Old 11-30-2015, 01:24 PM   #121
X-Raym
Human being with feelings
 
X-Raym's Avatar
 
Join Date: Apr 2013
Location: France
Posts: 6,108
Default

@cfillion
Hmmm maybe a dumb question, but is it possible to stay with my initial header format ?

I understand the benefit in term of performance of this extra top header section
Code:
 * @author X-Raym
 * @version 1.0
 * @changelog (2015-11-27)
 *   + Initial Release
 *
... but does it really worth it ? I mean is that really that faster to parse ? (theoretically yes because there is less lines, but how much milliseconds for 1000 scripts ?)

all Heda's scripts and mine have these header so it will be a real pain to rewrite all of them... so you have to understand that we are a bit cold at the idea of updating our whole databases...
And it breaks a bit the initial style hehe :P

----
I am for downloading the whole pack.
It is the current solution I propose. Way simpler for everyone.
X-Raym is offline   Reply With Quote
Old 11-30-2015, 01:46 PM   #122
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

Under "installing scripts" I mean two actions:
1) download/unzip whole repo (with functions, graphics etc), replace existed if need
2) add/remove scripts to action list by editing reaper-kb.ini file

So only second action should be customized while setup/later updating.

After some pm`s from different users:
I still think not anyone like header even if it most comfortable to get all possible info about script. I didn`t add it myself yet, because I feel it unuseful (mostly I can`t add any script related info like forum thread or release date), and only need for database creating (which also I don`t need since author and script name is enough for me as for regular user, anyway I can google any info about that, I guess Cockos forum always will be first link in results).
I know, to have all related info about script is good...but really who need it? User just wants script works however and by whoever it was written.

Quote:
Originally Posted by cfillion View Post
mpl, can you accept the authorization request for Travis-CI on ReaTeam I sent the other day
Didn`t saw anything, can you send again?
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate

Last edited by mpl; 11-30-2015 at 01:59 PM.
mpl is offline   Reply With Quote
Old 11-30-2015, 01:56 PM   #123
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default

One benefit of the different syntax is that it allows strict mistake detection:

Code:
 * Key Name: This line is discarded by the parser -> no error
 * @author X-Raym
 * @version 1.0
 * @changelo (2015-11-27)
 *   + Initial Release


I see three solutions:

1. I use your syntax instead (key: value is as easy to parse as @key value, it's just a bit more ambiguous with normal english)
I don't mind but I can't check for invalid keys if I do that unless every possible (and unused yet) keys are registered too.

2. Compatibility layer: supported keys are understood in your format too (but no error reporting for those)

3. Conversion tool
Key Name: Value --> @key_name value

Last edited by cfillion; 11-30-2015 at 03:25 PM.
cfillion is online now   Reply With Quote
Old 11-30-2015, 02:05 PM   #124
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

cfillion, X-Raym, please, let me know, again, which way we use header inside extension (How it will looks like and what the benefits of using it related to extension?).
Just thinking, I didn't add header and after filling whole repo I don't feel my scripts missed something, they just work and that is enough for me. So maybe forget temporary about metadata and concentrate on basic operations?
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate

Last edited by mpl; 11-30-2015 at 02:17 PM.
mpl is offline   Reply With Quote
Old 11-30-2015, 02:17 PM   #125
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default

Version is needed for releasing stable updates and Changelog (optional) to show what changed in a dialog similar to REAPER's(?). Since the extension simply installs everything in the background Author etc is not currently used.

So Version is really the only one required for the basic functionality.

Quote:
Originally Posted by mpl View Post
Didn`t saw anything, can you send again?
It should be in Profile > Organizations > Settings > Third-party access:

https://help.github.com/articles/app...-organization/

EDIT: About the Action List/reaper-kb.ini, the changes only take effect after REAPER is restarted.
For now I think the extension should focus on downloading and updating the files, and let the user load them in his Action List manually.

Last edited by cfillion; 11-30-2015 at 05:26 PM.
cfillion is online now   Reply With Quote
Old 11-30-2015, 02:38 PM   #126
X-Raym
Human being with feelings
 
X-Raym's Avatar
 
Join Date: Apr 2013
Location: France
Posts: 6,108
Default

