qutebrowser ":help" does not work

When I type ":help" in qutebrowser (on NixOS, latest master yesterday):
Unable to load page

Problem occurred while loading the URL qute://help/index.html

This most likely means the documentation was not generated properly. If you are
running qutebrowser from the git repository, please run scripts/asciidoc2html.py. If
you're running a released version this is a bug, please use :report to report it.

(I figure this is a nixpkgs packaging issue, so I'm not doing ":report".)
The funny thing is that the help is there on first startup.

@kirelagin

Works fine on 0.3.0 (built from this branch which is based on nixpkgs-unstable.

I asked @The-Compiler about this on irc, and it turns out the tarbal has the docs in them, whereas in the regular git release commit you have to generate them yourself.
Here is the solution; a different src.
I'll do a PR when I also get gstreamer working.

It seems the package is using the correct release tarball nowadays, so I think this can be closed if someone can verify :help works now?


Document Wireguard network requirements

Description
Add wireguard UDP port 51820 to network requirements.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

/sem-approve


RFC: A tag for customizing IntelliSense tooltips

In https://github.com/microsoft/tsdoc/issues/8#issuecomment-510733582 @bookmoons said:

Suggested in microsoft/vscode#77261 something like @extended and @extendedExample that's intended to land in docs but not be shown in smaller contexts like IntelliSense.
/**
 * Take over the specified world.
 *
 * @param world - The world to dominate.
 *
 * @remarks
 * Optimizes for minimum loss of life, minimum reputation harm, and maximum deviousness.
 * Leverages all available capital and human resources without remorse.
 *
 * @extended
 * The development of this procedure began with the Sumerian kings.
 * [... long history ...]
 *
 * @extendedExample
 * // Example of error handling based on the latest best practices
 * while (!retired) {
 *   try { takeOverWorld(earth) } catch (error) {
 *     if (error instanceof InsurrectionError) squashInsurrection()
 *     else if (error instanceof CoupError) decimateCoup()
 *     else if (error instanceof GlobalThermonuclearWarError) retire()
 *     [.. long example list .. ]
 *   }
 * }
 *
 * @beta
 */
export function takeOverWorld(world: World): void;

I've opened this issue so we can have a more focused discussion.

@bookmooons Are you sure that annotating core/extended text needs to be the burden of the documentation author?
Different IntelliSense engines may have different requirements:

the tooltip box may be larger or smaller
it may be scrollable or not
they may or may not know how to render certain constructs

I agree that the VS Code IntelliSense could present the content more nicely.  However, I wonder if that can be fixed without any additional accommodations from TSDoc.  Maybe there's already enough information in the AST to implement a reasonable heuristic.


Networktests

@grafnu PTAL

network_tests.py has the ability to perform four different tests, which can be selected simply through the command line args 'network.min_send', 'network.application.min_send', 'dhcp.long', 'ntp.update'

@pisuke improved network_tests to contain the following tests = 'network.min_send', 'network.application.min_send', 'dhcp.long', 'ntp.update' Dockerfile for each

network_tests.py has the ability to perform four different tests, which can be selected simply through the command line args 'network.min_send', 'network.application.min_send', 'dhcp.long', 'ntp.update'

@ewRedstone @grafnu please note that some of the suggested names of tests have changed and need your review. I've sent an email for this.

@grafnu PTAL

PTAL

@PTAL all comments addressed.
Faux containers coming soon.....

@grafnu PTAL

@ewRedstone Can you open a PR on https://github.com/arupiot/daq/pulls for this one too please?

@ewRedstone does all of this stuff need to be merged into arupiot/daq?


Rudy

we are here and be at the forefront, and be the first to try to be here
My address: 0x6E246BCd173eF53C44dD0109be3ff928E7C92F0e

@Rudisanjaya29 Thank you for your testing and we hope to continue delivering high quality features and improvements!


Implement serde::(De)Serialize for keyvalues_parser::Vdf

Implementing this would be a huge ergonomics win for users, but beyond that it would unlock much better fuzzing potential for keyvalues-serde. Right now keyvalues-serde currently fuzzes an arbitrarily generated "kitchen sink" style struct that holds a lot of different data types, but that doesn't catch issues like #45 :crying_cat_face:
Having this implemented would give us the arbitrarily dynamic type that would be perfect for fuzzing to find since it's the one type that describes all valid VDF blobs

We'll also want to implement it for Value to complete things. We should also be able to allow map types for the top-level pair. We just have to keep track of if we're in the top level in the deserializer, so that we don't won't wind up allowing things like
struct Foo {
    bar: Pair,
}

as valid


Create .gitpod.yml

Summary
This PR makes it so when this repository is used to create a Gitpod workspace, the configuration used will be delegated to the mattermost-gitpod-config repo through its .gitpod.yml file.
This allows multiple repos to share the same Gitpod config, as well as keeping the commit history minimal in regards to the individual repos.
https://www.gitpod.io/docs/references/gitpod-yml
https://www.gitpod.io/docs/multi-repo-workspaces
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-43928

Codecov Report

Merging #83 (6fdf097) into master (efd6640) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #83   +/-   ##
=======================================
  Coverage   16.18%   16.18%
=======================================
  Files          13       13
  Lines        1853     1853
=======================================
  Hits          300      300
  Misses       1534     1534
  Partials       19       19

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 efd6640...6fdf097. Read the comment docs.


Accessing current location from the marker

Hello,
This feature can be really helpful because apparently, if two widgets at the same time request for current location using geolocator, only one of them receives it. It is random. Sometimes the marker can show the current location and geolocator.getCurrentLocation resolves to null or vice versa. This is causing a big unresolvable issue in our app. Hope you can consider some accessor to the current location value that this widget resolves.

Agree, in flutter, it is wired if you are getting some value from some child widget.


Support detecting http response body type

I would like to be able to use linguist to detect the response body of a http response
Using the official format - https://www.w3.org/Protocols/rfc2616/rfc2616-sec6.html#sec6

Are you talking about... partial data streamed from the server? Or a complete response? (Which is fundamentally the same as saving the downloaded file and running github-linguist on the completed result…)

A complete response


Initial HSF work and quality-of-life improvements.

This encompasses all of the changes over the last month (almost), from both the mw-quality-of-life and mw-nu-hsf branches.
Highlights:

Clean up and format much of the code;
Fix and improve the various utility scripts;
Change the interface to use a single -s stage option instead of the variety of stage flags;
Remove use of Z3 from the frontend and model refinement stages, using it only in the Z3 backend; this allows for much easier expression manipulation;
Create a first-attempt Horn clause outputter for use with qarmc/hsf; this is not yet optimised enough for general use, and may have serious bugs, but can sometimes prove/refute small Starling cases;
We now use nunit 3 for tests; to run tests, use a nunit3 test runner on starling.exe;  the -t flag is gone;
Various fixes and optimisations.

Please test before merging, especially platform-sensitive aspects such as the scripts.

Examples and scripts all work on Windows.
Changes look good to me.


2FA connection fails

you can see my error :
Paullux@dvkbuntu:~/fbridge$ python3 fbridge.py
Logging in paulwoisard...
Please enter your 2FA code --> 677061
Submitting 2FA code.
Attempt #1 failed, retrying
Traceback (most recent call last):
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_client.py", line 205, in login
    self._state = State.login(
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_state.py", line 151, in login
    return cls.from_session(session=session)
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_state.py", line 190, in from_session
    revision = int(r.text.split('"client_revision":', 1)[1].split(",", 1)[0])
IndexError: list index out of range
Please enter your 2FA code --> 677061
Submitting 2FA code.
Saving browser.
Starting Facebook checkup flow.
Verifying login attempt.
Saving device again.
Attempt #2 failed, retrying
Traceback (most recent call last):
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_client.py", line 205, in login
    self._state = State.login(
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_state.py", line 153, in login
    raise _exception.FBchatUserError(
fbchat._exception.FBchatUserError: Login failed. Check email/password. (Failed on url: https://m.facebook.com/login/checkpoint/)
Please enter your 2FA code --> 016232
Submitting 2FA code.
Saving browser.
Starting Facebook checkup flow.
Verifying login attempt.
Saving device again.
Attempt #3 failed, retrying
Traceback (most recent call last):
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_client.py", line 205, in login
    self._state = State.login(
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_state.py", line 153, in login
    raise _exception.FBchatUserError(
fbchat._exception.FBchatUserError: Login failed. Check email/password. (Failed on url: https://m.facebook.com/login/checkpoint/)
Please enter your 2FA code --> 678383
Submitting 2FA code.
Attempt #4 failed, retrying
Traceback (most recent call last):
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_client.py", line 205, in login
    self._state = State.login(
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_state.py", line 151, in login
    return cls.from_session(session=session)
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_state.py", line 190, in from_session
    revision = int(r.text.split('"client_revision":', 1)[1].split(",", 1)[0])
IndexError: list index out of range
Please enter your 2FA code --> 678383
Submitting 2FA code.
Saving browser.
Starting Facebook checkup flow.
Verifying login attempt.
Saving device again.
Traceback (most recent call last):
  File "fbridge.py", line 100, in <module>
    client = FBListener(username, password)
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_client.py", line 103, in __init__
    self.login(email, password, max_tries, user_agent=user_agent)
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_client.py", line 205, in login
    self._state = State.login(
  File "/home/Paullux/.local/lib/python3.8/site-packages/fbchat/_state.py", line 153, in login
    raise _exception.FBchatUserError(
fbchat._exception.FBchatUserError: Login failed. Check email/password. (Failed on url: https://m.facebook.com/login/checkpoint/)

Yeah the fbchat module isn't maintained at the moment.
You can checkout my fork of this project: https://github.com/powerjungle/fbridge-asyncio
You can login using cookies: https://github.com/tulir/fbchat-asyncio#examples


Do not cloneNode() in constructor.

Doing so can lead to many unnecessary renders. Especially with components
like tonic-split and tonic-tabs where they use ${this.elements} or
they detect for left & right.
This is because render() get's called multiple times, especially if the
render is async since the cloneNode() will not have copied the children
that are asynchronously rendered.
By having render called more times then expected, the component will async render(); aka fetch and render multiple times (2 or 4) and that leads to
lots of unnecessary delays in interactivity.

I test this in components & logs and it seems fine.
It fixes the issue in logs for the AccountMenu component fetching settings / profiles multiple times.


(561) sustainable solution for users with multiple suppliers

This is the API work to manage users with multiple suppliers.
Adds the supplier name to the Task returned by the API.
Adds a User endpoint to the API that shows a given user name, email and a boolean to show if the user has multiple suppliers.
First part of:
https://trello.com/c/kFEu3JAd/561-sustainable-solution-for-users-with-multiple-suppliers

@leeky I'm hoping this is good now!

@mec Much better thanks - I just added one comment around the filtering by auth_id, otherwise this is good to go!

@leeky I've changed that index method.


Ensure wrap around UI design

With controller, its frustrating to move the UI selection around. Wrap around should be present everywhere possible.

Further handled by #111, but it's still not perfect. Some elements such as SpinBox's are wonky and consume focus events unreliably, which may require either changes in Godot or a special tool script to handle focusing and interactions done with controllers.

With #127 hack, most issues regarding wrap around behavior are fixed. A few corner cases are still left though, but the remaining issues should be treated separately. Overall, wrap around design is developed enough for me to consider this implemented.


LE version?

LE version have many ctd reasons and need powerfull tool for debug crashes.

Sorry, don't have LE and it's a different architecture.


JAXB generated TDML is not reproducible

Steps to reproduce:
sbt clean resources xjc
cp -R debugger/target/scala-2.12/src_managed_cxf/ /tmp
sbt clean resources xjc
diff -rq debugger/target/scala-2.12/src_managed_cxf/ /tmp/src_managed_cxf

Repeat lines 3 and 4 until it shows a diff.
So far I've been able to pretty easily reproduce this on Java 8 (OpenJDK Runtime Environment (Temurin)(build 1.8.0_432-b06)) and less likely but still reproducible on Java 11 (OpenJDK Runtime Environment Temurin-11.0.25+9 (build 11.0.25+9)) and so far not at all on Java 21. Not sure if this is a Java version issue, or some versions just make it more/less likely for some reason.
An example of a diff usually start with something like this:
diff -Naur debugger/target/scala-2.12/src_managed_cxf/org/apache/daffodil/tdml/DFDLBaseType.java /tmp/src_managed_cxf/org/apache/daffodil/tdml/DFDLBaseType.java
--- debugger/target/scala-2.12/src_managed_cxf/org/apache/daffodil/tdml/DFDLBaseType.java	2024-12-06 10:09:30.318167823 -0500
+++ /tmp/src_managed_cxf/org/apache/daffodil/tdml/DFDLBaseType.java	2024-12-06 10:09:16.896018288 -0500
@@ -24,10 +24,10 @@
  *     <extension base="{http://www.ogf.org/dfdl/dfdl-1.0/}DFDLType">
  *       <sequence>
  *       </sequence>
- *       <attGroup ref="{http://www.ogf.org/dfdl/dfdl-1.0/}MarkupAG"/>
- *       <attGroup ref="{http://www.ogf.org/dfdl/dfdl-1.0/}CommonAG"/>
  *       <attGroup ref="{http://www.ogf.org/dfdl/dfdl-1.0/}AlignmentAG"/>
  *       <attGroup ref="{http://www.ogf.org/dfdl/dfdl-1.0/}BaseAG"/>
+ *       <attGroup ref="{http://www.ogf.org/dfdl/dfdl-1.0/}MarkupAG"/>
+ *       <attGroup ref="{http://www.ogf.org/dfdl/dfdl-1.0/}CommonAG"/>
  *       <anyAttribute processContents='lax' namespace='urn:ogf:dfdl:2013:imp:daffodil.apache.org:2018:int urn:ogf:dfdl:2013:imp:daffodil.apache.org:2018:ext http://www.w3.org/XML/1998/namespace http://www.ibm.com/xmlns/dfdl/testData'/>
  *     </extension>
  *   </complexContent>
@@ -52,6 +52,36 @@
...

It's always related to attribute groups.
It looks like however xjc outputs attribute groups, it outputs their generated comments, members, and methods in a random order, instead of the order they are defined in the schema. This doesn't technically affect anything since the the order of attribute groups and their members/functions don't matter, but it does mean the compiled .class files have members/methods in a different order, which leads to non-reproducibility.

Looks like someone else ran into this recently: https://bugs.eclipse.org/bugs/show_bug.cgi?id=583485
So unlikely we are doing something wrong, but unfortunately there may not be an easy workaround. Maybe there's a linter or something that we can post process the generated xjc files to sort the methods and members?
Another option, it might be the case that all the incorrectly generated classes are related to objects that represent embedded schemas (DFDLChoiceType, DFDLElementTYpe, DFDLGroupType, SimpleType, etc). We probably don't need those objects at all. If we JAXB sources could treat the contents of <tdml:defineSchema> as an opaque blob of XML, then that might avoid the issue by just not generating things like DFDLChoiceType. We potentially have the issue with TDML related objects, but maybe it uses attributeGroups in a way that leads to deterministic output.


Infinite loop with errors after creating fresh user and visiting cart details page

Environment Details

Spartacus:  latest devleop (Latest commit 3d43cdb)
Browser: Chrome 79.0.3945.117
OS: Windows 10
Device: Dell Latitude5480
Backend server used: dev 17

Steps to Reproduce

create a brand new user
go to /cart page
(optionally you can add a product before visiting /cart page - no difference)

Observed Results

Cart not found error message is displayed. The infinite http calls are made to https://dev-com-17.accdemo.b2c.ydev.hybris.com:9002/rest/v2/electronics-spa/users/current/carts/selectivecart853aae42-4150-4e20-92a4-711bedd282b0?fields=DEFAULT,potentialProductPromotions,appliedProductPromotions,potentialOrderPromotions,appliedOrderPromotions,entries(totalPrice(formattedValue),product(images(FULL),stock(FULL)),basePrice(formattedValue),updateable),totalPrice(formattedValue),totalItems,totalPriceWithTax(formattedValue),totalDiscounts(value,formattedValue),subTotal(formattedValue),deliveryItemsQuantity,deliveryCost(formattedValue),totalTax(formattedValue,%20value),pickupItemsQuantity,net,appliedVouchers,productDiscounts(formattedValue),user&lang=en&curr=USD
Expected Results
No error occurs, nor the infinite loop with failing http calls.
Additional Information
Is it a problem of sample data recipe or our logic?

This is also happening with a registered user and it is preventing us to checkout. Tested on dev17.

This issue is caused by: in cart.effect.ts, we have:
if (cartNotFoundErrors.length > 0) {
           // Clear cart is responsible for removing cart in `cart` store feature.
           // Remove cart does the same thing, but in `multi-cart` store feature.
           return from([
                 new DeprecatedCartActions.ClearCart(),
                 new CartActions.RemoveCart(loadCartParams.cartId),
               ]);
            }

So, if cart not found, then the cart will be removed from store. And, in selective-cart.service.ts, we have this:
    tap(({ cart, loaded }) => {
        if (this.cartId && this.isEmpty(cart) && !loaded) {
          this.load();
        }
    }),

So, if cart is empty and not loaded, cart will be loaded.
To fix it, I remove that cartNotFoundErrors check. I don't know whether it is the best way to fix it.

After discussing with Marcin, we decided to leave cartNotFoundErrors check there, but add another condition.
if ( cartNotFoundErrors.length > 0 && payload.extraData && payload.extraData.active)

Tested on dev12, on chrome, safari, firefox.
No more infinite loop when we go to cart page.
Verified we can place an order.


AutoList POC

This is a proof-of-concept for #121
It still needs to be updated for all the properties that must be lists

This should now be ready for merge, but there's a few things to unpack here (and I had to get into the weeds of both Pydantic and Rob's clever property assignment stuff), so I'd like to go over it on the call.
Also a reminder to myself that I need to open a ticket about the __root__ item on Resource which is being weird (hooray).


Still Cannot Deploy

Describe the bug
Continues to give the same error I reported earlier about Build Command. Failed. Have tried numerous options like switching to Yarn instead of npm but of no avail. And credits getting consumed anyway. This is llike being looted.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-v1idza
Steps to reproduce
click on deploy and you will get the same error message.
Expected behavior
it should deploy and take the app live
Screen Recording / Screenshot

Platform

OS:  Windows,
Browser: Chrome,
Version: 130.0.6723.117

Additional context
No response

Appreciate the feedback! We are aware of this issue and are tracking it here: #2019


The neighborhood graph is not connected

Hi there, I am currently using the tapkee executable as a pipeline in my project in python, I am creating a text file with all my values to reduce wich looks like this :
0.301961,0.376471,0.443137,0,0.992157,0 0.188235,0.235294,0.298039,0.847059,0.752941,0 0.192157,0.239216,0.294118,0.854902,0.72549,0 0.266667,0.282353,0.380392,0.901961,0.505882,0.8 0.215686,0.211765,0.215686,0.913725,0.498039,0.8 0.25098,0.262745,0.286275,0.917647,0.494118,0.8 0.309804,0.305882,0.333333,0.905882,0.419608,0.8 0.258824,0.321569,0.396078,0.831373,0.811765,0 0.239216,0.298039,0.376471,0.831373,0.807843,0 0.231373,0.290196,0.368627,0.819608,0.741176,0 0.290196,0.360784,0.443137,0.784314,0.780392,0 0.301961,0.376471,0.443137,0,0.992157,0 0.188235,0.235294,0.298039,0.847059,0.752941,0 0.192157,0.239216,0.294118,0.854902,0.72549,0 0.266667,0.282353,0.380392,0.901961,0.505882,0.8 0.215686,0.211765,0.215686,0.913725,0.498039,0.8 0.25098,0.262745,0.286275,0.917647,0.494118,0.8 0.309804,0.305882,0.333333,0.905882,0.419608,0.8 0.258824,0.321569,0.396078,0.831373,0.811765,0
This example works fine but if I add one more line like
0.1,0.2,0.3,0.4,0.5,0
I get the message "[warning] The neighborhood graph is not connected"

So I check for solution and the only help I could find is this message that you wrote in your documentation.

Please note that "[warning] The neighborhood graph is not connected" message in most cases means
that ’tapkee’ run was unsuccessful. As a result, Tapkee() might return the matrix of NaN’s. One of
possible workarounds is to specify the higher number of neigbors (’-k’ option, default is 10). See
below for the example.

Then i tried to change the k values and it works when it's equal or superior to the number of rows divised by 2. So if I have 30 rows I have to make the neighbour values to 15 or higher.
Is it right or something else is wrong ? I am supposed to use this method with a lot of rows (4000 to
million), it gonna be long no ? I may not understand perfectly how it works.
Thanks for your help.

Hey! Sorry it took a while to respond you.
The problem is that the neighborhood strategy being used in Tapkee is dead simple: it finds the nearest neighbors for each point and then uses these connections exclusively. This means that if you add a point that is not a neighbor for any other point it gets 'dropped' out of graph (never reachable).
To solve the problem it might make sense to allow increasing the number of connections if some points lack them to be connected. Other mitigation might be to notify what are the disconnected points so that you can drop them from the dataset. I am not aware what mitigation might be the best, though.

Hi,
Only now I have gone through this. Just in case, excuses for the years-long lead time.
I am wondering what is the meaning of that a point is not a neighbor of any other point?
Even if a point is very, very far away from all the others, at least one of the others must be the closest to it. Unless there's some sort of maximum distance between neighbors, I am guessing.
I will start running the investigation together with one for ** On entry to DLASCL parameter number  4 had an illegal value I've been encountering and keep you posted.

I am wondering what is the meaning of that a point is not a neighbor of any other point?

All right, I think this little inquietude doesn't really matter: even if every point has  at least one neighbor the overall neighborhood graph could of course have more than one connected component

It is just a matter of a form of the distance matrix. Once it becomes a block matrix the whole thing stops working because eigendecomposition is not unique, I think.
Basically, every connected component should be embedded separately. But it doesn't make a lot of sense since we would not have any idea how they are positioned against each other. So that's why I think it is better to either:

Drop the smallest components
Try to fix the graph so that it is connected
Drop some outliers, maybe
Throw an exception

Basically, every connected component should be embedded separately. But it doesn't make a lot of sense since we would not have any idea how they are positioned against each other.

Ah, that's a good point, I didn't think about the separate embeddings.

So that's why I think it is better to either:

Drop the smallest components
Try to fix the graph so that it is connected
Drop some outliers, maybe
Throw an exception

This is what I thought earlier, it would the in the second point:
The issue is that given a number of neighbors (k) k_0 by the user (or the default one, if unprovided) the neighbor graph can be, let's say, unconnected and it is expected the result won't be trustworthy then.
If we would use a k equal to the number of points (or minus 1), then it would be surely connected, right?
From there, I think something that can be automated (perhaps as an optional routine enabled by default) is to rerun the neighbors binary-searching between k_0 and the number of points until one that produces a connected graph is found. It could of course be refined looking for the minimum k that makes it connected.


Add validation rule to check a model used to be the relationship table should has primary key

Every model in relationship must define primary key.
related to #151

legacy issue.


add support for SAVEPOINT statement

This PR adds support for postgres SAVEPOINT statements.
ref: https://www.postgresql.org/docs/8.1/sql-savepoint.html

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 11 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 90.462%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
7284

Relevant Lines:
8052

💛  - Coveralls

@alamb done. Thanks for reviewing :)
I'm using sqlparser at my project https://github.com/poonai/inspektor

Thanks again @poonai


MessagePackSerializer.Get is slow

I'm measuring own serializer(ZeroFormatter) and compare to other serializers.
I found MsgPack's MessagePackSerializer.Get is too slow.
Here is my benchmark.
https://gist.github.com/neuecc/52a5e0e07e25b52eead26f93ad2b52f9

If get serializer in outer iteration loop, works fine.
But "Get is slow" is critical issue.
Serializer is normaly called from Stream with type and called Get every time.
https://github.com/neuecc/LightNode/blob/master/Source/LightNode.Formatter.MsgPack/MsgPackContentFormatter.cs#L43
It means MsgPack-CLI is x10 slower than JSON.NET...

I cannot repro the issue. Coud you tell me testing environment?
My environment is Windows 10 Pro X64 AU, Core i7-3770, 16GB RAM, .NET 4.6.1 build with CLR 4.6.2, release build.
And my micro-repro code is following:
/* My output:
  Warm-up         :526,887.9us
  With-Creation   :56,118.0us
  Without-Creation:90.3us
  Via-MPS.Get()   :93.7us
*/
class Program
{
    static void Main( string[] args )
    {
        try
        {
            const int iteration = 1000;
            var sw = Stopwatch.StartNew();
            new SerializationContext().GetSerializer<Person>();
            sw.Stop();
            Console.WriteLine( $"Warm-up         :{sw.Elapsed.Ticks / 10.0:#,0.0}us" );
            sw.Reset();
            for ( var i = 0; i < iteration; i++ )
            {
                var context = new SerializationContext();
                sw.Start();
                context.GetSerializer<Person>();
                sw.Stop();
            }
            Console.WriteLine( $"With-Creation   :{sw.Elapsed.Ticks / 10.0 / iteration:#,0.0}us" );

            {
                sw.Reset();
                var context = new SerializationContext();
                for ( var i = 0; i < iteration; i++ )
                {
                    sw.Start();
                    context.GetSerializer<Person>();
                    sw.Stop();
                }
                Console.WriteLine( $"Without-Creation:{sw.Elapsed.Ticks / 10.0 / iteration:#,0.0}us" );
            }

            sw.Reset();
            for ( var i = 0; i < iteration; i++ )
            {
                sw.Start();
                MessagePackSerializer.Get<Person>();
                sw.Stop();
            }
            Console.WriteLine( $"Via-MPS.Get()   :{sw.Elapsed.Ticks / 10.0 / iteration:#,0.0}us" );
        }
        catch ( Exception ex )
        {
            Console.Error.WriteLine( ex );
        }
    }
}

Environment:
Windows 10 Pro x64
Intel Core i7-6700 3.40GHz
32GB RAM
.NET 4.5 release build.
<package id="MsgPack.Cli" version="0.9.0-beta1" targetFramework="net45" />
Your code compare get-speed from context or static direct.
I mentioned about serializer lookup speed.
var context = new SerializationContext();

// fast
var serializer = context.GetSerializer<Person>();
for ( var i = 0; i < iteration; i++ )
{
    serializer.Pack...
}

// slow
for ( var i = 0; i < iteration; i++ )
{
    context.GetSerializer<Person>Pack...
}

Get serializer everytime is slower is okay.
But serializer lookup speed is (too) slower than other serializers is not good.

I got it. Do you mean MessagePackSerializer.Get() should be over 10x than now?

Thank you, the answer is yes.

For this object embeds another I get 1750 ms (1.750 seconds) to obtain the Serializer:
    public class DeviceMessage<T>
    {
        [MessagePackMember(0)]
        public int dev_group;
        [MessagePackMember(1)]
        public int dev_level;
        [MessagePackMember(2)]
        public int dev_type;
        [MessagePackMember(3)]
        public int dev_msg_type;
        [MessagePackMember(4)]
        public T   dev_msg_body;

        public DeviceMessage(int dev_group, int dev_level, int dev_type, int dev_msg_type, T dev_msg_body)
        {
            this.dev_group = dev_group;
            this.dev_level = dev_level;
            this.dev_type = dev_type;
            this.dev_msg_type = dev_msg_type;
            this.dev_msg_body = dev_msg_body;
        }
    }

    public class D8RMessage<T>
    {
        [MessagePackMember(0)]
        public String sender;
        [MessagePackMember(1)]
        public String receiver;
        [MessagePackMember(2)]
        public int msgtype;
        [MessagePackMember(3)]
        public T body;

        public D8RMessage(String receiver, String sender, int msgtype, T body)
        {
            this.receiver = receiver;
            this.sender = sender;
            this.msgtype = msgtype;
            this.body = body;
        }
    }

This the Tester method:
           Stopwatch timer;

            timer = Stopwatch.StartNew();
            var serializer = MessagePackSerializer.Get<T>(context);
            timer.Stop();

            Console.WriteLine($"Got serializer in {timer.ElapsedMilliseconds}");

            using (MemoryStream stream = new System.IO.MemoryStream())
            {

                timer = Stopwatch.StartNew();
                serializer.Pack(stream, aType);
                timer.Stop();

                Console.WriteLine($"Packed in {timer.ElapsedMilliseconds} ms");

                stream.Position = 0;
                timer = Stopwatch.StartNew();
                var value = serializer.Unpack(stream);
                timer.Stop();

                Console.WriteLine($"Unpacked in {timer.ElapsedMilliseconds} ms");

                timer = Stopwatch.StartNew();
                MessagePackObject valueString = serializer.ToMessagePackObject(value);
                timer.Stop();

                Console.WriteLine($"aType (as json is) {valueString}");
                Console.WriteLine($"To print the json lasted {timer.ElapsedMilliseconds} ms");
            }
        }

And if I use this to serialize an object I got 1'750 ms!
            var D8RMessage3 = new D8RMessage<DeviceMessage<String>>("gmGateClosed", "dmInterfaceManager", 42, new DeviceMessage<string>(42, 69, 1, 16, "Cammina Cazzo!"));
            //TestMessagePack(D8RMessage3);

            Console.WriteLine($"Vediamo se MemoryStream è più veloce...");

            //var D8RMessage3 = new D8RMessage<DeviceMessage<String>>("gmGateClosed", "dmInterfaceManager", 42, new DeviceMessage<string>(42, 69, 1, 16, "Cammina Cazzo!"));
            TestMessagePackMemoryStream(D8RMessage3);

Thank you for your help.

I add another information serializing another Object of the same type but with different values hits a sort of cache as the time from 1750 ms becomes 0 (that is not measurable in ms) so it is a little better but in any case having to wait 1750 ms the first time is not acceptable for my type of applications.
There is maybe an work around?

When I serialise a simple object
public class Person
{
    public string Name { get; set; }
}

The initial call to MessagePackSerializer.Get takes several hundred milliseconds (340ms~) and the first call to serializer.Unpack() takes a further 30ms or so. Subsequent calls to both serializer.Pack() and serializer.Unpack() are fast.
The delay only seems to happen when the program first runs. I assume this behaviour is by-design, but is there a way to work around, or at least reduce the "slow start" behaviour?
Tested on MSgPack.Cli v1.0.0 targeting .NET Framework 4.5.
Code to reproduce:
using System;
using System.Diagnostics;
using System.IO;
using MsgPack.Serialization;

namespace ConsoleApp1
{
    public static class Program
    {
        static void Main()
        {
            Sw.Start();

            Console.WriteLine($"{Sw.ElapsedMilliseconds}ms");

            // create serializer
            var serializer = MessagePackSerializer.Get<Person>();

            Console.WriteLine($"{Sw.ElapsedMilliseconds}ms");

            for (var x = 0; x < 5; x++)
            {
                var msgPackBytes = Serialise(serializer, new Person { Name = "Bob" });
                var originalClass = Deserialise(serializer, msgPackBytes);

                Console.WriteLine($"{Sw.ElapsedMilliseconds}ms {originalClass.Name}");
            }

            Console.ReadLine();
        }

        static byte[] Serialise(MessagePackSerializer serializer, Person person)
        {
            using (var stream = new MemoryStream())
            {
                serializer.Pack(stream, person);
                return stream.ToArray();
            }
        }

        static Person Deserialise(MessagePackSerializer serializer, byte[] packedBytes)
        {
            using (var stream = new MemoryStream(packedBytes))
            {
                return (Person) serializer.Unpack(stream);
            }
        }

        static readonly Stopwatch Sw = new Stopwatch();
    }

    public class Person
    {
        public string Name { get; set; }
    }
}

Example output:
0ms
341ms
374ms Bob
374ms Bob
374ms Bob
374ms Bob
374ms Bob


Profile request

I'd like to have my profile on ghuser.io, thanks!

here you go :) https://ghuser.io/mohitmanuja


Steps to reproduce

Create an AnimationTree and 2 AnimationPlayers
Add an animation to each player and an animation track from the first playing the animation of the second
Create a Blend Tree with a TimeScale and make it play the first player's animation.
The first animation will play at the set time scale but the second won't.

Minimal reproduction project
Animate This.zip

@FreegleBarr Does this work on 3.4.2?

@Calinou No, I just checked and it also occurs on 3.4.2

It's more of an enhancement than a bug fix, but I think it's a necessary fix because without it, it's not possible to synchronize multiple AnimationPlayers' animation with changing their playback speed by AnimationTree. NodeAdd may can solve it in some extent, but it might not be sufficient for cases where it need to have multiple AnimationPlayer.


fix: deepcopy fail for coping model_instance config

link #609
When copying the configuration with model_instance, an error may arise because model_instance is typically an uncopyable object. To fix it, I attempted to use safe_deepcopy to replace the internal deepcopy during the copying of the LLM configuration.
The safe_deepcopy function first attempts to use the internal deepcopy, ensuring it doesn't disrupt the original usage. If that fails, it resorts to alternative copying logic, such as shallow copying. In the worst-case scenario, it returns the original object.
Note:
If there are any new graphs using deepcopy in future pull requests, special attention should be paid to copying the LLM configuration correctly.
Other Considerations:
This pull request may not represent the best practice for resolving the issue, but it offers a quick fix. Here are some alternative approaches that I believe are superior:

Using Pydantic Class Objects: Define the configuration with a Pydantic class object and implement a custom deepcopy method. This approach requires significant refactoring but is more elegant.
Replacing Internal Deepcopy: Utilize safe_deepcopy to replace the internal deepcopy. However, this might introduce unforeseen issues.
Alternative Deepcopy Packages: Consider using other deepcopy packages, but I have yet to find a suitable option.

In addition, there are any formattor about this project? seem I cannot find about it

wait a moment. find new problem

What's the problem?

What's the problem?

I found there is message missing when use copy(deep=True) in the pydantic model v1. You can see as below. Maybe this is pydanticv1 bug. so I try to use shallow copy when deepcopy failed.

Now is ok?

Now is ok?

yeah. Now is ok.

thank you!


How do I ...

/Qspectre
Where can i close this, my first installation failed

What package did you install?

@aphage
Could you please describe your question more clearly?
Please provide the steps or post the error info.

PS E:\install\vcpkg> ./bootstrap-vcpkg.bat
Building vcpkg.exe ...
C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Microsoft\VC\v160\Microsoft.CppBuild.targets(4
36,5): error MSB8040: 此项目需要缓解了 Spectre 漏洞的库。从 Visual Studio 安装程序(单个组件选项卡)为正在使用的任何工具集和体系结构安装它们。了解详细信息: https://aka.ms/Of
hn4c [E:\install\vcpkg\toolsrc\vcpkglib\vcpkglib.vcxproj]
E:\install\vcpkg\scripts\bootstrap.ps1 : Building vcpkg.exe failed. Please ensure you have installed Visual Studio with
the Desktop C++ workload and the Windows SDK for Desktop C++.
所在位置 行:1 字符: 4

& {& 'E:\install\vcpkg\scripts\bootstrap.ps1' }

+ CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
+ FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,bootstrap.ps1

error MSB8040: Spectre-mitigated libraries are required for this project

Okay, I manually modified the "toolsrc / vcpkg.sln" project properties and solved

Thanks guys


🔧 error: package com.mrousavy.camera.frameprocessor does not exist

How were you trying to build the app?
I tried building my app with "vision-camera-code-scanner": "^0.2.0" While on ReactNative 0.74 and Android 14, sdk 34
and I keep getting an error
Full build logs
/node_modules/vision-camera-code-scanner/android/src/main/java/com/visioncameracodescanner/VisionCameraCodeScannerPluginPackage.java:9:
error: package com.mrousavy.camera.frameprocessor does not exist
import com.mrousavy.camera.frameprocessor.FrameProcessorPlugin;

Project dependencies
"dependencies": {
   "react": "18.2.0",
   "react-hook-form": "^7.43.9",
   "react-native": "0.74.0",
   "react-native-vision-camera": "^4.3.2",
   "vision-camera-code-scanner": "^0.2.0"
}

VisionCamera Version
^4.3.2
Target platforms
Android
Operating system
MacOS
Can you build the VisionCamera Example app?
No, I cannot build the Example app either
Additional information

[ ] I am using Expo
[X] I have enabled Frame Processors (react-native-worklets-core)
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

@mrousavy I did search those 2 issues in the screenshot above but there was one solution and that was to use java 11 from AdoptOpenJDK and delete zulu 11 but I am on react-native 0.73.1 because Android 14 requires sdk 34 and that is the earliest version of react-native that supports sdk 34 but unfortunately 0.73.1 requires java 17 and currently I am on zulu 17 and getting an error related to the vision-camera-code-scanner package not being able to find the Barcode class from com.google.mlkit.vision.barcode

@mrousavy 2 years and not fix?

He created this error on purpose to monetize his component. This component is no longer open source. Consider using another module.


staleSaveError is a bad experience

VSCode Version: 1.14.1
OS Version: Win10.

Steps to Reproduce:

Open an existing file and make some edits.
Modify the file with another editor or write it with another program.
VS Code quietly when an edited file be modified externally.
staleSaveError popup when save the edited file in VS Code.

Expected results:

VS Code should not be quiet when an edited file be modified externally, it should popup an tip earlier to indicate the file has been modified externally, for avoid a greater merge conflict.
If possible, it should to guide more effectively or auto merge the conflicts (diff).
If possible, it should displaye an message (box / tip) to ask actions, like take current, take external, resolve the conflict, cancel (back to editing).
Allow the user to reload the file without saving, #17643.

Reproduces without extensions: Yes

@yfdyh000 the error message is guiding you to a diff editor to compare your version with the version on disk and provides actions to resolve the save conflict (take version from disk / take version from VS Code). Are you not seeing that?

I see it, but the message is too late and diff may be complicated, I might just want to give up the edited version (probably inadvertently), or get the conflict notification earlier.

@yfdyh000 I do not understand what you are asking for. If you have a save conflict you get:

a diff to see what is going on
an action to "take the version on disk"
an action to "overwrite the version on disk"

What is missing?

An earlier notice to aware it will be conflict, avoid a wider range of conflicts.
Maybe I thought adds noticeable and quick buttons in the notification, not just start to compare. Or larger buttons in the comparison toolbar, I was ignores them because they are small so it looks difficult to use / advanced.
In addition, if both content is accidentally the same, it still tell me to compare.
"Discard local changes and revert to content on disk" is vague, "your version" or "editor content" should be used, like "Click on Compare to compare your version with the one on disk.".

yeah there are other issues for this reported already
can you add this to https://github.com/Microsoft/vscode/issues/31063
can you add this to https://github.com/Microsoft/vscode/issues/31063
I can make that change to:

Use your changes and overwrite disk contents
and
Discard your changes and revert to content on disk

I pushed the wording change. Closing this one in favour of the other issues that got reported on this topic already.


OpenCV.projectPoints input type are confusing (maybe incorrect?)

OpenCV documentation for projectPoints describes all of the InputArray parameters as vectors or matrices. This agrees with how I see the function used by others in python from cv2.
The julia type signature of OpenCV.projectPoints requires AbstractArray{T, 3} for all of these, and it's not clear why these extra dimensions are needed or how they will be handled. This is the signature listed by methods(projectPoints):
projectPoints(objectPoints::Union{OpenCV.CxxMat, AbstractArray{T, 3} where T<:Union{Float32, Float64, Int16, Int32, Int8, UInt16, UInt8}},
              rvec::Union{OpenCV.CxxMat, AbstractArray{T, 3} where T<:Union{Float32, Float64, Int16, Int32, Int8, UInt16, UInt8}},
              tvec::Union{OpenCV.CxxMat, AbstractArray{T, 3} where T<:Union{Float32, Float64, Int16, Int32, Int8, UInt16, UInt8}},
              cameraMatrix::Union{OpenCV.CxxMat, AbstractArray{T, 3} where T<:Union{Float32, Float64, Int16, Int32, Int8, UInt16, UInt8}},
              distCoeffs::Union{OpenCV.CxxMat, AbstractArray{T, 3} where T<:Union{Float32, Float64, Int16, Int32, Int8, UInt16, UInt8}}
              ; imagePoints, jacobian, aspectRatio)

I came here for similar reason but specifically for the arguments of calibrateCamera.
Can you please post an example of a correct instance of the arguments for projectPoints or calibrateCamera?
I can't even seem to be able to create anything that fits the objectPoints::Union{OpenCV.CxxMat, AbstractArray{T, 3} where T<:Union{Float32, Float64, Int16, Int32, Int8, UInt16, UInt8}} "bill"...


uninitialized constant Fluent::Plugin::SyslogInput::PRIORITY_MAP

Getting this error after plugin is installed. Fluentd crashloops after.
[error]: unexpected error error_class=NameError error="uninitialized constant Fluent::Plugin::SyslogInput::PRIORITY_MAP"

Me too

If this maintainer isn't responsive, @i5okie any interest in forking this and making our own updated plugin?

https://github.com/ApplauseOSS/fluent-plugin-heroku-http/pull/7

@austenmadden thanks for this PR, I am also interested in this fix and willing to contribute to keep this project alive if needed

Sorry about the slow response. This got lost in the ~30K GitHub emails I get a day. 😁

I just pushed v0.0.2 to RubyGems
$ rake release[origin]
fluent-plugin-heroku-http 0.0.2 built to pkg/fluent-plugin-heroku-http-0.0.2.gem.
Tagged v0.0.2.
Pushed git commits and tags.
Pushing gem to https://rubygems.org...
Successfully registered gem: fluent-plugin-heroku-http (0.0.2)
Pushed fluent-plugin-heroku-http 0.0.2 to rubygems.org


Permalinks that bypass filenames

I want the option to set a different URL path for a file instead of it being foldername/filename

@PhilippeduPreez yes, these are "slugs". Can we have a deeper "so that" for this

one of the major problems of Obsidian Publish is the lack of permalinks, and the problem of having a readeable vault, with filenames == titles. (here is a discussion https://forum.obsidian.md/t/ability-to-specify-permalinks-in-frontmatter/8989 )
i think we have two solutions:

keep the url system as is, but add a "title" metafield that gets rendered everywhere in the frontend.
this way would keep the same "structure" in both Obsidian and in the website. i would take care about naming the files with a seo / rendered website POV
add a "slug" metafield that gets used when rendering the file's url.
slug should be optional (otherwise filename is used), and be just the last part of the url.. see how Hugo explains its path system: https://gohugo.io/content-management/organization/#path-breakdown-in-hugo

to tell it all, i think it would be wonderful to have both slug and title fields. maybe Obsidian team is already dealing with Titles in the new Frontmatter new features? shoudl we wait and see?


OU-454: Fix CSRF issues with proxy

When sent through a ConsolePlugin proxy, POST requests need to have the correct CSRF token to match their cookie. This PR looks to add that token when needed

/cherry-pick release-0.1


Outdated plugin folders not deleted from Jellyfin's config folder after updates.

Turns out that after each update of this plugin, Jellyscrub's previous (now old) folder within Jellyfin's config plugin folder is not deleted. They have to be manually deleted instead of the update taking care of it afterwards like other plugins.

This could be related to the ffmpeg process not being killed when jellyfin stops. Perhaps the the dll is not properly unloaded and is treated as in use. Not sure but will have to look into it.

Not sure if this is actually specific to jellyscrub. I installed an older version of AniDB and after it was updated through the task, jellyfin didn't auto-remove the folder of the older plugin on shutdown or subsequent startup. Do you have any other specific plugins that you've updated and had them remove the old folder?

Unfortunately, I have not checked before during updates. Especially since I have the auto-update task set for all plug-ins, I don't know when they're all updated, and therefore never checked the plugin folder before and after an update.
It could be that leaving the old plugin folders is the default and that some other task cleans up the unused folders eventually. Since I have some plugins that have auto-updated in the past, I would have had left-over folders. But since I first posted this issue, there were no outdated plugin folders apart from Jellyscrub. That is what led me to believe that Jellyscrub was at fault.
Maybe you're right. I just came across this when I was moving my config folders to a new location and noticed that the old plugin folder was remaining for Jellyscrub.

Jellyfin defaults to not deleting old plugins, but this functionality can be changed by setting <RemoveOldPlugins>false</RemoveOldPlugins> to true in the system.xml file in jellyfin's config folder.


[SPARK-21677][SQL] json_tuple throws NullPointException when column is null as string type

What changes were proposed in this pull request?
scala> Seq(("""{"Hyukjin": 224, "John": 1225}""")).toDS.selectExpr("json_tuple(value, trim(null))").show()
...
java.lang.NullPointerException
	at ...

Currently the null field name will throw NullPointException. As a given field name null can't be matched with any field names in json, we just output null as its column value. This PR achieves it by returning a very unlikely column name __NullFieldName in evaluation of the field names.
How was this patch tested?
Added unit test.

cc @viirya

cc @HyukjinKwon

Can one of the admins verify this patch?

ok to test

Test build #80579 has started for PR 18930 at commit ffa575a.

Test build #80579 has finished for PR 18930 at commit ffa575a.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80579/
Test FAILed.

Merged build finished. Test FAILed.

retest this please.

Test build #80587 has started for PR 18930 at commit ffa575a.

Test build #80587 has finished for PR 18930 at commit ffa575a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80587/
Test PASSed.

Merged build finished. Test PASSed.

Test build #80634 has started for PR 18930 at commit 5d71263.

Test build #80634 has finished for PR 18930 at commit 5d71263.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80634/
Test PASSed.

Test build #80656 has started for PR 18930 at commit 0078445.

Test build #80656 has finished for PR 18930 at commit 0078445.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80656/
Test FAILed.

Merged build finished. Test FAILed.

Test build #80664 has started for PR 18930 at commit 5c69df5.

Test build #80664 has finished for PR 18930 at commit 5c69df5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80664/
Test PASSed.

LGTM

Test build #80688 has started for PR 18930 at commit ab16929.

Test build #80688 has finished for PR 18930 at commit ab16929.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80688/
Test PASSed.

Test build #80740 has started for PR 18930 at commit e0e0c74.

Test build #80740 has finished for PR 18930 at commit e0e0c74.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80740/
Test FAILed.

Test build #80758 has started for PR 18930 at commit 5191ed4.

Test build #80758 has finished for PR 18930 at commit 5191ed4.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

retest this please.

Test build #80768 has started for PR 18930 at commit 5191ed4.

Test build #80768 has finished for PR 18930 at commit 5191ed4.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80768/
Test FAILed.

retest this please

Test build #80774 has started for PR 18930 at commit 5191ed4.

Test build #80774 has finished for PR 18930 at commit 5191ed4.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80774/
Test FAILed.

retest this please

Test build #80783 has started for PR 18930 at commit 5191ed4.

Test build #80783 has finished for PR 18930 at commit 5191ed4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80783/
Test PASSed.

LGTM

LGTM too.

Test build #80788 has started for PR 18930 at commit ff3b9da.

Test build #80788 has finished for PR 18930 at commit ff3b9da.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80788/
Test FAILed.

retest this please

Test build #80789 has finished for PR 18930 at commit ff3b9da.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80789/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Test build #80805 has started for PR 18930 at commit ff3b9da.

Test build #80805 has finished for PR 18930 at commit ff3b9da.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80805/
Test PASSed.

Merged build finished. Test PASSed.

Thanks! Merging to master.

Thanks @HyukjinKwon @gatorsmile

Thanks @viirya @HyukjinKwon @gatorsmile , I learned a lot from this journey.


.tmux.conf.local cannot be loaded recently

Environment and setting:
macOS X Bug Sur version 11.4
tmux version 3.2
zsh version 5.8 (x86_64-apple-darwin19.6.0)
iTerm2 version 3.4.8, without enabling "Use Unicode version 9+ widths" in iTerm2's Preference > Profiles > Text
echo $TERM shows xterm-256color outside tmux and screen-256color inside.
As title, I have used tmux for a long time. Recently after some updates such as oh-my-zsh and macOS, the tmux seems to get in trouble as below.

I have tried to re-install tmux, iTerm2 and re-source tmux files but nothing change. I found that .tmux.conf could be executed successfully by doing some test such as modifying setting in .tmux.conf, but the .tmux.conf.local file seems to failed to load successfully, which could be found by the un-modified status-bar. The user defined overrides command is written in .tmux.conf as same as the default file.
I am wondering what happens in the tmux. Although tmux is still working but I am missing the pretty theme...

Hello @ujkuo,
Please start the installation over then tell me if you're still having problem.
This means exiting tmux, making sure no tmux process is still running, deleting your ~/.tmux clone, your ~/.tmux.conf.local copy and the ~/.tmux.conf symlink.

Hello @gpakosz ,
After doing so, the configure executes successfully. Thank you so much!


Input data structure for molecule

Hi Roman,
I've noticed that DataSAIL doesn't currently accept InChI as input for molecules. However, I'm wondering if I could convert the InChI to ECFP and then provide it as input to DataSAIL? It seems that converting InChI to SMILES is quite risky, as indicated in these discussions:
https://github.com/rdkit/rdkit/issues/542
https://sourceforge.net/p/rdkit/mailman/message/36696861/
It would significantly enhance usability if DataSAIL could also accept ECFP as input for molecules, making it more universal.

Yes, fingerprint input may be an idea. What's also possible is to calculate the similarity-matrix on your own and input that with the --e-sim argument as in this example.

This has been solved with commit 35954ce.


[AMBARI-24001] Recommission action doesn't appear for decommissioned component

How was this patch tested?
21798 passing (39s)
48 pending

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2548/
Test PASSed.


Fix outdated URL for the ca-certificates package

This is my attempt at a fix for the outdated URL issue I spotted while perusing the docs earlier today.
Thanks in advance for your time reviewing this! 😃
Fixes #15985

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

There's a failing check. Asked someone to look into this.

782777


Ember ActionHandlerMixin for send and action

Define ActionHandlerMixin class (which mirrors Ember's ActionHandler mixin class) and apply this appropriately to match Ember's usage.

@jedmao could you review this PR?

:+1:

@jedmao Thanks.
I've also got some questions around the ember.d.ts file (I'm relatively new to TypeScript). Can I contact you in some convenient way?

My username at outlook dot com. Go for it.

@flcdrg thanks mate!
@jedmao thanks!


User analytics for xstate-wallet and web3torrent

Some tool which simply shows information about xstate-wallet usage in the wild. For example: how many wallets exist? How much money is in those wallets? How many API calls of various types have been made? What “sites” are the most used? What is the size of the average users store in KB? How many challenges occur in the wild? Again these can be “opt-in”, but it would be very helpful for us to understand how people are using this.

Added to Web3Torrent. Adding to wallet now!

@snario is this done in the wallet?

Add analytics (Amplitude?) and hook up to segment

I have narrowed the scope of this issue to simply enabling segment in xstate wallet (it is already enabled in web3torrent).
I'll tackle 1-3, and think about 4.
I will ignore 5-6, since they are pretty meaningless questions right now.


feat(Navbar): Added dark prop

If there is a light prop to add the navbar-light class, then to keep consistency there should also be a dark prop to add the navbar-dark class.
This PR adds the dark prop which will add the aforementioned class when set. With a test and the prop entry in the comments.

This is already addressed in the v5 branch which accounts for changes in bootstrap 4 beta


[BUG] Space on first word in Embedded Chat

See attached image.
The first word for the chatbot answers is always indented (there is a space). Has this been seen before? Do you know how I would fix it?
Thanks for all the great work!

resolved:


Wrapping number data in float

I was trying to use number range validator and it crashed because of an invalid comparison between str, data, and a number, min/max. When I wrapped the data field in float it was fixed.

If data is not already the correct type, there is something else wrong. If you believe this is an issue with WTForms and not your code, please open an issue with a minimal reproducible example and a full traceback.


user interface concept

@todo

[ ] specify user interface requirements
[ ] according to @okdistribute & @RangerMauve

check dashboard with data about the dat (health) of:

https://github.com/mafintosh/hypercore-stats-ui

[ ] evaluate inspiration from @okdistribute listed below

@ninabreznik

https://github.com/ninabreznik/pinning-service-webpage/blob/master/src/index.js
https://www.loom.com/share/506dc8e7934f45a79b0ec1ed5ca305a8

inspiration from karissa

https://www.npmjs.com/package/hypercore-stats-ui (see readme for example of block-level stats)
https://github.com/dat-land/dat-desktop (see readme for uTorrent-like stats)
https://webtorrent.io/ webtorrent network viz

inspiration from @ninabreznik

UX modules (rough sketch by Nina)

stats module (shows activity/health of your dat)

https://www.npmjs.com/package/hypercore-stats-ui (see readme for example of block-level stats)

register module (see hashbase for inspiration)
wallet module (profile, see old torrents for seeders/leechers ratio)    -

Did a first draft - to capture core features https://www.draw.io/#LDaDot

@ninabreznik you have to somehow screenshot or download/upload the file.
The link alone does not work

Really? The link to draw.io doesn't work?
On Sat, 8 Feb 2020, 22:49 Alexander Praetorius<EMAIL_ADDRESS>wrote:

@ninabreznik https://github.com/ninabreznik you have to somehow
screenshot or download/upload the file.
The link alone does not work
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/playproject-io/datdot-research/issues/4?email_source=notifications&email_token=AAIGZSZOWCYWYAOVH3BGR73RB6RN7A5CNFSM4KFUBVK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELGEBNQ#issuecomment-583811254,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIGZS2SGX7XQAF3LP3ECLTRB6RN7ANCNFSM4KFUBVKQ
.


Flask version correct

Python give " Error 'Flask' object has no attribute 'before_first_request' " because before_first_request has been removed on version 2.3

I'd rather fix the issue than force the older Flask version.


Update README.md

Add Status badge for openSSF, Dependency, CodeQL and License

Suppose we will merge PR#11 first, and then rebase this PR.


PTZ Support

Is your feature request related to a problem? Please describe.
One of my camera's uses the Pelco D PTZ standard, are there any plans to implement at least some of the PTZ protocols?

Even PTZ control in Camera.UI if not possible in the Home app would be welcome.

I'm fairly new to homebridge plugin development.
As said in this conversation, homekit supports PTZ.
This is what I have found:
TargetTiltAngle
TargetVerticalTiltAngle
TargetHorizontalTiltAngle
DigitalZoom
OpticalZoom

🎉 A new version of camera.ui
A new version of camera.ui is currently under active development. An initial alpha/beta release and previews are coming soon. Stay tuned for exciting updates: https://github.com/seydx/camera.ui/issues/448 .
This version will no longer be developed / fixed. The new version contains many novelties and fixes (most important is for HKSV recording).


'Delete' modal: improve pin guidance and overall language

I was about to fill this as separate issues, but I feel it's better to think about final UX that tries to address it all.
@ipfs-shipyard/gui

Unpin by default
Situation
Right now user can pin an item via Files screen, and then use the same GUI to remove it:

Problem
If deleted item was pinned, the low level pin remains, silently protecting the CID  from being garbage-collected.
This disconnect between low level pins and MFS will become a problem when we implement https://github.com/ipfs-shipyard/ipfs-companion/issues/888#issuecomment-689666068 as part of https://github.com/ipfs/ipfs-gui/issues/91. Pinning via IPFS Companion would create local pin AND add file to MFS.
Perhaps we should adjust the default behavior and remove local pin when a pinned resource is removed from MFS via GUI in Files screen?
Solution ideas
GUI users should not need to know what low level pins are, Files screen should be enough on its own for managing "pinned" data.
If they remove something from "Files", they expect it to not be kept around forever. There should be a way to keep the pin if a power user knows what they are doing, but it should not be the default behavior.
What if we extend the Delete dialog and display some additional UI if Deleted file is pinned?
Some ideas:

checkbox

[x] Remove local pin or.. [ ] Keep local pin
Q: should there be an option to remove remote pins too?

button

Delete and unpin (make it default)  and then regular Delete that keeps the pin is grayed out

No strong feeling here, but checkbox may be a better pattern if we want to provide other options, such as  "remove blocks from local repo" (see below)
Clear description of what "Delete" really means
aka MFS removal vs garbage collection vs removal working only on local node
Situation

If something is in MFS  it is implicitly pinned  (protected from GC).
If user removes something from MFS, even if it is not explicitly pinned, it is still in the repo until GC, and user will see the disk space used did not change
Even if content is GC'd, or every block is manually removed via ipfs block rm, the change is local, and the data may be safe and sound on a different node.

Problem
Due to this, "Delete" may not be the best description for the operation of "removing a pointer to a CID from MFS".
Solution ideas

give user an option to truly remove blocks from local repo: [ ] purge data blocks from local cache
rename? be more honest about operation being more a Forget/Discard than "Delete"

Apologies for a long brain dump, but wanted to have all concerns around Delete modal in one place, so we can figure out how to improve it in premaration for pinning services integration.
Feedback/ideas will be really appreciated!

I would go with Unpin and Delete (order makes more sense I guess?), it gives the subtle message to the user that the file should be unpinned first if he really wants to fully delete it :)

For your consideration/debate ...

@jessicaschilling YES PLEASE :+1:
To avoid questions "Why my repo size did not change when I removed this 2GB video" we could also add a third checkbox (unchecked by default) saying Run garbage collection in the background  or Proactively purge all unused blocks from the local cache
It will be easy to wire up – all  we need to do is to asynchronously run ipfs.repo.gc({quiet: true}) in the background.

Sounds good, but can introduce as-is first? A little concerned about cognitive overload for three check boxes when the third does something the user can do in the menu today.

@jessicaschilling that modal strikes me as crystal 🔮 clear.
Nice 👌
My only suggestions are visual tweaks to the component: to make the text darker (to add gravitas), consider left-aligning it (for readability), and possibly use a lighter font weight for the checkbox items.

@ericronne All good points - we've actually got a lack of consistency across modals (this is an existing one but just new text), so probably best fodder for its own issue.

@jessicaschilling YES PLEASE 👍
To avoid questions "Why my repo size did not change when I removed this 2GB video" we could also add a third checkbox (unchecked by default) saying Run garbage collection in the background or Proactively purge all unused blocks from the local cache or ... something more human 🤖
It will be easy to wire up – all we need to do is to asynchronously run ipfs.repo.gc({quiet: true}) in the background.

@lidel are we okay with keeping that a webui/desktop only feature, or should we request this functionality in go-ipfs?

@ericronne FYI, opened https://github.com/ipfs-shipyard/ipfs-webui/issues/1643 for modal standardization, but want to wait until a few other items get merged in order to avoid conflicts. Just wanted to confirm that it's on the radar.

This is ready for wiring up in https://github.com/ipfs-shipyard/ipfs-webui/pull/1644.

@lidel are we okay with keeping that a webui/desktop only feature, or should we request this functionality in go-ipfs?

@rafaelramalho19 The way I see it, CLI users can already trigger gc by executing ipfs repo gc after they remove things, so nothing is missing in go-ipfs. What we would do here is to give the same capability to GUI users :)


Disable text selection in menu with css

It looks weird to be able to select text in menu and UI where you shouldn't be able to.
Example:

solution:
https://stackoverflow.com/questions/826782/how-to-disable-text-selection-highlighting#4407335

@DataCourier Want to submit a PR?

Or else I can do it. Seems fairly simple thought 😄

I'll be getting familiar with the app over the weekends. I'll try to make some PRs, but if anyone wants to grab it go ahead.

Fixed by https://github.com/LN-Zap/zap-desktop/pull/296


Time shown on X-axis

Thank you very much for this panel, it is very useful.
I would like to request you if it is possible to show the time directly on the X-axis as for the typical Grafana graphs.
And again, great job!

+1 for us!

+1

!!

Looks like @AlexanderWert has made progress on this...  https://github.com/NovaTecConsulting/novatec-grafana-discrete-panel/issues/1

This is now in master and will be released with v0.0.7
https://github.com/grafana/grafana-plugin-repository/pull/245
Lets make new issues for improvements.


Enable full tracing tests on ROS 2 CI

(Issue moved from GitLab, see: https://gitlab.com/ros-tracing/ros2_tracing/-/issues/71)

This was done as part of https://github.com/ros2/ros2/issues/1177, so closing this.


fix(gridBehavior): changing the focus zone to embed

We recognized that additional 'div' wrapper  from focus zone was Firefox presented in accessibility DOM as focusable section.
Then for example NVDA navigated to the section and was narrating whole content of the 'div'.
Therefore changing the focus sone to 'embed'

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


After updating to the 1.21 server, the CPU usage is very high

Spark link
https://spark.lucko.me/YXsqCkLSXE
Description of issue
After updating to the 1.21 server, the CPU usage is very high, usually fluctuating from 5% to 50%. Even if the server has only one player, it will still occupy 0-10%, and even if there is no player, it will be the same
Purpur version
1.21-2280-930003b (MC: 1.21)*

You are running the latest version
Previous: 1.21-126-7e44684 (MC: 1.21)

Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] I ticked all the boxes without actually reading them
[X] My version of Minecraft is supported by Purpur.

Other
No response

Please send the spark profiler link (/spark profiler start) and after a few minutes do (/spark profiler stop). You sent a Heap Summary which doesn't help at all to solve your issue.
You can also join our Discord.

请发送 spark 分析器链接 （/spark profiler start），几分钟后执行 （/spark profiler stop）。您发送了堆摘要，这根本无助于解决您的问题。
您也可以加入我们的 Discord。

https://spark.lucko.me/9kqex2d8jF

请发送 spark 分析器链接 （/spark profiler start），几分钟后执行 （/spark profiler stop）。您发送了堆摘要，这根本无助于解决您的问题。
您也可以加入我们的 Discord。

Please send the spark profiler link (/spark profiler start) and after a few minutes do (/spark profiler stop). You sent a Heap Summary which doesn't help at all to solve your issue.
You can also join our Discord.

G1 Old 1300ms,Is it abnormal?

Please send the spark profiler link (/spark profiler start) and after a few minutes do (/spark profiler stop). You sent a Heap Summary which doesn't help at all to solve your issue.
You can also join our Discord.

G1 Old 1300ms,Is it abnormal?

Please send the spark profiler link (/spark profiler start) and after a few minutes do (/spark profiler stop). You sent a Heap Summary which doesn't help at all to solve your issue.
You can also join our Discord.

https://spark.lucko.me/9kqex2d8jF

Oh just noticed, there is a bit too much in SWAP, SWAP is when your RAM stuff is written onto a SSD or HDD(Which hurts SSD and performance)

Please join our discord for support. https://purpurmc.org/discord


Is it possible to get run label during the run?

We have RUN_DIR and RUN_ID but it seems run label isn't passed as an environment variable?

seems like an easy addition. We'll look into it for the next release.

@uduse please check the PR above and verify that it does what you need it to do.

@msarahan That's quick 😄 Yes this is what I want. I thought this will be a feature infuturer guild python API though.

We might want to slow down on this. I'm not sure adding env vars for additional run metadata is a good idea. The env space is something we could theoretically fill up with all sorts of useful run data - but there's a lot of metadata and the env would soon become filled with Guild stuff (def something we should avoid).
@uduse I think this is better provided as either an API, or something as a convention of reading the file .guild/attrs/label - which sits alongside all of the other run attributes, which you can read as needed.
The as-of-yet unreleased Guild API is guild._api and does have pretty easy support for this.
from guild import _api as gapi

print(gapi.current_run().get("label"))

This interface will not change and we'll always provide guild._api even when we promoted this, so you can trust this will work forever.

What I prefer about the env var approach is that it does not assume that guild is available inside the context of a run. In my opinion, we need to keep guild out of the run environment as soon as we can manage, but this proposed change would introduce a new, tighter coupling. It would also introduce a runtime requirement on Python, even if the run had nothing to do with Python.
I guess this is an argument for either providing the env var, or for providing some shorthand standard or command for reading the .guild/attrs/label file.


Fix address reuse if recipient asset !== L-BTC

related to https://github.com/vulpemventures/ldk/pull/119
The LDK fix lets us reuse the changeAddressGetter used in taxi pset. If the recipient asset is not L-BTC, a specific change address will be generated.
This PR also includes a small fix in txUpdater saga: now, we force re-updating of transactions where transfers.length === 0. It lets to fix bad Inbound/Outbound displaying in case of wallet with gap.
it closes #340
@tiero @Janaka-Steph please review

Shall we use ldk@next in marina? since we are going to support nicely<EMAIL_ADDRESS>

Shall we use ldk@next in marina? since we are going to support nicely liquidJS v6.x.x?

I would wait #345 (to test is everything is working well with update)

Test are failing?

in case of wallet with gap.

Not sure problem happens for a gap limit reached TBH, I guess would happen anyway

having problem trying to restore seed with a build for production
Restoration failed
Unsuported witness version (undefined), only 0 (blech32) or 1 (blech32m) are supported

included in #345


Drop YSQL tables only if they don't exist

Fixed couple apps so that they will only try to drop tables if they don't already exist. This was causing issues with repeated runs of yb-sample-apps.

LGTM
Verified that these should be the only places where 'IF EXISTS' was omitted


Question

How to modify or add my own translation

I tried to modify the en.sahih.xml it did not make any changes.
I want to add my own translation in our dialect, Thank you 👍


Where is export to GPX?

Hi There,
I see the export to FIT and TCX, but no export to GPX.... am I missing something?
(Strava's standard GPX export option is there, but that doesn't include trackpoint times)
Thanks!

I disable the GPX export if it's already an export option from Strava.

Would you consider including the your GPX export regardless? The built-in Strava GPX-export does not include times or watts or heartrate, etc.  Thanks!

All views show Sauce GPX ... but yes it has removed.


Maybe add Microsoft?

Not a small name on the list :)
https://microsoft.github.io/reactxp/blog/2017/04/06/introducing-reactxp.html

oh u right 👍 . Just did :)


Unable to bake NavigationMesh for certain meshes but it works through editor menu Mesh - Create Navigation Mesh.

Tested versions
4.3[2e7fc81]
System information
Windows 11 - Vulkan - Nvidia RTX 4070 - intel i5 13600KF
Issue description
For some reason when I add NavigationRegion3D I can't bake a navigation mesh for certain objects, no errors just no mesh appears but if I use the same MeshInstance3D and use the built-in menu Mesh - Create Navigation Mesh then it works, obviously this has a disadvantage that you can't change any nav mesh baking settings for it.
Similar thing happened to me on some other meshes too, this one is particularly maybe strange for nav mesh generation so I'm using it as an example (I also don't have any other anymore), it's a partition of sphere, just to be clear even if there wasn't the empty division in between the same issue occurs.
I understand the mesh is probably not standard but it works one way so I'd assume there should be a way to make it work both ways perhaps?
Steps to reproduce

Open the project, there are 3 NavigationRegions, one with the blen scene added the other with MeshInstance3D copied from the openedi inherited blend scene and the third one is the mesh resource saved through the import settings and assigned to a fresh new MeshInstance3D.
You can click on any region and at the top of the viewport press Bake NavigationMesh button and observe nothing changes.
Now click on the last MeshInstance3D and at the top of the viewport click on Mesh -> Create NavigationMesh and observe a new NavigationRegion3D is created with the mesh as its child and most importantly the navigation mesh is now visible and exists.

Minimal reproduction project (MRP)
NavigationBakeIssue.zip

The MRP is corrupted and missing the blend file to open.
From what I can see with the provided mesh resource alone that is a complete bogus mesh for baking navmesh as it has internal holes and flipped faces.

Hm, I downloaded the archive and it does have the blend file, when it comes to errors in the output when you open it I'm not sure why they started appearing but if you press the play button once they will be gone even on successive loads of the project.
I have three questions here if you don't mind:

I understand the mesh isn't ideal or anywhere near, as mentioned above I made this one on purpose since I currecnlty don't have any other. I had same issue with spheres and cubes for some reason.
Why does it generate the mesh polygon through Import Settings (setting to NavMesh only or NavMesh), does it fail since the mesh is bonkers and then it just loads the mesh data and uses that instead?
Same as 2. but this time through Mesh (menu) -> Create Navigation Mesh.

Since all these generate a NavigarionRegion3D I'd expect them all either fail or pass but a mix so where's the bug, is it in the Mesh menu and Import Settings for the mesh or in the NavigationRegion3D's baking instead?
As a side note, I'll take the 1. failure with regular shapes as per your suggestions that maybe there was some corruption, though I haven't found issues with the models before so maybe something during import can under some condition corrupt some data?

Except for the actual baking all the navmesh import or conversion options that exist around the Editor are just an array copy.
Not a single of those functions is capable of doing any verification that the mesh data makes any sense for navigation.

Thank you, it makes sense. I'm not sure why I had issues with couple models but it might have been just a corruption as you mentioned either in the blend file itself or when it was imported, it wasn't consistent with all my models from the same file at the time though.
Maybe this report could be closed then (or soon if no one else comes with the same issues)?

I think we can close this. Feel free to reply if you want to add something or think it should be kept open.


Update edition and nom dependency

Just a general housekeeping PR. Also, I was wondering if this repo is open to more PRs? I want to add parsers for things like tchar, token, #rule, etc. from the RFCs

Thanks! I would love to review more PRs. Could you say more about where these parsers are coming from?

Of course, in https://github.com/g1eny0ung/parse_link_header/issues/16 I'm going to be trying to implement the Link rule from rfc8288 which builds on a bunch of rules from rfc7230 (superseded by rfc9110). Specifically things like tchar, token, #rule. It feels like these three should be appropriate for this crate (so that they can be re-used elsewhere), and then the actual Link rule can go into https://github.com/g1eny0ung/parse_link_header/


Deployment script produces poor logs when failing on Function only deployment

Describe the bug
When trying to deploy using the function only deployment, poor error messages are given with little information.  Setting debug trace in PowerShell slightly helps.
To Reproduce
Steps to reproduce the behavior:
Run deploy script as follows
PS /home/craig/craig-workspace/azure-ipam/deploy> ./deploy.ps1 `
>>   -Location "uksouth" `
>>   -AsFunction
NOTE: IPAM Deployment Type: Function
INFO: Fetching Tenant ID from Azure PowerShell SDK
INFO: Fetching Azure Cloud type from Azure PowerShell SDK
INFO: Validating Azure Region selected for deployment
INFO: Azure Region validated successfully
INFO: Creating Azure IPAM Engine Application
INFO: Creating Azure IPAM Engine Service Principal
INFO: Creating Azure IPAM Engine Secret
INFO: Azure IPAM Engine & UI Applications/Service Principals created successfully
INFO: Logging in to Microsoft Graph
INFO: Granting admin consent for Azure Service Management API permissions assigned to IPAM Engine application
INFO: Admin consent for Azure Service Management API permissions granted successfully
INFO: Deploying IPAM bicep templates
ERROR: Unable to deploy Azure IPAM solution due to an exception, see logs for detailed information!
Run Log: ../logs/deploy_20230405121940PM.log
Error Log: ../logs/error_20230405121940PM.log

I have Global Admin and owner on my context subscription. I am connected using PowerShell and Azure-Cli on the same context.
Expected behavior
Logs saying what's wrong are more descriptive.
Deploy log
********************** PowerShell transcript start Start time:<PHONE_NUMBER>1940 Username: blah RunAs User: blah Configuration Name: Machine: blah (Unix <IP_ADDRESS>) Host Application: /opt/microsoft/powershell/7/pwsh.dll Process ID: 4753 PSVersion: 7.3.3 PSEdition: Core GitCommitId: 7.3.3 OS: Linux <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Jan 27 02:56:13 UTC 2023 Platform: Unix PSCompatibleVersions: 1.0, 2.0, 3.0, 4.0, 5.0, 5.1.10032.0, 6.0.0, 6.1.0, 6.2.0, 7.0.0, 7.1.0, 7.2.0, 7.3.3 PSRemotingProtocolVersion: 2.3 SerializationVersion: <IP_ADDRESS> WSManStackVersion: 3.0 ********************** NOTE: IPAM Deployment Type: Function INFO: Fetching Tenant ID from Azure PowerShell SDK INFO: Fetching Azure Cloud type from Azure PowerShell SDK INFO: Validating Azure Region selected for deployment INFO: Azure Region validated successfully INFO: Creating Azure IPAM Engine Application INFO: Creating Azure IPAM Engine Service Principal INFO: Creating Azure IPAM Engine Secret INFO: Azure IPAM Engine & UI Applications/Service Principals created successfully INFO: Logging in to Microsoft Graph INFO: Granting admin consent for Azure Service Management API permissions assigned to IPAM Engine application INFO: Admin consent for Azure Service Management API permissions granted successfully INFO: Deploying IPAM bicep templates PS>TerminatingError(New-AzDeployment): "The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: 12:19:54 - Error: Code=InvalidTemplateDeployment; Message=The template deployment failed with multiple errors. Please see details for more information. " >> TerminatingError(New-AzDeployment): "The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: 12:19:54 - Error: Code=InvalidTemplateDeployment; Message=The template deployment failed with multiple errors. Please see details for more information. " >> TerminatingError(New-AzDeployment): "The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: 12:19:54 - Error: Code=InvalidTemplateDeployment; Message=The template deployment failed with multiple errors. Please see details for more information. " ERROR: Unable to deploy Azure IPAM solution due to an exception, see logs for detailed information! Run Log: ../logs/deploy_20230405121940PM.log Error Log: ../logs/error_20230405121940PM.log ********************** PowerShell transcript end End time:<PHONE_NUMBER>1954 **********************

Error Log
New-AzDeployment: /home/craig/craig-workspace/azure-ipam/deploy/deploy.ps1:703 Line |  703 |        New-AzSubscriptionDeployment `      |        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~      | 12:19:54 - Error: Code=InvalidTemplateDeployment; Message=The template deployment failed with multiple errors. Please see details for more information.

Good morning @craigthackerx, and thank you so much for opening this issue!
At first glance, this doesn't appear per-se an Azure IPAM Deployment Script issue, but something within Azure is not allowing the deployment to complete. Can you try checking the Deployment results within the Azure portal?
You can find the Deployment details under the subscription like this:

I would expect you to see a failure here (mine shows success). Click on the failed deployment for additional details like so:

Here you should be able to see which component(s) failed, and perhaps a more clear reason as to why. It could be something like resource availability, quota limit, Azure Policy, etc.
I'm also more than happy to hop on a 1:1 session with you to drill down on the exact root cause at any time. Please feel free to reach out to me at<EMAIL_ADDRESS>and we can get something scheduled.
I know we'll get this resolved soon, looking forward to driving this to closure with you!

Good morning @craigthackerx, I just wanted to circle back with you on this issue to see if you've been able to look a bit deeper into where the deployment failed in the Azure portal, or if you'd like to schedule some time for us to dive into this together.
Let me know how things are going, we're always here to help!

Good morning @craigthackerx, I just wanted to circle back with you on this issue to see if you've been able to look a bit deeper into where the deployment failed in the Azure portal, or if you'd like to schedule some time for us to dive into this together.
Let me know how things are going, we're always here to help!

Apologies for the slow reply, I haven't had a chance to reply over the holidays.
Oddly, I struggled to replicate the issue on another tenant I had.  The tenant I was testing it in had more controls in place, so I find likely that something was interfering with the script, where as the Dev/Test tenant I have has little in the way of control.
You may wish to close this issue and I'll get back to you on here if I can replicate it, I can test what I wanted to test on my personal tenant.

Sounds great @craigthackerx! I've seen similar issues before where something like Azure Policy was preventing some portion of the Azure IPAM infrastructure from deploying.
In the future, you can add the "-Debug" flag to the Deploy.ps1 and it will produce EXTREMELY verbose logs (in the /logs folder). Generally we can parse through those to locate the more precise problem happening during the Bicep template deployment. As always, we're more than happy to assist with this once the Debug logs are generated.
For now I'll close this out, but we're here to help at any point in time!


Add emulatorID field to devices in daemon

Description
This adds an additional emulatorId to devices so allow clients to match them up (and hide already-running emulators from the "Launch Emulator" list). It's currently only exposed in the daemon, though we may also want to expose this in flutter devices (@mit-mit FYI).
@devoncarew @jonahwilliams
For Android this requires creating a connection to the device and running "avd name" (it doesn't work via adb) - see original investigations at https://github.com/flutter/flutter/pull/16705#issuecomment-382742079. It's more important now for VS Code, as the device selector is showing both devices and emulators, and the dupes are weird.
Here's how the Android Console works on a real emulator for reference:
dantup-macbookpro:inetutils-1.9 dantup$ telnet/telnet localhost 5554
Connected to localhost.
Escape character is '^]'.
Android Console: Authentication required
Android Console: type 'auth <auth_token>' to authenticate
Android Console: you can find your <auth_token> in
'/Users/dantup/.emulator_console_auth_token'
OK
avd name
Danny_Nexus
OK

Tests
This still needs tests - I want to be sure we're happy with this before proceeding (and I'm also open to ideas on how best to test this, the only significant code is in connecting to running emulators, and I'm not sure if we have much infrastructure for testing something like that.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read [Handling breaking changes]). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

Thanks!