Update managing-a-merge-queue.md

Clarify when merge_group is needed

Why:
The note mentioned both pull_request and push but seems to be relevant only to pull_request.
What's being changed (if available, include any code snippets, screenshots, or gifs):
Clarify the language in Managing a Merge Queue article
Check off the following:

[ ] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[X] For content changes, I have completed the self-review checklist.

@aldeed Thanks so much for opening a PR! I'll get this triaged for review ✨

Hey @aldeed! Thanks for bringing this up.
I had a chat with our engineering team about this change. While it is technically possible, we don't offer support for using the push event like this. The merge_group events are the explicit callbacks to execute required checks, which is why the docs emphasize using the merge_group event. For this reason, I'm going to close this PR and leave the docs as is.

@jc-clark The main problem with the current language is that when we turned on a merge queue and followed these instructions, all our checks ran twice because we already had push on that workflow. So adding merge_group trigger to a workflow that already runs on push is actually incorrect. Feel free to tweak the language, but I was trying to prevent others from spending hours trying to figure out this situation.

Ah, that makes sense @aldeed! And thank you for the deeper context here. I can go back to our engineering team and see what they say.


refactor: change the struct of LeafID

Regression test result
elapsed_milliseconds_ratio_mean=0.814 (current / reference)
The scores are perfectly the same.

I understand the code changes and see the reason for the speedup.
[Before]
After performing a kdtree search and getting the LeafID, feed it to the map to get the Leaf. In other words, it takes O(log N).
https://github.com/tier4/ndt_omp/pull/48/files#diff-9a6400f544541c9ebbf914629eb5208a2ec5fd097a8d62bba8579ecd2a2b658cL168
[After]
leaves_ is now simply std::vector, so we can get the leaf directly.
https://github.com/tier4/ndt_omp/pull/48/files#diff-9a6400f544541c9ebbf914629eb5208a2ec5fd097a8d62bba8579ecd2a2b658cR206

The spikes also seem to be just randomness.

All my concerns have been resolved.

I understand the code changes and see the reason for the speedup.
[Before] After performing a kdtree search and getting the LeafID, feed it to the map to get the Leaf. In other words, it takes O(log N). https://github.com/tier4/ndt_omp/pull/48/files#diff-9a6400f544541c9ebbf914629eb5208a2ec5fd097a8d62bba8579ecd2a2b658cL168
[After] leaves_ is now simply std::vector, so we can get the leaf directly. https://github.com/tier4/ndt_omp/pull/48/files#diff-9a6400f544541c9ebbf914629eb5208a2ec5fd097a8d62bba8579ecd2a2b658cR206

@SakodaShintaro In addition to this one, there are several other improvements

I removed the string index of leaves, since comparing long strings are costly
The getInverseCov, getMean, and similar functions to obtain the matrices of leaves now return reference and const reference to the corresponding matrices of the leaves. This is to avoid unecessary deep copies.

There are other modification in multigrid_ndt_omp_impl.hpp that remove the repetitive computations, but they belong to the speedup_align (merged). When using these modifications together, they reduce the computing cost of align.

@anhnv3991
I renamed leaves_ to leaf_ptrs_.
https://github.com/tier4/ndt_omp/pull/48/commits/9df1d913cfc34ae0dd0cd1bac3a4872d0ce58c74

The spikes also seem to be just randomness.

All my concerns have been resolved.

@SakodaShintaro I am curious about this randomness. Are there any part in your test code that involve random numbers? If that is not the case, then there must be something wrong in the new code. Because NDT is 100% deterministic, the matching results should be exactly the same in every run.


Add Node 0.12 support

Closes #35.
To add Node 0.12 support:

Replaced const and let with var.
Replaced arrow functions with regular old anonymous functions.
Replaced template literals with sad string concatenation.
Added object-assign ponyfill.

@ai: Before I merge and release, would you please try this branch in postcss-load-config to double-check that I caught everything you need?

Wow! You has really fast.
We could test right now on Travis CI. My tests are inside postcss-loader :).
But maybe @michael-ciniawsky could helps us with tests? Could you release new beta with master’s cosmiconfig in dependencies?

@ai: So this is one of the reasons I dropped Node 0.12 support — now I have a situation where Travis will fail unless I downgrade ESLint. Did you figure out a good work around for this?

That tool node-version-check works well for this problem ESLint-Node-version problem.

node-version-check looks awesome!

I think you could release it as a patch update, because there is no public API changes. Thanks for quick solution.


Error when create_feature_extractor in AlexNet

🐛 Describe the bug
When I try to obtain the feature of layer "classifier.4" in AlexNet, the program has reported an error. The code is as follows:
import torch
from torchvision.models import alexnet, AlexNet_Weights
from torchvision.models.feature_extraction import create_feature_extractor

model = alexnet(weights=AlexNet_Weights.IMAGENET1K_V1)
extractor = create_feature_extractor(model, {'classifier.4': 'feat'})
img = torch.rand(3,224,224)
out = extractor(img)

Error message
RuntimeError: mat1 and mat2 shapes cannot be multiplied (256x36 and 9216x4096)

I guess it is because that the shape of output from "flatten" of AlexNet is 256x36 rather than 9216.
Versions
Collecting environment information...
PyTorch version: 1.12.0+cu116
Is debug build: False
CUDA used to build PyTorch: 11.6
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.4 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.31

Python version: 3.9.12 (main, Jun  1 2022, 11:38:51)  [GCC 7.5.0] (64-bit runtime)
Python platform: Linux-5.4.0-117-generic-x86_64-with-glibc2.31
Is CUDA available: True
CUDA runtime version: 11.6.55
GPU models and configuration:
GPU 0: NVIDIA GeForce RTX 3090
GPU 1: NVIDIA GeForce RTX 3090
GPU 2: NVIDIA GeForce RTX 3090
GPU 3: NVIDIA GeForce RTX 3090
GPU 4: NVIDIA GeForce RTX 3090
GPU 5: NVIDIA GeForce RTX 3090
GPU 6: NVIDIA GeForce RTX 3090
GPU 7: NVIDIA GeForce RTX 3090

Nvidia driver version: 510.73.08
cuDNN version: Probably one of the following:
/usr/local/cuda-11.6/targets/x86_64-linux/lib/libcudnn.so.8.4.0
/usr/local/cuda-11.6/targets/x86_64-linux/lib/libcudnn_adv_infer.so.8.4.0
/usr/local/cuda-11.6/targets/x86_64-linux/lib/libcudnn_adv_train.so.8.4.0
/usr/local/cuda-11.6/targets/x86_64-linux/lib/libcudnn_cnn_infer.so.8.4.0
/usr/local/cuda-11.6/targets/x86_64-linux/lib/libcudnn_cnn_train.so.8.4.0
/usr/local/cuda-11.6/targets/x86_64-linux/lib/libcudnn_ops_infer.so.8.4.0
/usr/local/cuda-11.6/targets/x86_64-linux/lib/libcudnn_ops_train.so.8.4.0
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

Versions of relevant libraries:
[pip3] numpy==1.22.3
[pip3] torch==1.12.0+cu116
[pip3] torchmetrics==0.9.1
[pip3] torchtext==0.12.0
[pip3] torchvision==0.13.0+cu116
[conda] blas                      1.0                         mkl    defaults
[conda] cudatoolkit               11.3.1               h2bc3f7f_2    defaults
[conda] mkl                       2021.4.0           h06a4308_640    defaults
[conda] mkl-service               2.4.0            py39h7f8727e_0    defaults
[conda] mkl_fft                   1.3.1            py39hd3c417c_0    defaults
[conda] mkl_random                1.2.2            py39h51133e4_0    defaults
[conda] numpy                     1.22.3           py39he7a7128_0    defaults
[conda] numpy-base                1.22.3           py39hf524024_0    defaults
[conda] pytorch-mutex             1.0                        cuda    pytorch
[conda] torch                     1.12.0+cu116             pypi_0    pypi
[conda] torchmetrics              0.9.1                    pypi_0    pypi
[conda] torchtext                 0.12.0                     py39    pytorch
[conda] torchvision               0.13.0+cu116             pypi_0    pypi

@githwd2016 Thans for reporting.
The problem is you are throwing in a single image while the model expects a batch. Replace what you have with the following and it will work:
img = torch.rand(1, 3,224,224)
``

To confirm this is the case, remove completely the `create_feature_extractor` from your script and pass the image to the original model. If you do that, you will get the same error.

I believe this should solve your problem so I'll close the issue. If you continue facing problems feel free to reopen.


Opensuse leap 15.4 certificate cert.pem is invalid. Broken link http://bit.ly/ruby-ssl

Hello!
I'm running opensuse leap 15.4, and tried to install rbenv to manage my ruby versions. Once upon I installed ruby 2.7.2, when I tried to bundle install the gems of my project, I've got this error Could not verify the SSL certificate for https://rubygems.org/, and tried to reach the link http://bit.ly/ruby-ssl the link was broken. Opensuse leap 15.4 has currently openssl 1.1.1 in its official repository.
Before installing ruby 2.7.2, I compiled https://github.com/openssl/openssl/releases/tag/OpenSSL_1_1_1q with these flags: ./Configure --prefix=/opt/openssl_1_1_1q --openssldir=/opt/openssl_1_1_1q/ssl linux-x86_64 to install it in an isolated directory. Then I set the environment variable RUBY_CONFIGURE_OPTS=--with-openssl-dir=/opt/openssl_1_1_1q/ and finally rbenv install 2.7.2
After this build, I ran the command bundle install and stumble with the problem described on the first paragraph. Here is a screenshot:

I followed this guide, but I broke so many things in the process, even broke my system ruby when running gem update --system (with sudo) so I had to reinstall my OS.
After some tinkering, I finally found out that the cert.pem certificate was outdated. So I took this certificate: https://raw.githubusercontent.com/rubygems/rubygems/master/lib/rubygems/ssl_certs/rubygems.org/GlobalSignRootCA_R3.pem and save it under the file in /opt/openssl_1_1_1q/ssl/cert.pem. I discovered that was the right name and place for the file because after running the command curl -Lks 'https://git.io/rg-ssl' | ruby mentiond in the guide above, I've got this reply from the terminal:

:warning: :warning: :warning: :warning:  the screenshot has the route /opt/openssl_1_1_1d/ssl/cert.pem , but it should be /opt/openssl_1_1_1q/ssl/cert.pem, the screenshot is from another test I made. But it's the same process. :warning: :warning: :warning: :warning:
Once the file is created, I could connect to rubygems.

Thanks for sharing your problem and workaround!
Closing because rbenv itself doesn't deal with building Ruby nor running RubyGems. Therefore, this is a problem with how the OpenSSL installation itself is managed.

Hi @mislav , I understand the issue is off-topic here. Maybe I should post it on the ruby-build plugin repo?
Just wanted to give the heads-up, because this year I've had to change my laptop, and make a fresh install of rbenv in Ubuntu 22.04 and Open Suse Leap 15.4, in both OS I've had issues with openssl, I think it is because on Ubuntu they started using openssl 3.x on the official repository. IMHO a bad decision, because it is breaking a lot of stuff. That's the reason why I then switched back to OpenSuse that uses more stable software.

@KarlHeitmann Please see https://github.com/rbenv/ruby-build/issues/2043; maybe it's related?

Thank you, it may be related, I'll keep an eye on that thread. To see if I can do something there.


Added unit modifiers to master version

Added unit modifiers to the master branch.

Merging the new version, but will not update HED latest until all of the validators catch up.


Feature request: Ability to use folders without md-files

Sometimes I need to store images and other files outside the root page folder. See:

I don't need any md in it just because sometimes needs to delete files via FTP and can accidentally delete it.
But without md Media Manager don't work:
NOTE: You cannot add media files until you save the page. Just click 'Save' on top
Also I can't create nested folders without putting md in EVERY folder. See:

Grav don't have any option to specify template for nested folders.
Any ideas?

Currently, Pages expect a .md file into them, otherwise Admin thinks it's not a saved page yet. You can still make this work using file system / FTP instead of Admin, but I think this can be solved with Admin Pro which will feature a more advanced media management functionality.

Hmm.. this works for me, just create "add folder" rather than "add page":

This is actually an admin feature request.  Grav-core itself supports this fine, it's only the admin that doesn't.  Can you please re-open this issue here: https://github.com/getgrav/grav-plugin-admin/issues

@rhukster It also affects Grav core if we want to extend frontmatter options for nested folders template.
Ok. Done in Admin panel:
https://github.com/getgrav/grav-plugin-admin/issues/1000


[WIP] Enable alerts definitions with datasource for Middleware Server

Need https://github.com/ManageIQ/manageiq-ui-classic/pull/2309
Tasks

[X] DataSource - Connections Available
[X] DataSource - Connections In Use
[X] DataSource - Connections Timed Out
[X] DataSource - Connection Get Time
[X] DataSource - Connection Creation Time
[X] DataSource - Connection Wait Time

Let use operators <,<=,> and >=

Datasource connection create time average

Datasource connection get time average

Datasource connection max wait time

Datasource number available connections

Datasource number connections in use

Datasource number connections time out

@lucasponce  Can you please take a look? I think that Get Time, Creation Time and Wait Time is based in ms, maybe we should put ( ms ).
https://issues.jboss.org/browse/HAWKULAR-1255

@miq-bot add-label providers/middleware
@miq-bot add-label wip

@miq-bot remove-label wip

Thanks @gtanzillo for merge


Get Adrees of incoming message

How do I get the adress of a incoming message in fixed mode?

When in fixed mode you are broadcasting to any recievers on the correct channel and address. This can still be multiple devices. If you do not include any information in your broadcast about your own address, the recievers will not be able to tell where the message originated.
In your case you may wish to pre-fix the message data with the senders channel and address to obtain the senders address.


Question: Broken pipe

Hello, I have this error and I don't know how to fix it.
2023/09/24 05:16:14 INFO Searching for devices...
2023/09/24 05:16:15 INFO Connected to cast device. device="HUB Enrico"
2023/09/24 05:16:19 INFO Connected to cast device. device="Enrico TV"
2023/09/24 06:19:52 ERROR Lost connection to device. device="HUB Enrico" error="unable to write binary format: write tcp <IP_ADDRESS>:51232-><IP_ADDRESS>:8009: write: broken pipe"
2023/09/24 06:21:15 INFO Connected to cast device. device="HUB Enrico"

Getting this too after switching to a somewhat unstable wifi connection on the host, always put it down to that.
Try decreasing the discovery time as a workaround so your CC gets re-discovered faster.

Getting this too after switching to a somewhat unstable wifi connection on the host, always put it down to that.
Try decreasing the discovery time as a workaround so your CC gets re-discovered faster.

Thank you, I changed it to 2m in the config and still the same issue. My wifi is stable, I run my own dns and dhcp.

I meant that as a workaround until the issue is fixed. Doesn't help with the broken pipes, but your CC gets re-discovered faster after it so most times you don't notice it.

I've occasionally seen this too. The only thing I can think of is maybe the Cast device is expecting some sort of keepalive message, and when it doesn't get it, it closes the connection? I'll see if I can make this more reliable.
Do you get the error more often for a specific device?
In the mean time, the advice from @schumi4 is perfect. Any failed connection will be retried when the discovery is restarted.

Follow up to this:
I was used to seeing this all the time (sometimes 10+ times in one hour) when running over wifi on a rpi. Transitioned to running the docker container directly on my router and have had it happen maybe once or twice over a week. Pretty sure this is dUe to poor network connection on either side, CSS or CC.

I've occasionally seen this too. The only thing I can think of is maybe the Cast device is expecting some sort of keepalive message, and when it doesn't get it, it closes the connection? I'll see if I can make this more reliable.
Do you get the error more often for a specific device?
In the mean time, the advice from @schumi4 is perfect. Any failed connection will be retried when the discovery is restarted.

I think it's about the same.
I run it on a RPI4b with homeassistant, it's running as an addon.
The rpi is connected with ethernet, the two cast devices are wifi 2.4ghz connected. One is at 50cm from the router the other at 2 meter.
Starting...
/etc/cont-init.d/00-banner.sh: executing
-----------------------------------------------------------
 Add-on: Castsponsorskip
 skips sponsored YouTube content on all local Chromecasts
-----------------------------------------------------------
 Add-on version: 0.5.0
 You are running the latest version of this add-on.
 System: Home Assistant OS 10.5  (aarch64 / raspberrypi4-64)
 Home Assistant Core: 2023.9.3
 Home Assistant Supervisor: 2023.09.2
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums
-----------------------------------------------------------
 Provided by: https://github.com/alexbelgium/hassio-addons
-----------------------------------------------------------
/etc/cont-init.d/00-global_var.sh: executing
CSS_CATEGORIES='sponsor, into, outro, selfpromo'
CSS_DISCOVER_INTERVAL='1m'
CSS_MUTE_ADS='true'
CSS_YOUTUBE_API_KEY='AIzaSyDkOg9CzT9Z298HFHTfCHNwVMukgbpH89U'
/etc/cont-init.d/01-custom_script.sh: executing
[23:40:16] INFO: Execute /config/addons_autoscripts/sponsorblockcast.sh if existing
[23:40:17] INFO: ... no script found
/etc/cont-init.d/99-run.sh: executing
[23:40:22] INFO: Starting app
2023/09/26 23:40:22 INFO CastSponsorSkip v0.5.0 (73d1f7a)
2023/09/26 23:40:22 INFO Searching for devices...
2023/09/26 23:40:23 INFO Connected to cast device. device="HUB Enrico"
2023/09/27 03:58:14 ERROR Failed to connect to device. device="Enrico TV" error="device not found: "
2023/09/27 05:16:25 ERROR Lost connection to device. device="HUB Enrico" error="context deadline exceeded"
2023/09/27 05:18:13 ERROR Failed to connect to device. device="HUB Enrico" error="unable to connect to chromecast at '<IP_ADDRESS>:8009': dial tcp <IP_ADDRESS>:8009: i/o timeout"
2023/09/27 05:18:37 INFO Connected to cast device. device="HUB Enrico"
2023/09/28 03:56:14 ERROR Lost connection to device. device="HUB Enrico" error="unable to write binary format: write tcp <IP_ADDRESS>:60680-><IP_ADDRESS>:8009: write: broken pipe"
2023/09/28 03:56:24 INFO Connected to cast device. device="HUB Enrico"
2023/09/29 02:27:01 ERROR Lost connection to device. device="HUB Enrico" error="unable to write binary format: write tcp <IP_ADDRESS>:55726-><IP_ADDRESS>:8009: write: broken pipe"
2023/09/29 02:27:26 INFO Connected to cast device. device="HUB Enrico"
2023/09/30 02:40:03 ERROR Lost connection to device. device="HUB Enrico" error="unable to write binary format: write tcp <IP_ADDRESS>:32776-><IP_ADDRESS>:8009: write: broken pipe"
2023/09/30 02:40:27 INFO Connected to cast device. device="HUB Enrico"
2023/09/30 18:40:29 ERROR Lost connection to device. device="HUB Enrico" error="context deadline exceeded"
2023/09/30 18:41:28 INFO Connected to cast device. device="HUB Enrico"
2023/10/01 04:59:06 ERROR Lost connection to device. device="HUB Enrico" error="unable to write binary format: write tcp <IP_ADDRESS>:44510-><IP_ADDRESS>:8009: write: broken pipe"
2023/10/01 04:59:30 INFO Connected to cast device. device="HUB Enrico"
2023/10/01 14:48:22 INFO Detected video stream. device="HUB Enrico" video_id=lpEc5nHqO2c
2023/10/01 14:48:22 INFO No segments found for video. device="HUB Enrico" video_id=lpEc5nHqO2c
2023/10/01 14:51:32 INFO Detected video stream. device="HUB Enrico" video_id=K_Z1i8lVYj4
2023/10/01 14:51:32 INFO No segments found for video. device="HUB Enrico" video_id=K_Z1i8lVYj4
2023/10/02 02:45:31 ERROR Lost connection to device. device="HUB Enrico" error="unable to write binary format: write tcp <IP_ADDRESS>:60638-><IP_ADDRESS>:8009: write: broken pipe"
2023/10/02 02:45:33 INFO Connected to cast device. device="HUB Enrico"
2023/10/03 03:39:25 ERROR Lost connection to device. device="HUB Enrico" error="unable to write binary format: write tcp <IP_ADDRESS>:33882-><IP_ADDRESS>:8009: write: broken pipe"
2023/10/03 03:39:34 INFO Connected to cast device. device="HUB Enrico"

I've looked at some other Cast-related projects and it seems like this normal. Apparently a lot of Cast devices reboot overnight and install updates randomly. It looks like the general fix is to increase the number or retries and don't log the first few failures. I'll go ahead and do that for now. I'm tired of seeing these logs!

I'm testing an update locally which will attempt a full reconnect after a few failures in a row. If this fixes the issue, I'll go ahead and release it 🤞

@Ryckie @schumi4 This has been fixed in v0.5.2! Let me know if you still see any issues, but I've been running this update for a full day now with 0 broken pipe errors 🙂

I realized I should give some more technical info on the fix.

Before v0.5.2: if the device disconnected, CastSponsorSkip would retry a few times, then give up so that the device could be discovered during the next discovery. This could take a few minutes and resulted in the log messages you saw.
Starting with v0.5.2: when a device disconnects, CastSponsorSkip will instantly create a new connection to the device, and will only log an error if that reconnection fails. This seems to match the behavior of other Cast-related apps and has been working well for me. If the reconnection fails, it will still log an error and wait for discovery, but so far that hasn't happened for me!

@Ryckie @schumi4 I'd also suggest raising your discover interval back up to 5m! It doesn't hurt to keep it lower, but if the connection is stable then it's not necessary anymore.

It still doesn't rediscover for me until I restart the service after chromecast turned on, chromecast gen 1
Use case is TV and chromecast are powered off, only turn on at morning and night.
Oct 18 18:41:11 aml castsponsorskip[6959]: 2023/10/18 18:41:11 INFO CastSponsorSkip v0.5.3 (85f505b)
Oct 18 18:41:11 aml castsponsorskip[6959]: 2023/10/18 18:41:11 INFO Searching for devices...
Oct 18 18:41:11 aml castsponsorskip[6959]: 2023/10/18 18:41:11 INFO Connected to cast device. device="Master Bedroom TV"
Oct 18 18:41:12 aml castsponsorskip[6959]: 2023/10/18 18:41:12 INFO Detected video stream. device="Master Bedroom TV" video_id=LK6XFIHJQMg
Oct 18 18:41:13 aml castsponsorskip[6959]: 2023/10/18 18:41:13 INFO Found segments for video. device="Master Bedroom TV" segments=2
Oct 18 18:43:38 aml castsponsorskip[6959]: 2023/10/18 18:43:38 INFO Detected ad. Muting and attempting to skip... device="Master Bedroom TV"
Oct 18 18:43:52 aml castsponsorskip[6959]: 2023/10/18 18:43:52 INFO Skipped ad. device="Master Bedroom TV"
Oct 18 18:45:39 aml castsponsorskip[6959]: 2023/10/18 18:45:39 INFO Skipping to timestamp. device="Master Bedroom TV" category=sponsor from=1m43s to=2m14s
Oct 18 18:48:30 aml castsponsorskip[6959]: 2023/10/18 18:48:30 ERROR Lost connection to device. device="Master Bedroom TV" error="unable to write binary format: write tcp <IP_ADDRESS>:38068-><IP_ADDRESS>:8009: write: broken pipe"

already waited 10 minutes.

@leewhat Hmm I'll have to test taking some devices offline, but it should rediscover the device immediately when it comes back online. So it won't ever rediscover when you turn them back on?

@gabe565 yeah, it doesnt rediscover, I let the it ran wholeday.
here is the logs
-- Logs begin at Fri 2023-10-20 03:17:01 WIB, end at Fri 2023-10-20 15:37:44 WIB. --
Oct 20 06:56:29 aml systemd[1]: Stopping Skip YouTube ads and sponsorships on all local Google Cast devices...
Oct 20 06:56:29 aml castsponsorskip[18920]: 2023/10/20 06:56:29 INFO Gracefully closing connections... Press Ctrl+C again to force exit.
Oct 20 06:56:29 aml castsponsorskip[18920]: 2023/10/20 06:56:29 INFO Exiting.
Oct 20 06:56:29 aml systemd[1]: castsponsorskip.service: Succeeded.
Oct 20 06:56:29 aml systemd[1]: Stopped Skip YouTube ads and sponsorships on all local Google Cast devices.
Oct 20 15:29:34 aml systemd[1]: Started Skip YouTube ads and sponsorships on all local Google Cast devices.
Oct 20 15:29:34 aml castsponsorskip[16859]: time=2023-10-20T15:29:34.731+07:00 level=INFO msg="CastSponsorSkip v0.5.3 (85f505b)"
Oct 20 15:29:34 aml castsponsorskip[16859]: time=2023-10-20T15:29:34.732+07:00 level=INFO msg="Searching for devices..."
Oct 20 15:30:16 aml castsponsorskip[16859]: time=2023-10-20T15:30:16.972+07:00 level=DEBUG msg="Failed to connect to device. Retrying..." device="Master Bedroom TV" try=0 error="unable to connect to chromecast at '<IP_ADDRESS>:8009': read tcp <IP_ADDRESS>:58816-><IP_ADDRESS>:8009: read: connection reset by peer"
Oct 20 15:30:18 aml castsponsorskip[16859]: time=2023-10-20T15:30:18.578+07:00 level=INFO msg="Connected to cast device." device="Master Bedroom TV"
Oct 20 15:30:19 aml castsponsorskip[16859]: time=2023-10-20T15:30:19.079+07:00 level=DEBUG msg=Update device="Master Bedroom TV"
Oct 20 15:31:19 aml castsponsorskip[16859]: time=2023-10-20T15:31:19.089+07:00 level=DEBUG msg=Update device="Master Bedroom TV"
Oct 20 15:31:35 aml castsponsorskip[16859]: time=2023-10-20T15:31:35.873+07:00 level=DEBUG msg=Update device="Master Bedroom TV"
Oct 20 15:31:46 aml castsponsorskip[16859]: time=2023-10-20T15:31:46.723+07:00 level=DEBUG msg=Update device="Master Bedroom TV"
Oct 20 15:31:46 aml castsponsorskip[16859]: time=2023-10-20T15:31:46.966+07:00 level=INFO msg="Detected video stream." device="Master Bedroom TV" video_id=LK6XFIHJQMg
Oct 20 15:31:46 aml castsponsorskip[16859]: time=2023-10-20T15:31:46.967+07:00 level=DEBUG msg="Request segments" url="https://sponsor.ajay.app/api/skipSegments/f672?actionType=skip&actionType=mute&category=sponsor"
Oct 20 15:31:47 aml castsponsorskip[16859]: time=2023-10-20T15:31:47.222+07:00 level=DEBUG msg=Update device="Master Bedroom TV"
Oct 20 15:31:47 aml castsponsorskip[16859]: time=2023-10-20T15:31:47.722+07:00 level=DEBUG msg=Update device="Master Bedroom TV"
Oct 20 15:31:48 aml castsponsorskip[16859]: time=2023-10-20T15:31:48.059+07:00 level=INFO msg="Found segments for video."  Reconnecting..." device="Master Bedroom TV" error="context deadline exceeded"
Oct 20 15:32:43 aml castsponsorskip[16859]: time=2023-10-20T15:32:43.739+07:00 level=DEBUG msg="Failed to connect to device. Retrying..." device="Master Bedroom TV" try=0 error="unable to connect to chromecast at '<IP_ADDRESS>:8009': dial tcp <IP_ADDRESS>:8009: i/o timeout"
Oct 20 15:32:49 aml castsponsorskip[16859]: time=2023-10-20T15:32:49.242+07:00 level=DEBUG msg="Failed to connect to device. Retrying..." device="Master Bedroom TV" try=1 error="unable to connect to chromecast at '[<nil>]:0': dial tcp: lookup <nil>: no such host"
Oct 20 15:32:55 aml castsponsorskip[16859]: time=2023-10-20T15:32:55.245+07:00 level=DEBUG msg="Failed to connect to device. Retrying..." device="Master Bedroom TV" try=2 error="unable to connect to chromecast at '[<nil>]:0': dial tcp: lookup <nil>: no such host"
Oct 20 15:33:02 aml castsponsorskip[16859]: time=2023-10-20T15:33:02.248+07:00 level=DEBUG msg="Failed to connect to device. Retrying..." device="Master Bedroom TV" try=3 error="unable to connect to chromecast at '[<nil>]:0': dial tcp: lookup <nil>: no such host"
Oct 20 15:33:11 aml castsponsorskip[16859]: time=2023-10-20T15:33:11.249+07:00 level=DEBUG msg="Failed to connect to device. Retrying..." device="Master Bedroom TV" try=4 error="unable to connect to chromecast at '[<nil>]:0': dial tcp: lookup <nil>: no such host"
Oct 20 15:33:24 aml castsponsorskip[16859]: time=2023-10-20T15:33:24.252+07:00 level=DEBUG msg="Failed to connect to device. Retrying..." device="Master Bedroom TV" try=5 error="unable to connect to chromecast at '[<nil>]:0': dial tcp: lookup <nil>: no such host"
Oct 20 15:33:45 aml castsponsorskip[16859]: time=2023-10-20T15:33:45.254+07:00 level=DEBUG msg="Failed to reconnect." device="Master Bedroom TV" error="device not found: "
Oct 20 15:33:45 aml castsponsorskip[16859]: time=2023-10-20T15:33:45.254+07:00 level=ERROR msg="Lost connection to device." device="Master Bedroom TV" error="context deadline exceeded"
Oct 20 15:33:45 aml castsponsorskip[16859]: time=2023-10-20T15:33:45.255+07:00 level=DEBUG msg="Panic during close" device="Master Bedroom TV" error="runtime error: invalid memory address or nil pointer dereference" stack="goroutine 22 [running]:\nruntime/debug.Stack()\n\truntime/debug/stack.go:24 +0x64\ngithub.com/gabe565/castsponsorskip/internal/device.(*Device).closeApp.func1()\n\tgithub.com/gabe565/castsponsorskip/internal/device/watch.go:120 +0x40\npanic({0x72bd80?, 0xe1a5a0?})\n\truntime/panic.go:914 +0x218\ncrypto/tls.(*Conn).Close(0x71fc60?)\n\tcrypto/tls/conn.go:1403 +0x18\ngithub.com/vishen/go-chromecast/cast.(*Connection).Close(0x0?)\n\tgithub.com/vishen/go-chromecast@v0.3.2-0.20231012231916-ce26f51593c1/cast/connection.go:82 +0x94\ngithub.com/vishen/go-chromecast/application.(*Application).Close(0x4000061e78?, 0x24?)\n\tgithub.com/vishen/go-chromecast@v0.3.2-0.20231012231916-ce26f51593c1/application/application.go:431 +0x110\ngithub.com/gabe565/castsponsorskip/internal/device.(*Device).closeApp(0x0?)\n\tgithub.com/gabe565/castsponsorskip/internal/device/watch.go:124 +0x5c\ngithub.com/gabe565/castsponsorskip/internal/device.(*Device).Close(0x400023b080)\n\tgithub.com/gabe565/castsponsorskip/internal/device/watch.go:113 +0x84\ngithub.com/gabe565/castsponsorskip/cmd.run.func1.1()\n\tgithub.com/gabe565/castsponsorskip/cmd/cmd.go:104 +0xa4\ncreated by github.com/gabe565/castsponsorskip/cmd.run.func1 in goroutine 35\n\tgithub.com/gabe565/castsponsorskip/cmd/cmd.go:101 +0x50\n"
Oct 20 15:34:34 aml castsponsorskip[16859]: time=2023-10-20T15:34:34.742+07:00 level=DEBUG msg="Ignoring device." device="Master Bedroom TV" reason="Already connected"

@leewhat Thanks for the log! Was the end when you turned the device back on? If so, I think I know what's going on here and I can fix it

When I turn it on, there is no extra log.

@leewhat I believe I found the root cause of this. It has been fixed in v0.6.1. Would you mind verifying?

@gabe565 it works now, thank you!

@leewhat That's great to hear!


make github account

Daniel, Shmuel, Yonatan Open Git hub account
Amichai - will need to add you to the repository as contributors

Please mark this as done once you are all able to access this reposetory

Great!
Good practice is to add a comment when you close the issue.
Explaining what was done. What was fixed, enhanced, modified, documented, etc.


feat(wrangler): Updated Queues client to use new 'by-id' endpoints.

What this PR solves / how to test
Updates the Queues client to use new 'by-id' endpoints rather than the old 'by name' endpoints, which are in deprecation.
Fixes # MQ-553 internally
Author has addressed the following

Tests

[ ] TODO (before merge)
[x] Included
[ ] Not necessary because:

Changeset (Changeset guidelines)

[ ] TODO (before merge)
[x] Included
[ ] Not necessary because:

Public documentation

[ ] TODO (before merge)
[ ] Cloudflare docs PR(s): https://github.com/cloudflare/cloudflare-docs/pull/...
[x] Not necessary because: no change in the way wrangler is used, this is transparent.

LGTM. Have these changes been validated by someone on the queues team as well?


Update information-barriers-in-teams.md

Added a link to procedure article and made a few other little changes/fixes.

@MicrosoftHeidi thank you!


Use pooch

Start transition to pooch for fetching data.
The initial intent was to get rid completely of the example_utils in the docs, so that each notebook could be downloaded and run directly. The first step to achieve this is to move the code for fetching the data in src/nemos, and use pooch to download from osf.
There is one tutorial that relies on dandi, and that's a big dependency that I would rather not have as main dependency of nemos.
What should we do for that dataset?

In general, this looks good. Thoughts:
* Why not just put `download_dandi_data` into our fetch module? currently, we're only using it once, but it's feasible we'd use it more

I thought about it, it's a package with a lot of dependencies, and I was hesitant, moving it into the main package would make it a requirement. pynapple has it has an optional dependency, @gviejo what do you think about making dandi a dependency?
My personal inclination is not to add it as a requirement for downloading a single dataset.

* I don't love using pooch to grab code -- pooch feels more like it's meant for binary files, and just dropping random scripts into the users working directory feels fragile. I'm not sure of a better solution right now, but I posted in the US-RSE slack to see if anyone has suggestions

I agree with you, I did not like people having to manually download the scripts, and place it in the correct folder structure but I did not find a clear optimal solution. This one at least is safer because of the hashing and the download of the script based on the package version.

* If we _do_ end up going with pooch to get the utility scripts, I think we shouldn't use `if not path.exists`, but a try/except on the import.

Ok! let's wait until we get an opinion from the US-RSE people and finish this off.

Just pushed some updates here, following conversation on US-RSE slack, which we should review:

Pooch is now an optional dependency, which changes a bit of how fetch_data.py is structured. If the user tries to call fetch_data without pooch installed, they'll get an error telling them to install it.
moves download_dandi_data into fetch_data.py, which behaves similarly to fetch_data, but with dandi as the optional dependency instead of pooch
big restructure: makes the functions contained in the different *_utils/plotting.py scripts part of a hidden module, _documentation_utils. This is not imported in nemos/__init__.py  and so needs to be specifically imported: from nemos import _documentation_utils. This means it can contain import matplotlib (also an optional dependency), though we make sure to raise a more informative error message if the user manages to call it without it installed.

I think this ends up being cleaner, both for us and for users.
Those plotting functions in _documentation_utils should probably all have full docstrings now though, which should, among other things say WARNING: this function is intended for use in the documentation, and is not generalizable. It may not work well with other data or something similar

Codecov Report
Attention: Patch coverage is 7.45342% with 298 lines in your changes missing coverage. Please review.

Project coverage is 76.04%. Comparing base (32132b1) to head (69eaed7).
Report is 211 commits behind head on development.

Files with missing lines
Patch %
Lines

src/nemos/_documentation_utils/plotting.py
0.00%
270 Missing :warning:

src/nemos/fetch/fetch_data.py
43.75%
27 Missing :warning:

src/nemos/_documentation_utils/__init__.py
0.00%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (32132b1) and HEAD (69eaed7). Click for more details.
HEAD has 7 uploads less than BASE

Flag
BASE (32132b1)
HEAD (69eaed7)

9
2

Additional details and impacted files
@@               Coverage Diff                @@
##           development     #216       +/-   ##
================================================
- Coverage        97.30%   76.04%   -21.27%
================================================
  Files               18       24        +6
  Lines             1669     2350      +681
================================================
+ Hits              1624     1787      +163
- Misses              45      563      +518

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix: Speed up CircleCI integration test steps

What
The rails assets:precompile in unit tests takes <10s but in integration tests it was taking 2m+
This PR resequences the integration test steps - precompile now runs before installing chromedrivers, etc
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

Can't say i understand why (more assets to precompile due to browser tools?) but can see that it is fast to precompile so 👍

I suspect it's because it added chrome drivers, after restroing the cache and that invalidated the cache so it had to pull everything down again. 🤷♂


Silence the grep used for the version check.

Otherwise is generated output, and if you happen to have something like
"GREP_OPTIONS=--colour=auto" in your env, you get this color block when
this action happen.

@tbkka I believe this option on grep is standard, so it should work fine on Linux also.

Switch to /dev/null just to be safe.  tx.


Visual Studio build of vcpkg omits 11 source files, 152 test cases & 346 assertions

Describe the bug

vcpkgtest.vcxproj does not include all .cpp files in toolsrc\src\vcpkg-test
not all tests are compiled and run via Visual Studio

Environment

OS: Windows 10 Pro 2004 (OS Build 19041.329)
Compiler: see below

	Microsoft Visual Studio Community 2019
	Version 16.6.5
	VisualStudio.16.Release/16.6.5+30320.27
	Microsoft .NET Framework
	Version 4.8.04084

	Installed Version: Community

	Visual C++ 2019   00435-60000-00000-AA319
	Microsoft Visual C++ 2019

To Reproduce
Steps to reproduce the behavior:
A. Run all tests in Visual Studio, from vcpkg.sln

git checkout 0896cfcc301843b455bf14497a15b0e775735198
Open toolsrc\vcpkg.sln in Visual Studio
Do a Debug build
Run vcpkg-test

The output ends with:
===============================================================================
All tests passed (2616 assertions in 103 test cases)

B. Run all tests from a CMake-created build

git checkout 0896cfcc301843b455bf14497a15b0e775735198
For example, open the folder in CLion
Do a Debug build
Run vcpkg-test, from the console, or without using CLion's Catch2 adapter, so you see the total number of tests:

The output ends with:
===============================================================================
All tests passed (2962 assertions in 155 test cases)

Expected behavior
The same number of tests run, regardless of build method.
Additional context
Perhaps someone has forgotten to add new test source files to vcpkgtest.vcxproj?
When built from CMake, these test source files are loaded:

toolsrc\src\vcpkg-test\arguments.cpp
toolsrc\src\vcpkg-test\binarycaching.cpp
toolsrc\src\vcpkg-test\binaryconfigparser.cpp
toolsrc\src\vcpkg-test\catch.cpp
toolsrc\src\vcpkg-test\chrono.cpp
toolsrc\src\vcpkg-test\commands.build.cpp
toolsrc\src\vcpkg-test\commands.create.cpp
toolsrc\src\vcpkg-test\dependencies.cpp
toolsrc\src\vcpkg-test\files.cpp
toolsrc\src\vcpkg-test\hash.cpp
toolsrc\src\vcpkg-test\json.cpp
toolsrc\src\vcpkg-test\manifests.cpp
toolsrc\src\vcpkg-test\mockcmakevarsprovider.cpp
toolsrc\src\vcpkg-test\optional.cpp
toolsrc\src\vcpkg-test\paragraph.cpp
toolsrc\src\vcpkg-test\plan.cpp
toolsrc\src\vcpkg-test\specifier.cpp
toolsrc\src\vcpkg-test\statusparagraphs.cpp
toolsrc\src\vcpkg-test\strings.cpp
toolsrc\src\vcpkg-test\stringview.cpp
toolsrc\src\vcpkg-test\system.cpp
toolsrc\src\vcpkg-test\uint128.cpp
toolsrc\src\vcpkg-test\update.cpp
toolsrc\src\vcpkg-test\util.cpp

In Visual Studio, these test source files are loaded:
toolsrc\src\vcpkg-test\arguments.cpp
toolsrc\src\vcpkg-test\catch.cpp
toolsrc\src\vcpkg-test\chrono.cpp
toolsrc\src\vcpkg-test\dependencies.cpp
toolsrc\src\vcpkg-test\files.cpp
toolsrc\src\vcpkg-test\mockcmakevarsprovider.cpp
toolsrc\src\vcpkg-test\paragraph.cpp
toolsrc\src\vcpkg-test\plan.cpp
toolsrc\src\vcpkg-test\specifier.cpp
toolsrc\src\vcpkg-test\statusparagraphs.cpp
toolsrc\src\vcpkg-test\strings.cpp
toolsrc\src\vcpkg-test\update.cpp
toolsrc\src\vcpkg-test\util.cpp

/CC @strega-nil for info...

This is the issue with having two build systems... I'll fix it

Hi @strega-nil

This is the issue with having two build systems... I'll fix it

It’s maybe more to do with one of those systems being manually maintained, and people having to remember to do that.
Since the changing text in the .vcxproj - and maybe filter files too - only contains file-names, here is a CMake idea for a permanent [*] fix, to prevent anyone ever needing to manually maintain lists of files in these files in future:

Write code to teach the CMakeLists.txt files to generate the sections of VS configuration files that list files
Enable CMake to generate the content of the VS files with updated file list spliced in
Make CMake actually only overwrite the existing VS files if their content would be different (to prevent the annoyance of VS keeping on saying the project/solution has changed, every time CMake runs.

This way, after a source file is added, removed or renamed, as soon as CMake is rerun, the VS config will be updated, and show up as a diff in git. Usually I suspect those files would then be committed as part of the changeset that did the adding/moving/renaming.
By only auto-generating the file lists, and not all the compiler config, etc, this idea allows for all other VS configuration changes to be made in the IDE, as currently.
[*] Permanent until the VS team changes the formatting of the vcxproj/filter files (speaking from personal experience of about 15 years of maintaining scripts that do just this sort of thing to VS files!)

Visual Studio build of vcpkg no longer exists.


Relax bounds for AsRef.

I think the Type and Serialize parts of the AsRef<Path> + Type + Serialize weren't needed, because the path just gets turned into a byte slice anyway (which implements Type and Serialize).

Thanks!

I have released 0.2.6 with the changes on this PR in case you need it.


Upgrade ingress-nginx chart

Is your feature request related to a problem? Please describe.
From:
1.1.2
4.0.18

to:
appVersion: 1.7.1
version: 4.6.1

See: https://artifacthub.io/packages/helm/ingress-nginx/ingress-nginx
Describe the solution you'd like A clear and concise description of what you want to happen.
Describe alternatives you've considered A clear and concise description of any alternative solutions or features
you've considered.
Additional context Add any other context or screenshots about the feature request here.

Upgraded ingress-nginx chart to version 4.6.1 which uses nginx ingress version 1.7.1.
Tested upgrade path from v0.21.6 and v0.22.0 without any issues.
Created PR#1176


method is_callback_class_valid, db SELECTs

refactor SELECTs to class like CL_OO_CLASSNAME_SERVICE, not sure exactly what classes will work, dont have access to a system right now
alternatively, can an interface be added? so instead of checking the class, the class implements an interface?

closed, moving to #63


Upper triangular region of the matrix not filled up by the spectral_connectivity function

I am experiencing a problem with the spectral_connectivity function from the connectivity module. The outputted matrix's upper triangular region is not filled up. The code I am running is very similar to this one, but with my own data :

It appears as though the code enters the part where it should fill up the upper region, since it displays the text that happens right before it :

Maybe it doesn't enter the loop where the matrix is filled up, but I see no reason for that.
Another weird that happened, when I downloaded the sample data to execute the the above code, the first time I ran it the matrix appeared to be filled up. When I switched the parameter faverage to False to test all the functionalities that I need, the matrices were not filled up. After that, I switched back to faverage = True, to see if that was the problem, but the resulting matrix was not filled up either. Any help would be appreciated!

please share a script not a screen capture that allows to replicate the problem

Sorry about that, here is the code :
`import mne
from mne import io
from mne.connectivity import spectral_connectivity
from mne.datasets import sample
from mne.viz import plot_sensors_connectivity
#print(doc)
data_path = sample.data_path()
raw_fname = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw.fif'
event_fname = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw-eve.fif'
Setup for reading the raw data
raw = io.read_raw_fif(raw_fname)
events = mne.read_events(event_fname)
Add a bad channel
raw.info['bads'] += ['MEG 2443']
Pick MEG gradiometers
picks = mne.pick_types(raw.info, meg='grad', eeg=False, stim=False, eog=True,
exclude='bads')
Create epochs for the visual condition
event_id, tmin, tmax = 3, -0.2, 1.5  # need a long enough epoch for 5 cycles
epochs = mne.Epochs(raw, events, event_id, tmin, tmax, picks=picks,
baseline=(None, 0), reject=dict(grad=4000e-13, eog=150e-6))
Compute connectivity for band containing the evoked response.
We exclude the baseline period
fmin, fmax = 3., 9.
sfreq = raw.info['sfreq']  # the sampling frequency
tmin = 0.0  # exclude the baseline period
epochs.load_data().pick_types(meg='grad')  # just keep MEG and no EOG now
con, freqs, times, n_epochs, n_tapers = spectral_connectivity(
epochs, method='pli', mode='multitaper', sfreq=sfreq, fmin=fmin, fmax=fmax,
faverage=True, tmin=tmin, mt_adaptive=False, n_jobs=1)
print(con)`

Sorry about that, here is the code :
import mne
from mne import io
from mne.connectivity import spectral_connectivity
from mne.datasets import sample
from mne.viz import plot_sensors_connectivity

#print(__doc__)

data_path = sample.data_path()
raw_fname = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw.fif'
event_fname = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw-eve.fif'

# Setup for reading the raw data
raw = io.read_raw_fif(raw_fname)
events = mne.read_events(event_fname)

# Add a bad channel
raw.info['bads'] += ['MEG 2443']

# Pick MEG gradiometers
picks = mne.pick_types(raw.info, meg='grad', eeg=False, stim=False, eog=True,
                       exclude='bads')

# Create epochs for the visual condition
event_id, tmin, tmax = 3, -0.2, 1.5  # need a long enough epoch for 5 cycles
epochs = mne.Epochs(raw, events, event_id, tmin, tmax, picks=picks,
                    baseline=(None, 0), reject=dict(grad=4000e-13, eog=150e-6))

# Compute connectivity for band containing the evoked response.
# We exclude the baseline period
fmin, fmax = 3., 9.
sfreq = raw.info['sfreq']  # the sampling frequency
tmin = 0.0  # exclude the baseline period
epochs.load_data().pick_types(meg='grad')  # just keep MEG and no EOG now
con, freqs, times, n_epochs, n_tapers = spectral_connectivity(
    epochs, method='pli', mode='multitaper', sfreq=sfreq, fmin=fmin, fmax=fmax,
    faverage=True, tmin=tmin, mt_adaptive=False, n_jobs=1)

print(con)

I can replicate; plotting con yields this:

generated by:
import matplotlib.pyplot as plt
import mne
from mne.connectivity import spectral_connectivity
data_path = mne.datasets.sample.data_path()
raw_fname = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw.fif'
event_fname = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw-eve.fif'
# read raw data
raw = mne.io.read_raw_fif(raw_fname)
events = mne.read_events(event_fname)
# Add a bad channel
raw.info['bads'] += ['MEG 2443']
# Pick MEG gradiometers
picks = mne.pick_types(raw.info, meg='grad', eeg=False, stim=False, eog=True,
                       exclude='bads')
# Create epochs for the visual condition
event_id, tmin, tmax = 3, -0.2, 1.5  # need a long enough epoch for 5 cycles
epochs = mne.Epochs(raw, events, event_id, tmin, tmax, picks=picks,
                    baseline=(None, 0), reject=dict(grad=4000e-13, eog=150e-6))
# Compute connectivity for band containing the evoked response.
# We exclude the baseline period
epochs.load_data().pick_types(meg='grad')  # just keep MEG and no EOG now
con, freqs, times, n_epochs, n_tapers = spectral_connectivity(
    epochs, method='pli', mode='multitaper', sfreq=raw.info['sfreq'],
    fmin=3., fmax=9., faverage=True, tmin=0., mt_adaptive=False,
    n_jobs=1)
plt.imshow(con)
plt.show()

maybe it's on purpose as this matrix should be symmetric anyway? just asking

Yes it is a feature, not a bug. It has been like this for very long time now. You can always do C + C.T when you want to display it.

It is even documented in the API, see section Notes: https://mne.tools/stable/generated/mne.connectivity.spectral_connectivity.html#mne.connectivity.spectral_connectivity

Thank you, sorry to bother with something that was in the API

Perhaps the log message should be changed or removed then. It seems odd to say "filling the upper triangle" if we're filling it with zeros.

I agree, that definitely is confusing

I can take care of this!:)

