[CSL-23] Add more checks to MPC processing functions

@gromak

Pass SlotId so that message is completely ignored if it's not relevant for current slot. It's already stored in main Storage. Be careful to consider case when current slot is not the same as last seen by Storage (btw, I am not sure if it's possible at all).
For opening and share check that corresponding commitment exists.

@georgeee
Both items are already implemented in different ways


[Package Issue]: [ONLYOFFICE.DesktopEditors] Version <IP_ADDRESS>

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Other
Brief description of your issue
(1/3) Found ONLYOFFICE Desktop Editors [ONLYOFFICE.DesktopEditors] Version <IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://github.com/ONLYOFFICE/DesktopEditors/releases/download/v<IP_ADDRESS>/DesktopEditors_x64.exe
An unexpected error occurred while executing the command:
Download request status is not success.
0x80190194 : unknown error
Steps to reproduce
winget update onlyoffice
Actual behavior
(1/3) Found ONLYOFFICE Desktop Editors [ONLYOFFICE.DesktopEditors] Version <IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://github.com/ONLYOFFICE/DesktopEditors/releases/download/v<IP_ADDRESS>/DesktopEditors_x64.exe
An unexpected error occurred while executing the command:
Download request status is not success.
0x80190194 : unknown error
Expected behavior
update without error
Environment
winget --info
Windows Package Manager v1.4.10173
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.1105
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.19.10173.0

Screenshots and Logs
No response

https://github.com/microsoft/winget-pkgs/pull/95920

The problem is, Previously in Winget, OnlyOffice uses the .msi package but now it uses .exe. That's why.


Update utils.py

update compute_accuracy to deal with the cases where str_chosen and str_rej got the same scores, which is probably what the developers don't want

Can you elaborate a bit more why this is bad and what bad side effects this has?

When a model gives exactly same scores for the sentence to accept and the sentence to reject, the current evaluation metric will set this pair to be "correct". However, the above scenario is usually caused by the model having some problems in encoding sentences (e.g. over-quantized), or the two sentences exactly being the same, which we both don't expect. I think putting a warning here and defining this pair as "wrong" or "random" would be helpful for the developers to observe these problems.

I see, thanks! In that case I'd prefer to just add a warning but leave the computation logic the same. Does that make sense?

sure it makes sense. Thanks for your suggestions!


API to fetch Lexical data

Extend API to give additional info (transliteration, english, category etc) along with the Strongs number info.
[ ] Take a Strong number as an input and return the lexical info
[ ] Pre-populate the alignment data with the lexical info

The lexical info should be fetched from the VO-db

test comment: to test slack/github integration

test comment


Failed sending message to more than 300K users

Dears,
i am facing issue sending message to more than 300K users after upgrading to v5.2
we tried to send the message for first time and we get this error
Failed to prepare the message for sending:The orchestrator function 'SyncRecipientsOrchestrator' failed: "The activity function 'SyncAllUsersActivity' failed: "Timeout value of 01:00:00 was exceeded by function: SyncAllUsersActivity". See the function execution logs for additional details.". See the function execution logs for additional details.
then we increased the timeout for prep-function to be 8 hours, and we tried to send the message for second time but i was not delivered to all users with the below error message

anyone can advice for this issue

Hi @sayedpfe ,
Thanks for raising the query. Please send the export details of the message sent.

Hi @gsv022 ,
thanks so much for your reply, we couldn't export it, when we tried it gave us error

Hi @sayedpfe
Please give us sometime to check on the issue and will get back to you with proper response.

As confirmed over internal teams conversation for now no further steps need to be taken hence we are closing out the issue.


add support for api previews

The deployment statuses API has a preview for additional states. By adding a previews input to the action these can now be used as well
https://docs.github.com/en/rest/overview/api-previews#deployment-statuses
I've also updated the action dependencies.

Maybe the preview can even be passed by default, to not complicate things.


Access to Property Multiplicity

I have started to implement this project to generate c# classes and so far it has worked great. The main issue I have run into is accessing the multiplicity of a property. I don't see it in the Xmi classes and I don't really even see where it is located in the raw SysML XMI file (although I'm sure it's in there somewhere).
My temporary solution is to create a manual list to determine if a property is an array and also if it is required/optional.
My question is, is the Multiplicity something that is directly pulled from the SysML XMI file? Or is this something that is derived from other properties?
Any help would be appreciated.
Thanks,
-Patrick

@PatrickRitchie There should be representation for the XMI structures that indicate multiplicity now. However, I don't think the foreign relationships have been "stitched" into the transpiled model. Do you have a specific example/use case that I can focus on to resolve this issue?

I'm currently using a hard coded list to determine whether a property is an array or is required using the code below.
It would be nice if I could read this directly from the UmlProperty class as a hard coded list is not really scalable. Is this available in the latest version?
I'm also open to an alternative way to process this if there is something I'm doing incorrectly or something that could be done better.
Set IsArray and IsOptional properties:

https://github.com/TrakHound/MTConnect.NET/blob/master/libraries/MTConnect.NET-SysML/MTConnectPropertyModel.cs

public MTConnectPropertyModel(XmiDocument xmiDocument, string idPrefix, UmlProperty umlProperty)
{
    UmlId = umlProperty.Id;

    if (xmiDocument != null && umlProperty != null)
    {
        IsArray = ModelHelper.IsArray(xmiDocument, umlProperty.Id);
        IsOptional = ModelHelper.IsOptional(xmiDocument, umlProperty.Id);

        var propertyName = umlProperty.Name;
        if (propertyName.StartsWith("has") && propertyName != "hash") propertyName = propertyName.Substring(3);

        if (propertyName == "xlink:type") propertyName = "xLinkType";

        var name = propertyName.ToTitleCase();
        if (IsArray) name = ModelHelper.ConvertArrayName(name);

        Id = $"{idPrefix}.{name}";
        Name = name;
        DataType = ParseType(xmiDocument, umlProperty.Id, umlProperty.PropertyType);
        DataTypeUmlId = umlProperty.PropertyType;

        var description = umlProperty.Comments?.FirstOrDefault().Body;
        Description = ModelHelper.ProcessDescription(description);
        if (string.IsNullOrEmpty(Description)) Description = ModelHelper.GetClassDescription(xmiDocument, umlProperty.PropertyType);
    }
}

Hard coded method to determine if property is array:

https://github.com/TrakHound/MTConnect.NET/blob/master/libraries/MTConnect.NET-SysML/ModelHelper.cs#L366

public static bool IsArray(XmiDocument xmiDocument, string id)
{
    switch (id)
    {
        case "_19_0_3_91b028d_1579274935610_708920_3095": return true; // Devices.Component.Components
        case "_19_0_3_91b028d_1579274803419_180043_3064": return true; // Devices.Component.Compositions
        case "_19_0_3_45f01b9_1581211888318_232581_149": return true; // Devices.Component.References

        case "_19_0_3_68e0225_1633431910074_887850_97": return true; // Devices.Configuration.Relationships
        case "_19_0_3_68e0225_1633431923171_707595_113": return true;  // Devices.Configuration.CoordinateSystems
        case "_19_0_3_68e0225_1633431989416_861348_140": return true; // Devices.Configuration.Specifications
        case "_19_0_3_68e0225_1677585034568_640359_707": return true; // Devices.Configuration.ImageFiles

       ...
    }

    return false;
}

Hard coded method to determine if property is required or optional:

https://github.com/TrakHound/MTConnect.NET/blob/master/libraries/MTConnect.NET-SysML/ModelHelper.cs#L438

public static bool IsOptional(XmiDocument xmiDocument, string id)
{
    switch (id)
    {
        case "EAID_C7968EFA_A55E_4ccf_B7CA_CFE13D86C116": return true; // Devices.DataItem.Maximum
        case "EAID_72FA9526_7E5D_4084_A7F0_39FD402026E6": return true; // Devices.DataItem.Minimum
        case "EAID_7B83A56F_E21C_4aa3_9851_413935CE01A2": return true; // Devices.DataItem.Nominal

       ...
    }

    return false;
}


"value instanceof Function" used rather than "typeof value === 'function'"

I was trying to integrate our GWT application to this stomp-js library. I believe it the most official project for stomp-js?
i discovered that its not calling the correct parseConnect args lines, due to instance of not working over iframes etc.
Could the parseConnect() function be updated to support GWT properly?
from
if (args[1] instanceof Function) {
to
if (typeof args[1] === 'function' ) {
See https://github.com/paypal/paypal-checkout/issues/626 && https://github.com/webcomponents/custom-elements/issues/64#issuecomment-282686071 for similar issues.

Made this change in the GWT-compat branch. Please test by installing from that branch:
$ npm install stomp-js/stomp-websocket#GWT-compat

Shouldn't there be a triple equals in that change to avoid type conversion?

The file you are checking is probably the .cofee file, which gets compiled to .js.
Please check https://github.com/stomp-js/stomp-websocket/blob/30bf29ec0219c1333de0b977d5434242b2b0cf85/lib/stomp.js#L258

Indeed!! You are right sir. Thank you very much.
I tested it locally and it all works properly now.
I used https://github.com/gordon00/gwt-stomp and extended it to support the reconnect functionally

Released 4.0.4 with this change.

Planning for next version of stompjs is on the way. Your participation will be really appreciated, please join at https://github.com/stomp-js/stompjs/issues/1


Model Builder Error.

Problem encountered on https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/data
Operating System: windows
Error Encountered :
Model Builder Error.

Stack Trace:
at Microsoft.ML.ModelBuilder.AutoMLService.Experiments.AutoMLExperiment`3.d__23.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.ML.ModelBuilder.AutoMLEngine.d__28.MoveNext()

Hello,
Sorry you're hitting this error.

How long did you train for?
What scenario type did you choose?

Hey, thanks for choosing to help out.
I was trying to follow the sentiment analyzer tutorial on the ml.net docs online, that was where I hit the error. But I think I had a way through it recently, I simply tried the whole thing again, new project i mean, and it worked.
Although if such repeats itself again, I might not be able to solve it, that's the fear

@ZionLloyd05 Thanks for reporting! It sounds like you fixed the problem. If this repeats itself, please let us know by commenting here or opening a new issue. Thanks!


Does CereLink enable functionality similar to Central's "SingleNeuralChannel"?

Hi all,
In Central, there's an app to display activity from a single neural channel. This window is very useful and has 90% of the functionality that I need but unfortunately not quite enough; I have to write my own app. I'm curious about how this window achieves a couple different things.

Even if a channel is not configured to acquire continuous data, the window still displays the continuous data at the bottom. Even if this window is open and displaying continuous data, continuous data does not appear in the raster plot.
Even if a channel is not configured to acquired high-pass filtered data, this window still displays a high-pass filtered signal in the 'Strip' tab from which the threshold can be set.

How is (1) achieved? Does CereLink expose that functionality?
I guess that (2) can be achieved by filtering (1), though this seems dangerous if the filters don't match exactly those used by the NSP because then the threshold set by the user wouldn't perform exactly as expected on the NSP.

I have to look to remember (should ask Hyrum) but I remember there were some for-display data (i.e. low sample rate) that are sent. Also, there is a raw channel that can be configured along with the normal sample rate (both at the same time).
Everything should be in cbhwlib somewhere.

FYI, the reason I'm pursuing this is because I'm running into performance problems trying to plot 8 channels at 30 kHz + 4 channels at 2 kHz.
Central does not allow configuring a channel to transmit both SDK_SMPGRP_RAW and SDK_SMPGRP_RATE_30k, but RAW and something less than 30kHz is allowed. Is that a limitation of the NSP or just Central?
cbPKT_STREAMPREV sounds interesting. In its definition I don't see any data fields that would hold data. Can you indicate which?
The code comments suggest that cbPKT_STREAMPREV.chid should be 0x8000 + channel identifier. It seems like this chid doesn't meet any logical checks in ProcessIncomingPackets (there is no corresponding else for that second if). Would this be the right place to put it? I can attempt this, but I will only bother doing so if this gets me closer to getting easy-to-plot data.

Analysing the package traffic it seems like Central sends out a cbPKTTYPE_PREVSET every some 250ms. I didn't see a definition of the packet, but it just seems to be 0x8000 + chid, and empty payload. That gets confirmed by the NSP with cbPKTTYPE_PREVREP, followed by cbPKT_STREAMPREV every 20 ms. I guess Central then calculates the displayed value from rawmin and rawmax?

The comments describe what each field is. For example rawmin and rawmax are the minimum and maximum of the raw channel (the additional raw channel) for that time period (I think around 250ms). Or envmin and envmin are enevelope's min and max (that is shown in threshold). To plot, just plot a bar with those values, and it will look like zoomed-out version of the high sample rate data.
It is sent only for one channel currently under preview, Central sends some packet to tell which channel is currently open.
Central not configuring SDK_SMPGRP_RAW could be to avoid going above bandwidth I think.

I think monsource is the channel to be monitored

Thanks, I'll give it a try later this week.


C++/WinRT

It would be really great if all these pages documenting UWP could show C++/WinRT sample code. Unless it's made easy for people to get going C++/WinRT takeup is going to be very slow and it may fail to take off entirely. Good documentation with code samples is essential.

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

ID: 24777eff-7485-a1ba-17b8-d56547bbd26d
Version Independent ID: c5da8413-6441-3d07-4b84-02f145336771
Content: Menus and context menus - Windows UWP applications
Content Source: windows-apps-src/design/controls-and-patterns/menus.md
Product: uwp
GitHub Login: @jwmsft
Microsoft Alias: jimwalk

Hi, @egyik, thanks for filing this issue.
We don't provide example code in every language, but the fact that you're asking in this case means that there's a demand, so I'll take that into account with my planning and my TODO list.
FYI, the C++/WinRT documentation covers all kinds of techniques around using C++/WinRT in general. Handling events, authoring and consuming APIs, asynchronous patterns, some topics specifically about XAML, and so on. There are plenty of code examples in there showing all kinds of techniques. And there's the C++/WinRT API reference content, too. It's likely that the content there will be all you need to get started with menus. If you come across any specific scenarios or APIs that are challenging to use with a given feature area, then please let me know, and I'll see whether I can provide C++/WinRT code for them and/or fill any doc gaps.
~Steve

@egyik, have you run into any specific blockers with using the C++/WinRT language projection with these APIs? ~Steve

Hi, @egyik, I'll go ahead and close this issue now since it's been a while, and it feels resolved. Please reopen if I made a mistake in closing. ~Steve


Copy pattern from other route

It would be great if there was the possibility to copy/move a pattern from one route to other, in order to minimize work while creating patterns with similar itineraries.
It could be done either by duplicating a pattern and then assigning it to other route, or copying a pattern directly from another route or even bringing the patterns when you duplicate a route, as it was discussed in #259.

Hi @gbragaalves, We agree. It's a large missing use case to be able to copy patterns across routes. We have this in our roadmap, and I will update this issue when we have added the feature. Thanks for opening an issue for this!


Training programs needs a complete list

Program
ANE-Anesthesiology
ANE-Anes Cardiothoracic
ANE-Critical Care Medicine
ANE-CTCC Fellowship-Non Approved ACGME
ANE-Neuro Non-Approved
ANE-Pain Medicine
ANE-RAPPM-Non Approved ACGME
CHFM-Family Medicine
CHFM-Sports Medicine
DERM-Dermatology
DERM-Dermatopathology
EM-CCM Non-Approved
EM-EM Ultrasound Non-Approved
EM-Emergency Medicine
EM-Emergency Medical Services (EMS)
ENT-Otolaryngology
MED-Internal Medicine
MED-Advanced Endoscopy Non-Approved
MED-Advanced Heart Failure and Transplant Cardiology
MED-Cardiac Electrophysiology Non-ACGME Approved
MED-Cardiovascular disease
MED-Cardiovascular Non ACGME Approved
MED-Clinical Cardiac Electrophysiology
MED-Endocrinology
MED-Gastroenterology
MED-Geriatric Medicine
MED-Hematology/Oncology
MED-Infectious Disease
MED-Interventional Cardiology
MED-Interventional Pulmonary Non-Approved
MED-Nephrology
MED-Pulmonary disease & Critical Care medicine
MED-Rheumatology
MED-Sleep Medicine
MED-Transplant Hepatology
MED-Transplant Nephrology Non-Approved
NEURO-Behavioral Neurology
NEURO-Child
NEURO-Movement
NEURO-Neurology
NEURO-Vascular/Stroke
NS-Neurological Surgery
NS-Endovascular Surgical Neuroradiology
NS-Transition to Practice Non-Approved
OBGYN-Obstetrics & Gynecology
OBGYN-Maternal-Fetal Medicine
OPTH-Cornea Fellowship-Non Approved ACGME
OPTH-Ophthalmology
OPTH-Retinal Fellowship-Non Approved ACGME
ORTHO-Orthopaedic Surgery
ORTHO-Hand Surgery
ORTHO-Musculoskeletal Oncology
ORTHO-Adult Reconstructive Non-Approved
ORTHO-Shoulder/Elbow Fellow Non Approved ACGME
ORTHO-Trauma Fellow Non Approved ACGME
PATH-Pathology-anatomic and clinical
PATH-Cytopathology
PATH-Gastrointestinal Path-Non Approved ACGME
PATH-Hematopathology
PATH-Neuropathology
PATH-Surgical Path-Non Approved ACGME
PEDS-Pediatrics
PEDS-Child Neurology
PEDS-Pediatric Cardiology
PEDS-Pediatric Critical Care
PEDS-Pediatric Endocrinology
PEDS-Pediatric Gastroenterology
PEDS-Pediatric Hematology/Oncology
PEDS-Pediatric Neonatal-Perinatal
PEDS-Pediatric Nephrology
PEDS-Pediatric Pulmonary
PEDS-Pediatric Rheumatology
PEDS-Sports Medicine
PSYCH-Addiction Medicine
PSYCH-Child & Adolescent Psychiatry
PSYCH-Forensic Psychiatry
PSYCH-Geriatric Psychiatry
PSYCH-Psychiatry
RADONC-Radiation Oncology
RADONC-Non-Approved
RAD-Radiology-diagnostic
RAD-Physics-Non ACGME
RAD-CRQS Fellowship Non-Approved
RAD-Musculoskeletal Radiology-Non ACGME
RAD-Neuroradiology
RAD-Vascular/Interventional Radiology
SURG-Surgery
SURG-Integrated Plastics
SURG-Pediatric Surgery
SURG-Plastic Surgery
SURG-Surgical Critical Care
SURG-Thoracic Surgery
SURG-Vascular Surgery
URO-Urology

This is no longer required. Is now Self Classification


Upgrade Backstage to React 18

update dependencies to react 18
add PropsWithChildren to testutils
update components to use PropsWithChildren
update react hooks
drop @testing-library/react-hooks
fix updateImportState tests
update dependencies

Hey, I just made a Pull Request!

This pull-request updates the dependencies in Backstage from React 17 to 17 || 18.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

This PR is a duplicate of #17414 since the previous PR was closed out of being stale.

So it looks like the best way forward for the React 18 upgrade in this repository will be to break the version window policy for React. I've written an RFC for the matter in #18625.
The gist is that it will take an enormous effort to maintain compatibility (particularly in testing) of both React 17 and 18. Doing so would introduce a lot of complexity into the codebase, increasing the amount of work for both maintainers and contributors/developers.

#18714 Is another one that will need to get addressed before this can merge

not stale

not stale

not stale

Replaced by https://github.com/backstage/backstage/pull/20598 and others leading up to it


ENH: Require pyaml greater than 5.4

Closes #66

Codecov Report

Merging #67 (81165cc) into master (4cf13fb) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
- Coverage   88.83%   88.81%   -0.02%
==========================================
  Files          31       31
  Lines        2194     2191       -3
  Branches       50       50
==========================================
- Hits         1949     1946       -3
  Misses        235      235
  Partials       10       10

Impacted Files
Coverage Δ

avionix/chart/chart_builder.py
94.51% <0.00%> (-0.04%)
:arrow_down:

avionix/chart/chart_dependency.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Android 14 Beta 5 crop not working (Pixel 6 Pro)

[X] I have read the FAQ.
[X] I have searched in existing issues.

Environment

OS: Windows 11
scrcpy version: 2.1
installation method: Win x64 from GitHub
device model: Pixel 6 Pro
Android version: 14 Beta 4

Describe the bug
Tried to crop the black bar from the screen which is always visible on Pixel 6 Pro in Landscape mode. It worked perfectly on Android 13
but crop is completely ignored on Android 14 Beta 4
--crop 1440:2600:0:120 removed the black bar on Android 13 but doesn't do anything on Android 14
Note: Mouse position is still wrong if the crop is applied and also the scrcpy output says it has cropped
INFO: Texture: 1440x2480 vs. INFO: Texture: 1440x2600  if crop is 0
scrcpy.exe -w --video-bit-rate 25M --video-codec=h265 --audio-bit-rate=256K --audio-codec=opus --crop 1440:2600:0:120
scrcpy 2.1 <https://github.com/Genymobile/scrcpy>
C:\STORAGE\scrcpy-win64-v2.0\scrcpy-server: 1 file pushed, 0 skipped. 77.6 MB/s (56955 bytes in 0.001s)
[server] INFO: Device: [Google] google Pixel 6 Pro (Android 14)
INFO: Renderer: direct3d
INFO: Texture: 2480x1440
INFO: Texture: 1440x2480
[server] INFO: Device rotation requested: landscape
INFO: Texture: 2480x1440

@rom1v installed Beta 5 now, seems it's still broken. Do you have any idea what has changed from Android 13?

Nope, I don't know, they changed things related to display, which also broke --turn-screen-off (#3784). To be investigated (once the source code of Android 14 is released)…

FWIW: seems like the source is released now: https://android-developers.googleblog.com/2023/10/android-14-is-live-in-aosp.html
I can also confirm crop not working* on the Pixel 8 Pro, so nothing's changed there since beta it seems.
* The inputs do get "cropped" though, so any touches go to wrong places.

See https://github.com/Genymobile/scrcpy/issues/4011

@rom1v Thank you very much, can you provide updated binaries so I can test it?

Please test a new version in #5455, a binary is provided.

Thanks again, verified it's working as expected.


VEP option to return alleles in VCF format

Description
VEP has a new optional field to return the alleles in VCF format.
Use case
This new option allows to return the exact ref or alt alleles in case of an insertion or deletion.
Benefits
More output alleles info.
Possible Drawbacks
None
Testing
Run all tests and no regression detected.
Changelog
The default functionality is the same, there's only one more optional field in:
vep/species/hgvs
vep/species/id
vep/species/region

Coverage decreased (-0.0002%) to 44.284% when pulling c4c51d9eb05990e55d4c929149b6bc6079b9c624 on dglemos:vep/vcf_string into 545386fe7c5f52c2018e452836080c4fa665b914 on Ensembl:master.


Using TEST_JDK_HOME/java to launch ant for TKG compilation

move JAVA_HOME export to makefile

Signed-off-by: Renfei Wang<EMAIL_ADDRESS>

To keep the test story simple, we would like to use the same java to launch ant for both TKG compilation and test material compilation.


Request for adding DefectSAM

Hi,
Could you please consider adding this project?
Segment Anything in Defect Dectection.
paper link:
https://arxiv.org/abs/2311.10245
code link:
https://github.com/bozhenhhu/DefectSAM

Hi, we have added your paper to our list.

Thanks a lot !


Fix scrutinizer build bug (submodule related)

After adding support for it (see #13) the build is failing because the submodules are not being cloned:

The solution is to do something like:

closing since scrutinizer is not used


Register assignation to custodian when invitation is sent

When an admin tries to send an invitation to a custodian and assign a register, if the register has already a custodian or it's part of an invitation already sent, it doesn't show in the dropdown list but it can be typed. Once the invitation is sent, the custodian can accept the invitation but he/she is not going to have a register assigned.
Considering the 'Delete invitation' functionality does not exist, this can produce blocking situation difficult to unblock just using the RMT.
URL:
https://managing-registers-staging.cloudapps.digital/users/invitation/new?role=custodian
Steps to recreate:

Login as an admin
Click 'Manage custodians'
Invite a new custodian and use a already assigned Register.

Expected result:
a) A message that the register can not be assigned to the custodian because is already assigned
b) Restrict the text can be typed in the dropdown list to just the available options.
Note: Business should decided about this.
Actual result:
Invitation is sent as usual and can be accepted, leaving a custodian without Register
/cc @Maidment64

Option a) 'A message that the register cannot be assigned...' please!

Does not apply any more. It is not possible to assign already assigned registers any more


Handle task migrations correctly

Fixes #31.
This removes the previous use of CommitFailedException (which is Java-only) and instead mimics the Java Kafka Streams by translating certain error codes received when committing a task into a TaskMigratedError. This error is logged by StreamThread but otherwise seen as a signal to "try again".

Made code backwards-compatible with versions for confluent-kafka-python that lack INVALID_PRODUCER_EPOCH.


Add/small improvements

Small improvements.
To discuss: Can we equate<EMAIL_ADDRESS>with<EMAIL_ADDRESS>

To discuss: Can we equate<EMAIL_ADDRESS>with<EMAIL_ADDRESS>
What are the requirements for ActivityPub / Activity Streams? Is there any precedent conversation about this, by any chance?

What problem is this fixing?

GoToSocial seems to not support dots in pathes

@mattwiebe @jeherve if this is too tricky, I will remove it, to discuss it separately.
But there are some other improvements that should be merged!


update lottie dependencie to ^2.2.0

I just updated the version of lottie library

Lottie update is required. Can you please update your package?

@newtaDev is the repo still active?

@dagovalsusa  Thank you for the PR, really appreciate your effort will review and merge ASAP

@newtaDev is the repo still active?

Yes
And I also have a plan to rebuild this project....  will try do it ASAP

I need that update Lottie dependence too, please help. Thank you.

We are no longer using lottie
Thank you


Summary payload doesn't have few fields

Updated summary payload
Checklist:

[ ] verified by installing it in full mode and checked the payload. for each and every event
[ ] verified by installing it in Hyperion mode and checked the payload. for each and every event

Describe the bug
Updating summary payload
Expected behavior
Summary events should have few more fields
Current behavior
describe what you currently experienced from this process, and thereby explain the bug
Screenshots
If applicable, add screenshots to help explain your problem.
Logs
Paste any relevant application logs here.

Changes are done and it is released


bottomsheet add a position. slightly slightly ajar (100 dp) approximately.

Now the bottomsheet opens approximately in the middle of the screen. When you swipe down, it closes. Is it possible to add another position, approximately 100 meters high. That is, what would have turned out to be three positions: in the middle, full screen, and a slightly open position.

Android
Android 9
Device

@Bezlepkin i dont understand what you are talking about?
Please give more information when you open an issue.

So that you can open it:

full screen (as currently implemented)
in the middle of the screen (as already implemented)
to open a little bit (as in the video)

@Bezlepkin seems nice. I wont implement it until i need it. Sorry a lot to do / maintain. But if you want to have a go at it would love to have a PR :D

@Bezlepkin seems nice. I wont implement it until i need it. Sorry a lot to do / maintain. But if you want to have a go at it would love to have a PR :D

I would love to! But where to start? :)

@Bezlepkin first look at the native APIs to see how they handle it. Then try to find an API which would work for bot platforms then implement and test it

first look at the native APIs to see how they handle it. Then try to find an API which would work for bot platforms then implement and test it

For this implementation, you do not need to add code to Java Android?

@Bezlepkin there is no java code. It is all handled from the JS side (but still calling java code)

I have been interested in this behavior, in android it is very easy since it lets you indicate at the height that you want to open the bottomshet, in iOS I have no idea but I will try to take some time to investigate as well

@Bezlepkin I have it working in android but in ios I don't get it. have you been able to see any of this?

I have it working in android but in ios I don't get it. have you been able to see any of this?

No! How are you do it for Android?

Sorry @Bezlepkin, here you can see how to add this functionality only for android, the other options are not checked, only the default height that the developer wants. Repo

@farfromrefug I have been able to add this functionality for android here, but for ios it does not work: here. According to the documentation it should do that, I suspect that it is related to this method but I would not know how to do it. Do you know a "quick" way to implement this?

@vallemar not sure you can use the property to set it in N on iOS. Try to use setPrefered...
Also if you make a PR i am not sure about the name collapsedHeight i would prefer we use the name used on iOS or android

Además, si hace un PR, no estoy seguro del nombre collapsedHeight, preferiría que usemos el nombre que se usa en iOS o Android

Perfect, I would stick with the iOS one. I will not do the PR until I find a solution for iOS. Thanks! @Bezlepkin Have you been able to take a look at this?

@Bezlepkin @vallemar just released 6.0.2 with support for peekHeight on iOS/Android. Thanks @vallemar for your PR!

@Bezlepkin can you close this issue?


Add onMyMediaChange event api

I made the changes in the following files:

NameTag.css to convenient for my testing

draw_badge_api.ts:

add drawCameraSizeChange function inside the DrawBadgeApi, it takes the detected width and height as input.
Modify drawEverythingToImage takes the width and height as input and using the scale factors to determine the new axis values

app/main/page.tsx

add onMyMediaChange inside this file. Ideally, this should be inside the /lib/draw_badge_api, but I haven't figured out how to return the width and height as an event listener.

I don't have the extra camera, so I was not able to test how it performs on different types of video inputs.

@blickly can you take a look?

Thank you @kexinF for taking a stab at this. The changes were adapted into PR  #90 and merged into the repo.


Fix broken URL in tensorflow/java/README.md

Fixes broken URL in tensorflow/java/README.md, this fix fixes #12141.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Can one of the admins verify this patch?


Redis Queue Driver disable Continuous debug log in console

I have configured RedisQueue Driver for my vapor project. While running the app the console becomes flooded with continuous debug log from the driver connection. I can barely see the other logs. Is there any way to disable the RedisQueue driver connection log? Screenshot of my console is attached. (Notes: I am running the service with docker compose)

Hello @binarysid , I discussed the topic with @fabianfett . While we agree that it's verbose, this is expected behaviour when logging on a debug log level.
The current behaviour is mostly inline with the SSWG Log Level guide. However, as a results of yours logs we do think that the statement for finding connections in a pool should be a trace instead.
The solution is to increase the logging level to info instead.


Output random code

VSCode Version: Code 1.18.1 (929bacba01ef658b873545e26034d1a8067445e9, 2017-11-16T18:32:36.026Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author (truncated)
Version

project-manager
ale
0.23.1

java-debug
DSn
0.0.2

code-runner
for
0.8.6

vscode-javac
geo
0.0.47

beautify
Hoo
1.1.1

python
ms-
0.9.1

cpptools
ms-
0.14.5

csharp
ms-
1.13.1

team
ms-
1.122.0

java
red
0.16.0

vscode-icons
rob
7.19.0

python
tht
0.2.3

python-extended-snippets
tus
0.0.1

vscode-java-debug
vsc
0.4.0

vscode-java-pack
vsc
0.2.0

Steps to Reproduce:

Reproduces without extensions: Yes/No

Please give us more information, which your given information we can't reproduce it or can't do anything. :)
Also try to disable all extensions by using the --disable-extensions parameter.


App - Unhandled exception: SyntaxError: Unexpected token N in JSON at position 0

Not sure, but think this is an error with the json, and therefore perhaps in the docker container?
Get the following when trying to run
[2019-04-08T15:23:19.694] [INFO] server.ts - Starting Landroid Bridge... [2019-04-08T15:23:19.721] [INFO] server.ts - Setting port to 3000... [2019-04-08T15:23:20.268] [INFO] Mqtt - Connecting to MQTT Broker... [2019-04-08T15:23:20.296] [INFO] App - Adding static files path /usr/src/landroid-bridge/www [2019-04-08T15:23:20.301] [INFO] Scheduler - Skipping scheduler initialization (not enabled) [2019-04-08T15:23:20.339] [INFO] LandroidS - Initializing Landroid Cloud Service... [2019-04-08T15:23:20.353] [DEBUG] IoBrokerAdapter - UUID: 7a38a2f0-5a01-11e9-95b5-079d9f0c8e78 [2019-04-08T15:23:20.354] [WARN] IoBrokerAdapter - landroid-cloud-2.js will be replaced in next version by mqttCloud.js and worxConfig.js. Reason: better handling of models and mqttCloud supports more brands by a diffent config see also: https://www.npmjs.com/package/iobroker.kress [2019-04-08T15:23:20.357] [DEBUG] IoBrokerAdapter - API token set to *************** [2019-04-08T15:23:20.359] [DEBUG] IoBrokerAdapter - post:{"email":"@gmail.com","password":"","uuid":"*********","type":"app","platform":"android"} [2019-04-08T15:23:20.361] [DEBUG] IoBrokerAdapter - ***************** [2019-04-08T15:23:21.022] [ERROR] App - Unhandled exception: SyntaxError: Unexpected token N in JSON at position 0

Let's wait for an answer in this issue: https://github.com/weweave/landroid-bridge/issues/46

https://github.com/MeisterTR/ioBroker.landroid-s/commit/5b61b2ac81896cb809358d82c07011902308f27c perhaps this fix it, is it possible to update the hassio addon with these changes @stefanlaheij

Hi @olskar, I changed some code in the landroid-bridge project and pushed a new version of the add-on. For me it is working again!

Possible to implement the latest changes here @stefanlaheij ?
https://github.com/virtualzone/landroid-bridge/commits/master

Hi @olskar,
If have changed the repo from weweave to virtalzone. Is that what you meant?


Update vamas.py for floating coordinate in MAP file

Some MAP VAMAS file in XPS give coordinate in a floating format (example: -2.156)  when you read the file you get a string like '-2.156\n' which cause a fault when using int() I propose to use int(float('-2.156\n')) to get the int value of the coordinate

Thanks for reporting this. Unfortunately, I could never test MAP files.
Do you know if there are cases in which the value is closer to the next higher integer?
E.g. int(float(2.9) would give a value of 2. Is that the bahavior we want or does 3 make more sense?

Thanks for reporting this. Unfortunately, I could never test MAP files. Do you know if there are cases in which the value is closer to the next higher integer? E.g. int(float(2.9) would give a value of 2. Is that the behavior we want or does 3 make more sense?

Sorry, didn't see you question. Back to use this library, I don't know exactly what to answer. My case was a user with a MAP file with only one spectra. Maybe I should ask for an image to test the result of the data  extraction on real MAP file.

In general, it would be nice if we would have a bit more files, ideally from different machines of which we know the parameters that were set. Then we could write more tests to catch edge cases like this one. Personally, I just have access to one AES and one XPS.
Let me know if you want to share some files (can be meaningless data, as it gets added to this repo) or even write some tests. It would help a lot.


LIC not working

The issue is limited to Line Integral Convolution (LIC) not showing anything. For example in example Fluid_Basic_LIC, I can only see UI when I run the sketch. As soon as I press "FF" button, I can see fluids moving around. Back to LIC, nothing. Same issue repeats in all LIC examples.
I am on MacOS version 10.14, Radeon Pro 575 4096 MB. Can anyone please help me get around this problem please?
Btw, amazing library, really great work !

Just tried running it in windows machine, GeForce GTX 1080, same issue. Could it be related to a new version of Processing? I am using 3.5.3.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #28 (58d8d83) into main (79fc780) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   98.22%   98.22%
=======================================
  Files          19       19
  Lines         338      338
  Branches       33       33
=======================================
  Hits          332      332
  Misses          6        6

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

Codecov Report

Merging #28 (58d8d83) into main (79fc780) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   98.22%   98.22%
=======================================
  Files          19       19
  Lines         338      338
  Branches       33       33
=======================================
  Hits          332      332
  Misses          6        6

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


[freeimage] patch typedef for MacOS

The idea of the PR is to check if OBJC_BOOL_DEFINED is defined to avoid a second redefinition with a different type.
Fixes #38260

[X] Changes comply with the maintainer guide.
[ ] ~SHA512s are updated for each updated download.~
[ ] ~The "supports" clause reflects platforms that may be fixed by this new version.~
[ ] ~Any fixed CI baseline entries are removed from that file.~
[ ] ~Any patches that are no longer applied are deleted from the port's directory.~
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

@microsoft-github-policy-service agree company="Mega Limited"

@microsoft-github-policy-service agree company="Mega Limited (MEGA)"

Hi @WangWeiLin-MV
Is there something we can do to move this work forward?

You must click "Ready for review".


FIX: Use correct XPath syntax

Fixes compatibility with nokogiri 1.13.0

No longer necessary with nokogiri 1.13.1


flowcharting panel can not save

What did you do?
I installed flowcharting for my grafana, then I tried to draw a flow chart by XML source file generated by draw.io
What did you expect to see?
I see my chart, all good so far.

but when I try to save this panel, something weired happend.
What did you see instead? Under which circumstances?
when I click "back" button, my chart disappeared:

edit my panel again, all settings are gone:

Environment
grafana:v6.7.4(8e44bbc5f5)
flowcharting:0.9.1
Tried many times, nothing changed.
and no error logs in browser console
I'm not sure is this a bug or just a grafana version issue?
appreciated for any suggestions

Hi @bigHave
Can you open the console and see if you have an error ?

Hi @bigHave
Can you open the console and see if you have an error ?

Hi @algenty , thanks for replying, I opened the console but can not see any error.
but I found my issue is similar with #197 , so I tried version 0.9.1 and problem solved.


Configure distinct user agent

Instead of using the default User-Agent set by the SDK, configure a distinct one for cli.

Open question: What should the User Agent be?
SDKs are of the format: openfga-sdk {sdkId}/{packageVersion}, e.g. openfga-sdk go/0.1.1
Suggestion for the CLI would be: openfga-cli/{packageVersion} e.g. openfga-cli/0.0.1

Open question: What should the User Agent be?
SDKs are of the format: openfga-sdk {sdkId}/{packageVersion}, e.g. openfga-sdk go/0.1.1
Suggestion for the CLI would be: openfga-cli/{packageVersion} e.g. openfga-cli/0.0.1

your suggestion tracks 👍


invalid issuer in  UserPrincipalManager

JWTClaimsSet checks for issuer that is not equal to my token, how can I override this or resolve with configuration.
Environment

Spring boot starter:

active directory spring boot starter 2.1.6

OS Type: Windows/Linux/MacOS

Java version:

1.8

Summary
Issuer from azure ad is https://login.microsoftonline.com/uuid/v2.0
Code  throws BadJWTException because the issuer doesn't contain "https://sts.windows.net/" or "https://sts.chinacloudapi.cn/".
Reproduce steps
Add spring boot starter to my zuul project, then configured aad app, added properties and security config below.
`@Configuration
@EnableGlobalMethodSecurity(securedEnabled = true,
prePostEnabled = true)
public class SecurityConfig extends WebSecurityConfigurerAdapter {
@Autowired
private AADAuthenticationFilter aadAuthFilter;

@Override
protected void configure(HttpSecurity http) throws Exception {

    // Disable CSRF (cross site request forgery)
    http.cors().and().csrf().disable();

    // No session will be created or used by spring security
    http.sessionManagement().sessionCreationPolicy(SessionCreationPolicy.STATELESS);

    // Entry points
    http.authorizeRequests()
            .antMatchers("/signin").permitAll()
            // Disallow everything else..
            .anyRequest().authenticated();

    // If a user try to access a resource without having enough permissions
    http.exceptionHandling().accessDeniedPage("/login");
    http.addFilterBefore(aadAuthFilter, UsernamePasswordAuthenticationFilter.class);

    // Apply JWT
   // http.apply(new JwtTokenFilterConfigurer(jwtTokenProvider));

    // Optional, if you want to test the API from a browser
    // http.httpBasic();
}

@Override
public void configure(WebSecurity web) throws Exception {
    // Allow eureka client to be accessed without authentication
    web.ignoring().antMatchers("/*/")//
            .antMatchers("/eureka/**")//
            .antMatchers(HttpMethod.OPTIONS, "/**"); // Request type options should be allowed.
}

@Bean
public PasswordEncoder passwordEncoder() {
    return new BCryptPasswordEncoder(12);
}

@Bean
public AuthenticationManager customAuthenticationManager() throws Exception {
    return authenticationManager();
}

}
`
Expected Results
Validate token from issuer https://login.microsoftonline.com/uuid/v2.0
Actual Results
BadJWTException("Invalid token issuer")
`
private ConfigurableJWTProcessor<SecurityContext> getAadJwtTokenValidator(JWSAlgorithm jwsAlgorithm) {
     final ConfigurableJWTProcessor<SecurityContext> jwtProcessor = new DefaultJWTProcessor<>();

    final JWSKeySelector<SecurityContext> keySelector =
            new JWSVerificationKeySelector<>(jwsAlgorithm, keySource);
    jwtProcessor.setJWSKeySelector(keySelector);

    jwtProcessor.setJWTClaimsSetVerifier(new DefaultJWTClaimsVerifier<SecurityContext>() {
        @Override
        public void verify(JWTClaimsSet claimsSet, SecurityContext ctx) throws BadJWTException {
            super.verify(claimsSet, ctx);
            final String issuer = claimsSet.getIssuer();
            if (issuer == null || !issuer.contains("https://sts.windows.net/")
                    && !issuer.contains("https://sts.chinacloudapi.cn/")) {
                throw new BadJWTException("Invalid token issuer");
            }
        }
    });
    return jwtProcessor;
}

`

@jtc198322
Thanks for issue this. For double check, you only leverage AADAuthenticationFilter for your security configuration, right ?
The filter only take care of Auhorization header and validate it from below configuration
https://github.com/Microsoft/azure-spring-boot/blob/0caebdeaf694a69619eb07d39f1ca0ca3c81cc2f/azure-spring-boot/src/main/resources/serviceEndpoints.properties#L1-L8
You may need to provide more configuration about you code, so we can tell where is wired.

I didn't have most of these in place but I added a few. Same results
azure:
  activedirectory:
    clientId:  ***********************************
    clientSecret: *******************************
    activeDirectoryGroups: group1,group2
  service:
    endpoints:
      global:
        aad-signin-uri: https://login.microsoftonline.com/
        aadGraphApiUri: https://graph.windows.net/****************************

@jtc198322
In fact you may not have necessary to configure that, use default value from serviceEndpoints.properties is good enough

Most examples only indicate these...
azure:    activedirectory:     clientId:  ***********************************     clientSecret: *******************************     activeDirectoryGroups: group1,group2
Looks like the only way to avoid this code in the filter is  not to get into the below, should I already have any of these to make this false?
` if (principal == null || graphApiToken == null || graphApiToken.isEmpty()) {
principal = principalManager.buildUserPrincipal(idToken);
                final String tenantId = principal.getClaim().toString();
                graphApiToken = client.acquireTokenForGraphApi(idToken, tenantId).getAccessToken();

                principal.setUserGroups(client.getGroups(graphApiToken));

                request.getSession().setAttribute(CURRENT_USER_PRINCIPAL, principal);
                request.getSession().setAttribute(CURRENT_USER_PRINCIPAL_GRAPHAPI_TOKEN, graphApiToken);
            }`

Would it be possible to make the hard-corded strings of the issuer dynamic supported by properties? For a single tenant...
 @Override         public void verify(JWTClaimsSet claimsSet, SecurityContext ctx) throws BadJWTException {             super.verify(claimsSet, ctx);             final String issuer = claimsSet.getIssuer();             if (issuer == null || !issuer.contains("https://sts.windows.net/")                     && !issuer.contains("https://sts.chinacloudapi.cn/")) {                 throw new BadJWTException("Invalid token issuer");             }         }     });

@jtc198322
I am sorry it seems I am not get your point for this issue. For aad, the service-endpoint is something like fixed value, and it should works well. Are you try to customize this endpoint with aad ? I am not sure if it is possible or not. Please correct me if something misunderstanding.

Basically my issuer from Azure AD is not "https://sts.windows.net/")  or issuer.contains("https://sts.chinacloudapi.cn/"))
it is from https://login.microsoftonline.com/{uuid}/v2.0
and the jwt verification fails.

I have the same issue. I think the error occur if the setup is single tenant.

I think the problem is giving an azure AD v2 token to a azure AD v1 implementation.

Facing similar issue with single tenant application,  is there any solution to this issue?

I think I'm facing the same issue here. I'm using the AAD Spring Boot starter with Spring Boot 2.1.4 and I receive version 1.0 tokens. The access token is not a JWT, the id token is and it includes the attribute "ver": "1.0" and "iss": "https://sts.windows.net/our-tenant-id/", whereas it should be "iss": "https://login.microsoftonline.com/our-tenant-id/v2.0".
Here's my application.yml:
spring:
  application:
    name: app-name
  security:
    oauth2:
      client:
        registration:
          azure:
            client-id: my-client-id
            client-secret: my-client-secret
...
azure:
  activedirectory:
    tenant-id: our-tenant-id
    active-directory-groups: group1, group2, admin
...

How can I configure AAD in my app to generate ver 2.0 tokens?
If somebody tells me that this is a different issue, I'll create a new issue.

@berndgoetz if you are having this issue you are already generating v2.0 tokens, the springboot implementation verifies the tokens using the adal for java library that validates v1.0 tokens and checks the issuer whitout incluind the "https://login.microsoftonline.com" issuer. You can do your own implementation using msal for java to validate v2.0 tokens or change the tokens you are sending to v1.0. I have been working in the msal implementation but didn't finished it because of a problem with the obo flow here is a link to my issue: problem with obo flow

Wouldn't it be possible to get the correct issuer from https://login.microsoftonline.com/{tenant}/.well-known/openid-configuration or https://login.microsoftonline.com/{tenant}/v2.0/.well-known/openid-configuration for v2.0 respectively, instead of hardcoding them?

Hi all,
Even I am facing the same issue. any workaround so far.
I am generating a token from msal in the web app and trying to authenticate it in Java(web api) using this lib "azure-active-directory-spring-boot-starter" . the prolbem is the token generated in msal having "https://login.microsoftonline.com" as issuer but the principal class checks for "https://sts.windows.net"

now the issuer(https://login.microsoftonline.com) has been added to the verifier. And the change has been merged to the master. You can try it by using the latest SNAPSHOT or 2.1.7 dependency.

I am facing the same issuer problem. I downloaded 2.1.7 version

com.microsoft.azure
azure-spring-boot
2.1.7

But the code within 2.1.7 does not reflect "https://login.microsoftonline.com"
I can see the fix is available at https://github.com/microsoft/azure-spring-boot/blob/master/azure-spring-boot/src/main/java/com/microsoft/azure/spring/autoconfigure/aad/UserPrincipalManager.java
But the 2.1.7 jars downloaded by maven still contains the old code.
To double check I downloaded source code from https://github.com/microsoft/azure-spring-boot/releases/tag/2.1.7 and yes it still contains the old version of the code (i.e.  the fix to include "https://login.microsoftonline.com" within token verify is missing)
Can you please confirm which version should be downloaded to get the updates?

@sasmitanair Thanks for pointing out this. You're right about the 2.1.7 release, the change's not been released with it. So perhaps you could try it with our daily built snapshot.

Would that be 2.1.8-SNAPSHOT in the maven world? Can you please point me to the daily built snapshot version?


Query: Include: Collection SQL and generated code improvements

When generating collection include query models, the cloned parent subquery should explicitly project the required columns for join comparison and ordering.
Also tweaked generated code to no longer capture QueryContext in closures.

Ping @smitpatel @maumar

New version up.

Build failure
-- Missing NotNull annotations --
Microsoft.EntityFrameworkCore.Query.Internal.AnonymousObject.IsGetValueExpression[querySourceReferenceExpression]
Expected: False
Actual:   True

@maumar Bug in ApiConsistencyTest :trollface: out parameter! I am fixing.


Clicking NO does not render error

The phtml does not render error after clicking NO.
oauth/receivecode?error=access_denied&error_description=The+user+denied+access+to+your+application&state=xyz

Can you do a screencast, by any chance? I'm not sure I understand what you're trying to detail.

ping @basz

Of what i remember. The UI didn't respond appropriately whenever the user denied the authorization request. It should render the reason stated in the url.
Unfortunately I moved away from agility atm.

This repository has been closed and moved to laminas-api-tools/api-tools-oauth2; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-oauth2/issues/16.


Document string parameter passed to IN clause

Document unexpected behaviour when a string is passed as an argument to the IN clause

for https://code.djangoproject.com/ticket/29493


Locking Drupal VM box_version

Issue Type

Feature Idea

Your Environment
Vagrant 2.0.3
VirtualBox 5.2.12r122591
ansible 2.5.0
  config file = None
  configured module search path = [u'/Users/thomaswilhelm/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/Cellar/ansible/2.5.0/libexec/lib/python2.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.14 (default, Mar 22 2018, 15:04:47) [GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-9<IP_ADDRESS>)]

Your OS

macOS (10.13.6)

Summary
I'm trying to see if it's possible to lock the box_version of drupal vm. I have drupal vm config that works nicely with box version 1.1.8, but I had some issues when upgrading the box to 1.2.2. Until I've got the time to figure out what's going on, it is possible to lock the box version to 1.1.8 somehow? That way I know when passing my project over to other developers, I'm confident they won't run into the same issues.
I've found that in the Vagrantfile there's a setting config.vm.box_version which is perfect if I could set it, but can't see any way to pass this in from drupal vm config.yml, is it possible?

I wrote a patch that then allows me to set vagrant_box_version: '1.1.8' in my config.yml for anyone interested. Apply using composer patches.
lock-box-version.patch.txt

@ThomWilhelm - I generally recommend using the latest, as it's been tested fully with Drupal VM at least in a couple build environments, but I can see a use case of locking a box version.
Note that you could set/override the variable using a Vagrantfile.local file inside your project root instead of patching the Drupal VM Vagrantfile, but either way is fine.


Discuss also third-party store apps

Third-party store apps are separate from custom apps.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Feature: Allow displaying current values of specified VSCode settings in statusBarLabel

Hey I really like many of your extensions they are great for fine tuning the editor!
I've been trying to find an extension that would let me display in the statusbar the current value of my VSCode's global/workspace editor.fontSize.
I also use your Zoombar extension with a custom zoomUnit and swapped out the default keybindings for zoom in/out to use Zoombar for changing the UI size. I also have keybinding set for changing the editor font size and so I can easily adjust both the UI/text elements seperatly, it's quite nice especially for multiple monitors of different resolutions to quick adjust sizing.
As far as what sysinfo can display in the statusbar, it's a bit unclear in the README.md, but in the source code it seems like there isn't support for displaying any global/workspace settings values like editor.fontSize, but as I far as I understand it wouldn't be too difficult to include. I'll mess around with a fork for a bit and see what I can do and if you're available I could submit a PR as well :P
Thanks again for all your cool extension work!

Thank you for using my extensions!
I just released v2.5.1.
$(symbol-color) ${settings:workbench.colorTheme}
$(text-size) ${settings:editor.fontSize}
You can use specifications like these.
How about this?

Hey that's awesome! Exactly what I was hoping for, thanks for such a quick response and updates!


Curated list of Haskell packages

First off, great initiative! Hopefully this can replace haskell.org instead of competing with it, but I'll leave that for another discussion.

From what I could gather about the project, one of the goals was to make the website a stop for newcomers where they could navigate how to learn Haskell.
One of my major gripes as someone a bit inexperienced still, but following the community closely, is that I often have a hard time finding out what the recommended packages are for certain things.
There are efforts such as http://haskelliseasy.readthedocs.io that try to offer a more opinionated list of packages for various purposes, and I think this would be something that a Haskell site could benefit from - either by linking to the aforementioned site, or by supplying a list itself.
If this is a bit off scope with the project or goals of the project, feel free to close the issue. Just thought I'd add what I've found hard to navigate in my own experience.

I think it'd be better to leave it on haskelliseasy.com or reform haskelliseasy.com in some other way. Part of the point of the site is being based on personal experience or testing I've done with a beginner.
I don't include anything just because somebody asked me to. I don't think it's possible for a committee or group of people to reproduce that selectivity.
I'd call this out of scope for now, but if people are interested in digging into this problem and why I made haskelliseasy.com, I'd be willing to discuss it.

@bitemyapp I'd be interested in learning more of the subtle decisions that lead to the content that is currently presented on the site.

We may be able to break this up into two tiers:

Packages that are almost universally agreed to and ubiquitous: bytestring, text, vector, containers
Much more opinionated packages: conduit vs pipes, CSV or XML parsing libraries, etc. I'd say providing links to websites like haskelliseasy (or hosting opinionated pages on haskell-lang.org) could be appropriate.

Reasoning: let's share common resources that have no real dispute, and make it easy for people to educate themselves on more controversial issues easily.

@snoyberg I think that actually sounds like a great way to handle it.

An initial version of this is now available at: http://ci.haskell-lang.org/packages
I think this issue may be safe to close now, with further additions handled via PRs to https://github.com/haskell-lang/haskell-lang/edit/master/config/package-info.yaml. Any objection?

Merging into #27.


Update Nitrokey App to v1.3

Quick, not tested (yet) update to Nitrokey App v1.3.

After making all changes to the cask:

[ ] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked the cask was not already refused in closed issues.
[ ] Checked the cask is submitted to the correct repo.

Running script.sh...

>>> brew cask _audit_modified_casks 2f6ee57ce4939f47b77df8498f036b1160b62bbf...d042e45ef2baf280d228862580ff65df426dfebc
==> 1 modified cask: nitrokey
==> Downloading https://github.com/Nitrokey/nitrokey-app/releases/download/v1.3/Nitrokey.App.dmg
==> Verifying checksum for Cask nitrokey
audit for nitrokey: warning
 - appcast checkpoint mismatch
Expected: 557a05e9b7a326526bfadda52d670aa59097d2fb2c8ad42bdc1a05f119d19a23
Actual: 4ffd829b3bc2062a7d807bdc34a441f4b37f73b91cb3943e8ffdcfffad0fccc3
Error: audit failed for 1 cask: nitrokey

Running script.sh...

>>> brew cask _audit_modified_casks 2f6ee57ce4939f47b77df8498f036b1160b62bbf...ed03e991d8effed9f7635fae7b28271e0c19cc51
==> 1 modified cask: nitrokey
==> Downloading https://github.com/Nitrokey/nitrokey-app/releases/download/v1.3/Nitrokey.App.dmg
==> Verifying checksum for Cask nitrokey
audit for nitrokey: warning
 - appcast checkpoint mismatch
Expected: 557a05e9b7a326526bfadda52d670aa59097d2fb2c8ad42bdc1a05f119d19a23
Actual: 4ffd829b3bc2062a7d807bdc34a441f4b37f73b91cb3943e8ffdcfffad0fccc3
Error: audit failed for 1 cask: nitrokey

Please update the appcast checkpoint

@TravisBuddy stop

Thank you for correcting!


[terra-form-select] Opening one dropdown in a table will not close another opened dropdown

Bug Report
Description
When a terra-table contains a terra-form-select dropdown on each row, opening one select will not automatically close the previously opened select.
This works as expected on Chrome and Edge; only having the issue in IE.
Steps to Reproduce

Render a terra-table with a terra-form-select on each row.
Open a select on one row.
Open a select on a different row - the previous select remains open.
Open a select on another row - the previous two selects remain open.

Additional Context / Screenshots

Expected Behavior
When one select is opened, close other selects in the same table.
Environment

Component Name and Version: "terra-form-select": "^5.36.0", "terra-table": "^3.26.0",
Browser Name and Version: IE 10
Node/npm Version: [e.g. Node 8/npm 5]  node 10.16 / npm 6.9
Webpack Version: "webpack": "^4.30.0",
Operating System and version (desktop or mobile): Windows 10

This seems to be an issue with single-select variant on IE 10 and IE 11 -

This problem is exposed after https://github.com/cerner/terra-core/issues/2648 was fixed and maybe it was masked by the defect earlier.
At this point, I ain't 💯 sure what is really happening but in the code, I see that we query the DOM for terra-select-menu and compare the result against the currently focussed element in multiple places. I am assuming that when there are multiple select elements on the page, the intended element is not obtained and instead the first element seen in the DOM is obtained and compared against and blur handler (that closes the dropdown) is never actually getting invoked.
Instead of querying the DOM for the menu like that, I made sure that each individual Frame is aware of the Menu that it shows by adding a ref and it seems to have worked.
Is this the correct solution? I am not sure because I do not know why Frame does not maintain a ref its Menu in the current code.

cc / @StephenEsser

Thanks for looking into this @nramamurth. Any chance we could get a fix released within the next week or two? We're looking to release a project consuming this soon, and I anticipate this won't be an uncommon workflow.

@nramamurth
I think you are correct. The setTimeout is allowing two menus with the same ID to be rendered onto the page. When the comparison is made the wrong dom elements are being compared.
Two options come to mind that should resolve this issue.
Option 1: Using your solution, implement a refCallback on the menu and use that as the comparison.
Option 2: Implement a unique ID for each select menu. This file is already using lodash.uniqueid. That solution would look something like this.
I lean towards the first option, if possible, so that we are not concerned about querying the dom if we can define an explicit reference. However, the reason the current code doesn't implement a ref may be related to the aria-controls and aria-owns attributes are expecting an identifier:
https://github.com/cerner/terra-core/blob/master/packages/terra-form-select/src/single/Frame.jsx#L490
https://github.com/cerner/terra-core/blob/master/packages/terra-form-select/src/single/Frame.jsx#L496

@StephenEsser

the reason the current code doesn't implement a ref may be related to the aria-controls and aria-owns attributes are expecting an identifier:

I'd have missed that. Thanks for the heads-up!
How about we do both — have an unique ID for each select menu to support accessibility and also, maintain a ref so that we do not have to parse the DOM often (although I have read that performance difference between these approaches is quite negligible)? Maybe, it looks 'cleaner'.
When I do this, should I update the other variants  as well or it is better that we get to them when we get to flattening issues later at some point?

That approach sounds good to me. I'm not too strongly opinionated about using the ref vs querying the dom. I would implement whichever approach you feel works best.
I'd recommend not updating the other variants to keep the changes small unless they are also experiencing this same issue. It's easy to lose track of the original issue when there's a lot of changes in one PR.


[Error] : gateway  에 Service  에 있는 함수 가져와서 쓰려고 했는데 에러발생..

내가   import  를 못해주는건가 .. ?
더 알아보자 원래 안 되는건지 내가 모르는건지

[Nest] 40535  - 03/18/2023, 2:12:06 PM   ERROR [ExceptionHandler] Nest can't resolve dependencies of the ChannelsService (ChannelsRepository, UserRepository, ChannelMemberRepository, ?). Please make sure that the argument dependency at index [3] is available in the ChannelsModule context.

Potential solutions:
- Is ChannelsModule a valid NestJS module?
- If dependency is a provider, is it part of the current ChannelsModule?
- If dependency is exported from a separate @Module, is that module imported within ChannelsModule?
  @Module({
    imports: [ /* the Module containing dependency */ ]
  })

Error: Nest can't resolve dependencies of the ChannelsService (ChannelsRepository, UserRepository, ChannelMemberRepository, ?). Please make sure that the argument dependency at index [3] is available in the ChannelsModule context.

Potential solutions:
- Is ChannelsModule a valid NestJS module?
- If dependency is a provider, is it part of the current ChannelsModule?
- If dependency is exported from a separate @Module, is that module imported within ChannelsModule?
  @Module({
    imports: [ /* the Module containing dependency */ ]
  })

[Nest] 41682  - 03/18/2023, 2:53:31 PM   ERROR [ExceptionHandler] Classes annotated with @Injectable(), @Catch(), and @Controller() decorators must not appear in the "imports" array of a module.
Please remove "ChannelsService" (including forwarded occurrences, if any) from all of the "imports" arrays.

Scope [AppModule -> ChatsModule -> EventsModule]

Error: Classes annotated with @Injectable(), @Catch(), and @Controller() decorators must not appear in the "imports" array of a module.
Please remove "ChannelsService" (including forwarded occurrences, if any) from all of the "imports" arrays.

Scope [AppModule -> ChatsModule -> EventsModule]

    at DependenciesScanner.insertModule (/Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/scanner.js:76:19)
    at DependenciesScanner.scanForModules (/Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/scanner.js:34:43)
    at DependenciesScanner.scanForModules (/Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/scanner.js:61:43)
    at DependenciesScanner.scanForModules (/Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/scanner.js:61:32)
    at DependenciesScanner.scanForModules (/Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/scanner.js:61:32)
    at DependenciesScanner.scan (/Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/scanner.js:27:9)
    at /Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/nest-factory.js:107:17
    at Function.asyncRun (/Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/errors/exceptions-zone.js:22:13)
    at NestFactoryStatic.initialize (/Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/nest-factory.js:106:13)
    at NestFactoryStatic.create (/Users/jyp/Desktop/programming/42seoul/transcendence/together/backend/node_modules/@nestjs/core/nest-factory.js:42:9)
    ```

forwardRef(() => UsersModule) this keyword  사용 !!  서로 참조하는 양쪽에 다 해줘야 한다 !
https://docs.nestjs.com/fundamentals/circular-dependency


[Marker.io] image used in "getting started with prezzee api" is different

Steps to reproduce:
compare the image  used in "getting  started with prezzee api" with figma
Actual result: they are different
Expected result: please check this once. if done by client , wecan ignore and close.

Reported by: vinod<EMAIL_ADDRESS>- Contact via Marker.io
Source URL: https://wf-b.stg.prezzee.com/en-us/corporate-gifting-solutions/api?marker=true
Issue details: Open in Marker.io
Device typedesktopBrowserChrome <IP_ADDRESS>Screen Size1366 x 768OSWindows 10 (10.0.0)Viewport Size1366 x 641Pixel Ratio@1x

🤖 Marker.io bot - This issue was marked as ✅ Resolved in Marker.io


Forbidden trailing comma in Typings

There's a typo in the type definitions:
ERROR in redux-beacon/index.d.ts
(65,14): error TS1009: Trailing comma not allowed.

Hey @philipptri! Thanks for bringing this up. Trailing commas in function parameter lists are valid with my current Typescript version and tsconfig. May I ask what Typescript version you're using? Either way, I think I'd accept a PR to remove it from Redux Beacon's type definitions if you're up for it.

Hi @ttmarek! Good point, that might be the reason. We are still on 1.4.0 but we will update soon as well.
Thanks!


How to do soft scrubbing (volume censoring using the GLM)?

Describe the bug
XCPEngine gives the option to do either despike or volume censoring. However, despike is done with AFNI and it is not based on FD thresholds, while volume censoring is hard scrubbing (removal of volumes).
In order to only add the censored volumes to the GLM and do soft scrubbing, is there a command for that or should the regressors be added to confound2_custom.
Additional context
Add any other context about the problem here.

Checking the tutorial better, I think you call soft-scrubbing as spike regression and hard-scrubbing as volume censoring. With that in mind, how can we do spike regression only?
For example, on the design file, do we only do confound2_censor[2]=0 and confound2_framewise[2]=fds:0.25,dv:2? Or how to only do spike regression?
Thank you,
Eduardo

thank you @egarza
the despiking is done by AFNI's 3dDespike and not based on FD thresholds. this is done by adding DSP to the regression steps  regress_process[3]=DMT-DSP-TMP-REG
For the censoring, censoring are done by flagging volume with parameters(fd and dv) greater than the threshold. confound2_framewise[2]=fds:0.25,dv:2.  and  confound2_censor[2]=1 for spike regression.
Scrubbing also includes masking out the non-contiguous segments of data between outliers. The number of contiguous volumes required to survive masking is set flexibly by censor_contig[2]=X, where X is the minimum number of contiguous threshold-surviving time points required for those time points to survive masking.
if you identify volume with spike rather than using threshold, you can add it with confound2_custom[2].  It is tricky because you have to create a matrix. for instance if bold has total 10 volumes and volume 4 and 6 are flagged, the matrix will be 10 rows by 2 volumes flagged and volume 4 and 6 are labeled 1s in the  column separately ::
0 0
0 0
0 0
1 0
0 0
0 1
0 0
0 0
0 0
0 0

Hi Azeez,
Thanks for the quick answer. But what if I want the flagging of volumes with FD and DV, but I do not want the volume to be removed, I only want the volume to be regressed as a confound? Because right now my subjects end up with less volumes. Thank you
Eduardo

yes, there is full residualized  data in the regress folder regress/*_uncensored.nii.gz

I think we have a good idea of it, thank you


Rename repository to reflect wider scope than just observer API

As discussed in this thread (https://github.com/opentracing-contrib/java-observer/pull/3#issuecomment-312227092) we should rename this repo to something more appropriate to cover observer and filter API.
java-patterns - too generic
java-tracer-decorator and java-tracer-wrapper - I think these are too implementation specific, and focus just on the wrapping/decorator rather than on the APIs
java-api-extensions and java-tracer-extensions - these are in the right direction, wonder if a truncated form would work - e.g. java-ext

The most I like is
java-api-extensions. java-ext is not story telling at the first glance I don't know what is there.

why not simply use different repos?

@yurishkuro Was thinking that the filter/interceptor API would also use the SpanData to enable more interesting possibilities.

It's it worth creating intermodule dependencies because of one pojo?

They could all be in one module if preferred?

@pavolloffay @felixbarny @yurishkuro As part of this issue, would like to do the following:

[ ] Rename the repo to java-api-extensions (as proposed by Pavol)
[ ] Artifact id changed to opentracing-api-extensions
[ ] Place interfaces into package io.opentracing.contrib.api

Any objections?


"modern" mode

Inspired by Meteor 1.7 and the fact that you can easily ship ES2015 code and legacy code side-by-side, we can enable a mode where Vue CLI produces two bundles: one for legacy browsers using the specified browserslist values, and one that targets only browsers that support <script type="module">.

babel-preset-env supports targets: { esmodules: true } which allows us to easily target these modern browsers.
We can likely extract the corresponding browsers list for autoprefixer as well.

This should allow Vue CLI users to ship a significantly smaller, and decently more performant bundle to a large percentage of end users. In Vue 2.6 we can also add a build of Vue that ships native ES2015, which can be used in this mode.

That would be awesome! 😻

Closed via 204d8f07 and 0b42f2f7
Now this can be done simply with vue-cli-service build --modern.

The will definitely push the boundary!
Last year we did the experiment in one of our project with average UV of ten thousands. It's been great, along with lots of pitfalls though. By the time AFAIK, no other companies in China had ever thought of doing so. Later we pushed the solution to our most important project. And it went well since then. Currently we are planning a bigger move. Possibilities are out there.

This will definitely push the boundary!
Last year we did the experiment in one of our project with average UV of ten thousands. It's been great, along with lots of pitfalls though. By the time AFAIK, no other companies in China had ever thought of doing so. Later we pushed the solution to our most important project. And it went well since then. Currently we are planning a bigger move. Possibilities are out there.

@myst729 please test and review the feature for any particular pitfalls you have run into!

This is great and solves me a lot of headaches!
I noticed one downside is though that legacy browser will load the legacy files AND the modern files. Shouldn't they just load the legacy files?

@yyx990803 Is it possible to build only on es2015/modern mode? Without build 2x times.. one for legacy and another for modern?
with angular-cli@8, it is possible changing browserslist file.
I changed browserlistrc to list only moderns browsers, but don't work.
.browserslistrc:
> 4%
last 2 Chrome versions
last 2 Firefox versions
last 2 Safari versions

but don't work.

Insofar as...?

but don't work.

Insofar as...?

continued to generate two separate builds, modern and legacy.

Oh, so you changed the browserslist and till applied the --modern flag?
changing the browserslist to cover only modern browsers will make webpack build a bundle that contains only modern mode without the --modern flag.
And any additional "trickery" that "modern mode" does, i.e. changing the modern bundle's <script> elements to type="module" are not necessary as that's just a way to make the two bundles work side-by-side - which you don't want or need.

I'd still like to be able to trigger them separately without having to adjust browserslist. The main reason is my CI/CD deployment container sometimes runs out of allowed memory for my plan, trying to do both at once. If I could split them out into 2 separate actions I could avoid this.

changing the browserslist to cover only modern browsers will make webpack build a bundle that contains only modern mode without the --modern flag.

Can you please give example of corect  syntax to be put in browserslist, for whoever are not familiar with the meaning of i.e.:
> 1%
last 2 versions
not ie <= 8

@ttodua it's been a while since your question, but here is the extensive explanation: https://github.com/browserslist/browserslist#browserslistrc
TL;DR: each row is connected with an OR, therefore, "build for every browser version which has >1% market share OR the last 2 versions of a browser OR is not IE version <= 8.
I changed my file to be
> 1% and last 2 versions and not dead

This results in Vue CLI building only for modern browsers.


YARN-1903

Patch TestNMClient test
Handle separately Container transition NEW -> DONE upon KILL_CONTAINER
Add two more tests for killing a container

Can one of the admins verify this patch?


E2E: Update post list content ID

Summary
Update a ID for virtualized post list content to not collide with one we use for cypress e2e testinge.
Ticket Link
N/A

@sudheerDev Updating this ID, any impact for scrolling things if we do so?

@sudheerDev Updating this ID, any impact for scrolling things if we do so?

Nope all good.

@thekiiingbob We have the same ID postListContent   for IE11 in pist_list_ie.jsx but we are fine for now and we will be deleting that file for 5.16v as well.

@sudheerDev yeah, not an issue for Cypress as we can only run in chrome. Thanks!


VERBOSITY environment variable does not work as expected

What happened:
The hack/manifests.sh script uses the VERBOSITY environment and passes it as an argument to ./hack/build-manifests.sh, which in turn creates manifests like Kubevirt CR.
However, setting this environment variable does not change the generated CR in any way.
What you expected to happen:
When doing:
> export VERBOSITY=10
> make cluster-up && make cluster-sync
> # let it finish...
> k get -n kubevirt kubevirt kubevirt -o yaml

I expect to see all components (e.g. virt-handler, virt-launcher, etc) defined with the defined verbosity.
How to reproduce it (as minimally and precisely as possible):
Look at the previous point above.
Additional thoughts:
Since the variable does not work anyways, I think we can safely "break" backwards compatibility and change the variable name to KUBEVIRT_VERBOSITY for clearance.
Environment:

KubeVirt version (use virtctl version): v1.0.0

/label good-first-issue

@iholder101 can i work on this issue ? i need some time to get familiar with codebase and contrubuting guidelines .

Yes indeed @shubmjagtap! Thanks for reaching out to help!
Take your time and please let me know if you need any assistance.
/assign @shubmjagtap

/attempted

okay if  @Aryan8912 has attempted then let him work on this issue , i will unassign myself ( anyways im busy with other work )

is the issue is still on ? so that I can work on it ...

Ya work on it, I am busy other work

okay

@iholder101  yes maybe you are right , Iet me apply it and see the outcomes . Maybe it will resolve the conflicts .

/assign

/remove-lifecycle stale

/remove-lifecycle rotten

/remove-lifecycle stale


Can't launch Spyder through anaconda after trying to install jupyter package

Issue Report Checklist

[x] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Tried to install jupyter package using "conda install spyder-notebook -c conda-forge" and now cannot open spyder
What steps reproduce the problem?

Launching anaconda
Trying to launch spyder

What is the expected output? What do you see instead?
Spyder just does not open get error code
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE
Traceback (most recent call last):
File "/Users/anaconda3/bin/spyder", line 7, in
from spyder.app.start import main
File "/Users/anaconda3/lib/python3.12/site-packages/spyder/app/start.py", line 25, in
import ctypes
File "/Users/anaconda3/lib/python3.12/ctypes/__init__.py", line 8, in
from _ctypes import Union, Structure, Array
ImportError: dlopen(/Users/anaconda3/lib/python3.12/lib-dynload/_ctypes.cpython-312-darwin.so, 0x0002): symbol not found in flat namespace '_CFBundleGetVersionNumber'

Versions

Spyder version: 5.51
Python version:
Qt version:
PyQt version:
Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

python version:  3.12.4
qt version: qt-main                   5.15.2
pyqt version: 2.4.1
operating system: os

You are attempting to install a conda-forge (community channel) package into the Anaconda base Python environment which uses the anaconda (commercial channel).
The commercial channel is a limited set of packages, repackaged by Anaconda for use in their Anaconda base Python environment. The Anaconda base Python environment should generally be used "as is" with limited updates to the conda package manager and anaconda-navigator. Installation of community channel packages into the base Python environment, generally results in instabilities.
You should reinstall Anaconda and create a new Python environment (subinstallation of Python), with spyder-notebooks and the optional dependencies (seaborn installs, numpy, pandas and matplotlib, pyqt is needed for the interactive qt backend for matplotlib plots and the other dependencies are needed for pandas to read/write to various formats):
conda create -n spyder-env -c conda-forge python spyder spyder-notebook cython seaborn pyarrow sympy openpyxl xlrd xlsxwriter lxml sqlalchemy tabulate pyqt

Closing due to lack of response.


Alternatives to MIT

The post Deprecation Notice: MIT and BSD it's time to retire thirty-year-old academic licenses (WayBack Machine link) has a few good points as to why new software might want to avoid licenses like MIT or BSD, and provides an alternative. It might be a good idea to encourage alternatives to MIT or, at least, make note of them more prominently.

A first step for additional lax permissive licenses pertinent to this repo would be https://github.com/github/choosealicense.com/blob/gh-pages/CONTRIBUTING.md#adding-a-license
Feel fee to open individual issues or PRs for such licenses.


Request to regenerate the bisheng non-smoke tests

Summary:
Please regenerate all of the bisheng non-smoke tests in order to include a fix that allows them to run.
Note: We must verify that the resultant jobs have the default JDK_IMPL value set to "hotspot", even though the JDK_IMPL value in TEST_JOB_AUTO_GEN is set to "bisheng" (which I believe to be necessary so we  regenerate the right jobs).
Test Sets:
LEVELS: sanity,extended
GROUPS: openjdk,perf,system
JDK_VERSIONS: 8,11
JDK_IMPL: bisheng
ARCH_OS_LIST: aarch64_linux,x86-64_linux
and
LEVELS: sanity,extended,special
GROUPS: functional
JDK_VERSIONS: 8,11
JDK_IMPL: bisheng
ARCH_OS_LIST: aarch64_linux,x86-64_linux
Details:
In this issue, we discussed how setting the JDK_IMPL value to "bisheng" causes the smoke tests to fail before they can run, and how this value should instead be "hotspot".
This change was effected by creating a PR that sets the JDK_IMPL value correctly during test job generation, followed by a deletion and re-creation of the aforementioned smoke tests.
However, this was all done without considering the non-smoke tests, which are now failing.
@smlambert - Asking for your thoughts on this, please.

Jobs listed above regenerated via:
https://ci.adoptopenjdk.net/view/Test_grinder/job/Test_Job_Auto_Gen/264/
https://ci.adoptopenjdk.net/view/Test_grinder/job/Test_Job_Auto_Gen/265/ - openjdk regenerated separately to keep report dir equals true as default (when tests pass), unlike other test groups
https://ci.adoptopenjdk.net/view/Test_grinder/job/Test_Job_Auto_Gen/266/

Thanks Shelley. :)


Revert "Update exclusions"

Reverts hmcts/civil-ccd-definition#1447

Reverting commit as the permissions on the file have changed


Access 97

Please, can you tell me if it is possible to connect with Acces97 (16bits).
Thank you.

Yes you can.
Provider=Microsoft.Jet.OLEDB.4.0;Data Source=\xx1\db\xxx.mdb


🌐 Add Korean translation for Tutorial - Extra Models

Hello
I translated the Extra Models part of the tutorial.
Please review if there is anything wrong. thanks.
This PR translates tutorial/extra-models.md in Korean.
related: #2017

@hard-coders
Thank you for your review.
I accepted your points and pushed the new commit.


chore: Fetch all tags in deploy pipeline

Description

Fetch all tags in deploy pipeline

Checklist

[x] My code follows the existing style, code structure, and naming taxonomy
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have performed a self-review of my own code and included any verifying manual calculations
[x] I have added or updated unit tests that prove my fix is effective or that my feature works, and achieves sufficient code coverage.  New and existing unit tests pass locally and in the build (below) with my changes
[x] My changes generate no new warnings and running code analysis does not produce any issues
[x] I have added or run the performance tests that depict optimal execution times
[x] I have made corresponding changes to the documentation

nice.  that worked


UsernamelessExample login fails on Edge and Firefox

Windows 1903, Edge v44, Firefox 67
Register works, Login fails:
Edge error message: FAILTypeError: Cannot convert null or undefined to object
Firefox error message FAILTypeError: getAssert.allowCredentials is undefined

+1 not work in Edge for Mac Version <IP_ADDRESS> (Official build) Dev (64-bit)

@hrchu Fixed! Sorry for the long wait!


Sass preprocessor

This is a first preview which is able to compile *.scss files into *.css files using the preprocessor option added in version 0.8.1.
It can be used to solve https://github.com/divio/django-cms/issues/3600 by offering offline compilation of SCSS files, without requiring Ruby anymore.
To use it, simple add the new preprocessor compilescss to Sekizai's addtoblock templatetag:
{% addtoblock "css" preprocessor "sekizai.processors.sass_preprocessor.compilescss" %}<link href="{% static 'myapp/css/mystyle.scss' %}" rel="stylesheet" type="text/css" />{% endaddtoblock %}

Offline Compilation
By invoking ./manage.py compilescss Sekizai iterates over all templates and converts these SCSS files into CSS files. They are store side by side with their SCSS counterpart, which is great in combination with sourcemaps.
No SASS compiler is required on production systems!
This PR is prove of concept
It borrows quite a lot of code from django-compressor, therefore the latter is an additional requirement. If desired, I could integrate these classes from Compressor into Sekizai, in order to get rid of these additional dependencies.
I also would like to ask if this feature makes sense at all inside Sekizai. Otherwise I'd have to create another Django app, which offers this functionality separately. But since SASS is so popular in Django projects, I prefer to keep it inside the Sekizai project. Nobody is forced to use this preprocessor, so there is no additional bloat.
Please test this PR and consider it as a starting point for further discussions.

Hi, this looks like a great new feature, but as mentioned in #40, I do not want an actual implementation of pre-/post-processors in django-sekizai. These things should live in 3rd party applications (a new one or an existing one). As this patch does not rely on changes in django-sekizai, This should be very easy to do.


Forge caching issues

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (05e72b6 2022-05-19T00:03:30.647862736Z)
What command(s) is the bug in?
forge build
Operating System
Linux
Describe the bug
This is a consolidation of bugs that are suspected to be cache related:

https://github.com/foundry-rs/foundry/issues/1629
https://github.com/foundry-rs/foundry/issues/1344

Issue 1

[ ] Check if solved

Bug: Sometimes forge test does not run any test files.
Workaround: You can run forge clean before forge test
Issue 2

[ ] Check if solved

Bug: Sometimes a file is changed and that edit is not picked up by forge test, which will then not recompile that contract. There is no reliable way to reproduce this currently.

"issue 2" happened to me as well.
forge 0.2.0 (4fcd7e0 2022-05-26T00:08:37.693306Z)
I have two test files, and the ignored one is a collection of failing specs that use expectRevert()
A change made for the other test file is immediately detected and can be tested.

from tg

I can’t remember/figure out exactly when it happens but often I’d notice that only changed/recompiled test files are being run

it sounds like we're not getting all Tests on recompile
will try to pin this down now

encounter an issue probably related to this
2-line change from master branch: https://github.com/foundry-rs/foundry/compare/master...0xYYY:no-tests

add an invalid line in testdata/cheats/Cheats.sol, should cause compilation failure.
add a check in forge/src/multi_runner.rs test test_cheats: assert!(suite_result.len() > 0);
run cargo test test_cheats will fail with

     Running unittests (target/debug/deps/forge-b71ceab7b7a80d10)

running 1 test
test multi_runner::tests::test_cheats ... FAILED

failures:

---- multi_runner::tests::test_cheats stdout ----
thread 'multi_runner::tests::test_cheats' panicked at 'assertion failed: suite_result.len() > 0', forge/src/multi_runner.rs:1018:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    multi_runner::tests::test_cheats

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 6 filtered out; finished in 2.70s

error: test failed, to rerun pass '-p forge --lib'

remove the line in testdata/cheats/Cheats.sol and run cargo test test_cheats again, all tests pass

We haven't had caching issues in a long time, so I'm assuming this is resolved for now :smile:

I still met this issue with latest version:
% forge --version
forge 0.2.0 (1710187 2024-08-20T00:18:53.474556000Z)

But it's hard to reproduce, as it's gone after a manual forge clean.


Can we set metadata for HTTP with an environment variable or a static file

I've known how to set metadata for a HTTP so that we can select endpoint with the metadata:
...
route_config:
  name:
  virtual_hosts:
  metadata:
    filter_metadata:
      "envoy.lb":
         canary: true

In the example above, we can use canary: true to select endpoint in cluster.
My question is if it is possible to get the value of canary from an environment variable of OS or from a static file?
For example, after executing export var=true or there is a file containing var=true, I can configure my envoy as below:
...
route_config:
  name:
  virtual_hosts:
  metadata:
    filter_metadata:
    - "envoy.lb":
         canary: $var    # $var coming from an environment variable or a static file, Is this possible???

This is not supported.

@yanavlasov  I thought https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/matching/common_inputs/environment_variable/v3/input.proto could help me on this. Could you please show me how to use the common_inputs.environment_variable?


Implement email address verification in identity service

@tyleryasaka et al: from the UX perspective, email address verification usually involves receiving an email that contains a link, clicking that link, and the router doing something with a token contained in the URL. I assume that the plan is as follows:

DApp sends request to bridge server for a verification email to be generated
bridge server generates email
user clicks link in email
DApp* parses token
DApp* posts token to bridge server and stores the response in React state

Note that in step 4, this* is almost certainly a new and different tab/window than the one that initiated the verification. Assuming that the user may already have other unpublished identities or profile attributes in React state, they will be sitting with two different "sessions" that cannot be combined for a batched publication. So at a minimum, two transactions will be required to publish the changes. And most likely the user will abandon one of the two or get confused/frustrated.
Without local storage, how do we handle this email verification case?

@micahalcorn One solution is to just let the user copy/paste the code without ever having to navigate away from the page. The other solution that comes to mind is exactly what you mentioned, using local storage.
Either way, this sounds like a demo-dapp UI issue more than a bridge-server issue. Might be good to move this discussion over there.

Finished by https://github.com/OriginProtocol/bridge-server/pull/28


Fold changes from trak into the GitHub repository

Closes #10
Includes a number of new features. I have tried to keep things as close to TrakCare's code as possible. Notable differences:

All our configuration happens in the Utils method whereas theirs is done through the CodeTidy class.
They use %System.SQL.ShowPlan() to get the SQL query execution plan. That method is deprecated, so I have used %System.SQL.Explain() instead.
Method names are different between both code bases but the method bodies wherever possible are identical.

I have tested all the different configuration options and it all seems to work. I have committed the class I used for manual testing. This needs to be removed before merging with main.
I might have missed some bits of code, so please review this in detail if possible.

@isc-svelury the number one thing that would give me improved confidence in this change is a good set of regression tests. This is substantial enough that I kind of think we should put them in before merging, ideally with a passing set of tests before this change that also passes afterward.

Added unit tests for all configurables (and all options within) except the ones for excluding/including files and specific globals. I think they would be better suited for manual testing.
ReferenceClasses.cls has a method for each configurable. Within the method, each option possible option for the item is tested. The files used for testing are in before/ and the expected results are in after/. Generally, the files in after/ have the same name as the file in before/ with a suffix to indicate the option it corresponds to.
All these methods have a Boolean parameter DeleteCompFiles which is set to 1 by default. If set to 0, the comparison files generated will not be deleted. This is useful when debugging specific tests.


Update PyYaml and pydantic

[ x ] I have checked that this bug has not yet been reported
[ x ] I have checked that this bug exists in the latest version of pacsanini

Describe the bug
Dependabot alerts have been emitted for PyYaml (critical severity) and Pydantic (low severity). Although the CVEs don't relate to the specific ways that the dependencies are used in the project, the dependencies should still be updated to be safe.
Code To Reproduce
poetry add "pydantic>=1.7.4"
poetry add "PyYAML>=5.4"

Expected behavior
This should have no impact on the code. This is a just a question of patching some of the dependencies.
Additional context
CVEs:

pydantic: https://github.com/advisories/GHSA-5jqp-qgf6-3pvh
pyyaml: https://github.com/advisories/GHSA-8q59-q68h-6hv4

In addition to updating the versions of PyYAML and pydantic, the following changes should take place:

remove the requirements*.txt files: this turns out to be unnecessary because for development only poetry should be supported to install dependencies and when installing the actual project pip will understand the pyproject.toml syntax. Only the requirements_doc.txt file will be kept because it is the easiest (although not the cleanest) way for the readthedocs server to install the dependencies properly.
remove the setup_env.sh and export_requirements.sh scripts. The latter is because the requirements*.txt files will be removed. The former is because poetry should be used and this adds an "obscure" way to install the project.