Validation

Issue #, if available:
Description of changes:
Validation module. Permits to validate events and response against JSON schemas.
Checklist

[x] Meet tenets criteria
[x] Update tests
[x] Update docs
[ ] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

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

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

problem with git, will redo another branch...


Question on migrating to v3

I've got the following code running in v2.1.6.  I see the new 'anchor' concept.  What's the recommended way now to get children/parents as I'm doing below?  (assuming a child is the source and parent is target).
`        public PortModel InPort;  // Coming in from children
public PortModel OutPort;  // Heading out to parents
    public List<ObjectiveNodeModel?> ChildNodes
    {
        get
        {
            // Maybe I should return ID here, because I'm casting the basic node to an objectivenodemodel - things I've added like diagram, DataId won't be populated?  Or will they, because thats
            // what they really are?  Would have to test that one.
            // If it doesn't preserve the extras, return just Id then create a 'get node' method that you use as needed.

            if (usingPorts)
            {
                return diagram.Links.Where(x => x.TargetPort == this.InPort).Select(y => (ObjectiveNodeModel)y.SourcePort.Parent).ToList();

                //var incomingLinks = this.Links.Where(x => x.TargetPort == this.InPort);
                //return incomingLinks.Select(x => (ObjectiveNodeModel)x.SourceNode).ToList();
            }
            else
            {
                return diagram.Links.Where(x => x.TargetNode == this).Select(y => (ObjectiveNodeModel)y.SourceNode).ToList();
            }
        }
    }

    public List<ObjectiveNodeModel?> ParentNodes
    {
        get
        {
            if (usingPorts)
            {
                return diagram.Links.Where(x => x.SourcePort == this.OutPort).Select(y => (ObjectiveNodeModel)y.TargetPort.Parent).ToList();
            }
            else
            {
                return diagram.Links.Where(x => x.SourceNode == this).Select(y => (ObjectiveNodeModel)y.TargetNode).ToList();
            }
        }
    }`

Hello,
Very simple, check out this example:

Insteaf of: x.TargetPort == this.InPort
Use: x.Target is SinglePortAnchor spa && spa.Port == this.InPort

I am rewriting the website and documentation, once that's done 3.0.0 will be official, sorry for the trouble!

Great, thanks for that!


PullToRefresh implementation in Sliding Up Panel

Hi, I have some problem with using pullToRefresh libs.
When I sliding my ListView down, panel will starting collapse.
Thanks.

This is by design. You can't use pull to refresh inside of the panel. It's working as expected.


Google Summer of Code 2022

As in previous years, we will apply for GSoC 2022 under the umbrella of the Honeynet project. To get accepted by Google, we need to draft a convincing project idea for students to work on. My proposal would be to go for another "Pick-your-tasks" project as we did last year.
Let's use this issue to collect some ideas for tasks that are fun and easy to get started with. 😃 Of course, everyone is welcome to volunteer ideas.
Project Ideas
mitmproxy is a large project with a huge number of interesting areas to explore, down from low-level protocol work up to UX improvements. You know best what you are interested in, so why not get in touch with us and map out a custom GSoC project? Here are some ideas for inspiration:

Web UI (TypeScript/React.js): If you like to hack on front-end code, we have a vast array of ideas for mitmweb, our HTML frontend for mitmproxy. Just some ideas:

Add graphical editors for the map remote, map local, modify body, and modify headers settings.
Add body search filters
Improve the onboarding experience, e.g. by showing instructions on how to configure the proxy if no traffic has been captured yet.

Mitmproxy Core (Python):

Add a HAR Im- and Exporter to the core.
Many mitmproxy users want to use mitmproxy to just see all traffic of their current device without configuring individual applications, but that requires a complicated iptables configuration or undocumented scripts on Windows. You can make tremendous improvements to mitmproxy onboarding experience here! [#1261]
Many mitmproxy users only want to proxy one specific application. We already have a browser.start command to boot up a Chrome instance, it would be very useful to have similar wrappers for Firefox, Edge, Safari, Docker Containers, or Android Devices!
Add basic support for UDP Proxying

I do have a wish list and I think some of these things are both interesting and challenging enough for GSoC, but not too unrealistic:

async addon hooks with serialization (https://github.com/mitmproxy/mitmproxy/issues/4207). This would replace @concurrent so that addon hooks can be async and don't require a thread. This would allow hooks that take a longer time to not block the rest of the proxy. By "serialization" I mean that for a given connection client_connected is awaited before request is triggered. There should be no overlap or race-conditions compared to the regular blocking hooks
allow changing all options without restarting (https://github.com/mitmproxy/mitmproxy/issues/4269), e.g. start mitmdump without listening (--no-server), then use an add-on to start/stop listener on a given port
on top of the above: support multiple ports/configurations, also configurable live via addon without restarting the process (https://github.com/mitmproxy/mitmproxy/issues/4656)
on top of the above: expose all this in the mitmproxy UI (show a column with the listener port, allow adding/removing/enabling/disabling listeners)
improve guess_encoding, more tests, more robust (there was that one GIF server with UTF-8 encoding header). Accessing the text property should never crash
improved mitm.it onboarding, e.g. Android adb stuff. For first iteration it could list the adb commands step by step with required parameters pre filled. Next step would be automation. (https://github.com/mitmproxy/mitmproxy/issues/4838#issuecomment-965919966)
reduce PyInstaller size: not sure if this is realistic and interesting enough. But I wonder if a reduction by double digit % or even cutting it in half is realistic. I know the size comes from shipping with all dependencies, but how smart is PyInstaller? Do we ship files that are not actually needed?


Add ApolloCLI/ to .gitignore

Not adding the ApolloCLI folder to .gitignore causes this (very large) folder to get committed with your project. I have already added this to my own project, but think this should be added in the template, as well.
There may be a good reason not to do this, and, if so, please .gitignore this request.

Hi @joel-perry, I'd be happy to accept a PR if you've got the time.

@calvincestari Done


Platforms (OS) tested and know to run on.

Description
New users are in the dark about which platforms PyMarket is tested and known to run on: All?
There are no platform specific gotchas in the package dependencies?

I specified in the installation section that the library has been tested in Linux and Mac

I specified in the installation section that the library has been tested in Linux and Mac

I don't believe you can claim this is true.  See issue #11.  A user would really have to want to use this library to persevere through this setup.
Please update the install instructions to reflect:

What OS's pymarket has been tested to run on - So far it seems only Ubuntu Xenial 18.04?
The install instructions and minimum versions for the OS's listed in 1.

I don't understand why you don't believe in that claim.
There is a travis set up, running on linux and os: https://travis-ci.org/gus0k/pymarket/builds/597584718
As I explained while closing issue #11, that was a problem with the python version, not with the OS.

If you look at #11 you'll see it is a pip issue.
Your .travis.yml confirms that.
Nowhere do you state the latest pip has to be used, nor do you say how it should be upgraded.
Each distro has a 'supported' set of packages.  Each project has supported versions. Going outside of that generally means your on your own in terms of support contracts.
Please understand: very few users will preserver like this.

I updated the installation instructions to reflect how to upgrade pip and python version
https://pymarket.readthedocs.io/en/master/installation.html

I'm sure your users will appreciate the clarity.
Please change:

"PyMarket has been tested in Linux and mac OS."

to:
PyMarket has been tested in Ubutu Xenial (16.04) and mac OS.
In fact a find and replace /Linux/Ubuntu 16.04/, through out the code base is in order.

Done, I updated the docs with specific versions. Manjaro is my local development environment
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Wednesday, October 16, 2019 1:58 AM, Mark Van de Vyver<EMAIL_ADDRESS>wrote:

I'm sure your users will appreciate the clarity.
Please change:

"PyMarket has been tested in Linux and mac OS."

to:
PyMarket has been tested in Ubutu Xenial (16.04) and mac OS.
In fact a find and replace /Linux/Ubuntu 16.04/, through out the code base is in order.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


Footer

A proper footer should be made for the website to look good

I want to contribute to this section.

I would like to work on this issue. Can u plz assign me this issue

its done just you can modify the content


Updated python version 3.8 to 3.9

Required items, please complete
Change(s):

Python version updated 3.8 to 3.9

Reason for Change(s):

ICM

Version Updated:

3.9

Testing Completed:

Yes

closing this as zip is not correct will update in other PR


Add run queue lengths VM measurements

This PR adds two new VM measurements:

:total_run_queue_lengths - emits an event with a sum of run queue lengths, including dirty CPU run queue and without dirty IO run queue
:run_queue_lengths - emits an event for each normal run queue and dirty CPU run queue

@josevalim I know that you were concerned with efficiency here. We are not using erlang:statistics(run_queues) in the implementation, which is the inefficient method of getting the run queues length (probably because it fetches the lengths atomically, so maybe some kind of locking is involved).
 It’s already in the cod but I forgot to document it.


fix: correct textVariableAnchor property name

Fixed incorrect property name when setting text variable anchor offset in iOS symbol layer implementation. The property was incorrectly set as textVariableAnchor instead of textVariableAnchorOffset.

#145  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


gracefully handle ctrlc on windows by proactively the killing go process

Description
Closes https://github.com/vercel/turbo/issues/3427
An issue was introduced on windows by the go binary approach where on certain repos the go daemon process would not be killed correctly and hang. This fixes that issue by registering a ctrl-c handler
Testing Instructions
It is recommended that you download the kitchen-sink example on a windows machine. Start the daemon (the size of the repo seems to have an impact) and quit it. Before this change two ctrl-C handlers would fight for attention and the terminal would become unusable. After, one appears which you can dismiss with Y and then enter.

https://github.com/oconnor663/shared_child.rs/tree/master/src/sys

Yeah mostly to do with this note:

Compatibility note: The libc crate doesn’t currently support waitid on NetBSD or OpenBSD, or on older versions of OSX. There might also be some version of OSX where the waitid function exists but is broken. We can add a “best effort” workaround using waitpid for these platforms as we run into them. Please file an issue if you hit this.

Is it possible to write a test for this by sending a kill signal to the process? It might be good to have a baseline test if/when we run into this again

Yes, but we will hopefully be scrapping the go daemon in a week or two so I think we're ok to leave it out.


python37Packages.pybtex-docutils: 0.2.2 -> 1.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.pybtex-docutils is: "A docutils backend for pybtex"
meta.homepage for python37Packages.pybtex-docutils is: "https://github.com/mcmtroffaes/pybtex-docutils"
meta.changelog for python37Packages.pybtex-docutils is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.0.0 with grep in /nix/store/gkpypg9sprxgf3yy92wp3cf1jxzfh43p-python3.7-pybtex-docutils-1.0.0

directory tree listing: https://gist.github.com/3687c3ca9650af63de69a11e1a5600f2

du listing: https://gist.github.com/6b86acd560e77f314a265ad80083e442

Rebuild report (if merged into master) (click to expand)

24 total rebuild path(s)

6 package rebuild(s)

6 x86_64-linux rebuild(s)
6 i686-linux rebuild(s)
6 x86_64-darwin rebuild(s)
6 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python37Packages.pybtex-docutils
python37Packages.sphinxcontrib-bibtex
python38Packages.pybtex-docutils
python38Packages.sphinxcontrib-bibtex
python39Packages.pybtex-docutils
python39Packages.sphinxcontrib-bibtex

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gkpypg9sprxgf3yy92wp3cf1jxzfh43p-python3.7-pybtex-docutils-1.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.pybtex-docutils https://github.com/r-ryantm/nixpkgs/archive/bb2e0ce97d6a7dc84c67702dc99de9637732d485.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gkpypg9sprxgf3yy92wp3cf1jxzfh43p-python3.7-pybtex-docutils-1.0.0
ls -la /nix/store/gkpypg9sprxgf3yy92wp3cf1jxzfh43p-python3.7-pybtex-docutils-1.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages failed to build:

python37Packages.sphinxcontrib-bibtex
python38Packages.sphinxcontrib-bibtex
python39Packages.pybtex-docutils
python39Packages.sphinxcontrib-bibtex

2 packages built:

python37Packages.pybtex-docutils
python38Packages.pybtex-docutils

Maintainer pings

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109511 run on x86_64-darwin 1

6 packages built:

python37Packages.pybtex-docutils
python37Packages.sphinxcontrib-bibtex
python38Packages.pybtex-docutils
python38Packages.sphinxcontrib-bibtex
python39Packages.pybtex-docutils
python39Packages.sphinxcontrib-bibtex

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

python38Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

python39Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109511 run on x86_64-darwin 1

6 packages built:

python37Packages.pybtex-docutils
python37Packages.sphinxcontrib-bibtex
python38Packages.pybtex-docutils
python38Packages.sphinxcontrib-bibtex
python39Packages.pybtex-docutils
python39Packages.sphinxcontrib-bibtex

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

python38Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

python39Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109511 run on x86_64-linux 1

6 packages built:

python37Packages.pybtex-docutils
python37Packages.sphinxcontrib-bibtex
python38Packages.pybtex-docutils
python38Packages.sphinxcontrib-bibtex
python39Packages.pybtex-docutils
python39Packages.sphinxcontrib-bibtex

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

python38Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

python39Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109511 run on x86_64-linux 1

6 packages built:

python37Packages.pybtex-docutils
python37Packages.sphinxcontrib-bibtex
python38Packages.pybtex-docutils
python38Packages.sphinxcontrib-bibtex
python39Packages.pybtex-docutils
python39Packages.sphinxcontrib-bibtex

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

python38Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.

python39Packages.sphinxcontrib-bibtex:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;'and apythonImportsCheck`.


Fix table header when not responsive

Fix for #4222
WHY
BEFORE - What was wrong? What was happening before this PR?
Non responsive table kept a copy of the header, it always did, it's for a accessibility. Since the last changes on tables, I created a css rule that was forcing some padding on the first column, for better aligning, these rules conflict with this datatable hidden header, making it appear.
AFTER - What is happening after this PR?

How did you achieve that, in technical terms?
Those paddings are only added when the table is responsive.
Is it a breaking change?
Yes 😞, developers must republish the assets.
How can we test the before & after?
By enabling and disabling the responsive table.

Thank you @promatik !
Can confirm it works when unresponsive too, but the sort icons are "arrows" instead of "triangles" - why is that?

Also, when responsive, the sort icons now look wonky, they turned into a fat&short Star of David 😅

👀

Super funny it end up being like a Star of David 😅
Anyway, @tabacitu the changes I made shouldn't conflict with those icons, I updated this branch, and run a npm run prod, maybe the issue was that I was providing outdated sources. Can you check?
That should fix, but maybe you should run a build on your side.
By reading the changes once again, I'm pretty sure the issue is not in the code 😅

Mnope... npm run prod also doesn't fix the arrows for me:

on reponsive tables they're stars of David
on non-responsive tables they're arrows

So they're basically wrong in both instances.
--
Are you able to reproduce this @promatik ? If not, are you super-sure you're using this branch's assets?

@promatik @tabacitu the "pretty" icons are set by the responsive.datatables.js script, so when table is not responsive it will not use those icons. This has nothing to do with the change @promatik did here, it was happening before too.
Cheers

@promatik @tabacitu the "pretty" icons are set by the responsive.datatables.js script, so when table is not responsive it will not use those icons. This has nothing to do with the change @promatik did here, it was happening before too.

My suggestion with using responsiveTable instead of responsive still needs looking at @promatik , though - after that I think we're done here.

Tested, confirm it works, merging! Thanks @promatik .
For anybody stumbling into this, please note that in order to get the complete fix you need to do:
composer update backpack/crud
php artisan vendor:publish --provider="Backpack\CRUD\BackpackServiceProvider" --tag="assets" --force

in order to re-publish the CSS.

That's odd @Cocker ... I've just tried it again, and publishing the assets fixes it for me (with the public tag, indeed, sorry). I've also updated our demo to the latest version, and set the responsiveTable to false for FluentMonsters, so that you can see it should be working - https://demo.backpackforlaravel.com/admin/fluent-monster
Are you 100% sure you've re-published the bundle files too, and that the browser is loading them? Perhaps a Cmd+Shift+R / Ctrl+Shift+R (hard refresh) is needed in order to force the browser to re-fetch the cached assets?

@tabacitu This is what Fluent monsters looks or me:

Not all rows are aligned to left
My browser is Version 99.0.4844.82 (Official Build) (64-bit) on Ubuntu 21.10 if that helps
I did try a hard refresh, incognito mode both on my and your demo projects.

Ouch! @promatik - any idea why @Cocker is seeing the above?

@Cocker, this issue is now fixed on https://github.com/Laravel-Backpack/CRUD/pull/4490 👌
Thank you for the report and sorry for the delay.


Error when opening file with existing swapfile when using windowpicker

Opening a file for which you will be asked to open as readonly, delete swapfile, open readwrite, causes an error in neo-tree. The file is still opened as expected, but you have to dismiss the error message.
To reproduce:

Keep a file open in a secondary nvim instance
Start a new neovim and select the file in neo-tree
Answer neovim's builtin message window to open the file (e.g. select open as readonly)
select a window to open it in with the window-picker
experience the error message and dismiss it (the file is then opened as expected)

This is the message:
E5108: Error executing lua: ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:633: Vim(edit):E325: ATTENTION
stack traceback:
        [C]: in function 'cmd'
        ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:633: in function 'open_file'
        ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:482: in function 'open'
        ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:515: in function 'open_with_cmd'
        ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:640: in function 'open_with_window_picker'
        ...o-tree.nvim/lua/neo-tree/sources/filesystem/commands.lua:193: in function 'func'
        .../packer/start/neo-tree.nvim/lua/neo-tree/ui/renderer.lua:708: in function <.../packer/start/neo-tree.nvim/lua/neo-tree/ui/renderer.lua:706>

I think the latest commit in main fixes this problem. Can you give it a try and let me know?

Sorry, the bug still occurs. What's changed is that the message appears one step later now. Before the order was the following:

open file
answer neovim message
error
pick window
file opened

Now:

open file
answer neovim message
pick window
error
file opened

The message is slightly different due to changed line numbers:
E5108: Error executing lua: ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:644: Vim(edit):E325: ATTENTION
stack traceback:
        [C]: in function 'cmd'
        ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:644: in function 'open_file'
        ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:493: in function 'open'
        ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:526: in function 'open_with_cmd'
        ...t/neo-tree.nvim/lua/neo-tree/sources/common/commands.lua:657: in function 'open_with_window_picker'
        ...o-tree.nvim/lua/neo-tree/sources/filesystem/commands.lua:193: in function 'func'
        .../packer/start/neo-tree.nvim/lua/neo-tree/ui/renderer.lua:708: in function <.../packer/start/neo-tree.nvim/lua/neo-tree/ui/renderer.lua:706>

Sorry about that, the fix was incomplete before. Can you try again?

Now it works, thanks for fixing! :)


 (I did that both to get closer to the customer experience and because I often didn't have my phone available to 2-factor). I happened to login one day and discovered that the apps were costing quite a lot of money, and that was due mostly to the particular PostgreSQL tier chosen for the apps. I immediately deleted the apps, changed the PostgreSQL tier to the cheapest tier (consumption-based vs hourly), and resolved to never use my personal account again.
However, I'm concerned that azd makes it soo easy to deploy that other (non-MS) developers might azd up a template, forget to azd down, and accidentally run up a large bill.
I notice that in the Azure Portal, they sometimes display budget estimations. They definitely display it for PostgreSQL tier choosing, and it's very helpful for reasoning about tiers and for realizing that you may end up paying substantial $$.
I'm wondering if there's any possibility that the budget estimation information is available programmatically, so that azd could display something similar. Or, if you could at least warn when one of the resources has hourly based costs versus consumption-based costs. That could prevent other developers from having my experience.

Another short-term approach: In the review process, you could check that new templates use free or consumption-based tiers when possible. You could also add costs sections to the templates, like: https://github.com/pamelafox/fastapi-azure-function-apim#costs

We definitely want this but I don't think we have a robust way of doing this today so assigning to the backlog.


loadUserData throws exception if not logged in

When loadUserData is called before the user is logged in the following exception is thrown:
Caused by: org.json.JSONException: Value null of type org.json.JSONObject$1 cannot be converted to JSONObject
        at org.json.JSON.typeMismatch(JSON.java:111)
        at org.json.JSONObject.<init>(JSONObject.java:160)
        at org.json.JSONObject.<init>(JSONObject.java:173)
        at org.blockstack.android.sdk.BlockstackSession$loadUserData$1.onReceiveValue(BlockstackSession.kt:104)

This is due to the js method returning JSON.NULL which is converted to the string "null" that can't be parsed in the android method

Fixed thanks to @friedger 's pull request.


Demo text receiver doesn't work

I'm using edge 92.0.902.73.
And also not working on my iPhone, not muted, iOS 15. Can't see the console log, but apears the same message:
Sorry, it looks like this example is not supported by your browser. Please give permission to use the microphone or try again in Google Chrome or Microsoft Edge.

getUserMedia works for https and localhost. You can check the host.


What event triggered it?

I want it to trigger every five seconds
<vue-ellipse-progress animation="default 5000" :progress="progress"/>

@Guxuze Hi,
there are not triggers or events. If you want to "trigger" the initial animation, you can a) rerender the component or switch between b) loading or c) noData states.


Allowed booleans in requirements

I realized that I was thinking about this wrong, and that the pruning could be done in the constructors. The diff was also getting cluttered with unrelated changes, so this boils it down to just the boolean requirement stuff.

Actually, I was thinking something even better/worse, depending on how you look at it:
class OR:
    def __new__(cls, *args):
        if True in args:
            return True
        return super().__new__(cls, *args)

class AND:
    def __new__(cls, *args):
        if False in args:
            return False
        return super().__new__(cls, *args)

Would replace any creation of OR() with a True in it with just a True. And any AND() with a False with just a False. Then at the connection adding code, if a condition results in a full True or False, we can just make a condition less connection or no connection at all.
And where the list of requirements is constructed in the AND/OR constructors, we can just ignore True/False.

Oh, interesting, I didn't know new was a thing. Sounds good, I'll rework it.

It's one of those powerful but easy to abuse things. But in this case, I think it fits as it makes the logic still read clean, while getting a reasonable optimal solution.

Okay, that does feel better. I was wanting a way to keep the AND/ORs from being created in the first place without clutterieng the logic declaration, so __new__ is good to know about.

Don't mean to be a pest, but just checking in - is there anything you'd like me to change?

I have just been busy, I have not got around looking at it yet

Got it, no rush!


Added stringifiation for JsError

JsErrors from the play json parsing were being returned as JsResultExceptions,
which would then be sent as a string to the user, which is not a very
readable syntax error. This commit changes the behavior to use the syntax
shown on https://www.playframework.com/documentation/2.5.x/ScalaJson for
turning the JsError from parsing into a JSON string that can be used
as an error message.

Thanks!


Adding regexp expressions slows the linter by 2 secs or more

At least that's what I've observed in my codebase with go 1.18, which is currently about 5k lines of code.
Do you have any idea why? This is by far my favourite linter, it solves the biggest issue I have with golang.

This is weird and may be caused by complexity of your regexps.
We have a monorepo with codebase of 130k LOC, bare linter with no rexeps been running for 10 seconds, with one simple - 9.5 secs.
In future version cache for type regexp matching will be added (it reduces execution time for described codebase to 7 secs).
Ill dig into further optimisations later.

Sorry, this was a low effort issue I wrote. I think it was more related to golangci-lint and other linters, running exhaustruct by itself works great.


Add AnimationSystem and Sprite3dSystem system sets

The animation system updates texture_atlas.index. The sprite3d systems operate on texture_atlas.index. So I think that the sprite3d systems should run after animation.
This manifests in my project as me needing this system ordering:

In Update, I am changing the animation of my entity.
In PostUpdate, I am setting the Sprite3D anchor and custom_size but these values must be obtained from the current animation frame, and that requires using texture_atlas.index. This means this system needs to run after AnimationSystem. But it also needs to run before Sprite3dSystem because I am setting anchor and custom_size which Sprite3dSystem of course is looking at.

In project the use looks something like this:
app.add_systems(
    Update,
    update_animations,
    .run_if(in_state(AppState::MyState)),
);

app.add_systems(
    PostUpdate,
    update_anchor_and_size
        .after(AnimationSystem)
        .before(Sprite3dSystem)
        .run_if(in_state(AppState::MyState)),
);

Thanks 👌


Timezones in ubuntu based agent

We have tests running on the hosted linux preview agents that fail because timezones are missing.
When running a small test on the following docker images I get these results;
docker exec -it microsoft/vsts-agent:ubuntu-16.04-docker-1.12.1-standard

root@c7bad853c997:/var/temp# cat Program.cs
using System;

namespace temp
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Timezone test started.");
            var timezones = TimeZoneInfo.GetSystemTimeZones();
            foreach (var timeZoneInfo in timezones)
               Console.WriteLine(timeZoneInfo.Id);
            Console.WriteLine("Timezone test done.");
        }
    }
}
root@c7bad853c997:/var/temp# dotnet run
Timezone test started.
Timezone test done.
root@c7bad853c997:/var/temp# tzselect
/usr/bin/tzselect: line 180: /usr/share/zoneinfo/iso3166.tab: No such file or directory
/usr/bin/tzselect: time zone files are not set up correctly
root@c7bad853c997:/var/temp#

