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

Paper: Gagnon keke tahiri 2024 #907

Closed

Conversation

JustGag
Copy link
Contributor

@JustGag JustGag commented May 25, 2024

If you are creating this PR in order to submit a draft of your paper, please name your PR with Paper: <title>. An editor will then add a paper label and GitHub Actions will be run to check and build your paper.

See the project readme for more information.

Editor:

Reviewers:

@mepa mepa changed the title Gagnon keke tahiri 2024 Paper: Gagnon keke tahiri 2024 May 26, 2024
@mepa mepa added the paper This indicates that the PR in question is a paper label May 26, 2024
version: 1
project:
# Update this to match `scipy-2024-<folder>` the folder should be `<firstname_surname>`
id: scipy-2024-00_tex_template
Copy link
Collaborator

Choose a reason for hiding this comment

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

@JustGag if you update your id as described in the comment this should get the submission check action running as expected

Copy link
Contributor

Choose a reason for hiding this comment

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

Please update this to an ID similar to, which should match the folder name!

Suggested change
id: scipy-2024-00_tex_template
id: scipy-2024-gagnon_keke_tahiri

@@ -0,0 +1,21 @@
# See docs at: https://mystmd.org/guide/frontmatter
Copy link
Collaborator

Choose a reason for hiding this comment

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

@JustGag seems like this file was added in error, best to remove it asap to avoid confusion on your local machine. It should definitely be removed prior to merging.

Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove this file!

Copy link
Contributor

@rowanc1 rowanc1 left a comment

Choose a reason for hiding this comment

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

Hi @JustGag, could you please make a few changes to your PR for the preview to run?

  • delete top level myst.yml
  • remove changes to presentations folder
  • Rename folder to remove _2024 (all of the submissions in the folder will be for 2024!)
  • Update myst.yml ID to match folder name (follow instructions in the comment)

Let us know if you need any help. Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please remove the changes to this folder and restore the original PDF?

version: 1
project:
# Update this to match `scipy-2024-<folder>` the folder should be `<firstname_surname>`
id: scipy-2024-00_tex_template
Copy link
Contributor

Choose a reason for hiding this comment

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

Please update this to an ID similar to, which should match the folder name!

Suggested change
id: scipy-2024-00_tex_template
id: scipy-2024-gagnon_keke_tahiri

@@ -0,0 +1,21 @@
# See docs at: https://mystmd.org/guide/frontmatter
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove this file!

@TahiriNadia TahiriNadia deleted the Gagnon_Keke_Tahiri_2024 branch May 28, 2024 21:23
@hongsupshin
Copy link
Contributor

@JustGag @TahiriNadia Hi there, it seems like your team has created another PR #910 which seems to be a duplicate of this paper. If you're going to use the new PR, could you close this with your comment that confirms the decision, and modify the title of the new PR please?

@JustGag JustGag closed this May 30, 2024
@JustGag
Copy link
Contributor Author

JustGag commented May 30, 2024

Error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
paper This indicates that the PR in question is a paper
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants