Cannot start "Launch currently open script" with PHP executable path specified in settings

It is not possible to start debugging if PHP is specified in the settings:
    "php.executablePath": "D:\\tools\\php\\php-7.1.11-nts-Win32-VC14-x64\\php.exe",
    "php.validate.executablePath": "D:\\tools\\php\\php-7.1.11-nts-Win32-VC14-x64\\php.exe",

"Listen for XDebug" debugging is working (from the command line and from the browser). Debugging is also working if the PHP bin folder is added in the PATH variable.
Edit:
When starting an external console, following message is written:
'"php"' is not recognized as an internal or external command,
operable program or batch file.
Press any key to continue . . .

PHP version:  7.1.11
XDebug version:  v2.5.4
Adapter version:  1.12.3
Your launch.json:
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Listen for XDebug",
            "type": "php",
            "request": "launch",
            "port": 9000
        },
        {
            "name": "Launch currently open script",
            "type": "php",
            "request": "launch",
            "program": "${file}",
            "cwd": "${fileDirname}",
            "port": 9000
        }
    ]
}

XDebug php.ini config:
extension_dir = "D:\tools\php\php-7.1.11-nts-Win32-VC14-x64\ext"
zend_extension = "php_xdebug-2.5.4-7.1-vc14-nts-x86_64.dll"

[XDebug]
xdebug.remote_enable=1
xdebug.remote_autostart=1
xdebug.remote_port=9000

Developer Tools error Messages:
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:1311  INFO no standard startup: panel is active
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268 [Extension Host] DEBUG     Checking PHPLS_ALLOW_XDEBUG

t.log @ /C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268 [Extension Host] DEBUG     The xdebug extension is loaded (2.5.4)

t.log @ /C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268 [Extension Host] DEBUG     Process restarting (PHPLS_ALLOW_XDEBUG=internal|2.5.4|0)
DEBUG     Running D:\tools\php\php-7.1.11-nts-Win32-VC14-x64\php.exe -c C:\temp\BA8C.tmp c:\Users\JFlepp\.vscode\extensions\felixfbecker.php-intellisense-2.3.3\vendor\felixfbecker\language-server\bin\php-language-server.php --tcp=<IP_ADDRESS>:11914 --memory-limit=4095M

t.log @ /C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268 [Extension Host] DEBUG     Checking PHPLS_ALLOW_XDEBUG

t.log @ /C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268 [Extension Host] DEBUG     Restarted (52 ms). The xdebug extension is not loaded

t.log @ /C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:268 [Extension Host] PHP process connected

That setting is not mentioned in the readme - it is for a different extension


update extended_image version

plz update extended_image version to the latest in native_pdf_view, thanks

Published in 3.8.0


charmxi: Report errors and warnings in a standard format including file name and line number

Original issue: https://charm.cs.illinois.edu/redmine/issues/75

charmxi prints various error and warning messages when it finds problems with its input, but they are highly variable. At the very least, they should all report the file and line on which an issue arises, in a consistent format. Even better, the messages should follow some consistent grammar, too. If we want to be really fancy, printing the affected line with indicative markup like clang does would be awesome.

Original date: 2014-01-06 04:36:45

Inactive for 3 months; should be a small task; increasing the priority.


[20.03] wire-desktop: linux 3.18.2925 -> 3.19.2928, mac 3.18.3728 -> 3.19.3799

Motivation for this change
Backport wire-desktop: linux 3.18.2925 -> 3.19.2928, mac 3.18.3728 -> 3.19.3799 #94758
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build wire-desktop


update function scope

Hi, I find a scope problem in function call. see this code bellow:
// resursive calls
def fib(n)
  if n < 2 do
    return n;
  else do
    return fib(n-2) + fib(n-1);
  end
end

the result of fib(4) is 3. but if we modify the code a little as bellow
def fib(n)
  if n < 2 do
    return n;
  else do
    a = fib(n-2);
    c = fib(n-1);
    return a + c;
  end
end

then the result of fib(4) is 1, not 3.
I find this is because during function call, we give the function call a scope with current scope as its parent. so I fix it with a scope whose parent is null and it works.

this will cause we cannot access global variable in the function. Because we don't have a declaration  statement for variable, it's hard to determine  the proper scope of a variable.

Nice find @frozenxia! Thanks for the PR 👍


Node.js support

It'd be cool if there was a way to have downloaded packages form the node_modules file tree that's required for Node. Has there been any effort going in this direction with something using the hooks yet?
The reason I bring this up is that people seem pretty pissed at NPM, and this could be a great way to get a large number of users for the package.

Not to just plus 1 this and roll, but I would move to an npm alternative tomorrow as well

The best way forward with that would be to implement a gx-js (or similar) program with some of the hooks set up. The main one you would want is the install-path hook.
I do this in gx-go here: https://github.com/whyrusleeping/gx-go/blob/master/main.go#L498
Unless you need fancy other things, you might be able to just get away with it being a shell script (or, javascript)

A GitHub repo is now open for gx-node. Please come to discuss and contribute. :)

@whyrusleeping -- thanks.  I forked whyrusleepling/gx-go here sterpe/gx-js and modified the install-path hook to install to either $CWD/node_modules or "`which node`/../../lib/node_modules" with the --global flag.
This provides a barebones functionality with gx for anyone who wants to mess around further -- modules are installed to node_modules/gx/ipfs/$hash/$module_name.
Obviously still a bunch of work to do on this fork -- I didn't mess around with post-install hook (throws errors trying to do go stuff at the end) or anything but was able to publish some modules to IPFS and have the dependencies pulled down, etc.  I like gx.
https://ipfs.io/ipfs/QmSb1Uv2b5PgffNrHQhojLQMXNfqAuq67hAq34qSVQivEQ/
https://ipfs.io/ipfs/QmZ2vZ4Vn947wNRVt9jsPvTT7QKa5rnWMWsBnouvYPFnjo


Fix/increase walk interval

Description
Increase walk max interval to week instead of 12 hours.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Refactor/improvement
[ ] This change requires a documentation update

How Has This Been Tested?
unit test, integration test, manual test
Checklist

[x] My commit message is conventional
[x] I have run pre-commit on all files before creating the PR
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

Codecov Report

Merging #827 (d967513) into develop (974b1c8) will not change coverage.
Report is 4 commits behind head on develop.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff            @@
##           develop     #827   +/-   ##
========================================
  Coverage    87.54%   87.54%
========================================
  Files           27       27
  Lines         1830     1830
========================================
  Hits          1602     1602
  Misses         228      228

Files Changed
Coverage Δ

splunk_connect_for_snmp/common/inventory_record.py
99.07% <100.00%> (ø)

:tada: This PR is included in version 1.9.3-beta.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update library definitions in README to the latest version

I recently switched to this library for a project, and since I just copied the info from the readme I used an ancient version at first. That was especially annoying since many APIs that would've been helpful for me during migration didn't exist back in version 1.3.0 (like creating ServiceAccountCredentials from a PKCS8 string without having to pass JSON).
I'd love to prevent other users from making the same mistake as myself in the future through this README update.

Thanks for updating this )


tensorflow量化感知训练，加载成功build失败

I Start T2C Switcher...
D Optimizing network with broadcast_op, t2c_fc
E Catch exception when building RKNN model!
E Traceback (most recent call last):
E   File "rknn/base/RKNNlib/switcher/mode_switcher.py", line 353, in rknn.base.RKNNlib.switcher.mode_switcher.ModeSwitcher.build_mark
E   File "rknn/base/RKNNlib/switcher/mode_switcher.py", line 195, in rknn.base.RKNNlib.switcher.mode_switcher.ModeSwitcher.optimize_mark
E   File "rknn/base/RKNNlib/switcher/mark_optimize.py", line 656, in rknn.base.RKNNlib.switcher.mark_optimize.MarkOptimize.apply
E   File "rknn/base/RKNNlib/switcher/mark_optimize.py", line 652, in rknn.base.RKNNlib.switcher.mark_optimize.MarkOptimize.forward_mark
E   File "rknn/base/RKNNlib/switcher/mark_optimize.py", line 28, in rknn.base.RKNNlib.switcher.mark_optimize.fwd_upstream
E AttributeError: 'NoneType' object has no attribute 'reset_shape_format'
E Please feedback the detailed log file <log.txt> to the RKNN Toolkit development team.
E You can also check github issues: https://github.com/rockchip-linux/rknn-toolkit/issues

``


Interpolation Search in Dart

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
The interpolation search algorithm in Dart.
<EMAIL_ADDRESS>Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes #5263

merge conflicts resolved


Print warnings on not being able to load certificates

I have an SSL app, which ran but all SSL connections failed as the certificates were with incorrect permissions (system admin had messed up).
Knowing that the problem was with the loading of certificates would have been helpful to debug. It took us really long to figure out it was a file issue and not an OpenSSL version issue.
Is there a way to have debug information with the library, so we can catch errors like this?
Thanks,
Aisha

Didn't the listen function just fail?

No, it didn't fail. :open_mouth:
Trying to read the files was causing an ENOENT error but the listen proceeded and the server was running.

Possibly related, just to add:

I had an SSLApp silently fail (just returns empty response) when I accidentally used port 80, instead of 443.
ca_file_name isn't mentioned on https://unetworking.github.io/uWebSockets.js/generated/interfaces/appoptions.html, but I found it at https://github.com/uNetworking/uWebSockets/blob/master/src/App.h#L38. It still worked though even without providing it (maybe not needed at all).

I'm on Ubuntu 20.04, uWebSockets.js v18.12.0

Possibly related, just to add:

I had an SSLApp silently fail (just returns empty response) when I accidentally used port 80, instead of 443.
ca_file_name isn't mentioned on https://unetworking.github.io/uWebSockets.js/generated/interfaces/appoptions.html, but I found it at https://github.com/uNetworking/uWebSockets/blob/master/src/App.h#L38. It still worked though even without providing it (maybe not needed at all).

I'm on Ubuntu 20.04, uWebSockets.js v18.12.0

Update: it looks like it only works on chrome desktop, but not on firefox desktop and chrome android.
I've tried the following but no luck:
let app = null;
let port = null;

const endpoint_domain = 'mydomain.com';
const key_file_name = `/etc/letsencrypt/live/${endpoint_domain}/privkey.pem`;
const cert_file_name = `/etc/letsencrypt/live/${endpoint_domain}/cert.pem`;
const ca_file_name = `/etc/letsencrypt/live/${endpoint_domain}/chain.pem`;

if (fs.existsSync(key_file_name) === true) {
  assert(fs.existsSync(cert_file_name) === true);
  assert(fs.existsSync(ca_file_name) === true);

  app = uws.SSLApp({ key_file_name, cert_file_name, ca_file_name });
  port = 443;
} else {
  app = uws.App({});
  port = 8080;
}

Update: it looks like it only works on chrome desktop, but not on firefox desktop and chrome android.
I've tried the following but no luck:
let app = null;
let port = null;

const endpoint_domain = 'mydomain.com';
const key_file_name = `/etc/letsencrypt/live/${endpoint_domain}/privkey.pem`;
const cert_file_name = `/etc/letsencrypt/live/${endpoint_domain}/cert.pem`;
const ca_file_name = `/etc/letsencrypt/live/${endpoint_domain}/chain.pem`;

if (fs.existsSync(key_file_name) === true) {
  assert(fs.existsSync(cert_file_name) === true);
  assert(fs.existsSync(ca_file_name) === true);

  app = uws.SSLApp({ key_file_name, cert_file_name, ca_file_name });
  port = 443;
} else {
  app = uws.App({});
  port = 8080;
}

Maybe this should be tied up with logging

Maybe this should be tied up with logging

There also seems to be no warnings/errors if uWS is compiled without "WITH_OPENSSL=1" . It compiles and executes without using an ssl context even when the code tells it to.

Should the cert_file_name be changed to fullchain_file_name or cert_ca_file_name?
fullchain.pem = cert.pem + ca.pem, right?

Renaming things is always fun but breaks backwards compatibility. So even a bad name kept is better than a good name introduced in many cases. You still are allowed to pass only cert, it works both ways.

fullchain_file_name is a good name, it can be added as an alias


Typescript debugging

This launch configuration allows me to debug typescript code:
        {
             "name": "[Development] Debug Client",
             "type": "chrome",
             "request": "launch",
             "url": "${auto-detect-url}",
             "sourceMaps": true,
             "userDataDir": "${workspaceRoot}/.vscode/chrome",
             "webRoot": "${workspaceRoot}/Client"
         }

Note, that first need to launch app in separate process using dotnet run.
userDataDir option forces to launch the separate chrome intance and may be removed

Is typescript debugging currently not working?

doesn't work for me when [Development] Launch Web is used

Let me take a look haven't reconfigured it in a while.
Thanks for letting me know!

I tried the configuration above (ran dotnet run in a separate cmd window), but it gave me:

Cannot connect to runtime process, timeout after 10000 ms - (reason: Can't find a target that matches: ${auto-detect-url}.

Was there anything else that needed to be setup?

The other option is debugging within Chrome devtools:

Doing a Ctrl-P (or Command-P for MAC) has the file navigator pop-up, so you can just search for /containers or the filename specifically to easily get to things.

what if you specify url directly here: "url": "${auto-detect-url}", ?

Didn't work when I did localhost:5000 there either.

I have checked and found a little mess. The following are the results (current master + latest VSCode (osx) + latest Debugger for Chrome):

Debugging within Chrome devtools works like a sharm. So double thanks!)
Breakpoints in typescript files are not enabled if use [Development] Launch Web configuration .
Default VSCode instance raises Debug adapter process has terminated unexpectedly error if I try to debug.
VSCode Insiders version raises the same Cannot connect to runtime process, timeout after 10000 ms error, as you mentioned. Actually, the problem is that new Chrome instance openshttp://%24%7Bauto-detect-url%7D/ url (and so cannot attach).  However, if Chrome already has a tab opened with localhost:5000 url,  debugging is working well.
Specifying url directly working for me in VSCode Insiders version.

{
"name": "[Development] Debug Client",
"type": "chrome",
"request": "launch",
"url": "http://localhost:5000",
"sourceMaps": true,
"userDataDir": "${workspaceRoot}/.vscode/chrome",
"webRoot": "${workspaceRoot}/Client"
},

Double check shows that Insiders version of VSCode has Debugger for Chrome 2.5.3, but default VSCode only 2.5.1 (no update available). So partially problem may be also with extension itself.

Just tried updating to 2.5.3 (with the latest VSCode, not the insider one or anything) and unable to get it working.
It would be great to have this ability though! Hope we can figure it out soon..

Was looking at this a bit, and I've found a way to get it working. I'll tossing in a quick pull request consideration, though we should probably discuss some particulars and update the readme before merging it. This stuff is kinda rough right now... hopefully the various teams working on VS Code and extensions can smooth this out soon.

Yeah VSCode definitely doesn't make it too easy (at the moment) to setup debugging unless it's the most basic project. Checking out the PR later or tomorrow Stephen, great job!

Yeah, VS Code F5 doesn't seem to work very well.  It sometimes work and sometimes does not.
Seems like running via command line is more stable at the moment.
I cloned yesterday fyi

@jrmcdona have you tried after the latest vscode update as well as the updated version of this Repo?

@isaac2004 which debug settings do you use to launch your app?  Or what is your work flow you found best for debugging TypeScript.
I am having some luck with setting a breakpoint in Chrome dev tools sources on a TypeScript file.
But other times, the DIST folder is bundled with not TypeScript sources.
I am unable to see why this behavior is different at times.

@jrmcdona   I imagine your devtool is doing something weird. devtool setting in webpack determines if you get sourcemaps or not. What is it set to?

@isaac2004 think you are right there.  I had touched my webpack.config because I was adding in the enviornments files (http://tattoocoder.com/angular-cli-using-the-environment-option/) and tring to some and created a new script build:devMocked to get all mocked environment.   I must have done something wrong.
After putting back the web.config.js I can now run:

[Development] Launch the server (no browser)
[Development] Debug TypeScript

My breakpoints are hitting now.  Thanks


Remove "CHARSET" config option for MySQL, always use "utf8mb4"

In modern days, there is no reason to make users set "charset" anymore.
Close #25378

@techknowlogick I do not think this is "breaking"

Marking as breaking so that should any user still have just utf8 in their config we can add a note to the blog post with a link to the doctor documentation

Marking as breaking so that should any user still have just utf8 in their config we can add a note to the blog post with a link to the doctor documentation

I do not see a case how this change would affect end users, so I do not think it's worth to mark it as breaking or make end users spend time on reading it.

Edit, it also changes the config variable name.

The new "MYSQL_CHARSET" is just a hidden fallback in case some users really really like to do something, but, I think we should never mention it. Just like I_AM_BEING_UNSAFE_RUNNING_.....
I do not think any one really needs it.

It fully removes an option that users may have been using (even if they should be using the newer version), and also, due to the rename it means that users will have to use a new key in the future.

It fully removes an option that users may have been using (even if they should be using the newer version), and also, due to the rename it means that users will have to use a new key in the future.

I am pretty sure the removal affects nothing.
The "rename" won't be exposed for end users in any case.

It fully removes an option that users may have been using (even if they should be using the newer version), and also, due to the rename it means that users will have to use a new key in the future.

I am pretty sure the removal affects nothing.
The "rename" won't be exposed for end users in any case.

The Gitea instance will not work if database is utf8 but the connection string is utf8mb4

The Gitea instance will not work if database is utf8 but the connection string is utf8mb4

I have tested, it works flawlessly
If you have a bad case, please help me to know.

The Gitea instance will not work if database is utf8 but the connection string is utf8mb4

I have tested, it works flawlessly
If you have a bad case, please help me to know.

Have you tested inputting an emoji? Maybe we just need to say on the release blog that you need to convert your database from utf8 -> utf8mb4 manually.

Have you tested inputting an emoji? Maybe we just need to say on the release blog that you need to convert your database from utf8 -> utf8mb4 manually.

That's a longstanding problem. Not related to this PR.

I don't think it's necessary to backport to v1.20

I don't think it's necessary to backport to v1.20

Then 1.20 still needs other fix

But yes, we should document it as breaking when we don't use a key anymore.
And let it just be so that users can remove it from their config.

Oh, perhaps we should also remove the FAQ entry now.


Verification

The project I contribute to regularly is not on the list of projects but I would like to please be considered.
Last year I was on an open source grant working on rust-bitcoin (you can check my contributions here). This project is used by other projects on the list e.g., LDK, BDK (and many other bitcoin projects written in Rust). I will most likely be continuing on a grant working on this project this year.
If accepted I will be traveling over from Australia. I am willing and able, if needed, to help out with open source content (talk, tutorial, or workshop).
(I do have a single commit merged into bitcoin-core/secp256k1.)

Withdrawing application, thank you.


Code Review

Good project. Just need to fix minor issues

Fix issues with "Edit task"
Fix localstorage issues

Thanks a lot for noticing the bug in the code, turns out I had not run npm run build and pushed the bunble.js with the new changes in my javascript files.
I had done that with this commit...as such i have resolved the issue.


fix(sec): upgrade com.fasterxml.jackson.core:jackson-databind to <IP_ADDRESS>

What happened？
There are 1 security vulnerabilities found in com.fasterxml.jackson.core:jackson-databind 2.10.5

CVE-2020-36518

What did I do？
Upgrade com.fasterxml.jackson.core:jackson-databind from 2.10.5 to <IP_ADDRESS> for vulnerability fix
What did you expect to happen？
Ideally, no insecure libs should be used.
How was this patch tested?
Run mvn compile failed locally, couldn't complete the build process.
Run mvn clean test failed locally, unit-test couldn't pass.
The specification of the pull request
PR Specification from OSCS

Test failed, PTAL.


Fix typo

.Net Framework -> .NET Framework

@changeworld : Thanks for your contribution! The author, @billmath, has been notified to review your proposed change.

Verified the updates.


Reflect.companionOf not robust

While trying to upgrade the example jobs, I accidentally passed the Nothing$ object class into Reflect.companionOf. Reflect appended $ to the name and tried and failed to find the class Nothing$$.
Ideally Reflect would use scala reflection to find the companion object of a class, detecting and sanely handling the possibility that the class may be a companion object or have no companion object.

Note that there is a getCodec method in maestro's LoadEx class, which solves the same problem that this method is used to solve when called from Util.createHiveTableDescriptor. It solves the problem a little more intelligently, albeit also not robustly. Perhaps the duplicated logic is a sign this method should be in some omnitool-reflection project?


How to improve forecast efficiency.

Hello, Currently i’m using Holt winter model for daily sales forecasting. I have two different type of data. One is restaurant data and another one is lifestyle store data.
When I’ve forecasted restaurant data and match with actual data then prediction is approximately same but same model not work on lifestyle store data, the difference between actual sell and predicted sell is very large for lifestyle store data.
Please tell me how to improve this model
code snapshot is give below

Hi Sushil,
This is a pretty specific example and not something that we might more generally document for other customers.  I'd suggest that you work with the community team on reviewing and getting some recommendations for this...  https://community.influxdata.com
Thanks...


Facing problem in recording

Facing problem in recording 5K and 8K video recording.

please provide:

Hardware spec of machine
os
recorder settings.

Laptop series : https://www.asus.com/in/ROG-Republic-Of-Gamers/ROG-GX700VO/
RAM : 64 GB
Graphics : 8 GB Graphics card GTX 980
Windows : Windows 10 updated
Recorder setting : Resolution X4320_8k, Target fps : NTSC (29.9)

hum, re-reading the issue, what do you mean by "Facing problem"?
Also, does the console's output log contain any warnings or errors?

mp4 upper limit is 4k. (format specs)
added console errors to inform about that.
webm on the other hand, support up to 16k (but unity does not, 8k max)


Environment variable $HOME in docker-compose.yml is not expanded

Description
Copy the issue from https://github.com/runfinch/finch/issues/124.
Environment variable $HOME in docker-compose.yml is not expanded.
Steps to reproduce the issue

Create the docker-compose.yml below.

version: '3.7'
services:
  wordpress:
    image: wordpress:5.7
    restart: always
    ports:
      - 8080:80
    environment:
      WORDPRESS_DB_HOST: db
      WORDPRESS_DB_USER: exampleuser
      WORDPRESS_DB_PASSWORD: examplepass
      WORDPRESS_DB_NAME: exampledb
    volumes:
      - wordpress:/var/www/html
      - ${HOME}/hello.txt:/root/hello.txt
  db:
    image: mariadb:10.5
    restart: always
    environment:
      MYSQL_DATABASE: exampledb
      MYSQL_USER: exampleuser
      MYSQL_PASSWORD: examplepass
      MYSQL_RANDOM_ROOT_PASSWORD: '1'
    volumes:
      - db:/var/lib/mysql

volumes:
  wordpress:
  db:

Create a file to mount the volume.

echo $HOME
/Users/hieki

echo hello > ~/hello.txt
cat ~/hello.txt
hello

Run lima nerdctl compose up and it failed.

WARN[0000] Ignoring: volume: Bind: [CreateHostPath]
INFO[0000] Ensuring image wordpress:5.7
INFO[0000] Ensuring image mariadb:10.5
INFO[0000] Creating container finch_wordpress_1
INFO[0000] Re-creating container finch_db_1
FATA[0000] error while creating container finch_wordpress_1: exit status 1

Modify docker-compose.yml and rewrite the files to mount to absolute paths. It works fine.

version: '3.7'
services:
  wordpress:
    image: wordpress:5.7
    restart: always
    ports:
      - 8080:80
    environment:
      WORDPRESS_DB_HOST: db
      WORDPRESS_DB_USER: exampleuser
      WORDPRESS_DB_PASSWORD: examplepass
      WORDPRESS_DB_NAME: exampledb
    volumes:
      - wordpress:/var/www/html
      - /Users/hieki/hello.txt:/root/hello.txt
  db:
    image: mariadb:10.5
    restart: always
    environment:
      MYSQL_DATABASE: exampledb
      MYSQL_USER: exampleuser
      MYSQL_PASSWORD: examplepass
      MYSQL_RANDOM_ROOT_PASSWORD: '1'
    volumes:
      - db:/var/lib/mysql

volumes:
  wordpress:
  db:

Describe the results you received and expected
Environment variable $HOME in docker-compose.yml should be expanded.
What version of nerdctl are you using?
Client:
 Version:	v1.1.0
 OS/Arch:	linux/amd64
 Git commit:	18944bc70784dfa83010d37054d75487a58ab581
 buildctl:
  Version:	v0.10.6
  GitCommit:	0c9b5aeb269c740650786ba77d882b0259415ec7

Server:
 containerd:
  Version:	v1.6.12
  GitCommit:	a05d175400b1145e5e6a735a6710579d181e7fb0
 runc:
  Version:	1.1.4
  GitCommit:	v1.1.4-0-g5fd4c4d1

Are you using a variant of nerdctl? (e.g., Rancher Desktop)
Lima
Host information
Client:
 Namespace:	default
 Debug Mode:	false

Server:
 Server Version: v1.6.12
 Storage Driver: overlayfs
 Logging Driver: json-file
 Cgroup Driver: systemd
 Cgroup Version: 2
 Plugins:
  Log: fluentd journald json-file syslog
  Storage: native overlayfs fuse-overlayfs stargz
 Security Options:
  apparmor
  seccomp
   Profile: default
  cgroupns
  rootless
 Kernel Version: 5.19.0-26-generic
 Operating System: Ubuntu 22.10
 OSType: linux
 Architecture: x86_64
 CPUs: 4
 Total Memory: 3.825GiB
 Name: lima-default
 ID: 12637925-c1a5-4574-beda-a33046b62651

might be relevant to #1152 and #722

Duplicate of:

https://github.com/containerd/nerdctl/issues/996


Onbording> 31 days failing from backend API

As a superuser i want to see user with multiple status.
In below edge case i have selected both Active and Onboardingstatus>31 error has been observed below
{
"statusCode": 400,
"error": "Bad Request",
"message": ""state[1]" must be one of [OOO, ONBOARDING, IDLE, ACTIVE]"
}

Hey @iamitprakash can you confirm if the api used here is /users/search

Assigned to me.


tests: session expiration test

creates a session with expiration of 1 second
and waits 1.1 seconds before checking its value
I tested on the non-docker stores, let's see if the CI test passes on the ones with docker

Seems like this is passing.


fix(docs): some English typos

Possibly found some English typos.
I haven't changed the content of orangeHelvetica.css.map nor orangeHelvetica.min.css.map because I suppose they are generated automatically ; but I don't know exactly how to do that 😃

Thanks!


BUG: Inconsistent Output from BenchGlobal Compared to BenchLeastSquares in Dev Benchmarks

Describe your issue.
I've encountered an issue where the output from running the BenchGlobal benchmark is not formatted as a proper table, unlike the output from BenchLeastSquares. This inconsistency suggests there might be a bug in the BenchGlobal class.
Run the benchmark for BenchLeastSquares and BenchGlobal with the commands below.

               For parameters: 'Zirilli', '<nfev>', 'DIRECT'
               asv: skipped: NotImplementedError('skipped')

               For parameters: 'Zirilli', '<nfev>', 'SHGO'
               asv: skipped: NotImplementedError('skipped')

SciPy/NumPy/Python version and system information
1.13.0.dev0+1169.87f8428 1.26.3 sys.version_info(major=3, minor=9, micro=14, releaselevel='final', serial=0)

Build Dependencies:
  blas:
    detection method: pkgconfig
    found: true
    include directory: /opt/homebrew/Cellar/openblas/0.3.25/include
    lib directory: /opt/homebrew/Cellar/openblas/0.3.25/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP=1 VORTEX MAX_THREADS=56
    pc file directory: /opt/homebrew/opt/openblas/lib/pkgconfig
    version: 0.3.25
  lapack:
    detection method: pkgconfig
    found: true
    include directory: /opt/homebrew/Cellar/openblas/0.3.25/include
    lib directory: /opt/homebrew/Cellar/openblas/0.3.25/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP=1 VORTEX MAX_THREADS=56
    pc file directory: /opt/homebrew/opt/openblas/lib/pkgconfig
    version: 0.3.25
  pybind11:
    detection method: config-tool
    include directory: unknown
    name: pybind11
    version: 2.11.1
Compilers:
  c:
    commands: cc
    linker: ld64
    name: clang
    version: 15.0.0
  c++:
    commands: c++
    linker: ld64
    name: clang
    version: 15.0.0
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 3.0.8
  fortran:
    commands: gfortran
    linker: ld64
    name: gcc
    version: 13.2.0
  pythran:
    include directory: ../../../../../private/var/folders/s7/rbrc7kq522bbv0pb_bwfrt7w0000gp/T/pip-build-env-bblq6l9m/overlay/lib/python3.9/site-packages/pythran
    version: 0.15.0
Machine Information:
  build:
    cpu: aarch64
    endian: little
    family: aarch64
    system: darwin
  cross-compiled: false
  host:
    cpu: aarch64
    endian: little
    family: aarch64
    system: darwin
Python Information:
  path: /Users/tessavanderheiden/.venvs/scipy-dev/bin/python
  version: '3.9'

When I comment out def setup() (line) it seems to work:
    def setup(self, name, ret_value, solver):
        if name not in self._enabled_functions:
            raise NotImplementedError("skipped")

        # load json backing file
        with open(self.dump_fn) as f:
            self.results = json.load(f)

Can I make a PR?

If the issue is that all the BenchGlobal benchmarks are skipped by default, this is expected behavior. The environment variable SCIPY_XSLOW must be set to 1 for those to run, as documented in this comment.

My apologies for bothering you and thanks @lucascolley and @thalassemia for the comments.
I am using:
export SCIPY_GLOBAL_BENCH=AMGM
export SCIPY_XSLOW=1
export SCIPY_GLOBAL_BENCH_NUMTRIALS=10
python dev.py bench -t optimize.BenchGlobal

Which correctly shows results (and limits the run time of the global benchmarks).


Update to work with GraphQLSwift 0.4.0

I just started playing around with your other library, but was disappointed to find out that the latest version doesn't work with Graphiti. I tried forking the library and updating the Package.swift file, but there are a bunch of compile errors mostly due to changes in your Runtime dependency that I wasn't sure how to resolve. It would be great if these two could work together again.

fixed by #15


Login loop after a couple hours of use

Checklist

[X] I have read the FAQ
[X] I have checked if this issue already exists (both open and closed issues)
[X] I am using the most recent version of pogom from the master branch
[X] If this is an installation issue, I am posting my OS, OS version, python version, pip version
[X] I'm posting the complete log
[X] I'm using a descriptive title for this issue

My Issue
Begins to infinitely loop after a couple hours of use. Always 3 accounts involved in the loop, I'm going to go out on a limb and say that this number correlates with the number of threads (also 3). No debug log as yet. Running on Rasbian, current source from master, 56 accounts being used.

Apologies for the rushed name scrubbing, you should get the idea.

I confirm, same thing on Ubuntu server 15.04 x64. This loop actually made the map crash after a while.

I confirm this on Windows 10

Confirmed Debian x64

Confirmed Centos 7 x64, mine got into a login loop for 4 accs till my server got softbanned.

Same in docker ;)

Confirmed Win7 x64

Same on raspberry pi (debian).

Confirmed - Win10 x64

Confirmed - Win10 x64, 10 accounts 3 threads.

I have other scanners running and have not been softbanned since I stopped running pogom. This issue is causing the softbans. The logins are hammering the server and they are automatically timebanning IP addresses.

Typically restarting the python script has fixed the problem straight away, and has a few times now. Unless the restart has been perfectly timed, I'm not convinced it's an IP ban. Account soft ban? Maybe.

Although, as far as I'm aware, soft bans only stop Pokémon from showing up. This wouldn't explain the problems logging into an account.

I have the same problem. I logged the remaining time and it seems like auth_provider._ticket_expire is in the past.

got the same, it goes in an endless loop.
closed the scanner and started up just to see they are all softbanned due that.
possibly its due massive ( 40 accounts ) trying to login at the same time.
best way would be that we add a sleep timer for login accounts, so that we dont try to login with 40 accounts or even more at the same time.
so when you start the server it logs in with 1 account, few seconds later the second etc.etc.
it will take some time before all 40 accounts are logged in due that, but it will avoid that the server is bumping the accounts right away propably...

Another log. It seems like the _ticket_expire is already divided by 1000 so when it divides again before subtracting the current timestamp, I get a negative remaining time, which is smaller than 60 so it tries to log in again.

I have encountered the same problem on OS X with 160 accounts, 7 threads.
I wonder if it can be temporarily fixed by adding a simple check for negative numbers on line 301 in pogom/pgoapi/pgoapi.py.
Something like this
if remaining_time < 60 and remaining_time >= 0:

I wonder if it can be temporarily fixed by adding a simple check for negative numbers on line 301 in pogom/pgoapi/pgoapi.py.

I thinks it's normal to have values like a few seconds negative depending the call rate but not -1470000000..

Thanks for the log! It's helping us locate the bug. Though we don't have fix ready yet.
The issue lies deeper than the negative timestamps (we're using negative time diffs to force a re-login sometimes, the problem is what's causing that reset).

I have inserted some log statements into the code on this branch:
https://github.com/favll/pogom/tree/log-login-process
Please check it out and post your logs when the bug occurs.

hi,
here is the common output:

2016-08-11 10:57:27,548 [     pgoapi] [   INFO] Login successful: l09qXWwfkr6glLL
2016-08-11 10:57:27,548 [    rpc_api] [WARNING] No Session Ticket found - using OAUTH Access Token
2016-08-11 10:57:27,567 [   auth_ptc] [WARNING] access_token=TGT-13024835-NvVlT3Bno5BO3DwkOsMBez6zOUkTxprlHB6UaXWvrvAXZWJQzN-sso.pokemon.com&expires=10800 200 https://sso.pokemon.com/sso/oauth2.0/accessToken

Happens here too, Windows 10. Goes into a loop of attempting login, login expires, etc. With just 6 accounts. Restarting the server fixes it for a while.

Another log with the logging branch.
https://gist.github.com/midoritest/c7292257f4968d4e3591f3f6ff6ad11a

We need a log of when the error occurs.

The log in my comment above is from the time when the error happened. After that accs 208, 209, 214 looped over and over.

The log in my comment above is from the time when the error happened. After that accs 208, 209, 214 looped over and over.

We found the bug.  The server announces an expiry time of 3 hours for the access token, but the actual expiry time is 2 hours. Fix coming soon.

Do you think that the account are ban?

Sounds like you've figured it out, but for what its worth I seem to notice this coincides with the exact moment I get a soft-ip-ban.

Fixed in fd879e92e29da3cf38c1ebc093913512ef68dce0
Further improved in 1436d13d228b0cec81c26be456b7185d53971a50


Dead links

On pages:

https://docs.microsoft.com/en-us/powershell/module/wdac/Add-OdbcDsn?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/disable-odbcperfcounter?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/disable-wdacbidtrace?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/enable-odbcperfcounter?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/enable-wdacbidtrace?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/get-odbcdriver?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/get-odbcdsn?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/get-odbcperfcounter?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/get-wdacbidtrace?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/remove-odbcdsn?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/set-odbcdriver?view=winserver2012r2-ps
https://docs.microsoft.com/en-us/powershell/module/wdac/set-odbcdsn?view=winserver2012r2-ps

New-CimSession has forward link http://go.microsoft.com/fwlink/p/?LinkId=227967, which goes to https://docs.microsoft.com/en-us/powershell/module/cim/new-cimsession but this is a dead link (should be https://docs.microsoft.com/en-us/powershell/module/cimcmdlets/new-cimsession )
Get-CimSession has forward link http://go.microsoft.com/fwlink/p/?LinkId=227966, which goes to https://docs.microsoft.com/en-us/powershell/module/cim/get-cimsession but this is a dead link (should be https://docs.microsoft.com/en-us/powershell/module/cimcmdlets/get-cimsession )

@officedocsbot assign @yogkumgit

the link redirects have been updated and are going to the correct article.

@VLG17  Thank you very much for the contribution and sharing this explanation. @Styxxy
Hope this comment is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@officedocsbot close


Generated code wraps update sql data with parenthesis

After updating to v0.23.0 the generated code produces the error below in sql. I tried downgrading to v0.22.0 and everything works as expected.
This is a postgres database and we are not using any custom templates for bob.
postgres-1  | 2024-01-04 07:34:21.716 UTC [2062] ERROR:  syntax error at or near "=" at character 47
postgres-1  | 2024-01-04 07:34:21.716 UTC [2062] STATEMENT:  UPDATE "blocks" AS "blocks" SET
postgres-1  | 	("deleted_at" = $1)
postgres-1  | 	WHERE ("id" IN ($2))

postgres-1  | 2024-01-04 07:34:16.685 UTC [2050] ERROR:  syntax error at or near "=" at character 47
postgres-1  | 2024-01-04 07:34:16.685 UTC [2050] STATEMENT:  UPDATE "blocks" AS "blocks" SET
postgres-1  | 	("updated_at" = $1),
postgres-1  | 	("deleted_at" = $2)
postgres-1  | 	WHERE ("id" IN ($3))

