Add support for Apache HttpClient instrumentation

relates to https://github.com/elastic/apm-agent-java/issues/127

Codecov Report

Merging #151 into master will decrease coverage by 0.11%.
The diff coverage is 59.45%.

@@             Coverage Diff              @@
##             master     #151      +/-   ##
============================================
- Coverage      73.6%   73.49%   -0.12%
- Complexity      994     1002       +8
============================================
  Files           110      111       +1
  Lines          3702     3739      +37
  Branches        329      332       +3
============================================
+ Hits           2725     2748      +23
- Misses          819      831      +12
- Partials        158      160       +2

Impacted Files
Coverage Δ
Complexity Δ

.../co/elastic/apm/impl/transaction/AbstractSpan.java
91.66% <0%> (-3.99%)
20 <0> (ø)

...pm/httpclient/ApacheHttpClientInstrumentation.java
62.85% <62.85%> (ø)
8 <8> (?)

...stic/apm/report/IntakeV1ReportingEventHandler.java
73.33% <0%> (-4.45%)
12% <0%> (-1%)

...ration/source/PropertyFileConfigurationSource.java
40% <0%> (+2.85%)
7% <0%> (+1%)
:arrow_up:

...a/co/elastic/apm/logging/LoggingConfiguration.java
93.87% <0%> (+4.08%)
12% <0%> (ø)
: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 67407df...6f4f156. Read the comment docs.


Switch languages without page reload

Can we switch (natural) languages without reloading the page? With a page reload, the current state gets lost (code, input, output, ...).

Dodona itself also uses a page refresh to do this. Is there a use case for people programming first, then realizing the locale doesn't suit them to then change this locale?

Is there a use case for people programming first, then realizing the locale doesn't suit them to then change this locale?

There might be situations where poeple do not bother to change language before entering code and then get some feedback (for example, after a Python traceback) and realize that they would like to get feedback in a different language.


Get buffer icon from mode before falling to other options

Get buffer icon from mode before falling to other options

