Swift 6 linux builds fail with clang: error: unknown argument: '-index-store-path'

Not sure if it's all linux builds but it certainly seems to be common, for example Vapor and distributed-actors are affected.
It doesn't seem to be a problem with the swift base images themselves. All the following Swift 6 images we used can build Vapor:
✅ swiftlang/swift@sha256:f4ea7ab93663beac44cc5ec1a3ffcc7a7c33b9d2cd2dec4ad98dd1a739717e39
✅ swiftlang/swift@sha256:0f8bfdf7a1e34448c804aa290c5d02f6652cdef3d628d54dbdaa3e99a95bd22d
✅ swiftlang/swift@sha256:32cd5cc3e82012f138d5e255485a5ce6ec8fb10588e1ecef34c6af7e1291de48
✅ swiftlang/swift@sha256:30154112a700a5a95fd1760716bd2040e8b735f54f081a4865823abdec67d17e
✅ swiftlang/swift@sha256:c91ee3fa7f0e9b49459750cadfc39023e34732769f45122cb139850d6af4c6e9
That would mean it's one of the OS level dependencies we're installing that's interfering.
git clone https://github.com/vapor/vapor && cd vapor
mkdir .stats
docker run --rm -v "$PWD":/host -w /host swiftlang/swift@sha256:c91ee3fa7f0e9b49459750cadfc39023e34732769f45122cb139850d6af4c6e9 swift build --triple aarch64-unknown-linux-gnu -Xswiftc -Xfrontend -Xswiftc -stats-output-dir -Xswiftc -Xfrontend -Xswiftc .stats -Xswiftc -strict-concurrency=complete

(Use --triple aarch64-unknown-linux-gnu on Apple Silicon, --triple x86_64-unknown-linux-gnu on x86)

swift-cross-ui is failing builds across the board anyway, so I'll drop its dependencies for now.

[ ] test Vapor builds with updated base image
[ ] re-run linux builds

Vapor now passing:

Latest builder changes deployed.


Build failure: jenkins-job-builder

Steps To Reproduce
Steps to reproduce the behavior:

nix-build -A jenkins-job-builder

Build log
[...]
tests.test_plugins.PluginsTestScenarios.test_plugin_version_object_comparison(base_url2,s13)
--------------------------------------------------------------------------------------------

Captured traceback:
~~~~~~~~~~~~~~~~~~~
    Traceback (most recent call last):

      File "/build/python-jenkins-1.7.0/tests/test_plugins.py", line 311, in test_plugin_version_object_comparison
    plugin_info = j.get_plugins()[plugin_name]

      File "/build/python-jenkins-1.7.0/jenkins/__init__.py", line 939, in get_plugins
    plugins_data[keys] = plugins.Plugin(**plugin_data)

      File "/build/python-jenkins-1.7.0/jenkins/plugins.py", line 59, in __init__
    self['version'] = version

      File "/build/python-jenkins-1.7.0/jenkins/plugins.py", line 66, in __setitem__
    value = PluginVersion(value)

      File "/build/python-jenkins-1.7.0/jenkins/plugins.py", line 79, in __init__
    self.parsed_version = pkg_resources.parse_version(

      File "/nix/store/basb8wsyinlwbcawx0v9lklvv06qpph6-python3.10-setuptools-67.4.0/lib/python3.10/site-packages/pkg_resources/_vendor/packaging/version.py", line 197, in __init__
    raise InvalidVersion(f"Invalid version: '{version}'")

    pkg_resources.extern.packaging.version.InvalidVersion: Invalid version: '1.0a.preview'
[...]
======
Totals
======
Ran: 600 tests in 4.0974 sec.
 - Passed: 592
 - Skipped: 0
 - Expected Fail: 0
 - Unexpected Success: 0
 - Failed: 8
[...]
error: builder for '/nix/store/6p6xqf08cd3fj5lnz2yhlhhp74q10j6f-python3.10-python-jenkins-1.7.0.drv' failed with exit code 1;
       last 10 log lines:
       >
       > ==============
       > Worker Balance
       > ==============
       >  - Worker 0 (94 tests) => 0:00:04.199205
       >  - Worker 1 (139 tests) => 0:00:04.257260
       >  - Worker 2 (91 tests) => 0:00:00.617901
       >  - Worker 3 (92 tests) => 0:00:04.147683
       >  - Worker 4 (92 tests) => 0:00:04.156221
       >  - Worker 5 (92 tests) => 0:00:04.181178
       For full logs, run 'nix log /nix/store/6p6xqf08cd3fj5lnz2yhlhhp74q10j6f-python3.10-python-jenkins-1.7.0.drv'.
error: 1 dependencies of derivation '/nix/store/17m131aya6dd610k58fa056nakfidnzm-python3.10-jenkins-job-builder-4.3.0.drv' failed to build

Additional context
It's 8 tests blocking it from completing the build. It's a regression since release-22.11 -- now it's broken in both master and release-23.05 branches.
I think it's caused by 451c63214766d3bb8d078620cff4dc74463c5bc8 ("python310Packages.setuptools: 65.0.3 -> 67.4.0") and upstream did https://opendev.org/jjb/jenkins-job-builder/commit/571df29fd752cf80570f9648e1502d4571f9aaa6 to fix it (downgrading setuptools).
Notify maintainers
There are no maintainers listed for jenkins-job-builder, but since it's technically python-jenkins that fails to build: CC @gador.

Upstream issue: https://github.com/pypa/setuptools/issues/3772 ("InvalidVersion exception when invalid version used on Setuptools 66").

I think it's caused by https://github.com/NixOS/nixpkgs/commit/451c63214766d3bb8d078620cff4dc74463c5bc8 ("python310Packages.setuptools: 65.0.3 -> 67.4.0") [...]

Yes. Confirmed by git bisect. CC @mweinelt.

Fix (by disabling failing tests and relaxing setuptools version requirement): https://github.com/NixOS/nixpkgs/pull/238316


[stable/mercure] Add support for proxy headers

Signed-off-by: Kévin Dunglas<EMAIL_ADDRESS>What this PR does / why we need it:
Add support for the new option introduced in Mercure 0.6.0
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/lgtm

/retest


Improve markdown files content validation

Switching from defaults.yml to markdown files removed the automatic content validation rules we had. (see yml schema and tests)
We could improve validation for contributors and help maintenance. Here are a few things I have in mind:

[ ] Improve contributing documentation
[ ] Create a markdown template file that contributors could copy
[ ] Validate content in CI

Maybe using Marked?

Some contribution tips can be found in #391


Evaluate latest-changes query

latest-changes seems to be faster for area queries than achavi. Evaluate if this query produces the same result and indeed is faster (and why - maybe because not using changed?) and could be used instead.
[adiff:"' + last_week + '"][bbox:' + bbox + '][out:xml][timeout:22];
way->.ways;
(
    .ways >;
    node;
);
out meta;
.ways out geom meta;

https://github.com/tyrasd/latest-changes/blob/fb878927d2681777f619123f0a8ad3a05d6cdf8c/js/site.js#L47

changed is more expensive, as it has to evaluate every change in the given time range t0..t1. The query above simply returns the difference of the query, once executed at t0, and another time at t1. If your time range is rather large, this method might miss some changes that have a net zero effect (e.g. an edit which gets reverted very quickly).


EdgeOrder doesn't follow the directory structure

https://github.com/Azure/azure-rest-api-specs/tree/master/specification/edgeorder/resource-manager/Microsoft.EdgeOrder/2020-12-01-preview should be placed inside the preview folder, as are other specs in this repository

https://github.com/Azure/azure-rest-api-specs/pull/12510/files

https://github.com/Azure/azure-rest-api-specs/pull/12510/files

Thanks for feedback. We create a PR to fix it. For this case, we will create a valdiation rule to auto detect this case.

Thanks for feedback. We create a PR to fix it. For this case, we will create a valdiation rule to auto detect this case.


Errors while running the landslide4sense example

Hi, I've been trying to run the landslide4sense example and I've faced a few errors. I'll write down the steps I followed and how I worked around them.

I logged into the LandSlide4Sense challenge website to download the data.
From there I downloaded 3 .zip files (TrainData.zip, ValidData.zip and TestData.zip). However, since the _resources method in the Landslide4Sense class expects the filename to be called landslide4sense.tar (from the _TRAIN_VAL_ARCHIVES) I had to extract the TrainData.zip and compress it again into a .tar file named landslide4sense.tar.
Once I was able to run the example, I figured that by passing data_info=True I only got the paths to those .h5 files. But I wanted to get a tensor returned! So I set data_info=False
Now the problem was that the code expects a path like landslide/landslide4sense/train/img/ but extracting  landslide4sense.tar you end up with a folder named TrainData instead of train.  Once I renamed the folder it did actually work.

Honestly, I've been trying to work with torchgeo as well and I've faced errors there too. Since you rely on external data and you make assumptions about folder structure, with every slight change there is the risk of something breaking. (e.g. torchgeo uses Radiant MLHub to download a few datasets, and when there was a change on the mlhub side something broke).
In my opinion, there are two approaches one can take; either

host the data themselves  - and be in complete control of updating them; or
write 'integrity' tests for each dataset (e.g. checking whether the hash function has changed because of a slight change on the data provider's side, or whether the folder structure/naming has changed) and having people manually making tweaks...

Many thanks for reporting this issue and giving your comments.

About the naming of the .zip files. You may have notised that we developed this data loading code before the official test data were released, and based on our own wrapping method of the data. As the dataset provider has updated their data, we updated our code as well to match their provided .zip files (TrainData.zip, ValidData.zip, and TestData.zip).
The data_info option is supposed to specify whether to return the paths or to return the tensor. But we admit that the parameter naming here is a bit confusing, we will consider to change it in the future.
About the data folder structure. We planned to host some of the datasets and update them (for those who have no license issue). For the others, we will have to ask the user to download them theirself. For the integrity checking part, we have updated our codes to make it work.

Thanks again for posting these issues and comments. They really help to improve our repo.


Use gosu and execs for execution

Use exec at the end of scripts in order to simplify the process chain
and signal handling.  With this change, a SIGTERM sent by docker to
PID 1 will be propagated to the server binary for a clean shutdown.
Resolves #99

Want to do some more testing later (fresh data folder, user connect test, etc) before merging.

Thanks for taking the time to open this PR! Let me know when you feel it's ready to be merged and has been tested sufficiently, and I'll test it my end too :)

fixes #99

Good call on the exec usage. Will test this locally.

Testing has shown that docker stop is still resulting in the process immediately exiting instead of the server binary performing an orderly shutdown.  The latest changeset makes a slight change to run.sh, so that if desired, a user can override the UID/GID via docker run and use run.sh as the entrypoint, resulting in the server as UID 1.

I played with this for another couple hours, and was unfortunately unable to get signal passing to work with the sudo call.  Merging this partial fix does allow admins to bypass the sudo call after init.sh has been run at least once to set permissions on the volume mount.

More testing welcome, but it does appear to start the server correctly, and the environment appears intact:
$ docker exec -it testing /bin/bash
root@14bfcff225ce:/config# gosu steam env
PUID=1000
HOSTNAME=14bfcff225ce
SERVERGAMEPORT=7777
SERVERQUERYPORT=15777
PWD=/config
AUTOPAUSE=true
HOMEDIR=/home/steam
STEAMAPPID=1690800
SERVERIP=<IP_ADDRESS>
AUTOSAVEONDISCONNECT=true
AUTOSAVENUM=3
GAMECONFIGDIR=/config/gamefiles/FactoryGame/Saved
PGID=1000
STEAMBETA=false
CRASHREPORT=true
STEAMCMDDIR=/home/steam/steamcmd
SERVERBEACONPORT=15000
MAXPLAYERS=4
TERM=xterm
USER=steam
SKIPUPDATE=false
SHLVL=1
AUTOSAVEINTERVAL=300
DISABLESEASONALEVENTS=false
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
GAMESAVESDIR=/home/steam/.config/Epic/FactoryGame/Saved/SaveGames
DEBUG=false
_=/usr/sbin/gosu
HOME=/home/steam

Thanks for your continued efforts! Couple of questions. Firstly, why did you duplicate the mkdir into run.sh? This serves no purpose as far as I can tell.
Secondly, with the inclusion of gosu, is signal trapping functioning correctly? If not, I want to avoid including unnecessary dependencies. Your last response isn't overly clear, so I just wanted to clarify :)

Thanks for your continued efforts! Couple of questions. Firstly, why did you duplicate the mkdir into run.sh? This serves no purpose as far as I can tell.

This leaves open the possibility of changing the entrypoint when running a container, and still being able to run /home/steam/run.sh if the end-user so desires.  It does not appear to have a negative effect.

Secondly, with the inclusion of gosu, is signal trapping functioning correctly? If not, I want to avoid including unnecessary dependencies. Your last response isn't overly clear, so I just wanted to clarify :)

I tested by starting a container in the foreground, and issuing a stop from another terminal.  The server received the signal and performed an orderly shutdown.  I also tested on a detached container, and it was able to receive the signal as well.
[2021.12.02-19.18.53:466][  0FUnixPlatformMisc::RequestExitWithStatus
FUnixPlatformMisc::RequestExit
]LogWorld: BeginTearingDown for /Temp/Untitled_0
...
[2021.12.02-19.19.23:475][467]LogModuleManager: Shutting down and abandoning module RSA (3)
[2021.12.02-19.19.23:475][467]LogContentStreaming: Display: There are 1 unreleased StreamingManagers
[2021.12.02-19.19.23:478][467]LogExit: Exiting.
[2021.12.02-19.19.23:478][467]LogCore: FUnixPlatformMisc::RequestExit(bForce=false, ReturnCode=143)
Exiting abnormally (error code: 143)
Assertion failed: !HasCommands() [File:Runtime/RHI/Public/RHICommandList.h] [Line: 3816]

Pure virtual function called!
Signal 6 caught.

Thanks for your continued efforts! Couple of questions. Firstly, why did you duplicate the mkdir into run.sh? This serves no purpose as far as I can tell.

This leaves open the possibility of changing the entrypoint when running a container, and still being able to run /home/steam/run.sh if the end-user so desires.  It does not appear to have a negative effect.

It doesn't have a negative effect as such, it's just annoying to have a duplicate command as both instances must then be maintained.
If we're using gosu, then I think the possiblity of changing the entrypoint becomes an unnecessary and out-of-scope requirement to cater for.

It doesn't have a negative effect as such, it's just annoying to have a duplicate command as both instances must then be maintained.
If we're using gosu, then I think the possiblity of changing the entrypoint becomes an unnecessary and out-of-scope requirement to cater for.

Fair enough.  I'll remove it from the changeset.
While I'm making changes, does it make sense to pass "$@" through the exec stack to allow additional server binary command line parameters to be passed via docker's CMD parameter?
ie docker run --rm ... wolveix/satisfactory-server:latest -foo=bar would result in  Engine/Binaries/Linux/UE4Server-Linux-Shipping FactoryGame -log -NoSteamClient ... -foo=bar being eventually executed?

Appreciate it!
No, you're probably right. Please remove it.

Thanks for your hard work @bewing!


fix type in mergeConcurrently doc

fix https://github.com/cujojs/most/issues/335

Excellent, thanks @Sinewyk!


Clockemoji

@mamal72 please create another pull-request with https://github.com/slack-bolt/bolt-teamline/commit/efb64d0e07ad300b3d3724fd66c20f58ffbf1fbc removed


Allow const/static destructuring

I'll primarily defer to existing discussions requesting the same, see the following at internals.rust-lang:
allow destructuring patterns in const and static
destructuring declarations for const
It simply seems that it hasn't been considered to allow this yet? If so, may it be introduced?

I would like to voice a modest objection to doing this for static:
People often imagine const and static are "basically the same thing". But they aren't. A static is not like a let binding, and it denotes a "place in memory" in a more absolute sense, compared to the more nebulous sense of let.
// The reason we allow destructuring is that this,
let (x, y) = (5, 5);
let z = x + y;

// has, as far as a programmer is concerned, the same semantics as this:
let xy = (5, 5);
let z = xy.0 + xy.1;

// But in the case of static...
static XY: (i32, i32) = (5,5);
static (X: i32, Y: i32) = (5, 5);
// ...these are very different, in a way that is *immediately* relevant.

I think allowing this is allowing too much programmer confusion. But it seems eminently permissible for const declarations, which are nebulously located like let is. However, if people believe that to allow it for one must be to allow it for either, because they would like const and static to remain similar, then it would be preferable to allow it for neither.

// But in the case of static...
static XY: (i32, i32) = (5,5);
static (X: i32, Y: i32) = (5, 5);
// ...these are very different, in a way that is *immediately* relevant.

Do you intend to say that it's different because the memory locations are disconnected now?
Does anyone rely on such behaviour? I would see it permissable that X and Y alias the same memory location.

Yes. And the tuple memory representation is under such constraints that programmers will rely on the fact that XY.0 and XY.1 are, at furthest, adjacent in memory. They could certainly not alias if they were different values, and people do rely on the equivalence between homogenous tuples and homogenous arrays of equivalent length, especially when they don't need the order to be perfectly preserved, for various historical reasons that I am disinclined to chatter extensively about. Replace the parentheses with [] if you prefer.
Destructuring, however, allows them to be in wildly different regions of memory, putting entire gigabytes of empty addressing between them.

Destructuring, however, allows them to be in wildly different regions of memory, putting entire gigabytes of empty addressing between them.

Ah, this although somewhat exaggerated number, definitely helps understand the issue. I agree that, that's worth keeping in mind.
It's quite imaginable that this is a footgun.

Destructuring, however, allows them to be in wildly different regions of memory, putting entire gigabytes of empty addressing between them.

Ah, this (although somewhat exaggerated) number definitely helps understand the issue. I agree that, that's worth keeping in mind.
It's quite imaginable that this is a footgun.

I would like to voice a modest objection to doing this for static: People often imagine const and static are "basically the same thing". But they aren't. A static is not like a let binding, and it denotes a "place in memory" in a more absolute sense, compared to the more nebulous sense of let.
// The reason we allow destructuring is that this,
let (x, y) = (5, 5);
let z = x + y;

// has, as far as a programmer is concerned, the same semantics as this:
let xy = (5, 5);
let z = xy.0 + xy.1;

// But in the case of static...
static XY: (i32, i32) = (5,5);
static (X: i32, Y: i32) = (5, 5);
// ...these are very different, in a way that is *immediately* relevant.

I think allowing this is allowing too much programmer confusion. But it seems eminently permissible for const declarations, which are nebulously located like let is. However, if people believe that to allow it for one must be to allow it for either, because they would like const and static to remain similar, then it would be preferable to allow it for neither.

do only for const then

do only for const then

+1


Jupyter Variables disappear after clicking and other data viewer bugs

Issue Type: Bug
Upon clicking on variable exlorer in jupyter, a window in a lower editor pane appears and asks me to pick a variable. I pick a variable and Data Viewer opens up. And Jupyter Variables window disappears. Keeping the terminal pane open, even though it was not open. This is distracting me.
Data viewer does not refresh when dataframe refreshes, nor does have a button to do so.
It would be more intuitive behavior if cmd+click (or ctrl+click for non-mac users) would lead to data viewer instead of this additional flow.
Extension version: 2021.3.600686576
VS Code version: Code - Insiders 1.54.0-insider (Universal) (e590188f17162393f50feec19263398e6fe02d13, 2021-02-26T20:39:12.763Z)
OS version: Darwin x64 19.6.0

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
python383:30185418
pythonvspyt602:30263608
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstry244:30244315
pythonvsdeb440:30224570
pythonvsded773:30223139
pythonvspyt875:30259475
pythonvspyt639:30264824
pythontb:30258533

@nfx we've added a button to refresh the data viewer and we have also addressed the issue with the Jupyter variable window disappearing--the variable window will now remain open as long as there is a notebook open in the editor.
The refresh button should already be in your build, but the data viewer must be active and focused for the button to appear. The Jupyter variable window fix will become available in our upcoming March release.
https://github.com/microsoft/vscode-jupyter/issues/4562
https://github.com/microsoft/vscode-jupyter/pull/5006
https://github.com/microsoft/vscode-jupyter/issues/1143
https://github.com/microsoft/vscode-jupyter/pull/4951

@joyceerhl i'm fine with early builds :) though they have pytest behavior broken and are a bit slow ...

@nfx are you not seeing the refresh button / does it not work in v2021.3.600686576? This is what it's supposed to look like:

It'll only appear when you have set focus to the data viewer window.

I believe this can be closed. @nfx, please reopen if there's something remaining.


Question about weight decay

man i saw your chart weight decay=0.05 under the normal weight=1e-4,  weight decay=0.05 has better performance.
but its just on swin backbone in the table,
Will it better on the renset backbone with weight decay=0.05?
hope ur reply~

It is a good practice to use larger weight decay (stronger regularization) for the transformer-based backbones. So we guess this might not help for Conv-based backbones.

It is a good practice to use larger weight decay (stronger regularization) for the transformer-based backbones. So we guess this might not help for Conv-based backbones.

ok，i got it~
appreciate !!


Fixed std renaming bug.

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)

Emm, why is the CI failing

Emm, why is the CI failing

Flake


Cleanup of how fixtures customize a fixture

I noticed that sometimes the same fixture was being referenced twice in a given scenario. Something like the following.
Given a fixture app "empty-app"
# additional customization
And the Server is running at "empty-app"

This is equivalent to
Given a fixture app "empty-app"
# additional customization
Given a fixture app "empty-app"
And the Server is running

So it can just be replaced with
Given a fixture app "empty-app"
# additional customization
And the Server is running

This avoids pointless duplication of the fixture name.

Great. I think these are some leftover copy pasta. Thanks


[ARCTIC-1394] Optimize the execution time of drop table

fix #1394
Why are the changes needed?
After performing the drop table operation, if there are many hdfs files in the Arctic table, the file cleaning time will be very long, even up to 1-2 hours. If the user rebuilds the table during this process, the hdfs file of the new table may be cleaned up by the previous table deletion operation.
Brief change log
core/src/main/java/com/netease/arctic/catalog/BasicArcticCatalog.java
The doDropTable method has a purge parameter, which represents whether to clean up the file. The current operation is that regardless of whether the purge is true or false, the table deletion method of iceberg will be called first to delete the change and base tables respectively, and finally the table location directory will be cleared.
However, iceberg's table deletion method is slow to execute, and if it is decided to clear the table file, the previous steps are not required, and the table location directory in hdfs can be cleared directly.
Before：
if purge == true:
      1. delete change table (by iceberg)
      2. delete base table (by iceberg)
      3. delete table location (hdfs operations)
else:
      1. delete change table (by iceberg)
      2. delete base table (by iceberg)

After modification:
if purge == true:
      1. delete table location (hdfs operations)
else:
      1. delete change table (by iceberg)
      2. delete base table (by iceberg)

In the test, to delete an arctic table with 20,000 files, it takes about 10 minutes to use the previous method, but it only takes 1 second to use the modified method.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

@ShawHee Thanks a lot for your contribution!
From your description, the file system delete recursively API has a much better performance than Iceberg Tables drop table API.So deleting table location recursively when the purge configuration is enabled is indeed a good optimization method.
However, this optimization method still has a limitation, which is that all manifest/data files must be located under the table directory. In the current implementation of Arctic, this condition is satisfied. However, in future implementations, this limitation may be broken.
Therefore, I think this optimization can be merged in at the current stage, but additional comments should be added to explain the limitation.

LGTM

@zhoujinsong Thanks for your suggestions. I've added a comment to the code explaining the limitations of this optimization.


fix(json-field): switch from json to jsonb

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Welcome to the club

Hi and welcome to the club of IMAP client libraries for .NET ;-)

Thank you :)

Not sure if it's helpful, but if are in need of a MIME parser, you could use MimeKit so that you can focus on IMAP.
Another option is to just deal with streams. Another guy I know was working on a really nice IMAP library that supported command pipelining and was fully async (sadly using the Begin/End convention rather than async/await) who was just going the Stream route so that consumers could use any MIME parser they wanted.
I will note that my MimeKit library can convert System.Net.Mail.MailMessage into MimeMessage, but can not go in the other direction. That may be a downside if you want to pair your library with System.Net.Mail for SMTP support. That said, however, System.Net.Mail isn't all that great anyway and implementing SMTP support is trivial if you can do IMAP :)
Anyway... just some random thoughts.
Oh, and I like your class for UIDs - it's very similar to my UniqueId design ;-)

Thanks for the feedback.
This project was meant to be a fill in while I was running down some other work, but it seems to have taken on a life of its own. I'm definitely not planning on implementing SMTP myself, but I think that some SMTP integration is required to support some possible use cases that I can imagine (e.g. the automatic preparation of an email that a user has to edit or ok-to-send).
The library offers a stream with the whole message in it via the cMessage.Fetch(cSection ...) API (if you pass cSection.All as the section), so if someone wanted to hook up to an external mime parser or SMTP service then they could (and this will probably be required for the sending of sophisticated messages as the conversion to System.Net.Mail.MailMessage that I was planning was going to be very simple and will most likely be an incomplete one - when the time comes I'll include pointers to your project in the documentation if you approve).
The append APIs will definitely take a stream as one of the possible message data sources; I've already spent a bit of time including support for it in the command pipeline.
:)

This project was meant to be a fill in while I was running down some other work, but it seems to have taken on a life of its own.

Haha, this is exactly the same for my own IMAP, POP3, SMTP library (MailKit). :)
I was, at the time, hoping that one of the many other IMAP libraries would pick up MimeKit, but in the meantime I just wanted to verify that MimeKit had all of the bells and whistles to make it easy to use when writing SMTP, POP3, and IMAP clients.
Anyway, feel free to check out my MailKit library as well - I think you'll find some great design ideas that may help you, or at least might spark some spin-off clever ideas of your own.
The one thing I've still got to figure out is how to allow pipelining of commands.
Have you been able to solve that with your library? I've only skimmed over your source code so far but haven't had the chance to really play with it yet as I was busy rewriting my own library to be fully async.
One thought I had was to have my command-pipeline running in its own thread so that the higher-level public API would just queue commands on the engine which could peek-ahead in the command queue to see which commands could be pipelined (according to the rules in rfc3501), flush them, then read the tagged and untagged responses as they came in, emitting "completed" events (or mark the async Task as completed) so that the public API would know the command was finished and could return to the caller.

Pipelining: yes it works.
I solved it with an all async task approach with ‘locking’ to block the submitting of incompatible commands.
After doing some reading I ruled out using dedicated threads, as the command submitter and response receiver jobs spend most of their time waiting, which, according to my understanding, implies the use of tasks.
The locking approach is required for some commands, as you can’t even prepare the text of a command that includes MSNs if a command that allows EXPUNGEs to arrive is running at the time. The downside of the approach I took is that operations aren’t necessarily executed in the order that they are submitted to the library; e.g. a long run of compatible operations could hold up an incompatible one that was submitted in the middle. I’m not sure how likely that is to cause a problem in the real world however.
For examples of the pipelining in action: the library normally pipelines the NAMESPACE and ID commands at connect time, and if you don't have (or turn off) LIST-STATUS but ask for status information when doing a mailbox listing, it will pipeline all the STATUS commands. Those are both examples where the library is doing the pipelining internally, but it isn’t cheating – it submits the commands internally the same way as they would be from outside. There are tests in the testharness that make sure that the pipelining is working – the two I just found are ZTestSearch3 (which runs SEARCH and SORT at the same time and demonstrates the out of order execution issue) and ZTestPipelineCancellation (primarily a test that cancellation of a submitted command stops it being sent, but it runs two extended SEARCHes at the same time).
Doing the command pipelining was very frustrating and only occasionally satisfying (i.e. it was fun). Good luck!

I'll have to check out how you did it, thanks!


[Auto-generated] Update requirements.txt

Update requirements.txt
Automatically created PR from the ci_update_requirements.yml workflow.

Codecov Report
Patch and project coverage have no change.

Comparison is base (98379fd) 69.15% compared to head (4054bef) 69.15%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@                  Coverage Diff                   @@
##           ci/dependabot-updates     #309   +/-   ##
======================================================
  Coverage                  69.15%   69.15%
======================================================
  Files                         18       18
  Lines                        522      522
======================================================
  Hits                         361      361
  Misses                       161      161

Flag
Coverage Δ

pytest
69.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Inlay Hints dont work

Hi,
I am using LazyVim and I created a new file using your lazy config under the plugins directory. Everything works fine except for the inlay hints.
Could you please take a look?
Thanks!

Same here, there seems to be something broken with how it is enabled here: https://github.com/ray-x/go.nvim/blob/abd282564a31c5dec14e2038ebf04fdac9ea3278/lua/go/inlay.lua#L322

I get this error continuously. I can confirm that disabling inline hints fixes the issue.
Looking at the :help vim.lsp.inlay_hint, seems line enable takes a boolean, not the bufn. the bufnr should be passed to the second argument.
[dlevym@inspirion] ~ nvim --version
NVIM v0.10.1
Build type: Release
LuaJIT 2.1.1713773202
Run "nvim -V1 -v" for more info

I think this one was fixed. Please check and re-open if you see issues after the update. Thanks


Did we can calculate maximum message for Service plan S1?

Hi i have some question
Did we can calculate maximum message for Service plan S1?
e.g
companyAAA = 3000 employees
need send = 250 massage/month
on Service plan S1 can work ?
and how many estimate message limit for Service plan S1 ?
how to calculate

Hi @arruzarmas,
Thanks for raising the query. We will get back to you with proper response soon.

Hi @arruzarmas ,
We are closing this request as the we have provided the inputs for the query raised. If you face any issue with company communicator please raise a new issue for further assistance.


Thinkpad Helix Wacom Pen Not Rotating

The pen included with the Thinkpad Helix is not rotating. Have you used a pen with this script, or just touch?
Wacom handles pen detection a bit oddly, so I'm still working on it, but a potential fix is about 50% done, if you are interested.

i'm not sure what pen rotation means(?) but i'm on an asus ux370 w/ a pen.
But i don't use the pen tbh.
On Mon, 15 Oct 2018 at 14:18, Ben Yanke<EMAIL_ADDRESS>wrote:

The pen included with the Thinkpad Helix is not rotating. Have you used a
pen with this script, or just touch?
Wacom handles pen detection a bit oddly, so I'm still working on it, but a
potential fix is about 50% done, if you are interested.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/donbowman/kde-auto-rotate/issues/2, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AE5OkwSWSW5pwGpABb9UiOaDDf8h9lXoks5ulNGMgaJpZM4Xc4V7
.

Hello Ben,
have you found a way to make this script permanent?
I do have the same problem: manually enabled and started it works fine, but the corresponding (running!) auto-started process doesn't rotate (it seems to react on the sensor change though - journalctl) the screen :-(.
Thanks for any input,
pheidrias


Ampify Build stuck when i create in new region

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
arn:aws:amplify:eu-west-1:324106371807:apps/d2775g2vwzhi4n
Region
eu-west-1
Amplify Console feature
Backend builds
Describe the bug
I am trying to create an instance of amplify application in new region
When I connect to repo and branch, the build is stuck at backend phase
                                 # Starting phase: build
