FR: Long press for brightness

I've noticed while testing that it's quite easy to hit the right area of the icon instead of the actual icon when turning it off/on which opens the brightness options.
I would suggest that you introduce a long press on the button to open temperature/brightness rather than specifying regions of the button card.  Especially since color/temperature/brightness is something that isn't as commonly used as on/off and is currently covering 80% of the button card.

This is how HomeKit has implemented it, and it feels wrong at first. However, once you get used to it, it's much faster. I don't want to add long press just because we're used to it in Lovelace. The way I tried to solve it before was by adding double click instead, but it turns out Apple did it best, and that's what I'm after, the best implementation.
Left toggle "shortcut", right settings


Unavailble streams

Description
Hi there,
during the past days/weeks I've an worsening issue with my TC70 camera (which I believe is the same model as C200?).
It always starts with the HD stream not being available in Home Assistant anymore. When this happens, the SD stream usually is still available and when I check the Tapo app, the stream there seems to be HD still.
Later during the day, the SD stream also cuts off and is unavailable within HA. Sometimes the Tapo app is still working, but it also happened twice to me now, that even the Tapo app says, that I have "reached the device limit for playing this video". It's asking me to tap to retry, but this doesn't help either.
When this happens, the only fix is to reboot the camera.
What I mean by worsening is, that the HD stream did cut off like once a week, but the SD stream always stayed alive.
During the past week, both streams cut off once a day. So I am rebooting the camera every day.
I have the feeling that HA / your integration continuously pulls from the server which may be identified as a new device, or something.
Any idea how to fix this?
Thanks
André
PS: Maybe one more "hint": I've set the Stream identity on my dashboard to "live" rather than auto. For testing purpose I've just changed this to auto - which doesn't show me the stream on my Dashboard now anymore and instead just a grey error image, but maybe this helps.
Reproduction Steps
I have pretty much explained everything in the description.
Over time the streams just cut off, nothing special needs to be done from my end.
Expected behavior
No stream cuts :)
If applicable, add error logs.
No response
Device Firmware
1.3.11 Build 231121 Rel.39429n(4555)
Integration Version
5.4.13
Using stream component
Yes
Does camera work via official integrations?
N/A
Camera has all attributes filled out in developer tools
Yes
HASS Environment
HA VM under Proxmox
Search for similar issues
Yes
Additional information
No response

Check faq there is a limit of concurrent streams on camera you are hitting. App does not use rtsp stream.


Widevine CDM version selection and rollback support

This functionality has various aspects that are intertwined with:

The ability to backup Widevine CDM versions (up to 5, configurable) when updating
The ability to install/rollback a Widevine CDM version of your choice (from backup, or online)
~The ability to configure the interval to check for Widevine CDM updates~ #173
The ability to pin to a specific version (so a way to disable updating Widevine)

