Emily KL

Results 110 comments of Emily KL

Actually, hmm. It looks like the entire `reformat_code()` function is missing from this branch entirely. To be honest I'm not very enthusiastic about mixing codegen changes in with this PR....

In any case, the `reformat_code()` function is pretty minimal. It's just: ```python import subprocess ... def reformat_code(outdir): """Reformat Python code using settings in pyproject.toml.""" subprocess.call(["ruff", "format", *make_paths(outdir)]) ``` So the...

@daexs There are two changes I had to make in order to build the docs successfully on my machine: - Add `import argparse` at the top of `bin/utils.py` - Add...

@daexs Yeah after looking at the history I realized all the codegen refactor was done by Greg, so don't worry about it!

@afeld AFAIK `nbformat` is only required when working with Jupyter notebooks, is there a scenario you've encountered where you require `nbformat` when working outside Jupyter? In general we try to...

> What would be the downside of extending this to `'auto'` as well? @alexjohnson-kobold Good point! I'm happy with extending it to `'auto'` as well if you're in favor.

Hi @my-tien, this is very interesting, thanks for reporting. I tried to reproduce on my own machine and I got slightly different numbers for the first figure (see below) but...

Yes now that we've narrowed down the issue to Plotly.js I'll move the issue over. I'm not sure why the bottom margin gets larger. That does seem unintended, but I...

Thank you for the contribution @Rhoahndur ! Profiling tools would be a great addition to the library, and we like the simplicity of this approach. Could you explain why you...

@alexcjohnson Totally agree, this is needed! Also related: There's some weird behaviour with the `NAMESPACE` file (which I think is part of the R components generation process?) containing lines for...