gif-animation icon indicating copy to clipboard operation
gif-animation copied to clipboard

Looking for maintainer / project owner

Open extrapixel opened this issue 6 years ago • 39 comments

@01010101 @cgiles @jimcavoli @aib

extrapixel avatar Nov 27 '18 17:11 extrapixel

@villares @bryant1410

extrapixel avatar Nov 27 '18 17:11 extrapixel

I don't think I would be the right person because I've only ever used the library a few times and I'm not really a processing user.

But I wonder if simply giving people commit access would help. https://felixge.de/2013/03/11/the-pull-request-hack.html

aib avatar Dec 03 '18 09:12 aib

I might be able to maintain it... I'm the author of the VideoExport library for Processing and I combined several C GIF libraries last year for improved quantization.

The only thing is that I'm slightly busy atm. Is there much work to be done currently? Maybe make the 3.0 branch the default?

hamoid avatar Dec 03 '18 15:12 hamoid

@hamoid I think the 3.0 branch pretty much works as is - first thing would be to publish it so folks can use it again via the "new" processing library distribution method(s).

extrapixel avatar Dec 03 '18 16:12 extrapixel

So I volunteer to keep it running and respond to issues and PRs, basically, do maintenance work. Maybe you want to wait and see if there's different "candidates" and what is their expertise? Should anything be discussed via e-mail?

hamoid avatar Dec 03 '18 17:12 hamoid

That's great @hamoid , I've added you as a collaborator. If more interested people show up, I will add them too. Let's see how that goes, before we do the owner transfer. For openness and transparency I'd prefer to continue all communication here instead of email. Does that work for you?

First things would be to test the 3.0 branch - maybe check/fix open issues for 3.0. People say it woks, so there's probably not much to be done, except formalising a new release for the Processing IDE library manager. Another thing is to bring stuff from 3.0 branch to master and github pages.

extrapixel avatar Dec 03 '18 17:12 extrapixel

I could help with testing.

I personally think releasing 3.0 branch as default could be done immediately after confirming that the examples work in PDE 3.x -- and then later track down whether #13 is still an issue and how to fix.

If more people are using gifAnimation via Contributions manager and become familiar with it again then the pool of potential contributors should also increase.

jeremydouglass avatar Dec 03 '18 18:12 jeremydouglass

Thank you :) I'll compile the library locally and test on 3.0. I'm fine with discussing everything here.

hamoid avatar Dec 03 '18 18:12 hamoid

Funny question: how do you build the library? I don't see any of the files from the library template there (resources/, build.xml, build.properties). I can add those myself, but I was just wondering how do you do it @extrapixel ?

hamoid avatar Dec 03 '18 19:12 hamoid

How do you build the library?

I've just taken a quick look inside the 4 ".java" files of this library. This library only imports the vanilla Java classes and some Processing classes. Should be pretty easy to compile them all even manually by using just the "javac" command line.

GoToLoop avatar Dec 03 '18 20:12 GoToLoop

@hamoid, yes @GoToLoop is right, it will compile in the command line using javac without problems. I followed this information back then: https://github.com/processing/processing/wiki/Library-Basics

I agree @jeremydouglass . Would be great if you could you test branch 3.0 and also check wether the installation procedure described in the 3.0 branch readme is correct and works.

extrapixel avatar Dec 03 '18 22:12 extrapixel

Thanks @extrapixel that link helped (I always run an ant script to build my library). This is what I did:

