Compatibility with Expo Go

Hi,
I have been exploring the possibility of utilizing your library in conjunction with Expo Go, but it seems that the current version does not support this integration. I was wondering if you could kindly shed some light on this matter and explain the reasons behind this limitation.
Thank you very much for your time and attention. I greatly appreciate your hard work and dedication to creating such a valuable library.

Hello @joaovitorleffa,
I am sorry for the waiting, there weren't a lot of feedback so I didn't check the issues daily.
I haven't had experience working with Expo Go, and I am not sure whether it is possible to support these features in it.
Could you tell me how this is crucial for you? There are a lot of work in plan, and supporting Expo wasn't in them.
If it's possible I would recomend to use ReactNative cli, but if you have good resasons for using Expo let me know and I will try to help

Hello @ArturKalach, thanks for the feedback,
Currently, I'm working on a React Native CLI project, but there's a possibility that the project might migrate to Expo Bare Workflow in the future.
I also have no experience with Expo. But after creating this issue, I conducted some research to find possible solutions. It turns out that Expo doesn't support certain libraries. However, there is a workaround using the Expo Dev Client. By following these steps, I was able to successfully use the react-native-a11y library with Expo Go:

Start by building your Expo project using either expo run:ios or expo run:android.
Once the app is built, launch it with the Expo Dev Client by running expo start --dev-client. This will seamlessly integrate your library within the Expo Go environment.

I believe this solution should be sufficient to enable us to use the library with Expo Go.


fast-cli: 1.0.0 -> 3.0.1

Motivation for this change
Unpin fast-cli from version 1.  This adds support for the --upload flag so you can test upload speed in addition to download speed.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This PR is based on a slightly old version of master because I currently can't run the node-packages/generate.sh script, which encounters this error.  Therefore, I also haven't been able to run nixpkgs-review, which aborts when it hits a merge conflict.

Done, thanks @SuperSandro2000.


improve description of Extensions

I separated each Extensions into category and added description.

LGTM except two small comments.

Thanks for review! Fixed accordingly.

Thanks for the quick fixes. LGTM!


Implement #[derive(Query)] for enum types

#[derive(Query)] can now be applied to enum types
An Enum query will yield the first variant that satisfies an Entity (even if there are other variants which might have satisfied it)
Supports tuple enums, struct enums and will ignore any unit enum field.
Closes https://github.com/Ralith/hecs/issues/376

Don't forget to update the rustdoc for derive_query in macros/src/lib.rs, also.


Create 20220720-per-certificate-owner-ref.md

🌟 This design document was written by Maël Valais on 20 July 2022 in order to facilitate @RomanenkoDenys's feature request presented in #5158.

NONE

Irbe suggested that I follow the formatting mandated in the design template. I will address Josh's comments and reformat the document to match the expected template.

/test pull-cert-manager-master-make-test

I am unsure how to proceed with this design. @RomanenkoDenys would like to know what to do now.
The design has some missing sections but I don't think that's a stopper. This the discussion is settling down, I propose that we proceed with implementing that design.
Please let me know @JoshVanL if you have more comments.

Although this solution isn't perfect and might address a very limited set of users, we agreed on the fact that it makes sense to make --enable-certificate-owner-ref configurable at the Certificate level.
@wallrj Let me know if we still need work before this document gets LGTM'ed.

/remove-lifecycle rotten
/reopen

@inteon it looks like you deleted the branch...
@maelvls is there a plan for this?


oraclejdk8, buildpath for archive not recognized

I want to use the following package:
  environment.systemPackages = with pkgs; [
    oraclejdk8
  ];

Using just
nixos-rebuild switch

will yield the following error:
[root@laptop:/]# nixos-rebuild switch
building Nix...
building the system configuration...
these derivations will be built:
  /nix/store/62kwfp9gixsqaw66j1vqik4s89qskwkb-jdk-8u40-linux-x64.tar.gz.drv
  /nix/store/hkv5iv2n69b2m15hl1lq4dah6s4bakkb-oraclejdk-8u40.drv
  /nix/store/9cqqh4jfcraj40zs3hyw3gv76j9ffv87-system-path.drv
  /nix/store/9z7m3p71dmri6v8mwz9sdkkha9n3727f-system-crontab.drv
  /nix/store/gj5baymzqa8fs0c0gfai2l3w28x6aqyf-unit-script.drv
  /nix/store/7265v58pr44267glvy1kx7qjixkgk8gl-unit-cron.service.drv
  /nix/store/mkdvnw3z907zi2s250xxqbwv5qq56dyp-dbus-conf.drv
  /nix/store/il8kdnmjv17z44chkpn9fn4lb1jfbd7g-unit-polkit.service.drv
  /nix/store/yh14iccy5nfq06i5dqgyyd06i73a4rq9-system-units.drv
  /nix/store/95md0321g5dmn44hh9zki4j5xvakiv3m-etc.drv
  /nix/store/hc6ni58zss9hbnms65l3d7sd8qzfizif-nixos-15.05pre60707.4e51a46.drv
these paths will be fetched (10.57 MiB download, 34.75 MiB unpacked):
  /nix/store/49d293g968wcqnx7vfr2hhsw85lbklgz-libwebp-0.4.3
  /nix/store/5kvxbzv69x8zfyyr6rpv67cm7hh9vnwn-x265-1.5
  /nix/store/b745r22kafq8336h93v9abi93fqm02xh-libav-0.8.17
  /nix/store/w0mcblf6aywbyx7rxzqz0wd610lnfdmb-ffmpeg-2.5.4
fetching path ‘/nix/store/b745r22kafq8336h93v9abi93fqm02xh-libav-0.8.17’...
fetching path ‘/nix/store/49d293g968wcqnx7vfr2hhsw85lbklgz-libwebp-0.4.3’...
fetching path ‘/nix/store/5kvxbzv69x8zfyyr6rpv67cm7hh9vnwn-x265-1.5’...
building path(s) ‘/nix/store/sif69y5fk3ach38f3yi8yqyn15vw69id-jdk-8u40-linux-x64.tar.gz’

***
Unfortunately, we may not download file jdk-8u40-linux-x64.tar.gz automatically.
Please, go to http://www.oracle.com/technetwork/java/javase/downloads/jdk8-downloads-2133151.html, download it yourself, and add it to the Nix store
using either
  nix-store --add-fixed sha256 jdk-8u40-linux-x64.tar.gz
or
  nix-prefetch-url file://path/to/jdk-8u40-linux-x64.tar.gz

***

builder for ‘/nix/store/62kwfp9gixsqaw66j1vqik4s89qskwkb-jdk-8u40-linux-x64.tar.gz.drv’ failed to produce output path ‘/nix/store/sif69y5fk3ach38f3yi8yqyn15vw69id-jdk-8u40-linux-x64.tar.gz’
killing process 2547
killing process 2549
killing process 2548
cannot build derivation ‘/nix/store/hkv5iv2n69b2m15hl1lq4dah6s4bakkb-oraclejdk-8u40.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/9cqqh4jfcraj40zs3hyw3gv76j9ffv87-system-path.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/hc6ni58zss9hbnms65l3d7sd8qzfizif-nixos-15.05pre60707.4e51a46.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/hc6ni58zss9hbnms65l3d7sd8qzfizif-nixos-15.05pre60707.4e51a46.drv’ failed

So I went on the oracle site to download it. The sha256 fits:
[root@laptop:/etc/nixos/store]# sha256sum jdk-8u40-linux-x64.tar.gz
da1ad819ce7b7ec528264f831d88afaa5db34b7955e45422a7e380b1ead6b04d  jdk-8u40-linux-x64.tar.gz

I used both methodes described:
[root@laptop:/etc/nixos/store]# nix-store --add-fixed sha256 jdk-8u40-linux-x64.tar.gz
/nix/store/2q6cljnj4xhgshjihas8p9r1ksndvnmg-jdk-8u40-linux-x64.tar.gz

nix-prefetch-url file:///etc/nixos/store/jdk-8u40-linux-x64.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  165M  100  165M    0     0  2806M      0 --:--:-- --:--:-- --:--:-- 2806M
path is ‘/nix/store/2q6cljnj4xhgshjihas8p9r1ksndvnmg-jdk-8u40-linux-x64.tar.gz’
0kdhsvmb3073lwi59r2mg55v6pdamy41v0sg4qlcazkvrqcxh6ns

But when I try to rebuild it fails. Downloading it via wget throws the same error.
[root@laptop:/etc/nixos/store]#  wget --no-check-certificate --no-cookies --header "Cookie: oraclelicense=accept-securebackup-cookie" http://download.oracle.com/otn-pub/java/jdk/8u40-b26/jdk-8u40-linux-x64.tar.gz

I noticed that the path nix is looking for does not exist:
building path(s) ‘/nix/store/sif69y5fk3ach38f3yi8yqyn15vw69id-jdk-8u40-linux-x64.tar.gz’

should be:
/nix/store/2q6cljnj4xhgshjihas8p9r1ksndvnmg-jdk-8u40-linux-x64.tar.gz

So it is looking for wrong build paths, what can I do?
More information:
I am using the unstable channel and have a Lenovo y510p.

Same problem here.
I'd suggest installing proprietary software into user-local environments though.

Anyway, most certainly I'll look into this issue next weekend.

Builds and works. I call we close this issue.

That was fast :heart: :blue_heart: :green_heart:

@manpages 🙈


Always return a list from GetCommitsFromIDs

GetCommitsFromIDs is only used in one place: LoadPushCommits where
it expects that c.Commits is not nil.
This potentially nil set causes a NPE in in #12953
Fix #12953
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Codecov Report

Merging #12981 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #12981   +/-   ##
=======================================
  Coverage   42.47%   42.48%
=======================================
  Files         671      671
  Lines       73627    73625    -2
=======================================
+ Hits        31272    31276    +4
+ Misses      37291    37287    -4
+ Partials     5064     5062    -2

Impacted Files
Coverage Δ

modules/git/repo_commit.go
50.91% <ø> (+0.37%)
:arrow_up:

modules/charset/charset.go
68.53% <0.00%> (-4.50%)
:arrow_down:

modules/log/event.go
58.49% <0.00%> (-0.95%)
:arrow_down:

services/pull/pull.go
40.78% <0.00%> (+0.49%)
:arrow_up:

modules/queue/workerpool.go
60.00% <0.00%> (+1.22%)
:arrow_up:

modules/git/blame.go
77.19% <0.00%> (+1.75%)
:arrow_up:

modules/process/manager.go
75.00% <0.00%> (+2.50%)
:arrow_up:

models/unit.go
49.31% <0.00%> (+2.73%)
:arrow_up:

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 1bcf1ad...91bc4c8. Read the comment docs.


[IoT Hub] Increase SDK and API version

Related command
az iot hub
az iot hub create
az iot hub route test
az iot dps linked-hub create
Description
Increase the API version to 2022-04-30-preview and pin the min SDK to azure-mgmt-iothub==2.3.0. This will enable customers to modify the control plane of hubs that have features from the 2022-04-30-preview API, specifically the Cosmos DB endpoints.
The Cosmos DB Endpoint feature will be released in the azure iot extension.
There is also some small fixes:

the fileupload_storage_container_uri parameter is no longer used and is removed in this pr. This will remove the warning message below that occurs during hub creation.

the app_properties and system_properties for hub routes will now be converted to a dictionary (and thus make the routes test work)
the dps linked hub will now have an error message about location - so that if a hub is provided in a different subscription, then the user will be prompted to give the location. This will fix the customer confusion as seen in this issue: https://github.com/Azure/azure-cli/issues/22257

Testing Guide

Regenerated all affected iot commands. Also added a test to route test (that would have failed in the older version)
History Notes

[IoT Hub] Increase min SDK and API version
[IoT Hub] Remove warning during Hub creation and fix routes test
[IoT Dps] Improve error handling for linked hubs

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

IoT Hub

[IoT Hub] Increase min SDK and API version
[IoT Hub] Remove warning during Hub creation and fix routes test
[IoT Dps] Improve error handling for linked hubs

We usually only write the changes that can be perceived by users in the history, so I personally think the first and second points can be deleted.
Could you describe the third point in more detail?

[IoT Hub] Increase min SDK and API version
[IoT Hub] Remove warning during Hub creation and fix routes test
[IoT Dps] Improve error handling for linked hubs

We usually only write the changes that can be perceived by users and affect users' business in the history notes, so I personally think the first and second points can be deleted. Could you describe the third point in more detail?

the dps linked hub will now have an error message about location - so that if a hub is provided in a different subscription, then the user will be prompted to give the location. This will fix the customer confusion as seen in this issue: https://github.com/Azure/azure-cli/issues/22257

The second point will affect users too

In fact, the warning log has no real impact on the user's business, so we usually do not write the changes of warning log into the history notes.

[IoT] Fix https://github.com/Azure/azure-cli/issues/22257: az iot dps linked-hub create: Improve error handling for linked hubs

This is my updated history note, please see if it meets your expectations


[KYLIN-4971] Add new measure bitmap_map for count distinct at fronten…

…d. (#222)
Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Add bitmap_map measure.

only support segmentBuilder and auto add in Advanced Dict UI

Pull Request Test Coverage Report for Build 7070

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.001%) to 28.094%

Files with Coverage Reduction
New Missed Lines
%

core-dictionary/src/main/java/org/apache/kylin/dict/lookup/cache/RocksDBLookupTable.java
1
81.08%

server-base/src/main/java/org/apache/kylin/rest/util/QueryRequestLimits.java
1
47.62%

Totals

Change from base Build 7067:
0.001%

Covered Lines:
26715

Relevant Lines:
95091

💛  - Coveralls

@zhangayqian thxs, already fix.

@Ted-Jiang Nice！Can you merge this two commit to one commit?

@Ted-Jiang Nice！Can you merge this two commit to one commit?

sure,fix

LGTM


Rebrand to EnsonoBuild

📲 What
Rebranded the Independent Runner module to EnsonoBuild
Updated the style of the documentation.
🤔 Why
Amido has been part of Ensono for a long time now and so components should be branded as such.
🛠 How
Renamed AmidoBuild to EnsonoBuild.
Updated the references within the module to state EnsonoBuild
👀 Evidence
Artifacts - https://dev.azure.com/amido-dev/Amido-Stacks/_build/results?buildId=32202&view=artifacts&pathAsName=false&type=publishedArtifacts
🕵 How to test
Notes for QA

Noice, ty


Hosted Doxygen Documentation out of date?

According to your start page:
"The Doxygen reference docs are hosted online and generated as part of the build."
Leading to https://openthread.github.io/openthread/
But looking into files for e.g. platform abstraction shows many functions not documented on this website.
Are these functions undocumented or is something wrong with the automatic generation?

Unfortunately, while the docs are generated as part the build process, the docs are not automatically published as part of the continuous integration process.  We're working on a way to have the docs published automatically.  If you want the docs for a given commit ID, you can perform the standard bootstrap, configure, make process to generate the docs yourself.  You'll need doxygen for the basic doc support and graphviz if you want the call graphs as well.

BTW, I submitted #818 to update the hosted docs to ddd0704.

What about looking into the doc publishing mentioned in #486?

Using master branch to host autogenerated docs does not seem ideal.  It would mean nearly every commit would require another commit to update the docs in master branch.  Also, the full set of generated docs is nearing 350 MB, which I think is a bit hefty to include in master.

Good points! Didn't realize the docs feature had to be in master. #goodtoknow

Resolved by #818.


can you add helm list action

Can helm list action be added to verify the installation? or any workaround available to achieve that?

Hey @sushovan23
We have an update in the PR https://github.com/bitovi/github-actions-deploy-eks-helm/pull/28 to this action in order to support the helm list command.
You can test it out by changing your workflow yaml line to use a specific branch like this
    - name: Deploy Helm
      uses: bitovi/github-actions-deploy-eks-helm@27-add-helm-list-action

And set it up like the example in the README file in that branch. You can use the name attribute to filter your results.
Let us know how it goes! Thanks!

@LeoDiazL Thanks for the quick turnaround.
I'm facing following issue:
      - name: List Helm Release
        uses: bitovi/github-actions-deploy-eks-helm@27-add-helm-list-action
        with:
          aws-region: eu-west-1
          action: list
          cluster-name: Test
          namespace: test-ns
          name: app-test
          atomic: true
          timeout: 60s

