Remove repeated line and improve naming consistency

The line psi = tbs_document.post_sign_instructions needlessly appeared twice in the code.

The first part's output variable is used as output_handle in the second part. Make both names consistent.

Description of the changes
Small documentation fix in the interrupted signing example code.
Checklist
Please go over this checklist to increase the chances of your PR being worked on in a timely manner. Deviations are allowed with proper justification (see previous section).

[x] I have read the project's CoC and contribution guidelines.
[x] I understand and agree to the terms in the Developer Certificate of Origin as applied to this contribution.
[x] All new code in this PR has full test coverage.

For documentation contributions (delete if not applicable)

[x] I have built the HTML documentation locally, and verified that the changes behave correctly in-browser.

Thanks; will merge this.

Thank you for the merge, and thank you even more for pyHanko! With it I wrote a tool to sign PDFs using a remote signature API in just a couple of days.


Uses Custom Text for Different Geolocation Autofill Errors

What does this pull request aim to achieve? 💯
This PR intends to provide a more helpful message to users when autofilling the location input using the browser API fails.
Please be concise and link any related issue(s):
Resolves #1259

[X] I have checked that no one else is working on similar changes.
[X] I have read the latest README and understand the development workflow.
[X] The name of this branch is: feature/<branch_name>, or hotfix/<branch_name> if this is a hotfix. The branch is created in a cloned version of this repo, not a forked repo.
[X] This branch is rebased/merged with the latest staging (or the latest production if it is a hotfix).
[X] There are no merge conflicts.
[X] There are no console warnings and errors.
[X] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".

@EyeWumbo I went ahead and applied the format fix and will merge so we can get this in with next release to production.
I did assign myself but if you've an interest in taking on #786  let me know in slack.

Ah man, sorry about that. I got dragged away by some issues over the last few days. I'll check out #786


AspNet.Identity doesn't respect LowercaseUrls

Hi,
I have an application base on IdentityServer4 and AspNet.Identity. I use the following configuration to make all URLs lowercase:
services.Configure<RouteOptions>(options => options.LowercaseUrls = true);

But after the latest update (from Microsoft.AspNetCore.Identity v2.0.1 to v2.1.1) URL pointing to login page '/account/login' started to appear in pascal case like '/Account/Login'.
After some investigation I found that exact place where this string comes from is IdentityServiceCollectionExtensions.cs#L53:
.AddCookie(IdentityConstants.ApplicationScheme, o =>
            {
                o.LoginPath = new PathString("/Account/Login");
                o.Events = new CookieAuthenticationEvents
                {
                    OnValidatePrincipal = SecurityStampValidator.ValidatePrincipalAsync
                };
            })

These components don't use routing to generate urls (yet, @JamesNK), so there's no expectation that the route options would apply.


Create my-first-week-as-a-trainee-teacher

Title: My first week as a trainee teacher
Author: Cindy Ventura
Publish date: 20/09/21

Description: Cindy recently started the Postgraduate Diploma in Education (PGDE) teacher training course at Sheffield University. She kept a diary during her first week on the course and shares her experiences here.
Keywords: teacher training, trainee teacher, course, university-led teacher training,
Tags: becoming a teacher

@christinehitch I've added the metadata you suggested and placed the photo at the top of the post. All looks good to me!

I've added some suggestions @christinehitch @peteryates - hope that is useful!


Font face inside typi map

Hi Zell,
is there any way to insert a font face inside Typi map?
So what I'd like to do is insert the typeface in html{} with a normal typi-base map.
Thanks

Why not do this?
html {
  font-family: ‘your-typeface’;
}

That's obviously a normal solution, but if I use:
html {     font-family: "My Font", Arial, Helvetica, sans-serif; } @include typi-base ();
I get the html property twice:
`html {
font-family: "Inter-UI", Arial, Helvetica, sans-serif; }
html {
font-size: 100%;
line-height: 1.4; }
@media all and (min-width: 20em) {
html {
font-size: 100%; } }
@media all and (min-width: 48em) {
html {
font-size: 112.5%; } }
@media all and (min-width: 75em) {
html {
font-size: 125%; } }`
I'd like to insert the font-face inside a unique html {}

This shouldn't be a problem if you optimize your CSS with cssnano. It'll consolidate the html selector declarations into one.


[BUG] CTS failures

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Seeing CTS failures in both in-mem & graph (but this is on the experimental v4 build)- and I understand @dwolfson is also seeing in XTDB on v3.14
This was seen when testing v4 (java 17/gradle)
In my case I did a test run on Nov 22nd with in-memory using 'main'.
That is currently failing 16 tests of 6105 with 6089 passing
Expected Behavior
CTS to pass
Steps To Reproduce

Run CTS with current main - using inmemory for example
retrieve results with

http --timeout=600 --verify=no GET https://localhost:9443/servers/cts/open-metadata/conformance-suite/users/garygeeke/report > out.json

Print out the failing assertions with

cat out.json | jq -r '.testLabResults.testResultsFromWorkbenches[].failedTestCases[].unsuccessfulAssertions[]'

The result is
repository-relationship-reference-copy-lifecycle-03: DiscoveredNestedDataField reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: DiscoveredNestedDataField refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: RelatedKeyword reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: RelatedKeyword refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: LastAttachmentLink reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: LastAttachmentLink refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: AttachedRating reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: AttachedRating refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: DataFieldAnalysis reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: DataFieldAnalysis refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: AnnotationExtension reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: AnnotationExtension refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: DiscoveredAnnotation reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: DiscoveredAnnotation refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: SchemaAttributeDefinition reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: SchemaAttributeDefinition refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: SchemaTypeDefinition reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: SchemaTypeDefinition refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: TranslationLink reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: TranslationLink refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: AnnotationReviewLink reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: AnnotationReviewLink refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: DataClassDefinition reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: DataClassDefinition refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: DiscoveredDataField reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: DiscoveredDataField refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: AttachedTag reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: AttachedTag refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: SearchKeywordLink reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: SearchKeywordLink refreshed reference relationship matches original.
repository-relationship-reference-copy-lifecycle-03: DataProfileLogFile reference relationship matches the relationship that was saved.
repository-relationship-reference-copy-lifecycle-11: DataProfileLogFile refreshed reference relationship matches original.

Environment
- Egeria:
- OS:
- Java:
- Browser (for UI issues):
- Additional connectors and integration:

Any Further Information?
No response

The above output is from the graph repo

will repeat with current code from main

The CTS tests from main are clean.
This means it is one of

intermittent
specific to the v4 changes (gradle etc)
fixed by changes in main yet to be pushed to the v4 branch (~2 weeks)

Server log: https://gist.github.com/8e242fa5769a8856976122d5e29b7303
Nothing abnormal noticed

Having retested with https://github.com/odpi/egeria-charts/pull/225 (egeria-release-3.14), the inmemory CTS is failing in the same way
This is going to block the release

Looking through recent commits:

d669a357d66165a46a6d4b88abe86ce8f34b4307 made a change to reference relationships
f37c0ad2a0dbaff9d2ebc694d2141e49c2add9f2 affects the storage of classification/entities

@mandy-chessell Do you have any thoughts as to whether the above have changes may have resulted in this cts behaviour?

https://github.com/odpi/egeria/commit/d669a357d66165a46a6d4b88abe86ce8f34b4307 made a change to one of the converters in the OMAS layer to populate extra properties in an OMAS interface bean - highly unlikely to impact the CTS which operates at the OMRS level.
https://github.com/odpi/egeria/commit/f37c0ad2a0dbaff9d2ebc694d2141e49c2add9f2 made changes to entity/classifications and all of the failures are in the management of relationships.
I had a clean Cts run on my machine before pushing the changes into main.
The recent changes I made that broke the CTS was to add support for DEREGISTERED_REPOSITORY instance provenance type.  The CTS was storing an instance, retrieving it and using equals() to determine that it had not changed.  The problem was that the CTS was using a nonsense metadata collection id in the instance.  Before the DEREGISTERED_REPOSITORY instance provenance type fix, no check was made on the validity of the metadata collection id.  The local metadata collection now validates the metadata collection id to determine if it is an active member of the cohort.  If it is not then the instance provenance type is updated to DEREGISTERED_REPOSITORY before the instance is returned.  This change causes the equals() method to fail in the CTS.
The fix I added to the CTS was to update the instance provenance type to DEREGISTERED_REPOSITORY in the original instance before the test for equals().
I am thinking that the CTS fixes missed the branch or somehow I missed fixing the relationship tests.  But then I do not know how it worked on my machine.
I will run the CTS on main to see if the failure occurs on our latest code.  This will isolate whether there is a missing fix and we have a new/different problem, or the CTS fix missed the branch.
The fact that it fails on the latest V4.0 branch suggests a new problem.

The way to identify the test that is failing is to setach the CTS code for the part of the assertion message that does not include the typeName.
For example, for failing test message ….
repository-relationship-reference-copy-lifecycle-03: DiscoveredNestedDataField reference relationship matches the relationship that was saved.

Search for reference relationship matches the relationship that was saved
This gives you the assertion message.
private static final String assertion3      = testCaseId + "-03";
private static final String assertionMsg3   = " reference relationship matches the relationship that was saved.";

Find where it is used in the code and this give you the condition and profile
verifyCondition((newRelationship.equals(retrievedReferenceCopy)),
                assertion3,
                testTypeName + assertionMsg3,
                RepositoryConformanceProfileRequirement.REFERENCE_COPY_STORAGE.getProfileId(),
                RepositoryConformanceProfileRequirement.REFERENCE_COPY_STORAGE.getRequirementId());

Using this technique, the other failing test is
verifyCondition((newRelationship.equals(refreshedRelationshipRefCopy)),
                        assertion11,
                        testTypeName + assertionMsg11,
                        RepositoryConformanceProfileRequirement.REFERENCE_COPY_STORAGE.getProfileId(),
                        RepositoryConformanceProfileRequirement.REFERENCE_COPY_STORAGE.getRequirementId());

Both failures are using equals() to test that what was retrieved was what is expected.

Thanks - I had looked at the cts code,and established it was that comparison - was just wondering what you thought ref recent changes. It's been failing on v4 for a couple of weeks  (ON GRAPH ONLY)
I'm also today looking at https://github.com/odpi/egeria/issues/353 - so we can pick up on cts issues quicker & ensure consistency

If it is graph only it is probably a new issue and not related to the DEREGISTERED_REPOSITORY change which was in the OMRS and not the connectors.

Rerunning CTS on 3.14 (k8s, via egeria-cts chart)
: InMemory - PASS
: Graph - STILL RUNNING (will update when complete)

This is the comparison of the two relationships.  The retrieved headerVersion is 0 and should be 1 in the relationship, and each entityProxy.  In the comparison below, retrievedReferenceCopy is on the left and newRelationship is on the right.

The headerVersion property is used to discover whether the OMRS is talking to a future version of OMRS that uses later structures.   The value should be 1.  0 is the default value in the bean and suggests it is not being set.
It hints at a problem in the way instances are stored/retrieved from the graph store.  I do not know what has changed to cause it to fail now and why it is only affecting relationships.

Further investigation shows that RepositoryElementHeader where headerVersion is stored, does not have an equals() method.  This means that although this is two errors:

Missing equals() method in RepositoryElementHeader
GraphRepository not setting headerVersion correctly

It is not reason that the CTS is failing.

This is the comparison of the relationships for a failing case.

What we see is that the newRelationship incorrectly includes a null qualifiedName property in the entityProxies for each end of the relationship.  This is an invalid property for the type.
The error is in the CTS since it is setting up the newRelationship.  The question is, why has this changed?

I have located the source of the problem in EntityProxy.java.   It is setting up qualifiedName whether it is null or not.  Fixing that led to an NPE in Subject Area OMAS that was dereferencing uniqueProperties without checking it was not null.
The CTS is now running cleanly on graph

CTS tests

Run via egeria-cts k8s chart (as is usual for release)

Graph
Reporting test failures
        "testCaseCount": 6106,
        "testFailedCount": 1884,
        "testPassCount": 4222,

Summary output:
              Metadata sharing MANDATORY_PROFILE             NOT_CONFORMANT [  70597 /   1060 ]
              Reference copies  OPTIONAL_PROFILE             NOT_CONFORMANT [   6938 /   1884 ]
          Metadata maintenance  OPTIONAL_PROFILE             NOT_CONFORMANT [  13302 /    824 ]
                 Dynamic types  OPTIONAL_PROFILE             UNKNOWN_STATUS [      0 /      0 ]
                 Graph queries  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [    528 /      0 ]
             Historical search  OPTIONAL_PROFILE      CONFORMANT_NO_SUPPORT [    530 /      0 ]
                Entity proxies  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2759 /      0 ]
       Soft-delete and restore  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2592 /      0 ]
                Undo an update  OPTIONAL_PROFILE      CONFORMANT_NO_SUPPORT [    406 /      0 ]
           Reidentify instance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2650 /      0 ]
               Retype instance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  16365 /      0 ]
               Rehome instance  OPTIONAL_PROFILE             UNKNOWN_STATUS [      0 /      0 ]
                 Entity search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  62878 /      0 ]
           Relationship search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   8254 /      0 ]
        Entity advanced search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  44800 /      0 ]
  Relationship advanced search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   9312 /      0 ]

FAIL [241911/3768]
In-memory
Summary:
"testCaseCount": 6104,
"testFailedCount": 1884,
"testPassCount": 4220,
yet:
➜  inmem ~/src/cts/scripts/cts-analyze.py
              Metadata sharing MANDATORY_PROFILE    CONFORMANT_FULL_SUPPORT [  71657 /      0 ]
              Reference copies  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   8528 /      0 ]
          Metadata maintenance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  14063 /      0 ]
                 Dynamic types  OPTIONAL_PROFILE             UNKNOWN_STATUS [      0 /      0 ]
                 Graph queries  OPTIONAL_PROFILE CONFORMANT_PARTIAL_SUPPORT [    186 /      0 ]
             Historical search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [    530 /      0 ]
                Entity proxies  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2759 /      0 ]
       Soft-delete and restore  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2592 /      0 ]
                Undo an update  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   1218 /      0 ]
           Reidentify instance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2650 /      0 ]
               Retype instance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  16365 /      0 ]
               Rehome instance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   1590 /      0 ]
                 Entity search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  60722 /      0 ]
           Relationship search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   8232 /      0 ]
        Entity advanced search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  40464 /      0 ]
  Relationship advanced search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   9264 /      0 ]

PASS [240820/0]

The new, trial CTS test runs
See https://github.com/planetf1/cts.
These do not check the pass/fail count in the summary, instead relying on negative/positive evidence points in the full output
Both FAILED
Identical results from graph in terms of profile ( https://github.com/planetf1/cts/actions/runs/3715784372/jobs/6301358261 )
Inmemory showing a few issues https://github.com/planetf1/cts/actions/runs/3715784372/jobs/6301358199 - also a fail

Checked with MAIN
The SAME cts graph failures are reported
(inmemory running now)
Yet this is to be expected, since the PR https://github.com/odpi/egeria/pull/7247 is yet to be merged
Therefore the test may be invalid. Perhaps the container image (tagged 3.14) is cached ? Yet this wouldn't affect the trial test runs since there is no persistent container registry (k8s cluster built each time) and yet they had the same results

The inmemory CTS on 3.14 (which is using latest image) reports one exception:
                            "Unexpected Exception Exception : CTS test repository-entity-reference-copy-lifecycle caught exception AssertionFailureExc
eption from method saveEntityReferenceCopy whilst trying to save a reference copy of an entity of type Regulation.  Exception message was : Failed Ass
ertion:  reference entity retrieved with mappingProperties.Regulation.    This is the default value
 equivalent to null'}, replacedByTypeGUID='null', replacedByTypeName='null'}, TypeDefLink{guid='578a3500-9ad3-45fe-8ada-e4e9572c37c8', name='Governanc
eDefinition', status=TypeDefStatus{ordinal=1, name='ActiveTypeDef', description='TypeDef available and in use.  This is the default value equivalent to null'}, replacedByTypeGUID='null', replacedByTypeName='null'}, TypeDefLink{guid='a32316b8-dc8c-48c5-b12b-71c1b2a080bf', name='Referenceable', status
=TypeDefStatus{ordinal=1, name='ActiveTypeDef', description='TypeDef available and in use.  This is the default value equivalent to null'}, replacedBy
TypeGUID='null', replacedByTypeName='null'}, TypeDefLink{guid='4e7761e8-3969-4627-8f40-bfe3cde85a1d', name='OpenMetadataRoot', status=TypeDefStatus{or
dinal=1, name='ActiveTypeDef', description='TypeDef available and in use.  This is the default value equivalent to null'}, replacedByTypeGUID='null',
replacedByTypeName='null'}], validStatusList=[InstanceStatus{ordinal=15, name='Active', description='The instance is approved and in use.'}, InstanceS
tatus{ordinal=99, name='Deleted', description='The instance has been deleted and is no longer available.'}], validInstanceProperties=[qualifiedName, a
dditionalProperties, summary, implications, domainIdentifier, outcomes, scope, domain, description, title, priority, results, jurisdiction]}, instance
ProvenanceType=InstanceProvenanceType{ordinal=4, name='Deregistered Repository', description='The instance comes from a metadata repository that used
to be a member of the one of the local repository's cohorts but it has been deregistered. The metadata collection id remains the same. If the reposito
ry rejoins the cohort then these elements can be refreshed from the rejoining repository.'}, metadataCollectionId='remotec5-7484-4d71-923b-afb48f8e57a
1', metadataCollectionName='remote-metadataCollection-not-TUT_MDR', replicatedBy='null', instanceLicense='null', status=InstanceStatus{ordinal=15, nam
e='Active', description='The instance is approved and in use.'}, createdBy='OMAGServer', updatedBy='null', maintainedBy=null, createTime=Fri Dec 16 19
:44:48 UTC 2022, updateTime=null, version=1, statusOnDelete=null, mappingProperties={integerMappingPropertyKey=12, stringMappingPropertyKey=stringMapp
ingPropertyValue}, instanceURL='null', GUID='remote14-38ea-4540-bd4a-86bae7e97a49', reIdentifiedFromGUID='null'}"

I reran the tests (k8s) :

tests run clean on 3.13
inmem passes on 3.14 ( (as it did 3 other times, albeit prior to the latest fix). Suggests a one-off/intermittent  failure above)
graph FAILS on 3.14

Whilst graph consistently fails, I believe CTS failures for inmemory are inconsistent. To avoid more clutter I'll open up a new issue for inmemory

Unfortunately, the exception shown in comment https://github.com/odpi/egeria/issues/7148#issuecomment-1356124469 is an error in the CTS.  It is catching an assertion failure rather than an exception from Egeria.
The assertion failure it masks is probably the real problem.   I will merge the current PR and retest with the fix to the exception management.

If you want to try running the CTS pipelines

Will only work on merged updates
A new container image is made by the 'merge' pipeline at https://github.com/odpi/egeria/actions/workflows/merge-maven.yml. This typically takes an hour, but can take longer -- or fail (and must be restarted)
this image is posted to https://quay.io/repository/odpi/egeria?tab=tags - 'latest' is used for main.
Once the image is ready, the experimental cts github pipeline can be launched at https://github.com/planetf1/cts/actions/workflows/cts.yml by clicking on 'launch workflow'. It takes ~ 4.5 hours to cover our 3 repo connectors. (let me know if there are permission issues and you want to do this)
The output file can be downloaded (and we can figure out capture of additional info, or better interpretation of the results)

see CTS results after main -> https://github.com/planetf1/cts/actions/runs/3751230156 (though note inmem and xtdb are both showing some intermittency)

