Unable to connect to IPv4 IRC server when bindhost is IPv6 & IPv4

Hi
I have bindhost which has an IPv6 and an IPv4 address, when I connect to an IPv6 IRC server ZNC connects fine, when I connect to an IPv4  server I get
[08:43] <*status> Attempting to connect to [<IP_ADDRESS> 6667] ...
[08:43] <*status> Cannot connect to IRC (Invalid argument). Retrying...
If I run ZNC in debug mode I see the following
User [user] is connecting to [<IP_ADDRESS> 6667] ...
Csocket.cpp:880 Connect Failed. ERRNO [22] FD [6]
IRC::user == SockError(22 Invalid argument)
Is this something that can be fixed?
Work around for now is to create a bindhost which is the IPv4 address of which ZNC will use to connect from.
Regards

This should be fixed since 717d0596e3723a77d0158a633ec3c37f1d6f02d9. If you still have this issue please re-open.

I am having save problem
Disconnected from IRC (Server address is IPv6-only, but bindhost is IPv4-only). Reconnecting...
How I fix this ?

@fahadmehar try clearing your bindhost, /msg *status setuserbindhost or /msg *status setbindhost. If that doesn't help, give more details such as /msg *status version and output from znc -D showing the error.
Related: https://github.com/znc/znc/issues/241 as thanks to it people set <IP_ADDRESS> as bindhost.

It's /msg *status clearbindhost. <IP_ADDRESS> shouldn't be needed anymore.

This is my Ver
<*status> ZNC 1.6.4 - http://znc.in
<*status> IPv6: yes, SSL: yes, DNS: threads, charset: no

Please I really want to set VHost,

How I fix this '?
<*status> Disconnected from IRC (Server address is IPv6-only, but bindhost is IPv4-only). Reconnecting...

I clear my binhost so many times even with soap its not working...

What server are you trying to connect to? Try something that also has an IPv4 address (dunno, some freenode server?)

ON DALnet Server @psychon

Try <IP_ADDRESS> or <IP_ADDRESS> or <IP_ADDRESS> or <IP_ADDRESS> as server (random dalnet servers; however, I cannot find one with an IPv6-address; what server did you give ZNC to connect to?)

My IP address is IPv6 and They said my bIndhost is IPv6 only? I don't understand this?

Pardon me, but my psychic abilities are having a bad day: Who is "they" and what exactly do "they" say? Any more specific error message from ZNC?
Why do you even still have a bindhost after /msg *status clearbindhost? That should unset any bindhost.

Man, I want to use my VHOST which I set from www.freedns.afraid.org. On IPv6 Address. But when I set the BindHost I get this message

Uhm. That sounds more like ZNC's message is mixed up (is it?) and you are having an IPv6-bindhost, but DALnet does not have any IPv6 servers.
In that case, try /znc addserver ipv6.dal.net (or /znc addserver 2001:550:101:a::2 and /znc addserver 2a00:1a28:1100:4::a)

DALnet allow Ipv6 and on my ZNC panal i have put ipv6.dal.net server...

And After setting my Vhost on afraid and i cant able to add Ipv6 Ip on my panal. It says Invalid Hostname.

and it says my bindhost is ipv4 only
Disconnected from IRC (Server address is IPv6-only, but bindhost is IPv4-only). Reconnecting...

and it says my bindhost is ipv4 only
Disconnected from IRC (Server address is IPv6-only, but bindhost is IPv4-only). Reconnecting...

But didn't you say that your bindhost is an IPv6 address?
Here:

Man, I want to use my VHOST which I set from www.freedns.afraid.org. On IPv6 Address. But when I set the BindHost I get this message

I'm confused by your setup. And the use of "VHOST" confuses me further.
Can you tell me what you set up with freedns? (Which IP, which DNS name)
I think that you cannot use this to get a vhost...

@psychon  OK I can explain... I purchase VPS ok and I setup my ZNC and after setting up my ZNC i try to set Vhost for ZNC which i did from www.freedns.afraid.org. I put my IPV6 ip and set my VHost i try to put that vhost on znc. I get this message...

The BindHost in ZNC just selects the IP address that ZNC connects from. If you only have a single IP address, the BindHost option doesn't have any effect.
Since you mention VHost: When the IRC server sees an incoming connection, it only sees your IP address. It then looks up the PTR record of that IP address to get a domain. Then it resolves that domain into an IP address. If the result matches the IP address of the incoming connection, the domain is used for display purposes instead of the IP address.
So, just setting up a new DNS record with freedns does not help, because you also need the PTR DNS record of the IP address (which you cannot control in most cases).
All of the above is why the option in znc is called BindHost and not VHost.

@psychon  I think my VPS IP is Default IPV4 ? Maybe that's the issue?

IPV6 disable!!!
Copy:
net.ipv6.conf.all.disable_ipv6 = 1
net.ipv6.conf.default.disable_ipv6 = 1
net.ipv6.conf.lo.disable_ipv6 = 1
Edit: nano /etc/sysctl.conf
Paste and save!
and shutdown -r now

ZNC configuration no write bindhost!
And work :)


API Umbrella host value must be autofilled.

Steps:
1.Visit https://apinf.io
2.Sign in
3.Select 'API Backends' from menu
4.Click on 'Add API Backend'
5.Enter API name
6.Select the backend protocol from drop down menu
7.Enter the details of servers: Host and Port
8.Enter Frontend host and Backend host

Enduser doesn't know where the API Umbrella is used and must be defined in settings and autofilled.

Verified on latest develop version.


Get Ribbon Button by Name

First of all: Thanks for that great collection of winform elements.
Now my question: Is it possible to get a ribbon button by name? How can I search in a collection for this?
Thanks, Greetings from Germany!
Danny

Just a guess, but maybe something along these lines:
public KryptonRibbonTabCollection RibbonTabs { get; private set; }
foreach over the RibbonTabs -> Ribbon
foreach (KryptonRibbonGroup group in Ribbon.Groups)
etc.

Hey Smurf-IV,
thanks for your comment. This was my first idea but i cannot check the name of the button. My second idea was search for the Controls with the Find-Method. But in the controls-collection is no button control.
I programme in VB.NET: Me.Ribbon.Controls.Find()
I would like set a Button enable or disable. My Form with the Ribbon has a Interface-Method that I call from an another Form. My Method-Name: ChangeRibbonButtonStatus(ByVal Name As String, ByVal Status As Boolean). Now you know what I want with the Name of the Button. :D Perhaps anyone has a idea?!
Thanks for all, Danny


iOS Universal linking doesn't seem to work

I can't seem to get iOS universal links working. I've tried everything - deleting & re-installing, upgrading version number, trying different devices, simulator & real, and in each case the links open up in Safari instead of opening the app. Here are the steps I have taken to setup universal links:

Uploaded apple-app-site-association to an https server with json content-type. You can see per the attached that it is appearing on Apple's CDN:

I also validated the apple-app-site-association file via https://branch.io/resources/aasa-validator/ (worth noting that I was not able to validate with Apple's appsearch-validation-tool - received error: "Could not extract required information for Universal Links.")

Added the associated domain in my project capabilities:

Opening with /usr/bin/xcrun simctl openurl or clicking on the url on my mobile device (tried from notes app) still opens the url in safari instead of opening the app. I also followed the troubleshooting tips here: https://www.wwt.com/article/using-sysdiagnose-on-ios-to-troubleshoot-universal-links/ and found from my phone analytics logs that the app is not even reading the AASA file.
What could I be missing? Has anyone managed to get iOS universal linking to work?

Hello,
Do you have add :
"apps": [],
in your file 'apple-app-site-association' ?
"The apps key in an apple-app-site-association file must be present and its value must be an empty array, as shown in Listing 6-1."
https://developer.apple.com/library/archive/documentation/General/Conceptual/AppSearch/UniversalLinks.html#//apple_ref/doc/uid/TP40016308-CH12-SW2
Also check if you check 'Associated Domains' on your provisioning profile :)
I use the example code and it works on emulator and real device (via QRCode).

Hello,
Do you have add :
"apps": [],
in your file 'apple-app-site-association' ?
"The apps key in an apple-app-site-association file must be present and its value must be an empty array, as shown in Listing 6-1."
https://developer.apple.com/library/archive/documentation/General/Conceptual/AppSearch/UniversalLinks.html#//apple_ref/doc/uid/TP40016308-CH12-SW2
Also check if you check 'Associated Domains' on your provisioning profile :)
I use the example code and it works on emulator and real device (via QRCode).

@kmoreau thanks for the reply! The format I have for the apple-app-site-association file is following the newest format according to apple documentation. See here: https://developer.apple.com/documentation/xcode/supporting-associated-domains. That said, I also tried the former format and it did not work either.
Also the attached shows 'associated domains' included in my provisioning profile.

Which example code did you test with?
Thanks again!

Hello,
Do you have add :
"apps": [],
in your file 'apple-app-site-association' ?
"The apps key in an apple-app-site-association file must be present and its value must be an empty array, as shown in Listing 6-1."
https://developer.apple.com/library/archive/documentation/General/Conceptual/AppSearch/UniversalLinks.html#//apple_ref/doc/uid/TP40016308-CH12-SW2
Also check if you check 'Associated Domains' on your provisioning profile :)
I use the example code and it works on emulator and real device (via QRCode).

@kmoreau thanks so much for your response! I actually figure out the issue and it was a rookie mistake on my end. Just for reference in case anyone else runs into this, I had only created entitlements for profile - didn't even notice I wasn't on the "all" tab on Xcode. Once I created for all (debug, release, profile) everything worked as expected.


Dev fix bluetooth

It was not much in the end but was hard to notice.
The commit breaking it was
https://github.com/brilliantlabsAR/monocle-micropython/commits/861ec41
A chain of commit attempting to trace the changes is at
https://github.com/brilliantlabsAR/monocle-micropython/commits/b1ee71e

*ahem* I think Github is a bit broken here. Here is how it looks on my side:
* 19fe5e6 (origin/dev-fix-bluetooth, dev-fix-bluetooth) fix bluetooth
* 714318f (dev, origin/dev-raj, origin/dev) Merge pull request #102 from brilliantlabsAR/dev-raj


fix(kmod): support clone child exit events on s390x

What type of PR is this?
/kind bug
/kind feature
Any specific area of the project related to this PR?
/area driver-kmod
/area driver-bpf
Does this PR require a change in the driver versions?
What this PR does / why we need it:
This PR introduces 2 main changes:

enable support for BPF raw_tracepoint on s390x architectures. BPF has some problems with architectures like s390x as also described in this issue https://github.com/falcosecurity/libs/issues/497.  With this fix the BPF probe can works also on kernel versions >= 4.17, but the problem is that there are many instrumentation drops due to the use of bpf_probe_read() and bpf_probe_read_str() that don't work at all on architectures like s390x as you can read here. In the end I would say that we cannot use bpf on these architectures, at least for the moment

