There is no way in Pygeos to filter geometries with nan or inf coordinate values

There should be a way to assert geometries with missing or infinite values.
GeoPandas handles empty geometries s.is_empty or geometries with missing values s.isna().
a = pygeos.points(np.array([np.nan, np.inf]))
# <pygeos.Geometry POINT (nan inf)>

pygeos.is_geometry(a)
# True

pygeos.is_missing(a)
# False

pygeos.is_valid_input(a)
# True

The only workaround I could find is searching for 'Invalid Coordinate' in the string returned by pygeos.is_valid_reason but it is not safe:
has_invalid_coo = pd.Series(pygeos.is_valid_reason(df['geometry'].values)).str.contains('Invalid Coordinate')
# True

Maybe there's something that can be done with pygeos.get_coordinates.

I think that you should be able to do this with pygeos.get_coordinates indeed:
>>> coords, inds = pygeos.get_coordinates(a, return_index=True)
>>> inds[~np.isfinite(coords).any(1)]
array([0])

I don't think we need a separate function to achieve this, because we are thinking of forbidding NaN and inf in geometries anyway.
Please close the issue if this answered your questions.

@carlosg-m follow-up question for you: since you opened a few issues with questions related to NaN / Inf coordinates, do you have a specific use case for them? Or is it that they are contained in your data source, and thus you need to handle them in some way? (and what kind of data source does contain NaN/Inf coordinates?)
(it would be interesting for us to have some more context, also for deciding whether we could be more strict about forbidding such values or not when creating geometries. For example, if your have a geospatial data source that allows such values, it might be rather annoying to forbid such values, as in such a case you typically want to "fix" your geometries after reading the file has created them)

