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

2025-04-28-flow-with-what-you-know #38

Open
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

drscotthawley
Copy link

OpenReview Submission Thread

Checklist before opening a PR

  • I am opening a pull request against the main branch of the 2025 repo.
  • The title of my PR is exactly the name of my markdown file
    • i.e. _posts/2025-04-28-[SUBMISSION NAME].md would require a PR name 2025-04-28-[SUBMISSION NAME]
  • I have anonymized my post: my author's list is Anonymous, and there is no potential
    content which can reveal my/my collaborators identities.
  • My post matches the formatting requirements, including (but not limited to):
    • I have ONLY MODIFIED files in the following locations (failure to do so will result in
      your PR automatically being closed!):
      • a Markdown (or HTML) file in _posts/ with the format _posts/2025-04-28-[SUBMISSION NAME].md (or .html)
      • static image assets added to assets/img/2025-04-28-[SUBMISSION NAME]/
        *
      • interactive HTML figures added to assets/html/2025-04-28-[SUBMISSION NAME]/
      • citations in a bibtex file in assets/bibliography/2025-04-28-[SUBMISSION NAME].bib
    • I have a short 2-3 sentence abstract in the description field of my front-matter
    • I have a table of contents, formatted using the toc field of my front-matter
    • My bibliography is correctly formatted, using a .bibtex file as per the sample post

Any other comments

Please see question re. .mp4 placement asked in Issue #36

Copy link
Contributor

⚠️ We have detected a problem with your submission! ⚠️

You can only add/change/remove files related to your post, i.e. files that match one of these patterns: <_posts/SLUG.md, assets/img/SLUG/..., assets/html/SLUG/..., assets/bibliography/SLUG.bib>. But we found that you changed the following: <_posts/2025-04-28-Flow-With-What-You-Know.md & assets/bibliography/2025-04-28-Flow-With-What-You-Know.bib & assets/img/2025-04-28-Flow-With-What-You-Know/big_gaussian_and_spiral.png & assets/img/2025-04-28-Flow-With-What-You-Know/check_viz_b4_reflow.png & assets/img/2025-04-28-Flow-With-What-You-Know/cross_uncross_plot.png & assets/img/2025-04-28-Flow-With-What-You-Know/crossing_lines.png & assets/img/2025-04-28-Flow-With-What-You-Know/fm_vs_rf_streamvecs.mp4 & assets/img/2025-04-28-Flow-With-What-You-Know/gaussian_to_spiral_crossing_lines.png & assets/img/2025-04-28-Flow-With-What-You-Know/integrate_parabola_fwd_euler.png & assets/img/2025-04-28-Flow-With-What-You-Know/integrate_parabola_rk4.png & assets/img/2025-04-28-Flow-With-What-You-Know/last_epoch_reflow.png & assets/img/2025-04-28-Flow-With-What-You-Know/last_epoch_viz.png & assets/img/2025-04-28-Flow-With-What-You-Know/particles_fm.mp4 & assets/img/2025-04-28-Flow-With-What-You-Know/particles_fm_vs_rf.mp4 & assets/img/2025-04-28-Flow-With-What-You-Know/reflow_diagram.png & assets/img/2025-04-28-Flow-With-What-You-Know/reflowed_inp_gen_targ_traj.png & assets/img/2025-04-28-Flow-With-What-You-Know/transition_frames_example.png & assets/img/2025-04-28-Flow-With-What-You-Know/viz_test.png>. Also, make sure your PR's title (2025-04-28-flow-with-what-you-know) matches your post's slug!

Please make the aforementioned changes and re-submit :)

@busycalibrating
Copy link
Member

Hi, thanks for your submission! I believe we fixed our pipeline - could you please sync your fork with our changes and I will re-run the pipeline?

Thanks!

@drscotthawley
Copy link
Author

drscotthawley commented Nov 16, 2024

@busycalibrating Thanks for your reply. Couple questions...

  1. When I go to sync my fork, it says there's nothing to sync. Screenshot:
Screenshot 2024-11-15 at 6 22 53 PM

What to do in that case?

  1. Also, the above error message seems to be misreading the case of my PR, something somewhere is somehow converting it to lowercase, even though it's uppercase in all files and directory names:

Also, make sure your PR's title (2025-04-28-flow-with-what-you-know) matches your post's slug!

^^ but that's not the title of my PR. My PR is uppercase.

Is that to be expected?

@busycalibrating
Copy link
Member

Ah your workflow is using the correct github actions file, so you don't need to sync your branch.

Re 2: can you convert everything to lowercase (including this PR)? I think it's required...

@drscotthawley drscotthawley changed the title 2025-04-28-Flow-With-What-You-Know 2025-04-28-flow-with-what-you-know Nov 16, 2024
@drscotthawley
Copy link
Author

drscotthawley commented Nov 16, 2024

@busycalibrating Ok, switched everything to lowercase!

While I have your attention: re. Issue #36 : is my use of assets/img/ for .mp4 files ok? (maybe we'll just find out)

Thanks for your help.

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

@busycalibrating
Copy link
Member

Cool so looks like the pipeline works, but there seems to be some issues with your images rendering - I'm trying to figure out if it's something on our end or if it's something with how you named the files, looking into this now

Re: video, I suppose I'll add a new folder for those assets, for the moment you can leave it in your images folder

@EIFY
Copy link

EIFY commented Nov 16, 2024

@busycalibrating FYI, reference handling also seems broken...
Screenshot 2024-11-15 at 4 54 23 PM

@busycalibrating
Copy link
Member

Sorry this is going to be a bit annoying, but I think you'll need to change every reference to your original capitalized post to lower-case. This includes folders (e.g. assets/img/...), and references to filepaths. Eg this is the link in your post:

image

However if you manually look for the correct lower-case path, it renders:

image

@drscotthawley
Copy link
Author

Ok... global search and replace to effect lowercase.

btw I checked and the instructions just say "[SUBMISSION NAME]" (all caps!) and no mention of needing lowercase. The fact that the example file is lowercase wasn't sufficient guidance. I used to run a Jekyll site and used uppercase.
Anyway, we'll try this again! ;-)

@busycalibrating
Copy link
Member

Thanks for the feedback! Yeah you're totally right that was an oversight on our end - it was mostly to do with some hackery we did in our gh actions to do the auto build/deployment. I'll add some extra notes to the docs asap

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

@drscotthawley
Copy link
Author

ok i see one problem, with the wind map image. will fix it.....

@busycalibrating
Copy link
Member

Sweet I think the approvals are now correct now so I don't have to manually approve every update 😃

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

@busycalibrating
Copy link
Member

Great I think everything looks good! Let me know if you have any other issues :)

@drscotthawley
Copy link
Author

Ok, thanks David!

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

Copy link
Contributor

👋 Thanks for your submission! We have successfully built your website and we will push it shortly to the URL https://d2jud02ci9yv69.cloudfront.net/2025-04-28-flow-with-what-you-know-38/blog/flow-with-what-you-know/ !

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.

3 participants