We would also like to have a way to detect when Widevine playback fails (see #69), in that case pinning to a specific Widevine CDM would get overruled when it fails to work.
These features were listed in the Ideas in our Wiki
A rationale for giving this kind of control to the user is available from issues like:

Incompatibility with newest Widevine on ARM systems (netflix#465)
Stuttering playback of streams using new widevine (inputstream.adaptive#305)

I really like the pin idea, and rollback!
Hope you can implement it soon :)
If I may ask for an extra option: give a way to save previous version on a configurable path, ie nfs or SMB, because it's a lot to keep on some Raspberry Pi (~2gb). That would be awesome.
Thanks!

The implementation requires us (on ARM) to also keep track of the Chrome OS version in relation to the Widevine version. Since multiple Chrome OS releases ship with the same Widevine image (and we only know after extraction), recording which Widevine ships with what Widevine version will offer us a table to select the right Chrome OS image for a given Widevine release.
We may want to ship our own table as a reference, and InputStream Helper will continue to register newer versions going forward. I think this offers the best user experience on ARM.

From what I understood, not the full 2GB image is kept. The widevine files are extracted and the image is removed again.

So obvious and space saving!

Would be great to at least have an "Ignore" option when inputstreamhelper detects an outdated widevine version.
You probably already know, but just in case:
Right now, on ARM, you can't use plugins that use inputstreamhelper even though the outdated widevine version (4.10.1440.18) is able to play the streams.
I found that disabling inputstreamhelper is a workaround for the time being.
P.S.: Thank you very much for all of your great work!

@the-isz Please have a look at #191 if that somewhat relates to your situation. If I understand correctly you get an update message every time you try to play something? Not only each 14/31/... days?

I don't really understand what would be the difference between "pinning" to a specific version and disabling after installing that version.

In fact version 4.10.1440.18 is the latest available Widevine version.

@horstle Yes, exactly, I'm getting a popup telling me that a newer widevine version is needed to play the content every time I try to play a video. Then, I can either start the update or cancel. If I choose to cancel, the video isn't played.
@dagwieers Oh, OK. I thought I had read that there were 4.10.1440.19 versions out there for other platforms (e.g. x64) and I also saw that version in versions.txt. But if you say that .19 is the latest available then we might even have another problem here. Either:

inputstreamhelper is trying to update although the most recent version is already installed or
the fact that I'm manually updating widevine is confusing inputstreamhelper

FYI: I'm manually updating inputstreamhelper because I don't have enough disk space on my raspberry to download and unpack the Chrome OS recovery images.

@the-isz There is a new version called .19, but it's identical to the .18 version. That's one of the reasons it's trying to update. The other one is that it still checks the widevine_config.json to see which version is installed, which you presumably don't change (so it's your 2. assumption).
There is a fix for the disk space problem  in master, which should be released soon, but you're welcome to test now by downloading the repo as a .zip and installing manually.

FYI: I'm manually updating inputstreamhelper because I don't have enough disk space on my raspberry to download and unpack the Chrome OS recovery images.

If I'm correct, then InputStream Helper thinks you didn't install the last version. There's no binaries check, just it compares what it gets in its settings.xml file.
If you update manually, you should also update the settings.xml file so it matches the current version installed.

@horstle I actually manually update widevine_config.json as well.
@Smeulf Can you give me more details which settings.xml to update and what to update it with? Until now, libwidevinecdm.so and widevine_config.json used to be sufficient.

Can you give me more details which settings.xml to update and what to update it with? Until now, libwidevinecdm.so and widevine_config.json used to be sufficient

@the-isz Sorry, just checked again, and I was wrong thinking the current version is in settings.xml.

With PR #193 merged we can close this issue. Any issues with the current master branch should be opened as a new issue. Please test !


Minimum numpy requirement should be at least 2.0

Since we're using trapazoid function, minimum numpy version should be 2.0.

modify the pyproject.toml.


Rich text editor doesn't allow all content to be deleted.

Laravel Version: 5.4.25
Voyager Version: 0.11.12
PHP Version:5.6.3
Database Driver & Version: 10.0.30-MariaDB,

Rich text editor won't save empty content
Steps To Reproduce:
Created database table
Added Bread
configured form fields
I have a form with a rich text editor, I can add / edit the content but, if I delete all the content and save, the content remains.
Any idea?
Thanks

Hi @gavinj1999,
That is a known bug.

thanks for the reply, any workarounds?

Try to set validation for this field with sometimes and nullable parameters. "sometimes|nullable"

Does that work @AdrianKuriata?
I don't know any workaround atm @gavinj1999 :/ Please let us know if Adrian solution worked out.

@akazorg i hope it will work, i doing it in laravel when i want save null for any field. When you try save empty field with laravel, into $request this field will not figure so he cant save empty field. When you add sometimes and nullable, sometimes will be present and can be empty and will be only validating when you put something into field. You can set present, this will let you be empty but if ypu add parameter something like min:3 this will be validate so better solution is set sometimes|nullable if your field is not required. Check laravel docs.

Thank's Andrian, will check that PR asap.


test_coder_operators test fails

Information
The test_coder_operators test case fails due to an assertion error in both slow test and pr-tests even though it appears that the values compared are equal

Fixed when runtime service was removed https://github.com/Qiskit/qiskit-ibm-provider/pull/243


[Network] az network lb address-pool : Support subnet when creating or updating an IP-based backend pool of a load balancer.

Description

We have added subnet ID and IP address as a way to specify backend address when creating or updating an IP-based backend pool of a load balancer. This is related to this feature: #12937 however, we will start allowing customers to specify by subnet ID in place of vnet ID.
link issue : https://github.com/Azure/azure-cli/issues/16808
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network


Calculate hotplug attachment pod requests/limits based on VMI ratio

What this PR does / why we need it:
Have the hotplug attachment pod request/limit ratio follow the request/limit ratio of the VMI it is associated with. So if the request/limit ratio is 1 on the VMI, then the attachment pod will also have that ratio. This ONLY fixes the hotplug attachment pod for any ratio (ratios can be fractions as well). With higher ratios and a minimum request set in a LimitRange, it is possible this will result in the request value being below the minimum in the LimitRange, and the pod creation is rejected.
This change will result in most requests in the attachment pods being higher than the hard coded value before due to making sure we respect the ratio.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/approve

/hold


Flaw in logic?

I am unable to detect the exact flaw in the logic, but this scenario gives "Sorry, no other possible words.  Please check the result symbols you entered.
Please enter you last try as word:symbols" after the ORGAN guess.
The solution is RAGAS and all the words are in the English list files. I checked.
CLEAR:__+?
ORGAN: ?++
SUGAR:?++?
RAGAS:+++++

By default, the word-list loader ignores files ending in s that don't have two consecutive s at the end (like guess.) If you want to fix this, change the code:
self.word_list = Utility.load_word_list(self.word_list_file_path, word_length, True)

so that the last parameter is False (not True) so the word-list loader won't drop RAGAS from the list:
Please enter you last try as word:symbols
CLEAR:___+?
Suggested words (from simple word list):
        broad
        urban
        roman
        bryan
        brian
        array
        strap
        organ
        sarah
        spray
Please enter you last try as word:symbols
ORGAN:_?++_
Suggested words (from extended word list):
        ragas

@jasonvmiller  Thank you so much for pointing out the issue.   Frankly, the exclusion of plurals is crucial in improving performance.  I will see what would be the better way to exclude the plurals while not wrongly filtering out nouns ended with a single "s".
I am sorry for the slow reply.  I was busy working on a web-based UI for the solver, which will be out in the next 24 hours.  The web UI will expose the option of "including plurals" to the users.

@jason-chao tyvm for releasing this, it's wonderful and I enjoyed reading your solution!

@jasonvmiller  and @venomoustoad : please check the new web-based UI for the solver at https://wordle-solver.jasontc.net/.  This web UI provides the option to "include plurals" in the search for suggested words.   The issue with "RAGAS" case should be addressed when you check "include plurals".
I will also update the CLI version in the repo to allow users to turn on the "include plurals" option.

You may now pass --plurals in the arguments to wordle_solver.py and wordle_game.py to prevent the exclusion of plurals from the word list.


Messages not displaying correctly

General Checkups

[*] Have you checked that there isn't already an existing issue that describes what you report below?
[*] Have you checked that there isn't already an open pull requests for this issue/update/change?

Description
Messages do not display on the main page, but do 'stack up' on other pages.

fixed #914


Date posted

Add 'date posted' to case studies.

Coverage remained the same at 91.391% when pulling 6a5ea6baab4ef87e1b84444833467783833cafcb on date-posted into 37d4889786728cd692192f9aa1a7bc6174b0fc5f on master.


[azservicebus] Updating to go-amqp:ca689527ff4c98693075d2fbc0ef98ef415fd08b

Absorbing go-amqp@ca689527ff4c98693075d2fbc0ef98ef415fd08b
This contains a bug fix for settlement hanging if the receiver had been closed. This showed up in our constant-detach stress tests - we'd receive messages but then hang for an indeterminate time while trying to complete messages. The problem (from what I could trace from the log) was that the go-amqp Receiver didn't check the local link state before attempting to disposition a message. This would result in us waiting infinitely for a disposition response that would never happen.
The other changes are primarily just go-amqp transitioning from variadic config to plain old struct config.

(stress test launched as ripark-preview-disp-fix)

/azp run go - azservicebus

/azp run go - azservicebus

/azp run go - azservicebus


feat(network): add single node networking capabilities

This PR defines a new crate, to be used by the agent, when creating a new node or a new instance.
It allows :

[x] Creating a bridge to be used by virtual interfaces later on
[x] Setting up network configuration, and iptables rules
[x] Creating a network namespace to be used by instances
[x] Creating virtual interfaces for this namespace
[x] Setting up NAT/PAT rules for this interface
[x] Cleaning up both nodes & instances

The documentation has also been updated.

- delete api.proto, since we're not going the gRPC route anymore I understand you're going the rust crate way, that's good.
Please use a specific commit that would update the documentation and remove the grpc protos. Dont mix the proto removal with code that adds the new API.


Background Image is not Showing

Hello. I'm having this problem that should not be happening.
I've placed the background image in the resource folder, called it in the script, but nothing happens. Stays that pink and black image.
Take a look:

make a folder in the images folder labeled "textures" then use this: "imagens:textures/background.png",

Ok, I will try it.

make the .png in caps it worked for me

Ok, I will try it.

it worked?

I don't remember. Sorry.
¯_(ツ)_/¯


Unknown custom element: ,  and   and MDI icons not showing

On registering  VuetifyImageInput globally or in any individual component, i get errors in console saying:
[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.
[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.
[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.
Additionally, MDI-ICONS aren't showing

see #20 or #17


fix: bug fixes in run_vcpkg function

Fixes some bugs in the automatic installation of vcpkg and also automatic updating of the repository.

I have tested it, and it seems it works.


Add the option to Block Razer automatic installs

Type of Change

[x] New feature

Description
Added the option to block automatic installation of razer software using the code from the chris titus original Block-Razer repo.
Testing
Tested for the following cases:

Win 11 Fresh
Win 11 with script used from this repo

Issue related to PR

Resolves #550

Additional Information

Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no errors/warnings/merge conflicts.

Hello @xicor22,
Thank you for your PR but this would be a tweak and not a feature. Please could this be moved to tweaks.json under the Advanced Tweaks Section.

"WPFRazerBlock": {
    "Content": "Block Razer software autoinstall",
    "Description": "It disables the automatic driver installation and denies write permission of Razer folder to system which prevents the automatic install.",
    "category": "Essential Tweaks",
    "panel": "1",
    "Order": "a017_",
    "registry": [
      {
        "Path": "HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\DriverSearching",
        "Name": "SearchOrderConfig",
        "Type": "DWord",
        "Value": "0",
        "OriginalValue": "1"
      },
      {
        "Path": "HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Device Installer",
        "Name": "DisableCoInstallers",
        "Type": "DWord",
        "Value": "1",
        "OriginalValue": "0"
      }
    ],
    "InvokeScript": [
      "Remove-Item \"C:\\Windows\\Installer\\Razer\" -Recurse -Force",
      "New-Item -Path \"C:\\Windows\\Installer\" -Name \"Razer\" -ItemType \"directory\"",
      "$Acl = Get-Acl \"C:\\Windows\\Installer\\Razer\"",
      "$Ar = New-Object System.Security.AccessControl.FileSystemAccessRule(\"NT AUTHORITY\\SYSTEM\",\"Write\",\"ContainerInherit,ObjectInherit\",\"None\",\"Deny\")",
      "$Acl.SetAccessRule($Ar)",
      "Set-Acl \"C:\\Windows\\Installer\\Razer\" $Acl"
    ],
    "UndoScript": [
      "Remove-Item \"C:\\Windows\\Installer\\Razer\" -Recurse -Force",
      "New-Item -Path \"C:\\Windows\\Installer\" -Name \"Razer\" -ItemType \"directory\""
    ],
    "link": "https://christitustech.github.io/winutil/dev/tweaks/Essential-Tweaks/RazerBlock"
  }

I added this in the tweaks.json and got this error

I have the same format as other tweaks or have I missed something?

I accidently put this in the essential section, lemme change that.

"WPFRazerBlock": {
    "Content": "Block Razer software autoinstall",
    "Description": "It disables the automatic driver installation and denies write permission of Razer folder to system which prevents the automatic install.",
    "category": "Essential Tweaks",
    "panel": "1",
    "Order": "a017_",
    "registry": [
      {
        "Path": "HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\DriverSearching",
        "Name": "SearchOrderConfig",
        "Type": "DWord",
        "Value": "0",
        "OriginalValue": "1"
      },
      {
        "Path": "HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Device Installer",
        "Name": "DisableCoInstallers",
        "Type": "DWord",
        "Value": "1",
        "OriginalValue": "0"
      }
    ],
    "InvokeScript": [
      "Remove-Item \"C:\\Windows\\Installer\\Razer\" -Recurse -Force",
      "New-Item -Path \"C:\\Windows\\Installer\" -Name \"Razer\" -ItemType \"directory\"",
      "$Acl = Get-Acl \"C:\\Windows\\Installer\\Razer\"",
      "$Ar = New-Object System.Security.AccessControl.FileSystemAccessRule(\"NT AUTHORITY\\SYSTEM\",\"Write\",\"ContainerInherit,ObjectInherit\",\"None\",\"Deny\")",
      "$Acl.SetAccessRule($Ar)",
      "Set-Acl \"C:\\Windows\\Installer\\Razer\" $Acl"
    ],
    "UndoScript": [
      "Remove-Item \"C:\\Windows\\Installer\\Razer\" -Recurse -Force",
      "New-Item -Path \"C:\\Windows\\Installer\" -Name \"Razer\" -ItemType \"directory\""
    ],
    "link": "https://christitustech.github.io/winutil/dev/tweaks/Essential-Tweaks/RazerBlock"
  }

I added this in the tweaks.json and got this error
I have the same format as other tweaks or have I missed something?

This could be due to issues with tweaks currently.

even though i select another tweak to get around that bug, my razer block tweak is still not getting applied

"WPFTweaksRazerBlock": {
    "Content": "Block Razer software autoinstall",
    "Description": "It disables the automatic driver installation and denies write permission of Razer folder to system which prevents the automatic install.",
    "category": "z__Advanced Tweaks - CAUTION",
    "panel": "1",
    "Order": "a031_",
    "InvokeScript": [
      "
        function RazerBlock {
          $RazerPath = \"C:\\Windows\\Installer\\Razer\"
          Set-ItemProperty -Path \"HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\DriverSearching\" -Name \"SearchOrderConfig\" -Type DWord -Value 0
          Set-ItemProperty -Path \"HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Device Installer\" -Name \"DisableCoInstallers\" -Type DWord -Value 1
          Remove-Item $RazerPath -Recurse -Force
          New-Item -Path \"C:\\Windows\\Installer\\\" -Name \"Razer\" -ItemType \"directory\"
          $Acl = Get-Acl $RazerPath
          $Ar = New-Object System.Security.AccessControl.FileSystemAccessRule(\"NT AUTHORITY\\SYSTEM\",\"Write\",\"ContainerInherit,ObjectInherit\",\"None\",\"Deny\")
          $Acl.SetAccessRule($Ar)
          Set-Acl $RazerPath $Acl
          }
        RazerBlock
      "
    ],
    "UndoScript": [
      "
        function RazerUnblock {
          $RazerPath = \"C:\\Windows\\Installer\\Razer\"
          Set-ItemProperty -Path \"HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\DriverSearching\" -Name \"SearchOrderConfig\" -Type DWord -Value 1
          Set-ItemProperty -Path \"HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Device Installer\" -Name \"DisableCoInstallers\" -Type DWord -Value 0
          Remove-Item $RazerPath -Recurse -Force
          New-Item -Path \"C:\\Windows\\Installer\\\" -Name \"Razer\" -ItemType \"directory\"
          }
        RazerUnblock
      "
    ],
    "link": "https://christitustech.github.io/winutil/dev/tweaks/Essential-Tweaks/RazerBlock"

I looked at this for quite a while and cant seem to find any issues but the code in invoke script is not running.
The previous "select a tweaks" error was caused because I didn't put "Tweak" in "WPFRazerBlock".

Hello @xicor22, thanks for this PR. WinUtil offers a way of easily add registry changes like in following example.
  "WPFTweaksRazerBlock": {
    "Content": "Block Razer Software Installs",
    "Description": "Blocks ALL Razer Software installations. The hardware works fine without any software.",
    "category": "Essential Tweaks",
    "panel": "1",
    "Order": "a007_",
    "registry": [
      {
        "Path": "HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\DriverSearching",
        "Name": "SearchOrderConfig",
        "Value": "0",
        "OriginalValue": "1",
        "Type": "DWord"
      },
      {
        "Path": "HKLM:\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Device Installer",
        "Name": "DisableCoInstallers",
        "Value": "1",
        "OriginalValue": "0",
        "Type": "DWord"
      }
    ],
    "InvokeScript": [
      "

      "
    ],
    "UndoScript": [
      "
      "
    ]
  },

Hey @xicor22, thanks for the PR, you moved the reg changes into the invoke- & undo script again, could you please add them sepeately?
Also the Tweaks generally did not work temporarily, but should work now again. Also the function wrapping and calling kind of seems unneccessary due to it only beeing called there.
Otherwise I think it looks good :)

@MyDrift-user I'm sorry to ask you this but how do i open a new PR without the changes from this PR? I tried creating a new branch and creating a PR from that but this PR was also included in that one. I'm very new to code contribution so dont really know about this.

You forgot to remove the comma in feature.json


Increase Max File Size limit

On multiple case i have used values greater than 4096. We shouldn't be limiting it to a low value, epically when it comes to circular logging. Increase this value to something like 30-40GB so we don't run into odd issues like this.

Removed size limits from the maxsize var.  Should now accept any valid int.


Regional differences in number formats can cause OpenCAGE to destroy floats

An issue was discovered with regional number formats that causes floats to be converted into integers.  If the system region number format is in the format of: xx,xxx.yy (thousands separator is a comma and decimal indicator is a period) there is no problem.  However, if the system region number format is: xx.xx,yy (thousand separator is a period and decimal indicator is a comma) floats get converted to integers like so:
30.1 becomes 301
This of course, breaks things.
Issue can be reproduced by setting the system number format to e.g. English (Sweden) and attempting to edit nodes in OpenCAGE containing floats.  Saving and reloading the file will have them as integer numbers instead of floats.  Trying to restore the float by entering it in the field throws an exception.
Resetting the region to e.g. English (United Kingdom) or English (United States) causes the issue to go away.
In summary, OpenCAGE doesn't seem to take into consideration number formats in different regions resulting in broken maps.

Good spot! I assume this is targetted specifically at the script editor?

Yes, that's where the trouble lies.

Please could you provide screenshots of some examples where this occurs, and the entities/composites/levels that you've experienced it in? I'll get a fix up on staging once I can replicate it. Cheers!

I've just pushed an update out to the staging branch which should hopefully fix this issue. Within OpenCAGE's first screen, select the cog icon, tick "receive updates from staging branch" and click apply. You should receive the update, and be on version <IP_ADDRESS> when OpenCAGE reopens.
If this update hasn't fixed it, please could you provide screenshots of some examples where it occurs, and the entities/composites/levels that you've experienced it in?

Appears to be fixed in <IP_ADDRESS>.

Awesome, thanks!


fix: github workflow

Fixes #57

Added build step for swift, node and yarn
Fixes npm tests to refer to npm/test while looking for generated node_modules
Fixes npm parser which was erroring out with a nil pointer dereferencing error while parsing nested dependencies. (Although a better fix is to use types and not maps)

Finally a green tick 🎉
@cpanato @puerco @nishakm can you review and merge if this looks good?


CS GO Api

Hallo,
sorry for Posting this as an Issue, but have not found a better Place for.
Im searching for Information how to get Game events to External App, because im trying to get the same Features under Linux, with something Like this https://github.com/MatMoul/g810-led .
Can you give few Hints where i can start to understand how you grab from Games, for Example the Placed Bomb in CS.GO.
Best Regards
Andreas

This page has some useful information for csgo: https://developer.valvesoftware.com/wiki/Counter-Strike:_Global_Offensive_Game_State_Integration.
Dota 2 works in a similar way.


Issue 11439

https://github.com/zulip/zulip/issues/11439
When you click on your own name in the buddy list, we should go right to the popup
Testing Plan:
Click on my your name in buddy list and user info popup should appear instead of a Private message narrow with yourself.
GIFs or Screenshots:

@showell @timabbott Ready for review

@timabbott This LGTM.  It's a pretty concise way of doing it using selectors and a little extra markup.  The other alternative would be for the handler to check people.is_my_user_id or something like that and dispatch accordingly.
I think I like this approach.

Actually, one thing--I believe these can all be squashed into a single commit.

Thanks for the review. I will squash them into a single commit message.

FYI here's some feedback from Rishi:
https://chat.zulip.org/#narrow/stream/101-design/topic/user.20status.20design/near/694978
Tim, it's probably worth giving this a quick look, anyway, just to get a feel for it.

@showell Squashed the 3 commits into one.

Cool; I'm going to defer looking at this until I have a 2.0 release candidate posted.

@timabbott Please take a look when you get time. Thank you

The code is a fine implementation of the concept; I think there's some question on whether we want to make this change from a product perspective.  @thedeveloperr I encourage you to move on to other issues for now, and we'll leave this open until we can make a decision.


fix [].find error

Chrome (version: 44.0.2403.125) does not support  [].find, so change to [].filter.

Vue.js devtools version: 2.3.1

Why would you be using Chrome 44? It's 10 versions behind the current.

Coincidentally I had a discussion in an issue some days ago, with someone wroking in a bank that was "stuck" on v44 because of company policy. @simplesmiler suggested this could be because v44 was the last one to support the Netscape plugin AIP (NPAPI)

find stops after having found an element, while filter scans entire array even it already found one.

Closing as this is addressed by #240


Dealing with symlink pointing to directories (fixes #11)

Now shell-safe-rm will consider a symlink to a directory like a regular file; Added indentation in directory condition

This will solve #11

temporarily merge


Allow users to report stalls with no sausage sizzles without publicly exposing their location

Goes to the old Add Stall vs Report Stall debate.

https://twitter.com/geeky_dave/status/1109714112303366145
AB#39

Closing and merging into https://github.com/keithamoss/demsausage/issues/1855


"unsubscribe.bulkmailer.com" example is both listed as correct and incorrect

In the section https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/troubleshooting-mail-sent-to-office-365?view=o365-worldwide#ensure-that-email-message-content-is-transparent-and-traceable
The "unsubscribe.bulkmailer.com" example is both listed as correct and incorrect.

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

ID: 95417878-bc32-8b18-9900-6ad4f10da2a9
Version Independent ID: 30cd3f4e-9c51-f2fa-b6c0-036756f6b99e
Content: Troubleshooting mail sent to Microsoft 365 - Office 365
Content Source: microsoft-365/security/office-365-security/troubleshooting-mail-sent-to-office-365.md
Service: o365-seccomp
GitHub Login: @chrisda
Microsoft Alias: chrisda

@PatrickMessierGoC, I think I see what's going on (although it took me a minute):
The 'correct' redirects/links are correct because they all use the same domain (bulkmailer.com). The 'incorrect' redirect/links are incorrect because they all use different domains. The fact that bulkmailer.com appears under 'incorrect' isn't so important; what's important is bulkmailer.com is different than excite.com is different than yahoo.com. This aligns with the statement in the previous paragraph:

... If you have a lot of advertising or Unsubscribe links or Update the Profile links, they should all point to the same domain. ...

I added some clarifying text to Correct (all domains are the same) and Incorrect (all domains are different).
Thanks for the feedback,
Closing

@PatrickMessierGoC, I think I see what's going on (although it took me a minute):
The 'correct' redirects/links are correct because they all use the same domain (bulkmailer.com). The 'incorrect' redirect/links are incorrect because they all use different domains. The fact that bulkmailer.com appears under 'incorrect' isn't so important; what's important is bulkmailer.com is different than excite.com is different than yahoo.com. This aligns with the statement in the previous paragraph:

... If you have a lot of advertising or Unsubscribe links or Update the Profile links, they should all point to the same domain. ...

I added some clarifying text to Correct (all domains are the same) and Incorrect (all domains are different).
Thanks for the feedback,
Closing


Application Object Tree

In AX we have an Application object explorer, it is very much the same as C/AL Object Designer but in a tree view.
I'm so missing out this visibility when doing development in AL. A couple of these abilities would make the development much easier.

we able to see the existing standard code and even put a break point on them.
right click on any table/code unit public procedure to subcrible a pre and post event.

I'm looking forward for these features in the next AL update.

While waiting I recommend the AL Code Outline extension for VScode. It does some of what you request.

Thank you, I have been using that VS Code extension, It has many nice features .
I'm looking for the way to view existing standard actual code. Learning from existing standard code, the ready made library are very valuable.

Today the AL Explorer (for all objects including dependencies) and project outline should serve this need, so I am closing this issue.


Remove deprecated PatternFly components

Components that were deprecated as part of the v5 PatternFly release are still in use.
Those deprecated components should be replaced with the non-deprecated components or approaches PatternFly now recommends.

Closed with https://github.com/patternfly/patternfly-quickstarts/pull/263


AI can view Entertainment camera

About the PR

makes it so the AI can view the feed from entertainment cameras
Why / Balance

AI had no way of getting a  movable camera.
now reporter, borgs or other can drag a camera for checking out areas AI dont have visual on
Technical details

range is a little longer then the basic camera (7.5)
this is becuse of the microphone(range 10). and bigger lense. so its the same as a maxed out player can see.
Media

Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl:

add: AI can now view the Entertainment channels.

Summary by CodeRabbit

New Features

Introduced AI-driven vision capabilities to the wireless surveillance camera, enhancing its functionality.
Added a specified operational range of 9.5 units for improved monitoring capabilities.

i am a bit unsure of this.
this might make the AI confiscate the cameras from the reporter.
but i think this is better then making the cameras handheld. or making the Borgs act as AI eyes.

Did you test this in game? 9.5 is insanely large, and you cant even do .5s with the vision comp, it just gets rounded.

true the radius probably shouldn't be so large

i am a bit unsure of this. this might make the AI confiscate the cameras from the reporter. but i think this is better then making the cameras handheld. or making the Borgs act as AI eyes.

ai is literally at the bottom of the totem pole of the station. they are below crew and borgs can just ignore them

Did you test this in game? 9.5 is insanely large, and you cant even do .5s with the vision comp, it just gets rounded.
deafult is 7.5 i followed current standard.

true the radius probably shouldn't be so large
can turn it down to 7.5 same as wallmounted cameras. or you want it even lower?

wall mounted cameras dont have a 360 degree field also.
I'd make it 7.5 and test it, put a screenshot of it in the media part of the PR


Using babel-eslint parser makes standard run very slow

What version of standard?
12.0.1
What operating system, Node.js, and npm version?
Mac OS High Sierra 10.13.6
Nodejs 8.12.0
npm 6.4.1
yarn 1.10.1
What did you expect to happen?
I would expect standard to run only slightly slower if using the babel-eslint parser
What actually happened?
Standard checks a file in less than a second using the standard parser, but takes around five seconds using the babel-eslint parser. Is this expected behavior? Is there a way to inspect standard to see what might be going wrong? Thanks for this great library!

I think that this is an upstream issue with babel-eslint and/or eslint. Would you mind testing if the added delay is the same with just plain eslint without any rules added, and also file an upstream bug report? Thanks ☺


[Bug]: Cannot read properties of undefined (reading 'map') in v1.1.1-beta.38

What happened?
Updated to v1.1.1-beta.38, and recieving following errors now:
[29/03/2022, 18:04:46] Registering platform 'homebridge-eufy-security.EufySecurity'
[29/03/2022, 18:04:46] [homebridge-eufy-security] Loaded homebridge-eufy-security v1.1.1-beta.38 child bridge successfully
[29/03/2022, 18:04:46] Loaded 2 cached accessories from cachedAccessories.0EBB5A38298C.
[29/03/2022, 18:04:47] [homebridge-eufy-security] Country set: NL
[29/03/2022, 18:04:47] [homebridge-eufy-security] Unbridge set: false
[29/03/2022, 18:04:47] [homebridge-eufy-security] Finished initializing Eufy Security Platform
[29/03/2022, 18:04:47] Homebridge v1.4.0 (HAP v0.10.0) (homebridge-eufy-security) is running on port 53515.
TypeError: Cannot read properties of undefined (reading 'map')
    at DoorbellCameraAccessory.getCameraConfig (/usr/local/lib/node_modules/homebridge-eufy-security/src/accessories/CameraAccessory.ts:158:44)
    at new CameraAccessory (/usr/local/lib/node_modules/homebridge-eufy-security/src/accessories/CameraAccessory.ts:40:30)
    at new DoorbellCameraAccessory (/usr/local/lib/node_modules/homebridge-eufy-security/src/accessories/DoorbellCameraAccessory.ts:29:5)
    at EufySecurityPlatform.register_accessory (/usr/local/lib/node_modules/homebridge-eufy-security/src/platform.ts:393:13)
    at EufySecurityPlatform.discoverDevices (/usr/local/lib/node_modules/homebridge-eufy-security/src/platform.ts:300:14)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at HomebridgeAPI.<anonymous> (/usr/local/lib/node_modules/homebridge-eufy-security/src/platform.ts:150:7)
[29/03/2022, 18:04:52] [homebridge-eufy-security] Child bridge process ended
[29/03/2022, 18:04:52] [homebridge-eufy-security] Process Ended. Code: 1, Signal: null
[29/03/2022, 18:04:59] [homebridge-eufy-security] Restarting Process...

Device Type
Doorbell (Supported)
Plugin Version
beta (Best Effort)
HomeBridge Version
v1.4.x (Supported)
NodeJS Version
v16 (Supported)
Operating System
Raspbian 10+ (Supported)
Relevant log output
[29/03/2022, 18:04:46] Registering platform 'homebridge-eufy-security.EufySecurity'
[29/03/2022, 18:04:46] [homebridge-eufy-security] Loaded homebridge-eufy-security v1.1.1-beta.38 child bridge successfully
[29/03/2022, 18:04:46] Loaded 2 cached accessories from cachedAccessories.0EBB5A38298C.
[29/03/2022, 18:04:47] [homebridge-eufy-security] Country set: NL
[29/03/2022, 18:04:47] [homebridge-eufy-security] Unbridge set: false
[29/03/2022, 18:04:47] [homebridge-eufy-security] Finished initializing Eufy Security Platform
[29/03/2022, 18:04:47] Homebridge v1.4.0 (HAP v0.10.0) (homebridge-eufy-security) is running on port 53515.
TypeError: Cannot read properties of undefined (reading 'map')
    at DoorbellCameraAccessory.getCameraConfig (/usr/local/lib/node_modules/homebridge-eufy-security/src/accessories/CameraAccessory.ts:158:44)
    at new CameraAccessory (/usr/local/lib/node_modules/homebridge-eufy-security/src/accessories/CameraAccessory.ts:40:30)
    at new DoorbellCameraAccessory (/usr/local/lib/node_modules/homebridge-eufy-security/src/accessories/DoorbellCameraAccessory.ts:29:5)
    at EufySecurityPlatform.register_accessory (/usr/local/lib/node_modules/homebridge-eufy-security/src/platform.ts:393:13)
    at EufySecurityPlatform.discoverDevices (/usr/local/lib/node_modules/homebridge-eufy-security/src/platform.ts:300:14)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at HomebridgeAPI.<anonymous> (/usr/local/lib/node_modules/homebridge-eufy-security/src/platform.ts:150:7)
[29/03/2022, 18:04:52] [homebridge-eufy-security] Child bridge process ended
[29/03/2022, 18:04:52] [homebridge-eufy-security] Process Ended. Code: 1, Signal: null
[29/03/2022, 18:04:59] [homebridge-eufy-security] Restarting Process...

Got the same.

Working on it

I get the same.

Solved by manually adding the config posted by lenoxys. Unfortunately, RTSP doesn't really work. As with Camera UI, the stream is no longer accessible after a few minutes.

this is solved. sorry for the inconvenience!


[Feature Request] GeneralTest

Checklist

[X] I searched exising issues and no one else requests similar feature.
[X] I think that 25%+ users are positive towards this feature.

Describe the feature
GeneralTest
Additional context
No response

E02_01_List_of_Pin_Assignment.xlsx

考虑第一部分宽度，最好可设定
可以通过这个根据excel生成h文件

生成文件换行符问题 可能需要修改


Add number of matching listings to escape hatch

I think based on Tom's wireframes, we're no longer thinking about doing this?


Refactor Flickr to use ProviderDataIngester

Suggested Improvement

Refactor the Flickr provider script to use the new ProviderDataIngester base class
Benefit

More details in #229
Implementation

[ ] 🙋 I would be interested in implementing this feature.

Very excited for this one, in particular. There is one issue we aught to consider while working on it:
https://github.com/WordPress/openverse-catalog/issues/26


New rule: sl -> ls

When typing quickly, I make this mistake all the time. This addition allows you to unfuck sl, replacing it with ls.

 Isn't it quicker to simply type ls?

Yeah, but that's not as satisfying! The cd.. - > cd .. rule doesn't save characters either.

True...

This mistake is truly fuckable!!!

Thanks!


bind click and focus events when editable

When the editable option is set to true, the picker will not open when clicked. When the picker is editable, keyboard events are not bound to the input element. It looks like binding for click and focus events were moved inside the conditional and are only put in place when the picker is not editable.

There is a test failure in CI. When I ran test locally, the same test failed on master before I made any modifications. It looks like it might be a timezone/DST issue.

@DanielRuf 👍rebased

Thanks for your contribution and spotting this.


JRE SIGSEGV during AOSP compilation

Describe the bug
builds fail when anything that depends on JRE starts to compile, usually with either:
# Problematic frame:
# v  ~BufferBlob::vtable chunks

or
# Problematic frame:
# v  ~BufferBlob::vtable chunks

here are PID logs from the output
hs_err_pid73.log
hs_err_pid40079.log
hs_err_pid12464.log
hs_err_pid1253.log
Builds are run on an Ubuntu adm64 machine, the latest version available, rosetta enabled, and the newest version of Orbstack.
To Reproduce

Clone LineageOS 21
attempt to compile apollon

Expected behavior
on baremetal ubuntu builds don't fail, SIGSEGV shouldn't happen
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.6.4
Commit: b61f53a532892698cb1966f692dc7219035f0cad (v1.6.4)
System info:
macOS: 14.5 (23F79)
CPU: arm64, 8 cores
CPU model: Apple M2
Model: Mac14,2
Memory: 24 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-07-11T02-45-59.285792Z.zip

Screenshots and additional context (optional)
build.log
full build logs

Fixed for the next version.

is there access to an early/canary build for me to help test? (please ignore this message if not)

Released in hotfix v1.6.4 (17192).


Cleanup Rust annotations for Generic ML-KEM

The proofs for generic ML-KEM need to be made consistent with the latest Vector Trait annotations
and should be optimized so they do not run for memory and can be verified on CI.

[x] Ind_cca.rs (no lax functions, encapsulate/decapsulate get out of memory when verifying the post-conditions. I created #588)
[x] Ind_cpa.rs (#578)
[x] sampling.rs (1 lax function - it uses Rust_primitives.f_while_loop which doesn't support loop invariant. #570)
[x] serialize.rs (#587)
[x] matrix.rs (Mainstream)
[x] ntt.rs/invert_ntt.rss (#576)

I started a WIP branch for this issue https://github.com/cryspen/libcrux/tree/dev-generic-cleanup
Currently, it marks certain function in sampling.rs and serialize.rs as lax as they don't have proper pre-conditions yet (e.g. the function in serialize need to adapt the new conditions for vector trait). The branch also aims to figure a solution for functions in
Ind_cca.rs and Ind_cpa.rs that get out of memory.

I fixed the bug in generic polynomail.rs that blocks verifying the extracted code of constant ZETAS_TIMES_MONTGOMERY_R and I removed Libcrux_ml_kem.Polynomial.fst from ADMIT_MODULES in Makefile

@karthikbhargavan @mamonet Looks like everything is ticked off here. Is there still something open or can this be closed?

I need to merge dev to these PRs and do some modifications to get them work.

lets tick this off once the prs are merged today and then continue afterwards

This issue is completed!


Could not load helper Nightmare from module './helper/Nightmare'

What are you trying to achieve?
Hi @DavertMik , I'm trying to run tests with Nightmare (both nightmare and nightmare-upload have been installed globally):
codeceptjs run . --steps

What do you get instead?
Got following:
Could not load helper Nightmare from module './helper/Nightmare':
Required modules are not installed.

RUN: [sudo] npm install -g nightmare nightmare-upload
{}

Error:
    at createHelpers (/usr/lib/node_modules/codeceptjs/lib/container.js:29:13)
    at Object.create (/usr/lib/node_modules/codeceptjs/lib/container.js:102:15)
    at Codecept.init (/usr/lib/node_modules/codeceptjs/lib/codecept.js:43:15)
    at Command.module.exports (/usr/lib/node_modules/codeceptjs/lib/command/run.js:23:14)
    at Command.listener (/usr/lib/node_modules/codeceptjs/node_modules/commander/index.js:301:8)
    at emitTwo (events.js:106:13)
    at Command.emit (events.js:191:7)
    at Command.parseArgs (/usr/lib/node_modules/codeceptjs/node_modules/commander/index.js:615:12)
    at Command.parse (/usr/lib/node_modules/codeceptjs/node_modules/commander/index.js:458:21)
    at Object.<anonymous> (/usr/lib/node_modules/codeceptjs/bin/codecept.js:90:9)

My test code is:
Feature('Hellooworld');

Scenario('Login page loaded', (I) => {
  I.amOnPage('/');
  I.seeElement({name: 'password'});
});

Details

CodeceptJS version:
NodeJS Version:
Operating System:
Protractor || WebDriverIO || Nightmare version (if related)
Configuration file:

{
  "tests": "./tests/e2e/*_test.js",
  "timeout": 20000,
  "output": "./tests/e2e/output",
  "helpers": {
    "Nightmare": {
      "url": "http://localhost:3000",
      "show": false
    }
  },
  "include": {},
  "bootstrap": false,
  "mocha": {},
  "name": "ui"
}

Any thoughts about it?
Thanks

I met the same question after global installation.
I solved the problem by changing to local installation and edited the codecept.json as followed ：
"node_modules": {
    "Nightmare": {
      "url": "",
      "show": false,
      "restart": false
    }},

It works for me.

I have the same issue.
Following the steps here: http://codenroll.it/acceptance-testing-with-codecept-js/
I have nightmare and nightmare-upload installed locally.
codecept.json:
{
  "tests": "./tests/*.js",
  "timeout": 10000,
  "output": "./output",
  "helpers": {
    "Nightmare": {
      "url": "http://localhost:3000"
    }
  },
  "include": {},
  "bootstrap": false,
  "mocha": {},
  "name": "myb-e2e"
}

package.json
{
  ...
  "scripts": {
    "test": "codeceptjs run --steps"
  },
  ...
  "devDependencies": {
    "codeceptjs": "^0.5.1",
    "nightmare": "^2.10.0",
    "nightmare-upload": "^0.1.1"
  }
}

Running npm test causes:
/project> npm test

><EMAIL_ADDRESS>test /project
> codeceptjs run --steps

Could not load helper Nightmare from module './helper/Nightmare':
Required modules are not installed.

RUN: npm install --save-dev nightmare nightmare-upload
{}

Error:
    at createHelpers (/project/node_modules/codeceptjs/lib/container.js:29:13)
    at Object.create (/project/node_modules/codeceptjs/lib/container.js:102:15)
    at Codecept.init (/project/node_modules/codeceptjs/lib/codecept.js:43:15)
    at Command.module.exports (/project/node_modules/codeceptjs/lib/command/run.js:23:14)
    at Command.listener (/project/node_modules/commander/index.js:301:8)
    at emitTwo (events.js:106:13)
    at Command.emit (events.js:191:7)
    at Command.parseArgs (/project/node_modules/commander/index.js:615:12)
    at Command.parse (/project/node_modules/commander/index.js:458:21)
    at Object.<anonymous> (/project/node_modules/codeceptjs/bin/codecept.js:90:9)

I was about to install things globally but as stated above, that's an issue too.

Now as I expand my sample test to:
Feature('Retrieve booking');

Scenario('test something', (I) => {
	I.amOnPage('http://localhost:3000');
});

It errors with:
08:45> npm test

><EMAIL_ADDRESS>test /project
> codeceptjs run --steps --verbose

CodeceptJS v0.5.1
Using test root "/project"

Retrieve booking --
   [1] Starting recording promises
   Emitted | suite.before
 test something
   [2] Starting recording promises
   Emitted | test.before
   Emitted | test.start ([object Object])
   [2] Queued | throw error TypeError: I.amOnPage is not a function
   [2] Queued | fire test.passed
   [2] Queued | finish test
   [2] Error | TypeError: I.amOnPage is not a function
   [2] Starting <teardown> session
   Emitted | test.failed ([object Object])
   [2] <teardown> Queued | () => done(err)
   [2] <teardown> Stopping recording promises
 ✖ FAILED in 3ms

   Emitted | test.after
   [3] Starting recording promises
   Emitted | suite.after

-- FAILURES:

  1) Retrieve booking: test something:
     I.amOnPage is not a function
  ypeError: I.amOnPage is not a function
      at Test.Scenario (tests/retrieve_booking.js:5:4)
      at Context.test.fn (node_modules/codeceptjs/lib/scenario.js:29:24)

  FAIL  | 0 passed, 1 failed   // 13ms
   Emitted | global.result ([object Object])
npm ERR! Test failed.  See above for more details.

I had the same issue (Could not find helper Nightmare). It turned out I ran node_modules/codeceptjs instead of node_modules/codeceptjs-nightmare

This ticket is being closed due to inactivity. If this is still an active issue, then please reopen it, and we can revisit this.


WIP: Shellfmt

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

I think it will be difficult to:

find someone with the stamina needed to review this all,
avoid conflicts popping up during the review period

How about splitting this up by directory @janisz ?

I scrolled through the changes and so far I like them. I didn't give ✅ as this was still in WIP.

@janisz if possible, could WIP PRs be flagged as drafts in the future? I'm aware that this doesn't work always, especially when we're working on CI. Still, let's strive to only notify reviewers once the PR is ready to merge 😄


Make notes per denomination targeted by client configurable

The client always tries to have a specific minimum number of e-cash notes per denomination before issuing notes of the next bigger denomination. This target determines how many out-of-band spends can happen consecutively without the client coming back online.
Currently this target is hard-coded to 2, but should be configurable:
https://github.com/fedimint/fedimint/blob/33bfce1842cf856572586222e415355ee0535128/modules/fedimint-mint-client/src/lib.rs#L847-L855
But for this to work with the server we also need to make max_notes_per_denomination configurable which is currently always set to 3:
https://github.com/fedimint/fedimint/blob/33bfce1842cf856572586222e415355ee0535128/modules/fedimint-mint-common/src/lib.rs#L24
https://github.com/fedimint/fedimint/blob/33bfce1842cf856572586222e415355ee0535128/modules/fedimint-mint-server/src/lib.rs#L189
Afaik max_notes_per_denomination has to be target_per_denomination + denomination_base - 1. Currently we use base 2, so max_notes_per_denomination = target_per_denomination + 1, but to be future-proof and keep supporting arbitrary bases we should probably infer the base by looking at the second denomination (=base^1).

If no one else has started on this, I can give it a shot!

Go for it

Afaik max_notes_per_denomination has to be target_per_denomination + denomination_base - 1

If I understand correctly, the issue here is that we want to have enough notes of every denomination to be able to make x number of offline payments, each of potentially any size. For each offline payment, the worst case is that we need a number of notes for a single denomination that is one less than the base, so we're unable to use the next biggest denomination. For example, a base-2 payment of 1111 (or 15 in base-10) sats that's one away from being rounded up to 10000 (or 16 in base-10), or a base-10 payment of 999 sats that's one away from being rounded up to 1000. For base-2, our worst-case is 2-1=1 note per denomination per payment. For base-10, our worst-case is 10-1=9 notes per denomination per payment.
This sets the floor for the number of notes per denomination to target_per_denomination * (denomination_base - 1). However, we need to account for potential change notes to calculate the ceiling. Each denomination may have up to denomination_base - 1 notes as change due to having a total amount of ECash that can't be evenly broken into the denomination buckets.
So I think max_notes_per_denomination instead is (target_per_denomination * (denomination_base - 1)) + (denomination_base - 1) where the first section accounts for offline payments and the second section accounts for change. This can be simplified to the following:
((target_per_denomination + 1) * (denomination_base - 1))
Where the + 1 accounts for change and the - 1 gives us the maximum value before moving to a bigger denomination (the largest digit in base-10 is 9, which is 10-1).
Does that sound correct?


fixes problem with calling jMoment.utc without locale

fixes issue #66
Another solution is to change moment.utc as fallow:
jMoment.utc = function (input, format, lang, strict) {
    if (typeof lang === "boolean") {
        strict = lang;
        lang = undefined;
    }
    return makeMoment(input, format, lang, strict, true);
};

but I think the provided solution is more general.

thanks,
its available in version 3.3.3


DAC and ADC of ESP32S2 can't be used together in continuous mode. (IDFGH-10977)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I tried to generate a signal in DAC continuous mode and to sample the signal  using ADC continuous mode, when I discovered that these two can not be used in the same time. Firstly I started DAC to generate signal, the ADC, but it complains that is not available a dma channel, showing the error: E (3278) spi: alloc_dma_chan(212): no available dma channel
assert failed: dma_chan_free spi_common.c:331 (spi_dma_chan_enabled & BIT(dma_chan)).
If I not start the DAC, I can read in ADC continuous mode.

Hi @Qumbetlian , thanks for your reporting! And yes, the current ADC and DAC continuous drivers don't support to be used together, otherwise there will be a lot of limitations and workarounds to share the data and status between two separate drivers.
In your case you want to monitor the DAC output by ADC, you can refer to the example dac_continuous/signal_generator, which uses adc_oneshot to monitor the output.

Thank you L_KAYA for your response! I know about adc_oneshot and I tried it already, but, it seems that at the frequency that I need, adc_oneshot is to slow. I need to monitor the received signal by over 40K samples / second. After few experiments, I found out that if I configure the adc unit with:
adc_oneshot_unit_init_cfg_t adc_cfg = {
.unit_id = adc_unit,
.clk_src = ADC_RTC_CLK_SRC_RC_FAST,
.ulp_mode = ADC_ULP_MODE_RISCV,
};
it work little faster.
Do you know some settings that do adc one shot to work even faster?


Tolerance parameter is too high in example

The L-shaped example uses a tolerance of 1.0 yielding the point (0.5, 1.5). Reducing this parameter to 0.1 matches the expected value (0.5625, 0.5625).
Using geo = "0.14.0" and polylabel = "2.3.0"

Where are you seeing the tolerance of 1.0? It's 0.10 in both README.md and the notebook.

https://docs.rs/polylabel/2.3.0/polylabel/fn.polylabel.html

🤦♂ doctests were disabled. Fixed in b772e08, published to crates.io and docs.rs as 2.3.1. Thanks!


Feature/datepicker

The preview seems to have stopped working...

The preview seems to have stopped working...

I will investigate the problem


 Also, I know these images will be refreshed in the near future with Dashboard 2.0.

Had the wrong git info (CLA) - will resubmit.


test fix build error (Do not merge this PR)

Hi @superyyrrzz, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Hi @superyyrrzz, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


cs: check if (required) part of link is Signed

Check if a list of jmespath queries match the signatures present in the link. Used to enforce the signing of some parts of the link.

This change is

Nevermind...


Use ${command:python.interpreterPath} to get selected interpreter path in launch.json and tasks.json

For #11789
${config:python.interpreterPath} worked fine for launch.json but not for tasks.json. Having a command works for both. So I've renamed all instances of ${config:python.interpreterPath} to ${command:python.interpreterPath} to be consistent. Also did the migrations.

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[x] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #11840 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master   #11840   +/-   ##
=======================================
  Coverage   60.45%   60.46%
=======================================
  Files         631      632    +1
  Lines       34156    34179   +23
  Branches     4798     4800    +2
=======================================
+ Hits        20648    20665   +17
- Misses      12508    12512    +4
- Partials     1000     1002    +2

Impacted Files
Coverage Δ

...on/diagnostics/checks/invalidLaunchJsonDebugger.ts
94.11% <100.00%> (+0.07%)
:arrow_up:

.../diagnostics/checks/invalidPythonPathInDebugger.ts
86.36% <100.00%> (ø)

src/client/common/constants.ts
100.00% <100.00%> (ø)

...onfiguration/launch.json/interpreterPathCommand.ts
100.00% <100.00%> (ø)

...debugger/extension/configuration/resolvers/base.ts
91.02% <100.00%> (ø)

src/client/debugger/extension/serviceRegistry.ts
100.00% <100.00%> (ø)

src/datascience-ui/react-common/arePathsSame.ts
75.00% <0.00%> (-12.50%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


All finished surveys are evaluated as BEST regardless

It might be with how answered and question state is populated.
from #200 its mentioned that getMatchLevel, answered questioned and total questions is (0,0)

The number of questions and questions answered are both initialized to zero.  Because the rating is determined by the percentage of questions answered / questions that exist, it follows that (temporarily) the rating is perfect while the questions are still being fetched.  Write a special case for the indicator where the rating is :( while the number of questions is still 0.

so another case? what would you suggest the heart level and the copy be then for the survey complete case.

If the survey is complete (questions exist, and they have all been answered), that should be BEST


Invalid First XML Character 0xEF

An invalid XML character 0xEF exception is being thrown from Blob Batch, the XML chunk that contains this character doesn't get parsed as XML at the point the exception is thrown. Find the root location what the XML is being parsed in Azure Core.

Smoke tests have also encountered this in Windows environments -- https://dev.azure.com/azure-sdk/internal/_build/results?buildId=271402&view=logs&j=9d5db22d-3d98-5076-6ada-6891876c8311&t=5f0ec4c1-b78a-58e4-2905-0008c2d0ba42&l=236

@anuchandy and @srnagar, based on this being found outside of the scope I found it in I am going to revive the associated PR that was closed and take a deeper dive into find the root issue.

Fixed by #8687


Adds support for continuable reading in IonReaderContinuableTopLevelBinary.

Description of changes:
Builds on #510 by adding continuable (i.e. incremental) read support, allowing IonReaderContinuableTopLevelBinary to provide equivalent functionality to the existing incremental reader implementation.
When continuable reading is enabled (via IonReaderBuilder.withIncrementalReadingEnabled(true)), the reader will attempt to fully buffer each top-level value.
When continuable reading is disabled, the reader will not attempt to fully buffer values at any depth unless the user requests to parse a scalar. Additionally, receiving NEEDS_DATA from the underlying cursor causes the reader to infer end-of-stream, resulting in an IonException if a value is incomplete. This provides behavior equivalent to the existing non-incremental reader implementation, IonReaderBinaryUserX.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

When continuable reading is enabled (via IonReaderBuilder.withIncrementalReadingEnabled(true)), the reader will attempt to fully buffer each top-level value.

Do we plan to offer a mode where the reader will only buffer what is needed to produce a next event, and drop any previously buffered data as it goes? That could result in more efficient memory usage and potentially unblock folks with really really large TLVs.

Do we plan to offer a mode where the reader will only buffer what is needed to produce a next event, and drop any previously buffered data as it goes? That could result in more efficient memory usage and potentially unblock folks with really really large TLVs.

This is what non-incremental mode does. In that mode, only scalar values that the user requests to materialize need to be buffered in their entirety. It is impossible to get continuable behavior without having to buffer top-level values, but this functionality is only available through IonReaderContinuableApplicationBinary, as the behavior isn't compatible with IonReader interface semantics. That's why we need a different interface for general continuable parsing that doesn't require buffering TLVs, that's what I've proposed IonCursor (or something more user-friendly built on top of it) could be.


[1.9.4] No mouse in the Personal Chest inventory.

No mouse cursor when opening the Personal Chest:
https://youtu.be/mxoTECbrT4U

Don't see this occurring anymore, likely fixed due to the network handling changes.


Extension not found when filtering for the supported Visual Studio version

I have two Visual Studio extensions on Visual Studio marketplace, one supports VS 2019 and the other supports VS 2022.
I can find them by searching for the "diagram" keyword, but only if I set the Version filter dropdown to "Any". If I set the Version filter to the concrete VS version that they support (VS 2019 or VS 2022) then they cannot be found.
To Reproduce
Steps to reproduce the behavior:

Go to https://marketplace.visualstudio.com/vs
In the search box type "diagram" and click the search button
The first result (when sorted by relevance) is one of my extensions: "Codartis Diagram Tool for C# (VS2019)"
Now choose "Visual Studio 2019" in the version filter dropdown.
My extension is not in the result list any more. (Although it supports Visual Studio 2019, as confirmed by its page: https://marketplace.visualstudio.com/items?itemName=FerencVizkeleti.QuickDiagramToolforC )

Expected behavior
When I search for the "diagram" keyword and I set the Version filter to "Visual Studio 2019" then I expect my extension called "Codartis Diagram Tool for C# (VS2019)" to be in the result list.
Screenshots

Additional context

The same problem happens with my other extension that supports VS 2022. Its name is: "Codartis Diagram Tool for C# (VS2022)". When I search for the "diagram" keyword and "Any" version then it's in the result list. When I set the Version filter to "Visual Studio 2022" then my extension is not in the result list any more. Although it supports VS 2022 as confirmed by its page: https://marketplace.visualstudio.com/items?itemName=FerencVizkeleti.diagramtool-vs2022
The problem does not occur when I search for my extensions from Visual Studio's "Manage Extensions" dialog. If I search there for the "diagram" keyword then my extension is in the result list, as expected, both in VS 2019 and in VS 2022.

Thanks for bringing this to our notice. We are looking into this.

@kj0171 @prashantvc Any news on this? It's quite a problem for me because people can't find my extensions.

I've just given it a try and neither can I see the "Codartis Diagram Tool for C# (VS2019)" I thought it could be a Cache issue. This needs to be addressed @realvizu

@SaiKanth007 Hi! Any news on this issue? Thanks!

Hi @xavierdecoster ! I'd like to ask if there's any progress on this ticket? thanks!

What does your vsix manifest look like?

<?xml version="1.0" encoding="utf-8"?>
<PackageManifest Version="2.0.0" xmlns="http://schemas.microsoft.com/developer/vsx-schema/2011" xmlns:d="http://schemas.microsoft.com/developer/vsx-schema-design/2011">
    <Metadata>
        <Identity Id="09b0e9a9-7a0a-43c1-a55b-652f69d0232d" Version="1.3.0" Language="en-US" Publisher="Codartis" />
        <DisplayName>Codartis Diagram Tool for C# (VS2022)</DisplayName>
        <Description xml:space="preserve">Code visualization tool for C# to quickly explore, navigate and document source code structure and relationships.</Description>
        <MoreInfo>https://codartis.com</MoreInfo>
        <License>LICENSE.txt</License>
        <GettingStartedGuide>https://github.com/Codartis/DiagramTool/blob/main/help/Help.md#getting-started</GettingStartedGuide>
        <Icon>Resources\CodartisDiagramTool_128.ico</Icon>
        <PreviewImage>Resources\CodartisDiagramTool_256.png</PreviewImage>
        <Tags>code;visualize;codartis;quick;diagram;uml;c#;csharp;tool;inheritance;interface;codemap</Tags>
    </Metadata>
    <Installation>
        <InstallationTarget Id="Microsoft.VisualStudio.Community" Version="[17.2, 18.0)">
            <ProductArchitecture>amd64</ProductArchitecture>
        </InstallationTarget>
        <InstallationTarget Id="Microsoft.VisualStudio.Pro" Version="[17.2, 18.0)">
            <ProductArchitecture>amd64</ProductArchitecture>
        </InstallationTarget>
        <InstallationTarget Id="Microsoft.VisualStudio.Enterprise" Version="[17.2, 18.0)">
            <ProductArchitecture>amd64</ProductArchitecture>
        </InstallationTarget>
    </Installation>
    <Dependencies>
        <Dependency Id="Microsoft.Framework.NDP" DisplayName="Microsoft .NET Framework" d:Source="Manual" Version="[4.7.2,)" />
    </Dependencies>
    <Prerequisites>
        <Prerequisite Id="Microsoft.VisualStudio.Component.CoreEditor" Version="[17.0,)" DisplayName="Visual Studio core editor" />
        <Prerequisite Id="Microsoft.VisualStudio.Component.Roslyn.LanguageServices" Version="[17.0,)" DisplayName="C# and Visual Basic" />
    </Prerequisites>
    <Assets>
        <Asset Type="Microsoft.VisualStudio.VsPackage" d:Source="Project" d:ProjectName="%CurrentProject%" Path="|%CurrentProject%;PkgdefProjectOutputGroup|" />
    </Assets>
</PackageManifest>

I've tried to compare my vsixmanifest with some other ones that do appear correctly in the VS version filtered list, but I could not figure out what difference can cause it.

Maybe the extra space in the version range?

My other package (which also does not show up when filtered for VS version) does not have the extra space, but still does not work.
<?xml version="1.0" encoding="utf-8"?>
<PackageManifest Version="2.0.0" xmlns="http://schemas.microsoft.com/developer/vsx-schema/2011" xmlns:d="http://schemas.microsoft.com/developer/vsx-schema-design/2011">
    <Metadata>
        <Identity Id="7481ead5-87e4-4ac0-86d1-317e7adab60c" Version="1.3.0" Language="en-US" Publisher="Codartis" />
        <DisplayName>Codartis Diagram Tool for C# (VS2019)</DisplayName>
        <Description xml:space="preserve">Code visualization tool for C# to quickly explore, navigate and document source code structure and relationships.</Description>
        <MoreInfo>https://codartis.com</MoreInfo>
        <License>LICENSE.txt</License>
        <GettingStartedGuide>https://github.com/Codartis/DiagramTool/blob/main/help/Help.md#getting-started</GettingStartedGuide>
        <Icon>Resources\CodartisDiagramTool_128.ico</Icon>
        <PreviewImage>Resources\CodartisDiagramTool_256.png</PreviewImage>
        <Tags>code;visualize;codartis;quick;diagram;uml;c#;csharp;tool;inheritance;interface;codemap</Tags>
    </Metadata>
    <Installation>
        <InstallationTarget Id="Microsoft.VisualStudio.Community" Version="[16.11,17.0)" />
        <InstallationTarget Id="Microsoft.VisualStudio.Pro" Version="[16.11,17.0)"  />
        <InstallationTarget Id="Microsoft.VisualStudio.Enterprise" Version="[16.11,17.0)" />
    </Installation>
    <Dependencies>
        <Dependency Id="Microsoft.Framework.NDP" DisplayName="Microsoft .NET Framework" d:Source="Manual" Version="[4.7.2,)" />
    </Dependencies>
    <Assets>
        <Asset Type="Microsoft.VisualStudio.VsPackage" d:Source="Project" d:ProjectName="%CurrentProject%" Path="|%CurrentProject%;PkgdefProjectOutputGroup|" />
    </Assets>
    <Prerequisites>
        <Prerequisite Id="Microsoft.VisualStudio.Component.CoreEditor" Version="[16.0,)" DisplayName="Visual Studio core editor" />
        <Prerequisite Id="Microsoft.VisualStudio.Component.Roslyn.LanguageServices" Version="[16.0,)" DisplayName="C# and Visual Basic" />
    </Prerequisites>
</PackageManifest>

Also, I've found this package, which works as expected (shows up in the VS version filtered list), and also has the space in the version range.
Package: https://marketplace.visualstudio.com/items?itemName=AndreHass.YouML1
Vsixmanifest: https://github.com/ahass/YouML/blob/master/YouML/source.extension.vsixmanifest

Another potential cause: in my vsixmanifests the "from" part of the version range is not "16.0" or "17.0", but "16.11" and "17.2". If the VS version filter matches for exact "16.0" or "17.0", that would be an explanation for the problem.
However, I can't change the version ranges in may files to "16.0" and "17.0", because I depend on certain Roslyn versions that were introduced with the specified VS versions.

Closing this issue as I am no longer able to reproduce. Let us know if you continue to run into similar issues.

@xavierdecoster Did you test with a package that constrains support to 17.0 or newer (and not 17.0) ??
I only support 17.5 but have to frequently tell users to upgrade from older versions because the extension crashes - not a good UX


Break module into submodules

I want to keep the original idea as simple as possible, and keep it as light as possible.
So, I'm thinking about breaking the rip extra encoding into an extra module that will handle third party encoders and more advanced features (like Swagger/OpenAPI and HATEOAS).
As well as example could live in another modules, with all their dependencies.
But when you use rip, I wish you just need to depend on its module that mostly uses the standard library.
Also, that will trigger way less dependabot notifications.

Asked the experts on the #modules channel on the gopher slack

After testing, it was in fact optimized already. Just my go.sum containing all the deps sum in it made me think it was downloaded as well.
So no need for this.
Closing.


Watch region list

@marySalvi I'll pass it off to you now for the table pagination and filtering fixes.
@banesullivan I revised the method for adding/removing footprints in b7353ed. Let me know what you think of this new method. It seems that when cesium attempts to render the footprints for certain US regions (among the dataset you gave me), it gets a cesium error "invalid array length". Any ideas about this?


Convert the server and signer images to use alpine

Fixes #635.
Thanks to @ncopa for his solution at https://github.com/golang/go/issues/14851.

LGTM pending CI (since make integration tests this setup)

I'm having networking issues with compose again (while testing this PR) but I don't think they're related.
LGTM.


Cross-build for Scala Native

Also add a simple travis build config and compile magnolia for Scala 2.13.0-M3.
Closes #36

The travis build status is available here https://travis-ci.org/jodersky/magnolia

I just instructed Sonatype to publish it, so all being well, v0.7.1 should be available soon!

Sweet, thanks! I'm really bad at remembering publishing steps in sbt too, so I set up an automated script on most of my projects that publishes to sonatype when a release is pushed. https://github.com/drivergroup/spray-json-derivation/blob/475f6f6/.ci/build. Let me know if you'd like any help setting up something similar.

I would be interested in exploring that! @fommil has reported some problems with +publishSigned in the last couple of days (which I'm planning to look at some time, on several projects). sonatypeRelease has never worked for me, but I think the reason is that I never gave it my credentials anywhere...
Will you be coming to NE Scala in Boston next week?


Network configuration

I think it worth explaining the WDAV urls to whitelist as well
https://docs.microsoft.com/en-us/windows/security/threat-protection/windows-defender-antivirus/configure-network-connections-windows-defender-antivirus

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

ID: 1adce021-95d3-a946-5e9d-d818a88d3b0b
Version Independent ID: ff73e733-ce22-e37c-7e9b-6f477260dbca
Content: Microsoft Defender ATP production deployment - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/production-deployment.md
Product: w10
Technology: windows
GitHub Login: @mjcaparas
Microsoft Alias: macapara

Closing via merged PR #6793.


docs: add a note to add openai credits

📑 Description
Resolves the issue openapi API quota issue as discussed in the slack channel
Error: exhausted API quota for AI provider openai: error, status code: 429, message: You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.
✅ Checks

[X] My pull request adheres to the code style of this project
[X] My code requires changes to the documentation
[X] I have updated the documentation as required
[X] All the tests have passed

ℹ Additional Information

@AlexsJones
The preview URL is not working.
Here is the screenshot from my laptop

Thanks for the contribution @tchia04
Can you please restructure it to be a bit simpler to read?
E.g.
If you see this error message: "...."
Then do this...

Also it looks to be h1 size text, might be worth checking it's not too large.
Thanks!


RHIDP-3971 authorization introduction

IMPORTANT: Do Not Merge - To be merged by Docs Team Only
Version(s): 1.3.1

Issue: https://issues.redhat.com/browse/RHIDP-3971

Link to docs preview:

Reviews:

[ ] SME: @ mention assignee
[ ] QE: @ mention assignee
[ ] Docs review: @ mention assignee
[ ] Additional review: @mention assignee (by writer)

Additional information:

Looks good to me

LTGM, Fabrice. :)

/cherry-pick release-1.3


APPS-1131 Use go1.22 PathValue to parse path segment parameters

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (156a41a) 26.90% compared to head (f049260) 27.03%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #135      +/-   ##
==========================================
+ Coverage   26.90%   27.03%   +0.12%
==========================================
  Files          35       41       +6
  Lines        1613     1650      +37
==========================================
+ Hits          434      446      +12
- Misses       1121     1146      +25
  Partials       58       58

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


How to combine data updating in real time and loading more data on the pan?

Hello!
You provided two separate examples:

Updating data

Load more data on pan

Is it possible to combine them together? Maybe you have some examples or code snippet? I would like to update my data points in real time and have an opportunity to load more data on the pan.
Code :
updating data
`;
componentWillReceiveProps(nextProps) {
    const {data: newData} = nextProps;
    const {data: prevData} = this.state;
    const indexCalculator = discontinuousTimeScaleProviderBuilder().indexCalculator();
    const {index} = indexCalculator(newData);
    const xScaleProvider = discontinuousTimeScaleProviderBuilder()
        .withIndex(index);
    const {data: linearData, xScale, xAccessor, displayXAccessor} = xScaleProvider(newData.slice(-prevData.length));
    this.setState({
        linearData,
        data: linearData,
        xScale,
        xAccessor, displayXAccessor
    });
}`

load more data
`   }
onLoadMore = (start, end) => {
    if (Math.ceil(start) === end) return;
    const {data: prevData} = this.state;
    const {data: inputData} = this.props;

    if (inputData.length === prevData.length) return;
    const rowsToDownload = end - Math.ceil(start);
    const dataToCalculate = inputData
        .slice(-rowsToDownload - prevData.length, -prevData.length);
    const calculatedData = dataToCalculate;
    const indexCalculator = discontinuousTimeScaleProviderBuilder()
        .initialIndex(Math.ceil(start))
        .indexCalculator();
    const {index} = indexCalculator(
        calculatedData
            .slice(-rowsToDownload)
            .concat(prevData));
    const xScaleProvider = discontinuousTimeScaleProviderBuilder()
        .initialIndex(Math.ceil(start))
        .withIndex(index);
    const {data: linearData, xScale, xAccessor, displayXAccessor} = xScaleProvider(calculatedData.slice(-rowsToDownload).concat(prevData));
    this.setState({
        data: linearData,
        xScale,
        xAccessor,
        displayXAccessor,
    });
} `

Here is the issue that I get (Next update moves the chart to the right):

Hope you can help. Thanks in advance!

This seems more like a stackoverflow style question than an issue report.

Thanks so much for your response!
What exactly do you propose to do? I have to ask my question on
StackOverflow or change my question style?
2018-06-14 11:36 GMT+03:00 Karl Jensen<EMAIL_ADDRESS>
This seems more like a stackoverflow style question than an issue report.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rrag/react-stockcharts/issues/586#issuecomment-397217021,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASdTznSZBbzLhqgqEfCMvA_liiBtMhF4ks5t8iClgaJpZM4UluBt
.

@alexandrkondrachuk Why closed? Where  is the solution?

Hi, @hassantauqeer ! The issue has been closed because I found a solution by my self. If you are interested in my solution here is working code snippet https://codesandbox.io/s/k5q55018no .


Html_Table_Tags

Description 📜
Html_Table_Tags
1)table
2)tr
3)td
4)th
5)caption
6)colgroup
7)col
8)thead
9)tbody
10)tfoot
explanation with example
Fixes  #4179

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[x] Frontend Dev HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@ghulamyazdani
I have created a PR please check it.

Please assign me
Batch 2


Camera Controls not working on Mobile (Wasm)

It looks like the input RefCell is mutably borrowed somewhere else, here's the trace :
    at std::panicking::rust_panic_with_hook::h9d91fa0fae16500f (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[3427]:0x30db4a)
    at std::panicking::begin_panic_handler::{{closure}}::h2d82b5b2db976f3f (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[5345]:0x362b65)
    at std::sys_common::backtrace::__rust_end_short_backtrace::h24539600b7e2452c (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[14215]:0x4175f6)
    at rust_begin_unwind (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[12798]:0x4054ef)
    at core::panicking::panic_fmt::h91d2023e5afe1929 (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[14217]:0x41765a)
    at core::option::expect_none_failed::h203584ddbc785ca2 (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[6247]:0x3802e2)
    at core::result::Result<T,E>::expect::h2cecc8562387c163 (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[4601]:0x346426)
    at core::cell::RefCell<T>::borrow_mut::hac48a91c65b3f340 (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[5226]:0x35e8d0)
    at WASM::window::canvas::Window::add_touchmove_event_listener::{{closure}}::h664de35dc4bfa048 (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[240]:0xeaf13)
    at <dyn core::ops::function::FnMut<(A,)>+Output = R as wasm_bindgen::closure::WasmClosure>::describe::invoke::h7ed178cfd3254e2f (http://<IP_ADDRESS>:3000/2c704d02c033148a506a.module.wasm:wasm-function[3649]:0x3197c3)
    at __wbg_adapter_24 (http://<IP_ADDRESS>:3000/static/js/0.chunk.js:898:174)
    at HTMLDocument.real (http://<IP_ADDRESS>:3000/static/js/0.chunk.js:860:14)

Every touch event / add_touch* function in canvas::Window throws this.
The Wasm VM should be single threaded, so I really do not understand how two RefCells can simultaneously be borrowed.

Thanks for reporting, I will look into it a soon as I have time 💪

There were several errors, but it should be working now 👍


fix golint path

Golint has been moved.
More info here -> https://github.com/golang/lint#installation and golang/lint#417

Can one of the admins verify this patch?

@vladbarosan in fact I don't think we even use golint in CI anymore so if true feel free to remove it when creating v21.2.0.

This was fixed in the v21.2.0 release.


add dart implementation

Added dart code, open to comments

Could you please add a dart.yaml workflow file here that will run the dart tests? Something like:

On it.

Added workflow, your call on sub-module

I am sorry I messed it up


Can't parse GitHub releases feed

Hi,
It seems that a missing name property in GitHub Atom releases feed causes it to fail:

The example feed (I tried some others): https://github.com/sonatype/nexus-public/releases.atom
I use the latest plugin version with Grafana 8.5.2. Other feeds work.

@vfauth Thank you for opening the issue. I will take a look shortly.

@vfauth I have fixed the missing author name and added parsing for Author and Media:thumbnail fields. I will release an updated version in an hour or so to test.

Here is the code for Dynamic Text plugin to get you started:

{{title}}

{{date updated 'MMM, DD YYYY'}} by {{author}}

{{{content}}}

Please let me know if there is anything else.

@vfauth Could you please test and let me know if it works for you: https://github.com/VolkovLabs/volkovlabs-rss-datasource/releases/tag/v1.6.0

It works perfectly, thank you very much!!
However the change to extract the author name automatically, while an unquestionable improvement, will probably break some dashboards which used author.name, so maybe you could list it in the changelog/release announcement.

@vfauth Thank you. I  will update the changelog to make it clear.
But I don't think that using author.name is possible. Because Data Source returns [Object] string as a field and does not support JSON objects. Or was it possible?

It return an Object, but I was able to extract the author name with author.name, so I think the XML is parsed anyway.
At least, I was able to do it on the GitLab Blog RSS: https://about.gitlab.com/atom.xml

@vfauth Thank you for the feedback. Changelog updated and we released a new version: https://github.com/VolkovLabs/volkovlabs-rss-datasource/releases/tag/v1.6.2
Please let me know if there is anything else.


[java]Bloom filter read/write implementation

Reporter: Junjie Chen / @chenjunjiedada
Assignee: Junjie Chen / @chenjunjiedada
PRs and other links:

GitHub Pull Request #521
GitHub Pull Request #587

Note: This issue was originally created as PARQUET-1328. Please see the migration documentation for further details.

Gabor Szadovszky / @gszadovszky:
Removing 1.11.1 from Fix Version/s. (1.11.1 is a maintenance release and won't include any new features.)

Junjie Chen / @chenjunjiedada:
@zivanfi, Jim had reviewed some on this and we need some more reviews from committers, could you please help on this?


Clarify PersistentSessions setting and its usage.

Hi @hugomendix
Thanks for the pull request.
You've added some information relating to sticky sessions. The only reference I can find for sticky sessions is in the on-premises configuration documents. Can you let me know if I've missed it somewhere.
Are sticky sessions relevant for anyone else (Mendix Cloud/SAP/etc.). If so, should we have some documentation about what they are, what the default is, and how to change it?
If this is a non-Mendix concept, perhaps we could just link to an external document - providing users have some way of configuring whether sessions are sticky or not.

Hi @MarkvanMents,
I was referring to sticky sessions as a general mechanism that one could implement in a load-balancer (see for instance https://en.wikipedia.org/wiki/Load_balancing_(computing)#Persistence). AFAIK we generally do not have sticky sessions in our cloud, so it would indeed only apply to on-premise setups. Since Mendix 7 we advise to just not disable persistent sessions in a clustered setup, which is why I've also trimmed down the warning on 'negative impact' which sounds much too scary ;-) Note that some old (<=Mx6) customers may have a custom setup in our cloud that does provide sticky sessions.

@hugomendix
So would it be better to say:

In a clustered environment you must have persistent sessions. The only exception is for on-premises installations which have implemented sticky sessions.

We don't document for versions of Mendix below 6 as these are no longer supported, so your other use case shouldn't be an issue.


Update 27 rocm513

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-CPU please

jenkins: retest Ubuntu-GPU-multi please

jenkins: retest Ubuntu-CPU please

jenkins: retest Ubuntu-CPU please

jenkins: retest Ubuntu-sanity please

jenkins: retest Ubuntu-GPU-multi please

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-GPU-single please

jenkins: retest Ubuntu-GPU-single please

CI was not converging in a timely manner so I run this test on a dev node and it passed.  So I merged the PR.


Pass error through callback

Injects the seqid to allow the connection to reference the callback and then associates as many request specific errors with the callback as possible.
Also wraps the callback so that it only fires once.
TODOs logging, more unit tests
I need to find an a better unit testing library for Node that is Apache 2.0 compatible (e.g. upstreamable).
cc @Raynos @kriskowal @sectioneight @breerly @willyham

Closing. Meant to do on the Uber repo this isn't ready for upstream yet.


Added a script to make it a little easier for registering plugins

This should take away some of the tedium of moving around the files, and it ensures the versions are at least mapped correctly.

Thanks for the PR! LGTM


Improve printing of jsx props with Pexp_apply + spread with Pexp_apply/Pexp_fun

inline Pexp_apply where possible with brace hugging
improve formatting of spread children with Pexp_apply & Pexp_fun

Thank you for this change! –2 stops is huge.


Implement a Couchbase lifecycle workflow on APIv2

Couchbase cluster was released on APIv2 - https://instaclustr.redoc.ly/Current/tag/Couchbase-Provisioning-V2, we need to implement the full lifecycle workflow for it.
Things that need to be done:

CRUD
Validation

For now, this task is blocked. Got a response regarding it:
It’s still unconfirmed. The hope is that we can offer it like a normal application, but the ultimate decision on that is dependent on talks with couchbase that haven’t concluded (to my knowldege). So my advice would be don’t worry too much about couchbase for now, but keep it on the list of things to do in the future, since it may at some stage turn into a normal application offering.


How to get original referenced AST node using JSS?

I'm intending to build a web-side java code viewer to store and index all code which belong to different project.
I think may be such a project should work like this:

Gather all the source code. They may comes from source.jar which hold in central maven repository.
Build the AST nodes separately in different .java file.
Using JSS to calculate the references since we need to jump to the symbol declaration as common IDE does.

I have found that java-parser can generate AST node for each .java file. In the meantime , java-parser can also holds the token stream info like the start index and end index. Such an AST node information is quit useful since we need this to format the origin file into a highlight mode.
After that, we need to walk through the Syntax tree and for every node which is  reference-able, we need JSS to tell us which AST node it declared in.
For example:
in Person.java
class Person {
   Name name;
 }
in Name.java
class Name { String firstName; String secondName; }
In the view of Person.java page (remember we are build a code viewer, so we need to manipulate and demonstrate the Person.java in html), when user need to jump to Name, we need to know exactly which  AST node such a symbol is reference to. In the above case, we need to get the Name Class AST node  which is in Name.java file.
Is JSS capable to do this?  I know how to use JSS to solve the type of node, but I don't know how to fetch or point user to the origin declaration AST node in another file.
Am i missing something in the guide or the example ?

Yes, JSS can do that. When you resolve Name you will get a TypeDeclaration. Let's suppose it is a ClassDeclaration. Depending on where it comes from it could a JarClassDeclaration, a ReflectionClassDeclaration os a JavaParserClassDeclaration. In the latter case you can call getWrappedNode and get the JavaParser AST node, which knows the position in the source code of the declaration itself. Hope it helps.

It works, Thanks a lot !

You are welcome. Let us know about your website


Add a "Terminology" (or similar) section to the template

A terminology section would define jargon specific to an RFC.
A terminology helps RFC editing and reading:

Helps the author re-use the same terms to refer to identical concepts
Helps the author avoid mixing different ideas, on surface identical but not interchangeable
Helps the author identify the concepts they are using and structure their explanation with clearly defined concepts in mind
Helps the author limit how many different concepts they handle in the same RFC
The reader can refer to that section if they forgot the meaning of a particularly specialized term
The reader can pay closer attention to occurrence of terms specified in the terminology section
It makes RFCs more accessible, as it introduces context and jargon that non-experts may not know

This is different from the "User-facing explanation" concept sub-section, in that the terminology section is for the terms used in the RFC rather than just the ones introduced into the user-facing API.
Alternative names for this section are

Terminology
Jargon
syllabus
nomenclature

Strongly in favor of this. I think that my favorite terms are: "Definitions", "Glossary" and "Terminology".

Agreed, I started writing a RFC and this is the first thing I added. I vote for Terminology or Glossary.


Fix encoder attention mask input order for ORT

Fixes https://github.com/huggingface/optimum/issues/1158

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Reproduce results on KITTI test

Hi,
Could you provide the pretrained model to reproduce the tracking results on KITTI test? I found the pretrained weights which is for tracking on KITTI validation only.

Hi,
thanks for your interest in our work! The model trained on the full training set of KITTI is available under this link: https://tri-ml-public.s3.amazonaws.com/github/permatrack/kitti_full.pth