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

Comment out decapode UWD preview (for now) #30

Closed
brandomr opened this issue Nov 28, 2023 · 1 comment · Fixed by #25
Closed

Comment out decapode UWD preview (for now) #30

brandomr opened this issue Nov 28, 2023 · 1 comment · Fixed by #25
Assignees

Comments

@brandomr
Copy link
Contributor

Since there are issues with string escapes in Decapodes UWD previews cannot be consistently generated. This is a known bug in Decapodes that we expect TA2 to address; until then we should comment out this line.

As a workaround Terarium could generate its own UWD via the JSON from the preview. FYI @mwdchang @YohannParis

@fivegrant let's also remove that extraneous decapodes context file

Note: @pascaleproulx once this is completed Beaker will support Decapode composition: load 2 decapode AMRs, compose them together, and store the result. This satisfies a very minimal thin thread and resolves #25

@brandomr brandomr moved this to In Progress in Integration Nov 28, 2023
@brandomr
Copy link
Contributor Author

@pascaleproulx writing a decapode in decapod syntax and saving it as an AMR within beaker is already supported. The syntax would look something like:

glens_law = @decapode begin
  Γ::Form1
  (A,ρ,g,n)::Constant

  Γ == (2/(n+2))*A*(ρ*g)^n
end

In other words, we can support the Grigoriev Ice Cap example minus the UWD visualization, though I suspect it would be better to have a web-interactive visualization of this from the JSON anyway.

@fivegrant fivegrant linked a pull request Nov 28, 2023 that will close this issue
@github-project-automation github-project-automation bot moved this from In Progress to Done in Integration Nov 28, 2023
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 a pull request may close this issue.

2 participants