OpenCV is bugged in vcpkg when building with Clang

Describe the bug
The find_contours function of OpenCV returns points which are way outside of the bounds of the input image. This happens when installing opencv4 with vcpkg and building the project with Clang-13. This does not happen when building with GCC-10. Furthermore, manually installing OpenCV from source without vcpkg does not display this bug for both Clang-13 and GCC-10. The points seem to be off by a fixed offset, however this seems to change everytime the program is executed.
Environment

OS: Ubuntu 20.04.3 LTS 64-bit
Compiler:  clang version 13.0.1
vcpkg at commit 7e7dad5fe20cdc085731343e0e197a7ae655555b

To Reproduce
main.cpp:
#include <iostream>
#include <opencv2/opencv.hpp>

int main(){
    cv::Mat img{200, 200, CV_8U, 0.0};
    cv::Rect rect{50, 50, 100, 100};
    cv::Scalar col{255, 255, 255};
    cv::rectangle(img, rect, col, -1);

    std::vector<std::vector<cv::Point>> contours{};
    std::vector<cv::Vec4i> hierarchy{};
    cv::findContours(img.clone(), contours, hierarchy, cv::RETR_TREE, cv::CHAIN_APPROX_SIMPLE);

    for (const auto &points: contours)
        for (const auto &point: points)
            std::cout << point << std::endl;
    return 0;
}

CMakeLists.txt
cmake_minimum_required(VERSION 3.22)
project(opencv_bug)

set(CMAKE_CXX_STANDARD 17)

find_package(OpenCV REQUIRED)
include_directories(${OpenCV_INCLUDE_DIRS})
link_directories(${OpenCV_LIB_DIR})

add_executable(opencv_bug main.cpp)
target_link_libraries(opencv_bug ${OpenCV_LIBS})

vcpkg.json
{
  "name": "opencv-bug",
  "version-string": "1.0.0-SNAPSHOT",
  "dependencies": [{"name": "opencv4"}]
}

Then build with:

-DCMAKE_BUILD_TYPE=Debug
-DCMAKE_MAKE_PROGRAM=/usr/bin/ninja
-DCMAKE_C_COMPILER=/usr/bin/clang-13
-DCMAKE_CXX_COMPILER=/usr/bin/clang++-13
-DCMAKE_TOOLCHAIN_FILE=~/vcpkg/scripts/buildsystems/vcpkg.cmake
-G Ninja

Expected behavior
find_contours should return points which are inside of the image for both compilers. Specifically, the code snippit should print:
[50, 50]
[50, 149]
[149, 149]
[149, 50]

However, with clang it is printing points like these:
[41437298, 32814]
[41437298, 32913]
[41437397, 32913]
[41437397, 32814]

Additional context
Manually passing an offset of (0, 0) does not fix it. Nor does casting the output to different int-types. Also tested with older commits of vcpkg and still present there (431d3810aa9241aed49b14367be417f225d2af5e).

cc @cenit

I am building opencv using x64-windows-llvm triplet from here https://github.com/Neumann-A/my-vcpkg-triplets right now
@Neumann-A
i will let you know if i can reproduce errors

Also, I'd suggest to use a triplet for x64-linux-llvm instead of manually passing parameters to cmake when building, to have better consistency.
Please write a triplet file and share it here, so that I can test it later on an ubuntu machine

it looks like the library built by gcc-10 is not perfectly compatible with a downstream project built with clang-13?
Might it be some missing -fPIC somewhere in a dependency?
Why not defining your own triplet in vcpkg and let it build ports with clang, too? I never mix and match compilers, that might be a problem due to this situation?

Our build server is running Ubuntu, but we have several macOS developers who wanted to use clang on the server as well.  So as I understand, vcpkg will just use whatever compiler the triplet specifies and it doesn't follow the DCMAKE_CXX_COMPILER flag. Which is causing the vcpkg dependencies to be built with gcc, but the rest of the project with clang, is that correct? So to have vcpkg use different compilers or compiler flags I need to define a triplet and pass it to vcpkg. I can try doing that. Still, I wouldn't expect that to change not just the behaviour, but the correctness of the find_contours function of opencv.

I have been told that vcpkg has a strong tendency to mix toolchains!
@rodneysnacks You could check (or verify) your environment(s) so vcpkg doesn't get confused.
We don't use clang but will try to repro this issue locally.

Was busy last week and forgot to post an update. When vcpkg is also configured to use Clang, it worked without issues.  So the mixing of compilers was the issue. This can be closed as far as I'm concerned.
Sidenote, I used export CXX=... to get vcpkg to use Clang. If I want to do this via triplet I should append set(CMAKE_CXX_COMPILER ...) to it? Is there a way that is prefered? I kinda like the export, since I don't have to edit any files and I know that each command I execute will use the same compiler.

If I want to do this via triplet I should append set(CMAKE_CXX_COMPILER ...) to it? Is there a way that is prefered?

Yes, you should writer a new triplet.


chore: prevent middlewares/db to be active during locked phase

✅ Checklist

[ ] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

📝 Description
to generalize the issue that was fixed in #6863, we can prevent db to attempt to save app.json during the locked state
(this PR is living proposal, just created in order to discuss this)
❓ Context

JIRA or GitHub link:

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

the plan has been to reconsider this for a more general fix, but also a better implementation may be to move away from middlewares pattern and embrace LLM's pattern: just react from the store updates at the correct place of the app (= as a children of "IsUnlocked" instead of a global middleware listen)
https://github.com/LedgerHQ/ledger-live/blob/e541abf81b564f7175e1ed9931292e19d852af6b/apps/ledger-live-mobile/src/index.tsx#L165


Implement rotate stream

Take the pointerEventMetastream (#64) and return a stream of changes in the angle between two pointers.
Calculating the angle between two points should be basic trig.  Then, you're just diffing adjacent events to find the delta.  If that algorithm doesn't work, @maxw wrote some gesture recognizers for Form.  Take a peek at the source.
Open questions:

What if there are more than two pointers?

Same problem in #66; for now, you can probably ignore this scenario.

Pen

💻 I'm working on a diff at http://codereview.cc/D1692


[Suggestion]: Compatibility with LocalizedChat (or ProximityChat)

General Info

[ ] I would like to propose new content
[ ] I would like to propose changes
[X] I would like to propose another thing

Suggestion Description
Hey!
We would love to see your mod compatible with LocalizedChat, or ProximityChat (we can switch for this one if needed, both are same).
LocalizedChat
ProximityChat
Thanks!

What happens currently when the mods are used?

Bubble doesn't appear.


Filter

Filter!
You can filter

ratings,
features,
the difference between the start time of the ride and the time you searched for,
and sort by the aforementioned time difference, price, or relevance (a combination of the two)!

I also redesigned the top of the SearchSuggestionsPage a bit, since the filter button was an eyesore before. Let me know your feedback! (The row below the start and destination timeline is copied from the search ride and create drive pages and can be improved on when their time comes).

what does the shrug asset do? and the arb files should probably not be completely different?

You can easily test what the shrug asset does by trying to search for a ride from Poing to Titz!
Yeah I suspect the arb files are due to line endings with Windows, I'll fix that

When I try to select exclusive features the alert is behind the overlay so you don't see it

FYI: I refactored the filters into an own class because the SearchSuggestionPage became a bit messy. This should not affect functionality.


LN channels do not re-open and wallet does not receive funds back after HDD crash

Background
Had a working setup using stadicus guide, 7 channels open with autopilot. After a week I had a harddisk crash. I had to replace my harddisk and did a new setup of the node.
I imported my wallet seed to re-use my old wallet and hoping the channels would open again, or would receive my funds back, as the channels were not closed properly
Setup used the mainnet.
Your environment
"version": "0.4.2-beta
"operating system": "Raspbian Stretch"
"bitcoind": 17.0
Raspibol setup
Steps to reproduce
Setup a node on a pi using the Raspibolt guide. Fund the node and setup channels; Remove the harddisk. Format the harddisk and re-setup the node. Import your wallet with the previous received seed.
Expected behaviour

I would expect the funds to return to my wallet, as the node did not close the channels properly, because the harddisk crashed with the information on it. I would expect the channels to close, as they can not used/active anymore.
or
I would expect LND to see based on my wallet seed, what channels were open, and restore them.

I would at least like to restore the channels or force close them, but that seems not possible.
Actual behaviour
The wallet is empty. According to 1ML.com when I lookup my node, it has 10 channels open now. 3 of them are new and the other 7 channels that I had open previously are still open. The channels did not close and remain active, when my side of the channel is gone.

I imported my wallet seed to re-use my old wallet and hoping the channels would open again, or would receive my funds back, as the channels were not closed properly

The seed only contains on-chain keys, and info w.r.t the off-chain keys that we used. It doesn't yet include static channel information. Until that's in place, we recommend ensuring you're using a stable, reliable medium for a disk. This feature will be implemented soon, and can be tracked at #175.

"version": "0.4.2-beta

Also I recommend updating to 0.5, as a ton of bugs (and also generally performance improvements) have landed since then.


add column 'verified' to cloud-resources-list.md

Signed-off-by: Nicola115<EMAIL_ADDRESS>

I've add sort for docs. But it's strange in Chinese version docs. As the provider names both include Chinese and English, like 'AWS' and '阿里云'.

please add sort for docs

I've add sort for docs. But it's strange in Chinese version docs. As the provider names both include Chinese and English, like 'AWS' and '阿里云'.

@Nicola115 Will it cause the same big change as the random order when we update next time?

@Nicola115 Will it cause the same big change as the random order when we update next time?

Yes, you need to sort it after updates if you want to remain the alphabetical order. I've write a tool for sorting docs, and @zzxwill raise an issue for it. https://github.com/oam-dev/kubevela/issues/3536 Guess we might include the sorting in kubevela soon.


Exponential notation (e.g., 1e+140) is not parsed as a BigInt but as a double

Hi,
While trying to use json_bigint for learning about writing Flutter and dart apps I discovered the package doesn't recognize exponential notation for integers.
Using the example code as illustration,
String json =
    '{"sometext": "hello world!", "bignumber":<PHONE_NUMBER>00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000}';

correctly parses bignumber as a BigInt, but
String json = '{"sometext": "hello world!", "bignumber": 1e+140}';
incorrectly parses bignumber as a double (just as convert does).
In my current use case I am pulling the number from an external website so have no direct control over when it decides to hand me an integer in integer notation or in exponential notation.
(Also, as a side note, the example code is missing the '}' closing character on the json declaration such that running the example code results in a runtime parsing error.)

So I'm thinking the issue is caused by Dart itself.
At a low level Dart appears to convert numbers like 1e+140 to doubles, automatically, with no apparent option to keep it from doing so. Worse, even trying to sidestep the problem by using BigInt.parse('1e+140') results in a FormatException as, apparently, BigInt, itself, doesn't handle exponential notation, at all.
I was originally fooled by the documentation on this because I saw that BigInt.parse('7.5', radix: 16) worked fine until I realized it worked fine because e is a hex value not an exponential indicator.
Not sure why BigInt was designed in such a strangely broken way but at least I understand what it is doing now. I'm too new to Dart to know how this is supposed to be handled but I can only assume most, if not all, pathways will end up with big exponential notation values getting converted to a double which, basically, just sort of breaks everything.
Hopefully there is an answer out there but the only answer, at the moment, that I can see here is if we altered the parsing code to convert things like 1e+140 to a canonical string, first, and then parsing that with BigInt.parse() from there. The bigger problem would be going the other way as I'm thinking there are probably many edge cases that would get in the way such that encodeJson(decodeJson('1e+140')) may not be 1e+140.

I think you're right about converting it to a canonical form. I'm gonna try doing that.
As for the errors in the example code, I've fixed that, thanks for point it out.

I played with it a little a couple days ago as a Dart learning exercise. The numberPrimitive petitparser matcher already captures the form correctly so I thought about separating it into an integer and exponential notation form (which might be a decent answer) and handling it there but instead started modifying the _numOrBigInt method.
Immediately ran into the first "interesting" design problems:
1 to what extent should a BigInt raised to the BigInt be supported?
2 should an arbitrary chaining of exponents be supported? (.i.e, X1^Y2...^Xn)?
3 should there be an upper limit to an allowed value?
4 would it just be easier to have an option to decode exponential notation numbers (or any numbers, actually) into strings instead of turning them into number?
The third one struck me as important lest some malicious, or prankster, number source present a googolplex raised to the googolplex (10^10^100)^(10^10^100) raised to the ... you get the idea. Since "arbitrary" would let someone do such things, especially if chaining was supported, it would a lame if there was no way to control or limit that behavior.
While pondering how one might handle such issues it occurred to me that the allowing BigInt values to just not be turned into numbers might also be an answer. The tutorial I was going through only used the values for presenting (which would work fine), not calculating, and, more importantly, the website that the code interacts with uses the numbers for returning random trivia about the number which led to this additional, possible, feature for the encodeJson encoder:
5 allow strings formatted as numbers to be encoded as numbers instead of strings.
This last part would help support the canonical and reversible portions of the parser.
I started down this path, wrote some tests, updated the example code (to also include using the EncoderSettings variable which the original defined but never used), and then went "but what does ozaner want to do? Maybe the vision is entirely different than anything I just laid out."
So I sort of just stopped until maybe I heard back or just became super interested in seeing some of these changes I was pondering actually work. I did fork it off to safely play with it so if there was anything you found interesting (really only the example code is current in the forked repo) then feel free to grab it or let me know and maybe I could learn how to submit patches on github and send it your way all official like and stuff).
Thanks for making and posting the code. It has been fun to play with and learn from. And it also introduced me to petitparser which, it just so happens, is exactly the library I was needing for another little project piece of Dart code I'm playing around with right now. So json_bigint has been a big hit for me!

Ok I released an updated version of the package as v2.0.0, that adds support for this. You can see the documentation on pub.dev, but what's relevant to this discussion is that it adds DecoderSettings.
By default, the decoder still treats any number written in exponential notation (e.g. 1.212e+3, 2e4, 3e-4, etc.) as doubles. However, when  treatExpAsIntWhenPossible (one of the new decoder settings) is set to true, it tries to parse these exponential numbers and determine if they would evaluate to integers. If so, it converts them to integers instead of doubles.
For example:

1e6 = 1000000 is an int so it becomes a BigInt.
-23000e-3 = -23 is an int so it becomes a BigInt.
2.312e2 = 231.2 is not an int so it becomes a double.
43e-2 = .42 is not an int so it becomes a double.

You can look at the source code if your interested, but basically I just made a custom regex expression that runs on the string should it fail to parse as a BigInt. Then I do some calculations to make sure it evaluates to an integer with no decimal part, and return it as a BigInt if so.
As for representing large numbers as Strings, this is possible and is the most common way of getting around the problem of decoding BigIntegers from JSON in a lot of languages. You can just encode integers as strings and convert them back to integers yourself.
For my use case though, I didn't want to have my output JSON include quotations around the numbers. This meant I needed to be able to truly serialize BigInts rather than just their string counterparts. This is ultimately is why I made this library.
Anyway thanks for using and experimenting with the library, I hope this update allows you to decode exponentials as BigInts. Assuming I didn't break anything. If it doesn't work as intended, you can just reopen the issue.


Adding Support Raw Public Key mode (RFC7250)

Description
A lot of people ask for support of Raw Public Key.
This is a working base implementation, hope this could be useful.
Signed-off-by: Flavio Ferrandi<EMAIL_ADDRESS>Status
READY
Requires Backporting
NO
Migrations
NO
Additional comments
Todos

[x ] Tests
[x ] Documentation
[ ] Changelog updated
[ ] Backported

Steps to test or reproduce

Hi Flavio,
Thank you for your contribution. Do you intend to continue the work on this feature? Unfortunately, the Mbed TLS team won't have time to work on it in the foreseeable future.
Is this based on the previous work by https://github.com/ARMmbed/mbedtls/pull/336 and my continuation of it, or an independent reimplementation?
We will definitely not be able to merge this pull request in its present state because we won't accept a new feature without tests. There are three kinds of tests we can do:

Unit testing in test_suite_ssl. This isn't strictly necessary because a lot of the existing TLS functionality doesn't have unit tests, but it would be easier.
Interoperability testing. At a minimum, compat.sh would need to run an Mbed TLS client with an Mbed TLS server. OpenSSL doesn't support raw public keys. GnuTLS added support in 3.6.6; the most recent version of GnuTLS we currently use on our CI is 3.6.5 so we'd need to upgrade it.
Testing of relevant option combinations in ssl-opt.sh.

Not everything has to be done from the start or by the same person. It would be easier to have several small pull requests than a huge one. But each step does need to have at least some functional testing of the features that it introduces.

Hi Gilles,
yes, it's a port of https://github.com/ARMmbed/mbedtls/pull/336. This implementation is enough for implementing a working DTLS coap client, so I haven't planned any further development. If you tell me what kinds of test are best in this situation, I will try to do these so you can validate this PR.
Have a nice day, Flavio

Are you running this code against itself, or against another DTLS implementation that supports raw public keys?
We don't have well-defined test objectives, but for this feature, I think a minimum would be to add support for raw public keys to compat.sh, which tests Mbed TLS against itself, OpenSSL and GnuTLS. At first we'd only test Mbed TLS against itself since GnuTLS requires a more recent version that what we currently have on our CI. Ideally we'd also have extensive negative testing, but that's a pre-existing test gap for TLS in general so it isn't required to merge raw public key support.
In addition, we'd want to look at the line coverage. We'd look at the difference this patch makes to the figures reported by basic-build-test.sh. We'd look at what lines are not covered, and see if it's practical to add coverage for them.

I'm running running this code against a node js DTLS server that supports raw public keys.
I will look at compat.sh  and I will try to write a sample working client.
I keep you updated.

Hi @flaz83,
Thank you for your contribution!
A very useful first step towards the required level of testing would be the integration of raw public keys into the SSL test applications programs/ssl/ssl_{client|server}2, since it is those applications that are used in our testing scripts ssl-opt.sh and compat.sh. Have a look at the code of the examples, they should be straightforward to extend, but don't hesitate to come back to us if you get stuck with anything.
Thanks again for your work,
Hanno

integration of raw public keys into the SSL test applications programs/ssl/ssl_{client|server}2

Yes, thanks for mentioning that. It's a necessary step to doing anything beyond unit testing of individual functions.
Note that I've already started to do that in https://github.com/gilles-peskine-arm/mbedtls/tree/iotssl-222-raw-public-keys/spark . The work wasn't complete, and needs to be adapted, but I think it would be easier to start from what I did than to start from scratch.

Please note that there are a number of CI failures. Some are visible on Travis, but there's a known issue that the GnuTLS interoperability tests have been failing since the last Travis system image update and we haven't yet identified the cause. Jenkins runs a more extensive set of tests. I'm afraid we can't give direct access to the Jenkins results, but here's a summary of the failures. We can upload a copy of logs on demand (it's a moderately painful manual process, but it's doable).

Visual Studio builds are failing:
186>C:\builds\ws\3552-merge-H2L6XVY74JN5GHT6NGHZC\worktrees\tmpr8onvcfa\library\ssl_cli.c(516): warning C4267: '=' : conversion from 'size_t' to 'unsigned char', possible loss of data [C:\builds\ws\3552-merge-H2L6XVY74JN5GHT6NGHZC\worktrees\tmpr8onvcfa\cmake_solution\library\mbedtls.vcxproj]
186>C:\builds\ws\3552-merge-H2L6XVY74JN5GHT6NGHZC\worktrees\tmpr8onvcfa\library\ssl_cli.c(517): warning C4267: '=' : conversion from 'size_t' to 'unsigned char', possible loss of data [C:\builds\ws\3552-merge-H2L6XVY74JN5GHT6NGHZC\worktrees\tmpr8onvcfa\cmake_solution\library\mbedtls.vcxproj]
186>C:\builds\ws\3552-merge-H2L6XVY74JN5GHT6NGHZC\worktrees\tmpr8onvcfa\library\ssl_cli.c(553): warning C4267: '=' : conversion from 'size_t' to 'unsigned char', possible loss of data [C:\builds\ws\3552-merge-H2L6XVY74JN5GHT6NGHZC\worktrees\tmpr8onvcfa\cmake_solution\library\mbedtls.vcxproj]
186>C:\builds\ws\3552-merge-H2L6XVY74JN5GHT6NGHZC\worktrees\tmpr8onvcfa\library\ssl_cli.c(554): warning C4267: '=' : conversion from 'size_t' to 'unsigned char', possible loss of data [C:\builds\ws\3552-merge-H2L6XVY74JN5GHT6NGHZC\worktrees\tmpr8onvcfa\cmake_solution\library\mbedtls.vcxproj]

tests/scripts/check_files.py fails

tests/scripts/check-generated-files.sh fails. Run scripts/generate_errors.pl, scripts/generate_features.pl and scripts/generate_query_config.pl.

There are build failures in some non-default configurations. To reproduce:
tests/scripts/all.sh -k build_no_ssl_srv test_asan_remove_peer_certificate

A lot of existing tests are failing. To reproduce:
tests/scripts/all.sh -k test_default_cmake_gcc_asan

or
make CFLAGS='-fsanitize=address,undefined -fno-sanitize-recover=all' LDFLAGS='-fsanitize=address,undefined -fno-sanitize-recover=all'
tests/ssl-opt.sh
tests/compat.sh

At first glance all the failures seem to be due to a memory leak of a 616-byte object from allocated from mbedtls_ssl_parse_certificate in the client but I haven't checked all the logs.

There may be other failures that aren't immediately apparent but will become visible when the build failures and memory leak are fixed.


@azure/msal-angular getProfile() throws error in angular9-sample-app

Library