closed in #9147


Feedback Submission - how to guides - Simple Crowdfund - /how-to-guides/v3/pallet-design/crowdfund

What browsers are you seeing the problem on?
No response
Feedback
Grammar mistake in section 3. Declare your storage items:
https://docs.substrate.io/how-to-guides/v3/pallet-design/crowdfund/#3-declare-your-storage-items
Grammar mistake in the second sentence:
"First, deine the following types which will be used to declare our storage items:"
should be
"First, define the following types which will be used to declare our storage items:"

Thanks @patrickdevelopes, very much appreciated! 😃


Create our own instrumentation collection, prepare for ESM support and fix bugs

This replaces the @opentelemetry/auto-instrumentations-node with our own @solarwinds-apm/instrumentations package which does essentially the same thing but without depending on unnecessary packages and with more resource detectors enabled by default.
A lot of the initialisation code has been altered to be asynchronous, which makes it easier to reason about. The only things that actually need to be synchronous are reading the config then initializing the instrumentations based on it. Once that's done user code can start running without issues even if tracing isn't yet enabled and ready, since the module loading hooks are in place.
The config file loading has been simplified to all go through the same path which is a require call. There's no more optional dependency on json5 or ts-node, if users want comments they can switch their .json config to .js without any changes, and they can setup the TypeScript loader of their choice on their own without being limited to a given one.
A new @solarwinds-apm/module package was also added to bridge gaps between ESM and CJS. Previously the code contained many calls to the require global, which is not supported in ESM, and they are replaced using either the new module or import statements and calls which are properly translated by TypeScript depending on the target format. ESM is the default module format for AWS Lambda and for web code, and also the path forward for Node in the future. For full support some more changes and additions will still have to be made, but the biggest incompatibilities are now gone.
Also updated dependencies. Sorry this is a big one, feel free to ping me with any questions.

Thanks @raphael-theriault-swi ! I have left some very basic questions after a first look.
I am not going to be the best help on the ESM vs CJS compatibility stuff so please I encourage others to look at this too 😺

Thanks for the review


obselete methods..

UnityFactory.factory.LoadDragonBonesData("Ubbie/Ubbie"); // DragonBones file path (without suffix)
UnityFactory.factory.LoadTextureAtlasData("Ubbie/texture"); //Texture atlas file path (without suffix)
these functions have become obselete... how do i use the LoadData?

+1

Select the ske file
right-click -> Create -> DragonBones ->Armature Object / Armature Object (UGUI)

