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

Code suggestions #4

Open
falkamelung opened this issue Jan 24, 2024 · 0 comments
Open

Code suggestions #4

falkamelung opened this issue Jan 24, 2024 · 0 comments

Comments

@falkamelung
Copy link
Member

falkamelung commented Jan 24, 2024

Some comments on function call:

rain_plotter('bar', volcanos, rainfall, quantiles, rolling_number, scale, eruptions, elninos, lat_range, lon_range, by_season)
  • use plot_type='bar'
  • instead of volcanoes maybe sites_dict
  • instead of scale use log_flag=True
  • rename eruptions tovoclanic_events
  • are lat_range, lon_range needed?
  • by_season can be converted to a flag? The default is to use the full period and seasonal_flag=False.

helper_functions.py: Maybe separate in multiple scripts: plot_functions.py, el_nino_functions.py, helper_functions.py

  • plotting: try to do all the matplotlib settings in a separate function (configure_plot_settings.py) as much as possible.

  • we also want to create ax and fig in a separate function. If you have an idea on how to do this please do so. Else we will do this later.

  • We eventually want to separate notebooks from python scripts, i.e. the python scripts will end up in the precip repo. But I will ask Giacomo to do this once he puts all this together to a proper package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant