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

Copasi javascript support #2015

Merged
merged 36 commits into from
Oct 18, 2024
Merged

Copasi javascript support #2015

merged 36 commits into from
Oct 18, 2024

Conversation

whomingbird
Copy link
Contributor

…JavaScript library. This new feature in SEEK enables interpreting COPASI-supported models directly in the client using this library.
…application) when multiple items are associated with a model.
…e current user, use SpecialAuthCode to enable loading the model into CopasiUI (desktop application)
…lation page. These parameters can now be included when running model simulations, providing enhanced flexibility in configuration.
…ls that are not publicly accessible but available for download by the current user, requiring a special authentication code.
lib/seek/copasi/simulator.rb Fixed Show fixed Hide fixed
lib/seek/copasi/simulator.rb Fixed Show fixed Hide fixed
app/assets/javascripts/copasi/copasi.js Outdated Show resolved Hide resolved
app/assets/javascripts/copasi/copasijs.js Outdated Show resolved Hide resolved
app/assets/javascripts/plotly-2.27.0.min.js Outdated Show resolved Hide resolved
app/controllers/models_controller.rb Outdated Show resolved Hide resolved
app/helpers/assets_helper.rb Outdated Show resolved Hide resolved
@@ -40,6 +40,6 @@ def jws_online_logo
end

def show_copasi_button?
Seek::Config.copasi_enabled && (@display_model.policy.access_type == Policy::ALL_USERS) && @display_model.is_copasi_supported?
Seek::Config.copasi_enabled && @display_model.is_copasi_supported? && @display_model.is_downloadable? && @display_model.can_download?(current_user)
Copy link
Member

Choose a reason for hiding this comment

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

this duplicates and could reuse @display_model.can_run_copasi?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

can_run_copasi? is defined for the "model" model, can not be called by @display_model ( versioning model)

app/views/models/copasi_simulate.html.erb Outdated Show resolved Hide resolved
db/schema.rb Outdated Show resolved Hide resolved
lib/seek/permissions/translator.rb Outdated Show resolved Hide resolved
test/unit/model_type_handling_test.rb Outdated Show resolved Hide resolved
@stuzart stuzart added this to the 1.16.0 milestone Oct 15, 2024
stuzart
stuzart previously approved these changes Oct 17, 2024
@stuzart stuzart dismissed their stale review October 17, 2024 09:08

tests are failing

@whomingbird whomingbird merged commit d9ba580 into main Oct 18, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants