Bump pywebpush to latest 1.9.2

Description:
bump pywebpush to latest 1.9.2 for html5 push notifications
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

pywebpush needs cryptography 2.6.1, HA 0.91 needs cryptography 2.5.
homeassistant 0.91.0 has requirement cryptography==2.5, but you'll have cryptography 2.6.1 which is incompatible.

@perosb Can you please submit a new PR to fix the issue @bieniu has brought up

Hmm...weird that mine didn't complain.
@robbiet480
crypto and pyjqt are added under "Home Assistant Core". Maybe it's better to revert my PR?

Hey, this has broken web push notifications, see the issue.


Closed range of hours

Can i set a closed range of hours?
Example: i wanna the user to only be able to select hours from 11 to 18
Something like the minuteDivider option but for the hours
/// minuteDivider: [int] minute restriction, e.g. 5: every 5th minute will be shown (0, 5, 10, 15 ...)

@dylanwuzh

👀

closing as this wont be fixed


[Feature Request] CountryCode api-scraper [withdrawn codes][conflicting]

What TMM version are you using?
2.9.7
release, pre-release, nightly, or directly from GitHub/branch?
release
What is the actual behaviour?
CountryCode for for example germany only available for germany since 1990.
What is the expected behaviour?
In the following I will refer to the example of germany. The problem exists for other countries too.
For old movies, there should be germany available before 1990 and since 1990.
The following country codes should be available:

Code
Country Name
Alpha-2

276
Germany
DE

278
East Germany
DD

280
West Germany
DE

With the current enum CountryCode structure, it is not possible to add all of these country codes because they are conflicting in alpha-2 name which is the enum constant.
For example:
The scraper-imdb will never be able to scrape the correct release date for movies released before 1990. Best example here is Star Wars - Eposide IV. The release date in "west germany" was 1978-02-09.
To scrape the release date you have to request the realeaseinfo path of imdb for the movie. It contains the correct release date for "west germany" and later release dates for "germany".
Possible solutions:

Solution
Pros
Cons

scraper-imdb uses it's own individual solution
possible to scrape correct release dates
would not help other scrapers

CountryCode contains a list of names
Alternative names.
Code and alpha-3 would be missing. Most CountryCodes would have a list with one item.

CountryCode uses name as enum constant
name is unique
getByAlpha2 would always return a list, even though most countries only have one name. Enum constants would be long.

e.g. west germany with enum constant e.g. XWG
easy to implement
dirty solution, but imdb uses this constant as region name.

Steps to reproduce:
Doesn't matter.
Additional
Have you attached the Logfile from the day it happened?
Your Operating system? (win/mac/linux? version?)
Doesn't matter.

we've migrated v3 to GitLab (https://gitlab.com/tinyMediaManager/tinyMediaManager) along with all issues. The issues for v2 at GitHub will be closed - we will respond to the corresponding issue in GitLab in the future
br
Manuel


Unable to defocus certain fields in Safari

URL
No response
How to reproduce the issue?
When a feature is selected in 2.27.0 in Safari, I'm unable to move the keyboard focus away from fields like Operator in the sidebar. This makes it impossible to edit tags or use hotkeys. Firefox still works fine for me.
Screenshot(s) or anything else?
No response
Which deployed environments do you see the issue in?
No response
What version numbers does this issue effect?
2.27.0
Which browsers are you seeing this problem on?
Safari

I also have this issue with iD 2.27.0 on Safari

Has this been fixed in 2.27.1 ?

Yes, this is probably #9848.


PubmedReader

Hello! I started using the PubmedReader, but I'm experiencing some problems. It seems that the papers, which are returned have nothing to do with my search query. I type for example 'temporomandibular disorders' as the query and get results which have nothing to do with the particular disease. Anyone experienced something similar? Does the search query need some further specifications?

It runs here, but the returned results mentioned only "temporomandibular", not fully "temporomandibular disorders"
You can explore Pubmed parameters and open a PR if find a better way to achieve that, it will be very welcome!
parameters available on pubmed: https://www.ncbi.nlm.nih.gov/books/NBK25499/


Grouping Components by Category

As the component list grows, should we look at grouping them into folders by category eg: all form components into a forms  folder, charts into a charts folder etc.
I propose the following set of folders mirroring the component library

atomic
navigation
forms
loaders
charts

created a folder for form components in PR - #27 .


[iOS] Export kotlinx-datetime to the iOS Framework

Overview (Required)
To keep all conversion between kotlinx-datetime and Swift types consistently, export kotlinx-datetime's  Converters.kt to iOS.
Links

https://github.com/Kotlin/kotlinx-datetime

Can't toKotlinInstant function be used ?
We need that function too 😭

https://github.com/Kotlin/kotlinx-datetime/blob/master/core/darwin/src/Converters.kt#L31

Yes, we can, but have to use it this way.
ConvertersKt.toKotlinInstant(Date())


Need better platform code separation

I was interested in adding iOS, macOS and Windows support, but the layout of the code is not conducive to this.  src/cpuinfo_aarch64.c is highly Linux-specific.  For an ARM64 Windows implementation, where would the code go?  (In Windows, the ARM64 implementation would be based around IsProcessorFeaturePresent.)

Thank you very much for this feedback, I very much appreciate it.
I agree the code layout must be more principled to allow the matrix of OS/ARCH to be fully exposed.
How about cpuinfo_{arch}_{OS}.c? Any suggestion?

I'd personally do separate directory approach.
I'd vote on {OS}/{arch}.c.

I vote on the suggestion from gchatelet.

Using separate directories would allow for each OS to share code using additional files, still isolated from the rest.

@Androbin 's rationale makes sense. I'll play a bit with the layout and come back with a proposal.


Block width is not computed on Firefox nor IE.

Of course, it didn't work on Internet Explorer

Fixed by d3renderer


Made python kfserving.storage return uri if local path exists

The current kfserving.storage implementation only takes absolute local paths. It should also apply to any relative/absolute local path that exists on the local system.

This change is

/ok-to-test

Thanks Tommy! Sorry to thrash your implementation but it looks like I miscommunicated a bit.
I think the file:// convention is fine, but it's less and less common these days. Modern browsers support the convention, but I'm struggling to find other examples of libraries like this using it.
I'm happy to support it, but perhaps not as the recommended usage pattern as it results in somewhat awkward syntax.

Hi @ellis-bigelow, thanks for the feedback. So I added back os.path.exists and some test cases in my recent commit to also support local path without any prefix. Please let me know if you have any question.
Thanks.

Awesome thanks.
/lgtm
/approve


BitFenix API returning too many redirects

This unfortunately didn't solve the problem, but it at least seems local to me, due to a caching error.


Video transformation options as an Object instead of method chain

Hi,
I currently in process of migrating from cloudinary-core to @cloudinary/url-gen, and one of the biggest challenge I am facing is  around video transformations.
In my current implementation, I store all the transformations as an array of objects in database and use cloudinary.url method to inject the json for transformation.
const transformation = [
  {
    overlay: { text: "some text", fontSize: 40},
    gravity: "south",
    x: 100,
    y: 600,
    startOffset: "15",
    endOffset: "20",
    color: "#ffffff"
  }
]

const url = cloudinary.url("videoPublicId", {
  resource_type: "video",
  transformation,
  sourceTypes: ["mp4"]
})

Is there a way to do something similar using @cloudinary/url-gen? Or, do I have to resort to the method chain?
Any help is much appreciated!
Thanks!
Prabs

Hi Prabs,
Thanks for reaching out. So you should be able to use createCloudinaryLegacyURL to migrate easily. Please have a look at https://cloudinary.com/documentation/javascript1_migration_guide#migrating_delivery_urls and let me know ho wyou get on.
Thanks
Tom

Thanks Tom. It is definitely the right direction, but does not really solve my issue as the overlay still requires the function chain.
Ty

Hey @prabhatmyadbox
I just wrote this test which creates an overlay with a JSON

  it ('Accepts a JSON with an overlay', () => {
    const transformation = [
      {
        overlay: { text: "some text", font_size: 40, font_family: 'arial'},
        gravity: "south",
        x: 100,
        y: 600,
        startOffset: "15",
        endOffset: "20",
        color: "#ffffff"
      }
    ];

    const url = createCloudinaryLegacyURL("videoPublicId", {
      cloud_name: 'demo',
      resource_type: "video",
      transformation,
      sourceTypes: ["mp4"]
    });

    console.log(url);
  });

this is the output
http://res.cloudinary.com/demo/video/upload/co_rgb:ffffff,g_south,l_text:arial_40:some%20text,x_100,y_600/videoPublicId

Thanks Patrick for the the example.
I was finally able to get the desired result. And, I've realised where the error had occurred.
All my transformation keys had been converted to snakeCase (font_size, start_offset). But, for this function to work, the keys of transformation objects required to be camelCase while only the keys of overlay object needed to be snakeCase.
However, the LegacyITransforamtionOptions types list all the top level transformation key in snakeCase.
It appears that the type definition may not match how the method is behaving.
Thanks again!
Cheers!


Can't send or receive fragmented UDP packets

Environment
Windows build number:                         10.0.19041.0
Your Distribution version:                    Ubuntu 20.04
Whether the issue is on WSL 2 and/or WSL 1:   WSL 2

Steps to reproduce

I only ended up testing this issue with UDP and protocols based on it over IPv4.

