Added a generic SendMessage overload with lParam passed by value.

This generic overload of SendMessage
IntPtr SendMessage<TMsg, TWP, TLP>(HWND hWnd, TMsg msg, TWP wParam, ref TLP lParam)

It forces client code to pass lparam with the ref parameter modifier, even if the message doesn't mutate it.
This is redundant and misleading.
I added a new overload:
IntPtr SendMessage<TMsg, TWP, TLP>(HWND hWnd, TMsg msg, TWP wParam, TLP lParam)

LPARAM is defined in C++ as VOID* and must be a pointer. If a value remains unmodified, then it can only be an Int32. In that case, you should specifically convert the Int32 to IntPtr. The overload you've recommended allows for any structure to passed in without converting it to a pointer. This will certainly cause problems. What specific type are you passing that needs this?

LPARAM is defined in C++ as VOID* and must be a pointer. If a value remains unmodified, then it can only be an Int32. In that case, you should specifically convert the Int32 to IntPtr. The overload you've recommended allows for any structure to be passed in without converting it to a pointer. This will certainly cause problems. What specific type are you passing that needs this?

Would this be fixed with the unmanaged type constraint?


Roll Flutter Engine from dd67063fc921 to 8d9f17e07633 (1 revision)

https://github.com/flutter/engine/compare/dd67063fc921...8d9f17e07633
2023-04-21<EMAIL_ADDRESS>[web] Fix HashUrlStrategy.addPopStateListener (flutter/engine#41384)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Found 3 elements with non-unique id #button

What is affected?
Component
Description
if i have more then 1 button in a form with a type button on it, (to provent it from submiting the form)  the browser will throw an error with Found 3 elements with non-unique id #button
<form action="null">
      <md-outlined-text-field label="Password" type="password">
        <md-icon-button toggle type="button" slot="trailing-icon">
          <md-icon>visibility</md-icon>
          <md-icon slot="selected">visibility_off</md-icon>
        </md-icon-button>
      </md-outlined-text-field>

      <md-outlined-text-field label="Password" type="password">
        <md-icon-button toggle type="button" slot="trailing-icon">
          <md-icon>visibility</md-icon>
          <md-icon slot="selected">visibility_off</md-icon>
        </md-icon-button>
      </md-outlined-text-field>

      <md-outlined-button type="sbmit">Submit</md-outlined-button>
    </form>

Reproduction
Stsckblitz example https://stackblitz.com/edit/vitejs-vite-xdsjbm?file=index.html
Workaround
I havent found a workaround
Is this a regression?
No or unsure. This never worked, or I haven't tried before.
Affected versions
version 1.1.1
Browser/OS/Node environment
chrome browser version 120 on windows 11

This looks similar to https://github.com/material-components/material-web/issues/5213.
Are you using any tools or plugins to remove the shadow dom? The IDs are unique within each shadow root.

@asyncLiz I believe Microsoft Edge DevTools report this no matter what (in the new Issues tab, not sure if Chrome has it too). This is a bug on DevTools, not MWC's.

Generally I find the info on that tab unhelpful than helpful.

Can confirm as well, and OP's reproduction is sufficient:
Playground link
However it appears to be a Chromium bug:
https://bugs.chromium.org/p/chromium/issues/detail?id=1496312

Closing since this is a dev tools bug


Did not manage to run jasmine tests

I intended to run jasmine test on win7, phpstorm7, phpstorm karma plugin. I found multiple problems...
I installed karma from git bash:
npm install -g karma
Despite the statements in the documentation it did not install the karma-* plugins, so I had to install them manually:
npm install -g karma-jasmine
npm install -g karma-requirejs
...
After that I generated a test file with karma:
module.exports = function (config) {
    config.set({
        basePath: 'test',
        frameworks: ['jasmine', 'requirejs'],
        files: [
            {pattern: 'test-mixin.js', included: true}
        ],
        exclude: [

        ],
        reporters: ['progress'],
        port: 9876,
        colors: true,
        logLevel: config.LOG_INFO,
        autoWatch: true,
        browsers: ['Firefox'],
        captureTimeout: 60000,
        singleRun: false
    });
};

I tried to run (ofc I installed the plugins after found out by run that they are not installed), but I got the following phenomenons:

Sometimes I got port is already in use message, sot it automatically tries with other port (but nothing runs on the 9876 just karma). I tried with another port, I got the same results after some runs.
If I close the browser window manually it does not start a new window, just writes that server is not running. If I add the new window manually it starts complaining about port collisions, and opens a new window with different port.
I tried to change the browser to Chrome in the config file, but it still did open Firefox window.
I have confusing messages in phpstorm windows:

empty test suite (in event log)
no tests found
instantiating tests (for several minutes)
unable to attach test reporter to test framework or test framework quit unexpectedly
waiting for previous execution
[DEBUG] config - No config file specified
INFO [karma]: Delaying execution, these browsers are not ready: Firefox 26.0.0 (Windows 7)

I don't know what happens in the background, but the test obviously does not run. :S

(The code uses core js only, it does not require browser specific tests, so I'll change to jasmine-node instead of this.)

i was having the same problem
it was basically removing requirejs from the list of frameworks
frameworks: ['jasmine'],


UI Lockup on Message Interaction

Bug Report: UI Lockup on Message Interaction
Installation Method

Method used: Manual for Win10

Environment

Open WebUI Version: v0.3.28
Ollama Version: v0.3.11
Operating System: Windows 10
Browser: Version 129.0.6668.71 (Official Build) (64-bit)  , also tried on Edge with same behaviour.

Confirmation:

[x] I have read and followed all instructions provided in the README.md.
[ ] I am on the latest version of both Open WebUI and Ollama.
[x] I have included the browser console logs.
[x] I have provided the exact steps to reproduce the bug in the "Steps to Reproduce" section below.

Expected Behavior
The chat message should be clickable, allowing interaction without causing the browser to freeze or CPU to spike.
Actual Behavior
When clicking on a specific message (ID: 6fee83cf-52c6-4c20-bc44-390858f2e848), the UI freezes, and the CPU usage spikes to 100%. The browser becomes unresponsive until manually stopped.
Description
Bug Summary:
The UI becomes unresponsive when clicking on the last message (6fee83cf-52c6-4c20-bc44-390858f2e848) in the chat history, causing a CPU spike and browser freeze. (Multi-Model Messaging used)
Reproduction Details
Steps to Reproduce:

Load the attached chat history (file: chat-export-1727554950475.json). (not sure if possible via json, but found no other way)
Attempt to click on the message with ID: 6fee83cf-52c6-4c20-bc44-390858f2e848.
Observe the UI freezing and Chrome’s CPU usage spiking to 100%.

Logs and Screenshots
Browser Console Logs:
Message ID: 6fee83cf-52c6-4c20-bc44-390858f2e848
Stack Trace:
ye (MultiResponseMessages.svelte:191)
c (MultiResponseMessages.svelte:199)

Screenshots/Screen Recordings (if applicable):
[Attach screenshots or screen recordings if available]
Additional Information

This issue seems to be linked to the following snippet in the Svelte component:<div
   class="snap-center w-full max-w-full m-1 border {history.messages[messageId]
       ?.modelIdx == modelIdx
       ? `border-gray-100 dark:border-gray-800 border-[1.5px] ${$mobile ? 'min-w-full' : 'min-w-[32rem]'}`
       : `border-gray-50 dark:border-gray-850 border-dashed ${$mobile ? 'min-w-full' : 'min-w-80'}`
   } transition-all p-5 rounded-2xl"
   on:click={() => {
       if (messageId != _messageId) {
           let messageChildrenIds = history.messages[_messageId].childrenIds;
           while (messageChildrenIds.length !== 0) {
               messageId = messageChildrenIds.at(-1);
               messageChildrenIds = history.messages[_messageId].childrenIds;
           }
           history.currentId = _messageId;
           dispatch('change');
       }
   }}

The issue could relate to the logic in on:click that handles messageChildrenIds, possibly creating an infinite loop or resource lockup.

Note
Please ensure that the bug report follows the guidelines, including all required details, logs, and reproduction steps. Thank you!
chat-export-1727554950475.json

Good catch, fix on dev! Testing wanted here!


utils/ruby.sh: remove dependency for which command

[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.

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

which is a non-standard command for Bash (shellcheck SC2230). Besides, which command shows aliases as well but not in the desired format:
$ which -a python
/usr/bin/python

$ alias python='python3'

$ which -a python
python: aliased to python3
/usr/bin/python

Remove dependency for which command, see Homebrew/install#574 and Homebrew/install#575 for more details.

Do we want to use type -aP here too?

I think we could replace the function content (the for loop) with type -aP but keep function which_all here. I think it could be more elegant and readable.


CAMEL: Fixed checkstyle errors for connectors

Just cleaned those checkstyle errors in the connectors directory.

Thanks for the PR it has been merged. Do you mind closing this?

Thanks Claus, closing then.


Id refactor

@macieksmuga and I were discussing this.
There is a question of how Ids should be generated for objects inside the reference server.  So far we seem to be doing a concatenation of filenames / existing ids to produce a new unique id.  Another solution would be UUIDs.  A third would be using ids assigned by a third-party system, like a SQL database we use to store objects (in the graph).
Proposal: we should have a option in the server config to choose the id strategy we use at startup.  The code should be refactored to read this option and assign Ids based on the strategy chosen.
The advantage of this would be 1) more flexibility for the user 2) an easy way to compare which Id generation strategies would work best 3) whenever the Id generation method argument is definitively settled, the change on our end will be as simple as setting that to be the default serverconfig option.

