dandi / dandisets-healthstatus

Healthchecks of dandisets and support libraries (pynwb and matnwb)
0 stars 1 forks source link

Stores some `_lineno` within `versions:` record #46

Closed yarikoptic closed 1 year ago

yarikoptic commented 1 year ago

Trying #44 with

(venv) (base) dandi@drogon:~/cronlib/dandisets-healthstatus$ healthstatus test-files --save-results pynwb_open_load_ns /mnt/backup/dandi/dandisets-healthstatus/dandisets-fuse/000003/sub-YutaMouse33/sub-YutaMouse33_ses-YutaMouse33-150218_behavior+ecephys.nwb

resulted in the diff (will clean after filing this issue):

diff --git a/results/000003/status.yaml b/results/000003/status.yaml
index 3a034a8..dadbf55 100644
--- a/results/000003/status.yaml
+++ b/results/000003/status.yaml
@@ -1,7 +1,8 @@
+dandiset: '000003'
 dandiset_version: 9c6ab8750946cdac16c62057ec1f76f9ce954fe1
-last_run: 2023-02-02 16:56:48.227118-05:00
+last_run: '2023-02-02T16:56:48.227118-05:00'
 last_run_duration: 370696.096205
-last_run_ended: 2023-02-06 23:55:04.323323-05:00
+last_run_ended: '2023-02-06T23:55:04.323323-05:00'
 nassets: 101
 tests:
 - assets_nok: []
@@ -217,5 +218,8 @@ tests:
 untested: []
 versions:
   hdmf: 3.5.1
+  hdmf_lineno: '219'
   matnwb: v2.6.0.0
+  matnwb_lineno: '220'
   pynwb: 2.2.0
+  pynwb_lineno: '221'

I don't think addition of those _lineno for each record was intended, was it?