Add Double Take snapshots to HA Media Browser [FEAT]

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
You can access snapshot and video clips from Frigate through the HA Media Brower, it would be good if the Double Take snapshots could be accessed in the same way.
Another option would be a configuration setting to not save an image and instead just link to the image that should already be saved by Frigate. That however might complicate matters if Frigate is not installed on the same box as HA.
Additional context
This feature request is specifically for the latest version of DT which is installed as an HA Add On.

I have been playing with this some more and worked out what is happening.
When I click on media browser the top level folder (at least in my setup) are: 'Local Media', 'Netatmo' and 'Frigate'. When I drop down on the 'Local Media' folder I then see 'Double-take' and 'Frigate'.
I'm guessing this is working as expected, just strange Frigate is showing at the top level. Happy for this feature request to be closed given it looks like I just misunderstood what to expect.

Is there a chance an old version of Frigate was writing to the top level frigate folder?
I looked to see where the current Frigate add-on writes to and it appears to be /media/frigate and Double Take should be under /media/double-take/ as well.

My setup involves 2 different computers, one for HA and another one for Frigate/DoubleTake.
I use the Frigate proxy add-on to get access to the Frigate recordings under the Media browser in HA but the Double Take add-on does not add itself to the Media browser.
How can I add Double Take snapshots to the Media browser when using the Double Take Proxy add-on?
Thanks.

My setup involves 2 different computers, one for HA and another one for Frigate/DoubleTake.
I use the Frigate proxy add-on to get access to the Frigate recordings under the Media browser in HA but the Double Take add-on does not add itself to the Media browser.
How can I add Double Take snapshots to the Media browser when using the Double Take Proxy add-on?
Thanks.

Did you find any solution for this?


fix: Key host_name on website routing rules redirect

Description

Change the hostname key referenced by the redirect configuration on website routing rules from hostname to host_name.
Motivation and Context

The AWS provider attribute name is host_name, and it's also used as host_name in the complete example.
Breaking Changes

No
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

Thank you for this fix!

This PR is included in version 3.1.1 :tada:


add allowed_operators property in QdrantTranslator

Description:
This PR adds allowd_operators property to QdrantTranslator to fix the TypeError: can only join an iterable bug. This property is required in get_query_constructor_prompt in query_constructor\base.py:

allowed_operators=" | ".join(allowed_operators),

Issue:
#12061

thanks @xieqihui! i think the general fix for this would look something like #12349. Would love if you try and see if that fixes for you.
I think explicitly specifying operators is probably a good thing to do anyway though so i'm happy to land this PR as well


qtwebengine: Rename jasoncpp to jsoncpp

jasoncpp is not the name of the package included in meta-oe.  The correct
naming is jsoncpp.
Signed-off-by: Michael Davis<EMAIL_ADDRESS>

In master-next now, thanks.

In master as well now. Thanks


TypeError:  PerovskiteBuilder.generate_homogeneous_layer_charge_1() takes 8 positional arguments but 9 were given

With the minimal script pasted below, I'm running into the following error:
TypeError: PerovskiteBuilder.generate_homogeneous_layer_charge_1() takes 8 positional arguments but 9 were given
from LDHPbuilder.perovskite_builder import OrganicMolecule, InorganicMonolayer, PerovskiteBuilder
import numpy as np
import ase.io
from ase.io import read
monolayer = InorganicMonolayer.from_species_specification('Pb', 'I', 2)
print(monolayer)
new = read('alanine.xyz')
make the OrganicMolecule object
molecule = OrganicMolecule(
new,  # ase Atoms
1  # float
)
print(molecule)
make samples
pb = PerovskiteBuilder()
np.random.seed(1267)
samples = pb.generate_homogeneous_perovskite_samples(  # homogeneous since just one kind of molecule
monolayer,
molecule,
num_layers=2,  # should be a power of 2, but realistically 4 is already too big
num_samples=10,
max_num_attempts=6500,
# this can be either 'total_thickness' or 'half_thickness'. use half_thickness for long thin +1 molecules
stacking_method='total_thickness'
)
ase.io.write('samples_PbI.xyz', samples)
alanine.xyz file:
14
N     -0.594617   -1.101133    0.630510
C     -0.307489   -0.099434   -0.354650
C     -0.912135    1.238368   -0.022518
C      1.172908    0.015627   -0.596349
O      1.611156    0.846169   -1.430894
O      2.096587   -0.763633    0.068094
H     -0.020805   -1.952583    0.372780
H     -0.462514   -0.799137    1.601341
H     -1.600711   -1.406649    0.508749
H     -0.736620   -0.416212   -1.329365
H     -1.056710    1.388205    1.072018
H     -1.919878    1.378600   -0.474934
H     -0.232937    2.013062   -0.451274
H      2.963765   -0.341251    0.406494

just spotted the error - I will make a fix and then run some test today to make sure the system still performs the same on the benchmarks

This now done - let me know if it works


"sh: Syntax error: bad fd number" during build

On current master-next building on Ubuntu 16.04 LTS ppc64el
    LINKER     hbirt_test.bin
    LINKER     hbicore.bin
    LINKER     hbicore_test.bin
sh: 1: Syntax error: Bad fd number
sh: 1: Syntax error: Bad fd number
sh: 1: Syntax error: Bad fd number
sh: 1: Syntax error: Bad f    LINKER     hbirt_test.bin
    LINKER     hbicore.bin
    LINKER     hbicore_test.bin
sh: 1: Syntax error: Bad fd number
sh: 1: Syntax error: Bad fd number
sh: 1: Syntax error: Bad fd number
sh: 1: Syntax error: Bad fd number
cd build/buildpnor/ && /usr/bin/make --no-print-directory --no-builtin-rules --n
o-builtin-variables buildpnor
make[6]: Nothing to be done for 'buildpnor'.
true
../src/build/trace/tracehash_hb.pl -c -d ../obj -s ../img/hbotStringFile
    GENLIST    hbicore
    GENLIST    hbicore_test
    GENLIST    hbirt_test
    MAKE       kernel IMAGE
d number
cd build/buildpnor/ && /usr/bin/make --no-print-directory --no-builtin-rules --n
o-builtin-variables buildpnor
make[6]: Nothing to be done for 'buildpnor'.
true
../src/build/trace/tracehash_hb.pl -c -d ../obj -s ../img/hbotStringFile
    GENLIST    hbicore
    GENLIST    hbicore_test
    GENLIST    hbirt_test
    MAKE       kernel IMAGE

@stewart-ibm -- Clearly we never got to this (or at least to clearing out this error). Anything still need to be done here?


Can you consider adding a freehand canvas drawing feature in next version?

It would be a unique feature to add a freehand drawing component inside suneditor. It would be even better to have option to save that drawing to server database and get the url of imge in suneditor.

If you have a canvas tool, please let me know.
It will be applied after version 3.0.0.


Any possibility to use Segment Anything Model (SAM) for annotation class name based on color palette in the original images.

Any possibility to use Segment Anything Model (SAM) for annotation class name based on color palette in the original images.
    0: (202,150,150), # Class 1
    1: (198,186,100), # Class 2
    2: (167,183,186), # Class 3
    3: (255,255,133), # Class 4
    4: (192,192,206), # Class 5
    5: (32,80,160), # Class 6
    6: (193,134,1) # Class 7 etc.

Hello, SAM predicts class-agnostic masks, so the mask colors in the visualizations+code we provide are not meant to indicate object classes. Please see these other issues, which may be helpful: #155 #234 #49


Conditional logging of "Server"

Each time a rule triggers and is logged, it generates a line in the audit log with the "Server".
This wastes a lot of disk space and is not interesting if you have a good configuration management.
We should have a directive (or a compile flag) to disable that logging.
Ex. of compile flag: #marcstern@9e68805

I think there are two areas to consider here - we already allow configuration of which audit areas you want to store. I will say however that i see your point. I think a compile time flag might be a pretty good solution ... how would this apply to JSON?

A configuration directive (or option) is obviously more flexible, but I will obviously be OK with a compile time flag

In general I am going to highlight this as a feature request for libmodsecurity. The capability to determine the platform you are on deterministicly is useful.
An example would be SecRule PLATFORM "@contains IIS" XYZ"


[Feature Request] Update Native Select

Dear Evergreen-Team,
i really like the Components from Evergreen UI, but honestly, I don't like the design of the native select. In my view, this native element does not fit with the other components of the framework. I would like to bring the Design from the SelectMenu to the native Select. I would also create an option to put icons in front of a label, e.g. as Prop.

Hey @ITforHome, thanks for creating this issue! I believe there is still room for a NativeSelect alongside a themed on. In the design system we use the following design currently:

Although low-priority for us, I will keep this open!


s/pkgs.system/pkgs.hostPlatform.system/g

Describe your PR, what does it fix/add?
I have aliases disabled ever since a user of one of my repos sent a PR for it... so now I'm paying it forward I guess.
Thanks for hyprland... I think... still getting the initial config going.

@fufexan

Thanks for this.
Mind explaining what the benefit of using hostPlatform is? It's not directly clear to me.

pkgs.system is wrong in some scenarios, like cross-compiling. Also, is just doesn't exist for me, since I have these aliases disabled (forces me to notice things like this, keep up with tech rot/debt, etc).
In this case, when I cross-compile pkgs.system is going to be x86_64-linux, but pkgs.hostPlatform.system is going to be riscv64-linux, etc.

Ah, okay. That makes sense. Thanks for the explanation.

Btw prev.system exists in the overlay inside flake.nix. Shouldn't that be changed as well?

Okay, I tested, you're right, it was necessary if I forced the use of the overlay with nixpkgs.config.allowAliases = false. Tested with the change, it fixes eval. Should be good to go now.

Thank you!


WIP: Add Black to projecct

Adds black to the project as the code editor. Runs black on commit and uses it as a linter for PRs and pushes.

Waiting until the other 2 PRs are ready to merge before merging this one (this will require a reformat of all code)


ActiveMQ Out of Memory error in 2G limit container

Container is given 2G memory limit.
Environment: OpenShift Origin v1.3.1
How to reproduce:
oc create -f all-in-one-template.yaml
oc new-app -t hawkular-full

all-in-one-template.yaml.txt
hawkular-oom.log.txt

What version of services is that? And with what datastore?

0.23.0.Final-SNAPSHOT (commit 4250d21589cf10596ad4fad77cae54ca5a0468a0) with separate C*node container
Still a problem after MEM limit was raised to 4G
# docker stats e121042fb7af
CONTAINER           CPU %               MEM USAGE / LIMIT     MEM %               NET I/O               BLOCK I/O
e0c25e9b4143        0.53%               1.289 GB / 4.295 GB   30.01%              26.99 kB / 28.87 kB   486.4 kB / 28.89 MB

I'm having VPN issue so don't  have access to OpenShift Enterprise 3 to confirm.

This looks like the same issue I've seen before using a Fedora 24 host.
https://issues.jboss.org/browse/HAWKULAR-1131
I didn't see the issue when using the same container on a RHEL 7 or Fedora 23 host.

I can confirm that it's not a problem on OSE 3.2/RHEL 7.2 node
OSE v3.2 node

docker-common-1.10.3-44.el7.x86_64
RHEL 7.2

Origin node

v1.3 docker-common-1.10.3-54.gite03ddb8.fc24.x86_64
Fedora 24


Is it possible to keep the content container having its border radius when the screen is scroll down

I have the same problem

same issue here

Can you send me a screenshot ? I am not sure to understand your issue

@Nhacsam

Keeping the border-radius at the top left & right edge of the border while the screen was been scrolled down.

I've solved this by adding a negative top margin identical to my border radius to my main content view.

I've solved this by adding a negative top margin identical to my border radius to my main content view.

Can you please show an example of how you have applied this?
Thank you


Fix waveform and iOS compilation issue

Description
fix #129
fix #128
fix #130
Type of Change

When changing the frequency of waveform sounds using the flutter_soloud package, after a while, some background frequencies are heard along with the original frequency.

The waveform generation (on C+ side) used float numbers, but higher precision was needed to compute the waveform. Changing to doubles fixed the issue!
Minor changes for the other 2 issues that fix compiling on iOS.

[ ] ✨ New feature (non-breaking change which adds functionality)
[x] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[x] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

[nit] I think raising deployment target is definitely a breaking change. In the future, we should consider only doing it between major semver updates. This change broke the codelab and therefore might have broken others' code, too.
(Just to be clear, I'm glad these fixes landed regardless, and having the iOS version at 13 is probably good future-proof-wise. Just a heads up for the next time we need to change deployment targets.)

Sorry, I had to think of that! 🫣


Keyboard shortcut

I love this extension. Great work!
VSC on Mac has the default shortcut cmd + B to view/hide the sidebar. I changed it to ctrl + B.

ctrl + b has a default too, can't win really. Glad you found something that works for you.


VS Code asks the user to add input.json parameters for action that has no input arguments

Target components
action_server
What is wrong
The error is visible in VS Code extension, but I assume the validation logic comes from Action Server
Repro:

Create new action in VS Code that has no input arguments
Run the Action from the VS Code sidebar

Result: VS Code asks user to config input parameter in devdata/input.json.
Expected: No input args so query is misplaced
System info
Robocorp Code extension v1.19.1
Example script
No response

This issue has been moved here, under Sema4AI/actions repository.


chore: replace IATP by DCP and add OID4VC

What this PR changes/adds

Update reference to DCP instead of IATP
Add reference to OID4VC

Why it does that

Update to new name
The DSP specification is agnostic from the credential exchange protocol and should be open to other existing protocols

According to the PR etiquette, we should discuss this topic first and an issue or a discussion.

Hi @jimmarino,
Your message is raising the concern about the creation of a mandatory dependency between DSP and DCP. I don't recall that such a decision was voted in the EDWG.
Could you point to the normative parts of DSP that:

mandate that only DCP can be used
only non-human interaction are eligible for DSP.

?
An issue was opened here https://github.com/eclipse-dataspace-protocol-base/DataspaceProtocol/issues/65
Best regards,
Pierre

Hi @jimmarino,
Your message is raising the concern about the creation of a mandatory dependency between DSP and DCP. I don't recall that such a decision was voted in the EDWG.
Could you point to the normative parts of DSP that:

mandate that only DCP can be used
only non-human interaction are eligible for DSP.

?
An issue was opened here #65
Best regards, Pierre

I did not say that only DCP can be used. Please read my comments again. People are free to use whatever extensions they want. However, if they do not adhere to one of the baseline specifications, their solution will not be interoperable.
Regarding human interactions, please read the DSP specification, particularly the Dataspace Information Model. There is no concept of a human. This is a fundamental DSP architectural principle and a key to thoroughly understanding what a data space is.
Of course humans (End-Users) do exist, press buttons, request data, and potentially approve data transfers. However, these interactions are not reflected in DSP or the data space.

Now that we've established that DCP is not the only possible protocol, DCP is not mandatory either, because DCP is a claim protocol and not a base protocol.
And then, indeed, the Dataspace Information Model is agnostic from the type of participant, human or machine.
Which makes both DCP and OpenID4VC legit protocols to exchange credentials.

Now that we've established that DCP is not the only possible protocol, DCP is not mandatory either, because DCP is a claim protocol and not a base protocol.
And then, indeed, the Dataspace Information Model is agnostic from the type of participant, human or machine. Which makes both DCP and OpenID4VC legit protocols to exchange credentials.

Again, please read my comments carefully, as I did not say that. DCP is the protocol for decentralized identity in the EDWG. DSP has no concept of human interactions and they do not enter into the specification.
While you are free to argue your point, you must abide by a committer's decision. Unless you remove the reference to OpenID, your PR will be rejected and closed.

Hi @ticapix ,
nice catch. this needs to be changed following the ISO requirements. Overtaken by event and fixed in #58

Closing this PR, fixed in #58


Kafka Connector Connector Editor does not enable the "Submit" button when a value is pasted

Issue submitter TODO list

[X] I've looked up my issue in FAQ
[X] I've searched for an already existing issues here
[X] I've tried running main-labeled docker image and the issue still persists there
[X] I'm running a supported version of the application which is listed here

Describe the bug (actual behavior)
Kafka Connector Connector Editor does not enable the "Submit" button when a value is pasted
Expected behavior
It should be enabled
Your installation details
9c028edcafb4aa1e94c21e9d554951e55035009f
Steps to reproduce

Navigate to an existing running connector
Click on Config
Paste a value in the editor. For example :

Both using the keyboard shortcuts and Chrome dialog produce the same bug

{
	"connector.class": "com.google.pubsub.kafka.sink.CloudPubSubSinkConnector",
	"errors.tolerance": "all"
}

Expected: The "submit" button is enabled
Actual: The button is not enabled. I need to do manual edits to trigger the event that enables the button
Screenshots
https://github.com/user-attachments/assets/d8126730-19b6-4503-9e46-26cc6e4b5730

Logs
No response
Additional context
This issue may be related to https://github.com/kafbat/kafka-ui/issues/495 although the steps to reproduce are a bit different

hey hi @yeikel
I checked it today and could not reproduce a bug. The "Sumbit" functionality works ok for me
check video proof pls
https://github.com/user-attachments/assets/063431ff-592c-40c7-949c-a52d8d62e539

hey hi @yeikel I checked it today and could not reproduce a bug. The "Sumbit" functionality works ok for me check video proof pls
bug_works.mp4

Your reproducer is a bit different because you made an edit first and then pasted. Can you try, starting from a fresh edit, to paste a value directly?

Create a connector
Save
Navigate from scratch to to connector and click edit
Paste a value

Seems like a dupe of #514
There's a PR (#515), but we'd need some tests for that


bump aws-sdk-s3 from 0.38.0 to 1.4.0

aws-sdk-s3 has breaking changes in this upgrade: https://github.com/apache/incubator-opendal/pull/3693

Can I try it?


Strand is always "positive"

See TODO in server/reads.py. This surprised me when I was trying to debug a disagreement between samtools and the GA4GH server.

Thanks for all digging up these issues @danvk, the reads code does need a little love. The issue really is that we currently don't have anyone developing on the server who understands read data to any real degree, so we're doing a lot of guesswork. It's great to have some more eyes on the code.

There's an issue open for the general problem @jeromekelleher is referring to
https://github.com/ga4gh/server/issues/347

Should we close this issue?  It seems to be addressed by the more general issue #347

Let's keep this one open until we have a specific fix (we might forget to address this particular problem as part of the cleanup).

Is this still an issue? Should I include it in the baseline milestone?

Yes, this is still an issue. I don't think it's a baseline issue. We know we have problems with our read handling, and we need some expert input to make sure we're interpreting the spec properly.

Suggest asking the file formats task team for a review
Jerome Kelleher<EMAIL_ADDRESS>writes:

Yes, this is still an issue. I don't think it's a baseline issue. We know we
have problems with our read handling, and we need some expert input to make
sure we're interpreting the spec properly.
—
Reply to this email directly or view it on GitHub.*


Cannot run the latest macOS release. Crash

Process:               DRAGONGLASS [6316]
Path:                  /private/var/folders/*/DRAGONGLASS.app/Contents/MacOS/DRAGONGLASS
Identifier:            DRAGONGLASS
Version:               ???
Code Type:             X86-64 (Native)
Parent Process:        ??? [1]
Responsible:           DRAGONGLASS [6316]
User ID:               501

Date/Time:             2018-12-02 15:54:39.705 +0200
OS Version:            Mac OS X 10.14.1 (18B75)
Report Version:        12
Bridge OS Version:     3.0 (14Y667)

System Integrity Protection: enabled

Notes:                 Translocated Process

Crashed Thread:        0

Exception Type:        EXC_CRASH (SIGABRT)
Exception Codes:       0x0000000000000000, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY

Termination Reason:    DYLD, [0x1] Library missing

Application Specific Information:
dyld: launch, loading dependent libraries

Dyld Error Message:
  Library not loaded: @rpath/QtPrintSupport.framework/Versions/5/QtPrintSupport
  Referenced from: /private/var/folders/*/DRAGONGLASS.app/Contents/MacOS/DRAGONGLASS
  Reason: image not found

Binary Images:
       0x1041ff000 -        0x104f81ffb + (.. - $..) <3CBA8BB5-8DD3-36EE-926E-8506B1A7EA18> /var/folders/*/DRAGONGLASS.app/Contents/MacOS/DRAGONGLASS
       0x107f2c000 -        0x107faa6a7  dyld (635.2) <1780094A-8FE2-3EAA-B4A3-C4CF14BC5196> /usr/lib/dyld
    0x7fff40bbd000 -     0x7fff40bbdfff  com.apple.Carbon (158 - 158) <BF1EC28F-CBA8-3B2F-B9FA-E583BC2D6C1F> /System/Library/Frameworks/Carbon.framework/Versions/A/Carbon
    0x7fff410d6000 -     0x7fff410d6fff  com.apple.Cocoa (6.11 - 23) <4D5D4968-E233-3598-B434-E1009D48A628> /System/Library/Frameworks/Cocoa.framework/Versions/A/Cocoa
    0x7fff418f9000 -     0x7fff41d46fe7  com.apple.CoreFoundation (6.9 - 1560.12) <B5BDC307-6C89-3DE0-B4B9-4C2C650E4528> /System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation
    0x7fff44373000 -     0x7fff44405ff7  com.apple.framework.IOKit (2.0.2 - 1483.220.15) <B58031DE-A6B7-3685-840F-361670C06836> /System/Library/Frameworks/IOKit.framework/Versions/A/IOKit
    0x7fff4ba6f000 -     0x7fff4ba7eff3  com.apple.opengl (17.0.37 - 17.0.37) <6BCCA53E-5621-32ED-BA9A-38B1C2499ED5> /System/Library/Frameworks/OpenGL.framework/Versions/A/OpenGL

Model: MacBookPro14,3, BootROM <IP_ADDRESS>.0, 4 processors, Intel Core i7, 2,8 GHz, 16 GB, SMC 2.45f0
Network Service: Wi-Fi, AirPort, en0

@skibitsky thanks for the report. so this may be related to your qt version. i think we should be >= qt5.0
this is from binary or build?

@aejontargaryen yes, binary from release 2.2.0

This is fixed.  I'll provide build with static libraries in a few minutes.

@skibitsky
Mac OS Release build updated
reference b45336214f0b583f8f94b2dd30ffdebada4ab510

closing. #sword of the morning


Unable to cancel clang-format for multiple files in case of error in .clang-format

I'm using .clang-format file with StatementMacros key. This key is not known for default clang-format used by the clang-power-tools. (I don't know what is used by default, since I only installed Clang 9 in my system). When the path to clang-format.exe (9) is not specified in clang-power-tools settings, I've got the following error:

In case of requesting to format all files in solution I got new one error window as I clicked "Ok" button for every processed file. There is no way to cancel such operation without terminating the whole IDE or pressing Enter while files are exceeded.
I know that this situation is rare, but for the bunch operation it would be good to have an option to cancel processing the rest of files. E.g another button near "Ok" named "Cancel all".

Hi,
Clang Power Tools is always coming automatically with the latest version of Clang Format. In the next release, which will be released this week, v4.10.4, we will deliver the CPT extension with Clang Format v8.0.
Unfortunately, if the behavior of the Clang Format is changed over time this is nothing we can do about it. This is why we introduced the option to add the path to the .exe file. In this way, the user can choose the best version for his needs.
Regarding your issue, if are you running clang format command when the file is not open the command will not currently work properly. We have this issue in progress and you can find more details about it here: https://github.com/Caphyon/clang-power-tools/issues/537. It seems for us that the issue occurred on your machine is actually the same issue.
In the case you are thinking this is a new bug can you tell us if you are using a .clang-format file in your project? Can you send us a simple project reproducing this issue with the .clang-format config file in it?

I'm using not yet released Clang 9.0, hence some features used in .clang-format file are not recognized by Clang 8. To test the issue you just need add any typo in .clang-format file to make this file broken for clang-format utility. Then open a project/solution with enough number of files to process (my solution contains about 250 files (sources + headers)). A lot of files is needed to increase the time of clang-format procession and allow to user the time to cancel the long operation. But despite of existing a time, there is no way to cancel such an operation before all files were processed, since you will get an error popup (see my first post) and you need to close it manually for each file.

Hi,
In all this time, we made many changes to Clang Power Tools, a lot of new features and bug fixes are now available. Please check for updates. At this moment, the latest version is v7.1.
Can you tell us if this issue is still present on your machine?

I will close this issue because of the lack of information and because we cannot reproduce it anymore.


How should bad/stale providers be handled?

Testing #226668
Have a provider that just does this return new Promise(() => { /* NEVER resolve */ });, e.g it never returns and today it can stall the whole find widget. This is how we usually do it (e.g for suggest or references) but I wonder if search should do something more nuanced, like showing results from providers as they come?

@jrieken curious your thoughts on this now- not sure what the plans are for this work area


Changed the term "security group" to just "group" to express that thi…

…s also works for unified groups

Hi @rjong999, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@rjong999 - thanks very much for your contribution!
@curtand - FYI


update RA calls to new version

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

If code is regenerated using this tag, code will have breaking changes. This problem can be avoided if a new tag for package-2017-10-preview is added, and the tag for package-2015-07 is left as it originally was.

Refers to: specification/authorization/resource-manager/readme.md:34 in 2996419. [](commit_id = 2996419ad2be943dabc60b44b8503f2f894bfb9c, deletion_comment = False)

@mcardosos Also since there wasnt any examples files before,is it mandatory in this pr to include those files for the build to pass?
or can i create a separate pr for this alone

I would prefer for the examples to be included here.

@mcardosos have updated the examples.I am however not sure the reason the build is failing.
Could you shed some info regarding that and what needs to be done also?

@mcardosos there seems to be some error in the build stating message: 'Equivalent path already exists: /{roleDefinitionId}',even though there is only one path like that.
Could you let me know on what needs to be done to fix the build issue

Added some more comments about readonly properties.
There are model validation errors to be fixed. They can be fixed on the examples files.
This error on x-ms-paths can be safely igonred in this case :)

@mcardosos updated the examples,so that the model validation errors are fixed

@darshanhs90
I am not sure exactly why, but CI didn't run the linter on the 2017-10-01-preview files.
I just ran the linter on your changes so we can actually take a look into this issues in the new API swagger.
If you also want to run the linter on your machine so you also can look at this issues locally, you can do it like this...
cd azure-rest-api-specs\specification\authorization\resource-manager
npx<EMAIL_ADDRESS>--validation --azure-validator --message-format=json

I will add more comments on the new API swaggers based on this errors found by the linter.
{
  "type": "Error",
  "code": "OperationsAPIImplementation",
  "message": "Operations API must be implemented for '/providers/Microsoft.Authorization/operations'.",
  "id": "R3023",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization-RACalls.json:38: 2 ($.paths)",
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization.json:38: 2 ($.paths)"
  ]
}
{
  "type": "Warning",
  "code": "NonApplicationJsonType",
  "message": "Only 'application/json' content-type is supported by ARM.",
  "id": "R2004",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization.json:14: 4 ($.consumes[1])"
  ]
}
{
  "type": "Warning",
  "code": "NonApplicationJsonType",
  "message": "Only 'application/json' content-type is supported by ARM.",
  "id": "R2004",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization.json:18: 4 ($.produces[1])"
  ]
}
{
  "type": "Warning",
  "code": "OperationIdNounConflictingModelNames",
  "message": "OperationId has a noun that conflicts with one of the model names in definitions section. The model name will be disambiguated to 'ProviderOperationsMetadataModel'. Consider using the plural form of 'ProviderOperationsMetadata' to avoid this. Note: If you have already shipped an SDK on top of this spec, fixing this warning may introduce a breaking change.",
  "id": "R2063",
  "validationCategory": "SDKViolation",
  "providerNamespace": "Microsoft.Authorization",
  "resourceType": "providerOperations",
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization.json:191: 8 ($.paths[\"/providers/Microsoft.Authorization/providerOperations/{resourceProviderNamespace}\"].get.operationId)"
  ]
}
{
  "type": "Warning",
  "code": "OperationIdNounConflictingModelNames",
  "message": "OperationId has a noun that conflicts with one of the model names in definitions section. The model name will be disambiguated to 'ProviderOperationsMetadataModel'. Consider using the plural form of 'ProviderOperationsMetadata' to avoid this. Note: If you have already shipped an SDK on top of this spec, fixing this warning may introduce a breaking change.",
  "id": "R2063",
  "validationCategory": "SDKViolation",
  "providerNamespace": "Microsoft.Authorization",
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization.json:237: 8 ($.paths[\"/providers/Microsoft.Authorization/providerOperations\"].get.operationId)"
  ]
}
{
  "type": "Error",
  "code": "RequiredPropertiesMissingInResourceModel",
  "message": "Model definition 'RoleDefinition' must have the properties 'name', 'id' and 'type' in its hierarchy and these properties must be marked as readonly.",
  "id": "R2020",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization.json:709: 4 ($.definitions.RoleDefinition)"
  ]
}
{
  "type": "Error",
  "code": "RequiredPropertiesMissingInResourceModel",
  "message": "Model definition 'RoleAssignment' must have the properties 'name', 'id' and 'type' in its hierarchy and these properties must be marked as readonly.",
  "id": "R2020",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization-RACalls.json:529: 4 ($.definitions.RoleAssignment)"
  ]
}
{
  "type": "Warning",
  "code": "AvoidNestedProperties",
  "message": "Consider using x-ms-client-flatten to provide a better end user experience",
  "id": "R2001",
  "validationCategory": "SDKViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization.json:524: 8 ($.definitions.ClassicAdministrator.properties.properties)"
  ]
}
{
  "type": "Warning",
  "code": "AvoidNestedProperties",
  "message": "Consider using x-ms-client-flatten to provide a better end user experience",
  "id": "R2001",
  "validationCategory": "SDKViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization.json:600: 8 ($.definitions.ProviderOperation.properties.properties)"
  ]
}
{
  "type": "Warning",
  "code": "AvoidNestedProperties",
  "message": "Consider using x-ms-client-flatten to provide a better end user experience",
  "id": "R2001",
  "validationCategory": "SDKViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization.json:726: 8 ($.definitions.RoleDefinition.properties.properties)"
  ]
}
{
  "type": "Warning",
  "code": "EnumInsteadOfBoolean",
  "message": "Booleans are not descriptive and make them hard to use. Consider using string enums with allowed set of values defined. Property: canDelegate",
  "id": "R3018",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization-RACalls.json:501: 8 ($.definitions.RoleAssignmentFilter.properties.canDelegate)"
  ]
}
{
  "type": "Warning",
  "code": "EnumInsteadOfBoolean",
  "message": "Booleans are not descriptive and make them hard to use. Consider using string enums with allowed set of values defined. Property: canDelegate",
  "id": "R3018",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization-RACalls.json:522: 8 ($.definitions.RoleAssignmentPropertiesWithScope.properties.canDelegate)"
  ]
}
{
  "type": "Warning",
  "code": "AvoidNestedProperties",
  "message": "Consider using x-ms-client-flatten to provide a better end user experience",
  "id": "R2001",
  "validationCategory": "SDKViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization-RACalls.json:546: 8 ($.definitions.RoleAssignment.properties.properties)"
  ]
}
{
  "type": "Warning",
  "code": "EnumInsteadOfBoolean",
  "message": "Booleans are not descriptive and make them hard to use. Consider using string enums with allowed set of values defined. Property: canDelegate",
  "id": "R3018",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization-RACalls.json:579: 8 ($.definitions.RoleAssignmentProperties.properties.canDelegate)"
  ]
}
{
  "type": "Warning",
  "code": "AvoidNestedProperties",
  "message": "Consider using x-ms-client-flatten to provide a better end user experience",
  "id": "R2001",
  "validationCategory": "SDKViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization-RACalls.json:588: 8 ($.definitions.RoleAssignmentCreateParameters.properties.properties)"
  ]
}
{
  "type": "Warning",
  "code": "NonApplicationJsonType",
  "message": "Only 'application/json' content-type is supported by ARM.",
  "id": "R2004",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization-RACalls.json:14: 4 ($.consumes[1])"
  ]
}
{
  "type": "Warning",
  "code": "NonApplicationJsonType",
  "message": "Only 'application/json' content-type is supported by ARM.",
  "id": "R2004",
  "validationCategory": "RPCViolation",
  "providerNamespace": null,
  "resourceType": null,
  "sources": [
    "file:///C:/azure-rest-api-specs/specification/authorization/resource-manager/Microsoft.Authorization/2017-10-01-preview/authorization-RACalls.json:18: 4 ($.produces[1])"
  ]
}

Looks good. Signing Off!!

@mcardosos Fixed all the above issues

No modification for AutorestCI/azure-sdk-for-node

 https://github.com/Azure/azure-sdk-for-python/pull/1644

 https://github.com/Azure/azure-sdk-for-ruby/pull/1144


v1.60: "Auto language select" feature sets "CoffeeScript" language in "Git commit" files

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: v1.60
OS Version: Win 10

Steps to Reproduce:

Update to v1.60
run git commit
Notice selected language was automatically set to "CoffeeScript" instead of "Git commit message"

Workaround is probably to set "workbench.editor.languageDetection": false

@gjsjohnmurray the feature is nice, I'd rather not disable it completely.
But yeah, I might have do that until the issue is fixed

See #131565 for how you should be able to prevent a 'git-commit' file from switching to another language.

@TylerLeonhardt maybe https://github.com/microsoft/vscode/blob/70e30bc64e9ec7dd996259a00765f62a5e339f59/extensions/git/package.json#L2315-L2324 should disable language detection by default for 'git-commit' and 'git-rebase'?

specific workaround
"[git-commit]": {
        "workbench.editor.languageDetection": false
},

You can also do this for git-rebase

That will help with the dotenv issue I think too since we don't ship a language extension for that language in box.

fixed in ^

This will be in the 1.60.1 release targeted for next week.

@TylerLeonhardt good job, thank you

I'm still seeing this issue in 1.60.2. Was there a possible regression? Happy to provide any additional info needed, but the steps to reproduce in the original issue comment still apply.
Version: 1.60.2
Commit: 7f6ab5485bbc008386c4386d08766667e155244e
Date: 2021-09-22T11:59:27.195Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 18.7.0

@bcaplan I think you're seeing this race condition:
https://github.com/microsoft/vscode/issues/133506#issuecomment-928414654
which I plan on fixing this week.

@bcaplan I think you're seeing this race condition: #133506 (comment) which I plan on fixing this week.

@TylerLeonhardt I think you might be right, the file does open with the correct language selected, then quickly switches. I look forward to the fix, thanks for the response!


Bring Modernisation Platform view-only access into Terraform management

Brings the view-only access permission set for the modernisation-platform team into Terraform management.

@SteveMarshall 🙏🏼


Educational project

Hi
Here It is an educational project
https://github.com/joseluisgs/tenistas-rest-ktor-2022-2023
Ktor, reactive, SSL, JWT, websocket, validation, test, Koin DI, and others
Thank you

Added.

Thank you very much! 😀


I added a "/retry" command to retry for getting another answer.

Sometimes I want to re-roll the answer for something that fits better into the context. I am not sure if my implementation is correct, but it works for me like a charm so far.
I also added the missing parameter to the documentation for /pop.

Could you also update README.md? Thank you!

Also thanks for this PR, /retry seems will be a useful function to have

Well, I am not even a Python programmer (more the Go person) and I just skimmed over the implementation. I guess that checking that one pops up to and including the last user message is fine. I would need to figure out how to do that :)

I decided to check out how to make a better implementation for '/retry'. I think I got it. But need your review.

I would love to make something like "/retry Yes, " or "/retry No, " where the string is being added as the start of the assistant reply to further shape the results. But I am unsure if that is possible with the current design. I guess one would need to extend the function calls with something like "prepend" for the agent?

I updated the README and formatted the code. I hope this fits now.

I added the update for the pop <count> in the README and made the default to 3 instead of 2 (so it removes fun/assistant/user) which seems cleaner to me.

Thank you for the contribution! /retry, /rethink, and /rewrite are all really cool commands that I'm sure many others will find very useful, thank you for adding them and opening PRs to contribute back into the main repo.


Pinta is unable to open files from Finder

Description
If I try to open an image file with Pinta from Finder, I get the error

The document “{filename}” could not be opened. Pinta cannot open files in the “{format}” format.

This happens with any image file, regardless of the format (I tried png, jpg and bmp) and Pinta can open the same files if I use the Open option in its toolbar.
To Reproduce
Right-click an image file in Finder, select Open With > Pinta. Pinta opens and error appears.
Additional Info

Version
I'm running macOS Sonoma 14.6.1. I get this issue on the latest build of Pinta as well as the 2.2 preview build from here, but not release 2.1.2.

Just to clarify, this is only happening for you with the 2.2 builds, but 2.1 has worked fine for you?

Correct, 2.1.2 works correctly

Should be fixed now, thanks for catching this!


Implement new questions using facebook reactions

Currently it only shows the generic "Who reacted ..." because that's the only type of reactions received by the server.

@vincenzobaz @Rabyss Can I merge this?

No, I'll handle it soon.

Ok :)


x/mobile: Is it possible to send a function as argument from Java to golang?

I am using gomobile bind to generate binding for android. I have a use case where I want to write and export a function in golang which takes in a simple function as an argument like so:
func SetCallback(cb Callback)
where Callback is a user  defined type:
type Callback func(string)
However gomobile bind is not able to generate java bindings for this function. I have tried to do this directly without defining a Callback type, but no success. Is there any other way to do this?
Thank you for your time, looking forward to your response.

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Get the rest of the copyright symbols

Just to be thorough

Yeah, I though about that on the initial PR.
Thanks!


parallelism and NBuilder

Hi guys,
Not sure if anyone is actively contributing to this any more, looks like very minimal lately.
We had an issue when converting our Nunit tests to XUnit for parallel capabilities, and had make some mods to the source in order to get everything working, making the Builder and the BuilderSetup instance based rather than static.
Would you be interested in a pull request, or should we fork?  We'd love for this to be rolled in and not have to maintain our own copy, but there was no way to work around the issue.  If you're interested in looking at the changes we made let's chat!
-Jason

We've been experiencing similar problems. Would be great to have the fork available on GH to reference at least. Then it can be turned into a pull request potentially?

AutoFixture seems to be the way to go now.

I added the pull request.
https://github.com/garethdown44/nbuilder/pull/37
Works for us, the only difference is having to pass around a BuilderSetup() which we new up at the beginning of our tests

NBuilder is being maintained. I'm working on a catch-up release now. I'm still getting my sea-legs under me as co-manager of the project, so it will likely take a little while.

Polite bump - any news on this?

It looks like the pull request has been integrated. There are still a couple of bug fixes I want to finish before the next release, as well as reaching a final decision on what to do with the Silverlight build. You can track what's happening (or submit PR's) with the "Catch-up Release" milestone.


Unable to read language.json file

Hello,
I encountered an issue with the following code:
lange := carbon.NewLanguage()
lange.SetLocale("zh-CN")
car := carbon.SetLanguage(lange)
if car.Error != nil {
	log.Printf("Error Info %v\n", car.Error)
	return
}

offDutyTime := car.Now().StartOfDay().AddHours(18)
now := car.Now()
diff:= now.DiffInString(offDutyTime)
log.Printf("Diff: %v", diff)

golang version: such as 1.16
1.18.1
carbon version: such as 1.5.0
v2.1.8
time zone: such as Japan
China
I expected to get:
`Diff: 9小时`

But I actually get:
invalid locale file "lang\\zh-CN.json", please make sure the json file exists and is valid

Pic:

Thanks!

Sorry, I haven't tested it on the window platform, I will try to find a window platform device to test.