enable the CAPTURE_SCHED_PROC_FORK logic in order to catch the clone_child exit events on s390x architectures. Like in ARM64, the clone child exit event is not traced by the syscalls:sys_exit tracepoint, while the excve exit event, in this case, is traced! So to reuse the same logic introduced in ARM64 we have just to enable it in this way:
#if defined(CONFIG_ARM64) || defined(CONFIG_S390)
    #define CAPTURE_SCHED_PROC_FORK
#endif

For more info about this problem of the clone_child exit event take a look at the ARM64 fix.

Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
fix(kmod): support `clone child` exit events on `s390x`

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

This is truly a 💣 !
I'd also update the README about the supported architectures and the minimum kmod version + an eBPF note!
About the supported architectures: IMHO we can have libs repo that supports more architectures than Falco officially supports; it just means that Falco would work on those but has no officially provided packages + drivers builds.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 78a3ec8390bcac2e75361854c199e717bc7a49ae

LGTM label has been added.  Git tree hash: cbaf552d708b61f34a5ada5882260d106963bab3

Also, I really appreciate you taking the time to link the relevant kernel code on comments, make understanding and tracking the changes much easier!

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: d0d58e0af48d461ae24f8f375ca1fb46d3090bf8

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, Molter73
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,Molter73]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[cranelift-simplejit] Remove panic when passed PIC flag

Since PIC has no effect, there is no reason to disallow the option.
This simplifies my life a little and has no impact on runtime behavior
(other than not panicking).

PIC has an effect. It changes the relocation type to one that SimpleJIT can't handle, as it doesn't know how to create a GOT.

@jyn514, what do you think? Should we close this then?

Yes, I'm fine with that.


Feature/000/date picker a11y

:tada: This PR is included in version 1.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Exiting yazi leaves some ascii characters in terminal

What system are you running Yazi on?
Linux X11
What terminal are you running Yazi in?
rxvt-unicode (urxvt) v9.30
Did you try the latest code to see if this problem got fixed?
Tried, but the problem still
yazi --debug output

Yazi
    Version: 0.2.5 (VERGEN_IDEMPOTENT_OUTPUT 2024-05-11)
    OS: linux-x86_64 (unix)
    Debug: false

Emulator
    Emulator.via_env: ("rxvt-unicode-256color", "")
    Emulator.via_csi: Ok(Unknown([]))
    Emulator.detect: Unknown([])

Adaptor
    Adaptor.matches: X11

Desktop
    XDG_SESSION_TYPE: Some("x11")
    WAYLAND_DISPLAY: None
    DISPLAY: Some(":0")

SSH
    shared.in_ssh_connection: false

WSL
    /proc/sys/fs/binfmt_misc/WSLInterop: false

Variables
    SHELL: Some("/bin/bash")
    EDITOR: None
    ZELLIJ_SESSION_NAME: None
    YAZI_FILE_ONE: None
    YAZI_CONFIG_HOME: None

file(1)
    Version: Ok(Output { status: ExitStatus(unix_wait_status(0)), stdout: "file-5.41\nmagic file from /etc/magic:/usr/share/misc/magic\n", stderr: "" })