FYI python has a uuid module https://docs.python.org/2/library/uuid.html

I strongly advocate for UUIDs only.  We want to push GA4GH to go
to a GUID system, where GUIDs are either digests computed
directly from the data or UUID where digests will not work well.
There is still a need to link to external database through
secondary identifiers, however it opens up a lot of complexity
to no control the primary identifier space.
It's strategic for GA4GH to address the data identification
issue.  This is a huge problem for bioinformatics and genomics.
We need to demonstrate this in the server.
Since ids are wide spread through the server, It will also
simplify testing.
+1 UUIDs, -2^31 anything else
Danny Colligan<EMAIL_ADDRESS>writes:

@macieksmuga and I were discussing this.
There is a question of how Ids should be generated for objects inside the
reference server. So far we seem to be doing a concatenation of filenames /
existing ids to produce a new unique id. Another solution would be UUIDs. A
third would be using ids assigned by a third-party system, like a SQL database
we use to store objects (in the graph).
Proposal: we should have a option in the server config to choose the id
strategy we use at startup. The code should be refactored to read this option
and assign Ids based on the strategy chosen.
The advantage of this would be 1) more flexibility for the user 2) an easy way
to compare which Id generation strategies would work best 3) whenever the Id
generation method argument is definitively settled, the change on our end will
be as simple as setting that to be the default serverconfig option.
—
Reply to this email directly or view it on GitHub.*

UUIDs are great, but we need IDs to be stable across server restarts. We also need to be able to find a given object when we have its ID. So, when we need to support GET /variants/{id} we can know how to find it by decomposing the ID string. This is a strong requirement for me.
IDs here are internal to the implementation, and there is no requirement about their structure in the spec. Clients should not use the object IDs as anything other than an opaque identifier. I don't see a lot of point in giving admins choice in the ID selection strategy --- they're for the server's use, not anyone else's.

Yes, UUID should be used when stable external ids are required,
but something truly internal is up to the pleasure of the
developers.
I thought that external was the point.  Otherwise, why would
someone configuring the server care?

My understanding of the object ID fields is that they are internal IDs only, and are never intended to be used as a way of comparing objects across instances. Stable UUIDs for objects across repositories are a separate (and hugely important!) issue.

The id fields in the object are exposed to the user between ReST
requests and thus stable in some sense.  The scope of stable is
a big, unresolved issue within the DWG.  Right now, it's
undefined.
We (UCSC people I brainwash) feel that it's vital that ids be
stable be global for all conforming GA4GH servers.  This enables
a world of functional, such as reproducible, caching and
encouraging data sharing by making citation easy.
Jerome Kelleher<EMAIL_ADDRESS>writes:

My understanding of the object ID fields is that they are internal IDs only,
and are never intended to be used as a way of comparing objects across
instances. Stable UUIDs for objects across repositories are a separate (and
hugely important!) issue.
—
Reply to this email directly or view it on GitHub.*

Sure, the IDs must be stable on a given server, that seems clear. UUIDs for every read and variant though? That's a very strong requirement, and would need a huge lookup table for us to support it. We want to remain stateless and work directly from VCF and BAM. Requiring a UUID for each read and variant would make this impossible I think.
UUID for References, ReadGroups, VariantSets, etc, I totally agree with.

Agreed UUIDs should be on the immutable units, e.g. ReadGroups.
Ids within an immutable unit could be simple integers, however I think
they still should be globally stable withing the readgroup and all queries
need to be scoped with the immutable unit id.
Thus a read query would be (ReadGroupUUID readNumber).
I am really glad this is coming up.
Jerome Kelleher<EMAIL_ADDRESS>writes:

Sure, the IDs must be stable on a given server, that seems clear. UUIDs for
every read and variant though? That's a very strong requirement, and would need
a huge lookup table for us to support it. We want to remain stateless and work
directly from VCF and BAM. Requiring a UUID for each read and variant would
make this impossible I think.
UUID for References, ReadGroups, VariantSets, etc, I totally agree with.
—
Reply to this email directly or view it on GitHub.*

OK, that seems reasonable. So, is the idea is variantSetId is a UUID and is the same across different servers, and the variantId is then "{}:{}".format(variantSetId, variantNumber)? We need to make sure that we can efficiently retrieve the individual variants/reads, but I think something like this is workable.
So how do we define a UUID for a variant set? Isn't this something that a working group has been working on?

That is the idea, although I suspect for efficiency reasons we
will eventually have to support multiple variants, reads, etc,
per query.  But that could be restricted to a single UUID.
The containers and work flows group has been discussing data
digests as a way of generating GUIDs.  I am not sure were that
is at.  It kind of doesn't belong there, but because Matt is
interested in digests, it's being discussed there.
For anything that is large enough to generate a GUID from a data
digest, that should be the approach.
The UCSC BD2K center, which includes Berkeley, has put in a
request for a supplement to cover digests and identification.
Jerome Kelleher<EMAIL_ADDRESS>writes:

OK, that seems reasonable. So, is the idea is variantSetId is a UUID and is the
same across different servers, and the variantId is then "{}:{}".format
(variantSetId, variantNumber)? We need to make sure that we can efficiently
retrieve the individual variants/reads, but I think something like this is
workable.
So how do we define a UUID for a variant set? Isn't this something that a
working group has been working on?
—
Reply to this email directly or view it on GitHub.*

Thanks @diekhans and @jeromekelleher for the debate!
This is a critical issue to me right now, as I'm putting the finishing touches on the SQL schemas for reference/variation graphs, and I'd like to settle on the right way to do this. Sounds like I should switch to UUID's for persistent entities like ReferenceSets, and then have a compound (colon-separated works for me) ID for each Reference, with a simple integer identifier for the second part of this ID (mapping nicely to the numeric ID column as currently defined in the Reference table).

Whoops, reading more carefully: UUID for each reference sequence too. There will be an awful lot of these in the graph world...
Also, would callSets warrant a UUID as well?

The way to think about it is: what is the immutable unit.
Also, a UUID is only 128 bits in binary.
How many and how big.
CallSet for sure need a UUID.
Alleles too, since we want ti link to them from different
variants.
Maciek Smuga-Otto<EMAIL_ADDRESS>writes:

Whoops, reading more carefully: UUID for each reference sequence too. There
will be an awful lot of these in the graph world...
Also, would callSets warrant a UUID as well?
—
Reply to this email directly or view it on GitHub.*

Can I close this issue?  It seems like something that some working group will get back to us in the future on, and not really actionable right now...

I agree with @dcolligan - there are bigger fish to fry right now - but let's keep the above discussion in mind the next time we cycle around to this issue...

Yes, lets close this for now.


Lava quad

Finally creating a pull request for this.

Lava Quad in , a little off when you look at it from far away, the texture is a bit blurry
Added a end Portal functionality but that is commented out in this branch, I will do that in a seperate branch to keep this clean

WRONG

Lava quad could be it's own game at this point


Make it easier to find locations where ACA is available

Related command
az provider show locations
Is your feature request related to a problem? Please describe.
ACA is not available in all locations, and it saves a lot of time if a user can see where it is available before attempting a deployment.
The current command is pretty unintuitive and hard to remember:
az provider show -n Microsoft.App --query "resourceTypes[?resourceType=='managedEnvironments'].locations"
Describe the solution you'd like
We should have something that is simple and easy to remember like this:
az provider show locations -n ContainerApps
There should also be a command/param to filter to regions that not only have ACA deployed, but also have available capacity:
az provider show locations available -n ContainerApps
or
az provider show locations -n ContainerApps -onlyAvailable
(There is probably already a pattern for this, just providing a couple of examples.)
Describe alternatives you've considered
I've asked around and the ugly command above is the best anyone could think of.

@zhoxing-ms for awareness

Adding Service team to look into this.

@calvinsID Could you please look into this and provide an update ?

@charris-msft I don't see the REST service supports finding location in REST documents. Could you confirm that REST service supports this feature?
If so, please provide the REST docs and Swagger link

No, I can't confirm that. I'm not saying that it should work based on what we have currently implemented. Rather I believe we need to make whatever changes are required so that the command can be that simple across all providers.

OK, I think this feature needs to be supported by the REST service first, so that multiple client tools can support it.
And only if the REST service supports filtering locations can make those commands' performance be faster.

Makes sense. Can you forward it on to the correct team?

@tjegbejimba Could you help take look at this issue or transfer it to the right person?


Serialize make cluster-sync build + deploy steps

The way the make cluster-sync build + deploy steps are referenced means that make will try to parallelize them if -j > 1 is specfied (like I have via MAKEFLAGS in my .bashrc). cluster-sync was never working for me after a fresh checkout for example, I needed to invoke it twice
This fixes the rule to explicitly serialize the  cluster-build and cluster-deploy invocations

/release-note-none
/lgtm

/ok-to-test

/retest

@crobinso I can confirm this makes make cluster-sync work reliably on the first attempt for me :)
/lgtm

/approve
/retest


Switch illum filtering from ndi percentile filter to SciKit image rank filter

Ndi percentile filter causes memory error as a result of the very large (512px) radius used to filter which requires an 8*512^4 byte array in the SciPy implementation (https://github.com/scipy/scipy/issues/4878).
Switching to use scikit image median rank filter which is optimised to remember values of already visited pixels.

Coverage increased (+0.03%) to 70.583% when pulling 1af97941f7e33c15194d347656f20a4dcfcc353a on DragaDoncila:rankfilters into 46cdebc29ce530366bb2c1b4b9ce8c4f10fc700a on microscopium:master.

Whoo! =)

Nice work, @DragaDoncila  and @jamesryan094!


client leak lead to master memory cannot gc

when i use alluxio hadoop client,if i don't close FileSystem, the RetryHandlingMetaMasterConfigClient will always  connect master. this solution may lead to master memory cannot gc.  can we solve it?

could you provide more information such as logs and steps to reproduce?


Fix/settings bugs and added model SettingKeyType typing

This PR fixes various issues related to settings:

setting get cached even there are not saved yet (have no pk) which leads to trying to recreate them which fails
This was pretty hard to find, I located that issue with git bisect using 61481b4eb as bad commit, 0ffe7861a as good commit and got 16ff1936 as first bad commit.
add model key to SettingsKeyType (was actually the initiator of this PR)
plugin slugs that contain - can't use settings because the API re_path doesn't include - chars
related fields cannot be reseted, instead there is an js error thrown that data.value.toString() cannot be called on null
spelling mistakes discovered while debugging in the code (I have this extension installed to prevent them while I'm coding)

@wolflu05 some subtle fixes here, looks good :)


I start a dual-stack UDP listener:
net.ListenUDP("udp", &net.UDPAddr{IP: net.IPv4zero, Port: 0})

Then I send a UDP datagram (from localhost) to this listener and assert that this packet is received. At the same time, I capture all traffic on localhost using Wireshark.
https://gist.github.com/marten-seemann/bfa811133331b9c053137fd5df12638d
What did you see happen?
Running the test 10000 times, I pretty reliably get a test failure on macOS. The datagram is sent (as confirmed by the Wireshark trace), but it never arrives at the listener.
This is not caused by UDP packet loss (which shouldn't happen on localhost anyway). To make sure that this is not the cause of the problem, I added another test (TestUDPUnconnectedDualStackWithRetransmission), which retransmits the  datagram up to 50 times.

This bug is the source of a lot of flakiness in quic-go's test suite. We're running a lot of UDP transfers to test all facets of the protocol, enough to hit this bug on every other CI run or so.
What did you expect to see?
I expect the packet to be received reliably. This test should never fail.

Thanks for the report. Does the failure happens only on macOS? And it doesn't fail on other OSes? Does the failure reproduce if you don't capture traffic in Wireshark?

https://gist.github.com/marten-seemann/bfa811133331b9c053137fd5df12638d

Could you share a full buildable version of the test? Thanks.
cc @ianlancetaylor @neild

Does the failure happens only on macOS?

As far as I can tell, yes. I tested it on Linux and Windows (on CI), and it seems to work there.

Does the failure reproduce if you don't capture traffic in Wireshark?

Yes.

Could you share a full buildable version of the test? Thanks.

Sure! I updated the Gist: https://gist.github.com/marten-seemann/bfa811133331b9c053137fd5df12638d.

From what I can tell, this seems to be a macOS bug. I rewrote the test in C, and I'm seeing the same failure there: https://gist.github.com/marten-seemann/67eecb83006fdc020456821b69112385 (see comment there for how to run it).

This seems likely to be the cause of https://go.dev/issue/29225 as well.


Update office-365-management-activity-api-schema.md

@markjjo

Docs Build status updates of commit 4e82c5c:
:clock10: Pending: waiting for processors (12 builds ahead of you)
:warning: Docs Build is busy, currently there are 12 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

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

Docs Build status updates of commit 4e82c5c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

office-365-management-api/office-365-management-activity-api-schema.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Rust TLS version upgrade

@tusharmath there are some breaking changes in the latest version of rustls which I was able to fix but latest version of hyper-rustls still depends on the old version of rustls. I checked the git repo of hyper-rustls, they have already made the changes according to rustls 0.22.0 in their main branch but it is not published on crates.io. Directly putting the repo url in Cargo.toml fixes the issue but I don't know if we should do that or wait until latest changes are published for hyper-rustls>ASK FOR PERMISSION TO AUTHOR$<

Originally posted by @shashitnak in https://github.com/tailcallhq/tailcall/issues/712#issuecomment-1845325514
—%%%%NO CHANGES%%%%—

@Lo236816 updated the issue title and description to make it more readable.

Hopefully this gets some attention — https://github.com/rustls/hyper-rustls/issues/247


SensitivityLabelId param description in New/Set-UnifiedGroup

Issue #9978

Learn Build status updates of commit 827f9c7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-UnifiedGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-UnifiedGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

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


[Feature] Use for benchmarking agents like AutoGPT?

Hi,
Can this repo be used for testing and benchmarking general agent frameworks like AutoGPT? Are there docs around this or could you point me to the code files I would need to update to make this work?
Thanks!

Hi, @shruti222patel . Yes, it is possible. Writing a new client for each agent framework should be sufficient. You may refer to https://github.com/THUDM/AgentBench/blob/main/docs/Introduction_en.md#23-client. If this does not meet your requirement, you might have to change the core interaction logic in src/assigner.py.


White objects turn black when mobile devices are set to "dark" theme

For example, in the implementation of Go I recently did, the white stones turn black at night when my phone goes into nighttime dark mode. This leaves players with 2 sets of black stones - until the sun comes up and my phone changes back to daytime theme.

Indeed ! Thanks for reporting.

The tokens seem to be white, but the color in the color picker still looks black even though the RGB values show white. So, the color picker is still broken but the tokens work.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Jérémie Pardou-Piquemal<EMAIL_ADDRESS>Sent: Monday, March 1, 2021 4:38:42 PM
To: jrmi/airboardgame<EMAIL_ADDRESS>Cc: dudefaceguy<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [jrmi/airboardgame] White objects turn black when mobile devices are set to "dark" theme (#256)
Can you try this version https://603d5c2a01d02e2754890cfd--airboardgame.netlify.app ?
Also check that the daytime theme still works ;-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jrmi/airboardgame/issues/256#issuecomment-788315222, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATAYYGIDZGQQU72JQBOFUELTBQCOFANCNFSM4YMXNTGQ.

Dark mode is not so easy to bypass, especially with the actual browser support. Do know what is used on your mobile browser to switch to dark mode ? Some browser extension are aggressive on how they change the colors and you have no choice :cry:
I think this is the best I can do for now ;-)
I'll publish my modifications and close this ticket until tools improve.

I'm using Android Firefox with DarkReader. Your solution works well - the color picker is not as important as the pieces themselves.

From: Jérémie Pardou-Piquemal<EMAIL_ADDRESS>Sent: Tuesday, March 2, 2021 3:02 PM
To: jrmi/airboardgame<EMAIL_ADDRESS>Cc: dudefaceguy<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [jrmi/airboardgame] White objects turn black when mobile devices are set to "dark" theme (#256)
Dark mode is not so easy to bypass, especially with the actual browser support. Do know what is used on your mobile browser to switch to dark mode ? Some browser extension are aggressive on how they change the colors and you have no choice 😢
I think this is the best I can do for now ;-)
I'll publish my modifications and close this ticket until tools improve.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jrmi/airboardgame/issues/256#issuecomment-789176444, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATAYYGJBYKVACUBFUXOJYXDTBU75DANCNFSM4YMXNTGQ.


Add README.md deprecation notice

st2workroom is outdated and based on deprecated installer.
Point to st2vagrant repo instead.

Merged, thanks.


Use the cluster name in cluster.yaml in admin.conf

Currently the cluster config is created with cluster name as kubernetes. There is a cluster name in the cluster.yaml metadata.name. We should start using that name for the cluster so tools can be made to target any cluster in kubeconfig file.

cc @jessicaochen @rsdcastro - If no objections, I can start to look into this.

It does make sense to me.

Seems like the cluster name is hardcoded in kubeadm:
https://github.com/kubernetes/kubernetes/blob/999c2ae8716b76099b30da4cea973d0e2c737f77/cmd/kubeadm/app/phases/kubeconfig/kubeconfig.go#L191
There's been a PR to make it configurable, but it's been stale: https://github.com/kubernetes/kubernetes/pull/52470

Perhaps we can work on a new PR that focus on the cluster name to avoid the contention on the other point (user admins)?

I may be wrong but I think it is just the name in the kubeconfig file to reference the cluster. If you change the name in the kubeconfig file or set a new one with the name your want (https://kubernetes-v1-4.github.io/docs/user-guide/kubectl/kubectl_config_set-cluster/), you can still address the same cluster.
In that way, it really does not matter what kubeadm happens to dump as the cluster name on the master.

You are right @jessicaochen. I tried this with s/kubernetes/testcluster on my local kubeconfig file. I don't think we should be doing this in the final product, it's just hacky, and error prone. Instead the cluster name really just should be an option that kubeadm uses to save the original file.

I guess it depends on how important urgent this change is. Whether we want to wait for kubeadm to do it or we want an interim solution.

/label cluster-api

As a prototype we could live with managing a single cluster (or doing something hacky), but at least for an alpha/MVP, a good approach is to support cluster names.
I've already sent them a PR for it.

This issue was moved to kubernetes-sigs/cluster-api#49


How does react-navigation add buttons in the center?

How does react-navigation add buttons in the center?,Thank you!

That's not really supported. If you want a button in the center which navigates to a view without the TabBar, the only possibility would be to use a normal tab which looks like a button, connected to a screen which has tabBarVisible: false.
If you're searching for a way to customize this tab/button, I plan a rewrite of this component, see https://github.com/timomeh/react-native-material-bottom-navigation/issues/31#issuecomment-310229009.

think U!


What's the difference with firebase dynamic links?

On this page https://simonmarquis.github.io/Android-App-Linking you can see the difference between all kinds of links (for Android). This link in referenced in the README.md, under "The Ultimate Guide", you could have see it...


DotDeb proposes PHP7 for Jessie, find a way to use it

See https://github.com/team-rawbot/rawbot-virtualization/pull/42
Thanks @chregu

Thanks @chregu it should now work fine :)


Use vis.js with map

I want to be able to draw a network on a map. Like here http://geojson.io/
Is it possible to do this with vis.js?

Please make sure to read the following list before creating a new issue:

This issue tracker is not supposed to be used for questions on how to use visjs. It is intended to be used for bug reports and feature requests! In case you face yourself with a usage question, then post your question e.g. on stackoverflow tagged with "vis.js".

@albeliakov Everything is possible, but I don't think vis-network is a good fit here!


first commit of preliminary validation

Could you add a more detailed description on what this commit is doing? If you copied some of the files/functions from fbcode, maybe also link the code pointer and the places where you made changes so that the code reviewers will know what additionals were added (hence no need to review the entire files).


Documentation of keyboard shortcuts?

Would it be worth including a section on keyboard navigation?  There are the keyboard shortcuts that are documented in Mastodon itself, and then additional ones that don't seem to be documented anywhere other than in a thread: https://tootplanet.space/@YacYac/100551003949156532
[EDIT:  After some additional poking around, I found this commit https://github.com/cobodo/mastodon/commit/692c7aa8459e4c92a6bf6a65a296f12ab9ac4843, which seems to document all the current hotkeys.]
If this is something you'd like to add, I can write a section up and submit a pull request.

Please feel free! I don't know the keyboard shortcuts well myself, so I'd appreciate the contribution. :)

Since Mastodon has added a list of these in the Getting Started column, I've added a reference to that.


unify workflow file platform naming

Related to #86

Looks cool. I'm accepting this and doing some nitpicking before publishing the new version.


Allow integer renames for struct fields

Closes #1773 and related to PR #2056.
Extends the serde_rename and alias annotations on struct fields to support integer names. Any struct with an integer name is serialized as a map, similarly to structs with any flattened fields, avoiding the need for changes in existing serializer implementations.
Integer names are typed and serialized based on that type, but deserialization is only based on the integer value because deserializers often don't know the exact type of integers.
There are a few unrelated refactors at the start of the series as I was learning the project, then the bulk of the changes are to represent non-string renames (similar to #2056) and to handle the cases of deserialization.

This would be very useful, is there anyway that I could help to move this along if @AndrewScull is too busy?

This is very useful for my specific use case, where I have an enum to be serialized with adjacent tags. The tags however, are numbers.
The data I want to serialize looks like this:
{
  "op": 8,
  "d": {
    "v": 4,
    "heartbeat_interval" :13750.0
  }
}

This is the code that doesn't work unfortunately:
#[derive(Serialize, Deserialize, Debug)]
#[serde(tag = "op", content = "d")]
pub enum DiscordPayload {
    #[serde(rename = "8")]
    Hello(Hello),
}

It expects an identifier rather than an integer
Error deserializing payload: invalid type: integer `8`, expected variant identifier at line 1 column 7

This change does have a valid use case!

@frykher see #2056 for the MR pertaining to enums

I'd love to see this completed. It would be very useful for CBOR specifications that use integers in maps.

As an example of working around this limitation for CBOR encoding, I made this declarative macro to rename the fields to numbers for CTAP2 encoding. You can see it' use in passkey-types::ctap2::make_credential


ACJP-9 does not remove vertices and edges from the base graph when re…

…verting them (causes too many other problems due to NoSuchObjectExceptions)
@adityaburra @ninhnguyen @yetti @sjacob

Build finished.

👍


Gutter click action removes all breakpoints on a line

Set a breakpoint on a line, then set an inline breakpoint once the locations are revealed
Click the line again
All breakpoints in the line are removed, instead of just the line breakpoint. The inline breakpoint should be left intact 🐛

This seems to repro in stable, so not a regression from the context menu changes this iteration AFAICT (unless I've misunderstood the scenario):
https://user-images.githubusercontent.com/30305945/228630958-d87eee01-43b7-4f85-be3a-f01ac3f526ea.mp4
And also seems like it might have always been this way:
https://github.com/microsoft/vscode/blob/666e4172979f1414efdf83979efdbb445a21051c/src/vs/workbench/contrib/debug/browser/breakpointEditorContribution.ts#L303
Though, maybe worthwhile to change to the following since I imagine losing the inline breakpoints on a global gutter action can be frustrating and unexpected:
		const column = e.target.position.column;
		const breakpoint = this.debugService.getModel().getBreakpoints({ uri, lineNumber, column })[0];
		if (breakpoint) {
			this.debugService.removeBreakpoints(breakpoint.getId());
		}

Yea, apologies, I was just looking at this and see it wasn't touched by the context menu stuff.
I'm pretty sure this didn't used to happen 🤔 and if it did, then it shouldn't.

Actually, this is the same as when devtools does. So I think this behavior is fine, but the issue is that we don't show any inline decoration for the breakpoint if it's on the first non-whitespace character of the line. I will make a change so that we do show them inline if there's more than one breakpoint per line so that they can be addressed individually.


Support Weaviate Vector Database

Description
I'd like to use Weaviate as a local database directly in Mirascope to populate prompts for RAG, in a similar way that some other databases are supported.

Closed by #339


Restrict phone number field to the telephone input #192

Resolves #192
Description
This PR is adding validation on phone_number.  Number can not be blank and must be 10 digits long.
Before we move on and add specs we need clarification on the phone number format. Format we added does not allow any space or - between the digits. Once we clarify that is a format we want we will go ahead and add specs for student and user model as well.
Type of change

New feature which validates phone number.

Screenshots

@garettarrowood We added a simple validation for presence and for a length. Before we update the specs we wanted clarification on the phone number format. Right now we don't allow any space  between the digits  or any  -. Is that fine or do we want different format?

Our main branch has gotten a little out of whack.
If you'd like to continue to work on this, please cherry-pick these two commits and open up a fresh PR. Sorry for the inconvenience!
In regards to your question, I believe one solution may be to strip all -, (, ),   and . out of the input, then validate the length of 10 numeric characters.
cc/ @RudeChowder


Change all includes of this library to have prefix cifer

I noticed that there is a problem if you try to use CiFEr in another C application. The headers had no cifer/ prefix, which meant that they weren't found - this was causing errors. I changed the directory structure of the include directory so that all headers are inside a directory named cifer and are also included with the proper prefix.
Perhaps we could also add an application which uses the installed CiFEr library to the tests so that we also test for installation correctness. Might save some headaches in the future.

Looks good, thanks! I will merge.


Update azure-virtual-network-limits.md

Add clarification that the limit of NSG address prefixes applies separately to source and destination.

@anavinahar
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Add more filters

We want to support more filters
Reference:
https://shopify.github.io/liquid/
https://liquidjs.com/filters/overview.html
Source code:
https://github.com/stepci/liquidjs-lite/blob/main/src/filters.ts

I can take this one. I'll do a PR.

See PR #9 , I've implemented most of the Liquid filter functions for strings. I'll also update the README docs.


Reverse Proxy does not work with Odoo /longpoling (website chat)

Nevermind, It was my mistake, I made it work already.
Please, close this ticket. I tried to close it myself but I could not find any option here to close it.

It was a typo in the docker-compose.yml file. After fixing it everything is working nicely.


make a test compile again on Scala 2.13

now that Numeric has a parseString method
reference: https://github.com/scala/community-builds/issues/572

Can one of the admins verify this patch?

Jenkins, ok to test.

Thanks for the fix.
Please improve the commit message. I'd propose:
Fix 2.13 community build: add Numeric#parseString in RangesTest

Numeric#parseString got newly added in 2.13 (see scala/community-builds#572).

Also, this should target the 0.6.x branch. Otherwise we won't be able to test and release future 0.6.x releases for future milestones of Scala 2.13.
It will be forward-merged to master.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/4128/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5495/
Test PASSed.

I have addressed the review feedback.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/4129/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5496/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/4130/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5497/
Test PASSed.


Localization issue

When using another language than "English" -> not working at all, unable to create JournalNotes (ie the tick box is not displayed)

Thank you @sladecraven - will fix

Should be fixed! Please let me know if not


AttributeError: module 'asyncio' has no attribute 'run'

Traceback (most recent call last):
File "openredirex.py", line 126, in
main()
File "openredirex.py", line 106, in main
asyncio.run(redirme(url_list , args.payloads, args.keyword))
AttributeError: module 'asyncio' has no attribute 'run'

Please follow these steps to resolve the error

Install python3.7
sudo apt install python3.7-minimal

Install python3.7 dev
sudo apt-get install python3.7-dev

It will start working :)
Tested on ubuntu 18

Closing this issue as I have updated the entire script. File a new issue if you face any problems running it


Improve CI run time

What
Trello ticket link
Trello ticket link
This PR improves the time it takes our CI tests to run by:

Parallelising the integration tests so that one process runs the cucumber:ok task and one process runs the cucumber:preview_design_system task
Parallelising the MiniTest workflow using a matrix strategy.

Benchmark: Approximately 17m39s (average of the 10 most recent merge to main CI runs)
With the changes proposed, this PR reduces CI run time to approximately 7 minutes.
Why
This is to help reduce developer toil and time to deploy changes. The current run time is excessive and slows down development.
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

I've got some food for thought on this:
With the current implementation, MiniTest tests are manually assigned to nodes. This potentially puts us at risk of not running the full suite: if we've missed some tests, or if future tests get added to locations which lie outside of the assignment. This could be solved by randomly assigning tests across nodes, potentially resulting in a small increase in run time. The random assignment of tests would make it easier to scale up - as a node can be added without worrying about re-allocating tests across nodes - but increases complexity.

this is great 🚀

Seems like a good approach to me. Makes me feel a lot more comfortable about us not missing tests 👍


Community: Load a PDF document as per its Table of Contents

Description: Many PDFs such as textbooks come with a rich Table of Contents (also known as bookmarks). It would be helpful for a RAG system to see the entire content for a ToC entry in tandem, rather than limiting it to a single PDF page!
This PR adds adds a new PDF document loader which loads content of PDF by using its Table of Contents
Many PDFs have detailed Table of Contents. For example this book with the pdf has this Table of Contents structure...

PyPfium2Loader breaks the flow of content which logically continues on the next page.
This PR, loads content based on Toc entry. It can

Extract logical chunks within a single page
Extract logical chunks of document spanning across multiple pages
Is not limited by the page scope

Many PDFs have detailed Table of Contents. For example this book with the pdf has this Table of Contents structure...

PyPffium2Loader which breaks the flow of content which logically continues on the next page as it is part of the same ToC entry.
This PR, loads content based on Toc entry

I don't fully understand what this does (it looks like the docs page you added isn't actually populated), but I think it's unnecessary to add another PDF document loader because we have so many. Depending what the goal of loading in the TOC in this way is, couldn't this be added as some kind of postprocessing on documents in a guide instead?

Closing with recommendation of implementing as:

A guide (if possible to do downstream of PyPDFium2Parser);
Possibly a mode / parameter of the existing PyPDFium2Parser.


Feature: support historic information via services.opengeo.nl

I am aiming for a selection of a minimum date picker, and potentially selecting an available network. For now three files per day are provided.

It would be nice if https://services.opengeo.nl/ptds/ had a JSON file describing the available datasets, otherwise I have to do HTML parsing in Javascript (not fun) if I want to make the date/network picker dynamic
Something like:
[
  {
    "date": "2018-03-01",
    "url": "https://services.opengeo.nl/ptds/2018-03-01/",
    "datasets": [
      {
        "filename": "1-9-11-12-16-17.json",
        "lines": [1, 9, 11, 12, 16, 17]
      },
      {
        "filename": "1-15-19.json",
        "lines": [1, 15, 19]
      },
      {
        "filename": "2-3-4-6-44-83.json",
        "lines": [2, 3, 4, 6, 44, 83]
      }
    ]
  },
  {
    "date": "2018-03-02",
    "url": "https://services.opengeo.nl/ptds/2018-03-02/",
    "datasets": [
      {
        "filename": "1-9-11-12-16-17.json",
        "lines": [1, 9, 11, 12, 16, 17]
      },
      {
        "filename": "1-15-19.json",
        "lines": [1, 15, 19]
      },
      {
        "filename": "2-3-4-6-44-83.json",
        "lines": [2, 3, 4, 6, 44, 83]
      }
    ]
  }
]

Ah well of course I can also hardcode the datasets in the codebase, but it would mean having to update it every time you change something in your service

I'll create a file as such. Might also be a start for a network description.

https://services.opengeo.nl/ptds/index.json
Would an index by date maybe a better idea?

Doesn't make that big of a difference, logically I think an array makes more sense because it encodes the order, and the datasets are instrinsically ordered by date. Objects in javascript do not guarantee the order of the properties, so I would have to take care of that myself

I would say the most important part of this feature was implemented in the now-merged #38.
If we want more advanced parameters for the visualization, we also need a new structure for the index file as discussed

I consider this implemented.


Error: Player command failed: Restriction violated

Hi, currrently getting the following when running the pause operation:
~ ⏎  ❯❯❯❯ spotctl pause -d BedroomBeats
Error: Player command failed: Restriction violated
Usage:
  spotctl pause [flags]

Flags:
  -d, --device string   the name of device
  -h, --help            help for pause

2020/06/28 08:06:31 Player command failed: Restriction violated

I have re-logged in, and still experiencing this error

Nevermind, I see the tocken expired :|
"expiry":"2020-06-27T22:10:02.238049335+02:00"

This is still happening, even after I've refreshed the token by logging out and in again .. and advice/suggestions would be highly appreciated


[ZEPPELIN-2037] Restart button does not work

What is this PR for?
When I enable authentication with shiro or switch interpreter mode to Per user & isolated,  Restart does not work and it is possible to make zombie process.

Environment:
OS: CentOS 6
Zeppelin: master, branch-0.7
Configuration:
Enable authentication with shiro
Set interpreter mode to Per user & isolated(Option)

Test flow:

Execute simple code such as sc.version
Click "Restart" button
Execute simple code again
Shutdown zeppelin
Monitor java processes

There are two issues in Restart button.

closeAndRemoveInterpreterGroupByUser does not work in some situations
When  I enable authentication, then closeAndRemoveInterpreterGroupByUser will be called by clicking "Restart" button. The implement of closeAndRemoveInterpreterGroupByUser call dereference  only once, if remoteInterpreterProcess.referenceCount number larger than 1, then Restart button does not work.

closeAndRemoveInterpreterGroupByNoteId  cloud not get correct processKey
When I switch interpreter mode to Per user & isolated, closeAndRemoveInterpreterGroupByNoteId will be called by clicking "Restart" button.
And I will get intpKey=$username: and processKey=:, therefore, isEqualInterpreterKeyProcessKey be false forever.

I fix this issue by simple ways, but I only test spark, python, livy interpreters, maybe other interpreters have the same problem.
What type of PR is it?
Bug Fix
Todos

[ ] - Add unit test

What is the Jira issue?
ZEPPELIN-2037
How should this be tested?

Execute %spark sc.version on zeppelin
Click Restart button which in Notebook page or Interpreter Page
Monitor java processes by jps

Screenshots (if appropriate)
Questions:

Does the licenses files need update?
Is there breaking changes for older versions?
Does this needs documentation?

@cloverhearts Can you check it?

@jongyoul sure :)

