Skip to content

SAIS diff algorithm; VLE algorithms; feel free to tweak, cut & modify;#9

Closed
r-lyeh-archived wants to merge 1 commit into
mendsley:masterfrom
r-lyeh-archived:master
Closed

SAIS diff algorithm; VLE algorithms; feel free to tweak, cut & modify;#9
r-lyeh-archived wants to merge 1 commit into
mendsley:masterfrom
r-lyeh-archived:master

Conversation

@r-lyeh-archived

Copy link
Copy Markdown
  • I have updated README with the new directives.
  • I did not add another header/tag for the VLE variants (enabling VLE breaks backwards compatibility)

- added VLE streams
- compile as C++ bugfix
- fix for #5 and #1
- feel free to tweak, cut & modify
@r-lyeh-archived

Copy link
Copy Markdown
Author

I think that is all for now. I have had to merge on the run as my patch was based on the old version with h/c glued together : )

@CoolOppo

CoolOppo commented Apr 3, 2015

Copy link
Copy Markdown
Contributor

It might be a good idea to allow these to be used without recompiling the program, but through a flag you can call it with from your shell.

I was looking for an explanation for the differences you've introduced, but Google didn't really turn anything relevant up, and the README doesn't really explain the differences either. I can see that you definitely put some thought into this, but the code is really cryptic and difficult to get a hold of without taking a while to step through a bunch of parts in my head. Are you able to explain what exactly is going on from a higher-level point of view, and the advantages and disadvantages of these new algorithms?

@r-lyeh-archived

Copy link
Copy Markdown
Author

@CoolOppo, yeah sorry about the terse information.

@r-lyeh-archived

Copy link
Copy Markdown
Author

I could have sent two PR instead, but got it difficult to do so, as my fork was pretty outdated and diverged too much from base. Sorry about that.

@r-lyeh-archived

Copy link
Copy Markdown
Author

ping @mendsley

@otavio

otavio commented Jul 12, 2016

Copy link
Copy Markdown

Has this been abandonned? if so, why?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants