-
Notifications
You must be signed in to change notification settings - Fork 4
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
Finalize names of sound files #191
Comments
Thoughts about what to change:
@Ashton-Morris, any qualms with the above plan? |
I have no issues. I have the changed the names of those files on my computer. Did you want to change them on your end also, or have me send some? @zepumph |
I will rename now. I used this script to change all the staccato filenames: const fs = require( 'fs' );
const path = '../ratio-and-proportion/sounds/staccato/';
let files = fs.readdirSync( path );
files = files.filter( file => file.startsWith( 'marimba' ) );
files.forEach( fileName => {
const newFileName = fileName.replace( 'marimba-variation-v2-', 'staccato-' );
fs.renameSync( `${path}${fileName}`, `${path}${newFileName}` );
} ) |
Everything is renamed, now I just need to add the staccato sound |
Grabbing the wav files from #9 (comment) |
I used this script to rename and move wav files into assets: const fs = require( 'fs' );
const outputPath = '../ratio-and-proportion/assets/staccato/';
const inputPath = './rap-sounds/'
let files = fs.readdirSync( inputPath );
files = files.filter( file => file.endsWith( '.wav' ) );
files.forEach( fileName => {
const newFileName = fileName.replace( 'marimba-variation-v2-', 'staccato-' );
fs.copyFileSync( `${inputPath}${fileName}`, `${outputPath}${newFileName}` );
} ) Everything seems done to me now. @Ashton-Morris will you please take a look at file names in the following dirs to make sure all looks good from your end. Feel free to close. https://github.com/phetsims/ratio-and-proportion/tree/master/assets Thanks! |
Looks good |
I found https://drive.google.com/drive/u/1/folders/1A2SV3y_Qd0xAFx_IVCaMEqg9Mbq9wduk with outdated file names. Does this folder stick around past the design phase of a sim? We already store all the underlying |
I have copies of all of the in-developement working files on my computer so I don't thing we need this, but usually I just leave them there out of habit. |
It's redundant to have the same resources check into git as well as in drive. It also seems problematic to me to have them with outdated filenames that no longer link to the same resources in github, and (hopefully) on your computer. I would expect that from this issue, all of the filenames on your computer (wav/reaper-projects/tracks-in-reaper/mp3/etc) would have the new name. Thus something titled "marimba-variation-v2" is actually confusing and negative to stick around. Maybe we can improve the collaboration around keeping the asserts dir as this central scrap page: https://github.com/phetsims/ratio-and-proportion/tree/master/assets. Right now it is all top level, but there is nothing saying we couldn't have a sub-dir for
It is a PhET standard in general to try to maximize the percentage of resources that are in shared spaces. It doesn't always work (like I know it won't for reaper files), but in general the more in the PhET cloud things are, the more maintainable and transparent things are. Would it work for you to have these local files centered around local git checkouts of RaP and other sims? I'm happy to keep discussing. |
I see what you are saying. I think this ties into the soundboard idea that JB and I will be diving into. I can bring up this in relation during those talks because I do think it's good to have a better tracked workflow overall. |
I ceated phetsims/tambo#127 to track the tail of this conversation. I think this specific issue is ready to be closed. |
From #127 checklist:
We still have our prototype-like names of sounds in the project, and not all of them have associated
.wav
files in theassets
folder.I want to do this issue with @Ashton-Morris so that whatever names he has for files are updated in correlation with the github files.
The text was updated successfully, but these errors were encountered: