Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement #2

Merged
merged 9 commits into from
May 8, 2024
Merged

Enhancement #2

merged 9 commits into from
May 8, 2024

Conversation

zhu0619
Copy link
Contributor

@zhu0619 zhu0619 commented May 7, 2024

  • Logging image with bytes
  • Added distribution action
  • Added logic when discretization action before distribution
  • Refactored stereoactivity cliff detection
  • Update the tests accordingly
  • Update html report style

@zhu0619 zhu0619 requested a review from cwognum as a code owner May 7, 2024 15:52
@cwognum cwognum merged commit 493d11e into bootstrap May 8, 2024
4 checks passed
@cwognum cwognum deleted the dev branch May 8, 2024 00:06
cwognum added a commit that referenced this pull request May 8, 2024
* Basic repo structure

* Basic skeleton code

* First implementation of all major components: Action, Curator, Report, Broadcaster

* Added a functional module and added discretization

* Added deduplication

* Add stereoisomer AC detection

* Ported over test cases

* Made a good start on visualizations

* Tested with an integrated example - Added it as a tutorial

* Basic setup of docs

* Add HTML broadcaster

* Added chemspace visualization

* Added basic, untested CLI implementation

* Added visualization to stereoisomers with ACs

* Update README

* Fix bug

* Fix another bug

* Enhancement (#2)

* dev refactor

* fix test

* fix tests

* remove unused code

* refactor repo name

* Reviewed @zhu0619's changes. Switched to jinja2 for HTML  broadcaster

* Fixed bug in LoggerBroadcaster

* Fixed release CICD

* Fix CICD

---------

Co-authored-by: cwognum <[email protected]>

---------

Co-authored-by: Lu Zhu <[email protected]>
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