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

Collections API: Implement RasterLinesJoin #53

Closed
rajadain opened this issue Jul 31, 2017 · 4 comments
Closed

Collections API: Implement RasterLinesJoin #53

rajadain opened this issue Jul 31, 2017 · 4 comments
Assignees
Labels

Comments

@rajadain
Copy link
Member

Old implementation: https://github.com/WikiWatershed/mmw-geoprocessing/blob/develop/summary/src/main/scala/MapshedJob.scala#L77-L130

Sample request and response: nlcd-streams.zip

Depends on #51

@rajadain rajadain added this to the Collections API milestone Jul 31, 2017
@rajadain rajadain added the 1 label Jul 31, 2017
@ajrobbins ajrobbins added the WPF label Aug 2, 2017
@rajadain rajadain changed the title Implement RasterLinesJoin Collections API: Implement RasterLinesJoin Aug 8, 2017
@kellyi kellyi added in progress and removed queue labels Aug 24, 2017
@kellyi kellyi self-assigned this Aug 24, 2017
@kellyi
Copy link
Contributor

kellyi commented Aug 28, 2017

Have this set up to perform an operation on the input & return the map of values, but the values I've got are currently off by a bit from the expected result:

HTTP/1.1 200 OK                                          
Content-Length: 254                                      
Content-Type: application/json                           
Date: Mon, 28 Aug 2017 17:09:55 GMT                      
Server: akka-http/10.0.9                                 

{                                                        
    "result": {                                          
        "List(11)": 2983,                                
        "List(21)": 40599,                               
        "List(22)": 52563,                               
        "List(23)": 47320,                               
        "List(24)": 30416,                               
        "List(31)": 118,                                 
        "List(41)": 15735,                               
        "List(42)": 65,                                  
        "List(43)": 1001,                                
        "List(52)": 1820,                                
        "List(71)": 697,                                 
        "List(81)": 1433,                                
        "List(82)": 3087,                                
        "List(90)": 30546,                               
        "List(95)": 4236                                 
    }                                                    
}     

Investigating...

@kellyi
Copy link
Contributor

kellyi commented Aug 28, 2017

(Here's the expected response values from the zip file above, btw:)

{
  "result": {
    "List(11)": 2282,
    "List(21)": 34415,
    "List(22)": 45118,
    "List(23)": 39960,
    "List(24)": 25494,
    "List(31)": 101,
    "List(41)": 13336,
    "List(42)": 58,
    "List(43)": 841,
    "List(52)": 1562,
    "List(71)": 597,
    "List(81)": 1282,
    "List(82)": 2708,
    "List(90)": 24158,
    "List(95)": 3332
  }
}

@rajadain
Copy link
Member Author

@kellyi
Copy link
Contributor

kellyi commented Aug 28, 2017

Got another version of this working and/but its counts are slightly off in a different direction.

kellyi added a commit that referenced this issue Aug 29, 2017
Implement RasterLinesJoin operation

Connects #53
rajadain added a commit to WikiWatershed/model-my-watershed that referenced this issue Aug 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants