TypeError when running SaphireTest::logInWithPermission

Module version(s) affected
5.2.7
Description
I am in the process of testing one of my controllers using the FunctionalTest class. One of my tests makes use of the ::logInWithPermission method. Running this alone causes: TypeError "Return value must be of type bool, null returned.
I tested allowing the null to return, returning true and returning false in the function that produces this error. All of these ends up producing SilverStripe\Core\Injector\InjectorNotFoundException: Class SilverStripe\Versioned\RecursiveStagesInterface does not exist
I have silverstripe/versioned installed via composer so I am not sure why this next error is occuring.
How to reproduce
Test:
public function testCheckUserPerms()
{
    $this->logInWithPermission();
}

Possible Solution
I am providing a PR with this, I believe the config should return false on default in this scenario. The InjectionNotFoundException I am not sure about though.
Additional Context
This is a custom silverstripe install without the cms as we are not using Pages
1) Ordino\Lib\Tests\OControllerTests::testCheckUserPerms
TypeError: SilverStripe\ORM\Connect\DBQueryBuilder::shouldBuildTraceComment(): Return value must be of type bool, null returned

Expand to see stack trace
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/Connect/DBQueryBuilder.php:88
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/Connect/DBQueryBuilder.php:76
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/DB.php:141
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/Queries/SQLExpression.php:92
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/Queries/SQLExpression.php:112
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/DataList.php:1031
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/DataList.php:967
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/DataList.php:947
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/DataList.php:1669
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Dev/SapphireTest.php:976
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Dev/SapphireTest.php:1018
/srv/http/ordino-skeleton/ordino-lib/tests/controllers/OControllerTests.php:71

1) Ordino\Lib\Tests\OControllerTests::testCheckUserPerms
SilverStripe\Core\Injector\InjectorNotFoundException: Class SilverStripe\Versioned\RecursiveStagesInterface does not exist

Expand to see stack trace
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Core/Injector/InjectionCreator.php:29
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Core/Injector/Injector.php:631
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Core/Injector/Injector.php:1027
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Core/Injector/Injector.php:979
/srv/http/ordino-skeleton/vendor/silverstripe/versioned/src/RecursiveStagesService.php:28
/srv/http/ordino-skeleton/vendor/silverstripe/versioned/src/RecursivePublishable.php:462
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Core/Extension.php:142
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Core/Extensible.php:438
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/DataObject.php:1251
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/ORM/DataObject.php:1595
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Dev/SapphireTest.php:987
/srv/http/ordino-skeleton/vendor/silverstripe/framework/src/Dev/SapphireTest.php:1018
/srv/http/ordino-skeleton/ordino-lib/tests/controllers/OControllerTests.php:71

Fresh install with similar composer:
SilverStripe\Core\Injector\InjectorNotFoundException: Class SilverStripe\ErrorPage\ErrorPageControllerExtension does not exist in /srv/http/fresh/vendor/silverstripe/framework/src/Core/Injector/InjectionCreator.php:29

Expand to see stack trace
#0 /srv/http/fresh/vendor/silverstripe/framework/src/Core/Injector/Injector.php(631): SilverStripe\Core\Injector\InjectionCreator->create()
#1 /srv/http/fresh/vendor/silverstripe/framework/src/Core/Injector/Injector.php(1027): SilverStripe\Core\Injector\Injector->instantiate()
#2 /srv/http/fresh/vendor/silverstripe/framework/src/Core/Injector/Injector.php(979): SilverStripe\Core\Injector\Injector->getNamedService()
#3 /srv/http/fresh/vendor/silverstripe/framework/src/Core/Extensible.php(536): SilverStripe\Core\Injector\Injector->get()
#4 /srv/http/fresh/vendor/silverstripe/framework/src/Core/Extensible.php(436): SilverStripe\View\ViewableData->getExtensionInstances()
#5 /srv/http/fresh/vendor/silverstripe/framework/src/Control/Controller.php(117): SilverStripe\View\ViewableData->extend()
#6 /srv/http/fresh/vendor/silverstripe/framework/src/Dev/SapphireTest.php(896): SilverStripe\Control\Controller->doInit()
#7 [internal function]: SilverStripe\Dev\SapphireTest::SilverStripe\Dev\{closure}()
#8 /srv/http/fresh/vendor/silverstripe/framework/src/Control/HTTPApplication.php(137): call_user_func()
#9 /srv/http/fresh/vendor/silverstripe/framework/src/Control/Middleware/HTTPMiddlewareAware.php(65): SilverStripe\Control\HTTPApplication->SilverStripe\Control\{closure}()
#10 /srv/http/fresh/vendor/silverstripe/framework/src/Control/HTTPApplication.php(130): SilverStripe\Control\HTTPApplication->callMiddleware()
#11 /srv/http/fresh/vendor/silverstripe/framework/src/Dev/SapphireTest.php(884): SilverStripe\Control\HTTPApplication->execute()
#12 /srv/http/fresh/vendor/silverstripe/framework/src/Dev/SapphireTest.php(391): SilverStripe\Dev\SapphireTest::start()
#13 /srv/http/fresh/vendor/phpunit/phpunit/src/Framework/TestSuite.php(629): SilverStripe\Dev\SapphireTest::setUpBeforeClass()
#14 /srv/http/fresh/vendor/phpunit/phpunit/src/TextUI/TestRunner.php(651): PHPUnit\Framework\TestSuite->run()
#15 /srv/http/fresh/vendor/phpunit/phpunit/src/TextUI/Command.php(146): PHPUnit\TextUI\TestRunner->run()
#16 /srv/http/fresh/vendor/phpunit/phpunit/src/TextUI/Command.php(99): PHPUnit\TextUI\Command->run()
#17 /srv/http/fresh/vendor/phpunit/phpunit/phpunit(107): PHPUnit\TextUI\Command::main()
#18 /srv/http/fresh/vendor/bin/phpunit(122): include('...')
#19 {main}

Expand to see composer.json
{
    "name": "ordino-nz/ordino-skeleton",
    "description": "The Ordino Web Skeleton",
    "require": {
        "php": "^8.1",
        "silverstripe/recipe-core": "~5.2.0@stable",
        "silverstripe/admin": "^2.0",
        "silverstripe/mimevalidator": "~3.1.0@stable",
        "silverstripe/config": "~2.1.0@stable",
        "symfony/process": "^6.3",
        "ext-yaml": "*"
    },
    "require-dev": {
        "phpunit/phpunit": "^9.5",
        "mikey179/vfsstream": "^1.6.11",
        "squizlabs/php_codesniffer": "^3.7",
        "gskema/phpcs-type-sniff": "^82.2"
    },
    "config": {
        "process-timeout": 600,
        "allow-plugins": {
            "composer/installers": true,
            "silverstripe/recipe-plugin": true,
            "silverstripe/vendor-plugin": true
        }
    },
    "prefer-stable": true,
    "minimum-stability": "dev",
    "extra": {
        "project-files-installed": [
            ".htaccess",
            "app/.htaccess",
            "app/_config/mimevalidator.yml",
            "app/_config/mysite.yml"
        ]
    }
}

composer (fresh):
{
    "name": "silverstripe/installer",
    "type": "silverstripe-recipe",
    "description": "The SilverStripe Framework Installer",
    "require": {
        "php": "^8.1",
        "silverstripe/recipe-core": "~5.2.0@stable",
        "silverstripe/admin": "^2.0",
        "silverstripe/mimevalidator": "~3.1.0@stable",
        "silverstripe/config": "~2.1.0@stable",
        "symfony/process": "^6.3",
        "ext-yaml": "*"
    },
    "require-dev": {
        "phpunit/phpunit": "^9.5",
        "mikey179/vfsstream": "^1.6.11",
        "squizlabs/php_codesniffer": "^3.7",
        "gskema/phpcs-type-sniff": "^82.2"
    },
    "extra": {
        "project-files-installed": [
            ".htaccess",
            "app/.htaccess",
            "app/_config/mimevalidator.yml",
            "app/_config/mysite.yml",
            "app/src/Page.php",
            "app/src/PageController.php"
        ],
        "public-files-installed": [
            ".htaccess",
            "index.php",
            "web.config"
        ]
    },
    "config": {
        "process-timeout": 600,
        "allow-plugins": {
            "composer/installers": true,
            "silverstripe/recipe-plugin": true,
            "silverstripe/vendor-plugin": true
        }
    },
    "prefer-stable": true,
    "minimum-stability": "dev"
}

Validations

