gradients.norm(2, dim=1), dim=1?

@caogang Thanks for your good code! But something confuses me in gan_cifar10.py

dim = 1? Why it is only normed in the second axis? I think it should be normed across all axis but the batch axis.

Oh, very good question. I haven't found this problem before. I implemented this code following the author of WGAN-GP https://github.com/igul222/improved_wgan_training/blob/master/gan_cifar.py . Maybe it is a trick from the authors?:) However, do you try to norm across all axis but the batch axis? Maybe it would have better results:).  If you get a better results, please tell me and start a Pull Request. Thank you.

@caogang It is exactly normed across all axis but the batch axis just like it is in the paper. As shown below (from the authors' code https://github.com/igul222/improved_wgan_training/blob/master/gan_cifar.py), the output of the generator is reshaped to rank 2. In your code, the output of the generator is still rank 4.

yes, i also found this problem.

may be you can change the penalty term as below
gradient_penalty = ((gradients.view(gradients.size(0), -1).norm(
        2, dim=1) - 1)**2).mean() * LAMBDA

Yes, you are right. Thanks for your suggestions @LynnHo @eriche2016 . It is a bug :(, and it is fixed in the latest version. Hope it will be helpful.

still has this in the MNIST code...


Test OpenGl methods in FakeBatcher test

Part of the work to split Batcher in BatcherInterface and
PrimitiveAssembler.
(https://github.com/karupayun/orbit/commit/1d4d264fc5f1f90c033b62bc2d0b22f45750401f).
This class is testing different things (internal implementation, picking
and shared pickable) that now are going to be in separate files. As a
first step I'm testing the internal methods that are going to be used in
OpenGl implementation instead of the PrimitiveAssembler part.
In addition I'm renaming the internal methods to the original name since
they are going to be public functions used by the PrimitiveAssembler
(Henning's suggestion).
As a side effect, CaptureWindow complains correctly that they expect
that methods to be in OpenGlBatcher. This is because CaptureWindow
should use primitive assembler there. This is already in
https://github.com/karupayun/orbit/commit/1d4d264fc5f1f90c033b62bc2d0b22f45750401f,
but for now I just added a temporal TODO.

As little temporal detail, I add GetBatcherId() and GetNumElements()
temporary to Batcher.

2nd step in http://b/225173189

I fixed a rebased issue and force-push. Really sorry for that.
I've just added a renaming Add*** methods to Add***Helper and make 2 comments to tests in BatcherTests

Making it draft, this PR is not very clear and I will make a big PR instead.

Closing this PR as https://github.com/google/orbit/pull/3539 will be merged.


[BUG] "Voiceover" is inserted into Subtitles, Script and actual Voiceover

Describe the bug
The resulting video has "Voiceover" as subtitle and actual Voice at the beginning of each new clip within the output.
Expected behavior
No "Voiceover" in the result
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser Firefox
Python Version 3.11

Additional context
I suspect this is caused by the ChatGPT output, because "Voiceover" already appears in the debug output when it initially generates the text.

Try to tweak the prompt (inside gpt.py) as such it doesn't include those words.

Try to tweak the prompt (inside gpt.py) as such it doesn't include those words.

Wouldn't it be more reliable to just parse the script and delete all "voiceover"  before it is further used to generate subtitles and voice?

Try to tweak the prompt (inside gpt.py) as such it doesn't include those words.

Wouldn't it be more reliable to just parse the script and delete all "voiceover" before it is further used to generate subtitles and voice?

No, because it's not just "voiceover". There are a lot of keywords, and what if someone's script includes voiceover?


AMD/RequireJs Support?

Hi,
I just wanted to know if this works when loading scripts/modules (such as jQuery) using RequireJs and the AMD format?
Kind regards,
Mark

I think JSlim should work with RequireJs.  It doesn't really matter how you bring in the JavaScript library, just that you call it.


[nvidia-cutlass] New port with v3.3.0

New port with https://github.com/NVIDIA/cutlass/releases/tag/v3.5.0

Resolve #32648

Maybe we can name it with cutlass, like the repository name.
Here, I brought changes of the following PR

https://github.com/luncliff/vcpkg-registry/pull/191

The port can help CUDA GPU support of other ports

#36850

[x] Changes comply with the maintainer guide.
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

Libtorch is depending on this so adding this means decendoring it from libtorch.

I will check the port
On Fri, Jul 12, 2024, 22:45 Alexander Neumann @.***>
wrote:

Libtorch is depending on this so adding this means decendoring it from
libtorch.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/pull/39881#issuecomment-2225622442,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADLSYE7IINJE574H6OV5E6LZL7MV7AVCNFSM6AAAAABKY3CES2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRVGYZDENBUGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Port libtorch now uses nvidia-cutlass.
Seems like there is a problem finding CUDA compiler with the default option. Manually set CUTLASS_NATIVE_CUDA=OFF

Usage test pass with following triplets:
x64-windows
x64-windows-static


Custom description for referenced component is sometimes ignored (caching issue in upstream parser?)

This is a bit hard to describe, and I believe the actual bug here is in an upstream dependency, but I'm not sure where exactly to look.
When using $ref in a schema, it's possible to include a description both in the shared component being referenced (e.g. describing a type), and in the property itself (e.g. describing its usage in context). The relevant specs leave it up to implementations what to do here: show both descriptions, or over-write one with the other.
The parser used in OpenAPI Explorer appears to prefer the "local" description, discarding the one from the shared reference. However, it sometimes fails to do so.
Playing around with an example, this appears to be some kind of caching bug: if you always supply a description when using a particular component, that description is displayed. However, if you have previously referenced the component without overwriting its description, further custom descriptions are ignored.
For an example, load this OpenAPI description file with OpenAPI Explorer, and observe:

The top-level fields with custom descriptions display them
nested-one-custom displays its custom description - that component has only been used with a custom description, so is not cached(?)
nested-two-custom ignores the custom description - somehow the top-level two-default populated a cache(?)
double-nested-three-custom ignores the custom description - because nested-three-default populated the cache(?)
double-nested-four-custom ignores the custom description - because nested-four-default populated the cache(?)

Interesting. Had no idea there was a problem like that. The first step would be to dive into the spec parser, and see if the problem is the parsed spec is the issue or if the spec handling in this library is the problem.

This has been fixed in version 1.0 release og 1.0.496.

FYI @adamsussman, I saw your issue over in the RapiDoc repo. This repo is a fork of RapiDoc from pretty recently, and the issue regarding overwriting of descriptions has been fixed. Actually it was a change we made that enabled libraries such as the OpenAPI Explorer to support custom overrides. For the most part it is a drop in replacement for RapiDoc which gives you a lot of the same look and feel but has a number of these more annoying things fixed. It's not a simple 1:1 switch, but if you do find something not supported it would be great to let us know (hopefully those in a new issue ticket)

Thank you @wparad, will take a look!


Can't crawl localhost

Hi 👋🏻
I tried to crawl http://localhost:3000 but looks like i can't: [0000] ERROR Failed to parse domain
How could i fix that please?
Using the latest commit.

+1

Try http://localhost.localdomain
If it doesn't work, You can add that to your /etc/hosts
Something Like this
`[jodis@fedora ~]$ cat /etc/hosts
Loopback entries; do not change.
For historical reasons, localhost precedes localhost.localdomain:
<IP_ADDRESS>   localhost localhost.localdomain localhost4 localhost4.localdomain4
::1         localhost localhost.localdomain localhost6 localhost6.localdomain6
`

imo we shouldn't need to have a to specify an aliased domain for this, we should be able to scan localhost directly ...

imo we shouldn't need to have a to specify an aliased domain for this, we should be able to scan localhost directly ...

Yeah agreed, but I think the issue faced was resolved as soon as there was a 'period'(.) in the domain.
This could be fixed by the Team, but this is like, the temporary fix. I'm not 100% sure that's the reason for the issue, but this seemed to work for me then.


Should I use this repo or one of the forks?

I notice there are 807 forks to this repo.
Looking at Insights -> Contributors, it seems this repository is actively maintained.
Am I best off working from this repository, or are any of the forks noteworthy?
This is a general problem with starting on any open source project with multiple forks. I wish GitHub would display the number of commits in each fork when I list the forks. Otherwise there is no easy way to see how much effort has been put into various forks.

If you take a look at https://github.com/sammchardy/python-binance/network you can see there are only 70-80 forks listed, i belief thats because all the other forks dont have additional commits.
Some of the forks are trying to make something different, removed some dependencies or added some special things and some of them are merged with the main repo. You can observe this with the colored lines.

Hello Oliver, as I found you here I would like to ask you for help. If I use your unicorn-binance-websocket-api together with sammchardy's python-binance, then I get ImportError:
site-packages\unicorn_binance_websocket_api\unicorn_binance_websocket_api_connection.py", line 36, in
from websockets import connect
ImportError: cannot import name 'connect' from 'websockets'
Please help, I can only get both running by removing sammchardy's library from Python PATH before calling imports:
sys.path.remove(pytonInstallFolder + "\Python38\Lib\site-packages\binance")


rdt: use t.Run for running individual test cases

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (bea8ac4) 77.42% compared to head (275772a) 77.42%.

:exclamation: Current head 275772a differs from pull request most recent head 82c0fb8. Consider uploading reports for the commit 82c0fb8 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #117   +/-   ##
=======================================
  Coverage   77.42%   77.42%
=======================================
  Files          18       18
  Lines        1869     1869
=======================================
  Hits         1447     1447
  Misses        310      310
  Partials      112      112

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

@askervin @klihub diffstat looks big but this is essentially a one liner, everything else just indentation changes


[WIP] ✨ added jest-mock-router-v4

This is a react-router-4 mock utility.
Taking https://github.com/Shopify/sell-on-amazon/blob/master/app/ui/tests-utilities/router.ts
from sell-on-amazon and creating jest-mock-router-v4

This is currently failing build, and I also need to add test :)

I'm closing this for now, please feel free to re-open at a better time.


源码导入idea时报错： Incompatible because this component declares a component compatible with Java 11 and the consumer needed a component compatible with Java 8

project : > org.jetbrains.intellij:org.jetbrains.intellij.gradle.plugin:1.1.3
Caused by: org.gradle.internal.component.NoMatchingConfigurationSelectionException: No matching variant of org.jetbrains.intellij.plugins:gradle-intellij-plugin:1.1.3 was found. The consumer was configured to find a runtime of a library compatible with Java 8, packaged as a jar, and its dependencies declared externally but:

Variant 'apiElements' capability org.jetbrains.intellij.plugins:gradle-intellij-plugin:1.1.3 declares a library, packaged as a jar, and its dependencies declared externally:

Incompatible because this component declares an API of a component compatible with Java 11 and the consumer needed a runtime of a component compatible with Java 8

Variant 'runtimeElements' capability org.jetbrains.intellij.plugins:gradle-intellij-plugin:1.1.3 declares a runtime of a library, packaged as a jar, and its dependencies declared externally:

Incompatible because this component declares a component compatible with Java 11 and the consumer needed a component compatible with Java 8

What is your java version? This project required Java 11.


Use of Outdated Vulnerable Component<EMAIL_ADDRESS>

Describe the bug
While analyzing the apk generated of using FreeRasp 6.5.1 ( latest current version )
( Or specifically this android native implementation 'com.aheaditec.talsec.security:TalsecSecurity-Community-Flutter:9.1.0' in android build.gradle )
Analyzing the apk has detected the usage of<EMAIL_ADDRESS>To Reproduce
Use freerasp, analyze the .apk by any security scanning tool ( like MobSF)
Expected behavior
Inside lib/abi_folder/libclib.so files there are strings outlining that openssl version 1.1.1w is used
If Freerasp is confirmed to use this, please update it if possible
Please complete the following information:

Version of freeRASP: 6.5.1

Hello.
Thank you for bringing this issue to our attention. We'll take a closer look at it.

Hello @Maf-Dy ,
the issue has been fixed and will be part of next release.
Kind regards,
Talsec team


able to add blank values in keywords

Issues

If you're having trouble with Articulate we definitely want to help you. If you can provide a little bit of information it will make our troubleshooting job easier.

Noticed this, running some functionality re-testing, that blank keywords can be added.
Please describe the issue you are having here and answer the question below.

Also, this error message as a place holder could perhaps be me clear to just say "can't train because keyword value is blank IMO.
Error: child "examples" fails because ["examples" at position 2 fails because [child "value" fails because ["value" is not allowed to be empty]]]. 😖

0.27.2
win 10 pro
FF
no errors

@dcalvom can you confirm this and get it patched? No reason to interrupt the button work, just whenever you get to it.

Edit: Issue was also present in 27.1

Closed by 21abf670ea29f0ae19a3fca20beddd97b72229fd


SSCS-9787 Update hearing recording requests for other parties

https://tools.hmcts.net/jira/browse/SSCS-9787

Replaced by https://github.com/hmcts/sscs-ccd-definitions/pull/862


chore: small template fixes

Hey @ScapeStudio if you get some time can you check this PR to make sure we
didn't commit something extra by mistake
On Sun, Dec 12, 2021, 1:55 PM Patrick @.***> wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/sunday-club/learning-uniswap/pull/7
Commit Summary

9052ee1
https://github.com/sunday-club/learning-uniswap/pull/7/commits/9052ee1b10bd3abc778a55e86534c440cd7fa405
chore: small template fixes

File Changes
(5 files https://github.com/sunday-club/learning-uniswap/pull/7/files)

M .gitignore
https://github.com/sunday-club/learning-uniswap/pull/7/files#diff-bc37d034bad564583790a46f19d807abfe519c5671395fd494d8cce506c42947
(127)
D .gitignore copy
https://github.com/sunday-club/learning-uniswap/pull/7/files#diff-ab0846f3657c83d3f288e9c23d12c37b419f19f5d4ff9d0a253efddf6396c73c
(35)
D README copy.md
https://github.com/sunday-club/learning-uniswap/pull/7/files#diff-fa8e271dcd166e33780cee65eb09bad3f69f15f629663458ac1a6c1f94a313ed
(114)
M README.md
https://github.com/sunday-club/learning-uniswap/pull/7/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5
(115)
M hardhat.config.ts
https://github.com/sunday-club/learning-uniswap/pull/7/files#diff-57cf5378f8cab20b02b279097ba6fad08eb53f747d81770045dda9c5f95effe3
(9)

Patch Links:

https://github.com/sunday-club/learning-uniswap/pull/7.patch
https://github.com/sunday-club/learning-uniswap/pull/7.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sunday-club/learning-uniswap/pull/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJMBUNUOE6LZABEXUKTXITUQT42BANCNFSM5J4TTWPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

lgtm


Append the sourcefile where the error occurs

which is useful in case of a lot of inclusion (ie BootStrap)

this is a partial bad duplicate of #590 updated in #642


Training fails on Windows 10

The code trains fine on Linux but when I try to train it on Windows 10 with the same steps it results in this error. Any idea what is wrong here?
training start...
  0%|                                   | 0/500000 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "train.py", line 233, in <module>
    train(training_dbs, validation_db, args.start_iter, args.freeze) # 0
  File "train.py", line 158, in train
    = nnet.train(iteration, save, viz_split, **training)
  File "C:\Users\mamoo\OneDrive\Documents\GitHub\TuSimple\LSTR\nnet\py_factory.py", line 111, in train
    loss_kp = self.network(iteration,
  File "C:\Users\mamoo\anaconda3\lib\site-packages\torch\nn\modules\module.py", line 889, in _call_impl
    result = self.forward(*input, **kwargs)
  File "C:\Users\mamoo\OneDrive\Documents\GitHub\TuSimple\LSTR\models\py_utils\data_parallel.py", line 66, in forward
    inputs, kwargs = self.scatter(inputs, kwargs, self.device_ids, self.chunk_sizes)
  File "C:\Users\mamoo\OneDrive\Documents\GitHub\TuSimple\LSTR\models\py_utils\data_parallel.py", line 77, in scatter
    return scatter_kwargs(inputs, kwargs, device_ids, dim=self.dim, chunk_sizes=self.chunk_sizes)
  File "C:\Users\mamoo\OneDrive\Documents\GitHub\TuSimple\LSTR\models\py_utils\scatter_gather.py", line 30, in scatter_kwargs
    inputs = scatter(inputs, target_gpus, dim, chunk_sizes) if inputs else []
  File "C:\Users\mamoo\OneDrive\Documents\GitHub\TuSimple\LSTR\models\py_utils\scatter_gather.py", line 25, in scatter
    return scatter_map(inputs)
  File "C:\Users\mamoo\OneDrive\Documents\GitHub\TuSimple\LSTR\models\py_utils\scatter_gather.py", line 18, in scatter_map
    return list(zip(*map(scatter_map, obj)))
  File "C:\Users\mamoo\OneDrive\Documents\GitHub\TuSimple\LSTR\models\py_utils\scatter_gather.py", line 20, in scatter_map
    return list(map(list, zip(*map(scatter_map, obj))))
  File "C:\Users\mamoo\OneDrive\Documents\GitHub\TuSimple\LSTR\models\py_utils\scatter_gather.py", line 15, in scatter_map
    return Scatter.apply(target_gpus, chunk_sizes, dim, obj)
  File "C:\Users\mamoo\anaconda3\lib\site-packages\torch\nn\parallel\_functions.py", line 93, in forward
    outputs = comm.scatter(input, target_gpus, chunk_sizes, ctx.dim, streams)
  File "C:\Users\mamoo\anaconda3\lib\site-packages\torch\nn\parallel\comm.py", line 189, in scatter
    return tuple(torch._C._scatter(tensor, devices, chunk_sizes, dim, streams))
RuntimeError: start (0) + length (16) exceeds dimension size (1).

I have the same problem. Have you solved it now?

I have the same problem. Have you solved it now?

I tried changing / to \ in all the paths but still not running. I think its because of the slash somewhere because this error was coming when the folder structure wasn't like what the author mentioned. But I cannot find where its coming from. I am running the code on Linux for now.


Remove broken.

It is building fine locally, tested by myself and @SuperSandro2000 (who had added the broken tag).
Should this be tested on hydra before removing it?  I don't know if that is even possible.
Solves #119548

If you would, please reword your commit message to: dmd: mark unbroken (or something to this effect, the important part is the dmd:  prefix.)

Can do.  I change the title.  I am cloning nixpkgs in order to change the commit message.


[harfbuzz] Build error on x64-windows

Just downloaded and installed the latest vcpkg on a x64 Windows 10 machine.
Visuual Studio 14.0 installed on the system.
Installing freetype with no problems, but installing harfbuzz failed.
vcpkg.exe install harfbuzz fails with the following error.
.\vcpkg.exe install harfbuzz
Computing installation plan...
The following packages will be built and installed:
<EMAIL_ADDRESS>Detecting compiler hash for triplet x64-windows...
Compiler found: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64                                                 /cl.exe
Restored 0 package(s) from C:\Users\sven\AppData\Local\vcpkg\archives in 225 us.                                                  Use --debug to see more details.
Installing 1/1<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>-- Found Python version '3.12.7 at C:/Projects/vcpkg/downloads/tools/python/pyth                                                 on-3.12.7-x64/python.exe'
-- Using meson: C:/Projects/vcpkg/downloads/tools/meson-1.6.0-ac58a9/meson.py
-- Using cached harfbuzz-harfbuzz-10.1.0.tar.gz.
-- Cleaning sources at C:/Projects/vcpkg/buildtrees/harfbuzz/src/10.1.0-4a762381                                                 cc.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/Projects/vcpkg/downloads/harfbuzz-harfbuzz-10.1.0.tar.gz
-- Applying patch fix-win32-build.patch
-- Using source at C:/Projects/vcpkg/buildtrees/harfbuzz/src/10.1.0-4a762381cc.c                                                 lean
-- Using cached msys2-mingw-w64-x86_64-pkgconf-1~2.3.0-1-any.pkg.tar.zst.
-- Using cached msys2-msys2-runtime-3.5.4-2-x86_64.pkg.tar.zst.
-- Using msys root at C:/Projects/vcpkg/downloads/tools/msys2/21caed2f81ec917b
-- Configuring x64-windows-dbg
-- Getting CMake variables for x64-windows
-- Configuring x64-windows-dbg done
-- Configuring x64-windows-rel
-- Configuring x64-windows-rel done
-- Package x64-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:127 (message):
Command failed: C:/Projects/vcpkg/downloads/tools/ninja/1.10.2-windows/ninja                                                 .exe install -v
Working Directory: C:/Projects/vcpkg/buildtrees/harfbuzz/x64-windows-dbg
Error code: 1
See logs for more information:
C:\Projects\vcpkg\buildtrees\harfbuzz\package-x64-windows-dbg-out.log
Call Stack (most recent call first):
installed/x64-windows/share/vcpkg-tool-meson/vcpkg_install_meson.cmake:33 (vcp                                                 kg_execute_required_process)
ports/harfbuzz/portfile.cmake:90 (vcpkg_install_meson)
scripts/ports.cmake:192 (include)
error: building harfbuzz:x64-windows failed with: BUILD_FAILED
In the log file the error is related to a not existing type error in some header files.
[1/75] "C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "-Isrc\harfbuzz.dll.p" "-Isrc" "-I..\src\10.1.0-4a762381cc.clean\src" "-I." "-I..\src\10.1.0-4a762381cc.clean" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include/libpng16" "-IC:/Projects/vcpkg/installed/x64-windows/include" "/MDd" "/nologo" "/showIncludes" "/utf-8" "/W2" "/EHs-c-" "/std:c++14" "/Zi" "/wd4244" "/bigobj" "/utf-8" "-DHAVE_CONFIG_H" "-nologo" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "/bigobj" "-DHB_DLL_EXPORT" "/Fdsrc\harfbuzz.dll.p\hb-blob.cc.pdb" /Fosrc/harfbuzz.dll.p/hb-blob.cc.obj "/c" ../src/10.1.0-4a762381cc.clean/src/hb-blob.cc
FAILED: src/harfbuzz.dll.p/hb-blob.cc.obj
"C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "-Isrc\harfbuzz.dll.p" "-Isrc" "-I..\src\10.1.0-4a762381cc.clean\src" "-I." "-I..\src\10.1.0-4a762381cc.clean" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include/libpng16" "-IC:/Projects/vcpkg/installed/x64-windows/include" "/MDd" "/nologo" "/showIncludes" "/utf-8" "/W2" "/EHs-c-" "/std:c++14" "/Zi" "/wd4244" "/bigobj" "/utf-8" "-DHAVE_CONFIG_H" "-nologo" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "/bigobj" "-DHB_DLL_EXPORT" "/Fdsrc\harfbuzz.dll.p\hb-blob.cc.pdb" /Fosrc/harfbuzz.dll.p/hb-blob.cc.obj "/c" ../src/10.1.0-4a762381cc.clean/src/hb-blob.cc
cl : Command line warning D9025 : overriding '/EHs' with '/EHs-'
cl : Command line warning D9025 : overriding '/EHc' with '/EHc-'
cl : Command line warning D9025 : overriding '/Z7' with '/Zi'
cl : Command line warning D9025 : overriding '/EHs-' with '/EHs'
cl : Command line warning D9025 : overriding '/EHc-' with '/EHc'
cl : Command line warning D9025 : overriding '/Zi' with '/Z7'
c:\projects\vcpkg\buildtrees\harfbuzz\src\10.1.0-4a762381cc.clean\src\hb-vector.hh(505): error C2039: 'cmp': is not a member of 'hb_user_data_array_t::hb_user_data_item_t'
c:\projects\vcpkg\buildtrees\harfbuzz\src\10.1.0-4a762381cc.clean\src\hb-object.hh(162): note: see declaration of 'hb_user_data_array_t::hb_user_data_item_t'
c:\projects\vcpkg\buildtrees\harfbuzz\src\10.1.0-4a762381cc.clean\src\hb-object.hh(48): note: see reference to class template instantiation 'hb_vector_t<item_t,false>' being compiled
with
[
item_t=hb_user_data_array_t::hb_user_data_item_t
]
c:\projects\vcpkg\buildtrees\harfbuzz\src\10.1.0-4a762381cc.clean\src\hb-object.hh(174): note: see reference to class template instantiation 'hb_lockable_set_t<hb_user_data_array_t::hb_user_data_item_t,hb_mutex_t>' being compiled
[2/75] "C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "-Isrc\harfbuzz.dll.p" "-Isrc" "-I..\src\10.1.0-4a762381cc.clean\src" "-I." "-I..\src\10.1.0-4a762381cc.clean" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include/libpng16" "-IC:/Projects/vcpkg/installed/x64-windows/include" "/MDd" "/nologo" "/showIncludes" "/utf-8" "/W2" "/EHs-c-" "/std:c++14" "/Zi" "/wd4244" "/bigobj" "/utf-8" "-DHAVE_CONFIG_H" "-nologo" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "/bigobj" "-DHB_DLL_EXPORT" "/Fdsrc\harfbuzz.dll.p\hb-draw.cc.pdb" /Fosrc/harfbuzz.dll.p/hb-draw.cc.obj "/c" ../src/10.1.0-4a762381cc.clean/src/hb-draw.cc
FAILED: src/harfbuzz.dll.p/hb-draw.cc.obj
"C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "-Isrc\harfbuzz.dll.p" "-Isrc" "-I..\src\10.1.0-4a762381cc.clean\src" "-I." "-I..\src\10.1.0-4a762381cc.clean" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include/libpng16" "-IC:/Projects/vcpkg/installed/x64-windows/include" "/MDd" "/nologo" "/showIncludes" "/utf-8" "/W2" "/EHs-c-" "/std:c++14" "/Zi" "/wd4244" "/bigobj" "/utf-8" "-DHAVE_CONFIG_H" "-nologo" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "/bigobj" "-DHB_DLL_EXPORT" "/Fdsrc\harfbuzz.dll.p\hb-draw.cc.pdb" /Fosrc/harfbuzz.dll.p/hb-draw.cc.obj "/c" ../src/10.1.0-4a762381cc.clean/src/hb-draw.cc
cl : Command line warning D9025 : overriding '/EHs' with '/EHs-'
cl : Command line warning D9025 : overriding '/EHc' with '/EHc-'
cl : Command line warning D9025 : overriding '/Z7' with '/Zi'
cl : Command line warning D9025 : overriding '/EHs-' with '/EHs'
cl : Command line warning D9025 : overriding '/EHc-' with '/EHc'
cl : Command line warning D9025 : overriding '/Zi' with '/Z7'
c:\projects\vcpkg\buildtrees\harfbuzz\src\10.1.0-4a762381cc.clean\src\hb-vector.hh(505): error C2039: 'cmp': is not a member of 'hb_user_data_array_t::hb_user_data_item_t'
c:\projects\vcpkg\buildtrees\harfbuzz\src\10.1.0-4a762381cc.clean\src\hb-object.hh(162): note: see declaration of 'hb_user_data_array_t::hb_user_data_item_t'
c:\projects\vcpkg\buildtrees\harfbuzz\src\10.1.0-4a762381cc.clean\src\hb-object.hh(48): note: see reference to class template instantiation 'hb_vector_t<item_t,false>' being compiled
with
[
item_t=hb_user_data_array_t::hb_user_data_item_t
]
c:\projects\vcpkg\buildtrees\harfbuzz\src\10.1.0-4a762381cc.clean\src\hb-object.hh(174): note: see reference to class template instantiation 'hb_lockable_set_t<hb_user_data_array_t::hb_user_data_item_t,hb_mutex_t>' being compiled
[3/75] "C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "-Isrc\harfbuzz.dll.p" "-Isrc" "-I..\src\10.1.0-4a762381cc.clean\src" "-I." "-I..\src\10.1.0-4a762381cc.clean" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include" "-IC:/Projects/vcpkg/installed/x64-windows/debug/../include/libpng16" "-IC:/Projects/vcpkg/installed/x64-windows/include" "/MDd" "/nologo" "/showIncludes" "/utf-8" "/W2" "/EHs-c-" "/std:c++14" "/Zi" "/wd4244" "/bigobj" "/utf-8" "-DHAVE_CONFIG_H" "-nologo" "-DWIN32" "-D_WINDOWS" "-utf-8" "-GR" "-EHsc" "-MP" "-D_DEBUG" "-MDd" "-Z7" "-Ob0" "-Od" "-RTC1" "/bigobj" "-DHB_DLL_EXPORT" "/Fdsrc\harfbuzz.dll.p\hb-common.cc.pdb" /Fosrc/harfbuzz.dll.p/hb-common.cc.obj "/c" ../src/10.1.0-4a762381cc.clean/src/hb-common.cc
FAILED: src/harfbuzz.dll.p/hb-common.cc.obj
I have no idea how to fix this, tried several times re-installation of the whole vcpkg with no success.

Duplicate of https://github.com/microsoft/vcpkg/issues/37940, you could try to the high version of Visual Studio to install harfbuzz.

Of course, installing the latest Visual Studio and updating the libraries works fine, but this does not solve the problem if I want to use a specific Visual Studio version. The problem is that it's not easy to  download and install a specific vcpkg version (matching the Visual Studio version), so the only solution is updating Visual Studio.

This is an upstream issue. There is no magic in vcpkg making packages more compatible with older toolchains than supported by upstream.


DONT MERGE: Test commit

(cherry picked from commit fa6db4ea9e9499d58aabb841df9bad787819c99c)
Merge of #536 to 'master'

@kaa-leeroy test this fully please

@kaa-leeroy test this please


Unloading audio may lead to crashes when or after loading a level

FMOD issue. Audio unloading is disabled for the timebeing.

UPDATE: Music seems unaffected by this issue, only Sounds remain.


Error Building Jaxlib v0.4.30 on Jetson Orin

Description
When I attempt to build jaxlib v0.4.30 on a Jetson Orin with the command python build/build.py --enable_cuda --cuda_path /usr/local/cuda-12.2 --cudnn_path /usr/lib/aarch64-linux-gnu --cuda_version 12.2 --cudnn_version 8
I get the error
ERROR: /home/nvidia/.cache/bazel/_bazel_nvidia/a5643b5cc286b9b13a96818003a4a7dd/external/xla/xla/stream_executor/gpu/BUILD:349:19: Compiling xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc failed: (Exit 4): crosstool_wrapper_driver_is_not_gcc failed: error executing command (from target @xla//xla/stream_executor/gpu:gpu_timer_kernel_cuda)
The full output is below
python build/build.py --enable_cuda --cuda_path /usr/local/cuda-12.2 --cudnn_path /usr/lib/aarch64-linux-gnu --cuda_version 12.2 --cudnn_version 8

     _   _  __  __
    | | / \ \ \/ /
 _  | |/ _ \ \  /
| |_| / ___ \/  \
 \___/_/   \/_/\_\

Bazel binary path: ./bazel-6.5.0-linux-arm64
Bazel version: 6.5.0
Python binary path: /home/nvidia/jax/jax_env/bin/python
Python version: 3.10
Use clang: no
MKL-DNN enabled: yes
Target CPU: aarch64
Target CPU features: release
CUDA enabled: yes
CUDA toolkit path: /usr/local/cuda-12.2
CUDNN library path: /usr/lib/aarch64-linux-gnu
CUDA version: 12.2
CUDNN version: 8
NCCL enabled: yes
ROCm enabled: no

Building XLA and installing it in the jaxlib source tree...
./bazel-6.5.0-linux-arm64 run --verbose_failures=true //jaxlib/tools:build_wheel -- --output_path=/home/nvidia/jax/dist --jaxlib_git_hash=f4158ace933482844c145a6b919bf5dc86e084ba --cpu=aarch64
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=0 --terminal_columns=80
INFO: Reading rc options for 'run' from /home/nvidia/jax/.bazelrc:
  Inherited 'common' options: --experimental_repo_remote_exec
INFO: Reading rc options for 'run' from /home/nvidia/jax/.bazelrc:
  Inherited 'build' options: --nocheck_visibility --apple_platform_type=macos --macos_minimum_os=10.14 --announce_rc --define open_source_build=true --spawn_strategy=standalone --enable_platform_specific_config --experimental_cc_shared_library --define=no_aws_support=true --define=no_gcp_support=true --define=no_hdfs_support=true --define=no_kafka_support=true --define=no_ignite_support=true --define=grpc_no_ares=true --define=tsl_link_protobuf=true -c opt --config=short_logs --copt=-DMLIR_PYTHON_PACKAGE_PREFIX=jaxlib.mlir. --@xla//xla/python:enable_gpu=false
INFO: Reading rc options for 'run' from /home/nvidia/jax/.jax_configure.bazelrc:
  Inherited 'build' options: --strategy=Genrule=standalone --action_env CUDA_TOOLKIT_PATH=/usr/local/cuda-12.2 --action_env CUDNN_INSTALL_PATH=/usr/lib/aarch64-linux-gnu --action_env TF_CUDA_PATHS=/usr/local/cuda-12.2,/usr/lib/aarch64-linux-gnu --action_env TF_CUDA_VERSION=12.2 --action_env TF_CUDNN_VERSION=8 --config=mkl_open_source_only --config=cuda --repo_env HERMETIC_PYTHON_VERSION=3.10
INFO: Found applicable config definition build:short_logs in file /home/nvidia/jax/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:mkl_open_source_only in file /home/nvidia/jax/.bazelrc: --define=tensorflow_mkldnn_contraction_kernel=1
INFO: Found applicable config definition build:cuda in file /home/nvidia/jax/.bazelrc: --repo_env TF_NEED_CUDA=1 --repo_env TF_NCCL_USE_STUB=1 --action_env TF_CUDA_COMPUTE_CAPABILITIES=sm_50,sm_60,sm_70,sm_80,compute_90 --crosstool_top=@local_config_cuda//crosstool:toolchain --@local_config_cuda//:enable_cuda --@xla//xla/python:enable_gpu=true --@xla//xla/python:jax_cuda_pip_rpaths=true --define=xla_python_enable_gpu=true --linkopt=-Wl,--disable-new-dtags
INFO: Found applicable config definition build:linux in file /home/nvidia/jax/.bazelrc: --config=posix --copt=-Wno-unknown-warning-option --copt=-Wno-stringop-truncation --copt=-Wno-array-parameter
INFO: Found applicable config definition build:posix in file /home/nvidia/jax/.bazelrc: --copt=-fvisibility=hidden --copt=-Wno-sign-compare --cxxopt=-std=c++17 --host_cxxopt=-std=c++17
Loading:
DEBUG: /home/nvidia/.cache/bazel/_bazel_nvidia/a5643b5cc286b9b13a96818003a4a7dd/external/bazel_tools/tools/cpp/lib_cc_configure.bzl:118:10:
Auto-Configuration Warning: 'TMP' environment variable is not set, using 'C:\Windows\Temp' as default
DEBUG: /home/nvidia/.cache/bazel/_bazel_nvidia/a5643b5cc286b9b13a96818003a4a7dd/external/bazel_tools/tools/cpp/lib_cc_configure.bzl:118:10:
Auto-Configuration Warning: 'TMP' environment variable is not set, using 'C:\Windows\Temp' as default
Loading:
Loading: 2 packages loaded
Analyzing: target //jaxlib/tools:build_wheel (3 packages loaded, 0 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (36 packages loaded, 10 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (36 packages loaded, 10 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (36 packages loaded, 10 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (55 packages loaded, 216 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (57 packages loaded, 546 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (91 packages loaded, 3986 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (123 packages loaded, 4533 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (137 packages loaded, 4966 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (156 packages loaded, 5504 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (159 packages loaded, 7525 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (180 packages loaded, 9740 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (208 packages loaded, 13196 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (219 packages loaded, 16021 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (225 packages loaded, 18175 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (236 packages loaded, 20734 targets configured)
Analyzing: target //jaxlib/tools:build_wheel (238 packages loaded, 22135 targets configured)
INFO: Analyzed target //jaxlib/tools:build_wheel (239 packages loaded, 23156 targets configured).
 checking cached actions
INFO: Found 1 target...
[1 / 5] [Prepa] BazelWorkspaceStatusAction stable-status.txt
[24 / 3,232] Compiling third_party/f2reduce/f2reduce.cpp; 0s local ... (2 actions, 1 running)
[56 / 3,297] Compiling third_party/f2reduce/f2reduce.cpp; 1s local ... (4 actions, 3 running)
[182 / 3,363] Compiling llvm/lib/Demangle/RustDemangle.cpp [for tool]; 0s local ... (4 actions, 3 running)
[792 / 3,598] [Prepa] Executing genrule @local_config_cuda//cuda:cuda-lib; 4s ... (5 actions, 3 running)
[1,699 / 3,610] Executing genrule @local_config_cuda//cuda:cuda-lib; 1s local ... (5 actions, 4 running)
[1,824 / 3,642] Executing genrule @local_config_cuda//cuda:cuda-lib; 2s local ... (6 actions, 5 running)
[1,839 / 3,658] Executing genrule @local_config_cuda//cuda:cuda-lib; 3s local ... (6 actions running)
[1,844 / 3,661] Executing genrule @local_config_cuda//cuda:cuda-lib; 4s local ... (6 actions running)
[1,846 / 3,661] Executing genrule @local_config_cuda//cuda:cuda-lib; 5s local ... (6 actions running)
[1,854 / 3,704] Executing genrule @local_config_cuda//cuda:cuda-lib; 6s local ... (6 actions running)
[1,860 / 3,719] Compiling src/google/protobuf/wire_format_lite.cc [for tool]; 1s local ... (6 actions running)
[1,869 / 3,762] Compiling src/google/protobuf/wire_format_lite.cc [for tool]; 2s local ... (4 actions running)
[1,870 / 3,806] Compiling llvm/utils/TableGen/Attributes.cpp [for tool]; 2s local ... (6 actions running)
[2,561 / 5,511] Compiling llvm/lib/Demangle/ItaniumDemangle.cpp; 3s local ... (4 actions running)
[3,603 / 5,645] Compiling llvm/lib/Demangle/ItaniumDemangle.cpp; 4s local ... (5 actions running)
[3,675 / 5,873] Compiling llvm/lib/Demangle/ItaniumDemangle.cpp; 5s local ... (4 actions running)
[3,707 / 6,267] Compiling llvm/lib/Demangle/ItaniumDemangle.cpp; 6s local ... (6 actions running)
[3,709 / 6,267] Compiling llvm/lib/Demangle/ItaniumDemangle.cpp; 7s local ... (6 actions running)
[3,712 / 6,267] Compiling llvm/utils/TableGen/Basic/CodeGenIntrinsics.cpp [for tool]; 4s local ... (6 actions running)
[3,715 / 6,417] Compiling llvm/utils/TableGen/Basic/CodeGenIntrinsics.cpp [for tool]; 5s local ... (6 actions running)
[3,723 / 6,923] Compiling xla/service/gpu/stream_executor_util_kernel.cu.cc; 4s local ... (5 actions running)
[3,725 / 7,584] Compiling xla/service/gpu/stream_executor_util_kernel.cu.cc; 5s local ... (6 actions running)
[3,726 / 7,584] Compiling xla/service/gpu/stream_executor_util_kernel.cu.cc; 6s local ... (6 actions running)
[3,731 / 7,584] Compiling xla/service/gpu/stream_executor_util_kernel.cu.cc; 7s local ... (6 actions running)
[3,735 / 7,584] Compiling xla/service/gpu/stream_executor_util_kernel.cu.cc; 8s local ... (6 actions running)
[3,736 / 7,584] Compiling xla/service/gpu/stream_executor_util_kernel.cu.cc; 9s local ... (6 actions, 5 running)
[3,747 / 8,596] Compiling xla/service/gpu/stream_executor_util_kernel.cu.cc; 11s local ... (6 actions, 5 running)
[3,754 / 8,683] Compiling llvm/utils/TableGen/DirectiveEmitter.cpp [for tool]; 5s local ... (6 actions, 5 running)
.
.
.
[9,790 / 14,610] Compiling xla/service/gpu/kernels/topk_kernel_bfloat16.cu.cc; 468s local ... (6 actions running)
ERROR: /home/nvidia/.cache/bazel/_bazel_nvidia/a5643b5cc286b9b13a96818003a4a7dd/external/xla/xla/stream_executor/gpu/BUILD:349:19: Compiling xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc failed: (Exit 4): crosstool_wrapper_driver_is_not_gcc failed: error executing command (from target @xla//xla/stream_executor/gpu:gpu_timer_kernel_cuda)
  (cd /home/nvidia/.cache/bazel/_bazel_nvidia/a5643b5cc286b9b13a96818003a4a7dd/execroot/__main__ && \
  exec env - \
    CUDA_TOOLKIT_PATH=/usr/local/cuda-12.2 \
    CUDNN_INSTALL_PATH=/usr/lib/aarch64-linux-gnu \
    LD_LIBRARY_PATH=/home/nvidia/colcon_ws/install/zed_components/lib:/home/nvidia/colcon_ws/install/zed_interfaces/lib:/home/nvidia/colcon_ws/install/vesc_driver/lib:/home/nvidia/colcon_ws/install/vesc_ackermann/lib:/home/nvidia/colcon_ws/install/custom_ackermann/lib:/home/nvidia/colcon_ws/install/vesc_msgs/lib:/home/nvidia/colcon_ws/install/teleop_tools_msgs/lib:/opt/ros/humble/opt/rviz_ogre_vendor/lib:/opt/ros/humble/lib/aarch64-linux-gnu:/opt/ros/humble/lib:/usr/local/cuda-12.2/lib64: \
    PATH=/home/nvidia/jax/jax_env/bin:/opt/ros/humble/bin:/usr/local/cuda-12.2/bin:/home/nvidia/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/snap/bin \
    PWD=/proc/self/cwd \
    TF_CUDA_COMPUTE_CAPABILITIES=sm_50,sm_60,sm_70,sm_80,compute_90 \
    TF_CUDA_PATHS=/usr/local/cuda-12.2,/usr/lib/aarch64-linux-gnu \
    TF_CUDA_VERSION=12.2 \
    TF_CUDNN_VERSION=8 \
  external/local_config_cuda/crosstool/clang/bin/crosstool_wrapper_driver_is_not_gcc -MD -MF bazel-out/aarch64-opt/bin/external/xla/xla/stream_executor/gpu/_objs/gpu_timer_kernel_cuda/gpu_timer_kernel_cuda.cu.pic.d '-frandom-seed=bazel-out/aarch64-opt/bin/external/xla/xla/stream_executor/gpu/_objs/gpu_timer_kernel_cuda/gpu_timer_kernel_cuda.cu.pic.o' '-DEIGEN_MAX_ALIGN_BYTES=64' -DEIGEN_ALLOW_UNALIGNED_SCALARS '-DEIGEN_USE_AVX512_GEMM_KERNELS=0' -DHAVE_SYS_UIO_H -DTF_USE_SNAPPY '-DGOOGLE_CUDA=1' '-DBAZEL_CURRENT_REPOSITORY="xla"' -iquote external/xla -iquote bazel-out/aarch64-opt/bin/external/xla -iquote external/tsl -iquote bazel-out/aarch64-opt/bin/external/tsl -iquote external/eigen_archive -iquote bazel-out/aarch64-opt/bin/external/eigen_archive -iquote external/ml_dtypes -iquote bazel-out/aarch64-opt/bin/external/ml_dtypes -iquote external/com_google_absl -iquote bazel-out/aarch64-opt/bin/external/com_google_absl -iquote external/nsync -iquote bazel-out/aarch64-opt/bin/external/nsync -iquote external/local_config_cuda -iquote bazel-out/aarch64-opt/bin/external/local_config_cuda -iquote external/com_google_protobuf -iquote bazel-out/aarch64-opt/bin/external/com_google_protobuf -iquote external/zlib -iquote bazel-out/aarch64-opt/bin/external/zlib -iquote external/double_conversion -iquote bazel-out/aarch64-opt/bin/external/double_conversion -iquote external/snappy -iquote bazel-out/aarch64-opt/bin/external/snappy -iquote external/com_googlesource_code_re2 -iquote bazel-out/aarch64-opt/bin/external/com_googlesource_code_re2 -Ibazel-out/aarch64-opt/bin/external/ml_dtypes/_virtual_includes/float8 -Ibazel-out/aarch64-opt/bin/external/ml_dtypes/_virtual_includes/intn -Ibazel-out/aarch64-opt/bin/external/local_config_cuda/cuda/_virtual_includes/cuda_headers_virtual -isystem external/eigen_archive -isystem bazel-out/aarch64-opt/bin/external/eigen_archive -isystem external/eigen_archive/mkl_include -isystem bazel-out/aarch64-opt/bin/external/eigen_archive/mkl_include -isystem external/ml_dtypes -isystem bazel-out/aarch64-opt/bin/external/ml_dtypes -isystem external/ml_dtypes/ml_dtypes -isystem bazel-out/aarch64-opt/bin/external/ml_dtypes/ml_dtypes -isystem external/nsync/public -isystem bazel-out/aarch64-opt/bin/external/nsync/public -isystem external/local_config_cuda/cuda -isystem bazel-out/aarch64-opt/bin/external/local_config_cuda/cuda -isystem external/local_config_cuda/cuda/cuda/include -isystem bazel-out/aarch64-opt/bin/external/local_config_cuda/cuda/cuda/include -isystem external/com_google_protobuf/src -isystem bazel-out/aarch64-opt/bin/external/com_google_protobuf/src -isystem external/zlib -isystem bazel-out/aarch64-opt/bin/external/zlib -Wno-builtin-macro-redefined '-D__DATE__="redacted"' '-D__TIMESTAMP__="redacted"' '-D__TIME__="redacted"' -fPIC -U_FORTIFY_SOURCE '-D_FORTIFY_SOURCE=1' -fstack-protector -Wall -fno-omit-frame-pointer -no-canonical-prefixes -fno-canonical-system-headers -DNDEBUG -g0 -O2 -ffunction-sections -fdata-sections '-fvisibility=hidden' -Wno-sign-compare -Wno-unknown-warning-option -Wno-stringop-truncation -Wno-array-parameter '-DMLIR_PYTHON_PACKAGE_PREFIX=jaxlib.mlir.' '-std=c++17' -x cuda '-DGOOGLE_CUDA=1' '--cuda-gpu-arch=sm_50' '--cuda-gpu-arch=sm_60' '--cuda-gpu-arch=sm_70' '--cuda-gpu-arch=sm_80' '--cuda-include-ptx=sm_90' '--cuda-gpu-arch=sm_90' '-Xcuda-fatbinary=--compress-all' '-nvcc_options=expt-relaxed-constexpr' -c external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc -o bazel-out/aarch64-opt/bin/external/xla/xla/stream_executor/gpu/_objs/gpu_timer_kernel_cuda/gpu_timer_kernel_cuda.cu.pic.o)
# Configuration: 39ab92a7342907b6f4e73040033499a7cf871005fd5792c463496975725a1dc4
# Execution platform: @local_execution_config_platform//:platform
In file included from external/xla/xla/stream_executor/stream_executor.h:2,
                 from external/xla/xla/stream_executor/stream_executor_common.h:29,
                 from external/xla/xla/stream_executor/gpu/gpu_executor.h:60,
                 from external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc:19:
external/com_google_absl/absl/log/log.h:199: warning: "LOG" redefined
  199 | #define LOG(severity) ABSL_LOG_INTERNAL_LOG_IMPL(_##severity)
      |
In file included from external/tsl/tsl/platform/logging.h:26,
                 from external/xla/xla/stream_executor/device_memory.h:33,
                 from external/xla/xla/stream_executor/blas.h:35,
                 from external/xla/xla/stream_executor/gpu/gpu_executor.h:44,
                 from external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc:19:
external/tsl/tsl/platform/default/logging.h:165: note: this is the location of the previous definition
  165 | #define LOG(severity) _TF_LOG_##severity
      |
In file included from external/xla/xla/stream_executor/stream_executor.h:2,
                 from external/xla/xla/stream_executor/stream_executor_common.h:29,
                 from external/xla/xla/stream_executor/gpu/gpu_executor.h:60,
                 from external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc:19:
external/com_google_absl/absl/log/log.h:237: warning: "LOG_EVERY_N" redefined
  237 | #define LOG_EVERY_N(severity, n) \
      |
In file included from external/tsl/tsl/platform/logging.h:26,
                 from external/xla/xla/stream_executor/device_memory.h:33,
                 from external/xla/xla/stream_executor/blas.h:35,
                 from external/xla/xla/stream_executor/gpu/gpu_executor.h:44,
                 from external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc:19:
external/tsl/tsl/platform/default/logging.h:278: note: this is the location of the previous definition
  278 | #define LOG_EVERY_N(severity, n)                       \
      |
In file included from external/xla/xla/stream_executor/stream_executor.h:2,
                 from external/xla/xla/stream_executor/stream_executor_common.h:29,
                 from external/xla/xla/stream_executor/gpu/gpu_executor.h:60,
                 from external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc:19:
external/com_google_absl/absl/log/log.h:245: warning: "LOG_FIRST_N" redefined
  245 | #define LOG_FIRST_N(severity, n) \
      |
In file included from external/tsl/tsl/platform/logging.h:26,
                 from external/xla/xla/stream_executor/device_memory.h:33,
                 from external/xla/xla/stream_executor/blas.h:35,
                 from external/xla/xla/stream_executor/gpu/gpu_executor.h:44,
                 from external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc:19:
external/tsl/tsl/platform/default/logging.h:284: note: this is the location of the previous definition
  284 | #define LOG_FIRST_N(severity, n)                       \
      |
In file included from external/xla/xla/stream_executor/stream_executor.h:2,
                 from external/xla/xla/stream_executor/stream_executor_common.h:29,
                 from external/xla/xla/stream_executor/gpu/gpu_executor.h:60,
                 from external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc:19:
external/com_google_absl/absl/log/log.h:253: warning: "LOG_EVERY_POW_2" redefined
  253 | #define LOG_EVERY_POW_2(severity) \
      |
In file included from external/tsl/tsl/platform/logging.h:26,
                 from external/xla/xla/stream_executor/device_memory.h:33,
                 from external/xla/xla/stream_executor/blas.h:35,
                 from external/xla/xla/stream_executor/gpu/gpu_executor.h:44,
                 from external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc:19:
external/tsl/tsl/platform/default/logging.h:290: note: this is the location of the previous definition
  290 | #define LOG_EVERY_POW_2(severity)                         \
      |
In file included from external/xla/xla/stream_executor/stream_executor.h:2,
                 from external/xla/xla/stream_executor/stream_executor_common.h:29,
                 from external/xla/xla/stream_executor/gpu/gpu_executor.h:60,
                 from external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc:19:
external/com_google_absl/absl/log/log.h:265: warning: "LOG_EVERY_N_SEC" redefined
  .
.
.
  536 | #define DCHECK_GT(x, y) _TF_DCHECK_NOP(x, y)
      |
external/com_google_absl/absl/status/status.h(796): warning #2810-D: ignoring return value type with "nodiscard" attribute
      *this = std::move(new_status);
            ^

Remark: The warnings can be suppressed with "-diag-suppress <warning-number>"

external/com_google_absl/absl/status/internal/statusor_internal.h(240): warning #2810-D: ignoring return value type with "nodiscard" attribute
        status_ = OkStatus();
                ^

external/com_google_absl/absl/status/internal/statusor_internal.h(247): warning #2810-D: ignoring return value type with "nodiscard" attribute
      status_ = static_cast<absl::Status>(std::forward<U>(v));
              ^

/usr/lib/gcc/aarch64-linux-gnu/11/include/arm_neon.h(38): error: identifier "__Int8x8_t" is undefined
  typedef __Int8x8_t int8x8_t;
          ^

/usr/lib/gcc/aarch64-linux-gnu/11/include/arm_neon.h(39): error: identifier "__Int16x4_t" is undefined
  typedef __Int16x4_t int16x4_t;
          ^

/usr/lib/gcc/aarch64-linux-gnu/11/include/arm_neon.h(40): error: identifier "__Int32x2_t" is undefined
  typedef __Int32x2_t int32x2_t;
          ^

/usr/lib/gcc/aarch64-linux-gnu/11/include/arm_neon.h(41): error: identifier "__Int64x1_t" is undefined
  typedef __Int64x1_t int64x1_t;
          ^

.
.
.
/usr/lib/gcc/aarch64-linux-gnu/11/include/arm_neon.h(1308): error: identifier "__builtin_aarch64_addhn2v4si" is undefined
    return (uint16x8_t) __builtin_aarch64_addhn2v4si ((int16x4_t) __a,
                        ^

/usr/lib/gcc/aarch64-linux-gnu/11/include/arm_neon.h(1317): error: identifier "__builtin_aarch64_addhn2v2di" is undefined
    return (uint32x4_t) __builtin_aarch64_addhn2v2di ((int32x2_t) __a,
                        ^

Error limit reached.
100 errors detected in the compilation of "external/xla/xla/stream_executor/gpu/gpu_timer_kernel_cuda.cu.cc".
Compilation terminated.
Target //jaxlib/tools:build_wheel failed to build
INFO: Elapsed time: 1536.946s, Critical Path: 471.27s
INFO: 3903 processes: 22 internal, 3881 local.
FAILED: Build did NOT complete successfully
ERROR: Build failed. Not running target
Traceback (most recent call last):
  File "/home/nvidia/jax/build/build.py", line 726, in <module>
    main()
  File "/home/nvidia/jax/build/build.py", line 692, in main
    shell(build_cpu_wheel_command)
  File "/home/nvidia/jax/build/build.py", line 45, in shell
    output = subprocess.check_output(cmd)
  File "/usr/lib/python3.10/subprocess.py", line 421, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/usr/lib/python3.10/subprocess.py", line 526, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['./bazel-6.5.0-linux-arm64', 'run', '--verbose_failures=true', '//jaxlib/tools:build_wheel', '--', '--output_path=/home/nvidia/jax/dist', '--jaxlib_git_hash=f4158ace933482844c145a6b919bf5dc86e084ba', '--cpu=aarch64']' returned non-zero exit status 1.

System info (python version, jaxlib version, accelerator, etc.)
Bazel binary path: ./bazel-6.5.0-linux-arm64
Bazel version: 6.5.0
Python binary path: /home/nvidia/jax/jax_env/bin/python
Python version: 3.10
Use clang: no
MKL-DNN enabled: yes
Target CPU: aarch64
Target CPU features: release
CUDA enabled: yes
CUDA toolkit path: /usr/local/cuda-12.2
CUDNN library path: /usr/lib/aarch64-linux-gnu
CUDA version: 12.2
CUDNN version: 8
NCCL enabled: yes
ROCm enabled: no

I am using a Jetson Orin Nano with Tegra GPU running Jetpack 6
Cuda 12.2
cuDNN 8

Seeing the same issue on a Jetson Orin AGX with JP6. I think it should be fixable with the patch from https://github.com/tensorflow/tensorflow/issues/62490 (applied to xla -> tsl -> absl). I'm building with the patch now, I'll comment again if this fixes the issue.

FYI the above workaround works as a one-off; it would be nice to get the fix upstreamed once https://github.com/abseil/abseil-cpp/issues/1665 is resolved.

Thanks Kyle!

@kylestach Hi, I am facing the same issue here with the Jetson Orin Nano. Did you figure out ?

Hey @kylestach , I also stumbled upon the same problem and I don't get the solution. Maybe you could go into detail how exactly the fix works?

Note that this has been fixed upstream in absl: https://github.com/abseil/abseil-cpp/pull/1732. Maybe we can bump the vendored copy of absl before the next release?

There is already a PR to add this patch in XLA:
https://github.com/openxla/xla/issues/13924
So it should be fixed soon.

Did you link the wrong PR?

Updated my previous comment.

In that case can we bump the vendored copy of XLA? Or just stop vendoring things so it's no longer a JAX bug?

JAX seem to update the XLA commit everyday or so.
I looked, current JAX upstream point to an XLA that has the patch.
So taking today/tomorrow JAX nightly or building upstream should get you the fix.
If you try, can you confirm the fix works for you?

it is still not building...

openxla/xla#15687

how to apply it?

@jwknaup do yo know how to apply this patch while it is building?

I believe you have to copy the files over and overwrite the existing ones in your local XLA code. It's been a while, though, so I don't remember what exactly that looked like or where they were. Sorry, I'm not much help. 😢

I believe you have to copy the files over and overwrite the existing ones in your local XLA code. It's been a while, though, so I don't remember what exactly that looked like or where they were. Sorry, I'm not much help. 😢

well, I will wait for next release... because they change a lot of cuda gpus

I've studied last versions and now I did some progress. There are a bug that clang no detect automatically version of cuda, so in case that put only path not works. All must be match. Also

, be carefule with cudnn headers that are different.
the command is:
python3 build/build.py --enable_cuda --enable_nccl=False \
        --cuda_compute_capabilities=sm_87 \
        --cuda_version=12.2.0 --cudnn_version=8.9.4 \
        --bazel_options=--repo_env=LOCAL_CUDA_PATH="/usr/local/cuda-12.2" \
        --bazel_options=--repo_env=LOCAL_CUDNN_PATH="/usr/lib/aarch64-linux-gnu"

If you upgrade LLVM/clang first, that compiler error goes away (in my case with LLVM 18)
https://github.com/dusty-nv/jetson-containers/commit/2f22c19b04b3e4468430373887b52c1218225246#diff-d78aa7ea719da213abd6f6c1dba0b5a3a7c7aed87a5551fcbaccd86cabd1b5d1R7

@jwknaup you have now the wheels on http://jetson.webredirect.org/jp6/cu126.
Please if you get error
install:
pip3 install --no_deps *.whl

Close the issue :)


Accelerate Query::get

Objective

query::get is commonly used to retrieve an item from a specific entity. However, its performance may not be optimal when dealing with a group of entities, particularly for wide queries. In fact, it may even be slower than EntityHashmap.

Solution

In real-world scenarios, it's highly probable that the next entity belongs to the same archetype as the previous one , Therefore, caching the fetch operation can lead to faster retrieval.

Changelog

Added QueryEntityGetter
Added Query::getter(_mut)
Added QueryState::getter(_mut)

Performance

Best case (all entity in the same archetype),  2x-4x faster with  QueryEntityGetter::get , 4x-10x faster with QueryEntityGetter::get_unchecked .
Worst case (every entity with different archetypes and using safety get ), it has 10%~20% regresion.

Thanks for taking a swing at this! Haven't had the time for a full review yet, but a bit of a bikeshed on the name QueryEntityGetter -> PointQuery.
This should be done in a followup PR, but PointQuery implementing SystemParam would also be useful, though it may not be intuitive to use since immutable fetches require a mutable struct.

I did a benchmark on many_foxes with this PR, and I didn't observe any performance differences in propagate_transform. Only the use of the unsafe version seemed to yield around a 10% improvement.
TBH, the greatest value of this PR lies in providing a more versatile method for retrieving items from many entities than iter_many. However, I'm not certain if this approach is worthy.

Might want to test this against the visibility propagation system since that runs single-threaded and thus can needing fetch reinitialization as often as the transform propagate system.

Might want to test this against the visibility propagation system since that runs single-threaded and thus can needing fetch reinitialization as often as the transform propagate system.

run many_foxes with all entities Visibility changed every frame,
system{name="bevy_render::view::visibility::visibility_propagate_system"}
from 400us to 240 us


Use Pylint

Hi @yasmensarhan27,
Please use pylint to check our code. We are supposed to put the output of pylint in documentation.

@yasmensarhan27 I did pylint but could not get output from pytest. Also have a look at my latest pull request. I did some corrections to the code you uploaded.


Portfolio card title click

Changes

make only card title link
added word break for long words in card description
removed some console log leftover

Before

After

Codecov Report

Merging #266 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #266   +/-   ##
=======================================
  Coverage   83.22%   83.22%
=======================================
  Files         102      103    +1
  Lines        1192     1192
  Branches      101      102    +1
=======================================
  Hits          992      992
  Misses        181      181
  Partials       19       19

Impacted Files
Coverage Δ

...esentational-components/shared/conditional-link.js
100% <100%> (ø)

...ational-components/shared/service-offering-body.js
100% <100%> (ø)
:arrow_up:

src/smart-components/portfolio/portfolio-item.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


admin.TabularInline doesn't show anything

The inline shows on Django default and Grappelli, however, the entire section is empty on django-material

Yep, formsets is not supported yet

duplicate of  #1


A little bug in the function "cvui::rect"

Hello,When i running the exmple of mouse.cpp, i found a little bug in the function "cvui::rect",

when the value of the thePos.width and thtPos.height is zeros whill case the function addWeighted crushed ,so i use the following method to fix that

Hello and thank you for your suggestion! I'll implement your fix as soon as I have some free time to work on cvui.


[cxx-interop] Copy libstdc++ resources again if the sources changed

Previously if you changed libstdcxx.h or libstdcxx.modulemap locally and ran an incremental build, the artifacts weren't updated.

@swift-ci please smoke test


[MRG] decim parameter for epochs.plot.

Continuation to https://github.com/mne-tools/mne-python/pull/4346.
Also includes fixes to dead links.

works like a charm for me.
+1 for MRG

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3dc1962). Click here to learn what that means.
The diff coverage is 85.71%.

@@            Coverage Diff            @@
##             master    #4349   +/-   ##
=========================================
  Coverage          ?   86.06%
=========================================
  Files             ?      346
  Lines             ?    63068
  Branches          ?     9692
=========================================
  Hits              ?    54282
  Misses            ?     6116
  Partials          ?     2670

Thanks @jaeilepp !


Unreachable or Non-Existent Script in Profile

csb-accounts/profile.php calls for a non-existent or unreachable csb-content/js/profile.js returning a net::ERR_ABORTED 404 (Not Found) error in the Dev Console.
Browser: Google Chrome
OS: Windows 10

Fixed in #67


DefaultHttpHeader.names().toArray(...) may throw ArrayStoreException

Motivation:
DefaultHttpHeader.names() exposes HTTP header names as a Set. Converting the resulting set to an array using toArray(String[]) throws an exception: java.lang.ArrayStoreException: io.netty.util.AsciiString.
Modifications:

Remove our custom implementation of toArray(...) (and others) by just extending AbstractCollection.
Add unit test

Result:
Fixes [#7428].

@mikkokar PTAL as well

Thank you @normanmaurer for fixing this. That's what I also did to fillArray locally.

@Scottmitch @carl-mastrangelo please note that I found an "easier" fix which also takes care of correctly creating the array and remove a lot of code. I just extended AbstractCollection and so we get all for free ;) PTAL again

Cherry-picked into 4.1 (7aca99f9869634d1f65e017e62b5520cd8df8cf5). 4.0 is not affected

@normanmaurer +1


e2e:End-to-end test for the streaming pipeline

This PR contains stream mode tests for both parquet and fhir sink.
To test the stream mode, am counting the number of records in the source specifically Patients, Encounters and Observations and then am comparing this with the total number of sinked items in both parque and fhir sink
To test the parquet sink first I need to create patent, obs and encounter resources in openmrs, then doing a rowCount using the parquet-tool merge utility in the parque files directory where the patient resource is sinked. Am also doing the same for the obs and encounter resource to ensure the value in openmrs is equal to that in the sink directory
To test the fhir server stream sink am posting rest api requests to create a patient,encounter and obs resource using curl and then checking to verify that the exact resources are streamed to the hapi fhir

@gitcliff if you have created this PR for testing only, can you please change it to Draft and also make it clear at the top of you PR description?

@bashir2 @ibacher kindly review

@bashir2 kindly review

@bashir2 requesting for review

@bashir2 requesting for some review

@bashir2 requesting for review

@bashir2 when i add the ?_summary=count after the query parameters iehttp://localhost:8099/openmrs/ws/fhir2/R4/Patient/$1/?_summary=counti get the count of 0 patients,encounters and obs in the openmrs server. I though for the parquet streaming we are only concerned with validating the presence of the streamed resource in the parquet files

Closing as this is superseded by PR #218.


Client becomes unauthorized after 15m

I'm creating a client with a private key and application ID.  Instead of a creating a new client for each message, the client object is reused.  I added print statements into the code and verified that for every request, a new JWT is generated with a new JTI, ITA, and EXP.  After 15 minutes, sending a message errors out with an Authorization Error.  I noticed that the expiration time of the JWT is 15 minutes, but since the client is generating a new JWT for each request, I did not expect an authorization error.
Expected Behavior
The authorization should not fail after 15 minutes
Possible Solution
A workaround is to create a new client object for each request.
Steps to Reproduce (for bugs)

Create a client
send a message
wait for at least 15 minutes
send another message with the same client

Context
Just trying to send a message
Your Environment

Version used:
Python 3.10.10
vonage==3.13.1
vonage-jwt==1.1.0

Operating System and version: seen in Windows 11, python:3.10.10-slim-bullseye container

Hi, unsure why you're seeing this behaviour. The Vonage JWT library sets a default expiry time from the iat time set every time the method is called. Can you provide a code example so I can reproduce the issue?

Closing as no activity


Added exponential window

@jankoslavic no problem. One note about Github and TravisCI (which does automated test suite runs): each time you push a commit to Github on an open PR you trigger a full build for every supported Python version, so it's better to first make all your commits locally and then push them at once. You see now that after every commit you have a green check mark, which means several hours of CPU time per commit.
Related note: if a commit touches only documentation, there's no point in re-running tests. You can achieve that by adding [ci skip] to the end of your commit message.

@jankoslavic no problem. One note about Github and TravisCI (which does automated test suite runs): each time you push a commit to Github on an open PR you trigger a full build for every supported Python version, so it's better to first make all your commits locally and then push them at once. You see now that after every commit you have a green check mark, which means several hours of CPU time per commit.
Related note: if a commit touches only documentation, there's no point in re-running tests. You can achieve that by adding [ci skip] to the end of your commit message.

@jankoslavic here is a branch with some other minor fixes that are needed and the history and commit messages fixed up: https://github.com/rgommers/scipy/compare/pr/4348. Could you take that over and push it to this PR?
In case you need it, here's a way to do this:
git remote add rgommers https://github.com/rgommers/scipy.git
git fetch rgommers
git checkout -b pr/4348 rgommers/pr/4348
git push origin pr/4348:master -f   # assuming your own Github scipy repo is named origin

@jankoslavic here is a branch with some other minor fixes that are needed and the history and commit messages fixed up: https://github.com/rgommers/scipy/compare/pr/4348. Could you take that over and push it to this PR?
In case you need it, here's a way to do this:
git remote add rgommers https://github.com/rgommers/scipy.git
git fetch rgommers
git checkout -b pr/4348 rgommers/pr/4348
git push origin pr/4348:master -f   # assuming your own Github scipy repo is named origin

Note by the way that it's better not to develop on your own master branch. Instead, you should use your master branch to track scipy master, and create a new feature branch based on up-to-date master for each separate feature you want to work on.

Note by the way that it's better not to develop on your own master branch. Instead, you should use your master branch to track scipy master, and create a new feature branch based on up-to-date master for each separate feature you want to work on.

One question about the tau keyword. A description like For N=0 use tau=-(M-1)/ln(x) makes me think that tau=1 isn't the right default value. You want to pick a default value such that not specifying tau at all gives reasonable behavior. What would you suggest that value to be?

One question about the tau keyword. A description like For N=0 use tau=-(M-1)/ln(x) makes me think that tau=1 isn't the right default value. You want to pick a default value such that not specifying tau at all gives reasonable behavior. What would you suggest that value to be?

Dear @rgommers! Thank you for the notes above. I was not aware of the CPU hours behind.
I did perform the following actions:
git remote add rgommers https://github.com/rgommers/scipy.git
git fetch rgommers
git checkout -b pr/4348 rgommers/pr/4348
git push origin pr/4348:master -f   # assuming your own Github scipy repo is named or
Did I do it right?
With regards to 'tau'. 'tau=1' makes sense, but the decay is much too high for typical use.
What I need and is typical in signal processing for experimental modal analysis (EMA) is 'N=0' (1 at the start) and decaying till the end of the window. In EMA the exponential window is defined with one parameter only, i.e. decay rate 'd=1%' or 'd=0.01'. This means that at start it will be 1 and at the end it will be 0.01. For the us in EMA it would be more appropriate to use the parameter 'd' and the corresponding parameter 'tau' would be automatically defined.
However, due to generality the parameter 'N' is introduced, and then I proposed the parameter 'tau' which is also more general.
Would you prefer to have it defined with 'd'?

@jankoslavic you got the git stuff right.
Regarding tau, I think your reasoning makes sense. N, tau is indeed more general than d only, and N, d would make less sense.

Because this is a new function I'm going to ping the mailing list, but it should be good to go now.

Dear @rgommers after further research (especially with regards to the 'sym' parameter):
When the sym=True, then N makes no sense, as the window is symmetric only when N=(M-1)/2
For my use, I only need non-symmetric window with N=0; however, due to generality the window should probably by default be symmetric... What is the best way to handle this issue?
Further:

'N' and 'tau' can both be floats
in the def of 'get_window' function I have previously added to the exponential window parameter checking. As the window can have default parameters and therefore the function can be called with just M. I guess the parameter check should be removed.

thank you.

For spectral analysis one needs a non-symmetric window, with most likely N = 0, while for filter design a symmetric window with N=(M-1)/2 is needed. There are cases where you would like to manually set N but those are relatively rare I think. (The only case I can think of is an impulse response with a delayed maximum peak)
How about N = None as default value. Then, if sym == True we calculate N=(M-1)/2 and else N=0.   One check that needs to be done is that sym and N result in an error (ValueError).

I have prepared a new definition. @rgommers, @FRidh Do you like it?

@FRidh your proposal makes sense to me.
@jankoslavic looks good, but note that the test needs updating. TravisCI test run failed.

I have implemented the changes as disscused above. Now the test fails at:
w1 = window(7, *params, sym=True)
w2 = window(7, *params, sym=False)
assert_array_almost_equal(w1, w2)
If I change the default behaviour of the parameter N to:
N : float, optional
    Parameter defining the center location of the window function.
    If not given, then ``N = (M-1) / 2`` (the value used for ``sym = True``).

With this change the test would have same results at M=7.
@rgommers , @FRidh  do you agree? I am open for better suggestions.

The test should indeed fail because for N=0 the shape is not symmetric anymore. While I suggested N=0 as default value for sym==False I now think this is perhaps not such a good idea. Even though you most often would use N=0 when analyzing impulse responses, this means the left side of the sequence is not necessarily zero.
An (acoustic) impulse response generally starts with a zero so it's not a problem, however, to fit with the other windows here it might be better to use N=(M-1)/2 when N==None regardless of whether sym is True or False. This means the window will always "look" symmetric, and is going to zero at both sides, like the other windows do.
The sym option would then only calculate the window for M+1 samples and truncate the final coefficient, again like the other windows.
It might be worth adding a note then in the documentation that if you would like to use this window to analyze an impulse response, you would have to prepend L zeros where L is the length of your impulse reponse, and then use M=2L for window length.

I don't think changing the default value of N depending on the value of sym is a good plan.  I think that @FRidh's suggestion is a good one.  What about also renaming N to be center or center_position or something else that is descriptive.

Thinking about this some more, I wonder actually if it is even worthwhile to include N.  if one needs a shifted window, its easy enough to write oneself and IIRC none of the other windows include such a parameter.
If we keep it, how about two plots, one with the default N and one with it shifted.  This way the its obvious from a quick glance at the docs that it does produce both.

I think I agree with @ewmoore,  there are really few cases that you would want to set N that it might be better to leave it out, or else be really clear about what it does.

Thank you all for the comments.
The exponential window could be defined without the center (I prefer the name before N). In this case the exponential window definition is similar to most of other windows.
However, this symmetric window is something that one cannot use with impulse response (typical in acoustic or vibration analysis). Dropping half of the window does not sound good to me. Using a new parameter e.g. center looks a sound solution to me.
My proposal:
if sym:
center=(M-1)/2
elif center is None:
center=(M-1)/2
And I will try to clarify the documentation.
PS:
There is another window that is typically used in impact response measurements and is not symmetric. This is the Force window (see for example Fig 3 in http://www.chinaksi.com/doc/sem-org-IMAC-XV-15th-Int-15-40-2-Windows-Used-for-Impact-Testing.pdf). A typical force window is 5%: this means that the window is 1 for first 5% of time and 0 for the rest. There is a good reason for this: with impact/modal hammer you impact the structure for a few ms only and the rest of time there is no contact with the structure (you have only noise in the signal).

 My idea behind it is that it would fit with the other windows, however, it would make the function less usable because with this type of window you hardly ever want the window shape to be symmetric. So yes, let's make it one-sided then.
Looking at your proposal, you use the keyword sym to make the window shape symmetric. Just to be sure. The keyword sym stands for symmetric. It is used to distinguish from a periodic window (sym=False). The only difference between the two is that with sym=True you calculate M+1 coefficients and then purge the final coefficient while for sym=False you calculate for M coefficients. It does not mean that the window shape is symmetric.
Therefore I would say:
if not sym and not odd:  # Like in all the other windows
    M = M + 1
if center is None:
    center=(M-1)/2

if not sym and not odd: #
    w = w[:-1]

@FRidh thank you for your comments.
Your proposal is simpler. However, in this case one could use center=0 and sym=True which would result in a non-symmetric window (without any warning/error). The current code gives a warning.

dear @FRidh this was one of the initial versions, but it results in test failure:
w1 = window(7, *params, sym=True)
w2 = window(7, *params, sym=False)
assert_array_almost_equal(w1, w2)
dear @rgommers travis fails, but not due to the updated code. Should I push the latest code to your branch? (as previously)

travis fails, but not due to the updated code.

fwiw the travis error is due to the unrelated 50 minute time limit on tests

@jankoslavic see this notebook for how the exponential window as I wrote above performs. I think this is how it should behave. For comparison I also added some other windows, showing what sym does.

As you can see there is actually no need for sym with this window.

Dear @FRidh thank you for the comment and the notebook!
For my use, your version is just fine; but this will not perform fine for filter design which requires perfectly symmetric window when sym = True.
The sym help for other windows:
When True (default), generates a symmetric window, for use in filter design. When False, generates a periodic window, for use in spectral analysis.
What do you think?

Dear @jankoslavic , I must say I am not that familiar with filter design.
We have two cases that the window can be used, in filter design and in spectral analysis when operating with the DFT.
When taking the DFT of an odd numbered window the last coefficient should be truncated. This one is however restored by the first coefficient due to the periodicity of the DFT. This is done when sym==False.
When sym==True the window is said to be "symmetric". Does that mean the window shape is symmetric? Should, for filter design, the window be shaped symmetrically? I'm confused on this now.
Regardless of the answer, what I think is important to decide on, is whether in the periodic case ('sym==False`) the window has the same shape as in the filter design case (except for the last coefficient difference), or if the shape can be different. So, can it be that for the filter design case we have an exponential mirrored in the middle, whereas in the periodic case we do not? Does this have any consequences (e.g. the exponential decrease goes faster)?

Dear @FRidh! Yes, for the filter design the window should be symmetric about the central point.
Change of exponential decay? Yes, there can be a slight change of the decay it the + 1 point is added...

@jankoslavic thanks, that it definitely has to be symmetric about a central point is clear now.
Ok, I just checked your implementation again, and I agree also now with your implementation.

@jankoslavic, It looks like there is still @rgommers comment about cleaning up the example a little bit to be addressed.  It would also be good to write a test or two that looks at the output.  The existing test just checks that the sym flag is operating correctly.
Other than those two issues, this looks good to go.

Dear @ewmoore : thank you for your feedback. The test has been updated (actually the code). The test is similar with other windows. I guess further tests are not required. or?

Certain parts of scipy have rather poor test coverage, and parts of signal are one of those places.
Could you add a small test that looks at the returned values?  How about calling it with all parameters at the default and calling it having varied center or  tau or both.  So four calls to your function. I'd use assert_allclose (use an explicit value for rtol, you'll want it a little tighter than the default) and the current output as the expected value.  You don't need to make the window be very long, M could be say 4 or 5.  If you're feeling more ambitious, testing with both M even and M odd with sym true and sym false would be even better.
The idea here is that the tests should both ensure that everything is working, and that they stay that way in the future. So the functionality should be actually exercised by the tests. The window functions in general don't have adequate tests right now, but since this is a new feature, it may as well go in with adequate testing.

Also, could you squash this down to a single commit?

@ewmoore thank you for your comments. I do not really feel comfortable to squash it down to a single commit. Can you suggest me where to get more info about this?

What you want is an interactive rebase, try looking at the notes in the Numpy repo: http://docs.scipy.org/doc/numpy-dev/dev/gitwash/development_workflow.html#rewriting-commit-history

Dear @ewmoore Thank you for your help. I am really packed now. I guess I will leave this task for later.

all remaining comments were addressed in gh-4536, so closing this PR. Thanks @jankoslavic @FRidh @ewmoore !


Can't execute $emit tag  inside

I try $emit from child to parent it just work fine.
but when i try @click inside <nuxt-link> to execute $emit function it not work at all.
I don't know if i do something wrong?

I also try to bind @click on <nuxt-link> and it still not work

This may be a problem with nuxt-link but meanwhile you can try using @click.native to bypass this issue.

@pi0 TA Mate! but still not work :x

I think, the <a> inside <nuxt-link> is generate and overwrite by NuxtJs it self 🤔

@VadiForte what about this?
<template lang="pug">
  ul.menubar
    li(@click="closeMenu")
      nuxt-link(to="/") Home
    li
      nuxt-link(to="/product-service") Product & Service
    li
      nuxt-link(to="/about") About
    li
      nuxt-link(to="/") Request Demo
</template>

I can fire a custom method with the @click.native modifier, maybe the problem is your event handling. How and Where do you listen for menuWasClose?

@alexchopin Thank you alex and all you guy, it work fine now :D


Why is the shim running on it's own network ns?

Ref: https://github.com/containerd/runwasi/blob/71f8df9cc576ea9564b3ea692432e80b454da7e5/crates/containerd-shim-wasm/src/sandbox/shim/cli.rs#L56-L57
It is not clear to me the reason why the shim start function sets up it's own network ns. I checked out both the Go and Rust implementation of runc-shim, they aren't setting the same ns at shim start.
@jsturtevant pointed me to this https://github.com/containers/youki/issues/2473 from youki but it looks like youki was setting up all the namespaces for containers but somehow the network ns was ignored. I am raising the issue here for asking more investigation.
FYI @jprendes

Before youki we did everything ourselves.
It may not be necessary to do this anymore.

From https://github.com/containerd/runwasi/pull/327#issuecomment-1730071090, removing that setup_namespaces call means that the network ns is not set up at all.
At the time I thought that maybe youki wasn't setting up the network ns, but it does.
It would be great if anyone has further insight.

We removed it awhile back and broke everything 🤦  https://github.com/containerd/runwasi/pull/364

youki is only setting the Pid and User namespaces

That doesn't seem right :(

I removed it once and @jsturtevant needed to add it again here

youki is only setting the Pid and User namespaces

That doesn't seem right :(

That was me misreading the code. Youki sets Pid and User ns at one point, and the rest on a different place.
IIRC, Pid and User ns need to be set up in the intermediate process, while the rest are set up in the init process.

Youki sets Pid and User ns at one point, and the rest on a different place.

Okay, if this is true, then the shim doesn't need to set up its own network ns. There must be some root causes that we haven't yet discovered.

I've added some logging to runwasi and youki when setting namespaces, and got:
level=info msg=">>>>> runwasi > unshare(CloneFlags(CLONE_NEWNET))"
level=info msg="server listen started"
level=info msg="server started"
level=info msg="Shim successfully started, waiting for exit signal..."
level=debug msg="Got new client"
...
level=warn msg="Controller rdma is not yet implemented."
level=warn msg="Controller misc is not yet implemented."
level=debug msg="unshare or setns: LinuxNamespace { typ: Pid, path: None }"
level=info msg=">>>>> youki > unshare(CloneFlags(CLONE_NEWPID))"
level=debug msg="sending init pid (Pid(1883648))"
level=debug msg="unshare or setns: LinuxNamespace { typ: Uts, path: None }"
level=info msg=">>>>> youki > unshare(CloneFlags(CLONE_NEWUTS))"
level=debug msg="unshare or setns: LinuxNamespace { typ: Ipc, path: None }"
level=info msg=">>>>> youki > unshare(CloneFlags(CLONE_NEWIPC))"
level=debug msg="unshare or setns: LinuxNamespace { typ: Network, path: None }"
level=info msg=">>>>> youki > unshare(CloneFlags(CLONE_NEWNET))"
level=debug msg="unshare or setns: LinuxNamespace { typ: Cgroup, path: None }"
level=info msg=">>>>> youki > unshare(CloneFlags(CLONE_NEWCGROUP))"
level=debug msg="unshare or setns: LinuxNamespace { typ: Mount, path: None }"
level=info msg=">>>>> youki > unshare(CloneFlags(CLONE_NEWNS))"
level=debug msg="prepare rootfs rootfs="/var/lib/docker/rootfs/overlayfs/7c52b78c3a22d2a00cb67cdab9fd090241ef9cdf8f47bb0d7f20daba465662a2""
level=debug msg="mount root fs "/var/lib/docker/rootfs/overlayfs/7c52b78c3a22d2a00cb67cdab9fd090241ef9cdf8f47bb0d7f20daba465662a2""
level=debug msg="mounting Mount { destination: "/proc", typ: Some("proc"), source: Some("proc"), options: Some(["nosuid", "noexec", "nodev"]) }"
...

The network namespce is indeed being set twice.
By runwasi when the shim starts, and by youki on the init process.

It looks like after removing the network ns from shim start, the nginx pod had issues to start:
https://github.com/containerd/runwasi/actions/runs/8148869974/job/22272757552?pr=503#step:8:794

Pid and User ns need to be set up in the intermediate process

Just a drive by comment. The reason for this is that the pid and user namespace are a bit special and required special treatment. For the pid namespace, one can't simply join the pid namespace. Only the child process can join the new pid namespace, so we have to fork. In youki's case, we do this in the intermediate process to fork the init process.
user namespace is also special. In order to setup user namespace correctly, we need a parent process to setup the uid/gid mapping. This can't be set once inside the user namespace. So we join the user namespace when we fork the intermediate process from the main process. The intermediate process will join the user namespace and the main process will setup the uid/gid mapping.

@yihuaf Thanks for giving some context. Is what we are doing now, correct or do we need to make some changes in runwasi?

@yihuaf do you know if runwasi requires changes like https://github.com/containerd/runwasi/pull/503 or something else? I am still unclear if the network ns is youki's or runwasi's responsibility

the network ns was removed from: https://github.com/containerd/runwasi/pull/503. Closing this issue


Publish the minified crowd-2d-skeleton.js with CORS enabled

@arthur-putnam do you think you can publish the minified version of the crowd-2d-skeleton component with CORS enabled? For example at https://assets.crowd.aws/ like the https://assets.crowd.aws/crowd-html-elements.js?

Let me look into this and get back to you. If it is possible, It will likely take some time to get the necessary approvals and deployments in place.
If you are looking for a quicker solution, hosting the crowd-2d-skeleton.js would be the fastest path forward. The file can be hosted using any web hosting service. My hope is to have a blog published soon which will show how to do this in AWS. Specifically, it will show how to deploy an AWS CloudFormation stack using the AWS Cloud Development Kit (CDK) that can be used to run the crowd-2d-skeleton.js labeling workflows. This stack will setup the pre-annotation and post-annotation lambda functions as well as host the JavaScript file using Amazon S3 and Amazon CloudFront. I can keep you posted on this blogs status if that would be helpful.


gredis support tls

1. What version of Go and system type/arch are you using?
go version go1.14.4 darwin/amd64
2. What version of GoFrame are you using?
v1.13.1
3. Can this issue be reproduced with the latest release?
Yes
Currently gredis is not support tls, I want to make a pull request to enhance it

please assign to me if possible ？

my suggestion is add some parameter in the configStr
like ?idleTimeout=600&tls=true&skipVerify=true
is that ok ?

@xbkaishui It's awesome! Thanks!

no problem


clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

Error in ClashLeaders
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52
API returned non-200 status code: 429
View on Bugsnag
Stacktrace
clashleaders/clash/api.py:52 - find_clan_by_tag
clashleaders/model/clan.py:60 - fetch_and_save
clashleaders/scheduler.py:141 - index_random_war_clan

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been reopened in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been marked as fixed in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been reopened in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been marked as fixed in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been reopened in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been marked as fixed in Bugsnag
clashleaders.clash.api.ApiException in GET /search.json

An error linked to this issue has been reopened in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been marked as fixed in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been reopened in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been marked as fixed in Bugsnag
clashleaders.clash.api.ApiException in clashleaders/clash/api.py:52

An error linked to this issue has been reopened in Bugsnag
clashleaders.clash.api.ApiException in GET /clan/yf-jujyr0vq.xlsx


fails with onBeforeUpload returned false [403], but my onBeforeUpload returns true

I'm having an issue:

Give an expressive description of what is went wrong:

Upload fails with transport http, says config.onBeforeUpload() returned false [403]
wish this was easier

Version of Meteor-Files you're experiencing this issue:

ostrio:files 2.2.1

Version of Meteor you're experiencing this issue

2.7.1

Where this issue appears?

OS Linux, Chrome

Is it Client or Server issue?

client

Post Client and/or Server logs with enabled debug option, you can enable "debug" mode in Constructor

config.onBeforeUpload() returned false [403]

My code for FileCollection is basically copy/paste from GridFS guide:
// Based on https://github.com/veliovgroup/Meteor-Files/blob/master/docs/gridfs-bucket-integration.md#use-gridfs-with-gridfsbucket-as-a-storage

import {Meteor} from 'meteor/meteor'
import {FilesCollection} from 'meteor/ostrio:files'
import {createBucket} from './gridfs/createBucket'
import {createObjectId} from './gridfs/createObjectId'
import fs from 'fs'

type TODO = any

let tracksBucket: TODO

if (Meteor.isServer) {
	tracksBucket = createBucket('tracksBucket')
}

console.log('FilesCollection.schema', FilesCollection.schema)

export const Tracks = new FilesCollection({
	collectionName: 'tracks',
	allowClientCode: true,
	debug: Meteor.isServer && process.env.NODE_ENV === 'development',

	onBeforeUpload(file) {
		return true

		if (file.size <= 104_857_600 && /png|jpg|jpeg/i.test(file.extension)) {
			return true
		}

		return 'Please upload a file, with size equal or less than 100MB'
	},

	onAfterUpload(file) {
		// here we could manipulate our file and create a new version, for
		// example lower resolution version, etc...

		// then we read all versions we have got so far
		Object.keys(file.versions).forEach(versionName => {
			const metadata = {...file.meta, versionName, fileId: file._id}

			fs.createReadStream(file.versions[versionName].path)

				// upload the binary to the bucket
				.pipe(
					tracksBucket.openUploadStream(file.name, {
						contentType: file.type || 'binary/octet-stream',
						metadata,
					}),
				)

				// unlink the file from the fs on any error that occurred during
				// the upload to prevent zombie files
				.on('error', err => {
					console.error(err)
					this.unlink(this.collection.findOne(file._id), versionName) // Unlink files from FS
				})

				// once finished, attach the gridFS Object id on the
				// FilesCollection document's meta section and finally unlink
				// the upload file from the filesystem
				.on(
					'finish',
					Meteor.bindEnvironment(ver => {
						const property = `versions.${versionName}.meta.gridFsFileId`

						this.collection.update(file._id, {
							$set: {
								[property]: ver._id.toHexString(),
							},
						})

						this.unlink(this.collection.findOne(file._id), versionName) // Unlink files from FS
					}),
				)
		})
	},

	interceptDownload(http, file, versionName) {
		const {gridFsFileId} = file.versions[versionName].meta || {}

		if (gridFsFileId) {
			const gfsId = createObjectId({gridFsFileId})
			const readStream = tracksBucket.openDownloadStream(gfsId)

			readStream.on('data', data => {
				http.response.write(data)
			})

			readStream.on('end', () => {
				http.response.end('end')
			})

			readStream.on('error', () => {
				// not found probably
				// eslint-disable-next-line no-param-reassign
				http.response.statusCode = 404
				http.response.end('not found')
			})

			http.response.setHeader('Cache-Control', this.cacheControl)
			http.response.setHeader('Content-Disposition', `inline; filename="${file.name}"`)
		}

		return Boolean(gridFsFileId) // Serve file from either GridFS or FS if it wasn't uploaded yet
	},

	onAfterRemove(files) {
		files.forEach(file => {
			Object.keys(file.versions).forEach(versionName => {
				const gridFsFileId = (file.versions[versionName].meta || {}).gridFsFileId

				if (gridFsFileId) {
					const gfsId = createObjectId({gridFsFileId})

					tracksBucket.delete(gfsId, err => {
						if (err) console.error(err)
					})
				}
			})
		})
	},
})

if (Meteor.isServer) {
	Meteor.publish('tracks.all', function () {
		return Tracks.collection.find({})
	})
}

if (Meteor.isClient) Meteor.subscribe('tracks.all')

And here's the upload code:
	function upload(e: InputEvent) {
		e.preventDefault()

		const input = e.currentTarget as HTMLInputElement | null
		const file = input?.files?.[0]

		if (file) {
			// insert() docs: https://github.com/veliovgroup/Meteor-Files/blob/master/docs/insert.md
			let uploadInstance = Tracks.insert(
				{
					file,

					transport: 'http',

					// optional data to go with the file
					meta: {
						// userId: Meteor.userId(),  TODO
					},

					chunkSize: 'dynamic',
					allowWebWorkers: true, // If you see issues with uploads, change this to false

					onStart(_error, file) {
						console.log('start upload', file)
					},
					onAbort() {},
					onBeforeUpload() {},
					onError() {},
					onProgress(progress, file) {
						console.log('upload progress', progress, file)
					},
					onUploaded(error, file) {
						if (error) throw error
						console.log('done uploading', file)
					},
				},
				false,
			)

			uploadInstance.start()
		}
	}

Then when upload is called, the onUploaded method is passed that error.

If I remove transport: 'http' I get the same config.onBeforeUpload() returned false [403] error

Hello @trusktr ,
You have defined onBeforeUpload in .insert() method. Thought it's empty its return will be treated as undefined !== true;
Solution: remove onBeforeUpload hook or return true in the object passed to .insert() method

Oh, I see. So this example does it wrong?
https://github.com/Neobii/files-gridfs-autoform-example/blob/master/lib/images.js
Did the API change perhaps? Was that ok at some point?

Or does onBeforeUpload work in both places? I imagine the server needs to validate, because a client can easily hack around the onBeforeUpload on the client.

Does this example does it wrong?
https://github.com/Neobii/files-gridfs-autoform-example/blob/master/lib/images.js

No, I don't see onBeforeUpload defined in object passed into .insert() method

Hello @trusktr, was this one solved on your end?
Feel free to close it in case if the issue is solved on your end.