VSCode was not responding after leaving my Jupyter Notebook opened over night

Type: Bug
I used the following sample code to reproduce this problem on my Win11 Pro machine.
import matplotlib.pyplot as plt
import numpy as np

ax = plt.figure(figsize=(10, 10)).add_subplot(projection='3d')
X, Y = np.mgrid[0:6*np.pi:0.25, 0:4*np.pi:0.25]
Z = np.sqrt(np.abs(np.cos(X) + np.cos(Y)))

ax.plot_surface(
    X + 1e5,
    Y + 1e5,
    Z,
    cmap='autumn',
    cstride=2,
    rstride=2)

ax.set_xlabel("X label")
ax.set_ylabel("Y label")
ax.set_zlabel("Z label")
ax.set_zlim(0, 2)
plt.show()

Steps:

Launch a new instance of VSCode editor on my Win11Pro PC
Create a new jupyter notebook with the above sample code, save it
Left my PC and the notebook opened overnight

Observation:

Next day, my VSCode was not responding (frozen).

My Taskbar Manager showed eight instances of VSCode sitting idle

I restarted my VSCode, but a window popped up saying, "Another instance of Code is running but not responding."

About a minute later, VSCode restarted itself successfully, but this time I saw another message within VSCode saying:
"The editor could not be opened due to an unexpected error: cannot open resource with notebook editor type 'jupyter-notebook', please check if you have the right extension installed and enabled."

I restarted VSCode again, and my VSCode was finally back to normal.

This type of event happens to me almost every day.

I tried reinstalling my Win11Pro and everything (VSCode, miniconda, extensions, etc.), but it did not resolve the issue.

VS Code version: Code 1.74.3 (97dec172d3256f8ca4bfb2143f3f76b503ca0534, 2023-01-09T16:59:02.252Z)
OS version: Windows_NT x64 10.0.22623
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9700K CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
47.92GB (40.38GB free)

Process Argv
--crash-reporter-id 664d74f9-5373-49f9-8ca8-8d56f0a65f57

Screen Reader
no

VM
0%

Extensions (19)

Extension
Author (truncated)
Version

doxdocgen
csc
1.4.0

better-cpp-syntax
jef
1.17.2

isort
ms-
2022.8.0

python
ms-
2022.20.2

vscode-pylance
ms-
2023.2.10

jupyter
ms-
2022.11.1003412109

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.14

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

cmake-tools
ms-
1.13.45

cpptools
ms-
1.13.9

cpptools-extension-pack
ms-
1.3.0

powershell
ms-
2023.1.0

autodocstring
njp
0.6.1

partial-diff
ryu
1.4.3

vscode-fileutils
sle
3.10.0

cmake
twx
0.0.17

markdown-pdf
yza
1.4.4

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1cf:30587006
282f8724:30602487
pyind779cf:30657577
89544117:30613380
pythonsymbol12cf:30657549
2i9eh265:30646982

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.75.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Great, I will upgrade and repeat my test again tonight. Will update status tomorrow. Thanks.

I have upgraded my VSCode to version 1.75, and repeated the same test. This morning my VSCode was still not responding like before. My task manager showed that one out of eight VSCode instances runs at 17% CPU. WebView2GPU also runs at 17%. Only these two processes keep the CPU busy at that time (35%).

Instead of restarting my VSCode, I wait for VSCode to respond. About one minute later, VSCode resumed and worked normally. The task manager showed VSCode is back to idle.
Just wondering why did it freeze for a minute?

@deepak1556 do you have suggestions for me on how to troubleshoot the unresponsiveness?

Do you have the same issue as me?
Yesterday I left my Chrome browser open overnight, and this morning, both VSCode and Chrome were not responding. My CPU utilization went up to 70% (mostly by Chrome, VSCode, and Widgets.exe). Once I successfully closed my VSCode, my chrome browser returned to normal. I began to wonder, was this the VSCode problem or the Win11Pro problem?

From the symptom described in https://github.com/microsoft/vscode/issues/173198#issue-1568745228, it's likely the EH is not responding. Not sure about the issue @Scoodood was seeing.

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!

Sorry for the delayed response, @Scoodood can you share the full process command line of the VSCode process that is showing high CPU usage ?

@deepak1556 @rebornix Let's follow up here.

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!

@Scoodood just to confirm are you running in a RDP (remote desktop) session ? Does the issue happen with v1.78 which comes with an updated version of the runtime ?

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!


[core-auth] Improve documentation strings in core-auth

Fixes #5800.

Windows CI test runs failed due to timeout issue mentioned in #5820, but otherwise the CI runs succeeded.  Merging this.
Approved by adpship.


Properly deal with over-aligned T's in dynamic buffers

This PR allows two different use cases (1 is arguably a bug).

It is theoretically possible that the default alignment for https://docs.rs/wgpu/latest/wgpu/struct.Limits.html#structfield.min_storage_buffer_offset_alignment is lower than the most-aligned structure DVec4.
This lets you use the dynamic storage buffer api to build dynamically defined structures. You can set the designed alignment to 1, then you can use each "element" in the dynamic storage buffer as a member of a different type and it will automatically lay things out correctly.

the WG decided to require implementations to provide a dynamic offset limit that's min 32

Ahh, didn't realize that, makes sense.

Is the 2nd use case aimed at providing a way to lay out structs with fields that are unknowable at compile time?

Yeah - we have user defined shader inputs that we read and need to properly lay out. I have no particular cares about how we get there :) The reason I reached for the DynamicUniformBuffer is that the code was already there to do basically what I needed to do, just needed this extra alignment requirement.


Update to 2.2.1

No worries, I'd made the patch on my side last week but I hadn't seen the Github repo, I would've created the merge request earlier otherwise.


CHAOS-5762 Specify permissions required in every fault

Specify permissions required in every fault
Link to the specific fault from the fault-wise permissions page for Kubernetes fault

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://666ff4159eceaa0e7939545a--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66700c4ef9e524390bd3e08f--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66704351bbcb922a8e9cde22--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://667110a3ab3297afe682de1c--harness-developer.netlify.app


Add MicroBadger badge

We saw your image on Docker Hub at mudrod/mudrod, and we thought you might like this badge for your GitHub readme showing the image contents.
The badge shows the number of layers and download size of your Docker image, and links to our site where you can see the contents of each layer.
As you're using an automated build it will also show up in your Docker Hub description.

Thank you @microbadger :)


Error running command in existing container with exec

Provide additional environment details (AWS, VirtualBox, physical, etc.):

Desktop installation with Ubuntu 14.04 LTS on physical workstation

List the steps to reproduce the issue:

When attempting to debub networking issues in https://github.com/ros2/ros2/issues/79 , I tried to docker exec into talker container to inspect the network interface. However attaching an additional bash session into the container is not working for this scenario:

$ docker network create foo
$ docker run --rm \
    --publish-service=talker.foo \
    --name talker \
    osrf/ros2 \
    /root/ros2_ws//install/bin/talker
$ docker exec -it talker bash

Describe the results you received:

The daemon log is listed here:
http://pastebin.com/NGRXLUy2
Relevant output for docker exec command starts at line 120.

docker version:
Client:
 Version:      1.8.0-dev
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   a2ea8f2
 Built:        Sat Jul 25 22:03:05 UTC 2015
 OS/Arch:      linux/amd64
 Experimental: true

Server:
 Version:      1.8.0-dev
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   a2ea8f2
 Built:        Sat Jul 25 22:03:05 UTC 2015
 OS/Arch:      linux/amd64
 Experimental: true

docker info:
Containers: 2
Images: 120
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 135
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.16.0-30-generic
Operating System: Ubuntu 14.04.2 LTS
CPUs: 8
Total Memory: 15.63 GiB
Name: eve
ID: YDOY:4L4P:RWPC:FQOA:O7IS:LOZR:26TA:6P72:U2WT:Y5QU:5PDN:LUPO
Debug mode (server): true
File Descriptors: 10
Goroutines: 15
System Time: 2015-07-31T15:10:53.824492791-07:00
EventsListeners: 0
Init SHA1: b76fde836d83edef4fee74bbf1d0946c811150c4
Init Path: /usr/lib/docker/dockerinit
Docker Root Dir: /var/lib/docker
Username: ruffsl
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Experimental: true

uname -a:
Linux eve 3.16.0-30-generic #40~14.04.1-Ubuntu SMP Thu Jan 15 17:43:14 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

Without doing the network swap can you exec into the containers?

No, just executing the first three commands I referenced:
$ docker network create foo
$ docker run --rm \
    --publish-service=talker.foo \
    --name talker \
    osrf/ros2 \
    /root/ros2_ws//install/bin/talker
$ docker exec -it talker bash

is enough to produce this symptom, even before I begin to detach anything.

Is the talker container exiting?

Nope, the example process continues to "chatter" away until you explicitly stop it with a SIGINT or ^C if you started the container interactively with -it flag. Though, I've never seen the example prematurely exit either way I start the container.

Running with master I cannot reproduce. Can you try with a master build?
Is the network you are creating a bridge network or something else?

@cpuguy83 , the experimental build cited above was in fact creating a network of type bridge.
$ docker network ls
NETWORK ID          NAME                TYPE
20008b2f57d6        foo                 bridge
3ac33f0f66db        none                null
a0facfd11b48        host                host
b227dc024748        bridge              bridge

I just tried the available master binaries, but it does not have the network command. Is there something I need to do to enable the experimental features. Prior, I had just installed experimental versions of docker via the experimental READ.ME. My understands was that is to be a nightly build.

I just reinstalled docker from the experimental source and reproduced the same behavior, but it looks the build the scripts install is from Jul 30:
$ docker version
Client:
 Version:      1.8.0-dev
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   bfccd32
 Built:        Thu Jul 30 19:25:53 UTC 2015
 OS/Arch:      linux/amd64
 Experimental: true

Server:
 Version:      1.8.0-dev
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   bfccd32
 Built:        Thu Jul 30 19:25:53 UTC 2015
 OS/Arch:      linux/amd64
 Experimental: true

I'm a little weary of running other people's images. Are you able to repro with the busybox image?

I just tried busybox and got the same result. Note that this is still the Jul 30 build.
$ docker network create foo
$ docker run --rm \
    --publish-service=talker.foo \
    --name talker \
    busybox \
    ping localhost
$ docker exec -it talker sh

How can I enable the experimental commands from the master binaries? Should I just build from source, is there Dockerfile that does this already?

@ruffsl In order to build the experimental version from master, run
sudo make DOCKER_EXPERIMENTAL=true

Thanks @aboch , just tried the building from source with the DOCKER_EXPERIMENTAL flag, and the error behavior has ceased. So I guess one thing remaining is why the experimental debs from the install script are no longer "nightly"?

@ruffsl Glad it's resolved!
Pinging @jfrazelle (sorry!) regarding exp builds
Closing this issue since it looks like it is resolved.


Delegated Superseding Recovery Rules

Cooperative Recovery of Delegated Pre-Rotated Keys
Superseding Recovery
Supersede means that after an event has already been accepted as first seen
into a KEL that a different event with the same sequence number is accepted
that supersedes the pre-existing event at that sn. This enables the recovery of
events signed by compromised keys. The result of superseded recovery is that
the KEL is forked at the sn of the superseding event. All events in the
superseded branch of the fork still exist but, by virtue of being superseded,
are disputed. The set of superseding events in the superseding fork forms the authoritative
branch of the KEL. All the already seen superseded events in the superseded fork
still remain in the KEL and may be viewed in order of their original acceptance
because the database stores all accepted events in order of acceptance and
denotes this order using the first seen ordinal number, fn.
The fn is not the same as the sn (sequence number).
Each event accepted into a KEL has a unique fn but multiple events due to
recovery forks may share the same sn.
Superseding Rules for Recovery at given SN (sequence number)
A0. Any rotation event may supersede an interaction event at the same sn. (existing rule)
A1. A non-delegated rotation may not supersede another rotation at the same sn.  (modified rule)
A2. An interaction event may not supersede any event. ( existing rule).
(B. and C. below provide the new rules)
B.  A delegated rotation may supersede another delegated rotation at the same sn under either of the following conditions:
B1.  The superseding rotation's delegating event is later than (sn is higher)  the superseded rotation's delegating event in the delegator's KEL.
B2. The sn of the superseding rotation's delegating event is the same as the sn of the superseded rotation's delegating event in the delegator's KEL and the superseding rotation's delegating event is a rotation and the superseded rotation's delegating event is an interaction.
C. IF Neither A nor B is satisfied, then recursively apply rules A. and B. to the delegating events of those delegating events and so on until either  A. or B. is satisfied, or the root KEL of the delegation has been reached.
C1. If neither A. nor B. is satisfied by recursive application on the delegator's KEL (i.e. the root KEL of the delegation has been reached without satisfaction) then the superseding rotation is discarded. The terminal case of the recursive application will occur at the root KEL which by definition is non-delegated, wherefore either A. or B. must be satisfied, or else the superseding rotation must be discarded.

@SmithSamuelM hey,
is this feature related to the case where you'd like to revoke a QVI? So when the delegator delegates the QVI (delegatee), there's a way to revoke the QVI. In case I am using the wrong terminology, what I am asking is how to revoke a delegatee so that there is a way to remove it, i.e., in case of compromise.

@SmithSamuelM thanks.
I initially thought of this sophisticated mechanism described in the issue, enabling undoing IXNs, so that it inherently breaks the delegator<->delegatee relationship, thus exposing a mechanism to "revoke" the "trust" to given QVI.
Thinking of QVI revocation, I see cases like bankruptcy/not cooperative/anything else that fits into the "no longer trusted QVI" case.
In the case of revoking a QVI: is this all about eventually revoking ACDCs of QVI's that shall no longer be a QVI? Through TEL, I can find out the ACDC state so that a revoked QVI is discoverable by myself. The delegator<->delegatee relationship may still exist, yet it's broken due to the revoked ACDC.

@blelump The role of QVI is solely granted via the ACDC. The additional requirement that the QVI's Issuee AIDs in the ACDC be delegated is about increasing the security of the those AIDs not about granting QVI authority.

@SmithSamuelM, my question is then out of the scope of this issue. Sorry for that. Can you assist with where to look for granting QVI authority? I am missing this piece from a technical perspective.

@blelump.  The vLEI Ecosystem Governance Framework docs describe in details how a QVI is granted authority.
https://www.gleif.org/en/vlei/introducing-the-vlei-ecosystem-governance-framework


docs(blog): Fix typos discovered while reading

What I did
Here are some fixes for some typos and other minor errors I noticed while surveying your blog.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix register variables

registered variables are always exists even the task is skipped.
So we can't judge if a task is skipped by if registered variable is
defined or not
This patch is to fix the bug in pool_names.yml

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

tested on cisdev with cases:
Only run the tasks in pool_name.yml.

only rbd_hybrid pool exists.
No errors, and the variables are set correctly

{"hybrid_pool": "rbd_hybrid", "ssd_pool": "rbd_ssd"}
{"ceph_pools": {"rbd_hybrid": {"enabled": true, "pool_name": "rbd_hybrid", "volume_type": "CEPH_HYBRID"}, "rbd_ssd": {"enabled": false, "pool_name": "rbd_ssd", "volume_type": "CEPH_SSD"}}}
{"ceph_default_backend": "rbd_hybrid", "ceph_default_pool": "rbd_hybrid"}

rbd_hybrid and default pools exist.
No errors, and the variables are set correctly.

{"hybrid_pool": "rbd_hybrid", "ssd_pool": "default"}
{"ceph_pools": {"rbd_hybrid": {"enabled": true, "pool_name": "rbd_hybrid", "volume_type": "CEPH_HYBRID"}, "rbd_ssd": {"enabled": false, "pool_name": "default", "volume_type": "CEPH_SSD"}}}
{"ceph_default_backend": "rbd_hybrid", "ceph_default_pool": "rbd_hybrid"}

don;t we need this PR on master first?

No, we don't need this in master branch. As we discussed, I will remove pool_name.yml, and judge upgrade flag by running_version < installed_version.

ok to test

pool.yml also determines the pool names based on what is present. It was added to handle existing pool with default pool name. I am assuming you will handle that case in matser part of ceph ansible simplification card, right?

Oh, right. I will make the PR to master branch now...


Added option to show notification

Added a function that allows the user to display a notification of the applied layout (uses notify-send). The notification can be shown by adding --notification to the command (i.e. i3-instant-layout --notification mc).

Some extra notes I just discovered after using it for a bit more: This code works fine, except when using rofi, as notify-send doesn't allow for showing piped input.
I did find this simple bash workaround, which works fine for me, but I haven't implemented a proper python function to fix the issue as I found it to be minor. This is the bashfunction I used:
notifyLayout() {read notification; notify-send "layout" "$notification" "$@"}
which can be called like this:
i3-instant-layout --list | rofi -dmenu -i | i3-instant-layout - | awk '{print $2$3$4}' | notifyLayout
However, i found it to be way more simple to simple set the showNotification-variable to True at line 208, making it send a notification any time a layout gets applied (without the need for --notification).

Thanks for your contribution, looks good to me.


Loops

In directory is the block Suggestions if you click in connect normally will connect and add the channel but if the Person or Channel you see there is already in you list you will get "This web page has a redirect loop ERR_TOO_MANY_REDIRECTS "
You get also to many redirects and the page crash if the other server have any bad configuration and not connect

@royalterra This issue is closed because Redmatrix evolved into Hubzilla. When this is still an issue in the most recent version of Hubzilla, please recreate this issue over there. We're sorry for any inconvenience this will maybe give you.


Multi regression labels

Thanks for your great work!
Presently, I am researching some related issues, I am very interested in your codes. I am a non-computer grad student who is new to machine learning, and would like to know if your code can handle multi regression label problems, or what should I do if I want use in multi regression label problems?
I will be appreciated for your reply, thank you very much!

Thanks for your interest in our work. Unfortunately, CcGAN is only designed for scalar regression labels, and its current version cannot be applied to multivariate regression labels. We currently don't know how to modify it to fit the multivariate scenario.

Thanks for your interest in our work. Unfortunately, CcGAN is only designed for scalar regression labels, and its current version cannot be applied to multivariate regression labels. We currently don't know how to modify it to fit the multivariate scenario.

Sorry to keep bothering you. Your work helps me a lot.
I wonder if there is any other way to apply multiple continuous labels to the gan? Or are there any related papers to recommend?
Thanks very much for your reply!

You may simply concatenate your multivariate regression with the noise (for generator) or a hidden map (for discriminator). Some models for text-to-image generation may be helpful when texts are represented by a continuous vector. The following paper may be helpful:
Attribute2image: Conditional image generation from visual attributes

I have tried simply concatenate my multivariate regression with the noise (for generator) or a hidden map (for discriminator), but it seemed labels interfere with each other. I will try the models for text-to-image.
Appreciate your help ~
Thank you!

I have tried simply concatenate my multivariate regression with the noise (for generator) or a hidden map (for discriminator), but it seemed labels interfere with each other. I will try the models for text-to-image. Appreciate your help ~ Thank you!

You are welcome.

I have tried simply concatenate my multivariate regression with the noise (for generator) or a hidden map (for discriminator), but it seemed labels interfere with each other. I will try the models for text-to-image. Appreciate your help ~ Thank you!

Hi @ShenMinSM , I also encountered similar problem that requires to incorporate multi-labels. Did you get any new findings with this problem? It would be greatly appreciated if you could share any of your findings.
Also, thanks to @UBCDingXin for this great work. This method works perfectly for my project.
Thanks,
Feiyang

Hi, I found that maybe using several embeddings works well in my project but I am nor sure this also works in your project.
| |
沈玟
|
|
@.***
|
签名由网易邮箱大师定制
On 11/17/2022 00:56，Feiyang @.***> wrote：
I have tried simply concatenate my multivariate regression with the noise (for generator) or a hidden map (for discriminator), but it seemed labels interfere with each other. I will try the models for text-to-image. Appreciate your help ~ Thank you!
Hi @ShenMinSM , I also encountered similar problem that requires to incorporate multi-labels. Did you get any new findings with this problem? It would be greatly appreciated if you could share any of your findings.
Also, thanks to @UBCDingXin for this great work. This method works perfectly for my project.
Thanks,
Feiyang
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hi, I found that maybe using several embeddings works well in my project but I am nor sure this also works in your project. | | 沈玟 | | @.*** | 签名由网易邮箱大师定制 On 11/17/2022 00:56，Feiyang @.> wrote： I have tried simply concatenate my multivariate regression with the noise (for generator) or a hidden map (for discriminator), but it seemed labels interfere with each other. I will try the models for text-to-image. Appreciate your help ~ Thank you! Hi @ShenMinSM , I also encountered similar problem that requires to incorporate multi-labels. Did you get any new findings with this problem? It would be greatly appreciated if you could share any of your findings. Also, thanks to @UBCDingXin for this great work. This method works perfectly for my project. Thanks, Feiyang — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.>

Hi Min, really appreciate your reply. Can you explain a little bit more about "several embeddings"? Do you mean training different embeddings for different label variables? How to deal with the multiple embeddings? Concatenate them?
Thanks,
Feiyang


Using EntityManagerInterface instead of EntityManager

In current solution when generate method of Doctrine\ORM\Id\AbstractIdGenerator has EntityManager type hint you are not able to use entity manager decorated with EntityManagerDecorator. Simply changing solid implementation to interface makes this possible.

Closing as per https://github.com/doctrine/doctrine2/pull/1519#r40604606
We may fix this in 3.x though.


Do not use master for DEVEL build of CCPi code

@paskino @epapoutsellis I feel that our Travis or DEVEL builds shouldn't be failing because CCPi code is failing (as currently in #331). Presumably CCPi has its own testing suite. I therefore suggest to modify version_config.cmake to known-working commits (or tags) for the DEVEL build as well.
Obviously, that hash/tag should be such that the tests for master of SIRF work.
This is the same as what we do for Gadgetron etc. We had continuing Gadgetron build problems until we fixed the version.

Closing this as now only CCPi-FrameworkPlugins points to master and that's unlikely to cause a Travis problem as it's python only (just a folder copy by CMake I believe). All other CCPi tags have been tested and Travis is happy with them.


Fixed module paths to Gsl.Fft

The latest GSL release packages the library differently (using jbuilder), making it incompatible with Owl.  Gsl_fft was never supposed to be accessible, but leaked through anyway.  The correct way is to always use Gsl.? for accessing parts of the packed library.

Great! Markus, many thanks for fixing this :)

There is also another pull request in the OPAM repository for fixing version constraints: https://github.com/ocaml/opam-repository/pull/9994
It's not compiling right now, apparently due to some download problems with external libraries.  If it continues to have problems, you may want to check it to see whether there is a deeper problem with Owl dependencies.


Model revamp

Checkout and branched to another PR as this modification in the model data structure is not workable.


wrongly labeled examples?

I have noticed that in the training set the following are in the wrong folder (bird for the photo with a bike, bicycle for the photos with birds).
Here the list:
17e90d44fda65c25.jpg
42faeb7b75b57568.jpg
8a5d9bda1c8161d0.jpg
d8829a17b7d4b512.jpg

Interesting. I'll take a look at this and resolve it by EOD

Thanks for the bug report @cgrozea ! Created a PR to resolve this issue in #49


ValueError: Dimensions must be equal, 'conv4_3_norm/mul'

Hello, I have tried to run your project on Python3.6 but cannot load the pretrained model. Here's the error:
`Using TensorFlow backend.
2020-07-22 15:17:04.894949: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'libnvinfer.so.6'; dlerror: libnvinfer.so.6: cannot open shared object file: No such file or directory
2020-07-22 15:17:04.894999: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'libnvinfer_plugin.so.6'; dlerror: libnvinfer_plugin.so.6: cannot open shared object file: No such file or directory
2020-07-22 15:17:04.895006: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:30] Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2020-07-22 15:17:05.705543: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcuda.so.1
2020-07-22 15:17:05.728788: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-22 15:17:05.729311: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1555] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: GeForce GTX 1080 Ti computeCapability: 6.1
coreClock: 1.607GHz coreCount: 28 deviceMemorySize: 10.91GiB deviceMemoryBandwidth: 451.17GiB/s
2020-07-22 15:17:05.729495: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-07-22 15:17:05.730903: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-07-22 15:17:05.731847: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcufft.so.10
2020-07-22 15:17:05.732093: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcurand.so.10
2020-07-22 15:17:05.733549: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusolver.so.10
2020-07-22 15:17:05.734656: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusparse.so.10
2020-07-22 15:17:05.737928: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-07-22 15:17:05.738036: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-22 15:17:05.738481: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-22 15:17:05.738962: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1697] Adding visible gpu devices: 0
2020-07-22 15:17:05.739157: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2020-07-22 15:17:05.762832: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2020-07-22 15:17:05.763046: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x40f5a60 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-07-22 15:17:05.763059: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-07-22 15:17:05.828545: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-22 15:17:05.830034: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x40b24a0 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2020-07-22 15:17:05.830047: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): GeForce GTX 1080 Ti, Compute Capability 6.1
2020-07-22 15:17:05.830170: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-22 15:17:05.830480: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1555] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: GeForce GTX 1080 Ti computeCapability: 6.1
coreClock: 1.607GHz coreCount: 28 deviceMemorySize: 10.91GiB deviceMemoryBandwidth: 451.17GiB/s
2020-07-22 15:17:05.830510: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-07-22 15:17:05.830522: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-07-22 15:17:05.830533: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcufft.so.10
2020-07-22 15:17:05.830543: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcurand.so.10
2020-07-22 15:17:05.830553: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusolver.so.10
2020-07-22 15:17:05.830563: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusparse.so.10
2020-07-22 15:17:05.830573: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-07-22 15:17:05.830619: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-22 15:17:05.830940: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-22 15:17:05.831227: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1697] Adding visible gpu devices: 0
2020-07-22 15:17:05.831252: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-07-22 15:17:05.831964: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1096] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-07-22 15:17:05.831972: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1102]      0
2020-07-22 15:17:05.831976: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1115] 0:   N
2020-07-22 15:17:05.832042: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-22 15:17:05.832366: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-22 15:17:05.832673: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1241] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 9079 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1080 Ti, pci bus id: 0000:01:00.0, compute capability: 6.1)
tracking <tf.Variable 'conv4_3_norm/conv4_3_norm_gamma:0' shape=(64,) dtype=float32> gamma
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/framework/ops.py", line 1619, in _create_c_op
c_op = c_api.TF_FinishOperation(op_desc)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Dimensions must be equal, but are 512 and 64 for 'conv4_3_norm/mul' (op: 'Mul') with input shapes: [?,64,64,512], [64].
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/user/Projects/ssd_head_keras/evaluate.py", line 43, in
'compute_loss': ssd_loss.compute_loss})
File "/home/user/.local/lib/python3.6/site-packages/keras/engine/saving.py", line 492, in load_wrapper
return load_function(*args, **kwargs)
File "/home/user/.local/lib/python3.6/site-packages/keras/engine/saving.py", line 584, in load_model
model = _deserialize_model(h5dict, custom_objects, compile)
File "/home/user/.local/lib/python3.6/site-packages/keras/engine/saving.py", line 274, in _deserialize_model
model = model_from_config(model_config, custom_objects=custom_objects)
File "/home/user/.local/lib/python3.6/site-packages/keras/engine/saving.py", line 627, in model_from_config
return deserialize(config, custom_objects=custom_objects)
File "/home/user/.local/lib/python3.6/site-packages/keras/layers/init.py", line 168, in deserialize
printable_module_name='layer')
File "/home/user/.local/lib/python3.6/site-packages/keras/utils/generic_utils.py", line 147, in deserialize_keras_object
list(custom_objects.items())))
File "/home/user/.local/lib/python3.6/site-packages/keras/engine/network.py", line 1075, in from_config
process_node(layer, node_data)
File "/home/user/.local/lib/python3.6/site-packages/keras/engine/network.py", line 1025, in process_node
layer(unpack_singleton(input_tensors), **kwargs)
File "/home/user/.local/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py", line 75, in symbolic_fn_wrapper
return func(*args, **kwargs)
File "/home/user/.local/lib/python3.6/site-packages/keras/engine/base_layer.py", line 489, in call
output = self.call(inputs, **kwargs)
File "/home/user/Projects/ssd_head_keras/keras_layers/keras_layer_L2Normalization.py", line 63, in call
return output * self.gamma
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/ops/math_ops.py", line 915, in binary_op_wrapper
return func(x, y, name=name)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/ops/math_ops.py", line 1201, in _mul_dispatch
return gen_math_ops.mul(x, y, name=name)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/ops/gen_math_ops.py", line 6125, in mul
"Mul", x=x, y=y, name=name)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/framework/op_def_library.py", line 742, in _apply_op_helper
attrs=attr_protos, op_def=op_def)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/framework/func_graph.py", line 595, in _create_op_internal
compute_device)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/framework/ops.py", line 3322, in _create_op_internal
op_def=op_def)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/framework/ops.py", line 1786, in init
control_input_ops)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/framework/ops.py", line 1622, in _create_c_op
raise ValueError(str(e))
ValueError: Dimensions must be equal, but are 512 and 64 for 'conv4_3_norm/mul' (op: 'Mul') with input shapes: [?,64,64,512], [64].
Process finished with exit code 1
`

