-
Notifications
You must be signed in to change notification settings - Fork 32
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
BasicSSASolver not structuring output data correctly #50
Comments
And on a different machine, the values aren't delineated between .25 and .26, but using 1e3 and 1e-3 seem to break it. |
I have been looking into this issue today and trying to replicate it. I
have tried multiple values for k3 (1e-3, 1e-2, .1, .2, .25, .26, .3, .4,
.5) And have not been able to reproduce the error. I'm attaching the
notebook I used for testing.
…On Wed, Jan 30, 2019 at 9:06 AM Brian Drawert ***@***.***> wrote:
Assigned #50 <#50> to
@seanebum <https://github.com/seanebum>.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#50 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AVyQXiBI_68y3BgZXeGxLHeYIRpi1bfaks5vIacCgaJpZM4aVM9F>
.
|
What is interesting is that there is no code in python.basic_ssa_solver.py
to allow it to return information in any other structure. It only contains
the code for show_labels=True, and always returns
results[trajectory(list)][species_key(dict)][save_time(list)]
…On Thu, Jan 31, 2019 at 4:10 PM Sean Matthew ***@***.***> wrote:
I have been looking into this issue today and trying to replicate it. I
have tried multiple values for k3 (1e-3, 1e-2, .1, .2, .25, .26, .3, .4,
.5) And have not been able to reproduce the error. I'm attaching the
notebook I used for testing.
On Wed, Jan 30, 2019 at 9:06 AM Brian Drawert ***@***.***>
wrote:
> Assigned #50 <#50> to
> @seanebum <https://github.com/seanebum>.
>
> —
> You are receiving this because you were assigned.
> Reply to this email directly, view it on GitHub
> <#50 (comment)>, or mute
> the thread
> <https://github.com/notifications/unsubscribe-auth/AVyQXiBI_68y3BgZXeGxLHeYIRpi1bfaks5vIacCgaJpZM4aVM9F>
> .
>
|
This is now: Fix show_labels in BasicSSASolver |
Made vestigial by #57 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Model:
When running this model through the BasicSSASolver, the results array is not always structured in a manner consistent with the other solvers while at other times it is.
For instance:
When using the value k3 <= 0.25, I can print/plot results[0]['time']
When using the value k3 >= 0.26, I must use results['time'] instead
The text was updated successfully, but these errors were encountered: