CI: make actions locally testable

One can use https://github.com/nektos/act to test workflows locally, but the may need some tweaking to make them work.
See also https://www.redhat.com/en/blog/testing-github-actions-locally for additional improvements (e.g. mocking gh api)
Motivation:
Currently to test gh releases I needed to create a fork https://github.com/kraina-ai/srai-sandbox, which is a bit obrzydliwe (eng. disgusting). Other actions need commits to remote repo and can take quite some time to get a response.

See also how the playground approach is done in https://github.com/process-analytics/GitHub-Actions-playground


:up: Update dependencie

:rocket: deployment of nhsuk/connecting-to-services succeeded (http://connecting-to-services-pr-331.dev.beta.nhschoices.net)


Support input[type=search]

Pretty simple change! I have a local copy of this and it works great on search inputs.

LGTM :octocat:

Awesome! Let me know when this is released and I can replace my local copy w/ a bower copy.

@bhollis v1.2.0 is released :tada:

Thank you!

Hmm, 1.2.0 doesn't seem to have the updates in its dist folder.

@bhollis oops! I released v1.2.1 :sweat:


Doesn't build on recommended branches/toolchains

PyO3 won't build on my Ubuntu 18.04 (Python 3.6 through APT)
Rustup 1.14.0
Attempted to build on 4 different Rustc versions, listed below:
Rustc STABLE 1.29.0 (What I had installed before I read- failed because of the features macro)
Rustc NIGHTLY 1.32.0 (Lastest nightly- failed with a Rust compiler error)
Rustc NIGHTLY 1.29 2018-07-16 (Version said to be lowest compatible on docs.rs)
Rustc NIGHTLY 1.30 2018-08-18 (Version said to be lowest compatible on the Readme- failed saying the version was too low. Rustup actually didn't have a version dating 2018-08-18 and installed the closest I presume which was 2018-08-17)
About to try some more versions I guess

Working on Rustc NIGHTLY 1.31 2018-10-09

The requirement was involuntarily bumped by https://github.com/rust-lang/rust/issues/55380


Add Window settings

Hi !
This is a very interesting project.
I added the possibility to configure the context (currently vsync and multisampling).
Usage
Instead of
let window = Window::new("Triangle", Some((1280, 720))).unwrap();`

You use
let settings = WindowSettings { vsync: true, multisamples: 4 }; //These are the defaults by the way, compatible with previous hardcoded values.
// Or WindowSettings { multisamples: 4, ..Default::default() }; if only a subset of settings is needed
let window = Window::new_with_settings("Triangle", Some((1280, 720)), settings).unwrap();

Suggestions
Maybe you should start using rustfmt to format code, to keep a consistent style (especially when merging code from others) and allow to reformat code automatically. I tried to keep the existing style.
Also, continuous integration would be great, to check the code at least compiles on main platforms (Windows, Linux, MacOS) and to wasm. It could also enforce te code style. If you are interested, let me know, I could work on it and do a PR.

Hi @Mikayex
Nice contribution, thanks a lot 👍 I only had some minor renaming/documentation so I already merged it.
And I agree with your suggestions too. Until recently, this project has not seen any contributions from others than me so there was not really any need for formatting etc (I only recently added any kind of documentation). But it would make sense to use rustfmt. I have made a (gigantic) commit with formatting changes. About continuous integration, I have avoided that since unit tests doesn't really make sense for this crate, but you are right that compilation is also good to check and also formatting. If you want to set it up, you are more than welcome 👍


Polish toThrowErrorMatchingSnapshot.

Just a few small cleanups :)
cc @maximderbin

Current coverage is 87.45% (diff: 100%)

Merging #1828 into master will not change coverage

@@             master      #1828   diff @@
==========================================
  Files            38         38
  Lines          1220       1220
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1067       1067
  Misses          153        153
  Partials          0          0

Powered by Codecov. Last update 600c9c3...97bfd22


Fixed README logo image rendering

To address this issue, #23, which produces this cropped effect:

I  edited the logo image in Inkscape and made the page fit the content.

In addition to this I edited the html in the README to have:

I.e. I used width="50%" rather than width=400, since the former is more responsive to different screen sizes. However that may be personal. Furthermore I changed the image path to refer to the local image rather than the GitHub web link directly. But you may wish to revert that.

... this surely is an interesting phenomenon. I think changing the width and reference link is a good idea. Thank you!


Translator not handling /castsequence reset=combat Death's CaressSiedendes BlutSiedendes Blut, Herzstoß, , ",

Zalasir9:30 PM
check it is a mix of English and German
Sequences['DB_SquishyDK'] = {
author="Suiseiseki",
specID=250,
helpTxt = 'Talents 2112333',
StepFunction = GSStaticPriority,
lang="deDE",
PreMacro=[[
/cast [combat] Vampirblut
/Cast [combat] Tanzende Runenwaffe
]],
"/castsequence Markzerfetzer, Markzerfetzer, Markzerfetzer, Markzerfetzer, Todesstoß, , ",
"/castsequence reset=combat Death's CaressSiedendes BlutSiedendes Blut, Markzerfetzer, , ",
"/cast Todesstoß",
"/castsequence reset=combat Death's CaressSiedendes BlutSiedendes Blut, Herzstoß, , ",
"/cast Herzstoß",
"/cast Todesstoß",
"/cast Markzerfetzer",
PostMacro=[[
/TargetEnemy [noharm][dead] /Use [combat] /Use [combat] 14
]],
}
Show
TimothyLuke9:50 PM
do you ahve the original?
doh ! I just read the title
hmm its not linking "reset=combat " with out []'s

Ok two issues
reset=combat needs to be [reset=combat]
also the postmacro is now translating back out as
PostMacro=[[
/TargetEnemy [noharm][dead] /Use [combat] /Use [combat] 14
]],
}

castsequence is also adding in an extra comma at the end
"/castsequence Rasgamédulas, Rasgamédulas, Rasgamédulas, Rasgamédulas, Golpe letal**, ,** ",

also handle [[/cast [talent:7/1] Bladestorm;[talent:7/3] Dragon Roar]],


[Accessibility] [Mini Cart] Kebab menu buttons and close ("X") buttons are not labeled.

Steps to Reproduce

Enable NVDA screen reader.
Open page in Google Chrome.
Use TAB or TAB + SHIFT to navigate to the kebab menu and close buttons.
Notice the lack of meaningful name announced by NVDA.

Current Behavior
The kebab menu buttons and close buttons rely on svgs to visually convey their purpose. They lack informative alternative text or inline button labels. The purpose of the buttons is not conveyed to screen reader users.
Expected behavior/code
Ensure that all operable controls have accessible names. In the case of the button elements, inline text will suffice. This text can be visually hidden with CSS, although authors should not use display:none.
Note that the svgs should also be hidden from assistive technology with aria-hidden="true" and `focusable="false".
Environment
Chrome 92, Windows 10, NVDA 2020.4
AEM Core Components v. 2.17
Affected Users
Screen reader users, users of speech input
WCAG SC
4.1.2 Name, Role, Value

Tracked via CIF-2357


Prevent reftran printing new line to screen when run in quiet mode

This pull request prevents reftran from printing a new line to screen when run in quiet mode, bringing it into line with the behaviour of the other DynAdjust modules.

Coverage decreased (-0.02%) to 78.311% when pulling 06331755b879d7b505e3faf5b8725b0f40a9b529 on nicgowans:master into 73299348b5795571eb673644c0ef918894573f6b on icsm-au:master.

Thanks @nicgowans!

My pleasure, @rogerfraser!


Add styles for tasklist

Generate styles for tasklist preserving theme colors

to

A complete implement for pages-themes/hacker#75. Resolves #75
Preview a demo at https://davorpa.github.io/github-pages-themes-hacker/#what-about-task-lists

@parkr could you approval me to workflow?
My build and test env works fine
davorpa/github-pages-themes-hacker#5

This is a great idea and simple implementation.

This LGTM, though it would be great to test this on all major browsers.

What is the pointcut?

Some feedback in my scope.
Windows - Chrome
As seen as in PR description. I dont remember the version I took the snapshot with :worried:. Now I have 94.0.4606.81 and it looks the same.
Windows - Edge 94.0.992.38

Windows - IE 11.0.9600.19596

Need fix. Maybe due to missing ms- vendor property

(I left GitHub and can't merge this, but I give it my approval!)

در تاریخ چهارشنبه ۱۳ اکتبر ۲۰۲۱، ۲۳:۱۰ David Ordás <
@.***> نوشت:

Still better than what we started with!
I have in my backlog apply some IE11- hack to remove this decoration. I'll
also try to increase checkboxes size to be pixel perfect more or less.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pages-themes/hacker/pull/83#issuecomment-942657218,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOOITSZLOC6I3O7AUW5JWFLUGXOELANCNFSM5DDFHWQQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Would prefer this kept open.

Bump


Remove PIXI namespace?

Re. #174, here's an experimental no-PIXI build: phaser-test.js.zip
PIXI.Sprite is now Phaser.DisplaySprite and all other properties are moved to Phaser (e.g., Phaser.DisplayObject).

@samme Do you have somewhere the repository or a branch, where you've removed the pixi dependency?
Just wanted to look into the changes you made.

phaser-ce/tree/no-PIXI


Dead zone in drag-and-drop target

You have to move a cell up 100% of the way to the next cell to get it to drop. This is inconveniently far. Evan's system also displays a cursor change that suggests, possibly relatedly,  there's a dead zone between the hit box for dropping down to the next cell and the one before that where dropping does nothing, which is particularly annoying.
https://github.com/user-attachments/assets/9660b911-cd1b-41b0-bf9d-f7e1064202e3

Note relation to #142


Register listener

Issue Link: https://issues.apache.org/jira/browse/ARTEMIS-3085?filter=12350956
Hello @clebertsuconic
I have added registerIOCriticalErrorListener method to ActiveMQServer Interface,wrote the method's implementation and added IOCriticalErrorListenerTest also,though need suggestions on it.
Thank you
Somdatta

Please.. rebase against main:
git remote add github https://github.com/apache/activemq-artemis.git
git pull --rebase github main
git push origin register-listener -f

a merge commit on a pull request is never good.. you should always have your pull request against main. and if you update against main, please rebase and push -f

You should probably squash all your changes Into a single commit.
Also, you are adding the lister to a collection on ActiveMQServer, but you're not doing anything with it. you should register it to the StorageManager when it starts...
ultimately you should just delegate it from the storage manager.
Also: notice the critical error is meant for IO Storage.. not acceptors.. on your test you did an invalid TCP operation, but that wouldn't cause any damage.
We have some tests that are validating the IOCriticalException by simply calling an artificial method, generating a fake failure:
ShutdownOnCriticalIOErrorMoveNextTest was doing that, but that would be a bit convoluted for your test.
As for the change, you will have to change ActiveMQServerImpl a bit further.
Currently the ActiveMQServerImpl is creating a DefaultCriticalErrorListener that will register into the StorageManagers.
I suggest you add this ioCriticalErrorListener to your collection,
and then create a new Listener that will do the following:
ioCriticalErrorListeners.forEach((t) -> t.callitWhateverTheMethodName);

I went ahead and cleaned this up and merged it since it had been sitting idle for so long. Thanks for the contribution, @nbrendah!


Refusal to connect on SignUp with Twitter

It's been quite a few days. Whenever it is tried to Sign Up using Twitter the following loklak.net refused to connect. error is displayed (refer screenshot attached)
Without signing in, loklak.net works just fine.
Enabling/Disabling firewall doesn't help too.

This is known issue because the server side proxy for this has stopped and crashing because an existing pm2 instance is crashing. I've tried to restore the pm2 services but they just keep erroring. Looks like it needs a clean reinstall, we'll do it soon.
@aneeshd16 Could you help a little here ?

@sudheesh001 @aneeshd16
Works fine now. Closing this issue. Thanks :v:

Resolved this yesterday.


Add support for the Gateway object

The functionality I'm looking for is to iterate over gateways and their data sources:
https://learn.microsoft.com/en-us/rest/api/power-bi/gateways/get-gateways
https://learn.microsoft.com/en-us/rest/api/power-bi/gateways/get-datasources
Thank you and keep doing the great job with this package!

@baruchoxman - Have added Gateways. Feel free to take a look and let me know if there's any issues. To install:
pip install git+https://github.com/andrewvillazon/pbipy@45-add-support-for-the-gateway-object

To retrieve the Gateways there is a couple of new methods on the pbi client object: PowerBI.gateway for a single gateway, and PowerBI.gateways for all Gateways. Both return Gateway objects which have a Gateway.datasources method for all attached datasources.

Thanks, that works, feel free to merge!

Merged and released in: v2.11.0


some network related files are written to /var/lib/docker even if using the daemon's -g option

I see some files written in /var/lib/docker even if I use the -g option:
$ sudo docker info
Containers: 0
Images: 416
Server Version: 1.9.1
Storage Driver: overlay
Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.3.0-040300-generic
Operating System: Ubuntu 14.04.3 LTS
CPUs: 8
Total Memory: 7.706 GiB
Name: seb
ID: B2M7:KYAT:FVQA:TA6R:FLAN:DUF5:224N:JFD3:E4UF:KL2Y:FHDM:FPCI
Username: sebihgdata
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Cluster store: consul://localhost:8500
Cluster advertise: <IP_ADDRESS>:2375
$ sudo docker version
Client:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.2
Git commit:   a34a1d5
Built:        Fri Nov 20 13:12:04 UTC 2015
OS/Arch:      linux/amd64
Server:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.2
Git commit:   a34a1d5
Built:        Fri Nov 20 13:12:04 UTC 2015
OS/Arch:      linux/amd64
$ uname -a
Linux seb 4.3.0-040300-generic #201511020949 SMP Mon Nov 2 14:50:44 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
$ ps -aef|grep docker
root      4634     1  0 22:58 ?        00:00:01 /usr/bin/docker daemon --userland-proxy=false --cluster-advertise=wlan0:2375 --cluster-store consul://localhost:8500 --tlsverify --tlscacert=/etc/ssl/docker/ca.cert --tlscert=/etc/ssl/docker/swarm.cert --tlskey=/etc/ssl/docker/swarm.key -H tcp://<IP_ADDRESS>:2375 -H unix:///var/run/docker.sock --bip=<IP_ADDRESS>/16 --dns <IP_ADDRESS> --dns <IP_ADDRESS> -s overlay -g /home/seb/hgdata/deployments/docker
$ sudo ls -al /var/lib/docker
total 12
drw-------  3 root root 4096 nov 28 21:50 .
drwxr-xr-x 77 root root 4096 nov 28 21:50 ..
drw-------  3 root root 4096 dec  8 22:58 network
$ sudo ls -al /var/lib/docker/network/files
total 8
drw------- 2 root root 4096 dec  8 22:58 .
drw------- 3 root root 4096 dec  8 22:58 ..
srw------- 1 root root    0 dec  8 22:58 b673f9086e2d183e888d8febe2d75cde644d25ba60263d31c408ebb04834af7d.sock
$ sudo docker network ls
NETWORK ID          NAME                DRIVER
6ef5e8f4bb34        bridge              bridge
a261ee43d4c6        none                null
2a0486610826        host                host
s$ sudo service docker stop
docker stop/waiting
$ sudo ls -al /var/lib/docker/network/files
total 8
drw------- 2 root root 4096 dec  8 23:08 .
drw------- 3 root root 4096 dec  8 22:58 ..
$ sudo service docker start
docker start/running, process 4980
$ sudo ls -al /var/lib/docker/network/files
total 8
drw------- 2 root root 4096 dec  8 23:09 .
drw------- 3 root root 4096 dec  8 22:58 ..
srw------- 1 root root    0 dec  8 23:09 a9b1e47af2a285d4cb024ae29b34f18ae006a490c93460ac86eb979fa8f3458f.sock

Yes, the current libnetwork version has these paths hardcoded, but this is being fixed in https://github.com/docker/libnetwork/pull/742
/cc @aboch @calavera

This is a massive pain if one is dealing with a read-only / filesystem where /var/lib/docker cannot be created (and thus --graph is used to give Docker a different location to store files in). :cry:

Adding this to the milestone, per https://github.com/docker/libnetwork/issues/747#issuecomment-232667222

ping @mavenugo I see https://github.com/docker/libnetwork/pull/1328 was merged; was this issue resolved? (and do you know which release?)

@thaJeztah : is this issue fixed?
my env on centos 7:
Client:
 Version:      1.11.2
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   b9f10c9
 Built:        Wed Jun  1 21:23:11 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.2
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   b9f10c9
 Built:        Wed Jun  1 21:23:11 2016
 OS/Arch:      linux/amd64

i added the following to :  /etc/systemd/system/docker.service.d/override.conf
[Service]
ExecStart=
ExecStart=/usr/bin/docker daemon --storage-driver=overlay -H fd:// -g /datadisk/docker```

but still files are here:

ll /var/lib/docker/network/files/
total 0
srw-------. 1 root root 0 Nov 14 14:59 7d43489f7eb0e4f2b1ccdd1e9034ff8b9df4e5fec2fc064086c58c5e4d44f4ba.sock
srw-------. 1 root root 0 Nov 14 17:37 961b4fbb80bd0a2650a4e1f27b03c08ae2cdba7ce7a3b6974854dcd6b0f10032.sock

Looks like it's not in 1.11; https://github.com/docker/docker/blob/v1.11.2/vendor/src/github.com/docker/libnetwork/sandbox_externalkey_unix.go#L18
But the change is in 1.12; https://github.com/docker/docker/blob/v1.12.3/vendor/src/github.com/docker/libnetwork/sandbox_externalkey_unix.go#L18
Merged in https://github.com/docker/docker/pull/24237, and cherry-picked into 1.12 through; https://github.com/docker/docker/pull/25045
So looks like this can be closed

thanks for your response @thaJeztah.
when i move to docker version 1.12, when I use -g /datadisk/docker flag, will it create those network socket files to /run/docker/libnetwork/, right?

For future visitors who like me are wondering what that -g option (or its --graph long form cousin) does; apparently, it is the former nomer for what is now --data-root;

(let me x-post here as well); https://github.com/moby/moby/issues/21988#issuecomment-1720865253

Yes, correct. storage drivers used to be referred to as graph drivers. It was later changed to a more generic "storage drivers", because the "graph driver" name was based on how they were implemented, not what they did (and there were already plans to change the implementation; which is what became snapshotters in containerd.
(there's still a lot of references to "graph drivers" - at least in the code though!)


[Snyk] Upgrade refractor from 4.8.0 to 4.8.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade refractor from 4.8.0 to 4.8.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 2 months ago, on 2023-02-28.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Patch and project coverage have no change.

Comparison is base (d595695) 79.40% compared to head (6a3656a) 79.40%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #591   +/-   ##
=======================================
  Coverage   79.40%   79.40%
=======================================
  Files          31       31
  Lines         369      369
  Branches       60       60
=======================================
  Hits          293      293
  Misses         70       70
  Partials        6        6

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

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


Application Case Fixes - multipoint derivative dicts, early_stopping, composite materials, funtofem.out file for remote driver

Bug Fixes for High-Speed Civil Transport Aircraft Application Case
This application case we are using is exercising almost all of the current F2F code.

particular bug that arises when using CompositeFunctions for multiple scenarios with optimization. The analysis functions of a particular scenario are not differentiated with respect to aerodynamic discipline design variables with respect to the other scenarios. Yet the CompositeFunctions which combine information among scenarios such as TOGW will then fail unless all analysisFunctions have at least zero derivatives for each variable.
Early_stopping is included since it got mixed into these commits
Move mapbc files to the scenario folders
funtofem.out file was added since the aerodynamic and structural variable derivatives were actually not read from the sens files of the system call analysis run. Only shape variables were getting read from the *.sens files and this was messing up the optimization.

Had to start over and make a new branch using the parallel tacsAIMs (otherwise the CAD runtime in ESP/CAPS was atrociously slow almost 1:1 with the CFD).


feat: added multi pipeline support

Added an example demonstrating the multi pipeline support including reusing the same tasks within those pipelines.

Awesome, thanks!


AttributeError: partially initialized module 'yfinance' has no attribute 'Ticker' (most likely due to a circular import)

This error keep appearing.
Steps taken:
pip install yfinance
script:
import yfinance as yf
msft = yf.Ticker("MSFT")
msft = msft.history(period="1d")
print(msft)

I'm new with Python and also having this issue. How do you resolve this?


Support access logging in uber filter fuzz test

Description: Some filters (Tap, ESPv2 ServiceControl) implement custom access loggers. For completeness, fuzz these with the same upstream/downstream headers used for decoders/encoders.
Testing:

Ran with seed corpus and ensured access logger was called in Tap filter.
Ran with_libfuzzer for 5 minutes to check for crashes.

Signed-off-by: Teju Nareddy<EMAIL_ADDRESS>

cc @asraa

/retest

/test all


Cleanlight doesn't work

If you type /cleanlight with or without radius, you will get a positive feedback ingame, but nothing happens. The following error appears in the log: https://pastebin.com/ufKkURdP
The error happens world independent.
BKCommonLib version 1.15.2-v4-SNAPSHOT
LightCleaner version 1.15.2-v1
Paper 1.15.2 - 240
FAWE: 1.15.2 - 197

Server not supported, initialization failed
Are you actually using paper, or a weird fork of paper?

I am also getting the same issue with Paper - 245, same version of LightCleaner (1.15.2-v1). So glad I saw this comment because I genuinely thought maybe the server host banned the plugin or something

I'm using paper, but I activated no-tick-view-distance 10 which was implemented yesterday. Could this be releated?

Is it still an issue on newer versions of paper (so 260)?

Yes it does, tested it a minute ago

Alright, I'll have to go and fix support with the newest paper then. Something they changed must've bricked it

So checking back on the logs, it doesn't seen to be showing anything like an error or a warning on loading up the two plugins, but I did notice this warning straight after executing /cleanlight world... https://pastebin.com/HwBuwTJn

In the most likely case, paper devs de-obfuscated that field. Should be an easy fix

https://github.com/bergerhealer/BKCommonLib/commit/855f06a704f7bca8993b1d5899f27b5d6c622e0f
https://ci.mg-dev.eu/job/BKCommonLib/766/

Thanks it works :)


Create automatic Image slider for Landing Page

Description:

Create Automatic Image slider using JS that will display new image every 5 seconds on the landing page in the body section

Automatic Image slider created for landing page using JS.


watchdog: Prevent reload if a file gets closed without any changes

What's the issue?
watchdog 5.0 introduced a new signal that is fired when a file gets closed without any changes written.
This leads to random reloads when something opens and closes a file. opened events are already ignored, so we should ignore the new closed_no_write events too.
How to reproduce?
Update to watchdog 5.0+ and wait for the reloads to come in.
What's the expected behavior?
Werkzeug should only reload if there are actual changes to the code base.
Environment

Python version: 3.11.2
Werkzeug version: 3.0.4
watchdog version: 5.0.2

I'm using Django and django-extensions. I started getting reloads on every request. After doing some checks I was able to find that it started happening after upgrading werkzeug from 3.0.3 to 3.0.4

I'm using Django and django-extensions. I started getting reloads on every request. After doing some checks I was able to find that it started happening after upgrading werkzeug from 3.0.3 to 3.0.4

I am also having the same problem
I am using
werkzeug==3.0.4
watchdog==5.0.2
After a request i get bogus changes detected
backend-1            |  * Detected change in '/backend/common/schema.py', reloading backend-1            |  * Detected change in '/usr/local/lib/python3.12/site-packages/drf_spectacular/openapi.py', reloading backend-1            |  * Detected change in '/usr/local/lib/python3.12/site-packages/drf_spectacular/utils.py', reloading backend-1            |  * Detected change in '/usr/local/lib/python3.12/site-packages/drf_spectacular/generators.py', reloading backend-1            |  * Detected change in '/usr/local/lib/python3.12/site-packages/drf_spectacular/views.py', reloading backend-1            |  * Detected change in '/usr/local/lib/python3.12/site-packages/rest_framework/views.py', reloading backend-1            |  * Detected change in '/usr/local/lib/python3.12/site-packages/django/views/generic/base.py', reloading backend-1            |  * Detected change in '/usr/local/lib/python3.12/site-packages/django/views/decorators/csrf.py', reloading backend-1            |  * Detected change in '/backend/common/authentication.py', reloading postgres-1           | 2024-09-17 08:08:21.866 UTC [104] LOG:  could not receive data from client: Connection reset by peer backend-1            |  * Restarting with watchdog (inotify)

Opened a PR but needs feedback on direction: https://github.com/pallets/werkzeug/pull/2954#issuecomment-2396929700

Thanks @lociii  - much better solution. Closing mine

@ThiefMaster @davidism
This issue breaks runserver_plus from django-extensions for everyone not pinning watchdog to 4.x and using the file system events reload mechanism.
It would be really appreciated if my simple fix could be merged and released.

Thank you very much for reporting @lociii :)
I enforced watchdog version to be <5 in my project to fix it temporarily and move forward


className not set with CSSModules

Currently all my CSSModules className attributes are not being set on the HTML elements, however the classes are being injected into the head. I get no errors in the console nor terminal, and spent a couple hours trying to find answers online first, but came up empty.
The fusebox config I am running is as follows:
let isProduction = false;

const fuse = FuseBox.init({
  homeDir: 'src',
  output: 'dist/$name.js',
  hash: isProduction,
  sourceMaps: !isProduction,
  plugins: [
    BabelPlugin(),
    [
      SassPlugin(),
      CSSModules(),
      CSSPlugin(),
    ],
    WebIndexPlugin({
      template: 'public/index.html',
    }),
    isProduction && QuantumPlugin({
      treeshake: true,
      removeExportsInterop: false,
      uglify: true,
    }),
  ],
});

if (!isProduction) {
  fuse.dev({
    open: true,
    port: 8080,
  });
}

fuse
  .bundle('app')
  .instructions('> index.js')
  .watch()
  .hmr();

fuse.run();

my npm packages are
  "devDependencies": {
    "babel-core": "^6.26.0",
    "babel-plugin-transform-react-jsx": "^6.24.1",
    "babel-preset-latest": "^6.24.1",
    "fuse-box": "^2.2.2",
    "node-sass": "^4.5.3",
    "postcss-modules": "^0.8.0",
    "uglify-js": "^3.0.28"
  },
  "dependencies": {
    "react": "^15.6.1",
    "react-dom": "^15.6.1"
  }

That jsx module not adding the className:
import React, { Component } from 'react';
import styles from './App.scss';

class App extends Component {
  render() {
    return (
colored words
);
  }
}

export default App;

Am I perhaps missing something obvious, or have I stumbled across a bug?

And your jsx please

I've updated my original question with the JSX in question. Please note that this works perfectly fine in my WebPack configuration - I was simply comparing WebPack against FuseBox.

Unfortunately, I wasn't able to reproduce it. Here is my sass file
.hello {
    color: red;
}

The import (i am using typescript)
import styles from "./modules.scss";
console.log(styles);

The output;
{ "hello" : "_hello_foqbe_1" }

And the whole config:
const { FuseBox, WebIndexPlugin, SassPlugin, CSSModules, CSSPlugin } = require("fuse-box");
const fuse = FuseBox.init({
    homeDir: "src",
    output: "dist/$name.js",
    plugins: [
        WebIndexPlugin({
            template: "src/index.html",
        }), [
            SassPlugin(),
            CSSModules(),
            CSSPlugin(),

        ],
    ]
});

fuse.dev();
fuse.bundle("app").hmr().watch()
    .instructions(`>index.ts`);

fuse.run()

Try clearing the cache, if it doesn't help, I would appreciate if you create a repository where I can see the issue.
Cheers

You can find the code here:
https://github.com/Cyberlane/fuse-box-test

@Cyberlane you babel transpiler does it wrong
import styles from './App.scss';
console.log(styles);

means default that's a standard which is broken by one of the babel plugins. Which means that styles should be assigned to default.
Please upgrade to a more appropriate babel plugin, or at least the one that conforms to es6 standards. Alternatively, you can use
CSSModules({
  useDefault: false,
})

And continue using your implementation, but I would recommend align with the es6 spec.

@nchanged thanks for pointing out the issue, however I am going to have to throw this back at you.
Please update your own example repo, as my babel plugin was a copy from your example found here:
https://github.com/fuse-box/react-example

Fair enough @Cyberlane
This commit https://github.com/fuse-box/fuse-box/commit/3b793ca2e9fc118a7fb315db37d1ec6dfe7666e4 should "hack" it in<EMAIL_ADDRESS>I personally dislike babel, too many inconsistencies


Append group_id and event_id into context

Motivation
I want to know group_id from the received message of the user.
It comes with the body payload but not involved in further processing.
Possible stoppers
I am not sure that group_id is a universal field for all message types.

3.4.0 released, thanks.

3.4.0 released, thanks.


Minor Grammar Update

Fixed grammar issue in sentence.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azurermps-6.4.0/uninstall-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.4.0)

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

Thank you for your contribution!


RDKBDEV-2856 : Removed hardcoding of GponManager DM index to 1.

Reason for change:

Removed the retry mechanism in GponHal_get_init_data API since no failure is observed.
Removed hardcoding of index number to 1
Changes from Device.X_RDK_ONT.GemStats. to Device.X_RDK_ONT.Gem.

Test Procedure: Performed GponManager Sanity test.
Risks: None.

Internal ticket for tracking https://ccp.sys.comcast.net/browse/RDKCOM-5008


EMFILE error (too many open files) during neovim file watching on macos

When opening multiple large C# projects on my macOS machine, neovim enters into an infinite loop of errors like this:

On my neovim install, _watch.lua line 73 refers to this assert:

Based on googling it seems EMFILE means that we've reached some limit on mac around the number of open files.
This does not occur when "filewatching" is set to false

This could also be a neovim bug issue rather than roslyn.nvim

Nothing I can do about this. It is known the the neovim core team that filewatching is not optimal as it is really difficult.
This is exactly why I have that hacky filewatching option

Yep makes sense
For anyone else that has this problem -
I found some good ideas on solutions in this nvim issue comment
In particular, would be great if neovim had ability configuration options for excludes, as mentioned there.  Using exclude paths in hacks.lua, I was able to reduce my watcher count down by 1/4 which is enough for me to not see this problem anymore.


Migrate to PDM

Update the requirements, add pyproject.toml etc.

Done. As well as FastAPI migration.


XmlDoc: fix folding

Fix xml docs folding because of
https://github.com/dotnet/fsharp/pull/12484#issuecomment-988802545

@DedSec256 Could you also please look into adding some tests for folding features?


[docs/javadoc][hotfix] Corrected Join hint and misleadling pointer to Spargel

This PR adds the following patches:

in the Iteration programming guide, there was a misleading link to Spargel, which is deprecated; I switched it to point to Gelly;
while trying to use join hints, I saw that the JavaDoc for BROADCAST_HASH_SECOND was wrong; "Hint that the second join input is much smaller than the second". Fixed it to be smaller than the first :)

looks good, +1.

LGTM

Merging...


Adding Time To Live on Redis Vector Store Index

Is there a way we can add Time To live when storing vectors in Redis? If there isn't is there a plan to add it in the future?

This is definitely possible (given Redis was first and foremost a cache)!
We are working on something like this. Would be interested in talking to you about dev UX for this feature.

Yes I am available to chat!

Shoot me a note at<EMAIL_ADDRESS>and let's discuss

Shoot me a note at<EMAIL_ADDRESS>and let's discuss

I sent you an email via my personal mail. Richard.


Failed to upload backup

It appears the backup upload is not always successful. This could leave the backup inconsistent. I guess the most reasonable thing to do is to remember any failed backup and retry it.
[D] TIME: 2023-11-23T18:35:49.025189Z r: Response status code 502 Bad Gateway
[E] TIME: 2023-11-23T18:35:49.025743Z r: Failed to upload backup.
<title>502 Bad Gateway</title>

502 Bad Gateway

nginx/1.25.1

Related logs from nginx
/27 12:37:57 [error] 32#32: *350173 writev() failed (32: Broken pipe) while sending request to upstream, client: <IP_ADDRESS>, server: api.10101.finance, request: "POST /api/backup/026f77d9666b869ff8bea28552e6e7c33cd558331b5da7fb2e9b5ed6c3d94a80d9 HTTP/1.1", upstream: "http://<IP_ADDRESS>:8000/api/backup/026f77d9666b869ff8bea28552e6e7c33cd558331b5da7fb2e9b5ed6c3d94a80d9", host: "<IP_ADDRESS>"
2023/11/27 12:38:01 [error] 32#32: *350169 writev() failed (32: Broken pipe) while sending request to upstream, client: <IP_ADDRESS>, server: api.10101.finance, request: "POST /api/backup/026f77d9666b869ff8bea28552e6e7c33cd558331b5da7fb2e9b5ed6c3d94a80d9 HTTP/1.1", upstream: "http://<IP_ADDRESS>:8000/api/backup/026f77d9666b869ff8bea28552e6e7c33cd558331b5da7fb2e9b5ed6c3d94a80d9", host: "<IP_ADDRESS>"
2023/11/27 16:04:55 [error] 33#33: *356939 writev() failed (32: Broken pipe) while sending request to upstream, client: <IP_ADDRESS>, server: api.10101.finance, request: "POST /api/backup/02ef8fa81773a996bf87e3df16f567a0811fe89dbe55f0e6293526e7c913c394e2 HTTP/1.1", upstream: "http://<IP_ADDRESS>:8000/api/backup/02ef8fa81773a996bf87e3df16f567a0811fe89dbe55f0e6293526e7c913c394e2", host: "<IP_ADDRESS>"
2023/11/27 16:05:00 [error] 33#33: *356936 writev() failed (32: Broken pipe) while sending request to upstream, client: <IP_ADDRESS>, server: api.10101.finance, request: "POST /api/backup/02ef8fa81773a996bf87e3df16f567a0811fe89dbe55f0e6293526e7c913c394e2 HTTP/1.1", upstream: "http://<IP_ADDRESS>:8000/api/backup/02ef8fa81773a996bf87e3df16f567a0811fe89dbe55f0e6293526e7c913c394e2", host: "<IP_ADDRESS>"
2023/11/27 19:43:53 [error] 29#29: *363520 writev() failed (32: Broken pipe) while sending request to upstream, client: <IP_ADDRESS>, server: api.10101.finance, request: "POST /api/backup/03540d76709996665bf6078cff2851d9d5ec5027b89c2529be708c740617f82526 HTTP/1.1", upstream: "http://<IP_ADDRESS>:8000/api/backup/03540d76709996665bf6078cff2851d9d5ec5027b89c2529be708c740617f82526", host: "<IP_ADDRESS>"
2023/11/27 19:43:57 [error] 42#42: *363381 writev() failed (32: Broken pipe) while sending request to upstream, client: <IP_ADDRESS>, server: api.10101.finance, request: "POST /api/backup/03540d76709996665bf6078cff2851d9d5ec5027b89c2529be708c740617f82526 HTTP/1.1", upstream: "http://<IP_ADDRESS>:8000/api/backup/03540d76709996665bf6078cff2851d9d5ec5027b89c2529be708c740617f82526", host: "<IP_ADDRESS>"

It appears we can also run into a 503 when posting an order.
[E] TIME: 2023-11-28T15:13:20.053520Z r: Failed to post new order. Error: Could not create new order: Response { url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Ipv4(<IP_ADDRESS>)), port: None, path: "/api/orderbook/orders", query: None, fragment: None }, status: 503, headers: {"server": "nginx/1.25.1", "date": "Tue, 28 Nov 2023 15:13:20 GMT", "content-type": "application/json", "content-length": "70", "connection": "keep-alive"} } order_id: 78b4d819-8e65-40dd-801f-0cd4d93f7d92

related to https://github.com/get10101/10101/issues/1737


LibWeb/HTML: Implement download attribute

Required for Pixedit.
Ref: #102

It feels inaccurate to say that this is "implementing the download attribute" but sure, we are starting to implement it ;)


ch02: Typo in helloworld width/height

back-link: ch02/index.html#hello-world

This code in particular show a rectangle of 300 x 300 pixel geometry

Actual code example is 360 x 360

Seems I changed the code after the writing:-)


[BUG] Unable to select

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Device you are running wire-pod on (please complete the following information):

OS: [e.g. Debian, Windows]
Hardware: [e.g. Raspberry Pi]

Additional context
Add any other context about the problem here.

Accidentally pressed Shift+Enter to submit, will edit...


Remove unnecessary XSS decodes

Please test user-mgt stuff in UI because most of the changes are in that.

Fixed review comments with #1326


Re-generate a different random project name

The user might not like the random name that was generated, instead of forcing them to come up with something themselves we should just have a button (using a refresh icon similar to ♻) to generate a different one.

Awesome idea :-)
Le mar. 5 mars 2019 à 19:39, Tako Schotanus<EMAIL_ADDRESS>a
écrit :

The user might not like the random name that was generated, instead of
forcing them to come up with something themselves we should just have a
button (using a refresh icon similar to ♻) to generate a different one.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fabric8-launcher/launcher-frontend/issues/946, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACHvcLTpkQDgQjGaAdiCT5F3dJSIwp_vks5vTrnhgaJpZM4bfP4q
.

--
Andy Damevin


[WIP] RAND: Moving seeding out of the FIPS provider.

Due to a rule change by NIST about the validation requirements for operating system sources, it is necessary to move the seeding out of the FIPS provider and rely on a pull model instead.

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

This needs a rethink.  A provided seed source isn't possible inside the FIPS provider.  An alternate method of getting seed material in is required.

In addition to the seed callbacks, nonce ones are also required.
Also made the test RNG not a DRBG.  It isn't one and pretending to be one hid things that are better unhid.

Assuming the CIs are happy, should be good for a reviewing.

Ping for review.

Re-ping for review.  @openssl/committers ?

ping for review?

Still awaiting @openssl/committers review...

Review comments addressed.

Ping for rereview

Re-ping for review.

What's confusing, is that some of the review comments are about a commit that shouldn't be in this PR.

Could someone with access to an ARM64 system let me know what's going on with the failed test?
It passes  here under ASAN.

Feedback addressed

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

There were merge conflicts from #13417 which caused bit of pain.  They were function renaming issues, so no code changes.  I don't feel that reapproval is necessary however, I'll wait for the CIs before merging, just to be certain.

The no-deprecated builds are failing with:
ssl/ssl_lib.c: In function 'SSL_set0_tmp_dh_pkey':

ssl/ssl_lib.c:5963:9: error: implicit declaration of function 'SSLerr'; did you mean 'stderr'? [-Werror=implicit-function-declaration]

         SSLerr(0, SSL_R_DH_KEY_TOO_SMALL);

         ^~~~~~

