search

daynefiler / tcpl

Former repository for the tcpl R package project. New repo:
https://github.com/USEPA/CompTox-ToxCast-tcpl
4 stars 5 forks source link

Updates to RSQLite causes errors throughout package #5

Closed daynefiler closed 6 years ago

daynefiler commented 8 years ago

Received an email indicating the SQLite package is being updated and is causing issues in the tcpl package. Two issues here:

  1. the dbGetQuery function no longer appears to clear the result set. Should be an easy fix -- just manually clear the result set when the driver is SQLite.
  2. there seems to be a bug in the new SQLite version causing fields with NULL values to be loaded as a character, even when defined as 'double' in the database. For example, run class(tcplLoadData(lvl = 5)$modl_ga) with default package settings.
daynefiler commented 8 years ago

I notified the RSQLite developers (rstats-db/RSQLite#145) about a bug with data classes. They have decided to hold off on the planned RSQLite update for now.

daynefiler commented 6 years ago

Going to tentatively close this.

daynefiler commented 6 years ago

The data type issues and NULL values appear to be closed by rstats-db/RSQLite#160. PR #26 should will fully close the issue.