3.0 Bugs

This issue exists to keep track of packages that are currently failing to install with the dev branch of Commando and of bugs that currently need fixing prior to a public release.
Broken packages:

[ ]  putty.vm - seems like an upstream issue. choco install throws an error about invalid xml structure.
[ ]  metasploit.vm - checksum mismatch due to rolling installers. need to find a stable build link.
[ ]  hashcat.vm - fails on AMD processors. need to add compatibility.
[x]  lockhunter - upstream issue with the chocolatey package. won't fix.
[x] SublimeText3.PackageControl - upstream issue with the choco package. won't fix.
[ ] Add metapackages that remove shortcuts for some programs:

[ ] Google Chrome
[ ] Firefox
[ ] Obsidian
[ ] VSCode
[ ] VLC
[ ] Acrobat
[ ] Tor Browser (metapackage exists, needs a fix)

Bugs:

[x] Disable QuickEdit and Insert PowerShell modes to avoid the install freezing until a keypress
[ ] Pin a shortcut to PsExec to Lateral Movement from Sysinternals
[x] Standardize PowerShell tool shortcuts (.lnk, not ps1.lnk) (see PR in VM-Packages)
[x] Abort installation if the checks window is closed
[x] Remove the disk space requirement message for the custom profile
[x] Fix the profile manager erroring when continuing without a custom profile present

Categories that require more packages (solicit requests and add):

[ ] Privilege Escalation
[ ] Lateral Movement
[ ] Persistence

Closing due to a migration to a kanban internally


Figure out how to linting / config validation for terraform files in this template

Feature Request
Description of Problem:
...what problem are you trying to solve that the project doesn't currently solve?
...please resist the temptation to describe your request in terms of a solution.  Job Story form ("When [triggering condition], I want to [motivation/goal], so I can [outcome].") can help ensure you're expressing a problem statement.
Potential Solutions:
...clearly and concisely describe what you want to happen. Add any considered drawbacks.
... if you've considered alternatives, clearly and concisely describe those too.

I vote for terraform validate to be put as a githook in the template.

Having looked again at the unit testing, I vote for Kitchen Terraform:

Terratest requires an understanding of go-code to test terraform code; while I openly detest yaml, it's easier to manage than a compiled lang
The maintainer of Kitchen TF also works for HashiCorp, so comms might be easier in the interim.


Select drive as source

This is a three-purpose feature request.
Overtly, it's about having the ability to select a drive as a source. So, imagine having two sd drives connected, and selecting the one as a source, and the other as a destination. I think this is a perfectly legitimate Etcher feature. Interestingly, a system drive would be a legitimate thing to select as a source in this case.
Covertly, this is about allowing the selection of things "other than files from the filesystem" as sources. In the future we could add urls, torrents, and other things as sources. If we work out the interaction for having files and drives, we can extend it to other things as they become available.
Even more covertly, this is one of the features we'd need for a hardware version of Etcher. ;)

I like this idea. In order to achieve it, we should first sort out how to determine how much of the drive to copy, which is exactly what we need for the backup feature as well (https://github.com/resin-io/etcher/issues/266).
I'll mark this issue as "pending design", so we can start figuring out how to properly present this paradigm shift to the user while we sort out the technical side in the backup ticket,

/cc @konmouz

Theoretically, if you always represent these data sources/destinations as abstract streams, would you just be able to drop-in a disk stream in place of a file one (or a Node.js buffer or whatever internal structure you use)? My thinking is essentially that you could have both the source and destination be completely generic internally and then just specialize the UI to facilitate selection. With something like this, it would presumably be easy for someone to implement a new source or destination, because you wouldn't need to adapt internal switching logic.

@WasabiFan Yeah, and that's kinda covered by the etcher-image-write and etcher-image-stream modules as mentioned in https://github.com/resin-io/etcher/blob/master/docs/ARCHITECTURE.md
I agree with @jviotti that this PR sounds very much like a logical follow-on from #266

To me it sounds like a precursor. If you had a drive as a source and a file as a destination, voilà, backup

We can change the UI from 'Image' and 'Drives' to 'Source' and 'Destination'. Probably when you select a drive as source, it should look unavailable on the destination side. Regarding backup, it is true that it will be covered via this feature, however, it might worth having a 'backup' CTA as a more direct approach (mainly for new users).

agreed that there may be a need for a backup side-flow even if one could
put it together with these primitives.
having abstract source and destination/target selectors was one of the
earliest things we were talking about with @jviotti and I kinda let it drop
so we could focus on the core, interesting to see it coming back, means it
wasn't a terrible idea, probably.
--
Alexandros Marinos
Founder & CEO, Resin.io
+1<PHONE_NUMBER>
@alexandrosm
On Wed, Dec 7, 2016 at 9:03 AM, Konstantinos Mouzakis <
<EMAIL_ADDRESS>wrote:

We can change the UI from 'Image' and 'Drives' to 'Source' and
'Destination'. Probably when you select a drive as source, it should look
unavailable on the destination side. Regarding backup, it is true that it
will be covered via this feature, however, it might worth having a 'backup'
CTA as a more direct approach (mainly for new users).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/resin-io/etcher/issues/950#issuecomment-265393015,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLUCLRf85xmvrsQ90PHSe-0spCZ4_o6ks5rFnZOgaJpZM4LFwk3
.

I guess when selecting the 'Destination' as a file though, you'd only want to be able to select a single file, rather than selecting multiple files as in the multi-write mode. And it probably doesn't make much sense to read from a file and write to another file (unless you wanted to specifically enable that as a verification / testing mode).
So I guess the allowed combinations would look something like:

read-from-file -> write to one or more drives (typical disk-flashing mode)
read-from-drive -> write to one or more drives (disk-cloning mode)
read-from-drive -> write to a single file (backup mode)

Probably when you select a drive as source, it should look unavailable on the destination side.

Agreed. And when selecting a file as destination, you'd need to make sure it's not on the drive you're going to be reading from (which we can do, now that drivelist reports which mount-points each drive uses).
When reading-from-drive we'll need to unmount the drive first (so that we can access it in raw mode), so it might be a nice feature to re-mount the drive again at the same mount-point(s) after we've finished reading from it? (if it's possible to do that reliably)
Hmm, and perhaps that means we need another category of drives for drivelist to detect - drives that we "know" we can't unmount (which will include at a minimum the "boot drives"), and therefore will never be able to read from?

I think we should really consider this paradigm change now, since it means the whole Etcher workflow would change, and better to at least consider how this would look like before adding more features, otherwise it will be a pain to introduce later on.

read-from-file -> write to one or more drives (typical disk-flashing mode)

read-from-drive -> write to one or more drives (disk-cloning mode)
read-from-drive -> write to a single file (backup mode)

Just to add 'read-from-online source -> write to one or more drives' which I guess UI-wise would be slightly different from 'read-from-file -> write to one or more drives'.
Also, if it makes sense 'read-from-online source -> write to a single file'. Well, this sounds like download, so let me know if there is such a use case.

Not sure if 'read-from-online source -> write to a single file' "works" as a concept, but when doing 'read-from-online source -> write to one or more drives' it might be nice to have an option to cache (or save) the image file (and its metadata?) locally, so that if you want to flash an online-image multiple times, you can just use the saved-version instead of streaming it again.

Also, if it makes sense 'read-from-online source -> write to a single file'. Well, this sounds like download, so let me know if there is such a use case.

I think we can leave that out for now.

I am not sure why we are enumerating combinations if the two sides will be
independent?
--
Alexandros Marinos
Founder & CEO, Resin.io
+1<PHONE_NUMBER>
@alexandrosm
On Wed, Dec 7, 2016 at 2:33 PM, Juan Cruz Viotti<EMAIL_ADDRESS>wrote:

Also, if it makes sense 'read-from-online source -> write to a single
file'. Well, this sounds like download, so let me know if there is such a
use case.
I think we can leave that out for now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/resin-io/etcher/issues/950#issuecomment-265462019,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLUCBSAyHAN_yb7rqdFHyCHik5O3SABks5rFsPDgaJpZM4LFwk3
.

I guess for purposes like if you select an A drive as source, probably A will be unavailable on the destination side. Am I wrong?

sure, just want to make sure we are thinking about the two things as
independent and not tying ourselves to specific patterns.
maybe we should list the special cases instead (like the one you mentioned
above)?
--
Alexandros Marinos
Founder & CEO, Resin.io
+1<PHONE_NUMBER>
@alexandrosm
On Thu, Dec 8, 2016 at 10:33 AM, Konstantinos Mouzakis <
<EMAIL_ADDRESS>wrote:

I guess for purposes like if you select an A drive as source, probably A
will be unavailable on the destination side. Am I wrong?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/resin-io/etcher/issues/950#issuecomment-265707996,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLUCFLANjghbZV4hnMAcr4ws99MJmtdks5rF90DgaJpZM4LFwk3
.

For v.01, just to double check:

Can a destination be online?
Can you edit the files of a connected device or drive? Which means that source and destination is the same. Probably this is configuration.

Sorry if I was causing confusion. Yeah the two sides are "independent", but there are some combinations that don't make sense, so I was just trying to whitelist the valid combinations, rather than blacklist the invalid combinations.
In the current version of Etcher, the only "source" is "local file" and the only "destination" is "local drive".
I guess our currently-planned "sources" are: "local file", "online stream", "local drive"; and our "destinations" are: "local file", "local drive(s)". (But obviously there's nothing precluding other options being added to either of these lists).

I would replace "online stream" with simply "url" but otherwise yeah, it
all checks out. In cases like this I generally prefer blacklists than the
other way around, as they are true-er to the generative nature of
independent primitives.
--
Alexandros Marinos
Founder & CEO, Resin.io
+1<PHONE_NUMBER>
@alexandrosm
On Thu, Dec 8, 2016 at 11:21 AM, Andrew Scheller<EMAIL_ADDRESS>wrote:

Sorry if I was causing confusion. Yeah the two sides are "independent",
but there are some combinations that don't make sense, so I was just trying
to whitelist the valid combinations, rather than blacklist the invalid
combinations.
In the current version of Etcher, the only "source" is "local file" and
the only "destination" is "local drive".
I guess our currently-planned "sources" are: "local file", "online
stream", "local drive"; and our "destinations" are: "local file", "local
drive(s)". (But obviously there's nothing precluding other options being
added to either of these lists).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/resin-io/etcher/issues/950#issuecomment-265717510,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLUCPR2dawQYysFs3oQpuPrfYMSHLtGks5rF-hQgaJpZM4LFwk3
.

regrading "online stream", except of a url, would it be possible to have a dropdown/ filter search UI element to select an image from an online source? At least for popular images would be great. (e.g. type 'Ras' and get a list of different Raspbian Image Versions)

Fair enough. In that case the blacklist is:

local file -> local file
url -> local file

(and with the additional filtering that @konmouz mentioned of not being able to select the same local-drive as both source and destination; and if doing local-file -> local-drive or local-drive -> local-file we should ensure that the local-file isn't on a filesystem that's on the drive being written to or read from)

would it be possible to have a dropdown/ filter search UI element to select an image from an online source?

Yeah, something like that is planned in the Etcher roadmap :)  @alexandrosm might be able to find up the doc and send it to you on flowdock.

Done in the latest SDK


I am new to Skywalking . Want to install Skywalking and other agents

Hi Folks ,
I am new to skywalking and i want to install Skywalking and other agents like which can send logs, metrices and traces . Please help me to install all components and test it end to end . From official documentation i am unable to understand.
Thanks
Sajal

I am not sure what do you know unable to understand.

What would be installation steps to see metrices , logs and traces and Skywalking UI ?
Like what should i install first in AKS cluster apart from UI ?
 Take one at a step. All these you mentioned are not in one documentation.

Can you please just guide the steps as i am really very confused that what should be the steps ?
I am very new to this field. Please help me with the steps

What do you want to do with step one? You are listing many things at one time.

My bad . If i say that i want to start using Skywalking then what should be my first step for it's installation.

Eg : If i want to test traces in SW UI then what all should i install step by step ?

Set up OAP and UI, and install our javaagent by using -javaagent:.

Agent, https://skywalking.apache.org/docs/main/latest/en/setup/service-agent/java-agent/readme/
OAP + UI, https://skywalking.apache.org/docs/main/latest/en/setup/backend/backend-setup/

Thanks a lot wu-sheng . So step 1 and 2 will help me to collect only traces right ?

SkyWalking agent will help you on trace, metric, topology and even alarm. Some plugins could do log collecting.
Read the whole doc, it helps.

Thanks a lot Sir. I was able to get metrices and topology of bookinfo application. I followed steps from the documentation and was able to do. It's amazing.

Hi Sir ,
How to get K8s monitoring in Skywalking . I want cluster info like nodes , namespaces . How should i proceed ?

We have provided node tab, what is your question?

I want to see my Kubernetes cluster information like Cluster , node and service . But i am not getting that in SW UI. For that can you help me how can i get that ?
My SW UI that i had installed .

Maybe you are

Using an old version, upgrade the version. 8.5.0 began to provide k8s monitoring.
You used to set up an old version. Clean your browser local storage and cache.

where can i provide K8s monitoring , i mean in which config file ?

How about doc? Please don't rely on me for every answer. Read all documents.
https://skywalking.apache.org/docs/main/latest/en/setup/backend/backend-infrastructure-monitoring/#k8s-monitoring

Hi Sir,
I was able to get K8S monitoring UI .
Thanks

Hi Sir,
Can you please help me know how can i see trace and log in my Skywalking UI ? It is coming blank. I tried installing guestbook application to see it's traces and log in UI but nothing is coming. Do i need to install any other application like some Java application ?

Thanks
Sajal

You need agents, don't have to be a Java agent. Read the projects page, we have many.


there is a error in this

systemctl daemon-reload
service uwsgi restart
we can't restart the  uwsgi .

You'll have to be more specific than that.

oh，it seems that command “systemctl daemon-reload” is not useful for ubuntu14.04，is there any command can replace the “systemctl” ？

This tutorial is for Debian, and has not been tested on Ubuntu. You might be able to do: service uwsgi reload.


IPv6 interface local scope multicast TX

Please ignore the "net: Rename net_is_xxx...() functions to net_xxx_is...()" commit, it is part of #11042

Codecov Report

Merging #11044 into master will increase coverage by <.01%.
The diff coverage is 57.37%.

@@            Coverage Diff             @@
##           master   #11044      +/-   ##
==========================================
+ Coverage   53.25%   53.26%   +<.01%
==========================================
  Files         217      217
  Lines       26698    26700       +2
  Branches     5925     5926       +1
==========================================
+ Hits        14219    14221       +2
  Misses      10060    10060
  Partials     2419     2419

Impacted Files
Coverage Δ

subsys/net/l2/ethernet/arp.c
58.9% <0%> (ø)
:arrow_up:

subsys/net/ip/icmpv6.c
31.66% <0%> (ø)
:arrow_up:

subsys/net/lib/dns/resolve.c
36.22% <0%> (ø)
:arrow_up:

subsys/net/lib/app/client.c
52.52% <0%> (ø)
:arrow_up:

subsys/net/ip/icmpv4.c
32.62% <0%> (ø)
:arrow_up:

subsys/net/ip/rpl.c
13.97% <0%> (ø)
:arrow_up:

subsys/net/lib/coap_sock/coap_sock.c
60.13% <100%> (ø)
:arrow_up:

subsys/net/ip/connection.c
78.13% <100%> (ø)
:arrow_up:

subsys/net/ip/route.c
50.56% <100%> (ø)
:arrow_up:

subsys/net/lib/coap/coap.c
60.66% <100%> (ø)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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


Issue after HA 2024.4.0 Update

HA core version
2024.4.0
Intergration version
0.3.22
Device type and model
Air Conditioner 00000Q13
Used App
NetHomePlus
The description of problem
This message was displayed after the HA 2024.4.0 update.
Message:
The custom integration midea_ac_lan
This no longer works in version 2024.10.0. Please fix this before upgrading.
The custom integration "midea_ac_lan" implements the property "is_aux_heat" or uses the additional heating methods in a subclass of ClimateEntity.
Please create a bug report at https://github.com/georgezhao2010/midea_ac_lan/issues.
Once an updated version of "midea_ac_lan" is available, install it and restart Home Assistant to fix this issue

The logs
No response

Dupe #483


View job results

From my point of view the section about view job results is not correct for SDK v2. The second option states: A Jupyter notebook using the JobDetails Jupyter widget
I don't think that this works (at least I didn't get it to work, but maybe the package has to be installed additionally), and if it should work the documentation is not sufficient. Could you please check this out?

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

ID: 52d75487-5a8d-f735-bc34-8a5d90c52d86
Version Independent ID: b651cdfb-6d3b-8ce9-f665-be07b1f11273
Content: Evaluate AutoML experiment results - Azure Machine Learning
Content Source: articles/machine-learning/how-to-understand-automated-ml.md
Service: machine-learning
Sub-service: automl
GitHub Login: @manashgoswami
Microsoft Alias: magoswam

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

Hi @MichaelaHeindl
Thank you for bringing this to my attention. I understand that you are having trouble with the second option mentioned in the "View job results" section of the document. You are correct that the JobDetails Jupyter widget is not available in the Azure Machine Learning SDK v2.
Instead, you can use the RunDetails widget to view the job results in a Jupyter notebook. Here is an example code snippet that shows how to use the RunDetails widget:
from azureml.widgets import RunDetails

# get the latest run from the experiment
run = experiment.get_runs()[0]

# show the run details using the RunDetails widget
RunDetails(run).show()

This will display the run details, including the run status, logs, and output files, in a Jupyter notebook. You can also use the RunDetails widget to view the metrics and charts for the run.
I hope this helps! If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@MichaelaHeindl
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close


[Feature Request][Dropdowns] PreserveDefaultKeypressEvent flag

Is your feature request related to a problem? Please describe.
Hello.
By default, pressing the ENTER key and/or the SPACE key should dispatch a click event for some tagnames (especially on <button> and <a>).
For example, the expected default behaviour on a <button> should be :

SPACE: click
ENTER: click

And on a <a>:

SPACE: nothing
ENTER: click

When we're working on vanilla code, this is the behaviour of web browsers. It is provided for-free, and no more custom logic/magic is required.
Unfortunately, this behaviour is not preserved on Dropdown items. I could do some "magic" by implementing custom functions, as suggested in the documentation. But I think that's a pain we should be able to avoid.
At the moment, if I press the SPACE key or the ENTER key when the focus is on an <a> or a <button> within a dropdown menu, it just closes, and I don't think it is the desirable behaviour in this scenario.
For example, if I go on this demo: https://next-ui-toggler-bug-demo.vercel.app/
And then click on the "Patch Notes" dropdown menu trigger button,
And then press the "Tab" (or Up/Down) key to move the focus on "Dashboard" | "Discord Bot",
And then press Enter,
The dropdown just closes.
I would have liked this to reproduce the expected default behaviour: a click, and therefore a redirect to the corresponding page.
Describe the solution you'd like
I would like to have a PreserveDefaultKeypressEvent flag for the dropdowns items.
I think this bug is because of some preventDefault calls, which should be optional.
Describe alternatives you've considered
Implementing by hand the dispatch event... Really annoying.
Screenshots or Videos
No response
Code
:arrow_right: https://github.com/gustaveWPM/next-ui-toggler-bug-demo/blob/main/src/components/layouts/navbar/NavbarDropdown.tsx
Note: a more recent implementation of this (respecting some ARIA concerns) could be find there:
:arrow_right: https://github.com/gustaveWPM/dashboard_rtm/blob/main/src/components/layouts/navbar/NavbarDropdown.tsx

Hey @gustaveWPM could you share the dropdown implementation code?

Hello @jrgarciadev, thank you for your fast answer!
Sorry I forgot this important detail.
The code that contains my implementation of this dropdown can be found here
:arrow_right: https://github.com/gustaveWPM/next-ui-toggler-bug-demo/blob/main/src/components/layouts/navbar/NavbarDropdown.tsx

Ok I see, did you try using the useRouter hook + the Dropdown Menu onAction callback?
The support for Client Side Router will be available in v2.2.0 which means that only passing the href prop to the Dropdown items will automatically dispatch the navigation.
https://react-spectrum.adobe.com/react-aria/routing.html

Ok I see, did you try using the useRouter hook + the Dropdown Menu onAction callback?
The support for Client Side Router will be available in v2.2.0 which means that only passing the href prop to the Dropdown items will automatically dispatch the navigation.
https://react-spectrum.adobe.com/react-aria/routing.html

Thank you for the workaround!
Since this should be fixed in v2.2.0, I'd prefer to just wait. :)
However, I will give it a try if I have some time to experiment this.


ERROR: Timeout while waiting for panel to start.

Tested on Ubuntu 20/22 and Debian 12. The result is the same. I tried it on two different servers.
 ** Waiting for services to start...
    ✅ libertea-provider-trojan-ws started
    ✅ libertea-camouflage-nginx-fallback started
    ✅ libertea-provider-vmess-grpc started
    ✅ libertea-provider-vless-ws started
    ✅ libertea-provider-shadowsocks-v2ray started
    ✅ libertea-provider-trojan-grpc started
    ✅ libertea-log-parser started
    ✅ libertea-provider-vless-grpc started
    ✅ libertea-mongodb started
    ✅ libertea-rsyslog started
    ✅ libertea-haproxy started
    ❌ libertea-panel failed to start. Retrying...
*******************************************************
ERROR: Timeout while waiting for panel to start.
       Please open an issue on https://github.com/VZiChoushaDui/Libertea/issues/new
       and include the following information:

       - component name: libertea-panel
       - OS: "Debian GNU/Linux"
       - OS version: "12"
       - Docker version: Docker version 27.2.1, build 9e34c9b
       - Panel listening: False
       - Panel root status code: 000
       - Panel admin status code: 000
       Also include the output of the following command:
           tail -n 100 /tmp/libertea-panel.log

root@debian:~# tail -n 100 /tmp/libertea-panel.log
tail: cannot open '/tmp/libertea-panel.log' for reading: No such file or directory

Hmm. No panel log file? The service hasn't been started at all then.
Is systemd working? Run systemctl status libertea-panel and post the output.
We can't reproduce this, perhaps some weird configuration of os on your server provider's side is causing that. Would you mind sharing where you're getting your servers from?

Hmm. No panel log file? The service hasn't been started at all then.
Is systemd working? Run sudo systemctl status libertea-panel and post the output.
We can't reproduce this atm, perhaps some weird configuration of os on your server provider's side is causing that. Would you mind sharing where you're getting your servers from?

Ubuntu 22 this time:
ERROR: Timeout while waiting for panel to start.
       Please open an issue on https://github.com/VZiChoushaDui/Libertea/issues/new
       and include the following information:

       - component name: libertea-panel
       - OS: "Ubuntu"
       - OS version: "22.04"
       - Docker version: Docker version 27.2.1, build 9e34c9b
       - Panel listening: False
       - Panel root status code: 000
       - Panel admin status code: 000
       Also include the output of the following command:
           tail -n 100 /tmp/libertea-panel.log

root@ubuntu:~# sudo systemctl status libertea-panel
● libertea-panel.service - LiberteaPanel
     Loaded: loaded (/etc/systemd/system/libertea-panel.service; enabled; vendor preset: enabled)
     Active: activating (auto-restart) (Result: exit-code) since Mon 2024-09-16 00:40:52 UTC; 7s ago
    Process: 19345 ExecStart=/bin/bash /root/libertea/panel/serve.sh (code=exited, status=208/STDIN)
   Main PID: 19345 (code=exited, status=208/STDIN)
        CPU: 1ms

VPS at hostup.se

Please also post the output of this command:
tail -n 100 /tmp/libertea-panel.init.log

Please also post the output of this command:
tail -n 100 /tmp/libertea-panel.init.log

root@ubuntu:~# tail -n 100 /tmp/libertea-panel.init.log
tail: cannot open '/tmp/libertea-panel.init.log' for reading: No such file or directory


allow newer autocfg

went ahead and updated the documentation link too

See #9. You probably want to use https://github.com/mattwilkinsonn/rust-claims


dont set value in input

hi
thanks for your component
i test it i think have bug
https://stackblitz.com/edit/angular-ivy-aepwjb
please see this and help me

Hi, thanks for the report :)
Please checkout new version: 2.3.2

thanks
i check in stackblitz
worked


Remove yarn.lock from .gitignore

Lock files are designed to be submitted to source control for consistent builds. Ignoring yarn.lock conflicts with its intended purpose: https://classic.yarnpkg.com/en/docs/yarn-lock/#toc-check-into-source-control.

I'm not sure if yarn.lock was added to the gitignore for a reason I'm missing, but it seemed contradictory to its intended purpose as I mentioned above. I created this change since it was a quick fix. Feel free to merge or close as is appropriate.

@mezuzza I believe it was just to avoid confusion on the CI (for build and for dependent bot). So only single lock file is used (for NPM), I'd leave this as is, probably as the best option.

That makes some sense - though that somewhat implies both npm and yarn are being used to manage the project which sounds strange given that the project specifically uses npm.
Still, if you think it really doesn't make sense, would you be opposed to me modifying the comment to provide that context?

absolutely not, go for it, thx!


19.1.5: Cannot save aliases any more

Describe the bug
I just upgraded from 19.1.4 to 19.1.5 and noticed that you cannot click "Save" in the alias interface any more.
To Reproduce
Steps to reproduce the behavior:

Go to 'Firewall -> Aliases'
Click on 'Add new Alias'
Click "Save"
Nothing happens

Expected behavior
Alias should be saved.
Environment
Software version used and hardware type if relevant.
e.g.:
OPNsense 19.1.5 (amd64, OpenSSL).
APU2C4

Update to 19.1.5_1, went out half an hour ago.


Fix typo

There is a typo in variable name. I've also removed few confusing imports (not completely sure if they are not used from somewhere, but it should be safe to drop them).

ping @breillyr

LGTM


python highlight

Added highlight grammar in Commit 1bb89f5 for Python code block:

control keyword
constant
operator
data type
quotes
comment


URLs are always sent in absolute form (http://example.com/a/b/c), but normally only the path is required (/a/b/c)

It seems absolute URLs in request lines should only be sent to proxy servers [0], but idris-http always sends them regardless.
This is a problem for me because the webserver I am talking to (https://github.com/mozilla/geckodriver) does not accept them.
Can URLs always be sent in relative form instead? I can send a pull-request if you agree.
[0] https://www.w3.org/Protocols/rfc2616/rfc2616-sec5.html (section 5.1.2)

Thanks :+1:


'localhost' no longer valid as a server name

I have several scripts that route my RDP connections through SSH tunnels.  They connect to servers with with command-lines like:
xfreerdp /v:localhost:6901 /u: /d: /p: /cert:ignore
These scripts stopped working around release 6.3.2.
By setting the log-level to DEBUG I see these (somewhat misleading) errors:
[11:01:06:420] [3116933:002f8f89] [DEBUG][com.freerdp.core.nego] - [nego_send_negotiation_request]: RequestedProtocols: [SSL|HYBRID][0x00000003]
[11:01:06:420] [3116933:002f8f89] [ERROR][com.freerdp.core.transport] - [transport_default_write]: BIO_should_retry returned a system error 32: Broken pipe
[11:01:06:420] [3116933:002f8f89] [ERROR][com.freerdp.core] - [transport_default_write]: ERRCONNECT_CONNECT_TRANSPORT_FAILED [0x0002000D]
I have found a work-around.  If I replace 'localhost' with '<IP_ADDRESS>' my scripts continue to work.
I have also compiled a few different version  of xfreerdp and it looks like the failure is caused by this change to libfreerdp/core/tcp.c:
https://github.com/FreeRDP/FreeRDP/commit/562fa8a03d20957bff73fa3e1c5429aa3672dfd0#diff-c3b201d4c2b1152732b0975aa78f99c369ec784d5fe9385b902cbcfe21ff4397
If I back out that change to tcp.c, 'localhost' continues to work as a server name.
I tested this with several builds on Fedora Linux 40 and Raspbian Linux 64-bit.

@daj92109 then you should check your local setup.
localhost works quite well if it can be properly resolved. (we do not do that, we ask the system to resolve addresses for this project)
also, we have no version 6.3.2 ;)
most recent is 3.8.0
fyi: tested current freerdp on debian 12 and a self compiled version from master, both resolve localhost just fine on my systems.


Update execution engine to use Wasmtime 0.26 (duplicate)

Duplicate of #167 for the purpose of restarting the CI

It seems to make the crypto program I'm currently experimenting with run about 15% faster. Is that what you'd expect?

Great! I measured a 10% performance boost for object detection and video decoding


How to Make a Discord Bot using Python

We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
How to Make a Discord Bot using Python

Introduction
Discord is most used apps nowadays by companies, students and work professional and bots make it easy to protect our server from spams, greeting new people, verification , enjoyment and many more... .So I will give a full tutorial so that everyone can make their own bots using python for their servers,
Key takeaways:
What are the 3-5 most important things the reader should understand or be able to do after reading this article? Use this area to get your ideas down on the bulk of your article or tutorial.

Will learn how to make a bot using python
Add to the reader's skills to their most-using app and learn a bit about coding in python if not familiar yet.
How to use bot commands, checks, and converters.
How to create a Discord connection in Python.

References:
N/A
There is an article in EngEd about making a bot using node.js so I thought it would be helpful if one can also learn how to make it using python

seems like a helpful topic - lets please be sure it add value beyond what is in the official docs and that it does not overlap with any existing EngEd articles or incoming topic suggestions (if you haven't already) to avoid any potential article closure. - approved
@harshitab13
Do not forget to add soruces for sentences like : "Discord is most used apps nowadays by companies, students and work professional and bots make it easy to protect our server" - this is not an opinion statement therefore needs sources. 👍


how to disable past days

i want to disable past days in calendar
here is my current approach
      mapDays={({ date, isSameDate, today }) => {
              let isBooked = bookedDays.find((x) => isSameDate(date, x));
              if (
                date.year <= today.year &&
                date.month.index <= today.month.index &&
                date.day < today.day
              ) {
                return {
                  disabled: true,
                  style: { color: "#ccc" },
                };
              }
            }}

is there a more optimized way?

Maybe minDate or maxDate can help.


Hub Module Improvements

Summary

[x] Add additional options for peering to spoke vNets
[x] Add options for naming the default route

Specifications

Module/Configuration: Hub
Platform: Azure

Changed the way the hub manages peering to pass the variable directly to the network module


Update tryitnow URL

Will create anther one with the name change to "Free Account". So we can deploy this while waiting on the loc changes

looks good :shipit:


[soundtouch] enable static build

Describe the pull request

What does your PR fix?

Static build of soundtouch

Which triplets are supported/not supported? Have you updated the CI baseline?
all,!uwp, Yes

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Done

/azp run

Done

Needs to solve the conflicts here. @daschuer

This PR is obsolete. #20017 implemented support for static builds.

The only missing bit was the ci.baseline.txt  change. I have rebased this PR accordingly.

This PR is also (accidentally) included in https://github.com/microsoft/vcpkg/pull/20435

Thanks for your contribution! Unfortunately it looks like all the edits here are already applied in master :)


Code Climate

@stympy I was wondering if you could integrate Code Climate in this project with Travis. Code Climate is pretty powerful. We should definitely take advantage on this product to analyze the code and see what parts future PRs could improve.
I tried to create an account, but I couldn't link the Faker repository because I'm not the owner.
We should integrate Code Climate and add the maintainability and test coverage badges in the README.md.

Fixed in 1dcd059b

Awesome. Should we remove Coveralls or you believe we should work with both?

I don’t think we need both.

No problem. I just pushed a commit that removes the gem and its configurations 👍

Thanks!


Typescript type definition

Please add typescript type definition current version is useless.

Please see https://github.com/sendgrid/sendgrid-nodejs/pull/251 for progress, thanks!


Delete .github/ISSUE_TEMPLATE directory

:x: Build pagarme-sdk-dotnet 1.0.306 failed (commit https://github.com/guibranco/pagarme-sdk-dotnet/commit/5ea67dcde1 by @guibranco)

:white_check_mark: Build pagarme-sdk-dotnet 1.0.311 completed (commit https://github.com/guibranco/pagarme-sdk-dotnet/commit/27922b12fb by @guibranco)


Add MethodOverridingInitializer extension

Describe the change
This PR adds a new extension - MethodOverridingInitializer. This extension allows existing hash methods to be overridden if a hash passed in to the constructor contains such keys. For example:
class OverridingHash < Hash
  include Hashie::Extensions::MethodOverridingInitializer
end

hash = OverridingHash.new(zip: 'a-dee-doo-dah')
hash.zip #=> 'a-dee-doo-dah'
hash.__zip #=> [[[zip a-dee-doo-dah]]]

Issue
References #338
Design Decisions
I made this its own extension in case people want to restrict which methods can be overwritten. Using this extension only, they can pass in a hash of only the methods they want overwritten and prevent overwriting other methods.
Possible Alternatives
This could be included in the extension MethodAccessWithOverride instead of being it's own extension.

Great idea! A couple things I would like to see:

A reduction of the duplicated code. Since the redefine_method is the same as in the MethodOverridingWriter, it should be extracted into a helper module so we don't have it defined in multiple places.
We could ship this in a minor revision (3.6) as a new extension, but we shouldn't change the way that MethodAccessWithOverride works without a bump to 4.0. I think we could add a log message to the included hook for MethodAccessWithOverride if we want to make it a default in 4.0.

One additional thing that I'm thinking about here is that this somewhat replicates a subset of the behavior of MergeInitializer. I wonder if there is a clean way we can make it as an extension to MergeInitializer. I think we would have to add some sort of hook method that gets called for each key in MergeInitializer.
What are your thoughts about an approach like that?

I like that idea. We could maybe make it an optional named argument. So, for a normal hash, we would do something like: MyHash.new(initial_hash, override: true), and for a clash you could do MyClash.new(initial_hash, parent, override: true). Then if you don't want to override, we keep the original way the method was called with MyHash.new(initial_hash).

Neat

After implementing it both ways, I think it makes more sense to keep this under a MethodOverridingInitializer module. I don't really see this being used without MethodReader, but I can see people wanting to use MergeInitializer without overriding the methods.

Closed via cabb3e7


A problem occurred evaluating root project 'flutter_image_compress'.

FAILURE: Build failed with an exception.

Where:
Build file '/Users/apple/Developer/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_image_compress-1.0.0-nullsafety/android/build.gradle' line: 23

What went wrong:
A problem occurred evaluating root project 'flutter_image_compress'.

Plugin with id 'com.android.library' not found.

Exception: The plugin flutter_image_compress could not be built due to the issue above.

can anyone help me with this issue? Thanks

same here, have you found the solution?

Please submit new issues if it persists.


Use best practices for service registration

Following https://symfony.com/doc/current/bundles/best_practices.html#services:

If the bundle defines services, they must be prefixed with the bundle alias instead of using fully qualified class names like you do in your project services. For example, AcmeBlogBundle services must be prefixed with acme_blog. The reason is that bundles shouldn’t rely on features such as service autowiring or autoconfiguration to not impose an overhead when compiling application services.

In addition, services not meant to be used by the application directly, should be defined as private. For public services, aliases should be created from the interface/class to the service id. For example, in MonologBundle, an alias is created from Psr\Log\LoggerInterface to logger so that the LoggerInterface type-hint can be used for autowiring.
Services should not use autowiring or autoconfiguration. Instead, all services should be defined explicitly.

It would be nice to follow these practices in:
https://github.com/KnpLabs/DoctrineBehaviors/blob/16cf5617882d4e5828215f2018cb58b3d24c0ee7/config/services.php#L17-L20
I could give it a try if this is fine, the only problem I see is that UserProvider is using:
https://github.com/KnpLabs/DoctrineBehaviors/blob/16cf5617882d4e5828215f2018cb58b3d24c0ee7/src/Provider/UserProvider.php#L12-L16
~Security which is registered by symfony/security-bundle which is not a dependency (BTW looks like security.token_storage could be used instead).~ I would leave the Security service as it is for now.

Closing here, see https://github.com/KnpLabs/DoctrineBehaviors/pull/660#issuecomment-917385552


Use self when referencing cluster object

@mshuler can you review please?

+1 - feel free to merge to master, or let me know if you'd like me to merge.


[Feature]: SurrealDB support

What happened?
SurrealDB is one of the fastest growing Databases on GitHub! They have support for cloud and wasm databases.
I think it would be a great addition to the shuttle ecosystem.
They are still in Beta but they have support for all the major frameworks e.g Axum, Actix
I don't think it would be hard to implement, I could write a blog post if it is particularly easy.
Version
v.0.11.0
Which operating systems are you seeing the problem on?
Linux
Which CPU architectures are you seeing the problem on?
x86_64
Relevant log output
-

Duplicate declaration

[X] I have searched the issues and there are none like this.

Thanks for opening this Matthew! :heart: We're definitely interested in exploring this and we'd welcome a PR for it. Personally I don't know much about surrealdb yet, so if you have some ideas about how we might go about this that would be a much appreciated first step here.

/bounty

This issue is stalled due to uncertainties about how SurrealDB's license affects hosting on Shuttle. Once Surreal has their own cloud, there might be more activity on this front.

Surreal Cloud is now in beta, so we may be able to support some integration with Shuttle. For anyone who wants to use SurrealDb in the meantime, you can provision a database yourself in their cloud, then connect to it from your Shuttle application.


chore: add yarn scripts to simplify docs contribution

Motivation
Simplify docs contribution
Description
Added yarn scripts to simplify commands to run in order to contribute to docs. This would simplify instructions introduced in https://github.com/ChainSafe/lodestar/pull/6294 or better yet, you don't even need to read how to contribute because yarn scripts make it obvious.
Having all docs commands prefixed with docs: makes it also easier to figure out what commands exist as right now they are a bit scattered around.

Retroactive LGTM!   Good idea to include this 🚀


fix: add .js extension for relative import/exports in declarations

Context: https://github.com/nuxt/nuxt/issues/18426
Currently with Node16-style module resolution, the following will not be recognised by TS within a declaration file (imagine we have index.d.ts and other.d.ts):
export * from './other'

In order for this to be understood, it needs to have the JS file extension equivalent of the other dts file, which would be .js. (.mjs would be the equivalent for a .d.mts file).
Despite the fact that no .js file exists, TS understands the following to mean what we want:
export * from './other.js'

This PR adds support for a new flag (addRelativeDeclarationExtensions) which enables the addition of this extension within the outputted files. I've tested this within the Nuxt codebase both with and without Node16 style module resolution but it would benefit from broader testing as well, which is why I'm opening the PR for feedback at this point.
Remaining upstream todo would be to detect type import/exports within https://github.com/unjs/mlly.

@pi0 Might be worth merging https://github.com/unjs/mkdist/pull/110 first; I'm happy to update this PR subsequently as it's not that complex.

Let's quickly move forward for this. Looking for mlly PR for type support. Also 110 probably needs extra care for possible breaking changes.


[WFLY-16201] Upgrade Yasson to the final 3.0.0 version.

https://issues.redhat.com/browse/WFLY-16201
https://issues.redhat.com/browse/WFLY-15690

Thanks @jamezp


IN-675 Compare changed config to prod config

Separates workflow into two separate jobs

The first job detects if any config files are changed in the PR
The second runs all downstream steps only if first job is true

New steps in second workflow

One step to create and write out dictionary to JSON file containing info on config updated in PR and prod config in 001_Reference
Another step retrieves config info with jq and uses this to produce diff report comparing newly updated with prod config in 001_Reference, uploads this as artifact of workflow

Hello @rklocke! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file get_production_config.py:

Line 237:80: E501 line too long (81 > 79 characters)
Line 242:13: E128 continuation line under-indented for visual indent
Line 308:80: E501 line too long (81 > 79 characters)
Line 312:80: E501 line too long (97 > 79 characters)


Realm Browser navigation doesn't always go "back" to the proper place

Here is a video that shows the unexpected behavior:
https://www.dropbox.com/s/6dtitwguo0pgr7e/realm navigation.mov?dl=0

This issue was moved to realm/realm-browser#8


Seed from tables and refactor old code

Changelog:

Add capability on generate seeder file from tables
Add option table modes and model modes
Add capability generate from multiple models
Add capability generate from multiple tables
Add capability generate from all tables
Refactor code for better readability

Wrong branch, revert


The find dialog can be too wide

This is due to having a large string in your history

Should be fixed in builds >= 4.2.6105-dev.
There is now a size limit on the initial dialog size. You should still be able to resize the dialog to be wider if you need to, though.


Question: would caddy/letsencrypt fail to work with certain domain name extensions?

I'm using the latest version of Caddy and its working well for several domain names.
I'm unable to get a cert for a .studio domain name. Would the domain name be an issue?
Thank you.

What's the error message? :confused:

Closing unless there's any more information we can go off of and identify an issue. Thanks!

Hi @mholt My question was mainly to ask if a non-typical domain extension such as a *.studio would have a problem in getting a cert.
In the mean time, I've been able to get it working, so it doesn't seem to be an issue.

Should be fine as long as your ACME CA can verify it. (Let's Encrypt doesn't forbid .studio domains AFAIK.)


Add mechanism for token renewal

Uber object should store the token expiration date and renew the OAuth 2.0 access_token if date is reached

Hi, I would like to work on this. Kindly give me some suggestions to proceed.

@kp96 Love it! I would recommend the following:

check out branch v1.0.0. That way, we don't need to change your submission for the v1. There's quite some difference in terms of source code. You should get familiar with it
try to understand what Uber.js does. Most important, it uses the OAuth library to get the token
for the token renewal, you need to check for each HTTP call, if the existing token is still valid. You could integrate that check within the modifierMethodHelper method. The property tokenExpiration should be used for that
if renewal is required, use the method authorization with the property refresh_token. After that, execute the HTTP call as usual

As soon as you have the feature implemented, I'd ask you to add some tests. One test for the old-fashioned (node-back style) calls in this file: oauth.js and one test for the new async (promise-based) calls in this file: oauthAsync.js.
Let me know if you have any further questions. Super excited to get on board as a contributor :)

Thanks, That is all I needed as of now. Will start working on it asap.

👍

Hi,
As per your suggestions. I made the following changes.

Added a method isTokenStale() that checks if token is expired. Followed the 500 sec threshold as in the uber python sdk
Inside the modifierMethodHelper, I checked if the token is stale and if it is, I called the authorization method with refresh_token property.
Due to rest of the code being synchronous and only the refresh part is asynchronous I have moved the code into a function refreshCallback and called it after execution completes. I took this idea from here

However, I faced some issues regarding the tests part. The problem is I am very new to tdd and writing tests. I tried to manually check if the token refresh is happening by explicitly returning true from isTokenStale() method. I got this error from nock.
AssertionError: expected [Error: Nock: No match for request for every request method like PUT, POST and so on.. I tried to understand Nock library but not able to figure out the exact error. I have pushed the commit here. Kindly help me out with it and pardon me  if it sounds too silly.

@kp96 sounds great. Let's push through and get the testing issue out of the way! I quickly checked out your commit and ran the tests with this result:
  264 passing (1s)

----------------|----------|----------|----------|----------|----------------|
File            |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
----------------|----------|----------|----------|----------|----------------|
 lib/           |    97.83 |    92.31 |    96.88 |    97.83 |                |
  Uber.js       |    97.83 |    92.31 |    96.88 |    97.83 | 199,200,203,204 |
 lib/resources/ |      100 |      100 |      100 |      100 |                |
  Estimates.js  |      100 |      100 |      100 |      100 |                |
  Payment.js    |      100 |      100 |      100 |      100 |                |
  Places.js     |      100 |      100 |      100 |      100 |                |
  Products.js   |      100 |      100 |      100 |      100 |                |
  Reminders.js  |      100 |      100 |      100 |      100 |                |
  Requests.js   |      100 |      100 |      100 |      100 |                |
  User.js       |      100 |      100 |      100 |      100 |                |
----------------|----------|----------|----------|----------|----------------|
All files       |    99.01 |    96.61 |    98.84 |    99.01 |                |
----------------|----------|----------|----------|----------|----------------|

=============================== Coverage summary ===============================
Statements   : 99.01% ( 400/404 )
Branches     : 96.61% ( 228/236 )
Functions    : 98.84% ( 85/86 )
Lines        : 99.01% ( 400/404 )
================================================================================

As you can see, your additions show up as untested code lines (199,200,203,204). We'd need to test these to maintain a great coverage.
I didn't come across any errors with running the tests. However, this error you describe usually indicates that the mock server ("nock") we use for testing can't find the route you try to address.

did you run npm install?
did you update to the latest revision of the code?
do you run test with npm test?

I see that your test is quite complicated. In general, in order to test your more or less edge case, you'd need to:

add a new sample response file containing a token that has an expires_in value in the past. The file should be very similar to test/replies/token.json.

update the test/allTests.js and add a new nock, for instance:

.post('/oauth/token', {
            code: <your_new_code>
        })
        .replyWithFile(200, jp('<your_new_token_filename_without_extension>'))

It's important to understand that we use nock to check for the route ('/oauth/token') and for the submitted headers ('code') in order to match a route.

You have to replace <your_new_code> with some code that you'd use to target nock to a new route with the expired token response file you saved as <your_new_token_filename_without_extension>. We store all tokens in the test/common.js file. Add your new token and update the top require command in test/allTests.js to import your token. With that, you added a new route for a specific oauth token. Nock will respond with the expired token file. Great! Now, you should strat writing the unit tests.

update the file test/oauth.js and add a new describe part containing at leats one it unit test. Essentially, you'd describe a new authorization process. Within this test, you'd need to use your new code you added to test/common.js. Therefore, update the require command on top to add your new code. Then, in the it test, you essentially run an authorization command with that new token:

uber.authorization({
                authorization_code: <your_new_code>
            },
            function(err, res) {

In the callback, you'd have received the expired token. Now comes the interesting part. You have to make some call (any, let's call that call2). Your new feature should realize that the token is outdated and make a new call with a refresh_token to the login page of Uber. You might need to add another route to nock with your new refresh_token. As soon as your request is finished, in the callback of call2 you should check if the Uber object has a new, valid acess_token and an expiration date in the future.

similarly to what you did in the test/oauth.js file, you'd need to update the test/oauthAsync.js file. This file tests the async variants of the methods you updated. There should be no surprises here. The methods just don't have callbacks but promises you need to use. Keep in mind, every async method name appends with "Async".

That's quite an advanced test case you chose there buddy :) Let me know if you can get it done. Otherwise, I'll PR the tests to your fork and you can take these for your final PR :)

@agraebe Thanks for taking your time in giving me precise instructions. I will try to write tests on my own and if I face any issue, I will let you know. :+1:

@agraebe I have implemented all the tests as you described and completed the code coverage. Below is a gist of what I did.

Added required files in the test/replies folder.
Added required codes for various nocks in the common.js file and used them wherever required.
After getting an expired_token response from nock,  I called the uber.request method that  requires the access_token, and after the callback is called I have checked whether token_refresh happened or not.
Added an additional case where uber server may not be available during token_refresh, so that complete code is covered.
Added the above tests in both test/oauth.js and test/oauthAsync.js with their respective variants.

The latest updated commit in my fork is here. Let me know if anything else needs to be done. If that is all is needed, I will raise a PR. Thanks for your suggestions.

Should I raise a Pull Request?  A gentle reminder. Thanks

@kp96 Sorry for the late response! Your contribution looks great! Just create a PR and we can merge it after passing the tests... Are you interested in further tasks? We have some more in the backlog and are looking for passionate devs like you :)

Thanks! Yes, I am excited to take more tasks. :)


error when running the setup code

Hello, I am new to gallium os and Have been trying to run the setup code but I seem to run into an error for the switch statement. I assume this is because i am using the wrong version of python so I tried using a different version
output of pythong -V "python 2.7.17"
ouptut of python3 -V " python 3.6.9
here is the error error for "./setup-audio"
File "./setup-audio", line 156
match platform:
^
SyntaxError: invalid syntax
I get the same error for "python3 setup-audio"

GalliumOS is not supported


Dependency problems around jakarta.mail

If I only declare dependency on simple-java-mail, the program does not run as expect.
<dependency>
    <groupId>org.simplejavamail</groupId>
    <artifactId>simple-java-mail</artifactId>
    <version>8.1.3</version>
</dependency>

It will throw ClassNotFoundExceptions like:
java.lang.ClassNotFoundException: jakarta.mail.MessagingException

java.lang.ClassNotFoundException: jakarta.activation.FileTypeMap

It can be resolved by adding additional dependencies but it should not be the right behaviour.
Suggestions for the [simple-java-mail] staff:

Add those dependencies on simple-java-mail-8.1.3.pom
Use flatten plugin to simplify the pom before uploading artifacts

This can't be correct. I'll have a look later, but I'm highly skeptical. Are you sure your dependencies are not server-managed? Or that the transitive dependencies are not excluded somewhere?

I saw a really simlar issue here 432.
Additionally, I did not define any exclusion around jakata email.
However, it is possible that this issue is related to JDK versions.
I'm using JDK 11.

I just started an new empty Maven project, JDK 11, with just the simple java mail dependency. Here's the result of the dependency tree:

[INFO] Scanning for projects...
[INFO]
[INFO] -----------------< org.example:simple-java-mail-test >------------------
[INFO] Building simple-java-mail-test 1.0-SNAPSHOT
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-dependency-plugin:2.8:tree (default-cli) @ simple-java-mail-test ---
[INFO] org.example:simple-java-mail-test:jar:1.0-SNAPSHOT
[INFO] - org.simplejavamail:simple-java-mail:jar:8.1.3:compile
[INFO]    +- org.simplejavamail:core-module:jar:8.1.3:compile
[INFO]    |  +- com.sun.mail:jakarta.mail:jar:2.0.1:compile
[INFO]    |  +- com.sun.activation:jakarta.activation:jar:2.0.1:compile
[INFO]    |  - com.sanctionco.jmail:jmail:jar:1.4.1:compile
[INFO]    +- com.github.bbottema:jetbrains-runtime-annotations:jar:1.0.1:compile
[INFO]    +- com.pivovarit:throwing-function:jar:1.5.1:compile
[INFO]    - org.slf4j:slf4j-api:jar:1.7.32:compile
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  0.543 s
[INFO] Finished at: 2023-07-25T12:34:12+02:00
[INFO] ------------------------------------------------------------------------

Looks fine to me. This is so basic, I cannot conclude anything other than a user error, somehow. Your dependencies are not loaded properly in your project for some reason.

I have the same problem, using it in a Spring Boot Vaadin application. Needed to add
        <dependency>
            <groupId>jakarta.mail</groupId>
            <artifactId>jakarta.mail-api</artifactId>
        </dependency>

to my dependencies.
This should be already included in simple-java-mail?

Simple Java Mail works without the -api package, as it uses an implementation package directly (which includes all the same types).
<dependency>
    <groupId>com.sun.mail</groupId>
    <artifactId>jakarta.mail</artifactId>
    <version>2.0.1</version>
</dependency>

This library is used like this by thousands of people and I've recently tested this to double check it in a new clean project. If the -api dependency is needed, I can only guess it's because of Spring somehow.


Echarts 4.2.0 does not work in Chrome 73  (dev and Canary)  or higher.

General Questions

Issue Type

[ ] I have a question to ask about how to use ECharts to ...；我想提问如何使用 ECharts 实现某功能
[x] I have a bug to report；我想要报 bug
[ ] I have a feature to request, e.g.: I'd like a new feature that ...；我需要一个新功能
[ ] I have a feature to enhance, e.g.: The current feature should be improved in the way that ...；我希望改进某个功能
[ ] There's something wrong with the documents；文档有些问题
[ ] Others, or I'm not sure which issue types to choose here；其他，或我不知道应该选什么类型

Issue Details

Echarts 4.2.0 does not work in Chrome 73  (dev and Canary)  or higher.
Expected Behavior

Just rendering charts normally.
Current Behavior

If you are using Chrome 73(or 74,both latest dev and Canary have reached this version),Echarts does not render anything,even in offical demos,all you can see is a blank page.
Online Example

https://www.echartsjs.com/examples/editor.html?c=area-basic

Topics
Check those topics that are related. 选择相关主题。

[ ] Legend
[ ] Tooltip
[ ] Event
[ ] Performance
[ ] SVG
[ ] Map
[x] ECharts GL
[ ] Third-party libraries, e.g.: Vue.js, React

Anything Else We Need to Know

Environment

ECharts version；ECharts 版本: {4.2.0}

[x] It happens only on certain browsers or operating systems. 对于特定浏览器或操作系统才会出现的问题，请提供相应环境信息：{Chrome 74.0.3702.0 (dev or Canary)}

I have tried on Chrome 74.0.3712.0 (Official Build) canary (64-bit) (macOS 10.13.5)
using the link https://www.echartsjs.com/examples/editor.html?c=area-basic.
It seems to work normally.
Could I have your OS info?

Could I have your OS info?

Sorry to reply you so late.
My OS info: Windows 7(Ultimate).
The issue still happens in Chrome(dev) 74.0.3702.0,but now, on Chrome 74.0.3712.0 (Official Build),it works normally.
So...I think there's no need for us to care about this issue any more.


Link workspace locally

Added a script to link Workspace package locally. This has been done manually for some time now.
Not sure how much effort we should put into this convenience. Like e.x should it tell you if workspace is behind main by x commits?
Note: Encountered a weird bug where if I build workspace from root it behaves differently from when building a specific package. Unrelated to this pull request there might be some sort of bug with turbo caching or possibly tsup

@Gustav-Eikaas This should be mentioned in the README somewhere.

Yes you are right! Added it now


Log out of admin?

This seems ridiculous but I can't actually find the button to log out of the Admin account. We'll want to have this in case we're editing on a public computer.

add Logout button in admin w link to https://deep.pennds.org/admin/logout/


Tokens to AST

I utilized libcst to modify the approach for parsing the source code of the endpoints. With the generated tree, I designed a DFS like algorithm to retrieve all HTTPExceptions and functions.
I also made some changes on the dependencies to run on my environment.

We want to create a visitor instead of manually visiting the nodes.

Ok, I'll fix that

I think that the next step is to use libCST matchers to simplify the visitor

I've enabled the pipeline.
We need to fix the tests (or add them...) 👀

Feel free to remove support for python 3.6 and 3.7.

I do not know how to solve that conflict. Could you please suggest something to me, @Kludex?


x/build: may-start-trybots only applies to one's own CLs

In LUCI, may-start-trybots only grants permission to trigger a presubmit run on a CL you authored yourself. (Approvers can still trigger presubmits on any CL.)
This makes community members less able to shepherd CLs and therefore slows down the Go project as a whole. We want to fix this in LUCI (see https://crbug.com/1443582), but if it's causing enough trouble in the meantime we can look into workarounds. If that's important to you, please comment on or react to this issue.
cc @golang/release

This should be fixed now. Felix graciously tested triggering tests on https://go.dev/cl/554615.


iOS visibility hidden bug

iOS visibility hidden by default for a bar button is ignored when pushing with a bar button visible=false

Unable to reproduce on Android.
Reproduced on iOS only if setting false as a string instead of a boolean, a check is done native side to avoid crashes.


scim: Use setattr to set request.user as scim_client

This is a type-unsafe workaround before we can fix the problem that django_scim2 relies on request.user being present to authenticate requests.
Extracted from #18777 (cc @PIG208).

Merged, thanks @andersk and @PIG208!


no validation when training

is there validation when training waymo open dataset?

I want to print val loss after every epoch to monitor the training process

This is not supported now. Basically, Waymo validation will take ~1hr to run + ~0.5hr (or more than that) to evaluate on the validation set, which I feel is too long as a middle step between each training epoch. If you want to monitor the process, you can probably start a new single gpu job on the side to run dist_test.py and evaluate.
You can also train for shorter iteration / subsample dataset for faster iteration and only evaluate the last epoch

there is not evaluation for waymo in this repo, should I submit the prediction file to waymo official ?

In fact, i just want to use the val loss to monitor the training process, not the mAP metric

prediction
I could downsample the val set to reduce the tine cost. how can i add some code to get the val loss? thansks a lot from a new pointcloud object detection learner

i am training custom dataset ,I can't use waymo official metric

you can add some code here https://github.com/tianweiy/CenterPoint/blob/3c64610e78756990c8f67732749806a7a83caa5a/det3d/torchie/trainer/trainer.py#L423

and specify you want validation every x epoch at  ('val': x) https://github.com/tianweiy/CenterPoint/blob/3c64610e78756990c8f67732749806a7a83caa5a/configs/waymo/voxelnet/waymo_centerpoint_voxelnet_1x.py#L210


Display Message

If you have a picture or text and when you press it by using the left click mouse button a message "Copied!" should display. Today, there is a function for it but it is only for . Please take a look at the attached picture.
Is it possibly to do it?
Thank you!

I just do alert() message, users are one eye blind at times and you need to explicitly inform them about some event.

clipboard.js is not opinionated about tooltip implementations. It only does what it needs to do and you can handle that kind of user feedback on the success callback. If you need some inspiration check out the gh-pages from this project which contains the website.


OpenSSL 3.2.0, QUIC, macOS, error 56 on connected UDP socket

current translate_msg() function attempts to set ->msg_name (and ->msg_namelen) with BIO's peer name (connection destination) regardless if underlying socket is connected or not. Such implementation uncovers differences in socket implementation between various OSes.
As we have learned hard way sendmsg() and sendmmsg() on OpenBSD and (MacOS too) fail to send messages with ->msg_name being set on connected socket. In such case the caller receives EISCON errro.
I think translate_msg() caller should provide a hint to indicate whether we deal with connected (or un-connected) socket. For connected sockets the peer's name should not be set/filled by translate_msg(). On the other hand if socket is un-connected, then translate_msg() must populate ->msg_name and ->msg_namelen members.
The caller can use getpeername(2) to see if socket is connected. If getpeername() succeeds then we must be dealing with connected socket and translate_msg() must not set ->msg_name and ->msg_namelen members. If getpeername(2) fails, then translate_msg() must provide peer's name (destination address) in ->msg_name and set ->msg_namelen accordingly.
The propposed fix introduces is_connected() function, which applies getpeername() to socket bound to BIO instance. The dgram_sendmmsg() uses is_connected() as a hint for translate_msg() function, so msghdr gets initialized with respect to socket state.
The change also modifies existing test/quic_client_test.c so it also covers the case of connected socket. To keep things simple we can introduce optional argument connect_first to ./quic_client_test function. Without connect_first the test run as usual. With connect_first the test creates and connects socket first. Then it passes such socket to BIO sub-system to perform QUIC connect test as usual.
Fixes #23251

Checklist

[ ] documentation is added or updated
[x] tests are added or updated

On Mon, Jan 29, 2024 at 12:46:50AM -0800, Hugo Landau wrote:

@hlandau requested changes on this pull request.
__________________________________________________________________
In [1]crypto/bio/bss_dgram.c:

 mh->msg_iov         = iov;
 mh->msg_iovlen      = 1;
 mh->msg_control     = msg->local != NULL ? control : NULL;
 mh->msg_controllen  = msg->local != NULL ? BIO_CMSG_ALLOC_LEN : 0;
 mh->msg_flags       = 0;

}
+
+static int is_connected(BIO *b)
+{

struct sockaddr_storage ss;
socklen_t ss_len = sizeof(ss);

return (getpeername(b->num, (struct sockaddr *)&ss, &ss_len) == 0);

I do not think we want to add a syscall to every BIO_sendmmsg
invocation.
Whether a socket is connected is supposed to be tracked by
BIO_CTRL_DGRAM_SET_CONNECTED (data->connected).

yes, it makes sense. I'll send updated PR shortly. The new version
calls `getpeername()` in `dgram_ctrl()` when `BIO_C_SET_FD` command.
I think this is the kind of change Hugo expects.

On Mon, Jan 29, 2024 at 08:03:08AM -0800, Tom???? Mr??z wrote:

@t8m requested changes on this pull request.
__________________________________________________________________
In [1]crypto/bio/bss_dgram.c:

 mh->msg_iov         = iov;
 mh->msg_iovlen      = 1;
 mh->msg_control     = msg->local != NULL ? control : NULL;
 mh->msg_controllen  = msg->local != NULL ? BIO_CMSG_ALLOC_LEN : 0;
 mh->msg_flags       = 0;

}
+
Please drop this spurious addition.
__________________________________________________________________
In [2]test/quic_client_test.c:

if (test_get_argument_count() > 0) {

  mode = test_get_argument(0);

  if (TEST_ptr(mode) && strcasecmp(mode, "connect_first") == 0) {

      connect_socket();

  }

  if (cc_fd == INVALID_SOCKET) {

      TEST_error("Could not create connected socket");

      return 0;

  }

}

Not sure if it is worth changing it (probably not) but it would be much
easier to just make the test_quic_client to have 2 iterations.
(ADD_ALL_TESTS(test_quic_client, 2);) In one iteration you would use a
connected socket and in the other unconnected one.

I can give it a try, so we will be able to decide which variant will look better.

--
Reply to this email directly, [3]view it on GitHub, or [4]unsubscribe.
You are receiving this because you were assigned. Message ID:
@.***>
References

https://github.com/openssl/openssl/pull/23396#discussion_r1469812845
https://github.com/openssl/openssl/pull/23396#discussion_r1469818578
https://github.com/openssl/openssl/pull/23396#pullrequestreview-1849033275
https://github.com/notifications/unsubscribe-auth/AACSZTZLZBREHJYOXVF6C3DYQ7B3ZAVCNFSM6AAAAABCMDUEU2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQNBZGAZTGMRXGU

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This pull request is ready to merge

Merged to the master, 3.3 and 3.2 branches. Thank you.


dep ensure hangs when ssh key has passphrase

Looks like if there is a github private repo where you fetch the dependency from and your account has an authorised ssh key (which has a passphrase).
When performing a deb ensure was expecting it to prompt for my password however this wasn't the case, instead it would just hang.
Versions:
dep version being used is v0.4.1 installed via brew
Using go version: 1.9.4

Probably not the best solution but you can try it =)
git config --global url."git@github.com:".insteadOf "https://github.com/"

Could a user be alerted by dep in such case? please ? I assume it is a very common use case where people have passphrase on their keys ?

As a temp work around I did for this was to remove the passphrase from the key using ssh-keygen -p

I have this issue as well when running init, even with -gopath:  dep init -gopath -v --skip-tools

I'm having this same error on Ubuntu 16.04, with golang 1.10.1, and dep 1.10.1.

same problem here

Same here..
$ git config --global url."git@gitlab.company.com:".insteadOf "https://gitlab.company.com/"
$ dep ensure -add gitlab.company.com/{group}/module # occur error
$ dep ensure -add gitlab.company.com/{group}/module.git # fetch success

I use gitlab.
${PROJECT_PATH}/vendor/gitlab.company.com/group/go-fastping.git

How to remove git extension?

I found a solution:

first, change the url from https to ssh:

git config --global url."git@github.com:".insteadOf "https://github.com/"

second, config ssh-agent

eval ssh-agent -s ssh-add <path-to-privKey>
hope helpful!

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


Blocky noise on macOS in fullscreen mode

Blocky noise like this appear when application is in fullscreen mode and mouse is moved or something is typed:
.
There is lag when changing between windowed and fullscreen mode.
Windowed appear normal.
Appear normal in screenshot (in fullscreen mode).

Closing as we switched to OpenGL which should fix the issue


How-to: Authenticate using OpenID Connect 1.0 authorization_code flow

Publish a guide on How-to: Authenticate using OpenID Connect 1.0 authorization_code flow
Related gh-499

Closing this as a duplicate of gh-538
NOTE: The demo-sample is configured with formLogin() and social login (Google), which both use OpenID Connect 1.0 authorization_code flow.


lvgl font generation

Is your enhancement proposal related to a problem? Please describe.
Lvgl-library support utf. Zephyr uses some pregenerated font-files based on montserrat font (need to note that montserrat_8.c & montserrat_10.c available for selection during zephyr config are absent). But those files contained glyphs only for base ascii symbols.
Describe the solution you'd like
It will be helpfull to generate used in application font files during build time and also be able to configure additional characters to be included in the file (for example range 0x400-0x4FF for cyrillic, 0x100-0x17F for latin extended).

@nashif no idea abou lvlg 😅
Sorry

LVGL sources includes font-generator script (built_in_font_gen.py), which could be executed in build-time and generate font c-files. This script has an argument '--range' which could be used to select range of generated symbols. Also we can change font-family (for example use roboto fonts instead of montserrat) using '--range' argument.

Hey this has been sitting for ages and reading about the last update I think we can close it down. Feel free to reopen if there's something actionable still.


