isAvailable success function not working

I'm using the latest version of Phonegap and the npm install of your plugin.
    window.plugins.touchid.isAvailable(
	function(){alert('available')}, // success handler: TouchID available
	function(){alert('not available')} // error handler: no TouchID available
);

The error message works correctly in the browser and on Android, but the success alert isn't firing on my iPhone 6s.
This is my first day working with Phonegap so I have a bit of a curve.

I tried @vagishmm solution of downgrading the platform, but this caused a number of other problems for me, so I reverted.
I'm really stuck here. isAvailable is an important check to do, but it never, ever fires the success handler, be it on the emulator or a real phone. Without it, the plugin can't be used. That's a problem, since most alternatives appear to be forks of this.
Any idea what's involved in converting to the new platform versions?
@EddyVerbruggen have you been able to recreate this at all?

I got it working by re-adding ./www as a reference to the XCode project. I have no idea why that helped, and I have no idea why all the other plugins worked and this one didn't. But it least it got me past this problem.

@CobusKruger can you clarify what you did more specifically?

where can I this code
window.plugins.touchid.isAvailable(
function(){alert('available')}, // success handler: TouchID available
function(){alert('not available')} // error handler: no TouchID available
);
means in which file


Hint text in text field should wrap

The hint text in a text field doesn't automatically wrap by default. The text field doesn't seem to have properties to wrap the hint text either.

Related StackOverflow question: https://github.com/flutter/flutter/issues/11197

/cc @dvdwasibi who worked on hintText in #7942


Feature: Add support for Neovim Buffer Update Deltas

Today, Oni syncs the entire buffer when not in insert mode (so that it has the latest buffer state for language services). Until #7917, that was the best we could do.
However, with #7917, we can now hook up to the nvim_buf_lines_event and nvim_buf_changedtick_event, and only get the lines that have changed. This will be very helpful for performance, especially when manipulating large files. And it means we can potentially simplify our update management - we fork our buffer update handling depending on whether we are in insert mode or not (in insert mode, we push up a single line, whereas in normal mode, we push everything) - with this change, we could simply send up whatever the delta is.
On the vim side, a good starting point for this to look at the current synchronization logic:
https://github.com/onivim/oni/blob/68d1f1f816dab516709626499ab62f39fb523135/vim/core/oni-core-interop/plugin/init.vim#L37
We should be able to replace that OniNotifyBufferUpdate with the new update-delta events. In addition, we should be able to scrap this augroup here:
https://github.com/onivim/oni/blob/68d1f1f816dab516709626499ab62f39fb523135/vim/core/oni-core-interop/plugin/init.vim#L144
On the Oni side, we handle these updates here:
https://github.com/onivim/oni/blob/68d1f1f816dab516709626499ab62f39fb523135/browser/src/neovim/NeovimBufferUpdateManager.ts#L36
We should be able to simplify this - instead of splitting between notifyFullBufferUpdate and notifyIncrementalBufferUpdate, we can simply do a notifyDeltaBufferUpdate.
It'd be great to have some tests here that cover this surface area (when we get a delta update from Neovim, we incorporate that into our model we send to LSPs). And of course, we need to make sure it continues working with our set of LSPs.
Another good point of reference as an example would be the commit that @lunixbochs incorporated into ActualVim: https://github.com/lunixbochs/ActualVim/commit/414e8d4c3feb4c798a611c14ef47741e8b952146

It might be a bit before I get time to tackle this, so if anyone would like to take this on in the meantime, that'd be amazing! ❤  💯
A pre-requisite too is upgrading to the latest version of Neovim, but to test locally, we could use the debug.neovimPath configuration setting to grab a newer version of Neovim and override the bundled one Oni uses.

The Neovim node client also has a PR for this which is potentially useful as an example : https://github.com/neovim/node-client/pull/81


What about api?

nice app, but i am wondering if it has api code as well? would be great if you share API's or dashboard of somekind, would be great!

@GSTJ

Hey @Arvrairobo, this project only uses mocked data from a mock endpoint. The main focus was to display a nice front-end in React-Native.
I'm working on a spin-off called "Pegada," a dating app for dogs, which works 100% with a real backend. It's a closed repo, but I'll consider opening in the future.
The app is available in Brazil; I'm working on some big changes and should make it available worldwide soon.
https://pegada.app/

@GSTJ Hello! I want to recommend this app in China, can you sell it to me?


Build the editor extensions in Devlab9|Any CPU

This is the configuration and platform used by buildCC for releases.

Awesome! :100:


Entertainment api Coreelec

@SJunkies So I managed to install hyperion.ng as a service on Coreelec but without the entertainment-api. I grabbed a release for amlogic https://github.com/hyperion-project/hyperion.ng/releases and unzipped it with this command: curl -# -L --get <release_url> | tar --strip-components=1 -C /storage share/hyperion -xz
What I don't understand is that the release file has 2.0.0alpha4(now 5 just got updated) in the title but when extracted and booted up it says version 2.0.0.
At first I thought it was an issue with their pre compiled binaries so I cross compiled your entertainment branch on a debian laptop in docker for my amlogic platform. But it has the same issue. When I look into the build files it does include entertainment api in the code but when I start the service with the extracted binaries entertainment api is nowhere to be found in the web ui. And it says version 2.0.0 instead of 2.0.0alpha4.

@Rickexe
Sorry for the really late reply!
It sounds like you downloaded the right version, but it starts only the integrated version that comes with Coreelec!
BUT: The entertainment API is ready and will be merged (i hope) into the master branch soon! ;)
So there is a high chance, that the next Coreelec release could use / update to the next hyperion.ng version with the entertainment-api included! 🤞

That's fantastic news!!! I am soooo looking forward to it. I will finally get to see the full potential of my ambilight setup.

Do long do you think it will take? Are we talking in weeks or months...?

How long do you think it will take? Are we talking in weeks or months...?

I hope the merge in the master branch will take place in the next few days.

Oooh nice. Keep at it!

Hey, from what I can understand from the commits the api is now combined with the master branch?
Awesome work! Hopefully they will soon release a coreele version of it.


fix typo change "databaseaccount" to "mongocluster"

Database account are not the resource that creates the mongoDB vCore cluster it's for mongoDB only. The right one as mentioned in the bicep above at line 47 is mongocluster.

#sign-off

@gahl-levy

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

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

#sign-off


login to zoom with email and start an instant meeting

Describe the solution you'd like
I would like to be able to login to zoom with email and start an instant meeting, also it would be great if we can get the meeting id and pwd after starting the meeting.

This is a feature I would love to have.


Feature request: Add bookmarking conversations feature

Most of the conversations would be disposable, but I want to keep some of them for later follow ups, and it's easier to do so if I can bookmark them. Bookmarked conversations should bubble up to the top of the conversation list.

Good idea!

Please lmk if you have any plans to introduce this feature in the near future. If you're swamped with other tasks, no worries at all! I'd be more than happy to give it a shot and potentially create a PR for it later on, although I can't make any promises 😊


docker networks instead of links

docker 1.9 introduces the new docker networking and at the same time discourages docker links:
https://blog.docker.com/2015/11/docker-multi-host-networking-ga/
https://docs.docker.com/engine/userguide/networking/default_network/dockerlinks/
Do you have plans to support the feature? It would be interesting because it easily allows two-way-communication.

yes, i'm sure at some point it will be added. pull requests are always welcome.

Indeed we want to add new features introduced with Docker 1.9, although not quite clear how to do this for docker network yet.
The 0.14.x series is dedicated for adding new Docker 1.9 features. We will also have to think about how to handle backwards compatibility.

Network links are now supported for 0.14.0 with the new <net> configuration option. The network specified must exist already, though.
Might be a good idea to create a network on the fly if requested and not existant ?

Would be perfect if PropertyConfigHandler was also extended for this feature.

good point, added this, too.


Move storage to an abstract contract PRBProxyStorage

The idea is to move the storage to a dedicated abstract contract so that users can inherit it in their own target contracts.
As suggested by @SolcAndMe here.

Done.
https://github.com/PaulRBerg/prb-proxy/commit/1449dac4456820b6fa209750f40cbf6b1d3726f9


hosts.yml ranges

Q
A

Issue Type
Bug / Feature Request

Deployer Version
6

Local Machine OS
OS X 10.13

Remote Machine OS
Redhat 7

Description
Tried to add ranges to the host or hostname in the hosts.yml file
Not sure if this a bug or feature request but likely the latter
Steps to reproduce
production:
  hostname: www[01-05].example.com
  stage: production

Content of deploy.php

inventory('hosts.yml');

Output log
  The command "rm -f /var/www/example/.dep/deploy.lock" failed.
  Exit Code: -1 (Unknown error)
  Host Name: www[01:05].example.com
  ================
  ssh: Could not resolve hostname www[01:05].example.com: nodename nor servname provided, or not known

Oh this change looks promising https://github.com/deployphp/deployer/pull/1541

Will fix it.

Fixed.

Thanks @antonmedv
Should this work as well with the hostname key?
production:
  hostname: www[01-05].example.com
  stage: production

I gave it a whirl and doesn't seem to work with hostname key. But a big thanks, that was a fast fix!

I can work around that, thanks again for the fix


Please update hex package

Can you please push a new version to hex so we can use with phoenix 1.3. Thanks...

@smpallen99 done 👍

Thanks!! BTW, I just started using slim. I was amazed that, given my understanding for haml, that I was immediately productive. Seems like you have a lot more robust parser than we do in Calliope. When I get some time, I'm going to see If I can use your parser design to overhaul Calliope. However, right now I'm thinking I may like slim over haml :)

Thanks for the kind words @smpallen99!  Glad you were able to get up and running so quickly.  Huge thanks to @Rakoth who completely overhauled the parser to make it even better.


auto deregistry legacy servcie from consul make no effect

Describe the bug
could not deregistry the service from consul
spring cloud consul version: 2.2.7.RELEASE
Sample
this is my application.properties config:
# registry center
spring.cloud.consul.host=consul-1630121482-headless.reddwarf-pro.svc.cluster.local
spring.cloud.consule.port=8500
spring.cloud.consul.enabled=true
spring.cloud.consul.discovery.health-check-interval=15s
spring.cloud.consul.discovery.instance-id= ${spring.application.name}:${vcap.application.instance_id:${spring.application.instance_id:${random.value}}}
spring.cloud.consul.discovery.prefer-ip-address= true
spring.cloud.consul.discovery.deregister=false
# service check failed, delete service
spring.cloud.consul.discovery.health-check-critical-timeout=30s
spring.cloud.consul.discovery.heartbeat.enabled=true

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Compilation issue about weightOnlyBatchedGemvBs4Int8b.cu

When I follow the build instructions in your Readme to build TensorRT-LLM, it always gets stuck at 98%. The background compilation-related program status is R+ and S+.

I also encounter this problem when building the image.
nvidia-smi

nvcc
$ nvcc -V nvcc: NVIDIA (R) Cuda compiler driver Copyright (c) 2005-2023 NVIDIA Corporation Built on Tue_Jul_11_02:20:44_PDT_2023 Cuda compilation tools, release 12.2, V12.2.128 Build cuda_12.2.r12.2/compiler.33053471_0
pip list
Package                 Version

boltons                 23.0.0
brotlipy                0.7.0
certifi                 2023.5.7
cffi                    1.15.1
charset-normalizer      2.0.4
conda                   23.5.2
conda-content-trust     0.1.3
conda-libmamba-solver   23.5.0
conda-package-handling  2.1.0
conda_package_streaming 0.8.0
cryptography            41.0.4
filelock                3.12.4
fsspec                  2023.9.2
huggingface-hub         0.17.2
idna                    3.4
jsonpatch               1.32
jsonpointer             2.1
libmambapy              1.4.1
packaging               23.0
pip                     23.1.2
pluggy                  1.0.0
pycosat                 0.6.4
pycparser               2.21
pyOpenSSL               23.2.0
PySocks                 1.7.1
PyYAML                  6.0.1
requests                2.29.0
ruamel.yaml             0.17.21
setuptools              67.8.0
six                     1.16.0
toolz                   0.12.0
tqdm                    4.65.0
typing_extensions       4.8.0
urllib3                 1.26.16
wheel                   0.38.4
zstandard               0.19.0
cmake --version
cmake version 3.16.3
CMake suite maintained and supported by Kitware (kitware.com/cmake).

The issue with the compilation of weightOnlyBatchedGemvBs4Int8b.cu has been resolved. It was due to the compilation of weightOnlyBatchedGemvBs4Int8b.cu is too slow. However, there were some issues that occurred later.

The issue with the compilation of weightOnlyBatchedGemvBs4Int8b.cu has been resolved. It was due to the compilation of weightOnlyBatchedGemvBs4Int8b.cu is too slow. However, there were some issues that occurred later.

Pls try with this to ensure the library are properly fetched to your local storage via LFS.
https://github.com/NVIDIA/TensorRT-LLM/issues/100#issuecomment-1777104351


Add SkaffoldYamlGenerator

Fixes #10

If we ever plan on using a library to generate this in the future. We should just do that in this library. If that seems overkill for what this code is supposed to do, then this is okay.

anybody know if Skaffold has skaffold.yaml generating feature on the roadmap?

@patflynn Yep - we discussed this during the skaffold weekly @GoogleContainerTools/container-tools I believe @nkubala has some plans about it.

@patflynn @coollog yep I'm working on this right now, should have something to show soon


Getting "Refused to get unsafe header" errors when use in Chrome

I'm getting the following errors:
Refused to get unsafe header "Transfer-Encoding"
xhr.onreadystatechange
Refused to get unsafe header "X-Firefox-Spdy"
xhr.onreadystatechange

The are coming from process lib that's involved via xhr:
xhr.getResponseHeader('X-Firefox-Spdy')
xhr.getResponseHeader('Transfer-Encoding')

@mrkam This appears to be a CORS configuration issue. Please check if the endpoint has the CORS headers configured appropriately. The endpoint should expose the Transfer-Encoding value in the Access-Control-Expose-Headers header. Please check this stackoverflow thread for more details.

Yes, CORS is the reason but I've tried other approaches like chunked-request or direct use of XMLHttpRequest and all of them didn't have this problem. Although, I can request Transfer-Encoding header to be provided but what about the other one? "X-Firefox-Spdy"? Its name sounds quite random to me.

You're right, exposing Transfer-Encoding fixed the issue


kittiviewer:error: load kitti info fail, please check your backend!

Hello, your work has helped me a lot, but I can't load data when using kittiviewer. The error is as follows:

At the same time, the following information is obtained on the server side:

The abckend.py program does not seem to respond:

@traveller59

Got the same
I changed backend port to 16666 and it works

Got the same
I changed backend port to 16666 and it works
Thank you for your reply. But my problem still exists. I think this may be a problem that has nothing to do with the port. My running process is as follows:

Run backend.py on the server: python ./kittiviewer/backend/main.py main
Launch a local web server on the server: cd ./kittiviewer/frontend && python -m http.server
Since I want to see the effect in a local browser, I use the SSH -L command:
Ssh -L 16666:<IP_ADDRESS>:8000<EMAIL_ADDRESS>for listening to the local web server.
Finally I got the web interface but couldn't load the data.
I think the problem lies in the following two sentences:

@traveller59

Got the same.
I change the code in the main function of /backend/main.py
"app.run(host='<IP_ADDRESS>', threaded=True, port=port)" to "app.run(host='<IP_ADDRESS>', threaded=True, port=port)", and I type "server ip: port" not "<IP_ADDRESS>:port" into the "backend" box.
It works.

Try to use the same method, but get 'Error code: 404. Message: File not found.' after pressing F12. Any idea what I did wrong? Thanks @kerwinair

Did you solve the problem? @LittleYuanzi

@traveller59 Hi, There is no GUI in my server. So I try to open "Kitti Viewer Web" on another computer. However, Nothing happened and I got these errors :

Try to use the same method, but get 'Error code: 404. Message: File not found.' after pressing F12. Any idea what I did wrong? Thanks @kerwinair

hello ,How  do you solve it?I see the same problem

any solution?

Got the same
I changed backend port to 16666 and it works
Thank you for your reply. But my problem still exists. I think this may be a problem that has nothing to do with the port. My running process is as follows:

Run backend.py on the server: python ./kittiviewer/backend/main.py main
Launch a local web server on the server: cd ./kittiviewer/frontend && python -m http.server
Since I want to see the effect in a local browser, I use the SSH -L command:
Ssh -L 16666:<IP_ADDRESS>:8000<EMAIL_ADDRESS>for listening to the local web server.
Finally I got the web interface but couldn't load the data.
I think the problem lies in the following two sentences:

@

Got the same
I changed backend port to 16666 and it works
Thank you for your reply. But my problem still exists. I think this may be a problem that has nothing to do with the port. My running process is as follows:

Run backend.py on the server: python ./kittiviewer/backend/main.py main
Launch a local web server on the server: cd ./kittiviewer/frontend && python -m http.server
Since I want to see the effect in a local browser, I use the SSH -L command:
Ssh -L 16666:<IP_ADDRESS>:8000<EMAIL_ADDRESS>for listening to the local web server.
Finally I got the web interface but couldn't load the data.
I think the problem lies in the following two sentences:

@

@tyjiang1997 , Have you solve this problem?

@tyjiang1997 , Have you solve this problem?

@traveller59 Hi, There is no GUI in my server. So I try to open "Kitti Viewer Web" on another computer. However, Nothing happened and I got these errors :

@dongqiaqia Hey, How did you solve this problem? I am facing the same problem.


Gif support

Can you export gif like recordit app?

Hey, closing this as it's a dupe of https://github.com/wulkano/kap/issues/14.
It's in the works and should be landing soon! Thanks for your patience. 👌

@doot0 thanks and congrats, this app looks very nice

@sirgalleto Thanks boss! 🖖

Ops, closed by accident 😩


Fixing CI for macOS run

Running the script ./setup-environment.sh on the macOS side will always fail due to main and the current branch having different files.
Commented it out for now when I have some time to address it but won't cause any problems

trying to review
meanwhile, could you resolve the conflict?

yessir this is nice. can we also remove the submodule hash bump from this PR?

It should be fine @MaySoMusician , I see that the submodule is included- is that going to be a problem?

@nang-dev could you resolve conflicts again and approve this?

fixed the submodule. the diff is still there but i changed it to point to the correct hash. the submodule is same as main now.

hm i wonder why its still failing here #159


BCJSSE: client call - error after renegotiation

Hi,
in the master is added commit 18e1c3106bcfab7f646f993f56027fea7af24357 with acceptance of Renegotiation. I'm happy about that, cause I'm missing this functionality in one of our projects.
I tested this functionality during a call of server with a client certificate (tls 1.2 on Java 7) and it fixed the issue described here: https://github.com/bcgit/bc-java/issues/593#issuecomment-533518845
-->
ProvTlsClient - Client raised warning(1) no_renegotiation(100) alert: Renegotiation not supported`

But the communication still ends with:
ProvTlsClient notifyAlertRaised
WARNING: Client raised fatal(2) internal_error(80) alert: Failed to read record'.

I sought the reason and I found that the error is caused by validation in the completeHandshake() method in the TlsProtocol class. There is validation on 'appDataReady' if is this property set on true then the method throws an exception otherwise application sets the property on true and continues. The reason for the error is, that during the renegotiation is method completeHandshake() is called twice.
I tried to fix this issue by improving the appDataReady check about the ignore of this check during renegotiation, but I'm not sure if it is the correct way, to solve it.
After this update communication starts to work correctly.
Original check:
if (appDataReady ||
                    null == securityParameters.getLocalVerifyData() ||
                    null == securityParameters.getPeerVerifyData())
            {
                throw new TlsFatalAlert(AlertDescription.internal_error);
            }

Updated check:
if ((!(RenegotiationPolicy.ACCEPT == getRenegotiationPolicy() && securityParameters.isRenegotiating()) && appDataReady) ||
                null == securityParameters.getLocalVerifyData() ||
                null == securityParameters.getPeerVerifyData())
            {
                throw new TlsFatalAlert(AlertDescription.internal_error);
            }

Please, is it possible to check if is this solution correct? Otherwise, is possible to guide me in the right direction of a solution?
Kindly regards,
Vlastimil

I agree there is a bug here and your fix correctly addresses the immediate problem, although I will have to review it more fully.

@vjinochProfinit Are you able to confirm that this issue is fixed by the 1.70 release?

@peterdettman Cause issue probably was fixed, can this ticket by closed?


Upgrade CloudRun to v2

Closes https://github.com/piiano/vault/issues/3853

Migrate CloudRun resources to v2
Add startup and liveness probe
New variables to configure scaling
Automatically default to a zone for the bastion and Cloud SQL
Allow CloudSQL CMEK key to be set by variable, disable CMEK by default

Not possible to disable/migrate away from CMEK for existing installs without recreating the instance
Leaving KMS in-place for now to avoid destroying the key, to be removed in the future
KMS key field in resource has no effect after install

Upgrade google provider to v5
Upgrade CloudSQL module to latest
Upgrade VPC module to latest
Upgrade CloudSQL Postgres version to v15
Upgrade bastion OS to cos-113-lts, automatically pull newer images
Merge static CloudRun env vars with pvault_env_vars
Overall linting and cleanup

Deployed and tested in the sandbox.
@fs-dev1-vm-pvault-bastion ~ $ pvault selftest basic
Checking vault readiness... OK
Checking version... <IP_ADDRESS>04655447-g48d6d17, OK
Checking KMS configuration... KMS type: GCP, OK
Checking test collection recreation... OK
Checking encrypt and decrypt object... OK
Checking Object read/write... OK

Total warnings:  0

INF Command completed successfully

@horjulf Should we add a VERSION file or some other way to indicate that we made breaking changes?

@horjulf Should we add a VERSION file or some other way to indicate that we made breaking changes?

@nirhaas Good point, added some upgrade notes and a VERSION.
Can revisit later on better format and versioning system to publish the modules.


Vue Js warning "The el option shouild be a function that returns a per instance.."

Hi,
I have this warning
   vue.common.js?e881:1137[Vue warn]: The "el" option should be a function that returns a per-instance     value in component definitions.

Any ides how to remove this warning? My app works fine, just want to remove warning in console
       require('./packages');
       require('./components');
       require('./routes');

      const app = Vue.extend({
          el: '#app',
       });

Are you using the app variable later as new app()?
If not, you can simply change it to
const app = new Vue({
  el: '#app',
});

Only pass el at instantiation time, not extension time.


pytest filterwarning set too strict? (currently as error) causing the test suite failure with Python 3.14

I try to build blinker as RPM for Fedora Linux with Python 3.14.0a1.
The test suite fails because of a DeprecationWarning emited by asyncio.iscoroutinefunction which is used by pytest_asyncio dependency of blinker.
I suspect the error happens because of the strict filter applied to the warnings in the blinker's configuration: https://github.com/pallets-eco/blinker/blob/main/pyproject.toml#L40C1-L40C15
The traceback:
___________________ ERROR collecting tests/test_utilities.py ___________________
/usr/lib/python3.14/site-packages/pytest_asyncio/plugin.py:540: in pytest_pycollect_makeitem_convert_async_functions_to_subclass
    ] = hook_result.get_result()
/usr/lib/python3.14/site-packages/pytest_asyncio/plugin.py:522: in pytest_pycollect_makeitem_preprocess_async_fixtures
    _preprocess_async_fixtures(collector, _HOLDER)
/usr/lib/python3.14/site-packages/pytest_asyncio/plugin.py:213: in _preprocess_async_fixtures
    if fixturedef in processed_fixturedefs or not _is_coroutine_or_asyncgen(
/usr/lib/python3.14/site-packages/pytest_asyncio/plugin.py:169: in _is_coroutine_or_asyncgen
    return asyncio.iscoroutinefunction(obj) or inspect.isasyncgenfunction(obj)
/usr/lib64/python3.14/asyncio/coroutines.py:23: in iscoroutinefunction
    warnings._deprecated("asyncio.iscoroutinefunction",
/usr/lib64/python3.14/warnings.py:668: in _deprecated
    warn(msg, DeprecationWarning, stacklevel=3)
E   DeprecationWarning: 'asyncio.iscoroutinefunction' is deprecated and slated for removal in Python 3.16; use inspect.iscoroutinefunction() instead

During handling of the above exception, another exception occurred:
/usr/lib/python3.14/site-packages/pytest_asyncio/plugin.py:652: in _patched_collect
    return collector.__original_collect()
/usr/lib/python3.14/site-packages/_pytest/python.py:552: in collect
    return super().collect()
/usr/lib/python3.14/site-packages/_pytest/python.py:419: in collect
    res = ihook.pytest_pycollect_makeitem(
/usr/lib/python3.14/site-packages/pluggy/_hooks.py:513: in __call__
    return self._hookexec(self.name, self._hookimpls.copy(), kwargs, firstresult)
/usr/lib/python3.14/site-packages/pluggy/_manager.py:120: in _hookexec
    return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
/usr/lib/python3.14/site-packages/pluggy/_callers.py:50: in _warn_teardown_exception
    warnings.warn(PluggyTeardownRaisedWarning(msg), stacklevel=5)
E   pluggy.PluggyTeardownRaisedWarning: A plugin raised an exception during an old-style hookwrapper teardown.
E   Plugin: asyncio, Hook: pytest_pycollect_makeitem
E   DeprecationWarning: 'asyncio.iscoroutinefunction' is deprecated and slated for removal in Python 3.16; use inspect.iscoroutinefunction() instead
E   For more information see https://pluggy.readthedocs.io/en/stable/api_reference.html#pluggy.PluggyTeardownRaisedWarning
=========================== short test summary info ============================
ERROR tests/test_context.py - pluggy.PluggyTeardownRaisedWarning: A plugin ra...
ERROR tests/test_signals.py - pluggy.PluggyTeardownRaisedWarning: A plugin ra...
ERROR tests/test_utilities.py - pluggy.PluggyTeardownRaisedWarning: A plugin ...
!!!!!!!!!!!!!!!!!!! Interrupted: 3 errors during collection !!!!!!!!!!!!!!!!!!!!
============================== 3 errors in 0.49s ===============================

Python version: 3.14.0a1
Blinker version: 1.7.0

The setting is correct. It's good practice to treat warnings as errors in tests, so they can be detected early and fixed. We do not support Python 3.14 yet. Once it is in beta and available on GitHub CI, we can start testing it and fixing any compatibility issues.

That said, happy to review a PR early to add compatibility around any known deprecations.


Update WicketComponentExtensions.java

Set header value for 'Content-Security-Policy'

I will merge but extract to another method.


what is full name of rsu?

what does map_rsu.proto define, what is full name of rsu
thx

It is used for V2X communication, the full name is roadside units (RSUs).
The device paired with it is in the car, called platform for onboard (OBUs)
(road side)RSU <-> OBU(car side)

It is used for V2X communication, the full name is roadside units (RSUs).
The device paired with it is in the car, called platform for onboard (OBUs)
(road side)RSU <-> OBU(car side)

thanks


allow to renew credential based on userName

Fixes #9544
Status
ready
Description

if username is given in proxycfg, use it also for logonRenewMyProxy()
removes some fake error logging from vomsExtensionRenewal
Those functions are only used  by CRAB. You can safely merge and kindly add in next tag so that we can build a new TaskWorker with this and test in preprod.
I tested already in my VM.

Is it backward compatible (if not, which system it affects?)
YES
Related PRs
#9548 was a partial fix for same issue
#9577 was an earlier attempt which failed to properly rebase and has some discussions of the changes

from my side this this ready to merge. I tested it again in CRAB.

@belforte I have just created a new tag with this fix (and the UserFileCache one) in, please use:
1.3.2.pre6

thanks Alan


Add deb builder for ArduPilot version

add a simple way to create a deb file for the generator
also add a release build workflow. To trigger it, just create a github release with a tag and it will build the deb and put it as release artefact

Nice! Should this go upstream? They already build debs.

@khancyr Got any time to finish this up? It's really great.

A great next step would be to put this on an apt repo so we can have people apt install ardupilot-microxrceddsge


Download csv spreadsheet summarizing detections per hour and day

Soundscope needs to have the ability for the user to download a csv spreadsheet summarizing the number of detections for a given time frame.
SoundScope is already creating these summary tables for the hourly and daily visualization tabs. So, implementing this would consist of dowloading the daily and hourly pandas dataframes.
The csv spreadsheet needs to include the following metadata:

date of creation of the csv file
confidence threshold used
class label used (i.e. species/call type code)
name of user who generated the csv file
time zone (relative to UTC)

The csv file needs to have at least the follwoing columns:

Date in ISO 8601 compliant date/time format:  YYYYMMDDTHHMMSS
Number of detections

@namariD and Genevieve, you mentioned that the csv file should be compatible with other tools and databases like Makara and PACM. Could you please indicate what other columns you need in the csv file and in what format ?

After discussion with Annamaria dn Gen, the csv file should also include the following:

Date/time relative to the time zone of the recordings
Date/Time  relative to the analysis time zone (to be defined by the user when loading data to SoundScope)

Header should include:

Time zone of the recordings (i.e. UTC+ hour offset)
Time zone of the analysis (i.e. UTC+ hour offset)
version of SoundScope used

From Gen:
It would be helpful having the following columns:
-start date/time
-end date/time
-number of detections
-"Manual_Review"
-"Comments"
Exemple of spreadsheet currently used:
NAVFAC_NC_201710_HAT03B_RIWH_DAILY_GMT_MM.csv

Here is an example dolphin spreadsheet. The columns listed by Gen work well for us as well.
NEFSC_MA-RI_202011_COX01_dolphins_hourly.xlsx
All odontocete analysis is done in the timezone of the soundfiles and is hourly presence so we can adapt to whatever timezone is needed

In October, we will revisit this to have a template based on what the postgres database needs

Resolved in version 20241202

CSV button added to v20241202 to download daily and hourly detection summaries as csv files


Adding failover capability for vRO shim

As discussed with John and Alan via email.

@tkopton are you planning to fix the failing tests?

@alanjcastonguay Generally, yes. These is my very first try to contribute to a GitHub Python project / Travis etc. I suppose this is the reason:
E       UnboundLocalError: local variable 'r' referenced before assignment
loginsightwebhookdemo/init.py:243: UnboundLocalError
Let me check it. Thanks.

This is not a good idea. You've modified callapi with extra arguments. Instead, modify the caller to iterate over a list (array) and call callapi multiple times, once with each url until it succeeds.

As agreed, closing the request and changing the way how this feature might be implemented.

Would it be better to push this back onto architecture of vRO?  The shim should not have to monitor the state of vRO in my opinion.  vRO should be built behind a LB in the case where availability is required.  If anything, a queue mechanism would be helpful for the shim in the event an endpoint is down or unavailable.


[HOLD] Show empty policyExpenseChats in LHN just like defaultRooms

@neil-marcellini, please review when you get the chance
Details

Fixed Issues
Related to https://github.com/Expensify/Expensify/issues/201621
and follow up slack discussion here
Tests
Same as Web QA but done locally
Added tests to tests/unit/OptionsListUtilsTest.js to verify the latest changes to this logic works.

[x] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (PR Author) Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there’s a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained “why” the code was doing something instead of only explaining “what” the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named “index.js”. All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose and it is

[x] If a new CSS style is added I verified that:

[ ] A similar style doesn’t already exist
[ ] The style can’t be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.

PR Reviewer Checklist

[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

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

[ ] I verified there are no console errors (if there’s a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained “why” the code was doing something instead of only explaining “what” the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named “index.js”. All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn’t already exist
[ ] The style can’t be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.

QA Steps
This should be done on staging. No failure case needed since this is in beta, if this is not working let @TomatoToaster know so it can be fixed in a followup PR

Log into an account that doesn't have a workspace yet and create a new workspace.
Verify that you immediately see the workspace chat in the left hand nav
Go to Workspace Settings -> Manage Members and invite a user to the workspace
Verify that you immediately see another workspace chat in the left hand nav (this one should have the new user's name)

[x] Verify that no errors appear in the JS console

Screenshots

Web

Mobile Web

Desktop
(Looks same as Web)
iOS
(ignore the missing images in some workspace chats that's buggy and will be resolved with this issue)

Android

@neil-marcellini oh nice Neil you have some context on this already since you made the original change where this happens for defaultRooms.

Hold on, I realize now I gotta rework these tests a little bit.

Hold on, I realize now I gotta rework these tests a little bit.

Ah ok, I'll review again later.

Ok @neil-marcellini this should be good to review now, I've made the tests separate from the general reports since they aren't relevant to testing empty policyExpenseChats or defaultRooms in the sidebar.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @neil-marcellini in version: 1.1.56-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @francoisl in version: 1.1.56-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


bounding box

How could i get the bounding box of one point cloud? Thanks.
Shouyang

@lsymuyu, do you mean selection of ROI?

If you're asking for axis aligned bounding box, you could use numpy functions:
xyz = point_cloud.xyz  # this is a numpy array
print(np.min(xyz, axis=0))  # The lower left corner
print(np.max(xyz, axis=0))  # The upper right one

Is this what you meant?


Fix handling of control flow instructions in convert_to_target()

Summary
This commit fixes an issue in the convert_to_target() function where it wasn't looking for control flow instructions in the proper location. Typically the control flow instructions are put in the supported_instructions field of the BackendConfiguration, but the convert_to_target() function was ignoring this field. This commit updates the function to check supported_instructions for the control flow instructions. It doesn't more broadly look at the supported_instructions field, because on other backends this field is used to list the supported pulse instructions which might have name conflicts with other instructions and cause issues building a target.
Details and comments
Fixes #11872

@Mergifyio backport stable/0.46

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 89.302%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
5
92.19%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
58908

Relevant Lines:
65965

💛  - Coveralls


”Comandău” is listed in the drop-down menu

Description:
"Comandău" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Comandău" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Comandău, COVASNA (Comandău)".


Update gradients.json

I added an orange gradient based on the Html5 logo

Thanks for contributing 👍

Thanks!


Not holding storage owner type variable raises destoryed_error

In current codebase, without holding storage reference itself, it raise destroyed_error (Segmentation Fault).
To be honest, I'm not sure how I should handle this issue correctly.
Without handling this correctly, this library's users are likely to experience big confusion.
Any help or ideas would be greatly appreciated.
https://github.com/lablup/rraft-py/blob/main/harness/tests/integration_cases/test_raft.py#L287

This also should be fixed in #34.


Make npm ready

This is to address #44. All thats needed with this change is to run an npm publish on the repo. I've done this with v1.0.0 so now you can npm install -S leaflet.animatedmarker

why is this not getting merged?


Show GApps Browser in Share menu

GApps Browser is not shown in the Share menu, at least when I press the "Share" button in NewPipe.
This could come in handy for editing playlists in YouTube.
I guess it doesn't work from any other app, neither.

Please see #55


Fixed Repo.move() and added test

Repo.move() previously was broken and did not work as intended; this is a fix for this issue.

Deleted private function Repo._updateTree()
Updated Repo.commit() docs: as per https://developer.github.com/v3/git/commits/#create-a-commit, arg tree is a SHA string previously created (via Repo.createTree()), not the tree object itself.
Changed Repo.updateHead() API: added force arg to match Github API docs (https://developer.github.com/v3/git/refs/#update-a-reference).
Removed internal reference to deprecated Repo.updateTree(): previously referenced in Repo.move(); this has been addressed.
Added Repo.move() test: assert moved file has same contents, assert old path does not exist after update.

Should squash the commits when merging.

Thanks @cripplet. I'll get this out today or tomorrow.
Note to me: this is a breaking change.

@cripplet This is released in v2.0 which is building now


Firefox: screen recording broken. Error message: "DOMException: The object can not be found here."

Description
On Firefox the "official" demo https://collab-project.github.io/videojs-record/demo/screen-only.html doesn't work. After selecting either a window or the screen to share it fails with the following error message:
"DOMException: The object can not be found here."
Steps to reproduce
Go to demo:

https://collab-project.github.io/videojs-record/demo/screen-only.html
Select either a window or the screen to share
Browser dev console shows error

Results
Expected
It should start recording a screen cast.
Actual
It doesn't continue and shows an error
Error output
Browser dev console shows error:
DOMException: The object can not be found here.
Additional Information
Please include any additional information necessary here. Including the following:
versions
VIDEOJS: Using video.js 7.20.1 with videojs-record 4.6.0 and recordrtc 5.6.2
videojs
what version of videojs does this occur with?
7.20.1 (see demo)
browsers
Firefox 110.01 (64.bit)
OSes
MacOS

I tested it on Windows with Firefox 123.0b9 (64-bits) just now and it works fine here.


Support for auto formatting

Currently the macro doesn't get formatted with cargo fmt or rust-analyzer, is that possible to add to enable consistent correct formatting of these structs?

I don't think so since it is technically arbitrary syntax. It might be possible to do this with an IDE extension, however I am not currently developing something like that.
This might be an interesting edition to rustfmt. Perhaps a with --macro-fmt flag.


Can't add file type metadata to a Jupyter Notebook file

I tried to add file type metadata to a Jupyter Notebook file in a composite resource. I clicked on "Add metadata" button and have the following error. Does HydroShare support to add file type metadata to a Jupyter Notebook file? Thanks!
Resource link: https://www.hydroshare.org/resource/299c7d218c4e4594bfff573139d3d9b2/

This appears to be UX/front end related.  In testing I found

I can right click or click on the Add metadata button on a .ipynb file (or text file and presumably other files)
I get the message that content type creation was successful.  However the interface still shows "no metadata exists for this file".  If I click Add metadata a second time, I get the error Tian reported.
If I reload the page with the browser I am able to see and work with the content aggregation metadata.

This may be a regression introduced with efforts to make page loading more efficient (i.e. limit reloads).  @Maurier please check on this.  If this appears to be back end, @pkdash please check.

Undertood, I'll have a look.

@dtarb and @Maurier It is a UI issue. For certain type of aggregation creation, the file browsing area is not getting refreshed. Once the browsing area gets refreshed I am assuming the button to add metadata will not appear or get disabled.

@gantian127 Metadata can be added to any single file if it's not already part of an aggregation (except for fileset aggregation).

This issue is also fixed by https://github.com/hydroshare/hydroshare/pull/3677

Fix confirmed working on beta. Closing.


double_float move forward with formatted output streaming

Hello @ckormanyos @cosurgi
Standard-compliant formatted output streaming is now almost complete. However, one minor inconvenience/problem remains. In rare cases, cpp_double_float<float> does not round the given number accurately to the required precision. For example, for the number 4.933775e+04, using the std::ios::fixed flag and at a precision of 1, we have the following discrepancy:
expected: '49337.8' vs actual: '49337.7'
After extracting all the digits as done in this loop, f_prime in theory should be exactly equal to 5 (since that is the last left digit), but in practice, a small error term is observed. My guess explanation is that the division performed on line 658 is not exact.

Hi @sinandredemption thanks for moving forward on this.
I will look into the test case you mention. It seems like a probable edge case. The actual value of std::numeric_limits>::digits10 is 6, whereas max_digits10 is 9. We can look at the exact way that particular float value is stored in binary representation.
Rounding and floating-point representation is something that is generally difficult and ultimately, we might need a little help if we can't crack this case. I'll take a look soon and report back.

My guess explanation is that the division performed on line 658 is not exact.

At first I was thinking that maybe the division in that line
 auto f_prime = (f > FloatingPointType(0) ? f : -f) / cpp_double_float<FloatingPointType>::pow10(exp10);

happens between FloatingPointType/cpp_double_float<FloatingPointType> and the solution would be simple: just use a higher numerator precision cpp_double_float<FloatingPointType>. But this is not the case. Numerator appears to be const cpp_double_float<FloatingPointType>& f.
This however brings an extremely simple solution. We are doing stringstream conversions here, they are not supposed to be super fast. We could use cpp_double_float<cpp_double_float<FloatingPointType>> in the numerator and eliminate all problems altogether. This could work even without bringing in the dependency on cpp_quad_float<FloatingPointType> because by design we expect the cpp_double_float<cpp_double_float<…>> to work correctly if it is given itself as an argument.

Hello @cosurgi
Yes. That could be possible. The next thing I am going to work on is making sure, through robust testing, that the cascade construct `cpp_double_float<cpp_double_float<>> works just as intended.
However, another possible solution could be to code a fmod() implementation for cpp_double_float<>, and extracting the whole number digits (the digits that appear before the decimal point) through repeated division and fmod(). This should, intuitively, solve the precision problem encountered while divided by pow10(exp10) where exp10 is very large.
Kind regards,
Fahad.

Yes, using integer part separately could also help. This could be also related to the fact that in Table 3 and Table 4 in #21 the float has either 13 or 14 digits10.

Finished with the backend requirements of #34


Index.html page on gh-pages branch

Is your feature request related to a problem? Please describe.
There is no information at https://honeycombio.github.io/helm-charts which is confusing for those that navigate to it.
Describe the solution you'd like
Add some content that steers users in a better direction.
Describe alternatives you've considered
Additional context

Resolved as part of #84


Reloading - Shell take out of inventory

When creating a new lot, a select need to be added to select shells and remove them from inventory.
Default choice should be "Ignore" in case the use do not account empty shells.

Done.


Add a citation file?

Is your feature request related to a problem? Please describe.

I would like to cite QCElemental (and related packages) as software I am using, but there seems to be no clear way to do it (e.g. there are 29 contributors on GitHub -- alphabetical ordering of username?)
Describe the solution you'd like

Github supports CITATION.cff files, it'd be super helpful for people who want to cite the QC stack to have one.
https://citation-file-format.github.io/
Describe alternatives you've considered

Additional context

good point. qcengine has a codemeta, but qcel does not. I can see about a citation file here and/or a zenodo. for the QC stack, qcarchive as a whole and especially qcfractal have https://wires.onlinelibrary.wiley.com/doi/abs/10.1002/wcms.1491 . qcengine and sort-of qcel have https://aip.scitation.org/doi/pdf/10.1063/5.0059356 . qcschema has nothing journal-wise. i'll see about documenting better, but see if any of those could do in the interim.

Great, thank you for those links!

Add a note in README.md suggesting to use one of these two papers to cite QCElemental


UserRetriveService GetFirst(...) method returns null for User even if user exists in table Users

I was working on same database with same application hosted on different virtual machines, and occasionally with same username\password i could log in on one Web page but not on the other.
I debugged and saw that
var user = connection.TrySingle<Entities.UserRow>(criteria);
returns null in one case, but user is present in table.
If i try (on the same db\table)
var user = connection.List<Entities.UserRow>(criteria);
it will return list of users correctly.

Workaround is to manually go into DB and update\change username to lower\upper-case(one letter is enough) and i can log in again.

Eager to hear any idea, thanks in advance

I solved problem,
If you set up your database\table to be searched as key sensitive(CA) and you insert incorrect value for username (etc. Admin instead of admin). TwoLevelCache will cache it as invalid credentials
return TwoLevelCache.Get<UserDefinition>("UserByName_" + username.ToLowerInvariant(),                  TimeSpan.Zero, TimeSpan.FromDays(1), fld.GenerationKey, () =>             {                 using (var connection = SqlConnections.NewByKey("VTassure"))                     return GetFirst(connection, new Criteria(fld.Username) == username);             });
and because of  username.ToLowerInvariant() , it will be invalid for both "Admin" or "admin" (doesn`t care for upper\lower-case letters).But your Db\table doesn, so if you try then to put "admin" it will not go into database to look up for user, it will find it in cache and will reject.

Yes, if your user table is case sensitive you should change that code accordingly, maybe username.ToLowerInvariant() inside criteria as well. But you should be storing usernames in lower case also in DB. Anyway if possible use CI collation.


Laravel 8.45.0 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v8.45.0 branch.
Before merging, you need to:

Checkout the shift-ci-v8.45.0 branch
Review all pull request comments for additional changes
Run composer update
Test your application (no tests?)

:warning: You are running version 4.6.0 of spatie/laravel-translatable. The latest version is 5.0.0. You should review this dependency to see if you can upgrade to the latest version.

:information_source: If you are running PHP 8 be sure to run composer update locally to ensure you have the proper versions of your dependencies to reflect your PHP 8 environment.


Example project fails to build with NDK not installed error

Describe the bug
I tried to run the example project from the zxing_lib package (this https://github.com/shirne/zxing-dart/tree/master/example repository), but I got an error saying that NDK is not installed, even though I have installed it from the SDK Manager.
To Reproduce
Steps to reproduce the behaviour:

Clone the https://github.com/shirne/zxing-dart repository.
Open the example folder in Android Studio
Run the app on a Pixel 6 device in debug mode
See the error message in the console

Expected behaviour
I expected the app to build and run successfully, and to be able to scan barcodes using the camera.
Screenshots
Screenshot to help explain your problem:

Desktop:
OS: macOS Sonoma
Chip: Apple M2 Max
Smartphone:
Device: Pixel 6
Android version: 14
Additional context
I was able to run other example projects from different packages, such as google_ml_kit, without any issues.


The name pattern {_} doesn't work as expected

The pattern alloc::vec::{_}::set_len is expected to match alloc::vec::Vec<T, ...>::set_len (when using the Rust name pattern matcher) but doesn't (note that alloc::vec::_::set_len does).

the rust name matcher doesn't actually handle inherent impl blocks at the moment :sweat_smile:

Oh I see ^^
I'm surprised those are in the syntax at all for the moment: maybe we should (temporarily) panic if an impl block appears in a pattern after parsing?

I can't distinguish impl blocks from trait impls because we allow {Trait<Self>} syntax

What is the {Trait<Self>} syntax? (I'm not sure the Charon-ML name matcher allows the same)

it's equivalent to {impl Trait for Self}


Fails on Chinese people

face_recognition version:
Python version: 3.5.2
Operating System: Ubuntu 16.04

The library can't separate chinese people
I tried finding
https://www.google.com/url?sa=i&rct=j&q=&esrc=s&source=images&cd=&cad=rja&uact=8&ved=2ahUKEwjKwbLM6JTdAhWGMewKHekXBIYQjRx6BAgBEAU&url=https%3A%2F%2Flaotiantimes.com%2F2016%2F09%2F05%2Fchinese-prime-minister-to-visit-laos%2F&psig=AOvVaw0jNXiklPM4YaxtrqIEOjQe&ust=1535719348182900
in
https://www.google.com/url?sa=i&rct=j&q=&esrc=s&source=images&cd=&cad=rja&uact=8&ved=2ahUKEwiWwOmD55TdAhVNMewKHaGnBo0QjRx6BAgBEAU&url=https%3A%2F%2Fwww.japantimes.co.jp%2Fnews%2F2015%2F01%2F09%2Fnational%2Fhistory%2Fchina-brings-sex-slave-issue-spotlight%2F&psig=AOvVaw0jNXiklPM4YaxtrqIEOjQe&ust=1535719348182900
image and it identified the first person as the target person. Thus it can't accurately compare faces.

Correct. You may have some success lowering the threshold, but there are limitations to the accuracy here. Please see this for more details: https://github.com/ageitgey/face_recognition/wiki/Face-Recognition-Accuracy-Problems#question-face-recognition-works-well-with-european-individuals-but-overall-accuracy-is-lower-with-asian-individuals


Question about the DataPreprocessing when training ImageNet

Hi, Thanks for your wonderful work！
I notice that maskgit uses RandomResizedandCrop for data augmentation. But I find that in the code you adopt Cropping and flipping and comment out the Normalization (since I think VQGAN is trained with normalization). https://github.com/valeoai/Maskgit-pytorch/blob/b0b2b3cc11cffd0b159f22dc1c6e73a7e8b53db3/Trainer/trainer.py#L80 I am curious about the reason behind.
Thanks in advance!

Hi! Actually, I was not aware that RandomResizedandCrop is available directly from Pytorch... It might work, but be aware of the ratio aspect, otherwise the model will also generated "distorted" images.
About the official MaskGIT, since they do not realize the training code, I can not help you, sorry.
Best,
Victor

Yeah, but the inner function of RandomResizedandCrop is cropped first then Resized. I think it is not quite the same with the current operation.  I think the more feasible way is still using Resize and RandomCrop which is utilized in VQGAN Repo


add verbose toggle

verbose with the command line tool is really helpful we should expose this in the gui.

what if we always run it in verbose? since you only see the output when it is an error, anyway, it should be fine?

verbose is like, "really" verbose, since it also effects stuff like ofDirectory::listDir so you're seeing a ton of internal workings as it parses things...  it's maybe too much info IMO, let me add a toggle and let's see...


fix: use valid import/export syntax

Previously we were using the common syntax of importing/exporting files without the extension. However, this is not natively supported in browsers and also not in Node.js with ESM. This change adds a linting rule that automatically adds the .js extension
to any relative import.

LGTM 👍 I think you broke the record for most changed files in a pr

I'm torn between being proud and distraught 😅


Can't access MyWebApi from browser

I have the following docker-compose.yml but i can't access webapi from browser. Only webfrontend via http://localhost:5008
Can i access webapi?
docker-compose.yml
version: '3.4'
services:
webfrontend:
image: ${DOCKER_REGISTRY-}webfrontend
ports: ["5008:80"]
build:
context: .
dockerfile: WebFrontEnd/Dockerfile
mywebapi:
image: ${DOCKER_REGISTRY-}mywebapi
ports: ["5010:90"]
build:
context: .
dockerfile: MyWebApi/Dockerfile

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7aee46f8-9db1-33ef-b637-fc8e8881fab9
Version Independent ID: 6c469f9f-9751-669f-3dd4-c4fb49c9ae19
Content: Multicontainer tutorial using Docker Compose & ASP.NET Core - Visual Studio
Content Source: docs/containers/tutorial-multicontainer.md
Product: visual-studio-windows
Technology: vs-azure
GitHub Login: @ghogen
Microsoft Alias: ghogen

@vlapenkov -- thank you for your question.  This repo is solely for documentation issues, with limited support options.  Please consider these resources:

Visual Studio on Q&A
https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.
cc @ghogen for awareness


Can't pass a variable!

I cannot for the life of me get the
{% easy_map "[address]" 300 400 %}
tag to work unless I hard code the address in.
I have a variable {{ f.location.full_address }} which outputs like this:
Florida, Miami, Chucunantah Road
If I hard code that same output, the map works.
Can you help?

Bitbucket: https://bitbucket.org/kmike/django-easy-maps/issue/3
Originally reported by: Anonymous
Originally created at: 2011-07-13T02:02:15.352

Hi,
Hardcoded addresses should be in quotes, variable addresses should be without quotes.
So isn't this working?
{{{
{% easy_map f.location.full_address 300 400 %}
}}}

Original comment by: Mikhail Korobov

thanks, working great now, without curly brackets!

Original comment by: Anonymous

Yeah!
Thanks for the report. Closing this as invalid because there is an example with variable address in README and I don't know how to improve it to make less confusing.

Original comment by: Mikhail Korobov


Password reset conflict Laravel 5.3

Hi. This package has conflict with some 5.3 query builder functions.
BadMethodCallException in Builder.php line 2450: Call to undefined method Sofa\Eloquence\Query\Builder::notify()
How to fix this error?

it's not about the package - you probably forgot to use Notifiable on your model and sh.tty laravel exception handling for magic methods in eloquent lead you to Builder call.


Wrong collision response of test_move() when the CharacterBody2D is inside a One Way Collision

Godot version
4.0.alpha7
System information
Ubuntu 20.04.3 LTS
Issue description
Expected behavior: Collision detection with one-way CollisionShapes using test_move() only happens when the body is moving against the CollisionShape
What happens: test_move() detects one-way collisions even when moving along a vector with no y-component. The CharacterBody2D itself doesn't seem to detect a collision.
For example, if the one-way collision margin is the default of 1 and the safe margin of the CharacterBody is 0.01, the vertical space where this misfired collision can occur seems to be ~0.43px. This gets wider with bigger collision margins of the one-way collisions or with smaller safe margins used in test_move().
Steps to reproduce

Create a CharacterBody2D and a StaticBody2D with one-way collision.
Give the CharacterBody2D a script and run test_move() on the CharacterBody2D with its global_transform, a horizontal distance vector, and a small safe margin. The smaller it is, the easier to get a misfired collision.
Place the CharacterBody2D inside the very top of the StaticBody2D, while still being inside it. The one-way collision margin of the StaticBody2D can be increased, or the safe margin inside test_move reduced, to see the effect more clearly.

With the minimal reproduction project, just run it, read the script comments, and notice how for some time, the output logs print "true".
Minimal reproduction project
test_move bug.zip

I could replicate this on 4.0.alpha1

The following seems to be what happens in the engine in test_body_motion:
When the body is "deep" inside the one-way collision shape, i.e. in the recovery step each pair of contact points has a depth greater than cbk.valid_depth (i.e. greater than 1 here), the counter cbk.invalid_by_dir is incremented, and this causes the pair of shapes (the shape of the body and the one-way collision shape) to be added to excluded_shape_pairs. No recover_motion is calculated. Then, in the motion step there are no collision tests at all and the motion succeeds with safe = 1, which implies no collision is reported.
When the body is "near the surface" inside the one-way collision shape, i.e. in the recovery step each pair of contact points has a depth less than cbk.valid_depth (i.e less than 1 here), then the counter cbk.invalid_by_dir is not incremented, and the pair of shapes is not added to excluded_shape_pairs. Some recover_motion is calculated based purely on the contact points, which ignores the one-way collision direction (in this case, the recovery motion is directed upwards). Then, in the motion step there is a collision test (because shapes were not excluded). Namely there is an initial test, consisting of two steps, to decide if the body is stuck. Firstly it is checked whether there is any overlap, which is true because the recovery motion did not depenetrate the body completely. Secondly it is checked (if applicable, which it is here) whether the motion_normal points in the direction of the one-way direction. Now we note that motion_normal is computed as motion / motion.length(), which is Vector2(nan, nan) if motion = Vector2(0, 0). Hence the second test fails, and the body is detected to be stuck, which cancels the further kinematic solving and which sets safe = 0, which causes a collision to be reported.

@rburing While in the test project, I used Vector2(0, 0), this behavior actually occurred with horizontal vectors as well, such as Vector2(4, 0) which would be able to calculate a normal that isn't nan.

The "stuck" check I mentioned at the end is here: https://github.com/godotengine/godot/blob/563690347a6b9da4882bca15ebd9f3d79f30d3e6/servers/physics_2d/godot_space_2d.cpp#L788-L799
Indeed, it also decides that it's stuck for horizontal motion vectors, in which case motion_normal.dot(direction) == 0.0.
I'm not sure where to begin to fix the issue. cc @fabriceci

@rburing Would changing the condition to motion_normal.dot(direction) <= 0 (not strict inequality) be problematic in other ways?

The issue is valid in 4.2.dev4

Would this bug affect testing if there's a one-way collision anywhere Vector2.DOWN? I've been trying to detect if there are one-ways using test_move below enemies, but it can't find them, but it works fine with horizontal tests.


Add ability to resolve @scoped/packages subdirectories

Subdirectories autocompletion for @scoped/packages is not working:

Hi, I have fixed this problem, please try the newest version v1.5.0


Can't pass big number to smart contract method

Error: invalid number value (arg="", coderType="uint256", value=1000000000000000000, version=4.0.33)
at Object.n [as throwError] (pageHook.js:22646)
at I.encode (pageHook.js:22646)
at pageHook.js:22646
at Array.forEach ()
at X (pageHook.js:22646)
at ne.encode (pageHook.js:22646)
at ue.encode (pageHook.js:22646)
at e.value (pageHook.js:36175)
at pageHook.js:25264
at new Promise ()

Javascript does not support big numbers. Try to pass it as a string.

Let's try toSun or toBigNumber


Just a dice command

Dice command that will generate a random number between 1 and 6

conflicts?

also the code could be written better @MrM0nkeyz

Yeah you are right.

I am going to close this pr. We can get back to you once you got the updated code.

It's not an important command anyways so this doesn't need to be here.

It's fine.

I am just checking some commands, so like 5 minutes or less ;-;


Rollup not recalculated when order by field changes

When the order by field on a child record has changed, if no other field involved in the rollup has changed, the rollup is not recalculated.  The reason for this is that the order by field is not included in fieldsToSearchForChanges (https://github.com/afawcett/declarative-lookup-rollup-summaries/blob/master/rolluptool/src/classes/RollupService.cls#L540).  Order by influences query based rollups and therefore not including it could result in the rolled up value being incorrect.
Note that this issue existed prior to PR #227.
See https://github.com/afawcett/declarative-lookup-rollup-summaries/commit/3f2d31e9a7b9be7c7384e1886a82b9fa9fcbd6bc#diff-4aa41bc6f886aeba1cba33ce3962348cR1535 for demonstration of issue.  This test fails with prior code base on line https://github.com/afawcett/declarative-lookup-rollup-summaries/commit/3f2d31e9a7b9be7c7384e1886a82b9fa9fcbd6bc#diff-4aa41bc6f886aeba1cba33ce3962348cR1535 because the change to Amount__c is not considered material to rollup processing.

Thanks @jondavis9898 another good spot, this will be most acute on the concatenation based aggregate rollups as you say.

@jondavis9898 PR 244 fixes this i think, yes?

Yep, this one is fixed in PR #244.

Fixed in v1.24.


Plesk Onyx 17.5.3 NGINX Webmail Conf Logic Flaw

Line ~10 Did not also check for http2 in file \opt\psa\admin\conf\templates\default\webmail\nginxWebmailPartial.php

The bug was fixed in 526ef78


docs: k8s: updating docs for k8s v1.9, 1.10 and 1.11 support

updates docs

test-docs-please

Thanks a lot @tasdikrahman!


Remove index 2 value from example output

Remove index 2 value from example output as the ending index is exclusive
Fix the typo in the example output.
https://go.dev/play/p/6YZqXuuvvXa

Yep I have already done the exercises
PR is https://github.com/exercism/go/pull/2066


plex: <IP_ADDRESS>55-fef15d30c -> <IP_ADDRESS>79-424562606

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release 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.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 326379 run on x86_64-linux 1

3 packages built:

plex
plexRaw
plexRaw.basedb


Should fallback to $HOME/go when $GOPATH is not set

The default $GOPATH is $HOME/go. Instead of failing silently, you should fallback to $HOME/go in the event that $GOPATH is not set.

9fa172b4c95499b4620ac5f3a7d5add9fc854afa


Use AWS SDK v3

@tob1k the aws-sdk is already on  3.53

Yeah you're right.
Seems like the issue is because the version hasn't been bumped, the 0.2.0 version of the gem is what gets served up, and it's still locked in the gemspec to use the aws-sdk v2

@jnicklas can you take a look at this PR?

Thanks!


string.h / strcasestr() + strtok()

Hello world :-)
It would be really nice to have strcasestr() and strtok() functions in standard Zephyr C library and string.h:-)
Maybe Zephyr could use such functions from the build host OS when not available in Zephyr? I know that may introduce compatibility problems but for testing before they are available in the upstream that would be some sort of solution :-)
Thank you :-)
Tomek

Hi @cederom
The reentrant strtok_r(3) is available in Zephyr's minimal C library here and it is highly encouraged to use that instead of strtok(3) to eliminate race conditions.
Similarly, strncasecmp(3) is available in Zephyr's minimal C library here. Using the n variants of common string functions is preferred for most applications.
If those are insufficient for your needs, please consider opening a PR.

Thank you @cfriedt this is exactly what I need :-)
If this is internal Zephyr implementation why does not start it with k_ or z_ prefix? Like k_msleep()? :-)


Dark strata cobblestone mortar bug

Breaking a dark strata mortared cobblestone always drops default cobblestone.

Already fixed for next release :)


sendHeight not resizing child page

Using on the child page:
	<script type="text/javascript">         var pymChild = new pym.Child();         pymChild.sendHeight();     </script>
The sendHeight is not returning the page height of the child page after the filter has been selected and the user clicks "search."
Sample:  https://ymcanwnc.org/schedules-2/
Thoughts?

Hi,
I got similar issue but to me it happens two things :
When the object starts it doesn't return the child height and neither works the event sendHeight() that I use for making a responsive chart. Any idea?
Many thanks

Hi @starklmc, Let me try to understand, trying to take a look at your sample page if looks like if the pym child looses the reference to the parent id and that will cause the messaging to the parent not to work.
The sample is a bit complicated to follow through since there are many parts involved, could you produce a sample page just with the interaction with pym so that I can take a look at?

Hi @LuisSevillano, I would ask the same if you can generate a simplified page that I can take a look at it that would help me understand what the issue you are facing is....Take a look at the docs there are some examples there to get you started...we are using pym continuously to resize graphics at NPR so that's covered by pym.js for sure. are you using the renderCallback function? is your data dynamic?

@jjelosua thank you very much for your quick response. I've already solve my problem.
A css position: absolute attribute of two containers were causing some troubles not allowing pym to calculate the size of the child element.
Many thanks from Madrid!
Luis

The only workaround or hack I found is
create a hidden button with class update_height and add an event on click sendHeight because no matter what you do renderCallback does not work. Used it on my embed page here : Sample Page


Theme tokens with inline syntax and breakpoints seem to not be interpolated

It looks like theme tokens are not being interpolated when used in combination with breakpoints in the direct props style declarations.
// kuma.config.ts
import { createTheme } from '@kuma-ui/core'

const theme = createTheme({
  colors: {
    primary: 'blue',
    secondary: 'green',
  },
})

// Component.tsx
import { Box } from '@kuma-ui/core'

// the BUG: color will be "colors.primary" which is not a valid css value - it should be "blue" instead (with a media query for secondary color).
<Box
  color={['colors.primary', 'colors.secondary']}
>
  Text
</Box>

// I've also tried with t("") syntax, but it also does not interpolate (not sure if it should though)
<Box
  color={["t('colors.primary')", "t('colors.secondary')"]}
>
  Text
</Box>

Fixed in #381


Align async msg wording - change Message to Command

Our thinking and wording of asynchronous messaging, and distinction between events and messages should be aligned.
Let's align this page to the wording at

Messages can be classified into two main categories. If the producer expects an action from the consumer, that message is a command. If the message informs the consumer that an action has taken place, then the message is an event.
from Asynchronous messaging options in Azure

Thus I recommend changing "Message" to "Command" on this documentation page.
@clemensv: Your thoughts on this?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 08e12671-466a-8807-63bb-fb1923554d60
Version Independent ID: 88dcdcb5-e611-134d-5c2e-66c657416445
Content: Compare Azure messaging services - Azure Event Grid
Content Source: articles/event-grid/compare-messaging-services.md
Service: event-grid
GitHub Login: @spelluru
Microsoft Alias: spelluru

Hello @RicardoNiepel,
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@clemensv - please let me know what you think. Thanks.

We will track this internally. I will work with Clemens when has time to update the article. Thanks.
#please-close


Fixes the docker conductor:ui build.

There appears to be a weird error with the node dependencies and the current node version
used in the docker UI container image. You can find more information: https://github.com/gulpjs/gulp/issues/2162.
If you run the conductor UI container, it will not startup properly because the npm run build --server will fail with the following error:
../src/node_contextify.cc:631:static void node::contextify::ContextifyScript::New(const FunctionCallbackInfo<v8::Value> &): Assertion `args[1]->IsString()' failed.

Which is caused by the natives library. I simply just npm update in order to upgrade the packages.
This isn't caught in the Dockerfile for some reason. Even after using set -e, docker will run the command and continue on. Not sure why that happens, but you can run the command from the specific layer to see the real error message. Because the dist folder is never created, the startup.sh script will not run.

I fixed the merge conflict and simply just updated the natives package directly in the package-lock.json file to fix this issue. I'm unsure how to use npm to update with what was in master. For some reason, npm upgrade didn't do it's thing.
Anyways, the new lock file and natives 1.1.4 fixes the docker compose build.

I also met this problem. I changed the version of natives according to this pull request and I solved it. I think that this pull request should be merged.

Can we get this reviewed for a merge?  I too have to maintain this change in a fork
Also looks like upstream could use a rebase

Actually, I'm not able to reproduce the problem he reports in Docker when using the node:9-alpine image.  According to gulpjs/gulp#2162, this is a problem with gulp on Node 10 when the dependency natives < 1.1.3, which by default resolves to 1.1.2.
Node 9 is past end-of-life.  Node 10 is current LTS release.
As far as I can find, NPM doesn't provide for a way to force resolution without using special tools that would change the build toolchain, hence this being changed using the package-lock.json.   It would make sense to update the Docker base image to Node 10 along with this change.

I believe this should already be resolved in the newest conductor versions. Please reopen/submit a new PR if you still see the issue.


Add make target for mixer codegen

#8543

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: douglas-reid
If they are not already assigned, you can assign the PR to them by writing /assign @douglas-reid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

@douglas-reid Yeah it makes perfect sense to combine protoc and mixer codegen into one image. Let me update the PR to do that.
I don't think we want to include this tool in mixer image though. Essentially it is just a tool which is only used by adapter developer and not needed by mixer workload.

@douglas-reid After thinking a little bit more into it, I think it might not make sense to combine protoc and mixgen together in Istio release since it could introduce a dependency circle.. (API needs protoc to get update, while Istio needs API to build). cc @geeknoid who seems to be maintaining the protoc image.

@douglas-reid ptal?

Need to rebase, otherwise LGTM.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

@douglas-reid rebased. ptal, thanks!

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: douglas-reid
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

@douglas-reid  Can I also get a approval? :)

@bianpengyuan i don't have OWNERS. that approval must come from elsewhere.

@hklai @geeknoid Can you please take a look?

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

Needs approval from an approver in each of these files:

OWNERS [hklai]

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

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
55a88f51f8060e9e8daea3516be9339912a7c9b9
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
55a88f51f8060e9e8daea3516be9339912a7c9b9
link
/test istio-presubmit

prow/istio-unit-tests.sh
55a88f51f8060e9e8daea3516be9339912a7c9b9
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

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

Needs approval from an approver in each of these files:

OWNERS [hklai]

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

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
55a88f51f8060e9e8daea3516be9339912a7c9b9
link
/test istio-presubmit

prow/istio-unit-tests.sh
55a88f51f8060e9e8daea3516be9339912a7c9b9
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
83db2c9b0713435094750c55b9dd741ced721068
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

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

Needs approval from an approver in each of these files:

OWNERS [hklai]

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

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
83db2c9b0713435094750c55b9dd741ced721068
link
/test istio-integ-k8s-tests

prow/e2e-mixer-no_auth-mcp.sh
d460a2349086b7360a5a65aa8e607cab0680a3a1
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Vampire 5th Edition Updated translation and layout fixes

Apologies again and thank you for all your hard-work.
Hopefully the display issues is just linked to me not understanding how translation works.
Please keep up the great work.

You're welcome. Thanks for the help & patience. :)

No thank you. You have been great!

Huh not sure why there is a conflict. It should read Spent Experience however.

That is on me. I had to do a pull request for another project & during that process went ahead and updated the vampire 5e translations to hopefully fix those issues you were seeing from the Tuesday update. You'll want to pull down those updates that I just did less than an hour ago. Its likely what is causing the merge conflict.

Thanks I'll give it a go.
Yell if there is anything I can do to make your job easier.

Ha. It took me a few times to get the hang of it. :) This should be pretty easy. Just go to roll20 folder in Github. Do the Configure a remote for a fork then do the fetch the upstream directions. Provided those two go well you can push the updates to your fork.
You may need to make sure your code for this push is not in that roll20 folder as it could cause a merge conflict. This might be the rage quit spot. ;)

Ok I think that's my last commit for this week.
Thank you so much for spending time to fix the translation issue. It is really appreciated. I was dreading that people would see the poorly formed issue all week. I hope I haven't messed it up with the last commits.

This looks good. :) Hopefully this week is smoother for the vampire world.

Hopefully I didn't break the translations....


Application.create_bot() -  'str' object does not support item assignment

Summary
Application.create_bot() -  'str' object does not support item assignment
Reproduction Steps
app = await self.client.create_application(name) # works in my case
bot_app = await app.create_bot() # creates the error
'''
'str' object does not support item assignment
'''
Code
try:
            app = await self.client.create_application(name)
        except Exception as e:
            print(sty.fg.red + str(e) + sty.fg.rs)
            await ctx.send('Failed to create an application')
            return
        try:
            bot_app = await app.create_bot()
        except Exception as e:
            print(sty.fg.red + str(e) + sty.fg.rs)
            await ctx.send('Failed to create an bot to the application, the application was successfully created ...')
            return
        print('before token ...')
        token = await bot_app.reset_token()