/usr/local/bin/deploy: line 160: [: missing `]'
Error:  HELM_ACTION specified doesn't exist in this context. Please use 'install','uninstall' or 'list'
It's weird since error message says its there but doesn't look like it read it

Hi @sushovan23 there was a typo in an if statement.
If you could try it again, that would be great. Thanks for getting involved!


Manually Trigger Inline Completion

please complete the following information:

OS version: [e.g. Win 10, Mac Catalina]
linux
Editor: [e.g. vscode, intelij, sublime]
vscode & jetbrains suite
Editor version:
lastes for both
Programming language: [e.g. js, ts, c++]
all
TabNine extension version:
latest
Engine version (could be located within Tabnine Hub):
n/a

Issue Details:
Pretty often when using tabnine with intelligent editors i find the editor's internal behavior fighting with tabnine. Auto-indent is by far the biggest issue.  Often times i find myself hitting shift+tab then tab to trigger the completions. Also in certain cases the completion runs out but there is more code to be written on that line.
If we had a key combo that effectively allowed us to manually request an inline completion the tool would perform much better in many of the cases I am running into
gz#22255(related to Zendesk ticket #22255)

Hey @sam-ulrich1
There is a key binding named trigger.inline.suggestion in VSCode which does that.
Let me check if JetBrains has a similar shortcut.

I've tried it before posting, so let me know if it work for you.
Regarding JetBrains, there isn't a shortcut at the moment but we are working on adding it, and I'll update here once we have it.
In addition, It would like to see some examples of TabNine collisions with your IDEs, as this seems to be the root issue and usually we can address such issues, if you can, please send short videos of the issue to<EMAIL_ADDRESS>

Ya, I'll definitely send it over. Next time I run into it I'll take a few videos. Shouldn't be long


What is the status of opacity:0.99 fix/workaround

Bug description:
We want to remove opacity: 0.99 but not sure if this is fixed and don't want to hit users with bug if not fixed.
Once we discovered that on IOS webView make problem with other screens elements (make them white) and once we set opacity: 0.99 it worked.
Also this opacity fixed crashes on some android devices.
Now we have a problem on android 5 webView is there clickable HTML elements inside but it is blank/white.
If I remove opacity on android 5 it work.
To Reproduce:
Set webView style opacity: 0.99 and on android 5 it's blank.
Expected behavior:
WebView is visible on all phones without setting opacity.
Environment:

OS: Android
OS version: 5
react-native version: 0.60.5
react-native-webview version: 7.5.2

Has the same issue. In Android 5, opacity 0.99 will make webview blank

has this been resolved? or did it just get closed because of its staleness?

still searching for solutions


nginx: Add option for enabling pagespeed, option for additional modules

Motivation for this change
Make it easier to configure nginx servers using modules beyond those bundled by default, especially to automate the boilerplate required to use pagespeed.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

cc @3noch @adisbladis

@grahamcofborg test nginx-pagespeed

Failure on x86_64-linux (full log)
Partial log (click to expand)
error: attribute ‘nginx-pagespeed’ in selection path ‘tests.nginx-pagespeed’ not found

Failure on aarch64-linux (full log)
Partial log (click to expand)
error: attribute 'nginx-pagespeed' in selection path 'tests.nginx-pagespeed' not found

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘psol-<IP_ADDRESS>’ in /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/development/libraries/psol/generic.nix:9 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Partial log (click to expand)
make[1]: Leaving directory '/build/nginx-1.12.2'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/j53wj3v8vjh6fpkd1qwbh1s3llhcfm0c-nginx-1.12.2
shrinking /nix/store/j53wj3v8vjh6fpkd1qwbh1s3llhcfm0c-nginx-1.12.2/bin/nginx
strip is /nix/store/xmpjypwjmp2qi1chs5kr0hacnh161ls4-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/j53wj3v8vjh6fpkd1qwbh1s3llhcfm0c-nginx-1.12.2/bin
patching script interpreter paths in /nix/store/j53wj3v8vjh6fpkd1qwbh1s3llhcfm0c-nginx-1.12.2
checking for references to /build in /nix/store/j53wj3v8vjh6fpkd1qwbh1s3llhcfm0c-nginx-1.12.2...
/nix/store/j53wj3v8vjh6fpkd1qwbh1s3llhcfm0c-nginx-1.12.2
/nix/store/dilwjma3i6aicbi5gd6a1p3fzk4m9gs1-psol-<IP_ADDRESS>

Success on x86_64-linux (full log)
Partial log (click to expand)
test -d '/nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2/logs' \
	|| mkdir -p '/nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2/logs'
make[1]: Leaving directory '/tmp/nix-build-nginx-1.12.2.drv-0/nginx-1.12.2'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2
shrinking /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2/bin/nginx
strip is /nix/store/5qj61lcvzlap87rf6blvf8p577d482bv-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2/bin
patching script interpreter paths in /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2
checking for references to /tmp/nix-build-nginx-1.12.2.drv-0 in /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2...

Eep, apologies about that.  Dishonor on me and my family. :(.
@GrahamcOfBorg test nginx-pagespeed

Failure on aarch64-linux (full log)
Partial log (click to expand)
cannot build derivation '/nix/store/ifxv75bg73z8czi3h9bvjqm230hla22d-unit-nginx.service.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/113dyxahq52l7l1693nrs4jimzdwkcks-system-units.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/77cf4i1lx4g40vvrqdywnrz3gn2niq7c-etc.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/l69l8jaqbjzmi13gk58md60dv0mna4b8-nixos-system-webserver-18.03.git.7f8181d.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/rwspyna68883l8qc3qq8lfjb81mw5i4x-closure-info.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/mbzycn5q7ndn0c9dd6an2dayiaqcxvx1-run-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/0dwvvk9bnsbzriakgf4plvgy2zmnzw6z-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/k0rh09r0a87mxqlfxpwq9k95hnha0spy-nixos-test-driver-nginx-pagespeed.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/py97mp16a4irji509i37yc5mik52c8zq-vm-test-run-nginx-pagespeed.drv': 1 dependencies couldn't be built
error: build of '/nix/store/py97mp16a4irji509i37yc5mik52c8zq-vm-test-run-nginx-pagespeed.drv' failed

Success on x86_64-linux (full log)
Partial log (click to expand)
webserver# [    8.087609] systemd-shutdown[1]: All filesystems unmounted.
webserver# [    8.090730] ACPI: Preparing to enter system sleep state S5
webserver# [    8.091163] reboot: Power down
vde_switch: EOF data port: Interrupted system call
collecting coverage data
syncing
test script finished in 9.12s
vde_switch: EOF on stdin, cleaning up and exiting
cleaning up
/nix/store/2kg2nnbp6cycck2cghbwzcbrqrzcrmqj-vm-test-run-nginx-pagespeed

Haha, neat! FWIW aarch64 is because of builder's shell.

Failure on aarch64-linux (full log)
Partial log (click to expand)
checking for limits.h ... found
checking for sys/filio.h ... not found
checking for sys/param.h ... found
checking for sys/mount.h ... found
checking for sys/statvfs.h ... found
checking for crypt.h ... found
checking for Linux specific features
./configure: auto/os/linux: line 21: syntax error: support for $((arith)) is disabled
builder for '/nix/store/79mq11kgvav5msl27id1h6xn7zj8h42f-nginx-1.12.2.drv' failed with exit code 2
error: build of '/nix/store/79mq11kgvav5msl27id1h6xn7zj8h42f-nginx-1.12.2.drv' failed

Success on x86_64-darwin (full log)
Partial log (click to expand)
	|| mkdir -p '/nix/store/mddwj2kacypkxxkl0gvk2wchymipx4r4-nginx-1.12.2/logs'
test -d '/nix/store/mddwj2kacypkxxkl0gvk2wchymipx4r4-nginx-1.12.2/html' \
	|| cp -R html '/nix/store/mddwj2kacypkxxkl0gvk2wchymipx4r4-nginx-1.12.2'
test -d '/nix/store/mddwj2kacypkxxkl0gvk2wchymipx4r4-nginx-1.12.2/logs' \
	|| mkdir -p '/nix/store/mddwj2kacypkxxkl0gvk2wchymipx4r4-nginx-1.12.2/logs'
make[1]: Leaving directory '/private/tmp/nix-build-nginx-1.12.2.drv-0/nginx-1.12.2'
post-installation fixup
strip is /nix/store/5a88zk3jgimdmzg8rfhvm93kxib3njf9-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/mddwj2kacypkxxkl0gvk2wchymipx4r4-nginx-1.12.2/bin
patching script interpreter paths in /nix/store/mddwj2kacypkxxkl0gvk2wchymipx4r4-nginx-1.12.2

Success on x86_64-linux (full log)
Partial log (click to expand)
test -d '/nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2/logs' \
	|| mkdir -p '/nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2/logs'
make[1]: Leaving directory '/tmp/nix-build-nginx-1.12.2.drv-0/nginx-1.12.2'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2
shrinking /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2/bin/nginx
strip is /nix/store/5qj61lcvzlap87rf6blvf8p577d482bv-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2/bin
patching script interpreter paths in /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2
checking for references to /tmp/nix-build-nginx-1.12.2.drv-0 in /nix/store/8s9laic3cc3nivq9izqll31q8p2f6raz-nginx-1.12.2...

Needs minor touch-up for package vs cfg.package after preStart improvement, otherwise config is evaluated with wrong nginx (that doesn't have extra modules such as pagespeed built in), I have a branch with this fix.

Will just use this locally then, it's a bit much to go ahead with without more general interest.


✨ ✨ New layout ✨ ✨

✨ ✨ New layout ✨ ✨
This is an experimental new layout that we’re considering for more articles in the future!
What do you think about it? Let us know below. 💖
Originally posted by @janiceilene in https://github.com/github/docs/discussions/9369

Okay thanks تم الإرسال من هاتف Huawei الخاص بي-------- الرسالة الأصلية --------من: "welcome[bot]" @.>التاريخ: الاثنين، ١٤ مارس، ٢٠٢٢ ١٢:٣٥ صإلى: github/docs @.>نسخة إلى: MAFIAWA @.>, Author @.>الموضوع: Re: [github/docs] ## ✨ ✨ New layout ✨ ✨ (Issue #16227)
Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.
—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>


I run the file cli.py ,but have something wrong like this:

I run the file cli.py ,but have something wrong like this,who can help me?
Traceback (most recent call last):
  File "D:\DL\Segment\cellpose-main\cellpose\gui\gui.py", line 838, in calibrate_size
    diams, _ = self.model.sz.eval(self.stack[self.currentZ].copy(),
  File "D:\DL\Segment\cellpose-main\cellpose\models.py", line 522, in eval
    styles = self.cp.eval(x,
  File "D:\DL\Segment\cellpose-main\cellpose\models.py", line 297, in eval
    masks, styles, dP, cellprob, p = self._run_cp(x,
  File "D:\DL\Segment\cellpose-main\cellpose\models.py", line 361, in _run_cp
    yf, style = self._run_nets(img, net_avg=net_avg,
  File "D:\DL\Segment\cellpose-main\cellpose\core.py", line 367, in _run_nets
    y, style = self._run_net(img, augment=augment, tile=tile, tile_overlap=tile_overlap,
  File "D:\DL\Segment\cellpose-main\cellpose\core.py", line 446, in _run_net
    y, style = self._run_tiled(imgs, augment=augment, bsize=bsize,
  File "D:\DL\Segment\cellpose-main\cellpose\core.py", line 547, in _run_tiled
    y0, style = self.network(IMG[irange], return_conv=return_conv)
  File "D:\DL\Segment\cellpose-main\cellpose\core.py", line 319, in network
    y, style = self.net(X)
  File "C:\Anaconda\envs\fengchong\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\DL\Segment\cellpose-main\cellpose\resnet_torch.py", line 260, in forward
    T0 = self.upsample(style, T0, self.mkldnn)
  File "C:\Anaconda\envs\fengchong\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\DL\Segment\cellpose-main\cellpose\resnet_torch.py", line 210, in forward
    x = self.up[-1](xd[-1], xd[-1], style, mkldnn=mkldnn)
  File "C:\Anaconda\envs\fengchong\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\DL\Segment\cellpose-main\cellpose\resnet_torch.py", line 162, in forward
    x = self.proj(x) + self.conv[1](style, self.conv[0](x), y=y, mkldnn=mkldnn)
  File "C:\Anaconda\envs\fengchong\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\DL\Segment\cellpose-main\cellpose\resnet_torch.py", line 129, in forward
    feat = self.full(style)
  File "C:\Anaconda\envs\fengchong\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\Anaconda\envs\fengchong\lib\site-packages\torch\nn\modules\linear.py", line 114, in forward
    return F.linear(input, self.weight, self.bias)
TypeError: linear(): argument 'input' (position 1) must be Tensor, not NoneType

Hi @FCgogogo , can you provide the exact command you used to run cellpose?
For reference running cellpose via the CLI, here are the command line arguments and here are examples of running the CLI.

I'm closing this for now. For more help using the CLI, please refer to image.sc forum here.


Unresolved reference: StringQualifier

Describe the bug
Unresolved reference: StringQualifier when using @Named for constructor parameter in class marked with @Single.
To Reproduce
Steps to reproduce the behavior:

Setup a KMP project
Mark a class with @Single
Mark a constructor parameter with @Named
Build
See error

Expected behavior
Successful build
Koin project used and used version (please complete the following information):
koin-core version 3.2.0-beta-1
koin-ksp 1.0.0-beta-1
ksp version 1.6.10-1.0.4

Unresolved reference: StringQualifier

this is in generated module I guess

I have the same bug. There is missing the import path for StringQualifier in the generated module.

Thanks, should be quick to fix

fixed in next beta 1.0.0-beta-2 - d6725c4

@arnaudgiuliani 1.0.0-beta-2 was not successfully publish to maven repository.

it's online


Audio not working correctly

Hi @JMPerez! I really enjoy the idea of this project so thank you for your work!
Currently, I am having trouble getting the audio to work at all. I can queue a song and have it "play" which shows the progress bar moving from start to finish. Am I missing something?
Edit: I have tried running it both locally and on the demo herokuapp.

Do you mean you can't hear the audio on the tab?

Yes exactly. I am accessing the app through chrome

Duplicate of #14

Thanks. Excuse my ignorance


v1 parser not resolving common parameters that use relative references

Using the v1 parser, top level parameters that use relative references are not being resolved. Take for example parsing the following swagger fragment:
...
parameters:
  param2:
    $ref: "./parameters/params.yaml#/param2"
...

Actual results:
...
parameters:
  param2:
    $ref: "./parameters/params.yaml#/param2"
...

Expected results:
...
parameters:
  param2:
    name: "param2"
    in: "header"
    required: false
    type: "string"
...

closed by #1088

@gracekarina any thoughts on when a v1 release with this fix might be coming?


updating module names for builder

updates module names for new builder

Looks like the history on this one is borked but this is included in #1440 via https://github.com/bitovi/canjs/commit/c820813e07c017a6c7de5b4d874a3a128e7e4d33 now anyway.


Add multi-scale unwrapping

This PR adds basic multi-scale unwrapping functionality.
API updates
A new public function multiscale_unwrap() is added, which estimates unwrapped phase from an interferogram using a multi-resolution unwrapping strategy.
First, the interferogram is multilooked down to lower resolution and unwrapped to form a coarse estimate of the unwrapped phase. Then, the original interferogram is broken down into smaller tiles, which are each unwrapped independently. In order to mitigate phase discontinuities at tile boundaries, the phase values within each tile are then augmented by some number of phase cycles, using the coarse unwrapped estimate as a reference for the absolute phase. Specifically, the phase within each connected component is adjusted by a number of phase cycles in order to minimize its mean absolute difference with the low-resolution unwrapped phase estimate.
Test cases
Two simple test cases have been added to exercise the new functionality.
The first test case simulates a noise-less interferogram with a roughly diagonal phase gradient such that, if the interferogram were unwrapped by tiles without any post-processing correction, there would be a significant phase offset between each tile. It then checks that the resulting unwrapped phase field differs from the simulated absolute phase only by a fixed integer number of phase cycles, with no phase discontinuities between tiles.
The figure below demonstrates the test data and compares the result of the multi-scale unwrapping approach with a naive tile-based unwrapping approach with no secondary unwrapping. The interferogram was partitioned into a 2x2 grid of tiles for unwrapping. In the simple tile-based unwrapping result, large phase discontinuities are visible along the tile boundaries.

The second test case is similar to the first, but increases the complexity by partitioning the interferogram into several isolated regions of high-coherence pixels, each of which span multiple tiles and are separated by noisy pixels. The test checks the unwrapped phase values within each high-coherence region independently.
The figure below shows the test dataset unwrapped using SNAPHU. Note that the connected component labels show inconsistencies and mislabelings between tiles. This issue will be addressed in subsequent work.

Known limitations
The following known issues will be addressed in subsequent PRs:

Multi-scale unwrapping using PHASS is currently not working (ICU & SNAPHU are working and both are exercised by the test suite). The current behavior is that the code raises an exception saying that the unwrapped phase values are not congruent with the input wrapped phase.
This seems to be due to a small number of pixels in the unwrapped phase that are filled with a large negative value (-10000) that is inconsistent with the surrounding phase values. The pixels are labeled as belonging to a connected component so they should be valid.
I'll need to investigate this further.

Currently, each tile is unwrapped in serial, one-by-one, but in the future we could unwrap them in parallel (perhaps using dask).

As noted above, connected component labels from different tiles are inconsistent. I plan to address this in my next PR. For now, only the output unwrapped phase is reliable.

PHASS support should be fixed now.
In one of the unit tests, there was an issue where I wasn't properly excluding all invalid pixels when checking the unwrapped phase values. There were just a few invalid phase values, which wasn't enough to cause the test assertion to fail in the case of ICU or SNAPHU. But PHASS seems to set invalid unwrapped phase values to -10000, causing the test to fail.
The test implementation should be fixed now, and the test suite has been updated to run with PHASS in addition to ICU & SNAPHU.

@scottstanie thanks again for the review. I think I've addressed all of your comments in the latest commits, excepting this suggestion that I plan to implement in a future PR.


Label removal from wrapped object does not trigger label removal on managed cluster

Describe the bug
When label is removed from an object wrapped by ManifestWork, it is not removed from the object in the managed cluster.
To Reproduce

Setup OCM with https://open-cluster-management.io/getting-started/quick-start/
Use kubectl to create a ManifestWork on the hub with the following yaml file. Note the test: test label in the deployment metadata.

apiVersion: work.open-cluster-management.io/v1
kind: ManifestWork
metadata:
  namespace: cluster1
  name: my-first-work
spec:
  workload:
    manifests:
      - apiVersion: v1
        kind: ServiceAccount
        metadata:
          namespace: default
          name: my-sa
      - apiVersion: apps/v1
        kind: Deployment
        metadata:
          namespace: default
          name: nginx-deployment
          labels:
            app: nginx
            test: test
        spec:
          replicas: 1
          selector:
            matchLabels:
              app: nginx
          template:
            metadata:
              labels:
                app: nginx
            spec:
              serviceAccountName: my-sa
              containers:
                - name: nginx
                  image: nginx:1.14.2
                  ports:
                    - containerPort: 80

Ensure deployment nginx-deployment in cluster1 is labeled with the test label
kubectl --context kind-cluster1 get deployment nginx-deployment -o yaml
Edit the ManifestWork and remove the test label.
kubectl --context kind-hub edit manifestworks my-first-work -n cluster1
Deployment nginx-deployment in cluster1 is still labeled with the test label.

Expected behavior
In step 6 above, I expect the test label to be removed.
Environment ie: OCM version, Kubernetes version and provider:
kind v0.19.0 go1.19 linux/amd64
Additional context
Add new label and update a label value works as expected.

we use openshift/library-go to apply resources, the label is always merged. see mergemap here https://github.com/openshift/library-go/blob/master/pkg/operator/resource/resourcemerge/object_merger.go#L141
To remove the label, you will need to explicitly set it by adding a "-" suffix, e.g.
labels:
  test: test-

From looking at https://github.com/openshift/library-go/blob/master/pkg/operator/resource/resourcemerge/object_merger.go#L141 , I can see that the key is checked for  "-" suffix, not the value.
Anyway this edit is invalid for labels. Valid label key and value must begin and end with an alphanumeric character ([a-z0-9A-Z]) according to https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/#syntax-and-character-set

/assign


All npx prisma CLI commands fail

Bug description
Any npx prisma command I run results in the following error message:
npx prisma generate
node:internal/fs/utils:345
    throw err;
    ^

Error: ENOENT: no such file or directory, open '/Users/will/Documents/projects/plating-ai/node_modules/prisma/build/prisma_fmt_build_bg.wasm'
    at Object.openSync (node:fs:591:3)
    at Object.readFileSync (node:fs:459:35)
    at ../../node_modules/.pnpm/@prisma+prisma-fmt-wasm<EMAIL_ADDRESS>(/Users/will/Documents/projects/plating-ai/node_modules/prisma/build/index.js:12449:31)
    at __require (/Users/will/Documents/projects/plating-ai/node_modules/prisma/build/index.js:14:50)
    at Object.<anonymous> (/Users/will/Documents/projects/plating-ai/node_modules/prisma/build/index.js:91437:38)
    at Module._compile (node:internal/modules/cjs/loader:1105:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Module._load (node:internal/modules/cjs/loader:827:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12) {
  errno: -2,
  syscall: 'open',
  code: 'ENOENT',
  path: '/Users/will/Documents/projects/plating-ai/node_modules/prisma/build/prisma_fmt_build_bg.wasm'
}

Node.js v18.2.0

I saw this comment and found that there is indeed no node_modules/prisma/build/prisma_fmt_build_bg.wasm next to node_modules/prisma/build/index.js.
What I have tried so far to try and resolve the bug:

Uninstall and reinstall prisma and @prisma/client npm packages
npm i
use node node_modules/prisma/build/index.js migrate deploy instead of npx prisma migrate deploy, recommendation source

None of that has worked so far and this issue just seemingly randomly popped up yesterday. Please help, I can make any modifications to my database!
How to reproduce

Run any npx prisma CLI command

Expected behavior
No response
Prisma information
// This is your Prisma schema file,
// learn more about it in the docs: https://pris.ly/d/prisma-schema
generator client {
provider = "prisma-client-js"
}
datasource db {
provider = "postgresql"
url      = env("DATABASE_URL")
}
model User {
id                  String    @default(cuid()) @id
email               String    @unique
emailVerified       DateTime?
isAdmin             Boolean   @default(false)
accounts            Account[]
sessions            Session[]
images              Image[]
recipes             Recipe[]
conversations       Conversation[]
stripeCustomerId    String?   @unique
stripeSubscribedAt  DateTime?
isActive            Boolean   @default(false)
createdAt           DateTime  @default(now()) @map(name: "created_at")
updatedAt           DateTime  @updatedAt @map(name: "updated_at")
@@map(name: "users")
}
model Account {
id                 String   @id @default(cuid())
userId             String
type               String
provider           String
providerAccountId  String
refresh_token      String?  @db.Text
access_token       String?  @db.Text
expires_at         Int?
token_type         String?
scope              String?
id_token           String?  @db.Text
session_state      String?
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
@@unique([provider, providerAccountId])
}
model Session {
id           String   @id @default(cuid())
sessionToken String   @unique
userId       String
expires      DateTime
user         User     @relation(fields: [userId], references: [id], onDelete: Cascade)
}
model VerificationToken {
identifier String
token      String   @unique
expires    DateTime
@@unique([identifier, token])
}
model Image {
id          String    @default(cuid()) @id
imageKey    String
description String?
style       String?
owner       User      @relation(fields: [ownerId], references: [id])
ownerId     String
recipe      Recipe?   @relation(fields: [recipeId], references: [id])
recipeId    String?
isPublic    Boolean   @default(false)
createdAt   DateTime  @default(now()) @map(name: "created_at")
updatedAt   DateTime  @updatedAt @map(name: "updated_at")
}
model Recipe {
id            String    @default(cuid()) @id
owner         User      @relation(fields: [ownerId], references: [id])
ownerId       String
name          String
time          String
difficulty    String
tools         String[]
ingredients   String[]
instructions  String[]
isPublic      Boolean   @default(false)
images        Image[]
createdAt     DateTime  @default(now()) @map(name: "created_at")
updatedAt     DateTime  @updatedAt @map(name: "updated_at")
}
model Conversation {
id        String   @default(cuid()) @id
createdAt DateTime @default(now())
updatedAt DateTime @updatedAt
owner     User     @relation(fields: [ownerId], references: [id])
ownerId   String
messages  Message[]
title     String
}
model Message {
id              String   @default(cuid()) @id
createdAt       DateTime @default(now())
updatedAt       DateTime @updatedAt
conversation    Conversation @relation(fields: [conversationId], references: [id])
conversationId  String
role            String
text            String
}
Environment & setup

OS: macOS Monterey
Database: PostgreSQL 12.14
Node.js version: 18.2.0

Prisma Version
{
  "devDependencies": {
    "prisma": "^4.12.0"
  }
}

npx prisma -v commands are not working, so I retrieved the information from my package.json file.

Update
I created a nextjs test project and added prisma as outlined in this tutorial: https://vercel.com/guides/nextjs-prisma-postgres. Result: no issues.
Maybe something uninstalling/reinstall prisma in the original project didn't actually clear and reinstall the node_module files 🤔

I now have it working. After uninstalling/reinstalling @next-auth/prisma-adapter as well as prisma and @prisma/client, npx prisma CLI commands are now working again.
My best guess is that some weird stuff was going on in the dependency management coordination between those semi-related packages 🤷

found the reason of my case:
- ENV NODE_OPTIONS='-r @newrelic/next'

this new relic option must be removed before running prisma


Feature Suggestion: use Intellij's default reformat action when file extension is not supported by Clang-format

This is a great tool to replace clion's integrated reformatting function. So I set a global hotkey for clang-format. But clang-format will also reformat python files even though it doesn't know how to reformat python code. So my suggestion is: Add file extension filters, and let clang-format only runs on supported file extensions and redirect non-supported file to Intellij's default reformat action.

This is a very useful plugin. The feature/bug may be caused by how the plugin calls internally clang-format.
Through re-routing through a custom script I found that the -assume-filename option is being used
 $CLANG_FORMAT -style=file -output-replacements-xml -assume-filename=<file name> -cursor=<cursor position>

which according to (this)[https://releases.llvm.org/10.0.0/tools/clang/docs/ClangFormatStyleOptions.html#configuring-style-with-clang-format] seems to force the formatter to be executed regardless if the language extension is supported:
[..] When formatting standard input or a file that doesn’t have the extension corresponding to its language,
`-assume-filename=` option can be used to override the file name clang-format uses to detect the language. [..]

The -i <file names> option does not have this issue.
@mprobst how may we entice you to either add a regular expression filtering on thee supported filenames prior to invoking clang-format and/or to use the -i option instead? I'd offer a :beer: and a free tour of our facility. Many thanks in advance.

For those who need a workaround for this issue:
The following script may be usefull (N.B. needs executable flags, in the user's $PATH and e.g. named clang-format or clang-format-custom with the setting in the IDE adjusted accordingly):
#/bin/bash
arg3=$3
arg4=$4
newarg3=${arg3#*=}
newarg4=${arg4#*=}
match=$(echo $newarg3 | grep -Ei "\.(cc|cpp|cxx|c\+\+|h|hh|hpp|hxx|h\+\+|java)")
if [ -n "$match" ]; then
  /usr/bin/clang-format $1 $2 $4 -i $newarg3
else
  echo "<?xml version='1.0'?>"
  echo "<replacements xml:space='preserve' incomplete_format='false'>"
  echo "<cursor>${newarg4}</cursor>"
  echo "</replacements>"
fi

Enjoy! :smile:


error on MOT evaluation using run_mot_challenge.py

Hello ,
Im getting following error with this command
!python scripts/run_mot_challenge.py --DO_PREPROC False --BENCHMARK MOT20-train
raise TrackEvalException('no seqmap found: ' + os.path.basename(seqmap_file))
trackeval.utils.TrackEvalException: no seqmap found: MOT20-train-train.txt
Looking forward to your respond

How did you solve it?
I have the same problem and I'm looking for an answer.

hi everyone, any updates on how to fix it?


Frameworks compile error: 'Starboard/SmartTypes.h' file not found

With a clean build of develop using Visual Studio 2017 and the latest available winobjc-tools v0.2.170229-dev-20170404101752, I'm getting:
------ Build started: Project: StarboardLib, Configuration: Release Win32 ------
C:\develop\WinObjC\build\Starboard\lib\..\..\..\Frameworks\Starboard\PlatformSupport.cpp
C:\develop\WinObjC\Frameworks\include\Starboard.h(208,10): fatal error : 'Starboard/SmartTypes.h' file not found

I'm able to build with 96255883f4dff8c6d16fb49ecd067d56ea2a841d so this looks to be caused by #2419.

This is dupe of #2444 .

This is caused by us not having published an updated WinObjC.Language package. @ehren you may have to build the tools solution before you build WinObjC proper.

This isn't a blocker for me in any case (I rebased my functional test pull against the recent 96255883f4dff8c6d16fb49ecd067d56ea2a841d mentioned above - the functional tests also work fine in that commit)

chocolatey upgrade winobjc-tools --pre --version=0.2.170229-dev-20170404101752 --force should get you to the latest package from develop.  The issue would resolve itself once we have an official version published from master, which would be any day now.

Scratch that.  As @DHowett-MSFT stated, the header is from WinOBJC.Language package, which is not coming from chocolatey.

I was able to complete the build after the successful tools.sln build.

Closing as resolved! We'll point future folks here when/if they run into it.


Attribute filter fix

Allowing multiple attribute filters with multiple ranges;

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

193 of 221 (87.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 85.309%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

operators/src/source/ogr_source/dataset_iterator.rs
22
28
78.57%

operators/src/source/ogr_source/mod.rs
171
193
88.6%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
30004

Relevant Lines:
35171

💛  - Coveralls

bors merge


Using up Google API very quickly

Whenever I run the search code, the google JSON API key limit gets reached within a minute and opposite to what is claimed, only 5-20 profiles are scraped (I tried it with multiple API keys from diff. accounts). Is there a setting needed somewhere to be done?

Attaching search payload

Hi Yash,
By default the program automatically stores previously stored search terms and previously indexed accounts in .pkl files to prevent duplicate results. This is intentional as the google results do not update very often, but it is definitely challenging to work around when initially using the tool and setting it up. I think this might be why you are having difficulties.
To fix, you could just download the repository again, or go to /search_tool/data and delete all data in the indexed_queries.pkl and indexed_profiles.pkl files. If you would like to remove the duplicate checking functionality entirely, I think that you might be able to just check the repeat queries box in the UI, and change function load_indexed_profiles in the search_tool/indexed_data.py file to just always return [].
I hope this helps. If you are still having trouble, try searching again with a completely new search term you have never used before to see if that works, if not, I’d be more than happy to help.
Ethan
On Aug 22, 2023 at 5:15 AM -0400, Yash Bansal @.***>, wrote:
Whenever I run the search code, the google JSON API key limit gets reached within a minute and opposite to what is claimed, only 5-20 profiles are scraped (I tried it with multiple API keys from diff. accounts). Is there a setting needed somewhere to be done?
—
Reply to this email directly, view it on GitHubhttps://github.com/ethbak/connexion-for-linkedin/issues/1, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BATNCXEHTHFUPE57KLZZQ5TXWR2DNANCNFSM6AAAAAA3ZTA3QQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


How i add a errorView

I like add a View as navigation error by default --> .withDefaultErrorView()

AppLayout drawer = AppLayoutBuilder.get(variant)
...
.withNavigatorConsumer(navigator -> {/* Do someting with it */})
...


Zoom to fit is zooming out (making card smaller)

Are you using the latest version of the plugin?

[X] I have verified that I am using the latest version

Have you checked for similar issues?

[X] I have verified that there are no similar issues

What happened?
Zoom to fit  button is zooming out the view
What did you expect to happen?
"Zoom to fit" will maximalize cards to fit into screen
Relevant console output
No output.
Obsidian version
1.6.1
OS
macOS

I am in the live preview mode, I have only  1 card in my note, every time when I use zoom to fit it keeps getting smaller and smaller and it doesn't stop, the same way as zooming out behaviour


Fix clone(2) with double fork

Fix #185
This PR replaces clone(2) with double fork to create the container process. Double fork was the original implementation, and we wisely thought that we could improve it by using a single clone call. Turns out jokes on us that we can't use a single clone call. With clone, if we enter into a new pid namespace, we are OK. However, if we need to set_ns into an existing pid namespace, we are out of luck. So the right path forward needs to be double fork. For a more detailed discussion, see #185 why it doesn't work.
Also, turns out, this is a bigger change than I intended. So I apologize up front for the big change.

@yihuaf
Could you give me an example command that I can use to chuck that this has been fixed correctly?

Maybe, but if we don't use rootless, do we need to do double-fork?

@yihuaf
Could you give me an example command that I can use to check that this has been fixed correctly?

Try
docker run -it --rm --runtime youki --name youki busybox

and then
docker exec -it youki sh

In the shell using docker exec, check /proc/self and /proc/2 are set up correctly. Before, because exec didn't enter the correct pid namespace, /proc/<pid> was not configured correctly. In the main shell by docker run, you should also be able to see /proc/2 for the docker exec shell.
Pid 1 -> docker run shell
Pid 2 -> docker exec shell

Maybe, but if we don't use rootless, do we need to do double-fork?

There are 2 cases where we don't need to double fork.

If we don't use rootless and/or don't enter into an existing user namespace, we can enter into Pid namespace on the Youki main process and then fork the init process.
If we don't need to enter an existing Pid namespace through set_ns, we can use clone to enter user and pid namespace together and create the init process directly.

@yihuaf
Could you give me an example command that I can use to check that this has been fixed correctly?

Try
docker run -it --rm --runtime youki --name youki busybox

and then
docker exec -it youki sh

In the shell using docker exec, check /proc/self and /proc/2 are set up correctly. Before, because exec didn't enter the correct pid namespace, /proc/<pid> was not configured correctly. In the main shell by docker run, you should also be able to see /proc/2 for the docker exec shell.
Pid 1 -> docker run shell
Pid 2 -> docker exec shell

I was able to confirm the change. Thank you very much.
main
$ docker exec -it youki sh
/ # ps
PID   USER     TIME  COMMAND
    1 root      0:00 sh
    4 root      0:00 ps

this pr
$ docker exec -it youki sh
/ # ps
PID   USER     TIME  COMMAND
    1 root      0:00 sh
    2 root      0:00 sh
    3 root      0:00 ps
/ #

There are 2 cases where we don't need to double fork.

If we don't use rootless and/or don't enter into an existing user namespace, we can enter into Pid namespace on the Youki main process and then fork the init process.
If we don't need to enter an existing Pid namespace through set_ns, we can use clone to enter user and pid namespace together and create the init process directly.

Does this mean that if uid_mapping and gid_mapping are not needed, double-fork is not needed?
I thought it might be a good idea to improve the case where double-fork is not needed. However, once this PR is done, it's good.

I was hoping these PRs would pass with this PR fix, but it looks like not.
https://github.com/containers/youki/blob/9acc899239ee0b9d58ea1a81b69884d13ba561fe/integration_test.sh#L37-L40

I will not point out any details in this PR. We will not point out details in this PR, because this PR is very important and will be very hard to merge because of the large number of changes. Those corrections will be made in the PR little by little. I am looking forward to your reviews.

I was hoping these PRs would pass with this PR fix, but it looks like not.
https://github.com/containers/youki/blob/9acc899239ee0b9d58ea1a81b69884d13ba561fe/integration_test.sh#L37-L40

I will take a look at these tests right after this. #221 filed.

There are 2 cases where we don't need to double fork.

If we don't use rootless and/or don't enter into an existing user namespace, we can enter into Pid namespace on the Youki main process and then fork the init process.
If we don't need to enter an existing Pid namespace through set_ns, we can use clone to enter user and pid namespace together and create the init process directly.

Does this mean that if uid_mapping and gid_mapping are not needed, double-fork is not needed?
I thought it might be a good idea to improve the case where double-fork is not needed. However, once this PR is done, it's good.

Actually no. Without uid/gid mapping, we would still require double fork unless we are in one of two cases mentioned above. I think we can potentially improve this by differentiating which case to use, but it would take some effort to get all the corner cases right. The double fork implementation would cover all the cases and should be good enough for the moment.

I will not point out any details in this PR. We will not point out details in this PR, because this PR is very important and will be very hard to merge because of the large number of changes. Those corrections will be made in the PR little by little. I am looking forward to your reviews.
If you respond to some of my comments, I'll try to merge them.
Finally, it would be helpful if you could leave a brief summary of this issue on issue or on this PR.

That's probably a good idea, and I am happy to continue to fix all the details. I will summarize the issue here again, and #185 contains a history of how we got here.
Before this PR, the implementation was to use clone to create the container init process, joining new user and new pid namespaces together. However, in the case of docker exec, the container process will join existing namespaces user and pid namespaces. The set_ns to pid namespaces requires an extra fork to enter into the correct pid namespace. As a result, the container process didn't join the right pid namespaces. Many of the pid related operations were failing, such as looking at /proc/self and /proc/<pid>. The solution in this PR is to always double fork, one to enter user namespace and another fork to enter into pid namespace. We can't enter user and pid namespace together because entering into pid namespace requires CAP_SYSADMIN so we have to enter into user namespace first.


Deep nesting objects with custom __getattr__ function gives RecursionError in pydantic 1.9.1

Checks

[X] I added a descriptive title to this issue
[X] I have searched (google, github) for similar issues and couldn't find anything
[X] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.9.1
            pydantic compiled: True
                 install path: /home/sevaho/.cache/pypoetry/virtualenvs/trinity-M8VqpuQK-py3.9/lib/python3.9/site-packages/pydantic
               python version: 3.9.9 (main, Jan  9 2022, 13:11:20)  [GCC 11.1.0]
                     platform: Linux-5.17.1-arch1-1-x86_64-with-glibc2.35
     optional deps. installed: ['dotenv', 'typing-extensions']
...

Description
The following examples produce RecursionError when using pydantic 1.9.1, with 1.9.0 all works fine.
Examples
from pydantic import BaseModel

class State(object):
    def __getattr__(self, key):
        return self._state[key]

class FastAPI(object):
    def __init__(self):
        self.state = State()

class App(BaseModel):
    class Config:
        arbitrary_types_allowed = True

    web: FastAPI

class Test(BaseModel):
    app: App

# Works fine
App(web=FastAPI())

# Nesting another level gives errors
Test(app=App(web=FastAPI()))

Same use case with deep nesting fastapi:
from pydantic import BaseModel
from fastapi import FastAPI

class App(BaseModel):
    class Config:
        arbitrary_types_allowed = True

    web: FastAPI

class Test(BaseModel):
    app: App

# Works fine
App(web=FastAPI())

# Nesting another level gives errors
Test(app=App(web=FastAPI()))

Error output
web=<__main__.FastAPI object at 0x7f13cc80d5e0>
Traceback (most recent call last):
  File "/home/sevaho/gitlab/auth-trinity/test.py", line 27, in <module>
    Test(app=App(web=FastAPI()))
  File "pydantic/main.py", line 339, in pydantic.main.BaseModel.__init__
  File "pydantic/main.py", line 1038, in pydantic.main.validate_model
  File "pydantic/fields.py", line 857, in pydantic.fields.ModelField.validate
  File "pydantic/fields.py", line 1074, in pydantic.fields.ModelField._validate_singleton
  File "pydantic/fields.py", line 1121, in pydantic.fields.ModelField._apply_validators
  File "pydantic/class_validators.py", line 313, in pydantic.class_validators._generic_validator_basic.lambda12
  File "pydantic/main.py", line 679, in pydantic.main.BaseModel.validate
  File "pydantic/main.py", line 605, in pydantic.main.BaseModel._copy_and_set_values
  File "/usr/lib/python3.9/copy.py", line 146, in deepcopy
    y = copier(x, memo)
  File "/usr/lib/python3.9/copy.py", line 230, in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
  File "/usr/lib/python3.9/copy.py", line 172, in deepcopy
    y = _reconstruct(x, memo, *rv)
  File "/usr/lib/python3.9/copy.py", line 270, in _reconstruct
    state = deepcopy(state, memo)
  File "/usr/lib/python3.9/copy.py", line 146, in deepcopy
    y = copier(x, memo)
  File "/usr/lib/python3.9/copy.py", line 230, in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
  File "/usr/lib/python3.9/copy.py", line 151, in deepcopy
    copier = getattr(x, "__deepcopy__", None)
  File "/home/sevaho/gitlab/auth-trinity/test.py", line 6, in __getattr__
    return self._state[key]
  File "/home/sevaho/gitlab/auth-trinity/test.py", line 6, in __getattr__
    return self._state[key]
  File "/home/sevaho/gitlab/auth-trinity/test.py", line 6, in __getattr__
    return self._state[key]
  [Previous line repeated 976 more times]
RecursionError: maximum recursion depth exceeded

@sevaho thank you for the bug report. Could you please confirm that the bug is still relevant in Pydantic v2?

@lig I can confirm this is still happening in Pydantic 2.5.1

I very much doubt your getting the same error error as V1, @parmenashp beat to create a new issue with a MRE.
Closing this as we have no plans to work on fixing this in v1.


Add --interface-opts argument to "manage.py shell"

This PR adds a command-line argument to manage.py shell called "--interface-opts", which enables users to provide a string of additional command-line options to be passed along to the interactive interpreter when the interpreter is not "python".
For example, to start ipython with a classic prompt, with no "confirm-on-exit" behavior:
python manage.py shell --interface ipython --interface-opts '--no-confirm-exit --simple-prompt --classic'
Open to feedback and suggestions.

There was some previous discussion about removing the shell command with the rationale:

The downside of having "shell" in Django is that it's a non-trivial maintenance burden to decide which REPLs to support, add support for them, and then update/fix that support through the years. We've already seen a steady stream of tickets related to various edge-cases in IPython startup, not to mention the major API changes in recent IPython versions. There's no reason for Django to have to be maintaining code related to IPython."

In light of that, I think it's unlikely that there would be consensus to add more code related to the current barebones ipython/bython support.
p.s. For new feature suggestions like this, the first step is to create a Trac ticket rather than send a pull request.


fix: await-interaction fixer adds 'async'

Issue: #72
What Changed

await-interaction fixer now adds 'async' if not already present on function containing invocationsThatShouldBeAwaited

Checklist
Check the ones applicable to your change:

[X] Ran yarn update-all
[ ] Tests are updated
[ ] Documentation is updated

Change Type
Indicate the type of change your pull request is:

[ ] maintenance
[ ] documentation
[X] patch
[ ] minor
[ ] major

Hey @aurmer this is incredible! Thank you so much for your contributions ❤
I added a small fix and test cases, it all looks great 👍


gnome-extension-manager is broken on NixOS 24.11

Describe the bug
gnome-extension-manager is broken on NixOS 24.11, only a blank window is shown when run. Terminal logs:
(extension-manager:18498): Gtk-CRITICAL **: 19:03:25.374: Error building template class 'ExmWindow' for an instance of type 'ExmWindow': .:0:0 Invalid object type 'ExmInstalledPage'

(extension-manager:18498): GLib-GObject-CRITICAL **: 19:03:25.823: g_object_set: assertion 'G_IS_OBJECT (object)' failed

(extension-manager:18498): GLib-GObject-CRITICAL **: 19:03:25.823: g_object_set: assertion 'G_IS_OBJECT (object)' failed

(extension-manager:18498): GLib-GObject-CRITICAL **: 19:03:25.823: g_object_set: assertion 'G_IS_OBJECT (object)' failed

(extension-manager:18498): GLib-GObject-CRITICAL **: 19:03:25.823: g_object_bind_property_full: assertion 'G_IS_OBJECT (target)' failed

Steps To Reproduce
Steps to reproduce the behavior:

nix run github:nixos/nixpkgs/nixos-24.11#gnome-extension-manager

Expected behavior

The app should actually display something
Screenshots

Additional context
The Flatpak version works fine
Metadata

system: "x86_64-linux"
host os: Linux 6.6.60, NixOS, 24.11 (Vicuna), 24.11.20241116.057f63b
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.24.10
channels(root): ""
nixpkgs: /nix/store/idq1bmwwy9rlkc21hccvx42wlwpxsx1f-source

Notify maintainers

@foo-dogsquared

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Can confirm.

Possibly relevant:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1082621


Update README.md

I'm unable to approve my own pr. If anyone is willing to review it would be appreciated.

Thanks, you were suggested as someone to review.  I've taken over as maintainer of this project and am looking to work my way through the existing issues and make a few minor improvements moving forward.

Ishan, We should give Benjamin full ability to review and approve PRs.
On Mon, Feb 3, 2020 at 8:50 AM Benjamin F Knigge<EMAIL_ADDRESS>wrote:

Thanks, you were suggested as someone to review. I've taken over as
maintainer of this project and am looking to work my way through the
existing issues and make a few minor improvements moving forward.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/moovweb/gvm/pull/335?email_source=notifications&email_token=AAAPZKFNKLGQ26HJ6TS25OTRBBDONA5CNFSM4KO7UPX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKUR6ZQ#issuecomment-581508966,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPZKDM2VA4CBGVHTJSWJLRBBDONANCNFSM4KO7UPXQ
.

I have the ability to approve PR's just not the ability to approve my own.
I'm going to be looking at re-enabling travis as well and am not sure if I
have the proper permissions to do that.
Thanks,
Ben
On Mon, Feb 3, 2020 at 9:53 AM Ajay Kapur<EMAIL_ADDRESS>wrote:

Ishan, We should give Benjamin full ability to review and approve PRs.
On Mon, Feb 3, 2020 at 8:50 AM Benjamin F Knigge<EMAIL_ADDRESS>
wrote:

Thanks, you were suggested as someone to review. I've taken over as
maintainer of this project and am looking to work my way through the
existing issues and make a few minor improvements moving forward.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<
https://github.com/moovweb/gvm/pull/335?email_source=notifications&email_token=AAAPZKFNKLGQ26HJ6TS25OTRBBDONA5CNFSM4KO7UPX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKUR6ZQ#issuecomment-581508966
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAAPZKDM2VA4CBGVHTJSWJLRBBDONANCNFSM4KO7UPXQ
.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/moovweb/gvm/pull/335?email_source=notifications&email_token=ACJHMF7QIQ7LEMGW4QN4D2DRBBDYZA5CNFSM4KO7UPX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKUSI2I#issuecomment-581510249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACJHMFZEE2IYZQCOQ7WGRCDRBBDYZANCNFSM4KO7UPXQ
.

--
Benjamin Knigge
<EMAIL_ADDRESS>tel: +1-310-570-2880

@BenKnigge I requested to give you admin access or the highest possible. let me know if you run into permissions issues.

From our IT team:

[W]hat he seems to be running into was a master branch limit. I removed all limitations on master branch PRs; please have him try again.

Thanks
On Mon, Feb 3, 2020, 2:00 PM Ishan Anand<EMAIL_ADDRESS>wrote:

From our IT team:
[W]hat he seems to be running into was a master branch limit. I removed
all limitations on master branch PRs; please have him try again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/moovweb/gvm/pull/335?email_source=notifications&email_token=ACJHMF22IZTC2UZ6EU2ARK3RBCAWJA5CNFSM4KO7UPX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKVMBXY#issuecomment-581615839,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACJHMF6R7PEPTNXFTA7OW2LRBCAWJANCNFSM4KO7UPXQ
.


fix playwright._impl._errors.TargetClosedError: Target page, context or browser has been closed on Windows

Fix https://github.com/eggplants/deepl-cli/issues/221

according to chromium-dev:
The code for chrome is split between two DLLs, but there's some code duplicated (base, etc.). Only one DLL is loaded into each chrome process (chrome.dll for the browser, chrome_child.dll for others). We didn't attempt to keep single process working in that mode as it was unnecessary and potentially tricky/confusing given two copies of the same code in the same processes.
Setting chrome_multiple_dll=0 at gyp time is probably the simplest solution for single process mode in Chrome.

The main issue is that Chromium on Windows is built with chrome_multiple_dll=0, which does not support the --single-process flag. This leads to failures when attempting to run Chromium in a single process mode on Windows, the browser crashes immediately after launch.
Fix:
If the platform is Windows, the --single-process flag will be automatically disabled to ensure compatibility. On non-Windows platforms, the flag will still be used as expected.
Testing Results:
Tested on my fork, and the following outcomes were observed:

On Windows: --single-process is disabled, and Chromium runs without issues.
On Linux/macOS: --single-process works as expected, with no regressions or failures.

All relevant test have passed successfully in both environments.

@sakkyoi Thank you for your contribution!

https://github.com/eggplants/deepl-cli/releases/tag/v0.8.0


integration: V3 grpc with clientv3 (only Put)

@heyitsanthony If you haven't worked on integration test on clientv3, can I work on it?
Kind of duplicate with grpc test, but it's good to have both. Please review.
/cc @xiang90

one last bit--- v3_grpc_test.go is getting huge; the non-grpc stuff can go in v3_client_test.go

we should move this to clientv3/integration as I mentioned here https://github.com/coreos/etcd/pull/4332#issuecomment-176496427.
We should try our best to not introduce external dependency in our server tests.

@xiang90 Ok I will put these to the separate directory. Thanks

@heyitsanthony @xiang90 Just separated. PTAL. Thanks!

@gyuho Let's start with the simple test TestKVPut. TestKVPut should test put keys into etcd. After each put, we should do a get to get the put key. If it succeeds, the test passes.

@xiang90 Agree. This should be testing only clients.
Just cleaned up to have TestKVPut. PTAL. Thanks!

LGTM after fixing the last few comments.
Then we can test KVGet, KVRange, KVDelete, KVDeleteRange.

Thanks. Just fixed and updated test script to run integration test in clientv3. Will write tests for
other functions in a separate PR.

LGTM if test passes

Thanks


Interpreting stack/cabal projects

I think it would be great to have functions to initialize a hint session from a .cabal or stack.yaml file,
the same way as cabal repl or stack repl initializes ghci.
The functions could be for example:
runInterpreterWithStackFile, runInterpreterWithCabalFile
  :: (MonadIO m, MonadMask m)
  => FilePath
  -> InterpreterT m a
  -> m (Either InterpreterError a)

I've tried to implement that functionality myself by running stack repl --with-ghc echo to get the command line arguments normally passed to ghci, and forwarded them to unsafeRunInterpreterWithArgs, after removing the --interactive argument, which seems to be rejected by hint. With this approach I was able to interpret an example stack project, which was depending on another external package. The approach seems not very robust, though, and I'm not convinced how well it works in general. ;-)
Thanks for working on hint, by the way! :+1:
I'm currently trying to use it in my text editor to load a Haskell configuration project, which can depend on compiled editor plugins. :-)

+1, that would be useful for my project as well

If you send a PR, I'll be happy to review :)

Great that you guys are positive towards the idea! :-)
I've implemented a first prototype for runInterpreterWithStackFile here: 8381efdf969349c1e9e38d43d806b29989a4d5c6
However, I'm not very happy with the approach of parsing the output of stack --with-ghc=echo.
This causes an ugly dependency on how the stack executable prints to stdout.
I've reexported the functions in the Language.Haskell.Interpreter.Unsafe module, as stack forwards the ghc-options from the .cabal file to ghci, so I guess the same applies as for unsafeRunInterpreterWithArgs.
I wonder if it would be reasonable to use stack as a library instead. This may offer safety and speed, but on the other hand would add a big dependency. The latter could be mediated by having a hint-stack and hint-cabal package, so hint's dependencies can remain small.
I just had a quick look at the stack code base, but wasn't able to find a direct way for retrieving the ghci arguments. However, it might be not that difficult to reproduce the corresponding part from the stack executable. Here is the part in stack which creates the ghci arguments.

I agree that adding the dependency to hint itself would not be a good idea.
Hint itself doesn't depend on ghc being readily executable, so it would be unfortunate if now we depended on other tools like cabal or stack to be. Perhaps separate packages like you suggest would be the best way forward.
I have no idea about the repl command or much knowledge about cabal/stack in general, so my design input is fairly limited.

I agree that adding the dependency to hint itself would not be a good idea.
Hint itself doesn't depend on ghc being readily executable, so it would be unfortunate if now we depended on other tools like cabal or stack to be. Perhaps separate packages like you suggest would be the best way forward.

That makes sense.
What do you think about the approach used in the current prototype with spawning processes?
The only new static dependency would be process, but the stack-specific hint functions would require a stack executable on the runtime PATH to succeed. The main belly ache I get from this approach, is the whole stringyness and cross-platform support, but maybe those are wrong instincts in this case ;-)

I have no idea about the repl command or much knowledge about cabal/stack in general, so my design input is fairly limited.

I'm pretty much in the same situation. :-)
My hopes are that the problem can be solved by reusing much of the infrastructure already provided for ghci. Maybe we could get some input on the stack mailing list.
By the way, what exactly makes unsafeRunInterpreterWithArgs unsafe?
Are there arguments (besides --interactive) that ghci handles but hint doesn't?

But I also completely understand, if you want to avoid such runtime dependencies in an otherwise pure package for consistency.

I assume if they are separate packages, any other packages could depend on and only build what they need? For example, current users of hint could ignore hint-cabal and hint-stack. If that is possible, I'd definitely prefer using cabal/stack as libraries at compile time.
If it turns out to be impossible or ridiculously complex, then perhaps hacky command execution at runtime is a good option.

By the way, what exactly makes unsafeRunInterpreterWithArgs unsafe?

See https://github.com/mvdan/hint/pull/4. Basically, it's possible to make mistakes (e.g. typos in the type names) and the compiler won't catch them. Otherwise, the compiler will actually resolve the types.
As seen in the PR, unsafe is more powerful because, for example, it lets you come up with the types at runtime, not at compile time. Think of it as some sort of code generation. That's not possible at compile time, unless you have boilerplate for every single possible type.

These days, a more modern way to accomplish this would be to use https://github.com/mpickering/hie-bios to convert a stack or cabal project into the set of flags we need to pass to ghc.

(I didn't remove the "help wanted" tag because help is no longer wanted, but because that tag is redundant; I welcome help on all the issues, not just those which have the "help wanted" tag!)


test

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code cleanup/refactoring
[ ] Documentation update
[ ] This change requires a documentation update
[ ] CI system update
[ ] Test Coverage update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Below is just an example and you can delete it when you contribute.
Test Configuration:

Java: v11
LPVS Release: v1.x.x

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

[License Pre-Validation Service]  No license issue detected

[License Pre-Validation Service]  No license issue detected

[License Pre-Validation Service]  No license issue detected
Detected licenses:
File: src/main/java/com/lpvs/auth/OAuthAttributes.java
License(s): MIT (permitted)
Component: seb45_pre_023 (src/main/java/sixman/stackoverflow/auth/oauth/service/Provider.java)
Matched Lines: 1-54
Snippet Match: 98%

[License Pre-Validation Service]  No license issue detected
Detected licenses:
File: src/main/java/com/lpvs/auth/OAuthAttributes.java
License(s): MIT (permitted)
Component: seb45_pre_023 (src/main/java/sixman/stackoverflow/auth/oauth/service/Provider.java)
Matched Lines: 1-54
Snippet Match: 98%


Fix ingress backend broken SAN

Fix an issue where the trust domain is getting added to the full SAN provided in ingress backend's. This is accomplished by switching from adding service identity (which does not contain trust domain) to principals (which does), on the traffic policy rules and policy builder.

Codecov Report

Merging #4914 (26b6c33) into main (f768f64) will decrease coverage by 0.03%.
The diff coverage is 93.93%.

:exclamation: Current head 26b6c33 differs from pull request most recent head 1a6dc9d. Consider uploading reports for the commit 1a6dc9d to get more accurate results

@@            Coverage Diff             @@
##             main    #4914      +/-   ##
==========================================
- Coverage   68.67%   68.63%   -0.04%
==========================================
  Files         220      220
  Lines       15944    15939       -5
==========================================
- Hits        10949    10940       -9
- Misses       4943     4947       +4
  Partials       52       52

Flag
Coverage Δ

unittests
68.63% <93.93%> (-0.04%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/identity/types.go
74.07% <0.00%> (-5.93%)
:arrow_down:

pkg/catalog/catalog.go
91.30% <100.00%> (+1.30%)
:arrow_up:

pkg/catalog/inbound_traffic_policies.go
94.90% <100.00%> (ø)

pkg/catalog/ingress.go
96.26% <100.00%> (-0.04%)
:arrow_down:

pkg/envoy/lds/rbac.go
80.35% <100.00%> (-0.68%)
:arrow_down:

pkg/envoy/rbac/policy.go
100.00% <100.00%> (ø)

pkg/envoy/rds/route/rbac.go
92.85% <100.00%> (-0.70%)
:arrow_down:

pkg/trafficpolicy/trafficpolicy.go
96.15% <100.00%> (ø)

pkg/certificate/manager.go
76.70% <0.00%> (-0.81%)
:arrow_down:

pkg/httpserver/httpserver.go
84.90% <0.00%> (-0.55%)
:arrow_down:

... and 2 more

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 f768f64...1a6dc9d. Read the comment docs.


Compare feature with frontend and single-file diff

Gitea version (or commit ref): release/v1.1

Description
I cannot find a page or a button where I am able to choose two commits/tags/branches for comparsion. GitLab and GitHub provide such a feature. It is really helpful to undestand what is going on in your repo.

Having an additonal "Compare" button (or/and a "changes since this commit") while viewing a commit would also be great

To give a bit more ideas of what I think would be helpful for a better compare feature:
At the FSFE we switched from trac/SVN to Gitea to maintain our website. With trac, our voluntary translators were able to see the diff of a file between two freely choosable commits.
For example, with the file work.en.xhtml we would like to see the difference between the current version/latest change (4e89fd20cc) and a freely choosable other date/commit (e.g. d25c06af1e).
In trac, this diff is quite helpful for translators to know which bits have changed in a file over a certain span of time to update an outdated translated version.

Can we please rename this issue, e.g. to "Compare feature with frontend and single-file diff" or so? The current title doesn't seem to catch what we discussed here.

I would still find this highly interesting and relevant in order to make Gitea more professionally usable.

I would love to see this implemented.

Any update here?

 It's such a basic and important feature.

This feature would be much appreciated. I've used it a lot when using BitBucket in the past, and when using locally installed windows GUIs too. It's just that currently I am not allowed to install anything, otherwise it would be easy to work around this problem.


Diffing two empty files shows them as having changes

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

VS Code Version: 1.80.1
OS Version: macOS 13.4.1

Steps to Reproduce:

Diff two empty file
Notice the diff view shows "removed" and "added" lines

This issue is the same as #113318 but happens again

Can you share a screenshot?

An empty file compare to itself

I see, thanks!

To verify:
Select latest dev version and notice there is no red/green in this playground example.

The issue is verified. I am wondering if we want to see the commands starting with Compare active file with when there is no active file? I see this command while I have no active text editor.


emoji_rendering: Increase emoji size if its only message content.

Added new feature of rendering emoji at larger size when its the only content of the message .
The current sizes are :

32x32 :-  For devices larger than 768px.
22x22 :- For devices smaller than 768px.

ScreenShots :
Larger devices :

Smaller devices :

Closes #15390
Edit : The sizes of emoji can be discussed within CZO , or maybe shifted back to what @YashRE42 suggested here .

CZO topic : https://chat.zulip.org/#narrow/stream/6-frontend/topic/emoji.20size

@aryanshridhar Thanks for working on this!  This looks like a pretty promising PR.  Do you think you'll have time to address some of the comments?
(I apologize that my recent comments aren't on the PR itself...I got there from somewhere else and didn't notice that you had created a PR, but you should be able to find my comments.)

@aryanshridhar Thanks for working on this!  This looks like a pretty promising PR.  Do you think you'll have time to address some of the comments?
(I apologize that my recent comments aren't on the PR itself...I got there from somewhere else and didn't notice that you had created a PR, but you should be able to find my comments.)

@aryanshridhar Thanks for working on this! This looks like a pretty promising PR. Do you think you'll have time to address some of the comments?

Yeah sure, Will be excited to work on this new feature :), Will make the necessary changes as suggested and fix the failing tests.

@aryanshridhar Thanks for working on this! This looks like a pretty promising PR. Do you think you'll have time to address some of the comments?

Yeah sure, Will be excited to work on this new feature :), Will make the necessary changes as suggested and fix the failing tests.

Also @showell, Should enlargement of relam emojis be considered as well?

Updated the Pr to reflect the following changes:

Changed emoji-large class (that was previously being added) to large-emoji (as suggested by @showell earlier).
Current size of emoji being 32x32 for width > 768px and 22x22 for devices smaller than 768px.
Updated the translate_emoticon function to only convert if the text isn't already a valid emoji.(Due to issue listed here: 9d3345451b5ea2d324ba78a5164b69607ce3fb16)
Enlarges the emoji that were input as unicode/colon/emoticon (if page_params.translate_emoticons is set to true) or a realm_emoji.

Update Screenshots:

I merged the first (comment) commit as f19c7a2f690cb88aab8cb7b17aa5335995cfc5a3; posting comments on the rest.
Am I correct that this PR is primarily blocked on the CSS to make such messages not overflow their line?

@aryanshridhar
Is there any chance you could attempt this again?
We would love to use this because the emojis are just tiny on huge screens.


List and process only valid files in DBS

With this patch the agent should no longer create jobs with invalid files. Fixes #5589
It's not straight forward to test it, so I'll first check whether it does not break anything.
Don't merge yet please
@ticoann , I removed those changes to DBSInterface.py

I'd like to push it to production, @yuyiguo @ticoann could you please take a last look at it?
cmssrv113 has been operating with this patch for the last 24h and everything looks ok, we ran several workflows there.

Looks fine with me and it even tested. :smile:

I think need to get in GQ as well for correct splitting estimation. So I am merging it.

Thanks Seangchan! I'm pushing it now to all the agents then.
If things explode, you know what was the reason :-)


reset search filter on resetSuccess

Coverage increased (+0.01%) to 40.288% when pulling 7e5c8e9dcb4e780825b8098e6eca4f5c28de178c on embbnux:bugfix/contacts into 4c5aed350518e11e5cabc5540049c2ca99347352 on ringcentral:master.


unable to clone repo

I am not able to clone this repo on my local rstudio, and I'm not sure why. It keeps loading and compressing objects, never finishing. Are there any heavy files that are too heavy for my weak computer to handle? Or am I doing something wrong? I managed to clone the other repos

I am not able to clone this repo on my local rstudio, and I'm not sure why. It keeps loading and compressing objects, never finishing. Are there any heavy files that are too heavy for my weak computer to handle? Or am I doing something wrong? I managed to clone the other repos

And now i get this error, but unable to find a solution

This is working perfectly fine on my side when I try a new clone. Try deleting the repo and re-cloning it with either Rstudio, Git-gui, or GitHub Desktop. And if you still don't manage you can always download the archive, unzip it where you want the project to be and open it with Rstudio. I'm closing the issue because this is to my mind a local computer issue but do not hesitate to reopen it if you still have difficulties.


Attention and Memory performance is worse

Thanks for making this repo available!
Just want to post that when the memory grate and write attention are used together, the performance seems to be worse.

@geyang Thanks for the results.
Does it mean there are some bugs in this repo?


Unable to modify NBT tags dynamically.

Issue Description:
Trying to modify NBT tags in a function recipe gives the response of "Not supported yet"
What happens:
The above mentioned error occurs and my recipe isn't created.
What you expected to happen:
I'd like my recipe to work. It's possible I'm doing it wrong or that the feature is not yet available.
Script used (Please Pastebin or gist your script, posting an unpasted or ungist'd script will automatically close this issue):
https://pastebin.com/fm4ugihp
Minetweaker.log file (Please Pastebin or gist your file, posting an unpasted or ungist'd file will automatically close this issue):
https://pastebin.com/2DNNVJ9U

Affected Versions (Do not use "latest"):

Minecraft: 1.10.2
Forge: <IP_ADDRESS>7
Crafttweaker: 3.0.26

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

Alright after pouring over the code for a while longer it looks like the intended syntax is used here: https://pastebin.com/VDkHE1gg
If I do that though I get "Error: Not a Valid lvalue"
Is there's something I'm missing that lets me declare a mutable NBT tag?

Alright so for posterity it appears that NBT tags are immutable once created the correct syntax is:
https://pastebin.com/h4PW69ji
That apparently crashes if souls is not filled but I'll file a separate bug for that.


reply not possible on fully shown note

In the thread view, reply buttons are only on the selected note and its replies but not on the - fully visible - note the selected note was a reply to. I often tried to reply and found it annoying to have to select the note first.

So this is the thread view, so any of my replies would be on a different thread and thus not shown? I think that's fair. If you fear people would reply multiple times with their replies not showing, maybe change the reply button to a show my reply thread button after a reply.
Also if the above screenshot is a thread, it should highlight the selected message somehow. As it looks now, the Vitor comment looks highlighted. Xitter and other nostr clients slightly scale up the selected comment. I assume the Amine comment should be highlighted, right?


Implement hardstops for spinningBodiesOneDOF

Tickets addressed: bsk-#637 (partially)
Review: by file
Merge strategy: squash and merge

Description
It is not uncommon for realistic actuators have a non-finite range of motion (ie, if they don't use slip-rings and have cable bundles running through them which cannot rotate, or if there would be mechanical interferences with other parts of the structure at certain angles). Modeling these limits can be important to the practical use of these actuators and associated bodies.
To increase the fidelity of the spinningBodyOneDOF, this PR adds (optional) angular limits to the model. Once theta reaches these locations, the spinningBodyOneDOF will not move further than the boundary.
These limits are set to +/-Inf by default, so that they have no effect unless the caller explicitly sets them to a smaller range.
Verification
Added unit tests verifying that the model respects various configurations of the limits. Existing tests verify that not specifying the limits does not impact default behavior of model.
Documentation
Updated rst file associated with model to document the existence/example usage of these limits.
Future work
Its not of interest for my work, but for high fidelity dynamics simulation it would be possible to model the interaction of the spinningBody with the hardstop (by specifying the mechanical stiffness/damping of the angular limit).
Also, it seems that raising an exception (to stop the program) is the proper response for an invalid configuration (rather than just logging an exception, which the user might ignore (either intentionally or accidentally) and get invalid results). I implemented the ability to propagate a C++ exception to python in the SWIG wrapper for this function. Perhaps it would make sense to generalize this exception handling to make it more widely available to other basilisk modules.

@joaogvcarneiro , could you please take a look at this PR to your module?  Thanks.

Howdy @ascended121 , just checking that you are still actively pursuing this PR?  Thanks for the quick feedback.  The requested edits are fairly minor.

Howdy @ascended121 , just a quick ping to see if you are still active on this branch?  If not, I can try to pull this branch and make the required fixes before pushing?  Thanks for the quick feedback.


Fix LicenseTest.testLicense. Fixes #13217

change execution order to have all the files ready for testing.
Fixes #13217

With the changes in the lifecycle, the built server is unable to start :(
I guess we only need to change the lifecycle for the generate-licenses-html execution. The other ones should remain the same.

ouch, let me see


How to make java.sql.Timestamp return String

java.sql.Timestamp will generate Timestamp interface even though add JsonFormat annotation, how can I specific to use string class

Should be fixed by https://github.com/kbuntrock/openapi-maven-plugin/pull/175
Meanwhile, you can customize the modelAssociations, as described in the documentation: https://kbuntrock.github.io/openapi-maven-plugin/#/configuration?id=modelsassociations