CTS graph errors continue to be reported by the pipeline above.
Additionally, I did a standalone test on a 3 x 16GB/8CPU k8s cluster, and had the same results for graph, which continues to fail in 'main' with:
  graph ~/src/cts/scripts/cts-analyze.py
              Metadata sharing MANDATORY_PROFILE             NOT_CONFORMANT [  70833 /   1062 ]
              Reference copies  OPTIONAL_PROFILE             NOT_CONFORMANT [   6950 /   1887 ]
          Metadata maintenance  OPTIONAL_PROFILE             NOT_CONFORMANT [  13329 /    825 ]
                 Dynamic types  OPTIONAL_PROFILE             UNKNOWN_STATUS [      0 /      0 ]
                 Graph queries  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [    528 /      0 ]
             Historical search  OPTIONAL_PROFILE      CONFORMANT_NO_SUPPORT [    531 /      0 ]
                Entity proxies  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2782 /      0 ]
       Soft-delete and restore  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2598 /      0 ]
                Undo an update  OPTIONAL_PROFILE      CONFORMANT_NO_SUPPORT [    406 /      0 ]
           Reidentify instance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   2655 /      0 ]
               Retype instance  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  16365 /      0 ]
               Rehome instance  OPTIONAL_PROFILE             UNKNOWN_STATUS [      0 /      0 ]
                 Entity search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  62926 /      0 ]
           Relationship search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   8248 /      0 ]
        Entity advanced search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [  44800 /      0 ]
  Relationship advanced search  OPTIONAL_PROFILE    CONFORMANT_FULL_SUPPORT [   9312 /      0 ]

FAIL [242263/3774]

CTS is clean for inmemory & has been clean for a while, so the issue is addressed
(xtdb is failing, but this is fixed in v4 of the xtdb connector, which is not yet available due to dependency work still needed)


System.OverflowException in System.IO.Packaging.Package.Open

Scenario: we have a directory full of files that periodically should be added to a zip package.
On the first round anything runs perfectly, the package is created and all files are added.
Many other rounds runs perfectly and the result package is not to big (around 200MB) then suddenly an overflow exception is raised when the package is open another time.
Surprisingly the package is not corrupted and can be explored in Windows and all files can be extracted and read perfectly.
Stack trace:
System.OverflowException: Arithmetic operation resulted in an overflow.
at MS.Internal.IO.Zip.ZipIOExtraField.ParseRecord(BinaryReader reader, ZipIOZip64ExtraFieldUsage zip64extraFieldUsage, UInt16 expectedExtraFieldSize)
at MS.Internal.IO.Zip.ZipIOCentralDirectoryFileHeader.ParseRecord(BinaryReader reader, Encoding encoding)
at MS.Internal.IO.Zip.ZipIOCentralDirectoryBlock.ParseRecord(BinaryReader reader, Int64 centralDirectoryOffset, Int32 centralDirectoryCount, Int64 expectedCentralDirectorySize)
at MS.Internal.IO.Zip.ZipIOCentralDirectoryBlock.SeekableLoad(ZipIOBlockManager blockManager)
at MS.Internal.IO.Zip.ZipIOBlockManager.get_CentralDirectoryBlock()
at MS.Internal.IO.Zip.ZipArchive.GetFiles()
at System.IO.Packaging.ZipPackage.ContentTypeHelper..ctor(ZipArchive zipArchive, IgnoredItemHelper ignoredItemHelper)
at System.IO.Packaging.ZipPackage..ctor(String path, FileMode mode, FileAccess access, FileShare share, Boolean streaming)
at System.IO.Packaging.Package.Open(String path, FileMode packageMode, FileAccess packageAccess, FileShare packageShare, Boolean streaming)
at System.IO.Packaging.Package.Open(String path, FileMode packageMode)

What was the solution for this error?


Workspace and compilation unit caching

This PR includes the caching that @anpaz-msft set up, and additionally also keeps and updates a compilation unit instead of rebuilding it from scratch every time, and thus specifically only reloads references if it needs to.

I'll put up a separate PR for the Compilation Unit caching and in-memory resource processing.


Add support for IP range denylists to outgoing requests

We should not be handling requests to internal/RFC1918 addresses when we're publicly facing, and as usual allow user to override this list.
https://matrix-org.github.io/synapse/latest/usage/configuration/config_documentation.html#ip_range_blacklist
Conduit seems to support proxies so we will need to add code to check if we're using a proxy and ignore this list if so.
I think the Synapse defaults are fine here. We also should only support specifying subnets, if a user wants to explicitly exclude an IP for some reason then they can just do /32 or /128.

https://github.com/girlbossceo/conduwuit/pull/106
There are ways to bypass my implementation (and very likely Synapse's), but I think this is an acceptable risk at that point, and you should just be using a firewall rather than relying on the application.


[Bug] [Worker] fix: create linux user with home dir

Purpose of the pull request
fix #15669
Brief change log
add '-m' flag to 'useradd'
Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

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

Project coverage is 38.61%.  Consider uploading reports for the commit 8fe20d9 to get more accurate results

Files
Patch %
Lines

.../apache/dolphinscheduler/common/utils/OSUtils.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15670      +/-   ##
============================================
- Coverage     38.63%   38.61%   -0.03%
  Complexity     4795     4795
============================================
  Files          1316     1316
  Lines         44969    44969
  Branches       4817     4816       -1
============================================
- Hits          17373    17363      -10
- Misses        25713    25724      +11
+ Partials       1883     1882       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Should I use dev code now?


Icon URL in site extension package doesn't work over https

https://www.siteextensions.net/packages/search-portal/
The image URL for your package works as http, but fails as https because of invalid cert. Try https://reddog.io/img/reddog-60px.png. Any chance of fixing that? The Azure portal is making changes that causes the icon to be broken in this case.
Thanks!

@sandrinodimattia just a ping on this one, as the Portal owners are complaining about the broken icon :)

Hi @davidebbo sorry for the late reply! I've made the icon available through github.io
Can you update the icon to use this url? https://reddog-io.github.io/img/reddog-60px.png

@sandrinodimattia Thanks! Unfortunately, the way the gallery works, the only way to get it to properly update the image that it shows in the feed is to upload a new version of the package with the URL updated in the .nuspec file. So it's not something I can easily do since I don't have the source package.
You can't reuse the same version, so you can just bump it up from 1.3.1 to 1.3.2.

Ok, done! 👍

Thanks Sandrino! I just checked, and it no longer looks broken in the portal :)


Add failing test for write_to_stream

For the happy path, streaming writes are working really well.
However when there is an error returned from Vix.Vips.Image.write_to_stream/2, like when there is an invalid suffix/loader options, then then either:

The stream stalls until process timeout or
The writer process exists abnormally due to a match error on :ok = Image.write_to_stream(...)

Fixing the match error is straight forward.  The key issue is that the process structure isn't resilient to errors on Image.write_to_stream/2.  I've tried a number of approaches but each has resulted in subtle race conditions. I've tried trapping exits but I seem unable to get that working as intended too (and I think this is probably the cleanest approach).
I've added a failing test here so you can see and hopefully you have some smarter ideas than I do!!!

Yes, this is a bit tricky at the moment, but there are some options like custom errors. We can raise a well-defined error instead of match error for common types of error like invalid loader options etc.
Can share a bit more about process timeout error? An example test case would be a great help to test.

@akash-akya the timeout was not reliably reproducible.  Anyway, this additional commit is a partial attempt to find a clean method of propagating errors. It does the following:

Add a :error field to the Vix.Pipe struct to contain an error (a binary)
Support a new mode :error for a Vix.Pipe struct
Add Pipe.error/2 to put a Vix.Pipe into the error mode, with a message

As implemented this does not work correctly and the failing test now times out. I'm adding this here because you are much better at this than I am. But I think this might be a reasonable approach to take to managing errors in the streaming flow (reads and writes).  I will spend some more time on this if you think its a reasonable approach.

The idea would be that when the error is detected in a pipe that an exception is raised.  Since at that point we are not in the spawned writer process, the exception would be raised in the originating process and therefore can be rescued by client code.  Is that a reasonable understanding?

I've added a commit that raises in Vix.Pipe.read and Vix.Pipe.write is the pipe is in the error mode.  Interestingly, the failing test now times out when reading the pipe. Which I didn't expect since the read should suggest and its the write that should fail.  Now officially confused.

Thanks for the suggestions and taking your time to explore this :)
I agree with the approach you suggested. I was thinking the same thing. Basically capture the writer error and return it on read.
I have a draft here: https://github.com/akash-akya/vix/pull/52. Please take a look if possible. It is same as your approach, just refactored the overall structure a bit and moved the writer process to gensever which also simplify streaming. Let me know if you have concerns.
I'll update vips-source as well and will see if we can return a better error message instead of generic "failed to write".


Add 'Started' event

This is an event that is sent to widgets when the application first runs.
We likely want a fairly full suite of lifecycle events, but for the time being I think we should just add them as needed. This is an obvious candidate.

Let's call this fixed by #511


Test kit

I made a simple test kit to hep me test some of my actors. It might not be perfect but i thought it might be a good start. It's very similar to the Akka.net kit in terms of what you can test with it, although some names might be slightly different

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Dane seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Your credentials aren't after login with AAD

image
Integration with AAD doesn't work. Got message when login with AAD Your credentials aren't allowed.
DD v. 2.18.4 ( release mode )
Deployment method Docker Compose. OS Ubuntu 20.04
Steps to reproduce:
Open DD
Click Login with AAD
Enter credentials
Got message Your credentials aren't allowed

Got the same problem. Were you able to fix this?

