Is it possible to debug device's node environment like a Webview in Chrome Inspector?

I tried ConsoleSurface. It's cool but uncomfortable to debug. Is it possible to debug from PC?

How to run device's node with --inspect --debug-brk ?
https://medium.com/@paul_irish/debugging-node-js-nightlies-with-chrome-devtools-7c4a1b95ae27

At the moment, the debugging functionality of node has been disabled.  There was no particular reason for that other than I was trying to get the basic functionality up and running first.  If you want to try to get this working:

Look at NodeInstance.cpp -- I've commented out most of the inspector initialization; you would need to uncomment it and fix any dependencies
Add your flags in common.cpp in method ContextGroup::init_v8()

If you can get it running, please submit a pull request.  It would be great to get this in the build.

I had tried to build project, but every time I get the error:
Error:Execution failed for task ':LiquidCoreAndroid:ndkBuild'.

Process 'command '/home//android-sdk-linux/ndk-bundle/ndk-build'' finished with non-zero exit value 2

Any more information than that?  Is there a compiler error?    Can you check the other windows?  You can cd to the jni directory in a terminal and execute the ndk-build command directly. Any errors will show up in the terminal.

Android NDK: android-9 is unsupported. Using minimum supported version android-14.
[armeabi-v7a] Compile++ thumb: liquidcore <= NodeInstance.cpp
/home/ilyamuromets/android_projects/LiquidExample/LiquidCoreAndroid/jni/node/NodeInstance.cpp: In member function 'int NodeInstance::StartNodeInstance(void*)':
/home/ilyamuromets/android_projects/LiquidExample/LiquidCoreAndroid/jni/node/NodeInstance.cpp:450:49: error: 'StartDebug' was not declared in this scope
         StartDebug(env, path, debug_wait_connect);
                                                 ^
/home/ilyamuromets/android_projects/LiquidExample/LiquidCoreAndroid/jni/node/NodeInstance.cpp:466:24: error: 'EnableDebug' was not declared in this scope
         EnableDebug(env);
                        ^
make: *** [/home/ilyamuromets/android_projects/LiquidExample/LiquidCoreAndroid/obj/local/armeabi-v7a/objs/liquidcore/node/NodeInstance.o] Error 1

I had added #include "node.h" into NodeInstance.cpp and uncommented code.

I couldn't build project, because I haven't enough knowledge about C++. Could you help me?

Apologies again for my absence.  I am probably several weeks away from being able to spend any quality time on this.
The issue you are having is the reason that I commented out the code in the first place.
For a little background, node is not designed to be run as discrete instances in a single application.  In stock node, when you spawn a new process, you are literally creating a new process and running a brand new instance of node in that process.  If you dig in the code for process.spawn(), you will see that it runs node on the command line in a forked process!  It then uses sockets to communicate between the two of them.  All of the startup code is in node.cc.
For my purposes, that didn't really work.  Instead, I needed to be able to manage multiple processes inside the same app.  To do this, I rewrote (aka cut-and-pasted mostly) node.cc into a class-based implementation, NodeInstance.cpp.  My goal was to rewrite/copy as little as is absolutely necessary, so as to be able to keep up with node updates, and to minimize errors.  So what you see in NodeInstance.cpp is exactly that.  Where I can call back into the node library, I do so.  Where I cannot, I copied the code from node.cc to NodeInstance.cpp and massaged it where required to allow for multiple instances.
So I basically copied all the startup code and then systematically copied subroutines (mostly static functions) into the project until it compiled correctly.  As the inspector was not part of my MVP requirement, when I got compilation errors because of it, I decided to just disable it for now and deal with it later.
That's why you are getting the compilation errors when you uncomment the code.
So it is not going to be quite as easy as that.  You would have to do what I was doing, which is figure out which functions are missing, copy them in, rinse, repeat.  As you don't have practical working knowledge of C++, this is going to be difficult for you.  Unless you want to stumble your way through it (which I encourage!), just leave this issue open and I will put it on my to-do list.  But I have no timeframe to give.

@ericwlange  I saw the code has been changed, and I see the code for inspector init but I don't see anyway on how to enable it yet. Or is some documentation missing on this?

@ackava This is not yet turned on.  I had planned to get it into the 0.7.0 release but it fell off the priority list.  The code is all there.  It is just a matter of turning it on and testing.  It will likely only work on Android, though.  But I won't know for sure until I can get back to the feature.

@ericwlange is any news with this feature?
I need to find the way how to debug background.js, that running in LiquidCore.
Maybe you can orient me, what files I should look to enable debugging features?


BUG: fix handling of project name in package metadata

Just prototyping the solution for #481. More changes are needed to actually fix the bug, but these would be easier after #480 is merged, or at least merging the other PR first would avoid merge conflicts. I believe that none of the current users of meson-python exposes the bug, thus this is not really high priority yet.

@rgommers This is ready for review

We've accumulated several bug fixes and small new features for 0.15.0, and the PR backlog is pretty empty. WDYT about tagging a release now?


save_ensemble fails for data with large uint64 values

While working with the Full FG Dwarf Dipper Search notebook, I called Ensemble.save_ensemble after loading the LSDB catalogue dataframes into TAPE.
This ran for about 30 minutes (likely due to a table sync on a large unfiltered dataset) and then failed. Here is the full exception  but the pertinent failure is below with a uint64 -> int64 conversion failure.
ValueError("Failed to convert partition to expected pyarrow schema:
 ArrowInvalid('Integer value<PHONE_NUMBER>4587917312 not in range: 0 to<PHONE_NUMBER>854775807
 Conversion failed for column _hipscat_index with type uint64)

    Expected partition schema:
     ...
     _hipscat_index: int64
     ...

    Received partition schema:
     ...
     _hipscat_index: uint64
     ...

Note that PyArrow does have uint64 support.
I'm a little unclear why expected partition scheme is an int64, but Dask.to_parquet does allow you to explicitly check for the schema and we could potentially enforce this there. We should do a little investigation to see if we can avoid having to explicitly correct the schema ourselves.

It looks like the underlying issue may have begun in LSDB, further exploring the above example notebook.
We can see that we join two catalogs, each of which having indices with dtype uint64.

However the resulting catalog, _sources has an index with dtype int64
And it is this catalog's dataframe which we use as the source frame for the Ensemble, explaining the TapeSourceFrame.to_parquet error from above

Filed https://github.com/astronomy-commons/lsdb/issues/145 against LSDB for now

https://github.com/astronomy-commons/lsdb/issues/145 was resolved, and I will repot back once I've rerun the workflow


athenz-auth-core is not in maven central

I'm trying to use sia inside java, but couldn't find athenz-auth-core, can you help to fix this.
https://github.com/yahoo/athenz/blob/master/docs/dev_centralized_access.md
https://mvnrepository.com/search?q=athenz
thanks.

For now our libraries are only available from Bintray repository. Please take a look at the full java example documentation for centralized access:
https://github.com/yahoo/athenz/blob/2bb23838e09eda7d8ec67cace14dddcbe8fb6a49/docs/example_java_centralized_access.md#client-project-dependency-update


Open Speed dial a different direction

How would one customize your library so that the speed dial could open any direction (up, down, right or left)? Right now, it only shoots upwards.

@craig1123:
You can use the props positionV and positionH for control the SpeedDial position.
Example code
Full documentation

So positionV and positionH control the SpeedDial position. My question is about direction. Check out this library https://onsen.io/v2/docs/react/SpeedDial.html, if you type in "direction="left" on that speed dial, it opens to the left. None of the props for this library make the bubbleListItems slide out to the left/right

Published v0.2.0

npm install<EMAIL_ADDRESS>Example
Example code


Broken commands: kill + teleport

Both hit an NPE currently when executed in single player
Teleport has broken been broken longer but may be trickier to fix considering the suddenly moving player might trigger systems watching for weird behavior
Kill used to work - maybe it got shook up by recent changes (maybe the EntityRef client parameter?). Not too recent though, also broken in stable 49

Both commands exist twice: In CoreCommands and in PlayerSystem.
Similarly, debugTarget and setWorldTime have been moved to ClientCommand and also exist twice now.

Oho, so we had more duplication.
So really we need to remove those four from CoreCommands again then fix the two broken ones in PlayerSystem


R session aborted

Hello,
Thank you so much for developing so nice software.
When I ran following commands, it always shows R session aborted.
What can I do at present stage?
Thank you in advance for your great help!
Best,
Yue
da_res1 <- diffcyt(sce,
    formula = da_formula1, contrast = contrast,
    analysis_type = "DA", method_DA = "diffcyt-DA-GLMM",
    clustering_to_use = "merging1", verbose = FALSE)

What version of R are you using? What’s the output of your session info? Hard to advise without any information…

Hello HelenaLC,
Thank you so much for your quick response!
R version is R 4.4.0.
Please see following Screenshot.
Thank you in advance for your great help!
Best,
Yue

I'm not fully sure. But are you running this in VSCode ?
https://www.reddit.com/r/Ubuntu/comments/1d9ajpb/vscode_has_been_crashing_on_ubuntu_22044_lts_x86/
(at least several of the error message related to this are potentially VSCode-related)

Hello markrobinsonuzh,
Thank you so much for your great help!
I running all these commands in R studio version 4.4.0.
Thank you again and really appreciated!
Best,
Yue

Hi Yue,
Do you have the latest version of RStudio?
From what I've found, for example:
https://stackoverflow.com/questions/67638836/r-crashing-while-displaying-ggplot-after-update-process-memory-read-out-of-rang
https://issues.apache.org/jira/browse/ARROW-13037
I believe this is fully unrelated to diffcyt / cytofWorkflow.
Best, Mark

Hello markrobinsonuzh,
Thank you so much for your quick response!
My RStudio is 4.4.0. My Ubuntu environment is 18.04.5 LTS.
Is that because my Ubuntu environment is too old ?I should upgrade my Ubuntu environment?
Thank you in advance for your great help!
Best,
Yue

Your version of R is 4.4.0, but what version of RStudio do you have? See here:
https://posit.co/download/rstudio-desktop/
In any case, Ubuntu 18.04 is "end of life" for >1 year now, so it could be useful to upgrade:
https://ubuntu.com/blog/18-04-end-of-standard-support
I will close the issue because it does not appear to be a cytofWorkflow issue (I also checked building it locally on my hardward and it was fine), but rather a local installation issue.


first and second line inverted

settings in IDE:

output in discord:

as you can see first and second line are inverted

fixed by #273

fixed by #273

is not publish yet?

fixed by #273

is not publish yet?

not yet


Automatic implementation of GetHashcode and Equals in structs.

I think I just learned something...
So, do we really need to implement IEquatable for value objects?
https://stackoverflow.com/a/32797530/2409554

Hi @phmatray,
The motivation to implement IEquatable was to allow Dictionary, Collections of Value Objects.
https://docs.microsoft.com/en-us/dotnet/api/system.iequatable-1?view=netframework-4.8

The IEquatable interface is used by generic collection objects such as Dictionary<TKey,TValue>, List, and LinkedList when testing for equality in such methods as Contains, IndexOf, LastIndexOf, and Remove. It should be implemented for any object that might be stored in a generic collection.

Did you try removing the interface?

Hi @ivanpaulovich
I've found Donald Duck with an equality check in a collection without implementing IEquatable.

A pretty cool, not so well known advantage of Structs over Classes is that there is an automatic implementation of GetHashcode and Equals in structs.
That's pretty useful when keys are required for dictionaries

How could I go any further in these tests?
For the moment, this seems to confirm that IEquatable is not required.
    class Program
    {
        static void Main(string[] args)
        {
            var userCollection = new[]
            {
                new Person("Mickey", "Mouse"),
                new Person("Minnie", "Mouse"),
                new Person("Donald", "Duck")
            };

            var anotherUserWithSameProps = new Person("Donald", "Duck");

            Console.WriteLine("Equality check");
            foreach (var person in userCollection)
            {
                string output = person.Equals(anotherUserWithSameProps)
                    ? $"FOUND - {person}"
                    : person.ToString();

                Console.WriteLine(output);
            }
        }
    }

    struct Person
    {
        private readonly string _firstName;
        private readonly string _lastName;

        public Person(string firstName, string lastName)
        {
            _firstName = firstName;
            _lastName = lastName;
        }

        public override string ToString()
            => $"{_firstName} {_lastName}";
    }

How could I go any further in these tests?

After we remove the IEquatable interface and their related implementation, if tests are green we merge straight way.

I just downloaded a copy of the repository and made these changes. The tests remain green.

Let’s merge it ASAP.
@phmatray I invited you before, do you wanna be part of the project contributors?

@phmatray I invited you before, do you wanna be part of the project contributors?
Yes, of course :)