@carlosg-m follow-up question for you: since you opened a few issues with questions related to NaN / Inf coordinates, do you have a specific use case for them? Or is it that they are contained in your data source, and thus you need to handle them in some way? (and what kind of data source does contain NaN/Inf coordinates?

My use cases are related with data science and feature engineering for ML models or traditional ad-hoc queries.
I work in the energy sector, and we have custom made legacy GIS software.
The problem is that datasets that come from this systems are noisy and have all sorts of problems, including Infs, Nans, empty geometries, invalid geometries and so on. I try to preserve as much information as possible, for example a powerline might be misrepresented by a Linestring with a single point, however instead of discarding this invalid record I usually try to fix it and represented it as a single point.
I was using shapely together with pyspark to parse geojson from these legacy sources, but sometimes some geometries were so corrupted there was nothing that could be done.
However the last time I've encountered the Inf problem was when dealing with projections. The same dataset can contain geometries in WGS84, but also other coordinates in Cartesian planes that represent floor plans.
A very common preprocessing step we use is intersecting the geometries with the shape of the country they should be in using an official source, to filter out floor plans. When we project coordinates in ETRS89, the Cartesian coordinates from the floor plans, will yield Inf values, that later have to be dealt with.
During the development Pygeos you have to keep in mind that even if the input dataset is relatively clean and valid, some transformations may yield invalid or corrupt outputs. And there should be a way at least to ignore or discarding them without compromising vectorized operations.
There could be an option in some functions to ignore or raise errors given a corrupt input, like in Pandas {‘ignore’, ‘raise’, ‘coerce’}. However I have no idea if such behavior is in scope or aligned with your vision of what this lib should do. Since there is no Pygeos method to parse GeoJson you could argue that that all WKT or WKB input geometries that are consumed are assumed to have been serialized properly. But doing so you will force some users to use other libs for preprocessing, and benefit of performance will be lost.


Decide on and enforce consistent names for witnesses/delegates/committee members/etc.

Rename everything consistently within the codebase, whether part of the protocol or not.
For example, blocks currently have a field delegate_signature which is a straight up lie given the current ontology.

This isn't a hard fork.  It is an API change though.


Prepare kubevirt for sonobuoy conformance testing

What this PR does / why we need it:
Build and release a conformance container which can be used for third parties to test ther kubevirt installation for conformance.
This initial implementation only marks one first single test  as a conformance test.
The conformance binary which will be invoked by sonobuoy will do the following:

by default limit test selection to tests with the tag [Conformance]
by default skip all tests with the tag [Disruptive]
tell our test binary to write the junit results to the place where sonobuoy expects it
tell sonobuoy when the tests are done by creating the done file in the expected location

To generate a sonobuoy plugin for a released conformance test suite, run
$ sonobuoy gen plugin --name kubevirt-conformance --cmd /usr/bin/conformance --image kubevirt/conformance:<tag> -f junit > kubevirt.yaml

Then luanch the suite:
$ sonobuoy run --plugin kubevirt.yaml

Here a full flow in the development environment (note the registry:500 prefix and the :devel tag:
$ sonobuoy gen plugin --name kubevirt-conformance --cmd /usr/bin/conformance --image registry:5000/kubevirt/conformance:devel -f junit > conformance.yaml
sonobuoy gen plugin --name kubevirt-conformance --cmd /usr/bin/conformance --image registry:5000/kubevirt/conformance:devel -f junit > conformance.yaml
$ sonobuoy retrieve
202008201609_sonobuoy_8f8d0b0e-1d37-485a-b61d-bf7185198fbf.tar.gz
$ sonobuoy results<PHONE_NUMBER>09_sonobuoy_8f8d0b0e-1d37-485a-b61d-bf7185198fbf.tar.gz
Plugin: kubevirt-conformance
Status: passed
Total: 580
Passed: 1
Failed: 0
Skipped: 579

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Note, to try this out easily, I pushed the conformance binary on my docker hub account. It can be used via
sonobuoy gen plugin --name kubevirt-conformance --cmd /usr/bin/conformance --image docker.io/rmohr/conformance:latest -f junit > kubevirt.yaml

Release note:
Initial support for sonobuoy for conformance testing

/cc @phoracek
/cc @EdDev
Here a draft of what I would love to see regarding to conformance tests.

Please add some documentation under docs/. The PR description looks pretty good, it could be more or less recycled.

Done. What I don't like is that there is no testing for it. We should probably at least have a periodic job which executs the conformance tests.The user-guilde should also be updated I guess.

/lgtm

/retest

/approve

/retest


Build crate in a workspace

Make sure these boxes are checked! 📦✅

[X] You have the latest version of rustfmt installed and have your
cloned directory set to nightly

$ rustup override set nightly
$ rustup component add rustfmt-preview --toolchain nightly

[X] You ran rustfmt on the code base before submitting
[X] You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨

r=me, but looks like rustfmt on CI disagrees with your local rustfmt?

Huh. I did a rustup and still I'm not getting any more diffs... I wonder if Travis is using an old Rust??

Stable vs nightly perhaps?

I just double checked that our CI config was using nightly (and did https://github.com/rustwasm/wasm-pack/pull/432 at the same time) and confirmed that yes it is. And I am also using nightly...

Added some logging of the rustfmt version to #432

Looks like I am using an out-of-date rustfmt!
Mine:
$ cargo fmt --version
rustfmt 0.99.5-nightly (90692a5 2018-09-26)

Theirs:
$ cargo fmt --version
rustfmt 0.99.6-nightly (750b252 2018-10-18)

But I can't update mine further...
$ rustup update
info: syncing channel updates for 'stable-x86_64-unknown-linux-gnu'
info: syncing channel updates for 'beta-x86_64-unknown-linux-gnu'
info: syncing channel updates for 'nightly-x86_64-unknown-linux-gnu'
info: latest update on 2018-11-05, rust version 1.32.0-nightly (e6c5cf923 2018-11-04)
error: component 'clippy-preview' for 'x86_64-unknown-linux-gnu' is unavailable for download
info: checking for self-updates

       stable-x86_64-unknown-linux-gnu unchanged - rustc 1.30.0 (da5f414c2 2018-10-24)
         beta-x86_64-unknown-linux-gnu unchanged - rustc 1.31.0-beta.4 (04da282bb 2018-11-01)
  nightly-x86_64-unknown-linux-gnu update failed - rustc 1.31.0-nightly (2bd5993ca 2018-10-02)

$ cargo fmt --version
rustfmt 0.99.5-nightly (90692a5 2018-09-26)

Huh, looks like it was clippy holding me back
error: component 'clippy-preview' for 'x86_64-unknown-linux-gnu' is unavailable for download


Screen recording non-working in Wayland

The built-in GNOME Screenshot tool is missing the useful feature to easily take screen recordings. This is a really nice feature to have, and it's likely due to a system dependency missing.
All screen recording applications are seemingly non-functional. This includes (in my limited testing) OBS Studio and Kooha.

Non-flatpak system screenshot module incapable of screen capture

OBS Studio console traceback:
qt.qpa.qgnomeplatform: Could not find color scheme  ""
debug: Found portal inhibitor
debug: Attempted path: share/obs/obs-studio/locale/en-US.ini
debug: Attempted path: /app/share/obs/obs-studio/locale/en-US.ini
debug: Attempted path: share/obs/obs-studio/locale.ini
debug: Attempted path: /app/share/obs/obs-studio/locale.ini
debug: Attempted path: share/obs/obs-studio/themes/Grey.qss
debug: Attempted path: /app/share/obs/obs-studio/themes/Grey.qss
info: Platform: Wayland
info: CPU Name: AMD Ryzen 7 5800X3D 8-Core Processor
info: CPU Speed: 3592.345MHz
info: Physical Cores: 8, Logical Cores: 16
info: Physical Memory: 64212MB Total, 41936MB Free
info: Kernel Version: Linux 6.5.3-chos1-chimeraos-1
info: Flatpak Branch: stable
info: Flatpak Arch: x86_64
info: Flatpak Runtime: runtime/org.kde.Platform/x86_64/6.4
info: App Extensions:
info:  - org.freedesktop.LinuxAudio.Plugins.TAP
info:  - org.freedesktop.LinuxAudio.Plugins.swh
info:  - com.obsproject.Studio.Locale
info: Runtime Extensions:
info:  - org.freedesktop.Platform.GL.default
info:  - org.freedesktop.Platform.openh264
info:  - org.freedesktop.Platform.openh264
info:  - org.gtk.Gtk3theme.adw-gtk3
info:  - org.gtk.Gtk3theme.adw-gtk3-dark
info:  - org.kde.PlatformTheme.QGnomePlatform
info:  - org.kde.WaylandDecoration.QGnomePlatform-decoration
info:  - org.kde.Platform.Locale
info:  - org.freedesktop.Platform.GL.default
info: Flatpak Framework Version: 1.15.4
info: Desktop Environment: GNOME (gnome-session-oneshot)
info: Session Type: wayland
info: Qt Version: 6.4.3 (runtime), 6.4.3 (compiled)
info: Portable mode: false
info: OBS 29.1.3 (linux)
info: ---------------------------------
info: ---------------------------------
info: audio settings reset:
	samples per sec: 48000
	speakers:        2
	max buffering:   960 milliseconds
	buffering type:  dynamically increasing
info: ---------------------------------
info: Initializing OpenGL...
info: Using EGL/Wayland
info: Initialized EGL 1.5
info: Loading up OpenGL on adapter AMD AMD Radeon RX 7900 XTX (gfx1100, LLVM 15.0.7, DRM 3.54, 6.5.3-chos1-chimeraos-1)
info: OpenGL loaded successfully, version 4.6 (Core Profile) Mesa 23.1.8 (git-4baea976bb), shading language 4.60
info: ---------------------------------
info: video settings reset:
	base resolution:   1920x1080
	output resolution: 1920x1080
	downscale filter:  Bicubic
	fps:               60/1
	format:            NV12
	YUV mode:          Rec. 709/Partial
info: NV12 texture support not available
info: P010 texture support not available
info: Audio monitoring device:
	name: Default
	id: default
info: ---------------------------------
warning: No AJA devices found, skipping loading AJA UI plugin
warning: Failed to initialize module 'aja-output-ui.so'
warning: No AJA devices found, skipping loading AJA plugin
warning: Failed to initialize module 'aja.so'
warning: Failed to load 'en-US' text for module: 'decklink-captions.so'
warning: Failed to load 'en-US' text for module: 'decklink-output-ui.so'
libDeckLinkAPI.so: cannot open shared object file: No such file or directory
warning: A DeckLink iterator could not be created.  The DeckLink drivers may not be installed
warning: Failed to initialize module 'decklink.so'
info: [pipewire] No captures available
warning: v4l2loopback not installed, virtual camera disabled
info: [obs-browser]: Version 2.21.1
info: [obs-browser]: CEF Version 103.0.5060.134 (runtime), 103.0.0-5060-shared-textures_143.2591+g4204d54+chromium-103.0.5060.134 (compiled)
info: VAAPI: API version 1.18
info: FFmpeg VAAPI H264 encoding supported
info: FFmpeg VAAPI HEVC encoding supported
info: [obs-websocket] [obs_module_load] you can haz websockets (Version: 5.2.3 | RPC Version: 1)
info: [obs-websocket] [obs_module_load] Qt version (compile-time): 6.4.3 | Qt version (run-time): 6.4.3
info: [obs-websocket] [obs_module_load] Linked ASIO Version: 101201
info: [obs-websocket] [obs_module_load] Module loaded.
info: ---------------------------------
info:   Loaded Modules:
info:     text-freetype2.so
info:     rtmp-services.so
info:     obs-x264.so
info:     obs-websocket.so
info:     obs-vst.so
info:     obs-transitions.so
info:     obs-outputs.so
info:     obs-libfdk.so
info:     obs-filters.so
info:     obs-ffmpeg.so
info:     obs-browser.so
info:     linux-v4l2.so
info:     linux-pulseaudio.so
info:     linux-pipewire.so
info:     linux-jack.so
info:     linux-capture.so
info:     image-source.so
info:     frontend-tools.so
info:     decklink-output-ui.so
info:     decklink-captions.so
info: ---------------------------------
info: ==== Startup complete ===============================================
warning: signal_handler_connect: signal 'file_changed' not found

Kooha console traceback:
2023-10-09T12:07:32.793609Z  INFO kooha::application: Kooha (io.github.seadve.Kooha)
2023-10-09T12:07:32.793619Z  INFO kooha::application: Version: 2.2.4 ()
2023-10-09T12:07:32.793623Z  INFO kooha::application: Datadir: /app/share/kooha
2023-10-09T12:07:36.919423Z ERROR kooha::window: Failed to start recording

Caused by:
    0: Check out It Doesn't Work page for help.
    1: Failed to create ScreencastSession
    2: Failed to create session
    3: Failed to call `CreateSession` with parameters: Variant { ptr: 0x55e2b29d8360, type: VariantTy { inner: "(a{sv})" }, value: "({'session_handle_token': <'kooha_0'>, 'handle_token': <'kooha_1'>},)" }
    4: GDBus.Error:org.freedesktop.DBus.Error.UnknownMethod: No such interface “org.freedesktop.portal.ScreenCast” on object at path /org/freedesktop/portal/desktop

Journalctl
Oct 09 06:04:00 chimeraos systemd[839]: Started app-flatpak-com.obsproject.Studio-133029.scope.
Oct 09 06:04:01 chimeraos gnome-shell[3326]: Invalid window geometry for xdg_surface@38. Ignoring for now, but this will result in client termination in the future.
Oct 09 06:06:56 chimeraos polkitd[798]: Registered Authentication Agent for unix-process:133202:2390530 (system bus name :1.4820 [flatpak list], object path /org/freedesktop/PolicyKit1/AuthenticationAgent, locale en_US.UTF-8)
Oct 09 06:06:56 chimeraos polkitd[798]: Unregistered Authentication Agent for unix-process:133202:2390530 (system bus name :1.4820, object path /org/freedesktop/PolicyKit1/AuthenticationAgent, locale en_US.UTF-8)
Oct 09 06:06:59 chimeraos polkitd[798]: Registered Authentication Agent for unix-process:133212:2390823 (system bus name :1.4821 [flatpak list], object path /org/freedesktop/PolicyKit1/AuthenticationAgent, locale en_US.UTF-8)
Oct 09 06:06:59 chimeraos polkitd[798]: Unregistered Authentication Agent for unix-process:133212:2390823 (system bus name :1.4821, object path /org/freedesktop/PolicyKit1/AuthenticationAgent, locale en_US.UTF-8)
Oct 09 06:07:32 chimeraos systemd[839]: Started app-flatpak-io.github.seadve.Kooha-133236.scope.
Oct 09 06:07:36 chimeraos gnome-shell[3326]: meta_window_set_stack_position_no_sync: assertion 'window->stack_position >= 0' failed
Oct 09 06:09:40 chimeraos dbus-daemon[586]: [system] Activating via systemd: service name='org.freedesktop.home1' unit='dbus-org.freedesktop.home1.service' requested by ':1.4823' (uid=0 pid=133359 comm="sudo journalctl")
Oct 09 06:09:40 chimeraos dbus-daemon[586]: [system] Activation via systemd failed for unit 'dbus-org.freedesktop.home1.service': Unit dbus-org.freedesktop.home1.service not found.

This actually applies to all Pipewire recording applications. OBS Studio Flatpak is non-working, Kooha is as well. Likely missing deps or misconfigured core files?

If flatpak isn't working then this is certainly something to look into. While we don't intend to offer a full desktop experience, we don't want to inhibit users from expanding capabilities on their own through flatpaks. I don't do much screen capture myself so I'm not sure where to begin looking. Do you have any leads on where the pipewire stack is failing here?

If flatpak isn't working then this is certainly something to look into. While we don't intend to offer a full desktop experience, we don't want to inhibit users from expanding capabilities on their own through flatpaks. I don't do much screen capture myself so I'm not sure where to begin looking. Do you have any leads on where the pipewire stack is failing here?

@pastaq Solved! You guys forgot to enable the pipewire.service. To fix this, do the following:
$ systemctl enable --user pipewire.service
$ systemctl start --user pipewire.service

You will immediately get the screen recording option back in app programs, including the in-built GNOME screen recorder!

Odd. We use pipewire for all audio. Anyway, we can add it to the manifest if it drives this problem
https://github.com/ChimeraOS/chimeraos/blob/e31f8c3a17c73a274acc7916dab6007b66f6a397/manifest#L205


Bar code support

Hi Eric,
I have used your package for a small test project, and I have massive respect as it worked for me brilliantly.
I'd like to request support to detect and decode Bar codes as currently it doesn't. The pyzbar decoding inherently allows to decode bar codes.
I'm assuming the thought might have crossed your mind as well. right?

Hi!
Yes, it crossed my mind, but in the close future plans are to focus only on QRs.
That's because, in order to extract precise information like QR corners, rotation, transformation matrices... I'll have to rely in very specific QR features and characteristics.
Trying to go for additional data types, would decrease the detection-decoding performance and would limit the additional information about QRs that could be extracted.


chore(ci): checkout v2 uses deprecated node version

Running the test action in this repository currently results in a warning:

Instead of changing the default node version in the v2 checkout action, we can also just upgrade to v3.

Seems smart; I'll also turn on dependabot for this repo since we if we can avoid having to do this manually I think we should


Test pr ruty

/verified

/verified

/unverified

/approve

/verified

/unapprove

/unverified

/verified

/unverified

/approve

/unapprove

/approve

/verified

/unapprove

/approve

/verified

/approve

/unverified

/verified

/verified

/approve

/unverified


Question: Is there a working example?

Hey,
I'm currently trying to generate Azure Functions from an OpenApi Spec. But after executing the following command nothing gets generated (at least I can't see a new folder or any new created files):
autorest --azure-functions-typescript  --input-file:covid-apis.json --output-folder:./generated-azfunctions --verbose

As you can see, I tried it with the example covid-apis.json spec from the test/data/service-description-jsons folder.
I installed autorest via npm install -g autorest.
Does this module even work? This public announcement says TS is supported: https://azure.microsoft.com/en-us/updates/generate-a-new-function-app-from-an-openapi-specification/
Full output:
autorest --azure-functions-typescript  --input-file:covid-apis.json --output-folder:./generated-azfunctions --verbose
AutoRest code generation utility [cli version: 3.3.2; node: v12.19.1]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
No configuration found at 'file:///Users/wiedmannf/git/github.com/azure/autorest.azure-functions-typescript/test/data/service-description-jsons/'.
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/default-configuration.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/inspect.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/directives.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/pipeline.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/loader-openapi.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/miscellaneous.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-powershell.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-azureresourceschema.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-csharp.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-go.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-java.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-nodejs.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-php.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-python.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-ruby.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-typescript.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-validators.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-az.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-terraform.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/plugin-azure-functions.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/graphs.md'
> Including configuration file 'file:///Users/wiedmannf/.nvm/versions/node/v12.19.1/lib/node_modules/autorest/dist/resources/help-configuration.md'
> Loading AutoRest extension '@autorest/azure-functions-typescript' (latest->0.0.1-preview)
> Including extension configuration file 'file:///Users/wiedmannf/.autorest/@autorest_azure-functions-typescript<EMAIL_ADDRESS>> Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_azure-functions-typescript<EMAIL_ADDRESS>> Loading AutoRest extension '@autorest/modelerfour' (4.15.396->4.15.396)
> Including extension configuration file 'file:///Users/wiedmannf/.autorest/@autorest_modelerfour<EMAIL_ADDRESS>> Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_modelerfour<EMAIL_ADDRESS>NOTE: AutoRest core version selected from configuration: 3.0.6302.
   Loading AutoRest core      '/Users/wiedmannf/.autorest/@autorest_core@3.0.6302/node_modules/@autorest/core/dist' (3.0.6302)
No configuration found at 'file:///Users/wiedmannf/git/github.com/azure/autorest.azure-functions-typescript/test/data/service-description-jsons/'.
   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>   Including configuration file 'file:///Users/wiedmannf/.autorest/@autorest_core<EMAIL_ADDRESS>VERBOSE [0.9 s]: Reading OpenAPI 2.0 file file:///Users/wiedmannf/git/github.com/azure/autorest.azure-functions-typescript/test/data/service-description-jsons/covid-apis.json
[1.13 s] Generation Complete
autorest.azure-functions-typescript/test/da

I second this... the issue is different but I too have not been able to successfully generate Azure functions from this tool. I get the following error:
AutoRest code generation utility [cli version: 3.5.1; node: v14.18.2]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
SourceMapConsumer.initialize is a no-op when running in node.js
info    |    Loading AutoRest core      '/Users/nick/.autorest/@autorestcore@3.7.4/nodemodules/@autorest/core/dist' (3.7.4)
/Users/nick/.autorest/@autorest_azure-functions-typescript@0.0.1-preview/node_modules/@azure-tools/codemodel/dist/model/yaml-schema.js:38
exports.codeModelSchema = js_yaml_1.Schema.create(js_yaml_1.DEFAULT_SAFE_SCHEMA, [
                                           ^

TypeError: js_yaml_1.Schema.create is not a function
    at Object.<anonymous> (/Users/nick/.autorest/@autorest_azure-functions-typescript@0.0.1-preview/node_modules/@azure-tools/codemodel/dist/model/yaml-schema.js:38:44)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at require (internal/modules/cjs/helpers.js:93:18)
    at Object.<anonymous> (/Users/nick/.autorest/@autorest_azure-functions-typescript@0.0.1-preview/node_modules/@azure-tools/codemodel/dist/exports.js:42:14)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)

Same issue as @NickRees91 above.
TypeError: js_yaml_1.Schema.create is not a function
Have tried multiple JSON/YAML schemas which conform to OpenAPI specs.

The issue, I think, is caused by js-yaml:
js-yaml v3:
let schema3 = yaml.Schema.create(yaml.DEFAULT_SAFE_SCHEMA, [ customTags ])
js-yaml v4:
let schema3 = yaml.DEFAULT_SCHEMA.extend([ customTags ])
Looks like it has been built against js-yaml v3 (and is expecting Schema.create), but Autorest is pulling in 4.X of js-yaml which doesn't have that method. No idea how to resolve though.

Made a PR #3 that addresses this, got it to work on my end. You can pull from it in your local as well to use it.


Fix broken imports statements for AnakinParser

Resolve naming issues resulting from the refactoring.

Pull Request Test Coverage Report for Build 115

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.0%) to 72.175%

Totals

Change from base Build 110:
-1.0%

Covered Lines:
1437

Relevant Lines:
1991

💛  - Coveralls


add doc for alert insight agent

Description
Add doc for alert insight agent
Issues Resolved
Add doc for alert insight agent
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Need to add verification on insight API as well. You can see its API definition here: https://github.com/opensearch-project/dashboards-assistant/blob/50fef5c06993b110367314400f857042ef646568/server/routes/summary_routes.ts#L83, and write related test like summary API.

Ack. Updated.

Pinging @ruanyl for CI check and merge.


tablesnap crashes with md5 mismatch error from S3

Hi There!
I have tablesnap running on ubuntu with Cassandra 3.7, and it is crashing frequently with the below error (not always on the same file).
Is this normal?
Thank you for your help!
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]: Traceback (most recent call last):
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/bin/tablesnap", line 133, in worker
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     self.upload_sstable(bucket, keyname, f)
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/bin/tablesnap", line 394, in upload_sstable
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     encrypt_key=self.with_sse)
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/lib/python2.7/site-packages/boto/s3/key.py", line 1374, in set_contents_from_filename
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     encrypt_key=encrypt_key)
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/lib/python2.7/site-packages/boto/s3/key.py", line 1305, in set_contents_from_file
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     chunked_transfer=chunked_transfer, size=size)
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/lib/python2.7/site-packages/boto/s3/key.py", line 762, in send_file
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     chunked_transfer=chunked_transfer, size=size)
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/lib/python2.7/site-packages/boto/s3/key.py", line 963, in _send_file_internal
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     query_args=query_args
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/lib/python2.7/site-packages/boto/s3/connection.py", line 671, in make_request
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     retry_handler=retry_handler
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/lib/python2.7/site-packages/boto/connection.py", line 1071, in make_request
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     retry_handler=retry_handler)
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/lib/python2.7/site-packages/boto/connection.py", line 940, in _mexe
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     request.body, request.headers)
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:   File "/home/ubuntu/.local/lib/python2.7/site-packages/boto/s3/key.py", line 896, in sender
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]:     response.status, response.reason, body)
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]: S3ResponseError: S3ResponseError: 400 Bad Request
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]: <?xml version="1.0" encoding="UTF-8"?>
Nov 16 19:18:08 ip-10-0-0-29 tablesnap[14931]: <Error>BadDigest<Message>The Content-MD5 you specified did not match what we received.<

