- 
                Notifications
    
You must be signed in to change notification settings  - Fork 2
 
tutorial: Modify links to pattern tutorialName-Author1-Authorn_languageSuffix #9
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
Conversation
…ageSuffix` All the tutorials have their name changed to follow the pattern.
          ✅ Deploy Preview for training-slicer-org ready!
 To edit notification comments on pull requests, go to your Netlify project configuration.  | 
    
| 
           Thanks for working on this 🙏 So that we can document the guidelines for anyone working on contributing training materials, I suggest we spend more time describing the convention to have an ambiguous and systematic way of deriving the file name from the tutorial title. Looks like the convention seems to be further specified. I propose the following: - tutorialName-Author1-AuthorN_languageSuffix
+ [Slicer]<TutorialBaseName>[Tutorial]-<Author1>[-<Author2>[...]]_<language>[_<region>].(pdf|md|html)where: 
 References: 
 QuestionsCan we revisit the approach for generating the 
 | 
    
| 
           cc: @lassoan  | 
    
Related contributions: - Pull request reviews: - Slicer/training.slicer.org#8 (comment) - Slicer/training.slicer.org#9 (comment) - Pull requests: - SlicerLatinAmerica/SlicerTutorialMaker#67 - SlicerLatinAmerica/SlicerTutorialMaker#68
Related contributions: - Pull request reviews: - Slicer/training.slicer.org#8 (comment) - Slicer/training.slicer.org#9 (comment) - Pull requests: - SlicerLatinAmerica/SlicerTutorialMaker#67 - SlicerLatinAmerica/SlicerTutorialMaker#68
| 
           It would be hard to enforce a particular naming convention for files that are store outside this repository. For example, I would not want to go and ask Sonia to please remove your name from the tutorial filename. For any places where we want to have a string that is uniquely associated with a tutorial, we could use the tutorial ID. For example, we could use the tutorial ID for associating filenames of images that are stored in this repository. So, I think we can do without formalizing a   | 
    
| 
           Tutorials should ideally have all the source files stored on github and the PDFs generated from them are also stored in the same repository. In one of the commits, location of STC-VIS-102, STC-SEG-102, and STC-DEV-101 tutorials were changed, so I pushed a commit to restore the correct URLs. Any update to the tutorials are welcome, but any source file changes and new generated PDFs should be made in the tutorial's repository (and then we probably we don't need to change anything here). If there are scripts to generate the tutorial then those scripts should be stored in the tutorial's repository, too. With my commit, the suggested changes look good to me. @jcfr let us know if you agree. We need to merge these PR quickly because of reporting to CZI.  | 
    
All the tutorials have their name changed to follow the pattern.