kernel_spec_manager_class sometimes only works under ServerApp instead of JupyterApp in jupyter_config.json

Hello,
I've noticed an issue with Jupyter Lab finding kernels inside Conda environments using nb_conda_kernels on the managed notebooks on Google Cloud.
Here is the start of the server logs, if helpful:
Jul 12 20:13:40 twrightsman bash[414]: [W 2022-07-12 20:13:40.596 ServerApp] notebook_dir is deprecated, use root_dir
Jul 12 20:13:44 twrightsman bash[414]: [I 2022-07-12 20:13:44.912 ServerApp] beatrix_jupyterlab | extension was successfully linked.
Jul 12 20:13:44 twrightsman bash[414]: [W 2022-07-12 20:13:44.918 MathJaxExtension] Config option `kernel_spec_manager_class` not recognized by `MathJaxExtension`.
Jul 12 20:13:44 twrightsman bash[414]: [W 2022-07-12 20:13:44.921 MathJaxExtension] Config option `kernel_spec_manager_class` not recognized by `MathJaxExtension`.
Jul 12 20:13:44 twrightsman bash[414]: [W 2022-07-12 20:13:44.936 MathJaxExtension] Config option `kernel_spec_manager_class` not recognized by `MathJaxExtension`.
Jul 12 20:13:44 twrightsman bash[414]: [I 2022-07-12 20:13:44.940 ServerApp] jupyter_server_mathjax | extension was successfully linked.
Jul 12 20:13:44 twrightsman bash[414]: [I 2022-07-12 20:13:44.940 ServerApp] jupyter_server_proxy | extension was successfully linked.
Jul 12 20:13:44 twrightsman bash[414]: [W 2022-07-12 20:13:44.946 LabApp] Config option `kernel_spec_manager_class` not recognized by `LabApp`.
Jul 12 20:13:44 twrightsman bash[414]: [W 2022-07-12 20:13:44.958 LabApp] Config option `kernel_spec_manager_class` not recognized by `LabApp`.
Jul 12 20:13:44 twrightsman bash[414]: [W 2022-07-12 20:13:44.973 LabApp] Config option `kernel_spec_manager_class` not recognized by `LabApp`.
Jul 12 20:13:44 twrightsman bash[414]: [I 2022-07-12 20:13:44.975 ServerApp] jupyterlab | extension was successfully linked.
Jul 12 20:13:44 twrightsman bash[414]: [I 2022-07-12 20:13:44.975 ServerApp] jupyterlab_git | extension was successfully linked.
Jul 12 20:13:44 twrightsman bash[414]: [I 2022-07-12 20:13:44.975 ServerApp] jupytext | extension was successfully linked.
Jul 12 20:13:44 twrightsman bash[414]: [W 2022-07-12 20:13:44.979 NotebookApp] Config option `kernel_spec_manager_class` not recognized by `NotebookApp`.
Jul 12 20:13:45 twrightsman bash[414]: [W 2022-07-12 20:13:45.027 NotebookApp] Config option `kernel_spec_manager_class` not recognized by `NotebookApp`.
Jul 12 20:13:45 twrightsman bash[414]: [W 2022-07-12 20:13:45.045 NotebookApp] Config option `kernel_spec_manager_class` not recognized by `NotebookApp`.
Jul 12 20:13:45 twrightsman bash[414]: [I 2022-07-12 20:13:45.047 ServerApp] nbclassic | extension was successfully linked.
Jul 12 20:13:45 twrightsman bash[414]: [I 2022-07-12 20:13:45.047 ServerApp] nbdime | extension was successfully linked.
Jul 12 20:13:45 twrightsman bash[414]: [I 2022-07-12 20:13:45.108 ServerApp] nb_conda | extension was found and enabled by notebook_shim. Consider moving the extension to Jupyter Server's extension paths.
Jul 12 20:13:45 twrightsman bash[414]: [I 2022-07-12 20:13:45.108 ServerApp] nb_conda | extension was successfully linked.

The jupyter server version and verbose status output from nb_conda_kernels.install:
$ jupyter server --version
1.18.0
$ python -m nb_conda_kernels.install --status --verbose
Determining the status of nb_conda_kernels...
CONDA_PREFIX: /opt/conda
Entry points:
  - conda = nb_conda_kernels.discovery:CondaKernelProvider
  NOTE: entry points not used in Jupyter 7.3.4
Configuration files:
  - /etc/jupyter: <no files>
  - /usr/local/etc/jupyter: <no files>
  * $CONDA_PREFIX/etc/jupyter:
      jupyter_config.json: {
          "JupyterApp": {
            "kernel_spec_manager_class": "nb_conda_kernels.CondaKernelSpecManager"
          }
        }
      jupyter_notebook_config.json: {
          "NotebookApp": {
            "nbserver_extensions": {
              "nb_conda": true,
              "jupyter_http_over_ws": true
            }
          }
        }
  - ~/.local/etc/jupyter: <no files>
  - ~/.jupyter: <no files>
Status: enabled

I noticed that if I change the "JupyterApp" key in "$CONDA_PREFIX/etc/jupyter" to "ServerApp" and restart the server then the kernels in other environments are detected by Jupyter Lab, even if the extension is disabled (which it seems to revert to on server restart).
$ python -m nb_conda_kernels.install --status --verbose
Determining the status of nb_conda_kernels...
CONDA_PREFIX: /opt/conda
Entry points:
  - conda = nb_conda_kernels.discovery:CondaKernelProvider
  NOTE: entry points not used in Jupyter 7.3.4
Configuration files:
  - /etc/jupyter: <no files>
  - /usr/local/etc/jupyter: <no files>
  * $CONDA_PREFIX/etc/jupyter:
      jupyter_config.json: {
          "ServerApp": {
            "kernel_spec_manager_class": "nb_conda_kernels.CondaKernelSpecManager"
          }
        }
      jupyter_notebook_config.json: {
          "NotebookApp": {
            "nbserver_extensions": {
              "nb_conda": true,
              "jupyter_http_over_ws": true
            }
          }
        }
  - ~/.local/etc/jupyter: <no files>
  - ~/.jupyter: <no files>
Status: disabled

I unfortunately can't reproduce this on my personal laptop with fresh Conda installs of Jupyter Lab and nb_conda_kernels, the "JupyterLab" key works and I can discover kernels in other Conda environments. So I'm a bit lost at what might be occurring in the managed Jupyter environment...

Duplicate of #223 , closing in favor of that one


fix: onGestureEnd -> endWithSpring uses outdated data via useSharedValue

Relates to #574

You can't imagine how happy I am of seeing these PRs getting merged! Thanks for that! This is one of the bugs that was blocking my release.
I'm gonna try it today and give some feedback.
Thanks again!

You can't imagine how happy I am of seeing these PRs getting merged! Thanks for that! This is one of the bugs that was blocking my release.
I'm gonna try it today and give some feedback. Thanks again!

Sorry for keeping you waiting for so long.

Naaaa, no problem. Everything can wait in this life


Bump crate versions

Hi!
Could we get a new release soon? I would like to use https://github.com/bytecodealliance/wasm-tools/pull/1043 to support final types in SpiderMonkey.

Sure thing, I've opened https://github.com/bytecodealliance/wasm-tools/pull/1047 to do that

Closing as this was done awhile back


⚠ Spryr has degraded performance

In 7c0f37e, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 429
Response time: 11141 ms

Resolved: Spryr performance has improved in 09cc1b4 after 12 minutes.


show git log

please add some table with latest X commits...
there should be some butoon "reset to this"

also show changes when a commit is selected, text should be copiable.
maybe Sourcetree is a good inspiration:


[BUG] Remove deprecated warning on sgd_demo.ipynb sample notebook

Describe the bug
In sgd_demo.ipynb notebook, cell #8 outputs a deprecated warning:

It can be easily avoided by using max_iter instead of n_iter.
Hope it helps!
Miguel

closed with https://github.com/rapidsai/notebooks/pull/157


FHIR Repository API (in-memory)

Added an in-memory repository with search functionality and tests.
Question for reviewer:
Is the search functionality needed for the Bundle repository? (It is an easy add if it is needed)

@barhodes I appreciate the contributions to this PR. However, I would like to know why these changes are necessary. It would have been nice to have you review the code and request changes that I could apply. One change I added that you rolled back was ignoring the provided resource ID for the create operation (as the spec outlines). What is the reasoning to not ignore the ID? How does create differ from the update operation?

@barhodes I appreciate the contributions to this PR. However, I would like to know why these changes are necessary. It would have been nice to have you review the code and request changes that I could apply. One change I added that you rolled back was ignoring the provided resource ID for the create operation (as the spec outlines). What is the reasoning to not ignore the ID? How does create differ from the update operation?

I ran into issues using the IdDt type when matching that were resolved by using the Ids helper like we did in the TestRepository.  Looks like I got a little overzealous in merging those on the create and missed the random id getting created.  I'll correct that.  The rest of the changes were all done to ensure compatibility with hapi-fhir so the search parameters work correctly when using the Clinical Reasoning module against Smile CDR.