which isn't relevant to the changes here.

Merged to master.  Thanks for the reviews, feedback and suggestions.


Darek

Cleaning V.2

Module 01 - Making Datasets, Avoiding Leakage
Overall: You're off to a solid start with the Lending Club data! I can tell you've gotten really good at wrangling data into the shape and format that you want it 👍
Feedback: Get in the habit of being explicit about what you're doing and why. If you can explain your reasoning to a non-technical reader, you're quality of communication accounts for virtually every stakeholder you'll engage with. Leverage comments and text cells more.
Get the data

Glob FTW! For future reference, it can be easier to assign your path to a variable and then use that to work with multiple files inside the same notebook. For example:
path =  '/Users/YourUserName/Documents/etc/etc/FILES!'

Choose number of observations

I see that you split the data but it would be great to add a text cell or some commenting to explain how/why you decided to do so (code etiquette).
Nice job keeping it down to 141MB

Save your Data for reuse

👍

Choose Features

Great job killing whitespace and cleaning emp_length. Nicely done using anonymous functions
Same as above. Document your choices more explicitly with text cells / comments.

SC Review
Overall: You've come a helluva long way in 9 weeks.
Feedback: N/A | Keep doing what you're doing.
Making Datasets, Avoiding Leakage

Glob is a superpower. Remember that tomorrow in case you need to combine multiple csvs.
Key point of Data Hygiene in predictive modeling: leakage can easily lead you astray so proactively prevent it. Removing features is one really effective way to go about it. Consider Time-Series!

Start with Baselines

Not an explicit learning objective of this Sprint but so important that it's worth reiterating 👍

Class Imbalances

When confronting classification problems, accuracy may not be a good measure because it doesn't factor True Positive rate. But the threshold for this is contextual. You want to bear in mind the consequences of your choices when factoring this in. Par for the course re: Predictive Modeling & Decision Making.
If you determine that you can't afford false positives, use ROC AUC as a more appropriate metric for evaluating your model. By plotting the true positive rate vs the false positive rate at different thresholds, you can better understand your model's predictive power.

Making & Explaining Decisions

The whole point of Data Science (if you can reduce it down to just one) is to draw actionable insights. That means making decisions. In order to better calibrate that decision-making, we leverage Expected Values, and Shapley Value Plots. Review the lecture notebook for links to dive deeper into both of these topics if you wish!

Gradient Descent

How can you programaticaly minimize a value you care about? Gradient Descent refers to a method that iteratively discovers minima. The trick is understanding if they are local or global, for one. For another, computation is not free. The time and effort it takes your model to minimize it's errors is a consideration you must make. Calibrate the learning_rate as one knob/dial.


The country dropdown should be  in the Ascending order when the customer is adding the delivery address on the checkout page.

Issue Description
When we proceed to the checkout and there we fill in the address and select the country, the country format is appearing in Z-A. It must be in A-Z format.
**Bagisto Version - ** Master
Steps To Reproduce
1. Add any product to cart -> Proceed to checkout - > Fill the address manually -> Select Country
2. In Country Dropdown we can observe that the Z-A format is appearing.
Actual Result:  The country dropdown is in Descending order. Please find the screenshot Here
Expected Result:  The country dropdown should be in Ascending order.

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


proposal: AmeshPluginConfig support

AmeshPluginConfig Resource
The resource AmeshPluginConfig is a custom resource that defines all the plugin configurations that will be used by the APISIX.
A minimum sample configuration is as follows:
apiVersion: apisix.apache.org/v1alpha1
kind: AmeshPluginConfig
metadata:
  name: go-plugins
spec:
  type: pre-req
  plugins:
  - name: say
    config: "{\"body\":\"hello\"}"

This CRD indicates that the routes should be injected with the following plugin configuration:
{
  // ... Route configurations ...
  "plugins": {
    "ext-plugin-pre-req": {
       "conf": [
         {"name":"say", "value":"{\"body\":\"hello\"}"}
       ]
    }
  },
}

If you only want the plugin to run on specific workloads, you can add the selector field for filtering.
apiVersion: apisix.apache.org/v1alpha1
kind: AmeshPluginConfig
metadata:
  name: go-plugins
