System unavailable: build-osuosl-aix71-ppc64-2

Please put the system name in the title of this issue

https://ci.adoptopenjdk.net/computer/build-osuosl-aix71-ppc64-2/

Link to any log file showing the problem: https://ci.adoptopenjdk.net/computer/build-osuosl-aix71-ppc64-2/log

Please describe the issue: Problems with jenkins conneting to the buid machine to start the agent. This was seen after doing the work to move it to Temurin 17 for the agent in #2763 although the problem is occurring before it gets to tryin to invoke java. At the time of writing it's not showing anything in the startup log.

By means of comparison, build-osuosl-aix71-ppc64-1 is running ok.

This agent runs ok with Java 11, but has issues with JDK17

Is this open, or resolved?

Open at the moment, Im going to test with openjdk-17 to see if that has any differences from the temurin jdk

System is now running with JDK11 until the cause can be identified. Moving to March as the immediate problem is resolved but we need to diagnose the problem.

This issue has now been resolved ,with the addition of the prefixStartSlaveCmd='export LDR_CNTRL=MAXDATA=0x80000000 && parameter to jenkins, the agent now runs on JDK17


Fix serialization of LocalDate in custom structures

Tagging local dates with 1004, which are stored as elementValue in IssuerSignedItem doesn't work.
First approach doesn't solve it, see 92d71a4

See also https://github.com/a-sit-plus/mobile-driving-licence-credential/blob/main/mobiledrivinglicence/src/commonTest/kotlin/at/asitplus/wallet/mdl/CborSerializationTest.kt#L387

@nodh fixing the serializer is acutally a one-liner, but the deserializer still needs fixing

After Digging through deserialization, I have good and bad news
Good news
Our deserialization issue is absolutely solvable!
We need to the elementIdentifier to be parsed before the elementValue so we can perfectly decide which deserializer to use, when we encounder elementValue. I still don't quite know how we're supposed to verify the tags when reading everything back in, because tags are only read in decodeElementIndex and I don't know how we're supposed to use that (we anyhow shouldn't).
Bad news
We need to completely rework the internals of registerExtensionLibrary, which will also break its signature.
The naive approach replacing the current set of functions with maps (to have elementIdentfier-Strings map to deserializer functions won't get us all the way, I'm afraid: Just think of how many credential schemes support a Given Name or Date of Birth elementIdentifier with potentially clashing definitions.
What we need to do is maintain such maps for each registered credential scheme.
Problem is: I don't know where and how the credential scheme is used and if we can pass it to our deserializer somehow.
The protocol is scheme-aware so it should be possible to wire it up accordingly, but from the top of my head, I really don't know how. @nodh we need to talk about this particular issue. If you can walk me through the code for this part, I think I can work out the rest myself

@nodh see also my commits to your branch in VC-K

What's the deal with tag 24? I don't really know what's going on, but tried to preserve it

see #122

Fixed in #122


Multilingual indexing - documents of all languages but one deleted in index due to shared itemGuid

Describe the bug
Indexing of webpages translated into multiple languages seems to only store 1 language version for a page/item and deletes Lucene documents of other language versions in the index. Single Lucene document is only of single language, and as the Xperience content items are recognized using itemGuid (common for all lang.versions), the previously created ones are deleted in the index.
Is that expected behavior if you allow multiple languages for the index in Xperience? Or how should one adjust the code/setup/configuration properly for multilingual in current implementation?
To Reproduce
On XbyK instance with sample DancingGoat site,

Set up e.g. example indexing (from /xperience-by-kentico-lucene/tree/main/examples/DancingGoat/Search)
Create and configure an index e.g. for Article page type, with both English and Spanish languages
Rebuild the index
See number of indexed documents being 5 (6 articles, 1 single language is secured so is not indexed, 2 of other ones are multilingual). Inspect the actual documents e.g. using lucene tool (Luke) with their data.

Expected behavior
With previous steps, 7 indexed documents (5 in En lang., 2 in Es lang.) could be expected.
Generally, indexing either creates separate Lucene documents for different language versions of an Xperience page (or content item), or, the multilingual content is present within one Lucene document (doesn't seem preferable).
Library Version
Xperience by Kentico 29.1.5, xperience-by-kentico-lucene 7.0.1
Additional context
When indexing, after finding an existing item GUID  (part of base properties):
https://github.com/Kentico/xperience-by-kentico-lucene/blob/main/src/Kentico.Xperience.Lucene.Core/Indexing/DefaultLuceneTaskProcessor.cs#L135C9-L135C110
the code in https://github.com/Kentico/xperience-by-kentico-lucene/blob/main/src/Kentico.Xperience.Lucene.Core/Indexing/DefaultLuceneClient.cs#L27
deletes existing document, the code comment says:
                        // for now all changes are creates, update to be done later
                        // delete old document, there is no upsert nor update in Lucene

Hi @kentico-zdenekc , fix has been released in v7.0.2


PHDI Blob Trigger Help

Problem statement
We've got a blob trigger function app (https://github.com/CDCgov/prime-public-health-data-infrastructure/tree/main/src/FunctionApps/DSTP) that we haven't been able to get quite working with the various permissions in Azure. Can someone help identify exactly what's going wrong with the DSTP app?
What you need to know
We've tried to follow @JosiahSiegel's sample app here (https://github.com/CDCgov/prime-public-health-data-infrastructure/pull/75/files#diff-13f8d90368098e676a0f45f329867684d7c9ee90d3df22e014807a5fc67761d8), but haven't been able to get the output fully working.
Acceptance criteria

[ ] The DSTP app outputs something to the outbound container in the pitestdatasa storage account based on another blob being uploaded

Phdi were able to make blob triggers work


iron-list failing tests

Some of the tests fail when running pub run test -pdartium, but interestingly not when you run them manually.

Fixed these up


[BUG]final result = await PhotoManager.editor.deleteWithIds(ids)，接受不到返回值。

在android环境下，删除本地相册图片， 图片删除成功，但是
final result = await PhotoManager.editor.deleteWithIds(ids);
print('---------- delete, ${result.isNotEmpty}');
在result后的代码不执行。（也就是print这行代码不执行， 跑的demo也是同样情况）。是flutter版本问题还是其他，望解答。 急急急。

android os version?

By the way this also happens on iOS. It is not a android only issue.

I just saw this PR #354 does this fixes this issue? Anyone has test it out?

I've just tested it using:
  photo_manager:
    git:
      url: https://github.com/CaiJingLong/flutter_photo_manager.git
      ref: master

And I can confirm that the problem is still happening. Using the master branch the pic is not even deleted. On version 0.5.8 the pic is at least deleted the problem is that the code doesn't keep running.

android os version?

android version:10

android os version?

10,
_clickDeleteAction() {
PhotoManager.editor.deleteWithIds([widget.id]);
Navigator.pop(context);
}
我目前是这么使用的， 但是不能在删除操作执行完以后返回， 很难受。

Yep without the await it works but we won't know for sure if the picture was really deleted.
Still I found that when compiling using sdkVersion 29 in my device at least it only worked when combining with the permission:
android:requestLegacyExternalStorage="true"
On the AndroidManifest.

Fixed in #367.
Please wait for 0.6.0 version.


feat: update to eye v8.1.0

feat: update to eye v8.1.0

:tada: This PR is included in version 10.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v10.1.0

Your semantic-release bot :package::rocket:


Shared folder - Nuxt Blog annoucement

Hi!
It seems that the new shared folder feature annouced on the Nuxt Blog requires Nuxt V4 although the post makes it seem like it is also available for V3.
Maybe either the feature should be backported to V4 or the blog post should be updated accordingly?

It should be available in both, V3 and V4 in "the same place".

Here it is.
This example runs on Nuxt V3 and the shared folder doesn't work.
https://stackblitz.com/edit/github-aasqgz-two1vs?file=app.vue


[3.9-SB] Revamp Study Browser Panel Design: Redesign study browser panel with optimized layout and visual indicators.

What feature or change would you like to see made?
Redesign the Study Browser panel to align with the modern design of other viewer components. This should include:

Reducing wasted space and optimizing layout
Adding viewport indicators (e.g., dropdown to add display set as overlay to active viewport)
Incorporating visual indicators for display set status (loading, prefetching, warnings, etc.)
Adding sorting functions and sorting UI selector (series date, series number etc.)
Adding thumbnail and no thumbnail views
Explore horizontal view of the study browser
Better prior display support
Better sop class handler splitting support
study prefetcher

Why should we prioritize this feature?
The Study Browser panel has the oldest design and does not conform to the new UI standards. Revamping its design will provide a more consistent user experience across the application. Additionally, optimizing the layout and adding visual indicators will improve usability and make it easier for users to understand the state of display sets.

related https://github.com/OHIF/Viewers/issues/3363

Use Case: Handling Mixed SOP Classes in a Series
When a series contains multiple SOP classes, such as both video and images (where both could be considered secondary capture), the viewer should split them into distinct display sets. This can be achieved through a custom SOP class handler or via the UI. The options include:
Separate Display Sets: Create separate display sets for each type of content (video and images), ensuring they are easily identifiable as belonging to the same series. This can be done by: Implementing a custom SOP class handler to manage the split.
Combined Display Set: Develop a special kind of display set that can contain both types of data, clearly indicating that they originate from the same series. This could involve: Creating a new display set format that integrates both videos and images while maintaining clarity.
Enhanced Viewport: Introduce a new kind of viewport capable of displaying a stack of images alongside video. However, this approach might lead to confusion (just a thought).

Current need to finish ticket:

Displayset warning messages
Series number is no longer present
We need modality in the list view atleast or both (thumbnail and list)
Tracking identifer in the tracking panel
RT/SEG/SR labels etc in thumbnail view
border around images

https://github.com/OHIF/Viewers/pull/4254

This is done and merged code-wise. pending feedback from the community for the new iteration

Response from community feedback:

Change layout so that empty thumbnail (like seg) defaults to list view beneath the other thumbnails
Add More Options (...) button to study panel and add tag browser option to that button
Change default filter to all

@IbrahimCSAE I'm added a visual for each of the changes above.
Non thumbnail items in grid view
This is a behavior update, no design changes to components

Action button on Series items
This design was already completed, but since we had no actions, we didn't implement it. This will use the Button component (ghost variant) and the DropdownMenu component from this PR

Default to "ALL" when anything is opened from the Study List
No design changes, this is just a behavior change that "All" in the select menu will be the default when open

Done in

https://github.com/OHIF/Viewers/pull/4254
https://github.com/OHIF/Viewers/pull/4417


"Used coupon" - Gift page

Describe the bug
Everytime, even if you not used the coupon, we have this information "coupon already used" without doing anything. It's really bad for user experience and people need to ask another coupon (when they can).

Up! Just I can't assign someone for this issue


[SDP-1284] Fix bug introduced in PR #134 where the upload csv button would never be enabled

What
Fix bug introduced in PR #134 where the upload csv button would never be enabled.
Why
The bug was caused because the PR removed the {renderDropdownDefault(isWalletAssetsFetching)} line of code that inserts the (initial) "empty dropdown value" to the dropdown list, and it resulted in preventing the dropdown selection changed event from being called when the assets dropdown list contained a single item. That mistake prevents disbursement instructions from being uploaded.
Additional Change(s)
Updated the syntax of the javascript filter function to return explicit booleans.

stellar-disbursement-platform-frontend-preview is available here:http://sdp-frontend-pr140.previews.kube001.services.stellar-ops.com/

stellar-disbursement-platform-frontend-preview is available here:http://sdp-frontend-pr140.previews.kube001.services.stellar-ops.com/

stellar-disbursement-platform-frontend-preview is available here:http://sdp-frontend-pr140.previews.kube001.services.stellar-ops.com/


Removing User Group Member does not cause removal of related User's Access Control records

This is actually a security vulnerability. After a quick Base Application code analysis I discovered that "App ID" was a NULL GUID in Access Control (& related) records in v.16.1, but in v.18.0 it is filled with System Application's ID almost everywhere... except Access Control table (it's still a NULL GUID there). As a consequence, the following execution path does not cause the records to be deleted as expected:
UserGroupMember(OnDelete)->UserGroupAccessControl.RemoveUserGroupMember->RemovePermissionSetFromUser
...
AccessControlExists := AccessControl.Get(UserSecurityID, RoleID, SelectedCompany, ItemScope, AppID);
if AccessControlExists then begin
// AccessControlExists is always FALSE with non-NULL GUID AppID and NULL GUID existing in AccessControl table, so execution not reaches this place.
Reset;
SetRange("User Security ID", UserSecurityID);
SetRange("Role ID", RoleID);
SetRange("Company Name", SelectedCompany);
SetRange(Scope, ItemScope);
SetRange("App ID", AppID);
ReferenceExists := FindLast;
if not ReferenceExists then
AccessControl.Delete(true);
if ReferenceExists and ("User Group Code" = '') then
Delete;
end;
Bug observed in: Version: NO Business Central 18.0 (Platform 18.0.22893.23773 + Application 18.0.23013.23795)
Worked Ok in: Version: NO Business Central 16.1 (Platform 16.0.12630.12758 + Application 16.1.12805.0)

I have tried to "fix" the "App ID" value on "Access Control" record (i.e. set it to System Application's ID: '63CA2FA4-4F03-4F2B-A480-172FEF340D3F') in several ways:

OnBeforeInsert event - "App ID" actually showed up correctly (was the System Application's ID), but then disappeared in onAfterInsert event. Fixing it in OnBeforeInsert trigger did not work - "App ID" was still '00000000-0000-0000-0000-000000000000' in the database.

Via Job Queue scheduled in OnAfterInsert event handler - I stored RecordRef "Record ID to Process", and it actually got the correct (System Application's) "App ID" before I scheduled a JQ (!), but then record could not be found in JQ process (using just Find()), so I tried to Get(...) the record, replacing the "App ID" key component with a null Guid. Record was found, I could Rename(...) it without error, run Commit() just for extra "blessing", but "App ID" still remained '00000000-0000-0000-0000-000000000000' in the database.

After creating a record (via "User Group Member" creation), I changed manually in SQL the "App ID" of the inserted "Access Control", tried to remove "User Group Member" via User Card and... it worked! Relevant permissions were magically gone!

So I created SQL "ON [dbo].[Access Control] AFTER INSERT" trigger and everything began to work as it should - as in BC v.16...

It looks like "Access Control"."App ID" is partially processed by underlying BC code, but it becomes "partially unreadable" somewhere after OnBeforeInsert and OnAfterInsert events, and some built-in BC functions (such as RecRef.GetTable(...)) can still read it. However, is not stored correctly in the database.

I have tried to "fix" the "App ID" value on "Access Control" record (i.e. set it to System Application's ID: '63CA2FA4-4F03-4F2B-A480-172FEF340D3F') in several ways:

OnBeforeInsert event - "App ID" actually showed up correctly (was the System Application's ID), but then disappeared in onAfterInsert event. Fixing it in OnBeforeInsert trigger did not work - "App ID" was still '00000000-0000-0000-0000-000000000000' in the database.

Via Job Queue scheduled in OnAfterInsert event handler - I stored RecordRef "Record ID to Process", and it actually got the correct (System Application's) "App ID" before I scheduled a JQ (!), but then record could not be found in JQ process (using just Find()), so I tried to Get(...) the record, replacing the "App ID" key component with a null Guid. Record was found, I could Rename(...) it without error, run Commit() just for extra "blessing", but "App ID" still remained '00000000-0000-0000-0000-000000000000' in the database.

After creating a record (via "User Group Member" creation), I changed manually in SQL the "App ID" of the inserted "Access Control", tried to remove "User Group Member" via User Card and... it worked! Relevant permissions were magically gone!

So I created SQL "ON [dbo].[Access Control] AFTER INSERT" trigger and everything began to work as it should - as in BC v.16...
It looks like "Access Control"."App ID" is partially processed by underlying BC code, but it becomes "partially unreadable" somewhere after OnBeforeInsert event and before OnAfterInsert event, and some built-in BC functions (such as RecRef.GetTable(...)) can still read it. However, it is not stored correctly in the database.

I have tried to "fix" the "App ID" value on "Access Control" record (i.e. set it to System Application's ID: '63CA2FA4-4F03-4F2B-A480-172FEF340D3F') in several ways:

OnBeforeInsert event - "App ID" actually showed up correctly (was the System Application's ID), but then disappeared in onAfterInsert event. Fixing it in OnBeforeInsert trigger did not work - "App ID" was still '00000000-0000-0000-0000-000000000000' in the database.

Via Job Queue scheduled in OnAfterInsert event handler - I stored RecordRef "Record ID to Process", and it actually got the correct (System Application's) "App ID" before I scheduled a JQ (!), but then record could not be found in JQ process (using just Find()), so I tried to Get(...) the record, replacing the "App ID" key component with a null Guid. Record was found, I could Rename(...) it without error, run Commit() just for extra "blessing", but "App ID" still remained '00000000-0000-0000-0000-000000000000' in the database.

After creating a record (via "User Group Member" creation), I changed manually in SQL the "App ID" of the inserted "Access Control", tried to remove "User Group Member" via User Card and... it worked! Relevant permissions were magically gone!

So I created SQL "ON [dbo].[Access Control] AFTER INSERT" trigger and everything began to work as it should - as in BC v.16...

It looks like "Access Control"."App ID" is partially processed by underlying BC code, but it becomes "partially unreadable" somewhere after OnBeforeInsert event and before OnAfterInsert event, and some built-in BC functions (such as RecRef.GetTable(...)) can still read it. However, it is not stored correctly in the database.
I guess that using only Permission Sets with pre-defined Application ID could solve the issue, but this looks a bit crappy that so strange behavior occurs involving some activity that happens "behind the scenes" and breaks data (Access Control & related) integrity.

I have tried to "fix" the "App ID" value on "Access Control" record (i.e. set it to System Application's ID: '63CA2FA4-4F03-4F2B-A480-172FEF340D3F') in several ways:

OnBeforeInsert event - "App ID" actually showed up correctly (was the System Application's ID), but then disappeared in onAfterInsert event. Fixing it in OnBeforeInsert trigger did not work - "App ID" was still '00000000-0000-0000-0000-000000000000' in the database.

Via Job Queue scheduled in OnAfterInsert event handler - I stored RecordRef "Record ID to Process", and it actually got the correct (System Application's) "App ID" before I scheduled a JQ (!), but then record could not be found in JQ process (using just Find()), so I tried to Get(...) the record, replacing the "App ID" key component with a null Guid. Record was found, I could Rename(...) it without error, run Commit() just for extra "blessing", but "App ID" still remained '00000000-0000-0000-0000-000000000000' in the database.

After creating a record (via "User Group Member" creation), I changed manually in SQL the "App ID" of the inserted "Access Control", tried to remove "User Group Member" via User Card and... it worked! Relevant permissions were magically gone!

So I created SQL "ON [dbo].[Access Control] AFTER INSERT" trigger and everything began to work as it should - as in BC v.16...

It looks like "Access Control"."App ID" is partially processed by underlying BC code, but it becomes "partially unreadable" somewhere after OnBeforeInsert event and before OnAfterInsert event, and some built-in BC functions (such as RecRef.GetTable(...)) can still read it. However, it is not stored correctly in the database.
I guess that using only Permission Sets with pre-defined Application ID could potentially solve the issue, but this looks a bit crappy that so strange behavior occurs involving some activity that happens "behind the scenes" and breaks data (Access Control & related) integrity. I have observed (but no 100% sure) various behaviors (random? hopefully not...) in different cases - deleting of some User Group Members "cascaded" correctly removal of Access Control records, while some didn't. It's Friday afternoon, so I'm giving up on this research, but hopefully you will take a look at this. Have a good weekend! :)


Avoid integer overflow in xh_vm_map_gpa()

This is related to FreeBSD-SA-16:38.bhyve[0] AKA CVE-2016-1889 which was
discovered by Felix Wilhelm. In the context of hyperkit the code in question
runs as an unprivileged user.
Thanks to Gleb Smirnoff and the FreeBSD security team for advanced notice on
this issue.
[0] https://lists.freebsd.org/pipermail/freebsd-announce/2016-December/001773.html
Signed-off-by: Ian Campbell<EMAIL_ADDRESS>

LGTM

LGTM


Register w/Benevity

I have made initial contact with Benevity (Apple's method for giving). Their requirements for inclusion are not clearly stated on website. I was hoping to get ahead of the inclusion while we wait on 501c3 approval, although I think that is a requirement.

Submitted IRS determination letter to Benevity on 4/28.

Benevity has accepted us. Will be sending link today to create profile page within Benevity. Need signed verification letter from Citizen Labs that states that I can act on behalf of Citizen Labs.

All has been completed for this.


modify asr engine for paddlespeech server

Describe
modify the executor definition to be consistent with the ASRExecutor class.

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

参看 https://github.com/PaddlePaddle/PaddleSpeech/pull/2302


Compile errors with 1.8 targetted code

Trying to ditch my own internal Kotlin script usage with kscript
https://github.com/yschimke/oksocial-scratchpad/pull/1/files
Getting this error when running from a command line
$ ./src/main/kotlin/commands/giphysearch.kts trump
[kscript] [ERROR] compilation of './src/main/kotlin/commands/giphysearch.kts' failed
src/main/kotlin/commands/giphysearch.kts:17:21: error: cannot inline bytecode built with JVM target 1.8 into bytecode that is being built with JVM target 1.6. Please specify proper '-jvm-target' option
  val urls = client.query<SearchResults>(
                    ^

The kotlin default target is still 1.6. to compile against 1.8 you need to
provide the compiler flag via  //Compile_Opts or the corresponding
annotation. See readme.
Yuri Schimke<EMAIL_ADDRESS>schrieb am So., 25. März 2018,
09:47:

Trying to ditch my own internal Kotlin script usage with kscript
https://github.com/yschimke/oksocial-scratchpad/pull/1/files
Getting this error when running from a command line
$ ./src/main/kotlin/commands/giphysearch.kts trump
[kscript] [ERROR] compilation of './src/main/kotlin/commands/giphysearch.kts' failed
src/main/kotlin/commands/giphysearch.kts:17:21: error: cannot inline bytecode built with JVM target 1.8 into bytecode that is being built with JVM target 1.6. Please specify proper '-jvm-target' option
val urls = client.query(
^
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/holgerbrandl/kscript/issues/114, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMQ-EQFtzxDWuDmmHkZfLsc_ldppnQBks5th0uWgaJpZM4S6DBQ
.

Is there a real annotation behind file:CompilerOpts, it is flagged as red in Intellij, while DependsOn comes from import org.jetbrains.kotlin.script.util.DependsOn
@file:DependsOn("com.baulsupp:oksocial:1.37.0")
@file:CompilerOpts("-jvm-target 1.8")
//COMPILER_OPTS -jvm-target 1.8

n.m. found
implementation 'com.github.holgerbrandl:kscript-annotations:1.2'

It's added by default when using annotations in a script processed by kscript. Not sure what org.jetbrains.kotlin.script.util.DependsOn is all about, I'm seeing it for the first time.


Add 'USA' to the list of aliases

The list of aliases is obviously missing the USA. Please include! Thanks!

This PR does two things, and while I am reluctant but indifferent about the actual purpose, the additional changes I definitely do not accept.

@alcohol I can split this PR on two parts, which can be reviewed separately.
"Additional" changes allows lookup for country names like "Cote d’Ivoire" without using "ô" characters (I don't even have this character in my keyboard layout).

@alcohol I reverted changes for lookup the country names in ASCII.
So current PR is very simple, although I still plan to suggest ASCII lookup in another separate PR.


Conduit Workflow

DO NOT LOOK that I created the article to follow the author. This is extra work in this test, but I needed it for other purposes.

Well done!


Deleted users stay logged in

When deleted a user who is logged in can still access the site. They are still able to make some changes to things and will cause exceptions to be thrown.

An approach for this could be:

Set a global cache value indicating when/if a user was deleted (e.g. this could be a counter that starts at 0 when the site starts up)
In the login process, make a note of that global cache value in the session
When a user is deleted, update that global cache value (in the case of a counter, just increment it)
On page load, check the cached value and the session value - if they differ, the current user should be validated to still have an account (this would trigger item 2 above to happen again)

This issue can also come up when a head of household is signed in as a member of their household and someone adds the head of household to a different household causing them to no longer be a head of household with their ActiveUserId still being set to a household member.


Error when click on "visualize DB" in huge JPA diagram

Overview of the issue
In a diagram with more than 40 entities, don't allows visualize database when click right  button. And from the beging with this diagram also does not show the relation edges (the lines) correctly. Shows relation fields inside entities in both sides but missing line.
Expected Outcome
It's supoused to show database diagram.

Shows the error uploaded, and when closing error window keep working as usual.

ModelGeralJPA.zip
Just right click and select "visualize DB", and error came.
Jeddict Version: 4.4.1
Product Version: NetBeans IDE 8.2 (Build<PHONE_NUMBER>07)
Updates: NetBeans IDE is updated to version NetBeans 8.2 Patch 2
Java: 1.8.0_121; Java HotSpot(TM) 64-Bit Server VM 25.121-b13
Runtime: Java(TM) SE Runtime Environment 1.8.0_121-b13
System: Windows 10 version 10.0 running on amd64; Cp1252; es_AR (nb)
User directory: C:\Users\lean\AppData\Roaming\NetBeans\8.2
Cache directory: C:\Users\lean\AppData\Local\NetBeans\Cache\8.2
System specs:
Notebook Asus
Intel i7-6500U CPU @ 2.50GHz  2.59GHz
RAM 16GB (15.9 usable)
64 bit
Win 10 Home
OS compilation 16299.309

Hi @jGauravGupta,
look in this two entities: Venta & Entrada . The fields (entrada-venta) inside relation part are one edge, but line is gone.
Now i've updated to lastest version (4.7) in this picture below.
Thank you for having answered so fast!

Hi @lubriBoy ,
Had you created this diagram manually (by drag-n-drop of entities) or by reverse engineering the existing class.

First I had created three diagram separately.
This one have the relation between "Venta-Entrada".

Then generate source code from all of them and I did reverse engineering from that source code to unificate all three diagram.
So, could we say it's kind of drag and drop way, no?

https://github.com/jeddict/jCode/commit/3dc4f52d63b497994f569b2e7295cdb0d4b8ca66


Fully sync payload by traversing all links in Pando such that all of payload can be downloaded solely from Pando.

cc @kylehuntsman

Pando payload type is now updated to Any allowing dealbot to publish DAGs as payload which then allows Pando to traverse them via graphsync.
Next step is to update dealbot to publish metadata with the latest schema.

payload format changes are released and deployed in fil-prod.

Should this be captured on the Pando repo instead of here? Work from dealbot side as far as getting this done is now complete.


What error?

I run this command
slackin "your-team-id" "your-slack-token"
But I get this error. It's nonstop

That's how it works.
You can set the polling interval with this flag:
    -i, --interval <int>       How frequently (ms) to poll Slack [$SLACK_INTERVAL or 5000]


Function counts differently when collection from LINQ isn't materialized

After I updated my application from .net Core 2.0 to 2.1 (with packages) I've noticed that one function is returning only half records. So I was trying to debug this problem and check what I've found:

That method returns only 5 elements (what is totally incorrect). I put the same function (but with .ToList() before .Take(pageSize) in watch and it returns 10 elements (what is correct).
Generated SQL query that I catched in SQL Profiler returns 10 elements.
Steps to reproduce
The bug is in LINQ query
public Page<GroupTotal> Get(QueryParameter queryParameter)
{
    var summariesQuery = _context.Groups.AsNoTracking()
        .Where(g => g.IsActive)
        .GroupBy(g => new { g.GroupName, g.GroupId });

    return summariesQuery
        .Select(summary => new GroupTotal
        {
            Id = summary.Key.GroupId,
            Name = summary.Key.GroupName,
            Mode = summary.Select(s => s.Mode).ToArray() // THIS LINE MAKES PROBLEM
        })
        .ToPage(queryParameter.PageSize, queryParameter.PageNumber);
}

public static Page<T> ToPage<T>(this IQueryable<T> source, int pageSize, int pageNumber)
{
    var items = source.Skip(pageSize * (pageNumber - 1))
        .Take(pageSize)
        .ToList();

    return new Page<T>
    {
        PageSize = pageSize,
        PageNumber = pageNumber,
        Items = items,
        TotalCount = source.Count()
    };
}

If I'll remove Mode = summary.Select(s => s.Mode).ToArray() this line from my LINQ query then everything works fine.
Further technical details
EF Core version: EntityFrameworkCore.SqlServer 2.1.0
Database Provider: Microsoft.EntityFrameworkCore.SqlServer 2.1.0
Operating system: Windows 10
IDE: Visual Studio 2017 15.7.3

Perhaps it might be because of .GroupBy(g => new { g.GroupName, g.GroupId }); I heard that peopole have problem with GroupBy in LINQ

I’m not so expereinced in EF but i’m good in SQL and LINQ, so there are my 5 cents.
Your query is not effective for SQL Server and EF have to create special transformation of this query to:
var summariesGrouped = _context.Groups.AsNoTracking()
        .Where(g => g.IsActive)
        .Select(g => new { g.GroupName, g.GroupId })
        .Distinct();

var summariesPaged = summariesGrouped
        .Skip(pageSize * (pageNumber - 1))
        .Take(pageSize);

return summariesPaged.
        .Join(_context.Groups, l => new { l.GroupName, l.GroupId }, r => new { r.GroupName, r.GroupId },
              (l, r) => new { r.GroupName, r.GroupId, r.Mode })
        .AsEnumerable()  // enforce client side evaluation
        .GroupBy(g => new { g.GroupName, g.GroupId })
        .Select(summary => new GroupTotal
        {
            Id = summary.Key.GroupId,
            Name = summary.Key.GroupName,
            Mode = summary.Select(s => s.Mode).ToArray()
        });

If EF Core is not doing this non trivial transformation - your query is not effective and you will get client side evaluation of almost whole table on client side which equals to server death.
Wellcome to the real world.

@sdanyliv How you know that? How you know that will be transformated in that way? Or how can I learn it somehow to avoid not efficient LINQ queries?

@pklejnowski, i don‘t know how EF optimizes queries. It should, but it is on early stages of that, but possibly i’m wrong.
I have writen query that is most effective in your situation based on my experience. And just warning you - analyse SQL. If you will get expected result but SQL does not contain OFFSET, FETCH keywords - you have client evaluation.

@pklejnowski - The issues people having are around Group By with aggregate operators which would generate Server GROUP BY, which we started translating in 2.1 release. This is not that case.
@maumar - Would correlated subquery optimization apply in this case? There is ToArray but also GroupBy which would be client eval.

As I said in first post. Generated SQL looks fine and returns correct results. I have tracked it by sql profiler.

@pklejnowski - Can you post generated SQL(s)?

@pklejnowski, your line with comment // THIS LINE MAKES PROBLEM brings additional complexity for the query. That’s what i’m trying to explain.

@smitpatel
Here is the generated query tracked by SQL Profiler:
SELECT  	[g].[GroupId],  	[g].[GroupName], 	[g].[Mode] FROM  	[Read].[Groups] AS [g]  WHERE  	[g].[Mode] = @__mode_0  	ORDER BY [g].[GroupName], [g].[GroupId] OFFSET @__p_1 ROWS FETCH NEXT @__p_2 ROWS ONLY', 	N'@__mode_0 int, 	@__p_1 int, 	@__p_2 int', 	@__mode_0=1, 	@__p_1=0, 	@__p_2=10
And of course it return 10 records. I don't know why LINQ query returns only 5 though.

Intresting thing. I've returned to EntityFrameworkCore2.0 and I've checked generated SQL query and there is without OFFSET and FETCH part. So is totally inefficient in 2.0 and in 2.1 doesn't work ^^

@pklejnowski, last one try. Just trying to help you in this situation.
It is because your query is trying to retrieve details of grouped records and apply paging to that group. It's not trivial query without client evaluation and i have rewritten it to you to work as expected here. I know no one ORM that can do this automatically (I'm maintainer of one of them).
Query should have the following structure:

Group records (or distinct) and select only grouping key - result1
Apply paging to result1 - only at this stage you can apply paging!
Join result1 to same recordset (it filters records and paging will be applied correctly) and select only needed columns - result2
Group result2 on client side because we need details ummary.Select(s => s.Mode).ToArray()

Ahhh I see, I'll check this. By the way what is "client evaluation"? Can you explain it?

Client evaluation means loading almost all table to your application and group records on your machine.
It's very dangerous because of high memory usage and may lead to OutOfMemory.

Thank you. What characterizes not trivial query? In the future I would like to intuitively recognize which query may pose a problem.

@pklejnowski, there are a lot of them but usually you will have problem with grouping.
For GROUP BY query SQL Server can return only grouping keys and aggregation result with predefined (or custom) aggregation functions COUNT, SUM, MIN, MAX, ect..
SELECT
  GroupKey1,
  GroupKey2,
  COUNT(*),
  MIN(Field),
  ...
FROM SomeTable
GROUP BY
  GroupKey1,
  GroupKey2

It's easy and EF works well in that case and you can apply paging to this query. BUT if you need details of the grouped values - things become complicated.
For example change your query to get only first Mode:
summariesQuery
        .Select(summary => new GroupTotal
        {
            Id = summary.Key.GroupId,
            Name = summary.Key.GroupName,
            Mode = summary.Select(s => s.Mode).First() // THIS ALSO MAKES PROBLEM
        })

Solving this sample is ineffective even with my solution and can be done only by Window Functions, which are not supported by EF Core, and you have to write raw SQL. You can check my sample in our repository https://github.com/linq2db/linq2db/issues/1129#issuecomment-386910284 , but as mentioned before, this functionality is not available in EF Core, but not forever.

@sdanyliv I see. You explained a lot to me. I always thought that everything in EF+LINQ is happening magically in best way but clearly not.
Based on your example if I need for my Pager total count, then it wouldn't be a problem to add var totalCount = summariesGrouped.Count() at the end?

If total count of groups - yes, but notice that this will run additional query.

Yes I'm aware of that. But I have no choice :) I need to know how many of them are in the database, since I have to display page number list :)

I always thought that everything in EF+LINQ is happening magically in best way but clearly not.

Don't trust anyone :)
disabling-client-evaluation

This warning looks nice but is too sensitive I think. Because it throw exception even for _context.Groups.Max(g => g.year);

In this case also?
_context.Groups.AsNoTracking().Max(g => g.year);

Also. However on EntityFrameworkCore2.1 works fine even without AsNoTracking. So maybe they had bug in 2.0 version. Anyway should I turn on this option? When Client Evaluation can be useful?

We had discussion about this in our team. Conclusion is: never, if you do not need change tracking for these entities. You can always turn on client side evaluation in appropriate place via AsEnumerable() or by ToArrayAsync() extension methods.
Otherwise instead of SQL Server calculation you will get uncontrolled copy of the table on the client side.

@pklejnowski
_context.Groups.Max(g => g.Year)

should not cause client evaluation.  Can you post a full code sample that shows the problem?

@maumar
System.InvalidOperationException: 'Warning as error exception for warning 'Microsoft.EntityFrameworkCore.Query.QueryClientEvaluationWarning': The LINQ expression 'Max()' could not be translated and will be evaluated locally. To suppress this Exception use the DbContextOptionsBuilder.ConfigureWarnings API. ConfigureWarnings can be used when overriding the DbContext.OnConfiguring method or using AddDbContext on the application service provider.'

you only see this in 2.0 right? or 2.1 also?

Only 2.0, in 2.1 is fine

@sdanyliv
Client Evaluation is the new feature of EntityFrameworkCore? Is something like that for the old EF in .net 4.5 as well? I am shocked that there is very little information in google about Client Evaluation.

@pklejnowski, i'm just keeping one eye open on how things are going in EF Core. As i know this "feature" exists from first versions of EF Core.
Maybe i'm wrong but, from my observations, if EF Core can not generate SQL - client evaluation happens.

@sdanyliv one more question about this code you have suggested. Shouldn't I add  .Where(g => g.IsActive) to query since this condition is in summariesGrouped as well? Without that data might be filtered wrong I think?
So it would be:
return summariesPaged.
        .Join(_context.Groups, l => new { l.GroupName, l.GroupId }, r => new { r.GroupName, r.GroupId },
              (l, r) => new { r.GroupName, r.GroupId, r.Mode })
        .AsEnumerable()  // enforce client side evaluation
        .GroupBy(g => new { g.GroupName, g.GroupId })
        .Select(summary => new GroupTotal
        {
            Id = summary.Key.GroupId,
            Name = summary.Key.GroupName,
            Mode = summary.Select(s => s.Mode).ToArray()
        });

Yes, my fault. Looks like filter needed.

@pklejnowski - Based on code snippets posted in first post, I am not able to reproduce this issue. I am seeing correct number of records and SQL generated after applying Skip/Take.
Repro code:
using System;
using System.Linq;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Logging;

namespace EFSampleApp
{
    public class Program
    {
        public static void Main(string[] args)
        {
            using (var db = new MyContext())
            {
                // Recreate database
                db.Database.EnsureDeleted();
                db.Database.EnsureCreated();

                // Seed database
                db.AddRange(new Group
                {
                    GroupName = "Group1",
                    IsActive = true,
                    Mode = new Mode()
                },
                new Group
                {
                    GroupName = "Group2",
                    IsActive = true,
                    Mode = new Mode()
                }, new Group
                {
                    GroupName = "Group3",
                    IsActive = true,
                    Mode = new Mode()
                }, new Group
                {
                    GroupName = "Group4",
                    IsActive = true,
                    Mode = new Mode()
                }, new Group
                {
                    GroupName = "Group5",
                    IsActive = true,
                    Mode = new Mode()
                }, new Group
                {
                    GroupName = "Group6",
                    IsActive = true,
                    Mode = new Mode()
                }, new Group
                {
                    GroupName = "Group7",
                    IsActive = true,
                    Mode = new Mode()
                }, new Group
                {
                    GroupName = "Group8",
                    IsActive = true,
                    Mode = new Mode()
                }, new Group
                {
                    GroupName = "Group9",
                    IsActive = true,
                    Mode = new Mode()
                }, new Group
                {
                    GroupName = "Group10",
                    IsActive = true,
                    Mode = new Mode()
                });

                db.SaveChanges();
            }

            using (var db = new MyContext())
            {
                // Run queries
                var query = db.Groups.AsNoTracking()
                    .Where(g => g.IsActive)
                    .GroupBy(g => new { g.GroupName, g.GroupId })
                    .Select(summary => new GroupTotal
                    {

                        Id = summary.Key.GroupId,
                        Name = summary.Key.GroupName,
                        Mode = summary.Select(s => s.Mode).ToArray() // THIS LINE MAKES PROBLEM
                    })
                    .Skip(0)
                    .Take(10)
                    .ToList();

                Console.WriteLine(query.Count);
            }

            Console.WriteLine("Program finished.");
        }
    }

    public class MyContext : DbContext
    {
        private static ILoggerFactory LoggerFactory => new LoggerFactory().AddConsole(LogLevel.Trace);

        // Declare DBSets
        public DbSet<Blog> Blogs { get; set; }
        public DbSet<Group> Groups { get; set; }

        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        {
            // Select 1 provider
            optionsBuilder
                .UseSqlServer(@"Server=(localdb)\mssqllocaldb;Database=_ModelApp;Trusted_Connection=True;Connect Timeout=5;ConnectRetryCount=0")
                //.UseSqlite("filename=_modelApp.db")
                //.UseInMemoryDatabase(databaseName: "_modelApp")
                .EnableSensitiveDataLogging()
                .UseLoggerFactory(LoggerFactory);
        }

        protected override void OnModelCreating(ModelBuilder modelBuilder)
        {
            // Configure model
        }
    }

    public class Blog
    {
        public int Id { get; set; }
    }

    public class Group
    {
        public int GroupId { get; set; }
        public string GroupName { get; set; }
        public bool IsActive { get; set; }
        public Mode Mode { get; set; }
    }

    public class GroupTotal
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public Mode[] Mode { get; set; }
    }

    public class Mode
    {
        public int Id { get; set; }
    }
}

Please modify above repro code to demonstrate the issue you are seeing.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


User experience testing of CLI

DoD:
Go through each command in the CLI and try out each option. create issues for all issues encountered.

HI @kozbo. What is this linked to?

@justincc I think he meant to assign this to me and assigned it to you by mistake

Thanks @jnkrupp

Tested Working:
post-search
get-file
head-file
get-bundle
get-bundles-checkout
post-bundles-checkout
Bugs Found:
Login
Logout
download --bundle-uuid --replica aws : Error AUTH required
Awaiting Triage:
put-subscription
get-subscription
get-subscriptions
delete-subscription
delete-bundle
put-bundle
upload
put-file
Notes:
Need to fix login issues in order to be able to with required auth.
Need callback URL to test subscriptions

Created ticket #219 to start talking about help menu improvements.


5 min sensor update interval

Hi. On Decenber, I have noticed, that all sensors update interval is about 5 min. Before all sensors updated correctly (from June to November). I wonder if it is somethong wrong with integration or owlet istelf.

Couple of things that you could check if possible. There is the option to adjust the scan interval within the integration. The default is 5 seconds has this been changed on your system by accident perhaps? You can check as per the below screenshots.

If that’s still set at 5 seconds could you try to remove and reinstall the integration and see if the issue persists?
If the issue persists are there any errors in the home assistant logs relating to the owlet integration?
Failing that if you check the owlet app itself are the stats within the owlet app updating every 5 seconds as you’d expect?

I don't use app anymore, because HA integration is much better. I checked the app and is says "base disconnected". So my problem is owlet-end. Reboot doesnt work. I will try to reinstall base and sock and report if it works...

I don't use app anymore, because HA integration is much better. I checked the app and is says "base disconnected". So my problem is owlet-end. Reboot doesnt work. I will try to reinstall base and sock and report if it works...

Closing


reset password form and emails

[ ] Update the routes from /users/password/new to something like /passwords/reset.
[ ] Update the email content to use the existing copy and ensure the links use the correctly URL's here.
[ ] Skin the form better perhaps like the original

I'm pretty sure we support reseting through a JSON call. Shouldn't this just live in the front end?

Yeah probably. Depending on the Nginx config HTML requests to the api server can get to those forms. If users can we should probably skin them properly.

This is done now


[Bug]: Missing coverage from custom test environment

Version
29.0.3
Steps to reproduce

Clone my repo at https://github.com/marvasDE/jest-sample
Install packages via npm i
Run npm run test:coverage

Expected behavior
100 % Stmts covered in index.js
Actual behavior
83.33 % Stmts covered in index.js
Additional context
We have a global service class, but every line which should be covered with the global service is not covered. The test is running, because there is a console.log.
Uncomment Line 12 of sample.test.ts will fix the behavior, but we would like to use a global service, because in a real project we have multiple tests, which are using a global service to reduce memory usage.
Environment
System:
    OS: Linux 5.19 Arch Linux
    CPU: (8) x64 AMD Ryzen 7 2700U with Radeon Vega Mobile Gfx
  Binaries:
    Node: 18.9.0 - /usr/bin/node
    Yarn: 1.22.19 - /usr/bin/yarn
    npm: 8.19.1 - /usr/bin/npm
  npmPackages:
    jest: ^29.0.3 => 29.0.3

If your service is instantiated outside of a test (e.g. globalSetup or via a test environment) then that code runs outside of the context jest can pick up the coverage I think...


Use correct autoloader and config file when shopware is a composer dependency

1. Why is this change necessary?
When you write a plugin test in a composer-project environment like it is written in the dev docs you can refer to the correct testing bootstrap file but it does not load the correct configuration file.
2. What does this change do, exactly?
If the special case of a composer project environment is detected it uses a different autoload.php and config.php file.
3. Describe each step to reproduce the issue or behaviour.

Have a plugin in a composer project in custom/project/FooBar
Bootstrap tests<?php
// resides in custom/project/FooBar/Tests/Functional/Bootstrap.php
ini_set('error_reporting', E_ALL);
ini_set('display_errors', '1');
ini_set('display_startup_errors', '1');

require __DIR__ . '/../../../../../tests/Functional/bootstrap.php';

phpunit plugin folder
Cannot test as a wrong autoload.php is loadedWarning: require(/var/www/html/foobar/vendor/shopware/shopware/tests/Functional/autoload.php): failed to open stream: No such file or directory in /var/www/html/foobar/vendor/shopware/shopware/tests/Functional/bootstrap.php on line 30
Fatal error: require(): Failed opening required '/var/www/html/foobar/vendor/shopware/shopware/tests/Functional/autoload.php' (include_path='/var/www/html/foobar/engine/Library:/var/www/html/foobar/vendor/shopware/shopware/engine/Library:.:/usr/share/php') in /var/www/html/foobar/vendor/shopware/shopware/tests/Functional/bootstrap.php on line 30

Amend bootstrap require path
phpunit plugin folder
Cannot test as a wrong config.php is loadedFatal error:  Uncaught RuntimeException: Could not connect to database. Message from SQL Server: SQLSTATE[HY000] [2002] No such file or directory in /var/www/html/foobar/vendor/shopware/shopware/engine/Shopware/Components/DependencyInjection/Bridge/Db.php:78
Stack trace:
#0 /var/www/html/dynfoobaratics/vendor/shopware/shopware/engine/Shopware/Kernel.php(300): Shopware\Components\DependencyInjection\Bridge\Db::createPDO(Array)
#1 /var/www/html/foobar/vendor/shopware/shopware/tests/Functional/bootstrap.php(41): Shopware\Kernel->boot()
#2 /var/www/html/foobar/vendor/shopware/shopware/tests/Functional/bootstrap.php(67): TestKernel::start()
#3 /var/www/html/foobar/custom/project/FooBar/Tests/Functional/Bootstrap.php(7): require('/var/www/html/f...')
#4 /var/www/html/foobar/vendor/phpunit/phpunit/src/Util/Fileloader.php(64): include_once('/var/www/html/f...')
#5 /var/www/html/foobar/vendor/phpunit/phpunit/src/Util/Fileloader.php(48): PHPUnit\Util\Fileloader::load('/var/www/html/f...')
#6 /var/www/htm in /var/www/html/foobar/vendor/shopware/shopware/engine/Shopware/Components/DependencyInjection/Bridge/Db.php on line 78

Play some tetris on arcade machine to regain saneness

5. Which documentation changes (if any) need to be made because of this PR?
The dev docs should just have some paths to :clipboard: :ramen: from, depending on their test bootstrap location.
6. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

:ballot_box_with_check: I have written tests and verified that they fail without my change
:grin: it is THE test

Hey @JoshuaBehrens,
sounds like this should be fixed in the composer-project instead.
What do you think?
Any reasons against this?

Will there be a unified way to normalize a path to the tests bootstrap/kernel? I can't have plugins installed in a shopware standalone setup refer to a different bootstrap file than it would refer to in a composer project setup.
During plugin development I should just have to choose between 5.2 structure and pre-5.2 structure. But in a composer project this simply wont work. Although the tests folder is symlinked, __DIR__ uses the absolute resolved path to the file.
It is ok for me if you decline that the multiple .. might go to a completely different application and might break the tests as well. I am fine with a custom test kernel for composer projects as well. But this ends up in a complete mess in the test bootstrap per plugin where to get the correct test kernel from.

Regarding the configuration path. What about doing the same as done for the autoloader? I think of a loop that does the same like the autoloader loop but does not go any directories further then the autoloader parent directory.

Sounds like it might work, the only problem I see is that there is no single file path that might contain the config.php, but multiple. That might result in a lot of file checks

Can we somehow use the autoloader to find the configuration file? Like a configuration finder that just knows where it is?

Hey @JoshuaBehrens,
I don't see really the need to change this in shopware repository. What about placing in composer-project itself bootstrap files for Plugins, in the same place?
I tested it quickly and it works. I have created in composer-project template also the tests' folder with the same folder structure, but with the configuration for composer. For me it looks like your pain has been resolved with doing that.
Composer-Project: https://github.com/shopware/composer-project/commit/4bcc0cc6e93af356b76828f9ec842cc86fef95d6
FroshProfiler: https://github.com/FriendsOfShopware/FroshProfiler/commit/e7be5d4362a4f53f58fcd9fe3f0766e5fa663579
What do you think about that?

Hey @JoshuaBehrens,
you're still on it?

I tested it locally on a composer installation and a normal installation and it worked out as @shyim stated. But doesn't this mean, that every plugin has to have this testing setup like shyim? If I am right the current cli tools do not generate something similar to this.

Hey @JoshuaBehrens,
thank you very much for your contribution and your active communication about this issue.
Yet, we've had a discussion about this and we think this is a plugin issue, rather than a core issue.
Fixing this in the core brings some more difficulties, e.g. having people use the composer project and having them place the plugins into a non-standard directory - they'd have to fix their plugins anyway, with or without your change.
It would be highly appreciated to see a new PR from you, including a more compatible solution.

So this will result in a PR for the cli tools instead to create suitable plugin skeletons in the first place as well as some changes in the documentation. Thanks for the thoughts about it.


3.1.8 beta re2 bugs 2

Version
3.1.8  (0422f52)
Seed
R720-B0RE-ZXQ970Z5J4XHM07X84YU69AU0100H1C1100000ZC0YPJ82ZZZ
Scenario
RE 2 (Claire B)
Player Character
eveline
What happened?

room 111 unstop spin crank cut scene soft lock
log_pl1.txt
111 Claire B_00_2023-12-23 14^43.35.zip

room 213 power supply won't work . cannot access red card  (debug jump to room 211 , check weapon storage door light)
please refer :
https://github.com/IntelOrca/biorand/issues/722#issuecomment-1867804451

Probably same issue and might now be fixed


[spi_device] TPM interrupt for Write FIFO

Related Issue: https://github.com/lowRISC/opentitan/pull/15773
Current TPM IP does not provide an interrupt about the write transaction completion. Absent of the interrupt results the SW to wait until the transaction completed by reading the WrFIFO depth or polling the TPM CS# de-assertion.
A TPM transaction may took up to 55 us (64B + 1 Wait + address + cmd @ 10MHz). By introducing the completion interrupt for the Write FIFO, SW can switch the context.
Idea suggested by @tjaychen
CC: @weicaiyang @tjaychen @a-will

estimate 16
remaining 2023-03-21 16

Triaged for spi_device. Assigning to M2.5 with https://github.com/lowRISC/opentitan/labels/Priority%3AP2, though I'm not sure (1) whether this would free clock cycles on Ibex or on the TPM and (2) how this relates to requirements and capabilities of the product. Maybe @a-will knows? If not, I'll bring this up for discussion with the product team.

Triaged for spi_device. Assigning to M2.5 with  Priority:P2 Priority: medium , though I'm not sure (1) whether this would free clock cycles on Ibex or on the TPM and (2) how this relates to requirements and capabilities of the product. Maybe @a-will knows? If not, I'll bring this up for discussion with the product team.

This frees up cycles on Ibex. The current design only has an interrupt to inform software when the command and address phase completes (the "header"), but the transaction continues afterwards, potentially for a substantial amount of time (for the 64-byte payload case, especially).

For a SPI TPM that only does 4-byte payloads, the thinking (in the IP doc) was that the transmission time is too short for an interrupt to be useful. However, as payloads grow larger, the time duration grows larger, potentially making busy-waiting expensive. For the sake of prioritization, the product team may be helpful for determining which payload sizes are important.

As far as I know the only TPM register which is wider than 4 bytes is the FIFO register.  That is used all the time, though.  If we do not get interrupt at the end of a transaction involving the GSC receiving data through the FIFO register, that would be a problem.
We could busy-wait, but doing so would adversely affect our ability to verify performance of other aspects of our code, such as whether forwarding of characters from UART to USB encounters data loss during other system activity

Based on prioritisation discussion - this RTL change will likely need to go into the next cycle and use the SW workaround.
De-prioritising accordingly. If this does need to go in then it should be re-raised as a P0 @jesultra / @moidx / @msfschaffner / @GregAC

We missed the window for RTL changes, we'll move this to the backlog so that it can be considered for the next RTL freeze.


remove lastMessageTranslationKey from report collection

Explanation of Change

This PR removes lastMessageTranslationKey from report_, since looks like it is not used
Fixed Issues

$ https://github.com/Expensify/App/issues/51867
PROPOSAL:
Tests

Open any chat
Open Chat menu -> Add attachment
Select file and send attachment
When uploaded, verify the last message of the chat in LHN has changed from file name to [Attachment]

[x] Verify that no errors appear in the JS console

Offline tests

n/a
QA Steps

// TODO: These must be filled out, or the issue title must include "[No QA]."
Same as tests

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I added unit tests for any new feature or bug fix in this PR to help automatically prevent regressions in this user flow.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

https://github.com/user-attachments/assets/891f2201-3e10-4c74-b91a-4352dffa9c9d

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/c66a4afc-3fa4-4a15-901d-ecdbc47e0245

MacOS: Desktop

https://github.com/user-attachments/assets/9fedeb8d-d1c6-4f4c-bc0e-5410914a8284

Don't forget to remove it from the whitelistedReportKeys as well. Should we update that test somehow so that it's not so easy to forget?

blocked by https://github.com/Expensify/App/pull/54288

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Can siam-mot model be trained on a server with CUDA10.2

Hi~
I'm an undergraduate student. I rent a server with CUDA10.2 and can't install the new 11.0 version.(It didn't provide the permission of sudo commands)
Is it possible to train siam-mot with a CUDA10.2 server?
Thank U!

I cannot compile maskrcnn with cuda10.2, thus cannot run the code.
Could you please me how could you make it.
Thanks


uart-npcx: forward poll requests to fifo when running interrupt mode

Looking at the data sheet for the npcx section 4.15.5 CR_UART Core
Registers, the UICTRL register that is used in the npcx poll
functions is invalid when running in FIFO mode. Instead, calls to
uart_npcx_poll_in and uart_npcx_poll_out should be redirected to
their respective FIFO read/fill functions (when using interrupts).
Signed-off-by: Yuval Peress<EMAIL_ADDRESS>

Hey do we have any good reason to keep byte mode support at all? Thinking that we may just have the controller set in FIFO mode regardless of CONFIG_UART_INTERRUPT_DRIVEN, so that we don't have to have two different poll implementation to maintain and test for the poll API, as well as a change of behavior if the interrupt mode gets compiled out.

Hey do we have any good reason to keep byte mode support at all? Thinking that we may just have the controller set in FIFO mode regardless of CONFIG_UART_INTERRUPT_DRIVEN, so that we don't have to have two different poll implementation to maintain and test for the poll API, as well as a change of behavior if the interrupt mode gets compiled out.

NPCX5 and NPCX7 series support byte mode. I don't think it's our place to decide what consumers of Zephyr will need.


Feature/transition animatable

TransitionAnimatable Updated to use enums.
for instance :
var transitionAnimationType: String? { get set }
becomes
var transitionAnimationType: TransitionAnimationType? { get set }
I have one interrogation :
Should it be an optional or not (adding a none clause) ?
I don't know enough the TransitionAnimatable system to decide about this.
PS: Seems that I didn't made the last Xcode beta update.
There is maybe some errors, I will update it tomorrow

cool 👍
For the interrogation, make it optional will be the easiest way. If you want to add a none case to be as the others enum, you have to change the guard condition in the prepareForSegue of AnimatableViewController, and the configureNavigationControllerDelegate of AnimatableNavigationController, and I think that's all 👯

COOL, 👍 I have questions for you guys @lastMove @tbaranes , shall we make all enums conform to IBEnum has an option .none except for system built-in one like 'UIBlurEffect' ? I remember having .none can help us apply default value in the protocol.

That would be nice, moreover it's better to uniformise them, even for UIBlurEffectStyle, we can add a none case but just ignore it when we received that value.

I don't think we can add .none to UIBlurEffectStyle , it will be great if we can


A3.05 Mine Big Data

For github original see https://github.com/lsst-uk/lasair/projects/18

At the 28/APR/21 Lasair internal review it was agreed to close this activity.


Dying inside of a titan sometimes allows you to drop a titan just after

Some videos to illustrate
https://cdn.discordapp.com/attachments/977997240599728268/978002227404881990/First_game.mp4
https://cdn.discordapp.com/attachments/977997240599728268/978002259529072790/Second_game.mp4
It seems to work only when using "respawn as a titan" (or so it seems)

closing as potential duplicate of #339


Reset command needs a confirmation

Until a few minutes ago, I had a sizable cluster deployment running a few dozen things here at home.  I was investigating why kubelet wasn't garbage collecting snapshots and following this issue which contained advice for reconfiguring kubelet and then stopping and starting microk8s.
After configuring kubelet, I ran:
$ microk8s.stop && microk8s.start
This MicroK8s deployment is acting as a node in a cluster. Please use the microk8s stop on the master.
This MicroK8s deployment is acting as a node in a cluster. Please use the microk8s reset on the master.

Oh, I thought, I'll go over and reset the node from the master....
$ microk8s.reset kube-node-2
Disabling all addons.
Disabling addon : ambassador
Disabling addon : cilium
Disabling addon : dashboard
Disabling addon : dns
... snipped

And just like that, a cluster I had been managing for over a year went up in smoke.  My kid's minecraft server with months of progress.  My LDAP server.  My mySQL database.  All the persistent storage... gone.
It's my own fault.  reset is clearly not restart.  I made assumptions about what reset did without consulting the documentation first.  And the documentation is very clear about what reset does.
But:

the output of microk8s.stop && microk8s.start told me to run reset and it's 4am and I'm tired and just listened.
the fact that I passed a node name to the microk8s.reset command didn't cause it to fail
the command to completely wipe everything out never bothered to ask if I was certain I wanted to torch everything

I very strongly suggest that:

confirmation be added to the reset command.
the microk8s command have vastly improved help (microk8s help just lists the commands and none of the subcommands have help)

I understand your frustration and appreciate your calmness.
In https://github.com/ubuntu/microk8s/pull/1750 we try a few fixes based on your feedback. Your exact suggestions will need a bit more time to implement.
Apologies for the inconvenience we caused.

Is there any progress on adding an actual confirmation step or switch to the reset command?
It seems to be an incredibly bad piece of design and implementation - student project level - to have a command that will completely destroy a deployment including any persistent parts labelled as something innocuous as 'reset' and to then not back it up with at least a minimum of at least one 'are you sure you really want to do this?' step.
You can't leave a nuke lying around and not at least put a cover (and a proper label) on the big red button.


[Improvement][Task Instance] Throw an error message to tell the user that the file cannot be found instead of NPE during task execution

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
https://github.com/apache/dolphinscheduler/pull/13593#issuecomment-1438045083
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@SbloodyS
I have found a solution. The solution is that when the task runs, it will re-detect whether the resource still exists. If it does not exist, it will prompt. Can I submit code

@SbloodyS I have found a solution. The solution is that when the task runs, it will re-detect whether the resource still exists. If it does not exist, it will prompt. Can I submit code

Sure. Assigned to you. Looking forward to your contribution.

We all have this problem in 3.1. x. Do we need to submit it to 3.1.5-prepare as a bug

@SbloodyS We all have this problem in 3.1. x. Do we need to submit it to 3.1.5-prepare as a bug
#13656

Hi, @LiJie20190102 , if it's irrelevant to new resource center, you can directly submit to dev, and community will decide whether to merge it into release 3.1.5 or 3.2.0.

@SbloodyS We all have this problem in 3.1. x. Do we need to submit it to 3.1.5-prepare as a bug
#13656

Hi, @LiJie20190102 , if it's irrelevant to new resource center, you can directly submit to dev, and community will decide whether to merge it into release 3.1.5 or 3.2.0.

@SbloodyS @Radeity  After I read the relevant code carefully, I found that 3.1. x is not the same as the 3.2 version code. 3.2 will not have a NPE  error, and 3.1. x will have a NPE problem. The dev branch does not need to submit the corresponding code to deal with this problem. Can I only submit the code to 3.1.5 to fix this problem?

That's okay, you can submit to 3.1.5-prepare first.


1.4. beta3 - Wallet crashes when trying to apply encryption. (Win7/Win10)

When asked to be sure to set a passphrase, the wallet is crashing right away.
Edit:
When run as administrator, wallet will close after "OK" as proposed.
But no encryption is taking place.

Fixed with Spectre1.4.0beta4.


Improve Study Listing page

Reported by @AmandaBirmingham.

When looking at study search results in Qiita, I would love it if they were a bit easier to read.  The first thing I want to look at is the study title, which is currently squeezed into a very small box and highly text-wrapped, whereas “preparation data types” (which is usually a very short value like “16S”) gets biggest cell in the table!  I would be able to get the info I want from the table (and see more results on my screen at once) if the cell sizes were scaled differently, giving “title” a lot more room and everything else (I’m looking at you, “preparation data types” and “publications” :) less.

Then after further discussion we agreed that a solution would be to set the width percentage of the columns like this: 5-45-5-5-10-10-10-10.

That does not allow an admin to edit the users that the study is shared
with - currently this can only be done on this page.
On Sun, Mar 19, 2023 at 6:51 AM Antonio Gonzalez @.***>
wrote:

Reported by @AmandaBirmingham https://github.com/AmandaBirmingham.
When looking at study search results in Qiita, I would love it if they
were a bit easier to read. The first thing I want to look at is the study
title, which is currently squeezed into a very small box and highly
text-wrapped, whereas “preparation data types” (which is usually a very
short value like “16S”) gets biggest cell in the table! I would be able to
get the info I want from the table (and see more results on my screen at
once) if the cell sizes were scaled differently, giving “title” a lot more
room and everything else (I’m looking at you, “preparation data types” and
“publications” :) less.
[image: image001]
https://user-images.githubusercontent.com/2014559/226179976-493ef74a-bb01-4d1e-906c-5bd708e9d405.png
Then after further discussion we agreed that a solution would be to set
the width percentage of the columns like this: 5-45-5-5-10-10-10-10.
—
Reply to this email directly, view it on GitHub
https://github.com/qiita-spots/qiita/issues/3269, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPL3AFQP6JEE3PJTS6XCV3W44FOPANCNFSM6AAAAAAWAEBPVM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Gail Ackermann
Knight Lab
UCSD
@.*** @.***>

The idea is not to change the content of the columns, just the display width. In other words, the button you mention should be there. BTW maybe the confusion is that the screenshot is from "Public Studies" and not the "Your Studies" listing; however, the same idea of widths should be applied there.

thanks for the explanation.  I agree with the widths
On Sun, Mar 19, 2023 at 5:16 PM Antonio Gonzalez @.***>
wrote:

The idea is not to change the content of the columns, just the display
width. In other words, the button you mention should be there. BTW maybe
the confusion is that the screenshot is from "Public Studies" and not the
"Your Studies" listing; however, the same idea of widths should be applied
there.
—
Reply to this email directly, view it on GitHub
https://github.com/qiita-spots/qiita/issues/3269#issuecomment-1475450712,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPL3AG377D7IOFNKAXV73DW46OWDANCNFSM6AAAAAAWAEBPVM
.
You are receiving this because you commented.Message ID:
@.***>

--
Gail Ackermann
Knight Lab
UCSD
@.*** @.***>


fix: Using the wrong public key when associating tokens

fix Using the wrong public key when associating tokens.

This commit is to fix this error.


oracle load meta data get index info user schema error

Bug Report
Which version of ShardingSphere did you use?
4.0.0-RC2
Which project did you use? Sharding-JDBC or Sharding-Proxy?
Sharding-JDBC
Expected behavior
Actual behavior
Reason analyze (If you can)
 private Collection<String> getLogicIndexes(final Connection connection, final String catalog, final String actualTableName) throws SQLException {
        Collection<String> result = new HashSet<>();
        try (ResultSet resultSet = connection.getMetaData().getIndexInfo(catalog, catalog, actualTableName, false, false)) {
            while (resultSet.next()) {
                Optional<String> logicIndex = getLogicIndex(resultSet.getString("INDEX_NAME"), actualTableName);
                if (logicIndex.isPresent()) {
                    result.add(logicIndex.get());
                }
            }
        }
        return result;
    }

   ResultSet getIndexInfo(String catalog, String schema, String table, boolean unique, boolean approximate)
        throws SQLException;

oracle catalog is sid ，the second param is scheam，is not sid
Steps to reproduce the behavior, such as: SQL to execute, sharding rule configuration, when exception occur etc.
Example codes for reproduce this issue (such as a github link).

Hi, it is a easy issue, if anyone is interested in it, please raise a PR. Thanks
How to solve it?
If null == resultSet.getString(INDEX_NAME), then return Optional.absent() in getLogicIndex(final String actualIndexName, final String actualTableName).

I read the pssql、mysql and oracle jdbc source code，about catalog and schema method parameter，the result is :
1、All database's catalog  equal database name
2、oracle's schema equal username ,but other database's schema equal null.

I read the pgsql、mysql and oracle jdbc source code，about catalog and schema method parameter，the result is :
1、All database's catalog equal database name
2、oracle's schema equal username ,but other database's schema equal null.

Hi penguin, Appreciated your investigation, if available, we welcome your contribution. :)

I read the pgsql、mysql and oracle jdbc source code，about catalog and schema method parameter，the result is :
1、All database's catalog equal database name
2、oracle's schema equal username ,but other database's schema equal null.

It is great. Your detail investigation inspired us how to solve this issue!


Focus event and Mask clash???

Hi, I have an issue where I add a focus trigger on a field to call a javascript function. This works well for all fields except those that have MaskedEditor(). If I remove the MaskedEditor from the myPageRow.cs it works. I need to have both the Mask and my on focus trigger working.
What happens is that the Dialog does not even open. Here's the console output:

#495 Can you help me look good

Transform templates first

Reopen if not resolved.


new: support for tls session resumption

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "backend-scale",
    "component": "manipulate",
    "pr-id": "109",
    "commit-sha": "b7b4e02faa05196732b76d0080f0ad64b68b90c5"
  },
  {
    "project": "backend-scale",
    "component": "enforcerd",
    "pr-id": "1482",
    "commit-sha": "9dcd4539a4088c6386ea8608955ba969ad456a8c"
  }
]


Update Customizing Your Plugin's Listing to specify PyPI metadata as the only source for Documentation, User Support, Report Issues, and Twitter links

Updates were made as seen here.


doc(overview): combinatorics overview

Expand the graph theory section and write new sections for enumerative, set family and additive combinatorics as well Ramsey theory

As discussed on Zulip, we need to seriously discuss whether we want deeper nesting in this list like what you introduced. I am rather skeptical that combinatorics needs this when no other branch of maths needs it.

Given that this appears to be a controversial topic, I am going to close #18910 that only contains the >= change, so that the change will not be merged without a conclusion having been reached on this PR.

Since there were no more suggestions at https://leanprover.zulipchat.com/#narrow/channel/113488-general/topic/mathlib.20overview, I am marking this PR as awaiting review again


about channels

Hi,
Can you please tell me what's the reason for your choice of channels changes like[48,96,160,304]at each block respectively?

Hi @wwrong01,
Thank You for your interest in our work. We select the channels to have similar parameters as of MobileViT model.

Ok，I respect your prompt response！And ,from the papers I've read, a lot of the channels  choices are based on experience, like your article is inspired by MobileViT, like some articles inspired by swin transformer …maybe I can take that as a basis. And then there's the optimal choice based on their own experimental research.I just don't know how to make  choices.
---Original---
From: "Muhammad @.>
Date: Mon, Apr 10, 2023 20:16 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [mmaaz60/EdgeNeXt] about channels (Issue #21)
Hi @wwrong01,
Thank You for your interest in our work. We select the channels to have similar parameters as of MobileViT model.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Corrected the property name ACR SKU asserts on

PR Summary
Asserting the value of ACR Sku.tier is incorrect, it should be Sku.name. Made the replacement.
ref: https://docs.microsoft.com/en-us/azure/templates/microsoft.containerregistry/registries?tabs=bicep#sku
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] Change is not breaking
[x] This PR is ready to merge and is not Work in Progress
Rule changes

[ ] Unit tests created/ updated
[ ] Rule documentation created/ updated
[x] Link to a filed issue
[x] Change log has been updated with change under unreleased section

Happy for you to merge it @BernieWhite - I'll write a test for my next PR 😉


feat: add TeeResponseWriter

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


Implement plotting widget

Noticed this on the textual discord:

Really need to look into this (https://github.com/piccolomo/plotext/blob/master/readme/environments.md#rich).

Working on the implementation here: https://github.com/vkottler/conntextual/pull/24.


Inline Redux Action type to avoid dependency on Redux

Q
A

Fixed Issues?
N/A

Patch: Bug Fix?
Yes

Major: Breaking Change?
No

Minor: New Feature?
No

Tests Added + Pass?
Yes

Any Dependency Changes?
Yes

Adds redux to the dependencies of @redux-saga/types to fix this type error:
ERROR in ../../.yarn/cache/<EMAIL_ADDRESS>1:24-31
TS2307: Cannot find module 'redux' or its corresponding type declarations.
  > 1 | import { Action } from 'redux'
      |                        ^^^^^^^
    2 |
    3 | export type Saga<Args extends any[] = any[]> = (...args: Args) => Iterator<any>
    4 |

This looks good to me as well!  Thanks for the contribution


distro: SELinux should be the last stage

SELinux relabels the entire filesystem tree and should always be the last stage in the os pipelines.
Reported in, and closes: #3115

warning This PR introduces changes in at least one manifest (when comparing PR HEAD 946a7ac with the main merge-base 1096003). Please review the changes. The changes can be found in the artifacts of the Manifest-diff job [0] as manifests.diff.
[0] https://gitlab.com/redhat/services/products/image-builder/ci/osbuild-composer/-/jobs/3269161744/artifacts/browse

This is an expected change as the order of the stages changed in all these manifests.

There are failures in CI
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="    [/usr/lib/tmpfiles.d/journal-nocow.conf:26] Failed to resolve specifier: uninitialized /etc detected, skipping" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="    All rules containing unresolvable specifiers will be skipped." jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="    Failed to create file /sys/fs/selinux/checkreqprot: Read-only file system" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="    /run/osbuild/tree/etc/selinux/targeted/contexts/files/file_contexts: No such file or directory" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="    Traceback (most recent call last):" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="      File \"/run/osbuild/bin/org.osbuild.selinux\", line 75, in <module>" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="        r = main(args[\"tree\"], args[\"options\"])" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="      File \"/run/osbuild/bin/org.osbuild.selinux\", line 62, in main" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="        subprocess.run([\"setfiles\", \"-F\", \"-r\", f\"{tree}\", f\"{file_contexts}\", f\"{tree}\"], check=True)" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="      File \"/usr/lib64/python3.6/subprocess.py\", line 438, in run" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="        output=stdout, stderr=stderr)" jobId=528a52fe-f51a-48dc-808a-769fd349815d
Nov 07 05:23:32 rhel-8-7-nightly osbuild-worker[24603]: time="2022-11-07T05:23:32-05:00" level=info msg="    subprocess.CalledProcessError: Command '['setfiles', '-F', '-r', '/run/osbuild/tree', '/run/osbuild/tree/etc/selinux/targeted/contexts/files/file_contexts', '/run/osbuild/tree']' returned non-zero exit status 255." jobId=528a52fe-f51a-48dc-808a-769fd349815d

It seems a necessary file is missing after preptree which breaks all the osbuild things. @gicmo can it be moved before preptree but after the RHSMFacts stage?

It seems a necessary file is missing after preptree which breaks all the osbuild things. @gicmo can it be moved before preptree but after the RHSMFacts stage?

Doh, I didnt look properly, yes preptree is special too:

everything else
SELinux
preptree

Is the correct order. I think technically we don't need SELinux when we do preptree, since (rpm-)ostree does apply the policy itself.

These seem like legit failures in the ostree+greenboot tests on Fedora 37, but I can't figure out if they're transient or how my change is related to this (it's failing to SSH into a machine).

These seem like legit failures in the ostree+greenboot tests on Fedora 37, but I can't figure out if they're transient or how my change is related to this (it's failing to SSH into a machine).

A fixed was recently done for F37 and ostree, rebasing on main should give you that.

Hah, that's what I just did. :)


Report protocol buffer generation issue to Google

A protocol message with an underscore will not be translated correctly into Ruby. They're translated into constants, and constants with a leading underscore are private. They try to account for this by prepending PB_, but they still try to reference it without the prefix. 😮💨 🌴
For example, message _GetResponse will create PB__GetResponse but the rest of the generated code will still try to reference _GetResponse.

[x] Report the bug. https://github.com/grpc/grpc/issues/31551
[ ] Regenerate the proto code when fixed.

https://github.com/grpc/grpc/issues/31551
Looking at the bug report, it now looks like this problem is solvable.

Sorry, it was merged, but still in pre-release stage.

This was released in 1.55.0. https://github.com/grpc/grpc/commit/401c382e8a0cc5c6083d3d5edfc6cd12abeb2658


Improve the semantics of form's onInvalid action

Picking up what was discussed here:

validations should be shown after an eventually returned Promise from onInvalid has settled
provide some means to skip showing the validations based on the action's return value

Potentially a breaking change!

@jelhan would you be interested in working on this, as this was based on your suggestion?

@simonihmig, thanks a lot for picking this one up. I'm sorry for not replying earlier. Haven't had much time the last weeks.

provide some means to skip showing the validations based on the action's return value

I think this needs some more thoughts. E.g. it's not clear yet, when the validations should be shown again. Also we haven't seen an actual use case yet. If an application is able to resolve the validation errors automatically, it should also update the validation state. In that case there isn't any need to hide the validations. If the validation shouldn't be shown before a transaction caused by an invalid form submission, a long-pending promise could be returned. The same applies if some user interaction should be done on an invalid submission (e.g. showing a modal). Last but not least applications should be able to implement this on their own without overriding internal APIs as soon as we provide a way to control the visibility of validations in general (#364). So we should focus on that one first in my opinion.

I agree, see also my comment here https://github.com/kaliber5/ember-bootstrap/pull/820#issue-283854425.
I will add a note to #364 and keep that issue to track controlling the visibility of validations.

validations should be shown after an eventually returned Promise from onInvalid has settled

Implemented by #820, closing this.


Question: Lock down inbound traffic only to specific IP addresses and internal VNET

Hello,
Let's say I have a few APIs (Container Apps) that I want to expose via API Management (and therefore lock down the traffic to the outbound IP address of the API Management using https://learn.microsoft.com/en-gb/azure/container-apps/ingress?tabs=bash#ip-access-restrictions) and at the same time I want direct communication between the APIs that are deployed in the same Container Environment (using the Application Url or maybe Dapper and bypassing the API Management) - how would I go about and achieve that?
Would I have to include in the allow list both the API Management public outbound IP address and the Container Environment outbound IP address(es)? Would then the traffic between the APIs (Container Apps) be redirect via "the internet"?
Also, I believe the same question applies if I have a mix of public facing APIs and internal APIs. How do I lock the internal APIs down so the traffic is allowed only from the Container Environment network.
This is somewhat related to https://github.com/microsoft/azure-container-apps/issues/291

I believe if I understand the question and my understanding of the feature set, you can use the allow list for IPs to explicitly allow access from apim to certain apis. You do not need to add the outbound ip of the environment or the ips of any specific apps) i.e. if App A needs to talk to App B within the environment, that will happen via the FQDN that is generated when you enable ingress, it will not traverse over the public internet. This is also true with Dapr service invoke, but unlike communication via the FQDN, Dapr does not require any ingress settings to allow apps to talk to each other in the cluster.
If you want to mix both internal and external APIs i.e APIs available over public internet and those which are not, I would recommend deploying an internal-only, vnet integrated environment and using one of the following: app gateway, api management or azure front door to act as the public entrypoint. Customers would hit this front end "gateway" and the calls can be directed to the service in the environment based on your internal routing- the services that will be routable on the VNET must have "external" ingress setting. For those you don't want to expose, you can set "internal" as the ingress at the app level, meaning only other apps in the environment can access. The alternative if you don't want to add one of those (seems like you are using apim) would be to create the environment with an external endpoint and allow people to communicate directly with services that are "external" via their public FQDN.
I am making some assumptions here around your vnet config, you may not be using a version of apim that is vnet integrated, etc. so forgive me if this isn't totally applicable.
If you need additional insights or want to respond for clarity please do! We are working on docs for networking architectures so hopefully this will become more clear over time.

I'm looking for a mix of both internal and external APIs.
I agree that a custom VNET would make most sense, because you'd also want to control the access from the APIs to other resources, such as databases, storage accounts, etc.
I would want to use as a gateway the APIM (without a VNET integration however, since it involves significant costs), and rely only on its outbound IP address to allow traffic in on the external APIs. For the internal APIs I don't want to allow any external traffic at all.


Stored procedure mapping

Any plans to support stored procedure mapping introduced in EF 7
https://learn.microsoft.com/en-us/ef/core/what-is-new/ef-core-7.0/whatsnew#stored-procedure-mapping

Yes, stored procedure support is the next feature I plan to dig into.  Specifically as stand-alone functions similar to how they can be imported into an EDMX and executed from code.
The link you supplied is about linking stored procedure CUD operations to entities.
Example:  "People_Insert",         "People_Update",         "People_Delete"
That particular approach would require you to preconfigure the "People" entity in the rule file by identifying each CUD procedure, and its parameters.  and all those procedures must exist in the DB (scaffolding will not create them)..  Is this what you're looking for, or just the general ability to execute sprocs?

I'm interested in generating existing SPROCs and functions as defined in the EDMX.

Support just added for stored procedures and functions. (NEW AS OF JUNE 20, 2023 v1.2.28)
Regenerate the rules json file from EDMX in order to get the same structure that you had there.
If instead you just want to capture the sprocs in the DB without regenerating the json, you'll need to set "IncludeUnknownFunctions" to true on the schema (rule) in question.  You can use the EFR VSIX extension to check that box.
By default, unknown function capture is off.
This is a brand new, beta feature.  Let me know if there are any issues.

SQL's table-valued functions are also supported now, btw.


Setup work to support style utility components

There are no visual changes, these are just changes to lower level code (like design tokens) to support the other style utility PRs.
Just check that tests still pass and the code looks good. See other style utility PRs for additional context

npm run dist is failing locally, as well as in CI.


Missing Token for Audio Feature Methods

Hi there, thanks for this dope library!
Everything works fine except getting the audio features of a track (tempo, danceability etc..). The problem is, that the methods getTrackAudioFeatures and getTracksAudioFeatures do not send an access token, which is required, see spotify api docs for audio-features.
A simple fix could be:
Line 474, change
return this.api('/audio-features/' + track);

to
return this.api('/audio-features/' + track, 'GET', null, null, this._auth());

this will send the token in the header!
Similar need to append this._auth() to getTracksAudioFeatures.
I could send a PR if you want!
Have a nice day,
Felix

Thanks for spotting the bug. Will fix it within the next day

Hey @felixroos sorry for the delay.
Updated the package with the fix.

thanks a lot!