Fix object size explosion during normalization

How to limit objects from exploding in size during normalization (main problem: "Φ-dispatch" rule)?

This should be fixed by #173, right?

I think so


[Display Placer] Saving and retrieving preset does not seem to work

Extension – Display Placer
Author: @eluce2
Description / steps to reproduce

install DisplayPlacer plugin and Homebrew app
check with terminal to verify displayplacer is installed
Open the Switch Display Preset action
Select add new preset.
Give it some name
Save with cmd+enter
The new preset is nowhere to be found, even after restarting.

Raycast version
1.39.3
Hardware
M1 Macbook pro 14

Having same issue

I'm facing the same issue on Raycast v1.43.2


ls: cross-platform colors

Previously, the colors in ls were restricted to Unix. This PR removes that restriction and uses the lscolors crate instead of a custom implementation.

Plenty of code removed, the best kind of PR, bravo :)
There is a build failure but I am sure you know this!

@sylvestre Thanks, I hadn't seen that yet! It should be fixed now! One Ubuntu and FreeBSD still fail, but for unrelated reasons.

It should be all good now!

Great work, bravo :)


[DataFactory] Add explanation for using untrusted certificate

@yew : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@yew : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@yew : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@yew  looks good to me! TY!

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Adds litmus book to install openebs 0.6

What this PR does / why we need it:

This PR adds litmusbook for installing openebs v0.6 in kubernetes cluster .
This book has been tested on both AWS and GCP cluster .

Special notes for your reviewer:

Logs for litmus pod running this book in GCP cluster :

kubectl log -n litmus litmus-openebs-qk9bd
log is DEPRECATED and will be removed in a future version. Use logs instead.
No config file found; using defaults

PLAY [localhost] ***************************************************************

TASK [Applying openebs operator] ***********************************************

TASK [Applying openebs storageclass] *******************************************
FAILED - RETRYING: Checking Maya-API-Server is running (100 retries left).

TASK [Checking Maya-API-Server is running] *************************************

TASK [Checking OpenEBS-provisioner is running] *********************************

TASK [Checking OpenEBS-Snapshot-Operator is running] ***************************

PLAY RECAP *********************************************************************
<IP_ADDRESS>                  : ok=6    changed=5    unreachable=0    failed=0

Ansible Runner image has to be updated with openebs ansible playbook present in this PR .

Thanks @ashishranjan738 , Have requested some minor changes

<Forseeing some changes to the ansible-runner dockerfile content based on some WIP PRs, we will revisit it, going ahead with the merge for now>


will the project be further developed?

Hi @strukturart ,
at the moment this is not really well maintained.
One of the reasons is that I cam not able to dedicate the same amount of time to the project as I was some years ago.
Anyways, I am happy to review and merge PRs where I can. Please just keep in mind that this is something that I am doing besides my full-time job, so don't expect any communication of deadlines or such things (from my side... not sure about MazeMap)...
Greetings,
Niko

thank you for writing the plugin! I use it here in my osm app for KaiOs: https://github.com/strukturart/o.map
Everything works, only with useOnlyCache: true, no tiles are displayed. i wanted to use it to check if you downloaded all the necessary map data before going on a tour.

or am I misunderstanding useOnlyCache: true and you can't only see the local tiles?


Update variable declaration for IE.

