Question: Runtime representation

Does the runtime representation look like this:
{ there: 1, missing: undefined }

Or
{ there: 1 }

?

From JS perspective is there any difference?

let x = {};
let y = {foo: undefined};

x.hasOwnProperty("foo"); // false
y.hasOwnProperty("foo"); // true

Object.keys(x); // []
Object.keys(y); // ["foo"]

Well I presume there's a big difference memory-wise. I use this to construct some potentially very large records but practically tiny records in FFI. The reason I noticed is that react was complaining about a prop name that didn't exist, but I wasn't really passing that prop (Bing is quite a bit worse at generating FFI than ChatGPT by the way).
So it was actually nice to find out that that key I didn't have example coverage was wrong.
However I would prefer the {} version. I was wondering if it came from this lib, but it's probably from Record.Builder:
subContent ∷
  ∀ p kids. Coerce p DropdownMenuSubContentProps ⇒ ToJSX kids ⇒ p → kids → JSX
subContent props kids = React.element subContentImpl
  $ (coerce props ∷ DropdownMenuSubContentProps) # RB.build
      ( RB.insert (Proxy ∷ _ "children") kids
      )

That is strange. coerce props is noop (just unsafeCoerce) and RB.build uses copyRecord which actually iterates over existing JS object props.
The lib can add a prop to the record if you build it using:
recordWithAField :: { opt :: Opt Null }
recordWithAField = { opt: undefined }

But I would like to leave it as it seems handy - I can document this behavior. What do you think?
We can also remove first class undefined support and encourage merging. I'm not sure.


Add continuous scanning support

Please add support for continuous scanning, so that the barcode scanner does not close after a successful scan.
This could work like this:
var scans = 0,
    messageTimeout;
cordova.plugins.barcodeScanner.continuousScan(
    function(result) {
        // This method will get triggered for every barcode found (continuously)
        clearTimeout(messageTimeout);
        cordova.plugins.barcodeScanner.message('Product found');
        setTimeout(function() {
            messageTimeout = cordova.plugins.barcodeScanner.message(null);
        }, 1000);
        if (++scans === 10) {
            cordova.plugins.barcodeScanner.hide();
        }
    },
    function(error) {
        clearTimeout(messageTimeout);
        cordova.plugins.barcodeScanner.message('Scanning failed: ' + error);
        setTimeout(function() {
            messageTimeout = cordova.plugins.barcodeScanner.message(null);
        }, 1000);
    }
);

A work around I have used is to recursively call the scan method after every success until the user clicks cancel

I would also welcome this feature. It could for example "scan until user clicks done/back button" and then return results in array.. What do you think?

@Giriton I have just implemented such a workaround. I don't think it's quite as fast or elegant as just leaving the camera open but it does work. On iOS 9.3, the done button returns me to my Web UI, same with a Google Nexus 4 running Android 5.x. My "Samsung Galaxy Tab A" has a weird thing where if I tap the hardware back button to cancel, it closes the camera UI but also triggers a back call in the web ui, in my case either navigating me backwards or dismissing an open modal. So, I need to find a way for the back button, when in the camera UI, to not propagate to the web ui after dismissing the camera UI.

I agree, continuous scanning is indeed needed for real-world scanning apps. Calling the scan() method recursively works, but it has some lag to it and it decreases performance. Any update on this issue?

If anyone is willing to implement it: on Android we're using the same library as the NativeScript plugin which I've added continuous scan support to already. All required changes to the native library are already in place, so it should be relatively straightforward to add this to the Cordova plugin as well.
On iOS the NativeScript plugin uses my custom library, not the same as the one used in the Cordova plugin, so it may be a little harder.

+1
calling scan() recursively isn't a great solution

Duplicate of #69

+1
calling scan() recursively has some lag


Passing 3+ params to helper?

Hi, is it possible to pass 3+ parameters to a helper? It seems like it only sees the first two:
Handlebars.registerHelper('myHelper', function (param1, param2, param3) {
   return param1 + param2 + param3;
}

in template:
{{myHelper '1', '2', '3'}}
param3 is always undefined.
Am I doing it wrong?

Remove the commas:
{{myHelper '1' '2' '3'}}

Oh, yeah, thanks @dhollenbeck - it was a misspell in the question, I don't have the commas in my app actually, updated the question to fix that.
Though I seem to find the problem - I was passing a variable as a parameter which was not present in the current context. Once added, the problem was fixed and all three params were populated.
I was just googling for this and all the examples were presented with up to 2 variables, so it started to look like that's the limit.


Any suggestions on how to improve segmenting a fluffy animal?

Thanks for the model again. I tried it with my dog and the masking is not great for his fur.

Any suggestions? Thank you very much!

Reason: Honestly, it should be hard to segment due to the lack of this data in common segmentation datasets (e.g., SOD datasets, DIS datasets). However, this kind of data does appear a lot in image matting datasets.
Solution: I cannot download your uploaded image as it's shown as "private xxx" when clicked. But I made a screenshot of it (image quality is lower) and tried it with BiRefNet for portrait segmentation, of which the result is shown below. You can try that model on https://fal.ai/models/fal-ai/birefnet/playground (choose the model as portrait), where I collaborated with nice guys from that company to build the BiRefNet online demo.
The BiRefNet-portrait was not trained very seriously, and I plan to re-train it better in the next three days. I'll release the new model and update my online demo on HuggingFace with it.  Even though, the predicted result of the low-quality screenshot is still better than the previous one.
Once I finish the training, I'll tell you here if you want.

Awesome! It does look better, and I'm looking forward for the weight.  Few more questions:

Will we be able to host the weight ourselves?
Can we use the existing ComfyUI plugin for the portrait model? Is it just a weight swap?

Thank you!

Sure, all the content of my project is under the MIT license. It's fully open-source and even commercially available.
Regarding ComfyUI, there are several third-party implementations of adding BiRefNet as a node. However, they haven't been updated for a long time, and so many things have been updated. Therefore, they may cause some inconsistency problems. They did it with interest and enthusiasm and made it open-source, so I cannot urge them to update with my official latest versions.
Based on that, I will establish a ComfyUI node for BiRefNet by myself. It would take some time since I'm not familiar with ComfyUI, but since people like you do have this need, I'll surely do it.

Awesome! ComfyUI node will be so useful and the community will be benefit from it. Thank you!

Another great example here:

General Use Heavy:

Photoroom on iOS:

It would be great if the model can also trim the white border like photoroom as well. Thank you!

Thanks for your excellent suggestion! I'll check if this is a common problem in the predicted results of BiRefNet. If so, I'll try to think about how to solve it through training or some post-processing.
But I also feel pleased that BiRefNet performs better on some other points 😄

Omg yes, I didn't even catch it. Great job! I'm working on a large batch of photos that need to remove the background and I will post the results in this issue:
https://github.com/ZhengPeng7/BiRefNet/issues/44

BiRefNet-matting has been provided. Compared with previous BiRefNet-portrait, it's trained on much more matting data and has better matting performance on general scenes with different types of objects (human / animal / ...).
Back to the Samoyed, the result is now as below:


Avoid redundant calculations

What is this PR doing?
Avoids some redundant calculations in processBundle:

Duplicate calculation of publicKeyHash
Duplicate call to walletFromHash

It's also a bit simpler this way, imo.
How can these changes be manually tested?
Run the tests.
I also tried logging the gas used in should send ETH (empty call) and found this change reduced the gas from ~300k to ~280k.
Does this PR resolve or contribute to any issues?
Resolves #108.
Checklist

[x] I have manually tested these changes
[x] Post a link to the PR in the group chat

Guidelines

If your PR is not ready, mark it as a draft
The resolve conversation button is for reviewers, not authors

(But add a 'done' comment or similar)

@voltrevo Should this be targeting/comparing against audit-feedback instead?


Feature Request: Shortcuts for restarting Projects

A keyboard shortcut to restart/reboot+debug the selected/recognised current project would be very nice. Unfortunately I am not familiar with Jetbrain's IDE plugin development, but I hope this is doable.

You can create a shortcut for yourself. Just search for aspire in the keymap settings and you will see all the actions

Yes, but that is only useful to some extent because you still have to click the resource every time you want to restart it. Once it loses focus or you navigate to another tool window or editor it wont work. I imagined something more like the "Build current project" action, that detects what you are currently editing and then select the project accordingly. Not sure if something like that is planned. It was just an idea.

I won't add keyboard shortcuts, because it's hard to choose ones that are convenient for all keyboard layouts. But I made a fix to get a focused project from context. That way you can add a shortcut yourself, Rider will try to detect the project and restart/stop the corresponding resource.

Sounds great, thanks!


Removed duplicate foldingRangeProvider field in ServerCapabilities

Dear Good People at Microsoft LSP,
I found a duplicate field for foldingRangeProvider which I removed.
I am working on a Go implementation and stumbled on it while defining data structures. This is my first pull request on this project - I did read the contributing guideline and code of conduct, and hopefully I didn't violate either!
Kind regards,
Dayo

Thanks !


[10.x] Fix an error when $query->from is of type Illuminate\Database\Query\Expression

When passing a subquery referencing a different database to joinSub(), the prependDatabaseNameIfCrossDatabaseQuery() method would rewrite the "from" clause of the subquery. However, if $query->from is not a string but an Illuminate\Database\Query\Expression, the following error occurs:
TypeError: str_starts_with(): Argument #1 ($haystack) must be of type string, Illuminate\Database\Query\Expression given
To address this issue, I made modifications to prevent the replacement process when $query->from is an Illuminate\Database\Query\Expression instead of a string.

Hello, any other way to correct this error ?

@MarFelix
To avoid this issue, I think the only way is to pass the subquery's FROM clause as a string rather than as an Expression.
Example:
    /*
    $subquery = $builder2
                  ->from(new Raw('database2.posts'))  // Expression
                  ->select('user_id, title');
    */
    $subquery = $builder2
                  ->from('posts')  // string
                  ->select('user_id, title');

    $builder = $this->getBuilder();
    $builder->from('users')->joinSub($subquery, 'sub', function ($join) {
        $join->on('users.id', '=', 'sub.user_id');
    });

Refer to the unit tests in https://github.com/laravel/framework/pull/49937/files.


monterey leaves many running instances of nodejs

In the few days of testing (without ever rebooting my Windows), I started / exited / cleared Monterey cache many times. So, I observed that each time I start new instance of Monterey, open an existing application and run it, I would get higher and higher port number
Windows Task manager listed more than 10 instances of nodejs running that were not attached to anything - and since I was not doing anything else that requires nodejs, I am sure that all these were created by Monterey.
I believe that this is a consequence of https://github.com/monterey-framework/monterey/issues/160

we where talking about a global quit event when I was using just node .spawn for the terminal.
If we use nodes child process in monterey and dont manually stop them, they will be running after monterey quit.
(atm I dont know if we use this),
I know Ive tested the terminal I added for this by running gulp watch in it, and checked it stopped when I quit monterey.

This issue is well document (in Electron) so I do not think we need to discuss it. If we cannot relay on Electron runtime to exit all spawned child processes when the user initiates Monterey shutdown, then we need to do it ourselves. It is probably safer if we do it anyway.

need to see how long this holds water

I will be watching like a hawk


Any detailed documentation?

I want to use powerfulseal in our k8s cluster, is there any detailed documentation?

When I installed powerfulseal  with pip install powerfulseal, I encountered the following problem：

 ✘ zhanghui@慧儿 ~ powerfulseal --help
command not found: powerfulseal

Installation information is as follows：

Installing collected packages: prometheus-client, monotonic, humanfriendly, coloredlogs, pycparser, cffi, pynacl, asn1crypto, idna, enum34, ipaddress, cryptography, bcrypt, paramiko, spur, decorator, dogpile.cache, pbr, iso8601, certifi, chardet, requests, stevedore, os-service-types, keystoneauth1, munch, requestsexceptions, netifaces, jsonpointer, jsonpatch, appdirs, openstacksdk, websocket-client, httplib2, pyasn1-modules, oauth2client, kubernetes, itsdangerous, Werkzeug, MarkupSafe, Jinja2, click, Flask, flask-cors, future, ConfigArgParse, botocore, boto3, termcolor, flask-swagger-ui, functools32, jsonschema, powerfulseal
  The script humanfriendly is installed in '/Users/zhanghui/Library/Python/2.7/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
  The script coloredlogs is installed in '/Users/zhanghui/Library/Python/2.7/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
  The script pbr is installed in '/Users/zhanghui/Library/Python/2.7/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
  The script chardetect is installed in '/Users/zhanghui/Library/Python/2.7/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
  The script openstack-inventory is installed in '/Users/zhanghui/Library/Python/2.7/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
  The script flask is installed in '/Users/zhanghui/Library/Python/2.7/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
  The scripts futurize and pasteurize are installed in '/Users/zhanghui/Library/Python/2.7/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
  The script jsonschema is installed in '/Users/zhanghui/Library/Python/2.7/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
  The scripts powerfulseal and seal are installed in '/Users/zhanghui/Library/Python/2.7/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
Successfully installed ConfigArgParse-0.13.0 Flask-0.12.4 Jinja2-2.10 MarkupSafe-1.1.0 Werkzeug-0.14.1 appdirs-1.4.3 asn1crypto-0.24.0 bcrypt-3.1.5 boto3-1.9.71 botocore-1.12.71 certifi-2018.11.29 cffi-1.11.5 chardet-3.0.4 click-7.0 coloredlogs-10.0 cryptography-2.4.2 decorator-4.3.0 dogpile.cache-0.6.8 enum34-1.1.6 flask-cors-3.0.7 flask-swagger-ui-3.18.0 functools32-3.2.3.post2 future-0.17.1 httplib2-0.12.0 humanfriendly-4.17 idna-2.8 ipaddress-1.0.22 iso8601-0.1.12 itsdangerous-1.1.0 jsonpatch-1.23 jsonpointer-2.0 jsonschema-2.6.0 keystoneauth1-3.11.2 kubernetes-1.0.2 monotonic-1.5 munch-2.3.2 netifaces-0.10.7 oauth2client-4.1.3 openstacksdk-0.22.0 os-service-types-1.4.0 paramiko-2.4.2 pbr-5.1.1 powerfulseal-2.0.1 prometheus-client-0.3.1 pyasn1-modules-0.2.2 pycparser-2.19 pynacl-1.3.0 requests-2.21.0 requestsexceptions-1.4.0 spur-0.3.20 stevedore-1.30.0 termcolor-1.1.0 websocket-client-0.54.0

emmmm... It's a problem with environment variables, and I resolved it by increase the $PATH environment variable

/close


Q: How to download all videos from a channel correctly?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.11.11 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Let's say one wants to download all videos from e.g. Kurzgesagt - In a Nutshell. There are multiple URLs to choose from:

https://www.youtube.com/c/inanutshell/ (Main URL)
https://www.youtube.com/@kurzgesagt/videos (Videos tab)
https://www.youtube.com/@kurzgesagt/playlists (Playlists tab)

The goal would be to preserve the hierarchy, so playlists are placed in subfolders, but videos not within a playlist are also downloaded.
The main URL (log telling "[youtube:tab] Downloading all uploads of the channel. To download only the videos in a specific tab, pass the tab's URL") and the videos URL (log telling "[download] Downloading playlist: Kurzgesagt – In a Nutshell - Videos" with fetching five pages of videos) both only create one folder "Kurzgesagt – In a Nutshell - Videos" with 171 videos inside, no playlist folders (and no meta information in the files about e.g. playlists either)
Using the playlists URL gives me 249 videos, with 120 unique names, aka multiple copies in multiple playlist subfolders (see below for example), but missing 51 videos without a playlist.
e.g.
Kurzgesagt – In a Nutshell Playlists\Science and other stuff explained\[20141211] What Is Life？ Is Death Real？.mkv
Kurzgesagt – In a Nutshell Playlists\The Existential Crisis Playlist\[20141211] What Is Life？ Is Death Real？.mkv

So... currently one doesn't see any way to get playlists AND non-playlisted videos in one go without downloading everything twice and deleting copies manually afterwards. One can live with the copies in multiple playlists, as one can create a hard link between the files afterwards. But choosing between non-playlisted videos and playlists but missing videos is not fun.
Is there/Should there be an option to let yt-dlp download the playlists URL and then counterchecking with the videos URL to fetch any missing videos? If it stores the video URLs of the downloaded playlisted videos, it could then just skip those videos itself.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Download the playlist URL with --download-archive, and then the main page with same archive

Tested and works like a charm, thanks.


TMI2-520 - Removed unnecessary dependencies

Description
TMI2-520 - Remove unnecessary Find dependencies
husky and lint-staged have not been removed as they are required for the pre-commit checks (as I discovered when I tried to push this :skull: )
Type of change
Please check the relevant options.

[x] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] This change requires a documentation update.