As stated in the following issue TZDATA has been removed from the upstream ubuntu image for space savings.
This however causes parts of dotnet to malfunction because it depends on this data being available.
The intended solution is to manually add the required package.

Fixed in #39

Merged #39.


Not able to Handle ipv6 with location in conditional access policy

Hi,
We had enabled conditional access policy to block login outside of the country
But we found that this policy not working for IPV6 enables users as well as iPhone devices.
Kindly take an action on the same for resolution
just ran into this morning. the client uses an iPhone and the builtin mail app. sometimes it talks over ipv6 and without the "unknown" picked on my "USA only" policy, it fails. It works fine when going over ipv4, but you can't control it.
I am curious Microsoft 365 have millions of customer and thousands of organization which is using  M365 and conditional access policy.
How they are handling this threat or situation?
How Microsoft handles this issue with ipv6 as most people also working Remotely so, there might be situation like this as well.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 74b164a7-31c5-7cb3-c8e2-7533d375bd43
Version Independent ID: 11399252-d935-2cba-a414-a44e51e67007
Content: Location condition in Azure Active Directory Conditional Access
Content Source: articles/active-directory/conditional-access/location-condition.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@anomepani Thank you for your query. We will investigate and update the thread.

@anomepani  For the unknown country to be included under these scenarios you will need to use this checkbox which is by default not selected. Many of the times, when the country named is not picked due to IPv6 address selecting the option will still apply to your CA policies.

Let me know if you have any questions.

Include Unknown countries/region option is not trustable. How can we identity user is actually from USA only and actual user is logged in not hacker?
If Some malicious user or suspicious login activity comes from untrusted locations how can we identify and verify that this is expected activity.
This small checkbox can introduce bigger risk at any organization for compromised user credentials.
Please suggest trusted and secure options

