carissalow / rapids

Reproducible Analysis Pipeline for Data Streams
http://www.rapids.science/
GNU Affero General Public License v3.0
37 stars 20 forks source link

Fix minor app foreground, Bluetooth, keyboard, screen bugs #213

Closed jenniferfedor closed 1 year ago

jenniferfedor commented 1 year ago

This PR addresses several minor bugs identified recently:

  1. Time segments (rows) included in processed phone applications foreground features data were dependent on the order in which app categories and single apps were processed
  2. Phone screen episodes shorter than or longer than specified thresholds were being dropped based on their within-segment rather than total duration. We handle this correctly (i.e., drop based on total duration) for phone applications foreground
  3. Non-Unicode characters in phone Bluetooth bt_name and phone keyboard package_name, before_text, and current_text columns could not be parsed by pd.read_csv(). We add mutation scripts to coerce these columns into UTF-8 character encoding
  4. Minor corrections/updates to RAPIDS installation instructions for Docker and Ubuntu

Tests initially failed because R 4.3.0 was being installed during the tests workflow and this R version was incompatible with some of our package versions. This has now been fixed.