Attention is currently required from: dexter, laforge.
12 comments:
File pySim/esim/saip/batch.py:
Patch Set #6, Line 65: params: list[ParamAndSrc]=None,
IIRC the linter warns against passing empty lists as default argument, because in python there can a […]
Done
Patch Set #6, Line 92: csv_columns = next(self.csv_rows)
hm that's right, it has to be an iter(list), thx
Done
Patch Set #6, Line 97: csv_row = None
it is optional, configured by the caller, whether a CSV file is present. […]
Done
File pySim/esim/saip/param_source.py:
Patch Set #6, Line 31: class ParamSource:
That is right. […]
Done
Patch Set #6, Line 42: so the user can enter '0000' to get a four digit random number."""
Hmm. it's even worse, most subclasses completely replace this. I will make a better plan, thanks.
Done
yea you're right, sometimes i do sometimes i don't, thx. […]
Done
Patch Set #6, Line 65: def __init__(self, num_digits, first_value, last_value):
Maybe add type annotations and agree on one distinct type? At the moment this function can accepts a […]
Done
If this is a private/internal method, I would mark it with double underscore or single underscore?
not intended, in particular, to be internal
Patch Set #6, Line 95: last_value = int(last_str) if last_str is not None else "9" * len(first_str)
heh funny code =) you are right
Done
Patch Set #6, Line 121: val = random.randbytes(self.num_digits // 2) # TODO secure random source?
This uses the fact that all BinaryParameter can take a bytes as input value. […]
Done
Patch Set #6, Line 149: if val > self.val_first_last[1]:
self.last_value would be easier to read.
Done
Patch Set #6, Line 174: return val
it's more like […]
Done
To view, visit change 41845. To unsubscribe, or for help writing mail filters, visit settings.