"registry" argument doesn't work?

The official example led me to a weird issue, not sure is my fault or the API has been changed?
Code:
from prometheus_client import Counter, Gauge, Summary, Histogram
from prometheus_client.core import  CollectorRegistry

registry = CollectorRegistry(auto_describe=False)
requests_total = Counter("requests_total", "Total count of requests", registry=registry)

Error message
[pylint] E1123:Unexpected keyword argument 'registry' in constructor call
[pylint] E1120:No value for argument 'labelvalues' in constructor call

That works fine for me. Maybe you're using a very old version?

I'm using 0.1.1, is it the latest version? in this version, the construction function is:
class Histogram(object):
    ....
    def __init__(self, name, labelnames, labelvalues, buckets=(.005, .01, .025, .05, .075, .1, .25, .5, .75, 1.0, 2.5, 5.0, 7.5, 10.0, _INF)):

I see though..., the decorator _MetricWrapper has register argument, maybe something else causes the error...

@jeffwji I am using the same version and it's working, I am not getting the error that you are getting.
btw you can still import CollectorRegistry in the from prometheus_client import Counter, Gauge, Summary, Histogram because the CollectorRegistry is already imported in the prometheus_client.py CollectorRegistry = core.CollectorRegistry

I don't see any bug here.


Knife share ERROR: Description can't be blank

Description

I am getting the an error ERROR: Description can't be blank when attempting to upload a cookbook to the supermarket.
Chef Version

Chef Development Kit Version: 2.4.17
chef-client version: 13.6.4
delivery version: master (73ebb72a6c42b3d2ff5370c476be800fee7e5427)
berks version: 6.3.1
kitchen version: 1.19.2
inspec version: 1.45.13

Platform Version

OSX 10.11.6
Replication Case

knife cookbook site share go_chef "Programming Languages" -V
I have already berks installed and uploaded and can see my metadata file there too. My knife.rb is pointing to this directory as the one hosting my cookbooks so that isn't the issue and my metadata.rb file has a description in it.
https://github.com/bws0013/go_chef/blob/master/metadata.rb
Client Output

INFO: Using configuration from /Users/bensmith/.chef/knife.rb
INFO: Validating ruby files
INFO: Validating templates
INFO: Syntax OK
Generating metadata for go_chef from /var/folders/xf/q135cl3j00s959bx902md6fr0000gn/T/chef-go_chef-build20180114-68259-1z0anem/go_chef/metadata.rb
Making tarball go_chef.tgz
ERROR: Description can't be blank

Stacktrace

@bws0013 knife cookbook site share is deprecated and will remove in future any change that you can test using knife supermarket share ?

Hi @bws0013, Is any chance that you can test knife supermarket share with latest version chefdk/ chef-workstation ?

Closing, for now, if you still face the issue feel free to reopen it


D365 Site Has Uses Beyond Customer Service

I understand the Customer Service Scheduling is being deprecated and replaced with Universal Resource Scheduling.  That's fine, it never worked all that well anyway.  However, the Site entity has uses beyond Customer Service Scheduling.  It is the obvious out-of-the-box entity to record the organization's offices.
Yes, yes, I can create a custom entity to fulfill this role but why should I have to when there's already such an entity?  The replacement entity, Organizational Unit, has to be extended to have the same business value and the entity name is nonsensical to the average user.
The Site entity should be retained and made accessible in the Unified Interface.

Hello @PRMerger10 , your feedback was shared with the product team previously but this issue was never closed. Resolving it accordingly. Thank you for your feedback.


How to build Tflite-gpu to use GPU delegate, NNAPI delegate for devices like Hikey (AARCH64)

I was trying to use label_image.cpp example in Tflite. I was unable to make use of delegate options in the example after setting gl_backend, useNNAPI , accel and old accel , hexagon etc.. delegate flags to true.
Is GPU delegate only restricted to Android and Ios devices ?
If not how to build TFlite with gpu on other devices like host pc for simulation, Hikey(AArch64) and Raspberry Pi(armv7,rpi) ?
Is NNAPI & Hexagon delegate only restricted to Android and Ios devices ?
If not how to build TFlite with NNAPI on other devices like host pc for simulation, Hikey(AArch64) and Raspberry Pi(armv7,rpi) ?
Does devices having snapdragon soc's such as (835,660) which are running in Linux(development boards which dont have Android Os). If yes , how to build with the delegate and increase performance ?
I don't find any Makefile to build with these delegate options enabled. Could you provide steps to build (or) Makefile for making use of these delegate options?

Currently GPU delegate only works on Android and iOS.
And NNAPI is only available on Android.
https://developer.android.com/ndk/guides/neuralnetworks

Is there roadmap visibility when Raspberry Pi 4 GPU will be a supported TFLite GPU delegate?


[WIP] Yaml tests

add more yml test files and a .md file describing the test in the experimental.failing directory

Can one of the admins verify this patch?


WIP: Add operator-ci to execute integration tests for PRs, nightly, and releases

based on devconsole-operator

@nikhil-thomas this needs to be merged becausehttps://github.com/openshift/release/pull/3964  is dependent on it

/ok-to-test

/test lint


Need some policy examples

Grab some policy examples from apex examples.

Added the examples to the repo, but need them in the wiki


Cannot save .csv file

I am using ubuntu 16.04 and trying the example which has the following code
import twint c = twint.Config() c.Username = "twitter" c.Limit = 20 c.Store_csv = True c.Custom_csv = ["id", "user_id", "username", "tweet"] c.Output = "twitter.csv"
I donot understand why the data is not getting stored in the "twitter.csv" file, in fact the file is not generated.
TIA!

Looks like you're missing:
twint.Search(c)


《Decomposed Meta-Learning for Few-Shot Named Entity Recognition》 code path 404

Thanks for your interest in our paper and code!
We are going through the review process of open source code. This link will be available when the review is completed. : )

No description provided.

The code is available now!

Thank You ！！


[bug] Not able to launch app on Wayland

Describe the bug
Launching app not working on Wayland. I can see a window frame for some milliseconds and then it disappears. It seems to be working fine X11. Tested with npm and cargo.
Reproduction

