spectacle: fix cuda support

Description of changes
When cuda support is turned on spectacle will fail to compile with:
CMake Error at /nix/store/2r1vcpnld3nwpd1fs823xlawgw5j5c92-cmake-3.29.2/share/cmake-3.29/Modules/FindCUDA.cmake:883 (message):
  Specify CUDA_TOOLKIT_ROOT_DIR
Call Stack (most recent call first):
  /nix/store/x09lf32h3g5cic6783mh2n0hxbq2h83a-opencv-4.9.0/lib/cmake/opencv4/OpenCVConfig.cmake:86 (find_package)
  /nix/store/x09lf32h3g5cic6783mh2n0hxbq2h83a-opencv-4.9.0/lib/cmake/opencv4/OpenCVConfig.cmake:108 (find_host_package)
  CMakeLists.txt:82 (find_package)

-- Configuring incomplete, errors occurred!

This adds the optional cuda build inputs needed to avoid that
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

Could it be that this PR is obsolete with #314691?

Could it be that this PR is obsolete with #314691?

Trying to build this PR I get the following error:
opencv> CMake Error at modules/core/CMakeLists.txt:158 (message):
opencv>   CUDA: OpenCV requires enabled 'cudev' module from 'opencv_contrib'
opencv>   repository: https://github.com/opencv/opencv_contrib
opencv>
opencv>
opencv> -- Configuring incomplete, errors occurred!

I get this when I was using my nixos config with nixpkgs.config.cudaSupport = true; and also by running nix-build -A kdePackages.spectacle with only cudaSupport & allowUnfree enabled in my ~/.config/nixpkgs/config.nix

Then I probably need to disable cuda explicitly in that PR. I don't think we need it for spectacle in any way.

Please retest my other PR.

OpenCV should propagate CUDA in this case.

Nevertheless we should fix this.

Thanks for adding the fix to your PR, I'll close this one now


Incorrect sample code on PointerRoutedEventArgs

https://docs.microsoft.com/en-us/windows/winui/api/microsoft.ui.xaml.input.pointerroutedeventargs?view=winui-3.0
In this article, it explains how one can use PointerRoutedEventArgs to get various pieces of information about the pointer and do different tasks. The article is mostly good but, has one glaring problem

In most lines of code, "Windows.UI.Xaml.Input..." is used. However, for this piece of documentation, it is wrong. Both e.pointer and e.GetCurrentPoint(Target) return types from "Microsoft.UI.Xaml.Input..."

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

ID: 604ea511-cf47-0688-4999-aa507ca00fa4
Version Independent ID: 4d517ee9-a95a-d21c-ce5b-ddf733a79151
Content: PointerRoutedEventArgs Class (Microsoft.UI.Xaml.Input) - WinUI
Content Source: winui-api-build/xml/Microsoft.UI.Xaml.Input/PointerRoutedEventArgs.xml
Product: reunion
Technology: winui-reference
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

This topic has moved to the WinApp SDK ref doc set: https://docs.microsoft.com/en-us/windows/windows-app-sdk/api/winrt/microsoft.ui.xaml.input.pointerroutedeventargs?view=windows-app-sdk-1.1
Snippet has been updated.


Create new DNSSEC administration UI(s)

Create new DNSSEC administration UI(s) such that users have the ability to:

View keys and expiry data

Page structure should be organized such that it is usable; there could be hundreds of delivery services and several different CDNs per instance of Traffic Ops

Generate new keys for a particular delivery service or CDN TLD

Ideally this should be a popup that is overlaid on top of the admin page, including a date/time picker that defaults to the same effective date as used by the API but is selectable by the user
We should ensure that the date they select is within acceptable thresholds for effective date/expiration, while still allowing the user to roll keys in an emergency

Expire existing keys for a particular delivery service or CDN TLD

Defensive logic should be in place to prevent the user from expiring a key without a new key to take its place if DNSSEC is currently enabled

Enable or disable DNSSEC for an entire CDN TLD

If enabling, key generation process should run, taking into account whether keys exist and whether expiry data is valid

Same commits at #157


Android - Payments - Paypal account disappear if it was added offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the App and login
Navigate to Settings > Payments
Disable the internet connection in the device
Click/Tap on "Add Payment Method" and Click/Tap on "PayPal.me"
5.Enter any PayPal.me name, then click “Add PayPal account”
Go back online

Expected Result:
Paypal account method should not disappear
Actual Result:
Paypal account disappears
Workaround:
Unknown
Platform:

Where is this issue occurring?

Android

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/191775437-57dc8bee-f35e-4ac9-9503-59130b5a7e31.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Related offline issue https://github.com/Expensify/App/issues/10762
After reinstall the app , unable to reproduced it, payment button is disable.
https://user-images.githubusercontent.com/93399543/191778158-e1f055b7-251d-49ed-8ff9-ddf27f198c47.mp4


All wallets show in the drop down, regardless of what network is selected.

Here is a screenshot of the wallets my address is a signer on.

All these will appear regardless if I have mainnet selected in the network drop down, or any other network.  Two of these are deployed to mainnet, while the rest are on goerli.
This causes issues if you don't have the same network selected as the wallet is deployed to.  Is it possible to only show the wallets in the drop down that are deployed to the currently selected network?

Should be possible, I'll take a look at it!


Develop a suitable file format that users can input to pyMGM

References #11 and #13
[ ] - As a developer/user, I need a convenient file format that can handle raw data such as matrix entries (or possibly elliptical partial differential equations, or both, or any other component relevant to MGM problem input)
[ ] -  In addition, I need to develop a method in python which can successfully parse such a file, and load the data directly into the gallery class
[ ] - I also need a way to test that the data was successfully transferred over to entities in the gallery class

The typical way sparse matrices are stored is in CSR or CSC - see the scipy.sparse.csc_matrix and scipy.sparse.csr_matrix functions.
You can also load in MATLAB .mat files using scipy.io.loadmat.  This will be a convenient way to load in the current data files from the mgm/+util/matrices directory, which is what the gallery function loads in.  To get the matrices, nodes, right hand side, and exact solution from this data file you would do something like
from scipy.io import loadmat
m_dict = loadmat(`PoissonNeumann2DDiskFenicsP3.mat`)
Lh = m_dict['Lh']
x = m_dict['x']
fh = np.transpose(m_dict['fh'])
uexact = np.transpose(m_dict['uexact'])


lua rock dbus_proxy is not available in the neovim lua interpreter after installation

I think it has something to do with dbus_proxy entrypoint structure(it's a directory) but I don't think the rock is broken, maybe it relies on some deprecated import behaviour. I can't say for sure because I am not a lua developer.
moving init.lua out of dbus_proxy directory at ~/.local/share/nvim/lazy/luarocks/.rocks/share/lua/5.1 and renaming it to dbus_proxy.lua made it available

It seems like this is because of the way luarocks.nvim sets the package.path. It only looks for a <plugin>.lua file instead of a <plugin>/init.lua file in addition. Should be quite a simple fix - I'll be making a PR soon :)

this issue should be fixed in #5

Thank you for solving this issue as this is basically the only thing that stands between people who use lazy.nvim and them using neovim plugins that require luarocks.


New Mimoa26 interpreter

This is a pull request for the new Mimosa26 interpreter.
The new interpreter can roughly handle a trigger rate of 60kHz in realtime (single core performance).
Please add to the list if something is missing.

[x] Event number starts at 1

[x] Added converter for testbeam analysis

[x] Removed event selection

[x] Optimized correlation delay (tested with pymosa DAQ v1.0.0.dev0)

[x] Unit tests

[x] Documentation

Attachments
Fixed correlation offset with event number starting at 0:

Do you have an explanation why we see a correlation for more than 180 us, although the frame time is shorter?

The correlation should be visible for 2x 115.2us. The correlation length (115.2us + 1 column) stays the same when the correlation offset is changed.

Ok. I guess if you take just a few rows to check correlation the histogram is less Gaussian but a double S-Curve with plateau? If you increase the correlation window does the peak in the correlation histogram increase? That would be a cross check that the window is too small/not working as intended, no?

yes, I agree, that should be tested.

The correlation should be visible for 2x 115.2us. The correlation length (115.2us + 1 row) stays the same when the correlation offset is changed.

But this is only since the trigger timestamps are distributed within the Mimosa26 frame. If all timestamps would come in the same time, with respect to a Mimosa26 frame, we would have a perfect box curve with a width of 1 Mimosa26 frame, right?

@YannickDieter I think the development branch can be merged into master when the unittests are completely fixed. Especially check test_interpreter:create_raw_data() which sometimes gives different results than the interpreter.

Okay, I will check this as soon as possible. Before merging to master (or afterwards) we should also rename the project to pymosa_mimosa26_interpreter, no?


APEXCORE-734 StramLocalCluster may not terminate properly

@PramodSSImmaneni @tweise Please review

@PramodSSImmaneni @tweise Please review

@PramodSSImmaneni @tweise Please review

Will look into it

Can you rebase

Now it is not necessary to rebase unless there is a conflict. Please use "Rebase and merge" option on github.

Please take a look at the comment as well and address it when you get a chance.
Also, IMO, not having the conflict should not be the only criteria as there could be other logical problems after rebase and merge. I prefer the general guideline to committers not be an automatic rebase and merge but rather to make an assessment based on the situation like the changes made in the meantime and if needed do a rebase locally, check and make the appropriate decision. If needed go back to the contributor.
That being said, this particular PR likely can be rebased and merged from github as other recent changes or PRs in consideration that could get merged before this PR don't affect this.

The debug level is intentional. The StramLocalCluster main thread may be interrupted (please see description in JIRA). Warning is logged in case an event loop thread or container threads do not terminate, the extra warning is not necessary.
I guess we need to discuss the policy for the rebase on dev@apex. For smaller changes a committer does not have to require rebase and should be in a position to merge after review is done, IMO. If changes are limited to a file or a method that were not recently changed, the rebase should not be required as well. Squashing commits by a contributor is a must.

If CI fails as result of automatic rebase then the committer performing the push needs to be responsible. It is possible that a merge succeeds but the build doesn't pass.
@vrozov please take the discussion to the mailing list, as well as the one from the other PR.

Jenkins CI runs against the merge, not against the original commit.


2 commands dont work

in newest build 459 the commands  /i ,/v ,dont work

Yes, i am aware of this, Rocket Beta <IP_ADDRESS> (#461) for Unturned <IP_ADDRESS>  should have fixed it


New package: Stremio.StremioService version 0.1.13

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Resolve #199195

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

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

Note: <path> is the directory's name containing the manifest you're submitting.
Manual validation
Console logs
--> Disabling safety warning when running installers
Tip: you can type 'Update-EnvironmentVariables' to update your environment variables, such as after installing a new software.

--> Configuring Winget
Enabled admin setting 'LocalManifestFiles'.
Enabled admin setting 'LocalArchiveMalwareScanOverride'.

--> Installing the Manifest 0.1.13

Found Stremio Service [Stremio.StremioService] Version 0.1.13
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
This package requires the following dependencies:
  - Packages
      Microsoft.VCRedist.2015+.x64
(1/1) Found Microsoft Visual C++ 2015-2022 Redistributable (x64) [Microsoft.VCRedist.2015+.x64] Version 14.42.34433.0
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.visualstudio.microsoft.com/download/pr/c7dac50a-e3e8-40f6-bbb2-9cc4e3dfcabe/1821577409C35B2B9505AC833E246376CC68A8262972100444010B57226F0940/VC_redist.x64.exe
  ██████████████████████████████  24.4 MB / 24.4 MB
Successfully verified installer hash
Starting package install...
Successfully installed

Downloading https://dl.strem.io/stremio-service/v0.1.13/StremioServiceSetup.exe
  ██████████████████████████████  34.9 MB / 34.9 MB
Successfully verified installer hash
Starting package install...
Successfully installed

--> Refreshing environment variables

--> Comparing ARP Entries

DisplayName                                                        DisplayVersion Publisher             ProductCode                                Scope
-----------                                                        -------------- ---------             -----------                                -----
Microsoft Visual C++ 2015-2022 Redistributable (x64) - 14.42.34433 14.42.34433.0  Microsoft Corporation {804e7d66-ccc2-4c12-84ba-476da31d103d}     Mac...
Stremio Service version 0.1.13                                     0.1.13         Smart Code OOD        {DD3870DA-AF3C-4C73-B010-72944AB610C6}_is1 User

PS C:\Users\WDAGUtilityAccount\Desktop\winget-pkgs> Get-ARPTable | Format-Table

DisplayName                                                        DisplayVersion Publisher             ProductCode                                Scope
-----------                                                        -------------- ---------             -----------                                -----
Microsoft Edge                                                     100.0.1185.36  Microsoft Corporation Microsoft Edge                             Machine
Microsoft Edge Update                                              <IP_ADDRESS>                           Microsoft Edge Update                      Machine
Microsoft Visual C++ 2015-2022 Redistributable (x64) - 14.42.34433 14.42.34433.0  Microsoft Corporation {804e7d66-ccc2-4c12-84ba-476da31d103d}     Machine
Stremio Service version 0.1.13                                     0.1.13         Smart Code OOD        {DD3870DA-AF3C-4C73-B010-72944AB610C6}_is1 User

Screenshots

Microsoft Reviewers: Open in CodeFlow

Verify:

This is a tray-based app.


docs(installation): tegel design installation page

Describe pull-request
First draft at a tegelifyed installation page.

Uses highlight.js for syntax highlighting.
tegel.syntax.highlight.css is the theme file for the syntax highlighting

Solving issue
Fixes: https://tegel.atlassian.net/browse/DTS-1801
How to test

Go to storybook
Check in Installation
Check the style of the page

Nice! But could the text be a little bigger?
Feels to small for my eyes 👀


Bump up version

Sorry to bother you again, but I would need a new version so that I can put it in Sys::Chown metadata...

:+1:

Did you upload it also to CPAN? zef can only find 0.1.2 and 0.1.3 for the time being.


error rewards

What is your version Quests / build number 3.8.0
https://pastebin.com/CgkQ0sa7
create a rewards on order
ex:
/quests editor
1 test
2 beginning
3 finished
4 the npc (1)
11 new step
1 blocks
1 blocks break
1 define the names of the blocks (stone x 64)
all finished
Quests: test
12 (edit the rewards)
5 (set order of rewards)
1 (set order of rewards)
2 (set order display replacements)
as soon as I finished there is an error in the console

Thanks for the report. Please try the latest dev build and let me know if the issue persists (if so, include the new console error):
https://ci.codemc.io/job/PikaMug/job/Quests/
Sent with GitHawk

Thank you very much there is no problem :)


board reboot from API

Maarten,
I experience issue that at some random time gateway kinda works, sends stats to loraserver, but no lora data received. Power cycle helps.
I added:
// reboot
server.on("/REBOOT",  {
sendWebPage("","");             // Send the webPage string
server.sendHeader("Location", String("/"), true);
server.send ( 302, "text/plain", "");
ESP.restart();
});
to _wwwServer.
Then I can monitor packets received on node-red for example and send reboot to gateway which fixes issue. Not perfect, but works.
At least I did not see any other way to reboot gateway board. Or is there any?
Would you consider adding this to code, please?
Thanks!
Reggie

I take it that you want to included a REBOOT option in the Gateway Settings as well? We need a button to start the Reboot sequence.

Hello Maarten,
For my need enough that we have this via web call. For general use button could help as well for sure.
Thank you!
Reggie

Done in 6.1.5


Not rendering selectBoxIt correctly when in div with display:none

I have a  element in a div that is initially hidden (via display:none) when the page loads. When a user clicks a link that div is changed to display its content (including the  element).
The width of the selectBoxIt element doesn't appear to be figured correctly.
Here is a jFiddle showing the issue.
http://jsfiddle.net/dJzhV/1/
Thanks for the help and the great plugin.

I had a similar issue inside a hidden div using flexbox, and it was caused by max-height: 0px applied to:
.selectboxit-container .selectboxit-options
So then i just forced
.selectboxit-container .selectboxit-options { max-height: none !important; } via CSS and it's working fine.


Juju provider petstore action  & kubectl.sh fix

This PR adds a convenience command for the juju provider for launching the petstore example:
$ juju action do kubernetes-master/0 petstore

Credit for this work goes to @chuckbutler.
Also included: a fix to make cluster/kubectl.sh work with the juju provider.

LGTM

cool :) thanks !


Exclude some attributes from API listing

The CompilerGeneratedAttribute and EditorBrowsableAttribute are noisy in the API listing and not really necessary.

I removed EditorBrowsableAttribute since we discussed that originally, but after talking about how to best overload a method with default parameters and add EditorBrowsable, I wonder if we should keep it. @tg-msft thoughts?

I agree about [EditorBrowsable]. In many cases we manually add it and considering https://github.com/Azure/azure-sdk-tools/issues/366 I think people want to see it.


More tests.

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

Project coverage is 86.55%. Comparing base (4dfc588) to head (db5a469).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   84.78%   86.55%   +1.76%
==========================================
  Files          19       19
  Lines        1834     1822      -12
  Branches      249      247       -2
==========================================
+ Hits         1555     1577      +22
+ Misses        238      201      -37
- Partials       41       44       +3

Files
Coverage Δ

behavioral_control/my_router/views.py
72.98% <100.00%> (+5.46%)
:arrow_up:

... and 2 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


Adds support for arbitrary query data for NLU services

Some NLU service providers may support including arbitrary query data along with the utterance text. E.g., LUIS allows you to set query options and send external entities or dynamic lists. This change introduces the notion of an INLUQuery, which is an empty interface that services can optionally use to extract extra data from the input utterances file (rather than being limited to the text-only utterance string).
Fixes #42

Already merged via another PR.


[Event Request] Codeunit 90 Purch.-Post - OnBeforeNewInvoice

Hi, in the Codeunit 90 Purch.-Post, trigger CalcInvoice, would it be possible to get an event OnBeforeNewInVoice, like this :

?
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Fix exception type given invalid date or datetime

Codecov Report

Merging #67 (efe3fa1) into master (b0691ff) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #67   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          503       507    +4
  Branches        99        99
=========================================
+ Hits           503       507    +4

Impacted Files
Coverage Δ

tomli/_re.py
100.00% <ø> (ø)

tomli/_parser.py
100.00% <100.00%> (ø)

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 b0691ff...efe3fa1. Read the comment docs.


update docs with missing step for kind cluster creation

/kind bug
What steps did you take and what happened:
After installing Kind, GHC participants got connection refused error trying to run the quick-start.sh script and that was because they hadn't created a Kind cluster with kind create cluster. The "Getting Started" docs should be updated to include this step.

