Fix behavior of getElement* methods

Makes it so you can pass them to map()
Makes the document ones start from the document, not the body

For (2), instead of passing the start node as the second argument, the methods assume this is the start node. As it turns out, removing the second argument also fixes (1), because it no longer conflicts with the map index variable.
This does change the API but I dunno if anyone was legitimately using the parent arguments since they wouldn't have worked in the browser...

Lgtm. Consider putting the methods in Dom-element.js instead of seperate files

Feel free to merge

Moved to dom-element.js :+1:
$ git diff master... --stat *.js
 document.js    | 67 ++++++++-----------------------------------------------------------
 dom-element.js | 30 ++++++++++++++++++++++++++++--
 2 files changed, 36 insertions(+), 61 deletions(-)


Socket Hang Up error

Need to investigate what causes this error:
Upload failed!
socket hang up
Error: socket hang up
   at createHangUpError (_http_client.js:302:15)
   at TLSSocket.socketOnEnd (_http_client.js:394:23)
   at emitNone (events.js:91:20)
   at TLSSocket.emit (events.js:186:7)
   at endReadableNT (_stream_readable.js:974:12)
   at _combinedTickCallback (internal/process/next_tick.js:74:11)
   at process._tickCallback (internal/process/next_tick.js:98:9)

I haven't been able to reproduce the socket hang up error. However when running the tool on a repository with a large number of files, ZendeskClientWrapper will often need to retry uploads because it gets a 500 error from Zendesk.
The 500 errors seem to happen most often on the resource uploading step, when many connections are staying open at once. So I think it happens because of a limit by Zendesk's servers regarding the number of simultaneous connections. After some testing, I found that 500 errors seem to happen when more than 30 connections are made at once.
So I propose two things for this issue:

Implement request throttling so that no more than 30 connections are made at once.
Implement incremental "Retry After" times - each retry should wait an incrementally longer amount of time before retrying again:

1st retry: 500ms
2nd retry: 1000ms
3rd retry: 2000ms
4th retry: 4000ms
5th retry: 8000ms


"Failed to install support for joining sessions from a URL"

I've seen this error (or something to that effect, I didn't copy the exact message) pop up several times in the past couple days when opening a new window. Not sure why but let me know if I can provide more details...

@roblourens What OS were you on when you saw this?

MacOS

Can you run the "Live Share: Export Logs" command and send them over so we can take a look?
Also, does it appear if you run the "Live Share: Launcher Setup" command?

When I ran "Launcher Setup" I got Browser link launcher successfully installed. Exported the logs after that.
Untitled.zip

It's possible that my CPU was very busy at that moment but I wasn't doing anything that would mess with my actual Insiders install

I'm seeing the same error, also on MacOS. I've tried uninstalling/reinstalling the extension (with a restart of the editor in between). I also tried the launcher setup option and just got the same error popup again:

Attached are my logs:
liveshare.zip

Here's my output:
Argument: /Users/patrys/.vscode-insiders/extensions/ms-vsliveshare.vsliveshare-0.3.925/node_modules/@vsliveshare/vscode-launcher-osx/Live Share for VS Code.app/Contents/MacOS/Live Share for VS Code
Argument: -Register
Argument: --extensionRoot
Extension root passed as an argument: /Users/patrys/.vscode-insiders/extensions/ms-vsliveshare.vsliveshare-0.3.925
Bundle url is<EMAIL_ADDRESS>Set default handler returned with code: 0
Bundle identifier:com.microsoft.cascade.launcher.mac
Set default handler returned with code: 0
Extension not installed

I keep seeing this. Today I am seeing it every time I open a new window.

I have been seeing this on the Insiders build for some time.  I'm attaching my logs after a recent restart.
I see this error whenever I open or reload a window.
liveshare.zip

@patrys @aaroncraig10e @roblourens @tabacco
Are you guys on Mojave by chance?  #1278 was raised with this error specifically occuring on that OS.

I am

Yes. Seeing it on 10.14.1 (18B75)

Yes

That would explain why I can't repro and we haven't seen much traffic on it yet - it must be Mojave specific.
I've merged these details into the other issue since we have folks looking at the Mojave angle over there already..

I still have this issue
vs live share: v0.3.1156
vs code: v1.31.1

For contrast, I've been using Live Share EAP for a while now in both Code and VisStud Pro. The identical error started pooping out of VSCode about three days now.  Running on Windows 10 x64 with the Insider / Developer Options. Haven't noticed it on Ubuntu 16.04 or 19.xx,
I exported logs from VSCode, about two days worth. They are attached here as a Zip archive.
I attempted the manual launcher setup thing and it did the same thing.
Looking thru my runbooks last few days, the one thing that stands out are either the Congressional subpoenas (LOL) and a change to my Microsoft Account (added / changed email).
OneBigDumpLogs.zip

I am having the same issue, just sarted today, how to fix it?

I have the same issues, ever since I updated to Catalina.

I have the same issue since updating to 10.15 Catalina.

Same thing just updated to Catalina 10.15.2

Same thing to me

Same issue since installing Live Share Extension pack today.

Running macOS Catalina 10.15.3 (19D76).
vscode-live-share-logs.zip

No solution suggested? Having the same issue

same issue, any fixes?

This issue isn't fixed, please reopen!

This issue isn't fixed, please reopen!

Indeed, got this message again this morning as I opened VS Code 1.52.1.

Indeed, got this message again this morning as I opened VS Code 1.52.1.

I'm having this issue with every newly opened window as well.

I'm having this issue on MacOS (Big Sur) with every newly opened window as well.

I see this every time I open VSCode.

same issue

same thing to me

same issue

Ran into this same problem today (on macOS).  Solution for me:

Uninstall the VS Code the "Live Share" extension.
Quit VS Code
Remove all vestiges of the extension from ~/.vscode/extensions/ms-vsliveshare*
Start VSCode and reinstall the "Live Share" extension
Run the "Live Share: Launcher Setup" command

@scott-lc unfortunately this did not work for me. When I installed the "Live Share" extension I immediately got the "Failed to install..." message again. When I ran "Live Share: Launcher Setup" it also immediately triggered the "Failed to install..." message.

Step 3 was the trick for me, manually deleting the extension from the drive, and then re-installing it.    Take  look at the latest YYYMMDD_XXX_XXX_VSCode.log file.  I saw this error, which indicated to me that something was amiss.
[2021-04-15 16:43:24.333 Client E] Could not get file mode of /Users/XXX/.vscode/extensions/ms-vsliveshare.vsliveshare-1.0.4131/node_modules/@vsliveshare/vscode-launcher-osx/Live Share for VS Code.app: ENOENT: no such file or directory, stat '/Users/XXX/.vscode/extensions/ms-vsliveshare.vsliveshare-1.0.4131/node_modules/@vsliveshare/vscode-launcher-osx/Live Share for VS Code.app'

In my case, the "Live Share for VS Code.app" was entirely missing from my installation.

same issue here also.
MBP - M1 Big Sur 11.2.3
logs are here.
20210422_142224_16191013447800_VSCode.log

For me, this was caused by my M1 MacBook not yet having installed Rosetta 2.
Installing Rosetta 2 by running this in the Terminal solved it:
softwareupdate --install-rosetta

For me, this was caused by my M1 MacBook not yet having installed Rosetta 2.
Installing Rosetta 2 by running this in the Terminal solved it:
softwareupdate --install-rosetta

Installing Rosetta is not an option for all.
$ softwareupdate --install-rosetta
Installing Rosetta 2 on this system is not supported.
$ uname -p
i386

For me, this was caused by my M1 MacBook not yet having installed Rosetta 2.
Installing Rosetta 2 by running this in the Terminal solved it:
softwareupdate --install-rosetta

Installing Rosetta is not an option for all.
$ softwareupdate --install-rosetta
Installing Rosetta 2 on this system is not supported.
$ uname -p
i386

If it's not an option, you probably don't have an M1 mac and it's not needed.


Server to open the success, but the message is not received, it is like hanging up

I opened the server in the background, but it often can not accept the message, no response, Oddly, I do not stop the server,and it is running

when it runs in background for a  long time,it will get no response .but I have already make it run in background。

sorry,it works after I set the protocols。


potential fix for gh pages deploy

It may be that Travis fixes their issue over the weekend - they have a PR up already, and if so we can discard this. If they haven't merged theirs this may fix the build issue.

Still broken, merging.


Add python libs - DO NOT MERGE

Summary
‼ Do not merge this PR - just re-trigger automation off this branch once we have 👍 ‼
This PR improves the Python compilation process. It adds some dev libraries to the Docker image so Python can compile against the header files they provide, and it adds some commands to the make process to speed it up and reduce the filesize of the compiled python distribution.
All the dependencies are removed from the buildpack.toml to force a re-compilation of every dependency with the new libraries/options.
Once we are happy with this changeset, we shouldn't merge this PR. Instead, we should run the compilation action off of this branch. That will rebuild all the dependencies, and open a new PR with the new dependencies (and the commits on this branch).
We don't actually need a PR for the automation - just the branch - but I think opening a PR is the best way to review the changes, even if the PR itself isn't' required.
Use Cases
Resolves https://github.com/paketo-buildpacks/python/issues/549
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

I like the idea of dependency/README.md to give some explanation of the compilation decisions made.
How did you know to make the changes listed here? Comparison with the built-in compilation mechanism and what's installed on index.docker.io/jericop/amazonlinux-builder:base?

How did you know to make the changes listed here? Comparison with the built-in compilation mechanism and what's installed on index.docker.io/jericop/amazonlinux-builder:base?

Yes, and also looking at the output from python to determine missing module (see below) and from there a quick search for each module usually results in an associated library (e.g. curses results in the library ncurses):
The necessary bits to build these optional modules were not found:
_bz2                  _curses               _curses_panel
_lzma                 _tkinter              readline

With the initial approval from other maintainers, I'm going to initiate the automation based off of this branch. This will open the actual PR with the changes we want to merge. We can have future conversations to that PR.


ACCUMULO-4630 Move user manual to Accumulo website

removed user manual asciidoc and moved it to accumulo-website repo
binary tarball no longer contains user manual
moved configuration mgmt docs in 'config-header.txt' to website
ConfigurationDocGen.java now generates markdown for website during
core module build to core/target/generated-docs

See https://github.com/apache/accumulo-website/pull/11 for corresponding PR to add documentation to website.

The referenced JIRA issue remains open. Is there more work to be done?

I closed it


GVim Motif version menu Bug

Describe the bug
A clear and concise description of what the bug is.
(Issues related to the runtime files should be reported to their maintainer, check the file header.)
To Reproduce
Detailed steps to reproduce the behavior:

Run "gvim"
Resize the gvim window horizontally
The "Help" menu jumps to the right side of the window, where it belongs in Motif, but before resizing, the Help menu is next to "Window" menu.