@cloverhearts  Thanks for your review.
I seem closeAndRemoveInterpreterGroupByUser has been applied by https://issues.apache.org/jira/browse/ZEPPELIN-1770
I think closeAndRemoveInterpreterGroupByUser is most important feature on the multiple user environment, my users need to restart interpreter by themselves.
But the implementation of closeAndRemoveInterpreterGroupByUser cloud not do it now.
Did I miss something about this issue?

Thank you for your reply, @del680202
I understand your publicity purpose.
And I think it's right.
What I am worried call "terminateRemoteProcess ();" From "public void close (String sessionId) in InterpreterGroup.java".
"Close (String sessionId)" should only perform the action corresponding to the session.
And if all sessions are closed, the process must be terminated.
However, the current code always performs "terminateRemoteProcess ();" Within "close (String sessionId)".
"TerminateRemoteProcess ();" The function forces all sessions to be terminated and the process is terminated.
I think this is the wrong behavior of "close (String sessionId)".
We think this can be a problem, because we can write several users with an interpreter.

@cloverhearts Thanks for your advice
I will try to find the other way to solve this problem

Is it caused by ZEPPELIN-1921 ? It works in zeppelin 0.7 but not in master.
BTW, I think these methods are a little confusing, we might need to improve them later.  @jongyoul is working on refactor the interpreter component, we might also need to consider to clean up the APIs that those components should expose. (InterpreterFactory, InterpreterSetting, InterpreterGroup)

