Closed stevencox closed 5 years ago
Checked with Kara and Hao.... currently, "urban" is defined as "EstResidentialDensity >= 3". This may change, they are going to confer with Steve Appold on this issue.
The changes did not break the system.
Changed the workflow pair (.cwl and .yml) to 'workflow_5_v_3.cwl' and 'workflow_5_v_3.yml' respectively. Still stands to be improved...
I've created a second endpoint which utilizes 'TotalEDInpatientVisits > 2' as the feature variable. This is done in the 'server.py' file... however, I delineated this into two new files which are now titled 'icees_res_density_server.py' and 'icees_ed_visits_server.py' to represent this bifurcation. These two files must be operated in the same way which server.py has been prior to now.
The changes described in # 4 above, do not produce any change in the workflow output SIZE. In both cases, the output size is: "size": 883424,
Further, the checksum changes from test to test, even when nothing is changed and the original command is merely re-run. Current best guess is that there is a hard-limit to the size of the output but the gathering of contents is non-deterministic such that once ANY set of data of size = 883424, it is returned.
Thanks -
All issues addressed and the next set of concerns is addressed by issue #169
Bring the operation implementation currently in this service into alignment with current thinking on workflow five. (verify use of estimated population density, inequality values, p_value threshold, etc).
Verify all changes to that service continue to work with the workflow here.
Update the workflow name to something more readable.
Create a second endpoint using ed visits as a clustering criteria.