Enable Passwordless ssh connection for Multinodes docker

Description
passwordless ssh connection is needed for multinodes runs.
In order to have successful mpi init over multiple nodes, changes in the PR are needed.
Related Issue
NA
Changes Made

[x] The code follows the project's coding standards.
[x] No Intel Internal IP is present within the changes.
[ ] The documentation has been updated to reflect any changes in functionality.

Validation

[ ] I have tested any changes in container groups locally with test_runner.py with all existing tests passing, and I have added new tests where applicable.

@tylertitsworth
fixed some pre-commit issues. please help to merge it if it looks good to you.

@tylertitsworth
fixed some pre-commit issues. please help to merge it if it looks good to you.

You have an issue with the idp build of this container: https://github.com/intel/ai-containers/actions/runs/9474288343/job/26103601878#step:5:4409
And your lint is failing due to hadolint warnings.

@louie-tsai perhaps it would just be easier to get your feedback on #124 and merge that one.
Can you try testing with the image via GITHUB_RUN_NUMBER=554 docker compose pull multinode?
We can add a test to verify the ssh support and do some more to verify that the IDP openssl functions properly.

@tylertitsworth
fixed some pre-commit issues. please help to merge it if it looks good to you.

You have an issue with the idp build of this container: https://github.com/intel/ai-containers/actions/runs/9474288343/job/26103601878#step:5:4409
don't understand why we need to run apt-get inside conda environment. also didn't face the issue on our SPR machine.
"#66 ERROR: process "conda run -n idp /bin/bash -c apt-get update -y && apt-get install -y --no-install-recommends --fix-missing     python3-dev     gcc     libgl1-mesa-glx     libglib2.0-0     openssh-server     net-tools     virtualenv &&     rm  /etc/ssh/ssh_host_key     /etc/ssh/ssh_host_key.pub &&     apt-get clean &&     rm -rf /var/lib/apt/lists/*" did not complete successfully: exit code: 100"
is it a testing environment issue?
And your lint is failing due to hadolint warnings.

@louie-tsai perhaps it would just be easier to get your feedback on #124 and merge that one.
Can you try testing with the image via GITHUB_RUN_NUMBER=557 docker compose pull multinode? We can add a test to verify the ssh support and do some more to verify that the IDP openssl functions properly.

I don't see ssh client key handling there.  have you tested it on bare metal without k8s?

I don't see ssh client key handling there. have you tested it on bare metal without k8s?
@louie-tsai users can mount the key at runtime, that is the safest and intended way to communicate between nodes via docker.

don't understand why we need to run apt-get inside conda environment. also didn't face the issue on our SPR machine.

This is the environment that is built when PACKAGE_OPTION=idp, it's one of the versions of this multinode image.

is it a testing environment issue?

No, the weekly testing builds pytorch fine. https://github.com/intel/ai-containers/actions/runs/9432405319/job/26043878040

@tylertitsworth
fixed some pre-commit issues. please help to merge it if it looks good to you.

You have an issue with the idp build of this container: https://github.com/intel/ai-containers/actions/runs/9474288343/job/26103601878#step:5:4409
don't understand why we need to run apt-get inside conda environment. also didn't face the issue on our SPR machine.
"#66 ERROR: process "conda run -n idp /bin/bash -c apt-get update -y && apt-get install -y --no-install-recommends --fix-missing     python3-dev     gcc     libgl1-mesa-glx     libglib2.0-0     openssh-server     net-tools     virtualenv &&     rm  /etc/ssh/ssh_host___key     /etc/ssh/ssh_host___key.pub &&     apt-get clean &&     rm -rf /var/lib/apt/lists/*" did not complete successfully: exit code: 100"
is it a testing environment issue?
And your lint is failing due to hadolint warnings.

only saw below warning related to multinode session. Do we need to fix all warning?
There are warning for original Dockerfile. How do we pass hasolint for original Dockerfile?
"
Dockerfile:76 DL3006 warning: Always tag the version of an image explicitly
Dockerfile:78 DL3008 warning: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>
Dockerfile:122 SC2027 warning: The surrounding quotes actually unquote this. Remove or escape them.
Dockerfile:134 SC2027 warning: The surrounding quotes actually unquote this. Remove or escape them.
Dockerfile:153 DL3006 warning: Always tag the version of an image explicitly
"

I don't see ssh client key handling there. have you tested it on bare metal without k8s?

@louie-tsai users can mount the key at runtime, that is the safest and intended way to communicate between nodes via docker.

if they have 100 nodes, do they need to mount key 100 times?

only saw below warning related to multinode session. Do we need to fix all warning? There are warning for original Dockerfile. How do we pass hasolint for original Dockerfile? " Dockerfile:76 DL3006 warning: Always tag the version of an image explicitly Dockerfile:78 DL3008 warning: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version> Dockerfile:122 SC2027 warning: The surrounding quotes actually unquote this. Remove or escape them. Dockerfile:134 SC2027 warning: The surrounding quotes actually unquote this. Remove or escape them. Dockerfile:153 DL3006 warning: Always tag the version of an image explicitly"

We ignore some warnings in https://github.com/intel/ai-containers/blob/main/.github/linters/.hadolint.yaml, the lint error clearly says you only need to fix the SC2027 errors, which has to do with quoting.

if they have 100 nodes, do they need to mount key 100 times?

Yes, I would assume it would be automated from some blob storage. This is exactly how it works in k8s.

@tylertitsworth
fixed some pre-commit issues. please help to merge it if it looks good to you.

You have an issue with the idp build of this container: https://github.com/intel/ai-containers/actions/runs/9474288343/job/26103601878#step:5:4409

don't understand why we need to run apt-get inside conda environment. also didn't face the issue on our SPR machine. "#66 ERROR: process "conda run -n idp /bin/bash -c apt-get update -y && apt-get install -y --no-install-recommends --fix-missing python3-dev gcc libgl1-mesa-glx libglib2.0-0 openssh-server net-tools virtualenv && rm /etc/ssh/ssh_host___key /etc/ssh/ssh_host___key.pub && apt-get clean && rm -rf /var/lib/apt/lists/*" did not complete successfully: exit code: 100" is it a testing environment issue?

And your lint is failing due to hadolint warnings.

only saw below warning related to multinode session. Do we need to fix all warning? There are warning for original Dockerfile. How do we pass hasolint for original Dockerfile? " Dockerfile:76 DL3006 warning: Always tag the version of an image explicitly Dockerfile:78 DL3008 warning: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version> Dockerfile:122 SC2027 warning: The surrounding quotes actually unquote this. Remove or escape them. Dockerfile:134 SC2027 warning: The surrounding quotes actually unquote this. Remove or escape them. Dockerfile:153 DL3006 warning: Always tag the version of an image explicitly "

fixed the idp build issue accordingly

@louie-tsai let's move this discussion to https://github.com/intel/ai-containers/pull/129

Hi @louie-tsai, this PR has been open a while and might be stale. Do you want us to close the PR or keep it open for you?


python27 app manifest removed?

Hi
Just did a scoop status, and received the following message:-
scoop is up-to-date.
these app manifests have been removed:
    python27

I assume this means it was removed from the buckets in the repo. Is there a reason for that? I have a particular script that only works when I use python27 rather than python 3.5.1.
Regards
Mike

Hi Mike
It's been moved to the versions bucket. You just need to add it using scoop bucket add versions

Thanks so much, appreciate it!


Mobile Purchase

Is there any chance to get mobile purchase solution?
I mean implementing https://developer.apple.com/in-app-purchase/ and http://developer.android.com/google/play/billing/billing_overview.html payment methods that store, verify receipts for mobile devices.

This has to be supported by each pay method.  Which it is these days especially if you use the javascript token method of displaying the payment form.  Can I ask if apple pay, google pay is working for auth.net, paypal, or braintree with this module?


Fix sha256 binary for MacOS

MacOS doesn't have a sha256sum, only shasum

 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.

LGTM but these tasks are anyway not executed on a mac

I was just following the instructions to install drone from the source code:
https://github.com/drone/drone#from-source
Everything works fine on a MacOS except the bit at the end when it needs to generate the sha256 checksums.

The guide should call build_static because build generates the binaries for all architectures

I updated the README to instruct build_static instead of build. The cross compiled binaries and shasum should only be used by the build server, and don't need to run local. Sorry for the confusion
https://github.com/drone/drone/commit/bbd6b37e37507a0af65973006955501ad6224ec1#diff-04c6e90faac2675aa89e2176d2eec7d8


Support for tslint 6

grunt-tslint expects peer dependencies of v5.
v6 has been officially out for a few weeks now.
Can we get an update to support that?

@oranj would you be so kind as to submit a PR for this? change the dep to allow v5 OR v6.
note that the only change in v6 is that tslint is now officially deprecated in favor of @typescript-eslint.

There are a few other API changes in 6.0.0-beta0, as well as some bugfixes in 6.0.0, but none of them should require any functional code change in grunt-tslint. PRs welcome


::toFile task variants should ignore terminal width

I'm trying to get around a problem with our dependency tree being truncated as part of a CI task that doesn't have an interactive terminal, so uses the 80-character default and is truncating dependency versions (#173). This can be replicated pretty easily from the command line:
➜  stty cols 70 && sbt dependencyTree | grep jsr310
[info]   +-com.fasterxml.jackson.datatype:jackson-datatype-jsr310:2..

Unfortunately the terminal width also affects the output to file, which I don't think is intentional:
➜  stty cols 70 && sbt 'dependencyTree::toFile dependencies.txt'
...
[info] Wrote dependencyTree to 'dependencies.txt'
[success] Total time: 0 s, completed 05-Aug-2019 10:17:59
➜  grep jsr310 dependencies.txt
  +-com.fasterxml.jackson.datatype:jackson-datatype-jsr310:2..

Thanks, @matmannion. That cropping is indeed a bit useless for toFile. I'll see if I can do something for the next release.


Plz make lora and wildcard Autocomplete just like embedding

Plz make lora and wildcard Autocomplete just like embedding

Loras aren't part of the prompt, so how would you want that to work?
For wildcards you can just add your files to the list via the setting

Here are lora working in text encoder and translate to condition
https://github.com/skfoo/ComfyUI-Coziness

There are conditioning nodes that parse this out too, although I have no idea how they're supposed to work with no model output unless that's implicit which is kinda worse since it's hiding information about the underlying flow of things.  They really aren't part of the prompt (their keywords are, but the Coziness node doesn't seem to look those up and insert them nor does that work for everything) and wouldn't be in-place there.

For wildcards you can just add your files to the list via the setting

could you explain how exactly? i would like to have similar functionality as the a1111 extension in order to navigate the wildcard folder structure.
https://github.com/DominikDoom/a1111-sd-webui-tagcomplete

This has been done

thx, it works!


Document correctly that plot_ica_overlay() for Raw only plots the first 3 seconds of data

See https://mne.discourse.group/t/ica-plot-overlay-pan-forward-and-missing-data/2781

I'm fixing this.


Add a function for getting the name of a request

Signed-off-by: Uli Schlachter<EMAIL_ADDRESS>
This is how far I got before noticing that #314 does not really work. Dunno if this should be merged or not.

[ ] TODO: It just occurred to me that this should use the generated constants instead of having just the plain number

whoops, I mistakenly checked the TODO checkbox (and then unchecked it again), just in case you get some notification about me modifying this PR

Nope, no notifications. :-)

Can this be closed?

Dunno. I still feel like this could be useful, but so far have not come across a case where I would be able to use this information. If #314 is not possible... well, I guess then this should be closed. Do you agree that #314 is not possible and that the addition in this PR is not useful?

If #314 is not possible, I cannot think of any other use of this right now. Anyways, this is right now written for the old python generator, so if someday we want this, it would need to be done from scratch.


m64py: init at 0.2.4

Motivation for this change
This adds the package m64py:
https://github.com/mupen64plus/mupen64plus-ui-python
This is a Qt-based GUI for the mupen64plus N64 emulator.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @

If I remember correctly, m64py (and the underlying mupen) seemed pretty buggy, so I eventually gave up on using them.


Add Lebanese University Domain

@mhammadjaber00 I am sorry, the domain st.ul.edu.lb is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


ensure a secure version of Node.js is used

this PR makes sure that a secure version of the LTS version of Node.js gets used (see https://nodejs.org/en/blog/vulnerability/july-2017-security-releases)

engines is not for what version of Node is recommended, but what version of Node the library is compatible with.
Historically PDF.js was written to run in any modern browser (we specifically have compatibility.js to add missing functionality in some cases). To the best of my knowledge, we don't provide explicit support for a particular Node version (a very limited subset of the functionality is tested via Travis on the latest stable Node.js release). However, the examples are written using ES5 so that it works reasonably well if the user wants to experiment with PDF.js even if they don't have the latest and greatest version of Node.js

@Rob--W many CI and deployment tools will look at engines to determine what they need to install. Heroku, for instance, absolutely consults your package.json for an engines field to determine what it needs to install before it tries to run your code.

It seems that you are conflating "minimum supported version" and "desired/secure version".
IMO, engines in package.json in a library specifies the minimum runtime that is known to be compatible, with the implication that a lower version is likely unsupported & incompatible. This is not the case here, so I don't want to add the engines field with an unnecessarily high value.
And since you mention Heroku specifically, here is how they use "engines':

If you don’t specify a version, the latest Long-Term-Support release will be used.
-- https://devcenter.heroku.com/articles/nodejs-support#specifying-a-node-js-version

And although not documented in that article, what I found in practice is that if the engines.node field specifies a range/minimum, e.g. >=4.0.0, then heroku will install a recent version.


Fixed empty functions' names in perfomance panel

added html escaping in functions' names

Coverage increased (+0.08%) to 85.703% when pulling 03fb30a41fc50d440ff2ce0632325f6c72313877 on kammala:master into 511efff816c3a4bd2af52039674c326619229af9 on aio-libs:master.

Thanks, I will take a look soon.
On Wed, Jul 6, 2016, 10:36 Coveralls<EMAIL_ADDRESS>wrote:

[image: Coverage Status] https://coveralls.io/builds/6884693
Coverage increased (+0.08%) to 85.703% when pulling 03fb30a
https://github.com/aio-libs/aiohttp_debugtoolbar/commit/03fb30a41fc50d440ff2ce0632325f6c72313877
on kammala:master into 511efff
https://github.com/aio-libs/aiohttp_debugtoolbar/commit/511efff816c3a4bd2af52039674c326619229af9
on aio-libs:master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aio-libs/aiohttp_debugtoolbar/pull/43#issuecomment-230699870,
or mute the thread
https://github.com/notifications/unsubscribe/AANoZ0UY4Elxlpb4sr2EqPEHl0werzzsks5qS1sRgaJpZM4JF0uq
.

Thanks!


Admin panel error?

02:02:32.106 [info] Starting config update...
02:02:32.130 [info] Finishing config update...
02:02:32.153 request_id=jghitbt511sna12b0s8rni56bdl2nbug [info] Sent 500 in 48ms
02:02:32.172 [error] #PID<0.482.0> running Glasnost.Web.Endpoint terminated
Server: mydomain.tldt:80 (http)
Request: POST /admin/command_and_control
** (exit) an exception was raised:
    ** (Protocol.UndefinedError) protocol String.Chars not implemented for %HTTP                                                                                oison.Error{id: nil, reason: :nxdomain}
        (elixir) lib/string/chars.ex:3: String.Chars.impl_for!/1
        (elixir) lib/string/chars.ex:17: String.Chars.to_string/1
        (glasnost) lib/glasnost/web/controllers/admin_controller.ex:21: Glasnost                                                                                .Web.AdminController.command_and_control/2
        (glasnost) lib/glasnost/web/controllers/admin_controller.ex:1: Glasnost.                                                                                Web.AdminController.action/2
        (glasnost) lib/glasnost/web/controllers/admin_controller.ex:1: Glasnost.                                                                                Web.AdminController.phoenix_controller_pipeline/2
        (glasnost) lib/glasnost/web/endpoint.ex:1: Glasnost.Web.Endpoint.instrum                                                                                ent/4
        (phoenix) lib/phoenix/router.ex:277: Phoenix.Router.__call__/1
        (glasnost) lib/glasnost/web/endpoint.ex:1: Glasnost.Web.Endpoint.plug_bu                                                                                ilder_call/2

Internal server error when specifying a configuration URL. Is there a format that the config URL needs to follow?
On another note, if my server crashes how do I maintain the previous instance of docker? By starting up the docker container again won't the admin page reset (potentially enabling a malicious user to gain access to the admin panel?)

Is there a format that the config URL needs to follow?

Just the usual URL format including the http or https prefix. The url should point to a raw (as in github gist raw) valid JSON file.

By starting up the docker container again won't the admin page reset (potentially enabling a malicious user to gain access to the admin panel?)

Yep, Glasnost is in alpha and current admin workflow isn't ready for production yet. I will  implement proper authorization and authentication soon.

Ah, so I need to specify the full URL? I thought I just had to provide a keyword like 'secretadminpanel' instead of 'http://domain.tld/secretadminpanel' good to know 👍


Release request

I want kindly ask about merge current dependabot updates and release new version.
One fix is important for me:  #15

There is new version 1.1.0 - thanks


Swiping on a listview under perspective can cause erratic behaviour

Steps to reproduce
Have a ListView under a Transform with a perspective that imitates a "road".
If you swipe your finger outside of the visible bounds of the list, the list jumps to an unpredictable location, in a direction opposite to that of a swipe.
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final _scrollController = ScrollController();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: Transform(
        transform: Matrix4.identity()..setEntry(3, 1, -0.003),
        alignment: Alignment.bottomCenter,
        child: ListView.builder(
          controller: _scrollController,
          physics: const ClampingScrollPhysics(),
          reverse: true,
          itemBuilder: (context, index) => Container(
            color: index.isEven ? Colors.white : Colors.black12,
            alignment: Alignment.center,
            child: Text('Item $index'),
          ),
        ),
      ),
      floatingActionButton: FloatingActionButton(
        tooltip: 'Reset To Start of the List',
        onPressed: () => _scrollController.animateTo(
          0,
          duration: const Duration(seconds: 1),
          curve: Curves.easeInOut,
        ),
        child: const Icon(Icons.backspace_sharp),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Expected results
The list should behave "normally".
Actual results
The list jumps to an unpredictable location.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final _scrollController = ScrollController();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: Transform(
        transform: Matrix4.identity()..setEntry(3, 1, -0.003),
        alignment: Alignment.bottomCenter,
        child: ListView.builder(
          controller: _scrollController,
          physics: const ClampingScrollPhysics(),
          reverse: true,
          itemBuilder: (context, index) => Container(
            color: index.isEven ? Colors.white : Colors.black12,
            alignment: Alignment.center,
            child: Text('Item $index'),
          ),
        ),
      ),
      floatingActionButton: FloatingActionButton(
        tooltip: 'Reset To Start of the List',
        onPressed: () => _scrollController.animateTo(
          0,
          duration: const Duration(seconds: 1),
          curve: Curves.easeInOut,
        ),
        child: const Icon(Icons.backspace_sharp),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]
https://github.com/user-attachments/assets/ee2db010-33fb-4e71-bbca-5a5be825d067

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.22.3, on macOS 14.5 23F79 darwin-arm64, locale en-PL)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] IntelliJ IDEA Community Edition (version 2023.2.1)
[✓] VS Code (version 1.92.1)
[✓] Connected device (4 available)
[✓] Network resources

• No issues found!

Flutter version should not matter, we've had this for a long time.

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Reproducible using the code sample provided above.

flutter doctor -v
[!] Flutter (Channel stable, 3.24.1, on macOS 14.6.1 23G93 darwin-arm64, locale en-US)
    • Flutter version 3.24.1 on channel stable at /Users/deanli/dev/stable
    ! The flutter binary is not on your path. Consider adding /Users/deanli/dev/stable/bin to your path.
    ! The dart binary is not on your path. Consider adding /Users/deanli/dev/stable/bin to your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5874a72aa4 (6 days ago), 2024-08-20 16:46:00 -0500
    • Engine revision c9b9d5780d
    • Dart version 3.5.1
    • DevTools version 2.37.2
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[!] Flutter (Channel master, 3.25.0-1.0.pre.147, on macOS 14.6.1 23G93 darwin-arm64, locale en-US)
    • Flutter version 3.25.0-1.0.pre.147 on channel master at /Users/deanli/dev/master
    ! The flutter binary is not on your path. Consider adding /Users/deanli/dev/master/bin to your path.
    ! The dart binary is not on your path. Consider adding /Users/deanli/dev/master/bin to your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 99dc3beaf4 (6 hours ago), 2024-08-26 22:10:32 -0400
    • Engine revision b41ca79092
    • Dart version 3.6.0 (build 3.6.0-175.0.dev)
    • DevTools version 2.39.0-dev.15
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

This is likely because the transform also affects the coordinate plane the gesture is being applied to.


deps: Bump com_github_cncf_xds to cncf/xds@0c46c01

Changes: https://github.com/cncf/xds/compare/6b7cb9e...0c46c01

pgv_plugin_go : Use protoc_gen_validate instead of xds (cncf/xds#92)
cel: Add canonical CEL (dev.cel.expr) fields (cncf/xds#89)

dev_cel v0.15.0 dependency added
cel: Support canonical CEL in xds.type.v3.CelExpression

Commit Message:
deps: Bump `com_github_cncf_xds` to cncf/xds@0c46c01

Additional Description:
Changes: https://github.com/cncf/xds/compare/6b7cb9e...0c46c01

- pgv_plugin_go : Use protoc_gen_validate instead of xds(cncf/xds#92)
- cel: Add canonical CEL (dev.cel.expr) fields (cncf/xds#89)
  - [`dev_cel` v0.15.0]
    (https://github.com/google/cel-spec/releases/tag/v0.15.0)
    dependency added
  - cel: Support canonical CEL in xds.type.v3.CelExpression

Risk Level: Low
Testing: do_ci.sh
Docs Changes:
Release Notes:
Platform Specific Features:

@phlax  this is ready for review now.

/retest

Thank you for the quick review, @phlax!


Draft: Added transition handling between the side drawer and chats

@marcaaron
Details
Add a loading transition when chat is selected and the side drawer closes

Added a fullscreen loading component - just something for the time being - it needs to be revisited with proper design
Added withDrawerState HOC
Moved initial fetching calls to the ReportScreen
Unmount the comose input component when the drawer is extended - this fixes some issues with the keyboard staying open

Fixed Issues

Fixes #2154 fixes #2150
Tests

From the home screen
Have a few conversation
Select a conversation
The conversation should open with a brief loading

there should be a fullscreen loading indicator (centered)
on physical device sometimes loading happens so fast you might not see the loader. The transition should be smooth though

Go back and select another conversation

There's shouldn't be any glimpse of the previous conversation
The transition should feel as you are opening a new page

QA Steps
Same as the Tests above
Tested On

[x] Web
[ ] Mobile Web
[ ] Desktop
[x] iOS
[ ] Android

Screenshots

Web
https://user-images.githubusercontent.com/12156624/113462584-490c1d00-942a-11eb-9367-7e2ba83a7aad.mp4
Mobile Web

Desktop

iOS (Simulator)
https://user-images.githubusercontent.com/12156624/113461860-0ac12e80-9427-11eb-8aea-4965de1f16c1.mp4
iOS 11 Pro (Fix: Keyboard should close on back navigation)
https://user-images.githubusercontent.com/12156624/113478614-915f2580-9492-11eb-9f1c-eb4ece725029.mp4
Android

@marcaaron This is my rough sketch of how things can work
I want to share it early on to address any concerns
This is a draft - Please don't pay attention to stuff that needs to be cleaned up. I'm yet to move styles add documentation etc...
There are a few items I want to ask:

I've read you don't want to use promises - is the usage here OK - I can replace it with check in componentDidUpdate, but it won't be as elegant as now
I need to implement optimistic response - e.g. don't wait for fetchActions if the component has some data available to render immediately. I need to think about this but if you have an idea
The fullscreen loader might actual not be needed at all. I've tried without it and the transition is still very good. I replaced it with null 😄. I can add a sample video next week

So all in all I've still got some work to polish this but what do you say?

I've read you don't want to use promises - is the usage here OK - I can replace it with check in componentDidUpdate, but it won't be as elegant as now

What specific usage are you asking about here?

I need to implement optimistic response - e.g. don't wait for fetchActions if the component has some data available to render immediately. I need to think about this but if you have an idea

Not sure I understand this. I think you mean we want to display the component immediately if there is stored data before the network data finishes and updates it again with new data?

The fullscreen loader might actual not be needed at all. I've tried without it and the transition is still very good. I replaced it with null 😄. I can add a sample video next week

We can always remove this later if it’s not needed so I would say to keep for the time being.

I don't know why exactly this happens. But pressing on the left side of the chat returns focus on the compose input. I've removed all autofocusing logic whatsoever and it still does
The problem is, we unmount the compose input when the drawer is expanded. So when you click on the Header to go back (which is too on the left of course) you will get a critical error that if dismissed your stuck with unresponsive app. On production this directly force closes the app. Video bellow
This is Android only
https://user-images.githubusercontent.com/12156624/113907595-3a34ba00-97de-11eb-87c0-45c24ff8f093.mp4
Video shows:

tapping on the left side inside the screen hides and shortly after brings back the keyboard
tapping on the right side - closes the keyboard as expected - tapped outside of input
swiping the drawer open dismissed the keyboard as expected
pressing the back button in the header crashes the app because focus tries to return to the input field but the field is immediately unmounted by isDrawerOutOfTheWay resolving to false. This is unrelated to any of the existing autofocus logic because I disabled it everywhere and the issue would still happen..

I've even tried removing almost every other content with no luck. Things I've tried to figure out what causes this:

debugging - couldn't trace what is calling/setting focus on the field in js, so it's probably something native
removed all content from the sidebar/drawer - didn't help
removed everything inside ReportView but just added a plain {isDrawerOutOfTheWay && <TextInput placeholder="Test..." />} - no change still tries to focus the input
created a separate to recreate and post a ticket to react-native or react-native navigation but couldn't recreate the issue in an isolated snack...

I can leave the field mountend and set it as disabled (editable=false), but even then the keyboard stays open (though only on Android) when you go back to the LHN

Updated

I'm probably missing something obvious here, but why does this component mount and unmount now whereas before it did not?

https://github.com/Expensify/Expensify.cash/blob/master/src/pages/home/report/ReportView.js#L30-L34
I've moved this key prop to the wrapper view. Explicitly setting a key would re-mount a view (and it's children) when that key changes. It suggested by React so that you can avoid running error prone checks in didUpdate:
https://reactjs.org/blog/2018/06/07/you-probably-dont-need-derived-state.html#recommendation-fully-uncontrolled-component-with-a-key

Keys are usually used for dynamic lists but are also useful here. In our case, we could use the user ID to recreate the email input any time a new user is selected

Assuming no regressions in next week @kidroca this will be paid on 15 April 👍🏽

Opening the Url expensify.cash directly. shows this screen infinitely.

@kidroca I think this might is because we've switched from using the currentlyViewedReportID to the route.params.reportID. I missed that change in the PR. So, if we want to do it this way then we'll need to implement the solution discussed here:
https://expensify.slack.com/archives/C01GTK53T8Q/p1617908395456000
We can either:

Implement this fix now and not create a new issue for it
Revert this change by going back to using the currentlyViewedReportID temporarily and then create a new issue to switch things to route params.

The proper fix is to address the navigation it's related to other issues like the URL changing on mWeb, but I can't do it in the scope of the current ticket. If you want me to work on that I'll need a new ticket.
I can revert to using currentlyViewedReportID like it did before, though this will degrade the experience on Android. Will post a video

Here's how it would look like on Android when I revert to currentlyViewedReportID - the prop only updates after a delay.
https://user-images.githubusercontent.com/12156624/114208662-6418e880-9966-11eb-97bf-95745097de14.mp4
This is an issue with slow AsyncStorage access speed. I don't know how exactly ONYX works, but it might be possible to add a small proxy layer that persists data but keeps in in memory as well, so when it is accessed it can come instantly and don't have to be read from the file system first.
I think the iOS implementation of AsyncStorage probably does something like this as it works instantly there, so this might be a feature that can be implemented for AsyncStorage - configurable in memory proxy for fast access

I leave that up to you @kidroca. It's a regression from what we had before so we must do something for now :) I agree that switching the reportID source was probably not in scope so the fix for it doesn't need to be either.
p.s. I think maybe the video shared is to show some other behavior than what you are describing.

p.s. I think maybe the video shared is to show some other behavior than what you are describing.

Yep, sorry
Btw I thought of another patch fix that would better address this at the moment, I'll open a new PR. Should I copy the current PR description there?

Should I copy the current PR description there?

Ah, no need, linking back to this PR should be enough.

We found this issue that looks related to this PR https://github.com/Expensify/Expensify.cash/issues/2327. Not completely sure since the issue is reproducible in production.


Unify bias argument and initial_bias argument. Make bias deprecated.

Fix #525
As @tscohen and @beam2d are now discussing the design of Linear and Convolution2D, I should defer this PR until the discussion.

Memo

Add unittest that checks default behavior is unchanged
Consider if we should apply same change to Bilinear.

Parameter initialization is redesigned in #820. It is possible and better to work on what I want to do here in the PR. So I close this PR.


Added audio for multiple game states

I'm not sure if this is a wanted to feature but thought it might be fun to add little audio snippets for when different things happen. For example "That's Numberwang!" now plays a little audio snip from the show.

Oops meant to post this to the original repo my bad.


Bug with utf8-fdf-generator

Hi All,
Thanks so much for the lib.  It saved me so much time.
A customer discovered that if they entered a ( or ) in a text field that would be saved to a pdf, then it would crash.  We traced the bug back to this library and finally back to the utf8-fdf-generator lib.
The problem is that in the FDF spec (, ) and \ need to be escaped with a .  That's a simple enough change, but the utf8-fdf-generator was converting the values to UTF-16  and then back to UTF-8 on file save.  This ended up creating corrupted FDF files.
Long story short, it was easier for me to port in the generator lib and fix it here since this project has unit tests and they are both MIT license.  I put in a pull request for the generator lib, but it hasn't been updated in 4 years and the author isn't on github much, so I don't know how long it will be before its pulled.
I also updated the libs and modified the unit tests to show the bug and the fixes.
If you don't want to merge this, I totally understand since it's really an upstream problem.  I thought it would be helpful to anyone else that runs across this issue.
Thanks again,
James

@jbecwar maybe easier to just include your forked library rather than changing the code of this one?

I'd also like to see this merged. Including the fork as suggested sounds fine, but that means @jbecwar would first need to publish his fork on npm before this PR can be fixed up.

@jbecwar have you tried opening a pr for the utf library?

@whitef0x0
He opened it here a couple of months ago: https://github.com/Rhaseven7h/utf8-fdf-generator/pull/2
Although, as he mentioned, the author of that library doesn't seem to be very active.

@jasonphilips @johntayl maybe this would be the time to transfer pdffiller and this patched utf8 library to a pdffiller organization

@whitef0x0 @jasonphillips Copying this repo over to https://github.com/pdffillerjs
Will have to deprecate this one.

@johntayl if you transfer the repo you'll be able to keep all the stars and it'll have a redirect to it

@whitef0x0 I'll transfer it, I'm assuming I'd have to update the current npm package to use the new URL?

Any updates over this issue?


Configuration Parameter 7 is missing from Philio PSM02

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within ZWaveJS2MQTT?
NO, my problem is NOT within ZWaveJS2MQTT
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
Philio PSM02 (and similar Philio devices) use notification report rather sensor binary report for motion by default and as a result motion may not properly work,
The non-documented parameter 7 allows the sensor binary report to be enabled.
bit 1 (decimal 2) enable motion off report
bit 2 (decimal 4) enable super PIR sensitivity
bit 4 (decimal 16) 0 = notification report, 1 = sensor binary report
bit 5 (decimal 32) disable multi CC in auto report
bit 6 (decimal 64) disable report battery state when device triggered
Device information
Manufacturer: Philio
Model name: PSM02
Node ID in your network: 4
How are you using node-zwave-js?

[X] zwavejs2mqtt Docker image (latest)
[ ] zwavejs2mqtt Docker image (dev)
[ ] zwavejs2mqtt Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[X] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
version: Driver Version: 8.2.1, Server Version: 1.10.1
node-zwave-js branch:
zwavejs2mqtt branch: 0.24.1
Did you change anything?
no
If yes, what did you change?
No response
Did this work before?
No, it never worked anywhere
If yes, where did it work?
No response
Attach Driver Logfile
zwavejs_2021-09-18 (4).log

Many Philio config files are terrible, but we should at least add the param. Accepting PRs.

I am a newbie here. I assume PR stands for Pull Request. I will read through the documentation and try to submit one.
You mentioned that the Philio config files are terrible. In what way could they be improved? Looking at the one for the PSM02, it seems to contain the information which is publicly available: https://devices.zwave-js.io/?jumpTo=0x013c:0x0002:0x0002:0.0

I assume PR stands for Pull Request.

Bingo

In what way could they be improved?

We have a section in the docs detailing what we consider a good config file:
https://zwave-js.github.io/node-zwave-js/#/config-files/contributing-files
The Philio files often have labels/descriptions that don't yet match our style guide. And the parameters 6/7 that some of their devices have are only half documented there (min 0, max 127, user has to figure out what they do).
Here's an example of bad one: https://devices.zwave-js.io/?jumpTo=0x013c:0x0002:0x000d:0.0
And here's a better version of a similar device, using partial parameters to describe the user function settings: https://devices.zwave-js.io/?jumpTo=0x013c:0x0002:0x000e:0.0

Just to confirm, when creating a pull request, I pull the whole node-zwave-js and make changes to the Philio PSM02 config file?

Yep

Now that changes have been merged for PSM02, shall I make updates to PST02A and PST02B as well as they are very similar changes?

Sure, go for it :)

fixed by #3373


What about docs?

[ ] Create a doc folder
[ ] Create a new index.md document
[ ] Turn on Github pages
[ ] Decide if we want to go with sometime more complex


[Feature Request]: Vespa VectorDB Connection

Feature Description
My team is using Vespa as our VectorDB solution and we wanted to start using LLamaIndex in our project. We would just need to create a connector that can work with existing Vespa deployments so it would be a seamless connection with the library. I could work on this if possible.
Reason
No such connector exists at the moment.
Value of Feature
This would add to the already robust arsenal of Vector Databases supported by LLamaIndex and would allow our team to start using this library as well.

This notebook might be a good start for retriever abstraction.


Removal of deprecated code.

Codecov Report

Merging #706 (1d9d7d0) into main (9aaa8a5) will increase coverage by 0.03%.
The diff coverage is n/a.

: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     #706      +/-   ##
============================================
+ Coverage     92.78%   92.81%   +0.03%
+ Complexity     2578     2577       -1
============================================
  Files           281      281
  Lines          5113     5108       -5
  Branches        529      529
============================================
- Hits           4744     4741       -3
  Misses          237      237
+ Partials        132      130       -2

Impacted Files
Coverage Δ

...n/java/net/datafaker/providers/base/BaseFaker.java
90.99% <ø> (+3.16%)
:arrow_up:

...ain/java/net/datafaker/providers/base/Disease.java
100.00% <ø> (ø)

.../datafaker/transformations/sql/SqlTransformer.java
85.50% <0.00%> (-0.73%)
:arrow_down:

.../java/net/datafaker/service/FakeValuesService.java
84.08% <0.00%> (-0.20%)
:arrow_down:

...ain/java/net/datafaker/idnumbers/KoKrIdNumber.java
87.50% <0.00%> (+6.25%)
:arrow_up:

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


WSL2 - Error: connect ECONNREFUSED

I'm running Windows 11, my development environment is inside Debian running in WSL2.
If I have the extension running on a non-WSL instance of VSCode I'm able to connect to the DB.
If I have the extension running on my WSL instance of VSCode I'm not able to connect to the DB.
In both cases, the DB information is the same/correct.

Also getting this - any updates?

Same issue


Release latest fixes with explicit return types

Since the last v1.4.1 release there are some useful fixes https://github.com/Nyholm/psr7/compare/1.4.1...master
Can you publish a new release, please?

I'd also appriciate that


Create user-facing docsite using Sphinx

Use Sphinx and Sphinx auto-api extension to autogenerate docs that adds:

[ ] Changelog
[ ] Reference Guide (Concepts)
[ ] Setup & Installation instructions
[ ] Tutorial / Quick Start

Note: You can reuse TUTORIAL.md and REFERENCE.md by converting them into rst and including the rest using the include directive (eg.".. include:: ../README.rst" )
References:

Astronomer Providers
Docker project Docs and it's source-code.

Subtasks

[ ] Add CI step to Build docs so each PR is tested with docs.
[ ] Fix any failures when building the docs -- mostly around docstrings or missing descriptions
[ ] Instructions or automation around building the docs locally (nox or Makefile etc)

Follow-up task: https://github.com/astronomer/astro-sdk/issues/276

This task is not complete, please check the Issue description


add wanning msg for a bug on k8s on ginkgo removal brunch

I don't think its entirely necessary because we will bring in the change from #1720 when we do the git merge eventually.

merged on main


Uncaught exception when saving a model

Try this on a Mac and you'll get the following message:

Create a new model
Hit the save file button
Click on Cancel

This also happens when cancelling other dialogues.

Did you try out the latest nightly? Should be fixed there (via https://github.com/camunda/camunda-modeler/issues/337).


Temporary fix for dummy files in phony Ninja rules

Ninja rules can produce dummy files through an special case of a phony rule (see https://ninja-build.org/manual.html#_the_literal_phony_literal_rule ). This is used in the case of CMake-generated Ninja files, specifically for the install functionality.
This PR is a temporary fix that just declares every output as Optional instead of Required, so that BuildXL won't kill the build when the rule is executed and the dummy output is not there.
A more permanent fix is more complicated, and requires changing the Ninjson utility that we use to generate the BuildXL graphs. The internal bug-track for this is here.

/azp run Mac BuildXL PR Validation


Undercut NQ prices for NQ item & HQ prices for HQ item

Detects whether an item is NQ or HQ when opening the "Adjust Price" window, and undercuts NQ prices when selling a NQ item/HQ prices when selling a HQ item.

When "Undercut HQ prices" is checked, this has no effect and will retain the old behavior (it'll always undercut HQ prices), but it may be worth considering removing that config option
Shift still works as a toggle for NQ prices on HQ items / HQ prices on NQ items

LGTM


Fix doc links, add to "never exiting" section.

:+1:

+1, but it's always nice if a link to something goes directly to where the information being referenced appears, instead of to the top of a page that you have to read or search through.  This linked page is small enough that it doesn't really matter.

@rasahner I can do that, one sec

@rasahner updated the first link to include an anchor; I think the second one makes sense linking to the top of the page.

+1


qt5: 5.15.12 -> 5.15.14, 5.15.16 -> 5.15.17

Description of changes
Also drop a bunch of QtWebEngine Python3 stuff since it's now upstream.
Can go to staging-not-next, but maybe we can squeeze this in.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Uh, merged the backport already?  On staging-next we don't even have qtbase for x86_64-darwin right now:  https://hydra.nixos.org/build/262351201

The other darwin fails the same, too: https://hydra.nixos.org/build/262342932

I think the first real error is this one?
../../../include/QtCore/../../src/corelib/global/qcompilerdetection.h:1069:17: error: expected parameter declarator
    && (defined(__MAC_OS_X_VERSION_MIN_REQUIRED)  && __MAC_OS_X_VERSION_MIN_REQUIRED  < 140000
                ^
/nix/store/4p9q5hhj339yijzxf8jll6li35ngpj2s-libSystem-11.0.0/include/AvailabilityInternal.h:40:49: note: expanded from macro '__MAC_OS_X_VERSION_MIN_REQUIRED'
        #define __MAC_OS_X_VERSION_MIN_REQUIRED __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__
                                                ^

which sounds like they messed up their preprocessor spaghetti.


Updated zip file for non-submodule implementation

Question?
I would like to be able to use this theme, it is pretty incredible and the documentation is great!  Thank you!!
My deployment mechanism doesn't allow for submodules or hugo modules, so I am needing to create the themes/shadocs files and push during deployment.
I tried via downloading the zip but it seems to be missing quite a bit.  Is it possible this could be updated?  (example layout/partial/helpers)
I am trying via git clones of this project and the bulma project, but this is requiring identifying/removing all of the module/submodule pieces.
If there is another method that would be more simple please let me know.
Thank you.

Hi @sfjkastello ,
Glad you like the theme 😃
Indeed in your case, I guess that going for the zip would be the best option if you cannot use git to pull the repository and sub-modules.
Also as you mention the latest release was created a long ago and the zip was not up-to-date.
I've created a 1.2.0 of the theme and the new zip is published.
I'll try to release more often in the future. (Well... more than once a year 😄 ).
Regards,
Jordan.


feat: add catppuccin themes for godot 4

we recently updated our godot theme and added support for godot 4. i will raise a separate pull request against the 3.x branch to update those files too.

done! the spaces were added for aesthetic reasons in the theme picker, but i understand the concern.


Ignore env_prefix if validation_alias is set

This was the behavior in V1 and changed by mistake in V2.

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

Comparison is base (eb6ed32) 95.61% compared to head (dfbc683) 95.61%.

: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
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   95.61%   95.61%
=======================================
  Files           5        5
  Lines         251      251
  Branches       60       60
=======================================
  Hits          240      240
  Misses         10       10
  Partials        1        1

Impacted Files
Coverage Δ

pydantic_settings/sources.py
95.63% <100.00%> (ø)

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


mesh-expansion: reboot deletes required directory, sidecar startup fails

Bug Description
When a mesh-expansion vm with a sidecar reboots, the /var/run/secrets directory is deleted. As discussed in https://github.com/istio/istio/issues/35343 this doesn't seam to be a problem regarding the tokens/ subdirectory.
But with istio 1.14 the deletion leads to a failing SDS, because the sidecar wants to recreate /var/run/secrets to create /var/run/secrets/workload-spiffe-uds/socket. The directory creation fails with permission denied. Because of this the sidecar isn't able to come up.
Our puppet setup would recreate the directory, but since we're using istio's dns proxying, the puppet agent can not run, because dns isn't working without a running sidecar.
I can think of a workaround to recreate the directory after a reboot, but I'd argue that shouldn't be required for the package to be used. I understand that the content of the directory should be purged, but the directory itself shouldn't be deleted if it's required.
Also I would suggest to add this behavior to the docs page, took some time to figure out what the problem was.
Version
$ istioctl version
client version: 1.14.5
control plane version: 1.14.5
data plane version: 1.14.5 (89 proxies)

$ uname -a
Linux <hostname> 5.10.0-19-amd64 #1 SMP Debian 5.10.149-2 (2022-10-21) x86_64 GNU/Linux

Additional Information

Sidecar Logs
2022-12-29T08:08:51.616541Z	info	JWT policy is third-party-jwt
2022-12-29T08:08:51.616545Z	info	using credential fetcher of JWT type in cluster.local trust domain
2022-12-29T08:08:51.632722Z	info	dns	Starting local udp DNS server on <IP_ADDRESS>:15053
2022-12-29T08:08:51.633015Z	info	dns	Starting local tcp DNS server on <IP_ADDRESS>:15053
2022-12-29T08:08:51.633040Z	info	SDS socket not found. Starting Istio SDS Server
2022-12-29T08:08:51.633048Z	info	CA Endpoint istiod.istio-system.svc:15012, provider Citadel
2022-12-29T08:08:51.633561Z	info	Using CA istiod.istio-system.svc:15012 cert with certs: ./etc/certs/root-cert.pem
2022-12-29T08:08:51.634496Z	info	citadelclient	Citadel client using custom root cert: ./etc/certs/root-cert.pem
2022-12-29T08:08:51.652824Z	info	Opening status port 15020
2022-12-29T08:08:51.956885Z	info	ads	All caches have been synced up in 368.392041ms, marking server ready
2022-12-29T08:08:51.957100Z	warn	Failed to create directory for ./var/run/secrets/workload-spiffe-uds/socket: mkdir var/run/secrets: permission denied
2022-12-29T08:08:51.957140Z	error	sds	Failed to set up UDS path: failed to listen on unix socket "./var/run/secrets/workload-spiffe-uds/socket": listen unix ./var/run/secrets/workload-spiffe-uds/socket: bind: no such file or directory
2022-12-29T08:08:51.957145Z	info	sds	SDS server for workload certificates started, listening on "./var/run/secrets/workload-spiffe-uds/socket"
2022-12-29T08:08:51.957175Z	info	xdsproxy	Initializing with upstream address "istiod.istio-system.svc:15012" and cluster "kube-cluster"
2022-12-29T08:08:51.958777Z	info	starting Http service at <IP_ADDRESS>:15004
2022-12-29T08:08:51.979122Z	info	sds	Starting SDS grpc server
2022-12-29T08:08:51.979183Z	warn	Failed to create directory for ./var/run/secrets/workload-spiffe-uds/socket: mkdir var/run/secrets: permission denied
2022-12-29T08:08:51.979221Z	error	sds	SDS grpc server for workload proxies failed to set up UDS: failed to listen on unix socket "./var/run/secrets/workload-spiffe-uds/socket": listen unix ./var/run/secrets/workload-spiffe-uds/socket: bind: no such file or directory
2022-12-29T08:08:51.979409Z	info	Pilot SAN: [istiod.istio-system.svc]
2022-12-29T08:08:51.981411Z	info	Starting proxy agent
2022-12-29T08:08:51.981436Z	info	Epoch 0 starting
2022-12-29T08:08:51.981457Z	info	Envoy command: [-c etc/istio/proxy/envoy-rev0.json --restart-epoch 0 --drain-time-s 45 --drain-strategy immediate --parent-shutdown-time-s 60 --local-address-ip-version v4 --file-flush-interval-msec 1000 --disable-hot-restart --log-format %Y-%m-%dT%T.%fZ	%l	envoy %n	%v -l warning --component-log-level misc:error --concurrency 2]
2022-12-29T08:08:52.867830Z	warn	failed to fetch token from file: open ./var/run/secrets/tokens/istio-token: no such file or directory
2022-12-29T08:08:52.867885Z	info	xdsproxy	connected to upstream XDS server: istiod.istio-system.svc:15012
2022-12-29T08:08:52.988109Z	warn	Failed to create directory for ./var/run/secrets/workload-spiffe-uds/socket: mkdir var/run/secrets: permission denied
2022-12-29T08:08:52.988236Z	error	sds	SDS grpc server for workload proxies failed to set up UDS: failed to listen on unix socket "./var/run/secrets/workload-spiffe-uds/socket": listen unix ./var/run/secrets/workload-spiffe-uds/socket: bind: no such file or directory
2022-12-29T08:08:53.359444Z	warn	failed to fetch token from file: open ./var/run/secrets/tokens/istio-token: no such file or directory
2022-12-29T08:08:53.398696Z	info	cache	generated new workload certificate	latency=1.439793431s ttl=23h59m57.601313949s
2022-12-29T08:08:53.398741Z	info	cache	Root cert has changed, start rotating root cert
2022-12-29T08:08:53.398776Z	info	ads	XDS: Incremental Pushing:0 ConnectedEndpoints:0 Version:
2022-12-29T08:08:53.403606Z	info	cache	returned workload trust anchor from cache	ttl=23h59m57.596400108s
2022-12-29T08:08:54.988727Z	warn	Failed to create directory for ./var/run/secrets/workload-spiffe-uds/socket: mkdir var/run/secrets: permission denied
2022-12-29T08:08:54.988931Z	error	sds	SDS grpc server for workload proxies failed to set up UDS: failed to listen on unix socket "./var/run/secrets/workload-spiffe-uds/socket": listen unix ./var/run/secrets/workload-spiffe-uds/socket: bind: no such file or directory
2022-12-29T08:08:57.521451Z	warn	Failed to create directory for ./var/run/secrets/workload-spiffe-uds/socket: mkdir var/run/secrets: permission denied
2022-12-29T08:08:57.521525Z	error	sds	SDS grpc server for workload proxies failed to set up UDS: failed to listen on unix socket "./var/run/secrets/workload-spiffe-uds/socket": listen unix ./var/run/secrets/workload-spiffe-uds/socket: bind: no such file or directory
2022-12-29T08:09:05.522196Z	warn	Failed to create directory for ./var/run/secrets/workload-spiffe-uds/socket: mkdir var/run/secrets: permission denied
2022-12-29T08:09:05.522323Z	error	sds	SDS grpc server for workload proxies failed to set up UDS: failed to listen on unix socket "./var/run/secrets/workload-spiffe-uds/socket": listen unix ./var/run/secrets/workload-spiffe-uds/socket: bind: no such file or directory
2022-12-29T08:09:21.523459Z	warn	sds	SDS grpc server could not be started
2022-12-29T08:36:56.952157Z	warn	failed to fetch token from file: open ./var/run/secrets/tokens/istio-token: no such file or directory
2022-12-29T08:36:56.952399Z	info	xdsproxy	connected to upstream XDS server: istiod.istio-system.svc:15012
2022-12-29T09:05:45.822787Z	warn	failed to fetch token from file: open ./var/run/secrets/tokens/istio-token: no such file or directory
2022-12-29T09:05:45.822906Z	info	xdsproxy	connected to upstream XDS server: istiod.istio-system.svc:15012

Why do we not have permission to create folder, but can create files? Is it that typically the folder is created first then granted permission to istio-proxy?
The folder deletion is done by the OS btw, not Istio

Why do we not have permission to create folder, but can create files? Is it that typically the folder is created first then granted permission to istio-proxy?

Yes, from the installation guide

Install the token at /var/run/secrets/tokens:

$ sudo  mkdir -p /var/run/secrets/tokens
[...]

Transfer ownership of the files in /etc/certs/ and /var/lib/istio/envoy/ to the Istio proxy:

[...]
$ sudo chown -R istio-proxy [...] /var/run/secrets [...]

The folder deletion is done by the OS btw, not Istio

Thanks for pointing that out. Any advice/best practice on how to recreate the folder after reboot? I would suggest to add it to the docs page linked above.

A feasible solution would be to add creation and setting permission to systemd unit file :
ExecStart=/usr/bin/install -d -o istio-proxy -m 0755 /var/run/secrets

Solved with https://github.com/istio/istio/pull/43163


Consistent descriptions

Making all the descriptions consistent by transforming them into full sentences. Mainly for the welcome page of the frontend.
Also adding descriptions for gemss and mqdss.

I fixed a little thing which dismissed my (approving) review, can someone else check again?

Nice!
However I cannot say anything about the descriptions you added, can some of the others?

I'm not sure either...


Issue with gradle after adding android intent plus dependency

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Trying to build app -- not working after using android_intent_plus
Expected results
App can be build before
Actual results
Couldn't build the app after adding android_intent_plus dependency
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
![image](https://user-images.githubusercontent.com/90414446/234863558-44d595c2-45b7-44c5-bef8-8282540f07de.png)

Closing as duplicate of https://github.com/flutter/flutter/issues/125621.


bounding box detection is based on grid origin which could be out of bounds

Digging bug report from a user, which actually reveals an issue with the bounding box code:

build a grid
extend it to one side with many blocks
then build something at the end
and remove all the blocks from the origin, keeping only the end

The bounding box is somehow still centered on that grid origin, when it should in fact be more or less at the center of the grid.
This might be a known keen bug for keeping the grid origin outside of the actual grid, but maybe we can work around it by calculating the real center of blocks.
https://github.com/Casimir255/QuantumHangar/blob/master/Utilities/ParallelSpawner.cs#L244

@StalkR Two years later, I found the issue and the grid box should be working as expected now. This was fixed back in Jan

Well done! cheers :)


feat: Private Key Auth

DHE server context right-click context menu actions

Generate DHE keypair - will generate keypair, upload to server, and store in secret storage locally
Connect to server as another user (only visible if server is disconnected

Login via keypair - If a keypair has been generated, clicking on a DHE server node will allow picking username / pwd or key login

Discard connection action - shows up as trashcan on connected server node. Will discard DHE connection + any worker connections

Secret storage

Store generated keypair
Remember last login user / operateas for the server
Deephaven: Clear Secrets - command palette option to remove private keys + remembering user / operateas user info

Note: there are some debugging tools for visualizing the secret storage. In command palette:
> Developer: Toggle Developer Tools
> Developer: Log Storage Database Contents

The code should be ready for review. I do plan to update the DH npm package versions to non-alpha ones once https://github.com/deephaven-ent/iris/pull/2386 has merged and npm packages have been published, but there shouldn't be any api changes from that.


RGBW device not responding to scenes unless manually turned on first

Recently started experiencing an issue where an RGBWW strip I have will not respond to any scenes, unless I manually turn on the strip in Homekit before running the scene (then it works fine). I have a separate RGB (no W) strip which is not experiencing the issue, so I suspect it may be related to the existence of the white LED and how that’s handled.
Solutions attempted:

Pruning all MagicHome accessories from plugin
Recreating scenes from scratch
Power cycling LED strip and controller

I’m sure there’s more information needed, but I’m a novice at submitting big requests, so please let me know what other details I can provide.
Thanks!

Hey, thanks for letting us know. We have been experiencing issues with scenes since the start of the project. The issue lies in the fact that some of these controllers, on a firmware level, are not programmed to respond to commands if they are "powered off" as in set to 0% brightness. We try to compensate by adding an 'on' command followed by a delay but the problem still persists intermittently which is frustrating for everyone and hard to debug.
Could you please go into the plugin's settings and set the logging level to trace> restart HomeBridge > try to reproduce the issue > and then upload the log file (please don't paste it) here? I can't promise it will be solved soon as again, this is an ongoing and known issue, but any information we get helps us greatly. Especially logs.
Thank you,
-Z

Appreciate the reply. For what it’s worth, scenes have been working for this strip without much issue, it’s just in the last week or two that they’ve been having issues.
And my apologies, but I’m not seeing where the option to set the logging level to trace is located. Is that something I need to set in the config file?


Super-RMB stops working after some usage

Describe the Bug
Usually, Super-RMB opens Fly-Pie. After some time however a Super-RMB doesn't do anything and all subsequent Super-RMBs open the default GNOME window menu. Here is a screenrecording (the simulated shortcut maximizes the window the first time)
Screencast from 2022-11-18 21-09-46.webm
STR
Unfortunately, I can't reproduce it every time I use super-rmb. It just happens when using the Super-RMB activation at some point. It appears to be related to the Display Timeout setting. I can't reproduce it with a delay of 0. But with a delay of about 200 ms I can usually trigger it by spamming super-rmb in a few seconds. I can also reproduce it with all other extensions disabled. There is nothing in journalctl.
Expected Behavior
Super-RMB should still be working :)
System
Please complete the following information:

Fedora 37
Fly-Pie version 17 (EGO)
GNOME Shell version: 43.0

Thanks for the report! I can indeed reproduce this. I'll see if I find the reason :smile:

This was a really hard to find bug. In fact, it could not only be triggered by Super+RMB activation but also by normal hotkey activation. Just spamming the configured hotkey with a display timout > 0 could make Fly-Pie completely unresponsive until the session is restarted.
Anyway, I think it should work now!


[question] is the information on node state transmitted via mqtt?

Is there a way to know if a device is unavailable , in sleep state or dead via mqtt?

There is this setting in the settings page:

So I'd think yes, since you can disable it. Have you looked at the topics available?

There is this setting in the settings page:

So I'd think yes, since you can disable it. Have you looked at the topics available?

On my mqtt server, I get these topics:

On my mqtt server, I get these topics:

Thanks @LordMike

Thanks @LordMike


Update fingerprints.py

 https://useradmin.comma.ai/?onebox=3aa4a699dd10d0a8%7C00000002--2a290cb0a9

Your car should not be supported on master, master-ci, nightly branches and will be added to the next release when that becomes available!


Setting webpack_public_path does not reflect while importing images

Hello,
In my entry(main.js) file I have set __webpack_public_path__ at the first line as below
__webpack_public_path__ = 'http://some.cdn.com/some'
but while import images it just return the name
import SomeFriend from '../../images/some-friend';
(the above import is done in b.js which will be imported in main.js)
I even tried it setting next to the import
__webpack_public_path__ = 'http://some.cdn.com/some'
import SomeFriend from '../../images/some-friend';
when I do console.log(SomeFriend) it returns the file name as some-friend.jpg
Need experts advice. How to set __webpack_public_path__ so that all my image will be loaded from the give location.
Thanks & Regards
Adi

import is hoisted, so it always runs before the code inside of your module.
You probably have the use require in the entry point.

Great it works,
Thanks quick response.


[FEATURE] Plugin "Lombok" is incompatible (until build 201.SNAPSHOT < IU-202.6397.94).

Lombok plugin is not compatible with IDEA 2020.2

The InteliJ plugin can be found in a different repository, it seems like there already is a bug report for this issue: https://github.com/mplushnikov/lombok-intellij-plugin/issues/881
Disclaimer: I'm not a project owner.


CODEOWNERS: Make Hubble team (not docs-structure) own examples/hubble

GitHub team docs-structure is the fallback owner for examples/, but it would make more sense to have members of the Hubble team look at what's under examples/hubble/.

With approvals from sig-hubble, I think we can go ahead and don't need to wait for Aditi's review.


Visual Studio for Mac Compatibility Issue

I have encountered a problem while using NConcern and CNeptune in Visual Studio for Mac. When I checked the contents of the error message, the cause of the problem seems to be related to the file path combination. I tested your nuget package in macOS.
"    [ERROR] FATAL UNHANDLED EXCEPTION: System.IO.FileNotFoundException: Could not find file "/Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest\binAspNetMvcTest.dll"."
Instead of the path shown in the above log, the path below seems to be the correct one. It looks like your code needs to be modified to use the path delimiting string in the System.IO.Path class.
"/Users/rkttu/Projects/AspNetMvcTest/AspnetMvcTest/bin/AspNetMvcTest.dll"
I thought that there might be other problems, so I attached the entire log below.
Detailed MSBUILD log:
`Build started 4/16/2017 7:38:28 PM.

Project "/Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest/AspNetMvcTest.csproj" (Build target(s)):
Target GenerateTargetFrameworkMonikerAttribute:
Skipping target "GenerateTargetFrameworkMonikerAttribute" because all output files are up-to-date with respect to the input files.
Target CoreCompile:
/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5/csc.exe /noconfig /nowarn:1701,1702,2008 /nostdlib+ /errorreport:prompt /warn:4 /define:DEBUG /errorendlocation /preferreduilang:ko-KR /highentropyva+ /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/Microsoft.CSharp.dll /reference:/Users/rkttu/Projects/AspNetMvcTest/packages/Microsoft.Web.Infrastructure.<IP_ADDRESS>/lib/net40/Microsoft.Web.Infrastructure.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/mscorlib.dll /reference:/Users/rkttu/Projects/AspNetMvcTest/packages/MySql.Data.6.9.9/lib/net45/MySql.Data.dll /reference:/Users/rkttu/Projects/AspNetMvcTest/packages/NConcern.4.0.3/lib/NConcern.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.ComponentModel.DataAnnotations.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.Core.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.Web.Abstractions.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.Web.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.Web.DynamicData.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.Web.Extensions.dll /reference:/Users/rkttu/Projects/AspNetMvcTest/packages/Microsoft.AspNet.WebPages.3.2.3/lib/net45/System.Web.Helpers.dll /reference:/Users/rkttu/Projects/AspNetMvcTest/packages/Microsoft.AspNet.Mvc.5.2.3/lib/net45/System.Web.Mvc.dll /reference:/Users/rkttu/Projects/AspNetMvcTest/packages/Microsoft.AspNet.Razor.3.2.3/lib/net45/System.Web.Razor.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.Web.Routing.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.Web.Services.dll /reference:/Users/rkttu/Projects/AspNetMvcTest/packages/Microsoft.AspNet.WebPages.3.2.3/lib/net45/System.Web.WebPages.Deployment.dll /reference:/Users/rkttu/Projects/AspNetMvcTest/packages/Microsoft.AspNet.WebPages.3.2.3/lib/net45/System.Web.WebPages.dll /reference:/Users/rkttu/Projects/AspNetMvcTest/packages/Microsoft.AspNet.WebPages.3.2.3/lib/net45/System.Web.WebPages.Razor.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.Xml.dll /reference:/Library/Frameworks/Mono.framework/Versions/5.0.0/lib/mono/4.5-api/System.Xml.Linq.dll /debug+ /debug:portable /optimize- /out:obj/Debug/AspNetMvcTest.dll /subsystemversion:6.00 /target:library /utf8output Global.asax.cs App_Start/RouteConfig.cs Controllers/HomeController.cs Aspects/Logging.cs Attributes/ActivateTraceAttribute.cs "/var/folders/9m/fqmj_yjx28x7gb60pnyqyhvh0000gn/T/.NETFramework,Version=v4.5.AssemblyAttributes.cs"
Target _CopyFilesMarkedCopyLocal:
Copying file from "/Users/rkttu/Projects/AspNetMvcTest/packages/MySql.Data.6.9.9/lib/net45/MySql.Data.dll" to "bin/MySql.Data.dll".
Copying file from "/Users/rkttu/Projects/AspNetMvcTest/packages/NConcern.4.0.3/lib/NConcern.dll" to "bin/NConcern.dll".
Copying file from "/Users/rkttu/Projects/AspNetMvcTest/packages/NConcern.4.0.3/lib/NConcern.xml" to "bin/NConcern.xml".
Target CopyFilesToOutputDirectory:
Copying file from "obj/Debug/AspNetMvcTest.dll" to "bin/AspNetMvcTest.dll".
AspNetMvcTest -> /Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest/bin/AspNetMvcTest.dll
Copying file from "obj/Debug/AspNetMvcTest.pdb" to "bin/AspNetMvcTest.pdb".
Target CNeptune:
mono "/Users/rkttu/Projects/AspNetMvcTest/packages/CNeptune.1.1.0/build/CNeptune.exe" "/Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest/AspNetMvcTest.csproj" "Debug"
Unhandled Exception:
System.IO.FileNotFoundException: Could not find file "/Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest\binAspNetMvcTest.dll".
File name: '/Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest\binAspNetMvcTest.dll'
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share, System.Int32 bufferSize, System.Boolean anonymous, System.IO.FileOptions options) [0x0019c] in <164469fa6d844a078126fd8006807335>:0
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share) [0x00000] in <164469fa6d844a078126fd8006807335>:0
  at (wrapper remoting-invoke-with-check) System.IO.FileStream:.ctor (string,System.IO.FileMode,System.IO.FileAccess,System.IO.FileShare)
  at Mono.Cecil.ModuleDefinition.GetFileStream (System.String fileName, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share) [0x0001c] in <ebce9b93ad98499b85c2fc78d9656633>:0
  at Mono.Cecil.ModuleDefinition.ReadModule (System.String fileName, Mono.Cecil.ReaderParameters parameters) [0x00000] in <ebce9b93ad98499b85c2fc78d9656633>:0
  at Mono.Cecil.AssemblyDefinition.ReadAssembly (System.String fileName, Mono.Cecil.ReaderParameters parameters) [0x00000] in <ebce9b93ad98499b85c2fc78d9656633>:0
  at CNeptune.Program.Manage (System.String assembly) [0x0002d] in <1072b40fcc9947f48e666e8b38648f6c>:0
  at CNeptune.Program.Main (System.String[] arguments) [0x0012c] in <1072b40fcc9947f48e666e8b38648f6c>:0
[ERROR] FATAL UNHANDLED EXCEPTION: System.IO.FileNotFoundException: Could not find file "/Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest\binAspNetMvcTest.dll".
File name: '/Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest\binAspNetMvcTest.dll'
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share, System.Int32 bufferSize, System.Boolean anonymous, System.IO.FileOptions options) [0x0019c] in <164469fa6d844a078126fd8006807335>:0
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share) [0x00000] in <164469fa6d844a078126fd8006807335>:0
  at (wrapper remoting-invoke-with-check) System.IO.FileStream:.ctor (string,System.IO.FileMode,System.IO.FileAccess,System.IO.FileShare)
  at Mono.Cecil.ModuleDefinition.GetFileStream (System.String fileName, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share) [0x0001c] in <ebce9b93ad98499b85c2fc78d9656633>:0
  at Mono.Cecil.ModuleDefinition.ReadModule (System.String fileName, Mono.Cecil.ReaderParameters parameters) [0x00000] in <ebce9b93ad98499b85c2fc78d9656633>:0
  at Mono.Cecil.AssemblyDefinition.ReadAssembly (System.String fileName, Mono.Cecil.ReaderParameters parameters) [0x00000] in <ebce9b93ad98499b85c2fc78d9656633>:0
  at CNeptune.Program.Manage (System.String assembly) [0x0002d] in <1072b40fcc9947f48e666e8b38648f6c>:0
  at CNeptune.Program.Main (System.String[] arguments) [0x0012c] in <1072b40fcc9947f48e666e8b38648f6c>:0
/Users/rkttu/Projects/AspNetMvcTest/packages/CNeptune.1.1.0/build/CNeptune.targets(10,9): error MSB3073: The command "mono "/Users/rkttu/Projects/AspNetMvcTest/packages/CNeptune.1.1.0/build/CNeptune.exe" "/Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest/AspNetMvcTest.csproj" "Debug"" exited with code 1.

Done building target "CNeptune" in project "AspNetMvcTest.csproj" -- FAILED.
Done building project "AspNetMvcTest.csproj" -- FAILED.
Build FAILED.
/Users/rkttu/Projects/AspNetMvcTest/packages/CNeptune.1.1.0/build/CNeptune.targets(10,9): error MSB3073: The command "mono "/Users/rkttu/Projects/AspNetMvcTest/packages/CNeptune.1.1.0/build/CNeptune.exe" "/Users/rkttu/Projects/AspNetMvcTest/AspNetMvcTest/AspNetMvcTest.csproj" "Debug"" exited with code 1.
0 Warning(s)
1 Error(s)
Time Elapsed 00:00:01.09
`

Hi,
You're right concerning the origin of issue, it clearly seems that it is cause by path manipulation in CNeptune. Project file (.csproj) is parsed to build assembly path using project gile location and assembly name.
I probably need to include more checks to garantee this process.
Unfortunately, I actively work on Async support and fusion of CNeptune & NConcern to provide some exclusives advanced features while keeping performance as high as possible.
It may delay a little the fix you need to the middle/end of next week.


CI tests on Mumbai can run out of fake MATIC

Summary
test_mumbai.py has two tests. Each relies on fake MATIC being in two pre-specified accounts. We're good for local manually-initiated tests. But CI can run out of MATIC, in turn making the test fail. Example traceback below. This is bad, because unit tests should pass, not fail to to external factors like this.
To fix, test_mumbai can auto-grab more fake MATIC if needed.
Details below.
On local manually-initated tests
developers.md tells developers how to get that MATIC, as follows.
Some tests run on Mumbai (e.g. test_mumbai.py), which need fake MATIC. So you also need:
#set private keys of two remote accounts
export REMOTE_TEST_PRIVATE_KEY1=<your remote private key 1>
export REMOTE_TEST_PRIVATE_KEY2=<your remote private key 2>

These keys aren't public because bots could eat the fake MATIC. You need to generate your own, and fill them with a faucet; here's how. (Or, access-protected OPF keys).
How to fix in CI
To fix, test_mumbai can auto-grab more fake MATIC if needed.
Specifically: when test_mumbai runs, for each account, it can check to see if there's enough MATIC the account. If not, it can query a faucet API to get more.
Here's an article that uses a faucet api, and how to use it in code. https://medium.com/the-blockchain-presence-blog/getting-more-matic-on-polygon-mumbai-5db25d13480d
If that doesn't work, we can explore other options. This search may help: https://www.google.com/search?q=mumbai+faucet+api
Example traceback
>>> asset = ocean.assets.create_url_asset(name, url, alice_wallet)

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/ocean.py/ocean_lib/ocean/ocean_assets.py", line 266, in create_url_asset
    return self._create1(name, files, publisher_wallet)
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/ocean.py/ocean_lib/ocean/ocean_assets.py", line 305, in _create1
    asset = self.create(
  File "/home/trentmc/code/ocean.py/ocean_lib/ocean/ocean_assets.py", line 410, in create
    tx_id = data_nft_factory.deploy_erc721_contract(
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/ocean.py/ocean_lib/models/data_nft_factory.py", line 86, in deploy_erc721_contract
    return self.send_transaction(
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/ocean.py/ocean_lib/web3_internal/contract_base.py", line 201, in send_transaction
    return contract_function.transact(
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/ocean.py/ocean_lib/web3_internal/web3_overrides/contract.py", line 74, in transact
    return transact_with_contract_function(
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/ocean.py/ocean_lib/web3_internal/web3_overrides/contract.py", line 137, in transact_with_contract_function
    txn_hash = web3.eth.send_raw_transaction(raw_tx)
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/web3/eth.py", line 818, in send_raw_transaction
    return self._send_raw_transaction(transaction)
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/web3/module.py", line 57, in caller
    result = w3.manager.request_blocking(method_str,
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/web3/manager.py", line 198, in request_blocking
    return self.formatted_response(response,
  File "/home/trentmc/code/ocean.py/venv/lib/python3.10/site-packages/web3/manager.py", line 171, in formatted_response
    raise ValueError(response["error"])
ValueError: {'code': -32000, 'message': 'insufficient funds for gas * price + value'}

A partial fix: https://github.com/oceanprotocol/ocean.py/commit/e1b85c4ba0998c286e38a71c156219c59d195bf3
Before, Alice sent 0.001 MATIC to Bob. After, Alice sends 1e-8 MATIC:
"value": web3.toWei(1e-8, "ether")

How long will this last?
The standard MATIC faucet gives 0.2 MATIC on each dispense.
A typical tx fee is about: 0.000050715 MATIC = 5.7e-5 MATIC. That's one example. Here are all the txs for the Alice wallet.
Before, the dominant MATIC spending was Alice sending MATIC to Bob. 0.2 / 0.001 = 200 txs. Therefore it would fail for sure after 200 CI runs, assuming no top-ups.
Now, the dominant MATIC spending is tx fees. 0.2 / 5.7e-5 = 3500 txs. That helps! It means we don't have to go to the faucet as often.
But I'd still like to automatically fill when low. WIP...

Ideas & analysis

Idea: find an api to dispense. Pro: it's best because it directly solves the problem. Con: I can't find it; I searched for >30 min with no luck.
Idea: If Alice is low on funds, get from Bob. Con: only buys a bit of time
Idea: don't fail on "insufficient funds"; do fail on other errors. Pro: immediately stops all problems with gas. Pro: can be used locally no problem. Con: doesn't guarantee it happens in CI always.
Idea: manually get funds for 100 different wallets. Use alchemy faucet to get 1.0 each time (vs 0.2 for standard faucet).. Then programmatically have them send their eth to Alice. Pro: good for a while. Con: manually intensive
Idea: for Ocean tx test: switch from simple-but-higher-gas Ocean tx, to lowest-gas Ocean tx. (Send an existing datatoken?)
Idea: for Ocean test: don't do a write tx, just do a read, which isn't a tx. Eg on an existing data NFT or datatoken. So zero gasPro: zero gas. Con: not quite a full tx.

Recommendation:

do (3) - don't fail on "insufficient funds"; and
do (6) - a "read" test

In pursuing (1) - api, here's what I was trying.
>>> url = "https://api.faucet.matic.network/"
>>> json = {'network': 'mumbai', 'token': 'maticToken', 'address': '0xA54ABd42b11B7C97538CAD7C6A2820419ddF703E'}
>>> import requests
>>> response = requests.post(url, json=json)
>>> response.json()
{'error': {'message': 'Not found'}}

>>> url = 'https://api.faucet.matic.network/transferTokens'
>>> response = requests.post(url, json=json)
>>> response.json()
{'error': 'error processing request'}

Another error I would get in Polygon is:
    @staticmethod
    def formatted_response(
        response: RPCResponse,
        params: Any,
        error_formatters: Optional[Callable[..., Any]] = None,
        null_result_formatters: Optional[Callable[..., Any]] = None,
    ) -> Any:
        if "error" in response:
            apply_error_formatters(error_formatters, response)
>           raise ValueError(response["error"])
E           ValueError: {'code': -32000, 'message': 'replacement transaction underpriced'}

try (1) a bit more - api - with searches around https://api.faucet.matic.network/.

I tried. No luck.

do (3) - don't fail on "insufficient funds"

Done! This was the big one. See incoming merge, which will close this issue:)

(bonus) do (6) - a "read" test

I didn't bother, for here.

Some more digging by @AnaLoznianu :

It looks like previously, it was possible to fire this POST request(the same that https://faucet.polygon.technology/ uses):
"https://api.faucet.matic.network/transferTokens"
Body:   network: "mumbai"
address: 'your address'
token: "maticToken" (edited)
But now, that request always fails, they introduced a rechaptha token, rcToken that looks something like
"rcToken":"03AIIukziTBRU04TcjVkAzyjnKos7WPwrDgZXCUqw-Tk4XsfyXraokNxkEs-ufhZoeYBfQ6RP-Ni_llbDuw00z85TXT-TGry0wWNm6-nZQhtDB3meIoQESVH3fhwtpq7-HHpYV5STAr9BYfjqXHiN0bvq568bzr5bSCIcCHow-UpBdAZ832ASs5FBHlqaTb0NB3FKm0liZ3SC7Bg4voSgNCAGrasl2KYV94Ic3sGrbCBaWyW1C6VWlltuyvOcULU8imRPmSyWOeYVEDbwg3F3HGo5PQ1A5gV07fdGaCn5fL--USrT3PVd2FhNlf9OPoEJyujoEP4QMjZ1jzY767uhcxLZrxs8MC5IFGMOC_uUwRRUHJilE6CCv8zu8CgIMQD1xYH71W9A9IiGMZfNms03cRP0_zpXRAUse0IQlNk1eegX_AAgiwVnQq1jaOsa3LcwyZJZ-i_ntBAh7NHDVSjXTrayth-4z2C6WGGLHgztE7d0BuToiyENtFvUZ_D2CVYOy0fIglFIGpCta94ofXLVz6iqEdTcZznE6ddsempgaNRB2oB803A5XxeVgk0DXJ5CHly9bOpz9lWN4Rj6mbppSuvae9cCM97IJlw"

Talk to Polygon Support on the Live support page so that your issue can be looked into.


Update library locations for multi-lib *nix systems.

I currently maintain OpenImageIO for Fedora and I'm working on packaging OpenColorIO now.
These updates install 64 bit libraries into /usr/lib64 instead of /usr/lib on multi-lib linux systems.

Cool. Ill check this out soon.  I've just pushed a makefile update related to python installation, so this should probably be rebased...

I'm also working on unbundling everything in ./ext since Fedora doesn't allow bundled libraries. I'm trying to do it in a way that works for everyone by adding options to the main cmake config to use the system libraries but defaulting to off.
I'm more of a hacker than a programmer though so it's a lot of trial and error :)

hmmm.... some of the things in ext are not exposed in the public OCIO API; and not being a build expert I'd prefer to not expose additional runtime library dependencies. (tinyxml + yamlcpp, for example).
With your implementation, on a fedora build that had one of these libraries installed, would you link to the xml / yaml so(s), or would it use the .a statically at build time?  I'd hate to have the dependancies in the core library change depending on build options.  What if I pulled in these two libraries into 'core' as source files? (it used to be this way, actually).   I'm fine with picking up lcms, etc externally. But i'd like core to be self-contained...
@dbr, @malcolmhumphreys, @colindoncaster - thoughts?

I'm not sure I understand well enough to answer. I can't say that bundled libraries are absolutely forbidden in Fedora but it often requires an exemption from the board. Can you explain the need for static linking a little more? I have a rudimentary understanding of API's but that's about it.
As far as yaml-cpp, the current Fedora package does not provide static libraries, only shared.
I'll ask on the Fedora devel list as well. Others must have run into this situation.

Actually, thinking further the work you've done so far looks great.  Thanks!   Thinking further about ext handling, im totally willing to be overridden. (that even xml/yaml handling should come from the system).  Just note that both of these have patch files, both both code and build reasons. (Though our yaml change may be incorporated into the main library now? Ill have to check).
Also note that of the other libs in ext, only lcms / oiio is used by the command-line apps.   The remainder are used by the docs build target, and can be skipped for normal installs.  And the oiio tools should probably be skipped too - I'd like to move these into oiio (or an examples dir) instead.  They're really both 'toy' example apps.

I think you can hold off on asking Fedora for the moment. Let me investigate if the current yaml-cpp will suffice for our needs.
But... if you wanted to ask them one thing I'd be curious on their thoughts on how to publish PyOpenColorIO.  It both provides C++ symbols apps need to link against, and also is a python module.  What is the right place to install such a library?

I noticed the unittest.h in the oiio folder. Since I already maintain oiio for Fedora I looked for the same file. It is in the source but it's not installed. Should I ask oiio upstream to add this to /usr/include/OpenImageIO?
I would like to get the docs build working but that can always be fixed at a later date. I'll ask if bundled apps that aren't linked against (i.e. used for doc generation) is OK.
For the time being I put "lib" back on PyOpenColorIO and installed it to /usr/lib{,64} which I think is the right place. I'll ask Fedora-devel if maybe symlinks should also be put in /usr/lib{,64}/pythonX.X/site-packages.

For now I wouldn't unbundle the ext libraries, this would most likely needed to be done later to get the package accepted into the official package repositories ie fedora, debian, centos etc. I think their is still great value in getting rpm's and deb's built which can be installed on the most relevant distros.
We have to workout the best way to deal with the bundled libs @jeremyselan it doesn't seem to matter if they are in the source or not (http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries) reading this suggest its better if they are not. All the external deps would need to be shared libs.
I would prefer to make an application for an exception and take the lead from the fedora team. @hobbes1069 the main concern is to do with keeping the ocio library functioning in a dependable way across all platforms (kind of the whole point of it). Having to unbundle these internal libs (which are mainly used for serialisation) could mean the library behaves differently on a fedora vs debian vs centos machine.

oh I should say these changes look good! we should split this off into another thing about fedora packaging.

Perhaps we should close this then and move the discussion to ocio-dev?

Sure. Let's move it to ocio-dev.   Thanks for the fedora link, its opening my eyes as to how this may be a bit more work than i imagined to pull out the dependencies.  I also liked the video on that page, http://blip.tv/pycon-us-videos-2009-2010-2011/how-to-build-applications-linux-distributions-will-package-2083028   (audio only starts a few minutes in though).  Really makes you appreciate the role of packagers.

Committed in 33fca0a7072e4113b312583f1d2c5fc2837eb4f8


Team B: Refine initial requirements with the customer

The following initial requirements need refinement in order to create a set of broken down requirements to be entered into the SRS:
Accessibility

Gesture and Voice driven (All features)
Read for visually impaired (All features)

Search View

Search for a past email for a specified date range and sender
Store content of mailing cards to search in a user's storage (OCR search of images)
Provide feedback to USPS on searches conducted providing marketing support

Chatbot

Chatbot to provide access to search engine
Chatbot available through app (maybe website)

Broke this out into Issues by requirement.


Permit use of gif image format if prawn-gmagick is available

The prawn-gmagick gem adds gif support to Prawn (as well as a wide range of other formats). Therefore, permit use of the gif image format if the prawn-gmagick gem is loaded.
We may even want to consider loading the prawn-gmagick gem if it's installed.

Using prawn-gmagick will also allow the use of other formats such as TIFF.


Require go 1.19 for atomic.Int64

Synapse tried to build complement using the go version specificed in go.mod, and failed:
 📦 github.com/matrix-org/complement/internal/docker
Error: internal/docker/deployment.go:29:26: undefined: atomic.Int64

Does the setup-go in this repo need to be updated to enforce this? I.e. these have the same warning:

Warning: go-version input was not specified. The action will try to use pre-installed version.

Oh, ISWYM! It feels like it probably ought to.


BSP Documentation

The bsp file format should be documented so the project can be easier understood. Something like what I did for the .dem format.

It would be nice, but I don't have time right now. I'm busy working. You're free to do it if you like.
I used this[1] specification document, but I only used it to extract entities, textures and geometry.
Important things that I haven't done yet:

Lightmaps
Lightmap lump stores static lighting textures. I haven't yet implemented algorithm to use them. That's why there are no shadows and other light effects on maps. [2]

Visibility
Visibility lump has data that can be used to avoid rendering parts of the map that are not visible from current position of the camera. On small and medium maps, such as de_dust2, bkz_goldbhop, etc., it doesn't make much difference, but on maps like kz_world, nks_pupu, kz_gigablockier, etc., it would greatly improve performance. I get only ~15 fps on nks_pupu, but in-game it's stable 100 fps. [3]

[1] http://hlbsp.sourceforge.net/index.php?content=bspdef
[2] http://imgur.com/a/GPh8a (hlviewer left, in-game right)
[3] http://imgur.com/a/WpcsV

Added BSP documentation in ee537b05ef28eda5c1fd505cc41f09166ef00975


Allows pushing Lua code

This PR allows pushing Lua code.
For example:
lua.set("foo", hlua::functions_read::LuaCode("return 5"));

You can then do:
let r = lua.execute_code::<i32>("foo()").unwrap();
assert_eq!(r, 5);

Similar to LuaCode, also adds LuaCodeFromReader.
This change required modifying the Push trait to return a Result instead, in order to handle syntax errors.
More code cleanups will need to be made later. Since the code was quite dirty already, I don't think it's a problem to add even more panicking corner cases.

While it is maybe controversial to modify Push to return a Result, let's look at the alternatives:

Not add this feature. This would be unfortunate as there's currently no way to put a Lua function to a variable.
Use two steps: put the function on the stack with a custom method, then write it, with two different methods. However this causes tons of problems, like how to handle the fact that the user could pop something under the stack before poping the function at the top.
Panic if there's a syntax error. Meh.

I'm going to change type Err = (); into type Err = Void; where Void is a non-instantiable type.

The idea now is that set can still be used for anything except LuaCode and LuaCodeFromReader, while the new checked_set functions can be used with everything and returns a Result.


Fix readme markdown

Wow, seems no one noticed for almost 2 years. (or did github change its markdown renderer?)
Anyway, thanks, @OleksandrBezhan.