It wraps the data to be updated in parenthesis
blocks, err := models.Blocks.Query(...).All()
err = blocks.UpdateAll(ctx, storage.Bob, models.BlockSetter{DeletedAt: omitnull.From(time.Now())})

The same goes for the (*block).Update(...)

Fixed in v0.23.1
Although v0.23.2 has also been released with additional fixes.

Awesome, thanks!


NIFI-11408 enable user to disable gzip compression with PutGCSObject

Summary
NIFI-00000
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[ ] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Reviewing...

Tested the PR and confirmed the expected behavior. Merging. Thanks @joewitt


check brave browser for potential issue on trading page

I tested with Brave browser cand couldn't reproduce any browser-specific bug.


Fix reverse DNAT for externalTrafficPolicy=Local service with kube-proxy

See commits for details.
I'll rebase on https://github.com/cilium/cilium/pull/14675 once merged (the two PRs share the same first commit).

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-4.9

test-netnext

test-runtime

test-me-please

test-me-please

test-gke

test-me-please

test-gke

test-me-please

test-gke

GKE hit known flake #14915 . With CI signoff, this should be good to merge. @christarazi  / @michi-covalent could you check over the CI side here?

I updated the release note given this PR also fixed https://github.com/cilium/cilium/issues/12542.


when internet is disconnected the script keeps going to the next file to download

when instaloading a profile if internet is disconnected the script keeps sending request and showing errors but keeps going to the next file instead of waiting to finalize the file (until the internet is connected again) and then moving to the next one.
i think lack of this feature can cause inconsistencies when using instaloader where data consistency and validity is important.

Commit a1920b9 adds option --max-connection-attempts. With --max-connection-attempts=0, Instaloader retries failing connections infinitely without going to the next file. It is still possible to abort the retries with ^C.


FhirContext is not threadsave on initialization

The first time a resource is created with the FhirContext class the resource types are scanned ie. validateInitialized is called. If multiple parallel requests are incoming, the first will start the initialization and the second will skip this and use the uninitialized class which then results in a NPE.
The synchronization method in FhirContext.validateInitialized is not sufficient to cover this case.

Hi @jodue - Would you be able to provide code which shows how to replicate this?

public static void main( String args[] )     {         FhirContext dstu3FhirContext = FhirContext.forDstu3();         for ( int i = 0; i < 10; i++ )         {             new Thread( new Runnable()             {                 @Override                 public void run()                 {                     OperationOutcomeUtil.newInstance( dstu3FhirContext );                 }             } ).start();         }     }
Will result in multiple NPEs:
Exception in thread "Thread-4" java.lang.NullPointerException 	at ca.uhn.fhir.context.FhirContext.getResourceDefinition(FhirContext.java:376) 	at ca.uhn.fhir.util.OperationOutcomeUtil.newInstance(OperationOutcomeUtil.java:138) 	at com.agfa.mma.launchers.Test$1.run(Test.java:33) 	at java.lang.Thread.run(Thread.java:745)

Awesome, thanks! I'll have a look.

@jodue - Hmm, weird! I added a test method using your code, but it passes consistently for me.
Can you describe your set up? What version of HAPI are you using, what platform, what version of Java, etc?

Hi James, sadly i'm still using HAPI FHIR 2.3 and obviously should update! After trying my test case on version 3.0.0 i found out that this issue was already fixed in #610! Sorry for the additional work!
In the future i will try with the newest version before submitting a bug! :)


Ablility to display thumbnail?

Hi, this is just a question.  Can the first frame or thumbnail be displayed initially without auto play set to true?
I tried
player.Seek(1);
player.Play();
player.Pause();
on an h264 video with progressive playback format.  I thought this might get the first frame to load and pause, but it did not.  Any help would be appreciated.  thanks

The problem probably is that I'm trying to display these in a listview, with some sort of thumbnail or indicator that the item is a video.  I've set display controls to true, but they do not appear

I would avoid assigning a source to the player until you are ready to actually play it because that will consume system resources if the autoplay is set to true. You may need to use some other native API to capture screenshots from the video to display thumbnails. I'm not familiar with any immediately that would help with that but I would just swap the thumbnail out for a single instance of the player when a user taps on a thumbnail.

Hi was there any solution to this?  I would like to grab a snapshot of the video playing?


Unable to install on Big Sur with M1 chip: ld: library not found for -lodbc

PHP Driver version or file name:  5.10.0-beta1.
Client operating system: Big Sur
PHP version: 8.0.12
Microsoft ODBC Driver version: 2.3.9_1
Problem description:
My company bought a new macbook pro with M1 chip and I am trying to install the sqlsrv extensions for php for a week now, and still I didn't manage to do so.
The ODBC driver is installed correctly as I can see in the phpconfig() output.
Since I am dealing with a M1 chip, I am trying to install specifically version 5.10.0-beta1 of sqlsrv.
The install stops with the following:

ld: library not found for -lodbc
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [sqlsrv.la] Error 1
ERROR: `make' failed

Another thing I tried is installing this specific version of the driver on another mac (with i9 chip however) and copy sqlsrv.so and pdo_sqlsrv.so to the new macbook and into the extension_dir.
This results in php complaining:

PHP Warning:  PHP Startup: Unable to load dynamic library 'sqlsrv.so' (tried: /opt/homebrew/Cellar/php/8.0.12/lib/php/20200930/sqlsrv.so (dlopen(/opt/homebrew/Cellar/php/8.0.12/lib/php/20200930/sqlsrv.so, 9): no suitable image found.  Did find:
/opt/homebrew/Cellar/php/8.0.12/lib/php/20200930/sqlsrv.so: mach-o, but wrong architecture
/opt/homebrew/Cellar/php/8.0.12/lib/php/20200930/sqlsrv.so: stat() failed with errno=22), /opt/homebrew/Cellar/php/8.0.12/lib/php/20200930/sqlsrv.so.so (dlopen(/opt/homebrew/Cellar/php/8.0.12/lib/php/20200930/sqlsrv.so.so, 9): image not found)) in Unknown on line 0

By now I am out of ideas of what to do..

Hi @jotso have you followed the installation instruction?
What's the output when you type odbcinst -j?

Hi @yitam,
Thanks a ton! There was a magic line in the instructions which I missed out on:
sudo CXXFLAGS="-I/opt/homebrew/opt/unixodbc/include/" LDFLAGS="-L/opt/homebrew/lib/" pecl install sqlsrv
sudo CXXFLAGS="-I/opt/homebrew/opt/unixodbc/include/" LDFLAGS="-L/opt/homebrew/lib/" pecl install pdo_sqlsrv
Now that's installed, I am getting the following when running a test script:
SQLSTATE: 08001Code: -1Message: [Microsoft][ODBC Driver 17 for SQL Server]SSL Provider: [OpenSSL library could not be loaded, make sure OpenSSL 1.0 or 1.1 is installed]
in my phpconfig() it shows openssl is installed (version 1.1). I also installed unixodbc version <IP_ADDRESS>
And to come back to your question, the output of odbcinst -j is:
unixODBC 2.3.9
DRIVERS............: /opt/homebrew/etc/odbcinst.ini
SYSTEM DATA SOURCES: /opt/homebrew/etc/odbc.ini
FILE DATA SOURCES..: /opt/homebrew/etc/ODBCDataSources
USER DATA SOURCES..: /Users/bobvandoorn/.odbc.ini
SQLULEN Size.......: 8
SQLLEN Size........: 8
SQLSETPOSIROW Size.: 8
Any ideas would be great!

hi @jotso the ODBC team is working to fix this known issue with OpenSSL. Please use the workaround I suggested in #1319

Hi @yitam thanks again, I think we have it up and running now!

Great, @jotso ! Closing this issue now.


compare highscore stats

this will be awesome to the client, i think i havent seen anyone have this kind of thing on their clients.

maybe add another new slot to compare? but then it wouldnt be able to be part of lookup so you will have to type their full osrs name :(

or you can add this new addition on lookup section?

I think this was partially implemented by stat compare in opponent info and is probably no longer needed/relevant


Bottom up mode for custom menu

When in command line mode, the custom entry window opens up to an
unexpected height, depending on the current count of completion items.
The above makes it hard to anticipate where to look at, and makes life a bit
harder.
This patch adds an option to open the custom entries view in a bottom up
mode, and flips the regular behaviour of next/prev entry in this mode.
Setup is as easy as:
cmp.setup.cmdline(':', {
  view = {
    entries = {name = 'custom', selection_order = 'near_cursor' }
  }
}

@hrsh7th
Any chance of merging this?
10x

I think this must be smarter because the current option is only worth cmdline-completion.
For example, we can determine the direction via the window is upper the cursor or not.
I don't imagine the suitable option name but it should be changed, I think. e.g.: selection_order = 'top_down' | 'near_cursor'.

I like your idea.
Will try to implement.

Please consider the suitable option name :)

sure

@hrsh7th
I think this last commit fixes all issues.
If you have nothing else, I would appreciate if you can merge this.
Cheers.

Could you share an image or a gif of how this will look like?

@hrsh7th
Apologies if im pushing, but this has been ready for a while now.
Do you intend to merge?

LGTM.
I've checked the behavior. I don't like this behavior but someone might like this may be.

Thank you.
Ill add a wiki for this feature.

This PR broke command line autocompletion from Vim proper for me. When I open vim and do :e  I get an I^.

I can't reproduce. How do you determine the commit that broke the behavior?

I think this is something different. nvim now differentiats between
and .
On Fri, 8 Apr 2022, 17:26 hrsh7th, @.***> wrote:

I can't reproduce. How do you determine the commit that broke the behavior?
—
Reply to this email directly, view it on GitHub
https://github.com/hrsh7th/nvim-cmp/pull/848#issuecomment-1092920290,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFXXC2YP7OTWHGEG5RAGX3VEA6YTANCNFSM5RB7CSSQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

same to me, Tab in command line output like this

Tab and S-Tab mapped by cmp，but i do not set those in my cmp mapping config

This is not related to this PR. I've fixed it in latest main.


Add delete and list command

This isnt perfect, but this should handle the 90% case for when users dont want to have to install kind separately.

@nimakaviani @nabuskey Sorry I dropped this - it should be ready for re-review

Can you resolve the conflicts (rebase) please ?

@cmoulliard I moved this to match Kind's upstream behavior - for a 'get clusters' command I'm not sure its necessary to have templates/json/etc. IME its pretty common to simply list the clusters (this is very useful for e.g. shell tooling integration). If we want to add extra formatting dont let me stop anyone from making that an optional argument.

@nabuskey @nimakaviani Can you all re-review this please? thanks

@greghaynes once you resolve the conflict, this should be good to go.


Suggested improvements to beer song exercise

Related to the 99 Bottles of OOP book, we have a repository containing all the available BeerSong exercises from Exercism.
One reader, @dcarral, submitted a whole host of little fixes and improvements, that I'd like to bring into Exercism where appropriate.
Here are the fixes for the JavaScript exercise:

Improve JavaScript test suite by modifying expectations and adding a full lyrics test.
Adds empty implementation file in JavaScript: this way participants achieve failing test ('BeerSong is not a constructor') instead of module loading error when trying to require a non-existing file.

The changes are in this file:
https://github.com/dcarral/99bottles-polyglot/pull/11/files#diff-dfaa7cd627b791e136aaca588bee0772
I can't speak to the JavaScript aspect of these, but I think they're worth considering.

I wanted to add a vote in favor of adding implementation starter file to each exercise, would be a great QOL helper for experienced programmers and prevent a stumbling block for beginners.
I'd be happy to implement this and/or the other changes if desired.

@benfletcher @rchavarria @kytrinyx I think it could be valuable to add this case-by-case to certain exercises.  I am not in favor of adding stubs to all exercises, but that's just because there are so many different ways to implement most of the solutions in JavaScript.  I think stubs have value in two contexts:

Early in the track to help people who are new to the language, or to programming in general.
For specific problems with arcane or tricky APIs.

That's my 2 cents.

I am also not in favor of adding stubs to all the exercises for the reasons that @matthewmorgan mentions.

wontfix per #556.


Convert user docs from rST to MyST

MyST solves a lot of the annoyances I've encountered with rST. My experience so far is that it gives the best of both worlds: the sane, familiar syntax of Markdown, plus the handy features of rST and Sphinx directives.
There's no need to convert the auto-generated docs; this will only apply to the user guides.

Coverage remained the same at 97.66% when pulling dd79fbae922db99b35ad22f698766f590b60adf6 on JWCook:myst into 79f2dd9707666ca8fb9e63e331d71c0a376c029e on reclosedev:master.


Preserve path when copying resources

This change will let preserve relative path of all assets and not copy them to root folder
This change also would require to put pdxinfo in root of source code and therefore update template repository. May be we should introduce assets folder and pass it as an argument to PDC plugin?

Thanks for the PR, this seems like a nice feature to have. I like the idea of an assets folder argument that defaults to Resources or something, needing to put the pdxinfo at the root isn't ideal.

Going to close this as https://github.com/finnvoor/PlaydateKit/pull/103 seems to resolve it, but thanks for the initial contribution!


Fix type errors in Ember tests

#89 introduced some type errors in the Ember tests that I didn't notice because noEmitOnError was set to false. So set it to true and fix up the type errors.
In fixing the type errors, I realized that wrapping selector() has gotten a good bit more complex because selector()'s type is more complex, so I re-implemented the test to define a generic wrapSelector() function that can be used to produce a wrapper that transforms the arguments before passing them to selector(). I did the same for globalSelector() (whose type is even more complex). We could potentially put these in the library and export them to make them available to consumers, but I want to give them a little more time and get some feedback from the folks (or maybe just one person?) that wraps selector() before promoting them to a "public API."

@NullVoxPopuli I'd love for you to take a look at this, and also at #89 for context.
Basically, #89 adds some new type functionality but also makes the selector() (and globalSelector()) types more complex, so wrapping them (which was a use case that you originally brought to my attention) is more complex.
I don't have a specific ask here, other than taking a quick look and letting me know if you have any concerns/suggestions/feedback/whatever, or just an "LGTM" 😄


kiosk: remap esc to the controller's back button

Exit a game using the controller's back button (view button on newer controllers), not the start button (menu button on newer controllers). Start/view is used by the simulator to open the in-game settings menu.
Fixes https://github.com/microsoft/pxt-arcade/issues/6145

Looks okay to me! We had also discussed making the Y button function as a back button - is that still something we're considering, or did you decide against it?

Oh yes! I will handle this in a separate PR. Thanks for the reminder.


memory access violation in master

make test
npm test

><EMAIL_ADDRESS>test /home/artem/projects/vtcomposite
> tape test/*.test.js

TAP version 13
# [composite] success: buffer size stays the same when no compositing needed
[buffer size] cpp pre vtzero: 101067
[buffer size] cpp output:101067
ok 1 should be falsy
ok 2 same size
# [composite] success compositing - same layer name, same features, same zoom
free(): invalid pointer
Aborted (core dumped)
npm ERR! Test failed.  See above for more details.
Makefile:90: recipe for target 'test' failed
make: *** [test] Error 1

The above triggered by:  https://github.com/mapbox/vtcomposite/blob/master/test/vtcomposite.test.js#L43

This PR is in the merging queue once this issue is resolved
/cc @mapsam @millzpaugh

:+1:


[Bug]: Vercel postgres - Error: Connection terminated

Package
lucia-auth
Describe the bug
When attempting to use lucia with Next.js and vercel postgres, I'm getting the following:
Error: Connection terminated
    at cn.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3890:40)
    at Object.onceWrapper (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1127:141)
    at cn.emit (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1099:68)
    at x.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3737:19)
    at x.emit (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1098:37)
    at WebSocket.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3050:22)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:731:20)
    at WebSocket.dispatchEvent (node:internal/event_target:673:26)
    at fireEvent (eval at requireWithFakeGlobalScope (C:\Users\flomu\Desktop\Inventory\nextjs-dashboard\node_modules\next\dist\compiled\edge-runtime\index.js:1:657096), <anonymous>:12687:14)
    at TLSSocket.onSocketClose (eval at requireWithFakeGlobalScope (C:\Users\flomu\Desktop\Inventory\nextjs-dashboard\node_modules\next\dist\compiled\edge-runtime\index.js:1:657096), <anonymous>:12884:7)
Error: Connection terminated
    at cn.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3890:40)
    at Object.onceWrapper (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1127:141)
    at cn.emit (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1099:68)
    at x.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3737:19)
    at x.emit (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1098:37)
    at WebSocket.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3050:22)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:731:20)
    at WebSocket.dispatchEvent (node:internal/event_target:673:26)
    at fireEvent (eval at requireWithFakeGlobalScope (C:\Users\flomu\Desktop\Inventory\nextjs-dashboard\node_modules\next\dist\compiled\edge-runtime\index.js:1:657096), <anonymous>:12687:14)
    at TLSSocket.onSocketClose (eval at requireWithFakeGlobalScope (C:\Users\flomu\Desktop\Inventory\nextjs-dashboard\node_modules\next\dist\compiled\edge-runtime\index.js:1:657096), <anonymous>:12884:7)
 ⨯ unhandledRejection: Error: Connection terminated
    at cn.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3890:40)
    at Object.onceWrapper (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1127:141)
    at cn.emit (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1099:68)
    at x.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3737:19)
    at x.emit (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1098:37)
    at WebSocket.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3050:22)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:731:20)
    at WebSocket.dispatchEvent (node:internal/event_target:673:26)
    at fireEvent (eval at requireWithFakeGlobalScope (C:\Users\flomu\Desktop\Inventory\nextjs-dashboard\node_modules\next\dist\compiled\edge-runtime\index.js:1:657096), <anonymous>:12687:14)
    at TLSSocket.onSocketClose (eval at requireWithFakeGlobalScope (C:\Users\flomu\Desktop\Inventory\nextjs-dashboard\node_modules\next\dist\compiled\edge-runtime\index.js:1:657096), <anonymous>:12884:7)
 ⨯ unhandledRejection: Error: Connection terminated
    at cn.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3890:40)
    at Object.onceWrapper (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1127:141)
    at cn.emit (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1099:68)
    at x.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3737:19)
    at x.emit (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:1098:37)
    at WebSocket.eval (webpack-internal:///(middleware)/./node_modules/@neondatabase/serverless/index.js:3050:22)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:731:20)
    at WebSocket.dispatchEvent (node:internal/event_target:673:26)
    at fireEvent (eval at requireWithFakeGlobalScope (C:\Users\flomu\Desktop\Inventory\nextjs-dashboard\node_modules\next\dist\compiled\edge-runtime\index.js:1:657096), <anonymous>:12687:14)
    at TLSSocket.onSocketClose (eval at requireWithFakeGlobalScope (C:\Users\flomu\Desktop\Inventory\nextjs-dashboard\node_modules\next\dist\compiled\edge-runtime\index.js:1:657096), <anonymous>:12884:7)

Here's how I have it configured in my code, following the documentation example:
// lucia.ts
import { lucia } from "lucia";
import { pg } from "@lucia-auth/adapter-postgresql";
import { db } from "@vercel/postgres";
import { nextjs_future } from "lucia/middleware";
import "lucia/polyfill/node"; // polyfill for node 18 and below

export const auth = lucia({
	adapter: pg(db, {
		user: "auth_user",
		key: "user_key",
		session: "user_session"
	}),
	env: "DEV",
	middleware: nextjs_future(),
	sessionCookie: {
		expires: false
	},
});

// middleware.ts
export default async function middleware(req: NextRequest) {
  const url = req.nextUrl;

  try {
    const user = await auth.createUser({
      key: {
        providerId: "email",
        providerUserId<EMAIL_ADDRESS>        password: "!Login123"
      },
      attributes: {
        email<EMAIL_ADDRESS>				email_verified: false // `Boolean(false)` if stored as an integer
      } // expects `Lucia.DatabaseUserAttributes`
    });
    console.log(user)
  } catch (e) {
    if (e instanceof LuciaError && e.message === `AUTH_DUPLICATE_KEY_ID`) {
      // key already exists
      console.log('key already exists!')
    }
    // provided user attributes violates database rules (e.g. unique constraint)
    // or unexpected database errors
  }
...
...
...

I have created the three necessary tables in vercel postgres already.

Can you first verify that it’s an issue with Lucia and not how you setup your database connection?

@pilcrowOnPaper I have the app connected to the vercel postgres database already and can read and write data no problem.

Can you check whether you can query your database in the middleware?

Yes I am able to query another table from the middleware function

Are you querying using db instead of sql?

I'm able to query data with both db and sql from the middleware

Can you insert data with db and use transactions?

I realized I wasn't logging the actual error. It's saying [error: column "email" of relation "auth_user" does not exist]

You have to add the email and email_verified column to the user table


#2880 Update snapshots for autotests

How the feature works? / How did you fix the issue?
(Screenshots, videos, or GIFs, if applicable)
Check list

[ ] unit-tests written
[ ] e2e-tests written
[ ] documentation updated
[x] PR name follows the pattern #1234 – issue name
[x] branch name doesn't contain '#'
[x] PR is linked with the issue
[x] base branch (master or release/xx) is correct

Numerous tests are failing due to a design change. Their repair is necessary to capture the golden screenshot. The pull request remains open and in draft until then.

With the addition of 'offset' to the 'getBondsCoordinatesByAttributes' function, 'right-click' tests have been fixed. Updated screenshots. Updated the expected "gold" files for all tests and checked them. At the moment, 58 tests are falling. I am attaching the report after the last run.
playwright-report.zip


fix(logs): add/remove misplaced spaces and typo

Checklist
All items have to be completed before a PR is merged

[x] Adhere to Contribution Guidelines
[ ] Updates to Decision Records considered?
[ ] Updates to documentation at DX@Scale Guide considered?
[x] Tested changes?
[x] Unit Tests new and existing passing locally?

Codecov Report
Base: 60.48% // Head: 60.48% // No change to project coverage :thumbsup:

Coverage data is based on head (381aa71) compared to base (9b6dede).
 Consider uploading reports for the commit 2febbcc to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1236   +/-   ##
=======================================
  Coverage   60.48%   60.48%
=======================================
  Files          64       64
  Lines        2126     2126
  Branches      453      453
=======================================
  Hits         1286     1286
  Misses        788      788
  Partials       52       52

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Thanks @alanjaouen


Website Bug with Multiple Modals Example

The second Multiple Modals example on the website is reproducing the same issue that I am having when opening a modal and then triggering a second modal with the first modal's button. One the second modal is open the dimmer is stuck and there is not a way to close

I'm not experiencing these issues with the second example.
Can you describe the exact process you are using to reproduce the issue with that example, and what your browser version/OS is?

Yeah, Im on OSX 10.10.1 (14B25) and in both Chrome (41.0.2272.64) and Safari(8.0.2 ) when I navigate to the Modals examples and click run code on the second example for Multiple Modals the first popup shows. I click "Proceed" and then  "All Done" its closes the modal and leaves the dimmer up (see attached).
It seems to be a timing thing, when I first went back and spent more time clicking through it seemed to work, but if I click through quickly the problem arises. I am having the same problem on my project whereas I use one modal to open another and the dimmer stays put.

@jlukic
It looks like your latest updates fixed the bug, thank you for all your hard work! I am really digging Semantic-UI :)

Most likely this was related to webkit not using failSafe when animations are queued.


vcs: init at 1.13.4

Motivation for this change
Adding package Video Contact Sheet (vcs), a bash script that generates contact sheets from video files.
Things done

I've fixed all runtime binaries using wrapProgram, including coreutils and utillinux. Is that appropriate?
The script also pulls in dejavu_fonts to use as its default at runtime, increasing its closure size.
This has the same executable name, "vcs", as one provided by package vcstool. Is that a problem?

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

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

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

Result of nixpkgs-review pr 84287 1

1 package built:
- vcs


Setting up headscale-ui with Caddy on a VPS hosting headscale

Hi. I'm currently trying to get headscale-ui setup with headscale on my VPS (on Debian 11) using Caddy. I have headscale running on my VPS under the host static IP. I downloaded headscale-ui, unzipped, and placed the web folder under /var/www/html for Caddy. However, when I try to reach the server via my VPS's host IP, I get these errors in the console.

Here's my Caddyfile config:

I also tried hosting the files under /web through the Caddyfile (added /web to :80), but the files wouldn't even load in. Any suggestions? Thanks in advance.

For anyone else that may run into this issue:
I managed to solve this by following the instructions here
i had to add
handle_path /web* {   root * /var/www/html/web   file_server }
along with reverse_proxy localhost:8080 to setup a reverse proxy for headscale since it's running on that port in my VPS by default.
to my Caddyfile.


Allow custom steps / logic prior to publish

Describe Request:
Customer is publishing single orb to many organizations and using matrix to call publish job.
They thought about using pre-steps to run some envsubt on the yaml before publishing, but since pre-steps run before workspace from prior job is attached, there is nothing to work on.  Pre-step does not support special copmmands like attach_workspace so I am suggesting a parameter to publish job that is of steps type to run after checkout (#211 )/attach worksapce but before publish step.
- existing checkout or attach steps run
- custom steps parameter steps are executed
- existing publish is executed

Examples:
workflows:
  lint-pack:
    jobs:
      ...
      - orb-tools/publish:.
          matrix:
            parameters:
              org_name:
                - "org_1/resource_class"
                - "org_2/resource_class"
                - "org_3/resource_class"
              context:  [ orb-publishing ]
              pre_publish_steps:
                 - export ORG_NAME=<<matrix.org_name>>
                 - envsubst '$ORG_NAME' < ${ORB_DIR}/${ORB_FILE} > ${ORB_DIR}/${ORB_FILE}

Supporting Documentation Links:

https://circleci.com/developer/orbs/orb/circleci/orb-tools?version=12.0.0#jobs-publish onwards support a attach_workspace boolean parameter which would allow someone to set it false and then put their logic into a pre-steps block.
e.g. have a pre-steps block that does everything that the publish job normally does when attach_workspace is true and then carry out the customer-specific logic.
(Weirdly, the job also supports a checkout boolean that's declared and never used ... but that's already been reported in #211)


Show output of build_runner in debug console

https://github.com/aksharpatel47/vscode_flutter_helper/commit/15668fc99a2b3ceac2e68d48fe571af6a6cd2f1e adds the output of build_runner to OutputChannel


Update dropdown version for bib issue fix

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #213
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team
Summary by Sourcery
Update the auro-dropdown version to fix a specific issue and upgrade multiple dependencies to their latest versions for enhanced performance and security.
Bug Fixes:

Update auro-dropdown version to address a specific issue.

Enhancements:

Upgrade various dependencies to their latest versions for improved performance and security.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Temp fix of a crash caused by 0 shader address

@bjoeris not sure if this is the right fix since I am not sure if it is valid to have the addr to be 0 here. Could you verify if this is the correct fix? the problematic capture file is here:
https://drive.google.com/file/d/1-elAsbEDH2Ww4-zaPlnKLw88IQ2iyhu1/view

@bjoeris not sure if this is the right fix since I am not sure if it is valid to have the addr to be 0 here. Could you verify if this is the correct fix? the problematic capture file is here: https://drive.google.com/file/d/1-elAsbEDH2Ww4-zaPlnKLw88IQ2iyhu1/view

The traces I've tried all had UINT64_MAX for for missing shaders, but I think it's reasonably to also handle 0 as a missing shader.


simplify transactionpool by keeping only a single set of transactions

Currently we keep a set of transaction sets. Each transaction added to the txpool is added as a separate tx set. This works fine for the standard rules as all objects (transactions, output, inputs) are kept as known objects and in case such an object is spent (meaning it is known in tx pool and thus still unconfimed) than the transactions will be merged together and validated as a whole, allowing this behaviour.
As soon as you add new rules to the mix though, it doesn't play well with it, given it does not notice any ID-based conflicts and thus keeps them separate.
If instead we keep all transactions in a single set, we will at all times have a full view of the tx set and thus we will be able to handle unconfirmed objects to be spent as well as any other rules that apply to tx dependency.
We do need to guarantee that tx order is respected in terms of the dependency tree. Currently that would be guaranteed, but if we implement priority ordering in the future this would need to be respected as a master rule.

Discussion that triggered this proposal can be found at https://github.com/nbh-digital/goldchain/issues/36#issuecomment-515878182 (and the comments surrounding it).

As discusssed with the problem of the blockcreator removing the transactions from the pool, I wonder if it is the task of the blockcreator.
When the block is accepted in the consensus, the transactions are removed from the pool anyway.

I think you are right. Did a quick checkup and it does seem that we can just remove them automatically when receiving them as part of the ProcessConsensusChange callback the TxPool has implemented as part of its contract for being a ConsensusSet subscriber. Will just need to ensure that its TxPool subscribers are aware of this removal from the pool as well.

Going to keep this issue really simple. It can get out of scope very quickly, potentially breaking code without giving much benefits. Point is that there is a lot that can be improved in the txpool, but these should be separate issues.
I did already remove the purging from the code, as I do agree that this should not be done by the consensus.
But removing the txpool set or even sets of sets from the codebase might not be necessary for this issue. It could be a nice thing to do, but it's quite invasive and is not required for resolving this issue.
All we care for this issue is that we do not take care of related objects, and based on that validate the tx set on its own or within a greater context. Instead we want to validate based on everything always (everything meaning the current consensus state and tx pool state combined).

PurgeTransactionPool() method of the TransactionPool interface and its use by the BlockCreator (our miner) was added and used again since commit c047b23619d4e512a7f14b95f0f706631209bb87. Turns out that due to possible buggy behaviour when there is a lot of traffic, it does become necessary to not completely block that block creator to an indefinite halt.


Question: Install ESO via OLM (Operator Lifecycle Manager)?

Describe the solution you'd like
It would be awesome if ESO could be integrated into OLM, which is a marketplace for k8s operators, so for example ESO would be available at OperatorHub.io (by submitting the operator on both k8s and OpenShift catalogs).
What is the added value?
Helm is OK to manage simple k8s application installation, but when working with k8s operators, you need to manage operator lifecycle, upgrades, patches... OLM makes easier to manage the whole operator lifecycle at scale.
With OLM the operator installation can be done within a couple of clicks via UI, or with a few custom resources like OLM OperatorGroup and Subscription, that can be easily integrated into GitOps workflows with tools like ArgoCD on a declarative way without having to execute dynamic commands like helm install ....
Give us examples of the outcome
Most of current cloud native applications has k8s operators already on OLM (etcd, prometheus, grafana, vault, argoCD, flux, cert-manager, istio, knative...).
Some of them are even helm operators, operator-sdk framework has the operator-sdk helm version, theoretically it should easy to convert current helm charts into operators that can easily be integrated into OLM.
Observations (Constraints, Context, etc):
None.

@sevikarakose  @onselakin

As we needed to install the current external-secrets helm-chart operator via OLM, we have created a kind of new meta operator using operator-sdk, that is in charge of deploying current helm chart.
We have published it at https://github.com/3scale-ops/external-secrets-operator
Although the solution is not perfect (it could be better integrated into OLM, if for example CRDs were managed by OLM instead of being created by the operator), is the easiest way we have found to make usage of current helm-chart on a OLM way. It does not require any change on current helm-chart, making the maintenance easy by just updating the helm chart version on a Makefile, and executing an automated release pipeline with a GitHub action.

Awesome @slopezz! We've wanted to support OLM for a while (#244). Your helm-based implementation looks very promising - i particularly like the ability to customize the installation with the embedded values.yaml file.
We want to provide and maintain a package of our operator in both the community-operators and openshift space. I think it makes sense to couple the olm bundle process to the ESO release process so that every ESO release will publish a new bundle.
Do you think it makes sense to move your repo to the external-secrets org and we take care of bundling and releasing both helm and OLM artifacts?
Reminder to myself: flux2 has a different approach. They generate the bundle files with a couple of scripts which may be an alternative (though not very configurable for the user)

Sure @moolen, we are happy to transfer the repo to external-secrets org, so you could maintain both helm and OLM artifacts together (for the moment we are using our own OLM catalog managed with the repo with images on quay.io, so we can install it with OLM until you publish it on OperatorHub).
As I mentioned on my previous comment, the current implemented solution is not perfect, because the helm-chart is in charge of managing the CRDs, which is not safe as long as if you remove the OperatorConfig custom resource (so you delete the helm-char-operator managing CRDs) while there are many ClusterSecretsStore, SecretStore and ExternalSecrets deployed in the cluster, you might break all those resources with a deleted CRD.
Ideally, the CRDs management should be moved to OLM bundle files (like any other CRD), because with OLM, if you uninstall an operator, nothing happen to your current deployed custom resources (100% safe), because CRDs are not removed from the cluster.
I've seen there there is already a helm-chart var to not manage CRDs (by default installCRDs: true), and I've seen that with operator-sdk helm version, you can always override helm vars:
https://sdk.operatorframework.io/docs/building-operators/helm/reference/advanced_features/override_values/#basic-usage

So maybe it could be added in the operator watches.yaml:
# Use the 'create api' subcommand to add watches to this file.
- group: operator.external-secrets.io
  version: v1alpha1
  kind: OperatorConfig
  chart: helm-charts/external-secrets
  overrideValues:
    installCRDs: false
#+kubebuilder:scaffold:watch

And move the CRDs management to the OLM bundle files.

FYI: I experimented a bit with the flux approach to generate the ClusterServiceVersion manifest. However it's not possible to configure / apply patches to the OLM deployment. That means we can't even add annotations to service accounts or modify env vars in the deployment. This limits our users too much and i think it's a way better approach to have a operator that simply runs helm install with user-defined values.
I made some changes to the external-secrets-operator to make it possible to generate a bundle (instead of creating own images/catalog). I'll open a PR there soon.
For reference / my olm-generator branch: https://github.com/moolen/external-secrets/tree/experimental/olm-packaging/deploy/olm


[CL][Swap Router Module]: test CreatePool

Background
We should cover the following test cases:
https://github.com/osmosis-labs/osmosis/blob/0c2fa2d14a153b50d5d97525556ad828d3af6344/x/swaprouter/creator_test.go#L37-L39
Acceptance Criteria

tests added
all pass

It's blocked by #3245 . I'm willing to work on this issue after MsgCreateConcentratedPool  is implemented.

Basic CL test was added as a drive by


Cannot create instance with some ami on EC2, v0.12.2

I found it is happening for specific cases and specific host OS:

Ubuntu 14.04

aws_key=<...>
aws_access_key=<...>
vpc_id=<...>
instance_name=justsomename

ami=ami-5f709f34
docker-machine create --driver amazonec2 --amazonec2-secret-key ${aws_key} --amazonec2-instance-type t2.medium --amazonec2-ami ${ami} --amazonec2-access-key ${aws_access_key} --amazonec2-vpc-id ${vpc_id} --amazonec2-region us-east-1 ${instance_name}

Output
Running pre-create checks...
Creating machine...
(jenkins) Launching instance...
Waiting for machine to be running, this may take a few minutes...
Detecting operating system of created instance...
Waiting for SSH to be available...
Detecting the provisioner...
Provisioning with ubuntu(upstart)...
Installing Docker...
Copying certs to the local machine directory...
Copying certs to the remote machine...
Setting Docker configuration on the remote daemon...
Checking connection to Docker...
Error creating machine: Error checking the host: Error checking and/or regenerating the certs: There was an error validating certificates for host "<IP_ADDRESS>:2376": read tcp <IP_ADDRESS>:38799-><IP_ADDRESS>:2376: read: connection reset by peer
You can attempt to regenerate them using 'docker-machine regenerate-certs [name]'.
Be advised that this will trigger a Docker daemon restart which might stop running containers.

Ubuntu 16.04 (ami is different)

docker-machine rm -f ${instance_name}
ami=ami-cd0f5cb6
docker-machine create --driver amazonec2 --amazonec2-secret-key ${aws_key} --amazonec2-instance-type t2.medium --amazonec2-ami ${ami} --amazonec2-access-key ${aws_access_key} --amazonec2-vpc-id ${vpc_id} --amazonec2-region us-east-1 ${instance_name}

Output:
Running pre-create checks...
Creating machine...
(jenkins) Launching instance...
Waiting for machine to be running, this may take a few minutes...
Detecting operating system of created instance...
Waiting for SSH to be available...
Detecting the provisioner...
Provisioning with ubuntu(systemd)...
Installing Docker...
Copying certs to the local machine directory...
Copying certs to the remote machine...
Setting Docker configuration on the remote daemon...
Checking connection to Docker...
Docker is up and running!
To see how to connect your Docker Client to the Docker Engine running on this virtual machine, run: docker-machine env jenkins

v0.12.2:
docker-machine --version
docker-machine version 0.12.2, build 9371605

former ami (ami-5f709f34) used to work on older docker-machine 0.70 until start giving me errors (ssh connect retry) and I had to upgrade to the most recent one

Yes, some AMIs are not compatible with docker-machine's default provisioning because they're missing the aufs bits. Make sure you use AMIs that provide aufs, or use a different storage driver using the --engine-storage-driver option.