Myles Scolnick comments

Results 578 comments of


                                            Myles Scolnick

Support for Azure OpenAI with custom domain (Azure API / corporate environment)

hey @sanzoghenzo - that sounds like a reasonable change. would you be open to helping contribute this feature?

Support for Azure OpenAI with custom domain (Azure API / corporate environment)

I think it might be better to go with `[ai.azure]`. But we will still need to set which provider to use, so maybe we add `[ai.provider_type]` like here: https://github.com/marimo-team/marimo/issues/4318

Support for Azure OpenAI with custom domain (Azure API / corporate environment)

Hey @sanzoghenzo, anything we can help with on this PR? If you want to put up a draft PR, we can take a look early or make contributions

Support for Azure OpenAI with custom domain (Azure API / corporate environment)

This is now fixed by https://github.com/marimo-team/marimo/pull/6018 (which will be in the next release 0.15.0). Providers are encoded in the model ID: `azure/gpt-4o`, `openai/gpt-4o`

Single-item collections display as "1 items" instead of "1 item"

Looks like an issue with an upstream library: https://github.com/TexteaInc/json-viewer/pull/531

Single-item collections display as "1 items" instead of "1 item"

Closed by https://github.com/marimo-team/marimo/pull/3430

import order matters when running as `marimo edit my-notebook.py` vs `python my-notebook.py` on the same environment

@colobas thanks for reporting and the detailed issue. this is definitely an odd bug. we will look into this next

tuple dict keys display as strings

What is the expected output? This looks pretty correct to me?

The output shown for types of `list` and `tuple` is identical

This would maybe a be a bit tricky. These get converted to JSON types (that output is called the json-viewer), so set/list/tuple all get converted to `array`

The output shown for types of `list` and `tuple` is identical

@bluss i have a fix out for the case of `sys.version_info` (when an object extends list/tuple/dict): https://github.com/marimo-team/marimo/pull/3047