Support Get-PSHostProcessInfo and Enter-PSHostProcess on non-Windows

PR Summary
There a some differences in support of named pipes for Windows and non-Windows.  Need to see if we can get rid of the native code invocations and replace with just the managed code now that this works on Unix systems.  One difference is that only Windows supports Message type, but this doesn't seem to be a big factor.  Also, named pipes on Unix have a 104 character path limit.  On macOS, the $env:TMPDIR (on my system) is already 49 characters; corefx adds 12 more.  Since AppDomainName isn't really used, changed it from DefaultAppDomain to None to shorten the name.  Need to keep it since Windows PowerShell expects it.  Changed starttime part of pipe name to 8 hex characters which is to provide uniqueness to the pipe name.
Fix https://github.com/PowerShell/PowerShell/issues/5110
Possibly Fix https://github.com/PowerShell/PowerShell/issues/6435
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

@SteveL-MSFT
Regarding the server named pipe dispose, this is done in WindowsPowerShell via the CLR AppDomain unload event, which is not supported on CORECLR.  However, you can use the AppDomain.CurrentDomain.ProcessExit event that we use for AMSI un-initializing.  Unfortunately, these CLR events are not reliable, but they are better than nothing.
The start time added to the filename is intended to make the named pipe instance unique, to prevent a DOS or man-in-the-middle attack.  We will want to keep something like this, even if it means using something shorter.

I love when a PR has more deletions than insertions but more functionality

As for the domain name, do we really need it for the pipe name? Is multiple domain is going to be supported in .NET Core 3.0?
I have no problem keeping it for now until we get an accurate answer for the app domain support.

@daxian-dbw best I can tell, AppDomains are deprecated in .Net Core and not in the roadmap I can see for 3.0.  So it may be fine to remove it for non-Windows.  We'll have to keep it for Windows so that Windows PowerShell and PowerShell Core interop still works however.

To retain interoperability between Windows PowerShell and keep the code similar between Windows and non-Windows, I'll shorten the DefaultAppDomainName to 'none' but will keep it in the pipe name.

@SteveL-MSFT I forgot about maintaining compatibility with Windows PowerShell.  We definitely need to ensure we can still list/parse/connect to those hosts.

Ugh, with the changes I've made since, I can't connect to/from Windows PowerShell and PowerShell Core.  Debugging.

Looks like DefaultAppDomain is necessary on Windows as there is a check for that string and the starttime needs to stay as filetime integers on Windows to support interop between Windows PowerShell and PowerShell Core.  I believe everything else is addressed.  @PaulHigin  can you take another look?

@SteveL-MSFT  Yeah, for Windows we need to keep the DomainName to be able to connect to WindowsPowerShell host, and that means keeping the default domain name for Windows platform pipe names.

@PaulHigin yes, the recent commit has the same DefaultAppDomainName and starttime format on Windows and I verified manually it works from and to Windows PowerShell

@PaulHigin good suggestion.  Let me add a test for Windows PowerShell interop.

@SteveL-MSFT, there are tests about Enter-PSHostProcess failing in AppVeyor, can you please take a look?

@daxian-dbw it appears to be a problem with redirected input with PSReadLine and only on AppVeyor.  We're moving off AppVeyor (eventually), so I think we can ignore this otherwise we'll have to special case this by perhaps removing PSReadLine for this test

Given @SteveL-MSFT's clarification on the AppVeyor test failure, I'm merging this PR.


Blur event firing on toolbar click. Bookmark not maintained.

Hello! Not sure this is exactly a bug as it's just started occurring when I moved my toolbar element.
I'm having a bit of an issue with the blur event firing when I click on a toolbar item. This doesn't always happen which is odd. On normal command buttons it mostly works fine (though sometimes i'll click bold, it'll make the text bold, and THEN blur).
I have some custom 'dialogs' for font family and font size, they're still within the specified toolbar element but when I click them, the editor blurs before I can actually run a command to format the text. Normally this should be ok as the bookmark is saved for when the commands are run, however sometimes (and not always) it doesn't remember the bookmark, so when I run the command, it just inserts at the start of the editor,
Is there a set of specific classes that are sought after to not blur? Happy to look at the code specifically that stops toolbar from causing the blur and debug it, but wasn't sure where to start.
I'm not sure if it's related but I sometimes get this error when it blurs on the 'bold' command button in the toolbar;

Any suggestions / similar issues welcome!

Still haven't been able to resolve this issue.
If there's some more information I can provide, I'm happy to do so.