Seeing the same issue on DSE 4.8.10 (cassandra 2.1.15) and tablesnap 1.2.1
2017-11-15 01:34:07,096 CRITICAL Failed uploading /data/lib/cassandra/data/dev/locking/dev-locking-tmp-ka-383139-Data.db. Aborting.
Traceback (most recent call last):
  File "/usr/local/bin/tablesnap", line 133, in worker
    self.upload_sstable(bucket, keyname, f)
  File "/usr/local/bin/tablesnap", line 394, in upload_sstable
    encrypt_key=self.with_sse)
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/key.py", line 1362, in set_contents_from_filename
    encrypt_key=encrypt_key)
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/key.py", line 1293, in set_contents_from_file
    chunked_transfer=chunked_transfer, size=size)
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/key.py", line 750, in send_file
    chunked_transfer=chunked_transfer, size=size)
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/key.py", line 951, in _send_file_internal
    query_args=query_args
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/connection.py", line 668, in make_request
    retry_handler=retry_handler
  File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1071, in make_request
    retry_handler=retry_handler)
  File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 940, in _mexe
    request.body, request.headers)
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/key.py", line 884, in sender
    response.status, response.reason, body)
S3ResponseError: S3ResponseError: 400 Bad Request
<?xml version="1.0" encoding="UTF-8"?>
<Error>BadDigest<Message>The Content-MD5 you specified did not match what we received.</Message><ExpectedDigest>b762d0312863a16818d256ce3ad44dda</ExpectedDigest><CalculatedDigest>a8tB0+rMlQyxcHJLIbcP6A==</CalculatedDigest><RequestId>29890293DF4E2B84</RequestId><HostId>/rF6FmY2Q8biPxkIYFxbIw6hQagwwSQ5UWbqi8J1Eat33DBQ6Qell4QdLcIjmX192SbaFzuAex8=</HostId></Error>

Killed

@tightly-clutched I dug into this some more and it seems like this is being caused by the pyinotify library that tablesnap uses to identify when to copy files to S3. When you start tablesnap with the --auto-add argument, it passes that to pyinotify, and because of some side effect in pyinotify, that automatically subscribes the watcher to the IN_CREATE, event, which makes tablesnap start copying the files to S3 as soon as they are created.
So I think the fix is tablesnap needs to ignore IN_CREATE events unless you are specifically trying to watch for them (via the --listen-events argument). I'm going to try to submit a PR for this :man_shrugging:


Scrolling does not reset timer for the soft key bar

Scrolling the soft key bar does not reset the timer, so it will disappear when you're scrolling.

@alescdb I should have tested this during review :)

Yep that why I increased the autohide timeout :-)
I'll try to implement a onTouchEvent but they may be a conflict with repeatable keys.
Also when using the last git-master-release on Marshmallow preview 3, repeatable keys have a strange behaviour in some circontances.

Probably related:
https://github.com/connectbot/connectbot/issues/159
Regards


memory leak in mbedtls using net_app DTLS client

I am using the DTLS client in zephyr, and I found there is memory leak when run the following flow mutli times:

net_app_init_udp_client
net_app_set_cb
net_app_client_tls
net_app_connect
net_app_close
net_app_release
go to step1 and repeat

Seems the mebedtls_ssl_xxx_init functions in _net_app_tls_init function doesn't release the heap when calling net_app_release

I think I am also running into this. If I call mqtt_connect a couple of times I end up getting an error in my setup cert callback. Specifically, mbedtls_x509_crt_parse_der fails with MBEDTLS_ERR_X509_ALLOC_FAILED.
My callback is straight out of the mqtt example file.

Hello,
On 06/28/2018 11:50 PM, Zhenke Ma wrote:

I am using the DTLS client in zephyr, and I found there is memory leak
when run the following flow mutli times:

net_app_init_udp_client
net_app_set_cb
net_app_client_tls
net_app_connect
net_app_close
net_app_release
go to step1 and repeat

Seems the mebedtls_ssl_xxx_init functions in _net_app_tls_init
function doesn't release the heap when calling net_app_release

I have a WIP patchset which includes this hack to fix the leak you're
describing.  I need some time to submit an actual fix:
https://github.com/zephyrproject-rtos/zephyr/pull/8570/commits/82c5d82a09bcee33351a8f1ecc5aed7f490d4c63#diff-88ec0070db0157cb49aeb6609e210f1aR3883
Not sure if knowing where the leak is helps, but always good to get more
information.
NOTE: Added Jukka for reference

Mike

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zephyrproject-rtos/zephyr/issues/8631, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AI7BB8loGMuLiJ6JKwyGTaN4C2b5iHMZks5uBc5IgaJpZM4U8hm8.

Similar comment to what was posted to https://github.com/zephyrproject-rtos/zephyr/issues/8746 :
Recently, DTLS socket support was merged: https://github.com/zephyrproject-rtos/zephyr/pull/9338 , and that's going to be the new official API for (D)TLS usage. You are thus recommended to switch to it whenever possible. You may find that older net_app DTLS client is in "maintenance mode" (that actually depends on developers' time availability, and AFAIK, they're fully occupied). I thus lower priority of this ticket.

net-app is gone, so closing this issue


Derive Lines and Piece Stats from local computation instead of OCR

