Unable to update

Describe the bug
Unable to update
To Reproduce
rec.table('data').update({"cookie":cookie, "qrdata":qrdata, "otptime":'2023-01-08 13:21:52.676377'}).filter('email',"eq",email).execute()
Expected behavior
Update without error
Output
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Additional info
The update takes place, but the error is thrown

filter is a keyword in Python so I think the operation might not be performing what you expect it to be doing. Could you instead try using .eq?
rec.table('data').update({"cookie":cookie, "qrdata":qrdata, "otptime":'2023-01-08 13:21:52.676377'}).eq('email', email).execute()
Seems to work fine on my end. Going to close for now but let me know if there are further issues

Filter case seems to work fine on my end too - could you check that the input fields have valid types?

Closing for now but feel free to re-open if issue persists


Auth: add TLS client auth support

What this PR does / why we need it:
Adds a support for TLS client auth.
Which issue this PR fixes(if applicable):
fixes #1119
Special notes for reviewers:

Release note:

Added support for TLS client authentication against OpenStack for all components.

/check cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/ok-to-test

/test cloud-provider-openstack-acceptance-test-e2e-conformance

HTTPS is not enabled in my environment, @kayrus have you tested this PR?

HTTPS is not enabled in my environment, kayrus have you tested this PR?

@lingxiankong I also don't have TLS env. I added TLS auth support as it is done in common.

HTTPS is not enabled in my environment, kayrus have you tested this PR?

@lingxiankong I also don't have TLS env. I added TLS auth support as it is done in common.

Well, based on the environments of both two of us, TLS is not in common :-) (kidding)
I'd rather we only merge code that has been tested in real environment so that we are confident that the added options are useful. So hopefully there will be someone popping in asking for TLS support or have HTTPS supported can help to test.
/hold

@lingxiankong the TLS configuration code is quite common. See https://github.com/etcd-io/etcd/blob/b28a6272b73359591fe22bac5ac8cae8afdbe0cc/clientv3/yaml/config.go#L86 or https://github.com/gophercloud/utils/blob/e38af13814d1b40e82c64bb4d137b0634f629af0/terraform/auth/config.go#L192
From my perspective it is quite safe to merge it.

/test cloud-provider-openstack-acceptance-test-lb-octavia

/override openlab/cloud-provider-openstack-acceptance-test-csi-manila

/unhold
/lgtm

@jichenjc @ramineni could you please review this one?

/lgtm
looks like the manila CI still have some problem ... wait for that complete or maybe need trigger again

/lgtm
/approve


Unable to reliably build containers

I am having spurious results installing the visual studio build tools in a docker container. Here are the host system details:
Host OS

Windows 10 Pro
Version 1809
OS Build 17763.503

Docker Desktop

Version <IP_ADDRESS> (31259)
Build 8858db3

Dockerfile
# escape=`

# Two days ago I was successful with this docker base image:
#FROM microsoft/windowsservercore:ltsc2016

# I have tried the following which result in error codes 255 and<PHONE_NUMBER> from the Install.cmd  RUN command below
#FROM mcr.microsoft.com/windows/servercore:ltsc2019
#FROM microsoft/dotnet-framework:3.5-sdk-windowsservercore-1709
#FROM mcr.microsoft.com/windows:1809
FROM mcr.microsoft.com/windows:1809-amd64

# Reset the shell
SHELL ["cmd", "/S", "/C"]

# Set up environment to collect install errors.
COPY Install.cmd C:\TEMP\
ADD https://aka.ms/vscollect.exe C:\TEMP\collect.exe

# Download channel for fixed install.
ARG CHANNEL_URL=https://aka.ms/vs/16/release/channel
ADD ${CHANNEL_URL} C:\TEMP\VisualStudio.chman

# Download and install Build Tools for Visual Studio 2019 for native desktop workload.
ADD https://aka.ms/vs/16/release/vs_buildtools.exe C:\TEMP\vs_buildtools.exe
RUN C:\TEMP\Install.cmd C:\TEMP\vs_buildtools.exe --quiet --wait --norestart --nocache `
    --channelUri C:\TEMP\VisualStudio.chman `
    --installChannelUri C:\TEMP\VisualStudio.chman `
    --add Microsoft.VisualStudio.Workload.VCTools --includeRecommended`
    --installPath C:\BuildTools

# Use developer command prompt and start PowerShell if no other command specified.
ENTRYPOINT C:\BuildTools\Common7\Tools\VsDevCmd.bat -arch=x86 &&
CMD ["powershell.exe", "-NoLogo", "-ExecutionPolicy", "Bypass"]

SHELL ["powershell"]
RUN iex ((New-Object System.Net.WebClient). DownloadString('https://chocolatey.org/install.ps1'))
RUN choco install cmake --installargs 'ADD_CMAKE_TO_PATH=System' -y --no-progress

It seems there is no consistency in being able to install the build tools in any of the windows containers.
I was able to get the native-desktop image to build but it includes a bunch of stuff I didn't want and I was also interested in using VS 2019.
Additionally the top FROM directive (servercore:ltsc2019) worked two days ago but my computer was having issues with Explorer.exe so I reinstalled windows.
vslogs_2147483651.zip
vslogs_255.zip

There are restrictions, such as you can't install .NET Framework as part of the VS install into Windows containers, which don't install .NET Framework (at least certain versions) by default. Attempts will fail in unpredictable ways, which is why examples start with a dotnet-framework tagged image.
To troubleshoot further, logs are needed. Given the sporadic nature, this could be a connection issue through your configured network (nat, transparent, or otherwise).

I meant to include the logs. I have edited the original post and added the log. I did see the logs with 255 in the name was failing to add .NET. I will look at what .NET images are available and try that.

Could you recommend a FROM line for installing VS 2019? I have tried FROM mcr.microsoft.com/dotnet/framework/sdk:4.7.2 and `` both of which failed. See the attached logs.
vslogs_2147483651_4.7.2_sdk.zip
vslogs_2147483651_4.7.2-sdk-windowsservercore-1709.zip

Neither log is showing an install error. They are showing that the install succeeded, actually. Sure it's not one of the subsequent commands? The errors correspond to ERROR_CLIENT_SERVER_PARAMETERS_INVALID (could be an issue with your Docker install) and HRESULT_FROM_WIN32(ERROR_FILE_NOT_FOUND), which could be anywhere.

Given the logs zip were generated and only command that is capable of generated them is the VS install that would pretty conclusively point to the vs install failing. Am I thinking about that wrong? I am going to look through the logs and maybe capture the output from the docker run as well.

I have attached the shell output from running the log vslogs_2147483651_4.7.2-sdk-windowsservercore-1709.zip here. You can see at the end where I copy out the zip.
error_run.txt

I reinstalled windows and now the images are building reliably, I think something may have gotten broken with docker or another program I installed was interfering.


Does it help to cover the same area multiple times?

Dear community,
I'm opening this issue to discuss a conceptual doubt about mapping and localization, which then implies better (or worse) performances of Cartographer. I have found recently these mapping recommendations from NVIDIA, which contain the following sentence:

There is no need to drive through an already mapped area; it increases noise.

However, according to my experience with Cartographer, having more trajectories in the map helps the robot during the localization process since it increases the number of constraints which Cartographer uses to provide the pose estimate.
I would like to listen to your opinion about this, since after all my experience is quite limited. For example, do you think that it's better to map a corridor by driving the robot just one time along it or by driving it multiple times back and forth? Do you think this helps or it just increases noise as NVIDIA suggests?
Thank you for the help,
Marco.

I used to cover the same area multiple times when creating the initial map, but have stopped doing that. Now we just do it once, and check the quality of the generated map against a floorplan. What I found was that when I went through and inspected individual submaps in rviz, those from separate passes often were subtly misaligned, and while localisation worked fine at a macro level, it wasn't good enough for fine positioning.
Now just doing a single pass (and checking the map closely) we can get <2cm accuracy on a regular basis when doing pure localisation. Bonus that it's much quicker to map as well.


