AMP-SCZ / lochness

Download your data to a data lake.
Other
3 stars 1 forks source link

Wait for RPMS dump at different times according to AEST/AEDT #128

Closed tashrifbillah closed 1 year ago

tashrifbillah commented 1 year ago

Prescient Linux system appropriately updates system time.

Local time ``` Local time: Sun 2023-09-10 03:40:00 AEST Universal time: Sat 2023-09-09 17:40:00 UTC RTC time: Sun 2023-09-10 03:40:00 Time zone: Australia/Sydney (AEST, +1000) NTP enabled: yes NTP synchronized: yes RTC in local TZ: yes DST active: no Last DST change: DST ended at Sun 2023-04-02 02:59:59 AEDT Sun 2023-04-02 02:00:00 AEST Next DST change: DST begins (the clock jumps one hour forward) at Sun 2023-10-01 01:59:59 AEST Sun 2023-10-01 03:00:00 AEDT ```

I think it is the RPMS dump system at RPMS end that does not update according to daylight time. As a result, with time changes, data are dumped an hour away. On the other hand, Tashrif's modifications must run right after dump so REDCap-NDA compatible data can propagate through the system. Hence, moving forward, we shall wait at different times for the dump to complete. The way Tashrif has set it up now, human intervention will not be needed.

Disclosure: Tashrif has set up TIME_ZONE variable in /etc/profile.d/timezone.sh .

tashrifbillah commented 1 year ago

@dheshanm , this will prevent flow of wrong psychs follow-up forms. So we should not see those errors in bsub/*err files.

tashrifbillah commented 1 year ago

Right. Since you primarily maintain this repository, it is better that you merge it and deploy accordingly.

kcho commented 1 year ago

Merged and deployed on the Prescient data aggregation server. Thanks.