This issue is still present :( Can't find decision :(
пт, 10 лист. 2023, 12:59 користувач Jon @.***> пише:

Got the same problem. Were you able to fix this?
—
Reply to this email directly, view it on GitHub
https://github.com/DefectDojo/django-DefectDojo/issues/7994#issuecomment-1805513453,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIGZWY5IMJALZYUAN6W6XNDYDYCHZAVCNFSM6AAAAAAXFQNZKSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBVGUYTGNBVGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Could you please ask such things in the owasp slack channel?
please close this @mtesauro


Manjaro Gnome 不能启动Dochat，停在了Starting DoChat /dɑɑˈtʃæt/ ...

   ____         ____ _           _
  |  _ \  ___  / ___| |__   __ _| |_
  | | | |/ _ \| |   | '_ \ / _` | __|
  | |_| | (_) | |___| | | | (_| | |_
  |____/ \___/ \____|_| |_|\__,_|\__|

  https://github.com/huan/docker-wechat

            +--------------+
           /|             /|
          / |            / |
         *--+-----------*  |
         |  |           |  |
         |  |   盒装    |  |
         |  |   微信    |  |
         |  +-----------+--+
         | /            | /
         |/             |/
         *--------------*

  DoChat /dɑɑˈtʃæt/ (Docker-weChat) is:

  📦 a Docker image
  🤐 for running PC Windows WeChat
  💻 on your Linux desktop
  💖 by one-line of command

🚀 Pulling the docker image...
<IP_ADDRESS>: Pulling from zixia/wechat
Digest: sha256:e6e9d21c7cd1dfae0484e697f12f5f3c401de2f02e771d061868740e0d26549d
Status: Image is up to date for zixia/wechat:<IP_ADDRESS>
docker.io/zixia/wechat:<IP_ADDRESS>
🚀 Pulling the docker image done.
🚀 Starting DoChat /dɑɑˈtʃæt/ ...
停在了这一步，完全无反应。
   DE: GNOME 3.36.5
 /+++//+:`oo+o               /::--:.   WM: Mutter
 \+/+o+++`o++o               ++////.   WM Theme: Adwaita
  .++.o+++oo+:`             /dddhhh.   GTK Theme: Yaru-dark [GTK2/3]
       .+.o+oo:.          `oddhhhh+    Icon Theme: Yaru
        \+.++o+o``-````.:ohdhhhhh+     Font: Ubuntu 11
         `:o+++ `ohhhhhhhhyo++os:      Disk: 213G / 1.2T (20%)
           .o:`.syhhhhhhh/.oo++o`      CPU: Intel Core i7-10700 @ 16x 4.8GHz [85.0°C]
               /osyyyyyyo++ooo+++/     GPU: Quadro P2200
                   ````` +oo+++o\:     RAM: 8438MiB / 15725MiB

I also can't get it run on Ubuntu 20.04
Posting the debug info here
curl -sL https://raw.githubusercontent.com/huan/docker-wechat/master/dochat.sh   | DOCHAT_DEBUG=true bash

       ____         ____ _           _
      |  _ \  ___  / ___| |__   __ _| |_
      | | | |/ _ \| |   | '_ \ / _` | __|
      | |_| | (_) | |___| | | | (_| | |_
      |____/ \___/ \____|_| |_|\__,_|\__|

      https://github.com/huan/docker-wechat

                +--------------+
               /|             /|
              / |            / |
             *--+-----------*  |
             |  |           |  |
             |  |   盒装    |  |
             |  |   微信    |  |
             |  +-----------+--+
             | /            | /
             |/             |/
             *--------------*

      DoChat /dɑɑˈtʃæt/ (Docker-weChat) is:

      📦 a Docker image
      🤐 for running PC Windows WeChat
      💻 on your Linux desktop
      💖 by one-line of command

🚀 Pulling the docker image...

<IP_ADDRESS>: Pulling from zixia/wechat
Digest: sha256:e6e9d21c7cd1dfae0484e697f12f5f3c401de2f02e771d061868740e0d26549d
Status: Image is up to date for zixia/wechat:<IP_ADDRESS>
docker.io/zixia/wechat:<IP_ADDRESS>

🚀 Pulling the docker image done.
🚀 Starting DoChat /dɑɑˈtʃæt/ ...

++ id -u
+ '[' 0 -ne 0 ']'
+ '[' -n 29 ']'
+ groupmod -o -g 29 audio
+ '[' -n 44 ']'
+ groupmod -o -g 44 video
++ id -g user
+ '[' 998 '!=' 1000 ']'
+ groupmod -o -g 998 group
++ id -u user
+ '[' 1000 '!=' 1000 ']'
+ chown user:group '/home/user/.wine/drive_c/users/user/Application Data' '/home/user/WeChat Files'
+ export HOSTNAME=DoChat
+ HOSTNAME=DoChat
+ echo DoChat
+ hostname DoChat
+ exec gosu user:group /entrypoint.sh
++ id -u
+ '[' 1000 -ne 0 ']'
+ '[' -n true ']'
+ wine reg query 'HKEY_CURRENT_USER\Software\Tencent\WeChat'
reg��ϵͳ�޷��ҵ�ָ����ע�������ֵ
+ echo 'Register for Wechat not found ?'
+ exec wine 'C:\Program Files\Tencent\WeChat\WeChat.exe'
Register for Wechat not found ?
[0612/130536:INFO:exception_record.cc(518)] [QB]Process ID: 42 Type: 1
[0612/130536:ERROR:network_change_notifier_win.cc(156)] WSALookupServiceBegin failed with: 8

on  virtual machine  opensuse-15.2

我也卡在这里了，openSUSE leap 15.3

同上，请问有解决的吗？

I am using Manjaro KDE
maybe run xhost + before launching DoChat can solve this problem

I am using Manjaro KDE
maybe run xhost + before launching DoChat can solve this problem

I am using Manjaro KDE
maybe run xhost + before launching DoChat can solve this problem

I am using Ubuntu. I tried to run xhost + last night, but it still doesn't work. I think that is system lack of some tools or plugins? (I guess) rather than the problem of the docker's image.

I am using Manjaro KDE
maybe run xhost + before launching DoChat can solve this problem

I am using Ubuntu. I tried to run xhost + last night, but it still doesn't work. I think that is system lack of some tools or plugins? (I guess) rather than the problem of the docker's image.

Are there any output? Actually I also install Nvidia Container Toolkit to make dochat work well in my computer, you can know more at here #50 , hope it is helpful

I am using Manjaro KDE
maybe run xhost + before launching DoChat can solve this problem

I am using Ubuntu. I tried to run xhost + last night, but it still doesn't work. I think that is system lack of some tools or plugins? (I guess) rather than the problem of the docker's image.

Are there any output? Actually I also install Nvidia Container Toolkit to make dochat work well in my computer, you can know more at here #50 , hope it is helpful

I have no output or error info. I guess that isn't the problem of Nvidia. You know, I haven't ever learned docker knowledge, but still thank you for your reply.


Fix flaky tests

There are a couple of tests that are flaky (aka they fail unreliably every now and then). We should fix those.

I second that. One of them is RepositoriesHelper#render_push_activity creates the proper HTML for each kind of activity. I don't think I've come across any others so far.

There's another in the spec/features directory, but I don't remember which one.

PR #1011 doesn't seem to fix the failing build. See https://travis-ci.org/SUSE/Portus/jobs/147950881


Added www.tabsoverspaces.com site.

Thanks for the PR! We've simplified the process of adding your site/repo to the list. Instead of updating SITES.md you would add the utterances topic to your repo.


remove datamap export API

remove datamap export API
attempt to rewire datamap UI to look at the plus endpoint instead

Closes #
Code Changes

[ ] list your code changes here

Steps to Confirm

[ ] list any manual steps for reviewers to confirm the changes

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

Description Of Changes
Write some things here about the changes and any potential caveats

@allisonking i think this is just about good to go, if you could take another look - specifically at the FE pieces - that would be much appreciated!
@SteveDMurphy going to tag you as a reviewer here too, this is just the cleanup on fides corresponding to the addition to fidesplus
@TheAndrewJackson just an FYI since i know you're going to be working on datamap API code soon

i was just doing a final pass through myself, and realized that one thing i missed in the migration was moving the scope over to fidesplus - it's still defined here in core fides: https://github.com/ethyca/fides/blob/main/src/fides/api/ops/api/v1/scope_registry.py#L125
the new route in fidesplus references this scope from core fides, and that all seems to work fine. it's certainly a bit against the paradigm that's been set up, i think - where fidesplus endpoints have their scopes (and roles) defined in that code base. i'm not really sure what the end-user implications are, if any.
i was thinking of creating a follow up ticket for this migration rather than taking care of it immediately, since i think it's more of a tech-debt issue rather than one that impacts users significantly. but i could be wrong about that.
@pattisdr @allisonking - tagging you both since i think you are closest to some of the scopes/roles refactoring that was done, specifically with moving some of that to fidesplus? @SteveDMurphy also curious if you've got any thoughts off the top of your head.


fix(simple_planning_simulator): fix steering bias model

Description
In the old implementation, steering_bias was just added in the measurement, thus the behavior was "the vehicle yaw dynamics totally follow the steering command, but the steering output seems different". However, the expected behavior is "the steering output seems to be ideal, but the vehicle yaw dynamics does not match its steering angle". This PR fixes the behavior.
Related links
None
Tests performed

Add steer_bias param in yaml. (e.g. set to 0.01)
Set steering_offset.enable_auto_steering_offset_removal to false in mpc.param.yaml
Run psim
See the vehicle trajectory following is slightly biased to the right/left.

You can also check the steering offset topic /control/trajectory_follower/controller_node_exe/output/estimated_steer_offset converges to the bias value if you set the offset_removal true.
Notes for reviewers
None
Interface changes
None
Effects on system behavior
None by default.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.
[x] The PR has been properly tested.
[x] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[x] There are no open discussions or they are tracked via tickets.
[x] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Only yaw among the states has a biased value. each of the states should be unified as to whether it is an actual value or a sensor value.

fixed in https://github.com/autowarefoundation/autoware.universe/pull/6240/commits/e4dadc5b80fdc3266eec412535443bac680dead8
All states are actual values, and the values obtained by getSteer() are biased and used in the calculation of the steer_rate in calcModel() and published as an output report.

https://github.com/autowarefoundation/autoware.universe/assets/39142679/268f40cc-9378-453a-9295-3f541353c758

@TakaHoribe could you please check my commits? :pray:

@TakaHoribe
after https://github.com/autowarefoundation/autoware.universe/pull/6240/commits/c625256a3193ccad9ad625f7f0bfdb2d84bc2455

Add steer_bias param in yaml. (e.g. set to 0.01)
Set steering_offset.enable_auto_steering_offset_removal to false in mpc.param.yaml
Run psim
See the vehicle trajectory following is slightly biased to the right/left.

https://github.com/autowarefoundation/autoware.universe/assets/39142679/06da9803-af27-486c-8990-1c10506ce150

You can also check the steering offset topic /control/trajectory_follower/controller_node_exe/output/estimated_steer_offset converges to the bias value if you set the offset_removal true.

https://github.com/autowarefoundation/autoware.universe/assets/39142679/374a2400-5a25-4816-9824-e1992f78e86b
https://github.com/autowarefoundation/autoware.universe/assets/39142679/cf50e651-2fa4-4510-a718-987c1338a97f


Multiple updateNodeDataFromId not working?

I am trying to updateNodeDataFromId for data of 2 select dropdown fields in same node.
both are marked correctly but only the last one called works. In this case the "second" will work.
Code Sample:
editor.updateNodeDataFromId(newNodeID,{"first" : "test1"}); // wont work
editor.updateNodeDataFromId(newNodeID,{"second" : "test2"}); // works fine

Hello @sheksushant
Update node update all values.
Try:
editor.updateNodeDataFromId(newNodeID,{"first" : "test1","second" : "test2"});
Jero

Above code worked,
Thank you for the quick response.


{path} is logging {uri}

PR #1364 introduced a bug where {path} is now actually logging the full path?query rather than just the path when a rewrite has been performed.
I have a fix for this but I need to write tests to try to prevent something similar in the future.  Just wanted to log it so we know it is in progress.

Currently defered until after #1496

Fixed with #1481


Add promiseTimeout function

Codecov Report

Merging #80 (d59bcbf) into master (6b70565) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #80   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           30        31    +1
  Lines          303       313   +10
  Branches        28        28
=========================================
+ Hits           303       313   +10

Impacted Files
Coverage Δ

src/index.ts
100.00% <100.00%> (ø)

src/promiseTimeout/promiseTimeout.ts
100.00% <100.00%> (ø)

src/testHelpers.ts
100.00% <100.00%> (ø)

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 6b70565...d59bcbf. Read the comment docs.


Url gets parsed wrong in development mode

[EDIT: Being able to have relative references from CSS to assets work the same in dev and production is quite important]
I am using express together with socketstream. Besides that I am using bower for client dependency management.
I configured bower to download its contents into /client/code/system.
Bootstrap is also in system as bower dependency. I configured stylus to include the bootstrap css. Bootstrap loads its fonts (glyphicons) relative from the url ../fonts/fontfile.
That means, that requests in development go to /_serveDev/fonts/glyphicon.woff and in production mode to /assets/fonts/glyphicon.woff.
In development however the original url gets parsed as follows:
originalUrl: '/_serveDev/fonts/glyphicons-halflings-regular.woff',
  _parsedUrl:
   { protocol: null,
     slashes: null,
     auth: null,
     host: null,
     port: null,
     hostname: null,
     hash: null,
     search: '?glyphicons-halflings-regular.woff',
     query: 'glyphicons-halflings-regular.woff',
     pathname: '/_serveDev/fonts',
     path: '/_serveDev/fonts?glyphicons-halflings-regular.woff',
     href: '/_serveDev/fonts?glyphicons-halflings-regular.woff' },

You see that the file name gets changed to a parameter without value.
Whereas in production mode the same request is as follows:
originalUrl: '/assets/fonts/glyphicons-halflings-regular.woff',
  _parsedUrl:
   { protocol: null,
     slashes: null,
     auth: null,
     host: null,
     port: null,
     hostname: null,
     hash: null,
     search: null,
     query: null,
     pathname: '/assets/fonts/glyphicons-halflings-regular.woff',
     path: '/assets/fonts/glyphicons-halflings-regular.woff',
     href: '/assets/fonts/glyphicons-halflings-regular.woff' },

So in order to serve this file I have written following middleware:
var app = express();
app.get(/^\/(?:_serveDev|assets)\/fonts(?:\/(.+))?/, function(req, res){
  var file = req.params[0] || Object.keys(req.query)[0];
  res.sendfile(bowerpath + 'bootstrap/dist/fonts/' + file);
});

It's not a big issue, but why do requests to _serveDev get parsed different, than those to assets?

This is now fixed in 0.4, which we hope to release in a month or so. If you want to try it out check out the next branch.


chore: Remove GitHub token from docker action

What

Hopefully the toke isn't needed anymore

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 42.50%. Comparing base (3bf4492) to head (217211d).

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #76   +/-   ##
=========================================
  Coverage     42.50%   42.50%
  Complexity       51       51
=========================================
  Files            17       17
  Lines           320      320
  Branches         15       15
=========================================
  Hits            136      136
  Misses          172      172
  Partials         12       12

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Updates to automatically generated config text files.

img2img_restore_faces already added to the config list so uncommenting it in the text files offsets all the results. xyz plot img2img properties have img not txt in them.

Good catch.

Excited to be able to use it now, it's an essential feature in my mind so
thanks for writing it.
On Fri, Jun 30, 2023, 1:18 p.m. Zyin055 @.***> wrote:

Good catch.
—
Reply to this email directly, view it on GitHub
https://github.com/Zyin055/Config-Presets/pull/48#issuecomment-1615153573,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHU46AKKBJC2KR23YMY42LXN4YBHANCNFSM6AAAAAAZ2IPIXQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


basic_room.py example is broken

Maybe the interface to livekit package changed ?
Traceback (most recent call last):
  File "/home/basic_room.py", line 7, in <module>
    from livekit import api, rtc
ImportError: cannot import name 'api' from 'livekit' (unknown location)

Hey, please inside livekit-api :)


ENH: plot_topomap, plot_joint & plot: API homogeneity for scalings/scale parameters

Small API heterogenity: different plotting functions use similar parameters slightly differently (and named slightly differently)

scaling: evoked.plot(scalings=dict()) but evoked.plot_topomap(scale=dict() | float)
units: evoked.plot(units=dict()) but evoked.plot_topomap(unit=dict() | str)
max amplitude: evoked.plot(ylim=dict()) but evoked.plot_topomap(vmin=dict() | float, vmax=dict() | float())

Changing parameter names can be annoying for deprecation, but I suggest we at least make their input similar (e.g. always accepts both dict and float).
Additionally, evoked.plot_joint() could gather the parameters common to plot() and plot_topomap() to simplify it. WDTY @jona-sassenhagen ?

Somewhat related: #3164

Unity here would be greatly improved IMO.
We must be careful however with how different functions handle different channel types.

Can we split this into multiple smaller PRs?

It feels a bit redundant to have scaling and vmin, vmax in the first place. Could we get rid of scaling in for instance topomaps or is there a use case where you'd prefer using scaling?

to me scaling and vmin and vmax very different usage. I think we should
keep them.

I think you need scaling if you use for example stats.regression, where you plot t values on data channels.

Ugh I must get back to this. @jaeilepp can you maybe assign me so I don't forget?

Done.

:(

@Eric89GXL @agramfort do you have any idea about solving the different names for identical params for different functions (e.g., "unit" vs "units")?
I guess a **kwargs could do, but we never use that (in public functions), so ...

Can you give an example of what you are trying to do?

Oh it is in the top post.
I think the only clean way to fix it is to deprecate.


Apply global.proxy.excludeIPRanges to istio-cni

Enables the CNI plugin to respect global.proxy.excludeIPRanges configuration value.
Fixes #19939
Depends on istio/cni#293

Hi @cagataygurturk. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

See
#22902
istio/cni#293


Proposal: auto-complete import paths

Would be awesome if we could add auto-completion on import paths.
import {} from "./" // gives you an auto-complete suggestions

/// <reference path="./" // gives you an auto-complete suggestions

Dupe of #188


UserName field does not exist on some users

Category

[x] Bug

Describe the bug
When some SharePoint users are added to a user field, they do not have a UserName field (when fetching over REST).
Due to #4478 (closed as "by design"), this means trying to select the UserName field on multi-user field causes an error response if one of the users is missing this field.
In this case 2 of 15 AD-synced users are experiencing the problem. Both are fully licensed.
Steps to reproduce

Add a "broken" user to a multi user field
Send a request with $select=MultiUser/UserName&$expand=MultiUser
Observe that the server throws an error that the projected field cannot be selected
Try the same thing but with a single user field
Observe that the UserName field comes back null

Expected behavior
The user should always have a value for UserName, especially since it causes an error response otherwise.
Developer environment

OS: Windows 10, Arch Linux
Framework: Node.JS 10.x
Browser(s): Firefox 74, Chrome 80
Tooling: SPFx v1.10.0, Intellij Webstorm/PyCharm

This hasn't magically gone away


haskellPackages: fix libsystemd-journal

not sure why it broke in the first place, in an earlier version it had jailbreak = true; and sytstemd was set to the correct version in hackage-packages.nix. Did something with hackage2nix change @peti?

hackage2nix adds no more automatic jailbreaks. Why the reference to systemd doesn't work, however, I don't know. Anyhow, there are merge conflicts; could you please rebase the branch?

Why the reference to systemd doesn't work, however, I don't know.

I have to look into how cabal2nix references external libraries (if at all?).

Anyhow, there are merge conflicts; could you please rebase the branch?

I notice, yes; Is there a reason why these files are append-sorted instead of (pseudo-) alphabetically? The latter solves the merge conflict problem in e.g. python packages.

Is there a reason why these files are append-sorted instead of (pseudo-) alphabetically? The latter solves the merge conflict problem in e.g. python packages.

No particular reason, people just edit this file wherever they feel like it.

No particular reason, people just edit this file wherever they feel like it.

Would you object if I sorted it a bit?

Feel free to ... but please keep those files in synch between master and release-16.09, i.e. modify both branches.

fixed conflict @peti


trailing slash

Guzzle version(s) affected: 6.3.3
Hi,
making a GET request on this URL http://localhost/bpws2/app_dev.php/api/qrp/CFI/02493870188, Guzzle adds a trailing slash transforming the URL in http://localhost/bpws2/app_dev.php/api/cerved/qrp/CFI/02493870188/. If I add a character to the final path (http://localhost/bpws2/app_dev.php/api/cerved/qrp/CFI/a02493870188) then the URL is not modified.

There is a test here that proves that this is not happening. 🤔
Can you investigate further and elaborate?

In this thest the path is not ending with a numeric value, but with 'test'. In my scenario the path ends with only numeric characters '02493870188'. If the last element of the path contains an alphabetic character, then the we get the expected behaviour.

I am not able to reproduce your issue. Following a minimalistic example locally:
require "../vendor/autoload.php";

use GuzzleHttp\Client;
use GuzzleHttp\Psr7\Request;

$client = new Client([
    // Base URI is used with relative requests
    'base_uri' => 'http://httpbin.org',
    // You can set any number of default request options.
    'timeout' => 2.0,
]);
$request = new Request('GET', 'http://localhost:8081/bpws2/app_dev.php/api/qrp/CFI/02493870188');
$response = $client->send($request, [
    'timeout' => 2,
    'debug'=>true,
]);

Checking the server logs on my machine I notice that a request has been made to <IP_ADDRESS>:39090 [404]: /bpws2/app_dev.php/api/qrp/CFI/02493870188 - No such file or directory
No trailing slash is being added.

It looks more like your server is redirecting your request to one with a slash?

we recently tried to upgrade from 6.3 and now we can't get Guzzle to request URL's with a trailing slash (trailing slash is required). Seems like Guzzle is removing the slash (6.4/6.5/etc)... any way to get around this?

Please create a new with some code to reproduce your issue.


Support home assistant auto-discovery

As mentioned in #17, the tool should support auto discovery in a home assistant setup. This is achieved by publishing a certain topic to MQTT. More details:

discussion: https://github.com/DennisOSRM/hms-mqtt-publisher/issues/17#issuecomment-1804633765
HA documentation
OpenAI generated usage information (unverified)

implemented in #27


Incorrect drag and drop behavior of lines and elements

I copied the code from the CodeSandbox Demo and got a weird result. If you drag the elements,  the line connecting elements "begins to live its own life" and refuses to move behind the element. How to fix this?

I'm confused. Restarting VS Code solved the problem. Sorry to disturb you


Emerald: token queries, handle deterministic errors

okay so I'm treating three errors as deterministic (i.e. don't retry them, move on)

EVM "execution failed" error (code 2)
EVM "reverted" error (code 8)
if decoding the the output fails

current problems:

I don't have a test case for execution failure. https://github.com/oasisprotocol/oasis-sdk/blob/398c7891716b7d315a62ff426cebca9d008210eb/runtime-sdk/modules/evm/src/lib.rs#L187 looks like it should be sufficient to run a query with insufficient gas
the debug printout for these errors doesn't work well modules.EVMDeterministicError{error:(*fmt.wrapError)(0xc00007e020)}

oh shoot and I have the logic flipped

added evmCallWithABICustom to allow setting low gas. this successfully triggers an execution failed error. logic un-flipped. still need to do something about the debug print.

proceeding not to do anything about %#v formatting


NumberComparer

Fixes #87
Fixes #86

Codecov Report

Merging #88 into master will increase coverage by 0.71%.
The diff coverage is 83.15%.

@@            Coverage Diff             @@
##           master      #88      +/-   ##
==========================================
+ Coverage   70.48%   71.19%   +0.71%
==========================================
  Files         121      122       +1
  Lines        4872     4878       +6
==========================================
+ Hits         3434     3473      +39
+ Misses       1438     1405      -33

Impacted Files
Coverage Δ

...SourceCode.Clay.Json/Validation/LengthValidator.cs
61.9% <ø> (ø)
:arrow_up:

...ourceCode.Clay.Json/Validation/DecimalValidator.cs
64% <ø> (ø)
:arrow_up:

.../SourceCode.Clay.Json/Validation/Int64Validator.cs
64% <ø> (ø)
:arrow_up:

...SourceCode.Clay.Json/Validation/DoubleValidator.cs
64% <ø> (ø)
:arrow_up:

src/SourceCode.Clay.Buffers/HashCode.Fnv.cs
7.27% <ø> (ø)
:arrow_up:

.../SourceCode.Clay.Json/Pointers/JsonPointerToken.cs
52.63% <0%> (-2.93%)
:arrow_down:

src/SourceCode.Clay.OpenApi/ScalarValue.cs
13.33% <0%> (ø)
:arrow_up:

src/SourceCode.Clay.Data/SqlParser/SqlParamInfo.cs
37.03% <0%> (ø)
:arrow_up:

src/SourceCode.Clay.OpenApi/Callback.cs
65.21% <0%> (ø)
:arrow_up:

...ceCode.Clay.OpenApi/Expressions/FieldExpression.cs
76.37% <0%> (-0.43%)
:arrow_down:

... and 42 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 3952d14...b19725d. Read the comment docs.


[VxTextBuilder] Add VxTextTheme to change DefaultTextStyle of the text at theme level.

Is your feature request related to a problem? Please describe.
By default, VxTextBuilder uses default text style, and there's no way to change the default texts style on theme level. Normally, we could have wrap MaterialApp with DefaultTextStyle but there's already an open issue that Text doesn't inherit DefaultTextStyle beyond Scaffold. It's been 2 years to the issue, and it's really inconvenient to change the DefaultTextStyle for every page.
Describe the solution you'd like
The idea is to bring VxTextTheme which can be wrapped around MaterialApp to change the default text style through out the application. VxTextBuilder will give to priority to VxTextTheme if any or it will fall back to default.

i would like to work on this issue can you guide me though it @AyushBherwani1998  plz!! Thanks

@M-A-D-A-R-A the main issue here is how to access the TextStyle from parent. For an instance, we can introduce a VxTheme to define TextStyle but the main issue is how to access that VxTheme without context. Currently to build a text, we use this syntax "String".text.make(). For that TextStyle to be default we might have to change the syntax to "String".text(context).make(). We can also introduce a new extension of vXTextTheme to get the default TextStyle i.e. "String".text.vXTextTheme(context).make()

jfndklndslvkmflvmdflm


#160368988 e2e-testing

What does this PR do?
e2e-testing
Description of Task to be completed?

Setup e2e tests
Add e2e test files

How should this be manually tested?
Run yarn e2e-setup to setup selenium standalone
Run yarn test-e2e to run the tests
Any background context you want to provide?
End-to-end tests were added to validate and ensure the app meets user requirements
What are the relevant pivotal tracker stories?(if applicable)
#160368988

Pull Request Test Coverage Report for Build 259

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.76%

Totals

Change from base Build 251:
0.0%

Covered Lines:
1701

Relevant Lines:
1712

💛  - Coveralls


update the IRC room to point to #kolla

Thank you for contributing to stackforge/kolla!
stackforge/kolla uses Gerrit for code review.
Please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.

:( ok


BlockAtlas v1.1.15 platform bitcoin and ethereum error access denied

Summary of Bug

Expected Behavior

Steps to Reproduce

More Info (for devs / optional)
Request details:
{}

Response details:
{}

Hello,  i have a problem using api /v2/:token/tokens/:address with platform bitcoin and ethereum, msg='{status_code 503}'
in addition, service parser have same error.

Besides, api v3/tokens alaways return [] is it an error?
Looking forward to hearing from you


does not have sandbox access for frZQaeyWLUvLjeuEK43hmg and no access to InverseDeviceID

The following code gives me an error for me.
Swift 3.1, Xcode 8.3.2.
Thanks!
    ocr.recognize(image) { result in
        debugPrint("OCR Result: \(result)")
    }

2017-05-29 12:11:51.805177-0400 Test[2444:1199122] libMobileGestalt MobileGestaltSupport.m:153: pid 2444 (Test) does not have sandbox access for frZQaeyWLUvLjeuEK43hmg and IS NOT appropriately entitled
2017-05-29 12:11:51.805310-0400 Test[2444:1199122] WlibMobileGestalt MobileGestalt.c:550: no access to InverseDeviceID (see rdar://problem/11744455)
"OCR Result: "

same issue. any help?

Same here, any help much appreciated!


[feature request] Sync user data via Google Account

Since I have to switch between machines quite often (e.g., work/school and home), it would be nice to have data in KC3Kai sync between Chromes.  Not sure how complicated this would be to implement, but it should be possible (chrome.storage API?).

ah yes, this can be tracked on #17 which was an early suggestion but will be far into the future and has no definite implementation plans right now

Oops, somehow missed it in the Issue List.  And this time I actually checked. :/


Please provide more information.

How do I use this? I really have no clue how to get the referrer after Google depraced the referrer Intent Broadcast in March 2020. I need to get the referrer! Can you provide a little more information how setup this with your binding project in the ReadMe or give at least a little hint? I can't find anything about this topic in Xamarin Forums or Stackoverflow.

Hi
This is only a binding of the Google's referrer java library, you can learn more about the library in the official docs: https://developer.android.com/google/play/installreferrer/library?hl=pt-br#java
Here is a snipped of code that i current using in my apps, hope it helps:
InstallReferrerClient referrerClient = InstallReferrerClient.NewBuilder(context).Build();

referrerClient.StartConnection(new InstallReferrerStateListenerImpl {
    InstallReferrerSetupFinished = code => OnSetupFinished(referrerClient, code),
    InstallReferrerServiceDisconnected = OnInstallReferrerServiceDisconnected
});

private static void OnSetupFinished(InstallReferrerClient referrerClient, int code) {
    switch (code) {
        case InstallReferrerClient.InstallReferrerResponse.DeveloperError:
            TrackError("DeveloperError");
            break;
        case InstallReferrerClient.InstallReferrerResponse.PermissionError:
            TrackError("PermissionError");
            break;
        case InstallReferrerClient.InstallReferrerResponse.ServiceDisconnected:
            TrackError("ServiceDisconnected");
            break;
        case InstallReferrerClient.InstallReferrerResponse.ServiceUnavailable:
            TrackError("ServiceUnavailable");
            break;
        case InstallReferrerClient.InstallReferrerResponse.FeatureNotSupported:
            TrackError("FeatureNotSupported");
            break;
        case InstallReferrerClient.InstallReferrerResponse.Ok:
            OnConnectionEstablished(referrerClient);
            break;
    }
}

private static void OnConnectionEstablished(InstallReferrerClient referrerClient) {
    ReferrerDetails response = referrerClient.InstallReferrer;
    string referrerUrl = response.InstallReferrer;

    referrerClient?.EndConnection();
    referrerClient?.Dispose();
}

And the callbacks implementation :
 internal sealed class InstallReferrerStateListenerImpl : Java.Lang.Object, IInstallReferrerStateListener {

    public Action<int> InstallReferrerSetupFinished;
    public Action InstallReferrerServiceDisconnected;

    public void OnInstallReferrerServiceDisconnected() {
        InstallReferrerServiceDisconnected.Invoke();
    }

    public void OnInstallReferrerSetupFinished(int p0) {
        InstallReferrerSetupFinished.Invoke(p0);
    }

}

Thanks a lot, bro. I will take a look later.

It is working now. Thank you!

Hello @DouglasGiovanella & @joachimharms ,
Can you please help me to implement above code, so previously I have created "ReferrerReceiver : BroadcastReceiver" broadcastReceiver with previous way so now where above code required to add in mainactivity.cs ?
Also for above api does any change required in androidmanifest file ? and if i would like to test without moving to playstore how i can test it?
if you can help to resolve above queries it would be great help.
Many Thanks


fix: correctly handle hangup events for Android background isolates

Task linked: CU-8694xtdy8 Cannot decline the call from pop-up notification on Android


Build a Basic Web Scraper

Description:
Create a web scraper that extracts data from a specified website and displays it in a structured format (e.g., JSON).
Technologies: Python (Beautiful Soup/Scrapy)
Example:
Extract titles and links from a news website.

hello @strivedi4u ,
Please Assign This Issue. I want to contribute to this issue.


Dashboard feedback

2 fixes from #559

unhealthy gateways query fix

blue/black for histogram instead of green/red

rebased.
@R-Lawton if you have some time to review, I'd appreciate it.

👀


Add Does It ARM Apple Silicon App Test

This is a tool I built so people can test app compatibility with Apple Silicon without owning an actual Apple Silicon or Apple M1 Mac.
It isn't a Mac App but I feel like it could be a critical tool for Apple SIlicon Mac users but it may exactly fit in the main app list.
Let me know what you think.
Here's the info for what it's worth:
Put "Add ____" as Title

[x] Done

Provide a link to the proposed addition.
https://doesitarm.com/apple-silicon-app-test/
Explain why it should be added
So Mac users considering buying a new Apple SIlicon or Apple M1 Mac can test specific apps from their current mac to confirm compatibility.
Additional context
This browser-based tool is part of an open-source project dedicated to reporting App Compatibility for Apple Silicon Macs
Issue Checklist

[x] I have checked for other similar issues
[x] I have explained why this change is important
[x] I have added necessary documentation (if appropriate)

Hi @ThatGuySam,
Thanks for reaching out. For this project, there are two parts:

The github1s.com itself
Development of this project

For 1. I tested with M1 MacBook and it works well. And I don't think we need any additional check for ARM Apple Silicon with the web application itself since it's a pure web application.
For 2, we are blocked by this https://github.com/microsoft/playwright/issues/5443 which fails the VS Code build. Does the https://doesitarm.com/apple-silicon-app-test/ help to check whether it passes development process?

@xcv58 Was your original comment about testing all the apps listed?

@xcv58 Was your original comment about testing all the apps listed?

No, I thought it's for a different repo.
I think it's a good tool to be included in this repo, what category do you think it fits?

One option could be Utilities > System Related Tools
Another could be to list doesitarm.com under "Mac App Download Sites" (Since it does link to download pages) and then mention the tool as a note nested under the line for doesitarm.com
I want to make sure I respect the layout of the list as much as possible.

I suggest we add it to Utilities > System Related Tools.

Sounds good!
Do you need me to make a PR?

Sounds good!
Do you need me to make a PR?

Yes please make a PR if you would like do it. Thanks!

PR added! #874

Thank you @ThatGuySam 🙏


Compose the wallet from more purposeful structs rather than implementing all traits on one struct

At the moment we implement traits on the wallet and then pass the whole wallet in whenever we need certain functionality (e.g. the FeeEstimator and ChainMonitor in the ChannelManager).
It would be cleaner to create smaller dedicated structs that implement the necessary crates, that are initialized and passed in as needed and can be composed into being the wallet.

FeeEstimator has been decoupled, others still to go!

ideally, we would also move towards using ldk-node package when it matures enough - moving some of the functionality to external crates

@luckysori we discussed this in the past - if you want to re-open feel free to do so :)


Statement that allergies won't be sent to the server

Statement that allergies won't be sent to the server. It's obvious for us, but might not be for anyone.

I second that, and I don't think that's obvious as many of those preferences / settings for other services are stored in the cloud so that they are shared on your phone, computer etc.


fix: Add through to ManyToManyField

By the way, why other typings here using  = ...?
Should i also add = ... to this line?

yeah I think the ... isn't necessary for those attributes


Industrial turbine multiblock not registering to form 5x5x8

Please use the search functionality before reporting an issue. Also take a look at the closed issues!
Issue description: Industrial turbine multiblock not registering to form 5x5x8, infact no builds are forming
Steps to reproduce: This was built on a Superflat

build a standard 5x5x8 industrial turbine multiblock
try to have it form as a multiblock
fail

Version (make sure you are on the latest version before reporting):
**Forge: <IP_ADDRESS>11
**Mekanism: 1.12.2-<IP_ADDRESS>9
**Other relevant version:
Mekanism Tools 1.12.2-<IP_ADDRESS>9
Mekanism Generators 1.12.2-<IP_ADDRESS>9

If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
[gist/pastebin/etc link here]

It might just work as a multi block. It doesn't show the connected textures if you don't have CTM installed.

yeah looking at the Thermal towers (that appear to be working?) in the background, you're just missing CTM. Otherwise please post a full mod list.


Chinese translation of static class member initialization incorrect

Type of issue
Typo
Description
In English:
Static members are initialized before the static member is accessed for the first time and before the static constructor, if there's one, is called.
Chinese doc:
在首次访问静态成员之前以及在调用构造函数（如果有）之前，会初始化静态成员。
Should be:
在首次访问静态成员之前以及在调用静态构造函数（如果有）之前，会初始化静态成员。
Page URL
https://learn.microsoft.com/zh-cn/dotnet/csharp/programming-guide/classes-and-structs/static-classes-and-static-class-members?redirectedfrom=MSDN
Content source URL
https://github.com/dotnet/docs/blob/main/docs/csharp/programming-guide/classes-and-structs/static-classes-and-static-class-members.md
Document Version Independent Id
de2dac73-e810-dd42-f518-e50926b3747f
Article author
@BillWagner
Metadata

ID: 7f20db0d-ef5c-d8e7-2267-0f409c499e0f
Service: dotnet-csharp
Sub-service: fundamentals

Hi @klxiaoniu
I've opened a work item for this issue on our internal system for localization. Thank you for reporting it.
Because the source English text is correct, I'll close this issue.


chore: Stop automatic upgrade of some samples to .NET 8

For these, we need to confirm that the environments they are meant for support .NET 8 and documentation needs to be updated also.

I'll sefl merge this and test it by retrying #2719


Add Event Types filter and model config

Hello
This is a PR with update for mqtt to the new API version.
There is new configuration option for changing the model and enable amblegpt only for chosen labels.

Thanks @zeppelinlg!


[FR] UI filters for camera name and event type

First, apologies for my english, and my first time on github !
Just to preface my case and my tests, my need is:
I've got cameras that dont work well with frigate for their main_stream, they work well for the sub_stream and the person detection work as intended
For the face detection with double take, i need to use an overide of the snapshot to process the detectors at max resolution
I did a lot of trial and error and have some errors and requests:

detect:
match:
# save match images
save: true
# include base64 encoded string in api results and mqtt messages
# options: true, false, box
base64: false
# minimum confidence needed to consider a result a match
confidence: 60
# hours to keep match images until they are deleted
purge: 168
# minimum area in pixels to consider a result a match
min_area: 3600

min_area not taken into acount: the base value of 10000 is applied on detectors (compreface and deepstack)

frigate:
url: http://[ip]:5000
#object labels that are allowed for facial recognition
labels:
- person
attempts:
# number of times double take will request a frigate latest.jpg for facial recognition
latest: 0
# number of times double take will request a frigate snapshot.jpg for facial recognition
snapshot: 0
# process frigate images from frigate/+/person/snapshot topics
mqtt: false
# add a delay expressed in seconds between each detection loop
delay: 0

Even with this config, the default mqtt topics are processed, latest and snapshot are not (as intended).

cameras:
reolink_cam_01:
snapshot:
#   # process any jpeg encoded mqtt topic for facial recognition
#   topic:
#   # process any http image for facial recognition
url: http://[ip]:8123/local/test.jpg

url: http://[ip]:8123/local/test.jpg works but not with a refresh from the ui only after a hard reboot and not consistently, sometimes it takes many reboot/refresh for being taken into account

in the matches part of the ui, when the snapshot is processed, it's not listed under the cam entity but under "manual", to me the parameter is supposed to be an overide of the frigate snapshot.jpg

The url is processed only one time, for me if no match is found, the default attempts config for snapshot should be applied, and the overiden snapshot refreshed. with snapshot: 10 or 0 in the frigate - attempts config their is no differences

I tried to use a dynamiq url generated by an Api (in my case a reolink one : http://[ip]/cgi-bin/api.cgi?cmd=Snap&channel=0&rs=wuuPhkmUCeI9WG7C&user=[user]&password=[pass] ) but this is not taken into acount, I tried to send the same url via the double take api (curl --location --request GET 'url') but it was not processed either.

Request:
On the Macthes UI:
Beeing able to filter by camera entity
Beeing able to filter by trigered media (latest,snapshot,mqtt,...)

Hey @Sylvain-Laine, those config examples you posted are for the current beta build. I'm working on releasing 1.0.0 today as well which will include that.
I'll take a look at your other requests in a bit, but that should be the reason for the config inconsistencies.

Ok, thank you for the repply, i will update this issue, after the 1.0.0 release,
I toyed with the config in the past days, updated it today with the information available, but i already had the same problems before.
I hope i'll come back with good news :)

1.0.0 is released now.

Everything works thank you !
I didn't quite understand the configuration section for "camerar / [cam] / snapshot / url" I thought (falsly) that it was an override of the frigate snapshot, but this point is addressed in the issue #110
The support for filtering based off the camera and triggered media will be a great improvement for the ui !
Thank you

Hey @Sylvain-Laine, I got the new filters added to the beta build, docker pull jakowenko/double-take:beta. Let me know what you think!

Perfect ! Thanks

Juste a quick update, the filters do work great, but they are reset after a delete operation, this is just for quality of use.
This may need another [FR], but i was thinking of another filter which may be great for debugging, not so important for normal use: The ability to filter by frigate event (even the events with no return from detectors).

Glad they are working! I can work on the filter resetting logic and make it better. It's a little tricky because if no matches are in the database, then there's no filters, but as soon as one comes in I need to update all the filters so it shows on the page, this is when the "reset" of the filters happens.
I'll update it though so they don't reset after the delete operation. Only when the WebSocket receives new matches.
How did you think the other debug filter would work? No images would be saved for these events, so I'm not sure what to display.
Everything is written to a messages.log file, so you could always look through there for debugging the Frigate events too. Though not as clean as being on the UI.

Maybe a filter logic based on both trained folders / entry in the configuration instead of matches in database, it's a little more tricky because you still need to be able to shows matches in db that are no longer in the configuration after a change...
But as i said, this is juste quality of use, the filters are fine ! And the difference between v1.0.0 and the current beta build is perfect ! Great Job !
For the frigate event filter, im still thinking about it, i'll come back to you after having some time to formulate the use case in the details.

Hey @Sylvain-Laine, I got the new filters added to the beta build, docker pull jakowenko/double-take:beta. Let me know what you think!

can you do such a selection-filter accordingly also for the train-page?

@ozett the training page already filters images when you select a name from the dropdown. I don't see a lot of other value added by adding more filters. What else were you hoping to filter by?

i have now 5 page of faces.
i dont want to filter by person, i want to images regardless of the person-category
to improve quality i would check from time to time if images have a too low box-area.
or to sort out images that matched, but had not been trained on again


[Tree Taxonomy - Planning] Information Architecture

miro board: https://miro.com/app/board/uXjVPj0Fujg=/
Current State of I.A.

@zoobot im starting the information architecture to map out the tree taxonomy. Any suggestions?

updated information architecture. added tree request route, educational route on how to care for trees and what is a tree taxonomy.

want to show off the water source feature here eventually. adding it as a potential project.


.NET Core and xUnit cleanup for v0.9

Minor clean ups to resolve compiler warnings and remove files no longer required by newer project files.
Exposing internal visible classes to xUnit tests and marking RenderCleaner internal.

it's good to be merged as is, let me know if you want to merge it or hold for comments.

Yes we should change warnings to errors. I left one warning about an unassigned variable. It been bugging me so I really should fit it. I'll fix and re-push.

@vors Should be everything.


Use flutter Native window as transparent overlay window on desktop

Hi，
In desktop application development, it is very common to render content to the window through the main window handle, and create a new transparent window as an overlay to draw the UI.
After looking at the flutter window example, I found that flutter uses SetParent to attach the flutter Native Window to the main window.
That way, can I render the content onto the main window handle and let the flutter Native Window draw the UI as a transparent overlay?

There are similar issues filed for respective platform
windows: https://github.com/flutter/flutter/issues/71735
Linux: https://github.com/flutter/flutter/issues/66751
MacOs: https://github.com/flutter/flutter/issues/59969
Closing as a duplicate.

There are similar issues filed for respective platform
windows: #71735 Linux: #66751 MacOs: #59969
Closing as a duplicate.

I checked fluent_ Acrylic project. He uses SetWindowCompositionAttribute to realize the translucency of windows on windows platform. He can solve it #71735
But with this method, when I render the content to the main window, the translucent window still displays the content behind the main window rather than the content I want to render.
I used to use skia to draw the overlay window UI, Set the SKAlphaType of SKImageInfo to Premul,  and use WinAPI UpdateLayeredWindow in the overlay window to make the window translucent.
Translucent windows implemented using UpdateLayeredWindow can also be attached to the main window using SetParent.
and It can correctly display the content rendered in the main window.
Flitter also uses skia internally. flitter should be able to do it.


Creating account with 0 balance

Cannot change balance after so creating account with 0 balance is useless

Field now has 100,000 by default


Add sort functionality to data tables

Summary
Sort functionality allows the rows in a table to re-ordered based on the user's settings. Sorts can occur on different columns with different data types. Multiple sorts can be applied at same time.
💰 User value
Allows contents of data tables to be sorted so the content is easier to locate and consume.
📚 User Stories
If relevant, describe the high-level functionality as user stories.
As an ICDS user:
I need to be able to sort my data table by any column
So that I can order the data in a way that makes sense to me.
As an ICDS user:
I need to be able to sort by ascending or by descending
So that I can order the data either way.
As an ICDS user:
I need to be able to add alphanumeric sort, ascending or descending
So that I can order the data.
As an ICDS user:
I need to be able to add numeric sort, ascending or descending
So that I can order the data.
As an ICDS user:
I need to be able to add date sort, ascending or descending
So that I can order the data.
An an ICDS User
I need to be able to add multiple concurrent sorts, and see which sort is ordered where,
So that I have better control over my data.
📝 Acceptance Criteria
If relevant, describe in full detail the different interactions and edge cases that the component or patterns needs to fulfil.
Sorting order
Given that a column is sortable
When a user clicks on the sort icon button in the column header
Then the rows should be sorted initially in ascending order based on the data type:

Text: Alphanumeric sort (alphabetic)
Number or Currency: Numeric sort (lowest first)
Date: Date sort (oldest first)

Given that a column is sortable, and is already sorted ascending
When a user clicks on the sort icon button in the column header again
Then the rows should be sorted descending order based on the data type:

Text: Alphanumeric sort (alphabetic)
Number or Currency: Numeric sort (highest first)
Date: Date sort (newest first)

Given that a column is sortable, and is already sorted descending
When a user clicks on the sort icon button in the column header again
Then the sort should be removed and the rows return to their original unsorted order.
Custom sorting order
Given that a table is being set up, and a column is set to be sortable,
When setting its properties,
Then it should be possible to set a custom sorting order.
For example, instead of the default sorting order being 'ascending, descending, unsorted' it can be changed to 'ascending, unsorted' therefore not allowing a descending sort. Any combination of these can be used.
Default sort
Given that a table is being set up, and a column is set to be sortable
When setting its initial properties,
Then an initial sort can be added.
For example, a table showing band names could have a default sort applied to them so that they are initially sorted by age.
Sort exclusions
Given that a table is being set up and can be sorted
When setting its properties
Then certain rows can be excluded from sort to allow them to always appear at the top.
Multi-sort
Given that one column is already sorted
When clicking on another column and holding down the sort key (windows=ctrl, mac=cmd, or shift on either)
Then the new sort is added and each columns sort is identified by a numbered badge.
✏ Designs
If there's a Figma design file (or other mock-up), include it here.
## 🧾 Guidance
If there's written guidance or documentation, include it here.

AC to be moved to another ticket:
Sort exclusions
Given that a table is being set up and can be sorted
When setting its properties
Then certain rows can be excluded from sort to allow them to always appear at the top.
Multi-sort
Given that one column is already sorted
When clicking on another column and holding down the sort key (windows=ctrl, mac=cmd, or shift on either)
Then the new sort is added and each columns sort is identified by a numbered badge.


Is there Keypath support GraphQLMap

@lightsprint09: Going to close this now because I'm not sure what the issue is and it doesn't include a description, but feel free to add more information and reopen it.


Negative galaxy-galaxy correlation function

mock_observables.tpcf() returns negative value for the first bin, and also it is not monotonically decreasing over the first few rbins.
I suspect that might be due to insufficient number of randoms in computation of the 2pcf estimator.
This is the outcome of model.mock.compute_galaxy_clustering():
(array([  0.12239095,   0.17719995,   0.25655347,   0.37144299,
0.53778224,   0.77861137,   1.12728837,   1.63210958,
2.36299934,   3.42119545,   4.95327193,   7.17144145,
10.38294952,  15.03263208]),
array([ -1.        ,   0.12760279,   3.68148708,  11.60978411,
11.19051433,   7.99856754,   5.04833077,   3.03954944,
1.80054104,   1.0243043 ,   0.6106042 ,   0.34611761,
0.19264118,   0.10078104]))

I've been doing some tests on this issue with different r bins. I find that the issue mainly persists for r < 0.3.
If I specify r bin edges of
[  0.1          0.5          1.           1.2210553    1.49097605 ... ]
the xi values from compute_galaxy_clustering are
[ 9.61423222  7.83159952  5.06356391  4.03091415  3.00004464 ... ] .
No negative values and monotonically decreasing.
On the other hand if I specify r bin edges of
[  0.1, 0.3, 1., 1.2210553, 1.49097605 ... ]
the xi values from compute_galaxy_clustering are,
[ 2.18375258, 8.19514426,  5.17546647,  3.92509287,  2.95110477 ... ]

Xi = DD/RR - 1
RR is computed analytically, so that is not what's going on. A negative correlation function can, in principle, be a perfectly correct result (just look at the formula for the estimator). However, since you only see this in the inner bins, what is happening is just your point data happens to be sparse, so number counts are low due to random Poisson fluctuations. Make sense? Can you describe what problem this is causing for your code?

I see. It doesn't cause any problems with our code. The small scale behavior of the 2pcf was just unexpected.

Right, ok, that makes sense as a possibly alarming source of confusion, and you're not the first users to hit this source of confusion.  I think documenting it is the only way to handle this properly, since I can't think of a robust way to automatically detect what's going on here (unless you have suggestions?)
Do you have any suggestions for how to properly document this? Where did you look to see if this had been documented? Or if you didn't look, that's ok, but where might you have looked for some comment on this? It's just that this is likely to come up again.

I have a general suggestion:
In the two-point correlation function documentations, it would be helpful to include examples that demonstrate the calculation of 2-point functions (tpcf, wp, monopole, and quadrupole) for mock galaxy catalogs generated by Halotools.
I like the examples with randomly distributed set of points, and I find them useful for understanding how the underlying source-code works. But having examples that show the actual use of these two-point-functions in mock galaxy catalogs, would be more informative.
I'm not entirely sure if this was a relevant comment but I thought I'd bring it up.

@mjvakili I agreed with your comment and left this issue open as a reminder to myself. I now have some pretty extensive documentation on this:
http://halotools.readthedocs.org/en/latest/quickstart_and_tutorials/tutorials/catalog_analysis/galcat_analysis/index.html
Does this seem sufficient? If not, what else would help clarify? If so, and you think this addresses your confusion, would you just close the issue?

@aphearin : Yes, this looks great. Thanks. I'm closing the issue.
Sorry that my comment was not relevant to the title of the issue. I should have opened a different issue for this.


Make offset pane interactive

It would be cool if the offset pane was interactive so people could do things like manually enter custom offset coordinates or copy and paste an offset that they want to save and use again.

Oh, yeah this is a bug.  The offset panel field originally did allow these things.