Replace the KB Dialog with a InfoBar

This changes the keyboard warning from a dialog to an InfoBar, which
we just got in MUX 2.5. Some users were unhappy that we'd always display
the dialog. We learned from the input team that this service should
always be enabled. We're also learing from users that they don't always
want it enabled.
We're working with the Input team to help us figure out how this service
can be disabled and the Terminal work just fine. They're confident
that it shouldn't. For 99% of our users, they're right. So we don't
want to get rid of the dialog entirely, we want to understand how this
is possible. While we wait, let's make the message less aggressive.
This is instead of making a iKnowWhatImDoingDisableTheKeyboardWarning
setting to disable the dialog. Props to @cornem for suggesting the less
aggressive solution.
Validation Steps Performed
Tested manually, but by forcing the message to always display. Disabling
the service requires two full reboots, and ain't nobody got time for
that.
Closes #8228
Closes #4448, for now

Couple of thoughts.  Does the InfoBar follow the theme chosen for the profile/app - or does it pick Light or Dark theme based on the console's background colour's brightness so as to fit better?
Does it use the default severity colours built into the control's template - or do you map the colours to the profile's equivalent colour?
Green for Success
Yellow for Warning
Red for Critical
some kind of inverted colours for the default/informational colours?

Does it use the default severity colours built into the control's template

It's a pretty standard InfoBar, I'm really not doing anything to customize it. The coloration is based on whatever the application's theme is set to, and whatever the default styling for an InfoBar is.
Literally the only styling I've written for it is
<mux:InfoBar x:Name="KeyboardWarningInfoBar"
                     IsClosable="True"
                     IsIconVisible="True"
                     IsOpen="False"
                     Severity="Warning"
                     Message="{x:Bind KeyboardServiceDisabledText, Mode=OneWay}"/>

Does it use the default severity colours built into the control's template

It's a pretty standard InfoBar, I'm really not doing anything to customize it. The coloration is based on whatever the application's theme is set to, and whatever the default styling for an InfoBar is.
Literally the only styling I've written for it is
<mux:InfoBar x:Name="KeyboardWarningInfoBar"
                     IsClosable="True"
                     IsIconVisible="True"
                     IsOpen="False"
                     Severity="Warning"
                     Message="{x:Bind KeyboardServiceDisabledText, Mode=OneWay}"/>

The control can handle itself fine in terms of matching the App's RequestedTheme / The Theme set in Settings.  I suppose what I was wondering is if it should match the UI or if it should match the Profile, in terms of colour.
It seems to be in that between zone in that it could be part of the Tabs and Window Frame UI Zone, or it could be part of the Console area.
If the InfoBar persists when switching tabs, then it makes sense to match the App.

Yeah, so this one is app-affine and not terminal pane affine.
In general, there isn’t a well-defined “warning” color in a terminal application. We get to choose red, or yellow, or green or whatever but the user can reassign those color indices to be whatever color they want. There’s no contract that index 1 must be red, nor that an application use index 1 for warnings or errors.
It’s a neat idea, but I wouldn’t want to introduce too many dependencies on ill-defined things :smile:

Yeah, so this one is app-affine and not terminal pane affine.
In general, there isn’t a well-defined “warning” color in a terminal application. We get to choose red, or yellow, or green or whatever but the user can reassign those color indices to be whatever color they want. There’s no contract that index 1 must be red, nor that an application use index 1 for warnings or errors.
It’s a neat idea, but I wouldn’t want to introduce too many dependencies on ill-defined things 😄

Cool - I was just adding some thoughts to the discussion (or was that opening a discussion) - either way it makes sense for it to be app level!

(That said, I do like the idea. There has been some discussion in the .NET world and presumably the general CLI community about having something closer to semantic markup or more representative of application intent in terminal emulation. Not likely to to anywhere, but interesting to consider what a world is like where we give text meaning instead of just style!)

(That said, I do like the idea. There has been some discussion in the .NET world and presumably the general CLI community about having something closer to semantic markup or more representative of application intent in terminal emulation. Not likely to to anywhere, but interesting to consider what a world is like where we give text meaning instead of just style!)

Quite a departure from a "dumb" terminal churning through command codes...

Quite a departure

Eh, so is OSC 8 hyperlink support. :smile:
There's a fine line, for sure.

moved from the body:

If that InfoBar has to be dismissed manually then it's not any better than a popup. Is still annoying.

If that InfoBar has to be dismissed manually then it's not any better than a popup. Is still annoying.

If that InfoBar has to be dismissed manually then it's not any better than a popup. Is still annoying.

If that InfoBar has to be dismissed manually then it's not any better than a popup. Is still annoying.


How to benefit from a newer version of acs-engine on an existing cluster?

Is this a request for help?:
Yes
Is this an ISSUE or FEATURE REQUEST? (choose one):
QUESTION
What version of acs-engine?:
0.13.1
I'm sorry if this is not the right place but I don't find any other place to ask questions:
I have a cluster running created with acs-engine version 0.13.1. I noticed a newer version of acs-engine is now available, with a lot of bugfixes in it. How can I benefit from those bugfixes on my existing cluster, or is that not possible?  Or is it the same process as acs-engine upgrade?

Hi @mrdfuse,
There are two types of bugfixes that you'll see in acs-engine as releases progress forward (e.g., from v0.13.1 to v0.14.0:

improvements to reliability of cluster deployments, especially improving the breadth of the "types" of cluster configurations that are able to be reliably deployed
absorption of upstream Kubernetes fixes, especially w/ respect to the availability of newly released versions of Kubernetes

With respect to the 1st type, a new version of acs-engine does not add any value to your existing cluster (because it has already been deployed, and is running!).
With respect to the 2nd type, newer versions acs-engine may be able to move your cluster forward onto a newer version of Kubernetes, which could bring stability if the newer version of k8s includes fixes to behaviors that your workloads encounter.
acs-engine upgrade, unfortunately, is an alpha-ish feature at this point in terms of reliably enforcing cluster end state. We're working on publishing a whitelist of cluster configurations that upgrade works reliably against, but the nature of acs-engine generated clusters is that they tend to be distinct from one another (one of the chief purposes of acs-engine is to enable customization). Depending on how mission-critical your cluster is, you could do one of the following:

If your cluster is not mission critical, you could attempt an acs-engine upgrade against it using a newer version of acs-engine, knowing that (1) you may have to retry a few times to reach the desired cluster end state; and (2) there is the possibility that your cluster will be put into a state that is no longer "maintainable" by acs-engine
If your cluster is mission critical, I would recommend at the very least building another cluster with the same configuration, and then performing acs-engine upgrade against it using a newer version of acs-engine to analyze the compatilibity of your cluster w/ the current state of acs-engine upgrade. And then do that a few more times to get some real data about what this operation is likely to do to your production cluster. Finally, you could build a new cluster w/ a newer version of acs-engine, and then migrate your workloads to your new cluster, retiring the old one.

I hope that provides some realistic guidance!

@jackfrancis totally clear, thanks so much for the detailed explanation!!


Implementation of PyCaret Regression Module(Library) for Machine Learning - Python

PyCaret is an open-source, low-code machine learning library in Python that automates machine learning workflows. It is an end-to-end machine learning and model management tool that speeds up the experiment cycle exponentially and makes you more productive.
The PyCaret’s regression module has over 25 algorithms and 10 plots to analyze the performance of models. Be it hyper-parameter tuning, ensembling, or advanced techniques like stacking, PyCaret’s regression module has it all.
Here's the sample screenshot of the models comparison:

@Kumar-laxmi @072arushi

So you just want to some library functions?
This repo is for algorithms implemented without any library function

But by using pycaret library, people can choose what algorithm to choose
and that algo can be directly implemented using pycaret....!!
On Sat, Jun 24, 2023, 4:43 PM Kumar Laxmikant @.***>
wrote:

So you just want to some library functions?
This repo is for algorithms implemented without any library function
—
Reply to this email directly, view it on GitHub
https://github.com/Kumar-laxmi/Algorithms/issues/1566#issuecomment-1605397426,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASIKWJY6P4MSP2GRGXF6W3TXM3DUZANCNFSM6AAAAAAZSLV2EA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Help needed - SD card, Bootloader mode & run payload

Hi, I don't know if it is an issue or just my limited understanding. But, as there is no other help available, I would like to ask for your support here.
I use Cactus WHID and inserted SD card. I use WUD-Ducky for esp32-3.0.2-RC1.

SD Card
Even though reading through all articles here, I don't find a way to get SD card working in visible in Windows Explorer as USB drive. What am I missing / doing wrong? Do I need to adapt the code even?
Bootloader mode
I put WHID in Bootloader mode using magnet for hall sensor and reset button. I manage to upload the sketch. However, whenever I put the USB stick in any PC, it starts in bootloader mode. I have to press reset button before it starts correctly. How could I change?
Run Payload automatically
I would like to simulate autostart functionality. What do I need to do to provide a payload that starts automatically?

Please apologize if the topics would be misplaced here. Happy to get in contact with you (i.e., anybody who could support me) otherwise.

hey @j0mand

SD Card not visible in Windows Explorer

SD may fail on some Windows laptops, but I suspect it's more a consequence of another USB descriptor failing to load.
Since USB-CDC is supposedly disabled it can either be the keyboard or the mouse conflicting with an existing hybrid HID device (e.g. touchpad).
USBView may help identify the conflict.

whenever I put the USB stick in any PC, it starts in bootloader mode. I have to press reset button before it starts correctly.

Sounds like you did not remove the magnet from the hall sensor after flashing?
If you have to flash the WUD repeatedly I suggest you setup a ssid/pwd for the STA mode and use Arduino OTA update methode instead.
Setting up the ssid/pass can be done as a Ducky command:
SetSSID_STA your_router_ssid
SetPassword_STA your_router_pass

Run Payload automatically

This is in the roadmap and needs a few problems solved (logging race condition, SD mount/unmount detection) before implementation starts.
If you want to experiment, you can add this at the end of the setup() function:
   runpayload( SPIFFS, "/your_payload_file.quack.txt" );

'SPIFFS' was not declared in this scope :S
Sorry, I'm sure the solution is obvious, but I swear I've been trying to figure it out for a while.

Is there a way to set the default keyboard language so you don't have to change it every time?

@bomberman8 try LittleFS instead of SPIFFS

@bomberman8

Is there a way to set the default keyboard language so you don't have to change it every time?

a quick&dirty solution is to add this at the end of setup():
duckparser::parse( "LOCALE ES" ); // available language codes are in WiFiDuck/keyboard.cpp

a more comprehensive integration is to make the locale persistent using prefs::set() and prefs::get()
// called from setup()
char locale[8];
prefs::get( "locale", locale, 7, "US" ); // "US" is default if no pref exists, max 7 letters
String command = String("LOCALE ") + String(locale);
duckparser::parse( command.c_str() );

// WiFiDuck/keyboard.cpp
#include "keyboard.hpp"
#include "prefs.hpp" // <<<< add this include

namespace keyboard
{
  // modify setLocale()
  void setLocale( hid_layout_t* layout )
  {
    locale     = layout->locale;
    localecode = layout->code;
    prefs::set("locale", layout->locale ); // <<< add this line
  }

  // ......

}


An Attempt DOA submission

I have attempted DOA 2021 and here is my submission.

Hi @ASHIK11ab  Hope you're doing well.
Thank you for taking the time to solve the all the questions. We really appreciate your efforts. We will verify your submission and your certificate will be sent to you in your email. Thank you for participating. Keep an eye on our social media page for our upcoming events.

@ASHIK11ab Can you please specify your college/institution name for the certificate?

Sri Venkateswara College Of Engineering


remove module from structure inventory importer

please remove the module option here:

Looks like the module is also getting imported from old fda. Do you want me to remove that as well? Basically it is easy to remove the UI module row. Do you want me to do a deep scrub and remove the concept of module everywhere in the fda code?

Deep scrub, yes, please.

I'm on it.

This is completed on branch stage_damage_saving


OCPBUGS-24359: Fix including duplicate blobs in archive

Description
When constructing the archive at the end of the MirrorToDisk workflow, oc-mirror v2 was only checking if the blob was already included in previous archives by comparing its digest to the history map.
This is not sufficient, as the blob might already be included in the current tar, as several images may include the same blob.
This change compares the blob's digest to both the map of digests from history AND the map of digests included so far in archive, before taking the decision to include it in the archive.
Fixes # OCPBUGS-24359
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Perform a MirrorToDisk with ./bin/oc-mirror --v2 -c isc_23236.yaml file:///home/skhoury/23236 and this imageSetConfig:
kind: ImageSetConfiguration
apiVersion: mirror.openshift.io/v1alpha2
mirror:
  platform:
    channels:
    - name: stable-4.14
      minVersion: 4.14.2
      maxVersion: 4.14.3
    graph: true

Expected Outcome

The mirroring is a success.
The archive (mirror_000001.tar) size is 29G, compared to previously 93G. 29G is slightly more than v1-generated archive (mirror_seq1_000000.tar) with 28G.

╭─╼skhoury@fedora ~/23236
╰─ -$ ls -lha
total 57G
drwxr-xr-x. 1 skhoury skhoury  210 Dec  6 18:08 .
drwx------. 1 skhoury skhoury 7.7K Dec  6 15:35 ..
-rw-r--r--. 1 skhoury skhoury  29G Dec  6 18:10 mirror_000001.tar
-rw-r--r--. 1 skhoury skhoury  28G Dec  6 16:11 mirror_seq1_000000.tar
drwxr-xr-x. 1 skhoury skhoury    6 Dec  6 15:41 oc-mirror-workspace
drwxr-xr-x. 1 skhoury skhoury  212 Dec  6 18:08 working-dir

Moving the tar to another file (simulates sneaker-net, or moving the disk physically to the enclave), and performing a DiskToMirror succeeds.

@zhouying7780, @lmzuccarelli , @aguidirh the size difference between the v1 archive and v2 archive is the extra metadata we have in working-dir.
We can probably trim it down a knotch by excluding folder working-dir/release-images from the archive, which is the whole release image, with all its content. It gains ~1G. But I haven't implemented that yet. Please let me know.

/cherry-pick release-4.15

/lgtm


Websocket closes connection with EOF exception on "Switching Protocols" code 101

OKHttp3 version 3.10. Android application
After a successful connection and a short period of inactivity, the OKHttp3 websocket throws a EOFException in response to a code 101, "Switching Protocols" response. This then disconnects the socket and our application is forced to re-connect again.
Is there anyway to control this so the socket does not close ?
Is this expected behavior?
Our iOS and server side tests against the same server did not show this behaivor and remained connected until the applications disconnected them explicitely.
RealWebsocket class:
public void connect(OkHttpClient client) { ...
           try {
                RealWebSocket.this.listener.onOpen(RealWebSocket.this, response);
                String name = "OkHttp WebSocket " + request.url().redact();
                RealWebSocket.this.initReaderAndWriter(name, streams);
                streamAllocation.connection().socket().setSoTimeout(0);
                RealWebSocket.this.loopReader();
            } catch (Exception var6) {
                RealWebSocket.this.failWebSocket(var6, (Response)null);
            }

(Response return code 101)

Could you provide an executable test case? Nothing actionable here.

Issue Resolved:
Apologies, I should have specified that I am also using the wonderful STOMP client on top of the OKHttp3Client websockets:
https://github.com/NaikSoftware/StompProtocolAndroid by Nickolay Savchenko (Nickolay Savchenko
NaikSoftware). This library does an exceptional job of implementing the STOMP protocol integrated with OKHtttp3 websockets.
This is where I fell short by using the wrong StompClient.over method to provide my Url, which will create a default websocket instance that has a default socket read timeout (30 seconds ?). The  for me was to use the alternative StompClient.over method, allowing for a reference to an application instance of the OKHttp3Client. This allows the application set it's desired timeouts beyond any defaults.


Add Slack notification when E2E fails

Description
Whenever the E2E tests workflow fails, send a notification to the int-cp-operator channel.
How can this be tested?
Tested in sandbox workspace

Checklist
- [ ] Unit tests have been updated/added

[X] PR is labeled accordingly with a single label
[X] I have read and understood the contribution guidelines

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.16%.  Consider uploading reports for the commit 7b97fec to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2850   +/-   ##
=======================================
  Coverage   67.16%   67.16%
=======================================
  Files         251      251
  Lines       13693    13693
=======================================
  Hits         9197     9197
  Misses       3693     3693
  Partials      803      803

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


[nvidiadriver] Clarify what does "Active Support" means for Nvidia drivers

See https://github.com/endoflife-date/endoflife.date/pull/2962#discussion_r1190288038 :

This comment is outside the scope of this PR : @Evernow, @captn3m0 : do you know what does "Active Support" means for Nvidia drivers ?
When there is a date active support only last a few months. And when there is not, we have outdated information on most of the cycles (actively supported but EOL).
New features being added to a given branch. For example on Linux the most prevalent would likely be CUDA. NVIDIA isn't adding new CUDA version support to driver branches that aren't actively supported (like the R470 LTSB), while for Windows the most prevalent would likely be game feature updates with the same driver example as above.

