error when launching gradio app

when launching gradeio app i get  the following error: ImportError:
cannot import name 'Schema' from 'pydantic'

I am also experiencing this issue. Installed via docker and running python 3.10

Got this issue to, trying to run via collab

Same issue and no answer from the devs. Looks like it's happening to a lot of other people to (in other places), not sure how to fix it...

Yup, here with the same problem -
(threestudio) root@a100-instance-ajna:~/threestudio# python3 gradio_app.py launch
Traceback (most recent call last):
  File "gradio_app.py", line 13, in <module>
    import gradio as gr
  File "/usr/local/lib/python3.8/dist-packages/gradio/__init__.py", line 3, in <module>
    import gradio._simple_templates
  File "/usr/local/lib/python3.8/dist-packages/gradio/_simple_templates/__init__.py", line 1, in <module>
    from .simpledropdown import SimpleDropdown
  File "/usr/local/lib/python3.8/dist-packages/gradio/_simple_templates/simpledropdown.py", line 6, in <module>
    from gradio.components.base import FormComponent
  File "/usr/local/lib/python3.8/dist-packages/gradio/components/__init__.py", line 1, in <module>
    from gradio.components.annotated_image import AnnotatedImage
  File "/usr/local/lib/python3.8/dist-packages/gradio/components/annotated_image.py", line 11, in <module>
    from gradio import processing_utils, utils
  File "/usr/local/lib/python3.8/dist-packages/gradio/processing_utils.py", line 22, in <module>
    from gradio.data_classes import FileData, GradioModel, GradioRootModel
  File "/usr/local/lib/python3.8/dist-packages/gradio/data_classes.py", line 19, in <module>
    from pydantic import BaseModel, RootModel, ValidationError  # type: ignore
ImportError: cannot import name 'RootModel' from 'pydantic' (/usr/local/lib/python3.8/dist-packages/pydantic/__init__.cpython-38-x86_64-linux-gnu.so)

Same here. Wsl Ubuntu docker giving same errors.

Hi everyone! Sorry for the inconvenience. I already fixed this problem in the latest commit. Please pull the new code and try again. You may also need to reinstall gradio:
pip uninstall gradio gradio_client pydantic pydantic_core
pip install gradio==4.11.0

Note that if your network cannot connect to huggingface, remember to add HF_HUB_OFFLINE=1 DIFFUSERS_OFFLINE=1 TRANSFORMERS_OFFLINE=1 before the gradio launch command.
The gradio demo is a very basic interface for some text-to-3D pipelines, and does not support image-to-3D yet. Welcome the contribute and make it better!

Still facing the pydantic problem with a new error:
Traceback (most recent call last):
File "gradio_app.py", line 13, in
import gradio as gr
File "/usr/local/lib/python3.8/dist-packages/gradio/init.py", line 3, in
import gradio._simple_templates
File "/usr/local/lib/python3.8/dist-packages/gradio/_simple_templates/init.py", line 1, in
from .simpledropdown import SimpleDropdown
File "/usr/local/lib/python3.8/dist-packages/gradio/_simple_templates/simpledropdown.py", line 6, in
from gradio.components.base import FormComponent
File "/usr/local/lib/python3.8/dist-packages/gradio/components/init.py", line 1, in
from gradio.components.annotated_image import AnnotatedImage
File "/usr/local/lib/python3.8/dist-packages/gradio/components/annotated_image.py", line 11, in
from gradio import processing_utils, utils
File "/usr/local/lib/python3.8/dist-packages/gradio/processing_utils.py", line 22, in
from gradio.data_classes import FileData, GradioModel, GradioRootModel
File "/usr/local/lib/python3.8/dist-packages/gradio/data_classes.py", line 12, in
from fastapi import Request
File "/usr/local/lib/python3.8/dist-packages/fastapi/init.py", line 5, in
from .applications import FastAPI
File "/usr/local/lib/python3.8/dist-packages/fastapi/applications.py", line 3, in
from fastapi import routing
File "/usr/local/lib/python3.8/dist-packages/fastapi/routing.py", line 6, in
from fastapi import params
File "/usr/local/lib/python3.8/dist-packages/fastapi/params.py", line 4, in
from pydantic import Schema
ImportError: cannot import name 'Schema' from 'pydantic' (/usr/local/lib/python3.8/dist-packages/pydantic/init.py)

Also having this issue "ImportError: cannot import name 'Schema' from 'pydantic' " .... Using gradio==4.11.0

the problem seems to stem from old and mismatch version of fastapi  (against gradio==4.11.0 and pydantic==2.5.2)
pip install -U fastapi pydantic works well for me

On the docker container, I pip install fastapi and pydantic however when I run gradio I get the following
To create a public link, set share=True in launch().
Exception in thread Thread-4 (_do_normal_analytics_request):
Traceback (most recent call last):
File "/home/dreamer/.local/lib/python3.10/site-packages/httpcore/_exceptions.py", line 10, in map_exceptions
yield
File "/home/dreamer/.local/lib/python3.10/site-packages/httpcore/_backends/sync.py", line 168, in start_tls
raise exc
File "/home/dreamer/.local/lib/python3.10/site-packages/httpcore/_backends/sync.py", line 163, in start_tls
sock = ssl_context.wrap_socket(
File "/usr/lib/python3.10/ssl.py", line 513, in wrap_socket
return self.sslsocket_class._create(
File "/usr/lib/python3.10/ssl.py", line 1100, in _create
self.do_handshake()
File "/usr/lib/python3.10/ssl.py", line 1371, in do_handshake
self._sslobj.do_handshake()
TimeoutError: _ssl.c:990: The handshake operation timed out

the problem seems to stem from old and mismatch version of fastapi (against gradio==4.11.0 and pydantic==2.5.2)
pip install -U fastapi pydantic works well for me

Unfortunately this didn't work for me.
ImportError: /home/dreamer/.local/lib/python3.10/site-packages/tinycudann_bindings/_89_C.cpython-310-x86_64-linux-gnu.so: undefined symbol: _ZN2at4_ops5zeros4callEN3c108ArrayRefINS2_6SymIntEEENS2_8optionalINS2_10ScalarTypeEEENS6_INS2_6LayoutEEENS6_INS2_6DeviceEEENS6_IbEE
I think the problem is this:
pip install -U fastapi pydantic installs fastapi-0.112.0 and pydantic-2.8.2
pydantic==2.8.2 does not provide 'Schema', pydantic==1.10 provides it.
but gradio==4.11.0 requires pydantic>=2.0
also, lightning==2.0.0 requires fastapi<0.89.0


ci: add pushval action for building ModuleSite

current assumption is that the deploy action does work with a mutable lockfile
which is why the previously added build pushval actions do not reproduce failure we see no merging to master

Now that finally reproduced the failure I see in the master-merge deploy action:
error /home/runner/work/ModuleSite/ModuleSite/node_modules/node-sass: Command failed.
This seems to be related to the node-sass module, on which I found the following: https://stackoverflow.com/questions/70281346/node-js-sass-version-7-0-0-is-incompatible-with-4-0-0-5-0-0-6-0-0, suggesting to replace node-sass with sass


Shows the ignored packages

Waybar-updates worked perfectly until I added a package I wanted to ignore into /etc/pacman.conf.

IgnorePkg = my-pkg
NoUpgrade = my-pkg

It does skip it in pacman / yay but not in waybar-updates.

@iDenso hm, I didn't think about that. It should be relatively easy to support, but I am not sure how to reliable parse Pacman config using bash. Simple grep'ing should be enough, but I am not sure.

Oh, there is pacman-conf command. But I am not sure what param I should respect, probably NoUpgrade?

I just checked the arch wiki page and it seems like (if I understood correctly) the IgnorePkg don't touch to the package at all while the NoUpgrade don't touch to the current version of the package but keep downloading the newer version beside.
Tbh I'm not an expert at all but if you could add at least the IgnorePkg it would be really great 😃

@iDenso should be fixed with v0.2.8!


python3Packages.pyspnego: 0.3.1 -> 0.4.0

Motivation for this change
https://github.com/jborean93/pyspnego/releases/tag/v0.4.0
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 160440 run on x86_64-linux 1

5 packages marked as broken and skipped:

nagstamon
python310Packages.dask-xgboost
python39Packages.dask-xgboost
python39Packages.rl-coach
wapiti

70 packages failed to build:

dvc
dvc-with-remotes
python310Packages.aplpy
python310Packages.asdf
python310Packages.batchgenerators
python310Packages.caffe
python310Packages.dask-gateway
python310Packages.dask-image
python310Packages.dask-jobqueue
python310Packages.dask-mpi
python310Packages.distributed
python310Packages.env-canada
python310Packages.glymur
python310Packages.handout
python310Packages.httpx-ntlm
python310Packages.image-match
python310Packages.imagecorruptions
python310Packages.imageio
python310Packages.imgaug
python310Packages.intensity-normalization
python310Packages.keystoneauth1
python310Packages.moviepy
python310Packages.napari
python310Packages.napari-console
python310Packages.napari-svg
python310Packages.openstacksdk
python310Packages.osc-lib
python310Packages.pims
python310Packages.pyregion
python310Packages.pytest-astropy
python310Packages.pytest-astropy-header
python310Packages.python-cinderclient
python310Packages.python-glanceclient
python310Packages.python-heatclient
python310Packages.python-ironicclient
python310Packages.python-keystoneclient
python310Packages.python-manilaclient
python310Packages.python-novaclient
python310Packages.python-openstackclient
python310Packages.python-swiftclient
python310Packages.reproject
python310Packages.runway-python
python310Packages.s3fs
python310Packages.scikitimage
python310Packages.scmrepo
python310Packages.slicedimage
python310Packages.spectral-cube
python310Packages.streamz
python310Packages.sunpy
python310Packages.tifffile
python310Packages.tokenizers
python310Packages.transformers
python39Packages.aplpy
python39Packages.asdf
python39Packages.bpycv
python39Packages.dask-gateway
python39Packages.dask-jobqueue
python39Packages.dask-ml
python39Packages.dask-mpi
python39Packages.datashader
python39Packages.distributed
python39Packages.httpx-ntlm
python39Packages.python-glanceclient
python39Packages.s3fs
python39Packages.scmrepo
python39Packages.spectral-cube
python39Packages.streamz
python39Packages.stumpy
python39Packages.sunpy
python39Packages.zcs

83 packages built:

aws-adfs (python39Packages.aws-adfs)
home-assistant-component-tests.environment_canada
intensity-normalization (python39Packages.intensity-normalization)
napari (python39Packages.napari)
openstackclient (python39Packages.python-openstackclient)
python310Packages.aws-adfs
python310Packages.casa-formats-io
python310Packages.dask
python310Packages.dask-glm
python310Packages.datasets
python310Packages.exchangelib
python310Packages.fsspec
python310Packages.gcsfs
python310Packages.ibis-framework
python310Packages.jenkinsapi
python310Packages.pyfftw
python310Packages.pyspnego
python310Packages.requests-kerberos
python310Packages.smbprotocol
python39Packages.batchgenerators
python39Packages.boxx
python39Packages.caffe
python39Packages.casa-formats-io
python39Packages.clifford
python39Packages.dask
python39Packages.dask-glm
python39Packages.dask-image
python39Packages.datasets
python39Packages.env-canada
python39Packages.exchangelib
python39Packages.fastparquet
python39Packages.fsspec
python39Packages.gcsfs
python39Packages.glymur
python39Packages.handout
python39Packages.ibis-framework
python39Packages.image-match
python39Packages.imagecorruptions
python39Packages.imageio
python39Packages.imgaug
python39Packages.intake
python39Packages.intake-parquet
python39Packages.jenkinsapi
python39Packages.keystoneauth1
python39Packages.mask-rcnn
python39Packages.moviepy
python39Packages.napari-console
python39Packages.napari-svg
python39Packages.openstacksdk
python39Packages.osc-lib
python39Packages.pims
python39Packages.pyfftw
python39Packages.pyregion
python39Packages.pyspnego
python39Packages.pytest-astropy
python39Packages.pytest-astropy-header
python39Packages.python-cinderclient
python39Packages.python-heatclient
python39Packages.python-ironicclient
python39Packages.python-keystoneclient
python39Packages.python-manilaclient
python39Packages.python-novaclient
swiftclient (python39Packages.python-swiftclient)
python39Packages.qiskit
python39Packages.qiskit-machine-learning
python39Packages.reproject
python39Packages.requests-kerberos
python39Packages.runway-python
python39Packages.scikitimage
python39Packages.slicedimage
python39Packages.smbprotocol
python39Packages.spacy-transformers
python39Packages.sparse
python39Packages.stytra
python39Packages.tensorflow-datasets
python39Packages.tensorly
python39Packages.test-tube
python39Packages.tifffile
python39Packages.tokenizers
python39Packages.transformers
tartube
tartube-yt-dlp
tts

nixpkgs-review reveals no new failures on macos 10.15 (but many many existing failures)

Already in staging.


Resources regarding great APIs with the result pattern

Hi there!
I made an article demonstrating what a great API differs from a bad API. It contains examples using Fluent results and I had some good feedback from colleagues. I would be flattered if you could append it to your resource section in the readme. If not, feel free to close!
https://betterprogramming.pub/how-and-where-to-handle-exceptions-while-maintaining-a-great-api-e3df6c084746

Just saw that it is behind a paywall ... Won't put that into your repo. Sorry for the noise.


Ability to directly get / set the underlying Rule RemoteAddresses and LocalAddresses string

Is it possible (or could it be possible) to directly get/set the underlying Rule's RemoteAddresses and LocalAddresses properties through WIndowFirewallHelper?
I understand the entire objective is to safely wrap these properties using arrays of IAddress, but it would be nice to be able to get/set the raw string properties if for no other reason than debugging issues like my previously posted issue trying to assign a single IPV6 address.
EG:
var mystring=objRule.RemoteAddressesString;
and
objRule.RemoteAddresses=mystring;

I just made all rule's UnderlyingObjects public. Update your NuGet package.
var mystring = objRule.UnderlyingObject.RemoteAddressesString;
objRule.UnderlyingObject.RemoteAddressesString = mystring;
You might need to make a proxy form the NetFwTypeLib COM object.

This is most helpful for debugging issues like the recent IPV6 problem.
A little bit cumbersome figuring out the type of the underlying object, but in general this is fine.
Most everything should be done with FirewallHelper rather than directly on the underlying object, but the ability to get a reference to the underlying object is an excellent new capability.


fix:error link

Signed-off-by: Qi jin<EMAIL_ADDRESS>

check you cla

Thanks for first contribution!
But this change seems wired. the origin link https://www.sealos.io/docs/getting-started/build-example-cloudimage is accessible, wondering why  change to new link, witch url is 404?


Archlinux sftp  "Received unexpected end-of-file from SFTP server"

Distribution : Archlinux
Type : START1-S
Bug :
Unable to connect on sftp with good credentials.
Step to reproduce.
Start fresh instance START1-S with archlinux os.
Connect on sftp.
try again

Solution here: https://serverfault.com/a/770547/463439
You have to change Subsystem sftp /usr/lib/openssh/sftp-server into Subsystem sftp internal-sftp


postMessage not working

Description
I want to create a way for users to send a message to the app_home, and redirect it to another channel. So the workflow I thought was:
1- say something on app_home_opened (This is already done)
2- Listen to messages sent to the app_home through the event message.app_home, and then
3- Send it to a channel (the one that the user selected when he was installing the app)
What I did to get the text from the user was:
app.event("message.app_home", async ({ event: { text }, context }) => {
  try {
    const result = await app.client.chat.postMessage({
      token: context.botToken,
      channel: "general",
      text,
    });

    console.log(result);
  } catch (error) {
    console.error(error);
  }
});

But then, I get this message on the log:
[DEBUG]  web-api:WebClient:0 apiCall('chat.postMessage') start
[DEBUG]  web-api:WebClient:0 will perform http request
[DEBUG]  web-api:WebClient:0 http response received
[DEBUG]  bolt-app ack() begin
[DEBUG]  bolt-app ack() response sent
[DEBUG]  bolt-app ack() begin
[DEBUG]  bolt-app ack() response sent
[DEBUG]  bolt-app Conversation context failed loading for ID: D016SNBV71N, error: Conversation not found

I don't know what to do here.
And another question:
I would like to know how to get the channel ID that I select here:

What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[X] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[X] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.
[X] I've searched for any related issues and avoided creating a duplicate issue.

On the second question, check out Slack Developer Tools, great for getting IDs where needed (without opening up conversations in the browser).

One thing I can mention would be https://api.slack.com/events/message.app_home was for (deprecated) workspace apps. I may not fully understand your use case yet but if your app wants to subscribe to incoming messages in DM with a user,  https://api.slack.com/events/message with im:history permission is the right event to receive.

By the way, when I try to follow this example @seratch : https://slack.dev/bolt-js/tutorial/getting-started, it gives me a warning saying that the say function needs a text field.

@pedroapfilho
I guess you're using TypeScript, right? If so, we're aware of incompatibility issues around some code snippets in docs. Checking https://github.com/slackapi/node-slack-sdk/pull/853#issuecomment-641909673 would be helpful to know the reason why the text field is required in TypeScript now.
For the specific code snippet you've mentioned, I'll improve it by adding text arg in the code. Thanks for pointing this out!
Going back to your original question here, did the above answers help you out?

Actually it was more of an architectural error of mine, I was trying to create a type of chat-bot, but it wasn’t the best fit what I really wanted. I feel like the documentation lacks some more examples. I’m thinking about creating a PR with a few examples to the repo, so it’s going to be easier to understand how Slack works. What do you think? And your ‘typescript’ boilerplate is very helpful, I hope it was easier to find.

Thanks for your interests about the contribution! The team has been discussing the best way to manage those in the long run. We're not planning to have more examples in this repository at this moment (we may change our minds in the future, though). So, if you'll publish examples (that's really amazing!), go ahead in your own repository, first 🙇
Also, I'm glad to know my project template helped you out! Providing a kind of project template can be part of the above initiative, too. Hopefully, we can realize it as early as possible.

I just wanted to put here an example that I know, that helps a lot to create new apps with the framework: https://github.com/vercel/next.js/tree/canary/examples
One thing that I would suggest too, is to create some sort of create-bolt-app, with --template tags, that take templates from the example folder. It would make WAAAY easier to create new Bolt apps.


Training Anime like ARCANE (Netflix)

Quick question:
Is it possible to train with new style like from Netflix style animation "ARCANE" I really love their rendering of face.
If possible, is it hard, does it take a long time using M1?

It would be cool indeed. I think Face Portrait v2 is close to this.
I will try to train with my own dataset and will let you know if I succeed with ARCANE pictures

@Greg8978 Have you trained the model on your own dataset and what kind of dataset is it, facial animation? Looking forward for your update.

Hey there!
I collect some images directly from the show.
I try to train the model but it took 187 hours on my ubuntu.
I did not take the time to try to make the setup on my windows computer to use GPU for trainning.

Hey there!
I collect some images directly from the show. I try to train the model but it took 187 hours on my ubuntu. I did not take the time yet to make the setup on my windows computer to use GPU for trainning.

Would you mind sharing the dataset? I have enough GPU resources for training.

Sure, have a look here

@zhanglonghao1992, if you can share the training output it would make thinks easier for me ;)

@Greg8978  I failed to train face stylization on the data set you provided. I guess the training style data should contain a large number of clear faces. At present, I plan to collect more face images in Arcane for training.

Ha ok, thanks for the feedback.

this dataset might be of interest, not exactly arcane look but artstation has a realistic stylized look https://github.com/onion-liu/aahq-dataset

I'm training one but the results are not as good so far. will let you know if I get it work.
In the mean time, here are some super-cherry-picked golden samples:

any chance that you could release the checkpoint? the results look amazing already

I'm training one but the results are not as good so far. will let you know if I get it work. In the mean time, here are some super-cherry-picked golden samples:

Did you do face alignment when training face stylization?

Face Por

Thanks, your results looks fine enough. Could you please tell me which training code you use? I have another custom dataset and want to train it too.

I've also given a try to this style. So far I've got results like this on images in the wild.

I've also given a try to this style. So far I've got results like this on images in the wild.

That's cool! Would you mind sharing you training datasets and strategies?

I'm training one but the results are not as good so far. will let you know if I get it work. In the mean time, here are some super-cherry-picked golden samples:

Those are awesome! Is that a blend with the same Z or a projection?

I'm training one but the results are not as good so far. will let you know if I get it work. In the mean time, here are some super-cherry-picked golden samples:

Hi, how many pictures did you use to train stylegan model?

Amazing work! When I train stylegan model using screenshots in anime, there are always artifacts on the face. I am troubled by the lack of appropriate data sets. I wonder how many pictures did you use to train stylegan model? I would appreciate it if you could let me know. And if it is convenient, I would like to ask if you can release the data set?

I'm training one but the results are not as good so far. will let you know if I get it work. In the mean time, here are some super-cherry-picked golden samples:

Amazing work! When I train stylegan model using screenshots in anime, there are always artifacts on the face. I am troubled by the lack of appropriate data sets. I wonder how many pictures did you use to train stylegan model? I would appreciate it if you could let me know. And if it is convenient, I would like to ask if you can release the data set?

I'm training one but the results are not as good so far. will let you know if I get it work. In the mean time, here are some super-cherry-picked golden samples:

Awesome! Is this a StyleGAN result then do a animegan training?

Sure, have a look here

The transfer has expired, can you share it again?

It's easy to reproduce, just collect arcane video(s) (I used youtube trailers), then
ffmpeg -i Arcanevideo.mp4  fps=0.5 arcane%d.jpg

@bilal2vec These are super cherrypicked samples. The model's really fragile at the moment and have obvious normalization related artifacts for most of the images. I'm testing out some other techniques and trying to find a sweet spot between the quality and robustness.
@zhanglonghao1992  I didn't but it should help
@Sxela It's from a distilled pix2pix model
@rainsoulsrx @tinapan-pt  I've used about 500 images but it contains many duplicates cause it was taken from videos with limited characters
@chenhk-chn yup

@bryandlee Do you use pair data for training?

@bilal2vec These are super cherrypicked samples. The model's really fragile at the moment and have obvious normalization related artifacts for most of the images. I'm testing out some other techniques and trying to find a sweet spot between the quality and robustness.
@zhanglonghao1992 I didn't but it should help
@Sxela It's from a distilled pix2pix model
@rainsoulsrx @tinapan-pt I've used about 500 images but it contains many duplicates cause it was taken from videos with limited characters
@chenhk-chn yup

So besides these super cherrypicked samples, would  you please share some normal examples?

@zhanglonghao1992 yup
@rainsoulsrx  https://fragrant-chauffeur-53f.notion.site/Failures-d701a060e52046188b45823f56589093
It's pix2pixHD network with instance norm and I suspect that the black blobs are something similar to the "droplet artifact" in stylegan1.

@zhanglonghao1992 yup
@rainsoulsrx https://fragrant-chauffeur-53f.notion.site/Failures-d701a060e52046188b45823f56589093 It's pix2pixHD network with instance norm and I suspect that the black blobs are something similar to the "droplet artifact" in stylegan1.

Still not bad~
However in my experiment, I find it's hard to retain the 'style' when I train stylegan with less epoch, on the other hand, if I train stylegan more epoch, the 'style' become stronger, but I got more atifacts which will cause unpleasant data pair. just like below. So how do you make this balance?

@zhanglonghao1992 yup
@rainsoulsrx https://fragrant-chauffeur-53f.notion.site/Failures-d701a060e52046188b45823f56589093 It's pix2pixHD network (not the animegan architecture) with instance norm and I suspect that the black blobs are something similar to the "droplet artifact" in stylegan1.

You can try batchnorm with spectral norm, seems to work okay for me

@Sxela Thanks for the suggestions. I've actually tried a number of architectures including BN/SN but didn't really get the quality that I wanted. +) I just found out that you've trained a nice model. If anyone interested in arcane style, check this out Sxela/ArcaneGAN

@Sxela Thanks for the suggestions. I've actually tried a number of architectures including BN/SN but didn't really get the quality that I wanted. +) I just found out that you've trained a nice model. If anyone interested in arcane style, check this out Sxela/ArcaneGAN