Expected behavior
The Help menu should be on the right side of the window from start, not next to "Window" menu
Screenshots
If applicable, copy/paste the text or add screenshots to help explain your problem.

Environment (please complete the following information):

Vim version 8.2.2500
OS: Ubuntu 20.04, NetBSD 9.1
Terminal: GVIM

I'm afraid I do not know how to fix this.  I tried a few things, but the only thing that works is changing the width of the Vim window.
Perhaps there is a way to tell the menubar widget to recompute the menu positions?


llvmPackages: 5 -> 7 on Linux, keep Darwin at 5 until fixed

Replaces #39986.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

I've been using 7 as my default for over a month (ea00916ba11510fdf21c116d5204440c80391deb)
and so far so good -- although my usage certainly doesn't cover all packages impacted by this.

A miscompilation bug was mentioned on the other PR, it may be wise to check if LLVM 7 contains the fix (although this would mostly impact Darwin).

Failure on x86_64-darwin (full log)
Attempted: mesa
Partial log (click to expand)
cannot build derivation '/nix/store/ga3jscq5wjjzm7vkza0776xnd041q8az-apple-framework-ApplicationServices.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/c5ww790x688dbacb25xbd3djsg8dxszh-python2.7-pytest-3.9.2.drv': 19 dependencies couldn't be built
cannot build derivation '/nix/store/6zlvywkmim993jpz37d055n4j2xi9cww-apple-framework-CoreVideo.drv': 8 dependencies couldn't be built
cannot build derivation '/nix/store/i45kqjs4cvn6840wcv8vnjab7ccir7wy-apple-framework-Foundation.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/11sasaganqszjhj4hxixq8n6w35xlpa5-python2.7-Mako-1.0.7.drv': 11 dependencies couldn't be built
cannot build derivation '/nix/store/ackxc04k0g3g9qylsl9j5sslpliyhqs3-apple-framework-QuartzCore.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/sdccdnjmv91fgpak47x32p0h6mr9280z-apple-framework-Carbon.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/8ayxhrdkjv8v9l1f859s15hqx2a4y3j8-apple-lib-Xplugin.drv': 11 dependencies couldn't be built
cannot build derivation '/nix/store/5wb3q53012p5b4j59c7yl79w7rpslvnn-mesa-noglu-18.2.2.drv': 33 dependencies couldn't be built
error: build of '/nix/store/5wb3q53012p5b4j59c7yl79w7rpslvnn-mesa-noglu-18.2.2.drv' failed

Timed out, unknown build status on x86_64-linux (full log)
Attempted: mesa
Partial log (click to expand)
[ 81%] Building CXX object tools/llvm-dwp/CMakeFiles/llvm-dwp.dir/llvm-dwp.cpp.o
[ 81%] Building CXX object tools/llvm-cov/CMakeFiles/llvm-cov.dir/SourceCoverageViewHTML.cpp.o
[ 81%] Linking CXX executable ../../bin/llvm-dis
[ 81%] Built target llvm-dis
Scanning dependencies of target llvm-exegesis
[ 81%] Building CXX object tools/llvm-exegesis/CMakeFiles/llvm-exegesis.dir/llvm-exegesis.cpp.o
[ 81%] Linking CXX executable ../../bin/llvm-dwarfdump
building of '/nix/store/4sr7h8y8qx109f1hnm73blxd7nhhrfxw-llvm-7.0.0.drv' timed out after 1800 seconds
cannot build derivation '/nix/store/bfnnlly2dbsnipss189l1kyfadym3m2x-mesa-noglu-18.2.2.drv': 1 dependencies couldn't be built
error: build of '/nix/store/bfnnlly2dbsnipss189l1kyfadym3m2x-mesa-noglu-18.2.2.drv' failed

Success on aarch64-linux (full log)
Attempted: mesa
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/k91wcafyi3khzkacnd2v0yd620adds90-mesa-noglu-18.2.2-drivers/lib
patching script interpreter paths in /nix/store/k91wcafyi3khzkacnd2v0yd620adds90-mesa-noglu-18.2.2-drivers
checking for references to /build in /nix/store/k91wcafyi3khzkacnd2v0yd620adds90-mesa-noglu-18.2.2-drivers...
shrinking RPATHs of ELF executables and libraries in /nix/store/lqi8kbgqcbnb3mgdxaskn8iih9g2blxr-mesa-noglu-18.2.2-osmesa
shrinking /nix/store/lqi8kbgqcbnb3mgdxaskn8iih9g2blxr-mesa-noglu-18.2.2-osmesa/lib/libOSMesa.so.8.0.0
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/lqi8kbgqcbnb3mgdxaskn8iih9g2blxr-mesa-noglu-18.2.2-osmesa/lib
patching script interpreter paths in /nix/store/lqi8kbgqcbnb3mgdxaskn8iih9g2blxr-mesa-noglu-18.2.2-osmesa
checking for references to /build in /nix/store/lqi8kbgqcbnb3mgdxaskn8iih9g2blxr-mesa-noglu-18.2.2-osmesa...
/nix/store/0xkrw68haw0v6h8xjyw0117aji5ij9ga-mesa-noglu-18.2.2

Anyone have thoughts -- particularly "please wait"-- on this?

I don't really see a reason to diverge. I would propose to drop libxml2 by default and switch everything or nothing. The darwin builds will also give us a better overview of the impact.

We have to disable (libxml?) on darwin, or just by default to avoid bootstrapping problems.

Let's aim for this right after the 19.03 branch off. We still have quite a few things in staging that I am wanting to prioritize.

Sounds good for me! Thanks for making these calls! o7 (salute)
On Tue, 05 Feb 2019 16:19:04 -0800, Matthew Bauer<EMAIL_ADDRESS>wrote:

Let's aim for this right after the 19.03 branch off. We still have quite a few things in staging that I am wanting to prioritize.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/pull/49402#issuecomment-460856927Non-text part: text/html


refactor: update tests to use new standalone terraform-plugin-testing module

a go module was released last year to keep testing functionality separated from schema logic https://github.com/hashicorp/terraform-plugin-testing We need this replacement to use some functionality required to test migration from sdkv2 to framework, like plancheck https://developer.hashicorp.com/terraform/plugin/framework/migrating/testing#example
Keeping both github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource and github.com/hashicorp/terraform-plugin-testing/helper/resource imports produces a panic error https://github.com/equinix/terraform-provider-equinix/pull/406#issuecomment-1897294576 so it is required to fully replace all

Codecov Report
Attention: 765 lines in your changes are missing coverage. Please review.

Comparison is base (26b2f3d) 60.11% compared to head (e73b796) 58.82%.
Report is 136 commits behind head on main.

Files
Patch %
Lines

internal/errors/errors.go
7.05%
145 Missing :warning:

equinix/fabric_mapping_helper.go
0.00%
69 Missing :warning:

equinix/resource_metal_device.go
75.43%
42 Missing and 15 partials :warning:

equinix/resource_fabric_connection.go
0.00%
54 Missing :warning:

equinix/resource_fabric_cloud_router.go
0.00%
46 Missing :warning:

equinix/resource_network_device.go
2.50%
39 Missing :warning:

equinix/resource_fabric_routing_protocol.go
0.00%
26 Missing :warning:

equinix/resource_fabric_service_profile.go
0.00%
26 Missing :warning:

internal/resources/metal/metal_ssh_key/resource.go
73.62%
19 Missing and 5 partials :warning:

equinix/resource_metal_virtual_circuit.go
50.00%
13 Missing and 2 partials :warning:

... and 44 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #526      +/-   ##
==========================================
- Coverage   60.11%   58.82%   -1.30%
==========================================
  Files          99       96       -3
  Lines       20045    19857     -188
==========================================
- Hits        12051    11681     -370
- Misses       7691     7889     +198
+ Partials      303      287      -16

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


Query on Reference Data Store is not terminating n dashboard closure

I have a dashboard with an unbounded query of the Reference Data Store.
Start the query
Close the dashboard
The query is still visible and running in Server Tasks, I would expect it to be terminated by the system.
The query can be killed via Server Tasks immediately , so is responsive, and unlikely receiving a kill on dashboard close.

The code in stroom.pipeline.refdata.ReferenceDataServiceImpl#doSearch is already doing
                        if (taskContext.isTerminated()) {
                            throw new RuntimeException("Aborting search due to task termination");
                        }

