Kernel Compilation Issue with New Xtrack

Hi,
I upgraded to the latest xtrack (0.54.1) and the kernels compilation crashes. This can be observed simply running xsuite-prebuild.
Bellow the error stack from the command:
Xcoll not installed, skipping collimator elements
Traceback (most recent call last):
  File "/Users/felixsoubelet/.miniforge/envs/dev/bin/xsuite-prebuild", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/xsuite/cli.py", line 21, in main
    regenerate_kernels()
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/xtrack/prebuild_kernels.py", line 316, in regenerate_kernels
    tracker._build_kernel(
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/xtrack/tracker.py", line 624, in _build_kernel
    if self._context.allow_prebuilt_kernels:
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'ContextCpu' object has no attribute 'allow_prebuilt_kernels'

Do you have the latest xobjects?

Indeed I had not seen the new xobjects release.
I have updated it but am still getting a different error when running xsuite-prebuild. Below the full stack trace:
Compiling ContextCpu kernels...
default_only_xtrack_no_config.c:14605:8: error: redefinition of 'rng_get'
double rng_get (uint32_t *s1, uint32_t *s2, uint32_t *s3, uint32_t *s4 ){
       ^
default_only_xtrack_no_config.c:1769:8: note: previous definition is here
double rng_get (uint32_t *s1, uint32_t *s2, uint32_t *s3, uint32_t *s4 ){
       ^
default_only_xtrack_no_config.c:14616:6: error: redefinition of 'rng_set'
void rng_set (uint32_t *s1, uint32_t *s2, uint32_t *s3, uint32_t *s4, uint32_t s ){
     ^
default_only_xtrack_no_config.c:1780:6: note: previous definition is here
void rng_set (uint32_t *s1, uint32_t *s2, uint32_t *s3, uint32_t *s4, uint32_t s ){
     ^
2 errors generated.
Traceback (most recent call last):
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/unixccompiler.py", line 185, in _compile
    self.spawn(compiler_so + cc_args + [src, '-o', obj] + extra_postargs)
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/ccompiler.py", line 1041, in spawn
    spawn(cmd, dry_run=self.dry_run, **kwargs)
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/spawn.py", line 70, in spawn
    raise DistutilsExecError(
distutils.errors.DistutilsExecError: command '/Users/felixsoubelet/.miniforge/envs/dev/bin/arm64-apple-darwin20.0.0-clang' failed with exit code 1

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/cffi/ffiplatform.py", line 48, in _build
    dist.run_command('build_ext')
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/dist.py", line 989, in run_command
    super().run_command(command)
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
    cmd_obj.run()
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/command/build_ext.py", line 88, in run
    _build_ext.run(self)
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/command/build_ext.py", line 345, in run
    self.build_extensions()
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/command/build_ext.py", line 467, in build_extensions
    self._build_extensions_serial()
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/command/build_ext.py", line 493, in _build_extensions_serial
    self.build_extension(ext)
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/command/build_ext.py", line 249, in build_extension
    _build_ext.build_extension(self, ext)
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/Cython/Distutils/build_ext.py", line 135, in build_extension
    super(build_ext, self).build_extension(ext)
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/command/build_ext.py", line 548, in build_extension
    objects = self.compiler.compile(
              ^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/ccompiler.py", line 600, in compile
    self._compile(obj, src, ext, cc_args, extra_postargs, pp_opts)
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/setuptools/_distutils/unixccompiler.py", line 187, in _compile
    raise CompileError(msg)
distutils.errors.CompileError: command '/Users/felixsoubelet/.miniforge/envs/dev/bin/arm64-apple-darwin20.0.0-clang' failed with exit code 1

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/felixsoubelet/.miniforge/envs/dev/bin/xsuite-prebuild", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/xsuite/cli.py", line 21, in main
    regenerate_kernels()
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/xtrack/prebuild_kernels.py", line 316, in regenerate_kernels
    tracker._build_kernel(
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/xtrack/tracker.py", line 633, in _build_kernel
    out_kernels = context.build_kernels(
                  ^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/xobjects/context_cpu.py", line 311, in build_kernels
    so_file = self.compile_kernel(
              ^^^^^^^^^^^^^^^^^^^^
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/xobjects/context_cpu.py", line 437, in compile_kernel
    output_file = ffi_interface.compile(
                  ^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/cffi/api.py", line 725, in compile
    return recompile(self, module_name, source, tmpdir=tmpdir,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/cffi/recompiler.py", line 1564, in recompile
    outputfilename = ffiplatform.compile('.', ext,
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/cffi/ffiplatform.py", line 20, in compile
    outputfilename = _build(tmpdir, ext, compiler_verbose, debug)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/felixsoubelet/.miniforge/envs/dev/lib/python3.11/site-packages/cffi/ffiplatform.py", line 54, in _build
    raise VerificationError('%s: %s' % (e.__class__.__name__, e))
cffi.VerificationError: CompileError: command '/Users/felixsoubelet/.miniforge/envs/dev/bin/arm64-apple-darwin20.0.0-clang' failed with exit code 1

Old xpart? :P


Allow using a project-level catalog file as well

This is a draft PR, just in case - curious to hear if you'd consider adding this feature.
This looks in the project's configuration for a setting called xml.catalogFolder and if found, will try to load a catalog.xml file from that location and add it to the initialization options' catalogs array.
I'm not sure if the filename should be configurable as well - maybe it's simpler to name the setting xml.catalogFile instead, and expect the full path?
Cheers,
Chriztian

Thanks for the PR! This could definitely be added. I think it would make sense to specify the a file rather than a folder so you aren't locked into calling it "catalog.xml", unless this is a standard XML thing I'm not aware of?
You can also add a configWorkspace option to the extension.json too. This would allow people to configure it though the GUI as well.
I wonder if it makes sense to allow multiple catalogs per project rather than just one? If so it could be a pathArray configuration type that would be an array instead of a single string. It would be easier to decide this now to try to avoid future breaking changes.

Oh very nice - I didn't even think to see if a workspace level config was possible! I was really hoping for a GUI option of some kind. I'll try this out right away.
Makes total sense to go with files in a pathArray - hopefully I can translate that to the Type-/JavaScript as well :)

The update might actually be related to this nova 9.x issue — https://devforum.nova.app/t/regression-in-novas-lsp-configuration-support/1460/4
normally nova syncs the config from your project and global with the language server but there is a regression there. It might not be worth looking too far into it until that is resolved, hopefully in nova 9.2

It might be possible to use nova's config sync (when it is working again) to get this all working more automatically. If the config is named simply "xml.catalogs" it should line up with the language sever's configuration and nova will automatically tell the server about the value and update it when it changes. If we went this route then the initialisationOptions should be able to stay as it was and this config might automatically work.

Nova's configuration API is still broken 😞 https://devforum.nova.app/t/regression-in-novas-lsp-configuration-support/1460/15

That sounds great Robb - I've only been following along sporadically so wasn't really aware.
I'll convert this to a proper PR then and you can go ahead and merge it when you've confirmed it doesn't blow up your otherwise lovely extension 😁
Thanks!

Hey, I missed that you marked this as ready for review sorry! I'll look into this soon, apologies

No worries Rob 👍 (Though I no longer remember if I've verified that this actually works 😕 )

Hey, it looks like I was already testing setting "xml.catalog" from Nova's LanguageServer config integration and it seems to be working! I've just released 0.3.0 which contains this if you could try it out?
It doesn't need the xml-language-server.ts or extension.json changes now but but I'd still like to merge your doc changes, then you'd get properly credited GitHub wise!

Hi @robb-j
Thanks for the update! I've tried to see if I could make it work, but haven't yet succeeded in any of my projects, so I've created a very minimal test project that I hope you can try out and see if I'm just making things up 😅
It's located here: https://github.com/greystate/xml-reduced-case
When I open the "wordles.xml" file in Nova, I just get the "No grammar constraints (DTD or XML Schema)" message...
I've tried both having the catalog point to the "real" location (https://xmlns.greystate.dk/2022/wordles/wordles.xsd) and to the local copy that's included in the repo.
If I open the wordles.xml file in Xmplify.app (native macOS XML editor) and add the catalog file, it works "correctly", i.e. it understands the schema and provides help etc.
Hope I'm making sense :)
/Chriztian

Hey, I know I merged it but its taking a bit of investigation to get it working. Looking at your xsd:annotation's I think they aren't working because the LangaugeServer doesn't support that HTML-in-XML syntax? I'm by no means an expert on XSD, when I wrote the schema for nova syntaxes the LanguageServer treated the text inside there as markdown? I hacked around your file and putting plain text did get the hovers working!

<xsd:annotation>
  <xsd:documentation>
    One five letter word as one of 6 tries at solving the wordle (or 9 tries in a quordle).
  </xsd:documentation>
</xsd:annotation>

Cool @robb-j 👍 - I can totally work with that (Markdown, that is...)
I'll do some testing in my files - thanks!!

The catalogs aren’t working at the moment,  something in Nova or the LanguageServer changed and it’s not behaving well. I’m working on that over in #8, I should hopefully have a release for that soon.


Simplify rendering/editing sheet data between Vue app and backend

Branching this from #747 as it's a bit more complicated.
Developing complex features with zero design for planning has resulted in overly complex data structures in some cases. We should rethink this and simplify the data, if/where appropriate.
NOTE: If possible, this change should be implemented by making required changes to e.g. sheet editing configurations via a Django migration script, so that wiping out existing configurations would not be required.
Current Situation

There are currently multiple levels of describing nodes, columns and formatting in rendering/editing

Object Class (obj_cls)

Type of node (material or object)
Critical for all stages of ISAtab processing
In some cases, might be able to gather this info without passing it explicitly in Ajax view data

Item Type

Type of material node, used both in rendering and editing
SOURCE, SAMPLE, DATA, etc.
Critical for all stages of ISAtab processing

Header Type

Type of field in a process/material object
name, protocol, process_name, field..
field = e.g. characteristics and other columns of which there can be 0-N for a node

Column Type (colType or col_type)

Type of column, used for rendering
Null = default string rendering
Example: LINK_FILE for column which is supposed to render a link to iRODS for a DATA style material
Critical for rendering data and sorting columns
Some info can be derived from other sources

E.g. name in header type + DATA in item type = LINK_FILE (always)
However, a field may be of e.g. NUMERIC or UNIT type

Should at the very least be renamed into something more descriptive. Render type? Value type?

Format

Used in sheet edit config, defines the format of data accepted for a column
For e.g. characteristics, a single column may have one of multiple types of formatting
For e.g. name and protocol, the format` is always the same (redundant info?)
Critical for headers/columns with header type of field, not sure if for other types?

Naming of headers/columns

This may be inconsistent in places and confusing for the user
Also should consider the ISAtab/altamISA perspective
TODO: Catalogue how these terms are currently used

Plan for Improvement
?????? (-> profit)
Tasks
TBD (see previous list)

This should be done after we have done #994 and retired the existing app, as the current plan is to keep the new and old app side-by-side for at least one major release to ensure there are workaround for possible major bugs in the new implementation.
Hence, if we do the rewrite in v1.2 as planned at the time of writing, this would be done for v1.3 at the earliest.


[Static Adapter] Absolute file paths in generated HTML files

Describe the bug
When building the sveltekit app with the static adapter, the generated index.html inside /build/ contains only absolute links.  However, trying to run index.html leads to tons of errors because the absolute file paths cannot be resolved.
<link rel="modulepreload" href="/_app/start-47ad7042.js">
<link rel="modulepreload" href="/_app/chunks/vendor-986c0c4e.js">
<link rel="modulepreload" href="/_app/pages/__layout.svelte-76e90a6f.js">
<link rel="modulepreload" href="/_app/pages/index.svelte-3ba9ed59.js">
<link rel="modulepreload" href="/_app/chunks/Footer-4274319b.js">
<link rel="stylesheet" href="/_app/assets/start-61d1577b.css">
<link rel="stylesheet" href="/_app/assets/vendor-a0137619.css">
<link rel="stylesheet" href="/_app/assets/pages/__layout.svelte-9da8ffa3.css">
<link rel="stylesheet" href="/_app/assets/pages/index.svelte-1280cda4.css">
<link rel="stylesheet" href="/_app/assets/Footer-5bd6e039.css">

The solution would be to just insert the a dot before the starting / :
<link rel="modulepreload" href="./_app/start-47ad7042.js">
The given errors are related to CORS, since the path is pointing to the current drive ("D") in this case:

I am using the default configuration for the adapter. I wonder if this is intended or if there is a solution other than manually fixing all the paths.
Reproduction
The adapter: adapter: adapter({ 			// default options are shown 			pages: 'build', 			assets: 'build', 			fallback: null 		})
Logs
No response
System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (8) x64 Intel(R) Core(TM) i7-4790K CPU @ 4.00GHz
    Memory: 5.80 GB / 15.88 GB
  Binaries:
    Node: 14.16.1 - E:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 7.20.2 - E:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1023.0), Chromium (92.0.902.78)
    Internet Explorer: 11.0.19041.906
  npmPackages:
    @sveltejs/adapter-static: ^1.0.0-next.17 => 1.0.0-next.17
    @sveltejs/kit: next => 1.0.0-next.156
    svelte: ^3.34.0 => 3.42.2

Severity
serious, but I can work around it
Additional Information
No response

Build output are meant to be served through a live server and not with the file protocol. Either run a http server or use something like npx serve

is there an option to use the file protocol? it seems like it's also broken with Github pages.


Support printing features and returning keys when getting features from registry

Description

Provide function to show features details (including keys) in json format and return a [feature_name] = [typed keys] dict when 'get_features_from_registry'

How was this PR tested?

Tested locally, and got log like below:

Modified one of existing test cases to print out features details and to return key dicts

Does this PR introduce any user-facing changes?

[ ] No. You can skip the rest of this section.
[ ] Yes. Make sure to clarify your proposed changes.

LGTM, thanks @enya-yx for this!


checkpoint_challenge_required during login

[main] INFO  org.brunocvcunha.instagram4j.requests.InstagramRequest  - Reading InstagramLoginResult from {"lock": true, "status": "fail", "checkpoint_url": "https://i.instagram.com/challenge/4650947368/jrNub9nivY/", "error_type": "checkpoint_challenge_required", "message": "checkpoint_required"}
com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "lock" (class org.brunocvcunha.instagram4j.requests.payload.InstagramLoginResult), not marked as ignorable (2 known properties: "logged_in_user", "status"])

This can happen during Instagram4j.login(); and will result in an error since it cannot be casted.

Any update?

Fixed! Thanks


Error: the following clang command did not run successfully

Infer V1.2.0
macos 13.6.1
xcode: 15.0.1
1 error generated.
1 error generated.
1 error generated.
Error: the following clang command did not run successfully:
/Users/infer-osx-arm64-v1.2.0/lib/infer/facebook-clang-plugins/clang/install/bin/clang-18
@/Users/builds/hKr-Nzho/0/heiyeclient/-ios/infer-out/tmp/clang_command_.tmp.729459.txt
++Contents of '/Users/builds/hKr-Nzho/0/heiyeclient/-ios/infer-out/tmp/clang_command_.tmp.729459.txt':
"-cc1" "-load"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/infer/bin/../../facebook-clang-plugins/libtooling/build/FacebookClangPlugin.dylib"
"-plugin" "BiniouASTExporter" "-plugin-arg-BiniouASTExporter" "-"
"-plugin-arg-BiniouASTExporter" "PREPEND_CURRENT_DIR=1"
"-plugin-arg-BiniouASTExporter" "MAX_STRING_SIZE=65535" "-cc1" "-triple"
"x86_64-apple-ios13.0.0-simulator" "-Wundef-prefix=TARGET_OS_"
"-Werror=undef-prefix" "-Wdeprecated-objc-isa-usage"
"-Werror=deprecated-objc-isa-usage"
"-Werror=implicit-function-declaration" "-fsyntax-only" "-disable-free"
"-clear-ast-before-backend" "-disable-llvm-verifier"
"-discard-value-names" "-main-file-name" "HYCollectionViewController.m"
"-mrelocation-model" "pic" "-pic-level" "2" "-mframe-pointer=all"
"-ffp-contract=on" "-fno-rounding-math" "-funwind-tables=2"
"-target-sdk-version=17.0"
"-fcompatibility-qualified-id-block-type-checking"
"-fvisibility-inlines-hidden-static-local-var"
"-fbuiltin-headers-in-system-modules" "-fdefine-target-os-macros"
"-target-cpu" "core2" "-tune-cpu" "generic" "-debug-info-kind=standalone"
"-dwarf-version=4" "-debugger-tuning=lldb" "-fdebug-compilation-dir=/"
"-target-linker-version" "1015.7" "-fcoverage-compilation-dir=/"
"-resource-dir"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/facebook-clang-plugins/clang/install/lib/clang/18"
"-dependency-file"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/Objects-normal/x86_64/HYCollectionViewController.d"
"-MT" "dependencies" "-isysroot"
"/Applications/Xcode15.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX14.0.sdk"
"-isysroot"
"/Applications/Xcode15.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator17.0.sdk"
"-iquote"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/-generated-files.hmap"
"-ivfsoverlay"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/all-product-headers.yaml"
"-iquote"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/-project-headers.hmap"
"-include"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/Pods/Target Support Files//-prefix.pch"
"-include"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/infer/bin/../lib/clang_wrappers/global_defines.h"
"-D" "POD_CONFIGURATION_DEBUG=1" "-D" "DEBUG=1" "-D" "COCOAPODS=1" "-D"
"OBJC_OLD_DISPATCH_PROTOTYPES=0" "-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/-own-target-headers.hmap"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/-all-non-framework-target-headers.hmap"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Products/Debug-iphonesimulator//include"
"-I"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/Pods/Headers/Private"
"-I"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/Pods/Headers/Private/"
"-I"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/Pods/Headers/Public"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/DerivedSources-normal/x86_64"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/DerivedSources/x86_64"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/DerivedSources"
"-F/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Products/Debug-iphonesimulator/"
"-D" "NS_FORMAT_ARGUMENT(A)=" "-D" "_Nullable_result=_Nullable"
"-internal-isystem"
"/Applications/Xcode15.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator17.0.sdk/usr/local/include"
"-internal-isystem"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/facebook-clang-plugins/clang/install/lib/clang/18/include"
"-internal-externc-isystem"
"/Applications/Xcode15.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator17.0.sdk/usr/include"
"-O0" "-Wnon-modular-include-in-framework-module"
"-Werror=non-modular-include-in-framework-module" "-Wno-trigraphs"
"-Wno-missing-field-initializers" "-Wno-missing-prototypes"
"-Werror=return-type" "-Wdocumentation" "-Wunreachable-code"
"-Wno-implicit-atomic-properties" "-Werror=deprecated-objc-isa-usage"
"-Wno-objc-interface-ivars" "-Werror=objc-root-class"
"-Wno-arc-repeated-use-of-weak" "-Wimplicit-retain-self"
"-Wduplicate-method-match" "-Wno-missing-braces" "-Wparentheses"
"-Wswitch" "-Wunused-function" "-Wno-unused-label"
"-Wno-unused-parameter" "-Wunused-variable" "-Wunused-value"
"-Wempty-body" "-Wuninitialized" "-Wconditional-uninitialized"
"-Wno-unknown-pragmas" "-Wno-shadow" "-Wno-four-char-constants"
"-Wno-conversion" "-Wconstant-conversion" "-Wint-conversion"
"-Wbool-conversion" "-Wenum-conversion" "-Wno-float-conversion"
"-Wnon-literal-null-conversion" "-Wobjc-literal-conversion"
"-Wshorten-64-to-32" "-Wpointer-sign" "-Wno-newline-eof" "-Wno-selector"
"-Wno-strict-selector-match" "-Wundeclared-selector"
"-Wdeprecated-implementations" "-Wno-implicit-fallthrough" "-Wprotocol"
"-Wdeprecated-declarations" "-Wno-sign-conversion" "-Winfinite-recursion"
"-Wcomma" "-Wblock-capture-autoreleasing" "-Wstrict-prototypes"
"-Wno-semicolon-before-method-body" "-Wunguarded-availability"
"-Wno-ignored-optimization-argument" "-Wno-everything" "-std=gnu11"
"-ferror-limit" "19" "-fmacro-backtrace-limit=0" "-stack-protector" "1"
"-fno-builtin" "-fblocks" "-fencode-extended-block-signature"
"-fregister-global-dtors-with-atexit" "-fgnuc-version=4.2.1" "-fmodules"
"-fimplicit-module-maps"
"-fmodules-cache-path=/Users//Library/Developer/Xcode/DerivedData/ModuleCache.noindex"
"-fmodule-name=" "-fmodules-prune-interval=86400"
"-fmodules-prune-after=345600" "-fbuild-session-timestamp=1722308548"
"-fmodules-validate-once-per-build-session"
"-fmodules-validate-system-headers" "-fskip-odr-check-in-gmf"
"-fobjc-runtime=ios-13.0.0" "-fobjc-arc" "-fobjc-exceptions"
"-fexceptions" "-fpascal-strings" "-fmax-type-align=16"
"-fdiagnostics-show-note-include-stack" "-fasm-blocks"
"-serialize-diagnostic-file"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/Objects-normal/x86_64/HYCollectionViewController.dia"
"-D__GCC_HAVE_DWARF2_CFI_ASM=1" "-x" "objective-c"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/LocalPods//Classes/HYCollectionView/CollectionView/HYCollectionViewController.m"
"-O0" "-fno-builtin" "-include"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/infer/bin/../lib/clang_wrappers/global_defines.h"
"-Wno-everything"

PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace, preprocessed source, and associated run script.
Stack dump:0.2G] LocalPods/CommonTemp/Classes/Network/PPRequest.m
0.[ 0.3sProgram arguments: /Users/infer-osx-arm64-v1.2.0/lib/infer/facebook-clang-plugins/clang/install/bin/clang-18 @/Users/builds/hKr-Nzho/0/heiyeclient/xiaopeiban-ios/infer-out/tmp/clang_command_.tmp.44fba6.txtKeyValueStoreMgr+Live.m
1.[ 0.6s parser at end of filemp/Classes/Tools/KeyValueStore/KeyValueStoreMgr+Debug.m
Stack dump without symbol names (ensure you have llvm-symbolizer in your PATH or set the environment var LLVM_SYMBOLIZER_PATH to point to it):
0  clang-18                  0x00000001040721d8 llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) + 56
1  clang-18                  0x00000001040701dc llvm::sys::RunSignalHandlers() + 112lert.m
2  clang-18                  0x000000010407288c llvm::sys::PrintStackTraceOnErrorSignal(llvm::StringRef, bool) + 772
3  libsystem_platform.dylib  0x0000000195572584 _sigtramp + 56
4  clang-18                  0x0000000106864be4 clang::FieldDecl::getFieldIndex() const + 208
5  clang-18                  0x0000000106864be4 clang::FieldDecl::getFieldIndex() const + 208
6  FacebookClangPlugin.dylib 0x0000000108f407f0 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::dumpName(clang::NamedDecl const&) + 184
7  FacebookClangPlugin.dylib 0x0000000108f36f40 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::VisitFieldDecl(clang::FieldDecl const*) + 64
8  FacebookClangPlugin.dylib 0x0000000108f37344 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::VisitObjCIvarDecl(clang::ObjCIvarDecl const*) + 32
9  FacebookClangPlugin.dylib 0x0000000108f32254 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::dumpDecl(clang::Decl const*) + 276
10 FacebookClangPlugin.dylib 0x0000000108f3a918 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::VisitDeclContext(clang::DeclContext const*) + 556
11 FacebookClangPlugin.dylib 0x0000000108f34814 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::VisitObjCInterfaceDecl(clang::ObjCInterfaceDecl const*) + 60
12 FacebookClangPlugin.dylib 0x0000000108f32254 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::dumpDecl(clang::Decl const*) + 276
13 FacebookClangPlugin.dylib 0x0000000108f3a918 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::VisitDeclContext(clang::DeclContext const*) + 556
14 FacebookClangPlugin.dylib 0x0000000108f32c30 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::VisitTranslationUnitDecl(clang::TranslationUnitDecl const*) + 60
15 FacebookClangPlugin.dylib 0x0000000108f32254 ASTLib::ASTExporter<ATDWriter::BiniouWriterllvm::raw_ostream>::dumpDecl(clang::Decl const*) + 276
16 FacebookClangPlugin.dylib 0x0000000108f32104 ASTLib::ExporterASTConsumer<ATDWriter::BiniouWriterllvm::raw_ostream, true>::HandleTranslationUnit(clang::ASTContext&) + 160
17 clang-18                  0x0000000104af9f4c clang::MultiplexConsumer::HandleTranslationUnit(clang::ASTContext&) + 52
18 clang-18                  0x0000000105b2ef70 clang::ParseAST(clang::Sema&, bool, bool) + 648
19 clang-18                  0x0000000104abeb0c clang::FrontendAction::Execute() + 112
20 clang-18                  0x0000000104a4fc18 clang::CompilerInstance::ExecuteAction(clang::FrontendAction&) + 872
21 clang-18                  0x0000000104b3ff80 clang::ExecuteCompilerInvocation(clang::CompilerInstance*) + 500
22 clang-18                  0x00000001027877ac cc1_main(llvm::ArrayRef<char const*>, char const*, void*) + 3132
23 clang-18                  0x00000001027850d4 clang_main(int, char**, llvm::ToolContext const&) + 13920
24 clang-18                  0x0000000102784338 clang_main(int, char**, llvm::ToolContext const&) + 10436
25 clang-18                  0x0000000102791af0 main + 88
26 dyld                      0x00000001951b7154 start + 2476
ERROR RUNNING CAPTURE: Bi_util.Error("Corrupted data (string)")
Raised at Bi_util.error in file "src/bi_util.ml", line 3, characters 14-29
Called from Bi_io.read_untagged_string in file "src/bi_io.ml", line 575, characters 6-45
Called from Atdgen_runtime__Ob_run.read_string in file "atdgen-runtime/src/ob_run.ml" (inlined), line 181, characters 4-33
Called from ATDGenerated__Clang_ast_b.read_decl_info in file "src/atd/clang_ast_b.ml", line 34474, characters 17-117
Called from ATDGenerated__Clang_ast_b.get_decl_reader.(fun) in file "src/atd/clang_ast_b.ml", line 45034, characters 22-89
Called from Atdgen_runtime__Ob_run.read_array_value in file "atdgen-runtime/src/ob_run.ml", line 190, characters 27-38
Called from Atdgen_runtime__Ob_run.read_array in file "atdgen-runtime/src/ob_run.ml" (inlined), line 213, characters 4-34
Called from Atdgen_runtime__Ob_run.read_list in file "atdgen-runtime/src/ob_run.ml" (inlined), line 218, characters 16-36
Called from ATDGenerated__Clang_ast_b.read__decl_list in file "src/atd/clang_ast_b.ml", line 42907, characters 25-91
Called from ATDGenerated__Clang_ast_b.read_obj_c_method_decl_info in file "src/atd/clang_ast_b.ml", line 49707, characters 44-118
Called from ATDGenerated__Clang_ast_b.get_decl_reader.(fun) in file "src/atd/clang_ast_b.ml", line 44219, characters 22-102
Called from ATDGenerated__Clang_ast_b.get_decl_reader.(fun) in file "src/atd/clang_ast_b.ml" (inlined), line 43930, characters 8-1023
Called from Atdgen_runtime__Ob_run.read_array_value in file "atdgen-runtime/src/ob_run.ml" (inlined), line 192, characters 27-38
Called from Atdgen_runtime__Ob_run.read_array in file "atdgen-runtime/src/ob_run.ml" (inlined), line 213, characters 4-34
Called from Atdgen_runtime__Ob_run.read_list in file "atdgen-runtime/src/ob_run.ml" (inlined), line 218, characters 16-36
Called from ATDGenerated__Clang_ast_b.read__decl_list in file "src/atd/clang_ast_b.ml" (inlined), line 42907, characters 25-91
Called from ATDGenerated__Clang_ast_b.get_decl_reader.(fun) in file "src/atd/clang_ast_b.ml", line 44279, characters 22-90
Called from ATDGenerated__Clang_ast_b.get_decl_reader.(fun) in file "src/atd/clang_ast_b.ml" (inlined), line 43930, characters 8-1023
Called from Atdgen_runtime__Ob_run.read_array_value in file "atdgen-runtime/src/ob_run.ml" (inlined), line 192, characters 27-38
Called from Atdgen_runtime__Ob_run.read_array in file "atdgen-runtime/src/ob_run.ml" (inlined), line 213, characters 4-34
Called from Atdgen_runtime__Ob_run.read_list in file "atdgen-runtime/src/ob_run.ml" (inlined), line 218, characters 16-36
Called from ATDGenerated__Clang_ast_b.read__decl_list in file "src/atd/clang_ast_b.ml" (inlined), line 42907, characters 25-91
Called from ATDGenerated__Clang_ast_b.read_decl.(fun) in file "src/atd/clang_ast_b.ml", line 46207, characters 18-78
Called from Atdgen_runtime__Util.Biniou.from_channel in file "atdgen-runtime/src/util.ml" (inlined), line 30, characters 4-11
Called from ClangFrontend__Capture.validate_decl_from_channel in file "src/clang/Capture.ml", line 25, characters 2-113
...

Infer V1.2.0
macos 13.6.1
xcode: 15.0.1
1 error generated.
1 error generated.
1 error generated.
Error: the following clang command did not run successfully:
/Users/infer-osx-arm64-v1.2.0/lib/infer/facebook-clang-plugins/clang/install/bin/clang-18
@/Users/builds/hKr-Nzho/0/heiyeclient/-ios/infer-out/tmp/clang_command_.tmp.729459.txt
++Contents of '/Users/builds/hKr-Nzho/0/heiyeclient/-ios/infer-out/tmp/clang_command_.tmp.729459.txt':
"-cc1" "-load"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/infer/bin/../../facebook-clang-plugins/libtooling/build/FacebookClangPlugin.dylib"
"-plugin" "BiniouASTExporter" "-plugin-arg-BiniouASTExporter" "-"
"-plugin-arg-BiniouASTExporter" "PREPEND_CURRENT_DIR=1"
"-plugin-arg-BiniouASTExporter" "MAX_STRING_SIZE=65535" "-cc1" "-triple"
"x86_64-apple-ios13.0.0-simulator" "-Wundef-prefix=TARGET_OS_"
"-Werror=undef-prefix" "-Wdeprecated-objc-isa-usage"
"-Werror=deprecated-objc-isa-usage"
"-Werror=implicit-function-declaration" "-fsyntax-only" "-disable-free"
"-clear-ast-before-backend" "-disable-llvm-verifier"
"-discard-value-names" "-main-file-name" "HYCollectionViewController.m"
"-mrelocation-model" "pic" "-pic-level" "2" "-mframe-pointer=all"
"-ffp-contract=on" "-fno-rounding-math" "-funwind-tables=2"
"-target-sdk-version=17.0"
"-fcompatibility-qualified-id-block-type-checking"
"-fvisibility-inlines-hidden-static-local-var"
"-fbuiltin-headers-in-system-modules" "-fdefine-target-os-macros"
"-target-cpu" "core2" "-tune-cpu" "generic" "-debug-info-kind=standalone"
"-dwarf-version=4" "-debugger-tuning=lldb" "-fdebug-compilation-dir=/"
"-target-linker-version" "1015.7" "-fcoverage-compilation-dir=/"
"-resource-dir"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/facebook-clang-plugins/clang/install/lib/clang/18"
"-dependency-file"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/Objects-normal/x86_64/HYCollectionViewController.d"
"-MT" "dependencies" "-isysroot"
"/Applications/Xcode15.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX14.0.sdk"
"-isysroot"
"/Applications/Xcode15.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator17.0.sdk"
"-iquote"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/-generated-files.hmap"
"-ivfsoverlay"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/all-product-headers.yaml"
"-iquote"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/-project-headers.hmap"
"-include"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/Pods/Target Support Files//-prefix.pch"
"-include"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/infer/bin/../lib/clang_wrappers/global_defines.h"
"-D" "POD_CONFIGURATION_DEBUG=1" "-D" "DEBUG=1" "-D" "COCOAPODS=1" "-D"
"OBJC_OLD_DISPATCH_PROTOTYPES=0" "-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/-own-target-headers.hmap"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/-all-non-framework-target-headers.hmap"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Products/Debug-iphonesimulator//include"
"-I"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/Pods/Headers/Private"
"-I"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/Pods/Headers/Private/"
"-I"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/Pods/Headers/Public"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/DerivedSources-normal/x86_64"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/DerivedSources/x86_64"
"-I"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/DerivedSources"
"-F/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Products/Debug-iphonesimulator/"
"-D" "NS_FORMAT_ARGUMENT(A)=" "-D" "_Nullable_result=_Nullable"
"-internal-isystem"
"/Applications/Xcode15.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator17.0.sdk/usr/local/include"
"-internal-isystem"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/facebook-clang-plugins/clang/install/lib/clang/18/include"
"-internal-externc-isystem"
"/Applications/Xcode15.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator17.0.sdk/usr/include"
"-O0" "-Wnon-modular-include-in-framework-module"
"-Werror=non-modular-include-in-framework-module" "-Wno-trigraphs"
"-Wno-missing-field-initializers" "-Wno-missing-prototypes"
"-Werror=return-type" "-Wdocumentation" "-Wunreachable-code"
"-Wno-implicit-atomic-properties" "-Werror=deprecated-objc-isa-usage"
"-Wno-objc-interface-ivars" "-Werror=objc-root-class"
"-Wno-arc-repeated-use-of-weak" "-Wimplicit-retain-self"
"-Wduplicate-method-match" "-Wno-missing-braces" "-Wparentheses"
"-Wswitch" "-Wunused-function" "-Wno-unused-label"
"-Wno-unused-parameter" "-Wunused-variable" "-Wunused-value"
"-Wempty-body" "-Wuninitialized" "-Wconditional-uninitialized"
"-Wno-unknown-pragmas" "-Wno-shadow" "-Wno-four-char-constants"
"-Wno-conversion" "-Wconstant-conversion" "-Wint-conversion"
"-Wbool-conversion" "-Wenum-conversion" "-Wno-float-conversion"
"-Wnon-literal-null-conversion" "-Wobjc-literal-conversion"
"-Wshorten-64-to-32" "-Wpointer-sign" "-Wno-newline-eof" "-Wno-selector"
"-Wno-strict-selector-match" "-Wundeclared-selector"
"-Wdeprecated-implementations" "-Wno-implicit-fallthrough" "-Wprotocol"
"-Wdeprecated-declarations" "-Wno-sign-conversion" "-Winfinite-recursion"
"-Wcomma" "-Wblock-capture-autoreleasing" "-Wstrict-prototypes"
"-Wno-semicolon-before-method-body" "-Wunguarded-availability"
"-Wno-ignored-optimization-argument" "-Wno-everything" "-std=gnu11"
"-ferror-limit" "19" "-fmacro-backtrace-limit=0" "-stack-protector" "1"
"-fno-builtin" "-fblocks" "-fencode-extended-block-signature"
"-fregister-global-dtors-with-atexit" "-fgnuc-version=4.2.1" "-fmodules"
"-fimplicit-module-maps"
"-fmodules-cache-path=/Users//Library/Developer/Xcode/DerivedData/ModuleCache.noindex"
"-fmodule-name=" "-fmodules-prune-interval=86400"
"-fmodules-prune-after=345600" "-fbuild-session-timestamp=1722308548"
"-fmodules-validate-once-per-build-session"
"-fmodules-validate-system-headers" "-fskip-odr-check-in-gmf"
"-fobjc-runtime=ios-13.0.0" "-fobjc-arc" "-fobjc-exceptions"
"-fexceptions" "-fpascal-strings" "-fmax-type-align=16"
"-fdiagnostics-show-note-include-stack" "-fasm-blocks"
"-serialize-diagnostic-file"
"/Users//Library/Developer/Xcode/DerivedData/iMFlower-anxiincrhnzgmifhgkosvigizfwr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/.build/Objects-normal/x86_64/HYCollectionViewController.dia"
"-D__GCC_HAVE_DWARF2_CFI_ASM=1" "-x" "objective-c"
"/Users//builds/hKr-Nzho/0/heiyeclient/-ios/LocalPods//Classes/HYCollectionView/CollectionView/HYCollectionViewController.m"
"-O0" "-fno-builtin" "-include"
"/Users//infer-osx-arm64-v1.2.0/lib/infer/infer/bin/../lib/clang_wrappers/global_defines.h"
"-Wno-everything"


Better alternatives for hamburger icon in demos navigation?

Currently, the hamburger icon for the collapsed navigation in the demos uses some rather...fugly markup.
<button ...>
    Toggle navigation

</button>

While of course quite flexible, this feels like a bit of a hack. It would be cleaner to implement something with, say, SVG or an icon font. Unfortunately, it seems that Glyphicons Halflings don't include a classic menu/hamburger glyph - but the full glyphicons set (not halflings) does have a "show_lines" one.
Is there any chance @fat or @mdo could try and convince the maintainer(s) of Glyphicon to add an equivalent (and free) show_lines halfling?
Alternatively, Font Awesome does have a suitable character http://fortawesome.github.io/Font-Awesome/icon/bars/ (incidentally, it's what I used recently for http://www.paciellogroup.com)

@jcohenho One was added in 3.3.2 as: glyphicon glyphicon-menu-hamburger

@OKNoah awesome, thanks for the update!


Sampling a Grpc request by matching the method name

Provide hooks to be able to plugin a sampler and see the Grpc method name, params and the headers.
This will be helpful for both b3 and secondary sampling to pick which requests should be traced.
For example consider the following service topology that uses grpc for all of them.
playback -> auth
         -> license

In the above example, consider a scenario to sample 100% of requests to the method play(movieId=123) and the given arguments.
This can achieved today by implementing a grpc ServerInterceptor and running it before the brave.grpc.TracingServerInterceptor to check the conditions and inserting the b3 or secondary sampling header.

This implies making a centralized type to represent the grpc request as the parameters are in different places.
Because the types are very much library-specific (ex headers are "Metadata" not plain types), we'd need something here as opposed to an abstraction (over this and other grpc implementations like armeria and wire).
@jorgheymans do you feel up to trying to make a GrpcServerRequest type, then using it as the parameter to extractor.extract?

sure will give it a try !
On Sat, Sep 21, 2019 at 4:54 AM Adrian Cole<EMAIL_ADDRESS>wrote:

This implies making a centralized type to represent the grpc request as
the parameters are in different places.
Because the types are very much library-specific (ex headers are
"Metadata" not plain types), we'd need something here as opposed to an
abstraction (over this and other grpc implementations like armeria and
wire).
@jorgheymans https://github.com/jorgheymans do you feel up to trying to
make a GrpcServerRequest type, then using it as the parameter to
extractor.extract?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/brave/issues/993?email_source=notifications&email_token=AABPKAG5A7DQUTYN3OWF353QKWEE5A5CNFSM4IYSV6AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7II6NY#issuecomment-533761847,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABPKAELLGROC6EIDOGT6UDQKWEE5ANCNFSM4IYSV6AA
.

@jorgheymans lemme know if you've work in progress. Otherwise, I can give it a spike as hoping for an integrated demo in the next couple days before I disappear for springoneplatform

#1000 fixes this


Wrap exported components with React.forwardRef()

Feature request
It would be useful to wrap DailyAudio, DailyAudioTrack, and DailyVideo in React.forwardRef().
Why you need this
I’d love to use <DailyAudioTrack />, but it’s useful to have a reference to the DOM node for HTMLMediaElements. For instance, it’s not possible to set volume under the current implementation, since you’d need access to audioEl.current.volume.
Alternatives you've considered
I think it’s most useful to simply accept a ref and join it to the internal ref with something like useMergeRefs[^1]. That way, the user can use the ref however they need.
But if that’s not to your taste, you could also define specific methods with useImperativeHandle.
It could also be nice to accept declarative props (like volume) and handle all of the effects internally, but it’s less flexible and more work on your end.
Additional context
I also think it would be useful to accept arbitrary props and spread them onto the returned element, like so:
export default React.forwardRefs(function DailyAudioTrack ({
  onPlayFailed,
  sessionId,
  type = 'audio',
  ...rest,
}, externalRef) {
  const audioEl = useRef(null);

  // ...

  const ref = useMergedRef(audioEl, externalRef);
  return (
    <audio
      autoPlay
      data-session-id={sessionId}
      data-audio-type={type}
      playsInline
      ref={ref}
      { ...rest }
    />
  );
});

That way, you can pass in arbitrary properties like muted.

[^1]: Reference implementation:
```.js
// Takes multiple refs and merges them into a single ref so you can attach
// multiple refs to the same element.
export function useMergedRef(refs) {
  const filteredRefs = refs.filter(Boolean);
  const mergedRef = useCallback(el => {
    filteredRefs.forEach(ref => {
      if (typeof ref === 'function') {
        ref(el);
      } else {
        ref.current = el;
      }
    });
  }, filteredRefs);
  return mergedRef;
}
```

Thanks @rileyjshaw, this is an excellent feature request and your reasoning is very understandable.
I'll have a look into this and get back to you with updates.
Cheers,
Christian

@rileyjshaw I have a question for you about DailyAudio: what would you expect a forwarded ref to point to? Would that be the complete list HTMLAudioElement[] of rendered audio tracks?

@Regaddi great question! I hadn’t considered that. Here are some quick thoughts. I was thinking through the problem while typing this, so Option 4 is my favourite. Feel free to skim the rest  😄
Option 1: Do nothing
The use case for forwarding refs to DailyAudioTrack and DailyVideo are more directly evident. I think it would be fine to say “if you need convenience, go for DailyAudio. If you need configurability, go for DailyAudioTrack”.
…but it would be really nice if DailyAudio was also configurable, so on to:
Option 2: Define a set of acceptable props for both DailyAudio and DailyAudioTrack
Quoted from above:

you could also define specific methods with useImperativeHandle

It could also be nice to accept declarative props (like volume) and handle all of the effects internally, but it’s less flexible and more work on your end.

Since DailyAudio already uses DailyAudioTrack behind the scenes, you could use a similar signature for both components. Since it will take some work to define the signature anyway, hopefully props can be used for most configuration, and hide some of the imperative implementation required to interface with the underlying <audio> element.
This sounds like a lot of work, but I think a volume prop is probably the thing most people will actually want to use. You could forget other configuration options for now (and skip useImperativeHandle), and revisit it later if anyone creates a new issue.
Option 3: Return a list of HTMLAudioElement[] tracks
This might be hard to document, as it requires the library consumer to understand a bit about how the underlying component is put together. But it could be a useful “power user” feature, and allows for a lot of flexibility!
Option 4: Combine options 2 and 3
I think this is my favourite option of the bunch.

Create a declarative volume prop on DailyAudioTrack that works like this:

// Example stub:
const AudioExample = forwardRef(({ volume = 1, ...rest }, ref) => {
	const audioEl = useRef(null);

	// When volume changes, update the audio element’s volume.
	useEffect(() => {
		if (audioEl.current) {
			audioEl.current.volume = volume;
		}
	}, [volume]);

	// Initialize the audio element with the correct values.
	const audioCallback = useCallback(
		el => {
			if (el) {
				el.volume = volume;
			}
		},
		[volume]
	);

	const audioRef = useMergedRef([audioEl, audioCallback, ref]);

	return <audio ref={audioRef} autoPlay playsInline {...rest} />;
});

Allow additional arbitrary props to be spread onto the <audio> or <video> elements, in case someone wants to pass in a style object, overwrite playsInline, etc. (shown above)
Leave the rest of the wiring up to the library consumer by adding forwardRef to DailyAudioTrack (shown above) and exposing a list of refs from DailyAudio (discussed under Option 3).
If users really want other convenience props added in the future, or extra methods using useImperativeHandle, those can be added in the future. But my instinct is that volume, style, and a forwarded ref will cover almost all use cases.

Thanks a lot for the detailed input, @rileyjshaw!
I really appreciate your thinking here, it's invaluable!
I'd like to propose another, slightly adjusted, option, based on your options 2, 3 and 4:
Instead of applying a plain ref to DailyAudio, which would eventually hold a reference to all rendered <audio> elements (as proposed in option 3), I'd like to use that passed ref to setup an imperative handle with a few methods to query for the exact <audio> elements a developer would be interested in:
interface DailyAudioHandle {
  getAllAudio(): HTMLAudioElement[];
  getActiveSpeakerAudio(): HTMLAudioElement;
  getScreenAudio(): HTMLAudioElement[];
  getAudioBySessionId(id: string): HTMLAudioElement;
  getScreenAudioBySessionId(id: string): HTMLAudioElement;
}

export const DailyAudio = forwardRef<DailyAudioHandle, Props>((props, ref) => {
  const containerRef = useRef<HTMLDivElement>(null);

  useImperativeHandle(ref, () => ({
      getAllAudio: () => containerRef.current?.querySelectorAll('audio'),
      getActiveSpeakerAudio: () =>
        containerRef.current?.querySelector(
          `audio[data-session-id="${activeSpeakerId}"][data-audio-type="audio"]`
        ),
      getScreenAudio: () =>
        containerRef.current?.querySelectorAll(
          'audio[data-audio-type="screenAudio"]'
        ),
      getAudioBySessionId: (id: string) =>
        containerRef.current?.querySelector(
          `audio[data-session-id="${id}"][data-audio-type="audio"]`
        ),
      getScreenAudioBySessionId: (id: string) =>
        containerRef.current?.querySelector(
          `audio[data-session-id="${id}"][data-audio-type="screenAudio"]`
        ),
    }));

  return (
    <div ref={containerRef}>
      {/* rendered audio tracks *}
    </div>
  );
});

Rationale:

Returning a single HTMLAudioElement[] list means that developers have to manually filter for the desired <audio> element based on the data- attributes we render. Providing methods which already contain this filtering should result in less maintenance effort when updating to newer versions of @daily-co/daily-react.
Because the actual selectors stay contained within DailyAudio we can add tests for those, making sure that we don't accidentally break the imperative handle in the future.
I like the explicit nature of imperative handles in combination with strict types. It makes documentation a little easier and makes use of a known concept in React.

Let me know what you think, otherwise I'll probably go ahead with this approach ☺

@Regaddi this sounds so useful!! Great idea 😄


Update DR plans in light of new pipelines

Like all changes to infrastructure, since we're changing pipelines, we need to check/update DR plans:
https://app.zenhub.com/workspaces/cloud-platform-team-5ccb0b8a81f66118c983c189/issues/ministryofjustice/cloud-platform-user-guide/195

Moving issue to the correct repo: https://github.com/ministryofjustice/cloud-platform/issues/3424


Can you replicate the examples on macOS?

Let us know if you had to significantly deviate from the instructions!

I can do this

@tekjar Thanks! To clarify: do you mean you can run the examples on macOS? Or that you would test this (but haven't yet)?

Oops. Sorry for the confusion. Yet to test.

@japaric I have tested the example in Memory layout page and got the follow error :
error: language item required, but not found: `eh_personality`
error: aborting due to previous error
error: Could not compile `app`.
To learn more, run the command again with --verbose.

What could be the issue?
Thanks .

@odunboye sounds like you are compiling the program for the host instead of cross compiling. Did you set up a default build target in .cargo/config as covered in the last part of the first section (smallest std program)? If you missed it, it may be good to mention it again in the second section.

@japaric I was able to compile it by setting the .cargo/config content
Currently testing the rest of the examples on MacOS
Thanks

@japaric I was able to test the example up until the very last one where I can't generate the "release" version of the output like target/thumbv7m-none-eabi/release/app but was able to compile to the debug output.
What compilation flag compiles for the release version?

Try using the --release argument when building.
Note to self: add this missing info to the book.

I ran through the whole thing (almost) without issue on MacOS 10.12.6.
I did run into issues with the lldb related stuff, but it seems to be Python problems.  From the end of the "Memory layout" section:
$ lldb target/thumbv7m-none-eabi/debug/embedonomicon
(lldb) target create "target/thumbv7m-none-eabi/debug/embedonomicon"
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File<EMAIL_ADDRESS>line 52, in <module>
    import weakref
  File<EMAIL_ADDRESS>line 14, in <module>
    from _weakref import (
ImportError: cannot import name _remove_dead_weakref
Current executable set to 'target/thumbv7m-none-eabi/debug/embedonomicon' (arm).
(lldb) gdb-remote 3333
Process 1 stopped
* thread #1, stop reason = signal SIGTRAP
    frame #0: 0x00000008 embedonomicon`Reset at main.rs:9
   6
   7    #[panic_handler]
   8    fn panic(_panic: &PanicInfo) -> ! {
-> 9        loop {}
   10   }
   11
   12   // The reset handler
Target 0: (embedonomicon) stopped.
(lldb) disassemble -frame
embedonomicon`Reset:
->  0x8 <+0>:  sub    sp, #0x4
    0xa <+2>:  movs   r0, #0x2a
    0xc <+4>:  str    r0, [sp]
    0xe <+6>:  b      0x10                      ; <+8> at main.rs:18
    0x10 <+8>: b      0x10                      ; <+8> at main.rs:18
(lldb) print/x $sp
error: need to add support for DW_TAG_base_type '!' encoded with DW_ATE = 0x7, bit_size = 0
(unsigned int) $0 = 0x20010000
(lldb) step
Process 1 stopped
* thread #1, stop reason = step in
    frame #0: 0x0000000c embedonomicon`Reset at main.rs:15
   12   // The reset handler
   13   #[no_mangle]
   14   pub unsafe extern "C" fn Reset() -> ! {
-> 15       let _x = 42;
   16
   17       // can't return so we go into an infinite loop here
   18       loop {}
Target 0: (embedonomicon) stopped.
(lldb) step
Process 1 stopped
* thread #1, stop reason = step in
    frame #0: 0x0000000e embedonomicon`Reset at main.rs:18
   15       let _x = 42;
   16
   17       // can't return so we go into an infinite loop here
-> 18       loop {}
   19   }
   20
   21   // The reset vector, a pointer into the reset handler
Target 0: (embedonomicon) stopped.

However, things mostly worked :).  Looking at https://github.com/flutter/flutter/issues/17803, there is probably some brew/Python finagling that could be done to make it work.
Might be worth noting in the embedonomicon that installing QEMU and qemu-system-arm on MacOS is as simple as brew install qemu.  Googling "install qemu macos" and/or "install qemu-system-arm macos" doesn't yield a lot of clear information to get what is needed only for this guide (and not QEMU for emulating a Raspberry Pi or something else, which I saw a lot of).

@twilco thanks for the report. Some windows users are also having trouble with lldb so I think we'll recommend gdb instead of lldb.

Might be worth noting in the embedonomicon that installing QEMU and qemu-system-arm on MacOS is as simple as brew install qemu

Thanks for the info! I'll add a small section to the preface covering how to install the tools on macOS.


CrossAttention To CrossMamba

Dears Authors!
Thank you for your great work! I have a question about adapt from cross attention to cross mamba.
Can i modify mamba from this

to this (with source role similar to output of encoder in transformer and x role similar to query of decoder)

Do you think it will work, because the projection of source to B and C can help to cross feature with x ?

The source could have a different length to the target I think? So using the source to compute the projections won't give you the correct dimension?

Hmm, I think this approach can just work when x and source have the same length and cannot work when the length is different like cross attention.

@Kiet0712 Hey, I came across this paper: https://arxiv.org/abs/2407.10998 not sure if it meets your needs but take a look.

@Kiet0712 I think this could help https://arxiv.org/abs/2409.04803


Create Window_Sliding_Technique_Algo implementation in Java

Please follow the Contributing Guidelines & naming conventions in the guidelines accordingly. 🙂
Following naming guidelines is important to maintain consistency.😊
Please NOTE: In case, same implementation of an algorithm is submitted by someone else and it gets accepted, then your contribution won't be merged ( to avoid duplicates )

Can you provide more information about what exactly you want here?

Hey I would like to work on this.  Could you please assign it to me ?

Hey I would like to work on this. Could you please assign it to me ?

Already submitted  a PR for this Java implementation of Window_Sliding_Technique_Algo .
I hope it helps !


The new .pyw is not running

Before update I used to run .exe and software ran normally.
Did the installrequirements.bat part, then
Tried clicking on the ".pyw" file to open but it won't run. I had to type in "cmd" on explorar bar and run it that way. But I need to keep command prompt open or if not it will close.

Yo!
Inside /windows_src there's a TournamentStreamHelper.bat. Can you try copying that to the root folder and running from it?

Hey! Tried what you suggested but it had the same effect, seems like a command promt opens up and closes itself fast. Still  program does not run.

What happens if you open a Command Prompt and type python -version and press enter?
Also, is there anything in your log_error.txt after you try running it using the bat? Also, check if the bat's content is this: start "" pythonw src/TournamentStreamHelper.pyw (you can edit it using right click > open with notepad)

Hello! Did everything you said, got the updated 3.9 Python. Also log error doesnt show any mistake.
Saw the bat content and it said what you wrote to me.
Finally what I did was switch the bat information with python TournamentStreamHelper.pyw and now it opens up the program but still opens a command prompt.

I see... I'll have to investigate more towards this. I believe there is a way of running a bat in a way that it hides this command prompt. Also in theory a pyw should be detected as such and ran using pythonw.exe which is the executable that doesn't show a command prompt window.
Atm all the time I have available for this project I'm focusing in developing version 4.0 with many new features and fixes (it's almost a complete rewrite of the program), but I'll make sure to take some time to look into this!

So it runs! I can even close the Command promt now but did what i told you on the last message.
Its ok my guy, I reached out because maybe that was a common issue, trying to help out a bit. Your program is amazing.

Starting with 4.0, an exe will also be included!


PutAll() handling fix

PutAll() successfully adds the records to orbitdb, but they aren't added correctly to the index and can't be get()ed. This should fix that. Check the get() method above to understand the bug: It looks for the "paylod" value of a document but that isn't set correctly in the updateIndex() function.

LGTM, @phillmac can you confirm/


Add downloads and uploads tables in database

These tables can be inserted into when the save_squeak method is called in the controller.

Don't do this.
Instead, add an extra field to the squeak table to record where it was downloaded from.


feat: Add InlineDFG rewrite

Much as per the spec
Rejects cases where the DFG node has Order edges
Works for permutation (swap) DFGs!

Test failure is something to do with extension inference ("MismatchedConcreteWithLocations"), haven't figured out what extension annotation I need to add yet


February 14th, 2022 Community Meeting

Please add a topic in this thread and add a link to the GitHub issue associated with the topic.
Please make sure you give folks enough time to review/discuss the topic offline on GitHub before coming into the meeting
(optional) Paste the image of an animal 😸

https://github.com/shipwright-io/image/issues/4 -- Centralized repository for all the API/CRD resources (/cc @ricardomaraschini)

Image Abstraction Layer EP https://github.com/shipwright-io/community/pull/57

https://github.com/shipwright-io/community/discussions/60

Consolidate CRDs

Enrique: Tight relationship between CRDs and business logic
Jason: Tekton and Knative have CRDs and code also co-located
Sascha: we would have one more repo (build) which must get adopted after the API vs having at least one where one can do changes together
Otavio: we would need to change the APIs first as this is usually most discussed; then later the implementation can adopt
Enrique: why is OpenShift doing this -> Otavio: API stability, must not be broken
Sascha: would be easier once our API is more stable
Otavio: important is to review the API first -> Enrique: happens in the SHIP
Conclusion:

We made the point that APIs need to be discussed in Ships
As of today, moving the APIs into an own project is considered over-complicating the process
At some point in the future when we are more stable, this can be considered again

Image Abstraction Ship #27

Ricardo inviting everybody to read it: https://github.com/shipwright-io/community/pull/57
PoC code available in https://github.com/shipwright-io/image/, any code in there can still change. Is just a special situation because the overall component is new.

Owner rotation introduction by Enrique

Sascha: I like it, especially that we document what all needs to be looked at
Otavio: how is Tekton doing this? -> Enrique giving details
Jason: Tekton approach has same goal, but does not work perfectly because people are sometimes busy; not all owners / maintainers are doing it in Tekton
Enrique: should be a best-can-do jobrole

Enrique recommending https://github.com/shipwright-io/community/pull/53

Ricardo: seeing benefits of storing a tarball with the image proposal
Jason: Tekton had similar scenarios but never came to it

Matthias: playing around with OpenShift, what is the internal registry usually used for?

Ricardo: store image results of builds; cache images; disconnected environment where only the internal registry can go outside; oob references to common quay images

/close


Diff is not properly applied when using SingleIdSpace

Context

This is a bug I encountered when applying updates using a mapping with use_single_id_space: true
Expected Behavior

imposm diff should reinsert all corresponding ways when a node has been moved.

Here is a patch that adds a failing test
diff --git a/test/single_table.osc b/test/single_table.osc
index c349766..faf219a 100644
--- a/test/single_table.osc
+++ b/test/single_table.osc
@@ -23,4 +23,9 @@
       <tag k="building" v="office"/>
     </relation>
   </modify>
+
+  <modify>
+    <node id="20002" version="2" timestamp="2011-11-11T00:11:11Z" lat="43" lon="13"/>
+  </modify>
+
 </osmChange>
diff --git a/test/single_table_test.go b/test/single_table_test.go
index 19cf852..627b064 100644
--- a/test/single_table_test.go
+++ b/test/single_table_test.go
@@ -209,6 +209,10 @@ func TestSingleTable(t *testing.T) {
                }
        })

+       t.Run("NodeHasMoved", func(t *testing.T) {
+               ts.assertGeomLength(t, checkElem{"osm_all", -20201, "*", nil}, 222639)
+       })
+
        t.Run("Cleanup", func(t *testing.T) {
                ts.dropSchemas()
                if err := os.RemoveAll(ts.dir); err != nil {

Actual Behavior

The test fails, as if the ways that contain the node that have been updated were not in cache.
As far as I can understand, this is due to how the writer handle element ids.
On the initial import, the way Id is updated before being written to diffcache:  https://github.com/omniscale/imposm3/blob/db15f93da1b3bd600a1216a6386ccfb4bd7f02af/writer/ways.go#L91
Whereas the update process expect "normal" positive ids in cache.
Possible Fix

Here is a quick fix for my test case:
diff --git a/writer/ways.go b/writer/ways.go
index 4cc7749..ca16681 100644
--- a/writer/ways.go
+++ b/writer/ways.go
@@ -88,6 +88,7 @@ func (ww *WayWriter) loop() {
                        return true
                }

+               originalId := w.Id
                w.Id = ww.wayId(w.Id)

                var err error
@@ -123,6 +124,8 @@ func (ww *WayWriter) loop() {
                if (inserted || insertedPolygon) && ww.expireor != nil {
                        expire.ExpireProjectedNodes(ww.expireor, w.Nodes, ww.srid, insertedPolygon)
                }
+
+               w.Id = originalId
                if (inserted || insertedPolygon) && ww.diffCache != nil {
                        ww.diffCache.Coords.AddFromWay(w)
                }

But I am not very familiar with the implementation. Maybe this is expected in diffcache, and that's the update process that should be patched ?
Note that a similar issue may exist with relations, since the RelationWriter does the same kind of operations with element ids before writing to diffcache.

Did anyone get a chance to look into this?
Or would you have some tips to write proper test cases? Using assertGeomLength is a bit cumbersome.


Default value is not stored, which means the read value can change unexpectedly.

Imagine the following. Server has a random id - which can be reset if needed.
extension Defaults.Keys {
   static let vidServerId = Key<String>("vidServerId",default:UUID().uuidString)
}

##Expected

on first read, I get a random value
when I relaunch the app - I get the same value

##Actual
every time I launch the app - I get a new value.

It seems that the default is read each time on launch and saved into the default values.
This means that when you read a default value - you can't tell whether it is a saved value, or one that might change.
The same issue could cause wierd behaviour with user preferences
extension Defaults.Keys {
   static let importantChoice = Key<Bool>("choice",default:false)
}

After install:
User A left the choice unchanged
User B changed to true, then back to false
If I update my app to change the default
extension Defaults.Keys {
   static let importantChoice = Key<Bool>("choice",default:true)
}

Now - I have changed the setting for User A, but not for User B.

This is just how native NSUserDefaults works, and is in my opinion, the expected behavior. It's a default fallback value, not an initial value. Even if I agreed with you, it would be too late to change the behavior of this now.
To achieve what you want, you can do Defaults[.vidServerId] = Defaults[.vidServerId] at launch.


Document the new extra specs features

As part of https://github.com/cloudbase/garm/pull/143, we also added the ability to override the default runner installation template for providers that leverage the garm-provider-common package.
We need to document that feature.

That change also adds the ability to insert pre_install_scripts for Linux runners, which is also important to document.

This should be documented as part of each provider that decided to implement the functionality.


investigate following vs subscription preference data

[ ] can't unfollow if record has a :
[ ] Do we default to notifications enabled if not in following list?
[ ] test with ios app

@infinite-persistence confirmed this breaks all unfollowing ability when IOS users follow/unfollow something. This converts all # to :. And then the : records don't reflect in the UI when you unfollow.
This would be another reason to eventually do https://github.com/OdyseeTeam/odysee-frontend/issues/547


Add upgrade subcommand for self-upgrading

We'll need to flesh this out a bit more, I think it would be good to confirm with the user if they want to upgrade and provide a link the release notes so they can decide if they want to upgrade or not. Such as the typical [Y/n] option, and a -y flag can be added if they upgrade without being asked, or running in automation where they can't wait for user prompt.


fix(factory): unwrap config to not end up with tomlkit instances

Resolves python-poetry/poetry/issues/6201.

[ ] Added tests for changed code.
[ ] Updated documentation for changed code.

WIP for now, since we need to measure the impacts of this change, and this is missing tests.

Closing, since #483 was merged and we no longer use tomlkit in poetry-core


Add change log file and more badges

Extract change log into dedicated file
Add more badges to top of README file

Additional updates:

Include Python 3.7 in Tox
Use build stages on Travis CI

Fixes #16.

Codecov Report

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

@@          Coverage Diff          @@
##           master    #17   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines         150    150
=====================================
  Hits          150    150

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 bf1e37e...7b23b81. Read the comment docs.

Thank you @bittner


⚠ Bofrost.lu has degraded performance

In 0b760dd, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 986 ms

Resolved: Bofrost.lu performance has improved in 9d5a977 after 32 minutes.


quaternion vs. 6d representation

Hi,
I am curious about the use of quaternion and 6d representations for rotational parameters. I've noticed that you utilize a 6D representation in GenPose. My question is, when using quaternions for these rotational parameters, does it lead to a decrease in performance? Or are there any significant issues when deniose the quaternion parameter?

We chose the 6D representation in GenPose over quaternions to address the discontinuity of quaternions and Euler angles in Euclidean space, following the approaches in references [1, 2]. This ensures better performance and stability in handling rotational parameters.
[1] Wei Chen, Xi Jia, Hyung Jin Chang, Jinming Duan, Linlin Shen, and Ales Leonardis. Fs-net:
Fast shape-based network for category-level 6d object pose estimation with decoupled rotation
mechanism. In Proceedings of the IEEE/CVF Conference on Computer Vision and Pattern
Recognition, pages 1581–1590, 2021.
[2] Yi Zhou, Connelly Barnes, Jingwan Lu, Jimei Yang, and Hao Li. On the continuity of rotation
representations in neural networks. In Proceedings of the IEEE/CVF Conference on Computer
Vision and Pattern Recognition, pages 5745–5753, 2019.

Thank you for your reply. I was wondering if you have done any experiments to see how much the 6d representation improves performance over the quaternion representation in your task?

Thank you for your question. In our preliminary tests on data about 1/10th the size of our full dataset, the 6D representation showed an advantage (however, it was not an extremely significant leap). We didn't conduct these comparisons on the entire dataset, but we're planning to conduct more extensive testing in our future work. Keep an eye out for further updates!

Thank you so much.


MAINT: fft: Remove outdated documentation workaround

Reference issue
None filed.
What does this implement/fix?
In version 1.5.0 of SciPy, scipy.fft was a callable function. In order to support building Sphinx documentation, there was an environment variable _SCIPY_BUILDING_DOC which was used to disable this functionality because it interfered with building documentation somehow.
In #12558, this was deprecated and removed, and scipy.fft is no longer callable. The _SCIPY_BUILDING_DOC environment variable now has no purpose, and is not referenced anywhere else in the codebase. However, it was not removed from conf.py.
The docs still build correctly without this variable being defined.
Therefore, I propose that we remove it.
Additional information

Thanks @nickodell .


wip Remove numpy pinning

This PR removes the pinning for numpy that was used to resolve a conflict with pyarrow.
resolves #27

it seems that now it has some conflict with rbc 0.5.1:
func = <function add_number.<locals>.my_add_number at 0x7fee725c7ca0>
sig = Type(Type('float32',)|{'annotation': {}}, (Type('float32',)|{'annotation': {}}, Type('float32',)|{'annotation': {}}))
target = <rbc.targetinfo.TargetInfo object at 0x7fee728ba340>
typing_context = <numba.core.typing.context.Context object at 0x7fee725d49a0>
target_context = <rbc.irtools.JITRemoteCPUContext object at 0x7fee725d4df0>
pipeline_class = <class 'rbc.omnisci_backend.omnisci_pipeline.OmnisciCompilerPipeline'>
main_library = <Library 'rbc.irtools.compile_to_IR' at 0x7fee726faa00>
debug = False

    def compile_instance(func, sig,
                         target: TargetInfo,
                         typing_context,
                         target_context,
                         pipeline_class,
                         main_library,
                         debug=False):
        """Compile a function with given signature. Return function name when
        succesful.
        """
        flags = compiler.Flags()
>       flags.set('no_compile')
E       AttributeError: 'Flags' object has no attribute 'set'


clean up referenceScales

there's a lot of of inconsistency and TODOs in it

originally the goal was to clean up this class, but i've been slowly reorganizing it, and now it's more or less ok. the only remaining thing is that not every a(beta) function has the beta range over which its coefficients were determined. it would be nice if someone could do a little archaeology to figure this out.

i filled in the ones i could find


[BEAM-12153] Implement GroupByKey with CombinePerKey with Concatenate CombineFn

As flink runner does in GBK translation.
It is to avoid OOM due to the materialization of the collection in current GBK impl (see details in the ticket)
This PR is based on https://github.com/apache/beam/pull/15218 which makes the SS runner modifiable back again.
R: @ibzib
CC: @aromanenko-dev

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Spark StructuredStreaming ValidatesRunner

@ibzib thanks one again for prompt review !

Run Flink ValidatesRunner

Run Samza ValidatesRunner

Run Spark ValidatesRunner

Run Spark StructuredStreaming ValidatesRunner

rebased on master

Run Spark StructuredStreaming ValidatesRunner

Run Spark StructuredStreaming ValidatesRunner

Run Spark StructuredStreaming ValidatesRunner

Run Spark StructuredStreaming ValidatesRunner

Run Java Precommit

What is the next step on this PR?

@aaltay next step is getting back from vacation, which just happened :smile:
I'm gonna update the PR today.

@aaltay next step is getting back from vacation, which just happened 😄
I'm gonna update the PR today.

Welcome back :)

ES test split seems flaky (no split in my test)

rebased on master

@aaltay next step is getting back from vacation, which just happened smile
I'm gonna update the PR today.

Welcome back :)

thanks Ahmet !

Run Spark StructuredStreaming ValidatesRunner

@ibzib I addressed all your comments and validates runner tests pass. Still the java build is flaky with unrelated failures:
org.apache.beam.runners.flink.FlinkSavepointTest.testSavepointRestoreLegacy org.apache.beam.runners.dataflow.worker.StreamingDataflowWorkerTest.testHotKeyLoggingNotEnabled[1: [streamingEngine=true]] org.apache.beam.runners.dataflow.worker.StreamingDataflowWorkerTest.testHotKeyLogging[1: [streamingEngine=true]] org.apache.beam.sdk.io.elasticsearch.ElasticsearchIOTest.testSizes
Can we merge the PR ?

I did final commits cleanup, @aaltay if you're available, can we merge the PR ?

Run Spark StructuredStreaming ValidatesRunner

Thanks Etienne, I like all the deleted lines :)

@ibzib I addressed all your comments and validates runner tests pass. Still the java build is flaky with unrelated failures:

We don't want to merge PRs with test failures. If these tests are known flakes, you can propose sickbaying them.

I agree, see my comment in related ticket: https://issues.apache.org/jira/browse/BEAM-5172?focusedCommentId=17409348&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-17409348
Sickbaying them.

Run Spark StructuredStreaming ValidatesRunner

Run Spark StructuredStreaming ValidatesRunner

@ibzib I addressed all your latest comments and squashed. I also raised the priority of Elasticsearch falkes ticket that I ignored in this PR. Can we merge ?

@aaltay , @ibzib can we merge this PR ?

Run Spark StructuredStreaming ValidatesRunner

@ibzib all addressed, PTAL


Added vid.me ripper

Rips a video from vidme.
Does not support ripping all videos of a user

Thanks.


Fix component stacks for tsx, ts, and jsx files

Summary:
Component stacks with files ending in .ts, .tsx, or .jsx were skipped in LogBox reporting. This diff fixes the regex.
Changelog:
[General][Fixed] - Support .tsx, .ts, and .jsx in component stacks
Differential Revision: D54638526

I've also requested a cherry pick for 0.73 here => https://github.com/reactwg/react-native-releases/issues/133


Add which_vars to minimizeNelderMead

Added option to HARK.estimate.minimizeNelderMead to select which parameters are estimated by passing a vector of booleans of the same size as parameter_guess.  When not passed, it defaults to estimating all of the parameters (previous behavior).  Not a breaking change.
This code has been sitting on my computer for many months.  I only realized it wasn't in public HARK when I switched one of my projects from a local, private version of HARK to public package HARK.

This has been reviewed and is ready to merge.  Anybody seeing this who has permission should do it.


Updated app setting URL and Gatling tests

📲 What

Updated the app setting base URL to local one as dev ul does not work and make no sense when cqdrs-events template is used to replace the domain name.

Removed the Gatling test solution from the repo completely. A separate gatling repo is available already. So this isn't required in this repo any more.

✅ Acceptance criteria Checklist

[ ] Code peer reviewed?
[ ] Documentation has been updated to reflect the changes?
[ ] Passing all automated tests, including a successful deployment?
[ ] Passing any exploratory testing?
[ ] Rebased/merged with latest changes from development and re-tested?
[ ] Meeting the Coding Standards?

/azp run


Fix macOS title bar buttons not displaying properly.

In order for macOS windows to display buttons properly, we can't hide macOS native buttons, otherwise the title bar will appear on top of the buttons. The easiest manual solution is to refresh the window's flag, but for convenience, it's better to just not hide it.
Fix #60 .

好的，统一修改成NO应该能避免出现一些意料外的情况，已经修改。


[T3A3][W09-A3]Yim Chia Hui

Split Address into smaller classes Block, Unit, Street and PostalCode

Ready for review

Ack by closing PR.


Build error in TypeScript 5.0

error TS1239: Unable to resolve signature of parameter decorator when called as an expression. Argument of type 'undefined' is not assignable to parameter of type 'string | symbol'.

Related issues

https://github.com/nestjs/nest/pull/10970
https://github.com/microsoft/TypeScript/pull/53365

InjectPinoLogger type output from<EMAIL_ADDRESS>// AS-IS
export declare function InjectPinoLogger(context?: string): (target: object, key: string | symbol, index?: number | undefined) => void;
// TO-BE
export declare function InjectPinoLogger(context?: string): (target: object, key: string | symbol | undefined, index?: number | undefined) => void;

The above return type is created as the return type of the Inject function of @nestjs/common.
This issue has been fixed in<EMAIL_ADDRESS>In nestjs-pino, you need to build and deploy it again with the updated internal dependency version to change the type.

I have the same issue on my projects, and it's blocking us to use TS 5, do you need help @iamolegga?

This is not just an update of the typescript version and fixing what's broken. This should also be tested with different typescript versions of dependent projects. So, it requires a bit more time to not break all the things. I should be confident if this is a breaking change or not to choose the right semver for this update. Please be patient, or fork and fix it to make it work for your particular project

Sorry for the long waiting, the fix was published in 3.1.3


How can I obtain the file ram_plus_swin_large_14m.pth?

I want to test the recognition performance of RAM and RAM++on my traffic dataset, but I couldn't find the corresponding download link. I'm not sure if the author hasn't published it or if I haven't seen it

刚看到作者是复旦的博士大佬，不装了摊牌了，大哥这个plus的权重是没开源还是我没找到啊，如果没开源的话有没有在线的平台可以测试一下，想看一下实验效果


default response from getResponse

Fixes #166
Return default response from getResponse if getResponses does not have a matching status code.

I signed it!


wsk activation poll doesn't show anything after a trigger was created

The documentation https://new-console.ng.bluemix.net/docs/openwhisk/openwhisk_catalog.html#openwhisk_catalog_cloudant states in section "Listening for changes to a Cloudant database" that wsk activation poll should show when the trigger is executed.
But this only works after a rule has been created that includes that same trigger and an action.
Could the documentation be updated? Or even better log something when the trigger is executed.

I have noted a similar somewhat related behavior, that activation poll only registers information when "read" and "write" actions are invoked, but not when database content is changed or created using the Cloudant Dashboard (inside Bluemix)

This is a duplicate of #81.


CHW/ANC - View Past Encounters for an individual household member profile

Query and view all the past encounters for a patient and add pagination or continous scroll view
Access from menu item

View past encounters

[ ] View status of the task  - check status field
[ ] View detail of the encounter type  - check class.display field

sample encounter resource
{
  "resourceType": "Encounter",
  "id": "example",
  "text": {
    "status": "generated",
    "div": "
Encounter with patient @example
"
  },
  "status": "in-progress",
  "class": {
    "system": "http://terminology.hl7.org/CodeSystem/v3-ActCode",
    "code": "IMP",
    "display": "inpatient encounter".
  },
  "subject": {
    "reference": "Patient/example"
  }
}

@ellykits I think this is a good candidate to showcase continuous scroll view/loading new details. Let me know what you think. Thanks

@f-odhiambo were are we showing these encounters?
If it's in that "past encounters" section then I think we don't need to show some statuses since it's PAST encounters.
I think maybe there's need for us to discuss if we need all these statuses.
cc: @rowo

@f-odhiambo were are we showing these encounters?
If it's in that "past encounters" section then I think we don't need to show some statuses since it's PAST encounters.
I think maybe there's need for us to discuss if we need all these statuses.
cc: @rowo

Hi, @HenryRae This is accurate. You can progress based on the discussion we had on the call, to add icons for encounters that have been done in the past or cancelled. Future tasks may not apply here

@HenryRae For upcoming services/encounters will update the icons for this tickets #543

@syedowaisali90 Here are updated images and icons for update and also updated mock


Minor fixes for repository client

As requested in https://github.com/ubisoft/NGitLab/pull/40

Thanks @dianaMihaelaDiaconu for taking the time to make these changes.


Pipes don't update after language change

Is your feature request related to a problem? Please describe.
This may or may not be a feature request for updating the documentation:
When using .translate() within pure pipe's the pipe does not rerender when changing the language. This makes sense do to the design of pure pipes, but the docs may state an efficient way to deal with that.
Describe the solution you'd like
Update the docs to have a "Usage within pipes" topic. The only solution I can think of is also inputting the language which would trigger rerendering:
@Pipe({
  name: 'myAwesome'
})
export class MyAwesomePipe implements PipeTransform {
  constructor(private translocoService: TranslocoService) {
  }

  transform(state?: MyEnum, lang?: string): string {
    switch (state) {
      case MyEnum.A:
        return this.translocoService.translate('a');
      case FootballCup.Bundesliga2:
        return this.translocoService.translate('b');
      default:
        return '';
    }
  }
}

But then we also need the active language to be available within the template:
{{ myVar | myAwesome: activeLang }}

Which is kind of complicated because we need the translocoService be public avaiable within the template:
{{ myVar | myAwesome: (translocoService.langChanges$ | async) }}

I just figured out the possible approach I showed above does not work because langChanges$ triggers immediately and not after the language has been loaded so nothing changes.

Implement #461, please!

@MickL With which solution did you proceed? As I'm struggling with the same issue.

@pablomaurer did you try to use langChanged$ in your pipe?

Ah you still go the way to add the lang change as an param to re-trigger the rendering? I thought there's maybe a way without doing that. Because this way I have to add the translocoService everywhere where i use my custom pipes (don't like that).

+1


Smtp Problem (external email client)

Hello,
i'm can't send email from a external email client (thunderbird)
I receive this message when i try:
Um erro (SMTP) ocorreu ao enviar mensagens.
Resposta do servidor: “5.7.1 : Data command rejected: Blocked - contact<EMAIL_ADDRESS>for help - Suspicious HELO [[<IP_ADDRESS>]] and missing reverse DNS [<IP_ADDRESS>]”.

Hi,
I have the same exact issue. I also use Thunderbird.
Alert
An Outgoing server (SMTP) error occurred while sending mail. The server responded:  5.7.1 : Data command rejected: Blocked - contact<EMAIL_ADDRESS>for help - Suspicious HELO [...] and missing reverse DNS [...].

Sorry for the necromancy, but I have the same problem. It seems to be related to postfwd...


[Performance] Visual Studio locks up after a build if you have diagnostic build turned on

File -> New -> Console App (.NET Framework)
Add -> New -> Console App (.NET Core)
Tools -> Options -> Project and Solutions -> Build And Run -> MSBuild project build output verbosity -> Diagnostic
Click OK
Right-click on .NET Framework project and choose Rebuild

-- Notice that the build is instant and Visual Studio does not lock up

Right-click on .NET Core project and choose Rebuild

-- Notice that builds takes > 10 seconds and locks up Visual Studio during it --

https://github.com/dotnet/roslyn-project-system/issues/1384#issuecomment-277405543 We are spending most time in kernelbase!strlenW

I looked into the build trace, and it was clearly a performance issue in the output window side.
Name Inc %	Inc	Exc %	Exc	Fold	When
||+ msenv!CSUIBuilder::OutputStringNoPump 92.8	639,072	0.3	2,356	2,356 __399999599989999999999999999998
|| + msenv!CSUIBuilder::FBufferedOutputString 91.5	630,242	0.1	831	733 __399999599989999999999999999997
|| |+ msenv!CSUIBuilder::WriteBufferedOutputString	91.4	629,382	0.0	35	20 __399999599989999999999999999997
|| ||+ msenv!VsUtil::CString::Append 91.4	629,293	0.0	251	240	__399999599989999999999999999997
|| |||+ msenv!VsUtil::CString::Append 91.3	628,916	0.0	168	128	__399999599989999999999999999997
|| ||||+ kernelbase!lstrlenW 89.3	614,786	89.3	614,786	2,857	__399999599989999999999999999997
Basically, the code breaks down the input string into lines, and append them piece by piece to an internal buffer. Every appending, it will recalculate the length of the string in the buffer, and the length of the new string, although both of them known, and then reallocate the buffer. It is repeated for every line and every call. I think it should change the code to buffer the original strings into a list, and merge them together in one call, so the buffer size can be calculated before hand to prevent all extra allocation/copy and resizing. I will loop the right team. I don't think any change on the CPS side can fix or reduce this problem.

Is the reason we're not seeing this in legacy is because it writes a single chuck whereas we buffer?

I think David Pugh did some work in this area to fix some performance problems, and the change might change how strings are merged, and caused a regression here.  Also CPS buffering messages might make it worse, if the output window is slower than the build, the output messages will start to be queued in the internal buffer, and push to the output window more aggressively in a large batch.

@lifengl Do you have a bug on VSO side for this? Otherwise, I can open one and assign it to you so you can forward it to the correct team.

Bug 374371
Sent from my phone
On Feb 7, 2017, at 10:29 AM, Manish Vasani <notifications<EMAIL_ADDRESS>wrote:
@lifenglhttps://github.com/lifengl Do you have a bug on VSO side for this? Otherwise, I can open one and assign it to you so you can forward it to the correct team.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/roslyn-project-system/issues/1242#issuecomment-278095127, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALGWwtg2Hbtkwe6qe1I_CwXuBDTLIlhTks5raLfKgaJpZM4Lnp4d.

Fix is here: https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/56690.


Open in specific directory

It would be nice if one could open the file browser in a specific directory by giving the directory location as command line parameter.
This would be also helpful for other applications that want to use harbour-file-browser to open a specific directory.
Example: harbour-file-browser ~/Downloads
Expected result: Open File-Browser in the users Downloads directory

You can find a more advanced version of File Browser which accepts folders as command line parameters at https://github.com/ichthyosaurus/harbour-file-browser


final project submission

https://github.com/QMSS-G5072-2020/final_project_patel_pooja/commit/46a7226a2d775dd36099679c4801b3542b58d714
@SUN-Wenjun @lonadai0121

3/5 Satisfactory Project: lack of complexity and breadth in skills; should not include your own API key in your package
3/3 well commented
3/3 The code is efficient, correct, and minimal
3/3 The package structure is clear, easy to follow, and closely matches the structure discussed in class.
2/3 Documentation of the package and its contents is largely absent.
3/3 The package installs without problems.
1/3 Testing for the package is largely absent.


Optimize images in landing page

Optimize images in landing page , reference #176

@abhijeet007rocks8 @DhairyaBahl i would like to work on this