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

Update single_pendulums.jmd #890

Closed
wants to merge 6 commits into from
Closed

Conversation

Spinachboul
Copy link
Contributor

@Spinachboul Spinachboul commented Apr 2, 2024

Fixing the below issue
#785

  • The outputs are now not broken
  • No significant changes in the code, just some minor fixes.

The PR is ready to merge

@ChrisRackauckas

@ChrisRackauckas
Copy link
Member

That doesn't do anything about the loop?

@Spinachboul
Copy link
Contributor Author

No, the loops were running fine

@ChrisRackauckas
Copy link
Member

The plots are empty.

@Spinachboul
Copy link
Contributor Author

The individual plots are working fine now

@Spinachboul
Copy link
Contributor Author

Are the plots not working fine, or anything needs to be changed/fixed??
@ChrisRackauckas

@ChrisRackauckas
Copy link
Member

No plots are built, see the artifacts.

@Spinachboul
Copy link
Contributor Author

Like the plots are not visible??

@ChrisRackauckas
Copy link
Member

The artifacts showed the sol just errored.

@Spinachboul
Copy link
Contributor Author

Oh ok, I'll fix it soon!

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