Fix diff_drive accel limit (#242)

This fixes the issue (#242) where diff_drive_controller does not apply acceleration limits when provided.
It has been tested on Foxy, and tested to compile but NOT tested to run on Galactic. At present I can only test on Galactic with Gazebo, and with the latest versions of everything, starting my controller results in an unrelated error.
I've created a new issue (#251) to add a test for this in future.

I noticed the checks failed, one due to clang-format and one due to colcon?
I've fixed up the formatting in another commit, presumably they will squash when you merge? (Sorry, I'm a bit new to this)
Also, is there a clang-format reference that you are using so I can prevent this in future?


Support InstantX/InstantID

What does this PR do?
This PR adds support for InstantX/InstantID.
Link to the original repository: https://github.com/InstantID/InstantID
Some of the code was copied from the original repository:

The entire ip_adapter
GaudiStableDiffusionXLInstantIDPipeline is based on StableDiffusionXLInstantIDPipeline

A brief summary of the changes:

Added GaudiStableDiffusionXLInstantIDPipeline inherited from GaudiDiffusionPipeline and StableDiffusionXLControlNetPipeline for building Stable Diffusion XL pipelines
text_to_image_generation.py script was modified according to the existing logic to use InstantID pipeline. If a control image and Stable Diffusion XL model is specified, the InstandID functionality will be used

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests?

Hi @regisss, could you please leave you review in this PR? Thanks!

@alekseyfa , Can you also provide performance numbers.

@alekseyfa , Can you also provide performance numbers.

 They should pass
python -m pip install .[tests]
python -m pytest tests/test_diffusers.py -v -s -k test_stable_diffusion_xl_instantid

Thanks for your help! I've updated the tests, now everything passes


background image and icon always back to default if i dont chose image again.

Laravel Version: 5.5
Voyager Version: 1.1
PHP Version: 7
Database Driver & Version: mysql maria-db

Hi Dear,
when i go to setting and for example change site title or something else , if i dont chose image again all image change to default .
i dont want to select image again always  that edit some other text .
so how can i set image to prevent back to default ?
many thanks.

Please search before opening a new issue.
Duplicate of https://github.com/the-control-group/voyager/issues/3105 and #3059


Remove sjpeg

Sorry, I am aware, what was wrong with sjpeg?

Nothing “wrong” per se, but it’s not really clear that it warrants extra maintenance burden. However, the burden is also not that much, so we have decided to put off the removal for now.


olm: 3.1.4 -> 3.1.5

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/olm/versions.
meta.description for olm is: "Implements double cryptographic ratchet and Megolm ratchet"
meta.homepage for olm is: "https://matrix.org/git/olm/about"
meta.changelog for olm is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.1.5 with grep in /nix/store/5qvc3fafwaf2p5d9lgvlnvwgkvk79xn1-olm-3.1.5

directory tree listing: https://gist.github.com/e41a4f7ab8a5dc2469b013f080a3b06a

du listing: https://gist.github.com/d0d5a12f395c60ea6fef25187c2c0570

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

60 total rebuild path(s)

16 package rebuild(s)

16 x86_64-linux rebuild(s)
15 i686-linux rebuild(s)
14 x86_64-darwin rebuild(s)
15 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gomuks
matrixcli
mautrix-whatsapp
mtxclient
nheko
olm
pantalaimon
purple-matrix
python27Packages.python-olm
python37Packages.matrix-nio
python37Packages.python-olm
python38Packages.matrix-nio
python38Packages.python-olm
spectral
weechatScripts.weechat-matrix
weechatScripts.weechat-matrix-bridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5qvc3fafwaf2p5d9lgvlnvwgkvk79xn1-olm-3.1.5 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

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

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

1 package marked as broken and skipped:
- python27Packages.python-otr

4 packages failed to build:
- pantalaimon
- python37Packages.matrix-nio
- python38Packages.matrix-nio
- weechatScripts.weechat-matrix

12 packages built:
- gomuks
- matrique (spectral)
- matrixcli
- mautrix-whatsapp
- mtxclient
- nheko
- olm
- purple-matrix
- python27Packages.python-olm
- python37Packages.python-olm
- python38Packages.python-olm
- weechat-matrix-bridge (weechatScripts.weechat-matrix-bridge)

Maintainer pings

The packages failing to build are not regressions, according to hydra-check:
Build Status for nixpkgs.pantalaimon.x86_64-linux on unstable
✖ (Dependency failed) pantalaimon-0.6.1 from 2020-06-17 - https://hydra.nixos.org/build/122412621

Last Builds:
✖ (Dependency failed) pantalaimon-0.6.1 from 2020-06-15 - https://hydra.nixos.org/build/122265451
✖ (Failed) pantalaimon-0.6.1 from 2020-06-11 - https://hydra.nixos.org/build/122070144
✖ (Failed) pantalaimon-0.6.1 from 2020-06-03 - https://hydra.nixos.org/build/120807952
✔ pantalaimon-0.6.1 from 2020-05-28 - https://hydra.nixos.org/build/120783146
✔ pantalaimon-0.5.1 from 2020-05-13 - https://hydra.nixos.org/build/119124797
✔ pantalaimon-0.5.1 from 2020-05-12 - https://hydra.nixos.org/build/119087322
✔ pantalaimon-0.5.1 from 2020-05-08 - https://hydra.nixos.org/build/118569280
✔ pantalaimon-0.5.1 from 2020-05-06 - https://hydra.nixos.org/build/118329061
✔ pantalaimon-0.5.1 from 2020-04-28 - https://hydra.nixos.org/build/117852228
Build Status for nixpkgs.python38Packages.matrix-nio.x86_64-linux on unstable
✖ (Dependency failed) python3.8-matrix-nio-0.12.0 from 2020-06-17 - https://hydra.nixos.org/build/122412307

Last Builds:
✖ (Dependency failed) python3.8-nio-0.10.0 from 2020-06-15 - https://hydra.nixos.org/build/122261238
✖ (Dependency failed) python3.8-nio-0.6 from 2020-01-22 - https://hydra.nixos.org/build/110755720
✖ (Dependency failed) python3.8-nio-0.6 from 2020-01-21 - https://hydra.nixos.org/build/110566190
✖ (Dependency failed) python3.8-nio-0.6 from 2020-01-14 - https://hydra.nixos.org/build/110358554
✖ (Dependency failed) python3.8-nio-0.6 from 2020-01-13 - https://hydra.nixos.org/build/109773697
✖ (Dependency failed) python3.8-nio-0.6 from 2020-01-03 - https://hydra.nixos.org/build/109312141
✖ (Dependency failed) python3.8-nio-0.6 from 2019-12-31 - https://hydra.nixos.org/build/109245449
✖ (Dependency failed) python3.8-nio-0.6 from 2019-12-20 - https://hydra.nixos.org/build/108653843
✔ python3.8-nio-0.6 from 2019-12-19 - https://hydra.nixos.org/build/108477223
Build Status for nixpkgs.weechatScripts.weechat-matrix.x86_64-linux on unstable
✖ (Dependency failed) python3.8-weechat-matrix-0.1.0 from 2020-06-17 - https://hydra.nixos.org/build/122412398

Last Builds:
✖ (Dependency failed) python3.8-weechat-matrix-0.1.0 from 2020-06-15 - https://hydra.nixos.org/build/122268853
✔ python3.7-weechat-matrix-0.1.0 from 2020-06-11 - https://hydra.nixos.org/build/122081251
✔ python3.7-weechat-matrix-0.1.0 from 2020-05-29 - https://hydra.nixos.org/build/120829531
✔ python3.7-weechat-matrix-0.1.0 from 2020-05-13 - https://hydra.nixos.org/build/119124484
✔ python3.7-weechat-matrix-0.1.0 from 2020-05-12 - https://hydra.nixos.org/build/119100657
✔ python3.7-weechat-matrix-0.1.0 from 2020-05-10 - https://hydra.nixos.org/build/118765670

the previous failures have since been fixed :)


Flutter release apk size is very large in flutter sdk version 1.20

I am making a very simple app with 15 screens. And when i run this command flutter run --release. My release apk size is 26.2mb, which is very large.
My flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 1.20.2, on Linux, locale en_US.UTF-8)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
[✓] Android Studio (version 4.0)
[!] Connected device
    ! No devices available

! Doctor found issues in 1 category.

My pubspec.yaml file contain these dependencies:
  cupertino_icons: ^0.1.3
  flutter_svg: ^0.18.0
  provider: any
  image_picker: ^0.6.7+4
  menu_button: ^1.2.1
  pin_code_fields: ^5.0.1
  flutter_pdfview:
  path_provider:
  http:

These files takes the more space. Is there any way that we can reduce the release apk size. For apps like this, the size is very large. In my previous flutter sdk version 1.17 the release apk size of 30 screen app is not more than 15 mb.

Hi @umair-ilyas
With a flutter create default project, the result of flutter build apk --release is 15.1 MB on version 1.17.5 and 15.3 MB on both the latest stable 1.20.2 and latest master 1.22.0-2.0.pre.36. It could be that for a larger, more complex production app that this discrepency will get larger. However there was a massive amount of changes between these two versions.
In any case, please follow the advice of @siddharthsaini and as stated in the flutter tool:
You are building a fat APK that includes binaries for android-arm, android-arm64, android-x64.
If you are deploying the app to the Play Store, it's recommended to use app bundles or split the APK to reduce the APK
size.
    To generate an app bundle, run:
        flutter build appbundle --target-platform android-arm,android-arm64,android-x64
        Learn more on: https://developer.android.com/guide/app-bundle
    To split the APKs per ABI, run:
        flutter build apk --target-platform android-arm,android-arm64,android-x64 --split-per-abi
        Learn more on:  https://developer.android.com/studio/build/configure-apk-splits#configure-abi-split

I'm closing this issue since this isn't the intended build method for release and you are using third-party plugins. If you are still overshooting your targets with the mentioned advice, let me know and I'll reopen this issue.
Thank you


Dark Mode UI Error

By enabling the dark mode text is not visible in the cards and also footer icons is old type I can upgrade that also please assign to me.

The elements in the Community Section footer are not aligned properly, making the layout look messy.
please assign this task to me

i have already assigned this issue to someone if the person does not complete in next 2 days will assign to you

this issue has been solved, so we are closing this issue.


Simply removed some blank spaces README.md

@RocketChat/DesktopApp

Closes #ISSUE_NUMBER

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


不支持YouTube logo生成吗。。。

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

预计今晚上线

You can modify the parameters to create logo similar to youtube. Generally not an issue though


Previous states' controllers are executing in current state

Hi all,
I'm using angular 1.5.8 and ui-router v0.2.11. I have configured all the routing in app.js. and added all the controllers' reference in index.html file.
The problem I'm getting is when a state is changed, previous state's controller is being loaded in the current page. So the Restangular API call previous is executing in the current page on state load and also on scroll (as next API call are configured on scroll). It is breaking our functionality.
I have tried $state.reload(), but it also didn't work. But reloading a page works as expected.
I want only the controller of current state should be executed and other controllers should be unbound for current state.

will it be possible to update to 0.3.x and test?

Please update to a recent version and retest. If it still doesn't work, please create a plunker that demonstrates how to reproduce the issue.
http://bit.ly/UIR-Plunk

Hi, I have a similar issue in which the previous states' controller is being loaded first before transitioning to new state. Am I perhaps doing something wrong? Please open developer console in below plunker. Once open, click on the "Go to more bleh" button and then click on the Meh tab. Looking in the console you'll see the moreBleh's controller instance get's loaded again before completing the transition to the new bleh state controller.
https://plnkr.co/edit/HvFNHopTvGtr3kXHAi4M?p=preview

@IzakK your problem is due to an interaction between the way ui-bootstrap  and ui-router work.  As far as ui-router is concerned, this is expected behavior.
When you click the tab, ui-bootstrap re-renders the tab contents.  The contents includes a ui-view so it links a new ui-view.   The ui-view is then loading the controller for the current state (which targets the ui-view).
I think that's what's happening, at least.

I haven't found that ui-bootstrap's tabsets to play nicely with ui-router states and generally just use the bootstrap markup, ui-sref, and a single ui-view to do my tab implementations.

I'm closing this issue because the original reporter hasn't responded, and I think your issue  "works as designed" as far as I can tell.  If you are certain there's an issue with ui-router, feel free to open a new issue.


Observation period issues

Describe the bug
Why can't tenants use observers?
Steps to reproduce

Your setup

Laravel version: [e.g.7.24]
stancl/tenancy version: [e.g.3.1]
Storage driver: [e.g.mysql]

That's strange. Events work completely fine. Can you try using events in booted()?
If events work, observers should work too.


If PasswordLessPKCS12Keystore is created, it is created empty and not as copy of existing keystore

If we build an image with java 17 (paketo-buildpacks_bellsoft-liberica buildpack) the truststore at runtime contains all certificates that were added before using the ca-certificates buildpack (without cert embedding).
The same build with java 21 produces at runtime a truststore without the additional certificates
Expected Behavior
The truststore at runtime in a java 21 application contains the certificates added by the ca-certificates buildpack.
Current Behavior
It seems that the NewPasswordLessPKCS12Keystore is created empty and not as a copy of the truststore from the paketo-buildpacks_bellsoft-liberica/jre layer.
keystore.go: DetectKeystore -> NewPasswordLessPKCS12Keystore
Steps to Reproduce

build a java 21 application using the paketobuildpacks/builder-jammy-base and add custom certificates using the ca-certificates buildpack without cert embedding
run the container
check the truststore

Motivations
We need our own certificates. As a workaround we can embed the certificates to have them added to the truststore at startup.

This is a known issue, see #344.

can we close as duplicate?

I'd say this is basically resolved.
libjvm 1.44.1 has the code fix, bellsoft liberica (and the other JVM provider buildpacks) have 1.44.1 merged in.
We haven't cut a bellsoft release, but that should happen today or tomorrow and we should get the fix out in our Friday release.

can confirm this was released in latest https://github.com/paketo-buildpacks/bellsoft-liberica/releases/tag/v10.4.5  which is embedded itself in https://github.com/paketo-buildpacks/java/releases/tag/v10.7.0


README update: To install cmake 3.4.3 from source in Linux

Do most Linux systems have a new enough CMake in their package manager? If so, maybe we should just point there.

Hi Jordan,
No, Ubuntu default make version is 2.x and that gave me night mare of many days of weird build errors till I figured out that I needed cmake version 3.4.3 to be build from source

On Jul 29, 2016, at 5:58 PM, Jordan Rose<EMAIL_ADDRESS>wrote:
Do most Linux systems have a new enough CMake in their package manager? If so, maybe we should just point there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/apple/swift/pull/3879#issuecomment-236328239, or mute the thread https://github.com/notifications/unsubscribe-auth/AA15CgZoC2F9QqE495eOwDduKRR-NVfLks5qaqGngaJpZM4JYqrU.


Add Gateway and ClusterInfoImport CRDs

Add Gateway and ClusterInfoImport CRDs and update corresponding manifests

A sample of Gateway is like below:

apiVersion: multicluster.crd.antrea.io/v1alpha1
kind: Gateway
metadata:
  name: k8s-node-1
  namespace: kube-system
gatewayIP: <IP_ADDRESS>
internalIP: <IP_ADDRESS>

A sample of ClusterInfoImport is like below:

apiVersion: multicluster.crd.antrea.io/v1alpha1
kind: ClusterInfoImport
metadata:
  name: test-cluster-west-kube-system-clusterinfo
  namespace: kube-system
spec:
  clusterID: test-cluster-west
  gatewayNodeInfos:
  - gatewayIP: <IP_ADDRESS>
    name: k8s-node-1
  serviceCIDR: <IP_ADDRESS>/18

Two new fields 'ServiceCIDR' and 'GatewayIPPrecedence' are added in MultiClusterConfig.

By default, MC controller will detect ClusterIP range automatically. If admin set the
ServiceCIDR config manually, MC controller will use the value in the config.
By default, MC controller will chose InternalIP of a Node as the GatewayIP, if admin set
GatewayIPPrecedence config as 'public', it will use ExternalIP as the GatewayIP.

Signed-off-by: Lan Luo<EMAIL_ADDRESS>

Codecov Report

Merging #3689 (14b44f8) into main (70bbcb8) will decrease coverage by 26.07%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #3689       +/-   ##
===========================================
- Coverage   64.33%   38.26%   -26.08%
===========================================
  Files         281      114      -167
  Lines       39771    15565    -24206
===========================================
- Hits        25588     5956    -19632
+ Misses      12196     9135     -3061
+ Partials     1987      474     -1513

Flag
Coverage Δ

integration-tests
38.26% <ø> (?)

kind-e2e-tests
?

unit-tests
?

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

Impacted Files
Coverage Δ

pkg/apis/controlplane/install/install.go

...trollers/multicluster/resourceexport_controller.go

pkg/agent/controller/networkpolicy/fqdn.go

pkg/apis/controlplane/v1beta2/sets.go

...ntroller/networkpolicy/networkpolicy_controller.go

...piserver/registry/controlplane/egressgroup/rest.go

pkg/support/dump_others.go

pkg/ovs/openflow/ofctrl_packetin.go

pkg/agent/proxy/service.go

pkg/controller/networkpolicy/status_controller.go

... and 385 more


[5.4] Use array_wrap() in several places

This will replace blocks like:
if (! is_array($value)) {
    $value = [$value];
}

or
$value = is_array($value) ? $value : [$value];

with the use of array_wrap() helper function in several places.

Please use the Arr class rather than the global function, like the rest of the framework. :)


Allow loading *auth keys from Env variable exposed by CloudFormation and decoded by KMS

A new feature in AWS Lambda is the ability to load secrets from env variables.
See https://aws.amazon.com/blogs/aws/new-for-aws-lambda-environment-variables-and-serverless-application-model/
We want to support that for json2kinto editor_auth, reviewer_auth and auth

Is this addressed by #9?


Follow base plugin to v3

Updates:

When no public path set, don't default reference asset from root (e.g. /scripts.js). As this changes default behaviour, we signal with a major bump.
webpack figures out public path if not specified, by a "magic" auto path. This is stripped when templating the webpack generated asset.
Hint webpack 2 support
Fix bug when running with webpack 5 and rule configured with output.path. If [contenthash] used to generate filename, the templated asset used chunkhash.
Development deps updated
Run tests with supported webpack versions

Release:

First beta
Test
v3, when templated-assets-webpack-plugin release v3: https://github.com/jouni-kantola/templated-assets-webpack-plugin/issues/124

Ready to publish v3.0.0 with #45


[Enhancement] avoid copy to/from butil::IOBuff

Signed-off-by: fzhedu<EMAIL_ADDRESS>What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

chunkPB list is copied to/from IObuff when sending and receiving, this introduces extra two times of copy cost. This PR tries to avoid such copy, except that chunkPB should be copied if it will be sent by many times or its size > config::brpc_max_body_size when sending.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_be_unittest

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test


NotValidLength - 5000 chars limit?

Hi,
Is it possible to split a txt file in 5000 chars batches and submit them to google for translation? I suppose this error comes from this limit..
Thanks

deep_translator version:
Python version:
Operating System:

Description
Describe what you were trying to get done.
Tell us what happened, what went wrong, and what you expected to happen.
What I Did
Paste the command(s) you ran and the output.
If there was a crash, please include the traceback here.

@credo99 which error?

@credo99 you didnt post any error,  I don't understand your question


Add recipe for simulating synthetic lightcurves and fitting the PSD of an observed curve.

This PR adds a recipe which implements some functions to generate synthetic lightcurves, nominally:

the TK95 algorithm that will be added in the main gammapy code with 1.3
the more complex EMM algorithm

It also adds the function to create envelopes of the periodogram for a lightcurve, and a fitting function for the PSD of an observed curve. These function can also be found in the small package gammapy_SyLC that will probably be adapted to be used here in the future, once problems with installation are resolved.

Thanks a lot @cgalelli ...
@registerrier, should we precise in some headers the author, the date and the used Gammapy version?

Thanks a lot @cgalelli ... @registerrier, should we precise in some headers the author, the date and the used Gammapy version?

The gammapy version is defined in the env.yml. Maybe a print(gammapy.version) would be sufficient.
Regarding names, we have not done it yet, but we could/should. What would be the proposal for that? How does additional changes (maintenance, upgrades) is taken into account?

I finally could solve the problem with the import of gammapy_SyLC so the new version has everything implemented as a package function. Changed the text accordingly. Also added the author line at the end.

Thanks @registerrier , indeed i had made a bit of a mess in the restructuring of the notebook. Now it should be fine. I had also used an old local name for the notebook that was quite long and cumbersome -  returned to the name I had in the first commit.
The CI I think will still fail due to the import of gammapy_SyLC

Apparently it does not: i am trying to find an alternative way of installing the package in the environment file.


[323519703] Fix Oracle XML tasks

Existing Oracle XML tasks fail due to the missing argument placeholders in XML SQL query. What's more the XML metadata is retrieved sequentially for each object what makes it extremely slow. This PR fixes XML related issues and reduces extraction time:

Replace custom XML task with regular SQL query to improve the performance.
Add filters for *_TABLES and *_TASKS to exclude objects not existing in DBMS_METADATA.

Extraction time was reduced from over 10h to ~25 minutes.
XML Metadata is correctly extracted for all objects.

@shevek-google Would you have some time to re-review this PR?


Issue with libva-intel-drive 2.4.0

Just to let anyone know, if you're using an Intel HD Graphics card (Mine is the one integrated to an i5 4300u CPU on a thinkpad x240), you will have serious issues with that driver (Host System almost unresponsive, 1fps top...), solution is to downgrade it to 2.3.0 version, no issue at all again :)

Just to let anyone know, if you're using an Intel HD Graphics card (Mine is the one integrated to an i5 4300u CPU on a thinkpad x240), you will have serious issues with that driver (Host System almost unresponsive, 1fps top...), solution is to downgrade it to 2.3.0 version, no issue at all again :)

Thank you for your information, downgrading to 2.3.0 did the trick for my 4300m. :)

For debiian, it's the i965-intel-driver version 2.3.0, for reference.

Just to let anyone know, if you're using an Intel HD Graphics card (Mine is the one integrated to an i5 4300u CPU on a thinkpad x240), you will have serious issues with that driver (Host System almost unresponsive, 1fps top...), solution is to downgrade it to 2.3.0 version, no issue at all again :)

Thank you for your information, downgrading to 2.3.0 did the trick for my 4300m. :)

Just to know, did you succeed in making x265 stream working ? On windows it exit after a few seconds, on linux it's doesn't seems to work, even with hybrid driver for haswell.

does vainforeturns HEVC results ?

does vainforeturns HEVC results ?