@phmatray I invited you before, do you wanna be part of the project contributors?

Yes, of course 😄

I sent the invite


Arabic words with no spacing

Bug reports:
Using html2canvas messes up the spacing between Arabic words as you could see in this JSFiddle.
Note: html2canvas 0.4.1 works, but the quality isn't good enough, since v1.0.0 has the Scale option.
Specifications:

html2canvas version tested with: v1.0.0-rc.5
Browser & version: Firefox v77.0.1 - Google Chrome v83.0.4103.116
Operating system: Windows 10

@niklasvh I am also facing issue with Arabic. The last two words lose space between them. I have tried 0.5.0 but that version has quality issue and result is of poor quality.

@niklasvh I am also facing issue with Arabic. The last two words lose space between them. I have tried 0.5.0 but that version has quality issue and result is of poor quality.

You need to scale up the canvas, check my previous comment

@MuizMahdi  unfortunately I can't. I want this fix in latest without scaling the content. Reason is that we already have input control with default font size. If we scale than size of default font size increases.
Any plan for this fix in future?

@MuizMahdi unfortunately I can't. I want this fix in latest without scaling the content. Reason is that we already have input control with default font size. If we scale than size of default font size increases.
Any plan for this fix in future?

Did you figure this out?

@MuizMahdi unfortunately I can't. I want this fix in latest without scaling the content. Reason is that we already have input control with default font size. If we scale than size of default font size increases.
Any plan for this fix in future?

Did you figure this out?

No still no update.

@MuizMahdi unfortunately I can't. I want this fix in latest without scaling the content. Reason is that we already have input control with default font size. If we scale than size of default font size increases.
Any plan for this fix in future?

Did you figure this out?

No still no update.

@niklasvh need update on this issue because, its affecting the use case.

@niklasvh need update on this issue because, its affecting the use case.

I have the same problem.


Incorrect detection count threads on multiCPU motherboards

@ifratric, on Windows any version, count threads is incorrectly detected. In my case, I have motherboard with two sockets 18 cores and 36 threads, he determined count threads for only one processor.

