Skip to content

Conversation

zm711
Copy link
Contributor

@zm711 zm711 commented Oct 4, 2025

Waiting to see if we get through a couple more PRs in the next few days to update this PR.

Also waiting to add one additional person to author list once they give me their affiliation.

@samuelgarcia
Copy link
Contributor

OK for me.
I think that @h-mayorquin will be happy to have some PR merged for this release.

@h-mayorquin
Copy link
Contributor

I think that the over-segmentation fixes would probably be slower and can be delayed. I know that Zach would like to keep the schedule. But one that I would really like to add is:

#1790

which should be, 1) easy to review with around ten lines and, 2) only fixing a memory inefficiency (so not design considerations to discuss).

@zm711
Copy link
Contributor Author

zm711 commented Oct 9, 2025

Okay with #1790 being added in. Agree segmentation is a big PR. Let's wait on that one.

Also the person said they didn't want to be added to author list yet. So once #1790 is merged we are good to go.

@zm711 zm711 marked this pull request as ready for review October 13, 2025 11:42
@samuelgarcia samuelgarcia merged commit da45948 into NeuralEnsemble:master Oct 13, 2025
78 checks passed
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