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.

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

authority_score and hub_score values changed compared to CI values #333

Open
bburns632 opened this issue Nov 11, 2024 · 0 comments
Open
Assignees
Labels
ci update needed an update is needed to align with other dependencies

Comments

@bburns632
Copy link
Collaborator

These values now different in CI pipeline. May be related to updated described in #332

── Failure ('test-DirectedGraph.R:108:13'): DirectedGraph node measure values are expected for baseballstats, FunctionReporter ──
reporter$pkg_graph$node_measures(nodeMeas) not equivalent to ...[].
Dataset 'current' has rows not present in 'target'
Value testing for baseballstats, FunctionReporter : hubScore /n obj: c("OPS", "at_bats", "batting_avg", "on_base_pct", "slugging_avg") /n exp c("on_base_pct", "OPS", "slugging_avg", "at_bats", "batting_avg")
Value testing for baseballstats, FunctionReporter : hubScore /n obj: c(1, 0, 0.500007341756502, 0, 0.500007341756502) /n exp c(0, 1, 1, 0, 1)
── Failure ('test-DirectedGraph.R:108:13'): DirectedGraph node measure values are expected for baseballstats, FunctionReporter ──
reporter$pkg_graph$node_measures(nodeMeas) not equivalent to ...[].
Dataset 'current' has rows not present in 'target'
Value testing for baseballstats, FunctionReporter : authorityScore /n obj: c("OPS", "at_bats", "batting_avg", "on_base_pct", "slugging_avg") /n exp c("on_base_pct", "OPS", "slugging_avg", "at_bats", "batting_avg")
Value testing for baseballstats, FunctionReporter : authorityScore /n obj: c(0, 0.999991335461266, 1, 0, 1) /n exp c(0, 0, 0.5, 1, 0.5)

https://github.com/uptake/pkgnet/actions/runs/11769695286/job/32781078118

@bburns632 bburns632 added update needed an update is needed to align with other dependencies ci labels Nov 11, 2024
@bburns632 bburns632 self-assigned this Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci update needed an update is needed to align with other dependencies
Projects
None yet
Development

No branches or pull requests

1 participant