How Has This Been Tested?
Please describe the tests that you ran to verify your changes:

[ ] Unit Test

[ ] Integration Test (if applicable)

[ ] End to End Test (if applicable)

Screenshots (if appropriate):
Please attach screenshots of the change if it is a UI change:
Checklist:

[ ] If I have listed dependencies above, I have ensured that they are present in the target branch.
[ ] I have performed a self-review of my code.
[ ] I have commented my code in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation where applicable.
[ ] I have ran cypress tests and they all pass locally.

husky and lint-staged have not been removed as they are required for the pre-commit checks (as I discovered when I tried to push this 💀 )

Btw this is the same issue I flagged in my comment - the tool is obviously charting the dependencies as imported in the code, but both these dependencies are never imported in any code and are only referenced in our package.json file.


Update notebook handling to use new nbtx

See nbtx updates here: https://github.com/curvenote/nbtx/pull/15
Behaviour is mostly unchanged, file writing just happens more explicitly during mdast transform, rather than implicitly via nbtx and file factory.
See also #104 #107

Ok, I think this is good to go. A few more things to test before we go in, but should be good!


Nav bar issues

The "Contact" button in the navigation bar does not do anything.

Just a div id issue.

the github pages site has been updated but the netlify will be deployed later


[RFE] Utilize systemd's systext for image composability / extensability

Issue #443 is about deprecating Torcx in favor of systemd's sysext. But it focuses on the limited use case that Torcx was for, installing multiple versions of an application, usually a container runtime. But I think systemd sysext could be leveraged in a much broader and useful way now that systemd 249 is rolling though the channels.
Flatcar has always been a bare minimum Linux distro for containers. In essence all we need is kernel + systemd + update agent (of course, a small set of support tools). But we we see many usecases where it is viable to create an image rather than use containers for everything. I’d like to create a general mechanism that could be used to create customized image rather than just adding single application versions.
An issues with the torcx method is that it's not something that gets auto-updated. So that if I maintain my own Nebraska, I can have a pipeline where I consider an update to be the core Flatcar image + whatever sysext overlay(s) I have. Alternatively, it would be cool to have a tool where I could take my overlay(s) and create a merged image from that. That would simply retain our current update mechanism.
Ideally, as a project, we could maintain only a core image + a container overlay, and maybe a kubernetes overlay. Others could create their own overlays which the Flatcar team does not have to support or worry about but greatly expands the applicability of the Flatcar project and brings its features of auto-updates and immutability to many more areas.
But the key is that it would retain all those features: immutable, auto-updatable, minimal. In this way we're not just making another package manager. The focus should be on create images and having those be able to update.

This is also interesting for the OEM tools because they coupled with the Flatcar release version and tested together (currently they are not updated which causes problems, see https://github.com/flatcar-linux/Flatcar/issues/60). Having the OEM tools packaged as systemd-sysext image helps to be able to update them in a reliable way.
I like the proposal use extend Nebraska to be able to update a sysext image independently from or maybe even coupled to the Flatcar version. However, this does not enable rollbacks which I see as essential. Therefore, to cover automatic rollback, I would suggest to explore if the compressed OEM tools sysext images could even be stored in the /usr partition and loaded from there. This gives the rollback support for free and does not need any changes in Nebraska nor additional awareness when maintaining an own Nebraska server. The Challenge: The size limitation of the /usr partition has to be kept in mind but when we enable filesystem compression on the /usr partition we get more free space than now, and with sysext we can also remove some library duplications which currently increase the OEM tools size.
In addition, using sysext also works for the initramfs when passed by the boot loader. This would allow to inject custom OEM behavior into the early boot stage to load non-default kernel modules or similar.

one thing to work out about systemd-sysext, is:

how to make a workflow for having extensions available that are not enabled to be loaded
figure out how racy the merge/unmerge/refresh are. Say we land a new/updated extension, and run refresh, then it will momentarily unmount all extensions, and then re-merge and mount them. Possibly order dependent? seems racy.

one thing to work out about systemd-sysext, is:

how to make a workflow for having extensions available that are not enabled to be loaded

There's no mechanism to mark extensions as enabled or not. Either they are present and compatible with the system's version or not. So the recommended way to handle this seems to be by directly adding/removing symlinks (in the expected extension locations) to the actual images.

figure out how racy the merge/unmerge/refresh are. Say we land a new/updated extension, and run refresh, then it will momentarily unmount all extensions, and then re-merge and mount them. Possibly order dependent? seems racy.

We could make it so the recommended way to apply images will require a reboot. If an admin calls regresh/merge directly, that would be considered an approach for development.

We could make it so the recommended way to apply images will require a reboot.
If an admin calls refresh/merge directly, that would be considered an approach for development.

I strongly second this. Even more, we could consider making sysexts a provision time feature.
Otherwise we risk just re-inventing package management.

Regarding the raciness of refresh, the new mount beneath feature in the kernel is planned to be used for replacing the sysexts in one go instead of having a window where the overlay is missing.


[BUG] VST3 crash on Windows 10

@sdatkinson When loading NAM as a VST3 plugin on Windows 10, the plugin immediately crashes. At the same time, the standalone version runs well.

OS: Windows 10
Plugin version: v0.7.5
VST3
Studio One 6.1.1

Attaching the crash dump: Studio One_6_2_1_94935_Win x64_20230826_145409013.dmp.zip
Looks like some kind of null pointer exception but not sure how to debug this further:
EXCEPTION_RECORD:  (.exr -1)
ExceptionAddress:<PHONE_NUMBER>000000
   ExceptionCode: c0000005 (Access violation)
  ExceptionFlags: 00000000
NumberParameters: 2
   Parameter[0]:<PHONE_NUMBER>000008
   Parameter[1]:<PHONE_NUMBER>000000
Attempt to execute non-executable address<PHONE_NUMBER>000000

cc @olilarkin possibly iPlug-derived? I vaguely remember the stack trace consisting of some UI-related calls before it crashes... Anything I can do to help debug this?
(BTW, really looking forward to recording our new album with all-NAM guitars! But the plugin crashes on our guitarist's PC, no luck getting it to run yet 😅 Any pointers or workarounds highly appreciated!)

Update: Reaper managed to open the VST without issue on the same PC, so it must be something related to Studio One.

Thanks for the dump!

I vaguely remember the stack trace consisting of some UI-related calls before it crashes

Ar you able to get that stack trace by any chance?

not sure how to debug this further

Are you handy enough to be able to build from source and run in a debugger? I have to admit I haven't done it on Windows myself yet, but the XCode project for macOS includes a definition for running the plugin in Reaper, which I've found really helpful for debugging issues related to execution inside a DAW; presumably something similar might be able to be done in Windows & Visual Studio...

Reaper managed to open the VST without issue on the same PC, so it must be something related to Studio One.

What I think happens is that Reaper has some pretty capable exception handling that keep it running smoothly; I'm not sure I'd conclude it's a problem with Studio One in the sense that I'm not yet convinced that S1 is doing anything that's not a valid use of the plugin API. If we're able to get more insight into this, it may be helpful for spotting something lurking that could be tightened up 🙂


User's input is changing the 's value prop

Describe the bug
When assigning a const obj = ref({}) to a <FormKit type="form"> element  using the :value` prop, the  component is making changes to the prop.
Generally in Vue, components should not mutate props, so that behavior comes unexpected.
(see. https://vuejs.org/guide/components/props.html#one-way-data-flow)

When objects and arrays are passed as props, while the child component cannot mutate the prop binding, it will be able to mutate the object or array's nested properties. This is because in JavaScript objects and arrays are passed by reference, and it is unreasonably expensive for Vue to prevent such mutations.

The main drawback of such mutations is that it allows the child component to affect parent state in a way that isn't obvious to the parent component, potentially making it more difficult to reason about the data flow in the future. As a best practice, you should avoid such mutations unless the parent and child are tightly coupled by design. In most cases, the child should emit an event to let the parent perform the mutation.

If - as a Developer - I want my data (outside of the component) being mutated, I can  use v-model's two-way binding, which is available, too. https://formkit.com/essentials/forms#populating)
Reproduction

Open Playground: https://formkit.link/d586e48ef957b35f08e95aad67c3b389
The field is correctly pre-populated and also the debug output below
Type something in the input
Debug output changes, too, although v-model is not used.

Desktop (please complete the following information):

OS: Mac 11.6.2
Browser: Chrome 98.0.4758.80

In beta.3 we now clone the value prop before using it to prevent these kids of mutations.

If you change the playground example to beta.3 you can see the non-mutated props now

Love it! Thanks!


Installer script in devsds does not work if etcd is already installed with a different version

OpenSDS devsds installer do not consider version of etcd if etcd is already installed.
Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
When trying to install opensds using 'install/devsds/install.sh', in a system with already installed etcd, installer failed.
What you expected to happen:
Installer should be using a consistent version of etcd as configured.
How to reproduce it (as minimally and precisely as possible):
Install latest etcd 3.4 in the system and try to run installer script.

It's Fixed. The PR for fix is #1091


请问关于meta parser的相关资料？

关于生成代码部分，请问有没有对应的学习资料呢？

meta parser is a c++ reflection library implemented based on libclang, you may find some resources from these links:
Using Clang to generate C++ reflection data
metareflect
And our code will be open-source soon.
There are also some other methods to implement c++ reflection, such as rttr

This issue will be closed, plz reopen or create a new ticket if you have any problems.


Merge keyswitch from development into main (#10)

Added contents to prepare for v1.1 release
Added keyswitch kernel support
Added integrated kernel support for DyadicMultiply and Keyswitch
Deprecated standalone NTT/INTT
Updated documentation

Merge keyswitch from development into main to prepare for v1.1


Error when running yarn start

I'm getting an error when I run yarn start - UnhandledPromiseRejectionWarning: TypeError [ERR_INVALID_ARG_VALUE]: The argument 'id' must be a non-empty string. Received ''
Hare is my package.json file:

Same, with npm start. Moved back to 6.0.0 and now it works fine.

Yes, I have tried that but no results.

I removed the carco - yarn remove @craco/craco
Added 6.0.0 verison - yarn add<EMAIL_ADDRESS>
I have tried with 5.9.0 and 5.8.0, even I removed node_modules folder and the yarn.lock file, run yarn install, but I got the same error upon start

Don't forget to make craco.config.js file. 😉

Don't forget to make craco.config.js file. 😉

yep... I did create a file but it wasn't in the root directory, but in the src folder. I moved it and it works, thanks : )

Don't forget to make craco.config.js file. 😉

yep... I did create a file but it wasn't in the root directory, but in the src folder. I moved it and it works, thanks : )

I had this same issue. Didn't even notice I added it to the src folder by accident. Thanks for the info!


Concatenate/Append strings

How do I append a string to a variable etc? Like p := "[" + var + "]"

For your example, exactly as you've done. However, if you're appending string to a non-string and want a string result, you do p := string(nonStringVar) + "some string"

Can you add this to the documentation?

It's not clearly documented, but, we have Operators documented.
I thought about supporting non-string + string = string, but, the problem that

the other type may define its own + with string
it's not easy to implement universal string concat operation on all types (for example, even if we add string concat operator to all run-time types, it will not work wit custom types)

And, as @earncef said, string() function is still available if needed.

As of v1.23.0, builtin function format is a better way to format strings with non-string objects. It is described in the Builtins documentation.


fix: error with truncate logic

closes #1420
glaredb universalmind303/truncate-logic ≡34s
λ just run local --max-rows 10
GlareDB (v0.2.1)
Type \help for help.
Using in-memory catalog
〉select * from generate_series(1, 100000);
┌────────────────────────┐
│ generate_series        │
│ ──                     │
│ Int64                  │
╞════════════════════════╡
│ 1                      │
│ 2                      │
│ 3                      │
│ 4                      │
│ 5                      │
│ …                      │
│ 99997                  │
│ 99998                  │
│ 99999                  │
│ 100000                 │
│ ───                    │
│ 100000 rows (10 shown) │
└────────────────────────┘
〉

I did have one note.
would you expect the continuation row to count towards the max_rows?
like if i ran select * from generate_series(1, 100);
with max-rows = 4
would you expect [1,2,..,99,100] or would you expect [1,2,..,100]

I did have one note.
would you expect the continuation row to count towards the max_rows? like if i ran select * from generate_series(1, 100); with max-rows = 4 would you expect [1,2,..,99,100] or would you expect [1,2,..,100]

I would not expect it to count since it's not a "row" in this case.

closes #1412


Multiplayer Menu ip address hard to read while typing

Expected behavior:
I click into the text box, I see the current text with enough contrast from the input field box to see what I'm typing
Actual behavior:
I click the text box, the box gets highlighted near white, with white text on top. I have to blindly type the IP address
Potential Fix. Change the color of the highlight color, or the text file.
Discord Username:
AttackingHobo

The multiplayer menu is just an example you can modify or replace depending on your needs.  If you're not happy with the color of the text you can just change it to what you prefer.


trying to send a notification with a image loaded from include_bytes!() macro

hello.
love this crate.
im trying to send a notification with an icon or image loaded with include_bytes into the binary, but i keep getting the error WrongDataSize
the image im trying to load has this metadata:

thats all i know, but i feel that some metadata needed for this function image_data() its missing
here is the code snippet:
static NOTIFICATION_ICON: &[u8] =
    include_bytes!("../../static/icons/rule-202020.png");

fn image_data(width: usize, height: usize, bytes: &Vec<u8>) -> Vec<u8> {
    let size = width * height * 3;
    // let size = bytes.len();
    let mut image_data = vec![0; size];
    for (i, pixel) in (0..size).into_iter().zip(bytes.iter()) {
        image_data[i] = *pixel;
    }
    image_data
}

// the bytes of the image
fn main() {
    let vec_bytes = NOTIFICATION_ICON.to_vec();
    const HEIGHT: usize = 265;
    const WIDTH: usize = 265;

    Notification::new()
        .summary("Generated Image (.image_data())")
        .body("You should see stripes in this notification")
        .image_data(
  Image::from_rgb(
    WIDTH as i32, HEIGHT as i32,
    image_data(WIDTH, HEIGHT, &vec_bytes)).unwrap())
        .show().unwrap();

}

i tried to code snippet from the example with images from this repo.
with this snippet im getting this notif

something is missing.
how do i extract more metadata about the image?
what the number 3 represents from your example ?
if i am using the len of the bytes as size
fn image_data(width: usize, height: usize, bytes: &Vec<u8>) -> Vec<u8> {
    // let size = width * height * 3;
    let size = bytes.len();
    let mut image_data = vec![0; size];
    for (i, pixel) in (0..size).into_iter().zip(bytes.iter()) {
        image_data[i] = *pixel;
    }
    image_data
}

i get this error
[src/bin/notif.rs:53] &vec_bytes.len() = 62632
The application panicked (crashed).
Message:  called `Result::unwrap()` on an `Err` value: WrongDataSize
Location: src/bin/notif.rs:65

image metadata:

image size in bytes is: 62632
width: 265
height: 265
the number width * height * 3 (which is from the images example from this repo) is: 210675

the number 210675 > 62632. what im i missing here ?
and 62632(bytes) * 3 == 187896 which is also smaller than 210675
do i need to load the image in another way ?
what am i trying to acheive:
load the image into the binary of my application and use that bytes of the image in the notification function of this crate, to display my image into the nofitication

solved it.
the metadata that was required and was missing was provided by the crate image
here is the final snippet:
use image::ImageBuffer;
use image::Rgba;

use notify_rust::ImageError;
use notify_rust::Notification;
use notify_rust::Hint;
use notify_rust::Timeout;

use notify_rust::Image;

pub fn linux_notification_from_image_buffer<'a>(
    title: &'a str,
    message: &'a str,
    image_buffer: ImageBuffer<Rgba<u8>, Vec<u8>>,
    timeout: u32,
) {
    let image_data = image_buffer.to_vec();
    let height = image_buffer.height() as i32;
    let width = image_buffer.width() as i32;
    let image = Image::from_rgba(width, height, image_data).unwrap();

    Notification::new()
        .image_data(image)
        .summary(title)
        .body(message)
        .hint(Hint::Resident(true))
        .timeout(Timeout::Milliseconds(timeout * 1000))
        .show()
        .unwrap();
}

use color_backtrace::install;

static NOTIFICATION_ICON: &[u8] =
    include_bytes!("../../static/icons/rule-202020.png");

use image;

fn main() -> Result<(), ImageError> {
    install();

    let dynamic_image =
        image::load_from_memory(NOTIFICATION_ICON).unwrap();
    let image_buffer = dynamic_image.to_rgba8();
    linux_notification_from_image_buffer(
        "its working",
        "i've sent a notification from image loaded into the binary file of the app",
        image_buffer,
        10);
    Ok(())
}
}


Switch on debug logging with a flag on the CLI or with an environment…

… variable.
Note: probably easier to merge this PR before https://github.com/swisscom/korp/pull/4, as the latter one renames a file.

Just noticed how to do this in a more elegant way. Closing the PR and making a new one.


[BEAM-1726] TestFlinkRunner should assert PAssert success/failure counters

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Coverage increased (+0.01%) to 70.159% when pulling 354bbcc2d87164cd5bb94b7c71d50df1d4c17c93 on aviemzur:flink-test-runner-aggregators into 25b52c5acc3dddf8ee979b1b7a8c201b68c3f268 on apache:master.

Run Flink RunnableOnService

Run Flink RunnableOnService

Coverage decreased (-0.003%) to 70.146% when pulling a8721f147d3b74b50aa2b133d79e580c25e0e9ff on aviemzur:flink-test-runner-aggregators into 25b52c5acc3dddf8ee979b1b7a8c201b68c3f268 on apache:master.

Coverage increased (+0.01%) to 70.159% when pulling a8721f147d3b74b50aa2b133d79e580c25e0e9ff on aviemzur:flink-test-runner-aggregators into 25b52c5acc3dddf8ee979b1b7a8c201b68c3f268 on apache:master.

Run Flink RunnableOnService

Rebased on top of master.
There is still one ROS test failing in Flink Runner currently.

Coverage decreased (-0.007%) to 70.123% when pulling baad1702f0fbe4ad483bb0896510832472c2c72b on aviemzur:flink-test-runner-aggregators into 3f0fe91aabcd9623aff2f9f4bf9dce5038c287f5 on apache:master.

Coverage decreased (-0.01%) to 70.12% when pulling 56c09e3ac64c0919817d8a428179253d86506a6a on aviemzur:flink-test-runner-aggregators into 3f0fe91aabcd9623aff2f9f4bf9dce5038c287f5 on apache:master.

Run Flink RunnableOnService

Run Flink RunnableOnService

Retest this please.

Retest this please.

Retest this please.

This looks good to me now!
Might be good to have another pair of eyes on this, though.
R: @tgroh if you could have a look, that would be great.

Retest this please

Coverage decreased (-0.01%) to 70.12% when pulling 56c09e3ac64c0919817d8a428179253d86506a6a on aviemzur:flink-test-runner-aggregators into 3f0fe91aabcd9623aff2f9f4bf9dce5038c287f5 on apache:master.

Run Flink RunnableOnService

Retest this please.

Coverage increased (+0.03%) to 70.155% when pulling a974f3cd6e550706fce51b39ef23c9bca7cb8778 on aviemzur:flink-test-runner-aggregators into 3f0fe91aabcd9623aff2f9f4bf9dce5038c287f5 on apache:master.

Retest this please.

Coverage increased (+0.03%) to 70.159% when pulling a974f3cd6e550706fce51b39ef23c9bca7cb8778 on aviemzur:flink-test-runner-aggregators into 3f0fe91aabcd9623aff2f9f4bf9dce5038c287f5 on apache:master.

Run Flink RunnableOnService

Coverage decreased (-0.003%) to 70.159% when pulling a974f3cd6e550706fce51b39ef23c9bca7cb8778 on aviemzur:flink-test-runner-aggregators into f7855842d9c132473bef3096bba7cb9f5c281567 on apache:master.

Run Flink RunnableOnService

It looks like org.apache.beam.sdk.transforms.CombineTest.testSessionsCombineWithContext is flaky and sometimes passes and sometimes fails on not enough successful assertions.
This could either be an issue with the code it tests, or it could be an issue with the counters themselves that they are overcounting in some cases and that causes us to have incorrect test successes in this and other tests.

When https://github.com/apache/beam/pull/2417 is merged this PR will no longer be valid as it relies on the aggregators in PAssert.
Closing the PR and leaving the branch open for reference, as the other commits by @aljoscha will probably be needed when another solution is found


Secondary store indices & other derived views should feed off of a changelog

We maintain a bunch of secondary datastore indices and all kinds of derived views to power the different features of the viewer, which are currently maintained in parallel to the main store, inevitably leading to all the usual issues that come with multiple sources of truth.
We should have one and only one source of truth.
The store should publish a changelog, and secondary datastructures should be built and updated by subscribing to that changelog.
This is also a very common entrypoint for all sorts of plugins.
(I swear I've already written that issue before, where did it go :sob:)

Done!


train_cifar10_resnet.py multiple machines training error

try to run the train_cifar10_resnet with 2 machines using command like:
../../tools/launch.py -n 2 -H hosts --sync-dst-dir /tmp/mxnet python /tmp/mxnet/train_cifar10_resnet.py --kv-store dist_sync --batch-size 64 --gpus 0
it spits multiple error msgs of:
Traceback (most recent call last):
File "_ctypes/callbacks.c", line 314, in 'calling callback function'
File "/tmp/mxnet/mxnet/kvstore_server.py", line 38, in server_controller
optimizer = pickle.loads(cmd_body)
File "/usr/lib/python2.7/pickle.py", line 1382, in loads
return Unpickler(file).load()
File "/usr/lib/python2.7/pickle.py", line 858, in load
dispatchkey
File "/usr/lib/python2.7/pickle.py", line 1090, in load_global
klass = self.find_class(module, name)
File "/usr/lib/python2.7/pickle.py", line 1126, in find_class
klass = getattr(mod, name)
AttributeError: 'module' object has no attribute 'Nesterov'
I can successfully run the training on each of the machine locally without problem. However when running using two machines, it can't work. What could be the problem?

@mli

the problem is that Nesterov is not pickleable. Where is the source file? you'd better put it under python/mxnet.

The source file is in the example folder.
On Mon, Oct 10, 2016 at 9:24 AM, Mu Li<EMAIL_ADDRESS>wrote:

the problem is that Nesterov is not pickleable. Where is the source file?
you'd better put it under python/mxnet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/3458#issuecomment-252670519, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AESXbBgvwMyoEYyvfzhChkw9smmM71sJks5qymasgaJpZM4KPbA4
.

good to know it. @antinucleon i remember you have a better pickle solution

That will introduce dependence on dill
On Mon, Oct 10, 2016 at 11:16 Mu Li<EMAIL_ADDRESS>wrote:

good to know it. @antinucleon https://github.com/antinucleon i remember
you have a better pickle solution
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/3458#issuecomment-252698837, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABM13srs06mTWKLfhrksBYQZXHXflsQIks5qyoESgaJpZM4KPbA4
.

--
Sent from mobile phone

I have exactly the same problem as zhonghh

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Ignore snakeyaml major version update

It breaks k3s module. See #6684

Why is this ignored? snakeyaml-1.33 has CVE-2022-1471


JavaCall does not work with 32 bit Julia on Windows

Representative error: https://ci.appveyor.com/project/aviks/javacall-jl/build/1.0.42/job/8tjh878bbqf2h2dl
cc: @andygreenwell you've done some investigation around this i believe?

I get the same error: https://gist.github.com/AndyGreenwell/f0aff057e6d25a37609cae9c64ec837e
Setup under test:

Windows 7 32-bit
Julia 0.4.5
JavaCall master with the PR that uses WinReg.jl
JDK 1.8.0_77

How serious is (and difficult to fix) is non-32-bit support? [I guess this is independent of Windows.]
Most ARM CPUs are non-64-bit, whether of non-Android Linux or Windows Phone (I guess it doesn't have JVM..). I'm thinking of getting an ARM Linux (not Android, or possibly for both..). [32-bit [Windows] CPUs are on the way out, so maybe not too limiting there. Still, thought JDBC.jl was a nice option..]
[Yes, Dalvik is not that JVM. JavaCall doesn't support that or ART, is there an alternative, in that case?]

v0.4.0 tagged with support for julia 0.5

I managed to get JavaCall.init() to work as well as few calls to function on x86 Windows:
https://ci.appveyor.com/project/mkitti/javacall-jl/builds/32767716/job/6mog0jrkqc6s5ea0
This corresponds to 078278ec


update ginkgo-set target for go version 1.18, delete vendor folder

This PR

updates Makefile to fix ginkgo-set target by setting both GOPATH and GOBIN and use go install with version tag to be compatible with version 1.18 which is used in the build Dockerfile.
more details for go get/go install changes

removes the vendor folder to avoid vendor conflict from auto-generated modules.txt

separates test and test-verbose target

Signed-off-by: Sunyanan Choochotkaew<EMAIL_ADDRESS>

@sunya-ch I prefer to keep the vendor directory, in case the vendor pkgs are missing and it saves time on CI.
Can you vendor in the ginkgo pkg?

@sunya-ch I prefer to keep the vendor directory, in case the vendor pkgs are missing and it saves time on CI.
Can you vendor in the ginkgo pkg?

In this amend, I rerun the go mod vendor and have vendor folder back.
The previous generated vendor causes some permission issues but now it is fine I guess.


Fix/replace deprecated logger

Fix/replace deprecated logger

Thanks for submitting a pull request!
We will review it as soon as possible.

AuroraEditor/Base/TabBar/UI/TabBar/TabBar.swift:142:25 The compiler is unable to type-check this expression in reasonable time; try breaking up the expression into distinct sub-expressions

@aurora-care-bear please approve

Hey @0xWDG,
Thanks for your contribution.
Your PR has been merged!
Your name will appear on https://auroraeditor.com/#contributors in a few moments, if you have contributed to the editor.
Thanks again for your contribution!
Kind Regards,
Aurora Editor Team.


Rust bindings: please update protobuf dependency

https://github.com/stepancheg/rust-protobuf/pull/737 fixes a bug in the protobuf crate where it wouldn't interact properly with build systems that use RUSTC_WRAPPER. This was released as a semver-compatible update (v3.7.1), but this crate pins protobuf to =3.2.0 (which is quite unusual in Rust), so sadly reverse dependencies can't pick up the new protobuf.
Would it be possible to update the protobuf dependency?
This is a blocker for https://github.com/rust-lang/rust/pull/127682.

Btw, after posting this issue I got a strange email about a failed pipeline

Created a PR to delete the job. https://github.com/sourcegraph/scip/pull/285

The 3.7.1 version of the protobuf crate code doesn't compile with Rust 1.71.0 (version used in this repo), looking into upgrading that first.
error[E0445]: crate-private trait `MapFieldAccessor` in public interface
   --> /Users/varun/.local/share/mise/installs/rust/1.71.0/registry/src/index.crates.io-6f17d22bba15001f/protobuf-3.7.1/src/reflect/acc/v2/map.rs:115:1
    |
17  |   pub(crate) trait MapFieldAccessor: Send + Sync + 'static {
    |   -------------------------------------------------------- `MapFieldAccessor` declared as crate-private
...
115 | / pub fn make_map_simpler_accessor_new<M, T>(
116 | |     name: &'static str,
117 | |     get_field: for<'a> fn(&'a M) -> &'a T,
118 | |     mut_field: for<'a> fn(&'a mut M) -> &'a mut T,
...   |
121 | |     M: MessageFull,
122 | |     MapFieldAccessorImpl<M, T>: MapFieldAccessor,
    | |_________________________________________________^ can't leak crate-private trait

error[E0446]: private type `MapFieldAccessorImpl<M, T>` in public interface
   --> /Users/varun/.local/share/mise/installs/rust/1.71.0/registry/src/index.crates.io-6f17d22bba15001f/protobuf-3.7.1/src/reflect/acc/v2/map.rs:115:1
    |
33  |   struct MapFieldAccessorImpl<M, T>
    |   --------------------------------- `MapFieldAccessorImpl<M, T>` declared as private
...
115 | / pub fn make_map_simpler_accessor_new<M, T>(
116 | |     name: &'static str,
117 | |     get_field: for<'a> fn(&'a M) -> &'a T,
118 | |     mut_field: for<'a> fn(&'a mut M) -> &'a mut T,
...   |
121 | |     M: MessageFull,
122 | |     MapFieldAccessorImpl<M, T>: MapFieldAccessor,
    | |_________________________________________________^ can't leak private type

Some errors have detailed explanations: E0445, E0446.
For more information about an error, try `rustc --explain E0445`.

but this crate pins protobuf to =3.2.0 (which is quite unusual in Rust),

The reason for this is that the rust-protobuf set of crates doesn't make any guarantees on whether codegen by one version of the protobuf-codegen tool will be compatible with a different version of the protobuf crate. So we've erred on the side of caution by only having one pinned version to reduce the chances of errors downstream.

Thanks for doing the dependency bump :)


Remove version: FineprintSoftware.pdfFactory.Pro version 9.08

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://fineprint.com/release/pdf908pro.exe
OK
(Automated message - build 923)

URL: https://fineprint.com/release/pdf908pro.exe
OK
(Automated message - build 923)

Close with reason: Package still available.;


[webgpu]: deep refactoring

pipeline descriptions simplified
bind groups and layouts systematized
render target introduced (ready for composing and blending)
scene/shape opacity support introduced
code looks cleaner

@SergeyLebedkin Hello, I believe this type of significant refactoring requires a series of smaller-sized patches, as you've already mentioned in the list of changes:

pipeline descriptions simplified
bind groups and layouts systematized
render target introduced (ready for composing and blending)
scene/shape opacity support introduced
code looks cleaner

This would be helpful to track/review the changes from the git history more easily. Could you please create these patches individually?

I will separate this PR on 3 different PRs:

pipelines/layouts refactoring (they should be refactored together)
render target introduction
context and pipelines holder introduction


Document how to add App Icon

I could not find a way to choose an app icon for my Mac icon. Is this supported?
If yes, please add in docs.
If not, please support it

It's currently possible via the macIcon attribute in the construo block, but I plan on changing it so it's in the target block instead:
construo {
  macIcon.set(project.file("path/to/icon.icns")
}

Updated the title to document instead


Dead link error while checking link to a huge markdown file on GitHub

We have some false-positive reports while checking the documentation links from Travis if the target page is huge (e.g. SQL syntax documentation or Uber).
Could you please provide an option to specify timeout after which the link is marked as broken? Also it would be nice to see the reason why the link is considered broken (status code, connection timeout, etc.)

two issues that are now discussed in #72 and fixed by #55 for the verbosity


[IMPROVEMENT] Include the /proc/mounts file in the support-bundle.

Is your improvement request related to a feature? Please describe (👍 if you like this request)
Currently, we are relying on users to manually provide the mount list information to help troubleshoot mounting issues.
Describe the solution you'd like

[ ] Include the /proc/mounts file in the support-bundle.

Describe alternatives you've considered
None
Additional context
https://github.com/longhorn/longhorn/issues/5290#issuecomment-1730727469

@derekbit should we add multipath config if it exists?

@derekbit should we add multipath config if it exists?

Yeah, it is nice to have.


Regent compiler issue with nested references involving intXd

The following code:
import 'regent'
struct Foo {
  fld : int2d,
}
local task main()
  var is = ispace(int2d, {6,4})
  var r = region(is, Foo)
  for i in r do
    r[i].fld = i
  end
  for i in r do
    regentlib.assert(r[i].fld.x == i.x, '')
  end
end
regentlib.start(main)

fails during typechecking with:
dotx.rg:12: no field 'x' in type int2d
    regentlib.assert(r[i].fld.x == i.x, '')
                      ^

If you change it to use a temporary value, it works:
var tmp = r[i].fld
regentlib.assert(tmp.x == i.x, '')

This seems to have been fixed by some recent changes.


[Feature Request] Support for helpers in handlebars in compile mode

Feature request
What is motivation or use case for adding/changing the behavior?
Handlebars supports helpers to extend its functionality. For example, there are no comparison or coalesce operators besides the if statement, which expects a boolean value. This limitation leads to boilerplate code that converts every statement like {{#if (eq value "apple")}} to template({isApple: value === "apple"}) and {{#if isApple}}. Using it with HTML controls to set selected in <select><option ... makes the situation even worse.
Additionally, handlebars-loader supports this feature via the helperDirs option. Implementing this would improve feature parity between the two.
Describe the solution you'd like
I would love for helpers to be compiled into the template function, allowing them to be used in imported templates.
Side note: handlebars-loader doesn’t allow inline scripts in the configuration like this plugin does. I think this is a reasonable tradeoff to enable compiling helpers into the template.
Side note 2: It seems like this plugin doesn’t support ESM helpers:
Error [ERR_REQUIRE_ESM]: require() of ES Module /home/vagrant/workspace/website/src/handlebars/coalesce.js from /home/vagrant/workspace/website/.cache/pnpm/virtual-store-dir/html-bundler-webpack-plugin@4.9.2_handlebars<EMAIL_ADDRESS>not supported.
Instead change the require of coalesce.js in /home/vagrant/workspace/website/.cache/pnpm/virtual-store-dir/html-bundler-webpack-plugin@4.9.2_handlebars<EMAIL_ADDRESS>to a dynamic import() which is available in all CommonJS modules.

Describe alternatives you've considered

Introducing is* variables.
Using handlebars-loader without inheritance.

Appreciation for the useful project

[⭐] After the feature is implemented, do not forget to give a star ⭐

Thank you!

@Bessonov please try the v4.10.0.
Note: the helpers should not contains code for Node.JS or file system, e.g. path.resolve(), fs.readFile(), etc., because the helper's code will be called in browser runtime.
If the helpers option contains a directory to helper files, then helpers must be in CJS format. ESM is not supported.
If you have helpers in ESM format, then you can import in webpack config once and define the helpers option as the object:
const myHelper import from 'myHelper.js';
const HtmlBundlerPlugin = require('@test/html-bundler-webpack-plugin');
const Handlebars = require('handlebars');

module.exports = {
  plugins: [
    new HtmlBundlerPlugin({
      preprocessor: 'handlebars',
      preprocessorOptions: {
        helpers: {
          myHelper, // helper imported from ESM file

          // you can manual create a small simple helpers as a funciton:
          bold(options) {
            return new Handlebars.SafeString(`<strong>${options.fn(this)}</strong>`);
          },
        },
        partials: ['src/partials'],
      },
    }),
  ],
};

Hey @webdiscus,
Thank you! It resolves my use case!
If I understand it correctly, you are stringifying functions, which comes with several limitations. For example, I encountered the following issues:

Semicolons are mandatory (I don't use them, so an exclusion for dprint is needed).
Comments inside the helper functions aren't possible.
require doesn't work as expected (though it works if I provide the correct path where the template is executed, such as an absolute path).

The last point makes using libraries like handlebars-helpers more difficult. I believe they can be used if each helper is compiled into a single function (=without require) in some sort of preprocessing step.
In addition, my webpack configuration is esm-based, which made it a little bit more challenging to find a working combination of commonjs and esm. This setup works for me:
// webpack.config.mjs
import eq from './src/handlebars/eq.cjs'

// eq.cjs
// https://stackoverflow.com/questions/8853396/logical-operator-in-a-handlebars-js-if-conditional/31632215#31632215
module.exports = function () {
	function reduceOp(args, reducer) {
		args = Array.from(args);
		args.pop();
		const first = args.shift();
		return args.reduce(reducer, first);
	};
	return reduceOp(arguments, (a, b) => a === b);
}

Note: the helpers should not contains code for Node.JS or file system, e.g. path.resolve(), fs.readFile(), etc., because the helper's code will be called in browser runtime.

Nope, my runtime is still node :)
console.log(require('fs').readFileSync('/tmp/test'));

works just fine inside helpers! :smile:

@Bessonov

If I understand it correctly, you are stringifying functions, which comes with several limitations. For example, I encountered the following issues

yes, functions will be stringified. By stringify the source code can be modified. I don't know other way how to include helpers code into stringified template function when helpers are defined in options as a custom functions. This is the feature of the plugin.
I have an idea, if a helpers option is a path to helpers directory, then I can read source code as text directly from helper files and inject into template function as the string. I need to do an experiment.

@Bessonov
If I understand, my solution works only partially and can not cover all your use cases?

@webdiscus

If I understand, my solution works only partially and can not cover all your use cases?

At the moment, all my use cases are covered - thank you! (Of course, duplicating functions like reduceOp isn't best practice, but since it's in a very limited area, it's not an issue!)
I haven't explored the internals of handlebars-loader, but perhaps this could be useful for you: https://github.com/pcardune/handlebars-loader/blob/e678ba7aec1eb56bf70767ff1c11ef39281c22b7/index.js#L88

I have an idea, if a helpers option is a path to helpers directory, then I can read source code as text directly from helper files and inject into template function as the string. I need to do an experiment.

If I understand you correctly, this would solve the issue if everything is in the helpers folder, but not if some helpers require/import external modules. (At least after packaging the project, there will be no node_modules anymore, so require('whatever') wouldn't work anymore).
I don’t know much about webpack internals, so this might be a naïve question: could you use webpack for compiling helpers? Since you already control the entry points, maybe webpack could compile the entire chain of required/imported modules and include it in the compiled bundle like:
/***/ "./build/handlebars/utils.mjs":
/*!************************************!*\
  !*** ./build/handlebars/utils.mjs ***!
  \************************************/
/***/ ((__unused_webpack___webpack_module__, __webpack_exports__, __webpack_require__) => {

__webpack_require__.r(__webpack_exports__);
/* harmony export */ __webpack_require__.d(__webpack_exports__, {
/* harmony export */   reduceOp: () => (/* binding */ reduceOp)
/* harmony export */ });
// https://stackoverflow.com/questions/8853396/logical-operator-in-a-handlebars-js-if-conditional/31632215#31632215
function reduceOp(args, reducer) {
    args = Array.from(args);
    args.pop(); // => options
    const first = args.shift();
    return args.reduce(reducer, first);
}
//# sourceMappingURL=utils.mjs.map

/***/ }),

, so the helpers would "magically" work?

could you use webpack for compiling helpers?

Webpack already parses the JS code generated by the plugin, resolves all require and import statements, and injects the compiled code into the resulting compiled JS function. The magic of Webpack is already working.

this would solve the issue if everything is in the helpers folder, but not if some helpers require/import external modules. (At least after packaging the project, there will be no node_modules anymore, so require('whatever') wouldn't work anymore).

I have now other idea... I will generate the code for each helper founded in defined heplers directory:
`let helper = require('${helperFile}');
Handlebars.registerHelper('${name}', helper);`

So, Webpack compile all required files into code, including all required/imported modules from node_modules.

I haven't explored the internals of handlebars-loader, but perhaps this could be useful for you: https://github.com/pcardune/handlebars-loader/blob/e678ba7aec1eb56bf70767ff1c11ef39281c22b7/index.js#L88

The handlebars-loader does the same as I wrote above:
Line: 112
----
 else if (type === "helper") {
      if (foundHelpers["$" + name]) {
        return (
          "__default(require(" +
          loaderUtils.stringifyRequest(loaderApi, foundHelpers["$" + name]) +
          "))"
        );
      }
      foundHelpers["$" + name] = null;

Hm, I assumed that your require:
const Handlebars = require('handlebars');

works, because you put it in the compiled template here:
https://github.com/webdiscus/html-bundler-webpack-plugin/blob/33c99f3f7e837d0d1c6765699e41ae0dd2fad074/src/Loader/Preprocessors/Handlebars/index.js#L268 :
        var Handlebars = __webpack_require__(/*<EMAIL_ADDRESS>*/ "./.cache/pnpm/virtual-store-dir/handlebars@4.7.8/node_modules/handlebars/runtime.js");
        var data = {};

          Handlebars.registerHelper('italic', function (options) {  return new Handlebars.SafeString(`<i>${options.fn(this)}</i>`);});
          Handlebars.registerHelper('eq', function () { function reduceOp(args, reducer) {              args = Array.from(args);                args.pop();             const first = args.shift();             return args.reduce(reducer, first);     };      return reduceOp(arguments, (a, b) => a === b);});

At least I couldn't get require to work on another file.

I haven't explored the internals of handlebars-loader, but perhaps this could be useful for you: https://github.com/pcardune/handlebars-loader/blob/e678ba7aec1eb56bf70767ff1c11ef39281c22b7/index.js#L88

The handlebars-loader does the same as I wrote above:
[..]

You mean the result? Sure! But it seems that the way is different. For example, handlebars-loader can handle esm imports/exports. I found this document: https://github.com/handlebars-lang/handlebars.js/blob/master/docs/compiler-api.md


No model found, but I already ran the download model cell

When I try to run the create session cell, it says no model found.

official models or a custom one ?


TCP proxy from istio gateway to MTLS service fails.

I got a TCP proxy working great, with mtls disabled.
Same service, with a DestinationRule and policy setting mtls - fails. Spent some time debugging, will restart next week - we need to sort it out for 1.0
Please try lastest 1.0 build and create a TCP proxy with mtls enabled for any service, let me know if you get it working. (one way to do it is to declare the port as tcp on fortio, or use the test app)

@costinm , i would be great if you could provide the DestinationRule and AuthPolicy in this issue, which would make it easier for the reproduce. And have you tried the http proxy with mTLS?


Getting to know each other...

Hi everyone,
after our first Q&A session I was thinking it could be nice to know a bit more about us and our research interests  (also to team up for the Blog post). Do you agree?
I will start proposing a "template" but feel free to share anything you like about you. thanks!
Name Surname: Arianna Pisciella
Current position: 1st year PhD student at HEC Lausanne
Research interests/methods: ESG reporting, non-financial disclosure, real effects, textual analysis

Thank you Arianna, great idea!!
Name Surname: Bianca Minuth
Current position: 1st year PhD student at ESCP Business School in Paris
Research interests/methods: financial disclosure, social media, CEO personality traits, communication style, textual analysis

Hi!
I am Rusha and I am a first year student at NHH Norway. My research interests include corporate disclosures, corporate governance, tax research

Hi Everyone!
Name Surname: David Aguilera
Current position: 2st year PhD student at Universitat de Valencia
Research interests/methods: non-financial disclosure &  Corporate Governance (board diversity)


Edits to command help

I did a comprehensive review of the rhoas CLI command help and made edits to make the help more consistent and to add some additional detail for some of the features.
I did not review the Service Registry commands, but those can be done in a future PR.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation change
[ ] Other (please specify)

@wtrocki @rkpattnaik780 I made some updates based on your feedback - thanks! Let me know if everything looks good. Do I need to rebuild the help in order for the checks to pass? I was waiting until all of the reviews are complete.


Add an EVP demo for RSA key generation

Generate a 4096 bit key.  Include a validation step (separate function).
It should be the current best practice (fetch), have proper error handling and resource allocation/deallocation and be in the style of a known answer test.

Is this to add an example of an RSA key generation code snippet to the respective man page(s)?

It is adding a new program to the demo directory that generates an RSA key.
It's fine to do more, e.g. save to PEM file.

FWIW, I closed the pull request because I don't have my employer's permission to contribute it.


Get-Content -Wait fails on WSL2 files

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Map your WSL isntance to a drive letter.
For instance my W: drive is mapped to \wsl$\Ubuntu
Run Get-Content with the -Path and -Wait parameter to print out the text file and monitor it for changes:
Example: Get-Content -Path "W:\home\joseph\myvoipms\cron.log" -Wait
Expected behavior
Get-Content should wait for new content in the file, and print it as it comes in.

Actual behavior
Get-Content prints out the file, but then says "Get-Content: Incorrect function."

Error details
Exception             :
    Type            : System.ComponentModel.Win32Exception
    NativeErrorCode : 1
    ErrorCode       : -2147467259
    TargetSite      :
        Name          : WaitForChanges
        DeclaringType : [Microsoft.PowerShell.Commands.FileSystemContentReaderWriter]
        MemberType    : Method
        Module        : System.Management.Automation.dll
    Message         : Incorrect function.
    Source          : System.Management.Automation
    HResult         : -2147467259
    StackTrace      :
   at Microsoft.PowerShell.Commands.FileSystemContentReaderWriter.WaitForChanges(String filePath, FileMode fileMode,
FileAccess fileAccess, FileShare fileShare, Encoding fileEncoding)
   at Microsoft.PowerShell.Commands.FileSystemContentReaderWriter.ReadByLine(Boolean waitChanges, List`1 blocks,
Boolean readBackward)
   at Microsoft.PowerShell.Commands.FileSystemContentReaderWriter.Read(Int64 readCount)
   at Microsoft.PowerShell.Commands.GetContentCommand.ProcessRecord()
CategoryInfo          : InvalidOperation: (:) [Get-Content], Win32Exception
FullyQualifiedErrorId : ProviderContentReadError,Microsoft.PowerShell.Commands.GetContentCommand
InvocationInfo        :
    MyCommand        : Get-Content
    ScriptLineNumber : 1
    OffsetInLine     : 1
    HistoryId        : 3
    Line             : Get-Content -Path "W:\home\josephur\myvoipms\cron.log" -Wait
    Statement        : Get-Content -Path "W:\home\josephur\myvoipms\cron.log" -Wait
    PositionMessage  : At line:1 char:1
                       + Get-Content -Path "W:\home\josephur\myvoipms\cron.log" -Wait
                       + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : Get-Content
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo :
      0
      1

Environment data
Name                           Value
----                           -----
PSVersion                      7.4.3
PSEdition                      Core
GitCommitId                    7.4.3
OS                             Microsoft Windows 10.0.26100
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Does Linux PowerShell running within WSL happily wait on the same file?
What file system type does the drive appear as to Windows?
Get-Content

Wait is a dynamic parameter that the FileSystem provider adds to the Get-Content cmdlet. This parameter works only in file system drives. Wait can't be combined with Raw.

FileSystemContentStream.cs
WaitForChanges uses System.IO.FileSystemWatcher

Remote computers must have one of the required platforms installed for the component to function properly.

I don't see a list of required platforms for this function

@Josephur does this still repro for you in latest Windows?  Members of the cmdlet WG hasn't been able to repro this and works as expected

This issue still exists for me using the latest PowerShell 7.4.6
Here is a video of the bug:
Get-Content should with the -Wait switch latch to the file and continue to read it, I wonder if it's something to do with the mapped drive to WSL perhaps?

Oh and to answer your question, if I install Powershell within my Ubuntu WSL, and use the same command on the same file, it works fine.  It's only when Windows is using the mapped drive to access the Linux files when the error occurs.
Here's the details of the mapped drive:
C:\Users\Josephur>net use
New connections will be remembered.
Status       Local     Remote                    Network
         W:        \\wsl$\Ubuntu             Plan 9 Network Provider

This looks like an issue in the WSL file handler, I would suggest opening an issue in the WSL repo with a PowerShell 5.1 repro


Notification getting duplicate error.

Laravel Version: 5.4.16
PHP Version: 7.1
Database Driver & Version: MySQL
Queue Driver: Beanstalkd

Description:
Laravel notification getting duplicate error when using queue along batch notification to atleast ~100 users
Steps To Reproduce:

Create a notification with toDatabase
Make it ShouldQueue
Execute atleast ~100 users into that notification
You will get Duplicate entry in the notifications table

Attached Error in the notifications[exception] table
error.txt

Can't replicate your issue, a unique ID is always generated.

Make sure the notifiables list has unique entities and that you're not sending to the database channel multiple times.

I have the exact same issue happening on a production site. Still investigating what happened, but I've never experienced this until last week. It's happened multiple times on my queue, I'd say about 10 times total so far. If I come up with anything else I'll reply back.

Hi @apbassi89,
Did you solve it? I run into similar issue using code below:
$users->each(function($user) use($object){
    $result = $user->user_id === $object->user_id ? $object->xxx : $object->yyy;
    Notification::send($user, new SampleNotification($object,$result)); //SampleNotification implements ShouldQueue
});

3 users
2 have the same $result

@dariuszjastrzebski Unfortunately I spent several days on it and could not figure it out. I ended up writing a custom integration with Nexmo to send my SMS notifications, and replaced all the locations where I queued a notification with a queuing of my new SMS Job.
That solved the problem for me, but obviously that doesn't help anyone who relies on the built in notifications... Sorry about that.

I think, I am having the same issue with this error message
PDOException: SQLSTATE[23505]: Unique violation: 7 ERROR: duplicate key value violates unique constraint "notifications_pkey"
I am using postgres and my laravel verison is 5.6. I think the uuid generated by the system is not checking existence of 'id' before trying to insert it.

I am also getting the same issue. What is the solution so the we don't get duplicate notification error?
Attached screenshot for the same.

I have faced the same problem, the solution was to change the type of "id" column in notifications migration file from  $table->increments('id'); to $table->uuid('id')->primary();. and after that you have to refresh migrations.
another way is to change the column from database directly to type "char" with Length=36

I experienced this issue as well. When I used a single notification object with three channels (database, apple push notification, google push notification), I was getting duplicate notifications. When I split the three notifications out into three different notifications I no longer got duplicate notifications.

do you have an example, how to split and call three notifications?

Having this error in production as well
[2021-05-22 22:26:25] production.ERROR: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '707d4e26-af4e-4dc7-927f-6e414dbed562' for key 'PRIMARY' (SQL: insert into `notifications` (`id`, `type`, `data`, `read_at`, `notifiable_id`, `notifiable_type`, `updated_at`, `created_at`) values...

Anyone has a clue?

Here is how I split them into three instead of a single with all three in one via method array.
use NotificationChannels\Fcm\FcmChannel; class EventAndroidPushNotification extends Notification{    ... public function via($notifiable): array    {        return [FcmChannel::class];    } ... }
`
use NotificationChannels\Apn\ApnChannel;
class EventApplePushNotification extends Notification{
...
public function via($notifiable): array    {        return [ApnChannel::class];    }
...
}
`
class EventDatabaseNotification extends Notification{ ... public function via($notifiable): array    {        return ['database'];    } ... }

I am facing same issue in May 2022. Is there any solution?

The issue is because ID is not getting a value. It only happens on a project recently updated to Laravel 8. A fresh installation doesn't have this issue. @themsaid

It's been a long time and no one in the core team checking on this.
The solution for this is to use uuid, uuid is much more unique when running on the same micro time, concurrent process is inevitable and surely will occur a duplicated id.
Extend the model and replace the id as uuid and create an auto generated uuid on create event inside the model, that should solve your problem.


DRAKON: soviet flowchart

I am https://en.wikipedia.org/wiki/DRAKON fan. it is a soviet version of flowcharts, there are a lot of rules but it is ok.
Some time ago (maybe one year) I tryed to code Drakon into PlantUML but it was hard work.
Well, I add this issue as "enhancement"....cold war "towarish"....

Given how long ago this issue's activity died out, I'm closing it.
This allows us to maintain a fresh list of issues the community is interested in.


ARROW-2822: [C++] Zero padding bytes in PoolBuffer

See the JIRA discussion.

Codecov Report

Merging #2239 into master will increase coverage by 2.41%.
The diff coverage is 91.89%.

@@            Coverage Diff             @@
##           master    #2239      +/-   ##
==========================================
+ Coverage    84.2%   86.61%   +2.41%
==========================================
  Files         288      232      -56
  Lines       44069    41374    -2695
==========================================
- Hits        37107    35837    -1270
+ Misses       6931     5537    -1394
+ Partials       31        0      -31

Impacted Files
Coverage Δ

cpp/src/arrow/util/io-util.h
100% <ø> (ø)
:arrow_up:

cpp/src/arrow/io/hdfs.cc
0.33% <0%> (-0.01%)
:arrow_down:

cpp/src/arrow/ipc/metadata-internal.cc
91.63% <100%> (ø)
:arrow_up:

cpp/src/arrow/util/hash.cc
100% <100%> (ø)
:arrow_up:

cpp/src/arrow/io/file.cc
96.1% <100%> (+0.03%)
:arrow_up:

cpp/src/arrow/ipc/json-internal.cc
89.77% <100%> (-0.02%)
:arrow_down:

cpp/src/arrow/buffer-test.cc
100% <100%> (ø)
:arrow_up:

cpp/src/arrow/compute/kernels/hash.cc
95.63% <100%> (+0.01%)
:arrow_up:

cpp/src/arrow/test-util.h
74.61% <100%> (-1.17%)
:arrow_down:

cpp/src/arrow/builder.h
98.47% <100%> (+0.04%)
:arrow_up:

... and 63 more

Continue to review full report at Codecov.

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

Hi guys. I'll try to incorporate the changes and rework the warning logic to be more precise in the comming days (hopefully tomorrow).

Ok, so there was a possibility I totally haven't considered before: we can just remove the data() and null_bitmap from the builders. Amazingly, it doesn't break much besides couple of tests and the DictionaryBuilder. I adjusted DictionaryBuilder and it seems to work like a charm.
Does anyone know, if some 3rd party code relies on these methods? I would argue, they should be considered implementation detail, but we might want to put a deprecation warning before removing them alltogether.

Does anyone know, if some 3rd party code relies on these methods? I would argue, they should be considered implementation detail, but we might want to put a deprecation warning before removing them alltogether.

I would deprecate them first indeed.

Rebased on the master.

So, what orc adapter did was to use std::copy to copy data from int64_t* input to a Int32 builder by specifying data() as a target. This works because std::copy takes iterators and they just need to conform to the interator interface. IThe builders, on the other hand, either accept a vector or a raw pointer, both have to have the exact expected type to work.
My solution is to overload the AppendValues method to accept iterators. Then we can just use std::copy internally. The number of existing overloads with all possible combination of pointer and vector arguments suggests that we might want to switch to iterator-accepting interfaces at some point in the future, since they are more general. In the ideal world we'd have ranges, but alas...
An additional problem was that orc adapter used data pointer to directly add a computed value (time in nanoseconds). What is easily done in python with generators, is not supported in any current C++ version directly (excluding non-standard extensions). I added the LazyRange, a thin wrapper around a callable (e.g. lambda) which returns objects conforming to the iterator interface. What looks like a complicated construct, can be optimized by the compiler very well (I added some benchmarks).
I really think we should remove/deprecate data() and null_bitmap() from the builders, but it's apparent, we'd need some interface restructuring. I think the changes would be beneficial in the long term, but I'm also not happy about the size of this diff. I would suggest that as long as the general direction is acceptable, we fix this PR's code and leave the more farreaching consideration to be fixed in further PRs.

Ok, so funny thing: MSVC14 (i.e. 2015) issues warning on narrowing conversion when using std::copy. The funny part is that none other compiler, not even later MSVC versions, seem to be bothered by it.
Which raises the question, if the orc adapter in its current form has even been through CI, because [here|https://github.com/apache/arrow/blob/master/cpp/src/arrow/adapters/orc/adapter.cc#L535] it totaly copies int64_t to int32_t and smaller. Of course, it's not a problem, since we check the supplied type beforehand, but it shouldn't compile with MSVC 2015.
With the supplied changes it's actually pretty easy to fix the warning: just wrap the iterator with a lambda which does a static_cast to the right type. Should I do it? Should be 0 costs when compiled on Release.

@alendit can you confirm when the PR is ready for review again? Thanks for working on this!

I would suggest that as long as the general direction is acceptable, we fix this PR's code and leave the more farreaching consideration to be fixed in further PRs.

Sounds good to me

Reviewing this again now

:+1: ok, I'm merging so we can get your other PR rebased. Thanks @alendit @pitrou for your careful attention on this issue

Nice, glad it went through. Thx both of you for the help and patience :)
Once further question: for really small changes, like removing that one const, do I open a JIRA issue, too? Or is there a simplified procedure, so to not to swamp the queue?

You can open a PR but don't bother creating a JIRA. I'll merge once the build runs


BUG: File upload fails if full path is used and file is not in CWD

Describe the bug
When adding artifact with filename as full path, the filname is extracted and before upload the current wd path + filename is used to upload as seen below in neptune_sacred/impl/init.py
def artifact_event(self, name, filename, metadata=None, content_type=None):
    filename = os.path.split(filename)[-1]
    self._run[self.base_namespace][f"io_files/artifacts/{name}"].upload(filename)

this causes error for upload as below:
ERROR - neptune.internal.operation_processors.async_operation_processor - Error occurred during asynchronous operation processing: Cannot upload file "current_wd_path/filename": Path not found or is a not a file.
Expected behavior
No error and it should upload to neptune and visible to use in dashboard etc
Solution
remove line 139 #filename = os.path.split(filename)[-1]

Hey @naveenragur 👋
We just released neptune-sacred v1.0.1, which fixes this issue 🎉
Can you let me know if this works? And thanks again for reporting this! 🙏


Add Array#initialize signature that takes an Array

Add the version of the signature that allows an Array as a parameter to Array#initialize (as described in the ruby docs)
Motivation
https://github.com/sorbet/sorbet/issues/1913
Test plan
Added test case(s) in test/testdata/rbi/array.rb

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build result here:
→ https://go/builds/bui_HYvaoyWjHo3XhA

Sorry for the delay on looking at this, could you rebase it on master?

Sorry for the delay on looking at this, could you rebase it on master?

sure thing! Rebased into master

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build result here:
→ https://go/builds/bui_HYyc0LKsjs6Qta


fix: move class-level attributes to instance-level in Conversation class

Fixes #417
Why:
Class-level attributes persisted state between sessions. Making them instance-level ensures each session starts fresh.
What:
Moved _thread, _should_stop, _conversation_id, and _last_interrupt_id from class-level to instance-level within the init method.
Note: This pull request is a proof of concept, demonstrating how moving class-level attributes to instance-level resolves the session interference issue. I understand this can’t be merged directly due to the code generation process.

Thank you @acse-efk23! @tudor11l,  @lacop11 - do you think this is something we want to merge? Makes sense to isolate these at instance-level rather than class-level

@louisjoecodes we fixed the only one that was causing a problem which was the mutable (by not initialising it with a default value)
threading.Event = threading.Event()
I'm not a huge fan of class-level variables especially when they do not have the same value across instances. But it depends on what others think as well.

Thanks @acse-efk23 !
We have fixed the mutable _should_stop issue (I didn't notice your PR earlier unfortunately, would save us some time debugging!) but I agree we need a more consistent approach.
Note that these are not class variables but instance variables, IIUC. See https://docs.python.org/3/library/typing.html#typing.ClassVar.
The problem was the mutable default value. I think we should be fine just dropping all default values  and only keeping type hints on the class level, then putting all defaults into __init__. We can do it for the = None and = 0 even though they are not mutable and don't cause issues, just to be consistent and avoid accidentally adding more mutable defaults in future.
@acse-efk23 BTW:

I understand this can’t be merged directly due to the code generation process.
This particular file is not generated and is excluded in https://github.com/elevenlabs/elevenlabs-python/blob/main/.fernignore#L4 so it won't be overwritten.

If you would like to adjust the PR to keep the type hints as they were and just move the value assignment we would be happy to merge it. :)

Thank you for the review, @lacop11 . I have updated the pull request accordingly.

Looks good but the branch is out of date and conflicts, can you please rebase + resolve?

Thank you, @lacop11 . I have rebased and solved the conflicts.


How to use GridLayoutProvider

I am trying to use GridLayoutProvider , I am not sure about the function. Doc not clear can you please explain, what was the expectation function on the that

I'm in the same boat. I've been able to get it to render but can only manage to get 1 item per row.
Can't even find any examples with a full GitHub search.
Is GridLayoutProvider actually ready?  Thanks!


🐛 Rename APIExports to avoid collisions

Summary
This PR renames some APIExport objects, to avoid name collisions.
Related issue(s)
Fixes #1261

/approve


remember me

I am confused with remember me, when the cookie is set, then what to do when user is coming back in order to authenticate them? How to validate the cookie value or what should I do? _username and _password is a condition that remember me recognizes them to automatically allow user?

the firewall will automatically know this, you shouldn't have to do anything except configure the firewall to accept the remember me token

Thanks. Any idea about this? https://github.com/symfony/symfony/issues/24797

I did read https://silex.symfony.com/doc/2.0/providers/remember_me.html
and here is my code. I expected a cookie will be set, but I see no REMEMBERME cookie in developper tool. What wrong I did?
$app = new Silex\Application();
$app->register(new Silex\Provider\SecurityServiceProvider());
$app->register(new Silex\Provider\RememberMeServiceProvider());
$app['security.firewalls'] = array(
'my-firewall' => array(
'pattern'     => '^/secure$',
'remember_me' => array(
'key'                => 'Choose_A_Unique_Random_Key',
'always_remember_me' => true,
/* Other options */
),
),
);

Closing as support should be done on StackOverflow instead.


remove false assertions on attributes like required and recommended?

I systematically included some false assertions, although the general LinkML practice is to omit false attribute assertions

I am really inclined to move forward with this since the schema file is so large
this may improve the performance of some steps, and it may also save us from needing to use GitHub Large File storage for a while

there are 2708 lines matching the pattern  +.*: false\n


ChatGPT Hard-Fork

Hey there love this plugin 🔥
I've spun off a hard-fork here that is based on ChatGPT and removes some functionality that you have here (diverges quite a bit from this main)
Changes introduced:

Streaming completions with cancelling
ChatGPT-only calls for completions & edits
Removed some settings and loading state indicators that are no longer needed

Let me know if you'd like me to open a PR to merge those changes in 🙌

Hi, sounds promising! I'll play around with your fork this weekend.
How does the streaming work? It actually updates the buffer in realtime?

Cool 🙌 Streaming uses the --no-buffer option of curl together with the api’s stream: true option and inserts every new token into the buffer as they’re available.

I had a go with it over the weekend, awesome work!! The streaming really makes it feel more responsive!
I like the idea of using the Chat API, since it's faster and cheaper, but the results are worse than the regular completions/edits APIs. It seems like Chat has issues where it ignores the prompt and reverts to being a virtual assistant. I think they intend to fix that in a future model though, so let's keep an eye on it... At the pace that things are going this could be an option in a month or two.


Do not shade instrumentation and io.grpc.Context

io.grpc.Context is a global state (per-thread context propagation) that cannot be shaded in order to allow context propagation from the application.
com.google.instrumentation:instrumentation-api uses reflection to load the implementation and shading this package will create a different library that cannot load the implementation. One of the biggest problem is the interaction with the io.grpc.Context, you cannot get the same Context.Key (even if the same string is used) so if this library is shaded then interaction between libraries via the io.grpc.Context will be broken.

cc/ @ejona86 @zhangkun83 from gRPC

We have to support a variety of situations that require shading, including Dataflow v1, which has a dependency on grpc 0.13.0.  We would like to support tracing even in Dataflow v1, if possible.
I do understand that shading is a major source of problems.  For now, we can't submit this PR as is.

com.google.instrumentation:instrumentation-api uses reflection to load the implementation and shading this package will create a different library that cannot load the implementation.

You would need a shaded implementation as well, so that it will match the expected name.

@ejona86  you are correct but the problem is that we need multiple libraries (bigtable and the main application for example) to communicate to each other in the same binary. If bigtable library shades the implementation then the application must use the shaded API in order to pass a Span to bigtable otherwise is not going to work.
Same problem you have with grpc.Context if I set a deadline (or any other property) in my application bigtable library will not be able to read it because of this problem.

Shading is incompatible with those use cases. You have to either resolve the dependency problems to remove the need to shade, shade the other library instead of this one, or give up on the features.
The instrumentation library is one of the most important things to shade, since it is the least API stable. io.grpc.Context could be fine to not shade, except that we only split Context into a separate artifact in 1.0.1.

Given @ejona86 comments, it sounds like we should not merge this PR. @bogdandrutu are you ok with that?

We have plans to change around our shading logic over the next couple of weeks.  In most cases that matter, users will not use shaded Cloud Bigtable artifacts.  The only exceptions will be Dataflow v1 and Hadoop.  If users ask for propagation of Spans for those environments, we'll figure something out later.


Option to show alternative answers in the "expected answer" box

The motivation for this is to show something like "Expected answer: 1 or -1".
There could be an option "show as an expected answer" for alternatives, which would cause another input box to be shown in the "expected answer" box, containing this alternative's expected answer.


Alternative to Provider/wrapper that doesn't require id or class

The deprecated Provider/wrapper has the ideal API since you don't need to specify an id or class as the anchor. I haven't seen a way to achieve this with the latest API, but maybe I missed it. Can someone point me in the right direction?
https://react-tooltip.com/docs/examples/provider-wrapper
import { Tooltip, TooltipWrapper } from 'react-tooltip';

<TooltipWrapper content="I am using a global tooltip!">
   ◕‿‿◕
</TooltipWrapper>
<TooltipWrapper content="This is the same tooltip!">
   ◕‿‿◕
</TooltipWrapper>
// the tooltip component can be placed anywhere inside the provider
<Tooltip />

Ideal use case

Works great and do not have to manage anchors for all buttons/links.

    <TooltipWrapper
      place={place}
      content={message}
      positionStrategy={positionStrategy}
    >
      {children}
    </TooltipWrapper>

Not ideal

This doesn't work because all tooltips will have the same anchor tag.. so that means we always have to specify custom anchor for every button / link that needs a tool tip.

{children}

    <Tooltip
      anchorSelect={`.tooltipAnchor`}
      place={place}
      content={message}
      positionStrategy={positionStrategy}
    />

Support for the Provider/Wrapper was dropped because of the overhead for setting up, and complexity added to the tooltip internal code, so definitely avoid using it, since it will be removed in a future version.
To best help you here, can you be more specific on what you mean by not having to "manage anchors for all buttons"?
Have you looked at the data-tooltip-* attributes?
https://react-tooltip.com/docs/options#available-attributes
Does this not work for your use case?

@gabrieljablonski Yes you are right I can use data-tooltip-id and <Tooltip id="my-tooltip" /> Thanks for the quick help!

FYI,
Looks like I understood the documentation wrong, thought it meant to have the <Tooltip id="my-tooltip" /> on each component which caused rendering problems.
I think I was confused because other examples show it together...
◕‿‿◕
<Tooltip anchorSelect="#not-clickable">
  <button>You can't click me :(</button>
</Tooltip>

Maybe it makes sense to specify that <Tooltip id="my-tooltip" /> can be placed in the App.tsx?

Good to know.
Right now we feel like the documentation is clear enough. If more people end up having the same trouble, we'll consider updating it. Thanks for the suggestion though.


API for update product

Story: Update existing book
Note: As security approach is not yet decided for now we can proceed without Authentication and later add security support (most likely OAuth security)
Endpoint : PUT /api/products/{code}
Security: N/A
Request Body:
{
  "name": "Book Title", // mandatory
  "description": "book description", // optional
  "image_url": "https://images.com/1.png", // optional
  "price": 24.50 // mandatory
}

Success Response:
Status Code: 200
Body:
{
  "id": "uuid",
  "code": "ISBN",
  "name": "Book Title",
  "description": "book description",
  "image_url": "https://images.com/1.png",
  "price": 24.50
}

Error Response - Bad Request
Reasons: Missing required fields, Book with ISBN not exist etc.
Status Code: 400
Body:
  {
      "message": "Product with ISBN not exist"
  }

Can you also assign this one to me?