Expected Results
should give me a ApplicationBot Object back instead of the error
Actual Results
well, it gives me the ''str' object does not support item assignment' error
System Information

Python v3.10.6-final
discord.py-self v2.0.0-alpha

discord.py-self pkg_resources: v2.0.0a2

aiohttp v3.8.1
system info: Linux 5.18.19-3-MANJARO #1 SMP PREEMPT_DYNAMIC Tue Aug 23 15:56:13 UTC 2022

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shared the entire traceback.
[X] I am using a user token (and it isn't visible in the code).

Additional Information
No response

Discord changed how these endpoints worked and the fix has been part of an ever-growing stash I haven't finished yet. I'll just commit this separately.


For Inference on an external Soccer Video File, do I still need camera calibration for action spotting ?

I'm in assumption that only Calf, Calf GCN needs Camera calibration , does Netvlad++ needs camera calibration?

Also can I run this on frame by frame based scenarios? so to get exact timestamps

Hi @ShanuD11, I'm not part of the SoccerNet team, but in my understanding all 3 methods need frame features, but CALF CGN also needs the camera calibrations

So netvlad++ is totally based out of station temporal features, am I right?

As I understand it, yes (although you need annotations as well of course)

Hi @ShanuD11 ,
Only CALF_Calibration and CALF_Calibration_GCN require field calibration files. CALF, Pooling and TemporallyAwarePooling do not require field calibration files.
All those baselines are based on frame features extracted from videos. Most of them are spatio temporal features (i.e. they have a temporal receptive field and require a stack of frames), except ResNET that is spatial only, hence could be a fit for your scenario based on frames only.
 We do provide code for all those module, but you'll have to stack them in a single dataloader.


Create an asynchronous RMB client

Is your feature request related to a problem? Please describe
We currently only have the RMB direct client for zos calls. We need a way to be able to broadcast calls and perform action when the response is available without waiting for it.
Describe the solution you'd like
Create a new client that sends calls to RMB and registers a listener that waits for the response and perform a desired action.

Verified on devnet.
The new async client is used in the gpu indexer as seen in its implementation and the indexer is working fine and returns the gpu data without any issues. and was verified before in https://github.com/threefoldtech/tfgrid-sdk-go/issues/209#issuecomment-1649420555
and another issue was created for adding an example for the async client #268


Display border and edge effect for drawables with negative size

Negative size means the rectangle has either left > right or top > bottom in coordinates.
It is displayed as if the rectangle is flipped to the congruent positive size rectangle.

Closes ppy/osu#13406

(About commit history) I first tried to modify shader code but then I ended up with a simpler way by always normalizing the masking rectangle.

This blackness when flipped looks to be the wrapping mode. The following change fixes it:
diff --git a/osu.Framework.Tests/Visual/Containers/TestSceneSizing.cs b/osu.Framework.Tests/Visual/Containers/TestSceneSizing.cs
index 529af6b21f..2b033cc722 100644
--- a/osu.Framework.Tests/Visual/Containers/TestSceneSizing.cs
+++ b/osu.Framework.Tests/Visual/Containers/TestSceneSizing.cs
@@ -5,8 +5,11 @@
 using osu.Framework.Extensions.Color4Extensions;
 using osu.Framework.Graphics;
 using osu.Framework.Graphics.Containers;
+using osu.Framework.Graphics.OpenGL.Textures;
+using osu.Framework.Graphics.Primitives;
 using osu.Framework.Graphics.Shapes;
 using osu.Framework.Graphics.Sprites;
+using osu.Framework.Graphics.Textures;
 using osu.Framework.Input.Events;
 using osu.Framework.Utils;
 using osuTK;
@@ -1166,10 +1169,11 @@ protected override void OnDrag(DragEvent e)

         public InfofulBox()
         {
-            Add(new Box
+            Add(new Sprite
             {
                 RelativeSizeAxes = Axes.Both,
                 Depth = float.MaxValue,
+                Texture = Texture.WhitePixel.Crop(new RectangleF(0, 0, 1, 1), Axes.X, WrapMode.ClampToEdge, WrapMode.ClampToEdge)
             });
         }
     }

Is this known?

Actually, the blackness issue is also reproducible in master.
I cannot figure out what is causing the issue: the blackness is also gone when a texture ("sample-texture") is used instead of the WhitePixel, or even using Texture.WhitePixel.Crop(new RectangleF(0, 0, 1, 1)) with the None wrapping modes.
The issue doesn't affect the current osu-side use-case of border-only drawables (box.Colour = Color4.Transparent).
Therefore, I propose this PR be merged as is, potentially adding a comment to the test scene, unless a proper fix can be figured out quickly.

Ooh fun. The blackness is caused by the front to back pass. I've got a fix but will merge as is.


$mod aggregation operator allows the divisor to be zero

FerretDB version
commit.txt: https://github.com/FerretDB/FerretDB/commit/4a69d977535fc0a67e523095b4ddb22cf26d9079
version.txt: v1.2.0
Backend
PostgreSQL
Environment

OS: Ubuntu 22.10 x86_64
Deployment: v1.2.0
Deployment details: 23.0.3

What did you do?
See repro in #2781
What did you expect to see?
It should return a BadValue error in a match stage and Location16610 in a project stage.
What did you see instead?
No error

This issue is not a bug because we cannot validate operators that have not yet been implemented. I expected a "NotImplemented" error to be returned, hence the confusion.