smallAreaHealthStatisticsUnit / rapidInquiryFacility

The Rapid Inquiry Facility (RIF) helps epidemiologists and public health researchers in environmental health activities.
GNU Lesser General Public License v3.0
14 stars 5 forks source link

State 1 study setup modal errors #143

Closed peterhambly closed 5 years ago

peterhambly commented 5 years ago

Fix for issue: #138 State 1: study setup modal errors.

Also includes R NA/NAN fixes and control of additional modules. You will need to enable multiple covariates in frontEndParameters.json5 if you want to use this functionality.

devilgate commented 5 years ago

install.log

peterhambly commented 5 years ago

Have fixed DROP INDEX code

peterhambly commented 5 years ago

The front end multiple covariates has to be a multi-selector as it reads into an array. You would need to resurrect at least some of the old code and that is too complex given the time constraints

devilgate commented 5 years ago

That's all good, but new error:

psql:../psql_scripts/v4_0_user.sql:110: INFO:  +00001.40s  rif40_startup(): Deleted 2, created 3 tables/views/foreign data wrapper tables
DO
SELECT geography, numerator_table, theme_description, denominator_table, automatic 
  FROM rif40_num_denom;
psql:../psql_scripts/v4_0_user.sql:113: ERROR:  relation "rif40_num_denom" does not exist at character 92

install.log

peterhambly commented 5 years ago

Removed reference. Should be OK unless the scripts try to create a test study.

devilgate commented 5 years ago

It may be doing exactly that. Not sure, but:

psql:/Users/martin/Dropbox/work/sahsu/rapidInquiryFacility/rifDatabase/Postgres/psql_scripts/alter_scripts/v4_0_alter_7.sql:150: ERROR:  relation "rif40_num_denom" does not exist
LINE 7:    FROM rif40_num_denom z, rif40_tables a, rif40_table_outco...

install.log

peterhambly commented 5 years ago

Have modified alter 7 so that it recreates the rif40 schemas rif40_num_denom so all scripts will run correctly

devilgate commented 5 years ago

Not quite, I'm afraid:

GRANT SELECT ON TABLE rif40.rif40_numerator_outcome_columns TO rif_manager;
GRANT
Time: 0.277 ms
--\dS+ rif40.rif40_numerator_outcome_columns
\i ../../PLpgsql/rif40_sql_pkg/rif40_startup.sql
psql:/Users/martin/Dropbox/work/sahsu/rapidInquiryFacility/rifDatabase/Postgres/psql_scripts/alter_scripts/v4_0_alter_13.sql:136: ../../PLpgsql/rif40_sql_pkg/rif40_startup.sql: No such file or directory

install.log

peterhambly commented 5 years ago

Path fix so same relative path as other alter scripts

devilgate commented 5 years ago

Looking good now. Tested it on SQL Server as well, and that worked fine.