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

fix view shape for perlin #36

Merged
merged 1 commit into from
Feb 24, 2021
Merged

fix view shape for perlin #36

merged 1 commit into from
Feb 24, 2021

Conversation

rafaqz
Copy link
Member

@rafaqz rafaqz commented Feb 21, 2021

What the pull request does
Fixes row/ col order in views into the temp square array

Type of change
Please indicate the relevant option(s)

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ❇️ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📖 This change requires a documentation update

Checklist

  • The changes are documented
    • The docstrings of the different functions describe the arguments, purpose, and behavior
    • There are examples in the documentation website
  • The changes are tested
  • The changes do not modify the behavior of the previously existing functions
    • If they do, please explain why and how in the introduction paragraph
  • For new contributors - my name and information are added to .zenodo.json
  • The Project.toml field version has been updated
    • Change the last number for a v0.0.x series release, a bug fix, or a performance improvement
    • Change the middle number for additional features that do not break the current test suite (unless you fix a bug in the test suite)
    • Change the first number for changes that break the current test suite

@rafaqz rafaqz requested a review from tpoisot February 21, 2021 21:25
@rafaqz
Copy link
Member Author

rafaqz commented Feb 21, 2021

We probably need to run all the algorithms for rectangle and square outputs of a few sizes and mix of parameters, not necessarily testing the output just that it works.

@tpoisot
Copy link
Contributor

tpoisot commented Feb 22, 2021

Can you add some tests on combinations of sizes to make sure that it marches?

@rafaqz
Copy link
Member Author

rafaqz commented Feb 23, 2021

I was thinking we should just do that for all of the methods here in one place. Just loop over some sizes/shapes and run them all. I'll look at writing that.

@rafaqz
Copy link
Member Author

rafaqz commented Feb 24, 2021

Ok tests are in the other PR as they may need separate discussion. However this needs to merge for them to pass! I didnt think that through properly...

@mkborregaard
Copy link
Member

Just merge when you feel confident. We still have 0 users :-)

@rafaqz rafaqz merged commit d901551 into main Feb 24, 2021
@rafaqz rafaqz deleted the rs/perlin_array_shape branch February 24, 2021 12:35
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.

3 participants