search

Exawind / amr-wind

AMReX-based structured wind solver
https://exawind.github.io/amr-wind
Other
106 stars 83 forks source link

Fix to spinner sampler num_output_points #1000

Closed ndevelder closed 6 months ago

ndevelder commented 6 months ago

This was introduced when radar sampler was introduced and num_points() no longer was assumed as the number of sampler output points.

tonyinme commented 6 months ago

Looks good.

@ndevelder , would it make sense to fix this test as well in this PR?

tonyinme commented 6 months ago

@mbkuhn , this looks good, can we merge it?

mbkuhn commented 6 months ago

@mbkuhn , this looks good, can we merge it?

Ready to approve when the checks pass! Small typo in a comment, it appears. Sorry, Nate.