Conversation
|
Thanks for this. But this is only a small part of the complete fix. |
|
Ok. I should be able to implement the remainder of the changes needed this week and update this pull request with the new commits when I complete the necessary changes. |
TonioF
left a comment
There was a problem hiding this comment.
The ResamplingOp actually does not require many changes. Most work needs to be done here to adapt the spinner to support double values.
| private void reactToSourceProductChange(Product product) { | ||
| if (product != null) { | ||
| resolutionSpinner.setValue(determineResolutionFromProduct(product)); | ||
| resolutionSpinner.setValue(new Double(determineResolutionFromProduct(product))); |
There was a problem hiding this comment.
Rather than creating a Double object here, make determineResolutionFromProduct return a double in the first place.
This change allows the Resampling dialog to now successfully open and resample imagery, casting values to Double.