I am working on version 2.3 of the Dragon Bones animation.
Your answer has been helpful enough for me.
Thank you.
i'm 2.3ver DragonBones testing
-----Original Message-----
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Sent: 2017-07-21 (금) 19:33:37
Subject: Re: [DragonBones/DragonBonesCSharp] obselete methods.. (#34)
Select the ske fileright-click -> Create -> DragonBones ->Armature Object / Armature Object (UGUI)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Remove duplicate statement

Duplicate Statement:
Forward Slash (\) - Every \ used in the query clause must use escape character. For example, \ is evaluated as \ in the query clause.

@mario3113 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mario3113 Thanks for this typo fix.


ARROW-2981: [C++] improve clang-tidy usability

adds a docker-compose service for running clang-tidy
docker-compose runs as root, so the files touched by clang-tidy and clang-format were owned by root. They are now passed back to the user
clang-format is run after clang-tidy because the latter munges formatting

I ran clang-tidy then cleaned up the build errors in this branch to give an idea of what things are changed and what things can go wrong.

Codecov Report

Merging #4293 into master will increase coverage by 1.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4293      +/-   ##
==========================================
+ Coverage   87.99%   89.02%   +1.02%
==========================================
  Files         773      628     -145
  Lines       96308    85485   -10823
  Branches     1251        0    -1251
==========================================
- Hits        84747    76102    -8645
+ Misses      11325     9383    -1942
+ Partials      236        0     -236

Impacted Files
Coverage Δ

cpp/src/arrow/scalar.cc
57.89% <0%> (-15.44%)
:arrow_down:

cpp/src/arrow/array/builder_nested.cc
95% <0%> (-3.95%)
:arrow_down:

cpp/src/arrow/array/builder_dict.h
81.48% <0%> (-2.73%)
:arrow_down:

cpp/src/arrow/scalar.h
88.23% <0%> (-2.68%)
:arrow_down:

cpp/src/arrow/ipc/metadata-internal.cc
89.8% <0%> (-1.53%)
:arrow_down:

cpp/src/arrow/ipc/reader.cc
90.04% <0%> (-1.52%)
:arrow_down:

cpp/src/arrow/util/concatenate.cc
93.05% <0%> (-1.32%)
:arrow_down:

cpp/src/arrow/array.cc
90.46% <0%> (-0.2%)
:arrow_down:

cpp/src/parquet/arrow/writer.cc
96.44% <0%> (-0.19%)
:arrow_down:

cpp/src/arrow/compare.cc
80.56% <0%> (-0.13%)
:arrow_down:

... and 176 more

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 d141d78...bf6884d. Read the comment docs.

Is the only option to run this and have it apply changes, or is there a way to have it run a report for the purpose of applying manual fixes (similar to IWYU)?

It's not the only option; I could add format-check and clang-tidy-check endpoints which don't modify. This PR uses the modifying target because that's what docker-compose run format does

I think docker-compose run format is broken and should not modify the repository volume. It should only return failures and propagate the return code(s). I'd say that running su/sudo in the script blocking.

@fsaintjacques one of the objectives here is to enable auto-formatting to happen in Docker without having to configure clang-format in your local environment. This has been a problem for me when doing Windows development, for example

@bkietz can you hold off until we reach some consensus about the behavior of "format"?

So the objective of this JIRA is to allow more people to run clang-tidy and look at the output
How about this

Leave run format behavior unchanged -- that's out of scope
Run clang-tidy and output warnings, but do not modify the codebase

If we want to add a clang-tidy-fix target or something later we can do that.

@wesm done and green

I tried running the checks but it takes way too long. The following changes makes it acceptable, but less robust:
-Checks:          'clang-diagnostic-*,clang-analyzer-*,-clang-analyzer-alpha*,google-*,modernize-*,readability-*'
+Checks:          'clang-diagnostic-*,-clang-analyzer-*,-clang-analyzer-alpha*,google-*,modernize-*,readability-*'

After this, it seems somthing is wrong:
clang-tidy suggested fixes for /home/fsaintjacques/src/db/arrow/cpp/src/arrow/record_batch.cc:
clang-tidy suggested fixes for /home/fsaintjacques/src/db/arrow/cpp/src/arrow/compare.cc:
clang-tidy suggested fixes for /home/fsaintjacques/src/db/arrow/cpp/src/arrow/table_builder.cc:


/bin/zsh exits with [1] segfault after installing Prezto

Running it as a login shell, on a VPS over SSH.
I'm not sure what else to mention, so let me know.
Clarification: zsh didn't exit after installation. It exited whenever I tried running it subsequently.
Related question: http://superuser.com/questions/1006722/tmux-exits-with-exited-on-ubuntu-14-04

I just uninstalled Prezto and switched to oh-my-zsh, and everything works now.
I'd still like to try and help pin down the issue if possible.

What version of zsh do you have installed? A similar thing happened to me. However, after upgrading zsh from 5.0.2 to a newer version it works. I suspect it was something related to the async feature of the sorin prompt theme.

Hey, yep I was using 5.0.2, which is the latest in apt.
Did you compile the latest zsh from source or add a PPA?

I installed zsh 5.1.1 using linuxbrew which compiled it from source.

Got it, thanks!

Just wanted to chime in and say I also experienced this on a vanilla 14.04.3 install. The latest version of zsh on this platform is 5.0.2. But installing 5.2 doesn't experience this issue. Could be something to consider in documentation (unless I missed something).


Update Activity tab and Single Activity screen

With grouping internal transactions in Activities [1], there are some UI and UX improvements that we might want to consider, since there are now "standalone" and "grouped" transactions/activities:

[ ] The UI itself for the Activity tab [2].
[ ] Keeping them all of them expanded all the time might be a lot of noise? But tapping to expand means we need to add an additional placeholder row for the "main" transaction, or a button/toolbar to expand/collapse? Do we need a pseudo parent for the group?
[ ] When we tap on a group, what happens?
[ ] When we tap on a row in the group, what happens?
[ ] Also, recent transactions in token view, the screen displayed after tapping on a token. How do we display them and what do we display? The latter — do we show a group if it contains transactions that involve this token?

[1] The "internal" transactions are all token transfers, for now at least
[2] Screenshot:

@colourfreak bump, in case you missed this

@hboon
This is a bit more complex task. But I will be happy to work on this now.
First of all, we have a bug at the moment. It's impossible to Receive 5k ETH and 5k DAI. Something is wrong here.

What I suggest is add custom transaction types such as "Traded", "Purchased", "Contract Execution", "Approved". I requested this change long time ago, but I haven't seen it live. No idea where was that issue.
For the "Traded", when you go inside we should list all transactions inside it.

and we can even think of reducing this empty blank space in "Activity" under the amount by merging Transaction Details with this screen. I know we designed it to let token issuer place something there, but it turns out to be not confirmed. So I suggest a change.

@colourfreak will reply to the rest, but can you share the transaction hash or wallet address for where the problematic amount is displayed for?

@colourfreak will reply to the rest, but can you share the transaction hash or wallet address for where the problematic amount is displayed for?

Check out tomek.eth and the transaction on March 7th.

What I suggest is add custom transaction types such as "Traded", "Purchased", "Contract Execution", "Approved". I requested this change long time ago, but I haven't seen it live. No idea where was that issue.

Approvals for ERC20s was added for iOS recently (not in app store yet).
We can't do traded, purchased, etc since we don't have that information. We can (or at some point) will be able to see if there's a swap [1], but we can ony label it as a "swap", don't really know what's the intention of it.
Everything else that we don't label otherwise or is not crypto transfer is contract execution, do you want it to be labelled as such? Because it's basically means "unknown".
[1] Swap (at least to a certain extent, without server-side support) was already implemented in iOS. I remember hearing it doesn't work, so probably a bug. Will check

and we can even think of reducing this empty blank space in "Activity" under the amount by merging Transaction Details with this screen. I know we designed it to let token issuer place something there, but it turns out to be not confirmed. So I suggest a change.

To combine the single activity screen and the transaction details? Great! Is this particular one confirmed? We can break this out into a separate issue for implementation

Wrong DAI/ETH amount display fixed in #2770

and we can even think of reducing this empty blank space in "Activity" under the amount by merging Transaction Details with this screen. I know we designed it to let token issuer place something there, but it turns out to be not confirmed. So I suggest a change.

Splitting this out into a new issue: #2771


Make compatible with Node.js v22.11.0

Yep, these are the same failures I mentioned when I dm'd while we were both out.


Namespace problems

After template is rung the namespace in the project <project-name>.Content.Activities, etc. Following you examples online where did the ... .Content. ... come from It was not put in the  part of the CLI template command. This cause namespace warnings.

@RoyBradley I think this might actually be a little bug. I must have erroneously left that in from the the original project.  Surprising that you're the first to notice this. Will shortly update the template to remedy this

@RoyBradley Updated the template the Content has now been removed

Thank you. I have followed you for a few years. Way back then I found you auto mapper wrapper.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Gary Woodfine @.>
Sent: Friday, May 6, 2022 6:07:15 PM
To: threenine/api-template @.>
Cc: Roy W Bradley @.>; Mention @.>
Subject: Re: [threenine/api-template] Namespace problems (Issue #6)
@RoyBradleyhttps://github.com/RoyBradley I think this might actually be a little bug. I must have erroneously left that in from the the original project. Surprising that you're the first to notice this. Will shortly update the template to remedy this
—
Reply to this email directly, view it on GitHubhttps://github.com/threenine/api-template/issues/6#issuecomment-1120044141, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB76BK4BT6HAVFGJZVT7TATVIWJZHANCNFSM5U32633A.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks for persisting with my ramblings on the internet , hopefully they have been somewhat useful :-)

Ramble all you want; I really like your blogging. Your articles    Are first rate. So, I’m assuming that if I listen to your rambling, I will eventually I’ll find another piece of wisdom. :-)
Roy W Bradley
Get Outlook for iOShttps://aka.ms/o0ukef

From: Gary Woodfine @.>
Sent: Tuesday, May 10, 2022 2:59:20 AM
To: threenine/api-template @.>
Cc: Roy W Bradley @.>; Mention @.>
Subject: Re: [threenine/api-template] Namespace problems (Issue #6)
Thanks for persisting with my ramblings on the internet , hopefully they have been somewhat useful :-)
—
Reply to this email directly, view it on GitHubhttps://github.com/threenine/api-template/issues/6#issuecomment-1122003535, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB76BK5MWQ3BZ7727YVO6V3VJICMRANCNFSM5U32633A.
You are receiving this because you were mentioned.Message ID: @.***>

Bug was fixed and releases and is not available in 3.1.14 of the nuget package.
Going to go ahead and close this issue.  Thanks so much for the feedback

Thank you


HEP: Select another disk to store VM data during installation

https://github.com/harvester/harvester/issues/1728

I've created the issue for the version incompatibility check: https://github.com/harvester/harvester/issues/1860


Pp

Nice

Hello @Kakacuo ,
I will close this issue as no description is provided so I'm not sure what you are asking.
Best Regards,
Jon


How to get the sAMAccountName?

This is a very useful PS script so thank you greatly to the creator/contributors.   It would just be even better if I could get the sAMAccountName out of it.   Is this possible?

So I came across this script awhile ago for my security team and just recently they asked if I could update it with the SamAccountName. I had remembered this was asked so I thought I'd chime in and tell you how to add it. You'll need to update two sections, they'll be in bold. Hope this helps you and anyone looking for the same.
The first one here:
ForEach ($Member In $Members) {
Try {
$Name = $Member.GetType().InvokeMember("Name", 'GetProperty', $Null, $Member, $Null)
$Path = $Member.GetType().InvokeMember("ADsPath", 'GetProperty', $Null, $Member, $Null)
# Check if this member is a group.
$isGroup = ($Member.GetType().InvokeMember("Class", 'GetProperty', $Null, $Member, $Null) -eq "group")
If (($Path -like "/$Computer/")) {
$Type = 'Local'
} Else {$Type = 'Domain'}
New-Object PSObject -Property @{
Computername = $Computer
Name = $Name
Type = $Type
ParentGroup = $LocalGroup.Name[0]
isGroup = $isGroup
samaccountname = $Name
Depth = $Counter
}
The second one:
ForEach ($MemberDN In $ADGroup.Member) {
$MemberGroup = [ADSI]("LDAP://{0}" -f ($MemberDN -replace '/','/'))
New-Object PSObject -Property @{
Computername = $Computer
Name = $MemberGroup.name[0]
Type = 'Domain'
ParentGroup = $NTName
isGroup = ($MemberGroup.Class -eq "group")
samAccountName = Get-ADObject -Identity $MemberDN -Properties SamAccountName | Select-Object -ExpandProperty SamAccountName
Depth = $Counter
}


error when after upload image in custom server

i am trying to upload image to my server. Image was uploaded successfully

but i get an error

btw this is my froala options
let options = {
      heightMin: 400,
      imageUploadParam: 'file[]',
      imageUploadParams: {image: new Image().toString()},
      imageUploadURL: `${ENV.IMAGE_API}/upload`,
    };

    if (!_.isEmpty(user)) {
      options = Object.assign(options, {
        requestHeaders: {
          Authorization: user['token']
        }
      });
    }

can anyone solve this.. sorry for my bad english :(

@cahyowhy it looks like there is a problem on how the image is stored on server side. For tech support please reach us via our website: https://www.froala.com/wysiwyg-editor/contact.


WIP: major re-write

Goals:

remove ME
wrap configuration up into a class to enable multiple FS services to be talked to in the same process
add local caching option ( @arkilic have you been working on this? If not I will take it )
refactor in a way to make building a service on top easy

Are you going to get rid of ME for schema validation or this PR is limited to ME.Document removal?

Goal is to remove ME as a dep altogether so we can do per-instance connection management instead of global connection management.

:+1:

Is it worth even supporting a global singleton instance? Yes, users will need an instance that "just works,"  but databroker can make it for them. Is there some other reason we want this that I am not thinking of?

Maintaining the singleton should be a  mid-term back-compatibility feature.
Doing it this way I did not have to re-write any of the tests which gives me confidence I didn't screw it up.

Makes sense.

@danielballan and I agree with these changes


Allow trim to work when there is not enough frames.

This is what we need. When starting a stream, there are two cases:

There are already frames buffered in the ring buffer: we want to trim the input data to have just enough to not buffer
There are no (or not enough) frames in the ring buffer: we want to provide silence to the callback

get_linear_input_data() underruns gracefully and fill the remaining space with silence.

@ChunMinChang, this is similar to what we do in the mac backend, conceptually, but there is a way to know how much is buffered there, to not trim when there is not enough data, instead of trimming in all cases and returning early when there's nothing to do.

If input_frame_count > nframes > frames in input_buffer_manager, then we may have a out-of-sync problem between input_frame_count and input_buffer_manager. I'd prefer we totally remove the input_frame_count and only use input_buffer_manager to calculate frames, if this is the case (IIRC, this is what macos backend does).

I'll do this, it's better.


18024 Update DatePicker to not validate while still open

Issue #: /bcgov/entity#18024
Description of changes:

Fix UI issue described in image below
Only evaluate inputRules when dateText is not null
Set default for initialValue to be null instead of an empty string
I think it is ok if error message remains after initial input attempt (ie. They open the datepicker, click "Ok" or "Cancel" for an invalid date, then when they re-open the picker the validation message will not get cleared).
New story: datePickerValidation can be used for testing the fix

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the business-filings-ui license (Apache 2.0).

lerna notice
Successfully published:
 -<EMAIL_ADDRESS> -<EMAIL_ADDRESS> -<EMAIL_ADDRESS>lerna success published 3 packages


genotype pages not showing all associated terms

If I access wee1-delta from Mikel's paper , I only see a small fraction of the terms associated with wee1-delta?

#153
#214
https://github.com/pombase/pombase-chado/issues/577

Yep, sorry, Midori's right.  It's the same annoying redundancy problem.  I'm going to fix it soon.


Serve /crate/latest/crate rather than redirect

Part of #1438

Alright, I've added test cases for the crate page redirects, and the doc redirects are covered by the existing test cases, which I updated. Ready for re-review.

@jsha do you mind implementing my MatchVersion::Latest suggestion https://github.com/rust-lang/docs.rs/pull/1527#discussion_r739613990 ? Sorry to keep changing my mind, this should be the last time hopefully.

Alright, re-did with MatchSemver::Latest. That was less work than I had worried it would be, and you're right that it's tidier.

Hi @jsha, do you know when you'll get a chance to look at this again? It's almost done I think, it just needs a few updates to the tests :) (https://github.com/rust-lang/docs.rs/pull/1527#discussion_r748725526)

Thanks for the ping! I'll try to work on this this weekend. In addition to the tests there's also a pending request for a change in functionality (newest and * to redirect to latest instead of a specific version): https://github.com/rust-lang/docs.rs/pull/1527#discussion_r739744467

Updated and ready for re-review, pending resolution of https://github.com/rust-lang/docs.rs/pull/1527#discussion_r739744467.

@jsha Can you also update the crate details page to point to /latest unconditionally? Right now they point to the latest version number instead, these tabs:

the "platform" and "details" dropdowns are also using version numbers:

also if you don't mind it would be nice to remove the merge commit by rebasing, but I don't feel strongly about that.

Will do. Also there's a similar issue going from the doc page, via the menu, to the crate page - if you're on /latest/, it will send you to a specific version.
To clarify: you said "crate details page to point to /latest unconditionally," but I think the right logic is "all links to versioned URLs point to /latest if the current page is /latest, or the version of the current page otherwise." Do you agree?
Regarding rebases: My general preference is to squash everything into one commit before merge, and also to not rebase or squash mid-review, so reviewers can easily see the diffs between revisions. In other words, please never hesitate to ask me to rebase, I'm happy to do it. I'm also of course happy to adapt to a "rebase after each round of feedback" approach if that works better for you.

To clarify: you said "crate details page to point to /latest unconditionally," but I think the right logic is "all links to versioned URLs point to /latest if the current page is /latest, or the version of the current page otherwise." Do you agree?

Oh oops, good catch, that's what I meant :)

My general preference is to squash everything into one commit before merge, and also to not rebase or squash mid-review, so reviewers can easily see the diffs between revisions.

Sounds great, thanks!

Pushed some fixes to the crate page and friends. That wound up being more involved than I expected, so please take a close look. I added some tests for those behaviors.
I also added a "Permalink" under the crate menu to link to a versioned URL.

I also added a "Permalink" under the crate menu to link to a versioned URL.

Ooh, that reminds me - github/gitlab have a y keyboard shortcut which changes the current URL to have the version number, could we do that here too? In a follow-up PR is fine, I know this one is getting kind of big.
The last thing I think we should change is for "go to latest version" to use /latest/ as well.
The /target-redirect/ page doesn't seem to support /latest/:

/crate/{crate_name}/latest/builds/184 also is broken.
(I haven't looked at the code yet, I'll do that in a second.)

Ooh, that reminds me - github/gitlab have a y keyboard shortcut which changes the current URL to have the version number, could we do that here too? In a follow-up PR is fine, I know this one is getting kind of big.

Yep, let's do this in a followup PR.

The last thing I think we should change is for "go to latest version" to use /latest/ as well.

I'd like to propose also doing this in a followup PR. It's fairly isolatable and is in the general category of "start directing more traffic to the /latest/ URLs," along with updating the sitemap and changing crate.io.
I pushed some commits, but I just realize my fix for the builds page is incorrect (goes to a versioned page, but that's not actually what we want). I'll work on that some more and push again later today.

I pushed some commits, but I just realize my fix for the builds page is incorrect (goes to a versioned page, but that's not actually what we want). I'll work on that some more and push again later today.

Hmm, no that seems correct to me - we want to show a permalink because people often copy/paste it into a github issue.

This looks good to me as soon as you squash and fix the clippy lint :) thank you so much for sticking with this!


Update substrate to polkadot-v0.9.24

Can you increase the spec version of the node, to match the paracahin? integritee-network/parachain@642861b


please remove or rename the branch of revert-33692-🧹

the branch name include a special character, it seem not be standard name for branch.
Thanks, LiWang.

I'm not sure such a blanket character set restriction is possible, or warranted. Is this causing you a problem somehow? You can disregard the branch, revert branches are usually short-lived and deleted periodically. Otherwise, I'd need to see strong justification for setting a new character set policy, cleaning up 1 branch is not a solution, if there is a problem here.


Submit Biosimulators-smoldyn 2.73

Simulation tool

id: biosimulators-smoldyn
version: "2.73"
specificationsUrl: https://raw.githubusercontent.com/ssandrews/Smoldyn/810cabce43cf8cb59485e375dd7fd1e9f7468aa0/biosimulators.json
specificationsPatch:
version: "2.73"
image:
url: ghcr.io/biosimulators/smoldyn:2.73
digest: "sha256:dffd307d57ad878edd7a5ef4877d03bfb74d1fba88522e473eaacd5045d269c1"
validateImage: true
commitSimulator: true
Support for BioSimulators' conventions

[X] The simulation tool is provided as a Docker image
[X] The ENTRYPOINT of the image is a BioSimulators-compliant command-line interface
[X] This command-line interface supports the COMBINE/OMEX archive format
[X] This command-line interface supports the Simulation Experiment Description Markup Language (SED-ML) format
[X] This command-line interface supports the Hierarchical Data Format (HDF5)
[X] The capabilities of the tool are described using BioSimulators' format
[X] The modeling frameworks supported by the tool are described using Systems Biology Ontology (SBO) terms
[X] The algorithms supported by the tool are described using Kinetic Simulation Algorithm Ontology (KiSAO) ontology terms
[X] The model formats supported by the tool are described using EMBRACE Data and Methods (EDAM) ontology terms
[X] The licenses supported by the tool are described using Software Package Data Exchange (SPDX) identifiers
[X] The source code for the simulation tool is publicly available
[X] The simulation tool is available through a software repository such as CRAN or PyPI
[X] The simulation tool is registered with bio.tools
[X] The Docker image provides Open Containers Initiative-compliant metadata
[X] The Docker image provides BioContainers-compliant metadata
[X] The Docker image is registered with BioContainers

The validation/submission of your simulator failed.
- Simulator submissions should only use the keys "id", "version", "specificationsUrl", "specificationsPatch", validateImage", and "commitSimulator". The following keys are invalid:
-   - digest
-   - image
-   - url

The complete log of your validation/submission job, including further information about the failure, is available for 90 days here.
If you chose to validate your Docker image, the results of the validation of your image will be available shortly as a JSON file. A link to this file will be available for 90 days from the "Artifacts" section at the bottom of this page.
Once you have fixed the problem, edit the first block of this issue to re-initiate this validation.
The BioSimulators Team is happy to help. Questions and feedback can be directed to the BioSimulators Team by posting comments to this issues that reference the GitHub team @biosimulators/biosimulators (without the backticks).

Thank you @AlexPatrie for your submission to the BioSimulators simulator validation/submission system!
The BioSimulators validator bot is validating the specifications of your simulator, validating your Docker image, and committing your simulator to the BioSimulators registry.
We will discuss any concerns with your submission in this issue.
A complete log of your simulator submission job will be available for 90 days here. The results of the validation of your tool will also be saved as a JSON file. A link to this file will be available for 90 days from the "Artifacts" section at the bottom of this page.

- You (@AlexPatrie) do not have permissions to update the specifications of Smoldyn. Only the members of @biosimulators/smoldyn can update the specifications of Smoldyn. Please contact the members of this group to request permissions to update Smoldyn.


Dipoles

notebook for dipoles in whole spaces
util codes for plotting
currently these notebooks require the analytics branch of SimPEG

So in em_examples we are using dev/dcip branch... can we change to master? @lheagy

yep! feel free to update it - in environment.yml, just remove the @dcip/dev https://github.com/ubcgif/em_examples/blob/master/environment.yml#L12

want to merge ..., but it seems merge pull request is not activated ... any idea @lheagy

there are merge conflicts... you will need to merge in master before the pull request can be activated


[Feature Request]Transition to initial Chat page with keyboard commands

Describe the solution you'd like
Shift＋Command＋O
FYI: chatGPT
A clear and concise description of what you want to happen.
Why the solution needed
A clear and concise description of the reason why you want the solution.
Additional context
Add any other context or screenshots about the feature request here.
Implementation feasibility
Are you willing to discuss the solution with us, decide on the approach, and assist with the implementation?

[x] Yes
[ ] No

See : https://atmarkit.itmedia.co.jp/ait/articles/2405/17/news031.html#:~:text=新規チャットを開始する（1）,を開くことができる。


fix problems in the documentation

There were two problems
1. Key Imports were missing
2.The assignment of y_1 was incorrect

Nice, it's really good to have such a thorough proof-reader!
@MaximilianFranz: The PR looks good to me. If you agree, just merge it. Thanks.

Very thorough! Thanks @Tanmay-Kulkarni101!
Both were artefacts of copying code into the documentation.
Thanks for the fix, please keep on reporting. More is always better, in my opinion :)


Catch exceed character count error before submitting —  MastoHttpUnprocessableEntityError: Validation failed: Text character limit of 500 exceeded

Should be possible to catch it before posting since there's already a counter?

@hboon what does the character count show? (cropped below screenshot, near Post button)

@cheeaun I forgot, but definitely negative. Since it seem to allow clicking the post/submit button, I tried :)

@hboon ah ok, almost thought the count was wrong.
I'm not very confident with the character count logic on the front-end, or if they even match back-end's logic, or back-end's logic changed or differ due to different forks/instances/configs, so I feel it's better to let the back-end do the work 😂

Hah. Ok


MM-15505 Fix i18n fallback text for email to ldap password field

Summary
defaultMessage had a typo.
Ticket Link
https://mattermost.atlassian.net/browse/MM-15505

Removing @saturninoabril since he's out and adding @bradjcoughlin


Standardize Cop messages

This tries to bring a bit more consistency to the cop messages. Another
pass should be done to improve the wording, but this is something.

All messages now end in a period, ..
ALl cops with a single message constant are named MSG (I chose MSG
over MESSAGE because it's what rubocop expects).
Adjusted some message alignment and chose a consistent line
continuation pattern of not having an extra space before \.
Removed explicit MSG arguments where they were no longer needed.
Lifted one message into a frozen MSG constant.
Verified much of the above was correct with a script.

Quick and dirty script I used to check my work:
RuboCop::Cop::RSpec.constants.flat_map do |cop_name|
  cop = RuboCop::Cop::RSpec.const_get(cop_name)
  cop.constants
    .grep(/\A(MSG|MESSAGE)/)
    .map { |msg_const| ["#{cop}::#{msg_const}", cop.const_get(msg_const)] }
end.each do |const, str|
  puts "#{const} did not end in '.'" unless str =~ /\.\z/
  puts "#{const} should use MSG instead of MESSAGE '.'" if const =~ /MESSAGE/
end

:)


[aes] Add domain-oriented masking

This PR adds a new S-Box implementation for AES using domain-oriented masking (DOM).
To this end, the distribution of pseudo-random data inside the cipher core needs to be adapted as well. To ease review, I have factored out this first commit into #4642. Pleas only review commits 2 and 3 of this PR.
Note: this PR requires also #4629 .
The DOM S-Box added with this PR has a latency of 5 clock cycles, is unpipelined and requires 28 bits of randomness per evaluation (Canright no-reuse was at 18 bits per evaluation). Since we are using a 128-bit wide datapath with 16 S-Boxes in parallel, there is currently no benefit in using pipelining inside the S-Box. The pipelined versions in the original paper require between 2.6-2.8 kGEs. By avoiding pipelining and applying some additional, new optimizations, this implementation requires less than 1.4 kGE per S-Box. The overall resource consumption of AES with this S-Box is around 73 kGE.
Functional correctness of this implementation has been verified. The masking scheme has not yet been formally verified using the REBECCA tool: static analysis is passing, transient analysis is currently running (and most likely will run for a couple more days).
What's left to do:

[ ] Successfully pass transient analysis using REBECCA.
[ ] Perform initial SCA on FPGA.
[ ] Replace the (* keep = true *) attributes (Vivado and Yosys only) by prim_flop entities.

Update: The transient analysis in REBECCA has now been running for nearly 2 weeks without failure. First FPGA results using this implementation show massive leakage. The resistance against the correlation-enhanced collision attack is comparable with the unhardened implementation (~300000 traces, Canright no-reuse resists up to 1.2 Mio traces). :-(
Thus, I have now reworked the DOM S-Box to add more registers. Due to limitations of the REBECCA tool (doesn't like control signals such as register write enables inside the masked circuit) I can't formally verify the new implementation. However, on the FPGA we get substantial improvements in terms of resistance: with 2 Mio traces, just one single key byte difference can be guessed correctly! We would need at least 15 of 120 key byte differences to recover the key...
I will now clean up this new implementation and update this PR. Please wait with reviewing this and have a look at #4629 and #4642 first.

Update: The transient analysis in REBECCA has now been running for nearly 2 weeks without failure. First FPGA results using this implementation show massive leakage. The resistance against the correlation-enhanced collision attack is comparable with the unhardened implementation (~300000 traces, Canright no-reuse resists up to 1.2 Mio traces). :-(
Thus, I have now reworked the DOM S-Box to add more registers. Due to limitations of the REBECCA tool (doesn't like control signals such as register write enables inside the masked circuit) I can't formally verify the new implementation. However, on the FPGA we get substantial improvements in terms of resistance: with 2 Mio traces, just one single key byte difference can be guessed correctly! We would need at least 15 of 120 key byte differences to recover the key...
I will now clean up this new implementation and update this PR. Please wait with reviewing this and have a look at #4629 and #4642 first.

sorry i've been SUPER behind on the review here.
I will try to get to it next week, although I might just end up asking more
questions :)
On Wed, Jan 6, 2021 at 11:13 PM Pirmin Vogel<EMAIL_ADDRESS>wrote:

Update: The transient analysis in REBECCA has now been running for nearly
2 weeks without failure. First FPGA results using this implementation show
massive leakage. The resistance against the correlation-enhanced collision
attack is comparable with the unhardened implementation (~300000 traces,
Canright no-reuse resists up to 1.2 Mio traces). :-(
Thus, I have now reworked the DOM S-Box to add more registers. Due to
limitations of the REBECCA tool (doesn't like control signals such as
register write enables inside the masked circuit) I can't formally verify
the new implementation. However, on the FPGA we get substantial
improvements in terms of resistance: with 2 Mio traces, just one single key
byte difference can be guessed correctly! We would need at least 15 of 120
key byte differences to recover the key...
I will now clean up this new implementation and update this PR. Please
wait with reviewing this and have a look at #4629
https://github.com/lowRISC/opentitan/pull/4629 and #4642
https://github.com/lowRISC/opentitan/pull/4642 first.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/4643#issuecomment-755933153,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSVF4RU27LPFQBIVJYLSYVNK5ANCNFSM4VEPDXFQ
.

sorry i've been SUPER behind on the review here.
I will try to get to it next week, although I might just end up asking more
questions :)
On Wed, Jan 6, 2021 at 11:13 PM Pirmin Vogel<EMAIL_ADDRESS>wrote:

Update: The transient analysis in REBECCA has now been running for nearly
2 weeks without failure. First FPGA results using this implementation show
massive leakage. The resistance against the correlation-enhanced collision
attack is comparable with the unhardened implementation (~300000 traces,
Canright no-reuse resists up to 1.2 Mio traces). :-(
Thus, I have now reworked the DOM S-Box to add more registers. Due to
limitations of the REBECCA tool (doesn't like control signals such as
register write enables inside the masked circuit) I can't formally verify
the new implementation. However, on the FPGA we get substantial
improvements in terms of resistance: with 2 Mio traces, just one single key
byte difference can be guessed correctly! We would need at least 15 of 120
key byte differences to recover the key...
I will now clean up this new implementation and update this PR. Please
wait with reviewing this and have a look at #4629
https://github.com/lowRISC/opentitan/pull/4629 and #4642
https://github.com/lowRISC/opentitan/pull/4642 first.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/4643#issuecomment-755933153,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSVF4RU27LPFQBIVJYLSYVNK5ANCNFSM4VEPDXFQ
.

No worries @tjaychen , this PR is currently not so critical. I would prefer people having a look at my other AES PRs that are more urgent in particular #4629.

No worries @tjaychen , this PR is currently not so critical. I would prefer people having a look at my other AES PRs that are more urgent in particular #4629.

Thanks a lot @cdgori for your review and sorry for getting back so late. I had to prioritize other, somewhat less exciting tasks but hope to finish this one soon. I basically need cleanup the DOM S-Box (have a working example with more registers) and then scale down the PRNG.

Thanks a lot @cdgori for your review and sorry for getting back so late. I had to prioritize other, somewhat less exciting tasks but hope to finish this one soon. I basically need cleanup the DOM S-Box (have a working example with more registers) and then scale down the PRNG.

Update: I've cleaned up the DOM S-Box implementation that I've previously successfully measured on FPGA, shrinked the PRNG (gives us back almost 10kGE). Currently collecting new power traces and running formal verification.

Update: I've cleaned up the DOM S-Box implementation that I've previously successfully measured on FPGA, shrinked the PRNG (gives us back almost 10kGE). Currently collecting new power traces and running formal verification.

Update: had to do a rebase and fix a critical issue. I could confirm that the current version is again secure up to at least 2M traces on FPGA. Formal verification is still running.

Update: had to do a rebase and fix a critical issue. I could confirm that the current version is again secure up to at least 2M traces on FPGA. Formal verification is still running.

Thanks @cdgori for your feedback and review. And thanks @imphil for approving the PR.


Compile nbergst / akaze on TX2

when I try to make nbergst/akaze fork on tx2 it gives
`cc: error: unrecognized command line option '-msse2'
cc: error: unrecognized command line option '-msse3'
cc: error: unrecognized command line option '-msse4'
CMake Error at AKAZE_CUDA_generated_cuda_akaze.cu.o.cmake:207 (message):
Error generating
~cuda-akaze/build/src/CMakeFiles/AKAZE_CUDA.dir/lib/./AKAZE_CUDA_generated_cuda_akaze.cu.o
src/CMakeFiles/AKAZE_CUDA.dir/build.make:70: recipe for target 'src/CMakeFiles/AKAZE_CUDA.dir/lib/AKAZE_CUDA_generated_cuda_akaze.cu.o' failed
make[2]: *** [src/CMakeFiles/AKAZE_CUDA.dir/lib/AKAZE_CUDA_generated_cuda_akaze.cu.o] Error 1
CMakeFiles/Makefile2:168: recipe for target 'src/CMakeFiles/AKAZE_CUDA.dir/all' failed
make[1]: *** [src/CMakeFiles/AKAZE_CUDA.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2
`
when I removed the lines including "-msse2 -msse3 -msse4" on ./src/CMakelist.txt, in this time when making library  there is no error but in  runtime I mean when I try to command
./akaze_match ../../datasets/iguazu/img1.pgm  ../../datasets/iguazu/img4.pgm  ../../datasets/iguazu/H1to4p
it gives the following
allocating 83.657 Mbytes of gpu memory
allocating 83.657 Mbytes of gpu memory
safeCall() Runtime API error in file <~/vision-resource/cuda-akaze/src/lib/cudaImage.cu>, line 60 : unspecified launch failure
TX2:
Cuda 9.0
gcc 5.4.0
opencv 3.4.9
Thanks for your any help ...

Same for me with CUDA 10.2 and OpenCV 3.4.5.

Also this is what cuda-memcheck gives:
$ cuda-memcheck ./akaze_match ../../datasets/iguazu/img1.pgm  ../../datasets/iguazu/img3.pgm  ../../datasets/iguazu/H1to3p --descriptor 5
========= CUDA-MEMCHECK
allocating 83.657 Mbytes of gpu memory
safeCall() Runtime API error in file </.../cuda_akaze/src/lib/cuda_akaze.cu>, line 519 : all CUDA-capable devices are busy or unavailable.
========= Program hit cudaErrorDevicesUnavailable (error 46) due to "all CUDA-capable devices are busy or unavailable" on CUDA API call to cudaMallocPitch.
=========     Saved host backtrace up to driver entry point at error
=========     Host Frame:/usr/lib/aarch64-linux-gnu/tegra/libcuda.so.1 [0x2fd95c]
=========     Host Frame:./akaze_match [0x5a8f8]
========= ERROR SUMMARY: 1 error
As i can see from jtop my GPU is not busy:
NVIDIA Jetson TX2 - Jetpack 4.4 [L4T 32.4.3]
CPU1 [||||||||||||||                    Schedutil -  28%] 499MHz CPU4 [||||||||||||||                    Schedutil -  28%] 499MHz
CPU2 [  OFF                                                    ] CPU5 [||||||||||                        Schedutil -  20%] 498MHz
CPU3 [  OFF                                                    ] CPU6 [||||||||||||||                    Schedutil -  28%] 498MHz
Mem [||||||||||||||||||||||||||                                                                         2.0G/8.0GB] (lfb 1009x4MB)
Swp [                                                                                0.0GB/3.9GB] (cached 0MB)
EMC [|||||||                                                                                6%] 204MHz
GPU [                                                                                0%] 114MHz
Dsk [#################################################################################################              21.7GB/27.4GB]
┌────────────────── [info] ──────────────────┬───────── [Sensor] ── [Temp] ─────────┬──────── [Power/mW] ─ [Cur]  [Avr] ──────────┐
│UpT: 0 days 2:37:23                         │          BCPU       40.50C           │         4V0 WIFI     0      19              │
│FAN [                            0%] Ta=  0%│          GPU        38.50C           │         SYS CPU      384    410             │
│Jetson Clocks: inactive                     │          MCPU       40.50C           │         SYS DDR      401    391             │
│NV Power[3]: MAXP CORE ARM                  │          PLL        40.50C           │         SYS GPU      153    153             │
│─────────────── [HW engines] ───────────────│          Tboard     37.00C           │         SYS SOC      384    411             │
│APE: 150MHz                                 │          Tdiode     37.75C           │         ALL          2192   2184            │
│NVENC: [OFF]          NVDEC: [OFF]          │          thermal    39.70C           │                                             │
│NVJPG: [OFF]                                │                                      │                                             │
└────────────────────────────────────────────┴──────────────────────────────────────┴─────────────────────────────────────────────┘
1ALL  2GPU  3CPU  4MEM  5CTRL  6INFO  Quit                                                                        Raffaello Bonghi

Hi, are there any updates on this issue? I am struggling with the same issue.

Sorry, I am not actively working on this project at the moment. Unfortunately I cannot give you support about nbergst / akaze on TX2 since I did write the CUDA implementation myself. Have you tried contacting Niklas Bergström about this? His contact details are in his repo: https://github.com/nbergst/akaze
Regards,
Pablo

Sorry, I am not actively working on this project at the moment. Unfortunately I cannot give you support about nbergst / akaze on TX2 since I did write the CUDA implementation myself. Have you tried contacting Niklas Bergström about this? His contact details are in his repo: https://github.com/nbergst/akaze
Regards,
Pablo

Thank you for your reply! I have sent an email to Niklas. Hopefully he can help me further.
Regards,
Stijn


TraceId for empty header request

Hello!
I use your library with spring and it works greate for my tasks. But today i have a requirement to generate TraceId and spanId for requests that does not send that data to my service.
Is any simplest way do do that? May be property for spring properties? Or i should create prefilter for you web filter?
Just for info: reactive kafka library uses that logic from out of the box. I mean starting new trace Id for messages that has no TraceId.

Any Spring Boot application will do this out of the box through a Servlet filter or a Reactive WebFilter. Creation of a new trace id for any incoming request that has no trace id in the headers is a basic feature.