update variable declaration for IE.
There is a let declaration
let urlencoded=""
but modify value of  urlencoded later, suggest to change it as
var urlencoded="";
add declaration for
for(p in postData){
suggest to change it as
for(var p in postData){
If there's misunderstanding, I'm glad to receive your reply.
Thanks.

Hi,
Thank you for the modifications you are proposing.
I'm going to accept them.
However, it seems that you misunderstood the notion of "let/const" variables: a "let" variable is editable after declaration. A const is not.
The issue you were facing is due to IE support of this ES6 feature.
Thank you again for your contribution.
Ben


Resize image for nd

Is there a functionality of resize image for n-d where n > 2? It seems currently tensorflow only support resize images for 2d images (4D tensor). Could you add the functionality?

This is a fine feature request. Since it's a somewhat niche feature, we will not be able to work on this soon, but if someone else wants to take this on, I'd support it.

Though adding dimensions is outside the scope of tf.image in TensorFlow Core, it might be a welcome addition to the image operations in TensorFlow Addons. If you haven't already seen it, I'd also strongly recommend taking a look at TensorFlow Graphics! 😄
Transferring this issue to tensorflow/addons; @seanpmorgan for visibility.


Kale examples are failing

All the kale examples are failing docker images is from docker/kale/Dockerfile
kubeflow deployed on eks without rok
i am getting the following error for all the sample experiments
can i have some input on this please ?
Thank you

Hi there
Not sure if it's related with https://github.com/kubeflow-kale/kale/issues/377 because I'm installing some tensorflow packages ... maybe worth looking to jupyter-client version ...
The error that came out in my tests was:
  File "/usr/local/lib/python3.8/dist-packages/kale/common/jputils.py", line 210, in capture_streams
    msg_type = msg['header']['msg_type']
TypeError: 'coroutine' object is not subscriptable
/usr/lib/python3.8/threading.py:934: RuntimeWarning: coroutine 'ZMQSocketChannel.get_msg' was never awaited
  self._invoke_excepthook(self)

which is the same one of your screenshot @raghulkrishna
@StefanoFioravanzo @elikatsis
Best 👋🏾


Compute relationId in a more generic way

@dobe Please review
Should we document somewhere (where?) this limitation?

Current coverage is 81.14% (diff: 86.66%)

No coverage report found for relationid-union at 72f93c0.

Powered by Codecov. Last update 72f93c0...537cda6


Cannot reproduce results

Hello, and thank you for this useful code! I tried to reproduce the unsupervisd BERT+SimCSE results, but failed. My environment setup is as follows:

pytorch=1.7.1 (I also tested 1.8.0)
cudatoolkit=10.1
Single RTX 2080 Ti

The following script is the training script I used (exactly the same as run_unsup_example.sh).
python train.py \
    --model_name_or_path bert-base-uncased \
    --train_file data/wiki1m_for_simcse.txt \
    --output_dir result/my-unsup-simcse-bert-base-uncased \
    --num_train_epochs 1 \
    --per_device_train_batch_size 64 \
    --learning_rate 3e-5 \
    --max_seq_length 32 \
    --evaluation_strategy steps \
    --metric_for_best_model stsb_spearman \
    --load_best_model_at_end \
    --eval_steps 125 \
    --pooler_type cls \
    --mlp_only_train \
    --overwrite_output_dir \
    --temp 0.05 \
    --do_train \
    --do_eval \
    --fp16 \
    "$@"

Then, I obtained following evaluation results:
$ python evaluation.py --model_name_or_path result/my-unsup-simcse-bert-base-uncased/ --pooler cls_before_pooler --task_set sts --mode test
(some log ...)
+-------+-------+-------+-------+-------+--------------+-----------------+-------+
| STS12 | STS13 | STS14 | STS15 | STS16 | STSBenchmark | SICKRelatedness |  Avg. |
+-------+-------+-------+-------+-------+--------------+-----------------+-------+
| 65.14 | 79.35 | 70.48 | 80.72 | 76.45 |    74.21     |      70.97      | 73.90 |
+-------+-------+-------+-------+-------+--------------+-----------------+-------+

I also evaluated the pretrained model (this result are similar to results reported in #25):
$ python evaluation.py --model_name_or_path princeton-nlp/unsup-simcse-bert-base-uncased --pooler cls_before_pooler --task_set sts --mode test
(some log ...)
+-------+-------+-------+-------+-------+--------------+-----------------+-------+
| STS12 | STS13 | STS14 | STS15 | STS16 | STSBenchmark | SICKRelatedness |  Avg. |
+-------+-------+-------+-------+-------+--------------+-----------------+-------+
| 68.40 | 82.41 | 74.38 | 80.91 | 78.56 |    76.85     |      72.23      | 76.25 |
+-------+-------+-------+-------+-------+--------------+-----------------+-------+

I think the gap (2.35 in average) is too large. Is there something wrong in the above training/evaluation scripts? How to obtain ~76 results in STS tasks?

Hi, have you tried python simcse_to_huggingface.py --path result/my-unsup-simcse-bert-base-uncased/ to convert the model's state dict and config before evaluation?

Thank you for the quick response. I just tried to run the script before evaluation, but I obtained the same results..
$ python simcse_to_huggingface.py --path result/my-unsup-simcse-bert-base-uncased/
SimCSE checkpoint -> Huggingface checkpoint for result/my-unsup-simcse-bert-base-uncased/
$ python evaluation.py --model_name_or_path result/my-unsup-simcse-bert-base-uncased/ --pooler cls_before_pooler --task_set sts --mode test
(some log ...)
+-------+-------+-------+-------+-------+--------------+-----------------+-------+
| STS12 | STS13 | STS14 | STS15 | STS16 | STSBenchmark | SICKRelatedness |  Avg. |
+-------+-------+-------+-------+-------+--------------+-----------------+-------+
| 65.14 | 79.35 | 70.48 | 80.72 | 76.45 |    74.21     |      70.97      | 73.90 |
+-------+-------+-------+-------+-------+--------------+-----------------+-------+

In that case, I'm not quite sure how to interpret your results by now. I tried the scripts on google colab (pytorch=1.8.1 cuda=10.1 gpu=Tesla K80), and got an average performance of 75.20, similar to that reproduced in #25. Hopefully this will help.
Also, it seems to me that the intrinsic difference between GPU devices may affect the performance by up to 1 point, and the optimal hyperparamters are likely to alter cross different devices. So I'll suggest to try some simple parameter tuning on batch size, learning rate and pooling method on your own device, and see whether the results get better.

Thank you for the kind response. When I removed --fp16 option, I obtained a similar result (but, task-specific performance is still different from the pretrained model). Although I have known RTX 2000 series support mixed-precision, I guess that there is some issue.
+-------+-------+-------+-------+-------+--------------+-----------------+-------+
| STS12 | STS13 | STS14 | STS15 | STS16 | STSBenchmark | SICKRelatedness |  Avg. |
+-------+-------+-------+-------+-------+--------------+-----------------+-------+
| 69.15 | 82.25 | 74.72 | 81.63 | 78.63 |    78.39     |      69.97      | 76.39 |
+-------+-------+-------+-------+-------+--------------+-----------------+-------+

Hi,
Thanks for reporting this. I think it is mainly caused by differences between GPU versions and CUDA versions. The last reproduced result looks good to me though (fp16 does make a lot of variance).

I think so. Experiments without fp16 would be better for reproducing the reported results and testing other variants. I'm now closing this issue. Thanks again :)


Collection item template functionality should be moved in a tab

Regarding the work on usability/accessiblity done in #728 it is considered more appropriate to move the functionality for creating/editing a collection's item template in a separate tab

Agreed, this might be a good change to make. However, I'd like to have user feedback on this first, as the feature works as is.  Moving this to our 7.1 board to re-discuss during that release


bind: enable parallel building

Description of changes
because non-parallel is slow
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[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/)
23.05 Release Notes (or backporting 22.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.

Have you tried with --shuffle in makeFlags to see if there are any issues with parallel building?

I have now


Add continuous integration with Conda

@proyan Could you handle that?

See https://github.com/stack-of-tasks/pinocchio/blob/master/.github/workflows/macos-linux-conda.yml

Done with #13


Cannot add reactions to messages in forum threads

General Troubleshooting

[X] I have checked for similar issues on the Issue-tracker.
[X] I have updated to the latest JDA version
[X] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Expected Behaviour
When adding reactions to forum thread messages, JDA should be able to add reactions to messages in form threads
Code Example for Reproduction Steps
suspend fun executeFavCommand(storage: Storage, event: SlashCommandInteractionEvent) {
    val interaction = event.reply("Fetching candidates...").await()
    val original = interaction.retrieveOriginal().await()
    original.addReaction("👍").await()
    original.addReaction("👎").await()
}

Code for JDABuilder or DefaultShardManagerBuilder used
val jda = JDABuilder.createDefault(env.authToken).build()

Exception or Error
java.lang.NullPointerException: Cannot invoke "net.dv8tion.jda.api.entities.GuildChannel.getPermissionContainer()" because "channel" is null
at net.dv8tion.jda.internal.entities.mixin.channel.middleman.GuildChannelMixin.hasPermission(GuildChannelMixin.java:48)
at net.dv8tion.jda.internal.entities.mixin.channel.middleman.GuildChannelMixin.checkPermission(GuildChannelMixin.java:51)
at net.dv8tion.jda.internal.entities.mixin.channel.middleman.GuildMessageChannelMixin.checkCanAccessChannel(GuildMessageChannelMixin.java:107)
at net.dv8tion.jda.internal.entities.mixin.channel.middleman.MessageChannelMixin.addReactionBy

Forums are implemented by #2184, can you try with that instead?

@MinnDevelopment Will give it a try, thanks for the hint

Fixed in 5.0.0-alpha.19


Clarify whether trailing slash is meaningful for exact path match

What happened:
The current API spec does not clearly state if a trailing slash is meaningful in an exact path match. (It does clearly state that trailing slashes are ignored for prefix matches though).
https://github.com/kubernetes-sigs/gateway-api/blob/4b0c494fd206cb297ac7e3b0894d0ac9bee50fa9/apis/v1beta1/httproute_types.go#L272-L285
What you expected to happen:
Clear indication of whether or not trailing slash should matter in exact path match.

There may be some guidance in https://datatracker.ietf.org/doc/html/rfc3986/

not sure about other data planes, but in NGINX in the exact match the trailing slash matters. For example, an exact matchlocation = /abc/ { ... }  requires the request URI to be /abc/.

In 2017, when we started Emissary, I decided that the trailing slash would always be meaningful (mostly because the spec is unclear, and I thought that prefix: /foo matching /foobar/ would confuse people).
 So… not a slam-dunk but on balance I’d probably vote not to consider it significant.

No strong preference either way, but Ingress v1 conformance says they are significant: https://github.com/kubernetes-sigs/ingress-controller-conformance/blob/7ee2f3e2d283acb203be862cacd593c71afb1311/features/path_rules.feature#L195-L199 (if we care about consistency for users who may be relying on this)

+1 to significant. I would be extremely confused on any string match where I say exact: xyz and it doesn't match the literal xyz

+1 to significant, the Ingress precedent compels me.


Issue with import nannyml on Kaggle

Describe the bug
I have tried to reimplement one of NannyML examples in Kaggle. and When I used the import command I got the following error.
import nannyml as nml
from IPython.display import display
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots & scripts
If applicable, add screenshots or code snippets to help explain your problem.

Additional context
TypeError                                 Traceback (most recent call last)
/tmp/ipykernel_28/3486783819.py in
----> 1 import nannyml as nml
2 from IPython.display import display
/opt/conda/lib/python3.7/site-packages/nannyml/init.py in
49 from .drift import AlertCountRanker, CorrelationRanker, DataReconstructionDriftCalculator, UnivariateDriftCalculator
50 from .exceptions import ChunkerException, InvalidArgumentsException, MissingMetadataException
---> 51 from .io import DatabaseWriter, PickleFileWriter, RawFilesWriter
52 from .performance_calculation import PerformanceCalculator
53 from .performance_estimation import CBPE, DLE
/opt/conda/lib/python3.7/site-packages/nannyml/io/init.py in
4
5 from .base import Reader, Writer, WriterFactory
----> 6 from .db import DatabaseWriter
7 from .file_reader import FileReader
8 from .file_writer import FileWriter
/opt/conda/lib/python3.7/site-packages/nannyml/io/db/init.py in
3 #  License: Apache Software License 2.0
4
----> 5 from .database_writer import DatabaseWriter
/opt/conda/lib/python3.7/site-packages/nannyml/io/db/database_writer.py in
1 from typing import Any, Dict, Optional
2
----> 3 from sqlmodel import Session, SQLModel, create_engine, select
4
5 from nannyml._typing import Result
/opt/conda/lib/python3.7/site-packages/sqlmodel/init.py in
135
136 # Export SQLModel specifics (equivalent to Pydantic)
--> 137 from .main import SQLModel as SQLModel
138 from .main import Field as Field
139 from .main import Relationship as Relationship
/opt/conda/lib/python3.7/site-packages/sqlmodel/main.py in
24 )
25
---> 26 from pydantic import BaseConfig, BaseModel
27 from pydantic.errors import ConfigError, DictError
28 from pydantic.fields import SHAPE_SINGLETON
/opt/conda/lib/python3.7/site-packages/pydantic/init.cpython-37m-x86_64-linux-gnu.so in init pydantic.init()
/opt/conda/lib/python3.7/site-packages/pydantic/dataclasses.cpython-37m-x86_64-linux-gnu.so in init pydantic.dataclasses()
/opt/conda/lib/python3.7/site-packages/pydantic/main.cpython-37m-x86_64-linux-gnu.so in init pydantic.main()
TypeError: dataclass_transform() got an unexpected keyword argument 'field_specifiers'

Hey @koo-ec ,
there seems to have been some kind of breaking update in Pydantic recently, I found this referring to it: https://github.com/explosion/spaCy/issues/12034.
Could you check which version of Pydantic you have installed on that failing environment?


Adjusted create-budgets synopsis for better readability.

Adjusted create-budget synopsis for better readability.

@bandersmsft
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


[RFC] API planning

Broad Scope
This package will serve as the source for line search algorithms in SciML packages for nonlinear and optimization solvers. Multiple dispatch should be ensured so that downstream packages can use this package almost as a blackbox.
File/Code Organisation

An implementation of the Wolfe and Goldstein conditions - each as a struct. Associated dispatches for each for checking.
Each linesearch method will be defined as a struct with all the method specific hyperparameters and the common ones are arguments to the associated method dispatches
A kernel (function) for the method to be called in a loop, basically the update equation for the method.
Termination checking, no errors should be thrown but a nothing/NaN return when failure happens.
Extensions for supported AD packages.

We should also allow custom jac/jvp/vjp functions to be provided via kwargs and ignore them if they are not needed in the algorithm.

Line search method is used in NonlinearSolve.jl. Maybe we can consider integrating this part into LineSearch.jl.


[Event Hubs Client] May Change Log Updates

Summary
The focus of these changes is to update the change log content for the May milestone.
Last Upstream Rebase
Friday, May 1, 1:16pm (EDT)
References and Related Issues

Review and update documentation (May Milestone) (#9302)

/azp run net - eventhub - ci

/azp run net - eventhub - ci

/azp run net - eventhub - ci

/azp run net - eventhub - ci