perf: enable compress minify option for node target

Summary
For the Node target, keep output bundles as small as possible while retaining the necessary information for debugging, as Node outputs usually executed directly at runtime rather than being built again.
See https://github.com/web-infra-dev/rsbuild/pull/3966 for more.
Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

I found that this optimization breaks a lot of test snapshots, some of the changes are unexpected. For example, the code readability may be significantly worse.
I'm now tending to make this optimization opt-in. If users want a smaller Node.js bundle, they can manually set the minify configuration.

I'm now tending to make this optimization opt-in. If users want a smaller Node.js bundle, they can manually set the minify configuration.

Do we need to encapsulate a configuration, or we can just document how to configure it if a smaller bundle is needed?

I think adding documentation is good enough for this


Add NPM package extension

Problem
A use case has surfaced where we'd like to tag a request originating in Javascript with a trace ID. Since we use Log Weasel on the server, it would be nice if Log Weasel exposed a mechanism for easily creating an ID for this purpose on the client.
Solution
Add and publish an NPM package from Log Weasel that handles creating a Log Weasel formatted ID in JS.
Example usage in a consuming app
import * as LogWeasel from "@stitch-fix/log-weasel";

LogWeasel.init('CLIENT_HOME_UI');
LogWeasel.generateId(); // anywhere you want to generate a new Log Weasel ID

Checklist
Before Merging

[ ] If there is an RC on this branch, revert the version change in version.rb

After Merging
See the gem release process for a detailed list, but the gist of it is:

[ ] Fetch master locally and run the applicable rake version:* task on master to bump the version
[ ] Run rake release on master to release the new version on Gemfury
[ ] Add release notes - this is very important in helping other engineers understand what changed in the new version

@robwierzbowski / @benmvp I'd appreciate your expertise/feedback on the front-end config I added, and my overall approach.

@benmvp / @robwierzbowski this is ready for re-review


*: expose Leadership Transfer API to clients

Fix https://github.com/coreos/etcd/issues/7584.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0fe8fdc). Click here to learn what that means.
The diff coverage is 69.11%.

@@            Coverage Diff            @@
##             master    #8153   +/-   ##
=========================================
  Coverage          ?   76.43%
=========================================
  Files             ?      345
  Lines             ?    26946
  Branches          ?        0
=========================================
  Hits              ?    20596
  Misses            ?     4865
  Partials          ?     1485

Impacted Files
Coverage Δ

etcdserver/api/v3rpc/rpctypes/error.go
100% <ø> (ø)

etcdserver/api/v3rpc/util.go
100% <ø> (ø)

etcdserver/errors.go
0% <ø> (ø)

etcdctl/ctlv3/command/printer.go
39.78% <0%> (ø)

etcdctl/ctlv3/command/printer_simple.go
68.18% <100%> (ø)

integration/cluster.go
85.79% <100%> (ø)

etcdctl/ctlv3/ctl.go
100% <100%> (ø)

etcdserver/server.go
80.53% <100%> (ø)

proxy/grpcproxy/maintenance.go
46.87% <100%> (ø)

...xy/grpcproxy/adapter/maintenance_client_adapter.go
32.25% <100%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

@heyitsanthony PTAL. Thanks.

@heyitsanthony PTAL. Thanks.


Getting error while doing mvn test "The POM for javax.xml.bind:jaxb-api:jar:${jaxb.version} is missing, no dependency information available"

Hey team,
I cloned this project and getting error while doing mvn test in junit packages. It seems this packahge information is missing in  the solution.
The POM for javax.xml.bind:jaxb-api:jar:${jaxb.version} is missing, no dependency information available
[WARNING] The POM for javax.xml.bind:jaxb-api:jar:${jaxb.version} is missing, no dependency information available
[WARNING] The POM for com.sun.xml.bind:jaxb-core:jar:${jaxb.version} is missing, no dependency information available
[WARNING] The POM for com.sun.xml.bind:jaxb-impl:jar:${jaxb.version} is missing, no dependency information available
Getting below Error on compile on this :::

@guptaakhil please follow this process: https://github.com/intuit/karate/wiki/How-to-Submit-an-Issue


Add FallbackProvider and NullProvider

The trait world::Provider abstracts over types that can potentially provide a Chunk from a game world. We should add a helper type FallbackProvider that saves two arbitrary Provider, one of which is the primary provider while the other one is the fallback provider.
The type should also implement Provider by first trying to get the chunk from the primary provider and using the fallback provider in case the primary provider failed to deliver the chunk.
Additionally there should be a type NullProvider that always fails to provide a chunk. Both types are important for world provider composition.

Done in #19


write generated types to __types directories

fixes #4701
fixes #4973

Changes where generated types are written to:
-import type { RequestHandler } from './foo';
+import type { RequestHandler } from './__types/foo';

though the fix seems a bit... unidiomatic? would love a gut check from typescript-heads. little surprised this didn't come up sooner
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

If we do end up doing this, i wonder if we should make the same change to the JSDoc comments, and the examples in the docs:
-/** @type {import('./index').RequestHandler} */
+/** @type {import('./index.d').RequestHandler} */

It works either way, but there might be some value in consistency

I'm not a Typescript expert, but Microsoft seems to think having files with the same name, (e.g. foo.d.ts and foo.ts) is a bad idea - https://github.com/Microsoft/TypeScript/issues/7624
Maybe one solution would be to put the generated types file in types/foo.d.ts?
Tangentially related, I'd like to have my *.d.ts files in src/types/ and I see that that generated subtree of types follows this convention. Meaning, the generated type declarations are put in .sveltekit/types/. So maybe it's an opportunity to move app.d.ts into src/types/app.d.ts? That could clean up the conventions on where Svelte is storing types and also solve this issue because generated types would no longer be in src/routes/foo.d.ts and would instead be in src/types/routes/foo.d.ts.

I'm not a Typescript expert, but Microsoft seems to think having files with the same name, (e.g. foo.d.ts and foo.ts) is a bad idea - microsoft/TypeScript#7624

I second this. The TS resolution algorithm has a certain order to resolve files (ts>dts>js) and I think it wasn't intended to support two files which only differ by file ending. The more robust fix would therefore probably be to create a type file name that is distinct, for example taking advantage of our "double underscore is reserved": import("./__types.index"), or as a folder import("./__types/index")

I third it.

Updated to use __types, this is now ready for review

Yeah, this fits


Merging 1.2.0 release into master

Thanks @Athanaseus, want to check again? I see a new docker image has been pushed, so maybe things will work already...

Now working as expected.


Status / health check information for cohort connectivity

The objective of this issue is to discuss options and a proposed approach for exposing basic status / health check information on the connectivity of a given server to its cohort(s).
Understanding of current interactions:

Currently there are a number of OMRSTopicConnector instances that appear to define the interactions with a cohort
These appear to be managed in the OMRSCohortManager class
A number of instances of this OMRSCohortManager class (one per cohort?) are then available within the OMRSMetadataHighwayManager class, which manages the connectivity to each cohort that a local server is a member of
This OMRSMetadataHighwayManager class is then in turn exposed via APIs whose logic is implemented through the OMRSMetadataHighwayRESTServices class and ultimately bound to APIs (using Spring) in MetadataHighwayServicesResource
The actual connectivity to event buses within the OMRSTopicConnector appears to be through a list of OpenMetadataTopicConnector instances
Today there is no status information that is exposed through either the OMRSTopicConnector class (which itself implements a number of interfaces, notably OMRSTopic and OpenMetadataTopicListener as well as a base class ConnectorBase) or the OpenMetadataTopicConnector class (which also extends ConnectorBase and implements a different interface: OpenMetadataTopic)

Suggestions for providing status / health check information:

If we extend one or more of these interfaces / abstract classes with a method for retrieving status, this could then be served through underlying implementations
Examining an example underlying implementation (KafkaOpenMetadataTopicConnector, which extends OpenMetadataTopicListener), we can see that this ultimately extends the ConnectorBase abstract class
Considering further that it could be useful for any connector that can be started and disconnected to be able to communicate basic status / health-check information, perhaps it would make sense to extend the underlying Connector abstract class with such a status retrieval method (?)
There is already an isActive() method defined at the ConnectorBase level from which everything extends, but this is very binary and currently based purely on whether the connector has been started or disconnected

So as a proposed approach:

Add a method to the Connector abstract class to retrieve a connector status object (to be defined, but likely including at least an enumerated status (specific values to be defined), some more "free-form" informational field (string)?)
Provide a default implementation of this new method in ConnectorBase that simply re-uses the isActive() method also defined there to translate the binary (boolean) of isActive() into a basic status object
Allow any implementation of a Connector to override this new method with a more granular detection of various statuses (non-binary)
Use this connector-level method to surface a status (per cohort) in the OpenMetadataTopicConnector, etc classes upwards to the APIs

This would therefore not change any of the existing interfaces of a Connector while providing a default implementation of the logic that is based on already-existing and self-contained methods in the top-level abstract implementation (ConnectorBase), so I believe would also be backwards-compatible (?)

Thank you Chris @cmgrote for opening this issue. This will be useful information to know the status of cohort/s for a local server. The way it is proposed, it will be connector agnostic.

In addition to providing health check, we should also revisit the polling logic. Currently, Egeria tries to connect to Topic server in some sort of loop and fills up logs. This has been an issue whenever Kafka or server is not reachable. Logs are rolling over. We should poll in intervals not in a loop and also suppress logs if we can...

@guptaneeru probably multiple points here
a) Whether an audit event is generated within that connection attempt - I'd err on probably but haven't looked in enough detail at how tight that is.
b) The behaviour of the default audit log providers - for example it could handle repeated events better (last event occurred 10 times), or a wrapping logger could be provided.
c) More than b) The fact that the audit log framework is pluggable - so a new logger could be written to better suit your needs (including log cycling etc)?

Thank you @Nigel Nigle. How can I add our own audit logger?

@guptaneeru :

Info on the Audit Log Framework is at https://egeria.odpi.org/open-metadata-implementation/frameworks/audit-log-framework/
Using it at https://egeria.odpi.org/open-metadata-implementation/admin-services/docs/user/omag-server-platform-logging.html
The ALF code is at https://github.com/odpi/egeria/tree/master/open-metadata-implementation/frameworks/audit-log-framework
implementations of an audit logger at https://github.com/odpi/egeria/tree/master/open-metadata-implementation/adapters/open-connectors/repository-services-connectors/audit-log-connectors

In terms of the polling of kafka specifically. I've taken a look at the code. Issue #5681 touches on this but is very specifically about the state of the server. This issue is exploring specific approaches to understand more broadly the health of the system, which would likely address that issue. I think what's important is that the status can be understood at various levels (platform, server, connector(s) depending on the needs of the caller, and matching appropriate APIs that act upon the config)


General questions about SIFT3D outputs and usage

Dear Blaine,
I'm trying to motion correct a T1w EPI time-series, where the acquisition order of the slices is shuffled in each repetition, such that in each volume the slices have slightly different contrasts. As you can appreciate in the gif below, this change in contrast leads to an apparent "motion", characterized by a moving "nulling/dark band".

I've tried using ANTs with MI as a cost function for motion-correction, but my results are never as accurate as I would like them to be. I suppose this is because methods based on image intensities are confounded by the apparent change in contrast.
Recently, when reading a blog post on registering camera and satellite images, I learned about SIFT and decided to try those ideas to my problem. That is how I discovered your work.
Nonetheless, I haven't been able to get SIFT3D to work so far, but I suppose it's because I'm not using the CLI tools correctly.
Here's what I've tried:

Build SIFT3D with nifti support
Try registering two images with:

regSift3D --matches matches.csv \
          --warped warped.nii \
          --keys keys.nii.gz \
          --transform transform.csv \
          src.nii ref.nii

My output warped.nii looks wrong. Viewing it with freeview gives me a warning that the sform and qform of the warped.nii are incorrect, so I don't know whether the registration failed, or whether the nifti output is wrong.

I tried looking at transform.csv and figuring out whether I could use the values there to apply the transformation using AntsApplyTransforms.

The values in transform.csv look OK:
1.003655,0.006255,-0.005859,-0.649355
0.001542,1.000908,0.001695,-0.199589
-0.000418,-0.007386,1.009908,0.267528

I assume the first three columns represent a rotation matrix and the last column the translation coefficients. Is that correct?
Are the rotations computed with respect to the center of mass or the center of the image? Would you know the correct way to convert that output to ITK format?
Finally,  regarding the SIFT parameters, in your paper you cite the original SIFT work as a starting point to gain intuition about them, but from your experience which parameters matter most for accurately defining keypoints in brain MR images?
Thank you for sharing your work!
Daniel

Ok I do manage to convert the output to itk format by using Freesurfer's lta_convert.
For that I do the following:

Convert the transform.csv to a temporary transform.dsv file by changing commas to spaces.
Add a row with 0 0 0 1 to the txt file.
Call lta_convert  --infsl transform.dsv --src src.nii --trg trg.nii --outitk transform.txt

And once that is done I can apply the transform using AntsApplyTransforms without any issues.
The results at my end don't look perfect, but they don't look terrible either, so I guess I could improve them.
Another quick question though, is it possible to constrain SIFT3D to a rigid transform instead of affine, i.e. 6 DOF instead of 12?
Thank you!

Hi,
Thanks for sharing this very interesting application. I am eager to know if keypoint-based registration is suitable for this task. These methods tend to work well when there are local (not global) deformations between the objects or imaging protocols.
To answer your first question, the origin of the coordinate system is usually the top-left corner of the image. To be more precise, it is the first voxel stored in the file. AFAIK, the library reads the image raster directly; it does not apply any built-in image registrations that are supported in Nifti. This could definitely be improved, but it's tricky to keep track of the metadata. I usually use the Matlab wrappers and store the metadata separately.
Some Nifti readers complain about the image registration stored in the output Nifti files. Most will assume that the all-zeros matrix implies no registration should be used. If you need to preserve Nifti-specific metadata, Your approach of exporting the transform to another program seems appropriate.
As for your second question, the only transform that's currently supported is affine. However, you are free to export the matches, and do the RANSAC regression yourself with a different type of transform. I can provide some Matlab code for that. You are also free to implement some other transform in the C library, which would be even nicer :)
Thanks,
Blaine


Android device scrolling not working

How did you link to the library (autolinking, manual, rn link, cocoapods)?
Autolinking

What version of React Native?
0.74.5

What version of the library?
^1.4.0

iOS/Android version?
Android OS Version 34

Did the problem happen after updating React Native?
No

Are you using the library for the first time?
Yes

It's a bug? Provide a link to a minimal reproduction case.
scrolling in android device is not working

Have you checked the styling in the parent component? Please take a look at the limitations section on the README.


Upgrade pint==0.19.1

Upgrade pint, which now uses h to denote hours instead of Planck's constant (making my workaround redundant). Also prefer shorter units as a general rule.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 68.563%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
7034

Relevant Lines:
9732

💛  - Coveralls


RHIROS-445 use TableComposable component in diskUsageData function

Documentation requires update? :memo:

[ ] Yes
[X] No

Security Checklist :lock:
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
:guardsman: Checklist :dart:

[ ] Bugfix
[ ] New Feature
[X] Refactor
[ ] Unittests Added
[ ] DRY code
[ ] Dependency Added

Additional :mega:
Feel free to add any other relevant details such as links, notes, screenshots, here.

Codecov Report
Patch and project coverage have no change.

Comparison is base (f79ae88) 21.50% compared to head (dcb4072) 21.50%.

:exclamation: Current head dcb4072 differs from pull request most recent head 2c31d2d. Consider uploading reports for the commit 2c31d2d to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #197   +/-   ##
=======================================
  Coverage   21.50%   21.50%
=======================================
  Files          42       42
  Lines         865      865
  Branches      155      155
=======================================
  Hits          186      186
  Misses        632      632
  Partials       47       47

Impacted Files
Coverage Δ

src/Components/RosTable/RenderColumn.js
13.04% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@r14chandra , could you please attach screenshots that shows before-after change?
It would be helpful while reviewing.

@r14chandra , could you please attach screenshots that shows before-after change? It would be helpful while reviewing.

@kgaikwad There is no change in the UI. It is just a code refactor. But I will attach the screenshots in the description.


x/text: panic of index out of range in language.Matcher

What version of Go are you using (go version)?
go 1.9
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
amd64, darwin and linux.
What did you do?
language.MustParse accepts "lang_ZZ", e.g. en_ZZ, es_ZZ, etc. and returns a valid tag.
but m.Match(such a language tag) triggers a panic of "index out of range" at this lookup table: https://github.com/golang/text/blob/master/language/tables.go#L50
Sample code to reproduce the issue:
package main

import (
	"fmt"
	"golang.org/x/text/language"
)

func main() {
	supportedLanguages := []language.Tag{
		language.English,
		language.Arabic,
		language.BritishEnglish,
		language.Spanish,
		language.French,
		language.Italian,
		language.Japanese,
		language.Korean,
		language.Portuguese,
		language.Russian,
		language.SimplifiedChinese,
		language.TraditionalChinese,
	}

	tl := language.MustParse("en_ZZ")
	fmt.Printf("target language: %v\n", tl)

	m := language.NewMatcher(supportedLanguages)
	_, idx, _ := m.Match(tl)

	fmt.Printf("index matched: %v\n", idx)
}

Output:
target language: en-ZZ
panic: runtime error: index out of range

goroutine 1 [running]:
golang.org/x/text/language.regionGroupDist(0x139005701650135, 0x1650139)
	/Users/yb/go_root/src/golang.org/x/text/language/match.go:677 +0x13d
golang.org/x/text/language.(*bestMatch).update(0xc42003fbf0, 0xc4200722d0, 0x1650139, 0x0, 0x0, 0x101650057)
	/Users/ybi/go_root/src/golang.org/x/text/language/match.go:622 +0x28d
golang.org/x/text/language.(*matcher).getBest(0xc420072270, 0xc42000adc0, 0x1, 0x1, 0x0, 0x0, 0x0, 0x0, 0x0)
	/Users/yb/go_root/src/golang.org/x/text/language/match.go:504 +0x4a0
golang.org/x/text/language.(*matcher).Match(0xc420072270, 0xc42000adc0, 0x1, 0x1, 0x0, 0x0, 0x1161220, 0xc420072270, 0x2d952e1a19cc2e08)
	/Users/yb/go_root/src/golang.org/x/text/language/match.go:83 +0xd4
main.main()
	/Users/yb/test.go:28 +0x4d5
exit status 2

What did you expect to see?
either
(1) language.MustParse rejects "lang_ZZ" and throws an error;
or
(2) language.MustParse parses "lang_ZZ" to a tag of just "lang", similar to "lang_UND", e.g. language.MustParse of "en_ZZ" and "en" should return the identical result.
or
(3) language.Matcher supports ZZ country code and works properly.
What did you see instead?
language.MustParse accepts ZZ, but language.Matcher crashes on that.

Same reproduces in 1.10.3 darwin/amd64

This is still happening even on 1.11.1 darwin/amd64

Yes, because it is not fixed yet.
/cc @mpvl

Has this been fixed yet? It is a year later and I'm at go1.12.4 darwin/amd64 and it is occurring to my code.
What is actually wrong with the input? Maybe if I knew what was wrong with the input I could screen for it.
This is our production code used by hundreds of different people so I don't know in each case what is sent in by them.

Not familiar with the code that well and this could just be a symptom of a larger problem, but it looks like the cause of the panics are because of one byte in the regionToGroups byte array in the language/tables.go file that was removed (accidentally?) during the code refactoring on February 23, 2018.
https://github.com/golang/text/commit/b85db4f711a0f40224ea3a0b741a9f8c046fcb7f#diff-9b4a7a0caec1562de30551ab14452668L3346


More Features?

Hi!
I really appreciate this widget! Any plans to add more features? Such as due date, subtasks and notes?
Thanks!

Thanks for your comment @BigOlBooty Sorry I don't have a plan so far to continue working on this. Hope somebody can fork and move forward.


Support source language auto-detection in Translate

Issue #, if available:
#620
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

TODO: Test to make sure things don't fail if the identified language is in the list of target languages for translate or webcaptions.

It seems to make one duplicate listing of the source language subtitles in videojs. It also lists the source language in the Translation tab. The correct behavior is only list the source language once in the videojs subtitle options and not to list the source language at all in the Translation tab.

Alex already reviewed this PR, so I'm going to force the merge.


AddRowAPIEnabled does not work

Hi there,
You have the flag addrowapienabled but it is not working as it should. I should be able to add rows to the toable yet the Table throws an exception because the type is "AsAzure" instead of push.
Otherwise great library!!! Keep up the good work!