I want to learn more about kserve, I think updating this doc can be an easy step for me to learn and contribute more in the future.
So do I need to get assigned or shall I directly go for a PR?

@JaynouOliver You can just raise a PR.


fix: Add upgrade banner in the FC47 Primary Course card view

Relates to https://github.com/openedx/openedx-app-android/issues/340

@HamzaIsrar12 ready for another pass


[JBCS-634] decrease loops per vhosts for balancer changes

https://issues.jboss.org/browse/JBCS-634

Can one of the admins verify this patch? If it is not dangerous to our systems, juts tell me to "run_tests".  ( without the underscore and quotes :smiley: )

run_tests

run tests

Thanks @Karm

run tests

No sure why this is suddenly a problem
C:\workspace\workspace\mod_proxy_cluster-2.x-windows\DISTRO\
apache-lounge\label\w2k12r2\mod_proxy_cluster\native\mod_proxy_cluster\
mod_proxy_cluster.c(28): fatal error C1083:
Cannot open include file: 'apr_strings.h': No such file or directory

Fails on master too though.  Checking...

run tests

@aogburn @rhusar The only change I did was to switch from building with ApacheLounge httpd to building with our own httpd.
I suppose the packaging of ApacheLounge has changed, perhaps due to APR update and it no longer provides what I need for mod_proxy_cluster in the way I expect.
This PR did not break the smoke test. I'll look into Apache Lounge httpd next week...

@jfclere @Karm Not overly happy with this merge, really. First, it clearly duplicates code with add_balancers_workers function. An effort should be made to make the code manageable. Moreover, we require fixes to have an associated MODCLUSTER tracker, which this one does not, thus falls out of release notes and so on.


[Types] Fix error TS2309 in index.d.ts

Using simple-encryptor with typescript, I can't build the code due to error TS2309: An export assignment cannot be used in a module with other exported elements due to export = encryptorCreator in index.d.ts
Tested on typescript 2.9.2

I had done this also in my own fork (this was before the library had any typings): https://github.com/yonilerner/node-simple-encryptor/commit/d518c095f7845dce710a92d35c9ecf32b6ea824d

This looks good. I think I'm going do a major version bump as otherwise it'd break existing code using import/require style:
import encryptorCreator = require('simple-encryptor');
``

Merged and a v2.0.0 has been published to npm. Thanks!


feat!: Bump minimum Python version to 3.9

Also reverts old fixes specific to Python 3.6.
Python 3.9 is the last living release, supported until EOY25.

This PR is necessary to enable the PR testing workflow.


Add MapSet -> HashSet decoder/encoder support out of the box

Adds a new encoder/decoder for the MapSet.t()/HashSet<K> type.
Not sure how the rustler team feels about contributions like this that attempt to unwind and rebuild opaque elixir types. There are some obvious drawbacks here:

The internal representation of a MapSet could change at any time
Version number used is static in the code
Depending on how well tested user code is the version used by rustler could result in == comparisons failing silently for 2 "identical" mapsets. MapSet.equal?/2 however should always return the right result regardless...

I would be interested in feedback on how best to solve these problems and also your opinions on the value of encoding/decoding opaque types in the elixir stdlib like this...
I mostly did this work just to get hands-on in how encoders/decoders actually work so am happy for this to be closed if it isn't a direction the rustler team wish to head in.

Yeah, unfortunately this is relying on private information from MapSet which may change. Especially the assembling of MapSet from inside Rust.
The only way I can think for now is to have the conversion steps in Elixir. For example, a MapSet could become {:hash_set, MapSet.to_list(set)} in Elixir (which then becomes HashSet) and on the way back {:hash_set, keys} become MapSet.new(keys).

Thanks for the PR!
Unfortunately, as José mentioned, we want to avoid relying on private implementation details like this.
The best way of passing opaque types like this back and fourth would probably indeed be to do some of the conversion work on the Elixir side.
If you really need this for internal use, you could make a newtype for HashSet and implement the encoder/decoder for that. You are still relying in MapSet implementation details, so in this case you would be responsible for validating/updating every time you update Elixir. It is probably advised to avoid this if you can.


Load more ayah and render when user scroll down

Pagination is broken
If possible, maybe we can try upward pagination while working on this

Fixed pagination on staging. Let's add another issue for upward pagination.


EKF: Improve height reset behaviour

Addresses #96 and #91
Has been flown in SITL for normal operation and bench tested for a GPS height timeout
Height reset recovery from each of the three modes (Baro, GPS and range finder) needs to by tested. I have tried using SITL, but it does not support adding offsets to sensor outputs.

Failed bench testing with a manual offset added to the Z accelerometer offset to induce a loss of height fusion.

@priseborough Looked at this and generally looks good. I haven't seen any use of "_hgt_sensor_offset", is that missing? I would expect this value to be set when we are switching between the height sensors to maintain consistency.

_hgt_sensor_offset is included  to allow for the height to be maintained after a reset if required. Currently there are no resets that use it, however I have left it in because the requirements for reset behaviour when using different sensors could change when we do more testing.

I have completed testing using the replay and confirmed sensible switching and reset behaviour when using GPS and baro.


Attach binaries to release

As a user I want to attach freshly built binaries to a new release on GitHub so that other users can directly download them from there.
This should be optional since this is a use case only relevant for binaries and not for libraries.

Oops, just filed #133 which is for semantic-rs itself.
I don't think it should be semantic-rs' tasks to attach those files though, trust is a pretty good start for it

Agreed! Closing this one in favor of #133.


Fix some warnings

.../nox/nox/sessions.py:35: DeprecationWarning: invalid escape sequence \w
  path = re.sub("[^\w\s-]", "-", path).strip().lower()

.../nox/nox/sessions.py:36: DeprecationWarning: invalid escape sequence \s
  path = re.sub("[-\s]+", "-", path)

.../nox/.nox/tests-3-7/lib/python3.7/site-packages/markupsafe/__init__.py:13: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
  from collections import Mapping

-- Docs: https://docs.pytest.org/en/latest/warnings.html

@theacodes I wouldn't have seen this if I didn't make some tests fail. Worth changing how pytest is invoked so that warnings turn to errors?
The third is out of our control, but has been filed / fixed already: https://github.com/pallets/markupsafe/issues/99

Worth changing how pytest is invoked so that warnings turn to errors?

totally.
What are we using markupsafe for?

It's part of jinja2

ah
On Wed, Sep 19, 2018 at 1:54 PM Danny Hermes<EMAIL_ADDRESS>wrote:

It's part of jinja2
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/theacodes/nox/issues/137#issuecomment-422955036, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPUcz0MMBKxetQPbbvGEnIaKbKFOk0tks5ucq8bgaJpZM4Ww8Re
.


adds support for CLI usage

This is a pretty small change but it's very useful to people that want to run Reddit scripts from the CLI.

This worked great for me, thanks!

Second this.  Exactly what I was looking for.  Merge to main please.


[Helm][minor] Make "disabled" flag for worker groups optional

Why are these changes needed?
As mentioned by @tbabej in #537, an error will be reported if the disabled key on the default worker group is undefined. In old Helm versions, when disabled key is undefined, it will be set to nil. Hence, when we run the following command, it will report the error "incompatible types for comparison". Obviously, nil and boolean cannot be compared for equality. See https://github.com/helm/helm/issues/6376 for more details.
$ helm lint
==> Linting .
[ERROR] templates/: template: ray-cluster/templates/raycluster-cluster.yaml:105:9: executing "ray-cluster/templates/raycluster-cluster.yaml" at <ne .Values.worker.disabled true>: error calling ne: incompatible types for comparison

Error: 1 chart(s) linted, 1 chart(s) failed

In this PR, I give the disabled key a default value to make it become an optional field.

Helm version v3.4.1 => can reproduce #537
Helm version v3.9.4 => cannot reproduce #537

Related issue number
https://github.com/ray-project/kuberay/issues/544
https://github.com/ray-project/kuberay/pull/499
Checks
$ git diff | cat
diff --git a/helm-chart/ray-cluster/templates/raycluster-cluster.yaml b/helm-chart/ray-cluster/templates/raycluster-cluster.yaml
index 67922d4..5432068 100644
--- a/helm-chart/ray-cluster/templates/raycluster-cluster.yaml
+++ b/helm-chart/ray-cluster/templates/raycluster-cluster.yaml
@@ -102,7 +102,7 @@ spec:
 {{ include "ray-cluster.labels" $ | indent 10 }}
   {{- end }}
   {{- end }}
-  {{- if ne .Values.worker.disabled true }}
+  {{- if ne (.Values.worker.disabled | default false) true }}
   - rayStartParams:
     {{- range $key, $val := .Values.worker.initArgs }}
       {{ $key }}: {{ $val | quote }}
diff --git a/helm-chart/ray-cluster/values.yaml b/helm-chart/ray-cluster/values.yaml
index be80d86..a099a50 100644
--- a/helm-chart/ray-cluster/values.yaml
+++ b/helm-chart/ray-cluster/values.yaml
@@ -54,7 +54,7 @@ head:
 worker:
   # If you want to disable the default workergroup
   # change the line below
-  disabled: false
+  # disabled: false
   groupName: workergroup
   replicas: 1
   type: worker

$ helm lint
==> Linting .

1 chart(s) linted, 0 chart(s) failed

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

cc @DmitriGekhtman @tbabej

