Janek Klawe

Results 19 issues of Janek Klawe

This PR refactors refactors our task structure to make saving and loading into separate tasks. There should be no behavior changes (except for some slight changes in logging). There are...

Sometimes users pass a single argument to `@bn.outputs`, expecting it to work like `@bn.output`: builder = bn.FlowBuilder('test') @builder @bn.outputs('value') def _(): return 7 flow = builder.build() flow.get('value') This fails because...

Currently, when an entity has both persistence and memoization disabled, we store its value in memory for the duration the `Flow.get()` call (as documented [here](https://bionic.readthedocs.io/en/stable/concepts.html#disabling-in-memory-caching)). I think we should change...

The `FlowBuilder.merge` method excepts a flow, not a builder, which seems to be a common source of surprise. What's worse, if you pass it a builder you get a cryptic...

In general, almost any exception while reading a cached file will currently be reported as an `InvalidCacheStateError`, which means the user will be told the cache is corrupted and needs...

bug

Sometimes GCS file uploads (and presumably downloads) can time out (stack trace attached below). For most of these operations we use the GCS Python API rather than gsutil, so it's...

bug
enhancement

Currently the helper function `bionic.util.init_basic_logging` configures the global logging state, with a default level of `INFO` for all loggers. However, I suspect most users only use this because they want...

When an already-existing entity is defined with a function, its previous declaration (including any protocol, docstring, etc.) is erased. This is intended behavior, but it can be surprising -- especially...

Entities marked with `@changes_per_run` generate a new cache entry on disk each time they're computed, even though these cached entries are unlikely to ever be used again. We should write...

When we run `generate_test_compatibility_cache.py` to update our regression test vectors, it doesn't delete the old test vectors, but there's no point in keeping them around. It would be better if...

enhancement