Context
NestTrisChamps currently "reads" the value of lines and piece counters on the right., This is done by reading 3 digits per field, and in the case of lines, NestrisChamps assumes the line count will be below 500 for the majority of players, so it limits the detection of the first piece stats digits to be in (0, 1, 2, 3, 4)
As it turns out, Nestris players are now attempting to reach the point where the game breaks, like EricICX in his 6.4M game. That game had over 1400 lines, and several piece types above 500.
To capture values like these, NestrisChamps OCR would need to be upgraded to account for more characters (read the piece stats up to 600 or more, read lines, beyond 1600. Reading a greater character range takes more CPU cycle, and this only benefits a few top-level player at the detriment of the vast majority of NestrisChamps users.
An alternative to increasing the range of OCR is to realize that both lines and  pieces stats are values that increases continuously during a game. lines may increase by 1, 2, 3, 4, and piece stats should increase only one at a time.
It is therefore possible to derive the values by computation rather than OCR. By reading and comparing only the unit digit of the values, and comparing with the previously read value, a calculation to derive the new value in an ever increasing way is easy.
By doing this, OCR of the the full values can be done only at the beginning of a game, and for manageable ranges, say
This PR implements the optimizations mentioned above:

reduce the OCR range of the hundreds and tens digits for lines and piece stats, to just 0, 1, 2
produces final value from computation by inspecting changes to the units digit alone

Risk and drawbacks

Because OCR reports what it sees, if calibration is bad, and some values are broken, they will eventually self correct as new values are read correctly later. With computation, self-correction will be gone
NestrisChamps doesn't have a calibration mode distinct from the live capture of the game. Having the UI show computed value rather than OCR value might show values that are not expected, even thought he calibration is perfect. If a player calibrate from a video file, rather than a live stream, jumping the video playhead will not be understood by nestrischamps as a jump, and the computed value will be displayed incorrectly.

For example, say nestrischamps currently stores lines as 023, if the player jumps to say lines 119, nestrischamps will notice the unit value changes, but within the same tens cycle, and the computed result will display 039, rather than 119.
For normal games captured from a continuous video stream, that will not be a problem, but during calibration, players might be confused by the incorrect values.

One way to address the calibration display concern might be to have a 2-stage extraction of the lines and piece stats.
NestrisChamps users are expected to calibrate with a game at level 0, where lines and pieces stats will be very low (or or 1). deriving liens and pieces stats by computation is only necessary for game-breaking kind of games.
I'm thinking I will amend the code to:

read lines and piece stats from 000 to 299
Within a given game, if the value is less than 200, return the value as-is (i.e. read from OCR)
Within a game, when 200 is crossed, shift to derived mode, where only the units of the value is inspected for change.

The process resets at every game.
At calibration, even from a seekable video, we can assume a player will stay below level 29 (kill screen). With that assumption, lines will be below 230, and piece counters will mostly be below 100, so the behaviour described above should work fine. (See example values from EricICX's game for reference:

Perhaps if we want to maintain read-from-ocr for most normal games all the way to kill screen, maybe the below would be even better:

read lines from 000 to 399
read piece counters from 000 to 199

Within a given game

if lines is less than 300, return the value as-is (i.e. read from OCR).
When lines crosses 300, switch to compute mode
if piece counters are less than 100, return the value as-is (i.e. read from OCR).
When any piece counters crosses 100, switch to compute mode

Note that implementing the above will be cheaper than current nestrischamps which has a broader (and therefore more expensive) OCR range of 000 to 999 for lines, and 000 to 499 for piece counters.


Timeouts where headers have been sent are not included in outlierDetection

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
Whilst investigating https://github.com/istio/istio/issues/52879; i noticed that outlier detection was not working in that particular use case.
It looks to be because the response code was 200UT, because the destination sent the HTTP headers, but did not complete the body.  I would assume this is because its response code that's being considered, not response flags.
A UT (upstream timeout) is still an upstream failure in my eyes, and should be captured by outlier detection, and that upstream pod should be evicted as per the eviction policy.
Because of this; the pod was never evicted and errors continued.
Version
1.23

Additional Information
No response

Interesting....yeah this feels like an envoy behavioral decision; I'll try and find time to investigate


[GUI] Invalid attribute value: the text was not translated

The text was not translated into Portuguese.
Steps to reproduce the problem:

Click "Novo Modelo"
Click "Novo Atributo"
Select Attribute Type: "Valor Estático"
Select Value Type
Fill in value with invalid data
Click "Adicionar"

booleano

geo

ponto flutuante

inteiro

Affected Version: 58.2-20190321

Test OK.
The texts have been translated.

boolean:

geo:

float:

integer:

empty text:

Tested on Version: 62.3-20190603


Closing modal's with button in v1.4.0

May i know the css class to apply to modal dialog seconday button so that user can close it with cancel button too and X at the top. Right now i can do that if i add close css class for the button but this messes with the styles of btn css class. What data attribute do i add or css class do i apply for this functionality. I am using v1.4.0 ( I could update but will need a week before suggesting the update to project manager)

In 1.4 you had to do it manually with js - you could try asking in the mailing list if you need more help. thanks!


Switch from windows_package to package resource to prevent warnings on each run

Cookbook version
5.1.0
Chef-client version
PS C:\Windows\system32> chef-client -v
Chef: 12.16.42
Platform Details
Windows 2012R2
Scenario:
Getting a warning. Probably just need to switch 'windows_package' resources with plain old 'package'
[2016-11-14T10:07:17-05:00] WARN: Please use the package resource available in Chef Client 12.6+.
windows_package will be removed in the next major version release
of the Windows cookbook.

Steps to Reproduce:
I'm using the powershell5 recipe, but looks to be used also in powershell3.rb, powershell4.rb, as well as powershell5.rb
Expected Result:
Just needs update before next major release of Chef

This is something we'll probably do after Chef 13 is released in April. It drops support for Chef < 12.6, which is still pretty popular on the Windows side of things.

This issue can be closed as we are using built-in windows_package resource from 12.6+.

@phwelo v6.0 version of powershell cookbook fixes this issue.


Allow using ensure in a typed: strong file

Input
→ View on sorbet.run
# typed: strong

begin
  puts(1)
  puts(2)
ensure
  puts
end

Observed output
editor.rb:4: Conditional branch on T.untyped https://srb.help/7018
     4 |  puts(1)
          ^
  Got T.untyped originating from:
    editor.rb:4:
     4 |  puts(1)
          ^
  Note:
    Support for typed: strong is minimal. Consider using typed: strict instead.

editor.rb:4: Conditional branch on T.untyped https://srb.help/7018
     4 |  puts(1)
          ^
  Got T.untyped originating from:
    editor.rb:4:
     4 |  puts(1)
          ^
  Note:
    Support for typed: strong is minimal. Consider using typed: strict instead.
Errors: 2

Expected behavior

The "conditional branch" here is the synthetic one that Sorbet creates to support ensure (check the CFG).

Another test case:
Input
→ View on sorbet.run
# typed: strong

begin
  spot_interruption_thread = T.let(nil, T.nilable(Thread))
  if [true, false].sample
    spot_interruption_thread = Thread.new { puts('hello') }
  end
  puts('here')
ensure
  if spot_interruption_thread
    spot_interruption_thread.kill
  end
end

Observed output
editor.rb:4: Conditional branch on T.untyped https://srb.help/7018
     4 |  spot_interruption_thread = T.let(nil, T.nilable(Thread))
          ^
  Got T.untyped originating from:
    editor.rb:4:
     4 |  spot_interruption_thread = T.let(nil, T.nilable(Thread))
          ^
  Note:
    Support for typed: strong is minimal. Consider using typed: strict instead.

editor.rb:4: Conditional branch on T.untyped https://srb.help/7018
     4 |  spot_interruption_thread = T.let(nil, T.nilable(Thread))
          ^
  Got T.untyped originating from:
    editor.rb:4:
     4 |  spot_interruption_thread = T.let(nil, T.nilable(Thread))
          ^
  Note:
    Support for typed: strong is minimal. Consider using typed: strict instead.
Errors: 2

The typed: strong directive has been replaced with typed: strict as recommended.
The puts statement with an empty parentheses puts() has been replaced with puts('') to output an empty line.
The code is surrounded by begin and ensure blocks to ensure that the empty line is printed regardless of any exceptions that may occur.
By making these changes, the code should no longer produce the error messages related to the conditional branch on an untyped value.

typed: strong
begin
puts(1)
puts(2)
ensure
puts('')
end


refactor(stream): refactor non-append-only Top-N with cache

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
This implementation assumes that the value of offset and limit provided by user is small.
We use three BTreeMap (low, mid, high) to store rows in the range of [0, offset), [offset, offset+limit), [offset+limit, +Inf) respectively.
Currently the capacity of high cache is set to 1024 statically. Because we assume offset and limit is not large, the 1024 is a reasonable capacity for cache.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)
close #3999

I think it would be easier to implement a single heap (or BTreeMap) of size offset + limit?

Both LGTM

BTW Let's remove the old version of TopN after this PR got merged ☺

I think it would be easier to implement a single heap (or BTreeMap) of size offset + limit?
Use a single heap (e.g. BinaryHeap) is not possible to handle delete events. And if use a single BTreeMap, it needs to scan the cache every time for each input row to generate diff events. There are also corner cases that needed to be handled. The advantage of this implementation is that it is more or less an in-memory version of previous three state table version, in other words I can stand on the shoulder of giant.

I think it would be easier to implement a single heap (or BTreeMap) of size offset + limit?

Use a single heap (e.g. BinaryHeap) is not possible to handle delete events. And if use a single BTreeMap, it needs to scan the cache every time for each input row to generate diff events. There are also corner cases that needed to be handled. The advantage of this implementation is that it is more or less an in-memory version of previous three state table version, in other words I can stand on the shoulder of giant.


Release 1.0.2

@NikolayPianikov We don't usually do issues for releases, since there is really nothing to say about it. Generally the milestone gives any necessary info about the purpose of the release, if necessary.

@CharliePoole ok, I will take note of this


Event API documentation

Adds initial event API documentation. It outlines:

Registering listeners
HandlerLists
Custom Events

It's possible I got some implementation details incorrect... :)
I also started something of a "folder" structure with this, lmk if your feel it should be different

Yeah... fixed my bad english


[FLINK-2984] [ml] Extend libSVM file format support

Currently, FlinkML libSVM file reader cannot read some datasets (such as splice-site) in libSVM official site. This PR extends libSVM file format to liblinear equivalent. To avoid re-compiling regex pattern in every line, I changed the mapper from scala lambda function to RichFlatMapFunction.

Changes look good. +1 for merging.

Thanks for review @tillrohrmann! I'll merge this now.


Choose Cache Root window is not fully visible

Playnite: 10.34
NowPlaying: 1.4.1
Theme: Default
Theme API: 2.6.0
Choose Cache Root window button at the bottom are cut.

There is an error in Playnite.log:
ERROR|GameMenu:Failed to get menu items from plugin NowPlaying Game Cacher
System.InvalidOperationException: Sequence contains no elements
   at System.Linq.Enumerable.First[TSource](IEnumerable`1 source)
   at NowPlaying.NowPlaying.GetGameMenuItems(GetGameMenuItemsArgs args) in C:\Dev\GitHub\Playnite-NowPlaying\source\NowPlaying.cs:line 825
   at Playnite.DesktopApp.Controls.GameMenu.AddExtensionItems(List`1 games) in C:\projects\playnite\source\Playnite.DesktopApp\Controls\Menus\GameMenu.cs:line 586

Thanks. I'm reworking windows to better accommodate larger fonts. Stay tuned.

HI, the latest release should address this problem for you (v1.4.2). Please check it out.
--gr

Looks great now. Thanks for the fix!

Sure thing. closing this issue. --gr.


Count property name collision

Bug description

With the new introduction of count in prisma 2.2.0, I do now have a naming collision on the count property

How to reproduce
make a schema.prisma file with a model having a count property

model Example {
  id        String                      @id @default(cuid())
  count     Int
}

Then generate
prisma generate

Expected behavior
Since now, I had no conflict with the count. But the new agregation did type the count and creates 2 properties with the same name. I would expect prisma not to have too many reserved properties
Prisma information
see above
Environment & setup

OS: Mac OS
Database: PostgreSQL
Node.js version: v15.11.0
Prisma version:

prisma               : 2.20.1
@prisma/client       : 2.20.1
Current platform     : darwin
Query Engine         : query-engine 60ba6551f29b17d7d6ce479e5733c70d9c00860e (at node_modules/@prisma/engines/query-engine-darwin)
Migration Engine     : migration-engine-cli 60ba6551f29b17d7d6ce479e5733c70d9c00860e (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine : introspection-core 60ba6551f29b17d7d6ce479e5733c70d9c00860e (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary        : prisma-fmt 60ba6551f29b17d7d6ce479e5733c70d9c00860e (at node_modules/@prisma/engines/prisma-fmt-darwin)
Default Engines Hash : 60ba6551f29b17d7d6ce479e5733c70d9c00860e
Studio               : 0.365.0

Thanks for opening this issue. We fixed this in https://github.com/prisma/prisma/releases/tag/2.23.0.


UnboundLocalError: local variable 'idxs' referenced before assignment

Hello, when I run bash ./run_code.sh, the following error has occurred, and how can I solve this problem?
Traceback (most recent call last):
File "main.py", line 497, in
main(args)
File "main.py", line 448, in main
args=args
File "/content/evit/engine.py", line 56, in train_one_epoch
outputs = model(samples, keep_rate)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/parallel/distributed.py", line 963, in forward
output = self.module(*inputs[0], **kwargs[0])
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/content/evit/evit.py", line 442, in forward
x, _, idxs = self.forward_features(x, keep_rate, tokens, get_idx)
File "/content/evit/evit.py", line 437, in forward_features
return self.pre_logits(x[:, 0]), left_tokens, idxs
UnboundLocalError: local variable 'idxs' referenced before assignment

Hi, the bug was introduced accidentally by the visualization code that was newly added yesterday. I just fixed it. The code should be runnable now. Thank you for reporting this bug!


username , password, port

Hi, jlesage, i just downloaded the image and made it run with your default docker command. I want to build a spring application that will pull data from the filezilla server. I tried to connect to file filezilla gui but i don't know what the username and password is for this specific host. I tried to add a new server with host: localhost port:5800 and left the username and password emtpy but that didnt work out. How should i connect to the container and upload/download files from the container using the localhost:5800 gui of filezilla? What parameters do i need?
I cannot connect to the ftpserver using host:localhost or <IP_ADDRESS> username/password empty port:5800(tried also 20 and 21 ) ??

What is the credentials for logging into the client?

There is no credentials, unless you configured the container with the VNC_PASSWORD environment variable.


⚠ Bofrost.lu has degraded performance

In 8a5428e, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1089 ms

Resolved: Bofrost.lu performance has improved in 14897aa after 10 minutes.


Update Readme.md

I would like to let u know that I would love to see this project have a better and updated readme.md .
Pls assign this to me so that I can work on this on with discussion and suggestions .

hey @XxThunderBlastxX  I would be more than happy enough to contribute to your project by making  a proper readme file , please assign me this issue under JWOC.


作者你好，你肤色检测只用了150张图片训练，就能得到理想的效果？

作者你好，你肤色检测只用了150张图片训练，就能得到理想的效果？

How can you release your code and pretrained weight just trained on 150 images, the results is really bad!!!

Which I said is the skin segmentation model!!

Hey - I'd suggest trying out the accurate model first and checking your input resolution. I've managed to get decent results with that. Please can you post the images you're trying to run on.

OK, I will check it again, thank you!

I want to know which resolution did you train your skin model?

The model is trained on 480x480 crops; and people take up the majority of those images. If you resize your images to VGA (640x480) it should perform well.
https://github.com/WillBrennan/SemanticSegmentation/blob/master/semantic_segmentation/dataset.py#L50

OK, thank you very much!!!


Unexpected token

with return I get error

by removing it - no errors, and it works

By analyzing the blame information on this pull request, we identified @hramos and @scarlac to be potential reviewers.

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

There needs to be a return value... this is not the right fix.

https://facebook.github.io/react-native/docs/navigator.html there is no return in renderScene :)

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

There's an implicit return in those examples. Looking at your code the problem is that you added an explicit return when there should be an implicit one.

That is not my code it is code from example. And it does not work with return...

Your code and the code from the example are different because of the curly brace, take a closer look, read up on JS if you don't see the difference.

It is not like I don't believe in you :)  - but https://www.youtube.com/watch?v=MuG14Ke62YE


Allow compilation outside of KoLmafia directory.

This change should allow keeping this directory outside of the KoLmafia directory; it also removes code that ignores import errors when an imported file can't be found.

Thank you! This was on my to-do list.


Issue with UrlHelper

Hi,
I try to generate a link inside a template, using:
@Url.RouteUrl(<routename>)

RouteUrl returns null.
Thanks

Hi, Please assist with more info:
.NET Version:
Project Type:
Razor.Templating.Core Version:
Expected Output:
Sample repo to reproduce(if possible):

Hi @cadilhac, I had a look. @Url.RouteUrl will work only in MVC app as it requires the routing features which is not supported by the library.
A quick workaround is to call @Url.RouteUrl from the controller action method & then pass it to the ViewModel.
Hope it helps. Thanks.

Yes. Sorry, I had not been able to create a repro yet. And yes, I'm using MVC and was already creating the link in the controller as a workaround. Sad that you can't solve it. Cheers.

I've updated the documentation that UrlHelper is not supported. Hence closing. Please feel to reopen if there are any suggestions.
Thanks :)


Swagger json ver 3 does not included produces, and UI does not have Response Content Type Selection

By default the swagger output is version 3.0.1, and it doesn't include the "produces" array in the json output.  Instead, it includes the possible response content-types in the responses/content property.  However the swagger UI seems to require the "produces" array to display and populate the "Response Content Type" selector so that I can specify, via the UI, what content-type I want the response to be.  Is this intended behavior, because it seems like I need to revert back to V2 using the SerializeAsV2 option to get that functionality back.

Please provide a minimal reproducible example as a GitHub repository that demonstrates the issue you're experiencing so we can look into this further.


Adds rake task to register a local PARTNER_RECAP file into the database

Fixes #1422

@christinach Made the changes that you requested. Notice that I left it with the defaults for RECAP files if no DUMP_TYPE and DUMP_FILE_TYPE are indicated.

@christinach I am not sure I follow this: "Also a mapping between DUMP_TYPE and DUMP_FILE_TYPE as a comment in the beginning of the rake task would be good."


fix(logger): add setLevel function to set level programmatically

Issue number: #2303
Summary
Changes
When changing the logging level of a Logger via code, the logging level must be changed. With the current implementation, this was not happening because it was modifying the superclass and not the Logger instance. This PR aims to fix that.
User experience

Please share what the user experience looks like before and after this change

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[ ] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (653db45) 97.46% compared to head (56cf8d1) 97.47%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2320   +/-   ##
========================================
  Coverage    97.46%   97.47%
========================================
  Files          149      150    +1
  Lines         6901     6925   +24
  Branches       509      509
========================================
+ Hits          6726     6750   +24
  Misses         137      137
  Partials        38       38

Impacted Files
Coverage Δ

aws_lambda_powertools/logging/logger.py
99.42% <100.00%> (+0.01%)
:arrow_up:

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

NOTE for future when we do this properly...

Remove subclassing from logging.Logger) and subclass from object instead - already commented on why
Override __getattribute__ to proxy to self._logger -- had to remove it before, and add __getattr__ under TYPE_CHECKING due to Mypy not detecting non-existent attributes

    def __getattribute__(self, item):
        try:
            # Return actual item from our class first (e.g., append_keys, etc.)
            return object.__getattribute__(self, item)
        except AttributeError:
            # Return actual item from the underlying logger (e.g., setLevel, addHandler, etc.)
            return self._logger.__getattribute__(item)


Fix Appsignal when used with multiple DJ workers

Similar to the fix for Puma we need to let Appsignal know that the process has been forked so that it can re-establish connections. Unfortunately there is no good way to do this so we have to override the after_fork class method in an initializer to call Active Record's establish_connection as well as the Appsignal forked! method.
For some strange reason I was under the impression that the bin/delayed_job script was called 4 times when I wrote the previous fix in 6caec23 but on reflection that make no sense since it couldn't take advantage of copy-on-write memory saving.

LGTM


Fix Building-Locally.md and echoum.inx

I send a new PR to the develop branch instead of #886 and #888.
#886 simply solves the problem that occurs when copy-pasting commands.
#888 allows to build objects using Visual Studio 2002.

Actually, I'd prefer to NOT change echoum.inx. Add @NeoAdonis for comment.
Some people use the WDK sample as-is without any modification in production. That is bad, because if two people are doing this, their driver packages will conflict with each other's.
WDK tries to report such error early on. To do so, the samples are intentionally using the forbidden class name "Sample", forbidden service name like "echo". The sample now fails to build immediately. This forces the user to make some modification to the sample before publishing it as their own code.


Problem: Autoupdate does not work after login.

Example: Create agenda item.

works for me

Still buggy


function eth_workaround_eth_coinbase($val)

Hi @digitaldonkey ,
thanks for providing this interface and also the drupal/ethereum module!
I would like to contribute to this project too.
Currently I try to get the dev branch running on my environment but I have a problem, when I use the INFURA net because eth_coinbase is not supported by them.
I have found these workaround:
{
    // WORKAROUND: Catch a "405 Method Not Allowed'.
    if (isset($val['error']) && $val['error'] && $val['code'] == 405) {
        return '0x0000000000000000000000000000000000000000';
    } else {
        return $val;
    }
}

But as I see it they are not used at the moment, which breaks the drupal/ethereum module because ethereum-php throws an Exception which is not caught (Expected EthD20 at eth_coinbase (), couldn not be decoded. Value was: Array).
Where should this workaround be used? Can I help you with this? Would be happy to create a PR if you could point me into the right direction.
Thanks for your work!

Seems src/helpers/ethereum-client-workaround-helpers.php is not loaded.
--> https://github.com/digitaldonkey/ethereum-php/pull/14


fix: backButtonDispatcher not being null when routerConfig is provided

When using routerConfig an assertion fails in WidgetsApp.router.
This is caused by backButtonDispatcher defaulting to DefaultBackButtonDispatcher.
Pre-launch Checklist

[x] I have updated CHANGELOG.md with my changes
[x] I have run "dart format ." on the project
[x] I have added/updated relevant documentation

@bdlukaa this is required to use routerConfig

it sounds better

@bdlukaa is this what you had in mind?


python311Packages.torchmetrics: 1.2.1 -> 1.3.0

Description of changes
Changelog: https://github.com/Lightning-AI/torchmetrics/releases/tag/v1.3.0
cc @SomeoneSerge

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 280275 run on aarch64-linux 1

2 packages marked as broken and skipped:

piper-train
piper-train.dist

8 packages built:

python310Packages.pytorch-lightning
python310Packages.pytorch-lightning.dist
python310Packages.torchmetrics
python310Packages.torchmetrics.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist

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

10 packages marked as broken and skipped:

piper-train
piper-train.dist
python310Packages.elegy
python310Packages.elegy.dist
python310Packages.treex
python310Packages.treex.dist
python311Packages.elegy
python311Packages.elegy.dist
python311Packages.treex
python311Packages.treex.dist

8 packages built:

python310Packages.pytorch-lightning
python310Packages.pytorch-lightning.dist
python310Packages.torchmetrics
python310Packages.torchmetrics.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist


[Ruby] Add ruby_abi_version to exported symbols

The next Ruby version, 3.2, will have builtin ABI checking (see ruby/ruby#5474). This requires that the symbol ruby_abi_version is present in the shared object, otherwise the object fails to load.
For example, this is a small repro in Ruby 3.2:
$ ruby -Isrc/ruby/lib -e "require 'grpc'"
<internal:/home/peter/src/ruby/install/lib/ruby/3.2.0+0/rubygems/core_ext/kernel_require.rb>:85:in `require': /home/peter/src/grpc/src/ruby/lib/grpc/grpc_c.so: undefined symbol: ruby_abi_version - ruby_abi_version (LoadError)
	from <internal:/home/peter/src/ruby/install/lib/ruby/3.2.0+0/rubygems/core_ext/kernel_require.rb>:85:in `require'
	from /home/peter/src/grpc/src/ruby/lib/grpc/grpc.rb:22:in `<top (required)>'
	from /home/peter/src/grpc/src/ruby/lib/grpc.rb:19:in `require_relative'
	from /home/peter/src/grpc/src/ruby/lib/grpc.rb:19:in `<top (required)>'
	from <internal:/home/peter/src/ruby/install/lib/ruby/3.2.0+0/rubygems/core_ext/kernel_require.rb>:85:in `require'
	from <internal:/home/peter/src/ruby/install/lib/ruby/3.2.0+0/rubygems/core_ext/kernel_require.rb>:85:in `require'
	from -e:1:in `<main>'

@drfloob

@apolcyn any chance you could look at this?

Is this now mergeable? Is there any additional help needed?

C# macos failure is an infrastructural issue:
/bin/sh: /bin/sh: cannot execute binary file during compilation

cc @peterzhu2118 have any insight on https://github.com/grpc/grpc/issues/30976 ?

I opened #31151 which should fix the issue.

This change caused https://bugs.ruby-lang.org/issues/19082.

@apolcyn could you review the PR? This is affecting quite a few users.


🚀 clamp

this closes #29

 ✓ src/clamp.bench.ts > clamp performance 1298ms
     name               hz      min      max     mean      p75      p99     p995     p999     rme  samples
   · hidash clamp   112.50   8.7582  11.7683   8.8888   8.8400  11.7683  11.7683  11.7683  ±1.20%       57   fastest
   · lodash clamp  49.3294  19.8881  25.0428  20.2719  20.0420  25.0428  25.0428  25.0428  ±2.24%       25

  hidash clamp - src/clamp.bench.ts > clamp performance
    2.28x faster than lodash clamp

(저도 처음알았음)


second-order BK failure patch

This PR fixes the issue that when BK fails, the second-order perturbation solution triggers an error instead of returning an error message. I added a corresponding unit test as well.

CI passes.

I still don't understand why we need DifferenceEquations. Maybe discuss on slack?


Accept pending friendships

I want to be able to accept and ignore pending friendships, and I can't find a function to be able to do that anywhere in the documentation. I would be happy to contribute this in a PR, but I don't really know how to look at the requests the Android app makes. On the web though, it's with this endpoint:
POST https://www.instagram.com/web/friendships/{user_pk}/approve/
POST https://www.instagram.com/web/friendships/{user_pk}/ignore/

If trying to replicate in the app, you would have to have two accounts. One of them would need to be set to private, then the second would need to request to follow the private account. Then from the private account you would have a notification in your feed about the new request, and you could then accept or reject it there.

You're right, this hasn't been implemented yet. Thanks for the request. I will look at it this week!

Sorry for the delay, but this has now been implemented!
Check account_test.go for an example.

Thanks for taking this on! I really appreciate it.


Checking even and odd number - Ranjan Kumar Verma

I have a added a code to check even and odd number using C++. Please merge my pull request.

Could you change the path to /codes. Thank you!


Add AUR package to readme.

I created an AUR package for you (I prefer most things I install to be managed as packages for remembering what they are and to easily uninstall them). I added the location to the readme.

Thank you 👍


Remove SA1003 handling for tokens covered by other diagnostics

Fixes #1748

:+1: Looks good to me


Training datasets can not be downloaded

Training datasets can not be downloaded， can you share a new link ?

Thanks for your feedback. We will soon create a new link for the datasets

could you tell me what's the difference between MolNexTR and MolScribe?

Are the training data and augmentation methods different, or are the same data used but with different augmentation methods?

We implemented more advanced data augmentation algorithms and refined post-processing techniques for improved results.

We also use the different encoders; only decoder remains the same.


Windows PowerShell (Admin) on some computers with Windows 10 only Windows PowerShell work

Windows Terminal version
1.17.11461.0
Windows build number
10.0.19045.3393
Other Software
No response
Steps to reproduce
https://github.com/microsoft/terminal/assets/11423362/2be9454d-0b46-4baf-a4de-7a168b8536d4
Expected Behavior
Windows PowerShell (Admin) open Terminal Admin
Actual Behavior
Windows PowerShell (Admin) open poweshell admin

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #10276 for more discussion.
/dup #10276

Thanks for the warning, I'd looked for it but couldn't find it!

No worries - that's a wacky one that you'd have to know existed to be able to find it. Heck I knew it existed and still took me 10 minutes to find it 😛

No worries - that's a wacky one that you'd have to know existed to be able to find it. Heck I knew it existed and still took me 10 minutes to find it 😛

Are there any alternative solutions? The strange thing is that I have other computers just like it and it works as expected, same version all the same!

The strange thing is that I have other computers just like it and it works as expected

excuse me what 👀
The "default terminal" functionality definitely has never worked for elevated commandline applications. On Windows 11 the Win+X menu has "Terminal (Admin)" in it, which doesn't use the default terminal feature. Maybe that's what you've seen?

The strange thing is that I have other computers just like it and it works as expected

excuse me what 👀
The "default terminal" functionality definitely has never worked for elevated commandline applications. On Windows 11 the Win+X menu has "Terminal (Admin)" in it, which doesn't use the default terminal feature. Maybe that's what you've seen?

See video
Win10
https://github.com/microsoft/terminal/assets/11423362/00484b43-8b5a-4826-b6ac-17ed678bd86b

wat
Are you by any chance running with UAC entirely disabled?
reg query HKLM\SOFTWARE\Microsoft\Windows\CurrentVersion\Policies\System /v EnableLua should be able to tell us - 0 means UAC is disabled, anything else and it should be on.

That's right, it seems that whoever maintains the system used on this server has deactivated it, and activating it won't work again!


[SDK] API for endpoints should emit plural vips, not vip.

[WIP - Do NOT merge -- w/o the suggested change by Keith to emit names for the vip's ports, the consumer would not know which port speaks which protocol]

Changed endpoints API resource to emit vips (plural) instead of the now deprecated vip field.

There should not be more than 1 VIP per type of endpoint. Is there a situation where you believe this is not true?

@keithchambers an example of multiple vips is elastic's node which has an "http" and "transport" port exposed.

In this case, the entire response appears flawed. It should be something like:
{
  "http": {
    "vip": [
      "coordinator.elastic.l4lb.thisdcos.directory:9200"
    ],
    "address": [
      "<IP_ADDRESS>:1026"
    ],
    "dns": [
      "coordinator-0-node.elastic.mesos:1026"
    ]
  },
  "transport": {
    "vip": [
      "coordinator.elastic.l4lb.thisdcos.directory:9300"
    ],
    "address": [
      "<IP_ADDRESS>:1027"
    ],
    "dns": [
      "coordinator-0-node.elastic.mesos:1027"
    ]
  }
}

@paegun There is further work to be done before this is ready, correct? Can we close this PR in the meantime while that work is done?

w/o the suggested change by Keith to emit names for the vip's ports, the consumer would not know which port speaks which protocol

@paegun:  Consumers do know what protocols to speak to on which endpoint.  VIPs by design are defined against well known ports adhering to the underlying base technology's default scheme.
Additionally, introduction of field transport doesn't give much more of a clue, although it would make some sense to someone familiar with ElasticSearch, just as the VIPs with ports 9200 and 9300 do.
As it is today, multiple VIPs are defined per endpoint and information is being lost / overwritten, and this is a bug.  This fix is the minimal implementation of expected behavior for the current design.
@triclambert and @keithchambers:  The formatting change requested by @keithchambers is a feature request.  This PR addresses a bug within the context of the old design which already went through the scoping and requirements process.  @keithchambers, please submit your proposal as a feature request.

Agree with this bug fix, and a feature request for the format change.

FYI to manually retrigger a build you can post a comment of "retest this please" (please is required, lol)


deal with extensions when creating new note

Fixes
Fixes #53
Closes #53
Checklist

[x] I have made a change to this repository, be it functionality, testing, documentation, spelling, or grammar.
[x] I updated my branch with the master branch.
[x] I have commented any code I have modified or I did not modify any code
[ ] I have added necessary documentation about the functionality in an appropriate .md file
[ ] I have added the necessary testing to prove my fix is effective or my feature work, or I did not modify functionality.

Short description of what this PR does:
Does not append extension to the notename if it already has an extension.

Little more exploration of the program revealed that extra extension .md problem
was caused by new_note() calling open_note() with extension.
That shouldn't be happening because open_note() will add ext(through get_full_note_path()) if the filename doesn't end with $NOTES_EXT
P.S: sorry for being verbose

Good find! That totally makes sense, thanks for fixing it :+1:


How to handle render view exception to DefaultStatusCodeHandler

I am implementing a view engine. Inside the RenderView method, it might throw an exception. And I want to use the DefaultStatusCodeHandler to handle my exception.

I think there are two ways to throw the exception
First way is to throw inside RenderView method
public Response RenderView(ViewLocationResult viewLocationResult, dynamic model, IRenderContext renderContext)
{
    var html = MethodMightThrowException();
    return new HtmlResponse(contents: stream => AddHtmlToStream(html, stream));
}

In this case, the caller will eat up all exception and return an empty response.

Second way is to throw exception in HttpResponse content callback
public Response RenderView(ViewLocationResult viewLocationResult, dynamic model, IRenderContext renderContext)
{
    return new HtmlResponse(contents: stream =>
    {
        var html = MethodMightThrowException();
        AddHtmlToStream(html, stream);
    });
}

In this case, the under hosting platform catch the exception will re-throw it to use. In my case, the ASP.NET catch it and show the famous classic yellow exception page.

Both are not handling the exception to the DefaultStatusCodeHandler. How can I do that?

I found that the DefaultStatusCodeHandler actually get the exception from the context.Item[NancyEngine.ERROR_KEY], but I don't find any code setting this value.

I found several issues when making deeper debugging.

There is an extension to get Exception message from NancyEngine.ERROR_KEY but not set it. So I access it directly.

Proposal: besides get extension, add one set extension.
public Response RenderView(ViewLocationResult viewLocationResult, dynamic model, IRenderContext renderContext)
{
    try
    {
        var html = MethodMightThrowException();
        return new HtmlResponse(contents: stream => AddHtmlToStream(html, stream));
    }
    catch(Exception e)
    {
        renderContext.Context.Items[NancyEngine.ERROR_KEY] = e.ToString();
        var response = new HtmlResponse(HttpStatusCode.InternalServerError);
    }
}

After this changes, the DefaultStatusCodeHandler works and showing Error details are currently disabled. Please set StaticConfiguration.DisableErrorTraces = false; to enable..

But, after set StaticConfiguration.DisableErrorTraces = false; nothing is returned.

I take a deeper look into the code.

In DefaultStatusCodeHandler, it will only handle if response.Contents is null or is Response.NoBody. code
In ViewProcessor, if DisableErrorTraces=false it will wrap the response inside MaterialisingResponse. code Then the response check in DefaultStatusCodeHandler fails after this wrap.

So, the final response is an empty body.

/cc @grumpydev I saw you add MaterialisingResponse class in this PR #1504 . But I don't see any the pros from it. How do you think about this?
Besides, I think the Razor view engine encountered some similar issues too (#1316, #1397).


email and phone is not picked from standard Ga4 user_data

It would be nice if the templates would check the event data for standard Ga4 user_data
email: user_data.email
phone: user_data.phone
Just like you do with the facebook conversion API

done https://github.com/stape-io/gads-offline-conversion-tag/commit/0f9e4c832e0fdac18455a648c51adc1b6d75bb53


filter duplicate identifier

Some segue may have duplicate identifier

Looks good, but I'd like to wait until Swift 1.2 is released so that this can use the native Swift Set type rather than NSSet.

This has been fixed using the native Set type in Swift 1.2.


[Bug] The case in Figure 11 is not in the MMBench?

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] The bug has not been fixed in the latest version.

Type
I'm evaluating with the officially supported tasks/models/datasets.
Environment
The same as the official
Reproduces the problem - code/configuration sample
question_file = 'mmbench_dev_20230712.tsv'
question_df = pd.read_csv(question_file, sep='\t', encoding='utf-8')
physical_relation = question_df[question_df['category']=='physical_relation']
Reproduces the problem - command or script
NO
Reproduces the problem - error message
I iterate the physical_relation and did not find the data displayed in Figure 11.

Other information
No

@Richar-Du , the two cases do exist in MMBench, but in the test split. That's why you can not find them in the train split.


Accessing WebStream module after deploying trained model

I have a trained model that I deploy to the Vision AI camera (Reference - link). Once I am done with this, I need to view the output of the model and the camera field using the WebStream module (Reference - link). Once I add the WebStream module to the kit, I am able to access http://device_ip:3000, but I cannot view the stream. Am I doing something wrong?

Hi @MadanKrishnan97, we will get back to you on this.

Hi @MadanKrishnan97,  Can you please refer bellow mention link for your connectivity issue. https://azure.github.io/Vision-AI-DevKit-Pages/docs/Tutorial-HOL_Using_the_VisionSample/

Hi, Can you please refer below mention link for your issue. https://azure.github.io/Vision-AI-DevKit-Pages/docs/Deploy_Model_IoT_Hub/

Hi, Can you please refer below mention link for your issue. https://azure.github.io/Vision-AI-DevKit-Pages/docs/Deploy_Model_IoT_Hub/

Hello. Does it mean that I should not be doing the deployment step in this link https://github.com/microsoft/vision-ai-developer-kit/blob/master/machine-learning-notebooks/04-Deploy-Trained-Model.ipynb? Instead I should the generated URI and follow the deployment steps in the Azure wizard as suggested? Thanks!
PS: Sorry about closing the issue, it was a mistake.

Dear @MadanKrishnan97 Yes, both the doc is same its deep doc which I have shared you.

@MadanKrishnan97 Hi , Is this issue resolved ?

Hi @v-ragala @v-prsasa. I haven't had the opportunity to try out the suggested method. I will try it out soon and provide an update. Thank you for your patience.

@MadanKrishnan97 Thanks for the update. We will wait for your response.

@MadanKrishnan97 Any updates on this issue? Is it resolved?

Hi @v-prsasa. I am having issues with deploying custom models. Though the dlc conversion happens successfully, the deployment is not successful in the device. This is a different issue, and will create a new one for the same. But regarding the WebStream module, I could not get the video access and cannot confirm what the reason is. I got a workaround by using RTSP.

@MadanKrishnan97, Great! So we will go ahead and close this issue.

@MS-Nikunja Thank you. Yes, please do!


Update README with screenshots on how to customize keyboard shortcuts

Fixed in https://github.com/zenangst/MarvinXcode/commit/239ad53bd834192f6599d5241a0f2fa94d23d8b8


Add completed icon to step

Description
The update introduces an extra method in the Step class which allows setting a custom icon for completed steps. This new feature enhances the UI customization abilities, enabling users to further adjust the appearance of their progress steps based on their specific needs.
In the wizard blade view, new conditionals have been introduced. These conditionals check if a custom completed icon has been set by the user. If no custom icon has been set, the default icon is used. This ensures that the user interface remains consistent and intuitive, regardless of the customization level.
Visual changes
Before

After

Functional changes

Changes have been tested, and it is confirmed that they don't break existing functionality.
Up-to-date documentation has been included.
The code style has been fixed by running the composer cs command, ensuring it adheres to the project's style guide.

Thanks, especially for doing the docs screenshots

In a scenario where the wizard is skippable and the buttons are disabled e.g
->skippable()
->nextAction(fn (Action $action): Action => $action->extraAttributes(['x-show' => 'false']))
->previousAction(fn (Action $action): Action => $action->extraAttributes(['x-show' => 'false']))

The redundant checked icon is still displayed. Hopefully this video helps explain what I mean - https://jmp.sh/2YXDsYQN
I tried passing null to ->completedIcon(null) (https://filamentphp.com/docs/3.x/forms/layout/wizard#customizing-the-icon-for-completed-steps) but this has no impact due to the @else fallback.
Would it be possible to add the option to omit the completed icon by passing null? E.g ->completedIcon(null)


iCub Genova03 config update + index finger fix

This Pull Request addresses:

Index finger fix after PID re-tuning
General robot configuration fix, now tested on iCubOS running the Robotology-Superbuild v2024.02.0

This PR closes https://github.com/robotology/icub-tech-support/issues/1747 and https://github.com/robotology/icub-tech-support/issues/1674
To @sgiraz and @AntonioConsilvio who supervised these edits: this PR also fixes the wrong parts-calibration order.

OK from my side! Waiting for @AntonioConsilvio review.
cc @pattacini

Fine with me 👍🏻
Awaiting @AntonioConsilvio 's review then.


Update offsets

Mutually blocked with the test failure fixed by https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/820, so moving this commit there


WIP libsodium submodule

Don't download libsodium artifacts. Build it from source!
TODO: sometimes cargo build fails if you rebuild the project. Repeat cargo build and it will be build. Weird.
Profit:
~/sodiumoxide$ cargo tree
sodiumoxide v0.2.0 (/home/humbug/sodiumoxide)
├── libc v0.2.45
├── libsodium-sys v0.2.0 (/home/humbug/sodiumoxide/libsodium-sys)
│   └── libc v0.2.45 (*)
│   [build-dependencies]
│   ├── cc v1.0.28
│   └── pkg-config v0.3.14
└── serde v1.0.82
[dev-dependencies]
├── rmp-serde v0.13.7
│   ├── byteorder v1.2.7
│   ├── rmp v0.8.7
│   │   ├── byteorder v1.2.7 (*)
│   │   └── num-traits v0.1.43
│   │       └── num-traits v0.2.6
│   └── serde v1.0.82 (*)
├── rustc-serialize v0.3.24
├── serde v1.0.82 (*)
└── serde_json v1.0.33
    ├── itoa v0.4.3
    ├── ryu v0.2.7
    └── serde v1.0.82 (*)

Pull Request Test Coverage Report for Build 794

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.516%

Totals

Change from base Build 789:
0.0%

Covered Lines:
2482

Relevant Lines:
2626

💛  - Coveralls

So, being inexperienced in the world of C / C++, I'm noticing that I can't get this to build on my Ubuntu machine. My laptop is unable to run autogen.sh, and I don't know just yet what dependencies I'm missing or where to get them. I see no docs and I haven't fully researched it.
That said, if I download the released source directly and build it myself just using ./configure && make && make check, my laptop builds the library just fine.
So my question is, with this change, how many extra dependencies would need to be installed on a Rust developer's machine in order to use sodiumoxide in their project?
Would it be possible to get the release source code in the libsodium directory, rather than using the libsodium Git repository?

You should never run autogen.sh, unless you are making changes to the configure.ac code.
All the required files are already present both in releases and in the stable branch if you directly check out the git repository.

You should never run autogen.sh, unless you are making changes to the configure.ac code.
All the required files are already present both in releases and in the stable branch if you directly check out the git repository.

So then it looks like the submodule needs to be at commit 5bc564c, where 1.0.16 was merged into the stable branch. On my machine, I updated the submodule to that commit and removed the autogen.sh section from build.rs, and the build runs just fine.

Well, I am not quite sure whether we should build it from sources.
@jedisct1 is there a universal script to build linsodium on any OS?

@kpp There isn't, but ./configure && make check && make install will usually do the job.

@kpp Unless I'm mistaken, there's no way around building it from source for Unix-like OSes. Though building from source in Windows sounds like a pain. That looks like it requires installing the Windows 8.1 SDK. Would it be worth downloading the pre-compiled lib file when on Windows, but instead of using a SHA hash to verify it, use the sequoia_openpgp crate to verify the PGP signature?

@pcrockett @kpp Instead of relying on external tools, the signature can be verified with the minisign_verify crate.

@jedisct1 Of course... yeah the sequoia route is way overkill (and very beta). It didn't take me long to get minisign verification working based on the master branch. Decided to go ahead and open PR #329.

How do we handle builds for windows or other platforms. Also I would still like to keep the option of linking both dynamically and using a system libsodium.


Add clarification to docs around S3 secrets plugin interaction

After a great bit of trial and error, I discovered that the propagate-environment flag does not, in fact, result in proper loading of secrets injected by Buildkite's S3 Secrets plugin.
This PR adds adds clarification to documentation explaining how:

Environment variables imported from the Buildkite S3 secrets plugin will not properly load unless explicitly defined
The propagate-environment flag does not automatically import environment variables defined in the S3 secrets plugin environment hook

Yeah, propagate-environment flag would only propagate environment variables available in the step as stated in the docs. To import other environment variables , we need to explicitly pass in its key names under the 'environment' attribute of the docker plugin.
It's always good to have more clarification. Thanks @dannykansas for adding more clarification!


Application crash if a call is receive during game

On iOS devices, when someone call me during a game, the game crash when i go back to it.
I don't find it in background applications list.

Thanks for the report, I will investigate.

PhoneGap version abandoned: http://esviji.com/2016/02/android-and-ios-apps-development-stopped.html