NSLS-II / pyCHX

chx_analysis_codes
BSD 3-Clause "New" or "Revised" License
5 stars 6 forks source link

imports & params commented out from V1 release - can we delete or not? #44

Open ambarb opened 3 years ago

ambarb commented 3 years ago

This is a place holder to address lines of code that were commented out of V1. Are these lines used as on-the-fly configurations or can they be deleted?

SO far, V2 commented out can be differentiated from V1 commented out because the line will end with #common. For example in v2._commonspeckle.chx_xpcs_xsvs_jupyter_V1:

   1 #from pyCHX.v2._commonspeckle.chx_packages import * #common
   2 from pyCHX.v2._commonspeckle.chx_libs import markers, colors #common #TODO all other instances import with ()
   3 #from pyCHX.chx_generic_functions import get_short_long_labels_from_qval_dict
   4 #RUN_GUI = False
   5 #from pyCHX.chx_libs import markers
   6 import pandas as pds
   7 

First, we need to generate a report so someone knowledgeable can review the code. @ambarb will do that below, so assigning to her for now.