but suspect it also needs to check for tread interruption.
Also worth double checking that this is ok
        return lmdbEnvironment.getWithReadTxn(readTxn -> {
            try (final CursorIterable<ByteBuffer> keyValueDbIterable = keyValueStoreDb.getLmdbDbi().iterate(
                    readTxn, KeyRange.all());
                    final CursorIterable<ByteBuffer> rangeValueDbIterable = rangeStoreDb.getLmdbDbi().iterate(
                            readTxn, KeyRange.all())) {

i.e. we are using two iterables at once.  Pretty sure LMDB only has one cursor for the txn.  This may be ok as long as the two streams are not accessed concurrently.


No Icon for Lubbock Texas

The city needs to be appended to the list of cities located here: https://github.com/vbguyny/ws4kp/blob/0de6423c65a91bf277222b73a04271b89aec8bcb/Scripts/RegionalCities.js
In addition to the name, the latitude and longitude has to be indicated

I can't edit anything.

@Ethan8474 you need to fork this repo and make the fix on your machine.  Once you have made the change you can commit your changes and create a pull request.

Done, updated in v1.68 which is now live.


Added -D_GLIBCXX_USE_CXX11_ABI=0 to bazel build

Fixes issues with missing symbols when compiling with GCC 5.x. See
issue tensorflow/tensorflow#1419.

Can one of the admins verify this patch?

PR merged. Thanks, @aleist


Active account users can't change their password

I get this message:
$ jitsu users forgot jesusabdullah
info:   Welcome to Nodejitsu
info:   It worked if it ends with Nodejitsu ok
info:   Executing command users forgot jesusabdullah
info:   Request password reset for: jesusabdullah
error:  Error running command users forgot jesusabdullah
error:  Nodejitsu Error (401): Unauthorized
error:  Cannot reset password for inactive account.
info:   Nodejitsu not ok

I suspect this is due to a bug introduced by closing up #111. I'll look into it, but it'd be nice if someone could verify.

I got this problem here. So what can I do now?
$ jitsu login
info:   Welcome to Nodejitsu
info:   It worked if it ends with Nodejitsu ok
info:   Executing command login
prompt: username: jiyinyiyong
prompt: password:
error:  Unable to Authenticate as jiyinyiyong
error:  Nodejitsu Error (401): Not Authorized
prompt: username: ^C
$ jitsu help

Hi @jiyinyiyong,
Your issue is likely due to your account not being activated, and is unrelated to the issue discussed here. You may activate it at http://gimmesomebeta.jit.su . If you have further difficulties, please contact us at<EMAIL_ADDRESS>or at #nodejitsu on irc.freenode.net .

More detail. Here the error message looks the same.
$ jitsu users forgot jiyinyiyong
info:   Welcome to Nodejitsu
info:   It worked if it ends with Nodejitsu ok
info:   Executing command users forgot jiyinyiyong
info:   Request password reset for: jiyinyiyong
error:  Error running command users forgot jiyinyiyong
error:  Nodejitsu Error (401): Not Authorized
error:  Cannot reset password for inactive account.
info:   Nodejitsu not ok

error:  Cannot reset password for inactive account.

This confirms that you need to activate your account.

Finally I got the email on how to confirm my account.. now it's right.


Creating projectError: spawn npm ENOENT

Do you want to request a feature or report a bug?
Not sure if this is BUG or FEATURE
What is the current behaviour?
NPM is required to install preact using CLI, need an option to use yarn instead of npm or preact-cli should detect installed package manager
If the current behaviour is a bug, please provide the steps to reproduce.
$ preact create default frontend
ℹ INFO Assuming you meant preactjs-templates/default...
⠋ Creating projectError: spawn npm ENOENT

No issues if I'll install it by myself:
$ cd frontend && yarn install
...
success Saved lockfile.
Done in 82.35s.

What is the expected behaviour?
CLI should detect installed package manager and use it.
If this is a feature request, what is motivation or use case for changing the behaviour?
NPM is dying, most of the people already using Yarn. I'm not asking to replace NPM with yarn, but would be good if CLI can detect installed package manager. It's hard to get Node and NPM in sync on Debian
Please mention other relevant information.

Node version v8.9.3
npm version not installed
Operating system Debian 9
CLI version v2.1.1

@stiks preact help lists out all the commands which CLI has and preact create help list all the flags and gives detailed explanation for the same.

I am not able to reproduce this issue anymore. Please feel free to close this issue if you are still facing the issue.


Canvas drawing continuation Issue

Issue:
The drawing functionality is not working as expected. When the cursor leaves the canvas, the current line drawing should end. However, if the mouse button is held down while exiting the canvas area and then re-enters, the line continues from where it left off, creating an unnatural experience.
Steps to Reproduce:
Draw on the canvas while holding down the mouse button.
Continue holding the mouse button while moving the cursor outside the canvas.
Re-enter the canvas from any direction.
A line is drawn between the point where the cursor exited the canvas and the point where it re-enters.
Expected Behavior:
The drawing should be seamless. When the cursor moves outside the canvas boundaries, the drawing should stop. Upon re-entering the canvas, it should start as a new drawing rather than continuing from the previous one.

@vishanurag Can you please assign this to me?


Make lean a fully dynamic executable

Obsoleted by #209... for now

Actually there was nothing wrong with the preparatory commits pushed up to that point :smile:


Create LICENCE_FR

This is the French translation of the CC0 1.0 Universal License text.
What

This is the French translation of the CC0 1.0 Universal License text.

Refs

(this project accepts pull requests related to open issues)
(if suggesting a new feature or change, please discuss it in an issue first)
(if fixing a bug, there should be an issue describing it with steps to reproduce)
(only fixing a minor bug - broken link, typo - an issue is not needed)
(please link to the issue here)

Yes, indeed. I apologize. I took the time to write a completely new text following the standards of translating legal documents. The new text, apart from its faithfulness to the original, adheres to legal terminology.

Would you kindly accept my apologies and appreciate the effort put into the new text, please?


Docs: Generate component metadata only from library exposed components

Summary
React-docgen was parsing unnecessary files such as test files, internal components. Some of the files didn't have data to parse and causing irrelevant error messages.
The main motivation of this PR is to make the script work without generating error messages, which are just noises given the current state of it. What makes them just noises? Because the files where the errors are coming from, are not supposed to have metadata, so they doesn't need to be parsed by react-docgen.
So how do we exclude the unwanted files? We can add all of them to excluded paths list but they are a lot of them. Also, they are scattered among the component files and hard to identify by file name or folder they are in.
But we can identify the files that needs to be parsed easily! We can just look at the index.js file! That's why I think this approach is more accurate.
The previous logic read every single file in the given folder. Yes, we had a filter on css files and test files (but flowtest files were still slipping through, but this was an easy fix). However, there are files that are not components, that are internal components or just utilities.
What changed?

generteMetadata.js script was updated to parse only exported component and their subcomponents from index.js of the packages.
Bumped react-docgen to v6.0.4

parse method now return array of data

Ignoring node_modules in .flowconfig as it was casing type errors in CI

List of components that were removed after the update

DefaultAvatar
Foundation
AddCollaboratorsButton
CollaboratorAvatar
CollaboratorsCount
HoverOverlay
PositioningWrapper
Backdrop
Caret
InternalCheckbox
Collection
Item
Contents
Controller
InternalDatapoint
Trend
OptionItem
InternalLabel
InternalLink
InternalList
Message
MeasureItems
ScrollContainer
Action
Header
ExpandableItem
Title
ConfirmationPopover
InternalOverlayPanel
InternalPopover
InternalScrollBoundaryContainerWithForwardRef
ContentContainer
FullPage
PartialPage
PrimaryAction
GroupContent
GroupMobile
Mobile
PrimaryActionIconButton
TableOfContentsAnchor
TableOfContentsItemList
InternalTextField
InternalTextFieldIconButton
Controls
Playhead
AccessibilityPause
NewTabAccessibilityLabel
VisuallyHidden
AnimationContext
RequestAnimationFrameContext
OutsideEventBehavior
StopScrollBehavior
TrapFocusBehavior
ListProvider
NestingProvider
ScrollBoundaryContainerProvider
SideNavigationProvider
TableContext
FormErrorMessage
FormHelperText
FormHelperTextCounter
FormLabel
InternalDismissButton
MaybeTooltip
wrapWithComponent
InternalDateField
InternalDatePicker
TextInput

Links

Jira
[TDD](link to Paper doc)
[Figma](link to Figma file)

Checklist

[ ] Added unit and Flow Tests
[ ] Added documentation + accessibility tests
[ ] Verified accessibility: keyboard & screen reader interaction
[ ] Checked dark mode, responsiveness, and right-to-left support
[ ] Checked stakeholder feedback (e.g. Gestalt designers)

Thanks Diyor! I noticed that this does a little more than just exclude the folders. Mind adding more as what the changes are/improvements. How it works.
Can you verify there are no regressions? How do we know we didn't miss anything

Good point! I actually have snapshot of the only metadata. I compared them and put the list of removed components in the PR description. To be more confident with the output, I can do a deep comparison of every field and text. Hopefully, they'll be exactly the same :D .

Thanks Diyor! I noticed that this does a little more than just exclude the folders. Mind adding more as what the changes are/improvements. How it works.
Can you verify there are no regressions? How do we know we didn't miss anything

Good point! I actually have snapshot of the only metadata. I compared them and put the list of removed components in the PR description. To be more confident with the output, I can do a deep comparison of every field and text. Hopefully, they'll be exactly the same :D .

I checked the deep comparison of the new and old metadata. Aside from unnecessary components, nothing was removed. Instead, more fields were added. Like interfaces/types that are in the file, not just Props.


Wrong order of executed statements

Hi,
First off I just want to say that I am not sure whether to post this issue here, or on the sequel page, as I am not sure wherein lies the problem.
So here is what I am doing
I am using padrino + bacon + sequel + capybara + selenium | capybara-webkit to run tests.
When doing this with selenium, I get the output and result that I expect. See the following gist.
This snippet of code
https://gist.github.com/1504690
produces this out put
https://gist.github.com/f9a9a5bc122ff01d10f2
but with capybara-webkit I get this output
https://gist.github.com/42b9bee8347a7cbf5030
As you can see, the webkit output shows that when the button is clicked, the actions that happen within the controller are executed after all the actions in the test itself.
This seems very strange to me, as

The same setup works with selenium and browsing the code I haven't seen anything specific to transactional fixtures.
I am not using transactions for these tests at all

Now, as I said before I am not sure if this is a sequel issue or a capybara-webkit one. But seeing as I would prefer to use webkit over selenium, I thought I would try here first. I'll bug the sequel guy/s later if I get no luck here :)
Any ideas?

Sequel maintainer here.
You mentioned earlier to me that capybara-webkit was using a different thread (http://groups.google.com/group/sequel-talk/browse_thread/thread/dcfab7dcdbee9780), maybe if you join that thread after the click_button, things may work?  If I had to guess, this is a race condition.
I use Sequel with capybara with no issues, but not capybara-webkit, so this is outside of my area of expertise.  This does not appear to be a Sequel issue, though.

@jeremyevans Well, I was just going off of what is said in the capybara documentation here
https://groups.google.com/forum/#!msg/ruby-capybara/JI6JrirL9gM/R6YiXj4gi_UJ
The most important part is
What happens under Selenium is this: Capybara starts up your Rails app
using a webserver (usually webrick or thin) in a *separate thread* of
the *same process*.

I will see if there is a simple way for me to join thread after the click.
If that doesn't work, I will defer to the knowledge of the capybara-webkit developers.

/cc (sounds like a good bug!)

Well, there's always sleep 1 after the click_button. :)

Tried that (even with 5 seconds) still hung.
I wish there were an easy way to see how this is being executed compared to selenium.

I don't think this has anything to do with sequel. Most of capybara's Javascript drivers execute asynchronously. However, various drivers have found ways to automatically wait until some operations are complete so that some actions seem to execute synchronously. It seems like click_button in the Selenium driver's implementation executes synchronously.
Joining the threads doesn't make sense, because the background thread keeps running. It's an application server (generally WEBrick or Thin) running in a loop, so joining the thread will wait forever.
Capybara comes with a helper called wait_for that you can use to sync up the two threads. It takes a block that should return true once you know the action you're waiting for is complete. As an example, if you're expecting an action to create a new user, you could do something like this:
original_count = User.count
click_button "Signup"
wait_for { User.count == original_count + 1 }

That would get the original count, start processing the signup action, and repeatedly check the user count until it lined up. It will time out and raise an exception after two seconds by default if the condition is never met.
In general, I'd recommend testing things through the UI if you can anyway, so rather than checking that a user was created, I'd check that the page contains something that indicates the user was created. Helpers like have_css automatically use wait_for under the hood, so you don't have to worry about async issues if you use those.

@philly-mac for what it's worth @jferris' advice is golden here, wait_for{} is evil, but have_css (waits for the CSS to arrive) and have_no_css (wait for it to disappear) are exceptionally useful. I'm using them a lot in my pet project.

Cheers guys.
Those tips seemed to work with getting things in the correct order, but now when I run my tests I get this error
Capybara::Driver::Webkit::WebkitInvalidResponseError: Unable to load URL: http://<IP_ADDRESS>:49089/bookings/427

Not sure what this means. Any ideas on this?

That means that either your application responded with an error or you found a bug in capybara-webkit. I'd look through your test.log to see if there's anything interesting about that request.

Test log shows absolutely nothing out of the ordinary.
In fact I was getting those errors before, on every test run, I just thought it was the order of execution that was causing it to do that. But now it is still doing it which is why I am not sure what is causing it.

So upon further investigation it seems as if the problem may lie with padrino/sinatra.
The tests that I had before just tested to see if the url I was at matched a certain regex, and it did. But even when run using selenium nothing is loaded into the page, and I guess this is where the webkit driver is also reporting an error saying that it cannot load the page.
So I am guessing that if I can find out what is stopping the page from loading even in selenium, the webkit driver will work as expected.
Thanks for all the help guys, I will do some more investigating or bug the padrino guys for more help on this one.

Okay final word on this, and my sanity saved just in time for Christmas.
Turns out that the server which was being spun up was silently failing. And there was some error in the code, but I was just never seeing it.
After looking at this thread
https://github.com/jnicklas/capybara/issues/329
I added this code
Capybara.server do |app, port|
    require 'rack/handler/webrick'
    Rack::Handler::WEBrick.run(app, :Port => port, :AccessLog => [], :Logger => WEBrick::Log::new(Rails.root.join("log/capybara_test.log").to_s))
  end

I was actually able to see what was going on and where, and finally able to get things working again.
Silently swallowing up errors in testing is not a good thing at all. But glad I got it sorted, and maybe someone else that reads this may not have to go through this also.

@philly-mac What was the WEBrick error you were seeing and how did you resolve it?  I think I'm experiencing a similar issue.

@dantswain The error was not with webrick itself. It was with the code. But previously I was using thin to run locally, and it was swallowing up the errors and not reporting anything. So all I would know is that something wasn't working, but I had no idea why, or what. when I started using webrick it then started showing me the errors that thin was masking, allowing me to get everything working as it should.

Using @jferris' suggestion works great, with the exception that if your controller throws any errors you get no stack trace but only an inscrutable:
Unable to load URL: http://<IP_ADDRESS>:54473/model/1 (Capybara::Driver::Webkit::WebkitInvalidResponseError)

You can jump into the log to see the issue, but I was wondering if there is there a good way around that, like a convenient:
page.should_not be_an_invalid_response_so_show_me_a_stacktrace_or_the_page_source

@TALlama this is a general issue with asynchronous capybara drivers. Depending on your Rails settings, either a development 500 page is displayed to the headless browser that you don't see, or it bubbles up to thin/webrick and just goes in the log. Either way, the feedback isn't great.
I started work on a branch to fix this here: https://github.com/thoughtbot/capybara-webkit/tree/reraise_exceptions
There are still some issues, but you can try that out or contribute if you'd like. Right now the tests in that branch are hanging and I haven't figured out why.


Fix layout broken issue on metrics page

JIRA: https://issues.redhat.com/browse/RHOAIENG-2560
Description

The custom classname are mixed together when we merge feature branches into main, changed the classname to be more specific.

How Has This Been Tested?

Go to metrics page
Verify the issue is not there anymore and the layout is OK

Test Impact

N/A, layout changes.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

Tested, layout is fixed. New CSS class names look good!


[Documentation Issue] : Readme Update

Brief Description 📝
changed the View Demo hyperlink
Select the open-source program you are contributing to
Other
Code of Conduct

[X] I agree to the CONTRIBUTING GUIDELINES of this project.

Hey @shuvadeepmondal👋! Thanks for raising this issue and demonstrating your interest in contributing to our project!🚀
We value your engagement and eagerly anticipate your contributions. Should you wish to get involved, please use the /assignme command to self-assign the issue.
This issue provides an excellent opportunity for us to enhance our project, and we're excited to have you join us! Please take a moment to review our CONTRIBUTING GUIDELINES before getting started.
If you have any questions or need assistance, feel free to reach out to us on LinkedIn.❤

/assignme

Hello @shuvadeepmondal! We'd like to inform you that the issue has been resolved and is now closed. We appreciate your understanding and look forward to your continued involvement with our repository!🤗
If you encounter any further issues, please don't hesitate to reach out to us on LinkedIn. Thank you for your support!❣
Happy coding!✨ We hope to see you again soon!❣


IllegalArgumentException: Cannot open library: vips.dll -> ExceptionInInitializerError

SYMBOL_LOOKUP = SymbolLookup.libraryLookup(System.mapLibraryName("vips"), LIBRARY_ARENA)
	.or(SymbolLookup.loaderLookup())
	.or(Linker.nativeLinker().defaultLookup());

on Windows this tries to load "vips.dll", which is the wrong name. The library file of libvips 8.15.3 binary windows build is libvips-42.dll.
Note that the same system/setup works with an updated JVips (JNI bindings of libvips - the current github version uses 8.12.x though, so had to use a fork and upgrade that).

Thanks for raising an issue! This is interesting - the code snippet you included is directly generated by jextract (like everything in VipsRaw).
Specifically, the vips library name is passed to jextract, like so (in generate_ffm_bindings.sh):
"$JEXTRACT_DOWNLOAD_PATH"/bin/jextract \
--include-dir "/opt/homebrew/include/vips/" \
--include-dir "/opt/homebrew/include/" \
--include-dir "/opt/homebrew/include/glib-2.0" \
--include-dir "/opt/homebrew/lib/glib-2.0/include" \
--output core/src/main/java \
--target-package "app.photofox.vipsffm.jextract" \
--header-class-name "VipsRaw" \
--library "vips" \
@includes_filtered.txt \
"$LIBVIPS_ENTRY_PATH"

I don't have much experience with Windows systems - on Unix-like systems the library usually has a symlink added without the number in it. I see I have both libvips.dylib and libvips.42.dylib in my /opt/homebrew/lib/ folder.
I'm wondering if I'm just using the "wrong" library name, or if this is a limitation of the jextract output that needs a workaround. There are some docs for it here - what do you think 🤔

Seems to me the root issue in jextract is actually
System.mapLibraryName
which just maps libraries in windows by appending "dll".
Also searching
https://www.google.com/search?q=libvips+library+name+vips+vips-42
tells me that this might be mostly an issue due to libvips naming, though I must say in my linux and osx years I've seen tons of libs with numbers in the name - symlinked though, as you say. Unfortunately windows does no such thing, never has.
And when windows never symlinks any libs by default, I think one shouldn't assume symlinking for windows, so having people manually have to symlink libvips on windows (most people don't even know about mklink on windows in comparison to a simple ln -s on linux) feels very dirty tbh.
Would be nice if there was an option in jextract to provide a fallback library name, as it seems the java code would be able to do that with a  simple additional .or(...).
I'll see if I can find more info on jextract mailing lists and the like.

@lopcode very interesting in this context:
https://github.com/openjdk/jextract/pull/172/files

Also did a bit of digging in other libvips bindings and found this in the Ruby bindings: https://github.com/libvips/ruby-vips/blob/master/lib/vips.rb#L26
So they're being quite explicit about the expected library name on each platform. I think I think have to do something similar.

@lopcode found it:
https://cr.openjdk.org/~mcimadamore/panama/jextract_changes.html
in Change #8: Better library loading
looks to me like you could just do
-l vips -l libvips-42
and have it work for both *nix/bsd (macos/osx) systems and windows

@lopcode your link brings something very interesting - 42 seems to be the current abi number, so that can (and most probably will) change in the future. Which also means it would be beneficial in the long run to have some other solution than just hardcoding 42. But the number should be constant wrt. libvips version - the same libvips version will always have the same ABI version.

btw that same jextract code will also trigger a warning about needing an explicit flag, see:
libraryLookup is a restricted method of the Java platform.
Programs can only use libraryLookup when access to restricted methods is enabled.
Restricted methods are unsafe, and, if used incorrectly, might crash the JVM or result in memory corruption.

(I'm on a different computer right now, so I don't have the message at hand, but it says that you'll have to pass some enable-unrestricted flag in the future, or it won't work anymore)

Okay it's here:
https://openjdk.org/jeps/472
To enable native access for all code on the class path, use the following command-line option:

java --enable-native-access=ALL-UNNAMED

I don't currently understand how the jextract or chain can ever work - at least on my system, a library that can't be found will cause an IllegalArgumentException. So it's not possible to add multiple possible names because one will always fail.
I'm currently inclined to replace this piece of code with something custom that changes the library search name based on the operating system, similar to the ruby-vips snippet above.

Ok, #57 is an attempt to fix this issue on Windows. I included a property reference to allow consumers to change the ABI number, if they really need to.

I think I've fixed this with a custom SymbolLookup builder, which lets the library do a few searches for various platform-specific names: https://github.com/lopcode/vips-ffm/blob/main/core/src/main/java/app/photofox/vipsffm/VipsLibLookup.java#L10
Now I also understand the SymbolLookup.or() better - it's not about just "loading" the library, it's expanding the search space for symbols (so it tries the first loaded library for a symbol, then the next one, etc). I had to add lookup spaces for vips, glib, and gobject which I think makes sense. Although having to be explicit makes me nervous that it won't find other libs on Windows like libheif... but let's see how it goes.
I've also included a GitHub Actions check for Windows to prove it works, and stop me from accidentally breaking it in the future: https://github.com/lopcode/vips-ffm/blob/main/.github/workflows/checks.yml#L53
This is all in 0.5.0, if you could give it a try please (might need to given Maven Central a few minutes to publish) 🙁 And thanks again for the heads up, I wouldn't have noticed.

It finds the lib now (with 0.5.0), but crashes later on:
(process:19900): GLib-GObject-CRITICAL **: 16:54:52.044: g_object_set_is_valid_property: object class 'VipsIccTransform' has no property named 'outputProfile'
Issue seems to be this line in VImage:
VipsOption.String outputProfileOption = VipsOption.String("outputProfile", outputProfile);
https://www.libvips.org/API/current/libvips-colour.html#vips-icc-transform
The version 8.15.3 doesn't have that property on that method.

Runs on windows now. I'll post further issues separately, thanks!


Support open intervals for temporal operators

The extension from closed intervals [a, b] to open (a, b) and half open intervals (a, b] would be desirable.

Additionally, the support of negative time intervals [-10, 0] is also helpful.


Add export and import of csv files for vertica and mysql

add method to export query results to temporary csv files
add method for vertica and mysql to import csv files

Coverage increased (+0.1%) to 91.421% when pulling 2d7f458ec3bc9f0c529a30f65d8e02e256db53ec on RobinPapke:csv_import_export into 44318011c7cb3cfc97af20c3bc95b4cf8621d0f8 on kayak:master.


jedis 2.9.0 in Maven Central is broken, it does not contain the fix from pull request #1368

The pull request "Revert empty-list-on-transaction-rollback change. #1368" is supposed to be included in jedis 2.9.0, and it was actually merged into branch 2.9 (see commit "Revert empty-list-on-transaction-rollback change. (#1368)"), but all of this was done after 22-Jul-2016, which is the date of the last update of redis.clients:jedis:2.9.0 in Maven Central.
Thus redis.clients:jedis:2.9.0 in Maven Central does not contain the fix and is broken, and we had troubles because of this. So just for the sake of other people, please remove 2.9.0 from Maven Central (or update it) and definitely put an appropriate note in release 2.9.0 notes.

@stIncMale Thank you for interest in Jedis. I think, you misunderstood a bit. #1368 was created after release of jedis-2.9.0 and expected to be released in upcoming release (e.g. jedis-2.9.1).

@sazzad16 Ah, in that case, I'm sorry for the confusion :) Thank you!

@stIncMale Thank your for understanding :)

These are the only usage of import redis.clients.jedis.Transaction class across peregrine and monolith code base.


fix async-profiler install

Install async-profiler + perf and openjdk-11 debug symbols at the right time

Codecov Report

Merging #8404 (9d94d75) into master (975b11a) will decrease coverage by 40.52%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #8404       +/-   ##
=============================================
- Coverage     70.87%   30.35%   -40.53%
=============================================
  Files          1655     1643       -12
  Lines         86598    86244      -354
  Branches      13061    13023       -38
=============================================
- Hits          61377    26177    -35200
- Misses        20989    57700    +36711
+ Partials       4232     2367     -1865

Flag
Coverage Δ

integration1
28.53% <ø> (-0.16%)
:arrow_down:

integration2
27.31% <ø> (-0.04%)
:arrow_down:

unittests1
?

unittests2
?

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

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


Unresolved type Base64 in camel-jira native test

jdk11u-dev/master
graal/master
quarkus/master
camel-quarkus/master

Error: com.oracle.graal.pointsto.constraints.UnresolvedElementException: Discovered unresolved type during parsing: com.google.api.client.repackaged.org.apache.commons.codec.binary.Base64. To diagnose the issue you can use the --allow-incomplete-classpath option. The missing type is then reported at run time when it is accessed the first time.
Detailed message:
Trace:
	at parsing org.apache.camel.component.jira.oauth.JiraOAuthAuthenticationHandler.getOAuthRsaSigner(JiraOAuthAuthenticationHandler.java:72)
Call path from entry point to org.apache.camel.component.jira.oauth.JiraOAuthAuthenticationHandler.getOAuthRsaSigner(String):
	at org.apache.camel.component.jira.oauth.JiraOAuthAuthenticationHandler.getOAuthRsaSigner(JiraOAuthAuthenticationHandler.java:72)

Longer output

@claudio4j  can we replace this with Java's libraries instead of the repackaged commons-codec class ?

Yes it can be replaced. I will do it today or tomorrow, then will let you know here.

PR in place https://github.com/apache/camel/pull/3785

Thanks @claudio4j for the quick fix!

@claudio4j I'm trying to verify the fix, but have not yet managed to do so.
Last week patched camel 3.2.x with the fix (see branch), but was unable to get to the test failure. I encountered https://github.com/quarkusio/quarkus/issues/9103 and @ppalaga's PR didn't work when I first tried it. I'm trying again now that @ppalaga has a new fix and he's rebased the branch.

Somehow things have changed for Camel Jira native tests. They're now skipped unless you start a JIRA instance (e.g. docker) and set the right env variables. Can't find a CI run that would show an example so maybe Camel Jira native tests are not run? Discussion here.

The camel-jira tests in camel repo are all mockito tests, no need for a running jira instance.
But I see the JiraTest requires a running jira instance, however I didn't write the camel-jira quarkus code, so I am unsure about it.
Can you try run a docker jira server ?

@galderz could you please check whether this issue is still occurring? apache/camel#3785 mentioned above is a part of Camel 3.3.0 to which we upgraded recently.

How do you test this now? Seems like JIRA tests are now disabled. Do you have a profile in the maven tests that starts JIRA and tests this?

Actually, I think the issue is fixed based on testing I did before. I'll verify again. What happens is that even if JIRA tests are skipped, this issue happened during image generation, which happens regardless. I'll give it another spin in the next few days.

Fixed.


Add Appium to apps

(cc @jlipps)

Thanks! I would suggest adding some keywords like hybrid, automation, webdriver, etc.

Just added some more keywords. Thanks.

👍


Make formatting of smogon links consistent

If you're going to make formatting changes, can you change them all to ` quotes?

like throughout the project or for all links?

It eliminates the need for \" in links.
Do it for all links and surrounding text.

@Zarel can you merge this for now, I'll do the update to quotes when I'm not in the middle of recovering from wisdom tooth surgery

mmk


[component]: App store badge

Description
Create a CSS component for an app store badge.
Proposed markup

  Download the Post App on Google Play

Size of the image should be defined
Projects need to provide the app store badge locally (because of localization and requirement to keep the badge up to date)
A descriptive text should be provided for the link

Open issues:
Design:

[x]  Integrated new typography
[ ]  Spacing is defined
[x] Documentation final

Tokens:

[x]  Typography tokens ready

No response
Design
https://www.figma.com/design/JIT5AdGYqv6bDRpfBPV8XR/Foundations-%26-Components-Next-Level?node-id=1530-32646
Tokens

[x] Tokens for this component are ready

Tasks
### 💻 Tasks
- [x] Review Design (All states present? Possible to implement?)
- [ ] Add tokens for height
- [ ] HTML/CSS implementation
- [ ] Documentation in Storybook
- [ ] Write documentation about where to download the latest badge image assets
- [ ] Visual regression tests
- [ ] Design review

@alionazherdetska You can find the tokens now under Components/App store button. There is no border radius for focus as its set to 6 right now while the normal component has 4. I assume it will just be handled like everything else and get the border radius from the component itself, if anything changes I'll message you


[QUESTION] Add new notifications

Hello, I am recently creating a project in Laravel and I have decided to use Laravel 9 with AdminLTE 3. I had in another Laravel project a Toast alert system that works great and that I would like to implement instead of using sweetalert. My problem is that notifications outside the AdminLTE project are displayed in the upper right corner, but in the AdminLTE project they appear in the lower left corner, showing outside the "boundaries" of the screen. Which leads me to think that there is some conflict, I have changed all the classes of my alert system and it still gives me the same error, so I ask if it is possible to include a different alert system without it "breaking". A greeting and thanks in advance

Hi @alvarobm98 , if your toast is a third-party or a custom plugin, remember you need to configure the plugins files, before you can use it. Please, see https://github.com/jeroennoten/Laravel-AdminLTE/wiki/Plugins-Configuration

I have already tried that, as I have mentioned, the notifications work correctly in any Laravel project, except when I add them to a Laravel project with AdminLTE, it is as if there was some kind of conflict but I don't understand where it is because I ask if it was compatible adding another alert system other than swetalert2


Refactor to reduce replication and increase readability of TuoniC2.py

Key updates:

Typed parameters
Optimised http methods to reduce code re-use
Swapped to format strings
minimised for loops/ logic operators


Unshift a path to require.paths --include option

Is there a way to do this in mocha? I have a bunch of expresso tests that I am migrating over to mocha which used the --include option to shift the paths so that you could do var helper = require('lib/helpers/some_helper'). With mocha I now have to do var helper = require('../../lib/helpers/date_helper').

node doesn't support require.paths anymore


History back when photo gallery is opened

Instead of closing photo preview – (web) app is closing fully

That's my main issue while using Phanpy on my phone. The habit of swiping back to go back is too strong for me to remember not to use it in Phanpy so I'm closing the app and opening it back all the time.

It's really annoying me too on my phone, especially since when I open Phanpy back, it brings me back to the top of the list.

Should be working now on production site.

Yes, I saw it working yesterday evening! Thank you so much for this. It makes my experience with Phanpy on mobile much more enjoyable!


I go from 0.3.x to here (0.7.x) and get painful errors

previously I use
 import useScroll from 'scroll-behavior/lib/useStandardScroll';
and
import { Router, browserHistory } from 'react-router';
const history = useScroll(() => browserHistory)();

now use
import withScroll from 'scroll-behavior';
const history = withScroll(() => browserHistory)();

I get
main-e52c358c0540bca42e4b.js:2335 Uncaught TypeError: (0 , _scrollBehavior2.default)(...) is not a function

what should I do?

I installed react-router-scroll scroll-behavior
and use
import useScroll from 'react-router-scroll';
// const history = useScroll(() => browserHistory)();  //delete this line
  <Router
    render={(props) => {
      applyRouterMiddleware(useScroll());
        return <ReduxAsyncConnect {...props} helpers={{ client }} filter={item => !item.deferred} />;
    }
    }
    history={browserHistory}
  >

now it works


Padding option in centerCell method isn't working inX6 v2

Describe the bug
graph.centerCell(cell, { padding: { top: 300 } }), is something which we use, and we upgraded X6 version, it all stopped working
Your Example Website or App
NA
Steps to Reproduce the Bug or Issue
Position a cell using padding option, it wouldn't work
Expected behavior
Padding should be applied properly
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Can you provide a reproducible codesanox？

Here's the sandbox link
Please make sure you drag the node in the graph, and then click on center button on the right to center it


Implement an update_dependencies setting

In plan.go's upgrade, lint, and uninstall functions, if config.UpdateDependencies is true, there should be an additional Step that calls helm dependency update $chart. It needs to happen before the main command, but I don't think it matters whether it happens before or after InitKube.
The UpdateDependencies struct will look like Lint, though a little simpler:

It only needs Chart and cmd fields.
Its Prepare() method should require a nonempty Chart.
It doesn't need to pass any global flags other than --debug.
It doesn't need to pass any command-specific flags at all.

The draft documentation I'm writing for #8 mentions this setting but calls it out as "not yet implemented." Make sure that's updated as part of the work for this change.


feat: Add Bangla language support for Bangladeshi holidays

Contributing a new country ?

[ ] Have you checked to ensure there aren't other open Pull Requests for the same country?

Create a new class in the Countries directory. It should extend the Country class.
Add a test for the new country in the tests directory.
Run the tests so a snapshot gets created.
Verify the result in the newly created snapshot is correct.

"Thanks! Please check the changes. It returns Bangla names in English instead of Bangla font.

Thanks!


Question: How to write SQL INSERT INTO SELECT Statement?

I'm sure I'm not the only one but I can't find any documentation regarding this.
This SQL statement creates copies/duplicates.
How do I write SQL INSERT INTO SELECT Statement in Squel?
Example:
INSERT INTO table2
SELECT * FROM table1;
or
INSERT INTO table2
(column_name(s))
SELECT column_name(s)
FROM table1;
SQL Example taken from here:
http://www.w3schools.com/sql/sql_insert_into_select.asp
Squel insert() requires .set() and .set() requires 2 arguments which got me stuck.
Can this currently be converted from SQL to Squel?
Many Thanks

I would definitely be interested in that one (currently doing it manually (by appending my select to an insert string I create manually)

This is implemented in master, I just need to update the docs.

Awesome, let me know if I can help!

@hiddentao Nice work! Much more elegant than my current hack around! Can't wait to use it (and replace all my code...)
@pcothenet If you want a reference the relevant test is here: https://github.com/hiddentao/squel/blob/c5e6e136f25be5ffd7abe7ff184296f3603089fe/test/insert.test.coffee#L167-L177

Thanks! Will this be in 3.11.0?

Yes it will.
On 29/01/2015 13:45, Paul Cothenet wrote:

Thanks! Will this be in 3.11.0?
—
Reply to this email directly or view it on GitHub
https://github.com/hiddentao/squel/issues/53#issuecomment-71972301.

--
Ramesh Nair
Freelance Web Developer
www.hiddentao.com
www.github.com/hiddentao


Add release notes for 1.3.2

Description
Add release notes for 1.3.2
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks @andrross !


Add dict(*iterable) constructor

Fix issue #8328

Hi @stuartarchibald, I believe I have addressed all your comments. The current failure is not related to any changes introduced in my previous commit.
- Windows py310_np122

(testenv) D:\a\1\s>cmd /C conda install -q -y -c numba tbb=2021 tbb-devel
Collecting package metadata (current_repodata.json): ...working... failed

CondaHTTPError: HTTP 504 GATEWAY TIMEOUT for url <[https://conda.anaconda.org/numba/win-64/repodata.json>](https://conda.anaconda.org/numba/win-64/repodata.json%3E)
Elapsed: 01:00.076791
CF-RAY: 73f5e5f559ed2ad6-ORD

A remote server error occurred when trying to retrieve this URL.

@guilhermeleobas @kc611 -- may kindly request an update on the state of this PR?

Hi @esc, PR is waiting for another review


Inappropriate file formats are attached on the android phone

Invalid fb2 file format is attached only in android browsers.
my configs:
djsConfig = {
    maxFilesize: 20,
    maxThumbnailFilesize: 20,
    acceptedFiles: 'application/pdf,application/vnd.openxmlformats-officedocument.wordprocessingml.document,application/msword,image/bmp,image/jpeg,image/png,image/gif',
    dictRemoveFile: 'x',
    addRemoveLinks: true,
    autoProcessQueue: false,
    autoQueue: false,
  };

I found that the fb2 file type is an empty string, which is why it is attached to the form.


BSD Licesne Clarification

Which of the BSD Licenses described in this SPDX repository is actually being used?

The license is included in the repository. It is the BSD 2-clause license.

@davidfischer Thank you, I've asked because I'm using this specification to scan for OSS licenses in my code. The code scans the registered license name in Python repositories e.g. PyPi. Unfortunately, it says "BSD License (BSD)" instead of BSD 2 Clause, BSD 2-Clause or BSD-2-Clause.
This is actually very important because there are many software compliance tools that scan for used OSS libraries in code for legal reasons, as those must be listed in the code that uses them. It would be great, if you could change the the license name in PyPy to one of the options that I've mentioned above. So it will comply with the SPDX effort to standardise the usage of open source software licenses.

This project is not actively maintained and has been seeking maintainers. Feel free to open a pull request with the desired changes and I'll try to review them or ask the community to do so.


Mesh texturing: use longer rays inside the mesh?

In the mesh texturing code (here) a small ray is generated at each texel around the mesh surface, more precisely from x-n*raylen/2 to x+n*raylen/2.
Reflective surfaces are often modeled by nerf as semi-transparent surfaces with inner clouds. The nerf color around these surfaces may not reflect the real aspect of it, which comes from the clouds behind.
Since the textures are solid (no transparency - which is nice because rendering semi-transparent textures is not trivial), why not start much deeper inside the mesh? In fact, I would do rays that start from the farthest possible point behind the surface (eg scene bounds in that direction) and stop just outside of it (as it is done now). That way, we ensure that the texture looks like what would be seen if I have my eye just outside of the mesh.
This would also work better to extract the texture from a nerf, using a mesh computed by other means than nerfstudio.
Does that make sense?

Hey @f-dy, so I think it might make sense to extend the far_plane for the reasons you are suggesting. I could imagine we start the ray in the same spot, e.g., near_plane=x-offset and then set far_plane=X, where X is computed as the distance to exit the scene bounds. Alternatively, we could just set far_plane=scene_diagonal_length similar to how we set a big number in nerfacto here. That would avoid the ray <> aabb intersection test.


Improve training speed in parallel learning where workers is not powers of 2

Improve training speed in parallel learning where workers is not powers of 2

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: qrqpjxq sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the contribution.
Do you use the recursive-halving for the non-power-of-2 machines ?
Actually, we have this before, but it is not efficient compared with the pair wise exchange.
https://github.com/Microsoft/LightGBM/commit/7737b45e10203718ec682347d39524f436de3d01#diff-826309c5e7db10774b5a8999984b5ded
The reason, it has two additional process ( some nodes need to send/receive all data from its neighbor), and all other nodes are idle in this additional process, as a result, the overall time cost will be about 3x than the recursive-halving of power-of-2.
Though simple pair wise algorithm needs n-1 time communication, but it doesn't have additional cost.
Thus, it will be faster than the recursive-halving when machine is not power-of-2 .

I tried in our cluster, num_data is 7000w; num_features is 30; tree configuration: num_trees=50; num_leaves=512
when set:
num_workers = 64 it take 218s
num_workers = 65 it take 221s
num_workers = 127 it take 178s
if use pair wise algorithm, num_workers = 65 it take 1022s
(training time is relation to cluster load)
It seems in our cluster recursive-halving is faster, i don't know if just recursive-halving method work better in our environment.

@qrqpjxq okay, i see.
I think the reason the num_featrues is 30 in you tasks.
As a result, the communication volume is very small. So the communication times will be a largest cost in your task.
As a result, I think a better solution is:

when communication volume is large, we use the pair-wise solution.
when communication volume is small, we use the allgather to got all data from all machines, and do the reduce scatter locally.

What do you think ?

I think your idea will work well, can have a try.
and there are two cases if have any reference value：

in my test, num_features=30 and num_bins=9600(about, some feature is category)
before now, i have try AllreduceByAllGather to instead ReduceScatter(have redundant reduce compute locally) when workers is not power of 2, and training time reduce a litter.

@qrqpjxq sounds good.
The process of recursive halving when non-power-of-2 is not so elegant.
We can decide a threshold, for example, num_bins < 10000 (total communication size is about 200,000KB), And use the AllreduceByAllGather in that case.
Could you help for this ?

Ok, i will try it.
But in paper "optimization of collective communication operations in mpich"(52), it use bruck algorithm(AllreduceByAllGather used it) when total data gathered < 80KB.

@qrqpjxq I think the threshold could be changed. we also can let it configurable ( put it in the parameters.)

@qrqpjxq updates ?

i'm sorry for forget reply you. i'm very busy these days. i will do it in the latter days of holiday, or if you have time, can you do it?

@qrqpjxq It is not urgent. We can wait you to do this.

@qrqpjxq updates ?


ReadOnlyMany does not work with static provisioning

As a system admin I created the following static pv:
apiVersion: v1
kind: PersistentVolume
metadata:
  name: pv-images
  labels:
    id: images
spec:
  capacity:
    storage: 10Gi
  accessModes:
    - ReadOnlyMany
  csi:
    driver: ibm-spectrum-scale-csi
    volumeHandle:<PHONE_NUMBER>4479190260;099B6A42:5DC52700;path=/mnt/remote_fs0_1m/static/images

Unfortunately, a user can still write to the PV if he mounts the volume. This should not be allowed.
Please fix it as it affects my AI workloads.

Spectrum Scale CSI driver does not support ReadOnlyMany-
https://www.ibm.com/support/knowledgecenter/STXKQY_CSI_SHR/com.ibm.spectrum.scale.csi.v1r10.doc/bl1csi_limitations.html

Will you support ReadOnly Volumes created with static provisioning in 2.0.0?

No, There is no support of ReadOnly volumes in 2.0.0

The idea of this issue is to deliver this functionality in CSI, so please reopen and move it to the next release. The functionality should be supported.

Currently there is no way to support this but keeping this open as requirement.

Maybe you could call mmchattr -i yes fileOrDir for a static volume and then reset it again after deletion.

Very good suggestion. I will look into It.


How to use WebChat in ReactJS?

I have a question
I'm new to BotFramework-WebChat and I want to test my Bot (it works well in "Test in Web Chat" in Azure Bot - figure 1) in my web app using ReactJS. But it always warning "Taking longer than usual to connect." (figure 2).

After checked network tab in developer-tools, I saw that it has a request to https://directline.botframework.com/v3/directline/conversations but got error:

Here is my code:
const Chatbot = (props) => {
  const directLine = useMemo(
    () =>
      createDirectLine({
        token: "DIRECT_LINE_TOKEN",
      }),
    []
  );

  return <ReactWebChat directLine={directLine} />;
};

Any wrong here?

@baodv1001 - Please verify that "Always On" is enabled in your bot's app service resource. You can find this setting under Configuration => General settings. If you're experiencing this issue after a period of inactivity, this is likely why as the app service will go to "sleep" while waiting.
Looking at the bit of code you provided, I don't see anything that is jumping out at me.
If the above isn't the source of the problem, try comparing your setup against the provided samples in this repo. There are several that are built as React projects. For instance, look at the ChatComponent.js file in the "01.getting-started/g.hybrid-react-npm" sample.
You can also try this setup which is what I use in a React project of my own. You will need to make a few changes. For instance, I call on a locally-run token server to get my DirectLine token.
import React, { useState, useEffect, useCallback } from 'react';
import ReactWebChat, {
  createDirectLine,
  createStore,
} from 'botframework-webchat';
import { ConnectionStatus } from 'botframework-directlinejs';

const WebChat = props => {
  const [loading, setLoading] = useState(true);
  const [userID, setUserID] = useState('dl_abc123');
  const [username, setUsername] = useState('Bobby');
  const [store, setStore] = useState(null);
  const [directLine, setDirectLine] = useState(null);

  const webChatStore = createStore({}, ({ dispatch }) => next => action => {
    connectionStatusChanged(directLine.connectionStatus$);
    if (action.type === 'DIRECT_LINE/CONNECT_FULFILLED') {
      dispatch({
        type: 'WEB_CHAT/SEND_EVENT',
        payload: {
          name: 'webchat/join',
        },
      });
    }

    if (action.type === 'DIRECT_LINE/INCOMING_ACTIVITY') {
      console.log('INCOMING ACTIVITY ', action.payload.activity);
    }
    return next(action);
  });

  const fetchToken = useCallback(async () => {
    const res = await fetch('http://localhost:3500/directline/token', { method: 'POST' });
    const { token } = await res.json();
    await setDirectLine(await createDirectLine({ token }));
    setLoading(false);
  }, [setDirectLine]);

  const connectionStatusChanged = connectionStatus => {
    switch (connectionStatus.value) {
      case ConnectionStatus.Online:
        console.log('Connected')
    }
  };

  useEffect(() => {
    if (loading === false) {
      setStore(webChatStore);
    }
    if (loading === true) {
      fetchToken();
    }
  }, []);

  return loading === false && !!directLine ? (
    <ReactWebChat
      directLine={directLine}
      userID={userID}
      username={username}
      store={store}
    />
  ) : (

Loading...

  );
};

export default WebChat;

Closing this as resolved.


Duplicate gives no result using "withDecision=true"

Just after applying some decisions, it is not possible to get any data with decisions

https://github.com/Sp2000/colplus-backend/commit/02acc5d81bc3f0d15aef4797ce12da93aef3810e

This seems to be an issue again. See here.

Not really an issue with duplicates.
There are 1227 decsions for catalogue 3 and scarabs:
https://api.catalogue.life/decision?datasetKey=3&subjectDecisionKey=1027
All of them appear to be broken, i.e. the subject is lacking an id property that is required to bind the decision to the actual name usage. Therefore these decisions do not show up on duplicates or other usage responses like the tree API: https://api.catalogue.life/decision?datasetKey=3&subjectDecisionKey=1027&broken=true
As you can see in the logs none of the decisions did match.

Something seems off then, because the decisions were applied to the 2019-03-28 version of scarabs, which was replaced by the 2019-05-28 version but then when we saw no decisions we re-imported the 2019-03-28 version, so the decisions should match.

The logs complain about duplicate names so the decision cannot clearly be matched to one of the multiple usages:

EditorialDecision 4194 cannot be rematched to dataset 1027 - multiple names like SPECIES Copris graueri

Hmm, I wonder why there's no ID's in the decisions? Maybe it is a frontend bug after all?

I'll try to delete out the decisions and try again while watching what the frontend is sending to the API.

the ids is created by the import matching based on the other subject props.
But there are only logs for 25 duplicate names, so the other 12xx should be fine. Smells like a true bug...

found it

takes me ~30 minutes to deploy again

Alright, we'll try again after you deploy.

I deleted out all previous Scarabs decisions, and @yroskov re-applied the decisions to the 2019-03-28 version of Scarabs again, and then I re-imported the 2019-05-28 version and the decisions are still not remembered. I think the decisions had IDs, but then when we imported the 2019-05-28 version, the decisions might have lost their IDs. I'm double checking that now, because it seems strange.
The frontend is properly posting the decisions with IDs:
{
  "datasetKey": 3,
  "subjectDatasetKey": "1027",
  "subject": {
    "id": "22-.01-.05-.01-.002-.000-.102-.-",
    "name": "Amphimallon subparallelum",
    "authorship": "(Escalera, 1914)",
    "rank": "species"
  },
  "mode": "update",
  "status": "provisionally accepted"
}

not deployed yet ;)

not it is on both envs

I just re-tried it on dev where it was deployed (Backend version: 5409fd4 January 3, 2020 3:40 PM) and the subject IDs actually did disappear when the 2019-05-28 version was imported.

Nevermind, it worked on prod.

The logs claim that the names of the 52 decisions do not exist in the dataset. Please always check the logs, filtering by the logger name life.catalogue.dao.SubjectRematcher helps.
Checking the first log entry manually I see:

decision 3: SPECIES Anisoplia villosa Faldermann, 1836
search for Anisoplia villosa returns one usage with a different name: Anisoplia villosa (Goeze, 1777) Schoolmeesters P.
=> so no match is expected behavior in that case.

Decision 52, Clinteria malayensis Gestro, 1888 also has a different authorship Wallace, 1867 now, so a broken decision is expected.
subject IDs only ever remain if the matching succeeds. Otherwise they get nulled to indicate broken decisons. Thats by design.

subject IDs only ever remain if the matching succeeds. Otherwise they get nulled to indicate broken decisons. Thats by design.

Ah, that makes more sense.


ROX-11740: dackbox cluster sac tests

Description
The goal here is to have a test harness that ensures Scoped Access Control behaves similarly when the application is running against the bolt/rocksdb/bleve datastores and when it is running against the postgres datastores.
Checklist

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

If any of these don't apply, please comment below.
Testing Performed
Added tests.
Manual run of the added tests with the postgres datastor feature flag activated and deactivated.

/test style-checks


Antialias when using overlayWith

Is there a way to add anti-aliasing on the transparency border when using overlayWith? When I overlay a circle everything comes out okay, but the edges are sharp. Here's an example image:

Here's my code:
  const rect = Buffer.from(
    `<svg>
      <rect
        height="${size}"
        rx="${Math.floor(size / 2)}"
        ry="${Math.floor(size / 2)}"
        shape-rendering="auto" <- I have tried the other shape-rendering values
        width="${size}"
        x="0"
        y="0"
      />
    </svg>`
  );

  const buffer = await image
    .extract({
      height: radius * 2,
      left: Math.max(0, x - radius),
      top: Math.max(0, y - radius),
      width: radius * 2
    })
    .resize(size, size)
    .overlayWith(rect, { cutout: true })

Here's an image that has smoother edges (not sure how it was created):

Hello, the summary of the cutout logic in sharp is that it takes the alpha channel from the librsvg/cairo-rendered SVG and sticks it on the input. My best guess would be that there's a missing premultiplication step somewhere in the middle of the librsvg/cairo rendering, but I'm not sure there's much sharp can do here.
If you haven't already seen it, the existing overlayWith logic will be changing to take advantage of the relatively-new libvips composite logic as part of #728
As a workaround, perhaps try rendering at double the desired dimensions then perform a second pass to resize to half the dimensions as the Lanczos kernel will apply anti-aliasing.

@lovell thank you so much! The workaround is hugely helpful and rendering twice did the trick. Here's my code if it helps anyone else:
  // SVG now matches extract dimensions
  const rect = Buffer.from(
    `<svg>
      <rect
        height="${radius * 2}"
        rx="${radius}"
        ry="${radius}"
        shape-rendering="geometricPrecision"
        width="${radius * 2}"
        x="0"
        y="0"
      />
    </svg>`
  );

  // Extract and overlay in the first pass
  const buffer = await image
    .extract({
      height: radius * 2,
      left: Math.max(0, x - radius),
      top: Math.max(0, y - radius),
      width: radius * 2
    })
    .overlayWith(rect, { cutout: true })
    .toBuffer();

  // Resize after initial render
  await sharp(buffer)
    .resize(size, size)
    .toFile(`o_${fileName}.webp`, function(err, info) {
      console.log('err', err);
      console.log('info', info);
    });


Fix merge conflicts and update to main

You may need to check the conflicts that I fixed related to moving the LoggingWidget from MapControl to Map. I might have made a mistake.


dfm add does not check if file is already linked.

dfm add doesn't check if file is already symbolically linked, thus can lead to following error. if dfm add ran multiple times
-bash: cd: .emacs.d: Too many levels of symbolic links

dfm add now refuses to link symlinks and also attempt to work on files that don't exist.
I've uploaded version 2.2.2 with various bug fixes including this one to PyPi


[Site Request] add support for youtube created playlist

Checklist

[x] I'm reporting a site feature request
[x] I've verified that I'm running youtube-dlc version 2020.10.31
[x] I've searched the bugtracker for similar site feature requests including closed ones

Description

Youtube created Playlists cant be downloaded.
this is a example:
https://www.youtube.com/watch?v=bDVEdZaFHA8&list=RDCLAK5uy_kLWIr9gv1XLlPbaDS965-Db4TrBoUTxQ8
youtube-dlc only downloads first one since its in url so i tried converting it to normal playlist:
https://www.youtube.com/playlist?list=RDCLAK5uy_kLWIr9gv1XLlPbaDS965-Db4TrBoUTxQ8 (i know thats stupid but at least it opens in browser)
Verbose log with ' -f 140 --embed-thumbnail' args :
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-f', '140', '--embed-thumbnail', 'https://www.youtube.com/watch?v=bDVEdZaFHA8&list=RDCLAK5uy_kLWIr9gv1XLlPbaDS965-Db4TrBoUTxQ8']
[debug] Loading archive file None
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dlc version 2020.10.31
[debug] Python version 3.9.0 (CPython) - Linux-5.8.17-300.fc33.x86_64-x86_64-with-glibc2.32
[debug] exe versions: ffmpeg 4.3.1, ffprobe 4.3.1
[debug] Proxy map: {}
[youtube:playlist] Downloading playlist RDCLAK5uy_kLWIr9gv1XLlPbaDS965-Db4TrBoUTxQ8 - add --no-playlist to just download video bDVEdZaFHA8
[youtube:playlist] RDCLAK5uy_kLWIr9gv1XLlPbaDS965-Db4TrBoUTxQ8: Downloading page 1 of Youtube mix
[download] Downloading playlist: RDCLAK5uy_kLWIr9gv1XLlPbaDS965-Db4TrBoUTxQ8
[youtube:playlist] playlist RDCLAK5uy_kLWIr9gv1XLlPbaDS965-Db4TrBoUTxQ8: Collected 0 video ids (downloading 0 of them)
[download] Finished downloading playlist: RDCLAK5uy_kLWIr9gv1XLlPbaDS965-Db4TrBoUTxQ8

Possibly related to #57 as well. @insaneracist
I was able to confirm the same with "https://youtube.com/playlist?list=RDCLAK5uy_krjFmKbzWzkGvhqkYvvNnUbdrHy0QN1S8&feature=share&playnext=1".

It's assuming a playlist id starting with RD is a mix type, grabbing what it thinks is a video id from the last part of the playlist id, and then downloading an error page. It seems like playlists starting with RDCLAK5uy_ are generated from Youtube Music. eg
https://www.youtube.com/playlist?list=RDCLAK5uy_m_h-nx7OCFaq9AlyXv78lG0AuloqW_NUA
https://www.youtube.com/playlist?list=RDCLAK5uy_k8GvsGbBT2D5lvGqOc6gAitzrcWVvecYI


need to update extract_mpms_data() and plot_mpms_data() to take Js_T data

Currently the function looks for and extracts fc_data, zfc_data, rtsirm_cool_data, and rtsirm_warm_data. It needs to also look for and extract Js T data that is often measured upon cooling.
def extract_mpms_data(df, specimen_name):
    """
    Extracts and separates MPMS (Magnetic Property Measurement System) data
    for a specific specimen from a dataframe.

    This function filters data for a given specimen and separates it based on
    different MagIC measurement method codes. It specifically looks for data
    corresponding to 'LP-FC' (Field Cooled), 'LP-ZFC' (Zero Field Cooled),
    'LP-CW-SIRM:LP-MC' (Room Temperature SIRM measured upon cooling), and
    'LP-CW-SIRM:LP-MW' (Room Temperature SIRM measured upon Warming).

    Parameters:
        df (pandas.DataFrame): The dataframe containing MPMS measurement data.
        specimen_name (str): The name of the specimen to filter data for.

    Returns:
        tuple: A tuple containing four pandas.DataFrames:
            - fc_data: Data filtered for 'LP-FC' method.
            - zfc_data: Data filtered for 'LP-ZFC' method.
            - rtsirm_cool_data: Data filtered for 'LP-CW-SIRM:LP-MC' method.
            - rtsirm_warm_data: Data filtered for 'LP-CW-SIRM:LP-MW' method.

    Example:
        >>> fc, zfc, rtsirm_cool, rtsirm_warm = extract_mpms_data(measurements_df, 'Specimen_1')
    """

    specimen_df = df[df['specimen'] == specimen_name]

    fc_data = specimen_df[specimen_df['method_codes'].str.contains('LP-FC')]
    zfc_data = specimen_df[specimen_df['method_codes'].str.contains('LP-ZFC')]
    rtsirm_cool_data = specimen_df[specimen_df['method_codes'].str.contains('LP-CW-SIRM:LP-MC')]
    rtsirm_warm_data = specimen_df[specimen_df['method_codes'].str.contains('LP-CW-SIRM:LP-MW')]

    return fc_data, zfc_data, rtsirm_cool_data, rtsirm_warm_data

Need to make it so that it is an option to plot the Js-T data as well.

Need to make it so that it is an option to plot the Js-T data as well.

Possibly low-temperature loops as well. This is a good example of where the rockmag_functions library comes in helpful, as the functions for the VSM data can also be used on the low-temperature loops. Requires the upload of a loop correction file for the field steps.

Possibly low-temperature loops as well.

Agreed. My thought here is that developing the functions for the VSM data first (more likely to be used during IRM Summer School) makes sense. Those can then be brought in to enable plotting of low-temperature loops developed on the MPMS.

It likely makes sense to change the name of the extract_mpms_data to be more specific about what data are being extracted as this function is currently specific to dc temperature cycling data.


Update sdp_on_bf16 option for ST example

What does this PR do?
Update the argument options for sentence transformer examples with sdp_on_bf16 to solve the issue from
https://github.com/huggingface/optimum-habana/pull/1555/files
and tested/verified with results. Thanks.

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


--star_index argument is ignored with --aligner 'star_rsem' option

Description of the bug
In my pipeline I specified the path to both the STAR and the RSEM index, since they were generated independently
This is how the command looks like:
...
STAR_INDEX=/mnt/disks/data/resources/genomes/STAR/GRCh38_overhang_50bp
RSEM_INDEX=/mnt/disks/data/resources/genomes/RSEM/GRCh38_gencode

$NEXTFLOW run nf-core/rnaseq \
	--input data/samplesheet.csv \
	--fasta $GENOME_SEQUENCE \
	--gtf $GENOME_ANNOTATION --gencode \
	--star_index $STAR_INDEX \
	--rsem_index $RSEM_INDEX \
	--aligner 'star_rsem' \
	--skip_markduplicates \
	-profile docker \
	--max_cpus 16 \
	--max_memory '100.GB'

as you can see, the STAR index was manually generated with STAR in a separated folder and not with RSEM.
The problem I'm facing is that the pipeline is calling STAR with the path to the rsem_index in place of the proper one.
I think tha this is happening because with the option --aligner 'star_rsem' STAR is not called directly but through the RSEM function rsem-calculate-expression thus ignoring the --star_index argument
Expected behaviour
I'm aware that the simple fix is to generate the star index together with the RSEM one with the rsem-prepare-reference function but I think it would be worth adding a warning in case someone tries to specify the STAR index with the --aligner 'star_rsem'  option
Log files
Error executing process > 'RNASEQ:QUANTIFY_RSEM:RSEM_CALCULATEEXPRESSION (RT112_siNUMB_R1)'

Caused by:
  Process `RNASEQ:QUANTIFY_RSEM:RSEM_CALCULATEEXPRESSION (RT112_siNUMB_R1)` terminated with an error exit status (255)

Command executed:

  INDEX=`find -L ./ -name "*.grp" | sed 's/.grp//'`
  rsem-calculate-expression \
      --num-threads 6 \
      --temporary-folder ./tmp/ \
      --strandedness reverse \
      --paired-end \
      --star --star-output-genome-bam --star-gzipped-read-file --estimate-rspd --seed 1 \
      RT112_siNUMB_R1_1_val_1.fq.gz RT112_siNUMB_R1_2_val_2.fq.gz \
      $INDEX \
      RT112_siNUMB_R1

  rsem-calculate-expression --version | sed -e "s/Current version: RSEM v//g" > rsem.version.txt

Command exit status:
  255

Command output:
 STAR --genomeDir ./GRCh38_gencode  --outSAMunmapped Within  --outFilterType BySJout
 --outSAMattributes NH HI AS NM MD  --outFilterMultimapNmax 20  --outFilterMismatchNmax 999
 --outFilterMismatchNoverLmax 0.04  --alignIntronMin 20  --alignIntronMax 1000000  --alignMatesGapMax 1000000
 --alignSJoverhangMin 8  --alignSJDBoverhangMin 1  --sjdbScore 1  --runThreadN 6  --genomeLoad NoSharedMemory
 --outSAMtype BAM Unsorted  --quantMode TranscriptomeSAM  --outSAMheaderHD @HD VN:1.4 SO:unsorted
  --outFileNamePrefix ./tmp//RT112_siNUMB_R1  --readFilesCommand zcat
  --readFilesIn RT112_siNUMB_R1_1_val_1.fq.gz RT112_siNUMB_R1_2_val_2.fq.gz

System

Hardware: Google Cloud Engine VM
OS: Ubuntu 20

Nextflow Installation

Version: nextflow-21.02.0-edge-all

Container engine

Engine: Docked
Version: 20.10.3, build 48d30b5

Hi @ftucos ! Well spotted and apologies for the inconvenience! Yes, the pipeline has a number of workarounds to deal with STAR indices at the moment. The main reason I implemented things in this way is because we have to support an older version of STAR in this pipeline for legacy purposes in order to support the AWS iGenomes references we provide as standard (see footnote here). However, because we don't have any RSEM indices on AWS iGenomes we can use the latest version of STAR with that and so I added functionality to that module to build the entire index including the STAR files (see https://github.com/nf-core/rnaseq/issues/511). In any case, for the purposes of simplicity with the Nextflow implementation I would suggest that the STAR and RSEM indices are placed in the same folder and provided via --rsem_index

Hi @ftucos. As you suggested, the pipeline will now generate a warning when --aligner star_rsem, --rsem_index  and --star_index are provided together. Thanks for reporting!


Keras model doesn't compile for Edge-tpu if Dense layer contains [16, 32, 64, ...] neurons

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Google Colab
TensorFlow version (use command below): 1.15.0-dev20190626, 2.0-beta1
Python version: 3

Describe the current behavior
Couldn't convert keras model for Edge-tpu when Dense layer consists of [16, 32, 64, 128, 2^n and n>3]  neurons , but work for any other number.
Error:
 ERROR: :119 std::abs(input_product_scale - bias_scale) <= 1e-6 * std::min(input_product_scale, bias_scale) was not true.
ERROR: Node number 33 (FULLY_CONNECTED) failed to prepare.

Describe the expected behavior
Convert keras model for Edge-tpu successfully
Code to reproduce the issue
Colab to reproduce issue:
https://colab.research.google.com/drive/1H0L-UioDKHdl6QbT6OtJc5jplzWTi1qB

This looks like an Edge TPU issue that I'm not familiar with, so unassigning myself.

Is this still an issue? Can you check with TF2.0 and let us know whether the issue persists with latest TF version. Thanks!

@YaroslavSchubert Is this still an issue? If not, please close this issue. Thanks!

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Temporarily disable test stdlib/NSStringAPI.swift

It is taking too long to execute.

@swift-ci smoke test and merge


version 0.2.14

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/118677
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.14 -m "<description of version>" 70aae2d9ca1c3a56606921c0972c05668681464d
git push origin v0.2.14

Also, note the warning: Version 0.2.14 skips over 0.2.0
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.


277 - The multi-currency payment gateway on the CSPR blockchain - Milestone 3

This review was not approved by the CRDAO Voting Associates.


Default viewer for values based on key for associative structures

It would be great to be able to assign default viewers to values of associative structures like maps, without using metadata, which often is a pain.
Example:
I use µlog for logging and tracing, a log value may look like this:
{:user-agent nil,
 :mulog/duration 2173538,
 :mulog/namespace "my.namespace",
 :mulog/outcome :ok,
 :mulog/parent-trace nil,
 :http-status 200,
 :mulog/root-trace #mulog/flake "4t1B0IuXNLj7lPIClpVFiSU-MwnVGYs1",
 :mulog/timestamp<PHONE_NUMBER>823,
 :uri "/",
 :mulog/trace-id #mulog/flake "4t1B0IuXNLj7lPIClpVFiSU-MwnVGYs1",
 :request-method :get,
 :mulog/event-name :my.namespace/http-request}

It would be great if you could add a key → viewer config to portal, that declares the default viewer for the value of a key.
{:mulog/duration  :portal.viewer/duration-ns
 :mulog/timestamp :portal.viewer/relative-time

 ; maybe even a custom view function...
 :mulog/outcome (fn [value] [:span {:style {:color (if (#{:ok} value) "green" "red")}} value])}

I have code that solves this for me, but I think this could be a feature that would be really handy to have in portal itself, as it allows for simple customization, without the hassle of adding metadata to everything.
I would like feedback before I decide if this is something I even could contribute on my own. 🙂

@MrEbbinghaus  Gotcha, I think this is mostly already supported in Portal via :portal.viewer/for {} in metadata. For example, here is how it's used for portal.nrepl.


Resync Whitehall scheduled queue after data sync

The scheduled queue goes out of sync after a full data sync, and this rake task needs to be run to resync it again.
/cc @gpeng

:+1:


Replace spree_{get,post,put,delete} with normal methods

These are deprecated as of https://github.com/solidusio/solidus/pull/1330

:heart: :+1: