Add AWS::ApiGateway::GatewayResponse

Adds support for AWS::ApiGateway::GatewayResponse.
Unfortunately, there seems to currently be a bug in CloudFormation which blocks creating GatewayResponses (https://forums.aws.amazon.com/thread.jspa?messageID=801223). However, I think it's still worth adding support for creating these objects according to the spec in the docs.
It's my first PR and I'm not sure about the conventions in the project so please let me know if anything needs to be changed accordingly.

Thank you for sending in a PR and adding the test. Everything looked good to me.


fix: remove never used variables

Description
Fixed linter bugs, like as variable is assigned a value but never used
Checklist

[x] Read the contributing guidelines
[x] Commit messages follow the conventional commits spec
[x] Performed a self-review of the PR
[ ] Added automated tests
[x] Manually tested the change

LGTM, just wondering, do we need to remove this?
https://github.com/near/near-api-js/blob/dab149c54c757e75094ac256f187346d497ea926/.github/dependabot.yml#L22

LGTM, just wondering, do we need to remove this?
https://github.com/near/near-api-js/blob/dab149c54c757e75094ac256f187346d497ea926/.github/dependabot.yml#L22

Yes we do, good catch.

Thanks @BogdanovBogdan!


Discussion: Differences between Debug and Release builds

It can be a bit of a surprise when you see different behaviors between Debug and Release builds (as in the case of #1949). This is an attempt to document any existing such behavior differences in one place (and evaluate if there is a better way to achieve the same results without these differences).

Markup compiler uses reflection based provider instead of code gen to speed up debug builds. This behavior can be changed by adding false to the project file.

Release builds use .Net Native tool chain to improve performance.

Other ?

We should probably elaborate on what is different with .NET Native. Specifically we've seen ARM64 codegen issues in .NET Native and you have to sometimes use the rdinfo.xml file to customize what types are included for reflection.


[Bug Report] Slider change event not working on drag and drop

Element UI version
2.0.4
OS/Browsers version
All
Vue version
2.5.3
Reproduction Link
https://jsfiddle.net/778y6bm0/
Steps to reproduce
Drag and drop the slider
What is Expected?
New value to be set
What is actually happening?
New value is still the old value

change events will not fire if you use value instead of v-model. You need to listen to input in this case: https://jsfiddle.net/778y6bm0/1/


Allow disabling of CPUSchedulingPolicy for bbb-html5

Starting in BBB v2.2.31 bbb-html5.service now sets CPUSchedulingPolicy=fifo[1].
This causes a SETSCHEDULER error when running in an LXC container.
This PR disables this option using a systemd override when bbb_cpuschedule is false. This is also the recommended solution [2].
[1] https://github.com/bigbluebutton/bigbluebutton/releases/tag/v2.2.31
[2] https://docs.bigbluebutton.org/2.2/troubleshooting.html#bbb-html5-fails-to-start-with-a-setscheduler-error

The documentation also recommends to set Nice=-10. Is there a reason you did no set this here?

Otherwhise this PR looks good 👍🏼

The documentation also recommends to set Nice=-10. Is there a reason you did no set this here?

I think I just missed it. Fixed and rebased!


entry not found

Does this occur consistently?
Repro steps:

Action: appService.CreateWebApp
Error type: Error
Error Message: Entry not found in cache.
Version: 0.16.5
OS: win32
OS Release: 10.0.10240
Product: Visual Studio Code
Product Version: 1.45.0
Language: en

Call Stack
a.createError extension.js:1:49459
b.<anonymous> extension.js:229:174287
extension.js:229:168902extension.js:229:168902
extension.js:229:54224extension.js:229:54224
extension.js:229:52217extension.js:229:52217
extension.js:229:51839extension.js:229:51839
i.find extension.js:229:86332
extension.js:229:33268extension.js:229:33268

As a workaround you could try reloading your window or signing out and signing back in to your Azure Account using the command palette.
Unfortunately there's not much we can do in this extension, but this is being tracked upstream here: microsoft/vscode-azure-account#53


CSV Export

Use a library like https://www.npmjs.com/package/json2csv to convert JSON to CSV in the browser. This should download the current view.

I'm going to export these columns:
${Group}, Name, Organization, Stars, Market Cap, Funding, License
If we need more columns, I can add those columns later


tests: remove pytest.mark.xfail decorator from integration tests

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Removes the pytest.mark.xfail decorator as the measlinkbot is updated with the MeasurementLink version (<IP_ADDRESS>+d250) accommodating the multiplexer changes.
Why should this Pull Request be merged?
To validate the multiplexer test cases in the PR pipeline for all the upcoming PRs.
What testing has been done?
Tests should pass.

FYI. The integration tests are passing in the pipeline. 😃


tab "Following" as homepage ?

Dear Minimal Theme for Twitter,
This extension is fine, but could you propose an option to :
when I launch the url : https://twitter.com/home , currently it s arriving on "For You", and I need to click on "Following" to have only my following people.
Could you add a option to have automatically "Following" on the url https://twitter.com/home
Regards

Thanks @Hexerin :) yes it's in 5.0.2 — you can install the latest bundle manually if you want to have it right away: https://github.com/typefully/minimal-twitter/blob/main/MANUAL_INSTALLATION.MD

Thank you .
Tested ok on firefox.


App crash after join channel

Use the demo of your project, and then introduce another plugin: https://github.com/GuiminChu/flutter_nim. It is simply introduced without any modification in the code.
Run your demo. It will crash automatically after a while after startup.
=====
Flutter doctor：
[✓] Flutter (Channel stable, 2.8.0, on macOS 12.0.1 21A559 darwin-arm, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.62.3)
[✓] Connected device (3 available)
======
Environment:
iPhone XSMAX
iOS 15.1
NERtcSDK (3.9.0 & 4.5.2)
======
pubspec.yaml
environment:
sdk: ">=2.7.0 <3.0.0"
dependencies:
flutter:
sdk: flutter
nertc:
path: ../
cupertino_icons: ^0.1.3
permission_handler: ^5.0.1+1
http: ^0.12.2
shared_preferences: ^0.5.12+4
package_info: ^0.4.3+2
file_picker: ^2.1.5+1
fluttertoast: ^7.1.5
flutter_nim:
======
This is the console log before the crash：
FlutterCalled:EngineApi#joinChannel
flutter: onConnectionStateChanged#state:connecting, reason:joinChannel
[2021-12-16 21:54:54] [connect] Successful connection
[2021-12-16 21:54:54] [connect] WebSocket Connection <IP_ADDRESS>:8080 v-2 "WebSocket++/0.8.2" /?cid=1344428128847860&uid=1 101
flutter: onConnectionStateChanged#state:connected, reason:joinSucceed
noti:111reason:3 previousRoute:Speaker
flutter: onJoinChannel
FlutterCalled:DeviceManagerApi#isSpeakerphoneOn
CalledFlutter:onFrameResolutionChanged
CalledFlutter:onFirstFrameRendered
noti:111reason:3 previousRoute:Receiver
flutter: onFrameResolutionChanged: NERtcVideoValue{width: 1280, height: 720, rotation: 90}

thread #10, name = 'io.flutter.1.raster', stop reason = EXC_BREAKPOINT (code=1, subcode=0x1821aca68)
frame #0: 0x00000001821aca68 CoreFoundationCFGetTypeID + 148 CoreFoundationCFGetTypeID:
->  0x1821aca68 <+148>: brk    #0xc472
0x1821aca6c <+152>: orr    x8, x16, x8
0x1821aca70 <+156>: cbz    x8, 0x1821acae8           ; <+276>
0x1821aca74 <+160>: ubfx   x9, x9, #8, #10
Target 0: (youlian) stopped.
Lost connection to device.

Use the new version of the plug-in, it won’t crash anymore


Put explorer.html into a Jupyter notebook interface

We could start by just taking the existing code and making it work with a Jupyter notebook interface.

Fixed in acd7d1cde95b9e3ea6128f211821e7499a6cbafa


docs: fix invalid reference to chip revision

According to the official errata document for the ESP32-C3, revisions are named 0.1, 0.2 and so on (instead of 1, 2...).

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

Merged in https://github.com/espressif/esp-idf/commit/9cab62e734acc1446438e11a258cdbb756b320d5


Flaky-test: C++ ReaderTest.testReferenceLeak

ReaderTest.testReferenceLeak is flaky. It fails sporadically.
example failure
[ RUN      ] ReaderTest.testReferenceLeak
2022-03-16 10:03:01.669 INFO  [140180921047808] ClientConnection:189 | [<none> -> pulsar://localhost:6650] Create ClientConnection, timeout=10000
2022-03-16 10:03:01.669 INFO  [140180921047808] ConnectionPool:96 | Created connection for pulsar://localhost:6650
2022-03-16 10:03:01.670 INFO  [140180798908160] ClientConnection:375 | [<IP_ADDRESS>:57130 -> <IP_ADDRESS>:6650] Connected to broker
2022-03-16 10:03:01.676 INFO  [140180798908160] HandlerBase:64 | [persistent://public/default/testReferenceLeak, ] Getting connection from pool
2022-03-16 10:03:01.711 INFO  [140180798908160] ProducerImpl:189 | [persistent://public/default/testReferenceLeak, ] Created producer on broker [<IP_ADDRESS>:57130 -> <IP_ADDRESS>:6650]
2022-03-16 10:03:01.746 INFO  [140180798908160] HandlerBase:64 | [persistent://public/default/testReferenceLeak, reader-ef351ebab8, 0] Getting connection from pool
2022-03-16 10:03:01.748 INFO  [140180798908160] ConsumerImpl:224 | [persistent://public/default/testReferenceLeak, reader-ef351ebab8, 0] Created consumer on broker [<IP_ADDRESS>:57130 -> <IP_ADDRESS>:6650]
2022-03-16 10:03:01.748 INFO  [140180921047808] ReaderTest:447 | 1 consumer use count 3
2022-03-16 10:03:01.748 INFO  [140180921047808] ReaderTest:448 | 1 reader use count 0
2022-03-16 10:03:01.752 INFO  [140180921047808] ProducerImpl:686 | [persistent://public/default/testReferenceLeak, standalone-0-387] Closing producer for topic persistent://public/default/testReferenceLeak
2022-03-16 10:03:01.753 INFO  [140180798908160] ProducerImpl:729 | [persistent://public/default/testReferenceLeak, standalone-0-387] Closed producer
2022-03-16 10:03:01.753 INFO  [140180921047808] ConsumerImpl:999 | [persistent://public/default/testReferenceLeak, reader-ef351ebab8, 0] Closing consumer for topic persistent://public/default/testReferenceLeak
2022-03-16 10:03:01.753 INFO  [140180798908160] ConsumerImpl:1055 | [persistent://public/default/testReferenceLeak, reader-ef351ebab8, 0] Closed consumer 0
/pulsar/pulsar-client-cpp/tests/ReaderTest.cc:463: Failure
Value of: readerPtr.use_count()
  Actual: 0
Expected: 1
2022-03-16 10:03:01.753 INFO  [140180921047808] ProducerImpl:653 | Producer - [persistent://public/default/testReferenceLeak, standalone-0-387] , [batchMessageContainer = { BatchMessageContainer [size = 0] [bytes = 0] [maxSize = 1000] [maxBytes = 131072] [topicName = persistent://public/default/testReferenceLeak] [numberOfBatchesSent_ = 11] [averageBatchSize_ = 0.909091] }]
2022-03-16 10:03:01.754 INFO  [140180921047808] ClientConnection:1560 | [<IP_ADDRESS>:57130 -> <IP_ADDRESS>:6650] Connection closed
[  FAILED  ] ReaderTest.testReferenceLeak (86 ms)

I can reproduce it when the test was iterated for 50 times.
2022-03-24 18:40:25.090 INFO  [0x11d86e600] ClientConnection:189 | [<none> -> pulsar://localhost:6650] Create ClientConnection, timeout=10000
2022-03-24 18:40:25.090 INFO  [0x11d86e600] ConnectionPool:96 | Created connection for pulsar://localhost:6650
2022-03-24 18:40:25.091 INFO  [0x700007665000] ClientConnection:375 | [<IP_ADDRESS>:54854 -> <IP_ADDRESS>:6650] Connected to broker
2022-03-24 18:40:25.092 INFO  [0x700007665000] HandlerBase:64 | [persistent://public/default/testReferenceLeak, ] Getting connection from pool
2022-03-24 18:40:25.092 INFO  [0x700007665000] ProducerImpl:189 | [persistent://public/default/testReferenceLeak, ] Created producer on broker [<IP_ADDRESS>:54854 -> <IP_ADDRESS>:6650]
2022-03-24 18:40:25.110 INFO  [0x700007665000] HandlerBase:64 | [persistent://public/default/testReferenceLeak, reader-45b5d48620, 0] Getting connection from pool
2022-03-24 18:40:25.111 INFO  [0x700007665000] ConsumerImpl:224 | [persistent://public/default/testReferenceLeak, reader-45b5d48620, 0] Created consumer on broker [<IP_ADDRESS>:54854 -> <IP_ADDRESS>:6650]
2022-03-24 18:40:25.111 INFO  [0x11d86e600] ReaderTest:447 | 1 consumer use count 1
2022-03-24 18:40:25.111 INFO  [0x11d86e600] ReaderTest:448 | 1 reader use count 0
2022-03-24 18:40:25.116 INFO  [0x11d86e600] ProducerImpl:686 | [persistent://public/default/testReferenceLeak, standalone-0-344] Closing producer for topic persistent://public/default/testReferenceLeak
2022-03-24 18:40:25.118 INFO  [0x700007665000] ProducerImpl:729 | [persistent://public/default/testReferenceLeak, standalone-0-344] Closed producer
2022-03-24 18:40:25.118 INFO  [0x11d86e600] ConsumerImpl:999 | [persistent://public/default/testReferenceLeak, reader-45b5d48620, 0] Closing consumer for topic persistent://public/default/testReferenceLeak
2022-03-24 18:40:25.118 INFO  [0x700007665000] ConsumerImpl:1055 | [persistent://public/default/testReferenceLeak, reader-45b5d48620, 0] Closed consumer 0
/Users/xuyunze/github.com/BewareMyPower/pulsar/pulsar-client-cpp/tests/ReaderTest.cc:463: Failure
Expected equality of these values:
  1
  readerPtr.use_count()
    Which is: 0
2022-03-24 18:40:25.119 INFO  [0x11d86e600] ProducerImpl:653 | Producer - [persistent://public/default/testReferenceLeak, standalone-0-344] , [batchMessageContainer = { BatchMessageContainer [size = 0] [bytes = 0] [maxSize = 1000] [maxBytes = 131072] [topicName = persistent://public/default/testReferenceLeak] [numberOfBatchesSent_ = 11] [averageBatchSize_ = 0.909091] }]
2022-03-24 18:40:25.119 INFO  [0x11d86e600] ClientConnection:1559 | [<IP_ADDRESS>:54854 -> <IP_ADDRESS>:6650] Connection closed
2022-03-24 18:40:25.119 INFO  [0x11d86e600] ClientConnection:263 | [<IP_ADDRESS>:54854 -> <IP_ADDRESS>:6650] Destroyed connection

2022-03-24 18:40:34.076 INFO  [0x1141e3600] ClientConnection:189 | [<none> -> pulsar://localhost:6650] Create ClientConnection, timeout=10000
2022-03-24 18:40:34.076 INFO  [0x1141e3600] ConnectionPool:96 | Created connection for pulsar://localhost:6650
2022-03-24 18:40:34.076 INFO  [0x70000a907000] ClientConnection:375 | [<IP_ADDRESS>:54945 -> <IP_ADDRESS>:6650] Connected to broker
2022-03-24 18:40:34.077 INFO  [0x70000a907000] HandlerBase:64 | [persistent://public/default/testReferenceLeak, ] Getting connection from pool
2022-03-24 18:40:34.078 INFO  [0x70000a907000] ProducerImpl:189 | [persistent://public/default/testReferenceLeak, ] Created producer on broker [<IP_ADDRESS>:54945 -> <IP_ADDRESS>:6650]
2022-03-24 18:40:34.095 INFO  [0x70000a907000] HandlerBase:64 | [persistent://public/default/testReferenceLeak, reader-4b66d62984, 0] Getting connection from pool
2022-03-24 18:40:34.096 INFO  [0x70000a907000] ConsumerImpl:224 | [persistent://public/default/testReferenceLeak, reader-4b66d62984, 0] Created consumer on broker [<IP_ADDRESS>:54945 -> <IP_ADDRESS>:6650]
2022-03-24 18:40:34.097 INFO  [0x1141e3600] ReaderTest:447 | 1 consumer use count 1
2022-03-24 18:40:34.097 INFO  [0x1141e3600] ReaderTest:448 | 1 reader use count 1
2022-03-24 18:40:34.101 INFO  [0x1141e3600] ProducerImpl:686 | [persistent://public/default/testReferenceLeak, standalone-0-414] Closing producer for topic persistent://public/default/testReferenceLeak
2022-03-24 18:40:34.103 INFO  [0x70000a907000] ProducerImpl:729 | [persistent://public/default/testReferenceLeak, standalone-0-414] Closed producer
2022-03-24 18:40:34.103 INFO  [0x1141e3600] ConsumerImpl:999 | [persistent://public/default/testReferenceLeak, reader-4b66d62984, 0] Closing consumer for topic persistent://public/default/testReferenceLeak
2022-03-24 18:40:34.104 INFO  [0x70000a907000] ConsumerImpl:1055 | [persistent://public/default/testReferenceLeak, reader-4b66d62984, 0] Closed consumer 0
pulsar-client-cpp/tests/ReaderTest.cc:462: Failure
Expected equality of these values:
  1
  consumerPtr.use_count()
    Which is: 2


img tag shouldn't close

h('img').outerHTML produces  while it probably should produce .
What do you think?

Looks like this was just fixed in https://github.com/1N50MN14/html-element/commit/450252586486a1a0f0217e64faaf7f382daee13f

It's now in html-element 1.4.0.
I would bump its version in package.json but many tests are broken as it is.

run tests in hihat?
Most, if not all, tests that are broken, are related to the faux browser env (or element env).


‘zs’ may be used uninitialized in this function

warning: ‘zs’ may be used uninitialized in this function [-Wmaybe-uninitialized]
note: ‘zs’ was declared here
FN_DECIMAL xs, ys, zs;
FN_DECIMAL FastNoise::SingleValue(unsigned char offset, FN_DECIMAL x, FN_DECIMAL y, FN_DECIMAL z) const
{
    int x0 = FastFloor(x);
    int y0 = FastFloor(y);
    int z0 = FastFloor(z);
    int x1 = x0 + 1;
    int y1 = y0 + 1;
    int z1 = z0 + 1;

    FN_DECIMAL xs, ys, zs; // <-- THIS LINE GIVES WARNING
    switch (m_interp) // BECAUSE THIS SWITCH HAS NO DEFAULT
    {
    case Linear:
        xs = x - (FN_DECIMAL)x0;
        ys = y - (FN_DECIMAL)y0;
        zs = z - (FN_DECIMAL)z0;
        break;
    case Hermite:
        xs = InterpHermiteFunc(x - (FN_DECIMAL)x0);
        ys = InterpHermiteFunc(y - (FN_DECIMAL)y0);
        zs = InterpHermiteFunc(z - (FN_DECIMAL)z0);
        break;
    case Quintic:
        xs = InterpQuinticFunc(x - (FN_DECIMAL)x0);
        ys = InterpQuinticFunc(y - (FN_DECIMAL)y0);
        zs = InterpQuinticFunc(z - (FN_DECIMAL)z0);
        break;
    }

    FN_DECIMAL xf00 = Lerp(ValCoord3DFast(offset, x0, y0, z0), ValCoord3DFast(offset, x1, y0, z0), xs);
    FN_DECIMAL xf10 = Lerp(ValCoord3DFast(offset, x0, y1, z0), ValCoord3DFast(offset, x1, y1, z0), xs);
    FN_DECIMAL xf01 = Lerp(ValCoord3DFast(offset, x0, y0, z1), ValCoord3DFast(offset, x1, y0, z1), xs);
    FN_DECIMAL xf11 = Lerp(ValCoord3DFast(offset, x0, y1, z1), ValCoord3DFast(offset, x1, y1, z1), xs);

    FN_DECIMAL yf0 = Lerp(xf00, xf10, ys);
    FN_DECIMAL yf1 = Lerp(xf01, xf11, ys);

    return Lerp(yf0, yf1, zs);
}

That sounds like an error in your compiler because the switch statement covers all the possible values for m_interp, thus the default-code would be unreachable. In fact, I'm fairly sure clang will give you a warning if you do try adding a default case to it.

gcc version 9.3.0 (Arch Linux 9.3.0-1)
There are also missing // fall through comments by the way. As a result the the compiler warns about all the implicit fall through cases.
All I'm saying is that this lib would be great if it had these issues fixed. Don't take it personally. I am not saying that the creator of the lib is a bad programmer. I am just giving advice to make this lib more "multicultural" and more tolerant. But sure, feel free to ignore it. I have manually fixed these bugs in my fork of the lib anyway so it doesn't affect me any more.

I've noticed a lot of similar warnings which are a bit annoying but in no way detract from the usefulness of the library. g++ (gcc) 10.1.0 Manjaro linux54

I also used Manjaro btw but I thought it's more informative to just say Arch because Manjaro is Arch with a graphical installer, right? :D

@1Hyena yeah kinda, but manjaro also had fractionally older packages (so not at all an issue here, IIRC it's about 3 days before manjaro gets them after arch) and there's probably other minor changes but I agree, not worth worrying about tbh.


[Automation] Generate Fluent Lite from iothub#package-2020-03

[Automation] Generate Fluent Lite from iothub#package-2020-03

Here is the newly generated PR https://github.com/Azure/azure-sdk-for-java/pull/21383

I'll go ahead and close this PR then, since #21383 replaces it


Add to pin code keyboard automatic biometric scan and enable biometrics functionality

Add to pin code keyboard automatic biometric scan and enable biometrics functionality

Codecov Report

Merging #57 (7df43cc) into feature/open_source_widget_toolkit_pin (6ff1f78) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                           Coverage Diff                           @@
##           feature/open_source_widget_toolkit_pin      #57   +/-   ##
=======================================================================
  Coverage                                   65.77%   65.77%
=======================================================================
  Files                                         118      118
  Lines                                        4398     4398
=======================================================================
  Hits                                         2893     2893
  Misses                                       1505     1505

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


Cannot SSH into the same WSL 2 instance

Environment
Windows build number: 10.0.19041.0
Your Distribution version: 18.04
WSL 2

Steps to reproduce

Installed openssh-server
In /etc/ssh/sshd_config changed PubkeyAuthentication yes, ListenAddress <IP_ADDRESS>, Port 22222
Created a user for the purpose ssh-test-user. Placed the pub key in ~/.ssh/authorized_keys and the private key in ~/.ssh/id_rsa in the home of the user I am running the SSH command from
From one terminal ran /usr/sbin/sshd -d -p 22222, and from another - ssh localhost -p 22222 (Yes, I did try ssh-test-user@localhost first, but the problem does not seem to relate to the user)

WSL logs:
Sorry, I can't seem to collect the WSL logs with these instructions - the screens don't match anymore. If you need anything else, I'd be more than happy to provide.
Expected behavior
Succesful SSH connection.
Actual behavior
Got:
❯ /usr/sbin/sshd -d -p 22222
debug1: sshd version OpenSSH_7.6, OpenSSL 1.0.2n  7 Dec 2017
debug1: private host key #0: ssh-rsa SHA256:lBy7FAlvXOsLQ0WVtgvKhLkYam02th+WczjIEWaYN0k
debug1: private host key #1: ecdsa-sha2-nistp256 SHA256:WnpOfJoElUhjHmXpLpsdkztN7OETR4IjF/hzc6A2kVs
debug1: private host key #2: ssh-ed25519 SHA256:uTCvZl1d+elHU0UvSsuzGmvqFZFRTRDm5uDxw8GmcDo
debug1: rexec_argv[0]='/usr/sbin/sshd'
debug1: rexec_argv[1]='-d'
debug1: rexec_argv[2]='-p'
debug1: rexec_argv[3]='22222'
debug1: Set /proc/self/oom_score_adj from 0 to -1000
debug1: Bind to port 22222 on <IP_ADDRESS>.
Server listening on <IP_ADDRESS> port 22222.
debug1: Server will not fork when running in debugging mode.
debug1: rexec start in 4 out 4 newsock 4 pipe -1 sock 7
debug1: inetd sockets after dupping: 3, 3
debug1: Connection refused by tcp wrapper

and
❯ ssh localhost -p 22222
ssh_exchange_identification: read: Connection reset by peer

respectively.
Also

My /etc/hosts.allow and /etc/hosts.deny are currently empty. Tried putting things there, no luck
I tried disabling Windows Firewall
Tried wsl --terminate
netstat -au shows 22222 as being listened to by WSL
I tried other random ports also.

I appologise, I'm pretty sure it's just something I did wrong, but I can't understand what's the problem. I've read everything I can get my hands on, but noone seems to have the same problem (except when using port 22, which I changed).
Please help.

UPDATE: Apparently, all of a sudden I can't connect to the local MySQL server all of a sudden.
Tried removing the package, no luck yet. This is the apt log of the installation:
Start-Date: 2020-10-01  13:10:10
Commandline: apt-get install openssh-server
Install: libdevmapper1.02.1:amd64 (2:1.02.145-4.1ubuntu<IP_ADDRESS>, automatic), dmsetup:amd64 (2:1.02.145-4.1ubuntu<IP_ADDRESS>, automatic), networkd-dispatcher:amd64 (1.7-0ubuntu3.3, automatic), libkmod2:amd64 (24-1ubuntu3.5, automatic), python3-urllib3:amd64 (1.22-1ubuntu<IP_ADDRESS>, automatic), openssh-sftp-server:amd64 (1:7.6p1-4ubuntu0.3, automatic), python3-idna:amd64 (2.6-1, automatic), python3-six:amd64 (1.11.0-2, automatic), python3-requests:amd64 (2.18.4-2ubuntu0.1, automatic), systemd-sysv:amd64 (237-3ubuntu10.42, automatic), libpam-systemd:amd64 (237-3ubuntu10.42, automatic), ncurses-term:amd64 (6.1-1ubuntu1.18.04, automatic), python3-pkg-resources:amd64 (39.0.1-2, automatic), systemd:amd64 (237-3ubuntu10.42, automatic), openssh-server:amd64 (1:7.6p1-4ubuntu0.3), libnss-systemd:amd64 (237-3ubuntu10.42, automatic), libidn11:amd64 (1.33-2.1ubuntu1.2, automatic), ssh-import-id:amd64 (5.7-0ubuntu1.1, automatic), libcryptsetup12:amd64 (2:2.0.2-1ubuntu1.1, automatic), libjson-c3:amd64 (0.12.1-1.3ubuntu0.3, automatic), libip4tc0:amd64 (1.6.1-2ubuntu2, automatic), python3-certifi:amd64 (2018.1.18-2, automatic), python3-chardet:amd64 (3.0.4-1, automatic)
Upgrade: libsystemd0:amd64 (237-3ubuntu10.21, 237-3ubuntu10.42)
End-Date: 2020-10-01  13:10:17

Next:
Start-Date: 2020-10-01  17:29:38
Commandline: apt-get remove openssh-server
Remove: openssh-server:amd64 (1:7.6p1-4ubuntu0.3)
End-Date: 2020-10-01  17:29:38

Start-Date: 2020-10-01  17:37:55
Commandline: apt autoremove
Remove: python3-urllib3:amd64 (1.22-1ubuntu<IP_ADDRESS>), openssh-sftp-server:amd64 (1:7.6p1-4ubuntu0.3), python3-idna:amd64 (2.6-1), python3-six:amd64 (1.11.0-2), python3-requests:amd64 (2.18.4-2ubuntu0.1), ncurses-term:amd64 (6.1-1ubuntu1.18.04), python3-pkg-resources:amd64 (39.0.1-2), ssh-import-id:amd64 (5.7-0ubuntu1.1), python3-certifi:amd64 (2018.1.18-2), python3-chardet:amd64 (3.0.4-1)
End-Date: 2020-10-01  17:37:57

Allright, I am ashamed to admit, but it turns out that because I was missing two blank lines at the end of /etc/hosts.deny, it was denying everything. The moment I brought them back it worked. Not sure why this didn't affect apache.
I will test the SSH again, but I have the feeling it will work. I will post back so we can close this. I hope this help some other fool like me.

Yep. That was the problem, it works. Sorry about that. I hope it helps someone.


Wrapper class name

This prop will allow to add image rotation in wrapper library

+1 for this - It'd be really useful to be able to specify the wrapper class name.

any reasons not to merge this changes?

Thanks for the contribution! This is now available in v4.2.0


Difference between model_s012_final_yv and model_s012_final

Hi,
Thanks for releasing the code. May I know the difference between model_s012_final_yv and model_s012_final?

model_s012_final is used for davis 16 val, davis 17 val and davis 17 test.
model_s012_final_yv is used for youtube 19.
The only difference is that in the loss function (s012 stage), we reduce the weights of lmc by half.
Reason:
If the same model of davis is used, j&f is 0.830, which is still a superior result. But lowering lmc reduces the risk of overfitting the seen category on youtube-vos.

Thanks for the answer!


Test flakiness on "Check "result" failed"

The full error:
[FATAL] Check "result" failed. File "../src/libusb_js_proxy.cc", line 126, function "GetLibusbTransferContextChecked"

It's happening in a new (not landed yet) test that simulates the insert+connect+disconnect+remove sequence in a loop.

The crash is triggered by this assertion:
https://github.com/GoogleChromeLabs/chromeos_smart_card_connector/blob/8460ea382679779522105e8c07c3221cb819ce96/third_party/libusb/webport/src/libusb_js_proxy.cc#L126

Seems like another reincarnation of the unsafe multi-threaded handling of polling_transfer by CCID, around the same code that we already discovered to be problematic in #1078...
Basically, one thread is doing this (InterruptStop() from ccid_usb.c):
	transfer = atomic_load(&usbDevice[reader_index].polling_transfer);
	if (transfer)
	{
		int ret;

		ret = libusb_cancel_transfer(transfer);

meanwhile the other thread is doing (InterruptRead() from ccid_usb.c):
	atomic_store(&usbDevice[reader_index].polling_transfer, NULL);
	libusb_free_transfer(transfer);

So it's possible that by the time libusb_cancel_transfer() is called, the transfer pointer has already been deallocated by the other thread. The polling_transfer being an atomic variable doesn't really help, because this doesn't guarantee that the value that was read from it remains valid while it's passed to other libusb functions.


Possible bug with SparkPost coming up. (SparkPost changes in November 2018.)

Laravel Version: 5.7
PHP Version: 7.2

Description:
SparkPost just sent us the following email. I briefly examined Illuminate\Mail\Transport\SparkPostTransport and don't see anything that will break, but I just want to make sure there won't be an issue with the API changes coming up in November.
Here's what they sent us.

Dear SparkPost Customer,
We know how important it is that when you call the Transmissions API endpoint, we are able to accept your request quickly and return a response. We are continuously looking for ways to further improve the reliability of our service, provide consistently low API response times and improve the user experience. To that end, there are three changes planned that relate to how we handle errors:

Move single recipient template validation and message generation to be asynchronous.
Move all inline template syntax validation to be asynchronous, affecting both single and multi-recipient transmissions.
Deprecate and remove the extended error codes.

These changes only affect the user experience for error cases and will not change the user experience for any successful (HTTP 200 response) API requests. The changes will take effect on Nov 17, 2018 to give our customers time to prepare.
Making single recipient transmissions asynchronous
SparkPost will no longer perform synchronous template validation & message generation on single recipient transmissions. This means that the Transmissions API will no longer return a HTTP response of 4xx if there is a problem with the template syntax or substitution data. Instead, it will return a 200 response, and then a subsequent “Generation Failure” webhook event that will provide details on the failure. These events will be available via webhooks or the Message Events API/UI. This behavior for single recipient transmissions will be the same as the current behavior for multi-recipient transmissions.
This is similar to changes earlier this year in which we moved single recipient suppression checks to be asynchronous.
Making inline template syntax validation asynchronous
The second change to the Transmission API endpoint is that inline template syntax validation will no longer be performed synchronously. For both single and multi-recipient transmissions, the syntax validation will be done asynchronously, with a HTTP response of 4xx no longer returned for syntax error on inline templates. Instead, the API will return a 200 response, and then a subsequent “Generation Failure” webhook event that will provide details on the failure. It is recommended that the template preview API is used before injecting with an inline template.
Deprecating extended error codes for API Responses
In order to simplify the user experience, we are deprecating the API HTTP response with extended error codes. This will not affect the extended error codes that are returned via webhooks. These are descriptions and codes, in addition to the HTTP status and custom message, returned by some API endpoints including Transmissions, Templates, Sending Domains, Inbound Domains, Relay Webhooks, and Tracking Domains. These extra error fields add extra overhead and complexity to the client. We believe that the HTTP status code is usually sufficient with an optional error message when it adds necessary context to the client.
For example, a Transmissions API error currently looks like this:
HTTP 400
{
"errors": [
{
"description": "Unconfigured or unverified sending domain.",
"code": "7001",
"message": "Invalid domain"
}
]
}

Once the changes go into effect we will return just the HTTP status code and the optional error message. There will no longer be a “code” or “description” field returned, however, the content of the “description” field will be returned in the “message” field.
HTTP 400
{
"errors": [
{
"message": "Unconfigured or unverified sending domain." }
]
}

This change will eliminate redundant information from these API endpoint responses and make error responses consistent across all of our API endpoints.
Preparing For This Change
To prepare for these changes, we recommend that you put in place measures to begin using the Webhook or Message Event API data, if you are not already doing this. This will ensure a smooth transition. If you wish to validate a template before using it in a Transmission, you can use the template preview API endpoint or the UI. Also, if you have any code that is currently looking at the API error response for code, message, or description you will need to update those systems.
Making a Tough Decision At SparkPost, we take backwards-compatibility very seriously. We don’t make these types of changes lightly and appreciate that our customers have code in production using our API. We also take our reliability, performance, and usability very seriously and know that is one of the primary reasons why developers and enterprises choose SparkPost. We believe these updates will provide a much better experience for our customers without a major impact to existing customers.
Thank you,
Isaac Kim
Technical Product Manager

From what i understood this change in SparkPost only affects use cases when you use their email templates via API. I guess with Laravel we always define our own templates (*.blade.php) for mailables.

There doesnt seem to be any obvious breaking changes.
If anyone finds any - please submit a PR with the required fix.


Change project validate logic and the error message. rename script file

Purpose
This PR changes the project validation logic, fixes the error message, and renames the SQL script file.
Examples
Checklist

[x] Linked to an issue
[x] Updated the specification
[ ] Updated the changelog
[x] Added tests

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Danesh Kuruppu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Bookmark Support

Have all the builds on display by a bookmark-able link.
e.g. https://adoptium.net/temurin/releases/?os=win&arch=x64&package=jdk
Should show all the builds available for download, matching the signature (Windows + x64 + JDK), ordered from newest to oldest.
Without such support, having to re-select OS, architecture and package type over and over again becomes tedious and a time-waster.

duplicate of https://github.com/adoptium/website-v2/issues/373


Feature: Add Options for preview images in FileManager and bring back image popover on hover

Back in concrete5.6 we had enlarged preview images in the FileManager when hovering over images. This was very useful in projects where we managed a lot of pictures in the FileManager. It allowed us to make a clear destinctions between two somewhat similar images in the FileManager listing without having to open the detail preview.
With the limited 30x30 thumbnails we have to work with since 5.7 this kind of image management became much more tedious since the extra step of opening the full image and closing it again, just to make sure we are about to select the right one, is necessary more often than not.
This PR adds a few options that allow us to fine tune the way preview images are displayed in the file manager:
`
'file_manager' => [
    'images' => [
        /*
         * The style of preview image used in the file_manager
         *
         * @var string 'small'(default,30x30), 'large(60x60)' or 'full(size of file_manager_listing)'
         */
        'preview_image_size' => 'small',
        /*
         * Show file_manager_detail thumbnail as preview image in popover
         *
         * @var boolean
         */
        'preview_image_popover' => false
    ],
],`

I didn't want to introduce any new default tumbnail types, so the options are still rather limited. However. 'preview_image_size' => 'full' allows for full flexibility to enlarge the preview image to whatever size we want by adjusting the thumbnail size of 'file_manager_listing'  in /dashboard/system/files/thumbnails while 'preview_image_popover' brings back the hover style preview that we know from 5.6 and earlier.
What do you guys think? - Is this useful/ flexible enough to concider for the core?
I was contemplating on if/ where to put these options in the dashboard. /dashboard/system/files/image_uploading seems to be the most fitting place even though most options there are concerned with processing of uploaded images rather than options for the FileManager. - What do you think?

@haeflimi
Thank you for adding this.
If we switch the popover container and placement, we can offer an image preview in the file manager modal. Also, it appears that placement auto is not dynamically adjusting the popover position, using right appears to work a little better. There is still a little clipping in the file manager modal because of overflow set to auto.
container: '.ccm-file-manager-list',
placement: 'right',

Example:

I think it would be very useful to put the file manager preview_image_size and preview_image_popover config settings in the dashboard somewhere. Even with the preview image popover, I imagine many people will still want a larger default thumbnail, so making the setting easily accessible is important. I am not sure where to put the settings either.

@MrKarlDilkington and @Mesuva  Thanks for your feedback.
I've actually tried different configurations with placement and container attachment and found that auto was the only configuration that worked reliably even if it does appear a bit laggy when opening the popover. - I agree that the placement to the right feels the most intuitive and looks best. But i had a problem with the popover placement that i couldn't get around. I am suprised that you didn't encounter this because you 've obviously tested it too. Here's what happens on my dev. environment when i set the placement/ container as you suggest.:
When no images are cached in the Browser and the thumbnail is hovered for the first time the popover can't be placed correctly and appears as if an entry several lines down is hovered.:
https://www.useloom.com/share/6cf2dd48fcad496eb5dd5961e704b19f
On subsequent hovers it works correctly as expected.
I think the reason for that is the popover beeing initialized before the image is loaded and it's dimensions are known so the image can't be vertically centered in relation to the position where it should appear. Placing the popover auto (which seems to make it top or bottom depending on space available.) works around this problem by not having to account for the images hieght when placing the popover in the vertical axis.
The differnet container attachment @MrKarlDilkington suggested works better though, so i've addopted that. And while fiddling around a little more i've also removed the popover's width restriction so the images in the popover appear in the the dimensions that are defined for 'file_manager_detail'
If someone happens to have an idea on how to get around the vertical placement problem and make placement: right work properly. I'm all ears.

@haeflimi
I spent a while last night trying to find ways to add the popovers to the right, and each attempt failed in some way. Ultimately, to get them to position correctly to the right, you need to set an explicit height on the img element. The problem with that is at narrow screen widths, the images are not responsive.
Regarding where to add the config options in the dashboard, I think they fit on the Accessibility page.
Example:

@aembler
Are you open to having preview popups on by default, but also including the two Accessibility page options to turn the preview popups off and increase preview image size (large preview off by default)?
I've heard comments from some community members that they would prefer larger preview images. This offers something for everyone without affecting the default state.

I've Added the Options to the Dashboard as @MrKarlDilkington suggested. - But changed the default for the image popover to true as @aembler did.
I don't think we should force the popover upon everyone. - With changes like this, that in the end boil down to individual preference i think it's good to have the option to turn it off.
The clipping Problem in the Modal situation should be fixed now. For that i needed to change the overflow behaviour on the File Manager Dialog. - I hope by doing that i didn't break anything else. As far as i've tested it looks ok.
On the topic of mobile: Since the hover state isn't really a thing on mobile/ touch devices i don't think we have to do anything speciffically. - I might be wrong though.

@haeflimi
Removing overflow: visible; on div.ui-dialog .ui-dialog-content breaks modal scrolling and does not prevent popover preview clipping.

To prevent clipping, I believe we need to set the container property to an element that is "outside" or a parent of the modal. The element would need to be shared on front-end pages and dashboard pages.

@MrKarlDilkington
You are right. - Would have been to good to be true if  that worked  without breaking anything.

To prevent clipping, I believe we need to set the container property to an element that is "outside" or a parent of the modal. The element would need to be shared on front-end pages and dashboard pages.

I've tried that before. - The problem there was: At some point we lost the Styles for the popover element since all BS styles are namespaced.
Now i've undone the overflow change, Attached the popover to body and added/ included some extra css to style the popover and adjust the z-Index.
Now it seems to be working/ looking smoothly and without clipping.

@haeflimi
The popovers appear to "stutter" when you hover them on first load. They appear in one position, then switch to another, please see video.
https://www.useloom.com/share/e5a19ad9d8a14c3ca31f2476118868b1

@MrKarlDilkington
I see what you mean. On my local dev Environment this didn't really show since Images are loaded much quicker which makes the stutter bearly noticable.
The issue here is the auto placement for the popover. - it can only be positioned in the final position when the image is completely loaded. In the same way as the position: right method is unable to correctly position it in the vertical axis.
I think i've solved that now, which also makes placement: right work properly too.
I needed to deactivate the default popover hover trigger behaviour, load the image manually on hover and only trigger the popover when the image is completely loaded.

@haeflimi
I tested the latest changes and everything looks to be working as expected.

This is really nice, and it's something that's long overdue. This was a nice feature in 5.6 and I'd like to see it come back. You've done a lot of good work getting it closer.
I do have some concerns about what I'm seeing here. I'll just post them in no particular order:

We need to limit the size of the thumbnail somehow. Look what it looks like when I mouse over one of the image slider images:

I don't like how the popover arrow is actually hovering further down next to an image that isn't the one that I'm hovering over.
I think it'd be better if the image hovered below the item – right now if I accidentally hover over the thumbnail it obscures the metadata about the image i'm interested in.
I'm not sure we need all of these configuration settings. I think if we can get this working smoothly we can just default it to on and be done with it. Although I'm flexible on this last point.

Again, this is a really great start, and it looks like we're really close.

Again, this is a really great start, and it looks like we're really close.

Don't worry i haven't abandoned it. - I am still on it (when i find the time).
@aembler
I think you might not had the most recent version of the .js code after my last edit to the PR. (maybe because of the confilict on the file-manager.js. I dont really know how i am supposed to merge the grunt generated .js in case of a conflict - sorry about that.)
To address your concernes:

I've added a max-width of 75vw to the image in the popover. - This shoud not only improve things in the situation of a overly large image but also icrease usability on small screens.

This is related to the Clipping issue i had and should be resolved with my last edit. - The image/ arrow should appear in the right place. @MrKarlDilkington has tested/ alredy confirmed this too.

I haven't thought about the ability to see the metadata next to the image and it's a good point. - I've now changed the popover to use bottom placement. - There is a downside though: If the last image in the list is a large one. - It might not fit into the space between the bottom line of the File Entry and the end of the list and thuss can't be seen fully. - I don't see a easy way to have both (guaranteed visibility on the metadata using bottom placement and guaranteed full visibility of the preview image.)

I'd personally like to have the configuration options for different projects/ needs. So i'd rather not remove them.

I don't think I'm seeing the latest changes. I deleted my branch for this and pulled again from your branch, and merged the changes in, and it doesn't look any different to me. The arrow isn't in the right place, the window isn't limited in width. Is there something that may need to get added to the pull request?

Our client says file manager listing thumbnail is too small. I agree also. Let's merge this feature!

@hissy I think my requests need to be resolved before we can merge this in. See comment https://github.com/concrete5/concrete5/pull/6593#issuecomment-391492269
I pulled in updates but I don't think they actually fixed the problems. I'd be happy to be proven wrong though, or if someone can go through and make those corrects we could definitely get this into 8.5.

@hissy and @aembler
I dont really see what there is left for me to fix. - I've just re- tested this with the latest develop branch + the PR and it works nicely for me. As i understood it @MrKarlDilkington did some testing as well and it seemd to work for him too.
As i've stated in Mai: The only thing that i am not sure about is if the uglified file-manager.js merged correctly in this PR. This can be easily resolved however by running the grunt build script after merging. Something i'd assuem would be done anyway when building a new c5 release.
I've updated the PR with a freshly rebuilt Version of the File in question. But since this didn't seem to resolve your issues the last time around, i dont think it makes a difference.

Not sure what the issue was before, but I just merged this and I'm not seeing the same issues. Looks great! Thank you.

Is this functionality hidden behind a flag? I'm not seeing a preview image when hovering over images, I've upgraded an older 8.4 site to 8.5.1.


Fix RuboCop configuration

The current config is outdated, and fails with:
minitest-reporters master % bundle exec rubocop
.rubocop.yml: Metrics/LineLength has the wrong namespace - should be Layout
Error: The `Naming/UncommunicativeMethodParamName` cop has been renamed to `Naming/MethodParameterName`.
(obsolete configuration found in .rubocop.yml, please update it)
`Performance/*` has been extracted to the `rubocop-performance` gem.
(obsolete configuration found in .rubocop.yml, please update it)
`Rails/*` has been extracted to the `rubocop-rails` gem.
(obsolete configuration found in .rubocop.yml, please update it)

I've made a number of updates to make it runnable again. Once #324 is merged, this can enabled to run as part of CI.

Do you happen to know when the naming was changed in Rubocop?  What version?

Do you happen to know when the naming was changed in Rubocop? What version?

I think it should be in the changelog.

I see this is failing on some EOL Ruby versions:
https://github.com/andyw8/minitest-reporters/actions/runs/2049705204
What's the policy for supporting these?

I see this is failing on some EOL Ruby versions:
https://github.com/andyw8/minitest-reporters/actions/runs/2049705204
What's the policy for supporting these?

Ideally we want to support as many versions as possible.  Do you know what is wrong there?  Is there a way to have logs for actions runs?

I'm seeing an alert "1 workflow awaiting approval" so if you are able to approve that you'll see the full output.
On my branch I'm seeing:
      rubocop-performance was resolved to 1.0.0, which depends on
        ruby (>= 2.2.0)

But I'm confused as to where that's coming from since there's no dependency on rubocop-performance 🤔

Oh, wait, I see...

Ok, I've updated the config and it's passing with all the Ruby versions. I had to remove rubocop-performance gem since that doesn't support older Rubies.

Thank you for the contribution


Clustering transformers

PR for @kabir  aggregating 3 PRs relating to clustering transformers, namely https://github.com/wildfly/wildfly/pull/7390, https://github.com/wildfly/wildfly/pull/7290 and https://github.com/wildfly/wildfly/pull/7395.
Locally this passes all clustering tests with -Djboss.test.transformers.eap and passes mixed domain tests.

Linux with security manager Build 1299 is now running using a merge of 320a0a0718c9eec85db9f89e524e4571428e8b9a

Windows Build 1451 is now running using a merge of 320a0a0718c9eec85db9f89e524e4571428e8b9a

Linux Build 6371 is now running using a merge of 320a0a0718c9eec85db9f89e524e4571428e8b9a

Linux with security manager Build 1299 outcome was SUCCESS using a merge of 320a0a0718c9eec85db9f89e524e4571428e8b9a
Summary: Tests passed: 825, ignored: 331 Build time: 0:06:00

Windows Build 1451 outcome was SUCCESS using a merge of 320a0a0718c9eec85db9f89e524e4571428e8b9a
Summary: Tests passed: 2851, ignored: 387 Build time: 0:47:35

Linux Build 6371 outcome was SUCCESS using a merge of 320a0a0718c9eec85db9f89e524e4571428e8b9a
Summary: Tests passed: 2850, ignored: 387, muted: 1 Build time: 0:52:31

Linux Build 6373 is now running using a merge of 320a0a0718c9eec85db9f89e524e4571428e8b9a

last job ^^ is running with enabled eap transformers tests

Linux Build 6373 outcome was FAILURE using a merge of 320a0a0718c9eec85db9f89e524e4571428e8b9a
Summary: Tests failed: 2 (2 new), passed: 2866, ignored: 370 Build time: 0:49:19
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.mdb.ejb2x.MDB20TopicTestCase.testEjb20TopicMDBs: <no details avaliable>

org.jboss.as.test.integration.ejb.timerservice.mgmt.PersistentCalendarTimerManagementTestCase.testResourceExistence: <no details avaliable>

Closing, this is no longer needed, but proved that the other PRs are not causing regressions.


Completion is created only for the last prompt in the list of prompts

Hello!
Here is a sample code where the model is expected to return 4 choices for 2 prompts (2 for each prompt), according to the documentation - https://beta.openai.com/docs/api-reference/completions/create#completions/create-prompt:
import pandas as pd
import openai

openai.api_key = "YOUR_API_KEY"

prompts = [
    "example 1",
    "example 2",
]
generated = openai.Completion.create(model='davinci',
                                     prompt=prompts,
                                     max_tokens=10,
                                     best_of=4,
                                     n=2,
                                     echo=True,
                                     )
print(generated)

But get only 2 choices for last prompt (pay attention to indices, they do not start from 0):
<OpenAIObject text_completion id=cmpl-4zR7IQzUzg8ZQQAluCUkLfkJo3DmO at 0x1885f37c630> JSON: {
  "choices": [
    {
      "finish_reason": "length",
      "index": 4,
      "logprobs": null,
      "text": "example 2.10.1\" , \"subtitle\""
    },
    {
      "finish_reason": "length",
      "index": 5,
      "logprobs": null,
      "text": "example 2). 5 Add meat, cover pot with tight-"
    }
  ],
  "created":<PHONE_NUMBER>,
  "id": "cmpl-4zR7IQzUzg8ZQQAluCUkLfkJo3DmO",
  "model": "davinci",
  "object": "text_completion"
}

Similar code, but using JS lib works as expected.
Versions:
openai==0.18.1
python==3.9.7

Does anyone have any idea what went wrong? I apologize if the error is banal, but I spent quite a lot of time looking for it and failed. Thank you!

Hi @AlekseyPauls! Thanks for the issue.
Yea, that looks like a bug.  I believe the system is currently ranking all of the completions for all of the prompts together and returning the top n.

Hey @AlekseyPauls - thanks again for bringing this issue to our attention. It should be fixed now!
The index field of the returned choices should now start at 0 but be non-contiguous, this is intended behavior. For a prompt at index P, the corresponding completions in the returned choices list will have the index field containing with values from P * best_of to P * best_of + n - 1 inclusive.


Ref docs issue

Error:

##[error]Run-WithWarningFilter : : C:\hostedtoolcache\windows\Python\3.11.6\x64\Lib\site-packages\azure\ai\ml\ope
rations_batch_deployment_operations.py:docstring of
azure.ai.ml.operations._batch_deployment_operations.BatchDeploymentOperations:1: ERROR: Error in "literalinclude"
directive:
At D:\a_work\1\s\python\build.ps1:150 char:13
Run-WithWarningFilter -Command $sphinxBuildPath -Paramete ...
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
+ FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Run-WithWarningFilter
WARNING: : invalid option block.
WARNING: : System.Management.Automation.RemoteException
WARNING: : .. literalinclude:: ../samples/ml_samples_workspace.py
WARNING: :     :start-after: [START outbound_rule_begin_create]
WARNING: :     :end-before: [END outbound_rule_begin_create]
WARNING: :     :language: python
WARNING: :     :dedent: 8
WARNING: :     :caption: Create an FQDN outbound rule for a workspace with the given name,
WARNING: :     similar can be done for PrivateEndpointDestination or ServiceTagDestination.

For more details check here:

https://apidrop.visualstudio.com/Content CI/_build/results?buildId=399623&view=results

@danieljurek for notification.

@faynef - ack'ed. Thanks for noting this. We already have a PR in the works to fix this and all other sphinx issues in the ml package and we'll be subsequently turning on strict-sphinx to prevent these errors from being merged going forward.


No error message possible with SP initiated sign-in

The documentation should be updated to state that when using ADFS 2016 with Azure MFA as an IdP, if you perform SP initiated sign-in and the authenticating user is not registered with Azure MFA, it is not possible to present an error message to the user telling them to register with Azure MFA. Instead an assertion is send from ADFS to SP and an error message is displayed to the user.
It seems from my findings and from speaking with Microsoft support that displaying this error message via onload.js is only possible for IdP initiated sign-ins. I find this bizarre and it leads ultimately to a poor user experience.

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

ID: ae96cfcf-0a50-7e13-30c8-43e3730e184f
Version Independent ID: 9d8ed7b7-eec6-1b95-40b8-24a13361f66d
Content: Configure AD FS 2016 and Azure MFA
Content Source: WindowsServerDocs/identity/ad-fs/operations/configure-ad-fs-and-azure-mfa.md
Product: windows-server-threshold
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Yes, you're right !
The message is only displayed for IdP-init sign-ins. In case of the use of the SAML protocol, the assertion is sent to the app with a bad status when SP-init sign-ins.
The message should be displayed for SP-init in case of WS-Fed protocol
For the SAML app, you can try to access to the app via the IDP-init way via the URL :
https:///adfs/ls/idpinitiatedsigon.aspx?LogintoRP=

#please-close

#please-close


DMR: Replicate DMR functionality with NGINX

As an ARCHITECT
I WANT to have a version of DMR running on NGINX
SO that I can evaluate feature parity
Acceptance Criteria

[x] accept messages and custom headers at /messages endpoint, return 202 immediately
[x] redirect message according to X-Send-To header (currently only to Classifier and url hardcoded)
[x] send an error message to X-Sent-By if X-Send-To is not available
[ ] error payload base64 encoded
[ ] demo solution to team

In AC, "error payload base64 encoded" in unnecessary. Nginx will not encrypt (or ancode64 at this point) any of its messages.
Custom header to represent "Unable to send message" with initial message headers should be appropriate.

carried over to next sprint as https://github.com/buerokratt/DMR/issues/42

Delivered across 2 issues.


[BUG] vSphere cluster provisioning by cloning the already available VM take very long to provision.

Rancher Server Setup

Rancher version: v2.6-head
Installation option (Docker install/Helm Chart): Docker

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Information about the Cluster

Kubernetes version: v1.24.4
Cluster Type (Local/Downstream): Downstream - vSphere node driver

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom) Admin

Describe the bug

Cluster takes very long time (more than 45 minutes) to come up to an active state.
To Reproduce

Create a rancher server on vSphere 7 environment.
Go on to create a new vSphere node template.
Select the Clone an existing virtual machine in Creation method --> under the Instance Options.

In the Virtual machine option adjacent to the Instance options, select any virtual template.
Once, the template is created, go on to create a k8s cluster v1.24.4

Result
Notice the following error starts coming up while creating cluster and nodes gets deleted and new nodes get created.
Error creating machine: Error running "sudo apt-get update": ssh command error: command: sudo apt-get update err: exit status 255 output: Hit:1 https://download.docker.com/linux/ubuntu focal InRelease
However, if you wait for long time, cluster comes up in an active state after around 45-60 minutes.
Expected Result

Cluster should come up in an active state within average time span of 10 minutes.

/backport v2.6.9


restic and aliyun oss

What steps did you take and what happened:
i have use s3ForcePathStyle="false" when install velero
when create backup to aliyun oss though restic :
client.BucketExists: Please use virtual hosted style to access.\n\n: exit status
Environment:

Velero version (use velero version):  1.1.0
Kubernetes version (use kubectl version): 1.15.2
Kubernetes installer & version:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):centos 7.2

the error is
reate repository at s3:http://oss-cn-beijing.aliyuncs.com/build-xxx/xxx/restic/xxx failed: client.BucketExists: Please use virtual hosted style to access.\n\n: exit status 1
why restic use PathStyle ?

looks like its have not to supply virtual hosted style
return fmt.Sprintf("s3:%s/%s", strings.TrimSuffix(url, "/"), path.Join(bucket, prefix)), nil

@sunnoy I'm not totally sure if restic supports the virtual host style -- can you try running restic operations manually (outside Velero) using the virtual host style for your endpoint, and confirm whether or not it works?
ref. https://restic.readthedocs.io/en/latest/030_preparing_a_new_repo.html#amazon-s3

@sunnoy were you able to check this?


ZstdInputStream hangs for certain chunks

Hi there,
Here's a failing test:
@Test(timeout = 1000)
public void itCanStreamCompressedChunks() throws IOException {
    // The uncompressed input data for this test case
    byte[] expected = new byte[1024];

    // The decompressed output
    byte[] actual = new byte[expected.length];

    Random random = new SecureRandom();
    random.setSeed(0); // -- so that we get same random bytes each time.. If that makes sense.
    random.nextBytes(expected);

    PipedInputStream pin = new PipedInputStream();
    PipedOutputStream pout = new PipedOutputStream(pin);
    ZstdInputStream zin = new ZstdInputStream(pin);
    ZstdOutputStream zout = new ZstdOutputStream(pout);

    zin.setContinuous(true);
    zout.setCloseFrameOnFlush(false);
    zout.setLevel(3);

    zout.write(expected);
    zout.flush();

    int totalRead = 0;
    while (totalRead < actual.length) {
        totalRead +=
                zin.read(actual, totalRead, actual.length - totalRead);
    }

    assertThat(actual, equalTo(expected));
}

Using compile 'com.github.luben:zstd-jni:1.4.4-7'

Yes, that is strange:

the setContinuous(true) configures the ZstdInputStream to not assume the stream is finished even if underlying input stream tells it's finished.

setCloseFrameOnFlush(false) configures the output reader to not close the frame on flush but it looks that it also does not fully flushes the buffers.

I will investigate but it points to output stream flush issue.

Actually it's not problem with Zstd Input/Output streams. PipedInputStream has internal buffer of 1024 by default. So if the connected PipedOutputStream tries to write more data it will block until some of the data is consumed on the other end. In your example the data is compressed to more than 1024 bytes (as it's random) so it blocks on zout.flush() as there is nothing reading it on the other end. You can reproduce it even without Zstd in the mix, just pick buffer bigger than 1024 bytes.
Your example is solvable by setting bigger buffer, e.g. new PipedInputStream(1034). If you want to to be able to handle all output sizes produced by Zstd, make it 128 KB

You're right, sorry about that.
I'll form a new test that represents my problem correctly (I was not using
PipedInputStream when I discovered the issue).
On Mon, Mar 9, 2020, 7:16 PM luben karavelov<EMAIL_ADDRESS>wrote:

Actually it's not problem with Zstd Input/Output streams. PipedInputStream
has internal buffer of 1024 by default. So if the connected
PipedOutputStream tries to write more data it will block until some of
the data is consumed on the other end. In your example the data is
compressed to more than 1024 bytes (as it's random) so it blocks on
zout.flush() as there is nothing reading it on the other end. You can
reproduce it even without Zstd in the mix, just pick buffer bigger than
1024 bytes.
Your example is solvable by setting bigger buffer, e.g. new
PipedInputStream(1034). If you want to to be able to handle all output
sizes produced by Zstd, make it 128 KB
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/luben/zstd-jni/issues/116?email_source=notifications&email_token=AA4DXUQVZ5XTM3KRYL5ZZL3RGUP5BA5CNFSM4LEL4672YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOIFQ3Q#issuecomment-596662382,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4DXUQO7RVVKTA2WZ5LBKDRGUP5BANCNFSM4LEL467Q
.

Also, is 128KB the absolute output limit? What should I expect for inputs
that produce more than 128KB of compressed data? (this might be the cause
of what I'm experiencing).
On Mon, Mar 9, 2020, 7:16 PM luben karavelov<EMAIL_ADDRESS>wrote:

Closed #116 https://github.com/luben/zstd-jni/issues/116.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/luben/zstd-jni/issues/116?email_source=notifications&email_token=AA4DXURX5XABBRS2MEB3TXDRGUP7XA5CNFSM4LEL4672YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOXF3X3MI#event-3111615921,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4DXUVB7RKDAEP3DPKPL3TRGUP7XANCNFSM4LEL467Q
.

Hmm, actually you are right, 128 KB is not the absolute maximum. It's the internal size of the Zstd segment.
BTW, why do you need setContinuous(true) - this is solving very narrow use case (tail files that are not finished) but has high chance of deadlocks as it's up to you to decide if the file is actually finished or not.

Sorry for wasting your time, looks like this works well (even with very big inputs) in my new test...


reorganize query planning and execution

Moving query planning and execution to its own directory (name is lengthy - open to shorter suggestions!). Reorganize make_query_plan so that public methods live in the Public API section.
Part of the refactor in #1008

You can run typing_copilot init --overwrite to bypass the mypy and typing copilot CI blocks, otherwise they'll want you to type-hint all the code you moved, which is getting significantly refactored anyway in an immediate-future PR.


Test35 ironpython

Support test 35 in ironpython

Looks like iron python failed in #463 in https://github.com/pyansys/PyAEDT/runs/3622763506.  @maxcapodi78, any idea why these errors aren't being reported in the action?
Sorry for the approval, saw the green checkmark didn't realize that test failures wouldn't be displayed.  We need to fix that.

Looks like iron python failed in #463 in https://github.com/pyansys/PyAEDT/runs/3622763506. @maxcapodi78, any idea why these errors aren't being reported in the action?
Sorry for the approval, saw the green checkmark didn't realize that test failures wouldn't be displayed. We need to fix that.

@akaszynski  Massimo was going to take care of that in his next push (and I also fixed it here)

@akaszynski the ironphyton unit test cicd is fault tolerant because ironphyton is more unstable than cpython. furthermore one modification made in previous PR made failing all tests. I've fixed that and now tests looks much better

@akaszynski the ironphyton unit test cicd is fault tolerant because ironphyton is more unstable than cpython. furthermore one modification made in previous PR made failing all tests. I've fixed that and now tests looks much better

@maxcapodi78 It should be possible give an indicator about failing tests in PRs without automatically blocking PRs because of it

that would be great. let's talk tomorrow morning


power request on the same output type is loaded only once

What happened?
When try requesting two power models with the same output type, the only first success one will be applied.
set NODE_TOTAL_ESTIMATOR to true.
set NODE_TOTAL_INIT_URL to https://raw.githubusercontent.com/sunya-ch/kepler-model-db/css-117/models/v0.6/css-117/core96_v0.6/acpi/AbsPower/BPFIRQ/GradientBoostingRegressorTrainer_1.zip.
set NODE_COMPONENTS_ESTIMATOR to true.
set NODE_COMPONENTS_INIT_URL to https://raw.githubusercontent.com/sunya-ch/kepler-model-db/css-117/models/v0.6/css-117/core96_v0.6/rapl/AbsPower/BPFIRQ/KNeighborsRegressorTrainer_1.zip.
clean socket
get archived model
get init url https://raw.githubusercontent.com/sunya-ch/kepler-model-db/css-117/models/v0.6/css-117/core96_v0.6/acpi/AbsPower/BPFIRQ/GradientBoostingRegressorTrainer_1.zip
try getting archieved model from URL: https://raw.githubusercontent.com/sunya-ch/kepler-model-db/css-117/models/v0.6/css-117/core96_v0.6/acpi/AbsPower/BPFIRQ/GradientBoostingRegressorTrainer_1.zip for AbsPower
<Response [200]>
load model from config:  /mnt/download/acpi/AbsPower

What did you expect to happen?
It is a bug that must be fixed.
How can we reproduce it (as minimally and precisely as possible)?
Set initURL for NODE_TOTAL and NODE_COMPONENTS.
Anything else we need to know?
No response
Kepler image tag

Deployment

[ ] Model server
[ ] Estimator
[ ] Online trainer
[ ] Offline trainer
[ ] Profiler

Kepler model server image tag if deployed

Kepler estimator image tag if deployed

Kepler online trainer image tag if deployed

Kepler offline trainer image tag if deployed

Kepler profiler image tag if deployed

Kubernetes version

$ kubectl version
# paste output here

Install tools

Kepler deployment config

For on kubernetes:
$ KEPLER_NAMESPACE=kepler

# provide kepler configmap
$ kubectl get configmap kepler-cfm -n ${KEPLER_NAMESPACE}
# paste output here

# provide kepler model server configmap if Kepler Model Server is deployed
$ kubectl get configmap kepler-model-server-cfm -n ${KEPLER_NAMESPACE}
# paste output here

# provide kepler deployment description
$ kubectl describe deployment kepler-exporter -n ${KEPLER_NAMESPACE}

For standalone:
put your Kepler command argument here

Fixed by https://github.com/sustainable-computing-io/kepler-model-server/pull/171


CollectionView at second load shows empty

When I load the first time the view it works perfectly.

The problem appears when i go out of the view and then returning to the calendar view, the header with the months and days appears correctly but the cells with the days appears blank until I interact with it. Then it works correctly.

I set up the calendar view in the viewDidLoad
self.calendarCollectionView.visibleDates {             [unowned self] (visibleDates: DateSegmentInfo) in             self.setUpCalendarView(from: visibleDates)         }
self.calendarCollectionView.scrollToDate(Date())
I don't know if the error is from the collection view, but the other collectionViews I'm using in the app works correctly.
Thanks in advance for the help.

do you have that sample project zipped?
If yes, then can you drag it into this chat box?

Here you go!
Thank you.
sendCalendart.zip

Any updates on this, I'm experiencing the same issue

will have an update later today.

Thanks for the quick response @patchthecode

@patchthecode Not sure if that helps but this is only happening on the iPhone X, XR, XS, XSMax

For me it happens on an iPhone 6s if it helps in something

@patchthecode how did you go on Friday ?

It seems like It has something to do with the scrollToHeaderForDate() function. If I comment this line of code the calendar view displays correctly @patchthecode

I am not sure if the issue you are experiencing is the same is the original poster.
I know how to avoid the issue from happening, I am just trying to figure out why it is happening.

scrollToHeaderForDate()

did you mean scrollToDate()? I need to use that to change to it to the current month

@damiandara can you create a new issue, and upload your sample app just like user @eliomartinezb did?
I am still looking at his issue. But i believe yours may be different.

@eliomartinezb
Hey, i have made a fix.
Can you check master branch and see if it was fixed? You can find the instructions here.

@damiandara follow the instructions as well to test master branch to see if your issue was resolved.

@patchthecode hi there, thanks for the info, any way you could point me to the commit that fixed this issue?

This will be hard to find.
Are you experiencing this issue on version 7.1.7 ?

@patchthecode appreciate the quick response, thank you.

@patchthecode Is there any chance you can add a release tag as a patch for 7.1.x that includes 4fd1c04? This would help my project out a ton 😄


Laravel 7.6 - Unable to prepare route [api/user] for serialization. Uses Closure.

Why do I get the error below with a default install of Laravel 7.6 ?
$ laravel new test-site
...
$ cd test-site
$ touch database/db.sqlite
$ vim .env
DB_CONNECTION=sqlite
DB_DATABASE=/Users/me/test-site/database/db.sqlite
$ php artisan optimize
Configuration cache cleared!
Configuration cached successfully!
Route cache cleared!

   LogicException

  Unable to prepare route [api/user] for serialization. Uses Closure.

  at vendor/laravel/framework/src/Illuminate/Routing/Route.php:1081
    1077|      */
    1078|     public function prepareForSerialization()
    1079|     {
    1080|         if ($this->action['uses'] instanceof Closure) {
  > 1081|             throw new LogicException("Unable to prepare route [{$this->uri}] for serialization. Uses Closure.");
    1082|         }
    1083|
    1084|         $this->compileRoute();
    1085|

      +27 vendor frames
  28  artisan:37
      Illuminate\Foundation\Console\Kernel::handle(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))

Github issues are only for bug reporting. Please close this issue and search in the forums.  This is a very common error, with a very simple solution.
Thanks!


eval.py: error: unrecognized arguments: False

Hello!
Running code:

python train.py --uspto_50k dataset --use_rxn_class False
I get
eval.py : error: unrecognized arguments: False
Tell me what could be the matter?
Thank you.

try：python eval.py --dataset uspto_50k --experiments 27-06-2022--10-27-22

Hi, I am having the same issue, but with train.py. If I remove False as suggested above, I get other errors. How could I fix this?


Remove statics so users can use multiple SkittleLayouts

Hey did a release v2.1.1 with your changes thanks for the help


Reject more syntactically invalid Python programs

Summary
This commit adds some additional error checking to the parser such that assignments that are invalid syntax are rejected. This covers the obvious cases like 5 = 3 and some not so obvious cases like x + y = 42.
This does add an additional recursive call to the parser for the cases handling assignments. I had initially been concerned about doing this, but set_context is already doing recursion during assignments, so I didn't feel as though this was changing any fundamental performance characteristics of the parser. (Also, in practice, I would expect any such recursion here to be quite shallow since the recursion is done on the target of an assignment. Such things are rarely nested much in practice.)
Fixes #6895
Test Plan
I've added unit tests covering every case that is detected as invalid on an Expr.

@charliermarsh Can you check out the two commits I've just pushed? In particular, the stricter parser caused two tests to fail. I think my changes to those tests are correct, but I'd like to be cautious here!

Taking a closer look at those changes...

@BurntSushi - Those both look correct to me. I actually don't fully understand why app_label=\"core\", model=\"user\"", was previously marked as valid code, what was the "target" and the "value"?

was previously marked as valid code, what was the "target" and the "value"?

For x="a", y="b", it looks like it parsed as two targets, x and ("a", y) with a single value of b. It's not totally clear to me how it gets that particular parse. That first equals sign, I'd imagine, would inhibit picking up the second target.

Ahh ok, so it's like x = ("a", y) = "b". Makes sense-ish.

Those ecosystem changes look like strict improvements to me.


Remove csv dependency for mobile support

https://github.com/obsidianmd/obsidian-releases/pull/1204#issuecomment-1276549819

"csv": "^6.2.0"
Did you test that this works on mobile?
From what I can see this package will crash Obsidian on mobile as it depends on NodeJS API's.

This is resolved by PR #12


drill alerts from more than one alertmanager?

we have alertmanager instance inside each of our environments, if unsee can get data from all of them and display them together, that will be great.

Hmmm, I guess support for multiple instances will need to be added to fix #37, so I think it's reasonable to just allow passing multiple instances, update those concurrently and tag alerts with alertmanager instance name + add a filter for it.
For HA we should probably de-duplicate alerts, so maybe alerts should be tagged with list of instances reporting same alert.
Should be the next big feature to work on, I'll start working on it once I have some time. Thanks!

Similar use case here, multiple alertmanagers on different environments and we would like to have a single dashboard where we could see at a glance all the alerts that have triggered in all the environments.
It would be amazing to be able to use unsee for this job.

Wanted to second OP and vincepii — we also need data to be sourced from multiple alertmanagers.

Now that everyone mentioned it I really want to have that too ;)
I also need to do some refactoring around the code that fetches data from Alertmanager API, so that's a good opportunity to do that.

I looked into it a bit and even got some code written, things that need to be changed:

configuration syntax - need to support multiple URIs, should allow naming AM instances, if possible configuring timeouts, having a config file starts to be a good idea
silence and alert ids/fingerprints need to contain AM instance
silence API endpoint URI (used by silence form) needs to be instance aware, probably best if each alert was tagged with instane: [am1, am2], name -> uri mapping provided in the json reponse and form would send silences to all instance where alert is spotted (or even allow use to select instances where it should be silenced, probably the best)
internal metrics will need to have another dimension with the instance name
upstream errors are now per instance, not global
error handling needs to be refactored - it's no longer binary, one instance might be down but other up, so instead of fullscreen error we only need a top bar warning if it's only partial issue (fullscreen only if all am instances are down)

All of that on top of the obvious changes, so a bit of work and will probably take me a week or two.

Got some code in multi-upstream branch, it's working (as in should collect alerts from multiple Alertmanagers and deduplicate them), but it's not done, some features are missing, need to write new tests and clean up that branch. But it's usable for testing, ALERTMANAGER_URI now accepts multiple values separated by spaces, format is "name:uri name2:uri2", e.g. ALERTMANAGER_URI="prod:http://localhost staging:http://foo.bar"
It should be mergeable in a week or so, there's UI work needed which will require a bit of time, I don't want to overload it with yet another label, so will add some other UI elements to indicate which Alertmanager instance it was collected from (color bar on the side or something equally non-intrusive, but that must still allow filtering actions).

Thanks for working on this! I'd like to have it too

But it's usable for testing, ALERTMANAGER_URI now accepts multiple values separated by spaces, format is "name:uri name2:uri2", e.g. ALERTMANAGER_URI="prod:http://localhost staging:http://foo.bar"

Random nitpick: I'd use comma-separated list of values for alertmanager_uri, it would match what prometheus expects and it is easier to quote e.g. in the shell :D

All multi-value env options are space separated, so we would have to change everything for consistency. I'd rather keep it as is right now.

@prymitive ah, if space separated is already the rule then yes :+1:

What's on multi-upstream is almost feature complete, a filter support is missing and some UI tweaks are needed, but it should be usable. Long weekend might allow me to finish it, or maybe will make me lazy, no promises.

I think I'm done and all pieces are in place, PR awaits a review now.

PR merged, this will be in the next release, which will be tagged after some testing (and likely some fixes), a week or two will likely be needed.

unsee 0.7.0 was jut tagged and it includes those changes, see https://github.com/cloudflare/unsee/releases/tag/v0.7.0


Missing information from Monarch lite

In the alpha medikanren instance, that pulls in monarch lite, they are lacking phenotypes for many types of diabetes, including for example MODY11
But we definitely have these:
https://monarchinitiative.org/disease/MONDO:0013242#phenotypes
So either

bug in making monarch lite @kshefchek
bug in kgx that filters this @lhannest
bug in minikaner @gregr @webyrd

Running the Cypher,
MATCH (s:disease {id: "MONDO:0013242"})-[p]-(o) RETURN s,p,o

at http://scigraph.ncats.io yields:

Looks like the disease to phenotype associations are incomplete in Monarch Lite.

Correct, looks like it's only gene-disease, gene-phenotype, gene-gene - to allow us to test graph merging: https://github.com/monarch-initiative/monarch-cypher-queries/tree/master/src/main/cypher/kg-transform


LoadImages.hpp file

The header containing two functions is included in mian.cpp but not used there.
One of the functions is used in SideBarMenu.cpp. It sounds the other hasn't any usage. Why not removing that? (ofc, I've not loaded images with Dear ImGui, hence can't be quite sure)
What if the header is included in other TUs too? Then you'll get a symbol redefinition linker error most supposedly.

The header is incorporated into the main.cpp to prevent a redefinition linker error.
The first function is designed for loading images while the program is rendering, whereas the second function is intended for loading images before rendering. Currently, the first function is not in use, but its necessity for future use is uncertain.

1- It doesn't work that way, I suppose.
There should certainly be only one definition for any symbol, here your function, otherwise the linker confuses which one to choose. #include is also merely copy-paste, nothing that more. So including that header in main.cpp and using its function(s) in another translation unit then including that translation unit in main.cpp doesn't prevent any symbol redefinition error, but simply creates a couple of redundant complexities, to me.
When you create a header, you should consider it being potentially included in more than one TUs, therefore the symbols there ought to normally either have the static or inline specifier to prevent the redefection error. I prefer the latter since only one of the repetitious definitions will be picked up making the executable smaller in size.
Likely something like declaring a .hpp file with that header included and SetSideBarMenu function declared in it (defined in a .cpp file) would be better. That way you'd remove worries about the error and wouldn't include a .cpp file either.
2- The required function could be moved into where it's needed leaving out the excessive one.

Do you mean if i include the LoadImages.hpp in the main.cpp and use it in two different files like SideBarMenu.cpp and Devices.cpp this will conclude in an linker error ?

If you want to use stuff in that header in a TU, the header should be included. So the code will be copied into it. If used in more than one TU, more than one copy-paste will be done. Then you should get a linker error because more than one definition exist. To solve this, the static (or more preferably inline) specifier is normally required for the function(s).

https://godbolt.org/z/rvjdKW1nr
[100%] Linking CXX executable main
/opt/compiler-explorer/gcc-13.2.0/bin/../lib/gcc/x86_64-linux-gnu/13.2.0/../../../../x86_64-linux-gnu/bin/ld: CMakeFiles/main.dir/main.cpp.o: in function foo()': main.cpp:4: multiple definition of foo()'; CMakeFiles/main.dir/source.cpp.o:/app/source.cpp:4: first defined here

As well as, in Dear ImGui Wiki: Image Loading and Displaying Examples (https://github.com/ocornut/imgui/wiki/Image-Loading-and-Displaying-Examples#example-for-opengl-users) only stb_image.h is used seemingly. Where is stb_image_write.h applied in the code, please? Commenting out these two lines makes no sensible difference on my machine:

//#define STB_IMAGE_WRITE_IMPLEMENTATION
//#include "../stb_image/stb_image_write.h"

LoadTextureFromFile is the function to load image files and mentioned throughout that whole page as far as I see. Why did you go for LoadTextureFromHeader, and, where did you get this from, please (although their codes resemble each other)?

As well as, in Dear ImGui Wiki: Image Loading and Displaying Examples (https://github.com/ocornut/imgui/wiki/Image-Loading-and-Displaying-Examples#example-for-opengl-users) only stb_image.h is used seemingly. Where is stb_image_write.h applied in the code, please? Commenting out these two lines makes no sensible difference on my machine:

//#define STB_IMAGE_WRITE_IMPLEMENTATION
//#include "../stb_image/stb_image_write.h"

LoadTextureFromFile is the function to load image files and mentioned throughout that whole page as far as I see. Why did you go for LoadTextureFromHeader, and, where did you get this from, please (although their codes resemble each other)?

I included stb_image_write.h because at the point where i used the LoadTextureFromFile method it was needed. If it does not cause an error when deleted it can be deleted i suppose.
 Using LoadTexturefromFile the images where not displayed in the CI version because they where not found in the right path.

If you want to use stuff in that header in a TU, the header should be included. So the code will be copied into it. If used in more than one TU, more than one copy-paste will be done. Then you should get a linker error because more than one definition exist. To solve this, the static (or more preferably inline) specifier is normally required for the function(s).
https://godbolt.org/z/rvjdKW1nr [100%] Linking CXX executable main /opt/compiler-explorer/gcc-13.2.0/bin/../lib/gcc/x86_64-linux-gnu/13.2.0/../../../../x86_64-linux-gnu/bin/ld: CMakeFiles/main.dir/main.cpp.o: in function foo()': main.cpp:4: multiple definition of foo()'; CMakeFiles/main.dir/source.cpp.o:/app/source.cpp:4: first defined here

Why would you include the LoadImage.hpp twice in two different cpp files ? If it is included in the main.cpp it works in the SideBarMenu.cpp and every other file that is included afterwards. Do you mean changing the LoadImagefromHeader function to static?

Why would you include the LoadImage.hpp twice in two different cpp files

As mentioned earlier, a header is expected to be included in TUs. If you want to use it only in one single TU why did you create the header in the first place?

Do you mean changing the LoadImagefromHeader function to static?

Read above posts again, please.

I included stb_image_write.h because at the point where i used the LoadTextureFromFile method it was needed.

It's not required.

the images only have to be loaded once and not every frame

You're calling LoadTextureFromHeader on every frame.

Using LoadTexturefromFile the images where not displayed in the CI version because they where not found in the right path.

The CI version is not that important, supposedly. I don't know of any program sent out as a standalone executable file. Programs are sent out for users (or market) commonly as a pack comprising all dependencies needed for the executable to run correctly, such as images, sounds, .gif files, or even videos.
The stb_image.h library looks pretty much like our other external library imfilebrowser.h that we have a link to it in our repo.
Loading images using that library is not that complicated but quite easy and straightforward. It's not needed to load the file(s) on each frame either. Actually only the Dear ImGui widget ImGui::Image() is in the main loop, just like all other widgets.
Here is a resizable image in our project:

Why would you include the LoadImage.hpp twice in two different cpp files

As mentioned earlier, a header is expected to be included in TUs. If you want to use it only in one single TU why did you create the header in the first place?
If the header is included in the main.cpp it works. If this is not c++ standard we can change the function to static or inline specifier and load the .hpp in the files where it is needed.

the images only have to be loaded once and not every frame

You're calling LoadTextureFromHeader on every frame.
In the code there is a boolean "loaded_png", if the picture is loaded once this boolean is set to true, so the LoadTextureFromHeader function is only called once.

Using LoadTexturefromFile the images where not displayed in the CI version because they where not found in the right path.

The CI version is not that important, supposedly. I don't know of any program sent out as a standalone executable file. Programs are sent out for users (or market) commonly as a pack comprising all dependencies needed for the executable to run correctly, such as images, sounds, .gif files, or even videos.
Right now the CI version is tested by users, not a package version. If we want them to see the pictures we need to include the pictures via a .h fiile.
The stb_image.h library looks pretty much like our other external library imfilebrowser.h that we have a link to it in our repo. Loading images using that library is not that complicated but quite easy and straightforward. It's not needed to load the file(s) on each frame either. Actually only the Dear ImGui widget ImGui::Image() is in the main loop, just like all other widgets.
Here is a resizable image in our project:

Can you provide an example of the code you used for this?

That is exactly what i meant with c++ standard.

No. Standard C++ differs from and has nothing to do with that, usually. It's called "good code" and refers not specifically to C++ itself but all kind of programming.

Okay i get what you mean. It should check if ret is false not NULL. It is set to false if the image_data == NULL.

You could use this instead, I guess:
for (int w = 0; w < size; w++)
    if (!loaded_png[w]) {
      if (LoadTextureFromHeader(imagesNames[w], imagesLen[w],
                                &my_image_texture[w], &my_image_width[w],
                                &my_image_height[w]))
        loaded_png[w] = true;
      else
      {
        fmt::println("Error Loading Png.");
        loaded_png[w] = false;
      }
   }

static bool ret[size]; looks unnecessary too.

For the second part pls provide a code example. It is nice to know that it only takes a few lines of code but i would like to know what the exact code is.

Loading images is done more or less the exact same way as suggested in the Dear ImGui Docs.
The size used for ImGui::Image should be a fraction of main window size to let the logo fit its size with it.
I also used two ImGui::BeginChilds to have the main window in two sides.
This is part of the code (still under development):
  int logo_width{}, srch_dvc_width{};
  int logo_height{}, srch_dvc_height {};
  GLuint logo_texture{}, srch_dvc_texture{};

  auto render = [&]() {
    load_settings(config);
    SetupImGuiStyle(false, 0.99f);
    auto windowSize = ImGui::GetIO().DisplaySize;

    ImGui::Begin("OmniScopev2 Data Capture Tool", nullptr,
                     ImGuiWindowFlags_NoCollapse |
                     ImGuiWindowFlags_NoResize | ImGuiWindowFlags_NoMove);

     ImGui::BeginChild("Left Side",
     ImVec2(windowSize.x * 0.17f, windowSize.y), ImGuiChildFlags_AutoResizeX |
            ImGuiChildFlags_AutoResizeY | ImGuiChildFlags_AlwaysAutoResize);

     ImGui::Image((void*)(intptr_t)logo_texture, ImVec2(windowSize.x * 0.2f, windowSize.y * 0.2f));

     // Start only if devices are available, otherwise search for devices
      if (!sampler.has_value())
        if(ImGui::ImageButton("##search for dvc",
                              (void*)(intptr_t)srch_dvc_texture,
                              ImVec2(srch_dvc_width, srch_dvc_height))) {
          devices.clear();
          deviceManager.clearDevices();
          initDevices();
        }
   // TODO: rest of menu

Another option to access the .png files is to put their folder next to the .config files folders on the server so that that folder is also downloaded along with the .config folders, at the program start up.

You could use this instead, I guess:
for (int w = 0; w < size; w++)
    if (!loaded_png[w]) {
      if (LoadTextureFromHeader(imagesNames[w], imagesLen[w],
                                &my_image_texture[w], &my_image_width[w],
                                &my_image_height[w]))
        loaded_png[w] = true;
      else
      {
        fmt::println("Error Loading Png.");
        loaded_png[w] = false;
      }
   }

static bool ret[size]; looks unnecessary too.

This really seem to be a better version. I will change the code.

For the second part pls provide a code example. It is nice to know that it only takes a few lines of code but i would like to know what the exact code is.

Loading images is done more or less the exact same way as suggested in the Dear ImGui Docs. The size used for ImGui::Image should be a fraction of main window size to let the logo fit its size with it. I also used two ImGui::BeginChilds to have the main window in two sides.
This is part of the code (still under development):
  int logo_width{}, srch_dvc_width{};
  int logo_height{}, srch_dvc_height {};
  GLuint logo_texture{}, srch_dvc_texture{};

  auto render = [&]() {
    load_settings(config);
    SetupImGuiStyle(false, 0.99f);
    auto windowSize = ImGui::GetIO().DisplaySize;

    ImGui::Begin("OmniScopev2 Data Capture Tool", nullptr,
                     ImGuiWindowFlags_NoCollapse |
                     ImGuiWindowFlags_NoResize | ImGuiWindowFlags_NoMove);

     ImGui::BeginChild("Left Side",
     ImVec2(windowSize.x * 0.17f, windowSize.y), ImGuiChildFlags_AutoResizeX |
            ImGuiChildFlags_AutoResizeY | ImGuiChildFlags_AlwaysAutoResize);

     ImGui::Image((void*)(intptr_t)logo_texture, ImVec2(windowSize.x * 0.2f, windowSize.y * 0.2f));

     // Start only if devices are available, otherwise search for devices
      if (!sampler.has_value())
        if(ImGui::ImageButton("##search for dvc",
                              (void*)(intptr_t)srch_dvc_texture,
                              ImVec2(srch_dvc_width, srch_dvc_height))) {
          devices.clear();
          deviceManager.clearDevices();
          initDevices();
        }
   // TODO: rest of menu

Did you need to use the LoadImagesFromHeader or another load function before the images a rendered? Or was ImGui::Image() itself enough ?

LoadTextureFromFile is used to load the images.
Not sure if I got the rest of your question properly, but loading the images, as mentioned previously, is done only once and is outside the main loop (render).


Add support for Cloudflare R2 storage + github action

In CTI Butler we store CVE & CPE records.
b/c hundreds are added and updated each day, this means that someone needs to be running this script daily.
We may as well bring the whole flow into this logic using github actions
It will work something like this
### 0. r2 data structure

nvd-daily-jobs

cve

YYYY-MM

bundle.json
bundle.json

YYYY-MM

bundle.json
bundle.json

cpe

YYYY-MM
bundle.json
bundle.json
YYYY-MM

bundle.json
bundle.json

nvd_cve_daily
nvd_cpe_daily
1. run cpe/cve backfill daily
05:00 UTC each day two jobs will be run using GH actions
python3 cxe2stix_helper.py \
	--run_cve2stix \
	--last_modified_earliest YYYY-MM-DDT00:00:00 (yesterday) \
	--last_modified_latest YYYY-MM-DDT23:59:59 \
	--file_time_range 1d

This will create files like...
cve-bundle-2023_01_01-2023_01_01.json
python3 cxe2stix_helper.py \
	--run_cpe2stix \
	--last_modified_earliest YYYY-MM-DDT00:00:00 (yesterday) \
	--last_modified_latest YYYY-MM-DDT23:59:59 \
	--file_time_range 1d

This will create files like...
cve-bundle-2023_01_01-2023_01_01.json
2. upload cve/cpe bundle to R2
The script will then upload the respective Cloudflare R2 buckets
something along the lines of
https://community.dogesec.com/t/uploading-large-objects-to-cloudflare-r2-using-rclone/94
### note
it might be worth adding the logic in the script to bucket the output into directories.
If file_time_range is set at d then monthly directories are created (YYYY-MM) -- same logic as posting to cloudflare, if m is selected, then yearly directories are created (YYYY), if y selected then no directories created.

closing, will raise bug tickets individually


FormBuilderChipsInput : 'inputType != null': is not true (and other errors)

The FormBuilderChipsInput does not actually works without adding the following variables:
inputType: TextInputType.text,
obscureText: false,
autocorrect: false,
keyboardAppearance: Brightness.dark,
inputAction: TextInputAction.done,
textCapitalization: TextCapitalization.none,

Same issue here.  Just downloaded from Git and it does not run without the changes pointed out by the OP.

Fixed in v3.6.0.


Default push branch?

I'd love a setting where I can set my default branch to push to on the origin, instead of having to pick it from the list every time.

+1

+1 and if it just defaults to origin/master temporarily, then it would be helpful for simple workflow.

It should use the default upstream from git for that branch
If the branch as no default upstream it should default to origin/branch-name


sql task, can you add a semicolon and write multiple statements?

Describe the question
My sql tasks usually have multiple sql, so I want DS to support separated by semicolons and support multiple sql tasks at the same time
Which version of DolphinScheduler:
-[1.3.1]
Additional context
nothing.
Requirement or improvement
nothing.

This feature will be supported in the next version.

thanks!