Cannot open include file 'aspnetcore_msg.h'

PR #7706 is seeing this unrelated failure that seems to be coming from the IIS native code
e:\a\_work\1\s\src\servers\iis\aspnetcoremodulev2\commonlib\resources.h(6): fatal error C1083: Cannot open include file: 'aspnetcore_msg.h': No such file or directory (compiling source file GlobalVersionTests.cpp) [E:\A\_work\1\s\src\Servers\IIS\AspNetCoreModuleV2\CommonLibTests\CommonLibTests.vcxproj]

cc @muratg @Tratcher @jkotalik

That's an autogenerated file. I think it's an ordering issue with building CommonLibTests. It should always be built after the commonlib folder.

I presume all this means you have an idea of how to fix it? 😝

I think two builds of the same project are generating this file causing a conflict. We need to see if it can be move into a platform specific subdirectory. I can take a look.

https://github.com/aspnet/AspNetCore/blob/32aebef4d13e0d5fbe0af4a1453b1283627766fc/src/Servers/IIS/AspNetCoreModuleV2/CommonLib/CommonLib.vcxproj#L276


DataRoleHelper

I really like this new dev experience!
Is DataRoleHelper or an equivalent module available?

Awesome!!
Do you mean a section in the documentation where it talks about dataroles? If so, it's here (i also added it in the table of contents).
https://github.com/Microsoft/PowerBI-visuals-docs/blob/master/Capabilities/Capabilities.md#define-the-data-fields-your-visual-expects---dataroles

Related, but not quite what is in the documentation.
In the old code, we could define capabilities and then import DataRoleHelper like so.
import DataRoleHelper = powerbi.data.DataRoleHelper;
Later on, we could get the index of roles regardless of which order they were dragged into the field well. For example, if multiple measures were defined for Latitude, Longitude, Measure, and Description; we could easily get the index for each in the following manner:
lngIndex = DataRoleHelper.getMeasureIndexOfRole(dataViews[0].categorical.values.grouped(), "Longitude");    latIndex = DataRoleHelper.getMeasureIndexOfRole(dataViews[0].categorical.values.grouped(), "Latitude");    measureIndex = DataRoleHelper.getMeasureIndexOfRole(dataViews[0].categorical.values.grouped(), "Measure");    descriptionIndex = DataRoleHelper.getMeasureIndexOfRole(dataViews[0].categorical.values.grouped(), "Description");

Correct, that isn't available as yet. We will be adding more helper libraries to the mix soon that will allow you to read information out of the DataView in a systematic fashion.  I'll attach this issue to a milestone in the next couple of days so you'll be able to track when it get delivered.

Perfect, thanks

Hopefully we've answered your questions, let us know if you have more issues.

#21

completed


MinorVersionCompatibility fails in nightly workflow