As far as I am aware with few exceptions, NVIDIA usually does not backport features to driver branches that are not the latest one. There are occasional exceptions for this (beta branches for example sometimes end up with their features being backported to earlier branches) but I do not think they are important enough to mention.
Off topic: This is definitely something that needs to be clarified in a future PR, along with ideally automating this (there are many ways to potentially look at automating this, I've done a similar task for other situations like with this, but this discussion probably belongs in another PR or issue)

I think now the current page description is sufficient and does not need additional clarification. I merely document things for maintainers in #3303.


Configuration of interrupts from Supervisor and User mode

In CLINT mode, interrupts delegated to S mode would use the vector table stored at address stvec.BASE. However, in CLIC modes, there is no mention of supervisor/user mode equivalents of the new mtvt CSR. Do supervisor mode interrupts also use mtvt (a machine level CSR)?

From the memory map, the cliccfg register is only present in the Machine mode apertures. How would Supervisor and User mode code be able to correctly set levels, priorities, selective-vectoring and/or delegate interrupts without being able to determine the fields of cliccfg? Shouldn't there be a mention of setting up read-only permissions for this register from lower privilege modes?

Note that the 'm' is in a slightly different font, because it is a variable.  This is mentioned at the top of section 3 in the table, and below it.

Cliccfg, and clicintcfg, are memory mapped I/O registers, not CSRs.  So maybe this is just a matter of getting the memory permissions right for these memory regions?


Catch the deletion key before type assertion causes a panic

Handle cache.DeletedFinalStateUnknown keys in the virt-controller for VMIs.
fixes: https://github.com/kubevirt/kubevirt/issues/6418
Signed-off-by: Ryan Hallisey<EMAIL_ADDRESS>NONE

/retest

@rthallisey could you add a release-note? I think this is worth a note so that other people running into panics can find this fix more easily.

/lgtm
/approve

rebased

/lgtm
re-adding lgtm after rebase

@rmohr @davidvossel PR is approved.  Can you add lgtm?


ctr: Add sandbox flag to ctr run

Add a flag to be able to run a container in a given sandbox.

/retest

@Iceber Done, thanks!

/cherrypick release/1.7


WebKit8 OLED display not working

I have an ESP8266 with buildin OLED (https://www.aliexpress.com/item/ESP8266-WIFI-Chip-0-91-inch-OLED-CP2014-32Mb-Flash-ESP-8266-Module-Internet-of-things/32822165059.html)
It useses SDA=pin 2 and SCL=pin 14, and it works with Adafruuit_SSD1306 library but only when I define by using
Adafruit_SSD1306 display(OLED_RESET);
where OLED_RESET = 4
"Normal OLED modules" require OLED = 0;
I gues this is the reason it is not working with your library.
Any ideas on how to get it to work?

You have to use correct pins for constructor (supplied documentation is not correct):
SSD1306xxx display(0x3c, 4, 5);
and You have to reset display manually before library init
// reset display
pinMode(16, OUTPUT);
digitalWrite(16, LOW);
delay(50);
digitalWrite(16, HIGH);


Add basic behavior of authorization webhook

What this PR does / why we need it:
Add authorization webhook. In this PR, we only implement the default behavior.

[x] Delete RequestHeader because we can carry the version in gRPC metadata.
[x] Implement basic authorization webhook.
[x] Add access attributes such as key and verb to webhook request payload.

Which issue(s) this PR fixes:

Address #182
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation:

Checklist:

[x] Added relevant tests or not required
[x] Didn't break anything

The following additional work will be done in the follow-up PR.

[ ] Implement retry and retryBackoff.
[ ] Add authorization-methods flags.
[ ] Add authorization-cache-ttl flags (LRUExpireCache).
[ ] Provide a way to refresh the token.


c7i-flex instances are not correctly labeled with bandwidth

Description
Observed Behavior:
AWS EC2 flex instances (e.g. c7i-flex) do not get a instance-network-bandwidth labels, but other instance types do. The equivalent instance types (e.g. c7i) that do not have flex are properly labeled.
The code in bandwidth_gen.go expects the bandwidth to always be the 5th column in the documentation, but for flex instances it is the 6th column due to the new "Baseline performance" feature.
It appears that flex instances are very new. Documentation example here.
Expected Behavior:
All instances with explicit bandwidth values should have a instance-network-bandwidth label.
Reproduction Steps (Please include YAML):

Allow the use of c7i-flex instance types
Check the labels of an instance of that type, and there will be no instance-network-bandwidth label

Versions:

Chart Version: N/A
Kubernetes Version (kubectl version): v1.28.9-eks-036c24b

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

(I work with @craigandrews) Actually, we just noticed this is fixed on main: https://github.com/aws/karpenter-provider-aws/pull/6099 fixed it.

Closing this issue since the fix has already been added in #6099


handling softmax overflow

Softmax is prone to overflow, especially with "small" datatypes, such as fp16, and it can only get worse with fp8 etc.
This is well known and usually handled by subtracting a constant in the exponents. That constant drops out in the end, so it is mathematically equivalent (for infinite precision), however for finite precision math, it can make the difference from overflow to not overflow.
The question is how to chose that constant. Ideally one should know the histogram of the values. Alternatively, one could check for overflow at runtime, and adjust the constant at runtime, but that's a heavier calculation, so best avoided.
We should think how to handle this. The softmax API can accept more optional arguments to set the constant, or enable a dynamic determination. And migraphx can maybe use calibration data to deduce it. Not sure if this constant can be expressed in onnx...

That might be something that we can tune for MIGraphX. Specifically, we can run some calibration data to get a reasonable constant, then use it in the actual inference.


Fixed typo - principle instead of principal

Learn Build status updates of commit e481727:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/fundamentals/concept-learn-about-groups.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks @ClemensSzyperski !

#sign-off

#sign-off


Hi @jakoch,
Few questions:

Is this a new workflow or is it something that had been running successfully with old version of actions/cache?
If it is an old workflow, what version actions/cache were you using previously?

Hi kotewar,
this is an old workflow. it was running successfully before.
we used action/cache@v1 before and updated to v3.0.11.
we also updated the cache key, when updating the version number to v3: see change diff

@jakoch Can you try the same workflow with latest v3?

test with v3 and issue is resolved,
see https://github.com/jakoch/hikogui/actions/runs/3874064901/jobs/6604814406#step:6:74
thank you!


Client is lagging out averaging 500 ping ms

I tested the internet and everything, It works well, But the client is lagging out badly, I've updated to the latest version Bloxstrap.

ive tried unistalling it and resinstalling it lastly updating to the latest version of windows 11

As the preliminary instructions asked, does this occur only with Bloxstrap? https://github.com/pizzaboxer/bloxstrap/wiki/Switching-between-Roblox-and-Bloxstrap

Ive done that and fixed the issue, it does not occur with bloxstrap. it does it with other games


Enabling Accelerated Networking via Azure Portal

[As option to enable Accelerated Networking is available via Azure Portal, can we add this to the MSDocs, also would it need reboot? As I tried to enable it from portal and it did not reboot the servers. Is it correct when doing it via Portal?]

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

ID: e4be8e45-460a-fab4-519b-32d83382005c
Version Independent ID: 36a002a6-5813-da00-0a57-068eeaf491cf
Content: Create Windows VM with accelerated networking - Azure PowerShell
Content Source: articles/virtual-network/create-vm-accelerated-networking-powershell.md
Service: virtual-network
GitHub Login: @gsilva5
Microsoft Alias: gsilva

Hello @jyotiprakash90,
Thank you for your feedback! We will review and update as appropriate.

@jyotiprakash90, Appreciate your patience. Here is a document that explains about enabling the accelerated networking via Azure portal.
Also, this toggle switch behavior will not be available through portal after creation of the VM.
You can confirm whether the accelerated networking is enabled using this "lspci" command.
It should return similar output to this,
0000:00:00.0 Host bridge: Intel Corporation 440BX/ZX/DX - 82443BX/ZX/DX Host bridge (AGP disabled) (rev 03)
0000:00:07.0 ISA bridge: Intel Corporation 82371AB/EB/MB PIIX4 ISA (rev 01)
0000:00:07.1 IDE interface: Intel Corporation 82371AB/EB/MB PIIX4 IDE (rev 01)
0000:00:07.3 Bridge: Intel Corporation 82371AB/EB/MB PIIX4 ACPI (rev 02)
0000:00:08.0 VGA compatible controller: Microsoft Corporation Hyper-V virtual VGA
0001:00:02.0 Ethernet controller: Mellanox Technologies MT27500/MT27520 Family [ConnectX-3/ConnectX-3 Pro Virtual Function]

The ethernet controller indicates that Mellanox VF driver is activated then check for activity on the VF (virtual function) with the ethtool -S eth0 | grep vf_ command.
Ref: https://docs.microsoft.com/en-us/azure/virtual-network/create-vm-accelerated-networking-cli#portal-creation

Hi,
My question was towards this https://docs.microsoft.com/en-us/azure/virtual-network/create-vm-accelerated-networking-cli#enable-accelerated-networking-on-existing-https://docs.microsoft.com/en-us/azure/virtual-network/create-vm-accelerated-networking-cli#enable-accelerated-networking-on-existing-vmsvmshttps://docs.microsoft.com/en-us/azure/virtual-network/create-vm-accelerated-networking-cli#enable-accelerated-networking-on-existing-vms
I need to enable accelerated networking on an existing VM. While this option wasn't available via Azure portal earlier, now that it's available, the document you shared only mentions about CLI or PowerShell.
And as it mentions on doc that enabling accelerated networking requires deallocating the VM first, however it wasn't the case in my lab when I enabled it on an VM via Azure portal.
Could you please confirm the functionality around the same.
Regards,
JP

From: suvasara @.>
Sent: Tuesday, April 6, 2021 11:11:34 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Jyoti Prakash Nayak @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Enabling Accelerated Networking via Azure Portal (#73128)
@jyotiprakash90https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjyotiprakash90&data=04|01|Jyoti.Nayak%40microsoft.com|6e9d3e9177cb4b7eefea08d8f92338e3|72f988bf86f141af91ab2d7cd011db47|1|0|637533276985574132|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=zn9KHJG%2FFMXtw0JSr6h2csQHq7eK0oRW3MBKSGFkklg%3D&reserved=0, Appreciate your patience. Here is a documenthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fvirtual-network%2Fcreate-vm-accelerated-networking-cli%23create-a-linux-vm-with-azure-accelerated-networking&data=04|01|Jyoti.Nayak%40microsoft.com|6e9d3e9177cb4b7eefea08d8f92338e3|72f988bf86f141af91ab2d7cd011db47|1|0|637533276985584130|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=U5jF4AxOuF41LSr%2B%2FWmBH38CVTZx5U5cCSzifnqcYI4%3D&reserved=0 that explains about enabling the accelerated networking via Azure portal.
Also, this toggle switch behavior will not be available through portal after creation of the VM.
You can confirm whether the accelerated networking is enabled using this "lspci" command.
It should return similar output to this,
0000:00:00.0 Host bridge: Intel Corporation 440BX/ZX/DX - 82443BX/ZX/DX Host bridge (AGP disabled) (rev 03)
0000:00:07.0 ISA bridge: Intel Corporation 82371AB/EB/MB PIIX4 ISA (rev 01)
0000:00:07.1 IDE interface: Intel Corporation 82371AB/EB/MB PIIX4 IDE (rev 01)
0000:00:07.3 Bridge: Intel Corporation 82371AB/EB/MB PIIX4 ACPI (rev 02)
0000:00:08.0 VGA compatible controller: Microsoft Corporation Hyper-V virtual VGA
0001:00:02.0 Ethernet controller: Mellanox Technologies MT27500/MT27520 Family [ConnectX-3/ConnectX-3 Pro Virtual Function]
The ethernet controller indicates that Mellanox VF driver is activated then check for activity on the VF (virtual function) with the ethtool -S eth0 | grep vf_ command.
Ref: https://docs.microsoft.com/en-us/azure/virtual-network/create-vm-accelerated-networking-cli#portal-creationhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fvirtual-network%2Fcreate-vm-accelerated-networking-cli%23portal-creation&data=04|01|Jyoti.Nayak%40microsoft.com|6e9d3e9177cb4b7eefea08d8f92338e3|72f988bf86f141af91ab2d7cd011db47|1|0|637533276985594123|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Cd0Hfo%2BFlnlJJguLXyHnGSKlNXfUmSqDKlI5VDocLRM%3D&reserved=0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F73128%23issuecomment-814307346&data=04|01|Jyoti.Nayak%40microsoft.com|6e9d3e9177cb4b7eefea08d8f92338e3|72f988bf86f141af91ab2d7cd011db47|1|0|637533276985594123|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=9SBXqmdb9Ut6U9eHMsVeP8L7xH9QxwripAV%2BmM5hK7k%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPLZOXM6CUFLOVWIR4EN4NTTHNBU5ANCNFSM42GU6MBA&data=04|01|Jyoti.Nayak%40microsoft.com|6e9d3e9177cb4b7eefea08d8f92338e3|72f988bf86f141af91ab2d7cd011db47|1|0|637533276985604118|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=q56i%2Fmejb5AJoMHkgwHKWW8XDVT7fEXpWPYMcsIwCd0%3D&reserved=0.

@jyotiprakash90, here is the info from the doc I have highlighted in my previous response,

Unfortunately enabling accelerated networking on an existing VM is not available today from the portal side. Looks like the doc you were referring to needs a PR. We recommend you use PowerShell for now,
$nic = Get-AzNetworkInterface -ResourceGroupName "myResourceGroup" `
    -Name "myNic"

$nic.EnableAcceleratedNetworking = $true

$nic | Set-AzNetworkInterface

Hi,
This option is available now as below:
@.***D72BC2.0A7FDAC0]
Regards,
Jyoti Prakash
From: suvasara @.>
Sent: Wednesday, April 7, 2021 3:15 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Jyoti Prakash Nayak @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Enabling Accelerated Networking via Azure Portal (#73128)
@jyotiprakash90https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjyotiprakash90&data=04|01|Jyoti.Nayak%40microsoft.com|972be29a1a4b4dd0985c08d8f9a9c094|72f988bf86f141af91ab2d7cd011db47|1|0|637533854794431971|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PRLL%2FFQXWVzdf7pJKkM0b3K8cKji9cR8koR8raf6gx8%3D&reserved=0, here is the info from the doc I have highlighted in my previous response,
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F44890829%2F113845451-2758c000-97b3-11eb-9103-a919d7d32626.png&data=04|01|Jyoti.Nayak%40microsoft.com|972be29a1a4b4dd0985c08d8f9a9c094|72f988bf86f141af91ab2d7cd011db47|1|0|637533854794441969|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=MbivrMcuRtKd%2B5vf2mxq%2BCZNhmLymNGYA%2FQH%2BRV086U%3D&reserved=0
Unfortunately enabling accelerated networking on an existing VM is not available today from the portal side. Looks like the doc you were referring to needs a PR. We recommend you use PowerShell for now,
$nic = Get-AzNetworkInterface -ResourceGroupName "myResourceGroup" `
-Name "myNic"

$nic.EnableAcceleratedNetworking = $true
$nic | Set-AzNetworkInterface

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F73128%23issuecomment-814773243&data=04|01|Jyoti.Nayak%40microsoft.com|972be29a1a4b4dd0985c08d8f9a9c094|72f988bf86f141af91ab2d7cd011db47|1|0|637533854794441969|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=sL7%2FIEEuRj02932EC14wcWtYED0EETSc%2B%2BJIIk7gUUk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPLZOXLASUTVMVVA32FA6ETTHQSQFANCNFSM42GU6MBA&data=04|01|Jyoti.Nayak%40microsoft.com|972be29a1a4b4dd0985c08d8f9a9c094|72f988bf86f141af91ab2d7cd011db47|1|0|637533854794451965|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=cariElRYqozjlbq0iv0RG3leqe2t8oqLITiUPJWBkSM%3D&reserved=0.

@jyotiprakash90, @jyotiprakash90, My bad! I am checking in NIC properties blade instead of overview section.

But this feature failed to update the NIC when VM is on. So, you need to stop the VM for enabling accelarated networking on that nic.

@jyotiprakash90,
Update:
Also, we observed that there is a strange behavior with DS2_v2 virtual machine size. It needs VM to be in deallocated state for enabling accelerated networking on it. Whereas other sizes do enable this feature without need of VM reboot/deallocation.

@jyotiprakash90, this abnormal behavior has been reported to the concerned team. I will be adding you to the conversation.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

It is unclear whether from the portal.azure.com, if you enable "Accelerated networking", or if after disabling accelerated networking and re-enabling it, the machine needs to be deallocated (stopped from the portal).
Can you please clarify if it is required or not?

@GonzaloMurillo, there is no need to deallocate the virtual machine while enabling/disabling the "Accelerated Networking". But, for Ds2_v2 size it still needs the VM in stop state to enable "AN" on its nic. Also,
SS:


Added output information with path to  wrong JPEG file

Can one of the admins verify this patch?

@tensorflow-jenkins test this please

I switched this to raising a RuntimeError instead of killing the program.  If lint passes, this seems okay to me.


Add refresh tokens to /api/login/authorize and /api/login/generate sponses

Fixes #774

Codecov Report

Merging #776 into master will decrease coverage by -0.08%.
The diff coverage is 60.6%.

@@            Coverage Diff             @@
##           master     #776      +/-   ##
==========================================
- Coverage   72.12%   72.04%   -0.08%
==========================================
  Files          80       80
  Lines        4760     4787      +27
==========================================
+ Hits         3433     3449      +16
- Misses        996     1006      +10
- Partials      331      332       +1

Impacted Files
Coverage Δ

login.go
77.77% <100%> (+1.91%)
:white_check_mark:

login/service.go
70.9% <56.66%> (-2.63%)
:x:

Continue to review full report at Codecov.

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


Update to 20w09a

haven't done much testing myself, but there doesn't seem to be any major problems with farlands 20w07a unless i've missed something

Just updating the bundled libraries wouldn't be a bad idea anyways.

change to 08a

Done in 396958db3e623c38e226eb92540e2d0eb163d471

Builds will be released tomorrow, it's 4am here and I haven't really done any testing at all.


[FLINK-16627]  Flink-SQL json format add non-null optinon

What is the purpose of the change
json encode only with non-null fields
Brief change log

flink-sql json optinoal add 'json.encode.non-null'

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (don't know)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (not documented)

Does anyone have any suggestions ? or some one has another progress ?


docs: Document operator.unmanagedPodWatcher

Describe what the new operator.unmanagedPodWatcher option does in the
upgrade guide.

I can handle the backports for these since they'll conflict.

I'm not sure I follow you on the backports. I thought the option was new for 1.12, and I can't find it in older branches? Did I miss something? :thinking:

@qmonnet that option has been backported and will be available in the releases we announce today.


Feedback link is invisible.

feedback link is white on white background until you click it.  once clicked it is visible.

It's by design - only the box is visible, the text is visible when hovering.
We can choose another style. Should we make it almost invisible - i.e. light grey?


Missing tests to ensure formats ignore non-strings

All the current formats operate only on strings, but no tests ensure that other types are allowed.
These tests likely belong not in optional, since it's not optional behavior, even if validation is, but hopefully no one finds it confusing that there are tests both in optional and required for the same validator (format).

Closed in #277.


Migrate images to git lfs

Added in https://github.com/andy-voz/mova-cards/commit/55c60e140fb3140e773267876afaa433df2abc68


Problem with accent in the description of an application configuration

API-Mgr. CLI version
1.2.3
API-Manager and Service-Pack Version
7.7.20200730
Expected behavior
Having a config file such as :
{
"name": "Test API Specifique",
"description": "Accès aux API spécifiques",
...
...
}
Actual behavior
Having accents in the description return this error :
335    ERROR    APIMgrAppsAdapter| Error creating/updating application. Response-Code: 400. Got response: '{"errors":[{"code":400,"message":"Invalid JSON format"}]}'
**Log-Output **
335    ERROR    APIMgrAppsAdapter| Error creating/updating application. Response-Code: 400. Got response: '{"errors":[{"code":400,"message":"Invalid JSON format"}]}'
339    ERROR ApplicationImportApp| Error creating/updating application
com.axway.apim.lib.errorHandling.AppException: Error creating/updating application
at com.axway.apim.adapter.clientApps.APIMgrAppsAdapter.createOrUpdateApplication(APIMgrAppsAdapter.java:378)
at com.axway.apim.adapter.clientApps.APIMgrAppsAdapter.createApplication(APIMgrAppsAdapter.java:318)
at com.axway.apim.appimport.ClientAppImportManager.replicate(ClientAppImportManager.java:35)
at com.axway.apim.appimport.ClientApplicationImportApp.importApp(ClientApplicationImportApp.java:72)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at com.axway.apim.cli.APIManagerCLI.run(APIManagerCLI.java:142)
at com.axway.apim.cli.APIManagerCLI.main(APIManagerCLI.java:78)
Caused by: com.axway.apim.lib.errorHandling.AppException: Error creating/updating application.
at com.axway.apim.adapter.clientApps.APIMgrAppsAdapter.createOrUpdateApplication(APIMgrAppsAdapter.java:362)
... 9 more
Caused by: com.axway.apim.lib.errorHandling.AppException: Error creating/updating application. Response-Code: 400
at com.axway.apim.adapter.clientApps.APIMgrAppsAdapter.createOrUpdateApplication(APIMgrAppsAdapter.java:358)
... 9 more
Test case sample

Please describe the steps you have done to replicate the issue
use the sample in the wiki for application or just add a description with an accent and import it.

Thanks for reporting @Rennaud. Will be fixed as well with the next release.


Improve readme

I don't know why the ansible-connection parameters were included, they make no sense with the rest of the instructions being for installing to a remote server.
I also added a note for other types of keys, and re-numbered some steps.

#38


Multi-Touch triggers the relays as well

The same issue that is happening with swipe, also happens here.
if i multi-touch 2-3 buttons, they all toggle as well

Could you please update to the latest version and try again?
Please let me know the results.


"Contact your admin" message is added

A message
"Contact your administrator in case of issues"
is rendered above the Clear messages link within the Messages
list in case there's at least one user message rendered.
Fixes: https://github.com/oVirt/ovirt-web-ui/issues/92


Minor fixes in non-working RPM build script

Syntax error fix
Help message moved to a variable
New parameter - "final RPM destination absolute path"
Fixed copying final RPMs + added ability to specify destination RPM directory
Removed ./bin directory from source since it is not a part of code base and it wasn't possible to build RPMs without running compiling PhantomJS first by running ./build.sh
${builddir} must be also removed after rpmbuild process

Landed! Thank you!


Add instructions for an easier way of testing the plugin

As discussed in #330 I have added some files and a short description to the collaboration md file for you.
Have a look whether you think this will be valuable or not, and let me know.

👍


Adopt use of better-files library to clean up file IO code

I just came accross better-files
It looks like it could help us substantially clean up some of our file-based IO code, at the cost of an extra dependency). Just opening this as a lead for possible future investigation.

Related: #1320


jellyfin-media-player: 1.6.1 -> 1.7.0

Motivation for this change
Prepare for the upcoming 1.7.0 release:

https://github.com/jellyfin/jellyfin-media-player/releases/tag/v1.7.0-pre1
https://github.com/jellyfin/jellyfin-media-player/releases/tag/v1.7.0-pre2
https://github.com/jellyfin/jellyfin-media-player/releases/tag/v1.7.0-pre3
https://github.com/jellyfin/jellyfin-media-player/releases/tag/v1.7.0-pre4
https://github.com/jellyfin/jellyfin-media-player/releases/tag/v1.7.0

Now the jellyfin-media-player does not depend on a custom build jellyfin-web anymore, so we can reuse the jellyfin-web from nixpkgs.
This depends on the jellyfin-web changes from #144082 which i've added to this PR until that other PR is merged.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Rebased and updated to https://github.com/jellyfin/jellyfin-media-player/releases/tag/v1.7.0-pre4

Rebased and updated to the 1.7.0 stable release: https://github.com/jellyfin/jellyfin-media-player/releases/tag/v1.7.0

Rebased with the changes from #144082.


.NET 6.0 Enumerable.MaxBy conflicts with Enumerable.MaxBy

.NET 6.0 introduces built-in methods for MaxBy/MinBy which have the same signature as the IX ones.
As a result, after upgrading to .NET 6.0, I get the following error:
The call is ambiguous between the following methods or properties: 'System.Linq.EnumerableEx.MaxBy<TSource, TKey>(System.Collections.Generic.IEnumerable, System.Func<TSource, TKey>)' and 'System.Linq.Enumerable.MaxBy<TSource, TKey>(System.Collections.Generic.IEnumerable, System.Func<TSource, TKey>)'

Actually the return type is different as it is a List for IX but a single value for .NET 6.

We'll get this fixed soon. Are there any other conflicts due to new methods/overrides?

That's the only one I noticed but I'm not sure if there are any others.

Fixed in 5.1.

Hi @clairernovotny
Did you remove these methods altogether for NET 6.0?
We still would like to use them.
You see, the ones built-in return a single element where as the IX ones return a list.
Would it be possible to re-add them to NET 6.0 but use a different name to avoid a conflict?
Maybe MaxByAll? Or MaxByWithTies?
What do you think?

@terrajobst @bartdesmet for naming/suggestions.

If i'm not mistaken the problem with ambiguity is bigger and solution provided for MaxBy/MinBy in 5.1 might be accidentally breaked after future minor and major .NET releases
Current pattern for solving ambiguity that i see in sources is wrapping duplicating methods with compiler directive for hiding method when referencing from some target frameworks from reference assembly. For example hiding of MaxBy was implemented by following directive (MinBy solution is simular):
#if !(REFERENCE_ASSEMBLY && (NET6_0))
    public static IList<TSource> MaxBy<TSource, TKey>(this IEnumerable<TSource> source, Func<TSource, TKey> keySelector){...}
    public static IList<TSource> MaxBy<TSource, TKey>(this IEnumerable<TSource> source, Func<TSource, TKey> keySelector, IComparer<TKey> comparer){...}
    private static IList<TSource> ExtremaBy<TSource, TKey>(IEnumerable<TSource> source, Func<TSource, TKey> keySelector, Func<TKey, TKey, int> compare){...}
#endif

Problem that i see in this solution is that we are checking especially that are we targeting .net6.0 listed in reference assembly project targetFrameworks:
<TargetFrameworks>net4.8;netstandard2.1;net6.0</TargetFrameworks>
it works for now, but if we change target after .NET 7 release to <TargetFrameworks>net4.8;netstandard2.1;net7.0</TargetFrameworks>
it's going to break.
To fix it we must somehow set not just NET6_0, but something simular to NET6_0_And_FUTURE
If we use .net 5+ sdk for development process it can be achieved using  NET6_0_OR_GREATER constant
Actually problem is already hurting us:
Discribed pattern was introduced in https://github.com/dotnet/reactive/pull/743 (problem and solution desribed by @clairernovotny here and from then SkipLast/TakeLast methods are wrapped with following compiler directives
#if !(REFERENCE_ASSEMBLY && (NETCOREAPP2_1 || NETSTANDARD2_1))
    public static IEnumerable<TSource> SkipLast<TSource>(this IEnumerable<TSource> source, int count){...}
    public static IEnumerable<TSource> TakeLast<TSource>(this IEnumerable<TSource> source, int count){...}
#endif

ref assembly at that time targeted: <TargetFrameworks>netstandard1.0;netcoreapp2.0;netstandard2.1</TargetFrameworks>
time by time it has changed to current state resulting following demo project IxCompatibilityWithNet6.zip not compile now with error:
Test.cs(7, 21): [CS0121] The call is ambiguous between the following methods or properties: 'System.Linq.EnumerableEx.SkipLast<TSource>(System.Collections.Generic.IEnumerable<TSource>, int)' and 'System.Linq.Enumerable.SkipLast<TSource>(System.Collections.Generic.IEnumerable<TSource>, int)'
Csproj:
<Project Sdk="Microsoft.NET.Sdk">

    <PropertyGroup>
        <TargetFramework>net6.0</TargetFramework>
        <Nullable>enable</Nullable>
        <LangVersion>default</LangVersion>
    </PropertyGroup>

    <ItemGroup>
      <PackageReference Include="System.Interactive" Version="5.1.0" />
    </ItemGroup>

</Project>

Cs:
using System.Linq;

namespace IxCompatibilityWithNet6;

public sealed class Test
{
    public static void Run()
    {
        new[] { 1 }.SkipLast(1);
    }
}

Problem reproduces only on net6.0 as it is explicitly targeted in reference assembly

Using NET6_0_OR_GREATER constant sounds good.
However the issue here is that not only MaxBy and MinBy were added to .NET, but also that their return type is different.
We are targeting net6.0 but we still need to compile our code with the IX versions of the methods.
In Enumerable:
public static TSource? MaxBy<TSource,TKey> (this IEnumerable<TSource> source, Func<TSource,TKey> keySelector);
In EnumerableEx:
public static IList<TSource> MaxBy<TSource, TKey>(this IEnumerable<TSource> source, Func<TSource, TKey> keySelector);
Given that the return type is different, I believe the only option to make the IX MaxBy available in .NET 6+ is to rename it to disambiguate. For example, it could be named MaxByWithTies.

@clement911 sure, you right that we need renaming to keep originnally conflicting, but still useful methods available from API
My message is more about problems of existing "hiding" pattern.
I just think that hiding conflicting method for keeping code compilable and adding renamed copies for specific frameworks for keeping functionality available is for sure related, but pretty parallel topics.
So we can both set NET6_0_OR_GREATER for current MaxBy/MinBy(NETCOREAPP2_0_OR_GREATER for SkipLast/TakeLast) and add new MaxByWithTies/MinByWithTies methods.

That would be great

I've updated the code to use the OR_GREATER defines. There was a bug in the Extras that caused those not to be defined by default.

@clement911 @clairernovotny

Would it be possible to re-add them to NET 6.0 but use a different name to avoid a conflict?
Maybe MaxByAll? Or MaxByWithTies?

What do the IX ones? Assuming they include ties, then MaxByWithTies seems the most apt name to me.

The IX ones include ties. That's why they return a List. The .NET ones return a single object.

  DebugPanel.razor(140, 54): [CS0121] The call is ambiguous between the following methods or properties: 'System.Linq.EnumerableEx.TakeLast<TSource>(System.Collections.Generic.IEnumerable<TSource>, int)' and 'System.Linq.Enumerable.TakeLast<TSource>(System.Collections.Generic.IEnumerable<TSource>, int)'

@SicJG  This is exactly what I am seeing in an attempt to upgrade my application to .net 6 today. Should a new issue be opened here?

@bbarry i don't think so. Described problem was fixed in 90f8a4d9c510305bb93775b5ef3039f2c7908eda and merged to main.
We're just waiting for next release of Ix.Net, could be a patch with this fix only.
@clairernovotny do you know when it may happen?

Ix / Ix Async 6.0.1 were just published that have these fixes (and now include MinByWithTies/MaxByWithTies for those that need the original behavior).

Thank you very much @clairernovotny


Refactor.dynamic data

@gulfaraz nice. I've merged already, but will open a task in the same item to process the specific feedback here.
Yes, 'IBF-pipeline-output' and 'IBF-static-input' have partly merged into 'IBF-app' but have also partly disappeared.


Update to 2.0

Starting all changes for 2.0 both for the REDME.md and the .zip file itself

Note for myself:
Now that everything is moved to the GitHub app, I can do normal commits without it being painful

Moved this PR into a new Branch
(Still learning to use Github sorry)


Users cannot delete/deactivate themselves or last admin

https://github.com/rancher/rancher/issues/14818
There are tests for deleting yourself, but not for deleting the last admin, as the only way you could get to that point (past deleting oneself) is a race condition, where adminA deletes adminB at the same time as adminB deletes adminA (or any cycle of the sort).

Feedback given in person: simplify down to "cant delete yourself" and add a utility to re-add the admin user from the command line, similar to how we can reset password

Simplified the logic, added ensure-default-admin command which will add a default admin if none exists. If one does, it will ensure they still have admin privileges and that their user is active.

@maxkorp open up a documentation issue in rancher/docs and assign yourself and set it to the 2.1 milestone.
Documentation for this should wind up in the technical faq, close to where we document how to reset the admin password
https://rancher.com/docs/rancher/v2.x/en/faq/technical/#how-can-i-reset-the-admin-password

LGTM


Create Caramel

The Pull Request resolves Issue #[Issue-Number]
Description:
A brief description about the addition or edits made.
Notes: Delete all the text including this!

The #[Issue-Number] should be replaced with the specific Issue being referenced.

@lavendersnake
Create a json file and update the PR. Refer CONTRIBUTING.md for more details


Search: improved auto-complete suggestions menu

Improve the auto-suggestions by adding more infos (category, image & price)

 You should sign our Contributor License Agreement in order to get your pull request merged.

 You should sign our Contributor License Agreement in order to get your pull request merged.

Hi @redox , thank you for the PR. As we did a big release yesterday with Go Live version your branches now have merge conflicts. Can you please resolve merge conflicts and then we will handle the PR. Thanks.

Oh sure thing, I'll!

I get some errors while testing it today:

{
  message: "Something went wrong while processing the request. The error has been logged under code 20151217_082822_533_xjedcme. Your request id is 5af39742-d716-41fa-a536-ee2ef869f568."
  status: 500
  type: "internal_service_error"
}

@redox There are few small comments, please take a look at them.
We will check in our logs what was the problem with site service and if you are still getting 500 error please let us know.

@nemanjapopovic thank you for the review; I'm still unable to run it locally to be able to test/debug because of that 500 error (request id is 7e0994ee-4774-4adb-ba69-1132fe951705) -> any chance you could have a look why it says error 500?

Hi @redox , we looked in our logs. It seems that the problem is with subscriptions. Can you double check that your project is subscribed to Site Management package and also to the other packages. If it is and it is still not working can you please try creating new project and subscribing again to all needed packages. Let me know how it goes. Thanks.


Update zh_Hans Localizations. And more.

Add new Localizations.

"Alert.CancelInstall.Runtimes.Title"
"Alert.DeletePlatform.PrimaryButton"
"Alert.DeletePlatform.Title"
"Perform post-install steps"
"Error"
"Open In Rosetta"
"Platforms"
"PlatformsList.Title"

Fix some old Localizations. All contains "前置步骤" -> "安装后准备步骤".

Remove old or useless Localizations.

Create a file "String+.swift" to remove preview strings from localizations. Make it easier to work on translation.

Got it! I will try it this weekend.

I found that although Text support initializes with init(verbatim: ), Picker or TextField don't have alternatives for this.

@MattKiazyk I removed "String+" and modified all Texts to use init(verbatim: ). For "Picker" and "TextField" which are not supporting init(verbatim: ), I gave them labels with Texts. Please check it out.

@megabitsenmzq thanks so much for those changes - can you just rebase and I'll get that merged in

@megabitsenmzq thanks so much for those changes - can you just rebase and I'll get that merged in

Done!


docs: Update for developer tap migration

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Linuxbrew/developer is no more; long live Homebrew/linux-dev!

Thanks for the review @colindean!


Document Issues

https://github.com/Next2D/next2d.app/blob/2bd160c8e02ddb8bf4a165a593d19a110ecea6d5/language/en/framework-api.json#L35
contractor of the specified class means constructor of the specified class?
And, could you please explain the meaning of the following sentence?
https://github.com/Next2D/next2d.app/blob/2bd160c8e02ddb8bf4a165a593d19a110ecea6d5/language/en/framework-api.json#L18

@HerbertHe
Thanks issues.

contractor of the specified class means constructor of the specified class?
Yes, I am interested. That is correct.
The mechanism is to set variables in the constructor of the class specified in json and call the execute function.

https://github.com/Next2D/framework/blob/784d63af8640d5d1c367005b03dbf43f0a6b2156/src/Application.js#L514

And, could you please explain the meaning of the following sentence?
config.json can set fixed variables.
For example, if you change the URL of the API used in the local environment and the URL of the API used in production, you can manage different URLs by separating them with "local" and "prd".
You can use npm build --env="***" to write out the URLs for each environment.

Fixed variables that you want to use in all environments can be set to all to use the same fixed variables in all environments.
https://github.com/Next2D/framework-template/blob/main/template/src/config/config.json
The description was strange, so I made some changes.
https://github.com/Next2D/next2d.app/commit/2d281cf8f22f094ed1f472907ed4eef737399b9d

なのほど!


Search: Index group display name in DefaultCatalogCollatorFactory

Hey, I just made a Pull Request!

Added the spec.profile.displayName to the search index for Group kinds
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

This is Catalog-land, although search related...  So while I don't own the code and can't give you an approval, I'll weigh in with some context:
A totally valid use-case for collators is that they be replaced with custom implementations.  We threw in these DefaultFooCollator implementations in Catalog/TechDocs because we figured we could probably cover the basics for the 80%.  A custom implementation could be used to implement this exact change.  ...It could also be used to just add the display name as its own field.
This particular change will naturally change the way Search behaves for everyone who has not implemented a custom collator.  The question becomes: is this a change that would be beneficial to the 80%?  I personally appreciate that it makes users/groups a little more consistent.  It seems reasonable to expect that a group could be found by its display name.
🤷♂

We view this change as being beneficial to the 80% AND bringing consistency to the search experience of Users and Groups. That's why we decided to make the PR against the default collator.
One idea we had was to index on the metadata.title. We decided not to because that seemed like a more impactful change to the default


how to add multiple routes to multiple route tables

i want to add a list of routes to all my 3 public route tables

IMHO, you can use additional aws_route resources outside of this module but using this modules outputs.

hi all, this should be addressed in the next version which is under development here https://github.com/clowdhaus/terraform-aws-vpc-v4/tree/main/modules/subnets


add tags for versions

Which are automatically shown as releases on GitHub (and provide the option later to run a CI action like uploading to the marketplaces someday).

@usernamehw Would you apply a script that adds the existing versions as tags (then I'd create a script for that and post it here) and create tags for upcoming versions?
I'd like to subscribe to releases and also be able to get a fast overview of the changes between versions directly on GitHub - tags (or even "full" releases) would help a lot with that.

I think it was done at some point.

Rechecked: the last tag available is 1.2.0, maybe you have those all locally and need to only git push --tags to send them to this repo?

Maybe with this it will be synced in the future... "git.followTagsWhenSync": true,


Lambda layers rework

Description of changes

BuildStep  and package for layers
Refractored zipping logic from runtime plugins to function category

Issue #, if available

Description of how you validated changes
Added e2e  and unit tests
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request base (layers-rework@64cff13). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             layers-rework    #7085   +/-   ##
================================================
  Coverage                 ?   56.39%
================================================
  Files                    ?      445
  Lines                    ?    21871
  Branches                 ?     4382
================================================
  Hits                     ?    12335
  Misses                   ?     8757
  Partials                 ?      779

Continue to review full report at Codecov.

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


Discovery API

Discovery API
The discovery API aims to provide libp2p users (and other libp2p subsystems) a way for active peer discovery. Peers providing a service will use this API to advertise their presence in some namespace. On the other side, peers seeking a service use the API to discover peers that have previously advertised as service providers.
This issue aims to decide the following:

API
Libp2p Configuration
Implementations
Reprovider

API
Based on libp2p/go-libp2p-core/discovery/discovery.go

libp2p.discovery.advertise(namespace, options)
libp2p.discovery.findPeers(namespace, options)

A draft of it can be seen in libp2p/js-libp2p/blob/feat/discovery-api/doc/API.md#discoveryadvertise.
The discovery implementations (for now contentRouting modules and rendezvous) have some differences that will need to be considered.
Advertise
The advertise is trivial to implement, but the way it will be used over time to reprovide content needs to be considered. Rendezvous supports a given ttl for its records, but the DHT and the delegates currently have a fixed ttl.
🏁 For an initial solution, it will be fine to not support a ttl in advertise and default rendezvous to have the same to ease reprovide implementations. However, we should support this in the long run. I foresee application layer use cases interested in just announcing a service for a small interval and have application logic to understand if should advertise again or not. If we go with this solution, an issue to track an enhancement should be created.
Dropping this ttl for now would make the initial API for advertise: libp2p.discovery.advertise(namespace)
FindPeers
The findPeers API might need additional options from the Rendezvous and content routing current implementations. It should support an option for specifying the max number of distinct peers needed.
If we only have setup one Discovery implementation, it will be trivial to implement. However, if we setup multiple implementations we need to consider how they will be leverage, namely in Series or Parallel.
In series (perhaps following an order configured) will be easy to implement, and once we get the maxNumber we can just return. However, having a parallel approach would result in faster results. In a parallel approach, the implementations will need to be synchronised regarding when there are enough distinct peers. One possible solution is to add Abort Controllers support for rendezvous and content routing modules discovery. Once libp2p found enough peers, both would be aborted.
🏁 For an initial solution, we should probably do the discovery in Series. The main reasons for this are the easy implementation and the lack of Signed Peer Records in the content routing modules. We would use the content routing modules as a fall back, or if rendezvous is not setup. If we go with this solution, an issue to track an enhancement should be created.
Libp2p configuration
In the configuration side of things, we should probably not complicate until we work on improving the configuration. The simplest solution would be:
Libp2p.create({
  contentRouting: [...],
  dht: DHT,
  rendezvous: Rendezvous
})

Libp2p discovery module would be responsible for getting the appropriate modules if available. It would use the contentRouting API (which also includes the DHT).
The ideal solution would be:
Libp2p.create({
  discovery: [ ...]
})

But this introduces a lot of problems, as we would not be able to use libp2p.contentRouting API nor the DHT, as it is created inside libp2p. This would allow a configurable order for discovery modules though.
🏁I think we should go with the simplest approach and properly think about all the configuration issues at the same time, as we are already planning that.
Implementations
Rendezvous and ContentRouting modules will be used, but other modules might be implemented. There are two ways of adding support for the discovery API in these modules.
These modules can implement the API in their codebase, which would be as simples as mapping function namings and eventually rename option parameters (example: limit to maxNumProviders). But, if we do not support from now a configuration with discovery: [ ...], we can also start by having adapters in libp2p to do these mappings.
While adding these to the modules seems the best approach considering the other interfaces we have, it also feels odd to have these mappings in the module codebase. In the future, we can also have modules that extend these modules capabilities with the discovery API. Extend these modules would be tricky, it would depend on how we will go with the configuration improvements.
🏁 At this point, I would go with adapters in libp2p as this WIP branch.
Reprovider
We currently leave for the application layer reproviding content over time. However, one of the goals of the discovery API is to have libp2p subsystems leveraging it (like auto-relay). With this, auto-relay will need to have its own reprovide..
🏁 Taking into account the need to reprovide in the libp2p context, I think we should have a general purpose reprovider available. It would be used not only by the discovery API to reprovide over time (configurable), but also by the DHT. We just need to understand how we would connect all this, considering that the discovery API might use the DHT underneath.

@jacobheun let me know your thoughts on this.
🏁 are the points where we need a decision

Cross posting this:

Not going to block this PR on this (as we're doing it in other content/peer routing calls), but we should think about the parallel actions this is doing. Most people shouldn't be using two, but in the change they are, the parallel calls could cause some odd behavior. This is probably worth discussing in more depth when we look at the discovery api / configuration updates.

ref: https://github.com/libp2p/js-libp2p/pull/798#discussion_r530541555

Closing as this needs agreement at the specs level before defining the JS API, and even then it will likely be implemented as a service.


Add setting to relax SA1519 to allow consecutive with braces omitted on all but the last one

#2180 modified the rule to disallow this:
using (var a = GetDisposable())
using (var b = GetAnotherDisposable())
{
}

This was permitted in classic StyleCop. It would be helpful to have a setting that allows this behaviour to be retained.

:+1:

Good question. I just checked classic SC behaviour. The exception applies only to using. The following examples do result in SA1503.
for (int x = 0; x < 100; x++)
for (int y = 0; y < 100; y++)
{
}

while (true)
while (false)
{
}

if (true)
if (false)
{
}

Personally I think the classic behaviour is sensible. The concise form doesn't sacrifice readability for using directives, but for flow control structures requiring the braces is preferable.

:+1:  for using only

I'd prefer if the classic StyleCop behavior was the default with a setting to make it more strict if wanted/needed.

👍 for using only

:+1: for using only

:memo: We made the default compatible with StyleCop Classic so no updates to KnownChanges.md were required.


Support for CustomIpPrefix (BYOIP)

Resource Provider
NRP
Description of Feature or Work Requested
Need to have CLI support a new feature for customers to bring their own IP ranges to Azure, coming to Public Preview by 9/30.  This is all done using the CustomIpPrefix object - a new type of NRP resource.  Full PowerShell support is in progress; note there is already partial support .
There are also some modifications to the PublicIpPrefix and PublicIp classes as well.
So for CLI, we will need a new family of commands
az network custom-ip prefix
and updates to
az network public-ip prefix and az network public ip
Minimum API Version Required
2021-02-01
Swagger Link
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/network/resource-manager/Microsoft.Network/stable/2021-02-01/customIpPrefix.json
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/network/resource-manager/Microsoft.Network/stable/2021-02-01/publicIpPrefix.json
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/network/resource-manager/Microsoft.Network/stable/2021-02-01/publicIpAddress.json
Target Date
The Public Preview date is 9/30.  Updated PowerShell will roll out by 9/7, so we'd like this to be around the same time (early September).
Please contact<EMAIL_ADDRESS>for more details.

network

@yonzhan Could you help us understand the general timeframe when this can be executed?  The release date for the feature is 9/30, apologies for the late notification.  The good news is that the required pieces are already in the latest API.

This feature is planned in next sprint https://github.com/Azure/azure-cli/milestone/109

This issue was closed prematurely.  @kairu-ms please re-open or create a new issue for tracking


[Feat] Setup Blog Schema and Implement Blog Data Fetching from Database

Description
Description:
The blog functionality is not yet implemented in the application. To support blog posts, we first need to create a database schema for blogs and set up fetching data from the database. This will provide the foundation for adding blog-related features in future tasks, such as displaying blog summaries and individual blog content on the frontend.
Proposed Solution:

Define the Blog Schema:

Create a new Blog model in the schema.prisma file with fields such as:

id: Unique identifier for each blog.
title: Title of the blog post.
content: Main content of the blog.
image: URL for the blog's cover image.
authorId: Identifier linking the blog to a User.
createdAt: Timestamp of when the blog was created.
updatedAt: Timestamp of when the blog was last updated.

Establish relationships if needed (e.g., link blogs to User as author).
Run prisma db push to sync the new schema with the MongoDB database.

Create Sample Blog Data:

Add 2-3 initial blog entries to the database to facilitate testing of the fetching logic.

Set Up Data Fetching Logic:

Implement backend API endpoints to retrieve blog data.
Create endpoints such as:

GET /api/blogs: Fetch a list of blog summaries.
GET /api/blogs/:id: Fetch full details of a specific blog by id.

Ensure the data retrieval works correctly and returns the necessary fields for displaying on the frontend.

Expected Outcome:

A Blog schema is defined in the database, and blog data is stored correctly.
API endpoints for fetching blog summaries and individual blog details are functional.
Initial blog data is available in the database for testing further frontend implementation.

Additional Information:
This foundational setup will allow for a seamless transition to implementing the Blog Grid page and individual Blog Page functionality in a follow-up issue. The next step will focus on dynamically loading the blog content using this database connection.
Screenshots
Current blog page with static details:

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@kavya410004 You are assigned.


add definition drug dosage

Done:
https://github.com/DIDEO/DIDEO/commit/14e87ce7cb55c9566c9d28f5a5f9942fea60b2f6
https://github.com/DIDEO/DIDEO/commit/3d8beb7ab021b035926de2587408a39b572cc697


nixos/gns3-server: fix ubridge_path

Description of changes
Ubridge doesn't work since the path passed to gns3-server is to the nix store version rather than security wrapper created just before. This PR fixes this
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Now I get:
Could not determine uBridge version for /run/wrappers/bin/ubridge

I guess it has something to do with permissions. But since there is no logging in the gns3server for this part, I would probably need to patch the gns3-server package to get that message.
I have tried adding o=rx to security wrapper, but that didn't fix the problem. And since it runs fine when launched as normal user with docker and ubridge groups, I suspect it has something to do with systemd service config.

Created an issue for further discussion #292258


Added login_hint parameter in authorizationUrl

This PR enforces redirection to UAA authentication server during dashboard rendering flow.

Pull Request Test Coverage Report for Build 2099

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.0009%) to 89.333%

Totals

Change from base Build 2096:
0.0009%

Covered Lines:
8199

Relevant Lines:
8886

💛  - Coveralls


Error: utils.tree is not a function

Hey Bane!
I cloned the repo, but when I ran app.py, I got the message JS Error => TypeError: utils.tree is not a function. Do you know what's up?

I think you need to update trame-vuetify, trame-client, and well just update all of the trame packages. This is a new feature

Hmm, I've updated everything, but the error persists. It shows up once the new browser tab is opened but before the app can be displayed. Interestingly, that error message is printed over and over again, without stopping.
For reference, here are the versions I'm using (I've compared their versions  to those listed in their respective Github repos):
trame                         2.3.0
trame-client              2.6.0
trame-components  2.1.0
trame-deckgl            2.0.1
trame-leaflet             1.0.2
trame-markdown      2.0.2
trame-matplotlib       2.0.1
trame-plotly               2.1.0
trame-rca                   0.3.0
trame-router              2.0.1
trame-server              2.9.0
trame-simput             2.2.5
trame-vega                 2.0.2
trame-vtk                    2.0.17
trame-vuetify              2.1.0

I think you need to reset the browser cache or hard reload the webpage

I've even restarted my computer, but no luck! I'll ask someone to try it independently and let you know.


Hope to add custom proxy address.

https://github.com/Frooodle/Stirling-PDF/releases/download/v0.14.2/Stirling-PDF-with-login.jar
This file downloads too slow in China,so hope to add custom proxy address.

Added on sourceforge, is that any better?
https://sourceforge.net/projects/stirling-pdf/


Add UseContainerSupport flag to default value.yaml of k8s helm chart

Fix #17021

@ssz1997 Thanks for the offline discussion about the related issue, this PR add the necessary jvm flag UseContainerSupport to the values.yaml of kubernetes helm chart. Would you like to take a look at this PR？Thanks.

@maobaolong How about we make the global one enabeld by default, so there's no need to add it to each individual component?

@ssz1997 Thanks for your advice, PTAL.

@maobaolong I modified your PR to use the existing global jvmOptions. Do you think this is better?

@ssz1997 It is better, thanks for your help.

@maobaolong Could you help merge? I think you have the permission to do alluxio-bot, merge this please.


Enhanced desk to work nicely with the fish shell.

I enhanced desk to work nicer with the fish shell:

The function syntax is different (the function keyword is mandatory but the parentheses are missing).
The syntax for combining commands with &&/and is different too.

I tested it on my macbook and it seems to work nicely.
lint didn't get worse either. :-)

I did some additional tests and a small cleanup.
I can explain the changes I did and why I did them that way.
E.g. I removed the call to find in cmd_go because:

the desk name has to be unique or the function would break in a funny way
if all the names are unique they can very well be stored in the same directory
one less call to an external executable is always nice

P.S.: Thanks a lot for the great idea of desk! :smile:

I've absorbed this commit into https://github.com/jamesob/desk/pull/25. Thanks for the contribution, @ole108!


渲染存在空行

Describe the bug

在某一次更新后，渲染时会出现如图所示的空行
Reproduction
chrome、safari、edge 均复现
System Info
bytedance@C02F31JWMD6V ~/.ssh> npx envinfo --system --binaries --browsers
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y)

  System:
    OS: macOS 13.2.1
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 113.40 MB / 32.00 GB
    Shell: 3.6.0 - /usr/local/bin/fish
  Binaries:
    Node: 19.8.1 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.5.1 - /usr/local/bin/npm
  Browsers:
    Chrome: 111.0.5563.64
    Safari: 16.3

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

It doesn't seem like a bug, because the li element is wrapped in a p tag, which causes line breaks.


Possibly a bug - Pell's equation

What version of OR-Tools and what language are you using?
Version: 9.10.4067
Language: Python
What operating system (Linux, Windows, ...) and version?
Linux
This code:
#!/usr/bin/python3

from ortools.constraint_solver import pywrapcp

def main(D):
  solver = pywrapcp.Solver("...")

  max_limit=5*10**8
  x = solver.IntVar(0, max_limit, "x")
  y = solver.IntVar(0, max_limit, "y")

  solver.Add(x>0)
  solver.Add(y>0)
  # Pell's equation:
  solver.Add(x*x - D*y*y == 1)

  solution = solver.Assignment()

  db = solver.Phase([x,y], solver.CHOOSE_MIN_SIZE_LOWEST_MAX,
                    solver.ASSIGN_MIN_VALUE)

  solver.NewSearch(db)
  assert solver.NextSolution()
  print("x=%d y=%d D=%d" % (x.Value(), y.Value(), D))
  assert x.Value()**2 - D*(y.Value()**2) == 1

main(1)

Produces:
x=67108865 y=67108865 D=1
...
    assert x.Value()**2 - D*(y.Value()**2) == 1
AssertionError

Of course, this is incorrect solution. x==y==67108865 here.
% pip3 list | grep ortools
ortools             9.10.4067

Most likely you are overflowing int64.
If you try with cp-sat, it should return model invalid.

Here is the code for CP-SAT
from ortools.sat.python import cp_model

def main(coeff: int):
  model = cp_model.CpModel()

  max_limit=5*10**8
  x = model.new_int_var(1, max_limit, "x")
  y = model.new_int_var(1, max_limit, "y")

  # Pell's equation:
  x_square = model.new_int_var(1, max_limit * max_limit, "x_square")
  y_square = model.new_int_var(1, max_limit * max_limit, "y_square")
  model.add_multiplication_equality(x_square, x, x)
  model.add_multiplication_equality(y_square, y, y)
  model.add(x_square - coeff*y_square == 1)

  model.add_decision_strategy([x, y], cp_model.CHOOSE_MIN_DOMAIN_SIZE, cp_model.SELECT_MIN_VALUE)

  solver = cp_model.CpSolver()
  solver.parameters.num_workers = 12
  solver.parameters.log_search_progress = True
  solver.parameters.cp_model_presolve = True
  solver.parameters.cp_model_probing_level = 0

  result = solver.solve(model)
  if result == cp_model.OPTIMAL:
    print(f"x={solver.value(x)} y={solver.value(y)} D={D}")
    assert solver.value(x)**2 - D*(solver.value(y)**2) == 1

main(2)

It is surprisingly slow

Most likely you are overflowing int64.

Excuse me, but which variable/value has this type?

Most likely you are overflowing int64.

Excuse me, but which variable/value has this type?

All:
https://github.com/google/or-tools/blob/fd6f278e8c31fc6cfe8568c7e99b38a9f5fc5da4/ortools/constraint_solver/constraint_solver.h#L1104-L1105