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

mark k_means_heuristic_synth_small test with xfail #264

Merged
merged 1 commit into from
Jul 27, 2022

Conversation

jGaboardi
Copy link
Member

This PR provides a temporary solution to #213 until we figure out what to do more permanently.

@jGaboardi jGaboardi self-assigned this Jul 26, 2022
@jGaboardi jGaboardi requested a review from gegen07 July 26, 2022 20:22
@jGaboardi
Copy link
Member Author

xref #259

@codecov
Copy link

codecov bot commented Jul 26, 2022

Codecov Report

Merging #264 (fb27286) into main (a17c8da) will increase coverage by 0.2%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #264     +/-   ##
=======================================
+ Coverage   66.1%   66.3%   +0.2%     
=======================================
  Files         23      23             
  Lines       2582    2582             
  Branches     566     566             
=======================================
+ Hits        1706    1711      +5     
+ Misses       790     781      -9     
- Partials      86      90      +4     
Impacted Files Coverage Δ
spopt/_version.py 33.8% <0.0%> (+1.5%) ⬆️

@ljwolf
Copy link
Member

ljwolf commented Jul 27, 2022

Looks fine to me!

@jGaboardi jGaboardi merged commit 6879240 into pysal:main Jul 27, 2022
@jGaboardi jGaboardi deleted the skip_k_means_heuristic branch July 27, 2022 14:05
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.

None yet

2 participants