Thanks for reporting!
Looks like how we compute cpu_core_count (https://github.com/googleprojectzero/winafl/blob/master/afl-fuzz.c#L7821) would need to change. And I'm not currently sure if bind_to_free_cpu (https://github.com/googleprojectzero/winafl/blob/master/afl-fuzz.c#L475) would also need to be changed. In any case, a PR would be highly appreciated :)


Bugfix/copy changes

Title
Changes 👷♀
Copies and modals changed as in https://www.notion.so/oazo/Multiply-MVP-copy-changes-5a0bad7b570245cbbb6b3bc46b66bdba
How to test 🧪

- step 1 ...
Definition of done ✔

[ x] Acceptance criteria for each issue met
[x] Unit tests written where needed and passing
[ ] End-to-end tests for happy path
[x] Documentation updated
[x] Project builds without errors
[ ] Non-functional requirements met
[ ] Code reviewed and functionality tested by the reviewer (locally, Heroku, etc.)
[ ] Feature verified and accepted by Product Owner
[ ] Project deployed to production environment

I will add title to net value and make Liquidation price header smaller

One more thing would  be to ask for copy for those 2 cards (Manage Regular vault):

One more thing would be to ask for copy for those 2 cards (Manage Regular vault):

I'll ask @FBrinkkemper for those as we don't yet have them in notion

After changes:

new modals for old borrow vaults


pulumictl: init at 0.0.29

Motivation for this change
This may replace tf2pulumi at some point.
cc @mausch
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package built:

pulumictl

thanks, I needed the package to check the pulumi python package.


Importing flutter packages

Is there any way to import flutter packages for video,webview,skelton etc in flet

The packages must be implemented on both Flutter and Python side. You can check some of the closed PRs to see how it's done: https://github.com/flet-dev/flet/pulls?q=is%3Apr+is%3Aclosed
We are working on developer docs.


Segmentation fault when adding uiImage to container

I am getting a segmentation fault when adding a uiImage to a container. I got a crash when using GTK3 and also when trying to run the windows version under Wine. I don't have a minimal reproduction but I think I've got a good lead. The implementation of uiImage for unix isn't calling the macros that libui uses for creating the default handler functions. The segmentation fault is occuring in uiControlHandle.
This is as far as I got before running out of steam. I plan to revisit this later.

[ ] Tested on Windows
[ ] Tested on Mac
[x] Tested on GTK
[x] Tested in Wine
[ ] Minimal reproduction

uiImage is not a subclass of uiControl, that is why your application crashes.
There is currently no standalone image support in libui-ng. uiImage is solely used in conjunction with uiTable.
I guess this could be clearer in the docs?
We desperately need a standalone image support IMO.
Related to #163 which requests setting of the uiTable row height to use uiTable as a way of displaying images.


Website needs a meta image

I noticed the website (https://locust.io/ ) is missing a meta image, so different platforms guess differently as to what the image should be.
Google chat for example is picking Armin Ronacher's avatar image, which is confusing:

I'd love to fix this, but I think @heyman is the only one with access to the site itself.

Now fixed!


compat.py cleanup and type annotations

During digging inside djoser internals I discovered that there is some old code, so I desided to start remove piece by piece and to increase overal code quality. @tomwojcik  what you think?

Looks good. I will double check a few things soon, but it makes sense. Thank you


Scanner suddenly starts going backwards, then stops

Current Behavior
I am having an issue where, when scanning character development and material items, the scanner works fine for the first few pages then suddenly starts going backwards from the bottom and then stops when it reaches the top of the current page. I have used the scanner successfully many times before, and I even reinstalled and forced a 4.0 update to make sure everything was to be working as expected, with no luck. Not sure what I'm doing wrong, or if the scanner is just experiencing a bug.
https://github.com/Andrewthe13th/Inventory_Kamera/assets/142496346/8e3a45ab-8e39-4988-8604-9bec4801ffbe
Expected Behavior
The scanner to not skip rows or items
Reproducing a Behavior
I simply am using the scanner as intended.
Device OS
Windows 10
Genshin Impact Version
4.0
Inventory Kamera Version
1.3.9
Screen resolution, screen mode, and UI scale
Tried with both 1920 x 1080 Windowed and 1600 x 900 Windowed
Additional notes and remarks
No response
Evidence
InventoryKamera.log
Preflight Checklist

[X] I have checked for similar issues in the issue page, and I have not found any similar to mine.
[X] I am using the latest version of the scanner.
[X] I tried setting the scanning delay to maximum, and it still doesn't work.
[X] I have attached the video recording to the bug report.
[X] I have attached the error log to the bug report.
[X] I am available to later explains the issue.
[X] I have disabled all screen filtering software or plugins (i.e. HDR, Image Sharpening) while scanning.
[X] I use default in-game keybindings or have configured keybindings for the scanner.

This issue should be fixed in the latest release. Please reopen the issue if you find the problem still exists.


How to fill in "val_seq"?

When write "None", Program error.

Hi, thanks for your interest in our work.
When specifying val_seq = None, the sequence list would be automatically initialised in the corresponding dataloader. For example, please refer to L34 to L51 in data/dataloaders/dvs_loader.py.
If you are loading your own dataset via data/dataloaders/example_loader.py, no default sequence list is given. Then, the val_seq is needed to be manually specified as the list of sub-directories in your dataset folder.

Thank you very much for your reply!


Add Tokyo International University

Tokyo International University official website: https://www.tiu.ac.jp/
Following url contains contact email information of my enrolled major:
https://www.tiu.ac.jp/etrack/contact/prospective-students-contacts/
E-track students email domain: @al.tiu.ac.jp

@baykaa Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Update index.js

~15% accuracy improvements, same performance

Coverage remained the same at 100.0% when pulling ff0e1a4c837235e1d2df88838372bb9ad66655a9 on Murcurio:patch-1 into 1a8d3e59f59903a38bcdd080f2b3778b446f40c3 on kus:master.

+1


[libvmaf] New port

Fixes #41129.

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

@microsoft-github-policy-service agree

For naming this package (vmaf or libvmaf), Repology lists vmaf as the original project name. However, I chose libvmaf (though I'm open to changing it to vmaf if reviewers suggest) because we're shipping only the library, not the tools.
For example, on my Fedora 40 system, vmaf is split into three packages:
$ rpm -qa | grep vmaf
libvmaf-2.3.0-7.fc40.x86_64
vmaf-models-2.3.0-7.fc40.noarch
vmaf-2.3.0-7.fc40.x86_64

vmaf provides /usr/bin/vmaf, libvmaf provides /usr/lib64/libvmaf.so, and vmaf-models provides pre-trained models like /usr/share/model/vmaf_float_b_v0.6.3.json. So, I believe libvmaf is more appropriate.

Android and iOS compilation is disabled due to weird errors when compiling the arm-neon-android triplet. It seems that vcpkg/meson wrongly sets the sysroot, causing functions like fseeko, which aren't supported in Android API 21, to be used. According to the Android documentation (https://android.googlesource.com/platform/bionic/+/main/docs/32-bit-abi.md):

Android does not require the _LARGEFILE_SOURCE macro to be used to make fseeko and ftello available. Instead they're always available from API level 24 where they were introduced, and never available before then.

I need some more time to investigate this issue, and I don't think there are many people who want to run VMAF on Android or iOS (at least, not me).
[53/55] /Users/redacted/android/sdk/ndk/27.1.12297006/toolchains/llvm/prebuilt/darwin-x86_64/bin/clang++ --target=armv7-none-linux-androideabi21 -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -march=armv7-a -mthumb -Wformat -Werror=format-security -frtti -fexceptions -fPIC -fno-limit-debug-info --sysroot=/Users/redacted/android/sdk/ndk/27.1.12297006/toolchains/llvm/prebuilt/darwin-x86_64/sysroot -Isrc/libvmaf.a.p -Isrc -I../src/v3.0.0-2bc8652704.clean/libvmaf/src -Iinclude -I../src/v3.0.0-2bc8652704.clean/libvmaf/include -I../src/v3.0.0-2bc8652704.clean/libvmaf/src/feature -I../src/v3.0.0-2bc8652704.clean/libvmaf/src/feature/common -I/Users/redacted/src/workspace/test-vcpkg2/build/vcpkg_installed/arm-neon-android/include -fdiagnostics-color=always -D_LIBCPP_HARDENING_MODE=_LIBCPP_HARDENING_MODE_FAST -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=c++11 -g --target=armv7-none-linux-androideabi21 -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -march=armv7-a -mthumb -Wformat -Werror=format-security -frtti -fexceptions -fPIC -fno-limit-debug-info --sysroot=/Users/redacted/android/sdk/ndk/27.1.12297006/toolchains/llvm/prebuilt/darwin-x86_64/sysroot -fPIC -pthread -pedantic -DOC_NEW_STYLE_INCLUDES -MD -MQ src/libvmaf.a.p/svm.cpp.o -MF src/libvmaf.a.p/svm.cpp.o.d -o src/libvmaf.a.p/svm.cpp.o -c ../src/v3.0.0-2bc8652704.clean/libvmaf/src/svm.cpp
FAILED: src/libvmaf.a.p/svm.cpp.o
/Users/redacted/android/sdk/ndk/27.1.12297006/toolchains/llvm/prebuilt/darwin-x86_64/bin/clang++ --target=armv7-none-linux-androideabi21 -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -march=armv7-a -mthumb -Wformat -Werror=format-security -frtti -fexceptions -fPIC -fno-limit-debug-info --sysroot=/Users/redacted/android/sdk/ndk/27.1.12297006/toolchains/llvm/prebuilt/darwin-x86_64/sysroot -Isrc/libvmaf.a.p -Isrc -I../src/v3.0.0-2bc8652704.clean/libvmaf/src -Iinclude -I../src/v3.0.0-2bc8652704.clean/libvmaf/include -I../src/v3.0.0-2bc8652704.clean/libvmaf/src/feature -I../src/v3.0.0-2bc8652704.clean/libvmaf/src/feature/common -I/Users/redacted/src/workspace/test-vcpkg2/build/vcpkg_installed/arm-neon-android/include -fdiagnostics-color=always -D_LIBCPP_HARDENING_MODE=_LIBCPP_HARDENING_MODE_FAST -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=c++11 -g --target=armv7-none-linux-androideabi21 -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -march=armv7-a -mthumb -Wformat -Werror=format-security -frtti -fexceptions -fPIC -fno-limit-debug-info --sysroot=/Users/redacted/android/sdk/ndk/27.1.12297006/toolchains/llvm/prebuilt/darwin-x86_64/sysroot -fPIC -pthread -pedantic -DOC_NEW_STYLE_INCLUDES -MD -MQ src/libvmaf.a.p/svm.cpp.o -MF src/libvmaf.a.p/svm.cpp.o.d -o src/libvmaf.a.p/svm.cpp.o -c ../src/v3.0.0-2bc8652704.clean/libvmaf/src/svm.cpp
In file included from ../src/v3.0.0-2bc8652704.clean/libvmaf/src/svm.cpp:11:
/Users/redacted/android/sdk/ndk/27.1.12297006/toolchains/llvm/prebuilt/darwin-x86_64/sysroot/usr/include/c++/v1/fstream:869:9: error: no member named 'fseeko' in the global namespace; did you mean 'fseek'?
  869 |   if (::fseeko(__file_, __width > 0 ? __width * __off : 0, __whence))
      |       ~~^

It seems that vcpkg/meson wrongly sets the sysroot, causing functions like fseeko, which aren't supported in Android API 21, to be used.

vcpkg doesn't set the wrong sysroot. But many packages do configuration wrong, or make wrong assumptions. And the NDK really contributes to confusion.

Note that vcpkg builds with API level 21 by default. You can test locally with a higher API level e.g. using
vcpkg install libvmaf:arm64-android --cmake-args=-DVCPKG_CMAKE_SYSTEM_VERSION=24

If it builds successfully, the CI failure shall be recorded in scripts/ci.baseline.txt, similar to:
https://github.com/microsoft/vcpkg/blob/43b4ba9e5585f9b6111f11ce2284b1f9d5d04555/scripts/ci.baseline.txt#L564-L567
Some packages need even higher levels.

The libvmaf library itself doesn't use fseeko symbols. That's why I find this issue strange. The file /Users/redacted/android/sdk/ndk/27.1.12297006/toolchains/llvm/prebuilt/darwin-x86_64/sysroot/usr/include/c++/v1/fstream is clearly part of NDK's libc++, which is designed to handle various API levels. However, for some reason, it assumes fseeko is available when it's actually not. So I don't think limiting the API level is a long-term solution to this issue.

Some packages need even higher levels.

The issue is with meson, not vcpkg. I added a workaround for https://github.com/mesonbuild/meson/issues/3049 as a patch. If the API level is changed to 24 or above in the future, or if meson is fixed to no longer define _FILE_OFFSET_BITS=64 for Android targets, the workaround can be removed.


cannot import name 'BackTestStats' from 'finrl.trade.backtest'

I get this error now. Any suggestions?

we renamed the backtesting functions please change to from finrl.trade.backtest import backtest_stats, backtest_plot, get_daily_return, get_baseline.

First. I am amazed at how you all conceived of the idea of RL, implemented
it, and coded it. I looked at the program: WOW. You guys and gals are
brilliant!
Second. Thank you for your reply! I am really grateful, and nothing
expresses gratitude like money. Do you have a donation button on your
website? If so, I did not find it and would appreciate it if you would send
a link.
Third. Are you available for hire/consulting gigs?
My regards to you and contributors for the excellent work.
Stephen
Stephen Hobbs, M.Sc., MBA
Skype: stephenmhobbs
Cell:<PHONE_NUMBER>
On Sun, Mar 14, 2021 at 1:35 AM Bruce Yang @.***> wrote:

we renamed the backtesting functions please change to from
finrl.trade.backtest import backtest_stats, backtest_plot,
get_daily_return, get_baseline.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AI4Finance-LLC/FinRL-Library/issues/177#issuecomment-798843458,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2LS4T6FS754MFTZLYKDRDTDRKKXANCNFSM4ZEDOWXQ
.

Is there a way I could have found the change on my own w/out bothering you
and your team, of discovering this without submitting a help request?
I don't like bothering ppl and if I can learn what to do and what not to
do, it makes our lives better, faster, easier.
Stephen
Stephen Hobbs, M.Sc., MBA
Skype: stephenmhobbs
Cell:<PHONE_NUMBER>
On Sun, Mar 14, 2021 at 1:35 AM Bruce Yang @.***> wrote:

we renamed the backtesting functions please change to from
finrl.trade.backtest import backtest_stats, backtest_plot,
get_daily_return, get_baseline.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AI4Finance-LLC/FinRL-Library/issues/177#issuecomment-798843458,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2LS4T6FS754MFTZLYKDRDTDRKKXANCNFSM4ZEDOWXQ
.

Thx for your compliment!
Sorry for delayed reply.  Hope you enjoyed the codes.  If you have more suggestions (besides codes), you can speak in the slack channel.


Implement recommended changes from Leptos developer guide.

Closes: #225

Very good to have perfectly consistent formatting! I actually recommend augmenting this PR by making a mandatory check in CI that ensures that the formatting of the code never deviates from the standard checked for by 'cargo fmt'.
Needs to have conflicts resolved.


Langium Language Server Crashes with "TypeError: Cannot read properties of undefined (reading 'localeCompare')"

When using the word type in a Rule following another Rule that is not terminated with a ;, the Langium language server crashes.
Langium version: 3.2.0
Steps To Reproduce

Create a new prject using the steps from the Langium documentation and choose the defaults.
Open the project in Visual Studio Code
Paste the following code into the hello-world.langium:

grammar HelloWorld

entry Model:
    NonTerminatedRule

NonTerminatedRule:
    'hello' RuleWithType

RuleWithType:
    type='String'

I know the syntax is not valid, but I still expect the language server not to crash, because I have to restart Visual Studio Code each time that happens.

This has actually already been resolved, see https://github.com/eclipse-langium/langium/pull/1634. Our automated release job failed, because Microsoft decided to revoke our automation token (I don't know why).
I published the newest version of 3.2.0 manually. Closing.


Fixes "XML tag 'Tool is missing 'Tcp tag"

changed TCP to Tcp to match plugin tag
changed DI and DO to start from 0 so slider and SRC would be the same value

Hi, are you sure about adding the "0" IOs? I think KUKAs start at 1, so the controller's first output is 1, not 0. And in the plugin, since Grasshopper is index 0 based, you would use index 0 to select the DO 1. A bit confusing, I know.

Yes, kuka start at 1, but I don't think that's an issue, we chooses our IO setting base on what we see on the controller, so it's much more intuitive to use 27 if we see 27.
And since the controller would never show IO 0, there's no issues there.

I could add flag on the DO component to to switch between zero based and the controller's numbering.

I added the "useControllerNumbering" optional attribute to the IO element. This allows you to use the signal numbers directly in components like SetDO, WaitDI, etc. and ignores any IO names specified in the xml. This will work with KUKA, UR and Staubli, not with ABB, as it requires signal names.


Custom styling for match highlight?

Thanks for this great extension! Is there any way to style the match highlight? On my machine the highlight boxes are two thick, covering up part of the words. Also would be nice to define custom color, and to accent the match of the current selected line to make it stand out from the rest of the matches.

hello, i can take a look at this when i get back from my vacation may 15 :) . thanks for the feedback.

don't have bandwidth on this, feel free to reopen if you want to make an contribution.


Improved Issue-linking Logic

Description

When users link to a github issue on discord via repo#num (e.g. sir-lancebot#999), we currently (as of #998) display a purple tick all the time if it's a closed issue.
Instead, we should display a purple tick only when it has a linked PR which has been merged, and otherwise a red tick (the tick from before #998).
Reasoning

This change will allow users to easily see whether an issue was closed due to "not going to be implemented" / "duplicate" etc. or whether it was closed since it has been implemented by a PR.
Proposed Implementation

I'm not too sure about exact implementation details, but it should just be a case of using data provided by the GitHub API.
Additional Details

I spoke to @ChrisLovering about this yesterday and they seemed in favour, but wanted me to create an issue first to get some more feedback as to whether we'd like this & implementation etc.
Would you like to implement this yourself?

[x] I'd like to implement this feature myself
[ ] Anyone can implement this feature

GitHub itself doesn't provide this distinction (a closed issue is just a closed issue), so I'm not how this would be implemented in a practically useful way.
For example, one way of doing it would be checking for linked pull requests as you say. However, that brings the question: should we also check if the linked pull requests was merged or closed without merging? This also doesn't take into account issues closed by just a commit, or issues closed without any PRs needed (e.g. actioned items in meta or organisation repos), or just closed manually without linking what closed it specifically.
Another option would be to for e.g. an "approved" label, but that would have to be specific to a particular repo and would require consistent usage of labels which may not always happen.

GitHub itself doesn't provide this distinction (a closed issue is just a closed issue), so I'm not how this would be implemented in a practically useful way.
For example, one way of doing it would be checking for linked pull requests as you say. However, that brings the question: should we also check if the linked pull requests was merged or closed without merging? This also doesn't take into account issues closed by just a commit, or issues closed without any PRs needed (e.g. actioned items in meta or organisation repos), or just closed manually without linking what closed it specifically.
Another option would be to for e.g. an "approved" label, but that would have to be specific to a particular repo and would require consistent usage of labels which may not always happen.

Yes, I was gonna mention the same points.
Mostly, because as mentioned there are closed issues with no PR linked to it, but still.. they have been resolved, and closed manually.
If someone is curious about an issue, they could just click on the link sent by the bot and check it directly on GitHub.
Hence, there is no efficient way to correctly implement this, and ensure that it never shows false results at the same time.

Making the difference between closed resolved and unresolved is part of the GitHub roadmap and has not been implemented yet https://github.com/github/roadmap/issues/289

GitHub API now provides information on whether the issue was closed as resolved or not planned via the state_reason key.
See example: https://api.github.com/repos/python-discord/bot/issues/2875
Rather than checking for a linked PR which could complicate implementation, we could maybe check for this instead and display either a purple closed icon or github's gray closed icon.

Yeah, since that exists now it's definitely the way to go. Marked as approved


HeaderMatchesChecker : seemingly not working.

I have the following in the .xml file
<check class="org.scalastyle.file.HeaderMatchesChecker" level="warning"
		enabled="true">
		<parameters>
                <parameter name="header"><![CDATA[/**
  * Copyright (C) 2014-2018. Gavin Baumanis. All Rights Reserved.
  *
  * E<EMAIL_ADDRESS>  *
  */]]></parameter>
		</parameters>
	</check>

and I have the following at the start of the file;
/**
  * Copyright (C) 2014-2018. Gavin Baumanis. All Rights Reserved.
  *
  * E<EMAIL_ADDRESS>  *
  */

yet;
sbt scalastyle still reports

Header does not match expected text

I am happy to concede that I have something wrong in the XML...
But I have been trying to get this setting to work for more than two hours now and am now giving up and asking for some help - Thanks!

This works for me. Were you able to resolve this @gavinbaumanis . If not, what version of scalastyle are you using ? Any reproduction steps should help solving this faster

Hi there,
I have absolutely no idea what was causing the issue.
I ended up deleting my workspace, retrieving a fresh copy of the source code from version control and also re-installed sbt, too.
Most sensible answer is an issue in the XML - but I really don't know;
The code now passes stylecheck, correctly.


[Suggestion] Include in Commit window context menu

Thanks for your suggestion!
I will check the possibility of this and if it is possible, I'll implement it!
Meanwhile, feel free to open a PR

Meanwhile, feel free to open a PR

I just found out about your PR. Thanks, will check is asap!

https://github.com/Drjacky/Avocado/pull/113

@dgomolka Do you mean in the Commit Changes dialog? or Local Changes window?

Yeah I meant local changes 😄

https://github.com/Drjacky/Avocado/pull/113


KeyboardAvoidingView animation issues since 0.76+

Description
I've updated my project from 0.75.2 to 0.76.1 this morning, and I noticed that the KeyboardAvoidingView, which was previously animating properly, is no longer animating. It now jumps and doesn't follow the keyboard animation.
Not sure why in my project it doesn't work, but while creating the reproduction repo, I noticed the animation behaviour has changed and doesn't look native smooth anymore.
Steps to reproduce
Any new project with KeyboardAvoidingView
React Native Version
0.76.1
Affected Platforms
Runtime - iOS
Output of npx react-native info
System:
  OS: macOS 15.2
  CPU: (10) arm64 Apple M1 Pro
  Memory: 157.52 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 22.11.0
    path: ~/.nvm/versions/node/v22.11.0/bin/node
  Yarn:
    version: 1.22.17
    path: ~/.npm-global/bin/yarn
  npm:
    version: 10.9.0
    path: ~/.nvm/versions/node/v22.11.0/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /Users/Alexis/.rbenv/shims/pod

Stacktrace or Logs
No crash

Reproducer
https://github.com/alexmngn/reproducer-react-native-keyboard-avoiding
Screenshots and Videos
On my project
Before update, 0.75.2
https://github.com/user-attachments/assets/5089cd4d-fb2f-4bde-9a9b-a7790b989c23
After update, 0.76.1
https://github.com/user-attachments/assets/60bc6d95-c444-4492-887b-cdb7d4e562d1
On the reproducer repo
Before update, 0.75.2
https://github.com/user-attachments/assets/83df6cab-b4c4-4698-a458-291ab37f982c
After update, 0.76.1
https://github.com/user-attachments/assets/8fa6d39c-88af-4527-b41c-d7ae2bc8e7a2

Hey @alexmngn, thanks for opening this issue.
Does it happen with the new architecture too?

I've encountered a few issues with KeyboardAvoidingView since upgrading to 0.76. The first was due to a regression that was fixed in 0.76.1. The second is a bug in react-native-reanimated and New Architecture that seems to break onLayout, ultimately causing KeyboardAvoidingView to fail to calculate the correct height. The third is an issue with LayoutAnimation.configureNext(), which may be the root cause of this particular bug.

I've updated to RN0.76.3 and still encountering this issue

The Reanimated issue has been reported here: https://github.com/software-mansion/react-native-reanimated/issues/6751. However, there may be an issue with KeyboardAvoidingView without Reaniated given the reproducer above does not have Reanimated installed.

Any update on this?

The third is an https://github.com/facebook/react-native/issues/47617 with LayoutAnimation.configureNext(), which may be the root cause of this particular bug. (I've since discovered the LayoutAnimation.configureNext() issue seems to be related to react-native-reanimated.)

i can confirm the issue is not specifically reanimated. trying the repro without reanimated, the animation is still broken. #47617 is still valid to this one. the LayoutAnimation is not working on new architecture. turning back to old architecture should do the trick.


maxHeight and maxWidth are both required

If you only fill in maxHeight or maxWidth, the generated command will look something like:
....resize 100xNaN......

Which obviously will not resize the image :).
It would be a nice small enhancement to check if the properties are both present.

This was fixed in a previous release.


default cluster region

Towards: https://github.com/giantswarm/giantswarm/issues/14025
We want to default Spec.Provider.Region

tested this on gauss, works. :)


Freespace planner partial trajectory logic may shift turning point in manoeuvers

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
This is a similar but orthogonal issue to https://github.com/autowarefoundation/autoware.universe/issues/5988. I think the "index logic" implemented by getReversingIndices and getPartialTrajectory causes partial trajectories to miss the actual turn point:
https://github.com/autowarefoundation/autoware.universe/blob/5de111fe21701facb3a501bcbf0094577ad30667/planning/freespace_planner/src/freespace_planner/freespace_planner_node.cpp#L88-L102
https://github.com/autowarefoundation/autoware.universe/blob/5de111fe21701facb3a501bcbf0094577ad30667/planning/freespace_planner/src/freespace_planner/freespace_planner_node.cpp#L119-L141
If you consider the following example trajectory, going forward then backward:

idx
0
1
2
3
4
5
6
7

t(s)
0.0
1.0
2.0
3.0
4.0
5.0
6.0
7.0

vel(m/s)
1.0
3.0
1.0
±0.0
-1.0
-3.0
-1.0
0.0

x(m)
0.0
1.0
4.0
5.0
5.0
4.0
1.0
0.0

By the logic of getPartialTrajectory, the reversing index is 2 or 3 depending on the the sign of point 3. Then getPartialTrajectory extracts point from index 0 to 2 or 3, and override the velocity of the first and last point:

case 1: vel[3] > 0.0 (-> reversing index is 3), we get the partial trajectory:

idx
0
1
2
3

t(s)
0.0
1.0
2.0
3.0

vel(m/s)
3.0
3.0
1.0
0.0

x(m)
0.0
1.0
4.0
5.0

Changing the first point velocity is a bit strange here, but I guess it is to handle the case when we are at the turn point and avoid the first point of the partial trajectory being 0.0 (which would keep the vehicle stopped). That said, the current freespace planner implementation generate trajectories with a fixed 5km/s velocity target on all points so the velocity override has no effect in practice (velocity is later refined by motion_velocity_smoother)

case 2: vel[3] < 0.0 (-> reversing index is 2)

idx
0
1
2

t(s)
0.0
1.0
2.0

vel(m/s)
3.0
3.0
0.0

x(m)
0.0
1.0
4.0

You can see in this case we loose the point that was reaching x=5.0. Instead, Autoware would now stop at x=4.0.
One way to avoid this is to reduce the distance between trajectory points, but that means more computation. I think default A* minimum step distance is 0.4 meters, so Autoware may be constantly off by 0.4 meters from turn point.
Another way is to set the turning point on the next index (i.e. the first < 0.0 velocity). However I am afraid this could cause some strange behavior, as the last point of the partial trajectory could end up before the previous point.
I think the proper way to handle this would be to interpolate the turn point instead.
Any suggestion?
Expected behavior
Planned trajectory should be followed precisely
Actual behavior
With the current logic, Autoware may not be able to reach the turn point.
Steps to reproduce
N/A
Versions
No response
Possible causes
No response
Additional context
No response

Hello,
I've looked at this issue, but

I was not able to reproduce this issue on the current version. The partial trajectory is fully completed by the vehicle in autonomous mode.
For the example trajectory given,

idx
0
1
2
3
4
5
6
7

vel(m/s)
1.0
3.0
1.0
±0.0
-1.0
-3.0
-1.0
0.0

has velocity values that are zero, while the trajectory given to getReversingIndices would have a positive or negative non zero velocity value. Therefore the logic for finding the reversing indexes works.
createTrajectory function assigns this velocity profile before computing reversing indexes, each time it is called initially and during replanning in case of obstacles.
Trajectory createTrajectory(
  const PoseStamped & current_pose, const PlannerWaypoints & planner_waypoints,
  const double & velocity)
{
  Trajectory trajectory;
  trajectory.header = planner_waypoints.header;

  for (const auto & awp : planner_waypoints.waypoints) {
    TrajectoryPoint point;

    point.pose = awp.pose.pose;

    point.pose.position.z = current_pose.pose.position.z;  // height = const
    point.longitudinal_velocity_mps = velocity / 3.6;      // velocity = const

    // switch sign by forward/backward
    point.longitudinal_velocity_mps = (awp.is_back ? -1 : 1) * point.longitudinal_velocity_mps;

    trajectory.points.push_back(point);
  }

  return trajectory;
}

Assigning a non zero constant back/fwd velocity:

point.longitudinal_velocity_mps = velocity / 3.6;      // velocity = const
// switch sign by forward/backward
point.longitudinal_velocity_mps = (awp.is_back ? -1 : 1) * point.longitudinal_velocity_mps;`

So the trajectory used as input to getReversingIndices then getPartialTrajectory would look like this:

idx
0
1
2
3
4
5
6
7

vel(m/s)
5.0
5.0
5.0
5.0
-5.0
-5.0
-5.0
-5.0

reversing index set = [3],
Partial trajectory:

idx
0
1
2
3

vel(m/s)
5.0
5.0
5.0
0.0

The output partial trajectories are given to the velocity smoother.
createStopTrajectory publishes a trajectory where the end point velocity is set to zero, but is called on replanning phase, where a new complete global trajectory is made using createTrajectory


Bundling via esbuild.

Note:

Still not clear to me if node_modules can be bundled. I could not get it to be bundled, and without the node_modules pruned directory, the extension will not run. Documentation appears to state that VSCode will automatically install dependencies, but that does not appear to be the case.
The generated vsix file is around 26megs in size.

@khawkins @sfdctaka @maliroteh-sf

Cool, thanks for the feedback @khawkins ! Sorry this is taking so many iterations to get done!

Cool, thanks for the feedback @khawkins ! Sorry this is taking so many iterations to get done!

Haha no worries. This has been educational for me too. 🥲 😜

Another suggestion I have that would fit well with this PR—but could be a follow-on: we should probably add @vscode/vsce as a dev dependency to the project. Eventually/soon we're likely going to want to (semi-)automate creating VSIX packages, and that's going to be the easiest way to ensure that the packager is accessible in CI, similar to adding esbuild as we've done.

On second thought, let's just leave that for the story we have to automate publishing. It fits right into the requirements for that story. Disregard.


Add a single page UI to administer the application

As a Developer
I need the initial UI setup in the application
So that I can add UI functionalities for CRUD operations.
Details and Assumptions

Add a simple single-page UI file to the application.
Add the necessary configurations for UI to display.
This story will not include creating any CRUD functionality for the UI (only the basic setup).
Adding a static folder to the service, and copying all CSS and bootstrap files.
Add an empty index.html and corresponding rest-api.js file.
Replace the root URL in the service to point to the HTML.

Acceptance Criteria
Given the service is running and the initial UI setup is done
When I add CRUD functionality for UI
Then the UI should display those elements on the webpage.

Hey team! Please add your planning poker estimate with Zenhub @bf2285 @haoruigao @MahamArif @Matt-Kou @SteambreadC


Fails when .netrc is not readable

/home/a/.rvm/gems/ruby-2.3.6-n64/gems/netrc-0.11.0/lib/netrc.rb:45:in `check_permissions': Permission bits for '/home/a/.netrc' should be 0600, but are 0 (Netrc::Error)

Any follow up to this @ab ?


Update README.md

Change cibuild.cmd to build.cmd since there is no cibuild.cmd and build.cmd appears to the the correct command to run.

Closing for now. See https://github.com/Microsoft/msbuild/pull/3022/commits/2cf5d09bb8c772fb9607e5031696e3c11fe612cd for why.

You're confused because this is super confusing. We recently changed the entry point for our builds. Now, CI servers start at build/cibuild.cmd, but humans should be directed to build.cmd in the root, so this change is good. I'm going to reopen and accept it.


How to augment images and their (multiple) masks simultaneously, using masks as labels for Keras model?

Hi,
I'm not sure I've seen a way to use this package for multi-class semantic segmentation. I am aware that using DataPipeline, I can accomplish the first part of this question (augment images and their masks in parallel). However, I don't know how I could train a Keras model with this generator, denoting the masks as labels.
I think the nicest solution to this would be to simply allow the "labels" argument of keras_generator_from_array to also be an "array-like matrix" just like the "images" argument. The function would then apply the same operations to "images" and "labels." This would be immensely helpful, as Keras does not deal with multiple masks at once very nicely.
Even allowing the "labels" argument to be optional would be helpful (for my task, a whole-image label does not make sense). Then I could create two Keras generators from the same Pipeline (one for input images and one for masks), then zip them and feed that to Keras' fit_generator.

Issue 1: Interpolation of mask after random rotation
I have tried augmenting images and the corresponding mask in parallel. After random rotation, intermediate values appear in the binary mask probably due to interpolation. There is no API for the user to choose Nearest Neighbor Interpolation for augmenting masks. I think that would be useful. Though, it is not a big issue, it is something the user should be aware of.
Issue 2: Severe misalignment between image and mask after random distortion
This is a bug
def augmentation(x_root, y_root):
    p = Augmentor.Pipeline(x_root)
    p.ground_truth(y_root)

    # TODO: Bug fix: Augmentor can not synchronize the random distortion between image and label
    p.random_distortion(probability=1, grid_width=4, grid_height=4, magnitude=8)

    # save augmented images
    num_of_samples = int(1e2)
    p.sample(num_of_samples)

HI @holste1 - I will work on something to be able to do what you mentioned for the next version. Recently I have not had much time for Augmentor development, sorry for the delay in replying. I have marked this issue as an enhancement so it will get priority.


HTTP OTA update feature (not with DEFINES on compile time) (IDFGH-6824)

Is your feature request related to a problem? Please describe.
Kind of. I'm developing using the Arduino framework based on IDF. I would like to use HTTP or HTTPS without cert verification but the HttpsOTAUpdateClass (based on esp_https_ota.c) only works with HTTPS with cert verification. You allowed users to avoid this cert check by providing a define "CONFIG_ESP_HTTPS_OTA_ALLOW_HTTP" but that define is not visible from Arduino stack (IDF comes precompiled).
By the way, log messages are enables too (I see them on the Serial ouput which is not nice if you are using that UART port for something else). There's no way to disable them. For example:
E (119505) esp-tls: Failed to open new connection
E (119506) TRANSPORT_BASE: Failed to open a new connection
E (119513) HTTP_CLIENT: Connection failed, sock < 0
E (119515) esp_https_ota: Failed to open HTTP connection: ESP_ERR_HTTP_CONNECT
E (119521) esp_https_ota: Failed to establish HTTP connection
>W                LB111E (132588) esp-tls-mbedtls: mbedtls_ssl_handshake returned -0x7780
E (132589) esp-tls: Failed to open new connection
E (132589) TRANSPORT_BASE: Failed to open a new connection
E (132598) HTTP_CLIENT: Connection failed, sock < 0
E (132598) esp_https_ota: Failed to open HTTP connection: ESP_ERR_HTTP_CONNECT
E (132605) esp_https_ota: Failed to establish HTTP connection

Describe the solution you'd like
I would like to be hable to set the behaviour setting the right parameters on the passing esp_http_client_config_t

if plain HTTP must be used (most inscure) (http url)
if HTTPS without cert verification must be used (insecure I know) (https url with cert == NULL)
if HTTPS with cert verification must be used (secure) (https url with cert != NULL)

If you allow those configurations on esp_http_client(), it would provide much more functionality for Arduino users.
Describe alternatives you've considered
Handle everything myself manually with WiFiClientSecure/WiFiClient  but I liked the idea of using a IDF standard function for this.

FYI, this config was renamed in https://github.com/espressif/esp-idf/commit/fbc9c729b296edd4b1bfcb18b58dc2a99913874b. Looping in @me-no-dev for Arduino support.

I tried defining CONFIG_ESP_HTTPS_OTA_ALLOW_HTTP and CONFIG_OTA_ALLOW_HTTP but it fails with error:
esp_https_ota: No option for server verification is enabled in esp_http_client config.
It seems that when using Arduino core the IDF comes precompiled?


Azure Function Error

Hi,
i was following the tutorial Calling the PnP provisioning engine from a site script ( https://docs.microsoft.com/en-us/sharepoint/dev/declarative-customization/site-design-pnp-provisioning )
The Azure Function produce an error while proceeding.
The PnP Powershell function are not recognised and i have the following error :

2019-03-02T21:00:39.838 [Error] Connect-PnPOnline : The term 'Connect-PnPOnline' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
at run.ps1: line 3

I uploaded the powershell files following the instruction at the section "Upload the PnP PowerShell module for your Azure Function"
Anny help ?^^
Category

[ ] Question
[ ] Typo
[X] Bug
[ ] Additional article idea

Hi,
I resolved the first bug following this guideline https://stackoverflow.com/questions/37724769/how-to-install-a-powershell-module-in-an-azure-function/39985646#39985646
But now i have another Error
While executing i have an error message telling that i do not have enough privilege to perform the provision :
Apply-PnPProvisioningTemplate

CategoryInfo          : WriteError: (:) [Apply-PnPProvisioningTemplate], ServerUnauthorizedAccessException
FullyQualifiedErrorId : EXCEPTION,SharePointPnP.PowerShell.Commands.Provisioning.Site.ApplyProvisioningTemplate
2019-03-02T21:56:46.255 [Error] Exception while executing function: Functions.ApplyPnPProvisioningTemplate. Microsoft.Azure.WebJobs.Script: PowerShell script error. Microsoft.SharePoint.Client.Runtime: Access denied. You do not have permission to perform this action or access this resource.
2019-03-02T21:56:46.615 [Error] Function completed (Failure, Id=d2f5cade-3d39-4176-b571-1dfc29924086, Duration=15289ms)

Looks like you don't have permission to execute the PnP Template from the Azure Function. Did you configure the app permissions correctly?
I would confirm you gave the app tenant permissions and also confirm the client id and client secret are correct when the script is running.


setClientCredential() errors when no clientCertificate property is set

Core Library
MSAL Node (@azure/msal-node)
Core Library Version
2.11.0
Wrapper Library
Not Applicable
Wrapper Library Version
None
Public or Confidential Client?
Confidential
Description
This commit stops checking if config.auth.clientCertificate is actually defined.
CC: @Robbie-Microsoft
Error Message
TypeError: Cannot read properties of undefined (reading 'thumbprint')
MSAL Logs
No response
Network Trace (Preferrably Fiddler)

[ ] Sent
[ ] Pending

MSAL Configuration
auth: {
    authority: 'url',
    clientId: 'guid',
    clientSecret: 'secret',
}

Relevant Code Snippets
N/A

Reproduction Steps
Try authenticating without setting the clientCertificate property in the auth
Expected Behavior
Auth using clientSecret
Identity Provider
Entra ID (formerly Azure AD) / MSA
Browsers Affected (Select all that apply)
None (Server)
Regression
2.10.0
Source
External (Customer)

It's about this code, where this.config.auth.clientCertificate can be undefined:
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/2ff18f23f0631507ccbf966478b152b02c9d25af/lib/msal-node/src/client/ConfidentialClientApplication.ts#L223-L226

@RoelVB Thanks for reporting this. What flow are you using? ConfidentialClientApplication, ClientCredentialClient, etc

Encountered this using ConfidentialClientApplication

Thanks, I'll be looking into this immediately. FYI, unfortunately, our unit tests are lacking in this area. My next task was actually to address the unit tests for this area.

@RoelVB Sorry, I'm unable to reproduce this. Are you acquiring a token via cca.acquireTokenByClientCredential? Can you share your acquire token code with me here?

@Robbie-Microsoft
Yes. I'm calling acquireTokenByClientCredential.
This is the code that works on 2.10.0, but not on 2.11.0:
https://github.com/SMTP2Graph/SMTP2Graph/blob/aaa5f5b8db38230b8280eb6714819625b7530ee9/src/classes/Mailer.ts#L21-L31
This is the code that requests the token:
https://github.com/SMTP2Graph/SMTP2Graph/blob/aaa5f5b8db38230b8280eb6714819625b7530ee9/src/classes/Mailer.ts#L151-L161
If you want you can clone that repo to test. From the top of my head:

Clone
npm i
Upgrade @azure/msal-node to 2.11.0
Add an .env file with the contents:

CLIENTID="01234567-89ab-cdef-0123-456789abcdef"
CLIENTSECRET="VGhpcyBpcyB2ZXJ5IHNlY3JldCE="
CLIENTTENANT="contoso"
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Run npm test:send
The error will be in logs/error.log

Per the code in the first link, you are providing clientSecret AND clientCertificate. You can only provide one OR the other (OR clientAssertion). Can you delete the clientSecret line and try again?
However, I don't know why you'd receive the error you posted above: "TypeError: Cannot read properties of undefined (reading 'thumbprint')". I'll look into this tomorrow.
Let me know if deleting the clientSecret line and using ONLY clientCertificate works.

Sorry, the supplied code can be a little confusing. clientSecret or clientCertificate is defined, not both. The other will be undefined.
I will probably be able to do some more tests in about 8-9 hours from now.

@Robbie-Microsoft
I did some more testing and found something interesting.
This works:
auth: {
    authority: '<url>',
    clientId: '<guid>',
    clientSecret: '<secret>',
}

This throws the error mentioned:
auth: {
    authority: '<url>',
    clientId: '<guid>',
    clientSecret: '<secret>',
    clientCertificate: undefined,
}

So as of 2.11.0 the code checks if the key exists, but not if it's actually defined.
The 2.10.0 code had this to make sure it's defined:
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/e6d18c5140359145e0142d4ee4854367fd5da078/lib/msal-node/src/client/ConfidentialClientApplication.ts#L224-L227
And because of this line, the issues is not caught while building (or by your IDE):
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/aa24330717b5e1f2f98c9d534d3766caefbaa452/lib/msal-node/src/config/Configuration.ts#L192
Because now all auth properties are typed as "defined".

I came to the same conclusion as you and then came back to this page to see that you posted what I was thinking :) . I was able to reproduce your error by passing this into the config for the ConfidentialClientApplication:
clientSecret: "fake_secret", clientCertificate: undefined,
I think this is technically an error on your end, as clientCertificate should not be passed in as undefined, even if it's not defined. Also, you're technically passing in both clientSecret and clientCertificate (even though it's undefined) when you should only be passing in one or the other (or clientAssertion). If you pass in both clientSecret and clientCertificate (expected values instead of undefined), then you would receive the expected error: "An application should at most have one credential".
I believe you would have prevented this error on your end if you typed your config as Configuration, the type from msal-node that ConfidentialClientApplication expects to be passed into it.
You could do something like:

define #msalClient without clientSecret or clientCertificate
if Config.clientCertificateThumbprint && Config.clientCertificateKeyPath
add clientCertificate to #msalClient
else if clientSecret, add clientSecretto@msalClient`
else, throw error

Regardless, the code here is not working as expected, and should throw an error if clientCertificate is passed in as undefined.
I will get a PR out soon that checks if clientCertificate is passed in as undefined and throw an appropriate error if so

I believe you would have prevented this error on your end if you typed your config as Configuration, the type from msal-node that ConfidentialClientApplication expects to be passed into it.

You mean this Configuration?
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/5eabb42fe253ca40be6ded4158e63da2b8a2c6ec/lib/msal-node/src/config/Configuration.ts#L121-L127
That's what I use and it accepts auth.clientCertificate to be undefined.
But for some reason the typing is changed here, and used internally, which will make you miss some potential issues:
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/5eabb42fe253ca40be6ded4158e63da2b8a2c6ec/lib/msal-node/src/config/Configuration.ts#L191-L197

So this method should transform Configuration to NodeConfiguration, but it's not guaranteed:
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/22651b30bce9244cce2d7ffcfa1a81e4c44be133/lib/msal-node/src/config/Configuration.ts#L210-L243
In my case this line:
auth: { ...DEFAULT_AUTH_OPTIONS, ...auth },

Actually does this:
auth: { ...{clientCertificate: {/* default config */}}, ...{clientCertificate: undefined} },

Which results in:
auth: { clientCertificate: undefined },

A fix for this could be:
auth: { ...DEFAULT_AUTH_OPTIONS, ...auth, clientCertificate: { ...DEFAULT_AUTH_OPTIONS.clientCertificate, ...auth.clientCertificate } },

But to be sure, you would also have to do this for DEFAULT_AUTH_OPTIONS.azureCloudOptions and the same for DEFAULT_TELEMETRY_OPTIONS.application

Using new ConfidentialClientApplication was working for me in @azure/msal-common: 14.12.0, but now in @azure/msal-common: 14.13.0 I'm getting errors "TypeError: Cannot read properties of undefined (reading 'thumbprint')".

However, I don't believe it's our intention to allow you to pass in clientCertificate as undefined.

TypeScript rules implies that clientCertificate?: {...} allows for undefined to be directly assigned to to the property. This is relevant when trying to set these values from a configuration and all the values are going to be set to something. The code should check for value existence, not just the object key being defined.

We encountered the same issue with the BotFramework SDK. We're using the client ID and client secret as the client credential. Below are the call stacks. And it seems there is no way to get around this.

@RoelVB I agree with your analysis about what the root cause is. However, this behavior (root cause) already existed before the regression, and the PR I linked fixes the regression.
I'm going to keep the PR as is, and have created a new GitHub issue to track the root cause: #7221


Subnet service tag

There is a service tag for "VirtualNetwork" but oftentimes we would like to restrict communication to only the subnet which the resource is deployed in. Currently we enter in the CIDR for the subnet by hand, but a service tag for the subnet which an NSG is attached to would be very helpful.

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

ID: 5e905802-0b08-4b3b-25ec-0f88c023bc32
Version Independent ID: f31325e5-49ea-b28c-336e-10f2a3fc12b4
Content: Azure security groups overview
Content Source: articles/virtual-network/security-overview.md
Service: virtual-network
GitHub Login: @jimdial
Microsoft Alias: jdial

@asasine Thanks for the feedback. We are actively investigating and will get back to you soon.

@asasine These issues are designed for issues with the docs. if you would like to request a feature, please leave your feedback Here.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Just Adding the apk file of Findissues Android Application

Enhancement
Made a simple application that we can use from our phone when we don't have our laptop with us.

I think it would be better to just open the repo link in the default browser. Because you first of all are not logged in to GitHub what is really annoying when you want to comment on the issue and the other thing is you can't use passkeys as a login method when using your in App browser version of GitHub. But other than that I like it 👍

And I don't know to really explain but there is this strange behaviour that when starting the app it shows issues who were created recently but not ass recently as the website shows them so for example when the most recent issue in the app is 8 minutes ago the most recent one on the website is 4 minutes ago. But what the real problem ist that when you click on a issue and then go back to the app all issues that are shown are from 1 day ago. Here is a video:
https://github.com/anand346/findissues/assets/79466158/427f7763-33f4-47bb-945c-b411d2e0298f
And the second thing is that I think it would be great if we could have something to reload the page like in the browser so just pull down to reload and show newer issues.

I will try to fix it 👍

@SpatzlHD I have fixed the issues that you have told me can you now take a look at the apk file ?

@SpatzlHD I have fixed the issues that you have told me can you now take a look at the apk file ?

I will take a look at it in something like 4 hours!


[CAPI] Move objects with force move label

Description of changes:
CAPI has a force move label that helps clusterctl identify if extra objects should be moved. This change will make --filter-cluster also pick up objects that have the force move label only if they are not a direct dependent of any cluster.
Objects that have force move and depend on cluster being filtered will be moved, Objects having force move and depend on cluster not being filtered for will be dropped from the tree and will not be moved.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve
/cherrypick release-0.16


Can't read over 127 holding registers

Sorry！ My english is pool.
21 Jun 12:48:39 - TypeError: "value" argument is out of bounds
at checkInt (buffer.js:1044:11)
at Buffer.writeUInt8 (buffer.js:1092:5)
at Object. (C:\Users\Kevin.node-red\node_modules\node-red-contrib-modbus\node_modules\node-modbus\node-modbus\handler\server\ReadHoldingRegisters.js:58:14)
at ontimeout (timers.js:380:14)
at tryOnTimeout (timers.js:244:5)
at Timer.listOnTimeout (timers.js:214:5)

Please, post your flow to read or try translate.google.com! ;)

I changed the  Quantity of [Automatic Read FC3] node in the example to 128.
This my flow.
[{"id":"fac3ce5e.ab6dc","type":"modbus-read","z":"ea701a17.43ed38","name":"Automatic Read FC3","showStatusActivities":false,"showErrors":false,"unitid":"","dataType":"HoldingRegister","adr":"200","quantity":"128","rate":"1","rateUnit":"s","server":"24f2a88d.c6acd8","x":265,"y":128,"wires":[["e209e1a3.a8351"],["2b8c015f.aafa1e","68632849.14f568","f99a6a7b.f67478"]]},{"id":"24f2a88d.c6acd8","type":"modbus-client","z":"","name":"Local Write VServer","clienttype":"tcp","bufferCommands":true,"stateLogEnabled":false,"tcpHost":"<IP_ADDRESS>","tcpPort":"10502","serialPort":"/dev/ttyUSB","serialType":"RTU-BUFFERD","serialBaudrate":"9600","serialDatabits":"8","serialStopbits":"1","serialParity":"none","serialConnectionDelay":"100","unit_id":1,"commandDelay":1,"clientTimeout":1000,"reconnectTimeout":2000}]
Thank you！

When I use the raspberry to read my HMI device when there is a similar problem, when I read the number is greater than 127, node output the number of the value will be wrong.
When I set the number to 128, the number of output values is 0. When set to 130, the number of output values is 2.
[{"id":"20b5ffca.4eb1","type":"modbus-read","z":"40cb00ad.b82bb","name":"","showStatusActivities":false,"showErrors":false,"unitid":"1","dataType":"HoldingRegister","adr":"1101","quantity":"130","rate":"5","rateUnit":"s","server":"94460b98.86a5e8","x":127.10000610351562,"y":149,"wires":[["e83d2013.827d6"],[]]},{"id":"94460b98.86a5e8","type":"modbus-client","z":"","name":"","clienttype":"tcp","bufferCommands":true,"stateLogEnabled":false,"tcpHost":"<IP_ADDRESS>","tcpPort":"502","tcpType":"DEFAULT","serialPort":"/dev/ttyUSB","serialType":"RTU-BUFFERD","serialBaudrate":"9600","serialDatabits":"8","serialStopbits":"1","serialParity":"none","serialConnectionDelay":"100","unit_id":1,"commandDelay":1,"clientTimeout":1000,"reconnectTimeout":2000}]
Thank you very much.

Quite simply, the Modbus protocol itself doesn't allow for reading more than 125 registers in a single function code 3 request.
In a Modbus request, only 1 byte is allocated to specify the number of Modbus registers requested, which means we don't have a way to request more than 127. A modbus message is limited to 256 bytes, which, when accounting for the 5 bytes that comprise the header and CRC , leaves us with 251 bytes for register values, or 125 2-byte register values.

Although this item is closed. The solution is plausible but I think the modbus code should be written in such a manner that the request will not quit node red... Suppose someone would love to write a 'virus' to quit all the modbus devices with this code than it's an easy job. I'll hope you could consider to change the code.


chore: update CODEOWNERS

:tada: This PR is included in version 0.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


add gateway address for the static route

This adds gateway address while configuring advertised static routes
on the client and endpoint interfaces. currently it's missing in the
kernel forwarder.
Signed-off-by: Periyasamy Palanisamy<EMAIL_ADDRESS>
Description
This change just retrieves corresponding gw ip address from the
IPContextobject and set it as gw ip address for the given route.
Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[x] Tested locally
[ ] Have not tested

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

/cc @JanScheurich


Add support for sending custom email to only the administrators

https://github.com/zulip/zulip/issues/13413

[x]  Add an option --admins-only to support sending an email to only the administrators of a realm (or server).

Testing Plan:
Added unit test, manually tested ./manage.py send_custom_email with and without new --admins-only flag.

I merged as the series ending with e1849b63c14f909bb81ff366876d427325756dbd, after squashing your commits, fixing the typo, and improving error handling.
Thanks @Wowol!
If you want to setup a production server in a VM somewhere (maybe using docker-zulip) to debug the last issue, that'd be awesome.


Cloud Provider should not default to External

With the current settings, --cloud-provider=external is set by default.
Node taints will be present with this setting and the cluster will not be ready to run various workloads.
There should be a variable that determines if this setting is applied in the templates/userdata or not.
The documentation for the variable should state what will happen one way or the other.
https://github.com/search?q=repo%3Aequinix-labs%2Fterraform-equinix-kubernetes-cluster external&type=code
Today, users must install CPEM, which will clear the taint. This is demonstrated in the example:
https://github.com/equinix-labs/terraform-equinix-kubernetes-cluster/blob/main/examples/cpem-add-on/main.tf
When adding this variable, the example should also toggle the new variable. This will result in the module including --cloud-provider=external.  CPEM can not be installed if this setting is not present.

Using a lightweight, unopinionated, toggle will allow this provider to be used in settings where CPEM is not desired or alternative L2 friendly cloud-providers (such as kube-vip-cloud-provider) can be used.


Polybar kills all children of processes it spawns?

So, I have a shell script that launches mpv in the background to connect to my remote MPD server:
$ cat $(which stream-mpd)
#!/bin/bash
pkill -f "mpv http://mysever"
mpv http://myserver &

If I launch this from the command line, everything works as it should, mpv runs in the background even if I close the terminal, etc. However, if I try to hook it up to my mpd module in polybar (so that I can actually listen to the stream that I am remote-controlling with the module) like so:
format-online = %{A3:stream-mpd:}<label-song> %{A}  <icon-prev> <icon-seekb> <icon-stop> <toggle> <icon-seekf> <icon-next>  <icon-repeat> <icon-random>

then something strange happens: the mpv process is killed as soon as stream-mpd exits. I can confirm this by adding a sleep 10 as the last line of my script: polybar will freeze up for 10 seconds since the script is running, but then once the 10 seconds are up, mpv is killed too! I can't explain why that happens - is polybar going out of its way to clean up the children of all the processes it spawns? Is there a way to accomplish what I'm after (which is to start a background task as an input trigger)?

I don't exactly know how polybar handles its child processes, but it makes sense that it cleans up after itself, otherwise you end up with a bunch of processes spawned by polybar which run in the background (not necessarily doing anything or outputting text to the bar without the bar listening for it) or you may even get zombie processes, if they're not handled correctly.
But what you want to achieve is possible inside the script, with one of the following ways:
nohup:
nohup mpv http://myserver &>/dev/null &

disown:
mpv http://myserver &
disown

setsid:
setsid mpv http://myserver &>/dev/null < /dev/null &

The setsid version is probably the one that will always work, it directly attaches the process to the init process. It is possible that if you use the nohup or disown variants that the mpv process is still terminated when the script is terminated but that may depend on the shell and on how the script is terminated.

Thank you for the super-thorough explanation. The setsid solution is indeed best. Cheers!

Just FYI, I have just pushed #2248 which will make polybar no longer kill children spawned through click (and scroll) commands. Programs launched through polybar now act more or less the same as if they were started from some application launcher.

@patrick96 I am on 3.5.0, and now, even the simplest command executed with mouse click (e.g. notify-send "hello" "world") leaves a zombie process. See the screenshot below. These processes do not seem to disappear after a while. They (obviously) are removed after polybar is restarted.

@pawel-szopinski Hmm, I though I tested this exact thing, but I guess I did not. I fixed this in #2285, this will be in the 3.5.1 release.

@patrick96 big thanks for the quick fix! I can confirm it is indeed fixed in 3.5.1.

Thank you!


Adding Generic Field

Adding one more field, as the textField does not support Term vectors and postion offset, which is required for phrase search

@sandeepreddy602  @chenlica  Please review these changes.  I had to add one more field because term vectors are not supported in the predefined fields. And since we are always calculating the span information, we should always use this Field instead of text Field.

@akshaybetala I didn't quite follow the introduction of "GenericField".  Can you explain both here and in comments what it is about?  Can we have a better name for it?

Per our discussion today, please remove this GenericField.  Instead, extend the existing TextField to enable positional indexing.


All playback fails

I'm submitting a ...

[ x ] bug report
[ ] feature request
[ ] support request

General infomration
Browsing works, video playback fails. ("License update not successful".)
Prerequisites

[x] Are you running the latest version?
0.14.2

Description
Steps to Reproduce
Select a show and try to watch it.
Context (Environment)
Brand new Vero 4k running OSMC 2017.09-2, kernel 14.29-37, Kodi 18.0-ALPHA1
Running dpkg -l | grep mediacenter-osmc prints
ii  vero3-mediacenter-osmc               17.8-118                           armhf        Media Center package for OSMC
But I am running Kodi 18-
Installation

[x] I installed the plugin via zip from the Releases page

Operating System

[x] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[ ] Linux (ARM)
[ ] Android

Debug log
https://gist.github.com/felixyz/9bfdf0e50b85a681cf1359a63f659c65

Currently a bug setting the correct ESN for Android. Fix will be pushed this evening.

Fixed in 0.12.5, you might need to clear (empty & save) the ESN field in your settings

Sorry, still not working for me:
21:52:55.783 T:4113760816 WARNING: Previous line repeats 36 times.
21:52:55.783 T:4113760816  NOTICE: VideoPlayer::OpenFile: http://localhost:57149/manifest?id=80087549
21:52:55.784 T:3697239024  NOTICE: Creating InputStream
21:53:00.622 T:3746427888   ERROR: Traceback (most recent call last):
21:53:00.623 T:3746427888   ERROR:   File "/usr/lib/python2.7/SocketServer.py", line 295, in _handle_request_noblock
21:53:00.624 T:3746427888   ERROR:     self.process_request(request, client_address)
21:53:00.624 T:3746427888   ERROR:   File "/usr/lib/python2.7/SocketServer.py", line 321, in process_request
21:53:00.624 T:3746427888   ERROR:     self.finish_request(request, client_address)
21:53:00.624 T:3746427888   ERROR:   File "/usr/lib/python2.7/SocketServer.py", line 334, in finish_request
21:53:00.625 T:3746427888   ERROR:     self.RequestHandlerClass(request, client_address, self)
21:53:00.625 T:3746427888   ERROR:   File "/usr/lib/python2.7/SocketServer.py", line 655, in __init__
21:53:00.625 T:3746427888   ERROR:     self.handle()
21:53:00.625 T:3746427888   ERROR:   File "/usr/lib/python2.7/BaseHTTPServer.py", line 340, in handle
21:53:00.626 T:3746427888   ERROR:     self.handle_one_request()
21:53:00.627 T:3746427888   ERROR:   File "/usr/lib/python2.7/BaseHTTPServer.py", line 328, in handle_one_request
21:53:00.627 T:3746427888   ERROR:     method()
21:53:00.628 T:3746427888   ERROR:   File "/home/osmc/.kodi/addons/plugin.video.netflix/resources/lib/MSLHttpRequestHandler.py", line 41, in do_POST
21:53:00.629 T:3746427888   ERROR:     self.wfile.write(base64.standard_b64decode(b64license))
21:53:00.629 T:3746427888   ERROR:   File "/usr/lib/python2.7/base64.py", line 93, in standard_b64decode
21:53:00.630 T:3746427888   ERROR:     return b64decode(s)
21:53:00.630 T:3746427888   ERROR:   File "/usr/lib/python2.7/base64.py", line 73, in b64decode
21:53:00.630 T:3746427888   ERROR:     return binascii.a2b_base64(s)
21:53:00.630 T:3746427888   ERROR: TypeError: must be string or buffer, not bool
21:53:00.634 T:3697239024   ERROR: AddOnLog: InputStream Adaptive: License update not successful (no keys)
21:53:00.634 T:3697239024   ERROR: AddOnLog: InputStream Adaptive: Initialize failed (SingleSampleDecrypter)
21:53:00.636 T:3697239024   ERROR: CVideoPlayer::OpenInputStream - error opening [http://localhost:57149/manifest?id=80087549]
21:53:00.637 T:3697239024  NOTICE: CVideoPlayer::OnExit()

With 0.12.6.
Again, I'm running on a Vero 4K (osmc).
I must admit I don't know how to clear (empty & save) the ESN field. In which settings? Or from CLI?

Navigate to the addons settings & there to the "Expert" session.
Just click on that ESN field & remove all chars. Then reboot Kodi or your device (just for the sake of safety) & it should work again.

I get exactly the same error using 0.12.7. I am also using a Vero 4k. Complete stacktrace:
19:18:44.505 T:3696784368   ERROR: Traceback (most recent call last): 19:18:44.505 T:3696784368   ERROR:   File "/usr/lib/python2.7/SocketServer.py", line 295, in _handle_request_noblock 19:18:44.505 T:3696784368   ERROR:     self.process_request(request, client_address) 19:18:44.505 T:3696784368   ERROR:   File "/usr/lib/python2.7/SocketServer.py", line 321, in process_request 19:18:44.506 T:3696784368   ERROR:     self.finish_request(request, client_address) 19:18:44.506 T:3696784368   ERROR:   File "/usr/lib/python2.7/SocketServer.py", line 334, in finish_request 19:18:44.506 T:3696784368   ERROR:     self.RequestHandlerClass(request, client_address, self) 19:18:44.506 T:3696784368   ERROR:   File "/usr/lib/python2.7/SocketServer.py", line 655, in __init__ 19:18:44.506 T:3696784368   ERROR:     self.handle() 19:18:44.506 T:3696784368   ERROR:   File "/usr/lib/python2.7/BaseHTTPServer.py", line 340, in handle 19:18:44.506 T:3696784368   ERROR:     self.handle_one_request() 19:18:44.506 T:3696784368   ERROR:   File "/usr/lib/python2.7/BaseHTTPServer.py", line 328, in handle_one_request 19:18:44.506 T:3696784368   ERROR:     method() 19:18:44.506 T:3696784368   ERROR:   File "/home/osmc/.kodi/addons/plugin.video.netflix/resources/lib/MSLHttpRequestHandler.py", line 41, in do_POST 19:18:44.506 T:3696784368   ERROR:     self.wfile.write(base64.standard_b64decode(b64license)) 19:18:44.506 T:3696784368   ERROR:   File "/usr/lib/python2.7/base64.py", line 93, in standard_b64decode 19:18:44.506 T:3696784368   ERROR:     return b64decode(s) 19:18:44.506 T:3696784368   ERROR:   File "/usr/lib/python2.7/base64.py", line 73, in b64decode 19:18:44.506 T:3696784368   ERROR:     return binascii.a2b_base64(s) 19:18:44.506 T:3696784368   ERROR: TypeError: must be string or buffer, not bool 19:18:44.547 T:4016042992   ERROR: AddOnLog: InputStream Adaptive: License update not successful (no keys) 19:18:44.547 T:4016042992   ERROR: AddOnLog: InputStream Adaptive: Initialize failed (SingleSampleDecrypter)
Clearing the ESN field and rebooting did not help for me. Any other idea how to fix this? Thanks a lot!


Input.is_mouse_button_pressed() is failing during double clicks

Godot version:
v3.0.5
OS/device including version:
Windows 10 on Lenovo Ideapad
Issue description:
When polling for input in _process(),  Input.is_mouse_button_pressed(BUTTON_LEFT) doesn't return true for a rapid second click. It appears as if some unseen double clicking functionality is blocking it.
Using is_action_* with the Input Map works as expected:
Input.is_action_pressed("left_click")
No other Input.is_* methods seem to have this issue, only is_mouse_button_pressed().
Steps to reproduce:
Run the line following in _process():
print(Input.is_mouse_button_pressed(BUTTON_LEFT))
Try doing two clicks in a row at different speeds. I found that, while it should be printing true whenever the button is down, it prints false during additional clicks that occur within <1s of the first.
Minimal reproduction project:
The above line of code is all that's required, I'm happy to put together a test project if necessary :)

Ran into this issue in 3.0.6 as well.


工具库工程&文档能力支持

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

可升级站点依赖<EMAIL_ADDRESS>


Ready to add a new city to the project - Rixensart

Hi,
I have replicated the EqualStreetNames project in my city, Rixensart in Wallonia, following the instruction in the README.md file.
I now would like to add it as new city to the project.
I think best would be that I keep the ownership of my repository.
My repository is here: https://github.com/vdarmsta/EqualStreetNames-Rixensart
Let me know if all is in order or if anything is missing.
Thanks in advance,
Vincent

Awesome!
You should have received an invitation to join @EqualStreetNames/rixensart team.
Once you've joined you should be able to transfer your repository to @EqualStreetNames organization and I'll take care of the next steps.

Don't forget to add the MAPBOX_TOKEN secret !

Don't forget to add the MAPBOX_TOKEN secret !

Hi Jonathan, I thought I already replied to you, but not sure it was sent or not.
Anyway, could you help me about how to create the MapBox_Token and also, where to add it? I haven't found that info in the README. What I have found is the link to the Mapbox doc. I followed it and I'm ready to create a token, but not sure what I have to select for the Token Scope and if I can add something in the token restriction (URL) or not (if yes, which URL can I put)?
Thanks in advance and sorry for the delay in my reaction.

[...] but not sure what I have to select for the Token Scope [...]

You can select everything.

[...] and if I can add something in the token restriction (URL) or not (if yes, which URL can I put)?

You can put https://rixensart.equalstreetnames.be in the list of URLs.

Anyway, could you help me about how to create the MapBox_Token and also, where to add it?

Once you have your token, you need to create a MAPBOX_TOKEN in your repository secrets.
https://github.com/EqualStreetNames/equalstreetnames-rixensart/settings/secrets/actions (in "Repository secrets")

Hi,
Ok, this is done.
Let me know if this works and if there is still something to do on my side.
Best regards,
Vincent
On 27/04/2022 08:39, Jonathan Beliën wrote:

[...] but not sure what I have to select for the Token Scope [...]

You can select everything.
image
https://user-images.githubusercontent.com/1150563/165456404-a1060b67-61d6-4cee-baef-33239ffefe41.png
[...] and if I can add something in the token restriction (URL) or
not (if yes, which URL can I put)?

You can put |https://rixensart.equalstreetnames.be| in the list of URLs.
Anyway, could you help me about how to create the MapBox_Token and
also, where to add it?

Once you have your token, you need to create a |MAPBOX_TOKEN| in your
repository secrets.
https://github.com/EqualStreetNames/equalstreetnames-rixensart/settings/secrets/actions
(in "Repository secrets")
—
Reply to this email directly, view it on GitHub
https://github.com/EqualStreetNames/equalstreetnames/issues/519#issuecomment-1110602395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGHIEIXTVZDKP4HH3KPV623VHDOLFANCNFSM5S72R7LA.
You are receiving this because you authored the thread.Message ID:
@.***>

Data update and deployment are on their way!
I'll take care of the DNS settings this weekend. 😃

🚀 Deployed: https://rixensart.equalstreetnames.be/


Set constraints for one off tasks

Fixes https://github.com/remind101/empire/issues/806
Fixes https://github.com/remind101/empire/issues/746
Broken by https://github.com/remind101/empire/pull/792/files#diff-a0afb4d13dc79c28cad59d370ba902caL66.
This adds the constraints back in, and adds the following:

You can now provide the -s flag with emp run to specify the constraints (e.g. emp run migration -a acme-inc -s PX)
Labels are now set on attached containers started with Docker.
Memory and CPU constraints are now set on attached containers started with Docker.

+1


Redash

Summary
Onboarding Redash. PostgreSQL and Redis are included by the redash community chart.

[ ] My change requires a change to the documentation and I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

@tony0306 I think we should actually focus on lightdash not redash

We can still put this in though, there are people who do want it

I think that I'm pretty close to finish it, once done I will move to lightdash as well. I guess next week this might be done as well.


Cow: Watch tasconnectlogistics.com.au

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

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

Approved by Jeff Schaller in Charcoal HQ


Can this be modeled on top of async hooks

Things like asynchronous operations at the very least could be expressed as (virtual) async resources. An interesting question would be if this is sufficient to express something like a "diagnostics channel" by filtering for specific async_hooks resources.

I believe that async_hooks is not enough by itself, because of issues with userspace queueing. Essentially, if I have code like the following:
// Async context 1
function foo() {
  doAsyncThing("data", function () { console.log('Called back eventually')})
}

// Async context 2
function bar() {
  doAsyncThing("more data", function () { console.log('called back after starting in a different context');})
}

function doAsyncThing() {
  //...
  manager.jobs.push({data, callback});
  //...
}

// Elsewhere, asynchronously
manager.onFreeWorker = function () {
  worker.jobs = this.jobs;
  this.jobs = [];
  worker.work().then((completedJobs) => {
    for (job of completedJobs) {
      job.callback();
    }
  }
}

What context are the callbacks invoked in? There are at least 3 different asynchronous contexts here (suppose the first 2 are different HTTP requests coming in, and the worker is communicating the jobs with an out-of-process worker like a database).
My understanding of async hooks would let you associate a single stack of calls together, for example the promises in onFreeWorker might be associated with the context that invoked onFreeWorker, but that isn't either of the 2 http contexts. And each callback invoked at the end is invoked synchronously, within a single async context, so how would they be distinguished and assigned to the correct original context?

I wasn't even talking about the async tracking, more about the data being emitted / the "diagnostics event bus". E.g. creating an AsyncResource that represents an individual mongodb interaction.
The userland callback queue can be implemented using the embedder API of async hooks so the context tracking shouldn't be an issue for properly instrumented libraries in node 8+.


Open-Data ?

Hello @mstuyts ,
Awesome work ! 👍
I do the same (kind of) work on my own ; I'll check if the WMS services I use are in your list.
IMHO, it would also be interesting to show if the WMS service provides open-data or not.

This project migrated to GitLab. This issue can now be found on https://gitlab.com/GIS-projects/Belgium-WMS/issues/1.


Local user fix

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md

Thanks for the fix, I'd remove the 2 lines above as those aren't needed anymore, and only bump the patch version of the chart.