Listen for incoming UDP packets on port 8080 on some external host using nc -ul 8080.
Open a socket to the external host inside WSL: nc -u <remote address> 8080
Send a long (> 1471 characters excluding the \n) line / packet from WSL (doesn't arrive at external)
Send a short line such as test from WSL to punch through the NAT for the next step (does arrive at external)
Send a long (as above) reply from the external host (doesn't arrive at WSL)

External host console:
[user@hostname ~]$ nc -ul 8080
test
testaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
^C

WSL console:
user@ubuntu:~$ nc -u <external host address redacted> 8080
testaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
test
^C

If instead executed directly (outside of WSL) using a version of nc compiled for Windows, all packets arrive as expected.

WSL logs: Sadly I am not able to include logs due to Feedback being disabled.
Expected behavior

All packets are received at their respective destinations.
Actual behavior

Long packets are dropped somewhere between the physical NIC and WSL. This appears to happen whenever the IP datagram would exceed 1500 bytes in size. This issue seems related to #4517, however the proposed solution of lowering the MTU on WSL doesn't work for UDP since it disregards the MTU entirely and I am not connected to a VPN or similar. Running sudo tcpdump under WSL will produce output similar to the following upon sending these packets:
05:03:37.622552 IP <IP_ADDRESS>.50284 > <external host redacted>.8080: UDP, bad length 1476 > 1472

Running Wireshark or tcpdump on the Windows host while receiving such long packets will show the fragmented package arriving, and Wireshark succeeds in displaying the reassembled UDP packet. Running sudo tcpdump in WSL produces no output related to the large packet.

Great analysis, thank-you. Was able to reproduce here. FWIW, Hyper-V with a bridged network is okay. Your step 3 can be quantified with something like the following, for (eg) 5000 'a' characters plus a newline:
$ printf '%*s\n' 5000 | tr ' ' a | nc -u <IP_ADDRESS> 8080

This issue seems related to #4517

It is almost certainly the cause of at least some percentage of people's ssh woes as well.

Have experienced the same problem over a long period sending UDP packages from a container in Docker for Windows. This was a problem while running Docker on Hyper-V and still is a problem running Docker in WSL2. Between the Docker containers packages are sent and received correctly. Trying to receive packages from the Docker container at the Windows host, it seems like fragmented UDP packages exceeding MTU are dropped.

Have experienced the same problem over a long period sending UDP packages from a container in Docker for Windows. This was a problem while running Docker on Hyper-V and still is a problem running Docker in WSL2. Between the Docker containers packages are sent and received correctly. Trying to receive packages from the Docker container at the Windows host, it seems like fragmented UDP packages exceeding MTU are dropped.

Experiencing the same problem here.
Any information on how/when this will be fixed?
I have a service which send stats by UDP and so it's unusable from inside the WSL...

Experiencing the same problem here.
Any information on how/when this will be fixed?
I have a service which send stats by UDP and so it's unusable from inside the WSL...

Until this is fixed, I use socat to do UDP tunelling over TCP.
Install it on both server and client (WSL)
sudo apt install socat

In the following example I forward the 8094 UDP port to the remote server:
Log with SSH on the remote server and launch the following command:
socat TCP4-LISTEN:8094,fork UDP4:localhost:8094

Finally open a terminal on the local client machine (WSL) and execute the following command:
sudo socat UDP4-LISTEN:8094,fork TCP4:<IP_ADDRESS>:8094

Now on the local client machine (WSL) if you push with UDP on localhost::8094 then all is redirected on the remote server via TCP

Until this is fixed, I use socat to do UDP tunelling over TCP.
Install it on both server and client (WSL)
sudo apt install socat

In the following example I forward the 8094 UDP port to the remote server:
Log with SSH on the remote server and launch the following command:
socat TCP4-LISTEN:8094,fork UDP4:localhost:8094

Finally open a terminal on the local client machine (WSL) and execute the following command:
sudo socat UDP4-LISTEN:8094,fork TCP4:<IP_ADDRESS>:8094

Now on the local client machine (WSL) if you push with UDP on localhost::8094 then all is redirected on the remote server via TCP

Hi, I think it also affects TCP traffic through a VPN interface where is lower MTU.

I workaroud it by ip route change $DEFAULT_ROUTE advmss $ADVMSS. But it is annoying.

This still does not work as of Kernel version: 5.10.16.

Any ETA?

This is a major bummer. Took a long time to figure out what was happening. Running snmpbulkwalk from within a WSL instance...
# Works
snmpbulkwalk -c public -v 2c -<IP_ADDRESS>

# Fails - results in fragmented UDP packets that can be seen on wireshark on host but are not routed back to WSL instance
snmpbulkwalk -c public -v 2c -C r1024 <IP_ADDRESS>

Any update on this?

Have the same problem to send WakeOnLan (udp) packets from wsl... Will try the TCP routing workaround!

As mentioned by jstarks here, there's a workaround that might help you

Same problem, waiting for solution.

Just a note that the issue seems to that all fragmented packets from the wsl2 host are dropped; I first experienced it with icmp, confirmed with udp and also tcp.  An easy to reproduce case is to set the mtu in the WSL host lower, say to 1000; then send packets (icmp, udp or tcp) of size 900 and you will see them go through at various points in the network; then send the same packet of size 1100, so the WSL kernel fragments the packet, and it won't make it past the wsl interface to any other interface on the host nor to the network.
As everyone, would sure love a fix.  (I'll probably just switch to a vm, it makes WSL networking pretty flaky.)

It seems like this is still an active issue sending fragmented UDP packets from WSL2 in 2024. Could someone please re-open this issue. Wondering if there are any updates or plans to fix this?


build: Update to header 1.3.232

Update known-good
Generate source

CI Vulkan-Tools build queued with queue ID 21908.

CI Vulkan-Tools build # 817 running.

CI Vulkan-Tools build # 817 passed.


Base lambda value in the log file

The base value of lambda according to the prototxt file it is 1000 in the log file that you have attached on the google drive here: https://drive.google.com/drive/folders/1kpGGvb5Nv0EmDicW2ue8LdUVjI0Ht1PP. However, in the same training log the lambda was set as 5 from the first iteration.
I was wondering if we set the base as 1000, should not it be starting from 1000 and then decrease it to the minimum lambda, which is 5 in this case. Could you please look into your log file: https://drive.google.com/drive/folders/1kpGGvb5Nv0EmDicW2ue8LdUVjI0Ht1PP?
Thanks.

I realized what was happening there. I think the lambda value is decided by the model based on the weights that we supply. If the pre-trained model is good enough then eventhough we provide the pretty high base lambda, it is going to take it as lambda_min.


MONIT-40224: ebpfSampler

This is for a New Sampling Strategy for eBPF traffic. The design doc is here https://confluence.eng.vmware.com/pages/viewpage.action?spaceKey=WFENG&title=Design+Doc+for+a+New+Sampling+Strategy+for+eBPF+traffic
There are two more tickets related to this
MONIT-40225 | Configuration and parameter tuning of ebpfSampler
MONIT-40444 | ebpf script change for eBPF sampler

Not sure why Java CI build failed. On my local, it has been successful.

Note: @zen10001  to change base branch to dev


Create robot that automatically sends the contributor an invitation to join the GitHub org when their PR is merged

Do you want to request a feature or report a bug?
feature
What is the motivation / use case for changing the behavior?

Hey @ghuntley, this seems like a really cool project. I'm interested in taking this one up! So I create a new repository and send a link when it's ready?

@erickzhao neat. I'd love have this in existence. Right now I'm doing this manually ugh.
Goals

When someone submits a pull request

If user is already part of the organization AND has a pending invitation to join the organization-> do nothing
If user is not part of the organization AND does not have a pending invitation to join the organization -> send them an invite

How
I'd recommend starting by forking https://github.com/behaviorbot/welcome and coming up for a name for your new robot (badge?) then moving on to studying how the GitHub API works.
The bot would be a maintainer in the ReactiveUI Community team should grant them privileges to invite/add people to the team.

Once you have done some study then I'd suggest creating your own GitHub organization (it's free) and get it working over there. Keep us updated as you progress and if you have any questions then just ask :)

Thanks a lot for the tips! I'll get started as soon as possible. :)

Hey @ghuntley, small update on my progress so far! I managed a minimal viable product up, where I run the bot locally with npm start and it does its stuff.
You can check it out here: https://github.com/erickzhao/badge

(The screenshot shows a simulated webhook, but
What it currently does:

Receives webhook from GitHub
Checks for pull_request.closed action.
Does nothing if action user is a bot, if pull request was closed without merging, or if repository is not part of an organization.
If all checks are passed, use GitHub API to send organization request to user.

Todo:

[ ] Write tests
[ ] Clean up code
[ ] Write usage docs
[ ] Deploy

Possible improvements:

[ ] Add CI
[ ] Allow users to customize the organization that they're invited to
[ ] If user is invited, give users the option to enable the bot to comment on the PR that was merged.
[ ] Add CONTRIBUTING.md

I'll be improving it (and figuring out how to deploy to the cloud) pretty soon. I'm pretty swamped with schoolwork until Friday, though, so look for another update probably this weekend!

I've added it to the ReactiveUI organization so you now have your very first user. Before progressing too much further I'd suggest implementing analytics/telemetry/log capture via Sentry.io (information on this is in the Probot documentation). Then move onto creating some tests, then finally setting up CI with Heroku. Once you have all that setup then move onto progress/other features such as:
Provide configuration option via yaml that allows a consumer to specify whether addOrgMembership or addTeamMembership is used. Currently L24 always uses addOrgMembership:
      response = await context.github.orgs.addOrgMembership(memberPayload);

Would like to have see configuration option that allows maintainers to specify that the contributor is invited to a team and have configuration option to specify the identifier of the team that the contributor is invited to.

@ghuntley I just enabled 2FA and accepted the organization invitation. 😄
I'll work on the things you mentioned first! I'll update the checklist as I go and leave another comment when I'm done. :)
By the way, do you know of a way we can make this count as a Hacktoberfest project, since I won't be submitting any PRs directly to any repositories?

It works!

By the way, do you know of a way we can make this count as a Hacktoberfest project, since I won't be submitting any PRs directly to any repositories?

Send in a pull-request to the reactiveui/website repository tagging this issue using this language in the commit message fixes https://github.com/reactiveui/ReactiveUI/issues/1501 explaining that people who contribute get an invitation to join the ReactiveUI org on this page https://reactiveui.net/docs/thanks/

Hey @ghuntley! Four midterms, one assignment, and one interview later, I've gotten a decent amount of progress on badge so far. I've been updating my checklist as I've been going, but the summary is that I got Sentry and Travis CI working, wrote a bunch of unit tests (should cover most cases), and tested it in badge-test-repo.
My only issue right now is getting context.github.paginate to mock correctly in unit tests, so I haven't enabled it. Right now, badge is only getting the first page of teams for an org and checking the team name in the config against those, so organizations with over 30 teams might experience some difficulties.
Everything else should be working smoothly, though! Let me know if you have any other concerns about functionality or code quality.
Also, I'll make the PR to ReactiveUI/website right now. :)
Cheers!

Small update: showed the app to the Probot maintainers and decided to change badge to invite-contributors as it is clearer (albeit less sexy) and avoids confusion with repository badges/shields. :)
https://github.com/probot/probot.github.io/pull/78

invite-contributors is a great name. That's a really good change. Don't forget to rename the badge.yml. I've had to temporarily disable the robot as it looks like something occurred that shouldn't of happened. The robot demoted my access. Ooops. Can you please investigate & resolve 👍

Don't forget to rename the badge.yml

Good catch! I've scrubbed the name badge away from package.json and test/index.js as well.

The robot demoted my access on the organization.

I noticed that this bug occurred 7 days ago. Could you potentially check again? I think that this case was handled in a later commit. From then on, the bot only invites if getOrgMembership() throws an error.
I ran a simulation where I invited myself into badge-test-repo, and it ran into the return clause I was already in the organization.

The terminates if user was already in organization test should already handle this case as well. Let me know if this still occurs on your end, and I'll try to take a deeper look.

Hey @ghuntley have you tried using the app lately? I've also gotten it to work for outside collaborators! :)

Yup - we are using it in production. Thank-you @erickzhao


chunks=True

h5py has support for automatically picking an appropriate size of chunks based on the type and shape if chunks is set to True. Would be nice if similar behavior could be provided in Zarr.

It's already there :-)
I copied the h5py implementation, except I tweaked the algorithm to make
bigger chunk sizes as in my experience h5py chunks are too small.
On Tuesday, January 24, 2017, jakirkham<EMAIL_ADDRESS>wrote:

h5py has support for automatically picking an appropriate size of chunks
based on the type and shape if chunks is set to True. Would be nice if
similar behavior could be provided in Zarr.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alimanfoo/zarr/issues/107, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAq8Ql9-TnJnPn47p9kQEVvyzCeI_bTIks5rVkKFgaJpZM4LsnYs
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721

Oops 😳. Missed that.
Should we change this to a doc issue then?

Yes good idea, I hadn't got around to documenting.
On Tuesday, January 24, 2017, jakirkham<EMAIL_ADDRESS>wrote:

Oops 😳. Missed that.
Should we change this to a doc issue then?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/alimanfoo/zarr/issues/107#issuecomment-274899962, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAq8Qky8o-3VCR7-p3mMP-Inf3kQ03ksks5rVkkhgaJpZM4LsnYs
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721


Improve security guide

Improving Security Guide per suggestion in #100 .

good point @meryllblanchet, updated.


creates convertDate helper function and adds field_mobile_app_date to…

… api response for a campaign.
Fixes #4723

@chloealee looks like this branch needs a rebase against dev!

@angaither thanks for that and for sitting with me to rebase!

thanks, @weerd ! Updated all the changes we just reviewed together :)