Text Opener
    default: Some(Opener { run: "${EDITOR:=vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block: Some(Opener { run: "${EDITOR:=vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

tmux
    TMUX: false

Ueberzug++
    Version: Err(Os { code: 2, kind: NotFound, message: "No such file or directory" })

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "/home/david/.local/state/yazi" directory.

yazi.log:
  2024-05-11T20:56:02.500109Z  WARN yazi_adaptor::emulator: [Adaptor] No special environment variables detected
    at /home/david/.cargo/registry/src/index.crates.io-6f17d22bba15001f/yazi-adaptor-0.2.5/src/emulator.rs:66

  2024-05-11T20:56:02.500132Z  WARN yazi_adaptor::emulator: [Adaptor] Unknown TERM_PROGRAM:
    at /home/david/.cargo/registry/src/index.crates.io-6f17d22bba15001f/yazi-adaptor-0.2.5/src/emulator.rs:80

  2024-05-11T20:56:02.500135Z  WARN yazi_adaptor::emulator: [Adaptor] Unknown TERM: rxvt-unicode-256color
    at /home/david/.cargo/registry/src/index.crates.io-6f17d22bba15001f/yazi-adaptor-0.2.5/src/emulator.rs:87

  2024-05-11T20:56:02.502460Z  WARN yazi_adaptor::ueberzug: ueberzugpp spawning failed: No such file or directory (os error 2)
    at /home/david/.cargo/registry/src/index.crates.io-6f17d22bba15001f/yazi-adaptor-0.2.5/src/ueberzug.rs:84

Describe the bug
When run in rxvt-unicode, after exiting yazi, the characters Gi=31,s=1,v=1,a=q,t=d,f=24;AAAA are left in the terminal.

Expected Behavior
No leftover characters from yazi after exiting.
To Reproduce

Install urxvt (sudo apt install rxvt-unicode)
Run rxvt-unicode
Run yazi
Press q to exit

Configuration
no config
Anything else?
"Unknown TERM: rxvt-unicode-256color" in yazi.log might be relevant.

Thanks for the report!
It seems like urxvt doesn't recognize Kitty's graphics protocol detecting sequence, perhaps the easiest way to fix it is to add urxvt's TERM value rxvt-unicode-256color to the list of known terminals:
https://github.com/sxyazi/yazi/blob/c1e1f26c4c388ed4a39de117062a2db364535a22/yazi-adaptor/src/emulator.rs#L83-L89
so further CSI process can be skipped. Would you like to raise a PR for it?


Fixed performance tests

When testing MR please deploy your own hyperfoil due to CPU performance capability of single instance

test_rhsso_tokens still fails for me with:
testsuite/tests/performance/smoke/test_smoke_rhsso_tokens.py:71: in test_rhsso_tokens     benchmark = setup_benchmark.create_benchmark() testsuite/perf_utils.py:140: in create_benchmark     return self.factory.benchmark(benchmark).create() ../../.local/share/virtualenvs/3scale-tests-Hj_-YXcM/lib/python3.10/site-packages/hyperfoil/factories/hyperfoil_factory.py:30: in create     self._client.benchmark.create(files=self._files) ../../.local/share/virtualenvs/3scale-tests-Hj_-YXcM/lib/python3.10/site-packages/hyperfoil/clients.py:20: in create     response = self.rest.post(url=self.url, json=params, **kwargs) ../../.local/share/virtualenvs/3scale-tests-Hj_-YXcM/lib/python3.10/site-packages/hyperfoil/client.py:76: in post     return self.request('POST', *args, **kwargs) ../../.local/share/virtualenvs/3scale-tests-Hj_-YXcM/lib/python3.10/site-packages/hyperfoil/client.py:70: in request     return self._process_response(response) ../../.local/share/virtualenvs/3scale-tests-Hj_-YXcM/lib/python3.10/site-packages/hyperfoil/client.py:82: in _process_response     raise ApiClientError(response.status_code, response.content) E   hyperfoil.errors.ApiClientError: Response 400: b'Cannot read benchmark: line 6, column 3: : Benchmark parsing error; See below: \n- host: https://perf-6my-ficap-smoke-rdu-3scale-apicast-production.ficap.apps.${CLUSTER_DOMAIN}:443\n  sharedConnections: 20\n- host: http://no-ssl-sso-tools.apps.${CLUSTER_DOMAIN}:80/auth/\n  ^ HERE\n  sharedConnections: 100\nname: test_smoke_rhsso_tokens\n: : Host must not contain any path: http://no-ssl-sso-tools.apps.${CLUSTER_DOMAIN}:80/auth/'; None

test_rhsso_tokens still fails for me with:

#195 is not included here, but you can test it by rebasing


Stopped working after A15 update (Google App v<IP_ADDRESS>.arm64)

I'm not sure if this is due to Androi 15 update or not, but I was still able to open the RSS feed on A14

It's due to Android 15 yes. They completely changed how the feed works and Discover Killer may be dead as a result, I've not had chance to check yet.

In Smart Space it does show the expanded one, but I assume this is because you're hooking into Pixel Launcher and SystemUI and also because it does a different thing because of the At A Glance widgets. I'm not sure if this is really what would make custom overlay on Google App works again tho

Smartspacer is different and was easy to upgrade to the new method. A fully customisable overlay like Discover Killer is not so easy.

hope it can work again! Loved using this but a15 breaks it :(

I love it too ❤


TypeError: response.enhance is not a function

If an error is thrown when doing a request and you're using the RetryMiddleware then the initial error will be swallowed and Mappersmith will throw an error instead. This makes it impossible to debug the actual problem.
Exception has occurred: TypeError
TypeError: response.enhance is not a function
    at enhancedResponse (/Users/kenneth.gibson/klarna/klarna-app/services/orders/node_modules/mappersmith/middleware/retry/v2/index.js:117:19)
    at /Users/kenneth.gibson/klarna/klarna-app/services/orders/node_modules/mappersmith/middleware/retry/v2/index.js:87:88
    at <anonymous>
    at process._tickDomainCallback (internal/process/next_tick.js:228:7)

I think some middleware is changing the response because the retry middleware is prepared to handle failures. In this case, it is receiving something different. Maybe I could log the response if it isn't a Response object. WDYT?

Solved with the following: https://github.com/tulios/mappersmith/pull/118


fix: Headway headsign logic

Notion task: Multi-line headways include the correct destination
Something I did not think about when implementing headway mode was how to headsign matchers are there to determine the direction of the affected range of an alert. This means that headway needs the opposite direction's headsign (sounds so obvious now so not sure how I didn't think about this).
This PR adds an additional element to the headsign matcher tuples. I would rather solve this in the logic, but that seems risky because we are looking at platform_ids. Directional alerts may cause an ID we expect not to be present (or vice-versa). Because we know that the headsign matcher works for alerts, if we also hardcode the headsign for headway we will know for sure that the logic is correct.
Biggest concern: this affects anything that implements the SingleAlertWidget protocol. The upside is it can just ignore the fourth element if it does not have headway mode (which is every existing alert since headway is handled in departures).

[ ] Tests added?

Definitely agree with you on the new values in prefare_alert_headsign_matchers, so I thought about this again and came up with something else. Headsign matchers now use maps instead of tuples. This means we can add headway headsigns to just the configs that need it. So that value is gone entirely from prefare and DUP scenarios that do not display headway (mostly Kenmore stuff).


Provide template for gitlab CI

Create a .gitlab-ci.yml similar to : https://github.com/samsung-cnct/container-fluent-bit/issues/11
Helpful samples:
https://github.com/samsung-cnct/cyklops-events/pull/7/files
https://git.cnct.io/venezia/install-mongodb/blob/master/.gitlab-ci.yml

ToReview:

Added gitlab-ci.yml template to solas-container: https://github.com/samsung-cnct/solas-container/pull/18
Updated linked reference docs in solas: https://github.com/samsung-cnct/solas/pull/33

Things to consider:

Since ultimate integration with gitlab is still pending, the instructions on how to add a newly created repository to gitlab are still missing. Possibly wait until that is done?
There will also be some discussion on how to handle version tags that may or may not affect the way the publish stage should look.
A (pending integration) working example lives here.

@guineveresaenger I think this is looking good and we should start merging/closing stuff today. If it develops like the original solas work, the next few ports to GitLab will probably result in more tweaks and clarifications to the instructions. I would be interested to do a port today!

Fixed by https://github.com/samsung-cnct/solas-container/pull/18


chore: Bump dependencies to latest

[ ] I have added automatic tests where applicable
[ ] The PR title is suitable as a release note
[ ] The PR contains a description of what has been changed
[ ] The description contains manual test instructions

Changes

A single PR for all the dependency bumps.

Bumps all dependencies to latest, including node to 20
Exports were missing the types field, making it not work with TypesScript when moduleResolution is set to Node16 or NodeNext. I fixed this as well.

Test

Try it out, check the build output for any major changes compared to the old build output.

/home/runner/work/ftrack-web-widget/ftrack-web-widget/src/ftrackWidget.ts
  0:0  error  Parsing error: DeprecationError: 'originalKeywordKind' has been deprecated since v5.0.0 and can no longer be used. Use 'identifierToKeywordKind(identifier)' instead

/home/runner/work/ftrack-web-widget/ftrack-web-widget/vite.config.ts
  0:0  error  Parsing error: DeprecationError: 'originalKeywordKind' has been deprecated since v5.0.0 and can no longer be used. Use 'identifierToKeywordKind(identifier)' instead

Should we add a renovate config as well?


docs(2.3.x) create the 2.3.x doc tree by copying over 2.2.x directory

Adds autodocs generated from Kong 2.3

@lena-larionova apologies, I didn't see the note about the missing entry in kong_versions.yml . I thought it was already on the release/2.3 branch (notice this PR is against that branch, not against master. We will have to merge release/2.3 into master in another pr in order for it to be public)

@lena-larionova apologies, I didn't see the note about the missing entry in kong_versions.yml . I thought it was already on the release/2.3 branch (notice this PR is against that branch, not against master. We will have to merge release/2.3 into master in another pr in order for it to be public)


Let's try using doctrine to parse jsdoc

See https://www.npmjs.com/package/doctrine.

:+1:!

It looks like the issues with doctrine have been cleared up so we can totally use it.

I'm working on adding this.

Awesome!

Cool!

It still needs some tweaks but it's on it's way.

Let me know if you need any help on this.  I'm quite interested

I'll have to update the FP docs generation a bit with this. If I notice anything odd, I'll let you know.

I'll have to update the FP docs generation a bit with this. If I notice anything odd, I'll let you know.

Don't worry about that until we officially bump. Things are still baking at the moment.

:+1:


Workspace - Company type dropdown in Adding bank account page (Step 2) is broken

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging.new.expensify.com
Go to Settings > Workspace > any workspace > Bank account
Click Add manually
Proceed to Step 2
Click on the Company type dropdown

Expected Result:
Company type dropdown is not broken
Actual Result:
Company type dropdown is broken
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.69-10
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/a966e88a-e090-414b-a268-cdc248facf66
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Proposal
Problem
Company type dropdown in Adding bank account page (Step 2) is broken
Root cause
we are mapping a translation key which is a string to picker items.
Changes
Create a array with incorporation types in constants, map the array to items like below
                    items={_.map(CONST.INCORPORATION_TYPES, (value) => ({ value: translate(`companyStep.incorporationTypes.${value}`),

This is a regression of this PR

Looks like this is going to be fixed by https://github.com/Expensify/App/pull/27358

Closing this one out since it's fixed here


docs: Update Vault issuer docs for k8s 1.24

In response to the issue: https://github.com/cert-manager/cert-manager/issues/4144
I made a comment about how to workaround the change.
This PR is to attempt to address the k8s 1.24 change in the cert-manager documentation

I've tested this on kind with k8s 1.24:
Server Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.0", GitCommit:"4ce5a8954017644c5420bae81d72b09b735c21f0", GitTreeState:"clean", BuildDate:"2022-05-19T15:39:43Z", GoVersion:"go1.18.1", Compiler:"gc", Platform:"linux/amd64"}

This pattern also matches the note on the hashicrop resource too.

Just a question for maintainers whether the headings used for the k8s examples makes sense when you look at the navigation bar to the right. Screenshot of what I mean:

I'm not sure the side bar makes it apparent they are sub sections of "Authenticating with Kubernetes Service Accounts". Perhaps I should replace the heading text with just bold or some other formatting so it doesn't appear in the side bar?
Or beyond a certain heading level, does the site ignore the heading?

Thanks @irbekrm

IMO the header could stay as is in this PR

Could you readd the /lgtm label please? (since I changes the headers to remove <= and >= symbols)

Thanks for working on this @hawksight !
/lgtm
/hold cancel

@irbekrm it appeared to be some spelling issues causing the failure. I hope I've corrected them in 23f2d19 if you could re-label this PR again. Re:

New changes are detected. LGTM label has been removed.

Is there a pre-commit in this repo btw?

Is there a pre-commit in this repo btw?

No, there isn't..
You're thinking to run the verify script locally before committing? Yeah that could be good. One minor issue is that the verify script requires various node things to be installed though- wouldn't want to make it a requirement for external contributors who are just submitting some small fix. However, perhaps it's possible to somehow package the dependencies into an image for example

/lgtm

You're thinking to run the verify script locally before committing? Yeah that could be good. One minor issue is that the verify script requires various node things to be installed though- wouldn't want to make it a requirement for external contributors who are just submitting some small fix. However, perhaps it's possible to somehow package the dependencies into an image for example

Pre-commit is opt-in in that you have to have run the setup task for a repository otherwise it takes no effect and acts like normal. So that should help, that most don't need to do anything and can commit / PR as normal.
Yer it doesn't have to be everything, but we could just keep it to simple things, like spell checking. There are some built in actions for some simpler stuff. Mainly I was just thinking of ways to avoid the cycle of, review, label then it fails for a spelling or formatting issue.... and then it needs reviewed and labelled again.
Also thank you for seeing this through! :)


Replace pkg/units with the docker/go-units package

As mentioned here, the units library in docker/pkg will be replaced by the go-units.
Note This update isn't supposed to change any functionality of the package.

Oh, thanks!
ping @calavera :-)

cc @calavera
re-review, please? :smile_cat:

this has just been changed in Docker and is now breaking builds of dockerclient

I'd be soooo happy to see this land. This just became an issue for us.

LGTM

LGTM but I don't have access to merge in this repo

Thanks @ripcurld00d!


feat: add useCancelListing (#154)

Current dependencies on/for this PR:

main

PR #153

PR #155

PR #156

PR #157

PR #158   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: LhE62Pbrj0P1k6gpAemc

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: W6it8zzsk3uVZvwrcgQc


Revert "[update] removes .travis.yml"

Reverts dollarshaveclub/ember-router-scroll#114

@yowainwright I need approvals of these reverts before I can merge, if you don't mind please.


MemoryError when calling to_categorical(y)

I recently hit a MemoryError when using to_categorical(y) (with a big dataset, 'y.shape = (10000,256,256,4)', 16 categories). The error was in the line np.zeros((n, num_clases)), which, according to the numpy docs, defaults to numpy.float64.
I fixed the issue by calling np.zeros((n, num_clases), dtype=numpy.int8) instead. It could also be numpy.bool. Could there be any unforeseen side effects of this? Should I submit a pull request?

The MemoryError is caused because you are loading the dataset into your RAM and there is no more memory in your RAM.

I know, my point is that the datatype is much heavier than necessary, and I
should be able to load that dataset into a 48GB RAM.
On Jul 2, 2017 8:28 PM, "abrad1212"<EMAIL_ADDRESS>wrote:
The MemoryError is caused because you are loading the dataset into your RAM
and there is no more memory in your RAM.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/7167#issuecomment-312541741, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADH-4eWHYPBTlTcqbdltSh6fMIZ_aKrmks5sKF_3gaJpZM4OIq0a
.


Parameterize cache key to support calls with different inputs

Right now, CoroutineCache doesn't support caching similar data requests with different parameters.
Lets say, we are caching a search request for Products with string parameter 'productName'. Any such request would be cached under the same key, that was set in  @ProviderKey("ProductsSearch"). So we may encounter following situation:

Search Products with 'snacks' substring;
This request is being stored under "ProductsSearch" cache;
Search Products with 'water' substring;
Instead of making actual call, CoroutineCache will return results stored for 'snacks' request.

I suggest to add another lambda into getData signature, that will be written by CoroutinesCache user and that will do required modifications to @ProviderKey based on input parameters.
Example:
interface CacheProviders {

    @ProviderKey("TestKey", EntryClass(Data::class))
    @LifeTime(value = 1L, unit = TimeUnit.MINUTES)
    @Expirable
    @UseIfExpired
    suspend fun getData(dataProvider: suspend () -> Data,
                                      keyModifier: (String) -> String = { s -> s }): Data
}

If such modifier is present, than base key will be modified according to provided lambda. Default  implementation can be created, that takes a list of Objects to calculate overall hash of parameters.

As I found out, it's extremely hard to work with KCallables in such solution, as each parametrized call is converted into generated class with contains a field with parameters. So that it makes a nightmare to call them via reflection.
So I suggest another approach now:
interface DataProvider {
suspend fun getData(): Data
fun parametrizeKey(baseKey: String): String = baseKey
}
interface CacheProviders {
@ProviderKey("TestKey", EntryClass(Data::class))
@LifeTime(value = 1L, unit = TimeUnit.MINUTES)
@Expirable
@UseIfExpired
suspend fun getData(dataProvider: DataProvider): Data

}


Logger doesn't log my custom log

I am having issues changing the splitting method and I want to debug using the logs. I added some custom logs in ingest_service.py like this:
logger.info("SemanticSplitterNodeParser: =%s", dir(node_parser))
These are the default logs in this file which work:
    def ingest_file(self, file_name: str, file_data: Path) -> list[IngestedDoc]:
        logger.info("Ingesting file_name=%s", file_name)
        documents = self.ingest_component.ingest(file_name, file_data)
        logger.info("Finished ingestion file_name=%s", file_name)
        return [IngestedDoc.from_document(document) for document in documents]

Both this logs are execute when I embed a file. So I also tried adding a log like this:
    def ingest_file(self, file_name: str, file_data: Path) -> list[IngestedDoc]:
        logger.info("Ingesting file_name=%s", file_name)
        documents = self.ingest_component.ingest(file_name, file_data)
        logger.info("Finished ingestion file_name=%s", file_name)
        logger.info("LOG TEST=%s", file_name)
        return [IngestedDoc.from_document(document) for document in documents]

Even if I add a log like this it still won't be displayed, only the two default logs are displayed like this:
14:44:44.484 [INFO    ] private_gpt.server.ingest.ingest_service - Ingesting file_names=['file.pdf']
14:44:49.258 [INFO    ] private_gpt.server.ingest.ingest_service - Finished ingestion file_name=['file.pdf']
I run privategpt with:
PGPT_PROFILES=vllm make run
Thanks for any help in advance!

How did you sort this out?


Bad Gradle include snippet

I suggest to replace this
repositories {
    mavenCentral()
}

dependencies {
    compile ("me.lucko.luckperms:luckperms-api:4.3")
}

with this
repositories {
    mavenCentral()
}

dependencies {
    implementation 'me.lucko.luckperms:luckperms-api:4.3'
}

implementation configuration is faster than deprecated compile.
There's no reasons to use GStrings when you don't need ${...} pattern syntax.
Braces are not necessary.

You could always make a pull request, to make the changes then Luck can decide if the changes are good or not. Would be quicker than using an issue.

It's for wiki. Can I PR it?

Yeah you can, i've done it before.

There's a repository containing all wiki pages.
You can contribute to that, to suggest changes. I sadly don't have a link, but it's either one of Luck's repos or one of the LuckPerms organisation.

ok, i'll try to find

It's here: https://github.com/LuckPerms/wiki
I'm fine with the changes, but don't really know enough about the implementation configuration to say if it's valid or not.
@kashike almighty gradle master, any opinion?

@lucko
https://docs.gradle.org/current/userguide/java_library_plugin.html#sec:java_library_separation

@lucko Yes, api and implementation should be used instead of compile now.

I've updated the wiki page, thanks for the suggestion. :)


Topic detail with consumer offsets

Hi,
we extended the existing topic detail page. We added a "More details..." button that expands the topic detail page and adds a table with current offsets for all consumers on a given topic. Now the topic detail page not only displays consumer names but it also displays current offsets for those consumers on a single page.
This extension proved to be useful for us while we were migrating consumers from old topic to a new topic and therefore we wanted to monitor old topic until we saw no lag on any of those consumers. On a  new topic we wanted to monitor that all consumers now switched to it are just fine and happily consuming messages. Although the same could be achieved in original state it was crucial for us to see those details on a single page at the same time.
Technically we extracted respective part of group.html into group.html.inc and included that part into both group.html and a new page topic-consumers.html which is now opened from topic-detail.html. We also splitted consumers on a page accoring to their activity so in the end we display two tables - one for active consumers and one for inactive consumers.
Screenshot of a changed page is here:
Screenshot 1 - Topic detail page with a new button More details...
Screenshot 2 - The page reloaded after clicking More details ...

This looks awesome, I like some of the refactor too. Can one of you sent us a copy of the contributor agreement linked here:
https://github.com/quantifind/KafkaOffsetMonitor#contributing
sadly, I am not allowed to merge without that.

Thank you for preaccepting the request. I will try to process the necessary contributor agreement within our company as soon as possible.

Hi there,
Please find the two licence agreements signed by us as contributors in<EMAIL_ADDRESS>Thank you for your patience.

excellent, the code looks good to me, I want to test it and then I'll merge the PR next week.


Add SSL checks for OVN

Since OVN also uses SSL, we should check those certificates.

lets revisit this because ... https://bugs.launchpad.net/charm-ovn-central/+bug/1979539


Make it run on 2022 and 2023 datasets; and sign(genWeight) scaling

As the title suggests, this PR is for enabling 2022 and 2023 runs.
Both years are split in two periods with the following year parameters to distinguish them:
2022_preEE, 2022_postEE, 2023_preBPix, 2023_postBPix
In addition to that, there are incompatibilities of the NanoAOD variables between Run3 and Run2.
In particular, Jet.puID does not exist. To avoid the crash I create a new parameter, params['run_period']='Run3' which is used to distinguish a Run3 processor.
There is also incompatible Electron id variable: id: mvaFall17V2Iso_WP80  for Run2 and  id: mvaIso_WP80  for Run3. I did not think of anything better than to change it manually when running over Run3. But this should be dealt with in a better way somehow.
Other changes in this PR are some improvements for the old runner script.
The PR is ready for review.

Pushed another commit to solve the #130

Thanks @andreypz having a look!

Yes, we will test the new runner and implement the modifiers needed to run at RWTH Aachen cluster. One of our new student is looking into this.


MM-8777: Highlighting usernames on exact mention

Summary
When you have a username with ., - or _, wasn't well managed by the
highlighting library. I improved the regular expression on maching usernames
after the @ sign. And adding the highlight of the not @ signed version of the
username.
I'm not if here I'm breaking other expected behaviors.
Ticket Link
MM-8777
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed

I think the ticket here is incorrect. @user.name should be highlighted in this case, but none of user.name should be highlighted since the search is for @user.name specifically

Ok, but the search results return me user.name results too. So I have highlighted both.

Here I think the behavior must be consistent between back and front, if we return results for user.name and @user.name we must highlight both, If we only highlight the @user.name string, we must return only @user.name results. But I'm only 3/5 about it, because of course, it depends on the UX decision made for this.

@jasonblais Could you validate MM-8777 ticket and let us know what is the expected behavior? Thanks!

@saturninoabril @jespino I tested and in the scenario given in the ticket, user.name doesn't trigger mentions. Only @user.name does.
That makes sense, as the user only wants to get mentions for @user.name.
Hence, the expected behaviour is: Only messages containing @user.name are returned in recent mentions search in RHS, and properly highlighted.

Ok, I'm agree with that, but then, the backend is not working properly, because it is returning "user.name" results too, so... is there a backend bug too?

Looks like there's a known search issue, which was closed as won't fix. https://mattermost.atlassian.net/browse/MM-5206
George and Eric would know if this is expected to work on elasticsearch though. I'm not 100% sure what's expected to work/not work with search.

Ok, then, now have the expected behavior, only @user.name is highlighted. But still returning @user.name and user.name results.

@jespino so it's fixed apart from the issue described here (and closed as won't fix)? https://mattermost.atlassian.net/browse/MM-5206

Yes, is not directly related. This PR solve the highlighting in the frontend part. The backend works exactly how was working before. Now, you receive results from user.name and @user.name but only highlights the @user.name texts.

I just checked and we do still return user.name and @user.name when you search @user.name on Elasticsearch (assuming we're using that on pre-release). I'm not sure if there's anything we can do to prevent that though since the logic for determining at mentions is really complicated due to the fact that they can contain sometimes contain periods.
To solve the discrepency between the returned results and the search highlighting, we'd probably need to implement the highlighting based off of the matches in Elasticsearch instead of doing it after the fact like we are now


Ran migrate instead of baseline, got error; now can't run baseline because of history table

Which version and edition of Flyway are you using?
6.0.4, community edition
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using (type & version)?
SQL Server 2016
Which operating system are you using?
Windows 10
What did you do?
First I tried a migration, which failed, because I didn't create a baseline before. After that I ran the baseline command.
What did you expect to see?
The baseline should be stored. This worked like that in version 5.2.4
What did you see instead?
Baseline not stored in the table.

The problem is not related to the migration. To reproduce what I meant, then just take an empty database and run a baseline. Then delete the only row from the flyway table and run the same baseline again.
I would expect that the baseline would add one row to the flyway table, but it doesn't. The flyway table stays empty.

OK, thankyou. So the specific problem is that because the migration failed, it left an empty schema history table which could then not be baselined? If that's so then I understand the problem now. We will need to carefully consider a behaviour change, as a migration could potentially fail part way through and a subsequent baseline might be misleading.

Yes, you are correct. And as stated, this behavior was different (and according to me correct) in 5.2.4.

@kippm You say:

The problem is not related to the migration. To reproduce what I meant, then just take an empty database and run a baseline. Then delete the only row from the flyway table and run the same baseline again. I would expect that the baseline would add one row to the flyway table, but it doesn't.

Manually altering the schema history table is likely to cause Flyway to enter an inconsistent state. Therefore we can't really support workflows that involve such manual alterations.
In this scenario, you've manually altered the schema history such that it's empty. In this case the Baseline command doesn't do anything. I think this is the correct behavior. All you need to do is run clean, or manually delete the schema history table, and run baseline again.
As much as it is a behavior change between Flyway versions it doesn't seem particularly significant to me.

Actually we'll make baseline produce an exception in this case to make it more clear that baselining an empty schema table is invalid.


Crash in background

Scoop caught this error today. No idea when it happened, but the device was lying idle for a long time when I picked it up and noticed this crash.
Apps is at version 12.
FATAL EXCEPTION: main
Process: app.grapheneos.apps, PID: 18896
java.lang.ExceptionInInitializerError
	at app.grapheneos.apps.util.UtilsKt.a(Unknown Source:4)
	at app.grapheneos.apps.autoupdate.AutoUpdateJob.onStartJob(Unknown Source:12)
	at android.app.job.JobService$1.onStartJob(JobService.java:62)
	at android.app.job.JobServiceEngine$JobHandler.handleMessage(JobServiceEngine.java:105)
	at android.os.Handler.dispatchMessage(Handler.java:106)
	at android.os.Looper.loopOnce(Looper.java:201)
	at android.os.Looper.loop(Looper.java:288)
	at android.app.ActivityThread.main(ActivityThread.java:7870)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003)
Caused by: j3.h: lateinit property baseAppContext has not been initialized
	at app.grapheneos.apps.core.GlobalsKt.<clinit>(Unknown Source:195)
	... 11 more

cc @muhomorr

@opusforlife2 By "Scoop" you mean https://github.com/TacoTheDank/Scoop ?
Which device and OS version are you on?

Unable to reproduce this issue.
baseAppContext is unconditionally initialized in ApplicationImpl,attachBaseContext() which should always be the first app method that is executed by the OS when the app is being instantiated.

There's an AOSP bug that would be able to cause this issue:
https://issuetracker.google.com/issues/160946170
Do you have a an auto-backup app or service installed?

@opusforlife2 Yes


setgroups broken

$ cat /tmp/apt/Dockerfile
FROM debian
RUN for f in uid_map gid_map setgroups;do echo "#$f"; cat /proc/self/$f; done
RUN apt update

$ docker run -it --rm -v /tmp/apt:/foo -w /foo -v /tmp --privileged img build -t foo .
Building docker.io/library/foo:latest
Setting up the rootfs... this may take a bit.
RUN [/bin/sh -c for f in uid_map gid_map setgroups;do echo "#$f"; cat /proc/self/$f; done]
--->
#uid_map
         0          0          1
         1          1      65536
#gid_map
         0          0          1
         1          1      65536
#setgroups
deny
<--- r3942e87odgx0tu831zsxrxlv 0 <nil>
RUN [/bin/sh -c apt update]
--->

WARNING: apt does not have a stable CLI interface. Use with caution in scripts.

E: setgroups 65534 failed - setgroups (1: Operation not permitted)
E: setgroups 0 failed - setgroups (1: Operation not permitted)
Reading package lists...
E: setgroups 65534 failed - setgroups (1: Operation not permitted)
E: setgroups 0 failed - setgroups (1: Operation not permitted)
E: Method gave invalid 400 URI Failure message: Failed to setgroups - setgroups (1: Operation not permitted)
E: Method gave invalid 400 URI Failure message: Failed to setgroups - setgroups (1: Operation not permitted)
E: Method http has died unexpectedly!
E: Sub-process http returned an error code (112)
<--- quk7snb2cmsx3llpmpeeogix2 100 <nil>
solving failed: executor failed running [/bin/sh -c apt update]: exit code 100

Seems failing this check https://github.com/jessfraz/runc/blob/d5cb71bbe08dc3ca135837187e281af5f336fd1c/libcontainer/container_linux.go#L1806

https://github.com/AkihiroSuda/runc/commit/343b7bc4a662c7cac6829562cd13fc019aef6d64 works for me
@jessfraz Could you cherry-pick the commit to your fork?
@cyphar Is this commit safe to be proposed to the upstream? I'm not sure why this check exists.

Will do!!!
On Mon, May 21, 2018 at 23:59 Akihiro Suda<EMAIL_ADDRESS>wrote:

AkihiroSuda/runc@343b7bc
https://github.com/AkihiroSuda/runc/commit/343b7bc4a662c7cac6829562cd13fc019aef6d64
works for me
@jessfraz https://github.com/jessfraz Could you cherry-pick the commit
to your fork?
@cyphar https://github.com/cyphar Is this commit safe to be proposed to
the upstream? I'm not sure why this check exists.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/genuinetools/img/issues/96#issuecomment-390883690,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYNbFW_37KcKUV_aeoH_NSr8crxLRBCks5t07dIgaJpZM4UH5Xo
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

updated and triggered a new build of the image :)

note to self to add e2e testing for the image :)

@AkihiroSuda From memory the logic was that you need to have CAP_SETGID in order to be able to set setgroups to allow. I'm not sure why that code was there in the first place -- this assumption isn't true and never has been. Not to mention that it doesn't make sense given the newgidmap support. :+1: to submitting a patch that removes it.

@cyphar Thanks, added the commit to https://github.com/opencontainers/runc/pull/1688

Opened https://github.com/genuinetools/img/pull/98 for adding test


WindowOverflow4 at ??:?

I see this line:
0x40080000: _WindowOverflow4 at ??:?
repeated twice toward the beginning of the device boot.
What does this mean? Is this going to break something later?
This occurs even when I use the default partition table.
Flashing binaries to serial port /dev/tty.SLAB_USBtoUART (app at offset 0x10000)...
esptool.py v2.0-beta2
Connecting........_____....._____....._____.....__
Uploading stub...
Running stub...
Stub running...
Changing baud rate to 921600
Changed.
Attaching SPI flash...
Configuring flash size...
Auto-detected Flash size: 4MB
Wrote 16384 bytes at 0x0000e000 in 0.2 seconds (657.2 kbit/s)...
Hash of data verified.
Flash params set to 0x0220
Wrote 16384 bytes at 0x00001000 in 0.2 seconds (650.8 kbit/s)...
Hash of data verified.
Wrote 802816 bytes at 0x00010000 in 10.9 seconds (590.7 kbit/s)...
Hash of data verified.
Wrote 16384 bytes at 0x00008000 in 0.2 seconds (661.4 kbit/s)...
Hash of data verified.

Leaving...
Hard resetting...
MONITOR
--- idf_monitor on /dev/tty.SLAB_USBtoUART 115200 ---
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
ets Jun  8 2016 00:22:57

rst:0x1 (POWERON_RESET),boot:0x17 (SPI_FAST_FLASH_BOOT)
ets Jun  8 2016 00:22:57

rst:0x10 (RTCWDT_RTC_RESET),boot:0x17 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0x00
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0008,len:8
load:0x3fff0010,len:2488
load:0x40078000,len:7128
load:0x40080000,len:252
>>>0x40080000: _WindowOverflow4 at ??:?
entry 0x40080034
>>>0x40080034: _WindowOverflow4 at ??:?
I (1214) cpu_start: Pro cpu up.
I (1215) cpu_start: Single core mode
I (1216) heap_alloc_caps: Initializing. RAM available for dynamic allocation:
I (1229) heap_alloc_caps: At 3FFAFF10 len 000000F0 (0 KiB): DRAM
I (1249) heap_alloc_caps: At 3FFC90B8 len 00016F48 (91 KiB): DRAM
I (1270) heap_alloc_caps: At 3FFE0440 len 00003BC0 (14 KiB): D/IRAM
I (1291) heap_alloc_caps: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (1313) heap_alloc_caps: At 4009B824 len 000047DC (17 KiB): IRAM

What does this mean? Is this going to break something later?

Short answer: It's not a problem, it's an unintended consequence of the new idf_monitor tool.
Long answer:
Anything which is printed that looks like a code address is looked up to a symbol/function/code name in the ELF file.
In this case:
load:0x40080000,len:252
>>>0x40080000: _WindowOverflow4 at ??:?
entry 0x40080034
>>>0x40080034: _WindowOverflow4 at ??:?

Are the section load & entry pointer addresses, printed by the bootloader. idf_monitor sees the addresses looks them up (via addr2line) for symbol/function names and finds the "function" _WindowOverflow4 (an Xtensa exception vector) and ??:? for the source file and line. None of which is very useful!
Probably we can find a way print something less ominous sounding here (possibly by injecting a symbol at the beginning of the ELF file.)

Ok, thanks for the quick reply!
I thought it was something wrong with my partition scheme.
BTW loving the inline stack trace decoding.


Add NotebookIndex to the cache

Summary
This PR updates the FileCache to include an optional NotebookIndex to support caching for Jupyter Notebooks.
We only require the index to compute the diagnostics and thus we don't really need to store the entire Notebook on the Diagnostics struct. This means we only need the index to be stored in the cache to reconstruct the Diagnostics.
Test Plan
Update an existing test case to run over the fixtures under ruff_notebook crate where there are multiple Jupyter Notebook.
Locally, the following commands were run in order:

Remove the cache: rm -rf .ruff_cache
Run without cache: cargo run --bin ruff -- check --isolated crates/ruff_notebook/resources/test/fixtures/jupyter/unused_variable.ipynb --no-cache
Run with cache: cargo run --bin ruff -- check --isolated crates/ruff_notebook/resources/test/fixtures/jupyter/unused_variable.ipynb
Check whether the .ruff_cache directory was created or not
Run with cache again and verify: cargo run --bin ruff -- check --isolated crates/ruff_notebook/resources/test/fixtures/jupyter/unused_variable.ipynb

fixes: #6671

Current dependencies on/for this PR:

main

PR #6863   👈

This comment was auto-generated by Graphite.

Does the cached output now match the uncached output?

Yes, I've expanded the existing test case to test against the fixtures in ruff_notebook crate which contains a number of notebooks. I've also tested locally using the mentioned steps in the PR description and the output match.

How does the performance between cached and uncached notebooks compare?

Size
With a test directory containing a few repositories with Jupyter notebooks:
$ tokei ../test-repos/notebooks -t "Jupyter Notebooks"
===============================================================================
 Language            Files        Lines         Code     Comments       Blanks
===============================================================================
 Jupyter Notebooks    1221            0            0            0            0
 |- Markdown           272        14168          422        10048         3698
 |- Python             273        47807        37056         4219         6532
 (Total)                          61975        37478        14267        10230
===============================================================================
 Total                1221            0            0            0            0
===============================================================================

The disk size has increased because of the addition of notebook index stored in the cache:
$ du -h -d 1 ../cache-test
 12M	../cache-test/no-notebook-index
 14M	../cache-test/with-notebook-index

Performace
With the following ruff.toml:
include = ["*.ipynb"]

Earlier, the output of a cached run on Jupyter Notebooks were incorrect because the cell information was not stored in the cache. With that in mind, the benchmarks are as follows:

Uncached vs Cached runs on Jupyter Notebooks only file set:

$ hyperfine --warmup 5 \
  "./target/release/ruff check -e --config ./ruff.toml --no-cache --select=ALL ../test-repos/notebooks" \
  "./target/release/ruff check -e --config ./ruff.toml --cache-dir ../cache-test/notebook-index --select=ALL ../test-repos/notebooks"
Benchmark 1: ./target/release/ruff check -e --config ./ruff.toml --no-cache --select=ALL ../test-repos/notebooks
  Time (mean ± σ):      1.127 s ±  0.073 s    [User: 6.342 s, System: 0.391 s]
  Range (min … max):    1.018 s …  1.246 s    10 runs

Benchmark 2: ./target/release/ruff check -e --config ./ruff.toml --cache-dir ../cache-test/notebook-index --select=ALL ../test-repos/notebooks
  Time (mean ± σ):     192.9 ms ±   3.8 ms    [User: 277.7 ms, System: 138.7 ms]
  Range (min … max):   185.8 ms … 198.9 ms    15 runs

Summary
  ./target/release/ruff check -e --config ./ruff.toml --cache-dir ../cache-test/notebook-index --select=ALL ../test-repos/notebooks ran
    5.84 ± 0.40 times faster than ./target/release/ruff check -e --config ./ruff.toml --no-cache --select=ALL ../test-repos/notebooks

The numbers are in line with non-Jupyter Notebook dataset

Cached with and without NotebookIndex in the cache:

$ hyperfine --warmup 5 \
  "./../ruff/target/release/ruff check -e --config ./ruff.toml --cache-dir ../cache-test/ruff-old --select=ALL ../test-repos/notebooks" \
  "./target/release/ruff check -e --config ./ruff.toml --cache-dir ../cache-test/ruff-new --select=ALL ../test-repos/notebooks"
Benchmark 1: ./../ruff/target/release/ruff check -e --config ./ruff.toml --cache-dir ../cache-test/ruff-old --select=ALL ../test-repos/notebooks
  Time (mean ± σ):     181.6 ms ±   3.6 ms    [User: 261.9 ms, System: 134.0 ms]
  Range (min … max):   177.5 ms … 188.5 ms    16 runs

Benchmark 2: ./target/release/ruff check -e --config ./ruff.toml --cache-dir ../cache-test/ruff-new --select=ALL ../test-repos/notebooks
  Time (mean ± σ):     188.6 ms ±   4.3 ms    [User: 276.8 ms, System: 134.8 ms]
  Range (min … max):   181.7 ms … 197.1 ms    15 runs

Summary
  ./../ruff/target/release/ruff check -e --config ./ruff.toml --cache-dir ../cache-test/ruff-old --select=ALL ../test-repos/notebooks ran
    1.04 ± 0.03 times faster than ./target/release/ruff check -e --config ./ruff.toml --cache-dir ../cache-test/ruff-new --select=ALL ../test-repos/notebooks

Here, we can see that the performance has gone down a bit because of the new NotebookIndex which needs to be cloned from the cache to be used on Diagnostics. Both FileCache and Diagnostics owns the NotebookIndex which is the reason for cloning.


Extension Not Found in Mozilla Firefox

Hi,
Thank you so much for this extension.
I just tried adding it to Firefox but I was directed to a page that said the extension is no longer available.
Have you removed the extension from Firefox?

@SimeonOA I have not removed it, but perhaps firefox took it down for some reason.

Same here, can't find it on the firefox add-ons website anywhere.

Here it is!
https://addons.mozilla.org/en-US/firefox/addon/twitter-bookmarks-search/
I don't know why it doesn't show in search


Add bot for updating Galata tests automatically

Adding a comment containing "update galata references" will trigger an action that pushes a commit to the branch with the new test references.

Tested in https://github.com/martinRenou/bqplot/pull/4

cc. @ibdafna

meeseeksdev please backport to 0.12.x


Fixed Backpacks, Vests

Tell me WHO DID THIS

Thank fuck!

Also I don't know what the fluff is this but can you change the single line if's to still have no brackets to be on two lines because this is really not readable


Use monai-weekly throughout

Since the pep8 refactor, we check if monai is installed, and install it if not:
!python -c "import monai" || pip install -q "monai[pillow, tqdm]"

We should really be using a more up-to-date version of monai, so should probably switch to using monai-weekly throughout:
!python -c "import monai" || pip install -q "monai-weekly[pillow, tqdm]"

addressed by https://github.com/Project-MONAI/tutorials/pull/133 please file new tickets for any follow-up


Is this project still maintained?

Summary
Looking at the contributor stats it looks like development has slowed down. Should I still depend on this repository for my project?
Additional Details
I was caught off guard by a severe memory leak issue raised more than a year back. This issue forced me to refactor out any and all Result type responses from my code. I don't see it getting fixed anytime soon. Should I be worried?

Development of this project has slowed, and I am the only maintainer. I still try to merge small features and bug fixes, and you can look at the commit history and decide whether that's active enough for you. Fundamental issues like the one you pointed out I don't have time to work.


libglm-dev is needed to compile (Linux, Debian)

The libglm-dev package is needed on Debian to build
See the following.
Build Failed
you > make
[  1%] Building CXX object base/CMakeFiles/base.dir/VulkanDebug.cpp.o
In file included from /bench/github_local/DOWN_ONLY/Vulkan/base/VulkanDebug.cpp:9:
/bench/github_local/DOWN_ONLY/Vulkan/base/VulkanDebug.h:23:10: fatal error: glm/glm.hpp: No such file or directory
 #include <glm/glm.hpp>
          ^~~~~~~~~~~~~
compilation terminated.
make[2]: *** [base/CMakeFiles/base.dir/build.make:141: base/CMakeFiles/base.dir/VulkanDebug.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:91: base/CMakeFiles/base.dir/all] Error 2
make: *** [Makefile:84: all] Error 2

install libglm-dev
you > sudo apt-file search glm.hpp
libglm-dev: /usr/include/glm/glm.hpp

you > dpkg -L libglm-dev
dpkg-query: package 'libglm-dev' is not installed
Use dpkg --contents (= dpkg-deb --contents) to list archive files contents.

you > sudo aptitude install libglm-dev
The following NEW packages will be installed:
  libglm-dev
0 packages upgraded, 1 newly installed, 0 to remove and 33 not upgraded.
Need to get 175 kB of archives. After unpacking 2,187 kB will be used.
Get: 1 http://ftp.citylink.co.nz/debian buster/main amd64 libglm-dev all <IP_ADDRESS>-2 [175 kB]
Fetched 175 kB in 0s (1,135 kB/s)
(Reading database ... 389474 files and directories currently installed.)
Preparing to unpack .../libglm-dev_<IP_ADDRESS>-2_all.deb ...
Unpacking libglm-dev (<IP_ADDRESS>-2) ...
Setting up libglm-dev (<IP_ADDRESS>-2) ...
======  How can you help?  (doc: https://wiki.debian.org/how-can-i-help ) ======

New packages where help is needed, including orphaned ones (from WNPP):
 - glm - https://bugs.debian.org/963901 - O (Orphaned)

Build successfully
you > make
[  1%] Building CXX object base/CMakeFiles/base.dir/VulkanDebug.cpp.o
[  1%] Building CXX object base/CMakeFiles/base.dir/VulkanDevice.cpp.o
[  2%] Building CXX object base/CMakeFiles/base.dir/VulkanSwapChain.cpp.o
[  2%] Building CXX object base/CMakeFiles/base.dir/VulkanTexture.cpp.o
[  3%] Building CXX object base/CMakeFiles/base.dir/VulkanTools.cpp.o
[  4%] Building CXX object base/CMakeFiles/base.dir/VulkanUIOverlay.cpp.o
[  4%] Building CXX object base/CMakeFiles/base.dir/VulkanglTFModel.cpp.o
[  5%] Building CXX object base/CMakeFiles/base.dir/vulkanexamplebase.cpp.o
[  5%] Building C object base/CMakeFiles/base.dir/__/external/ktx/lib/texture.c.o
[  6%] Building C object base/CMakeFiles/base.dir/__/external/ktx/lib/hashlist.c.o
[  7%] Building C object base/CMakeFiles/base.dir/__/external/ktx/lib/checkheader.c.o
[  7%] Building C object base/CMakeFiles/base.dir/__/external/ktx/lib/swap.c.o
[  8%] Building C object base/CMakeFiles/base.dir/__/external/ktx/lib/memstream.c.o
[  8%] Building C object base/CMakeFiles/base.dir/__/external/ktx/lib/filestream.c.o
[  9%] Linking CXX static library libbase.a
[ 12%] Built target base
Scanning dependencies of target viewportarray
[ 13%] Building CXX object examples/CMakeFiles/viewportarray.dir/viewportarray/viewportarray.cpp.o
[ 13%] Linking CXX executable ../bin/viewportarray
      :
      :
      :
[ 98%] Building CXX object examples/CMakeFiles/subpasses.dir/subpasses/subpasses.cpp.o
[ 99%] Linking CXX executable ../bin/subpasses
[ 99%] Built target subpasses
Scanning dependencies of target terraintessellation
[100%] Building CXX object examples/CMakeFiles/terraintessellation.dir/terraintessellation/terraintessellation.cpp.o
[100%] Linking CXX executable ../bin/terraintessellation
[100%] Built target terraintessellation

This can be closed with the addition of libglm-dev to the build requirements on Linux.
Prob should be tagged "Linux"

It's included as a git submodule. If you didn't clone with git clone --recursive as instructed, run git submodule init and git submodule update.

As noted by @eloj you need to clone the sub modules.


CMake Package Config support added

see https://github.com/jrl-umi3218/jrl-cmakemodules/pull/103

this is failing because the  jrl-umi3218/jrl-cmakemodules branch needs to be merged first so the submodule clone is failing, thus the cmake run won't work right

@gergondet do you know what's going on with this travis ci error? It seems to fail to check out a subrepository:

Submodule path 'tests/shared-tests': checked out '3b1a5b76465dcb7135b6dd30c6cb705df42ec817'
Unable to checkout '5e0e3a97b748aec181d01b666db44c938042a335' in submodule path 'cmake'
The command "eval git submodule update --init --recursive" failed. Retrying, 2 of 3.

It indicates you pushed the wrong reference for the cmake submodule. (indeed 5e0e3a97b748aec181d01b666db44c938042a335 is not a valid reference but it is one for your fork)
My best guess is to do something like that in your local sch-core copy:
git submodule sync
cd cmake
git remote update
git checkout origin/master
cd ../
git commit -m 'Fix cmake submodule'

okay hopefully fixed now!

Coverage remained the same at 56.888% when pulling e8f5e4d3b6ecfadc69dd273c878c5992c4c1c68c on ahundt:package into 5c3796f51ed7f0ef37a7add338e1b41c8fe06f7d on jrl-umi3218:master.

Coverage remained the same at 56.888% when pulling e8f5e4d3b6ecfadc69dd273c878c5992c4c1c68c on ahundt:package into 5c3796f51ed7f0ef37a7add338e1b41c8fe06f7d on jrl-umi3218:master.

Coverage decreased (-0.5%) to 56.379% when pulling da3c5cbc6625673e34cfa6ec2ed9f43acf91a4f6 on ahundt:package into 5c3796f51ed7f0ef37a7add338e1b41c8fe06f7d on jrl-umi3218:master.

@gergondet can this be merged?


Press and hold keys on OSX causes suggestions popup

OS version and name: OSX 10.12.1
Hyper.app version: <IP_ADDRESS>3
Relevent information from devtools (CMD+SHIFT+I on Mac OS, CTRL+SHIFT+I elsewhere): NA
The issue is reproducible in vanilla Hyper.app: Yes

Issue
Just updated to 1.0.0, now when I'm in neovim if i press and hold down the 'L' key instead of repeating the movements, it also flickers the popup to enter 'ł'. It also seems to affect all keys as pressing and holding jkli keys seems sluggish. Was fine before this and is still fine in iterm. See attached gif which captures a subset of the popups flashing

@matheuss  Related to hyperCaret

Same issue here. Experienced on the same OS inside normal vim.

Noticed this too. Anyone have any ideas as to why this would happen here and not in iTerm? You can also disable this on a per app basis. https://gist.github.com/rastasheep/bfc8266eeb58b899054c

We could probably use this. Maybe in a plugin? https://github.com/electron/electron/pull/8084

Yeah, this is pretty annoying. Solved it by running
defaults write co.zeit.hyper ApplePressAndHoldEnabled -bool false

This should be make in to a plugin.

systemPreferences.setUserDefault from electron can be used to disable ApplePressAndHoldEnabled as @xvalentino pointed out.
However I think it should be part of core rather than a plugin. Accent pop-up is designed for input fields where caret doesn't move when user holds a key. Hyper caret behaviour is different, it always moves, so there's no point to have the pop-up window as it appears and disappears without letting user to select an accent.

This is still happening with Hyper 2.0 on Mojave. Is there a reason this was closed and abandoned?

@sukima This was fixed in 2017 but broke again recently. Looks like @chabou fixed it a few days ago, see https://github.com/zeit/hyper/issues/3317 This fix should be live whenever v2.1.1 comes out.

This still seems to be a real issue in Hyper 3.x.x

Also seeing this in Hyper 3.0.2


When the code will be made public, and whether the parameter of Bert will change during training

Hi,
Thanks for the interesting work.
May I ask when the code will be made public, and whether the parameter of Bert will change during training?
Thanks.

Hi,
We plan to release the code in late January. All parameters are tuned during training.
Hope this helps!


https mixed content errors

First off, this is an amazing library. I created a local server and was able to quickly start visualizing large datasets with MapLibre GL JS in-house. The repo documentation was excellent in getting this done.
When I tried this on a public-facing site with an SSL certificate, the Martin tileset server uses HTTP and is blocked by the browser.
My public setup uses a Digital Ocean droplet with a Let's Encrypt SSL certificate. PostgreSQL uses these certificates, and Martin is able to connect to the database using the ssl_cert in the config options and ?sslmode=require in the connection string.
I can get a list of sources at http://domain.org:3000/catalog, and the tilesets can be accessed at http://domain.org:3000/{table_name}.
However, if I try to access these over HTTPS, the browser returns an ERR_SSL_PROTOCOL_ERROR. If I use HTTP, it's a Mixed Content error.
Hopefully, I'm missing something simple here. Any insight would be greatly appreciated. If this issue can be explained in the docs, I'd be happy to submit a PR.

Hi @boydx, it clearly sounds like something unrelated to Martin itself - as we never have handled HTTPS traffic. I am sure Actix lib could be made to handle the TLS termination properly, but this is outside of the scope for Martin itself, at least for now. Most of the time, these tasks are done with NGINX wrapper, or some other such mechanism.


Could not resolve dependency

hrtorbov@Hristo-Torbovs-MacBook gigs.com % npx react-native init mobile --template @ui-kitten/template-ts

               ######                ######
             ###     ####        ####     ###
            ##          ###    ###          ##
            ##             ####             ##
            ##             ####             ##
            ##           ##    ##           ##
            ##         ###      ###         ##
             ##  ########################  ##
          ######    ###            ###    ######
      ###     ##    ##              ##    ##     ###
   ###         ## ###      ####      ### ##         ###
  ##           ####      ########      ####           ##
 ##             ###     ##########     ###             ##
  ##           ####      ########      ####           ##
   ###         ## ###      ####      ### ##         ###
      ###     ##    ##              ##    ##     ###
          ######    ###            ###    ######
             ##  ########################  ##
            ##         ###      ###         ##
            ##           ##    ##           ##
            ##             ####             ##
            ##             ####             ##
            ##          ###    ###          ##
             ###     ####        ####     ###
               ######                ######

                  Welcome to React Native!
                 Learn once, write anywhere

✔ Downloading template
✔ Copying template
✔ Processing template
✖ Installing dependencies
error Error: Command failed: npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react-native-svg
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@ui-kitten/components
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/hrtorbov/.npm/eresolve-report.txt for a full report.

npm ERR!      /Users/hrtorbov/.npm/_logs/2020-11-21T10_24_16_881Z-debug.log

I think that's because node@15. When I downgraded to @14 it worked.

I think that's because node@15. When I downgraded to @14 it worked.

You have dependency conflict (incorrect and potentially broken dependency) as it says, So try to to run the command with --force, or --legacy-peer-deps.
Try this command:
npm install --save --legacy-peer-deps

You have dependency conflict (incorrect and potentially broken dependency) as it says, So try to to run the command with --force, or --legacy-peer-deps.
Try this command:
npm install --save --legacy-peer-deps

You have dependency conflict (incorrect and potentially broken dependency) as it says, So try to to run the command with --force, or --legacy-peer-deps.
Try this command:
npm install --save --legacy-peer-deps

I doubt it. They might have fixed it now, but when I tried it back then it was on a brand new machine, freshly installed everything. I'll close the issue since I'm not sure if it's fixed already or not.

You have dependency conflict (incorrect and potentially broken dependency) as it says, So try to to run the command with --force, or --legacy-peer-deps.
Try this command:
npm install --save --legacy-peer-deps

I doubt it. They might have fixed it now, but when I tried it back then it was on a brand new machine, freshly installed everything. I'll close the issue since I'm not sure if it's fixed already or not.

Hi @doanxuanhung1997 @volkan-a @artyorsh
I am getting the error of the unable to resolve dependency tree
node_modules/semantic-release
npm ERR<EMAIL_ADDRESS>from the root project
and I tried the above command of the npm install --save --legacy-peer-deps and also npm install --force, and npm cache but still i get the same error not sure how to solve it
Kindly help.. attaching the screen shot

Also the package.json file
{
"name": "sbc.clientmanagement.ui",
"version": "0.0.0-dev",
"repository": {
"type": "git",
"url": "https://github.com/Sage/sbc.accountants.clientmanagement.ui.git"
},
"main": "src/Index.tsx",
"dependencies": {
"@axe-core/react": "^4.0.0",
"@flatfile/sdk": "^2.0.0-rc7",
<EMAIL_ADDRESS>"^2.1.4",
<EMAIL_ADDRESS>"^5.1.0",
<EMAIL_ADDRESS>"^1.3.4",
<EMAIL_ADDRESS>"^10.4.0",
<EMAIL_ADDRESS>"^1.8.0",
"@types/resize-observer-browser": "^0.1.5",
"carbon-react": "106.1.4",
"core-js": "^3.8.1",
"cross-env": "^7.0.3",
"dotenv": "^8.2.0",
"file-loader": "^6.2.0",
"formik": "^2.1.5",
"i18next": "^20.3.0",
"identity-obj-proxy": "^3.0.0",
"lodash": "^4.17.21",
"lz-string": "^1.4.4",
"moment": "^2.29.3",
"normalizr": "^3.6.0",
"odata-query": "^6.7.0",
"react": "^16.13.1",
"react-dom": "^16.13.1",
"react-i18next": "^11.9.0",
"react-markdown": "^6.0.3",
"react-router": "6.0.0-beta.3",
"react-router-dom": "6.0.0-beta.3",
"semantic-release": "^17.4.7",
"single-spa-react": "^3.2.0",
"uuid": "^8.3.2",
"waait": "^1.0.5",
"yup": "^0.29.1"
},
"devDependencies": {
"@babel/core": "^7.10.5",
"@babel/preset-env": "^7.14.2",
"@babel/preset-react": "^7.12.0",
"@babel/preset-typescript": "^7.12.7",
"@commitlint/cli": "^12.1.1",
"@commitlint/config-conventional": "^12.1.1",
"@sage/design-tokens": "^2.25.0",
<EMAIL_ADDRESS>"^1.1.0",
"@semantic-release/changelog": "^5.0.1",
"@semantic-release/exec": "^6.0.2",
"@semantic-release/git": "^9.0.0",
"@testing-library/jest-dom": "^5.11.1",
"@testing-library/react": "^11.2.7",
"@testing-library/react-hooks": "^6.0.0",
"@testing-library/user-event": "^13.1.9",
"@types/enzyme": "^3.10.5",
"@types/enzyme-adapter-react-16": "^1.0.6",
"@types/history": "^4.7.9",
"@types/i18n-js": "^3.0.3",
"@types/jest": "^26.0.5",
"@types/jsdom": "^16.2.6",
"@types/lodash": "^4.14.173",
"@types/lz-string": "^1.3.34",
"@types/node-fetch": "^2.5.7",
"@types/react-dom": "^16.9.8",
"@types/react-router-dom": "^5.1.5",
"@types/service_worker_api": "0.0.9",
"@types/single-spa-react": "^3.0.1",
"@types/styled-components": "^5.1.9",
"@types/styled-system": "^5.1.11",
"@types/systemjs": "^6.1.1",
"@types/uuid": "^8.3.0",
"@types/yup": "^0.29.3",
"@typescript-eslint/eslint-plugin": "^4.10.0",
"@typescript-eslint/parser": "^4.28.5",
"acorn": "^8.7.1",
"babel-loader": "^8.1.0",
"clean-webpack-plugin": "^3.0.0",
"commitizen": "^4.2.4",
"conventional-changelog-conventionalcommits": "^4.5.0",
"copy-webpack-plugin": "^9.0.1",
"css-loader": "^5.0.1",
"cz-conventional-changelog": "^3.3.0",
"draft-js": "^0.11.7",
"enzyme": "^3.11.0",
"enzyme-adapter-react-16": "^1.15.2",
"enzyme-to-json": "^3.5.0",
"eslint": "^7.18.0",
"eslint-config-prettier": "^6.11.0",
"eslint-plugin-import": "^2.21.2",
"eslint-plugin-jest": "^24.1.3",
"eslint-plugin-jest-dom": "^3.6.5",
"eslint-plugin-jsx-a11y": "^6.3.1",
"eslint-plugin-prettier": "^3.1.4",
"eslint-plugin-react": "^7.20.0",
"eslint-plugin-react-hooks": "^4.2.0",
"eslint-plugin-testing-library": "^5.0.1",
"fetch-mock": "^9.10.4",
"fs": "0.0.1-security",
"history": "^5.1.0",
"husky": "^4.3.8",
"install": "^0.13.0",
"jest": "^26.1.0",
"jest-teamcity-reporter": "^0.9.0",
"jest-watch-typeahead": "^0.6.1",
"jsdom": "^16.6.0",
"lint-staged": "^10.2.11",
"mini-css-extract-plugin": "^2.4.3",
"mockdate": "^3.0.5",
"msw": "^0.28.1",
"node-fetch": "^2.6.7",
"prettier": "^2.1.2",
"prettier-eslint-cli": "^5.0.0",
"style-loader": "^1.2.1",
"styled-components": "^4.4.1",
"systemjs-webpack-interop": "^2.1.1",
"typescript": "4.1.6",
"webpack": "^5.61.0",
"webpack-cli": "^4.9.1",
"webpack-dev-server": "^4.7.3",
"whatwg-fetch": "^3.0.0"
},
"scripts": {
"compile": "tsc",
"start": "cross-env NODE_ENV=development webpack-dev-server",
"build": "cross-env NODE_ENV=production webpack",
"test": "jest --watchAll=true",
"test:coverage": "npm test -- --coverage --watchAll=false",
"test:debug": "npm test -- --detectOpenHandles",
"test:silent": "npm test -- --silent",
"lint": "npm run compile && prettier --check "./src//*.{ts,tsx}" && eslint "./src//.{ts,tsx}"  --max-warnings 31",
"lint-staged": "lint-staged",
"format": "prettier --write "./src/**/.{ts,tsx}"",
"commit-msg": "commitlint -E HUSKY_GIT_PARAMS",
"commit": "cz",
"prepublishOnly": "npm run build",
"release": "semantic-release"
},
"husky": {
"hooks": {
"pre-commit": "npm run lint-staged",
"commit-msg": "commitlint -E HUSKY_GIT_PARAMS"
}
},
"commitlint": {
"extends": [
"@commitlint/config-conventional"
]
},
"lint-staged": {
"src/**/*.{tsx,ts}": [
"npm run format",
"npm run lint -- --fix"
]
},
"config": {
"commitizen": {
"path": "./node_modules/cz-conventional-changelog"
}
},
"browserslist": {
"production": [
"last 1 chrome version",
"last 1 firefox version",
"last 1 safari version",
"last 1 edge version"
],
"development": [
"last 1 chrome version",
"last 1 firefox version",
"last 1 safari version",
"last 1 edge version"
]
},
"testEnvironment": "jsdom"
}
Also the pacak


Pam's Harvestcraft fails to download, 400 Bad Request

Relevant log output:
java.io.IOException: Error 400 at https://addons.cursecdn.com/files/2428/363/Pam's HarvestCraft 1.10.2j.jar
	at com.github.franckyi.cmpdl.task.CustomTask.getLocation(CustomTask.java:55)
	at com.github.franckyi.cmpdl.task.CustomTask.crawl(CustomTask.java:41)
	at com.github.franckyi.cmpdl.task.CustomTask.crawl(CustomTask.java:42)
	at com.github.franckyi.cmpdl.task.DownloadModsTask$DownloadModTask.call0(DownloadModsTask.java:66)
	at com.github.franckyi.cmpdl.task.DownloadModsTask$DownloadModTask.call0(DownloadModsTask.java:55)
	at com.github.franckyi.cmpdl.task.CustomTask.call(CustomTask.java:17)
	at javafx.concurrent.Task$TaskCallable.call(Task.java:1423)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.lang.Thread.run(Thread.java:748)

wget has no problem downloading the same URL:
$ wget "https://addons.cursecdn.com/files/2428/363/Pam's HarvestCraft 1.10.2j.jar"
--2018-04-04 17:41:03--  https://addons.cursecdn.com/files/2428/363/Pam's%20HarvestCraft%201.10.2j.jar
Resolving addons.cursecdn.com (addons.cursecdn.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to addons.cursecdn.com (addons.cursecdn.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Moved Temporarily
Location: https://files.forgecdn.net/files/2428/363/Pam's%20HarvestCraft%201.10.2j.jar [following]
--2018-04-04 17:41:03--  https://files.forgecdn.net/files/2428/363/Pam's%20HarvestCraft%201.10.2j.jar
Resolving files.forgecdn.net (files.forgecdn.net)... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to files.forgecdn.net (files.forgecdn.net)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://media.forgecdn.net/files/2428/363/Pam%27s+HarvestCraft+1.10.2j.jar [following]
--2018-04-04 17:41:03--  https://media.forgecdn.net/files/2428/363/Pam%27s+HarvestCraft+1.10.2j.jar
Resolving media.forgecdn.net (media.forgecdn.net)... <IP_ADDRESS>
Connecting to media.forgecdn.net (media.forgecdn.net)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 3240818 (3.1M) [application/java-archive]
Saving to: ‘Pam's HarvestCraft 1.10.2j.jar’

Pam's HarvestCraft 1.10.2j 100%[=====================================>]   3.09M  9.47MB/s    in 0.3s

2018-04-04 17:41:04 (9.47 MB/s) - ‘Pam's HarvestCraft 1.10.2j.jar’ saved [3240818/3240818]

Perhaps the quote isn't escaped correctly?

Won't fix for 2.0.0 release. Sorry for the inconvenience.
It will be fixed for 2.1.0 beta.


Make rust test-suite pass

Currently test suite in test-suite directory is not passing

@barankaraaslan which platform did you run the tests on?

@barankaraaslan closing this issue as I can successfully run the tests. Feel free to re-open with more info if you run into it again!


Support for multidim outputs

Closes #252
Thanks to Kazuki for updating asdfghjkl on PyPI, we can now handle 3D tensor outputs (e.g. in LLMs) with GLM predictive.

WIP adding an article based on lm_example.py to the docs.

@runame @aleximmer ready to review!


test more cases in playbook

This does not change anything except testing locally.

Will be handled in #108


Protocol for requesting all necessary votes for round

Related to #19 : it's possible (if messages for recent rounds are unavailable) that we can get stuck in a state where we keep trying to skip to higher and higher rounds but none of them become completable. In this case, we should be able to request messages for those rounds (as we know they exist) in order to make the round completable.

On this crate we added support for processing catch-up responses. The protocol for requesting a catch-up is undefined here and only implemented in Substrate. Do you think this can be closed?

This issue AFAIK is more about requesting votes for old rounds. But with catch-ups, it might not be necessary.


Updated example_config.yml

Added an example of overriding Product's $db array with an adjusted data type.  Thank you.

Thanks!


adding PROPERTYSTATUS option to allcomponents.ps1

According to the V2 documentation the Property JSON object can have the following attributes:
@param name String property name
@param value String property value
@param status AttributeStatus property status
PlatformConfigurationV2Factory.java considers the optional PROPERTYSTATUS attribute in line 202:
final JsonNode statusNode = property.has(Json.PROPERTYSTATUS.name()) ? property.get(Json.PROPERTYSTATUS.name()) : null;
However, the corresponding Powershell script "allcomponents.ps1" which generates "localhost-componentlist.json"
does not offer the option to specify a PROPERTYSTATUS.
I have modified "allcomponents.ps1" so that the user can add a PROPERTYSTATUS.
I will also adjust the .sh script accordingly if the Powershell script change is accepted.

I like the idea of adding a case for 3 arguments to the jsonProperty function. That will help others extend the script for their use cases.

Thanks for your reply! I've updated the bash script version now.

Thank you for the suggestion and patience during the holidays.


Fix SxS Dev12 and Dev14 build errors

When you have multiple versions of VS installed, you will get build
errors about assembly versions.  This should fix that by importing the
settings file and specifying the version on those assemblies

Reviewed by @HuanhuanSunMSFT


Stuck on map loading screen when using generated Presets.json

Hello, so I am not sure if I am just really stupid or it just isn't working. When I generate config (Presets.json) with web UI and replace it in my profile, result is a few errors in log and I am not able to land on moon anymore. I tried multiple variants but even if I use new profile with only BepInExPack and LethalQuantities in it, and generate raw .json from web without changing anything, it doesn't work. I can only land on moon if I don't touch original .json file.
LogOutput.log

Ok, I think I know what might be causing this error. Can you try with this preset?
Presets.json
It seems like I didn't account for the case where no options were selected

Yes, with this one it is working. This problem also occured with this preset where I changed/added only enemies spawn weights on all maps and scrap amount (+dungeon size) on one.
Presets.json

Alright. This will be fixed next update. Thanks!

Great, thank you very much! Really like these mod possibilities, looking forward to using it 👍


Fix dashboard test and add sign_in helpers

This should resolve the problem with the login therefore we can close https://github.com/mikevo/uno-heterogeneous-knowledge/pull/36

Nice, thanks for fixing this. I'll close #36 as well.


How did the MCQ seq2seq results are evaluated?

In other words, MCQ seq2seq model generates the answers. How do you convert it to accuracy?

We used the MCQ evaluator from Unified QA to obtain the accuracies. I will add the script here later.

Thank you for your quick reply!

Hey @yana-xuyan, I have added the evaluation script in the latest commit.
You will need to rerun python prepare.py once in the experiments/mcq directory.

Thanks a lot! That would be really helpful!


Missing URL encoding for Links with parameters by sharing on Facebook and Whatsapp

For instance I would like to share this url https://canitravel.net/countries/?from=176&passport=176
as you can see there are from and passport as parameters in the above link
on whatsapp the request made would be
https://api.whatsapp.com/send?text=https://canitravel.net/countries/?from=176&passport=176
which results in https://canitravel.net/countries/?from=176, missing the passport parameter
I think it would be only a matter of encoding the url in the text parameter of the request to whatsapp.
What do you think, would it be possible?
Thanks,
Super nice component by the way!

@gallifilo
Good catch, I'll try to fix it in a day or two

Fixed in v1.0.9


Duplicate dom in v-if's change duplicates select tag change in v-else/v-else-if/other v-if

Version
2.5.16
Reproduction link
https://jsfiddle.net/adrielsantiago/vfao15ku/5/
Steps to reproduce

Change select in first to "B".
Click the second tab.
ER: the select in "second" will still have it's default value.
AR: the select in "second" was changed to the index of the first select.

What is expected?
The select in "second" will still have it's default value.
What is actually happening?
The select in "second" was changed to the index of the first select.

If you add an ID or a class to either container div to make them non identical, it won't happen.

If we add a key to each div, it is not reproducible. The reason I think it happens is because vue is trying to reuse the select, which it does reuse because it's the same, but then the selected property get's brought along with it.

This is expected. It doesn't happen if you bind the selected attribute or use v-model.
You can also use a key if you don't need to host the variable


fix remote cursor to maintain in quill example

What this PR does / why we need it?
This is Draft PR to share issues while fixing #553
so it works incorrect now.
Any background context you want to provide?
I added some simple code to set selection in case which needs to adjust remote selection, but it works wrong.
procedure is as follows.

client A selects 123 . (selection range is {index: 5, length: 3})
client B places cursor before 1 and insert text 'a'
since insertPosition(5) is equal to or less than remote cursor position(5), we need to adjust remote selection range {index: 5, length: 3} to {index: 6, length: 3} by using quill.setSelection
that range which is sent to selection-change event changes to {index: 6, length: 2} . this is tricky.

What are the relevant tickets?

Fixes # 553
Checklist

[ ] Added relevant tests or not required
[ ] Didn't break anything

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

@chacha912 hi, i made draft PR to share issue with you as you suggested in https://github.com/yorkie-team/yorkie-js-sdk/issues/553#issuecomment-1668951013.
any help would be appreciated.

Thank you for sharing the code :)
I think that the reason selection-change event is being called twice is because Quill is already handling it internally, but you're explicitly calling setSelection again. As mentioned in the issue, during a remote edit, Quill correctly manages one's own selection for proper display.(However, since the source of the event is the api, this selection-change event is not transmitted to Yorkie peers.) Therefore, when you call setSelection again during a remote edit, it duplicates Quill's actions.

And if your intention is to modify local selection when a remote edit occurs and transmit it to other peers, this approach could result in other peers having to wait to receive the selection change.
To illustrate, consider the figure below. The gray highlight represents one's own cursor information. In the second line(issue), one's own cursor is maintained correctly, but the remote cursor is displayed inaccurately. In the third line(solution 1), for 'B', the selection changes from [6,9] to [7,10]. Transmitting this change to other peers could potentially lead to issues.

I think an alternative approach: when an edit operation occurs, update all remote cursors accordingly. I found a similar mechanism in CodeMirror, where marker spans are updated using stretchSpansOverChange within the editor(but i'm not sure).

Feel free to let me know if I've correctly understood your intention, or if there's an alternate approach you'd like to suggest.

@chacha912 i really appreciate your help 🙏
i tried other approaches and the approach to change remote selection by updating presence with not calling internal setSelection in quill. and it worked well for me.

I think an alternative approach: when an edit operation occurs, update all remote cursors accordingly. I found a similar mechanism in CodeMirror, where marker spans are updated using stretchSpansOverChange within the editor(but i'm not sure).

i tried the approach you suggested above also, but using cursors could not solve problem. maybe it seems related to quill's internal setSelection (i will look around more)
i updated commit on new approach and will see if there is another problem in my commit again.

@ed-jinyoung-park Thanks! I've checked the parts you revised.
As mentioned in the comment above, it would be a good to consider the following case as well.
In cases where synchronization takes a long time, there is still an issue. You can test this by setting the syncLoopDuration time on the client, as shown below.
const client = new yorkie.Client('http://localhost:8080', {
  syncLoopDuration: 2000,
});

In the case of CodeMirror, the remote cursor position is updated immediately. (I tested whether the 234 selection remains intact when the right client selects 234 and the left client enters a.)

However, with Quill, since it needs to receive remote operation and then transmit modified selection, you can observe that during this interval, it doesn't represent remote selection correctly. (When a is entered, remote selection initially displays as 123, and only upon receiving the modified selection does it switch back to 234.)

Thanks for your contribution. It was my understanding that this PR had to go a different way. I think it would be good to close this PR for now and reopen a new PR.


Add pause button plugin?

This is mainly a UI thing since the chart already has .pause() and .resume() with all the pausing logic built in, so it might not really make sense to add a plugin that just adds a button. Maybe an example?

Not happening for now - ui is an implementation concern


How to set chart's height to 100%

Question
According to https://apexcharts.com/docs/options/chart/height/, you can set the height to a fixed number, but is there a way to make it 100% height of it's parent?

Just corrected it.
Previously height: 100% didn't work as expected even though parent height was provided.
You will have to give the parent element some height and then if you give options.chart.height: 100%, then the chart will take the parent's height