please make sure that the dataset was also created with the library (or with the API directly) with the appropriate settings (Push-Dataset)
You cannot just add rows to any Dataset
If the type is "AsAzure" then this actually means that the dataset connect to Azure Analysis Services and the type of a dataset cannot be changed after it was created

Gerhard,
the dataset was created through the powerbi UI. But I get the same result
when creating it through the API, it gets returned as "Azure" the second
time I connect.
Carsten
Skype: hesscarsten
Tel:<PHONE_NUMBER>
https://calendly.com/carstenhess
On Mon, Apr 8, 2019 at 12:28 AM Gerhard Brueckl<EMAIL_ADDRESS>wrote:

please make sure that the dataset was also created with the library (or
with the API directly) with the appropriate settings (Push-Dataset)
You cannot just add rows to any Dataset
If the type is "AsAzure" then this actually means that the dataset connect
to Azure Analysis Services and the type of a dataset cannot be changed
after it was created
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gbrueckl/PowerBI.API.Client/issues/13#issuecomment-480679207,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AN5iNX96PyaeSTM3ZpTusDYdRwUZAVhOks5vesV8gaJpZM4ccb11
.

When creating it, did you use PBIDefaultMode.Push?
Please see https://github.com/gbrueckl/PowerBI.API.Client/blob/master/SampleApplication/Program.cs line 173 onward where you can see a full sample of how to use the Push-API

Yes I used push as the default mode.
Carsten
On Mon, Apr 8, 2019 at 7:21 AM Gerhard Brueckl<EMAIL_ADDRESS>wrote:

When creating it, did you use PBIDefaultMode.Push?
Please see
https://github.com/gbrueckl/PowerBI.API.Client/blob/master/SampleApplication/Program.cs
line 173 onward where you can see a full sample of how to use the Push-API
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gbrueckl/PowerBI.API.Client/issues/13#issuecomment-480791328,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AN5iNWJLgCyz-5TrUOu5tnutJH8941wPks5veyZCgaJpZM4ccb11
.

and did you also try this when creating a whole new dataset and not using an existing?
can you run the sample code just to test whether it is a general bug or something specific to your environment/setup/code?

closing due to no further feedback


Re: 'Local Land Charges Search'

