Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix histogram feature grouping on next-release-devel #646

Open
wants to merge 6 commits into
base: next-release-devel
Choose a base branch
from

Conversation

fotonick
Copy link

Same goal as before, but since it is an API-breaking change, it is now rebased to next-release-devel. Note that I also had to fix a Pie::new() doctest that failed to compile.

@fotonick
Copy link
Author

@AaronErhardt I fixed the CI build on this branch, in addition to my original patch. I hope it reduces chaos just a little for you.

If it's further helpful, I can commit a Cross.toml to aid multi-arch testing. From my Mac, I was able to cross test --target x86_64-unknown-linux-gnu to reproduce and debug the CI failures.

@fotonick fotonick force-pushed the fix_histogram_feature_grouping_2 branch from 9995160 to 298f718 Compare September 28, 2024 08:34
@fotonick fotonick force-pushed the fix_histogram_feature_grouping_2 branch from 298f718 to cb71888 Compare March 3, 2025 04:16
@fotonick
Copy link
Author

fotonick commented Mar 3, 2025

You're quite right that the last patch to datetime.rs (pulling from master) was a big step backwards. Sorry about that. This should be better. I think it leaves the codebase better than how I found it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants