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.

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

Improve results plotting logic, bounded to pyCSEP refactoring . #26

Open
Tracked by #40
pabloitu opened this issue Sep 10, 2024 · 0 comments
Open
Tracked by #40

Improve results plotting logic, bounded to pyCSEP refactoring . #26

pabloitu opened this issue Sep 10, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@pabloitu
Copy link
Collaborator

pabloitu commented Sep 10, 2024

There is current plot refactoring PR in pycsep (SCECcode/pycsep#263). Once this goes through, floatcsep needs to restructure the plotting logic to connect the Test function to a Plot function. Some ideas for improvement

  • Always require the user to insert a plot function to a test config. From the Plot function signature/name, derive the logic (e.g. all forecasts, one forecast, all time windows, etc.)
  • Define defaults for each test function. This implies functions are automatically passed, and only arguments are optional for the user. Any other plotting logic should be done through a plot hook.
@pabloitu pabloitu self-assigned this Sep 10, 2024
@pabloitu pabloitu mentioned this issue Oct 2, 2024
15 tasks
@pabloitu pabloitu added the enhancement New feature or request label Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant