Skip to content

Commit

Permalink
Opt-in to uploading profiles from why.log() (#1571)
Browse files Browse the repository at this point in the history
## Description

Separates the notion of authentication mode (authenticated, anonymous,
local) from whether or not `why.log()` should automatically upload
profiles.

## Changes

- Makes `allow_local` default to `True`
- Adds `upload_on_log` configuration parameter orthogonal to session
type (aka authentication mode)
- Prompts for `upload_on_log` in interactive sessions
- `WhyLabsWriter::write()` will warn about possible redundant uploads if
called in a session with `update_on_log` enabled

<!-- Reference related commits, issues and pull requests. Type `#` and
select from the list. -->

Closes whlabs/whylogs#1570


- [ ] I have reviewed the [Guidelines for Contributing](CONTRIBUTING.md)
and the [Code of Conduct](CODE_OF_CONDUCT.md).
  • Loading branch information
richard-rogers authored Oct 1, 2024
1 parent 89fec66 commit 7a5c611
Show file tree
Hide file tree
Showing 17 changed files with 1,454 additions and 1,273 deletions.
4 changes: 2 additions & 2 deletions python/examples/basic/Getting_Started_with_UDFs.ipynb
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@
"metadata": {},
"outputs": [],
"source": [
"%pip install whylogs"
"%pip install 'whylogs>=1.5.0'"
]
},
{
Expand Down Expand Up @@ -133,7 +133,7 @@
"source": [
"import whylogs as why\n",
"\n",
"why.init()\n",
"why.init(force_local=True)\n",
"\n",
"results = why.log(df, name=\"udf_demo\", schema=custom_schema)\n",
"results.view().to_pandas()"
Expand Down
Loading

0 comments on commit 7a5c611

Please sign in to comment.