Same problem for me!

Hi, could you please let us know your exact combination of library versions (i.e. Python, TF, Keras, etc.)?
Thanks.

Hi, I'm getting the exact same error when trying to load the pre-trained model. Python 3.7, TF 1.14, keras 2.2.4. Anyone figured out a solution yet?


Adding clusterAttribute causes tick error

Describe the bug
With the following definition
<GraphCanvas nodes={displayNodes} edges={displayEdges}
          // clusterAttribute="schema"
          edgeInterpolation="curved"
          edgeLabelPosition="above"
          labelType="edges"
          ref={graphRef}
          onCanvasClick={onCanvasClick}
          onNodeClick={onNodeClick}

          sizingType={displayNodes.length > 0 ? "pagerank" : undefined}
          draggable={true}
/>

if clusterAttribute is enabled then the following error is generated
ERROR
Cannot read properties of undefined (reading 'tick')
TypeError: Cannot read properties of undefined (reading 'tick')
    at force (http://localhost:3000/static/js/bundle.js:508477:21)
    at http://localhost:3000/static/js/bundle.js:500073:9
    at Map.forEach (<anonymous>)
    at Object.tick (http://localhost:3000/static/js/bundle.js:500072:14)
    at Object.step (http://localhost:3000/static/js/bundle.js:508837:11)
    at run (http://localhost:3000/static/js/bundle.js:508425:25)
    at http://localhost:3000/static/js/bundle.js:508431:5
    at new Promise (<anonymous>)
    at tick (http://localhost:3000/static/js/bundle.js:508421:10)
    at http://localhost:3000/static/js/bundle.js:509950:13

the clusters are shown (takes some time) when the error is closed and the error is not redisplayed unless the graph is closed and re-opened

Steps to Reproduce the Bug or Issue
Create a node dataset with the cluster information in data : {schema : "123"}
turn on clusterAttribute
Expected behavior
Just expect the nodes to be displayed
Screenshots or Videos
No response
Platform
"reagraph": "^4.15.18",
"@types/react": "^17.0.9",
"@types/react-dom": "^17.0.9",
"react": "^18.2.0",
"react-dom": "^18.2.0",
Windows 11
Chrome Version 122.0.6261.95 (Official Build) (64-bit) (although same issue in Edge)
Node 18.19.0

Your Example Website or App
No response
Additional context
No response

Can you share a example? Looks like all the demos work fine.

What do you want the example of ... the data .. or ?

Hi @amcdnl I added you to my sandbox with the data, but the sandbox is complaining about import
Let me know if you want me to recreate a smaller repo ... somewhere.

Is this still an issue?

I removed the feature from my app .. .so it may still be a problem, but I am not using the feature


NNVM / TVM Support

Any plan to support NNVM / TVM ? which is very similar with MXNet. See NNVM / TVM graph json specification: https://docs.tvm.ai/dev/nnvm_json_spec.html
I could open it with *-symbol.json filename like MXNet. If you could support it more, will be nicer. For example, coloring. Currently, I open it , which just black / white.

Is the format the same as MXNet or are they just similar by accident? Is there a model zoo for TVM? From the description it seems to be not possible to detect if a json file is TVM or MXNet so that are the same. If that's the case feel free to submit a PR with improvements.

@FrozenGene can you please work with the TVM team and provide:

A model zoo or official reference files.
A clear description or spec if TVM files are a special case of MXNet or a separate format.
How to detect if a file is TVM file. Are all TVM files named -tvm.json and did you change the TVM code to make sure this is the case. If not, are you planing to fix TVM to make it possible to easily detect the format?
Why is the TVM format different from MXNet instead of just being compliant with MXNet?
Most TVM operators seem to be fused so they wouldn't match with existing colors for known layers. How is this expected to work? Is there a spec listing all possible operators?

Thanks for quick response. For TVM or MXNet, we could not control the name of .json file, which will be decided by user. As you did before, you assume the user will provide one .json file ends with -symbol.json, which will be MXNet model. So, I follow your way and think of one way that we can assume TVM user will provide the .json file ends with -tvm.json.
Then I answer your questions.

TVM haven’t one model zoo, I will talk about TVM team about this thing

2.TVM’ graph (called NNVM graph) is a plain json, MXNet also levarge NNVM, so MXNet’s model is also one NNVM graph. So they have many common places and format. But TVM has its unique ops like you have added.

Answered before.

TVM is a inference framework (can accept CoreML model, tensorflow model. MXNet model and so on), which is different with MXNet. MXNet focus on training, not inference. But they are both from DMLC community.

All TVM Ops will start with fuse_, because TVM will do graph fusion optimization. So if I have one convolution op, TVM graph will have fuse_conv2d, if I have batchnorm after it, I will have broadcast_add_broadcast_mul (TVM will do one optimization called SimplyInference and split batchnorm into add and multiply for precomuting), then I will have fuse_conv2d_broadcast_add_broadcast_mul. I know it have some strange places for parsing, but I will advice we parse convolution firstly, because in TVM, we will do graph fusion optimization defaultly and convolution will be much likely be the begining, because convolution can not be fused by other operators. So we can assume TVM op’s name will start fuse_, then if we have conv2d next, we mark it convolution’s color, then we can have one good place to start.

It would help the have a few sample files available. I was not able to find a single TVM json file online.

It would help to have a few sample files available. Online search came back empty.
If a file includes tvm_op, can it be assumed that all nodes will be of type tvm_op and hence the file must be a TVM file and should be using different metadata?

It would help to have a few sample files available. Online search came back empty.
If a file includes tvm_op, can it be assumed that all nodes will be of type tvm_op and hence the file must be a TVM file and should be using different metadata?

TVM Only contain tvm_op (which is used for normal operators, like conv2d) or null (which is used for input or params like weights), so if a file includes tvm_op, you can assume it is TVM.
I can provide one model file mobilenet v1 model. see zip compressed file here:
mobilenet-v1-tvm.json.zip
If you need more models, I can help to construct it.


Nix Primitives.cs - types now defined in main repo

Description of Changes
These types have been moved to the main repo, where they are used by bindings-csharp as well.
Requires SpacetimeDB PRs

https://github.com/clockworklabs/SpacetimeDB/pull/1477

It looks like the tests are failing. I get this locally as well:
$ dotnet test
  Determining projects to restore...
  Restored /mnt/nutera/work/spacetimedb-csharp-sdk/examples/quickstart/client/client.csproj (in 869 ms).
  Restored /mnt/nutera/work/spacetimedb-csharp-sdk/SpacetimeDB.ClientSDK.csproj (in 869 ms).
  Restored /mnt/nutera/work/spacetimedb-csharp-sdk/tests/tests.csproj (in 878 ms).
CSC : warning CS8785: Generator 'Type' failed to generate source. It will not contribute to the output and compilation errors may occur as a result. Exception was of type 'InvalidOperationException' with message 'Could not resolve type U128'. [/mnt/nutera/work/spacetimedb-csharp-sdk/SpacetimeDB.ClientSDK.csproj]
/mnt/nutera/work/spacetimedb-csharp-sdk/src/SpacetimeDB/ClientApi/EnergyQuanta.cs(20,10): error CS0246: The type or namespace name 'U128' could not be found (are you missing a using directive or an assembly reference?) [/mnt/nutera/work/spacetimedb-csharp-sdk/SpacetimeDB.ClientSDK.csproj]

I think it's because this repo references 0.11 but SpacetimeDB master already set version for C# deps to 0.12. https://github.com/clockworklabs/SpacetimeDB/blob/01d4ed163f90b3fffbba34b3fee5c715b48b4144/crates/bindings-csharp/BSATN.Runtime/BSATN.Runtime.csproj#L5

I think it's because this repo references 0.11
https://github.com/clockworklabs/spacetimedb-csharp-sdk/blob/fb0cd6ba2ac096dd58226146fe6a5e7bbeb86bea/SpacetimeDB.ClientSDK.csproj#L25
but SpacetimeDB master already set version for C# deps to 0.12. https://github.com/clockworklabs/SpacetimeDB/blob/01d4ed163f90b3fffbba34b3fee5c715b48b4144/crates/bindings-csharp/BSATN.Runtime/BSATN.Runtime.csproj#L5

Ah yeah good catch. Ideally we would've bumped these versions when we merged the original breaking PRs but :melting_face:

Ok now only genuine tests are failing. I'll try to fix those up too. cc @Centril


Update manifests/e/Eugeny/Tabby/1.0.125/Eugeny.Tabby.locale.en-US.yaml

Fixes an issue where this package has multiple different monikers
Microsoft Reviewers: Open in CodeFlow

[Policy] Reset Labels


Extend the types of urls that PutFile accepts

Right now PutFile will allow you to specify a file's data as a URL to scrape... with the caveat that it has to be an http:// url.
This has proven to be a pretty nice workflow for people so we should consider extending this to allow scraping from other sources. Sources that I think are worth considering for the first pass at this:

s3
gcs
git

It might also be worth adding some wildcard logic so you can put the contents of an entire bucket at once.

This was fixed by #1059 .


Use Makefiles to build, test the catalog function images

This was mentioned in https://github.com/GoogleContainerTools/kpt-functions-catalog/pull/123#discussion_r528900499
We should have top-level Makefile that defines the build, test tasks as targets.
For each task, if it can be done in a few lines of shell script, we can inline it. Otherwise, we should have a dedicated script for that task. This applies to not only golang-based fn but also ts-based fn.
The current state is not what we want eventually.
e.g. we have script calling makein functions/go/publish-functions.sh.

@Shell32-Natsu looks like you moved the issue to Done but didn't close it.  is it closed?

Yes. I thought it will be automatically closed if I moved it to done.

This task completed by https://github.com/GoogleContainerTools/kpt-functions-catalog/pull/129

This task completed by https://github.com/GoogleContainerTools/kpt-functions-catalog/pull/129

I thought it would be automatically closed if I moved it to done.

@Shell32-Natsu You can close an issue in the kpt repo from a PR in the kpt-functions-catalog repo by writing fixes: GoogleContainerTools/kpt#{PR_number} in the PR body. When the PR is merged, the issue will be automatically closed.

I thought it would be automatically closed if I moved it to done.

@Shell32-Natsu You can close an issue in the kpt repo from a PR in the kpt-functions-catalog repo by writing fixes: GoogleContainerTools/kpt#{PR_number} in the PR body. When the PR is merged, the issue will be automatically closed.


buildAst fixe crash with method call syntax

Ready.

Can be merged together with #48 without conflicts.


how make something like this

__init__.py

def create_app(test_config=True):

    app = Flask(__name__, instance_relative_config=True)

    app.config.from_object(__name__)

    db = Database(app)

    app.__setattr__('db', db)

    return app

app.py

from flask import current_app
def close_request():
    current_app.__getattr__('db').close()

Why mix the server and the DB? should they not be 2 separate entities, in terms of functionality as well as structure?

It's example only .. not for close database
IN use flask-peewee some action need database parameter

Please use Stack Overflow for questions about your own code. This tracker is for issues related to the project itself. Be sure to include a minimal, complete, and verifiable example.


Version management

There are a few tools that allow for fairly nice automatic versioning. Something to think about but certainly not necessary.
https://docs.openstack.org/pbr/latest/user/features.html#version
And I think ParmEd uses https://github.com/warner/python-versioneer

+1 for versioneer

Versioneer for what we are doing would be extremely useful as we need the compute stack to be reproducible. +1 for versionner!

I'd love to add versionner, this is going to be a bit tricky though as the versioneer setup has to come after the skeleton has been created. So I can template a skeleton, replace the items in the _version.py and versioneer.py that is created, back-port them, and see if that works? We would also have to include git as a requirement for projects since versioneer won't work without it really.
I'm also a bit worried about the barrier to entry on version number here since the initial versions will be difficult to read with the default style. Looks like this: {{PACKAGE}}-0+untagged.1.gc1968f9
One option is I could have a cookiecutter post-generate hook to do a
git init
git tag 0.0.0

which should make it so if they do a setup right then it will have the correct version. Or I could just leave instructions (which I will do anyways). Thoughts?

It looks like there are some cookiecutter repos that have versioneer pre-installed:
https://github.com/tylerdave/cookiecutter-python-package/tree/master/{{cookiecutter.repo_name}}

Great find. That looks like they are doing exactly what I was thinking about doing, except I was also considering to an initial git init; git tag 0.0.0 as a post-gen hook, which I would also like some thoughts on.

I may cannibalize some ideas from the base repo for cookiecutter-pypackage, but it also looks like overkill

I was also considering to an initial git init; git tag 0.0.0 as a post-gen hook, which I would also like some thoughts on.

That sounds like a great default IMO


Adding feature for dense 'nearest neighbor' nuclei

For particularly dense images, many nuclei appear when using the neighboring nuclei filter. Can we add an additional visibility feature to only filter to nuclei directly contacting the current region? For the existing filter, some of the nuclei that appear are not relevant visually when providing feedback. Picture below as an example:

Remember that you can manually turn on and off visibility for any region by entering Region Visibility mode and clicking the regions you want to turn on/off.
Beyond that, we could add a user control for the radius used to count as a neighbor, and put it in settings. What do you think @JustKeepClimbing ?

Hi @davidborland , sorry for the delay in responding. I recall the visible toggle, but having a more advanced control to define radius would be helpful with denser images. This may be a moot point though as Segmentor evolves.

Initial version of neighbor radius control, and improved method for calculating neighbor distance implemented.


Preview showing first scene title

In my test application I have the configuration (cut down version) as shown below. When the app starts, the preview page is displaying the title from the first scene which i think is incorrect. It is picking up the title from the default. Ideally what i would like would be a new configuration parameter, previewtitle, that would display a title for the preview page.
The other option that would be useful would be to have a parameter to change the titlle off the button (pnlm-load-button) i.e.  "loadButton": "
Click to
start
tour
" etc.
pannellum.viewer('panorama', {
    "type": "equirectangular",
    "preview": "/images/streetview.png",
    "autoLoad": false,
    "default": {
        "firstScene": "Livingroom",
        "sceneFadeDuration": 2000
    },
    "scenes": {
        "Livingroom": {
            "title": "Livingroom",
        }
    }
});

I just added loadButtonLabel, previewTitle, and previewAuthor configuration options in 624c1c11244fc8ad3310a1315c944f94c915ef6e.

Thank you!


Enable autobuild of members in landscape

Signed-off-by: John Mertic<EMAIL_ADDRESS>

Build failed because of:
No cached entry, and OpenNebula (member) has issues with logo: opennebula.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/opennebula.svg'
No cached entry, and Marvell International Ltd (member) has issues with twitter: https://twitter.com/marvellsemi, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Item OpenNebula (member) has no image_data

Build failed because of:
No cached entry, and OpenNebula has issues with logo: OpenNebula.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/OpenNebula.svg'
No cached entry, and Marvell International Ltd (member) has issues with twitter: https://twitter.com/marvellsemi, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Item OpenNebula has no image_data


Installation issues under Linux Mint

This looks like a great package for symfony, but I have a couple of issues.

At the 4th step you write to edit the app/config/parameters.yml, but unfortunately there is no such file
At the 4th step you write to execute "composer install", but I get the following messages:

Do not run Composer as root/super user! See https://getcomposer.org/root for details
Composer could not find a composer.json file in /var/www/symfony
To initialize a project, please create a composer.json file as described in the https://getcomposer.org/ "Getting Started" section

I guess for the last problem I should download the json file from composer website. is this right?
Thank you in advance for your help...

Hi,
thanks for your feedback.

the app/config/parameters.yml file is your Symfony parameter file.
It's not a real problem (IMHO). To avoid this message, this line should solve this problem
RUN usermod -u 1000 www-data

cheers

Thanks for your reply. At the second bullet the problem is that there is no "composer.json" file at the folder. Unless this should be run under a specific folder, which is not mentioned at your guide steps.

My bad! I didn't read the full message...

I guess for the last problem I should download the json file from composer website. is this right?

Do you have an existing Symfony project? If yes, you probably miss the part where you mount the Symfony App volume to your containers.

Sorry for my late reply.
I do not have an existing project. As a matter of fact, I am trying to set up a dev environment to start learning to code on symfony, and I think that Docker is the best solution on that.
I have a mounted volume of the host computer, that is visible by the container. I didn't change anything of your compose yml file.
So, If I need to start developing on symfony with your docker containers, I should install manually symfony? Can you provide info on that?

I dont think that learning both Docker and Symfony is the best idea.
As it stand, you can't install Symfony in this Docker stack. You need to do it outside.
# install symfony installer
~/sites/lab »  sudo curl -LsS https://symfony.com/installer -o /usr/local/bin/symfony
~/sites/lab »  sudo chmod a+x /usr/local/bin/symfony
# create the app
~/sites/lab »  create app
~/sites/lab » symfony new app

 Downloading Symfony...

    5.3 MiB/5.3 MiB ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓  100%

 Preparing project...

 ✔  Symfony 3.2.1 was successfully installed. Now you can:

    * Change your current directory to /Users/maxence.poutord/sites/lab/app

    * Configure your application in app/config/parameters.yml file.

    * Run your application:
        1. Execute the php bin/console server:start command.
        2. Browse to the http://localhost:8000 URL.

    * Read the documentation at http://symfony.com/doc

# then come back to the docker stack and restart the readme
~/sites/lab » cd docker-symfony


[Issue]: latestSupportingAssetId is incorrect in migrated ISRA resulting in incorrect new supporting asset ID

Issue Description
The issue is from an sra that was saved from an xml. There are 4 supporting assets and when you add a new supporting asset, the ID is still 4.
Duplicate Steps

Add a supporting asset when the asset ID order in the table is 4, 1, 2, 3
Notice that the new ID is 4

Looking at the sra file, "latestSupportingAssetId": 3. The issue is when the file was saved from xml, the latestSupportingAssetID may have been based on the last asset ID on the table.
Expected Results
The supporting asset ID cannot be a duplicate of an existing asset ID
Version Detected
1.1.0 (Default)
Attachments
No response

Workaround is set to set  latestSupportingAssetId manually in the SRA file.

Set the latestSupportingAssetId in the SRA file before adding the supporting asset. It's better to re-open the file after editing manually. Example: If the highest ID is 4 then set latestSupportingAssetId to 4


Add support for int4 weight-only QAT

Summary: This commit adds support for int4 weight-only QAT, which simulates the numerics of the existing
Int4WeightOnlyQuantizer. The main motivation for this is to provide an end-to-end path for running QAT and lowering to the efficient int4 tinygemm cuda kernel. To enable this, we have to add new fake quantization primitives to match the numerics of the tinygemm kernel, and this required refactoring existing quant primitives to skip dtype casting.
Test Plan:
python test/quantization/test_qat.py -k test_qat_4w_linear
Reviewers: jerryzh168, msaroufim
Subscribers: jerryzh168, msaroufim, HDCharles, supriyar

Cool! Do you think we could also merge a table showing accuracy and speedup in the QAT folder? I'd like to use that in a new training section this week

Cool! Do you think we could also merge a table showing accuracy and speedup in the QAT folder? I'd like to use that in a new training section this week

Yeah setting up the experiments now, hoping to get those numbers soon

What happened with this PR? Are we still planning on merging it?

What happened with this PR? Are we still planning on merging it?

Yes, Jerry and I discussed offline. I plan to refactor the existing quant primitives to make a common fake_quantize_affine first, then update this PR to use that


ffi: another batch of fixes from pyo3-ffi-check

Fix for PyTypeObject on Python 3.8, as well as removing a lot of stuff not supported on PyPy. I was also able to add support for datetime folds on PyPy.

@mejrs as I've got CI green I'm going to merge this now. Might conflict slightly with the code.rs changes in #2409 - sorry!

@mejrs as I've got CI green I'm going to merge this now. Might conflict slightly with the code.rs changes in #2409 - sorry!

It wasn't so bad :)


Channel forced closed cannot spend onchain balance

A channel was forced closed the funds show in my onchain wallet but if I go to spend them or swap to a lightning channel the app shows no funds available.  I have tried resetting and restoring but I still have the same problem.  On-chain tx shows over 12 confirmations.

mutiny-logs.txt

May take up to 3 days for the sweep transaction to funds to become available. There should be a 'sweep transaction' afterwards. Please check back over the next few days and let us know if it doesn't appear.

Oh yes I forgot about the wait for a force close.  Thanks


AssetLoader and ImageLoader on password protected pages on Chrome

Apparently it is impossible to load atlases on Chrome (a JSON file that points to an image file), when working on password protected pages with basic authentication. Chrome console shows a 401 (Authorization Required) error on the image and does not allow to download it to the browser:
GET http://webpage/confetti/confetti-spritesheet.png 401 (Authorization Required)

The funny thing is that you can load the image directly without any problem:
var texture = PIXI.Texture.fromImage("http://webpage/confetti/confetti-spritesheet.png"); // this works!

Here is a probable reason: http://stackoverflow.com/questions/19244006/chrome-basic-auth-image-requests-return-401-but-not-when-called-directly
But this is not completely clear why we can't load an image with AssetLoader, but we can with Texture.fromImage(). Someone has an idea on how to fix this or a workaround?

Pretty sure this is a duplicate of #1269 or #1234


Remove button - how to add remove button on corner of text box

Hello thanx for ur code its very useful for me but i am not able to add a remove button in the text box
i.e, it must dissapear when text box dissapears
please reply ASAP

hello @vrsrohit . In MotionEntity.java there are methods draw() and drawSelectedBg(). Use them to draw anything on top you need.

Hi i have similarly problem, but i have drawn it but now i confused to make listener on it? @AndriyBas Could you give me some advice?


Can not use !negative pattern to exclude directories in multiple webpack configs

Issue description or question
I fail to use !negative pattern to exclude directories in multiple webpack configs to prevent this plugin from deleting nested paths. After building, the nested directory abc is deleted.
Webpack Config
const {CleanWebpackPlugin} = require('clean-webpack-plugin');

module.exports = [
  {
    entry: 'src/index.js',
    output: {
        path: path.resolve(__dirname, 'dist')
    }
    plugins: [
       new CleanWebpackPlugin({
          cleanOnceBeforeBuildPatterns: ['**/*', '!abc/**/*'],
       })
    ]
  },
  {
    entry: 'src/abc/abc.js',
    output: {
        path: path.resolve(__dirname, 'dist', 'abc')
    }
    plugins: [
       new CleanWebpackPlugin({
         cleanOnceBeforeBuildPatterns: ['**/*', '!def/**/*'],
       })
    ]
  },
  {
    entry: 'src/def/def.js',
    output: {
        path: path.resolve(__dirname, 'dist', 'abc', 'def')
    }
    plugins: [
       new CleanWebpackPlugin()
    ]
  }
];

Environment
 System:
    OS: Windows 10
    CPU: (8) x64 Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz
    Memory: 398.06 MB / 15.91 GB
  Binaries:
    Node: 10.13.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.13.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 6.4.1 - C:\Program Files\nodejs\npm.CMD

@popolan1986 I ran into the same problem and replaced the '!excluded_dir//*' pattern with  '!excluded_dir/'. That does seem to work.

@sirockin , I know why the nested directory is deleted. abc/**/* matches abc's nested directories but not itself so the clean plugin deletes abc. I can use Braced section to exclude abc directory like this "!abc{, /*}" (= !abc, !abc//*).

@popolan1986 is your issue resolved? Can I close this?

Yes, you can.


python310Packages.python-box: 6.0.2 -> 6.10.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.python-box is: "Python dictionaries with advanced dot notation access"
meta.homepage for python310Packages.python-box is: "https://github.com/cdgriffith/Box"
meta.changelog for python310Packages.python-box is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.10.0 with grep in /nix/store/kyfpz6fsvsqlqmilcpjzxn2qyxhrv6vm-python3.10-python-box-6.10.0
found 6.10.0 in filename of file in /nix/store/kyfpz6fsvsqlqmilcpjzxn2qyxhrv6vm-python3.10-python-box-6.10.0

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.pytenable
python310Packages.python-box
python310Packages.restfly
python39Packages.pytenable
python39Packages.python-box
python39Packages.restfly

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kyfpz6fsvsqlqmilcpjzxn2qyxhrv6vm-python3.10-python-box-6.10.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.python-box https://github.com/r-ryantm/nixpkgs/archive/04c151b9e35c1cd7ba6f03426d97a17aea251436.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kyfpz6fsvsqlqmilcpjzxn2qyxhrv6vm-python3.10-python-box-6.10.0
ls -la /nix/store/kyfpz6fsvsqlqmilcpjzxn2qyxhrv6vm-python3.10-python-box-6.10.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

6 packages built:

python310Packages.pytenable
python310Packages.python-box
python310Packages.restfly
python39Packages.pytenable
python39Packages.python-box
python39Packages.restfly

Maintainer pings
cc @fabaff for testing.

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

6 packages built:

python310Packages.pytenable
python310Packages.python-box
python310Packages.restfly
python39Packages.pytenable
python39Packages.python-box
python39Packages.restfly


Update blade names and link references for SA

Renamed the "Job" blade to the "Overview" blade, as it is shown in the portal. Also updated where the link to the Query blade is.

@chazbot7 : Thanks for your contribution! The author, @jseb225, has been notified to review your proposed change.

Thanks a lot for the suggestion. Looks good #sign-off


Conditional return types don't work

Bug Report

🔎 Search Terms

conditional return type alias generics
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about conditional return types

⏯ Playground Link

Playground link with relevant code
💻 Code

type Test<T> = T extends string ? number : boolean;

function foo<T>(x: T): Test<T>{
  if(typeof x === "string"){
    return 5;
  }
  return true;
}

🙁 Actual behavior

Both branches have different flavors of error 2322
🙂 Expected behavior

the typeof guard should have let Typescript know it was dealing with a string and should have now which branch of the conditional type it should be allowed to return. The same outside of the type guard since it returns and control flow analysis should have treated that like an implicit else.

What I'm really trying to do:
I'm trying to write a function that when giving a string returns Wrapped
The closest I got this to working is with overloads but then I couldn't get it working with downstream functions even if they had overloads. https://www.typescriptlang.org/play?ts=4.4.3#code/JYOwLgpgTgZghgYwgAgOpTgB0xAJgHgBUA+ZAbwChlkoI5cB7EAGwE9kxWcAuZAIgDuGbHj4BuKjTqMW7AG5xmAVwi9CEgL4VtMJSARhgTZAiZzoYABQAPXgGcwUUAHMAlMl7osOAg6chnYgldfUNjUxBzKDAiYhtPYR9Y9wTvPFjgvQMjEBMzC3jkPQBrEAYBEBSikFLy3MpqamAYS04cBhhka2QAXj7+Pxc+VwbGsdowJSh6ttV+ITTcPgAaZAVlOesNCTGtcYhJ6a6KLR0ssNycKABbYDs7YHMAMXOc2MLBgOQAHzRE9JKZQqxCqXhEBEBdVIowmU1yESiVmsrk02jRQA
So I tried to be smart about doing the conditional in the return type and saw this bug.

duplicate of #33912


Use /v2 suffix for new go modules v2

Hey @matcornic
thanks for merging my changes (#44) related to supporting go modules. Because you created a new major version (v2) for the go module changes, we should reflect that in go.mod.
In go modules, if the major version changes, the import path also changes. So I also updated all the examples and the go get in the README.
Thanks,
Atrox

Codecov Report

Merging #45 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   98.45%   98.45%
=======================================
  Files           3        3
  Lines         907      907
=======================================
  Hits          893      893
  Misses          8        8
  Partials        6        6

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f305906...f79c2e7. Read the comment docs.

Thanks for the quick merge @matcornic! Could you release a new version v2.0.1?

Thanks for the quick merge @matcornic! Could you release a new version v2.0.1?

Done :)

Awesome, thank you very much. Now everything runs smoothly for me. :)


cleanup ingress and use route rule as defacto config

This should alleviate some of the ingress woes by pointing people to use istio route rules, and a dummy ingress.

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed


[AMO-128509] - Ad Cloud Search Metadata Schema

Please link to the issue #…

@lrosenthol @prabhum2

@adcloud team - @jishah-adobe I would like to udnertand the use-cases and why this separate record - time-series schema is created ; where can I find out more about this?

@adcloud team - @jishah-adobe I would like to udnertand the use-cases and why this separate record - time-series schema is created ; where can I find out more about this?

@ktukker-adobe - We are moving our reporting data from relational databases to Ad Cloud Platform. The aggregated metrics data falls under two categories - performance (cost/clicks/impressions) and conversions (registrations, orders, revenue).  Performance Metrics are pulled from ad platforms (search/social) and data we receive is aggregated at daily level across collection of attributes (e.g ad, device and match types).  The data we receive doesn't fall under time-series events.  However, data platform currently lacks mechanism to define custom defined partitions - in our case assume by day).  Also this data gets reconciled and needs to be re-processed at a bare minimum 3 times.  To facilitate some level of partitioning scheme the proposal from XDM team was to define our own time-series class, mix-in and data-types under ad-cloud namespace and then use them to define schemas per tenant.  The same theory holds true for attribute/entity tables.  They do not fall under profile a.k.a record class. We were able to define them as a custom/ad-hoc schemas in earlier version of XDM but since that restriction was lifted and schemas were forced to extend from a class we are building our attribute schemas using profile class though none of the fields will be populated and left empty. We can schedule a quick meeting on this if it helps to move faster.

This PR looks good to me. Only waiting on a formal OK from PM around this new class which will appear in the UI along side the other two core classes.

My Open questions:

In the IRD on the wiki (https://wiki.corp.adobe.com/pages/viewpage.action?spaceKey=EfficientFrontier&title=XDM+Schema+Review), the baseschema (class) is listed on the wiki as : “profile-adcloud” – I don’t see this defined in this PR. If it would be defined, what does a “Profile AdCloud” describe? Is is the consumer who clicks on an add? Is it the Adobe Account (Customer) who has accounts with google and other ad spaces? (“Profile” is too generic of a description).

In Experience Platform “context/profile” soon will be renamed to “context/individual” to better represent the definition: A consumer, it’s attributes, behavior, identities and segment membership. In future, definitions for “Enterprise”, “Household” and “Neighborhood” might be defined. These all represent "Profiles" of different entities. Where does this object fit in?

Current definition of “adgroup-all.schema.json” extends “context/profile” – is this the actual “profile-adcloud” mentioned above? Is this modeling correct? If I look at the definitions, these look more “account” level (Adobe customer) than “Individual” (Consumer level) definitions


Nixpkgs Architecture: Simple Package Paths RFC draft implementation test [skip editorconfig]

Description of changes
In recent months, the Nixpkgs Architecture Team has been working on a draft RFC to make the first large-scale improvement of the nixpkgs architecture. Feedback appreciated, feel free to open issues/PRs to that RFC draft repository!
As part of that effort I'm writing a tool in Rust to automatically perform the tree-wide migration proposed by the RFC draft. The tool is a bit hacky still, but it's at the point where I'd like to have CI evaluate the result to see what breaks, which is what this PR is about. Feedback for the tooling is also appreciated, it should run as part of CI in the end.
This might not be the final state of the RFC but it's pretty close to what we hope to do.

[skip editorconfig] should help with this.

@zowoq Not entirely how that works, see here, but that can't disable just a specific workflow, it disables all of them which I don't want

https://github.com/NixOS/nixpkgs/blob/31ce2fc58405e1ef3cfc96755c4970716c5ece5a/.github/workflows/editorconfig.yml#L14
[skip editorconfig] skips only the editorconfig check

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2022-01-23-nixpkgs-architecture-team-meeting-26/24892/1

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2023-09-05-nixpkgs-architecture-team-meeting-43/32635/1

I'm planning to implement this in two steps:

Implement a CI check to require new packages to use pkgs/by-name if possible. This implements the exclusion list from part 2, but doesn't migrate any packages yet.
This could benefit from https://github.com/NixOS/nixpkgs/issues/256788

The mass migration itself, which now only needs to do three things for each top-level package using callPackage:

Move all of its files to pkgs/by-name
Remove it from the exclusion list
If it uses an empty second argument, remove its definition from all-packages.nix. This is also planned to be enforced with https://github.com/NixOS/nixpkgs/pull/256792

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/ci-will-soon-enforce-pkgs-by-name-for-new-packages/38098/1

FYI the Nixpkgs Architecture Team link is incorrect or dead.

Note that @Aleksanaa did a large part of the automated migration in https://github.com/NixOS/nixpkgs/pull/354531, thank you! There's still more that can be migrated manually, so I'll leave this PR open for now until I or somebody else can do it


ERROR: could not find /var/run/openvswitch

This is for running ovs tool on a remotehost endpoint.  FWIW the directory /var/run/openvswitch does exist

This is probably an issue with mount points used for remotehost endpoint, and not with the ovs tool itself.


[DO NOT MERGE, DRAFT] Bhavikmfujikosu/eval sample geval

This is currently in draft. This PR is created just to test if CI pipeline passes
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
All Promptflow Contribution checklist:

[ ] The pull request does not introduce [breaking changes].
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

@microsoft/prompt-flow-approvers
This PR is currently in draft mode and I created this to test CI pipeline. Once we get this PR ready for review, we'll update the title of PR and tag you again! :)

@crazygao sorry I think your review got dismissed by my push (no more changed needed from us), could you please add your comments/review again


move implementions of util functions from ds3.c to ds3_utils.c, to ma…

…tch definitions in ds3_utils.h

ds3_utils.c needs to be added to the build.  As it is now with this merge, all the functions in that file are not available in the built library.


Optimize cbrtf by removing checks for non-normal floats

This code removes checks for NaN, +/-INF and subnormal numbers. This should be a safe assumption to make for all realistic input values. This presumably leads to more optimization due to less branching and maybe even auto-vectorization, but i honestly haven't checked. It's faster either way.
I originally wanted to refactor the code too, but I don't think that's necessary.


Input - Check to see if the user already chose a letter

Only a unique letter counts as a guess. So a user can't enter.

I would like to work on that

alright @abderra7 it's yours!


Retention of Azure AD reports

The following paragraph of the document seems to be incorrect:
"Events in the Azure AD audit report are retained for 180 days."
It depends on the Azure AD licenses and will be stored 7-90 days
(reference: https://docs.microsoft.com/en-us/azure/active-directory/reports-monitoring/reference-reports-data-retention).

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

ID: f7c9d466-1830-ebaf-066f-8b53b57b6917
Version Independent ID: 414715c5-d5f7-823a-3869-d8eb08448dbd
Content: Azure logging and auditing
Content Source: articles/security/azure-log-audit.md
Service: security
GitHub Login: @unifycloud
Microsoft Alias: TomSh

@ThomasLive, Thanks for the feedback!  We are taking a look into this and will get back to you soon.

@ThomasLive Thank you for bringing this to our notice . We have initiated an update on our end and it will reflect on the page in the next release cycle in 24-48 hours. We will close this issue now.
Thank you.


Aliases are forced as lower case and projects identifiers to integers

@sephii With the taxi-tempo plugin, we can use JIRA tickets as aliases, like INFRA-5 or OPS-318. However, because of how taxi currently works, we're forced to write them in lower case in our timesheet files.
In the same type of issues, the project identifier is forced to be an integer, where I would have used the first part of the JIRA issue ("INFRA"), which lead us to do quite a hack to convert it back and forth and therefore the aliases generated in ~/.taxirc are obfuscated (where they wouldn't need to be).
Any chance these constraints might be made optional / removed ?

Can you describe what's happening when you use a non-lowercase alias? I don't remember any constraint regarding aliases but it's been a long time since I last worked on the parser. ^^
For the project id constraint I wouldn't see a problem accepting an str type. I'd be happy to review a PR, or this will have to wait a bit until I find some time to work on this.

Fixed it! The issue with uppercase aliases in timesheet entries was that it resulted in an "inexistent alias" warning and the entry was skipped.


🐛 Bug - Uncaught (in promise) TypeError: unref(...).stop is not a function

@google-translate-select bug report
Summary

When using this component with vue-router, when I hit the back button in the browser I'm getting the following error:
Uncaught (in promise) TypeError: unref(...).stop is not a function...google-translate-select.vue?vue&type=script&lang.ts:381

Environment & Usage

Using Vue3 and Chrome browser.
My component usage looks like so:
<template>

              Translate

        <GoogleTranslateSelect
            default-language-code="en"
            default-page-language-code="en-GB"
            :fetch-browser-language="true"
            @select="handleGoogleTranslateSelect"
            class="border border-gray-400 rounded"
            trigger="click"
        />

</template>

I'm also getting this error using this package with Nuxt. It looks like a TypeError gets thrown because googleTranslateOriginSelectObserve and htmlAttrLangObserve are null during the onBeforeUnmount hook

This doesn't seem to break anything in production, but it breaks development builds in Nuxt

@ubercj PR solved. Please check it again.

You can check version 0.1.2, Thanks @ubercj 👏


prevent label from being added to the css applied

Because this doesn't seem right and I think people will be concerned about it and not use the label feature.

Also, side question, what would happen if I nested the label? Nothing?

I could totally remove the label from the actual css being injected, not a problem.
Nested labels should get combined; idea being to show labels of all the styles being composed. In a previous version, it would sorta approximate the tree of labels and show it in the data attribute value itself, dialled it back to simple concatenation. I'm happy to iterate on this based on feedback, let me know what works best.

Concatenating makes sense :+1:

Also: css({label: 'foo'})...
I would expect that to be: css-foo-nil
but it's "css-foo-1cepw5k"

I thought about it, and I think it labels should participate in hashing, if only to differentiate origins. I'm going to leave that behaviour as is for now, but let me know if it causes actual problems.
(there's also the fact that it would be expensive to remove the labels before hashing, but we'll cross that bridge if and when we come to it)
published 2.20.37, that doesn't include the label in the generated css 🎊

Perfect! Thanks!


Bugfix/um 678 memory leaks

LGTM

LGTM

LGTM!

LGTM

LGTM

LGTM

LGTM

LGTM

I beat you to it! 😂

LGTM

@mcfadden8 I will merge this, ya?


Spotify Artist Search

It is now possible to search artists via the spotify rest api
This is PR is work in progress
Tests are still missing
Some todos in code
Caching of auth token still missing

 Spotify Search

Master merged
Following now also works on Spotfiy
Tests still missing, coming next 💦

I proudly present: the Spotify search :)
Have fun with this gigantic PR 😂


luci-mod-network: disambiguate DHCP IP set help text

Despite this string existing for a while now, it rarely got translated, and where it did, it was equally obscure. A hint that the source string was unclear.
Disambiguated.

Merged, thanks!


Example inadequacy

These examples are not very useful if you don't get to see a simple real-world configuration object.

Please check the new documentation effort. Feel free to comment there.

@bebraw Thank you! :)


The project was wrong while building in yarn

Multiple attempts to delete node_modules to no avail

Not much is happening in build-geo. Could be a network issue. Does your server have external access to download links? I believe its failing when trying to download the country db from github.

I'm debugging locally My computer is a Mac m1 pro chip

build-geo 中没有发生太多事情。可能是网络问题。您的服务器是否具有下载链接的外部访问权限？我相信它在尝试从 github 下载国家数据库时失败了。

I also get the error in centos 7：
error /opt/umami/node_modules/@prisma/engines: Command failed.
Exit code: 1
Command: node scripts/postinstall.js
Arguments:
Directory: /opt/umami/node_modules/@prisma/engines
Output:
node:internal/process/promises:279
            triggerUncaughtException(err, true /* fromPromise */);
            ^

Error: read ECONNRESET
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:217:20) {
  errno: -104,
  code: 'ECONNRESET',
  syscall: 'read',

node:16.18.1


Error in the framework.cmake file.

In the current framework.cmake file there is an error line 49.
The "include(cmake/configure_catkin.cmake)" needs to be changed in "include(cmake/configure/catkin.cmake)".
With the current version ROS users cannot build the package as the catkin_make command produces an error due to this line.

Sorry about that - I just recently moved some files and this was an oversight. I've just pushed the fix that you suggested.
Thanks for your feedback.


consider for rendering Display instead of Debug for errors reporting

Rust's Error type error type requires both a Debug and Display implementation. One for developer debugging and one for human display. When a crowbar lambda fails it prints the errors Debug out which typically is not useful for humans but when lambda reports these errors to user Debug it what they say.
I ran some experiments and feel pretty confident that Display would be a much better way to go for the way the std::error::Error type and crates like failure represent errors for human consumption.
#[macro_use]
extern crate cpython;
#[macro_use]
extern crate crowbar;

use std::error;
use std::fmt;

#[derive(Debug)]
struct E(String);

/*impl fmt::Debug for E {
    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
        write!(f, "this is a more manual debug")
    }
}*/

impl error::Error for E {
    fn description(&self) -> &str {
        "this is a description"
    }
}

impl fmt::Display for E {
    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
        write!(f, "an error occured while trying perform a thing")
    }
}
lambda!(|event, _| {
    println!("invoked with {:?}", event);
    // this produces `SystemError: Rust panic`
    //panic!("nope");
    // this produces `RuntimeError: E("foo")`
    // but could produce `RuntimeError: an error occured while trying to perform a thing` if crowbar printed Display
    let result: Result<&str, Box<_>> = Err(E("foo".into()).into());
    result
});

Fixed by https://github.com/ilianaw/rust-crowbar/pull/43


TypeError: type() doesn't support MRO entry resolution

Selene version 1.0.1 does not work with Python 3.7.
Simply importing selene.api
from selene.api import *

causes the following error:
  File "/path/to/src/tests.py", line 1, in <module>
    from selene.api import *
  File "/path/to/venv/lib/python3.7/site-packages/selene/api/__init__.py", line 23, in <module>
    from selene import config, browser, browsers
  File "/path/to/venv/lib/python3.7/site-packages/selene/browser.py", line 28, in <module>
    import selene.driver
  File "/path/to/venv/lib/python3.7/site-packages/selene/driver.py", line 33, in <module>
    from selene.elements import SeleneElement, SeleneCollection
  File "/path/to/venv/lib/python3.7/site-packages/selene/elements.py", line 50, in <module>
    from selene.wait import wait_for
  File "/path/to/venv/lib/python3.7/site-packages/selene/wait.py", line 97, in <module>
    class IFn(with_metaclass(ABCMeta, Generic[T, R])):  # todo: consider using Callable[[T], R] instead
  File "/path/to/venv/lib/python3.7/site-packages/future/utils/__init__.py", line 139, in __new__
    return meta(name, bases, d)
  File "/path/to/venv/lib/python3.7/abc.py", line 126, in __new__
    cls = super().__new__(mcls, name, bases, namespace, **kwargs)
TypeError: type() doesn't support MRO entry resolution; use types.new_class()

Hey thank you for the issue!
Now it should be fixed in https://pypi.org/project/selene/1.0.2/
I mixed some commits before releasing 1.0.1 so it happened to be broken. Now everything should be fie.
I also recommend to switch to 2.x versions of Selene. It's yet prel-apha release, but seems to be good enough to try. We checked a migration on a few projects and it was fine...

Thanks, 1.0.2 works!


Use ZeroSSL API for secondary certificate creation instead of Certbot

Is your feature request related to a problem? Please describe.
Creation of certificates with Certbot takes ~1m and we can only create one at a time
Describe the solution you'd like
Replace Certbot with our Python script that uses the ZeroSSL API. We can then create certificates in parallel, and each cert only takes ~20s
Describe alternatives you've considered
We could potentially use DNS validation for Certbot rather than the HTTP callback in use at present.
Additional context
get_certs.py

@athandle can you please get an update from @kumarnarendra701 on where he is with this ahead of the Monday sprint planning.

sure @cpswan , he deployed changes to Buzz today. He is facing some challenge

Below tasks completed for parallel SSL/Stack generation -

SSL generation via ZeroSSL API.
Tested on Wasp env.
Implement secret management(encpass) to store DO and ZeroSSL keys.
Create a readme document for secret management.

Partially complete
Implemented on Buzz but getting below errors, so reverting back to old script and shared logs to the dev team to understand the issue.
Logs -
INFO|2021-07-09 18:24:47.964836|AtSecondaryServer|Secure Socket open for @shallowwarabeuta !
FINER|2021-07-09 18:24:47.964882|AtSecondaryServer|serverSocket _listen : SecureServerSocket
Unhandled exception:
FileSystemException: Failed to initialize file system entity watcher (OS Error: Too many open files, errno = 24)
SEVERE|2021-07-09 18:24:48.241437|AtSecondaryServer|SocketException: Write failed (OS Error: Connection reset by peer, errno = 104), address = <IP_ADDRESS>, port = 2787
SEVERE|2021-07-09 18:24:48.537791|AtSecondaryServer|SocketException: Write failed (OS Error: Connection reset by peer, errno = 104), address = <IP_ADDRESS>, port = 2787
SEVERE|2021-07-09 18:24:53.117861|AtSecondaryServer|HandshakeException: Connection terminated during handshake
SEVERE|2021-07-09 18:24:53.138394|AtSecondaryServer|SocketException: OS Error: Connection reset by peer, errno = 104, address = <IP_ADDRESS>, port = 2787
SEVERE|2021-07-09 18:25:04.352246|AtSecondaryServer|SocketException: OS Error: Connection reset by peer, errno = 104, address = <IP_ADDRESS>, port = 2787
SEVERE|2021-07-09 18:25:06.078548|AtSecondaryServer|HandshakeException: Connection terminated during handshake
SEVERE|2021-07-09 18:25:06.097298|AtSecondaryServer|SocketException: Write failed (OS Error: Connection reset by peer, errno = 104), address = <IP_ADDRESS>, port = 2787

@kalluriramkumar @murali-shris @cconstab please help @kumarnarendra701

Fixed this issue and implemented the new version of SSL generation on Buzz.

@kumarnarendra701 has this been stable for long enough for us to plan the rollout to Hornet?

@cpswan - I have already rollout to Buzz and informed Sita to perform one round of testing once I ll get the go-ahead then roll out to the hornet.

@ksanty we can close this ticket as https://github.com/atsign-foundation/at_server/issues/232 is open and we can work over there


nixos/qmk: ensure plugdev groups exists

QMK's udev rules are setting hidraw devices's group to plugdev.
Without the plugdev group, attaching a QMK device produces an error message like this:
Dec 06 17:14:25 ru systemd-udevd[1031635]: /nix/store/83da29946qcpxb8v60vmwy770mj9jci1-udev-rules/50-qmk.rules:71 Unknown group 'plugdev', ignoring.

The corresponding udev rule is https://github.com/qmk/qmk_firmware/blob/d915ab5/util/udev/50-qmk.rules#L71
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.05 Release Notes (or backporting 23.05 and 23.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

You would also need to be made a member of that group in order for it to actually do anything for you. I'm not really sure what the best UX would be here. Any thoughts?

There are a few scenarios:

no plugdev group and therefore no user added to the group
plugdev group exists but no user is added to it
plugdev group exists and user is added

In case of scenario 1, we get an error that the group doesn’t exist which is at least some indication that things will not work.
In scenario 2, things will just silently fail.
Scenario 3 obviously works.
I’m thinking something along the lines of a warning in case the group is defined but it has no members.

I’m thinking something along the lines of a warning in case the group is defined but it has no members.

In general that sounds good to me.
There's also the scenario that the wrong user got added to the group, or a user was forgotten to be added to the group. Arguably that's a less likely scenario, but it will also produce a silent error.
Either way, adding safeguards to prevent empty groups seems out of scope for this PR and requires more thought. E.g. will we need the possibility to disable the warning in context of mutable users or DynamicUser=, etc.

The general plugdev handling is a separate issue - let's merge this first


Improving detection for PowerShell (dev)

Codecov Report

Merging #352 (4b16fa2) into dev (de1de13) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #352      +/-   ##
==========================================
- Coverage   67.27%   67.12%   -0.16%
==========================================
  Files         120      120
  Lines       10583    10583
==========================================
- Hits         7120     7104      -16
- Misses       3463     3479      +16

Impacted Files
Coverage Δ

assemblyline/common/identify.py
89.18% <ø> (ø)

assemblyline/common/backupmanager.py
69.58% <0.00%> (-5.84%)
:arrow_down:

assemblyline/datasource/alert.py
88.46% <0.00%> (-3.85%)
:arrow_down:

assemblyline/common/heuristics.py
80.76% <0.00%> (-1.29%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de1de13...4b16fa2. Read the comment docs.