2022-01-06T10:54:53.871Z [INFO]: Note: It is recommended to run this command from the root of your app directory
2022-01-06T10:54:54.130Z [INFO]: Amplify AppID found: dvb6mzqluh031. Amplify App name is: meaplatform
2022-01-06T10:54:54.225Z [INFO]: Adding backend environment luelle to AWS Amplify app: dvb6mzqluh031
2022-01-06T10:54:54.401Z [WARNING]: - Initializing project in the cloud...
2022-01-06T10:54:59.635Z [INFO]:
2022-01-06T10:54:59.638Z [INFO]: CREATE_IN_PROGRESS amplify-meaplatformmeaplatfo-luelle-105453 AWS::CloudFormation::Stack Thu Jan 06 2022 10:54:54 GMT+0000 (Coordinated Universal Time) User Initiated
                                 CREATE_IN_PROGRESS DeploymentBucket                           AWS::S3::Bucket            Thu Jan 06 2022 10:54:58 GMT+0000 (Coordinated Universal Time)
                                 CREATE_IN_PROGRESS UnauthRole                                 AWS::IAM::Role             Thu Jan 06 2022 10:54:58 GMT+0000 (Coordinated Universal Time)
                                 CREATE_IN_PROGRESS AuthRole                                   AWS::IAM::Role             Thu Jan 06 2022 10:54:58 GMT+0000 (Coordinated Universal Time)
                                 CREATE_IN_PROGRESS UnauthRole                                 AWS::IAM::Role             Thu Jan 06 2022 10:54:59 GMT+0000 (Coordinated Universal Time) Resource creation Initiated
2022-01-06T10:55:04.636Z [INFO]:
2022-01-06T10:55:04.638Z [INFO]: CREATE_IN_PROGRESS DeploymentBucket AWS::S3::Bucket Thu Jan 06 2022 10:54:59 GMT+0000 (Coordinated Universal Time) Resource creation Initiated
                                 CREATE_IN_PROGRESS AuthRole         AWS::IAM::Role  Thu Jan 06 2022 10:54:59 GMT+0000 (Coordinated Universal Time) Resource creation Initiated
2022-01-06T10:55:14.639Z [INFO]:
2022-01-06T10:55:14.640Z [INFO]: CREATE_COMPLETE UnauthRole AWS::IAM::Role Thu Jan 06 2022 10:55:14 GMT+0000 (Coordinated Universal Time)
2022-01-06T10:55:19.654Z [INFO]:
2022-01-06T10:55:19.656Z [INFO]: CREATE_COMPLETE AuthRole AWS::IAM::Role Thu Jan 06 2022 10:55:14 GMT+0000 (Coordinated Universal Time)
2022-01-06T10:55:24.636Z [INFO]:

And ultimately times out, there is no error or anything to indicate what is going wrong
Expected behavior
Build should succeed or error that needs to be fixed in build should be shown. Even if a permission issue, it should indicate what is getting missed.
Reproduction steps

Rebuild the app for above ARN
Build will get stuck in backend build phase

Build Settings
version: 1
backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - yarn install
    build:
      commands:
        - yarn run build
  artifacts:
    baseDirectory: build
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

Additional information
No response

I created a new app meaplatformie in eu-west1 and ran into same issue of deployment getting stuck
App ARN
arn:aws:amplify:eu-west-1:324106371807:apps/dmyim1snahlt7

Hi @SantoshRaoMea thanks for raising this issue. It's stalling because behind the scenes you're being prompted to select an AWS region. I'm brining this up to the team for further investigation.

Hi, can you try to check your .gitignore file to ensure all these files are ignored? Thanks!
https://docs.amplify.aws/cli/reference/files/#gitignore

@hloriana , just to aid the investigation, I am using an Aurora data source on a cluster, and there is only one cluster and 1 database in the region of eu-west-1 account

I tried recreating app again, just to check with latest CLI (Sorry that I recreated, wanted to keep the region clean). Issue still persists, app details bellow
App name
meaplatform
App ARN
arn:aws:amplify:eu-west-1:324106371807:apps/dtesxd31tv0np

@hloriana would there be any workaround for the moment, to get past the prompts by supplying the region in some way

Hi VIctor,
All these files are ignored.
my .gitignore for reference as well. Also i double checked the repo doesnt have any of these files as well.
See https://help.github.com/articles/ignoring-files/ for more about ignoring files.
dependencies
/node_modules
/.pnp
.pnp.js
testing
/coverage
production
/build
misc
.DS_Store
.env.local
.env.development.local
.env.test.local
.env.production.local
npm-debug.log*
yarn-debug.log*
yarn-error.log*
*.xlsx
.secret-*
**.sample
#amplify-do-not-edit-begin
amplify/#current-cloud-backend
amplify/.config/local-*
amplify/logs
amplify/mock-data
amplify/backend/amplify-meta.json
amplify/backend/awscloudformation
amplify/backend/.temp
build/
dist/
node_modules/
aws-exports.js
awsconfiguration.json
amplifyconfiguration.json
amplifyconfiguration.dart
amplify-build-config.json
amplify-gradle-config.json
amplifytools.xcconfig
#amplify-do-not-edit-end
Regards
Santosh