[x]<EMAIL_ADDRESS>
Description
The current angular9-sample-app throws this error when attempting to view profile information received from the login flow:

core.js:3828 ERROR Error: Uncaught (in promise): InteractionRequiredAuthError: AADSTS65001: The user or administrator has not consented to use the application with ID '6226576d-37e9-49eb-b201-ec1eeb0029b6' named 'msal-angular2'. Send an interactive authorization request for this user and resource.
Trace ID: 57f15927-decc-4ecb-a520-2302241b5900
Correlation ID: 167a98a2-9b60-44b1-801c-1ef993e701c7
Timestamp: 2020-03-20 16:19:52Z
InteractionRequiredAuthError: AADSTS65001: The user or administrator has not consented to use the application with ID '6226576d-37e9-49eb-b201-ec1eeb0029b6' named 'msal-angular2'. Send an interactive authorization request for this user and resource.
Trace ID: 57f15927-decc-4ecb-a520-2302241b5900
Correlation ID: 167a98a2-9b60-44b1-801c-1ef993e701c7
Timestamp: 2020-03-20 16:19:52Z
at InteractionRequiredAuthError.AuthError [as constructor] (AuthError.js:22)
at InteractionRequiredAuthError.ServerError [as constructor] (ServerError.js:22)
at new InteractionRequiredAuthError (InteractionRequiredAuthError.js:24)
at MsalService.push../node_modules/msal/lib-es6/UserAgentApplication.js.UserAgentApplication.saveTokenFromHash (UserAgentApplication.js:1259)
at MsalService.push../node_modules/msal/lib-es6/UserAgentApplication.js.UserAgentApplication.processCallBack (UserAgentApplication.js:814)
at MsalService.push../node_modules/msal/lib-es6/UserAgentApplication.js.UserAgentApplication.handleAuthenticationResponse (UserAgentApplication.js:904)
at MsalService. (UserAgentApplication.js:644)
at step (tslib.es6.js:99)
at Object.next (tslib.es6.js:80)
at fulfilled (tslib.es6.js:70)
at resolvePromise (zone.js:836)
at resolvePromise (zone.js:795)
at zone.js:897
at ZoneDelegate.invokeTask (zone.js:431)
at Object.onInvokeTask (core.js:27323)
at ZoneDelegate.invokeTask (zone.js:430)
at Zone.runTask (zone.js:198)
at drainMicroTaskQueue (zone.js:611)
at ZoneTask.invokeTask (zone.js:517)
at ZoneTask.invoke (zone.js:502)

This error is thrown at this method here:
const GRAPH_ENDPOINT = "https://graph.microsoft.com/v1.0/me";
...
getProfile() {
    this.http
      .get(GRAPH_ENDPOINT)
      .toPromise()
      .then(profile => {
        console.log(profile);
        this.profile = profile;
      });
}

Is there some additional configuration that has to happen in order to access the signed in profile information? Thanks!

What is the difference between the profile at the GRAPH_ENDPOINT, and getting account information via authService: MsalService?
For example:
console.log(this.authService.getAccount().name);
console.log(this.authService.getAccount().userName);
console.log(this.authService.getAccount().accountIdentifier);

What is the difference between the profile at the GRAPH_ENDPOINT, and getting account information via authService: MsalService?
For example:
console.log(this.authService.getAccount().name);
console.log(this.authService.getAccount().userName);
console.log(this.authService.getAccount().accountIdentifier);

The difference is where the information comes from, and what consent do you need to get that. User info from getAccount() comes from AAD itself, which you consent with the profile scope (@jasonnutter to confirm). This is due to AAD - OpenID compliance, and acts as the UserInfo endpoint. On the other hand, profile info from Graph API endpoint uses user.read scope.
I'm looking into replicating your issue.

@CAlex-Wilson I had the same issue as well. It seems the consent scopes are not passed during the login request -likely an issue with Angular factory configuration. Quick dirty solution for now could be to ask for the token with the scope explicitly, like
ngOnInit() {
    this.getProfile();
    this.authService.acquireTokenPopup({scopes: ["user.read"]}); // here
  }

  getProfile() {
    this.http.get(GRAPH_ENDPOINT)
      .toPromise().then(profile => {
        this.profile = profile;
      }).catch(err => {
        console.log(err);
      });;
  }

You can read a bit about AAD consents while this being solved : )

@derisen Okay that makes sense, thank you for the explanation! I added the acquireTokenPopup({scopes: ["user.read"]}) fix above and now it logs the error properly as shown below. I gave the sample app the permissions it requested and it still seems to fail. Is this expected behavior then?
MSAL Logging:  Sat, 21 Mar 2020 00:52:09 GMT:d52be319-fad4-4ac7-8a58-e7a5b9b8747a-1.2.2-Error
Error when acquiring token for scopes: user.read InteractionRequiredAuthError:
The request was denied because one or more scopes requested are unauthorized or expired.
The user must first sign in and grant the client application access to the requested scope.