spec:
  type: pre-req
  plugins:
  - name: say
    config: "{\"body\":\"hello\"}"
  # selectors
  nodeSelector:
    kubernetes.io/hostname: node1
  selector:
    matchLabels:
      component: redis
    matchExpressions:
    - key: zone
      operator: In
      values:
      - foo
      - bar
    - key: environment
      operator: NotIn
      values:
      - dev

For more information about selectors, please refer to Resources that support set-based requirements.
Custom Plugins
Apart from the official plugins, users can also add custom plugins to the Amesh.
To make the image used by the sidecar include user custom plugins, the user needs to build a new image based on Amesh's APISIX image and the user's plugins.
Test Plan
We need a complete and solid test suite to validate this functionality, including the following cases:

[ ]  Test the plugin configuration is injected into the routes
[ ]  Test the plugin with a custom selector
[ ]  Test the plugin definition updates are reflected

SGTM

Done


benchmark test fail to run with telemetry v2 nullvm on both sidecar mode

When running telemetry v2 nullvm setting on both sidecar mode, we are getting error message "Error: Aborting because error -1 for http://fortioserver:8080/echo?size=1024: ", need to investigate why.
Also data for nomixer on both sidecar mode seems not correct though.
@carolynhu @mandarjog

I know most http load client would quit if the client/server has too many errors.

The issue was caused by using outdated envoyfilter url.  PR to update merged.


Make the SDK be more configurable

background
https://kaligo.atlassian.net/browse/GHP-3713
design
Expose two new configurations for the SDK:

client_config: define the configurations for the workflow client
payload_converters_options:  define the configuration for the payload converters

Sample configurations
Temporal.configure do |config|
  config.host = ENV.fetch("TEMPORAL_HOST", "localhost")
  config.port = Integer(ENV.fetch("TEMPORAL_PORT", 7233))
  config.namespace = ENV.fetch("TEMPORAL_NAMESPACE", "nydus")
  config.credentials = GRPC::Core::ChannelCredentials.new(root_cert, client_key, client_chain)

  # NEW CONFIGS

  config.client_config = {
    channel_args: {
      ::GRPC::Core::Channel::SSL_TARGET => ENV["TEMPORAL_SERVER_NAME"]
    }
    # more configs for the workflow client go here
  }
  config.payload_converters_options = {
    'json/plain' => {
      circular: true
      # more configs for JSON converter go here
    }
   # more configs for other converter go here
  }
end

Overall LGTM, could we also add some documentations to readme?

Self-QA

[x] setup a Cashback auto-redemption on Anz staging, the according workflow completes successfully without any error


powerpc-eabi-gcc: No such file or directory

i cloned the grrlib source code and entered the directory GRRLIB and ran make clean but when i run make all i get this
`

pngu.c
make[2]: powerpc-eabi-gcc: No such file or directory
make[2]: *** [pngu.o] Error 1
make[1]: *** [all] Error 1
make: *** [all] Error 1
`

It seems to be a problem with your toolchain. Can you compile other Wii homebrews?
If you need help with devkitPro you can look at https://devkitpro.org/wiki/Getting_Started

i can compile the wii examples template but i don't think that counts

Are you on Linux, Windows or in a Docker container?

macos

Can you paste the whole output when using make clean all in the GRRLIB folder, for example:
make[1]: Entering directory '/workspaces/GRRLIB/GRRLIB/GRRLIB'
rm -f GRRLIB_gecko.o GRRLIB_print.o GRRLIB_core.o GRRLIB_ttf.o GRRLIB_texEdit.o GRRLIB_fileIO.o GRRLIB_bmf.o GRRLIB_fbAdvanced.o GRRLIB_bmfx.o GRRLIB_render.o GRRLIB_3D.o GRRLIB_snapshot.o GRRLIB_gecko.d GRRLIB_print.d GRRLIB_core.d GRRLIB_ttf.d GRRLIB_texEdit.d GRRLIB_fileIO.d GRRLIB_bmf.d GRRLIB_fbAdvanced.d GRRLIB_bmfx.d GRRLIB_render.d GRRLIB_3D.d GRRLIB_snapshot.d libgrrlib.a
make[1]: Leaving directory '/workspaces/GRRLIB/GRRLIB/GRRLIB'
make[1]: Entering directory '/workspaces/GRRLIB/GRRLIB/lib'
make[2]: Entering directory '/workspaces/GRRLIB/GRRLIB/lib/pngu'
rm -f pngu.o pngu.d libpngu.a
make[2]: Leaving directory '/workspaces/GRRLIB/GRRLIB/lib/pngu'
make[1]: Leaving directory '/workspaces/GRRLIB/GRRLIB/lib'
-----------------------------------------------------------
make[1]: Entering directory '/workspaces/GRRLIB/GRRLIB/GRRLIB'
GRRLIB_gecko.c
GRRLIB_print.c
GRRLIB_core.c
GRRLIB_ttf.c
GRRLIB_texEdit.c
GRRLIB_fileIO.c
GRRLIB_bmf.c
GRRLIB_fbAdvanced.c
GRRLIB_bmfx.c
GRRLIB_render.c
GRRLIB_3D.c
GRRLIB_snapshot.c
powerpc-eabi-gcc-ar -r libgrrlib.a GRRLIB_gecko.o GRRLIB_print.o GRRLIB_core.o GRRLIB_ttf.o GRRLIB_texEdit.o GRRLIB_fileIO.o GRRLIB_bmf.o GRRLIB_fbAdvanced.o GRRLIB_bmfx.o GRRLIB_render.o GRRLIB_3D.o GRRLIB_snapshot.o
/opt/devkitpro/devkitPPC/bin/../lib/gcc/powerpc-eabi/12.1.0/../../../../powerpc-eabi/bin/ar: creating libgrrlib.a
make[1]: Leaving directory '/workspaces/GRRLIB/GRRLIB/GRRLIB'
-----------------------------------------------------------
make[1]: Entering directory '/workspaces/GRRLIB/GRRLIB/lib'
-----------------------------------------------------------
make[2]: Entering directory '/workspaces/GRRLIB/GRRLIB/lib/pngu'
pngu.c
powerpc-eabi-gcc-ar -r libpngu.a pngu.o
/opt/devkitpro/devkitPPC/bin/../lib/gcc/powerpc-eabi/12.1.0/../../../../powerpc-eabi/bin/ar: creating libpngu.a
make[2]: Leaving directory '/workspaces/GRRLIB/GRRLIB/lib/pngu'
make[1]: Leaving directory '/workspaces/GRRLIB/GRRLIB/lib'

 anthonyvaldes@aj  ~  cd Downloads/GRRLIB-4.4.1/  anthonyvaldes@aj  ~/Downloads/GRRLIB-4.4.1  ls CHANGELOG.md   LICENCE.md     docs           grrlib.doxygen GRRLIB         README.md      examples  anthonyvaldes@aj  ~/Downloads/GRRLIB-4.4.1  cd grrlib  anthonyvaldes@aj  ~/Downloads/GRRLIB-4.4.1/grrlib  cd grrlib  anthonyvaldes@aj  ~/Downloads/GRRLIB-4.4.1/grrlib/grrlib  ls GRRLIB_3D.c         GRRLIB_fbAdvanced.c GRRLIB_render.c     Makefile GRRLIB_bmf.c        GRRLIB_fileIO.c     GRRLIB_snapshot.c   grep GRRLIB_bmfx.c       GRRLIB_gecko.c      GRRLIB_texEdit.c    grrlib GRRLIB_core.c       GRRLIB_print.c      GRRLIB_ttf.c        grrlib.h  anthonyvaldes@aj  ~/Downloads/GRRLIB-4.4.1/grrlib/grrlib  make clean all rm -f GRRLIB_3D.o GRRLIB_bmf.o GRRLIB_bmfx.o GRRLIB_core.o GRRLIB_fbAdvanced.o GRRLIB_fileIO.o GRRLIB_gecko.o GRRLIB_print.o GRRLIB_render.o GRRLIB_snapshot.o GRRLIB_texEdit.o GRRLIB_ttf.o GRRLIB_3D.d GRRLIB_bmf.d GRRLIB_bmfx.d GRRLIB_core.d GRRLIB_fbAdvanced.d GRRLIB_fileIO.d GRRLIB_gecko.d GRRLIB_print.d GRRLIB_render.d GRRLIB_snapshot.d GRRLIB_texEdit.d GRRLIB_ttf.d libgrrlib.a GRRLIB_3D.c make: powerpc-eabi-gcc: No such file or directory make: *** [GRRLIB_3D.o] Error 1  ✘ anthonyvaldes@aj  ~/Downloads/GRRLIB-4.4.1/grrlib/grrlib 

When you type echo $DEVKITPPC in your console where you are trying to compile, what do you get?

/opt/devkitpro/devkitPPC

And when you type /opt/devkitpro/devkitPPC/bin/powerpc-eabi-gcc --version

powerpc-eabi-gcc (devkitPPC release 41) 12.1.0
Copyright (C) 2022 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Maybe you have an other version of the compiler! Can you simply try powerpc-eabi-gcc --version

zsh: command not found: powerpc-eabi-gcc

The only thing I could think of is that you might have not done step 3:

To get the Xcode command line tools run xcode-select --install from Terminal.
Use the pkg installer to install devkitPro pacman.
Reboot your mac to get environment set.
Use dkp-pacman to install your chosen tools as per Unix like platforms below.

Here is a post of someone having a similar problem: https://stackoverflow.com/questions/71047549/no-such-file-or-directory-arm-none-eabi-gcc-error-trying-to-make-micropython

i installed it a few days ago and i rebooted after

ill reboot again

i added export DEVKITPPC to my zshrc because i forgot but it still doesn't work

i added /opt/devkitpro/devkitPPC/bin to my path and now it compiles

forgot to make the last comment a close with comment comment so im closing it now


stm32cubemx: wrap with FHSEnv

Description of changes
As of recently STM32CubeMX is using JxBrowser for, now mandatory, user authentication.
Same browser is extracted from STM32CubeMX install binaries into ~/.stm32cubemx directory and depends on the system shared libraries.
This PR wrapps STM32CubeMX with FHSEnv in order call JxBrowser.
Package remains mostly the same except a minor change where stdenv is replaced by stdenvNoCC.
STM32CubeMX login page takes a while to appear on Wayland (Sway) but works mostly fine.
Fixes: #272342
cc @angaz @wucke13

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Tested running the application using nixpkgs-review. Tested creating a new project, and generating the code. The login window opened, it didn't take in my key presses very well. I had to type the same key multiple times before it accepted a key. Very annoying for the password, but I would consider this a bug in the app, not the nix configuration.
From what I can tell, it's working as expected, and the mentioned bug is resolved.

I found copy/paste to be best solution for credentials input.
I have a feeling those issues are related to GPU acceleration in JxBrowser i.e. Chromium.
It's possible to disable hwaccel by using chrome  cmdline flags, but exec string is hardcoded in STM32CubeMX and since login is required only once I think it's not worth to persue further.
Thanks for testing and review.