Thank you for mentioning, it means a lot to me! Can't wait to see your model from DeepStudio, it looks much better in my opinion, its temporal consistency is just outstanding.

@Sxela  No problem! Great work man

练，如果我成功使用 ARCANE 图片，我会通知您

May I know how to train?

DeepStu

May I know how to train?

@zhanglonghao1992是的
@rainsoulsrx https://fragrant-chauffeur-53f.notion.site/Failures-d701a060e52046188b45823f56589093 这是具有实例规范的 pix2pixHD 网络（不是 animegan 架构），我怀疑黑色斑点类似于 stylegan1 中的“液滴工件”。

You do not have access to Bryan's Notion. Please contact an admin to add you as a member.


Add support of SAMPLE ClickHouse clause

Coverage increased (+0.005%) to 98.425% when pulling f78214334659f1cf9518135a392ec71d837ec77d on Kadiska:gt/clickhouse_sample into 30574f997c80851f7e940ad09a63e14a98871dd3 on kayak:master.

Could you pull from the master branch so that the newest CI test kick in?

done, it has been rebase to master


enterprise wallet / General Information

One can remove
Blockckain / Tezos  and address and keep only DID
Website : Talao.io and email contact should be moved to another place (above didkit ???) in order to avoid confusion with the information about the enterprise itself

To do:

Remove website and contact email from General information page.
If DID method is web: don't display the following field in General information page: blockchain, address and copy address button.


Write tests with guesswork

We can do it! Write tests for this library written with this library!

Doing it!


Auto-fill the MAKE variable.

We should add MAKE as a special key for the Vars struct (much like .RECIPEPREFIX and set it to however the current program was invoked.

You can assign me this one, I should have time to work on it this week

Awesome; thank you! Let me know if you have any questions.

I got around to working on this one, I will implement it as the GNU docs specify, I will also add a rudimentary impl of the MAKEFLAGS variable.


Fix replication event for removing metadata

It seems like the removal of metadata in the Swarm.leave call
is not properly replicated.
The handle_replica_event for :update_meta is always merging
the local with the remote meta data which means existing
keys always stay in the meta dictionary. Thus, the removal
of keys is never taking affect on remote nodes.
Broadcasting specific add_meta and remove_meta replication
events instead of a more general update_meta event which gives
remote nodes more context to apply the correct state.
Updated also adding metadata, even though it was working, but
it might be better to keep these in-sync.

Great catch! I don't know why that never occurred to me, but yeah, a single update event is not sufficient for sure. I'll review this ASAP


fixed the broken import statement in error.js

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Error msg and fixing compilation for Windows and MAC

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@darshan-sj Please reopen when you pick this work again.


How to catch errors

I have deletePlace function in my project. When I send DELETE without authorization to some resource server I've got the error 401, how can I catch errors in amygdala?
this.deletePlace = function(place_id) {
	return store.remove('places', {
		'id': "/" + place_id
	}).done(function() {
		ee.emit('change');
	})
}

Error example caused by lack of authorization
DELETE http://localhost/api/v1/places/qx9143jmj7 401 (Unauthorized)
XHR finished loading: DELETE "http://localhost/api/v1/places/qx9143jmj7"
Uncaught #<XMLHttpRequest>

@debian-sh you can add an extra callback on done():
this.deletePlace = function(place_id) {
    return store.remove('places', {
        'id': "/" + place_id
    }).done(function() {
        ee.emit('change');
    }, function(xhr) {
       alert('error');
    })
}


[W5.11][T09-1]Saif Uddin Mahmud

No JUnit tests were added due to lack of understanding.

Looks good to me!

However, you did not follow proper OOP procedure where you only modified some of the higher level functions. And there was also no Junit testing to ensure that your modification is fully working.

@Psyf

Write commit messages in recommended format, i.e. rename instead of renamed.
Some comments added. Please close the PR after reading them.


tests: disable slow bench test

tombstone_scan is slow to start and it doesn't have to be run
every time.

Can we rewrite it using batch op? I guess most time is spent on writing.

I guess it can be done. But write 100,000 keys to disk every time tests are run is not a good idea.

can we still use cargo bench to run it explicitly?

You can use cargo bench -- --ignored to run it.


/b option not documented

The /b (for bindings?) option is presented in an example, but is not otherwise documented on this page.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e1a5d57a-9b6d-97b5-68ef-596ea616ab3f
Version Independent ID: 74c36f72-cfee-5fbb-e410-fb4cb7449cdc
Content: netcfg
Content Source: WindowsServerDocs/administration/windows-commands/netcfg.md
Product: windows-server
Technology: manage-windows-commands
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Fair enough, there is also a typo and slightly bad choice of word connected to that typo as well. The page should have had more UNIX-like descriptions of the parameters, including mnemonic phrases to aid in associating the parameters with their description. I will open a PR to make some adjustments and suggest a row in the table for the /b paramater.

Please note that running the netcfg command on Windows 10 version 1909 will list more options in the examples than those present on this page, as well as using a dash (-) character instead of the slash (/) as the indicating character for that parameter.

Microsoft Windows [Version 10.0.18363.693]
(c) 2019 Microsoft Corporation. All rights reserved.

C:\WINDOWS\system32>netcfg /?
netcfg [-v] [-winpe] [-l <full-path-to-component-INF>] -c <p|s|c>
       -i <comp-id>

    -winpe installs TCP/IP, NetBIOS and Microsoft Client for Windows
           preinstallation environment
    -l     provides the location of INF
    -c     provides the class of the component to be installed (p == Protocol,
           s == Service, c == Client)
    -i     provides the component ID

    The arguments must be passed in the order shown.

    Examples:

    netcfg -l c:\oemdir\myprot.inf -c p -i myprot

        Installs protocol 'myprot' using c:\oemdir\myprot.inf

    netcfg -c s -i MS_Server

        Installs service 'MS_Server'

OR

netcfg [-v] -winpe

    Example:

    netcfg -v -winpe

        Installs TCP/IP, NetBIOS, and Microsoft Client for Windows
        preinstallation environment

OR

netcfg [-v] -q <comp-id>

    Example:

    netcfg -q MS_IPX

        Displays if component 'MS_IPX' is installed

OR

netcfg [-v] -u <comp-id>

    Example:

    netcfg -u MS_IPX

        Uninstalls component 'MS_IPX'

OR

netcfg [-v] -s <a|n>

    -s  provides the type of components to show (a == adapters,
        n == net components)

    Example:

    netcfg -s n

        Shows all installed net components

OR

netcfg [-v] -b <comp-id>

    Example:

    netcfg -b ms_tcpip

        Shows binding paths containing 'MS_TCPIP'

OR

netcfg [-v] -m

    Example:

    netcfg -m

        Outputs the binding map to NetworkBindingMap.txt in the current directory.
        -v will also display the binding map to the console.

OR

netcfg -d

    Example:

    netcfg -d

        Performs a cleanup on all networking devices.
                This will require a reboot.

OR

netcfg -x

    Example:

    netcfg -x

        Performs a cleanup on networking devices, skipping those without physical
        object names.
                This will require a reboot.

General Notes:
  -v    Run in verbose (detailed) mode
  -?    Displays this help information

C:\WINDOWS\system32>_

Also, I don't know if this is indicative of this command being updated closer to what will be available on Windows Server 2019 (I don't have access to WS 2019 yet).

