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

Make the number of analysis nodes a parameter #173

Closed
ranocha opened this issue Sep 8, 2020 · 1 comment · Fixed by #187 or #200
Closed

Make the number of analysis nodes a parameter #173

ranocha opened this issue Sep 8, 2020 · 1 comment · Fixed by #187 or #200
Assignees
@ranocha
Labels
good first issue Good for newcomers

Comments

@ranocha
Copy link
Member

ranocha commented Sep 8, 2020

Right now, the polynomial degree for analysis is hardcoded to be twice as large as the one for the solution. We should make that one a parameter, at least when we move toward Taal: Trixi as a library.
@ranocha ranocha added the feature label Sep 8, 2020
@gregorgassner
Copy link
Contributor

Agreed. That is just a leftover when we first implemented the analysis...

@sloede sloede added the good first issue Good for newcomers label Sep 10, 2020
@ranocha ranocha mentioned this issue Sep 22, 2020
Merged
36 tasks
@ranocha ranocha self-assigned this Oct 1, 2020
@ranocha ranocha mentioned this issue Oct 6, 2020
Merged
45 tasks
@ranocha ranocha removed the feature label Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ranocha ranocha

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

WIP: Taal ranocha/Trixi.jl
Merge Taal into master trixi-framework/Trixi.jl
3 participants
@sloede @ranocha @gregorgassner