Problem with 'Local Land Charges Search' (https://www.api.gov.uk/hmlr/local-land-charges-search/)
Can the two asterisks please be removed from this line:
https://bgtest.landregistry.gov.uk/b2b/ECLLC_StubService**
Many thanks,
Simon.

Hi, can this please be bumped up please?

A fix has been has been merged. @SPD-LR Could you please check the entry to make sure you are happy with the fix.

Hi Rob,
That's great, much appreciated.
Simon.
From: Rob Nichols @.>
Sent: 24 March 2023 10:06
To: co-cddo/api-catalogue @.>
Cc: Devey, Simon @.>; Mention @.>
Subject: Re: [co-cddo/api-catalogue] Re: 'Local Land Charges Search' (Issue #595)
A fix has been has been merged. @SPD-LRhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSPD-LR&data=05|01|simon.devey%40landregistry.gov.uk|507d9f747267411d2add08db2c4f6dce|236309a8284b4e57a7b3fc767736f69d|0|0|638152491889252187|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=AhBXnC%2BqWP5OUqkuJESWQkg%2B7ZWwewK6o%2Bcq%2BKHRR%2BE%3D&reserved=0 Could you please check the entry to make sure you are happy with the fix.

Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fco-cddo%2Fapi-catalogue%2Fissues%2F595%23issuecomment-1482548804&data=05|01|simon.devey%40landregistry.gov.uk|507d9f747267411d2add08db2c4f6dce|236309a8284b4e57a7b3fc767736f69d|0|0|638152491889252187|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=7dUYx%2B8VSg1IQunNJ%2B5mWEGW6BjJvDsmQfM8o52HwvI%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA3AYUWHOBIK4YGJESKVEWHTW5VW2BANCNFSM6AAAAAAVCC2PHQ&data=05|01|simon.devey%40landregistry.gov.uk|507d9f747267411d2add08db2c4f6dce|236309a8284b4e57a7b3fc767736f69d|0|0|638152491889252187|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=HIWeY9ANZVhHYSH5tzSO8ZcAvv%2Bav1j7rUc5XgIJRIc%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>HM Land Registry's purpose is to protect your land ownership and provide services and data that underpin an efficient and informed property market. Our vision is: "A world-leading property market as part of a thriving economy and a sustainable future".
We check all mail and attachments for known viruses. However, you are advised that you open any attachments at your own risk. If you have received this email and it was not intended for you, please let us know, then delete it.
We welcome correspondence in English and Welsh.
To see how HM Land Registry treats your personal information, read our Personal information charter: www.gov.uk/government/organisations/land-registry/about/personal-information-charter.


The message was not confirmed by RabbitMQ

We are seeing the following happening every so often:
Content-Type:	application/vnd.masstransit+json
MT-Reason:	fault
MT-Fault-Message:	rabbitmq://rmqcluster:5672/bus-XPRMW1PAPP02-xactprmservices-rrdoyybkdo6kt4o3bdkyucmjr5 => The message was not confirmed by RabbitMQ
MT-Fault-Timestamp:	2016-11-10T21:54:47.6453945Z
MT-Fault-StackTrace:	at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at MassTransit.RabbitMqTransport.Contexts.RabbitMqModelContext.<MassTransit-RabbitMqTransport-ModelContext-BasicPublishAsync>d__17.MoveNext()
at MassTransit.RabbitMqTransport.RabbitMqSendTransport.<>c__DisplayClass6_0`1.<<MassTransit-Transports-ISendTransport-Send>b__1>d.MoveNext()
at MassTransit.RabbitMqTransport.RabbitMqSendTransport.<>c__DisplayClass6_0`1.<<MassTransit-Transports-ISendTransport-Send>b__1>d.MoveNext()
at MassTransit.Pipeline.Filters.AsyncDelegateFilter`1.<Send>d__3.MoveNext()
at MassTransit.RabbitMqTransport.Pipeline.PrepareSendExchangeFilter.<MassTransit-Pipeline-IFilter<MassTransit-RabbitMqTransport-ModelContext>-Send>d__5.MoveNext()
at MassTransit.RabbitMqTransport.Integration.RabbitMqModelCache.<SendUsingExistingModel>d__11.MoveNext()
at MassTransit.RabbitMqTransport.RabbitMqSendTransport.<MassTransit-Transports-ISendTransport-Send>d__6`1.MoveNext()
at MassTransit.Pipeline.Filters.DeserializeFilter.<Send>d__4.MoveNext()
at MassTransit.Pipeline.Filters.RescueReceiveContextFilter`1.<MassTransit-Pipeline-IFilter<MassTransit-ReceiveContext>-Send>d__5.MoveNext()
MT-Host-MachineName:	XPRMW1PAPP02
MT-Host-ProcessName:	xactprmservices
MT-Host-ProcessId:	34596
MT-Host-Assembly:	xactprmservices
MT-Host-AssemblyVersion:	<IP_ADDRESS>61
MT-Host-MassTransitVersion:	<IP_ADDRESS>8
MT-Host-FrameworkVersion:	4.0.30319.34209
MT-Host-OperatingSystemVersion:	Microsoft Windows NT 6.2.9200.0

What does this mean?

Does this have to do with publish confirms? Is there a way to tell MassTransit not to enable those?

You can configure the Host (or the endpoint, depending upon which version you are using) to ignore PublisherConfiguration by setting:
cfg.PublisherConfirmation = false;
This will ignore confirmations from RabbitMQ on published messages.

Hey guys,
Can I use cfg.PublisherConfirmation = false; in MassTransit v 2.9.
I sent a message with ExpireDate and sometimes I get an exception "publisher did not confirm message"

If you get that error look into the broker RabbitMQ logs for related errors.

check the times from the servers. If they are different enough they will through this error as MassTransit performs a check related these times.


Unexpected tokens in vcd that loads in gtkwave

Running the load_signals example on the attached vcd  (zipped to be able to upload here) file yields the following output:
It took 59.343885ms to load the header of /data/study/master/thesis/work/toplevel/out.vcd
[00:00:00] ░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ 0 B (0.000%)                                                                                thread 'thread '<unnamed>' panicked at wellen/src/wavemem.rs:490:25:
index out of bounds: the len is 7993 but the index is<PHONE_NUMBER>1105
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
<unnamed>' panicked at wellen/src/vcd.rs:1171:33:
Unexpected tokens: `jm` and `0km` (1067 lines after header)
thread '<unnamed>' panicked at wellen/src/vcd.rs:1171:33:
Unexpected tokens: `E^` and `0F^` (81 lines after header)
thread '<unnamed>' panicked at wellen/src/vcd.rs:1171:33:
Unexpected tokens: `$D` and `0%D` (833 lines after header)
thread '<unnamed>' panicked at wellen/src/vcd.rs:1171:33:
Unexpected tokens: `aJ` and `0bJ` (155 lines after header)
thread '<unnamed>' panicked at wellen/src/vcd.rs:1171:33:
Unexpected tokens: `9s` and `0:s` (197 lines after header)
thread '<unnamed>' panicked at wellen/src/vcd.rs:1171:33:
Unexpected tokens: `vX` and `0wX` (943 lines after header)
thread '<unnamed>' panicked at wellen/src/vcd.rs:1171:33:
Unexpected tokens: `!/` and `0"/` (705 lines after header)
thread '<unnamed>' panicked at wellen/src/vcd.rs:1171:33:
Unexpected tokens: `j5` and `0k5` (47 lines after header)
[00:00:00] ███████████████████████░░░░░░░░░░░░░░░░░ 336.33 kB (57.560%)

while gtkwave loads it without problems.
If you need a smaller file I can probably produce one, let me know if that would be useful.

I am looking into this now (finally!).
This is what wellen is currently stumbling over:
$var wire 0 H read_idx $end
...
b H

I.e., we currently do not deal well with 0-bit signals

This should be fixed in wellen 0.13.3.
Please re-open this issue if the problem persists.


Yarn workspaces  - EventSource's response has a MIME type ("text/html") that is not "text/event-stream"

I've a project with a yarn workspaces architecture, following this example form the next.js documentation, that looks like:
packages
|- webapp
| |- pages
| |- ...
| |- server.js
|- libraryA
|- libraryB
server.js looks like
const cacheableResponse = require('cacheable-response');

const ssrCache = cacheableResponse({
  get: async ({ req, res, pagePath, queryParams }) => ({
    data: await app.renderToHTML(req, res, pagePath, queryParams),
    ttl: 7200000 // 2 hours
  }),
  send: ({ data, res }) => res.send(data)
});

app.prepare()
  .then(() => {
    const server = express();

    server.use('/_next', express.static(path.join(__dirname, '.next')));
    server.use('/static', express.static(path.join(__dirname, 'static')));

    ....

    server.get('/account', (req, res) => {
        return ssrCache({ req, res, pagePath: '/account' });
    });

    ...
  }
);

And I'm getting this errors on the client
EventSource's response has a MIME type ("text/html") that is not "text/event-stream". Aborting the connection.

 Error with on-demand-entries-ping: Unexpected token < in JSON at position 0

Which makes the website eventually fail until I make several hard refreshes.
Do you have any idea why?
Or maybe do you have a working example with yarn workspaces?
Also I had to specify server.use('/_next', express.static(path.join(__dirname, '.next'))); and server.use('/static', express.static(path.join(__dirname, 'static'))); because it didn't work without it.
Thanks 🙂

No idea about yarn.
Did you try just the nextjs example, without your extra specific things?


Bump the sbt-spiewak plugin

We should get out this mostly unscathed: I think all our repos are main instead of master.  If we've missed any, let's fix them.
Also lets us turn off fail fast, which I think we should do on the main repo, where the build is less stable.
Will release this as 0.7.0.

I am wondering whether we really want to use sbt release (and not rather sbt ci-release) here? What about adding
githubWorkflowPublish := Seq(WorkflowStep.Sbt(List("ci-release")))

both in Http4sOrgPlugin and build.sbt?

I am wondering whether we really want to use sbt release (and not rather sbt ci-release) here? What about adding
githubWorkflowPublish := Seq(WorkflowStep.Sbt(List("ci-release")))

both in Http4sOrgPlugin and build.sbt?

Nevermind, I guess https://github.com/djspiewak/sbt-spiewak#ci-release is supposed to be used in consumbers of sbt-http4s-org?

Nevermind, I guess https://github.com/djspiewak/sbt-spiewak#ci-release is supposed to be used in consumbers of sbt-http4s-org?

Yes, I've had good luck with that release plugin from sbt-spiewak.  The only rough edge I've found is that it uses a different PGP plugin than many are used to, and the PGP keys need to be regenerated without a passphrase.

Yes, I've had good luck with that release plugin from sbt-spiewak.  The only rough edge I've found is that it uses a different PGP plugin than many are used to, and the PGP keys need to be regenerated without a passphrase.


After update I get "Theory must have test data" for AutoData attributes

I have updated to latest xunit for visual studio 2017 and also updated all packages for xunit including AutoFixture. And now I get the compiler error:  "Theory must have test data"
 [Theory, AutoData]         public void CanMapSearchDirectionToModel(SearchDirection direction)         {  ..... }
I have no clue on have to fix it, can you help me?
this is my package.config file
<?xml version="1.0" encoding="utf-8"?>
<packages>
  <package id="AutoFixture" version="3.51.0" targetFramework="net461" />
  <package id="AutoFixture.Xunit" version="3.51.0" targetFramework="net461" />
  <package id="AutoMapper" version="6.1.1" targetFramework="net461" />
  <package id="Castle.Core" version="4.2.0" targetFramework="net452" />
  <package id="FluentAssertions" version="4.19.4" targetFramework="net461" />
  <package id="Microsoft.AspNet.WebApi.Client" version="5.2.3" targetFramework="net452" />
  <package id="Microsoft.AspNet.WebApi.Core" version="5.2.3" targetFramework="net452" />
  <package id="Microsoft.Owin" version="3.0.1" targetFramework="net452" />
  <package id="Microsoft.Owin.Hosting" version="3.0.1" targetFramework="net452" />
  <package id="Microsoft.Owin.Security" version="3.0.1" targetFramework="net452" />
  <package id="Microsoft.Owin.Testing" version="3.0.1" targetFramework="net452" />
  <package id="Miracle.Settings" version="<IP_ADDRESS>" targetFramework="net461" />
  <package id="Moq" version="4.7.137" targetFramework="net452" />
  <package id="morelinq" version="2.6.0" targetFramework="net461" />
  <package id="Newtonsoft.Json" version="7.0.1" targetFramework="net452" />
  <package id="Owin" version="1.0" targetFramework="net452" />
  <package id="SemanticComparison" version="3.51.0" targetFramework="net461" />
  <package id="ServiceStack.Common" version="3.9.71" targetFramework="net45" />
  <package id="ServiceStack.Text" version="3.9.71" targetFramework="net452" />
  <package id="SimpleInjector" version="2.8.3" targetFramework="net452" />
  <package id="System.ValueTuple" version="4.4.0" targetFramework="net461" />
  <package id="xunit" version="2.3.0" targetFramework="net461" />
  <package id="xunit.abstractions" version="2.0.1" targetFramework="net461" />
  <package id="xunit.analyzers" version="0.7.0" targetFramework="net461" />
  <package id="xunit.assert" version="2.3.0" targetFramework="net461" />
  <package id="xunit.core" version="2.3.0" targetFramework="net461" />
  <package id="xunit.extensibility.core" version="2.3.0" targetFramework="net461" />
  <package id="xunit.extensibility.execution" version="2.3.0" targetFramework="net461" />
  <package id="xunit.extensions" version="<IP_ADDRESS>6" targetFramework="net461" />
</packages>

Yep, it's our good old known issue about xUnit integration package. Please install AutoFixture.Xunit2 instead of AutoFixture.Xunit. The latter one is for xunit 1 😄
It will be fixed since v4: https://github.com/AutoFixture/AutoFixture/pull/835.

Closing this one as I'm 99.9% sure my suggestion solves the issue. Please re-open if it doesn't.


(? - 23w31a) Scroller

This texture file was added in 23w32a, so I presume this was a hardcoded, generated texture before this point.


Added Programiz.com, a free step-by-step tutorial for Python Beginners

This commit includes Programiz.com, which offers free, step-by-step Python tutorials designed for beginners.
With its easy-to-follow programming tutorials and interactive coding interfaces, learners can practice and test their coding skills directly on the website.

Sorry, we don't really advertise like this.


How to store relation id as object id?

@jenssegers
I hope you are doing good!
I have an issue with relationship id, which is store as string.
How I can store that companyId as Object id?
$user = new User;
$user->companyId        = $companyId;
$user->save();

any solutions?

I was looking into that briefly a while ago and didn't find a quick solution without having to extend or rewrite the library (and large parts of it when you're dealing with many-to-many relations). The issue is that the foreign key assignment goes through the normal accessors and converts ObjectId's to strings.
In addition to that Eloquent uses the id as an array index at some point while managing the relations which doesn't work if the foreign keys aren't strings or integers.
Might look into it again in the future but might not be able to find a solution. The main issue for us was not being able to use lookups during aggregation since MongoDB can't automatically convert between string and object ids.
It would probable require a major change to the functionality of the library basically making it incompatible with data of the current version and thus not be developed. Just speculating though.

$user = new User; $user->companyId = new MongoDB\BSON\ObjectId($companyId); $user->save();

That only works on one-to-many relations, not many-to-many and also means you have to manually store the foreign key.
If a customer can have many users and a user can belong to many customers it stores the ids on both objects as an array of strings. And it will crash when loading the relations if you try to manually store them as ObjectIds instead. At least with this kind of relation it is not possible to use the relation functionality of the ORM at all if you need the foreign keys to be stored as objects.

I also believe that a major change is required to achieve this.
I did it in a project I'm working on by extending almost all the relationship classes (like BelongsToMany, HasMany, etc), and overwriting their getParentKey() method (to return a MongoDB\BSON\ObjectId instance), as well as the BelongsToMany::attach(), BelongsToMany::buildDictionary() among other methods.
I'm sure my implementation does not cover all the functionality of relationships, so I will work on it from time to time.

I also believe that a major change is required to achieve this.
I did it in a project I'm working on by extending almost all the relationship classes (like BelongsToMany, HasMany, etc), and overwriting their getParentKey() method (to return a MongoDB\BSON\ObjectId instance), as well as the BelongsToMany::attach(), BelongsToMany::buildDictionary() among other methods.
I'm sure my implementation does not cover all the functionality of relationships, so I will work on it from time to time.

Can you share your implementation?


ModuleNotFoundError: No module named 'ts.torch_handler.handler_box'

Hi @ben-omji
How to solve this issue?
ModuleNotFoundError: No module named 'ts.torch_handler.handler_box'
Best regards,
PeterPham

@ThanhPham1987
Could you give me more information about your issue?
And please show me if you have any changes to the code.

Hi @ben-omji . When I run this command line:./run_ts.sh coco_yolov5s.mar. And I access container. I see the log inside container. I see this bug "ModuleNotFoundError: No module named 'ts.torch_handler.handler_box' "
Best regards,
PeterPham

@ThanhPham1987
Updating torch serve repository version from 0.7.1 -> 0.8.1 solve that issue.
Check the line 17 of setup.sh


GWRResultsLite class

Returning a lighter GWRResultsLite class in bandwidth searching, so it can use maths in diagnostics.py.
I perfer using _compute_betas_gwr  for simplicity for now, since I found spglm.iwls is also doing global GLMs, and I am afraid of messing up with that.

Look ok to me. I don't quite follow your second point about _compute_betas_gwr versus spglm.iwls...when you say messing with it, is that in reference to exploring sparse data structures?
Do you have any benchmark of the speedups of using the Lite results class versus not using it?

I think spglm.iwls serves a more general purpose not only for gwr but also for other models. The only line needs for a gaussian gwr is this, so I think its more straightforward to just call it.

Sure makes sense, I thought that was what you were already doing from your
previous optimization PR, so wasn't sure what the alternative would be. The
simpler, the better!
On Sun, Mar 18, 2018 at 11:00 AM, Ziqi Li<EMAIL_ADDRESS>wrote:

I think spglm.iwls serves a more general purpose not only for gwr but
also for other models. The only line needs for a gaussian gwr is this
https://github.com/pysal/spglm/blob/b225342e941faf828d5cb071fe705c4ca57c30b1/spglm/iwls.py#L76,
so I think its more straightforward to just call it.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/pysal/gwr/pull/25#issuecomment-374024112, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHvdzWC2dyv_atE3CYAT_7us8_NR63S4ks5tfqDCgaJpZM4Ssz2k
.

Haven't done a benchmark, but after this PR, the expectation should be very similar to this #19, just made it more organized~

Ok, just trying to get a sense of where the speed-up comes from i.e.,
primarily from the avoidance of collecting as many model output or simply
from using _compute_betas_gwr instead of the full iwls routine.
On Sun, Mar 18, 2018 at 11:11 AM, Ziqi Li<EMAIL_ADDRESS>wrote:

Haven't done a benchmark, but after this PR, the expectation should be
very similar to this #19 https://github.com/pysal/gwr/issues/19, just
made it more organized~
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/pysal/gwr/pull/25#issuecomment-374025955, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHvdze8woQ7nMBlclb0zoNNf73aPjioWks5tfqNsgaJpZM4Ssz2k
.

If total runtime is 185 seconds, it is like return GWRResults() takes ~35 secs, and iwls takes ~130 secs. After speedup, total is 27 seconds, it is like _compute_betas_gwr takes 24 secs, and return GWRResultsLite() takes nearly zero seconds. I would say major speedup from not calling iwls but returning a lighter output also matters.

Ok, sounds good!


Fix: The pull requests view uses the Link-muted CSS class to display inline usernames

This PR ensures that users their actual name is displayed on the pull requests overview page.
Before:

After:

Presumably, at some point github changed from muted-link to Link--muted:

@ForbesLindesay : Is this still of interest?

Closing this out as there doesn't seem interest and I'd like to keep my created PR list at 0.


Add customQuery option

In some cases, the source url has another parameter name like q instead of query, so this commit adds customQuery to replace that name.

Thanks @THernandez03
This can be done with combine option this way:
combine: ({ query }) => ({
  q: query
})

I have updated the readme and passed object with default params to combine function.
p.s. Better late than never %)


