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.

Sign up for GitHub

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.8.0 #189

Closed
wants to merge 1 commit into from
Closed

chore: release v0.8.0 #189

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 11 additions & 11 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ authors = [
]
documentation = "https://docs.rs/crate/augurs"
repository = "https://github.com/grafana/augurs"
version = "0.7.0"
version = "0.8.0"
edition = "2021"
keywords = [
"analysis",
Expand All @@ -29,16 +29,16 @@ keywords = [

[workspace.dependencies]
augurs = { path = "crates/augurs" }
augurs-changepoint = { version = "0.7.0", path = "crates/augurs-changepoint" }
augurs-clustering = { version = "0.7.0", path = "crates/augurs-clustering" }
augurs-core = { version = "0.7.0", path = "crates/augurs-core" }
augurs-dtw = { version = "0.7.0", path = "crates/augurs-dtw" }
augurs-ets = { version = "0.7.0", path = "crates/augurs-ets" }
augurs-forecaster = { version = "0.7.0", path = "crates/augurs-forecaster" }
augurs-mstl = { version = "0.7.0", path = "crates/augurs-mstl" }
augurs-outlier = { version = "0.7.0", path = "crates/augurs-outlier" }
augurs-prophet = { version = "0.7.0", path = "crates/augurs-prophet" }
augurs-seasons = { version = "0.7.0", path = "crates/augurs-seasons" }
augurs-changepoint = { version = "0.8.0", path = "crates/augurs-changepoint" }
augurs-clustering = { version = "0.8.0", path = "crates/augurs-clustering" }
augurs-core = { version = "0.8.0", path = "crates/augurs-core" }
augurs-dtw = { version = "0.8.0", path = "crates/augurs-dtw" }
augurs-ets = { version = "0.8.0", path = "crates/augurs-ets" }
augurs-forecaster = { version = "0.8.0", path = "crates/augurs-forecaster" }
augurs-mstl = { version = "0.8.0", path = "crates/augurs-mstl" }
augurs-outlier = { version = "0.8.0", path = "crates/augurs-outlier" }
augurs-prophet = { version = "0.8.0", path = "crates/augurs-prophet" }
augurs-seasons = { version = "0.8.0", path = "crates/augurs-seasons" }
augurs-testing = { path = "crates/augurs-testing" }

augurs-core-js = { path = "js/augurs-core-js" }
Expand Down
15 changes: 15 additions & 0 deletions crates/augurs-forecaster/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,21 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

## [0.8.0](https://github.com/grafana/augurs/compare/augurs-forecaster-v0.7.0...augurs-forecaster-v0.8.0) - 2024-12-16

### Added

- add 'transforms' JS crate and include in augurs JS bindings (#195)

Comment on lines +11 to +14
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue

Document breaking changes in the Added section.

The PR objectives mention that augurs-forecaster has breaking changes due to a non-exhaustive enum. This should be clearly marked as breaking in the changelog.

Apply this diff to clearly indicate the breaking change:

 ### Added
 
- - add 'transforms' JS crate and include in augurs JS bindings (#195)
+ - [**breaking**] add 'transforms' JS crate and include in augurs JS bindings (#195)
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
### Added
- add 'transforms' JS crate and include in augurs JS bindings (#195)
### Added
- [**breaking**] add 'transforms' JS crate and include in augurs JS bindings (#195)

### Fixed

- make Transform enum non-exhaustive (#194)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Document impact of non-exhaustive Transform enum.

The change to make Transform enum non-exhaustive is a breaking change that requires pattern matching to include a wildcard pattern. This impact should be documented to help users update their code.

Apply this diff to add migration instructions:

- - make Transform enum non-exhaustive (#194)
+ - [**breaking**] make Transform enum non-exhaustive (#194) - requires adding a wildcard pattern (`_`) to Transform enum matches
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- make Transform enum non-exhaustive (#194)
- [**breaking**] make Transform enum non-exhaustive (#194) - requires adding a wildcard pattern (`_`) to Transform enum matches


### Other

- precalculate offset and scale factor for min-max scale transformer (#196)
- Add power transformation logic to forecaster transforms ([#185](https://github.com/grafana/augurs/pull/185))
Comment on lines +19 to +22
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Improve changelog organization by properly categorizing changes.

The entries in the "Other" section appear to be functional changes that should be categorized more specifically:

  • "precalculate offset..." is a performance enhancement that should go under "Changed" or "Fixed"
  • "Add power transformation..." is a new feature that should go under "Added"

Apply this diff to reorganize the changes:

 ### Added
 - add 'transforms' JS crate and include in augurs JS bindings (#195)
+- Add power transformation logic to forecaster transforms ([#185](https://github.com/grafana/augurs/pull/185))
 
-### Fixed
+### Changed
 - make Transform enum non-exhaustive (#194)
+- precalculate offset and scale factor for min-max scale transformer (#196)
-
-### Other
-
-- precalculate offset and scale factor for min-max scale transformer (#196)
-- Add power transformation logic to forecaster transforms ([#185](https://github.com/grafana/augurs/pull/185))
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
### Other
- precalculate offset and scale factor for min-max scale transformer (#196)
- Add power transformation logic to forecaster transforms ([#185](https://github.com/grafana/augurs/pull/185))
### Added
- add 'transforms' JS crate and include in augurs JS bindings (#195)
- Add power transformation logic to forecaster transforms ([#185](https://github.com/grafana/augurs/pull/185))
### Changed
- make Transform enum non-exhaustive (#194)
- precalculate offset and scale factor for min-max scale transformer (#196)


## [0.7.0](https://github.com/grafana/augurs/compare/augurs-forecaster-v0.6.3...augurs-forecaster-v0.7.0) - 2024-11-25

### Other
Expand Down
14 changes: 14 additions & 0 deletions crates/augurs-prophet/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

## [0.8.0](https://github.com/grafana/augurs/compare/augurs-prophet-v0.7.0...augurs-prophet-v0.8.0) - 2024-12-16

### Added

- add Forecaster wrapper for Prophet (#191)

### Fixed

- add explicit link to chrono method (#192)

### Other

- *(deps)* update statrs requirement from 0.17.1 to 0.18.0 (#187)

## [0.7.0](https://github.com/grafana/augurs/compare/augurs-prophet-v0.6.3...augurs-prophet-v0.7.0) - 2024-11-25

### Breaking Changes
Expand Down