115115
116116
117117# TODO: storing parameters in separate table as an extension (dropping
118- # the column parametenrs would be much nicer
118+ # the column parameters would be much nicer
119119
120120# TODO: metadata split between well known columns and maybe something else is
121121# not such a good idea. The problem is if we allow for specific columns then
122- # how do the user/us know which are metatadata ? I THINK the only sane solution
122+ # how do the user/us know which are metadata ? I THINK the only sane solution
123123# is to store JSON in a column called metadata
124124
125- # TODO: fixix a subset of metadata that we define well known (and create them)
125+ # TODO: fixing a subset of metadata that we define well known (and create them)
126126# i.e. no dynamic creation of metadata columns, but add stuff to
127127# a json inside a 'metadata' column
128128
129129
130130class _BackgroundWriter (Thread ):
131131 """
132- Write the results from the DataSet's dataqueue in a new thread
132+ Write the results from the DataSet's data queue in a new thread
133133 """
134134
135135 def __init__ (self , queue : Queue [Any ], conn : AtomicConnection ):
@@ -574,7 +574,7 @@ def add_metadata(self, tag: str, metadata: Any) -> None:
574574 """
575575 Adds metadata to the :class:`.DataSet`. The metadata is stored under the
576576 provided tag. Note that None is not allowed as a metadata value, and the
577- tag has to be a valid python identified (e.g. containing alphanumeric
577+ tag has to be a valid python identifier (e.g. containing alphanumeric
578578 characters and underscores).
579579
580580 Args:
@@ -614,7 +614,7 @@ def pristine(self) -> bool:
614614 """
615615 Is this :class:`.DataSet` pristine? A pristine :class:`.DataSet` has not yet been started,
616616 meaning that parameters can still be added and removed, but results
617- can not be added.
617+ cannot be added.
618618 """
619619 return not (self ._started or self ._completed )
620620
@@ -629,7 +629,7 @@ def running(self) -> bool:
629629 @property
630630 def started (self ) -> bool :
631631 """
632- Has this :class:`.DataSet` been started? A :class:`.DataSet` not started can not have any
632+ Has this :class:`.DataSet` been started? A :class:`.DataSet` not started cannot have any
633633 results added to it.
634634 """
635635 return self ._started
@@ -712,7 +712,7 @@ def _perform_start_actions(self, start_bg_writer: bool) -> None:
712712
713713 def mark_completed (self ) -> None :
714714 """
715- Mark :class:`.DataSet` as complete and thus read only and notify the subscribers
715+ Mark :class:`.DataSet` as complete and thus read- only and notify the subscribers
716716 """
717717 if self .completed :
718718 return
@@ -1688,7 +1688,7 @@ def load_by_id(run_id: int, conn: AtomicConnection | None = None) -> DataSetProt
16881688 If no connection is provided, lookup is performed in the database file that
16891689 is specified in the config.
16901690
1691- Note that the ``run_id`` used in this function in not preserved when copying
1691+ Note that the ``run_id`` used in this function is not preserved when copying
16921692 data to another db file. We recommend using :func:`.load_by_run_spec` which
16931693 does not have this issue and is significantly more flexible.
16941694
@@ -1774,7 +1774,7 @@ def load_by_counter(
17741774
17751775 Lookup is performed in the database file that is specified in the config.
17761776
1777- Note that the `counter` used in this function in not preserved when copying
1777+ Note that the `counter` used in this function is not preserved when copying
17781778 data to another db file. We recommend using :func:`.load_by_run_spec` which
17791779 does not have this issue and is significantly more flexible.
17801780
0 commit comments