[Documentation Update]: Ethical Rules

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description
I would like to see an update in documentation regarding ethical code of conduct of the contributors, people should not be solving the problems identified by someone who raised the issue and is capable of solving the problem by own. One should preferably find or create issue who is not assigned to anyone
Suggested Change
Write it in form of "Ethical Rules" point wise
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Full Name
Om Karmakar
Participant Role
GSSoc extended ,Hacktoberfest

@UTSAVS26 Please assign me, I have submitted the PR already, also do give the tabels of Hacktoberfest and GSSoc extended
Thank you!


[MockFileSystem] Directory discover could return the directory self

Consider the following code:
      var mfs = new MockFileSystem();
      mfs.Directory.CreateDirectory("dir");
      var directoriesRelative = mfs.Directory.GetDirectories("dir");
      var directoriesAbs = mfs.Directory.GetDirectories(mfs.Path.GetFullPath("dir"));

You will see that directoriesRelative contains the dir folder itself, while directoriesAbs is empty as expected.
This is a critical bug because it forces to rewrite production code to cover it later by tests.
Likely, in order to fix the issue the following two lines should be reordered: MockDirectory.cs.

Hi Zvirja, I am looking into your issue.
Can you elaborate which lines you mean?
only one line is highighted

Good catch :) I believe that these two lines should be reordered among themselves.
Will be really thankful if this bug is fixed.

Fixed in #258, shipped to NuGet as <IP_ADDRESS>


docker tags

Is it possible for you to push docker releases with same release tags?
That would make it possible to allow more stable deployments using for example a specific release tag
docker pull mrin9/rapidoc:7.4.2

Yes, I need to streamline the docker build pipeline, as on today, it triggers a build whenever something is pushed to github.

Gr8. Nerver trind it bit this might help.
https://github.com/marketplace/actions/build-tag-publish-docker

From release 8.0.0 docker images can be pulled using tags
https://hub.docker.com/r/mrin9/rapidoc/tags


Failed assertion in read_frame_op

