Skip to content

Refactor: minor README improvements + code syntax adjustments#35

Merged
carloalbertobarbano merged 3 commits intoEIDOSLAB:developmentfrom
andreped:development
May 25, 2023
Merged

Refactor: minor README improvements + code syntax adjustments#35
carloalbertobarbano merged 3 commits intoEIDOSLAB:developmentfrom
andreped:development

Conversation

@andreped
Copy link
Collaborator

@andreped andreped commented Jan 28, 2023

Modifications to the README:

  • Old algorithms bullet points replaced by implementation table
  • Removed old implementation table
  • Specified that runtime results are for Macenko only
  • Minor inconsistency fixes

Modifications to the code:

  • All modifications were syntax-related, and do not change any of the actual behaviour

@andreped
Copy link
Collaborator Author

Otherwise, I believe this branch can be merged and a new release can be made.

Jax backend support, as well as support for the vahadane algorithm can be made in the future. Not feasible to add for this release.

@andreped andreped changed the title Refactor: minor README improvements Refactor: minor README improvements + code syntax adjustments Jan 28, 2023
@andreped
Copy link
Collaborator Author

andreped commented Mar 2, 2023

@carloalbertobarbano Note that I have improved documentations in this PR. We have also added support for modified reinhard, which was not documented in the existing README.

Also, what about JAX support? Vahadane is not ready to be added, but I believe the Macenko with JAX support should be ready, no?

@carloalbertobarbano carloalbertobarbano merged commit 6db513b into EIDOSLAB:development May 25, 2023
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.

2 participants