I had the same problem， windows10, go.1.18.3, carbon: v2.1.8
func TestWeek(t *testing.T) { 	lang := carbon.NewLanguage() 	lang.SetLocale("zh-CN") 	c := carbon.SetLanguage(lang) 	if c.Error != nil { 		// 错误处理 		t.Error(c.Error) 	} 	fmt.Println(c.ToWeekString()) }
=== RUN   TestWeek
a_test.go:15: invalid locale file "lang\zh-CN.json", please make sure the json file exists and is valid
--- FAIL: TestWeek (0.00s)
FAIL

It has been fixed in v1.7.1 or v2.2.1


Exchanged token not getting refreshed during token refresh in requestCustomGrant

Describe the issue:
When an application is using a custom grant to do a token exchange, the first token is obtained from a particular IDP and then it is exchanged for another token from another IDP using a token service.
As of now, the SDK initiates a refresh token call whenever a 401 response is received to a network call and gets a new token from the first IDP.
The issue is that although the token from the first IDP is obtained from the refresh token grant, the token exchange with the second IDP doesn't happen  and hence, the access token stored in the worker is the token from the first IDP and all other network calls to the backend REST APIs fail since they are expecting a new token from the second IDP.
How to reproduce:

Expected behavior:
When the SDK refreshes a token, it would be expected to see if a custom grant has been configured and if so, initiates the custom grant as well.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

@thivi This is already fixed in SDK so I think we can close the issue now ?

Closing based on https://github.com/asgardeo/asgardeo-auth-react-sdk/issues/70#issuecomment-1285840878


Rather poor level of detail on prerequisites

Especially for "Verify that the account that runs the Asynchronous Processing Service has read access to private keys of the certificate." it would be helpful to say how to do that and who to resolve. Steps are basically these:

Run mmc.
File -> Add/remove snap-in...
Add "Certificates" (Computer account, Local Computer).
Navigate to Certificates/Personal/Certificates.
Right-click the certificate you're trying to use for authentication with Exchange Online -> All Tasks -> Manage Private Keys.
Add the service account the Asynchronous Service runs under and give it read permission.
Restart the "Microsoft Dynamics CRM Asynchronous Processing Service" and the "Microsoft Dynamics CRM Asynchronous Processing Service (maintenance)" services.

(thanks to https://community.dynamics.com/crm/f/microsoft-dynamics-crm-forum/207762/crm-hybrid-configuration-error-aquiring-sic-token-from-acs-has-failed)
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d4a7165a-d7a7-fecd-3260-f6be07a6f331
Version Independent ID: aa92c5c1-3e94-ae45-5e0c-8d5edd4f4f5f
Content: Connect Exchange Online to Dynamics 365 Customer Engagement (on-premises)
Content Source: ce/customerengagement/on-premises/admin/connect-dynamics-365-on-premises-exchange-online.md
Product: d365ce-op
GitHub Login: @Mattp123
Microsoft Alias: matp

Thanks for your comment, @JamesHorsley. This issue has been assigned to the doc author for review.

@JamesHorsley - Thank you for your feedback. The procedure has been added to the article and will be published live shortly.
Closing issue.


RCL should show .csproj

See https://github.com/aspnet/Razor/issues/2391

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

ID: d92ffa7c-fd65-7dc3-194e-6a67a8a7f176
Version Independent ID: 2df11a37-d64e-6353-bd92-0b7729070c3e
Content: Reusable Razor UI in class libraries with ASP.NET Core
Content Source: aspnetcore/mvc/razor-pages/ui-class.md
Product: asp.net-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Fixed by #7888


edsl_with_cloud_providers.ipynb fails

I added the "skip-execution" tag to avoid failing in testing.

@rbyh I don't think we should delete the notebook. It shows the users how to connect to edsl with their custom-deployed models.


Straight line annotation

Paid Pro user...
I need to annotate some images for semantic segmentation, where the annotation is a thick straight line.
When selecting draw new mask there is an option to change the thickness of the mark.
I need something similar when annotating with polyline.
Thanks!

Dublicate of #2553


using Q script for AI samples on Devcloud

Existing Sample Changes
Description
Make DevCloud usage on AI samples more user friendly, and could submit a task to a specific Intel Architecture such as ICX.
External Dependencies
NA
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[ X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ X] Command Line

@bdmoore1
could you help me to review it?
thanks

This may seem obvious, but between step 7. "You will see your login at the prompt" and step 8 about the readme, we need to tell the user to actually get the samples! It's very disjointed how it is now since they are sitting at a prompt, but there isn't a readme anywhere. ideally we'd provide a link to pull the repo.
addressed this feedback with related modification. we will guide them to how to get samples section accordingly.

@tomashek @bdmoore1
please help me to review the contents.
the main change is adding a Q script for DevCloud job submission.


Update README.md

Updated README.md in order to provide a more proper way to install the application with one easy-to-use command.

@OpenSource03 Thank you for this and my apologies for not having those requirements listed before!


fix(tabs): Restore the highlight style when tab is selected

Related Ticket(s)
Closes #11147
Description
The current tab is not correctly highlighted, the border-bottom color doesn't have the correct color:
The Storybook is correctly showing the color but I cannot reproduce it in my project. I created a reproduction of the bug in stackblitz.
Storybook:

Stackblitz

Looking at the code in this line of the scss, the [selected] is appended to the tabs so the built selector is:
:host(cds-tabs)[selected].
This is not a valid selector and it should be:
:host(cds-tabs[selected]).
I confirmed the compilation with the compiled source in the navigator.
Changelog
New

Set the style when the tab is selected with the correct selector.

Changed
Removed

Deleted the scss code that used the wrong selector.

@siaikin @kennylam @ariellalgilmore after this PR the height of the tabs is wrong (48px):

I expect something like (42px):

Real example from:

To:

@siaikin @kennylam @ariellalgilmore after this PR the height of the tabs is wrong (48px):

https://www.figma.com/file/YAnB1jKx0yCUL29j6uSLpg/(v11)-All-themes---Carbon-Design-System?type=design&node-id=27723-299438&mode=design&t=w37vXSCI2zyq4S2u-4
Looking at the figma file, the height should be 40px

@abdonrd Thanks for the head's up! Issue created https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/11540.


Install and configure Next app

@garyttierney @danielgrant Would you be able to assist on getting buildkite up and running? This app has its own self contained Dockerfile and a handful of similar environment variables.

Moved to wellcomecollection.org repo:
https://github.com/wellcomecollection/wellcomecollection.org/pull/6088


Failed to create task or type PCTConnection

I am going through the quick start in your documentation and the build is failing on step 2.
Here is the full output from the command line:
C:\Users\Administrator\Progress\Developer Studio 4.3.1\workspace\MyProject>ant -v
Apache Ant(TM) version 1.9.6 compiled on June 29 2015
Trying the default build file: build.xml
Buildfile: C:\Users\Administrator\Progress\Developer Studio 4.3.1\workspace\MyProject\build.xml
Detected Java version: 1.8 in: C:\Progra~1\Java\jdk1.8.0_60\jre
Detected OS: Windows 7
parsing buildfile C:\Users\Administrator\Progress\Developer Studio 4.3.1
\workspace\MyProject\build.xml with URI = file:/C:/Users/Administrator/Progress/Develope
r%20Studio%204.3.1/workspace/MyProject/build.xml
Project base dir set to: C:\Users\Administrator\Progress\Developer Studio 4.3.1
\workspace\MyProject
parsing buildfile jar:file:/C:/apache-ant-1.9.6/lib/ant.jar!/org/apache/tools/ant/antlib.xml with URI =
jar:file:/C:/apache-ant-1.9.6/lib/ant.jar!/org/apache/to
ols/ant/antlib.xml from a zip file
 [property] Loading Environment env.
Property "env.PCT_HOME" has not been set
dropping C:\Users\Administrator\Progress\Developer Studio 4.3.1
\workspace\MyProject\${env.PCT_HOME}\lib\PCT.jar from path as it doesn't exist
Build sequence for target(s) `build' is [build]
Complete build sequence is [build, ]

build:
[PCTVersion] PCT Version 1.1-pre - Build 194
[ProgressVersion] dlcHome attribute not set. Falling back to DLC environment variable :
c:\progress\v115
[ProgressVersion] OpenEdge version found : OpenEdge Release 11.5.1 as of Wed May  6 19:01:46 EDT 2015
[ProgressVersion] Using object : com.phenix.pct.ProgressV114
[PCTCreateBase] OpenEdge version found : OpenEdge Release 11.5.1 as of Wed May  6
19:01:46 EDT 2015
[PCTCreateBase] Using object : com.phenix.pct.ProgressV114
[PCTCreateBase] Copying empty DB to db
[PCTCreateBase] Current OS is Windows 7
[PCTCreateBase] Output redirected to C:\Users\Administrator\Progress\Developer Studio 4.3.
\workspace\MyProject\db\db.procopy.log
[PCTCreateBase] Setting environment variable: DLC=c:\progress\v115
[PCTCreateBase] Executing 'c:\progress\v115\bin\_dbutil.exe' with arguments:
[PCTCreateBase] 'procopy'
[PCTCreateBase] 'c:\progress\v115\empty8'
[PCTCreateBase] 'db'
[PCTCreateBase]
[PCTCreateBase] The ' characters around the executable and arguments are
[PCTCreateBase] not part of the command.
[PCTCreateBase] OpenEdge version found : OpenEdge Release 11.5.1 as of Wed May  6
19:01:46 EDT 2015
[PCTCreateBase] Using object : com.phenix.pct.ProgressV114
[PCTCreateBase] cpstream is not defined. You may have problems if file names have characters
outside of ASCII range. You really should define it !
[PCTCreateBase] Current OS is Windows 7
[PCTCreateBase] Setting environment variable: DLC=c:\progress\v115
[PCTCreateBase] Executing 'c:\progress\v115\bin\_progres.exe' with arguments:
[PCTCreateBase] '-b'
[PCTCreateBase] '-q'
[PCTCreateBase] '-p'
[PCTCreateBase] 'C:\Users\SCHNEI~1\AppData\Local\Temp\pctinit29301.p'
[PCTCreateBase]
[PCTCreateBase] The ' characters around the executable and arguments are
[PCTCreateBase] not part of the command.
[PCTCreateBase] Loading C:\Users\Administrator\Progress\Developer Studio 4.3.
\workspace\MyProject\db\db.df in database
[mkdir] Skipping C:\Users\Administrator\Progress\Developer Studio 4.3.
\workspace\MyProject\build because it already exists.

BUILD FAILED
C:\Users\Administrator\Progress\Developer Studio 4.3.1\workspace\MyProject\build.xml:13:
Problem: failed to create task or type PCTConnection
Cause: The name is undefined.
Action: Check the spelling.
Action: Check that any custom tasks/types have been declared.
Action: Check that any <presetdef>/<macrodef> declarations have taken place.

    at org.apache.tools.ant.UnknownElement.getNotFoundException(UnknownElement.java:500)
    at org.apache.tools.ant.UnknownElement.makeObject(UnknownElement.java:432)
    at org.apache.tools.ant.UnknownElement.maybeConfigure(UnknownElement.java:164)
    at org.apache.tools.ant.Task.perform(Task.java:347)
    at org.apache.tools.ant.Target.execute(Target.java:435)
    at org.apache.tools.ant.Target.performTasks(Target.java:456)
    at org.apache.tools.ant.Project.executeSortedTargets(Project.java:1405)
    at org.apache.tools.ant.Project.executeTarget(Project.java:1376)
    at org.apache.tools.ant.helper.DefaultExecutor.executeTargets(DefaultExecutor.java:41)
    at org.apache.tools.ant.Project.executeTargets(Project.java:1260)
    at org.apache.tools.ant.Main.runBuild(Main.java:853)
    at org.apache.tools.ant.Main.startAnt(Main.java:235)
    at org.apache.tools.ant.launch.Launcher.run(Launcher.java:285)
    at org.apache.tools.ant.launch.Launcher.main(Launcher.java:112)

Total time: 1 second

C:\Users\Administrator\Progress\Developer Studio 4.3.1\workspace\MyProject>

I got through step 1 ok but connecting to the database before building doesn't seem to be working for me.
The db.lg and db.procopy files seem to indicate that the database was created ok.  I can provide if necessary.
I am new to Progress OpenEdge and Developer Studio so please go easy on me if I am missed something silly.
I have been following the QuickStart and have been copying and pasting from that so there shouldn't be any issues with the build.xml or .p or .df files.  I can of course provide those if necessary.

Hi,
You may want to add <typedef resource="types.properties" /> after the <taskdef> node.
That's written in the InstallDocumentation page, but not in the QuickStart page, I'll update this part.
Gilles

Thanks jakejustus that got me past that problem.  Unfortunately now I have another one :(
When I try to build this is now what I get:
C:\Users\Administrator\Progress\Developer Studio 4.3.1\workspace\MyProject>ant -v
Apache Ant(TM) version 1.9.6 compiled on June 29 2015
Trying the default build file: build.xml
Buildfile: C:\Users\Administrator\Progress\Developer Studio 4.3.1\workspace\MyProject\build.xml
Detected Java version: 1.8 in: C:\Progra~1\Java\jdk1.8.0_60\jre
Detected OS: Windows 7
parsing buildfile C:\Users\Administrator\Progress\Developer Studio 4.3.1
\workspace\MyProject\build.xml with URI = file:/C:/Users/Administrator/Progress/Develope
r%20Studio%204.3.1/workspace/MyProject/build.xml
Project base dir set to: C:\Users\Administrator\Progress\Developer Studio 4.3.1
\workspace\MyProject
parsing buildfile jar:file:/C:/apache-ant-1.9.6/lib/ant.jar!/org/apache/tools/ant/antlib.xml with URI =
jar:file:/C:/apache-ant-1.9.6/lib/ant.jar!/org/apache/to
ols/ant/antlib.xml from a zip file
 [property] Loading Environment env.
Build sequence for target(s) `build' is [build]
Complete build sequence is [build, ]

build:
[PCTVersion] PCT Version 1.1-pre - Build 194
[ProgressVersion] dlcHome attribute not set. Falling back to DLC environment variable :
c:\progress\v115
[ProgressVersion] OpenEdge version found : OpenEdge Release 11.5.1 as of Wed May  6
19:01:46 EDT 2015
[ProgressVersion] Using object : com.phenix.pct.ProgressV114
[PCTCreateBase] OpenEdge version found : OpenEdge Release 11.5.1 as of Wed May  6
19:01:46 EDT 2015
[PCTCreateBase] Using object : com.phenix.pct.ProgressV114
[PCTCreateBase] Database db already exists
[mkdir] Skipping C:\Users\Administrator\Progress\Developer Studio 4.3.1
\workspace\MyProject\build because it already exists.
[PCTCompile] OpenEdge version found : OpenEdge Release 11.5.1 as of Wed May  6 19:01:46
EDT 2015
[PCTCompile] Using object : com.phenix.pct.ProgressV114
[PCTCompile] PCTCompile - Progress Code Compiler
[PCTCompile] cpstream is not defined. You may have problems if file names have characters
outside of ASCII range. You really should define it !
[PCTCompile] Current OS is Windows 7
[PCTCompile] Setting environment variable: DLC=c:\progress\v115
[PCTCompile] Executing 'c:\progress\v115\bin\_progres.exe' with arguments:
[PCTCompile] '-b'
[PCTCompile] '-q'
[PCTCompile] '-param'
[PCTCompile] 'C:\Users\SCHNEI~1\AppData\Local\Temp\pct_params20161.txt'
[PCTCompile] '-p'
[PCTCompile] 'C:\Users\SCHNEI~1\AppData\Local\Temp\pctinit41160.p'
[PCTCompile]
[PCTCompile] The ' characters around the executable and arguments are
[PCTCompile] not part of the command.
[PCTCompile] PROPATH : C:\Users\SCHNEI~1
\AppData\Local\Temp\pct36016.pl,.,c:\progress\v115\tty,c:\progress\v115
\tty\ablunit.pl,c:\progress\v115\tty\adecomm.pl,c
:\progress\v115\tty\adecomp.pl,c:\progress\v115\tty\adeedit.pl,c:\progress\v115
\tty\adeshar.pl,c:\progress\v115\tty\dataadmin.pl,c:\progress\v115\tty\OpenEdge.B
 usinessLogic.pl,c:\progress\v115\tty\OpenEdge.Core.pl,c:\progress\v115
\tty\OpenEdge.ServerAdmin.pl,c:\progress\v115\tty\prodict.pl,c:\progress\v115,c:\progress\
v115\bin
 [PCTCompile] RUN pct/v11/pctCompile.p
 [PCTCompile] Switching to fileset C:\Users\Administrator\Progress\Developer Studio 4.3.1
 \workspace\MyProject\src
 [PCTCompile] Compiling test2.p in directory C:\Users\Administrator\Progress\Developer Studio
4.3.1\workspace\MyProject\src TO C:\Users\Administrator\Progress\De
veloper Studio 4.3.1\workspace\MyProject\build/
[PCTCompile] Error compiling file C:\Users\Administrator\Progress\Developer Studio 4.3.1
\workspace\MyProject\src\test2.p at line 1 column 8
[PCTCompile] CREATE MyTable.
[PCTCompile] -------^
[PCTCompile] Unknown or ambiguous table MyTable. (725)
[PCTCompile] ** C:\Users\Administrator\Progress\Developer Studio 4.3.1
\workspace\MyProject\src\test2.p Could not understand line 1. (196)
[PCTCompile] ** Unknown table name MyTable. (200)
[PCTCompile] ** C:\Users\Administrator\Progress\Developer Studio 4.3.1
\workspace\MyProject\src\test2.p Could not understand line 2. (196)
[PCTCompile]
[PCTCompile] 0 file(s) compiled
[PCTCompile] Failed to compile  1  file(s)
[PCTCompile] Return value : 10

BUILD FAILED
C:\Users\Administrator\Progress\Developer Studio 4.3.1\workspace\MyProject\build.xml:15: Return
code : 10
    at com.phenix.pct.PCTRun.execute(PCTRun.java:739)
    at com.phenix.pct.PCTCompile.execute(PCTCompile.java:683)
    at org.apache.tools.ant.UnknownElement.execute(UnknownElement.java:293)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:497)
    at org.apache.tools.ant.dispatch.DispatchUtils.execute(DispatchUtils.java:106)
    at org.apache.tools.ant.Task.perform(Task.java:348)
    at org.apache.tools.ant.Target.execute(Target.java:435)
    at org.apache.tools.ant.Target.performTasks(Target.java:456)
    at org.apache.tools.ant.Project.executeSortedTargets(Project.java:1405)
    at org.apache.tools.ant.Project.executeTarget(Project.java:1376)
    at org.apache.tools.ant.helper.DefaultExecutor.executeTargets(DefaultExecutor.java:41)
    at org.apache.tools.ant.Project.executeTargets(Project.java:1260)
    at org.apache.tools.ant.Main.runBuild(Main.java:853)
    at org.apache.tools.ant.Main.startAnt(Main.java:235)
    at org.apache.tools.ant.launch.Launcher.run(Launcher.java:285)
    at org.apache.tools.ant.launch.Launcher.main(Launcher.java:112)

 Total time: 0 seconds

 C:\Users\Administrator\Progress\Developer Studio 4.3.1\workspace\MyProject>

I have opened the database using Data Administrator and have run the code in test2.p in data administrator and it all works fine.  It would seem that in the build it isn't finding/connecting to the database properly for the code in test.p to compile properly.  The problem is that because I am a noob in OpenEdge and ant builds and PCT I'm not sure what the problem is or how to fix it.  Any help would be appreciated.  Thanks.

I managed to figure it out.  I had the PCTConnection element outside (above) the PCTCompile element.  I moved the PCTConnection to be a child element of PCTCompile and now everything is ok.

Quickstart page has been updated. Closing issue, please open a new one if you have additional issues.


Add files, fix bugs, and support MP3.

Fixes https://github.com/TitchMW/outerwildsmusicplayer/issues/1

:/


'En dash expected' for something that is not a number range

In a tikz environment I have a label which is like R8 1k\Omega. I get the inspection Number ranges should use an en dash (–) instead of any other dash/hyphen. This is not at all a number range, and that's why I'm not using a dash or hyphen. What was the intended use case for this inspection?

Could you give me the full tikz environment contents?

Sure, if you want, here it is:
\documentclass{article}
\usepackage{circuitikz}
\usepackage{tikz}
\begin{document}

    \begin{center}\begin{circuitikz}
                      \draw[dashed]
                      (2,10) to (5,10)
                      to (5,14)
                      to (2,14)
                      to (2,10)
                      (3.5,13) node[align=left] {Lenze\\ frequency\\ inverter}
                      ;
                      \draw
                      %E4 and its components (Arduino I/O, transistors)
                      (10,0) node {E4}
                      (1,1.225) node[align=center] {Arduino\\ I/O}
                      (10,2) node [pnp] (pnpE4) {}
                      (pnpE4.B) node[right=8mm, align=left] {Q6\\ BC557}
                      (5,1.225) node [npn] (npnE4) {}
                      (npnE4.B) node[right=8mm, align=left] {Q5\\ BC547}
                      %Low end stop and its components
                      (0.5,7.225) node[align=center] {Arduino\\ I/O}
                      (10,8) node [pnp] (pnpLow) {}
                      (pnpLow.B) node[right=8mm, align=left] {Q4\\ BC557}
                      (4, 7.225) node [npn] (npnLow) {}
                      (npnLow.B) node[right=8mm, align=left] {Q3\\ BC547}
                      %High end stop and its components
                      (0.5, 15.725) node[align=center] {Arduino\\ I/O}
                      (10,16.5) node[pnp] (pnpHigh) {}
                      (pnpHigh.B) node[right=8mm, align=left] {Q2\\ BC557}
                      (4, 15.725) node[npn] (npnHigh) {}
                      (npnHigh.B) node[right=8mm, align=left] {Q1\\ BC547}
                      %Frequency Invertor and components (including line to GNDs)
                      (3.5,11) node[sground] {}
                      to (2,11)
                      (1.2,11.5) node[align=left] {39 \\(GND)}
                      (2,11) to[short,-*] (-1,11)
                      (5.7,12) node[align=left] {28 \\ enable}
                      (5,11.5) to (6.5,11.5) % fi to right
                      %Arduino GND
                      (-3,11) node[align=left] {Arduino\\ GND}
                      (-2,11) to (-1,11)
                      (-1,0) to (-1,14.955) % line on the left that connects GND
                      (12,4) to (12,18.5) % line on the right that connects 20
                      %circuit around E4
                      (10,0.25) to (pnpE4.C)
                      (pnpE4.B) to[R=R8 1k$\Omega$] (npnE4.C)
                      (npnE4.E) to (5,0)
                      to (-1,0)
                      (npnE4.B) to[R, l=R7 4.8k$\Omega$] (2.1, 1.225)
                      (9,2) node[circ] {}
                      to[R,align=left, l=R9\\ 10k$\Omega$] (9,4)
                      to (12,4)
                      (10,4) node[circ] {}
                      to (pnpE4.E)
                      %circuit around low end stop
                      (pnpLow.B) to[R=R5 1k$\Omega$] (7,8)
                      to (npnLow.C)
                      (9,8) node[circ] {}
                      to[R, align=left, l=R6\\ 10k$\Omega$] (9,10)
                      to[short, -*] (12,10)
                      (pnpLow.E) to (10,10) node[circ] {}
                      (pnpLow.C) to[switch, align=left, l=low\\ end stop] (10,6)
                      to (6.5,6)
                      to (6.5,10) % up to high end stop
                      (npnLow.B) to[R=R4 4.7k$\Omega$] (1.2, 7.225)
                      (npnLow.E) to (4,6)
                      to[short, -*] (-1,6)
                      %circuit around high end stop
                      (pnpHigh.B) to[R=R2 1k$\Omega$] (7,16.5)
                      to (npnHigh.C)
                      (9,16.5) to[R, align=left, l=R3\\ 10k$\Omega$, *-] (9,18.5)
                      to (12,18.5)
                      (pnpHigh.E) to[short, -*] (10,18.5)
                      (pnpHigh.C) to (11.5,15.74) % from Q2 to right ...
                      (11.5,15.74) to (11.5,7.225) % ... to right of Q4 ...
                      (11.5,7.225) to[short,-*] (10,7.225) % ... to Q4
                      (npnHigh.E) to (-1,14.955)
                      (npnHigh.B) to[R=R1 4.7k$\Omega$] (1.2, 15.725)
                      % draw end stop
                      (6.5,11.5) to[switch, align=right, l=high end stop] (6.5,10)
                      ;
    \end{circuitikz}\end{center}

\end{document}

Another case: January 4 1999 is also not a number range.

January 4 1999 should be written as January 4, 1999.
Cannot catch ALL exceptions. We are making an inspection suppression system for b0.6 - so you can always exclude them yourselves later. For now. Fixed the problem with [R=R8 1k.


Symlinks in src/pages, multiple pages source directories

Description
Hello! I want to place my pages files not only inside "./pages" or "./src/pages". I also want to be possible store it inside "../../another-package/src/etc/my-page". I was expected if I do ln -s ../../another-package/src/etc ./src/pages/another-package-symlink that will work out of the box. So I want to have many places to store my pages. How can I do it now? Is it possible? Thanks a lot!

It will be perfect to have setting like pagesDirsGlobs. For example looks like now it is ["./pages/**/*", "./pages/**/*"]. In my case I want make it ["./pages/**/*", "./pages/**/*", "./../../another-package/src/**/*Page"]


Import pdfme with a script element

Is your feature request related to a problem? Please describe.
I want to use pdfme by importing it with a script element. I don't use nodejs, typescript, webpack, etc.
Describe the solution you'd like
I want to import pdfme with the script tag like pdfjs
<script type="module">
import pdfjsDist from<EMAIL_ADDRESS></script>

or pdf-lib
<script src="https://unpkg.com/pdf-lib"></script>
Describe alternatives you've considered
None
Additional context
No response

This is not a feature request.
Please continue at https://github.com/pdfme/pdfme/discussions/455

@hand-dot I understand, can you help me on that discussion? I'd greatly appreciate it. Thanks...


nix tools use ascii instead of unicode apostrophe

Hi all,
Is it at all possible to refactor nix tools in general so that instead of outputting unicode stuff like
instantiated ‘entropy-0.3.6.tar.gz’ -> ‘/nix/store/r7nsxd6k5ig8z6748rl49yzsc74pi89a-entropy-0.3.6.tar.gz.drv’
(unicode ‘’, i.e. 0x2018, 0x2019)
to use ascii ', i.e. 0x27
This would make it easier to write tools which parse the output, and also make it slightly easier to copy / paste terminal output :)
I can find the relevant code and patch it but I was wondering specifically if this will cause breaking changes in the toolchain

Hmm, looks like this issue is more appropriate in nix/. Moving it over


Update rooms.js

Added groundFloor, speakers, and computers mapping for building rooms.

@Gaelan Gaelan said this is implemented this backwards to the current setup, so we removed it.


add support for routing

@timglabisch what did you mean with routing? Could you provide a description or could we close this issue?


Remeasure block decorations if content changes

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions

Description of the Change
Since #13880 the height of a block decorations doesn't get recalculated anymore if the content changes. This brings back the behavior of Atom 1.18. For more information on how to reproduce and GIFs see #14836
Alternate Designs
I considered adding mutation observers to decoration-manager but this way we have a finer control over which decoration to schedule for remeasurment.
Why Should This Be In Core?
This fixes a :bug: 😄
Benefits
This makes block decoration rendering consistent with the way it was in Atom 1.18
Possible Drawbacks
I don't know how performance critical multiple MutationObserver's are.
Applicable Issues
Fixes #14836
/cc: @nathansobo @as-cii

Thank you for the pull-request, @lgeiger! 🙇
The reason why we decided not to automatically remeasure items when they change can essentially be attributed to performance, because we were unclear on the cost of observing a potentially large number of DOM nodes (e.g. see https://github.com/atom/atom/pull/13795).
On the other hand, users of this API should know in advance when they are about to change a decoration and, as such, can manually invalidate its dimensions and let Atom re-measure it. I am honestly quite surprised that it used to work in Atom 1.18, as I don't remember adding any code that supported this behavior.
With that in mind, does using invalidateBlockDecorationDimensions work for your use case? If not, can you describe a scenario where it wouldn't be applicable?
Thanks!

Thanks for your answer, I understand the performance considerations.
I didn't know of invalidateBlockDecorationDimensions since it isn't documented anywhere (in fact the entire TextEditorElement class isn't documented) and it used to work in Atom 1.18, so I guess package Authors aren't aware of that change. It might be good to mention it in the release notes, since this may break other packages as well.
This change requires quite some refactorings in Hydrogen, but you're right we can use Reacts onDidUpdate handler to schedule a recalculation and don't need to rely on MutationObservers.
I'll try it in the next few days to see if this solution is applicable.

I tried to manually invalidate the block decorations in Hydrogen (We use React so I simply made us of onDidUpdate).
While this works for most of the cases, some outputs may allow users to dynamically change DOM dimensions. Here is a example where this is the case:

If this PR isn't accepted we'll still need to use MutationsObservers inside the package code base, at least I don't know of any better way of doing this. I can imagine this beeing a problem that's also relevant for other packages.
@as-cii What about adding another decorationParam for configuring if a block decoration should be automatically invalidated if the dimensions change?

Thanks for providing this scenario @lgeiger: now I am definitely on board with remeasuring block decorations when their size changes. I think we could take a slightly different approach, which should also be more efficient. In particular:

Add a ResizeObserver to on-screen block decorations. When a block decoration goes out of screen we should stop observing it. Viceversa, when a block decoration appears on screen we should register a new ResizeObserver.

One thing to experiment with here is to reuse the same ResizeObserver and simply call observe/unobserve, as opposed to creating a new observer for each decoration.

When getting a resize callback, call invalidateBlockDecorationDimensions.

Another thing that might be worth experimenting with here is to avoid re-measuring the block decoration's item through invalidateBlockDecorationDimensions and, instead, simply update lineTopIndex with the values provided by the ResizeObserver callback.

I can provide more guidance on this if you are interested in changing this pull-request to do what I described above! 👍

@as-cii Thanks for the great suggestions!
I think point 2 should be fairly easy to implement. Using a global resize observer may require more code change, I'll take a look to see if I can make it work. Thanks for your help 👍

I tried using a global ResizeObserver in ecd24ade6f60239ed23665957e2c69971ecd9891.
Unfortunately calling this.scheduleUpdate() inside the observer callback causes a re-measurement and therefor continuously calls the callback until the block decoration is remove or moved off screen.
I tried using this.updateSyncAfterMeasuringContent() as well which is better but the callback still gets called far too many times (e.g. on scroll, editor resize or mouseover events).
@as-cii Any suggestions?

Closing in favor of #14929


first attempt at downloading & formatting temperature data

When I run the changed lines I get en error "Peer certificate cannot be authenticated with given CA certificates." How were you able to get around this?

I'm not sure - i never got that error. We can change it back if it continues to not work!


Implemented a new feature that Hashie::Mash.load accepts a Pathname object #331

https://github.com/intridea/hashie/issues/331

This code is good.
I want to leave this open for others to comment, in theory the caller can just do to_s on the Pathname and we won't need this merged. I don't see many gems supporting Pathname directly, I am not convinced we should.

@dblock thank you very much for taking your time to review the code. I hope to see others' feedbacks or opinions.

@gipcompany why do you feel that we need to be able to handle Pathname objects instead of a plain string?
Can't you just do Mash.load(Pathname('your_path').to_s) ?

The only thing I'm not a fan of is the switching based on class. An alternative to this would be to just always call #to_s on the parameter, a la:
@file_path = file_path.to_s

This doesn't break the current implementation and allows you to use Pathname or any other type of file pointer that responds to #to_s. Since we're expecting a string here anyway, it's semantically the same but expands the usable parameter space.

The issue with things like to_s is that you start opening files by passing garbage in, and that is bound to give us some security issues somewhere in a stack of things at some point.

To bad this didn't get merged. I subclass Mash just to get this... o well.
I also don't understand the argument about passing garbage in. If it resolves to an existing filename, then apparently it isn't garbage, and in the end the user chooses what they pass in... and as far as I'm concerned it's their responsibility not to pass in "garbage" that resolves to "/etc/shadow". In any case files here should be opened read only, so what's the security implication?
I think  it makes sense that code that operates on files can at least take in Pathname instead of a string. But personally why not everything that responds to #to_str, since it's up to the user to send something sensible in. If you want to be sure that it is a path, and not garbage, it's a small fix:
path.kind_of?( Pathname ) and path = path.to_s

@najamelan I think you're right, and I am ok with the code as it stands. @michaelherold LMK how you feel about this?
file_path.is_a?(Pathname) ? file_path.to_s : file_path

This should be rebased.

LGTM. :+1:

@gipcompany sorry for the delay, see above, would appreciate if you could help wrap this up

@dblock Thanks

@dblock let me see. I would like to catch up with the current status. Should I resolve this PR not to be conflicted and re-push it?

Lets fix the build. ActiveSupport 5 is in the way now I think with older rubies, just add ~> 4.x into https://github.com/intridea/hashie/blob/master/Gemfile#L15 for now, that should fix it.

Try to rebase this against master. You should end up with a single commit here. Git can be fun.

Merged via https://github.com/intridea/hashie/commit/bbafaded9d5d8619d5000a3a10c17d66ee8830a0.

Wow, this being merged in public is my first time experience. Thank you everyone for each support and thank you, @dblock for your continuous support.

Also hashes in commits that contain dictionary words, like fade in bbafade are a big deal, congrats on your first open source contribution!

I'm sorry to arrive a bit late with this, but as I'm implementing something similar, I came to the following conclusion:
Pathname, File, but also Dir all implement #to_path which gives the string path... In some sense it seems more logical to check for objects responding to #to_path and to use that instead of #to_s. It would require a check for directory? if you don't support loading all files in a directory though.

You have some good points @najamelan, feel free to PR an update!


feat: support immutable interfaced target types

support not only ImmutableList<T> but also the matching interfaces IImmutableList<T>

Codecov Report

Merging #365 (578124f) into main (685d6ee) will increase coverage by 0.06%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #365      +/-   ##
==========================================
+ Coverage   91.91%   91.97%   +0.06%
==========================================
  Files         126      126
  Lines        3921     3940      +19
  Branches      544      548       +4
==========================================
+ Hits         3604     3624      +20
+ Misses        210      209       -1
  Partials      107      107

Impacted Files
Coverage Δ

...iptors/MappingBuilders/DictionaryMappingBuilder.cs
86.36% <100.00%> (+0.31%)
:arrow_up:

...iptors/MappingBuilders/EnumerableMappingBuilder.cs
94.51% <100.00%> (+0.43%)
:arrow_up:

src/Riok.Mapperly/Descriptors/WellKnownTypes.cs
96.15% <100.00%> (+2.40%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Refactor work order constructors.

This PR refactors the WO constructors so that:

WO itself is self-contained.
Worker does not need any more info to execute WOs.
Align with our goal to add WO serialization for the distributed version.

@cramja @saketj Check out CreateIndexOperator.[ch]pp (merged in #42) for code style fixes, although I probably should do so in a separate PR.

Thanks @zuyu. Will also take note of IWYU conformity in future PRs too.

@hbdeshmukh Any comments on the PR? Looks good to me.
@zuyu Sorry but the master has moved, so this branch has conflicts. Can you resolve and we should be ready to merge this PR.

@pateljm Please take another look. Thanks!

Looks good. Thanks @zuyu for this PR. Thanks @hbdeshmukh for the review. Merging now.


Free trial period for in-app-purchase

It might not be the responsibility of this library, however I wonder if anyone have implemented the free-trial period for in-app-purchase with this lib? (For example check if the free trial period is over, etc)  Or, simply just set up the free-trail period in iTunes-connect and forget it, don't have to code anything in react native side?
My understanding is (auto-renewal case):
(1) FIRST, user have to 'purchase' not matter the free trial period is set or not.
(2) if the app is configured to have free trial period, THEN user can use the app in that free trial period without being charged
(3) when the free trial period is over, apple's system starts to auto renew user's purchase, user still can use the app, but now user will get charged from then on.
I'l appreciate any experiences if you have some to share. Thank you! :-)

There is nothing to set up in this Library, so I am closing this.

@kawashimaken It would be nice if you can send a PR to the readme for with the setup you did. others might benefit from that.

It's my pleasure!
https://github.com/chirag04/react-native-in-app-utils/pull/83


More details on Allowed and Not Allowed

Worked with Defender AV PM to explain what Allowed and Not Allowed do for the following CSPs:
Defender/AllowArchiveScanning
Defender/AllowBehaviorMonitoring
Defender/AllowCloudProtection
Defender/AllowEmailScanning
Defender/AllowFullScanOnMappedNetworkDrives
Defender/AllowFullScanRemovableDriveScanning
Defender/AllowRealtimeMonitoring
Defender/AllowScanningNetworkFiles
Defender/AllowUserUIAccess

May I suggest either a comma or a dash between the existing Allowed / Not allowed text and the added descriptions? I think adding a comma or dash (turns into en dash on the page) would put the focus back on the values 0/1 as well as showing that the added text is additional explanation to the Allowed / Not allowed text.

@manikadhiman The PR that provides further details on "allowed" and "not allowed" for certain CSPs in the Policy CSP - Defender page is available for review. Thanks.

#sign-off  PR is ready to merge to master/live.

Hi @MandiOhlinger, Thanks for the updates. Could you please incorporate edits suggested by @illfated? Thanks!

@ManikaDhiman No - I don't feel comfortable making those changes. I use these CSPs, but I'm not with that specific doc team.

@MandiOhlinger. No worries. I have made the edits. Thanks for updating this doc!

👍 Good job. Could not have picked a better solution myself.


Two successful messages about moving

When moving a record to the first/last position

Heya! Thanks for reporting this. It's now fixed in version 3.3.0. Good luck!


iOS v15.4 and above not showing over tabs when inner tab was press in transaction details

https://explorer.xpxsirius.io/#/tx/C1FE6677A39DF3D04DF4E1DC05381E1608185A929ED72FCC6CEBEDB26E0F4C20

tested on v16.7.2


[SPARK-31784][CORE][TEST] Fix test BarrierTaskContextSuite."share messages with allGather() call"

What changes were proposed in this pull request?

Change from messages.toList.iterator to Iterator.single(messages.toList).
Why are the changes needed?

In this test, the expected result of rdd2.collect().head should actually be List("0", "1", "2", "3") but is "0" now.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Updated test.
Thanks @WeichenXu123 reported this problem.

ping @sarthfrey @WeichenXu123 @jiangxb1987

Test build #122920 has started for PR 28596 at commit 2be2ca7.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27563/
Test PASSed.

Test build #122920 has finished for PR 28596 at commit 2be2ca7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122920/
Test PASSed.

retest this please

Test build #122949 has started for PR 28596 at commit 2be2ca7.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27591/
Test PASSed.

Test build #122949 has finished for PR 28596 at commit 2be2ca7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122949/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27600/
Test PASSed.

Test build #122959 has started for PR 28596 at commit 2be2ca7.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27601/
Test PASSed.

Test build #122962 has started for PR 28596 at commit 021c0a5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27604/
Test PASSed.

Test build #122959 has finished for PR 28596 at commit 2be2ca7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122959/
Test PASSed.

LGTM

Thanks, merged to master/3.0 !

Test build #122962 has finished for PR 28596 at commit 021c0a5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122962/
Test PASSed.

thanks all!


kamel run runtime/examples/Sample.java fails with java.lang.NoClassDefFoundError

Exception in thread "main" org.joor.ReflectException: java.lang.reflect.InvocationTargetException
| at org.joor.Reflect.on(Reflect.java:781)
  | at org.joor.Reflect.call(Reflect.java:463)
  | at org.joor.Compile.compile(Compile.java:73)
  | at org.joor.Reflect.compile(Reflect.java:77)
  | at org.apache.camel.k.jvm.RoutesLoaders$2.load(RoutesLoaders.java:90)
  | at org.apache.camel.k.jvm.Application.main(Application.java:34)
  | Caused by: java.lang.reflect.InvocationTargetException
  | at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  | at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  | at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  | at java.lang.reflect.Method.invoke(Method.java:498)
  | at org.joor.Reflect.on(Reflect.java:777)
  | ... 5 more
  | Caused by: java.lang.NoClassDefFoundError: etc/camel/Sample (wrong name: Sample)
  | at java.lang.ClassLoader.defineClass1(Native Method)
  | at java.lang.ClassLoader.defineClass(ClassLoader.java:763)
  | at java.lang.ClassLoader.defineClass(ClassLoader.java:642)
  | ... 10 more

Same here. @lburgazzoli wdyt?

I need to check how the things are compiled now, can you try to set the class package as etc.camel ?

@lburgazzoli I tried with package and it's working fine, but don't know why this default package?

It is because I've changed how the integration is given to the runtime images: before it was included as a resource, now it is on the file system and the class name is computed from the path, I need to re-think how this is computed, maybe using an env var to tell the image the base path for integrations.

This seems to be fixed.

Still happening...


Make istio-remote an Istio Helm values installation

Currently istio-remote (for VPN connectivity and Split Horizon EDS) is a separated Helm chart/release.
We can achieve the same goal with having another values within the Istio chart tailored for the remote clusters installations.
What we gain:

Better user experience
Simpler and less confusing docs
Easier to maintain and avoids (templates) duplications
Get rid of the need to create the charts packages (helm dep update) by moving the subcharts back to the Istio release

We can remove the istio-remote chart and move subcharts back to istio or we can keep the istio-remote and mark it deprecated. Current PR removes it.

per @ymesika MC istio-remote chart is currently broken as it is missing some of the latest update from istio chart, thus it is critical for us to get this fixes in 1.1
cc @wenchenglu @duderino

Pending PR: https://github.com/istio/istio/pull/11307

@ymesika when this is done, please update the Setup the remote cluster section in https://preliminary.istio.io/docs/examples/multicluster/split-horizon-eds/  It currently references the istio-remote folder


[package request] WebGrab Plus

Can someone please add a 'WebGrab Plus' package here (to streamline installation for TVheadend users)
WebGrab Plus manual setup instructions
http://www.webgrabplus.com/documentation/installation/synology-nas
http://www.webgrabplus.com/node/22
http://webgrabplus.com/documentation/interact-your-pvr-software/tvheadend

Hello. Not the right project to submit such request. Work is in progress, please follow https://github.com/SynoCommunity/spksrc/pull/3639


Issue/5780 attributes tab

Description
Added the attributes tab to the instance details page.
it features a tree table which can be sorted, expanded, collapsed.
It features a readonly Json-code viewer (the expert mode will be added in a next task)
It features a compare tool to compare different version and different attribute sets.
closes #5780
https://github.com/user-attachments/assets/c56ecbeb-3d61-49ab-81c2-05a69f089071

Processing this pull request

Merged into branches master in 4ab0b2e004b509721ae8bb4373e93c2b3b38f361


feat: mark optional hyperparameters as keyword only

Closes #278.
Summary of Changes

Mark optional hyperparameters as keyword-only
Add guideline to consider marking optional parameters as keyword-only

🦙 MegaLinter status: ✅ SUCCESS

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ MARKDOWN
markdown-link-check
1

0
0.92s

✅ PYTHON
black
11
0
0
0.7s

✅ PYTHON
mypy
11

0
1.69s

✅ PYTHON
ruff
11
0
0
0.04s

✅ REPOSITORY
git_diff
yes

no
0.03s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by

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

v0.12.0
GitHub release

Your semantic-release bot :package::rocket:


Revert "update engine to get fix for dartbug.com/26550 (#4289)"

This reverts commit 215e1b89edd455a0d6f214038fea7d70ba05ceca.
It broke the Mac build: https://build.chromium.org/p/client.flutter/builders/Mac/builds/1097

Landing this to clear up the tree.