@del680202

When I enable authentication with shiro or switch interpreter mode to Per user & isolated, Restart does not work and it is possible to make zombie process.

This issue has been solved by #2149. And the issue status was updated to RESOLVED in https://issues.apache.org/jira/browse/ZEPPELIN-2037. Can you check again?

@AhyoungRyu  Thanks, I will check it.

@AhyoungRyu
I test #2149 in my machine.
It works well, Thanks.
But I seem when I edit&save interpreter config then the zombie process will be occurred.
Anyway, Restart button looks well, I close this PR, thanks.
Test cases:
Global mode
Restart by button in notebook page:  SUCCESS
Restart by button in interpreter page: SUCCESS
Per user & isolated mode
Restart by button in notebook page:  SUCCESS
Restart by button in interpreter page: SUCCESS

@del680202 Thanks for your time and effort to test it out.

But I seem when I edit&save interpreter config then the zombie process will be occurred

I checked it with current branch-0.7 and I could see the interpreter process was killed.
Actually just tested in basic case (anonymous / global mode) with Spark interpreter.

Run some paragraphs \w Spark intp
Edit zeppelin.spark.maxResult from 1000 to 20  in the interpreter setting page  -> save
The remote interpreter process was killed

Was your case different with mine?

@AhyoungRyu  Thanks for your reply.

But I seem when I edit&save interpreter config then the zombie process will be occurred

Sorry, I miss this test case.
I test it by current branch-0.7 too.
My test steps as below:

Execute sample code with %python in global mode.
Switch python interpreter to per user mode
Restart/Shutdown Zeppelin

It looks like a issue of switching interpreter mode function.

@del680202  I can not reproduce the issue on 0.7.1 rc2, regarding your instruction, when you do step 2, zeppelin would ask you to restart the interpreter, did you click the button to restart it ?

@del680202 Also tested on both latest branch-0.7 and rc2 but couldn't reproduce. Here is my steps.

Activate shiro -> login
Run a paragraph with python intp(globally shared) -> 1 RemoteInterpreterServer shows up
Edit python interpreter as "Per User / isolated" -> Click save btn (in this step the interpreter will be restarted automatically) -> the previous RemoteInterpreterServer was killed

I checked jps in every steps, but couldn't see zombie remote interpreter process. Can you check again you were testing in latest branch-0.7? Or you can simply check it with 0.7.1-rc2 binary[1] as @zjffdu did.
[1] https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.7.1-rc2/

@zjffdu  @AhyoungRyu
Yes, I tested it by latest branch-0.7.

I tested this problem with %python  and %pyspark at the same time

Enable shiro for login
Execute sample code by %python and  %pyspark
.
Switch spark interpreter from global mode to Per user mode and save it.
Switch python interpreter from Per user mode to global mode save it.
Shutdown zeppelin

It was success when I change spark interpreter setting, but it make zombie process when I change python interpreter setting.

I'm testing it now and found something weird.

@del680202 I've made a PR for handling what you found. Can you please test it with that PR? #2191

@jongyoul Thanks, I will test it later

@del680202 Thanks, if you finish to test it, we will make a new rc including it.


Deleted machines don't show up in discovery again

While setting up my test cluster at home I realised that if I add a MAC to a new machine, then delete the machine, then plug the MAC again, it won't show up in the discovery list for the second time.
I don't know exactly what's going on, but it seems that deletion is not clearing everything (maybe related to #93?) and this not showing up in the discovery (because it has "already seen it").
After #93 is fixed, I'll try again, and if the issue is solved, I'll close this one.

#93 has nothing to do with networking.

Yup, figured out after I said it. :)


Release 0.17.1

Scheduled release date: April 16, 2024
This is a tracker to aggregate issues that should be part of 0.17.1.
Please add specific Issues or PRs prefixed with the "Depends on" keyword
(and remember that PRs need to be backported to the release-0.17 branch).
Make sure to follow the manual steps listed below before starting the release process.
Manual Steps
For detailed information regarding each step, please check: https://submariner.io/development/building-testing/ci-maintenance/

[ ] Custom GitHub Actions
[ ] GitHub Actions
[ ] Kubernetes Versions
[ ] Shipyard Linting Image Software

Depends on https://github.com/submariner-io/lighthouse/pull/1528