I am operating some long running websocket clients and every few days I get the assertion failed in do_close case of the switch statement:
Assertion `! d.ws.wr_block_' failed.
Is this a bug in the library or does it imply I am doing something wrong?
This was beast/websocket/impl/read.ipp line 454 as of commit 1ab7a2f04ca9a0b35f2032877cab78d94e96ebad.

This means you are trying to close when you have an active write. Close counts as a write, you can't do both simultaneously.

I should add some comments around that assert, it happens semi-often.

I am not calling any close function. Is there something else that triggers the close?

If you have a read operation pending, and a close frame is received, the implementation will automatically send a close frame in response. Although a write is performed at the low level, the implementation coordinates with caller-initiated writes to ensure that there is no block (wr_block_ is a flag indicating that a write operation is in progress).
It is possible, although unlikely, that a bug exists. I will investigate. This assertion can also happen if you attempt two writes at the same time. In the meanwhile, if you want to help track this down please review your code and make absolutely certain that you are not trying to perform two writes at the same time.

Upon reviewing the asynchronous read implementation, I can see how there might be a small window allowing a race condition. Please give this branch a try and let me know if any of the new asserts go off:
https://github.com/vinniefalco/Beast/commits/fix

I'm running the new branch now.

I'm having a similar problem. But it crashes instead.
I never call close nor delete the connections. I always have a pending read, but writes are made synchronously and I only have one thread.

@felipealmeida Did you run the debug version?

I'm always compiling with debug, yes.

I'm using C++1y in GCC

The problem is always reproducible in my case. I'm not sure it is a Beast bug however.

OK.  I'm moving the websocket stream while a pending operation exists, by using a std::vector.

Ah, you can't move websocket streams while there are pending operations :) Good catch. But I still think there's a bug from the original description in this issue. I have some ideas about it, I will push a possible fix shortly.

I rewrote history in the fix branch to include a possible fix, please try it and let me know if the asserts go away (don't forget to run a debug build!)
https://github.com/vinniefalco/Beast/commits/fix

So...I guess you haven't seen the assert anymore?


detecting focus on any field ?

this might be very obvious but how can i detect that field has focus , because on mobile the keyboard moves up and i need to adjust the UI accordingly.

Each Field component is given a meta.active flag that you could detect changing from false to true. Something like that?

Yes, but it would be nice if it was in the passed down props. Because if I
use mapStateToProps to pass in that in, the app throws an error on startup
because the that state has not been initialized in redux.
On Sep 15, 2016 7:51 AM, "Erik Rasmussen"<EMAIL_ADDRESS>wrote:

Each Field component is given a meta.active
http://redux-form.com/6.0.5/docs/api/Field.md/#-meta-active-boolean-
flag that you could detect changing from false to true. Something like
that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/erikras/redux-form/issues/1777#issuecomment-247351136,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANOMEc7aqOM39cL7fwDGDRMudM9gMUOLks5qqVt5gaJpZM4J98-l
.

+1
I am looking to implement a checkout payment credit card form with a visual credit card reference (using https://github.com/JohnyDays/react-credit-card) and I need to tell the Card component which field currently holds focus (so that the visual reference can spin around to display the CVC when the user is inputting that information).

Ideally the form's would receive a prop containing the entire 'state' (values, meta, errors etc) of its Fields and the user could destructure the state accordingly.
Currently I'm planning to use the parse/normalize hooks and store the focus state myself.

Not passing all the values/meta/errors as props as a default behaviour is, to my understanding, an intentional part of the api that helps keep it performant by default.  Otherwise you drift towards the v5 api which was revamped partly because of this problem.
Closing this usage question as it is nine months old since last update. There are now many ways provided to access the state information required, namely the provided selectors.


AMQ-4598 purge enhancement

Reset queue counter during "purge" to the number of inflight messages, so that they are not "hidden" while still being processed.

This issue was fixed in the upstream, mind closing the PR ?


Fix features for gizmo menu

Brief description
Fix features for the Gizmo Menu project settings (shortcut for python type of usage and file type of usage functionality)

Same problem here, solved with this MR.

thanks @friquette ;)


Fix soundness bug in aggregator pi datahash

Description
constraint
// 3 batch_data_hash and chunk[i].pi_hash use a same chunk[i].data_hash when
// chunk[i] is not padded
//
// batchDataHash = keccak(chunk[0].dataHash || ... || chunk[k-1].dataHash)
//
// chunk[i].piHash =
//     keccak(
//        &chain id ||
//        chunk[i].prevStateRoot ||
//        chunk[i].postStateRoot ||
//        chunk[i].withdrawRoot  ||
//        chunk[i].datahash)

was implemented incorrectly. The one gets implemented was
// 8. batch data hash is correct w.r.t. its RLCs

This PR fixes this issue.
Issue Link
[link issue here]
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Contents

[item]

Rationale
[design decisions and extended information]
How Has This Been Tested?
tested mock  prover with mock chunks
cargo test --release

closed as integrated to #712


python312Packages.nibabel: unbreak by disabling test requiring distutils

Description of changes
Package presumably never worked with Python 3.12 but some dependency was recently enabled for this interpreter.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 301089 run on x86_64-linux 1

12 packages failed to build:

python312Packages.dcmstack
python312Packages.dcmstack.dist
python312Packages.dipy
python312Packages.dipy.dist
python312Packages.heudiconv
python312Packages.heudiconv.dist
python312Packages.nipype
python312Packages.nipype.dist
python312Packages.pybids
python312Packages.pybids.dist
python312Packages.templateflow
python312Packages.templateflow.dist

16 packages built:

python312Packages.dicom2nifti
python312Packages.dicom2nifti.dist
python312Packages.nibabel
python312Packages.nibabel.dist
python312Packages.nilearn
python312Packages.nilearn.dist
python312Packages.nipy
python312Packages.nipy.dist
python312Packages.nitime
python312Packages.nitime.dist
python312Packages.nitransforms
python312Packages.nitransforms.dist
python312Packages.pymedio
python312Packages.pymedio.dist
python312Packages.torchio
python312Packages.torchio.dist

all failures exist on master


Implement Numa affinity for worker threads

This PR introduces numa awareness for the static_thread_pool.
Summary of Changes

[x] There is a new CMake option STDEXEC_ENABLE_NUMA to explicitly opt-in numa awareness. Notice that we need to link stdexec with libnuma, if enabled. We might also introduce a new CMake target especially for exec or for exec::static_thread_pool.
[x] The constructor of static_thread_pool takes an additional pointer to exec::numa_policy, which defaults to exec::get_default_numa_policy(). This Numa policy defines a distribution mapping, which maps a thread number to a Numa node. It also provides a member function to bind the current thread to a specified Numa node. If STDEXEC_ENABLE_NUMA is false, then the numa policy is doing nothing.
[x] We use a stateful exec::numa_allocator<T> that allocates memory on a specified numa node.
[x] Introduce member functions static_thread_pool::get_scheduler_on(numa_node_mask) and static_thread_pool::get_scheduler_on(cpu_mask) to return a scheduler that schedules with specified constraints.
[x] Worker threads try to steal from other worker threads on their own Numa node.  When this fails a certain amount of times every other worker thread becomes a stealing target.

API Design
This PR introduces the interface numa_policy which is defined as
  struct numa_policy {
    virtual std::size_t num_nodes() = 0;
    virtual std::size_t num_cpus(int node) = 0;
    virtual int bind_to_node(int node) = 0;
    virtual std::size_t thread_index_to_node(std::size_t index) = 0;
  };

The thread pool takes a pointer to numa_policy as an optional argument to customize the distribution of worker threads to Numa nodes.
The static_thread_pool has the following ways to get a scheduler with certain properties
    // Returns a scheduler without any constraints
    scheduler get_scheduler() noexcept;

    // Returns a scheduler that schedules on a specific worker thread (enumerated from 0...N-1)
    scheduler get_scheduler_on_thread(std::size_t threadIndex) noexcept;

    // Returns a scheduler that schedules only on worker threads that run on one of the specified nodes
    scheduler get_constrained_scheduler(const nodemask& constraints) noexcept;

Benchmarks
We perform a benchmark on a machine with 2 numa nodes. Each node has 14 cores (28 threads).
  Model name:            Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz
    CPU family:          6
    Model:               79
    Thread(s) per core:  2
    Core(s) per socket:  14
    Socket(s):           2

We perform the nested schedule benchmark from the last PR and we see that the scaling is better with numa affinity enabled.
The OS scheduler does an amazing job when having less than 28 threads.

Max throughput

No Numa
With Numa

5.27+e08
6.4e+08

Which is an improvement of roughly 20%

@maikel I can run this code if you need numa machines to benchmark on.

/ok to test

/ok to test

/ok to test

We have discussed that the planned improvements to this happen in an extra PR.

/ok to test


make it work with shadow dom

This seems to fix shadow dom issues.
The basic issue is that with a dom structure like :

document

body

div#shadow-host

#shadow-root

div#wrapper

onClickOutsideDecoratedComponent

div#clicker

In the above case the mousedown listener is registered on document and when clicking on div#clicker the handler is called with an event with target = div#shadow-host where as we really want it to start at div#wrapper
So the solution is to add the event listeners to both document and #shadow-root
However this causes the handler is triggered twice, once with event.target = div#clicker, then once with event.target=div#shadow-host.  So we tag the event and only handle it once, from div#clicker
Then when walking up the dom in findHighest we have to continue via the host element so that we traverse the host document as well

I've opened this as a draft PR since I really just wanted to get a discussion going on the correct solution for this

fair enough - I'm not sure this is a solvable issue in this way since the shadow dom is supposed to be genuinely "hidden" from the page, so as far as the browser can tell the page, there's nothing "in" the shadow DOM, the event simply started in the (custom) element that is found in the regular DOM.
What's the concrete use case we're trying to solve for here, though? Because React itself doesn't use shadow DOM, that's more a custom elements thing (which I would love to see get so easy to use they basically make React obsolete =D).

My primary case is css isolation.
By rendering into a shadow DOM I am albe to use my component in contexts where I have no control over the styling of the page.  For example if I use semantic ui and a third party use bootstrap there will be no css clashes when embedding my control
I am using a component (react-datepicker) that uses onclickoutside to hide the calendar popup when you click outside the popup.  However when used within a shadow dom, clicks within the calendar popup always close the popup due to the above mentioned reason.

@cameronbraid
Got the same issue in shadowDOM.
Another solution maybe checking the event.composed flag.
If it's true, then call the event.composedPath(), inside the returned array the first one is the real target.
In older browsers, if we use the webcomponentsjs polyfill, shadowRoot is a virtual element in memory,
it seems addEventListener is not working sometimes, haven't chase why.

Looking at Event.composed is a good call, because it lets us access Event.composedPath, which is probably the exact thing we want to be looking at in this case, rather than having to do a "while not at top" loop.

@Pomax @cameronbraid
I've raised another pull request #324 .
Please have a look.
The polyfill is for custom web components API. Not for this lib. This lib works quite well in Chrome and other modern browsers. :)

Faced same issue when using react-datepicker in shadowRoot.  Patch applied as "temporary workaround":
diff --git a/src/index.js b/src/index.js
index 3e66bf8..3ec0e85 100644
--- a/src/index.js
+++ b/src/index.js
@@ -131,6 +131,8 @@ export default function onClickOutsideHOC(WrappedComponent, config) {
      * for clicks and touches outside of this element.
      */
     enableOnClickOutside = () => {
+      const document = this.componentNode && this.componentNode.getRootNode ? this.componentNode.getRootNode() : document;
+
       if (typeof document === 'undefined' || enabledInstances[this._uid]) {
         return;
       }
@@ -178,6 +180,8 @@ export default function onClickOutsideHOC(WrappedComponent, config) {
      * for clicks and touches outside of this element.
      */
     disableOnClickOutside = () => {
+      const document = this.componentNode && this.componentNode.getRootNode ? this.componentNode.getRootNode() : document;
+
       delete enabledInstances[this._uid];
       const fn = handlersMap[this._uid];

quite similar to solution in this PR, but a bit simpler :)
I can't use composedPath, cuz in my env shadowTree is attached in closed mode.

closing due to inactivity (mine, mostly, and unless someone is going to fund me working on this again, it's unlikely that'll change)

closing due to inactivity (mine, mostly, and unless someone is going to fund me working on this again, it's unlikely that'll change)


New feature-added a contact form and finish the mobile version

Added a contact form and finish the mobile version with Formspree API.

Hi @YugynDprodigy10 ,
Status: Invalidated 🔴
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
This milestone is a pair-programming activity. You have submitted a project, that does not have any contribution from your partner. Please make sure your partner contributes to the code before submitting another request or if you are working solo, please attach a screenshot to the PR description showing where student success asked you to do so.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.

That is because I did not have a coding partner for the week. I did it solo

HI @YugynDprodigy10 👋🏼
Can you please provide proof about working solo ? it can be a screenshot of the email you get from Mircoverse student success that proves you've been unpaired for this week 👍🏼

HI @YugynDprodigy10 👋🏼
Can you please provide proof about working solo ? it can be a screenshot of the email you get from Mircoverse student success that proves you've been unpaired for this week 👍🏼


Source code dummy devices

Hello,
I'd like to know if there is any possibility to view the code for the dummy devices?
I'd like to use it as workflow inspiration for my own devices.
Thanks,

The dummy devices are a simple Node JS application, all the source code can be found in the tutorials.Step-by-Step  repository.

https://github.com/FIWARE/tutorials.Step-by-Step/tree/master/context-provider/controllers/iot
https://github.com/FIWARE/tutorials.Step-by-Step/blob/master/context-provider/models/devices.js
https://github.com/FIWARE/tutorials.Step-by-Step/blob/master/context-provider/models/command/ultralight.js
https://github.com/FIWARE/tutorials.Step-by-Step/blob/master/context-provider/models/measure/ultralight.js

Thank you very much, I was simply looking in the wrong repo :)


Should also mention Azure AD Role 'Application Administrator'

Under section #required-permissions - where it says:
"If your account is assigned to the User role, but the app registration setting is limited to admin users, ask your administrator to either assign you to the global administrator role, or to enable users to register apps."
Surely it should also mention the Azure AD Role 'Application Administrator' someone granted this role will also be able to create a service principal without requiring to be a full Global Administrator (or asking a Global Administrator to do this for them)?
https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/directory-assign-admin-roles#application-administrator

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

ID: 11aaa091-1108-52f6-0066-0f4e939aeef4
Version Independent ID: dbc7e37e-87a7-c124-3e4c-47aa060e6cf3
Content: Create identity for Azure app in portal
Content Source: articles/active-directory/develop/howto-create-service-principal-portal.md
Service: active-directory
GitHub Login: @CelesteDG
Microsoft Alias: celested

@rohancragg
If the setiing "User can register applications" is set to NO, then having the role of Application Admin would not suffice. However, as per the Admin Roles document you have referred, with the role of "Application Developer" - user should be able to register the application.
We would have this checked and update as appropriate.

@CelesteDG
In the Admin Roles document it states for the role of Application Developer - Users in this role can create application registrations when the "Users can register applications" setting is set to No.
Could this be verified and corrected in this document ?

Hi @neeleshray - yes I agree, it was 'Application Developer' I had in mind when i suggested this change rather than 'Application Administrator': https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/directory-assign-admin-roles#application-developer

@rohancragg and @neeleshray - Thank you both very much for the feedback. I had updated the paragraph to the following to leave room for any future roles that may get added that also have permissions to create/manage app registrations (rather than calling out specific roles).

Updated topic is now live. #please-close


I am not able to Download the Course

Checklist

[x ] I'm reporting a broken site support issue
[x ] I've verified that I'm running youtube-dl version 2<IP_ADDRESS>
[x ] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar bug reports including closed ones
[x ] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpspbsbtmq\build\youtube_dl\extractor\common.py", line 627, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpspbsbtmq\build\youtube_dl\YoutubeDL.py", line 2238, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

## Description

C:\myyoutubedll>youtube-dl --username "anil-sharma-ramdev" --password "AnilRamdev666@" --verbose --sleep-interval 120 --o "C:/MyVideos/%(playlist)s/%(chapter_number)s - %(chapter)s/%(playlist_index)s - %(title)s.%(ext)s" "https://app.pluralsight.com/library/courses/snowforce-2020-session-24/"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--username', 'PRIVATE', '--password', 'PRIVATE', '--verbose', '--sleep-interval', '120', '--o', 'C:/MyVideos/%(playlist)s/%(chapter_number)s - %(chapter)s/%(playlist_index)s - %(title)s.%(ext)s', 'https://app.pluralsight.com/library/courses/snowforce-2020-session-24/']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: none
[debug] Proxy map: {}
[pluralsight:course] snowforce-2020-session-24: Downloading JSON metadata
[download] Downloading playlist: Monitor Salesforce Limits on Salesforce with an Unlocked Package
[pluralsight:course] playlist Monitor Salesforce Limits on Salesforce with an Unlocked Package: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[pluralsight] Downloading login page
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpspbsbtmq\build\youtube_dl\extractor\common.py", line 627, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpspbsbtmq\build\youtube_dl\YoutubeDL.py", line 2238, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

WRITE DESCRIPTION HERE

Duplicate of #29776, now.


Support access to related model on Django 2+

field.rel was removed in Django 2.  These changes maintain compatibility with Django<2

Awesome, thank you.


Support S3-backed repositories

It would be nice if it were easy to have ACI repositories in an S3 bucket.  This should be simple, in that S3 can serve HTTP, except that HTTPS is a little tricky with S3.
An S3 backed website aci.example.com is reachable on http://aci.example.com, but not https://aci.example.com.  It is reachable on  https://s3-<region>.amazonaws.com/aci.example.com
I see two options (and we may want to do both):

Add s3 support via an s3:// schema, and add s3:// to the discovery sequence.  This would also support private repositories.
Attempt to recognize sites that are hosted using S3 CNAMEs, and use the alternative (https://s3-<region>.amazonaws.com/bucket/...) method of accessing them.   We could either detect the CNAME in DNS (hacky) or look for the "Server: AmazonS3" header on http (also pretty hacky, and security concerns of http).

Personally, I think the first option is much better; it feels much less hacky, it allows for private repos, and I can imagine that we make the discovery mechanism configurable in future, and this would tie in well with that.

To be clear, the HTTP vs HTTPS problem really happens in two places: rkt trust, which will not accept http-only sites in discovery, and when we download the "latest" version (which we should probably only do over https, although currently we tolerate http)
It would also be great to allow private S3 buckets, of course!

@justinsb Do you want to put up a proposal for 1) so we can talk through it?

I did a simple PoC that allows rkt trust s3://<bucket>/pubkeys.gpg  just to make sure this made some sense.  Lots of work still to do (particularly around credentials): https://github.com/justinsb/rkt/commit/a249a764b59e1a0f29306ba960d658b1441b5c0e
What is the right way to make a concrete proposal?  Code or words or both?  Where do the words go (and do you have an example of a good proposal)?

@justinsb I realised I didn't quite grok your point earlier about adding s3 to the discovery sequence. Are you just talking about that being one of the possible schemes returning during meta-discovery (e.g.  analogous to the hdfs example we mention in the spec today), or are you talking about a different kind of discovery process?

I don't think I myself have fully grokked the discovery process, so this may not make sense!
Certainly, it would be great if we supported s3:// urls if we find them.  i.e. we should honor this:
<meta name="ac-discovery" content="example.com s3://mybucket/{name}-{version}-{os}-{arch}.{ext}">

And this should support S3 credentials for private buckets (I would suggest this should happen by default).
I think this support and rkt trust s3://<bucket> should be uncontroversial (other than feature creep).
But, the missing link that I'd like to bridge is that it would be great to be able to store a private ACI repo in a private S3 bucket, and have that work (without having to set up SSL certificates for the discovery mechanism on S3 and without having to open the bucket up publicly).  I'm not sure what the best way to achieve this is.  I think we want the s3:// url added to the sequence of discovery URLs.
The problem is that S3 bucket names don't require DNS control (e.g. I could register a bucket justin.coreos.com`).  In theory, this doesn't matter because the trust mechanism should prevent any problems, but I can see this biting someone eventually.
Two options that might work:

maybe we could let a user add a path to their discovery mechanism, and support s3:// (and hdfs:// etc)
maybe we could have a "scoped trust" that would achieve this, so I would do rkt trust --root s3://aci.justinsb.com/pubkeys.gpg --basedir s3://aci.justinsb.com/  This would let me run any ACI, as long as it was signed by my key, but only if downloaded from s3://aci.justinsb.com.  And this would implicitly extend the discovery path.


Migrated errors for models package #2

Description
This PR fixes:
#3485
#3486
#3487
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #3528 (d2b3e57) into master (008287e) will decrease coverage by 2.22%.
The diff coverage is 0.47%.

:exclamation: Current head d2b3e57 differs from pull request most recent head e0e28a1. Consider uploading reports for the commit e0e28a1 to get more accurate results

@@            Coverage Diff             @@
##           master    #3528      +/-   ##
==========================================
- Coverage   12.63%   10.41%   -2.23%
==========================================
  Files          37       37
  Lines        2929     3227     +298
==========================================
- Hits          370      336      -34
- Misses       2467     2824     +357
+ Partials       92       67      -25

Flag
Coverage Δ

gointegrationtests
?

unittests
10.41% <0.47%> (-1.00%)
:arrow_down:

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/config/config.go
0.92% <0.00%> (ø)

mesheryctl/internal/cli/root/perf/perf.go
26.66% <0.00%> (-23.34%)
:arrow_down:

mesheryctl/internal/cli/root/system/channel.go
16.49% <0.00%> (-2.33%)
:arrow_down:

mesheryctl/internal/cli/root/system/config.go
1.11% <ø> (ø)