Thanks for the fix!
To finish this up, could you return values.yaml to its state prior to https://github.com/ray-project/kuberay/pull/543?
(That will make it clearer that the disabled field is optional.)

@DmitriGekhtman Thank you for your review! I have updated it.

Merging before CI finishes, as Helm charts aren't yet tested in CI.


mdl-layout__header--waterfall-hide-top won't work in IE11

IE11 still hides the bottom row after adding mdl-layout__header--waterfall-hide-top
Here is the example:
https://codepen.io/anon/pen/BKgjew

It works perfectly in my computer in IE 11. Please refer the screens.
First I got a blank white screen and after 5 sec it fully loaded up.
http://i.imgur.com/HGdvrED.png
http://i.imgur.com/KsufGn6.png

It is not working. It's supposed to be liked this:

I added some custom CSS and it seems working for now.
.is-compact.waterfall-hide-top .mdl-layout__header-row {
    top: -100%;
    position: relative;
}
.is-compact.waterfall-hide-top { /* Chrome & Opera */
    max-height: 64px;
    height: 100%;
}

I used waterfall-hide-top instead of mdl-layout__header--waterfall-hide-top
https://codepen.io/anon/pen/xVogPp


Node.js v22 support

Could you please add v22 support for node.js as it is the latest approved version and it generates warning that this plugin does not support it. Thanks.

I will add it soon

Hi @tothbe
Node.js v22 is supported in version 2.1.3 and later.

Thanks man, you are just amazing.


add product detailes while ordering

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
@pooranjoyb
[Feature] Add Product Name while ordering product #256
Fixes #256 (issue)
Type of change
Please give a X on it which is applicable

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactor Code
[ ] A documentation update
[ ] Others(mentioned in the issue number)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test A Describe here
Test B Describe here (if Requred)
Screenshorts and Vedios:

give screenshorts and vedio of the changes you made
Checklist:
give a X on it which is applicable

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my 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 new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

@pooranjoyb
please chech this and merge it

Thank you @pooranjoyb


yatas: init at 1.3.3

Description of changes
Tool to audit AWS infrastructure for misconfiguration or potential security issues
https://github.com/padok-team/YATAS

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

yatas


Adding ADL Initiative projects

Thanks for submitting a pull request! Below are a few things you can do to help us more quickly review your changes.
Checklist
I have…

[X] run the build process locally (./scripts/build) and make sure it builds correctly.

Summary of Changes
This pull request adds the projects from ADL Initiative so that they are included in code.json

Hi @Elliottdotgov can you merge this pull request?


publish<EMAIL_ADDRESS>

Requested by: @AbhiPrasad
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Targets

[x] npm
[x] github
[ ] gcs
[ ] registry
[ ] aws-lambda-layer

Closing due to https://github.com/getsentry/publish/issues/643


Save files uploaded but not downloaded to Deck

Describe the bug
After setting up the save directory of a Prism Launcher instance, the save data gets uploaded. However when I do the same setup on the deck and try to sync, the folder remains empty. When I attempt to sync the game, the app reports "no changes", implying that it thinks the files are there but when you open the directory in the file explorer the directory is empty.
To Reproduce
Steps to reproduce the behavior:

Install Prism Launcher. Create a Minecraft instance.
In Prism Launcher, select your instance and click Edit -> Worlds -> Open Folder.
Add this path to OpenCloudSaves.
Do the same for your Steam Deck.
From your desktop PC, sync your files so it uploads the save data to cloud.
Check your cloud backup and see that the files are uploaded.
From your Steam Deck, sync your files so it downloads the save data from the cloud.
It will report that there are things to download. Accept and wait for it to finish.
Open the directory where the save data is supposed to go.
The directory will be empty.
Sync again and it will report that there are no changes.

Expected behavior
Save data appears in the specified directory.
Screenshots
N/A
Desktop (please complete the following information):

OS: Steam Deck OS
Version: Whatever the latest is

Additional context
In my case, directory for the save data in my Steam Deck is home/deck/.var/app/org.prismlauncher.PrismLauncher/data/PrismLauncher/instances/Yoru/.minecraft/saves

Thanks for the report. I'll target this for 0.17.

This should be fixed in 0.17.4


G5 - Language Detector

Group - 5
*Joe
*Preston
*Sharma