Released: https://github.com/submariner-io/releases/releases/tag/v0.17.1


Rename src/cc_catalog_airflow/dags/provider_api_scripts/modules to be more descriptive

We will also have to rename a large number of imports when we do this, since modules.etlMods is used in nearly all Provider API Scripts.

This will be addressed by the Provider API Script rewrites.
See #238 #247 #252 #253 #254 #255 #256


chore(deps): Update Guava in Firestore and Google Sheets plugins

Update Guava to v32, to get fix for CVE-2023-2976.

/ok-to-test

/build-deploy-preview skip-tests=true

/build-deploy-preview

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true


Command pviews doesn't work in Xcode 7.3

i just install chisel, i try other commands like pvc work fine, but pviews doesn't work, nothing print and nothing running...can someone help?

https://github.com/facebook/chisel/issues/149
Fixed in Xcode 7.3.1

Thanks for the report @kenny1269. As @khramtsoff pointed to, the problem (which is an Xcode bug) is fixed in 7.3.1.


Document how to execute a command in a pod with the Rancher API

This issue represents a small piece of the work in https://github.com/rancher/rancher-docs/issues/173

Consolidating into https://github.com/rancher/rancher-docs/issues/583.


Add documentation about fine grained webhook token

Fixes #1037
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>
Changes

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

Codecov Report

Merging #1038 (805608c) into main (12dcb84) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1038      +/-   ##
==========================================
- Coverage   61.36%   61.28%   -0.08%
==========================================
  Files          99       99
  Lines        6041     6024      -17
==========================================
- Hits         3707     3692      -15
+ Misses       1986     1985       -1
+ Partials      348      347       -1

Impacted Files
Coverage Δ

pkg/resolve/resolve.go
88.54% <0.00%> (-1.28%)
:arrow_down:

pkg/adapter/tls.go
100.00% <0.00%> (+11.76%)
:arrow_up:

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

good to go for y'all ?


modify 2.6.5 release log (replace pr link with issue link)

What is the purpose of the change
replace pr link with issue link
Brief changelog
replace pr link with issue link

Codecov Report

Merging #2716 into 2.6.5-release will increase coverage by 0.04%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##             2.6.5-release    #2716      +/-   ##
===================================================
+ Coverage            47.35%   47.39%   +0.04%
- Complexity            4283     4286       +3
===================================================
  Files                  554      554
  Lines                24366    24366
  Branches              4305     4305
===================================================
+ Hits                 11538    11549      +11
+ Misses               11044    11036       -8
+ Partials              1784     1781       -3

Impacted Files
Coverage Δ
Complexity Δ

...ba/dubbo/remoting/transport/netty/NettyServer.java
67.85% <0%> (-3.58%)
8% <0%> (-1%)

...baba/dubbo/remoting/transport/mina/MinaClient.java
59.37% <0%> (+1.56%)
9% <0%> (+1%)
:arrow_up:

.../com/alibaba/dubbo/monitor/dubbo/DubboMonitor.java
89.71% <0%> (+1.86%)
15% <0%> (ø)
:arrow_down:

...rpc/protocol/dubbo/telnet/InvokeTelnetHandler.java
57.83% <0%> (+3.61%)
15% <0%> (+2%)
:arrow_up:

...va/com/alibaba/dubbo/config/DubboShutdownHook.java
73.91% <0%> (+30.43%)
6% <0%> (+1%)
:arrow_up:

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 c84d79f...4d6c44a. Read the comment docs.


Missing private endpoint for CosmosDB PostgreSQL Cluster

The listed private endpoints here, are missing the "coordinator" sub-resource, for CosmosDB PostgreSQL Cluster.

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

ID: b006a616-2f2f-8ecf-8b80-fd0a3cc62090
Version Independent ID: 92c54dc6-8ddf-b1f7-4a58-d61b8cd7fe25
Content: What is a private endpoint? - Azure Private Link
Content Source: articles/private-link/private-endpoint-overview.md
Service: private-link
GitHub Login: @asudbring
Microsoft Alias: allensu

@DevMan01
Thanks for your feedback! We will investigate and update as appropriate.

@DevMan01 , I've updated the missing private endpoint for CosmosDB PostgreSQL Cluster as below:

The changes will reflect soon.


Add posix_spawn bindings

@alexcrichton Were you working on using posix_spawn in Command or just adding the bindings? I get significant performance improvement on FreeBSD by using vfork, so was looking into writing posix_spawn support.

@bdrewery initially yeah I was curious if we could implement an optimization for Command to at least attempt to use posix_spawn for some cases, but unfortunately the semantics of posix_spawn on a nonexistent executable are different than fork/exec, so it ended up being a dead end for libstd.
That being said, I think it'd be great to implement this in crates.io!

@alexcrichton What's the difference you're seeing?

Command::new("nonexistent").spawn() returns an error today, but with posix_spawn it returns a success and the call to wait() fails. I wasn't sure how you could actually tell whether the process exec'd correctly or not.

I think we could just try_wait (to use waitpid(WNOHANG)) after and check for exit status 127 here https://github.com/rust-lang/rust/compare/master...alexcrichton:posix-spawn#diff-86e199423278a641f73ed69c42c9f733R40 and return the expected error. Some platforms would allow skipping that such as FreeBSD which just returns ENOENT in errno after posix_spawn.

For context I picked up Rust just last week and the first thing I prototyped hit major overhead in fork on FreeBSD https://github.com/rayon-rs/rayon/issues/540 which vfork or posix_spawn fixes. So I'd really like to see this fixed by default for FreeBSD at least.

@bdrewery isn't try_wait racy though? We wouldn't know if the process exited quickly or if it failed to spawn I'd imagine. Additionally would that be guaranteed to catch all process spawn failures?

@alexcrichton yeah true. At least on FreeBSD and DragonFlyBSD because vfork is used, the parent process is suspended until the exec; but a ENOENT/ENOEXEC is returned for those platforms anyway. OpenBSD doesn't use vfork, and NetBSD's syscall implementation doesn't appear to have the suspend behavior always. Using posix_spawn for FreeBSD/DragonFlyBSD rather than vfork is still preferred since there's some special handling for threads in FreeBSD's posix_spawn that could not be reproduced otherwise. I'm going to take your commit and polish it on FreeBSD.

@alexcrichton looking over glibc it looks like its latest posix_spawn can return ENOENT in the case you're talking about as it uses a pipe to send the error code back up to the parent https://github.com/bminor/glibc/commit/9ff72da471a509a8c19791efe469f47fa6977410#diff-35e1fecdd831511357340526ff4c5bacR285. There are also some tests for the behavior: https://github.com/bminor/glibc/commit/9ff72da471a509a8c19791efe469f47fa6977410#diff-d5ff29c03cf203a4861e0ed3e2ec63ddR41
https://sourceware.org/bugzilla/show_bug.cgi?id=10354#c12
      2. Child must synchronize with parent to enforce stack deallocation and
         to possible return execv issues.

