Skip to content

Commit

Permalink
Merge pull request #109 from dihm/break_runmanager_dep
Browse files Browse the repository at this point in the history
Break dependency on runmanager by using `labscript_utils.shot_utils.get_shot_globals`
  • Loading branch information
dihm authored Oct 12, 2023
2 parents 91b87a9 + f926975 commit 4674e8b
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
4 changes: 2 additions & 2 deletions lyse/dataframe_utilities.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
import labscript_utils.shared_drive
from labscript_utils.connections import _ensure_str
from labscript_utils.properties import get_attributes
import runmanager
from labscript_utils.shot_utils import get_shot_globals


def asdatetime(timestr):
Expand All @@ -26,7 +26,7 @@ def asdatetime(timestr):
return pandas.Timestamp(timestr, tz=tz)

def get_nested_dict_from_shot(filepath):
row = runmanager.get_shot_globals(filepath)
row = get_shot_globals(filepath)
with h5py.File(filepath,'r') as h5_file:
if 'results' in h5_file:
for groupname in h5_file['results']:
Expand Down
1 change: 0 additions & 1 deletion readthedocs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,5 +23,4 @@ python:
path: .
extra_requirements:
- docs
system_packages: true

3 changes: 1 addition & 2 deletions setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,11 @@ install_requires =
desktop-app>=0.1.2
h5py
importlib_metadata
labscript_utils>=3.0.0
labscript_utils>=3.3.0
matplotlib
numpy
pandas>=0.21
qtutils>=2.2.2
runmanager>=3.0.0
scipy
tzlocal
zprocess>=2.2.2
Expand Down

0 comments on commit 4674e8b

Please sign in to comment.