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.

Sign up for GitHub

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

Jump to bottom

ENH: Tom august updates to cass_coopmans_I #402

Merged
merged 25 commits into from
Aug 18, 2024
Merged

ENH: Tom august updates to cass_coopmans_I #402

merged 25 commits into from
Aug 18, 2024

Conversation

mmcky
Copy link
Contributor

@mmcky mmcky commented Aug 7, 2024

No description provided.

@mmcky mmcky changed the title ENH: Tom august updates to lectures ENH: Tom august updates to case_coopmans_I Aug 7, 2024
@mmcky
Copy link
Contributor Author

mmcky commented Aug 7, 2024 •
edited
Loading

@shizejin it looks like there is an execution issue on this branch (not producing the plot)

T = paths[0].size - 1

is paths defined?

@shizejin
Copy link
Member

shizejin commented Aug 7, 2024

Thanks @mmcky. I tried to replicate the execution error on my local machine, but it builds well. Could you see more details about the execution issue from the server to better understand what's going wrong?

The paths should be have been created by the line paths = shooting(pp, 0.2, 0.3, T=10) in the cell above, and Tom and I didn't touch anything about the shooting function in this PR. Maybe there is something wrong when calling the shooting function, but I am sure what's causing it.

@mmcky mmcky changed the title ENH: Tom august updates to case_coopmans_I ENH: Tom august updates to cass_coopmans_I Aug 16, 2024
mmcky added 19 commits August 17, 2024 14:10
@mmcky
minor updates and adjustments
a3e70f7
@mmcky
update docker container
cf6d5e7
@mmcky
switch to using our github based container registery
d64375e
@mmcky
Merge branch 'main' into tom_august
fe62655
@mmcky
re-enablele build cache from 404
5e44020
@mmcky
move code-block above numref
32462a3
@mmcky
try having stable manifold references after the figure is generated.
1956930
@mmcky
try having stable manifold references after the figure is generated.
e33272c
@mmcky
Revert "try having stable manifold references after the figure is gen…
dce0733 
…erated."

This reverts commit e33272c.
@mmcky
Revert "try having stable manifold references after the figure is gen…
f5311a8 
…erated."

This reverts commit 1956930.
@mmcky
Revert "move code-block above numref"
ad50fe0 
This reverts commit 32462a3.
@mmcky
convert numref to ref for code generated image
49d8437
@mmcky
Revert "convert numref to ref for code generated image"
be27132 
This reverts commit 49d8437.
@mmcky
convert stable_manifold figure to figure directive for now
5d23832
@mmcky
remove duplicate label
47cb156
@mmcky
enable numfig=true
3426b1e
@mmcky
restore to code generated figure and reset to numref as should work
3a17e21
@mmcky
enable a full build
664aff5
@mmcky
alternative link syntax
49ceec3
@github-actions GitHub Actions
Copy link

🚀 Deployed on https://66c1ab85f1bcf7448a0d5f2f--nostalgic-wright-5fa355.netlify.app

@mmcky
Copy link
Contributor Author

mmcky commented Aug 18, 2024

it appears there was a strange interaction with the build cache so a full build succeeds. Merging this now.

@mmcky mmcky merged commit 94e1f42 into main Aug 18, 2024
4 of 5 checks passed
@mmcky mmcky deleted the tom_august branch August 18, 2024 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mmcky @shizejin @thomassargent30