I guess the only problem with Linux now would be ensuring its glibc has this newer posix_spawn at runtime.
I made a few changes to your implementation (namely the *sys::os::environ one was needed and no need to zero objects which have their own initializers):
diff --git src/libstd/sys/unix/process/process_unix.rs src/libstd/sys/unix/process/process_unix.rs
index d66c237514..fa66245abb 100644
--- src/libstd/sys/unix/process/process_unix.rs
+++ src/libstd/sys/unix/process/process_unix.rs
@@ -24,7 +24,6 @@ impl Command {
                  -> io::Result<(Process, StdioPipes)> {
         use sys;

-
         const CLOEXEC_MSG_FOOTER: &'static [u8] = b"NOEX";

         let envp = self.capture_env();
@@ -237,7 +236,7 @@ impl Command {
     }

     #[cfg(not(any(target_os = "linux", target_os = "macos")))]
-    fn posix_spawn(&mut self, stdio: &ChildPipes, envp: Option<&CStringArray>)
+    fn posix_spawn(&mut self, _stdio: &ChildPipes, _envp: Option<&CStringArray>)
         -> io::Result<Option<Process>>
     {
         Ok(None)
@@ -280,8 +279,8 @@ impl Command {
         }

         unsafe {
-            let mut file_actions = PosixSpawnFileActions(mem::zeroed());
-            let mut attrs = PosixSpawnattr(mem::zeroed());
+            let mut file_actions = PosixSpawnFileActions(mem::uninitialized());
+            let mut attrs = PosixSpawnattr(mem::uninitialized());

             libc::posix_spawnattr_init(&mut attrs.0);
             libc::posix_spawn_file_actions_init(&mut file_actions.0);
@@ -302,7 +301,7 @@ impl Command {
                                                            libc::STDERR_FILENO))?;
             }

-            let mut set: libc::sigset_t = mem::zeroed();
+            let mut set: libc::sigset_t = mem::uninitialized();
             cvt(libc::sigemptyset(&mut set))?;
             cvt(libc::posix_spawnattr_setsigmask(&mut attrs.0,
                                                  &set))?;
@@ -315,7 +314,7 @@ impl Command {
             cvt(libc::posix_spawnattr_setflags(&mut attrs.0, flags as _))?;

             let envp = envp.map(|c| c.as_ptr())
-                .unwrap_or(sys::os::environ() as *const _);
+                .unwrap_or(*sys::os::environ() as *const _);
             let ret = libc::posix_spawnp(
                 &mut p.pid,
                 self.get_argv()[0],

As a sidenote it turns out that posix_spawn still has like 12% overhead vs vfork on FreeBSD due to the file actions using malloc. That's an implementation detail that we'll improve in FreeBSD's libc though.


docs: terminate third sprint

:tada: This PR is included in version 0.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


setting aws_account_id doesn't ensure all resources are created in that account

I am trying out this template for EKS cluster creation right now.
While doing the apply, i was wondering why my vpc endpoint resources did not show up in the new subaccount that i created.
Turns out they were created in the main account i was using, even though i set aws_account_id to the sub account.
That is not ideal or obvious to a new user and i assume it also is a bug?
These resources i can see in my main account, which should be in the new sub account instead:

vpc
subnets
routing tables
igw
egress igw
eip
endpoints
nat gw

I see that it says in the requirements [AWS CLI](https://docs.aws.amazon.com/cli/latest/userguide/getting-started-install.html) configured with the account you want to deploy into, however my assumption was that my profile should have the permissions needed to create the resources.
Why else would there be a aws_account_id variable?
It seems i will have to use the iam_role option then to enforce where to spawn the resources, will check that out.
When destroying the incorrectly created resources, i do now get:
╷
│ Error: expected "url" url to not be empty, got
│
│   with data.flux_sync.main[0],
│   on flux2.tf line 103, in data "flux_sync" "main":
│  103:   url         = local.flux2["github_url"]
│
╵
╷
│ Error: error reading EKS Cluster (cluster-name): couldn't find resource
│
│   with data.aws_eks_cluster.cluster,
│   on provider-local.tf line 33, in data "aws_eks_cluster" "cluster":
│   33: data "aws_eks_cluster" "cluster" {
│
╵

not ideal, because we wanted to use flux2 without github.
Will try that again with a demo url set.
Setting a demo url did actually not let me remove the resources, so i manually removed them.

@applike-ss Hi, as you might have notice in https://github.com/particuleio/teks/pull/133, to enable multi-account support, you need to use a dedicated assume role in the terragrunt.hcl in each environment

Thanks, that's what i'm trying right now.
Having some issues still like #134, however i am refactoring it a bit so i don't actually need the kms module.

i got my cluster up and running, though not with all the addons that it should have. closing it and creating a new one for the current issue


MomentJS dependency

Can't the momentjs dependency be removed? It seems to be used for simple 3 date functions which are supported by the default Date() object in javascript, while using 500kb of space.

This has been fixed. mermaid uses moment-mini instead.


bug: respect cursor's shape

Did you check docs and existing issues?

[X] I have read all the noice.nvim docs
[X] I have searched the existing issues of noice.nvim
[X] I have searched the exsiting issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.0-dev
Operating system/version
archlinux latest
Describe the bug
Whenever i use this plugin, my cursor shape change from _ to block, i force set cursor shape from alacritty and not set it on nvim.
how to preserve the cursor shape?
Steps To Reproduce
just use this plugin
Expected Behavior
cursor shape not change when using this plugin
Repro
No response

Noice uses your Neovim cursor settings, not sure how that works with Alacritty. Noice supports Neovim cursors.

@folke : there's something that reset cursor shape to block, like:
1. map / to :silent! /
2. nvim
3. press /
4. press <esc>
5. :qa
6. now nvim exit and user dropped to shell with a block cursor shape

Is there a way to get insert mode cursor shape in the fine cmdline box?

@folke : there's something that reset cursor shape to block, like:
1. map / to :silent! /
2. nvim
3. press /
4. press <esc>
5. :qa
6. now nvim exit and user dropped to shell with a block cursor shape

It's possible to write a script to avoid it.

https://github.com/folke/noice.nvim/issues/552

#552

So isn't there any way to set the cursor shape? so the only thing I can to avoid from changing the cursor shape of the shell might be write a script like echo -n $'\e[0 q'


Theme only applies to code edtior (not status bar or background of sidebar/tabs)

Not sure if I'm doing something wrong, but unlike on the screenshots here, the theme only applies to the code editor for me, all other colors stay the default light/dark.
I'm setting the theme with the command 'Preferences: Color Theme', do I need to do something else to have it apply globally?

I'm not sure what's wrong, but let's try to look into this.
Do you have any other extension that's affecting the workbench?
Could you show your full settings.json (for both user & workplace) so that I could have a look.

Hi, thanks for taking the time to reply!
My user settings.json:
{
    "workbench.activityBar.visible": false,
    "workbench.colorTheme": "Gruvbox Dark (Hard)",
    "window.menuBarVisibility": "toggle",
    "editor.minimap.enabled": false,
}

The workspace settings are empty.
I just disabled all other extensions and reloaded, no change.
Maybe I should have mentioned: I'm not using the Microsoft binary release of vscode, I am building from source according to their instructions, I just built it from the master of https://github.com/Microsoft/vscode again to make sure I have the latest version, it says version 1.29.0 in the about menu, do you think that could be the issue?

Thanks for your work on this extension, looks really good!


Markdown not being parsed

I'm trying to set up a very basic test page using hologram.
When I run hologram, it doesn't seem to parse my markdown syntax in the comment section of the scss.
Running off of what the hologram init gave me.
hologram_config.yml:
# Hologram will run from same directory where this config file resides
# All paths should be relative to there

# The directory containing the source files to parse recursively
source: ./wwwroot/dist

# The directory that hologram will build to
destination: ./docs

# The assets needed to build the docs (includes header.html,
# footer.html, etc)
# You may put doc related assets here too: images, css, etc.
documentation_assets: ./doc_assets

# The folder that contains templates for rendering code examples.
# If you want to change the way code examples appear in the styleguide,
# modify the files in this folder
code_example_templates: ./code_example_templates

# The folder that contains custom code example renderers.
# If you want to create additional renderers that are not provided
# by Hologram (i.e. coffeescript renderer, jade renderer, etc)
# place them in this folder
# code_example_renderers: ./code_example_renderers

# Any other asset folders that need to be copied to the destination
# folder. Typically this will include the css that you are trying to
# document. May also include additional folders as needed.
dependencies:
    - ./wwwroot/dist

# Mark which category should be the index page
# Alternatively, you may have an index.md in the documentation assets
# folder instead of specifying this config.
# index: test

# To additionally output navigation for top level sections, set the value to
# 'section'. To output navigation for sub-sections,
# set the value to `all`
nav_level: all

# Hologram displays warnings when there are issues with your docs
# (e.g. if a component's parent is not found, if the _header.html and/or
#  _footer.html files aren't found)
# If you want Hologram to exit on these warnings, set the value to 'true'
# (Default value is 'false')
exit_on_warnings: false

site.scss:
$test-color: red !default;

/*doc
---
title: First
name: first
category: basics
---
    ```html_example
    <div class="test">test</div>
    ```

    ## Test
    ##Test
*/
.test {
    color: $test-color;
}

The result:

Am I missing something? Seems like this should be working.

I think your markdown is indented too far and is being viewed as code, does this work:
$test-color: red !default;

/*doc
---
title: First
name: first
category: basics
---

```html_example
<div class="test">test</div>
```

## Test
##Test
*/
.test {
    color: $test-color;
}

Well, color me embarrassed. That's all it was. Much appreciated. VS did some auto-indentation, so I figured it was right... Figured Microsoft was right... First mistake ;) Thanks!