Cool. Thanks for updating the platforms. Everything looks good to me.
I'm not certain if the commit history is correct. Should changing the platforms be in it's own commit?

I squashed it, but I guess it would make more sense to have separate commit now that I think more about it.
Thanks for approwing it again, so I'll not touch it anymore.

I squashed it, but I guess it would make more sense to have separate commit now that I think more about it. Thanks for approving it again, so I'll not touch it anymore.

Yeah, my comment was more for future reviewers. Maybe it's something they would want.

I squashed it, but I guess it would make more sense to have separate commit now that I think more about it. Thanks for approving it again, so I'll not touch it anymore.

Yeah, my comment was more for future reviewers. In case they don't see that it's been changed. Maybe it's something they would want.

Understand, added separate commit.


Use JUnit parameterized runner for dialect-sensitive integration tests

Two commits:

Inline base class. It is clearer to have all test code inlined. If you must make a separate utility method, better to do it in the same class. Inheritance is almost never good in test classes.
Use JUnit Parameterized test runner instead of utility method. Also converted to a PTransform instead of a method, which is good Beam style for pipelines. Also made the parameterized tests enclosed, because we could also have ITs that do not depend on SQL dialect.

The two commits are independent, please do not squash (for history to be most readable)

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

run sql postcommit

I tried this idea already but results in an illegal BQ table name error.

The test name can be customized. Here is an example: https://github.com/junit-team/junit4/wiki/parameterized-tests#example
However, it always add [ and ] into the test name, and test name are used to create temporary BQ table. Thus it leads a illegal BQ table name.

I'll fix up the test BQ to not create invalid names.

run sql postcommit

run sql postcommit

run sql postcommit

Getting an interesting failure that doesn't seem related. There may be something fragile being exposed by a minor change in how the test is executed.

(https://builds.apache.org/job/beam_PostCommit_SQL_PR/125/testReport/junit/org.apache.beam.sdk.extensions.sql.meta.provider.datacatalog/DataCatalogBigQueryIT$DialectSensitiveTests/testReadWrite_ZETASQL_/)

I notice that the CalciteQueryPlanner and ZetaSQLQueryPlanner do not share the codepath to set up the cluster's metatadata provider.

run sql postcommit

Filed https://issues.apache.org/jira/browse/BEAM-8453 for JmsIOTest failure.
Filed https://issues.apache.org/jira/browse/BEAM-8454 for Java SDK harness failure.

run java precommit

run sql postcommit

Got multiple local successes. Running here: https://gradle.com/s/l572gifqnx5pg
The metadata provider issue appears to be a flake. The two planners set up metadata totally differently. We should fix that.
Here one success: https://gradle.com/s/l572gifqnx5pg


Bundle is locked to twig-bundle 5.4 rather then ^5.4

Compose is locked on 5.4 rather locking to any 5.4.*
"symfony/twig-bundle": "5.4 || ^6.0"

Thanks @On5-Repos for reporting this issue.
It should be fixed with 2b2f950.
Can you please verify whether it works fine for you and close the ticket afterwards?

its working, thanks.


Image Batch: Input image dimensions do not match for images

Getting error: What am I doing wrong? It worked a few weeks ago but now doesn't.
_Error occurred when executing Image Batch:
WAS Image Batch Warning: Input image dimensions do not match for images: ['images_b']
File "E:\ComfyUI\execution.py", line 153, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "E:\ComfyUI\execution.py", line 83, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "E:\ComfyUI\execution.py", line 76, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "E:\ComfyUI\custom_nodes\was-node-suite-comfyui\WAS_Node_Suite.py", line 4712, in image_batch
self._check_image_dimensions(batched_tensors, image_names)
File "E:\ComfyUI\custom_nodes\was-node-suite-comfyui\WAS_Node_Suite.py", line 4703, in check_image_dimensions
raise ValueError(f"WAS Image Batch Warning: Input image dimensions do not match for images: {mismatched_images}")

I have the same issue. Did you get this resolved?

I just adjusted the logic that the original author did. Seems they
attempted to check all dimensions at once rather than using first as a
guide for rest.
On Sun, Jan 21, 2024, 10:43 AM rainstormads @.***>
wrote:

I have the same issue. Did you get this resolved?
—
Reply to this email directly, view it on GitHub
https://github.com/WASasquatch/was-node-suite-comfyui/issues/276#issuecomment-1902724592,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIZEZLPKDUK2TSI5YILOKLYPVOWVAVCNFSM6AAAAAA7TJVSZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBSG4ZDINJZGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


MacOS root user is not able to use Twingate (MacOS: 12.6.6)

I have been using your application twingate for MacOS which I download over the AppStore. It works fine on the user account. However when I try to use twingate as the Root user using the root login on MacOS it fails.
I have troubleshooted the issue and this is the error:
2024-04-20 20:52:15.744+0200 [ERROR] [client] Unable to evaluate DeviceOwnerAuthenticationWithBiometrics policy, code: -8, exception: Biometry is locked out.
2024-04-20 20:52:15.803+0200 [ERROR] [client] -[VPNManager finishStartWithError:] [Line 374] Start finished with error Error Domain=com.anchorfree.hydra.manager Code=4 "failed_to_connect" UserInfo={NSLocalizedDescription=failed_to_connect}
2024-04-20 20:52:15.803+0200 [ERROR] [client] Connect failed: failed_to_connect

While digging into it deeper the error is all about the fact that twingate checks the biometry and if its locked out it dose not allow the connection to the network. While this is ok for most cases in the case of the root user the biometry is always locked out cause biometry is not needed.
Hope someone can reach out and offer a fix for this.

I am sorry but I really need help here.

The issue here is likely with your VPN configuration and not with the biometric check as we handle that error. Validate that the Twingate VPN profile is enabled in System Settings.

Could you please be more specific under which system settings?

See my comment here: https://github.com/Twingate/pulumi-twingate/issues/50#issuecomment-2096714653
Please reach out via the Help Center or the Twingate community forum (links above), as this is not a place for technical support that isn't related to Pulumi.


Add info to change installer UI language

The installer itself can have its UI language changed after the fact.

Hi @artl93, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Art Leonard). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

