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

Ruben new benchmarks #228

Closed
wants to merge 11 commits into from
Closed

Ruben new benchmarks #228

wants to merge 11 commits into from

Conversation

rbnmj
Copy link
Collaborator

@rbnmj rbnmj commented Apr 24, 2023

New benchmarks: FixedSalinity_SimpleKiwi, OGSLargeScale3D_SimpleKiwi, & NetworkOGSLargeScale3D.

Reference files need revision.

SimpleKiwi: Output for growth, ag_resources, bg_resources, & salinity is not available (NaN) in SimpleKiwi. Not sure if it's better to remove these entries completely for SimpleKiwi or if this is an error.

test.pty Outdated Show resolved Hide resolved
@mcwimm
Copy link
Collaborator

mcwimm commented Apr 24, 2023

SimpleKiwi: Output for growth, ag_resources, bg_resources, & salinity is not available (NaN) in SimpleKiwi. Not sure if it's better to remove these entries completely for SimpleKiwi or if this is an error.

Good observation. I just saw in the [documentation)(https://pymanga.github.io/pyMANGA/project_dox__MangaProject__tree_dynamics__tree_growth_and_death__SimpleKiwi__SimpleKiwi.html) that there's no output available. That needs to be fixed (Issue #225 ).

Copy link
Collaborator

@mcwimm mcwimm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good for the start :)

I'm not sure if we shoud first restructure the ModuleBenchmarks folder before you proceed. Issue #222 contains suggestions for the new structure. @jvollhueter what do you think?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please apply the same changes regarding outputs and naming here

@mcwimm mcwimm force-pushed the ruben_benchmarks branch from 64aa766 to 0f384bc Compare May 16, 2023 11:15
@mcwimm
Copy link
Collaborator

mcwimm commented Jun 6, 2023

@rbnmj can we close this PR as this is now done in #245 and #237 ?

@rbnmj rbnmj closed this Jun 6, 2023
@rbnmj rbnmj deleted the ruben_benchmarks branch June 7, 2023 17:59
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

Successfully merging this pull request may close these issues.

2 participants