@anomepani  Currently the Country does not have IPV6 data, if you must use IPv6 with country specific input in request, then you might need to wait as this is a known gap and things will improve in near future.

@anomepani  We will close this now, if you have any follow up questions, please do tag me in your comment and we will be happy to help you.

@anomepani Currently the Country does not have IPV6 data, if you must use IPv6 with country-specific input in request, then you might need to wait as this is a known gap and things will improve in near future.

@vipulsparsh Could you suggest an alternative secure way to handle those users with Ipv6 internet so we can not open all unknown location but can limit lower surface for security impact. Any suggestion for secure organization with conditional access policy with this type of scenario most welcome.

@anomepani  May be you can use AAD Identity Protection. Have a look here, if you can visit this : https://docs.microsoft.com/en-us/azure/active-directory/identity-protection/overview-identity-protection

As we have Azure AD premiun 1 license, I don't think so this option available for us.
How other  pharma,  government, financial,  proprietary business organizations which is using office 365 will handle this type of situation.
There may be some secure alternative?


Gen 2 support?

Is it still the case that there is zero support for Gen 2 VMs? See for example http://www.aidanfinn.com/?p=21416

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

ID: 1b9c0d7a-bbf5-eccc-f2ca-a825bf42c82f
Version Independent ID: 58aa93af-4030-1541-a14c-c66c683ccf7c
Content: Prepare a Windows VHD to upload to Azure
Content Source: articles/virtual-machines/windows/prepare-for-upload-vhd-image.md
Service: virtual-machines-windows
GitHub Login: @glimoli
Microsoft Alias: genli

@jtuliani as the article mentions, we do support Gen2 but only in Azure Confidential Computing
https://azure.microsoft.com/en-us/solutions/confidential-compute/
For general Azure deployments we still do not support Gen2
Also, in order to create a Confidential Compute VM you cannot upload your own VHD. So this doc is still correctly.

Thank you for clarifying! Super stuff


feat: Add pt-Br translation

Added Brazilian Portuguese translation

@robsonpiere much appreciated, thank you!

Published in https://github.com/PenguinOfWar/strapi-plugin-paper-trail/releases/tag/v0.3.2 / https://www.npmjs.com/package/strapi-plugin-paper-trail


How should we represent in-flight spans in the UI

Currently, an in-flight span shows up as NaN (as it doesn't have a duration). Is this ok? how could we make it look better without misleading people into thinking an in-flight span is actually complete.

$ curl -s localhost:9411/api/v1/spans -H'Content-type: application/json' -d '[
  {
    "traceId": "77047bf9cf04ed5d",
    "id": "77047bf9cf04ed5d",
    "name": "get",
    "timestamp":<PHONE_NUMBER>945904,
    "duration": 156470,
    "annotations": [
      {
        "timestamp":<PHONE_NUMBER>945904,
        "value": "sr",
        "endpoint": {
          "serviceName": "client"
        }
      },
      {
        "timestamp":<PHONE_NUMBER>102374,
        "value": "ss",
        "endpoint": {
          "serviceName": "client"
        }
      }
    ]
  },
  {
    "traceId": "77047bf9cf04ed5d",
    "id": "40ef4e93ab3b1507",
    "name": "get",
    "parentId": "77047bf9cf04ed5d",
    "annotations": [
      {
        "timestamp":<PHONE_NUMBER>100852,
        "value": "cs",
        "endpoint": {
          "serviceName": "client"
        }
      }
    ]
  }
]'

cc @dragontree101 @cburroughs

In Lens, in-flight spans are shown correctly (no Nan) but there is no visual indication either that it's in-flight. A subtle hint or mouse-over that the request is in-flight is probably still useful.


Discrepancy in return value for SADD

Using redis-rb:
[0] pry(main)> Redis.new.sadd "temp", ["123"]
=> 1
[1] pry(main)> Redis.new.sadd "temp", "12"
=> true

Using mock_redis:
[0] pry(main)> MockRedis.new.sadd "temp", ["123"]
=> true
[1] pry(main)> MockRedis.new.sadd "temp", "12"
=> true

This is probably because of here:
https://github.com/brigade/mock_redis/blob/master/lib/mock_redis/set_methods.rb#L10
I guess this is also an issue with redis-rb. Since the return value of SADD should always be an integer.

Fixed in #109.


The type or namespace name VisualStudio' does not exist in the namespace Microsoft'.

error CS0234: The type or namespace name VisualStudio' does not exist in the namespaceMicrosoft'. Are you missing an assembly reference? screen shot: https://fourq-dropshare.s3-us-west-2.amazonaws.com/v5OkL1EJlI
namespace NextGenAPI.Tests.EntityBuilders
{
    using System;

    using Microsoft.VisualStudio.TestTools.UnitTesting;

