Transferring files between express instances.

Hello,
What is the best way to transfer a file from a private internal express instance to another? Currently, I'm trying to send a file from PRIV to PUB, where PUB then relays the download to the user.

User makes a request to PUB.
PUB requests file form PRIV. PRIV uses res.download() to send file.
PUB gets response, clones response headers, executes res.send(res.body).
User downloads a 0-byte file. It's appropriately named but contents are invalid.

What am I missing for step 3? Must I save res.body to a local file and use res.download() again?
Your help is much appreciated!

You probably want to pipe things together -- on PUB, resFromPriv.pipe(res).

(you're making a limited case of HTTP proxy in this case.)

Ah, ill give it a try, thanks.

This did the trick. However, I had to do some finagling to get POST/FILE requests to process without hanging up. Turns out you need to perform proxy pipes before the bodyParse middleware, or you must buffer the request. I went with the pre-parse option.  Thanks!

Correct, because in Node.js, you can only read a stream one time, so once the body parser consumes the request stream, the stream will no longer have data to read.

@aredridel and @dougwilson : Would you mind pointing me in the right direction for migrating away from http.createClient to http.request? I can't seem to find a solution that pipes the original request without having to parse request.method. I'd rather not use a third party request module if I can avoid it. Here's the original createClient code.
var proxy = http.createClient(port,host);
var proxyRequest = proxy.request(req.method, url, req.headers);
proxyRequest.on('response', function (proxyResponse) {
  proxyResponse.pipe(res);
});
req.pipe(proxyRequest);

I would highly suggest you use a module like "request" to do this, otherwise all you're going to do is end up rewriting that module all over again as you run into each edge case the module solves.
As for how to do it without non-core modules, you may want to ask over on the core Node.js board for help, as I have never attempted to replicate the request module.

Also http-proxy module is a good choice. If you are looking to replicate them, you could always at least take a look at what they are doing in their source code as a reference :)

@dougwilson Definitely not looking to replicate. IYO, what's the best option to support my use case with as few lines of code as possible? http.createClient was perfect because I didn't need to write a case for each request method (the only exception was with file uploads).

Hm, I'm just not sure if there is a full description of your use case besides just "I need a reverse proxy". Can you provide a detailed description? Assume you are describing with enough detail where I could write an implementation that is not just a simple reverse proxy.

Basically, the simplest solution would be the following:
var request = require('request')

app.get('/some/file', function (req, res, next) {
  request('http://server/some/file')
  .on('error', next)
  .pipe(res)
})

That simple example, though, uses the request module; the equivalent in Node.js core would likely be many, many lines, especially because the .pipe method from the request module is not the same at all as the .pipe method in Node.js core, as it does header copying and other things.

And using http-proxy if you also wanted to forward the full request, the example would be the following:
var httpProxy = require('http-proxy')

// it is very important in this example that you do not read from req before this
// i.e. you cannot use body-parser prior to this or you'll loose the body
app.all('/some/route', function (req, res) {
  httpProxy.web(req, res, { target: 'http://server' }) // will request http://server/some/route
})

I was writing a bunch of stuff and then you posted what looks like a perfect solution. In your httpProxy case, would it also forward attached files?

Yes, it does forward attached files.

@dougwilson Thanks for your help, http-proxy was the way to go! Three lines, covers all request methods, handles uploads and downloads perfectly!
Question: Would it be better for performance to move createProxyServer() outside of the route?
app.all('/some/route', function (req, res) {
  var target = 'http://priv/some/route';
  var proxy = httpProxy.createProxyServer(); // questionable placement?
  proxy.web(req, res,{ target:target, ignorePath:true })
})

Would it be better for performance to move createProxyServer() outside of the route?

Looking at their code, I would definitely say yes :)

Sweet, thanks. You've helped me solve the original ticket and optimize :+1:


Constant blinking

The taskbar blinks, as long as the application is in use, when opening applications, which have not been opened in a while.

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


Slow booting time

Description
I find in one of our servers that if install this package laravel have a big slow booting. More than 4 seconds. If we remove this package all work correctly.
Possible some app provider register? Don't know, or need some php library version. Not errors on logs. Only slow booting.
Your environment
PHP 8.1.8
Laravel 9

cache your files using the php artisan icons:cache command it will work fast

Cache your files using the php artisan icons:cache command it will work fast

php artisan icons:cache
doesn't solve the problem at all, my boot time w/o blade fontawesome is 70 ms and with this package ( cached ) is 400 ms

I can confirm. On a new installation the boot time jumps from ~140 to ~280-300ms

Yeah, there are definitely issues with the way Blade UI Icons work. 🤷🏻 I think it's just because there are so many files, not really much we can do.
If you find a solution, feel free to PR it. 👍🏻

php artisan icons:cache worked for me. I had 27 seconds loadtime before caching.


[typing]Cannot get type 'Request' from outside of @types/multer

TypeScript Version:  2.0.0 beta
Code
installed @types/express and @types/multer .
import * as express from "express";
import * as multer from "multer";

type Request = express.Request & multer.Request;

Expected behavior:
no error
Actual behavior:
error

What's the error?

error TS2305 module multer has no exported nember Request.

Seems I can use Express.Request
strange

but the Express.Request is from express, not multer. through the Request in multer is declared in namespace Express


fix: coverage count

Instead of full match, start to count as soon
as there is 1 match
@andremedeiros pls give feedback

@jrainville it definitively does count things a lot more.
However if you debug on remix for example you can see that behaviour too. (passing multiple times on the same line)

Hmm, well as long it doesn't affect (badly) the percentages, I guess it's fine.


Acquire the default MainContext when initializing GTK

It is required for GTK to be able to own the default main context on the
thread that initialized it, among other things to run its main loop on
that.

Make this more explicit by already acquiring it here during
initialization. This also ensures that other API that checks that this
very thread owns the main context works out of the box.

This depends on https://github.com/gtk-rs/glib/pull/447

Good idea, thanks, 👍

Restarted CI.

This will be green once https://github.com/gtk-rs/gtk/pull/779 is merged.

Restarted CI.

@GuillaumeGomez green apart from appveyor being broken :) I'll fix that later... but it's worrying that gtk tests don't work there. The setup seems broken.

appveyor worked, travis didn't. Is it expected?


gramatic errors

The resulted system model is wrong, I found the following errors:
RosPublisher { -> RosPublishers {
RosSubscriber { -> RosSubscribers {
RosServiceServer { -> RosSrvServers {
RosActionClient { -> RosActionClients {
RosActionServer { -> RosActionServers {

RefServiceServer -> RefServer
RefActionClient -> RefClient
RefActionServer -> RefServer

Should be okay with #3.


Breakout

Thank you very much for a comprehensive implementation.
I ran Breakout with the current configuration, except changing the actors from 350 to 4 since I ran
into memory problems with Ray. I am using the same setup as specified tested on, except GTX 1060.
The code reports one line, but never updates this line. On Tensorboard I see progress, but at zero rewards.
Any advice?

Hi,
About the line in the console displaying the progress, it is normal that there is only one but it should be updated. If the number of self played games, the loss or the number of training steps does not change, there is a problem.
About the reward that remains at 0, it depends on the number of selfplayed games, it may be that it takes a lot of games to get a first reward. In particular, I remember that on breakout, it must first learn to press fire to start the game and then catch the ball. At first it should play almost in random mode. It can therefore take a little time during which it stays at zero.
However, we haven't had time yet to test enough breakout and Atari games. This requires a lot of computing power and can take a long time with only 4 actors. DeepMind reported using 350 actors.

Hi,
Thanks for the quick response.
Could you provide the version of Breakout that you used to test. I would
like to see some progress and then start working on it further.
On Thu, May 21, 2020 at 2:30 PM Werner Duvaud<EMAIL_ADDRESS>wrote:

Hi,
About the line in the console displaying the progress, it is normal that
there is only one but it should be updated. If the number of self played
games, the loss or the number of training steps does not change, there is a
problem.
About the reward that remains at 0, it depends on the number of selfplayed
games, it may be that it takes a lot of games to get a first reward. In
particular, I remember that on breakout, it must first learn to press fire
to start the game and then catch the ball. At first it should play almost
in random mode. It can therefore take a little time during which it stays
at zero.
However, we haven't had time yet to test enough breakout and Atari games.
This requires a lot of computing power and can take a long time with only 4
actors. DeepMind reported using 350 actors.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/werner-duvaud/muzero-general/issues/53#issuecomment-632059163,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALB2BSJDCKXMRKIR7OQDOXLRSUNFFANCNFSM4NGROGCQ
.

We used the version available in the games folder.
We just decreased some parameters (num_actors, num_channels, batch_size, etc.) to run it on a less powerful server.

Thanks, yes I meant provide these parameters.
I will look at the updates to determine what to use.
Thanks again for a comprehensive implementation. I researched thoroughly and decided to build upon your work since it is such a good inspiration. I hope to give some positive feedback, also on other Atari games.

Great, don't hesitate to join the discord to discuss the results and share your experience.
About hyperparameters, I do not remember exactly what I had. I lowered them until it worked. The parameters that I changed are as follows:
num_actors, num_channels, num_reduced_channels, batch_size, stacked_observations.
The hardware was: RTX 2080, 128GO RAM, 12 core Intel Xeon E5v3


Build failure: aws-sdk-cpp on x86_64-darwin

https://hydra.nixos.org/job/nixpkgs/trunk/aws-sdk-cpp.x86_64-darwin
Steps To Reproduce
Steps to reproduce the behavior:

build aws-sdk-cpp on x86_64-darwin

Build log

/tmp/nix-build-aws-sdk-cpp-1.11.336.drv-2/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:158: Failure
Expected equality of these values:
  ACCESS_KEY_ID_2
    Which is: "accessKeyId2"
  actualCredentials.GetAWSAccessKeyId().c_str()
    Which is: ""
[  FAILED  ] STSProfileCredentialsProviderTest.TestCredentialsLoadAndCache (0 ms)
/tmp/nix-build-aws-sdk-cpp-1.11.336.drv-2/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:235: Failure
Expected equality of these values:
  1
  stsCallCounter
    Which is: 0
[  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleWithStaticAndSourceProfile (0 ms)
/tmp/nix-build-aws-sdk-cpp-1.11.336.drv-2/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:285: Failure
Expected equality of these values:
  ACCESS_KEY_ID_2
    Which is: "accessKeyId2"
  actualCredentials.GetAWSAccessKeyId().c_str()
    Which is: ""
[  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleWithProcessCredentials (0 ms)
/tmp/nix-build-aws-sdk-cpp-1.11.336.drv-2/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:478: Failure
Value of: actualCredentials.IsExpiredOrEmpty()
  Actual: true
Expected: false
[  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleRecursively (0 ms)
/tmp/nix-build-aws-sdk-cpp-1.11.336.drv-2/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:526: Failure
Value of: actualCredentials.IsExpiredOrEmpty()
  Actual: true
Expected: false
[  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleSelfReferencing (0 ms)
/tmp/nix-build-aws-sdk-cpp-1.11.336.drv-2/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:581: Failure
Value of: actualCredentials.IsExpiredOrEmpty()
  Actual: true
Expected: false
[  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleSelfReferencingSourceProfile (0 ms)
[==========] 22 tests from 4 test suites ran. (6097 ms total)
[  PASSED  ] 16 tests.
make[2]: *** [tests/aws-cpp-sdk-identity-management-tests/CMakeFiles/aws-cpp-sdk-identity-management-tests.dir/build.make:185: tests/aws-cpp-sdk-identity-management-tests/aws-cpp-sdk-identity-management-tests] Error 1
make[2]: *** Deleting file 'tests/aws-cpp-sdk-identity-management-tests/aws-cpp-sdk-identity-management-tests'
make[1]: *** [CMakeFiles/Makefile2:869: tests/aws-cpp-sdk-identity-management-tests/CMakeFiles/aws-cpp-sdk-identity-management-tests.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

Notify maintainers
cc @orivej
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.48, NixOS, 24.11 (Vicuna), 24.11.20240831.12228ff`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.5`
 - nixpkgs: `/nix/store/bd4fmzws6n5542khxbifbkr6nrygi232-source`

Add a :+1: reaction to issues you find important.

I successfully built master’s aws-sdk-cpp locally on x86_64-darwin. It’s either a flaky build or a cursed Rosetta 2 issue. I’m going to try again on the community builder.

It breaks on the community builder 🫠

FWIW these are the failing tests:
aws-sdk-cpp-x86_64-darwin> /tmp/nix-build-aws-sdk-cpp-1.11.336.drv-4/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:158: Failure
aws-sdk-cpp-x86_64-darwin> Expected equality of these values:
aws-sdk-cpp-x86_64-darwin>   ACCESS_KEY_ID_2
aws-sdk-cpp-x86_64-darwin>     Which is: "accessKeyId2"
aws-sdk-cpp-x86_64-darwin>   actualCredentials.GetAWSAccessKeyId().c_str()
aws-sdk-cpp-x86_64-darwin>     Which is: ""
aws-sdk-cpp-x86_64-darwin> [  FAILED  ] STSProfileCredentialsProviderTest.TestCredentialsLoadAndCache (0 ms)
aws-sdk-cpp-x86_64-darwin> /tmp/nix-build-aws-sdk-cpp-1.11.336.drv-4/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:235: Failure
aws-sdk-cpp-x86_64-darwin> Expected equality of these values:
aws-sdk-cpp-x86_64-darwin>   1
aws-sdk-cpp-x86_64-darwin>   stsCallCounter
aws-sdk-cpp-x86_64-darwin>     Which is: 0
aws-sdk-cpp-x86_64-darwin> [  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleWithStaticAndSourceProfile (0 ms)
aws-sdk-cpp-x86_64-darwin> /tmp/nix-build-aws-sdk-cpp-1.11.336.drv-4/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:285: Failure
aws-sdk-cpp-x86_64-darwin> Expected equality of these values:
aws-sdk-cpp-x86_64-darwin>   ACCESS_KEY_ID_2
aws-sdk-cpp-x86_64-darwin>     Which is: "accessKeyId2"
aws-sdk-cpp-x86_64-darwin>   actualCredentials.GetAWSAccessKeyId().c_str()
aws-sdk-cpp-x86_64-darwin>     Which is: ""
aws-sdk-cpp-x86_64-darwin> [  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleWithProcessCredentials (0 ms)
aws-sdk-cpp-x86_64-darwin> /tmp/nix-build-aws-sdk-cpp-1.11.336.drv-4/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:478: Failure
aws-sdk-cpp-x86_64-darwin> Value of: actualCredentials.IsExpiredOrEmpty()
aws-sdk-cpp-x86_64-darwin>   Actual: true
aws-sdk-cpp-x86_64-darwin> Expected: false
aws-sdk-cpp-x86_64-darwin> [  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleRecursively (0 ms)
aws-sdk-cpp-x86_64-darwin> /tmp/nix-build-aws-sdk-cpp-1.11.336.drv-4/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:526: Failure
aws-sdk-cpp-x86_64-darwin> Value of: actualCredentials.IsExpiredOrEmpty()
aws-sdk-cpp-x86_64-darwin>   Actual: true
aws-sdk-cpp-x86_64-darwin> Expected: false
aws-sdk-cpp-x86_64-darwin> [  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleSelfReferencing (0 ms)
aws-sdk-cpp-x86_64-darwin> /tmp/nix-build-aws-sdk-cpp-1.11.336.drv-4/source/tests/aws-cpp-sdk-identity-management-tests/auth/STSProfileCredentialsProviderTest.cpp:581: Failure
aws-sdk-cpp-x86_64-darwin> Value of: actualCredentials.IsExpiredOrEmpty()
aws-sdk-cpp-x86_64-darwin>   Actual: true
aws-sdk-cpp-x86_64-darwin> Expected: false
aws-sdk-cpp-x86_64-darwin> [  FAILED  ] STSProfileCredentialsProviderTest.AssumeRoleSelfReferencingSourceProfile (0 ms)
aws-sdk-cpp-x86_64-darwin> [==========] 22 tests from 4 test suites ran. (6084 ms total)
aws-sdk-cpp-x86_64-darwin> [  PASSED  ] 16 tests.

I’m inclined to simply disable them on x86_64-darwin because I don’t think I feel like figuring out why on Earth this would happen.

Reproduced one success via Rosetta 2 on the community builder and one thanks to @grahamc. It’s just flaky. Asking the Hydra wizards to try restarting it again.

Hydra since managed to build it.


Rename imodel-bridge to itwin-connector-framework

Includes both package renames and symbol renames.

@calebmshafer I'll start making changes again once we hear back from instant on.

@calebmshafer I'll start making changes again once we hear back from instant on.

@zachzhu2016 sounds good. In the meantime I'm going to close my pull request that you pull into this branch.

@zachzhu2016 you're free to update this and merge it whenever you'd like now that 3.0 has started on master.

Abandon rename changes as we're moving imodel-bridge out of mono-repo to https://github.com/iTwin/connector-framework


出现段错误

Vim: Caught deadly signal SEGV
Error detected while processing function 112_PollServerReady[7]..112_Pyeval:Vim: Finished.
line    4:
Exception MemoryError: MemoryError() in <module 'threading' from '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/threading.pyc'> ignored
Segmentation fault: 11
您好，在安装完之后，出现了以上的段错误，请问您是怎么解决的呢？

@MarcusNerva 你好，请问你用的是什么系统呢？编译ycm的时候报错没有呢？

(base) bismarck@xiehanhuadeMBP2 .vimplus$ ./install.sh Platform type: Darwin Find /Users/bismarck/.vimrc already exists,backup /Users/bismarck/.vimrc to /Users/bismarck/.vimrc_bak_20200802215417? [Y/N] N Find /Users/bismarck/.vimrc.custom.plugins already exists,backup /Users/bismarck/.vimrc.custom.plugins to /Users/bismarck/.vimrc.custom.plugins_bak_20200802215419? [Y/N] N Find /Users/bismarck/.vimrc.custom.config already exists,backup /Users/bismarck/.vimrc.custom.config to /Users/bismarck/.vimrc.custom.config_bak_20200802215420? [Y/N] N Find /Users/bismarck/.vim already exists,backup /Users/bismarck/.vim to /Users/bismarck/.vim_bak_20200802215421? [Y/N] N xcode-select: error: command line tools are already installed, use "Software Update" to install updates Warning: vim 8.2.0850 is already installed and up-to-date To reinstall 8.2.0850, runbrew reinstall vimWarning: ctags 5.8_1 is already installed and up-to-date To reinstall 5.8_1, runbrew reinstall ctags==> Downloading https://mirrors.ustc.edu.cn/homebrew-bottles/bottles/gmp-6.2.0.mojave.bottle.tar.gz Already downloaded: /Users/bismarck/Library/Caches/Homebrew/downloads/3c651a65c777317ba1980355a0c302d57d42db28924c7a471cd8f5b50294376d--gmp-6.2.0.mojave.bottle.tar.gz ==> Downloading https://mirrors.ustc.edu.cn/homebrew-bottles/bottles/isl-0.22.1.mojave.bottle.tar.gz Already downloaded: /Users/bismarck/Library/Caches/Homebrew/downloads/14b6edf98e4be7ce4b767ed1e525f23ef164fd2d898882bc1e23f5f2e5554ab6--isl-0.22.1.mojave.bottle.tar.gz ==> Downloading https://mirrors.ustc.edu.cn/homebrew-bottles/bottles/gcc-9.3.0_1.mojave.bottle.tar.gz Already downloaded: /Users/bismarck/Library/Caches/Homebrew/downloads/584679ce1d8f39829cbf68fbe1f5957656523e05d9ad8f35388bbe1866527aa1--gcc-9.3.0_1.mojave.bottle.tar.gz Error: gcc 9.2.0_3 is already installed To upgrade to 9.3.0_1, runbrew upgrade gcc. ==> Downloading https://mirrors.ustc.edu.cn/homebrew-bottles/bottles/cmake-3.17.3.mojave.bottle.tar.gz Already downloaded: /Users/bismarck/Library/Caches/Homebrew/downloads/d6e7c12c3d161b0617f1cecd191d4f0619f4ab5146d52b05d829b83f2e882492--cmake-3.17.3.mojave.bottle.tar.gz Error: cmake 3.16.2 is already installed To upgrade to 3.17.3, run brew upgrade cmake. ==> Downloading https://beyondgrep.com/ack-v3.3.1 Already downloaded: /Users/bismarck/Library/Caches/Homebrew/downloads/e2eb0cdbad3035791e95d763143cd83170f5871125bf10c7311b33e4fab3fb8e--ack-v3.3.1 Error: ack 3.2.0 is already installed To upgrade to 3.3.1, run brew upgrade ack. mkdir: /Users/bismarck/.vim: File exists fatal: destination path '/Users/bismarck/.vim/plugged/YouCompleteMe' already exists and is not an empty directory. Please choose to compile ycm with python2 or python3, if there is a problem with the current selection, please choose another one. [2/3] 2 Compile ycm with python2. Searching Python 2.7 libraries... #######library_dirs############ ['/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/config', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib64', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib'] #######library_dirs############ filename:libpython2.7.dylib Found Python library: /System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/config/libpython2.7.dylib Found Python headers folder: /System/Library/Frameworks/Python.framework/Versions/2.7/include/python2.7 -- The C compiler identification is AppleClang <IP_ADDRESS>10046 -- The CXX compiler identification is AppleClang <IP_ADDRESS>10046 -- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/cc -- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/cc -- works -- Detecting C compiler ABI info -- Detecting C compiler ABI info - done -- Detecting C compile features -- Detecting C compile features - done -- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/c++ -- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/c++ -- works -- Detecting CXX compiler ABI info -- Detecting CXX compiler ABI info - done -- Detecting CXX compile features -- Detecting CXX compile features - done -- Found PythonLibs: /System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/config/libpython2.7.dylib (found suitable version "2.7.10", minimum required is "2.7")  -- Using libclang archive: /Users/bismarck/.vim/plugged/YouCompleteMe/third_party/ycmd/cpp/../clang_archives/libclang-8.0.0-x86_64-apple-darwin.tar.bz2 -- Using libclang to provide semantic completion for C/C++/ObjC -- Using external libclang: /private/var/folders/gz/l_dvw9kn6wx00lv6cn8drrth0000gn/T/ycm_build_5X12dJ/lib/libclang.dylib -- NOT using clang-tidy for static analysis. -- Configuring done -- Generating done -- Build files have been written to: /private/var/folders/gz/l_dvw9kn6wx00lv6cn8drrth0000gn/T/ycm_build_5X12dJ Scanning dependencies of target BoostParts [  1%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/filesystem/src/codecvt_error_category.cpp.o [ 11%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/filesystem/src/utf8_codecvt_facet.cpp.o [ 11%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/filesystem/src/operations.cpp.o [ 13%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/filesystem/src/unique_path.cpp.o [ 13%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/filesystem/src/path_traits.cpp.o [ 13%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/filesystem/src/path.cpp.o [ 13%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/filesystem/src/portability.cpp.o [ 15%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/filesystem/src/windows_file_codecvt.cpp.o [ 17%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/c_regex_traits.cpp.o [ 19%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/cpp_regex_traits.cpp.o [ 21%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/cregex.cpp.o [ 23%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/fileiter.cpp.o [ 25%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/icu.cpp.o [ 27%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/instances.cpp.o [ 29%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/posix_api.cpp.o [ 31%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/regex.cpp.o [ 33%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/regex_debug.cpp.o [ 37%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/regex_traits_defaults.cpp.o [ 37%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/regex_raw_buffer.cpp.o [ 39%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/static_mutex.cpp.o [ 41%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/usinstances.cpp.o [ 43%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/w32_regex_traits.cpp.o [ 45%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/wc_regex_traits.cpp.o [ 47%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/wide_posix_api.cpp.o [ 49%] Building CXX object BoostParts/CMakeFiles/BoostParts.dir/libs/regex/src/winstances.cpp.o [ 50%] Linking CXX static library libBoostParts.a /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(windows_file_codecvt.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(icu.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(instances.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(regex_debug.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(usinstances.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(w32_regex_traits.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(winstances.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(windows_file_codecvt.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(icu.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(instances.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(regex_debug.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(usinstances.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(w32_regex_traits.cpp.o) has no symbols /Library/Developer/CommandLineTools/usr/bin/ranlib: file: libBoostParts.a(winstances.cpp.o) has no symbols [ 50%] Built target BoostParts Scanning dependencies of target ycm_core [ 52%] Building CXX object ycm/CMakeFiles/ycm_core.dir/Candidate.cpp.o [ 58%] Building CXX object ycm/CMakeFiles/ycm_core.dir/Character.cpp.o [ 58%] Building CXX object ycm/CMakeFiles/ycm_core.dir/CandidateRepository.cpp.o [ 58%] Building CXX object ycm/CMakeFiles/ycm_core.dir/CharacterRepository.cpp.o [ 60%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ClangCompleter/ClangCompleter.cpp.o [ 66%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ClangCompleter/ClangUtils.cpp.o [ 66%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ClangCompleter/ClangHelpers.cpp.o [ 66%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ClangCompleter/CompilationDatabase.cpp.o [ 68%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ClangCompleter/CompletionData.cpp.o [ 70%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ClangCompleter/Documentation.cpp.o [ 72%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ClangCompleter/Range.cpp.o [ 74%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ClangCompleter/TranslationUnit.cpp.o [ 76%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ClangCompleter/TranslationUnitStore.cpp.o [ 78%] Building CXX object ycm/CMakeFiles/ycm_core.dir/CodePoint.cpp.o [ 80%] Building CXX object ycm/CMakeFiles/ycm_core.dir/CodePointRepository.cpp.o [ 82%] Building CXX object ycm/CMakeFiles/ycm_core.dir/IdentifierCompleter.cpp.o [ 84%] Building CXX object ycm/CMakeFiles/ycm_core.dir/IdentifierDatabase.cpp.o [ 86%] Building CXX object ycm/CMakeFiles/ycm_core.dir/IdentifierUtils.cpp.o [ 88%] Building CXX object ycm/CMakeFiles/ycm_core.dir/PythonSupport.cpp.o [ 90%] Building CXX object ycm/CMakeFiles/ycm_core.dir/Result.cpp.o [ 92%] Building CXX object ycm/CMakeFiles/ycm_core.dir/Utils.cpp.o [ 94%] Building CXX object ycm/CMakeFiles/ycm_core.dir/Word.cpp.o [ 96%] Building CXX object ycm/CMakeFiles/ycm_core.dir/versioning.cpp.o [ 98%] Building CXX object ycm/CMakeFiles/ycm_core.dir/ycm_core.cpp.o [100%] Linking CXX shared library /Users/bismarck/.vim/plugged/YouCompleteMe/third_party/ycmd/ycm_core.so [100%] Built target ycm_core -- The C compiler identification is AppleClang <IP_ADDRESS>10046 -- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/cc -- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/cc -- works -- Detecting C compiler ABI info -- Detecting C compiler ABI info - done -- Detecting C compile features -- Detecting C compile features - done -- Found PythonLibs: /System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/config/libpython2.7.dylib (found version "2.7.10")  -- Configuring done -- Generating done -- Build files have been written to: /private/var/folders/gz/l_dvw9kn6wx00lv6cn8drrth0000gn/T/regex_build_gY5Y6C Scanning dependencies of target _regex [ 66%] Building C object CMakeFiles/_regex.dir/regex_2/_regex.c.o [ 66%] Building C object CMakeFiles/_regex.dir/regex_2/_regex_unicode.c.o [100%] Linking C shared library /Users/bismarck/.vim/plugged/YouCompleteMe/third_party/ycmd/third_party/cregex/regex_2/_regex.so [100%] Built target _regex         __                __            __   __/_/___ ___  ____  / /_  _______  \ \ / / / __ __ / __ / / / / / /
\ V / / / / / / / // / / // (_  )
_/// // // ,//_/__/
//                     ...is now installed!
Just enjoy it!
p.s. Follow me at https://github.com/chxuan.
It takes 0 minutes.
`
您好我的系统是MacOS 10.14。请问这个问题是由什么引起的呢？我在别的地方看到是因为vim -lua导致的，我重新安装了vim 使它+lua了，但是依然存在这个问题。

各位，我的问题解决了，是由anaconda自带python编译YouCompleteMe导致的。通过和 @chxuan 的沟通，他建议我使用系统自带的python来编译。
请注意，采用系统自带的python并不用卸载anaconda，我将install.sh脚本中关于ycm下载安装的部分进行了更改，原本是：
`function install_ycm()
{
git clone https://gitee.com/chxuan/YouCompleteMe-clang.git ~/.vim/plugged/YouCompleteMe
cd ~/.vim/plugged/YouCompleteMe

read -p "Please choose to compile ycm with python2 or python3, if there is a problem with the current selection, please choose another one. [2/3] " version
if [[ $version == "2" ]]; then
    echo "Compile ycm with python2."
    python2.7 ./install.py --clang-completer
else
    echo "Compile ycm with python3."
    python3 ./install.py --clang-completer
fi

}我将if [[ $version == "2" ]]; then
echo "Compile ycm with python2."
python2.7 ./install.py --clang-completer处更改成了(因为我自带的python是2.7版本的，而且path是/usr/bin/python)：if [[ $version == "2" ]]; then
echo "Compile ycm with python2."
/usr/bin/python ./install.py --clang-completer`
之后重新编译了ycm，就解决了问题！
我的系统是MacOS 10.14.5。希望这个issue可以帮助与我有同样问题的人。


Use external cloud provider instead of aws

What would you like to be added:
I would like the kubelet to be initialized & use the external cloud provider rather than AWS. We should provide the user an option to customize which provider it uses on the worker node.
Why is this needed:
Using an external cloud provider is the default way that EKS creates clusters post 1.22. We should give the option to customers to use the external cloud provider rather than the in-tree aws provider.

We're targeting 1.25 to switch to the external cloud provider, but we don't currently plan to make this configurable. It'll just toggle based on the k8s version.


Create a logger for aframe visualization

AFrame exporter needs to include or not a logger to register preferred in the pharo side. events like

nodes used with popup.
camera move
type of device
session time.

Suggestions you may use AJAX from javascript with Zinc server from Pharo in order to get this

Aframe is not longer provided, then the logger is not need it anymore


Add timeout feature for geemap.zonal_statistics

Hello! I have stumbled upon a timeout issue when I run zonal statistics. After a couple of hours, sometime the function hangs at the "Calculating Statistics" stage. Is it possible to add a timeout feature, such that the functions returns a timeout error? Here's an example from the requests library: https://docs.python-requests.org/en/latest/user/quickstart/#timeouts.
Cheers!
Sasha

Good suggestion. It has been implemented. Update the package using geemap.update_package(). The default timeout is 300 seconds.


hop-node: Remove dead code

Thanks for catching that. Removed https://github.com/hop-protocol/hop/pull/313/commits/8da37ccb7eb4fde61c0f9b7f9a902ae6b1846596
Agree, removed https://github.com/hop-protocol/hop/pull/313/commits/8da37ccb7eb4fde61c0f9b7f9a902ae6b1846596
The test suite needs an overhaul and the helpers file has some useful methods so I'm not going to touch the test files for now. The test suite can be cleaned up in a later PR


Bypass-all-cookies-except

For some reasons I like to have the inverse logic from the current "bypass-on-cookie" feature.
I do not want to define cookies which bypass but I like to define cookies which are ok for caching.
When a request contains a single cookie that is not on the cookie whitelist then it should bypass the cache. Caching is only allowed when all cookies are on the whitelist.
Main reasons:

new unknown cookies from plugins etc.
security: when updates change the cookie name then protected pages may accidently being cached with the current bypass-on-cookie feature

Closing this as the main repo has moved over to workers-sdk monorepo, located here: https://github.com/cloudflare/workers-sdk/tree/main/templates.
Feel free to reopen this issue in that repo if you're still experiencing this issue.


Fix: credential_process to merge .aws/config instead of .aws/credentials into generated file

When using credential_process --generate, use credential_process.destinationConfig instead of file.destination variable to determine a config file to open/merge from. This also syncs up the -o flag (output, which is written to credential_process.destinationConfig) to read and output to the same filepath.

Go tests build result:
[WARNING] The 'rev' field of repo 'https://github.com/dnephin/pre-commit-golang' appears to be a mutable reference (moving tag / branch).  Mutable references are never updated after first install and are not supported.  See https://pre-commit.com/#using-the-latest-version-for-a-repository for more details.  Hint: `pre-commit autoupdate` often fixes this.


merge(sync) master changes to develop branch

Codecov Report

Merging #4 into develop will decrease coverage by 0.28%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop       #4      +/-   ##
===========================================
- Coverage    77.29%   77.01%   -0.29%
===========================================
  Files           47       47
  Lines         9484     9332     -152
===========================================
- Hits          7331     7187     -144
+ Misses        2153     2145       -8

Impacted Files
Coverage Δ

superset/__init__.py
72.5% <ø> (-0.89%)
:arrow_down:

superset/connectors/base/views.py
71.42% <0%> (-10.39%)
:arrow_down:

superset/dict_import_export_util.py
21.05% <0%> (-7.52%)
:arrow_down:

superset/cache_util.py
48.38% <0%> (-5.9%)
:arrow_down:

superset/legacy.py
13.43% <0%> (-4.88%)
:arrow_down:

superset/stats_logger.py
60.97% <0%> (-3.47%)
:arrow_down:

superset/translations/utils.py
86.66% <0%> (-2.81%)
:arrow_down:

superset/views/annotations.py
73.8% <0%> (-2.28%)
:arrow_down:

superset/connectors/sqla/views.py
63.06% <0%> (-1.29%)
:arrow_down:

superset/connectors/connector_registry.py
88.57% <0%> (-1.18%)
:arrow_down:

... and 36 more

Continue to review full report at Codecov.

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


Improve documentation on refractored config

What problem or use case are you trying to solve?
Improve documentation for config updated in https://github.com/OpenDevin/OpenDevin/pull/1552
Describe the UX of the solution you'd like
Update the documentation sites with different config & example config.toml for people to directly copy?
Do you have thoughts on the technical implementation?
Describe alternatives you've considered
Additional context

Great point, I'll do that.
There is make setup-config which creates a working file, it has just a few variables though, just enough to start the app with a regular model.
The .env vars continue to work, they're the same vars.

This should be fixed, by https://github.com/OpenDevin/OpenDevin/pull/1929 and the examples in tutorials/readme for evaluation. Please feel free to reopen if not.


Install latest ansible with required modules.

The latest CentOS stream could not install ansible2.9.* which include all modules we need, now it just install latest ansible with core only, we need to install other modules manually. For RHEL, ansible 2.9 will be installed.
Signed-off-by: CS Zhang<EMAIL_ADDRESS>

@cs-zhang please squash your commits into 1.

@cs-zhang any change at https://github.com/ocp-power-automation/ocp4-upi-powervs/blob/d7a5ffb36121bdc64068ed202971365210551a40/variables.tf#L324? In the description of this PR it is said latest ansible core will be installed.

@yussufsh  For rhel, the yum still install ansible 2.9.x with the enabled repo, and ansible-galaxy still work. Only the centos will install latest ansible.

/approve
/cc @sajauddin for final approval


Show load and file_extensions in accessors

Like #26 this seemed an interesting intellectual exercise.
This functionality looks scary and technically it is internals.
But I have been using it in ExprTools.jl in like every version of julia back til 1.0
Having to use an internal method for this is another manifestation of https://github.com/JuliaLang/julia/issues/20555
I put this in a helper function so that after #26 we could add it to that list also

OK actually to keep people from getting confused, could we make the type_accessors a complete standalone "visible" method? It is a bit different from accessors (they don't "eat" an actual object).
Other than that this looks wunderbar, thanks again!
PS I see you rebased, perfect. :D

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (a2e4441) to head (86a1e15).

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #27   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          267       280   +13
=========================================
+ Hits           267       280   +13

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

I think it is pretty clear from how it is printed:

julia> AbstractFBCModels.required_accessors()
[1] reactions(a::AbstractFBCModels.AbstractFBCModel) @ AbstractFBCModels none:0
[2] n_reactions(a::AbstractFBCModels.AbstractFBCModel) @ AbstractFBCModels none:0
[3] metabolites(a::AbstractFBCModels.AbstractFBCModel) @ AbstractFBCModels none:0
...
[11] save(a::AbstractFBCModels.AbstractFBCModel, path::String) @ AbstractFBCModels none:0
[12] load(::Type{A}, path::String) where A<:AbstractFBCModel @ AbstractFBCModels ~/repos/AbstractFBCModels.jl/src/io.jl:7
[13] filename_extensions(::Type{A}) where A<:AbstractFBCModel @ AbstractFBCModels ~/repos/AbstractFBCModels.jl/src/io.jl:25

but up to you.
Its getting late here so I will leave it with you, if that ok?
Feel free to push to this branch before you merge.

actually yeah with the types it looks completely transparent. Thanks a lot!


More examples

I wanted to get started with chrome testing using client library. But not able to find relevant sample code. If there was a sample code how to start browser and go to URL(eg google.com) and close the browser that will be great help for people who are getting started.
Eg :+1:
I used
  service, err :=
    selenium.NewChromeDriverService("/home/johndoe/workspace/toolchain/x86_64-linux/chromedriver",
    4444)
  caps := selenium.Capabilities{"browserName": "chrome"}
  wd, err := selenium.NewRemote(caps, "https://www.google.com")
  wd.Refresh()
  if err != nil {
    glog.Errorf("Failed")
    panic(err) // panic is used only as an example and is not otherwise recommended.
  }
  defer service.Stop()

And the above code did not start browser but did not fail. If there was a sample code I could have just coded based on that.

Thanks for the feedback and suggestions.
Did you see the example code in the documentation?
https://godoc.org/github.com/tebeka/selenium
(Click "Example" to expand it. It doesn't seem to have a direct link).
Also, in general, a good way to understand how to use any API is by looking at its unit tests:
https://github.com/tebeka/selenium/blob/master/remote_test.go
I'll definitely add more examples to the documentation, since we only have the one and it doesn't exercise a huge swath of the API.

Units test are too complex to write a simple program that will open a browser

Dear @arpitjindal97 ,
Thanks for this life / time saving snipet . It worked for me .
Also "browserPath" is unused in this code, and It doesn t look for chromedriver in the current Bin directory.
It looks in $PATH on my Ubuntu 18.04.
I suppose its the missing part...passing "browserPath" to the webdriver instentiation...
Have a nice day
For google to crawl : how to make tebeka selenium go webdriver work with chrome web browser

On macOS you can:
brew cask install chromedriver

For anyone wondering, here's how you use the browserPath and start chrome/chromium headless:
package main

import (
	"fmt"
	"github.com/tebeka/selenium"
	"github.com/tebeka/selenium/chrome"
	"net"
	"os"
	"os/exec"
	"strconv"
)

const (
    proxyPort = 1337
)

func main() {
	browserPath := GetBrowserPath("chromium")
	port, err := pickUnusedPort()

	var opts []selenium.ServiceOption
	service, err := selenium.NewChromeDriverService("chromedriver",
		port, opts...)

	if err != nil {
		fmt.Printf("Error starting the ChromeDriver server: %v", err)
	}

	caps := selenium.Capabilities{
		"browserName": "chrome",
	}

        // NB: Path is the important part here
	caps.AddChrome(
		chrome.Capabilities{
			Path: browserPath,
			Args: []string{
				"--window-size=1920,1080",
				"--headless",
				"--no-sandbox",
			},
		},
	)

	wd, err := selenium.NewRemote(caps, "http://<IP_ADDRESS>:"+strconv.Itoa(port)+"/wd/hub")
	if err != nil {
		panic(err)
	}

	wd.Refresh()

	wd.Get("https://google.com")
	defer service.Stop()
}

func pickUnusedPort() (int, error) {
	addr, err := net.ResolveTCPAddr("tcp", "<IP_ADDRESS>:0")
	if err != nil {
		return 0, err
	}

	l, err := net.ListenTCP("tcp", addr)
	if err != nil {
		return 0, err
	}
	port := l.Addr().(*net.TCPAddr).Port
	if err := l.Close(); err != nil {
		return 0, err
	}
	return port, nil
}

func GetBrowserPath(browser string) string {
	if _, err := os.Stat(browser); err != nil {
		path, err := exec.LookPath(browser)
		if err != nil {
			panic("Browser binary path not found")
		}
		return path
	}
	return browser
}


Implemented update feature and made the update script safer

Implemented update feature and made the update script safer,
you can now run code --update to update the software.
If an update fails the old software will no longer be deleted.

Huh? Did something go wrong? All the threads are marked as resolved, but the only additional commit is the merge of my own changes into your PR. None of the suggestions are actually there. >_>

And also, you might've overlooked this alternative backup approach I mentioned in the main review comment I posted:

First download the files into a separate "staging" directory (maybe code_update or something). Then, once everything was successful, replace the existing files. In the case of errors just don't copy them and get rid of the code_update directory again.

I personally think this might be a cleaner approach, but I'd be happy to hear your thoughts as well. Maybe there are some benefits with the current implementation that I'm overlooking. If not, do you want to do those changes yourself or should we merge this PR for now and I'll do it myself?

Hmm maybe there are some issues with github, i wrote this message once and then it just disappeared :/
I can't see any "apply" button other then this warning:

Only those with write access to this repository can merge pull requests.

As for your suggestion i think it sounds great, it may depending on how it is implemented result in fewer read/write cycles too. Since we can abort install early if just one of the files fail. I suggest however that we merge this PR for now and maybe one of us could work more on it later :)

You might need to unresolve the threads first. These buttons should then work again and add the commits to your own branch.

Unless I don't understand how GitHub works and I have to click those buttons myself (which would seem odd to me).

Right, im eating dinner at the moment but ill check it out asap. :)

No worries, I got time ^^
I added you on discord btw (was linked on your profile) if you want to chat some more about general stuff (e.g. about my dev setup, how to use git, etc...)


speedtest feature

helo
i hope looking glass will have feature to do speedtest.. not from client to LookingGlass server.. but from LookingGlass to another ip address outside.

If you run configure.sh, you will get this option:

Enter the size of test files in MB (Example: 25MB 50MB 100MB)

By generate these files, the client have the option to download a test file to measure the download speed.
Or what kind of speedtest are you after? Do you have an example?

i mean speedtest between LookingGlass script.
if i have 3 server in 3 different countries which also using LookingGlass.
user want to know from server A to server B how fast they are? note: user does not have root access..they just want to know the speed result only.
hemm...possible?

A simple solution is to make a PHP script that executes a curl or wget command and print the result. Example of a curl command with nice outout.
Then you simply link to this script on the LookingGlass page.

LookingGlass curent can't test to another server, LG can get IP server chose to test with host?


random principle

From @redprop on January 30, 2015 22:17
Click on connections / ifpending , then random profile , the message " Requested profile is unavailable "
Copied from original issue: friendica/red#882

From @friendica on January 30, 2015 22:31
[master ef04d21] fix random profiles
Should be fixed now


Auto expand rows

When running the app on iPhone 6, after adding the third picture, it should be on a line below, instead of scrolling off to the side. The same way the text entry field expands downwards, the container of the cells should also expand accordingly.
(BTW, fantastic work done on the StackViewController)

The demo app's implementation (ImageAttachmentView) uses a UIStackView to lay out the thumbnail images horizontally and UIStackView is limited to a single row or column.
Laying the thumbnails out in a single line is a limitation of a stack view, not a bug with the project.
From Apple's docs on UIStackView:

The UIStackView class provides a streamlined interface for laying out a collection of views in either a column or a row

@GJNilsen This was intentional, as explained by @jessedc (thanks!)


Got error while starting example - Module build failed: Error: Cannot find module '@babel/core'

Here is the error log:

yarn run v1.13.0
$ webpack-dev-server --hot --host <IP_ADDRESS> --config=./webpack.dev.config.js
The setup option is deprecated and will be removed in v3. Please update your config to use before
Project is running at http://<IP_ADDRESS>:8080/
webpack output is served from /
Content not from webpack is served from D:\workspaces-test\frameless-titlebar-master\example\dist
Hash: 1caf05bacc4337ba547c
Version: webpack 3.12.0
Time: 861ms
Asset       Size  Chunks                    Chunk Names
bundle.js     301 kB       0  [emitted]  [big]  main
bundle.js.map     347 kB       0  [emitted]         main
index.html  183 bytes          [emitted]
[0] (webpack)/hot/log.js 1.04 kB {0} [built]
[2] (webpack)/hot/emitter.js 77 bytes {0} [built]
[3] multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 webpack/hot/dev-server ./src/index.js 52 bytes {0} [built]
[4] (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 7.93 kB {0} [built]
[5] external "url" 42 bytes {0} [not cacheable]
[6] ./node_modules/strip-ansi/index.js 161 bytes {0} [built]
[8] ./node_modules/loglevel/lib/loglevel.js 7.86 kB {0} [built]
[9] (webpack)-dev-server/client/socket.js 1.08 kB {0} [built]
[11] (webpack)-dev-server/client/overlay.js 3.67 kB {0} [built]
[12] ./node_modules/ansi-html/index.js 4.26 kB {0} [built]
[13] ./node_modules/html-entities/index.js 231 bytes {0} [built]
[16] (webpack)/hot nonrecursive ^./log$ 170 bytes {0} [built]
[18] (webpack)/hot/dev-server.js 1.61 kB {0} [built]
[19] (webpack)/hot/log-apply-result.js 1.31 kB {0} [built]
[20] ./src/index.js 1.64 kB {0} [not cacheable] [built] [failed] [1 error]
+ 6 hidden modules
ERROR in ./src/index.js
Module build failed: Error: Cannot find module '@babel/core'
babel-loader@8 requires Babel 7.x (the package '@babel/core'). If you'd like to use Babel 6.x ('babel-core'), you should install 'babel-loader@7'.
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:581:15)

Did you follow https://github.com/Cristian006/frameless-titlebar#try-the-example and yarn install the dependancies in both the root directory of frameless-titlebar and the example directory?

thy for your reply. It works now.
So I can not use the /example as a template ?
How should I change the /example project to run it without root directory?
regards
dit

I recommend using something like https://github.com/electron-react-boilerplate/electron-react-boilerplate since the example was just meant to showcase the titlebar, not necessarily meant as a project boilerplate. To answer your question though, if you go into the example folder and delete line #41 from package.json that will remove the outer folder as a project dependency. Then run yarn add frameless-titlebar to install frameless-titlebar as a regular node module 👍


Updated Teams.js with tokenReceived event

This 'tokenReceived' event will allow the caller to ensure the API token has been retrieved before spawning any new instances of the bot.
I identified this issue when attempting to spawn a bot instance controller.spawn({ serviceUrl: ... }) directly after initializing the bot controller Botkit.teamsbot({ clientId: 'xxx, clientSecret: 'xxx' });.  Unfortunatly the token had not yet been obtained, which resulted in the bot instance without the API token.
A workaround was utilising setTimeout, but Ben suggested this change (Thanks again for the help).

Hi @benbrown is it possible to get this stale tag removed again?

@markgrahamlp I will do my best to get this published as 0.7.7 in the not too distant future.

not stale

Hi @benbrown can this stale tag be removed again?


copy error message

Location
/mywallet
Expected behavior
Shortcode or address is wrong length.
Actual behavior
Error. Shortcode or address is the wrong length.
Steps to reproduce the problem
my wallet > send SWT > type in wrong address
check with @faffydee if he agrees with dropping the word "error"

I would propose checking with faffy before making issue?


Add a script to run a single integration test

Added a npm script called test-single which can be use as:
npm run test-single -- endpoints/address/tests/integration_test.js to only run the test which was passed to the script.
node test/run_integration_test_single.js <path to test> will work the same.

some of these tests are really flaky :/

Bus tests failing again for no particular reason.
Some of these tests are really flaky

Yup. It's because we don't mock or stub out any of the sources so we need to make requests to them that might fail for a number of reasons. The original reasoning was that it would alert us to breaking changes to the source sites during our nightly CI runs.
I'll take a look at what we can do to fix this and open up a issue/pr with a proposal.


Add simulate-sce module

This draft PR starts to add a simulate-sce module.
The R scripts are derived from https://github.com/AlexsLemonade/OpenScPCA-analysis/tree/63f6d91464ef86bf15b8c2e9c7e56f048e4c1a9b/analyses/simulate-sce, with additional nextflow wrapping.
The main processes are pretty straightforward, though I used the stageAs directive a few times to simplify managing files that will have the same names for input and output.
I also implemented a few Util functions, namely to get lists of projects and project paths from an S3 release bucket.
Note that the way this is implemented is a divergence from my initial idea: I now think we will likely want to have some standard input channels to subworkflows, which may be just a channel of [project_id, project_path] as I have done here.
While I initially thought I would have the whole project directory as inputs to processes themselves, I quickly realized that this resulted in lots of extra staged files, and I wanted to be able to work in parallel by sample, so I also implemented some channel logic to create sets of input files for each sample in a project. I expect that some of this will be generally useful as well, so I expect to move some part or parts of it to Utils.
The idea would be to have custom functions that take a channel of Projects and emit channels with specific subsets of the files: most likely with only processed rds or hdf5 files. I think that would happen in a later PR, and maybe after we have a few modules implemented to see what patterns emerge.
Right now this workflow works locally with a local R install, but the Docker image specified is not actually not correct, as it does not have splatter installed (and it has a lot of extras!). I plan to make a proper Docker image in openscpca-analysis, then we can use that here. But I thought that I would get this draft up for a look before I went too far along.

I have now run this and with the added bug fixes, everything worked! A few random failures, as would be expected, but every sample now has simulated data. Ready for another look @allyhawkins


Cannot invoke "com.reandroid.arsc.value.Entry.isScalar()" because "item" is null

Describe the bug
Running APKEditor with any of the options in any modes besides info generates a NullPointerException in the main thread almost immediately.
To Reproduce
Steps to reproduce the behavior:

Used version 'APKEditor-1.4.1.'
Operating system 'Windows 10'
Command 'java -jar APKEditor-1.4.1.jar d -i .\myapp.apk' However using all options this still fails

Log/Stacktrace
java.lang.NullPointerException: Cannot invoke "com.reandroid.arsc.value.Entry.isScalar()" because "item" is null
        at com.reandroid.apk.ApkModule.lambda$filterResFileEntries$1(ApkModule.java:648)
        at com.reandroid.arsc.item.StringItem.lambda$getUsers$0(StringItem.java:77)
        at com.reandroid.utils.collection.ComputeIterator.getNext(ComputeIterator.java:52)
        at com.reandroid.utils.collection.ComputeIterator.hasNext(ComputeIterator.java:38)
        at com.reandroid.utils.collection.CollectionUtil.toList(CollectionUtil.java:180)
        at com.reandroid.apk.ApkModule.filterResFileEntries(ApkModule.java:657)
        at com.reandroid.apk.ApkModule.listResFiles(ApkModule.java:595)
        at com.reandroid.apk.ApkModule.listResFiles(ApkModule.java:581)
        at com.reandroid.apkeditor.protect.DirectoryConfuser.confuse(DirectoryConfuser.java:44)
        at com.reandroid.apkeditor.protect.Protector.runCommand(Protector.java:59)
        at com.reandroid.apkeditor.Options.runCommand(Options.java:59)
        at com.reandroid.apkeditor.Main.run(Main.java:136)
        at com.reandroid.apkeditor.Main.execute(Main.java:72)
        at com.reandroid.apkeditor.Main.main(Main.java:57)

Used apk file
I Cannot disclose the name of the app due to NDA contract. but this app is a real piece of work with multiple layers of obfuscation.
Additional context
apktool can decompile this but fails to recompile because of the DUMMY VALUE in the assets and the obfuscated resources. (this is my best guess so far).

I believe this issue is already fixed on this Commit, try with latest build from here

Thanks a lot, using the new commit I was able to both decompile and recompile the app. Can you summarize how you bypass the resource obfuscation issue? I have tried everything and failed.


Set current/default application at the start of component create

Resolves #291

@kadel @containscafeine can you have please review this ?

Blocked on #330

Isn't this PR the same as https://github.com/redhat-developer/odo/pull/358/files ?
@containscafeine @surajnarwade ?


zeek: 4.2.2 -> 5.1.2

Description of changes
https://github.com/zeek/zeek/releases/tag/v5.0.0
I removed the output splitting because of hard to track reference cycles.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@ofborg eval

Converted to draft because split outputs are actually desirable. I don't know how to find the problematic references though.

is there something missing to mark it as ready for review?

is there something missing to mark it as ready for review?

Mostly time to get it working properly. If anybody who knows a little about Zeek wants to help it would be greatly appreciated.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1657

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/824


Relative Time

Docs:
https://shoelace.style/components/relative-time
Source:
https://github.com/shoelace-style/shoelace/tree/next/src/components/relative-time

There isn't a Figma component to create for this.


[Bug]: Unix time Block Function does not accepte  !

Version of the software
0.26
Operating system
Windows 10
Browser / Native
Native
What happened?
A bug happened!
Last time i create a config but when I parse a unix time variable to translate it to date I found that the unix date function block does not accepte the </> feature so fix that please as soon as possible its important.
king of regards
Aymen
Relevant LoliCode if needed
No response

It accepts an integer, you can just use the VAR mode and put the name of your parsed timestamp (just the name of the variable without <these>) and it will automatically cast from string to int.


test: adding wait vs non wait actions test

implementation of new test to validate the use of Wait Actions VS Non Wait Actions during the train dialog creation.

Wait action: After the system takes a "wait" action, it will stop taking actions and wait for user input.
Non-wait action: After the system takes a "non-wait" action, it will immediately choose another action (without waiting for user input)
Also, added new  data-testids for entity creation checkboxes.
finally, I also modified function names to match action + element name.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


new layout issues

missing background color for inactive tab:

BNB also has (-0.15% fee)

if I have BUSD balance and allowance, BNB and WBNB should shows "BNB + BUSD" and "WBNB + BUSD", in this case, there no -0.15% fee tag

Value button overflow:

mobile UI broken when error message is too long

(low priority) parse the error message and display only the reason
call revert exception; VM Exception while processing transaction: reverted with panic code 17 [ See: https://links.ethers.org/v5-errors-CALL_EXCEPTION ] (method="multiSwap((address,address,uint256,uint256,bytes32),(uint256,uint256,uint256,uint256)[])", data="0x4e487b710000000000000000000000000000000000000000000000000000000000000011", errorArgs=[{"type":"BigNumber","hex":"0x11"}], errorName="Panic", errorSignature="Panic(uint256)", reason=null, code=CALL_EXCEPTION, version=abi/5.7.0) should show only Panic(0x11)

:tab

CP decimals is 18

what's with the 2 -> change?

missing history tab in mobile layout

wrong history display
https://bscscan.com/tx/0xcef522e494790d5fc41cf21bd1840fd6d8a6e0d8733cde0ba1be497941cd0262#eventlog

should be +0.5 Cake -> 0.97... CP

history must sort newest to oldest

Remove: click % => expected

Remove: input number => no change

incorrect amount removed

link preview


Fix: target line is not changed

Lines aren't swapped if the origin line is longer than the target.  The
origin line is replaced by the target line, but the target line remains
unchanged.  The cause is passing 1-indexed number of line
nvim_buf_get_offset,  but the function accepts a 0-indexed line
number.
Consider the following example
very long and descriptive line
short line

Pick the long line and swap it with the short one. You'll get
short line
short line

Thanks for the PR ! You're so right, I merge and I'll fix the tests ;)


Azure SDK Review - [Azure SDK]

New SDK Review meeting has been requested.
Service Name: Azure SDK
Review Created By: Ronnie Geraghty
Review Date: 05/31/2023 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: .NET,
Description: Test for Interest Group Invites. This meeting will be cancelled.
Detailed meeting information and documents provided can be accessed here

Cancelled by: Ronnie Geraghty


SC-44 # Set scope without needing --project flag

Changed

BREAKING CHANGE: <project_path> input option to a --cwd flag for all CLI commands

BREAKING CHANGE: bm server scope --project <project> now sets project using bm server scope <project>

LGTM 👍


Derive Insertable for struct / Arc / Rc rather then for plain reference

Currently there's only implementation for &'a struct when Insertable is auto-derived https://github.com/diesel-rs/diesel/blob/master/diesel/src/macros/insertable.rs#L103
This is kind of painful when working with multithreading, because multithreading doesn't understand lifetimes. E.g. I have this method:
    pub fn create(&self, payload: NewUser) -> Box<Future<Item=User, Error=Error> {
        let conn = self.get_connection();

        Box::new(
            self.cpu_pool.spawn_fn(move || {
                let query = diesel::insert_into(users).values(&payload);
                query.get_result(&*conn).map_err(|e| Error::from(e))
            })
        )
    }

I want to generalize all the async stuff into one method to reduce boilerplate. This function works fine for 'static queries:
    fn execute_query<T: Send + 'static, U: LoadQuery<PgConnection, T> + Send + 'static>(&self, query: U) -> RepoFuture<T> {
        let conn = match self.r2d2_pool.get() {
            Ok(connection) => connection,
            Err(_) => return Box::new(future::err(Error::Connection("Cannot connect to users db".to_string())))
        };

        Box::new(
            self.cpu_pool.spawn_fn(move || {
                query.get_result::<T>(&*conn).map_err(|e| Error::from(e))
            })
        )
    }

But this thing let query = diesel::insert_into(users).values(&payload); isn't static, but has 'a lifetime that is equal to payload. To avoid this, I tried to implement smth like this:
    fn execute_query_fn<U, Q, F, P>(&self, payload: P, query_fn: F) -> RepoFuture<U>
        where
            U: Send + 'static,
            Q: LoadQuery<PgConnection, U> ,
            F: for<'a> Fn(&'a P) -> Q<'a> + Send + 'static,
            P: Send + 'static
    {
        let conn = match self.r2d2_pool.get() {
            Ok(connection) => connection,
            Err(_) => return Box::new(future::err(Error::Connection("Cannot connect to users db".to_string())))
        };

        Box::new(
            self.cpu_pool.spawn_fn(move || {
                let pl = payload;
                let query = query_fn(&pl);
                query.get_result::<U>(&*conn).map_err(|e| Error::from(e))
            })
        )
    }

But this fails due to rust compile limitations F: for<'a> Fn(&'a P) -> Q<'a> + Send + 'static, i.e. it doesn't allow to specify lifetime for generic parameter. The corresponding Rust RFC is in progress https://github.com/rust-lang/rfcs/blob/master/text/1598-generic_associated_types.md.
All this pain comes from the fact that I have to use reference to payload here let query = diesel::insert_into(users).values(&payload);. Because only reference auto-derived for Insertable. If it was smth like struct or Arc everything would work ok.
Does it make sense to implement these?

@sgrif  Yeah, that would solve the issue


Things still open after Profile deleted

I went back into the Guest account that I had made, and I decided to delete the account. I tried recreating the Guest account without the password box filled in it said "Password Required" I logged back in to the Admin account the Management box was still opened with the words "Password Required" still there with guest in the account name box and no password in the password box.

@Roystermeat This issue has been resolved for version 0.1.406 of Ruthenium. Thank you for your cooperation.


Question about the zoom-in

Hi,
Thanks for your excellent work! Here I have a question. I noticed here is a 'zoom-in' operation in the figure of your paper. However, there is no implementation in your repo. So, could you introduce the method you used for changing ROI?
Regards,
Rui

There is no "zoom-in" operation used in this work. The input image is the original image, we just show the "zoomed-in part" for better visualization.


Incorrect documentation...this line is referring to modifying the interface for the IListInfoWebPartProps. This is located inside a file called "IListInfoWebPartProps.ts".

So that is not in a ts file of it's own at all. It's just declared as an interface in the file he mentioned.

Hi @kevinguyer The documentation is correct. This line is referring to modifying the interface for the IListInfoWebPartProps. This is located inside a file called "IListInfoWebPartProps.ts".
I have circled it in red in the below image.

Originally posted by @bcameron1231 in https://github.com/SharePoint/sp-dev-docs/issue_comments/476906438

If you can provide context where you saw this, it would help to get the doc fixed.
The web part property pane interfaces used to be in their own files, but have since been merged to be included in the web part themselves. Looks like the doc just needs an update.


Resolve the GrainType from the provided generic grain value

closes #140

Hey @seniorquico,
Sorry it was late on Friday and didn't give a good description. This corrects the GrainType on the created grains to be the same as Orleans so anything that is looking at the Type value, including any aliases, are correct. I can add a test for that specific case.
We had some code that looks the GrainType and was failing since it was an invalid value.
I used a DI container since the GrainTypeResolver has an interesting dependency tree and this approach handles Orleans changing how this value is generated since it's extensible. I could make it more explicit but there might have to be manual changes if Orleans changes functionality.
I'd rather not be creating the container and resolving the value each Silo but TestKit seems to isolate itself on that level so it seemed sensible. It could be made private and static so it's only resolved since it shouldn't change but didn't want to assume.

@seniorquico Sorry for the delay, added tests showing what this change provides.

@seniorquico Another gentle nudge on this since all our tests are broken without this change and theres no way for me to hack around it so it's blocking us. Happy to make more changes if you'd like

@seniorquico Another bump on this. Happy to change approach with some feedback.
Looked at the latest changes with grain context. It gets me some way to being able to do this externally to testkit but even if I override the GrainContext with the "correct" grain id with the grain type correctly, the context gets replaced since the comparison is done on the whole grain id.
If you'd rather not the service provider route so the grain types are correct for all grains I could instead expose an overload of GetOrAddGrainContext that takes a whole GrainId and allows you to set that?

HI, sorry to keep prodding, I work with @Romanx - any chance of progressing this, we could really do with it :)  Thanks!


Exception check added in client when sending insertion data.

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
For changelog. Remove if this is non-significant change.
Category (leave one):

Improvement

Short description (up to few sentences):
Server exception got while sending insertion data by is now being processed in client as well.

Still fails. (

Not fully checked yet


Implement bundle validation

The validation steps does following checks:

operator-sdk bundle validation
yaml lint
check of version range annotation
check of cert project id

JIRA: ISV-848

@amisstea @MarcinGinszt Please review


Address permission support + enabled/disabled customer support

Add support of address's access attribute to allow or refuse create/update/delete address on account and cart pages.
Add support of enable customer attribute.

tested and works fine, thanks. We cannot merge until we merge changes odoo side
https://github.com/shopinvader/odoo-shopinvader/pull/471

It's merged in v13


How to bundle when addLayers: false

The docs say:
When false, you must include the Datadog Lambda library in your functions’ deployment packages.

But there are no instructions on how to actually do this. I tried adding import "datadog-lambda-js"; to the top of my handlers, so the datadog libs would get bundled (and they did), but that didn't work.

Then after browsing through your repos, I came upon this:
export const jsHandler = "node_modules/datadog-lambda-js/dist/handler.handler";

So I assumed, I'd have to add the entire folder on that specific location, but that didn't work either as it then started throwing errors as it was missing other libraries (Error: Cannot find module 'hot-shots').
So, how are we supposed to "include the Datadog Lambda library in your functions' deployment packages"?
why?
Having to exclude these libraries from the bundler, makes our functions break when trying to run them locally with sam local start-api, so we'd have to always comment the exclude rule just to run them locally.
Also I thought that by using our bundler (esbuild), we should be able to get better minification and reduce the overall package size.

Hi @vieirai, thanks for reaching out!
You're likely missing the dd-trace dependency, which uses hot-shots.
If you'd like to use the npm packages instead of Lambda Layers, you'll need to follow the custom installation option, following the Option B guide in the documentation.
npm install datadog-lambda-js dd-trace
Please let me know if this helps.
Thanks!

Hi @astuyve , thank you for looking into this.
So, after playing around with the Custom installation option, I ended up applying the datadog wrapper around my handler and adding the layer explicitly as per the docs.
This way I let my bundler package the and tree shake the libraries and then the extension deals with the logs.
I still think the documentation around the addLayers toggle on the SAM macro is misleading though, as the only way to bring the libraries in myself, was not to use the macro.

Thanks for the note!
If you wrap the handler yourself using the datadog method, then there is less of a reason to use the SAM macro. But the macro acts to redirect the handler (which doesn't require the datadog method be explicitly called), which makes it still useful when paired with addLayers: false.

Thanks,
AJ


{Feature Request} Ability to subdivide bookmarks into groups

My list of bookmarks has grown so much that I am having to scroll through them to find the right ones.  I have taken to changing my naming conventions to group associated bookmarks.
Couple options for request...

HR rule to allow me to separate groups within the long list
{Better] Is there a way to create folders within singular imported XML?  That way I could have all of my systems for a specific purpose in a sub folder that is synchronized singularly.

You can move each folder wherever you want, just like you would move normal bookmarks. You can't move individual items in the folder though if that's what you were trying to do.


Add CI/test step to Jenkins

Needs:

Way to run tests w/o browser
Way to aggregate test results in, say, JUnit format for display/plugins


minor renaming and refactoring in envoy filters

No functional change
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[X] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@istio/wg-user-experience-maintainers can you PTAL?

@ramaraochavali: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
37b69e8da6591cc08b3a1a6ae46e63b69367959b
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-multicluster-tests_istio


Confusing hover outline for disabled input elements

I'm not sure if it's a bug, but I'm just confused about outline style for hovered disabled inputs - Input, DatePicker components. It just seem that i can change the value of input. Should it change outline color?

For example there is no any ui changes for disabled hovered inputs in Material UI library

Thank you for letting us know. The issue will be fixed in the coming release.


Connection timeout for every request after hour(s) of idle time

Environment
Knex version: ^0.20.10,
Database + version: MSSQL ^6.1.0
OS: Windows 10 Server 2016
@smorey2
So I get this error everytime I leave the app running idle for an hour or more, and run some queries to db.

This is a multi-tenant app. So I had to provide a knex instance every time the user makes a query. Is this related?

This is also the same for the basic queries including select * from rooms. And rooms only contain 3 - 10 items with 5-6 columns.
This is my knex config:

Maybe this one is related?

I think this is more relevant.

Even adjusting the timeout doesn't take effect

It's the same even here. Is the transaction locking the table?

I'll close this. I got and error 'transaction tried to rollback but transaction is still ongoing'(non-verbatim) earlier. So that must've locked the db.

@derencel20 : Howdy!  So, it looks like that timeout is coming from the mssql library itself (as opposed to the knex pooling).  I think you can adjust that timeout by specifying it as part of the connection: section of your knexfile.js.  For ex:
{
  connection: {
    user: "blah",
    password: "blahblah",
    timeout: 30000,
  }
}

Based upon skimming the code, it looks like it will forward this value to the underlying mssql library.


I can not run correctly robonomics for move_arm_client.py for KUKA manipulator

Waiting for payment
====================
Version: 0.21.0-31a135b-x86_64-linux-gnu
0: sp_panic_handler::set::{{closure}}
1: std::panicking::rust_panic_with_hook
at /rustc/d7f94516345a36ddfcd68cbdf1df835d356795c3/src/libstd/panicking.rs:530
2: rust_begin_unwind
at /rustc/d7f94516345a36ddfcd68cbdf1df835d356795c3/src/libstd/panicking.rs:437
3: core::panicking::panic_fmt
at /rustc/d7f94516345a36ddfcd68cbdf1df835d356795c3/src/libcore/panicking.rs:85
4: core::option::expect_none_failed
at /rustc/d7f94516345a36ddfcd68cbdf1df835d356795c3/src/libcore/option.rs:1273
5: <core::future::from_generator::GenFuture as core::future::future::Future>::poll
6: std::thread::local::LocalKey::with
7: async_task::raw::RawTask<F,R,S,T>::run
8: multitask::Ticker::tick
9: async_executor::Executor::run
10: core::ops::function::FnOnce::call_once{{vtable.shim}}
11: <alloc::boxed::Box as core::ops::function::FnOnce>::call_once
at /rustc/d7f94516345a36ddfcd68cbdf1df835d356795c3/src/liballoc/boxed.rs:1081
<alloc::boxed::Box as core::ops::function::FnOnce>::call_once
at /rustc/d7f94516345a36ddfcd68cbdf1df835d356795c3/src/liballoc/boxed.rs:1081
std::sys::unix::thread::Thread::new::thread_start
at /rustc/d7f94516345a36ddfcd68cbdf1df835d356795c3/src/libstd/sys/unix/thread.rs:87
12: start_thread
13: __clone
Thread 'async-std/runtime' panicked at 'called Result::unwrap() on an Err value: Codec(Error("No such variant in enum Phase"))', robonomics/io/src/source/virt.rs:94
This is a bug. Please report it at:
https://github.com/airalab/robonomics/issues/new

Hello! Thank you for report!
Could you describe step by step how to repeat this error, please.

I have solved this error. I did not executed the command for removing robonomics database before running it.
Thanks for your attention


Simple estimator for scripts being tested

This provides tasty-plutus an additional option, allowing a rough estimate of how much time and space a script execution under test will take. This is not a full-blown benchmarking solution, as it's fairly inexact, but it can be used to give a back-of-envelope estimate.
Closes #194 . Seem OK by you @maciej-bendkowski, @samuelWilliams99?

One more comment, I think that --estimate-only is a bit misleading. Specifically, it performing cost estimates but, at the same time, also running off-chain tests. Could we skip these tests when invoking --estimate-only?

tasty-plutus does not concern itself with off-chain anything. I can't skip tests I don't control in tasty - that's the provider's own responsibility.
This is further evidence that we need something like plutus-bench, which does benchmarking of this stuff properly.


run L1 resolution tests on mainnet

Complete https://docs.google.com/spreadsheets/d/19LP_bgipXZL8BxyinnUAODlcWk_soTNmSHgaHpVkges/edit?usp=share_link

Hey team! Please add your planning poker estimate with Zenhub @bilbeyt @GabrielBuragev @istankovic


Add count of translations

A summation/count for the three translation tabs would be nice. (like in "Kata" for "Completed","Unfinished", "Authored".

Just came across this today and was about to enter my own ticket when I found this one.  I'd say just put the numbers right where I have the red box next to each type.

The verbosity of the lists makes it a little hard to get a quick count on how many you have pending approval.


a question

when my controller has a collectionview or tableview, how to resolve the complexity in reshresh/endRefresh/loadMore/endLoadMore

this four state is so bad that repeated in every controller

What did you mean 'complexcity'?

@devxoul  i mean if my viewcontroller have a collectionview, and you know collectionview may have refresh/loadmore action to pull new data from server,so how to modify this state convenient with reactor

collectionView
.rx
.refresh
.map { Reactor.Action.refresh }
.bind(to: viewModel.action)
.disposed(by: disposeBag)
    collectionView
        .rx
        .loadMore
        .map { Reactor.Action.loadMore }
        .bind(to: viewModel.action)
        .disposed(by: disposeBag)

    reactor
        .state
        .map { $0.isRefreshing }
        .distinctUntilChanged()
        .bind(to: collectionView.rx.isRefreshing)
        .disposed(by: disposeBag)

    reactor
        .state
        .map { $0.isLoadingMore }
        .distinctUntilChanged()
        .bind(to: collectionView.rx.isLoadingMore)
        .disposed(by: disposeBag)

    reactor
        .state
        .map { $0.dataLoadedAll }
        .distinctUntilChanged()
        .bind(to: collectionView.rx.isNoMoreData)
        .disposed(by: disposeBag)

    reactor
        .state
        .map { $0.isLoading }
        .distinctUntilChanged()
        .bind(to: rx.isLoading)
        .disposed(by: disposeBag)

now i have to write this code in every controller, it's so ugly

Why not just create a utility method to do those things in once?

Is it possible to solve this with inheritance somehow?


Update index.md

added a missing noun for the sake of understanding

Thanks @silvermanj7!
Thanks,
Dan


nvm not available in non-interactive shells

See #1717 for installation details.  Sorry if you already know these shell details - I had to review them to write this up.
Interactive shells are typed into.  E.g. typing the command nvm at the flashing cursor.
This bug is about nvm not being available in non-interactive shells.  E.g. this test.sh:
$ echo -e '#!/bin/bash \n nvm version' > test.sh
$ chmod +x test.sh
$ cat test.sh
#!/bin/bash
 nvm version
$ ./test.sh
./test.sh: line 2: nvm: command not found

This is because nvm isn't on the path, which probably causes alot of other bugs.
Linking nvm.sh into the path doesn't work because nvm.sh doesn't appear to do anything when run directly:
./.nvm/nvm.sh version
<no output>

So, I'm a bit stuck on how to use nvm non-interactively and have to put this crufty code everywhere:
#/bin/bash
[ -s "$HOME/.nvm/nvm.sh" ] && source "$HOME/.nvm/nvm.sh" && nvm install

Which is kind of annoying.  It would be nice if nvm was actually installed onto the path.

nvm.sh is not runnable, not a binary, never will show up in a PATH, and doesn't take a version argument. It's a sourced shell function. Try . ~/.nvm/nvm.sh.

@ljharb Hey Jordan, no problem.  I get that.  But it doesn't show up in non-interactive shells, so it can't be scripted.  I noticed a nvm-exec file but didn't understand it's purpose.
What about a nvm file that sources nvm.sh and runs it with arguments?
The installer could link this into ~/bin or if no directory explain what to do.

I'd also like it to show up in non-interactive shells.

@MichaelJCole that's exactly what nvm-exec is for.

Aha!  Great!  Ok, that's the same direction.  For whatever reason, nvm-exec doesn't work for me in Zsh or Bash
nvm works great, but nvm-exec returns no output.  The umbrella w/ rain means the command returned a failure code to the shell :-}
Am I using it wrong?  Or did I miss some documentation?
michael@mcdesktop ~
☔  ls $NVM_DIR
alias            Dockerfile  Makefile  package.json  test
bash_completion  install.sh  nvm-exec  README.md     update_test_mocks.sh
CONTRIBUTING.md  LICENSE.md  nvm.sh    ROADMAP.md    versions

michael@mcdesktop ~
☀  bash $NVM_DIR/nvm-exec

michael@mcdesktop ~
☀  nvm

Node Version Manager
... (output truncated)
Note:
  to remove, delete, or uninstall nvm - just remove the `$NVM_DIR` folder (usually `~/.nvm`)

michael@mcdesktop ~
☀  bash $NVM_DIR/nvm-exec

michael@mcdesktop ~
☀  ./$NVM_DIR/nvm-exec
zsh: no such file or directory: .//home/michael/.nvm/nvm-exec

michael@mcdesktop ~
☔  bash
michael@mcdesktop:~$ nvm

Node Version Manager
... (output truncated)
Note:
  to remove, delete, or uninstall nvm - just remove the `$NVM_DIR` folder (usually `~/.nvm`)

michael@mcdesktop:~$ echo $NVM_DIR
/home/michael/.nvm
michael@mcdesktop:~$ .$NVM_DIR/nvm-exec
bash: ./home/michael/.nvm/nvm-exec: No such file or directory
michael@mcdesktop:~$ bash $NVM_DIR/nvm-exec
michael@mcdesktop:~$

That's because you're running bash $NVM_DIR/nvm-exec - nvm-exec is an executable. Try nvm-exec $version $command.

@ljharb I think nvm-exec is broken or not installing correctly.  It doesn't seem to do anything.
michael@mcdesktop ~
☔  ./.nvm/nvm-exec ls
bin  Desktop  Documents  Downloads  Music  NULL  Pictures  scm  snap  Templates  Videos

@MichaelJCole that’s how it’s supposed to work. It takes an arbitrary command and executes it with a particular version of node nvm used.

@ljharb How do I use nvm in a non-interactive shell without having to source it each time?

I mean, really, you don't - one way or another, nvm.sh must be sourced to use nvm.
However, if you execute nvm-exec with the proper arguments, it will source it each time for you.

@ljharb ok, this isn't fitting my model, but that's ok.
For most projects, I've got a preview.sh or deploy.sh to start them up.  Something like this for a hexo project:
#!/bin/bash

# Load nvm (node version manager), install node (version in .nvmrc), and npm install packages
[ -s "$HOME/.nvm/nvm.sh" ] && source "$HOME/.nvm/nvm.sh" && nvm use
# Use local-npm if running
wget -qO /dev/null 'http://<IP_ADDRESS>:5080/_browse/' && npm set registry http://<IP_ADDRESS>:5080 && echo '----> local-npm found <----'
# Npm install if not already.
[ ! -d "node_modules" ] && npm install

hexo server

Looking at this again, I'm whining about the first parts of the first line.  So maybe not a big deal.  Mostly, some other developer is going to look at it and throw a WTF exception if they don't know what NVM is.
Do you have any suggestions or do you see a better way?  Thanks!

I don't think there's a better way, no (altho the && nvm use is redundant; nvm auto-runs that for you).

Ok, cool.  Thanks!

I would like to use "nvm use" in my cron scripts and am unable to run it. Neither nvm-exec nor nvm.sh are working...
~/.nvm/nvm.sh use v11
^ does nothing and does not switch to version 11...
~/.nvm/nvm-exec use v11
^ outputs the error: "/home/ubuntu/.nvm/nvm-exec: line 15: exec: use: not found"
@ljharb what's the correct way to do this?

@JimLynchCodes your cron scripts need to source nvm.sh to be able to use nvm.

To run nvm use via ssh remote command as part of a deployment script doing as ljharb commented makes it work.
source ~/.nvm/nvm.sh
nvm use v12.4.0

  sshCommands="$(cat <<END_HEREDOC
# Install api server dependencies.
source ~/.nvm/nvm.sh

echo "Switching to appropriate node.js version using nvm"
if ! nvm use v12.4.0 ; then
  echo "Error using nvm to switch node.js version."
  exit 14
fi

echo "Installing server api dependencies"
if ! yarn ; then
  echo "Error installing server api dependencies."
  #exit 15
fi
END_HEREDOC
)"

ssh ${deploymentUsername}@${deploymentHost} -t "$sshCommands"

In windows I had to do this
C:\Node\nvm\nvm use 14.15.4

Where C:\Node\nvm is the directory where I stored nvm
Then I restarted the Jenkins server

In windows I had to do this
C:\Node\nvm\nvm use 14.15.4

Where C:\Node\nvm is the directory where I stored nvm
Then I restarted the Jenkins server

@promisepreston nvm does not support windows, unless you're using WSL. Perhaps you're using nvm-windows, a different project?

@promisepreston nvm does not support windows, unless you're using WSL. Perhaps you're using nvm-windows, a different project?


Fixed computation of selectOffsetBot variable

This is a fix for the issue [https://github.com/silviomoreto/bootstrap-select/issues/743]

Fixed in v1.7.0-rc1.


Allow output filenames to have yml suffix

In our internal project, we have almost 5000 individual YAML files that
we generate. For historic reasons, we continue to name the files .yml
instead of .yaml.
By enabling kubecfg to write the files with a .yml suffix, we can
produce these files for our CI/CD pipeline measurably faster.
I used hyperfine to benchmark the current mode (my build of kubecfg, do
not specify --format, followed by find | read | mv), vs. the same build
of kubecfg with --format=yml, no call to find.
Benchmark #1: GENMODE=current make generate
Time (mean ± σ):     13.384 s ±  0.459 s    [User: 79.086 s, System: 19.813 s]
Range (min … max):   12.775 s … 14.280 s    10 runs
Benchmark #2: GENMODE=newkubecfg make generate
Time (mean ± σ):     11.166 s ±  0.216 s    [User: 73.907 s, System: 9.905 s]
Range (min … max):   10.858 s … 11.544 s    10 runs
Summary
'GENMODE=newkubecfg make generate' ran
1.20 ± 0.05 times faster than 'GENMODE=current make generate'

Thanks!

Actually I think I got triggery-happy.
The -o is for the output format. While the problem we're having here is the filename extension of the files created when we use --export-dir. The problem is caused because --export-filename-format doesn't include the extension.
I think what we should do is to add a new --export-filename-extension flag, defaulting to empty string. When that flag is not empty it will override whatever filename extension would be chosen by the -o flag.

@mark-rushakoff do you want to do that second PR or should I do it?

I think you could knock that out faster than I could @mkmik -- sorry this change didn't cover it.


There are no directories in pricing version

views directory exists in demo version but not in standard+ version (next.js + typescript)
Can i get the views directory??

We have provided the next.js version as a Skeleton. So you integrate or start the new project quickly.
As this issue is for the pro version.  We are closing here. You can contact our support team for further discussion.


Any further development?

Thanks for this hidden gem, great work!
I have a coupe of feature requests, but I'm not sure if you are still interested in working on this.

Not really, I consider this done for my purposes. Feel free to fork though :)

OK, thanks.


Atom tree-view git highlighting isn't always being refreshed in realtime on OSX

Green (added) and orange (modified) colors sometimes remain until I exit from full screen mode, switch desktops or click on the items  in the tree-view. This is pretty annoying and misleading.
Atom Version: 1.5.3
OS X version: 10.11.3

/cc @joshaber

here is a video demo of the bug (link expires in 7 days) http://cloud.vatsaev.com/data/public/atom-treeview-bug

I got that 'bug' when upgrading to 1.9 (1.9.1 and 1.9.2) with a slightly different behaviour,
I have absolutely no orange or green in the tree-view, but if I chose one 'modified or added' file it gains its supposed color, and loses it after sometime (very short time)


Please invite me to the Recode-Hive GitHub Community Organization

Name
Shivam Prasad Singh
Discord Username (if applicable)
shiv007_70175
Additional Context
Please invite me to the Recode-Hive GitHub Community Organization. I'm excited to connect with like-minded developers, contribute to innovative projects, and grow in a supportive community. With my passion for open-source and commitment to continuous learning, I’m eager to make meaningful contributions. I look forward to sharing knowledge, gaining insights, and helping build impactful projects together.

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


nit: error generating maven dependencies report

Running maven site on a project that uses HikariCP I get 3 warnings like the following.  Is the build non-standard in some way?
[WARNING] Unable to process class com/zaxxer/hikari/pool/HikariPool.class in JarAnalyzer File /Users/ezra/.m2/repository/com/zaxxer/HikariCP/2.3.2/HikariCP-2.3.2.jar
org.apache.bcel.classfile.ClassFormatException: Invalid byte tag in constant pool: 18

HikariCP-2.3.2.jar is a Java 8 jar, built in a standard fashion.  The issue seems to be with JarAnalyzer not HikariCP.  You might try using the HikariCP-java6-2.3.2.jar artifact instead if you need to run JarAnalyzer on it.


my cmd crashes whenever i use the command node.js

i cant open the file with node directly
like when i choose to open the file with node.js it just turns the app off

type this


Add link to email

Welcome email does not have a link for me to click on.

(That link should log me in automatically - without having to type any passwords)


feat(docz): add Description component

Description
Adds the <Description> component to print component descriptions, as per #736
Example
MyComponent.tsx
import React from "react";

/**
 * The greatest component in the world
 */
export const MyComponent = () => <div />;

export default MyComponent;

MyComponent.mdx
import { Description } from "docz";
import MyComponent from "./MyComponent";

# My Component

<Description of={MyComponent} />

Screenshots

Before
After

@pedronauck what do you think about adding this?

Hi Jess, sorry for the delayed answer. So, about this component, I think that is so minimal its behavior and I don't know it will be used at all, the component description can simply write on your mdx however you want. You can use it on your project, but to put inside the core I think that's not a good idea.

Hey Pedro, thanks for responding!
With all due respect, to say that the component description can be written on your mdx misses the point that JSDoc documentation is just as useful for in-editor documentation as it is for generating static documentation.  Frankly, the in-editor documentation like shown below is the more important use case.

In using docz without this component I have to provide the component description in the JSDoc and then (as you said) also copy and paste the same description into the MDX, which I've found to be very tedious, inefficient, and more difficult to maintain.  In other words, I think it's a bad practice.
I think just having this component readily available will encourage people to "do the right thing" and put descriptions on their components since it is then very easy to display those descriptions. I think it makes a very natural sibling to the Props component.  And, since it is so simple it should not be a headache to maintain in core.

I have to agree with @pedronauck here. If you (@jchadwick) feel that this is the right way I'd urge you to maintain it as a separate repo. If it gains traction I think it'd be fair to think about merging it into this repo, but as a general consumer of docz I can't see the immediate added value.

Withdrawing PR

@jchadwick If you maintain this as a separate package then it may be worth adding a section to the documentation to let users know about it. Whilst it has been decided this won't be added to the main repo right now I'm sure some users will find it useful :+1:


Add bash tests for all example scripts

Within here, I suspect that the light client doesn't like txs that don't include the --fee flag
https://github.com/tendermint/basecoin/tree/master/cmd/basecli

Tested in new tests/cli tests on the pr and it works. Fix will be in master with the 0.6.0 release.

Closed with #145