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

Feature/return results option #18

Merged
merged 3 commits into from
Feb 8, 2021
Merged

Feature/return results option #18

merged 3 commits into from
Feb 8, 2021

Conversation

apizzuto
Copy link
Contributor

@apizzuto apizzuto commented Feb 5, 2021

This does not change any default behavior, but adds two features:

  • The option to return a dictionary to the user with the results of the simulation (includes info that would go into the header written to the file, as well as the simulation results)
  • The option to suppress printed headers, when using verbose=False. This is helpful when calling FIRESONG simulations many times and you don't want to have too much printed output

@itaboada itaboada self-requested a review February 8, 2021 21:22
@itaboada itaboada merged commit fe907f1 into base Feb 8, 2021
@apizzuto apizzuto mentioned this pull request Feb 10, 2021
Closed
@apizzuto apizzuto deleted the feature/return-results-option branch March 18, 2021 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itaboada itaboada Awaiting requested review from itaboada

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@apizzuto @itaboada