This PR resolves issue #39 and #40. The logic in reactive values for the DEA model has been reworked so that reactive values are not updated immediately. This should eliminate error messages in the console when the user uploads a new data set with different column names than the previous data set. New tests have also been introduced to the port argument of the run_pioneer function. Port numbers must be within the range 3000 through 65535, and not in the list of port numbers considered unsafe by Google Chrome. Previously, if an unsafe port was chosen, the app would start, but only show a blank page. In addition, this PR changes the following:
runPioneeR has been renamed to run_pioneer
A new function – unset_env_vars – has been added to clean environment variables set by run_pioneer (environment variables are normally unset, but will not if the app quits unexpectedly)
The label for the ID variable in the UI is not "Firm ID" which makes more sense for the user
This PR resolves issue #39 and #40. The logic in reactive values for the DEA model has been reworked so that reactive values are not updated immediately. This should eliminate error messages in the console when the user uploads a new data set with different column names than the previous data set. New tests have also been introduced to the
portargument of therun_pioneerfunction. Port numbers must be within the range 3000 through 65535, and not in the list of port numbers considered unsafe by Google Chrome. Previously, if an unsafe port was chosen, the app would start, but only show a blank page. In addition, this PR changes the following:runPioneeRhas been renamed torun_pioneerunset_env_vars– has been added to clean environment variables set byrun_pioneer(environment variables are normally unset, but will not if the app quits unexpectedly)