I was still working on it... :-(

As a suggestion, for .NET we have a label called WIP to indicate topics are not ready to be merged yet.

Not sure why your commenting on an closed pull request with unrelated changes.

Each pull request and commit should be always ONE complete change.
When the pull request is closed, the comments left are absolutely ignored because they cannot be acted upon. The request is closed.

From: Terry G. Lee<EMAIL_ADDRESS>Sent: Wednesday, November 30, 2016 2:50 PM
To: Microsoft/visualstudio-docs<EMAIL_ADDRESS>Cc: Art Leonard<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/visualstudio-docs] Add info to change installer UI language (#10)
@TerryGLee commented on this pull request.

In docs/install/install-visual-studio.mdhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fvisualstudio-docs%2Fpull%2F10&data=02|01|artl%40microsoft.com|d066826869c245edaf2908d419733df6|72f988bf86f141af91ab2d7cd011db47|1|0|636161430130011206&sdata=DlhEG8ftwMMajxRfWkzEWNGYeWVQizuzaFeZWfVhUcQ%3D&reserved=0:

@@ -93,6 +93,11 @@ To install Visual Studio 2017 RC in a language of your choosing, click the **Lan

[!IMPORTANT]

While Visual Studio 2017 RC in general is supported for use in a production environment, those workloads and components that are marked "Preview" in the installation UI are not supported for use in a production environment.

+## Changing the installer Language
Another comment: This section should not appear after the important "production environment" note. In fact, that note should be the last text on the page, directly before the See Also links. (We're hoping that people will see this note before they comment on specific things that we already know aren't working yet in this RC, as the note appears (or should appear) close to the "Post Comment" box.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fvisualstudio-docs%2Fpull%2F10&data=02|01|artl%40microsoft.com|d066826869c245edaf2908d419733df6|72f988bf86f141af91ab2d7cd011db47|1|0|636161430130011206&sdata=DlhEG8ftwMMajxRfWkzEWNGYeWVQizuzaFeZWfVhUcQ%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJZjM270Etq6DsQYGRseO3YdcCwSCxnDks5rDf2jgaJpZM4K-bxO&data=02|01|artl%40microsoft.com|d066826869c245edaf2908d419733df6|72f988bf86f141af91ab2d7cd011db47|1|0|636161430130011206&sdata=%2BpzE6orhxaGi0%2B4u367ex7qbS39WFZ%2Bn8c0c6uyS1lk%3D&reserved=0.

I still think this guidance is horribly unprofessional for technical documentation. This guidance must be used in an indeterminate sense: You can.. you should… you may… Otherwise it supposes too much of the reader and otherwise sounds like something I was never allowed to submit in any paper past the seventh grade.
From: Terry G. Lee<EMAIL_ADDRESS>Sent: Wednesday, November 30, 2016 2:46 PM
To: Microsoft/visualstudio-docs<EMAIL_ADDRESS>Cc: Art Leonard<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/visualstudio-docs] Add info to change installer UI language (#10)
@TerryGLee commented on this pull request.

In docs/install/install-visual-studio.mdhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fvisualstudio-docs%2Fpull%2F10&data=02|01|artl%40microsoft.com|1217100469ae44c3283708d41972acdd|72f988bf86f141af91ab2d7cd011db47|1|0|636161427695974759&sdata=wifXnWiW8VSO3LJWTs3Yj8v7Lz8L9qls27FwOnxBDPU%3D&reserved=0:

@@ -93,6 +93,11 @@ To install Visual Studio 2017 RC in a language of your choosing, click the **Lan

[!IMPORTANT]

While Visual Studio 2017 RC in general is supported for use in a production environment, those workloads and components that are marked "Preview" in the installation UI are not supported for use in a production environment.

+## Changing the installer Language

+By default, the installer program tries to match the language of the operating system when run for the first time. The installer will remember this setting. The setting can later be chagned by running the installer from the command line. For example, the user can force the installer to run in Englishby running vs_installer.exe --locale en-US". This setting will then be remembered when the installer is run the next time.
As well, I'd like to word-craft this a bit:
By default, the installer program tries to match the language of the operating system when it is run for the first time. The installer will remember this setting. You can change the setting later by running the installer from the command line. For example, you can force the installer to run in English by running the vs_installer.exe --locale en-US" command. This setting will then be remembered when the installer is run the next time.
It's best to say "you" when you can, as a best practice.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fvisualstudio-docs%2Fpull%2F10&data=02|01|artl%40microsoft.com|1217100469ae44c3283708d41972acdd|72f988bf86f141af91ab2d7cd011db47|1|0|636161427695974759&sdata=wifXnWiW8VSO3LJWTs3Yj8v7Lz8L9qls27FwOnxBDPU%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJZjMz0QpiRXpjmDTXzAf4sIz93h60Jjks5rDfyvgaJpZM4K-bxO&data=02|01|artl%40microsoft.com|1217100469ae44c3283708d41972acdd|72f988bf86f141af91ab2d7cd011db47|1|0|636161427695974759&sdata=MVH88nWstLjCplkPZuW5ac9e1bhVwQSP08tGXHjdcp8%3D&reserved=0.


Ensure Netty is usable on Java7

Motivation:
When adding SNIMatcher support we missed to use static delegating methods and so may try to load classes that not exists in Java7. Which will lead to errors.
Modifications:

Correctly only try to load classes when running on java8+
Ensure Java8+ related tests only run when using java8+

Result:
Fixes [#6700]

@normanmaurer The example app from #6700 exits normally with these changes. 👍

Cherry-picked into 4.1 (aab89b058e2e31fb50f5df98325dc40db61fbced) and 4.0 (0bc8ca5af5a7095cf940addc2df194dc136669ee)


kernels working on a given set of features

NB - tests for this feature are missing

Example with mol features:
import pandas as pd

import bofire.strategies.api as strategies
from bofire.data_models.domain import api as domain_api
from bofire.data_models.features import api as features_api
from bofire.data_models.kernels import api as kernels_api
from bofire.data_models.molfeatures import api as molfeatures_api
from bofire.data_models.priors.api import HVARFNER_LENGTHSCALE_PRIOR
from bofire.data_models.strategies import api as strategies_api
from bofire.data_models.surrogates import api as surrogates_api

domain = domain_api.Domain(
    inputs=domain_api.Inputs(
        features=[
            features_api.ContinuousInput(key="x1", bounds=(-1, 1)),
            features_api.ContinuousInput(key="x2", bounds=(-1, 1)),
            features_api.CategoricalMolecularInput(
                key="mol", categories=["CO", "CCO", "CCCO"]
            ),
        ]
    ),
    outputs=domain_api.Outputs(features=[features_api.ContinuousOutput(key="f")]),
)

strategy = strategies.map(
    strategies_api.SoboStrategy(
        domain=domain,
        surrogate_specs=surrogates_api.BotorchSurrogates(
            surrogates=[
                surrogates_api.SingleTaskGPSurrogate(
                    inputs=domain.inputs,
                    outputs=domain.outputs,
                    input_preprocessing_specs={
                        "mol": molfeatures_api.Fingerprints(),
                    },
                    kernel=kernels_api.AdditiveKernel(
                        kernels=[
                            kernels_api.RBFKernel(
                                ard=True,
                                lengthscale_prior=HVARFNER_LENGTHSCALE_PRIOR(),
                                features=["x1", "x2"],
                            ),
                            kernels_api.TanimotoKernel(
                                features=["mol"],
                            ),
                        ]
                    ),
                )
            ]
        ),
    )
)

strategy.tell(
    experiments=pd.DataFrame(
        [
            {"x1": 0.2, "x2": 0.4, "mol": "CO", "f": 1.0},
            {"x1": 0.4, "x2": 0.2, "mol": "CCO", "f": 2.0},
            {"x1": 0.6, "x2": 0.6, "mol": "CCCO", "f": 3.0},
        ]
    )
)
candidates = strategy.ask(candidate_count=1)
print(candidates)

@e-dorigatti: This looks really great and will make the GP section so much easier, we can then get rid of a lot of GP classes! Thank you very much!

Hi @jduerholt thanks for the kind feedback and pointers :) this sounds like a very important use-case to implement and in the next weeks I will look into it. My first idea would be to go via the input_preprocessing_specs and try to mimic what Fingerprints is doing, probably by recycling some functions from CategoricalInput, and indeed creating a new kernel for the Hamming distance.
Going through an input transform sounds (a lot?) more involved, why do you think that would be preferable to the preprocessing specs? Maybe I should learn more about botorch first.. ;)

Hi @e-dorigatti,
a few more notes/explanations from my side:

There are two types of transforms in principle:

Everything that we have in the input_transform_specs, this is executed before the actual optimization/model fitting. For examples, categorical features are transformed/expanded into Onehots or molecular ones into fingerprints, than the model is fitted upon this representation and also the optimizer on top of the acquisition function is working on top of this represenation. Currently, we can give there either one of CategoricalEncodingEnums or one of the molfeatures classes. This is also a bit of inconsistent here, and we have to tidy it up at some point. The molfeatures classes was added later, and we found there that simple enums were not enough as the transform need to be parameterized.
Then we have the botorch input transforms, this is executed on the fly during gp fitting, optimization. A transform(s) is added to the GP and the data is transformed before it is actually entering the kernel. We use them for their flexibility, for example we could have two models in an optimization and the one could be equipped with a Normalize transform and the other one with an InputStandardize transform. Then one GP would be fitted to the normalized (between 0 and 1) data and the other one to the standardized data (zero mean, unit varriance), and still both of them could be used in combination during optimization. Another example is filtering of features. It could be the case, that we have build two surrogates, but in one of them one feature is not present (perhaps, because we know that it has not predictive power for the output of this model) and now we want to use both surrogates in optimization. In this case, BoFire is automatically combining both Models in botorch's ModelList and adds a filter features transform before the first one that filters those features out that should be not part of the optimization (could be in principle also done with the approach here in the PR). Another example is the PiecewieseLinearGP. For the optimizer, the input to the model are the nodes of the piecewiselinear, but the nodes are on the fly duing optimization expanded (in a differentiable manner) to the discretized piecewiselinear on which then the Wasserstein Kernel is applied. Currently, desite chosing which scaler to use, botorch input transform are not explicitly exposed to BoFires API.

We decided at some point that, categoricals have to be encoded for use in surrogates/optimization always as one-hot, as this allows both for dealing with them as normal features treated as the continuous ones without special kernels and using them in a MixedGP botorch OneHotToNumeric transform. Of course, also other design choices would be possible here, which have drawbacks in itself.
Regarding this PR, an 80% solution which would make different implementations for and MixedSingleTaskGPMixedTanimotoGPSurrogate obsolete would be to have the HammingKernel being able to act directly on the onehot encoded values. This would mean that one would move the OneHotToNumeric transform into the kernel. Furthermore, also the MultiTaskGP could then benefit from this as we would then have the same flexibility there. Of course we could also lift the restriction of having categoricals always as one-hots.
In a next step, one should then make the already eposed scaling transforms (Normalize, Standardize) configurable by giving the user the explicit option which features these transform are acting on.

In general, botorch input transforms are a great way to build up complex differentiable optimization pipelines, on the other hand it is very challenging to come up with a general framework, parameterizied by feature names that can compose different input transforms on different features including the bookkeeping of indices.
I hope his posts helps ;)

Thank you for the thorough explanation @jduerholt, this clears up a lot of doubts! So the key difference is that the optimizer of the acquisition function is working with the features that result after applying the input preprocessing specs, but before the input transform.
What are the reasons that led you to decide that categoricals have to be one-hotted? Currently because I don't know your motivations it looks a bit silly since you use the input transform to do the one-hot, then the OneHotToNumeric to undo the one-hot, as the Hamming kernel does not need one-hots in the first place. If we now add more code to deal with the fact that this specific transform (and only this one?) changes the input dimensionality, it would seem like over-complicating things even further (but it would not be that complicated in the end). Maybe, given that with this PR you can have kernels focusing on specific (subsets of) features, it is now unnecessary to force categoricals to be one-hot? Moreover, because molecules are in the end a special type of categorical, it would make sense imho to convert the molecular featurizers to transforms rather than preprocessing.
I am now wondering whether it is necessary to expose botorch's input transforms, or it would be okay to put these transformations within the kernels themselves, so that users would specify the features that the kernel is acting on as well as how they should be transformed. I think this would be simpler to implement, and not less powerful than botorch's input transforms unless I am missing something. It also makes sense because, in some way, kernels are the only reason why we need input transforms in the first place, and different kernels could work on different representations of the same feature.
What do you think? In the end, I think that both options can be implemented relatively easily, but perhaps one is more disruptive than the other. I think you have a better perspective and ability to judge :)

Will provide you a thorough answer tomorrow ;)

What are the reasons that led you to decide that categoricals have to be one-hotted? Currently because I don't know your motivations it looks a bit silly since you use the input transform to do the one-hot, then the OneHotToNumeric to undo the one-hot, as the Hamming kernel does not need one-hots in the first place.

The reason was that I wanted to have things too flexible:

I wanted to be able to combine a SingleTaskGPSurrogate which needs to onehot the categoricals with a MixedSingleTaskGPSurrogate in the same optimization. For this, there is the need that for both of them the same features enter the GP. So I decided for this complicated solution using the OneHotToNumeric transform, as I saw that it was already available in botorch. And then I just said that CategoricalnputFeatures always has to be one hot.
But this kind of constraint is already relaxed for the following features which all inherit from CategoricalInputFeature:

CategoricalDescriptorInput: Here also CategoricalEncodingEnum.DESCRIPTOR is allowed.
CategoricalMolecularInput: Here also the vairous mofeature transforms are allowed.
TaskInput: If you want to use it within a MultiTaskGPSurrogate, it has to be ordinal encoded.

Moreover, because molecules are in the end a special type of categorical, it would make sense imho to convert the molecular featurizers to transforms rather than preprocessing.

You are right, they are special forms of the categorical features. But I would not convert them to botorch input transforms, for the following reasons:

Botorch input transforms are super nice if you have to change something between the input representation for the optimizer and the representation of the GP. Especially, if you want it in differentiable manner, as we do it in the PiecewiseLinearGPSurrogate. This is not really the case for the molecular featureizers. The featurization is non differentiable and we can do it easily upfront.
The current setup for the transforms is working and I do not see a need to change it.

I am now wondering whether it is necessary to expose botorch's input transforms, or it would be okay to put these transformations within the kernels themselves, so that users would specify the features that the kernel is acting on as well as how they should be transformed. I think this would be simpler to implement, and not less powerful than botorch's input transforms unless I am missing something. It also makes sense because, in some way, kernels are the only reason why we need input transforms in the first place, and different kernels could work on different representations of the same feature.

Two reasons for keeping it:

Practical reason; The transforms are deeply baked into botorch and also well tested. If we want to do it in the kernels, we have to implement it ourselves.
Computational reason: the botorch/gpytorch gps store the training data x*which is used then used at inference time to calculate the covariance matrix/kernel with respect to the test data x: k(x,x*). The trick is that it stores the transformed training data, so you do not need to transform the training data at every inference step. All these mechanism are already implemented and well tested especially on the botorch side. For this reason, in general I want to keep the trasforms, but of course there could be exceptions for edge cases for it.

What I would do now:

Release the restriction that CategoricalInputss has to be always one hot encoded and allow also for ordinal encodings.
Fix get_categorical_combinations in BotorchStrategy so that it also works for categoricals encoded as ordinals.
Implement in get_categorical_combinations also the generation of the molecular ones. Currently you can only run optimizations over molecular spaces if you do not have only continuus ones.
Drop the options in BoTorchStrategy for the categorical_method, descriptor_method und discrete_method. Depending on the surrogate, we had here so far the possibility to use continuous relaxations of them during the optimization, which ofc can lead to wrong optima. So I would drop this support (makes the code much easier), especially as we get the GA by @LukasHebing and @sleweke-bayer for mixed spaces.
Change the ScalerEnums into actual classes in which we can define on which features/feature types the corresponding transforms should apply.
Try to write the MixedSingleTaskGPSurrogate in plain bofire as a SingleTaskGPSurrogate which the explicit kernels and see that we get the same results.
Remove all necessary GP implementations and write helper methods that helps you to generate the complex specs.

This should overall, simplify our codebase a lot and will give in general much more flexibility. Of course, there will be also edge cases. What do you think?
Ofc, this should not all be done in this PR by you. So I would suggest, that we try to get this PR (which does not breaks anything) in as fast as possible, agree on the changes from above and then distribute the taks. Fine for you?

Thank you for the clarifications @jduerholt. Following your suggestions I have implemented the Hamming kernel working on one-hot features, which is instantiated by the kernel mapper transparently for the user, and verified that one can get the same results as MixedSingleTaskGP (or even better with a different kernel)

Apologies, I got lost debugging those tests that were failing on GH but not locally, then I noticed other PRs are suffering from the same issue. According to the print statements I added it seems like an issue with differentiation in the formulaic package?
>       assert np.allclose(B, J)
E       assert False
E        +  where False = <function allclose at 0x7f392a5de930>(array([[0., 1., 0., 0., 0., 2.],\n       [0., 0., 1., 0., 0., 1.],\n       [0., 0., 0., 1., 6., 0.]]), array([[[0, 1, 0, 0, 0, 2],\n        [0, 0, 1, 0, 0, 1],\n        [0, 0, 0, 1, 0, 6]]]))
E        +    where <function allclose at 0x7f392a5de930> = np.allclose

tests/bofire/strategies/doe/test_objective.py:52: AssertionError
----------------------------- Captured stdout call -----------------------------
model 1 + x1 + x2 + x3 + x3 ** 2 + x1:x2
diff x1 0 + 1 + 0 + 0 + 0 + x2
diff x2 0 + 0 + 1 + 0 + 0 + x1
diff x3 0 + 0 + 0 + 1 + 0 + (2*x3)

Where diff x3 should be 0 + 0 + 0 + 1 + (2*x3) + 0 (full output here)


IE9/IE8 Form doesn't show until the "F12 developper tools" is opened

In IE9/IE8, in my app, the Form doesn't show or isn't render (I am not sure) until the "F12 developper tools" is opened and the page refreshed. After the first time the developer tool is shown, not matter if the developper tools is still open or not, the form will successfully be displayed on the page if I refresh the page.
Have you already seen that problem?

Never heard of it. Have you seen the docs on supporting IE 8?
http://docs.angular-formly.com/v6.13.0/docs/support-ie-8

Kent C. Dodds

(Sent from my mobile device, please forgive typos or brevity)
On Jun 11, 2015 7:04 PM, "samuelcoutu"<EMAIL_ADDRESS>wrote:

In IE9/IE8, in my app, the Form doesn't show or isn't render (I am not
sure) until the "F12 developper tools" is opened and the page refreshed.
After the first time the developer tool is shown, not matter if the
developper tools is still open or not, the form will successfully be
displayed on the page if I refresh the page.
Have you already seen that problem?
—
Reply to this email directly or view it on GitHub
https://github.com/formly-js/angular-formly/issues/349.

<canned-message>
Thanks for your interest in angular-formly. The issues are for bug reports/feature requests, so I'm going to go ahead and close this issue. If you have a question or need help, please watch this screencast for an explination of the most effective/proper way to get help using angular-formly.
Summarized:

Learn angular-formly basics
Check the examples
Reproduce the issue
Ask on Stack Overflow
Ask on the mailing list
Ask on Chat

P.S. sorry for the canned message, this just happens a lot :smile:
</canned message>

@samuelcoutu every time I see this, it has to do with using an API that is only available in the developer mode of IE8. And the most likely culprit is console.log. Using stuff like console.log will crash your page while in normal mode, but it will work fine in developer mode.

Ah, good point @koenweyn. angular-formly does use console.warn in several places when you're using its API incorrectly as well. I recommend disabling api-check as well as warnings when you ship to production (leave it on in development though). I will add this recommendation to the docs for supporting IE 8.

Thanks a lot for helping. This was causing my issue.

👍

If $log.warn was used instead of console.warn would that also resolve this issue?  I am speaking exclusively about angular-formly rather than api-check.

Yes, angular-formly uses $log

Kent C. Dodds

(Sent from my mobile device, please forgive typos or brevity)
On Jun 12, 2015 7:49 AM, "Caleb Kniffen"<EMAIL_ADDRESS>wrote:

If $log.warn was used instead of console.warn would that also resolve
this issue? I am speaking exclusively about angular-formly rather than
api-check.
—
Reply to this email directly or view it on GitHub
https://github.com/formly-js/angular-formly/issues/349#issuecomment-111497931
.


Segfault in libc's getdelim() due to NPE when using ag's -z option for compressed files

ag -z crashes with a segfault due to passing a null file pointer to iogetdelim() (glibc)
# stack trace:
                Stack trace of thread 18478:
                #0  0x00007ffff6fbe88c __getdelim (libc.so.6)
                #1  0x0000000000407ddf search_stream (ag)
                #2  0x000000000040823a search_file (ag)
                #3  0x00000000004082ea search_file_worker (ag)
                #4  0x00007ffff7324ea5 start_thread (libpthread.so.0)
                #5  0x00007ffff704d96d __clone (libc.so.6)

# Debugger info
(gdb) bt full
#0  _IO_getdelim (lineptr=0x7ffff3939da8, n=0x7ffff3939db0, delimiter=10, fp=0x0) at iogetdelim.c:53
        _IO_acquire_lock_file = <optimized out>
        result = <optimized out>
        cur_len = 0

# iogetdelim.c
/* Read up to (and including) a TERMINATOR from FP into *LINEPTR
   (and null-terminate it).  *LINEPTR is a pointer returned from malloc (or
   NULL), pointing to *N characters of space.  It is realloc'ed as
   necessary.  Returns the number of characters read (not including the
   null terminator), or -1 on error or EOF.  */
ssize_t
_IO_getdelim (char **lineptr, size_t *n, int delimiter, FILE *fp)
{
  ssize_t result;
  ssize_t cur_len = 0;
  ssize_t len;
  if (lineptr == NULL || n == NULL)
    {
      __set_errno (EINVAL);
      return -1;
    }
  CHECK_FILE (fp, -1);
  _IO_acquire_lock (fp);
  if (_IO_ferror_unlocked (fp))
    {
      result = -1;
      goto unlock_return;
    }
  if (*lineptr == NULL || *n == 0)
    {
      *n = 120;
      *lineptr = (char *) malloc (*n);
      if (*lineptr == NULL)
        {
          result = -1;
          goto unlock_return;
        }
    }

# Macro definition
# ifdef __EXCEPTIONS
#  define _IO_acquire_lock(_fp) \
  do {                                                                              \
    FILE *_IO_acquire_lock_file                                                      \
        __attribute__((cleanup (_IO_acquire_lock_fct)))                              \
        = (_fp);                                                              \
    _IO_flockfile (_IO_acquire_lock_file);
# else
#  define _IO_acquire_lock(_fp) _IO_acquire_lock_needs_exceptions_enabled
# endif
# define _IO_release_lock(_fp) ; } while (0)
#endif

Try
https://github.com/ggreer/the_silver_searcher/issues/1349
https://github.com/ggreer/the_silver_searcher/issues/1349#issuecomment-567665686
https://github.com/ggreer/the_silver_searcher/issues/1349#issuecomment-567705544


ci: add openmp test

Description

Add test script including OpenMP flag (--//:has_openmp)
Add test_output flag for every test script

Could you split the commits into 1. leave test outputs on error. 2. add openmp config.


Concurrent pinging gives spurious results

Hello,
I am new to Rust and I am trying to ping multiple IP addresses at the same time using threads. When I use the ping crate sequentially, the result is as expected: IP addresses which are online yield Ok, IP addresses which are not online yield Err. However, when I try to ping multiple addresses concurrently, the behavior is "random", i.e., the return value from ping::ping does not reflect whether the address is available or not.
Minimal Example
Here is a minimal example:
use std::net::IpAddr;
use std::str::FromStr;
use std::thread;
use std::time::Duration;

use ping;

fn is_online(ip: IpAddr) -> bool {
    let timeout = Duration::from_millis(50);
    match ping::ping(ip, Some(timeout), None, None, None, None) {
        Ok(_) => { println!("{}: online", ip); true },
        Err(_) => { println!("{}: offline", ip); false },
    }
}

fn main() {
    let ips = [
        "<IP_ADDRESS>", // is available in my network
        "<IP_ADDRESS>", // is not available in my network
        "<IP_ADDRESS>", // is not available in my network
        "<IP_ADDRESS>", // is not available in my network
    ];

    let mut handles = vec![];
    for ip in ips {
        let addr = IpAddr::from_str(ip).unwrap();
        let handle = thread::spawn(move || {
            is_online(addr);
        });
        handles.push(handle);
    }

    for handle in handles {
        handle.join().unwrap();
    }
}

In my network, only <IP_ADDRESS> is available, but I see this output
<IP_ADDRESS>: online
<IP_ADDRESS>: online
<IP_ADDRESS>: online
<IP_ADDRESS>: online

or also this output:
<IP_ADDRESS>: online
<IP_ADDRESS>: offline
<IP_ADDRESS>: offline
<IP_ADDRESS>: offline

Additional Remarks
I have seen the exact same problem when I was using ping with tokio and green threads.
As said, I am new to Rust and maybe the code has some inherent logic error which I don't see.  :-)
Version Information
I am using ping version 0.4.0 with rustc 1.67.1 (d5a82bbd2 2023-02-07).

This is because current implementation of this crate is not concurrency safe. Current implementation do not check the ident which is received to the specified one, so calling ping multiple times in a short duration, the "offline" may receive the "online" one's reply.
We should and the ident check, and wait until we got the right reply, or the timeout occurred.
I'll try to fix this if I have time, and PR is welcomed.

Thanks for the answer, that makes it clear!
I'll have a look tomorrow evening. I'll keep you updated.

@aisk: I've creted a PR, see https://github.com/aisk/ping/pull/7

Hi :) Is this issue is resolve?

would be awesome if the PR could be picked-up again :)


Updateed org-homepage.css

Added comments before the styling started and rearranged the code without changing anything :-)

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