One small comment, other than that looks great! :+1:


snarkos: 2.0.2 -> 2.1.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/AleoHQ/snarkOS/releases.
meta.description for snarkos is: A Decentralized Operating System for Zero-Knowledge Applications
meta.homepage for snarkos is: https://snarkos.org
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.1.1 with grep in /nix/store/lfgrqinw81ksvm3rr4lwpp1rpa9jyf9w-snarkos-2.1.1
found 2.1.1 in filename of file in /nix/store/lfgrqinw81ksvm3rr4lwpp1rpa9jyf9w-snarkos-2.1.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
snarkos

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lfgrqinw81ksvm3rr4lwpp1rpa9jyf9w-snarkos-2.1.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A snarkos https://github.com/r-ryantm/nixpkgs/archive/8c2e243480cd43254c6d2338e6a5e49801db32ec.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lfgrqinw81ksvm3rr4lwpp1rpa9jyf9w-snarkos-2.1.1
ls -la /nix/store/lfgrqinw81ksvm3rr4lwpp1rpa9jyf9w-snarkos-2.1.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

snarkos

Maintainer pings
cc @happysalada for testing.

Result of nixpkgs-review pr 238406 run on x86_64-linux 1

1 package built:

snarkos


How to troubleshoot ALB not appearing for subdomain

Hello, I've gotten my ALB to work in the past using the following config and the domain domain.com, yet when I create the ALB for a subdomain, such as foobar.domain.com, I can't see my ALB created in AWS.
Note that I've created my cluster using kops with RBAC enabled:
kops create cluster --zones us-east-1a,us-east-1b,us-east-1c,us-east-1e --node-size t2.micro --node-count 2 --node-volume-size 15 --master-size t2.micro --authorization RBAC --node-size t2.micro --master-size t2.micro --master-count 1 --master-volume-size 15 --ssh-public-key ~/.ssh/id_rsa.pub <MY CLUSTER NAME> --yes

Here's my config:
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: fcc
  namespace: default
  annotations:
    kubernetes.io/ingress.class: alb
    alb.ingress.kubernetes.io/scheme: internet-facing
    alb.ingress.kubernetes.io/certificate-arn: "arn:aws:acm:us-east-1:redacted:certificate/redacted"
    alb.ingress.kubernetes.io/listen-ports: '[{"HTTP": 80}, {"HTTPS":443}]'
    alb.ingress.kubernetes.io/actions.ssl-redirect: '{"Type": "redirect", "RedirectConfig": { "Protocol": "HTTPS", "Port": "443", "StatusCode": "HTTP_301"}}'