InteractionRequiredAuthError: The request was denied because one or more scopes requested are unauthorized or expired. The user must first sign in and grant the client application access to the requested scope.
    at InteractionRequiredAuthError.AuthError [as constructor] (http://localhost:4200/vendor.js:77015:28)
    at InteractionRequiredAuthError.ServerError [as constructor] (http://localhost:4200/vendor.js:77505:28)
    at new InteractionRequiredAuthError (http://localhost:4200/vendor.js:77442:28)
    at MsalService.push../node_modules/msal/lib-es6/UserAgentApplication.js.UserAgentApplication.saveTokenFromHash (http://localhost:4200/vendor.js:75484:25)
    at MsalService.push../node_modules/msal/lib-es6/UserAgentApplication.js.UserAgentApplication.processCallBack (http://localhost:4200/vendor.js:75039:29)
    at MsalService.push../node_modules/msal/lib-es6/UserAgentApplication.js.UserAgentApplication.handleAuthenticationResponse (http://localhost:4200/vendor.js:75129:14)
    at MsalService.<anonymous> (http://localhost:4200/vendor.js:74869:34)
    at step (http://localhost:4200/vendor.js:98568:23)
    at Object.next (http://localhost:4200/vendor.js:98549:53)
    at fulfilled (http://localhost:4200/vendor.js:98539:58)

MSAL Logging:  Sat, 21 Mar 2020 00:52:09 GMT:d52be319-fad4-4ac7-8a58-e7a5b9b8747a-1.2.2-Error Error when acquiring token for scopes : user.read ClientAuthError: User cancelled the flow.

core.js:3828 ERROR Error: Uncaught (in promise): ClientAuthError: User cancelled the flow.
ClientAuthError: User cancelled the flow.
    at ClientAuthError.AuthError [as constructor] (AuthError.js:22)
    at new ClientAuthError (ClientAuthError.js:105)
    at Function.push../node_modules/msal/lib-es6/error/ClientAuthError.js.ClientAuthError.createUserCancelledError (ClientAuthError.js:152)
    at WindowUtils.js:35
    at ZoneDelegate.invokeTask (zone.js:431)
    at Object.onInvokeTask (core.js:27323)
    at ZoneDelegate.invokeTask (zone.js:430)
    at Zone.runTask (zone.js:198)
    at ZoneTask.invokeTask (zone.js:513)
    at ZoneTask.invoke (zone.js:502)
    at resolvePromise (zone.js:836)
    at zone.js:901
    at ZoneDelegate.invokeTask (zone.js:431)
    at Object.onInvokeTask (core.js:27323)
    at ZoneDelegate.invokeTask (zone.js:430)
    at Zone.runTask (zone.js:198)
    at drainMicroTaskQueue (zone.js:611)
    at ZoneTask.invokeTask (zone.js:517)
    at ZoneTask.invoke (zone.js:502)
    at timer (zone.js:3041)

sorry, just to confirm, did you restart the application and clean the browser cache?

@derisen

sorry, just to confirm, did you restart the application and clean the browser cache?

That was the issue! Thank you for your help! I can confirm that your temporary fix works as expected.

@CAlex-Wilson cheers! just reopening to track the sample fix

@CAlex-Wilson Can you try replacing the clientId in teh sample with a client that was created in your tenant?

@CAlex-Wilson Can you try replacing the clientId in teh sample with a clientId for an application that was created in your tenant?

I replaced with clientID with mine and it works as expected without the fix provided above.

@CAlex-Wilson Great...the clientId that is provided by default is one we use for testing, and may not work outside the MSFT tenant (it should work with MSA accounts). I'll make sure that this is noted in the samples.

Hi,
I see this was closed but I'm wondering if there was an actual fix to the code or was the profile ngOnInit workaround above the "solution".
The docs seem to indicate that you can prompt the user for MS Graph "User.Read" consent during login but this still does not work for me (following the docs config exactly as-is). The user is only prompted to consent to "View your basic profile" and "Maintain access to data..."
So the question is, can you prompt for MS Graph API permissions at the same time as login? This older post seems to indicate that you cannot but that is from about 1.5 years ago.
My versions:
"@azure/msal-angular": "^1.0.0",
"msal": "^1.3.2",

Thanks in advance.

@jimbarrett33 Please open a new issue, thanks!


[SPARK-39789] Remove unused method and redundant throw exception declare

What changes were proposed in this pull request?
Review code and found some issue about unused method and redundant throw exception declare.
Why are the changes needed?
Eliminate redundant information and make the code cleaner
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.

Can one of the admins verify this patch?

i will close


Do not support routing

In order to help us troubleshoot issues with this project, you must provide the following details:

ElasticDump version
Elasticsearch version
Node.js version
Full command you are having issue with (with inputs and outputs)
A data set you can share publicly which reproduces the problem

It's very important that you take the time to formulate your questions so that we can aid you in a timely manner.
Yes routing is supported. If the data that is dumped contains routing metdata elasticsearch-dump will make use of that field.
Additionally you can use the ---params to add the routing param to the destination index.
NB: This will route all documents to the particular route
There is also the --tranform if you want to add deterministic routing..


Automated Package Upgrade

:tada: This PR is included in version 7.7.100 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Filecoin integration

Coverage decreased (-0.1%) to 40.557% when pulling ef5e2257cf75a7748351dedb3e4e02ac92517b4a on filecoinIntegration into e07c2ccc6c1e95535e76ceb5deca2faf42acbabf on master.


docs(README.md): sqlite storage is now supported in the go SDK

Ref https://github.com/fermyon/spin/pull/1714

[ ] Depends on https://github.com/fermyon/developer/pull/765#issuecomment-1713077854

We probably should wait until we get docs on the developer site after the next release.

I created a PR for the developer site https://github.com/fermyon/developer/pull/883


facilityType-activity equivalence mapping

facility-types.jsonld currently contains a mapping between facilityType and activity. In order for this to be useful for backwards compatibility when dealing with datasets that currently use activity for FacilityUse this mapping needs to be an equivalence. This PR (https://github.com/openactive/facility-types/pull/4) addresses the cases where the mapping are not equivalent.
Going forward it would be helpful to formally define the nature of this mapping to ensure the mapping can be used consistently by all implementers.
For example the activity Tennis should map to Tennis Courts but not to Indoor Sports Hall.
Indoor Sports Hall should not match to any activities as it is not equivalent to any specific activities.
Additionally it would be good to provide guidance for operators that still use systems that use activities to describe a FacilityUse so that they know which activity would result in the expected facilityType.
For example an operator would use this mapping to manually tag a Tennis Court with the activity Tennis to ensure that it appears alongside other facilityType: Tennis Court in searches.

Do you/we know which users still use activities to describe FacilityUse? I can try to get them along to a W3C call to discuss.

I think it's predominantly Legend and Gladstone Booking Systems at the moment

Squash court currently has two activity mappings:

Squash
Racketball

Racketball is not mapped to "Racquetball Court" as Racketball and Racquetball are different sports (with different court sizes, rules, and balls).


Hybrid Encryption Removal

This feature does not come backed into the Rust Crypto suite of algos. It was something was implemented at the C# layer initially. This should be moved into the Rust layer because the marshalling between all three method calls causing some unnecessary overhead.
This can be mimicked in Rust.

The implementation was removed from the SDK and needs to be implemented in Rust.

Closed we are not longer going through with this.

This article has come to my attention.

This was removed and can be closed. HPKE will be implemented in #133


Investigate if we can improve schema Validation Error reporting

We use the jsonschema module to validate messages against our schemas.
At least in the manner we currently utilise it some of the messages for failed validation aren't as useful as they could be.
e.g. if someone attempts to send data containing a key that the schema has tagged 'disallowed', then the reported error is something like:
"[<ValidationError: "{'type': ['array', 'boolean', 'integer', 'number', 'null', 'object', 'string']} is not allowed for 'BadVALUE'">]

which is citing the value of the disallowed key when it would be more useful to cite the name of the key.
Hopefully, certainly in whatever version is current for python 3.9+, we can improve on this.  If needs be we can investigate any alternative modules for validation.

https://python-jsonschema.readthedocs.io/en/stable/errors/
Specifically: https://python-jsonschema.readthedocs.io/en/stable/errors/#jsonschema.exceptions.ValidationError.json_path bit will help improve it.

Looks like useful feedback to pass back to clients, certainly.


Spark Hive tools doesn't work because of pythonpath configuration

I am not able to use Synapse Pyspark interactive because of following error.

In the log, I see below error
Unable to write to Workspace Settings because python.pythonPath is not a registered configuration.
How to resolve this problem?

Hello @sambathraj .sorry for the inconvenience. If you are trying to use Synapse pyspark interactive , please use synapse studio instead, thanks. Since it's dependency will not be maintained anymore, it will also not be maintained. Currently, we are updating doc.

Thanks for the reply. We are trying to use the VSCode for debugging a library code. We were able to use it with a local installation of pyspark.
How to overcome this error? where do we need to set the  python.pythonPath configuration? I tried to setup manually and it doesn't work.
{
"python.pythonPath": "C:\Users\xxxxxxx\.msvscode.hdinsight\hdinsightJupyter\Scripts\python.exe"
}

can you try jupyter extension v2022.1.1001614873 and python extensiton v2021.12.1559732655? thanks

Now that VSCode extension is retiring as mentioned in the doc, what are the plans to use PyCharm/VSCode to develop .py scripts instead of notebooks on Synapse?

@zesluo Can you comment on the question above? Is there an alternative to developing for Synapse if this extension no longer supports that?


scolling

This directive is saving me a great amount of time, for that I thank you.
However I have only one issue/request that I would normally take the time to figure out and implement myself - but I don't have the time.
Is there a simple way when expanding/collapsing panels to get the scroll position to set to the top of the newly expanded pane?
I've got about 5 panes that are all parts of a large medical form (with lots of form fields) and when one section is finished, I want to collapse it - however the scroll position remains at the bottom of the pane.
I hope that make sense. Thanks!

You're welcome.
First, please upgrade vAccordion to the v1.2.1 release. Then add the following code to your controller to get the top position of the currently expanded v-pane element:
$scope.$on('vAccordion:onExpandAnimationEnd', function (event) {
  var paneElement = event.targetScope.paneElement;
  console.log( paneElement[0].offsetTop );
});

I hope this will help.


I want to do an AWESOME Pullrequest - .obj material renderer!

Hi Mate,
I'm in love with your project and I'd like to contribute. I want to add a way to render materials on the object.
Could you give me some advice on what I need and how to get started?
Cheers 🍻

Wow, mate! Your willingness to contribute and add a .obj material rendering feature to this project is seriously impressive. I have been waiting for this feature for so long, and I'm beyond excited to see someone taking the initiative to bring it to life.
Your dedication and passion for the project are truly admirable. It's great to see someone so enthusiastic about contributing to something they love. I can't thank you enough for all the hard work you're willing to put in.
 I'm sure many people, like me, will be over the moon when this feature is finally implemented. Your efforts will not go unnoticed, and you'll be celebrated by many for the amazing work you're doing.
Thank you for being such an inspiration


Science goggle are bugged in Science protolathe

Producing science goggles in the science protolathe (at least on Ministation) produces nothing and takes resources (but it can still produce other things, like power cells).
Reproduction:
1.) Go to a science protolathe with glass and metal.
2.) Insert said glass and metal into protolathe.
3.) Select science goggles and add it to the queue.
4.) Click "Process Queue".
Example(s):

No there. Open issue on unitystation/unitystation


DIfference in the results from HF demo and COlab

Hi,
I am trying this model on Colab and I am seeing difference in the results compared to the results on Huggingfaces demo link.
I tried multiple variants (R101-3x-deforn, R50, R50-3x, R50-3x-deform). Huggingface results seem superior to me w.r.t mask accuracy. I have downloaded the models from the drive linkes in the readme.
Here is my code
config_file = "/content/transfiner/configs/transfiner/mask_rcnn_R_50_FPN_3x.yaml"

cfg = get_cfg()
cfg.merge_from_file(config_file)
cfg.MODEL.WEIGHTS = "/content/transfiner/pretrained_models/output_3x_transfiner_r50.pth"

cfg.MODEL.RETINANET.SCORE_THRESH_TEST = .5
cfg.MODEL.ROI_HEADS.SCORE_THRESH_TEST = .5
cfg.MODEL.PANOPTIC_FPN.COMBINE.INSTANCES_CONFIDENCE_THRESH = .5
cfg.freeze()

im = cv2.imread("/content/img_000124785.jpg")
predictor = DefaultPredictor(cfg)
outputs = predictor(im)
v = Visualizer(im[:, :, ::-1], MetadataCatalog.get(cfg.DATASETS.TRAIN[0]), scale=1.2)
out = v.draw_instance_predictions(outputs["instances"].to("cpu"))
cv2_imshow(out.get_image()[:, :, ::-1])

I see this notice in the logs while I do the prediction which looks sucpicious
The checkpoint state_dict contains keys that are not used by the model:
  roi_heads.mask_pooler.conv_norm_relus_semantic.0.{bias, weight}
  roi_heads.mask_pooler.conv_norm_relus_semantic.2.{bias, weight}
  roi_heads.mask_pooler.conv_norm_relus_semantic.4.{bias, weight}
  roi_heads.mask_pooler.conv_norm_relus_semantic.6.{bias, weight}
  roi_heads.mask_head.deconv_bo.{bias, weight}
  roi_heads.mask_head.predictor_bo.{bias, weight}
  roi_heads.mask_head.encoder.layers.0.self_attn.{in_proj_bias, in_proj_weight}
  roi_heads.mask_head.encoder.layers.0.self_attn.out_proj.{bias, weight}
  roi_heads.mask_head.encoder.layers.0.linear1.{bias, weight}
  roi_heads.mask_head.encoder.layers.0.linear2.{bias, weight}
  roi_heads.mask_head.encoder.layers.0.norm1.{bias, weight}
  roi_heads.mask_head.encoder.layers.0.norm2.{bias, weight}
  roi_heads.mask_head.encoder.layers.1.self_attn.{in_proj_bias, in_proj_weight}
  roi_heads.mask_head.encoder.layers.1.self_attn.out_proj.{bias, weight}
  roi_heads.mask_head.encoder.layers.1.linear1.{bias, weight}
  roi_heads.mask_head.encoder.layers.1.linear2.{bias, weight}
  roi_heads.mask_head.encoder.layers.1.norm1.{bias, weight}
  roi_heads.mask_head.encoder.layers.1.norm2.{bias, weight}
  roi_heads.mask_head.encoder.layers.2.self_attn.{in_proj_bias, in_proj_weight}
  roi_heads.mask_head.encoder.layers.2.self_attn.out_proj.{bias, weight}
  roi_heads.mask_head.encoder.layers.2.linear1.{bias, weight}
  roi_heads.mask_head.encoder.layers.2.linear2.{bias, weight}
  roi_heads.mask_head.encoder.layers.2.norm1.{bias, weight}
  roi_heads.mask_head.encoder.layers.2.norm2.{bias, weight}
  roi_heads.mask_head.encoder.conv_fuse.{bias, weight}
  roi_heads.mask_head.encoder.conv_r1.0.{bias, weight}
  roi_heads.mask_head.encoder.conv_r1.2.{bias, weight}
  roi_heads.mask_head.mask_fcn_uncertain1.{bias, weight}
  roi_heads.mask_head.mask_fcn_uncertain2.{bias, weight}
  roi_heads.mask_head.mask_fcn_uncertain3.{bias, weight}
  roi_heads.mask_head.mask_fcn_uncertain4.{bias, weight}
  roi_heads.mask_head.deconv_uncertain.{bias, weight}
  roi_heads.mask_head.predictor_uncertain.{bias, weight}
  roi_heads.mask_head.predictor_semantic_s.{bias, weight}

could this be the reason for the difference in results. And in general, the accuracy of masks seem to be not good even with the R101-3x-deform model which has highest mAP compared to the results that I am seeing in the huggingface demo.
Any idea what might be happenning here?
TIA.

Thanks.
I installed detectron2 using pip first and then installed transfiner by doing python3 setup.py build develop..
Let me follow the instructions in this repo fully and try again (I remember I had some installation issues on the colab which is why I installed detectron2 separately.). Are you aware of any compatability issues with different detectron2 versions.

It indeed seems to be an issue with some dependency compatability issue.
Earlier I was installing detectron2 first which is installing a bunch of dependecies unlike the instructions in the readme of this repo..
that somehow led to the issue of model not getting loaded properly.
The reason that I was donig that is that the package "omegaconf" is getting installed by the transfiner installation script at a place that is not visible in colab.
Things are working as expected if If I dont' install detectron2 first. And the output masks are much much better :)
I would still like to understand where the problem is exactly, but please feel free to close this issue.

when you install detectron2 first and using "from detectron2 import xxx", the files of original detectron2 package is invoked instead of our modified mask head file.

Hi @lkeab
While the outputs are much better, I still see these logs now while trying to use the demo.py in Colab.
I suppose that this is also not supposed to happen, any idea what might be wrong?
fpn input shapes: {'res2': ShapeSpec(channels=256, height=None, width=None, stride=4), 'res3': ShapeSpec(channels=512, height=None, width=None, stride=8), 'res4': ShapeSpec(channels=1024, height=None, width=None, stride=16), 'res5': ShapeSpec(channels=2048, height=None, width=None, stride=32)}
[06/15 07:28:33 fvcore.common.checkpoint]: [Checkpointer] Loading from ./pretrained_models/output_3x_transfiner_r101_deform.pth ...
WARNING [06/15 07:28:33 fvcore.common.checkpoint]: Some model parameters or buffers are not found in the checkpoint:
roi_heads.mask_head.deconv_bo.{bias, weight}
roi_heads.mask_head.predictor_bo.{bias, weight}
  0% 0/1 [00:00<?, ?it/s]/content/transfiner/detectron2/modeling/roi_heads/fast_rcnn.py:154: UserWarning: This overload of nonzero is deprecated:
	nonzero()
Consider using one of the following signatures instead:
	nonzero(*, bool as_tuple) (Triggered internally at  /pytorch/torch/csrc/utils/python_arg_parser.cpp:882.)
  filter_inds = filter_mask.nonzero()
/usr/local/lib/python3.7/dist-packages/torch/nn/functional.py:3063: UserWarning: Default upsampling behavior when mode=bilinear is changed to align_corners=False since 0.4.0. Please specify align_corners=True if the old behavior is desired. See the documentation of nn.Upsample for details.
  "See the documentation of nn.Upsample for details.".format(mode))

configs/transfiner/mask_rcnn_R_101_FPN_3x_deform.yaml is the config being used.
I have followed the instructions from the readme and the python version is also 3.7. Could this still be because of some installation issue?

I downloaded the latest checkpoint from drive and I don't see these warnings any longer. Thanks.
I was using a checkpoint that I downloaded a couple of months back (April).. I am not sure if something changed or If I downloaded it wrong.


Permit components to validate-as-multiple in single mode.

Overview

The File component's validated submissionValue() returns data that is structured differently from the JSON submission document.
This causes form.io to be unable to display/edit files when shown after saving, if you're persisting the validated/cleaned data.
I resolve this by adding a new method for validate() to use when determining if we need to validate in single or multi-value mode. File forces itself to always be validated as multiple values, but is still presenting to the outside world as a single-value field when configured that way.
Checklist

[ ] tests/ added or updated
[ ] CHANGELOG.md's Unreleased section is updated
[ ] Documentation is updated

Coverage increased (+0.4%) to 81.024% when pulling 8723214c0f1d63f18b6d7f4fd1a7c61261ab322e on invalid-file-submission-data into 3ad37b16f4e2573b1419329ea2922ec8b39192c1 on develop.


Default to files:read scope for Figma provider

Closes #1827 by defaulting to files:read oAuth scope for Figma as per the issue.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 57.033%

Totals

Change from base Build<PHONE_NUMBER>9:
-0.002%

Covered Lines:
9545

Relevant Lines:
16736

💛  - Coveralls

At first glance,  Figma docs don't need updating (couldn't find scope via ctrl-F)


icloud photos not working

Great module...but can't seem to get icloud photos to work.  This is how I read the docs, but it doesn't seem to work.
source:  "icloud: www.icloud.com/sharedalbum/#1234abcd"
Can you post an example of the source line?  Thanks in advance.

It should just be source: "icloud:1234abcd"

Worked like a charm...thank you for the excellent module and support.


fix(material/sidenav): only trap focus when backdrop is enabled

Correct when Sidenav enabled focus trapping. When backdrop is show, trap focus. Do no trap focus when backdrop is not shown.
Existing behavior is that Sidenav traps focus whenever it is not in side mode. This causes the end user to not be able to interact with the sidenav content when the mode is push/over, backdrop is disabled and using ConfigurableFocusTrapFactory (#26572).
With this commit applied, Sidenav always traps focus when backdrop is shown. Sidenav never traps focus when backdrop is not shown, regardless of what mode the sidenav is in, focus trapping will respect if the backdrop is shown or not shown.
Fix this issue by correcting boolean logic for detecting if backdrop is enabled and using that logic to determine when to trap focus. Add an example that injects ConfigurableFocusTrapFactory.
Fix #26572

Since this was last reviewed: fixed issue with showing backdrop when there are both a start and end sidenav. Add _drawerHasBackdrop method to simplify logic.


editor shift+click not working

working in editor but not MacOS; assumed to be an integration issue

dupe: https://github.com/lockbook/lockbook/issues/1665


Replace ErrNotFound with boolean return value

Removes ErrNotFound and replaces its use with an additional boolean return value from Find, FindRaw and Delete.
Depending on your view of whether ErrNotFound is a true error, this changes the semantics of Delete to make it not an "error" to attempt to delete a missing key.
Closes #82.

I'm going to merge without further changes, but

Set() and SetRaw() could do with the same treatment for a consistent API

You mean returning a bool indicating if the collection changed? Hmm. We did add SetIfAbsent with such an output, though of course slightly different semantics. I think part of the motivation there was to avoid changing the widely-used and simple Set() API.

I'm going to merge without further changes, but

Set() and SetRaw() could do with the same treatment for a consistent API

You mean returning a bool indicating if the collection changed? Hmm. We did add SetIfAbsent with such an output, though of course slightly different semantics. I think part of the motivation there was to avoid changing the widely-used and simple Set() API.


Common Gateway Interface issue on azure app service, spring cloud gateway

While performing load testing on a spring cloud gateway I am getting a request failure rate of about 30% - 40% and 0% on local and the error I get on each failed request is "The Specified CGI application encountered an error and the server terminated the process."
Requests count 2500
Ramp up period 5s
App service plan : P1V2
I have tried the following things:

Increasing Heap Memory
By changing underlying web server to appache Tom cat instead of reactor netty  and changing max thread count.
Increasing SSL handshake timeout
Upgrading Spring boot version to 3.0.2 & Spring cloud version to 2022.01.1
The services which are being called are working fine on the same plan and can handle the load

Hi, @Mohit00021
Thanks for reaching out.
Here is my current understanding:

You are running your app in Azure App Service.
Your system has a module develop by yourself with spring-cloud-gateway.
Your system has no relationship with Spring Cloud Azure and Azure Spring Apps.

Correct me if my understanding is not right.

Yes, @chenrujun your understanding is correct am I at the wrong place to ask this?

If the problem exists in your localhost, you can create an issue here: https://github.com/spring-cloud/spring-cloud-gateway

Closing this issue. Because there is no update for more than 2 weeks.


Running hackjohn from cron

Hi there--very cool project.  I can run hackjohn from the shell, but not from cron--when it runs from cron it does not seem to be able to successfully load the anaconda environment.  Do you mind sharing more details on how you managed to configure it with cron?
I have created a shell script to be executed by cron.  I have tried every possible permutation of:
cd /Users/friedman/hackjohn
conda activate hackjohn  or source /anaconda2/bin/activate hackjohn
/anaconda2/envs/hackjohn/bin/python hackjohn.py or python hackjohn.py or
hackjohn.py (with the proper shebang line at the top)
Could you share insight about how you got cron to properly activate the anaconda environment and use the proper version of python? thanks so much:)

Hey @ndfriedman thanks for your interest and sorry to hear about your cron environment issues.
What worked for me was to include a shebang as the first line of hackjohn.py like:
#!/home/dhimmel/anaconda3/envs/hackjohn/bin/python

This shebang should point to the python binary installed in the conda environment. Can you check that if you run the path to the python binary in a terminal that it opens python?
For example, when I run /home/dhimmel/anaconda3/envs/hackjohn/bin/python in my terminal, a Python session launches like:
Python 3.6.6 |Anaconda, Inc.| (default, Jun 28 2018, 17:14:51)
[GCC 7.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.


Benchmark

Setup Peloton on MemSQL machine

Micro benchmark
Without logging
0.5 1 10 16328.4

Hi, I ran ycsb benchmark in my branch and it works fine. probably I have fixed certain bugs when refactoring the code :-) the throughput I got is much higher (2-3x) than yours. I ran the experiments on a virtual machine. I will test the scalability later.

@yingjunwu I pulled in your mvcc-old-to-new branch this afternoon and re-ran the benchmark:

@eric-haibin-lin my branch is mvcc-new-to-old... the GC is disabled by default. You may enable it by change the parameter in line: https://github.com/yingjunwu/peloton/blob/mvcc-new-to-old/src/backend/gc/gc_manager_factory.h#L22 to GC_TYPE_VACUUM. Plus, you need to enlarge queue size to, say, 100,000 set in line: https://github.com/yingjunwu/peloton/blob/mvcc-new-to-old/src/backend/gc/gc_manager.h#L31and L32.
In fact, the single-thread GC does not work well. My new YCSB can report the throughput for every 0.1 seconds. You can see how the GC influence the system throughput.

@eric-haibin-lin I highly suggest you to check the throughput log recorded in outputfile.summary. GC is very important in multi-versioned systems. Hyrise's design, which is implemented in Joy's old branch, does not implements GC. Plus, it does not distinguish primary index from secondary indexes. In addition, its measurement scheme implemented in the old-version YCSB benchmark renders the reported numbers meaningless.

50% Update with garbage collection ON.  Before tuning flush frequency.

Not sure whether you have merged my PR to the master branch at cmudb, but the result seems to be reasonable, as we found that there might be some bottlenecks on the storage layer.

Btw, what's the scale factor?

I only merged with the new-to-old branch, and turned on GC as you mentioned previously. I haven't merged with cooperative GC branch yet. The scale factor is 1. What scale do you usually test with?

Reasonable. Now you can try to merge with my mvcc-cooperative-gc branch and see what will happen. You can set the scale factor to 10 or even larger.


Can the GPU simulator simulate different GPU interconnect topologies such as RING and ALL-TO-ALL?

If so, please guide me on how to set up these parameters in the simulation.

You can check the mesh connector implementation here https://github.com/sarchlab/akita/blob/v3/noc/networking/mesh/mesh.go.
Creating a ring topology is basically creating a ring connector like the mesh connector, but it will be much simpler.


feat(html-content): innerHTML property

Alternative for pull requests #63 and #64,
and admittedly this one is my favorite implementation.

What kind of change does this PR introduce?
Close: https://github.com/akserg/ng2-toasty/issues/3

What is the current behavior? (You can also link to an open issue here)
Not supporting 'html' content in toasts

What is the new behavior (if this is a feature change)?
Supporting ability to projecting 'html' templates in toasts

Other information:
Testings using this branch:
ng2-webpack-demo/toasty-html-content

Using that code:
options = {
 title: 'HTML title: Toast It!',
 msg: 'HTML message: Mmmm, tasties...'
};

this.toastyService.default(toastOptions);

Gives the result:

I would happy to review and discussing

Nice, any chance to have this pulled in master?

Hi Gil,
That solution much better than in pull requests #63 and #64.
Thank you for your valuable input.

@akserg thanks :)


Support Promise v3

We should support Promise v3 for Async v2, v3 and v4. I don't think implementing this here requires a lot of work, the only outstanding feature that would be debatable is cancellation semantics as discussed in #42.
I'm not currently working on this, but figured it makes sense to report here in order to track this feature request. In case anybody feels like picking this up, any input and PRs would be much appreciated :+1:

Closed via #46, #47 and #48 :shipit:


Remove some copies from the queue binding implementation

Remove some copies from the queue binding's send() method
By not copying data out of perfectly good Strings into new heap arrays
when we don't need to.
Remove a copy when calling queue.sendBatch with "bytes" content
Because we synchronously consume the arraybuffer in order to base64-
encode it further down in sendBatch, there's no point in making a full
copy of it.
In theory we could just always call BufferSource::detach() to avoid
ever needing a copy, but I'm a little worried about the unintuitive
behavior of an arraybuffer being emptied out whenever you pass it into a
queue method. While it'd be really nice for performance, I'd hate to see
it cause a serious customer bug.

The latest force-push was just to rebase on main after two weeks of changes had built up. The only actual code change was the new commit to use detach().
There's no other changes needed here as far as I know, so if someone approves then I'll plan on merging it once it's green.


Convert Class Files to Jasm

Please create a tool to convert .class files into .jasm files.  This will make reading .class files much easier than the output from javap.

I wonder if having a disassembler would allow using jasm for patching Android apps. Given you can already convert dex files to jar (with the conveniently named dex2jar tool), this might open the door for an alternative to patching smali code (via baksmali).

I wonder if having a disassembler would allow using jasm for patching Android apps. Given you can already convert dex files to jar (with the conveniently named dex2jar tool), this might open the door for an alternative to patching smali code (via baksmali).

dex2jar is a very old, kinda unmaintained tool with no support for newest DEX versions, translating debug info, handling weird edge cases etc. (see here) So I would NOT see it as a valid replacement/alternative to baksmali/smali.
It's still good for some cases obviously, but I wouldn't use it for a full roundtrip (dex -> jar -> dex) on an average app and expect it to work.

This is now available in develop. To disassemble a .class file src/com/example/MyClass.class to the src directory with the command-line tool, you can now do e.g:
jasm -d -i classes -o src com/example/MyClass.class
Scheduled for 0.6.0 release this weekend.

@roscopeco Awesome!  Thank you!


Submit MASSpy 0.1.6

id: masspy
version: "0.1.6"
specificationsUrl: https://raw.githubusercontent.com/biosimulators/Biosimulators_MASSpy/1b88c8d252035cc5aab63013d6d17407827d9e03/biosimulators.json
specificationsPatch:
version: "0.1.6"
image:
url: ghcr.io/biosimulators/biosimulators_masspy/masspy:0.1.6
digest: "sha256:651a5591e74a3d828c71435a2819fd728a1f2d0cf7345bd5729b694240ee2f6d"
validateImage: true
commitSimulator: true

Changelog

Rerun with Singularity 3.9,5

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

The specifications of your simulator is valid!

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

The specifications of your simulator is valid!

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

The specifications of your simulator is valid!

Will be superseded by 0.0.9 which has much better performance due to requiring libRoadRunner<2.2.0 and numpy<1.22.


Yank broken CPUSummary version

@DilumAluthge

Looks like it needs a rebase?

Looks like it needs a rebase?

Done


Fix the embedding size lookup in Categorify op

When columns are jointly encoded, the embedding size needs to be looked up by the shared category name, instead of the column name.

rerun tests

rerun tests


zones/steal re-execution of main only works for ES modules

This issue is to circular back and fix it once the demo app is working.

This needs to be fixed to make production work.


There are some error when i run "/examples/openasr2021/sigapo/local/dump_wav.sh"

Describe the bug
The audio file is not executable
So I modified the code as follows:
for slice in ls $logdir/wav_*.slice; do
{
name=basename -s .slice $slice
mkdir -p ${data}/wavs/${name}
cat ${slice} | awk -F ',' -v sox=$sox -v data=pwd/$data/wavs/$name
-v logdir=$logdir -v name=$name '{  during=$4-$3 ;
cmd=""  sox  " "$2 " " data "/" $1 ".wav" " trim " $3 " " during;
system(cmd)
printf("%s %s/%s.wav\n", $1, data, $1);
}' |
sort > ${data}/wavs_${name}.scp || exit 1;
} &
done
wait

nothing to say


fixed cleanup order to uninstall operator before cleaning sa to fix intermittent failures

https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/2108/

Run the entire suites on Jenkin and post the result.


[bugfix] Fix documentation of sanity functions accepting a filename

Fixes #1646.

Codecov Report

Merging #1666 (f849866) into master (01d7efc) will decrease coverage by 0.29%.
The diff coverage is 45.28%.

@@            Coverage Diff             @@
##           master    #1666      +/-   ##
==========================================
- Coverage   87.54%   87.24%   -0.30%
==========================================
  Files          45       45
  Lines        7492     7545      +53
==========================================
+ Hits         6559     6583      +24
- Misses        933      962      +29

Impacted Files
Coverage Δ

reframe/utility/sanity.py
97.45% <ø> (ø)

reframe/core/modules.py
56.62% <45.28%> (-1.21%)
: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 7ddadba...f849866. Read the comment docs.


Client::Handler() API: only .body(), or .body() and .end()?

There has been some discussion about whether...
(Current state, pseudocode)
Client::Handler()
  .body([](auto&& response) {
    // Will be called for every response AND one more time when the stream terminates.
    // Have to check whether `response` is valid here. Invalid response = terminated stream.
  });

... is confusing. The discussed alternative is something like...
(Alternative 1, pseudocode)
Client::Handler()
  .body([](auto&& response) {
    // Called for every response (only); `response` is always valid.
  })
  .end([]() {
    // Handle the end of the stream, if needed.
  });

@benh can share some insight as to why we picked the current state. After that we can discuss whether we now prefer an alternative.

Thanks @rjhuijsman for this.
The semantics of stream in stout-eventuals does split up the EOF signal into a separate ended() callback (see below ^ for why this was called ended() but not end()), but the semantics of ended() are that you should not do anything else with the stream at that point, and abstractions like Loop() make sure to not pass you a reference to the stream so you don't make that error.
With gRPC however the "stream" from the server (or client) call might be ended, but that doesn't mean that you can't still use the call to send more requests (or responses). It all depends on the underlying semantics/expectations of the gRPC method. IMHO the real issue is gRPC is based on a "channel" abstraction not a "stream", and stout-eventuals doesn't have something like that built-in (yet).
So the semantics were going to be a bit different no matter what: either we'd have an ended() where you still might use the reference to the call or we'd have a different body(). I ended up deciding to change the semantics of body() because:

We already had some special semantics  in body() regarding the need to relinquish the borrowed_ptr in order for us to read the next request/response from the channel. It seemed nicer to lump all the things that needed to be "taught/learned" in one place.
Programmers familiar with gRPC APIs already follow a pattern where you they specifically have to check for EOF (or failure) when reading requests/responses:

auto state = ...;
while (true) {
  if (reader->Read(&response)) {
    // ... update 'state' based on response (and possibly generate a new request).
    call.Write(request);
  } else { // .. handle failure or EOF.
    // ... use 'state' to possibly send a final request and finish.
    call.WriteLast(request);
    call.Finish(status);
}

Extrapolating this gives us a body() that closely resembles this pattern:
Client::Handler()
  .body([state = ...](auto& call, auto&& response) {
    if (response) {
      // .. update 'state' based on 'response' and (possibly generate a new request).
      call.Write(request);
    } else { // .. handle failure or EOF.
      // ... use 'state' to possibly send a final request and finish.
      call.Write(request);
      call.Finish(status);
    }
  });

The ended() version isn't that far off from the while loop version, but because they are in different scopes you do have more work to do, in particular you need to use context() rather than lambda captures:
Client::Handler()
  .context(State{})
  .body([](auto& state, auto& call, auto&& response) {
    // .. update 'state' based on 'response' and (possibly generate a new request).
    call.Write(request);
  })
  .ended([](auto& state, auto& call) { // .. handle failure or EOF.
    // ... use 'state' to possibly send a final request and finish.
    call.Write(request);
    call.Finish(status);
  });

Ok, hopefully that's some helpful history, let's discuss!
^ stout-eventuals used ended() not end() because in the original design there were "helper" functions to invoke each of the callbacks such as start() and next(), and body() and some compilers didn't like end() because it's apparently a builtin somewhere? That being said, the current design doesn't have these helpers anymore in lieu of just calling the function on the continuation directly, i.e., k.Ended() so we could now go and fix up the tense to be consistent!

Thanks Ben!
Just to make sure that we have all the design alternatives on the table, is there also a possibility that we'd handle failure separately from normal termination? For example:
Client::Handler()
  .context(State{})
  .body([](auto& state, auto& call, auto&& response) {
    // .. update 'state' based on 'response' and (possibly generate a new request).
    call.Write(request);
  })
  .ended([](auto& state, auto& call) { // Handle EOF.
    // ... use 'state' to possibly send a final request and finish.
    call.Write(request);
    call.Finish(status);
  })
  .failed([](auto& state, auto& call) { // Handle failure.
    // ...
  });

As for tenses, I quite like the past tense - it makes it clearer to me I'm not giving an order but handling a state. In fact, I might lean into that: instead of...
.body([](..., auto&& response) { ... });

... (which I find a slightly confusing name), I'd consider ...
.responded([](..., auto&& response) { ... });

Closing this issue after recent refactorings.


segfault with mutli-thread since the port ptr become null

use the PR https://github.com/intel/DML/pull/18
performance test to run multi-thread with the following command
./examples/dml_example_c_api_perftest 128 10000 4096 0 16
allocatate the 4096 aligned src=0x7f3a7a388000, dst=0x7f3a79b86000
jobs=0x7f3a7c82f010
jobs=0x7f3a7c83ac10
jobs=0x7f3a7c846810
jobs=0x7f3a7c852410
jobs=0x7f3a7c85e010
jobs=0x7f3a7c869c10
jobs=0x7f3a7c875810
jobs=0x7f3a7c881410
jobs=0x7f3a7c88d010
jobs=0x7f3a7c898c10
Starting example for multi-job memory move jobs=0x7f3a7c846810:
jobs=0x7f3a7c8a4810
Starting example for multi-job memory move jobs=0x7f3a7c83ac10:
Starting example for multi-job memory move jobs=0x7f3a7c82f010:
jobs=0x7f3a7c8b0410
jobs=0x7f3a7c8bc010
Starting example for multi-job memory move jobs=0x7f3a7c869c10:
Starting example for multi-job memory move jobs=0x7f3a7c85e010:
jobs=0x7f3a7c8c7c10
Starting example for multi-job memory move jobs=0x7f3a7c852410:
jobs=0x7f3a7c8d3810
Starting example for multi-job memory move jobs=0x7f3a7c8c7c10:
jobs=0x7f3a7c8df410
Starting example for multi-job memory move jobs=0x7f3a7c8d3810:
Starting example for multi-job memory move jobs=0x7f3a7c898c10:
Starting example for multi-job memory move jobs=0x7f3a7c8b0410:
Starting example for multi-job memory move jobs=0x7f3a7c8df410:
Starting example for multi-job memory move jobs=0x7f3a7c881410:
Starting example for multi-job memory move jobs=0x7f3a7c88d010:
Starting example for multi-job memory move jobs=0x7f3a7c8a4810:
Starting example for multi-job memory move jobs=0x7f3a7c8bc010:
Starting example for multi-job memory move jobs=0x7f3a7c875810:
Segmentation fault (core dumped)
using gdb we will find that the port ptr is null:
Thread 13 "dml_example_c_a" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fffef1f4700 (LWP 123148)]
0x0000000000422057 in dml::core::dispatcher::hw_queue::enqueue_descriptor (this=0x62dc38 dml::core::dispatcher::instance+56, desc_ptr=0x7ffff7fac4c0) at /home/dennis/DML/sources/core/src/hw_dispatcher/hw_queue.cpp:92
92                           : "a"(current_place_ptr), "d"(desc_ptr));
Missing separate debuginfos, use: yum debuginfo-install libgcc-8.5.0-4.el8_5.x86_64 libpmem-1.6.1-1.el8.x86_64 libstdc++-8.5.0-4.el8_5.x86_64 libuuid-2.32.1-28.el8.x86_64
(gdb) p current_place_ptr
$1 = (void *) 0x0
(gdb)
From the logic, the port ptr can't be null since the portal_mask_never change, but in this case the portal_mask_ changed to zero, so suspect some data overflow overwrite these data can cause the issue.

@guoanwu Will apply this PR: https://github.com/intel/DML/pull/15  for a test?

Hi @guoanwu, this should be addressed with https://github.com/intel/DML/commit/e44443c24d53552b248b9869b1b16f89cd970f52, please let me know if you could verify or if I could close the issue.

Verified that it works correctly now, so closing the issue.


Either and EitherStrict Generic Parameter ordering

First off, great library. Thank you for publishing this. My only concern is with the ordering of the generic parameter ordering for Either<R, L> and EitherStrict<R, L>. Visually, it's confusing to see the left argument of the generic be the right and vise versa. It seems to make the following very counter intuitive:
Either.Right<string, bool>(() => "Hello!");

From a purely directional standpoint I think it might make more sense to reverse their ordering and use <L, R>. That way
Either.Right<string, bool>(() => false);

Refers, not only to the right generic argument, but also the right monad.

I have finally come around to this thinking, I always had in my mind that the 'subject' of the monad would be first in the list of generic arguments.  On my other library on here (language-ext), I did the same.
It previously felt odd having this:
    Either<Error,int>

But I am finally convinced that actually for readability sake you do want the Right on the right and the Left on the left.
The changes are checked in, I will do a new release and nuget package soon.  Thanks for the feedback :)

Awesome @louthy, thanks a lot!


Add cross compilation stuff

I was chatting with @retep998 the other day about wanting to get "how to cross compile to windows" stuff going. They were kind enough to send me a gist. Here it is.
This is very obviously still a skeleton, but that's what the forge is for! Maybe we can get people excited to fill it out.

I really hope someone on non-Windows can test out those instructions and provide corrections and clarifications.


[animations] minimum size option for scale

Problem
When windows scale onto the screen when at the start of the animation they are really small they look pixialted and bad
Solution
Is it possible to have an optional minimum scale option for the scale animation eg:
animation=windows,1,5,scale,70% # <- last value is minimum scale option

This would mean that windows start there scale animation at 70% so the animation looks more like it does on gnome

done in 64e80991ca335c4ef7da736bb5582747caca284a
really cool effect, ngl.
how 2 use (example):
animation=windows,1,8,default,popin 80%


Disable ext4 journaling to extend life of drives

Turning off ext4 journaling can extend the life of flash memory, especially cheaper SD cards and eMMC modules. Unfortunately, this option cannot be toggled when a system is booted with the rootfs read write. It would be nice if the ext4 journal was disabled when creating the image. This can be accomplished with tune2fs:
# tune2fs -l /dev/mmcblk1p7 | grep 'features'
Filesystem features:      has_journal ext_attr resize_inode dir_index filetype needs_recovery extent 64bit flex_bg sparse_super large_file huge_file dir_nlink extra_isize metadata_csum
# tune2fs -O ^has_journal /dev/mmcblk1p7
# tune2fs -l /dev/mmcblk1p7 | grep 'features'
Filesystem features:      ext_attr resize_inode dir_index filetype extent 64bit flex_bg sparse_super large_file huge_file dir_nlink extra_isize metadata_csum

Had some bad luck with this -- will reopen if I get it working.


Fix drop_duplicates for multi-index columns support.

Codecov Report

Merging #856 into master will decrease coverage by 2.17%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #856      +/-   ##
==========================================
- Coverage   94.22%   92.04%   -2.18%
==========================================
  Files          34       34
  Lines        6147     6150       +3
==========================================
- Hits         5792     5661     -131
- Misses        355      489     +134

Impacted Files
Coverage Δ

databricks/koalas/frame.py
94.51% <100%> (-1.59%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.54% <0%> (-72.73%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
77.5% <0%> (-7.5%)
:arrow_down:

databricks/conftest.py
90.69% <0%> (-6.98%)
:arrow_down:

databricks/koalas/namespace.py
85.01% <0%> (-1.31%)
:arrow_down:

databricks/koalas/plot.py
93.09% <0%> (-0.96%)
:arrow_down:

databricks/koalas/utils.py
97.24% <0%> (-0.69%)
:arrow_down:

databricks/koalas/testing/utils.py
77.35% <0%> (-0.63%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


ADD LUSD to OP V3 - Review

Please consider AIP.md PR : https://github.com/aave/aip/pull/356
-Proposal Payload
-deployed IR strategy
-payload & proposal creation scripts
-updated MakeFile
-Test & generated diff report

PR deprecated in favor of https://github.com/bgd-labs/aave-proposals/pull/107 using the new listing engine


Stuck sending heartbeat

With this autoscale event…

the following lifecycled events happened…

and then the heartbeat just kept going, instead of going through to https://github.com/lox/lifecycled/blob/master/daemon.go#L87
This could be a bug in…
https://github.com/buildkite/buildkite-aws-stack/blob/4b653d0cb9e4941816baeeee2bad8367c421ed0e/packer/conf/buildkite-agent/scripts/buildkite-agent-lifecycled-handler
https://github.com/buildkite/buildkite-aws-stack/blob/4b653d0cb9e4941816baeeee2bad8367c421ed0e/templates/buildkite-elastic.yml#L310

I've changed it to https://github.com/buildkite/buildkite-aws-stack/commit/26280084620ba78029c4eea23d4b7484b54abdd2 for now to see if it fixes it…


[issue] adding domains fail due to colon

I was adding some domains and I had this error:
[ERROR] too many values to unpack (expected 2)
I discovered it was because of a IPv6 (therefore the : symbol)
Doing this solved the issue:
cat domains.txt | grep -v ":" | bbrf domain add - -s subfinder --show-new

Hi, I think the same thing should be implemented for bbrf domain update as well. For now ipv6 works fine in bbrf domain add only.


[5.2] Fix key collisions when adding closures to global scopes

Fix for #12530.
Unless @acasar can think of a better solution :)

@vlakoff Thanks for taking a look into this. :+1:
I was maybe thinking to remove all random stuff an let php choose the next free index:
return static::$globalScopes[get_called_class()][] = $scope;

What do you think?

I thought about this too. But I feared collision with some named scope 0.
A maybe better implementation would be a stack, not only for closures but also named scopes. It would still require a map, allowing user to get a registered scope by its name.
I thought about this so the scopes would be executed in the order they have been registered.

Updated PR using the great suggestion of @GrahamCampbell.

:+1:


Average engineering effort to deploy a new language + total length of time

User Story
As a Scrum Master and Product Owner,
I want to determine on average how many engineering story points it takes ship a new language FTM and the total length of time across the entire process by completing 3 new languages, Arabic, Brazilian Portuguese, and Punjabi,
So that I can more accurately forecast how long it would take to complete all 47+ remaining language builds.

Outcome: Automation efforts allowed us to go from 2 days per language to a few hours.


Flakey: node::tests::test_node_add_taggded_blob_event

https://github.com/n0-computer/iroh/actions/runs/9266460538/job/25492087993?pr=2330

Likely only flaky because the timeout is too low. Fixed by #2450.


Feature/nxl<PHONE_NUMBER> sidebar grid

Monday.com Item
Monday Item ID:<PHONE_NUMBER>
Description
Made gridded selection component with custom callback, columns and width constrained by its container.

How Has This Been Tested?
In storybook, use npm run storybook

Screenshots/GIF (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Bypassing review, approving, squashing and merging.


when will mono build be used in actual production

About mono build, when will it be used in actual production, or when will the alpha phase end.

There is no ETA for the Mono build being marked as production-ready, but the main missing features such as iOS export and AOT compilation are expected to be added in Godot 4.0.

There's no ETA for the Mono build being marked as production-ready, but the main missing features such as iOS export and AOT compilation are expected to be added in Godot 4.0.

Thank you for your reply, aside from platform support and feature support, I would like to know what other specific reasons for "cannot be used for actual production",  if there is any, such as unstable and major changes at any time.If these problems do not exist, I think it can be used for actual production for me.

Due to the Vulkan merge, followed by recent core refactorings including the OS/DisplayServer split, the master build is currently unstable. Other than the main missing features, these seem to be the current things that need to be stabilized

The information dialog about it being alpha will be removed in 4.0. It's already quite stable as of 3.2, but there are a lot of breaking changes incoming for C# in 4.0, so I decided not to remove it until then.


Use Vectorized Partition Kernel for Window Frame Calculation

Is your feature request related to a problem or challenge?
BuiltInWindowExpr currently makes use of the vectorised partition function when evaluating window frames with ranks, but falls back to a very inefficient mechanism based on performing scans using Vec<ScalarValue> as a comparison key for window functions that rely on the window frame, including user defined window functions.
Describe the solution you'd like
I may be missing something but it looks like window functions are only evaluated on data that is already sorted, as such it should be possible to identify the partition boundaries in a single pass, and then use this information to efficiently extract the window boundaries.
Tagging @mustafasrepo and @alamb who may have more context on this design
Describe alternatives you've considered
No response
Additional context
#7517 tracks remove ScalarValue's implementation of Ord to cleanup the code and discourage this sort of approach in future

When a window function with rank is used.  A single pass on the whole data is sufficient. What we need is finding partition boundaries in the data. However, for window frames (with groups, and range) for each row we find the preceding and following boundary (corresponding to each row, determined by values of the row.) Hence this calculation should be done for each row(storing last boundaries found, helps in decreasing search space) on the subset of the table(around row of the interest.)
I am not sure whether window frame boundary search can be vectorized. That being said, I guess instead of Vec<ScalarValue> we can use Row type.

I may be missing something but it looks like window functions are only evaluated on data that is already sorted, as such it should be possible to identify the partition boundaries in a single pass, and then use this information to efficiently extract the window boundaries.

The window functions are indeed always evaluated on data that is sorted.
We might be able to operate directly on Array for windows that are defined as a row size, and as @mustafasrepo  suggests operate on Rows for windows defined by value


 Using it delicately.
Git branches should describe what this feature(or fix) is for by their name.
There is "bible" rule for git branch naimg.
feature/something-is-new
fix/something-is-fixed
You can practice how to co-work on same git repository.
Comments

When I ask to Chat-GPT like this please recommend good git branch naming rules for web product

Naming conventions for Git branches in web product development can greatly improve the organization and clarity of your repository. Here are some recommended rules for naming branches:

Feature Branches: When working on a new feature, use the prefix "feature/" followed by a descriptive name for the feature. For example, feature/user-registration or feature/dark-mode.

Bugfix Branches: If you're fixing a bug, use the prefix "bugfix/" followed by a brief description of the bug. For example, bugfix/login-validation or bugfix/404-page.

Hotfix Branches: Hotfix branches are used for urgent fixes to the production environment. Use the prefix "hotfix/" followed by a brief description of the issue. For example, hotfix/security-issue or hotfix/payment-error.

Release Branches: When preparing for a release, create a branch with the prefix "release/" followed by the version number. For example, release/1.0.0 or release/2.1.3.

Refactor Branches: If you're performing significant refactoring, use the prefix "refactor/" followed by a brief description of the refactoring task. For example, refactor/navigation-menu or refactor/auth-service.

Merge or Integration Branches: Use the prefix "merge/" or "integration/" when combining multiple feature branches or preparing for a larger integration. For example, merge/feature-branch-1 or integration/staging-to-master.

Documentation Branches: When updating or adding documentation, use the prefix "docs/" followed by a brief description. For example, docs/readme-update or docs/user-guide.

Experimental Branches: For experimental features or ideas that may not make it to production, use the prefix "experiment/" followed by a description. For example, experiment/new-search-algorithm or experiment/reactive-ui.

Temporary Branches: For temporary work that may not be ready for a commit or push, use the prefix "tmp/" followed by a brief description. For example, tmp/fix-broken-tests or tmp/style-tweaks.

Remember to keep the branch names descriptive and concise to help team members easily understand the purpose of each branch. Consistency in branch naming can make it easier to manage and track the different branches throughout the development process. Additionally, encourage team members to follow these conventions to maintain a clean and organized Git history.

This is a great idea.
We will discuss this on our next meeting on Aug 8th and apply this convention from then :)
Thanks for the suggestion!


Unsupported ltm node

Environment

AS3 converter: 1.11.0-RC1 used as service with "serve" flag

Summary
When passing ltm node to ACC converter, I got this as unsupported-  I figured it out by altering the code and forcing printing unsupported obejcts to output, because of the first issue
Steps To Reproduce
Steps to reproduce the behavior:
bigip.conf.zip
Convert attached bigip.conf to observe the issue
docker run --rm -v "$PWD":/app/data f5-as3-config-converter:1.11.0-RC1  -o test.json -c /data/bigip.conf --unsupported
Expected Behavior
"ltm node" should not be classified as unsupported

This was resolved in ACC 1.12


add claims to clusteroauth

Signed-off-by: Chris Ahl<EMAIL_ADDRESS>

/hold
wait for #64 is merged then I can rebuild image and test


Crash Report

So I wanted to play the game with some mods on, leading me to download block launcher, as soon as I opened the app, it crashed, I'm using a Samsung Tab 4 and I was wondering if this device was incompatible and if it could be fixed.

Crashed for no reason i uninstalled and reinstalled 15 times and nothing happend 😣 i hope this gets fixes

Ps im usin :  a lg samsung tab


Problem with mouse events handlers in Firefox

Resize and drag doesn't work properly in Firefox, but samples on https://frappe.io/gantt work perfectly.

Frappe Gantt version 0.3.0
Firefox 75.0

I've repaired it by replacement e.offsetX/Y to e.layerX/Y in mouse event handlers. And with switch-off of animateSVG for Mozilla Firefox.

Hi! this is what I'm looking for. Where did you change offset to layer? exerywhere it occurs? does this work in the other Browsers as well? and how do you "switch-off" animateSVG? could you please tell me more?

Hi! It works perfectly for me. However, I've tested it only on Chrome and Firefox.
I've made 4 changes in class Gantt
2 in bind_bar_events():
            $.on(this.$svg, 'mousedown', '.bar-wrapper, .handle', (e, element) => {
                ...
                x_on_start = e.layerX;
                y_on_start = e.layerY;

            $.on(this.$svg, 'mousemove', e => {
                if (!action_in_progress()) return;
                const dx = e.layerX - x_on_start;
                const dy = e.layerY - y_on_start;

2 in bind_bar_progress():
            $.on(this.$svg, 'mousedown', '.handle.progress', (e, handle) => {
                is_resizing = true;
                x_on_start = e.layerX;
                y_on_start = e.layerY;

            $.on(this.$svg, 'mousemove', e => {
                if (!is_resizing) return;
                let dx = e.layerX - x_on_start;
                let dy = e.layerY - y_on_start;

And 2 in class Bar
            if (navigator.userAgent.indexOf("Firefox") === -1) {
                animateSVG(this.$bar, 'width', 0, this.width);
            }

            if (navigator.userAgent.indexOf("Firefox") === -1) {
                animateSVG(this.$bar_progress, 'width', 0, this.progress_width);
            }

Thanks @RuslanGabbasov, very helpful!


Vulnerability roundup 99: fastjson-0.99.9: 1 advisory [9.8]

search, files

[ ] CVE-2017-18349 CVSSv3=9.8 (nixos-unstable)

Scanned versions: nixos-unstable: 64b46178838.
Cc @nequissimus

false positive


kgsearch: can not find data with ids

After resolving #764, get another problem.
As we can see, freebase dump data list many object in the first column.

When requesting the object with ids, the response contains empty results.

{u'@context': {u'goog': u'http://schema.googleapis.com/', u'kg': u'http://g.co/kg', u'@vocab': u'http://schema.org/', u'EntitySearchResult': u'goog:EntitySearchResult', u'detailedDescription': u'goog:detailedDescription'}, u'itemListElement': [], u'@type': u'ItemList'}

scripts/google_knowledge_query.py

"""Example of Python client calling Knowledge Graph Search API."""
import json
import urllib

api_keys="xxxx"

def query(params):
    service_url = 'https://kgsearch.googleapis.com/v1/entities:search'
    url = service_url + '?' + urllib.urlencode(params)
    print url
    response = json.loads(urllib.urlopen(url).read())
    dir(response)
    print(response)
    # for element in response['itemListElement']:
    # for element in response:
        # print "query "+ element['result']['name'] + ' (' + str(element['resultScore']) + ')'

def main():
    params1 = {
        'query': 'Taylor Swift',
        'limit': 10,
        # 'ids':'g/11b5l_c1f6',
        'indent': True,
        'key': api_keys,
    }
    params2 = {
        # 'query': 'Taylor Swift',
        # 'limit': 10,
        # 'ids':'/m/0w1hzqb',
        'ids': '/m/010mgs42',
        'indent': True,
        'key': api_keys,
    }
    query(params2)

if __name__ == '__main__':
    main()

Well, it does not make sense.  Because the object should be somewhere.
Thanks!

cc @jonparrott
@Samurais does this also occur in the Node SDK, or is it a Python-specific issue?

@Samurais can you share the JavaScript you're using?

We need more info to do anything here.  Let us know if you'd still like us to take a look!


Textsize to 14 and use Bold.

I think this way the labels are more readable.
I also tried to change Open Sans and Arial Unicode to bold but it looks like it's not hosted on the cdn.
before:

after:

What if you have 100 devices?

before:

after:

I like the before version.
I also see that you changed font to Noto Sans Bold. But we also use some other fonts. Which means it will be inconsistent.

I know, that's why I said: "I also tried to change Open Sans and Arial Unicode to bold but it looks like it's not hosted on the cdn"

Yeah, but then it doesn't make sense to change only one.

I recommend submitting a feature request and we see what people want. Again, I personally prefer smaller font.


ShadowSocks support?

Wireguard is nice but ShadowSocks is better because it doesn't keep session alive. THis is great for smartphone cause the data usage costs money.

Shadowsocks is proxy and Wireguard is VPN. Different species.

ShadowSOCKS is in a grayarea between a Proxy and VPN. While ShadowSOCKS may look like a VPN if you are using a client like Outline or Shadowsocks Android, the underlying networking componets are not Layer 2 or 3 like a VPN but more like Layer 4 where it can transport TCP/UDP data over a proxy-like connection that does not create a virtual network interface in the same way a real VPN does.
ShadowSOCKS can create a virtual network interface like in the case of Outline, but that cannot communicate on a LAN segment to other devices on the VPN's network like a real L2/L3 tunnel can. Making it kinda useless.
You are better off just setting up a ShadowSOCKS server along side SoftEther like I did on my home VPN machine.


Don't create PR if running against a PR

Description
Closes #16.
Checklist

[x] Tested functionality against a test repository (see "How to test changes")

https://github.com/cole-h/update-flake-lock-test/runs/6278183888?check_suite_focus=true

[ ] Added or updated relevant documentation (leave unchecked if not applicable)

The action still runs and GitHub marks it as successful, so this isn't something we can do (to our satisfaction) in this repo.


fix: handle bar chart flickering

High Level Overview of Change

The bar chart for Upgrade Status is flickering due to:

Bar animation per data refetch
A strange data point from validators list with undefined fields:
{"ledger_index":74661353,"ledger_hash":"613E298A8C0AEB816D16AA61952E0834BBD9B5E5677EA3E9A2413118EE074363"}

This PR will fix the above issues.
Context of Change

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (non-breaking change that only restructures code)
[ ] Tests (You added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation Updates
[ ] Release

TypeScript/Hooks Update

[ ] Updated files to React Hooks
[ ] Updated files to TypeScript

Before / After

Test Plan

Can you add a test for this weird case.

Do we know what's causing that weird test case? Is there a bug in the VHS?

Do we know what's causing that weird test case? Is there a bug in the VHS?

It's the Streams component of the Explorer that caused this issue as we investigated. Will resolve it in another ticket.


[fixed] maybeScrollItemIntoView exception when itemNode is not found

It's difficult to know why you want to perform this change, since you've provided no context, but I'm assuming it's because you've implemented a renderMenu function which doesn't always return every item to be rendered. This edge case is already documented:

Ensure the returned tree includes every entry in items or else the highlight order and keyboard navigation logic will break.