Closed ConnectedSystems closed 2 years ago
Closes #92
@ryanheneghan @Rosejoycrocker
Completely forgot to say: As part of this set of changes I had to update ReefConditionIndex()
to use pre-calculated metrics values.
Thanks Takuya, I'll take a look at this and provide feedback asap. Apologies for the silence - I've had something urgent come up that's due Thursday that is taking all my attention.
On Tue, Feb 15, 2022 at 5:16 PM Takuya Iwanaga @.***> wrote:
@ryanheneghan https://github.com/ryanheneghan @Rosejoycrocker https://github.com/Rosejoycrocker
Completely forgot to say: As part of this set of changes I had to update ReefConditionIndex() to use pre-calculated metrics values.
— Reply to this email directly, view it on GitHub https://github.com/open-AIMS/ADRIA_repo/pull/95#issuecomment-1039933037, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMUXJVG2UILRQKIIWE5DDFLU3H4S5ANCNFSM5ONMZBUA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>
Closing in favour of #103 which includes this change
Sorry, in favour of #101 not #103
Added a new
summarizeMetrics()
function that takes a collated set of metrics and extracts the summary statistics across the scenarios run.Includes
plotTrajectory()
function to quickly visualize results.In visualizing results, I am assuming the mean of standard deviation is acceptable - I have heard no objections so far!