jiayuasu opened a new pull request, #994:
URL: https://github.com/apache/sedona/pull/994

   
   ## Did you read the Contributor Guide?
   
   - Yes, I have read [Contributor 
Rules](https://sedona.apache.org/latest-snapshot/community/rule/) and 
[Contributor Development 
Guide](https://sedona.apache.org/latest-snapshot/community/develop/)
   
   ## Is this PR related to a JIRA ticket?
   
   - Yes, the URL of the associated JIRA ticket is 
https://issues.apache.org/jira/browse/SEDONA-XXX. The PR name follows the 
format `[SEDONA-XXX] my subject`.
   
   ## What changes were proposed in this PR?
   
   1. Rename the function to RS_AsBase64
   2. Fix the wrong name of RasterOutputTests. This stopped JUnit from picking 
up the tests when run with `mvn install`
   3. Use another image for the test as sometimes (for some images) Java8 and 
Java11 produce different base64 strings given the same input image.
   4. Improve the documentation to explain how this function works. In 
particular, this function only visualizes the first 4 bands regardless how many 
bands the input raster could have.
   
   ## How was this patch tested?
   
   Passed unit tests
   
   ## Did this PR include necessary documentation updates?
   
   - Yes, I have updated the documentation update.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to