Main icon indicating copy to clipboard operation
Main copied to clipboard

The spartan movie...

Open yossigil opened this issue 9 years ago • 24 comments

Add a new handler and a new command, to do spartanization step by step. This means, find a tip, apply it, repeat. I think this would be fun! What do you think @OriRoth

yossigil avatar Oct 04 '16 16:10 yossigil

Do this second... It is important since it would be a good selling point. But, it may take more than 10 minutes (not much more though)

yossigil avatar Oct 06 '16 11:10 yossigil

The Spartan movie is out, and it is better than 300! (https://en.wikipedia.org/wiki/300_(film)) Use keyboard shortcut Ctrl+Alt+M to start the movie. Beware: run time is estimated to be [number of spartanization suggestions] * 1 + 2 seconds, so it is not recommended to run it on large projects.

OriRoth avatar Oct 07 '16 17:10 OriRoth

@orimarco , @M4tt30rru

yossigil avatar Oct 08 '16 09:10 yossigil

spartanization

OriRoth avatar Oct 08 '16 10:10 OriRoth

Now also on the readme page: https://github.com/SpartanRefactoring/Spartanizer @yossigil

OriRoth avatar Oct 08 '16 10:10 OriRoth

:+1:

yossigil avatar Oct 08 '16 17:10 yossigil

@yossigil I saw you are interested in improving the SpartanMovie. The current implementation works, but is defiantly non releasable - do you want me to make it more usable? (require some work, mostly answering threading related issues) (and more organized code)

OriRoth avatar Oct 17 '16 13:10 OriRoth

No. This is not a product, but I hope @OElmak will do something about it.

yossigil avatar Oct 18 '16 10:10 yossigil

@OElmak check your Telegram I have sent you a message. If you take this challenge you'll need some background story.

OriRoth avatar Oct 18 '16 13:10 OriRoth

@M4tt30rru We need your urgent help here.

yossigil avatar Dec 07 '16 21:12 yossigil

Why? This is a pure GUI issue... threads and stuff.

OriRoth avatar Dec 07 '16 21:12 OriRoth

@M4tt30rru has an implementation and a demo

yossigil avatar Dec 07 '16 22:12 yossigil

Matteo?

yossigil avatar Apr 20 '17 19:04 yossigil

@yossigil you may consider giving it to the yearly project team... to be done later this semester.

OriRoth avatar Apr 20 '17 22:04 OriRoth

i want of off load you guys, as much as I can.

yossigil avatar Apr 21 '17 03:04 yossigil

@yossigil I wrote to you on Telegram yesterday. @OriRoth I can start working on this today. I was abroad till yesterday.

M4tt30rru avatar Apr 21 '17 07:04 M4tt30rru

The Spartan movie works on my computer, even if it does not show the progress (as in the video that @OriRoth posted on the main page). I tested it on a number of projects. I can start working on this. Are there any other improvements should I work on?

M4tt30rru avatar Apr 22 '17 08:04 M4tt30rru

Can you publish it?

yossigil avatar Apr 22 '17 08:04 yossigil

I pushed the changes. It is now on the main menu.

M4tt30rru avatar Apr 22 '17 09:04 M4tt30rru

But isn't this the old version of the spartanizer?

yossigil avatar Apr 22 '17 09:04 yossigil

Exactly, this was actually my point. There is the problem I reported: it "freezes" and take some time to finish in some projects. I am working now on this "freezing" problem. For the rest it seems it is working fine.

M4tt30rru avatar Apr 22 '17 09:04 M4tt30rru

Great

yossigil avatar Apr 26 '17 05:04 yossigil

Done?

yossigil avatar May 25 '17 19:05 yossigil

There are some progress, but I am still working on it.

M4tt30rru avatar May 26 '17 07:05 M4tt30rru