spec:
  rules:
    - host: foobar.domain.com
      http:
        paths:
          - path: /*
            backend:
              serviceName: ssl-redirect
              servicePort: use-annotation
          - path: /*
            backend:
              serviceName: fcc
              servicePort: 80
---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  labels:
    app: alb-ingress-controller
  name: alb-ingress-controller
rules:
  - apiGroups:
      - ""
      - extensions
    resources:
      - configmaps
      - endpoints
      - events
      - ingresses
      - ingresses/status
      - services
    verbs:
      - create
      - get
      - list
      - update
      - watch
      - patch
  - apiGroups:
      - ""
      - extensions
    resources:
      - nodes
      - pods
      - secrets
      - services
      - namespaces
    verbs:
      - get
      - list
      - watch
---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  labels:
    app: alb-ingress-controller
  name: alb-ingress-controller
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: alb-ingress-controller
subjects:
  - kind: ServiceAccount
    name: alb-ingress
    namespace: kube-system
---
apiVersion: v1
kind: ServiceAccount
metadata:
  labels:
    app: alb-ingress-controller
  name: alb-ingress
  namespace: kube-system
---
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: alb-ingress-controller
  name: alb-ingress-controller
  namespace: kube-system
spec:
  replicas: 1
  selector:
    matchLabels:
      app: alb-ingress-controller
  strategy:
    rollingUpdate:
      maxSurge: 1
      maxUnavailable: 1
    type: RollingUpdate
  template:
    metadata:
      creationTimestamp: null
      labels:
        app: alb-ingress-controller
    spec:
      containers:
        - image: docker.io/amazon/aws-alb-ingress-controller:v1.1.0
          imagePullPolicy: Always
          args:
            - --ingress-class=alb
            - --cluster-name=<MY CLUSTER CREATED BY KOPS>
          name: server
          resources: {}
          terminationMessagePath: /dev/termination-log
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      securityContext: {}
      terminationGracePeriodSeconds: 30
      serviceAccountName: alb-ingress
      serviceAccount: alb-ingress
---
apiVersion: v1
kind: Service
# ... my service ...
---
apiVersion: apps/v1
kind: Deployment
# ... my deployment ...

Here's my Ingress description:
kubectl describe ing/fcc
Name:             fcc
Namespace:        default
Address:
Default backend:  default-http-backend:80 (<none>)
Rules:
  Host                     Path  Backends
  ----                     ----  --------
  staging.seenickcode.com
                           /*   ssl-redirect:use-annotation (<none>)
                           /*   fcc:80 (<IP_ADDRESS>:8888,<IP_ADDRESS>:8888)
Annotations:
  alb.ingress.kubernetes.io/actions.ssl-redirect:    {"Type": "redirect", "RedirectConfig": { "Protocol": "HTTPS", "Port": "443", "StatusCode": "HTTP_301"}}
  alb.ingress.kubernetes.io/certificate-arn:         arn:aws:acm:us-east-1:redacted:certificate/redacted
  alb.ingress.kubernetes.io/listen-ports:            [{"HTTP": 80}, {"HTTPS":443}]
  alb.ingress.kubernetes.io/scheme:                  internet-facing
  kubectl.kubernetes.io/last-applied-configuration:  {"apiVersion":"extensions/v1beta1","kind":"Ingress","metadata":{"annotations":{"alb.ingress.kubernetes.io/actions.ssl-redirect":"{\"Type\": \"redirect\", \"RedirectConfig\": { \"Protocol\": \"HTTPS\", \"Port\": \"443\", \"StatusCode\": \"HTTP_301\"}}","alb.ingress.kubernetes.io/certificate-arn":"arn:aws:acm:us-east-1:redacted:certificate/reacted","alb.ingress.kubernetes.io/listen-ports":"[{\"HTTP\": 80}, {\"HTTPS\":443}]","alb.ingress.kubernetes.io/scheme":"internet-facing","kubernetes.io/ingress.class":"alb"},"name":"fcc","namespace":"default"},"spec":{"rules":[{"host":"foobar.domain.com","http":{"paths":[{"backend":{"serviceName":"ssl-redirect","servicePort":"use-annotation"},"path":"/*"},{"backend":{"serviceName":"fcc","servicePort":80},"path":"/*"}]}}]}}

  kubernetes.io/ingress.class:  alb
Events:                         <none>

So the issues is that my ALB isn't appearing in AWS. (this config worked in the past btw, but not when I was using a subdomain. What is the recommended way to troubleshoot this?

Hi,
You can take a look at logs from the controller.
kubectl logs $(kubectl get pods -n kube-system | grep alb-ingress-controller | grep Running | awk '{print $1}') -n kube-system

I can help debug if you post the logs here or send to<EMAIL_ADDRESS>

I get
E0430 23:28:58.385792       1 :0] kubebuilder/controller "msg"="Reconciler error" "error"="failed to build LoadBalancer configuration due to failed to get AWS tags. Error: AccessDeniedException: User: arn:aws:sts::redacted:assumed-role/nodes.foobar.domain.com/i-0f1d33033b689b7be is not authorized to perform: tag:GetResources\n\tstatus code: 400, request id: ba767c2a-6b9f-11e9-8050-8f4b21b5e25e"  "controller"="alb-ingress-controller" "request"={"Namespace":"default","Name":"fcc"}

@seenickcode
You need to add permissions to worker node, so ALB Ingress Controller can call AWS API on behalf of you.
Below are the commands:
# change to your aws region
export REGION=us-west-2
export ALB_POLICY_ARN=$(aws iam create-policy --region=$REGION --policy-name aws-alb-ingress-controller --policy-document "https://raw.githubusercontent.com/kubernetes-sigs/aws-alb-ingress-controller/master/docs/examples/iam-policy.json" --query "Policy.Arn" | sed 's/"//g')
echo $ALB_POLICY_ARN
# change to your worker node's instance profile name, you can find it on `AWS Console->EC2-> YourWokerNode->Description->IAM Role`
export NODE_ROLE_NAME=<name-of-your-worker-node-role>
aws iam attach-role-policy --region=$REGION --role-name=$NODE_ROLE_NAME --policy-arn=$ALB_POLICY_ARN

After that, you can delete & reinstall the controller, and the issue will gone.
(Alternatively, if you add a dummy annotation to your ingress will work too without reinstall contoller)

Great, that did it. Since the node's role name is the tied to the cluster name using kops, I failed to attach this policy to the new role name.
Thank you again for the very fast response, it's much appreciated.


Is there a block update function?

Is there a block update function? So every tick, each block calls a function or something similar. In this case I want to make my cactus grow every once in a while.

Hi - the way I do this is to use custom block events to track when cactus blocks are added to or removed from the world, and then run a tick handler to occasionally update existing blocks.
E.g.:
noa.registry.registerBlock( cactusID, {
    // block settings, material, opacity, etc...
    onSet: addCactus,
    onUnset: removeCactus,
    onLoad: addCactus,
    onUnload: removeCactus,
})

function addCactus(x,y,z) {    // remember there's a cactus at (x,y,z)...
function removeCactus(x,y,z) { // forget there's a cactus at (x,y,z)...
noa.on('tick', function() {    // do something with all current cactus blocks...

Block event meanings are explained here, hope that helps.

Thanks for the answer! Sorry for asking a lot of questions!

Not at all, please assume every question is something I've forgotten to document! Thanks!


[CI] Fix asan mixin

Signed-off-by: Esteban Martinena<EMAIL_ADDRESS>

This project has no c++ code, so ASAN is not useful.


controller: Replica scheduled to the node that shouldn't able to scheduled

It happens rarely. When create a big volume, the replica sometimes will schedule to the disk that shouldn't able to scheduled.

Fixed.


useI18n() does not return locales when you use Per-component translations

Version
@nuxtjs/i18n: 8.0.0-beta.4-27812700.b832f07
nuxt: 3.0.0
Nuxt configuration
Please change to [x] if relevant for this issue:

[x] Applies to a site deployed to a static server (site generated with nuxt generate)
[x] Applies to a site deployed to a server with a Node backend

@nuxtjs/i18n configuration
i18n: {
  locales: [
    {
      code: 'en',
      iso: 'en',
      name: 'English',
    },
    {
      code: 'ja',
      iso: 'ja-JP',
      name: '日本語',
    },
  ],
  vueI18n: {
    legacy: false,
  },
},

Reproduction Link

https://github.com/phanect/repro-nuxt-i18n-lang-switcher
Steps to reproduce
If you use per-component translations (i.e. add <i18n> block in the Vue SFC), useI18n() returns i18n object without locales, therefore the lang switcher does not work.
If you remove the <i18n> block from the SFC, useI18n() works properly and you can access i18n.locales properly, so I think <i18n> block causes this problem.

Create a page component and add following code in <script setup> block:

const { locales, t } = useI18n();
const switchLocalePath = useSwitchLocalePath();

Add <i18n> block in the page component
List available locales as following code:

<a
  v-for="locale in locales"
  :key="locale.code"
  :href="switchLocalePath(locale.code)"
>
  {{ locale.name }}
</a>

What is Expected?
List of available locales are shown
What is actually happening?
List of available locales are not shown

Same problem #1658

@andreidmour I don't think this issue is a duplicate of #1658.
You didn't even extract locales from the return value of useI18n() in #1658. This issue is about locales, so I think #1658 is different problem from this issue.

Thank you for your reporting!
I've just reproduce this issue.
This issue is a bug.
I'll try to fix.

I've just fixed this issue.
You can use edge version<EMAIL_ADDRESS>Please try it!
Thanks!

@kazupon Thanks, I have confirmed the issue is fixed on my end.
Now useI18n() returns i18n.locales properly.


How can i upload an image? I am trying to drag the image till the text field, but i am always receiving this error message "后端配置项没有正常加载，上传插件不能正常使用！", What should i do to fix it?

This error is thought out from the core text editor inside. You need to config your server to use the upload function.
I'm afraid this vue-editor doesn't suit for you if you can not read Chinese, because this project is only a wrapper of UEditor, the core text editor inside.

Thaks for your answer, I already find another solution for my inssue. Thank
you anyway!
On Sun, Apr 14, 2019 at 11:05 PM SilentTiger<EMAIL_ADDRESS>wrote:

This error is thought out from the core text editor inside. You need to
config your server to use the upload function.
I'm afraid this vue-editor doesn't suit for you if you can not read
Chinese, because this project is only a wrapper of UEditor
https://github.com/fex-team/ueditor, the core text editor inside.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SilentTiger/vue-ueditor/issues/30#issuecomment-483083660,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ar68Up0-QKn-t2Cz3D2aPCAOkDT-Ka1Vks5vg95ngaJpZM4cqpEK
.


ID11018 - Updated screenshots and instructions to match redwood theme

Add here your WMS ID
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

Hi Anoosha, they match the new redwood theme and how we will start accessing Graph Studio from now one.

@ramumg - We will approve the PR once the UI changes are live.


Responsive & Performance

New responsive mobile header, media queries optimization, and language dropdown functionality fix. Added dynamic media and sizes attribute, Implemented FontPie's font optimization techniques, disabled lazy loading for above-the-fold images to improve performance.
Description

Created a new responsive mobile header
Added media queries to optimize the look of the page inside the author editor mode
Fixed the functionality of the language dropdown, which was not showing the selected language
Enabled dynamic media for the images and added a sizes attribute
Disabled lazy loading for above-the-fold images, which was delaying the largest contentful paint
Increased website performance by optimizing font delivery with FontPie

Related Issue

https://github.com/adobe/aem-site-template-standard/issues/98
https://github.com/adobe/aem-site-template-standard/issues/99
Motivation and Context
How Has This Been Tested?

I tested the changes on different devices and screen sizes and I also used https://pagespeed.web.dev/ to measure the website's performance.
Screenshots (if appropriate):
Responsive

Original performance score vs new score

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

The footer doesn't seem to be centered. Not sure if this was already the case before but it's worth aligning I guess...

Made changes to center the footer:

@vladbailescu The PR is ready for review. Could you please have a look or dispatch ? Thank you !

@vladbailescu  That's weird indeed. I can't seem to reproduce it and I tried all browsers. This is how it's supposed to look like:

Do you have this glitch also when you resize to mobile screen ?


Transfer labels

From @lfunkhouser on October 3, 2017 16:48
From @lfunkhouser on October 2, 2017 15:24
This issue is used to transfer issues to another repo
Copied from original issue: Esri/solutions-grg-widget#119
Copied from original issue: Esri/solutions-geoevent-java#79

This issue was moved to ArcGIS/solutions-erg-widget#8

This issue was moved to Esri/solutions-geoevent-java#78

This issue was moved to Esri/military-features-data#272


chore(docs): update Authentication Service documentation

chore(docs): update Authentication Service documentation

We need to tag the service before we can merge this @malta895 @pmarchini FYI
Let's finish the QA for the new authn service feature and then remember to merge this! 💪

We need to tag the service before we can merge this @malta895 @pmarchini FYI
Let's finish the QA for the new authn service feature and then remember to merge this! 💪

I confirm the QA was successful and we tagged the v3.6.0 of the service, we can merge this

I confirm the QA was successful and we tagged the v3.6.0 of the service, we can merge this

great!
Let's wait for the next PR to align the changelog and then we can merge!

We have the changelog with also the patch v3.6.1


253 implement extractor filtering by category

This PR does the following:

backend on the endpoint GET /listeners add new query parameter category to enable filtering
backend create new endpoint GET /listeners/category to list all the distinct categories in the whole listener collection
frontend implement the filter radio button groups
search and filter are independent of each other: i.e. when searching filter will jump to "all"; when filtering the searchbox will clear out

I noticed that there is only "EXTRACT" as the category in most of the V1 extractors; mannually added geospatial as an example

This is based off PR 252; just for ease of review i point this to 252


Encountering 'dict object' has no attribute 'bridge_name' and 'mac_address' in Ansible Playbook For ABI

With latest Pull of main branch. Facing issue while triggering ABI Playbook. Seems Few variables name has been changed.

This issue has been resolved. PR: https://github.com/IBM/Ansible-OpenShift-Provisioning/pull/290


[CWS-2398] Add syscall context to mount event

What does this PR do?
This PR adds syscall arguments to the mount event.
It also increases the max entries of the syscall_ctx map from 1024 to 8192 entries in order to account for the increased number of syscalls supporting context resolution.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


sct_viewer: fix typo and improve help

version 3.0.6.
typo:

help:
https://sourceforge.net/p/spinalcordtoolbox/wiki/sct_label_vertebrae/attachment/label_vertebrae_viewer.png
would be more intuitive to show the red dot where it should be.

Deprecated ?! or fixed ?!
Whoops, we can't find that page.


Detect typing in omnibox

Probably will be a little roundabout.
Allow the user to set GXM as a "search engine" (chrome_settings_overrides, search_provider) so we can detect typing in the omnibox.
Pass through searches and suggestions to an engine of the user's choice (grab a list using search api).

okay looking at this now this seems like it's gonna be way more complex than i thought 😭
firefox's search api doesn't actually return any of the urls i need (query url and autocomplete url)
i'd probably need to ask the user to input a link to the correct OpenSearch xml description. (for reference, here's Google's.)
then, search urls MUST be an https link. i would rather not host an entire server for this, for longevity and privacy reasons
LibRedirect gets around this by giving Firefox a URL that will never lead anywhere, and then intercepting the network request to redirect it to the correct place. yikes. that's not ideal but i guess it could work.
but also, would that even be possible for autocomplete? i'd have to try and find out, but worst case scenario we'd need the user to forfeit autocomplete...
also also, guessing i can't change search engine icon at runtime either, so it'd be stuck as either the GXM logo or a generic magnifying glass.
all of these hoops and hurdles to jump through just to get a scuffed search engine passthrough... is it really worth all of this for some typing sounds?

currently trying to figure this out again, this might be even more impossible than i thought it was??

keyword is absolutely required for the extension to detect keystrokes, regardless of whether or not the extension is currently the default search engine, without a "gx " at the beginning we don't know anything
can't detect autocomplete web requests, as those are made on a level higher than extensions can observe. so we can't detect typing on the default engine OR our own(!!)
can't detect keystrokes using command api as those require a modifier to be valid (Ctrl or Shift)

tl;dr: there are zero ways of detecting even just keypresses in the address bar without forcing people to type a mandatory word at the beginning of every search
i genuinely think that webextensions are too limited to implement this in a seamless way. :(
if anyone has any ideas or suggestions feel free to leave them in, i think i have to give up on this for now 💔

if anyone genuinely wants to be typing in "gx my search term here" to look things up then i guess i'll implement it?? but i don't really think enough people would be okay with that to warrant me spending time on it

HANG ON. I JUST FOUND SOMETHING. WE MIGHT BE COOKING

"fixed" by #20


v2ray api stats - not work, in log [block]

What version of V2Ray are you using?
V2Ray v5.1.0 (user) 20221127-040503 (go1.19.3 linux/amd64)
What's your scenario of using V2Ray?
try view stats
use command
v2ray api stats
What problems have you encountered?
failed to dial <IP_ADDRESS>:8080
What's your expectation?
see statistics
Please attach your configuration here
Server configuration:
 {
  "api": {
    "services": [
      "StatsService"
    ],
    "tag": "api"
  },
  "inbounds": [
    {
      "listen": "XXXXXX",
      "port": 443,
      "protocol": "vless",
      "settings": {
        "clients": [

          {
            "id": "XXXXXX",
            "level": 0
          }
        ],
        "decryption": "none",
        "fallbacks": [
          {
            "dest": "/dev/shm/default.sock",
            "xver": 1
          },
          {
            "alpn": "h2",
            "dest": "/dev/shm/h2c.sock",
            "xver": 1
          }
        ]
      },
      "sniffing": {
        "destOverride": [
          "http",
          "tls"
        ],
        "enabled": true
      },
      "streamSettings": {
        "network": "tcp",
        "security": "tls",
        "tlsSettings": {
          "alpn": [
            "h2",
            "http/1.1"
          ],
          "certificates": [
            {
              "certificateFile": "/etc/ssl/v2ray/fullchain.pem",
              "keyFile": "/etc/ssl/v2ray/privkey.pem"
            }
          ]
        }
      }
    },
    {
      "listen": "<IP_ADDRESS>",
      "port": 8080,
      "protocol": "dokodemo-door",
      "settings": {
        "address": "<IP_ADDRESS>"
      },
      "tag": "api"
    }
  ],
  "log": {
    "access": "/var/log/v2ray/v2ray-access.log",
    "error": "/var/log/v2ray/v2ray-error.log",
    "loglevel": "debug"
  },
  "outbounds": [
    {
      "protocol": "freedom",
      "settings": {},
      "tag": "direct"
    },
    {
      "protocol": "blackhole",
      "settings": {},
      "tag": "block"
    }
  ],
  "policy": {
    "levels": {
      "0": {
        "statsUserDownlink": true,
        "statsUserUplink": true
      }
    },
    "system": {
      "statsInboundDownlink": true,
      "statsInboundUplink": true,
      "statsOutboundDownlink": true,
      "statsOutboundUplink": true
    }
  },
  "routing": {
    "domainStrategy": "AsIs",
    "rules": [
      {
        "ip": [
          "geoip:private"
        ],
        "outboundTag": "block",
        "type": "field"
      },
      {
        "inboundTag": [
          "api"
        ],
        "outboundTag": "api",
        "type": "field"
      }
    ]
  },
  "routingRules": {
    "inboundTag": [
      "api"
    ],
    "outboundTag": "api",
    "type": "field"
  },
  "stats": {}
}

Client configuration:

Please attach error logs here
Server error log:
2022/11/27 05:14:43 [Debug] [4047726615] proxy/dokodemo: processing connection from: <IP_ADDRESS>:35562
2022/11/27 05:14:43 [Info] [4047726615] proxy/dokodemo: received request for <IP_ADDRESS>:35562
2022/11/27 05:14:43 [Info] [4047726615] app/dispatcher: taking detour [block] for [tcp:<IP_ADDRESS>:0]
2022/11/27 05:14:43 [Info] [4047726615] app/proxyman/inbound: connection ends > proxy/dokodemo: connection ends > proxy/dokodemo: failed to transport response > io: rea
d/write on closed pipe

Client error log:

Please attach access log here
2022/11/27 05:33:31 <IP_ADDRESS>:49244 accepted tcp:<IP_ADDRESS>:0 [block]
2022/11/27 05:33:32 <IP_ADDRESS>:49258 accepted tcp:<IP_ADDRESS>:0 [block]
2022/11/27 05:33:34 <IP_ADDRESS>:49272 accepted tcp:<IP_ADDRESS>:0 [block]

Other configurations (such as Nginx) and logs here
If V2Ray cannot start up, please attach output from --test command
If V2Ray service is abnormal, please attach journal log here

Your config file is wrong.


workbench.list.openMode to control single click vs double click to open

VSCode Version: 1.20.0
OS Version: Windows 10

-- Sorry for the English, I used Google Translator --
When using "workbench.list.openMode" to control single click vs. double click to open ", I thought it would not affect "workbench.editor.enablePreview".
1- That is, when "workbench.list.openMode: doubleClick" and "workbench.editor.enablePreview: true" would affect only the folders, and when
"workbench.list.openMode: doubleClick" and "workbench.editor.enablePreview: false" would affect everything for double click.
2- An alternative would be to configure extensions of file types supported by VSCode that would NOT enter the preview. By default, all files would enter the preview.
For me the first option would solve. Does this setting make sense? 🤔

Does this issue occur when all extensions are disabled?: Yes

@Tekbr there is no relation between the 2 settings and I have no idea what you refer to with regards to "would affect only the folders".
The setting is very simple:

singleClick: folders expand on single click, items expand on single click
doubleClick: folders expand with double click, items open with double click

Since we also "pin" items on double click ("pin" is the opposite of "preview"), it means that opening items with double click will "pin" them.
Are you referring to being able to open an item in "preview" with a double click?

@bpasero Let's try again!! If you still have questions, please ask me. And if it now makes any sense to you the explanation.
Version - 1.19.3
Settings: workbench.editor.enablePreview: true
1- Folders and Files opened in a single click
2- Files being opened in Preview.
Version - 1.19.3
Settings: workbench.editor.enablePreview: false
1- Folders and Files opened in a single click
2- Since files opened directly for editing.
Version - 1.20.0
Settings: workbench.list.openMode: singleClick
workbench.editor.enablePreview: true
1- Folders and Files open in a single click
2- Being what files open in Preview.
Version - 1.20.0
Settings: workbench.list.openMode: singleClick
workbench.editor.enablePreview: false
1- Folders and Files open in a single click
2- Since files open directly to edit.
Version - 1.20.0 - Expected
Settings: workbench.list.openMode: doubleClick
workbench.editor.enablePreview: true
1- Folders opened in double click
2- There is no possibility to see file in the preview.
NOTE: Expectation would be to preview the files only with singleClick. With a double click it would open directly to edit
Version - 1.20.0
Settings: workbench.list.openMode: doubleClick
workbench.editor.enablePreview: false
1- Folders and Files opened in double click
2- Since files opened directly for editing.
That is, in my opinion, workbench.editor.enablePreview: true becomes useless when workbench.list.openMode is set up with doubleClick. Or I'm doing something wrong.

Version - 1.20.0 - Expected
Settings: workbench.list.openMode: doubleClick
workbench.editor.enablePreview: true
1- Folders opened in double click
2- There is no possibility to see file in the preview.
NOTE: Expectation would be to preview the files only with singleClick. With a double click it would open directly to edit

I do not think that is what people want. People want to only open a file on double click, neither preview nor in edit mode. If we would change that, these people would basically have to make changes to 2 settings to get the desired behaviour. It is true that these people probably also do not want the preview editor in the first place but they might not be aware of the setting.
The preview feature can still be used, e.g. if you open an entry from quick open. There are other ways how editors can also be opened programmatically in preview mode.
I am closing this as designed until more people complain about it.

-- Sorry for the English, I used Google Translator --

I do not think that is what people want. People want to only open a file on double click, neither preview nor in edit mode.

@bpasero But before having this function, they already put workbench.editor.enablePreview: false, not to have the preview. It soon makes sense if the preview is true to show and doubleClick active.
With workbench.editor.enablePreview: false, it works the way it is. As I mentioned in the previous review.

If we would change that, these people would basically have to make changes to 2 settings to get the desired behaviour.

And as a single function you are removing an existing function.

It is true that these people probably also do not want the preview editor in the first place but they might not be aware of the setting.

The same way I had to go back to the previous version to check the reason. I was not aware that by enabling doubleClick with preview, the preview would not work.

The preview feature can still be used, e.g. if you open an entry from quick open. There are other ways how editors can also be opened programmatically in preview mode.

Yes, but it only works for some recent files and not for everyone.


Don't crash when building docs

I moved the @github/markdown-toolbar-element import into the render function like suggested in https://www.gatsbyjs.com/docs/debugging-html-builds/
Closes https://github.com/github/markdown-toolbar-element/issues/68
Screenshots
Please provide before/after screenshots for any visual changes
Merge checklist

[ ] ~Added/updated tests~
[ ] ~Added/updated documentation~
[ ] Tested in Chrome
[x] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

cc/ @keithamus

Responded on the upstream issue, let me know what you think!
Separately, would this run into the same mix-and-match issue described in: https://github.com/primer/react/pull/2339 ? Specifically:

This mix of require and import confuses transpilers and occasionally breaks things in unexpected and hard-to-debug ways (for example, see this Slack thread).

I pushed a commit that utilizes useEffect and dynamic imports so that we can avoid require calls if we want.

@koddsson sorry about the dynamic import stuff 😞 Seems like the same issue that @iansan5653 ran into.
For the mix-and-match, this would end up being a bundler behavior, right? If I understand right, require() is undefined in an ESM scope.
Also wanted to check-in with the deploy preview, it seems like the build is failing with this error:

Also wanted to check-in with the deploy preview, it seems like the build is failing with this error:

I don't see this anywhere. Do you have a link to this UI?

@koddsson let me know if this works! https://github.com/primer/react/actions/runs/3144532666/jobs/5110661581
I think it's showing up for the Deploy (fork) workflow

I annoyingly can't reproduce the deploy error locally. Any ideas? Can you reproduce it?

I annoyingly can't reproduce the deploy error locally. Any ideas? Can you reproduce it?

Seems like this was a issue with me not having push access to this repo. Seems to be running green now 👍🏻


where to set batcheize and num of iteration

thks  !!

 and .
Number of iterations = (epochs * total_images) / batch_size


add myself as a code owner

Limiting code owner list to DRIs that approve and merge pull requests.


Implemented copyItem method

Would it be possible to have a test for this method?

I agree with @pushkarnk on getting a test for new code.

Closing in favor of #847


excon is a dependency

I'm kinda getting back to Ruby development after a few year hiatus. So I'm sorry if this is incorrect. I couldn't seem to use wikidata-client until I had manually installed excon in my own project. This seemed like an oversight?

Thanks for the report.
Why did you remove the Gemfile.lock?

Because the Travis build failed when it was checked in. I read at http://stackoverflow.com/questions/4151495/should-gemfile-lock-be-included-in-gitignore that it is recommended practice not to check on age file.lock files for gem projects.


OWNERS title

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

✨PR Status ✨

Condition

Has Required Labels approved, lgtm
❌

No Blocking Labels wip, hold
✅

This Pull Request will not merge until all required tests have passed

/lgtm


getSelectedText appears to return Clipboard.readText() sometimes

Description
I am wanting to create an extension which creates a markdown link using the URL from the clipboard and the currently selected text.
The extension code I've put together works for the most part but getSelectedText and Clipboard.readText() often return the same results when they should 100% not.
Importantly this appears to be a case of which app you're in. VSCode is a good case to recreate this. Selecting text from a textarea (like the one I'm entering in now does appear to work correctly)
Steps To Reproduce
Checkout this repo - https://github.com/omonk/linkdown

npm run dev to start up the extension
Select and copy url (https://google.com)
Observe the url is in your clipboard history
Select a new portion of text from an application
Run linky command
See that the markdown created is either the URL twice https://google.com even when the selected text is not the link or JS throws with the error that the URL is invalid, as the URL that has been provided by Clipboard.history is in fact selected text.

Current Behaviour

Clipboard.readText() returns selected text
getSelectedText() returns the Clipboard.readText()

Expected Behaviour

Clipboard.readText() returns the the correct Clipboard.readText()
getSelectedText() returns the correct selected text

Yeah the heuristic of the getSelectedText is to try to use accessibilities APIs but if it can't, it'll try to send a copy command to get the selection. We found that some apps don't behave nicely and we have workaround that with the clipboard :/
Not much we can do here...


Implement complex norm in optimizers

As proposed in #196, I reviewed all transformations in transform.py and optimizers in alias.py, and replaced x**2 with (x.conj() * x).real to support complex numbers.
I think we need to double check the correctness of all changes. What tests do you think we should add?
I'm not sure how to generalize the transform sm3 to complex numbers. The optimizers fromage and noisy_sgd should be working, but in the tests I can't find a learning rate to make them work well.

I rewrote AliasTest.test_optimization in alias_test.py to test for complex parameters on two target functions. By the way, I added the missing tests for all optimizers in alias.py.

this should also use the complex norm in gradient clipping. See #287

Now I've also implemented complex gradient clipping in this PR (and closed #161), and I hope it doesn't make this PR look too large.

I'd suggest renaming abs_sqr to abs2, which is the name used in Scipy, C++ Eigen, Julia ...

@mkunesch If you could spare the time, could we get a review on this PR?

I'd suggest renaming abs_sqr to abs2, which is the name used in Scipy, C++ Eigen, Julia ...

Yes I agree

I'd suggest renaming abs_sqr to abs2, which is the name used in Scipy, C++ Eigen, Julia ...

In scipy it has been decided to name it abs_sq, not abs2 FYI.

I'd suggest renaming abs_sqr to abs2, which is the name used in Scipy, C++ Eigen, Julia ...

In scipy it has been decided to name it abs_sq, not abs2 FYI.

@wdphy16 could you update the name to abs_sq for (forward?) consistency with scipy, please?

@wdphy16 could you update the name to abs_sq for (forward?) consistency with scipy, please?

Done that

@hbq1 After you merge this PR and #288 could you also tag a new release?

@PhilipVinc we'll release a new version early next week if you don't mind (releasing something Friday evening might turn into a restless "weekend" ;) )

It's ok, thank you!


[stub] add stub_tcp_upstream support

This allows using TCP instead of UDP for a specific stub zone
Fixes #94

I did not update the smallapps as I was not compiling with make and those are skipped.
I would be happy to get those to compile if the idea is agreed upon.

smallapps was only a couple of changes away, so I went ahead and applied changes.

@chantra - i've prepared PR based on your changes which also allows using tcp upstreams in forward zones

Should the be closed, when PR #519 were merged?

Thanks folks for pushing it through.


school library: add basic UI

In this PR:

Created a app.rb file that has methods for:

List all books.
List all people.
Create a person (teacher or student).
Create a book.
Create a rental.
List all rentals for a given person id.

Created main.rb that has methods:

Present the user with a list of options to perform.
Lets users choose an option.
If needed, ask for parameters for the option.
Have a way to quit the app.

Hi @thecodechaser, thanks for your review.
The app is not running because my executable file is not main.rb is the one named main.
I've removed the unnecessary files.

Hi @thecodechaser, thanks for your review. The app is not running because my executable file is not main.rb is the one named main.
I've removed the unnecessary files.

Hi, @Alexr16 Okay, but it is great practice to name all ruby files with the .rb.

Hi @thecodechaser, thanks for your review. The app is not running because my executable file is not main.rb is the one named main.
I've removed the unnecessary files.

@Alexr16 Ohk that's good, If everything is good you can go for another review, Good luck


Pretty print of <script> in html causes extra lines to be added on each subsequent pretty print

Take https://github.com/revelc/formatter-maven-plugin/blob/main/src/test/resources/someFile.html and add

Can we merge this issue with #1689?

@damienhunter30 This is a different issue.  A script tag with content in it triggers a new line.  So each time the file is reformatted, it gets another new line.  So this continues to grow out of control.  Issue #1689 simply is extra space being left at end of various lines (and always a tag ending >).  In that case there is only ever one there, it doesn't add more extra spaces, just leaves them on lines that they exist instead of properly trimming.

@hazendaz can you add a small reproducing testcase here? I haven't been able to repro it using try.jsoup

Can you expose all parameters there?  That site only does single spacing so most of the issues don't expose there.  Need 4 character spacing at least.  I can get the remainder later but suspect that is core to most issues.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Jonathan Hedley @.>
Sent: Monday, December 27, 2021 9:08:17 PM
To: jhy/jsoup @.>
Cc: Jeremy Landis @.>; Mention @.>
Subject: Re: [jhy/jsoup] Pretty print of <script> in html causes extra lines to be added on each subsequent pretty print (Issue #1688)
@hazendazhttps://github.com/hazendaz can you add a small reproducing testcase here? I haven't been able to repro it using try.jsouphttps://try.jsoup.org/~Hn6jvmLvU88RUcPDhlcdShk3T0U
—
Reply to this email directly, view it on GitHubhttps://github.com/jhy/jsoup/issues/1688#issuecomment-1001831655, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAHODIZQH5CAREMPSBYLDDTUTELZDANCNFSM5KMIYVEA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

Here is a link to config used.  https://github.com/revelc/formatter-maven-plugin/blob/main/src/main/resources/formatter-maven-plugin/jsoup/html.properties
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Jeremy Landis @.>
Sent: Monday, December 27, 2021 10:21:06 PM
To: jhy/jsoup @.>
Subject: Re: [jhy/jsoup] Pretty print of <script> in html causes extra lines to be added on each subsequent pretty print (Issue #1688)
Can you expose all parameters there?  That site only does single spacing so most of the issues don't expose there.  Need 4 character spacing at least.  I can get the remainder later but suspect that is core to most issues.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Jonathan Hedley @.>
Sent: Monday, December 27, 2021 9:08:17 PM
To: jhy/jsoup @.>
Cc: Jeremy Landis @.>; Mention @.>
Subject: Re: [jhy/jsoup] Pretty print of <script> in html causes extra lines to be added on each subsequent pretty print (Issue #1688)
@hazendazhttps://github.com/hazendaz can you add a small reproducing testcase here? I haven't been able to repro it using try.jsouphttps://try.jsoup.org/~Hn6jvmLvU88RUcPDhlcdShk3T0U
—
Reply to this email directly, view it on GitHubhttps://github.com/jhy/jsoup/issues/1688#issuecomment-1001831655, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAHODIZQH5CAREMPSBYLDDTUTELZDANCNFSM5KMIYVEA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks, fixed. The issue was that when pretty printing in outlineMode, the closing </script> tag was incorrectly being indented, which caused extra newlines to be added to the script data nodes.

This is confirmed fixed.


Changing the way AP issue is reported

Hi @gsmet, @gunnarmorling!
Finally had a chance to look into that AP problem. First approach that I took a few days ago is in this PR. We don't have a way to distinguish when the error should be reported on List annotation or on an element of that list (because if it's a list annotation it is passed to MultiValuedChecks and checks receive list elements and know nothing about the wrapping annotation).
I don't really like the way I've changed the code to support such possibility (to choose on which annotation to report on). But more cleaner ways required much more refactoring so I didn't want to do that (as maybe you don't think that such thing is needed (possibility to report on either list or element annotation)).
I think that for this particular case it will be more suitable to report an error on List annotation rather than on the list element.
But this got me thinking more - we do have other tests with list annotations and they are working just fine. So why would this one behave like that ? The only difference that I've found is that in other cases list annotations are using arrays to declare list elements. And when I changed MixDirectAnnotationAndListContainerAnnotation to do the same - it helped
@Length.List(
		{ @Length(min = 45) }
)

instead of
@Length.List(@Length(min = 45))

the error is reported on correct line (well on the line where the list element is located)
So maybe we would not need to support "multiple error lines" ...

The only difference that I've found is that in other cases list annotations are using arrays to declare list elements. And when I changed MixDirectAnnotationAndListContainerAnnotation to do the same - it helped

Ah ah, I also played the Spot the 7 differences game but didn't find this one...
So I suppose it's a bug in JDK 8 fixed in JDK 9.
I would be for just fixing the test. People using List will have multiple annotations in it (or they won't need it anyway). I think it's sufficiently a corner case so that we ignore it, especially because it's fixed in the next JDK version.
I just wanted to have stable tests for JDK 8 and 9 so let's just add the braces and leave with it.
WDYT?
Thanks for getting to the bottom of it!

@gsmet I think that it's a good idea to start with fixing the test - I'll rework this commit to do just that. (add braces to annotation)
As for the check itself.. I'd say that there's still something that we can improve. In next case:
@Length(min = 5)
@Length.List({
		@Length(min = 45),
		@Length(min = 46),
		@Length(min = 47)
})
@Target({ ANNOTATION_TYPE, METHOD, CONSTRUCTOR })
@Retention(RUNTIME)
@Constraint(validatedBy = MixDirectAnnotationAndListContainerAnnotation.MixDirectAnnotationAndListContainerAnnotationValidator.class)
@Documented
public @interface MixDirectAnnotationAndListContainerAnnotation {

AP will report 3 errors for each @Length(min = 45), @Length(min = 46), @Length(min = 47) that there's a mix of @Length(min = 5) and @Length.List(...
But we should come up with something better than what I've done in the initial commit :)

Merged, thanks!


SlimPad Added

Ready to submit your project? Nice job! Hopefully you've read the submission guidelines. If not, make sure to read them first.
If you're unsure about the format, make sure to check out the Orpheuspad example and follow the format there.
If this PR isn't a submission, that's okay too! If there's anything about this PR template that seems confusing, let me know in #hackpad on the slack!

@a3acx  Thanks for reminding. I just added boot.py :)

@qcoral  Thanks!!!


Does not work

Hello,
This particular government environment setting does not work. The azure file storage location will still point to commercial and error out
. As well, the setup process that is identified before installation doesn't follow the flow of how the app installation works. The application never presents a "welcome" in such a way to edit the config file to the setting listed. The application installs and even setting the config option after installation doesn't take effect.

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

ID: 7ceaa921-290f-7730-e787-2ee040d616ae
Version Independent ID: a0140e51-138c-7cf5-1f6c-ac66a0588a6c
Content: Migration Manager Government Cloud settings - Migrate to SharePoint and OneDrive
Content Source: migration/mm-gov-cloud.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


bluetooth headset button support

As riot has bluetooth headset support it must answer and release the call by bluetooth headset button.

looks like TelecomManager should be used for this
jitsi meet ends call with bluetooth button so it's code can be used as a sample
similar discussion with linphone:
https://github.com/BelledonneCommunications/linphone-android/issues/574
https://github.com/BelledonneCommunications/linphone-android/issues/618


RecyclerView

Hello, when create RecyclerView with GridLayoutManager, foreground in items of recyclerview did not work properly, some item have foreground other not. Tested with ForegroundImageView. When using RecyclerView with LinearLayoutManager everything works good.

Is there a chance you can provide some sample code where this is occurring?

My item loyout: http://pastebin.com/pu0vAtUP
Simple RecyclerView adapter: http://pastebin.com/QEHCq9LH
Code to initialize RecyclerView and adapter: http://pastebin.com/4Rh2N9Jn
I hope this helps, but I think that the problem can be with RelativeLayout.

Looking at your item layout, why not just use android:background on the more button? It does not already have a background, so it should work just fine. Also, what version of Android are you seeing this on? 6.0+ defers to the Android implementation, so it might be a problem with the platform.

If i set image background not a app:srcCompat, then i can't set image padding and android:tint (color for vector drawable), use android 5.0.

I tested more, and create a simple layout, forground not work: http://pastebin.com/gdzpMsL2


Fix up changelog, readme, audit

Docs

[x] fixes #152
[x] updates changelog

Deps

[x] resolve package-lock.json
[x] apply npm audit (one minor remains, investigating...)
[x] randomly pick one of #86 or #82 unless somebody weighs in...

:bell: @conda-incubator/setup-miniconda-team

Thanks!


When I execute a teleport command while on a train, it freezes.

Expected Behaviour
I tried to run a teleport command to go home 8km away.
Actual Behaviour
The moment I executed the command described below, Minecraft stopped working.
By the way, the world data is not damaged and is safe.
Reproduction Steps
Steps to reproduce the behaviour:
1.Enter the following command while on the train. (Coordinates are arbitrary. )

/tp @p 801.93 74.00 4725.34 267.51 33.64

that's all
Crash Report
None
Screenshots
None
Environment

Minecraft Version:1.17.1
Installed Mods:FabricAPI, Optifine
OS : Windows 10 home(21H1)
CPU: AMD Ryzen 3600
GPU: GeForce GTX 1660 SUPER
RAM: 8GB*2
HDD+SSD : 2TB+500GB

Yeah, the train wants you to be in it until you get out. Push shift or get off the train at a station to do that.

Also despite the insanely fast computer you have, the trains aren't meant to handle people suddenly moving 8km away.
And @p is nearest player, to teleport yourself use @s.

Seems kinda like how I crashed Centown. Except I don't think I was on a
train.

Can you please confirm if this still happens in the latest version? Do you also have the logs and crash report?

I verified it in the same world with MTR-1.17.1-3.0.0-beta-8.
Even if I executed it with the following command, it did not freeze.

/tp @p ~ ~ ~100000
However, when I executed it with the command copied by F4 + C, it temporarily froze.
/execute in minecraft:overworld run tp @s 4041.50 57.50 -4480.50 283.32 58.40
Also, it has not been teleported to the specified coordinates.

There is no crash report because it was killed in Task Manager.

Can you please confirm if this happens in the latest version again? Thanks a lot!


Crate type should be cdylib rather than dylib ?

The documentation mentions that the crate type should be set to dylib so that a shared object loadable by Java is produced. I had to set it to cdylib, dylib produces objects that link dynamically to libstd.

Yes. It should be cdylib.


Open data enhancements

Makes API more gwpy-like (e.g t0 -> start, tf -> end)
If an IFO name is passed as a channel to fetch_timeseries, will query open data
If a specific open data flag is passed to query_segments will query open data (see here).

closes #20 #23


Error: Cannot find module '../build/index'

I am getting following error in the latest version.
linkchecker Documents/openshift-docs/install_config/persistent_storage/
module.js:491
    throw err;
    ^

Error: Cannot find module '../build/index'
    at Function.Module._resolveFilename (module.js:489:15)
    at Function.Module._load (module.js:439:25)
    at Module.require (module.js:517:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/home/gnelson/.nvm/versions/node/v8.4.0/lib/node_modules/@oasp/asciidoc-link-checker/bin/linkchecker:5:20)
    at Module._compile (module.js:573:30)
    at Object.Module._extensions..js (module.js:584:10)
    at Module.load (module.js:507:32)
    at tryModuleLoad (module.js:470:12)
    at Function.Module._load (module.js:462:3)

Version:
├──<EMAIL_ADDRESS>(github:oasp/asciidoc-link-checker#ac64831a376cd642f27ed09d334a1ad4e6a16708)
Node: v8.4.0
NPM: 5.4.1

Hi @gaurav-nelson, sorry for the delayed response.
After last update the /build folder, as it is auto generated when you compile the code with npm run build or yarn build was removed from the repository.
If you install the tool globally with npm i -g @oasp/asciidoc-link-checker you will find the /build directory in your global node_modules/@oasp/asciidoc-link-checker folder as it is uploaded in the npm registry.

@sjimenez77 Thank you for the update. Installing with npm i -g @oasp/asciidoc-link-checker fixed the issue.


New release?

The latest bug fixes are not in the 2.0.3 release. Could you create a new one so we can just install the library using npm without having to reference the GitHub repo directly?
Thanks!

done


i18n(pt-br): translating guides/pages file

Description

Adding the translation of the guides/pages.mdx missing file.

@Fryuni thank you for the review. I did the requested changes but there's still one issue I wasn't able to solve, as pointed here: https://github.com/withastro/starlight/actions/runs/8299714503/job/22716051747#step:6:484
Do you have a clue, please? Thank you.

@Fryuni thank you for the review. I did the requested changes but there's still one issue I wasn't able to solve, as pointed here: https://github.com/withastro/starlight/actions/runs/8299714503/job/22716051747#step:6:484 Do you have a clue, please? Thank you.

I did dig a little more and ended with just one last broken link.

@Fryuni thank you for the review. I did the requested changes but there's still one issue I wasn't able to solve, as pointed here: https://github.com/withastro/starlight/actions/runs/8299714503/job/22716051747#step:6:484 Do you have a clue, please? Thank you.

I did dig a little more and ended with just one last broken link.

awesome @Fryuni and @HiDeoo ! Thank you for your time reviewing it. I think all tests are fixes now. Thank you so much.


Sciview crashes ImageJ when starting

I just installed SciView. I'm using ImageJ 1.53c with Java v 1.8.0_172. When I open the SciView plugin in shows a starting screen for about a second and then the starting screen and ImageJ close abruptly. Any advice on how to troubleshoot this?

There is a compatibility issue between Java8 and sciview. We require Java 11. The fix for the moment is to just use Java11.
Part of the next steps for sciview/Fiji involve upgrading Fiji to Java 11, this issue is tracking that: https://github.com/scenerygraphics/sciview/issues/520


FilterPlug: Fix compatibility with legacy SubGraphs and Dots

Note this causes a failure in UnionFilterTest which was explicitly ensuring these connections can't be made.
I'm not sure what you think about this @johnhaddon ... the problem seems to come up because we have a bunch of published tools with filters routed through Dots, and the way Dots are serialized into the files means that the dot['out'].setInput( dot['in'] ) call gets rejected...
My change here seemed like the simplest workaround, though it obviously breaks that UnionFilter test. Maybe a better solution would be some sort of childAdded connection for Dots that makes the internal connection occur earlier? We'd still be stuck with the problem my new test case is pointing out, which is that you can't plug the output of an Int-Dot with a Filter source into anything...

Actually, as soon as I finished writing the PR I thought of a better way. Hopefully this seems more acceptable now. I'll probably just merge it if Travis passes so we can get a patch release out today.

Note that the test isn't really anything todo with FilterSwitch, that was just the first FilterPlug test file I found, since we don't seem to have one just about the plugs...


Fixes broken Rabbit exchange checks.

Fixes https://github.com/JasperFx/wolverine/issues/367.
Adds test coverage for RabbitMqExchange environment checks.
Changes exchangeName in Rabbit MQ end_to_end.use_fan_out_exchange() test from "fanout" to "fanout1" to avoid future collisions with the fanout exchange type.
Modifies RabbitMqExchange to reference exchanges by name rather than by type.

Got it, thanks!


Let the single cupertino picker use the magnifier too

Actually this is probably not the right approach. Since iOS pickers by default have this visual effect, we should make it the default in CupertinoPicker. Edit the list wheel viewport doc template to not have the default value in the template. Append the default in the Cupertino widget doc.

Actually the way Apple does this is not consistent. I just recently updated my iPod and the countdown timer no longer has this magnifier effect. However the others still do, so I don't know whether we should have this by default or not.
In my opinion I would rather stick with the non-magnifying style for all pickers, firstly because it is in the newest update so it is newer than the other one, secondly because it is easier to layout.


Data Quality from Alpaca and Quandl

Daily data from Alapca started around 2008 but data quality before 2015 could be bad.  Example, data for EEM from 2008 to 2010 have large fluctuations.
Write a script to catch the following behaviours in time series:
Large fluctuations over a period: Can use absolute change (A stock should not go more than 50% up or down in a day usually) or volatility (Reject periods with unreasonably high volatility with prices fluctuating between two values)
Daily data from Quandl could have a lot of missing entries, especially back-month futures.
Analyse the distribution of missing data.
Consider using other data sources such as Alpha-vantage and Yahoo Finance.

Data from Alapca for TSLA is wrong as it is not split adjusted.


[aon_timer] Consider requirements for clk_aon_i vs clk_i

The current aon_timer testbench has a limitation so we must start clk_aon_i before clk_i can start (https://github.com/lowRISC/opentitan/issues/6821). This is not an issue for earlgrey as the aon clock is always active before other clocks (the FSM in pwrmgr that enables the fast clocks is clocked from the aon clock).
We should consider either making this an explicit design constraint or fixing the limitation to allow the fast clock to start with the aon clock in the aon_timer testbench.
I'd favour the former as it seems like a reasonable design constraint.

I agree with making this an explicit design constraint for the aon_timer testbench.

This constraint sounds sensible to me as well. So we need to update the aon_timer documentation/specification and constrain the DV environment appropriately? Or does the DV environment already do this correctly?

Assigning @GregAC to please take a look and propose a resolution -- feel free to delegate the implementation ofc

By @GregAC: This seems to be a doc-only issue. We don't care about starting the other clocks before the AON clock for Earlgrey. But we should confirm this with the PD team.
By @rswarbrick: We should add an assertion / constraint in the chip-level testbench to ensure this always holds.

Confirmed by PD team that clk_aon_i is always active before clk_i


Document Doom runtime optimizations

Doom's startup optimizations are well documented. For example,
https://www.reddit.com/r/emacs/comments/f3ed3r/how_is_doom_emacs_so_damn_fast/
and
https://github.com/hlissner/doom-emacs/blob/develop/docs/faq.org#how-does-doom-start-up-so-quickly
I do not think the runtime optimizations are well documented. I have been picking some seemingly useful snippets from the repo for my own use, but I don't have a great handle on them. These include
;; Optimizations
(setq-default bidi-display-reordering 'left-to-right
              bidi-paragraph-direction 'left-to-right)
(setq bidi-inhibit-bpa t)
(setq-default cursor-in-non-selected-windows nil)
(setq fast-but-imprecise-scrolling t) ;; <-- this one is great on Windows 10!
(setq frame-inhibit-implied-resize t)
(setq ffap-machine-p-known 'reject)
(setq inhibit-compacting-font-caches t)

and a few others. It would be great to centralize Doom's runtime optimizations into a document with some commentary.

I am interested in writing about it, but the list is hopelessly long and ever changing. The ones you've captured are some of the most stable. Perhaps sometimes after new years I'll look into adding a section for it in the FAQ or our Discourse.


HDDS-7508. [Snapshot] Finish TestOMSnapshotDAG, restore CompactionNode SnapshotID field for debugging

What changes were proposed in this pull request?
This is a follow-up to https://github.com/apache/ozone/pull/3824. See jira HDDS-7508 description.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7508
How was this patch tested?

[x] Integration test TestOMSnapshotDAG.

@GeorgeJahad @neils-dev

These methods don't appear to be used.  Can they be removed from TestRocksDBCheckpointDiffer.java?
updateRocksDBInstance
testDefaultColumnFamilyOriginal
testCertainColumnFamily

Are these methods still needed?:
traverseGraph
printMutableGraphFromAGivenNode
printMutableGraph
They seem like they might be useful tools, so it might be good to keep them, but they don't appear to be checked by any tests.  So there is no incentive to keep maintaining them, or awareness of when they've stopped working.

It seems like internalGetSSTDiffList() is the heart of the compaction diff code.   But it's not clear the tests fully excercise it.  It contains 5 or 6 different if statements to handle all the different possible cases, and I can't tell whether the tests hit each of those cases.

Will remove those unused test helper methods.

These methods don't appear to be used. Can they be removed from TestRocksDBCheckpointDiffer.java?
updateRocksDBInstance testDefaultColumnFamilyOriginal testCertainColumnFamily

Thanks. Will remove those unused test helper methods.

Are these methods still needed?: traverseGraph printMutableGraphFromAGivenNode printMutableGraph
They seem like they might be useful tools, so it might be good to keep them, but they don't appear to be checked by any tests. So there is no incentive to keep maintaining them, or awareness of when they've stopped working.

Make sense. Will move them to the test class.

LGTM. But I suspect the result can be different if the RocksDB behavior changes. We should have a way to directly call the
rocksDB compaction lister and generate the deterministic DAG that we want to generate.Such a DAG traversal can give a constant output result.
I guess the current PR can go in as it is. But this is the idea for a next PR that will be very deterministic.

Thanks @GeorgeJahad @prashantpogde @hemantk-12 for reviewing this.


Removed the" Reconfigure the Node's Kubelet in a Live Cluster " page

#31585
Removed the" Reconfigure kubelet " file

Hi @AtriChatterjee1 , this PR has 2 issues:

Only need to include one language (such as en), and leave other languages to different localization team
Squash your commits into one please

/hold

Still working on this?

When would this PR be OK to unhold?

When would this PR be OK to unhold?

Remove the change to zh-cn page from this PR; 2. resolve the conflict.

Due to the author's lack of response to comments or feedback for a period of 2 weeks or more, we consider this pull request (PR) as inactive. If necessary, please resubmit it for further consideration.
Refer to 'When to close Pull Requests'.
/close


Make the website static

Current architecture is:
browser --> nginx
              - /* ------------> node.js frontend
              - /api, /auth ---> Python aiohttp backend ---> MongoDB

It is possible to make the frontend part generated to set of static files, so no node.js process would bee needed on run time (only on build time). Those static files could be served by nginx, by some CDN from static storage (like AWS S3) or by the aiohttp server.
browser --> nginx or CDN
              - /* ------------> static files (next.js export)
              - /api, /auth ---> Python aiohttp backend ---> MongoDB

Or just
browser --> Python aiohttp server --> static files (next.js export)
                                  --> MongoDB

The deployment could be easier then.
The only limitation is that we don't want to have to deal with cross-origin requests, so both frontend/static files and backend should be on the same hostname.
Proof of concept: https://github.com/messa/aiohttp-nextjs-demo-chat

BTW. similar issue for python.cz: https://github.com/pyvec/python.cz/issues/263

So far I think that SSR improves UX massively and making the website static - having to load all dynamic content via AJAX after the static HTML is loaded - is not an optimal approach for such application. Right now the only such dynamically loaded page is the task submit page and there are already some UX issues with it.