Context
--> Running end to end tests
go run ./test/e2e
test-e2e2024/0[5](https://github.com/celestiaorg/celestia-app/actions/runs/8997023702/job/24714448567#step:5:6)/08 06:01:04 No particular test specified. Running all tests.
test-e2e2024/05/08 0[6](https://github.com/celestiaorg/celestia-app/actions/runs/8997023702/job/24714448567#step:5:7):01:04 go run ./test/e2e <test_name> to run a specific test
test-e2e2024/05/08 06:01:04 Valid tests are: MinorVersionCompatibility, MajorUpgradeToV2, E2ESimple

test-e2e2024/05/0[8](https://github.com/celestiaorg/celestia-app/actions/runs/8997023702/job/24714448567#step:5:9) 06:01:04 === RUN MinorVersionCompatibility
test-e2e2024/05/08 06:01:04 no versions to test
exit status 1
make: *** [Makefile:140: test-e2e] Error 1

https://github.com/celestiaorg/celestia-app/actions/runs/8997023702/job/24714448567
Problem
The nightly e2e test has been failing for a while.
Proposal

Fix it so it passes
Instead of running it nightly, run it on every PR and make it an optional check. Currently, no one notices when the nightly fails because we have no hooks set up to alert us of failures.

Fixed the issue where it wasn't able to fetch tags but now i get a port forwarding error in MinorVersionCompatibilitiy.
The test is working locally but the error in ci could be happening because the gh pipeline workers are very limited. looking into this with devops @smuu

Instead of running it nightly, run it on every PR and make it an optional check. Currently, no one notices when the nightly fails because we have no hooks set up to alert us of failures.

imo, this does not have to be tied to fixing this test, and instead could be a separate issue. That separate could optionally be tied to enabling alerts in slack after something fails the tests when merging to main.

not being able to fetch tags was the least of our worries. we're not able to start testnet at first because of port forwarding now different knuu issues are coming up. you can view the runs in ci on my pr. working on resolving this with @smuu


add interval to prometheus service monitor

We have a use-case where we need to modify the default scrape interval
on a number of apps, including coredns.
This PR simply adds support to change it. (It should be backwards
compatible with existing implmentations.)
Test/verification:
$ helm template charts/coredns --set prometheus.monitor.enabled=true --set prometheus.monitor.interval=60s |grep servicemonitor.yaml -A 22
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: RELEASE-NAME-coredns
  labels:
    app.kubernetes.io/managed-by: "Helm"
    app.kubernetes.io/instance: "RELEASE-NAME"
    helm.sh/chart: "coredns-1.16.5"
    k8s-app: "coredns"
    kubernetes.io/cluster-service: "true"
    kubernetes.io/name: "CoreDNS"
    app.kubernetes.io/name: coredns
spec:
  selector:
    matchLabels:
      app.kubernetes.io/instance: "RELEASE-NAME"
      k8s-app: "coredns"
      app.kubernetes.io/name: coredns
      app.kubernetes.io/component: metrics
  endpoints:
    - port: metrics
      interval: 60s

I followed the instructions for "signoff", but get a permission denied. (It seems odd that I would need to "signoff on my own PR". So, I'm guessing someone from the actual coredns team needs to signoff...)
How do I proceed?

I followed the instructions for "signoff", but get a permission denied. (It seems odd that I would need to "signoff on my own PR". So, I'm guessing someone from the actual coredns team needs to signoff...)
How do I proceed?

In a PR, you are pushing your branch to your GitHub fork (craig5/coredns-helm). So a "permission denied" result would coming from your fork, not coredns/helm (i.e. not something under coredns control). Perhaps you are not authenticated locally, or logged in with another account, or something like that.

Thanks!
Ok, I got confused with the instructions talking about "checkout the PR branch locally"...
I just amended the commit in my branch in my forked repo and it seems to be fine. :)
Sorry about the confusion!


Passing context or metadata into component cards/atoms

I'd like to be able to pass some arbitrary metadata about a mobiledoc down into the Ember components that render cards and/or atoms. Is there any good way to do this at the moment?
Here's a use case: I have some metadata about a mobiledoc's author. I'd like to refer to this in building the payload for certain atoms.
At the moment, it seems to me that the only way to do this is to subclass the mobiledoc-editor component and write a custom template that passes in the additional properties here. That doesn't seem optimal to me, because there's no way to inherit a parent template and override only a subsection thereof. Among other things, it's hard for maintainability; every time the component template changed upstream, consumers would have to manually review changes to the code upstream and update...
I understand if you don't want unnecessary clutter in the code, but I think it would be quite helpful to provide some way for consumers to pass context into their atom/card components. Any suggestions? (I'd be happy to provide a PR if appropriate.)

@decasia I believe you want to pass cardOptions to the editor? They are then passed options to card invocations.
See: https://github.com/bustlelabs/mobiledoc-kit/blob/master/CARDS.md

ah, grand. Thanks!

Hi @mixonic, I checked into this a bit more and it looks like there are a couple of changes that we'd need in ember-mobiledoc-editor in order to use the options hash in an Ember atom/card component. Basically, to get the end user's cardOptions passed into the ember card components, we'd need to do the following:

merge the consumer's options hash into the cardOptions callback hash that ember-mobiledoc-editor uses.
pass the options hash into the Ember component invocations.

I've tested this myself in this branch and it seems to work well. Would you be amenable to seeing this as a PR? I'd be happy to add unit tests.

@decasia that change seems totally reasonable to me- would be great to have in a PR with some tests 😄 Please tag me on the PR to review!


[flakey] webhook.TestAdmissionInvalidResponseForResource

=== RUN   TestAdmissionInvalidResponseForResource
[2366](https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative_pkg/2425/pull-knative-pkg-unit-tests/1493620921280237568#1:build-log.txt%3A2366)
    logger.go:130: 2022-02-15T16:23:56.397Z	INFO	webhook/webhook.go:200	Informers have been synced, unblocking admission webhooks.
[2367](https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative_pkg/2425/pull-knative-pkg-unit-tests/1493620921280237568#1:build-log.txt%3A2367)
2022/02/15 16:23:56 http: TLS handshake error from <IP_ADDRESS>:50900: EOF
[2368](https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative_pkg/2425/pull-knative-pkg-unit-tests/1493620921280237568#1:build-log.txt%3A2368)
    logger.go:130: 2022-02-15T16:23:56.551Z	ERROR	webhook/webhook.go:150	failed to fetch secret	{"error": "secret \"webhook-certs\" not found"}
[2369](https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative_pkg/2425/pull-knative-pkg-unit-tests/1493620921280237568#1:build-log.txt%3A2369)
2022/02/15 16:23:56 http: TLS handshake error from <IP_ADDRESS>:50902: tls: no certificates configured
[2370](https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative_pkg/2425/pull-knative-pkg-unit-tests/1493620921280237568#1:build-log.txt%3A2370)
    admission_integration_test.go:403: Failed to receive response Get "https://<IP_ADDRESS>:42363/booger": remote error: tls: unrecognized name
[2371](https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative_pkg/2425/pull-knative-pkg-unit-tests/1493620921280237568#1:build-log.txt%3A2371)
    logger.go:130: 2022-02-15T16:23:56.552Z	INFO	webhook/webhook.go:242	Starting to fail readiness probes...
[2372](https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative_pkg/2425/pull-knative-pkg-unit-tests/1493620921280237568#1:build-log.txt%3A2372)
--- FAIL: TestAdmissionInvalidResponseForResource (0.26s)

Here: https://github.com/knative/pkg/pull/2425
https://prow.knative.dev/view/gs/knative-prow/pr-logs/pull/knative_pkg/2425/pull-knative-pkg-unit-tests/1493620921280237568

/lifecycle frozen

@dprotaso is this still an issue?


Portal confuses values that are equal but of different types

I was able to reproduce the problem I reported in this slack discussion using plain clojure types.
(tap> {:a [[1]]})
(tap> {:a ['(1)]})

Here one object contains a vector [1] and another contains a list (1). However, the portal shows both as vectors.

If I clear it and tap them in the opposite order, both will be shown as lists:

This time I wasn't able to observe diferrent values by accessing portal selection, but tap-list clearly shows two different types:
@#'portal.runtime/tap-list ; => #<Atom@aeae4fd: ({:a [(1)]} {:a [[1]]})>

When types are important this becomes extremely confusing since you're used to trust what you see in portal.

Thanks for the bug report, should be fixed in 0.55.0.

Thanks for the bug report, should be fixed in 0.55.0.

Awesome, thanks a lot for a quick fix!

Unfortunately I just found that this didn't address my original problem.

So if I'm reading the fix correctly, it takes hashcode and meta into account but it doesn't take type into account. I'm pretty sure adding type to the hash would address my problem. As a workaround I could also force my custom objects to generate a hashcode that's different from the one that regular Clojure collections produce.
In general, I think we should expect key clashes when storing data in a cache. So when two or more different objects share the same key, we should still be able to distinguish them in the cache.
Thoughts?

Redefining hashcode function didn't work. hash function in Clojure ignores different values as can be seen here:
; Here is what I see after tapping two of my objects:
(map #(-> % :metrics first hash) @@#'portal.runtime/tap-list)
(-724974908 -724974908) ; identical values here

(map #(-> % :metrics first type) @@#'portal.runtime/tap-list)
(clojure.lang.PersistentArrayMap cash.Metric) ; different types here

(map #(-> % :metrics first .hashCode) @@#'portal.runtime/tap-list)
(1655286045<PHONE_NUMBER>) ; different values here

With the example provided above:
(tap> {:a [[1]]})
(tap> {:a ['(1)]})

I get:

The inner list / vector types are now correctly sent to the client.
Is there another example you are trying out?
The solution involves adding the additional hash-like number computed by:
(defn- hash+ [x]
  (cond
    (map? x)
    (reduce-kv
     (fn [out k v]
       (+ out (hash+ k) (hash+ v)))
     (+ 1 (if (sorted? x) 1 0) (hash+ (meta x)))
     x)

    (coll? x)
    (reduce
     (fn [out v]
       (+ out (hash+ v)))
     (+ (cond
          (list? x)   3
          (set? x)    (if (sorted? x) 4 5)
          (vector? x) 6
          :else       7)
        (hash+ (meta x)))
     x)

    :else
    (cond-> (hash x)
      (can-meta? x)
      (+ (hash+ (meta x))))))

Which assigns different values to different shapes based on various Clojure type predicates. This will easily produce collisions which is why the value is still part of the cache key.
here are the current tests.


To expose device's bond state

Hello guys,
Is it possible to expose the device's bond state on Advertisement class? For example:
public actual class Advertisement(
    private val scanResult: ScanResult,
) {

    internal val bluetoothDevice: BluetoothDevice
        get() = scanResult.device

    public actual val name: String?
        get() = bluetoothDevice.name

    public actual val bondState: String?
        get() = bluetoothDevice.bondState

    .......
}

Seems reasonable. From a quick search, it appears this status is unavailable for Apple and JavaScript, so this can be an Android specific property.

Until 0.4.0 is released, this can be tested via:
import java.net.URI

repositories {
    maven { url = URI("https://oss.sonatype.org/content/repositories/snapshots") }
}

dependencies {
    implementation("com.juul.kable:core:0.3.0-issue-65-SNAPSHOT")
}


BUG: Extraneous positional argument when running npm init foo

Congrats on the launch! Very exciting.
Describe the bug
npm init foo downloads the create-foo package and runs the "bin" specified in its package.json. It's commonly used to scaffold new projects.
It looks like turbo init behaves similarly to npm init, but turbo init foo (and by extension npm init foo) fails:

Link to the blitz that caused the error
https://stackblitz.com/edit/node-qcu4bd
To Reproduce
Steps to reproduce the behavior:

Go to https://stackblitz.com/edit/node-qcu4bd
Click into the terminal
Type npm init foo or turbo init foo

Expected behavior
The same behaviour as npm init foo
Version of webcontainer
Hash: 7286dbe8d47d3473cde7be488b7d048fb59b05f2
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Browser name  = Chrome
Full version  = 90.0.4430.93
Major version = 90
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/90.0.4430.93 Safari/537.36
performance.memory = {
totalJSHeapSize: 55442824,
usedJSHeapSize: 51747536,
jsHeapSizeLimit:<PHONE_NUMBER>
}
Additional context
Add any other context about the problem here.

@Rich-Harris A small update from our side. There's an internal PR for it so it shouldn't take too long before it lands. Keep you posted 🙏 .

@Rich-Harris So we fixed the npm init thing and it seems to work. BUT, some create-* packages uses git under the hood, like npm init svelte-app. We are working on our git internals and expose a git CLI as well, but right now that causes other issues.

I am closing this because it's the error remaining is not related to extraneous positional arguments as this particular issue has been fixed.


Dark image

Hi,
great job.
I have a problem.
I do not understand if it depends on the picam.
The image I see is very dark.
What can I do? Put two IR LEDs
Thanks!

Hi tbop-R.
Basically the Pi Camera will adjust the brightness automatically if there is too much light.
But requires a certain light level to produce usable images.
Those IR LEDs will only help if you have a NoIR version of the Pi Camera.
But it always depends on your use case.


Exception handling should be improved in fetch_remote_file.

https://github.com/telekom/mltb2/blob/e8ac1939b2f80d0375cb0a333baa85bcf71e0731/mltb2/files.py#L51

There is nothing I want to improve at the moment.


Identity Server 4 Running Behind a load Balancer

I have setup Identity Server 4 for my project using Entity Framework. I already configured the service to use a persisted grant Store and a Signed Certificate.

services.AddIdentityServer()
        .AddSigningCredential(Config.GetSigningCertificate())
        .AddResourceOwnerValidator<ResourceOwnerPasswordValidator>()
        .AddProfileService<ProfileService>()
        .AddConfigurationStore(builder =>
                    builder.UseSqlServer(connectionString, options =>
                        options.MigrationsAssembly(migrationsAssembly)))
        .AddOperationalStore(builder =>
                    builder.UseSqlServer(connectionString, options =>
                        options.MigrationsAssembly(migrationsAssembly)));

Here is the configuration of the service.
The problem is when I run my server behind a load balancer with for exemple 2 identic instances handling all the request, the server where the user did not logged in fail to decode the JWT token, leading to 401 unauthorized errors.
I'm assuming the sigining method of the tokens or their encription is the problem but I cannot find a way to solve this.
Here is the rest of my configuration.
The Configure:
 app.UseIdentityServerAuthentication(new IdentityServerAuthenticationOptions
{
      Authority = url,
      // Authority = "http://localhost:5000",
      AllowedScopes = { "WebAPI" },
      RequireHttpsMetadata = false,
      AutomaticAuthenticate = true,
      AutomaticChallenge = true,
})

the Client:
 new Client
{     ClientId = "Angular2SPA",
     AllowedGrantTypes = GrantTypes.ResourceOwnerPassword, // Resource Owner Password Credential grant.
     AllowAccessTokensViaBrowser = true,
     RequireClientSecret = false, // This client does not need a secret to request tokens from the token endpoint.
     AccessTokenLifetime = 7200, // Lifetime of access token in seconds.
     AllowedScopes = {
                       IdentityServerConstants.StandardScopes.OpenId, // For UserInfo endpoint.
                       IdentityServerConstants.StandardScopes.Profile,
                       "roles",
                       "WebAPI"
                      },
     AllowOfflineAccess = true, // For refresh token.
     AccessTokenType = AccessTokenType.Jwt
}

I also implemented my own IResourceOwnerPasswordValidator and IProfileService based on EF validations.
Any idea why is this happening?

fail to decode the JWT token

Who fails to decode token? did you turn on logging where this happens?

There is 2 servers working in parallel and the one that did not provided the token fail to validate it and returns 401 unauthorized on the apis routes.

Then turn on logging in the API to see what exactly is going wrong...

Hey, I found the error, the Elastic Beanstalk env was not the same on the 2 servers, leading to an authority error.You can close the issue.


When will release version 1.7.4?

When will release version 1.7.4?

I've just gone ahead and published it... let me know if you have any further issues!

Thank you.

Hello, I can't find the 1.7.4 version on npm. Can you please publish it?

I guess it was published again as 1.7.3 but under different name
https://www.npmjs.com/package/angularjs-audio
(via https://github.com/danielstern/ngAudio/issues/205)


Compilation error: Type 'Binding' has no member 'doubleColumn'

Hi, I'm currently on the ios17-refactor branch, and it's not compiling. It requires passing the binding. So, using .constant(.doubleColumn) instead of just .doubleColumn solves the issue.

Thanks, I've just fixed the issue and also merged the branch to main.


Sound burst playing pcm bytes

Hello and thanks for the plugin. I set up a TCP socket and use it to stream PCM data. When I restore the data in a byte array and import it to sound using ImportAudioFromRawBuffer function and play it, there is a sound burst at the beginning and end of the audio. Is there any way to solve this problem?

Hello. Thanks for reporting the issue.
The error was that the value was not calculated correctly during RAW transcoding. Instead of "-1:1" the value went to "0:1".
Already fixed.


Make sure that Netlib LAPACK respects FFLAGS

OpenBLAS allows users to specify FFLAGS and then uses override to append additional
options. However, without such an override in lapack's make.inc, lapack would use the external
FFLAGS, rather than the ones being computed by OpenBLAS. For example the DEBUG=1 flag
would not apply to LAPACK code. This is all a bit messy but forced by the integration with netlib
lapack. Note that CFLAGS already has this override for the same reason. It is possible that
other variables here should have a similar override, but I think for most of the other ones, OpenBLAS's
build system does not append to the flags passed in by the user.
cc @staticfloat @ianatol

Right, thanks. The overrides have been added on a case-by-case basis so far, and I do not think any of the remaining variables need them.
(DEBUG=1 may be a corner case, as in if you set your own FFLAGS you might as well put the -g there if you know you need it, but things could get interesting if the predefined FFLAGS dropped the recursive that a number of functions in Reference-LAPACK need)


fix issue_comm error

If the comm_size of a COMM_COLL_NODE node is 0 and the issue_comm function is called, the child nodes of this node cannot be released.

Hello, @SallyChangLiu,
Thank you for identifying the error and creating a pull request. I believe that communication nodes with a comm_size of 0 shouldn't actually appear. Instead of releasing child nodes, would it be better to print an error message and abort the execution?

Hello, @SallyChangLiu, Thank you for identifying the error and creating a pull request. I believe that communication nodes with a comm_size of 0 shouldn't actually appear. Instead of releasing child nodes, would it be better to print an error message and abort the execution?

In text2chakra_converter.py file, a node with comm_size=0 will be constructed, so I checked whether comm_size is 0 before issue_comm is called. You can also fix this by modifying the text2chakra_converter.py file so that nodes with comm_size 0 are not built, and printing an error message in the issue() function. @TaekyungHeo

Hi, I just pushed another PR that verifies the node's field validity before issuing nodes in PR #120. Should cover this case.
I think even if these traces are invalid and shouldn't logically appear, they still might appear from the buggy converter. Then it will be very helpful to debug the converter by capturing these buggy traces.

We have decided to raise an error when the communication size is zero in collective communication operators. Moreover, this corner case should be covered by https://github.com/astra-sim/astra-sim/pull/132. I will now close this PR.


build: upgrade java core to 9.10.1

PR summary
This PR upgrades our java core dependency to version 9.10.1
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

:tada: This PR is included in version 0.18.9 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Consider moving to a template generation model

Currently, OpenIddict.Mvc exposes a special (generic) controller and a few embedded assets/views to handle the entire authorization flow. Though really convenient for people who don't want to customize the controller or the view models, this approach offers a less flexible experience than the generated controller/models used by Identity.
The question is: should we move to a template generation model (probably based on yeoman) or should we keep the existing approach?
/cc @damccull @ilmax @Bartmax

There isn't a really and standard way of managing controllers/view. That's something I would expect to change soon since is a request for various authors, and even more now that everything is more "composable".
That being said, I think the normal workflow for someone is to :

install package
quickly make it work and get a feel of it
customize little
use it lightly
customize heavily.

If we fail to #2 or #5 we put user satisfaction in danger.
I'm not sure if everything can be done, here some ideas:
Create a separate package. OpenIddict.Frontend with a dependency on core. But instead of having everything embedded and compiled to install the actual controllers / views in the project. (This is not very upgrade able friendly)
create a template for all the controllers/views. (Identity does this for file new, but file new is an awful experience, if we can do like right click add new item select openiddict client views and controller  and then pick some info (like dbcontext, user entity) that would be great.
Yeoman have some limitations (Don't know exactly) and I think that's one reason why aspnet didn't go heavily with it.
We also need to think cross platform solution so maybe we can do a "command" like EntityFramework to create/download the assets.
For client side normally one goes with npm/bower. We can explore if those solution can be useful. Right now there is no npm not bower package that includes .cshtml but doesn't mean they won't exist. We would need some task for grunt/gulp to actually copy this files in the correct location so looks like a lot of investment but I won't discard without taking a look at it.
Can we have a post restore command on package? So that after installing the dll can execute something that creates/download the assets? Can this be optional?
We can also ask the asp.net for help and guidance, they are aware of authors of packages with views/controllers needs. While they don't have a solution yet, they may have some ideas.

Here's my first round of thoughts on the matter... I'm sure additional information could influence how I feel, so do help me out if I have something wrong here :)
In my travels creating packages for a different framework and language (although with similar challenges and structures to ASP.NET & C#), I landed on the practice of including controllers in my package, but not trying to wire them into the routing system.
If the project dev wanted to use my defaults, I'd offer a facility that quickly sets up all the routes to point to my default controllers.  In C#, this would likely done as an optional extension method on the MVC route building interface.
If however they didn't want to use my defaults, they could either create their own controllers and integrate with the library manually, or they could even extend my default controllers and override specific parts of functionality.
This approach resulted in maximum flexibility.  Devs could slide up and down the scale based on their needs without being restricted at any point.  It also obviated scaffolding for the most part and played nicely with the framework, allowing DI to continue to shine.
Openiddict seems close to this, although I wonder if it could just be rolled in with the main OpenIdConnectServer repository at that point.  The main thing that would have to change is that it would have to not use attributes for routing and middleware/filtering.
tldr;

I use nodejs stuff for front-end, but I don't like having to use yeoman for ASP.NET
Don't use attribute routing
Rely on the language and DI to offer extension points
Based on the last 2 items, Merge openiddict with OpenIdConnect

This is a hard one :)
Security is hard, and so provide maximum flexibility yet ensuring the correct behavior of the system should be hard as well.
I completely agree with @Bartmax for the typical user workflow, so we have to get up & running very quickly (e.g. provide a meta package that brings in all required dependencies)
We need to figure out which pieces of the puzzle are the most frequently changed/replaced by the end user and factor that pieces out.
IMO the UI is the part that will probably be replaced most of the time, but there are a discrete amount of interaction between front-end and back-end that are not so obvious to the average developer (read "not a security guy that loves reading RFCs" :smile: ) so we need to document all those interactions to provide all necessary guidance.
I think that template generation can be a nice approach, I'm not sure however if a VS template can be installed via nuget (a VS plug-in would be less than ideal IMO), and we should also keep in mind cross platform as well as mentioned by @Bartmax.
I like the cli commands approach but I have no experience on this subject and looking through EF issues seems to be not so easy to implement it right.

See related issues
#123

Closing, as OpenIddict.Mvc and OpenIddict.Assets have been removed and people are now encouraged to create their own controller.


Netmask is not applied properly

There is an issue documented here that showcases a problem with how the netmask filters out the address range.
I think this has to do with how the network length is converted to a "character length." See here:
https://github.com/ipgen/rust/blob/041184c27e011bae9a9cfb3b21d25150b47c21e1/src/lib.rs#L75
In the aforementioned issue, a prefix length of 7 was given. The network_len would then be calculated as 1 "character," leaving 3 bits to be modified by the address rather than preserved by the netmask.

I will be working on a significant rework that uses a bit system instead of a character system. Stay tuned, @rushmorem.

Said rework has been completed in #5


navbar logo not showing on scroll

Hi, thanks for the awesome template.
When i try scrlling down the page the nav logo disapears and goes to the alt text.
How can i fix this to show the logo src image?

I ran into the same issue. All I had to do is fix the path to the logo in /js/main.js
hope this helps

Hi, thanks for the awesome template.
When i try scrlling down the page the nav logo disapears and goes to the alt text.
How can i fix this to show the logo src image?

I ran into the same issue. All I had to do is fix the path to the logo in /js/main.js
hope this helps


[BUG] Changed node type from master to worker and longhorn not change node from unschedulable to schedulable

Describe the bug
Longhorn not recognize the change of new node type on change from master to worker.
To Reproduce

Delete node from cluster
Reinstall k8s
Join again to cluster (with same name)
Longhorn not schedule any disk to it (maybe has the old info of the cluster)

Expected behavior
Longhorn schedule disk to the new node
Log
Environment:

Longhorn version: 1.2.2
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Helm
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: RKE2

Number of management node in the cluster: 3
Number of worker node in the cluster: 5

Node config

OS type and version: Ubuntu 20.04
CPU per node: 56
Memory per node: 128G
Disk type(e.g. SSD/NVMe): FCoE
Network bandwidth between the nodes: 10gbps

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetal
Number of Longhorn volumes in the cluster: +20

Additional context
Using rke2

@lfdominguez
How did you delete RKE2 node from cluster?


Pass SVN revision to DoubleCmd build

Currently the field "revision" is not set in the DCmd about dialog. It's probably caused by the fact that it is build from the exported version exported from SVN. The DoubleCmd build script could be examined to find a way to provide that information via an extra parameter or an environment variable.

Implemented by patching DCmd build scripts to leverage provided SVN revision:


Dear, I have a bug with my code of training with using the  maskscoring_rcnn.detectron2

Traceback (most recent call last):
File "trainRes101.py", line 284, in
args=(args,),
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron2/detectron2/detectron2/engine/launch.py", line 79, in launch
daemon=False,
File "/home/jsxian2/anaconda3/envs/maskScoringOne/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 230, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/home/jsxian2/anaconda3/envs/maskScoringOne/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 188, in start_processes
while not context.join():
File "/home/jsxian2/anaconda3/envs/maskScoringOne/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 150, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
-- Process 1 terminated with the following error:
Traceback (most recent call last):
File "/home/jsxian2/anaconda3/envs/maskScoringOne/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 59, in _wrap
fn(i, *args)
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron2/detectron2/detectron2/engine/launch.py", line 125, in _distributed_worker
main_func(*args)
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron2/maskscoring_rcnn.detectron2/tools/trainRes101.py", line 247, in main
cfg = setup(args)
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron2/maskscoring_rcnn.detectron2/tools/trainRes101.py", line 206, in setup
cfg.merge_from_file(args.config_file)
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron2/detectron2/detectron2/config/config.py", line 54, in merge_from_file
self.merge_from_other_cfg(loaded_cfg)
File "/home/jsxian2/anaconda3/envs/maskScoringOne/lib/python3.7/site-packages/fvcore/common/config.py", line 123, in merge_from_other_cfg
return super().merge_from_other_cfg(cfg_other)
File "/home/jsxian2/anaconda3/envs/maskScoringOne/lib/python3.7/site-packages/yacs/config.py", line 217, in merge_from_other_cfg
_merge_a_into_b(cfg_other, self, self, [])
File "/home/jsxian2/anaconda3/envs/maskScoringOne/lib/python3.7/site-packages/yacs/config.py", line 478, in _merge_a_into_b
_merge_a_into_b(v, b[k], root, key_list + [k])
File "/home/jsxian2/anaconda3/envs/maskScoringOne/lib/python3.7/site-packages/yacs/config.py", line 491, in _merge_a_into_b
raise KeyError("Non-existent config key: {}".format(full_key))
KeyError: 'Non-existent config key: MODEL.MASKIOU_ON'
Can you teach me? please, Thank you!

Traceback (most recent call last):
File "train_net.py", line 162, in
args=(args,),
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron22/detectron2/detectron2/engine/launch.py", line 79, in launch
daemon=False,
File "/home/jsxian2/anaconda3/envs/maskScoringTwo/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 230, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/home/jsxian2/anaconda3/envs/maskScoringTwo/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 188, in start_processes
while not context.join():
File "/home/jsxian2/anaconda3/envs/maskScoringTwo/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 150, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
-- Process 1 terminated with the following error:
Traceback (most recent call last):
File "/home/jsxian2/anaconda3/envs/maskScoringTwo/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 59, in _wrap
fn(i, *args)
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron22/detectron2/detectron2/engine/launch.py", line 125, in _distributed_worker
main_func(*args)
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron22/maskscoring_rcnn.detectron2/tools/train_net.py", line 126, in main
cfg = setup(args)
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron22/maskscoring_rcnn.detectron2/tools/train_net.py", line 118, in setup
cfg.merge_from_file(args.config_file)
File "/gpfs1/scratch/jsxian2/maskscoring_rcnn_detectron22/detectron2/detectron2/config/config.py", line 54, in merge_from_file
self.merge_from_other_cfg(loaded_cfg)
File "/home/jsxian2/anaconda3/envs/maskScoringTwo/lib/python3.7/site-packages/fvcore/common/config.py", line 123, in merge_from_other_cfg
return super().merge_from_other_cfg(cfg_other)
File "/home/jsxian2/anaconda3/envs/maskScoringTwo/lib/python3.7/site-packages/yacs/config.py", line 217, in merge_from_other_cfg
_merge_a_into_b(cfg_other, self, self, [])
File "/home/jsxian2/anaconda3/envs/maskScoringTwo/lib/python3.7/site-packages/yacs/config.py", line 478, in _merge_a_into_b
_merge_a_into_b(v, b[k], root, key_list + [k])
File "/home/jsxian2/anaconda3/envs/maskScoringTwo/lib/python3.7/site-packages/yacs/config.py", line 491, in _merge_a_into_b
raise KeyError("Non-existent config key: {}".format(full_key))
KeyError: 'Non-existent config key: MODEL.MASKIOU_ON'
When I use train_net.py,  the error still happens.

what bash command did you use?

python train_net.py --config-file ../configs/COCO-InstanceSegmentation/mask_rcnn_R_101_FPN_3x.yaml --num-gpus 2 SOLVER.IMS_PER_BATCH 6 SOLVER.BASE_LR 0.002

with MASKIOU_ON ?

You mean that  the command should be
python train_net.py --config-file ../configs/COCO-InstanceSegmentation/mask_rcnn_R_101_FPN_3x.yaml --num-gpus 2 MODEL.MASK_ON True  MODEL.MASKIOU_ON True  SOLVER.IMS_PER_BATCH 6 SOLVER.BASE_LR 0.002
right?
but I insert the MASK_ON: True and MASKIOU_ON: True in the yaml file

@lsrock1 I'm getting 'Non-existent config key: MODEL.MASKIOU_ON' as well.
What are the exact steps to integrate this implementation on standard detectron2 setup?

Really sorry for the late response, i will fix it soon.

The default config of this implementation has maskiou option.
I think you use the original detectron2 train script or modules.

What files should be copied/merged to the main detectron environment @lsrock1?

@AKMourato
https://github.com/lsrock1/maskscoring_rcnn.detectron2/blob/2702b2a1e2efbeb7adf4d9de2a88976121afd822/detectron2/config/defaults.py#L27
Maskiou options in defaults.py


Bump Emscripten version

Should fix the "unexpected hash" errors caused by https://dl.bintray.com/homebrew/mirror/jpeg-9c.tar.gz returning "Forbidden!" again...
TODO: boilerplate too

Hmm, seems a lot slower...

Possibly because Release is now -O3. https://github.com/emscripten-core/emscripten/blob/main/ChangeLog.md#2012-01092021

12m :grimacing:
I guess we still have the option of overriding that back to -O2. If it ain't broke...

Tried to get ccache working (support added recently) didn't succeed, so forced it back to -O2 for now.
(Also tried just overriding for linking, which had about half the effect of doing it everywhere)

Was worth a try I suppose! Thank you.
Looks like users will build with the default options, too, so we don't lose anything by speeding up CI.

TODO: boilerplate too

...since this just got me again: https://github.com/32blit/32blit-boilerplate/pull/21


Smarter values of top_n for One Hot Encoder in AutoML

Currently, AutoMLSearch will only fit OneHotEncoders with top_n set to 10.
This can be problematic because a user can have data with more than 10 categories, e.g. 50 uses states) and the 40 least frequent states in the data get lumped together.
We currently have two workarounds for this:

Pass in a pipeline_parameters argument to AutoML where the value of top_n is set by the user. This may not be the best because this value of top_n would be applied for all categorical features.
User can create a DAG with multiple OHE (one for each categorical feature) where top_n varies for each OHE.

Ideally, AutoMLSearch would set a smart value of top_n for each categorical feature automatically that the user could then change depending on their domain knowledge.
This is closely related to #1728 but I feel it's a separate as it relates specifically to top_n in the OHE.
Maybe it makes sense to mark this as blocked by #1728.

@freddyaboulton great point. I agree that a) we should support setting top_n and other OHE params per-feature, and that b) having a "smart" value for top_n could be beneficial.
I will file a) separately. For b): do you have an idea for a method or heuristic? Let's discuss.


Update vminsights-enable-hybrid.md

Updated agent version number and SHA keys

@kinghorn : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@kinghorn: I'm sorry - only the author of this article, @bwren, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Let me know if you need more information about working in the private repo. I can send email that has the info.

Not sure how it went to the public repo - I just clicked the Edit link on the docs page like I normally do in MS Edge.  Will ping you tomorrow to see what has changed and how I can work w/the private repo

tried edit link again in the doc and this time it worked as usual and I was able to merge it myself - thx fpr the help!


[Bug] MDX23C-8KFFT-InstVoc_HQ bug on linux (Google Colab)

I was testing the MDX23C-8KFFT-InstVoc_HQ on Google Colab, and I was surprised when I heard the result, the audio was slow, the singer was singing slowly and the audio length was longer. I tested the same song on Windows with the same settings, and the results were normal.
Here, the code I used, both in Colab and on Windows:
MDX23C = models.MDXC(name="MDX23C-8KFFT-InstVoc_HQ", other_metadata={'is_mdx_c_seg_def': True,'segment_size': 384,'batch_size': 8,'overlap_mdx23': 8,'semitone_shift': 0},device=device, logger=None)
res = MDX23C(input_file)
vocals = res["vocals"]
af.write(f"{no_inst_folder}/{basename}_MDX23C.wav", vocals, MDX23C.sample_rate)

Here, the link to the songs:
https://drive.google.com/drive/folders/11aete_dd56XqR68P2cr_BMRlPhvHb7W0?usp=drive_link
And also an Audacity photo of the songs:

Are you sure that the input_file had 44100 sampling rate? The current code doesn’t resample automatically.

@MohannadEhabBarakat Yes, I'm sure, I don't think I've ever used Hi-Res audio in separation. All the audio I use comes from Deezer

@MohannadEhabBarakat And also, I used the same audio in both Windows and Collab and had different results, which I found strange. Maybe it's something to do with package versions. Here is the requirements file that I used in colab
requirements.txt

I just tested with two models, a VR (karokee_4band_v2_sn) and an MDX (Reverb HQ), and both gave normal results. I remembered that in the last tests I did, I used videos from YT, not from Deezer, but I don't think this is a problem, since the normal results from VR and MDX were using a video from YT

I was testing the HQ4, it also has this same problem, both on Windows and Linux. It looks like the semitone_shift is wrong

@MohannadEhabBarakat And also, I used the same audio in both Windows and Colab and had different results, which I found strange. Maybe it's something to do with package versions. Here is the requirements file that I used in colab. I'm going to test it today with VR models with the same package versions, and write what the results were requirements.txt

I think that might be caused because of package versions or resampling algorithms. I noticed that UVR GUI used different resampling according to the OS. I'm not sure why they did it but I just followed them to replicate the same results. For the package versions unfortunately even using the same versions might not solve the issue; As some libraries will have different implementations on different OSs (even with the same version). The workaround that worked for me in the past was to wrap everything in a docker file. Which is basically unifying the OS.
As I'm back now I'll be working on:

Fixing the bugs you found
Adding new docs
Adding new weights (at least the ones you tested)

So if you can send me an email with your findings and the current bugs, it will help me a lot 🤗.

@MohannadEhabBarakat And also, I used the same audio in both Windows and Colab and had different results, which I found strange. Maybe it's something to do with package versions. Here is the requirements file that I used in colab. I'm going to test it today with VR models with the same package versions, and write what the results were requirements.txt

I think that might be caused because of package versions or resampling algorithms. I noticed that UVR GUI used different resampling according to the OS. I'm not sure why they did it but I just followed them to replicate the same results. For the package versions unfortunately even using the same versions might not solve the issue; As some libraries will have different implementations on different OSs (even with the same version). The workaround that worked for me in the past was to wrap everything in a docker file. Which is basically unifying the OS.
As I'm back now I'll be working on:

Fixing the bugs you found
Adding new docs
Adding new weights (at least the ones you tested)

So if you can send me an email with your findings and the current bugs, it will help me a lot 🤗<EMAIL_ADDRESS>
I can try to help, but I don't know if it would be of much help, since I don't use most models, and I end up using only specific ones. In fact, I tested a model that is not available in the UVR repository, but that works both in UVR and in your code. If you want to take a look at this model I'm referring to, I uploaded it to the link below:
https://github.com/ShiromiyaG/RVC-AI-Cover-Maker/releases


Implement soft validations (map existing) for TDWG Test and Assertion Standard

Likely need to setup a config map to point to where the soft/validations are implemented to keep track what is done.
https://github.com/tdwg/bdq/projects/2

Love it


HADOOP-18407. Improve readVectored() api spec

part of HADOOP-18103.

Description of PR
How was this patch tested?
Not required as it is a doc only change.
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

CC @steveloughran

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 50s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
markdownlint
0m  0s

markdownlint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
40m  0s

trunk passed

+1 :green_heart:
compile
24m 51s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
21m  0s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 44s

trunk passed

+1 :green_heart:
mvnsite
2m  2s

trunk passed

+1 :green_heart:
javadoc
1m 39s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
1m 12s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 14s

trunk passed

+1 :green_heart:
shadedclient
24m 18s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 11s

the patch passed

+1 :green_heart:
compile
23m 48s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javac
23m 48s

the patch passed

+1 :green_heart:
compile
21m 40s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
21m 40s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m 40s

the patch passed

+1 :green_heart:
mvnsite
2m 12s

the patch passed

+1 :green_heart:
javadoc
1m 39s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
1m 15s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 16s

the patch passed

+1 :green_heart:
shadedclient
25m 14s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
19m  3s

hadoop-common in the patch passed.

+1 :green_heart:
asflicense
1m 34s

The patch does not generate ASF License warnings.

224m 42s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4760/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4760

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets markdownlint

uname
Linux 16edb2be83be 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 82d080042f2ce0379a05a27e7293eab1bb1a9c5b

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4760/1/testReport/

Max. process+thread count
1251 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4760/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

💔 -1 overall
Yetus failing because of no new tests were added. So ignoring that as it is a doc only change.


Fix amqp channel and connection creation in order to prevent "unknown delivery tag"

Pull Request type

[x] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Changes in this PR
After this PR amqp connection and channel will be created once instead of closing and opening channels each time there is a failure.
Upgrade amqp-client version to 5.13.0
Describe the new behavior from this PR, and why it's needed
Amqp connection/channel objects will be created once and amqp client will be responsible of restoring channels in case of a failure.
After this PR Conductor will be able to acknowledge messages with delivery tags from previous channels and process those messages successfully
Issue
https://github.com/Netflix/conductor/issues/2382
Alternatives considered
No

@ritu-p , can you please review this PR?

@aravindanr  @apanicker-nflx
As part of this PR i'm upgrading the version of the amqp-client since the current version (5.9.0) contains an issue related to consumer recovery and consumer tags.
Since I updated the relevant dependencies.lock files, the compilation creates the final jar with the right amqp-client version, but if i delete the dependencies.lock files and run "./gradlew generateLock updateLock saveLock" to recreate them, the lock files of the server and test-harness will have the older version 5.9.0 and not 5.13.0. This situation will cause that the final boot jar will have the amp-client:5.9.0 and not 5.13.0.
Updating the following files fix the problem:
./server/build.gradle
implementation( group: 'com.rabbitmq', name: 'amqp-client'){ version{require "${revAmqpClient}"}}
and adding to ./server/build.gradle
testImplementation( group: 'com.rabbitmq', name: 'amqp-client'){ version{require "${revAmqpClient}"}}
Are you ok with this approach or you have a better idea?
Thanks

@ritu-p @aravindanr
I made the requested changes and update the unit-test.
Can you please review and merge if everything is ok?
Thanks

@heinrichcohn I have approved the changes


Explore XAI method DeepLift and Shap

[ ] How to run it?

[ ] Do they work on our data? (or can we replicate author's results?)

[ ] How can we implement it into Dianna?

Either images or text or both.

There are two solutions: (1) use onnx model and borrow some functions from other DeepLift/DeepLiftSHAP implementations (e.g. captum) to produce our own implementations, specifically for onnx models. (2) convert onnx models to torch models and wrap up captum explainer directly (a bit nasty). We need to decide which direction to go.

There are two solutions: (1) use onnx model and borrow some functions from other DeepLift/DeepLiftSHAP implementations (e.g. captum) to produce our own implementations, specifically for onnx models. (2) convert onnx models to torch models and wrap up captum explainer directly (a bit nasty). We need to decide which direction to go.

I imagined option (1) when writing the proposal.

Consulting more carefully the document with selected XAI methods for DIANNA, I see we voted No for DeepLIFT!
Is this the best method to be under SHAP?

During the standup we decided to go for option 2 first (quick conversion of the model to pytorch, and then wrap captum). We can then later see if we want to expand that or go for option 1 completely.


Add telemetry event in StorageHelper

to see how often this is invoked before https://github.com/AzureAD/ad-accounts-for-android/pull/1196 is done. (Ideally, none).

Thanks @iambmelt @shahzaibj !


Redirect style will create new user

it's fine to link account in popup style, but wechat-mp require to use redirect style, which is a limit of the wechat. could you please support link account with redirect style?

I'll have to look into how wechat works and from that figure out where the issue is. I don't use wechat so I won't get to this any time soon, so if someone else wants to look into this, it will be much appreciated.

I've looked through the code and I don't see where the issue can originate. Did you make any progress on this @zhaoyao91 ? Did you try the newest version?

This is not related to wechat.
I use google and facebook, and when the flow is set to redirect a new user is created instead of adding the information to the logged in user

The code that I am talking about is in accounts-oauth:
Meteor.startup(() => {
  const oauth = OAuth.getDataAfterRedirect();
  if (! oauth)
    return;

  // We'll only have the credentialSecret if the login completed
  // successfully.  However we still call the login method anyway to
  // retrieve the error if the login was unsuccessful.

  const methodName = 'login';
  const { credentialToken, credentialSecret } = oauth;
  const methodArguments = [{ oauth: { credentialToken, credentialSecret } }];

  Accounts.callLoginMethod({
    methodArguments,
    userCallback: err => {
      // The redirect login flow is complete.  Construct an
      // `attemptInfo` object with the login result, and report back
      // to the code which initiated the login attempt
      // (e.g. accounts-ui, when that package is being used).
      err = convertError(err);
      Accounts._pageLoadLogin({
        type: oauth.loginService,
        allowed: !err,
        error: err,
        methodName,
        methodArguments,
      });
    }
  });
});

This will call the normal oauth login handler instead of the link

@MastroLindus Thanks a lot for locating the code. I will see what we can do to fix this in core.

@MastroLindus Thanks a lot for locating the code. I will see what we can do to fix this in core.

Same issue here with Google (linkWithGoogle) and loginStyle: "redirect" calls
Some progress on this issue ?


todo-syntax false positives for substrings and lowercase

Comments with words partially containing the word "todo" are marked the todo-syntax error, even if it is not a TODO comment.
E.g. in a comment with the word Todorov (could be a name of a method, class, variable, etc.), the issue happens, and I have to either disable tailor on the comment or misspell the word. Which are both bad, since I want to keep tailor on, while I correctly spell the word in the comment.
I suggest assuming the todo-syntax mark only if there is a TODO word that, either:

is NOT substring of another (like in "Todorov")
is the first word of the comment
is all in capital case.

Thanks for reporting @joaoscalett! Based on the original intent of this rule, we've decided to switch to only flagging comments that contain "todo" as isolated words when they are in an incorrect format (as you've suggested). However, we will still flag instances regardless of case and location of the word because we want to encourage the "preferred syntax" when TODO tasks are mentioned later in a comment or with different capitalization.
Thanks for your suggestions, the fix is forthcoming!


Introduce Deno tasks and remove --no-check flag

Signed-off-by: Jheyson Saavedra<EMAIL_ADDRESS>

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


Add support for augs/auras/cloaks/aetheria

Aug
Lum
Cloak stuff like http://acpedia.org/wiki/Cloaked_in_Skill
Surges (Aetheria)

Initial progress on XP augs on https://github.com/amoeba/accharplanner/tree/add-augmentations branch. I think I just need to add support for importing.

Closing in favor of more issues rather than this one staying open forever.


Organise and detail the wiki section ⬆

The goal is to make the whole repo the easiest possible to navigate and understand for any newcomer in the lab. The tree command can help to produce an arborescence of the files as I mentioned on Slack.

I like the way the documentation is going, but I feel it's missing a little something to tie it together on the home-landing page. It's main theme should be "Welcome new fiber tower developper!" and, after redirecting a user to the "User guide" page with a link, give to the newcomer an overview that helps her or him get started in the advent everybody is too occupied to help right away. I guess it should be along the lines of what @grudule explained to @alexmomo1212 at the beginning of the summer.
I think it needs a section on how our workflow to work with github, small new features branches to be merged in the development branch first, test an executable, pull request to the main brach with code review and /fast-forward merge (assuming with fix he missing token!), etc. I think one of the first things @grudule did in the lab was a diagram for this workflow isn't it? I think it's worth putting it in that section.
I would also definitely bring the functional diagram on the homepage, is the original of that diagram in the repo somewhere btw? I think it needs a little updating and very much replacing "laser sensor" by "diameter sensor" please.
I think a section giving some info on the relations between the code and the circuit diagram would also help, since otherwise one as to dig in the code to figure out the ins and outs on the Arduino and drivers. Actually, those pins do not figure as some variables or arguments somewhere in the GUI_doc?
Thanks in advance for these improvements!


Need to re-design Playback feature

Buttons next and back should be inside message box. At the beginning of the chat back button should be disabled; at the end - next button should be disabled
The message box in Playback should be the same size as in new chat.
Stop button should have another icon and should be moved to chat header

Expected design:

Please see Figma here (available for DIAL dev team only): https://www.figma.com/file/lwL5jneveZczAj5RLcLFH2/DIAL-Chat?type=design&node-id=2889-8559&mode=design&t=i4XpbdbFspb5xHrH-4

New design is verified on staging server successfully.


Support TraceFlags in logging exporter

The collector logs exporter should parse the LogRecord TraceFlags field to indicate if the trace is sampled.
It currently uses gcp.trace_sampled (https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/pull/444) which maps to the GCP LogEntry traceSampled field. This matches the spec we have, where all gcp.* attributes map to the matching GCP-specific log field: https://github.com/open-telemetry/opentelemetry-specification/blob/dc78006c12d9767fd2e35b691706c7572a76fd43/specification/logs/data-model-appendix.md#google-cloud-logging, but TraceFlags should also be respected and mapped to this.
Slack thread: https://cloud-native.slack.com/archives/C043XTR0JHF/p1689196581019039

Opened #690 to add this


Support new format input files for NEGF++.

Initial draft for the support of nextnano++-style input files for the NEGF C++ version.
Execution of the InputFile object is working.
Sweep execution is also working.
To do for the future:

Implement unit tests of NEGF++ methods

HI @Chikuwaq ! Could you, please, add an example of nextnano.NEGF++ input file to tests/datafiles/NEGF folder?

@HeorhiiY Added an input file and the database in the .negf format. The compatibilities are as follows:

C# version
Supports XML input files and XML databases only.
C++ version
Supports both XML and .negf formats for input files and databases. They can be mixed; the user can use XML input file with .negf database and vice versa.

Thanks for your work, @Chikuwaq ! I am merging branch and closing the request


Improve bundler size after tree shaking

What problem does this feature solve?
I noticed that in some cases, the bundle size of Rspack / Rsbuild is still larger than that of Vite after using new tree shaking.
I wonder if this is a bug or if the tree shaking strategy is different.
Test case
This is my test case:
https://github.com/chenjiahan/rsbuild-repro-tree-shaking-size

source code:

import { createApp } from "vue";
import { Button, Swipe } from "vant";

console.log(createApp, Button, Swipe);

npx rsbuild build with new tree shaking (91.9 kB):

➜  npx rsbuild build

  Rsbuild v0.3.2

ready   Client compiled in 0.32 s
info    Production file sizes:

  File                                Size       Gzipped
  dist/index.html                     0.26 kB    0.20 kB
  dist/static/js/index.1a4d2b1d.js    91.9 kB    33.9 kB

  Total size:  92.1 kB
  Gzipped size:  34.1 kB

npx rsbuild build without new tree shaking  (361.4 kB):

➜  npx rsbuild build

ready   Client compiled in 0.34 s
info    Production file sizes:

  File                                Size        Gzipped
  dist/index.html                     0.26 kB     0.20 kB
  dist/static/js/index.e227bbd7.js    361.4 kB    108.4 kB

  Total size:  361.7 kB
  Gzipped size:  108.6 kB

npx vite build (67.83 kB):

➜  npx vite build
vite v5.0.11 building for production...
✓ 277 modules transformed.
dist/index.html                 0.30 kB │ gzip:  0.23 kB
dist/assets/index-tluu1stB.js  67.83 kB │ gzip: 27.65 kB
✓ built in 660ms

Can you try webpack? It would not be considered as a bug, if our bundle size isn't bigger than webpack (at least for current stage).

Sure, I will add webpack soon 😄

I have updated the demo and added webpack.
The bundle size of webpack is 81.0 kB, smaller than Rspack, but still larger than Vite/Rollup. Maybe we can align with webpack first.
> rsbuild build --config ./rsbuild.config.webpack.mjs

  Rsbuild v0.3.2

ready   Client compiled in 0.76 s
info    Production file sizes:

  File                                Size       Gzipped
  dist/index.html                     0.32 kB    0.22 kB
  dist/static/js/index.81390de4.js    1.4 kB     0.77 kB
  dist/static/js/3.45f7ec6d.js        81.0 kB    30.6 kB

  Total size:  82.8 kB
  Gzipped size:  31.5 kB

Rspack: 91.9 kB
webpack: 81.0 kB
rollup: 67.83 kB

rspack patch

webpack without concatenate module

The rest size diff caused by concatenateModule

after enable concatenateModules the size reduced to


@chenglou

Has a lot of amazing talks, couldn't find one about him being a maintainer even though he maintains some amazing projects

Thanks for the suggestion! This list is meant to aggregate content produced by maintainers, rather than a list of maintainers themselves. If you do ever find a talk by chenglou that'd be good for this list, feel free to open another PR.

There's a lot of amazing content by him, but not about maintaining projects as far as I'm aware. Maybe an opportunity for an interview 😉


Join github.com/cdcgov

What type of help do you need?

[ ] Question
[ ] New Repo
[ ] Delete Repo
[x] User Membership (please make sure new members are familiar with the CDC open practices and set up their profile with name and org info to help people collaborate with them)
[*] Other

I need to join https://github.com/cdcgov in order to get access to the HL7 DEX repos.
I am a new developer on the DEX team.
Thanks
Emil Hristov<EMAIL_ADDRESS>

Added you to cdcgov. You will want to contact Leslyn McNabb directly on write access to the DEX repos, and I would prefer it if your team members were also grouped under a Github team.


Build from sources failed on Ubuntu 16.04

I'm trying to install Caffe2 on Ubuntu 16.04, getting a build error. I have deleted the build folder and tried again but the same problem.
[ 97%] Building CXX object caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o /home/hannan/caffe2/caffe2/python/pybind_state.cc: In member function ‘virtual bool caffe2::python::PythonOpBase::RunOnDevice()’: /home/hannan/caffe2/caffe2/python/pybind_state.cc:202:74: error: ‘PyString_AsString’ was not declared in this scope              PyString_AsString(trace_vec[i]->tb_frame->f_code->co_filename);                                                                           ^ /home/hannan/caffe2/caffe2/python/pybind_state.cc: In lambda function: /home/hannan/caffe2/caffe2/python/pybind_state.cc:322:41: error: ‘PyString_Check’ was not declared in this scope              if (PyString_Check(arg.ptr())) { // string                                          ^ /home/hannan/caffe2/caffe2/python/pybind_state.cc: In lambda function: /home/hannan/caffe2/caffe2/python/pybind_state.cc:824:37: error: ‘PyString_Check’ was not declared in this scope          if (PyString_Check(arg.ptr())) { // string                                      ^ caffe2/CMakeFiles/caffe2_pybind11_state.dir/build.make:62: recipe for target 'caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o' failed make[3]: *** [caffe2/CMakeFiles/caffe2_pybind11_state.dir/python/pybind_state.cc.o] Error 1 make[3]: Leaving directory '/home/hannan/caffe2/build' CMakeFiles/Makefile2:3304: recipe for target 'caffe2/CMakeFiles/caffe2_pybind11_state.dir/all' failed make[2]: *** [caffe2/CMakeFiles/caffe2_pybind11_state.dir/all] Error 2 make[2]: Leaving directory '/home/hannan/caffe2/build' Makefile:138: recipe for target 'all' failed make[1]: *** [all] Error 2 make[1]: Leaving directory '/home/hannan/caffe2/build' Makefile:4: recipe for target 'all' failed make: *** [all] Error 2

I encountered the same error today, with ubuntu 16.04

same error

Hi there, had the same problem, solved it by using a virtualenv with python2.7 instead of python 3.5


Saving database clears undo history

Describe the bug
Saving the database wipes undo and redo history, making it impossible to undo action taken before a save.
To Reproduce

Open a Ghidra database and perform any modifying action (create a struct, rename a variable, add comment, etc).
The option to undo this action is available.
Save the database.
The undo option is no longer available.

Expected behavior
Saving should not change the ability to undo an action, since saving often and early is an important habit, and this bug discourages that.
Environment:

OS: Windows 10 Pro Version 1809
Java Version: 11.0.3
Ghidra Version: 9.0.4

The code currently works as designed, based upon how our database works.
I believe it would be more useful to support undo regardless of the files saved state, but that would require a major change to our system.


Update return type hint in SimpleHeaderSet

Fixes return type int according to line 175
Found while running https://github.com/phpstan/phpstan on my code

Q
A

Bug fix?
no (really just an inline comment fix)

New feature?
no

Doc update?
no

BC breaks?
no

Deprecations?
no

Fixed tickets
#...

License
MIT

duplicate of #1260

duplicate of #1260


clean all zero storages at index generation time

Problem
Split from https://github.com/anza-xyz/agave/pull/3188
We notice there are all zero account storages when we generate index at startup.
We should add them as candidates for "clean".
Summary of Changes

Add all zero account storages to clean when we generate index.
Add a stat for all zero account storages at start up.
Refactor to use local sum of stats before update the outer atomic stats

Fixes #

Are there perf numbers for how this impacts a node?
How many stores are only accounts with zero lamports?

This has been a issue for a while, especially with long running skip rewrite nodes,
see https://github.com/anza-xyz/agave/pull/2880 for the discussion.
I will start a node to gather the stats.

Note that this PR it self doesn't fully solve the problem.
https://github.com/anza-xyz/agave/pull/2880#issuecomment-2344990011
But this is a good step to the solution.
To fully solve the problem, we will need "shrink" PR, which handles single slot one ref zero accounts.
https://github.com/anza-xyz/agave/pull/3188

@HaoranYi check this out, too.
https://github.com/anza-xyz/agave/pull/2880/files
I added something here to force us to shrink these all zero lamport account storages. This work is so old I've lost my memory of how well this worked or didn't. I think without the full solution, it isn't really helpful to run clean unless the entire storage is zero lamport. However, it seems highly unlikely with the code we have today that we'll end up with an entire slot being zero lamport accounts. It definitely is orthogonal to the other pr, so this should definitely be split out and put in or closed independently.

The problem we see is that with long running skip rewrites, we are accumulating all zero lamport storages up to 16K.
That problem happens before we fix shrink to remove single ref zero lamport accounts.
Maybe we don't have this problem now.
But, PR2880 is the work motivated from the above problem and try to fix it in general when there are all zero lamport storage at start up. It will protect us from malicious attach by flooding zero lamport storages at start up.
@jeffwashington Your are right that the additional force shrink in this commit https://github.com/anza-xyz/agave/pull/2880/commits/a5d6aac327afe0dcc51185639a58da7aa2ec63ea is crucial.
As said in the comments https://github.com/anza-xyz/agave/pull/2880#issuecomment-2346508244,
we removed 16K dead slot slots with force shrink.
Without "force shrink", we are unable to remove those dead slots by just adding them to clean.
I have wrote up a theory for the problem before on slack. Now I repost it here.
Consider the following example.
  200 100
X   1   0
Y       0
Z       0
W       0
U       0

Here is how the clean algorithm works.
At startup, slot 100 have all zeros, inserted into clean and kick off clean.
clean starts:
- populate candidate keys (X, Y, Z, W, U) and candidate slots (200, 100).
- first reclaim for non-zero keys X/100
- However, X/100 can't be dropped because 100's total count = 5
- it will add 100 to shrink candidates ()
- in calc dependency, we remove 100 from count_map and later visit to
handle_claim for all zeros keys (Y-U) won't visit 100 because it is not found
in count_map (unfortunately)
Shrink starts
- look at candidate 100 (), but its shrink live ratio is not below 80%. So
it was skipped.
In the end 100 wasn't cleaned and won't be visited again unless X was rewritten. With skip rewrite and X being cold, slot 100 will unlikely to be touched again.
The new commit that force shrinking all zero slots, will let shrink work on
100 regardless. And thanks for the recent handling of all zeros added to shrink, shrink will see that all accounts in 100 are dead or single_ref_zero_accounts, and it can be dropped. hooray!
This explains what happens for slot 276499184 too.
https://anza-xyz.slack.com/archives/C0735T9CVPU/p1726063943441339
Where the first handle_reclaim runs, it only sees 5 accounts are dead out of 545. All
the rest are live zeros (single-ref zeros). Send it to shrink, which then skip it because of live ratio is
too low.
force shrink solves the problem to be working because it bypass the shrink productive check and shrink them anyway.
And "the change to keep track of single ref zero lamports will address part of this." (#3188) will also likely to solve the problem too, because shrink will be kicked off since Y-U are all track as single-ref zero accounts.

My node restart with this PR crashed ...
[2024-10-16T22:15:42.414152173Z ERROR solana_metrics::metrics] datapoint: panic program="validator" thread="solAccountsLo10" one=1i message="panicked at accounts-db/src/account_storage.rs:80:9:
    self.no_shrink_in_progress(): 296018580" location="accounts-db/src/account_storage.rs:80:9" version="2.1.0 (src:022d218e; feat:3176011155, client:Agave)"

I restarted my node.
Unfortunately, because the node was offline for too long. It started from a downloaded snapshot. So it doesn't actually test the fix.
But the good news is that there is no all zero slots from mnt snapshot.
[2024-10-17T20:08:28.436858014Z
INFO
solana_metrics::metrics]
datapoint:
generate_index
overall_us=202250480i
total_us=117120112i
scan_stores_us=1234552i
insertion_time_us=43268269i
min_bin_size_in_mem=1395i
max_bin_size_in_mem=75762i
storage_size_storages_us=72285i
index_flush_us=16536621i
total_rent_paying=0i
amount_to_top_off_rent=0i
total_items_including_duplicates=725196049i
total_items_in_mem=42697085i
accounts_data_len_dedup_time_us=64350759i
total_duplicate_slot_keys=122260784i
total_num_unique_duplicate_keys=42697085i
num_duplicate_accounts=79563699i
populate_duplicate_keys_us=4136265i
total_slots=432200i
slots_to_clean=432195i
copy_data_us=33458274i
par_duplicates_lt_hash_us=0i

I tested it again with a restart on another node that's running skip rewrites. This time it finds 398 all zero slots at startup, and didn't crash.
sol@jeff-dev-sv-3:~$  grep "all_accounts_are_zero_lamports_slots" logs/solana-validator.log | awk '{for (i=1;i<=NF;i++) print $i}'
[2024-10-17T22:26:46.145469392Z
INFO
solana_metrics::metrics]
datapoint:
...
all_accounts_are_zero_lamports_slots=398i (***)

I made a few more commits to refactor the statup stats, i.e. accumulate the stat for a chunk then update the atomic outer stats.

Rebase and fix conflicts after #3265.
There should be no functional changes.


Support Visual Studio 2017

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
The theme should support Visual Studio 2017 too.

Hi, material theme is made for Visual Studio Code, if you want it to be ported for other software, you need to hope that someone will do it, or make it yourself. Here you can check the officials portings:
https://github.com/equinusocio/vsc-material-theme#official-portings


Make sure not to require react before NODE_ENV has been set

If NODE_ENV is undefined when we require react it defaults to the development bundle which breaks next start since once it's required it is cached.
Fixes: #7167
Closes: #7190

Closes https://github.com/zeit/next.js/issues/7082 too

Awesome :D


trying to catch a MRZ

Hello
I'm trying to scan the MRZ ( Machine Readable Zone ) of an ID card.
example here : [https://www.ibz.rrn.fgov.be/typo3temp/pics/476d8015a3.jpg](sample picture)
It's not very consistent, usually the first line is not getting picked up. Are there specific options i could try out ?
I tried the SetTryHard but there's no real difference in my case.

It might just be too much for the platform API to process, though that might also depend on how you're providing the image to it. Try preprocessing the image to make the text as readable as possible: https://medium.com/technovators/survey-on-image-preprocessing-techniques-to-improve-ocr-accuracy-616ddb931b76


fix(curriculum): small English update

Checklist:

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the main branch of freeCodeCamp.
[ ] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

Oh, I'm sorry @moT01. I made a fast edit directly on Github and must have overlooked that. Will do that next time.


Use IPv6 VPN transfernetwork

What this PR does / why we need it:
Replace the existing <IP_ADDRESS>/24 network which is currently in use for the vpn transfernetwork with a IPv6 ULA address space. This frees the usable networks ranges for shoots.
Which issue(s) this PR fixes:
Fixes #
Is a simpler approach than what @timebertt did in https://github.com/gardener/gardener/pull/9597
Special notes for your reviewer:
Release note:

Thank you @majst01 for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.

/close in favour of #99


[bug] updater cannot launch app after installation

Describe the bug
App is able to detect a new version is available,  and gets exited immediately after calling installUpdate(), also brings up the installer.
however, app does not get relaunched after installation finished somehow.
I was following the instructions on the doc that said I should call 1. installUpdate then 2. relaunch.
But app immediately exited after calling installUpdate and relaunch never have a chance to run.
I've also tried emit('tauri://update-install'), it has the same result which exits the app, and never bring it back.
Reproduction
No response
Expected behavior
No response
Platform and versions
Environment
  › OS: Windows 10.0.22000 X64
  › Webview2: 101.0.1210.53
  › MSVC:
      - Visual Studio Community 2022
      - Visual Studio Build Tools 2017
  › Node.js: 16.15.0
  › npm: 8.5.5
  › pnpm: Not installed!
  › yarn: 1.22.18
  › rustup: 1.24.3
  › rustc: 1.61.0
  › cargo: 1.61.0
  › Rust toolchain: stable-x86_64-pc-windows-msvc

Packages
  › @tauri-apps/cli [NPM]: 1.0.0-rc.12
  › @tauri-apps/api [NPM]: 1.0.0-rc.6
  › tauri [RUST]: 1.0.0-rc.12,
  › tauri-build [RUST]: 1.0.0-rc.10,
  › tao [RUST]: 0.9.1,
  › wry [RUST]: 0.17.0,

App
  › build-type: bundle
  › CSP: unset
  › distDir: ../webapp
  › devPath: http://<IP_ADDRESS>:3000/

App directory structure
  ├─ .git
  ├─ dist
  ├─ node_modules
  ├─ src
  ├─ src-tauri
  └─ webapp

Stack trace
No response
Additional context
https://discord.com/channels/616186924390023171/731495028677148753/979313103902765066

Maybe we can force the launch [App Name] option from the CLI, i'm not sure. Because currently we force exit after running msiexec and leave the restart to the UI.

@lucasfernog

leave the restart to the UI

So, to be more precise, does it mean we need to manually open the app after updating? at least for now?

same issue


Talk integration

Need to record events in talk - collect, comment, post, hashtag
Also look at creating script for after-the-fact posting (per previous issue)

not really a geordi code issue; closing and tracking elsewhere


Inference.run throws when passing Variables of type PARAMETER as 'constants' argument

Describe the bug
When passing Variables of type PARAMETER as 'constants' argument to the constructor of Inference (or child classes), this line throws during the execution of Inference's run method since the Variables that were passed as 'constants' are in self._var_trans but not in the 'kw' input argument.
Expected behavior
Execution does not throw and Variables passed as 'constants' argument to the Inference constructor, even if they are of type PARAMETER, are treated as constants and not optimized in training.
Desktop:

OS: Ubuntu 18.04.2
Python version: 3.6
MXNet version: 1.4.1
MXFusion version: 0.3.1
MXNet context: CPU

The first question I have for this is whether it even makes sense to pass PARAMETER type variables to the 'constants' argument in Inference?
If it does make sense, it looks like the way to fix it is by adding a check in the prepare_executor method of the InferenceAlgorithm to not add 'constants' to self._var_trans. This is because if they are constants they shouldn't need to be transformed anyways, and we can just get them from the constants dictionary. Though that is a slightly funny interpretation of the transformation interface I think.

Thanks @meissnereric. This issue is not blocking me. I think there is value on supporting a PARAMETER to be passed as 'constant' argument of Inference. Several use-cases could benefit from it. If not supported, I suggest (1) making very clear in documentation it's not supported, (2) describing an alternative, and (3) issuing a warning if PARAMETER is passed as 'constant' to facilitate debugging.


Question: is it possible to prevent value change when clicking outside slider-thumb ?

The idea is to use this slider on ionic v1 and allow the user to scroll the page starting from an element containing a slider.
In that case, the slider should modify the value only if dragged from the slider-thumb. Is it possible ?

You can set the onlyBindHandles to true.

Demo: http://jsfiddle.net/dsu2kjdz/


[dv] Add asserts to check alerts for memory integrity failures

We should wait to merge this until CI is fixed right?


Shared mailbox direct logon

Hello,
Still not clear, is it possible to use Shared mailbox for direct logon only to Exchange Online services, as for example for scanner, can we set up some shared<EMAIL_ADDRESS>reset a password via Azure/Admin and add it into scanner settings (same for some web apps, like a ticketing system). I know it is working if we would be doing this and this document informs when to use license for a shared mailbox: https://docs.microsoft.com/en-us/microsoft-365/admin/email/remove-license-from-shared-mailbox?view=o365-worldwide , so in our scenario, it is safe to use!? The general article also says that if a hacker can still creds of the shared mailbox then can access it, but the same happens with a normal user account and delegated access, only the hacker needs some time to test access to the GAL list of shared mailboxes of the organization by running some script wouldn't take a long time...
Just curious if we can use Shared mailbox and reset a password for it and use it for:

Scanner
Ticketing portal
Direct outlook setup, like secretary use only<EMAIL_ADDRESS>100% of her working time
It is working now, but is it allowed? what will be happening if we use like this long time?

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

ID: 289f2440-9c44-7d93-20e3-baa9e9892cd9
Version Independent ID: abc4e209-a071-d28a-cb01-aa72bbaee7c0
Content: About shared mailboxes - Microsoft 365 admin
Content Source: microsoft-365/admin/email/about-shared-mailboxes.md
Service: o365-administration
GitHub Login: @SKjerland
Microsoft Alias: sharik

@lightupdifire Thank you for your feedback. A shared mailbox is not intended for direct sign-in via outlook, ticketing tool, or a device like a scanner by its associated user account. You should always block sign-in for the shared mailbox account and keep it blocked. It is not designed for that and If you would like to use it with a password assigning the license is the correct way.
Hope this helps!
Thanks
Sri

@lightupdifire Thank you for your feedback. A shared mailbox is not intended for direct sign-in via outlook, ticketing tool, or a device like a scanner by its associated user account. You should always block sign-in for the shared mailbox account and keep it blocked. It is not designed for that and If you would like to use it with a password assigning the license is the correct way.
Hope this helps!
Thanks
Sri

@msbemba
Thanks, Sri! Strange that it is not well explained in this document and most advanced IT knows that it is working just fine even without a license... and the sentence "it is not designed" doesn't really say it is not allowed to use this way :) so a little bit confused, especially knowing some setups based on Microsoft docs, where in the Windows Servers 2019 we must follow steps from Windows Server 2003 as an example... On one side I can understand your strategy and if you find that it is nothing to update in the doc, like a more clear explanation, then you can close this case.

@msbemba
Thanks, Sri! Strange that it is not well explained in this document and most advanced IT knows that it is working just fine even without a license... and the sentence "it is not designed" doesn't really say it is not allowed to use this way :) so a little bit confused, especially knowing some setups based on Microsoft docs, where in the Windows Servers 2019 we must follow steps from Windows Server 2003 as an example... On one side I can understand your strategy and if you find that it is nothing to update in the doc, like a more clear explanation, then you can close this case.

@lightupdifire Every shared mailbox has a corresponding user account. Notice how you weren't asked to provide a password when you created the shared mailbox? The account has a password, but it's system-generated (unknown). You aren't supposed to use the account to log in to the shared mailbox. More details can be found here - https://docs.microsoft.com/en-us/microsoft-365/admin/email/create-a-shared-mailbox?view=o365-worldwide#block-sign-in-for-the-shared-mailbox-account
I have added this to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/4034
Thanks
Sri

@msbemba
In the last 5 years, for about 1000 users, I have integrated all services from Microsoft 365 E3 license, but before that, I have migrated Exchange 2016 on-premise to Exchange Online, AD DS to AD Connect + AD FS, File Servers to SharePoint Online, SCCM to Intune, user MFA to Conditional Access MFA, data protection to DLP and AIP, on-premise legacy tools to RDS and WVD, so I have noticed A LOT :)
But have you noticed that you can Enable a Shared mailbox and reset a password for it and use it together with Conditional Access MFA so the account stays protected with MFA?
Thanks for calling the new PR! will wait for an update.

@msbemba Thank you very much for the contribution and updating the documentation with PR. @lightupdifire Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Request support for Azure openAi

Request support for Azure openAi

It's certainly an easy add but there's a small issue, I don't have any credit in Azure so i can't really test it...

A simple solution is to use a proxy such as https://github.com/coolxyz/cloudflare-azure-openai-proxy/

Added custom OpenAI API URL support in the latest build.
Closed as completed.


Update HyPhy to 2.5.7

On usegalaxy.org

The build script doesn't work on more than one toolset at once, you'll have to split these into two PRs.

@galaxybot deploy this


ViewController Containment support

Hi @lkzhao,
thanks for great library.
Currently I'm prototyping an app in Storyboard and would like to use Hero for some transitions.
Is it possible to use Hero with ContainerView and Embed segue?
I saw a similar issue has been raised (#153), but has this functionality been implemented?

ContainerView is not supported at the moment. But there is an API available called
  Hero.shared.transition(from: UIViewController, to: UIViewController, in view: UIView, completion: ((Bool) -> Void)? = nil)

It basically allows you to provide two VC and a containerView and apply transition animation between the first view controller to the second one. try that out and see if it works for you.

That might work for my case, I need to try.
This could be a "nice to have" feature, as it allows using Hero as a prototyping framework.
Storyboards have a drawback with reusability. It can be solved easily by using References and VC Containment where ContainerView points to a Storyboard reference.
Incorporating Hero with its default transitions will make these prototypes look extremely realistic, and they can be done as quickly, as prototypes done in specialized software (e.g. Origami / Pixate etc).

@richardtop Did this solve it for you? I'm also after container support.

@J7mbo I finished the prototype faster than I was able to try this feature.

Hi,
First of all thanks for such useful library 👍
I am using multiple ContainerViews in my "TargetVC" connected to tabs (to be able to switch in between). These ContainerViews embed UITableViewControllers. Data is loaded from network. Network data is small and total time to get data is around 50ms.
As you mentioned above i tried to use "Hero.shared.transition" but i guess i could not apply correctly :(
Lets say there are :

HomeVC
TargetVC
. containerView
. - TargetVC_ChildTableVC

To be able to have a smooth cascade animation (as in Documentation) where shall i implement "Hero.shared.transition"
Another question; if i want to display "TargetVC_ChildTableVC" to come from bottom of the screen (at the same time with other animations) after table view contents loaded/rendered . What shall i need to do?
Best

If i can manage to do :
I want to display "TargetVC_ChildTableVC" to come from bottom of the screen after table view contents loaded/rendered. Currently Hero animates the ContainerView first and then load the table data of embedded TableViewController.
will be more than enough for me.

I'm just evaluating Hero at this point and haven't written any Hero code. It looks really nice - exactly the kind of morphing I was looking for. However, this "indirect view controller transition" issue is a potential sticking point for my use case as well.
I'll outline my specific VC structure and use case below just for reference, but I don't want to bury the lede. The main reason I'm posting here is to ask what the current status on this is and to ask if you'd be open to a PR in this area if I were to implement it. If so, do you have any specific thoughts or plans regarding the API for it?
In my app, every state for an area is a separate view controller. So, for example, in the Search function, there's one VC that owns the overall search function. I'm using a UISearchController, which by design segregates the presentation of search results into a separate view controller. That view controller in turn is just a big container view that swaps in sub-controllers for states such as "searching spinner animation", "there was a network error", or the presentation of actual search results in a table view.
I outline this in terms of Search, but it's really a more general pattern. For example, one transition I would like to animate is from a tap on a search result to a detail presentation:

I just want to do the obvious transition animation anchored to the image. But the destination is actually a PodcastViewController that manages a separate container view, into which (in the typical case) it installs a PodcastStandardStateViewController.
It's possible that the existing API might handle this just fine. I'm just a bit unsure about the common container view. There's really nothing in common between the two view hierarchies until you get back into the navigation controller.

OK, I've looked into this a bit more and here's what I've found.
Multiply-nested view controllers aren't an issue at all since Hero looks primarily at the view hierarchy. It doesn't really care that much about view controllers; all the view controller stuff in the API is just there to impedance-match with Apple's underlying transition API. As long as you get the "before" and "after" states set up fully before transitions, it just works.
The real bête noir in my setup turns out to be UISearchController. The way it's implemented is kind of wacky - look at the view debugger in Xcode when you're browsing search results and you'll see that you're off in some modal presentation that has nothing to do with the navigation controller hierarchy perceived by the user. When you transition to a detail view, there's no obvious way for Hero to get its hands on the right pair of root views.
I actually tried implementing a little hack that would let view controllers say "this is my real view" to Hero, but it caused crashes, so it didn't seem like a promising avenue. I ended up essentially reimplementing my own version of UISearchController that uses a saner containment hierarchy, and that seems to be working fine.
Hero transition animations for container views also work fine, with some additional glue added. The issue is that you have to continue to do the manual view-controller-hierarchy management while also calling Hero.shared.transition, and it's not entirely clear at exactly what point that call should go. This is what I've been using; seems to work so far, but I'm not 100% sure there isn't a latent problem here:
    func transitionTo(_ controller: UIViewController) {
        if let highest = childViewControllers.last {
            if Hero.shared.isTransitioning {
                // print("Transitioning!")
                Hero.shared.complete(finished: true)
            }
            highest.willMove(toParentViewController: nil)
            highest.removeFromParentViewController()
            addChildViewController(controller)
            Hero.shared.transition(from: highest, to: controller, in: containerView) { finished in
                controller.didMove(toParentViewController: self)
            }
        } else {
            add(controller, inView: containerView)
        }
    }

    // Called on parent view controller
    func add(_ child: UIViewController?, inView destination: UIView? = nil) {
        if let child = child {
            addChildViewController(child)
            let targetView = destination ?? view!
            targetView.addSubview(child.view)
            child.view.frame = targetView.bounds
            child.didMove(toParentViewController: self)
        }
    }

I'm not sure that the handling of the cases where Hero already isTransitioning are handled correctly, either. (In my application, there can be a very quick transition from a "loading" view controller to a standard detail view, faster than the animation time.)
Bottom line for me: I think all the pieces are already in Hero. It might be helpful to include something like the transitionTo method above that includes view controller changes, just because it's not self-evident how to write this without some experimentation. Either that, or provide some kind of stepwise API that does not manage the subview change (e.g., recordInitialView() followed by recordTargetView() and animate()).


Create a input section in the api

Create some POST api functions to update the text to fetch

done


Encountered some error when evaluating

Hello thank you for the great work
I have some problems when I try to evaluate the metrics.
I use the CUDA11.0 and the pytorch 1.7.1
   conda install pytorch==1.7.1 torchvision==0.8.2 torchaudio==0.7.2 cudatoolkit=11.0 -c pytorch
And I download the trained weights

I load the weight as
-w /media/ee904/Data_stored/PGP/PGP_lr-scheduler/archive/data.pkl
then the error pop up I think it's about the weight format..
Can you help me? Or should I need to use the older version of CUDA?

I think there could be different versions between train and test

Loading the .tar file itself as weights should solve your problem.

Yes, please directly pass the .tar file as the weights.
-w /media/ee904/Data_stored/PGP/PGP_lr-scheduler.tar