Good idea to count word frequency and then select the correct language. In main.py file if you correct
from languages import LANGUAGES
``` with \n
```python
from .languages import LANGUAGES
``` all checks will be ok.

Easy to read. Smart to remove the punctuation too, so less room for bugs.

I agree with Nick and Nektarios. It was good to count frequency of common words as a there can be overlap. For our group we were only checking if they showed up at all. Also it was good to take it a step further and remove punctuation. Forward thinking, fellas!


How to resolve the problem: ImportError: Cannot Import Name version ？

jiangjiang@jiangjiangdeMacBook-Pro src % python -m openne --help
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/Users/jiangjiang/Desktop/baseline_code/OpenNE/src/openne/main.py", line 7, in
from . import node2vec
File "openne/node2vec.py", line 3, in
from gensim.models import Word2Vec
File "/Users/jiangjiang/Library/Python/2.7/lib/python/site-packages/gensim/init.py", line 6, in
from gensim import parsing, matutils, interfaces, corpora, models, similarities, summarization, utils  # noqa:F401
File "/Users/jiangjiang/Library/Python/2.7/lib/python/site-packages/gensim/models/init.py", line 23, in
from . import wrappers  # noqa:F401
File "/Users/jiangjiang/Library/Python/2.7/lib/python/site-packages/gensim/models/wrappers/init.py", line 5, in
from .ldamallet import LdaMallet  # noqa:F401
File "/Users/jiangjiang/Library/Python/2.7/lib/python/site-packages/gensim/models/wrappers/ldamallet.py", line 40, in
from smart_open import smart_open
File "/Users/jiangjiang/Library/Python/2.7/lib/python/site-packages/smart_open/init.py", line 26, in
from smart_open import version
ImportError: cannot import name version

Hi, cddjiang97,
You are using Python 2.7 but Python 3.7 is required.


Fix: Fixed an issue where tags would sometimes display with the wrong color

Resolved / Related Issues
To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

Closes #16548

Steps used to test these changes
Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

Open a folder containing some tagged items
Change sort order
Then, check that tag icons in the following controls are using the appropiate color : Settings/Tags, Sidebar>Tags, Home>Tags Widget, Preview Pane, Right click context menu

Notes
I changed the bindings (I think we may remove that VisualState - our DependencyProperty doesn't seem to be bindable).
I don't know if this is the appropriate way to address the issue

I think you don't need newValue parameter, just use DP value


Entry title with tags

If the original XML has an item with a title with HTML tags in it, the corresponding entry's title field will be an object.
E.g. here's a snippet from the SciencesNews RSS stream:
        <item>
            <title>A new
                T. rex
                exhibit takes a deep dive into the iconic dinosaur </title>
            <link>https://www.sciencenews.org/article/new-t-rex-exhibit-takes-deep-dive-iconic-dinosaur</link>
            <description>“T. rex: The Ultimate Predator,” a new exhibit in New York City, draws on the latest science to provide a fresh look at Tyrannosaurus rex and its relatives.</description>
            <category><category domain="https://www.sciencenews.org/topic/animals">Life & Evolution/Animals</category>
 <category domain="https://www.sciencenews.org/topic/paleontology">Life & Evolution/Paleontology</category>
 <category domain="https://www.sciencenews.org/topic/science-society">Humans & Society/Science & Society</category>
</category>
            <comments>https://www.sciencenews.org/article/new-t-rex-exhibit-takes-deep-dive-iconic-dinosaur</comments>
            <enclosure length="34247" type="image/jpeg" url="http://www.sciencenews.org/sites/default/files/styles/620px_wide/public/2019/03/main/articles/031319_cg_trex_feat.jpg?itok=jSO4aNYg"/>
            <guid isPermaLink="true">https://www.sciencenews.org/article/new-t-rex-exhibit-takes-deep-dive-iconic-dinosaur</guid>
            <pubDate>Fri, 15 Mar 2019 07:00:06 -0400</pubDate>
            <source url="https://www.sciencenews.org/feeds/headlines.rss">Latest Headlines | Science News</source>
            <dc:creator>Carolyn Gramling</dc:creator>
        </item>

I got this title:
title: {
    _: "A new  exhibit takes a deep dive into the iconic dinosaur ",
    em: ["T. rex"]
}

I don't know how to restore the title from this, since the place for "T. rex" seems to be lost.
P.S. Here is the full RSS file in case you need it.

I believe putting HTML inside the title is invalid RSS. The "correct" solution here (assuming I'm not mistaken) would be to contact the RSS publisher and ask them to remove the HTML in the title.
That said, if you want to add a fix for this, I'd accept a PR. I'm not sure there's an easy fix though - I believe the xml2js object (the output of our XML parser) will always dismantle this tag. But there may be some function available like node.toString() that will give you the full title with the HTML.


not support  HikariDataSource connect pool

Could you elaborate? This worked for me.

Sorry, this issue double Datasource configuration error. I had to solute it


typos and comma fixes. paragraph line breaks changed.

changes

there were some strange line breaks. maybe a setting of your editor, or just a different style to do it? (I removed the linebreaks in larger paragraphs). please use --word-diff to see the wording changes I made within those paragraphs
please double check if I understood correctly how the forcing of redeploys of snapshots work. I changed the wording there into the opposite, so I hope I got it right :)

questions

how to 'install' this? (I tried using install.sh and the comments in 'Deployment' section but did not succeed yet)
do I have to set the MARATHON_URL in my environment? (I suspect that this is optional)
if the installation fails, there is no feedback (e.g. when setting a wrong version string)
what is the required python version?
what do you mean by 'positional arguments'? I have not heard this before.
is this supposed to be part of this repo? "src/resources/repository/com/meltwater/myservice-snapshot"
what is the syntax for specifying the versions based on? (if that is an existing way to do it, maybe one could just point to the spec)
If I was using lighter and a deployment as been done, how would I know that it has worked? Do I get a confirmation message, or a log files written, or similar?
could hipchat, newrelic, and datadog be grouped in a section called 'integrations'? If yes, then they could potentially be moved to the end of the README?

Thanks for the feedback! I've fixed the spelling and punctuation problems but left the line breaks so that non-line-wrapping editors won't have issues. Answering the questions

Good point. I've added the Installation section with how to make use of Lighter
I've removed this setting as it's not really useful
If the version can't be resolved from Maven then an error is showed and an exit code it returned
We build the Python project into static binaries for Linux and Mac OSX, so no Python version is actually required.  We use Python 2.7 at the moment, but can change without problems if we so desire (the static binary builds in the python runtime)
Positional argument is standard lingo for the commands, e.g. the "deploy" or "verify" commands. That printout is actually generated by the standard argparse module, so not something we've specified.
Yes, it's a file used by the unit tests.
The syntax for specifying version ranges is documented in the readme. It's based on the usual inclusive/exclusive range operators [] and () and inspired by how Apache Ivy does it. Unfortunately there's no real standard for it I could find, and we don't (want to) support the full syntax that Ivy has
If lighter performs a deployment there's an info log line on the console saying that it's deployed service-x
Good suggestion, I've moved them to an Integrations section at the bottom.


Enter your number without spaces or dashes

from the bbs log-in page:

Enter your number without spaces or dashes

can the backend just strip  , -, and . (but leave + for prefix)?

Patch welcome!l

i played with this a bit, and while it's easy to strip those characters from the input (request.payload.phone.replace(/[^\d+]/g, '')), the one thing is that there's a pretty strict validation on the input already. (index.js#L116)
is it worth making the validation looser (maybe just Joi.string()) so that more lenient input can be entered?

It probably makes more sense to just allow the characters you expect people to use in phone numbers, since that set should be fairly small, right?

internationally, most phone numbers are 10 digits, but that's not always true. validation for phone numbers is always tricky; validating using custom regexes misses corner cases and forces honest phone numbers (e.g.,<PHONE_NUMBER>) to be rejected.
something like this would work:
request.payload.phone.match(/\d/g).length >= 8

HTML5 to the rescue ? input type="tel" -> link

i'd be happy to take this on if @isaacs wants to assign me, otherwise i defer to @cvan for advice on how best to proceed ?

Probably one of these would help? https://www.npmjs.com/search?q=libphonenumber
Don't stand on ceremony waiting on me for assignments :)  Go nuts.

@isaacs understood, i was just trying to keep within the guidelines
i'll start working on this as a client-side validation solution.

related #53 (for the server module implementation)

i realise that i have only moved the validation onto the client by using a relatively new tag syntax which may not degrade well (although browser support looks ok), it does however avoid the big red form error which i consider a UI improvement.
we do however still need the big red error for cases that try to subvert the client-validation.. something simple like parseInt( phone.replace(\^\+\, '') ) if thats a 10+ digit number then it's ok
that is where i think this issue dovetails into #53 "us numbers with/without country code should resolve to same account"

support across mobile devices could be improved with shims (scroll down to "form validation")


Arnej/workaround strtof bug

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@baldersheim please review and merge
@havardpe please review also

@havardpe PTAL


Many false positive even for close face detection

Hi,
I have a good working setup of Reolink (for 1920p) with snapshots taken on the highest resolution over to DoubleTake and CompreFace.
The issue I have is that CompreFace gives me 98% accuracy even for unknown people standing close to the camera.  Sometime it even mark my ankle as my face with 92%.  This happens also with other people I'm training for, even that I only train for relatively closer images.
I also only train based on what the camera detect, didn't upload any other image.
Question is - Is this something to do with the Camera, or should I upload a more rich images of myself first?
Thanks,
Tal.

Ideally, training images should be taken on the same camera and same conditions as during recognition.
If it's not possible - it's better to use images with the best quality for training.
One detail, I recommend using only one image per subject to avoid false positives.
One more idea, have you tried custom builds? They should be more accurate:
https://github.com/exadel-inc/CompreFace/blob/master/docs/Custom-builds.md

Thanks, can you please elaborate what did you refer with "one image per subject"? Are you saying that more than one image within a training is less beneficial?

Imagine you are a security guy and you were given a photo of John - the person needs to recognize.
It's not an easy task to recognize a person using a photo, so there is a chance that you won't recognize John when you see him, or there is a chance that you will recognize another person as John
So now, imagine you were given the second photo of John.  The chance that you recognize John will increase, even if he doesn't look like in the first photo, he will probably look similar in the second photo. But the chance that somebody else would look like John in one of the photos also increased. So there is a bigger chance of recognizing another person as John.
What would you do if you met a person who looks similar to John from one photo, but different from another photo?
The logic will depend on your needs:

you say that this is John only if he looks similar to both photos
you say that this is John only if he looks similar to one of the photos
you say that this is John only if he looks similar to 50% of photos (if you have more than 2 photos)
in the case of an automated security system, you can automatically approve a person who looks like john on all photos, if not, send it to human check
If you have several people to recognize, you can take top 5 results and make a logic on how to determine the person.
Because of all those possibilities, we return the similarity to each photo, and you should decide how your system should behave.
Loading only one image per person and using a threshold on similarity is the simplest way to decrease incorrect recognitions. This is why I usually recommend it for most systems.

Thanks for the detailed explanation.
Thanks,
Tal.

From: Pospielov Serhii @.>
Sent: Tuesday, July 25, 2023 6:50:28 PM
To: exadel-inc/CompreFace @.>
Cc: Tal Yanai @.>; Author @.>
Subject: Re: [exadel-inc/CompreFace] Many false positive even for close face detection (Issue #1093)
Imagine you are a security guy and you were given a photo of John - the person needs to recognize.
It's not an easy task to recognize a person using a photo, so there is a chance that you won't recognize John when you see him, or there is a chance that you will recognize another person as John
So now, imagine you were given the second photo of John.  The chance that you recognize John will increase, even if he doesn't look like in the first photo, he will probably look similar in the second photo. But the chance that somebody else would look like John in one of the photos also increased. So there is a bigger chance of recognizing another person as John.
What would you do if you met a person who looks similar to John from one photo, but different from another photo?
The logic will depend on your needs:

you say that this is John only if he looks similar to both photos
you say that this is John only if he looks similar to one of the photos
you say that this is John only if he looks similar to 50% of photos (if you have more than 2 photos)
in the case of an automated security system, you can automatically approve a person who looks like john on all photos, if not, send it to human check
If you have several people to recognize, you can take top 5 results and make a logic on how to determine the person.
Because of all those possibilities, we return the similarity to each photo, and you should decide how your system should behave.
Loading only one image per person and using a threshold on similarity is the simplest way to decrease incorrect recognitions. This is why I usually recommend it for most systems.

—
Reply to this email directly, view it on GitHubhttps://github.com/exadel-inc/CompreFace/issues/1093#issuecomment-1650098326, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH43C7I6UJNTI6VVLBWDKDXR7TMJANCNFSM6AAAAAAZZXMXUA.
You are receiving this because you authored the thread.Message ID: @.***>


Timesheet - 06/05/2021 - Bucky Roberts

Task 1
Marketing meeting with Tristan and Bojan.
Proof of Work

https://docs.google.com/document/d/1WDc3izgpXjd10JyrSqdRuh2BT27MMWdCwTSxigMQDRU/edit?usp=sharing

Time Spent
1.5 hours

Task 2
Writing roadmap tickets and chatting with various team members about features.
Proof of Work

https://github.com/thenewboston-developers/Management/projects/11

Time Spent
1 hours

Total Time Spent
2.5 hours
Account Number
a37e2836805975f334108b55523634c995bd2a4db610062f404510617e83126f

Approved


Suggestion: Add an apt repository

An apt repository would make it easy for users of debian derivatives to install Lepton. In addition, it should be fairly trivial to set up, I should have thought - since Lepton for linux is distributed as a single binary already. Tools exist to automatically generate packages for multiple linux-based systems at once, too.

@sbrl
Thanks for suggestion ;) Is there any problem when using the AppImage format in Debian?
Since I don't have a Debian system for testing, I would prefer a format that can run everywhere for different Linux.  (AppImage is the best option I can find at this moment.)

@hackjutsu Nope, there isn't a particular problem with it! It's just that since I am on a debian-based system (Ubuntu 17.04, actually), I much prefer to have everything update through apt - that way I only have 1 thing to keep track of to update my laptop here. Other distributions will be the same - just with different package manager.
The AppImage system you're using currently looks like it it would lend itself quite well to automatic packaging.

I would like to keep this post open to see whether other people have similar requests.
Related issues:
https://github.com/hackjutsu/Lepton/issues/57
https://github.com/hackjutsu/Lepton/issues/127


Add race condition flag

Making a test branch to see if the jobs will fail if I put in the -race flag in the build and the test step.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 5a03ea22c383dd7554bddde8ce5cd09b4e5caa25


hack/*: Put kubelet-pod.uuid into a less volatile place.

Otherwise the kubelet-pod.uuid won't survice the reboot, which makes
ExecStartPre=-/usr/bin/rkt rm meaningless.

Without this, every time the node is rebooted, there will be garbage rkt pods, they won't be cleaned up until the next run of rkt gc service (which is run every 24hrs).


chore(deps): Update the Dev Container (2023 Q3)

Closes #1806
Notes

#1808
#1809

Description
Biannual update of the devcontainer. See #1806 for the documentation of the decision-making process.
How to test the new devcontainer
Build the devcontainer image inside your current devcontainer after having checked out the PR branch:
$ devcontainer build --workspace-folder tools/devcontainers/sage/
...
{"outcome":"success","imageName":["vsc-sage-fab0756cad848c05da6a50951535dd55d12c8faf5c2101949d7f7cabf9c2088c"]}

Start the devcontainer. If the image has not been built previously, it will be built now.
$ devcontainer up --workspace-folder tools/devcontainers/sage/

Step inside the running devcontainer:
$ docker exec -it sage_devcontainer bash
To run a command as administrator (user "root"), use "sudo <command>".
See "man sudo_root" for details.

vscode@f82a8bc3fa7f:/$

Clone Sage Monorepo inside the devcontainer:
cd ~
git clone https://github.com/Sage-Bionetworks/sage-monorepo.git

Create a checklist of tools and tasks to test before merging this PR, which will then publish the image of the new devcontainer to GHCR.

Failed to build the dev container image from the host
The issue is related to installing Node.js
## Run `sudo apt-get install -y nodejs` to install Node.js 18.x and npm
## You may also need development tools to build native addons:
     sudo apt-get install gcc g++ make
## To install the Yarn package manager, run:
     curl -sL https://dl.yarnpkg.com/debian/pubkey.gpg | gpg --dearmor | sudo tee /usr/share/keyrings/yarnkey.gpg >/dev/null
     echo "deb [signed-by=/usr/share/keyrings/yarnkey.gpg] https://dl.yarnpkg.com/debian stable main" | sudo tee /etc/apt/sources.list.d/yarn.list
     sudo apt-get update && sudo apt-get install yarn

+ curl -sSL https://cli.github.com/packages/githubcli-archive-keyring.gpg
+ gpg --dearmor -o /usr/share/keyrings/githubcli-archive-keyring.gpg
curl: (60) SSL: no alternative certificate subject name matches target host name 'cli.github.com'
More details here: https://curl.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.
gpg: no valid OpenPGP data found.

This is the version of curl that has an issue with the certificate in Ubuntu 22.04:
+ curl --version
curl 7.81.0 (x86_64-pc-linux-gnu) libcurl/7.81.0 OpenSSL/3.0.2 zlib/1.2.11 brotli/1.0.9 zstd/1.4.8 libidn2/2.3.2 libpsl/0.21.0 (+libidn2/2.3.2) libssh/0.9.6/openssl/zlib nghttp2/1.43.0 librtmp/2.3 OpenLDAP/2.5.14
Release-Date: 2022-01-05


adds cancellation refinement to download manuscripts component

this removes the use of sweet alert to provide loading state
it moves loading state inline into the file list
handles cancellation of upload inline as well
this makes a few minor tweaks to support multiple files
TODO:
[ ] add new download-service image once https://github.com/OA-PASS/pass-download-service/pull/3 is merged (here and in pass-docker)

Updated download-service image will be added in a separate PR


[Concurrency] Attempt to unbreak embedded build

Description: Unbreaks embedded build which cannot touch metadata field as it does not exist. No idea why this "worked" before, as this code was here for years and could not have worked on embedded.
Scope/Impact: Compilation fix for embedded.
Risk: Low
Testing: CI build
Reviewed by: @bnbarham
Original PR: https://github.com/swiftlang/swift/pull/75006
Radar: rdar://131191938

@swift-ci please test


missing_argument error when trying to use provided example

Not sure if this is still a working utility, getting missing_argument error for the following execution:
slack-cleaner --token $TOKEN --message --channel general --user "*"

Version: v0.7.4
Python Version: 3.8

Additional context
I've also tried to use slack-cleaner2 and get a similar error but with just a bit more text

This is the scopes my current application has installed into the workspace:

according to the docs that happens when you have an organization wide token, see https://api.slack.com/methods/conversations.list#arg_team_id

add support for team_id in https://github.com/sgratzl/slack_cleaner2/releases/tag/v3.0.7

so for slack-cleaner-2 to use a team_id would it look like this:
s = SlackCleaner('xoxp-token', 'team-id')

nevermind got it
s = SlackCleaner('xoxp-token', team_id='team-id')


Ng lint produces to multiple xml content in single file

🐞 Ng lint produces to Multiple XML content in single file

- [ x] new
- [x ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [x ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression? Yes
Angular 5 Was not having this issue. Once we moved to Angular 7 we are facing this issue.
Description
when we use ng lint for checkstyle it produces multiple xml content. This result in parsing failure of the xml.
🔬 Minimal Reproduction
when i run this
ng lint --format checkstyle > result-checkstyle.xml;
the result file contains :
<?xml version="1.0" encoding="utf-8"?><checkstyle version="4.3"><file name="../shoppingcart-ang7/shoppingcart/src/app/app.component.ts"><error line="208" column="160" severity="error" message="trailing whitespace" source="failure.tslint.no-trailing-whitespace" /></file></checkstyle>
<?xml version="1.0" encoding="utf-8"?><checkstyle version="4.3"></checkstyle>

This result in xml parsing error.

Duplicate of https://github.com/angular/angular-cli/issues/14659


Test mutation of ctor arguments

At present, it's cumbersome to test mutation of constructor arguments of mutable classes.
Consider the following class
public final class ListAdder<T> implements Consumer<T> {
    private final List<T> list;

    public ListAdder(List<T> list) {
        this.list = list;
    }

    public void accept(T element) {
        list.add(element);
    }
}

something like this might do the trick:
assertThat(param->new ListAdder(param),
     mutatesParameter(
          ArrayList::new,
          soIt(iterates("a")),
          when(sucessfully(adder->adder.accept("a")))));

closed by #95


fix: #[borsh_skip] on field of struct enum variant (BorshSerialize)

Resolves #173

@dj8yfo Let's use insta crate for snapshot testing, see this issue on near-sdk-rs and my comment mentioned there: https://github.com/near/near-sdk-rs/issues/1038

the actual fix is in this commit https://github.com/near/borsh-rs/pull/174/commits/fbe04fc9d48de93b54f0afcbf8a0187a4ac67c55

i've included change Span::call_site -> Span::mixed_site on three lines in commit


Send job fail notifications to PagerDuty

This might also be completed using webhooks (#59), but I'm going to keep it open as there may be an advantage to not using webhooks for PagerDuty specifically


Feat: Replaicing react hot toast with  Sonner toasts

Closes #555
✅ Checklist

[x] I have followed every step in the contributing guide
[x] The PR title follows the convention.
[x] I ran and tested the code works

Testing

Created new stories for UIs and toasts

Changelog

Removes react-hot-toast based Toast
Adds Sonner based toast

Screenshots

💯

I have decided to create a new PR considering it's easier to do this than fix the other branch :(
@ericallam


Disable Generating Unused Files for Distribution

This pull request resolves #170 by disabling ncc from generating the source map and license files for distribution.

There's seems an issue with the standard-usage job on macOS. But since changes in this PR should not affect that job, we can assume that the job should be passed.


Added track name list to the sidebar on smaller screens for filtering sessions

Partially Fixes the issue #1317
Changes:

Adds mobile view of the track name list for filtering sessions

Test Server
http://secure-meadow-20680.herokuapp.com
Demo Website
http://princu7.github.io/websiteTest
@aayusharora @geekyd @sumedh123 @harshitagupta30 Please review. Thanks :)

Here are some of the screenshots

@aayusharora @geekyd @sumedh123 @harshitagupta30 Please review. Thanks :)

@Princu7 Great work! Can we hide the side menu when the user clicks on the track titles?
It will improve the UX. What do you think?

@Princu7 Also one more suggestion, can we blur the other part of the screen when the side bar is active? Or this can be done in other separate issue also.

@sumedh123 Thanks for the suggestion. Yes, we should do it in a separate issue later when major issues are resolved. That would be more apt I guess. @geekyd Yes, that is a good option. I will try to change to it. Thanks!

Made the suggested changes @geekyd Here is the test server link. Now on clicking the name of the track in the sidebar will close it.
Test Server
http://secure-meadow-20680.herokuapp.com
Test Site
http://princu7.github.io/websiteTest


Update build settings

See: https://github.com/bumptech/glide/pull/318

I close this pull-req because of https://github.com/bumptech/glide/pull/318#issuecomment-71769779


Renaming and cleanup

Codecov Report

Merging #7 into master will decrease coverage by 0.85%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
- Coverage   92.92%   92.07%   -0.86%
==========================================
  Files           2        2
  Lines          99      101       +2
  Branches       18       17       -1
==========================================
+ Hits           92       93       +1
  Misses          6        6
- Partials        1        2       +1

Impacted Files
Coverage Δ

...iegsa/wiremockextension/ManagedWireMockServer.java
50% <ø> (ø)
:arrow_up:

...enspiegsa/wiremockextension/WireMockExtension.java
97.75% <91.66%> (-1.1%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix on MemorySource.load_from_file()

Simple typo in MemorySource

Codecov Report

Merging #151 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #151   +/-   ##
=======================================
  Coverage   97.38%   97.38%
=======================================
  Files          56       56
  Lines        5540     5540
=======================================
  Hits         5395     5395
  Misses        145      145

Impacted Files
Coverage Δ

stix2/datastore/memory.py
98.88% <0%> (ø)
:arrow_up:

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 e92db24...0bb1bb3. Read the comment docs.

Good catch! Thanks

The line was uncovered initially. Though, it can be addressed when the datastore coverage fixes are done.


[SR-15993] @MainActor type restricted from implementing simple protocol

Previous ID
SR-15993

Radar
None

Original Reporter
@OneSadCookie

Type
Bug

Environment
Xcode 13.3 / Swift 5.6

Additional Detail from JIRA

Votes
0

Component/s

Labels
Bug

Assignee
None

Priority
Medium

md5: e9434d87b0433c2a29beaee1eba08ea1

Issue Description:
Given the following code:
protocol HasAString {
    var string: String { get }
}

@MainActor
class Foo: HasAString {
    var string: String
    init(string: String) {
        self.string = string
    }
}

And compiling with -warn-concurrency, I get an error I don't expect:
swift -warn-concurrency example2.swift
example2.swift:7:9: warning: property 'string' isolated to global actor 'MainActor' can not satisfy corresponding requirement from protocol 'HasAString'
    var string: String
        ^
example2.swift:2:9: note: 'string' declared here
    var string: String { get }
        ^

What confuses me here is that nothing here is Sendable, so a Foo instance can't escape the main actor, even if upcast to any HasAString, so why is it unsafe for it to implement this protocol?

This is still an issue in Xcode 14b1, though you no longer need -warn-concurrency; the warning is emitted by default.

I think (?) this is because @MainActor does cause an implicit conformance to Sendable for a non-public type, but this is still incredibly unintuitive (and incredibly unhelpful; we have a lot of existing code that presumes that UIKit subclasses can be made to conform to protocols)


Update zk timeout to 60 seconds Fix test assertion order

This PR will address some of the failures due to ZK connection timeouts

merged with pint


[Improvement] Eliminate raw use of parameterized class

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
When I'm browsing the Uniffle codebase,  a lot of raw use of parameterized class occurred, especially in test classes.
For instance, the iterator parameter and product2 variable is raw use of parameterized class
https://github.com/apache/incubator-uniffle/blob/66c752fe3ecd6d4979d3539fe22265d4239156a9/client-spark/common/src/test/java/org/apache/spark/shuffle/reader/AbstractRssReaderTest.java#L51-L63
How should we improve?
Eliminate the raw use of parameterized class as much as possible.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Hi @advancedxy
I'd like to handle this issue if you agreed. Thank you.

I assigned to you.


Update pull request

I see that in your code you return if the pull request already exists.
My use case is to open and a pull request from master to branch prod and I would like to auto update the description of this pull request with the list of pull requests that have been merge into master since the last merge of master into prod'
God this looks complicated said like this.
Workflow
Merge pull request 13 from feature1 branch to master -> Create pull request 14 from master to prod with a description like
## Pull request created at {date} to release to prod

## Features Merged
#14 Feature1

Merge pull request 14 from feature2 branch to master -> Update pull request 14  with something like
## Pull request created at {date} to release to prod
Added description by the user

## Features Merged
#14 Feature1
#15 Feature2

Obviously I would need to add a line at the end of the description or split the body on "## Features Merged" to reconstruct the list
Does it makes sense to you?

hey @Benoss - if you open a PR and push any changes to that branch, that alone will update the PR. I would suggest taking an approach that is less hard coding in some specific action for your use case, and more putting together steps like modular pieces. More specifically, given some variable set by pull request action, you instead comment again on the PR. E.g., modify this recipe to check the return codes and then act accordingly. If you want to set up a recipe like that and then show the various outputs, what we might need to update is making sure that the PR number for the existing PR is returned. Please give that a start and then link me to the running action and we can go from there.

Yeah I realise that's a bit complex, first step is probably just a variable to allow updates :)

Sorry I think you are misunderstanding - given that a PR is already open, if you just push again to that branch, the PR is updated. There is no special action needed on the part of the action here if the PR is already open :) If you, however, want to post an additional comment, I'd suggest setting up a recipe as I suggested, printing out those output variables, and then if something is missing we can add it. But I think you should be able to just parse them, determine based on the exit code that it's open, and then post a comment. See snakemake for an example of doing this - it posts a comment on the PR based on needing black formatting.
https://github.com/snakemake/snakemake/blob/master/.github/workflows/main.yml#L25
You would want a similar thing, but have the if condition triggered based on the PR already being open (the status code) and the number (the pr number).

The other thing that would be good is to be able to set the the author of the PR to the person that pushed. Also be able to use an existing pull request template in the repo if exists

The author is likely available as a variable in the GitHub payload, and environment inputs already are defined to add assignees, reviewers, or team reviewers, so you should already be able to do this. If you need a custom message or title, just set those as envars too. I’m not sure anymore what you are asking for development or feature or help wise, so I’m going to close the issue. If/when you implement something and have an actual use case that we can debug, happy to help again.


[BREAKING] Use babel-preset-env instead of babel-preset-latest

From Babel's Slack:

Not compatible with uglify-js.

Detect uglifyjs-brunch and set "uglify" as target. Easy.
https://github.com/babel/babel-preset-env/pull/178

@shvaikalesh looks good. Seems like a possible solution.

Yes, just add uglify: true for production.

Should be fixed by #53.


Compiler warning: value used before set in PyInt_AsSsize_t(x)

Cython 0.28.5 generates the following code (which seems to be there since earlier version of cython).
static CYTHON_INLINE Py_ssize_t __Pyx_PyIndex_AsSsize_t(PyObject* b) {
  Py_ssize_t ival;
  PyObject *x;
#if PY_MAJOR_VERSION < 3
  if (likely(PyInt_CheckExact(b))) {
    if (sizeof(Py_ssize_t) >= sizeof(long))
        return PyInt_AS_LONG(b);
    else
        return PyInt_AsSsize_t(x);
  }
#endif
...

The compiler complains about x not being initialized, e.g., warning #592: variable "x" is used before its value is set return PyInt_AsSsize_t(x);.
So I'm not sure but it seems that it should be b instead of x here?!

Interesting. :)
I don't know any architecture where sizeof(Py_ssize_t) < sizeof(long), so this is probably just a safety fallback that's not used in practice, but you're right, it should use b.


Provide User Storage to Researchers across Windows and Linux VMs

As a Workspace User I want a storage area for my sole use, mounted into my virtual machine, so that I have my own space to work in.
Note:
Although this storage needs to be workspace specific it does not need to be private to the user, as the workspace is the security boundary. Although private storage for each user within a workspace has been discussed with users, after discussion it has been deemed not appropriate.
Acceptance criteria:

[ ] Per user storage available in home directory of Windows VMs
[ ] Per user storage available in home directory of Linux VMs

Open to suggestions as to what "home directory" means, and as to what is appropriate.

@marrobi is this an accurate reading of the requirements...?

Researchers, within a workspace, should be given a folder that is clearly "theirs". e.g. the folder name is based upon their username, or the folder is their home folder, or something similar. In my case, that folder might be named martinpeck
This folder should be present when they first attach to a VM within a workspace
This folder is shared across all VMs within that workspace
This folder is shared across Windows and Linux VMs within that workspace
Other users within the workspace have their own folders
Other users within the workspace might be able to traverse the filesystem and see the contents of another user's folder. Once in another user's folder, that user can manipulate (create/read/update/delete) files.
These user folders are not shared across workspaces

sorry...should have numbered that list to make it easier to discuss (done)

Correct, add to no 3 - "shared across all their VMs"

On the file system previously used a users object id as the azure file share name.

I'm assuming that a user does not know their objectid though? Do you mean their AAD objectid

If we mount the same shared storage to each VM, can't users create their own folder structure there?

I agree @tamirkamara. If they have shared storage, the folder structure they create (whether it be based upon usernames or otherwise) is up to them.
@marrobi other than the convenience of knowing "when I join a workspace I can assume there's a folder with my name on it"  is there some other reason we need to do this?

Two scenarios that need covering. Although wouldn't priotise this, has been asked for, but not a blocker.

I do some work on a VM. I store that work in my home folder. I resize that VM. I lose my work. Hence some shared storage should be mounted to the common profile directories - home and desktop spring to mind.

Although data does not need to be secured privately, I want to have my own space where it is not easy for other users to move/delete my files.

First one is more pressing as think it will be an issue before long.


Four column design

As a user, I want to organize my types and type references in a four-column design.
ACs:

[ ] there are the columns Bay Types, Conductive Equipment Types, Function Types and Logical Node Types from left to right
[ ] the columns can be shown and hidden as before
[ ] the columns have corresponding headings


Eterna3D on mobile - zooming while rotate/pan selected is clunky

For puzzles featuring Eterna3D, there are currently 3 ways of navigating the 3D structure - rotating, panning, and zooming. Currently, there is a dropdown menu where players can select each of these navigation tools individually. And through this menu, the rotating, panning, and zooming tools work well.
However, I noticed that while the rotating or panning tools are selected, it is also possible to zoom by using 2 fingers on the screen instead of 1. When doing this, the behavior is very clunky - the amount of zoom that occurs is relatively small and also inconsistent.
From the player perspective, it would be nicer if this "two-finger" zooming exhibited similar visual behavior as when the zoom tool is selected.

This is actually handled by NGL, not us directly. I'll track this in the EternaJS repo though.


Python: support GitHub in update script

Issue description
The update script uses the PyPI API to determine whether updates are available and uses the hashes it provides.
Some expressions do not use fetchPyPi or fetchurl with an url pointing to PyPI`. In those cases, we shouldn't use the PyPI API.
Therefore, an initial check should be included that checks what kind of repository is used. Then, we can also support GitHub for the few scripts that use it.