@cfillion
Quote:
1. I use your syntax instead
Well, it is not bad if we don't have proper debug for keys as they are rarely buggy (it is often written from script to script with copy paste).

Focusing on the system that is already in place will be easier for us.... And you will be able to make tests on hundreds scripts pack without waiting :P
X-Raym is offline   Reply With Quote
Old 11-30-2015, 05:19 PM   #127
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default

Ok, I did the first solution and added support for your syntax + disabled the error reporting of unknown keys.

For example this script: https://github.com/X-Raym/REAPER-Rea...%20console.eel
...is understood like this:



I noticed some of your scripts have their Version tag below a blank line. These tags are not detected (I have to draw a line between the header and the code):

Code:
--[[
 * Author: ok
 * License: ok

 * Version: the parser stops before getting down here...
--]]
This would work instead:

Code:
--[[
 * Author: ok
 * License: ok
 *
 * Version: ok
--]]
EDIT:
Quote:
Originally Posted by X-Raym View Post
Well, it is not bad if we don't have proper debug for keys as they are rarely buggy (it is often written from script to script with copy paste).
https://github.com/X-Raym/REAPER-Rea...3&q=reposotory

Last edited by cfillion; 11-30-2015 at 10:13 PM.
cfillion is online now   Reply With Quote
Old 11-30-2015, 07:04 PM   #128
X-Raym
Human being with feelings
 
X-Raym's Avatar
 
Join Date: Apr 2013
Location: France
Posts: 6,108
Default

@cfillion
Nicely done !

Reposotory... Hmmm... ^^

Well, in half of my scripts "last version" is, Last Version & Last Version Dates not included in the scripts header, but only included on changelog.
I was tired to write infos twice --'

Would it be too much to ask to have version infos taken from first entry in changelog ?

Thanks again for your work on this !
X-Raym is offline   Reply With Quote
Old 12-02-2015, 03:21 AM   #129
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

Added to my repo header:

Quote:
--[[
* Script name (if file name will be rewrited by someone)
* Lua script for Cockos REAPER (let any human found this knows what is this code for)
* Author: Michael Pilyavskiy (mpl) (also if someone wants to rename or if we`ll decide to exclude scripter names from common script name to make Action List clean)
* Author URL: http://forum.cockos.com/member.php?u=70694 (as basic contact)
* Licence: GPL v3
* Version: 1.0 (I think it is better way to point this directly, and changelog could be described below the header if user really interested what was changes since last version)
]]
(empty line)
(code)
Since I use script for that (of course I didn`t it manually), I can edit common repo scripts this way, if this header is ok.
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate

Last edited by mpl; 12-02-2015 at 04:56 AM.
mpl is offline   Reply With Quote
Old 12-02-2015, 04:34 AM   #130
X-Raym
Human being with feelings
 
X-Raym's Avatar
 
Join Date: Apr 2013
Location: France
Posts: 6,108
Default

@mpl
I guess this could be the minimum fields !
But I'm surprised that you don't put
Quote:
* ReaScript Name:
* Description:
as prefixes.

Prifexes is what make parsing possible easier (a lot more easier than checking line number).
X-Raym is offline   Reply With Quote
Old 12-02-2015, 04:42 AM   #131
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

There are desciptions already in some non-selfexplainable scripts but as a comment or another way, I`ll add it to desciption prefix later.
Script Name - I firstly look at your first script in first folder:
Quote:
/**
* Display selected tracks and takes color in the console
* EEL Script for Reaper
* Displays tracks and takes color in RGB and HEX values in the console.

* Author: X-Raym
* Author URl: http://extremraym.com
* Reposotory: GitHub > X-Raym > EEL Scripts for Cockos REAPER
* Repository URl: https://github.com/X-Raym/REAPER-EEL-Scripts
* Source URl: https://github.com/X-Raym/REAPER-EEL-Scripts/Display selected tracks and takes color in the console.eel
* Licence: GPL v3
* Forum Thread: Script: Display selected tracks and takes color in the console
* Forum Thread URl: http://forum.cockos.com/showthread.p...57#post1480557
* Version: 1.0
* Version Date: 2015-15-02
* REAPER: 4.76
*/
So I though parser will automatically guess first line as reascript name (there is no prefix / no another prefix / "ReaScript name" prefix).
I don`t think I should add whole header mostly with empty values, but only minimum fields + what I think header for current script should comes with. I also don`t put supported version since I supposed user installed latest official releases of REAPER and SWS, so I willn`t get expected bugreport.

I also did put /* instead /**. Why did you use what?
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate

Last edited by mpl; 12-02-2015 at 04:55 AM.
mpl is offline   Reply With Quote
Old 12-02-2015, 04:54 AM   #132
X-Raym
Human being with feelings
 
X-Raym's Avatar
 
Join Date: Apr 2013
Location: France
Posts: 6,108
Default

The script script my repo, you should check the last one hehe :P

Okay for the minimum header fields,

But really what make parsing easy is that all pattern matching are the same,
if it has to be adjust once time for prefix, one time for line number... Not very easy and extensible.
And I know it because I already made a script header parser :P

Note that I also use ReaScript Name:
instead of just Names or just ReaScript
so anyone can find that it is a ReaScript (for REAPER)... and that it is its name (which can differ from its file name, action name, undo name etc...)




EDIT : Oh yeah, and I think Repo URL is important for cfillion
X-Raym is offline   Reply With Quote
Old 12-02-2015, 05:08 AM   #133
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

I should point somehow anyway what language used for this script.
Repository URL (why lower L?) - ok.

So
Quote:
--[[
* ReaScript Name: Bypass all FX except instruments on all tracks
* Lua Script for Cockos Reaper (I don`t think it should be parsed, who cares it is lua or eel script. So I added it if someone will download single script, delete its name with extension and willn`t know what extension add)
* Author: Michael Pilyavskiy (mpl)
* Author URL: http://forum.cockos.com/member.php?u=70694
* Repository URL: https://github.com/MichaelPilyavskiy/ReaScripts
* Licence: GPL v3
* Version: 1.0
]]
(empty line)
(code)
is ok (for now replacing such thing not so painful for me as its automated)?
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate
mpl is offline   Reply With Quote
Old 12-02-2015, 05:22 AM   #134
X-Raym
Human being with feelings
 
X-Raym's Avatar
 
Join Date: Apr 2013
Location: France
Posts: 6,108
Default

In fact it is not a L it is a I

URI = URL + URN



I agree I don't see the URN part of it, but it is that that is URI that is used on WordPress plugins and Theme Header:
File Header « WordPress Codex

WordPress › Header Requirements | Plugin Developer Handbook | WordPress Developer Resources

So I'll stay with it :P

----
Saying that I'm afraid I also put a L everywhere. Damn ^^
Fortunately we can do batch replacement in all file.
---

EDIT: your forgot Description: prefix :P
and just like me, replace URL by URI

Last edited by X-Raym; 12-02-2015 at 06:10 AM.
X-Raym is offline   Reply With Quote
Old 12-02-2015, 06:17 AM   #135
X-Raym
Human being with feelings
 
X-Raym's Avatar
 
Join Date: Apr 2013
Location: France
Posts: 6,108
Default

Best commit ever:

URI not URl · X-Raym/REAPER-ReaScripts@2a42303

^^
X-Raym is offline   Reply With Quote
Old 12-02-2015, 06:34 AM   #136
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

Just to know - how you did that (via git shell I guess)?
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate
mpl is offline   Reply With Quote
Old 12-02-2015, 07:00 AM   #137
X-Raym
Human being with feelings
 
X-Raym's Avatar
 
Join Date: Apr 2013
Location: France
Posts: 6,108
Default

@mpl
Search and Replace in All folder and subfolder files in Notepad++ :P (other code editor have feature)
then, a simple commit like the other, using GitHub APP.

Took me about 10 seconds to do the whole process.

That's the power of unique prefix !
X-Raym is offline   Reply With Quote
Old 12-02-2015, 01:05 PM   #138
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default

mpl, all 65 scripts on your repository are passing the metadata test: http://sprunge.us/TOfB
This mean they can all be indexed and installed using the manager without further modification.

On ReaTeam/ReaScripts I added a version tag to most of the script in the metadata branch. Test results after the change: http://sprunge.us/bQEH (20 failures).
The remaining scripts don't have any information at all...
(I used 0.2015.12.25 format on the scripts without existing version numbers. Versions are sorted numerically so the zero is important to not shadow a future v1 release)

I don't think I can enable travis on the repository myself as it requires admin rights. Can you go on this page: https://travis-ci.org/profile/ReaTeam and enable it on ReaScripts for me?

------

Quote:
Originally Posted by X-Raym View Post
Would it be too much to ask to have version infos taken from first entry in changelog ?
Well, the content of a multiline tag must be indented to be understood unambiguously:

Code:
--[[
 * Changelog:
 *   This indented block of text is the changelog content.
 *   There is no ambiguity with the next line:
 *
 * This line is not part of the changelog.
--]]
In some of your scripts (for example this one: https://github.com/X-Raym/REAPER-Rea...tems%20end.lua) the changelog is both below an empty line and with unindented parts. It is not practical to parse...
I think I could write a short script to fix this AND add the missing Version tag since you are using this same syntax almost everywhere. I think it's simpler to do this rather than adding tricky workarounds in the parser.

Quote:
Originally Posted by X-Raym View Post
EDIT : Oh yeah, and I think Repo URL is important for cfillion
I use git directly to get this information. It can be in the header too but for the indexer it's not necessary.

Quote:
Originally Posted by mpl View Post
So I though parser will automatically guess first line as reascript name (there is no prefix / no another prefix / "ReaScript name" prefix).
I'd prefer if everything is prefixed the same way. But right now the manager does not need these tags anyway (only version and changelog are used) so any style is ok.

Quote:
Originally Posted by mpl View Post
I don`t think I should add whole header mostly with empty values, but only minimum fields + what I think header for current script should comes with.
I think that's the right thing to do: tags with empty values risk causing validation errors in the future because they are considered as boolean flags (but as long as they are unused they are just silently discarded).

------

About user-configurable repositories (which is almost finished, by the way):

I think the easiest way to subscribe to a new repository would be to have a special file written by the repo maintainer downloaded then imported into the extension by the user (a bit like REAPER licenses are):

File Name.ReaPackRemote
Code:
Developer Name
https://github.com/dev_user/ReaScripts/raw/master/index.xml
Extension Menu > ReaPack > Import new repository...

The first line determines the name of the folder where the scripts will be downloaded. It would have to be a unique value for every repository.

Last edited by cfillion; 12-02-2015 at 01:30 PM.
cfillion is online now   Reply With Quote
Old 12-02-2015, 01:25 PM   #139
X-Raym
Human being with feelings
 
X-Raym's Avatar
 
Join Date: Apr 2013
Location: France
Posts: 6,108
Default

@cfillion
Ok, If you propose a way to add Version in header more easily in bulk editing, I can do it.
X-Raym is offline   Reply With Quote
Old 12-02-2015, 01:51 PM   #140
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

cfillion, please take a look. Did I added repos to Travis? (I'm from smartphone these days).
I'll add header to common repo later.
What about line
Quote:
* EEL script for Cockos Reaper
Parser will just ignore that line, right? Or I should add some prefix also?
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate
mpl is offline   Reply With Quote
Old 12-02-2015, 02:03 PM   #141
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default

It works!


Full Test Log (when you click on "Details" on the PR): https://travis-ci.org/ReaTeam/ReaScr...uilds/94500875
It says which scripts have yet to be fixed and what is wrong with them.

Please push your header changes on the metadata branch instead of master so it will be added to the pull request and automatically re-tested by travis.
https://github.com/ReaTeam/ReaScripts/pull/2

Quote:
Originally Posted by mpl View Post
Parser will just ignore that line, right? Or I should add some prefix also?
Yes it's simply ignored.

Last edited by cfillion; 12-02-2015 at 02:46 PM.
cfillion is online now   Reply With Quote
Old 12-02-2015, 02:09 PM   #142
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

Done.
Now header for my and for common repo should be parsable. I see you already started adding headers manually, but I added it automatically by script, so it looks same as my and X-Raym repos. I added header roughly by Lua script, which parsed scripter name and script name into related fields. So, it needs to edit some of them more or less, I just added strong header to make all scripts at least parseable.

I`m not competely understand what Travis is (I understood, that is for checking synthax or something but how related to extension?), so if these things only useful to build extension and you need them, I can give you admin rights (you`re already have write permissions, is`t it?). Note, I still new to github so I can flounder in git actions and damage something (I think I already did something non-expected by myself and merged metadata to master, is that so?). I gave you green light by adding to owners so do what you want/need
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate

Last edited by mpl; 12-03-2015 at 02:57 AM.
mpl is offline   Reply With Quote
Old 12-03-2015, 02:31 AM   #143
Sexan
Human being with feelings
 
Sexan's Avatar
 
Join Date: Jun 2009
Location: Croatia
Posts: 2,452
Default

sorry guys, but any answer to my previous post #119? or I can stop updating it with headers and remove them?
Sexan is offline   Reply With Quote
Old 12-03-2015, 02:50 AM   #144
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

Sexan, here my thoughts:
1) I can add this if Tritone not against it (I wrote him pm already)
2) What it counts, and why 1 minute instead, say, project state change + mouse context changes + transport play/rec time?
3) difference from "split at time selection"?
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate
mpl is offline   Reply With Quote
Old 12-03-2015, 05:09 AM   #145
Sexan
Human being with feelings
 
Sexan's Avatar
 
Join Date: Jun 2009
Location: Croatia
Posts: 2,452
Default

Oh, I'm not very good at explanations, timer counts time and afk is set to 1 minute which can be modified (don't know what actual time is good but for me 1 min after I'm gone is good),but if you have better explanation please use it.

Smart split is very different then "split at time selection" :
it is combination of it with "split at edit cursor", when using sws "split items at time selection (if exists) else edit cursor" you can't split at edit cursor within time selection .

This one allows editing at time selection and while time selection still exists it allows to split anywhere at edit cursor.
So its a 1 shortcut to do split at edit cursor and time selection without of limitations of not being able to split at edit cursor in the time selection.

my script


sws split at timeselection edit cursor

so the short summary , there is no action (1 shortcut) that allows splitting IN the time selection unless using more shortcuts.
btw SWS is required because I'm using "Make crossfade at left of edit cursor"

Last edited by Sexan; 12-03-2015 at 05:25 AM.
Sexan is offline   Reply With Quote
Old 12-03-2015, 09:22 AM   #146
Xenakios
Human being with feelings
 
Xenakios's Avatar
 
Join Date: Feb 2007
Location: Oulu, Finland
Posts: 7,992
Default

Interesting stuff in this thread! Keep up working on it!

I am not sure if I have anything useful and interesting enough to contribute to the script repo, most of my scripts are completely specific to my own workflows but I can take a look if I happen to have something worth sharing.
__________________
For info on SWS Reaper extension plugin (including Xenakios' previous extension/actions) :
http://www.sws-extension.org/
https://github.com/Jeff0S/sws
--
Xenakios blog (about HourGlass, Paul(X)Stretch and λ) :
http://xenakios.wordpress.com/
Xenakios is offline   Reply With Quote
Old 12-03-2015, 12:17 PM   #147
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default

Quote:
Originally Posted by mpl View Post
I`m not competely understand what Travis is (I understood, that is for checking synthax or something but how related to extension?), so if these things only useful to build extension and you need them, I can give you admin rights (you`re already have write permissions, is`t it?). Note, I still new to github so I can flounder in git actions and damage something (I think I already did something non-expected by myself and merged metadata to master, is that so?). I gave you green light by adding to owners so do what you want/need
Many thanks! Travis is for checking new commits and pull requests so we can know if the metadata are parsable and ok before contributions are merged.

This is part of the build system to generate the package database the extension is using.

You can run the build scripts on your own computer (the code is written in Ruby so it must be installed beforehand) with these commands:
Code:
cd Path/To/ReaScripts

bundle install # installs the metadata parser, reapack indexer and their dependencies
bundle update # updates the parser and indexer if I made change to them

bundle exec rake # checks the metadata of every script (this is what Travis runs)
bundle exec rake index # generates or updates index.xml for the extension
The last command (rake index) must be ran once in a while to release updates to the users, as "index.xml" contains the full list of scripts in a machine-readable format for the extension.

(I developed the tools on OS X, but it should work fine on Windows too)

In summary:

1. The scripts must have at least a Version tag and optionally a Changelog tag. For now everything else is ignored.
2. Travis ensures every new contribution on the repository matches the first rule.
3. The repository must be processed once in a while in order to generate a file named "index.xml" using command `bundle exec rake index`
4. The extension download and read that index.xml file to know what it must do on the user's computer.

EDIT: Oops, in the last commit you deleted the build system files (Rakefile, Gemfile, .gitignore, .travis.yml and index.xml). No trouble, thanks to git nothing is lost.

Last edited by cfillion; 12-03-2015 at 12:32 PM.
cfillion is online now   Reply With Quote
Old 12-03-2015, 12:35 PM   #148
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

I thought it is something auto generated from metadata branch since there aren`t headers in scripts inside this branch (or they written another way). As you see, I cnanged all headers in master, so be free to add to master what you need.
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate

Last edited by mpl; 12-03-2015 at 12:55 PM.
mpl is offline   Reply With Quote
Old 12-03-2015, 12:51 PM   #149
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default

All green! https://travis-ci.org/ReaTeam/ReaScr...uilds/94713319
Thanks to your commit the header of all 81 scripts is OK (the work in the metadata branch is finished so I deleted it).

In the meantime I'm working on making the extension remember what it previously installed so it can do efficient updates.
cfillion is online now   Reply With Quote
Old 12-03-2015, 12:56 PM   #150
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

cfillion, I didn`t got it: should every repo holder install all of this to make it compatible wit extension? If so, it look some complex to me (and for X-Raym, I guess) since I didn`t figured out what should I do step-by-step. Also, can we exclude these files from zip package, for example when user download it directly from github (I remember I saw "gitignore" or something)?

Sexan, just ran smart split and get sexan_SmartSplit.lua:52: bad argument #1 to 'GetMediaItemInfo_Value' (MediaItem expected), just selected 2 items and ran. Please fix possible bugs and make a good description since script name/first run not self-explainable. I also didn`t get what project timer do. If it should counts time when project is opened, I don`t understand why did you add 1 minute timer. And if it counts "action" time when user do something with this project (which is true) then your script not implemented correctly (it counts also time when project "sleep").
Sorry for writing this. When I add any script to repo I test it, so I can`t add it if it has bug at first run and if I don`t understand what it does actually while there is no any description so I can`t use it.
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate

Last edited by mpl; 12-03-2015 at 01:28 PM.
mpl is offline   Reply With Quote
Old 12-03-2015, 01:22 PM   #151
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default

Yes, unless some other way to generate index.xml is made. I hope it's not too complex to use...

I'll try making step-by-step instructions later. Are you using Windows?

Quote:
Originally Posted by mpl View Post
Also, can we exclude these files from zip package, for example when user download it directly from github (I remember I saw "gitignore" or something)?
It's possible using branches, but why? They are very small files and it's easier to use them on master.
Most of the code I wrote is hosted elsewhere on github so these files are only containing the minimum.
(https://github.com/cfillion/metaheader and https://github.com/cfillion/reapack-index)

Here is a blank repository with only the build files: https://github.com/cfillion/reapack-repository-tools

Last edited by cfillion; 12-03-2015 at 05:41 PM.
cfillion is online now   Reply With Quote
Old 12-03-2015, 01:30 PM   #152
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

Yes, instruction how-to-use / what-to-do would be nice.
I`m using Windows but no problem to do something in OSX (via VirtualBox).
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate
mpl is offline   Reply With Quote
Old 12-03-2015, 02:54 PM   #153
Michael Ellis
Human being with feelings
 
Michael Ellis's Avatar
 
Join Date: Nov 2015
Location: Weaverville NC
Posts: 59
Default

Hi all,
I've just published a Python ReaScript on GitHub for creating rehearsal tracks from audio files. See the discussion at

http://forum.cockos.com/showpost.php...3&postcount=15

for more details.

The project link on GitHub is https://github.com/Michael-F-Ellis/Practice-Track

I'll confess I've only skimmed this thread, but it sounds like good work is being done toward creating some sort of central repository for ReaScripts. If what I've done seems worth adding, please let me know how to package it to fit your format.

Cheers,
Mike
__________________
Write music faster with Tbon. https://github.com/Michael-F-Ellis/tbon
Mac mini, 2.3 Ghz core i7, 16GB Ram, Linnstrument #437, Technics P30
"What survives every change of system is melody." -- Igor Stravinsky
Michael Ellis is offline   Reply With Quote
Old 12-03-2015, 02:56 PM   #154
Sexan
Human being with feelings
 
Sexan's Avatar
 
Join Date: Jun 2009
Location: Croatia
Posts: 2,452
Default

never mind,I see this is way over my abilities to maintain,I will remove the links.I can't reproduce that bug with split item, I'm using it daily for a long time now an not seen any bug (especially that kind of error which is crucial for a script to work)
Nevertheless looking forward for this tool.
Sexan is offline   Reply With Quote
Old 12-03-2015, 02:58 PM   #155
semiquaver
Human being with feelings
 
Join Date: Jun 2008
Posts: 4,914
Default

Quote:
Originally Posted by Xenakios View Post
Interesting stuff in this thread! Keep up working on it!

I am not sure if I have anything useful and interesting enough to contribute to the script repo, most of my scripts are completely specific to my own workflows but I can take a look if I happen to have something worth sharing.
don't hold out on us Xenakios!
semiquaver is offline   Reply With Quote
Old 12-03-2015, 03:22 PM   #156
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default

Michael Ellis, firstly, nice idea.
As you can see, I don`t add any python script since it needs so much additional actions for regular user (should I mention python slow as hell?). Even install of this already painfull, while all of this could be rewrited and wrapped into single (and relatively simple) EEL/Lua script so user will not need to install anything and just use it from action list.

Anyway, if someone will rewrite it into EEL/Lua and point your name in scriptname/credits - is it will be ok for you?
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate

Last edited by mpl; 12-03-2015 at 04:04 PM.
mpl is offline   Reply With Quote
Old 12-03-2015, 05:54 PM   #157
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default


Last edited by cfillion; 12-08-2015 at 06:38 PM.
cfillion is online now   Reply With Quote
Old 12-03-2015, 09:22 PM   #158
Michael Ellis
Human being with feelings
 
Michael Ellis's Avatar
 
Join Date: Nov 2015
Location: Weaverville NC
Posts: 59
Default

Quote:
Originally Posted by mpl View Post
Michael Ellis, firstly, nice idea.
...
Anyway, if someone will rewrite it into EEL/Lua and point your name in scriptname/credits - is it will be ok for you?
Thanks. If someone wants to rewrite it, that's perfectly ok with me.
__________________
Write music faster with Tbon. https://github.com/Michael-F-Ellis/tbon
Mac mini, 2.3 Ghz core i7, 16GB Ram, Linnstrument #437, Technics P30
"What survives every change of system is melody." -- Igor Stravinsky
Michael Ellis is offline   Reply With Quote
Old 12-03-2015, 11:11 PM   #159
mpl
Human being with feelings
 
mpl's Avatar
 
Join Date: Oct 2013
Location: Moscow, Russia
Posts: 2,548
Default


Should I do something else before these actions?
__________________
SoundCloud | MPL Scripts discussion | ReaPack | Donate
mpl is offline   Reply With Quote
Old 12-04-2015, 12:53 AM   #160
cfillion
Human being with feelings
 
cfillion's Avatar
 
Join Date: May 2015
Location: Québec, Canada
Posts: 2,887
Default

EDIT: Oh, this is on your own repository. Download these files and retry `bundle install`: https://github.com/cfillion/reapack-repository-tools.

Gemfile, Rakefile and .gitignore in the ReaScripts directory.

Last edited by cfillion; 12-04-2015 at 01:13 AM.
cfillion is online now   Reply With Quote
Reply

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT -7. The time now is 02:09 AM.


Powered by vBulletin® Version 3.8.11
Copyright ©2000 - 2019, vBulletin Solutions Inc.