I seem to have resolved this issue (mostly - still think there's a couple of minor issues).
After heavily trawling through the examples and trying to replicate it, I had to add the following mousedown event listener function to anything that is in any of the toolbar elements;
function( event ) {
    event.preventDefault();
    var b = editor.composer.selection.getBookmark();
    setTimeout(function() {
          editor.composer.selection.setBookmark(b);
    }, 0);
};

This may already be documented somewhere (as it is in the example code), but if not, would be good to put something in about building custom toolbars that this may be necessary.


⚠ Bofrost.be has degraded performance

In b83453d, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1055 ms

Resolved: Bofrost.be performance has improved in 666db84.


Failed to get http response, status code: [403]

I have 2 consecutive device test tasks. The first one is working with success but the second one is failing with this error.
Failed to get http response, status code: [403]
Why is this error happening?
First successful test;
+------------------------------------------------------------------------------+
| (10<EMAIL_ADDRESS>                              |
+------------------------------------------------------------------------------+
| id: virtual-device-testing-for-android                                       |
| version: 0.9.7                                                               |
| collection: https://github.com/bitrise-io/bitrise-steplib.git                |
| toolkit: go                                                                  |
| time: 2017-10-10T16:19:43Z                                                   |
+------------------------------------------------------------------------------+
|                                                                              |
INFO[16:19:43] Start installing (golang) with apt-get
INFO[16:19:43]  * [OK] Step dependency (go) installed, available.
Configs:

ApkPath: /bitrise/deploy/app-development-debug.apk
TestTimeout: 900
DirectoriesToPull:
EnvironmentVariables:
TestDevices:

Model         API Level   Locale     Orientation
NexusLowRes   24          portrait   en

AppPackageID:
TestType: instrumentation
TestApkPath: /bitrise/deploy/app-development-debug-androidTest.apk
InstTestPackageID:
InstTestRunnerClass:
InstTestTargets:
Upload APKs
=> APKs uploaded
Start test
=> Test started
Waiting for test results
Validating
(1/1) running
(0/1) running
=> Test finished
Test results:
Model         API Level   Locale   Orientation   Outcome
NexusLowRes   24          en       portrait      success
|                                                                              |
+---+---------------------------------------------------------------+----------+
| ✓ |<EMAIL_ADDRESS>                    | 268 sec  |
+---+---------------------------------------------------------------+----------+

Second failed test;
+------------------------------------------------------------------------------+
| (11<EMAIL_ADDRESS>                              |
+------------------------------------------------------------------------------+
| id: virtual-device-testing-for-android                                       |
| version: 0.9.7                                                               |
| collection: https://github.com/bitrise-io/bitrise-steplib.git                |
| toolkit: go                                                                  |
| time: 2017-10-10T16:24:10Z                                                   |
+------------------------------------------------------------------------------+
|                                                                              |
INFO[16:24:10] Start installing (golang) with apt-get
INFO[16:24:10]  * [OK] Step dependency (go) installed, available.
Configs:

ApkPath: /bitrise/deploy/app-development-debug.apk
TestTimeout: 900
DirectoriesToPull:
EnvironmentVariables:
TestDevices:

Model    API Level   Locale     Orientation
Nexus4   21          portrait   en

AppPackageID:
TestType: instrumentation
TestApkPath: /bitrise/deploy/app-development-debug-androidTest.apk
InstTestPackageID:
InstTestRunnerClass:
InstTestTargets:
Upload APKs
Failed to get http response, status code: [403]
|                                                                              |
+---+---------------------------------------------------------------+----------+
| x |<EMAIL_ADDRESS>(exit code: 1)       | 1.33 sec |
+---+---------------------------------------------------------------+----------+
| Issue tracker: ...se-steplib/steps-virtual-device-testing-for-android/issues |
| Source: ...thub.com/bitrise-steplib/steps-virtual-device-testing-for-android |
+---+---------------------------------------------------------------+----------+

Hi,
Did you turn on the Virtual Device Testing for that app? You can do it on the Settings tab of the app on bitrise.io - guide/blog post: https://blog.bitrise.io/introducing-solid-and-snappy-virtual-device-testing-for-android

@bkmalkoc did the instructions above fix the issue (to enable Virtual Device Testing for the app)?

Closing as we did not receive any update / confirmation. @bkmalkoc feel free to comment and we'll reopen this ticket! :)

Hi Viktor,
Yes I have absolutely it is turned on. That's how the first test got successful. Unfortunately not the second one, second test is failed. What could be the reason?
The reason for creating second virtual test step is I want to run on different devices. Instead is there a way to define two(or more) devices in one virtual device test step?

That works now. Thank you for solution. @viktorbenei

Any time @bkmalkoc ;)


fix ci

Description
Backports: https://github.com/opensearch-project/reporting/pull/737 to 2.x line since the upstream breaking changes were backported to the 2.x line. Also updates some dependencies to make the build pass.
Upstream changes: https://github.com/opensearch-project/OpenSearch/pull/8540
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

With a change like this, feels like there is some common change across all the files. Could you please add to the description so I can know what is the change and the motivation?

@pjfitzgibbons updated the description of the PR


Typo "Azure container instances"→"Azure Container Instances"

https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/index-hosting
#PingMSFTDocs

#sign-off

The article in this PR no longer exists in the main repo. I'll close the PR.


Different symbols have the same name and therefore the same anchor

It's possible to have functions, methods, operators etc. with the same names, but different types. One example would be the following class declaration:
public class SomeClass {

    public func someMethod(arg: String) {

    }

    public func someMethod(arg: Int) {

    }
}

This creates symbols with exactly the same name in the documentation:

And for both methods we then create the same anchor #someclass.somemethod(arg:), therefore creating invalid HTML and making it impossible to directly link to the second method.

The same happens when functions differ only in generics:
public class SomeClass {
    public func onMessage(action: () -> Void) {}

    public func onMessage<T>(action: (T) -> Void) {}
}


slurm submiited job failed silentely

Snakemake version
6.7.0
Describe the bug
The stdout of snakemake (as show below) remains as the job is normal while actually, the job has already failed.
Minimal example
Building DAG of jobs...
Using shell: /usr/bin/bash
Provided cluster nodes: 1
Job stats:
job          count    min threads    max threads
---------  -------  -------------  -------------
all              1              1              1
qc_before        9              1              1
total           10              1              1

Select jobs to execute...

[Wed Sep  1 16:04:01 2021]
rule qc_before:
    input: /home/mewu3/data/PV_L003_R1.fastq.gz, /home/mewu3/data/PV_L003_R1.fastq.gz
    output: /home/mewu3/test/reports/qc/PV-L003_R1.html, /home/mewu3/test/reports/qc/PV-L003_R2.html
    log: /home/mewu3/test/logs/PV-L003_qcBefore.log
    jobid: 9
    wildcards: SAMPLE=PV-L003
    resources: tmpdir=/tmp

Submitted job 9 with external jobid 'Submitted batch job 39550942'.

Additional context
The failure message was received by mail.

This is likely not a snakemake bug but rather an issue with your cluster (slurm) profile/interface. Are you using a profile created using this cookiecutter? What is the snakemake command you are using and what is the final output? or does it just hang thinking the job runs indefinitely?

Command-line: atlas run --profile cluster --jobs 3
And yes the profile was created using cookiecutter.

Could it be that your log output directories don't exist? That makes slurm fail silently. I'm talking about the directories that follow #SBATCH etc
What's your submit script? This is what mine looks like:
#!/bin/sh

#SBATCH --output=log/%j.out                 # where to store the output ( %j is the JOBID )
#SBATCH --error=log/%j.err                  # where to store error messages

# activate conda environment
source $HOME/miniconda3/etc/profile.d/conda.sh
conda activate nextstrain

{exec_job}

Hey, I'm the developer of the above-mentioned snakemake pipeline and cluster profile.
I updated the cluster profile. Now upon error, the clusterwrapper communicate the error to the snakemake via stdout.
Before I printed everything to the stderr.
Ideally, snakemake would also listen to the stderr.


feat: add maintain() method for CacheSessionPool

Adds a public method for Spanner\Session\CacheSessionPool to maintain cached sessions and keep them alive.
Related to issue #2411. Implementation details are described here.

@larkee it would be great if you could also look at this PR.


Implement a way to pass a string as a query argument to perform vector search

As part of working on queries that involve ext::ai capabilities a user would need to pass text embeddings as query parameters. A text embedding is a numeric representation of text that enables us to semantically compare different pieces of text using math.
Embeddings are obtained by passing text to an API like this. They are then passed into a query as an array<float32>. It is ~2000 elements long, however, which makes it difficult to input using existing UI widgets.

Current workflow:

Set up some way externally (e.g. a Python script) to do the API request to transform text into an embedding
Copy the embedding, paste it as a string parameter, then parse that in the query into array<float32>...? Haven't tried that personally, not sure if that'd work.

Desired workflow:

Once the UI detects an array<float32> parameter, it offers a choice between the normal widget and a text embedding generation widget.
When selecting the second one, the user can pass their text string as an input. They can also pick which of the configured embedding models to use (if multiple are configured in the AI tab).
The UI handles the API request and passes an array<float32> to the DB.

@jaclarke, @1st1 also asked that you have a chat with him about this when you get around to it.


Assertion failed: (thread_id_key != 0x7777), function find_thread_id_key, file coroutine.cc, line 134.[BUG]

Describe the bug
Getting the above error when running packer start on a new M1 macbook pro.
To Reproduce
Steps to reproduce the behavior:

yarn global add @hayes0724/shopify-packer
packer start

System (please complete the following information):

OS: OSX 12.0
Node Version: v16.13.0
NPM Version: 8.1.0

Packer (please complete the following information):

Version v2.2.0

I have the same error and i tried to install as new project, as well as an existing one

@Teunel
Have you tried installing node 14 or 15 instead? I run 14.17.3 or 15 stable.

Official message @ fibers page:
Update [April 13th, 2021] -- Fibers is not compatible with nodejs v16.0.0 or later. Unfortunately, v8 commit dacc2fee0f is a breaking change and workarounds are non-trivial.
/Downgrading to v15 is not recommended, because doesn't have LTS - Long Term Support/
So I just used to downgrade to the latest 14.18.3 LTS and the issue is resolved.


[Enhancement] Add proxy support

While testing our own addon's proxy support I discovered that inputstreamhelper does not adhere to Kodi's proxy configuration, which breaks playing DRM streams.
It is not that hard to make addon's using Requests to use Kodi's proxy settings. You can find our implementation at: https://github.com/pietje666/plugin.video.vrt.nu/pull/87
The testing methodology I used was to disable the default gateway (or in LibreELEC, configure a non-existing gateway) and then test your local proxy and see if everything works as expected.

Hi!
Is there any actual explanation why this is needed to be done by the add-on? You would assume that Kodi would use its proxy settings system wide, because I can imagine that there's a lot of add-on that won't handle this proxy stuff themselves.

Well, everything that uses internal functionality (like ListItem) does use global Kodi settings, but you are using the python Requests library independently, and it doesn't know about Kodi's proxy settings.
This patch gets Kodi's proxy settings and applies them to your requests-invocation.
And indeed, there are quite some addon's that do not have (full) proxy support (netflix, youtube, etc), or have their own proxy support (retrospect). Or our own vrt.nu addon which did use Kodi's proxy for everything that was handled by Kodi, but not for the stuff that used the requests library.

BTW Feel free to configure a proxy in Kodi and test it yourself, your addon will not be using the Kodi proxy. So DRM playback will simply fail in environments that require a proxy to be used.


Replace sidekiq with delayed_job

We don't need the efficiency of a Redis-backed job queue, since we really don't process that many jobs.
Switching to delayed_job would remove the dependency on Redis.

I can tackle this.


s390x: Add support for atomic operations (part 1)

This adds full back-end support for the Fence, AtomicLoad
and AtomicStore operations, and partial support for the
AtomicCas and AtomicRmw operations.
The missing pieces include sub-word operations, operations
on little-endian memory requiring byte-swapping, and some
of the subtypes of AtomicRmw -- everything that cannot be
implemented without a compare-and-swap loop.  This will be
done in a follow-up patch.
This patch already suffices to make the test suite green
again after a recent change that now requires atomic
operations when accessing the heap.

FYI @cfallin - as discussed in our call yesterday.


Requested File

Tabula API version: <IP_ADDRESS>92117
Filename: Test 2016 Adopted Budget.pdf
Internal Server Error (500)
   Request Method:
    POST

Test 2016 Adopted Budget.pdf
    Request URL:
    http://<IP_ADDRESS>:8080/pdf/eeef67c8c1723baa349e9f9a6087027e5091b6f1/data

Hi @hjhirsch,
Your document contains images of a document. Tabula only works with PDFs that contain text.
Nevertheless, you shouldn't have gotten an error.


Feature: Support batch jobs [ie lists of files]

Hi there, I'd like to request that AzCopy add a 'batch job' feature. The scenario here is that I have a large quantity of individual copies that need to be made, but I'd like to run only one instance of AzCopy which treats the group as a single job.
Proposed approach:
.\azcopy batch <batch path>
Where <batch path> points to a text file. The text file would contain the exact same parameters that would be used if executing azcopy directly, one per line. Say I have three files I want to copy as one job, the file could have:
cp "C:\file1.txt" "https://account.blob.core.windows.net/container/file1.txt" --content-type "plain/text"
cp "C:\file2.xml" "https://account.blob.core.windows.net/container/file2.xml" --content-type "application/xml"
cp "C:\file3.bin" "https://account.blob.core.windows.net/container/file3.bin"

The three files would be copied in parallel, as one job.
In my real-life scenario I have ~2M select files that I need to copy and ideally treat as one job.
Thanks!

Hi @jcarnahan, thanks for reaching out!
To understand your scenario better, is there any pattern in your 2M files? How are you selecting them?
I'm asking this because AzCopy has pattern matching available, so you could easily select files/directories matching the given pattern, just like the copy command on linux. Please refer to ./azcopy cp --help to see examples.

The pattern matching feature will not assist me with my scenario. The files I would like to copy were all created with a random GUID in their name, and reside in the same container + virtual directory as ~10M other files also using random guids in their name.

Hi @jcarnahan, is it possible to just upload the root folder? It sounds like all your files (with GUIDs as name) are stored together, right?
If I'm understanding your scenario correctly, you are generating the files with GUIDs, and you want to upload them as they become available, right? Maybe that's why you couldn't upload the root folder?

Let me add a bit more detail - my storage account has one container, and inside that container I have ~12M blobs, all which use the same naming pattern, which is a unique GUID. There is a specific set of 2M blobs that I want to copy via AzCopy. My ask is for some way that I can point AzCopy at a file that lists all 2M files I want to copy, and have AzCopy them all as one job. I only want to make a copy of this set of blobs once, and ideally without invoking AzCopy 2M times.

@jcarnahan thanks for the clarification, now I understand. I'll discuss with the team and get back to you.

One possible workaround.
If azcopy supports symlinks (resolving symlinks to realpaths), then we could have a directory with just symlinks to files you want to copy -- this would serve as a hacky batch.  You can create these symlinks using a bash/Powershell/etc from any suitable source of your choosing.  @jcarnahan looks like you are on Windows but last I've seen there are symlinks there too.
And actually it is a good question: @zezha-msft  how does azcopy treat synlinks?

Hi @ppanyukov, great suggestion!
On the copy command, there is indeed a flag --follow-symlinks which should accomplish what you are suggesting.

I can visualize how that might work for an upload scenario, but how would
that work in my scenario, where I want to download a large # of select
blobs from my blob storage account?
On Wed, Feb 27, 2019 at 11:37 AM Ze Qian Zhang<EMAIL_ADDRESS>wrote:

Hi @ppanyukov https://github.com/ppanyukov, great suggestion!
On the copy command, there is indeed a flag --follow-symlinks which
should accomplish what you are suggesting.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-storage-azcopy/issues/196#issuecomment-468000733,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzNvoOVZ7r28L3qt1fNQ6vi54xGI-Wyks5vRt6TgaJpZM4aiK7P
.

Hello,
Have you any ETA for this feature ?
I have the same problem. I have a large number of files in an Azure Blob and I need to copy them in another. I can't use wildcard, and I cannot use directory copy.
I have on one side files stored using hash as name 0f/14/487ef487a4874154, and on the other side I need to resolve them with a full path like /some/directory/file.ext
It's would be great to be able to give in a file (or by reading STDIN) the list of files to copy (like source/destination on each line) and then let all the work be done on the server side.

We now have the ability to supply a list of files, but not the ability to rename them.  See https://github.com/Azure/azure-storage-azcopy/wiki/Listing-specific-files-to-transfer

I'm going to close this issue now, because the basic support is there now (i.e. for a list of files, for both upload and download).  Right now, the list-of-files parameter is only documented at the URL listed in my comment above.  In release 10.4, it will be included in the publicly documented list of parameters, as show by "azcopy copy --help"
@socolin if you'd like renaming as a feature, please log that as a separate issue, thanks.
@jcarnahan if you need to specific specific content types, per file, rather than letting AzCopy figure out the type for each one, then please also log that as a separate issue, thanks.

Hi @JohnRusk ,
Quick question/reminder. On MacOS with brew i get the version 10.7.0, but the cli help still doesn't mention this option (at least i haven't found it) . Is this intentional?
$ azcopy --version
azcopy version 10.7.0

$ azcopy copy --help |grep -i list-of-files

Yes, it's intentional. See https://github.com/Azure/azure-storage-azcopy/wiki/Listing-specific-files-to-transfer
(BTW, I'm not actually working on AzCopy any more, but got the notification because you @-mentioned me)

Does this support copying a list of files from blob storage?  Or only a list of files off the local hard disk?

Hi @micktion, it supports all source types.

Hi @zezha-msft, great.  What would azcopy command look like? And what would be the contents of a file if you wanted to copy a list of blobs from blob containers in one azure account to another azure blob storage, using Shared Access Signatures for both the source and destination?  Like do you include the full URL for each blob plus the SAS in the batch file? Are there any examples of this around?

@micktion the flag is explained here: https://github.com/Azure/azure-storage-azcopy/wiki/Listing-specific-files-to-transfer
The command would look like:
AzCopy copy "https://mysourceaccount.blob.core.windows.net/mycontainer?[SAS]" "https://mydestinationaccount.blob.core.windows.net/mycontainer?[SAS]" --list-of-files fullPathToYourTextFile
The file list should only contain the blob names, each located on a different line.

Please give it a try and let us know if you have any more question.

@zezha-msft thanks

Hi @zezha-msft, sorry for reviving an old thread. Just looking to check: does this functionality also apply for other azcopy operations? Specifically, I'd also like to be able to specify the --list-of-files parameter for the rm operation so I can remove the files after the preceding copy has been completed. Thanks.


Add a view that shows current amount of funds in the account

This will be easier to add now that there’s the drawer navigator


1.4.20 Broke downloads pretty much entirely, only downloading a few items

I downloaded 1.4 yesterday and downloaded a few of my favorite artist's DA and Tumblr page. I went to set downloads to run while I was sleeping, but there was an update available, so I updated. Then I went to set my downloads, and each download, despite being of large albums, finished after 10 items or less. What gives? Going to backdate.

Additionally, I cannot completely uninstall the new version of the jar. Even ending the process TREE I was still unable to delete the .jar. To backdate and use the same folder I had to name the new jar ripme1.jar

Correction: I have tested with albums I have already downloaded and moved elsewhere. Nothing is downloading now. Instead, it spits out "download complete" then a red "Null", and that's it. No clue what is causing this, even moved folders and redownloaded the old jar

What urls are you trying to download?

Works perfectly for me when I tried a favs folder of mine. Are you using Oracle Java SE 8 Update 131 which has same bit width as your OS ?
Are you on Window$ 10 ?
If you're on Window$, start Command Prompt as Admin and run this to force shut down regardless of stuck processes (still not guaranteed):
shutdown /s /t 0 /f

On Linux, BSD and other NIX systems you shouldn't need any commands to shut down even if a process is stuck unless it's a very important one; as root run shutdown now.

I downloaded multiple Tumblr and Deviant Art pages on the first non-error'd run. The one I used as a test once I saw downloads were failing was http://deadbones5.tumblr.com/ . Again, what it's doing now is spitting out "download complete" and then "null" after only seconds and without downloading anything. All URLs are doing this now.
As for Windows, I am indeed using Windows 10. As for Java, I dont know what version I was on, but it was working fine initially. I did not update my Java but the errors started anyway, and updating my Java didnt fix it

Tested with a deviant art site and it seems to be working? The first deviant art i tested only downloaded 10 and then errored as well though.

I cleared my Java installs and reinstalled and rebooted, tested the DA site that failed before and it's working. Tumblr still broken but that's a single issue not widespread like I initially had so I'll close this

Updated my post.

@rautamiekka As a matter of being helpful, unless there's some evidence that Windows is fundamentally at fault for something, let's avoid blaming the OS and try to help users come to a resolution. In my experience, Java is just as likely to be inexplicably broken on any platform.


Need More Details On Setting Up Cross Tenant Access For Teams Shared Channels

Need More Details On Setting Up Cross Tenant Access For Teams Shared Channels

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

ID: 20f0db07-fbeb-7a8e-aa94-297b5f808216
Version Independent ID: 0fec134e-c8a3-50ec-7094-31ca70cb42ba
Content: Microsoft 365 inter-tenant collaboration - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/microsoft-365-inter-tenant-collaboration.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@kelleyvice-msft Please help us in resolving the issue. Thanks

In my understanding, documentation for Teams Shared Channels across tenants isn't available at this time as the feature is still in Public Preview. Additional information about Shared Channels is available here: https://docs.microsoft.com/en-us/MicrosoftTeams/shared-channels. You can check https://techcommunity.microsoft.com/t5/microsoft-teams-blog/bg-p/MicrosoftTeamsBlog or subscribe to the RSS feed from the blog for new information.


make install-tests (feature request)

Hello,
would it be possible to include one target in the Makefile in order to install the tests?
Thanks,
Vincent.

Why do you want to install the tests?

In order to not copy them manually to the target. It's more handy if we can do something like "make install-tests" or something like that in order to have the tests installed into (for instance) /usr/share/openblas/{test,ctest,utest}/


Extract PlotAssert library

On Kotlin 1.3 it should be easier.  Let's wait for it.

Filed a bug for Kotlin: https://youtrack.jetbrains.net/issue/KT-29100 (may not be really a bug, I just want to ask at the source how it should be done)


Add support for expression-based index INCLUDE

With the new generic IndexBuilder<TEntity> added in https://github.com/aspnet/EntityFrameworkCore/pull/14641, we should be able to add INCLUDE columns to an index using expressions.

Uh, I have no idea how I missed/forgot this, but it looks like I included this in #699 (see comment). Just ignore this.
One thing that we probably should do is follow the EF team and remove ForNpgsqlHasIndex and advise people to just use HasIndex (https://github.com/aspnet/EntityFrameworkCore/pull/14641).

Uh, I have no idea how I missed/forgot this

That's the good kind of omission/forgetfulness 😆

Are we comfortable with removing the method directly, or should we delegate to HasIndex and put an ObsoleteAttribute on ForNpgsqlHasIndex for now?

I definitely think we should follow EF Core's lead on this one and remove it entirely (3.0 is a major version). Do you want to submit a PR for this?

Do you want to submit a PR for this?

Sure. I don't think we have any usages of this in the code base, so it'll be a remove-only PR 😊


feat(parser): Initial support for parsing flags in JS

Initial support for parsing flags in JS
Add and update JS parsing rules that are affected by the Unicode flag u. Specifically, with the u flag:

\u marks the start of a Unicode escape sequence, otherwise it matches the character u
\p{UnicodeProperty} matches a Unicode property escape, otherwise it matches the character p followed by the string {UnicodeProperty}
a character identity escape \x is only valid if x is a syntax character ^ $ \ . * + ? ( ) [ ] { } | or /. Without the u flag, a character identity escape accepts other characters

Add and update tests for parsing rules that are affected by the Unicode flag u

Sorry I didn't get back to this for a while. I thought I was waiting on you 🙈. Feel free to merge this!

Perfect 👌


stderr output when checking for agent version

If I try to check for the agent version ./sbin/circonus-unified-agentd --version
I noticed that I get the following output:

2022/11/10 08:32:44 proto: duplicate proto type registered: k8s.io.apimachinery.pkg.api.resource.Quantity
circonus-unified-agent 0.2.0 (git: master 0483f4a v0.2.0) built:2022-11-09T20:00:39Z

After digging deeper, I noticed that it seems that the first line (what is bolded) is actually stderr that is being displayed. Would it be possible to not show stderr particularly when simply checking the agent version? It's possible this shows up when trying to call other flags as well.

Thanks @bp88 for pointing this out, we tracked down the source of this stderr message and removed it. The fix should be included in the next release.


Implement edge type enum per algorithm

Currently we give our edges implicitly meaning based on context of the algorithms they are used in.
But edges have specific different meanings in different algorithms. One option would be to find a common superset between those algorithms. Another option would be to have one EdgeType enum class per Algorithm.
This could look something like this:

class PCEdgeTypes(EdgeType):
     DIRECTED_EDGE = "directed"
     UNDIRECTED_EDGE = "undirected"

     @pre
     @on_updated([PCEdgeTypes.UNDIRECTED_EDGE], [PCEdgeTypes.DIRECTED_EDGE])
     @classmethod
     def check_update_of_undirected_edge_possible(cls, node_a, node_b, graph, operations):
         pass

This also means that a PipelineStep needs to explicitly tell what kind of edge types it requires. And that the edge type enum object can be configured at the creation of a model.

solved with hooks.


Business use

I'm planning to use it at work.
I'm curious about security.
Could the code of my company's equipment be leaked?

Hey, it depends on whether you're using a local model or API based provider. If using local models within Ollama everything should stay locally with nothing leaked, but obviously I cannot guarantee it for every decision made by users.
Regards,


Ectoplasma seems too difficult

How is the player supposed to get enough Ectoplasm to make Ectoplasma for stuff like Spectre String? It requires 1000 mb per craft, but melting Ectoplasm only makes 10 mb. Do you just need to set up a spirit spawner and kill them with magic? It seems a bit extreme to need 100 Ectoplasm per Spectre String, especially since Spectre String is used for Fluxed String for Dense ME Cables.

This is specifically in Nightmare Mode btw

@milesman34 sorry for the late answer. once you obtain some ectoplasma from a spirit you can righ-click it on a (vanilla) sapling to create a spectre tree, which can be harvested for more ectoplasma. this process can also be automated as it drops spectre saplings.
The reason it only gives 10mb is, that setting up a farm for a tree isn't really hard to accomplish.
Link to the Random Things Wiki

with version 2.2.0 (waits for approval by curseforge) i also increased the amount of ectoplasm from 10 to 50 mb per item.


Un-added the PR test

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Module build failed: SyntaxError: Unexpected token (2:2)

The boiler project in https://learnnextjs.com does not run due " Module build failed: SyntaxError: Unexpected token (2:2) " error pops up. I have found out it has something to do with my Babel settings but I have not found the exact solution yet.
Expected Behavior:
Hot reload should output a Hello World to the browser window
Current Behavior:
The browser outputs:
Module build failed: SyntaxError: Unexpected token (2:2)

Thanks alluster. I've been spenting hours to trying to resolve the same problem. Finally, my Next works like a charm too, using ur suggestion. Really appreciated.
I'm doing a Udemy course to develop a blockchain project. At the same time, trying to seek alliances from the community, to find collaborators, on a non-profit basis, to develop blockchain solutions for the South-east Asia small and medium enterprise. If the ideas are able to reach proof-of-concept, we can then develop them into for-profit business solutions. Any advice from you?


Image pre-processing about the synapse dateset

While reproducing your multi-organ segmentation work, I noticed that the number of channels after image preprocessing is 1, which does not match the number of input channels (3) set in the source code you provided. Why is this?

Hello, you can check lines 28 to 29 in the vmunet.py file. When the number of input image channels is 1, it will perform a repeat operation.
    def forward(self, x):
        if x.size()[1] == 1:
            x = x.repeat(1,3,1,1)
        logits = self.vmunet(x)
        if self.num_classes == 1: return torch.sigmoid(logits)
        else: return logits


Buil responsive pricing section and footer

Improved the pricing section and footer for the application

thanks for the help!


hanging note on ending screen.

Ending screen (happy ending) has a hanging note in the sound channel 3.

fixed in v20.


Open Widget on right-click

Hi,
I am trying to open a custom widget that i am developing on right-click event on map but struggling to do so. Even google street view widget is not getting opened in this calcite version of CMV.
Can anyone help....!

Not sure if this was broken then or was subsequently resolved but it works today. closing...


HOTFIX: Fixing issue with circle plots

Purpose
Hotfix for drawing circle plots in survey collection
Related Issues
Closes COL-###
Submission Checklist

[x] Included Jira issue in the PR title (e.g. COL-### Did something here)
[x] Code passes linter rules for each file you updated. To lint all files at once, run npm run lint. To just lint one specific file run npx quick-lint-js src/js/<file-you-changed> --snarky.
[x] No new reflection warnings (clojure -M:check-reflection)

this PR is related to the issue raised in #1656


popular tv show section data

Create lambda that calls https://www.episodate.com/api/most-popular?page=1 and saves to a file in s3. Create an event rule to run the lambda once a week.
Save the id, name, perma link, and image thumbnail path for top 20 most popular shows. Save as a json file.

Should be done. Leaving open until UI is done.


Tests for ConstSimplifier arithmetical operations

Hello :octocat:
This adds several property-based tests and some unit tests to the ConstSimplifier class, fixes some bugs on its base class ConstSimplifierBase, and includes some missing simplification scenarios.

A new jar is built in order to reuse tlair's IrGenerator in tla-pp
New errors are introduced for the following scenarios:
a. Division by 0
b. Mod (%) by 0
c. Negative expoents
d. Expoents bigger than an Integer (we used to try and make the operation used doubles, but that was not working)
e. Expoential operations that would overflow BigInt
TLA builder for TlaInt now accepts BigInts as well

The tests are structured like this:

Generate any TLA expression, then combine with operations and parameters that result in a simplification:

any expression with  with <operation's neutral element> should equal the expression itself.

i.e. x + 0 = x

any expression with   with <operation's absorbing element> should equal the same constant result regardless of the expression

i.e. x * 0 = 0, x % 1 = 0
These tests also include assertions like x / x = 1 since the testing structure is the same
There are two groups for this: simplifications that result in 0 and simplifications that result in 1

Generate pairs of numbers, combine them in an arithmetical expression and check that the result of the simplification equals the evaluation of the expression
For exponential operations, use fixed values (in regular unit tests) since the result is highly dependent on the combination of the two values and that's a pain to generate.

Helps with https://github.com/informalsystems/apalache/issues/451 but is not enough for closing it. More tests to come!

[X] Tests added for any new code
[X] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] Documentation added for any new functionality
[ ] Entry added to UNRELEASED.md for any new functionality

My second major comment has nothing to do with the contents of this PR (and is not directed at Gabriela), but concerns the simplifier itself: Why are the simplification methods, which are by all counts transformations, not using TransformationTracker? We're losing the ability to track source information. (this is the reason for all the ex @ .../return ex suggestions, in a tracked system you'd want to avoid copy-creation)

These simplifications are happening in the rewriting rules. If we start tracking changes there, we will create a massive map, for no reason. It probably makes sense to decorate simplification in the optimization pass, but not in the rewriting rules.


Add landing locations on planets

Short Description
When you get closer to planets, cities should appear with specific docking locations for you to visit.
More Details
We will need to create at least a few different types of cities (I imagine floating cities, polluted surface cities, and ultramodern cities). When you approach a city, you can request docking permission, which will provide you with directions to the docking location within the city. We'll also need to set up a camera rotation system (so the camera is facing down relative to the planet), which will help the player orient themselves.
Related Ideas
I'll need to make some soundscapes for the cities as well.

Different types of cities are wrapped into the Location concept in #66, which will make it possible to have multiple landing sites and adaptive states (e.g. finding a crashlanded floating city because the temperature was too low to support balloons).


minimal fix for plugin testing

While developing new plugins, I noticed that once in a while some will not load correctly. After further investigation, it resulted that if the plugin relied on a plugin-type and the plugin-type had a filename that was alphanumerically higher than the plugin - the system reset will fail because the plugin-type will not be added before the plugin.
This PR makes sure that plugin-types are always activated before regular plugins; solving the above issue.

Coverage increased (+0.001%) to 92.894% when pulling 51f53512fa19f99ee25dcf7ef61a4125dd50e696 on antgonza:min-fix-for-plugin-testing into e0428d53912ee228ff635d5cafb4f16a6227222f on qiita-spots:dev.


Absent documentation about configuration

As for me, configuration retreiving is the hardest thing in PHPixie to understand. Documentation about configuration absent, so any analysis can be performed only by source code. Examples:
Local config
I call "local", because this configuration settings are specific to bungle. All example pieces of code ececuted in action (HTTPProcessor). For example, this data will be read from bundles\app\assets\config\pony.php:
$value=$this->builder->bundleConfig()->get('pony.princess')

Or better way to receive two or more values:
$cfg=$this->builder->bundleConfig()->slice('pony');
$value=$cfg->get('princess');

Ok, but this method is not described in documentation. I wanted to use it as i18n tool, retreiving translation strings.
Global config
I call "global", because this configuration settings are common for all bundles. Again, all pieces of code are running in scope of action (HTTPProcessor). For example, this data will be read from assets\config\pony.php
$value=$this->builder->frameworkbuilder()->configuration()->config()->get('pony.princess');

Or better way to receive two or more values
$cfg=$this->builder->frameworkbuilder()->configuration()->config()->slice('pony');
$value=$cfg->get('princess');

But this code is ugly! This $this->builder->frameworkbuilder()->configuration()->config() of anomalous length? It was better to create shortcut, $this->builder->appConfig()->get(...). For myself, I, certainly, did it, here is a solution:
// bundles/app/src/HTTP/Processor.php
	public function __get($name){
		if($name==='bundleConfig')
			return $this->builder->bundleConfig();
		if($name==='appConfig')
			return $this->builder->frameworkbuilder()->configuration()->config();
		if($name==='frameworkBuilder')
			return $this->builder->frameworkBuilder();
		$components=$this->builder->components();
		if(method_exists($components,$name))
			return call_user_func(array($components, $name));
		$trace = debug_backtrace();
		trigger_error(
			'Неопределенное свойство в '.get_class($this).': ' . $name .
			' в файле ' . $trace[0]['file'] .
			' на строке ' . $trace[0]['line'],
			E_USER_NOTICE
		);
		return null;
	}

After this trick, I can get configuration and components like this:
$orm=$this->orm;
$bundleValue=$this->bundleConfig->get('pony.princess');
$appValue=$this->appConfig->get('pony.princess');

So, the pony solution is more beautiful, than pixie, isn't it?
Override configuration
Another mess with configuration in PHPixie. Is there a shortcut method to get hierarchy config data? For example: I wish to create a forum bundle. I put content of this bundle into bundles/forum folder, add link to it into src/Project/Framework/Bundles.php and wish to configure it. A forum has local config, located in bundles/forum/assets/config/forum.php, it contains "default" values, like default page title, default banner and so on. A user may wants to override some of data, like default banner (maybe, depending on selected language or moon's phase). This is like an overlay, but target bundle (forum) shouldn't know about it.
This behaviour was default in Kohana framework: core-specific values can be overridden by module-specific values (the same as bundle) and both of them can be overridden by application-specific values. This works for i18n too. But the great disappointment was that Kohana reads all configuration files at startup and it wasn't very fast. But there were one great advantage: you may see EVERY configuration option in file, copy them into application/config folder and change everything you want (or delete values, that you didn't plan to change, they will be read automatically).
Has PHPixie such convinient instrument? Configuration overlay is slightly different, they mush be merged manually, not automatic.
Assets
Some strange things happens. As I thought, assets is parts of the project, which are not a code (images, css, templates, config and so on). So, the location of such files is very important. But the documentation has many lacks.
For example, in bundles/app/assets/config/templateLocator.php, there is a line:
'directory' => 'templates',

But there are no instruction, why I should use EXACTLY this type of code? Why this:
'directory' => realpath(__DIR__.'../templates'),

will lead to error, because engine cannot find template? This is absolute path to template folder, so it should work perfect!
OK, return to our assets. A code:
$path=$this->builder->assetsRoot()->path();

is not described in documentation, gives us full path to bundles\app/assets/ - good. But if I want to receive global assets, the same code lead to a fail:
$path=$this->builder->frameworkbuilder()->assets()->frameworkAssetsRoot()->path('');

This retreive invalid path: vendor\phpixie\framework\assets/. I wish simple path to /assets.

did you see this:
https://habrahabr.ru/post/301152/

did you see this?:
Yes, I do. More, I've read Updates to configuration..., but it was even more confusing. The main pitfalls:

article don't contain information about framework's version, for which it is suitable. Pity. I can only guess about appropriate version. More, the framework itself has no (or I don't know about it) internal feature, that can tell about it's version (ok, version of each bundle). This can be useful, for example, to print in site's footer: "Powered by PHPixie <IP_ADDRESS>"
articles scattered around: the most useful information is in QuickStart guide, some information should be collected on Components page and even more unstructured information located on habr.
no chances to fix this. For example, there are some places in docs, where no filenames (where presented code should locate) shown: chapter Configuration in ORM article. If docs was on Github, I could fork them, fix and make pull request. But there is only old, PHPixie 2.0's documentation here, it don't worth to be corrected.

TemplateLocator is configured rlative to the assets folder
Hmm, maybe, it's worth to test path? Something like IsAbsolutePath?
frameworkAssetsRoot is the assets folder of the framework bundle
Ok, how to get path to /assets folder "right way"? And why in this case "$this->builder->frameworkbuilder()->configuration()" gets application-specific configuration, but "$this->builder->frameworkbuilder()->assets()" gets framework-specific assets?
By the way, the PHPixie 2.0 architecture was very like to another PHP framework, Kohana. Modules, hierarcial configuration and file system, even facades Cache, DB and so on - are the same. Most likely, this is an accident. For me, such HMVC architecture is very convinient, but quite slow.


perf(lexer): try perfect hash table for keyword lookup

Building on #140 Try a faster hashing function (CityHash) from fasthash.
Since rust-phf don't support custom hasher yet, I wrote our own macros to generate specialized keyword hash tables and tried included the macro in the source code of oxc_parser. However there's some weird issues I can't resolve (It compiles and runs when building the whole workspace but fails to run benchmarks or examples). Therefore I expanded the macro offline and put the file in crates/oxc_parser/lexer/keyword_generated.rs

Seems fasthash cannot build on Windows 🤦♂.

I think hashing on less values is the key here, then the next step is to change the hashing function, and then the phf table.

It seems the performance of match_keyword function is so small that, you need at least a constant 5-10ms change on the typescript benchmark for it to be noticable.

Shall we try and make the token values lazy (for keywords only) and then come back to this?

I'll work on this.

I'm actually running some experiments to remove materializing token values for both keywords and identifiers. I noticed the into_bump_str can already turn an owned string to an allocator-managed reference. I'm currently looking into making a new variantTokenValue::Str(&'a str) and move materialization away from parsing logic.

I changed it on main.

and move materialization away from parsing logic

This may require a lot of work, but you can try. The only problem I have with String<'a> is that all downstream code will become very hard to write.

(you probably already thought of it, but just in case)

TIL! Thank you again for looking into the dark arts of Rust.

    let string: &'static str = unsafe { &*(string as *const str) };

This is making me really uncomfortable, but I'll try it regardless 🤯

I've attached a profiling result generated from pprof-rs for parsing lodash.js. Maybe the time spent on lexer is really trivial so it's not worth all the hassle.
⬇ Time related to lexer in a 30s sample.

I couldn't find a way to display the whole picture nicely... I thought of introducing pprof to the codebase but it seg-faulted on typescript.js

@YangchenYe323 Yes, let's stop here, in conclusion the speed improvement is negligible compared to the rest.
All your work is this still valuable and we learned so much in the process, some of the code is also improved along the way. We also recorded all the process for future references.
Again, thank you so much for trying things out and learning with us on this topic.


Quests, Items, Objects, Units

Added:
Quests:
You Reap What You Sow
Fallen Adventurers
Preventive Strike
Trapped in the Nightmare
Serpentbloom
Down With the Sickness
Preventing Poison
Kodo Hunt
Mother of the Hollow
Troubles From Distant Lands
Trader's Misfortune
Mastering the Arcane
Arcane Arms
A Glittering Opportunity
A Bloody Good Deed
Items:
Country Pumpkin Seeds
Mountain Berries Seeds
Striped Melon Seeds
Magic Mushroom Spores
Squealer Thornmantle's Mane
Sickly Gazelle Flesh Sample
Summer Dew
Life's Dawn
Vulpa Bloom
Moontouched Wood
Crystal of the Serpent
Everchanging Essence
Objects:
Ripe Garden Pumpkin
Garden Berry Bush
Ripe Garden Watermelon
Summer Dew
Life's Dawn
Vulpa Bloom
Mysterious Glittering Object
Units:
Kern Mosshoof
Chok'Garok
Ureda
Kheyna Spinpistol
Aneka Konko
I believe some of the quests need races,faction,classes added to them which I will visit later.

I have reviewed and checked all changes and everything seems to work well.

Thanks! All merged :)


Compatibility issues

The current version only works with Mac OS.
I've only included my own binary of EnergiBridge.
Any help is appreciated!

The reason might be because, the energibridge binary might be compiled specifically for macOS. If this binary contains macOS-specific system calls or dependencies, it will not run on other operating systems.
The best solution is by mocking the Subprocess call to allow testing of the subprocess functionality without depending on the actual energibridge binary
I will open a pull request with the probable solution

Hi @Khaleelhabeeb
Thanks for your input! Indeed, you're right: the binary I am importing in the project was compiled for my machine only.
Another feature that could help here would be to allow users to configure their own binary. This could be set in a file stored in the home directory or in the project directory.


Update WETH

:clipboard: Add associated issues, tickets, docs URL here.
Overview
Describe what your Pull Request is about in a few sentences.
This PR deprecates the old WETH9 and uses the new WETH which loads the token name and token symbol from the L1Block contract
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Update boba-bindings and boba-chain-ops

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.

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

Project coverage is 29.14%. Comparing base (503d43c) to head (cb45139).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                    Coverage Diff                    @@
##           override-bridgeERC20     #205       +/-   ##
=========================================================
- Coverage                 40.13%   29.14%   -10.99%
=========================================================
  Files                        75       65       -10
  Lines                      5183     4104     -1079
  Branches                    826      826
=========================================================
- Hits                       2080     1196      -884
+ Misses                     2996     2833      -163
+ Partials                    107       75       -32

Flag
Coverage Δ

cannon-go-tests
?

chain-mon-tests
27.14% <ø> (ø)

common-ts-tests
26.72% <ø> (ø)

contracts-ts-tests
12.25% <ø> (ø)

core-utils-tests
44.03% <ø> (ø)

sdk-tests
34.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 10 files with indirect coverage changes


Python: updated openai chat sample

Motivation and Context

fix chat_gpt_api.py sample
Description

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

Python 3.8 Test Coverage Report • FileStmtsMissCoverMissingTOTAL540799482% report-only-changed-files is enabled. No files were changed during this commit :)
Python 3.8 Unit Test Overview

Tests
Skipped
Failures
Errors
Time

1214
11 :zzz:
0 :x:
0 :fire:
22.052s :stopwatch:

Hi @eavanvalkenburg  @moonbox3  this thing was highlighted by me, and you have created a separate PR , and did not add me here. may I ask why?
what of my contribution, you just closed that.
please clear me things otherwise I will complain to Microsoft about this.

is this PR counted as mine contribution?


low-level prototype 2

wire frame of Customer online order page

Pages:
PayDay, Edit employee, edit inventory


Include TableNo on Fields.Lookup Page

If table numbers are not meant to dissappear soon, it would be nice if we would have the 'Table No' field available on the 'FieldsLookup' page (hidden by default?). Our consultant often now the table no. by heart and searching seems faster (in RTC)
Having table no / table name vs field no / field name seems logic?
https://github.com/microsoft/ALAppExtensions/blob/master/Modules/System/Field Selection/src/FieldsLookup.Page.al

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Xpath works with UIautomator but same xpath does not work with UIautomator 2

xpath which does not work with Uiautomator2
@FindBy(xpath="//android.widget.EditText[@resource-id='com.test.android.app:id/edit_text'][@NAF='true']")
Please help.
Appium version- 1.6.5(command line)
uiautomator2 version-
<EMAIL_ADDRESS><EMAIL_ADDRESS>

This issue doesn't seem to be related to Robot Framework itself.


Ethernet: gptp doesn't work at all on STM32H5

check for gPTP issue on STM32H5
link to:
stm32h5: samples/net/gptp doesn't work at all #62465

it not the right way
the issue is in HAL
so I close this PR


WIP: Hook up doorlist UI to database

Still a WIP, but what do you think of adding a "checked-in" column to the tickets table?
Done so far

Use new query for Doorlist endpoint query & add checkedin to result
Align data with datagrid in doorlist React component
Add event name to doorlist title

To do

Update Showings page to get showings from database
a. Group showings by play
Send selected showing's ID in URL, and use as param in query string
Create endpoint to update "checkedin" status for tickets on checkbox click

Also, can you help me understand what the self join to tickets is doing?

Whoops that's my bad, the self join serves no purpose. It looks like I had accidently left it in there while I was experimenting with different queries. For the finalized version of the query, we can drop the left join tickets as tix2 on tix.ticketno = tix2.ticketno bit.

Sounds good. Thanks Francis - the self join bit is removed.


How many BoF Dinner assistants needed?

Currently there are 8 on the volunteer grid: http://volunteer.pydx.org:8081/volunteer/grid/
Is 8 the correct number?

Hi Chris,
We discussed this tonight and, unless we get 50 new people registered in
the next week, we're likely going to only need at most 2 volunteers for
this shift.
Thank you for getting the app out!
On Sep 29, 2015 9:20 PM, "Chris Freeman"<EMAIL_ADDRESS>wrote:

Currently there are 8 on the volunteer grid:
http://volunteer.pydx.org:8081/volunteer/grid/
Is 8 the correct number?
—
Reply to this email directly or view it on GitHub
https://github.com/ChrisFreeman/pydx-volunteer-2015-issues/issues/11.

Change max volunteers to two from 8.


Add support for GCS storage

This seems to work well for us. Not too sure not touched PHP code in years, but more than happy to invest some more time into it.
I think we should use the URL abstraction of laravel-filesystem rather than building it in our code base (getPublicUrl). This would make it easier to support signed temporary URLs as a next step

Hi @simonswine,
Thank you for working on this and providing the code in this pull request.
To be totally honest I'm not sure GCS is something I'd like to include and support at this time. It hasn't really been requested before so the overall benefit is likely limited at this stage.
From a quick search GCS appears to have an s3 compatible API, Maybe that could be used instead? From checking support for Minio and ceph I know other S3 API compatible services can work so maybe GCS can too.

The S3-compatible API in GCS is generally not recommended -- it can incur extra costs, and authentication with it doesn't benefit from IAM permissions. I'd personally love to see support for this, since it would make our deployment of BookStack that much cleaner.

Hi @ssddanbrown,
thanks for taking a look and replying, it fair enough if you decide not to include support for GCS and potentially other Cloud providers (it is open source after all)...
I was well aware of the compatibility that GCS provides, but security wise it is a no-go for any serious use case (as @nilium points out). I am happy to take another look at this PR if you decide to change your mind and get it in a mergeable shape.
Whoever you gonna decide, thanks for Bookstack it works really well for our use case

Since this issue was created I have had to use GCS myself so I can appreciate why the s3-compatible API isn't optimal. That said, GCS support really has had very little demand so I'd still not look to support it at this time. I kinda see this as horizontal growth of the platform as explained in my comment here.
If you still want to patch this in yourself, I'd be happy to accept a fresh pull request for some of the public URL generation cleanup in this to make maintaining such a patch easier for yourself.
Additionally, it may be worth noting there appears to be a more recently supported GCS Laravel library here:
https://github.com/Superbalist/laravel-google-cloud-storage
Due to the above I'm going to close this. Thank you still @simonswine for originally spending some time and offering this pull request, and for your understanding in your previous comment.


Exception while decompiling 180138aa0: Decompiler process died

Describe the bug
sample.txt (Rename as .DLL)
In this sample, the function at 0x180138aa0 fails to decompile, its a huge function that initializes a lot of stack strings in part using SSE instructions. The CFG could compute and shows a very vertical graph, but everything indicates me that this function is a real one and not just added for anti-analysis. Another detail is that the stack for this function is very big, and as a side note, trying to emulate this function with Ghidra emulator seems to fail because the emulator doesnt allocate enough stack and there seems to be no option to customize it. https://github.com/NationalSecurityAgency/ghidra/blob/33f90ba1c168cf4fa2b7631c7f0787d774c25c1d/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/service/emulation/ProgramEmulationUtils.java#L470
To Reproduce
Load the sample and analyze
Go to.. 0x180138aa0 then create a function there because auto analysis doesnt seem to create it, watch decompiler crash
I tried modifying various options/limits of the decompiler, nothing worked
Expected behavior
Decompiled function showing right
Environment (please complete the following information):

OS: Windows 10 22H2
Java Version: JDK 17.0.10
Ghidra Version: 11.0.1
Ghidra Origin: official release

Additional context
When the process dies I can see a WerFault.exe process which means it crashed

Seems I was able to get it to run for longer by modifying the decompiler.exe stack size in its headers, like so in CFF Explorer:

So its crashing due to running out of stack space?

73GB of memory usage and still going. Should it ever consume that much memory? Either way I only have 32GB so it probably swapped by now either way my system still running OK.

I am reproducing


IDR about deployment page

This moves the former README-os-idr.md into idr-playbooks/files/html and adds a HTML, wrapper so it appears alongside the current /about page. I've updated the file to correspond to the latest IDR deployment playbooks, and added some background information.

Just put this on-hold....we forgot to discuss it in the last IDR meeting

This needs discussion since it's included in the intended IDR deployment branch https://github.com/openmicroscopy/infrastructure/pull/159 If we don't want a deployment doc alongside the about page I'll remove it.

Last commit adds a link to /jupyter

I've manually copied the state as of https://github.com/openmicroscopy/infrastructure/pull/154/commits/69baf1b326ba49835d06bc380baa1e90e7d553ac to the new repo. Happy for this to go in as the current (0.3.0-rc2) way of deploying; or happy to move straight to use of jekyll-playbooks.

I'll merge then.


build: Move non-macro variables into settings.cpp

Since APP_VERSION is different for every commit and almost all file
include settings.hpp, the whole project has to be rebuilt for every
commit. With this, hopefully, this can be greatly reduced and only
changed files need to be rebuilt. This will also help ccache

It's only the default for functions, and I can remove it there if you want. But for variables it changes the semantics, and because they are const, you get an uninitialized const error when you remove it.
Looking at this again, I think I will move everything that doesn't has to be used in a macro into a variable.


Deprecated of docker ps since and before options for v1.12

Deprecated note https://github.com/docker/docker/blob/master/docs/deprecated.md#docker-ps-before-and-since-options
Signed-off-by: Wen Cheng Ma<EMAIL_ADDRESS>

@thaJeztah PTAL,  and can you re-trigger  the winddowsTP5 check ? thanks!

Nice! :heart: these cleanups
Perhaps we should change "Target For Removal In Release: v1.12" to "Removed in release v1.12" (I see we didn't update other deprecated options, but we can do those in a separate PR)
Otherwise looks good 👍

@thaJeztah  deprecated document has been updated  ad you said,  thanks.

great! LGTM

@thaJeztah  looks experimental and windowsTP5 check jobs failed unrelated reason.

LGTM (assuming grep doesn't fail :p)

LGTM 🐼


Update whats-new.md

Add Layout support office and html files

@laujan
The warnings in this PR need to be resolved before we can merge.
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Fix: klog log verbosity.

Signed-off-by: xuezhaojun<EMAIL_ADDRESS>

/assign @zhiweiyin318

As in https://github.com/kubernetes/klog/blob/main/README.md#how-to-use-klog says,
Use klog.InitFlags(nil) explicitly for initializing global flags as we no longer use init() method to register the flags

/lgtm
/approve


Sidekiq does not die with Ctrl-C on foreman

Procfile
test1: bundle exec sidekiq -q test1
test2: bundle exec sidekiq -q test2
test3: bundle exec sidekiq -q test3

MyWorker.rb
class MyWorker
  include Sidekiq::Worker
  sidekiq_options retry: false

  def perform(params)
    Sidekiq.logger.info { "start" }
    sleep 10
    Sidekiq.logger.info { "finish" }
  end
end

reproduce:
$ bundle exec rails c
rails> Sidekiq::Client.enqueue_to('test1', MyWorker, 'foo')

then, Ctrl-C on foreman
$ bundle exec foreman s
(omit)
01:18:58 test1.1 | 2015-04-22T16:18:58.954Z 8101 TID-otp6bh2ho INFO: Starting processing, hit Ctrl-C to stop
01:19:24 test1.1 | 2015-04-22T16:19:24.574Z 8101 TID-otolbbd84 MyWorker JID-ba53c839c56c5cdb53578efe INFO: start
01:19:24 test1.1 | 2015-04-22T16:19:24.574Z 8101 TID-otolbbd84 MyWorker JID-ba53c839c56c5cdb53578efe INFO: start
^CSIGINT received
01:19:27 system  | sending SIGTERM to all processes
01:19:28 test3.1 | exited with code 0

ps. one of three processes survived this time
$ ps -ef | grep sidekiq
sonots      7169     1  1 01:13 pts/6    00:00:04 sidekiq 3.3.4 argus [0 of 25 busy]

It seems this is caused because foreman does not wait until all of its children dies.

Issued to sidekiq, too https://github.com/mperham/sidekiq/issues/2312

Sidekiq does not die immediately upon Ctrl-C, it can take up to the shutdown timeout in seconds (default is 8 sec).  How long does Foreman wait for child processes?

It looked foreman does not wait at all cuurently, i.e., sends TERM and exit immediately.
Waiting 10 seconds as default would be sufficient like heroku waits 10 seconds.

I'm running into this too.  Foreman should definitely wait until all the children are finished before exiting.  Doesn't seem like leaving before your workers leave is something that a foreman would do. :wink:

+1! I've hit this issue too. Running foreman via upstart, and everytime the service is restarted we get a new sidekiq worker.

I've made this patch for my use : https://github.com/jbox-web/deploy-it/commit/f5fed452d4885b7a91bf8984588aca760ba8a1ba, it seems to do the trick.

Should be fixed now


chore: Rename MyProfileHeaderMyCollectionAndSavedWorks

Description
This is an attempt to rename the MyProfileHeaderMyCollectionAndSavedWorks component and related URLs and components.
QA Test Case(s)

Test Case
Feature
Environment
Acceptance Criteria
Setup Instructions/Link

PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Closing this PR for inactivity, please feel free to reopen it if it's still a work in progress.


Typo fix in sentry-kubernetes/README.md

a small typo in sentry-kubernetes/README.md line 13
There are many such typos in the directory /incubator.

/assign @seanknox

/ok-to-test

/lgtm


Write a Haskell program to print elements of an array

Description
Write a Haskell program to print elements of an array
Input  : [1, 2, 3, 4, 5]
Output : 1 2 3 4 5

How to contribute

Save the solution in print_elements_of_an_array.hs file
Add print_elements_of_an_array.hs file in program/print-elements-of-an-array folder

!assign

I've already given the solution in #7101. Please close the issue


What is difference between @mediapipe/tasks-vision and @mediapipe/selfie_segmentation on npm?

I am working on a video conference project using JavaScript, and I want to incorporate real-time selfie segmentation on the webcam using MediaPipe npm packages. However, I noticed that there are two different packages available on npm. The MediaPipe website uses the @mediapipe/tasks-vision npm package to demonstrate image segmentation, and provides demo code (available at https://codepen.io/mediapipe-preview/pen/xxJNjbN). However, it seems that the webcam demo is a little laggy and not very smooth. On the other hand, I found another demo (https://github.com/ayushgdev/MediaPipeCodeSamples/blob/main/Vanilla JS/selfie segmentation with bg blur.html) that uses the @mediapipe/selfie_segmentation npm package to demonstrate background blurring except for the selfie segmentation. This demo is much smoother than the former.
My question is: what is the difference between @mediapipe/tasks-vision and @mediapipe/selfie_segmentation on npm? Is using @mediapipe/selfie_segmentation a better and smoother option compared to using @mediapipe/tasks-vision?
Thank you very much!

@hongruzhu good question! Google Meet uses a proprietary model to implement real-time selfie segmentation on the input video feed, which unfortunately is not open-sourced. You can refer to this issue to know more.
Coming back to your question, both @mediapipe/tasks-vision & @mediapipe/selfie_segmentation are capable of performing realtime segmentation on the input feed. In terms of performance and accuracy, both solutions are based on state-of-the-art machine learning models (assuming you're using up-to-date versions) and can provide high-quality results. The choice between them will depend on the specific requirements of the project & system configuration of your device (since both runs on-device).
But in general, @mediapipe/tasks-vision is a more general-purpose solution for performing various computer vision tasks, including object detection, face detection, and segmentation. It allows for greater customization and flexibility, but may require more effort to implement for specific use cases.

Hello @hongruzhu
We are building a set of new, improved MediaPipe Solutions to help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions. And we request the MediaPipe developer community help us uncover the issues and make the APIs more resilient. As part of the new APIs, we extend the computer vision related solutions as @mediapipe/tasks-vision in javascript world. These tasks include image segmentation(this includes the selfie segmentation), object detection, hand gesture recognition, etc.
The @mediapipe/selfie_segmentation package is the legacy solution offering which provided capability of selfie segmentation. However, we are ending support for these MediaPipe Legacy Solutions, and upgrading the others. The libraries, documentation, and source code for all the MediapPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions.

@hongruzhu
So will talk first about the codepen, the model they have implemented there is the "deeplab_v3" model but in the documentation they have only mentioned the Selfie Segmentation model don't know why it is like that.
But I replaced the url for deeplab_v3 with the selfie_segmenter model "https://storage.googleapis.com/mediapipe-models/image_segmenter/selfie_segmenter/float16/latest/selfie_segmenter.tflite?v=aljali.mediapipestudio_20230621_1811_RC00" and I got very good results.
The lag which you described about is not there with this model.
Now talking about the difference between @mediapipe/tasks-vision and @mediapipe/selfie_segmentation, so after updating the model url as described above there is not much difference in both the models just I observed that the edges were sharp in the taskvision model then the other model/

It seems, that @mediapipe/selfie_segmentation cannot be started from a WebWorker. At least, I had this issue in my use case.
It's also mentioned here https://github.com/google/mediapipe/issues/3659#issuecomment-1289300054
In contrast, @mediapipe/tasks_vision should work fine for this purpose as mentioned here  https://github.com/google/mediapipe/issues/3659#issuecomment-1705136767
Haven't checked this yet myself. But it's confirmed here https://github.com/google/mediapipe/issues/3659#issuecomment-1718467419


Make RpcMock test case a standalone class

This will make having rpc calls mocked optional for developers to
either include in there test cases or no.

+1


Wireguard-ui generating invalid config files on restart

We are usng the latest docker version of wireguard from dockerhub (super basic config, most things left as default except for passwords). The first time the container starts file, creates an empty DB and we can add some connections via the GUI and apply them. Then when we do docker compose down, and then up, we see that all tunnels are down in the log. The only way to fix it is to remove config/wg0.conf and then restart, edit a connection and then apply the config to regenerate a valid config.  Here is the log output after a restart:
**** Server mode is selected ****
**** SERVERURL var is either not set or is set to "auto", setting external IP to auto detected value of <IP_ADDRESS> ****
**** External server port is set to 51820.   Existing configs are used. ****
[custom-init] No custom files found, skipping...
.:53
 ****
**** All tunnels are now down. Please fix the tunnel config /config/wg_confs/wg0.conf and restart the container ****
[ls.io-init] done.

The problem is likely that the config is also being added to config/wg_confs so it exists twice.

Seems to be related to a commit from october 2023
https://github.com/linuxserver/docker-wireguard/commit/36ca38850c150fa82a068f2543eb26e37919e810
by using this image: linuxserver/wireguard:v1.0.20210914-ls4
it should be working, but in my case i cant connect at the time but this can be related to another problem in my case

Can confirm that it works with the specified image version of linuxserver.io's wireguard image.
If i have time in the future i maybe create a PR for this here but at the moment i don't know anything about go.

Just tested with the image provided above and it survives a restart now. Thanks for the update. I will keep an eye out for this in future releases.

Another solution that worked for me: just mount the config volume to the directory that linuxserver now expects it to be in
version: "3"

services:
  wireguard:
    image: linuxserver/wireguard:latest
    cap_add:
      - NET_ADMIN
    volumes:
-     - config:/config
+     - config:/config/wg_confs
    ports:
      - "51820:51820/udp"
...

Disclosure: I'm not familiar enough with Wireguard or wireguard-ui to know if there are any bad implications to moving the whole mount point for the config directory

Another solution that worked for me: just mount the config volume to the directory that the linuxserver container now expects it to be in
version: "3"

services:
  wireguard:
    image: linuxserver/wireguard:latest
    cap_add:
      - NET_ADMIN
    volumes:
-     - config:/config
+     - config:/config/wg_confs
    ports:
      - "51820:51820/udp"
...

Disclosure: I'm not familiar enough with Wireguard or wireguard-ui to know if there are any bad implications to moving the whole mount point for the config directory

Are you using this config yourself? linuxserver/wireguard:latest creates the subfolders coredns, templates and wg_confs, which means mounting with - config:/config/wg_confs will create the path /config/wg_confs/wg_confs. If youre going to move the mount point, then you should at least do it for the wireguard-ui container, not the wireguard container. Also. i tried, but did not make it work...

Two solutions that worked for me:
Symlink

Stop wireguard-ui
Remove config/wg0.conf
Make a symlink to the correct file config/wg0.conf > config/wg_confs/wg0.conf
Start wireguard-ui

Configuration
Apparently wireguard-ui can be configured to point to a different configuration file, in the case of the subdirectory wg_confs you should set it up like this (In docker-compose):
...
services:
  wireguard-ui:
    environment:
      - WGUI_CONFIG_FILE_PATH=/etc/wireguard/wg_confs/wg0.conf
...

Although I've had issues with this environment variable not being picked up with some reason, so I'm just using a symlink at the moment.


⚠ ClamAV has degraded performance

In c6b22aa, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 593 ms

Resolved: ClamAV performance has improved in 892ee20.


Support for custom colors for jupyterhub theme

Users need a convenient way to control the:

primary
secondary
text color
accent color

there are currently 6 colors used in the template:
#652e8e #474173 #957da6 #e8e8e8 #00a3b0 #32c544
these colors will have to be ordered according to how the present.
currently the style sheets are outside the templates directory, but to configure them they'll need to be templates, i think.

@tonyfast yes they will need to be in templates. I think templates/page.html. https://github.com/Quansight/qhub-jupyterhub-theme/blob/master/custom/css/style.css is short and could just be inlined into templates/page.html

Re-assigning @tylerpotts to issue.


"no gamepads detected" - xbox360 wireless controller, ubuntu 15.10 headless - missing dependency??

hey there,
i installed ES on a headless ubuntu 15.10 server from the ppa. It does start but my gamepad (wireless xbox360 controller) is not detected, however.
The controller itself is working perfectly, though, as i can run jstest with the registered device (/dev/input/js0) from the commandline: every button input works the way expected.
i use nodm to start emulationstation and have also tried to run openbox before ES.
IF, however, I install a full desktop environment (lxde-core), ES started within LXDE perfectly detects my gamepad.
What am I missing here? What package / setting is installed with the desktop environment that helps to make the gamepad work in ES??
thanks for your hints

you have to set  in es_settings.cfg

I'm running into the same issue, but on Windows 10 with a xbox360 fight stick. When I'm prompted to hold a button I do so and nothing happens after 10-15sec. Per above, I looked for a es_settings.cfg file in my .emulationstation folder but I don't have one. What am I missing?


Support discounts specifying fields in the discount UI

Currently custom discounts cant specify fields the filament UI.
This PR attempts to fix that by allowing discounts to specify 3 'magic' methods:
lunarPanelSchema(): array
The Filament schema for the fields to be displayed when the discount type is specified
lunarPanelOnFill(array $data): array
To allow the discount to modify the data associated with the discount before its rendered (only called if the discount type is selected)
lunarPanelOnSave(array $data): array
To allow the discount to modify the data associated with the discount before its saved  (only called if the discount type is selected)
Here is an example of how it is used, in a Amount off (Retail price) discount that we use in one of our systems:
    public function lunarPanelSchema(): array
    {
        $currencies = Currency::get();

        $currencyInputs = [];

        foreach ($currencies as $currency) {
            $currencyInputs[] = Forms\Components\TextInput::make(
                'data.retail.fixed_values.'.$currency->code
            )
            ->label($currency->name)
            ->afterStateHydrated(function (Forms\Components\TextInput $component, $state) use ($currencies) {
                $currencyCode = last(explode('.', $component->getStatePath()));
                $currency = $currencies->first(
                    fn ($currency) => $currency->code == $currencyCode
                );

                if ($currency) {
                    $component->state($state / $currency->factor);
                }
            });
        }

        return [
            Forms\Components\Toggle::make('data.retail.fixed_value')->live(),
            Forms\Components\TextInput::make('data.retail.percentage')->visible(
                fn (Forms\Get $get) => ! $get('data.retail.fixed_value')
            )->numeric(),
            Forms\Components\Group::make(
                $currencyInputs
            )->visible(
                fn (Forms\Get $get) => (bool) $get('data.retail.fixed_value')
            )->columns(3),
        ];
    }

    public function lunarPanelOnFill(array $data): array
    {
        if (! Arr::get($data, 'data.retail', false)) {
            $dataValues = $data['data'] ?? [];

            $dataValues['retail'] = [];

            if (isset($dataValues['fixed_value'])) {
                $dataValues['retail']['fixed_value'] = $dataValues['fixed_value'];
                unset($dataValues['fixed_value']);
            }

            if (isset($dataValues['fixed_values'])) {
                $dataValues['retail']['fixed_values'] = $dataValues['fixed_values'];
                unset($dataValues['fixed_values']);
            }

            if (isset($dataValues['percentage'])) {
                $dataValues['retail']['percentage'] = $dataValues['percentage'];
                unset($dataValues['percentage']);
            }

            $data['data'] = $dataValues;
        }

        return $data;
    }

    public function lunarPanelOnSave(array $data): array
    {
        if ($retail = Arr::get($data, 'data.retail', false)) {
            $data['data'] = array_merge($data['data'], $retail);
            unset($data['data']['retail']);

            $currencies = Currency::enabled()->get();
            $fixedPrices = $data['data']['fixed_values'] ?? [];
            foreach ($fixedPrices as $currencyCode => $fixedPrice) {
                $currency = $currencies->first(
                    fn ($currency) => $currency->code == $currencyCode
                );

                if (! $currency) {
                    continue;
                }
                $data['data']['fixed_values'][$currencyCode] = (int) round($fixedPrice * $currency->factor);
            }
        }

        return $data;
    }

As you can see we are using some of the same data values as the standard "Amount off" discount, but we have to prefix them in the schema to avoid collisions on the filament side... then on fill/save we add/undo those prefixes so they dont get saved to the database. This is just a simple example to show how they can used and why they are necessary.
The benefit of this approach is that the discount is just registered once, making the DX experience simpler.
        Discounts::addType(
            DiscountTypes\AmountOffRetail::class
        );

Thanks @ryanmitchell Going to try and get my head into and take a look at this soon :) @glennjacobs did you have any opinions in the meantime?

@ryanmitchell I think this approach is fine. Here are my thoughts...
The new methods don't implement an interface but I think they should. The core provides the DiscountTypeInterface and I feel like the admin package should provide its own interface for these three new methods.
And then finally we could do with updating the documentation to cover this.

@glennjacobs thanks for reviewing and your feedback. I've pushed up the changes requested.

Just checking in on this - it would be great to get reviewed again to make the next alpha if possible.


use example config

Warnings

:warning:

Please include an updated CHANGELOG.md file.This way we can keep track of all the contributions.

  Generated by :no_entry_sign: dangerJS


Adding return type int to queryScalar method docs #18424

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
#18424

Thanks.


Failure to open Microsoft Copilot App

Status
Available in
Channel type

In preview
Windows 11 Insider Preview Build 26298.5011
Canary Channel

Dev Home version
1.1501.521.0
Dev Home Azure Extension version
1.1501.521.0
Windows build number
10.0.26298.5011
Other software
OS Build Version: 10.0.26298.5011.ge_release_svc_im_current_directadept.240610-1312
.NET Version: .NET <IP_ADDRESS>
Steps to reproduce the bug

Try restart your computer
Try open Microsoft Copilot App again

Expected result
Microsoft Copilot App will work
Actual result
Error loading and opening the Microsoft Copilot App
Included System Information
CPU: 11th Gen Intel(R) Core(TM) i7-11370H @ 4.85GHz
Physical Memory: 39.78GB (14.21GB free)
Drive Storage: 121.21GB
Processor Architecture: x64

It sounds like the issue you're having is not related to Dev Home, so this isn't the correct place to open it. Please open your issue in Feedback Hub, and it will be seen by the right people to help.

@IDOWNGRADEDYTAPP it looks like you've reported several issues unrelated to Dev Home here at GitHub in the Dev Home repository. This repository is focused solely on Dev Home. If you wish to report issues with other products, please report them at their GitHub repositories or use Feedback Hub.


feat: add main and types field

Also updated prettier config so that it matches what you are doing (adding semi: true)
Fixes #4

Thanks


Sharding strategy in 405b paper

Quote from paper

Our use of FSDP for Llama 3 shards optimizer states and gradients, but for model shards we do not reshard after forward computation to avoid an extra all-gather communication during backward passes.

My question is doesn't this result in having all of the model in memory by the end of forward? Not clear on how they accomplished this.
I believe this is related to SHARD_GRAD_OP for pytorch fsdp:

SHARD_GRAD_OP: Gradients and optimizer states are sharded during computation, and additionally, parameters are sharded outside computation. For the parameters, this strategy unshards before the forward, does not reshard them after the forward, and only reshards them after the backward computation. The sharded optimizer states are updated locally per rank. Inside no_sync(), the parameters are not resharded after the backward computation.

It seems we need to use device_mesh when using SHARD_GRAD_OP/HYBRID_SHARD. That was the missing piece when I tried this before


Fixes Spelling and Formatting errors

Fixes #1

This is awesome, a huge thank you from the team. I think we owe something. Do you want to contribution listed in another file here, or perhaps a free month of one of the paid plans. Let us know.
Thank you,
Warren

Hi @wparad, I am glad to connect with you and thanks for merging the PR.
It would be great if you could add me to the contribution list.


OpenTelemetry demo: we cannot send traces from the frontend through ingress

I'm running the OpenTelemetry demo in Kubernetes. It's in EKS on AWS. I added an ingress (backed by an AWS ALB), so that it is publicly accessible. The ingress routes /v1/traces to the collector, and everything else to the frontend-proxy.
This resulted in 502 errors from the ALB when trying to connect to the collector, while the frontend-proxy endpoints worked fine.
The ALB access logs indicated that it was unable to establish a connection to the collector.
Thanks to @TylerHelmuth, the solution is to add to the collector's OTLP HTTP receiver configuration:
endpoint: <IP_ADDRESS>:4318
or
endpoint: ${MY_POD_IP}:4381
<IP_ADDRESS> works for both port forwarding and ingress; ${MY_POD_IP} works for ingress but not port-forwarding.
The current setting of endpoint: <IP_ADDRESS>:4381 works ONLY for port-forwarding.
Consider going back to <IP_ADDRESS>, to work for both? I spent days troubleshooting this 😢

@puckpuck we caused this via https://github.com/open-telemetry/opentelemetry-helm-charts/pull/735.
If putting the demo behind an ingress is a common enough practice (which I assume it is based on the number of vendor forks of the demo), I think we do want to make it easier to handle this situation.  I don't feel comfortable using <IP_ADDRESS> as a default value, even in the demo, since we'd be breaking our own Security Best Practices.  Maybe there is a way to clearly document how the default value affects the ingress in the values.yaml and in the README?


Queue connection

noticed a public constructor in a singleton class - made it provate so have to use getInstance()
queue connection should now retry if it gets shutdown

Good stuff, thanks Dave.


This commit will create helper_functions.py

Correct me if I am wrong, should the main function be defined first before other functions, or it should be the last function defined in the script?


unit test to reproduce the issue

This change is

Relates to Issue 89
https://github.com/apex-enterprise-patterns/at4dx/issues/89


Update zhcn locale and upload zhcn words

By the way, dear Zrips, could I have your permission to repost this resource to a Chinese Minecraft forum MCBBS(http://www.mcbbs.net)? Just for advertising.
Of course I will provide your download link and not leak the jar file, just the link.
If you agree, I would appreciate it a lot!
PS. I tried to ask on spigotmc forum, but seems you didn't reply.

I dont really have time for forum post anymore, so mainly answering only to pm's or over github.
If you wish you can post it at MCBBS, just remember to include github links for issue reports.


Mypy not auto on

You might want to squash this

I tried to add MyPy to the README but I got crazy unfixable merge conflicts. Please add after merging.

Please keep this in #439.

#439 is closed now, please re-open and merge this.

ok, should be good to go now

Thanks a lot!


updated client.go

This PR is for fixing for lint issues.

@ashiscs thx for this PR as well.
Can you please get this fixed:

https://github.com/layer5io/meshery-octarine/pull/26/checks?check_run_id=256191057#step:4:97
https://github.com/layer5io/meshery-octarine/pull/26/checks?check_run_id=256191057#step:4:97

Sir,i am closing this issue, as issue no #26 is addressing this!


Change legacy central url

search.maven.org is the legacy search platform with an uptime of 95,88% in the last 90 days (see https://status.maven.org/).
central.sonatype.com is the modern search platform with an uptime of 100%.
So please change the default of centralUrl to https://central.sonatype.com/solrsearch/select

The problem I have with it is that the solrsearch is not part of their documented API
https://central.sonatype.com/api-doc
Their help is still referring to 'legacy search'
https://central.sonatype.org/pages/support/#consumers


stdenv: add runPhase function

Motivation for this change
This is useful for running phases inside a nix-shell, using buildPhase for example will always execute make even when a derivation has a custom buildPhase.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested stdenv.mkDerivation
[x] Fits CONTRIBUTING.md.

Another solution for this behaviour would be to do this in the functions for each phase, that way we would get the expected behaviour without having to use the runPhase function.

I think it's better if this is fixed in the phases, that way it will work as expected.


add support to php 8

Thanks for the PR. I did not merge because I wanted to fix the build errors for PHP 8. They are fixed now and I shall release soon