[X] Check that there isn't already an issue that reports the same bug
[X] Double check that your reproduction steps work in a fresh installation of silverstripe/installer (with any code examples you've provided)

PRs

https://github.com/silverstripe/silverstripe-framework/pull/11241

Got this error weirdly when doing an update from v5.0. Although it was a caching issue, it gave me a fright. If this issue is still outstanding, can it be merged please 🙂 /cc @GuySartorelli

PR merged.
It will be included in the April 2025 minor release.


Wire dart2js through flutter tool, add compilation test

Allows compilation to a stub dart:ui implementation for testing/tooling.
Requires https://github.com/flutter/engine/pull/7739 for missing members + libraries

We've determined that conditional exports are not handled correctly.

We've determined that conditional exports are not handled correctly.

Is it something @vsmenon can help with?

The test did fail correctly but I also did not set the correct exitCode in the tool

Should I remove the config imports, and roll this with the test expected to fail?

What would be the value of that?

We'd at least have a way to keep track of what is currently blocking us

There is an open bug with conditional exports in particular:
https://github.com/dart-lang/sdk/issues/34179
Are imports working?

Should I remove the config imports, and roll this with the test expected to fail?

I'm a fan of tests that are expected to fail! They are good at reminding you to later add a test that expects it to not fail.

I've tried a few different combinations of importing/exporting and I cant reproduce the successful compile anymore. It seems as if the imports are ignore by dart2js, or we're not using the config imports correctly.

This might just be a case of dart2js being too eager with detecting overflowing integer values

Holding off on this til I have a bug to link to against dart2js or config imports

This was a big case of holding it wrong, everything is working now


typegenAutoConfig support for modular context types

We have a use-case for something like this:
typegenAutoConfig: {
  contextType: 'ContextA.Context & ContextB.Context & ContextC.Context',
  sources: [
    { alias: 'ContextA', source: '...a.ts', onlyTypes: [] },
    { alias: 'ContextB', source: '...b.ts', onlyTypes: [] },
    { alias: 'ContextC', source: '...c.ts', onlyTypes: [] },
  ]
}

It is possible by using manual typegenConfig. But going manual ~gives up~ (https://github.com/prisma-labs/nexus/issues/323#issuecomment-554016318) a non-trivial amount of feature-set.
So, got thinking, is our use-case generic enough a pattern to warrant inclusion into typegenAutoConfig?
The pattern is modular context sources. It doesn’t work right now, at least in part (haven’t digested the whole module logic yet) because there is this regex assumption:
    const forceImports = new Set(
      objValues(backingTypeMap)
        .concat(contextType || "")
        .map((t) => {
          const match = t.match(/^(\w+)\./);
          return match ? match[1] : null;
        })
        .filter((f) => f)
    );

'A.A & B.B & C.C'.match(/^(\w+)\./)

[ 'A.',
  'A',
  index: 0,
  input: 'A.A & B.B & C.C',
  groups: undefined ]

meaning B C will not be forced imports, meaning those modular context contributions will be TS type reference errors in the typegen file because they are never imported.
~It would be relatively trivial to to add support for TS & construct into the above regex. Here's a working draft (^(\w+)\.|\s+&\s+(\w+)\.).~
~It might be a bit magical, but of course we would document it, and it seems intuitive.~
-- edit see later proposals below.

Revising part of the idea. Another way to make the type ref would be:
contextType: 'MergedContext'

Given an additional header is added like:
interface MergedContext extends ContextA.Context, ContextB.Context, ContextC.Context

And doing this has material impact on the TS IDE intellisense DX.
So we need a more general solution than my regex tweak suggestion.
I think the simplest thing we can for this feature for this API is add a new contextType option to sources.
If a source is marked as being a forContext then it forces it being an import.
typegenAutoConfig: {
  contextType: 'ContextA.Context & ContextB.Context & ContextC.Context',
  sources: [
    { alias: 'ContextA', source: '...a.ts', onlyTypes: [], forContext: true },
    { alias: 'ContextB', source: '...b.ts', onlyTypes: [], forContext: true },
    { alias: 'ContextC', source: '...c.ts', onlyTypes: [], forContext: true },
  ]
}

This issue needs to be considered in conjunction with what we do in #265.

So its not accurate to say:

But going manual gives up a non-trivial amount of feature-set.

We can wrap e.g.:
      nexusConfig.typegenConfig = async (schema, outputPath) => {
        const configurator = await typegenAutoConfig(autoConfig)
        const config = await configurator(schema, outputPath)

It reduces some of my motivation for this issue. However I still think like having an easy way to express multiple contexts with the autoConfig API could make sense.

Neat idea - I'm initially curious about how/where you're generating these programmatically, mostly for my own reference.
I've also wanted a way to declare context typings dynamically as something that a plugin can modify / add typings for if there's a place it needs to tack local state for the request, e.g. a DataLoader plugin which automatically creates DataLoaders for types/fields and builds them into an object hanging off the request context.
I agree that anything we do here should be part of #265 - I think that an object | array would be fine here, and multiple contexts would just be unioned automatically?

Hey @tgriesser

I'm initially curious about how/where you're generating these programmatically, mostly for my own reference.

The use-case is that the prototype we're building allows something as follows:
app.use(plugin1).use(plugin2).use(plugin3)

Each plugin may optionally contribute toward the app's graphql resolver context.
Make sense?

I've also wanted a way to declare context typings dynamically as something that a plugin can modify / add typings for if there's a place it needs to tack local state for the request, e.g. a DataLoader plugin which automatically creates DataLoaders for types/fields and builds them into an object hanging off the request context.

👍
I wonder if this is related: Currently we have e.g. plugin1 write to disk something like:
      import { Photon } from '${GENERATED_PHOTON_OUTPUT_PATH}'

      export type Context = {
        photon: Photon
      }

      export const context: Context = {
        photon: new Photon(),
      }

Our prototype then has makeSchema reference that file. This works, but of cousre sub-optimal (end-user experiences perf loss b/c IO, plugin author works harder to implement correctly, etc.).
If Nexus plugins could provide context types as strings rather than (in addition to) file paths then we could stay completely in memory I think.
So does this fall under your point or on this second point are we talking about two different ideas?


task/151/refactor-art-scroller

Codecov Report

Merging #152 (e4c4ecb) into main (93c526d) will increase coverage by 0.06%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   97.46%   97.53%   +0.06%
==========================================
  Files          83       84       +1
  Lines        1541     1582      +41
  Branches      303      300       -3
==========================================
+ Hits         1502     1543      +41
  Misses         39       39

Impacted Files
Coverage Δ

client/src/components/ArtScroller.tsx
100.00% <ø> (ø)

client/src/actions/art-scroller-actions.ts
100.00% <100.00%> (ø)

client/src/components/ArtScroller.style.tsx
100.00% <100.00%> (ø)

client/src/reducers/artScrollerReducer.ts
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Symmetry puzzles

Requires changing the editor to always add symmetrical start points, requiring a symmetrical board, and drawing multiple snakes at once (where one is primary, the other passive). May also require some modifications to Path/validate to allow multiple movement paths.

Did not attach pull requests so far, but this implements non-colored symmetry: https://github.com/thefifthmatt/windmill-client/pull/10


Improve border styling classes

Separate border style and width from border color. Add border-sm, border-md, and border-lg for varying border widths.

Separated classes for color and border size.


First merge of Migraphx worker

New worker for inference server; Dockerfile updated to install ROCm and Migraphx libraries in support, 2 example Python scripts and automated test case.

Migraphx is AMD's library for inference on a GPU.  This adds GPU support to the Inference Server.

Can one of the admins verify this patch?

Need to add the file imagenet-simple-labels.json to the tests directory, since QA wants to use it.

Need to add the file imagenet-simple-labels.json to the tests directory, since QA wants to use it.

ZenDNN added the labels here already


Using with MS/TP Router

Node Version: 10.15.0
Node BACstack Version: 0.2.4

Question

Note: Make sure you have read the FAQs
before logging this issue.
Feature Request / Question
I am trying to use node-bacstack with devices behind a MS/TP router. It is unclear to me how to use node-bacstack to accomplish this. The NET and ADR is not obvious or clear to me. I am trying to discover the devices behind device 156 and read/write their properties. I'm new to MS/TP.
Below is the WHOIS return from /examples/read-device.js

Device 156 is the MS/TP router that has child devices/properties. The readproperties function included in the /examples/read-devices.js retuned this JSON for device 156

A bacnet explorer returned this for device 156

Here is code. Mostly unchanged from the example.
read-device.zip

Can you please run your code with debug?
DEBUG=bacnet:* node yourfile.js
And post the log here

@mwitt1337
I had the same issue and found there is support for passing in the network number and mac addr.
For me BACnet router details
ip: <IP_ADDRESS>
network num: 3
ms/tp mac num: 6
Working code example that was tested a few months ago
// different ways to do a who is including a network whoIs
bacnetClient.whoIs();
bacnetClient.whoIs({'net':0xFFFF});
bacnetClient.whoIs({'net':0xFFFF,'interface':'<IP_ADDRESS>'});
bacnetClient.whoIs({'net':3});

//  read a point through a router to an MSTP device
bacnetClient.readProperty({address: '<IP_ADDRESS>',net:3,adr:[6]} , {type: 1, instance: 1}, 85, (err, value) => {
  console.log('value: ', value);
  console.log('err: ', err);
});

In theory when you use on('iAm') to get discovered devices then the net/adr should be included in the "found device details".
@mwitt1337 please try to send the "whoIs" as above ... and then we can see in the log output if the found evices include the details as needed.
Then we can enhance the docs and example

So here are the commands I ran and the debug log output
bacnetClient.whoIs();
sent whoIs<PHONE_NUMBER>063
examples/read-device.js:468
Found Device 3001 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 1234 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 9101 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 156 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
bacnetClient.whoIs({'net':0xFFFF});
sent whoIs<PHONE_NUMBER>669
examples/read-device.js:468
Found Device 3001 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 1234 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 9101 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 156 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 985911 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 862527 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 48871 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
bacnetClient.whoIs({'net':0xFFFF,'interface':'<IP_ADDRESS>'});
sent whoIs<PHONE_NUMBER>212
examples/read-device.js:468
Found Device 3001 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 156 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 1234 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 9101 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 48871 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 985911 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 862527 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
bacnetClient.whoIs({'net':3});
sent whoIs<PHONE_NUMBER>268
bacnetClient.whoIs({'net':0});
sent whoIs<PHONE_NUMBER>749
examples/read-device.js:468
Found Device 3001 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 156 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 9101 on {"address":"<IP_ADDRESS>","forwardedFrom":null}
examples/read-device.js:490
Found Device 1234 on {"address":"<IP_ADDRESS>","forwardedFrom":null}

Yes the debug should be waaaaay more. Please try against and use
DEBUG=bacnet* node Myfile.js
Please

Sorry, I'm still newer to node and I'm using VSCode. What I posted
previously is from the debug console. Is there a logfile I should be
looking at?
Running this in a javascript debug terminal returns errors DEBUG=bacnet* is
not recognized as the name of a cmdlet. I'm missing something fundamental
here.
Running DEBUG=Bacnet*; node read-device.js works but doesn't return
any further detail then what I've posted before.
On Sat, Jul 25, 2020 at 2:17 AM Ingo Fischer<EMAIL_ADDRESS>wrote:

Yes the debug should be waaaaay more. Please try against and use
DEBUG=bacnet* node Myfile.js
Please
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BiancoRoyal/node-bacstack/issues/28#issuecomment-663816369,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGLMPHVCCUNVMPJENKIOG2TR5J2FZANCNFSM4PD7RPAA
.

Just open a shell and "cd" in the relevant directory and run it from there as described above

Thanks for being patient with me ;) Got the debug to work. I loaded the
source into a folder but didn't run npm install...
[image: image.png]
On Mon, Jul 27, 2020 at 12:41 PM Ingo Fischer<EMAIL_ADDRESS>wrote:

Just open a shell and "cd" in the relevant directory and run it from there
as described above
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BiancoRoyal/node-bacstack/issues/28#issuecomment-664507774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGLMPHVI4RYMWAXMIUB4CILR5WU4BANCNFSM4PD7RPAA
.

Hi, read some more here about debug https://github.com/BiancoRoyal/node-red-contrib-modbus/wiki/DEBUG
But with bacnet ;)

The image from my email response didn't post. Here is the debug output. It really doesn't seem to have much more information. Does DEBUG=bacnet* already include all of the options specified in the link?

which way you sent the whois command?

bacnetClient.whoIs({'net':0xFFFF});
bacnetClient.whoIs({'net':0xFFFF,'interface':'<IP_ADDRESS>'});
bacnetClient.whoIs({'net':3});

?

I ran the who is command 4 different ways and attached a text file of the outputs.
net 0.txt
net 0xFFFF.txt
net 3.txt
new 0xFFFF with interface.txt

Are you using wireshark to debug?
Have you tried to read a point value or anything different to a whoIs?

I was able to make some progress on one of the devices.
  bacnetClient.on('listening', () => {    bacnetClient.readProperty({address: '<IP_ADDRESS>',net:2002,adr:[36]} , {type: 8, instance: 48871}, 76, (err, value) => {     printResultObject(48871,value);     console.log('value: ', value);     console.log('err: ', err);    });
I attached a json file with the printResultObject value.
resultobject.txt
Below is a screen shot of the json from the attached file visualized. At the top the Type:8 and Instance: 48871 matches the light panel I was trying to read values from. The child objects at the bottom are confusing me because they are Type:12 which I haven't found what object type 12 is. The value is Type: 2 which is analog output and instance: 601 must be the instance of that analog value.
I was able to read the current value of instance 601
  bacnetClient.on('listening', () => {    bacnetClient.readProperty({address: '<IP_ADDRESS>',net:2002,adr:[36]} , {type: 2, instance: 601}, 85, (err, value) => {     printResultObject(601,value);     console.log('value: ', value);     console.log('err: ', err);    });
This is a major break through and hopefully this helps anyone else trying to sort out reading properties. Next I will be testing writing to these properties.

But you were still not able to get that device by a whoIs call?

No, the devices in the object list are not coming back with the whois. The
MSTP bridge will return with the whois but none of the child devices.
Malachi Witt
440-387-8043
On Mon, Aug 3, 2020, 2:16 AM Ingo Fischer<EMAIL_ADDRESS>wrote:

But you were still not able to get that device by a whoIs call?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BiancoRoyal/node-bacstack/issues/28#issuecomment-667826304,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGLMPHVJWRAH64YT43OX723R6ZI43ANCNFSM4PD7RPAA
.

These are buttons and switches  on a light panel so I don't know if they
should be responding to a whois
Malachi Witt
440-387-8043
On Mon, Aug 3, 2020, 7:20 AM Malachi Witt<EMAIL_ADDRESS>wrote:

No, the devices in the object list are not coming back with the whois. The
MSTP bridge will return with the whois but none of the child devices.
Malachi Witt
440-387-8043
On Mon, Aug 3, 2020, 2:16 AM Ingo Fischer<EMAIL_ADDRESS>wrote:

But you were still not able to get that device by a whoIs call?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BiancoRoyal/node-bacstack/issues/28#issuecomment-667826304,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGLMPHVJWRAH64YT43OX723R6ZI43ANCNFSM4PD7RPAA
.

Interesting is - when checking your screenshot above - that the router device returns some more data in the response. maybe we need to look in how/that they are parsed correctly :-(
Maybe @NateZimmer have an idea? The logic should be the same as in the official repo

To people having the issues:

Ensure you run wireshark while performing tests. Wireshark will give you a nice dissection of the who-is response / generic bacnet responses. From there you can tell if the sub devices are actually being reported. Ideally somehow upload the exported pcap filtering on the bacnet port.

@mwitt1337 Regarding this example:

Here the network number appears to be 1
Here the network address appears to be a 6 byte MAC address. I take it, this is not a MS/TP device then? MS/TP devices will have a network address of a single byte. TBH I haven't tried/tested on my personal fork of addressing by 6byte MAC address. I wonder if whatever bacnet explorer you are using might be masking the network address though its possible it could be the MAC address. Wireshark will tell you the answer
You should first:

Send a who-is to the bacnet router
Create an iAm event listener and log the iAm responses you get

From there you will know how to address the sub devices
To this repo's maintainers:
What did you do with the iAm event emitter? https://github.com/fh1ch/node-bacstack/blob/master/lib/client.js#L290 .
The iAm event emitter should contain:

Address:
-- Router IP
-- Network number
-- Network address
Other info (device ID, max apdu, ...etc)


User signup allows duplicate emails

User is able to signup with an email that is already used.

Need to update the migration files.


Finish C API for HostSession

What
Add missing methods (host, etc) to the HostSession C API
Why
This is required so that a C ManagerInterface implementation can call Host functionality (and log messages, etc...)

C-Api needs more holistic thinking, out of date.


Add inpainting support with Differential Diffusion [WIP]

This is @andreasjansson's WIP to add Differential Diffusion to SD3.

Cherry picked @lucataco's changes from https://github.com/replicate/cog-stable-diffusion-3/pull/5 to get this working.

When using a mask and image, I'm seeing this error:

Prediction failed.
name 'ensure_image_size_multiple_of_eight' is not defined

https://replicate.com/zeke/sd3-inpainting-with-differential-diffusion?prediction=b3bdebghg1rgm0cgab0sdm6zkw

Prediction failed.
name 'sixteen' is not defined

https://replicate.com/p/8qaa3xn5bnrgp0cgabn8gbjrtr

It's working!
https://replicate.com/p/4h0wr0mb2nrgj0cgac99sr9hc0

Hey, the number of outputs being forced to 1 for inpainting is a bit weird. If it's just for differential diffusion, then it's fine, but we should probably remove that parameter to keep things clean
But if this is just adding inpainting on top of the existing text-to-image stuff (which is what it looks like), then I think this code is actually okay as is. I haven't run the code myself, but I've played around with the model at https://replicate.com/zeke/sd3-inpainting-with-differential-diffusion and it seems to work well
I love how it follows the same format as https://github.com/replicate/cog-sdxl/blob/main/predict.py (sdxl version)

On https://github.com/replicate/cog-stable-diffusion-3/blob/b4878b8a66bfd7da1ba905fb39967e612394dda9/predict.py#L197, make num_inference_steps = 28 a parameter to the predict method and have 28 as the upper limit.
I think we should also move:
    negative_prompt: str = Input(
        description="Input negative prompt",
        default="",
    )

to the bottom because SD3 struggles to properly use negative prompts.

@zsxkib I've got a few things going on, so this is not top priority for me at the moment. Feel free to push your suggested changes directly to this branch if you like.


Weak-perspective camera coordinates

Hello, great job.
I've a question about the estimated perspective camera position. I assume that the SMPL model goes as (x,y,z) and after watching the values at the estimated camera I noticed that the first coefficient was big ( x? ) , but I expected the last coordinate to be big ( z )

If you look at how we are computing the orthographic projection:
def orthographic_projection(X, camera):
    """Perform orthographic projection of 3D points X using the camera parameters
    Args:
        X: size = [B, N, 3]
        camera: size = [B, 3]
    Returns:
        Projected 2D points -- size = [B, N, 2]
    """
    camera = camera.view(-1, 1, 3)
    X_trans = X[:, :, :2] + camera[:, :, 1:]
    shape = X_trans.shape
    X_2d = (camera[:, :, 0] * X_trans.view(shape[0], -1)).view(shape)
    return X_2d

You can see that the first coefficient in the camera is the scaling factor whereas the second and third the translation in the x and y direction. So more formally, the transformation we are doing is
[X,Y,Z] -> camera[0] * [X+camera[1], Y+camera[2]]

I hope this helps

Thanks for the fast reply.
This in fact helps alot.

Hello, I can't figure how to use the predicted camera parameters in order to project the predicted points of the model onto the image.

The orthographic projection function that I mentioned above projects the mesh in the normalized image plane. If you want to do the projection in pixel coordinates, you have to convert the projected coordinates from [-1,1] to [0,224] by a simple translation and scaling.
You can also do the perspective projection from scratch as K * [R,t] * [X,Y,Z] as we show in demo.py. Assuming a fixed focal length, first we get the camera translation
camera_translation = torch.stack([pred_camera[:,1], pred_camera[:,2], 2*FOCAL_LENGTH/(224 * pred_camera[:,0] +1e-9)],dim=-1)

This is t in the above projection equation. R will be the identity matrix and K will be
        FOCAL_LENGTH                0                  224/2
K =          0                 FOCAL_LENGTH.           224/2
             0                      0                    1

Thanks once again for the helpful response. Closing it, for now...

Hi,
Can you tell me how you arrived at the tz term in the translation expression
camera_translation = torch.stack([pred_camera[:,1], pred_camera[:,2], 2*FOCAL_LENGTH/(224 * pred_camera[:,0] +1e-9)],dim=-1)? It would be helpful if you could explain the reasoning.

Same question. If you have any mathematical solution, please let me know, thank you.


Image entity and more

I want to build a more customizable draft-based editor for my client work, and this was the most reasonable starting point that I found.  I have a few separate goals that are different from this repo, but a lot of what I'm doing could easily be backported here.  In my fork (https://github.com/thehumaneffort/rich-edit), I have a few different things going on.  I wanted to file nicely separated PRs for them, but they wound up getting conflated.  There are a few different pieces:

Image entity (image-entity branch)
HTML embedding (develop branch)
Usage outside of a React app. (i.e. compilation into a stand-alone JS utility without dependencies) (not done yet)
JSON in-and-out (so I can store JSON in a HAStore and do intelligent things for previews & more.ds
Docker-based development path. (docker-dev).

This is more to let you know that I'm running with this as a starting point, and am happy to help move this project forward, but I don't have a lot of time for nice PRs yet 😢 .

Thanks a lot for your work and letting us know about it!


Remove temporary replace directive from go.mod of mmagent

This replace directive was temporary and enabled to make changes both in pillar and wwan microservice within the same PR. Now that the PR is merged, we can point mmagent to pillar from the upstream EVE repo.

Get rid of those "replace" everywhere! :-)


Auto-loading of the input system behaved falsely, due to a stuck EditorPrefs variable

As described by  @SamuelAsherRivello here: https://github.com/SamuelAsherRivello/rmc-mini-mvcs/pull/5#issuecomment-1878000588 the auto-loading of the input system behaved falsely due to a stuck EditorPrefs variable.
I used a Editor feature that behaves like PlayerPrefs. It is called EditorPrefs and creates data point in the Registry of the users PC.
EditorPrefs.SetBool("PackageInstalled", false);
EditorPrefs.SetBool("PackageInstalled", true);

So if you ever installed it and didn't click the 'Reset PackageInstalled Switch' the PackageInstalled variable in the registry stays true. I think I might be able to provide a fix. By resetting the _PackageInstalled Variable if it exists to false everytime it checks and only set it to true if it is present already.
if (EditorPrefs.HasKey("PackageInstalled"))
    EditorPrefs.SetBool("PackageInstalled", false);

The we can also get rid of the button. (Where I also made a variable name copy paste mistake. 😅)
Originally posted by @KReguieg in https://github.com/SamuelAsherRivello/rmc-mini-mvcs/issues/5#issuecomment-1879834581

Thanks! I'll check it soon

Tested with:
New project, no registry key available and it works. ✅
Existing project, registry key available and both values and it works as well. ✅
Will create a new PR from a fix branch. 🍻

Corresponding PR: #7


Add Total inference to mlops-machine-learning-model

Relevant information

Added a golden and summary metric, hopefully it will be shown in the entity explorer

Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

 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.

Hi,
Is there anything I should be doing for the validation step?
Thanks,
Yaron

@albertoolivan Is there anything else I should do to merge?
The workflow is awaiting approval


[Question] LiveKit with local network and no internet connection.

Hi!
Hope you can give me any hint of the implementation I'm trying please.
I'm using LiveKit in a local network, it's a Windows app, sometimes with no internet connection but all the devices are connected to the same network. It turns out that when there's no internet connection LiveKit doesn't work and I get an error in signal_client.dart line 121.
I'd like to know if this approach is possible with the current version of LiveKit.
I'm using the following stack:

Flutter (Channel stable, 3.22.2, on Microsoft Windows [Version 10.0.22631.3737], locale en-US)
livekit_client: 2.1.5 (Flutter version).
I'm using this LiveKit server version (1.6.1)

Thanks in advance. I'd really appreciate any hint or response. And any additional info I'll be willing to provide you.

Please do not file duplicate tickets across repos.


tap to click is not working on arch+gnome on hp chromebook 14 (intel model)

after a chroagh update, tap to click stopped to work on gnome.
the mousepad scrolling works great.
the feauture works on other non-gnome desktop environments.
PS: sorry for my horrid, tarzanesque english
PS2: chroagh is cool :)

Install libinput


[To Main] In-app role management

Issue #: https://apps.itsm.gov.bc.ca/jira/browse/DESENG-574
Description of changes:
- Introduced new database models to manage user roles and user-to-role mappings in the Met database.
- Revamped the authentication validation process to rely on roles retrieved directly from the database,
rather than relying solely on token-based roles.
- Eliminated redundant or unused roles from the system.
- Updated unit tests to to fetch roles directly from the database instead of relying on role generation from the factory and eliminated the Keycloak mock.
- Renamed admin and viewer role.
- Removed the usage of CSS API for role management.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report
Attention: Patch coverage is 92.04545% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 75.66%. Comparing base (eeb51e3) to head (395493d).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2470      +/-   ##
==========================================
- Coverage   75.69%   75.66%   -0.03%
==========================================
  Files         586      590       +4
  Lines       21013    21049      +36
  Branches     1529     1528       -1
==========================================
+ Hits        15905    15927      +22
- Misses       4862     4876      +14
  Partials      246      246

Flag
Coverage Δ

metapi
87.83% <92.04%> (-0.10%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

met-api/src/met_api/config.py
97.33% <ø> (ø)

met-api/src/met_api/models/__init__.py
100.00% <100.00%> (ø)

met-api/src/met_api/models/user_role.py
100.00% <100.00%> (ø)

met-api/src/met_api/resources/staff_user.py
93.75% <100.00%> (+0.05%)
:arrow_up:

met-api/src/met_api/services/authorization.py
89.47% <100.00%> (+0.58%)
:arrow_up:

...et-api/src/met_api/services/participant_service.py
83.33% <ø> (-2.39%)
:arrow_down:

.../met_api/services/staff_user_membership_service.py
86.04% <100.00%> (ø)

.../met_api/services/user_group_membership_service.py
100.00% <100.00%> (ø)

met-api/src/met_api/utils/constants.py
80.00% <100.00%> (-15.00%)
:arrow_down:

met-api/src/met_api/utils/enums.py
100.00% <100.00%> (ø)

... and 10 more

... and 5 files with indirect coverage changes


Support get name of seat default from ini

ivi-input-modules: read default seat name from weston.ini file
On weston 10, the default seat name is "default". Recently, the  wayland backend is using the 'seat0' is the default seat name.
This made unexpectations on the input examples and tests.  To avoid this situation in the future, the default seat name
should be configured in weston.ini file.
ilmInput: support ilm_getDefaultSeat api to return default seat name
Sometimes the client need to know the default seat is available or not, or it wants to know which seat is default among multi       seats.
The seat_create_event should add more an argument for identification of default seat. The ilm_getDefaultSeat is going to query the list of seats to find the default seat name.
ilm_input_test: update tests with default seat
Currently, tests are using the 'default' as the default seat name. In the case, the default seat name is not the 'default', tests will
return a failure.
To avoid this, need to check the default seat is available or not. If default seat available, will run tests. otherwise.

This is the new sequence for get the name of seat default:

Tested-by: Au Doan Ngoc<EMAIL_ADDRESS>

@khangtb1 ,

To avoid this, need to check the default seat is available or not.
If default seat available, will run tests. otherwise.

In the commit message, above statement is incomplete.

Recently, the
wayland backend is using the 'seat0' is the default seat name.

It good to mentione corresponding weston version in the comment rather than "Recently".

This made unexpectations on the input examples and tests.

Statement correction please. e.g. "This resulted in unexpected test results for input examples and tests".


Installation guide for Astro

Describe the feature
I was struggling to get Formkit to work in Astro.build. The docs only explain for Vue and Nuxt.
Ended up getting it to work, so for the people also struggling:
astro.config.mjs
import { defineConfig } from 'astro/config';
import vue from '@astrojs/vue';

export default defineConfig({
  integrations: [
    vue({ appEntrypoint: '/src/pages/_app' })
  ],

src/pages/_app.ts
import type { App } from 'vue';
import { plugin, defaultConfig } from '@formkit/vue'

export default (app: App) => {
    app.use(plugin, defaultConfig);
}

Since Astro is quiet popular and getting more fans every day, I think it would be great to add it to the docs. Especially since a lot of Astro users come from tools like 11ty, hugo and Jekyll, not vue.

@silveltman Thanks for the request! We'll get this added.

We added a section in the intallation guide on how to integrate Astro and Formkit.

Looks neat 👌🏼
I also tried configuring formkit in a formkit.config.js file in my Astro app, expecting it to work since Astro uses vite too. Or does this only work in nuxt?
Here  you briefly mention Nuxt, but no Astro

I believe its a nuxt only think, to add the configuration you can still use the formkit.config.js file, but you need to import it and add to the config.
// formkit.config.ts
import { fr } from '@formkit/i18n'
import { DefaultConfigOptions } from '@formkit/vue'

const config: DefaultConfigOptions = {
  locales: { fr },
  locale: 'fr',
}

export default config

// src/pages/_app.ts
import type { App } from 'vue'
import { plugin } from '@formkit/vue'
import config from 'formkit.config'

export default (app: App) => {
  app.use(plugin, config)
}

makes sense. Will give it a try, thanks a lot!


Dev branch

Summary

Adding functionalities for the following endpoints

  1. /order/{id}/submission/{submissionId}

  2. /order/{id}/submission

  3./order/{id}/rejection/{rejectionId}

  4./order/{id}/rejection

  5./order/{id}/amendment/{amendmentId}

  6./order/{id}/amendment

  7./order/{id}/acceptance/{acceptanceId}

  8./order/{id}/acceptance

Adding field called 'type' which takes values Submission, Rejection, Acceptance, Amendment for the corresponding actions while inserting into order_transaction table

Adding validation for all order get endpoints which required id

Raise PR to develop branch


Illustrations test unreadable in dark mode

The text on illustrations are unreadable when dark theme is active. Readable on light theme:

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

ID: c5064f6a-f469-4b7c-101c-e62038f1efb6
Version Independent ID: 4e830181-4699-703f-cf74-5c468628c082
Content: Track Data Changes - SQL Server
Content Source: docs/relational-databases/track-changes/track-data-changes-sql-server.md
Product: sql
Technology: ****
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@burgieb -- thank you for reporting this.  I am updating the article.


Next release

//cc @palainp this is a combination of #147 #142 #144, and also updates the opam-repository to mirage-qubes 0.9.3. I updated the sha256.
Would you mind to see whether you get the same result, and if yes, I'd be keen on merging this and cut a fresh 0.8.1 release.

It's good to me! Thanks for the next release.
I got the same sha256, and the firewall starts correctly+support load as before.


java.lang.NullPointerException: Cannot read the array length because "" is null?

// __ java.lang.NullPointerException: Cannot read the array length because "" is null
$ which java
.../graalvm-ce-java17-22.2.0/bin/java
$ java -version
openjdk version "17.0.4" 2022-07-19
OpenJDK Runtime Environment GraalVM CE 22.2.0 (build 17.0.4+8-jvmci-22.2-b06)
OpenJDK 64-Bit Server VM GraalVM CE 22.2.0 (build 17.0.4+8-jvmci-22.2-b06, mixed mode, sharing)
$ which javac
.../graalvm-ce-java17-22.2.0/bin/javac
$ javac -version
javac 17.0.4
$ ls -l ./gcxml.jar
-rwxrwxrwx 1 root root 52922 Oct  8 04:24 ./gcxml.jar
$ file ./gcxml.jar
./gcxml.jar: Java archive data (JAR)
$ sha256sum --binary ./gcxml.jar
746a6a37d5bd80929a78522a079b964a0879305714f0223b7f9b316a97411bb7 *./gcxml.jar
$ java -jar ./gcxml.jar -xml
gcxml 0.5.0
java.lang.NullPointerException: Cannot read the array length because "" is null
GENERAL:
-version                print version information (current: 0.5.0)
-help                   show list of available commands
CONVERSIONS:
-all                    convert latest TSV to XML and TSV sub-versions for each specific PDF version
-xml           convert TSV to XML for specified PDF version (or all if no version is specified)
-tsv                    create TSV files for each PDF version
QUERIES:
-sin <version | -all>   return all keys introduced in ("since") a specified PDF version (or all)
-dep <version | -all>   return all keys deprecated in a specified PDF version (or all)
-kc                     return every key name and their occurrence counts for each version of PDF
-po key<,key1,...>      return list of potential objects based on a set of given keys for each version of PDF
-sc                     list special cases for every PDF version
-so                     return objects that are not defined to have key Type, or where the Type key is specified as optional
Note: output might be too long to display in terminal, so it is recommended to redirect the output to file (eg  > report.txt)
$

The documentation for the gcxml utility is poor, but you need to run it from the arlington-pdf-model folder. See https://github.com/pdf-association/arlington-pdf-model#gcxml-java where it says:

The Java gcxml.jar file must be run in this top-level folder (such that ./tsv/ and ./xml/ are both sub-folders).

So CLI needs to be:
java -jar ./gcxml/dist/gcxml.jar -xml

I should also add that gcxml has not been fully maintained since the majority of users are using the master TSV data or JSON/Jupyter data models. Some of its features may no longer work with the latest round of changes to the internal grammar.
gcxml is currently only being used to convert the TSV data to an XML equivalent, but the XML processing has likely rotted.


How to patch libraries that make copy of requests.Session()

Problem:

https://github.com/pkkid/python-plexapi/blob/4.5.2/plexapi/server.py#L107

    def __init__(self, baseurl=None, token=None, session=None, timeout=None):
...
        self._session = session or requests.Session()

It saves a copy of the requests object, so, therefore requests_cache can't do monkey-patch on/off.
What I want to do is make a decorator for methods in a class that does with requests_cache.enabled(trakt_cache, expire_after=expire_after):

https://github.com/Taxel/PlexTraktSync/pull/295/files

See my research:

https://github.com/Taxel/PlexTraktSync/issues/307#issuecomment-830227300

Yeah, that looks better. This should work as long as there's only one session object, and only one thread using it.


Capability map computing technology pillar - add missing capabilities

:white_check_mark: Checklist

[x] This pull request has a meaningful title.
[x] If your changes are not yet ready to merge, you have marked this pull request as a draft pull request.

:ballot_box_with_check: Maintainers' checklist

[x] This pull request has had the appropriate labels assigned
[x] This pull request has been added to the SATRE backlog project board
[ ] This pull request has been assigned to one or more maintainers

:arrow_heading_up: Summary
This PR should add (and remove if needed) any missing capabilities from the computing technology pillar, that @machintim has added to his capability decomposition.
So far:

data analytics

:closed_umbrella: Related issues

:raising_hand: Acknowledging contributors

[ ] All contributors to this pull request are already named in the table of contributors in the README file.
[ ] The following people should be added to the table of contributors in the README file:

Please update the capability map to include changes from other PRs.

Closing this, will open a new PR if this does get added


Angular v2 sample events

Coverage remained the same at 83.279% when pulling 36f84760855e8a2b3e2d37c994e24528e796560c on angular-sample-broadcast into 94f05fd71e9028f5f32b07f45075e6c76cf21cb9 on dev.

@rellis-of-rhindleton Thanks again for your feedback in #1244. We took it into account here. Please let us know what you think of MsalBroadcastService's use of observables.

It looks good to me. Thanks for doing it.
You may want to make msalSubject$ a get-only property so people don't shoot themselves in the foot, but I guess that isn't likely.


MarkSeries datapoint fill white renders fill black and side effect of VerticalRectSeries fill black

Clicking the button in example causes the MarkSeries to get a fill field added with value '#ffffff'.
When this occurs the fill of mark on screen changes to black and the fill on screen of the VerticalRectSeries also changes to black despite its data series not being modified.
Extra noticed behaviour if I set the property fill on MarkSeries to say '#ffff00' it sets it and the
fill value in the data points does not override it when toggled, the VerticalRectSeries still gets black fill side effect.
This example demonstrates https://codesandbox.io/s/reactvis-test-test-markseries-fill-8wtht
The behaviour occurs in Firefox and Chrome and Edge on my Windows 10 desktop.
"react": "^16.8.6",
"react-dom": "^16.8.6",
"react-vis": "^1.11.7",

I am going to close this, as once I set colorType to "literal" on mark and rect series, i can set literals on the data points in the series to get expected behavior.


q parameter is always appended to the URL even if empty

Click https://npmgraph.js.org/
Click anywhere on the page to un-focus the field
Notice that the URL changed to https://npmgraph.js.org/?q=

The code in question: vscode.code-workspace


Migrate to .NET 8

.NET 8 is a TLS version, and it would be preferred to upgrade to that.

Solved


Fix v1p1beta1 unit tests

No idea.
On Fri, Mar 16, 2018, 6:17 PM Tres Seaver<EMAIL_ADDRESS>wrote:

@tseaver approved this pull request.
How did we get merged with the borked test?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-python/pull/5064#pullrequestreview-104751545,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc9odJja8RmUul87KpLvvhmuvqAMQks5tfGQLgaJpZM4Sug2i
.


Add support for Vespa Node's and indexing with additional fields

Description
This PR adds support for indexing nodes to vespa with custom fields other than text and metadata.
Issue #13319
Motivation
A schema in Vespa has many fields that can be searched from and referenced in ranking profiles. This PR enables users to define the values for their vespa fields with a VespaNode.
Type of Change

[✓] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[ ✓] Added new unit/integration tests

Suggested Checklist:

[✓ ] I have performed a self-review of my own code
[ ✓] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[ ✓] My changes generate no new warnings
[ ✓] I have added tests that prove my fix is effective or that my feature works
[ ✓] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

Notes
Development decisions

Tests now try to get the local running vespa instance rather than deploying each time when tests run. This is done to make TDD possible. A caveat is to check the locally running vespa instances to avoid making accidental changes to other instances
Nodes are added and deleted before and after each test. This is to avoid conflicts between tests and versioning issues
When adding a node, an isinstance check checks for whether the passed in nodes are Vespa and if so add the vespa_fields to the fields in entry
When querying, the fields of the hits are checked for length. If its bigger than 5, its assumed that custom fields have been added and its treated as a vespa node. This check could be made safer

Future Considerations

Vespa Node's only define a dictionary and doesn't do any type checking. To avoid this, support for customizing Vespa Node could be added. The fields added to new node would be treated as vespa fields. This could be done with pydantic however is out of the scope of #13319

Note to maintainers
Please don't merge this before @thomasht86 reviews and confirms the changes

Hi great you liked it! I am overwhelmed with work on my side rn so can't say for sure but will try if I get some free time on my hands 🤟


Document testing part

Add some documentation inside tests directory about testing involve in sdk.
Currently while running unit tests error is thrown
└ make test-php-unit
vendor/bin/phpunit --configuration ./phpunit.xml --testsuite unit
PHPUnit requires the "dom", "json", "libxml", "mbstring", "tokenizer", "xml", "xmlwriter" extensions, but the "dom", "mbstring", "xml", "xmlwriter" extensions are not available.
make: *** [Makefile:24: test-php-unit] Error 1

Fixes

need to install mbstring in php8.2

Run php phan test gives following error
vendor/bin/phan --config-file .phan/config.php --require-config-exists
ERROR: The php-ast extension must be loaded in order for Phan to work. Either install and enable php-ast, or invoke Phan with the CLI option --allow-polyfill-parser (which is noticeably slower)
php-ast can be installed in the following ways:

1. Unix (PECL): Run 'pecl install ast' and add extension=ast.so to your php.ini.

2. Unix (Compile): Download https://github.com/nikic/php-ast then compile and install the extension as follows:

   cd path/to/php-ast
   phpize
   ./configure
   make
   sudo make install

   Additionally, add extension=ast.so to your php.ini file.
For more information, see https://github.com/phan/phan/wiki/Getting-Started#installing-dependencies
make: *** [Makefile:39: test-php-phan] Error 1

can we add this to the composer.json file?

@individual-it i tried to added in json file but it says our system miss this packages
Problem 1
    - Root composer.json requires PHP extension ext-curl * but it is missing from your system. Install or enable PHP's curl extension.
  Problem 2
    - Root composer.json requires PHP extension ext-ast * but it is missing from your system. Install or enable PHP's ast extension.

can be done under https://github.com/owncloud/ocis-php-sdk/issues/191


build fail: could not find a package configuration file provided by "costmap_queue"

Need to install the following packages before a successful build:
costmap_queue, dwb_critics, move_base_msgs, rospy_message_converter

Modified readme to include the above information.


Leading incorrectly defined

In typography, leading is the distance between 2 baselines of text as shown here:

However, in typst, leading is the space between the bottom of a line and the top of the next line; as shown in this picture which was generated using this code, with zero leading:
#set text(size: 20pt) #set par(leading: 0pt) #lorem(35)

Ok, thanks for the clarification.

If leading means inserted space (lead) between lines, why do lines overlap when I set leading to zero?

If leading means inserted space (lead) between lines, why do lines overlap when I set leading to zero?

Because the bounding box of text isn't exact, depending on what top-edge and bottom-edge are set to

leading is by default defined to 0.65 em (according to the docs) and I get zero "baselineskip" when I set leading to -0.65 em. Why has this to do with top-edge and bottom-edge?

leading is by default defined to 0.65 em (according to the docs) and I get zero "baselineskip" when I set leading to -0.65 em. Why has this to do with top-edge and bottom-edge?

Because leading is by definition the definition between the bottom edge of one line and the top edge of the next.

Ok, but then if you want to follow the "pre-digital meaning" of leading, you should define top-edge to "ascender" and bottom-edge to "descender".
If I want to define an exact baselineskip as it is now, I have to set them both to zero and set the leading to the baselineskip I want.

TL;DR: Typst currently produces a line height of 1.2 (baseline to baseline) for the 0.5em "leading" setting. If that is "incorrect" may be up for debate, but at the very least it is weird and not intuitive by modern standards. So unless Typst wants to dethrone Word for its funny line height handling, this should get fixed.

The term "leading" actually refers to the lead (as in lead, the metal) that typesetters used to space out lines. Following this you could argue it is the distance from the descender (e.g., the lowest point of ")", "y" or "g" etc.) to the ascender of the next line (often "I", "f", or ")"). At least this is the definition that Sheldon Cooper would give in "Fun with Flags".
It used to look like this:

Image Source: http://georgi3porgi3.blogspot.com/2017/07/typography-tangles-leading-kerning-what.html
That being said… I have probably used around 20-30 typesetting and layout tools in the last +10 years, and not a single one uses leading this way. Most sensible tools use a dimensionless factor, so 1.5 = 1.5em = 150% (baseline to baseline, in CSS dimensionless and percentage are not exactly the same, but the same for the same font size – different story). Microsoft Word does something odd unless you select the "line spacing exactly" option, but then again, I would NOT (!) consider Word a "sane" tool.
The typst documentation currently states that leading is "The spacing between lines," which is a bit vague… if we're talking about baselines (which is now the common understanding outside Sheldon Coopers "Fun with Flags"), then the documentation is indeed incorrect.
BUT… on top of the pedantry:
#set text(font: "Arial", 10pt)
#set par(leading: 0.5em)

This results\
in a line height of 1.2 instead of 1.5 (baseline to baseline, as in HTML, Illustrator and Figma)

If there is a good reason for this, I'm eager to learn it. But what currently happens is that 40% (?) of the leading value gets added between the bottom and top of the next line. So the term "incorrectly" might be subjective, but I'd say it is at least very, very odd.
There is probably a reason this gets mentioned over #159 over #4224 and over #5225 (my own, indirectly related, issue) and over again.

@woerndl See https://github.com/typst/typst/issues/4224#issuecomment-2423254670.

@laurmaedje Thanks, Laurenz! By the way, congratulations on 0.12. Fantastic release—keep up the great work! 🎉


📝 Fastspeech2 implementation.

Related to #38. This PR is an implementation of FastSpeech 2 (https://arxiv.org/abs/2006.04558) paper. Note that this PR is just implement FastSpeech2 first, Fastspeech2s will implement after i can reproduce fastspeech2.

[x] Add code to extract f0 and energy from preprocess.
[x] Add code for FastSpeech2.
[x] Add FastSpeech2Trainer.
[ ] Update Full ReadMe
[ ] Upload pretrained.

@dathudeptrai  datasets.AudioDataset  not found

Be patient;))))

Be patient;))))

ok  :grin: :exploding_head:


fix: SELinux denials - dbus send msgs, squid port connect

squid port is needed when host-metering is using a proxy that uses port 3128 via the HTTS_PROXY env var.
dbus send message denial was observed after subscription-manager calls.

RPM packages with this build are available in my branch of copr: https://copr.fedorainfracloud.org/coprs/pvoborni/host-metering-dev/ , build: https://copr.fedorainfracloud.org/coprs/pvoborni/host-metering-dev/build/6548283/
It can be used to verify that it works.
The policy change was developed with a help from @wrabcak

Verified with the build rpm:
# getenforce
Enforcing

# rpm -qa host-metering\*
host-metering-0.1.0-git202310191511G753617c9.el7.x86_64
host-metering-selinux-0.1.0-git202310191511G753617c9.el7.noarch

# cat /etc/systemd/system/host-metering.service.d/00-proxy.conf
[Service]
Environment=HTTP_PROXY=http://[XXXXX]:3128
Environment=HTTPS_PROXY=http://[XXXXX]:3128

# journalctl -xe
[...]
Oct 26 15:04:07 ip-10-31-62-249.us-east-1.aws.redhat.com host-metering[9692]: Collecting metrics...
Oct 26 15:04:07 ip-10-31-62-249.us-east-1.aws.redhat.com host-metering[9692]: Metrics collected
Oct 26 15:04:07 ip-10-31-62-249.us-east-1.aws.redhat.com host-metering[9692]: Initiating notification request...
Oct 26 15:04:07 ip-10-31-62-249.us-east-1.aws.redhat.com host-metering[9692]: Sending 1 sample(s)...
Oct 26 15:04:07 ip-10-31-62-249.us-east-1.aws.redhat.com host-metering[9692]: Notification successful
Oct 26 15:04:07 ip-10-31-62-249.us-east-1.aws.redhat.com host-metering[9692]: Server fully started


COM: Rework components to be 'mobile first'

Which is basically a hipster way of saying "make them work when they're small". Fiddly stuff with the media queries. Setting some standards for new components.

Moved to wishlist. Not going to be done in time.

I can vouch that the mobile experience on the new LDJam.com could be better. It's workable, and at least it lets me zoom to a proper level. But there's some small enhancements that could make the experience a bit better.
Maybe I'll tinker around with some of the basic settings to improve the main experience, without fiddling too much with the separate components.

Ya, all I have time for right now is making sure it works (mobile), not making sure it works well.
This is a pretty important part of the site I'm discovering. Voting for themes on mobile was way more popular than I expected.


App crashes just after camera preview is shown (Android 6)

Expected Behaviour
Camera preview opens, i see myself (in front camera) until I use a QR code.
Actual Behaviour
Camera preview opens, i see myself (in front camera) for 0.5 second, the whole app crashes.
Reproduce Scenario (including but not limited to)

It happens only on one particular device with Android 6 with only one (front) camera.
Other camera apps works well on this device.
Another cordova camera plugin also works well (https://github.com/cordova-plugin-camera-preview/cordova-plugin-camera-preview).

Steps to Reproduce

It happens every time I run this plugin. With default configuration and also with configuration:
formats: "QR_CODE"

Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android 6.0
Cordova CLI version and cordova platform version
cordova --version : 7.1.0
cordova platforms : android 6.3.0

Plugin version
cordova plugin version | grep phonegap-plugin-barcodescanner : 7.0.2

Logs taken while reproducing problem
(full logcat with other info about camera and formats selected are in attached here
errorLog.txt
)
FATAL EXCEPTION: Thread-496
Process: com.our.packagename.ui, PID: 8391
java.lang.ArrayIndexOutOfBoundsException: src.length=460800 srcPos=461160 dst.length=810000 dstPos=46800 length=1200
	at java.lang.System.arraycopy(Native Method)
	at com.google.zxing.PlanarYUVLuminanceSource.getMatrix(PlanarYUVLuminanceSource.java:101)
	at com.google.zxing.common.HybridBinarizer.getBlackMatrix(HybridBinarizer.java:70)
	at com.google.zxing.BinaryBitmap.getBlackMatrix(BinaryBitmap.java:85)
	at com.google.zxing.qrcode.QRCodeReader.decode(QRCodeReader.java:77)
	at com.google.zxing.MultiFormatReader.decodeInternal(MultiFormatReader.java:171)
	at com.google.zxing.MultiFormatReader.decodeWithState(MultiFormatReader.java:85)
	at com.google.zxing.client.android.DecodeHandler.decode(DecodeHandler.java:96)
	at com.google.zxing.client.android.DecodeHandler.handleMessage(DecodeHandler.java:60)
	at android.os.Handler.dispatchMessage(Handler.java:102)
	at android.os.Looper.loop(Looper.java:148)
	at com.google.zxing.client.android.DecodeThread.run(DecodeThread.java:110)

@Giriton yeah, that crash is coming from deep inside the ZXing library the plugin uses for decoding barcodes.

@macdonst I see, thank you for clarification. As far as I found out, this plugin uses barcodescanner-release-2.1.5.aar form here https://github.com/EddyVerbruggen/barcodescanner-lib-aar , correct?
On that page, @EddyVerbruggen says:
Update any source files as needed (current version is: https://github.com/zxing/zxing/releases/tag/BS-4.7.6)
However that version 4.7.6 is from September 2016. Current newest version 4.7.7 is from October 2017. So my question is, is it possible to perhaps update version of BS (Barcode Scanner) used from ZXing to current 4.7.7 ?
In ZXyng, there is a commit that did something around this very row, where the error appears from:
https://github.com/zxing/zxing/commit/2f11529aa35e01354f9036c2aa7747ab23a604ef#diff-3ea115340f67fb0e31e24e4cd9ae7582
So updating Barcode Scanner.aar would update also ZXyng engine used in Barcode Scanner.aar and if problem persist, I know it is still not fixed problem of ZXyng. In that case I can open issue there. Until then, I can not be sure whether the mentioned commit was or was not included in currently used 4.7.6.
Hope it makes a sense :-)
What do you think?

One of my users is facing a similar issue with ZXing library and the crash log looks similar to what you have mentioned.
Android Version is 6.0
Would love to see if there is any update on this issue.

I have detected a similar error on a device with android 7.
Camera error after opening.

Same here, only my app crashes immediately after i open scanner

Also on windows 8.1 this issue

Same here, too.

I really need this barcode scanner works on Android 6, at least!!!

I have same problem.


These are not integer fields, they are booleans. That seems to matter

Which version of postfixadmin are you running?

Seems to be 2.3.6

Indeed, BooleanField looks better. Thank you!


Peering Request

Full Name
Human-life Information Platforms Institute
ASN
63806
PeeringDB Integration

[X] My PeeringDB Entries are current
[ ] Peer at ALL EXCHANGES (and skip the Peer Address(es) field below)

Peer Address(es)

<IP_ADDRESS>
2001:df3:14c0:8000::132

Multihop

[X] Multihop Peering Request

Updates to closed #187.
We kindly ask for multihop peering, if it is possible.


Images scaling inconsistent across devices

Steps to Reproduce
Flutter is rendering same image and text with different sizes on different phones. Check the screenshot below. Left: OnePlus 7, right : Redmi Note 4. Ignore the spacing, check the image and text is bigger on the right one(Redmi). I assume this issue is regarding screen density.

Flutter Doctor
[√] Flutter (Channel unknown, v1.9.1+hotfix.5, on Microsoft Windows [Version 10.0.17763.914], locale en-IN)
    • Flutter version 1.9.1+hotfix.5 at C:\Users\heysu\Documents\flutter
    • Framework revision 1aedbb1835 (3 months ago), 2019-10-17 08:37:27 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\heysu\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 28.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[√] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 42.1.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[√] VS Code (version 1.41.1)
    • VS Code at C:\Users\heysu\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.7.1

[√] Connected device (1 available)
    • Redmi Note 4 • 46e8c6c00504 • android-arm64 • Android 7.0 (API 24)

• No issues found!

Hi @Sp4Rx
I see there's an issue addressing the case you described.
Please follow up on that issue,
I'm closing the current one as duplicate.
If you disagree please write in the comments
and I will reopen it.
Thank you

Hey @VladyslavBondarenk
Thanks for responding. The issue you mentioned is for text scaling only. What about the image scaling?
Above all flutter takes dp as unit right? I don't understand why am I facing this issue

Hi @Sp4Rx
could you please provide a minimal code sample?
thank you

The images look like they're at the same scaling to me.
It's that your positioned widget is 100 from the bottom - which is 100 logical pixels - which is different depending on your devicePixelRatio.
The sample you gave is different from the images - can you share updated images with the sample on your two test devices?
Can you also have the sample print out WidgetsBinding.instance.window.physicalSize and WidgetsBinding.instance.window.devicePixelRatio?

I'm going to reluctantly close this issue for now - if you have additional details to add please feel free to comment here and we can reopen.
Without further details, this looks like it's all WAI to me.


flutter drive often unable to connect to iOS simulator

$ flutter drive --target=test_driver/main_screen.dart
Using device iPhone 7.
Starting application: test_driver/main_screen.dart
Starting Xcode build...
 ├─Assembling Flutter resources...                    5.4s

 └─Compiling, linking and signing...                 21.7s

Xcode build done.                                           42.2s
	[1.1 5ADBF246-FE89-4BD3-859C-25ECE9B4C1F1 2001:b011:2013:18be:c95f:63b6:b85d:5578.60603<->2404:6800:4012:1::200a.443]
	Connected Path: satisfied (Path is satisfied), interface: en0
	Duration: 0.438s, DNS @0.003s took 0.021s, TCP @0.027s took 0.019s, TLS took 0.353s
	bytes in/out: 3325/1065, packets in/out: 10/10, rtt: 0.016s, retransmitted packets: 0, out-of-order packets: 0
5.3.0 - [Firebase/Core][I-COR000003] The default Firebase app has not yet been configured. Add `[FIRApp configure];` (`FirebaseApp.configure()` in Swift) to your application initialization. Read more: https://goo.gl/ctyzm8.
5.3.0 - [Firebase/Analytics][I-ACS023007] Firebase Analytics v.50001000 started
	[4.1 6B1BD2AD-1F1B-4A79-A986-574F15261F45 2001:b011:2013:18be:c95f:63b6:b85d:5578.60615<->2404:6800:4008:803::200e.443]
	Connected Path: satisfied (Path is satisfied), interface: en0
	Duration: 0.116s, DNS @0.000s took 0.002s, TCP @0.003s took 0.021s, TLS took 0.056s
	bytes in/out: 3881/808, packets in/out: 10/9, rtt: 0.018s, retransmitted packets: 0, out-of-order packets: 0
	[5.1 BD18113F-DE5C-465F-BC6E-DB5C70D34729 2001:b011:2013:18be:c95f:63b6:b85d:5578.60618<->2404:6800:4008:803::200e.443]
	Connected Path: satisfied (Path is satisfied), interface: en0
	Duration: 0.102s, DNS @0.000s took 0.002s, TCP @0.005s took 0.019s, TLS took 0.050s
	bytes in/out: 3589/1262, packets in/out: 9/10, rtt: 0.018s, retransmitted packets: 0, out-of-order packets: 0

Meanwhile, the app is stuck in a half-launched state (I can see the splash screen / launch image), but no further progress is made.
The firebase messages can be ignored, I guess.
flutter doctor
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel dev, v0.6.0, on Mac OS X 10.13.6 17G65, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.2)
[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
[✓] Android Studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] Android Studio (version 3.1)
[✓] VS Code (version 1.26.1)
[✓] Connected devices (1 available)