mesheryctl/internal/cli/root/system/start.go
1.27% <0.00%> (-0.24%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/stop.go
0.93% <0.00%> (-0.16%)
:arrow_down:

mesheryctl/internal/cli/root/system/system.go
69.23% <ø> (ø)

mesheryctl/internal/cli/root/system/update.go
1.13% <0.00%> (-0.22%)
:arrow_down:

mesheryctl/pkg/utils/auth.go
0.00% <0.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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


[BUG] Overlay for allsky version ${ALLSKY_VERSION}

Camera model
ZWO 178mc
Operating System
Bookworm
Allsky version
Version: v2023.05.01_05
Pi and amount of memory (512 MB, 1, 2, 4, or 8 GB)
4b 8gb
Bug Description
the overlay variable  ${ALLSKY_VERSION}  returns the wron value (v2023.03.31_tbd)
Log / configuration files
No response

Solved.  Reloaded config.sh

@BobW55, would you please attach ~/allsky/config/logs/install.sh.log ?
It might help me determine why the version number was wrong.


Hcf 466 cli extended table

Made as requested on the slack #cli channel by dkoper.

Hi @andreas-kupries , I think there might have been an issue introduced as part of this change
https://www.pivotaltracker.com/story/show/117404629


Simplifying Condition Clauses

This proposal adjust Swift grammar to enable the arbitrary mix of expressions among conditions rather
than constraining them before other conditions. Under this proposal, expressions are no longer limited
to where clauses after the initial list of Boolean conditions.
Swift-evolution thread:
[Pitch] making where and ,	interchangeable in guard conditions

Although I can probably fix this up to specify that expressions must be Boolean, I'm going to leave this for now as it's not pressing. Thanks again.


prepare_font_for_shipment (future? tool): delete/strip unreachable glyphs from the fonts we ship + noto_lint change

delete/strip unreachable glyphs from the fonts we ship even if we keep them in the sources.
Also:
Please change in noto_lint.py "E <reachable>" to "W <reachable>"

I'd rather leave it as an error and explicitly disable particular glyphs in particular fonts.
This often is a real error in which the designers and harfbuzz disagree about whether a glyph can actually be generated from a text sequence, and we need to catch these.

agreed with argument: "I'd rather leave it as an error and explicitly disable particular glyphs in particular fonts." thanks for pointing this out.

Whether we strip unreachable glyphs from the font binaries or not seems to best be done via a flag passed to the toolchain.  My guess is we should strip them by default.  The designs are in the source files and that's the only place we need to keep them if someone wants to make use of them in the future.

Only strip the unreachable glyphs if you are 100% sure that your process is correctly identifying glyphs as unreachable.
If you strip them and the glyphs are actually used in a complex OpenType feature, then you will have output issues with the build font.

Karen Dupre
Principal Quality Engineer & Font Librarian

600 Unicorn Park Drive 3rd Floor
Woburn, Massachusetts 01801
United States

Telephone +1<PHONE_NUMBER>

monotype.comhttp://www.monotype.com/
From: dougfelt<EMAIL_ADDRESS>Sent: Friday, December 09, 2016 12:59 PM
To: googlei18n/nototools<EMAIL_ADDRESS>Subject: Re: [googlei18n/nototools] prepare_font_for_shipment (future? tool): delete/strip unreachable glyphs from the fonts we ship + noto_lint change (#335)
Whether we strip unreachable glyphs from the font binaries or not seems to best be done via a flag passed to the toolchain. My guess is we should strip them by default. The designs are in the source files and that's the only place we need to keep them if someone wants to make use of them in the future.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/googlei18n/nototools/issues/335#issuecomment-266079009, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGWBJo7ZnKH_4HntvYHoAvMXM2uTmSh7ks5rGZbqgaJpZM4LHJ8v.

If harfbuzz thinks they're unreachable, we'll have output issues regardless on platforms like ChromeOS and Android that use harfbuzz.  This will be the case whether we strip them or not.  So we should strip them, test the fonts as usual, and if there's a bug we'll find out.

@jamesgk says the sources have flags that can be used to prevent glyphs from being emitted, and that the pipeline respects that.   So it seems the best course of action is to have lint continue to report these, and the designers either fix them, or (if they want to keep the glyph design around) tag them in the sources to mark them as not to be emitted, or demonstrate that fontTools is in error and they can indeed be reached.

Sounds like a good course of action. Could you add/write something that we
could post as a guideline for anybody that edits the sources and we can
close this? What do you think?
On Tue, Dec 13, 2016 at 4:00 PM, dougfelt<EMAIL_ADDRESS>wrote:

@jamesgk https://github.com/jamesgk says the sources have flags that
can be used to prevent glyphs from being emitted, and that the pipeline
respects that. So it seems the best course of action is to have lint
continue to report these, and the designers either fix them, or (if they
want to keep the glyph design around) tag them in the sources to mark them
as not to be emitted, or demonstrate that fontTools is in error and they
can indeed be reached.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlei18n/nototools/issues/335#issuecomment-266901035,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMtJf31D1h6jPwXOjk3zXOYh00CDptonks5rHzGSgaJpZM4LHJ8v
.

--
Marek Z Jeziorek [ 老马 ] |<EMAIL_ADDRESS>|<PHONE_NUMBER>

@jamesgk, does the Glyphs App provide a UI for setting this flag?  What is it?

When you right-click on a Glyph in the main font UI, there's a checkbox for exporting the Glyph.

Alternatively, in the Font Info -> Instances menu, you can add a custom parameter "Keep Glyphs" with a list of glyphs to keep, one list for each master. Seems less practical but I've seen that for Devanagari.

So it's not the case that all glyphs are exported by default?  You have to explicitly mark each one you want to export, or add them to that list?

No no, the checkbox is checked by default for all glyphs.

So, I'm going to say leave lint as-is and report this as a bug.


Can't access the unix fd from unprivileged lxc container

Hi @Arkq,
Right now on my system. Bluez is running in the host, and bluealsa is running withing a privileged lxc container. and I mount the DBUS socket to the lxc container. Everything works good.
But when I change the container to a unprivileged container, it seems bluealsa can't work properly. For the unprivileged container, I have setup uidmap, to map the uid 0 in the container to uid 10000 in the host.
When bluealsa try to acquire the transport, it report error in function transport_acquire_bt_a2dp() at the line:
t->bt_fd = g_unix_fd_list_get(fd_list, 0, &err);

The error log is :
g_unix_fd_list_get: assertion 'g_is_unix_fd_list (list)' failed

Generally when unprivileged container want to access a unix socket in the host, we need to bind this unix socket to a file, and explicitly mount the file in the container as well, then container can open this file to access the socket. That is how we did with the dbus socket.
But with the A2DP transport unix socket, it is not bind to any file, so unprivileged container can't access that socket. Not sure if there is any way to resolve this problem.

Bluealsa utilizes file descriptor passing via Unix socket in two places -
acquiring A2DP transport (mechanism exposed by bluez D-Bus API) and
providing PCM pipe and control socket to clients. This way of transferring
FDs is rather tightly bound to bluez-alsa architecture (it simplifies
open/close synchronization), so if there is no way of transferring FD via
Unix socket to unprivileged container, I'm afraid there is no much we can
do :/


[stable/hlf-peer] Configure logging

What this PR does / why we need it:
This PR allows the operator to specify the log level of the Peer. HLF is kind of verbose in debug mode, filling the disk quite fast and making the reading of logs painful.
Which issue this PR fixes
N/A
Special notes for your reviewer:
I've changed the defaults to reflect the default values of Hyperledger 1.3: https://github.com/hyperledger/fabric/blob/release-1.3/sampleconfig/core.yaml#L37
Regarding higher version of hyperledger tho the control of the logging is controlled by the variable FABRIC_LOGGING_SPEC (https://hyperledger-fabric.readthedocs.io/en/release-1.4/logging-control.html#logging-specification). I didn't added it there tho to keep the PR simple and don't bring >=1.4 specific configurations in the codebase for now.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @alexvicegrab @nicolapaoli

/ok-to-test

@nicolapaoli, this PR should not cause any issue with previous, as it merely converts a set hardcoded values to variables, so I will approve. We can easily update the Nephos example values.yaml to use debug mode if needed.
/approve

@ClementGautier thanks for your contribution!

/lgtm

@ClementGautier aha, please check conflicts and resolve. Thanks!

@alexvicegrab it should be good now ;) Thx !

/lgtm


upgrade maven-jar-plugin

Purpose

upgrade maven-jar-plugin to non-obsolete version
Tests

want to see how E2E CI reacts to that simple change (extracted from #2519), and investigate if anything goes wrong
API and Format

Documentation

ok, basic compilation issue "Error:  Failed to execute goal org.apache.maven.plugins:maven-jar-plugin:3.4.2:jar (default) on project paimon-s3-impl: You have to use a classifier to attach supplemental artifacts to the project instead of replacing them. -> [Help 1]"
I can work on it...


Create a authentication using Firebase

Can now sign in with Authenticator Firebase. You still do not need to be signed in to view the Map.


fix pubspec description

[yak_flutter] add
[yak_flutter] make descriptio in pubspec between 60 to 180 char


[4.2 03-26-2018] SILGen: Don't postpone the cleanup of the convertEscapeToNoEsca…

…pe inside an bind optional
This would violate dominance since the convertEscapeToNoEscape
instruction does not dominate the postpone point.

@swift-ci Please test

#15610

Build failed
Swift Test Linux Platform
Git Sha - 952fdac4f8349b8a49a31b37423032cfbb8d956b

Build failed
Swift Test OS X Platform
Git Sha - 952fdac4f8349b8a49a31b37423032cfbb8d956b


go-swagger install error

Problem statement
when running $ GO111MODULE=off go get -u github.com/go-swagger/go-swagger/cmd/swagger
I get the following errors:
package embed: unrecognized import path "embed" (import path does not begin with hostname)
package io/fs: unrecognized import path "io/fs" (import path does not begin with hostname)
package github.com/hashicorp/hcl/hcl/printer: cannot find package "github.com/hashicorp/hcl/hcl/printer" in any of:
I have ${GOPATH}/bin in my path
Please remove the sections that don't apply
Steps to reproduce
$ GO111MODULE=off go get -u github.com/go-swagger/go-swagger/cmd/swagger
Environment
swagger version:  v1.30.4
go version: 1.13.0
OS:   OSX 12.6.3

Too old go version.

ok thank you i was able to run $ go install github.com/go-swagger/go-swagger/cmd/swagger@latest and that resolved the issue thank you


importer: zfs listing could yield a description separated via whitespace

I'm leaving this as is, not the prettiest thing in the world, but it seems ZFS is not built for scripting... ;)


v0.10.0 doesn't compatible with create-react-app

I created a repo at https://github.com/cpunion/reproduce-import-graphiql-css-error to reproduce this issue. It created by create-react-app, and create-react-app uses css-loader and postcss-loader to process .css files. So maybe it doesn't compatible with css-loader or postcss-loader.
Steps:

Checkout the 2nd commit https://github.com/cpunion/reproduce-import-graphiql-css-error/commit/9e1681c55da453633edc25efaa917468b384f38e, it imports css of<EMAIL_ADDRESS>by import 'graphiql/graphiql.css', and it works fine.

Checkout the last commit https://github.com/cpunion/reproduce-import-graphiql-css-error/commit/3ac8408011baf3fa34a6e95f1913eabe9dcb52a0, it uses<EMAIL_ADDRESS>and doesn't work.

Errors:
Failed to compile.

Error in ./~/graphiql/graphiql.css
Syntax error: Unexpected token  in JSON at position 17977
    at JSON.parse (<anonymous>)
 @ ./~/graphiql/graphiql.css 4:14-107 13:2-17:4 14:20-113

I found a problem in node_modules/graphiql/graphiql.css by binary search (Cut a half every time :-) )，I found there is a line at graphiql.css:699 breaks loaders.
Please see https://gist.github.com/cpunion/f60fcca11569ce7a6a958a8a4407e20f#file-graphiql-css-L699

Also seeing 0.10.0 fail with postcss-loader when doing @import 'graphiql/graphiql.css';

Let me take a look at this, and if necessary I'll cut another release today.

I think the easiest way forward is to remove the source map lines from the built graphiql.css. I don't think they are very useful anyway…

Fixed in v0.10.1 via #440. Thanks!


Add unit tests for String functions in org.jkiss.utils.CommonUtils

Hi,
I've analysed your code base and noticed that org.jkiss.utils.CommonUtils in the bundles module is not fully tested.
I've written some tests for the functions in this class with the help of Diffblue Cover.
Due to the size of the class, the tests in this PR just cover the functions that return a String value.
Hopefully these tests should help you detect any regressions caused by future code changes. If you would find it useful to have additional tests written for this repository, I would be more than happy to look at other particular classes that you consider important.

Looks great, thank you very much!
We constantly lack unit tests, it is awesome when someone contribute one :)
You could take a look at org.jkiss.utils package in general. There are a few other very generic classes like XMLBuilder, IntKeyMap and LongKeyMap.


Replace Carthage with SPM for Ably Project and test suite

Context
Regarding https://github.com/Carthage/Carthage/issues/3019, Carthage builds will fail in Xcode 12 when using beta or latest stable, the installation fails with something like: error: Unable to determine target or target not included.
We are using Carthage to build our dependencies while developing and running tests. Our customers can use Carthage to add our library to their project.
The reason
The reason of this issue is the new Xcode 12 behaviour, which Carthage doesn't handle correctly right now. See https://github.com/Carthage/Carthage/issues/3019 for details.
Current solution
We can wait until Carthage fix the issue or use the Swift Package Manager or CocoaPods. Since we need to add support to Swift Package Manager (https://github.com/ably/ably-cocoa/issues/819), I propose to use SPM in Ably project and test suite.
┆Issue is synchronized with this Jira Uncategorised by Unito

Can be closed if we decide to remove carthage entirely under #1137.


fix ambiguities

Aqua is set to check ambiguities.

Solves #36

Related changes:

Some tests change from NotANumber to ArgumentError #38
Standalone +∞ now refers to RealInfinity(). Some +/- test results have changed accordingly, although not consistent. #39
Support (fld|cld|div)(∞, x) where x<0
Aqua tests are moved to the end of tests to make sure things work before fixing ambiguities.
Support infinite Complex{float} + ∞
min|max are set to follow Julia base convention, that is, min(x,y)=x and max(x,y)=y if x==y. This impacts ℵ₀ and ∞ specifically. #41

A crucial line somehow loses coverage. Removing the line results in failed tests. I've added a note to the line.

Related: https://github.com/JuliaApproximation/ClassicalOrthogonalPolynomials.jl/issues/154


Missing docker compose --env-file option

Hello,
I haven't the .env file in the same location of the compose .yml configurations, so that I need to specify a custom .env path location but currently python on whales does not support this option
I did some tests on my local copy and this is working for me:
https://github.com/mdantonio/python-on-whales/commit/1bcf7962a41ce91bfbe1128fbcdb3112fa09d400
Of course I can send a PR, but tests are missing there

Hi again! Thank you for the issue, i'll take a look :)

The option has been added, I'll make a new release after a few other PRs are merged.

Thank you! I will test it as soon as the new version will be released

The release is done. v0.21.0. Thank you for your help!

Many thanks to you for your support!
I confirm that everything works as expected
I think that I will open you other issues in the following days while I will continue with the integration into my applications, I hope I don't bother you too much :)

If this package can be of use to you, that makes me happy. Please keep opening issues, it's very good for me to know what the users want and what I should focus my attention on. I also appeciate when you give me code diffs because then making a pull request is quite fast :)