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

phase out complex arrays (when using the nonoscillatory option) #381

Open
slayoo opened this issue Apr 14, 2023 · 5 comments
Open

phase out complex arrays (when using the nonoscillatory option) #381

slayoo opened this issue Apr 14, 2023 · 5 comments
Labels
no-activity

Comments

@slayoo
Copy link
Member

slayoo commented Apr 14, 2023

reasons:

  • principle of least surprise (complex type suggestes complex number, but we just use it for pairs of numbers)
  • does not follow Options.dtype
  • makes introduction of buffers tricky
  • adds fishy boilerplate code to boundary_condition classes
@github-actions
Copy link

Stale issue message

@github-actions
Copy link

Stale issue message

@github-actions
Copy link

Stale issue message

@slayoo slayoo reopened this Oct 29, 2023
@github-actions GitHub Actions
Copy link

Stale issue message

@github-actions github-actions bot closed this as completed Jan 5, 2024
@slayoo slayoo reopened this Mar 8, 2024
@github-actions GitHub Actions
Copy link

Stale issue message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
no-activity
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@slayoo