Are there any updates on this issue, please?

I assume #99635 should have closed this; reopen if I'm wrong.


Bluetooth: L2CAP: ECRED fixes

@Vudentz which releases should this be backported to? Please add the labels for those (the automatic backport PR creation works again, but not for the generic "Backport" label)

@Vudentz which releases should this be backported to? Please add the labels for those (the automatic backport PR creation works again, but not for the generic "Backport" label)

Ive added the version where ECRED support has already been introduced.

AFAIK ECRED was first introduced in the 2.3 release, should we add it there as well?

AFAIK ECRED was first introduced in the 2.3 release, should we add it there as well?

Backports only go back 2 releases.


Javascript is required to use the site

When javascript is disabled, the page is unusable - every subpage is covered by full screen welcome page (see the screenshot below presenting /members subpage - notice the url bar and active no-script plugin)

Działa


"indirect call to null" \ "null function or function signature mismatch" error only on web exports

Godot version
v4.0.beta.custom_build [f692b4768]
System information
Linux
Issue description
I get these error only on the web export (both on localhost and in production).
Here is what I get on Firefox:

and on Chrome:

The errors appeared with the recent update, had no problem beforehand. And they only appear on the web export.
Steps to reproduce
I'm not sure exactly what caused this error,  'LetterSquaresGenerator.gd' is a class that inherits Node, so this line shouldn't throw an error:
@onready var _generator: LetterSquaresGenerator = $LetterSquaresGenerator
And even if didn't inherit from Node, and this line really is an error, why would the error only appear on web?
Minimal reproduction project
don't have one, sorry :(

