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 'Running processes' section for aiida 2.0 #423

Merged
merged 1 commit into from
Sep 20, 2022
Merged

Update 'Running processes' section for aiida 2.0 #423

merged 1 commit into from
Sep 20, 2022

Conversation

ramirezfranciscof
Copy link
Member

No description provided.

Copy link
Member

@chrisjsewell chrisjsewell left a comment

Choose a reason for hiding this comment

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

Thanks, I see though wget [https://aiida-tutorials.readthedocs.io/en/tutorial-2021-intro](https://aiida-tutorials.readthedocs.io/en/tutorial-2021-intro/_downloads/ea4486c5b943d172e522da9bc9dc56ae/Si.pbe-n-rrkjus_psl.1.0.0.UPF), which should be changed to the correct branch

@ramirezfranciscof
Copy link
Member Author

ramirezfranciscof commented Aug 29, 2022

Ah, I understood we were doing all the 2.0 updates first and then any actual content, but I can do that now if you prefer.

The only issue is that there is no tutorial-2022-intro deployed right now (see here) so that link is just going to be broken...

@ramirezfranciscof
Copy link
Member Author

Thanks, I see though wget [https://aiida-tutorials.readthedocs.io/en/tutorial-2021-intro](https://aiida-tutorials.readthedocs.io/en/tutorial-2021-intro/_downloads/ea4486c5b943d172e522da9bc9dc56ae/Si.pbe-n-rrkjus_psl.1.0.0.UPF), which should be changed to the correct branch

On second thought @chrisjsewell I'm not exactly sure what to do here, because from what I could understand that link was not only added after the website was deployed, but it seems to have been generated at deploy time too (the _downloads/<HASH>), so I can't really anticipate what the final link will look like right?

@chrisjsewell
Copy link
Member

He i think it would be better to just use the sphinx download role

@ramirezfranciscof
Copy link
Member Author

You mean this? Right, I think this is what we do for the downloadable links embedded in the text, but I don't see that having the option to explicitly write the weblink address, does it? (I remember searching during the last tutorial but couldn't find anything)

@mbercx
Copy link
Member

mbercx commented Sep 20, 2022

Merging this as is, have opened issue regarding the wget code snippets:

#430

@mbercx mbercx merged commit 155a093 into aiidateam:tutorial-2022-intro Sep 20, 2022
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.

None yet

3 participants