(Ref. #3997 )


Getting error: TypeError: Cannot read property 'replace' of undefined

Hello!
Getting the error while reading this file.
This file is not damaged: opens correctly in Numbers (version 5.3 (5989)). Also correctly parsed by other libraries.
demo repository (file included): https://github.com/vishenkov/xlsx-error-demo
sandbox: https://codesandbox.io/s/github/vishenkov/xlsx-error-demo
file: https://drive.google.com/file/d/1lSR5ZLxQkm0OYLxveYYPVXbpz9t5SExw/view?usp=sharing
/xlsx-error-demo/node_modules/xlsx/xlsx.js:10015
                comment.t = rt.t.replace(/\r\n/g,"\n").replace(/\r/g,"\n");
                                 ^

TypeError: Cannot read property 'replace' of undefined
    at /xlsx-error-demo/node_modules/xlsx/xlsx.js:10015:20
    at Array.forEach (<anonymous>)
    at parse_comments_xml (/xlsx-error-demo/node_modules/xlsx/xlsx.js:10003:65)
    at parse_cmnt (/xlsx-error-demo/node_modules/xlsx/xlsx.js:15094:9)
    at parse_comments (/xlsx-error-demo/node_modules/xlsx/xlsx.js:9945:16)
    at parse_zip (/xlsx-error-demo/node_modules/xlsx/xlsx.js:20021:19)
    at read_zip (/xlsx-error-demo/node_modules/xlsx/xlsx.js:20249:9)
    at readSync (/xlsx-error-demo/node_modules/xlsx/xlsx.js:20314:69)
    at Object.readFileSync (/xlsx-error-demo/node_modules/xlsx/xlsx.js:20328:9)
    at Object.<anonymous> (/xlsx-error-demo/index.js:3:23)

The underlying issue has been resolved in version 0.15.0

Thanks


VS-code server web view, infinite loading jupyter notebook.

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: code-server: v4.18.0
Code: 1.83.1
OS Version: Browser: Mozilla/5.0 (Windows NT 10.9; Win64; x64) Chrome/<IP_ADDRESS>

Steps to Reproduce:

Install latest vscode server on ubuntu with curl -fsSL ....
Launch with code-server bind to 8081 and auth password
Load vscode in browser
Install jupyter extension, python extension
Either open an existing notebook or click new file jupyter notebook
The bar at the top loads infinitely, kernel dot shows active. Able to see and connect to kernels. Buttons for + code and + markdown, run all are available. Able to convert notebook to .py and execute and see result.
unable to view the jupyter cells, main code window is blank white with inifinte loading at top in a blue bar.
Tried uninstalling and reinstalling vs-code and individual extensions and disabling all others.
Tried downgrading each extension.
A message that extension activation has failed occurs.

Expected:
The normal cells with interactive functions should appear.

***: Very similar issue to https://github.com/microsoft/vscode-jupyter/issues/7352 however this is using the web browser and vs-code server.

@SamoraHunter may I ask where do you get code-server?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


policy: Fix cilium policy trace output when only deny rules are applied

Currently, deny rules are not counted in *Repository.GetRulesMatching().
Therefore, this function returns false even though a rule is applied if it is a deny rule.
This causes a wrong output of policy trace commnad in some cases.
This PR makes this function to count ingress/egress deny rules.
Also, update the e2e test case for checking policy trace works correctly when there is only a deny rule exists.

test-me-please

Both team review requests are covered and all required tests are passing. Marking ready to merge.


Feature/cu kk7a7m player controller navigation

Animation bugs fixes
Code improvements
NavMesh implemented (dash disabled because it's not implement with navmesh yet)

Task linked: CU-kk7a7m Player controller implement navigation movement


convert tensor type to match for torch.matmul

fixes:
  File "/opt/conda/lib/python3.10/site-packages/peft/tuners/lora.py", line 686, in forward
    result = super().forward(x)
  File "/workspace/alpaca_lora_4bit/autograd_4bit.py", line 63, in forward
    out = mm4b.matmul4bit(x, self.qweight, self.scales,
  File "/workspace/alpaca_lora_4bit/matmul_utils_4bit.py", line 109, in matmul4bit
    output = _matmul4bit_v1_recons(x, qweight, scales, zeros)
  File "/workspace/alpaca_lora_4bit/matmul_utils_4bit.py", line 81, in _matmul4bit_v1_recons
    output = torch.matmul(x, buffer)
RuntimeError: expected scalar type Float but found Half

Thanks for your fixing this! Already applied another patch to the file


Add pagination for the GetJobStderr command

The current implementation does not allow to implement tail -f since pagination is not supported. I propose to add several modifications to the command.

Add parameters offset and limit to the comand options. They will specify read offset and maximum number of bytes to read.
Add annotations end_offset and total_size to the command response annotations. They will tell the index of the last byte read and the total size of the stderr. Example of the response annotations in here: https://github.com/ytsaurus/ytsaurus/blob/b8937b8484b25935074668a769d7425aa08be467/yt/yt/client/driver/table_commands.cpp#L840

The implementation of the GetJobStderr is here. Note that there are two implementations: one for the running job (FromNode) and one for the finished job (FromArchive).

Could we close this issue?


[numpy] Fix less/greater bug with scalar input

Description

FFI greater
Fix a bug in ufunc op: less/greater/less_equal/great_equal
described in #18594

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@mxnet-bot run ci [centos-cpu, unix-cpu]

@mxnet-bot run ci [centos-cpu, unix-cpu]

@mxnet-bot run ci [centos-cpu, unix-cpu]


[easy init] set allowed values for location parameter to define the known valid locations for container-apps env

Design and implement one strategy to limit the location for a deployment based on the resources discovered during easy init

We also need to handle this when Postgres is detected, since flexible server has a very limited set of regions - https://learn.microsoft.com/en-us/azure/postgresql/flexible-server/overview#azure-regions

If we end up hardcoding the list, we'll continue to have to stay in sync with allowed regions by the service as the service expands their footprint. If there's an API exposed for allowed regions, that may be the best option.

@weikanglim do we have any API exposed for allowed regions based on service?

@rajeshkamal5050 No, it doesn't exist currently.


Add values to the bar plots for better visibility

Right now we quote numbers from Fig.2, but some of the values are hard to read off, in particular those close to 1.0 because it's very close to the x-axis. By adding the values on top, the issue can be avoided.

The bar label in the third plot overlaps with the title.

@leofang @asmeurer I'll add the values manually. The PDF and SVG currently in the figure are generated after post-processing in an SVG editor.

I would prefer to avoid that if possible. We'd have to redo it if I need to rerun the branchmarks (like for #15).

Then you need to find a way to reliably add the a-d labels, so that they can be referred to individually.

If we move up the max from 80 to, say, 85 or 90, the label should fit in. Shall we do that in favor of (simple) automation?

If we move up the max from 80 to, say, 85 or 90, the label should fit in. Shall we do that in favor of (simple) automation?

Done in c714e23.

Then we need to find a way to reliably add the a-d labels programmatically, so that the bar charts can be referred to individually.

I'm checking how this works
https://matplotlib.org/stable/gallery/text_labels_and_annotations/label_subplots.html
will send a follow-up PR once I figure out.

Thanks, @leofang!


1.40 Front rendering has changed

With the new version on Mac 1.40 i've noticed that font rendering and background colors changed, screenshot here:
Left is 1.40 - Right is 1.39
https://ibb.co/g9hVxhr
1.39
https://ibb.co/5rPVL20
1.40
https://ibb.co/L58Qj0h
I much prefer how the fonts and colors looked on the 1.39 version. Is this a bug? Or you removed the subpixel (if this is how it's called) font rendering?

/duplicate - see #84154


Problem-matchers for error messages that span multiple lines

VSCode already supports multiline problem matchers where the error as a whole is split over multiple lines, so long as no component of the error is split, e.g.
/home/ljw/code/file.vb:(15,3)-(16,8)
Error: unexpected type

However, it doesn't support the case where the error MESSAGE is split over multiple lines. It's only able to pick up an error message from one of the lines. E.g.
/home/ljw/code/file.vb: line 15, cols 3-5
error: unexpected type
   You provided "int" but "string" is expected
   Suggestion: try CStr(x) or turn Option Strict Off

In issue #9635, @AshleyYakeley suggested the idea of an "append" property. You'd write a problem-matcher like the following. The meaning of "append" is that each time it's encountered, it adds a new line of content to the error message string.
{ "regexp": "^(.*): line (\d+), cols (\d+)-(\d+)$", "file":1, "line":2, "col": 3, "endCol": 4},
{ "regexp": "^(error): (.*)$", "severity":1, "appendMessage":2},
{ "regexp": "^   (.*)$", "appendMessage":1}

Such an "append" property would have solved all the scenarios that folks have posted on #9635, as well as previous issues. I think it's a beautiful. (Pragmatically, there's zero need for anything other than "message" to be appended).
@dbaeumer commented (before he'd seen the append idea) that he'd need to invent a whole json programming language to solve declaratively the problem of multi-line message strings. @alexr00 therefore closed the existing issue, saying it would be solved by programmatic task matching #59337.
I think programmatic problem-matching is a much bigger hammer than is warranted here. I'm therefore opening a new task. Declarative problem-matchers in tasks.json are one of my top favorite features of VSCode. I love how they empower everyone to be able to glue together their favorite tools without having to code extensions to VSCode. I think that programmatic matchers are an unnecessarily high barrier-to-entry for problem matchers. I think that having to write (install, deploy, maintain) an extension will be far too high a burden for rank-and-file developers.
I'm writing this task in the hope that @dbaeumer  and @alexr00 might reconsider their decision on the "append" flag... :)
For what it's worth, here are two more languages that need multi-line messages for their problem matchers. Rust:
warning: unused variable: `decls`
  --> hphp/hack/src/facebook/hh_linearize/main.rs:69:9
   |
69 |     let decls = decl_rust::direct_decl_parser::parse_decls(
   |         ^^^^^ help: if this is intentional, prefix it with an underscore: `_decls`
   |
   = note: `#[warn(unused_variables)]` on by default

Ocaml:
File "/data/users/ljw/fbsource/fbcode/hphp/hack/src/hh_client.ml", line 27, characters 9-24:
Error: Unbound value Random.self_int
Hint: Did you mean self_init?

cc. @o-lim, @massimocode, @indiejames, @jpihl, @cmeiklejohn who all also reported needing multiline messages-strings out of their problem matchers.

Another language example, fwiw—based on the examples above I think stylus' compilation errors might get some utility from this
Example Error Outputs:
/usr/local/lib/node_modules/stylus/bin/stylus:685
              throw err;
              ^
ParseError: ./invalid-parse.styl:3:19
   1|  body
   2|     form input
   3|         == padding 5px
------------------------^

illegal unary "==", missing left-hand operand

    at Parser.error (/usr/local/lib/node_modules/stylus/lib/parser.js:259:11)
    at Parser.equality (/usr/local/lib/node_modules/stylus/lib/parser.js:2009:23)
    at Parser.typecheck (/usr/local/lib/node_modules/stylus/lib/parser.js:1990:21)
    at Parser.logical (/usr/local/lib/node_modules/stylus/lib/parser.js:1977:21)
    at Parser.ternary (/usr/local/lib/node_modules/stylus/lib/parser.js:1961:21)
    at Parser.negation (/usr/local/lib/node_modules/stylus/lib/parser.js:1953:17)
    at Parser.expression (/usr/local/lib/node_modules/stylus/lib/parser.js:1932:24)
    at Parser.stmt (/usr/local/lib/node_modules/stylus/lib/parser.js:891:25)
    at Parser.statement (/usr/local/lib/node_modules/stylus/lib/parser.js:750:21)
    at Parser.block (/usr/local/lib/node_modules/stylus/lib/parser.js:938:21)

/tmp/zshz10JcE

/usr/local/lib/node_modules/stylus/bin/stylus:685
              throw err;
              ^
ParseError: /tmp/zshz10JcE:4:1
   1|
   2| div
   3|   invalid thing "
   4|
------^
   5|

expected "indent", got "outdent"

    at Parser.error (/usr/local/lib/node_modules/stylus/lib/parser.js:259:11)
    at Parser.expect (/usr/local/lib/node_modules/stylus/lib/parser.js:287:12)
    at Parser.block (/usr/local/lib/node_modules/stylus/lib/parser.js:918:12)
    at Parser.selector (/usr/local/lib/node_modules/stylus/lib/parser.js:1640:24)
    at Parser.property (/usr/local/lib/node_modules/stylus/lib/parser.js:1583:51)

/duplicate #9635

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


fixed russian translation for css text article

This article seems to be translated via google translate or something similar.
Part of the article wasn't translated.
Translated the missing parts.
Made the article more human-readable.
Fixed typos.
Fixed styles.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@focuse Thank you for  creating this PR.  However, since it was created, the underlying document has changed.  Can you please review and resolve the current merge conflicts?  It could be the changes to the underlying document are so significant that your changes are no longer relevant.  If the latter is the case, it might be easier to just close this PR and create a new PR with any changes you stillOtro ejemplo con un atributo href feel are important to make.
If you need any assistance or have questions about resolving the merge conflicts, please reply back or contact someone in the Contributors Room on Gitter.
Thank you.

We are restructuring a lot of the code and learning resources referenced in your pull request, so for now I am going to go ahead and close this pull request. Thanks again for opening this pull request and for contributing to the community.


wrote the code for divide by zero error.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request, but this adds unnecessary information and isn't properly formatted. So we have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


In A1 the player is not sent to next scene

Repro
Start game from preloader
Reach A1
Reach the teleporter
Actual Result
nothing happens
Expected Result
Get sent to the next level

The way how the backend function in future build will be very diffrent from the build you've tested on.
Please reverify the gameflow in next patch.

the player is new sent to the next scene but instead of the next level it is the main menu

Issue fixed in commit 102
please reverfiy


Consider not exporting >4 joint vertex influences.

If it is reasonably possible to (non-destructively) remove extra vertex influences during export, perhaps the exporter should (a) do that automatically, or (b) have an opt-in setting to include additional influences. Almost no realtime engines support >4 influences, and this is a common source of errors for users who don't know they need to apply Blender's Limit Total function.

@donmccurdy what do you think would be the proper way to handle this. Not adding more than 4 influences would be destructive, but just issuing a warning might get overlooked by the user.

The destructive case I definitely want to avoid would be modifying the model during export, in a way that persists in the editor. Stripping additional influences during export, unless the user checks a box to enable >4, might be OK I think.
Is there any way for the user to see warnings, without opening Blender via CLI? That would be helpful if so.

there now is an option in the exporter to allow more than 4 influences. Currently the user is still only warned via cli, see #89

Looks good, thanks!


lodash: changed _.isPlainObject() method

https://lodash.com/docs#isPlainObject
https://lodash.com/docs#_ (description of the chainable wrapper)

@chrootsu thanks mate!


Support for multiple xwayland instances

Would be better to let compositors start as many xwayland instances as they want.
I don't think a lot of things are blocking this. We probably just need to make the compositor set DISPLAY instead of setting it automatically. Since the compositor already sets WAYLAND_DISPLAY it pobably makes more sense anyway.

Out of curiosity what is the use case for this? Sandboxing?

Yeah

Could this also be used to fix the annoying blurry upscale for X11 windows?

No.


How to add custom logic to the navbar?

Hi,
I am trying to experiment with Aurelia and want to add some custom logic to the nav-bar inside the skeletons. For example I want to add a form to login to my app inside the navigation bar, or I want to add a Login or Logout link based on authentication status.
However, when I add a nav-bar.js file, this does not get picked up. (e.g. I create a submit() function there and call it from the <form submit.delegate="submit()">.  I keep getting the following error: aurelia-binding.js:1926 Uncaught Error: submit is not a function
I read through the following guide but it doesn't really help:
https://gist.github.com/kristianmandrup/e1099f54bbb7f6968af7
In nav-bar.html let extract the block for <div class="navbar-header"> into its own component. Since it belongs to Navbar, we create a subfolder src/navbar and add a nav-header.html file there.
...
We make router available inside the new component by binding it via router.bind="router". We now get an error about a missing file nav-header.js. Aurelia expects every component to consist of a html file with the template (layout) and a js file of the same name which contains a class with the functionality.

We need to create a file src/navbar/nav-header.js

Am I missing something?
Thanks for the help.

Github is for bugs and features. The best place to get help is on gitter or SO.

@jseminck - have you found a solution to this?


linux/mman.h include?

Secure arena (crypto/mem_sec.c) includes linux/mman.h.
Issue is that header linux/mman.h does not exist on Android for API < 21.
It seems to me on library builds fine without warnings.
What about to remove "#  include <linux/mman.h>" from crypto/mem_sec.c?

That include is necessary to detect the definition of MLOCK_ONFAULT (if supported by the kernel in question); removing the include would remove support for MLOCK_ONFAULT.

kaduk wrote:

That include is necessary to detect the definition of MLOCK_ONFAULT (if supported by the kernel in question); removing the include would remove support for MLOCK_ONFAULT.

Ok.
On Android MLOCK_ONFAULT  is not defined in platform headers.  In
"unified headers" is defined for some platform.
The I guess it could be excluded only for Android.

How old is Android 21?
We do not want to clutter our code with ifdef for old platforms.

Rich Salz wrote:

How old is Android 21?
We do not want to clutter our code with ifdef for old platforms.

For instance API level 19-20 correspond to Android KitKat 4.4 - 4.4.4
and 4.4.4 is still supported (security patches).

Is there a no-secure-memory configure option?

Is there a no-secure-memory configure option?

Nope.  But that could certainly be arranged.
Another way to handle it could be to guard with a check for __ANDROID__ and whatever macro there might be that indicated the API level.  Perhaps with an extra Android-specific include?

Richard Levitte wrote:

Another way to handle it could be to guard with a check for __ANDROID__ and whatever macro there might be that indicated the API level.  Perhaps with an extra Android-specific include?

Most of functionality is obtained from <sys/mman.h>. It seems to me on
Android <linux/mman.h> could be excluded unconditionally.

maybe <sys/mman.h> works in general ?
Also the linux man page says:
       #include <sys/mman.h>

       void *mmap(void *addr, size_t length, int prot, int flags,
                  int fd, off_t offset);

Bernd Edlinger wrote:

maybe <sys/mman.h> works in general ?
yes and no : Please see previous post - linux/mman.h is to found
definition of MLOCK_ONFAULT.

MLOCK_ONFAULT is currently 0x1 do you expect linux to ever change that ?

Well, that's not too hard to solve, is it?
: ; cc foo.c
: levitte@lapdog:~
: ; ./a.out
1
: levitte@lapdog:~
: ; cat foo.c
#include <stdio.h>
#include <sys/mman.h>
#ifdef linux
# include <linux/mman.h>
#endif

int main()
{
  printf("%d\n", MLOCK_ONFAULT);
}

(perhaps replace #ifdef linux with something that is better tuned for Android's different APIs)

Bernd Edlinger wrote:

MLOCK_ONFAULT is currently 0x1 do you expect linux to ever change that ?
Wrong direction. Value itself is not important. Please read again post
from "kaduk".

If you use gcc the following can be used to find candidates:
gcc -x c -g3 -E /dev/null |grep define
so one of these could be used to detect "linux":
#define __gnu_linux__ 1
#define __linux 1
#define __linux__ 1
#define linux 1
#define __unix 1
#define __unix__ 1
#define unix 1

Don’t go there.  ☺

Maybe ifdef ANDROID will work ?

I mean don't try to add our own define for the MLOCK magic number :)
We can always add -DOPENSSL_SYS_ANDROID to our config definitions.

but is andoid garanteed to never have any linux/mman.h ??

(But I know @dot-asm prefers to use system-defined values if possible)

Richard Levitte wrote:

Well, that's not too hard to solve, is it?

It is solved in my local repository as:
#if !defined(ANDROID)
include <linux/mman.h>
#endif
linux and unix are compiler defines declared for android targets as
well.

is this gcc?

Does it matter?  Android is a phone O/S and MLOCK_ONFAULT seems not necessary there.

@levitte  we already use that define:
crypto/dso/dso_dlfcn.c:        defined(__ANDROID__)
crypto/async/arch/async_posix.h:    && !defined(__ANDROID__) && !defined(__OpenBSD__)
include/openssl/rand.h:# if defined(__ANDROID__) && defined(__NDK_FPABI__)

Cool, so something like this?
# if defined(linux) && !defined(__ANDROID__)

Richard Levitte wrote:

Cool, so something like this?
# if defined(linux) && !defined(__ANDROID__)

inclusion  is already in # if defined(OPENSSL_SYS_LINUX)
may be  defined(linux) is duplicate ?

Yes, linux is duplicate, and __ANDROID__ is probably enough.
Generally I think __linux__ or __linux are more portable than linux
because without the underscore it is just an ordinary name that could
have no special meaning on another O/S.

Resolved in #5584.


Setting the highlight parentheses style: what I am I doing wrong

I am trying to define a custom style for the HighlightMatchingBracketProcessor. Here is a simplified version of what I am doing:
from prompt_toolkit.shortcuts import prompt
from prompt_toolkit.layout.processors import HighlightMatchingBracketProcessor
from prompt_toolkit.token import Token
from prompt_toolkit.styles import style_from_pygments

from pygments.styles.tango import TangoStyle

style_extra = {
    Token.SearchMatch:   "bg:#0000ff", # blue
}

prompt(">>> ", extra_input_processors=[HighlightMatchingBracketProcessor()],
    style=style_from_pygments(TangoStyle, style_extra))

It isn't working. The highlighted parentheses are still gray, instead of blue. Am I doing something wrong or is there a bug here?

Oh derp, it's the wrong token (read the wrong part of the processors.py file). It should be Token.MatchingBracket.Other.


Dropdown/autocomplete option selection fails when using validation and auto-animate

Reproduction
https://formkit.link/ffef849e7595eb250f73bd8abacea5a6
Describe the bug
Seems the validation-visibility="live" on Dropdown and Autocomplete input types clashes with auto-animate's functionality, preventing the option you clicked from the list from being selected on the first attempt. Once the error has been displayed, selecting an option again does work.

Environment
• OS: MacOS
• Browser: Chrome
• Version 120.0.6099.216

@andrewIAED Appreciate the reproduction. We'll take a look at this.

@andrewIAED This is fixed at v0.121.1 of FormKitPro.


Created Apply Page

Added Application forms, requirement documents, and a picture.

In the future, please make changes to a branch inside this repo instead of from a personal fork. That way we can easily see your changes by running it ourselves.


frontend: fix progress ring background in darkmode

Background of the progress ring was white in darkmode. Changed to be dark gray.

current master:

with this commit (ffd255cbc71614d8a346df60b2cf0a676d48878b)

Ideally we had a darker gray that is still lighter than the dark background. But we don't use it yet and I didn't want to introduce a new color for this.
cc @shonsirsha @jadzeidan


Async

Closes #30

@dragonnn I hope you don't mind I took your commits. Do let me know if you do!

No problem with that! Glad it helps and it is gonna be merged into display-interface. I recently didn't had much time to return to my project with uses it so it got sidetracked.

Replaced by #37.


Doesn't Work, please fix!

The timer doesn't count down. It stays at what it was set at, please fix this! I love this plug-in idea, if only it was functional!

Hold shift, then right click the interface
Came here after not being able to get it working myself and poked around a little. It looks like a lot of RL interfaces have additional options when right clicking while shift is held 🙂
If the project owner sees this comment, I wonder if it's worth adding it as a FAQ in the README?


FeatureReq: RUN function should show same file list as INCLUDE does

Describe the solution you'd like
INCLUDE function displays a list of files to reference. The RUN function does not but should so the same thing. It's syntax is the same except it immediately executes the code on those COD scripts.
Describe possible alternatives you've considered
N/A
Additional context
N/A

@DyoungMCWI, I did not rebuild the VLX, but please test this when you get a chance and if you agree it is corrected, then close this issues.

Looks to be resolved. Nice work. Apologies for the delay in testing. Some issues with compiling the VSIX which I was able to finally resolve on my new system.


Benchmarking issue141

Reference Issues/PRs
Initial PR dealing with issue #141.
What does this implement/fix? Explain your changes.

added additional unit tests for evaluator
introduced functionality for calculating the runtime for fit and predict methods
added some docstrings

Does your contribution introduce a new dependency? If yes, which one?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors.
[ ] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[ ] I've added unit tests and made sure they pass locally.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

Any other comments?

Hi Viktor, a quick note: we changed our git workflow (see #360), we're now creating branches from master and merge into master directly, rather than going through dev.

Thanks, I missed this. I will create a new PR.


HttpServletResponse::sendRedirect(String) broken in JSPs (EE10)

I believe we found another bug while testing our upgrade to EE10 (Java 21) on App Engine Standard.
If we attempt a redirect from a JSP (via HttpServletResponse::sendRedirect(String) like response.sendRedirect("/some-path");), it works properly locally but crashes when running via our deployed App Engine application (with an error like: Request failed: Unexpected Error: java.io.IOException: written 2 > 0 content-length). When sending a redirect from a non-JSP HttpServlet, there are no issues; so it appears to be an issue specific to handling redirects from JSPs.
This can be reproduced with my sample web app: https://github.com/MuffinTheMan/appengine-standard-ee10-sample/tree/3-redirects (see the README.md for instructions--I have been testing with a custom runtime built from this commit: https://github.com/GoogleCloudPlatform/appengine-java-standard/commit/0f190e891a5f8f76b24fa8e46fca64ed15e544a1 due to https://github.com/GoogleCloudPlatform/appengine-java-standard/issues/316).
From the README:
Broken

Navigate to /redirect-test

Local App: redirected to /jsp
Deployed App: the browser displays an error like Request failed: Unexpected Error: java.io.IOException: written 2 > 0 content-length

Working

Navigate to /redirect

Local App: redirected to /jsp
Deployed App: redirected to /jsp

I assume the runtime code used is from head, right?

@ludoch the latest commit in my local version from which I had built the jars was https://github.com/GoogleCloudPlatform/appengine-java-standard/commit/0f190e891a5f8f76b24fa8e46fca64ed15e544a1
However, I just checked out the latest appengine-java-standard:main and built new jars to test. Same issue with the latest.

I think this is a bug in Jetty 12 EE10, I will open a PR to fix.


Update Glyphs to 2.2.2-822

Thank you for the contribution. It was merged directly as 657eba104e96356c62a0bb83f8a57a878199bce7 to keep commit history cleaner. Your contribution is still credited to you.


Improve Issuer pub key discovery, while verifying an sd-jwt-vc

Currently, library expects that the issue of an sd-jwt-vc uses one of the three supported methods:

SD-JWT-VC issuer metadata
X.509 Certs
DID

Perhaps, the first two methods are not mutually exclusive.
This means that an issuer may choose to:

Have as iss claim an HTTPS URI and optionally a kid claim in the JWS Header (enabling SD-JWT-VC Issuer metadata), and at the same time
Have an x5c claim in the JWS header (enabling X509 Certs)

If that's the case, library should

stop treating like an error the above condition (where both methods are eligible)
allow caller to indicate a preference in which method to use.

This is related to https://github.com/eu-digital-identity-wallet/eudi-srv-pid-issuer/issues/247


Docs: Contributor's guide and spec-template

Summary of the Pull Request
This PR contains the first draft of our contributor's guide and spec template.
References
N/A

PR Checklist

[X] Partially implements #890
[X] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed/ N/A
[X] Requires documentation to be updated
[X] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #890

This will also resolve #750 and with a little additional language #865


Update systemd.md - add EXTRA_FILESYSTEMS

Hi, you should also update now systemd.md to reflect EXTRA_FILESYSTEMS
https://github.com/henrygd/beszel/blob/d2284c3fed0fa48f1b8f97902cc963f05812e098/supplemental/guides/systemd.md?plain=1#L113
Environment="EXTRA_FILESYSTEMS=sdb"
Thanks also for super cool piece of art :)

No problem. Added commented out examples to the guide and script.
Also added a note about how to update the unit files.


Urxvt terminal dependencie in vim mode

Just want to mention I was not able to run vim-mode (t and shift T) in my gnome terminal. However, after I installed Urxvt did it work properly.

Hi, thanks for mentioning!
While urxvt is used by default, you can configure the script to use gnome terminal instead. Have at #12.


RFP - Quassel IRC

Quassel IRC is a modern, cross-platform, distributed IRC client, meaning that one (or multiple) client(s) can attach to and detach from a central core.
Windows installer containing core, client and the standalone client. Git builds can be found here or here (unofficial).

https://chocolatey.org/packages/quassel


Disable update check by default when ENABLE_MU and USE_MU install options is set to 1

Summary of the new feature / enhancement
https://learn.microsoft.com/en-us/powershell/scripting/install/installing-powershell-on-windows?view=powershell-7.4#support-for-microsoft-update-in-powershell-72-and-newer
When this options is set to 1 (which is default) powershell will be automatically updated through windows update, so i don't see a point in having (annoying) update check when auto-update is enabled by default.
Proposed technical implementation details (optional)
No response

POWERSHELL_UPDATECHECK not working?
[environment]::SetEnvironmentVariable('POWERSHELL_UPDATECHECK','Off','User')

Off turns off the update notification feature

While -nologo and POWERSHELL_UPDATECHECK will work they are yet another thing to configure when you have already told the system what you want it to do.
If I install via an MSI, the MSI mechanism should be the single thing that defines all the configuration to make my selection work correctly.
Likewise if I use Winget, then I want that to manage my updates and all relevant settings.
I should not have to chase down additional settings to disable behaviour I thought I had already specified.

Raymond Chen has a question for developers - "What if every application did this?". Windows has a central place for managing updates, there is no need for every application to perform the checks and nag the user.

And yet, office seems to now manage its own updates, edge does, VS code, does, my adobe apps do, my non-adobe PDF reader does, and notepad++ does.
What seems to be uniquely annoying with PowerShell the executable checks with github to see if there is a new version, it is unaware of which of multiple mechanisms installed the present version and whether that mechanism can update it.
I have quoted Chen's Whenever there is a coordination problem, somebody says, "Hey, let's create a process!
 Many times over the years, and  "What if every application did this?" is really talking about what I have come to call Developer-driven Development (DDD).  When did a user ever say "What I really want is more status messages from this program". or "I need a program to tell me the instant there is any kind of update available".   In VS code the PowerShell extension used to say "Hey PowerShell's been updated, I can install the new version, now or click here and I'll never mention it again", and that was pretty good. But the about the third or fourth time you get "Oi! Stupid! There's an update! Go do stuff to install it!" and the stuff does no work, you start to get negative feelings about to product.
If the defaults were reversed and instead of -Nologo getting rid of the messages we had -Logo to show them how many people would complain "when I start PS it goes straight to a > , how do I get loads of messages without putting them in my profile ?"

And yet, office seems to now manage its own updates, edge does, VS code, does, my adobe apps do, my non-adobe PDF reader does, and notepad++ does.

I have no problem with applications managing their own content that is outside of the operating system delivery mechanism, in the case of PowerShell that would be updating modules from the PowerShell Gallery.
But if an application has to manage its own updates it means that the software delivery system is fundamentally broken.
How many times has "bash" said to you that it needs updating?
Now in the case of an MSI, it can manage environment variables so it can set the POWERSHELL_UPDATECHECK at the system level at the installation time, and automatically remove the variable at uninstall time. That is really what MSI are designed to do. If you want to override the user level version of it, you are welcome to do that.

Which does not really help

or check out the release page
powershell-7.4.0-linux-musl-x64.tar.gz is available on release page

Just because there is a tar to download does not mean I want or need to upgrade it.

I know. But I've never used a package manager to install pwsh on the distros I've used, with the exception of NixOS (then there was not it in repo, it was 6+ with reduces functionality in snap). I think many users install pwsh by unpacking the archive.
Imho, the most convenient choice:

no dependencies
everything in one directory
you choose the directory yourself
you can write a script for this (github api and install-postinstall)
need not wait for the maintainers to deign to add a package

With Debian/amd64 you just add the microsoft.packages.com repository and off you go, with updates!
Ubuntu 22.04 does not place nicely with microsoft.packages.com because the dotnet sdk from ubuntu conflict with those from Microsoft.
The alpine PowerShell APK is good because it does not duplicate the dotnet runtime in the PowerShell package, it reuses the existing dotnet runtime APK, just like CBL-Mariner. CBL-Mariner also supports PowerShell on arm64 as an rpm.
On macOS I use the PKG but have to repackage the ARM64 one because the official release still requires Rosetta.

With Debian/amd64 you just add the microsoft.packages.com repository and off you go, with updates!

These are deb, rpm or zipper packages, but I told about tarballs. If I used Debian I wont add microsoft repo and install pwsh with apt or dpkg.

Imho, the most convenient choice:

no dependencies                      --  still has dependencies, you just don't know what they are and they aren't automatically resolved
everything in one directory                   ---- in the case of PowerShell it lives under /opt/microsoft/powershell/7 except for a symbolic link in /usr/bin and basic help
you choose the directory yourself        ---- no need to choose a directory, it already has a standard place to go
you can write a script for this (github api and install-postinstall)     ------ the package manager and the install scripts in the package already do all this
need not wait for the maintainers to deign to add a package.     ----- true, but you can host your own apt or yum repository if you absolutely must have the upto date version

I do not trust packages from the AUR, and there is no powershell package in the main repositories and is not expected.

I maintained a PKGBUILD for PowerShell for ArchLinux but have not updated it since 7.3.4. It builds the package directly from the same tarballs you are referring to.

I do not trust packages from the AUR, and there is no powershell package in the main repositories and is not expected.

I maintained a PKGBUILD for PowerShell for ArchLinux but have not updated it since 7.3.4. It builds the package directly from the same tarballs you are referring to.

I have updated the checksums for 7.4.0.

I maintained a PKGBUILD for PowerShell for ArchLinux

This good. Could I recommend you to add one more post-install action? In order of proper working of Test-Connection cmdlet execute after:
mv "$pkgname-$pkgver/pwsh" "$pkgdir/opt/microsoft/powershell/7"

this command:
setcap cap_net_raw=eip "$pkgdir/opt/microsoft/powershell/7/pwsh"

from the same tarballs you are referring to

I was reffering to musl tarball (for alpine or musl void)

The mv you mention is part of the package function, I believe I need a complete new function like the following?
post_install() {
    setcap cap_net_raw=eip "$pkgdir/opt/microsoft/powershell/7/pwsh"
}

Should it be like that?
The setcap needs to get executed after the installation.

I maintained a PKGBUILD for PowerShell for ArchLinux

What is your powershell?

I maintained a PKGBUILD for PowerShell for ArchLinux

Which is your powershell?

None of those. I do not publish in the Arch Linux repositories. I maintain a PKGBUILD file built directly from official binaries.

This is the Linux equivalent, where a package manages the global environment variables. It would be good if the MSI did the same in the MSI/WiX/Windows way.

The idea is for people who don't trust unofficial packages and want to build from a PKGBUILD file they can see in order to validate what it is doing and where the source/binaries come from.

Any user can just clone git of any package and read PKGBUILD.
For me, I don't trust AUR powershell packages, not for security reasons, but because they offer an official installation option that is different from mine, from the $PSHome location to the contents of the man page.


Fix: Module deprecated warnings

Coverage remained the same at 95.283% when pulling 1f33fde8bc8acd28d34dd2c415890c6503a26a53 on dineshvgp:fix-deprecated-warnings into aa34f0d68390ba39771fc75c209567e967e58614 on coryhouse:master.

Thanks! 👍

Looks like I didn't have to make the issue about switching to env. Change look great!


Replace chrono with time

This PR removes all our direct dependencies on the chrono crate and introduces a dependency on time instead.
fixes issues #23 and #24

This touches too much of Cargo.lock. Lets wait for the other Cargo.lock upgrades first.

bors r+

bors r-


Authorize 401 returned with Response Body

I have implemented an MVC Controller protected with an authorize filter using a policy
 [Authorize("dataEventRecordsAdmin")]
 [HttpGet("{id}")]
 public IActionResult Get(long id)
 {
    return Ok(_dataEventRecordRepository.Get(id));
 }

I send a request with an authorized user but has no rights to this resource. Fails the Policy.
I expect a 403 back with no data in the body.
I receive a 401 with data in the body. The body should be empty.
Policy def:
 services.AddAuthorization(options =>
            {
                options.AddPolicy("dataEventRecordsAdmin", policyAdmin =>
                {
                    policyAdmin.RequireClaim("role", "dataEventRecords.admin");
                });
                options.AddPolicy("dataEventRecordsUser", policyUser =>
                {
                    policyUser.RequireClaim("role",  "dataEventRecords.user");
                });

            });

app.UseJwtBearerAuthentication(options =>
            {
                options.Authority = "https://localhost:44345";
                options.Audience = "https://localhost:44345/resources";
                options.AutomaticAuthenticate = true;
            });

Sorry I figured it out
To return a Forbidden 403, an AutomaticChallenge=true  in the UseJwtBearerAuthentication is required
options.AutomaticChallenge = true;
The body was returned because it was cached in the browser for a previous request.


support for getting hosts lists from configMaps

I've got all of my apps' configs in configmaps, but when I try to do something like this, it fails with a syntax error:
spec:
  tls:
  - secretName: appname-tls
    hosts:
      - valueFrom:
          configMapKeyRef:
            name: appname
            key: LOCAL_DOMAIN

It'd be super useful to be able to generate the hosts list from configMap entries so we're not having to hardcode these values.

This is not supported by kubernetes Ingress objects. To be honest I dount that it is likely they gonna support your use case, but the right repo to raise the issue would be in github.com/kubernetes/kubernetes


task deploy:vendors uses composer install install ... instead of composer install

Deployer version: 7.0.2
Deployment OS: MacOs

I am using the Statamic recipe, but also tried it with just common recipe, without any change.
vendor/bin/dep deploy production
task deploy:info
[production] info deploying HEAD
task deploy:setup
task deploy:lock
task deploy:release
task deploy:update_code
task deploy:shared
task deploy:writable
task deploy:vendors
[production]  error  in vendors.php on line 31:
[production] run cd /var/www/virtual/user/deployer/releases/9 && /usr/bin/php /usr/local/bin/composer install install --dev --prefer-dist --no-progress --no-interaction --optimize-autoloader 2>&1
[production] You are using the deprecated option "--dev". It has no effect and will break in Composer 3.
[production] Invalid argument install. Use "composer require install" instead to add packages to your composer.json.
[production] exit code 1 (General error)
task deploy:failed
task deploy:unlock

Made the error myself by adding set('composer_options', ...) to my deploy.php.


Improvements to documentation

This PR includes some improvements to the documentation:

Fixed some links to the API docs that were not working in the docs after merging #239.
Simplified more of the sub-submodule imports to submodule imports in the docs.
In the coil optimization tutorial, the minimal example was simplified, and a new section was added to list the many available regularization terms.
stage_two_optimization.py was split into a simpler minimal version and a version with all the regularization terms.

I agree that the base classes Curve, Surface, and MagneticField aren't user-facing so we shouldn't need to import them into simsopt.geo and simsopt.field in principle. However, when I tried this approach, I experienced two problems. First, then Curve, Surface, and MagneticField don't appear anywhere in the API docs, not even in the more detailed API docs "for developers". This is a problem because then some important user-facing methods like .plot() and .to_vtk() aren't in the API docs anywhere. Second, if Curve, Surface, and MagneticField aren't imported into simsopt.geo and simsopt.field, then it doesn't seem possible to link to these classes in the .rst docs files. For instance, in this approach, then where :obj:'simsopt.geo.curve.Curve' appears in one of the .rst files, the hyperlink to the developer API docs does not work. Do you see some other way around these problems?

Ok.  Makes sense. In that case your changes are correct.
On Sat, Jun 25, 2022, 2:10 PM Matt Landreman @.***>
wrote:

I agree that the base classes Curve, Surface, and MagneticField aren't
user-facing so we shouldn't need to import them into simsopt.geo and
simsopt.field in principle. However, when I tried this approach, I
experienced two problems. First, then Curve, Surface, and MagneticField
don't appear anywhere in the API docs, not even in the more detailed API
docs "for developers". This is a problem because then some important
user-facing methods like .plot() and .to_vtk() aren't in the API docs
anywhere. Second, if Curve, Surface, and MagneticField aren't imported
into simsopt.geo and simsopt.field, then it doesn't seem possible to link
to these classes in the .rst docs files. For instance, in this approach,
then where :obj:'simsopt.geo.curve.Curve' appears in one of the .rst
files, the hyperlink to the developer API docs does not work. Do you see
some other way around these problems?
—
Reply to this email directly, view it on GitHub
https://github.com/hiddenSymmetries/simsopt/pull/241#issuecomment-1166336647,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA62VECAJYFQE7QB3WCMFGDVQ5DQ3ANCNFSM5Z2QSYFA
.
You are receiving this because your review was requested.Message ID:
@.***>


fix false positive for ellipsis

Long time no see. 😁
See https://github.com/golangci/golangci-lint/issues/991. Trying to pre-allocate when ... is in play in a for loop gets messy. Also revised the logic to better handle multiple assignments, and added a test.

Just like old times, you come and improve everything :smile: Double thank you for the tests. It was pretty easy to add Buildkite builds for nakedred, I can probably do the same thing in prealloc as a followup PR.


[Model Level] Add Bert, Resnet18, Whisper Model

Add Model

Bert
Resnet18
Whisper

this branch is opened for ospp check

Correctness in X86 avx512 has been verified.


Enhancement: useRecoilHook return includes prop of all current states of atoms/selectors passed in

Sometimes a custom hook modifies state as a side-effect and does not return that state as a prop. We would like to test whether the side effect works as expected.
Example: You have a datapoint of type Foo accessed via const foo = atom<Foo>(...). You have a dialog to edit Foo, which maintains data of type FormState and that is returned from the dialog's controller, which is a custom hook. But the controller also has the function saveChanges: () => void that updates the foo atom when the user clicks "save" button.
So you have a controller returning { formState: FormState; saveChanges: () => void }.
To fully test the controller, we need a way of doing
test('saveChanges function saves formState changes in foo state', () => {
    const _ = renderRecoilHook(myController, {
        states: [{ recoilState: foo, initialValue: 'foo' }],
       formState: [{recoilState: myFormState, initialValue: { newFoo: 'bar' }]
    })
    act(() => _.result.current.saveChanges())

    expect(_.states.current['foo']).toHaveValue('bar')
})

If renderHook returned the queryBySelector-type things, we could pass in a wrapper that returns <input readOnly data-testid="foo-state" value={JSON.stringify(myFooState)}/> and then do expect(getByTestId('foo-state')).toHaveValue('bar') but it doesn't, so no dice.

I figured out I could wrap my hook with a wrapper hook and have the wrapper return all my hook's properties + whatever other state I wanted to test, and then just test that from { result }


bug: IonContent scrolls behind IonFooter

Bug Report
Ionic version:
[x] 5.1.0
Current behavior:
With this DOM layout (React style):
<IonPage>
  <IonHeader>
    <IonToolbar>...</IonToolbar>
  </IonHeader>
  <IonContent>...</IonContent>
  <IonFooter>
    <IonToolbar>...</IonToolbar>
  </IonFooter>
</IonPage

IonContent will set below IonHeader and thus not scroll behind IonHeader which is great. Unfortunately the IonFooter sits on top of the IonContent so that the content scrolls behind the footer.
Expected behavior:
The IonContent sits above IonFooter, not behind.
Steps to reproduce:
See sample DOM layout above.
Other information:

Ionic info:

...
Ionic Framework : @ionic/react 5.1.0
...

I didn't pay attention that I was using <IonContent fullscreen> which causes this issue. Hence no bug, closing.


Add BlockNumber to parsed event

Event will look like:
{"Name":"NameOfTheEvent","Event":{"BlockNumber":842108,...}

@kompotkot : Please bump version, we already released 0.1.1


Making get_ functions a method of Network class?

Feature context
I believe that the tool can have a bit more of OOPness if the get_ functions (e.g. on.get_junction) are methods of the Network class. As such, we will be using syntax like
network.get_junction()

instead of
on.get_junction(network)

Proposed Solution
Include the get functions into the Network class via wrapper of the already implemented functions

Hi!
Sorry for getting back to you so late. I'd definitely approve of a change like this (and I think @steffelbauer would concur). The current structure is rather old and we kept it so that old code of ours stays compatible for the time being. In the future (i.e., version 1.0.0), more OOPness would improve OOPNET's overall quality. The only thing that worries me a bit, is that we have a lot of those getter functions. Adding them all to the Network class as methods might overcrowd the class. We should probably reduce the number of functions to a more reasonable number (e.g., get_node might replace get_junction, get_tank and get_reservoir). Maybe we can also come up with a better solutions for getting lists of components of a certain type (e.g., get_tanks or get_pipes).
At the moment, I don't have the time to work on this but we always encourage PRs and I'd certainly help discussing various approaches to this issue or reviewing code, if you would be open to working on this.

Hi
I don't see why "overcrowding" the class with methods is a drawback. The way I see it, it's always better to have a class method than a "hack" to get what you need. This also aims at explicit rather than implicit syntax.
I can work on updating the getters onto the class, but first, let's agree its a direction that would be in line with the development of the library.
Cheers!

Thanks for your input!
I concur with your statement that a class method is more desirable. But let me explain, what I meant above:
We have more than 50 of those getter functions. Plus, if we integrate the getters into the Network class, we should probably also add the adders and removers (again another 20 or so functions) to the Network class while we are at it.
Don’t get me wrong, I am in favor of a change into the direction you propose, but IMHO we should think how we want to do that and what we want the API to look like (since this is a rather large change). Having ~70 new public methods in the Network class without structure doesn’t seem like the best way forward to me – even if it is more object-oriented than the current solution. Something like network.junctions.get_by_id(id="J-1") seems more reasonable to me personally for instance. However, this would be a larger change to the general Network class than what you propose.
@steffelbauer what do you think?


When i open patient it stuck on loading image

Im not using dockers everything is configure by the docs.
But when i click on the patient name it doesnt load the images.

That's a very odd issue. It looks like it retrieved the study metadata correctly. I have no idea where this exception is happening or what is causing it. Do you have any more information?

Does this happen for all images?
Does this happen on a development version or in production (looks like production due to the filenames)?


feat: Cloud Gateway operations

Description

We needed an operations page that talked about how to provision, upgrade, and delete a CGW.

DOCU-3749
Testing instructions
Preview link: https://deploy-preview-7156--kongdocs.netlify.app/konnect/gateway-manager/provision-cloud-gateway/
Checklist

[x] Review label added
[x] Conditional version tags added, if applicable.

For example, if this change is for an upcoming 3.6 release, enclose your content in {% if_version gte:3.6.x %} <content> {% endif_version %} tags (or if_plugin_version tags for plugins).
Use any of the following keys:

gte:<version> - greater than or equal to a specific version
lte:<version> - less than or equal to a specific version
eq:<version> - exactly equal to a specific version

You can do the same for older versions.

Broken links are expected since the specs aren't added yet.


Can't add New Projects to empty favorite folder

Reproduction steps

Create an empty folder outside of enso

Add it to local using the Add Folder button

Click on EmptyFolder in Enso

Click on New Empty Project

Expected behaviour:
A new project gets created in EmptyFolder
Actual behaviour:
A new project gets created in my default local folder
Version 2024.4.1-nightly.2024.9.6
Build 6c80f8f05eac7ebe5dfd8479e81458f5ba2f1185
Electron 31.2.0
Chrome 126.0.6478.127
User Agent Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) enso/2024.4.1-nightly.2024.9.6 Chrome/126.0.6478.127 Electron/31.2.0 Safari/537.36

The new folder button also has the same behaviour

Closing as it seems fixed.


Migrate Index Coop space to ENS

Hi! What is your PR about?

[ ] Add images for your space
[ ] Add custom domain for your space
[x] Add alias for migrate your space
[ ] Add or edit a skin
~Edit your space with GitHub?~ (deprecated) Please migrate your space to ENS https://docs.snapshot.page/spaces/migrate
~Create your space with GitHub~ (deprecated) Create your space with ENS instead https://docs.snapshot.page/spaces/create

@ncitron Just FYI we will wait for this proposal to end before doing the migration: https://snapshot.org/#/index-coop.eth/proposal/QmdgNav7hJBCwk1MQyEvV1Y2Lm83pDHegK2bEAV73cmL7i


[AndroidManifest] Add support for <property> element via PropertyAttribute.

Fixes: https://github.com/xamarin/xamarin-android/issues/8729
Adds support for specifying the AndroidManifest.xml <property> element via a new [Property] attribute.  This appears to be the exact same as <meta-data> but with a different name, so it is implemented exactly the same.
Example:
[Service (Name = "fooService", ForegroundServiceType = Android.Content.PM.ForegroundService.TypeSpecialUse)]
[Property ("android.app.PROPERTY_SPECIAL_USE_FGS_SUBTYPE", Value = "explanation_for_special_use")]
public class FooService : Service
{ ... }

creates:
<service android:name="fooService" android:foregroundServiceType="specialUse">
  <property android:name="android.app.PROPERTY_SPECIAL_USE_FGS_SUBTYPE"
      android:value="explanation_for_special_use" />
</service>

This element can be applied to <application>, <activity>, <provider>, <receiver>, and <service> elements.

Proposed commit message:
Fixes: https://github.com/xamarin/xamarin-android/issues/8729

API-31 added support for a [`<property/>`][0] element within
`AndroidManifest.xml`, which can be contained within `<activity/>`,
`<application/>`, `<provider/>`, `<receiver/>`, and `<service/>`
elements.

Add a new `Android.App.PropertyAttribute` custom attribute which will
emit `<property/>` within `AndroidManifest.xml`:

	[AttributeUsage(AttributeTargets.Assembly | AttributeTargets.Class,
	        AllowMultiple = true, Inherited = false)]
	public partial class PropertyAttribute : Attribute {

	    public string Name      {get;}
	    public string? Resource {get;}
	    public string? Value    {get;}

	    public PropertyAttribute (string name);
	}

Semantics otherwise appear to be identical to
`<meta-data/>`/`MetaDataAttribute`: only `Resource` or `Value` can
be specified, etc.  (This is checked by `aapt2`.)

Using `[assembly:Property(…)]` will result in `<property/>` being
emitted within the `<application/>` element.

For example:

	[Service (Name = "fooService", ForegroundServiceType = Android.Content.PM.ForegroundService.TypeSpecialUse)]
	[Property ("android.app.PROPERTY_SPECIAL_USE_FGS_SUBTYPE", Value = "explanation_for_special_use")]
	public partial class FooService : Service {
	}

will emit the `AndroidManifest.xml` fragment:

	<service android:name="fooService" android:foregroundServiceType="specialUse">
	  <property
	      android:name="android.app.PROPERTY_SPECIAL_USE_FGS_SUBTYPE"
	      android:value="explanation_for_special_use"
	  />
	</service>

[0]: https://developer.android.com/guide/topics/manifest/property-element

Hi, what version will it appear in?

It will be in .NET 9. Preview 7.


compdb runextractor should allow continuing on error

Currently cmake/compdb extraction will exit if any of the subcommands fails.  We should allow continuing to extract to allow partial builds to extract as much as possible, rather than merely all-or-nothing.  It appears as though the use of errgroup here: https://github.com/kythe/kythe/blob/41af1b272687fe7183419e53f9c6e1ed1b8d9a0c/kythe/go/extractors/config/runextractor/compdb/compdb.go#L56 is at fault

Would it be also possible to report the source files that failed extraction?  It'd help to keep track of paths with known failure and alert on new failures.  No worries if this is infeasible - the failure counter is a good approximation.

@justbuchanan 's change should log the underlying errors when a compilation fails, but it isn't really feasible to dig into those and find the specific source files which failed, IMO

No worries.  Thanks for the fix @justbuchanan.

Sure thing, @pirama-arumuga-nainar. I added a bit more detail to the error message in https://github.com/kythe/kythe/pull/4546, which includes the source file.


ESM -> CJS

Refactor code to build to commonjs as es modules can import from commonjs files normally (at least from what i tested) but vice versa cannot be said as dynamic import is required to load es modules from commonjs.
Should fix this in v2

Fixed in v2 released


Add mat4.multQuat

When creating modelMatrix from rotation in pex-renderer i'm missing the following function
  var tempMat4multQuatMat4 = mat4.create();
  function mat4multQuat(m, q) {
    mat4.fromQuat(tempMat4multQuatMat4, q);
    mat4.mult(m, tempMat4multQuatMat4);
    return m;
  }

Do you think it would be worth adding?

Looks like pex-renderer use case is actually translation + rotation.

Is using fromTranslationRotationScale(a, translationVec3, rotationQuat, [1,1,1]) an option?
or adding fromTranslationRotation(a, translationVec3, rotationQuat,) another valid option?

I think they'd be both faster than running two consecutive mat4 operations (fromQuat/mult).

For building camera.invViewMatrix indeed i could just default to above and scale [1,1,1] for the below i would need to assume that we use component.transform() and position,rotation,scale are set.
function updateModelMatrix(matrix, transform) {
  mat4.identity(matrix);
  if (transform.position) mat4.translate(matrix, transform.position);
  if (transform.rotation) mat4multQuat(matrix, transform.rotation);
  if (transform.scale) mat4.scale(matrix, transform.scale);
  if (transform.matrix) mat4.mult(matrix, transform.matrix);
}


RISC-V binaries

I'd love to have some RISC-V support in this library; I'd be happy to contribute.

Sure, go ahead :)


Fixed height for SS on home page removed.

Signed-off-by: Krishna Gupta<EMAIL_ADDRESS>Fixed height for SS on home page removed.
Description
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Looking good, @Krishna2323! 👍


Remove controller from unit tests

retest this please

@admiyo one of the functional tests failed, could you check if there is something behind it?

That is just the reabse:  have not addressed comments yet.