Can you still reproduce this issue in the latest 4.0 beta?

Oh I posted this quite awhile ago and the problem is gone in the latest version, web export working fine again :) I will close the issue


.objrefs problem with so -bf imd

This pull request includes a small but important change to the ImpactMarkdown class in the cli/src/builders/imd.ts file. The change ensures that the logs variable is initialized to an empty array if no logs are found for the given ileObject.relativePath.

cli/src/builders/imd.ts: Modified the getLogsFor method to return an empty array if no logs are found for the ileObject.relativePath.the same issue as mentioned #79 (.objrefs problem with so -bf imd).

Thanks @marcin-ogon!


[NO  MRG] MNT: Re-rendered with conda-smithy 3.0.0.rc2 and pinning 2018.03.03

Check conda-smithy=3.0.0

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The recipe must have some tests.

Intriguing @isuruf! Thanks for putting this together. So is this based on some development work for conda-smithy somewhere?

So is this based on some development work for conda-smithy somewhere?

No. Just hand editing. As a start I'd like to get https://github.com/conda-forge/conda-smithy/pull/710 in and then work on it after 3.0.0. Also there's a PR at https://github.com/conda-forge/conda-forge-ci-setup-feedstock/pull/7 too

Is this still needed given PR ( https://github.com/conda-forge/occt-feedstock/pull/23 ) went in?

Right. Everything should be included already.


Load history from API

Hello, Fredrik. Thanks for the great component that you created.
Is there a possibility to obtain the history from an api, instead of the local storage?  In this case I think it could be the initial messages, coming from an api.
I saw your comment on the issue below, but still, I don't know how I could read the history of an api.
https://github.com/FredrikOseberg/react-chatbot-kit/issues/4#issuecomment-683711856
Thank you.

Hello, Fredrik. Thanks for the great component that you created.
Is there a possibility to obtain the history from an api, instead of the local storage? In this case I think it could be the initial messages, coming from an api.
I saw your comment on the issue below, but still, I don't know how I could read the history of an api.
#4 (comment)
Thank you.

Hi, thanks for the kind words.
Yeah, you can obtain the message history from an API, there's a few ways you could do this currently.
One way would be to implement it in the loadMessages function:
const loadMessages = async () => {
     const res = await getMessageHistory()
     const data = await res.json()

     return data;
}

<Chatbot ... messageHistory={loadMessages()} />

Another way would be to inject it directly into your config:
// fetch the history from wherever you want in your app and load the config dynamically

const getConfig = (messageHistory) => {
     return {
          initialMessages: [...messageHistory]
     }
}

<Chatbot config={getConfig(messageHistory)} />

Does that work for you?

It definitely works for me, Fredrik. Thank you for your quick response.


Records Management compliance statement

Can we please include which Records Management standard this functionality is compliant with? It doesn't comply with ISO 15489 or ISO 16175 which are applicable to Australia, New Zealand and the United Kingdom. It does comply with US Federal Records Management.

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

ID: cc52e4d6-6373-204d-9eb2-d8c396375fb1
Version Independent ID: 106caf2c-6de5-3cc9-6cdd-4dc81dcbcc8b
Content: Records management - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/records-management.md
Service: o365-seccomp
GitHub Login: @cabailey
Microsoft Alias: cabailey

@gavinmckay  Thanks for your feedback. These technical docs focus on how to use the records management solution in Microsoft 365.  When implemented, this feature can help you be compliant with the regulations you need to follow. Certifications for our products are typically documented in the Service Trust Portal (https://servicetrust.microsoft.com/).  If you can't find what you're looking for there and it's important to your deployment, let your Microsoft Account Manager know.


.formkit-control-label class in "repeater" and possible others are visible

Reproduction

Describe the bug
.formkit-control-label class should have e.x. z-index: -1 and / or opactiy: 0, or another way of hiding as position: absolute and left: -999px are still on screen if they are in a position: relative container.
What exactly are these labels used for if they are not visible? Screen readers? Any "right" solution to that, except for changing the class?
Environment
• OS: Windows
• Browser: Chrome
• Version: 106.0.5249.119

Are you saying this is a bug with the genesis theme, or your personal theme? The parent .formkit-controls has overflow:hidden in genesis so all those values are not visible no matter how wide the screen is. The labels are there for accessibility purposes. Can you provide a genesis reproduction of this issue?

@justin-schroeder Here you go: https://formkit.link/eef9b50248b1e52701debb58cb07afbc
If you size it further to the right, you can see it. At least on my screen it is there without changing the size at all.

I would recommend to use overflow: hidden on .formkit-fieldset, which seem to fix this, and maybe increase left: -999px to more than that?

Should have position:relative on .formkit-controls. We wouldn't want the fieldset to have overflow hidden or children couldn't overflow their container (like dropdowns). You can add that as a workaround for now:
.formkit-controls {
  position: relative;
}

This will eventually make it into the pro package too.

position: relative has been added to .formkit-controls in the pro package.


Keyboard shortcuts list (or any other list we can scroll) column headers should always be visible

When listing keyboard shortcuts, keep the column headers visible (fixed) when scrolling down.
In fact, please consider making this a default whenever you show a list that we can scroll.

@sandy081 , can I work on this ?
Some starter points will be helpful , Thanks :)

Keybindings editor rendering code is here - https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/preferences/browser/keybindingsEditor.ts#L1
Please go through it and let me know if you have any questions.

@sandy081 , Thanks :)
https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/preferences/browser/keybindingsEditor.ts#L720 , This is where the headers are being set , But I did not quiet get how to add those CSS kind of attributes to this ( Like position:"fixed" ), if you could throw some more insights it will be helpful :)
Also should we consider changing it generally when a list is scrollable ?