fix error while resolving (helm-icons--get-icon 'fallback) with 'all-the-icons provider (#16)

I got bitten by the error in #16 today and would love to see this merged.

Ah, this is what I was attempting to do when I first added ati. Nice!

anything preventing this from being merged?


Generic ephemeral volume support

Details
I would like to have support for generic ephemeral volumes in the common chart.
https://kubernetes.io/docs/concepts/storage/ephemeral-volumes/#generic-ephemeral-volumes
Helm chart name:

Describe the solution you'd like:

Anything else you would like to add:
Its not currently supported atm.

Additional Information:

I'm more opposed to this, and chance you could open a PR?


[WIP] ARROW-831: switch from boost::regex to std::regex

I'm able to replicate this error locally with Xcode 6.4, and
static const std::regex DECIMAL_PATTERN("(\\+?|-?)((0*)(\\d*))(\\.(\\d+))?", std::regex::ECMAScript);

doesn't work either. So we'll need to rewrite this regex to work with C++11 regexes

Do trivial regexen match?

I think this might be a bug with XCode 6 since this seems to work fine with gcc 4.9 and MSVC =(

I think the best option may be to write a POSIX-compliant regex (the current regex doesn't compile with std::regex::extended), any takers? There's a separate issue which is that the Status code in the Decimal constructor when parsing a string goes unchecked.

std::regex seems to not be supported in g++ <= 4.8:
http://stackoverflow.com/questions/12530406/is-gcc-4-8-or-earlier-buggy-about-regular-expressions
Do we still want this for 0.3 then? Supporting g++ 4.8 seems important, afaik it is the standard in ubuntu 14.04, which is still widely used.

OK, that sounds good for now. Eventually I want to add https://github.com/google/re2 to our toolchain for faster string analytics based on regexen, and at that point maybe we can switch away from std::regex. It's too bad that the C++11 regex implementation took a while to reach maturity.

Now that https://github.com/apache/arrow/pull/567 is in, let's close this? Thanks


Launching containers using Docker Compose on Windows system may result in: backend_app        | exec /usr/backend/entrypoint.sh: no such file or directory

During the Windows build startup, the error '| exec /usr/backend/entrypoint.sh: no such file or directory' occurs, causing the entrypoint.sh script to be unable to execute.
Execution failed!
The same issue occurs on my computer, company computers, and my colleagues' computers. How can we resolve this?

I think all Windows users will encounter this problem. （Docker version 25.0.3）

I saw your log of docker daemon. It means you docker cannot create container successfully. @Micost uses Windows platform. However, he uses WSL2(unix like environment). So, Maybe we can discuss this in next meeting.

@arvin936 is working on these kind of issues. I believe we should remove chmod +x entry_point.sh in Dockerfile.

@arvin936 is working on these kind of issues. I believe we should remove chmod +x entry_point.sh in Dockerfile.

I think the key to the problem is not to perform the build process on Windows, we should perform these steps during the release of the image. For newcomers who are still trying to run the program, the build process should not be their current focus.


Better handling of nospectrum/nodigital

Replace nodigital/nospectrum args in ScopeApp/OscilloscopeWindow with bitmask so we can request any set of channels
Add checkboxes to startup/connection dialog for initial channel set, allowing any of analog/digital/spectrum. Enforce at least one being checked.

Change of plans, just hide spectrum/digital channels at startup unless they're on already.


Update Modin Get Started Readme with node and mode instruction

Introduction
This template serves 2 purposes and will need to be edited accordingly prior to submitting the PR

Changes to existing samples
Submission of new Samples

Existing Sample Changes
Description
Made edits to the Running Samples in Devcloud section to enable users to select the correct mode and node to run their sample.
I added a jump link at the top to skip to the DevCloud directions. I am not sure if there are steps required before running a sample on the devcloud. For example, does the user need to build modin for CPU or activate conda before using the DevCloud? If the answer is yes, then the jump link to the DevCloud directions need to move down below the required steps.
NOTE: Prior to accepting pull request, please review the commented sections in the README.md:

If your sample runs in batch mode, please delete the section on how to run in interactive mode. Joe Oster requested this because interactive mode can be troublesome so we would rather have users use batch mode when possible.
In the node table, if a node is applicable to this sample, please change the node name to BOLD so users can easily identify which nodes are available.

Fixes Issue#
External Dependencies
None.
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Implement fixes for ONSAM Jiras
[ x] Documentation

How Has This Been Tested?
Text was reviewed and approved by Dan Petre, Donna Boyer, and Joe Oster.

[ ] Command Line
[ ] oneapi-cli
[ ] Visual Studio
[ ] Eclipse IDE
[x ] VSCode
[ ] When compiling the compliler flag "-Wall -Wformat-security -Werror=format-security" was used

Closing pull request. We are waiting until @raoberman can come up with a solution to get the Modin sample to work on DevCloud.


change colour and point type based on previous suggestions

At the moment a new scatterplot group causes the same combo (Orange and Circle) to be selected. The problem i believe is the lack of ability to update the popupmodal, maybe a reactive dataframe?

Should be working now.


Fetch correct dictionary in case of process.env.ORION_APPID

A small oversight in my previous (honoured) pull-request.
Also in case of server-side use of orion.dictionary.get(), make sure the correct query parameters are provided to the findOne() call.
Regards,
Joris

Nice


Tabs CSS does not cope with use of dropdown component for narrow version

Detailed description
The tabs component when narrower than 768px (42rem in v10) shows a dropdown like replacement. Use of an actual dropdown makes component implementation simpler as dropdown functionality only needs defining in one place.

When using a real dropdown component for the narrow version the styling of the dropdown is compromised. Producing this effect.

tabs component

Use of a dropdown component should be supported.

Chrome

9.87.2

carbon-components-vue

Steps to reproduce the issue
Additional information
Tabs issue before, with standard behaviour CSS hidden for demonstration purposes.
https://codepen.io/lee-chase/pen/aMPGNW
Tabs after fix
https://codepen.io/lee-chase/pen/MxZGpe
NOTE: The fix could be achieved by only applying bx--tabs-trigger classes when bx--dropdown is not also a class.

Hi @lee-chase thank you for bringing up this topic! The idea for reusing dropdown style for narrow-mode tabs is interesting, several things comes up on this topic here:

Would applications really like doubled markup?
Need to get JS implementations (Vue, React, vanilla, etc...) in sync, with their API in consideration (e.g. event models, tab panel integration)
Possible future change in narrow mode UX (If our designer comes up with a different UX for narrow mode, would we want to rip/replace the dropdown portion and probably make another component work as narrow mode tab UI?)

Good questions @asudoh
To your first point, the double markup is a bit of a pain but a relatively small one as we're looking at a maximum of six tabs. Alternatively, we could use javascript to control the switch allowing us to refine how tabs behave when users have sidebars or other width constraints.
With regards to the API I would not expect the user of a tab component to make any changes, so the same events are raised. (This happens in the current Vue implementation).
Yes, I would expect to replace the component based on future changes. It would be a surprise if the alternative we designed was not a component in its own right.
In general, I would like to see us re-use our components where ever possible. That includes the handlebars where it would be nice to see us use partials.


failed to transfer SendPort to dart from c

failed to transfer SendPort to dart from c,use sample code as follow:
https://github.com/dart-lang/sdk/tree/master/samples/ffi/async
thank you!

Can you be more specific?  What in particular is failing?  What version of Dart are you on?

Can you be more specific? What in particular is failing? What version of Dart are you on?

Thanks for the reply, I got the value of receive_port_ through Dart_NewNativePort_DL. But the execution to Dart_PostCObject_DL will return an error, so it prompts "Failed to send message, invalid port or isolate died".
When I create Dart_Handle through Dart_NewSendPort, I am prompted, "Isolate is not created or Dart_EnterScope is not executed"
I create an Isolate through Dart_CreateIsolateGroup, and when compiling, I am prompted that "Dart_CreateIsolateGroup is not defined"
I have copied files such as dart_api.h, so I currently don't know how to create the Isolate correctly.
The code uses PendingCall as shown below:
`class PendingCall {
public:
PendingCall(void** buffer, size_t* length)
: response_buffer_(buffer), response_length_(length) {
receive_port_ =
Dart_NewNativePort_DL("cpp-response", &PendingCall::HandleResponse,
/handle_concurrently=/false);
}
~PendingCall() { Dart_CloseNativePort_DL(receive_port_); }
Dart_Port port() const { return receive_port_; }
void PostAndWait(Dart_Port port, Dart_CObject* object) {
std::unique_lockstd::mutex lock(mutex);
const bool success = Dart_PostCObject_DL(port, object);
if (!success) FATAL("Failed to send message, invalid port or isolate died");

printf("C   :  Waiting for result.\n");
while (!notified) {
  cv.wait(lock);
}

}
static void HandleResponse(Dart_Port p, Dart_CObject* message) {
if (message->type != Dart_CObject_kArray) {
FATAL("C   :   Wrong Data: message->type != Dart_CObject_kArray.\n");
}
Dart_CObject** c_response_args = message->value.as_array.values;
Dart_CObject* c_pending_call = c_response_args[0];
Dart_CObject* c_message = c_response_args[1];
printf("C   :   HandleResponse (call: %" Px ", message: %" Px ").\n",
reinterpret_cast<intptr_t>(c_pending_call),
reinterpret_cast<intptr_t>(c_message));
auto pending_call = reinterpret_cast<PendingCall*>(
    c_pending_call->type == Dart_CObject_kInt64
        ? c_pending_call->value.as_int64
        : c_pending_call->value.as_int32);

pending_call->ResolveCall(c_message);

}
private:
static bool NonEmptyBuffer(void** value) { return *value != nullptr; }
void ResolveCall(Dart_CObject* bytes) {
assert(bytes->type == Dart_CObject_kTypedData);
if (bytes->type != Dart_CObject_kTypedData) {
FATAL("C   :   Wrong Data: bytes->type != Dart_CObject_kTypedData.\n");
}
const intptr_t response_length = bytes->value.as_typed_data.length;
const uint8_t* response_buffer = bytes->value.as_typed_data.values;
printf("C   :    ResolveCall(length: %" Pd ", buffer: %" Px ").\n",
response_length, reinterpret_cast<intptr_t>(response_buffer));
void* buffer = malloc(response_length);
memmove(buffer, response_buffer, response_length);

*response_buffer_ = buffer;
*response_length_ = response_length;

printf("C   :     Notify result ready.\n");
notified = true;
cv.notify_one();

}
std::mutex mutex;
std::condition_variable cv;
bool notified = false;
Dart_Port receive_port_;
void** response_buffer_;
size_t* response_length_;
};`
thanks for help。

//cc @aam

@LF-H sorry it took a while to get back to your issue.
Are you still facing the problem?
You can take a look at dart sdk test code to see how you can create isolates: https://github.com/dart-lang/sdk/blob/a155e2b521088d38/runtime/bin/ffi_test/ffi_test_functions_vmspecific.cc#L356 for example.


Hack Session 516 ✼ Remotely

23.03.2022
Wrap-up of our Gōng-fu I/O Weekly · Hack Session #516 ✼ Remotely ✼ meetup.
Projects
• ‹Bouboucle› generative art with Javascript — @andreaskundig and @david-hodgetts
• ‹Distributed Arcade› Game World & Boards with H3 & hex-coordinate systems — @olange and @rudifa
See also

#515 Previous session
#517 Next session

We prepared the migration of our ‹h3-minimap› Observable notebook source code to an ‹h3-worldmap› web component on GitHub and published an initial (so far, useless) version on NPMjs.org.


Update 143-quicksort.md

Here is a video.
This project was made a copuple of years ago and uses all sorts of fancy data structures and interfaces ... just for the sake

Thank you ❤

To keep in mind: check the length of the title and create a test for the title length if necessary.


relax 0850a (location bar autocomplete)

IMO this is a pretty fundamental and awesome feature for most people. also see https://github.com/ghacksuserjs/ghacks-user.js/issues/438

IMO we can also remove 'browser.urlbar.autoFill.typed' because 'browser.urlbar.autoFill' disables '.typed' as well and people who want autofill probably also want typed stuff autofilled. (removing it instead of moving to 9999 for the FF62 diffs)

we're jumping the gun a little here - we have a couple of issues open - sharing overrides and a discussion/vote (only as an indicator, not binding) re cookies, etc
We haven't decided what we're doing with the master & extra stickies/js-examples re do we make the master hardened and provide relaxed items, or more relaxed and provide hardened items. And I would consider this item (location bar dropdown etc) as only a small part of a bigger set to do with shoulder surfers and to some degree local data (eg history, forms etc) - and we should relax all of those "together" IMO
PS: created a separate issue for autofill


DE-6498: Add Storybook

I Added storybook for API docs and development of the components.
Here I also did a few drive-by fixes of some bugs and the API, but more will come in DE-6506. There I will also fix the rest of the stories, bc. some of them currently do not work.

Okay, I fixed the Jest config, now it's really ready for review :)

Thanks!


update extension icons

The icons has been updated, now update the gallery to use the new icons.

I have already deleted the sqlserver.png from these extensions when i was updating the icon.


updated code copied from server

this brach is a clone of all the code changed we did on aws

Add .gradle to gitignore pls


Error build

In file included from /root/tdbot/td/td/telegram/AnimationsManager.cpp:21:
/root/tdbot/td/td/telegram/Td.h:249:50: error: ‘td::StateManager::State’ has not been declared
249 |   void on_connection_state_changed(StateManager::State new_state);
|                                                  ^~~~~
/root/tdbot/td/td/telegram/Td.h:277:17: error: ‘State’ in ‘class td::StateManager’ does not name a type
277 |   StateManager::State connection_state_;
|                 ^~~~~
/root/tdbot/td/td/telegram/Td.h:348:91: error: ‘td::StateManager::State’ has not been declared
348 | d_api::ConnectionState> get_connection_state_object(StateManager::State state);
|                                                                   ^~~~~
/root/tdbot/td/td/telegram/Td.h:1054:38: error: ‘td::td_api::getChatStatisticsUrl’ has not been declared
1054 |   void on_request(uint64 id, td_api::getChatStatisticsUrl &request);
|                                      ^~~~~~~~~~~~~~~~~~~~
/root/tdbot/td/td/telegram/Td.h:1176:44: error: ‘getChatThemes’ in namespace ‘td::td_api’ does not name a type; did you mean ‘setChatTheme’?
1176 |   void on_request(uint64 id, const td_api::getChatThemes &request);
|                                            ^~~~~~~~~~~~~
|                                            setChatTheme
make[3]: *** [td/CMakeFiles/tdcore.dir/build.make:310: td/CMakeFiles/tdcore.dir/td/telegram/AnimationsManager.cpp.o] Error 1
make[2]: *** [CMakeFiles/Makefile2:529: td/CMakeFiles/tdcore.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:411: CMakeFiles/telegram-bot.dir/rule] Error 2
make: *** [Makefile:164: telegram-bot] Error 2

You have wrong Td.h header.

/root/tdbot/td/td/telegram/Td.cpp:93:10: fatal error: td/telegram/SponsoredMessages.h: No such file or directory
93 | #include "td/telegram/SponsoredMessages.h"
|          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[3]: *** [td/CMakeFiles/tdcore.dir/build.make:2013: td/CMakeFiles/tdcore.dir/td/telegram/Td.cpp.o] Error 1
make[2]: *** [CMakeFiles/Makefile2:529: td/CMakeFiles/tdcore.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:411: CMakeFiles/telegram-bot.dir/rule] Error 2
make: *** [Makefile:164: telegram-bot] Error 2


Pushing to docker hub. Error: Index response didn't contain any endpoints

Description
When pushing to docker hub, it starts the push, but, before it can get anywhere, it gives this error.
Steps to reproduce the issue:

docker login

docker login
Login with your Docker ID to push and pull images from Docker Hub. If you don't have a Docker ID, head over to https://hub.docker.com to create one.
Username: fustbariclation
Password:
Login Succeeded

docker push

docker push hub.docker.com/fustbariclation/blue:latest

Describe the results you received:
Tue 25 Oct 2016 14:16:19 SAST
The push refers to a repository [hub.docker.com/fustbariclation/blue]
d46afcd68ca7: Preparing
a2eb9966230c: Preparing
c2ade7efa02d: Preparing
c998354a0a8a: Preparing
67aca2b232ea: Preparing
b0afbd3c7b31: Waiting
38ece981f0d9: Waiting
3ac0d1046245: Waiting
4ce5da20b321: Waiting
a0dc2eaec3c3: Waiting
ca20b906b87c: Waiting
30596d6a4f4c: Waiting
ec96d31f8eaf: Waiting
e3ac10d00c60: Waiting
2c2153fbd032: Waiting
d9a069c1d0fc: Waiting
a5eb0fc1decb: Waiting
b2ac5371e0f2: Waiting
142a601d9793: Waiting
Index response didn't contain any endpoints

Describe the results you expected:
I expected it to push the segments.
Additional information you deem important (e.g. issue happens only occasionally):
It happens consistently. I've seen some reports of other people having this issue, but not any fixes. On report said it was a problem with DNS - but that seems to be working OK:
ping index.docker.io
PING index.docker.io (<IP_ADDRESS>): 56 data bytes
Request timeout for icmp_seq 0
Request timeout for icmp_seq 1
Request timeout for icmp_seq 2
Request timeout for icmp_seq 3

I'm not sure if that host is supposed to have icmp enabled, so I don't know if there's any significance in it not responding to a 'ping'.
Output of docker version:
Client:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.7.1
 Git commit:   6f9534c
 Built:        Thu Sep  8 10:31:18 2016
 OS/Arch:      darwin/amd64

Server:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:        Thu Aug 18 17:52:38 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 43
Server Version: 1.12.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 27
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: overlay null bridge host
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.20-moby
Operating System: Alpine Linux v3.4
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 10.73 GiB
Name: moby
ID: 6DPA:LPBI:N7WO:W26J:36TZ:6AKY:QHNS:LQYI:4IQE:UUKR:OEWV:XLBH
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 33
 Goroutines: 99
 System Time: 2016-10-25T12:47:17.232063687Z
 EventsListeners: 1
No Proxy: *.local, 169.254/16
Registry: https://index.docker.io/v1/
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
OS/X - 10.12

ping @dmp42 any ideas?

Yes.
Don't:
docker push hub.docker.com/fustbariclation/blue:latest
Do:
docker push fustbariclation/blue:latest

Oh, gee, I should learn to read. Thanks @dmp42
I'll close this issue as this is not a bug but I think the above should help you get your issue sorted

I am having this same issue, and the work-around also fails. Here's my first attempt (like the OP's problem):
$ docker login -u drstrangepork -p $DOCKERHUB_PASSWORD hub.docker.com
Login Succeeded
$ docker tag drstrangepork/awscli-runner hub.docker.com/drstrangepork/awscli-runner
$ docker push hub.docker.com/drstrangepork/awscli-runner
The push refers to a repository [hub.docker.com/drstrangepork/awscli-runner]
Index response didn't contain any endpoints

Then I followed your advice and got this:
$ docker login -u drstrangepork -p $DOCKERHUB_PASSWORD hub.docker.com
Login Succeeded
$ docker push drstrangepork/awscli-runner
The push refers to a repository [docker.io/drstrangepork/awscli-runner]
denied: requested access to the resource is denied

Not using hub.docker.com with docker login doesn't work either:
$ docker login -u drstrangepork -p $DOCKERHUB_PASSWORD
Error response from daemon: Get https://registry-1.docker.io/v2/: unauthorized: incorrect username or password

I wonder why is it closed. I have exactly the same issue.
Docker version is this:
Client:
 Version:      17.04.0-ce
 API version:  1.28
 Go version:   go1.7.5
 Git commit:   4845c56
 Built:        Mon Apr  3 18:07:42 2017
 OS/Arch:      linux/amd64

Server:
 Version:      17.04.0-ce
 API version:  1.28 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   4845c56
 Built:        Mon Apr  3 18:07:42 2017
 OS/Arch:      linux/amd64
 Experimental: false

This issue is caused by daocloud.io (It's a docker mirror in China).
Please try to tag your image like
docker tag daocloud.io/[daocloud_id]/[image_name] [dockerhub_id]/[image_name]

to make a new tag without the daocloud.io part.
Then try docker push [dockerhub_id]/[image_name].

Yes


Allow tab color to be modified

It would be useful to have a Tab Rules option to change the tab color to a user-defined hex color. This way a user could organize tabs by color groupings (especially useful if lots of tabs open in a browser window).
The reasoning behind this is analogous to what can be done with the <meta name="theme-color" content="#{hexcolor}"> tag in controlling the display of the "tab" color on mobile: https://developers.google.com/web/fundamentals/design-and-ux/browser-customization/

Hello,
As I know you can't set a color on a Chrome tab on desktop. Beside, Chrome mobile does not support extensions.
Best

Indeed. Maybe another option is to "colorify" the icon? :)


Release 1.0.5 on Bower

Hi there.
$ bower info angular-material-sass-files

Returns only 0.8.3 as available version.
Are there plans for new releases on bower? Would be nice to have them, otherwise I think we can change the Readme to avoid old installs. WDYT?

Should be available now. Thanks for reporting!

Thanks!


aws-sdk-js

Is https://github.com/aws/aws-sdk-js with the react-native branch the successor to this repo?

Please use AWS Amplify moving forward: https://github.com/aws/aws-amplify


Automated snapshot creation

Create snapshots available on IPFS:

fetch the latest sentinel release and npm ci
for every configured network:

run npm build-snap
add the created DB file to IPFS (ipfs --api <api addr> add <file>)
get hash/CID of uploaded file
publish (or push) hash/CID

(maybe) somehow notify about new snapshots

Point sentinels repo to latest snapshots:
Option push:
Have a local copy of the git repo, modify manifest.json with the updated hashes/CIDs, commit to a branch, push to GH and create a PR on GB.
Option pull a:
Have a github action (which trigger?) pull the latest snapshot hashes/CIDs and update manifest.json accordingly.
Option pull b:
Let a git hook pull the latest snapshot hashes/CIDs and update manifest.json accordingly.
For the pull options, we also need a way for how the repo updating agent can access the data (hashes/CIDs) provided by the snapshot creation agent. If running on a dev machine, it could be just an ssh <machine> "cat file" .... Else (e.g. GH action) it could be any of a dedicated API/webservice provided by the creation agent, DNS entries, ENS (just kidding - as long as no L2 ENS), IPNS (bad past experiences...), ...?

favorite option:
Push: after snapshot generation, the snapshot creation agent triggers a GH action via webhook. That action shall update manifest.json in a dedicated branch and create a PR if none exists currently.
Additionally, that GH action shall verify the provided snapshot hashes/CIDs, e.g. with npm run verify-snapshot <network>.

@ngmachado example for IPFS pinning:
sentinel@buck-1:~/superfluid-sentinel/snapshots$ ipfs --api /ip4/<IP_ADDRESS>/tcp/5001 add snapshot_4_1643374733.sqlite.gz
added QmaJWnQ9K3Y6e75szA3itYGd5MRwxetpEJHr4pseEEUEU1 snapshot_4_1643374733.sqlite.gz
 2.06 MiB / 2.06 MiB [=============================================================================================================================================================================================] 100.00%

The IPFS API addr should be configurable.
If you want to use that API from other machines than buck-1, let me know - need to add permission to its firewall then (if open to the Internet, anybody could upload stuff there).

From the last PR the verify-snapshot was rename to test-sync

We can document the process in the process section of https://github.com/superfluid-finance/superfluid-sentinel/wiki/About-Fastsync


Analyzing data from Neuropixels 2.0 with 4 shanks

Hi,
I'm using 2.0 Neuropixel probes with 4 shanks. Following the 'Analyze Neuropixels datasets' tutorial I see that si.plot_probe_map(raw_rec, ax=ax, with_channel_ids=True) produces a plot with the wrong channel mapping, where all channels appear on the first shank, even though the four shanks are plotted.

With ax.set_ylim(-50, 3000):

According to this issue the problem with plotting 4 shank probes was solved by upgrading to version 0.97.1, but I'm running version 0.99.0.dev0 and something similar is happening.
I run the whole tutorial and the shanks are being represented as a single one on the sortingview widget as well, so I guess they are just being treated as a single shank throughout the tutorial?
What is recommended for these probes? Should I just split the files into one per shank and analyze them separately like here, or is there a way to treat them as a single probe with four shanks?
Thanks!!

Hi @ja-94
As mentioned here (https://github.com/SpikeInterface/spikeinterface/issues/1923) the problem has been fixed in main via #205. We'll make a new release soon!


Why is _CVBooster object hidden class?

Some of Kaggler use out-of-fold prediction (oof) which is calculated with validation dataset for stacking, blending, evaluation etc. In order to  calculate oof with lgb.cv(), we need to return Boosters however it is not returned from the function. If I customize to return _CVBooster object, it can be calculated, but _CVBooster is hidden class, so I think it is not appropriate to return officially.
Is it possible to remove prefix underscore from  _CVBooster? Then I can customize to return the object on lgb.cv().
I would appreciate it if you could advise to it.
_CVBooster
https://github.com/Microsoft/LightGBM/blob/master/python-package/lightgbm/engine.py#L248-L267
Example
https://www.kaggle.com/kenmatsu4/using-trained-booster-from-lightgbm-cv-w-callback

ping @StrikerRUS

Thank you for your comment!  Minimum requirements are:

Returning list of Boosters
Returning best_iteration to identify which iteration is best for predict when using early stopping.

In addition to above, current _CVBooster.__getattr__ function is useful for batch operation of boosters on each fold. The following link is the part of usage of the _CVBooster.__getattr__ calculating prediction result for test data.
https://github.com/matsuken92/LightGBM_test/blob/master/src/cv_booster_test.py#L117
I will send my idea as PR, so I would appreciate it if you could review it then!

@matsuken92 I see! Thanks for the clarification!
Of course, I'll be happy to review it.
Can you please also think about backward compatibility in terms of cv function return type? Maybe a new key in results dict or something else, idk...

Closing in favor of being in #2302. We decided to keep all feature requests in one place.
Welcome to contribute this feature! Please re-open (or post a comment if you are not a topic starter) this issue if you are actively working on implementing this feature.

Closed via #3204.


Fix/payment tests

Explain your changes:
This PR is trying to fix the flakiness of the payment test by reduce the fee after we change the snark coordinator key
Explain how you tested your changes:
*
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet


Bugfix: Make blankslate-narrow responsive

Using max-width instead of width allows the blankslate to shrink on smaller screens.
Fixes: #553.
Sorry for the branch name, I made this change right on GitHub.
/cc @primer/ds-core

It's also possible that .container could benefit from the same adjustment: https://github.com/primer/primer/blob/master/modules/primer-layout/lib/container.scss#L4.
The other container flavors are using max-width so it would also be more consistent.

Sure thing, @shawnbot! Would it be helpful to make another pull request for the .container width to max-width adjustment I mentioned earlier?


Transparent Navigation Bar

Using the plugin makes the bottom navigation bar transparent.  Using the opaque theme UnityStatusBarTheme makes it solid again, but then the top status bar is gray.
Device -> Pixel 2 XL running Android 8.1.0

What is your expected behavior?

It is expected that the plugin will not affect the navigation bar.  It is also expected that the opaque version of the status bar would be the same color as the opaque version of the navigation bar (black).

Sorry for a very late answer.
You can configure the transparency of the navigation bar on your own, by changing the flag
true
in style definitions (values-v21 and values-v19), and then repacking/recompiling the aar.
On the opaque status bar color - it's controlled by the theme, in this case Theme.Material.Light.NoActionBar.Fullscreen. Unity by default uses Light themes, but I'd agree a dark one by default here makes more sense. I'll fix that.

Non-light theme used in 0.5 release.


No underscores allowed in group/artifact

Currently it is not possible to use _ in group/artifact identifier. This limits easy integration since database table names usually don't permit - in names, but allow _. So it is always necessary to transform names from database -> databus and vice versa.

seems weird as SHACL allows underscores, see:
https://dbpedia.gitbook.io/databus/model/model/dataid
sh:pattern "/[a-zA-Z0-9\\-_]{4,}/[a-zA-Z0-9\\-_\\.]{1,}/[a-zA-Z0-9\\-_\\.]{1,}/[a-zA-Z0-9\\-_\\.]{1,}#Dataset$" ;

now working, added test.


🐛 Fix OpenAPI definition generation

This is supposed to implement the same change as https://github.com/matrix-org/matrix.org/commit/d14b93a0bd80da4c112ba75c7bc1693aaec6fe61
Can I please have a sanity check?
Signed-off-by: Alexandre Franke<EMAIL_ADDRESS>

looks like you may need to rebase your branch or otherwise merge main - the failing job was removed

at least I think it was


Networking Plugin: failed to apply changes (10 - Operation not supported)

Description
Using a Raspberry Pi 3 RDK build, the networking plugin fails to bring up the dobby0 bridge device.
Visible with network types nat or none.
Workaround is to only use open network type to circumvent having to create a veth device.
Expected Behaviour
No error when launching container.
Actual Behaviour
The networking plugin fails, the container starts if the plugin is set to required.false, but the network set up does not work.
Logs:
0000000454.428183 ERR: < M:Netlink.cpp F:applyChangesToLink L:565 > failed to apply changes (10 - Operation not supported)

Steps to Reproduce
Start any container with networking type in the plugin set to none or nat.
Platform/Environment

Raspberry Pi 3 Model B v1.2, basic RDK RPi build
Crun version: 0.14.1
Dobby version: 3.0

The method Netlink::ifaceUp causes the bug, and a bridge device is not created.
There have been issues with certain versions of libnl and creating bridge devices. In rdkPlugins/Networking/CMakeLists.txt, if the last line is uncommented, a workaround is used to fix that issue.
I tried using the workaround but the issue still persists. Will need to try again and give it another go.

Any updates on this issue?
Problem still persists on dunfell-2023q2.
I tried using open network as a temporary workaround but container still has no connection.


treatment of default column aliases when none are specified

note: this is cross-posted here from the SQLite3 forums.
Consider what the SQLite CLI does when returning results with same column names from different tables but no aliases.
$ sqlite3
SQLite version 3.44.2 2023-11-24 11:41:44
Enter ".help" for usage hints.
Connected to a transient in-memory database.
Use ".open FILENAME" to reopen on a persistent database.
sqlite> CREATE TABLE one (id INTEGER PRIMARY KEY, a, b);
sqlite> CREATE TABLE two (id INTEGER PRIMARY KEY, c, d, one_id, a);
sqlite> INSERT INTO one (a, b) VALUES ('foo', 'bar'), ('baz', 'qux');
sqlite> INSERT INTO two (c, d, one_id, a) VALUES ('frob', 'blah', 1, 'foo'), ('rand', 'comt', 2, 'baz');
sqlite> SELECT id, a FROM one;
id  a
--  ---
1   foo
2   baz
sqlite> SELECT c, d, id, a FROM two;
c     d     id  a
----  ----  --  ---
frob  blah  1   foo
rand  comt  2   baz
sqlite> SELECT * FROM one JOIN two ON one.id = two.one_id;
id  a    b    id  c     d     one_id  a
--  ---  ---  --  ----  ----  ------  ---
1   foo  bar  1   frob  blah  1       foo
2   baz  qux  2   rand  comt  2       baz
sqlite> SELECT id, a, b, id, c, d, one_id, a FROM one JOIN two ON one.id = two.one_id;
Parse error: ambiguous column name: id
  SELECT id, a, b, id, c, d, one_id, a FROM one JOIN two ON one.id = two.one_id;
         ^--- error here
sqlite> SELECT one.id, one.a, one.b, two.id, two.c, two.d, two.one_id, two.a FROM one JOIN two ON one.id = two.one_id;
id  a    b    id  c     d     one_id  a
--  ---  ---  --  ----  ----  ------  ---
1   foo  bar  1   frob  blah  1       foo
2   baz  qux  2   rand  comt  2       baz

So, SQLite (the lib) is happy to output columns with the same default aliases, when none are provided, as shown in SELECT * FROM one JOIN two ON one.id = two.one_id but at least the SQLite CLI doesn't like it when ambiguous column names are provided in a query.
better-sqlite3, on the other hand, silently removes the column two.a from the result because the returning object is a JavaScript object, and an object can't have two keys with the same name.
function selects() {
    const res1 = db.prepare('SELECT id, a FROM one').all();
    console.log(' res1\n', '-'.repeat(50), '\n', res1, '\n');

    const res2 = db.prepare('SELECT c, d, id, a FROM two').all();
    console.log(' res2\n', '-'.repeat(50), '\n', res2, '\n');

    const res3 = db.prepare(`SELECT * FROM one JOIN two ON one.id = two.one_id`).all();
    console.log(' res3\n', '-'.repeat(50), '\n', res3, '\n');

    const res4 = db.prepare(`SELECT one.id, one.a, one.b, two.id, two.c, two.d, two.one_id, two.a FROM one JOIN two ON one.id = two.one_id`).all();
    console.log(' res4\n', '-'.repeat(50), '\n', res4, '\n');

    const res5 = db.prepare(`SELECT one.id AS one_id, one.a AS one_a, one.b AS one_b, two.id AS two_id, two.c AS two_c, two.d AS two_d, two.one_id AS two_one_id, two.a AS two_a FROM one JOIN two ON one.id = two.one_id`).all();
    console.log(' res5\n', '-'.repeat(50), '\n', res5, '\n');
}

 res1
 --------------------------------------------------
 [ { id: 1, a: 'foo' }, { id: 2, a: 'baz' } ]

 res2
 --------------------------------------------------
 [
  { c: 'frob', d: 'blah', id: 1, a: 'foo' },
  { c: 'rand', d: 'comt', id: 2, a: 'baz' }
]

 res3
 --------------------------------------------------
 [
  { id: 1, a: 'foo', b: 'bar', c: 'frob', d: 'blah', one_id: 1 },
  { id: 2, a: 'baz', b: 'qux', c: 'rand', d: 'comt', one_id: 2 }
]

 res4
 --------------------------------------------------
 [
  { id: 1, a: 'foo', b: 'bar', c: 'frob', d: 'blah', one_id: 1 },
  { id: 2, a: 'baz', b: 'qux', c: 'rand', d: 'comt', one_id: 2 }
]

 res5
 --------------------------------------------------
 [
  {
    one_id: 1,
    one_a: 'foo',
    one_b: 'bar',
    two_id: 1,
    two_c: 'frob',
    two_d: 'blah',
    two_one_id: 1,
    two_a: 'foo'
  },
  {
    one_id: 2,
    one_a: 'baz',
    one_b: 'qux',
    two_id: 2,
    two_c: 'rand',
    two_d: 'comt',
    two_one_id: 2,
    two_a: 'baz'
  }
]

Now that I know this behavior, I can code around it, at least until either better-sqlite3 or SQLite change their behavior. Frankly, I don't know what the right solution should be. I am assuming SQLite behaves the way it does because, perhaps all SQL engines do the same thing. better-sqlite3, on the other hand, does the best with what it is provided -- a resulting data set that is modeled as an array (table) of objects (column-value pairs). But the objects can't have duplicate keys. Hence, we have a problem unless SQLite sends some other info in the result.
Of course, I am reporting this to the better-sqlite3 folks as well.

Thanks for taking the time to describe this issue.
I’d strongly suggest altering your query to use unique column names, though. I suspect duplicate column or alias names in your SELECT may cause other non deterministic behaviors (as with any rdbms), including references in ORDER BY and GROUP BY clauses.

.expand(true)

Causes the prepared statement to return data namespaced by table. Each key in a row object will be a table name, and each corresponding value will be a nested object that contains the associated column data. This is useful when performing a JOIN between two tables that have overlapping column names. If a result column is an expression or subquery, it will be available within the special $ namespace.

Alternatively .raw(true) will also give you all data. Combine it with .columns()

yea, you are correct. Now I know that better-sqlite3 does offer the tools I need to handle this curious situation. The only possible improvement would be if better-sqlite3 figured out that the result had ambiguous names and automagically used .expand(true). In any case, many thanks for the explanation.


Eraser tool is not working

Hi, We're benchmarking labelling tools, and Labelstudio seems really promising, great job.
Only there's a critical bug in our experience: The impossibility to erase and thus to refine segmentation with brush labels.
The eraser tool is simply not working in the latest production release available: 1.6.0

Anyone knows a previous version where it worked ?

Ok I managed to pinpoint the faulty commit: 6cdfe354
I started the app with the previous commit e516329c, and the eraser tool worked as expected.

Linking to main repo issue: https://github.com/heartexlabs/label-studio/issues/867

@jeremyben Thank you for your great exploration, I've created a ticket, hope our frontend team will fix it soon.

Same problem here, I hope it gets fixed soon. :)

@omaralvarez Have you already tried the latest develop branch from the main LS repo?

No, I have not, glad to hear it. Thanks!


[VS Code] Joining collaboration session: Collaboration session not found.

Error:Joining collaboration session: Collaboration session not found.Steps to Reproduce:1.2.||Version Data||-:|:-||**extensionName**|VSLS||**extensionVersion**|1.0.540||**protocolVersion**|2.2||**applicationName**|VSCode||**applicationVersion**|1.32.3||**platformName**|MacOS||**platformVersion**|18.6.0|

Thanks for reporting this, @BryanDungan. Is it possible that the host ended the session before you tried to join it? What were you doing at the time that this happened? Were you ever able to join a session?

He's right next to me, so no
On Thu, Jul 18, 2019, 3:30 PM Monica Williams<EMAIL_ADDRESS>wrote:

Thanks for reporting this, @BryanDungan https://github.com/BryanDungan.
Is it possible that the host ended the session before you tried to join it?
What were you doing at the time that this happened? Were you ever able to
join a session?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/live-share/issues/2529?email_source=notifications&email_token=ALEWIJLNMDCCH3Z6LRQTDMDQADAE5A5CNFSM4IE6QJP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2JRFQI#issuecomment-512955073,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALEWIJKNVPIGPOWYGVD26ATQADAE5ANCNFSM4IE6QJPQ
.

It happens randomly.
On Thu, Jul 18, 2019, 3:30 PM Monica Williams<EMAIL_ADDRESS>wrote:

Thanks for reporting this, @BryanDungan https://github.com/BryanDungan.
Is it possible that the host ended the session before you tried to join it?
What were you doing at the time that this happened? Were you ever able to
join a session?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/live-share/issues/2529?email_source=notifications&email_token=ALEWIJLNMDCCH3Z6LRQTDMDQADAE5A5CNFSM4IE6QJP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2JRFQI#issuecomment-512955073,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALEWIJKNVPIGPOWYGVD26ATQADAE5ANCNFSM4IE6QJPQ
.

Your logs might provide us with more information on this issue. If you experience it again, can you attach them here so I can take a look? It would be best to get your logs and his. You can access them via the Live Share: Export Logs command from the command palette (ctrl + shift + p) and attach them to this issue.


Add detailed steps

Add a new page with more detailed steps about how to use this plug-in.

@cking0987 Alright, ready for merging! (I don't have permission to merge, even after approving the changes.)


"vm create" generates deployment names that are too long

When vreating VM's with names that are already quite long, the cli generates deployment names for parts of the deployment that are too long:
$ az vm create --name Canonical-UbuntuServer-14-04-5-LTS --resource-group paulmey-resourcegroup --location westus2 --size Standard_DS1_v2 --image Canonical:UbuntuServer:14-04-5-LTS:latest
At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details. {
  "error": {
    "code": "InvalidDeployment",
    "message": "The provided deployment name 'Canonical-UbuntuServer-14-04-5-LTSCanonical-UbuntuServer-14-04-5-LTSVMNicmac' has a length of '76' which exceeds the maximum length of '64'. Please see https://aka.ms/arm-deploy for usage details."
  }
}  Correlation ID: 6408ac66-4b22-4980-xxxx-8bbe4f417139

Thanks for your report @paulmey! We'll take a look and issue a fix.

@paulmey what is your output for az --version for the vm module?  The CLI generates a semi-random deployment name for VM create like "vm_deploy_f32e9e43..."  It sounds like you may have an outdated VM module.

vm (0.1.0b11)
Time to update... Thanks!

Version 2.0.1 should solve your problem. :smile:


INSERT INTO SELECT does not throw an error / warning when some inserts fail

CrateDB version
4.6.8 / 4.7.0
CrateDB setup information
No response
Steps to Reproduce
CREATE TABLE insert_into_source ( ts TIMESTAMP, ts_g TIMESTAMP);
CREATE TABLE insert_into_sink ( ts TIMESTAMP, ts_g TIMESTAMP GENERATED ALWAYS AS date_trunc('minute',ts));

INSERT INTO insert_into_source VALUES (now(),date_trunc('minute',now())+1), (now(),date_trunc('minute',now()));
--INSERT OK, 2 records affected (0.016 seconds)

INSERT INTO insert_into_sink (ts,ts_g) SELECT ts, ts_g FROM insert_into_source;
--INSERT OK, 1 record affected (0.011 seconds)

Expected Result
Statement fails with
Failed to execute upsert shardId=[insert_into_sink][0] id=XOARjn8B6RWc_1mx5EhS
error=java.lang.IllegalArgumentException:
Given value null for generated column ts_g does not match calculation date_trunc('minute', ts) =<PHONE_NUMBER>000

Actual Result
no error, just
INSERT OK, 1 record affected (0.011 seconds)
a lower number than expected

Update
This might not be a bug after all, however the situation could be maybe improved to give the user some way to check if an INSERT INTO SELECT partially failed.
INSERT INTO insert_into_sink VALUES (now(),date_trunc('minute',now())+1), (now(),date_trunc('minute',now()));

returns
Error!

SQLParseException[Given value<PHONE_NUMBER>001 for generated column ts_g does not match calculation date_trunc('minute', ts) =<PHONE_NUMBER>000]

COPY FROM has the optionRETURN SUMMARY to see how many rows failed to be copied and now also the fail_fast option to stop copy operations in best effort manner when insert error occur.

Discussion outcome: Could add a session setting that allows partial failures and causes a "fail fast" behavior (first error encountered gets propagated)

Starting from version 5.5, UPDATE statements will continue on row failures instead of always showing an error.
The affected rows are displayed by the resulting row count. Related PR

Session setting that allows partial failures and causes a "fail fast" behavior (first error encountered gets propagated)
The new session setting should behave in a similar way for UPDATE statements.

One possible option to give back a warning could be by means of a NoticeResponse with severity WARNING see https://www.postgresql.org/docs/current/protocol-flow.html and https://www.postgresql.org/docs/current/protocol-error-fields.html

We had another discussion about this and concluded that the NoticeResponse would be the nicer option.
Partly because of bulk request handling where individual "batch" error reporting would work via HTTP, but is problematic for PostgreSQL. PG/DB-API clients typically only expose a row-count array or failure, not one failure per batch.
The other main advantage of a NoticeResponse is that it can be active by default, so users can always get the additional info.
It's more effort to implement, as it also requires a HTTP response format addition and updates in crash and the admin-ui to show the warnings. It won't make it for 5.7 but I'd target this for 5.8 instead but that's TBD

Relates to https://github.com/crate/crate/issues/8312 and https://github.com/crate/crate/issues/8949

Main user facing problem:
Confusion if some rows are not inserted (for example because of constraint violation).
Current behaviour: Errors are swallowed, number of successfully inserted rows is returned.
Listing all possible options to change behaviour from the user perspective to get additional feedback:

Number of inserted rows is returned, errors aren’t swallowed but redirected to sys.operations
To lookup the errors, users need some identifier.

HTTP could include it in the response
for PG protocol we could implement session_id and users would lookup by session_id in sys.operations.
See https://github.com/crate/crate/issues/14939)

Number of inserted rows is returned and errors are emitted as warning (see comment above for details)

Number of inserted rows is returned. In case of error, show first error and short-circuit the operation (number of inserts is discarded/unknown to a user). Short circuit would be a breaking change unless it’s behind a flag/setting.

Number of inserted rows is returned. In case of error, show first error AND number of inserted rows and short-circuit the operation. Short circuit would be a breaking change unless it’s behind a flag/setting.

Introduce optional RETURN SUMMARY clause for UPDATE/DELETE/INSERT in addition to RETURNING.
Can’t use both RETURN SUMMARY and RETURNING at the same time

From: https://github.com/crate/crate/issues/16435

Problem Statement
Currently, whenever a bulk insertion partially or fully fails it is hard to understand what went wrong. For that, the recommended approach is to rerun each record individually to get further insights into what happened.
At some point, due to memory concerns, many error messages were removed, which resulted in little to none errors being reported on bulk insertions.

Possible Solutions
Use extra flags on the bulk request to adjust the error reporting behavior of the bulk operation


Use docker image with Postgresql

I've just tried setting up portus with postgresql database, and realized that the docker image does ship a different database config file, which doesn't allow setting the adapter and expects different env variables than the database.yml file in this repository. While this is not that really bad (I'll just pass through my own database.yml) it is pretty confusing, as it behaves differently than what is described in http://port.us.org/docs/database.html

Actually, this is bad. The pg gem is not bundled in the docker container (anymore?), so it won't run with a passed through database.yml.
From the docs linked above:

Note that these dependencies have already been bundled in the official Portus image.

So this again is in conflict with the official documentation.

They don't fully support Postgres yet you have to use MySQL until they sort it out there are few issues explaining why, I'm waiting for this feature as well as atm I have MySQL (for Portus) and Postgres (for Clair) running and it will be mint to have just one DB 👍

@zot24 thanks for the info, but that should still be communicated in the documentation then :D

@jcgruenhage this is due to a bug: #1418. Once we fix this (which should happen as soon as possible), you'll be able to interact with the database adapter as documented 😅

I'm waiting for this feature as well as atm I have MySQL (for Portus) and Postgres (for Clair) running and it will be mint to have just one DB

See #1499 😉 It's a breaking change (e.g. we'd have to provide a migrator from mysql to postgresql data) and we were short in time. Feel free to leave a comment in #1499 if you have any suggestions on the topic 😃
Closing this now since it's already tracked in #1418, but feel free to submit new comments here if you have more doubts 😉


Maps within wire-elements/modal

When I attempt to use laravel-maps within a wire-elements modal I get the following error.
@LarsWiegers do you have any recommendations for how to address this? I am currently just using one of the examples from the readme for testing purposes.

        <x-maps-leaflet :centerPoint="['lat' => 52.16, 'long' => 5]"></x-maps-leaflet>

The above html code is a part of the following livewire component that has the attached class.
<?php

namespace App\Http\Livewire\Modals;

use LivewireUI\Modal\ModalComponent;

class ViewMap extends ModalComponent
{
    public function render()
    {
        return view('livewire.modals.view-map');
    }
}

I am able to use this component when not including it in a wire-element/modal but I am unsure why I get this error when attempting to load the map within the modal.
Unhandled Promise Rejection: ReferenceError: Can't find variable: L

It appears that if I take the script and duplicate it right before the closing body tag I am able to get the modal to properly render without any issue.

The library does not currently support livewire, but if you got it working awesome!
To be honest i dont use livewire. But feel free to send in a pr for supporting it.

@ryangurn pls see my last message

Closing this as @ryangurn did not respond, feel free to reopen if this is something you want to move further.


FSDP training raise "KeyError: 'ShardingStrategy.NO_SHARD'"

System Info
torch                         2.0.1
torchaudio                    2.0.2
torchvision                   0.15.2
Information

[ ] The official example scripts
[ ] My own modified scripts

🐛 Describe the bug
Hi, i can train the asr_librispeech finetuning code use DDP, however, when i switch to FSDP, an exception raised.
Error logs
Traceback (most recent call last):
File "examples/asr_librispeech/finetune_asr.py", line 41, in main_hydra
train(kwargs)
File "/SLAM-LLM/src/slam_llm/pipeline/finetune.py", line 167, in main
model = FSDP(
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/fully_sharded_data_parallel.py", line 391, in init
_auto_wrap(auto_wrap_kwargs, fsdp_kwargs, FullyShardedDataParallel)
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/_wrap_utils.py", line 73, in _auto_wrap
_recursive_wrap(**auto_wrap_kwargs, **fsdp_kwargs)
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/wrap.py", line 370, in _recursive_wrap
wrapped_child, num_wrapped_params = _recursive_wrap(
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/wrap.py", line 370, in _recursive_wrap
wrapped_child, num_wrapped_params = _recursive_wrap(
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/wrap.py", line 370, in _recursive_wrap
wrapped_child, num_wrapped_params = _recursive_wrap(
[Previous line repeated 3 more times]
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/wrap.py", line 388, in _recursive_wrap
return _wrap(module, wrapper_cls, **kwargs), nonwrapped_numel
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/wrap.py", line 317, in _wrap
return wrapper_cls(module, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/fully_sharded_data_parallel.py", line 408, in init
_init_param_handle_from_module(
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/_init_utils.py", line 429, in _init_param_handle_from_module
_init_param_handle_from_params(state, managed_params, fully_sharded_module)
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/fsdp/_init_utils.py", line 529, in _init_param_handle_from_params
SHARDING_STRATEGY_MAP[state.sharding_strategy],
KeyError: 'ShardingStrategy.NO_SHARD'
Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 2031656) of binary: /usr/bin/python3
Traceback (most recent call last):
File "/usr/local/bin/torchrun", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/elastic/multiprocessing/errors/init.py", line 346, in wrapper
return f(*args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/run.py", line 794, in main
run(args)
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/run.py", line 785, in run
elastic_launch(
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/launcher/api.py", line 134, in call
return launch_agent(self._config, self._entrypoint, list(args))
File "/usr/local/lib/python3.8/dist-packages/torch/distributed/launcher/api.py", line 250, in launch_agent
raise ChildFailedError(
Expected behavior
How to modify to use FSDP for speeding up? Thanks a lot! :D

Have you solved it?

Also run into the same issue, tried to modify:
sharding_strategy: str = "NO_SHARD"
to
sharding_strategy: str = "FULL_SHARD"
raised error message : "KeyError: 'ShardingStrategy.FULL_SHARD'"
Any idea how to solve it ?

Maybe you guys can try to comment this line in finetune.py:
sharding_strategy=fsdp_config.sharding_strategy
The reason might be an extra blanket or some operators when we transfer the parameters.
And the defualt value for sharding_strategy is actually FULL_SHARD
But I am not sure whether it will keep well in training since I just trained the loss to nan.

Any progress on that?


[Feature][JsonSplit-api]processDefinition view-tree interface

from #5498
Change the request parameter processDefinitionId to processDefinitionCode,including the front end and controller interface

I will implement the back-end java code logic


Issues setting up git authentication with github CLI

Platform or tool
GitHub
Desired or expected behavior
I authenticated my github CLI via op's GitHub plugin, using the Personal Access Token stored in my vault. It works fine when I call gh manually; however, when I try to authenticate git using gh (I wanted to use Personal Access Token for authenticating git, as I was trying to push a commit with a github workflow change), it failed (see below).
This was even after I ran gh auth setup-git, which I understand is a way for git to authenticate directly using gh CLI (which is now authenticated by the op plugin) without having to resort to git-credentials-manager.
When I inspected git settings, turns out the issue was that git was trying to use credential helper of !/opt/homebrew/bin/gh auth git-credential (default behaviour of running git auth setup-git), which was using gh without the aliasing that the op plugin sourcing does!
So I ended up having to manually set the credential helper as gh auth git-credential, but it doesn't really sit right with me.

Is this the "right way" to authenticate git and gh?
If so, could this be better documented? I looked all around in the 1pw dev docs and I didn't see anything about authenticating git itself (which is super important if you want to use your PAT to authenticate).

Thanks.
Current behavior
! [remote rejected] master -> master (refusing to allow an OAuth App to create or update workflow `.github/workflows/ci.yml` without `workflow` scope)

Relevant log output
No response
op CLI version
2.13.1

Hi @JaneJeon, I am sorry we missed this earlier.
I stumbled upon this issue when investigating a different one. You are right in that the path should be op plugin run -- gh in .gitconfig so that it uses the 1Password-stored secret. And I hear that this manual editing isn't elegant. I'll raise this internally, thank you for reporting!

Thank you. For the moment, I've simply resorted to using an SSH key for authentication and signing w/ github.

Thanks, but as I've said above, I've currently resorted to using 1P + GH over SSH, and I'd much rather have proper gh/git authentication support baked with 1p CLI.

Assuming you don't have GitHub token stored on the Apple Keychain and have it only on the 1Password vault, you can replace !/opt/homebrew/bin/gh with op plugin run -- gh on .gitconfig. That seems to do the trick for me.

gh has a command gh auth setup-git which puts in the credential helper entries. It would be cool if when aliased via op that the call for gh auth setup-git would produce the op run version instead. Since you are wrapping gh, can you just intercept and answer this directly without letting it go to gh? Also could intercept login, logout, and refresh and give a message that 1Password is managing the GitHub auth.


[vault] out of etcd migration example

https://github.com/improbable-eng/vault-kv-extract

https://www.vaultproject.io/docs/commands/operator/migrate.html

For this the https://www.vaultproject.io/docs/commands/operator/migrate/ is the right solution.


The "Reactiveusercontrol" name does not exist in the  namespace "Http://reactiveui.net"

We will close your issue without comment if you delete or do not fill out the issue checklist and provide ALL the requested information. This repository's issues are reserved for feature requests and bug reports. For support questions, please ask on StackOverflow: https://stackoverflow.com/questions/tagged/reactiveui
Please delete this note before pressing the submit button
Do you want to request a feature or report a bug?
 ReactiveUserControl

<rxui:ReactiveUserControl
x:Class="views:YourView"
x:TypeArguments="vms:YourViewModel"
xmlns:rxui="http://reactiveui.net"
xmlns:views="clr-namespace:Foo.Bar.Views"
xmlns:vms="clr-namespace:Foo.Bar.ViewModels">

</rxui:ReactiveUserControl>

What is the current behavior?
I can't use xaml syntax rxui:ReactiveUserControl implement my Wpf View.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
this is AppView.xaml code:
<rxui:ReactiveUserControl
        x:Class="views:AppView"
        x:TypeArguments="vms:AppViewModel"
        xmlns:rxui="http://reactiveui.net"
        xmlns:views="clr-namespace:Love.Wpf.Views"
        xmlns:vms="clr-namespace:Love.Wpf.ViewModels"
             Height="450" Width="800">
    <!--<Grid>

    </Grid>-->
</rxui:ReactiveUserControl>

this is AppView.xml.cs code:
    public partial class AppView : ReactiveUserControl<AppViewModel>
    {
        public AppView()
        {
            InitializeComponent();
        }
    }

What is the expected behavior?
XDG0008 The "Reactiveusercontrol" name does not exist in the  namespace "Http://reactiveui.net". LOVE.WPF Appview.xaml

XDG0008	命名空间“http://reactiveui.net”中不存在“ReactiveUserControl”名称。	Love.Wpf	AppView.xaml

XLS0507 The type "Reactiveusercontrol" cannot be used as an object element because it is not public, or there is no public parameterless constructor or type converter defined. LOVE.WPF Appview.xaml

XLS0507	类型“ReactiveUserControl”不能用作对象元素，因为它不是公共的，或者未定义公共无参数构造函数或类型转换器。	Love.Wpf	AppView.xaml

What is the motivation / use case for changing the behavior?
I don't know..
Which versions of ReactiveUI, and which platform / OS are affected by this issue? Did this work in previous versions of ReativeUI? Please also test with the latest stable and development snapshot
ReactiveUI.Wpf:8.3.1
VS2017:15.8.0 Preview 2.0
Other information (e.g. stacktraces, related issues, suggestions how to fix)
Error XDG0006 namespace prefix "x" has not been defined.	LOVE.WPF Appview.xaml 2

 Error XLS0413 the attribute ' Class ' was not found in type ' Reactiveusercontrol '.
LOVE.WPF Appview.xaml 2

错误	XDG0006	命名空间前缀“x”尚未定义。	Love.Wpf	AppView.xaml	2

错误	XLS0413	未在类型“ReactiveUserControl”中找到属性“Class”。

X:class= "Views:appview" is invalid. ' Views:appview ' is an invalid class name.
Error CS0103 the name "InitializeComponent" LOVE.WPF error does not exist in the current context XDG0008 namespace "clr-namespace:love.wpf.viewmodels" does not exist Appviewmodel "name.
The severity code indicates that the project file line suppresses display status
Error XLS0301 the character sequence "views:" should not be in this position.
x:Class=“views:AppView”无效。“views:AppView”是无效的  类名称。
错误	CS0103	当前上下文中不存在名称“InitializeComponent”	Love.Wpf
错误	XDG0008	命名空间“clr-namespace:Love.Wpf.ViewModels”中不存在“AppViewModel”名称。
严重性	代码	说明	项目	文件	行	禁止显示状态
错误	XLS0301	在该位置不应为字符序列“views:”。

this is AppView.xaml code:

<rxui:ReactiveUserControl
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
x:Class="views:AppView"
x:TypeArguments="vms:AppViewModel"
xmlns:rxui="http://reactiveui.net"
xmlns:views="clr-namespace:Love.Wpf.Views"
xmlns:vms="clr-namespace:Love.Wpf.ViewModels"
Height="450" Width="800">
<!--
</Grid>-->

</rxui:ReactiveUserControl>

Your x:Class looks wrong, should be something like
x:Class="NameSpace.To.View"

This is unrelated to the reactive user control so closing this issue

@glennawatson
so, ReactiveUserControl api doc is error? Can I fix?

@glennawatson
thanks a million.


Fix License Headers

The headers for license files are currently EPL - either change the target license, or update the headers

Per this discussion: https://opensource.stackexchange.com/questions/4960/can-a-team-be-the-copyright-holder-mit, it appears "The Corona IDE@GitHub Authors" would be an acceptable copyright header

Updated to the MIT license - headers, documentation, and distributions updated


[FSunstone] Remote Authorization Support in FireEdge Sunstone

Description
FireEdge Sunstone needs to exhibit support for Remote Authentication In order to integrate with external authentication systems (such as Kerberos or SAML based ones like Shibboleth). This will allow the deployment of FireEdge Sunstone in environments where a user can be authenticated in several services, including FireEdge Sunstone, using a Single Sign On mechanism.
Use case
Remote Authorization in FireEdge Sunstone assumes that users are precreated in OpenNebula using the ‘public’ authorization driver. These users should exist in the OpenNebula DB for the authentication to succeed, regardless of the authentication coming from the SSO auth backend.
The interaction with the remote authentication backend (SAML Identity Provider, IdP) is delegated to a higher level proxy, like for instance apache httpd shibboleth module. Upon successful authentication the proxy must pass onto FireEdge Sunstone the username as a header (X_AUTH_USERNAME) or environment variable (REMOTE_USER).
Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

@tinova PR to approve:

[CODE] one-6.6: https://github.com/OpenNebula/one-ee/pull/2558
[CODE] master: https://github.com/OpenNebula/one-ee/pull/2557


Add a quirk for google-cloud-bigquery

On conda-forge, the package google-cloud-bigquery is quite bloated because it contains all the extras.  The pypi package google-cloud-bigquery is instead google-cloud-bigquery-core on conda-forge:
https://github.com/conda-forge/google-cloud-bigquery-feedstock/blob/main/recipe/meta.yaml#L70

Codecov Report

Merging #396 (63b0231) into main (9c3242e) will increase coverage by 0.70%.
The diff coverage is 92.82%.

@@            Coverage Diff             @@
##             main     #396      +/-   ##
==========================================
+ Coverage   91.30%   92.01%   +0.70%
==========================================
  Files          17       16       -1
  Lines        1576     1703     +127
==========================================
+ Hits         1439     1567     +128
+ Misses        137      136       -1

Flag
Coverage Δ

unittests
92.01% <92.82%> (+0.70%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

setup.py
0.00% <0.00%> (ø)

grayskull/base/github.py
72.83% <72.83%> (ø)

grayskull/license/discovery.py
94.87% <78.12%> (-2.28%)
:arrow_down:

grayskull/strategy/abstract_strategy.py
83.33% <83.33%> (ø)

grayskull/__main__.py
95.60% <93.18%> (-2.55%)
:arrow_down:

grayskull/base/factory.py
91.17% <93.33%> (+1.17%)
:arrow_up:

grayskull/strategy/py_base.py
95.00% <95.00%> (ø)

grayskull/config.py
96.96% <96.96%> (ø)

grayskull/utils.py
98.12% <97.58%> (-1.88%)
:arrow_down:

grayskull/strategy/pypi.py
97.68% <97.68%> (ø)

... and 7 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add web endpoint to the inference server

This lets you do
modal serve src.inference
in one window, and
curl 'https://modal-labs--example-axolotl-inference-web-dev.modal.run?input=what+time+is+it' in another window
it streams!
Note that most of the complexity here is that we want to support an unparametrized class constructor, so we need to pick the run id automatically. In order to do that it was slightly easier to change the argument to be run_name rather than the full path. If run_name isn't provided (which it never is in the case of web endpoints, since we don't support parametrized functions with web endpoints), then it just picks the last run by doing a directory listing.

I think the run_name change makes sense even just on its own but can we update the readme too?


Account linking stopped working with the new Facebook limited login

Description
Recently https://github.com/facebook/facebook-ios-sdk created a new release which conforms with Apple Privacy Manifest but now the token returned from the limited login no longer works to link a user to an account.

android works correctly

(on IOS - FB LIMITED LOGIN) For example if i login with my gmail and then i login with my facebook which uses the same email and want to link with credential i receive this error: Duplicate credential received. Please try again with a new credential
Reproducing the issue
No response
Firebase SDK Version
10.26
Xcode Version
15.4
Installation Method
CocoaPods
Firebase Product(s)
Authentication
Targeted Platforms
iOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

Thanks for reporting, @Daniel3711997. I tried linking a Google account to a Facebook account with the same email, but I wasn't able to get the duplicate credential error. I just want to clarify if you're trying to link the Facebook account to an email/password or to a Google account through Google Sign-In?

i got the duplicate error after i logged in with google signin and after that tried to link the facebook account

Thanks for clarifying, @Daniel3711997. I tried to link the Google Sign-In credential to Facebook account, but the issue is not reproducible. Is it possible for you to share an MCVE or detailed steps to reproduce the issue?

the issue still persists, link the facebook account to a google account (but with limited login from Facebook)

Hello, This is Shohag. I am a professional Ethical Hacker,
Security Specialist as well as Penetration Tester. I can test the following vulnerability and I will provide you with a perfect assessment of your website security
which makes your website safe and sound.
My Experience:
💗. Recovering Hacked WordPress Website
💗. Full Vulnerability Scan and WordPress website malware removal
💗. Vulnerability Assessment
💗. Antispam protection
💗. Analysis of database servers
💗. Restore a hacked website
💗. Ethical Hacking(White Hat Hacker)
💗. Hacked account recovery
💗. Password recovery
💗. Decode encrypted files
💗. Vulnerability Testing
💗....my WhatsApp number +8801315423936

@google-oss-bot, recently I have integrated Facebook limited login now I am getting error (Duplicate credential received. Please try again with a new credential.) with firebase auth Facebook limited login. Please help me to fix this error.
let idTokenString = AuthenticationToken.current?.tokenString
let nonce = self.generateNonce().sha256()
self.authCredential = OAuthProvider.credential(
withProviderID: SignInMethod.facebook.provider,
idToken: idTokenString!,
rawNonce: nonce
)
guard let authCredential = authCredential else {
return
}
Auth.auth().signIn(with: authCredential) { (user, error) in
if let error = error {
print(error.localizedDescription)
self.handleAuthError(error: error, signInMethod: signInMethod)
}
}

@Hiteshboricha-Pomelo, may I know if you're generating a new nonce every sign in? You may also check this related issue.

yes I was generating a new nonce every sign in. I read the document again for limited login and now it's fixed. Issue with Nonce only. Thanks for the quickly response

still happening

@Daniel3711997 please provide a sample app that can reproduce the issue to investigate.

@google-oss-bot, recently I have integrated Facebook limited login now I am getting error (Duplicate credential received. Please try again with a new credential.) with firebase auth Facebook limited login. Please help me to fix this error.
let idTokenString = AuthenticationToken.current?.tokenString let nonce = self.generateNonce().sha256() self.authCredential = OAuthProvider.credential( withProviderID: SignInMethod.facebook.provider, idToken: idTokenString!, rawNonce: nonce )
guard let authCredential = authCredential else { return } Auth.auth().signIn(with: authCredential) { (user, error) in if let error = error { print(error.localizedDescription) self.handleAuthError(error: error, signInMethod: signInMethod) } }

he already provided one

@Daniel3711997, I tried to login using my Google account and then link to my Facebook account with the same email using Facebook limited login by following the steps here, but it's not reproducible on my end. Could you confirm if we're on the same page? Also, could you provide your Podfile.lock file?


added pulls option

See also https://plothist.readthedocs.io/en/pulls/advanced/advanced_examples.html
Will also add some documentation example with compare_two_hist and one with pulls displayed.

Should compare_data_mc()  be renamed to account the fact that the mc_hist_list argument is a list of mc_contributions (and not a simple mc hist, as we could expect from compare_data_mc)?

compare_data_mc() uses compare_two_hist(), all the comparison is now computed in compare_two_hist() and nothing in compare_data_mc(). Needs to check closely the definitions of the uncertainties.

Should compare_data_mc() be renamed to account the fact that the mc_hist_list argument is a list of mc_contributions (and not a simple mc hist, as we could expect from compare_data_mc)?

The list can contain only one component if needed, so I am not sure we need to change the name

Every points of the bike discussion have been addressed.


Fix handling of multiple statements.

When a query contained multiple statements any statement which did not
return data (e.g. SET, CREATE DATABASE, etc) would be ignored unless
it was the last statement.

This needs a test.

Test added.

LGTM


Keep display awake when webcam is in fullscreen

Hey,
first of all I really love this app.
One thing that annoys me a little bit is the fact that the display doesn't stay on when I fullscreen the webcam. Is this something you would add as a feature?

Hi @vordenken ,
Under Settings --> Appearance you can disable Auto-Lock when idle and the app will not let the iOS turn off the screen when the app is used. It is not the exact same thing you asked but the closest.
Would this solve your need?
Regards,
Gaston

Hi @gdombiak ,
thanks for your answer. I know that this is a feature in iOS but this would mean my phone wouldn't auto-lock at all.
What I mean is that the display only been kept on while displaying the webcam in OctoPod. Similar to when watching Youtube, Netflix or any other video-type content.

hey @vordenken ,
I'm referring to a setting inside of OctoPod itself. Go to Settings --> Appearance in OctoPod app to control this setting. Let me know if this would help with your request. I still see value in your request since that could help others with the same request.
Thanks,
Gaston


Improve build time

Description:
Improve the build time of the EngEd repo.

Converted the following partials to partialCached components reducing the build time and number of times each partial is processed/computed:

footer.html
header.html
eng-ed-banner.html
footer-scripts.html
head-scripts.html

Added test-build.yml GitHub action to test the build on a PR to master.

How to Test:
Compare results of running hugo --templateMetrics --templateMetricsHints on both master and this branch locally. Build time results will vary with each build. However the results should show a decrease in build time as well as a decrease in the count column for many of the partials with a high cache potential score.
Initial build time report:

Improved build time report. reduced build by about 1 second:

🚀


Incorrect union output

Not 100% sure but looks like a bug, Clipper1 gave correct result on this input.
The problem is that I am trying to simulate SimplifyPolygons function from Clipper1, and I see that some input edges are missing from the output.
Pay attention that the input contour is self-overlapping on problematic edges.
TEST(Clipper2Tests, IncorrectUnion)
{
	Clipper2Lib::Paths64 paths = {
		{{-712628032, -796916864}, {712628160, -796916864},
		 {712628160, 482344832},  {-712628032, 482344832}, {-712628032, -691533376}},
		{{-148008768, -708073600},
		 {-151864192, -700430272},
		 {-148008832, -708073600},
		 {-146168576, -711381632}},
		{{-94181952, 251733760}, {-92439040, 251733760}, {-89137664, 253527744},
		 {-87006208, 253527744}, {-85212224, 255321728}, {-82534848, 255321728},
		 {-79233472, 257115648}, {-74448576, 257115648}, {-72654592, 258909632},
		 {-69329280, 258909632}, {-72654720, 258909632}, {-74448704, 257115648},
		 {-79233472, 257115648}, {-74448640, 257115648}, {-72654656, 258909632},
		 {-61891136, 260703488}, {-61891072, 260703488}, {-61414400, 261180160},
		 {60994224, 262497536},  {62311488, 261180160},  {62788156, 260703488},
		 {68170112, 260703488},  {69963968, 258909632},  {73551808, 258909632},
		 {75345792, 257115648},  {79233600, 257115648},  {75345856, 257115648},
		 {73551872, 258909632},  {73551808, 258909632},  {75345792, 257115648},
		 {79233600, 257115648},  {82534976, 255321728},  {86109376, 255321728},
		 {87903360, 253527744},  {91491304, 253527744},  {92439168, 252579840},
		 {95556928, 251256064},  {97052960, 249760096}}};

	paths.push_back(paths[0]);
	paths.push_back(paths[1]);
	paths.push_back(paths[2]);
	Clipper2Lib::Clipper64 c;
	c.AddSubject(paths);
	Clipper2Lib::Paths64 result;
	c.Execute(Clipper2Lib::ClipType::Union, Clipper2Lib::FillRule::Positive, result);

        ASSERT_EQ(result.size(), 2); // Returns 3 contours instead of two
	/*
        // The following code generates expected output on the same input in Clipper1,
        // If I am not mistaken it is equivalent to the Clipper2 code from above
        ClipperLib::Paths c1Paths;
	for (auto& path : paths)
	{
		c1Paths.push_back({});
		for (auto& point : path)
		{
			c1Paths.back().emplace_back(point.x, point.y);
		}
	}
	ClipperLib::SimplifyPolygons(c1Paths, ClipperLib::pftPositive);
       */
}

Clipper1 result:

I can't duplicate this. Please retest with the latest commit.

Thanks!
Seems to be fixed by #720


Fix for IE, to make it stable.

When the event is sent (ex. start dragging) slow browser (for example IE) is still didn't react on it and we send another event. So we need to wait a bit. No idea how to get the information that event is processed. So we wait for 0.5 s. For me it works all the time now.
I'm using Selenium Grid infrastructure with  IE 11 slaves on Win 7 VMs.

Could you unify indent to 2 width space, please?
Because it is hard to see what is changed.
(I know, ?w=1 hack, but we cannot comment to changes when use this hack)

Sure, have done it. Sorry IntelliJ IDEA has 4 spaces for tab by default.

@acierto Thanks! I have a question.
Did you try shorter wait durations?
I think shortest one is the best because it make test fast.

You can make those values configuration in html-dnd, but not waiting at all makes this lib very flaky for the usage.

Hi man, do you have any comments regarding PR to merge it?
The tests are not failing anymore after the last change, hopefully you will accept the PR and we can use your library directly instead of own fork!
If you have any comments, please let me know

+1. It works flawlessly.

We should make test pass to keep stable.
So, I'm thinking about how to make it be testable.
Please wait few days.

Let me clarify what you mean, as the first sentence doesn't make any sense for me.
Tests are already stable and they pass.
Do you want extra tests to reveal the current issue with IE e2e tests and then verify with this PR that the problem will be resolved?

Oops, I missed the CI status... I'm sorry.
Now, I'm reviewing the patch again.

Fair enough, what do you think about it?
https://github.com/zengfenfei/delay

I think that using "Execute Async Script" API is the better way.
How do you feel it?

I'm gonna send new Pull request for your branch about it.

If it would be the single change how will it resolve the issue? As still 4 events won't wait for each other.
Basically with
// This is the only change: executeScript -> executeAsyncScript
driver.executeAsyncScript(dragAndDrop, draggable, droppable);

you guarantee that the end result will be synchronised, but the problem is also happening if another event is triggered with not waiting till the first one finished.
We can fix it or with callback hell as it is now, or with Promises, what will look better and as an end accord we can use executeAsyncScript to get rid of sleep.

I sent you an invitation in GoogleChat, maybe we can faster discuss it there.

I sent you an invitation in GoogleChat, maybe we can faster discuss it there.

Hmm, I feel it is hard because I'm not good at speaking English.
I like text-based conversation because some translation tools can help me.


not working for windows

when emmet is already installed on windows 8, I write div.row , press Tab and it'snot working, thank you.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/23688494-not-working-for-windows?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github).

I'm not sure but I think this is a general issue with emmet on atom, did the autocomplete-plus API change?
I'm having the same issue in Arch Linux with latest atom, packages

I have the same problem, doesn't work with IDs neither.
e.g. Pressing tab after writing "div#mydiv" just adds a tab.

Make sure your document syntax is set to HTML

Thanks, that worked! It was a PHP file, I'll check if there's a plugin that allows PHP and HTML syntax at the same time.


TCP RTT in Frame metadata

To enable more accurate latency tracking than simple ICMP echo requests, add the TCP RTT for each packet to the encapsulating Frame. This could get a bit weird with retransmitted Frames but still useful.

This is actually way more complex than I thought, as the TCP context isn't passed through to the reassembler. Not actually sure the best way to do this other than either ignoring reassembly like Machina, or maybe implementing a custom protocol layer for Frames, but no clue how TCP reassembly would work with that.


Proofread and copyedit the spec

A native English speaker with specification-writing experience needs to spend some time rewording the specification.

I'm informed by @maxandersen that this issue report could be taken the wrong way. To clarify: as a native speaker with lots of experience at writing specs, I am of course volunteering to do this work.

Hey, I got your point, and you are right; there are a lot of issues for improvement. I have almost zero experience writing a spec from scratch and am not a native speaker.
Thank you. I will need as much help as needed, mainly because I do it in my free time as a community person.

Additional work that is needed under this issue:

[ ] The Copyright section has a number of variables that are not being substituted when I build locally, as well as in the officially-built M1 spec doc, https://jakarta.ee/specifications/data/1.0/data-1.0.0-m1.
[ ] Section 1.3. Conventions is empty. We should consider removing it unless there are plans to author something for that section.
[ ] Section 1.4. Jakarta Data Project Team should be updated to include the names of people who have made significant contributions to the spec. We should consider voting in some of these people as committers and add them under that section as well.


Suspected Memory Leak in Ressource

The Issue
When using sampleTerrainMostDetailed often (thousands of times), the memory usage slowly rises.
Inspections
In node this leads to a large external memory usage over time. We managed to replicate the problem in the browser (see this sandcastle gist). This leads to the following profile over 90 seconds:

For some reason, the listeners keep on rising, we are currently trying to get to the bottom of this and would welcome any input.

Hi,
we have done some further investigation:
The Problem seems to be the new Ressource Class, we changed the CesiumTerrainProvider to use fetch instead of the Ressource class, and the Problem seems to be gone, for sampleTerrainMostDetailed.
See : https://github.com/virtualcitySYSTEMS/cesium/commit/293cdde65b16d22966bef61c64fec5bf1a188ed5

Your sample code never runs the render loop, which means that RequestScheduler.update() is never called, which is the function that cleans up activeRequests.

@shunter Thanks, when i add RequestScheduler.update() after each sampleTerrain call the memory footprint keeps stable. Maybe this could be documented for methods which can be used without the render loop?

Could there be a way for the RequestScheduler to auto update in node environments? Running cesium headless and having the need to clean up after it manually seems unintuitive.

I somehow missed this issue the first time around but @kring just ran into it.
Couldn't we just have RequestSchedule run update on itself no more than once a frame?  Rather than installing a constant even into the event loop, we could have it use process.nextTick and only install the one time handler after a request has resolved.
I may not be explaining it well, but I think this is an easy fix.


Max Child Blocks

We can limit blocks by type but it would be nice to limit child blocks to a set number to allow a choice of which blocks they want to use.
Currently if I have 3 child blocks I can set a max of 3 to each meaning they can get up to 9 child blocks. If I set 1 max to each type they can only use 3 but have to use one of each.

I agree, this would be a useful feature. As mentioned in #40, there's a lot of other rule-based options that would be useful too, so I've been thinking of ways to incorporate them all in a UI-friendly way, which is why I haven't added this one in yet. But it comes up a lot so I'll add it in sooner than later.

Now implemented in 1.4.0 which is currently still in testing. If you're interested you can grab the early release from the master branch.

Nice will do next week.


NeighborArray is now fixed size

Remove the ability for NeighborArray to grow. Always allocates it one larger than requested, to reserve space for a temporary neighbor.

Fixes https://github.com/apache/lucene/issues/11783

I was thinking a better approach would be to leave it to uses of NeighborArray to define maxSize, and not add +1 in the NeighborArray class itself as this PR suggests

I guess I was thinking that since this class only has a single use, it wouldn't matter? But it definitely is better encapsulation to move the sizing logic to the place where we know how many we need. +1 to have consumers do it, especially since at least in one place they already do :) I'll follow up with a patch

Also -- now that I see this I realize that most likely we are never exercising this resize capability, so removing it won't really help performance / memory usage as I was hoping. But it still seems like a good cleanup?

Wow, lots of fun discussion here, including specifics of how Java conditionals are evaluated.  @msokolov is this still relevant?  The HNSW code has been red-hot lately; maybe this change was already effectively done?

Thanks @msokolov.  This looks like a nice tool, helpful for giving demos of cool Lucene features at conferences, but it looks like consensus is we should not add it to Lucene?  Maybe luceneutil instead?


Hardware revision register

It looks like we don't have a register for reading the hardware version yet but we should probably have one, e.g. to populate the device id used during bootstrap and reporting purposes. context
lc_ctrl seems like a good place for this register.

yeah sorry, i completely forgot something like this is needed. We need to put it "somewhere".
Life cycle makes sense, as it's somewhere we can read out along with device ID.
@moidx
@msfschaffner
@cindychip
I think how this will work is that it will be defined as a top level constant that is then fed into places.
So one would be lc_ctrl, and we can feed it into other places as needed.
Any thoughts @msfschaffner ?

This would be easy to add, and LC makes sense since this will allow us to read it even in locked / raw life cycle states (via the LC TAP) where the processor cannot run.
We would have to coordinate with DV, though, since this will need a minor update to the DV env.
@cindychip this should not be much work, right?
CC: @NigelScales


Fix incorrect project path resolver

Address:

https://github.com/omnisharp/omnisharp-vscode/issues/453
https://github.com/OmniSharp/omnisharp-roslyn/issues/587

/cc @david-driscoll @DustinCampbell

Since this is blocking scenario in VS Code. I will not wait for a sign off. Please feel free to comment even after I merge this change, LGTM (pat on my own back)


Link to "Intranet lifecycle" is broken

[Enter feedback here]
Link to Intranet lifecycle is broken: https://raw.githubusercontent.com/MicrosoftDocs/OfficeDocs-SharePoint/live/SharePoint/SharePointOnline/spodownloads/Intranet%20lifecycle.pdf
guessing it should be this one?: https://raw.githubusercontent.com/MicrosoftDocs/OfficeDocs-SharePoint/public/SharePoint/SharePointOnline/spodownloads/Intranet%20lifecycle.pdf

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

ID: 4ab8a948-7330-5238-1c04-4a95db4cda5d
Version Independent ID: 4597b4bb-3cc6-99b2-ca68-2006cb7c73a5
Content: Intelligent intranet overview - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/intelligent-internet-overview.md
Service: sharepoint-online
GitHub Login: @MattEEvans
Microsoft Alias: matteva

Thanks for your feedback!

Link was fixed


Don't run installation checks

I don't think we want to remove them completely. We just should run them on merge


limitation: Secure Boot is not supported using custom kernel (in this case, fsync)

You need to use kernel signer action from EyeCantCU to achieve that.
For now, please note that limitation in README.
Relevant issue:
https://github.com/blue-build/modules/issues/148

yooo thats why it didnt work! Ill revert my revert with the proper github action now! thank you very much!

Heyo! Now that I've changed the main build action to a custom one for metadata, we can integrate the kernel signer!

This issue should be fixed now! I just need to test this out on a VM to make sure this works first

This issue is blocked in upstream due to the way rpm-ostree cliwrap install-to-root / is handled in bluebuild-cli for installing custom kernels.
Related issue:
https://github.com/blue-build/cli/issues/119

This issue should be fixed now with the newest bluebuild CLI! ill be testing this out and checking if this works

This issue should be fixed now with the newest bluebuild CLI! ill be testing this out and checking if this works


Warning: the gem 'rails-assets-angular' was found in multiple sources.

After running bundle
Warning: the gem 'rails-assets-angular' was found in multiple sources.
Installed from: https://rubygems.org/
Also found in:
  * http://rails-assets.org/
You should add a source requirement to restrict this gem to your preferred source.
For example:
    gem 'rails-assets-angular', :source => 'https://rubygems.org/'

And here is mysterious rails-assets-angular gem on rubygems.org
https://rubygems.org/gems/rails-assets-angular

https://github.com/rubygems/rubygems/issues/1148

It's known issue. We notified about it on our Twitter: https://twitter.com/railsassets
We requested forced yank of rails-assets-angular gem in https://github.com/rubygems/rubygems.org/issues/857
Thus issue is connected with bundler CVE: http://bundler.io/blog/2014/08/14/bundler-may-install-gems-from-a-different-source-than-expected-cve-2013-0334.html
The solution is the same:

Use rails-assets gems in source blocks of Gemfile, like described on https://rails-assets.org
Remove manually all previously installed rails-assets gems. After installation bundler cannot differentiate between different sources. It's described in https://github.com/bundler/bundler/issues/3360

rails-assets-angular gem has been yanked, but still you need to use block source syntax (CVE).


Bind keys to arbitrary commands

So, for instance, I could do something like this:
  - { key: N, mods: Control|Shift, command: "alacritty" }

And then press ^+shift+n to spawn a new terminal

I think we have another issue for this somewhere... can't find it though. This feature is definitely something we plan to implement.


spring-cloud-starter-dubbo

dobbo is 2.7.8   always  No provider available from registry
After restarting the service

try 2.2.6.RELEASE ？

try 2.2.6.RELEASE ？

不行 试了

try 2.2.6.RELEASE ？

不行 试了

No, I tried that


volumes are not mounting properly

using the machine resource, I define volumes such as:
:volumes => ["/Users/testuser:/home/testuser"],
The docker run command that is issued does mount the volumes (if I run it manually it works), but then in the DockerContainerMachine::converge function the container is renamed and the volumes are lost (the directory exists but is not mounted, no files)

I'll also note that I think it was broken in this commit https://github.com/chef/chef-provisioning-docker/commit/07753c4520d3364124dbdca434cd102e471b8bb4#diff-82b3f52918fe92819012dcf0d787c347
If I change the code back to transport.execute it seems to keep the volumes mounted

At this point I can confirm when transport.container.commit is called none of the options are getting passed, and subsequently when image.run is called none of these options exist on the container, therefore the volumes are missing
I am not sure on the correct fix, I think we want to invoke Docker::Image.create with the correct options OR just call transport.execute as it was doing previous to the change above.  Look forward to hearing thoughts

this works on the new master, however, there is another semi-related bug I'll submit a patch for


read_parquet with arrow_to_pandas can cause incorrect _meta

Describe the issue:
If users provide a arrow_to_pandas argument that affects the dtypes, the Dask DataFrame's _meta will be incorrect.
Minimal Complete Verifiable Example:
>>> import pandas as pd
>>> import dask.dataframe as dd
>>> df = pd.DataFrame({"A": [pd.Timestamp("2000-01-01")]})
>>> df.to_parquet("/tmp/test.parquet")
>>> ddf = dd.read_parquet("/tmp/test.parquet", arrow_to_pandas={"timestamp_as_object": True})
>>> ddf.A.dtype
dtype('<M8[ns]')
>>> ddf.compute().A.dtype
dtype('O')

I don't know exactly how _meta is constructed from the parquet / arrow metadata, but if there's call to .to_pandas() maybe fixing this is as easy as including the user-provided argument.

II think this is something which came up last year: https://github.com/dask/dask/issues/7219 .   We could do some kind of additional reseting of meta in https://github.com/dask/dask/blob/08db480f9387aab3c93c4f256b7c4c726a7314fb/dask/dataframe/io/parquet/core.py#L473-L495

We can certainly fix this by updating _create_dd_meta for the arrow engine to use _arrow_table_to_pandas