From: Victor Ding @.>
Sent: Tuesday, January 11, 2022 4:15 AM
To: aws-amplify/amplify-console @.>
Cc: Santosh Rao @.>; Mention @.>
Subject: Re: [aws-amplify/amplify-console] Ampify Build stuck when i create in new region (Issue #2489)
Hi, can you try to check your .gitignore file to ensure all these files are ignored? Thanks!
https://docs.amplify.aws/cli/reference/files/#gitignore
—
Reply to this email directly, view it on GitHubhttps://github.com/aws-amplify/amplify-console/issues/2489#issuecomment-1009421109, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVXJOAVNEJD7SOQLVJKLJ5TUVNOSFANCNFSM5LMCTRRQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>
meaplatform http://www.meaplatform.com
linkedin.com/company/meaplatformhttps://www.linkedin.com/company/meaplatform
twitter.com/meaplatformhttp://twitter.com/meaplatform
Confidentiality Note: This e-mail is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. Dissemination, distribution or copying of this e-mail or the information herein by anyone other than the intended recipient, or an employee or agent responsible for delivering the message to the intended recipient, is prohibited. If you have received this e-mail in error, please notify us immediately and destroy the original message and all copies
meaplatform http://www.meaplatform.com
linkedin.com/company/meaplatformhttps://www.linkedin.com/company/meaplatform
twitter.com/meaplatformhttp://twitter.com/meaplatform
Confidentiality Note: This e-mail is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. Dissemination, distribution or copying of this e-mail or the information herein by anyone other than the intended recipient, or an employee or agent responsible for delivering the message to the intended recipient, is prohibited. If you have received this e-mail in error, please notify us immediately and destroy the original message and all copies

I could get this working, steps i followed *though a work around"

Created a New App in the new region
Connected to Github
Let the Backend build hang and timeout
Pulled the code locally along with new env
This causes many backend files related to resourced to be deleted
Restored the deleted files
In amplify/team-provider-info.json copied over categories from a working env
Adjusted the region and ARN settings where applicable in the copied over data
Pushed to Git and let it build

Hi @SantoshRaoMea, glad to see you were able to find a workaround and thank you for providing feedback on the logging process. If you run into this issue again feel free to reopen this.


KAFKA-8410: Update the docs to reference the new PAPI

Since the old Processor API is now deprecated, we need to
update the documentation to steer people to the new API.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Here are the rendered docs for the reviewers:

Thanks so much for the reviews, everyone!
I've merged and cherry-picked to 3.0 (cc @kkonstantine )


contacts_service/contacts_service-Swift.h' file not found

I have seen many people mention this problem without any solution. All these issues were closed due to timeout. I need an accurate solution.thanks.

Hi there!  This is a community project (my time is very limited, sorry).  Please feel free to contribute your PR for review... thanks!


installation: test Invenio with "master" version of most important packages

[ ] Flask
[ ] Jinja2
[ ] Werkzeug
[ ] SQLAlchemy
[ ] Flask-*
[ ] workflow #2177
[ ] dictdiffer
[ ] ... ?

Closed via 88fb1e0c333ec60c58d76e33488cd14fe3745ec3.


need Help on cell selection Ui

@nadbm while selecting cell range UI looks something need to improve, is there any option to do

Are you talking about the white spacing?
The cellspacing can be added via the css. Otherwise there isnt any option to make it look like that


fix: add dns search record limit note.

This PR is going to fix #18900 , adding a note for the removing of 6 DNS search records limit in glibc package versions at or exceeding glibc-2.17-222.

/lgtm
/approve


Change grpc version from 1.35 to 1.40

Update version of grpc in response to recent update on their side.
Builds and tests locally.

Unfortunately the front running of at least boringssl is still necessary.


cluster operators degraded

If gatekeeper is not configured correctly, it can cause the cluster operators to go into a degraded state as they cannot reconcile correctly. the network operator is the main one you see always:
2021/03/16 10:28:12 reconciling (/v1, Kind=Namespace) /openshift-multus
2021/03/16 10:28:12 could not apply (/v1, Kind=Namespace) /openshift-multus: could not update object (/v1, Kind=Namespace) /openshift-multus: admission webhook "check-ignore-label.gatekeeper.sh" denied the request: Only exempt namespace can have the admission.gatekeeper.sh/ignore label

need to add the below flag to the controller for all core namespaces:

https://github.com/open-policy-agent/gatekeeper/blob/release-3.2/pkg/webhook/namespacelabel.go#L25


A11y: When Google Street View is open, focus does not move to the street view

Describe the task
When opening Google Street view for a single property, the focus is still on the single property sidepanel where you triggered the view from.
Here's what the experience is like for screenreaders:
https://www.loom.com/share/605fba1977af458293337021e074ed47?sid=998d512e-f4a3-4d6f-8cf0-c08847a260c7
Acceptance Criteria

[ ] When opening the full screen Google street view, focus will move from the trigger button to the street view container.

Note: this issue is just for moving focus, not for fully working keyboard/screenreader use within the street view

Hi, i'd like to assign myself this task if it is still available. Thank you!

Just sharing an update, I have focus applied to the streetview but now that I am in here I am thinking we should add a couple more updates like:
-Close iFrame on pressing ESC key
-Have Close button receive focus first
So I am working on having those items included. @bacitracin @marvieqa let me know if there is anything else you think should be added


[GUI] cache number of device per page

The number of devices per page should be cached into the user's browser or something like that.
Is really annoying change it every time you come back to the device screen.
This issue is related to usability.

It will no longer be prioritized.


Allow tracking number to be passed to orderFulfill

I want to merge this change because it allows to add optional tracking_number to orderFulfill mutation.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-7831 Allow tracking number to be passed to orderFulfill


optimize rootref resolution

Coverage decreased (-0.3%) to 88.569% when pulling 247b4f04b4e2ab0814515915bb5434b682490904 on optimize-rootref-resolution into c1a75224cc0927020b9416e46958d870dc59f460 on master.


fix(treeindex, #24): optimize scan performance

It has known issues on AArch64. Checking it on Intel.

Passed? Missing/inappropriate memory fences somewhere?


Us21 egiftcar

Pull request for user story 21

We need a refactor in the code!
So keep rocking bro!
Thank you
See you soon...


use as SliverAppBar

I want a SliverAppBar with gradient .. so far I have been using the normal one but can I use GradientAppBar as the SliverAppBar ?

I'll look into it when I'm back from holiday. As far as I've seen the SliverAppBar is a completely AppBar, I'll probably add a SliverGradientAppBar.
On 30 Jul 2019 12:37, Adib Mohsin<EMAIL_ADDRESS>wrote:
I want a SliverAppBar with gradient .. so far I have been using the normal one but can I use GradientAppBar as the SliverAppBar ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/joostlek/GradientAppBar/issues/10?email_source=notifications&email_token=ABWBN26UIQYIETREUN7W7ELQCAKWRA5CNFSM4IH24XU2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HCINJHA, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABWBN2Y4C6MUMDPLJ3YXFXDQCAKWRANCNFSM4IH24XUQ.


Change "Github" from a button to a ribbon on the background image.

Similar to what we see on http://www.grpc.io/

We can have our designer create the ribbon if you can't find a open source version.


default delete-collection-workers to 16

this hopefully gets density jobs of > 100 nodes to successfully terminate
their e2e test namespaces under a hardcoded 5m threshold set in
kubernetes/test/e2e/framework/framework.go
this matches the setting used by some of the kubernetes/test-infra
jenkins jobs, eg: kubernetes-gce-scalability-release-1.3

retest this please

lgtm


Bot occasionally logs the same error "no such host"

From time to time my bot logs the same error:
2024/12/13 09:17:19 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host
2024/12/13 09:17:19 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host
2024/12/13 09:17:20 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host
2024/12/13 09:17:31 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host
2024/12/13 09:17:32 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host
2024/12/13 09:47:59 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host
2024/12/13 09:47:59 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host
2024/12/13 09:48:00 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host
2024/12/13 09:48:10 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host
2024/12/13 09:48:11 [TGBOT] [ERROR] error get updates, error do request for method getUpdates, Post "https://api.telegram.org/botxxxxxxxxxx:xxxxxxxxxxxxxxxxGXrNOqmGrQ78BC6cyD8/getUpdates": dial tcp: lookup api.telegram.org: no such host

This error doesn't really affect the bot's performance.
But my terminal logs are now practically useless because I can't read them properly.
What should I do?

lookup api.telegram.org: no such host seems as your network problems


Synchronize with original repository

Finally!

Sorry, wrong merge. I was going to merge into our internal repository.
Fixed already.


verison 2.2.7 magento2 order problem

Hi, we are using iyzipay at our website for credit card payments.
The problem is if the customer use iyzico as payment type magento order is not created in the system. Even in case the payment successfull magento does not create the order. And customer gets and error didt get the SUCCESS PAGE.
Please look the case and support us. We just upgrade from 1.9 and we are using iyzipay for 3 years.

Magento 2.2.7 order would not created in magento aftrer iyzico payment.
response did not return to site. and order would not placed...

Hello @secaliskan, from the iyzico developer team
Our integration team has contacted you to help you.
Thank you for your valuable feedback.


Add the ability to pause the remote pod

Today, mirrord runs side-by-side with the existing pod/s. Many users want to "replace" the remote pod.
I think the best, nicest, approach wouldn't be to really replace the replicaset with our agent, but still run it side by side, making the agent send SIGSTOP to the pod's process, then SIGCONT when agent finishes running.
This way we're just pausing the activity of the original pod.
Small hint-  I think we should have a loop that sends SIGSTOP every few seconds, as there might be some OS stuff that will send SIGCONT without us knowing/wanting.
Suggestion - flag would be --pause -p

We need to test it on multiple runtimes, so this should be covered in e2e, not just integration testing.
Manually test few frameworks (Python, Java, Go) to see how they behave with this method.
Check a scenario where the remote pod has health checks - I assume the local service would answer for those so k8s won't mark it as not ready / dead

Should we use docker's pause command if docker is the container's runtime?
And maybe if it's not docker we can still use cgroup freezer which is apparently what docker uses to pause?

I wouldn't bother with docker pause as it's not the common runtime.
cgroup freezer sounds cool but also overkill for now if we already went with the sigstop..

Saw your reply too late. But now we have a neat implementation using the docker/containerd API to pause the container instead of doing it ourselves :)

Is adding this feature to the IDEs part of this issue or should I open separate issues for that?
(PRs would be separate either way I guess)

Is adding this feature to the IDEs part of this issue or should I open separate issues for that? (PRs would be separate either way I guess)

No, this feature should be possible to enable from the config file - VS Code already uses it and IntelliJ is underway.

Stable status tracked in #809, support for ephemeral tracked in #795.

That exactly what is preventing me from using Telepresence 2.
Many thanks 🙏🎉

@bcouetil Thanks for letting us know! It'll make us prioritize stabilizing this feature :) #809
Let us know if you have any more feature requests/feedback.


Enable deleting connections

Fixes #57
Uses the same interaction as creating the statements in the first place: right click statement, click 'delete connection', click other statement to delete connection with it.
Changes summary:

add deleteConnection function
separate drawConnection as function
use statement.id instead of rowindex

That's odd, are you sure you're on the right branch? One minor thing I realized you have to pay attention to is to click the shape, not the text in the shape as that doesn't pass the click event through to it's parent (yet).
Should I make a separate PR for that, or include it in this one?

I have tried again: apparently is not working on my local. We can chekck it tmr in person.

UPDATE: Just realized how it works. Better to add a comment on the help page to clarify how the deleting connection works.

Thanks for checking and approving @ermannolocascio , happy to hear you got it to work in the end. How could I have changed the PR description to make more clear how the functionality is supposed to work?


BSA Linux - Test cases like 863 for RCiEP and i-EP should not be run

This issue is created for BSA Linux for a similar reason to https://github.com/ARM-software/bsa-acs/issues/40.

Hi @sunnywang-arm,
Can you confirm if the issue is observed with the BSA binary containing  #40 fix. (https://github.com/ARM-software/bsa-acs/pull/51)
Binaries build after 2nd Sep will have the fix.
Regards,
ACS team

The issue still exists with the image build after 2nd Sep.  As our offline discussion, The root cuase can be that the bsa 863 has actually run for RCiEP device, as 863 doesn't rely on bdf table but linux apis


Compile exception when passing null as component attribute

Describe the bug
When null is put into a components attribute within a .razor/.cshtml file, a compilation error is reported.
Example:
<NavMenuItem href="/modules" Title="Module" Icon="IconType.ViewDashboard" Match="@(null)">
The reported error is (freely translated):
CS0121: The call does not distinguish clearly between the members or properties "RuntimeHelpers.TypeCheck<T>(T)" and "RuntimeHelpers.TypeCheck<T>(Func<Task>)"
The razor compiler translates the above attribute assignment to:
 builder.AddAttribute(37, "Match", Microsoft.AspNetCore.Components.RuntimeHelpers.TypeCheck<Microsoft.AspNetCore.Components.Routing.NavLinkMatch?>(null));

I assume you're seeing this error in the editor, and not when you build from the command line correct?

@rynowak Exactly

OK cool then. You're seeing this as result of some hacks we had to do in preview 3. We added some new/language features in that release but did have a VS release that would update the editor.
This is already fixed for preview 4.


epic(site): UIPlayground integration

Closes: #1867
I'd recommend checking out this PR visually as well (especially in terms of movement and content of the ESL pages).

:tada: This PR is included in version 5.0.0-beta.4 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Build failed on Windows

I just installed llama-cpp-2 on Windows and when tried to build it failed with error.
Error
cargo build -p llama-cpp-2
   Compiling windows_x86_64_msvc v0.52.6
   Compiling windows-targets v0.52.6
   Compiling windows-sys v0.52.0
   Compiling libloading v0.8.5
   Compiling clang-sys v1.8.1
   Compiling errno v0.3.9
   Compiling home v0.5.9
   Compiling rustix v0.38.34
   Compiling which v4.4.2
   Compiling bindgen v0.69.4
   Compiling llama-cpp-sys-2 v0.1.69
   Compiling llama-cpp-2 v0.1.69
error[E0271]: type mismatch resolving `<LlamaTokenAttr as RawBitFlags>::Numeric == i32`
  --> C:\Users\User\.cargo\registry\src\index.crates.io-6f17d22bba15001f\llama-cpp-2-0.1.69\src\token_type.rs:45:17
   |
45 |         Ok(Self(BitFlags::from_bits(value).map_err(|e| {
   |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `u32`, found `i32`

For more information about this error, try `rustc --explain E0271`.
error: could not compile `llama-cpp-2` (lib) due to 1 previous error

Version
llama-cpp-2 = { version = "0.1.68", features = [] }

was fixed by #488 and could be closed, but isn't released yet to crates.io because of #518


addiotionalTextEdits should be in initializationOptions?

Hello, I have a problem with double import with pyright and golang. How to apply addiotionalTextEdits option?
https://github.com/yegappan/lsp/issues/259

seems pyright did not support auto import unless pylance, how did you make it work?
anyway, history or information or description in #259 should have been answer your question in this lsp side, please donot repeatly dup open tickets.
// IMO we perhaps cannot offer tech support, specially via this way.


What am I doing wrong?

include 'WavesPhpAPI/vendor/autoload.php';
$senderPublicKey = '// triple checked',
$senderPrivateKey = '//triple checked',
$assetId = '//triple checked';
$recipient = '//triple checked';
$amount = 100;
$fee = 100000;
$timestamp = time() * 1000;
$assets = new \Wavesapi\Assets([
  'host'    => 'http://localhost:6869,
  'api_key' => '//triple checked'
]);
$assets->setCredentials($senderPublicKey, $senderPrivateKey);
$assets->broadcastTransfer([
  'assetId'       =>  $assetId,
  'recipient'     =>  $recipient,
  'amount'        =>  $amount,
  'fee'           =>  $fee,
  'feeAssetId'    =>  null,
  'timestamp'     =>  $timestamp,
  'attachment'    =>  'test'
]);

The asset is not sponsored so I am assuming I leave feeAssetId null.
I am also assuming this isn't actually connecting to my local node which I believe is the problem.
For example if I do:
$assets = new \Wavesapi\Assets([
  'host'    => 'http://localhost:6869,
  'api_key' => '//triple checked'
]);
echo $assets->getBalance($addr);

Nothing is returned. Yet I can do
$test = file_get_contents('http://localhost:6869/assets/balance/'.$addr);
var_dump($test);

and that works

Okay so I have been working on this all day trying to figure it out.
So far I believe the problem is 'Array to string conversion' issues with php ver 7.
When I do a var_dump on the $assets, every variable is the same value (the last value in the loop).
Still trying to figure out the actual fix to this.

So to get this working in php 7.2 to send assets here is what I had to do:
Step 1:
modify line 18 of Transfer.php
change $txType to $type
modify line 34 of Transfer.php
change txType to type
Step 2:
modify line 33 of DataModel.php
change  $this->$f[0] to  $this->{$f[0]}
modify line 46 of DataModel.php
change $value = $this->$row[0]; to $value = $this->{$row[0]};
modify line 98 of DataModel.php
change $arr[$f[0]] = $this->$f[0]; to $arr[$f[0]] = $this->{$f[0]};
Step 3:
The script doesn't seem to actually send the data to the api, even changing the url's to the correct urls so I had to use curl
  $ch = curl_init();
  curl_setopt($ch, CURLOPT_URL, 'http://localhost:6786/transactions/broadcast');
  curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
  curl_setopt($ch, CURLOPT_POST, 1);
  curl_setopt($ch, CURLOPT_POSTFIELDS, $assets->data);
  $headers = array();
  $headers[] = 'Accept: application/json';
  $headers[] = 'Content-Type: application/json';
  curl_setopt($ch, CURLOPT_HTTPHEADER, $headers);
  $result = curl_exec($ch);
  if (curl_errno($ch)) {
    echo 'Error:' . curl_error($ch);
  }
  curl_close($ch);

This works to send an asset. I am sure plenty of other things are still broken but thought this might help others. :)


Tooltips quickly disappear after appearing

Video demonstration:
https://github.com/win32ss/supermium/assets/118223558/84a0e765-f4c8-4c10-b67c-fb26b8d53bc3
(stretched due to recorder setting)
When you go over a link to view the description (or full name of the video in the case of Youtube), the tooltip appears for a very brief moment and disappears.
Interestingly enough, if I make the tooltip appear outside of the browser window, it doesn't disappear until I move the cursor out again (works as intended).
OS: Windows 7 Professional 64-bit
Backend: GPU (ANGLE_D3D11)

I have noticed this issue in testing on XP. I have just determined this to be part of the far-reaching "lazy compositor" bug that appears to be responsible for many graphical glitches in Supermium 119:

Issues #124 and #130

Issues with text randomly disappearing

Issues with graphical elements from other webpages or the browser UI appearing within the space of web pages

Unfortunately, the fix (to forcefully update the HUD of the web page view) is currently bundled with a debugging method where newly-rendered graphical elements are highlighted in green (--ui-show-paint-rects). If the rebase to 121 does not fix this issue, like how similar issues occurred in 117 but not 118, the proposed fix will be implemented in Supermium 121.

i have the same issue, plus graphic corruption in facebook page.
This temp solve, works more or less well, except in outlook page, the tooltip appears & disappear fast
But the rest seems to work fine
"chrome.exe --enable-features=DisableVideoOverlayIfMoving"
OS: Windows 7 Professional 32-bit

Updated to v121, the problem continues.

I've been getting the this as well

If this is gonna help: In the good version (Chromium 109), tooltips were rendered by system (Windows). In "bad" versions, tooltips were "trying to be" rendered by the browser (System tooltip has system color scheme while CHRM tooltip has the browser's own color scheme)

Yes, I had been looking into taking many UI elements back into Windows' control (not just tooltips but also menus and toolbars, and possibly even reducing the custom titlebar to where only the omnibox and tabs are rendered by the browser).
This was also tied to another rendering issue that was presently in 117 and 119 (but not 115, 118 and 121) on some of my test setups, so I was a little surprised to see it persisted for others. But I think the restoration of native elements will reduce the dependency on the frequently-changing, frequently-breaking Chromium composition and rendering components.


Move per-deployment assets from this repo & legacy repos to osparc-config

All per-deployment config, secret,s, assets, such as:

Maintenance Pages HTML
Passwords
Customizations oer deployment/product
... now are placed in the git repo osparc-configs.

### Tasks
- [ ] https://github.com/ITISFoundation/osparc-ops-environments/issues/546
- [ ] https://github.com/ITISFoundation/osparc-ops-environments/issues/229

Closing as all tasks mentioned are done


Migrate to spring Neo4j

This feature is not needed anymore. Neo4j is not queried directly from PathwayMatcher anymore.


Support for more keyboard locales?

Is support for more languages planned in the near future? Our users need support for Russian (Cyrillic) and Turkish.

Yes, I have more languages planned for August and September.
You can also implement new locales yourself, the same way English is currently implemented.

Is it possible to add the Arabic language?

I think the keyboard itself is pretty straightforward, but I got stuck with the RTL aspects of it.

I think the keyboard itself is pretty straightforward, but I got stuck with the RTL aspects of it.

I hope you can add Arabic language soon, there is millions of users need this language, what is the problem with RTL?

For me, it was things like editing the localizable string, designing the RTL parts in the library (is it even needed?) and figure out how RTL works. I mean, should the keyboard ignore the RTL locale and use LTR (feels wrong), should the layout do everything reverse etc.

For me, it was things like editing the localizable string, designing the RTL parts in the library (is it even needed?) and figure out how RTL works. I mean, should the keyboard ignore the RTL locale and use LTR (feels wrong), should the layout do everything reverse etc. What if your device uses a non-RTL language and enables the arabic keyboard?

You don't need to support RTL with Arabic language in keyboard layout, RTL for phrases, I'm working on keyboard on android and I support Arabic language without RTL and it works without problem.

How did you implement the circle with two lines above? It just adds lines above the current character?

In my keyboard on Android I didn't support RTL at all, even in the phrases, for me this does not affect the performance of the keyboard and it works properly. just support RTL with Arabic in keyboard settings if you want.
I didn't undrstand what you mean by circal with two lines above?

If you look at this image, the images are mirrored, but placed on the same position on the keyboard.
I've circled the "two line" button below. Do you have a screenshot of your Android keyboard?

I didn't see any problem with the layout, and the circle with two line it's Arabic Diacritics https://en.wikipedia.org/wiki/Arabic_diacritics
and this is the layout from android

So, the diacritics part is optional?

How about Icelandic ?

@lumpidu I'll add it to the next minor version 👍

If I want to contribute an Icelandic language pack, what exactly do you need ? Is there a how-to for creating a language pack ?

I have some instructions in the readmes, but I’m sure they’re not as instructive as they should be.

In the above pics the only difference I can spot is the currency symbol. This symbol changes depending on the dialled in general language. E.g. if I set my general language to German, there is a € symbol even for an Icelandic keyboard. For Icelandic, there is a kr symbol and I suppose for U.S. language there is a $ as on the right picture.

@lumpidu Yeah, but both above are Icelandic, the left is an iPhone 8 and the right an iPhone 13. The devices have the same region. Not sure what's going on.

@m3toh I have just released 5.5, which has support for Farsi, Kurdish Sorani and Arabic. 👍

The upcoming 5.6 will add support for Brazilian, Irish, Portuguese and Turkish.

I will add support for Greek in the next version, but I create a separate ticket for that, so I'm closing this.


FAB is hard to push

Only the part of the FAB that's on the toolbar is clickable. The part that floats over content can't be clicked.

Fixed! I added a subview to the SingleSubVC that collects the taps underneath where the FAB would be


Deis Logs in K8S

Kubernetes provides a number of useful logging abstractions. We need to determine the best way to use these services. We need to determine if other services (like logspout) are still necessary.

@technosophos I've been knee deep in logging as of late, so I'd like to help out with this.  My initial thoughts are that logspout won't be needed (seems k8s likes fluentd and has nice support for it), but that logger might still serve a useful function for anyone who simply doesn't want or need to ship their logs to somewhere outside the cluster, but would like to still aggregate them somewhere within the cluster and access them with deis logs.

fluentd performs the same task as what logspout does differently, though the end result is the same (logs need to get shipped to a syslog server). The trick is getting it installed on every node when global scheduling affinity is not supported yet on kubernetes.
I agree about logger still being a required component OOTB. There are blog posts on how some users have set up fluentd with elasticsearch but we should see if we can still ship syslog messages to a logger.

How does k8s schedule its own system components and "add-ons" to every node?  Obviously, that's a concern whether we stick with logspout or switch to fluentd.  k8s' own logging docs talk about clusters created with support for cluster-level logging, but they never actually show you how that's enabled.  (Or at least I'm not seeing it.)

How does k8s schedule its own system components and "add-ons" to every node?

It's part of the bootstrap process, so that's not something we can plug into very easily since we're targeting both existing and new clusters.

@krancour you are right, the k8s docs about cluster-level logging is not clear. Here's a post with more details
https://github.com/kubernetes/kubernetes/blob/master/cluster/saltbase/salt/fluentd-es/fluentd-es.yaml
One advantage of switching to fluentd is that is possible to provide the same functionality (forward to an external syslog server) and add new ones like forward logs to riemann https://github.com/wkf/fluent-plugin-riemann (I'm sure @jchauncey could be interested in this)

It's part of the bootstrap process

Is possible to just run kubeletet create -f. Is possible to run that after the k8s cluster is created

+1 on fluentd

Is possible to just run kubeletet create -f. Is possible to run that after the k8s cluster is created

Well, yes and no. To explain how this works (for others who may not know), the salt bootstrap process installs the manifest to /etc/kubernetes/manifests/fluentd-es.yaml on every node, which the kubelet will then read on boot and start that pod on every node that has that manifest file on-disk. We cannot run kubelet create -f on a node that we do not have SSH access to, such as the case with users who may want to run Deis on Google Container Engine. You only get access to the kubectl API so that's what we're restricted to. I recall somewhere on the k8s issue list that someone has found a workaround using port affinity, so that will likely be the workaround for Deis agents.

this might also be a good time to investigate running back to rsyslog rather than deis/logger. See https://github.com/deis/deis/pull/4502#issuecomment-142321849

see https://github.com/deis/logger.


Adding support for shahmukhi punjabi months

I am from Western Punjab where these months are frequently used and understood in shahmukhi. Hence adding these names.
Signed-off-by: H. M. Irfan Sadiq<EMAIL_ADDRESS>

Thanks for adding this @hmsadiq Will take a look as soon as I can

@poudel : Any suggestions or improvements?


Named parameters in while loop

Windows 10

DBeaver 7.2.2

Oracle 11G Express

I want to bind parameters inside of an anonymous block... but in a while loop. When I do this, dbeaver only request the parameters one time, and then uses the same parameters to the next iterations.
Step 1:

Step:2

Thank you for the report.

Some clarifications were required. The behavior you described happens on the server side. Therefore it works as expected.

yeah, but how I can clean these parameterse, cause it uses the same values for every iteration, and I dont want that

I believe there's nothing dbeaver can do. That begin/end block is seen by Oracle as a single statement. The database itself does not support getting multiple parameters for them.


[Question] GQTy cli uses "gqty.config.js" file for generating pages?

Hi.
I have the need to add some headers into the GQTY introspection.
The command gqty generate seems to work fine but when next build generates the pages the config it's not used.
so my question is GQTy cli uses "gqty.config.js" file for generating pages?
Many thanks
My config file:

I don't think I understand, gqty doesn't generate any pages, and the whole point of gqty is to only generate once, since all the queries are generated on runtime

The config is used on both "gqty generate" and cli programmatic usage https://gqty.dev/docs/cli/programmatic

Maybe it's a feature of FaustJS that uses the GraphQL endpoint to get the page data to generate it at NextJS' build time.
I'll try to goo deep into this.

If I am understanding the question correctly, gqty.config.js is only used when you generate the client.
Common use cases of next.js only imports the generated client, it should not be aware of gqty.config.js at all.
If you want to include custom headers at runtime, e.g. when you are doing next build, you may have to edit the fetch code of the generated client.

@aitoraznar checkout https://codesandbox.io/s/gqty-sandbox-forked-o9int?from-embed=&file=/src/gqty/index.ts:803-874 as an example of how to provide a custom queryFetcher to your gqty client. As mentioned above, the headers in the gqty config are only for the introspection request (which is why they are under the introspection property).

@aitoraznar please reopen and add more context if you find this is still an issue.


How to get both image & video types in listAlbums & Sorting in Pagination

I use PhotoGallery.listAlbums and mediumType prop required a object, not a List.  I expected result like that:
PhotoGallery.listAlbums(
        mediumType: [MediumType.video, MediumType.image],
      );

not
PhotoGallery.listAlbums(
        mediumType: MediumType.image, //or MediumType.video
      );

Because I wanna get all types of album.
And how to sorting list media after I get a new pagination? Sorry my English is not good. I hope you understand.

@Firelands128  Please help me..

I have the same problem
Is your problem solved?

Support this feature by making "mediumType" to be optional to be able to allow fetch both types of media after version 2.0.0.


"SecurityGroup" emits security groups and directory roles

If the app manifest is configured to issue groups claim with "groupMembershipClaims": "SecurityGroup", directory role information is also added to groups claim. This is explained here: Understanding the Microsoft Entra app manifest - Microsoft identity platform | Microsoft Learn. This behavior is not properly documented across Entra docs.

@billmath @cilwerner

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

Thanks, @billmath.
@cilwerner - Can you review the changes to the Identity Platform articles?
#hold-off

LGTM
#sign-off


Fix casting to float when assigning numeric values; fixes normalization of integer arrays

PR Checklist

[x] This comment contains a description of changes (with reason)
[x] Referenced issue is linked Fixes #836
[x] If you've fixed a bug or added code that should be tested, add tests!
[ ] Documentation in docs is updated

Description of changes
Newly, casting adata.X to the dtype of the array it is assigned values from.

Technical details
adata.X being of a dtype which annot be casted to the dtype of values (e.g. object if containing string) should not be a prominent problem, as this will trigger the set_numeric_values earlier during _assert_encoded.

Additional context

Resolved 264 packages in 3.18s
  × Failed to download and build `numpy==1.25.2`
  ╰─▶ Build backend failed to determine requirements with `build_wheel()`
      (exit status: 1)

      [stderr]
      Traceback (most recent call last):
        File "<string>", line 8, in <module>
        File
      "/home/docs/.cache/uv/builds-v0/.tmpFdZf6K/lib/python3.12/site-packages/setuptools/__init__.py",
      line 10, in <module>
          import distutils.core
      ModuleNotFoundError: No module named 'distutils'

      hint: `distutils` was removed from the standard library in Python 3.12.
      Consider adding a constraint (like `numpy >1.25.2`) to avoid building a
      version of `numpy` that depends on `distutils`.
  help: `numpy` (v1.25.2) was included because `ehrapy` (v0.11.0) depends on
        `lifelines` (v0.30.0) which depends on `numpy`

Why the heck is this happening now again. Sigh...


Make db:tunnel more windows-friendly (wip)

Use win32-process gem to spawn and kill processes.
SIGHUP (or unix signals for that matter) is not supported on
windows - ruby wrapper supports the equivalent of SIGINT though, so
use that.
Since subprocess terminates immediately on windows, there's no need
to wait when stopping tunnel
Add new_pgroup flag, since that seems to be the only way to be able
to SIGINT a subprocess. This flag is invalid on *nix, so we have to
ensure it's only used on windows
Add appveyor gem when testing on windows

Current coverage is 88.68% (diff: 77.77%)

Merging #196 into master will decrease coverage by 0.11%

@@             master       #196   diff @@
==========================================
  Files            37         37
  Lines          1152       1158     +6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1023       1027     +4
- Misses          129        131     +2
  Partials          0          0

Powered by Codecov. Last update ba26b3d...1d8ad52

@krallin do you have any suggestions regarding this? This feels a bit too if-defy for me at the moment, so suggestions are welcome 😄

Ha, one last thing. What do we use the win32-process gem for here? Looking through the docs it appears to re-implement Process.kill, so maybe that's where we use it, but... what breaks if we don't use this gem at all?

Can we use a single regexp / method to detect Windows? Perhaps even reuse Gem's?

I'll reuse the conditional that exists in Gem.win_platform?.
Can we make the Windows constants in Tunnel constant, or use subclasses? (I imagine this might address your if-def-style concern!)
I went with the constant method. I was considering subclassing the Tunnel class, but I felt that it would be too heavyweight just for a couple of constants.

What happens when the SSL_X vars aren't set? Do we care? Is this something we'll address in packaging in the Toolbelt?

In the tests, not much, since we mock the calls to the API. Codecov fails when trying to call codecov.io with the error below though:
C:/Ruby21-x64/lib/ruby/2.1.0/net/http.rb:923:in `connect': SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed (OpenSSL::SSL::SSLError)
This will definitely be something that should be addressed in the Toolbelt, since all calls to our API will otherwise fail.

Can we use a single regexp / method to detect Windows? Perhaps even reuse Gem's?

I'll reuse the conditional that exists in Gem.win_platform?.

Sounds great; thanks!

Can we make the Windows constants in Tunnel constant, or use subclasses? (I imagine this might address your if-def-style concern!)

I went with the constant method. I was considering subclassing the Tunnel class, but I felt that it would be too heavyweight just for a couple of constants.

Sounds good; thanks!

What happens when the SSL_X vars aren't set? Do we care? Is this something we'll address in packaging in the Toolbelt?

In the tests, not much, since we mock the calls to the API. Codecov fails when trying to call codecov.io

This will definitely be something that should be addressed in the Toolbelt, since all calls to our API will otherwise fail.

Thanks. I was indeed more concerned about runtime rather than just tests 😄.
For the toolbelt, IIRC Chef packages the Mozilla CA bundle via this software definition: https://github.com/chef/omnibus-software/blob/master/config/software/cacerts.rb, so we should be able to reuse this.
Thanks!

Ha, one last thing. What do we use the win32-process gem for here? Looking through the docs it appears to re-implement Process.kill, so maybe that's where we use it, but... what breaks if we don't use this gem at all?

Nothing breaks; the original reason I added it was because I got a warning that "a file descriptor could not be opened" (which obviously is a bit hard on Windows 😉) but everything worked otherwise fine. Now I cannot even reproduce the file descriptor warning, so I'm thinking it might be worthwhile to just remove the win32-process gem.

Ha, one last thing. What do we use the win32-process gem for here? Looking through the docs it appears to re-implement Process.kill, so maybe that's where we use it, but... what breaks if we don't use this gem at all?

Nothing breaks; the original reason I added it was because I got a warning that "a file descriptor could not be opened" (which obviously is a bit hard on Windows 😉) but everything worked otherwise fine. Now I cannot even reproduce the file descriptor warning, so I'm thinking it might be worthwhile to just remove the win32-process gem.

Thanks for checking! As far as I'm concerned, I'd rather we remove it until we establish that it's needed, but I'm fine with keeping it if you think that's the smarter move here.

Ha, one last thing. What do we use the win32-process gem for here? Looking through the docs it appears to re-implement Process.kill, so maybe that's where we use it, but... what breaks if we don't use this gem at all?
Nothing breaks; the original reason I added it was because I got a warning that "a file descriptor could not be opened" (which obviously is a bit hard on Windows 😉) but everything worked otherwise fine. Now I cannot even reproduce the file descriptor warning, so I'm thinking it might be worthwhile to just remove the win32-process gem.
Thanks for checking! As far as I'm concerned, I'd rather we remove it until we establish that it's needed, but I'm fine with keeping it if you think that's the smarter move here.

Actually @krallin , it looks like the CI fails consistently without having the win32-process gem. I don't know why that's the case, but without it the tunneling tests consistently hangs indefinitely. I haven't been able to reproduce this with the win32-process gem. Do you have any ideas?

Ha, one last thing. What do we use the win32-process gem for here? Looking through the docs it appears to re-implement Process.kill, so maybe that's where we use it, but... what breaks if we don't use this gem at all?

Nothing breaks; the original reason I added it was because I got a warning that "a file descriptor could not be opened" (which obviously is a bit hard on Windows 😉) but everything worked otherwise fine. Now I cannot even reproduce the file descriptor warning, so I'm thinking it might be worthwhile to just remove the win32-process gem.

Thanks for checking! As far as I'm concerned, I'd rather we remove it until we establish that it's needed, but I'm fine with keeping it if you think that's the smarter move here.

Actually @krallin , it looks like the CI fails consistently without having the win32-process gem. I don't know why that's the case, but without it the tunneling tests consistently hangs indefinitely. I haven't been able to reproduce this with the win32-process gem. Do you have any ideas?

None; evidently this is a pretty good reason to keep it around! It'd be nice to try and investigate why this is happening, but I have a feeling that this is probably not going to be worth the effort, especially if you're unable to repro locally.
Thanks a lot for double-checking! 😄
Cheers,

Alright, then I'll merge this. Thanks! 😄


pull request for e2e test at 12/28/2016 08:15:06

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


FFaker::Color.hex_code

I am not sure, why am i getting the following NoMethodError. I guess, it was supposed to work as:
[92] pry(main)> FFaker::Color.name
"blue"
But while trying to access hex_code and others except name method.
[84] pry(main)> FFaker::Color.hex_code
NoMethodError: undefined method hex_code' for FFaker::Color:Module from (pry):80:in pry'

Hi @veekram the reason behind Faker::Color.name is working correctly, and Faker::Color.hex_code is not is, In current version of ffaker i.e 2.2.0 the FFaker::Color module contains only one method which is name. I think we can able to use all other methods i.e hex_code, rgb_array,  rgb_list, rgba_array, rgba_list, hsl_array, hsl_list, hsla_array, hsla_list in next release.

@SagareGanesh Thank you for your response. I went through the documentation and found out those methods included. Thanks for letting me about it. Let's make it available on the next release. Cheeers!


[Bug] Resizing a widget breaks DnD in release environment.

Description
[What happened]
Steps to reproduce the behaviour:
Add steps to reproduce this behaviour, include console / network logs & screenshots

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

Important Details

Version: [Cloud / Self-Hosted vx.x]
OS: [e.g.MacOSX]
Browser [e.g. chrome, safari]
Environment [production, release, deploy preview]

@marks0351 this seems like a critical issue. Can you add steps?

@yatinappsmith seems like a release blocker

its a release-blocker yes @Nikhil-Nandagopal

@marks0351 please update the description

@vivekverma2312 mark this as a release blocker

@vivekverma2312 isn't this verified? Kindly update

@somangshu The fix is working fine on release. Can we cherry pick this to prod.

I believe this was already present on release and should have gotten promoted. @marks0351 please confirm

@somangshu this is in release branch of CE and EE but not in master branch should i cherry-pick?


[v5] Update Devtools in production docs section

Describe the bug
Devtools in production docs code section is outdated.
This import:
const ReactQueryDevtoolsProduction = React.lazy(() =>
  import('@tanstack/react-query-devtools/build/lib/index.prod.js').then(
    (d) => ({
      default: d.ReactQueryDevtools,
    }),
  ),
)

throws error:
Cannot find module<EMAIL_ADDRESS>or its corresponding type declarations.ts(2307)

Your minimal, reproducible example
https://tanstack.com/query/v5/docs/react/devtools#devtools-in-production
Steps to reproduce

Update react-query version to:
"@tanstack/react-query": "5.0.0-beta.23",
"@tanstack/react-query-devtools": "5.0.0-beta.23",

Copy-paste this example to your project:
https://tanstack.com/query/v5/docs/react/devtools#devtools-in-production

TS linter will mark this as an error

Expected behavior
@tanstack/react-query-devtools/build/ has legacy and modern folders in it.
I guess this code should look like this?
const ReactQueryDevtoolsProduction = React.lazy(() =>
  import('@tanstack/react-query-devtools/build/modern/production.js').then(
    (d) => ({
      default: d.ReactQueryDevtools,
    }),
  ),
)

How often does this bug happen?
Every time
Screenshots or Videos

Platform

Tanstack Query adapter
react-query
TanStack Query version
5
TypeScript version
5.2.2
Additional context
No response

good catch. can you please raise a PR to update that section?
https://tanstack.com/query/v5/docs/react/devtools#devtools-in-production
I think the modern bundlers section should be unaffected because we've updated the package.json exports. I'm wondering if we should revert the order in the guide, because most bundlers should understand package.json exports by now ...

Hi @TkDodo
What is the alternative of /build/lib/index.prod.js for version 5, when is used with legacy tool like CRA?

build/modern/production.cjs I think

@TkDodo shows error on the terminal for all possibilities ``modernorlegacy. The error is like: Package path ./build/modern/production.cjs is not exported from package```

hmm @lachlancollins can you maybe have a look here please?

Hi @snelsi, if I'm not mistaken, you should just import from @tanstack/react-query-devtools/production rather than referencing the direct file, and your bundler should resolve the file. Also I don't think you need to wrap in lazy, check out this example. Docs might need updating, lmk how it goes!

@lachlancollins
I don't think importing @tanstack/react-query-devtools/production works. 😥
At least not in my Next.js app

However, import "@tanstack/react-query-devtools" seems to work fine?

Uh, I think I'm just missing
"module": "nodenext",
"moduleResolution": "nodenext",

in my tsconfig.json

someone please update the documentation on how the imports really work (on the rc branch). Thanks.

After the release this is still an issue.
I checked the source and the package.json states the following exports
"exports": {
    ".": {
      "import": {
        "types": "./build/modern/index.d.ts",
        "default": "./build/modern/index.js"
      },
      "require": {
        "types": "./build/modern/index.d.cts",
        "default": "./build/modern/index.cjs"
      }
    },
    "./production": {
      "import": {
        "types": "./build/modern/production.d.ts",
        "default": "./build/modern/production.js"
      },
      "require": {
        "types": "./build/modern/production.d.cts",
        "default": "./build/modern/production.cjs"
      }
    },
    "./package.json": "./package.json"
  },

Thinking the /production should be the one I want in this case.
When I use the following code inside my NextJS app
const ReactQueryDevtoolsProduction = React.lazy(() =>
  import('@tanstack/react-query-devtools/production').then(imp => ({
    default: imp.ReactQueryDevtools,
  })),
);

I still get Cannot find module '@tanstack/react-query-devtools/production' or its corresponding type declarations.ts(2307)
It works fine though - in production and development build of nextjs.
Anyone knows how to fix this properly? If so I can also update the README with the resulting changes

yeah I haven't looked into this, but we need to straighten this out. @DamianOsipiuk @lachlancollins with the new bundling setup, do you know what's the best way to import the devtools in production ?

Just a confirmation, that i wasn't able to make work lazy import of production bundle in legacy way(modern one is not accessible to me as i can't adjust TS config module properties to required)

@todorone @bastibuck @snelsi
Could any of you try locally patching devtools package and add following to package.json -> exports?
"exports": {
  ...
  "./build/modern/production.js": {
    "types": "./build/modern/production.d.ts",
    "import": "./build/modern/production.js",
    "default": "./build/modern/production.cjs"
  },
},

And then try
const ReactQueryDevtoolsProduction = React.lazy(() =>
  import('@tanstack/react-query-devtools/build/modern/production.js').then(
    (d) => ({
      default: d.ReactQueryDevtools,
    }),
  ),
)

@todorone @bastibuck @snelsi Could any of you try locally patching devtools package and add following to package.json -> exports?
"exports": {
  ...
  "./build/modern/production.js": {
    "import": {
      "types": "./build/modern/production.d.ts",
      "default": "./build/modern/production.js"
    },
    "require": {
      "types": "./build/modern/production.d.cts",
      "default": "./build/modern/production.cjs"
    }
  },
},

And then try
const ReactQueryDevtoolsProduction = React.lazy(() =>
  import('@tanstack/react-query-devtools/build/modern/production.js').then(
    (d) => ({
      default: d.ReactQueryDevtools,
    }),
  ),
)

This works in next.js with following tsconfig:
"module": "esnext",
"moduleResolution": "node",

nice, then lets put that in the docs and @DamianOsipiuk I guess it means changing exports too ?

Just updated it in my app and it works with the new version.
IMO this can be closed

After adding moduleResolution: NodeNext and module: NodeNext I am now getting the following error
The inferred type of 'ReactQueryDevtoolsProduction' cannot be named without a reference to<EMAIL_ADDRESS>This is likely not portable. A type annotation is necessary.ts(2742)
Also, changing the moduleResolution and module to NodeNext messes up with the project such that every other import requires to have the file extension. And without NodeNext, then @tanstack/react-router routes has type issues

After adding moduleResolution: NodeNext and module: NodeNext I am now getting the following error
The inferred type of 'ReactQueryDevtoolsProduction' cannot be named without a reference to<EMAIL_ADDRESS>This is likely not portable. A type annotation is necessary.ts(2742)
Also, changing the moduleResolution and module to NodeNext messes up with the project such that every other import requires to have the file extension. And without NodeNext, then @tanstack/react-router routes has type issues

Hi @melvinotieno , this is a separate issue. Could you please create a new issue and a repro?


Clear out attribute table when no events are found

Should revert to how the attribute table looks in the template

fixed


feat: integrate WeaveID login

Integrate WeaveID based on https://community.xyz/opportunity#QfwCoWMZq-L1WZe580RpsOQLjWWq4iXGDW7Zvifi1_4

Thanks for the contribution! I'm not sure if this is just me, but it seems that I'm unable to add another keyfile using this method after logging in for the first time.
CC @martonlederer

Thanks for the contribution! I'm not sure if this is just me, but it seems that I'm unable to add another keyfile using this method after logging in for the first time.
CC @martonlederer

@t8 I managed to reproduce it. I will start working on a fix later today!

Fixed a bug that caused the profiles to become undefined when the user tried logging in using the same key file twice.

@leNicDev Was playing around with this and encountered an annoying behavior with the WeaveID modal. When trying to go out of the modal by clicking on the right side of the page the file selector is triggered.

@leNicDev Was playing around with this and encountered an annoying behavior with the WeaveID modal. When trying to go out of the modal by clicking on the right side of the page the file selector is triggered.

I think this is an issue with the z-index of either WeaveID or our input. Any thoughts @martonlederer?

@littledivy @t8 Fixed it by increasing the z-index of #popupdarkbg. Might be a possible improvement for WeaveID.


New version: Waterfox.Waterfox version G6.0.9

Pull request has been created with komac v2.1.1 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

(Automated response - build 776.)

Manual Validation ended with:

Hi @Exorcism0666,
The installation isn't unattended. Is there an installer switch to have the package install silently?
(Automated response - build 788.)


neat thing

This looks great. It's awesome to see the cool vfx and graphics that you're contributing to the mod!
For posterity, here's what this looks like in game:

Going to go ahead and merge this.


Updates Channel.js to Repeater.js

Motivation
Make the showcase section more up-to-date. This PR fixes the Issue #4550.
Have you read the Contributing Guidelines on pull requests?
Yes!
Test Plan
Just visit a showcase page from this branch to see the result.
Related PRs
There are no related PRs.

@lucalves You might want to change your top post:
This PR fixes the Issue #4550. -> Fixes #4550.
When you use the Fixes #issue_number keyword GitHub will automatically close the linked issue when the PR gets merged.
Read the GitHub docs, linking a pull request to an issue using a keyword to learn more about this.

Thanks, fixes https://github.com/facebook/docusaurus/issues/4550