    using NextGen.DAL.Models;
....

@CodisRedding Have you figured thi out?

Maybe you still do not have the sdk for visual studio installed.
Go to https://my.visualstudio.com/downloads and look for the SDK as "visual studio 2015 sdk" download and install.

After installing the SDK add a reference to the required assemblies to your project as described here.
Good luck.


WIP: ConvertFrom/To-CliXml

Fix #8311

PR Context

PR Checklist

[ ] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[X] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Please don't mark milestone until merged


stm32: mco: Remove deprecated Kconfig configuration

change the configuration of the MCO 1 or 2 output for the stm32 series
Kconfig is no more used, set the configuration by the DTS  as speciied by the  dts/bindings/clock/st,stm32-clock-mco.yaml
See the refman of the stm32 to put the correct clock source and the prescaler value
&mco2 {
	clocks = <&rcc STM32_SRC_PLLI2S_R MCO2_SEL(1)>;
	prescaler = <MCO2_PRE(5)>; /* div by 5 --> set 0b111 in the MCO2PRE of the RCC CFGR */
	pinctrl-0 = <&rcc_mco_2_pc9>;
	pinctrl-names = "default";
	status = "okay";
};

Fixes https://github.com/zephyrproject-rtos/zephyr/issues/80885

add the MCO prescaler : division factor  as common to stm32 series, not defined yet all the stm32 series
Use the same LL_RCC_PLLI2S_ConfigDomain_I2S to configure the PLL I2S, only the div M factor changes depending on the stm32 serie

@FRASTM Please rebase to fix CI issue

rebased on 065bd322200ccf2f9d717fd258ec966a522ce733

rebase on  af68d97507188249e2f9677119fc716a4d04989e

@nordic-krch PTAL

All changes are in ST specific file, reassigning.

Area matches: {<Area STM32 Platforms>: 20, <Area Drivers: Clock control>: 8, <Area Release Notes>: 1, <Area Samples>: 0}
labels: {'area: Clock control', 'platform: STM32', 'area: Samples', 'Release Notes'}
collab: ['erwango', 'FRASTM', 'gautierg-st', 'GeorgeCGV', 'marwaiehm-st', 'mathieuchopstm', 'nordic-krch', 'fabiobaltieri', 'kartben', 'nashif']
Submitted by: FRASTM
candidate maintainers: {'erwango': 20, 'nordic-krch': 8, 'fabiobaltieri': 1, 'kartben': 1}
Picked assignees: ['nordic-krch'] (27.59% ownership)

shrugs


aws-alb-ingress-controller deletes manually added listeners and security groups

Hi Team,
I manually added some listeners in the ALB to expose one of my service.
I found teh listener was automatically deleted by ingress controller.
Similarly, I added security group also for the load balance.It also got deleted.
Can you please help me here?

The ALB Ingress Controller is driving the desired state of ALB to match the specified spec on Ingress Object. So any direct modification to ALB via AWS API will be overwritten.
For SecurityGroups, you can use alb.ingress.kubernetes.io/security-groups annotation to control securityGroups, the draw back is you need to modify securityGroup on nodes on allow inbound access from the ALB securityGroups you specified.
For Listeners, you can use alb.ingress.kubernetes.io/listen-ports annotation to control the listen ports(if all listen ports have same routing rule/backend.  If it's different routing rule/backend, there is an alpha version to let u do that: https://github.com/kubernetes-sigs/aws-alb-ingress-controller/releases/tag/v1.2.0-alpha.1.
Also, i think an additional annotation like alb.ingress.kubernetes.io/listen-ports-unmanaged maybe a good idea.

Thanks.
Can you please tell me when can we use this in production  https://github.com/kubernetes-sigs/aws-alb-ingress-controller/releases/tag/v1.2.0-alpha.1?
We really have a use case where we expose different microservices on different ports.


Add IQaudIO Pi-DAC+ to configuration table

Change-type: patch
Update DAC configuration table to include the IQaudIO Pi-DAC+, which works using dtoverlay value iqaudio-dacplus
Hey there, appreciated contributor!
Have you taken a look at the contribution guidelines for this project? To enable our CI to work correctly and version the releases of the project, you need to ensure your commit messages contain the required information detailed here. Thank you!

Hey @michaellwilks this is great, thanks! You'll need to update the git commit message to include the change type footer (not the Github comment), then the CI will pass and we can merge this

We'll need to bump the number to 13 now, too

Glad it worked Mikey!
I fixed the indexes.

@balena-ci retest


book: Update step on getting gtk4 crate version

Use placeholder value that follows the semantic versioning and find gtk4 crate version using crate search.

Thanks for your contribution @kianmeng .
There is indeed room for improvement, but I think there is a better alternative to your PR.
We should instead replace all cases where dependencies are introduced with calls of cargo add.
Are you interested in picking this up?
If yes, make sure to disable your markdown formatter to avoid big diffs.

@Hofer-Julian Sounds good. Should I close this PR and create a new one?


Internal server error

I've been trying to install it over Linux, Docker and Docker compose but i'm always getting Internal server error. What could i do?

Same page displayed for me. Here are the logs from the docker container.
[2024-12-07 01:08:21,245] ERROR in app: Exception on / [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 2528, in wsgi_app
    response = self.full_dispatch_request()
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 1825, in full_dispatch_request
    rv = self.handle_user_exception(e)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 1823, in full_dispatch_request
    rv = self.dispatch_request()
         ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/flask/app.py", line 1799, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/ytdlp2STRM/ui/routes.py", line 15, in index
    crons = _ui.crons
            ^^^^^^^^^
  File "/opt/ytdlp2STRM/ui/ui.py", line 97, in crons
    with open(self.crons_file, 'r') as file:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: 'config/crons.json'

Did you create a blank crontab file?

Did you create a blank cron file?

Please @xJCPMSx share the console output error or something similar. You can check https://github.com/fe80Grau/ytdlp2STRM/issues/88#issuecomment-2524736274, it may be a problem related to some configuration file


Add /snowflake command

This command allows you to generate timestamps by giving it a snowflake id. Below is a gif of how it works:

(this pr does not include context menu commands)
Because discord snowflakes exceed the max limit of the ApplicationCommandOptionType.Number I had to resort to a string input. As you can see in the gif above, invalid snowflakes will be dealed with properly.
I have no idea if I implemented this properly. I tried to keep to the current structure as much as possible. I'm mainly worried about the translations not being done correctly, since I just added them to src/locales/en-US/translation.json. Feel free to change anything you don't like. Any feedback is welcome!

Implements #31


sys: windows: use golang.org/x/sys/windows

similar to https://github.com/moby/moby/pull/33399 and https://github.com/moby/moby/pull/40645

Codecov Report

Merging #4093 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4093      +/-   ##
==========================================
- Coverage   42.54%   42.53%   -0.01%
==========================================
  Files         130      130
  Lines       14828    14828
==========================================
- Hits         6308     6307       -1
- Misses       7600     7602       +2
+ Partials      920      919       -1

Flag
Coverage Δ

#linux
45.89% <ø> (-0.01%)
:arrow_down:

#windows
38.22% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

snapshots/devmapper/pool_device.go
55.69% <0%> (-0.41%)
:arrow_down:

Continue to review full report at Codecov.

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


nvmeof-recoverer: implement node selection for reassigning failed OSD pods

Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

need to resolve the conflict

I don't totally understand this PR.

Oh sorry, I fixed it


copr: set Expr.tp in RpnFnScalarEvaluator

What have you changed?
Please explain in detail what the changes are in this PR and why they are needed:
In RpnFnScalarEvaluator, evaluate_raw will construct some Expr then evaluate it for test, but only Expr.field_type is set correctly, Expr.tp is left to be default value (NULL).
If some builtin_function try to optimize by the expression node's type, then the test will break.
What is the type of the changes?
Pick one of the following and delete the others:

Bugfix (a change which fixes an issue)

How is the PR tested?

Unit test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Any examples? (optional)

/test

/merge


Mixit recipe

Simple MixIT recipe on WHAM clean using always MoMs with 4 speakers.

One thing could be confusing is having this in WHAM. We use WHAM clean and MixIT works only on WHAM clean.
Because WHAM clean is same as WSJ0-2mix we might want to move this there.

One thing could be confusing is having this in WHAM. We use WHAM clean and MixIT works only on WHAM clean.
Because WHAM clean is same as WSJ0-2mix we might want to move this there.

Didn't think about that.
We can hard code the task in the train file.. And the README is pretty clear so I think it's fine.

It's very frustrating to copy paste all this content for a "simple" MixIT example though.

I agree, I can use symlink maybe for the whole local folder.
But conf.yml has to be different so it has to be moved somewhere else. Like in a conf directory as Kaldi does.

Probably can be improved a lot by using a custom dataloader and do the "magic" there by creating MoMs with not always 2 speakers. But this recipe it is just a proof of concept and should be enough to show MIxIT.

But this recipe it is just a proof of concept and should be enough to show MIxIT.

Completely agree. This is enough IMO
I'm waiting for the comment in the eval file (and the pl module maybe) and I'll merge.

Added comments on both eval.py and validation_step


allow configuring externalModules for typescript bundler

e.g. hnsw-lib, currently hardcoded

closing for now, can re-open later if desired


App still informs about booster vaccination

Technical details

Device name: Samsung A5 (2017)
Android version: 8.0
App version: 2.13.2

Describe the bug
After importing the certificate of a booster vaccination, the app still informs about the missing booster vaccination.
Only the number of dates differs since the last vaccination differs to this screenshot
https://www.coronawarn.app/assets/img/blog/2021-11-04-booster-notifications/benachrichtigung-johnson.png
Steps to reproduce the issue

Get vaccinated with Johnson
Import the certificate
Get vaccinated with Comirnaty
Import the certificate

Expected behaviour
The information about the missing bosster vaccination should go away
Additional context
May be related to #4269 #4343 #4334

Internal Tracking ID: EXPOSUREAPP-10232

@DerVogel2020 thanks for raising this. The text should disappear within 24 hours. For several technical reasons, this logic is currently only checked once a day and unfortunately not when a new certificate is registered in the app. We're working on it...

Same as #4334?

@Ein-Tim Almost, since there the comirnaty vaccination did not become valid immediately. Here the validity and priority etc  is fine, only the info text should disappear.

@DerVogel2020 thanks for raising this. The text should disappear within 24 hours. For several technical reasons, this logic is currently only checked once a day and unfortunately not when a new certificate is registered in the app. We're working on it...

@mlenkeit thanks. Now (after less than 16 hours) the message is gone and replaced by "Sie haben nun alle derzeit geplanten Impfungen erhalten. Ihr Impfschutz ist vollständig."

Still getting the message, even 19 days later

This issue also occured for my mother, who had Biontech Comirnaty all 3(!) times. She gets the booster notification even though she was boostered 74 days ago, which is 3/3 vaccinations for her.

(images were shared with her full consent)

@laundmo
The fix for this issue is showing up in pre-release version v2.15.0-RC0.
Cancel booster indication (EXPOSUREAPP-10232) #4383
Version 2.15 has not yet been released.

There was an announcement this morning for the release of CWA 2.15. See https://www.coronawarn.app/en/blog/2021-12-20-cwa-2-15/.

I closed this now, because 2.15 is available.

I closed this now, because 2.15 is available.


Question about frustum loss

Hello, I have a question about your work.
I noticed that the config of symphonies(symphonies.yaml) includes a ‘frustum loss’ and that frustum loss is measured. However, the paper does not provide an explanation about frustum loss. Will the explanation be added to the accepted paper?

Uh-oh, I' think we will remove the frestum loss from the config, it doesn't influence the performance according to our experiments.
BTW, the frustum loss is introduced by MonoScene.
Thanks for your carefulness : )

I got it. Thank you for the quick and kind answer!


[1.9] Implement EnderDragon.getHealingCrystal().

SpongeAPI | SpongeCommon
I heard @gabizou say some time ago that DataManipulators are no longer allowed to store Entitys. So instead of implementing HealingSourceData, I just moved it into EnderDragon and implemented it there.

@Zidane It looks like you merged the API changes; could you also get the implementation merged in? I fixed the merge conflicts.


Network worker sometimes freezes

It seems that, occasionally, the processing of this loop takes more than a few minutes: https://github.com/paritytech/substrate/blob/f6d66db4a44596630943b4905e871125b9a490c6/client/network/src/service.rs#L1417
We should figure out what's happening precisely.
For note keeping, this happened on Parity's kusama-sentry-ew4-0 on August 12th from 17:10 to 17:15 Berlin time.

Should be fixed by https://github.com/paritytech/substrate/pull/6903


Group is created twice, gets a "phantom peer" assigned and can't be deleted

Describe the problem
I tried to create a new Group during the creation of an Access Control Policy. The policy was supposed to have that same group as Source and Destination.
After doing so, I got not one but two new Groups (both with exactly the same name) under "Settings -> Groups".
Moreover both those Groups allegedly have a Peer assigned and therefore can't be deleted. Just that there is not one single Peer existing in that Netbird instance.
So I also can't get rid of the duplicated group because in order to do so I'd have to delete a "phantom peer" first, which I can't because there are 0 peers.
To Reproduce
I have not yet been able to reproduce the behaviour.
My (unproven) assumption is, that somehow by using the same group name in the Source and the Destination field I could trigger the creation of the same (or rather an identically named) group twice. Maybe because the creation of the "first" group (Source field) hasn't been finished when I entered the same name in the Destination field. But as I said: I couldn't reproduce it, so that's just a speculation.
Expected behavior
I expect not to have duplicate groups and phantom peers in my Netbird instance.
Are you using NetBird Cloud?
self-hosted
NetBird version
0.25.6
Screenshots

Here you can see the same group twice, with each a peer assigned. (Don't mind the assigned setup-key, that one I added later but it had no influence on the situation.)

Here you can see that there are no peers in that same instance. (In fact at the point where the duplicate groups were created, there never had been a peer in that freshly installed instance.)
Additional context
I'd love to know how to get rid of the duplicate groups.

Thanks for the (again) very fast fix.
I can confirm that after the update the "phantom users" disappeared and the duplicated groups could be deleted.


Switch pynvml to nvidia-ml-py

Fixes: #5517
Link to PyPI for nvidia-ml-py here showing usage remaining the same as previous pynvml package.

A6000 test running successfully as well:
https://github.com/microsoft/DeepSpeed/actions/runs/9071316498/job/24924771808


[ON-HOLD] - feat[swa-271] make SubmitArtworkOverview the standard flow

The type of this PR is: feat
SWA-271
Make SubmitArtworkOverview the standard flow
Echo PR https://github.com/artsy/echo/pull/210
docs https://github.com/artsy/eigen/blob/main/docs/developing_a_feature.md
For follow up things to do, please check SWA-270

This PR resolves []
Description

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

ⓘ 'User-facing' changes will be published as release notes.

Cross-platform user-facing changes

Rename modal title - patrinoua

iOS user-facing changes

Android user-facing changes

Dev changes

This PR contains the following changes:

Cross-platform user-facing changes (Rename modal title - patrinoua)

  Generated by :no_entry_sign: dangerJS against 7e28ce74405b525510cabb0c7e666c4079aea4be

Thank you @AlicanAkyuz 🙏 I've updated the PR can you maybe have another look?

all good with the release. it was cut from main a few commits back, so no worries. we can merge when we want.
in general, main should be pretty much unblocked from releases. we can merge stuff to main whenever we want. what we are trying now is to cut a release from main (which means making a new branch from main), and only adding hotfixes etc on there, so we should be very much unblocked from main and releases 👍

@pvinis we are still unsure if we want to release this in the next release as we are still working on some bugs/improvements. does it still make sense to merge now?

ah gotcha, you mean next release. I would suggest (if possible) to put things behind a flag. that way we can merge and not have it released/visible to users. if a flag wouldn't work, then holding the PR is ok, but it's the harder option I feel like.

understood, thank you. since this is removing a flag already, it looks like keeping it on hold is the only option.

oh, and I just checked the changes, and it's actually the flag! 😅
if we are not sure we will be releasing this, we have some options:

close the PR and open it when we are sure
keep the PR until we are sure
connect that flag to echo, turn the flag in echo to false, them merge this PR here. this way we control the release from echo.

up to you and your team :)

I'd say let's keep it on hold as-is if there aren't other opinions. in the next couple of days it'll be clearer if we want to release this in the next release.

Replaced by https://github.com/artsy/eigen/pull/6399

Replaced by https://github.com/artsy/eigen/pull/6399


Broken link

On github ...

After clicking the provided link ...

Hello @neokio . Thanks for your issue! We are changing the website structure and some links are not working, sorry. We will fix it ASAP.


Awaiting Terminal processId never returns value on Windows

The PowerShell extension creates a console using the vscode.window.createTerminal API and then tries to log its PID with terminal.processId.
A simplified version of this startup might be something like:
const pwshTerminal = vscode.window.createTerminal({
    name: 'PowerShell Console',
    shellPath: 'C:\\Program Files\\PowerShell\\7-preview\\pwsh.exe',
    shellArgs: ['-NonInteractive', '-NoProfile'], // Options suppress things like PSReadLine, so simplify repro
    hideFromUser: false});

pwshTerminal.processId.then((pid) => console.log(pid));

In https://github.com/PowerShell/vscode-powershell/pull/2482 I changed our then() call to an async/await version, like:
const pwshTerminal = vscode.window.createTerminal({
    name: 'PowerShell Console',
    shellPath: 'C:\\Program Files\\PowerShell\\7-preview\\pwsh.exe',
    shellArgs: ['-NonInteractive', '-NoProfile'], // Options suppress things like PSReadLine, so simplify repro
    hideFromUser: false});

const pid = await pwshTerminal.processId;
console.log(pid);

This led to https://github.com/PowerShell/vscode-powershell/issues/2491, where the extension client would not progress past this await call (as discovered with log points).
As I say in https://github.com/PowerShell/vscode-powershell/issues/2491#issuecomment-593500969, I can reproduce this with the following change from https://github.com/PowerShell/vscode-powershell/commit/d431617d6997c75fdce35342323e016f1bbaf271:
diff --git a/src/process.ts b/src/process.ts
index aa957b5..b70966e 100644
--- a/src/process.ts
+++ b/src/process.ts
@@ -128,7 +128,9 @@ export class PowerShellProcess {

         // Log that the PowerShell terminal process has been started
         this.log.write("Registering terminal PID log callback");
-        this.consoleTerminal.processId.then((pid) => this.logTerminalPid(pid, pwshName));
+        const pid = await this.consoleTerminal.processId;
+        this.logTerminalPid(pid, pwshName);
+        // this.consoleTerminal.processId.then((pid) => this.logTerminalPid(pid, pwshName));

         return sessionDetails;
     }

So startup pauses, since the client never opens a named pipe. Debugging the server shows it waiting on a named pipe connection.
In the client, pausing the debugger shows us stopped here:

Log points on each line showed us progressing through the startup code as expected until awaiting the processId value of the terminal, at which point we stopped progressing.
We worked around this issue https://github.com/PowerShell/vscode-powershell/pull/2498 by moving back to the .then() (I suspect there are startups where we never log the PID).
I've only seen this on Windows. My machine's OS version, where I've managed to reproduce:
OS Name:                   Microsoft Windows 10 Enterprise Insider Preview
OS Version:                10.0.19535 N/A Build 19535

This is running VSCode 1.43.0-insider (user setup).

I know the repro steps here aren't ideal. If there's any advice on how to work with the debugger to open up closures or trace the issue in some other way, I'm happy to follow that to dig deeper in my repro environment.

Also experiencing this. I've tried @rjmholt's work around but my then is never resolved and my extension load just hangs.


Fix istioctl version information (blocking release)

The version library moved to another repo, so we need to update the
location here. ldflags locations are relative to the GOPATH, so we have
to do the long vendor directory, not the stanard istio.io/pkg.
Fixes https://github.com/istio/istio/issues/14182

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: douglas-reid
If they are not already assigned, you can assign the PR to them by writing /assign @douglas-reid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

bin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

This won't work once the vendor directory gets removed. I don't think it's imminent, but we will be setting ourselves up for failure at some point.

This is the only change that works right now. It is not possible to make a fix that works with and without vendor AFAIK. And we need this ASAP as it is blocking releases.
Also I don't think we decided to remove vendor

I recall one of the calls where we mentioned going to go mod and the removal of vendor at some point in the future (I recall it was after we got the CI working with go mod instead of vendor). But it's not imminent, and maybe with things like splitting repos, it will be harder to actually do (obviously, we would need a fix for this).
I don't have a better solution at the moment either.

/lgtm
Small cosmetic comments:
On the title (blocking release), I think we could/should use a label release-showstopper or something similar. It'd help a lot to triage these as P0 automatically after review. Thoughts? Maybe @geeknoid
has already a process that describes these showstopper interaction.

The version library moved to another repo
Is there a specific commit that created this problem (would be good to refer to it here for tracking purpose).

This is the only change that works right now. It is not possible to make a fix that works with and without vendor AFAIK.
I was curious that the change (for a shell script) doesn't have a variable ${VENDOR_PATH} that could be "parametrized" if we want to make that kind of change less ugly.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: fpesce, howardjohn
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: douglas-reid
If they are not already assigned, you can assign the PR to them by writing /assign @douglas-reid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

bin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: costinm, fpesce, howardjohn
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

bin/OWNERS [costinm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test e2e-simpleTests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simpleTests-master

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test e2e-simpleTests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test e2e-simpleTests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test integ-security-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid I think we might reconsider moving version package out of the module. We should not use vendor to set build variables.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/integ-security-k8s-presubmit-tests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test integ-security-k8s-presubmit-tests-master

prow/e2e-simpleTests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test e2e-simpleTests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simpleTests-master
/test istio-pilot-e2e-envoyv2-v1alpha3-master

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test integ-security-k8s-presubmit-tests-master

prow/e2e-simpleTests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test e2e-simpleTests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test integ-security-k8s-presubmit-tests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

prow/e2e-simpleTests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test e2e-simpleTests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

We're definitely going to need to revise this.
We'll be building our binaries out of different repos. We're also probably going to drop the vendor directory in some/all our repos.
But this'll work for now.

/test e2e-simpleTests-master
/test istio-pilot-e2e-envoyv2-v1alpha3-master

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test integ-security-k8s-presubmit-tests-master

prow/e2e-mixer-no_auth.sh
325d1cab1b9b6860900d989bcd910d01a897f0cb
link
/test e2e-mixer-no_auth-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth-master


first attempt at pd-nilwind alias code

( note: errors with sudo make install, namely
install: cannot stat '/lib/libwinpthread-1.dll': No such file or directory pd-lib-builder/Makefile.pdlibbuilder:1112: recipe for target 'install-datafiles' failed
)
Anyways, basically copying the pertinent bits of the nilwind Makefile, (the end of it, and the forLinux bit at the top and removing mentions of cyclist), seems like you have to do sudo make install-aliases and it works (didn't find anything for building within directory)

also i guess the forLinux bit means that the code only works for linux installs?


odo dev no longer working with java-quarkus starter projects - E2E tests broken

/kind bug
/area testing
What versions of software are you using?

Using the community starter project:

odo init --name my-quarkus-app --devfile java-quarkus --starter community

odo dev

Using the redhat-product starter project:

odo init --name my-quarkus-rh-app --devfile java-quarkus --starter redhat-product

odo dev

Actual behavior
Build of the project inside the container created by odo dev used to work before, but it is no longer working now, because of the following error:
tools: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project code-with-quarkus:
Fatal error compiling: error: release version 17 not supported -> [Help 1]

This is because code.quarkus.io is defaulting the Java version to 17 for generated projects, unless told otherwise. The problem is that the Devfile does not specify an explicit Java version, and the container component is using a Java 11 image: registry.access.redhat.com/ubi8/openjdk-11
This breaks the following E2E test: https://github.com/redhat-developer/odo/blob/main/tests/e2escenarios/e2e_devfile_test.go#L75
Expected behavior
Those starter projects should work as before, so users can quickly play with odo and quarkus :-)
Workaround (for users)
2 solutions:

either change the maven.compiler.release property to 11 in the pom.xml file
or change the container component image to a Java 17 image, like: maven:3-eclipse-temurin-17-alpine or registry.access.redhat.com/ubi8/openjdk-17

Workaround (for our E2E tests)
None so far. The Devfile Stack for java-quarkus should either :

define a Java 17 image explicitly in the container component (with the risk that this issue will reproduce again when Quarkus switches to the next LTS version of Java - not until at least 2029, that said)
or, to be safe, explicitly specify a Java version in the starter project download URL; this version should be compatible with the container component image: https://github.com/devfile/registry/blob/main/stacks/java-quarkus/devfile.yaml#L12-L18

Any logs, error output, etc?

"odo logs" output
...

tools: [INFO] ------------------------------------------------------------------------
tools: [INFO] Total time:  2.513 s
tools: [INFO] Finished at: 2022-09-30T09:55:51Z
tools: [INFO] ------------------------------------------------------------------------
tools: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project code-with-quarkus: Fatal error compiling: error: release version 17 not supported -> [Help 1]
tools: [ERROR]
tools: [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
tools: [ERROR] Re-run Maven using the -X switch to enable full debug logging.
tools: [ERROR]
tools: [ERROR] For more information about the errors and possible solutions, please read the following articles:
tools: [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
tools: [INFO] Scanning for projects...
tools: [INFO]
tools: [INFO] ---------------------< org.acme:code-with-quarkus >---------------------
tools: [INFO] Building code-with-quarkus 1.0.0-SNAPSHOT
tools: [INFO] --------------------------------[ jar ]---------------------------------
tools: [INFO]
tools: [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ code-with-quarkus ---
tools: [INFO] Using 'UTF-8' encoding to copy filtered resources.
tools: [INFO] Copying 2 resources
tools: [INFO]
tools: [INFO] --- quarkus-maven-plugin:2.13.0.Final:generate-code (default) @ code-with-quarkus ---
tools: [INFO]
tools: [INFO] --- maven-compiler-plugin:3.8.1:compile (default-compile) @ code-with-quarkus ---
tools: [INFO] Changes detected - recompiling the module!
tools: [INFO] Compiling 2 source files to /projects/target/classes
tools: [INFO] ------------------------------------------------------------------------
tools: [INFO] BUILD FAILURE
tools: [INFO] ------------------------------------------------------------------------
tools: [INFO] Total time:  2.932 s
tools: [INFO] Finished at: 2022-09-30T09:56:05Z
tools: [INFO] ------------------------------------------------------------------------
tools: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project code-with-quarkus: Fatal error compiling: error: release version 17 not supported -> [Help 1]
tools: [ERROR]
tools: [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
tools: [ERROR] Re-run Maven using the -X switch to enable full debug logging.
tools: [ERROR]
tools: [ERROR] For more information about the errors and possible solutions, please read the following articles:
tools: [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
tools: [INFO] Scanning for projects...

Just saw this issue reported on the Devfile registry side: https://github.com/devfile/api/issues/949
So this issue should be fixed by the related PR: https://github.com/devfile/registry/pull/131

devfile in registry was fixed https://github.com/devfile/registry/pull/131
this should be now in staging

devfile in registry was fixed devfile/registry#131
this should be now in staging

Thanks for the information.
@anandrkskd Can you please revert the changes done in #6184 when you get a chance? Thanks.
/assign anandrkskd


Trouble with syncing with my actual hue sync box

Hello,
I'm totally new with home assistant and I'm still learning and eager to learn too.  I had no trouble to install the huesyncbox addon and he did find my hue sync box and I was able to put to configured it into HA but he don't recognize any of the 4 HDMI config and I was wondering if I had to do this manually of it should sync everything from my box like the phillips hue lights system?
Thanks for all help you can provide,
Ghislain

I am not exactly sure what you mean with "recognize any of the 4 HDMI config", but lets try to clarify it.
The HDMI ports are exposed as "sources" in the Mediaplayer. You can see on the card in the userinterface after pressing the ⋮ button (when the box is on, HA hides sources selection when device is off).
You can change sources though the user interface or through automations or service calls, but in that regard it is the same as any other mediaplayer that exposes sources.

I am not exactly sure what you mean with "recognize any of the 4 HDMI config", but lets try to clarify it.
The HDMI ports are exposed as "sources" in the Mediaplayer. You can see on the card in the userinterface after pressing the ⋮ button (when the box is on, HA hides sources selection when device is off).
You can change sources though the user interface or through automations or service calls, but in that regard it is the same as any other mediaplayer that exposes sources.

Hello it did work correctly after i follow up your instruction, pretty cool, thanks a lot

Cleaning up some of the issues.
It seems this was resolved so closing it.


fix: Update agones example

What does this PR do?
It fixes the destroy and NLB annotations
Motivation
Found bug while testing
More

[x] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[ ] Yes, I have updated the docs for this feature
[x] Yes, I ran pre-commit run -a with this PR

For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

LGTM


Release v0.7.0 beta.2

[Fix] Publish source .ts files in a different directory than the corresponding .js and .d.ts.

Closes #53

Codecov Report

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

@@          Coverage Diff           @@
##           master     #54   +/-   ##
======================================
  Coverage    65.9%   65.9%
======================================
  Files          56      56
  Lines        1710    1710
  Branches      355     355
======================================
  Hits         1127    1127
  Misses        439     439
  Partials      144     144

Continue to review full report at Codecov.

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


add a query parameter to disable redis caching

By default, Redis caching is enabled when REDIS_HOST and REDIS_PORT env variables are provided. Sometime, for debugging purpose, just to rule out the issue with the caching, we want to query BTE without caching. An extra query parameter like caching=false can be added to temporarily disable the caching.
Relevant code is here:
https://github.com/biothings/bte_trapi_query_graph_handler/blob/main/src/cache_handler.js

Just to clarify, you want an option for the user to specify a caching param in the request?

This is what I have in mind:
Dy default, the query is submitted like this:
POST /v1/query (with the body as the query graph)
With an optional query parameter, we can make the same query like this:
POST /v1/query?caching=false (body is the same query graph)
to disable the default caching behavior.

When I post a query to https://api.bte.ncats.io/v1/query?caching=false, I get the following error:
{
    "error": "Your input query graph is invalid",
    "more_info": [
        "query should NOT have additional properties 'caching'"
    ]
}

I've tested a handful of queries with the same behavior, so it does not appear to be query-dependent.

@andrewsu should I also modify BTE's smartapi registration to include the caching parameter (for query endpoints)? https://github.com/NCATS-Tangerine/translator-api-registry/blob/32737525ff94ad8f5223d8a4f77031d6cca4d91e/biothings_explorer/smartapi.yaml#L100
I have a bit of an idea of how to do that, but not fully sure about the openapi/smartapi spec...

huh, interesting question.  Clearly the local version is necessary and used in this validation step: https://github.com/biothings/BioThings_Explorer_TRAPI/blob/main/src/middlewares/validate.js#L18.  But the local version has also diverged from the smartAPI version here https://github.com/NCATS-Tangerine/translator-api-registry/blob/master/biothings_explorer/smartapi.yaml.  I don't particularly like to have two versions hanging around, but we also don't want to validate every query against a file in another repo.
In the interest of expediency, I'm fine merging this PR just to fix the issue.  But probably we should discuss if we want to change how we handle this...

@andrewsu my understanding is that the local version is meant to match the smartapi version...

@colleenXu you may try the latest BTE metadata from SmartAPI locally to see if any further changes are needed.
And I agree that this file should be sync'ed with the smartapi version.

@newgene @andrewsu to clarify: what endpoints have the caching parameter?
Currently the PR for the local file #350 and the smartapi registration yaml https://github.com/NCATS-Tangerine/translator-api-registry/blob/master/biothings_explorer/smartapi.yaml are identical. They have the caching parameter specified on the /query endpoints only (v1/query, v1/smartapi/{}/query, and v1/team/{}/query)....not /asyncquery...


Reduce dependencies

The biggest goal for Modclean 3.0.0 is to reduce dependencies across the board. I will either remove dependencies altogether, write custom functionality within this module, or find better dependencies.

Dependencies have been greatly reduced and some replaced with smaller, better maintained dependencies.


Redfish configuration failed ACS v4.19.0

Failed to execute HTTP GET request retry attempt 2/2 [URL: https://<IP_ADDRESS>/redfish/v1/Systems/] due to exception javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
then I tried this commands
openssl s_client -servername [redfish IP] -connect [Redfish IP]:5000 </dev/null | sed -ne '/-BEGIN CERTIFICATE-/,/-END CERTIFICATE-/p'>/tmp/redfish.crt
keytool -import -alias [Redfish IP] -keystore /path-to-java/lib/security/cacerts -file /tmp/redfish.crt
(node185) Failed to execute HTTP GET request retry attempt 2/2 [URL: https://<IP_ADDRESS>/redfish/v1/Systems/] due to exception javax.net.ssl.SSLPeerUnverifiedException: Certificate for <<IP_ADDRESS>> doesn't match any of the subject alternative names: []

I don't have a redfish compliant server to check/test, cc @GabrielBrascher @wido - can you review and advise?

@psyma Take a look at: https://docs.cloudstack.apache.org/en/latest/adminguide/hosts.html#out-of-band-management
Have you tried the redfish.ignore.ssl to turn it off and on?
As it's not a valid certificate you probably have these issues. Have you tried without SSL (just to see if this works)

@rohityadavcloud @wido Thanks for the attention.
We disregard using the redfish instead we use IPMI.


MVI example

thanks all for the links. read them.
I actually prefer this to Redux..  Here why :
It reminds me of when i used to build CMS systems and you use "Intents" to also navigate across between modules passing "action data with it". These Intents being loosely coupled was very important, because Modules dont known about each other.
Also it allows you to set a "Intent Contex" also which is basically a global context like perhaps a set of name / value pairs, and then each module just does its thing based on that context, leading to a view that can be made up of a ton of different Modules.

This is the same thing you do with GraphQL gateways on the server.
Behind each GraphQL resolver is "modules" which are often GRPC based MicroService's that return the data. When all the data comes back to the GraphQL layerfrom each microservice it goes back up the pipe to the client as a GraphQL response.


vp9_lp: enable vp9_lp encoding for gst-msdk

This commit is based on https://github.com/intel/vaapi-fits/pull/102
Signed-off-by: Zhu Qingliang<EMAIL_ADDRESS>

checked with local CI smoke test. passed

looks good by local CI full test on encode/vp9


Fix CTS mutable_dispatch memory leak

Current class wrapper of the CTS test framework allows getting the pointer of the private member object. This puts the object at risk of losing its original value if the pointer gets reassigned, causing a memory leak and potentially other problems.
This happens to the "clMemWrapper kernel" used by
mutable_command_work_groups tests, where the "clMemWrapper kernel" gets initialised by the default basic setup function and then it gets reassigned by the build_program_create_kernel_helper() helper  function through pointer.
This patch fixes this issue by updating mutable_command_work_groups tests: instead of calling basic setup function and then initialise the "clMemWrapper kernel" object again in the helper function, it now overrides the basic setup function to make sure the "clMemWrapper kernel" will be assigned only once.

Merging as discussed in the November 5th teleconference.


Instructions required to run reproducible system integration tests or their subset against a test instance on a custom configuration

ISSUE TYPE

Other

COMPONENT NAME
Sytem integation testing wrt general /all

CLOUDSTACK VERSION
4.17.1 and upwards

OS / ENVIRONMENT

SUMMARY
Requirement: need to run existing system integration tests, or their subset, against an existing test instance.
One main feature is that it uses vmware hypervisor.
What needs to be done to setup a test system and configure it to access that test instance?
STEPS TO REPRODUCE
What I have done so far: followed a GitHub workflow test iteration e.g.
# https://github.com/apache/cloudstack/actions/runs/5963475928/workflow
...
mvn clean install -am -Dsimulator -Dnoredist -P developer,systemvm -pl plugins/hypervisors/kvm
mvn clean install -am -Dsimulator -Dnoredist -P developer,systemvm -pl plugins/hypervisors/vmware

Also, I've tried to run a Python test manually in a cloned repo for the branch 4.17.1:
> python3 test_volumes.py

EXPECTED RESULTS
I thought that anything will happen, ending in an error like "can't access test system" or similar.
ACTUAL RESULTS
Immediate process breakup:
> mvn clean install -am -Dsimulator -Dnoredist -P developer,systemvm -pl plugins/hypervisors/vmware
> org.apache.maven.MavenExecutionException: Could not find the selected project in the reactor: plugins/hypervisors/

For the manual test:
> python3 test_volumes.py
> [WARNING]: cannot initiate Sentry error reporting: [Errno 6] No such device or address. (UserWarning)
> ImportError: cannot import name 'escape' from 'jinja2' (/usr/local/lib/python3.9/dist-packages/jinja2/__init__.py)

Final thoughts
As it seems [1] the jinja dependency error comes from the fact that the environment uses Flask 1.x. and Jinja2 3.x which are not compatible through the reqired "escape" component by Flask which has been dropped in Jinja.
[1] https://stackoverflow.com/questions/71718167/importerror-cannot-import-name-escape-from-jinja2

@synergiator

as for the first error , i think the issue is that -pl doesn't take a directory but a project name
the second, the tests are not plain python but nosetests. I poften have to repeat a single test in our lab and use https://github.com/DaanHoogland/trivia/blob/main/singletest.sh . It does some magic around environ definition that you'll have to mimic so the tests know what for instance you hypervisor type and MS IP are. There are some pages on https://cwiki.apache.org/confluence/display/CLOUDSTACK/QA around this. Some may be outdated but its a good starting point. (be grateful if you poinnt uus to improvements)

@synergiator I will close this issue. please reopen if you think it still needs addressing or open a new one if you will.


3.0.0 Release

#163, #173 - remove unnecessary dependencies (@MagicAndre1981)
#164 - add guide link to README (@andreyleskov)

Is anything withholding a release of 3.0?


Add missing dependency to SwiftSyntaxMacroExpansion.

Cherry-pick https://github.com/apple/swift-syntax/pull/2285 to package-release/509.

The SwiftSyntaxMacroExpansion module imports SwiftOperators but does not depend on it explicitly in Package.swift. This is no big deal in Swift 5.x, but I believe in Swift 6 this will become an error.
Also, other packages that depend on this package seem to have stricter build systems, and so the lack of this dependency is actually causing CI failures. For example, by updating OpenAPIGenerator to use SwiftSyntax 509 there is a CI failure due to this problem:

error: Target SwiftSyntaxMacroExpansion imports another target (SwiftOperators) in the package without declaring it a dependency.

Closing in favor of https://github.com/apple/swift-syntax/pull/2295


Cannot download pre-trained because not have Baidu account

Hi, thanks for your great works. But I have some issue when I download your pre-trained weight in Baidu (I cannot create account in Baidu because I don't have Chinese phone number, so I cannot download it from Baidu). Can you please upload pretrained model in google drive or another online storage ?
Thanks for your help

Hi, update with a Google link, please have a try.

Hi, thanks for your reply.
My google account is<EMAIL_ADDRESS>Can you share the pre-trained model via this google account ?
Thank you

You can download them directly
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Phat Thanh Pham @.>
发送时间: Monday, August 26, 2024 11:59:11 PM
收件人: GANWANSHUI/GaussianOcc @.>
抄送: GANWANSHUI @.>; Comment @.>
主题: Re: [GANWANSHUI/GaussianOcc] Cannot download pre-trained because not have Baidu account (Issue #7)
Hi, thanks for your reply.
My google account is<EMAIL_ADDRESS>Can you share the pre-trained model via this google account ?
Thank you
―
Reply to this email directly, view it on GitHubhttps://github.com/GANWANSHUI/GaussianOcc/issues/7#issuecomment-2310424774, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKJKE4ETPCCA6QYZCFVJMF3ZTM7D7AVCNFSM6AAAAABNECOJYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJQGQZDINZXGQ.
You are receiving this because you commented.Message ID: @.***>

https://drive.google.com/drive/u/0/mobile/folders/1ojxpFjj1bG4s5EiteLsqFnAz7liDhgq7

Oh thanks for your help, I can visit this google drive link.
Thank you so much


Native Editor - Cell hover state (grey) - Styling and cell-move and run button interaction with monaco editor

Run and move cell up and down commands have potential issues here with detecting mouse enter and exit in combination with the monaco editor.
○ 3 days
§ Don't want flashing + detecting mouse enter with editor

THis is part of the add hover work

Validated