javac -source 1.6 -target 1.6 -d . -classpath /usr/share/processing/core/library/core.jar src/*.java
jar -cf gifAnimation.jar gifAnimation

The first command creates a gifAnimation folder with the .class files, the second command makes the jar out of those .class files.

I'll test the jar tomorrow.

hamoid avatar Dec 03 '18 22:12 hamoid

alright! also, feel free to create new dev branch, including the jar etc, so the others can use your version to test.

extrapixel avatar Dec 03 '18 22:12 extrapixel

Today I tested the produced jar file with the latest Processing and both examples worked fine (display gif and export P3D to gif). I'll create the dev branch tomorrow :)

hamoid avatar Dec 04 '18 19:12 hamoid

I tested the dev branch on Windows 10 Edu 64bit, and everything works fine. Now all the issues (including #13) are solved. I think the only pending work is just to comply with the advertising protocol before contacting the Processing Librarian Elie Zananiri to restore access from the Processing IDE.

Pending tasks:

  • [x] Make two changes in the file library.properties,

  • change: url = http://github.com/01010101/GifAnimation to: url = https://github.com/extrapixel/gif-animation

  • change: minRevision = 3.0 to: minRevision = 0246 so as it matches the revision number of Processing 3.0

  • [x] Create the GifAnimation.txt and GifAnimation.zip files

  • [x] Contact Elie Zananiri

By the way, it could be the right time to update the page http://extrapixel.github.io/gif-animation/ , and to patch the broken link http://www.extrapixel.ch/ on the "GifAnimation" entry on https://processing.org/reference/libraries/

I volunteer to help @hamoid on this final tasks, if necessary.

AlvaroHermida avatar Dec 30 '18 03:12 AlvaroHermida

The PR #21 tries to simplify the promotion of the current dev branch for advertising on the Processing IDE. After merging both on the master branch it is necessary to check if the following url's works (they depend on the merged contents):

  • [ ] https://github.com/extrapixel/gif-animation/blob/master/GifAnimation.txt
  • [ ] https://github.com/extrapixel/gif-animation/blob/master/GifAnimation.zip

...and we will be ready to contact Elie Zananiri.

AlvaroHermida avatar Dec 30 '18 05:12 AlvaroHermida

Thank you for the help @AlvaroHermida ! :) I will take a look at this in the next days. Happy end of year!

hamoid avatar Dec 30 '18 09:12 hamoid

Hi @AlvaroHermida , thank you very much! I'v added you as a collaborator - if you want, please go ahead with the merging.

extrapixel avatar Dec 30 '18 10:12 extrapixel

Thanks! I think we may wait a few days to enjoy the New Year's Eve and polish some details (particularly the final url's, since they may change to something like "https://github.com/extrapixel/gif-animation/blob/3.0/gifAnimation.txt" after changing the default branch).

In fact, I still need to understand why the url https://github.com/extrapixel/gif-animation/archive/3.0.zip works, while I cannot access the archive folder from the web interface. Any hint will be appreciated.

Happy end of year!

AlvaroHermida avatar Dec 30 '18 13:12 AlvaroHermida

Sorry, I clicked the wrong button... ("Close and comment" instead of "Comment")

AlvaroHermida avatar Dec 30 '18 13:12 AlvaroHermida

Any hint will be appreciated.

It's the link that is provided by the GitHub website: screen shot 2018-12-30 at 14 51 38

extrapixel avatar Dec 30 '18 13:12 extrapixel

Thanks again!

So, since we need to host the .txt and the .zip on the same static folder, these files will need manual updating on every new release.

You will need to provide a static link to the latest version of your library, and host a copy of your library.properties alongside it. The result on your server should look something like this:

http://www.example.com/awesomelib.zip (The latest version of awesomelib) http://www.example.com/awesomelib.txt (A copy of the library.properties file contained in awesomelib.zip) ... On our end, we'll only record the location of awesomelib.txt, and we'll assume that a copy of the library file is hosted at an address by the same name, except ending in zip

I have updated #21 accordingly, to use the following url's

  • [ ] https://github.com/extrapixel/gif-animation/blob/3.0/gifAnimation.txt
  • [ ] https://github.com/extrapixel/gif-animation/blob/3.0/gifAnimation.zip

I'll go now to standby mode 😴 until @hamoid checks the PR. See you next year!

AlvaroHermida avatar Dec 30 '18 14:12 AlvaroHermida

Should we host those two files in a GitHub Page? I think only @extrapixel can enable that feature, if desired. It looks something like this: 2019-01-04-145608_792x434_scrot We could choose docs/ and then commit into that folder the binary and the txt file. Any time we update those files in the repo, they should become available in the Library Manager in the PDE. What do you think?

hamoid avatar Jan 04 '19 13:01 hamoid

@hamoid GitHub pages are currently in the gh-pages-branch: https://github.com/extrapixel/gif-animation/tree/gh-pages

Do you want it to move to the docs folder?

extrapixel avatar Jan 04 '19 14:01 extrapixel

Ah, I didn't know it was set up already. It's visible at https://extrapixel.github.io/gif-animation/

We only need to post the zip and the txt to that repo, right?

hamoid avatar Jan 04 '19 14:01 hamoid

We only need to post the zip and the txt to that repo, right?

That should work. We could also have a distribution folder within the master branch to host these files directly in the repo: https://github.com/extrapixel/gif-animation/raw/master/distribution/gifAnimation.zip https://github.com/extrapixel/gif-animation/raw/master/distribution/gifAnimation.txt

This could make the release/update process a tad easier.

extrapixel avatar Jan 04 '19 14:01 extrapixel

Yes, the ./raw/master/distribution trick will facilitate future updates, assuming it qualifies as "static link" for the Contribution Manager updating script. Thanks for the info, @extrapixel!

I have edited again the README.md file to reflect the new repository address, and I have created and populated the distribution folder (both in patch-1 and in master, just in case).

The next steps after @hamoid greenlights the latest changes are just to merge #21, change the default branch, and write to the librarian. If you want to delegate this tasks, just tell me :) .

AlvaroHermida avatar Jan 07 '19 03:01 AlvaroHermida

Hi! Sorry for the delay... I went for a long trip and it slipped my mind. I tried the download link from your README, installed it and both examples work fine in Linux. Green light on my side :)

hamoid avatar Feb 07 '19 05:02 hamoid

@AlvaroHermida from my point of view, you can proceed with the merging etc. Thanks!!!!

extrapixel avatar Feb 07 '19 13:02 extrapixel

Thanks for thinking of me; I've really not done much with the project recently, so I think I'm probably better left as just a contributor for now, but thanks for the nod. It sounds like some of the fine folks here have things covered. Should that change and the need arise again, I'd be happy to reconsider in the future.

jimcavoli avatar Feb 09 '19 03:02 jimcavoli