buildRustPackage is not very nix-ish

Hi, I started developping some rust code on nixos, and I find that while buildRustPackage is great for having all of crates.io available, the experience is not quite on par with, say, that of developping in haskell or ocaml on nix.
Specifically:

Documentation is a bit terse (I only found this wiki page: https://nixos.org/wiki/Rust), and it is not clear if it is up to date.
In Rust projects with both Rust and C dependencies, having the C dependencies known of nix, and the Rust dependencies known to Cargo is not convenient: Cargo.lock cannot be generated from within a nix-shell because of depsSha256, so one needs the awkward nix-shell -p cargo -p rustc -p
C dependencies of Rust dependencies of a project need to be discovered and added manually to default.nix (dependencies are forgotten when going through Cargo-land)
When using nix-shell, builds of rust dependencies are not shared between rust projects.

I'm grouping these into one issue because I'm not sure which of these sub-issues can be fixed in buildRustPackage, and what would need something more akin to cabal2nix where rust dependencies of rust packages are made manifest to nix/within nixpkgs.

I agree with your points.
Regarding your 3rd point, that may not be very easy to accomplish. I was thinking we'd need something that parses Cargo.toml, but even Cargo.toml doesn't seem to tell you what C dependencies you need, see for example: https://github.com/brson/rust-sdl/blob/master/Cargo.toml
This may be even harder to figure out when libraries are dynamically loaded via dlopen. Case in point, the x11-dl library, which loads its dependencies dynamically: https://github.com/Daggerbot/x11-rs/blob/master/x11-dl/Cargo.toml
If I understand correctly your 4th point, I think it may also not be easy accomplish at this point in time (but I hope I'm wrong). As far as I know, there is no way to tell cargo how to use an already-built library. I think I asked the Cargo developers how to do this, and IIRC they told me this is not supported because Rust doesn't have a stable ABI yet (which is probably a poor excuse in Nix's case).

Regarding the 3rd point, perhaps we could have something that crawls a source tree looking for stuff like #[link(name="SDL")], and then we just need a map from the SO name to the Nixpkgs package name. Perhaps a similar thing could be done for dlopen (at least where dlopen is used with a string litteral).
That might pull in some false positives, but then it's just a matter of removing the unnecessary stuff, which is much easier IMO than trying to hunt down the dependencies in the first place.

And regarding the 4th point, as @wizeman notes:

If I understand correctly your 4th point, I think it may also not be easy to accomplish at this point in time (but I hope I'm wrong). As far as I know, there is no way to tell cargo how to use an already-built library. I think I asked the Cargo developers how to do this, and IIRC they told me this is not supported because Rust doesn't have a stable ABI yet (which is probably a poor excuse in Nix's case).

That's precisely what I recall from the last time I looked into Rust on Nix.

As far as I know, there is no way to tell cargo how to use an already-built library.

I'd make these two simplish requests of them:

Allow configuring/building dependencies with a downstream lockfile
Allow an unstable way to pass arbitrary flags to rustc without cargo complaining.

Then we should be to do the rest ourselves.

Allow configuring/building dependencies with a downstream lockfile

Don't overrides allow to do that?

I think the whole rust situation is bound to become worse for some time. I was trying to update cargo to 0.10.0 version, but it breaks buildRustPackage function.
-- Cargo needs write access to registry as they implemented file locking (https://github.com/rust-lang/cargo/pull/2486). And it seems that they now try to write .cargo-lock-index file to registry directory. This can be mitigated by copying whole registry to package build directory with write permissions.
-- Cargo fetch now barfs at NixOS hack with file:///dev/null as it seems that they always try to update registry.
    Updating registry `file:///dev/null`
error: failed to fetch `file:///dev/null`

Caused by:
  [6/-1] Could not find repository from '/dev/null'

If somebody could look into this, as I don't have enough knowledge and experience with this kind of stuff :) Or maybe we should wait for: https://github.com/rust-lang/cargo/pull/2361

Could  cargo-local-registry be used here to improve the situation?

Not very much :(. The probable is caching rlibs, not source.

Could we use multiple output and have each rust package output both its binaries / rlibs and its source. Then each rust package in nix could depend on the "source" output of its cargo dependencies. I think this would solve issues 2 and 3 of my original report, since rust-to-rust dependencies would be tracked by nix.

See this Reddit discussion + blog post for an alternative approach. Looks interesting.

And the relevant nixpkgs issue ticket: #24991.

Can we close this because Carnix and buildRustCrate?

I think so. There are still some points which could be improved, but we'd better handle that separate tickets.

For that matter, is there a proposal to replace all buildRustPackage with Carnix?

No. Carnix still generates very large nix expressions that are not shared between packages.

Well we have a copy with some fixes in the nix-community group. It's technically a fork but just to have a "backup" of what's proposed for carnix and a copy of the released versions. There was some attempt at mirroring cargo into nixpkgs but as @Mic92 said it's generating very large expressions. Those missing features and the mentioned use-cases are definitely things I and others want to have in carnix. We still have to discuss with the original author. It will probably happen around nixcon.. (Excuses the lack of links & mentions, typed on a crippled phone)

it's generating very large expressions.

All the language infrastructures are hitting this. I really want to open the discussion on import-from-derivation to fix the problem once and for all.

@andir where is that fork? The nest doesn't seem to have any way to search, and I can't see it in your github. I want to contribute.

this looks promising: https://www.hadean.com/blog/managing-rust-dependencies-with-nix-part-ii

@illegalprime It doesnt, since it uses cargo in the build.
@andir Carnix is a mess. There might be some code worth saving thou.

@lifranc cargo has a lot of stuff in it that would be hard to maintain in a rewrite (features, editions, etc.), as long as nix has fully control over building each crate, what's wrong with building it with cargo?

what's wrong with building it with cargo?

Never used that word. not promising is what I expressed. I'm certain you can see why.

@lifranc hmm... what do you think would be a good solution? (maybe that involves a contribution to rustc or cargo, but that's even more hoops to jump through)

@illegalprime What carnix does (try to do maybe); pull all the data needed into nix expressions. The expressions carnix outputs have the wrong structure imo and feel like they stop at an intermediary step (stil lots of functions and boilerplate code in files that should only contain crates' metadata). So either fix carnix or do it over.


In the meantime though the LaTeX works locally so that's that.


Add Header to Workspaces Doc for Website

Changes
Our website needs a header on the docs that it consumes to indicate
ordering and page title.
This PR adds the needed header to the Workspaces doc and inserts it
just before PipelineResources in the ordering.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Commit messages follow commit message best practices

/test pull-tekton-pipeline-build-tests


ghc-ncurses6-8.4.4 references libtinfo

stack setup on a system with ncurses and without libtinfo downloads ghc-8.4.4-x86_64-fedora27-linux.tar.xz as ghc-ncurses6-8.4.4 , but as a binary in the tarbal references libtinfo, the setup fails.
Installing libtinfo on the system fixes the situation, but when libtinfo is available, stack setup downloads the same ghc-8.4.4-x86_64-fedora27-linux.tar.xz as ghc-tinfo6-8.4.4 this time, so I guess the ghc-ncurses6-8.4.4 is intented to be installed on libtinfo-less systems and referencing libtinfo is a bug in that case.
this is the output of the failing setup:
Writing implicit global project config file to: /home/marian/.stack/global-project/stack.yaml
Note: You can change the snapshot via the resolver field there.
Using latest snapshot resolver: lts-12.21
Downloaded lts-12.21 build plan.
Preparing to install GHC (ncurses6) to an isolated location.
This will not interfere with any system-level installation.
Downloaded ghc-ncurses6-8.4.4.
Installing GHC ...
Received ExitFailure 2 when running
Raw command: /usr/bin/gmake install
Run from: /home/marian/.stack/programs/x86_64-linux/ghc-ncurses6-8.4.4.temp/ghc-8.4.4/

Thanks for your thorough debugging.  You're right, it shouldn't try to install the fedora27 (libtinfo linked) bindist on systems that only have libncurses6.  The distro transition to libtinfo/libncurses6 was messy for a while, and stack's approach to choosing a bindist wasn't quite right, and looks like things got a bit confused.
Now, the majority of distros include libtinfo6 (which may just be a symlink to libncurses6), and libtinfo6 is what the official GHC bindists are linked to.  Stack is also better at choosing the right bindist now.
At this point, I think we could remove the ncurses6 entries from stack-setup-2.yaml.  This means that in your situation you'd have gotten an error message saying that there's no ncurses6 bindist available, which is much more sensible.
It might also be good to make this error message a bit more friendly for this situation and have it point the user to a FAQ entry.

(I've now removed the fedora27 bindists from the ncurses6 variant entries)

On 12/3/18 10:51 PM, Emanuel Borsboom wrote:

Thanks for your thorough debugging. You're right, it shouldn't try to
install the fedora27 (libtinfo linked) bindist on systems that only have
libncurses6. The distro transition to libtinfo/libncurses6 was messy for
a while, and stack's approach to choosing a bindist wasn't quite right,
and looks like things got a bit confused.

Yes, I think so too.
Mine system is Gentoo, which only builds libtinfo when explicitly requested.

On 12/3/18 10:59 PM, Emanuel Borsboom wrote:

(I've now removed the |fedora27| bindists from the |ncurses6| variant
entries)

It works now, thank you.


Adding read and write advice to jkind api (plus some kind2 stuff)

Squashed to master as 9c68d796dec3213c9b1dd3f19893cf446cc595ce


pixi global list fails when empty global environment is present

Checks

[x] I have checked that this issue has not already been reported.

[x] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
[envs.some_dev]
channels = ["conda-forge"]
dependencies = {}
exposed = {}

Issue description
If an empty environment is present in pixi global manifest as in the example
it will make pixi global list fail with the following error:
× No package records found in /Users/graf/.pixi/envs/some_dev/conda-meta
Expected behavior
I think we should not allow such (empty) environment declaration in manifest.

I think we should not allow such (empty) environment declaration in manifest.

I was starting out with that, but I remember that it made things a bit awkward in a couple of places. I think we should just fix pixi global list :)


101 improve phase processing

See #101

Can you take a look if these changes work? I have not yet been able to fully test them...

Looking at the Tests error log it is saying there's an attribute error in your code when calling:
Measurand.active_power_import.value

Looking at the Tests error log it is saying there's an attribute error in your code when calling:
Measurand.active_power_import.value

Should be: power_active_import

Ok to merge?


Fix unstable test and nightly build

What is the purpose of the change
This PR fixes #12 and #10.
Brief change log

Add option to nightly build twine command to skip uploading wheel if the wheel already exists
Add the ".csv" suffix to the randomly generated filename to fix unstable test

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Thanks for the PR! LGTM.


Partition of set into K subsets of equal sum.

• Brute force -> Efficient Solution
• Step by Step

Hey @aaryahjolia  ,
I'm Darshan Jain, and I'm thrilled to be a part of this GSSOC'23. I bring along my expertise in Data Structures and Algorithms in C++. I'm passionate about open source projects and eager to contribute and collaborate with fellow enthusiasts.
Can you please assign this to me.


WIP: Pending Transaction RPC

Pull Request type
Please add the labels corresponding to the type of changes your PR introduces:

Feature

What is the current behavior?
Resolves: #204
What is the new behavior?
Does this introduce a breaking change?
No
Other information
Rebase from https://github.com/keep-starknet-strange/madara/pull/293 ; I cannot push to the original branch on finiam repo

Could you please check that the pending transaction vector is also empty until the moment of finalizing the block?
It does not block this PR, just that it's a behavior that will cause this endpoint to almost always return an empty vector.


Refactor documentation

Refactoring documentation to fit the Diátaxis structure
Tasks remaining:

[ ] Generate vimdocs from reference.md
[ ] Generate API reference
[ ] Lint for broken markdown links
[ ] Additional guides/tutorials

Fantastic, thank you!

Messed this one up. Had the wrong tab open and thought I was merging a PR in another repo. DERP. Continued in #30.


Windows CI: Upload test results to AZP

Commit Message:
Windows: Upload test results to AZP

Requires removing --output_base flag to Bazel startup options
The TEST_TMPDIR and BUILD_DIR bind mounts from the host are where
Bazel will place build output etc.
Allowing Bazel to place build and test output there will give the host
access to the data to upload
Only riskiness involves Windows and long paths, adds a good few characters
to the exec root/output base Bazel uses
Should help with diagnosing test flakiness on Windows, having test results actually
uploaded to AZP so we can track failures over time

Additional Description: N/A
Risk Level: Low
Testing: N/A, CI change
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

@envoyproxy/windows-dev

See https://dev.azure.com/cncf/envoy/_build/results?buildId=57958&view=ms.vss-test-web.build-test-results-tab should be able to filter now for windows test results

/wait
trying to minimize path length increases

/retest

merged master to get coverage to hopefully pass, shouldnt be affected by this change


Kustomize KEP 1755 failing test

What happened?
The kustomize e2e test "TestOperatorBasic" recently started to consistently (if not always) fail.
This failure is on the registry configuration check that was added by this PR : https://github.com/apache/camel-k/pull/4849.

Steps to reproduce
No response
Relevant log output
operator_test.go:76:
          Expected
              <string>:
          to equal
              <string>: kind-registry:5000

Camel K version
main branch

@squakez I'll take care of it.

@squakez The last tests are clean, the issue is fixed.


[RFC]: add math/base/special/rad2degf

Description
This RFC proposes adding math/base/special/rad2degf, which would be the the single-precision equivalent for math/base/special/rad2deg.
float stdlib_base_rad2degf( const float x )

Related Issues
None.
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

I'd like to work on this.

#2126 would be a pre-requisite for this one. While it isn't directly useful from the implementation point of view, it is required for bencharks and examples.


Make tomte a dev dependency

Proposed changes
tomte doesn't need to be a core dependency of this repo. Move it to a dev dependency to avoid unnecessary dependency restrictions on dependant projects.
Types of changes

[x] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Thanks @evangriffiths !

Thanks for the merge @0xArdi


Corrected 'registrating' to 'registering'

TIL: "registrating" is to select and combine pipe organ stops.  :)

@rcabr : Thanks for your contribution! The author, @wmgries, has been notified to review your proposed change.

#sign-off
Thanks @rcabr for fixing this typo!


Implement mutation observer

As part of #102, we need an observer that hooks into analyses*, scours compiler logs, and does the following for each compilation:

If we detect the words MUTATION HIT: in a log line, parse an integer and increment a hit counter for that mutant;
If we do so and the compilation was deemed interesting, declare that mutant killed by that compilation and send a signal upstream (ie to whatever is controlling the mutant time-slicing).

In both situations, we would probably also have a log file open and dump information about the hit/kill.
I'm not yet sure whether this would happen synchronously as part of the call into the observer, or involve a channel of some sort.

It hasn't been decided yet whether it hooks into these at the machine level or global level, and so whether this would be an analysis or a cycle-analysis.

[ ] Implement and test line parsing
[ ] Implement (and test?) per-compilation behaviour
[ ] Implement (and test?) observer boilerplate

[ ] Implement and test line parsing
[ ] Implement (and test?) per-compilation behaviour
[ ] Implement (and test?) observer boilerplate

This is probably, actually, best done as an analysis pass, followed by an observer that scrapes the analyser output.  I might just do that.

This is probably, actually, best done as an analysis pass, followed by an observer that scrapes the analyser output.  I might just do that.

Done (albeit somewhat hardcoded in for now).

Done (albeit somewhat hardcoded in for now).


Falsely reported "Call to undefined static method" if __construct may throw an Exception

Larastan Version: 2.6.0
--level used: 5

Description
A 'Call to undefined static method' error occurs when the constructor of the target class of Facade may throw an exception.
If there is no part that throws an exception, no error will occur.
Laravel code where the issue was found
app/Foo.php:
<?php

namespace App;

class Foo
{
    public function __construct()
    {
        if (config('app.foo') === null) {
            throw new \Exception('app.foo required'); // error will not occur if this line is commented out
        }
    }
    public function foo(): void
    {
        //
    }
}

app/FooFacade.php:
<?php

namespace App;

use Illuminate\Support\Facades\Facade;

class FooFacade extends Facade
{
    protected static function getFacadeAccessor()
    {
        return Foo::class;
    }
}

app/SomeClass.php:
<?php

namespace App;

class SomeClass
{
    public function callFoo(): void
    {
        FooFacade::foo();
    }
}

Note: The result cache may cause changes to the file that do not change the results, so either clear-result-cache before executing the analyze command or specify the --debug option to the analyse command.

Larastan boots your service container and so the __construct method is called during the static analysis since the class is being resolved using the container. In this case, Larastan is throwing the exception since config('app.foo') === null

Exactly what @CalebDW said. Also you can add /** @mixin Foo */ to top of FooFacade, so Larastan will not have to resolve the facade itself.


Massive errors for "on player places/breaks block"

Spigot 1.12.1 (65e8124-bde2a93) and Denizen Developmental build 240
When I use on player places human_skull and on player breaks human_skull, I seem to be getting errors.
Errors for placing the human skull: https://pastebin.com/LryTxyBu
Errors for breaking the human skull: https://pastebin.com/5tGeXpne
Script: https://pastebin.com/PA7aNVgs

It came to my attention that "human_skull" is not a real material. The script is still the same, just replace the human_skull parts with skull.


Typo

Sorry, don't know how this space got into my last PR :/

Fixed this in https://github.com/lamby/django-email-from-template/commit/80ab9786e913dcb4acd5d289a0da44dcfc1931db - I think your PR here diverged from master.

Released as 2.4.1.

Oops, thanks 😅


add pagination

This feature will not be implemented because notifications are simple objects with short time to live indeed it doesn't require pagination.


How to train this code on our own data set.?

@wkentaro Respected Sir, Can you please guide me how to run this code on our Custom images.
And How to get "frame-0000XX.pose.txt" files as an input for this code.??
I will be waiting for your kind responce..

How to get "frame-0000XX.pose.txt" files as an input for this code.?

You can correct data with a hand-mounted camera on a robot.
In the example, this repo is using data from this paper:
https://github.com/andyzeng/apc-vision-toolbox


Various portico updates for 1.9.

@timabbott would be great to get this (or at least the first two commits) merged and deployed before the release. I think replacing our heros with pictures significantly changes the initial impressions of the pages.
Not the greatest pictures, but I found for now:

Merged, thanks @rishig!


reformatting and --profile check on !debug builds

I needed to set a custom profile, as we ran into a link error on lto = fat. Setting a profile was not allowed on a release build.

Hi @boymaas, sorry for the long delay!
I'll merge your PR, but can you tell me more about your issues with profiles and LTO?
Thanks for this PR!


minor bug fix for esp-idf

fix infinite recursive call in ao_beginTransaction
define dtostrf function for esp-idf (it's undefined)

Hi @nelopauselli, thank you very much for your contribution!


Make RestClient members virtual

Hi RingCentral,
Would you please make the members of RestClient virtual?
We would like to override them with custom implementations, specifically to help with token management.

Could you please specify which members?
As mentioned here https://github.com/ringcentral/RingCentral.Net/issues/26#issuecomment-694341340 we are migrating to an extension architecture in next major version. So inherit and override may not be the way we recommend.

Since 5.0.0-beta1, this project has been migrated to an extension based architecture.
So if you want to extend the SDK, create an extension. There are already a couple of extensions FYI: https://github.com/ringcentral/RingCentral.Net#extensions


JSDoc Type hints not working for imported types.

Let me know if this is actually a TypeScript bug.  While there isn't explicitly any TypeScript involved (except the typings file in the source) I recognize that a lot of the JavaScript intellisense is sourced from the TypeScript engine.

VSCode Version: 1.10.1
OS Version: Windows 10

Steps to Reproduce:
var Foo = require('foo');
/** @type {Foo} */
var foo; // `foo` is of type `any` according to tooltip.

var Foo = require('foo');
var foo = new Foo(); // `foo` is of type `Foo` according to tooltip.

/** @type {WebGLRenderingContext} */
var foo; // `foo` is of type `WebGLRenderingContext` according to tooltip.

The above illustrates that while VSCode knows about and understands the imported type Foo (the package has a typing file).  When using JSDoc for type hinting in JavaScript, VSCode doesn't appear to recognize it in the JSDoc.  The last example shows that the JSDoc hinting is working in general by providing a globally scoped/available type in the doc and seeing that VSCode does the right thing.

Thanks. This is a duplicate of https://github.com/Microsoft/TypeScript/issues/14056 being tracked on the TypeScript side


hello world blog post

Now that we have a public talk, and a few projects in the works, I think it is a good time to tell people about 2i2c! This is a quick blog post to describe what 2i2c is, why it exists, and what it aims to do. It is sort-of a blog post version of our JupyterCon talk.
What do people think? I would love any comments or feedback. I'd like to get this out somewhat soon, so that we can follow up in the future with more focused blog posts on what we're up to, and make any big announcements that might be in the pipeline :-)
cc @2i2c-org/founders
closes https://github.com/2i2c-org/meta/issues/53

This is looking good @choldgraf! One quick thought: do we want to put an invite there to join the slack? I don't know what we want the slack to be, but it a goal is to have place for community chat, the blog would be an easy way to invite folks in

Thanks for this, @choldgraf!!!
I think https://docs.google.com/document/d/1A2lEjvmZuhmNpPc1qBnT4QLFmKza7p8Io9w36UHCIT0/edit is an importat differentiator for us, and I'd love to get it out at the same time...

Interesting point about the Slack...I can think of two approaches there:

We create a temporary "invite link" for the slack and direct people to it. We treat the Slack as a public place for 2i2c-relevant conversation.
We point people to the mailing list, and in a subsequent email to that list, send this Slack link. That way there is some degree of filtering for engagement that happens (because we have to pay for each additional person on the Slack, so don't wanna blow up our costs with a bunch of lurkers)

What do you think?

Link to issue about 'right to fork' document #23.

Update
I made a few more updates to this post (as well as the website) to try and capture more of the "kinds" of hubs that we offer, and what kinds of use-cases they're meant for. Also incorporated some of @lheagy's feedback above.

Update
Just took another pass and cleaned up some of the text. I added a "Get involved" section that includes a link to Google forms for the mailing list as well as list to sign up for Slack. If enough people want to sign up for slack we can create a dedicated #community single-user guest channel, but if not then we can probably just invite them as members to the Slack. @lheagy what do you think about that?
I'll plan to post this tomorrow (Tuesday) unless folks think that we should act differently. Please do give comments and suggestions in the meantime!


Relicense to LGPL2.1+

ack from<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

@SvetaHBar Can you approve this re-license?
Thank you.

Other authors are from Red Hat where legal team approved license change.

In case anyone interested, the changes was made by script of https://github.com/johann-petrak/licenseheaders
python ./licenseheaders.py  -t lgpl-v2.1 -o 'Red Hat, Inc' -d ~/Source/nmstate/ -y 2018-2019 -n 'nmstate' -u https://github.com/nmstate/nmstate/

Approved

New patch uploaded.
Changes:

Fixed the incorrect year comment.
Change to https for license link.
Removed the github project link.


fix to show spinner when promise is being loaded and refresh the list wh...

...en the promise has returned data.

Sorry, what exactly this PR is doing? It's not nice changing the tab spacing. Could you please update your PR without changing it? If your IDE is doing it, try in a different IDE or at plunkr, please.
The title is cut. I couldn't get it all.


Update-flake8-bugbear

Created for inspecting the dependabot PR https://github.com/sanger/lighthouse/pull/881
Bumps flake8-bugbear from 23.7.10 to 24.2.6.

Release notes
Commits

updated-dependencies:

dependency-name: flake8-bugbear dependency-type: direct:development update-type: version-update:semver-major ...

Closes #
Changes proposed in this pull request

Instructions for Reviewers
[All PRs] - Confirm PR template filled
[Feature Branches] - Review code
[Production Merges to main]
    - Check story numbers included
    - Check for debug code
    - Check version

Closing this as it was for inspection of the issues in another PR and it is no longer necessary.


Set dialect to C++11

For projects that did not specify C+11 as the dialect, std library classes and types could be shown as error with a red underline.

Tested on Linux and Windows.


Synchronize submodules

The submodules are out of sync and the code will not build. This updates the submodules to their latest master state.
git submodule foreach git pull origin master

Synchronize submodule: rapidjson
Synchronize submodule: websocketpp

Merged into https://github.com/socketio/socket.io-client-cpp/commit/af68bf3067ab45dc6a53261284e0da9afd21b636. Thanks for the pull request.

Merged into https://github.com/socketio/socket.io-client-cpp/commit/af68bf3067ab45dc6a53261284e0da9afd21b636. Thanks for the pull request.


Long (and sometimes failing) native compilation with .NET native compiler

Bug
When System.Reactive is used inside a Universal Windows Platform (UWP) application built in Release mode with .NET native (DNN) compilation enabled, native compilation time is extremely long to the point where it sometimes fails with out-of-memory errors thrown by the ILC compiler.
It has been suggested in #899 to add the property <Use64BitCompiler>true</Use64BitCompiler> to the UWP project, however this does not guarantee success.
Even with this property added to our application, our Windows store app which successfully built with System.Reactive 4.0 no longer compiles with .NET native when using System.Reactive 4.1 or higher. We are building on a powerful CI/CD pipeline with 16GB RAM and the ILC compiler often fails with out-of-memory errors such as:
Microsoft.NetNative.targets(801,5): error : Internal compiler error: Exception of type 'System.OutOfMemoryException' was thrown.

Which library version?

System.Reactive 4.1 and later.

What are the platform(s), environment(s) and related component version(s)?

Universal Windows Platform (UWP) application.

What is the use case or problem?

.NET native compilation takes a long time and often fails with out-of-memory errors.

What is the expected outcome?

.NET native compilation is successful and fast, or at least as fast as when System.Reactive 4.0 is used.

What is the actual outcome?

.NET native compilation is either:

Many magnitudes slower to complete
Fails entirely with out-of-memory errors

Do you have a code snippet or project that reproduces the problem?

Theory:
System.Reactive 4.1.x and greater causes the .NET native compiler to attempt to generate a huge amount of code, leading to long compile times.
Test:
NET Native Benchmarks.zip
The attached zip contains four UWP sample projects based on the latest blank app template.
Each sample uses a simple Observable.Timer on a 30s delay to log a timestamp to the console. The implementation is kept deliberately simple to provide an effective benchmark for comparing the .NET native compilation time between System.Reactive package versions.
Results:

System.Reactive 4.0.0 (DotNetNativeSystemRX40) >= 30s to compile
System.Reactive 4.1.6 (DotNetNativeSystemRX41) >= 1m 30s to compile
System.Reactive 4.2.0 (DotNetNativeSystemRX42) >= 1m 30s to compile
System.Reactive 4.3.1 (DotNetNativeSystemRX43) >= 1m 30s to compile

Analysis:
Thanks to @MattWhilden for the documentation on understanding the output of the ILC compiler.

One of the most helpful things for my day to day is understanding where the compiler puts various artifacts related to it’s mechanations. For the most part, these end up under obj<arch>\Release\ilc\intermediate.`

Each of the folders/items here is interestering in it’s own right but the ILTransformed directory is often the most helpful for issues like this. This directory contains most of the juicy artifacts from the “analysis” phase of compilation. Most importantly you’ll see:

*.csv files with general information
A closure file containing a list of all the dlls your app references
.csv with pdb and assembly info
A file to track all forwarded types
A file showing the result of all the decisions about which types need to be saved/generated etc due to reflection analysis
*.ildll files which are app dlls we’ve done various transformations to (still openable by ILSPY)
*.ilpdb files that help us write the final native pdb files

The most useful file in all of this is the Reflection log. It’s somewhat helpful to understand how the compiler thinks about reflection visibility etc.

I recommend glancing through this page to give yourself a sense of the things it’s trying to say. In particular, things marked as anything but Browse will cause the later phases of the compiler to need to do heavy lifting.

Looking at the size of the reflog for System.Reactive 4.3.1, the problem is clearly seen.
Diffing the two logs shows a huge number of members from the System.Reactive.Concurrency namespace now being marked as necessary. They also seem to all be marked as Dynamic which will cause the native code generator to need to kick in for each of them.
Possible Solution:
Add runtime directives into the System.Reactive package to instruct the .NET native compiler which types will never be dynamically invoked with reflection.

This is fixed now in 4.2.2 and 4.3.2.

Thanks @onovotny, this great news!
We appreciate the hard work.


docs: prepare changelog for v1.22.0 release

Codecov Report
Patch and project coverage have no change.

Comparison is base (4f6bd9a) 35.63% compared to head (cf80c15) 35.63%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #937   +/-   ##
=======================================
  Coverage   35.63%   35.63%
=======================================
  Files          92       92
  Lines       11342    11342
=======================================
  Hits         4042     4042
  Misses       6907     6907
  Partials      393      393

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


InMemoryDataset: RAM or GPU memory?

❓ Questions & Help
The description for InMemoryDataset is: Dataset base class for creating graph datasets which fit completely into memory. See here for the accompanying tutorial.
Is this referring to RAM or GPU memory?
Thank you!

This is referring to RAM. I will make this more clear in the documentation.

Thanks for clearing that up!


GH-370: Implement ability to create JARs from existing workspaces

Closes GH-370.

Codecov Report

Merging #371 (6c9f062) into main (20f40ff) will increase coverage by 0.03%.
The diff coverage is 80.76%.

:exclamation: Current head 6c9f062 differs from pull request most recent head 204df3a. Consider uploading reports for the commit 204df3a to get more accurate results

: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     #371      +/-   ##
==========================================
+ Coverage   81.21%   81.24%   +0.03%
==========================================
  Files          91       92       +1
  Lines        2816     2837      +21
  Branches      234      234
==========================================
+ Hits         2287     2305      +18
- Misses        480      483       +3
  Partials       49       49

Impacted Files
Coverage Δ

...b/ascopes/jct/workspaces/impl/FileBuilderImpl.java
43.18% <37.50%> (-3.16%)
:arrow_down:

...ub/ascopes/jct/workspaces/impl/JarFactoryImpl.java
100.00% <100.00%> (ø)


Consul persistance

Currently, internal use of Consul image assumes temporary character of stored data. However, it might be useful to have long-term storage as well, so Consul should store its data on disk, similar to how it is implemented in MariaDB image.

Implemented in ce74f1a


Insiders: Window Drag and Drop not working

Type: Bug
Windows 11

Open VS Code

Don't go fullscreen, or de-maximize if fullscreen
Make a nice regular 4:3 window size so that some of the menu is cut off by the ridiculously large command bar, and the Layout icons are nearly touching it

Try to drag the window to another location
Note you can't drag by the menu bar, or the command bar, or the layout bar - everything which overlays the title bar also breaks standard title bar behaviour (i.e. being able to move the window)

Expected:

Be able to "drag through" any title bar overlays, or
Make them smaller or resizable, or
Mark a very clear "grab handle" which is designed for use like a regular title bar!

I'm experiencing a similar issue with VS Code. After updating to the latest Insiders build, I'm no longer able to click and drag the window by the top menubar. This issue only occurs with VS Code, as other windows (e.g., Windows Explorer, browsers) are still draggable.
Details of my setup:
Visual Studio Code - Insiders
Version: 1.95.0-insider (user setup)
Commit: 7cdb7ee9df511ac3b6586be865219f49ed858d8b
Date: 2024-10-21T17:55:54.285Z (13 hours ago)
Electron: 32.1.2
ElectronBuildId: 10323173
Chromium: 128.0.6613.162
Node.js: 20.17.0
V8: 12.8.374.33-electron.0
OS: Windows_NT x64 10.0.22631

It seems to have started with this latest update. Would appreciate any insights or if others are encountering this issue!

Fixed by https://github.com/microsoft/vscode/commit/4f50e36812ff1a60b1efc46781a81a3060313720

Newest Insiders includes the fix


Node 10 support

Although Node 8 is under long-term support, Node 9 is current. Any install via brew or chocolatey will default to Node 9, which creates a bad experience for local developers that install node for the first time on a machine and must downgrade to 8.
UPDATE: node 10 just released

We want to update to (at least<EMAIL_ADDRESS>and<EMAIL_ADDRESS>There's an error in grpc's index.d.ts file that keeps us from being able to use<EMAIL_ADDRESS>in the same project: https://github.com/grpc/grpc-node/issues/176

why is this breaking? Does it mean we will drop node 8?

I don't think we've decided whether or not we're keeping node 8 as a backup (likely yes? @paulbatum), but we definitely will be changing the default version to 10

thanks @mhoeger I was asking in the context of the email thread about handling WEBSITE_NODE_DEFAULT_VERSION, I'll respond on that thread

re-opening to track pulling to functions-host repo


Feature flag to print in a non-deferred format.

I'm using defmt as my logging library. I like the efficiency that it enables. However sometimes, I want to be able to view the data without running a printer on the host machine to decode the data. I know that this is antithetical to the design of defmt, but there are several motivating reasons:

Often, I cannot use probe-rs based tools, which means that I need to rely on other tools to read UART, for example
Often these other tools expect utf-8 encoded data, or don't handle reconnection well, or print data other than the raw bytes to their stdout. Basically these tools tend to not work well as a source for defmt-print's pipe.
Sometimes I cannot even get a stdout for the serial data. This happens for example when I'm debugging my hardware with sigrok and a logic analyzer. defmt is not one of the supported encodings for the serial data, I can really only pick hex, ascii, etc.
Sometimes the binary that corresponds to the firmware is unknown.  Maybe my types don't even support Debug and thats going to require a big refactor! It would be nice to have the comfort of knowing that for sure I'll be able to use defmt and choose at the last minute before I ship my firmware whether the logs will be deferred or not.

For the exclusive purpose of enabling easier debugging in these scenarios, would it be possible to turn off the "deferred" part of defmt via a feature flag?

I'd recommend you to do something similar to what smoltcp does (link):
#[cfg(not(test))]
#[cfg(feature = "log")]
macro_rules! net_log {
    (trace, $($arg:expr),*) => { log::trace!($($arg),*) };
    (debug, $($arg:expr),*) => { log::debug!($($arg),*) };
}

#[cfg(feature = "defmt")]
macro_rules! net_log {
    (trace, $($arg:expr),*) => { defmt::trace!($($arg),*) };
    (debug, $($arg:expr),*) => { defmt::debug!($($arg),*) };
}

Then you can select if you want to use defmt or log with a cargo feature of your application.

This won't let me use the same defmt global logger though, right? I rely on those for transport of the serialized data. I would need to implement a new one for each transport type, that uses log instead, which seems quite painful.

This won't let me use the same defmt global logger though, right? I rely on those for transport of the serialized data. I would need to implement a new one for each transport type, that uses log instead, which seems quite painful.

Regarding the transport, you could try out https://crates.io/crates/rtt-logger/, which seems to enable using log together with rtt.

And many types might not implement Debug, they may only implement defmt::Format

As long as you are in control of the types this seems solvable, but I agree that this becomes annoying if it is a foreign type. Let me think about it.

Regarding the transport, you could try out https://crates.io/crates/rtt-logger/, which seems to enable using log together with rtt.

My firmware uses several different defmt implementations. Some for UART some for RTT some for usb serial, and depending on the platform I'm targetting, these implementations may use entirely different crates. I don't think its really great for firmware authors to need to add even more dependencies and complicated cfg gating when they could just reuse the existing defmt logger ecosystem.

This seems to be a pre-requisite to #716 (although I suspect that could be achieved by loading/embedding the decoder info into the binary).
Having the #[cfg(...)] for logging is "servicable" but has limitations (e.g. slightly different formatting args, different traits required, ...) and it would be a nice QoL improvement if a crate could just use defmt for logging across all targets (device/host/serial console)

Motivation

Not needing a device/emulator to check defmt logging messages from calling a function
Remove duplication: redirecting-to/co-existance with log/tracing would remove the need to duplicate/cfg!(...) logging statements depending on the target in many cases
currently I often end up with both log and defmt as optional dependencies and a couple of #[cfg(...)] statements to make it work (crate level logging modules + extras when formatting compatibility is an issue).
cargo build --all-targets --all-features could work which helps with having standard CI scripts


move find_package or test dependencies to test block

This prevents them from being used when not installed (when not building tests).

Example failure: http://build.ros2.org/view/Bdev/job/Bdev__rviz__ubuntu_bionic_amd64/2/

Example failure: http://build.ros2.org/view/Bdev/job/Bdev__rviz__ubuntu_bionic_amd64/2/


[Breaking change] Use dependencies from crates.io

This will allow publishing glium on crates.io.

This is a breaking change, as people who use glium will also need to use the crates.io versions of cgmath, nalgebra and/or image if they want to use these types with glium.


[FR] Add Helm Repo

Is your feature request related to a problem? Please describe.
It's annoying when Helm Charts aren't available online so they can be added with helm repo add.
Describe the solution you'd like
Add Helm repo for this project backed by GitHub pages.
Describe alternatives you've considered
Not using this Operator or its Chart.

I have been helping a little towards this end. @kubealex seems enthusiastic about moving in the direction you are suggesting.
https://github.com/kubealex/k8s-mediaserver-operator/issues/87
I know for now you can clone the repo, cd to the [helm-charts/k8s-mediaserver](https://github.com/kubealex/k8s-mediaserver-operator/tree/master/helm-charts/k8s-mediaserver) directory.
From there you can helm [install|template] <release-name> ./


Hitbox for playing a video prevents clicking of images at center to view

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #nightly-bugs.

Fixed. my beta branch


Support for inference on LMDB

A (small) step towards support for non-image data: this makes it possible to perform inference on any 3D blob. Blobs may not be "proper" images (e.g. because they don't have a standard number of channels).
close #619
close #630
A new field and a new button were added to allow inference on LMDB database:

Results are shown as follows (using DB key instead of image path):

Progress

[x] Add "Test DB" form to generic model show page
[x] Create new route
[x] Update inference job to take in a path to a database
[x] Update inference tool to parse an LMDB database
[x] Add tests

Tests added in latest commit.

I am interested in using Digits with my Multispectral images (4 image channels).  Since most multispectral images can be output as a separate 8 bit grayscale images (one for each channel), I was wondering if we could get Digits to accept each  multispectral images as separate channels images for both training and classification. For example the file-list could contain the image name, channel number and category.
Image1.jpg c1 1
Image1.jpg c2 1
Image1.jpg c3 1
Image1.jpg c4 1
Image2.jpg c1 1
Image2.jpg c2 1
Image2.jpg c3 1
Image2.jpg c4 1
Image3.jpg c1 2
Image3.jpg c2 2
Image3.jpg c3 2
Image3.jpg c4 2

Hi @ajcampisi DIGITS does not support the format you propose out of the box. Are you saying that each image channel - taken in isolation - is sufficient to classify the image? In that case why don't you convert the 4-channel image to grayscale with:
convert <input> -colorspace Gray <output>

and then feed those grayscale images to DIGITS?

Hi Greg,
Thanks for the reply.  No unfortunately it is not sufficient an individual channel image to be used for classification.  All four channels images need to be feed into the network as four separate network inputs and the combined image data from all four network channels can then be used by the network for both classification and training.


Refactoring remote layer & GrpcNet

As discussed in #411 I refactored the remote layer and abstracted the Remote class to give us the ability to choose between GrpcCore and GrpcNet.
The Cluster layer uses the extensions to access the remote layer.
Before creating the cluster instance with this PR, we need to add the Remote plugin to an actor system.

This is great 🎉
Some questions,
If someone uses the Grpc.NET version, and does so from an Asp.NET core app, is there a separate startup in that case or can it still safely do all the bootstrapping of configuring kestrel etc seen here? https://github.com/AsynkronIT/protoactor-dotnet/pull/585/files#diff-08dae864959e2426048096def86528a42ebf4988ddaf15e0b62096e79034d520R33
Have you done any benchmarks, e.g. RemoteBenchmarks comparing the two versions?

It's the same startup.
For example this is a regular AspNet app :
// This method gets called by the runtime. Use this method to add services to the container.
// For more information on how to configure your application, visit https://go.microsoft.com/fwlink/?LinkID=398940
public void ConfigureServices(IServiceCollection services)
{
    services.AddLogging(loggingBuilder =>
    {
        loggingBuilder.AddSeq(Configuration.GetSection("Seq"));
    });
    services.AddGrpc(options =>
    {
        options.MaxReceiveMessageSize = null;
        options.EnableDetailedErrors = true;
    });
    services.AddSingleton(sp => new ActorSystem().WithServiceProvider(sp));
    services.AddRemote((sp) =>
    {
        RemoteConfig remoteConfig;
        if (HostingEnvironment.IsDevelopment())
            remoteConfig = RemoteConfig.BindToLocalhost(80);
        else
            remoteConfig = RemoteConfig.BinToAllInterfaces(Environment.MachineName, 80);
        remoteConfig = remoteConfig.WithProtoMessages(Messages.ProtosReflection.Descriptor);
        remoteConfig = remoteConfig.WithRemoteKind("HelloActor", Props.FromProducer(() => ActivatorUtilities.GetServiceOrCreateInstance<HelloActor>(sp)));
        return remoteConfig;
    });
    ConsulProvider clusterProvider = new ConsulProvider(new ConsulProviderConfig(), c =>
    {
        c.Address = new Uri($"http://{Configuration.GetValue("ConsulHostname", "<IP_ADDRESS>")}:8500");
    });
    services.AddClustering((remoteConfig, sp) =>
        ClusterConfig.Setup("StabilityTestAsp", clusterProvider, new PartitionIdentityLookup(), remoteConfig)
        .WithClusterKind<HelloActor>(sp, "HelloActor")
        .RegisterHelloGrainKind(sp)
    );
    // services.AddGrains<Grains>((grains, sp) => grains.RegisterHelloGrainFactory(() => sp.GetRequiredService<IHelloGrain>()));
    services.AddTransient<IHelloGrain, HelloGrain>();
}

// This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
public void Configure(IApplicationBuilder app, IWebHostEnvironment env, ILoggerFactory loggerFactory)
{
    Log.SetLoggerFactory(loggerFactory);
    if (env.IsDevelopment())
    {
        app.UseDeveloperExceptionPage();
    }

    app.UseRouting();

    app.UseEndpoints(endpoints =>
        {
            endpoints.MapGet("/",
                async context =>
                {
                    await context.Response.WriteAsync(
                        "Communication with gRPC endpoints must be made through a gRPC client. To learn how to create a client, visit: https://go.microsoft.com/fwlink/?linkid=2086909"
                    );
                }
            );
        }
    );
    app.UseProtoRemote();
}

Some thoughts.
The effort to make this backwards compatible is impressive, I do think there is an inconsistency compared to how other parts of PA works.
e.g. Persistence has providers. Cluster has providers.
One could argue that the IRemote also are providers, but this is my current gut feeling, that maybe the Remote object should be the same, and injected with e.g a TransportProvider or something like that.

One thing I am missing here is a test suite. Ideally we can abstract out a common spec which both providers can be run against.
Have a look at the way it is done for both cluster and identity, where you can run the same tests against multiple implementations. A separate fixture can be provided for each Remote type.


Is there any update on profiler's TensorCores eligible rules?

@yisitu
I find the rules code to judge whether a kernel uses Tensor Cores is 1~2 years old. I'm not sure whether there should be new updates, along with latest hardward chips or library updates.
BTW, how these rules come from? Is there any doc that lists all kernels which uses Tensor Cores?
https://github.com/tensorflow/tensorflow/blob/5dcfc51118817f27fad5246812d83e5dccdc5f72/tensorflow/core/profiler/utils/kernel_stats_utils.cc#L95-L119

@trisolaran  these rules need to be updated for Ampere and beyond.

@yisitu Thanks! But how do you get the rules? Maybe I could contribute if I know the way to get them.

Hi Gao,
It is a heuristic and obtained mainly through observation, empirical data, and research. I agree that it would be great if NVIDIA documentation lists all kernels which uses Tensor Cores. Alternatively, you could read the cuDNN documentation to see the prereqs for using TensorCores and observe what kernels are launched - but this requires significant amount of effort.
Situ

Hi yisitu,
Thanks a lot for response!
Do you mean trying to run a lot of classic models on different NVIDIA archtectures(such as Turing and Volta), and using nvprof and tensor_precision_fu_utilization metric to check each kernel, recording every one with this metric valid? This way is also time consuming and can't cover full kernel set.
Yes, I don't find any list provided by NVIDIA. The document is not detail enough to kernel level.

It is not so much about the models as it is about the resulting cuDNN kernels for each GPU architecture. You can simplify this a little more to cover just the cuDNN TensorCore use cases.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


WFLY-17355 Upgrade Infinispan to 14.0.3.Final

https://issues.redhat.com/browse/WFLY-17355

Thanks @pferraro

@pferraro Looks good; thanks for the cleanup.


Formula style test fails in "brew audit --new-formula" command

The brew audit formula style checker fails with the commit<PHONE_NUMBER>a18f87f538f188417fcd954a0deac2 as below.
$ brew audit --new-formula Formula/box.rb
box:
  * C: 2: col 60: Description shouldn't end with a full stop
  * C: 6: col 3: `version` (line 6) should be put before `sha256` (line 5)
  * C: 13: col 3: Use 2 (not 4) spaces for indentation.
  * C: 17: col 3: Use 2 (not 4) spaces for indentation.
  * Stable: version 3.0.0 is redundant with version scanned from URL
Error: 5 problems in 1 formula detected
$ tree
.
├── Formula
│   └── box.rb
├── LICENSE
└── README.md

1 directory, 3 files
$ brew --version
Homebrew 2.1.1
Homebrew/homebrew-core (git revision 65a4; last commit 2019-05-01)
Homebrew/homebrew-cask (git revision 6f43f; last commit 2019-05-02)

https://github.com/KEINOS/homebrew-box/blob/6022112866a18f87f538f188417fcd954a0deac2/Formula/box.rb#L1-L19

--fix option should fix most of the problem.
$ brew audit --fix --new-formula Formula/box.rb
box:
  * C: 2: col 60: [Corrected] Description shouldn't end with a full stop
  * C: 6: col 3: [Corrected] `version` (line 6) should be put before `sha256` (line 5)
  * C: 13: col 3: [Corrected] Use 2 (not 4) spaces for indentation.
  * C: 17: col 3: [Corrected] Use 2 (not 4) spaces for indentation.
  * Stable: version 3.0.0 is redundant with version scanned from URL
Error: 5 problems in 1 formula detected, 4 problems corrected

--- a/Formula/box.rb
+++ b/Formula/box.rb
@@ -1,19 +1,19 @@
 class Box < Formula
-  desc "📦🚀 Fast, zero config application bundler with PHARs."
+  desc "📦🚀 Fast, zero config application bundler with PHARs"
   homepage "https://github.com/humbug/box"
   url "https://github.com/humbug/box/releases/download/3.0.0/box.phar"
-  sha256 "397f012cae0e7124cc2f949daa45cef594bfe2784c700fe33724dea43ce89777"
   version "3.0.0"
+  sha256 "397f012cae0e7124cc2f949daa45cef594bfe2784c700fe33724dea43ce89777"

   bottle :unneeded

   depends_on "php" if MacOS.version <= :el_capitan

   def install
-      bin.install "box.phar" => "box"
+    bin.install "box.phar" => "box"
   end

   test do
-      shell_output("#{bin}/box --version").include?(version)
+    shell_output("#{bin}/box --version").include?(version)
   end
 end

$ brew audit --new-formula Formula/box.rb
box:
  * Stable: version 3.0.0 is redundant with version scanned from URL
Error: 1 problem in 1 formula detected

This can be fixed by deleting the version which can be notice by the URL.
-    version "3.0.0"


Vtk file can't be read if there is only mesh points (no associate data)

I have a structured mesh as a vtk file that contains only mesh points and there is no associated data in that file besides the nodes. Using meshio.read('mesh.vtk') gives the following error that ties to line 123 in vtk_io.py because it seeks GetData() in a file that does not have any:
# Explicitly extract points, cells, point data, field data
    points = numpy.copy(numpy_support.vtk_to_numpy(
            vtk_mesh.GetPoints().GetData()
            ))

Any suggestion? Could you include the capability to read just the mesh file? I was interested in converting vtk mesh to gmsh...so, if you think there will be further problems during conversion because of missing associated data then I would be interested to know. Thanks.

All of meshio's data types revolve around unstructured meshes, so no wonder this doesn't work.

Could you include the capability to read just the mesh file?

I guess it would be possible to get something to work, but that part would be completely disconnected from everything else in meshio since there is no other structured mesh type I know of. Given that, I guess a reader could be another standalone program. Your best bet for now would be to take some inspiration from vtk_io.py in meshio and write it yourself.


[wip] Rework Miniforge inputs

follow-on to #133
addresses concerns from https://github.com/conda-incubator/setup-miniconda/pull/137#issuecomment-752682346
[x] stop doing fragile github actions stuff
[x] make miniforge-version the indicator for using download-miniforge
[x] make Miniforge3 the default miniforge-variant
[x] update docs
[x] update example 10 test

Is this ready for review @bollwyvl ?


add code of conduct

I want to add code of conduct.Please assign this issue to me

I too wanted to add code of conduct, if needed you can assign me this issue!


feat: Artwork grid save icon

This PR resolves CX-3138
Description
Artwork grid save icon.
PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.
[ ] I have prefixed changes that need to be tested during a release QA with [NEEDS EXTERNAL QA] on the changelog.

To the reviewers 👀

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

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Closing this PR in favor of https://github.com/artsy/eigen/pull/7708. Sorry I couldn't for some reason pull up your branch

@MounirDhahri closing this since we never closed it :D


Feature request: zero-downtime upgrade

Hi, thanks for the tool - it comes really handy and works great out of the box with minimal setup required, exactly how it should.
But unfortunately, it doesn't provide zero-downtime upgrade feature (--start-before-stopped is a bit different). For me it seems it can be easily done with this tool, following this strategy for example:

stack must have service alias "SERVICENAME-alias" for the service to be upgraded
when upgrade is requested, add new service like "SERVICENAME-upgraded"
add the "SERVICENAME-upgraded" to "SERVICENAME-alias" links
perform some kind of check against "SERVICENAME-upgraded" to ensure it have started (this seems tricky since different services require different checks)
remove original SERVICENAME from SERVICENAME-alias
rename SERVICENAME-upgraded to just SERVICENAME

I agree this would be a nice feature but it should be implemented in Rancher itself and exposed through the API. I don't think the scope of this tool should be expanded to support operations that Rancher itself doesn't not supply.
Perhaps file an issue on their Github repo requesting this feature.
Thanks!


error compiling on OSX with Opencv 3.4.3

Describe the bug
Error compiling bsglibrary on OSX 10.11.6 with opencv 3.4.3.
/.../bgslibrary-master/FrameProcessor.h:74:5: error: unknown type name 'GMG'; did you mean
'GMM'?
GMG* gmg;
^~~
GMM
To Reproduce
do it again..
Expected behavior
no errors
System information
Desktop (please complete the following information):

OSX 10.11.6 El Capitan:
C++, XCode

Additional context
Previous compilation with Opencv 3.4.1 had no problems.

I got the same error with OpenCV 3.4.3:
==============================
error: ‘GMG’ does not name a type
GMG gmg;*
Did you solve that?

I did a little try but ran in more errors. So for me it's up to the programmers to do some homework.

Thank you @pauloord123 @jesuinovieira for reporting this issue. I am looking at it.

It's interesting to note that I'm using Linux, but the same error occurred.

Hi @pauloord123 @jesuinovieira can you test my last commit? I fixed the macro preprocessor for the GMG algorithm in https://github.com/andrewssobral/bgslibrary/commit/f65078c621dfa70258b513a28eb8810b714ec07b.


Plan for Spin Wasm interface migrations

We know we will be migrating from the pre-component wit-bindgen ABI to the component canonical ABI "soonish". We expect to later make various breaking changes to trigger and host component interfaces, especially as new Wasm features stabilize (resources, async, streams, future WASI versions, etc).

Initial thoughts:

For the ABI change, support both ABIs for "a while", issuing deprecation warnings in Spin for a release or two before removing the old ABI. If we can figure out a reasonable way to wrap the new ABI in an adapter component, great.
For future interface changes, write an adapter component for each breaking change and have Spin (spin build? up?) wrap old components with those as needed. If we can have a nice generic mechanism for this it could make updates relatively painless.

For future interface changes, I was wondering about the practicality of versioning the interfaces e.g. redis-1.0, redis-2.0, etc., which would provide some explicitness and provide for meaningful detection and diagnostics.  I'm particularly thinking about Wasm components built against old interfaces and deployed to Cloud: we'd like them to get something like "Fermyon Cloud no longer supports foo-1.0, please rebuild against..." rather than "link error, required import bar not found".  (Needless to say, any implementation would indeed lean heavily on adapters - this is more about how we detect and diagnose a mismatch.)
Looking at Luke's Wasm Day presentation, though, he doesn't show any versioning of interfaces there.  But I feel like issuing new WITs with the same name but different members is going to lead to confusion long term.  Is versioning being covered in the component model discussions, and if so what is the thinking?

Is versioning being covered in the component model discussions, and if so what is the thinking?

I don't know that it has been formalized anywhere but my understanding is that breaking changes would at least conventionally involve a change to the interface name. @fibonacci1729 @dicej

I've been looking at how best to support the "old" ABI. The least-bad option at this point seems to be to fork wit-bindgen at tag v0.2.0 and update wasmtime there. Assuming that works out in the short term I think we can stick with that strategy until/unless it becomes a maintenance burden at which point we can reevaluate.

BTW, we discussed component interface versioning recently with Luke and others, and there was consensus that we'll want more than just a naming convention, and that each interface the component imports or exports should be explicitly versioned.  The details of how that is represented in the tooling (e.g. inside the WIT files themselves vs. in metadata elsewhere) are yet to be determined, however.

WIT packages now include a semver version, which will help here.

This is mostly irrelevant now due to spin-componentize.


Zip headers not found. Probably not a zip file at ...

I'm using zip4j-2.5.1 on Android (minSdkVersion16). I have an encrypted zip file that needs to unzip, but I got this error from some(about 10%) of my users!
Zip headers not found. Probably not a zip file at ...

Can you share the zip file? Without a zip file in this case, it will be hard to say what might have happened.

It's containing a small SQLite database(about 10 MB). Usually, I use WinRAR to make that zip file with password & default options.
It's not possible to share the zip file.

Usually, I use WinRAR to make that zip file with password & default options.

Could it be that it was a rar file by mistake? As mentioned earlier, without the zip file, it will be hard to say for me. All I can say for now is that, as the error message says, zip4j was unable to find the relevant zip headers in the file. For now, my conclusion would be that it most likely is not a zip file.

No, It's a zip file I saved with .mp3 format! Decryption works fine on my devices, AVDs and on the device of most of my users.

What do you mean by a zip file with .mp3 format?

Closing this issue as there is no further feedback.


Poll all announce URLs in torrent

For now announce polling stops on first working url found. This is no good because "working" doesn't means "have enough peers". First working tracker may return 1 slow peer, but other ones having hundreds of fast peers will be never accessed. So we need an option to poll all trackers in torrent, regardless of returned peers and tracker status.

I'm currently improving the tracker list view and will make sure this gets added.

This still isn't implemented; exposing libtorrent's lt::settings_pack::announce_to_all_trackers in the session settings should work? In addition, active_tracker_limit should probably be wired up to the UI, as it's currently only accessible by manually modifying PicoTorrent.sqlite.
https://github.com/picotorrent/picotorrent/blob/3ffaa83ea48fd8fdbd584e2f00bdccb001c518a9/src/picotorrent/session.cpp#L126-L133

Hi! There's a big rewrite in the works, and there's a lot of settings which will be exposed in the preferences dialog. These two among them.
I'll reopen this to keep track of it,


Add test context utility to uv-interpreter

Follow-up to https://github.com/astral-sh/uv/pull/3797 to clean up the test isolation in uv-interpreter.
I still want to expose a CLI at some point like uv python <...> for discovery and test from there, hopefully this will make that transition simpler.

The diff is basically unreadable, I'd recommend just giving the new file a brief read.


Minor alsa backend tweaks

Silence a clippy warning and we don't need std::cmp::min when usize already has min.

Thanks mate


Efficiently batch insert to indexed tree. Insertions should handle arbitrary sizes

Much the same as task #383 we need to efficiently batch insert items into the indexed tree. We should be able to efficiently insert batches of any size. Some work in this area has already been done as part of building a rollup.

I will start working on this once https://github.com/AztecProtocol/aztec-packages/pull/951 is reviewed because the insertion algorithm will most likely be similar.


Fix regression with lazy properties and add another test case [4.0]

Description: When type checking an expression without applying the solution, we would set the types of anonymous closure parameters to ErrorType. This resulted in SILGen crashes because of un-typechecked AST when a lazy property was initialized by an expression containing a closure with anonymous parameters.

Origination: Regression from another set of fixes to 'lazy' in 4.0.

Tested: New test added. Also add another test for an odd behavior with 'lazy' that worked in 3.1 but we don't want to allow anymore. This second part is not really necessary for 4.0 branch, but it's just a new test.

Reviewed by: Pavel Yaskevich

Radar: rdar://problem/33219081

@swift-ci Please test

This is failing in ASan unfortunately. I'll have a corrected version soon.

@swift-ci Please test

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 9365ada474d9aee113936ad4fb42dd2b74c2f5f3
Test requested by - @slavapestov

@swift-ci Please test

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - a12b3f83b12b2c106ad61f4af7a1cdd0fcd0ca95
Test requested by - @slavapestov


[CARBONDATA-2688][CarbonStore] Support SQL in REST API

Support SQL interface in Horizon service.
Support REST client for SQL
This PR is based on #2473

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5778/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7020/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5799/

Merge to carbonstore branch

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7032/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5772/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5808/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5778/


inform where we can find tesseract.exe

Your Feature Request
Could you please write in the github where we can find the .exe to use it from commande line. I have downloaded the latest version (.zip file) but i cannot find it

What about reading documentation? Or Readme?

https://tesseract-ocr.github.io/tessdoc/Installation.html


Learn getSwitchState: undefined

When I try to learn a code I get the above message? This is a new installation and the 'Learn' light is activated but always the undefined response?
2018-06-13T10:07:04.176376347Z [2018-6-13 10:07:04] [Broadlink RM] Learn Code (ready)
2018-06-13T10:07:04.206609113Z [2018-6-13 10:07:04] [Broadlink RM] Learn getSwitchState: undefined
2018-06-13T10:07:14.180243582Z [2018-6-13 10:07:14] [Broadlink RM] [INFO] Learn Code (stopped - 10s timeout)
2018-06-13T10:07:14.186526135Z [2018-6-13 10:07:14] [Broadlink RM] [INFO] Learn Code (stopped)

I think I have resolved this by getting closer to the RMPro to record the hex code. I now have on/off codes for a device and they do work but it takes ages to send the thousand or so characters?
The codes recorded are from Trust Smart RF switches.

I have the same problem but on RM Mini3, same message and no IR code learning is possible. (RM mini3 doesn't support RF)

Hey @Cappe77, please re-open if you didn't manage to figure this out.


Break Navigation Overview Section into smaller parts

I think it's fine for now, although we should probably break the Overview section into smaller parts. For example: Become a maintainer and Running Integration Tests are Dex development related topics.

Originally posted by @sagikazarmark in https://github.com/dexidp/website/pull/59#issuecomment-716044707

This is fixed after migrating to docsy.


Where should reference results be stored?

Reported by beutlich on 30 Apr 2014 10:14 UTC
I'd like to move the reference results of the ModelicaTest.Tables test models from the obsolete dev branch to ModelicaTest. Where should they belong to?
From the Buildings library I observed that reference results of all example models are located directly under the Resources/ReferenceResults/ directory.
I propose to add the reference results to ModelicaTest/Resources/ReferenceResults/ in a similar way. Please note that there already is ModelicaTest/Resources/Scripts/Dymola.
One other idea is to use an addtional subdirectory, i.e. ModelicaTest/Resources/Tables/ReferenceResults/.
The names will either be of format ModelicaTest.Tables.CombiTable1D.Test1.csv or with underscores ModelicaTest_Tables_CombiTable1D_Test1.csv (like the Buildings reference results). I have no strong opinion on the names but prefer the names with dots.
This discussion ticket is open for two weeks and expires 14th of May 2014. If there is no discussion at all I consider this as an agreement and will perform the above proposed changes.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1472

Comment by sjoelund.se on 30 Apr 2014 10:20 UTC
#1392 is related

Comment by beutlich on 22 May 2014 11:03 UTC
Resolved by d8b4e1a616bf307dfd80872dcab1fd1ae4479dab.

Comment by sjoelund.se on 22 May 2014 13:56 UTC
I still think these were added in the wrong place. #1392 says the results should be stored:

Hierarchical
Not in the library svn but in a separate path (only for reference results)
Not tool-specific libraries (the library officer chooses the reference tool)

Comment by beutlich on 17 Jun 2014 16:53 UTC
Leo, as we've just discussed, please verify if these results can be reproduced with the tools used for regression testing and delete it again from public svn (i.e. revert d8b4e1a616bf307dfd80872dcab1fd1ae4479dab) once your results are available online.

Comment by beutlich on 11 Aug 2014 12:44 UTC
No reply from Leo :-(
Anyway, reverted by f67ca03eb6535d19070e5df35312de7a1cef6c5e.

Modified by beutlich on 13 Oct 2015 20:51 UTC


Add a build script for w3m, a pager with WWW capabilities

The w3m tool is particularly useful when rendering HTML pages into plain
text, as Git for Windows needs to do with its release notes.
Signed-off-by: Johannes Schindelin<EMAIL_ADDRESS>

Hold on, I get a funny error linking the setmode function when building this in one of my VMs. Will investigate.

Okay, fixed.

Ping?

Forgot about this.. Life got hectic .. again. Alexey is busy with work until early next year, so packaging will have to wait until then. We need to get some infrastructure sorted out. I'm going to try to arrange some in the new year. I promise.

Alexey is busy with work until early next year, so packaging will have to wait until then.

Thanks for the clarification!


Update Herbarium 2022

Hello,
I added the current year's competition page to the repo.

@kiatchuan, thank you for reviewing. What else process is needed to merge this pull request to visipedia?

Apologies, forgot to merge it. It should be merged now.

Apologies, forgot to merge it. It should be merged now.

Thank you, Kiat!


⚠ Bofrost.be has degraded performance

In 07feb32, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1496 ms

Resolved: Bofrost.be performance has improved in 1bc3e8a.


create system users

Let Airlock create new ceph users as system users

Codecov Report

Merging #20 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
+ Coverage   86.02%   86.04%   +0.02%
==========================================
  Files          28       28
  Lines         558      559       +1
  Branches       50       51       +1
==========================================
+ Hits          480      481       +1
  Misses         78       78

Impacted Files
Coverage Δ

...ck/proxy/handler/radosgw/RadosGatewayHandler.scala
94.28% <100%> (+0.16%)
:arrow_up:

Continue to review full report at Codecov.

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


New Enhancement: add gpt 4o mini from open ai

What features would you like to see added?
There is a new model which is named gpt 4o which was released by open ai.can you add this yo librechat
More details
As gpt 4o is priced the lowest of all ai models on top it's good to use.
Which components are impacted by your request?
Other
Pictures
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Already implemented https://github.com/danny-avila/LibreChat/pull/3384


Unable to find a specification for `ChainPageCollectionView'

I am trying to install ChainPageCollectionView using pod and am getting this error.

pod repo update in terminal
solved it, sorry for opening an issue.

no problem😀


Can provide a way to achieve loop

as title

circular option is intended for the same purpose. Set it to true to start from top again and false if you want to start list in reverse when completed.
Let me know if it serve the purpose.

I know circular , what i mean is the seamless loop，not back to the top. Such as when the first loop end, copy the loop data and push it.I have tried to modify your code but failed.

Now I understand, but currently it is not possible. I will add this in next version.

Any update on loop? Thanks
@mubaidr

I do plant to push this feature over the coming days. Stay tuned!

Great!
Can you tell me what's your thoughts? How to copy the loop data and push it?
Many thanks

This should work:

Use computed list instead of directly rendering prop words
Add new option continuous to component as prop
If the new option (continuous) is true then on completion (final item), slice all the items from the list and insert into array at the end. https://github.com/mubaidr/vue-swimlane/blob/master/src/component/vue-swimlane.vue#L155

You are most welcome to try and implement this, It would be appreciated. 👍

I probably know how to do it.But I have too much work to do this month , I will try after the work is done.

I have cleaned and refactor the plugin code to improve readability and major layout refinements.
Continuous option can be implemented here: https://github.com/mubaidr/vue-swimlane/blob/master/src/component/vue-swimlane.vue#L156

Continuous option is implemented but the animation breaks for this option. Seems like have to opt for Vue transitions for this.


Moving servicequotas (1) resource to v1beta1 version

Description of your changes
Adding new resources from group servicequotas:

[x] aws_servicequotas_service_quota

Fixes #323
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
aws_servicequotas_service_quota - manually and

/test-examples="examples/servicequotas/servicequota.yaml"

/test-examples="examples/servicequotas/servicequota.yaml"


MAJOR ISSUES!

Describe your bug here.
So there's this bug where it just tells me that "Std.is is deprecated" and something with my VC version. Can anyone help?
Command Prompt/Terminal logs (if existing)
C:/HaxeToolkit/haxe/lib/hscript/2,4,0/hscript/Interp.hx:145: characters 8-18 : Warning : Std.is is deprecated. Use Std.isOfType instead.
C:/HaxeToolkit/haxe/lib/hscript/2,4,0/hscript/Interp.hx:180: characters 8-18 : Warning : Std.is is deprecated. Use Std.isOfType instead.
C:/HaxeToolkit/haxe/lib/hscript/2,4,0/hscript/Interp.hx:221: characters 8-18 : Warning : Std.is is deprecated. Use Std.isOfType instead.
C:/HaxeToolkit/haxe/lib/hscript/2,4,0/hscript/Interp.hx:515: characters 8-18 : Warning : Std.is is deprecated. Use Std.isOfType instead.
C:/HaxeToolkit/haxe/lib/hscript/2,4,0/hscript/Interp.hx:483: characters 37-56 : Warning : Std.is is deprecated. Use Std.isOfType instead.
C:/HaxeToolkit/haxe/lib/hscript/2,4,0/hscript/Interp.hx:484: characters 31-47 : Warning : Std.is is deprecated. Use Std.isOfType instead.
C:/HaxeToolkit/haxe/lib/hscript/2,4,0/hscript/Interp.hx:646: characters 10-42 : Warning : Std.is is deprecated. Use Std.isOfType instead.
source/options/NoteOffsetState.hx:57: characters 3-39 : Warning : `FlxCamera.defaultCameras` is deprecated, use `FlxG.cameras.setDefaultDrawTarget` instead
C:\Users\TKEFNFAR420\Desktop\Friday Night Funkin\Base Game (Psych Engine)\Source Code\FNF-PsychEngine-main\export\release\windows\obj>setlocal enabledelayedexpansion
Error: 64bit is not automatically supported for this version of VC. Set HXCPP_MSVC_CUSTOM and manually configure the executable, library and include paths
Error: Error: 64bit is not automatically supported for this version of VC. Set HXCPP_MSVC_CUSTOM and manually configure the executable, library and include paths

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I did not edit anything.

It doesnt affect anything


Improve the onboarding in the readme.

Guide for setting the database up.
Guide for checking that the system is running.

@ryanbnl thank you!
@f11h can you have a look at the changes?

Thanks for the detailed description @ryanbnl !


[UTIL] Library-Inline-Snapshot

Library Functions (@emma.inline Annotation)
Work in progress, some things work, some things don't. Comments and hints are very much appreciated!

[x] Inlining of public methods on annotated objects (@emma.inline object Util { /*[...] */ })
[x] Inlining of annotated methods (object SomeUtil { @emma.inline def myFunc /* [...] */ })
[x] Overloading
[ ] Type parameters
[ ] Non public modifiers

Limitations

Cannot @emma.inline a top-level object (a limitation of macro annotations)

Next steps

More efficient inlining at runtime --> Non hack-y repair of Select(This(_), name) artifacts
Type parameters

Dependencies

Macro Paradise (due to Macro Annotations)

Alternatives

Perform the bulk of inlining at annotation / static time. Was not possible due to Tree-shenanigans involving multiple universes and weird compiler infinite loops while creating a Type.checked version to resolve overloading (working on that as well).

Is there any chance for this to work with decorators (@emma.inline implicit class Folds[A](self: DataBag[A]) extends AnyVal { ... })? Then we could define all folds as inlined methods.

In principe the macro annotation should be able to work with traits as well, but we need to rethink the implementation of the lifting and splicing of the method bodies.

This is still on my radar, we should try to pick it up and migrate it to the newir in the next weeks.
@joroKr21, @ParkL maybe you can collaborate on that.

I am picking up this and will merge it while changing the approach a bit.
WIP can be tracked here.

Will port this to #253.

Compared to this list of features, #253 will have
Library Functions (@emma.lib Annotation)
This is basically done and allows for polymorphic function abstraction with overloading (see the examples in #252).

[x] Inlining of apply methods on annotated objects (@emma.lib object plus { /*[...] */ })
[x] Overloading
[x] Type parameters
[ ] Non public modifiers

Limitations

Cannot @emma.lib a nested object (a limitation of macro annotations)

Next steps

[x] More efficient inlining at runtime --> Non hack-y repair of Select(This(_), name) artifacts
[x] Type parameters

Dependencies

Macro Paradise (due to Macro Annotations)

Alternatives

Perform the bulk of inlining at annotation / static time. Was not possible due to Tree-shenanigans involving multiple universes and weird compiler infinite loops while creating a Type.checked version to resolve overloading (working on that as well).

I managed to implement the alternative described by @ParkL in the lib macro by merely type-checking per DefDef rather than the enclosing, annotated ModuleDef.
This works, if the DefDef code does not refer to other members of the enclosing object.


How negative value in a registers are handled?

Could @bit-hack please explain how negative values in registers are handled?
For example say ADDI r5, r0, 100 and say r0 has value -100 (in 2's complement)
will following give correct answer ?
82    case 0: // ADDI    Aidan Dodds, 20/09/20 16:10 • tidy and fix SRAI bug
  1     rv->X[rd] = (int32_t)(rv->X[rs1]) + imm;
  2     break;

because X stores unsigned integer AFAIU then above seems not handled correctly.

Hi Vivek,
The code above is correct.
The array of uint32_t X[32] for the register file is just a container for 32bit data.  What's important here is that the operands are cast correctly prior to performing signed or unsigned operations.
In the example above, you can see that for the signed ADDI instruction, the value read from X[rs1] is cast to a signed integer, which means the operation is performing signed arithmetic.  Once the operation is computed, its converted back to an unsigned result and stored away.
The bit pattern for signed and unsigned is the same so this is perfectly correct and fairly basic behavior in C.
To get a better grasp of this, you could read up about casting and type promotion.
Remember, the key is not what type the data is stored as, but how its cast before the arithmetic is performed.

Don't we need to store negative numbers in 2' complement ?
Also I don't think what above you described is true for Rust.


Error on Google Chrome

When opening promised window on Chrome I got this error
A boolean is being passed as a fourth parameter to window.open. This is not used and may cause an exception in a future release.

Must be a recent change in the API, will fix in the next version. Thanks @ekabolotina!

Should be fixed on latest (v 4.1.0). Thanks for reporting it!


Does not correctly report Android 11

os.version is null for the following user agent string:
Mozilla/5.0 (Linux; Android 11; SAMSUNG SM-N986U) AppleWebKit/537.36 (KHTML, like Gecko) SamsungBrowser/14.2 Chrome/87.0.4280.141 Mobile Safari/537.36

This was fixed in v2.2.10. This issue was using v2.2.9.


About large depthwise conv2d kernel speed

您好，我做了一个关于您提供的大卷积核算子速度的实验。对比pytorch的Conv2d
    if torch.cuda.is_available():
        x = torch.randn(64, 1000, 32, 32).cuda()
        x1 = torch.randn(64, 1000, 32, 32).cuda()
        m1 = DepthWiseConv2dImplicitGEMM(1000, 31, bias=False).cuda()
        m2 = nn.Conv2d(1000, 1000, 31, padding=31// 2, bias=False, groups=1000).cuda()

        start = time.time()
        y2 = m2(x1)
        y2.mean().backward()
        end = time.time()

        start1 = time.time()
        y1 = m1(x)
        y1.mean().backward()
        end1 = time.time()

        print("time:",end-start,end1-start1)

结果：
time: 0.0003426074981689453 0.0003437995910644531
经过测试，他们前向和后向所花费的时间几乎是相同的。是pytorch做了相关优化吗？
pytorch 1.9.0  python3.9  cuda 10.2

Compare time with torch.cuda.synchronize()
By the way, did you have any success with the installation?
I am failing to install in colab environment.
So I'm not sure if this answer will solve your problem.

Thanks. I try your method and It works.
real time:
Conv2d 1.4116837978363037
DepthWiseConv2dImplicitGEMM 0.19309020042419434
For your question,actually I compiled with python 3.8.3  torch 1.8.0 cuda 10.2
It stuck with pytorch 1.9.0 , anyway, I dont know why.

solved
Compare time with torch.cuda.synchronize()
thanks to klae01


FILE_ID.DIZ plz?

No biggy and super glad to see Icy Term getting dev attention;
can you add a FILE_ID.DIZ release archives?
http://www.textfiles.com/artscene/information/faq-file_id.html
As long as its in the first / directory of the ZIP/TAR, bbSes could display detailed info about Icy, or whats in this release update, etc...

Added a file_id.diz to the 0.2.5 release thought that I'm a bit faster with 0.3.0 but it'll take longer than expected :/.


Introduction to Blazor

Fixes #10
Includes all updates from original aspnet/Docs PR: https://github.com/aspnet/Docs/pull/5791

ty @danroth27 Updates made. :+1:


3D rendering in the console

I love C
https://www.youtube.com/watch?v=L4_wtMmpEaE

Closing this one in lieu of the discussion at #696


Godot 4.0 beta 5 crashes when loading extension

Godot version
4.0 beta 5
Plugin version
v1.0.0-beta4 - Libdatachannel 0.17.10+git
System information
Pop!_OS 22.04 LTS
Issue description
I'm attempting to use the WebRTC GDNative Extension with the webrtc_signaling example with Godot 4.0 beta 5.
At first, I could open the project but the extension wouldn't load, citing that the object files included with the extension were missing from ~/.local/share/lib/. After copying the files included in godot-extension-webrtc/lib into that directory and closing the editor, I can no longer open the project at all. Instead, I get this error:

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.0.beta5.official (89a33d28f00fec579184fb7193790d40aa09b45b)
Dumping the backtrace. Please include this when reporting the bug on: https://github.com/godotengine/godot/issues
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x42520) [0x7f150ea42520] (??:0)
[2] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0x429d7e8] (??:0)
[3] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0x43e433b] (??:0)
[4] godot::String::init_bindings() (??:0)
[5] godot::Variant::init_bindings() (??:0)
[6] godot::GDExtensionBinding::init(GDNativeInterface const*, void*, GDNativeInitialization*) (??:0)
[7] /home/ben/.local/share/applications/../lib/libwebrtc_native.linux.template_debug.x86_64.so(webrtc_extension_init+0x5d) [0x7f1508be4e3d] (??:0)
[8] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0x42b69b1] (??:0)
[9] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0x431a436] (??:0)
[10] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0x402f792] (??:0)
[11] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0x4042599] (??:0)
[12] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0x408f11f] (??:0)
[13] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0x4318146] (??:0)
[14] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0xd9dd29] (??:0)
[15] /lib/x86_64-linux-gnu/libc.so.6(+0x29d90) [0x7f150ea29d90] (??:0)
[16] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0x80) [0x7f150ea29e40] (??:0)
[17] /home/ben/.local/share/applications/Godot_v4.0-beta5_linux.x86_64() [0xdbe18e] (??:0)
-- END OF BACKTRACE --
================================================================
[1]    30336 IOT instruction (core dumped)  ~/.local/share/applications/Godot_v4.0-beta5_linux.x86_64 project.godot

(I've also made an issue on the main Godot repo because I wasn't totally sure where the problem might be)
Steps to reproduce

Unzip the provided project files
Copy contents of godot-extension-webrtc/lib to ~/.local/share/lib
Attempt to open project with Godot 4.0 beta 5

Minimal reproduction project
webrtc_signaling_w_extension.zip

v1.0.0-beta4 is compatible with Godot 4.0-beta4.
There hasn't been a release compatible with beta5 yet.

Released 1.0.0-beta5


Failed to login to Garmin

It seems that Garmin Connect has changed something regarding the logging mechanism, so the first step keep failing.

Fixed in release 0.2.1


Document accessing dropbox instance

Detailing accessing the dropzone instance so any event listener can be added

Hey @silverbackdan,
Thanks for contributing one more approach for listening to events!  :tada:


Photographers should be credited

Site should include credit for photographers.

Photos taken from pexels.com, which provides guidance on crediting.

Yes, when downloading images a suggested credit is provided.  Adding these to the front-matter using the tag image-credit will ensure the attribute appears in the generated set.


New version: DataAssimilationBenchmarks v0.2.1

Registering package: DataAssimilationBenchmarks
Repository: https://github.com/cgrudz/DataAssimilationBenchmarks.jl
Created by: @cgrudz
Version: v0.2.1
Commit: 8fee05c78c6ba73b51a6f98aff469b516adcabdf
Git reference: HEAD

Please simply remove this pull request, this was done in error trying to troubleshoot the issue with registrator.


Add an automation test for the log out process in Wallet

As per @rolsonquadras, we should create a new spec file describing the log out workflow in https://github.com/trustbloc/sandbox/tree/main/test/ui-automation/test/specs.

This has been implemented


If the news and news publisher feature in dataset

The content feature is composed of a 300-dimensional user comment feature and a 10-dimensional user profile feature. But it seems that in the code, the first feature of each graph is the news (the root). So is that means the first content feature of each graph is the feature of news and the profile of publisher , not the feature of user comment and user profile?
Also, is the first spacy feature of each graph is the recent 200 news of the publisher?

So is that means the first content feature of each graph is the feature of news and the profile of publisher, not the feature of user comment and user profile?

Yes, the first feature in each graph represents the news encoding plus the profile feature of the Twitter account who first tweets the news.

Also, is the first spacy feature of each graph is the recent 200 news of the publisher?

The news content feature is only encoded from the current news content, it does not include the historical news since we didn't know the news publisher info.


TS2742: The inferred type of 'clickListener' cannot be named without a reference... This is likely not portable. A type annotation is necessary.

» bazel build //packages/routing
INFO: Invocation ID: bf0b4d82-c91a-4854-ad1b-3942c28e7c89
INFO: Analyzed target //packages/routing:routing (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
INFO: Writing explanation of rebuilds to 'bazel-explain.txt'
ERROR: /home/dzintars/code/github.com/dzintars/prime/src/packages/routing/BUILD.bazel:5:11: Compiling TypeScript (devmode) //packages/routing:routing failed (Exit 1): tsc_wrapped.sh failed: error executing command
  (cd /home/dzintars/.cache/bazel/_bazel_dzintars/372a4c8711393e1e5266b713c6d61d3c/execroot/oswee && \
  exec env - \
    PATH=/home/dzintars/.cache/bazelisk/downloads/bazelbuild/bazel-3.7.0-linux-x86_64/bin:/home/dzintars/.yarn/bin:/home/dzintars/.config/yarn/global/node_modules/.bin:/home/dzintars/.yarn/bin:/home/dzintars/.config/yarn/global/node_modules/.bin:/usr/local/bin:/usr/local/sbin:/usr/bin:/usr/sbin:/home/dzintars/bin:/usr/local/go/bin:/usr/local/go/bin:/usr/local/go:/home/dzintars/go:/home/dzintars/go/bin:/home/dzintars/bin:/usr/local/go/bin:/usr/local/go:/home/dzintars/go:/home/dzintars/go/bin \
<EMAIL_ADDRESS>@@bazel-out/k8-fastbuild/bin/packages/routing/routing_es5_tsconfig.json)
Execution platform: @local_config_platform//:host
packages/routing/src/lib/sagas.ts:85:11 - error TS2742: The inferred type of 'clickListener' cannot be named without a reference to '../../../../external/npm/node_modules/redux-saga'. This is likely not portable. A type annotation is necessary.

85 function* clickListener() {
             ~~~~~~~~~~~~~

Target //packages/routing:routing failed to build
INFO: Elapsed time: 0.533s, Critical Path: 0.34s
INFO: 2 processes: 2 internal.
FAILED: Build did NOT complete successfully

Just returned any type for now. Have no clue how to type it properly.
function clickChannel(): any {

Later will disable any and will fix all of them.


Create .gitkeep

This PR was raised by mistake. I am ready with the recipe and had checked in, raised a different PR as well. If possible this PR can be deleted. Thanks.

Okay. Closing.


Is hot reload for user and password creds available?

Greetings!
We have a task to hot reload creds to etcd (they can be changed from vault). Is it allowed in your library? How to process situations where watches on key are already set and currently listening? Should I recreate client every time creds change? How would you do this?
Thank you.

@njhill help please


(WIP) Added support for AWS ECR

Hi, wanted to get feedback on a more elegant way of integrating ECR, if that is something that would interest the Hephy team.
So far as I'm aware the authorization token is not guaranteed to stay static so I decided to modify the controller to pull the auth details if the registry username is aws-ecr.
Unsure of how up to date the contributing documentation is so please let me know if I've missed something glaring.
TODO

[ ] Make region configurable.

Hi @mrferos , thank you for the PR!
Could you reformat the commit messages to form to our style? We use a tool that takes them and creates changelogs when making releases and that's why this is necessary.
https://docs.teamhephy.com/contributing/submitting-a-pull-request/#commit-style

@Cryptophobia I totally missed that in the documentation, the pre-existing support for ECR. I can close this PR and just depend on the existing solution?

@Cryptophobia I totally missed that in the documentation, the pre-existing support for ECR. I can close this PR and just depend on the existing solution?

Yes, I hope the documentation helps. Let us know if you are having any issues and you can always reach us on slack with any questions slack.teamhephy.com . :+1:


Appstudio update test-component-pac-igoh

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-igoh-on-pull-request has successfully validated your commit.


Update to gaffer 0.7.5

Merged into develop.

See Gaffer 0.7.5 release notes: https://github.com/gchq/Gaffer/releases/tag/gaffer2-0.7.5