ProvTools / provR

GNU General Public License v2.0
7 stars 3 forks source link

json parse error from cleanR #68

Closed MKLau closed 7 years ago

MKLau commented 7 years ago

This is for the tfjmp@capture pull request.

Error: parse error: unallowed token at this point in JSON text : "", "rdt:sourcedScripts" : , "rdt:scriptTimeStamp" : "", " (right here) ------^ In addition: Warning message: In .ddg.insert.error.message(error.msg) : No binding for ddg.start.time . DDG may be incorrect!

tfjmp commented 7 years ago

Can you check that it does not happened with the master branch?

Can you post the whole JSON output?

Also could you come up with a few lines of scripts so we can add cleanR regression test to the automated test?

Thanks!

MKLau commented 7 years ago

Looks like this is a combination of the prov.json not being empty intially and cleanR not correctly initializing.

tfjmp commented 7 years ago

Any progress on getting the data?

MKLau commented 7 years ago

Sorry, I should have closed this after my last comment. It's a cleanR issue, I need to adjust the API.

tfjmp commented 7 years ago

Did you do: Also could you come up with a few lines of scripts so we can add cleanR regression test to the automated test?

MKLau commented 7 years ago

No, but it's on my desk to add this to the cleanR issue system

MKLau commented 7 years ago

Hopefully, I'll get to it tomorrow afternoon

MKLau commented 7 years ago

The issue itself, that is. I'll just go add this to cleanR now