Benchmark crashes in Rust (wasm module)

Added super-simple BenchmarkDotNet project at https://github.com/christophwille/csharp-opa-wasm/blob/master/spikes/BenchmarkOpaWasm/Program.cs
This crashes before taking measurements, details see attached log file. Seems some issue with the underlying wasm/runtime?:
thread '<unnamed>' panicked at 'allocate_for_function: "The paging file is too small for this operation to complete. (os error 1455)"', src\libcore\result.rs:1188:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
ExitCode != 0

BenchmarkOpaWasm.WasmPolicyExecution-20200309-115410.log

@peterhuene Do you know where this might be coming from? (so I can report it at the correct place). Thanks.

This is coming from Wasmtime. allocate_for_function is called when allocating trampolines per function and it was being done very inefficiently until recently.  Does the wasm module have a lot of functions perhaps?  Let me track down to see when (and if) the fixes around this made it in.

The wasm is in this repo at https://github.com/christophwille/csharp-opa-wasm/tree/master/sample-policies
It is super-simple, I have attached the wasm2wat for it here.  policy.wasm2wat.txt

So there were multiple fixes in this area, one of which made it in to 0.12.0 (https://github.com/bytecodealliance/wasmtime/pull/947) and one that is not yet in (https://github.com/bytecodealliance/wasmtime/pull/957).
The former reduced the number of trampolines generated from per-function to per-signature, which should greatly reduce the needed number of trampolines, allowing some trampolines to be shared between functions with the same signature.
The latter is to prevent 64K allocations per trampoline which is really inefficient.  We hope to have that fix in soon.
I'd still only expect this to be a problem with a module with a lot of distinct signatures or in a memory-constrained environment.

The module is super small with not many signatures, so I can't explain why you would be running into this issue.  I'll see if I can reproduce.

You may want to see if adding an IDisposable implementation to OpaPolicy to dispose of the instance and then disposing the policy in the benchmark method helps at all.  I suspect the benchmarking is iterating heavily on that method and thus creating a lot of instances without putting any pressure on the GC because it's all native allocations being done.

I'll try that. I hadn't added IDisposable because OpaPolicy itself isn't holding the external resources (well, and I haven't decided on any sort of object model yet).

I tested it with minimal changes as can be seen in https://github.com/christophwille/csharp-opa-wasm/commit/6fc45a2d708e095029e9959f17b654556e81bd5e. Yes, collecting the instances makes all the difference.

Closing for Wasmtime 0.15


Take a photo and store it on the device

Hello,
I am in great need of some guidance. I am trying to develop a photo mode system for my AR application, meaning I would like to allow the user to take a photo of AR object as well as what the camera sees and store it to his device.
I have tried using render targets but it seems that export render target does not work on android, as I get a corrupted png image.
Any tips on where to look for this kind of functionality?
Many thanks

https://www.unrealengine.com/marketplace/zh-CN/slug/rave-mobile-photo-gallery

https://www.unrealengine.com/marketplace/zh-CN/slug/rave-mobile-photo-gallery

Thanks for that. Didnt know about this plugin... Have you actually used it?

I use the c++ code to achieve.
If you want to do it in C++ code, you can refer to this.
http://www.pianshen.com/article/2544198620/


M1 Build attempt

First attempt to get M1 build on the workflows

Just wanted to confirm, does the ARM Mac releases still require rosetta at the moment? I prefer not to download rosetta and the Mac release is asking me to download it still

We still don't have an automated m1 build. Github offers m1 runners but I don't have access to m1 hardware myself to debug it and set it up.
For now native m1 build is only available if you build it yourself, sorry;/

@XapaJIaMnu I have a personal M3 MacBook now, when I find some time I can give it a go.

Build seems to be working, signing fails though. Definitely an expired key, but also the arm build has some parse errors of which I don't know the source.


N64 and psp emulators not show in the menu

Ppsspp emulator and n64 emulator cant show in the menu.....and cant access to the roms and emulators..... any ideas to fix this?? Thanks!!

What is your device and did you add your games in the proper format to the proper folders so that they should show?
https://github.com/christianhaitian/arkos/wiki/ArkOS-Emulators-and-Ports-information#nintendo-64
https://github.com/christianhaitian/arkos/wiki/ArkOS-Emulators-and-Ports-information#playstation-portable-psp

any update on this?


Advanced search item detail viewers redux

This adds advanced search features developed in the iDC UI Theme and adds support of media viewers for a variety of media types for the item detail page and provides general styling for the item detail page.
To test:
Viewers:

pdfs: add a repository item and an associated document media type with a pdf attached, see that pdf.js viewer displays the pdf on the item detail page.
oral history item: add a repository item and an associated audio media type with an mp3 audio file attached and an associated document media type with a pdf attached, see that both the audio player and the pdf.js viewer are rendered on the item detail page.
paged content/complex media: add a paged content repository item and add at least one page repository item that is a member of the paged content item you just created and add an associated image media type to the page item. See that mirador.js viewer will display the pages on the paged content item and the individual page items will display the image without a viewer on their detail pages.
video: add a moving image repository item and an associated video media type with a mp4 video attached. See that the video is rendered on the item detail page with controls.
openseadragon item: add an image repository item and use the openseadragon display hint and add an associated image media type. See that open seadragon viewer renders in the item detail page.

Advanced Search:

With some content added, visit the /advanced-search route by directly manipulating the url in the browser or by clicking the "Advanced Search" text under the search input located in the page header.
Note that the advanced search controls will display above the list of results. Perform searches utilizing these controls and the newly added filters available in the left sidebar of the page.
It should be possible to perform both word proximity searches and general advanced search queries by adding search parameters as needed.

When adding a document as a repo item, it seems that selecting the PDFjs display hint at the bottom will result in 2 PDF viewers on screen, one of which fails to load content

When adding a document as a repo item, it seems that selecting the PDFjs display hint at the bottom will result in 2 PDF viewers on screen, one of which fails to load content because of CORS error

Yeah this is a known issue. I think we need to disable PDF.js display hint, but I'd like to do it in a follow up PR.

The video player doesn't seem to restrict the video size much. I uploaded a vertical video from my phone and it looked like it was restricted in width, but not height resulting in a video player that was maybe twice my screen height. Definitely a low priority thing

I left some lower priority comments, otherwise, looks good

Will address these comments in follow up PRs.


Fix bit shifting

Need to bitshift these after they're int eh UIn64, otherwise they fall out of the UInt8 before the convsersion.

This is a duplicate of #327


BLE Library and HTTP Library Interference

Hi, i've a proyect with both library. i've a code that connects to bluetooth and connects to Wi-Fi. Both work ,but not the way that i want. The codes for each library are in "setup". First i put the Bluetooth's code then the HTTPS, the problem of this is that the Bluetooth works ,but the HTTPS doesn't (it can't download a file from a website). And if i put the HTTPS over the BLE's code. HTTPS works, but BLE doesn't (I try to send a massage to ESP32 ,but it doesn't receive).
What could be the problem? ,and what can i do?

memory shortage?

I don't think so, i was investigating in these hours. And i found a comment that "the Bluetooth and Wifi use the same antenna or something like that" ,and that it's difficult to use it at the same time. @chegewara Do you have any idea?

Yes, my idea is memory shortage, because i never had any issue to use wifi and ble at the same time, and i did many such projects, including the one i am working with now. I just had to switch to NimBLE, because of memory shortage this time, and only this time, but this is huge project, including LTE.

NimBLE? , My proyect is huge too.

https://github.com/h2zero/NimBLE-Arduino

@chegewara Thanks, I'll try change the libraries.

@chegewara I have problems to adapt it. i use
#include <BLEDevice.h>
#include <BLEUtils.h>
#include <BLEServer.h>
#include <BLE2902.h>


Fixed character size for unicode when specifying :size for doom-unicode-font

[X] I have searched the FAQ and the documentation for solutions.
[X] I have searched the issue tracker for similar issues (the closed issues too).
[X] I have searched the Discourse for any errors and solutions.
[X] I can reproduce my issue on the latest commit of Doom Emacs.
[X] I have read How to Debug Issues and found no solution.

What did you expect to happen?
I expected to be able to use Cyrillic symbols the same way I use the latin ones.
What actually happened?
In org mode, when I create new text headers (outline1,2,3 etc.) and use Cyrillic symbols in it, the symbols height is somehow ~50% of what it should be, please see screenshot:

Corresponding section from my config.el:
(setq doom-font (font-spec :family "Iosevka Term" :size 18 :weight 'medium)
      doom-variable-pitch-font (font-spec :family "Iosevka Term") ; inherits `doom-font''s :size
      doom-unicode-font (font-spec :family "Iosevka Term" :size 18)
      doom-big-font (font-spec :family "Iosevka Term" :size 21))

Describe your attempts to resolve the issue
I've tried to use :ui +unicode package, and it falls back to a different font, where this issue is not happening.
However, if I understand correctly, Iosevka has a full support of Cyrillic, and this issue happens only with the headers in org mode.
Steps to reproduce

Create a new org mode document
Set font to Iosevka Term
Copy this text

* Header заголовок

* Заголовок

System Information
https://pastebin.com/QuNti0tq

Does removing the :size from your doom-unicode-font fix it? (You shouldn't be setting one anyway)

It does fix it! Thanks a lot 🙏

Same problem and the size removing doesn't help:

(setq doom-font (font-spec :family "JetBrains Mono" :size 13 :weight 'normal)
      doom-variable-pitch-font (font-spec :family "JetBrains Mono")
      doom-unicode-font (font-spec :family "JetBrains Mono")
      doom-big-font (font-spec :family "JetBrains Mono" :size 15))

Differences
OS and font: Windows 10 and JetBrains Mono
System Information
https://pastebin.com/H0udrn3a

@hlissner  should I create an another issue or may reopen this?

Try call M-x describe-char at cyrillic symbols. Don't know why but see a Consolas font:
    harfbuzz:-outline-Consolas-bold-normal-normal-mono-13-*-*-*-c-*-iso10646-1 (#x1EA)

I think the issue is in a unicode package. It doesn't apply doom-unicode font for all possible unicode blocks so have to add it in your config.el for needed blocks
(after! unicode-fonts
  (dolist (unicode-block '("Cyrillic"
                           "Cyrillic Supplement"))
      (push "JetBrains Mono" (cadr (assoc unicode-block unicode-fonts-block-font-mapping)))))


Docs: how to mixin from local file?

Hi, thanks so much for this library! Has really helped cdk-nextjs offer a flexible/customizable L3 construct to deploy Next.js on AWS.
Given the example from the docs:
new ProjenStruct(project, { name: 'MyProjectOptions' })
  .mixin(Struct.fromFqn('projen.typescript.TypeScriptProjectOptions'))
  .omit('sampleCode', 'projenrcTs', 'projenrcTsOptions');

How could I adjust this example to create a new interface from a local file like:
new ProjenStruct(project, { name: 'MyProjectOptions' })
  .mixin(Struct.fromLocal('./my/file/path/TypeScriptProjectOptions.ts'))
  .omit('sampleCode', 'projenrcTs', 'projenrcTsOptions');

I'm not sure that's possible. jsii-struct-builder is using a jsii assembly for reflection on existing interfaces. What you are suggesting is pointing to a raw TS file that could contain anything! Even if we assume that it's just an interface, we would now need to reflect on it and that's kind-of difficult.
Taking one step back, may I ask what you are trying to achieve with it?

@mrgrain, ah I see. I didn't know this package relied on jsii assembly for reflection of existing interfaces. In cdk-nextjs project linked above, I'm exposing an overrides property on the props of the main construct: Nextjs. I'm using this package to create optional interfaces for CDK Prop types used like FunctionProps, DistributionProps etc so we don't have to use any. I'd also like to offer overrides for constructs created locally in this package like NextjsBucketDeployment which is is used in NextjsServer and NextjsStaticAssets. I didn't think I could use Struct.fromFqn to reference that construct so that's why I created this issue.

Hm, I see. So something you actually can do, is reference a FQN from the local assembly. E.g.: Struct.fromFqn('cdk-nextjs. NextjsBucketDeployment'). Only thing to consider is that there's a bootstrapping issue because you need the local .jsii to be already available. 🤔

Oh I see. I thought Struct.fromFqn looked in node_modules so that wouldn't work. Yes, I would have a circular dependency issue :/

Whoops, didn't mean to close.

I thought Struct.fromFqn looked in node_modules so that wouldn't work

Haha yeah. I only implemented that recently: https://github.com/mrgrain/jsii-struct-builder/pull/159

Haha, well I appreciate you implementing that! Any ideas on how to get around bootstrap issue?

Any ideas on how to get around bootstrap issue?

You will have to run projen compile first, which unlike projen build doesn't do a projen default at the start.

Otherwise you could define all your interfaces (even the manual ones) with jsii-struct-builder. That is possibly a bit annoying, but would give you a single source of thruth.

@mrgrain, I think I'll try to modify the GitHub action to run projen compile first instead of just running projen build. Is there  a simple or documented way to do that?
Here is an example of our GitHub Action (failing because I need to run projen compile first): https://github.com/jetbridge/cdk-nextjs/actions/runs/7171276028/job/19525876169?pr=181

Not this specifically, but I reckon you could patch the workflow files directly using the low level API:
https://projen.io/escape-hatches.html#object-file-patches

Worked perfectly. Thanks so much, @mrgrain!


Fix wrong import

Fixes https://github.com/enram/vptstools/issues/57

@peterdesmet I'm merging this asap as this import issue could potentially break imports when user install the current version/master


Event dates not recognized

Just imported GC36P0T, no event date shown.

I was going to ask @Bananeweizen if it was ok to do it. If you want, I can take care of it (it's not a merge, technically it will be a cherry-pick followed by a reverse merge from release to master to remember that we have this patch on both branches).

It looks like good fix and should not do anything bad.

I would be glad if Samuel could do that as he seems to be much more a master of the git magics. :)

Done.


tree-wide: use a standard project layout

Keep main.go files under cmd/ and internal libraries under internal/.
This will allow us to add more exutables under cmd/ (whereas only one
was possible when main.go was kept in the root).
Signed-off-by: Tom Gundersen<EMAIL_ADDRESS>

lgtm


Release Pipeline - update to publish to Red Hat Scan Registry and Submit Image Cert Results

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
To support publishing to Red Hat Marketplace, images need to be pushed to the red hat scan registry on a regular release cadence, followed by executing the preflight utility and submitting the results.
An example workflow is provided to show how this can be executed via a gh workflow.
Proposed Solution
jobs:
  build-push-image:
    runs-on: ubuntu-latest
    outputs:
      redhat_scan_tag: ${{ steps.redhat_scan_tag.outputs.tag }}
    steps:
    - name: "Redhat Meta: Add scan tag to list of redhat tags"
      id: rh-scan-meta
      run : echo "scan.connect.redhat.com/ospid-f8304390-ff4c-4b7f-8c7e-1ed13370b3c2/kong-ingress-controller:version"

    - name: docker login to RH Scan Registry
      uses: docker/login-action@v2
      with:
        registry: scan.connect.redhat.com
        username: ${{ secrets.RH_USERNAME }}
        password: ${{ secrets.RH_TOKEN }}
    - name: Redhat Image Build and Push
      run: echo "build stage"
    - name: setup output
      id: redhat_scan_tag
      run: |
        echo "::set-output name=tag::${{ steps.rh-scan-meta.outputs.tags }}"

  rh-scan-registry-submit:
    runs-on: ubuntu-latest
    needs: build-push-image
    steps:
      - name: check preflight can see docker-config
        uses: addnab/docker-run-action@v3
        with:
          image: quay.io/opdev/preflight:stable
          run: |
            echo ${{ secrets.RH_AUTH_CONFIG }} > ./dockerconfig.json
            preflight check container  ${{ needs.build-push-image.redhat_scan_tag }}  --submit \
            --pyxis-api-token=${{ secrets.RH_PYXIS_TOKEN }} \
            --certification-project-id=${{ secrets.RH_PROJECT_ID }} \
             --docker-config=./dockerconfig.json

Additional information
The partner engineering team can help retrieve any secrets needed from the red hat portal
Acceptance Criteria

[ ] no breaking changes to the current release pipeline
[ ] the release pipeline should push the new image with appropriate tags to Red Hat scan registry
[ ] the release pipeline should run the preflight utility and submit results to the Red Hat scan registry

Commented in the internal partner engineering team issue about the place where we'll get the credentials to the RH Connect Docker registry.

Blocked by credentials for a technical push account being available.


CLI fails in windows for users with spaces in their username

This issue is reporting a bug in the code of Perseus. Details of the scope will be available in issue labels.
The author described their issue as follows:

Perseus build functions fail if you have a space in your username on windows.
Known to affect perseus build/export/serve
Since perseus defaults to using the cache directory in windows, the call to "powershell -command C:\Users\First Last..." doesn't work due to the space.

The steps to reproduce this issue are as follows:

Create a windows user with a space in the username.

A minimum reproducible example is available at <>.

Hydration-related: false
The author is willing to attempt a fix: true

Tribble internal data
dHJpYmJsZS1yZXBvcnRlZCxDLWJ1ZyxBLWNsaSxhdXRob3Itd2lsbGluZy10by1pbXBs

Well that's a problem. What's the exact error message you're getting?

 [1/2] 📦 Exporting your app's pages...
 [2/2] 🏗  Building your app to Wasm...❌
C:\Users\Isaac : The term 'C:\Users\Isaac' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path
was included, verify that the path is correct and try again.
At line:1 char:1
+ C:\Users\Isaac Tay\AppData\Local\perseus\perseus_cli\cache\tools\wasm ...
+ ~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\Users\Isaac:String) [], CommandNotFoundException
  [1/2] 📦 Exporting your app's pages...✅
  [2/2] 🏗  Building your app to Wasm...❌

It should be fixable by changing the tool path for windows from "C:\Users..." to "& 'C:\Users...' ", though I'm not sure if that's the best fix for this

 ⠄ [1/2] 📦 Exporting your app's pages...
  [2/2] 🏗  Building your app to Wasm...❌
C:\Users\Isaac : The term 'C:\Users\Isaac' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path
was included, verify that the path is correct and try again.
At line:1 char:1
+ C:\Users\Isaac Tay\AppData\Local\perseus\perseus_cli\cache\tools\wasm ...
+ ~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\Users\Isaac:String) [], CommandNotFoundException
  [1/2] 📦 Exporting your app's pages...✅
  [2/2] 🏗  Building your app to Wasm...❌

It should be fixable by modifying the toolpath for window from "C:\Users..." to "& 'C:\Users...' ".
Sorry for closing the issue by accident


Jesse: Watch canadarxfast.com

Jesse requests the watch of the watch_keyword canadarxfast\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

canadarxfast\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


[Test](build index) enhance build index with clone test case with retry logic #42348

cherry pick from #42348

run buildall


Add logs and telemetry for Entry Widget

What was solved?
This PR adds logs to EntryWidget, and aligning iOS with Android logs. Because logger was added as a dependency to QueueMonitor, it was necessary to move queueMonitor from Glia environment to Glia's class property due to Logger being initialized during GliaSDK initialization. Because of that some changes can be seen in tests as well.
MOB-3481
Release notes:

[ ] Feature
[X] Ignore
[ ] Release notes (Is it clear from the description here?)
[ ] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Additional info:

[ ] Is the feature sufficiently tested? All tests fixed? Necessary unit, acceptance, snapshots added? Check that at least new public classes & methods are covered with unit tests
[ ] Did you add logging beneficial for troubleshooting of customer issues?
[ ] Did you add new logging? We would like the logging between platforms to be similar. Refer to Logging from iOS SDKs → Things to consider for newly added logs in Confluence for more information.

Screenshots:

fixed b1c382a

!squash


Adding more changes

Pull Request Motivation

Kind

Release Note

NONE

Just merge this into your branch. Run go run cmd/cmptest/main.go to run the main program!


Fix for #65, got lucky that this was a one-line change. Whew!

Is this fix always correct in all possible-scenarios, i.e. in all OSGI and non-OSGI scenarios? If not, may be it should be an opt-in solution which can be enabled via an option in a configuration file?

I thought about that too. In non-OSGi environments, there should be only one classloader, so the set will be superfluous.
Without having a copy of the BundleContext, I don't know of a way to tell if one is running under OSGi or not. For instance, the existence of the headers in the MANIFEST.MF only means a JAR can be run as a bundle, not that it is running as a bundle.
If we went the configuration file route, what if it was opt-out and then remove the setting if nobody used it a few revisions from now? This would reduce long term complexity. Just a thought.

I see that KryoSerialization is using system.dynamicAccess.getClassFor, which is going to use the system.dynamicAccess.classLoader. So it seems like a misfeature that the classloader Kryo itself is using would be different than the one system.dynamicAccess.getClassFor is using, no?

Any reason we can't get this pushed? With #67 fixed, it would be nice to use mainline sources at the next release! :)

Thanks so much!


ExportSVG() is broken

ezdxf: 0.16.5
python: 3.11
build123d: dev branch
from build123d import *

svg = ExportSVG()

results in:
Traceback (most recent call last):
  File "/home/gordon/mysrc/cadquery/export.py", line 3, in <module>
    svg = ExportSVG()
          ^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/build123d/exporters.py", line 944, in __init__
    self.add_layer(
  File "/usr/lib/python3.11/site-packages/build123d/exporters.py", line 991, in add_layer
    layer = ExportSVG._Layer(
            ^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/build123d/exporters.py", line 907, in __init__
    self.fill_color = convert_color(fill_color)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/build123d/exporters.py", line 902, in convert_color
    if isinstance(c, RGB):
       ^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/typing.py", line 1328, in __instancecheck__
    return self.__subclasscheck__(type(obj))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/typing.py", line 1331, in __subclasscheck__
    raise TypeError("Subscripted generics cannot be used with"
TypeError: Subscripted generics cannot be used with class and instance checks

Try upgrading ezdxf - I think that will fix the problem.

Yep, it works with 1.3.0. Did not check dependencies from pyproject.toml carefully, sorry.

The better solution is to add a minimum to the pyproject.toml for ezdxf but I haven't done that yet. I think this is the first dependency that's had this type of problem.


Add links from WinRT Display.Core APIs to their COM interop interfaces

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

@stevewhims - Could you review this pull request?
Note that if the updates are accepted, the locale (en-us) needs to be removed from the new URLs. (The PR review team can do this before merging.)

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


Setting MoCap for dm_control simulations.

Hello, how can I set the position and orientation of a MoCap object in dm_control? The MjData object has mocap_pos and mocap_quat attributes, but unlike gym no set_mocap_pos and set_mocap_quat functions.

You can do this simply by writing into the mocap_pos and mocap_quat arrays in physics.data, e.g.
from dm_control import mujoco

# Trivial model containing just a single mocap body with a cube geom
physics = mujoco.Physics.from_xml_string("""
<mujoco>
  <worldbody>
    <body name='my_mocap_body' pos='1 0 0.5' mocap='true'>
      <geom name='mocap_cube' type='box' size='0.05 0.05 0.05' rgba='1 0 0 0.5'/>
    </body>
  </worldbody>
</mujoco>
""")

print(physics.named.data.geom_xpos)
# FieldIndexer(geom_xpos):
#                x         y         z
# 0 mocap_cube [ 1         0         0.5     ]

# Translate 'my_mocap_body' +1 meter in the z direction
physics.named.data.mocap_pos['my_mocap_body', 'z'] += 1.0

# Recompute the forward dynamics to update the geom position (we could also call
# `.step()` if we also wanted to advance the simulation)
physics.forward()

# The geom is now in the new position
print(physics.named.data.geom_xpos)
# FieldIndexer(geom_xpos):
#                x         y         z
# 0 mocap_cube [ 1         0         1.5     ]

The tutorial colab has loads more examples of how to read and update the simulation state.


feat: delete duplicate image

I deleted the duplicate skill image (PowerBI). Please check!

Thank you for deleting it!


fix: [SPT-1889] updated modal scrollable shades to fix resize inheritance

SPT-1889
How to test

Go to [/?path=/story/overlays-modal--modal](https://picasso.toptal.net/SPT-1889-fix-modal-scrollable-shades-inheritance-on-resize/?path=/story/overlays-modal--modal and open one of the modals
Resize the window height to make the content scrollable
Scroll so the shade effect will be in place
Resize back to non-scrollable content, the shade effect should dissapear
Compare to current

Screenshots
https://user-images.githubusercontent.com/79158829/132189780-86ad126e-d00d-4311-94ed-a7e8a9c5c093.mov
https://user-images.githubusercontent.com/79158829/132189784-2d12f0f7-34c0-4df9-ab53-a6dcf384614e.mov
Review

[ ] Read CONTRIBUTING.md and Component API principles
[ ] Annotate all props in component with documentation
[ ] Create examples for component
[ ] Ensure that deployed demo has expected results and good examples
[ ] Ensure that tests pass by running yarn test
[ ] Ensure that visuals tests pass by running yarn test:visual. If not - check the documentation how to fix visual tests
[ ] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.

PR commands

List of available commands:

@toptal-bot run all - Run whole pipeline
@toptal-bot run build - Check build
@toptal-bot run visual - Run visual tests
@toptal-bot run deploy:documentation - Deploy documentation
@toptal-bot run package:alpha-release - Release alpha version

I have it manually checked after adding window.addEventListener('resize', updateShades)
Works better in contrast to the original version. Top shadow disappears.

Great if we have some throttling for reducing an amount of calls.

https://user-images.githubusercontent.com/79158829/132337429-5fb925fc-c494-4b62-b3b6-e90ee7861b4f.mov
https://user-images.githubusercontent.com/79158829/132337073-472564a0-219e-4fc6-bb82-ab97619b50f3.mov
This also fixes shades not appearing when resizing to scrollable content

@TomasSlama 47d921d

@toptal-anvil ping reviewers


Add dnsmasq with container and use the container IP as DNS server.

So we can use podman ip option to run a container with specific ip, so we are now running a dnsmasq container on <IP_ADDRESS> and binding it to host 53/udp using as privileged container. As part of our VM setting we using the crc.testing as search option and also container IP as 1st nameserver. Now all the crc.testing query resolved by that container and also openshift dns container not complain about not having an upstream server.
Moving forward we need to convert this as systemd service.

Will check/look at this tomorrow morning. Now a little bit too late ;-)
On Sun, Jun 2, 2019 at 9:21 PM Praveen Kumar<EMAIL_ADDRESS>wrote:

So we can use podman ip option to run a container with specific ip, so we
are now running a dnsmasq container on <IP_ADDRESS> and binding it to host
53/udp using as privileged container. As part of our VM setting we using
the crc.testing as search option and also container IP as 1st nameserver.
Now all the crc.testing query resolved by that container and also
openshift dns container not complain about not having an upstream server.

Reply to this email directly, view it on GitHub
https://github.com/code-ready/crc/pull/176?email_source=notifications&email_token=AAAAOZQ54N4MK3LFTFZIMY3PYPCOLA5CNFSM4HSCUL7KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GXEUKZA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAAOZWFQR3PLY7Q5NHKPZLPYPCOLANCNFSM4HSCUL7A
.

--
Gerard Braad | http://gbraad.nl
[ Doing Open Source Matters ]

I would prefer we also set the correct nameservers for the local /etc/resolv.conf. We can prepare this with a setup to change the permissions...
and use network.ResolvFileValues to create the file. the setup is pretty much like https://github.com/code-ready/crc/blob/master/pkg/crc/services/dns/dns.go#L61-L72
Something like:
	orgResolvValues := network.GetResolvValuesFromHost()
	// override resolv.conf file
	nameserver := network.NameServer{IPAddress: serviceConfig.HostIP}
	nameservers := []network.NameServer{nameserver}
	nameservers = append(nameservers, orgResolvValues.NameServers...)

	resolvFileValues := network.ResolvFileValues{
		SearchDomains: orgResolvValues.SearchDomains,
		NameServers:   nameservers}

	network.CreateResolvFileOnHost(resolvFileValues)


Update tests to run more robustly locally

To stay aligned with the other WebdriverIO projects we will move to Grunt instead of using a shell script. Thanks for putting up the PR though.


Children entities with PK can not be mapped to list in aggregate

Hi!
Not sure if this is an actual bug or expected behavior.
I have two tables, Parent and Child, where a parent can have many children, like so:
CREATE TABLE parent (
  p_id UUID PRIMARY KEY DEFAULT GEN_RANDOM_UUID(),
  name VARCHAR
);

CREATE TABLE child (
  c_id UUID PRIMARY KEY DEFAULT GEN_RANDOM_UUID(),
  name VARCHAR,
  p_id UUID NOT NULL REFERENCES parent(p_id)
);

Then I define these tables in Kotlin as:
@Table("parent")
data class Parent(
    @Id
    val pId: UUID?,
    val name: String,

    @MappedCollection(idColumn = "p_id", keyColumn = "c_id")
    val children: List<Child>
)

@Table("child")
data class Child(
    @Id
    val cId: UUID?,
    val name: String
)

Next I try to create a new entity through the default repository
@Repository
interface ParentRepository : CrudRepository<Parent, UUID> {
}

val parent = Parent(null, "Parent", listOf(Child(null, "Child")))
repo.save(parent)

When I do this, I get the following error:
org.postgresql.util.PSQLException: ERROR: column "c_id" is of type uuid but expression is of type integer
  Hint: You will need to rewrite or cast the expression.

By debugging the insert code I can see that Spring tries to insert an integer value of 0 as the primary key for the child table.
If I change my parent class to use a Set instead of a List everything works as expected and UUIDs get generated for both the Child and the Parent.
@Table("parent")
data class Parent(
    @Id
    val pId: UUID?,
    val name: String,

    @MappedCollection(idColumn = "p_id", keyColumn = "c_id")
    val children: Set<Child>
)

Thank you

Please provide a Minimimal Reproducable Example, preferable as a Github repository. Make sure to include the database, either as an in memory database or if that is not possible using Testcontainers.

Hi, I have created a reproducable example here https://github.com/olofholmlund/spring-data-bug
It uses testcontainers so docker must be running. Run the test class DemoApplicationTests to see the working example using a Set and the broken example using a List

The problem is that you specify to store the key column of the list, i.e. it's index in c_id by specifying
@MappedCollection(idColumn = "p_id", keyColumn = "c_id")
val children: Set<Child>

But the same column is used for the id of the Child entity:
@Id
val cId: UUID?,

With a Set this works because it doesn't have a "key".
To fix this change keyColumn to "index" for example and add an appropriate column to the child table:
CREATE TABLE IF NOT EXISTS child
(
    c_id UUID PRIMARY KEY DEFAULT gen_random_uuid(),
    name VARCHAR,
    p_id UUID NOT NULL REFERENCES parent (p_id),
    index INTEGER
);

I'll send a PR to that effect to your reproducer


Notebook help shortcuts not present

The (maybe ipythons?) shortcut of shift+tab that show the docstring of the function under cursor in a floating window (and expanding the window with with double tap of this same combination) is not present.
When someone tackles this, I think it's also important to to make the text selectable so that it can be copy-pasted into the notebook.

Working in master, although there is a discussion here #714 about how it all should work:
Closing as resolved and duplicate.


Failing test cases

Hi,
When I run "python2.7 setup.py test", a number of test cases report failures. This is from git revision dfbcecad9ad5ccef267fb96ac3d1a84f7c530645 running on Debian 7.
I would attach the full output as a text file, but github won't let me; it says "Attaching documents requires write permission to this repository.  Try again with a PNG, GIF, or JPG.". Here are just a couple of snippets from the output instead:
test_oct (tests.cq_test.TestNSGT) ... /home/gson/sw/nsgt/nsgt/nsgfwin_sl.py:64: UserWarning: Q-factor too high for frequencies 52.15,53.82,55.54,57.32,59.15,61.05,63.00,65.02,67.10,69.25,71.46,73.75,76.11,78.55,81.06,83.66,86.33,89.10,91.95,94.89,97.93,101.06,104.30,107.64,111.08,114.64,118.31,122.09,126.00,130.03,134.20,138.49,142.92,147.50,152.22,157.09,162.12,167.31,172.67,178.19,183.90,189.78,195.86,202.13,208.60,215.27,222.16,229.27,236.61,244.19,252.00,260.07,268.39,276.98,285.85,295.00,304.44,314.19,324.24,334.62,345.33,356.39,367.79,379.57,391.71,404.25,417.19,430.55,444.33,458.55,473.23
warn("Q-factor too high for frequencies %s"%",".join("%.2f"%fi for fi in f[q >= qneeded]))

and
======================================================================
FAIL: gtest_oct (tests.slicq_test.TestNSGT_slices)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/gson/sw/nsgt/tests/slicq_test.py", line 53, in gtest_oct
    self.runit(siglen, fmin, fmax, obins, sllen, trlen, real)
  File "/home/gson/sw/nsgt/tests/slicq_test.py", line 18, in runit
    self.assertTrue(close)
AssertionError: False is not true

Hi, thanks for the report.
This is a error just in the unit tests - should be fixed with commit d9d94adfa3ca9d2d26c1e111fbd7a69243374a1e.


Handle redirects

Gmail always opens URLs using a redirect service such as https://www.google.com/url?q=http://my.real.url
It would be great if choose-browser could resolve the real url.

Btw, long-clicking the link in gmail and selecting "open browser" puts the real url into the intent.
Anyway. thanks for the fix.


will pts provide an api to render in existing canvas provided by the user?

user html (user will decide where to put it)(some times there's some state issue if it is put elsewhere)

<canvas id="user-canvas">
</canvas>

pts
 var space = Pts.SpaceContextOnExistingCanvas(..getElementById(...)..)

Thanks that's exactly what i need.


IKEA Trådfri: configuration variable allow_tradfri_groups: false ignored

Home Assistant release with the issue: 0.70.1

Last working Home Assistant release (if known): n/A
Operating environment (Hass.io/Docker/Windows/etc.): Docker @ Synology

Component/platform: https://www.home-assistant.io/components/tradfri/

Description of problem:
The configuration setting allow_tradfri_groups: false is ignored. Trådfri groups are auto discovered regardless of this setting.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
tradfri:
  allow_tradfri_groups: false

Traceback (if applicable):

Additional information:
Manually deleting the Trådfri groups from entity_registry.yaml doesn't help either. The groups are re-discovered on next HA startup.

I've noticed the exact same thing.
Just tested it on a clean installation, groups are imported, while the parameter has been set.

allow_tradfri_groups: false is ignored in my hass setup too...

same here.

I think it would be good if the integration would support the import of groups from trådfri. I don't want to configure groups within HA if I already have them in the "native" system.


Latency between clicking button in Android and Crownstone responding

The latency between clicking a button in iOS is really good. On Android it takes up to six seconds before the Crownstone reacts. The latency needs to be on the same level of iOS for a good user experience.

Android does have its issues with connecting: each phone model behaves very different. Nonetheless, we're trying to improve it now, with some success!


wrap existing C driver in lua?

Would it be simpler to wrap in lua the existing cassandra driver written in C? (C++ actually, but it's interface is plain C.) For reference: https://github.com/datastax/cpp-driver
I am aware that this would complicate the portability but it would also profit from the large datastax community.

We need to leverage the openresty cosocket API and the portability is indeed another issue.

Oh, and the rewrite allows for cluster awareness, which is available through ngx.shared.DICT, hence even if we wrap parts of the C driver, we need custom Lua code on top. At the end this is not a big deal.


Unable to bring up catalog questions when added per environment

Unable to bring up catalog questions when catalog added per environment.

This appears to be the UI not sending the project ID header for that request...

Version - master 5/17
Verified fixed


Takeoff - allow publishing NAN longitude and latitude

A NAN is interpreted in the FlightTaskAuto as a non-valid global coordinate and sets a local position.
If a zero is sent instead, the global coordinate is recognized as valid and will be executed.
This is a problem when the global position is gained for the first time after takeoff and that a valid global reference did not exist before
fixes #12919
To reproduce:

Trigger an automatic takeoff without having a global position valid but optical flow only (ekf2 stop, gps stop, ekf2 start)
Enable gps in air (gps start ...)
The drone does not fly towards longitude/latitude 0 anymore

@PX4/testflights Could you test that PR please?
What's important here is to test autonomous takeoffs:

start mission from ground
takeoff triggered from ground station.

Tested on CUAC+ V5:
Modes Tested
Mission Plan Mode (Automated): Good.
RTL: Good.
- Procedure

Armed form QGC.
Took-off on mission plan mode
. Note that commands were sent form QGC (Armed and takeoff)
Once all waypoint completed vehicle switched to RTL and landed as expected.
Good flight in general.
Notes:
No issues noted, good flight in general.

Log:
https://review.px4.io/plot_app?log=de1e7c8b-b555-4706-b168-2d9a6d6bf4db
Tested on PixRacer V4:
Modes Tested
Mission Plan Mode (Automated): Good.
RTL: Good.
- Procedure

Armed form QGC.
Took-off on mission plan mode
. Note that commands were sent form QGC (Armed and takeoff)
Once all waypoint completed vehicle switched to RTL and landed as expected.
Good flight in general.
Notes:
No issues noted, good flight in general.

Log:
https://review.px4.io/plot_app?log=0228f5f2-5470-4d2f-9a60-cfa1091b1366

Tested on pixhawk4 f-450
Modes Tested
Mission Plan Mode (Automated): Good.
RTL: Good.

Procedure

Armed form QGC.
Took-off on mission plan mode
. Note that commands were sent form QGC (Armed and takeoff)
Once all waypoint completed vehicle switched to RTL and landed as expected.
Good flight in general.
Notes:
No issues noted, good flight in general.
Log:
https://review.px4.io/plot_app?log=0ca6226f-71c0-4774-b79b-00bc7986c79f

@jorge789 @dannyfpv Thanks guys, that's all good.


Webviz fails to initialize when plots are being displayed

I'm currently having a issue when intializing webviz with the layout containing plots. This seems to only happen to Windows as i tried it on Ubuntu and it works fine.

Are there any updates regarding this?

Hi @zhihao95, I'm not able to reproduce the problem. Could you post the version of chrome you are using? Also, are there any JS errors in the console?
Here's a generic page on the error, but I don't think it's much help: https://support.google.com/chrome/answer/95669

I'm using Chrome version 88.0.4324.150 (Official Build) (64-bit). Also, we are using remote bag and remote layout to load the bags and layouts.
I couldn't get the JS error to show using chrome (it just crashes the page and shows either STATUS_ACCESS_VIOLATION or STATUS_NOT_IMPLEMENTED)
However, this shows if I use firefox instead


Update quickstart.md

Why:
Closes [issue link]

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Madison chambers or Chamberlin POW MIA Afghanistan IRAC,  impersonated by Pamela mceandry, calling themselves Steve and Mary Female identity change to Paris Pichai or Pichilli Apaguasa. Secured identity with "memory" software accounts of DreamWorks high jacked used against her by known criminals and theives.Stalking of female documented with federal restraining order permanent. Paris<PHONE_NUMBER> Sent from my  U.S.Cellular© Smartphone-------- Original message --------From: mc4928184 @.> Date: 2/28/22  15:53  (GMT-05:00) To: @. Subject: FW: Urgent HelpSent from my  U.S.Cellular© Smartphone-------- Original message --------From: mc4928184 @.> Date: 2/28/22  3:51 PM  (GMT-05:00) To: @. Subject: FW: Urgent Data pushesMasked ipPhone info changesAll obsevedSent from my  U.S.Cellular© Smartphone-------- Original message --------From: mc4928184 @.> Date: 2/24/22  12:07 AM  (GMT-05:00) To: @. Subject: Urgent Crime Sent from my  U.S.Cellular© SmartphoneSent from my  U.S.Cellular© Smartphone
-------- Original message --------From: Inukurthi Dileep @.> Date: 3/4/22  03:28  (GMT-05:00) To: github/docs @.> Cc: Subscribed @.***> Subject: [github/docs] Update quickstart.md (PR #16047)
Why:
Closes [issue link]
What's being changed:
Check off the following:
I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
For content changes, I have completed the self-review checklist.
Writer impact (This section is for GitHub staff members only):
This pull request impacts the contribution experience
I have added the 'writer impact' label
I have added a description and/or a video demo of the changes below (e.g. a "before and after video")
You can view, comment on, or merge this pull request online at:
  https://github.com/github/docs/pull/16047
Commit Summary
1585510  Update quickstart.md
File Changes  (1 file)
M
content/codespaces/getting-started/quickstart.md
(4)

Patch Links:
https://github.com/github/docs/pull/16047.patch
https://github.com/github/docs/pull/16047.diff
—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

@Dileepinukurthi
Thanks so much for opening a PR! I'll get this triaged for review :zap:

Hi @Dileepinukurthi - thanks for offering to improve our docs ✨
We can't accept these updates because we follow strict content models for our content and also we believe deep dive is the correct US spelling.
Thanks for your interest 🙂

By n sonic


Fix disabled menu items in project feature

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
Fixed #4026.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage: 68.559%. Remained the same when pulling ffff8aeb910010f31ff09bd3b07e2ddcb6045454 on kazuhitoyokoi:master-fixmenu4project into ae92ea94764a1a6ddd9ed019fc996fdfd10be329 on node-red:master.

Thank you!


DINO with more than 6 layers

I am training DINO on my own dataset of enc/dec layers of 4/4, 5/5, 6/6, 4/6 and all the result seems reasonable.
But when it comes to 7/7 or 8/8, the evaluation results go wrong. And I tried to set the encoder config num_cp=7, but it still doesn't work.
the model config is below:
model = dict(     as_two_stage=True,     backbone=dict(         depth=50,         frozen_stages=1,         init_cfg=dict(checkpoint='torchvision://resnet50', type='Pretrained'),         norm_cfg=dict(requires_grad=False, type='BN'),         norm_eval=True,         num_stages=4,         out_indices=(             0,             1,             2,             3,         ),         style='pytorch',         type='ResNet',         with_cp=True),     bbox_head=dict(         loss_bbox=dict(loss_weight=5.0, type='L1Loss'),         loss_cls=dict(             alpha=0.25,             gamma=2.0,             loss_weight=1.0,             type='FocalLoss',             use_sigmoid=True),         loss_iou=dict(loss_weight=2.0, type='GIoULoss'),         num_classes=1,         sync_cls_avg_factor=True,         type='DINOHead'),     data_preprocessor=dict(         bgr_to_rgb=True,         mean=[             123.675,             116.28,             103.53,         ],         pad_size_divisor=1,         std=[             58.395,             57.12,             57.375,         ],         type='DetDataPreprocessor'),     decoder=dict(         layer_cfg=dict(             cross_attn_cfg=dict(dropout=0.0, embed_dims=256, num_levels=5),             ffn_cfg=dict(                 embed_dims=256, feedforward_channels=2048, ffn_drop=0.0),             self_attn_cfg=dict(dropout=0.0, embed_dims=256, num_heads=8)),         num_layers=7,         post_norm_cfg=None,         return_intermediate=True),     dn_cfg=dict(         box_noise_scale=1.0,         group_cfg=dict(dynamic=True, num_dn_queries=150, num_groups=None),         label_noise_scale=0.5),     encoder=dict(         layer_cfg=dict(             ffn_cfg=dict(                 embed_dims=256, feedforward_channels=2048, ffn_drop=0.0),             self_attn_cfg=dict(dropout=0.0, embed_dims=256, num_levels=5)),         num_cp=7,         num_layers=7),     neck=dict(         act_cfg=None,         in_channels=[             256,             512,             1024,             2048,         ],         kernel_size=1,         norm_cfg=dict(num_groups=32, type='GN'),         num_outs=5,         out_channels=256,         type='ChannelMapper'),     num_feature_levels=5,     num_queries=1500,     positional_encoding=dict(         normalize=True, num_feats=128, offset=0.0, temperature=20),     test_cfg=dict(max_per_img=512),     train_cfg=dict(         assigner=dict(             match_costs=[                 dict(type='FocalLossCost', weight=2.0),                 dict(box_format='xywh', type='BBoxL1Cost', weight=5.0),                 dict(iou_mode='giou', type='IoUCost', weight=2.0),             ],             type='HungarianAssigner')),     type='DINO',     with_box_refine=True)
the loss began with 55.6 and ended with 21.9307, but the evaluation result is bbox_mAP_copypaste: 0.00170 0.028 0.000 0.001 0.006 0.005 0.010 0.031 0.050
The loss of normal model ends with around 3, is the layers larger than 6 automatically frozen?

Maybe you are using the pretrained model, which use 6 layers. If you lessen the layer, you are still using the trained weight. I you add extra layer, those are randomly initialized and need large scale dataset to be trained.
Its just a guess.

Maybe you are using the pretrained model, which use 6 layers. If you lessen the layer, you are still using the trained weight. I you add extra layer, those are randomly initialized and need large scale dataset to be trained.
Its just a guess.

Thanks for replying! But I checked the log and it showed that onlly the backbone is loaded from pretrained model. And the init information of layer 7 and layers before are the same, which is The value is the same before and after calling init_weights of DINO, so I think they were all trained from scratch.
And I think when the model mismatches with the checkpoint, it will still load the matched parameters and init the rest?


Issues with custom dialer

I want to customize the dialer so that it can dial outgoing connection using IPv6 address.
However, this library will prefer DNS resolution by default. In a dual-stack environment and the client use socks5h (which means the hostname will not be resolved by the client), the resolved IP address might be IPv4, thus making the dialing failed.
Although I found a workaround that can pretend the name resolution success with empty result:
func (d YourResolver) Resolve(ctx context.Context, name string) (context.Context, net.IP, error) {
	return ctx, []byte{}, nil
}

And it can trick the address formatter to use the FQDN instead:
https://github.com/things-go/go-socks5/blob/6191a3489f54e890dbaeafc168c470c9fb2135e4/statute/addr.go#L19-L26
It depends on the fallback of the error case, so it's not ideal to implement like this.
Considering we need to have backwards compatibility, we also can't change the resolve logic here:
https://github.com/things-go/go-socks5/blob/6191a3489f54e890dbaeafc168c470c9fb2135e4/handle.go#L50-L65
So I'm creating an issue looking for ideas to solve this issue.

Maybe your question is to force your custom resolver to return an ipv6 address?
You can try net.ResolveIPAddr("ip6", name) or other go resolver, maybe like following:
func (d DNSResolver) Resolve(ctx context.Context, name string) (context.Context, net.IP, error) {
	addr, err := net.ResolveIPAddr("ip6", name)
	if err != nil {
		addr, err = net.ResolveIPAddr("ip4", name)
		if err != nil {
			return ctx, nil, err
		}
		return ctx, addr.IP, nil
	}
	return ctx, addr.IP, nil
}

Maybe your question is to force your custom resolver to return an ipv6 address?
You can try net.ResolveIPAddr("ip6", name) or other go resolver, maybe like following:
func (d DNSResolver) Resolve(ctx context.Context, name string) (context.Context, net.IP, error) {
	addr, err := net.ResolveIPAddr("ip6", name)
	if err != nil {
		addr, err = net.ResolveIPAddr("ip4", name)
		if err != nil {
			return ctx, nil, err
		}
		return ctx, addr.IP, nil
	}
	return ctx, addr.IP, nil
}

So what I'm trying to do is that, I want to use user a for IPv4 outbound and b for IPv6 outbound.
Although I can use ctx for storing user information, it will be way easier if we have the option to skip the domain resolution.


Update ws.py

Coverage remained the same at 63.922% when pulling ddff8ab1165500fc6438851f566bbfe230f4c748 on naveenuchiha:patch-1 into 4a7409a8493b6ca63dbd9cd41f5acbbcb71a6905 on Charcoal-SE:master.

Hi! Welcome to SmokeDetector. While we appreciate your contribution, would you mind taking a look at our continuous integration and at least pass the checks?
Also, can you please describe why you're removing those blank lines? It seems to violate our code conventions.
Before we understand the intention behind this PR, we have to label it as [invalid]. Thank you for your understanding.


conditional model

I would like to express my sincere gratitude for your recent series of remarkable works. I am a student in the Department of Biostatistics and a beginner in diffusion models. To deepen my understanding of latent diffusion models (LDM), I have been studying your code. In your paper DiscDiff: Latent Diffusion Model for DNA Sequence Generation, you mentioned that the current diffusion model is a conditional model. However, the provided code appears to be for an unconditional model. I wanted to clarify: to achieve specific generation for the 15 species mentioned in the paper, should we modify the UNet2DModel in diffusers to UNet2DConditionModel, or will the code for conditional generation be updated in the future?
Apologies if this question seems a bit naive, but I am genuinely looking forward to your guidance. Thank you in advance for your time and assistance!

Hello @Kee-Qin ,
We will release the code. This repo is for the workshop paper initially, so the unconditional version of the model is sufficient for reproducing the results. Now, we have a follow-up work  Paper Link which will be presented in NeurIPS 2024. We will release the code around the neurIPS conference 2024 (in about 20 days) in the this new repo https://github.com/Zehui127/Absorb-Escape
The delay is because of the new version of the paper contains multiple systems, including the species conditioned DiscDiff.


Toggle pretty print successful eval result under the code

When standing inside successful evaluation, Ctrl+I should toggle between compact one-line truncated view (first) and expanded pretty-printed view (second):

Note: do not use clojure.pprint, better implement it yourself: same idea, but sans the part where it tries to fit keys/values into 80 chars

Note: do not use clojure.pprint, better implement it yourself: same idea, but sans the part where it tries to fit keys/values into 80 chars

Does clojure.pprint/*print-right-margin* not work for keys/values?


ACCUMULO-4722 Use Objects.equals in Pairs class"

ACCUMULO-4722 Use Objects.equals() in Pair equals method
Removed private equals method in Pairs class and instead made use of the
java.util.Objects.equals method. Updated the test class to verify
several variations of the equals method.

ACCUMULO-4722 Use Objects.equals in Pairs class
Made and additional update Pairs.java to replace the private hashCode method with Objects.hashCode(). I compared the performance of Objects.hash(Object... values) vs Objects.hashCode(Object o) prior to making the update and noticed that in my local tests the use of Objects.hashCode was noticeably faster than Objects.hash.

@jmark99 Looks good.  You finished with updates?  I can merge your changes if so.

Hi Keith, yes it can be merged. Thanks!


Dead links

The links to the O'Reilly Guides, as well as the Example PAM application are all returning a 404.

@Jnchi  You can use web.archive.org

Writing PAM Modules, Part One
Writing PAM Modules, Part Two
Writing PAM Modules, Part Three


option: ampExperimentToken

Adds a meta tag to the head to use AMP experiments.
@ljharb @jeremyhuang3627

LGTM.


Adding new Jupyter Notebook named adder_implementation.ipynb and updating readme file with it.

@heyshakya thanks for the contributions!
This PR is also removing xls-workshop-openlane-ja.ipynb is that intentional?

/cc @tdene, since I believe he originally authored the notebook

adder_implementation.ipynb

Can you suffix this with openlane and sky130?

Yeah correct, "/cc tdene, since I believe he originally authored the notebook"
I found this on Twitter and thought of going through it.
Understood it, so thought of putting it in the repo.

Can you suffix this with openlane and sky130?
No, I don't know how to do it.

No, I don't know how to do it.

sorry for not being precise, I meant renaming the file.

I have changed the file name from adder_implementation to adder-openlane-sky130
Please close this PR

can we also rebase the change, and update the notebook to use the same conda-eda install process as the other ones? (see the setup cell in https://github.com/chipsalliance/silicon-notebooks/blob/main/digital-inverter-openlane.ipynb)

Hi @proppy ,
Hope you are doing well.
I went through the files and tried to do what you said to do,
But I was unable to do that.
Can you do that, I am unable to do that.
Regards,
Abhishek


(Do not merge) Changes to deploy on openstack (in dhcp network)

Hi @bacastelli
Some of the changes i made to the code, to allow for deployments on our specific openstack cluster.
These changes should not be merged (for obvious reasons).
I am not sure if these problems/features will be used by customers. So it might not be worth the effort to create a proper fix.
One more problem i have, which i have not fixed in metro: when vsc is deployed in a dhcp managed subnet, it receives the dns-domain and nameservers, however i have configured a different one in metro deployment, that one is not added as primary or secondary dns.

I am going to close this request just to keep it out of the way. I found the information contained herein to be quite helpful. We will still have access to the PR after it's closed for future reference. Thank you.


Refactor Person and UniquePersonList

Model Classes:
Person --> Guest
UniquePersonList --> UniqueGuestList
Several other test classes were refactored when refactoring the above 2 classes. Note: Remember to refactor those test classes in a later PR.

Good work on the refactoring!


Why was the sourcetype changed on the isc dhcpd sourcetype??

https://github.com/splunk/splunk-connect-for-syslog/pull/1772
Why did the sourcetype get changed from isc:dhcp to isc:dhcpd??
Docs indicate that it should be isc:dhcp
https://splunk.github.io/splunk-connect-for-syslog/1555/sources/vendor/ISC/dhcpd/
Also the Splunk add-on to use with this datasource (as documented) should be isc:dhcp
https://docs.splunk.com/Documentation/AddOns/released/ISCDHCP/Sourcetypes

The document is not updated correct, we will fix it
If you need to change the sourcetype urgently please update splunk_metadata.csv and restart sc4s
isc_dhcpd,sourcetype,isc:dhcp

What is going to be the fix then? Are you going to leave the default sourcetype isc:dhcp or are you leaving it isc:dhcpd?  Because leaving it isc:dhcpd means that a workaround is required in order for this data source to work with the ISC DHCP app..
https://docs.splunk.com/Documentation/AddOns/released/ISCDHCP/Sourcetypes

we are not going to change it to dhcpd as it can impact other customers as well, what we are going to do is:

Correct the doc
Provide the local parser to
override the oob parser if a customer need it.


Add a cranelift-compile fuzz target

Add a cranelift-compile fuzz target that generates test cases to try to compile for all supported architectures.

Trevor tells me that he ran this fuzz target for four hours without finding any bugs, which actually has me concerned that it may not be testing what we wanted to test.

That's weird, when I ran this it crashed almost immediately even without a previous corpus.
Ran it again with (This is just the run string that I usually use for fuzzgen):
cargo +nightly fuzz run --no-default-features cranelift-compile -- -rss_limit_mb=8192 -max_len=128 -runs=5000000
Gives me the following input (base64'd):
////L/////////////////8K/wAAXQAAAAD/AQAAAA==

Crashes with:
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: Undeclared function u0:0 is referenced by u1:0!', fuzz/fuzz_targets/cranelift-compile.rs:60:53
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

A follow up from the issue above, the default fuzzgen config generates random function references even if they don't actually exist. This works for icache, but not for fuzzgen since there we actually try to do some linking (via cranelift-jit).
Altering that config for this target does get it past that error.

Huh, I guess I'm surprised that cranelift-jit works at all for cross-compiling. I don't think we should try to test those code paths when cross-compiling. So I think this is another argument for merging this new fuzz target into cranelift-icache, or at least making it more closely resemble that fuzz target.

Could this perhaps get added as a mode to cranelift-fuzzgen instead of adding a new fuzz-target? Mostly due to how oss-fuzz allocates time I think it would be best to not expand the time allocated for Cranelift-specific fuzzing since the time alloted to cranelift-fuzzgen is probably good enough, although I could see this going both ways.
My thinking is that the first byte of the input, for example, and its lowest bit would indicate "run in an interpreter" or "compile, possibly a cross-compile"

Merging this into icache would have the same effect right? We would keep the same number of targets as we do today (i.e. fuzzgen and icache). I think that would be the best way to go.

Oh dear sorry disregard me, indeed!


Mac silicon compatibility

Hello, I was reviewing the code to handle the installation of java, but I still want to clarify a question, can download the version of java for mac silicon?
in advance good day

Current version (3.3.9) would not install arm version of java. I implemented this on 'v3.4.0' branch but I haven't test yet (I don't have mac)

Ok, So I'll try it with my mac.


Can’t search: MDB_READERS_FULL

Describe the bug
Hello, I was running a bunch of requests in parallel and then meilisearch throwed a few hundred times this error:
{
  "code": "internal",
  "link": "https://docs.meilisearch.com/errors#internal",
  "message": "An internal error has occurred. `MDB_READERS_FULL: Environment maxreaders limit reached`.",
  "type": "internal"
}

I would like to know how many connection we can do at the same time?
And also this limit should be specified in the known limits documentation page.
To Reproduce
Steps to reproduce the behavior:

Run meilisearch
Send requests

Expected behavior
It execute the search
Screenshots
If applicable, add screenshots to help explain your problem.
Meilisearch version: main branch
Additional context
Additional information that may be relevant to the issue.
I’m running macOS

Thanks @irevoire
Is it something we also had in the previous versions? (v0.27.0 or v0.28.0)

Hey,
I had an idea that maybe we could send a 429 Too Many Requests instead of an internal error. But we could find more and better ideas by looking at the Retry-After guide by Mozilla.

Maybe we could create a « search service » that holds ONE read transaction and make every search goes through.
And every 2s, it could drop it and open a new one to get the latest version of the DB?

Why not but I find it a little bit more complex as there is more synchronization to do between thread and workers, also doing so forces us to share the read transaction between thread (Sync) and enable the related cargo feature of heed.

Yeah, that's sure, but it does fix the issue instead of letting the user handle it.

share the read transaction between thread

Do you think it would solve the issue to have one read transaction by thread instead of one for everyone?
But I'm afraid some people will have more than 126 threads and thus will get the error right away 😬

What I worry about is that it is not recommended to keep a read transaction for too long as it can block write transactions (as there can only be 1 read transaction and 1 write transaction at the same time) and keeping a read transaction for too long can force the database to grow in size.
Therefore, I am worried about how will you make sure to drop the read transaction after 2 seconds if there is no read transaction and only a writing one. Will you use channels to send read transaction requests to a function that is running in a separate thread that is dynamically handling the 2 seconds timeout to drop the transaction? That's a quite complex system.

Huum, I don't think we'll need such a complex system, but I'm not sure yet; I think before choosing any solution, it would be good to make a POC of this one since it's the only one really solving the issue instead of making it appear later.

Another solution could simply be to either:

Let the user specify how many parallel readers he wants to allow using a CLI argument.
Or multiply the number of available CPUs by the current default value of 126 (allowing 126 readers by thread).

But why is there a limit in the beginning? Will it throw an error if we ask for too many readers?

So after reading a little bit more about the consequences of setting this parameter to a high value, we can as it doesn't impact performances.

Closing this in favor of https://github.com/meilisearch/meilisearch/issues/2786

Hi @Kerollmops and @curquiza 👋
I was just wondering if I could get some clarification on this one.
What exactly do we mean by "at the same time"?
I'm trying to understand what the implication is for the cloud team.
Is this, "126 requests started within the same millisecond"? Or do we mean that 126 is the maximum number of requests that can be in a "processing state" by Meilisearch concurrently?

Or do we mean that 126 is the maximum number of requests that can be in a "processing state" by Meilisearch concurrently?

This one, this is the number of readers, so the number of "queued" request that Meilisearch can have
#2786 is closed so now it's not 126 but 1024 😇 (released in v0.30.0)


Visualization of contemporary lake temperature trends

@jread-usgs @gjahansen2
Ideas/feedback needed here.
One can go a pretty simple route. State-wide lake average summertime surf water temps. Based on WiLMA modeling. The timeseries is noisy but there is a trend to it. Doesn't pop like I usually would want a vis to.

Another thing we talked about is two maps, before/after (1980's, 2000's, or something like that). Overall differences aren't huge. Cross-lake differences are larger making such comparison challenging.

Thoughts?

What about maps of change, or percent change? Like this, but with a metric
that makes sense to people (percent increase?) or if you base it on summer
average temperatures, you could say degree increase
[image: Inline image 1]
On Thu, Jun 9, 2016 at 2:59 PM, Luke Winslow<EMAIL_ADDRESS>wrote:

@jread-usgs https://github.com/jread-usgs @gjahansen2
https://github.com/gjahansen2
Ideas/feedback needed here.
One can go a pretty simple route. State-wide lake average summertime surf
water temps. Based on WiLMA modeling. The timeseries is noisy but there is
a trend to it. Doesn't pop like I usually would want a vis to.
[image: wi_avg_jas]
https://cloud.githubusercontent.com/assets/347600/15944257/65130844-2e52-11e6-8b32-4707bbcb2ac9.png
Another thing we talked about is two maps, before/after (1980's, 2000's,
or something like that). Overall differences aren't huge. Cross-lake
differences are larger making such comparison challenging.
[image: 1980s_jas_map]
https://cloud.githubusercontent.com/assets/347600/15944328/bdffc74e-2e52-11e6-8715-a405c5dc8a19.png
[image: 2000s_jas_map]
https://cloud.githubusercontent.com/assets/347600/15944329/bdffe58a-2e52-11e6-9aa3-2cc3e6269d30.png
Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/USGS-VIZLAB/climate-fish-habitat/issues/15#issuecomment-225008725,
or mute the thread
https://github.com/notifications/unsubscribe/ASiU6kaidEQ7rFlIflj_SJZQ8mVndvJaks5qKHCjgaJpZM4IyWC_
.

--
Gretchen Hansen
https://gretchenhansen.squarespace.com

Yeah, I like that. But it lacks the interactive features that @jread-usgs craves

Though I don't think I'll use % change. I'll play around with just degC change.

degC change seems fine, and we don't need it to be interactive. Perhaps just a smaller static figure that is inset within the text?


how can i change the direction of the days from right to left?

I want to know if there is a way to make the days of the week and the date to start from the right side.
In Hebrew/Arabic you start from the right side.
I thought the locale will fix it but it doesn't.
I've tried looking in the props but didn't see any relavent prop.
Thanks

Thanks @sarahmarciano.
did you changed the direction to RTL?
if i add to the calendar 'direction: RTL' then it start from the right but then if i will change the locale back to English it will stay RTL and that is no good.

yes, indeed I've changed the direction to RTL,
what I can suggest you if you need your app to be flexible is to have two divs one with direction of rtl and one with direction of ltr and render the div depending on the locale direction
your code will look something like this: (I did not test this snippet of code but you got the idea...)
const rtlLocales = [
    'ae',	/* Avestan */
    'ar',   /* 'العربية', Arabic */
    'arc',  /* Aramaic */
    'bcc',  /* 'بلوچی مکرانی', Southern Balochi */
    'bqi',  /* 'بختياري', Bakthiari */
    'ckb',  /* 'Soranî / کوردی', Sorani */
    'dv',   /* Dhivehi */
    'fa',   /* 'فارسی', Persian */
    'glk',  /* 'گیلکی', Gilaki */
    'he',   /* 'עברית', Hebrew */
    'ku',   /* 'Kurdî / كوردی', Kurdish */
    'mzn',  /* 'مازِرونی', Mazanderani */
    'nqo',  /* N'Ko */
    'pnb',  /* 'پنجابی', Western Punjabi */
    'ps',   /* 'پښتو', Pashto, */
    'sd',   /* 'سنڌي', Sindhi */
    'ug',   /* 'Uyghurche / ئۇيغۇرچە', Uyghur */
    'ur',   /* 'اردو', Urdu */
    'yi'    /* 'ייִדיש', Yiddish */
];

function isLocaleRtl(locale) {
    return rtlLocales.indexOf(locale.split('-')[0]) !== -1;
}

render() {
        const { locale } = this.props;
        if (isLocaleRtl(locale)) {
            return (
                <div dir='rtl'>
                    <DatePicker
                        selected={date}
                        onChange={(date) => setEventDate(date)}
                        locale={locale}
                        minDate={moment()}
                    />
                </div>
            );
        }
        return (
            <DatePicker
                selected={date}
                onChange={(date) => setEventDate(date)}
                locale={locale}
                minDate={moment()}
            />
        );
    }

I have a problen with the keyboard arrows, when I press the left key, it acts as right. and vice versa...
does anyone got a solution?
Thanks!

Hi, I tried adding the  to solve the direction of the arrows in an RTL lang. (right arrow to move to previous month, left to move to next month ) - but it didnt work.
Any suggestions...?
Thanks!

Hi, I tried adding the  to solve the direction of the arrows in an RTL lang. (right arrow to move to previous month, left to move to next month ) - but it didn't work.
Any suggestions...?
Thanks!

it is possible with 2 simple css rules:
.react-datepicker__navigation--previous {
  left: unset !important;
  right: 2px;
  transform: rotateY(180deg);
}
.react-datepicker__navigation--next {
  right: unset !important;
  left: 2px;
  transform: rotateY(180deg);
}


F2 refined hev split

Still a work in progress, but I wanted to get this here to make it easier to see changes and such

@calbaker should this be closed?


Six commits: two typo fixes, one update and 3 suggestions.

Six commits: two typo fixes, one update and 3 suggestions.

Can you split off the suggestions?

Hi josegonzalez. I thought I should just separate commits, and not the pull requests. My mistake. I will do it now.

@josegonzalez, all commits are ready. Let me know if there is something more I can do.

Thanks!


shop=nutrition_supplements

http://wiki.openstreetmap.org/wiki/Tag:shop%3Dnutrition_supplements
http://wiki.openstreetmap.org/wiki/File:Nutrition_supplements.svg
http://wiki.openstreetmap.org/w/index.php?title=Proposed_features/nutrition_supplements&oldid=1229431

Only 132 uses

@pnorman I would say these shops have a 250-1.5K convenience shops :1 nutrition_supplements ratio IRL. And you probably wouldn't find these shops in low populated cities/undeveloped areas.
Yellow pages in my locality lists ~30K items but only 11 nutrition_supplements shops.

On the other hand, this tag

only in English language at wiki and
a 6-7 month old tag? most items in Europe were created just 3-4 months ago
was  never visible in presets
only English-speaking Taginfo users know it's presence in OSM and other users who have been pointed to this tag individually via diverse communication channels (at least 3 objects were tagged after my recommendation)

I think it's a good tag.   It obviously corresponds to a real kind of a store that people want to map.  And including all the various alternate tags like shop=[supplements|nutrition|vitamin|vitamins] it adds up to almost 500 uses.
It's better to establish the tag now than to encourage more tag fragmentation or mistagging while waiting for OSM to decide about something.  Going forward, "reasonableness" should be the deciding factor for adding a preset - not whether it is "approved" or frequently used or has a wiki page (All of those things are still super important, but they are red herrings that fail us sometimes).  see also: #2943


Debugging 2

Debugging
MNT: Re-rendered with conda-build 3.21.7, conda-smithy 3.16.2, and conda-forge-pinning 2<IP_ADDRESS>.09.01
Adding deps
Adding deps
Adding debug info
Adding output
Fixing output
printing
WIP: more debugging

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Fixed in https://github.com/scikit-build/scikit-build/pull/646


an error in "Point_cloud_data_sub_sampling_with_Python"

Hello, I am not sure you maintain the code now or not, but in the function def grid_subsampling(points, voxel_size), there is no reference for subsampled_points, I think the code is not runnable.
Thank you.

@whuwuteng thank you for noticing! I updated the Notebook to function properly! Have fun,
Florent


Bluetooth keyboard not in /dev/input/by-path/ or by-id

I have a bluetooth keyboard that does not show up in  /dev/input/by-path/ or by-id at least not with kbd in the name.
Is it possible to connect a Bluetooth keyboard with mouseless?
Thanks for your help!

Yes that should be possible, but I am not sure if every keyboard shows up in one of the two directories and has kbd in it, so if there are other ones that sound plausible you might try them.
Otherwise one can also always use the device /dev/input/eventXY (here XY is a number), you could use libinput list-devices to find out which is the right one, or run journalctl -f and then plug in the keyboard. But this device can change when the keyboard gets replugged.

With cat /proc/bus/input/devices I can get the event number and put that into the config.yaml. This works but as you wrote the event number might change after reboot.
Are there no other options? Manually updating the config.yaml is quite tedious.

Yes one could update the config file automatically before starting mouseless with something list this:
#!/usr/bin/env sh
device=$(libinput list-devices | grep -i 'KEYBOARD_NAME' -A10 | grep 'Capabilities.*keyboard *$' -C 5 | grep "Kernel:" | sed 's_.*\(/dev/input/event[0-9]*\).*_\1_')
[ -z "$device" ] && echo "device not found" && exit 1
echo "device found: $device"
sed -i 's_- "/dev/input/event[0-9]*"_- "'"$device"'"_' ~/.config/mouseless/config.yaml
mouseless --config ~/.config/mouseless/config.yaml

Here you have to replace KEYBOARD_NAME with the name of your keyboard as it is shown in libinput list-devices, and if it finds the device, it replaces it in the config file, where a device with name /dev/input/eventXY has to be present. This is a little hacky but I currently do know a better option.


[Form Recognizer] Sovereign clouds support

Packages impacted by this PR
@azure/ai-form-recognizer
Issues associated with this PR
Fixes https://github.com/Azure/azure-sdk-for-js/issues/19442
Describe the problem that is addressed by this PR

Adds sovereign cloud support to the form recognizer package

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?

Adds the audience option in the client options to support the UsGov and China clouds

Based on the prior art and the architecture board discussions(links below), this design was chosen.

Provide a list of related PRs/issues

https://github.com/Azure/azure-sdk-for-js/issues/17563
https://github.com/Azure/azure-sdk/issues/3353
https://github.com/Azure/azure-sdk/issues/2905
https://github.com/Azure/azure-sdk/issues/3752
https://github.com/Azure/azure-sdk-for-java/pull/25517
.NET - https://github.com/Azure/azure-sdk-for-net/pull/23141
JS - https://github.com/Azure/azure-sdk-for-js/blob/809f05c652ac9bb0fc08ce7048d8b5e33b9fc3e0/sdk/containerregistry/container-registry/src/containerRegistryClient.ts#L36-L41
JAVA - https://github.com/samvaity/azure-sdk-for-java/blob/e0d055cbe7f575c5ba004d1b07a9a48487cf6f6e/sdk/formrecognizer/azure-ai-formrecognizer/src/main/java/com/azure/ai/formrecognizer/models/FormRecognizerAudience.java#L11-L19
https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/604/Sovereign-Cloud-Testing

Are there test cases added in this PR? (If not, why?)
No, but the existing test cases are updated to handle the multiple clouds by checking the endpoint.
Checklists

[x] Added impacted package name to the issue description
[x] Added a changelog (if necessary)

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

subscription().subscriptionId

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

API changes have been detected in @azure/ai-form-recognizer. You can review API changes here
API changes
+     audience?: string;

/azp run js - ai-form-recognizer - tests-weekly

API changes have been detected in @azure/ai-form-recognizer. You can review API changes here
API changes
+ export declare enum FormRecognizerAudience
+     audience?: string;

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

/azp run js - ai-form-recognizer - tests-weekly

Waiting for https://github.com/Azure/azure-sdk-for-js/issues/20773

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-form-recognizer
azure-ai-form-recognizer

/azp run js - ai-form-recognizer - tests-weekly


Auto Install Script

I created an easy install script for convoy. it would save a lot of time deploying and installing convoy on multi hosts...
Source Code
Usage:
curl -#L https://git.io/convoy | sudo bash

Example output:
root@Ni8CAC2:/|>  curl -#L https://git.io/convoy | sudo bash
######################################################################## 100.0%
Downloading convoy ...
Installing convoy ...
'convoy/convoy' -> '/usr/local/bin/convoy'
'convoy/convoy-pdata_tools' -> '/usr/local/bin/convoy-pdata_tools'
mkdir: created directory '/etc/docker/plugins/'
Installing convoy service...
Created symlink from /etc/systemd/system/multi-user.target.wants/convoy.service to /etc/systemd/system/convoy.service.
Convoy is installed!
run 'convoy daemon' with approtiate options then run 'systemctl start convoy'

@pi0
Thanks for the script!
Though it's hard to included it in the official convoy doc, because it's very hard to be a good universal solution.
For example, it's assuming everyone is using systemd, which can be broken for quite some people.
We're working towards a docker container approach which we think is more universal applicable. I can redirect people to here if they're asking for auto start script though. :)

run 'convoy daemon' with approtiate options then run 'systemctl start convoy'

what's this line mean?

@juglans To be honest i was for months ago so i forgot things 😂 But it appears that some initial configuration was needed or if i am wrong just tell me to remove that line and simply starting daemon

add something to end of ExecStart=/usr/local/bin/convoy daemon?
maybe like this,
...
[Service]
ExecStart=/usr/local/bin/convoy daemon --drivers vfs --driver-opts vfs.path=/mnt
...

3q also

@juglans Just added VFS_PATH param. what's your idea migrating that script from gists to this repo?

I just want to run this Unit in my Docker swarm nodes...:)

Ok :))


Superforms hangs vite dev

[X] Before posting an issue, read the FAQ and search the previous issues.

Description
A clear and concise description of what the bug is, and, unless obvious, what you expected instead.
My App is getting more complicated and it seems that the vite dev hangs in superforms converter regularly. I even found a function cal to which I can comment and it will work out. I have 2 more similar function calls elsewhere - but 3rd one kills the vite dev.
If applicable, a MRE
Use one of these stackblitz templates to create a minimal reproducible example that you can link to here:
I would love to build something similar, but I don't know if similar conditions are possible to build

Zod
Valibot

vite dev --debug log
vite:import-analysis 0.22ms [1 imports rewritten] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/string.js +9ms
vite:transform 0.27ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/string.js +9ms
vite:load 9.50ms [fs] /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/number.js +0ms
vite:import-analysis 0.08ms [1 imports rewritten] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/number.js +0ms
vite:transform 0.11ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/number.js +0ms
vite:load 9.63ms [fs] /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/boolean.js +0ms
vite:import-analysis 0.06ms [1 imports rewritten] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/boolean.js +0ms
vite:transform 0.08ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/boolean.js +0ms
vite:load 9.75ms [fs] /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/date.js +0ms
vite:import-analysis 0.06ms [1 imports rewritten] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/date.js +0ms
vite:transform 0.09ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/date.js +0ms
vite:load 9.86ms [fs] /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/array.js +0ms
vite:import-analysis 0.06ms [1 imports rewritten] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/array.js +0ms
vite:transform 0.08ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/array.js +0ms
vite:load 9.97ms [fs] /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/object.js +0ms
vite:import-analysis 0.06ms [1 imports rewritten] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/object.js +0ms
vite:transform 0.08ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/object.js +0ms
vite:load 10.07ms [fs] /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/tuple.js +0ms
vite:import-analysis 0.07ms [1 imports rewritten] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/tuple.js +1ms
vite:transform 0.09ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/tuple.js +1ms
vite:load 10.19ms [fs] /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/mixed.js +1ms
vite:import-analysis 0.05ms [1 imports rewritten] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/mixed.js +0ms
vite:transform 0.08ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/mixed.js +0ms
vite:load 10.29ms [fs] /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/lazy.js +0ms
vite:import-analysis 0.06ms [1 imports rewritten] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/lazy.js +0ms
vite:transform 0.08ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/lazy.js +0ms
vite:load 4.16ms [fs] /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/common.js +3ms
vite:import-analysis 0.01ms [no imports] node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/common.js +3ms
vite:transform 0.06ms /node_modules/sveltekit-superforms/dist/adapters/yup-to-json-schema/converters/common.js +3ms

Minor update, if I comment the call to that particular function, and run dev without it and once chrome loads the page uncomment the function again and refresh -- all works fine. But as soon as I restart it - hangs again at the same place.

Does it hang when it processes the yup-to-json-schema files? If you can reproduce it in a repo, I can take a closer look.

This issue never appeared any more, so closing


SIGSEGV on state.didSet

In an app we recently released to the iOS App Store, we've started seeing the following crash within a couple seconds of opening the app and calling some store.dispatch() functions:

Crashed: com.apple.main-thread
SIGSEGV 0x0000000000000010

Crashed: com.apple.main-thread
0  libobjc.A.dylib                0x1f40 objc_msgSend + 32
1  libswiftCore.dylib             0x1cf6c8 __CocoaDictionary.Iterator.nextKey() + 100
2  libswiftCore.dylib             0x1cf8c0 protocol witness for IteratorProtocol.next() in conformance Set<A>.Iterator + 256
3  libswiftCore.dylib             0xd9364 Sequence.forEach(_:) + 632
4  App                            0x3e135c Store.state.didset +<PHONE_NUMBER> (<compiler-generated>:4341896028)
5  App                            0x3e1684 Store.state.setter + 20 (Store.swift:20)
6  App                            0x3e2770 Store._defaultDispatch(action:) + 173 (Store.swift:173)
7  App                            0x3e3c20 partial apply for closure #1 in Store.createDispatchFunction() +<PHONE_NUMBER> (<compiler-generated>:4341906464)
8  App                            0x3e3c68 partial apply for thunk for @escaping @callee_guaranteed (@in_guaranteed Action) -> (@out ()) +<PHONE_NUMBER> (<compiler-generated>:4341906536)
9  App                            0x3e3d58 partial apply for thunk for @escaping @callee_guaranteed (@in_guaranteed Action) -> () +<PHONE_NUMBER> (<compiler-generated>:4341906776)
10 App                            0x389660 partial apply for thunk for @escaping @callee_guaranteed (@in_guaranteed Action) -> (@out ()) +<PHONE_NUMBER> (<compiler-generated>:4341536352)
11 App                            0x3e5e5c closure #1 in closure #1 in closure #1 in createThunkMiddleware<A>() + 22 (createThunkMiddleware.swift:22)
12 App                            0x3897ac partial apply for thunk for @escaping @callee_guaranteed (@in_guaranteed Action) -> () +<PHONE_NUMBER> (<compiler-generated>:4341536684)
13 App                            0x3e3c68 partial apply for thunk for @escaping @callee_guaranteed (@in_guaranteed Action) -> (@out ()) +<PHONE_NUMBER> (<compiler-generated>:4341906536)
14 App                            0x3e3d58 partial apply for thunk for @escaping @callee_guaranteed (@in_guaranteed Action) -> () +<PHONE_NUMBER> (<compiler-generated>:4341906776)
15 App                            0x3e3c68 partial apply for thunk for @escaping @callee_guaranteed (@in_guaranteed Action) -> (@out ()) +<PHONE_NUMBER> (<compiler-generated>:4341906536)
16 App                            0x3e27d0 Store.dispatch(_:) +<PHONE_NUMBER> (<compiler-generated>:4341901264)

So far, we haven't been able to reproduce it locally, so I'm guessing it has to do with a race condition somewhere.
Any thoughts on what might be causing this or ideas on how to better debug it?

Is there some other error code for a sub-category?
https://developer.apple.com/documentation/xcode/investigating-memory-access-crashes
Never had an iOS app crash tbh :)
Since you speak of race conditions: I do wonder if you do something odd around the use of createThunkMiddleware. I'd try to symbolicate the crash log first and inspect the call stack further.

@mhamann Did you figure out a cause for this?

Thanks for checking in @DivineDominion. I must've missed your original reply. We did solve this. We were using Combine within an Actor, and that seems to have been the trigger for this issue. Somehow, the Actor's subscription to the store was causing memory to be accessed in an unsafe way, and thus the sigsev crash.
My iOS dev knowledge isn't deep enough to know whether or not there is a good way to solve this, but we ended up moving the code from our Combine logic into ReSwift middleware. It ends up making the dispatch flow async, which is somewhat annoying, so other approaches might be better.

@mhamann Great, then I'll close this issue!
For some async tasks, try out https://github.com/ReSwift/ReSwift-Thunk/ to dispatch an action that can perform something complex, access the state in the process, and dispatch actions to indicate progress. Like the HTTP request/pending/result triad.


fix ValidateCPUPeroid

Ⅰ. Describe what this PR did
#1637 : add unit-test for apis/opts/cpu.go ValidateCPUPeroid
Ⅱ. Does this pull request fix one issue?
NONE
Ⅲ. Describe how you did it
NONE
Ⅳ. Describe how to verify it
NONE
Ⅴ. Special notes for reviews
NONE

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jiahao.mjh seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1655      +/-   ##
==========================================
+ Coverage   41.16%   41.18%   +0.02%
==========================================
  Files         274      274
  Lines       18091    18089       -2
==========================================
+ Hits         7447     7450       +3
+ Misses       9729     9725       -4
+ Partials      915      914       -1

Impacted Files
Coverage Δ

ctrd/image.go
75.27% <0%> (-2.75%)
:arrow_down:

daemon/mgr/system.go
75% <0%> (-1.73%)
:arrow_down:

cli/container.go
0% <0%> (ø)
:arrow_up:

cli/common_flags.go
0% <0%> (ø)
:arrow_up:

daemon/mgr/container.go
50.23% <0%> (+0.15%)
:arrow_up:

apis/server/utils.go
66.66% <0%> (+4.76%)
:arrow_up:

apis/opts/cpu.go
50% <0%> (+50%)
:arrow_up:


Danger

Restrict changing only one test/extension per PR
Fail travis on danger error

Current coverage is 46.36% (diff: 100%)

Merging #241 into master will not change coverage

@@             master       #241   diff @@
==========================================
  Files            64         64
  Lines          2737       2737
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1269       1269
  Misses         1468       1468
  Partials          0          0

Powered by Codecov. Last update 84921c6...f1a2ba0

I really liked your idea!!! Danger is all about being creative!


MNT: Increase minimum tedana version

There is a new tedana release (0.0.9), in which the masking we apply is made less aggressive. This should address user-reported issues with multi-echo coverage in fMRIPrep (e.g., here, here, and https://github.com/ME-ICA/tedana/issues/600).
Pinging @emdupre and @jbteves because they were involved in the release.
Changes proposed in this pull request

Increase the minimum tedana version to 0.0.9, now that there is a new release.

Documentation that should be reviewed

None

@tsalo A couple points:

Can you merge master?
Can you explain what tedana needs scipy 1.3.3+ for? Are there bugs we should be expecting to see using tedana 0.0.9a1 that would be fixed by upgrading scipy?

Can you merge master?

Can do.

Can you explain what tedana needs scipy 1.3.3+ for? Are there bugs we should be expecting to see using tedana 0.0.9a1 that would be fixed by upgrading scipy? Also scikit-learn. (Looking at https://github.com/ME-ICA/tedana/pull/575/files#diff-9184081828959d098f4c5ff1be1111dbc955d9284508838db2182fd6863af7aa.)

We updated the minimum requirement in https://github.com/ME-ICA/tedana/pull/518 to fix test failures, but we never successfully pinned down the cause of the failing tests, and we aren't sure if 1.3.3 is necessary to fix things.
@emdupre @jbteves what do you think is the best solution? Should we test out the scipy version fMRIPrep has pinned in Dockerfile (1.1.0) with tedana to see if it breaks things and, if not, make our next release with that version in our requirements? Then I could change this PR to pin to 0.0.10 instead of 0.0.9.

@emdupre @jbteves what do you think is the best solution? Should we test out the scipy version fMRIPrep has pinned in Dockerfile (1.1.0) with tedana to see if it breaks things and, if not, make our next release with that version in our requirements? Then I could change this PR to pin to 0.0.10 instead of 0.0.9.

I can open a PR on tedana now and see if it passes our test suite as-is ! We can re-visit once we have those results.

@tsalo @emdupre I see that https://github.com/ME-ICA/tedana/pull/723 went in, so it looks like master will at some point ask for tedana ~= 0.0.10. What are your thoughts on hard-pinning 0.0.9a1 for the LTS (#2403), or should I go with tedana >=0.0.9a1,<0.0.11,!=0.0.9?

To be honest, I'm not sure what changes are acceptable under the LTS version, but if it is acceptable to update tedana when that will change the results somewhat (combined data will be masked less aggressively), then I think that would be preferable for most users. If not, though, then your solution to pin 0.0.10 in master and 0.0.9a1 in 20.2.1 sounds good.

I agree with @effigies that it depends on whether you consider it a bugfix or not. I am not familiar enough with the multi-echo branch of the pipeline to make a decision right now.

It comes down to whether you consider the change in masking a bug fix or a substantive change in processing. If it turns data from unusable to usable, then I'd consider it a bug fix. If, for data that was already masked sensibly, the change in masks is beyond the run-to-run variability, then I would err on the side of it not being a bug fix.

Thanks all ! To weigh in here: it's not that previous versions gave incorrect results, but those results may not have aligned with user expectations based on how this method was previously described in the literature. Essentially: While the previous code correctly generates T2* maps, it only does so in voxels where multiple echos have signal. Since multi-echo acquisitions are often described as increasing signal in regions which typically suffer from dropout, users were surprised that this was not the case in their processed data (because these dropout-prone regions typically only have signal in the first echo and so were not included in the generated T2* maps before this release).
So, I'd say it's really a judgment call. The maps from either tedana release are technically correct, but the maps from the more recent release will better match most user expectations. I can of course go into more detail on why we switched strategies for this t2smap workflow, if that helps in making this decision !

Thanks for the context, Elizabeth. Sounds like the more conservative choice is to consider it an improvement rather than a bug fix, but there's an argument here that users find the current results less usable.
If it were me, I'd probably go the conservative route, but I'll defer to Basille and Pierre if they think it makes sense to include this update.

@oesteban I think the goal was to move to 0.10+ once that was released.

Ugh, thanks for the heads-up. I guess merging this can't harm.


Constructor visibility must be public

see https://docs.abapopenchecks.org/checks/90/

merged, closing