Nope, only VP8 and h264... :(

Hello! Do you mind pasting the vainfo of both version so that we can compare please? Thanks :D

I don't have any results for HEVC in vainfo, so I didn't even tried making an x265 stream working...
But, for your information, I'm not using the beta version because it's closing itself with a "unknown error" after a few minutes (error: L-100). (I don't have any problem running the official .AppImage since I installed the beta and downgrade my drivers.)
I don't know if it can be of any help, but here is the vainfo for my system. It is exactly the same for the 2.4.0 and 2.3.0 version:

vainfo: VA-API version: 1.7 (libva 2.7.1)
vainfo: Driver version: Intel i965 driver for Intel(R) Haswell Mobile - 2.3.0
vainfo: Supported profile and entrypoints
VAProfileMPEG2Simple            : VAEntrypointVLD
VAProfileMPEG2Simple            : VAEntrypointEncSlice
VAProfileMPEG2Main              : VAEntrypointVLD
VAProfileMPEG2Main              : VAEntrypointEncSlice
VAProfileH264ConstrainedBaseline: VAEntrypointVLD
VAProfileH264ConstrainedBaseline: VAEntrypointEncSlice
VAProfileH264Main               : VAEntrypointVLD
VAProfileH264Main               : VAEntrypointEncSlice
VAProfileH264High               : VAEntrypointVLD
VAProfileH264High               : VAEntrypointEncSlice
VAProfileH264MultiviewHigh      : VAEntrypointVLD
VAProfileH264MultiviewHigh      : VAEntrypointEncSlice
VAProfileH264StereoHigh         : VAEntrypointVLD
VAProfileH264StereoHigh         : VAEntrypointEncSlice
VAProfileVC1Simple              : VAEntrypointVLD
VAProfileVC1Main                : VAEntrypointVLD
VAProfileVC1Advanced            : VAEntrypointVLD
VAProfileNone                   : VAEntrypointVideoProc
VAProfileJPEGBaseline           : VAEntrypointVLD

vainfo: VA-API version: 1.7 (libva 2.7.1)
vainfo: Driver version: Intel i965 driver for Intel(R) Haswell Mobile - 2.3.0
vainfo: Supported profile and entrypoints
VAProfileMPEG2Simple            : VAEntrypointVLD
VAProfileMPEG2Simple            : VAEntrypointEncSlice
VAProfileMPEG2Main              : VAEntrypointVLD
VAProfileMPEG2Main              : VAEntrypointEncSlice
VAProfileH264ConstrainedBaseline: VAEntrypointVLD
VAProfileH264ConstrainedBaseline: VAEntrypointEncSlice
VAProfileH264Main               : VAEntrypointVLD
VAProfileH264Main               : VAEntrypointEncSlice
VAProfileH264High               : VAEntrypointVLD
VAProfileH264High               : VAEntrypointEncSlice
VAProfileH264MultiviewHigh      : VAEntrypointVLD
VAProfileH264MultiviewHigh      : VAEntrypointEncSlice
VAProfileH264StereoHigh         : VAEntrypointVLD
VAProfileH264StereoHigh         : VAEntrypointEncSlice
VAProfileVC1Simple              : VAEntrypointVLD
VAProfileVC1Main                : VAEntrypointVLD
VAProfileVC1Advanced            : VAEntrypointVLD
VAProfileNone                   : VAEntrypointVideoProc
VAProfileJPEGBaseline           : VAEntrypointVLD

HEVC/H265 won't work if not supported and reported back by vainfo, x265 (as software decoding) isn't supported by Shadow only hardware. Don't worry about official appimage of Shadow, it's in fact a legacy version kept for compatibility until better solutions are found. The version to use on linux and that is being iterated on is Beta, and you should be able to also use Alpha.

Hi,
I have the same issue on my Thinkpad X230 (on Arch Linux).
The shadow client did not work correctly with libva-intel-driver version 2.4 (the image returned is not clear and blocked, all the system freezes)
With libva-intel-driver in version 2.3.0 all work well !
Workaround (Arch Linux) :
Downgrade the version of libva-intel-driver
wget http://archlinux.arkena.net/archive/packages/l/libva-intel-driver/libva-intel-driver-2.3.0-2-x86_64.pkg.tar.xz
sudo pacman -U libva-intel-driver-2.3.0-2-x86_64.pkg.tar.xz

My Hardware

Intel(R) Core(TM) i5-3320M CPU @ 2.60GHz
Intel HD Graphics 4000

vainfo: VA-API version: 1.9 (libva 2.9.1)
vainfo: Driver version: Intel i965 driver for Intel(R) Ivybridge Mobile - 2.3.0
vainfo: Supported profile and entrypoints
      VAProfileMPEG2Simple            :	VAEntrypointVLD
      VAProfileMPEG2Simple            :	VAEntrypointEncSlice
      VAProfileMPEG2Main              :	VAEntrypointVLD
      VAProfileMPEG2Main              :	VAEntrypointEncSlice
      VAProfileH264ConstrainedBaseline:	VAEntrypointVLD
      VAProfileH264ConstrainedBaseline:	VAEntrypointEncSlice
      VAProfileH264Main               :	VAEntrypointVLD
      VAProfileH264Main               :	VAEntrypointEncSlice
      VAProfileH264High               :	VAEntrypointVLD
      VAProfileH264High               :	VAEntrypointEncSlice
      VAProfileH264StereoHigh         :	VAEntrypointVLD
      VAProfileVC1Simple              :	VAEntrypointVLD
      VAProfileVC1Main                :	VAEntrypointVLD
      VAProfileVC1Advanced            :	VAEntrypointVLD
      VAProfileNone                   :	VAEntrypointVideoProc
      VAProfileJPEGBaseline           :	VAEntrypointVLD


libc/pthread: return EBUSY when barrier is in use

Summary
specified here:
https://pubs.opengroup.org/onlinepubs/009695399/functions/pthread_barrier_destroy.html
Impact
More confirm to the standard.
Testing

Please, squash these two commits: https://github.com/apache/incubator-nuttx/pull/4129/commits/ea0ddd188e222dadcaa6e0860ad97ebf79656969 / https://github.com/apache/incubator-nuttx/pull/4129/commits/790639c769bf4fea1677e7bb16fb069b7a77fa22

Done.

do you have any real code relying on pthread_barrier_destroy detecting it?

when pthread_barrier_init is called, the barrier is an uninitialized garbage. it's value should not be checked.

LTP testcase call pthread_barrier_init twice, and expect the second call return fail:
https://github.com/linux-test-project/ltp/blob/master/testcases/open_posix_testsuite/conformance/interfaces/pthread_barrier_init/4-1.c#L81-L85
https://github.com/linux-test-project/ltp/blob/master/testcases/open_posix_testsuite/conformance/interfaces/pthread_barrier_init/4-1.c#L113-L123
Do you have better method to fix it?

do you have any real code relying on pthread_barrier_destroy detecting it?

No, but it specify in the standard and check by LTP:
https://github.com/linux-test-project/ltp/blob/master/testcases/open_posix_testsuite/conformance/interfaces/pthread_barrier_destroy/2-1.c#L121-L131
We must to follow.

@xiaoxiang781216 please see the latest issue: https://pubs.opengroup.org/onlinepubs/9699919799/functions/pthread_barrier_destroy.html
It says (at the bottom):

The [EBUSY] error for a barrier that is in use or an already initialized barrier object is removed; this condition results in undefined behavior.

Ok, the change just check destroy path now, please take a look @yamt

my reading of the standard is that if an implementation detects the condition, it can report EBUSY.
it seems fine not to detect the condition.
IMO this is something which shouldn't be "fixed".
ltp is just broken or probably checking too specific implementation behavior.

i glanced at the latest nptl code of pthread_barrier_destroy.
it doesn't seem to return EBUSY either.
(older versions seem to do)

my reading of the standard is that if an implementation detects the condition, it can report EBUSY.
it seems fine not to detect the condition.
IMO this is something which shouldn't be "fixed".
ltp is just broken or probably checking too specific implementation behavior.

It isn't an implementation behaviour, it specify and recommand in standard:
[EBUSY]
The implementation has detected an attempt to destroy a barrier while it is in use (for example, while being used in a pthread_barrier_wait() call) by another thread.

i think it's better not to "fix" this.

And openbsd also return EBUSY too:
https://github.com/openbsd/src/blob/2207c4325726fdc5c4bcd0011af0fdf7d3dab137/lib/librthread/rthread_barrier.c#L86-L89

i think it's better not to "fix" this.

@yamt please give a reasonable explain why it's better to not fix it, even the standard explicitly state that "it is recommended that the function should fail and report an [EBUSY] error."

i think it's better not to "fix" this.

@yamt please give a reasonable explain why it's better to not fix it, even the standard explicitly state that "it is recommended that the function should fail and report an [EBUSY] error."

the EBUSY here is optional. (that's why the standard removed it from its ERRORS section.)
a portable application can not rely on it anyway.
less code is better for us.

a portable application can not rely on it anyway.

i guess i will change my mind if you give me an example of real applications relying on this EBUSY detection.
(real == not a test code)


Incompatibility with Windows 10 Pro version 1903

Camera Model                       |  Intel RealSense D435 |
Firmware Version                   | Signed_Image_UVC_5_11_6_250.bin  |
Operating System & Version |   Win 10 Professional for Workstations ver. 1903 |
Intel Device Driver Version    | <IP_ADDRESS>  (dated 1 April 2019) |
Kernel Version (Linux Only)    |  N/A                                         |
Platform                                 | PC |
SDK Version                           |  <IP_ADDRESS>9                          |
Language                               |  N/A                          |
Segment			              |  AR                          |
Intel RealSense Viewer and Depth Quality Tool no longer work after upgrade to Windows 10 Pro version 1903. When I launch the applications, nothing happens.
I was using the software without any issues in 1803, just before Windows Update requested authorisation for installing the 1903 update. On completion of the update, the RealSense software no longer worked.
Remedies attempted:
I uninstalled the SDK and reinstalled. No improvement.
Microsoft Support Case:
I opened a ticket with Microsoft Support, case number:<PHONE_NUMBER>
Microsoft's Support recommended me "to contact the manufacturer of your device to get if the have it latest version that is tested to run on Windows 10 1903 or we can go back to the previous build of Windows 10.".
Can you please confirm if this is an incompatibility with Windows 10 Pro 1903 and, if so, consider how to resolve this?
Thank you

Hello ntsarb,
You can visit my site and solve your problem manually
I have mentioned a simple and effective process on open dll file website.
If you will do not solve your problem then mail me via mail.
thanks
David Thomas

Hello ntsarb,
Did you resolve your problem?

Hello ntsarb,
Did you resolve your problem?

Thanks for your message. I had stored the camera in a drawer, as it wasn't working.
I reinstalled SDK Version <IP_ADDRESS>9, today, and it looks like it's back on duty.


Invert open/close

Hi, I have some covers and when they are open the card shows a closed shutter.
So if it's possible to invert this with an option in the card thois would be great

Hello @jurgen272 ,
I just updated the card for your request. You can now use a setting "invert_percentage" to fit your needs :) .
Tell me if it works correctly so I can close this issue.

Hi,
The card is ok now.
Thank you very much
J
Deejayfool<EMAIL_ADDRESS>schreef op 27 juni 2020 12:21:24 CEST:

Hello @jurgen272 ,
I just updated the card for your request. You can now use a setting
"invert_percentage" to fit your needs :) .
Tell me if it works correctly so I can close this issue.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Deejayfool/hass-shutter-card/issues/5#issuecomment-650537816


Search

Search for competitions
Search for athletes in or across all competitions
Filter for location,date, discipline (in competition)

https://github.com/lucaong/minisearch for local inbrowser search


add CSS3ColorsSwift

add CSS3ColorsSwift
Project URL
https://github.com/WorldDownTown/CSS3ColorsSwift
Description
A UIColor extension with CSS3 Colors name.
Why it should be included to awesome-ios (optional)
Because this library is simpler than other similar libraries. and you can write web color like UIColor in Swift 3.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

  :no_entry_sign:
  Found 1 link issue

Link issue by awesome_bot

Line
Status
Link

422
404
https://github.com/faunadb/faunadb-swift

  Generated by :no_entry_sign: danger

Cool library, @WorldDownTown. I find those names really useful, specially when testing quick mvps.
Thanks for contributing! 👍


Add date functions to core JS to allow current year to be added to footers in lesson code

Feature Request
var d = new Date();
var year = d.getFullYear().toString();

Enabling ©2016 FCC Kitchen to become ©2018 FCC Kitchen

Hello, can you provide a link to a page you are speaking about? Thank you.

https://learn.freecodecamp.org/responsive-web-design/applied-visual-design/adjust-the-color-of-various-elements-to-complementary-colors/

Hey MaX,
The code isn't generated quite the way you think it is, what you see is actually from json which functions aren't possible. However we agree that it is outdated and will work on changing that particular instance and related instances throughout the curriculum.


Run Dart finalizer callback for external types when they fail to post

Fix ZeroCopyBuffer memory leak related to discussion in https://github.com/sunshine-protocol/allo-isolate/issues/6.
Even though this memory leak seems unlikely, there should be an attempt to handle failure to post

Hello @trueb2!
Thank you very much!
Well, that sounds good, but did you also check out #7? I'm afraid without any proper unit tests here for that PR specifically, we could be doing double-free here.
I actually totally forgot about #6 to continue diving in the dartvm code, but anyway, is there a way you can make a small test for this?

Hi @shekohex, Thanks for supporting this project! I really appreciate the ability to integrate async Rust with Flutter.
I ran the change for a couple days with real-world data and added calls to post before the store pointer is set, other than that I don't add more test cases. The current container tests already simulates the VM calling the finalizer. We could still mock a random failure into the post interface and assert success and failure appropriately.
Failure to post or retry post could end up leaking expensive Dart allocations waiting for a message on the native port. I am thinking that the post function or another exposed function should block until success.
I see cargo fmt touched several files. Do you want me to keep or remove those changes?

see cargo fmt touched several files. Do you want me to keep or remove those changes?

Try cargo +nightly fmt since we use nightly in the CI for now.


PR for issue #36

generate only for css modules
replaced allGenerate with enabledMode flag
Improved npm link on example
Updated example - removed non-css modules .d.ts files that were previously present

Linting/Prettier passes

Amazing!! Thanks!!


Create a registration page to take info not covered by Github- codewar name, cohort, user type etc.

finished by this Tuesday.


QT 5.15 compatibility

QT 5.15 compatibility and some other minor fixes

I cannot specify only the new commits, and some old also included.

I have revised your Qt patches to implement backward compatibility with Qt v4.8<PHONE_NUMBER>67f0895c47983b942ae142ce9beb81

They seem fine, merged separately just in case


Does the project want the domain FONT.PARTS?

Hello:
I registered a bunch of domains as a joke, simply redirecting most of them to https://copypaste.wtf:

While doing this I noticed FONT.PARTS. I registered this because I'm familiar w/this project. If you want this domain I don't mind giving it for free, for now it's a redirect to the GitHub.

Hey @ctrlcctrlv, this is really kind, but we don't want the overhead of dealing with a custom URL, as the GitHub/readthedocs does well for our needs. Would just let this one expire come next year.


No such file errors after update

Issue Type: Bug
Install SFTP 1.12.9 SFTP/FTP sync Extension by liximomo, https://github.com/liximomo/vscode-sftp.
An update to VS Code Insiders after Friday 4/23/2021 cause this extension to fail. Every time you save a file the extension SFTPs it to a remote server. I didn't try over the weekend but come Monday, it began to fail on every update with a notification "No such file", "Source: SFTP (Extension)". The detail output is:
[04-27 08:17:32] [info] [file-save] d:\dev\fas-dev\web\src\verification\fas-document-select-card.js
[04-27 08:17:33] [error] Error: No such file
	at SFTPStream._transform (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\sftp.js:412:27)
	at SFTPStream.Transform._read (internal/streams/transform.js:205:10)
	at SFTPStream._read (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\sftp.js:183:15)
	at SFTPStream.Transform._write (internal/streams/transform.js:193:12)
	at writeOrBuffer (internal/streams/writable.js:358:12)
	at SFTPStream.Writable.write (internal/streams/writable.js:303:10)
	at Channel.ondata (internal/streams/readable.js:719:22)
	at Channel.emit (events.js:315:20)
	at addChunk (internal/streams/readable.js:309:12)
	at readableAddChunk (internal/streams/readable.js:284:9)
	at Channel.Readable.push (internal/streams/readable.js:223:10)
	at SSH2Stream.<anonymous> (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2\lib\Channel.js:167:15)
	at SSH2Stream.emit (events.js:315:20)
	at parsePacket (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\ssh.js:3288:10)
	at SSH2Stream._transform (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\ssh.js:694:13)
	at SSH2Stream.Transform._read (internal/streams/transform.js:205:10)
	at SSH2Stream._read (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\ssh.js:253:15)
	at SSH2Stream.Transform._write (internal/streams/transform.js:193:12)
	at writeOrBuffer (internal/streams/writable.js:358:12)
	at SSH2Stream.Writable.write (internal/streams/writable.js:303:10)
	at Socket.ondata (internal/streams/readable.js:719:22)
	at Socket.emit (events.js:315:20)
	at addChunk (internal/streams/readable.js:309:12)
	at readableAddChunk (internal/streams/readable.js:284:9)
	at Socket.Readable.push (internal/streams/readable.js:223:10)
	at TCP.onStreamRead (internal/stream_base_commons.js:188:23) when local ➞ remote d:\dev\fas-dev\web\src\verification\fas-document-select-card.js

VS Code version: Code - Insiders 1.56.0-insider (9a4d524f909e09079fec7abb82f2efde2172da7c, 2021-04-27T06:15:26.914Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2208)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.62GB (6.14GB free)

Process Argv
--crash-reporter-id 45e19e72-f964-447a-91b9-2b35016194eb

Screen Reader
no

VM
0%

Extensions (20)

Extension
Author (truncated)
Version

numbered-bookmarks
ale
8.0.3

es6-string-css
bas
0.1.0

pgformatter
bra
1.17.0

better-toml
bun
0.3.2

markdown-table-prettify
dar
3.4.0

dart-code
Dar
3.21.1

flutter
Dar
3.21.0

prettier-vscode
esb
6.3.2

php-intellisense
fel
2.3.14

go
gol
0.24.2

vscode-drawio
hed
1.4.0

php-cs-fixer
jun
0.1.158

vscode-phpfmt
kok
1.0.30

sftp
lix
1.12.9

camelcasenavigation
map
1.1.3

git-graph
mhu
1.30.0

vscode-docker
ms-
1.12.1

remote-ssh
ms-
0.65.4

remote-ssh-edit
ms-
0.65.4

es6-string-html
Tob
2.8.0

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
python383:30185418
pythonvspyt602:30291494
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstry244:30244315
pythonvspyt639:30291487
pythontb:30258533
pythonvspyt551cf:30291413
vspre833:30267464
pythonptprofiler:30281269
vshan820cf:30276953
pythondataviewer:30285072
vscus158:30286553
vscgsv2:30286872

@sandy081 I think this extension might be doing something with virtual workspaces. I think you made some changes in this area recently, so giving to you to investigate.

I have just changed the enablement of extensions in virtual workspaces. Can you please help me in understanding what is the issue here and how does enablement impacts this?

@sandy081 Please forward the issue to someone else. As inbox tracker, I thought there was a chance that one of the recent changes related to virtual workspaces could have caused this.

@mbinette I am sorry that above info is not helpful to diagnose the issue. I would recommend to file the issue against the extension first so that extension can analyze and diagnose from its side and see if there is an issue in the extension or if it finds some feature is not working as expected in VS Code then they can file an extension against VS Code providing proper steps to reproduce.

FYI @jrieken @bpasero if you changed something recently in this area. This extension hasn't been update in 2 years and stopped working on Monday's insiders.

Just got to know that (thanks @alexdima) the extension has not been updated from last 2 years. So it could be a change in VSC that caused this. Hence reopening and asking others to jump on and provide some insights.
Meanwhile @mbinette can you please provide steps to reproduce so that we can diagnose it easily?

The steps to reproduce are simple but does require you to have a remote SSH or SFTP service setup. I have a local Windows 10 workstation that syncs files over SSH to a Google virtual server. Follow the directions for the extension to setup the Host, Username, Password or Key, and the local and remote directories to sync. Then use the extension to sync files and you will get the error every time. Some files are correctly synced but others will have its contents removed  or truncated (file still exists).
I've installed the non-insiders edition of VS Code and the extension still works. Has Electron or Node been updated in Insiders Edition since last Friday? The errors are coming from Node but couldn't figure out if it had actually changed and when.

@mbinette That stack appears multiple times in issues in that npm module repository and it looks like it is caused by any server-side error.

Yes insiders has bumped node from 12.18.3 to 14.16.1, so very likely some behavior change is in play here. Can you provide the debug logs from the extension https://github.com/liximomo/vscode-sftp#debug

Here is the debug log. Doesn't look very helpful.
[04-29 14:32:19] [info] [file-save] d:\dev\fas-dev\build\k8s\tmpl\api-web.yaml [04-29 14:32:19] [trace] handle upload file for d:\dev\fas-dev\build\k8s\tmpl\api-web.yaml [04-29 14:32:19] [debug] Outgoing: Writing MKDIR [04-29 14:32:19] [debug] Outgoing: Writing CHANNEL_DATA (0) [04-29 14:32:19] [debug] Outgoing: Writing LSTAT [04-29 14:32:19] [debug] Outgoing: Writing CHANNEL_DATA (0) [04-29 14:32:19] [debug] Outgoing: Writing OPEN [04-29 14:32:19] [debug] Outgoing: Writing CHANNEL_DATA (0) [04-29 14:32:19] [debug] Outgoing: Writing FSTAT [04-29 14:32:19] [debug] Outgoing: Writing CHANNEL_DATA (0) [04-29 14:32:19] [debug] Outgoing: Writing FSETSTAT [04-29 14:32:19] [debug] Outgoing: Writing CHANNEL_DATA (0) [04-29 14:32:19] [debug] Outgoing: Writing WRITE [04-29 14:32:19] [debug] Outgoing: Writing CHANNEL_DATA (0) [04-29 14:32:19] [debug] Outgoing: Writing CLOSE [04-29 14:32:19] [debug] Outgoing: Writing CHANNEL_DATA (0) [04-29 14:32:19] [debug] Outgoing: Writing FSETSTAT [04-29 14:32:19] [debug] Outgoing: Writing CHANNEL_DATA (0) [04-29 14:32:19] [debug] Outgoing: Writing CLOSE [04-29 14:32:19] [debug] Outgoing: Writing CHANNEL_DATA (0) [04-29 14:32:19] [error] Error: No such file 	at SFTPStream._transform (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\sftp.js:412:27) 	at SFTPStream.Transform._read (internal/streams/transform.js:205:10) 	at SFTPStream._read (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\sftp.js:183:15) 	at SFTPStream.Transform._write (internal/streams/transform.js:193:12) 	at writeOrBuffer (internal/streams/writable.js:358:12) 	at SFTPStream.Writable.write (internal/streams/writable.js:303:10) 	at Channel.ondata (internal/streams/readable.js:719:22) 	at Channel.emit (events.js:315:20) 	at addChunk (internal/streams/readable.js:309:12) 	at readableAddChunk (internal/streams/readable.js:284:9) 	at Channel.Readable.push (internal/streams/readable.js:223:10) 	at SSH2Stream.<anonymous> (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2\lib\Channel.js:167:15) 	at SSH2Stream.emit (events.js:315:20) 	at parsePacket (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\ssh.js:3288:10) 	at SSH2Stream._transform (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\ssh.js:694:13) 	at SSH2Stream.Transform._read (internal/streams/transform.js:205:10) 	at SSH2Stream._read (c:\Users\mbine\.vscode-insiders\extensions\liximomo.sftp-1.12.9\node_modules\ssh2-streams\lib\ssh.js:253:15) 	at SSH2Stream.Transform._write (internal/streams/transform.js:193:12) 	at writeOrBuffer (internal/streams/writable.js:358:12) 	at SSH2Stream.Writable.write (internal/streams/writable.js:303:10) 	at Socket.ondata (internal/streams/readable.js:719:22) 	at Socket.emit (events.js:315:20) 	at addChunk (internal/streams/readable.js:309:12) 	at readableAddChunk (internal/streams/readable.js:284:9) 	at Socket.Readable.push (internal/streams/readable.js:223:10) 	at TCP.onStreamRead (internal/stream_base_commons.js:188:23) when local ➞ remote d:\dev\fas-dev\build\k8s\tmpl\api-web.yaml [04-29 14:32:24] [debug] Outgoing: Writing ping (GLOBAL_REQUEST<EMAIL_ADDRESS>

Thanks for the log, the error seems to be thrown from the server after FSETSTAT operation when syncing the file when local ➞ remote d:\dev\fas-dev\build\k8s\tmpl\api-web.yaml. I am not sure what causes this, one thing to check if the directory and file are present in the server. It is best if this issue is debugged by the extension author, closing as such.
Side note: noticed that https://github.com/liximomo/vscode-sftp is not using the latest version of ssh2 module, so maybe it has fixes for node 14 compatibility.

Hi all, wasn't having a problem until I just now upgraded vscode to 1.56.2.
Now the error is
[05-14 11:29:14] [warn] Can't set modified time to the file because Invalid handle
[05-14 11:29:14] [error] Error: Invalid handle
"Caused by extension" lmao, you just broke 845K installs.


Check wheter downloaded data are indeed of the size specified in RequestAsk

The CID that is linked to a Request can have a different size compared to what is specified in the Request's Ask, we should double check that the downloaded size is not different.

Agreed, but how can this be done without downloading the data? Is this information available in the manifest?


PowerShell task example reports that it can not recognize the term of "name"

##[error]name : The term 'name' is not recognized as the name of a cmdlet, function, script file, or operable program. Check

the example is as follows:
steps:
- powershell: Write-Host Hello $(name)
  displayName: Say hello
  name: firstStep
  workingDirectory: $(build.sourcesDirectory)
  failOnStderr: true
  env:
    name: Microsoft

I think the name is targeted to {evn.name} whose value is "Microsoft"
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7458d5b2-1bb7-7d87-7ce6-2b0c430b0155
Version Independent ID: edf5bbd3-a3f0-5426-e985-31290f8afd05
Content: YAML schema - Azure Pipelines
Content Source: docs/pipelines/yaml-schema.md
Product: devops
Technology: devops-cicd
GitHub Login: @steved0x
Microsoft Alias: sdanie

https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/utility/powershell?view=azure-devops
according to this doc, we should change the $(name) -> $env:name and it will work.
the modified command is as follows:

and the output is:


await user in @lucia-auth/sveltekit

Fixes crucial bug caused by pending Promise returned from load function. Caused by async transformUserData being left as a pending promise
This is the error caused if the bug is left like this:
Data returned from `load` while rendering / is not serializable: Cannot stringify arbitrary non-POJOs (data._lucia.user)

This is the returned value before
{
  _lucia: {
    user: Promise { [Object] },
    sessionChecksum: '...'
  }
}

This is the expected return value
{
  _lucia: {
    user: {
      userId: '...',
      username: '....',
      handle: 'mavthedev'
    },
    sessionChecksum: '...'
  }
}

To work around this, you can change your +layout.server.ts to this
import { handleServerSession } from "@lucia-auth/sveltekit";
import type { LayoutServerLoad } from "./$types";

export const load: LayoutServerLoad = async (e) => {
    const data = await (handleServerSession())(e)
    data._lucia.user = await data._lucia.user
    return data
}

I can’t seem to recreate this issue. Not type errors as well. Also, if this is an issue, it’s an issue with handleHooks() and maybe validateSessionUser()

I can’t seem to recreate this issue.
The issue can be recreated by making the transformUserData function async
Caused by async transformUserData being left as a pending promise


:s command errors without in pairs/ in ipairs.

What are the steps that lead to the bug occurring? Or, how frequently does the bug occur?
if you run a script line that only does "for i, v in Player:GetChildren()" for example, then the command will error.

What would be the normally expected or desired result?
The command should run as intended.

What is the actual result (the bug)?
Command Errors.

What is the impact of this bug?
Less knowledgable players may not be able to understand the bug/why their command works on roblox but not via the s command. Fortunately able to be fixed by just adding ipairs/pairs.

(Any additional context or information eg. in the form of error messages or screenshots.)

Not really a bug per se, but just because Adonis code execution currently only supports Lua (which Roblox previously used) and thus Luau-exclusive syntax and features such as __iter, compound assignments, type annotations etc. won't be compatible at the moment.

Not really a bug per se, but just because Adonis code execution currently only supports Lua (which Roblox previously used) and thus Luau-exclusive syntax and features such as __iter, compound assignments, type annotations etc. won't be compatible at the moment.

In plain english, use ipairs(Player:GetChildren()).

This is not really a bug, more of the fact that the sandbox module FiOne does not support loops without pairs, ipairs, etc...


Remove interim BBs and update the logic

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #20175
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #20174 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #20174   +/-   ##
=======================================
  Coverage    19.9%    19.9%
=======================================
  Files          55       55
  Lines        1437     1437
  Branches      218      218
=======================================
  Hits          286      286
  Misses       1138     1138
  Partials       13       13

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d2e9977...8d02a96. Read the comment docs.


Should we rename APIs? sangte -> atom

As a Korean, I really liked the name sangte since it just  makes sense.
However, after putting myself into foreign developers' shoes, I thought that they might not like it because they are not used to this word.
Thus, I am planning to change the name as below. I need some feedback! Should I go for it? Or stick with the current naming?
sangte -> atom
useSangte -> useAtom
useSangteActions -> useAtomActions
useSangteValue -> useAtomValue
useSetSangte -> useSetAtom
useSangteStore -> useAtomStore
useSangteCallback -> useAtomCallback
useResetSangte -> useResetAtom
useResetAllSangte -> useResetAllAtoms

In my opinion, current name sangte is okay and maybe(?) acceptable for foreign developers. (I'm korean though :))
But Following Recoil's case, they provide hooks like useRecoilState, useRecoilValue, we can do something like useSangteState, useSangteValue.
Or re-export everything like radix-ui's approach is worth considering :)
export function sangte () {...};
export const atom = sangte;

Jotai means a state in Japanese.
From the same perspective, The name Sangte looks okay.

In my opinion, current name sangte is okay and maybe(?) acceptable for foreign developers. (I'm korean though :)) But Following Recoil's case, they provide hooks like useRecoilState, useRecoilValue, we can do something like useSangteState, useSangteValue. Or re-export everything like radix-ui's approach is worth considering :)
export function sangte () {...};
export const atom = sangte;

Thank you for good suggestions! It seems like it would be nice to rename useSangte to useSangteState, because non-korean people might find it confusing.
I am planning to make these aliases by re-exporting!

atom -> sangte
useSangteState -> useSangte

Jotai means a state in Japanese. I guess you have already known this. From the same perspective, The name Sangte looks okay.

Agreed, but jotai does not have name 'jotai' in their apis.

Jotai means a state in Japanese. I guess you have already known this. From the same perspective, The name Sangte looks okay.

@kyun
Agreed, but jotai does not have name 'jotai' in their apis.

Agree to change names to some extent.
Zustand and Jotai each have the meaning of state, but they are not found in apis.
Such as useSangte seem quite interesting to Koreans, but people in other countries may not.
I think function names seem to be more important to describe which way it uses(Atomic or Flux or something).

useJotai()
useJotaiState()
...

What if jotai used the "Jotai" to their APIs, it is okay to me.
Because I have already known what "Jotai" is. (I didn't know that meaning before knowing "Jotai")
Almost of developers who try to use "Sangte", they may know what "Sangte" is.
so, I don't think "Sangte" makes them confused.

Thank you for sharing your thoughts everyone!
My current plan is:
Create alias atom for sangte, useSangteState for useSangte and keep all the API name as is.
There might be some changes after getting more feedbacks from other developers later on.
By the way I am planning to make a video about this library (in English) sooner or later!


launch factory app outside statechange callbacks

Closing this outdated PR. Please submit this to the latest sprint branch if needed.
Please include a JIRA ticket number in the PR comments.


fix: Support disable the Zookeeper Curator EnsembleTracker

What is the purpose of the change
fix #14232
Support disable the Zookeeper Curator EnsembleTracker
Brief changelog
Support disable the Zookeeper Curator EnsembleTracker
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Thank you very much for the prompt repair.
maybe org.apache.dubbo.registry.zookeeper.utilCuratorFrameworkUtils also need change?


Easier customization for image for note with other type [DITA OT 1.8]

Let's say we have DITA notes with othertype specified on it:
     <note type="other" othertype="computer-only">

The XSLT template:
<xsl:template match="*[contains(@class,' topic/note ')]" mode="setNoteImagePath">

located in:
 DITA-OT/plugins/org.dita.pdf2/xsl/fo/commons.xsl

has a commented out part which would have matched an image based on the othertype value, making it easier to customize the image.
Why would that part be commented out.

Seems to be related to #1234.


Move scripting in Travis config into script!

In order to more easily run the build process outside of travis for other consumers of the project and for us locally, we should move the bash script part of the Travis script into a script!
See:
https://github.com/wellcometrust/platform-api/blob/master/.travis.yml#L56

Yep. I was planning to do this with Alice when we made the build script changes.


Add /who page with player directory

Moving this to the Compendium (will add an indicator for online status, as well as a separate view to show those who are currently online).


Allow regex in routes

Adds regex routes as suggested in issue #1061.

I know this is broken code. I'm hoping that somebody who knows D will understand the intent and fix it or replace it with better.

The problem is that while the implementation in the old naive router implementation is straightforward, implementing this for the new DFA based router is a lot more involved. Apart from that it doesn't mix with the wildcard support, so there would have to be something that tells regex based routes apart from normal ones.
Another possibility, which other frameworks offer, is to only support the regex for placeholder variables:
router.get("/items/:(var:[0-9]+)/something", ...);

It would then be implemented as step after the basic route matching.

@s-ludwig As indicated in issue #1061, I don't have much to go on for how routes are supposed to work. I think I imitated the behavior that was in the code for wildcards, if by "wildcard" you mean "*". I don't know anything about a new DFA-based router. Is it documented somewhere?
I did note issues with "*" in the comments of the code. The Route constructor is overloaded to accept both a string and a regex.
router.get(regex(r"/items/(?P<var>[0-9]+)/something"), ...);
```
should work just fine.

Okay, using separate overloads would indeed avoid the issues in mixing the two approaches.
The DFA (erroneously called "tree" in the code, it's really a directed graph) is used by default instead of the linear Route probing: https://github.com/rejectedsoftware/vibe.d/blob/1908505af05154b172f97baca8624c2ff953cf60/source/vibe/http/router.d#L110
I'll fix the documentation later, but currently wildcards and placeholder variables are relatively limited (but useful enough for all usual needs):

Wildcards may only be at the end of the match string and match any string: /some/path* for example matches /some/pathxyz or /some/path/asdasd)
Placeholders must be at the end of the match string, or have to be followed by a slash: /foo/:placeholder/bar or /foo:placeholder are valid, but /foo/:placeholder1-:placeholder2 doesn't work.

Le 7 avr. 2015 14:05, "Sönke Ludwig"<EMAIL_ADDRESS>a écrit :

Another possibility, which other frameworks offer, is to only support the
regex for placeholder variables:
router.get("/items/:(var:[0-9]+)/something", ...);
It would then be implemented as step after the basic route matching.
—
Reply to this email directly or view it on GitHub.

The syntax will more likely be Python's named group, which std.regex
supports (-> "/items/:(?P[0-9]+)/something/").
That would indeed be very cool to have.

Closing this, since the implementation approach doesn't work anymore. I'll open a separate ticket instead.


WIP: Add tile pointing to build repo

I've updated this, it should work now.
I'm not sure how happy I'm with Manifest for this


ocf_datapipes==1.2.44

Pull Request
Description
Upgrade datamodel, fix for 15 minute data
How Has This Been Tested?

Ci tests

ran locally with changes

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Ive pushed this one thourgh too, @dfulu but any comments are very welcome


Update to sbt 0.13.16-RC1

and remove the workarounds needed for an older sbt version.

Cheers, @2m.


cmd/containerd: resolve config relative binary location

The default configuration is now resolved relative to installation
location of the binary. For example, if the binary is run from
/usr/local/bin/containerd, it now will find the configuration in
/usr/local/etc/containerd/config.toml. Overriding the config still
works as normal.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

Codecov Report

Merging #1323 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1323   +/-   ##
=======================================
  Coverage   35.56%   35.56%
=======================================
  Files          23       23
  Lines        2874     2874
=======================================
  Hits         1022     1022
  Misses       1627     1627
  Partials      225      225

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e6ba48f...9c23622. Read the comment docs.

IIUC, when containerd is installed on /usr/bin, the config file name becomes /usr/etc/containerd/config.toml.
But I've never seen /usr/etc directory...

FHS says Note that /usr/etc is still not allowed: programs in /usr should place configuration files in /etc.
https://refspecs.linuxfoundation.org/FHS_3.0/fhs/ch04s09.html

I don't think I would move this, it adds more confusion to where things are located.

@crosbymichael Fair enough. Would it be better to search through a few common locations, such as /etc, /usr/local/etc or do we just leave this as is?
Mostly, this was added to ease installation from tar to single root, but I removed the configuration file from #1322, so this isn't necessary.

For now, one absolute default location is best.  The amount of confusion this would cause vs what we gain will not be worth it


Update disable-dpi-awareness.md

#5664, Write-the-Docs-2020

@AlexB-SheldonMFG : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@gregvanl I've applied the spelling fix. I'll merge the PR now. Thanks.


Local testing when utilizing WebSocket

Writing a slackbot, pretty basic so far. To avoid exposing a public endpoint, I'm trying to use WebSocketMode to listen for events etc.
I have a container running the listener, and it works well. However, when doing development locally, having two listeners open seems to be an issue. Do you all have a way you do development with websocketmode?

Hi @tehreet, thanks for asking the question?

However, when doing development locally, having two listeners open seems to be an issue. Do you all have a way you do development with websocketmode?

Does this mean you have two connections for a single Slack app - 1) deployed app's connection 2) connection on your local machine (local dev)? If so, as you are experiencing, the Slack server-side sends events to your connections in a round-robin manner.
If you want to have development version of your app, the current recommendation is to create a new app for development/testing with the same set of OAuth scopes and functionalities.
I hope this was helpful to you. If everything is clear, may I ask you to close this issue?


Directions incorrect on debug screen.

SpongeForge spongeforge-1.8-1577-2.1-DEV-884

Vanilla 1.8.8

For future reference, only SpongeForge can be on the client (so issues like this should be made in SpongeForge).

I thought it may have been implying a deeper issue, rather then just the debug screen as I was struggling with directions again, but it was my own code fail instead.


Bugfix/restrict python version compatibility

Closes #1704
Thank you @tanmaik! There were a few tests failing on our part that I had to dig around and fix to get all tests to pass.

Disclaimer: This review was made by a crew of AI Agents.
Code Review Comment for CrewAI PR #1736
Overview
The modifications presented in this pull request effectively remove support for Python 3.13 and standardize compatibility to Python versions 3.10 to 3.12. The changes span various configuration files, dependency adjustments, and necessary test modifications to ensure our CI/CD processes align with the new standards.
Key Observations and Recommendations
1. Python Version Compatibility
The requirement in pyproject.toml has been updated as follows:
# Current
requires-python = ">=3.10,<=3.12"

# Suggested improvement
requires-python = ">=3.10,<3.13"

This minor change provides explicit boundaries for version support, helping to clarify compatibility for users.
2. Configuration Files
You're doing a great job in streamlining our configuration files alongside this PR. The modifications in files like .github/ISSUE_TEMPLATE/bug_report.yml and README.md offer clear communication regarding our support directions.
3. Test Enhancements
Test changes have correctly accounted for the removal of Python 3.13 compatibility. However, readability can be improved by utilizing textwrap.dedent() for multiline strings, as shown below:
# Current
@patch("builtins.open", ...)
read_data="""
[project]
name = "test_project"
version = "0.1.0"
requires-python = ">=3.10,<=3.12"
"""

# Suggested enhancement
@patch("builtins.open", ...)
read_data=textwrap.dedent("""
    [project]
    name = "test_project"
    version = "0.1.0"
    requires-python = ">=3.10,<3.13"
""")

4. Version Validation
Consider implementing a Python version validation function to provide an upfront check ensuring users adhere to our version requirements:
def validate_python_version():
    import sys
    if sys.version_info >= (3, 13):
        raise RuntimeError("Python 3.13 is not supported. Use Python >=3.10,<3.13")

5. Documentation Updates
Enhancing our documentation around version compatibility is crucial. A potential addition to installation.mdx:
## Version Compatibility
CrewAI requires Python 3.10-3.12 for optimal performance and security.
### Version Check
```bash
python3 --version  # Ensure version is between 3.10 and 3.12

This addition would assist users in ensuring they meet the compatibility requirements before installation.
Conclusion
The changes you’ve made effectively manage the transition from Python 3.13 while cleaning up our codebase. Although these adjustments may add short-term burden for users currently on Python 3.13, the long-term benefits in code maintainability, clarity, and enhanced documentation substantially outweigh initial inconveniences.
By implementing the recommendations above, we can further bolster our code’s robustness and user guidance. Great work on pushing this forward!

awesome thanks @bhancockio
this is my first ever pr : )


How to printf a float number?

When I use " printf("%f\r\n", 1/3); " , I got only " f "?
How can I print a float number?

ch32v003fun provides a stripped-down version of printf() which doesn't support floating point numbers. Use of floating point on the ch32v003 MCU is discouraged due to the heavy code size requirements. If you require floating point output you will need to provide that yourself.

Okay, understood and thanks!

If, having heeded @emeb's sound advice, you still really need float support, then maybe consider using a self-contained "tiny" printf() implementation, ideally one that doesn't use heap which should generally be avoided in a resource constrained embedded environment. There are several such implementations available. For example;

https://github.com/mpaland/printf
https://github.com/eyalroz/printf
https://github.com/cjlano/tinyprintf
https://www.menie.org/georges/embedded/small_printf_source_code.html

WCH's properiatary toolchain does provide a printfloat library that can do that, and that works just fine: https://github.com/Community-PIO-CH32V/platform-ch32v/issues/41#issuecomment-1792337029
However, that is so big in code size anyways that you should never ever use it. Avoid floating pointing at all costs, or do cheaper fixed-point arithmetic instead


chore: Bump<EMAIL_ADDRESS>(#84089)

Fixes https://github.com/microsoft/vscode/issues/84343

Oh nevermind, I was not aware 6.1.4 was already released a while ago...


Fix KeyError with removing SSH connection

Closes #15
(well, I think it does.  can't test since my Pi + FrameOS has crashed and not recovered)

Closing in favor of https://github.com/FrameOS/frameos/pull/17

Haha, sorry for that :)


Show DFI in Token and coins separately

What would you like to be added:
Please list DFI in the wallet as Tokens and Coins separately, and perhaps make a button to change Token into coins.
Also it would be great if amount of coins cannot get below 1 DFI (turn this feature on and off in the configuration if you think somebody should be able to send all its DFI)
Why is this needed:
There are soo many support questions from people who do not know that there are different types of DFI. They get Error messages and cannot use the app without support. This is very bad for user experience.
And using the command line is good for tech people but not for the normal users.
It generates a lot of frustration if the app doesn't work as a normal user would expect it. Even if it is from the blockchain view business as it should be.
So I think these little enhancements will increase the usability for normal users by a lot and help the general acceptance for DFI.

Hi @wmbst , thanks for your feedback, the team is aware and working on improving what you mentioned.

Hi @wmbst, we have added a split of display for UTXO and Token in >v2.3.5. Please take a look. Thanks!


Live preview isn't working

It's not expected to work at this stage but it's for the record.


the video window may close unexpectedly in the middle of a sentence

Hello.
Sometimes the video window may close unexpectedly in the middle of a sentence when the assistant is saying something. If you start talking yourself, so that the assistant begins to regenerate the text and voice it, the window will appear again.
I found out that this is a bug in silly_extras.bat. It wants to open a wav file, but can't. Any ideas why?
cv2: missing video frame
Exception in thread Thread-4 (wav2lip_server_play_init): Traceback (most recent call last):
	File "D:\miniconda\envs\extras\Lib\threading.py", line 1045, in _bootstrap_inner
		self.run()
	File "D:\miniconda\envs\extras\Lib\threading.py", line 982, in run
		self._target(*self._args, *self._kwargs)
	File "D:\talk-llama-fast\xtts\SillyTavern-Extras\modules/wav2lip\server_wav2lip.py", line 342, in wav2lip_server_play_init
	next_video_chunk_global = play_video_with_audio(video_file_path, audio_file_path, True, next_video_chunk_global, rand_caption)

	File "D:\talk-llama-fast\xtts\SillyTavern-Extras\modules/wav2lip\server_wav2lip.py", line 475, in play_video_with_audio
	wf = wave.open(audio_file, 'rb') # Open the next audio file

	File "D:\miniconda\envs\extras\Lib\wave.py", line 631, in open
		return Wave_read(f)
	File "D:\miniconda\envs\extras\Lib\wave.py", line 283, in __init__
		self.initfp(f)
	File "D:\miniconda\envs\extras\lib\wave.py", line 274, in initfp
	raise Error('fmt chunk and/or data chunk missing') wave.Error: fmt chunk and/or data chunk missing<PHONE_NUMBER>.074945 in wav2lip gen server chunk:23_2
in wav2lip_server_generate: is busy: 0, face_detect_running: 0, chunk: 23, chunk needed: 23, reply: 2

something wrong with wav format, but i don't know what causes it. It just happens sometimes


/conf.d is not used for logging

https://github.com/DataDog/datadog-agent/tree/master/Dockerfiles/agent is saying to mount a volume to /conf.d/ but when trying to start the datadog/agent:latest docker image, I see the following output:
[ AGENT ] 2018-02-22 17:06:35 UTC | ERROR | (start.go:220 in StartAgent) | Could not start logs-agent: could not find any valid logs configuration file in /etc/datadog-agent/conf.d

Hi @mattwilliamson ,
Indeed, *.yaml files are copied, see https://github.com/DataDog/datadog-agent/blob/master/Dockerfiles/agent/entrypoint/89-copy-customfiles.sh
Could you please post the yaml file you exposed in the container, so @NBParis can have a look at its syntax?

Getting the same error.
docker-compose
  datadog:
    container_name: dd-agent
    image: datadog/agent:latest
    links:
     - kafka
     - zookeeper
     - influxdb
     - grafana
    environment:
     - DD_HOSTNAME=ddhostname
     - DD_API_KEY=<<secret key>>
     - NON_LOCAL_TRAFFIC=true
     - DD_APM_ENABLED=true
     - DD_LOGS_ENABLED=true
     - DD_PROCESS_AGENT_ENABLED=true
    volumes:
     - /var/run/docker.sock:/var/run/docker.sock
     - /proc/:/host/proc/:ro
     - /sys/fs/cgroup/:/host/sys/fs/cgroup:ro
     - /opt/datadog-agent/run:/opt/datadog-agent/run:rw
     - /opt/datadog-agent/conf.d:/conf.d:ro

/opt/datadog-agent/conf.d/docker.d/conf.yaml
logs:
    - type: docker
      service: docker

Anything I'm missing?

Hello @AGutan,
The configuration seems indeed to be correct. We will double check with the exact same config to see if we can replicate this issue.
Meanwhile, could you try to add this environment variable to your configuration:
- DD_LOGS_CONFIG_CONTAINER_COLLECT_ALL=true

This should have the exact same behaviour as your configuration.
Thanks

@NBParis indeed DD_LOGS_CONFIG_CONTAINER_COLLECT_ALL=true did the trick. Might be a good idea to document this, since I haven't seen the parameter in the docs.

Indeed the in-app documentation is not totally up to date, but you should be able to find it in the official documentation: https://docs.datadoghq.com/logs/docker/#option-2-container-installation
I'll make sure we update the in-app documentation as well.
Let us know if we can close this issue (we will anyway keep looking at the original issue).

@NBParis yes, you can close the issue. Thanks for quick assistance!


Investigate obtaining cosmic.vcf for hg38

Cosmic is useful for MuTect calling:
"In contrast, COSMIC is a more highly validated resource, so it is used essentially as a whitelist to "rescue" candidate mutations that would otherwise be rejected for being in the panel of normals and/or dbSNP. We expect that anything that is really somatic that is flagged as such in dbSNP will also be in COSMIC, so we can rely on COSMIC to rescue those sites."

Not very hard to do. Make an account on sanger and download from their website. Alternatively, you can use this to get it
https://github.com/arkal/docker_scripts/blob/master/prepare.py#L788-L840
you just need to provide a username and password form Sanger

@arkal Thanks!


[cherry-pick 2.3] [Enhancement] speed up trash sweep

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Currently, when the disk water level is high, the sweeper thread removes the trash directory files to reclaim disk space, but the sweeper thread sleeps too long, the default value ranges from 3 minutes to 1 hour, which could cause data GC to be delayed when the critical water level is reached.
This PR ensures that the sweeper thread is awakened in time when the critical disk water level is reached. At the same time the original behavior of the sweeper thread is preserved.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.23 out of 24 committers have signed the CLA.:white_check_mark: liuyehcf:white_check_mark: xiaoyong-z:white_check_mark: padmejin:white_check_mark: waittttting:white_check_mark: Astralidea:white_check_mark: EsoragotoSpirit:white_check_mark: amber-create:white_check_mark: hellolilyliuyi:white_check_mark: ZiheLiu:white_check_mark: rickif:white_check_mark: Seaven:white_check_mark: femiiii:white_check_mark: gengjun-git:white_check_mark: trueeyu:white_check_mark: evelynzhaojie:white_check_mark: stdpain:white_check_mark: zombee0:white_check_mark: HangyuanLiu:white_check_mark: DorianZheng:white_check_mark: Smith-Cruise:white_check_mark: zaorangyang:white_check_mark: Crystal-LiuJing:white_check_mark: dirtysalt:x: mergify[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


kustomize overlays should be shown with their path

this is the https://github.com/redhat-developer/gitops-repo-example repo - Monokle Cloud shows

whereas desktop shows

This is a problem that has been identified in more places, if the same resource names are reused in different folders, you get different copies of the same resource, and there is no information to distingish them until you click on them (to show the file), making it hard to show

thanks @chargio - this has actually been discussed almost since the beginning - see https://github.com/kubeshop/monokle/issues/424 for some possible solutions - cc @julioramosest

I would:

Opt for having the file path next to the name label when hovering (not a tooltip, since that's slower)
since filepath can be long and first parent folders being the same, I would show only the last 2 folders, this way: .../folder/folder
also truncating full length according to pane width, as we do with other file names
color for the file path: grey-7

weirdly enough this works like cloud for some repos - for example


Add fog, volumetric fog and particle effect limit options

Contains
Yet another partial fix for #1144
How to test
Additions can be seen in video settings menu.
For testing, particle effect limit: picking block from ground generates multiple particles during animation.
For testing: fog: blocks that are far from the player will be foggy or not according to setting. For future, there can be extra options for fog like density, colour, equation (linear, exp or etc.).
For volumetric fog, I couldn't find a proper test case for trying out volumetric fog implementation in "combine" shader. But, my current addition can turn it on or off. It would be great if someone can provide instructions for me to test recent additions.

Can one of the admins please verify this patch?

add to whitelist
Hey @tdgunes and thanks :-)
On volumetric fog I wonder if it is even enabled at present? I remembered an old video showcasing it nearly three years ago by @begla where he would enable / increase volumetric fog via sliders in TeraEd (an alternatively game facade launching with some extra graphics config)
https://www.youtube.com/watch?v=cXiX5ZJT_-k
I just realized that video is unlisted rather than published, so very few people have ever seen it. Maybe it was implemented but never fully finished up for actual use? That was back around the end of our 3d wizardry heavy period.
@emanuele3d might have a related clue or two

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/462/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/465/
Hooray Jenkins reported success with all tests good!

Hehe, probably best that volumetric fog is off by default :D

Otherwise yeah the switches work and that was the plan :-)


cmd/list: support listing formulae installed on request or automatically

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Sample usage:
$ brew ls --manual
gcc
llvm
[...]
$ brew ls --auto
grpc
protobuf
[...]
$ brew ls --manual --auto
gcc: manual
grpc: auto
llvm: manual
protobuf: auto
[...]

Resolves #17117.

On a related note, this logic is extracted from a local command brew mark that I wrote some time ago but haven't had a chance to consider contributing yet. It marks a formula as installed on request or automatically, and is analogous to apt-mark, dnf mark, port {,un}setrequested, etc.
The reason I wrote it is mainly because brew upgrade and brew reinstall can change a formula's status from automatically installed to installed on request, which is not always desired. Sometimes I do brew reinstall to work around the tab bug (#16032) so that I can safely use brew autoremove to prune the dependencies that are actually not needed anymore. But as an unintended consequence (to me), the formula itself becomes installed on request, so it can no longer be autoremoved later. brew mark --auto <formula> became a solution for me.
After a quick search I do see previous request for such functionality (e.g., #10754), but that was not implemented in the end. I'd like to raise this to see if there is still interest in this; if so, I can prepare a PR for that.