Just as a general observation, how do you go about debugging this? Like for web development you would open up a browser and check output. How is this done for vscode? e.g. where do you preview the HTML/CSS changes you are making?

Hey @Benny1007 , You can use Debugger for chrome
Also, All changes should be seen once you rebuild.
Refer this for more (https://github.com/Microsoft/vscode/wiki/How-to-Contribute#build-and-run)

@sandy081 I spent a few hours on this yesterday, and I'm able to render the headings as part of the header (rather than the list). This is what it looks like:

Code needs to be cleaned up (and definitely added to the List or WorkbenchList API), but this is how I did it: https://github.com/Microsoft/vscode/pull/65892/files
Would you mind directing me to where in the code base the header is actually added to the DOM? I can't seem to find it. Also, if you have any thoughts/suggestions, I'd really appreciate it 😄 .

Fixed by #41558


Add db agnostic computations service API tests

This PR adds a new abstract ComputationsServiceTest class which is supposed to define test cases for all Duchy Computations Service APIs. This PR only implements two test cases for createComputation rpc.
This PR also adds a Spanner implementation of this test which runs all API tests against the Duchy Computations Service with spanner database.

This change is


convert expressions to classes

Don't know how you have gotten away with calling obj.constructor() without using new.

Nice progress.

Don't know how you have gotten away with calling obj.constructor() without using new.

By the magic of ES3