Add ROAS plot

Add a method to compute the return of ad spend for certain channels and plot it. The plot could follow the style of figure 3 from Jin et al 2017, which I'll copy down here just as a reference.

Challenge: We need to do a forward pass to get these results.

Could the "channel_contributions" Deterministic be used for this?
For instance,
# (chain, draw, date, channel)
mmm.fit_result["channel_contribution"] / mmm.l

Have any thoughts on the difficulty of this @juanitorduz?

Yes! We just need to be careful with the adstock contribution (similar as you did with the out-of-sample feature ;) )

Yes! We just need to be careful with the adstock contribution (similar as you did with the out-of-sample feature ;) )

Would this affect more than the first l_max time periods?


Actually exposing memehoovy

What is your question?


eslint-module-utils: filePath in parserOptions

Refs https://github.com/benmosher/eslint-plugin-import/issues/839

Coverage increased (+0.003%) to 95.062% when pulling b4d75c80c5eba883b9e2314094e253a4e70a06f2 on sompylasar:patch-2 into 1377f55bc3d003519a74a89b048e11b20d53990e on benmosher:master.

@ljharb OK, will add, what specifically? Assert the path to be actually passed?

@sompylasar that's ideal, yes

@ljharb

I need a spy to intercept parser call in the test. I've implemented a naive one in 6 lines, but if you'd like to import a library like sinon or chai-spies instead, let me know.
I've bumped the eslint-module-utils version to 2.1.0, added lines to both CHANGELOGs, and updated the eslint-plugin-import dependency version of eslint-module-utils.

Now CI fail with No compatible version found<EMAIL_ADDRESS>how do you update these versions? @ljharb

It'd have to be npm published - but that won't happen until after it's merged.

@ljharb Reverted: https://github.com/benmosher/eslint-plugin-import/pull/840/commits/7712ce132ec38e38e0c756695efd10df7c2c3eeb

I'd probably bring in sinon (more deps is a good thing), but this is fine for now.

👍 I'm for sinon, too. I'd also use chai assert instead of expect (too much magic), but it was already there, so consistency...

Coverage increased (+0.05%) to 95.111% when pulling 7712ce132ec38e38e0c756695efd10df7c2c3eeb on sompylasar:patch-2 into 1377f55bc3d003519a74a89b048e11b20d53990e on benmosher:master.

@benmosher Updated, PTAL. Replaced hand-made spy with sinon.spy.

Coverage decreased (-1.2%) to 93.816% when pulling 7ac5e8f24fe23c726e350168faa9d64380c0f26c on sompylasar:patch-2 into 2f690b4388552f36b07541c99cb82cab631f62cc on benmosher:master.

Coverage increased (+0.04%) to 95.097% when pulling 7ac5e8f24fe23c726e350168faa9d64380c0f26c on sompylasar:patch-2 into 2f690b4388552f36b07541c99cb82cab631f62cc on benmosher:master.

Coverage increased (+0.07%) to 95.125% when pulling d397b9b7ff6a2cc0c800cfeea3ad28427f9b05ec on sompylasar:patch-2 into 2f690b4388552f36b07541c99cb82cab631f62cc on benmosher:master.

Coverage decreased (-1.2%) to 93.857% when pulling 57327424f0a3fb3c37e7e8c8199f6f8558d8e837 on sompylasar:patch-2 into 2f690b4388552f36b07541c99cb82cab631f62cc on benmosher:master.


Netty 4.1.34 and Linux kernel 5.0.x don’t play nice

I’m not confident in the exact culprit for this issue, but wanted to add it here in case someone else hits it and decides to Google.
There’s something wrong with netty and kernels 5.0 through 5.0.3 (haven’t tested 5.0.4 or 5.0.5 but the Changelogs don’t seem applicable).
I’m not sure if this is a JDK issue but I tried OpenJDK 11 and 12.
Basically, on 5.0.x I see a lot of extra WRITABILITY_CHANGED events in the netty debug output that don’t happen on any older kernel version (I tested the latest 4.14.x through 4.20.x).
Along with the extra events, the data netty writes to the socket doesn’t seem to be fully flushed or something, and the other end of the socket will time out thinking it has to read more data.
Unfortunately as this happened in production on a canary deployment it’s hard for me to test it again without annoying users. Maybe 5.0.x has some changes to epoll the JDK isn’t aware of?

Can you show me how you bootstrap your channel ?

Am 02.04.2019 um 02:22 schrieb Omar Kilani<EMAIL_ADDRESS>I’m not confident in the exact culprit for this issue, but wanted to add it here in case someone else hits it and decides to Google.
There’s something wrong with netty and kernels 5.0 through 5.0.3 (haven’t tested 5.0.4 or 5.0.5 but the Changelogs don’t seem applicable).
I’m not sure if this is a JDK issue but I tried OpenJDK 11 and 12.
Basically, on 5.0.x I see a lot of extra WRITABILITY_CHANGED events in the netty debug output that don’t happen on any older kernel version (I tested the latest 4.14.x through 4.20.x).
Along with the extra events, the data netty writes to the socket doesn’t seem to be fully flushed or something, and the other end of the socket will time out thinking it has to read more data.
Unfortunately as this happened in production on a canary deployment it’s hard for me to test it again without annoying users. Maybe 5.0.x has some changes to epoll the JDK isn’t aware of?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi Norman,
Sure:
https://github.com/unfiltered/unfiltered/blob/0.10.x/netty-server/src/main/scala/Server.scala#L110

Can you try to use EpollServerSocketChannel and EpollEventLoopGroup and let me know how it works out ?

Am 02.04.2019 um 07:08 schrieb Omar Kilani<EMAIL_ADDRESS>Hi Norman,
Sure:
https://github.com/unfiltered/unfiltered/blob/0.10.x/netty-server/src/main/scala/Server.scala#L110
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Will do.
I added support for native transports to unfiltered:
https://github.com/unfiltered/unfiltered/pull/440
Will test it in some real apps soon.

Hi Norman,
Amazingly, I was able to repro this with a smallish unfiltered example.
One interesting thing to note is that I could repro it with HTTP keepalive off. With it on it doesn't trigger whatever the issue is.
Anyway, with native transports it works either way. So I'm not sure what that says. Are you interested in the repro case if it uses scala/unfiltered, or should I work on a "netty only" Java example?
Thanks!

Just for my own notes, this is still broken in 5.0.5. I'm going to test on 5.0.6.
Is it fair to say that if it works with native transports we can rule out an issue in netty and it's more likely some JDK issue?

@omarkilani yes I would say so.

@omarkilani any update ?

@carl-mastrangelo @ejona86 @vkostyukov did you hear of anything like this in your env ?

I had to pause my research into this but 5.0.6 made the problem much more frequent.
I’ll test 5.0.7 today.  It’s a very weird sort of issue, but I did manage to pcap it in production so I’m trying to study the wireshark output.
Happy to share that with you once I make sure there’s no customer data in it.

@omarkilani would be interested

I've not seen anything similar to this in our environment, but we aren't running Linux 5.0 and we aren't running OpenJDK 11... I am aware of one binary that would be using OpenJDK 11+grpc, but it still wouldn't be running on Linux 5.0.

@omarkilani I just talked with @franz1981 and he mentioned that maybe it is related to edge vs level triggered. NIO uses level triggered while our native epoll transport uses edge triggered by default. Could you try to switch to level triggered for epoll and see if you see the same effect ? Would be interesting to see the outcome :)
bs.option(EpollChannelOptions.EPOLL_MODE, EpollMode.LEVEL_TRIGGERED)
and if its a ServerBootstrap also:
bs.childOption(EpollChannelOptions.EPOLL_MODE, EpollMode.LEVEL_TRIGGERED)

I tested 5.0.7 and it's still broken. However, I've narrowed the issue down to 7 kernel commits.
So I'm going to revert each one and recompile and retest.

Nice! Keep us posted

Am 14.04.2019 um 04:05 schrieb Omar Kilani<EMAIL_ADDRESS>I tested 5.0.7 and it's still broken. However, I've narrowed the issue down to 7 kernel commits.
So I'm going to revert each one and recompile and retest.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Just an update on this. I was able to make nginx fail in a similar way using epoll, so I’m going to close this.
My hunch on the cause is this commit (or the ones surrounding it, but they’re all interdependent so it’s not really possible to revert just one):
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v5.1-rc5&id=c5a282e9635e9c7382821565083db5d260085e3e
The issue is hard to reproduce because you need to have a lot of epoll usage on the system to trigger whatever it is. So just running the jvm, you need a bunch of CPUs and a load tester with a lot of clients, etc.
It’s easier when you run more things that use epoll. So I chained:
nginx - haproxy - jvm/netty
Basically just proxying the request but generating a lot of epoll events in the kernel. With this chain nginx fails after about 1000 requests from 100 concurrent clients (the error message references epoll_wait, but I had to stop my testing for today so I forget the exact message. Since it’s not netty related it doesn’t matter I guess).
Weirdly enough, I can never get haproxy to fail. The jvm fails in epoll after something like 9,000,000 requests if load tested directly.
So, however haproxy uses epoll doesn’t trigger whatever the issue is.
Anyway, this thing is annoying me a lot, but every time I need to test it I need to use a VM with 64+ cores which costs a bunch of money. So I’m not sure I can keep chasing it. Hopefully someone else hits it and hasn’t more bandwidth. It’s one of those things that’s hard to believe is happening but there’s definitely something happening, I swear. :)


FAT-846 - List records from marc21 prefix, from and resumption token …

Purpose
FAT-846 - Karate tests: List records in marc21 format with starting date

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


ResourceQuota Enhancement

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:
https://github.com/harvester/harvester/issues/3124
Solution:
https://github.com/chrisho/harvester/blob/maintenance-resource-hep/enhancements/20230228-resource-quota-enhancement.md
Related Issue:
https://github.com/harvester/harvester/issues/3124
Test plan:
https://github.com/chrisho/harvester/blob/maintenance-resource-hep/enhancements/20230228-resource-quota-enhancement.md#test-plan

Re-design


Adds route list_routes to support performance testing

@FBRTMaka @lukecampbell I tried to be all fancy and add some logic to support running one test_*.py file at a time, but I am running into the error seen in the travis output.  Any ideas?


Update variables.css to make consistent layout

Updated according to description
Checklist:

[x ] I have read and followed the contribution guidelines.
[ x] I have read and followed the how to open a pull request guide.
[ x] My pull request targets the main branch of freeCodeCamp.
[x ] I have tested these changes either locally on my machine, or GitPod.

Closes issue #53656
I have edited the variable as requested

Hey @yoesk8
Thank you for opening this pull request.
As there was earlier PR - #53664 - making all necessary changes, merged recently, we are going to close this one.
Thank you and happy coding! 😊

If you have any questions, feel free to ask questions on the 'Contributors' category on our forum or the contributors chat room.


[Feature request] Default update strategy

Hello.
If I don't specify the GVK of the children, then the default policy is OnDelete, which means I can't change the state of the created children. This makes it difficult to make more dynamic controllers.
I need a function that allows you to specify a default strategy for children, without explicitly specifying their list. This will allow you to make much more dynamic code.
I created a PR where this is implemented. If no default policy is specified, then the current algorithm remains virtually unchanged. This gives full backwards compatibility.
https://github.com/metacontroller/metacontroller/pull/662

hi @zefir01, GVK of children is required, not sure what you want to achieve. Please elaborate on that with some use case

I want to be able to make more complex controllers where it is not known in advance which children will be created.
Now I'm making a framework that makes it more convenient to work with crossplane. and I need this feature. the main idea is to make it as simple as teraform by taking care of all the CRD and controller work.

Close as no plan to implement


feat(icon): add static to css folder icon

Changes proposed:

[x] Add
[ ] Delete
[ ] Fix
[ ] Prepare

Folders named static are usually used for storing CSS files of project.

HI @abhayhk2001, unfortunately I will have to close your PR. static is not only used for css. I've seen static folders being used for html, images and all other kind of assets.
I'd suggest you to use the customization feature.


Allow arbitrary expressions in loop bounds

Currently we have Integer there. We should allow arbitrary Expr, but keep the semantics: the compiler will throw an error if such an Expr doesn't evaluate to a constant statically.

After fixing #54 by merging #77 we could provide a straightforward implementation using #76 and requiring that loop bounds are typeable without any external dependencies.


Nuxt CodeSandBox Examples - Moved?

Version
v2.11.0
Reproduction link
https://codesandbox.io/s/github/nuxt/nuxt.js/tree/dev/examples/hello-world?from-embed
Steps to reproduce
1 - Navigate to the 'Examples' section of Nuxt
2 - Choose any example
3 - CodeSandBox fails to load
Have the Examples been moved but the CodeSandBox hasn't been updated?
What is expected ?
See a Live Example of some of the functions/features of Nuxt
What is actually happening?
No CodeSandBox is found

This bug report is available on Nuxt community (#c10348)

They seem to be back now...
Sorry!


PS-3299 batch file input to abs workspace

Increase speed of loading to abs
https://keboola.atlassian.net/browse/PS-3299

@pivnicek this needs either to be closed or moved to monorepo


Home World of first Character you log in for the day

Instead of showing home world of current character it shows home world of the first character you logged in when you launched the client. Home world does not change when you log in a character on another world.
Here it shows Home (Phantom) even tho this characters home world is Sagittarius

I hadn't considered that, I cache home world information when first logging in, but it should be cleared when changing characters


Add CI support

As discussed in #20 this adds support for travis-ci builds based on arch-travis.
I have added make install for wlc as suggested by @Luminarys

LGTM. :+1:


tortoise: unexpected behavior when encoding abstain votes

 {"L":"ERROR","T":"2022-07-14T03:52:29.007-0400","N":"fce93.trtl         ","M":"layers that are not terminated should have been encoded earlier","node_id":"fce93f3b8757a66bc1f0a58683a8819aab6345e577f1ceeec0c79ccd5ec54487","module":"trtl","sessionId":"b5089a86-2a87-4607-97d3-62ec37bfcf4c","base_layer":"11","last_layer":"80","block_layer":13,"block":"d664d9f0cc","local_vote_reason":"hare","local_vote":"abstain","block_id":"d664d9f0cc","layer_id":13,"name":"trtl"}

it happens when node didn't make progress for longer than zdist

i think it was fixed in my latest refactoring, will reopen if i see it again


Microsoft.WindowsAzure.MobileServices Language Locale Setting

Hi All
I’m using the Microsoft.WindowsAzure.MobileServices MobileServiceClient together with AAD B2C Authentication.
the LoginAsyn does not support setting the language Locale.
Problem Description.docx
I need this to be solved.
Can anyone help me on this?
Many thanks in advance.
René

@modulus37 Thanks for using Azure Mobile Apps. As we talked over mail, we don’t support setting the locale this way.
Please refer to the SDK readme that highlights the future of Azure Mobile Apps. We are not adding new features to the product. Please let me know if you have any other questions.


Support converting options to enums (case-insensitive)

Please include the following with each issue:
1. Describe the bug
In our process of moving away from options to enums, we sometimes have option field that have all the same option values, except for some casing differences. Changing from option to enum seems only supported if the values match 100%, including case matching. Question is to remove the casing restriction.
References to field names / enum values seem to be case-insensitive, so it would make sense to apply the same for options to enum conversion too.
2. To Reproduce
Steps to reproduce the behavior:
I've a field 'Contact Type' with optionmembers : To, CC, BCC.

I want to replace the option by the standard enum "Email Recipient Type", with values : To, Cc, Bcc

The AL compiler allows to reference an enum value using both BCC and Bcc (so case-insensitive)

However, the compiler sees the change from option to enum as non-supported

3. Expected behavior
Please allow changing option fields to enum as long as the list of optionmembers matches the enum values, irrespective their casing.
5. Versions:
Name: AL Language
Id: ms-dynamics-smb.al
Description: AL development tools for Dynamics 365 Business Central
Version: 9.0.542050
Publisher: Microsoft
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=ms-dynamics-smb.al

@fvet - The rule is not case sensitive. The problem here is that the enum in the System Application defines "Bcc" with the ID 3, while it had 2 as ordinal value in your field definition. This is why you get an error from the AppSourceCop mentioning that an enum with value 2 is missing now.
The enum in the system app looks like this:
enum 8901 "Email Recipient Type"
{
    value(0; "To") { }
    value(1; "Cc") { }
    value(3; "Bcc") {  }
}

From the AppSourceCop's point of view, this is then as expected


dev: add ENVOY_DOCKER_SKIP_PULL to skip docker pull

Signed-off-by: spacewander<EMAIL_ADDRESS>
Commit Message: dev: add ENVOY_DOCKER_SKIP_PULL to skip docker pull
Additional Description: it  saves lots of time when developing with run_envoy_docker.sh wrapper
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

happy to land this - separating the pull part was mostly done so we could see how much CI time was being used to pull the image
im wondering if we should make it the default to not pull separately

If we make it the default, we need to update CI to always pull the latest one, otherwise a stale image may be used.

... we need to update CI to always pull the latest one ...

i think its ok - we dont need to worry about that so much as the build image uses a "unique" hash for its tag (of course it could theoretically change but wont unless something has gone wrong)

Should I update the PR to make the docker pull non-default now, or should we wait for more opinions from the other people?


Clicking Privacy & Security Policy and Terms and Conditions links deletes field data

Describe the bug
At the bottom of the registration page, where the user enters their address & details, clicking the links for both privacy & security policy and Terms and Conditions will delete everything entered in the registration fields.
To Reproduce
Steps to reproduce the behavior:

Get to the registration page
Fill out address fields either manually, or using the GPS location
Click on 'Privacy & Security Policy' or 'Terms and Conditions'
Notice that the fields are cleared

Expected behavior
Data entered by the user shouldnt be cleared by clicking on either links, forcing them to re-enter all information.
Screenshots
Video of the issue here: https://drive.google.com/file/d/1exmr5YEPDA9Seswm90G9ahAyxbRsI2MQ/view?usp=sharing
Smartphone:

OS: iOS
Browser: Safari
Version: 13.1

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.

I have pushed a fix for this

Current Behaviour is as expected. Regression testing is done. Clicking on 'Privacy and Security Policy' and 'Terms and Conditions' doesn't delete the information entered for registration


Honeycomb not rendering in older browsers

The viz of the progress of the different project look wired in my browser (Safari).

This is due to lack of full CSS3 support. We should fallback to rendering a grid or even only the list view.

The ideal solution here would be to have CSS 2-specific rendering. I haven't revisited this in a while, but just a couple of bullet points on this:

The grid renders fine in Safari on an old iPhone 5 which is almost 10 years old, and I haven't heard about this complaint from other users.
I'm revamping the interface of dribdat in the PWA https://github.com/dribdat/backboard project where we can do more for retro devices.
The new search interface in the about page (which I want to soon make more prominent), profile and resources areas, have a simpler grid for project display. If there is interest, I will make it an option for the event grid, too.

In other words, if this is still an issue, please feel free to re-open!


Users getting Welcome Name you have an account on Farm. Enjoy! even though they do not

Screenshots attached for two different users reporting they are getting the Welcome Name you have an account on Farm. Enjoy! even though they do not have an account though hippo at all. One Login ID is agphinne and the other is clmperez.
At this moment in time, one of those users has submitted an account request, and the other is expected to do so soon.

I'll circle back to this another time, but what I could find was that these users were erroneously routed from the home component to // instead of //create because their status was already set as such in the db somehow

I have suspicions about where the problem lies, but want to wait and see if another user or I can reproduce this error*


Refactor WebPage Sample

Now implementation showcases standalone dependent resources and classic version. Add e2e tests later in a new PR.

LGTM. Should we also use the "classic" mode in the tests?

The plan is to use both, in E2E tests (so 2 tests). Not sure yet how. Bit will manage it somehow.


bump  identity-workflow-impl-bps Public version

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2739509635

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2739509635
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2740896991

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2740896991
Status: failure


Applicability of RDT-1B on UR5 Robotic Arm and Fine-Tuning Requirements

Hi,
First, congratulations on the excellent work with RDT-1B! 🥳 Your approach to diffusion-based bimanual manipulation and the introduction of a unified action space are both impressive and inspiring.
I am interested in adapting your method for use on a different robotic setup, specifically a UR5 robotic arm. I have a few questions regarding this adaptation:

Compatibility: Given the unified action space you've introduced, would it be feasible to apply RDT-1B on a UR5 arm setup? Are there specific adjustments or considerations you would recommend?

Fine-Tuning Requirements: Assuming additional fine-tuning is needed for optimal performance on UR5, could you provide guidance on the approximate data volume and computational resources (e.g., GPU requirements) necessary for fine-tuning?

Any insights or recommendations would be greatly appreciated. Thank you again for your work on this exciting project! 🙏

Thanks for your attention!
Compatibility: Yes, it is compatible. No adjustment is needed. You can control a UR5 with either a joint-based or EEF-based controller.
Fine-Tuning Requirements: At least 24GB VRAM.


Value of anomaly percent does not change according to the labelled curves

The Anomaly percent does not change.

Hello, I have repaired, please verify

Thanks. Please take a look at the new issue, the latest code makes web interface inaccessable.

Hello, this problem is being repaired, thank you


QRX fixes and added functionality

Issue #, if available:
Description of changes:

_model.py:
1.A. Fixed the issue of the last cell being to small by making the last cell be itself merged with the penultimate cell. This improves performance significantly.
1.B. Added model_is_already_trained to fit function. If True, don't fit the underlying model. This allows one to apply QRX to pre-trained models.
_predictor.py: Added train_QRX_only_using_timestep. If it is -1, nothing changes. Otherwise, it should be an integer between 0 and forecast horizon - 1. In that case, instead of creating a different partition for each timestep in the forecast horizon (and therefore train a different underlying model for each timestep in the forecast horizon), it will only train one underlying model (and therefore have the same partition for all timesteps) based on the the target from the train_QRX_only_using_timestep^{th} timestep. This is an experimental option, which cuts down on training time significantly, while degrading performance in general. On the other hand, it allows you to use more context windows to create the partition.
Changed the name clump_size to min_bin_size, which is more descriptive.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

marked this as breaking to prevent it from ending up in a previous release branch


GUACAMOLE-302: Incorrect password field focus

Note: This is not specific to just password fields, but any field type in general.

Update: Added the two pieces of documentation requested above.

Update: Fixed the default value of $scope.relevantField = {}; as well as the documentation.

Update: Changes ready for review.

Update: Unneeded whitespaces removed as well the extra guac-focus. Also the documentation is altered as per your suggestion. Ready for review.


Wrong autocomplete for predicates

Environment data
dotnet --info output:
Программы командной строки .NET (2.1.4)

Product Information:
 Version:            2.1.4
 Commit SHA-1 hash:  5e8add2190

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.13
 OS Platform: Darwin
 RID:         osx.10.12-x64
 Base Path:   /usr/local/share/dotnet/sdk/2.1.4/

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.5
  Build    : 17373eb129b3b05aa18ece963f8795d65ef8ea54

VS Code version:
C# Extension version:
1.13.0
Steps to reproduce
Type ( in collections and then letter c.
Expected  behavior
No intellisense suggestions
Actual behavior
Autocomplition for c = char. And if i type "c=>c." it will complete with "c=>char". How to disable that?

Could not reproduce this behavior with the current release.


Edit: AMC (Afghanistan)

Channel ID (required)
AMC.af
Channel Name
No response
Alternative Names
No response
Network
No response
Owners
No response
Country
No response
Subdivision
No response
City
No response
Broadcast Area
No response
Languages
No response
Categories
No response
NSFW
None
Launched
No response
Closed
No response
Replaced By
No response
Website
No response
Logo
https://play-lh.googleusercontent.com/MYcrmTg-2FfK3mVvNsjttNsXXTLfUG7i3QjMd0fzticooxMwLMK0NQdyQSMnF0iC9lg
Notes
The logo for the channel is recent, it should be replaced with the current one.

@freearhey
Ok, I have updated the request.


*: save metadata of regions to leveldb

What problem does this PR solve?
This Version is used for testing first, not completed.
What is changed and how it works?
Check List
Tests

Unit test
Manual test (add detailed scripts or steps below)

@siddontang  we need load regions from regions to speed up recover router of the regions.
PTAL @disksing @rleungx @liukun4515

/rebuild

PTAL @disksing @rleungx @liukun4515


fix: wrapping and height calculations

Changes
Content used to cut off if the height of the window was smaller than the content height.
Wrapped content broke formatting and caused content to get cut off.
The offset calculations were off and wouldn't scroll to show hidden content.
There was an off by one error for content that exceeded the window height (would always hide the title of the first field).
Other considerations
Some fieds (confirm, select, input) have the option to be shown as inline. When shown as inline, the styles do not wrap (intentional)
Buttons will stack vertically if they don't fit horizontally. Button widths will also match (do we want that by default or only when stacked?)
Fixes
https://github.com/charmbracelet/huh/issues/398
https://github.com/charmbracelet/huh/issues/429
Maybe others (need to confirm)
Here's what this PR looks like:
https://github.com/user-attachments/assets/b3588782-33d4-4e11-8105-2f6b7335df6a
Previous behaviour:
https://github.com/user-attachments/assets/0e77808e-5895-45d4-b075-f299babecf2b
A few enhancements I'd still like to see: huh does a lot of redraws that cause rendering issues when not in alt screen. You come across that behaviour when resizing the window a lot while running an example (e.g. burger)

feat: set group with from size msg
feat: handle WindowSizeMsg + base height on rendered content
feat: render prefix before newlines
feat: update viewport on resize, cursor != height so no scroll...
fix: always update viewport height
fix: add option prefix
fix(multiselect): width calculation
fix: calculate offset from previous field heights
feat: add calculate wrapping helper
fix(select): wrap select field
feat: make group set field widths accounting for styles
feat: set width
feat: stack buttons in narrow window
feat: add Theme function to Fields
feat: calculate width based on field theme
fix(group): fix height calculation
feat: remove FullWidth helper (not needed)
fix(group): fix off by one at top for overflowed content

In case we don't want to merge the stacked buttons here, I have another version of this branch where I removed those features. https://github.com/charmbracelet/huh/tree/horizontal-btns

rip, looks like these changes might be breaking grid layout...

In the image above we're missing options 4-6. This is happening at all widths. Absolutely tragic 😭
Build info:
macOS, kitty, no multiplexer, just running the layouts/grid example


Authentication failure on rails api

Dear,
Could you please give me a some clue that can't authenticate because of InvalidAuthenticityToken problem?
Rails: 6.0.0 (installed api only mode)
ominiauth-azure-activedirecotry-v2 : 1.0.0
I'm following the article, but I got below issue:
E, [2022-04-13T13:10:55.623352 #24682] ERROR -- omniauth: (azure_activedirectory_v2) Authentication failure! ActionController::InvalidAuthenticityToken: ActionController::InvalidAuthenticityToken, ActionController::InvalidAuthenticityToken
So I tried to skip_before_action :verify_authenticity_token, but there's another issue that rails-api don't have this method by default.
In this case, how can I fix and go ahead?
Best,

It was correct - after I changed from ActionController::API to ActionController::Base on application_controller.rb, everything works well.
class ApplicationController < ActionController::Base

Glad you got it sorted.
It doesn't make sense to try and use Azure AD on an API-only application since the user must have a front-end UI to interact with in order to be able to use the Azure AD

Glad you got it sorted :-)


[Feature] InjectorService, add manually a service (allready constructed).

InjectorService can use a service built manually and/or not typed.
Here's how it works:
// fooservice.ts
import {InjectorService} from "ts-express-decorators";

export interface IMyFooService {
   getFoo(): string;
}

export type MyFooService = IMyFooService;
export const MyFooService = Function;

InjectorService.factory(MyFooService, {
     getFoo:  () => "test"
});

// otherservice.ts
import {MyFooService} from "./fooservice.ts";

@Service()
export default class OtherService {
     constructor(myFooService: MyFooService){
           console.log(myFooService.getFoo()); /// "test"
     }
}

This feature will be released for the v1.4.0-1 (beta) (soon)

This example is still valid and working?
I tried it and got:

Process finished with exit code 0

I don't know how to set debugging mode (And I would be grateful if you advice) so I dived into the module to find this error being thrown:

TypeError: _this.expressApplication.use is not a function
at /home/mouneer/djaty/djaty-server/node_modules/ts-express-decorators/lib/services/controller.js:136:50
at Array.forEach (native)

I use

"ts-express-decorators": "^1.4.0-20",

Any thoughts?

Hello @m0uneer,
Thanks for your issue because you found a big fail with the InjectorService.
Effectively the example does not work. I've only test with ExpressApplication as Factory but I never test with other factory. My mistake.
So I've fix the problem with the 1.4.0-21 and the right example is the following:
// fooservice.ts
import {InjectorService} from "ts-express-decorators";

export interface IMyFooService {
   getFoo(): string;
}

export type MyFooService = IMyFooService;
export const MyFooService = Symbol("MyFooService");

InjectorService.factory(MyFooService, {
     getFoo:  () => "test"
});

// otherservice.ts
import {MyFooService} from "./fooservice.ts";

@Service()
export default class OtherService {
     constructor(@Inject(MyFooService) myFooService: MyFooService){
           console.log(myFooService.getFoo()); /// "test"
     }
}

Note: For a factory you must use the Inject decorator. I haven't found a solution without decorator :/

Great, this works. Thanks a lot.
Regarding the Inject decorator, why don't you read the function string, parse it then inject the required dependency?


Add KMP to JS modules

Describe the suggestion:

[x] Please add the KMP to JS modules

Additional context
Add any other context about the problem here.
@behzodhalil

public actual fun info(message: String, tag: String) {}

If there are any specific dependencies that the module requires, please inform me @Sardor-M!


Darwin Support

Would https://github.com/spacemonkeygo/monotime/blob/master/mono_linux_amd64.s also work on darwin/amd64?

No, that's a Linux-specific system call


Update pyrainbird

Description:
Updates the pyrainbird library to 0.1.6 which just fixes a bug where pycryptodome was missing from the install_requires field causing issues if you didn't already have it installed.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Did you test this? Does pycryptodome work fine for you?

Yeah, i've installed it locally on my home-assistant instance (by doing pip install --upgrade pyrainbird==0.1.6 in a new venv after setting up hass again) and ran through a full manual test before I made the PR.

Successor of #14588.

I assume this might be a good place to start looking for help with this component? Point me elsewhere if not.
I've just update to Hassio .71 with the hopes that rainbird would start working again. Unfortunately it seems not. I get the following in my HA logs on boot. Any help would be appreciated. I'll add that I've just started using HA in the last 2 months and I'm far from a unix pro....
The errors reported are essentially the same seen before updating.
Timer got out of sync. Resetting
7:11 PM core.py (ERROR)
Unable to prepare setup for platform switch.rainbird: Could not setup all dependencies.
7:11 PM setup.py (ERROR)
Unable to setup dependencies of switch.rainbird. Setup failed for dependencies: rainbird
7:11 PM setup.py (ERROR)
Unable to prepare setup for platform sensor.rainbird: Could not setup all dependencies.
7:11 PM setup.py (ERROR)
Unable to setup dependencies of sensor.rainbird. Setup failed for dependencies: rainbird
7:11 PM setup.py (ERROR)
Setup failed for rainbird: Component failed to initialize.
7:11 PM setup.py (ERROR)
Error getting state. Possible configuration issues
7:11 PM components/rainbird.py (ERROR)
Request resulted in no response
7:11 PM components/rainbird.py (WARNING)
Setup of rainbird is taking over 10 seconds.
7:10 PM bootstrap.py (WARNING)
Setup of config is taking over 10 seconds.
7:10 PM bootstrap.py (WARNING)

No this is not the right place for support. Please use the forum or chat.
https://www.home-assistant.io/help/#communication-channels
If you suspect a bug, please file an issue here at github.


Intellisense deletes number before identifier

Steps to reproduce, with Intellisense enabled:

Type a_0 = 1
Type 2 a in a new expression.
With the cursor after the a, press Enter to fill in the intellisense replacement
The latex is now a_0 instead of 2 a_0.

https://github.com/user-attachments/assets/38c60ace-23f1-43bb-9e02-6a45879b9a62

This only occurs when auto-subscriptify is enabled. It also occurs when just typing left-to-right, without pressing Enter to fill in the replacement.


ObjectManipulator with offset origin (HostTransform) causes MinMaxScaleConstraint to behave oddly

Describe the bug
When ObjectManipulator is configured with a HostTransform with an offset origin, MinMaxScaleConstraint gets very confused and will make the object drift in the direction of the offset origin when the min/max scale limit is hit.
To reproduce
Steps to reproduce the behavior:

Make empty GameObject
Make child object, offset by some amount
Put ObjManip on child object, make HostTransform parent
Add MinMaxScaleConstraint
Scale object with hands; observe weird drifting

Expected behavior
MinMaxScaleConstraint should behave the same with or without a HostTransform offset
Notes
Impacting internal customer.

ISSUE MIGRATION
Issue migrated from: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/10954

Original comment by:
 Zee2

@JonathanPalmerGD


can't run dora -help

      My ubuntu version is aarch 18.04 ,and whern i  install dora-rsv0.2.4,i try to run 'dora --help ',but it shows :
             bash: /home/agilex/dora_main/dora: Permission denied
      After i run 'chmod +x /home/agilex/dora_main/dora' ,i run 'dora --help' again ,it shows :
             bash: /home/agilex/dora_main/dora: cannot execute binary file:exec format error
      i had try v0.2.3 and v0.2.4,it shows the same issue;By the way ,my python is 3.7 , my motherboard is Jetson nano.
     Thanks for your reply

Successfully installed dora-rs-0.2.4
agilex@agilex:~/dora-v0.2.4-aarch64-Linux$ PATH=$PATH:$(pwd)
agilex@agilex:~/dora-v0.2.4-aarch64-Linux$ dora --help
bash: /home/agilex/dora_main/dora: cannot execute binary file: Exec format error
agilex@agilex:~/dora-v0.2.4-aarch64-Linux$ dora --version
bash: /home/agilex/dora_main/dora: cannot execute binary file: Exec format error
agilex@agilex:~/dora-v0.2.4-aarch64-Linux$ dora up
bash: /home/agilex/dora_main/dora: cannot execute binary file: Exec format error

HI Latitude9527, did you directly unzip these dora binary commands and set them to the environment variable PATH?
https://github.com/dora-rs/dora/releases/download/v0.2.4/dora-v0.2.4-aarch64-Linux.zip

Thanks ,I have replaced the latest installation package, and now it can run successfully

Nice. make persistent efforts 💪💪💪.

Thanks ,I have replaced the latest installation package, and now it can run successfully

Nice. make persistent efforts 💪💪💪.


Update surefire and failsafe plugins to 2.20

Motivation
Maven 3.5.0 was released a few days ago and the current versions of the surefire and failsafe plugins no longer work.
https://maven.apache.org/docs/3.5.0/release-notes.html
Modifications
Update surefire and failsafe plugins to 2.20
Result
Fixes issue #6666 and it's possible to build Netty with Maven 3.5.0
Tested versions:
$ mvn -version
Apache Maven 3.3.9 (NON-CANONICAL_2015-11-23T13:17:27+03:00_root; 2015-11-23T05:17:27-05:00)
Maven home: /opt/maven
Java version: 1.8.0_121, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-openjdk/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "4.10.11-1-arch", arch: "amd64", family: "unix"
$ mvn -version
Apache Maven 3.5.0 (NON-CANONICAL_2017-04-10T13:56:20+03:00_root; 2017-04-10T06:56:20-04:00)
Maven home: /opt/maven
Java version: 1.8.0_121, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-openjdk/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "4.10.11-1-arch", arch: "amd64", family: "unix"

Actually, doesn't fix it - closing...


How to get/set "assigned to" field?

Hi,
how to get/set native field "assigned to"? What is ID for this field?
Thank you.

Hi. assigned_to_id

Hi,
it's look like doesn't work. I created custom fields with User type. The field ID is 44.
#ID current field is 44
if (cfs[44].blank?)
User.current.id
else
cfs[44]
end
Previous code work perfectly if there is no below part of the code:
Tracker ID for tracker "Request for vacation" - 12
ID customer to process request for vacation
if (tracker_id == 12)
assigned_to_id = 28
else
cfs[44]
end
After the second part of the code field cfs[44] filled by value 28, which I try to assign to "Assigned to" field.
I would like to make dynamic issue assigning.

If you would like to make dynamic issue assigning, you should find another plugin or create your own http://www.redmine.org/projects/redmine/wiki/Plugin_Tutorial

I don't understand. "Assign to" field is read-only? Why I can't programmatically change "Assign to " field?

You could, but this plugin made only to calculate custom field values.

Can I change the field "Assigned to" from a script in the plugin "computed custom field"?

Unfortunately, you can't.

That's Redmine API restrictions, or the plugin doesn't afford this?

Can I set a category from the plugin script?

Maybe I was wrong, you could try self.assigned_to_id = 28, but if it would works it would be side effect of computation of custom field value.

Hi,
self.assigned_to_id = 28 -  it works. Thank you.


trying to understand use cases & a feature request

It's pretty clear why I'd use create_and_open. But use case of rest two of functions aren't much clear to me

open_under_cursor : maybe useful when re-editing the image.
search_and_open : don't know how to use this, probably same as re-editing the image. Instead

search_and_open would have been useful if would be able to duplicate the figure and create a new figure. Not sure if telescope supports some callback i.e. the selected figure copied to new path and opened in inkscape; code for figure is either inserted or copied for pasting when a particular key-combination is pressed in picker.
I've tried to replicate your code for my use case - I use inkscape to create figures. nvim with plug. Did spend fair bit of time figuring out how to make it work - new to entire ecosystem. I did remove all ai related code, and refactored it to create a figure directory as sibling to open file.

Hello,
Thanks for sharing your thoughts. I would usually write LaTeX documents or presentations with Beamer and sometimes, as I iterate over the documents I need to open a particular figure to edit. I found that most of the times, the most convenient way is to get my cursor under the figure I want to edit, and then hit io to open it and edit. The other way is to just search through the illustrations to open and edit. These are meant for when you want to edit an illustration.
Making a copy of a particular illustration is an interesting idea and I can look into implementing it! I do find myself lately going to the terminal to copy an svg to another one.
What's your use case?
Are you using vimplug as your plugin manager? If so, I am happy to look into supporting it. A little busy this week, but I can look into it this weekend.

Thanks, can you share the modifications for running this with vimplug? If trivial, I may be able to push an implementation sooner for vimplug.
Regarding dimensions, how often do you need new dimensions? I was thinking initially of having a list of “templates” SVGs (you can choose the dimensions there) and then illustrate.nvim in the future would allow you to create a new SVG from a selected template. So if you tend to have 3-4 different dimensions for your SVGs you will have 3–4 templates and then on creation you will pick one.

Hello @anandkumar89,
I have a working version of the copy-and-open functionality, as well as changes to get the plugin working with vim-plug out of the box.
Can I ask if you could give it a test to see if it works on your end? You can find instructions of how to install it with vim-plug in the readme file.
Also note the new function illustrate_finder.search_create_copy_and_open which will first open search to find an existing file to copy, then it will ask for a new name, and then it will open the new copy in the default app.
It will be good to test this with my copy using 'rpapallas/illustrate.nvim' to see if it will work out of the box, as show below:
local vim = vim
local Plug = vim.fn['plug#']

vim.call('plug#begin')
    Plug 'rpapallas/illustrate.nvim'
    Plug 'rcarriga/nvim-notify'
    Plug 'nvim-lua/plenary.nvim'
    Plug 'nvim-telescope/telescope.nvim'
vim.call('plug#end')

local illustrate = require('illustrate')
local illustrate_finder = require('illustrate.finder')
vim.keymap.set('n', '<leader>is', function() illustrate.create_and_open_svg() end, {})
vim.keymap.set('n', '<leader>ia', function() illustrate.create_and_open_ai() end, {})
vim.keymap.set('n', '<leader>io', function() illustrate.open_under_cursor() end, {})
vim.keymap.set('n', '<leader>if', function() illustrate_finder.search_and_open() end, {})
vim.keymap.set('n', '<leader>ic', function() illustrate_finder.search_create_copy_and_open() end, {})

Thanks!

Interesting, that's a good catch. I will look into this.
Did you have success with using the new changes and vimplug out-of-the-box with the latest changes?

Thanks for helping with testing. I created a new issue for the relative paths: https://github.com/rpapallas/illustrate.nvim/issues/3
I am closing this one since it is now done.


scheduled tasks not cleaning up on fatal error

Laravel Version: 5.4.0
PHP Version: 5.6
Database Driver & Version: SQLSERVER 2012

Description:
I tried a pull request but failed the StyleCI test, but I have no clue why.
https://github.com/laravel/framework/pull/19419/commits
Anyways, if a fatal error occurs in the task, the mutex file doesn't get cleaned up if  ->withoutOverlapping() is used when scheduling the task.
By adding https://github.com/laravel/framework/pull/19419/files#diff-dc33b115806d4b2fa1499d3fc46bbd4fR61
register_shutdown_function(function()  {
    $this->removeMutex();
});

at https://github.com/laravel/framework/blob/5.4/src/Illuminate/Console/Scheduling/CallbackEvent.php#L61
the problem is resolved, since it will always clean up the mutex file when the script exists.
Steps To Reproduce:
      $schedule->call(function() {

            $worker = new Worker();
            $worker->workAlready();
        })
        ->name('automatic_email_sending')
        ->withoutOverlapping()
        ->when(function() {

            // weekdays 7:45 - 8:30
            // weekends 8:45 - 18:30
            $now = Carbon::now();

            if($now->dayOfWeek == Carbon::SUNDAY) {
                return false;
            }

            $start = $now->copy()->setTime(7,45, 0);
            $end = $now->copy()->setTime(19,30, 0);

            $random = new Random($now->diffInDays(Carbon::createFromDate(2017,1,1)));

            $int = $random->nextInt(0,45);
            $start->addMinutes($int);

            if ($now->isWeekend()) {
                $start->addHour();
                $end->subHour();
            }

            return $now->between($start, $end);
        });

-- Worker.php
    namespace Tschallacka\Workers;
    /**
      * cause a fatal error, Class Tschallacka\Workers\Carbon not found
      **/
    class Worker {
         public function workAlready() {
              $now = Carbon::now()
         }
    }

Moving the discussion to the opened PR.


Panic on exec.go

Hi, I'm getting a panic in exec.go when I am running the introspection query (I've tried both cURL and GraphQL IDE) and finding it very difficult to debug. Any help/suggestions appreciated. Below you'll find my stack trace. I omitted the full paths of some of my directories for sake of privacy.
Here's the introspection query I'm trying:
query IntrospectionQuery {
    __schema {
      queryType { name }
      mutationType { name }
    }
}

Here's my schema:
package campaigns

// Schema is the schema for campaigns
var Schema = `
  schema {
    query:    Query
    mutation: Mutation
  }

  type Query {
  }

  type Mutation {
    CreateCampaign(campaign: CampaignInput!): Campaign
  }

  type Campaign {
    uuid: ID!
    userUUID: ID!
    packageUUID: ID!
  }

  input CampaignInput {
    userUUID: ID!
    packageUUID: ID!
  }
`

Here's the stack trace of the panic I'm seeing:
goroutine 8 [running]:
...vendor/github.com/neelance/graphql-go/internal/exec.(*request).handlePanic(0xc420013fc0)
	go/src/github.com/spbk/worf/vendor/github.com/neelance/graphql-go/internal/exec/exec.go:310 +0x123
panic(0x4cc180, 0xc4200180a0)
	/usr/local/Cellar/go/1.7.4_2/libexec/src/runtime/panic.go:458 +0x243
.../vendor/github.com/neelance/graphql-go/internal/exec.ExecuteRequest.func1(0xc420013fc0, 0x0, 0x0, 0x7f89e0, 0xc420311e30, 0xc420012980, 0xc420012440, 0x213b00, 0x0, 0x0)
	go/src/github.com/spbk/worf/vendor/github.com/neelance/graphql-go/internal/exec/exec.go:340 +0x68
vendor/github.com/neelance/graphql-go/internal/exec.ExecuteRequest(0x7f89e0, 0xc420311e30, 0xc420012440, 0xc4203ab670, 0x0, 0x0, 0x0, 0x7faf80, 0x835c68, 0x9ad348, ...)
	go/src/github.com/spbk/worf/vendor/github.com/neelance/graphql-go/internal/exec/exec.go:341 +0x26a
vendor/github.com/neelance/graphql-go.(*Schema).Exec(0xc4203ab190, 0x7f89e0, 0xc4203d1aa0, 0xc4201d0c00, 0x5a6, 0x0, 0x0, 0x0, 0x0)
	go/src/github.com/spbk/worf/vendor/github.com/neelance/graphql-go/graphql.go:121 +0x3cd
github.com/spbk/worf/api.(*graphQLHandler).ServeHTTP(0xc42002c120, 0x7f81e0, 0xc4204204e0, 0xc4200d6780)
	go/src/github.com/spbk/worf/api/api.go:60 +0x1a8
github.com/spbk/worf/vendor/github.com/gorilla/mux.(*Router).ServeHTTP(0xc4203a4320, 0x7f81e0, 0xc4204204e0, 0xc4200d6780)
	go/src/github.com/spbk/worf/vendor/github.com/gorilla/mux/mux.go:114 +0x10d
net/http.serverHandler.ServeHTTP(0xc420380280, 0x7f81e0, 0xc4204204e0, 0xc4200d65a0)
	/usr/local/Cellar/go/1.7.4_2/libexec/src/net/http/server.go:2202 +0x7d
net/http.(*conn).serve(0xc420380300, 0x7f8920, 0xc420012780)
	/usr/local/Cellar/go/1.7.4_2/libexec/src/net/http/server.go:1579 +0x4b7
created by net/http.(*Server).Serve
	/usr/local/Cellar/go/1.7.4_2/libexec/src/net/http/server.go:2293 +0x44d

Note, as soon as I remove mutations from my schema, the introspection query works.

I figured it out. My schema didn't have a query type defined and it appears the library can't handle that case (perhaps this is a constraint in the spec).

@robmadden Please open an issue suggesting more descriptive/helpful error messages in this case.

@getaaron no.


RegexNotFoundError

% youtube-dl https://www.youtube.com/watch\?v\=54x43SXNPhI --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.youtube.com/watch?v=54x43SXNPhI', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: 9d0b7361d
[debug] Python version 3.10.12 (CPython) - macOS-13.2.1-arm64-arm-64bit
[debug] exe versions: ffmpeg 6.0, ffprobe 6.0, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] 54x43SXNPhI: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/opt/homebrew/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/opt/homebrew/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/opt/homebrew/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/opt/homebrew/lib/python3.10/site-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/opt/homebrew/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)

ERROR: Unable to extract uploader id

Already reported: #31530
Also read:
#30839 !


how do I use .eslintrc for npm start and browser console linting

I am working on a react project that was created with create-react-app. We are using eslint and have added a few tweaks. However, it appears that the browser console (and npm start) use some other configuration. We are seeing warnings about things that do not matter and are 100% style choices. They are coming from eslint rules, but apparently not using our .eslintrc, as they do not show up when we run eslint against the project. How do I get npm start and the browser console to use .eslintrc.json so I can override those warnings?

You can not override our ESLint configuration -- you must eject to do so. If you found configuration which enforces style, please open an issue and we will consider disabling the rules.

I did figure out how to get the warnings to show up in our linting, but obviously turned off that one rule - as mentioned that still only affects running eslint directly and the warnings still appear in the browser.
react/jsx-equals-spacing is a style choice, is it not? I'm not aware of any potential bugs or errors that come out of that. And I could be mistaken, but it looks like that rule originates from this project/team. Why would that ever have been added even as a warning?
One question about ejecting... If we eject, does the browser/npm start then honor our .eslintrc.{js,json,yaml} or do we have to make changes somewhere else?

We are a little bit stricter in JSX because beginners often misunderstand how it works and produce bad markup when they deviate from the recommended style. We only enforce the parts that are overwhelmingly accepted in the React community, and don’t enforce anything that is debatable (e.g. where to close JSX tags).

When you say 'enforce', you must mean as errors?... These are not errors. They are warnings and they clutter the console for no good reason and make it harder for me to see the messages I'm actually interested in. I can still run my app, I just have to sift through unnecessary warnings.
This warning 'enforces' no spaces even though the style itself allows you to specify a number of spaces. (As an aside, I would contend that it should also support aligning equal signs when the attributes are on multiple lines). However, as I understand it, nothing other than the 0-space setting may be used, as it is baked in.
I have no problem with saving noobs from themselves, but using the .eslintrc provided by the user does exactly that. The users that need the 'help' won't know to change .eslintrc and the ones that do should be assumed to know what they're doing - principle of least surprise. The amount of time I spent tracking this down when the .eslintrc didn't 'just work' for this is really, really annoying and doubly so by the fact that I am not 'trusted' to change it for my own app.
I hacked the module locally to squash this message for myself, but that is very obviously not a long term solution and doesn't save other members of the team from being subjected to it. That's a terrible solution, but the only reasonable way I could get around it, which is just plain silly, not to mention more 'dangerous' than allowing me to just override things in .eslintrc.

When you say 'enforce', you must mean as errors

No, as warnings. I’d be okay with removing these rules from our config though, I guess, as you’re right these are stylistical rules, and we avoid them. Do you want to send a PR?

They are warnings and they clutter the console for no good reason and make it harder for me to see the messages I'm actually interested in

This is a completely different problem (and a bug). When we display ESLint errors (and break the build), we should hide the warnings. I’ll file a new issue for this 😉

Filed https://github.com/facebookincubator/create-react-app/issues/2056 and https://github.com/facebookincubator/create-react-app/issues/2057.


feature: add CSV creation to locale scripts

Description
There was a request from people working on our localization to provide the localization files in a tabular text form, so this script adds that transformation. It also adds the CSV files to our .gitignore.
Task Item
#minor

Coverage remained the same at 55.313% when pulling 3b640ceaf210fc1d4a282960be19b269057ae2ac on beyackle/l10nTSV into 9ba5e8412eb81852fdd140b692ad46528f9c7b1b on main.

I'm going to temporarily close this one and come back to it as part of a later effort to automate more of the documentation tasks (once that becomes a priority for me again). It doesn't need to stay open that whole time.


Search/ filter computed columns

Is there anyway to search or filter the computed columns in the search or filter bar like you can with non-computed columns? e.g. "computedColumnName" > "5". Thanks for the great plug-in!

If you mean global Kibana search/filter bar, this is not possible as this search/filter is done server-side on elasticsearch data, while enhanced table computed columns are computed and rendered front-side.
That said, in enhanced table options, you can enable a “filter bar” that allows you to filter table rows, similarly to Kibana global search bar.

But the filter bar only always you to search for specific values through out the rows. You can't look up a value in only a specific column and get back the rows where that matches, right?

Indeed.
You can do it through index pattern scripted fields.
or, in enhanced table, using “rows computed filter” option.

Given that question is answered, I close the issue.


Add mediafiles attachments to agenda items

like in motions, only as reference to mediafiles.

Means CustomSlide model internally.

See my branch.

see #1681


SCSB endpoint should return 404 when it his suppressed records

SCSB will ask for suppressed records because sometimes these are sent to Recap. We can't get them from the Alma API, though. In the code, we get the item and the holdings but then the request for the bib record returns an error (e.g. https://api-na.hosted.exlibrisgroup.com/almaws/v1/bibs/9958708973506421?apikey=)
We can return a 404 for this; distinguishing between this and the other 404s wouldn’t be helpful.
honeybadger issue is
https://app.honeybadger.io/projects/54497/faults/74759178
Use case background
When Recap staff receive an item and scan it into the the SCSB system (aka “accession” it), their system hits the bibdata scsb endpoint to pull in the data. If there’s any problem at this point the item will have status “incomplete”. If the problem was a 500 or other error from bibdata, it gets “Dummy Title” and “Dummy Author” as the data values.
Recap staff regularly pull up and investigate the report of incomplete records, so there is already workflow around handling errors.
The records should clear from incomplete status once #1049 is implemented because Recap staff will re-save the records that don't look right to get them to re-export to SCSB.

@eliotjordan @hackartisan Is this closed? I see that linked PR (#1077), but no history of this auto-closing.

@tpendragon sorry; because it merged to a non-default branch it doesn't auto-close. We have to remember to close issues when we merge.


[beta] Add solutions to the Applied Visual Design challenges

Issue Description
To allow npm test to properly test challenges, I think we should add solutions for each challenge.
This can be done by adding a solution as a one-line string to each challenge's solutions array.
A solution should be based on the challengeSeed.

I'm working on a PR for this.

As I couldn't get my solutions to pass npm test, I did some research. Whatever I did, the test runner returned ✖ SyntaxError: Unexpected token <.
After investigating other seed-files, I found that no HTML-based challenges even uses the solutions array. Next, I looked for the npm test script in package.json and found the file responsible for running tests, seed/test-challenges.js.
After reading that code, I understand why my tests weren't working. It's because these lines expect challenge solutions to consist of pure JS-code.
Should we extend seed/test-challenges.js to also allow testing of HTML and CSS challenges?
To do that, we could probably use something like https://github.com/krasimir/atomus to simulate the browser environment. It seems quite simple to just inject the solution HTML and CSS into this library and then run tests as usual.
Note that this library uses a fixed version of jQuery (1.X), so if some of our tests depend on newer versions of jQuery, we'd possibly need to submit a PR to Atomus to conditionally use other another jQuery-version.
Conclusion
By adding support for testing all types of challenges, we would be able to detect more errors before they make it into production. And hopefully, we can leverage existing tools to make this update not require more than ~40 lines of additional code.
@freeCodeCamp/moderators and especially @BerkeleyTrue, what do you think about this?
I would be happy to try extending the existing npm run test-challenges implementation if you think it's worth the effort :blush:

@Greenheart yes - imho we should extend test-challenges.js to include HTML/CSS solutions.
@BerkeleyTrue do we plan to continue using test-challenges.js for the foreseeable future? Is it worth the effort of extending?

@QuincyLarson @Greenheart We currently cannot test all the challenges because the testing framework is not set up for that.
I have https://github.com/freeCodeCamp/freeCodeCamp/issues/4201 which will allow us to test all the challenges.

افكر امسح النظام من عندي لحين وصول اموالي


Enhance Cluster Documentation

Hey guys,
When creating a microk8s cluster with more than two nodes and the private registry enabled on the master, image pulling won't work initially because of the registry running on http instead of https.
Users have to extend /var/snap/microk8s/current/args/containerd-template.toml
And add the following lines after line 53:
[plugins.cri.registry.mirrors."your-registry-ip:32000"]
          endpoint = ["http://your-registry-ip:32000"]

I guess some service needs to be restarted afterwards, as well!? I didn't know which one so I just restarted the system and afterwards it was working.
Currently there's no hint in the documentation, which would be good as I just stumbled across this issue.
Best regards,
Joschi

I can create a pull request for the docs if you wan't, but maybe there's a more convenient way to configure this step?


Util.addGestureDetector is deprecated

Description
I'm starting to develop games with flame, but I'm facing this problem described above when I try to add a detector to receive the touches made to the screen. I already tried to downgrade my flame version, which ended that problem, but caused another one that prevented me from running the program. The android studio says I should try to use an equivalent to the addGestureDetector in the Game class in order to solve this, but this class doesn't seen to have any function related to that. What should I do?
Development environment
Flame version: 0.28.0
Flutter doctor:
 Include your flutter doctor output here
[√] Flutter (Channel stable, 1.22.3, on Microsoft Windows [versÃ£o 10.0.18363.1139], locale pt-BR)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[√] Android Studio (version 4.0)

Runtime
This issue is related to running on which platform? (Select all that apply)

[x ] Android :android:
[ ] iOS :apple:
[ ] Web :browser:

Minimal reproducible code (Required for bugs)
Util flameUtil = Util();
flameUtil.fullScreen();
flameUtil.setOrientation(DeviceOrientation.portraitUp);
LangawGame game = LangawGame();
runApp(game.widget);
TapGestureRecognizer tapper = TapGestureRecognizer();
tapper.onTapDown = game.onTapDown;
flameUtil.addGestureRecognizer(tapper); (this part is deprecated)

That API is deprecated, you can still use it, but on future versions it will be removed. You can find the new API and an example on how to use on the site: https://flame-engine.org/docs/input.md#example

Thank you, very much! Orgulho de brasileiros tendo sucesso!

🇧🇷🇧🇷🇧🇷🎉🎉🎉


Change canonical loc to be end of vector.

Change canonical loc to be end of vector.
Reduces copying overhead on symbols with many locs.
Motivation

Test plan

Existing tests should catch any behavior changes.

I made the change that Jake requested.


Question: Does this code work for WP multisite installs?

Does this code work for WP multisite installs?
Any special considerations?

Basically, the script checks if the file exists on disk and serves it directly.  There shouldn't be any difference between single installation and multisite.


Update snapshot file

This updates a test snapshot.

What's the goal of this?

Ah, apologies. It was an attempt to run the test suite, since the Makefile was giving unhelpful errors.
I managed to run it now and see this is intentional.
When running just cargo test the tests fail, which is surprising at first.

What's the error you get when running make test? It passes on my end and in CI, but we should ensure it passes on all systems, so that's helpful feedback if it fails...


Update EmbeddedMode.md

Docs Build status updates of commit f08e22a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/develop-your-app/EmbeddedMode.md
:white_check_mark:Succeeded

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


Strange "xsv search" behavior?

> echo '<IP_ADDRESS>,one.one.one.one' | xsv search hello
<IP_ADDRESS>,one.one.one.one
$ xsv --version
0.13.0

That's not strange. xsv treats all CSV files as having heads by default. Please consider reading the xsv search -h output, and in particular, see the --no-headers flag.


feat: support prefix based parsing for scopes

Hi @levivannoort, this PR addresses the prefix-based parsing for scopes, implementing regex pattern support. It also updates the scope-checking conditions to ensure the workflow functions correctly even with a single scope. If you have any feedback or suggestions, please let me know. Thanks!

Hi @paopa, thanks for picking up the suggestion I addressed in the previous pull request, this is a welcome addition.


fix: disable checking for void return

Closes #1197
✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
Changed www's .eslintrc.json to stop checking functions for void returns. This allows external libraries like react-hook-form to be implemented, as their documentation that recommends feeding a return-promising function to one that expects to return void. This was despite react-hook-form being built in TypeScript.
Disabling this check will allow other library code using less strict versions of TypeScript to be implemented without errors, and is the official TS ESLint solution:
https://github.com/typescript-eslint/typescript-eslint/pull/4623
https://github.com/react-hook-form/react-hook-form/discussions/9325#discussioncomment-4060566

just realized that my issue is with the .eslintrc.json file that you get when you create-t3-app, not the .eslintrc.cjs that is present in the repo that scaffolds t3 apps. Not sure how to change what the .eslintrc.json rules are for people that create a t3 app, but this pull request is irrelevant to the actual issue #1197

Might be able to make this even more specific to just this conflicting return types issue by using:
"@typescript-eslint/no-misused-promises": [
"error",
{
"checksVoidReturn": {
"returns": false
}
}
]
"For example, if you don't mind that passing a () => Promise to a () => void parameter or JSX attribute can lead to a
floating unhandled Promise." Might need a comment mentioning this, so any imported library code needs to explicitly handle
unresolved Promise errors.
Quote TS ESLint docs: https://typescript-eslint.io/rules/no-misused-promises/

Might be able to make this even more specific to just this conflicting return types issue by using:
"@typescript-eslint/no-misused-promises": [
    "error",
    {
      "checksVoidReturn": {
        "returns": false
      }
    }
  ]

"For example, if you don't mind that passing a () => Promise to a () => void parameter or JSX attribute can lead to a floating unhandled Promise." Might need a comment mentioning this, so any imported library code needs to explicitly handle unresolved Promise errors.
Quote TS ESLint docs: https://typescript-eslint.io/rules/no-misused-promises/

What do you think is the best approach? If eslint says this is the way we should probably follow it?

I think a doc change. Possibly in FAQs we'd add in a section on "Type errors with external libraries", where we can link recommended ESLint workarounds / settings.
The framework is so type strict currently that a suggested list of workarounds for small, specific stuff like Promises in a void return from a 3rd party library would be a cool feature to have, reduce stack overflow / google searching, and improve DX.
I'll change my commit from the config change to an FAQ change once I'm home later. Thank you @juliusmarminge !

I'll close this but feel free to open a PR for a docs addition when you got time :)


[Feature request] Why not just update the repository for Cosmic

Ubuntu Cosmic (18.10) prerelease user here :)
Why not just make repositories work for 18.10? i think that there isnt so much differences on this one?

I've uploaded packages for cosmic.


Facilitate managing Docker Swarms

Use-case: I have a docker swarm. I'd like to create a new connection for a swarm manager, adding various children configurations, then clone the entry once for every other manager node I have. Currently cloning seems to only clone the top-level connection, forcing me to recreate the children by hand.

Related use-case, it would be helpful to have Docker Swarm related functionality, like applying an action across all manager or worker nodes. For example, if xpipe was swarm-aware then once I add a manager node it might offer to add all other manager nodes (and maybe worker nodes) automatically.

Following up on https://github.com/xpipe-io/xpipe/issues/124#issuecomment-1868673371
The reason you want to copy the children nodes (at least the ones that were added manually by the user) is that some services are not auto-detected. For example, I sometimes run JVMs with JDWP (debugging) enabled inside a container and I want to establish an SSH tunnel from my local machine to the host in order to attach a debugger to the running application. I don't think xpipe will auto-detect such a thing so I need to manually recreate it once per node.

I see the use case, let me think about how to handle this

I might be overthinking this. I only have two use-cases for now:

Establish a tunnel to debug a remote JVM.
Establish a tunnel in order to manage/debug the swarm using https://github.com/jesseduffield/lazydocker.

In practice, connecting to any of the manager nodes is sufficient for lazydocker. For debugging the JVM, I need to connect to the right node.
The point is that I probably don't need the original use-case of adding the entire swarm to xpipe. Lazydocker already provides a lot of functionality so, unless you plan to reimplement all its functionality, it's sufficient for me to just connect to a single node and switch over to it.
There is still value in making it easier to attach a debugger. I'm thinking of doing the following:

Add -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=*:5005 to all the JVMs I launch.
Run docker with -p 5005 which will map the port to a random host port
Use docker port <container-id> to look up the host port number
Create an ssh tunnel from localhost:5005 to the docker host at that port number.

xpipe could theoretically automate steps 3 and 4. A user could connect xpipe to a docker swarm, select a service they'd like to debug (establish a tunnel) and attach a debugger. Anyway... this is just me thinking out loud. I'll let you figure out whether this is something you want to pursue.

So I am not that familiar with swarms, but right now if you add a docker context on the system to xpipe, it should show you all containers. When you refresh that, it should automatically add a services entry for each container. If you refresh that, it should show you the docker ports as services and you can tunnel them from there.
I don't know how many systems you use, but if that is something you want to fully automate that process including adding the JVM debugger args and exposing ports, you can look into the API at https://github.com/xpipe-io/xpipe-python-api


update get full exception info when exception has InnerException

when exception has InnerException
use exception.Message will not show full info
sometimes the InnerException will be very helpfull

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

The MessageFormatter in LoggerExtensions only keep the second parameter, never use the parameter ex , so the InnerException will be ignored

Then the issue should be addressed in Logging, not here.

I create a issue in https://github.com/aspnet/Logging/issues/675

Updated.
Here's some background #442. It's an unfortunate design flaw of logging.
So all logger must be handle exception and user's message both.
Although the code will log redundant ex.Message in https://github.com/fs7744/Diagnostics/blob/62cc1bd80bc9eeaf78189f68936f58e5e98c5239/src/Microsoft.AspNetCore.Diagnostics/ExceptionHandler/ExceptionHandlerMiddleware.cs#L48
But it not big problem.


Api match mds schema

Tests pass, but we do not have a doct conda package it can see yet.

@arkilic @danielballan In metadataservice the events are stored with data/timestamp as two dictionaries.  In this implementation (and previously) they are stored as a (value, timestamp) tuple in a single data dict.
Pros of changing schema (now)

already doing a migration, so this is a great time to do it if we are going to do it
store on disk matching the in-memory representation
reduces thrashing on both way in and out

cons of changing schema

store in form that matches read, however that data gets re-formatted on the way out of bs to the {data:{}, timestamp: {}} form.
current schema help to enforce that we always have value + timestamp.

On net, I think changing the schema now is the better option.

Agreed. Less migrations :+1:


[INFRA] add bids_spec pdfs from before move to GitHub

In this PR I am adding the bids specification PDF files for the versions that were released prior to our move to GitHub as a platform.
See https://github.com/bids-standard/bids-specification/issues/392 for context.
I am adding the following versions:
1.0.0
1.0.1
1.0.2
1.1.0
1.1.1

I took the pdfs from the old bids website, where they were originally hosted and where they are still archived: https://github.com/bids-standard/bids-website/tree/gh-pages/old_website
I propose to not include the release candidate versions (rc), because they shouldn't be treated equal to a release. Side note: I cannot find 1.0.0-rc3 anywhere.
Note that the pdf for version 1.1.0 was 6MB of size, so I removed the embedded fonts to shrink it. Furthermore, I ran all of the files through some online shrinking service, further reducing the size. To my inspection, the quality has not suffered from this treatment.
To discuss:

[ ] where exactly to store the PDFs --> I am currently proposing a separate directory from the root ./historical_pdfs
[ ] how to link to the historical PDFs. This could be done in a separate PR. One place could be the changelog file, where each version heading is a link to that version. See also: https://github.com/bids-standard/bids-specification/pull/405#discussion_r376162587

Please no rendered PDFs as is in this repo. It will be of no benefit to any contributor - it will make repo grow, clone and fetch slow down, etc
Use git lfs, annex, it just place into a separate git repo, which you can include as optimal submodule here.

Please no rendered PDFs as is in this repo

Usually I would agree, but these PDFs are arguably a special case.

