-
Notifications
You must be signed in to change notification settings - Fork 578
anr-aapg-fanmade:0.1.0 #1991
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
anr-aapg-fanmade:0.1.0 #1991
Conversation
b6ce66e
to
4b9d46f
Compare
I updated the package to take the warnings of the CI into account. Two warnings remain about named argument to the package configuration function that are acronyms in uppercase (AAPG and CES). The automated warnings worry that this may be an instance of CamelCase and recommend kebab-case instead. But I think that in this case keeping the acronyms capitalized makes things significantly more readable for package users, so I propose to keep the argument names that way. |
For the CI warnings, this is definitely a false positive, don't worry about it. For the template part, I think it could make sense to have a single template with both files (and the default Regarding the license of these files, I will assume that you wanted to write "can't" here:
In that case, I don't think it will change anything to keep them out of the template, as packages distributed to Typst Universe must be under an open-source license. The part you are quoting is for the opposite situation: when a template is under an open-source license (as it should be) but the users may want to share their derivative works under an incompatible license. If I'm mistaken and you actually wanted to write "can", I don't think there is any issue here, even if you were to put these two files in a template. |
You are correct, I meant "cannot", I will edit to fix this. The The style code itself in Each file comes with an explicit mention in the header of its licensing regime, in particular first-proposal.typ and second-proposal.typ have an explicit header mentioning that their content is not open source. My understanding is that there are other packages that include assets that are not themselves under an open-source license, typically the logo of the institution for institution-specific templates (for example this logo, which is probably covered by the following rules restricting its usage to schorlarly purposes). I view including
This is a good suggestion, thanks. Given our discussion on the copyright of these files, I am not sure that this would be acceptable for inclusion in Typst packages. (My very own interpretation of your packaging rules is that it is clearly okay to include the files as differently-licensed assets of the package, but only probably okay to expose them as templates.) If you converge on a decision on this aspect, and the decision is that offering these files as templates would be acceptable, then I will do as you suggest. |
(Note: if the CI system has a github repository somewhere, and you think it would be reasonably feasible to tweak the rules to not-warn on all-caps argument names, which are probably acronyms and not CamelCase identifiers, then I could consider trying to contribute this.) |
I think it would be fine to ship these two files as a template. Regarding the CI system, the code is available here. Thank you for offering your help. |
4b9d46f
to
d4db206
Compare
Thanks for the guidance! I updated the PR to offer |
The CI fails with an error because it considers second-proposal.png (the thumbnail) as a "compiled document". The code in package-check fails on foo.png if foo.typ also exist. I will silence the error by renaming the file as second-proposal-thumbnail.png. |
d4db206
to
24f9389
Compare
The actual issue here is that files for the template should be in their own subdirectory. Only files in this directory will be copied when starting a project with |
Hello! Do you think you will soon be able to fix the package structure as explained in my previous message? No worries if not, but I will then close this PR as inactive (you will just have to open a new one once you fixed the issue). |
@elegaanz thanks for the ping. I updated the package following your recommendation, with a |
2b3c0c2
to
5069b6d
Compare
@CL4R3T You released meppp:0.2.1, so you probably want to have a look at this pull request. If you want this update to be merged, please leave a comment stating so. Without your permission, the pull request will not be merged. |
@SillyFreak You released linguify:0.4.2, so you probably want to have a look at this pull request. If you want this update to be merged, please leave a comment stating so. Without your permission, the pull request will not be merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're getting there, just one small thing about the thumbnails location, and it should be good to merge!
[template] | ||
path = "template" | ||
entrypoint = "second-proposal.typ" | ||
thumbnail = "second-proposal-thumbnail.png" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thumbnail files should be outside of the template directory: this directory only contains files that will be copied to the user's project. The thumbnail
key in the manifest is relative to the root of the package, not to the template directory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I also changed the thumbnails from .png to .webp, following the discussion in #2007 .
I believe that the bot messages come from the fact that I temporarily pushed a version of this PR sitting on top of some local experiments on mine regarding #2024, which removes duplicated files and replaces them by a symlink. Sorry for the trouble. |
Thank you. |
I am submitting
This package consistents in a style library for ANR AAPG grant proposals -- grant proposals of French National Research Agency (ANR), the main funding body for French research grants. It would of use to anyone interested in using Typst to write an ANR grant proposal, instead of the official .docx files provided or the unofficial LaTeX packages.
I have read and followed the submission guidelines and, in particular, I
typst.toml
file with all required keysREADME.md
with documentation for my packageLICENSE
file or linked one in myREADME.md
exclude
d PDFs or README images, if any, but not the LICENSENotes
ANR submissions are in two phases (first submit a pre-proposal, then if approved write a full proposal).
The ANR funding agency provides .docx templates that contain the skeleton of a submission, including recommended layout, section titles, and also a bunch of recommendations on how to fill each section. The expected workflow for those .docx files is that grant authors fill out each section, and gradually remove the recommendations as they go along.
The typst package proposes the same workflow with two
.typ
files:first-proposal.typ
andsecond-proposal.typ
, which contain in particular the relevant ANR recommendations (they differ between the two phases).It would be nice to expose first-proposal.typ and second-proposal.typ as typst Templates, instead of forcing users to find the README file, find the link, and copy them. This is not done for two reasons:
cancannot be released under an open-source license, because they contain the ANR recommendations verbatim. The Typst packaging requirements demand that a template file can be distribued under an open-source license by users "after being modified through normal use". One plausible interpretation of "normal use" could be that users will indeed manually remove all ANR recommendations from the output, and then the result could be published under the terms of their choice (... modulo the issue that the titles of each section are provided by the ANR as well).