Thanks for the info! So this PR is kind of in a weird state as you are both a contributor and a reviewer. Should I review your code or will someone else be providing that review?

Yeah, I've asked JP to review.  You are more than welcome to review as well.


Update to the latest version of the maven-plugin

update dependency to maven-plugin version 2.12.1
adapt the configure-branch-entries.jelly accordingly

Are there any plans when the next release (containing this change) will take place?

With the large bump in core version I want to make this part of the 0.4 release, which is going to include some other things I'm currently working on.  No definitive answer other than "soon" :)


[Liger] add native liger-kernl orpo loss

What does this PR do?
Adds support for Liger ORPO loss kernel to the ORPO Trainer natively.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

2 questions/remarks:

can you run benchmark so that we can (1) quantify the improvement and (2) check that results with and without liger are the same
we could have an additional tag for the hub when a model is trained with liger

I think we should bump liger version to v0.5 (it doesn't include the loss before), see https://github.com/linkedin/Liger-Kernel/releases/tag/v0.5.0

waiting on https://github.com/linkedin/Liger-Kernel/pull/486

waiting on https://github.com/huggingface/trl/pull/2502

@kashif can you share the curves once it's ready?

needs: https://github.com/linkedin/Liger-Kernel/pull/503


Mirrored video results for DyNeRF dataset

Hi,
Thanks for your nice work!
Here is a small issue about the mirrored video results (flipped the left and right). It will appear when we rendering the DyNeRF dataset.

I found that the poses returned by average_poses and viewmatrix in 4DGaussians/scene/neural_3D_dataset_NDC.py were left-handed.
Should we change these to right-handed instead?
i.e.,
    # 4. Compute the x axis
    x = normalize(np.cross(y_, z))  # (3)

    # 5. Compute the y axis (as z and x are normalized, y is already of norm 1)
    y = np.cross(z, x)  # (3)

and
    m[:3] = np.stack([vec0, vec1, vec2, pos], 1)

Thank you for your suggestion!
Rendering validation video will not affect experiment results in my paper and only for visualization. And I'll fix it in the next version.


resolver: spurious regression in resolution of globals

$ starlark  # at revision 6ddc71c
Welcome to Starlark (go.starlark.net)
>>> x = 1
>>> x += 2
Traceback (most recent call last):
  <stdin>:1:1: in <toplevel>
Error: global variable x referenced before assignment

I suspect this is a REPL bug: the first statement creates x=1 in the globals; the REPL then adds x=1 in the predeclared for the second statement. The x += 1 is resolved as a global because it's an assignment, but there is no global x. Perhaps x +=1 should not be treated as a binding since the binding must already exist.

It works as expected on 50ca820fafb9

This was issue #224, fixed by 77c1099ca038b502b6f4607b00edae412647ba37.


Imagemagick path?

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 2.2.9

Your OS

Windows 10 Pro

Summary
Just ran 'vagrant up --provision' using:
vagrant_box: geerlingguy/centos7
php_version: "7.3"
php_packages_extra:

"php-bcmath.x86_64"
"php-imagick"

phpinfo shows imagick ini here: /etc/php.d/40-imagick.ini
Website working great (Drupal 8.9.0), went on to enable imagemagick module v3.1.
Then went to configure it in admin/config/media/image-toolkit
There it asks for the path to the package executables. Where do I find this in DrupalVM?
I tried /usr/bin/convert and that is not correct. So how can I find the path to enter here?
Sorry for the newbie question.
Thank you.

While you installed the PHP extensionphp-imagick, you didn't install imagemagick itself.
On your config.yml, you need to install it. Find the extra_packages line with sqlite under it. Under that, add imagemagick,
extra_packages:
  - sqlite
  - imagemagick

What @rubenvarela says is correct—you also need to install the imagemagick package itself, in addition to the PHP library for it. Once you do that, it should be available in /usr/bin/convert.
Thanks!


New folders

Hi!
I use your theme for my personal projects and at work as well, this is really aesthetic thing.
What do you think about adding icon for folder 'actions'?
It would be nice to have sth like that, relating to naming folders in react web projects which use redux.
Maybe do you also consider setting icon for 'redux' folder?

Hello
I have thought about that i will add it ofcourse


Parse error for .babelrc files with comments or unquoted keys

It looks like this module is using JSON.parse(source) to parse the file. That fails unless it's JSON, which is not always (or even usually?) the case. As mentioned here, the format is JSON5, which allows comments, unquoted keys etc.

I’d accept a PR to fix this.