They will not change, so they won't blow up the history beyond their initial file size
They will remain the only PDFs added to the repo, because the "new" pdf (see #135 and connected issues/prs) will be generated on each build (and "stamped" according to its version), but stored either on ReadTheDocs or CircleCI (we have to check)
Conceptually it makes sense to have all specifications in one place
The drop in speed for cloning etc. will not be very noticable

But let's hear what other contributors have to say.

Conceptually it makes sense to have all specifications in one place
...
It is good to have the old versions here, rather than scattered over the intertubes.

... and the counter argument to those is in the aforementioned item:

the "new" pdf (see #135 and connected issues/prs) will be generated on each build (and "stamped" according to its version), but stored either on ReadTheDocs or CircleCI (we have to check)

So -- they will be spread around, unless we centralize explicitly.  E.g. this repository could reference (urls) them from the corresponding locations (a dedicated repo with historical PDFs and RTD for new ones).  Or we could even setup automated population of that dedicated repo with newer versions as well.
I really do not see sticking historical PDFs (only) into this repository (of sources) solving a problem of scattering PDFs around.

What about having the PDFs at a place where they will remain persistent and each get their own  DOI? E.g. Zenodo? This could be done for past PDF versions, but in principle also for current and future versions (when rendered as PDF).

Heya! I like the idea of putting the PDFs in a zenodo archive.
We could make one repository, and then upload the PDF files in the order of oldest to newest, doing a new release (giving a new doi) for each. That means we can have one DOI for each version, and one link that will go to the latest version of the DOI (https://help.zenodo.org/#versioning).
That way we can keep the persistant archive of all the PDFs in one place.
We could maybe add a section to the specification on "how to find previous versions" and include those dois?

I'm :+1: for Zenodo archiving the old version. One concern I'd have is to ensure that the PDF can be directly accessed by URL, and not just the landing page.
When a GitHub release is archived to Zenodo, it's zipped, rather than making all of the individual files directly available. So I'd just want to check that not everything gets hidden in a zip and turned into a multi-click exercise.

Has somebody already started posting these to Zenodo? Should I do that? Do we need to finally set up the bids-maintenance user (#332) so that no one person becomes a bottleneck for working on Zenodo?

Has somebody already started posting these to Zenodo?

no, not yet

Should I do that?

I think we should first address your own concern: "ensure that the PDF can be directly accessed by URL, and not just the landing page"
If Zenodo does not work for this, we could think about alternatives:

figshare? (I would prefer not to use it)
An OSF page for each spec? ... owned by a BIDS OSF user? (similar to the bids-maintenance user)

Do we need to finally set up the bids-maintenance user (#332) so that no one person becomes a bottleneck for working on Zenodo?

yes! And share the password for that user among the maintainers.

Zenodo entry: https://zenodo.org/record/3686062
Direct download link: https://zenodo.org/record/3686062/files/bids_spec1.0.0.pdf
I used the date of https://github.com/bids-standard/bids-website/commit/74303972bae4f411bb2aaf3f9205d99283f380f3 as the publication date, and retrieved the author list from https://www.frontiersin.org/10.3389/conf.fnins.2015.91.00056/event_abstract as a first approximation. Metadata can still be edited, but the data archive itself is no longer editable.
We should be able to add new versions straightforwardly. The contributors list shows up in 1.1.0, so we can be a little more definite at that point.
The @bids-maintainer user created the entry, so anybody with access to that (I can share the password over a secure channel) can push new versions.

great, thanks a lot @effigies.
If I understand correctly, we'll have a single DOI for the PDFs ... but different download links? See this screenshot from the zenodo page:

Remaining TO DOs:

[ ] upload the remaining "old" BIDS versions one by one, getting a new download link for each, and making sure to adjust the contributors starting with version 1.1.0
[ ] add a section to the BIDS specification about where to find the PDFs of the specification ... and determine where that section should be in the spec
[ ] editing the Release Protocol to include a short section on how to generate PDF for a version of the spec, and then how to upload it to Zenodo

There will be a common DOI and a specific DOI per version.

actually, looking at my To Do list in https://github.com/bids-standard/bids-specification/pull/407#issuecomment-590831912
I realized that if we are to get DOIs for each spec, we have to get back to https://github.com/bids-standard/bids-specification/issues/66
and discussing the authorship order for each spec. That was easy for the version that @effigies uploaded (because it already existed). For the other specs, we'll have to first find a consensus.

I am adding the remaining PDFs one by one to zenodo now. Just for documentation purposes, I get the publication dates from these commits:

1.0.1 https://github.com/bids-standard/bids-website/commit/ff9ccf43c56c8b492a0d2c5b431be057e0e99f7e
1.0.2 https://github.com/bids-standard/bids-website/commit/ddd73c3e2a0ca737ce65062c78145c6d6cf194e8
1.1.0 https://github.com/bids-standard/bids-website/commit/d0a0570cdded7ccea057aec05a257b8949b87dd6
1.1.1 https://github.com/bids-standard/bids-website/commit/6ea0e1a3c5e606f6bba6c11aa02221f629faced7

I renamed https://github.com/bids-standard/bids-specification/issues/66 and edited the OP to reflect  that we need to finish discussing author order.
Discussion can continue there, I am closing this PR, because it's original goal has been achieved in other ways.


feat: simplify Result.from and .combine

Thank you for this proposition! This make the codebase much simpler and easy to extend, it helped me a lot to find a solution for doing Result.combine from promises directly.
As this PR include also Result.from promise, that I do not agree for, I migrated the simplifications you proposes in other PR, together with UnknownError proposition: https://github.com/wiktor-obrebski/type-safe-errors/pull/21


javax.crypto.IllegalBlockSizeException on few devices

For some reason devices like ZTE V6 Plus (6.0) throw javax.crypto.IllegalBlockSizeException where others, mainly mainstream Samsung, Huawei, Sony work as expected.
RxFingerprint.encrypt(EncryptionMethod.AES, context, "a", "1234567890123456").subscribe()
At least one fingerprint has been enrolled.
Any ideas?

Do you have a more detailed Stacktrace for the exception? Which version of RxFingerprint are you currently using?

Unfortunately not. It's 2.2.1.

Without much more details it's hard to guess what actually is the problem in this case. For AES the key size is set to 256 by default which should be valid. Might be that the implementation on the ZTE device is somehow incompatible with this.

Yes...
Let me take a peek into firebase. May there's more to work with.

Exception javax.crypto.IllegalBlockSizeException: android.security.keystore.AndroidKeyStoreCipherSpiBase.engineDoFinal (AndroidKeyStoreCipherSpiBase.java:519) javax.crypto.Cipher.doFinal (Cipher.java:2056) com.a.a.g.a (SourceFile:110) com.a.a.i$2.onAuthenticationSucceeded (SourceFile:114) android.hardware.fingerprint.FingerprintManager$MyHandler.sendAuthenticatedSucceeded (FingerprintManager.java:855) android.hardware.fingerprint.FingerprintManager$MyHandler.handleMessage (FingerprintManager.java:803) android.os.Handler.dispatchMessage (Handler.java:110) android.os.Looper.loop (Looper.java:203) android.app.ActivityThread.main (ActivityThread.java:6255) java.lang.reflect.Method.invoke (Method.java) com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run (ZygoteInit.java:1063) com.android.internal.os.ZygoteInit.main (ZygoteInit.java:924)

Another device facing the same problem: One Plus 5T
But only after creating new fingerprints from the android settings. Old fingerprints worked well with the lib.

Interesting. Unfortunately I have neither device at hand to test it. Also the Stacktrace is not being much more helpful.
Which encryption method are you using? Did you change this at some point?

Same for me :/
Untouched encryption method is AES. See my code above.
It's strange..
`
@Override
| protected final byte[] engineDoFinal(byte[] input, int inputOffset, int inputLen)
  | throws IllegalBlockSizeException, BadPaddingException {
  | if (mCachedException != null) {
  | throw (IllegalBlockSizeException)
  | new IllegalBlockSizeException().initCause(mCachedException);
  | }
  |
  | try {
  | ensureKeystoreOperationInitialized();
  | } catch (InvalidKeyException | InvalidAlgorithmParameterException e) {
  | throw (IllegalBlockSizeException) new IllegalBlockSizeException().initCause(e);
  | }
  |
  | byte[] output;
  | try {
  | flushAAD();
  | byte[] additionalEntropy =
  | KeyStoreCryptoOperationUtils.getRandomBytesToMixIntoKeystoreRng(
  | mRng, getAdditionalEntropyAmountForFinish());
  | output = mMainDataStreamer.doFinal(
  | input, inputOffset, inputLen,
  | null, // no signature involved
  | additionalEntropy);
  | } catch (KeyStoreException e) {
  | switch (e.getErrorCode()) {
  | case KeymasterDefs.KM_ERROR_INVALID_INPUT_LENGTH:
  | throw (IllegalBlockSizeException) new IllegalBlockSizeException().initCause(e);
  | case KeymasterDefs.KM_ERROR_INVALID_ARGUMENT:
  | throw (BadPaddingException) new BadPaddingException().initCause(e);
  | case KeymasterDefs.KM_ERROR_VERIFICATION_FAILED:
  | throw (AEADBadTagException) new AEADBadTagException().initCause(e);
  | default:
| //line 519
  | throw (IllegalBlockSizeException) new IllegalBlockSizeException().initCause(e);
  | }
  | }
  |
  | resetWhilePreservingInitState();
  | return output;
  | }
`

Hey Mauin, here's another Stacktrace from Android Vitals:
java.lang.NullPointerException:    at com.mtramin.rxfingerprint.AesDecryptionObservable (SourceFile:95)   at com.mtramin.rxfingerprint.FingerprintObservable$2.onAuthenticationSucceeded (SourceFile:105)   at android.hardware.fingerprint.FingerprintManager$MyHandler.sendAuthenticatedSucceeded (FingerprintManager.java:1005)   at android.hardware.fingerprint.FingerprintManager$MyHandler.handleMessage (FingerprintManager.java:926)   at android.os.Handler.dispatchMessage (Handler.java:105)   at android.os.Looper.loop (Looper.java:164)   at android.app.ActivityThread.main (ActivityThread.java:6809)   at java.lang.reflect.Method.invoke (Native Method)   at com.android.internal.os.Zygote$MethodAndArgsCaller.run (Zygote.java:240)   at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:767)

Just going around and have similar issue. Try it in android studio emulator on Nexus One with api 23. You will probably get the same error. This could help you with debugging

any update on this? I have same problem


ensure a linebreak after config validation errors and do not parse config for help

Description
This PR:

ensures that a line break is printed after the config validation error output
ensures that configuration is not parsed for help topics (but only on execution of the command)

Related Issue

Fixes #4402

Motivation and Context

make oCIS intuitive to use
How Has This Been Tested?

have a non configured oCIS (rm -rf ~/.ocis)

run ocis storage-users uploads list --help, it succeeds with this PR but not with master
run ocis server, it has a proper linebreak at the end of the error message
run ocis migrate shares --help, it succeds with this PR but not with master
run ocis migrate publicshares --help, it succeds with this PR but not with master

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[x] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

Great, thanks 👍

Maybe to re-kick CI to see if it turns green post the not starting issue last week...?


Cannot Upload

Hi Martin,
I am creating a new form with the upload component and when clicking on the upload files button nothing happens.
When I checked the console it has the following error:
uploader.js:340 Uncaught TypeError: $(...).render is not a function
    at uploader.js:340
    at uploader.js:344

It's quite weird because I have used the component on another website which is working fine, I've followed the exact steps but can't get this working.
Thanks for your help.

This is not a problem with the plugin at all, I think the theme JS (bootstrap) is the issue here.
If anyone has any solution for this kindly let me know please.
Thanks.

What was solved your problem?

I have solved my issue. Problem was in my thema layout I forget added:
After all *.CSS linked:
{% styles %}
After all *.JS script:
{% scripts %}
This two line of code solve my problems and now everything works correctly.


ci: add token for semantic release

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


exit - navigation

It would be great to use '0' everywhere in the menu tree to navigate one level up.
At the moment I find it confusing and in some places not logical that for example when you are inside a sub-menu of >Appearance you need a different number depending on how many options are available.
Also in this situation I'd expect 'cancel' to directed me back to 'Appearance' and not two levels up to the main menu.
I know that mhwd-tui is actually not the same piece of software, but of course when navigating the options you have the feeling that you stay inside the same application.
Therefor it is confusing that in mhwd-tui exit navigation is different again.
I would expect when I am for example inside the option 'Install kernel', that '0' whould bring me back to mhwd-tui.
Ctrl+c should maybe be the shortcut to abort and exit bmenu altogether, no matter where you currently are.

Yeah, there is much room for improvement here. I agree with your idea. Unfortunately I know this side of bash poorly. Patches are welcome =). But I'll see what I can do about that. Items using select are difficult, because numbers are assigned to items automatically. I use it mainly in places where I don't know number of items beforehand (like choosing kernels).

You're doing a fantastic job, @Chrysostomus ! I would not even know where to begin... But I will see what I can do if I find some time, soon :)
Thank you!


Add a helper to sign API responses with self-authenticated JWTs

When sending a response from a server API you would typically add a AUTHORIZATION header with a self-authenticated JWT such that subsequent API calls can use a refreshed JWT.
Consider adding a helper to sign any response.
Possible implementation:
trait JwtResponseHelper {

  def selfAuthenticationTokenGenerator: SelfAuthenticationTokenGenerator
  def hostLogger: HostContextLogger

  def selfAuthenticationErrorResult =
    InternalServerError("Failed to generate self-authenticated JWT")

  /** Signs the given result using a JWT.
    *
    * Sings the result by adding an AUTHORIZATION header of the format 'JWT $token'.
    *
    * @param result Result to sign
    * @param hostUser Atlassian host user associated with this operation
    * @return Either the signed result or a JWT signing error
    */
  def withJwtAuthorizationHeaders(result: Result)(
      implicit hostUser: AtlassianHostUser): Either[JwtSigningError, Result] = {
    selfAuthenticationTokenGenerator
      .createSelfAuthenticationToken(hostUser)
      .map { token =>
        result.withHeaders(AUTHORIZATION -> s"JWT $token")
      }
  }

  /** Signs the given result using a JWT.
    *
    * Sings the result by adding an AUTHORIZATION header of the format 'JWT $token'.
    *
    * This is a convenience function that logs an error and returns either the
    * default error result or any result given as error result.
    *
    * @param result Result to sign
    * @param onError Result to return if signing the token fails
    * @param hostUser Atlassian host user associated with this operation
    * @return Signed result or error result if signing failed
    */
  def withJwtAuthorizationHeaders(result: Result,
                                  onError: Option[Result] = None)(
      implicit hostUser: AtlassianHostUser): Result = {
    withJwtAuthorizationHeaders(result) match {
      case Right(token) => result.withHeaders(AUTHORIZATION -> s"JWT $token")
      case Left(error) =>
        hostLogger.error(
          s"Failed to generate self-authenticated JWT: ${error.getMessage}")
        onError.getOrElse(selfAuthenticationErrorResult)
    }
  }

}

In the example above it's probably better to remove the logger dependency as each class that mixes in this trait is forced to provide this logger.

The header used in the response should be configurable, instead of using hard-coded AUTHORIZATION header. AUTHORIZATION should not be used anyways because it's not a valid HTTP response header.

Fixed by #20


Added homebrew install paths to Cake tool resolver

Currently if you install cake via homebrew and execute it using the global alias cake and then proceed to use CakeExecuteScript (..) it will fail to find Cake.exe to run.
This patch simply looks for Cake.exe in the paths that homebrew will install cake and sorts it so that the newest version is returned first.

@Redth when you have the opportunity could you rebase this into 1 commit.

Squashed. :)

@Redth merged, thanks! :+1:


feat: add possibility to create infinite toast by using 0 in duration

Thank you for the suggestion, appreciate it! I went with a slightly different approach here #46.


Internal error when pinging with dgram socket

As the title suggests, I am getting an internal error when calling ping::dgramsock::ping. I am not getting this error when running the raw socket version, but as the crate already suggests, that requires me to run the program with sudo.
I have been able to figure out where the error is coming from exactly, namely the last part of the ping function, that being the decoding of the response packet. The error in the decoding happens when checking whether the version is 4, this is because it is actually 0.
I do not know what this means, but that is what I managed to extrapolate. The data returned when using a dgram socket is the following:
00 00 A2 EF 00 07 00 01 2E DD 45 29 13 D0 DC 21
AA 78 A4 7D 84 B4 0D 0E 7D 66 2D C4 56 90 16 9C

I cannot provide the data returned when using a raw socket as getting CodeLLDB to run the program with sudo is a pain.
I am running the program on a Raspberry Pi 4B running Raspberry Pi OS Bookworm (Debian 12) 64 bit.

I just guess, maybe there is another udp packet send to your process, and ping try to decoded it and centainly failed.
#20 just fixed it and shipped with v0.5.2, you can retry with this version.

During my own debugging, I found that on our system, neither case provided proper IP packet headers when using dgram sockets. I found logic in the code to only decode IPv4 headers, but decode IPv6 data directly. Upon reverting this so IPv4 and v6 both parse only the ICMP data directly from the socket, the pings worked, but the validation stage failed. Upon further research, it seems as though the Linux kernel may be assigning random ident values when performing unprivileged pings, I assume as a safety measure (similar to how one cannot bind to ports below 1024 without special capabilities), which invalidates the check within the ping library itself. I fixed it for us, but it's not the cleanest solution, and I'm not 100% sure if this is the correct solution, since the code functions fine as it is on Mac OS. Posting to hopefully provide insight and further discussion to find a better solution.

It could also be relevant that I was building for x86_64-unknown-linux-musl. I would imagine that musllibc isn't doing anything different from glibc here, but it is a possibility.

Ok, I'll try to re-produce this error with x86_64-unknown-linux-mus when I have time. Can you also provice what Distro and kernel version for your system?

We are using RedHat kernel 4.18.0-372.9.1.el8.x86_64
Thanks!

This also happens for me on Ubuntu 22.04, kernel 6.5.  I will try and investigate when I have some time.


Working example GET JSON not working

When I click the link for GET JSON the page just alerts "error". I looked at the used url and it seems that the JSON you used to get there is not on the server anymore.

Thanks, it's fixed now.


ENH: Provide public API for fast DisjointSet subset size.

Reference issue

Closes #17285.
What does this implement/fix?

Provide public API to quickly get the size of a DisjointSet subset.
This information is already stored internally as disjoint_set._sizes; this PR simply adds a public method to access it.
Additional information

@fuglede is this good to go for you?

Yes. Lint check is failing but for a file that hasn't changed here.