Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Check input to compute_spectrum_fdtd for sanity #55

Open
3 tasks
Tracked by #71
gomezzz opened this issue Feb 24, 2022 · 0 comments
Open
3 tasks
Tracked by #71

Check input to compute_spectrum_fdtd for sanity #55

gomezzz opened this issue Feb 24, 2022 · 0 comments
Labels
waveregime_internship

Comments

@gomezzz
Copy link
Collaborator

gomezzz commented Feb 24, 2022 •
edited
Loading

Feature

Desired Behavior / Functionality

Things to check:

  • are there sources (should always be one? then check that)
  • are there detectors
  • assert that any other variables that need to be set are there

We need to do this as users will mess it up, always :)

What Needs to Be Done

  • Add assertions for above
  • Add any others you may find sensible @torbjornstoro

How Can It Be Tested

Run with bogus values to see if errors are thrown.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
waveregime_internship
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gomezzz @torbjornstoro