Create app using sh <(curl https://create.tauri.app/sh)
Go into project directory.
Run npx tauri dev

Expected behavior
The App should launch in both X11 and Wayland
Platform and versions
Environment
  › OS: Pop!_OS 22.04 X64
  › Node.js: 16.14.0
  › npm: 8.19.2
  › pnpm: Not installed!
  › yarn: Not installed!
  › rustup: 1.25.1
  › rustc: 1.64.0
  › cargo: 1.64.0
  › Rust toolchain: stable-x86_64-unknown-linux-gnu

Packages
  › @tauri-apps/cli [NPM]: 1.1.1
  › @tauri-apps/api [NPM]: 1.1.0
  › tauri [RUST]: 1.1.1,
  › tauri-build [RUST]: 1.1.1,
  › tao [RUST]: 0.14.0,
  › wry [RUST]: 0.21.1,

App
  › build-type: bundle
  › CSP: unset
  › distDir: ../dist
  › devPath: http://localhost:1420/
  › framework: React (Next.js)
  › bundler: Webpack

App directory structure
  ├─ .vscode
  ├─ src
  ├─ src-tauri
  ├─ node_modules
  └─ .next

Stack trace
No response
Additional context
No response

Please provide more context, like stack traces or error messages. tauri apps work on wayland.

Sorry if I am not doing the correct thing, but I am struggling to get a good stack trace. Is there a specific command that produces a proper stack trace?
Using gdb and cargo I was able to get the following
Breakpoint 1, 0x0000555555608010 in cargo_tauri::main ()
(gdb)
Single stepping until exit from function _ZN11cargo_tauri4main17h0179a274777cef81E,
which has no line number information.
[New Thread 0x7ffff7bff640 (LWP 529965)]
[New Thread 0x7ffff79fe640 (LWP 529966)]
[Detaching after vfork from child process 529967]
[Thread 0x7ffff79fe640 (LWP 529966) exited]
[Thread 0x7ffff7bff640 (LWP 529965) exited]
[Detaching after vfork from child process 529968]
[New Thread 0x7ffff79fe640 (LWP 529969)]
[New Thread 0x7ffff7bff640 (LWP 529970)]
[Detaching after vfork from child process 529971]
[New Thread 0x7ffff77fd640 (LWP 529972)]
[New Thread 0x7ffff75fc640 (LWP 529973)]
        Info Watching /home/jorge/test/src-tauri for changes...
    Finished dev [unoptimized + debuginfo] target(s) in 0.08s
[Thread 0x7ffff79fe640 (LWP 529969) exited]
[Detaching after vfork from child process 529975]
[New Thread 0x7ffff79fe640 (LWP 529976)]
[Thread 0x7ffff7bff640 (LWP 529970) exited]
[Thread 0x7ffff79fe640 (LWP 529976) exited]
[Thread 0x7ffff75fc640 (LWP 529973) exited]
[Thread 0x7ffff77fd640 (LWP 529972) exited]
[Inferior 1 (process 529940) exited normally]

Can you try setting this env var: GDK_BACKEND="x11" and try again?

Thanks but it didn't help.
┌─🦝─[/home/jorge/js_projects/test]
└─ echo $GDK_BACKEND
x11
┌─🦝─[/home/jorge/js_projects/test]
└─ npx tauri dev
     Running BeforeDevCommand (`npm run dev`)

><EMAIL_ADDRESS>dev
> vite

  VITE v3.1.8  ready in 224 ms

  ➜  Local:   http://<IP_ADDRESS>:1420/
  ➜  Network: use --host to expose
        Info Watching /home/jorge/js_projects/test/src-tauri for changes...
    Finished dev [unoptimized + debuginfo] target(s) in 0.08s

This was an issue with the my installation of linux or other required packages, not tauri. A fresh installation of linux solved everything. Tested on both PopOS and Fedora and it works as expected.


iPhone device listed twice

SPR 0.2.16 on Clearfog.
I had an existing iPhone device which I duplicated (from Safari) so that it would use the same iCloud Keychain password. After adding the device I refreshed the browser (because there's a bug where after adding a device it immediately pops up a modal). Then I renamed the device from "EVA Phone 12 mini #copy" to "EVA Phone 13 mini".
The state seemed to get a little messed up because there was an entry showing "EVA Phone 13 mini" with no PSK (showing as "N/A" on the device list, like a hardwired device), and no groups, but the correct MAC address. And then the "EVA Phone 12 mini #copy" is populated with a WPA key but no MAC address.

The MAC shown here matches what was displayed on the iPhone's setting screen.

The phone is happy connected at <IP_ADDRESS> and once I added it to LAN/WAN/DNS groups it was able to access the internet. I can ping the device from another host and it responds:
$ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=4.35 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=64 time=20.5 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=64 time=40.5 ms
^C
--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 2003ms
rtt min/avg/max/mdev = 4.346/21.774/40.455/14.768 ms

The MAC address for this device does NOT appear in the sae_passwords file. Therefore I'm surprised it is able to join the network.

However there is a device ff:ff:ff:ff:ff:ff with the same shared password, not sure if that plays into it.

Subsequently the iPhone dropped from the network and then reattached with a new MAC address, which then got assigned to the pending entry which had the right PSK.

Here are the new phone connecting as :6a and then again as :30:
{"Event":"AP-STA-CONNECTED","Iface":"wlan1.4115","MAC":"...:30","Router":"","Status":""}
info
8/10/2023, 9:44:40 PM
{"Event":"AP-STA-CONNECTED","Iface":"wlan0.4108","MAC":"...:6a","Router":"","Status":""}
info
8/10/2023, 9:29:09 PM
{"Event":"AP-STA-CONNECTED","Iface":"wlan0.4108","MAC":"...:6a","Router":"","Status":""}
info
8/10/2023, 9:29:06 PM
{"Event":"AP-STA-CONNECTED","Iface":"wlan0.4107","MAC":"...:6a","Router":"","Status":""}

Phone connected again as :6a and I was able to compare the sae_passwords file. Indeed it seems to be the ff:ff:ff:ff:ff:ff entry that appears when this device can connect:
$ diff /tmp/{,no-}connect-sort.txt
0a1
>
9d9
< [shared PSK between my Apple devices]|mac=ff:ff:ff:ff:ff:ff

Ok, here's my theory. The issue involves having two device entries:

One with the station MAC address assigned, but no PSK as if it's a wired device.
One that's pending, but with the PSK entry populated (because it was duplicated from another entry).

When SPR regenerates the sae_passwords file, it adds a wildcard entry for the pending device, entry 2:
https://github.com/spr-networks/super/blob/99d7281e66e0d27e05ed3c4fa0744779a8c70c3d/api/code/radios.go#L56-L60
The station connects, and provides the password that matches the wildcard entry, so it gets authed.
Somewhere else in the system it uses the station MAC address to decide what IP address, groups, tags, etc. to apply. This matches device entry 1, before it decides to overwrite the "pending" entry. Thus the device gets assigned <IP_ADDRESS>.

@lts-rad Does this reproduce for you, maybe one of the parts of this issue?

Duplicate a Wi-Fi client. New entry is added with pending MAC address.
Wi-Fi client connects; you get a notification in the browser, but the device list in the browser doesn't update the show the MAC address. The devices.json file is updated to replace the pending entry with the fixed MAC address.
Rename the device. This causes a write to the devices.json file. Because the browser still thinks the device is pending, it ends up creating a new pending entry with the new name, and the same Wi-Fi password.


Fix for issue #23

Stop measuring layout height of zero when offered full width (but zero height).

The typo and if-else exists in the current code; perhaps rather create a task to clean up the project code as a whole?

I think the if-else could be included in this PR, but there is a great need for the whole project to be cleaned up to be honest. Maybe I'll get around to it one weekend!


[improve][broker] Improve knowBrokers update

Motivation
Improve knowBrokers update
Modifications
Improve knowBrokers update
Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/AnonHxy/pulsar/pull/37

/pulsarbot rerun-failure-checks

the code here. it seems a fix to avoid a problem. context here https://github.com/apache/pulsar/pull/16690

knownBrokers？

@AnonHxy hi i found some previous fix on the ConcurrentModificationException when on knownBrokers field.
and in my opinion this method will have a chance to execute concurrently.
So if thread1 is try to figure out which is newAddBroker and which is deadBroker. thread2 clear the Collection will there an missing update？maybe an synchronized is need WDYT？

knownBrokers？

Great. Have corrected the title. Thanks

@AnonHxy hi i found some previous fix on the ConcurrentModificationException when on knownBrokers field. and in my opinion this method will have a chance to execute concurrently. So if thread1 is try to figure out which is newAddBroker and which is deadBroker. thread2 clear the Collection will there an missing update？maybe an synchronized is need WDYT？

Make sense


Optimize lit_get_magic_string_utf8.

Return item by index instead of using switch.

LGTM

Looks good to me

make push


How to understand param --padding_constant && --segm_downsampling_rate

How to understand param --padding_constant && --segm_downsampling_rate？
Besides, what if i want to replace encoder network you provided？I try to use mobilenet as an encoder，but，when training，size of pred and target are not match。 what should i do to solve the problem and adapt my own encoder network？
hope your response，Thanks！
@metalbubble @quantombone

Padding constant is the maximum down-sampling rate of the neural networks. For example, a ResNet w/o dilated conv has padding constant 32.
Segmentation downsampling rate is the down-sampling rate of ground-truth segmentation map when calculating loss. For example, UPerNet calculates such loss on a map with down-sampling rate 4.

@Tete-Xiao The two params must be set in testing?


Sharkdp.Bat version 0.22.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

The Windows Package Manager policies requires that the InstallerUrl comes directly from the ISV's release location.

https://learn.microsoft.com/en-us/windows/package-manager/package/repository#error-labels
Please add Urls from the official source.

original url only provide zip format. I believe providing it as an installer url wont work.

https://github.com/microsoft/winget-pkgs/pull/93774#issuecomment-1383185816

#93774 (comment)

thank you :)


Add animation timeline for easier debugging

I aim to fix the animation bugs, but it was very cumbersome/confusing to only use f12/devtools to peek into all the animation curves/keys, which might need an inspection to figure out the bugs. This PR generates a pretty dense canvas to visualize the animation data, so its possible to just pick what you want with a simple mouse click, then inspect the selection in f12/devtools via viewer.selectedAnimKey.
Online demo: https://killtube.org/playcanvas-gltf/viewer/

I confirm I have signed the Contributor License Agreement.

Thanks! 👍


Elixir support still a problem

See title. There's been several tickets auto closed by stale bot.

Sorry for the long delay here, I'll take a look at a few of them.
In the meantime though, PRs are very much welcome.

No problem, check out some of the closed prs I think one of them may fix it

Hey @srcrip can you take a look at the related PR (https://github.com/windwp/nvim-ts-autotag/pull/206) and let me know if that works for Elixir? I have exactly 0 Elixir experience, so I need another person to let me know 🙂

Thanks @PriceHiller, I think this branch does in fact fix it for normal html tags in heex files.
If possible though, heex in elixir also has this other html-like tag that looks like this:
    <.stepper current_step={@current_step} target={@myself} class="lg:hidden -ml-1">
      <:step name={:personal}>
        1.
        Personal Info
      </:step>

      <:step name={:work_experience}>
        2.
        Work Experience
      </:step>

      <:step name={:education}>
        3. Education
      </:step>

      <:step name={:other}>
        4. Other
      </:step>

      <:step name={:review}>
        5. Review
      </:step>
    </.stepper>

The <.stepper> is a phoenix component and the <:step> is a slot. It would be great if this plugin could successfully rename them too.

I've merge in #206, but I'm not sure how viable the alternative heex tags are to add here as we are using binds to set the tags.
It would require some custom logic specifically for heex files.

Does the Treesitter objects cover the entire tag identifier including the . at the beginning? If not does there need to be a treesitter change for that?

@PriceHiller it seems the config adds an alias for elixir = html but I think you still need to add heex as its a separate TreeSitter grammar


Hi , after starting game i see error

A catastropic failure occured while loading the modloader. (https://user-images.githubusercontent.com/66868587/148458478-f6ab0723-4fef-446a-9e24-b16917699117.png)

Have you tried --no-mods?

Have you tried --no-mods?

yes i tried, it doesn't work

Have you tried --no-mods?

In my case OneLoader stops at the preparing stage with --no-mods

Is the issue still occuring?

Did you correctly install the modloader?

I'm also having this issue, nothing works

Can't reproduce the issue, not enough information was provided


Swagger changes for Stop and Resume protection APIs

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@mentat9 / @weidongxu-microsoft : These changes are only in -preview version which is not live yet. Hence it should not break customers - as flagged in the PR.

@pratikjoshi14
Is there any SDK released for this preview version?
The factor on live or not, SDK released or not is only factors to consider, you still need approval from Jeffrey.

We published a powershell sdk for specific private preview customers who wanted to get early sneak peek. but no .net or other sdks were generated..


Update request

The cx contacted me they cannot find the columns explained this document, when they selected Edit columns button in the Virtual Machines pane (#4 in the instruction).
It seems like this works for the classic VMs but not for ARM VMs. As long as I checked the portal, I found there is the "Maintenance" menu in the menu tab, and it took me to the maintenance menu for VMs to see the right columns.
Could you please correct how to for ARM VMs, so that some customers don't get confused. Thank you!

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

ID: 289dc246-31cd-2705-45b2-69e8d7e66246
Version Independent ID: 9d55f15f-befc-762a-259b-f1b250d004cb
Content: Use the portal for maintenance notifications - Azure Virtual Machines
Content Source: articles/virtual-machines/maintenance-notifications-portal.md
Service: virtual-machines
Sub-service: maintenance-control
GitHub Login: @shants123
Microsoft Alias: shants

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thanks for providing feedback that helps improve our documentation. This issue has been resolved and we are closing the issue. #please-close


Overwrite unsealkey

In order to ensure we don't compromise the unseal key, we should overwrite the buffer with nonsense once the key has been used

Fixed by #7


MRG: Tracker iterable and adding logging for tracker dealer

Changes TrackerDealer to iterate through trial information rather than trackers. Edit: Also adding some logging into the TrackerDealer so that it will show up in the tab file (or whatever the callback function dictates) similar to instances of TrackerUD and TrackerBinom.

Now that I'm starting to try to use this, a couple of small API questions:

Why change_criteria and change_rule instead of change_reversals=None, change_trials=None (you can set one or both)?

For step_size_up / step_size_down lists, shouldn't change_criteria be 1 less than the length of these, since you only need to make N-1 change-size transitions for N step sizes?

I think I got my answer to (2):
Traceback (most recent call last):
  File "/home/larsoner/Documents/python/larsoner/adaptivetempcoh/adaptivetempcoh.py", line 65, in <module>
    change_rule, x_min, x_max) for sv in start_value]
  File "/home/larsoner/Documents/python/expyfun/expyfun/stimuli/_tracker.py", line 134, in __init__
    raise ValueError('First element of change_criteria must be 0.')
ValueError: First element of change_criteria must be 0.

I'd rather it just be N-1 in length...

And for (1), another more flexible API would be e.g., stop=dict(reversals=30, trials=20), change=dict(reversals=[5, 2])

Sorry I didn't catch these on the first go-around. We can stick with the current API if it makes your lives difficult to change them.

It's not a big deal to make changes right now. The first element must be 0 thing was a choice I wrestled with a bunch. So if you prefer it the other way I'm OK with that.
In principle I like stop=dict(reversals=30, trials=20). That's actually pretty nice. Almost always people use one or the other, but I see no reason not to make it more flexible. That change dict though is weird. I cannot imagine a case where you would want to change based on trials or reversals. It would be super messy. But I guess we could change it for consistency.
The thing that gets complicated is that TrackerDealer needs to keep pace on either trials or reversals. It can't do both. So if you have two stop rules, things can get weird. It also currently uses stop_rule for pacing because that's what makes the tracks stop around the same time without having a bunch of trials from one condition left to finish up. However, the pacing code as written should be able to handle the case where one tracker needs to be finished up.
I would say to minimize the load on @maddycapp27 and keep target movement to a minimum, poke around a bit more and comment here for things you think should be changed, and then we can decide and address them all in one go. Sound OK?

Happy to make whatever changes! (Happy in a "it will happen sometime next week" way).

Yeah that's fine. One reason dict is nice is that if there is ever another reason we want to add, we can do it without parameter explosion. But if that's unlikely (probably is), then the stop_reversals=None, stop_trials=None (where you mest define exactly one) might be the best way to go.
Also I'm not sure if these counts are working the way I'd expect. I wanted a step size of 4 for 3 trials, 2 for 7 trials, and 1 for the remainder:
step_size_up, step_size_down = [4, 2, 1], [4, 2, 1]
change_criteria = [0, 3, 7]
stop_criterion = 30
stop_rule = 'reversals'
x_min, x_max = 1, 20
start_value = [x_min, x_max]

But this was the track (fixed response True @ level > 10):

The elements of change_criteria are indices, not counts. (That's why the first one is currently required to be 0). It looks like in your tracks, you are getting changes at 3 and 7 reversals.

So, try change_criteria = [0, 3, 10] and change_rule = reversals and see if that does what you want.

Counts seems more intuitive to me than indices. To do indices I'd actually figure out the counts I wanted and cumsum.

Ahh yes I see it's properly documented already and I just didn't do my homework. My bad!
But do you find counts to be at least as intuitive? Either way you'll have to do a diff or cumsum operation to get between the two representations. But with the counts rather than indices we at least get rid of that pesky 0 :)

Nah, actually to me it seems more intuitive to say it changed at X and Y reversals. But I don't honestly care that much (it was another thing that I debated over when initially coding it).

I think I'm pro-indices--makes it easier to think about how "soon" to change step size based on the stop condition.

Fair enough, we can keep that. How about:

change the name to change_indices, because they are indices into the reversals (or trials)
remove the zero, enforce list length = step size length -1

Then I don't think I would have needed to look at the docs to get it right :)

Sure, if @maddycapp27 is OK with it.

Flake8
https://travis-ci.org/LABSN/expyfun/jobs/242121725
Failing example(s)
https://circleci.com/gh/maddycapp27/expyfun/49?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

I think the move from change_criteria to change_indices is finally debugged (although the tests will have the final say on that once they find). @Eric89GXL @drammock Looking at your comments the other potential changes were:

Offering both a reversal and a trial stopping criteria: Is this a "whichever happens first" or "both must happen" kind of deal? Not adverse to having this functionality, but not sure how high priority the change is based on previous discussion.
Putting labels in the plot function: How do you feel about the plot function also calling the plot_thresh function by default (perhaps include an argument of threshold=True)?

Was/is there anything else?

My initial reaction was definitely either, then I hesitated, then I settled at "either, right?" So: either, right?

I feel fine about it if there's a flag.

regarding the reversals: would it work to have a min_trials, max_trials, and min_reversals parameters, where defaults are min_trials=0, max_trials=np.inf?  That way you just check against all 3 params each time, the default case responds only to min_reversals, but it's easy to put in trial ceiling/floor as desired.

agree with @rkmaddox: fine if there's a flag

Hmm, I like that. In the case where you hit max_trials before min_reversals I would say that the tracker should stop (i.e. max_trials carries more importance than min_reversals). Agree/disagree?
Dumb question: what do you mean by flag?

"flag" = function parameter (usually a boolean one).  Meaning you can turn off the behavior if you don't want it, i.e., by passing plot_thresholds=False

by the way, I notice all the commits on this thread say they're coming from @rkmaddox but based on the conversation it sounds like @maddycapp27 is actually doing the coding.  Seems like maybe you're working on a shared computer and didn't update the git config?  (Might be an argument in favor of having separate user accounts on the machine...)

Yes you are correct, I should have made it clear that I'm doing the changes. We'll figure out an arrangement to fix the ambiguity soonish.

I ended up making parameters for stop_reversals and stop_trials instead of doing min/max_trials and min_reversals. The latter was making modification to the TrackerDealer rather arduous (read: frustrating/time consuming).

Don't forget to ping for review when changing to MRG, as there is not title-change GitHub notification

Okay I actually did a track while half paying attention to the (poorly-constructed) stimuli:

You can see for a while in a couple of them I was actually at ceiling (level 1) performance. I'm thinking that these should perhaps effectively count as reversals...? In the limit where a subject really is at ceiling based on the stimuli, then it seems like the tracker should be able to capture that, and it would if these counted as reversals.

Also, perhaps at least the x-axes of all trackers should be linked (so they're on the same scale), and then make linking the y-axes optional (since for from-above and from-below it can be nice for camparing)?

then make linking the y-axes optional (since for from-above and from-below it can be nice for camparing)?

Actually perhaps the options should be ylim='auto' (current) or 'extrema' (new default that uses min/max?) or 'linked' (which is 'auto' but with mpl linking).
This should probably be a separate PR since it will involve some thinking, and I don't want to hold up this PR for it.

@Eric89GXL with the plotting options, where would these go? The only plotting function is in the tracker, not the dealer, so having linked as an option wouldn't make any sense. But, I agree that it should be saved for another PR. There are plenty of enhancements that can be made (I want to add a ML psychometric function fit, for example) that don't affect how it is used in the experiment. We need to get that down so that @maddycapp27 can move forward, and then we can add ENH PR's as we use it some more.

Also, per an ongoing email conversation with Stecker, if you're bouncing off your upper difficulty limit a lot then your steps are too big.

Also, per an ongoing email conversation with Stecker, if you're bouncing off your upper difficulty limit a lot then your steps are too big.

Well I wasn't really bouncing so much as staying there. That's why I think they should count as "reversals". They would:

Help you get to the finer resolution more quickly if using multiple sizes, and (more critically)
Prevent infinite trial numbers if someone really can do the hardest difficulty consistently.

If we don't want to count staying at ceiling or floor from trial-to-trial as reversals, then we probably need to make the stop_trials argument mandatory (non-None), lest some poor subjects be left doing the task forever.

Hmm. If there are subjects that can consistently perform the task at the hardest difficulty level, then I think an adaptive track probably isn't the right way to find the threshold. That would mean that the threshold was somewhere below the hardest condition, and regardless of what you count as reversals, the threshold you calculate will be meaningless.
I think for this reason I am against counting them as reversals. It would basically speed up and disguise getting a meaningless answer. I think we can assume a reasonable amount of experimenter care that should prevent those edge cases from happening.

I don't love the idea of forcing stop_trials to be int. For me, I question if this would prematurely end a run more often than it aborted an endless run. But I also can't think of a better solution.
Putting a warning in seems reasonable. I would imagine that you wouldn't want to warn at a case like trial 7 in the second Run of your example, but would specifically want to warn about cases where there were multiple consecutive ceiling/floor responses.

Seems to me like the solution is setting stop_trials to be a finite int if you're concerned that a run might take too long. Especially since np.inf can't be cast to int, in cases where you don't want an upper limit on trials, you have to do some arbitrarily large int, or allow a flag like -1 or something, both of which seem messy. We could add to the docs the possibility of a track taking too long, or a poorly constructed one never converging.
But a warning for threshold seems fine. Eventually we will want to add plan to add a different threshold function that uses the the ML PF fit, rather than averaging reversals. The latter is a quick and dirty estimate.

I am still pondering the reversals thing.

I don't love the idea of forcing stop_trials to be int. For me, I question if this would prematurely end a run more often than it aborted an endless run.

Well then we can disallow None, and force people to do np.inf if they are confident enough (but recommend in the docs that they don't do this unless they're certain no participant can achieve ceiling perf).

Especially since np.inf can't be cast to int, in cases where you don't want an upper limit on trials, you have to do some arbitrarily large int, or allow a flag like -1 or something, both of which seem messy.

We can cast to float if want np.inf to be a possibility for those experimenters who are sure subjects can never ceiling out.

I am still pondering the reversals thing.

The track I posted is actually an interesting case. My thresh is probably more like 3 than 1 in my task. I actually know that 1 is chance level by construction -- it's a 2I-2AFC task where intervals 1 and 2 only meaningfully differ if level > 1.
I'm also not sure the "right" way to handle these things, because they can happen by chance in 2AFC tasks where there is some upper limit on the stimulus difficulty level.
Maybe the best thing to do for now (to get this PR in!) is to force stop_trials to be int | float, recommend people pass an int to prevent infinite length in the case of ceiling performance, but allow them to use np.inf if they want. That way we can keep thinking about how to deal with repeated ceiling performance.

Well then we can disallow None, and force people to do np.inf

stop_trials is recast as np.inf if it's None in the function call--is there significance to having the user input np.inf explicitly?

stop_trials is recast as np.inf if it's None in the function call--is there significance to having the user input np.inf explicitly?

It's not a big difference, but it is a little bit more explicit IMO in terms of forcing people to realize they could theoretically have an infinite number of trials. But if you prefer None, I can live with it as long as we're clear it's the same as np.inf, i.e. it could be infinitely long in the docs.

@maddycapp27 Functionally, no, but it forces the experimenter to ponder the infinite.

For the reversals thing, whether or not it is a good idea will depend on how you calculate your threshold. If you are doing mean of reversals (the standard, but not great way), then your threshold won't be affected if you immediately reverse at ceiling, so it's fine.  However, it was a bad track anyway. You also probably won't be able to do a great ML fit, but I don't think calling it a reversal will help.
So I think it is OK to call it a reversal when you hit the floor/ceiling? Because that is basically the level you will stay at until you get a real reversal anyway. Does that seem right, both?

Example broken
https://circleci.com/gh/maddycapp27/expyfun/62?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
Thanks for sticking with this @maddycapp27 I think we're almost there

I think we're almost there

Fingers crossed!

So I think it is OK to call it a reversal when you hit the floor/ceiling? Because that is basically the level you will stay at until you get a real reversal anyway. Does that seem right, both?

Yeah I'm fine with that. We could add a param that is repeat_limit='reversal' (new default) or 'ignore' (current behavior), unless you thinks YAGNI and we can just effectively use at_ceiling='reversal' without exposing the param.

Let's do repeat_limit='reversal' without exposing the parameter for now, if that's OK with @maddycapp27.

Let's do repeat_limit='reversal' without exposing the parameter for now, if that's OK

In this PR?

I think this PR, at least here on out, should be focused on nailing down the behavior of the tracker during an experiment. Any other plotting or analysis things that come up can be done later. So I think repeat_limit should be in this PR, since it will in fact affect experiment flow.

(Sorry. This has kind of become the PR from hell)

(Sorry. This has kind of become the PR from hell)

On the contrary. Consider yourself lucky by considering the two interrelated cautionary tales from MNE which together contain > 1000 comments. @rkmaddox was also involved in those, just sayin'.
I did learn an important link from those, though. (If you maintain a GitHub repository, there is a big green "Merge" button to click to accept changes.)

Maybe that means any PR I get involved in goes to hell?

The most-commented PR on this repo is #301...

I just deleted that branch for closure.

just FYI, I'm not likely to be able to review this...  super busy the rest of this week and next week, and then off to Kavli Institute for 2 weeks.  Forge ahead without me!

@drammock That's awesome, enjoy! Too bad you're not heading up UR way, we're only a few hours north.

@Eric89GXL I'm good. Thanks :)

I think I am +1 for merge. It seems to have converged on experimental flow, at least for a properly set up track.

Thanks for working through this @maddycapp27 !

So _repeat_limit = 'reversal' is set but never used. Can you make a PR to fix this behavior?

(i.e., this PR didn't do what we had talked about it doing when at an extreme, namely treat it as a reversal)

I can look at it tomorrow if that's ok?

Sure, that would be good


Fix some missing code in step-by-step walk through

What does this PR do?

Fixes #10518
To make it easier for beginners to get started with PyTorch Lightning, I've fixed some errors I encountered in Get Started page.
This pull request is Ready for review.
If there is any problem with this change, please let me know. I'll be happy to fix it!
Does your PR introduce any breaking changes? If yes, please list them.

None
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

ci-tests in CircleCI is failing, but this is not because of this change, but because the installation of jsonnet is failing.
#!/bin/bash -eo pipefail
go get github.com/google/go-jsonnet/cmd/jsonnet

unrecognized import path "gopkg.in/yaml.v2": reading https://gopkg.in/yaml.v2?go-get=1: 502 Bad Gateway
	server response: Cannot obtain refs from GitHub: cannot talk to GitHub: Get https://github.com/go-yaml/yaml.git/info/refs?service=git-upload-pack: net/http: request canceled (Client.Timeout exceeded while awaiting headers)

Exited with code exit status 1
CircleCI received exit code 1

Hi @tchaton.
Thank you for your review! I applied suggestions.
In addition to the review, I noticed that doctest fails by my change. I'll fix that too.

I added a commit to fix the missing testoutput to prevent doctest from failing.

@awaelchli @akihironitta
Thank you very much for the review!
Fixed as follows!

Remove unnecessary pip install torchvision  description
Move the import to the other imports location at the top of the section


Chair texture fixing on the E44 Yellow Head

It was reported that the texture of a chair was wrongly flipped, so I updated and made some changes to the model, so that now it returns to normal.
Nothing big modifications to the texture file, it's just I flipped the chair around inside the .bbmodel so that it faces the right position.

There are actually a few more seats that needs to be fixed, so I fixed it directly in beta-8.


Set Corners and Shadow of AnimatableView by Code

Hi,
what a great framework, speeds up my development cycle 👍
Probably in contrary to the most, I use this by Code as I need a little more fine grain about the animation details depending on different adaptive size classes (device traits).
I noticed a bug: When setting rounded corners and shadow by code, the rounded corners are ignored. I know why (maskToBounds, clipToBounds), but I am wondering if I can invoke your internal calls manually?
e.g. when using a UIPresentationController I do the following (stripped out unrelated stuff):
    override func presentationTransitionWillBegin() {
        dimmingView.frame = containerView?.bounds ?? CGRect.zero
        containerView?.addSubview(dimmingView)
        dimmingView.addSubview(presentedViewController.view)

        if let view = presentedViewController.view as? AnimatableView {
            view.cornerRadius = 20
            view.shadowColor = UIColor.blackColor()
            view.shadowRadius = 10
            view.shadowOpacity = 0.5
        }
      ...
  }

As said shadow is applied, corner ignored.
Next I am searching if you have any ready-to-use Pop animation as UIViewControllerAnimatedTransitioning which I can use in the above UIPresentationController in my own adaptive UIViewControllerTransitioningDelegate (returns different UIViewControllerAnimatedTransitioning depending on source trait)..
If you can point me towards the right direction, any help would be welcome :)
Cheers,
Frederik

@winkelsdorf thanks for reporting this, I need more time to investigate that, I did a quick try on IB, it is working fine, if you can share a demo project with just the problem bit, it will be faster.

Did you set the shadowOffset?  Did you use any view inspector like RevealApp to find out the runtime attributes, especially for the corner? IF some one reset it, find out where and why?
We call commonSetup() in ShadowDesignable to set the layer.masksToBounds, can you toggle it and see, but it may not help the corner though.

Just wanted to let you know: I'll check back in a few days, currently busy with a release..

Don't know about your main issue, but I can help with the second question:

if you have any ready-to-use Pop animation as UIViewControllerAnimatedTransitioning which I can use in the above UIPresentationController in my own adaptive UIViewControllerTransitioningDelegate (returns different UIViewControllerAnimatedTransitioning depending on source trait)..

We have to update our documentation about that part, but you can find all the animations ready to use that we have in IBAnimatable:  TransitionAnimationType
In IBAnimatable, we are using them with custom segues, custom UINavigationController, and ContainerViewController, so you can probably find your way to use them with your own UIViewControllerTransitioningDelegate`.
You can see each animation in the demo project by clicking on forgot password.

We have a set of transition animators can be used in UIPresentationController but don't have a Pop animation. I think it is a common requirement to have animatable UIPresentationController, I am going to create a card for it and close this issue for now.
@winkelsdorf if you have any question, or you'd like to provide more information, please let us know, thanks.


Speed up commit operations with partial index updates

    /// Cherry-pick a commit in memory and return the resulting tree.
    ///
    /// The `libgit2` routines operate on entire `Index`es, which contain one
    /// entry per file in the repository. When operating on a large repository,
    /// this is prohibitively slow, as it takes several seconds just to write
    /// the index to disk. To improve performance, we reduce the size of the
    /// involved indexes by filtering out any unchanged entries from the input
    /// trees, then call into `libgit2`, then add back the unchanged entries to
    /// the output tree.

See https://github.com/arxanas/git-branchless/blob/331b7cf2a37d00a3d54ba5df00f3d702aa7b3948/src/git/repo.rs#L774-L789

I wonder if we can and should create a crate for sharing complex details like this.  It could serve dual purposes of also documenting how to implement higher level git operations with libgit2, something that is lacking regardless of language.  Pretty much the only resource I found was (pygit's recipes.
The big risk is people wanting these functions customized every which way that we might as well not provide them.  If we can find a strict definition of what we accept (no touching working tree, conflicts are report as errors, etc) it'd help.
Maybe we could create an org we can collaborate in and I could move https://github.com/crate-ci/git-config-env over there (I put it there just to not have it in my personal repos which I try to avid).

I would be open to splitting the stuff under git-branchless/git into its own crate — I kind of wanted to do that anyways to improve compilation times. It wraps a lot of the libgit2 stuff into its own APIs, as per the commentary here, so it's not clear what functionality we wouldn't want to wrap.
That being said, it seems like it would slow down development to split up that crate from the rest of the repository, since development does tend to cross-cut at times.

I very much see libgit2 wrappers being too coupled to individual projects to be worth splitting out, unless they get very mature.
I was more envisioning a git-recipes crate that provides one off snippets that people can find useful like

Functions for cherry-pick, squash, etc
Auth logic (see https://github.com/rust-lang/git2-rs/issues/600)

So its mostly a question of whether the algorithms can be split out from the abstractions.

Naming is hard :).  I was looking to create an org to put these under but it looks like someone has the username git-rs.  If anyone has alternative names for what to name this common ground for working on git related libraries and tools, I'm open to ideas


Allow 'ctrl' key to be used for select-many

I believe that this should do the trick for the issue raised by @powersa in #249.
Note: I haven't actually tested this, just read through the code and made the PR via my browser.

:clap:
Some information about browsers and OS differences: http://stackoverflow.com/questions/6245868/which-key-does-the-e-metakey-refer-to-in-javascript-mouseevent


[test/ctc] Code quality: undefined-behavior

ctc.cpp:130: The ctc_alpha_cpu function contains write access to an array by negative offset. This code leads to undefined behavior and should be fixed.
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/develop/test/ctc.cpp#L130-L144
To reproduce:
CXX=/opt/rocm/llvm/bin/clang++ CXXFLAGS='-Werror'  cmake -DMIOPEN_TEST_FLAGS=' --disable-verification-cache ' -DCMAKE_BUILD_TYPE=debug -DCMAKE_CXX_FLAGS_DEBUG='-g -fno-omit-frame-pointer -fsanitize=undefined -fno-sanitize-recover=undefined ' -DBUILD_DEV=On ..
make check

...
/MLOpen/test/ctc.cpp:130:28: runtime error: addition of unsigned offset to 0x000001e4b630 overflowed to 0x000001e4b628
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /MLOpen/test/ctc.cpp:130:28 in
CMake Error at test_test_ctc.cmake:7 (message):
...

The problem is that outside of the function is not aware of such restrictions. And the method of setting arguments does not require passing a pointer not to the beginning of the array. Someone else can use it and get an error.

Check #886

@shurale-nkn Please specify urgency label for this issue and for #886


Not able to call any method using yml file

@benas
I feel this is the basic functionality required to invoke a fucntion in the action. i just needed a clarity on that how we can do that.

Is this problem solved now


fix: 🔥 aws adaptar as qwik citiy middleware

Overview
🌐 Live Demo: https://f5sdphvtj5.execute-api.us-west-1.amazonaws.com/
Updated a few dependencies like serverless, serverless-offline, also I migrate most of the logic to a specific middleware

What is it?

[x] Feature / enhancement
[ ] Bug
[ ] Docs / tests / types / typos

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

@wmertens @zanettin

@leifermendez Great! My only question is, why does qwik-city need the extra dependencies?


Quotes in src/main/helm/values.yaml get arbitrarily dropped in target/heml/openshift/*/values.yaml

If I have this in src/main/helm/values.yaml:
opensearch:
  limits:
    cpu: 2000m
    memory: 3Gi
  requests:
    cpu: 1000m
    memory: 2Gi
  java-opts: "-Xms1g -Xmx1g"

Then the resulting file at target/heml/openshift/*/values.yaml doesn't have quotes around the java options (see last line), and is therefore invalid:
---
app:
  [...]
opensearch:
  limits:
    cpu: 2000m
    memory: 3Gi
  requests:
    cpu: 1000m
    memory: 2Gi
  java-opts: -Xms1g -Xmx1g

Workaround: insert a space before the -:
opensearch:
  limits:
    cpu: 2000m
    memory: 3Gi
  requests:
    cpu: 1000m
    memory: 2Gi
  java-opts: " -Xms1g -Xmx1g"

Then the file will be syntactically correct, at least:
---
app:
  [...]
opensearch:
  limits:
    cpu: 2000m
    memory: 3Gi
  requests:
    cpu: 1000m
    memory: 2Gi
  java-opts: ' -Xms1g -Xmx1g'

I'm not sure if there is something we can do to improve this (since we delegate this on the jackson serializer), but will take into a look into this next Tuesday. Thanks for reporting!

Thanks!


Public 360 Commit

Please check the following conditions for your PR.

[ ] apiDefinition.swagger.json is validated using paconn validate command.
[ ] apiProperties.json has a valid brand color. Invalid brand colors are #007ee5 and #ffffff.

@Pranav2889 Also please make sure to sign the CLA that was linked to by the CLA bot in the comments. Thanks!

@Pranav2889 Please move the connector files from custom-connectors/Public360/Artifacts/apiProperties.json to certified-connectors/Public360

@Pranav2889 Please delete the extra files, i.e. icon.png and settings.json from the PR as they are not required for open sourcing your connector.

Create a seperate one


Explainability metrics

explainability metrics
feature importance plots
tutorials

Also, for lime, am I right in thinking that the contrast metrics are not included here?
Is it "replaced" with the importance spread for features? I guess for each feature, you get their range and calculate the entropy of the ranks compared to if it was the same rank in each sample, which give you some sort of contrast metric?
Do we have demos of the effectiveness of these metrics? i.e. what are expected "good" values? Or can we only figure that out by comparing to a benchmark explainable model?


If two modals share the same v-if condition the calculated z-index is incorrect, resulting in popover misbehaviour.

Describe the bug
If two modals share the same v-if condition, initially false and then becoming true, the z-index of the first modal will be too high. Popovers placed on the first modal will then not appear (due to their lower z-index).
This is also related to the ticket here: https://github.com/uiv-lib/uiv/issues/341 - if popovers respected modal z-index I don't think this would be an issue.
To Reproduce
Please see demonstrative example:
https://codesandbox.io/s/uiv-bug-o5tyh?fontsize=14&hidenavigation=1&theme=dark

Two modals must be created and depend on the same v-if condition, initially false (if_condition)
The first modal must begin v-modelled on to a false variable (v_model_a)
A button must set the if condition and v-model attributes of the first Modal to true (if_condition and v_model_a).

When clicking "Open Modal" the popover inside the modal will not appear. If the modal is then closed and opened, it will appear.
Additional context
I have examined the UIV Modal code and understand the cause, but I cannot suggest the fix.

Inside Modal.vue, the function getOpenModalNum gets the number of open modals by checking how many .modal-backdrop elements exist in the DOM.
Inside the $toggle() method the z-index of the opening Modal is given an offset depending on how many modals are open.

But no modal was open when the Modal was opened and $toggle() was called, so why was the z-index offset? The answer is because:

In Modal.vue, the .modal-backdrop div initially exists in the template and is removed in mounted().
Therefore in my example, as soon as the v-if condition becomes true, the .modal-backdrop for both modals are inserted in to the DOM - however mounted() is first called for Modal A and completes execution (opens the modal) before the mounted() is called for Modal B which would have removed its own .modal-backdrop. Therefore getOpenModalNum in $toggle on the first run for Modal A produces 1, as it detects the backdrop of Modal B.

You may find it strange that I am using both v-if and v-model for the modal, however I do this a lot over my project when the slotted contents of the modal require require an object not to be null. This object is tested in the v-if of the modal.
You may say I should control the Modal slotted contents with a v-if. However I am often wrapping Modal in my own wrapper component that takes the object - and placing v-if on that wrapper instead, not the Modal. The wrapper expects that the object is non-null, as it often kicks off APIs on mounts and it would be more work to ensure the wrapper could handle null values.
Workaround Found:
Use $this.nextTick around the v-model attribute change. This allows both Modal's mounted() to run before displaying the first modal.

I understand what you said. Thanks for the very clear description. I will find out a solution for this.

@StefanCardnell pls try 0.36.1

Have tried 0.36.1 and the issue seems to be fixed. Thanks for the resolution @wxsms.


Auto update not working

ℹ Computer information

PowerToys version: 0.29.3
PowerToy Utility: Application
Running PowerToys as Admin: Yes
Windows build number: [run "winver"] 1909 (18363.1316)

📝 Provide detailed reproduction steps (if any)

… Open Power Toys
…Check for Updates
…Update available

✔ Expected result : With Auto update turned ON, PowerToys did not update to the latest version
What is the expected result of the above steps?
❌ Actual result : Power Toys gives me notification that update is available when clicked "Check Updates"
What is the actual result of the above steps?
📷 Screenshots

Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form No

do you have notifications turned off / focus assist enabled?  we alert via notification and check every 24 hours

That was it. Thank you


Provider - Custom

I apologize if this has been covered before...
But would you please consider the possibility of an "other" option (accompanied by a dynamic textbox) to specify providers that are not listed?  It would also be nice to have that for the location and label fields.
Thanks

you can add your own providers already more>>provider>> add a new provider

Oops, I did not see that.  Thanks very much, I greatly appreciate it.


Track sent but not received in application Application Insights

I installed version in 6.0.1, imported the module and specified the right instrumentation key from my Application Insights App (type: node.js). A track is sent with status code 200 but I don't see any data shown in Azure.

Is there something I overlook here?

I was expecting exceptions to show up in the Failure Panel in Azure. Now I found out that in there only server side failures are shown. After looking under Browser in Azure I found the received exceptions I was looking for. My bad.


changed author link

@rachelappel  simple author link change. Please merge.

@rachelappel @danroth27  must be bogus build errors. False positives are not cool, I've seen so many that's why I agreed with @Erikre that his build error was probably  bogus. In his case, he did have a problem so you always need to check.

The bogus build errors were fixed in 054a7527f3e0dd91019f2fec0cdb2272cf726371
Any PRs opened after that should not have false positives.

Changes merged in a3fdb002394f15082c1fe337e90e7ec2fd2b147a


Improve user experience for SQL UI

The following improvements were made

use filtering capabilities of el-tree-v2 to allow searching for non-empty tables, searching by database, table and column name
add a shortcut to run SQL queries
provide more clear table statistics
make tabs which require additional backends (ML and graph) switchable

In additionto UX imprrovements I added a Helm chart which is useful for deployment in Kubernetes.
Finally, ClickHouse login credentials can be populated via URL parameters. This allows to build "deep links" into ClickHouse instances. In my case there's a kind of portal with many ClickHouse instances and we want to simplify the user access to those.

@wangqinghuan and other devs. Would be great to get this merged.


Fix RDKit SMILES parsing

Changelogs

Fixes issue #22 by making generated SAFE strings more compatible with RDKit
Add support for labelling attachment points with branch opening label (see https://github.com/datamol-io/safe/issues/22#issuecomment-1874194848)

Checklist:

[x] Add tests to cover the fixed bug(s) or the new introduced feature(s) (if appropriate).
[ ] Update the API documentation if a new function is added, or an existing one is deleted. Eventually consider making a new tutorial for new features.
[x] Write concise and explanatory changelogs below.
[x] If possible, assign one of the following labels to the PR: feature, fix or test (or ask a maintainer to do it for you).

The original discussion related to this PR can be found here: https://github.com/datamol-io/safe/issues/22.

Ping @hjuinj  for review.

Thank you, Emmanuel. LGTM May I ask will the pip package be automatically updated upon merging?

Yes, there will be a new release after merging. I will ping you back here when it's available.

PyPi version released. conda-forge will be later in the week.

Thanks Emmanuel for the super swift resolution, and kudos on the great work on SAFE.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1480

Questions ?
Please refer to the Backport tool documentation


[ELPA] Fit latest elpa into the code

Several issues with the latest ELPA were addressed:

elpa now defines it's own double_complex and float_complex macros. To avoid the name collistion, the decision is taken to remove all our using double_complex = std::complex<double> aliases
linking problem is reported here: https://github.com/marekandreas/elpa/issues/30
our elpa.hpp header was removed as unnecessary

Anything after 7.4 would be ok in terms of API. We can make the minor release any time before the CP2K release.


14.3 Icon GAN: 'Model' object has no attribute 'inbound_nodes'

Notebook 14.3. Icon GAN
execution of def gan(g, d) / def get_internal_updates(model) fails with error 'Model' object has no attribute 'inbound_nodes'

inbound_nodes was renamed to _inbound_nodes (added underscore). Issue closed

Thanks. I was just looking at this: https://stackoverflow.com/questions/48329150/keras-inputlayer-object-has-no-attribute-inbound-nodes-when-converting-to-cor


Unverified VS Code Publisher

Describe the bug
A third-party-extension security rater (similar to Snyk) has given this repo's VS Code Extension a "medium" threat level due to:

Publisher didn't verify their listed domain ownership. Publisher verification is a good practice to ensure the publisher is who they say they are. Yet, VS Code publisher verification process is not rigorous enough.

Link here.
Expected behavior
Please consider if it would be simple and convenient to become "verified" as a publisher. If so, perhaps it may be something you would be willing to do. Or not!
Original error
[Not applicable]
Screenshots
[Not applicable]

Also, really love this extension. Congratulations on its success!

Hi, thanks for reporting this. FYI, I'm not the extension author; I'm just an enthusiast who contributes a little bit.
Looking into this for a few minutes I found this article: https://medium.com/@amitassaraf/3-6-uncovering-design-flaws-in-the-visual-studio-code-marketplace-ea1d8e8b0171
This explains why extensiontotal marks it as a medium-level threat. Although I agree with their assessment about lacking verification on the VS Code extension marketplace, this warning (in my understanding) will appear on any extension where the listed homepage/repository is pointing to a domain they have not verified ownership of.
As this is the actual repo and homepage of the ts-pretty-errors extension, in this case the warning is just exactly what it is: a warning. Using an actual verified domain as the homepage for the extension seems like a bit much just to get rid of a warning on a third-party site.
I think they point out a very valid flaw, I hope the VS Code team takes it seriously and works to improve this attack vector.
But it also reads like an advertisement for extensiontool as a product. So do keep that in mind.

Thank you @jb-asi and @kevinramharak,
Actually, I've been waiting for approval from Microsoft for a very long time.
I fulfilled the requirements but still no comment from their side.
If anyone can help speed things up it will be really appreciated


Add github action for Documenter.jl

didn't realize we already had documenter.jl running in CI.yml


Bug: PTZ speed broken on BirdDog P200A2_A3 5.5.114

We recently upgraded the BirdDog P200 firmware to latest (5.5.114) as well as upgrading Companion from 3.1-ish to 3.4.3. Some - but not all - of our PTZ buttons in Companion 3.4.3 started panning super super slowly (like, 1 pixel per second). However, some were fine.
Looking in the JSON for the buttons I see:
Working Button:
"options": {
	"val": "left",
	"posPan": "0000",
	"posTilt": "0000",
	"override": true,
	**"panSpeed": 11,**
	**"tiltSpeed": 9,**
	"value": 88
},

Not working button:
"options": {
	"val": "right",
	"posPan": "0000",
	"posTilt": "0000",
	"override": true,
	"value": 89
},

I think - but cannot verify - that a "working button" was a button we created at some point in the past?
The non-working button is one created with v3.4.3. I wonder if the "value" parameter is either gone or otherwise doesn't do anything?
Also note in the BirdDog Camera Control UI the Pan Speed is a value from 0-21. In Companion, the "Speed Override" control is a value from 0-255, which seems like a mismatch.
It's also possible it had an issue with the Module Model being set to Auto-Detect, I changed it to P200 A2_A3 but that didn't seem to make difference.
Thanks!
--
Matt

@MattRogish I think I found the issue with the pan/tilt speed override not working, and that should be fixed for new buttons in the latest beta. Let me know if creating a new button still results in issues.
In regards to the speed values, the module should recognize the speed mode (either 0-21 or 0-255) set by the camera but it sounds like it's possibly not doing that. I will look into that more.


Interface

It would be a good idea to change the interface, to make it better and more professional.

It would be good to make it possible to download a list of tasks from the internet

Add an other status for a task than done or not : like for example in progress.


Scaffold a NixOS module able to produce a script

The idea is that given a NixOS module like
{
imports = [spo-anywhere.nixosModules.spo-anywhere];

spo-anywhere = {
  blockProducer = {
    # ...
  };
  target = "X.X.X.X";
};

we get a derivation in the NixOS configuration attrset that depends on those options.
For instance running
nix build .#nixosConfigurations.foo.config.system.build.spoAnywhereScript
we should get a bash script that runs:
nixos-anywhere --flake /nix/store/.../foo-config-flake#foo --target X.X.X.X
scp -r $1 X.X.X.X:/var/lib/mykeys

For this issue purposes let's focus only on the structure and let's merge it as soon as it works.

I made target an argument of the script in the same way that key-dir is (i.e. Usage: spo-deploy-script --target <target> --key-dir <key-dir>).


do not create clone (instanceDoc) in model [8]

This should be faster, as we can pass the complete model to the XPath evaluator:
In all expressions, detect paths starting with / and prepend /instance[0/
concat(/path/to/node1, path/to/node2) => concat(/instance[0]/path/to/node1, /instance[0]/path/to/node2)
It is worth considering doing this in the XPath Evaluator itself, though that would technically break it.

We'd have to remove repeat templates from the model and instead add those as data to a repeat.

[ ] fix cloning repeats (after init)
[ ] fix tests
[ ] fix itemsets not updating after manually cloning repeat with template
[ ] clean up evaluate function
[ ] do another performance comparison with master

remaining checkbox is moved to its own issue: #266


Upgrade Ruby to 2.3.0

Upgrade Ruby to 2.3.0
https://www.ruby-lang.org/en/news/2015/12/25/ruby-2-3-0-released/

Thanks


Mockingjay references XCTest

I’d like to use Mockingjay in a test version of my app, where I mock some responses for demo mode.
But Mockingjay references XCTest and can therefore not be used outside of test projects.
Can Mockingjay be split up into a main pod + subpod for the XCTest dependency?

Ok. I’m sorry! I should’ve looked into the podspec first 🤦♂


fix nginx-udp-and-udp on same port

In nginx-ingress-controller if using
        - --udp-services-configmap=$(POD_NAMESPACE)/nginx-ingress-udp
        - --tcp-services-configmap=$(POD_NAMESPACE)/nginx-ingress-tcp

configs with same port, 53 for kube-dns as example:
apiVersion: v1
kind: ConfigMap
metadata:
  name: nginx-ingress-udp
data:
  53: "kube-system/kube-dns:53"

and
apiVersion: v1
kind: ConfigMap
metadata:
  name: nginx-ingress-tcp
data:
  53: "kube-system/kube-dns:53"

in nginx.conf will be created two upstreams with same name:
upstream kube-system-kube-dns-53 {...}

and nginx failing to start

This change is

/lgtm

Coverage remained the same at 43.178% when pulling 23c45340be3f92b0fe9d616ad31b63c017b21f6d on lkirill:nginx-udp-and-udp-fix into 4f04537eb0806d4d4ad622323141beb650427b01 on kubernetes:master.

@lkirill thanks!


Thanos Compactor Fails to Progress, Goes OOM

Thanos, Prometheus and Golang version used
thanos, version 0.1.0-rc.2 (branch: master, revision: 8b7169b78f5b1cf76424cabb2e7586f7219ae11b)
What happened
Compactor runs, uses up 23GB of RAM, gets killed, rinse and repeat
Full logs to relevant components
level=info ts=2018-09-13T08:28:28.280610487Z caller=downsample.go:213 msg="downloaded block" id=01CMEKAPKRKB35ST3SJA9Q594B du...
level=info ts=2018-09-13T08:25:46.085395594Z caller=compact.go:132 msg="start of GC"
level=info ts=2018-09-13T08:25:46.147804176Z caller=compact.go:171 msg="start first pass of downsampling"
level=info ts=2018-09-13T08:25:44.303181744Z caller=compact.go:126 msg="start sync of metas"
level=info ts=2018-09-13T08:25:44.302939213Z caller=compact.go:231 msg="starting compact node"
level=info ts=2018-09-13T08:25:44.303183157Z caller=main.go:243 msg="Listening for metrics" address=<IP_ADDRESS>:10902

**Anything else we need to know**

The log messages always seem to reference block 01CMEKAPKRKB35ST3SJA9Q594B.
It takes about 20 minutes from container startup to OOM, which is at 23GB.

When it ooms? After what exactly log line? The logs you provided does not suggest anything.
Indeed, that was the last log entry. After that k8s kills the pod.

Can you try to do kubectl logs -p <container>

Nice, so clearly downsampling being not optmized. Let's move discussion to the original ticket: https://github.com/improbable-eng/thanos/issues/297


Add RegisterDelegate with parameters returning object for the requested runtime-known service type

Remove the gap in API because we have one for the RegisterDelegate of IResolverContext
This overload maybe useful for the Prism.Extensions.DryIoc

Example without this feature:
using System;
using DryIoc;

public class Program
{
	public static void Main()
	{
		var c = new Container();

		c.Register<MyService>();

		c.Register(typeof(MyService), made: Made.Of(FactoryMethod.Of<Program>("Activate")), setup: Setup.Decorator);

		Console.WriteLine(c.Resolve<MyService>());
	}

	public class MyService {}

	public static MyService Activate(MyService s)
	{
		// do stuff
		return s;
	}
}

When implemented you may write:
c.RegisterDelegate(typeof(MyService), typeof(MyService), (object s) => { /*do stuff*/; return s; }, setup: Setup.Decorator);


Feature request: InternalBinaryWrite configurable to force writing non optional fields

Looking at the spec of proto3 the behavior of not writing fields when the value is equivalent to the default is expected

Also note that if a scalar message field is set to its default, the value will not be serialized on the wire.

In order to be compatible with proto2 serialization it would be great to have an option that allows the override of this behavior and serializes always non-optional fields, the changes required seem to be fairly small. It could be even automatic detecting the syntax statement when reading proto.
Let me know if a PR related with this would be of interest.

Full proto2 support requires quite a few changes, see https://github.com/timostamm/protobuf-ts/blob/master/MANUAL.md#proto2-support
I think it would translate into new code paths for the plugin, the runtime, and in the generated code as well. This will increase complexity quite a bit, and might also impact performance for the reflection based operations.
Maybe there is a way to selectively only special case proto2 required fields, but I'm not sure what exactly this would entail, and I'm wondering how far it would get us towards full proto2 compatibility. The conformance tests unfortunately only cover proto3.
Do you have a use case for serializing proto2 messages?

In this case I need to be compatible with some proto2 schemas between different programming languages. The only incompatibility encountered using this framework was the required fields not being written on serialization, but reading the provided link I see it being a bit more involved. I dont think this point 4 sentence is accurate, the framework infers and generates correctly all interface types.

required label is not supported: If a field has the required label, the type will be myField: bool with the default value false.

I was able to move forward by adding some quick and dirty code to InternalBinaryWrite.scalar. If protobuf-ts prefers to stay on proto3 only support what framework would you recommend for proto2 typescript code generations?

It looks like the official implementation and protobuf.js both support proto2. I recommend going with one of them. However, if you have a migration to proto3 on your roadmap somewhere, I would personally use protobuf-ts with a patch.


[Bug]: Motion detection/ person detection not working

Version
1.7.1
Node version
Docker
Operating System type
Docker
Operating system version
HAOS 11.1 / Eufy Security 7.7.3
Describe the bug
Motion detection and Person detection is always Clear. (No change in status to detected)
To reproduce

Walk in front of the wired doorbell (T8200X)
Go to Logbook or History
Notice that nothing is registered against Doorbell Motion Detected or Doorbell Person Detected (binary sensors)

Screenshots & Logfiles
home-assistant_eufy_security_2023-11-08T01-46-05.507Z.log
402f1039_eufy_security_ws_2023-11-08T01-40-00.924Z.log
Additional context

See also issue noted with Eufy Security integration
https://github.com/fuatakgun/eufy_security/issues/986

I have the same issue, I don't appear to receive events from my 2C cameras. I need to restart the Add-on for any updates to be received.

In my case, a restart does not solve the problem.  The sensors have never changed even with the numerous version updates.PremSent from my iPadOn Nov 14, 2023, at 3:15 AM, manlymgmt-anthony @.***> wrote:
I have the same issue, I don't appear to receive events from my 2C cameras. I need to restart the Add-on for any updates to be received.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Same issue for me I put a restart add on Eufy WS every hour but it's not a real solution

+1 using T8222 Door bell.

I have the same issue for all my wired cameras.

Same for me. No motion detection at all and some strange stuff in the logs of the addon:
          524           ],           errorCodes: [             'ETIMEDOUT',             'ECONNRESET',             'EADDRINUSE',             'ECONNREFUSED',             'EPIPE',             'ENOTFOUND',             'ENETUNREACH',             'EAI_AGAIN'           ],           maxRetryAfter: undefined,           calculateDelay: [Function: calculateDelay],           backoffLimit: Infinity,           noise: 100         },         localAddress: undefined,         method: 'POST',         createConnection: undefined,         cacheOptions: {           shared: undefined,           cacheHeuristic: undefined,           immutableMinTimeToLive: undefined,           ignoreCargoCult: undefined         },         https: {           alpnProtocols: undefined,           rejectUnauthorized: undefined,           checkServerIdentity: undefined,           certificateAuthority: undefined,           key: undefined,           certificate: undefined,           passphrase: undefined,           pfx: undefined,           ciphers: undefined,           honorCipherOrder: undefined,           minVersion: undefined,           maxVersion: undefined,           signatureAlgorithms: undefined,           tlsSessionLifetime: undefined,           dhparam: undefined,           ecdhCurve: undefined,           certificateRevocationLists: undefined         },         encoding: undefined,         resolveBodyOnly: false,         isStream: false,         responseType: 'json',         url: URL {           href: 'https://firebaseinstallations.googleapis.com/v1/projects/batterycam-3250a/installations',           origin: 'https://firebaseinstallations.googleapis.com',           protocol: 'https:',           username: '',           password: '',           host: 'firebaseinstallations.googleapis.com',           hostname: 'firebaseinstallations.googleapis.com',           port: '',           pathname: '/v1/projects/batterycam-3250a/installations',           search: '',           searchParams: URLSearchParams {},           hash: ''         },         pagination: {           transform: [Function: transform],           paginate: [Function: paginate],           filter: [Function: filter],           shouldContinue: [Function: shouldContinue],           countLimit: Infinity,           backoff: 0,           requestLimit: 10000,           stackAllItems: false         },         setHost: true,         maxHeaderSize: undefined,         signal: undefined,         enableUnixSockets: false       }     },     message: 'FidRegistrationFailedError: FID registration failed',     context: {       fid: 'ftCEprWMxxIbVTSO31BJEL'     },     stacktrace: 'FidRegistrationFailedError: FID registration failed\n' +       '    at PushNotificationService.registerFid (/usr/src/app/node_modules/eufy-security-client/build/push/service.js:136:19)\n' +       '    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n' +       '    at async PushNotificationService.createPushCredentials (/usr/src/app/node_modules/eufy-security-client/build/push/service.js:200:16)\n' +       '    at async PushNotificationService._open (/usr/src/app/node_modules/eufy-security-client/build/push/service.js:709:32)\n' +       '    at async PushNotificationService.open (/usr/src/app/node_modules/eufy-security-client/build/push/service.js:771:13)\n' +       '    at async Timeout._onTimeout (/usr/src/app/node_modules/eufy-security-client/build/push/service.js:781:21)'   },   credentials: undefined,   renew: false } 2024-03-14 15:20:30.562	INFO	eufy-security-ws:eufy-security-client	[main] Push notification connection closed 2024-03-14 15:20:30.563	ERROR	eufy-security-ws:eufy-security-client	[push] [PushNotificationService._open] Push notifications are disabled, because the registration failed! {   credentials: undefined,   renew: false } 2024-03-14 15:20:30.565	INFO	eufy-security-ws:eufy-security-client	[push] [PushNotificationService.open] Retry to register/login for push notification in 30 seconds...

My S330 detects motion but does not yield a persons name. Looking at the app though the person is being recognized. The battery powered cameras work just fine.
So the Data that is being received by the doorbell looks as follows:
{
  "message": {
    "name": "Doorbell",
    "event_time":<PHONE_NUMBER>921,
    "type": 91,
    "station_sn": "T8030Pxx",
    "device_sn": "T8213Pxx",
    "push_time":<PHONE_NUMBER>531,
    "title": "Eufy Security",
    "content": "Someone has been spotted",
    "channel": 2,
    "cipher": 0,
    "event_session": "20240710_173050",
    "event_type": 3102,
    "file_path": "/media/mmcblk0p1/Camera02//zx/hdd_data1/Camera02/202407/20240710173049/20240710173049.zxvideo.dat",
    "pic_url": "https://security-app-eu.eufylife.com/v1/s/g/REDACTED",
    "push_count": 2,
    "notification_style": 3
  }
}

The cameras however:
{
  "message": {
    "name": "Parking Lot",
    "event_time":<PHONE_NUMBER>121,
    "type": 23,
    "station_sn": "T8030Pxxx",
    "device_sn": "T8161Pxxx",
    "push_time":<PHONE_NUMBER>721,
    "title": "Eufy Security",
    "content": "Vehicle has been spotted",
    "channel": 0,
    "cipher": 0,
    "event_session": "20240710_173004",
    "event_type": 3107,
    "file_path": "/zx/hdd_data1/Camera00/202407/20240710173000/20240710173000.zxvideo",
    "pic_url": "https://security-app-eu.eufylife.com/v1/s/g/REDACTED",
    "push_count": 1,
    "notification_style": 2,
    "storage_type": 1,
    "msg_type": 18,
    "person_name": null,
    "person_id": null,
    "tfcard_status": null,
    "user_type": null,
    "user_name": null,
    "station_guard_mode": null,
    "station_current_mode": 1,
    "alarm_delay": null,
    "sound_alarm": null
  }
}

Note that it makes no difference if a vehicle, pet or person has been detected. The cameras will always yield the full info block with potential recognized persons but the door bell will never include such information in the push notification.
So the issue is that either the API does not send such information or the data model does not handle it.


🐛 Accepts dRep Registration with Mismatched Anchor URL and Registration Data

Area
Other
Domain
preview.gov.tools
Which wallet were you using?
No response
Context
When attempting to register a dRep with a valid field entered, but with an anchor URL that does not exactly match the registration data, the system currently accepts the registration without displaying the error message: Your External Data Does Not Match the Original File. Instead, the registration proceeds successfully, and the dRep information displayed includes the data from the anchor URL.
Steps to reproduce

Connect your wallet.
Navigate to the dRep registration section.
Fill in the valid registration data and proceed.
Enter an anchor URL with information that differs from the dRep registration data.
Observe the transaction process.
Review the registered dRep information.

Actual behavior
The system accepts the dRep registration despite the mismatch between the anchor URL and the registration data.

Anchor URL: https://metadata.cardanoapi.io/data/dRep
Registration Data:
Hello.jsonId
{
  "@context": {
    "CIP100": "https://github.com/cardano-foundation/CIPs/blob/master/CIP-0100/README.md#",
    "CIP119": "https://github.com/cardano-foundation/CIPs/blob/master/CIP-0119/README.md#",
    "hashAlgorithm": "CIP100:hashAlgorithm",
    "body": {
      "@id": "CIP119:body",
      "@context": {
        "references": {
          "@id": "CIP119:references",
          "@container": "@set",
          "@context": {
            "GovernanceMetadata": "CIP100:GovernanceMetadataReference",
            "Identity": "CIP100:IdentityReference",
            "Link": "CIP100:LinkReference",
            "Other": "CIP100:OtherReference",
            "label": "CIP100:reference-label",
            "uri": "CIP100:reference-uri",
            "referenceHash": {
              "@id": "CIP119:referenceHash",
              "@context": {
                "hashDigest": "CIP119:hashDigest",
                "hashAlgorithm": "CIP100:hashAlgorithm"
              }
            }
          }
        },
        "paymentAddress": "CIP119:paymentAddress",
        "givenName": "CIP119:givenName",
        "image": "CIP119:image",
        "objectives": "CIP119:objectives",
        "motivations": "CIP119:motivations",
        "qualifications": "CIP119:qualifications",
        "doNotList": "CIP119:doNotList"
      }
    },
    "authors": {
      "@id": "CIP100:authors",
      "@container": "@set",
      "@context": {
        "name": "http://xmlns.com/foaf/0.1/name",
        "witness": {
          "@id": "CIP100:witness",
          "@context": {
            "witnessAlgorithm": "CIP100:witnessAlgorithm",
            "publicKey": "CIP100:publicKey",
            "signature": "CIP100:signature"
          }
        }
      }
    }
  },
  "authors": [],
  "hashAlgorithm": "blake2b-256",
  "body": {
    "givenName": "Hello ",
    "motivations": "test",
    "objectives": "test",
    "paymentAddress": "addr_test1qqk7c8nvx7yeutsuhgj92lulfnc6cyest4vdmycn9z077t3pmhhpdhdmyvqysl53q3387ruud46klus49vs7acxdpunshlrgcl",
    "qualifications": "test",
    "references": [
      {
        "@type": "Other",
        "label": "Label",
        "uri": "test.com"
      }
    ]
  }
}

Preview dRep Info Post-Registration:

Expected behavior
An error message should be displayed indicating that the external data does not match the original file, preventing the registration from proceeding.

I can't reproduce that - could you please provide more info from the network tab for the /validate request?

@MSzalowski It can no longer be reproduced. It seems your changes have resolved the issue.

Closing as per the comment above.


Use CLI for bundling

Description

Switch to @swup/cli for bundling
Correctly separate dependencies from dev dependencies
Includes version bump and changelog update

Drive-By

Code cleanup, more readable logging

Checks

[x] The PR is submitted to the master branch
[x] The code was linted before pushing (npm run lint)

@daun Ready :)


[moleculer-mail] had issues importing in to a typescript moleculer project

When I try to include moleculer-mail into a typescript moleculer service, I kept greeting an error about missing types.
I did try making my  own local type.d.ts, but even that did't wor
// type.d.ts
declare module 'moleculer-mail`

Did you solve this?


[QUESTION] Trying to install Version 5

We are trying to install version 5 of the sppkg. We are now seeing that "Installing SharePoint add-ins from the SharePoint Store will not be possible as of July 1, 2024"
How do we deploy to our tenant now?

@mabaugh83 Microsoft 365 Learning Pathways is not a SharePoint add-in. It is a custom web part using the SharePoint framework. It is not effected by this announcement. You can follow the instructions in the ReadMe for installation.


[DOCFIX] Clean up scalability tuning doc

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.5 please


Close modal

Adds a cancel button to photo upload modal. Clicking outside form closes modal. Modal closes after hitting submit.

Hey @appleJax I already implemented this


Attempt to de-reference a null object from RollupJob (as seen on Apex Jobs page)

caused by: System.NullPointerException: Attempt to de-reference a null object
Class.dlrs.RollupService.processScheduleItems: line 356, column 1
Class.dlrs.RollupJob.execute: line 48, column 1

Cause: This appears to be caused when using Schedule Mode and having upgraded to v2.0 with existing 'Lookup Rollup Summary Schedule Items' records in place (see corresponding tab). The new field 'Lookup Rollup Summary 2' on this object will not be populated on pre-existing records (new records are fine) and this causes the problem. Add this field to the layout and list view and sort by it to check.
Workaround: is to populate the 'Lookup Rollup Summary 2' field with the value (ID) of 'Lookup Rollup Summary' for each effected record (mass edit on list view is good for this) or delete the effected records and do Calculate to ensure all rollups are updated, then new Lookup Rollup Summary Schedule Items records will have this field populated correctly.
Code Fix is to ensure this code tolerates either the 'Lookup Rollup Summary' or 'Lookup Rollup Summary 2' fields containing a value, not just the new one.

Fixed v2.3.

I just upgraded from 2.4.2 to 2.11 and started seeing this exception. Also could this issue potentially impact other scheduled jobs? I am looking into it further.

Thanks @rajarak12 that is odd, this fix seemed to be ok at the time i have not heard of any issues until now. Maybe try the workaround described above? It is also possible its another issue. This will only impact your DLRS scheduled job called RollupJob not any others.

Hi was there any resolution to this error.  I'm on v2.9 and received the following error message for a portion of failed batches.
Failed to process batch for class 'dlrs.RollupJob' for job id '7074600002fNQY7
caused by: System.NullPointerException: Attempt to de-reference a null object
Class.dlrs.RollupService.processScheduleItems: line 380, column 1
Class.dlrs.RollupJob.execute: line 48, column 1


Supplier is assignable iff it has a contract

If a supplier doesn't have a contract it cannot be assigned.
Check if the bucket link for this supplier is non empty.
Best to wrap this logic in a getter on the supplier model (such as hasContract)
In the new supplier wizard it is OK to upload the file later, but make it clear that until that step is completed supplier cannot be assigned
Also to make it clear why it cannot be assigned, consider maybe displaying a prominent message on the supplier's page.

works


Cannot compile - use of 'clone_from_slice'

I cannot compile jpeg-decoder on my system. The compile throws the following error during build.
   Compiling jpeg-decoder v0.1.2
/Users/arsalanahmad/.cargo/registry/src/github.com-88ac128001ac3a9a/jpeg-decoder-0.1.2/src/decoder.rs:465:38: 465:136 error: use of unstable library feature 'clone_from_slice' (see issue #27750)
/Users/arsalanahmad/.cargo/registry/src/github.com-88ac128001ac3a9a/jpeg-decoder-0.1.2/src/decoder.rs:465                                     .clone_from_slice(&self.coefficients[scan.component_indices[i]][block_offset .. block_offset + 64]);

I'm on OS X and running rustc 1.6.0.

clone_from_slice was made stable in rust 1.7, so that version or higher is what's supported.


unknown keyword platform RC2

I have installed the recently release CORE R2 Tooling and created a new project. An attempt to restore packages fails with the above given error.
Here the command: dotnet restore
error: TODO: unknown keyword platform
unknown keyword platform
Any reason why this is happening

Some possible solutions:

Check your global.json or projects properties. You may be specifing old runtime.
Have you uninstalled all previous versions of .NET CLI (if any)
Have you installed the latest NuGet Package Manager Extension? Available here: https://dist.nuget.org/visualstudio-2015-vsix/v3.5.0-beta/NuGet.Tools.vsix
Check your NuGet feeds - they should be the official ones.

You have an old version of the RC2 tooling. Run dotnet --version on the command line to see what version is being used.

@davidfowl I downloaded the tooling installation file about 5 hours ago based on the link provided on http://dot.net page.

@takazm can you share your project.json file, and if you have any particular NuGet.config file that you're using?
@pakrym @anurse @pranavkm any idea what this error might be?

'type:platform'  vs 'target:platform' should be the first one.

@takazm , can you run dotnet --version. It's quite likely you have an older dotnet CLI installed somewhere on your disk.

running dotnet --version gives me the following output:
.NET Command Line Tools (1.0.0-beta-001598)                                                                                Product Information:                                                                                Version:     1.0.0-beta-001598                                                                                Commit Sha:  7582649f88                                                                                Runtime Environment:                                                                                OS Name:     Windows                                                                                OS Version:  10.0.14342                                                                                OS Platform: Windows                                                                                Runtime Id:  win10-x64
I have also noticed that when I create a new project using VS or through dotnet new; when I open the project it says dotnet is not installed but I did install and can see the files in the folder specified in the path

my global.json looks like this:
{
"projects": [ "src", "test" ],
"sdk": {
"version": "1.0.0-preview1-002702"
}
}
and the project.json:
{
"dependencies": {
"Microsoft.NETCore.App": {
"version": "1.0.0-rc2-3002702",
"type": "platform"
},
"Microsoft.ApplicationInsights.AspNetCore": "1.0.0-rc2-final",
"Microsoft.AspNetCore.Diagnostics": "1.0.0-rc2-final",
"Microsoft.AspNetCore.Mvc": "1.0.0-rc2-final",
"Microsoft.AspNetCore.Razor.Tools": {
"version": "1.0.0-preview1-final",
"type": "build"
},
"Microsoft.AspNetCore.Server.IISIntegration": "1.0.0-rc2-final",
"Microsoft.AspNetCore.Server.Kestrel": "1.0.0-rc2-final",
"Microsoft.AspNetCore.StaticFiles": "1.0.0-rc2-final",
"Microsoft.Extensions.Configuration.EnvironmentVariables": "1.0.0-rc2-final",
"Microsoft.Extensions.Configuration.Json": "1.0.0-rc2-final",
"Microsoft.Extensions.Logging": "1.0.0-rc2-final",
"Microsoft.Extensions.Logging.Console": "1.0.0-rc2-final",
"Microsoft.Extensions.Logging.Debug": "1.0.0-rc2-final",
"Microsoft.VisualStudio.Web.BrowserLink.Loader": "14.0.0-rc2-final"
},
"tools": {
"Microsoft.AspNetCore.Razor.Tools": {
"version": "1.0.0-preview1-final",
"imports": "portable-net45+win8+dnxcore50"
},
"Microsoft.AspNetCore.Server.IISIntegration.Tools": {
"version": "1.0.0-preview1-final",
"imports": "portable-net45+win8+dnxcore50"
}
},
"frameworks": {
"netcoreapp1.0": {
"imports": [
"dotnet5.6",
"dnxcore50",
"portable-net45+win8"
]
}
},
"buildOptions": {
"emitEntryPoint": true,
"preserveCompilationContext": true
},
"runtimeOptions": {
"gcServer": true
},
"publishOptions": {
"include": [
"wwwroot",
"Views",
"appsettings.json",
"web.config"
]
},
"scripts": {
"prepublish": [ "npm install", "bower install", "gulp clean", "gulp min" ],
"postpublish": [ "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%" ]
}
}

The CLI version is pretty ancient. I'd recommend uninstalling older versions of the CLI (check Control Panel -> Programs & Features) and also remove anything under %LocalAppData%\Microsoft\dotnet. You can install the CLI from https://go.microsoft.com/fwlink/?LinkID=798398. Odd that the tooling you installed didn't install the right CLI with it.

trying to uninstall old version but, i dont see %LocalAppData%\Microsoft\dotnet or the entry in programs in control panel (see image)

Can you run where dotnet to figure out where it's picking up the version of dotnet you have?

i did run that one and its in \program files\dotnet
Do i just delete the files or have to do a proper uninstall of which i dont see an uninstaller

Removing C:\Program Files\dotnet from your path might be an alternative to deleting it. Once done, reinstall the CLI from https://go.microsoft.com/fwlink/?LinkID=798398 for good measure.

Did install just fine on separate my personal laptop and seems to have no issue at all. I will have to uninstall everything including VS and try again. Thanks guys

@takazm I'm glad you got it working. There were at a few points in time some issues with the installers, especially for those who installed nightly builds and then the official release. So hopefully it was just a one-time issue.


Orsi/sample apps

Pull request to merge Orsi Academy sample apps into Holohub public repository.

@JasperHofman The datasets is now live, can you update the link?

@jjomier I added the lines back


standardizing pubsub doc tags

Please do not merge, coordinating with doc updates to avoid broken links.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #659   +/-   ##
=========================================
  Coverage     55.84%   55.84%
  Complexity      202      202
=========================================
  Files            74       74
  Lines          2052     2052
  Branches        131      131
=========================================
  Hits           1146     1146
  Misses          874      874
  Partials         32       32

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 de6a65b...790312b. Read the comment docs.

Merging, no code changes.