-
Notifications
You must be signed in to change notification settings - Fork 8
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
chore: release v0.9.0 #232
base: main
Are you sure you want to change the base?
Conversation
WalkthroughThis pull request updates the versioning across the Augurs workspace, incrementing the main package and all related dependencies from Changes
Possibly related PRs
Poem
π Recent review detailsConfiguration used: CodeRabbit UI π Files selected for processing (8)
π§ Files skipped from review as they are similar to previous changes (4)
π Additional comments (5)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? πͺ§ TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
π§Ή Nitpick comments (3)
crates/augurs-dtw/CHANGELOG.md (1)
13-13
: Remove incorrect component prefix in changelog entry.The
*(forecaster)*
prefix seems out of place in theaugurs-dtw
crate's changelog. This prefix suggests the feature belongs to the forecaster component, which might confuse users looking for DTW-specific changes.-*(forecaster)* add NaN handling to MinMaxScaler and StandardScaler (#227) +add NaN handling to MinMaxScaler and StandardScaler (#227)crates/augurs-outlier/CHANGELOG.md (1)
13-13
: Remove incorrect component prefix in changelog entry.The
*(forecaster)*
prefix is incorrectly included in theaugurs-outlier
crate's changelog. This prefix should be removed to maintain consistency with the component's scope.-*(forecaster)* add NaN handling to MinMaxScaler and StandardScaler (#227) +add NaN handling to MinMaxScaler and StandardScaler (#227)crates/augurs-core/CHANGELOG.md (1)
13-13
: Remove incorrect component prefix in changelog entry.The
*(forecaster)*
prefix should be removed from theaugurs-core
crate's changelog as it incorrectly suggests this is a forecaster-specific change.-*(forecaster)* add NaN handling to MinMaxScaler and StandardScaler (#227) +add NaN handling to MinMaxScaler and StandardScaler (#227)
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π Files selected for processing (6)
Cargo.toml
(2 hunks)crates/augurs-core/CHANGELOG.md
(1 hunks)crates/augurs-dtw/CHANGELOG.md
(1 hunks)crates/augurs-forecaster/CHANGELOG.md
(1 hunks)crates/augurs-outlier/CHANGELOG.md
(1 hunks)crates/augurs-prophet/CHANGELOG.md
(1 hunks)
β° Context from checks skipped due to timeout of 90000ms (21)
- GitHub Check: macos (macos-14, aarch64)
- GitHub Check: macos (macos-13, x86_64)
- GitHub Check: windows (windows-latest, x86)
- GitHub Check: windows (windows-latest, x64)
- GitHub Check: linux (ubuntu-22.04, ppc64le)
- GitHub Check: musllinux (ubuntu-22.04, armv7)
- GitHub Check: linux (ubuntu-22.04, s390x)
- GitHub Check: linux (ubuntu-22.04, armv7)
- GitHub Check: musllinux (ubuntu-22.04, aarch64)
- GitHub Check: linux (ubuntu-22.04, aarch64)
- GitHub Check: Test Book
- GitHub Check: musllinux (ubuntu-22.04, x86)
- GitHub Check: Tests
- GitHub Check: linux (ubuntu-22.04, x86)
- GitHub Check: Rustfmt
- GitHub Check: musllinux (ubuntu-22.04, x86_64)
- GitHub Check: Clippy
- GitHub Check: linux (ubuntu-22.04, x86_64)
- GitHub Check: Prophet WASMStan component
- GitHub Check: JS tests
- GitHub Check: Check
π Additional comments (4)
Cargo.toml (1)
21-21
: LGTM! Version updates are consistent.The version bump to 0.8.2 is consistently applied across the workspace package and all augurs-* dependencies.
Also applies to: 32-41
crates/augurs-forecaster/CHANGELOG.md (1)
13-13
: LGTM! Changelog entry is correctly formatted.The changelog entry is properly formatted with the correct component prefix, as this is the forecaster crate where the NaN handling feature was implemented.
crates/augurs-prophet/CHANGELOG.md (2)
10-11
: Verify the comparison link for version 0.8.2The version comparison link appears to be correctly formatted, but let's verify it resolves properly.
β Verification successful
The 404 error for the comparison link is expected
The changelog entry is for an upcoming release (v0.8.2) scheduled for January 11, 2025. The comparison link will work once the tag
augurs-prophet-v0.8.2
is created during the release process. The current latest version is v0.8.1, released on January 7, 2025.π Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the comparison link for version 0.8.2 resolves properly # Expected: Link should return a 200 status code curl -s -o /dev/null -w "%{http_code}" "https://github.com/grafana/augurs/compare/augurs-prophet-v0.8.1...augurs-prophet-v0.8.2"Length of output: 132
Script:
#!/bin/bash # List tags containing "prophet" to understand the naming pattern git tag | grep -i "prophet" # Show 5 most recent tags to verify the current format git tag --sort=-creatordate | head -n 5Length of output: 69
Script:
#!/bin/bash # Fetch all tags git fetch --tags # List tags again git tag | grep -i "prophet" # Check releases through GitHub API gh release list --repo grafana/augurs --limit 5Length of output: 17733
12-14
: Verify consistency of NaN handling changes across componentsThe addition of NaN handling is documented consistently with the forecaster prefix. Let's verify this change is documented similarly across all affected components.
β Verification successful
NaN handling changes are consistently documented
The NaN handling changes for MinMaxScaler and StandardScaler are properly documented across all affected components (core, forecaster, and prophet) with consistent "(forecaster)" prefix and PR reference (#227).
π Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify NaN handling documentation across components # Expected: Similar entries should exist in other component changelogs # Search for similar NaN handling entries in other component changelogs rg -l "add NaN handling to MinMaxScaler and StandardScaler" "crates/*/CHANGELOG.md"Length of output: 147
Script:
#!/bin/bash # First, find all CHANGELOG.md files echo "=== Searching for CHANGELOG.md files ===" fd -t f "CHANGELOG.md$" # Then, find implementations of MinMaxScaler and StandardScaler echo -e "\n=== Searching for Scaler implementations ===" ast-grep --pattern 'struct $_(MinMaxScaler|StandardScaler)' # Search for any NaN-related entries in changelogs with context echo -e "\n=== Searching for NaN-related entries in changelogs ===" rg -i "nan" -A 2 "CHANGELOG.md"Length of output: 930
Script:
#!/bin/bash # Search for NaN-related entries in all changelogs echo "=== Searching for NaN-related entries in changelogs ===" rg -i "nan" "crates/*/CHANGELOG.md" # Look for Scaler implementations with a more flexible pattern echo -e "\n=== Searching for Scaler implementations ===" ast-grep --pattern 'struct $_Scaler' # Search specifically in forecaster-related files echo -e "\n=== Searching in forecaster files ===" rg -l "Scaler" "crates/augurs-forecaster/**/*"Length of output: 570
Script:
#!/bin/bash # Search for NaN-related entries in changelogs echo "=== Searching for NaN-related entries in changelogs ===" rg -i "nan" crates/augurs-prophet/CHANGELOG.md crates/augurs-forecaster/CHANGELOG.md crates/augurs-core/CHANGELOG.md # Look for Scaler implementations echo -e "\n=== Searching for Scaler implementations ===" fd -e rs . crates/augurs-forecaster crates/augurs-core | xargs rg "Scaler" -A 5Length of output: 17437
bc00d91
to
09d648c
Compare
π€ New release
augurs
: 0.8.1 -> 0.9.0 (β API compatible changes)augurs-changepoint
: 0.8.1 -> 0.9.0augurs-core
: 0.8.1 -> 0.9.0 (β API compatible changes)augurs-clustering
: 0.8.1 -> 0.9.0 (β API compatible changes)augurs-dtw
: 0.8.1 -> 0.9.0 (β API compatible changes)augurs-ets
: 0.8.1 -> 0.9.0augurs-mstl
: 0.8.1 -> 0.9.0augurs-forecaster
: 0.8.1 -> 0.9.0 (β API compatible changes)augurs-outlier
: 0.8.1 -> 0.9.0 (β API compatible changes)augurs-prophet
: 0.8.1 -> 0.9.0 (β API compatible changes)augurs-seasons
: 0.8.1 -> 0.9.0Changelog
augurs
augurs-changepoint
augurs-core
augurs-clustering
augurs-dtw
augurs-ets
augurs-mstl
augurs-forecaster
augurs-outlier
augurs-prophet
augurs-seasons
This PR was generated with release-plz.
Summary by CodeRabbit
augurs
dependencies from version 0.8.1 to 0.9.0MinMaxScaler
andStandardScaler
componentsDbscanCluster
type replacingisize
in clustering functionality