Repository navigation
docs(changelog): record the graph builder, test teardown and MolNet changes - #47
Merged
Merged
Conversation
…hanges Three changes merged without a CHANGELOG entry. The graph builder now assembles molecular graphs on the host and returns float32 batches under x64, with refusals that name their cause; the test suite collects garbage after GPU-marked tests only; and the MolNet benchmark's quick mode featurizes only the molecules it keeps, with its split indexing the molecules the splitter chose. Each entry carries the measurements its change was merged with.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
#43, #44 and #45 merged without a CHANGELOG entry. This repository's CHANGELOG records CI, test-suite and benchmark changes as well as library ones (for example the shard and CI entries under 0.1.6 and 0.1.7), so all three belong under Unreleased.
Evidence
Each entry's figures are the ones its PR was merged with:
pre-commit run --all-filespasses all 24 hooks. The CHANGELOG is not part of the docs build.Merge Danger
Door: two-way.
Blast Radius: none.
Documentation only.