• No issues found!

May depend on the signing configuration used when installing the main app to the device previously. Once I uninstalled the App manually (or used xcrun simctl uninstall), flutter drive was able to function.
This may indicate a further bug when the app has not been installed successfully.


Update scalatest to 3.2.12

Updates org.scalatest:scalatest from 3.2.10 to 3.2.12.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatest", artifactId = "scalatest" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequest = { frequency = "@monthly" },
  dependency = { groupId = "org.scalatest", artifactId = "scalatest" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

[error] (zincTesting / update) found version conflict(s) in library dependencies; some are suspected to be binary incompatible:
[error]
[error] 	* org.scala-lang.modules:scala-xml_2.12:2.1.0 (early-semver) is selected over 1.0.6
[error] 	    +- org.scalatest:scalatest-core_2.12:3.2.12           (depends on 2.1.0)
[error] 	    +- org.scala-lang:scala-compiler:2.12.15              (depends on 1.0.6)


How to calculate the palm distance from the camera

Plugin Version or Commit ID
v0.12.0
Unity Version
2022.3.4f1
Your Host OS
macOS Ventura 13.6.1
Target Platform
Android
Description
Hi, I am currently working on an AR project in Unity, utilizing the hand tracking solution provided by MediaPipe through AR Foundation. My goal is to accurately determine the palm distance from the camera for better interaction with virtual objects.
Currently, I've implemented the hand tracking solution to work with unity arfoundation. However, I'm facing a challenge in ensuring that these annotations are appropriately positioned in 3D space relative to the AR camera. I aim to have the annotations at the correct distance from the camera, allowing users to interact with objects at varying depths using their hands.
To address this issue, I attempted a solution involving placing my hand at different distances from the camera and recording the distance between two specific balls in the hand tracking solution. By analyzing the data, I identified a quadratic pattern and derived the quadratic coefficient to calculate palm distance at runtime. While this approach provided some success, I encountered issues with the annotation not mapping accurately in certain scenarios.
I'm reaching out to seek your guidance. Are there better methods or best practices for accurately determining palm distance in an AR environment using MediaPipe in Unity? I would greatly appreciate any insights or recommendations you may have on improving the accuracy and reliability of palm distance calculations.
Here is a sample of how mine looks like.
https://github.com/homuler/MediaPipeUnityPlugin/assets/63505340/ba9f9bc1-4596-440e-9d7a-a5d683bb5df9
Code to Reproduce the issue
No response
Additional Context
No response

As far as I know, we cannot estimate the depth with MediaPipe, or if we can, the accuracy might not be high.
Therefore, I think you either need to do it by yourself (as you've already done) or combine it with a depth camera for better results.
For accurate information, please ask within the official MediaPipe community.