fix: archway channel event

Description:
Commit Message
fix: archway eventlogs for channels

see the guidelines for commit messages.
Changelog Entry
version: <log entry>

Checklist:

[ ] I have performed a self-review of my own code
[ ] I have documented my code in accordance with the documentation guidelines
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have run the unit tests
[ ] I only have one commit (if not, squash them into one commit).
[ ] I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Codecov Report

Merging #458 (3b8e233) into main (d6f1986) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #458      +/-   ##
============================================
+ Coverage     67.19%   67.21%   +0.01%
  Complexity      553      553
============================================
  Files           191      191
  Lines         17395    17405      +10
  Branches        350      350
============================================
+ Hits          11688    11698      +10
  Misses         5528     5528
  Partials        179      179

Flag
Coverage Δ

rust
65.03% <100.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...osmwasm-vm/cw-ibc-core/src/ics04_channel/events.rs
99.56% <100.00%> (+0.01%)
:arrow_up:


Introduce health-checks and readiness checks for mesh serve command

Awesome!

Related: https://github.com/Urigo/graphql-mesh/issues/1561
@chompy18 fyi

Awesome!
Thanks @dotansimha


Update DeployBuilder

Update the `DeployBuilder to allow setting a gas_price for a deploy.

go go gadget go

bors r+

bors r+


Uncaught TypeError: Cannot read property 'element' of undefined

[Enter steps to reproduce:]

...
...

Atom: 1.36.0 x64
Electron: 2.0.18
OS: Microsoft Windows 10 Home Single Language
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'element' of undefined
At C:\Users\Annoymous\AppData\Local\atom\app-1.36.0\resources\app\static\<embedded>:11

TypeError: Cannot read property 'element' of undefined
    at TextEditorComponent.measureLongestLineWidth (~/AppData/Local/atom/app-1.36.0/resources/app/static/<embedded>:11:104563)
    at TextEditorComponent.measureContentDuringUpdateSync (~/AppData/Local/atom/app-1.36.0/resources/app/static/<embedded>:11:70559)
    at e.readDocument (~/AppData/Local/atom/app-1.36.0/resources/app/static/<embedded>:11:67706)
    at ViewRegistry.performDocumentUpdate (~/AppData/Local/atom/app-1.36.0/resources/app/static/<embedded>:11:326811)

Commands
     -6:33.3.0 core:paste (input.hidden-input)
     -6:13.2.0 atom-beautify:beautify-editor (input.hidden-input)
  2x -0:25.1.0 editor:newline (input.hidden-input)

Non-Core Packages
atom-beautify 0.33.4
atom-html-preview 0.2.6
color-picker 2.3.0
csscomb 0.3.1
emmet 2.4.3
file-type-icons 1.3.4
html-outliner 0.1.1
ocaml-indent 0.7.1

Thanks for the report!  Can you confirm a few things for us?

What were you doing at the time of the error?
Is the error is reproducible?
If reproducible, do you see the error in safe mode (atom --safe)?

Had the same problema. it started yesterday at night (when Atom probably updated after it was closed). Experiencing strange situations here. When I click a file from treeview, it pins the tab up there in the window. But it doesn't show the code. And I can't access de file unless I click the tab. If I have other files opened, I cant navigate to that new file. It's like it's not there. And when that happens, if I enter that file (by clicking the tab), it doesn't do things like "automatic brackets/parentesis". Using.
When I click a file in the treeview, console logs this:
Uncaught (in promise) Cannot get services before activation
Promise rejected (async)
t.exports.TreeView.fileViewEntryClicked @ VM22 <embedded>:14
t.exports.TreeView.entryClicked @ VM22 <embedded>:14
(anonymous) @ VM22 <embedded>:14

It does not happen in --safe mode


Add support for AWS ECR

Pricing details: https://aws.amazon.com/ecr/pricing/
Terraform resources: aws_ecr_*

Can I try this one out?

@emhagman of course, thank you!
There's information for adding new resources here: https://github.com/infracost/infracost/blob/master/CONTRIBUTING.md#adding-new-resources

@emhagman of course, thank you!
There's information for adding new resources here: https://github.com/infracost/infracost/blob/master/CONTRIBUTING.md#adding-new-resources

Great, thanks :)

@emhagman are you still interested in working on this? it should be fairly quite. Also FYI we're willing to pay contributors to add resources, ping me, alikhajeh1, on Discord if you're interested.

@alikhajeh1 I'll pick this one up.

The PR for this has been merged. FYI @emhagman and @syphernl it'll be in the next release, version v0.7.8, which should go live tmrw.


Use mobx transformers on state function to memoize already computed values

This features delivers:

Usage of transformers to aggregate the state
The ability to track particular models as opposed to all models
Naming the state for debugging purposes

Also, this fixes the two observable layers approach we used to have.

This PR has introduced a breaking change as before we used to define the state as a function that returns a function. (cannot recall exactly the reason why)
But now, all state aggregators definition is just a single function.


Update daemonset to apps/v1 api version

The extensions/v1beta1 API version has been deprecated since kubernetes 1.8 (https://github.com/kubernetes/kubernetes/pull/51828).
Update the fluentd DaemonSets to apiVersion: apps/v1.

It is also necessary to set the selector and template metadata
missing required field "selector" in io.k8s.api.apps.v1.DaemonSetSpec
e.g.
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: fluentd-stackdriver
  namespace: kube-system
  labels:
    k8s-app: fluentd-logging
    version: v1
spec:
  selector:
    matchLabels:
      name: fluentd-stackdriver
  template:
    metadata:
      labels:
        name: fluentd-stackdriver
        k8s-app: fluentd-logging
        version: v1
    spec:
    ...

This issue should be mainly fixed in https://github.com/fluent/fluentd-kubernetes-daemonset/commit/ae351a6fbd32b3f904a2160ddd92b417fb26160f, https://github.com/fluent/fluentd-kubernetes-daemonset/commit/d8480890d557fc3b3c7036938668af8e58b5934e, and https://github.com/fluent/fluentd-kubernetes-daemonset/commit/bc2b417f9d7bdeaca8211e93eb75ea6cc889d9d5.
Closing.


sort order of param/vars/resources/outputs (etc)

Should we standardize the ordering of the top level keywords in modules?  Thinking:

It's often easier to reason over when this is consistent...
params/outputs define the contract, having that be easy to discern is useful
should be easy to automate

I agree we should standardize for params/outputs. Params should definitely all be declared at the top. Outputs could either be 2nd or last, but should also be grouped together.
I don't feel as strongly about the order of variables/resources.
Created this issue in bicep to track: https://github.com/Azure/bicep/issues/6602

Dear @bmoore-msft, Azure Verified Modules (AVM) will become the single Microsoft standard for Bicep modules. Transitioning to AVM, requests for new features or bug fixes will need to be submitted against the appropriate AVM module via AVM Module Issues.
As part of this transition, the new AVM CI environment is replacing the existing CI formerly leveraged within the Bicep Registry Modules (BRM) repository. As a result of this, issues and features related to the existing BRM CI will be closed. If you think your issue still needs addressing, please raise a new issue related to the AVM CI, here.
See this informational notice for more details on the upcoming changes.
Thank you for your understanding!


Hacky youtube search

:ship:

Fixed correctly in c9a60a1561247b5ccb92f9aa6a05e4365cb8ee2c .... You're fired @LinuxMercedes :fire:


Fix link to PDF file

It seems a new PDF file got added. This PR changes the link to that PDF file in the README.

You might also consider just naming the file manual.pdf to avoid this issue in the future 😄

I just fixed some typos and indeed renamed it so the readme doesn't need to be updated :) thanks!


docs: Example on adding metadata to a text-classification dataset

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.

Links to be corrected.

Closes #4181
Type of change
(Remember to title the PR according to the type of change)

[ ] Documentation update

How Has This Been Tested
(Please describe the tests that you ran to verify your changes.)

[ ] sphinx-autobuild (read Developer Documentation for more details)

Checklist

[ ] I added relevant documentation
[ ] I followed the style guidelines of this project
[ ] I did a self-review of my code
[ ] I made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I filled out the contributor form (see text above)
[ ] I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

@kursathalat, can you also align with the most recent phrasing and formatting in this pull? https://github.com/argilla-io/argilla/pull/4208

Addressed in https://github.com/argilla-io/argilla/pull/4313


chore: bdd-gh-1592622724 to 0.0.1

chore: Promote bdd-gh-1592622724 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


No-coalescing + pixel callback = ?

Currently when disabling coalescing and using the pixel callback function, it still returns image-sized frames. This is probably not what we want in that case; it should return a frame-sized frame instead, like it does with the buffer methods. That or we should just not allow using that combination (for now).

The pipeline has probably fixed this. Is it still a problem?

please re-open if the problem is still there


MITREATT&CK

MITRE ATT&CK for Cloud Matrix
Fixes #
Proposed Changes

This file already exists in master branch with same name. If you want some changes I think you should move the file so that it gets deleted from master and then we will push it to solution. Otherwise it will create a copy.

Yes, these are being migrated from Workbooks to Solutions. Upon GA, I'll depreciate the Workbooks upon GA


Hardware support documentation and gfx1031

Hi! I was checking out the new HIP SDK for windows and I noticed that the hardware support page refers to the 67xx GPUs as gfx1032. I'm pretty sure this is incorrect? All other documentation I can find refers to Navi 22 GPUs as target gfx1031-and there are reports of success compiling HIP for that target on the linux side of things. Is this an error, and if so can the documentation be corrected and support clarified? Appreciate it.

@Maetveis can you double check the tables? I think we probably made a typo.

gfx1031 is not yet supported AFAIK.
Tensorflow says "Ignoring visible gpu device (device: 0, name: , pci bus id: 0000:03:00.0) with AMDGPU version : gfx1031. The supported AMDGPU versions are gfx1030, gfx900, gfx906, gfx908, gfx90a."
Pytorch recognizes the GPU, but seg faults if any operation is done on the tensors stored on the GPU.
My device is 6800m (Asus G513QY laptop).

@cgmb HSA_OVERRIDE_GFX_VERSION works BTW.

I double checked the architecture targets on both the windows and linux sides of the documentation. They are accurate and need no further changes

I double checked the architecture targets on both the windows and linux sides of the documentation. They are accurate and need no further changes


Fixing pxt checkdocs

Properly check urls and snippets, fail build when there is a documentation error

This would avoid #2756 in the future?

@microsoftsam no, it does not check for missing doc pages used in pxtarget

@microsoftsam needs cherrypick in v2


color code is truncated in android colors.xml

steps:

pubspec.yaml:

flutter_native_splash:
  image: assets/images/splash.png
  color: 080582

flutter pub run flutter_native_splash:create

Actual result: res/values/colors.xml value:
 <color name="splash_color">#80582</color>
ExpectedResult:
 <color name="splash_color">#080582</color>

color: "080582"

you can maunlly change it in res/values/colors.xml file


feat(themes): rebuild component to support themes #115

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #115
Summary:
Major refactor of code to support the new theme structure
Type of change:
Please delete options that are not relevant.

[x] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

Surge demo deployment succeeded! 🚀🚀🚀
Auro Web Component Generator

The demo is not accurate.

Please delete stale branches.
https://github.com/AlaskaAirlines/auro-accordion/branches/stale

I am assuming that this issue was also addressed with this work? https://github.com/AlaskaAirlines/auro-accordion/issues/116

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


map markers do not display popups when clicked

@ilyankou - Would you please look into this issue? I happened to notice it while finishing up the image log. Thanks!

Also, one minor addition: please add our standard "view the code on GitHub" link at the bottom of the map

Added link to GitHub code.
Not sure about the popups, it works for me in both Firefox and Chrome. Could you see if there are any errors in the console?

Would you please try it in Safari desktop? No popups for me in that browser, and no errors or anything in the Console.
I'm running Safari 14.0.2, and to be honest, it's been throwing occasional glitches elsewhere, such as failure to work when clicking "browse/upload" buttons. So maybe it's me?

Looks like a bug: https://github.com/Leaflet/Leaflet/issues/7331
I disabled tap as they recommend. It now works in Safari in both desktop and iphone.


Progress

Hi , I Keep logging the progress amount but it will not go up to 100 but the files is already downloaded

Hello @Mahan3340 can you share the download link or example code?

Hey , I just deleted the line Checking for the downloadID and Status in onUpdate Listener and its just working fine (don't know if this was the problem) but just there is another quick thing that I would appreciate if you add it soon and that's somehow be able to disable the auto download when pushing new downloads to Fetch instance cause its not working even when I call pause() method just after I push the new file to the queue and it just keeps downloading the first file .
thanks .

It may be best for you to store your requests in a list. When you are ready for the download to begin, enqueue that list using fetch.enqueue(list);

Goood ! and wont this act like the normal fetch.enqueue(a single request ) ? I mean starting the download immediately

Yes. Once you enqueue the first download in the list will begin to download.

ahaa got it thank you so much


PyTorch version for reproducing the result

I want to reproduce the result of "3D Pose (H36M-SH, scratch), 39.1mm", but I only can get 40.0mm.  So I want to know what is the  PyTorch version you used to train the model?

Did you adjust batch_size and learning_rate in the code?
Using the most recent PyTorch for the installation guide seems to be recommended.

I change the batch_size from 32 to 4 and 8, because I only have one 3090 GPU with 24GB.

Hi, if you change the batch size, you may need to adjust the lr accordingly. Thank you @HoBeom for pointing out!


select knob seems backwards

Every time I use the select knob object syntax I write it incorrectly, but it also is more limiting.  IMO it makes more sense that the dropdown would show the keys and on selection pass the value.
Additionally, the current setup restricts what can be passed as the prop value to strings or numbers because object keys are more restricted.  i.e. you can't have a function or an object in the key position, but you could in the value.
I realize this would be a breaking change.

Add to this that you cannot set null as a key, either.  I went through some of our stories and found cases where people expected that they could use 'null': null inside the select.  Needless to say, it doesn't work that way.

YES. Please fix this. It makes 0 sense why it was done this way. It is completely backwards.

Oh yeah, I was just wondering why my numbers were all getting converted to strings. I was using this:
select('Size', { 4: 4, 6: 6, 9: 9 }, 4)

This definitely feels backwards. The main reason is because when you use this syntax, your keys must be strings, but the values can be any type.


[Feature] vehicles inventory

Hello @ImagicTheCat, is it possible to put directly in the nearest owned vehicle's inventory harvested items? It's hard to implement if it doesn't exists?
Thanks in advance

I'm simulating things right now about harvest, gang weed operations, etc... It's not only hard to do, it's also not very interesting. Repeating the chest "put/get" can be a boring process, but it is exactly how it is working in real life, you need to carry things to your trunk. This is labor, I will not make the process of earning money easy. But maybe in a distant future, expansive machines to automate the process could exists.

Okay, thanks for informations :). Good idea to choose expansives machines to automate the process.


ffmpeginterop vs Vlc for winrt?

Since you guyx have good experience in ffmpeg stuff, I was wondering whether you are aware of the vlc build for win rt? which is also open source? Probably you do know abt that
vlc-winrt by official vlc team it is code for the vlc uwp app on the store.
https://code.videolan.org/videolan/vlc-winrt
Vlc Media Element it is sort of a custom Control made to immitate MediaElement control of the uwp element using the libvlc as playing source of the files.
https://github.com/kakone/VLC.MediaElement
libvlc-uwp is the port of libvlc library ( open source vlc library ) for uwp
https://github.com/ZhaoYunxiong/libVLCX.UWP
So the reason I wanted to provide you guyx with all these sources about vlc stuff on uwp is because I wanted to have a discussion about ffmpeginterop on uwp vs vlc on uwp  so that we can discuss pros and cons of each. and maybe improve this fork with some features from vlc as well?
So far what I know abt these 2 is that with ffmpeginterop we can use MediaPlayerElement and hence more features of the platform control of uwp. But with VLC.MediaElement we r limited to the features which were immitated by the creator of that repo. but if we see the code of vlc-winrt-uwp the app which is already on store from vlc team, it seems to do a lot of features, including all kinds of subs and audio stream switching as well. I am not sure about which is more performant or efficient though :)
So let the discusiion begin :) @lukasf  @mcosmin222

The biggest problem with VLC and any of its code components is the license. If you use it, you have to make all your app open source, which is unsuitable for a wide range of applications (basically, you can't monetize it). And reverse engineering would take too long and not yield any benefits in the long run. If we were to integrate VLC components into ffmpeginterop, ffmpeginterop would also switch to the more restrictive licensing.
Microsoft has a good team of engineers it is just that they lack direction. VLC does have a lot of bugs, especially on mobile (it hicks up on Android, and it never really worked well for me on windows mobile).
And besides, VLC lib is a fork of FFMPEG.
FFMPEG interop can plug into any windows rendering component. We are not limited to MediaPlayerElement. We can use MediaElement as well, and we can also create custom rendering surfaces. But the moment we do that, we give up a lot, and since there is just a few of us doing it, mostly in our spare time, I do not see the benefits. We would basically have to keep up with Microsoft and recover at the same time. In my app I've been implementing lots of custom stuff in the past because Windows's equivalents were bad or didn't offer nearly the same features, but this changed in windows 10 with the rapid feature releases.

good point about the liscence, I didnt know tht so thanks for the info. so yeah we haveto keep focused and keep improving the ffmpeginterop for uwp.


Consider installer that doesn't require npm

It'd be great to get npm off of my dev system -- any chance elm-cli-tooling could have an installer that doesn't require npm?

Hi! I’ve thought a lot about that, which is why I’ve made sure that nothing relies on Node.js so everything can be re-implemented using something else. What would need to be done is:

Someone to write the tool in some other language.
Figure out to install that tool in a fast, light-weight and secure way (almost a chicken and egg problem).

Personally I won’t work on this because:

I want to focus on elm-watch and elm-safe-virtual-dom.
When I get back to this tool, I’d rather spend time on #22 and #93, and eventually #88.
I need npm anyway in my project, so having a small dependency-free npm package is hard to beat.


Worker Input Form Field Errors

from: https://daemo-test.herokuapp.com/getting-started
potential user errors: users can insert an infinite series of numbers and letters in worker setup text fields.
the text field requiring the last four of the SSN allows greater than 4 non-numeric characters.
In a situation when the user introduces either false, error filled, or foreign information, this instance of Daemo will not complete the transaction with Stripe and will keep spinning in the upper part of the window under the heading box.

yes, once you submit the form there will be an error telling you that the values are invalid.. closing


Asking about features this offers vs other Edgar tools

I'm new to Edgar filings so I'm trying to understand the difference between
edgarminers/python-edgar updated 3 months ago, 265 stars, 24 waters, 62 forks
sec-edgar-downloader updated 5 months ago, 265 stars, 11 Watchers, 75 Forks
pyedgar updated 8 months ago 27 stars, 3 Watchers, 11 Forks
Should they work together or are they similar competitors but sibling rivals ?
There are other EDGAR tools and I'm trying to understand how to evaluate them
Describe the solution you'd like
I was looking for a single solution that also worked with pandas, perhaps a scraper, that saves to a database but i'm again still researching edgar and that data impacting my models. I work with a ton of data right now at about 17 tb domestically so I would be a heavy user and would like to contribute if its a fit.
Some of them have search functions others do not
Search by stock ticker
Search by Central Index Key (CIK)
No Search by SEC CIK SEC CIK lookup tool if you cannot find an appropriate ticker
Search for 351 filings - list here
Describe alternatives you've considered
Rolled my own based off what I found online but your script would solve some issues.
Additional context
Pandas integration, database advice

sec-edgar-downloader downloads any form given the CIK or ticker
python-edgar downloads the entire index of all submissions URLs for all CIKs. Up to you to filter the CIKs you want and download the submissions that you need.
Hope that helps.


chore: trailing | undefined in types.

This was bothering me, so just a quick mechanical cleanup. In all the types we define, putting | undefined at the end instead of the beginning to be consistent.

No changeset here, no functional changes.

Weird test failure https://github.com/cloudflare/wrangler2/runs/6048122620?check_suite_focus=true

Landing this because it's safe and and it's the weekend and I don't want folks to be rebasing later.


Overlay in Dropdown menu

Bug report
Current Behavior
Dropdown Menu has default background overlay, which covers everything around Content when it is open and makes every hover and cursor disable
Expected behavior
Not covering anything around Content
Reproducible example
https://codesandbox.io/p/sandbox/radix-ui-react-dropdown-menu-dialog-bug-forked-nz4jvk
Open Context on the top right corner and move cursor around on Trigger button or other buttons, you will se there is no cursor: pointer anymore
Suggested solution
I need to turn that off, so maybe workaround or feature please
Additional context
Topbar is disable when I move cursor around, it makes my UI lose UX a lot.
@radix-ui/react-popover does not have this issue
Your environment
REACT
If anyone has the same issue or knows the solution please tell me :)

This actually is a better UX, it prevents the user from accidental click on something outside of dropdown menu or scrolling the dropdown menu open outside of the viewport.
What you are looking for is the modal prop of DropdownMenu.Root: https://www.radix-ui.com/primitives/docs/components/dropdown-menu#root

I'd also like an option to disable this overlay. My team is considering moving our old project to radix, and since we already have dropdowns that don't have this overlay, this is not acceptable for our product and design.
Also, I don't think it's obvious that current behavior a better UX either, as one would expect a visible barrier that will show the user he can't click anything ATM, like with a modal. This transparent overlay that you're not even expecting to be there when clicking is frustrating.
Other than this quirk, we're really liking this library and we'd really like to use it.


What the difference between Standard TAPIR and Online TAPIR ?

Is the former model offline track ?

Right, Standard TAPIR conduct temporal convolution on all frames (including future frames) to predict point location at every frame. While Online TAPIR replaces the temporal convolutions with similarly-shaped causal convolutions, such that the activation of each unit at time t depends only on the activations from times ≤ t.


KB832524 no longer available

https://support.microsoft.com/en-us/help/832524

Sorry, page not found
Try searching Microsoft Support to find a solution

This KB refer in https://docs.microsoft.com/en-us/sql/t-sql/database-console-commands/dbcc-traceon-trace-flags-transact-sql?view=sql-server-2017 for trace flag 1201
Please return it.

@ktaranov Hi Konstantin, We do not know why the KB832524 webpage was removed, but it was removed intentionally.  I will make one inquiry to try learning why it was removed, and I will send further @ notifications to you if I am successful.
I must now close this Issue 2385.
Thank you for submitting this Issue. We hope to hear from you again.

@WilliamAntonRohm Hi William, can you please fix/remove the broken link, and then paste the PR number/link in here.  Thanks.

Note: I sent email to Edison D. inquiring about the old KB 832524 webpage.


Hundreds of false errors in .NET SDK solution with multi-targeting projects

From @AArnott on March 7, 2017 23:17
Version Used:
VS 2017 RTW (dev15rtwsvc 26228.04)
Steps to Reproduce:
Disable NuGet package restore in the IDE to workaround https://github.com/NuGet/Home/issues/4764
git clone https://github.com/aarnott/pinvoke
cd pinvoke
git checkout c5642c29ddd
.\build -Restore -Build
src\pinvoke.sln

Open BCrypt.cs (from the BCrypt project).
Opening Hid.cs and other files also does this.
Expected Behavior:
No errors
Actual Behavior:
The error list is full of false errors.

Copied from original issue: dotnet/roslyn#17614

I think we'll probably want to take first look at this. I'll move it, and reopen if it's not us.

FWIW, I've noticed that sometimes no errors show up. But it's usually like this.

@AArnott With latest bits and all the changes from https://github.com/dotnet/project-system/issues/2297 - I'm not seeing these errors, however, I am seeing lots of design-time build errors around your CodeGeneration AppDomain that you are firing up expecting to find v14 dlls.

When I try on the latest d15rel, nuget is badly broken and I can't make any progress building pinvoke itself.
But on the latest public release (15.2, 26430.14) restore fails with https://github.com/NuGet/Home/issues/4753 which I can workaround and build succeeds.
After VS is loaded, I still see hundreds of compile errors after I open a source file. And I also notice that the TFM context switcher on the document incorrectly says $(DesktopAndPortableFrameworks), which is the macro I use in my project file, as if it was reading the unevaluated MSBuild syntax instead of the evaluated, semicolon-delimited list.

@AArnott We only added support for evaluated values in <TargetFrameworks> in 15.3. Can you try a recent 15.3 build?

Oh I see you tried that - when you say NuGet is badly broken, what do you mean?

Actually, the screenshot above was taken on 15.2 (to avoid all the nuget failures). On 15.3, I see the context switcher works properly. Thanks for the tip.
I'll have to retry on the nuget stuff to report. But generally I file those issues on nuget/home directly.

I was running into design-time build failures - if they are running in the path that we're running into figure out PackageReferences, then this will block restore.

Thanks for the tip. I'll try to find that out by adding Message tasks to my codegen targets and then running restore at the command line. Should that effectively reproduce what VS might be doing?
FWIW, I'm in 15.3 right now (26621.2 d15rel), and I'm seeing 94 errors in the Error List of my open PInvoke.sln solution (master branch). None of which are "real".

@AArnott - any current status here?

@Pilchie My last attempt at opening PInvoke with 15.5 was awesome. Everything worked better than it ever has (at least since Dev14). :)

@AArnott - I saw your mail on that, wanted to make sure it was the same project.  Going to close this for now then.  Let us know if (when) you run into future issues!


Windows 11 Won't Install

Hello:
I have been trying for a couple of days to get Windows 11 installed and is having issues. What am I doing wrong? ☹

The released version of Windows 11 will install no without issues.


Why is this needed

I am trying to understand why it is needed to "copy" the SLDS assets from the node_modules folder into the assets directory. This feels a little janky. Why can we not just reference the SLDS npm package in the lwc.config.json file here https://github.com/trailheadapps/ecars/blob/main/apps/ecars-pwa/lwc.config.json

@banderson5144 The lwc.config.json is there to specify module resolution for LWC components and third-party LWC components (Like https://www.npmjs.com/package/lightning-base-components). The third-party components should be exposed as LWC.
The SLDS bundle is not an LWC Component or Exposed as a CSS-only Module (CSS-only modules for SLDS will be overkill and cause performance bottlenecks) and hence we need to copy the SLDS assets.
Let me know if you have any other queries on this.


How to access logs in web/celery instances in our demo infrastructure setup?

Currently, /management/ returns HTTP 500, but I’m unable to access error logs. The error is not reproducible on my instance brought up using docker-compose from latest snapshot.

@kwkwan can you help @strogonoff find a way to access the logs?

I can setup a manual github workflow action to get logs of the web and celery pods.

@kwkwan that's clever! Thanks!


Enabled e2e test

Lets trigger another Github actions build and make sure it getting passed

Conflicts pr which is very old. Hence closing


docs(networking-manager): added NetworkingManager documentation

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


ngbDate invalid in Dynamic Form Date Control

Hello,
I get an invalid error in Date field control.
Original source code.
Thanks for your help.

Hello,
I get an invalid error in Date field control.
Original source code.
Thanks for your help.

I fix this problem by updating ng-bootstrap to version 5.

strange..I haven't seen this issue with 4.x release of ng-bootstrap.

i have this problem too

packages have been updated to recent versions. Please let me know if such issue still exists. Thanks


Public/Private Dependency issues

Hey all,
I am back with a bug, and I have already checked that I am on the latest version of Pipenv ;)
The Problem
We have a couple of private dependencies and plenty of public dependencies.
When trying to install with pipenv install with no package, we always fail to install in a clean environment.
The issue is that no matter which source is used, the underlying pip install -r <requirements> command fails. Either the public dependencies don't exist in the private repo, or the private dependencies don't exist in the public repo, causing either command to fail.
I have found the part of the code that causes this. In the pip_install function in the cli module, the sources are tried until they succeed, and if all sources fail, only the last is reported. For awhile, it just looked like my installation couldn't find Flask and gunicorn, an error that I thought was strange, but could deal with by scripting in anger.
A Solution
One way of fixing this might be to store the source used for a package in the lockfile (or somewhere) and during install create separate requirements.txt files for each source to install from.
If that isn't possible/simple or there is an obviously better way to do this, I would be happy to help/write code for this.
A Note
I would also prefer it if every unsuccessful command could be logged, preferably in the format of command-followed-by-error. While it might be more information than others want, I might have figured out this bug weeks ago if I had seen one more error message, rather than just the last one. (This may be part of a PR, if I happen to be the one who writes it)
– Isaac

Hey @isaacsanders, thanks for opening this. Could you clarify a few things for me really quickly. Are you installing from a Pipfile or requirements.txt? Could you provide the contents of that file, and if you need to remove any sensitive data just make sure the structure of the requirement is preserved?
I'm not entirely sure how this would fail to find those packages since it should check every source before throwing an error. I'm particularly interested in the values of your sources again this time.

I am installing from a Pipfile, but pipenv appears to internally generate a requirements.txt tempfile.

@isaacsanders, it does create one, installs from it, and then deletes the file. Could you provide the contents of the Pipfile?

It does check every source, but either repo doesn't mirror every package in the requirements.txt, so it fails for both, each for missing a package that the other source has.

This is in the pattern of our Pipfile
[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true

[[source]]
url = "https://packages.example.com/pypi/simple"
verify_ssl = true

[packages]
requests = "*"
private-package = "*"

There are two distinct sets, and instead of unioning the sources, pipenv only succeeds on installing when the intersection satisfies the requirements.txt.

@isaacsanders, thanks for the clarification, I see what you're saying now. So we need to eliminate what's been installed from one source before moving onto the next, or create a way to detect what's already been installed. I'm open to reviewing PRs for this or I'll try to take a look at this as soon as I can.

I have a fix. pip has a --extra-index-url option. Instead of looping over the sources, I will just create a list of these --extra-index-url options and append them to the list of options.

@nateprewitt @kennethreitz There is a question however.
If the package is in two repositories, which one will it install?

@isaacsanders, we currently default to the order they are written into the Pipfile. You would need to place your internal source before the external to achieve your desired behaviour.
In the event you have mixed requirements (multiple packages in both repos with different precedence), we don't have a good solution for that. We could store a package by package requirement, or an extra variable for an inline table which allows optional explicit definitions for sources. I'm not sure if we're at a point to entertain that yet though. @kennethreitz, any thoughts here?

we can't support every use case.


Iam policies - Add Conditions

Added support for IAM policy conditions

This looks good to me. I merged in your other PRs and now this one has conflicts. Care to rebase?


dev: add more test coverage for the incremental merkle tree

The actual merkle tree hook contract
(cf: https://github.com/astraly-labs/hyperlane_starknet/blob/main/contracts/src/contracts/hooks/merkle_tree_hook.cairo) lacks testing for the internal functions. The idea is to try every single function to avoid unexpected outputs.

related issue #37


Bug: not all unicode characters are shown (Hebrew in my case)

V<IP_ADDRESS>
I try to switch to Hebrew using ALT+SHIFT, and then I type in it, but I don't see the text from Carnac on the screen showing Hebrew. It shows English instead.
For example, I type "ד" and I see "s" instead (same key, but different character should be written as I'm on Hebrew language and that's what was typed.
I think most apps fail to do this correctly. But I've found one that does:
https://keypressosd.com/

I was looking for a similar issue. Currently, on an azerty keyboard, if I type "déjà", it shows as 3d2j03.
Actually, the top row seems to be treated like a qwerty keyboard, or something.
Mmm, not only that. It is OK if I type a or q, but <> show as |, and so on. All non-alphabetic characters doesn't follow azerty keyboard layout.
Is it something that can be adjusted somewhere?

I think I got some fork of Carnec that allows to show Hebrew characters too, but it has various issues, including showing as if I write in Hebrew even though I write in English, and writing the order of letters in the opposite direction (Hebrew is RTL language, so the text seems in the opposite direction).
I don't remember where I got it, but here are the files:
carnac.zip

I think I got some fork of Carnec that allows to show Hebrew characters too, but it has various issues, including showing as if I write in Hebrew even though I write in English, and writing the order of letters in the opposite direction (Hebrew is RTL language, so the text seems in the opposite direction).
I don't remember where I got it, but here are the files:
carnac.zip

This is solved my problem too.

I should have put a link to it. No idea where I got it from :(

@mugiseyebrows How come the app isn't updated anymore? There are plenty of forks out there, waiting to be merged...

@AndroidDeveloperLB seems like author left his project for some reason, last commit was 4 years ago, no PR will be merged ever, probably.
Mine fork works correctly when you have multiple keyboard layouts (multiple languages) in your system.


Set up mutations and JWT logic

As a user, I can create my own account and login

As a logged in user, I can create thoughts, react to thoughts, and add friends

Users can create an account, login and create thoughts, reactions and add friends, closing issue -KF.


Added pagination style of pager for twitter bootstrap

See http://twitter.github.com/bootstrap/#tables

Woops, compile issue (dumb) :) Fixing


Reduce duplicated effort for module+global/script typings

Currently when we create typings that can be consumed as module AND global/script, we need to write it twice.
e.g.:
https://github.com/typed-contrib/knockout/blob/master/index.d.ts
https://github.com/typed-contrib/knockout/blob/master/global/index.d.ts
and
https://github.com/typed-typings/typed-iscroll/blob/master/iscroll.d.ts
https://github.com/typed-typings/typed-iscroll/blob/master/iscroll/iscroll.d.ts
This is due to TypeScript does not allow this:
declare namespace abc {
  import main = require('./abcModule');
  export = main;

or similar construct.
Is there an issue opened on https://github.com/Microsoft/TypeScript can be referenced and tracked here?

Doesn't TypeScript 2.0 UMD support fix this specific issue?

https://github.com/Microsoft/TypeScript/issues/7125
Yeah, just need to refresh my memory and connecting the dots.
Can't wait for it to come out.

Workaround by @basarat
https://github.com/Microsoft/TypeScript/issues/7352#issuecomment-191547232

@unional, thanks for the link to the workaround by @basarat.  I'm afraid I don't quite follow how this approach could be applied to a type definition.  If it's not too much trouble, would you be able to illustrate how this could be applied in the case of say iScroll or Knockout?  I'd really like to understand this better.

I need to build an example to test the statement that I think it might not work.
The workaround is exposing them through two files, global.d.ts and globalAugment.d.ts and add them to typings.json/files

@johnnyreilly
Modules stay modules
Basically if a file is declared as a module, there is no way to move it into the global namespace without thinking too hard:
declare module "foo" {
    export var foo: any;
}

Any attempt to use this foo module would need an import:
import * as foo from "foo";

That would essentially make the file a module and therefore cannot pollute the global namespace.
Hack to move it into the global
The one linked here : https://github.com/Microsoft/TypeScript/issues/7352#issuecomment-191547232 works by pollute the global namespace seperately:
interface My {    // To introduce the name `My` to the global namespace
}

And then augment it from the module:
import foo = require('foo');
declare global {
    interface My {
        user: typeof foo;
    }
}

etc.
Note this is a very bad workaround for decreasing code duplication as we need two files (global.d.ts and my.d.ts) to put a module into the global declaration space.
Long term fix
UMD module declarations : https://github.com/Microsoft/TypeScript/issues/7125 now allow you to create a declaration file that is a module:
export function doThing(): string;
export function doTheOtherThing(): void;

export as namespace myLib;

That can be imported as a module:
import * as foo from "./foo";

or can be imported globally:
/// <reference path="my-lib.d.ts" />

myLib.doThing();

So we actually don't need any duplication effort on our part. The effort moves into the hands of the library consumer (either global or as a module). Hope that helps :rose:

@Basarat, do you think that if it is being referenced multiple times, would it cause duplicate identifier error?  That's what I want to test about. :rose:

Thanks @basarat - I need to let this sink in a bit but I think I follow you.
So it's not really worth advising people to use the hack; just wait for UMD to ship.  (Which looking at the talks from Build is probably only about 2 months or so) :tulip:

do you think that if it is being referenced multiple times, would it cause duplicate identifier error

@unional are you asking about umd or hack?

Hack.

do you think that if it is being referenced multiple times, would it cause duplicate identifier error

@unional Not if its referenced multiple times pointing to the same file. However if its across different files its effectively the following and will result in an error:
declare global {
	interface Array<T> {
		foo:number; // Duplicate ident error
	}
}

declare global {
	interface Array<T> {
		foo:number; // Duplicate ident error
	}
}

Great. If that is the case, then it might be possible that we can start doing it today in typings. Since they are ambient, the dependencies aren't followed not inlined.
Therefore it should (crossing fingers) not cause problem at the end consumer.
Thanks for your great hack! :rose:

...nor inlined....
Phone keyboard doesn't have "nor " in the dictionary........

Just tried it. Having a bit of issue. ping @blakeembrey .
I've upload an example:
https://github.com/typed-typings/typed-iscroll/tree/master/global/iscroll
The problem I have is when I run typings bundle -o out --ambient on it, it fail with:
typings ERR! message Attempted to compile "iscroll" as an ambient module, but it looks like an external module.

typings ERR! cwd /Users/hwong/github/typed-typings/typed-iscroll/global/iscroll/out
typings ERR! system Darwin 14.5.0
typings ERR! command "/usr/local/bin/node" "/usr/local/bin/typings" "bundle" "-o" "out" "--ambient"
typings ERR! node -v v4.3.2
typings ERR! typings -v 0.7.12

typings ERR! If you need help, you may report this error at:
typings ERR!   <https://github.com/typings/typings/issues>

The message is right that indeed iscrollAugment.d.ts is an external module.....but.....

@unional What's the expected behaviour?

@blakeembrey how to make it work? :stuck_out_tongue:
When used, it needs to be in two separate files. Maybe index.d.ts and indexAugment.d.ts?
// main.d.ts
/// <reference path="main/ambient/iscroll/index.d.ts" />
/// <reference path="main/ambient/iscroll/indexAugment.d.ts" />

But on the other hand, it may not worthwhile because it could be too complicate to implement for a hack that will go away when 2.0 lands.

Don't you just bundle it without --ambient? It doesn't matter that it gets wrapped up, declare global is still good.

Without --ambient you get another error because iscroll.d.ts is ambient:
typings bundle -o out
typings ERR! message Unable to compile "iscroll", the typings are meant to be installed as ambient but attempted to be compiled as an external module

typings ERR! cwd /Users/hwong/github/typed-typings/typed-iscroll/global/iscroll
typings ERR! system Darwin 14.5.0
typings ERR! command "/usr/local/bin/node" "/usr/local/bin/typings" "bundle" "-o" "out"
typings ERR! node -v v4.3.2
typings ERR! typings -v 0.7.12

typings ERR! If you need help, you may report this error at:
typings ERR!   <https://github.com/typings/typings/issues>

Bundle the definition itself?

Bundle the definition itself?

What do you mean?
// typings.json
{
  "name": "iscroll",
  "main": "iscroll.d.ts",
  "ambient": true,
  "files": [
    "iscroll.d.ts",
    "iscrollAugment.d.ts"
  ],
  "version": "5.1.3",
  "homepage": "https://github.com/cubiq/iscroll",
  "devDependencies": {
    "blue-tape": "registry:npm/blue-tape#0.1.0+20160322235613"
  },
  "ambientDevDependencies": {
    "node": "registry:dt/node#4.0.0+20160319033040"
  }
}

I don't know what to say, can you provide a more isolated example? Looking at https://github.com/typed-typings/typed-iscroll/blob/master/global/iscroll/typings.json#L6-L7 you are trying to merge an ambient and external version which won't work. Why don't you make it only external versions?

Create an issue on typed-iscroll: https://github.com/typed-typings/typed-iscroll/issues/4

Maybe it's worth just waiting for 2.0? If it's only a couple of months away then this might be more trouble than it's worth. Also you might get people in the future looking at Typings with the hack in place and then thinking the hack is good practice.

Also you might get people in the future looking at Typings with the hack in place and then thinking the hack is good practice

:100: I am sure this wasn't intended as a part of declare global augmentation and we just found a way to abuse it. But then again I'm not the one feeling the pain of writing these duplicate definitions so whatever you chose :rose: :bow:

:smile:  maybe we can change topic on how to get typings ready for UMD. :wink:
We still want to have a separate file for UMD. But now it will be in top-level import/export format. So we need to detect it is UMD and make it an exception to the current validation.

FWIW the hack is simpler now as you no longer need a separate file for introducing the name into the global namespace: https://github.com/Microsoft/TypeScript/issues/7161#issuecomment-210743869 :rose: I still can't make the choice of what is the right way to do it though, and I leave that to your brilliant hands :rose:

Just think of an idea as I'm looking into fixing some issue on env-atom:

Create typings in module format, as in the source code.
Use typings bundle to create global typings
Use typings.json/files to add those bundled global typings.

@blakeembrey what do you think?

@unional That could work. I think if it works it's good. You may want to tweak the bundle command to omit dependencies just in case, but seems reasonable to do it this way and end up with just one type of bundle.


[WIP] Add priority and fairness

Started drafting doc for https://github.com/kubernetes/enhancements/issues/1040

Hey, what's going on here?  I am only asking to merge one commit, but this PR somehow has 33!

Oh, I see I goofed up on the commit to start from.  Fixed now.

I spotted some typos that I hope aren't controversial, and I also commented on the structure of the page. I hope this feedback is useful.

/remove-language es

/remove-language fr

/remove-language id

/remove-language zh

/cc @yue9944882 @lavalamp

Page preview:
https://5d579d7acb65dd0009e4cc90--kubernetes-io-vnext-staging.netlify.com/docs/concepts/cluster-administration/priority-and-fairness/

/remove-area blog

Thanks for the help.  I made the easy revisions.

/cc @deads2k @lavalamp @yliaog @aaron-prindle @adohe

Because there's clearly some TODO-type content in this PR, I will
/retitle [WIP] Add priority and fairness
and I hope that's OK.

/milestone clear

Why close this instead of put it in the next release?

/reopen

@MikeSpreitzer v1.16 is now released
What would you like to do with these changes? Perhaps target the release for v1.17?
/lifecycle stale

@MikeSpreitzer Please feel free to /reopen this when you're ready to update the base branch.
/close


gist:links should be symmetric

since gist:links property is defined as

Relates a NetworkLink to a NetworkNode that it connects to another node. Used when the connections are undirected, or the direction is not known.

shouldn't it be declared as symmetric, i.e. an owl:SymmetricProperty?

No. The subject is a NetworkLink, the object is a NetworkNode. The model looks as follows:

links is used when the link is undirected, or direction is unknown. linksFrom and linksTo are used to indirect direction of the link.


Please change the name

Creating an app for the "average person" and calling it felony does not do the greater cryptographic community any favors in the PR battle we have with the NSA, FBI et al that argue encryption allows for illegality.

Thought what someone wrote on HN was appropriate:
A new app called Felony allows ISIS and online pedophiles to communicate in secret with ease.

Hmm @henryboldi just closed this serious issue without even so much as a comment. That doesn't communicate that this is a project that really cares about promoting the use of encryption.

 Please change the name to something that won't drive potential users away.

Ok guys, let's put down the pitchforks, I think he got the message.

+1 for changing this name, this is bad marketing (though I personally think it's pretty funny). I think a good deal of potential users will be turned off of using this for fear of oppression etc.

@henryboldi

Change the name. It's all fun and games, until brute governments with zero patience arrest someone, and politically charged headlines make the legitimate use of encryption harder.
This is plainly irresponsible, and unwelcome in the broader cryptography and infosec communities.
Closing the issue without even discussing sets a pretty negative collaboration tone, too. You have great momentum in just one day, don't squander it by alienating possible supporters and contributors.
You have a well thought out, well designed application here. Don't throw that away for the sake of a joke.
As menzoic on HN suggested, what about "Freedom"? It sends the right message.


[Bug]: widgets aren't loading in edit mode for certain apps

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I open some app in edit mode, the widgets don't load in.
Refer to these recordings for reference:

https://www.loom.com/share/4a6d71be64804f1aa25038da1180ce64
https://www.loom.com/share/44b1a162abe944d59511c8210d6ff0d3
For some users it was solved by going to preview mode, then back to edit mode, for others it is solved by opening a new page where the widgets load in, then coming back. Force refreshing and closing and reopening does not solve this
Steps To Reproduce
This issue isn't consistent or reproducible. it happens often with certain apps with certain users, but when another user opens the same app, it works fine
Public Sample App
No response
Version
v1.7.1 Cloud

Closing due to duplicate issue


open pr guide

Most beginners have difficulty in opening a PR, a page like https://sprig.hackclub.com/get would be perfect

So you would like a md page explaining how to fork the project, create branch, make changes, create pull request?

@LucasHT22 what if we just added a link to this github page first contionutions?


[DCOS-50376]

[DCOS-50376]
What does this PR aim to do?

Bump base tech version for HDFS to cdh-6.1.1

How was this PR tested?

By running a CI build

Closing this PR.


Issue 218 aws irsa basic auth provider

Description
Added the AwsEcrBasic auth provider that retrieves AWS ECR auth credentials, using the AWS SDK for Go v2. Ratify pod must be enabled with IAM Roles for Service Accounts (IRSA). Includes documentation about setup and verification.
Additional information about the IRSA and AWS SDK for Go v2 approach can be had from this post.
Modified httpserver/Dockerfile to accept new ARG to enable GOPROXY ENV config.
Fixes #218
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Local build:
docker build -f ./httpserver/Dockerfile -t ratify:test --no-cache=true --build-arg GOPROXY=direct .

Ratify helm install from file system:
helm install ratify --atomic --namespace ratify ratify/charts/ratify/ \
--values values.yaml

ratify-configuration configmap included correct auth provider selection:
      "stores": {
        "version": "1.0.0",
        "plugins": [
            {
                "name": "oras"
                ,
                "auth-provider": {
                    "name": "aws-ecr-basic"
                }
            }
        ]
      },

ratify pod log output was:
time="2022-06-22T00:17:34Z" level=info msg="configuration successfully loaded."
time="2022-06-22T00:17:34Z" level=info msg="selected auth provider: aws-ecr-basic"
time="2022-06-22T00:17:34Z" level=info msg="stores successfully created. number of stores 1"
time="2022-06-22T00:17:34Z" level=info msg="1 notary verification certificates loaded from path '/usr/local/ratify-certs'"
time="2022-06-22T00:17:34Z" level=info msg="verifiers successfully created. number of verifiers 1"
time="2022-06-22T00:17:34Z" level=info msg="selected policy provider: configPolicy"
time="2022-06-22T00:17:34Z" level=info msg="policies successfully created."
time="2022-06-22T00:17:34Z" level=info msg="starting server at:6001"

Checklist:

[X] Does the affected code have corresponding tests?
[X] Are the changes documented, not just with inline documentation, but also with conceptual documentation such as an overview of a new feature, or task-based documentation like a tutorial? Consider if this change should be announced on your project blog.
[ ] Does this introduce breaking changes that would require an announcement or bumping the major version?
[X] Do all new files have appropriate license header?

@susanshi @akashsinghal @etrexel Based on last night's meeting, I updated snake case auth provider values to camelCase.

@jimmyraywv thanks for taking the time to update some of the other config entries to camel case. We opened up a separate PR #235 to address this case issue and a few other config issues in the docs and code. Would you mind reverting the casing changes apart from those AWS specific?
@susanshi and I think the PR is ready to merge. Could you fix the merge conflicts?

@susanshi this is the same issue we were seeing this week. Even when running go mod tidy locally, there's some dependencies we have that are throwing errors since the packages have been renamed. I tried to checkout @jimmyraywv's PR but the only thing that worked for me was to delete the go.sum and rebuild the file.

created issue https://github.com/deislabs/ratify/issues/245 to track the fix for go.mod file

@jimmyraywv Thanks for being patient with us all these package issues! We've finally resolved them. I've created a PR to your fork updating your branch. If you could take it a look and try the ECR/EKS workflow to make sure everything is still good, and then merge it into your branch. Then we'll go ahead and merge your branch into main and push the release. https://github.com/jimmyraywv/ratify/pull/2

I merged the PR to my PR branch. My local build was successful.


KAFKA-9652: Fix throttle metric in RequestChannel and request log due to KIP-219

After KIP-219, responses are sent immediately and we rely on a combination
of clients and muting of the channel to throttle. The result of this is that
we need to track apiThrottleTimeMs as an explicit value instead of
inferring it. On the other hand,  we no longer need
apiRemoteCompleteTimeNanos.
Extend BaseQuotaTest to verify that throttle time in the request channel
metrics are being set. Given the nature of the throttling numbers, the test
is not particularly precise.
I included a few clean-ups:

Pass KafkaMetric to QuotaViolationException so that the caller doesn't
have to retrieve it from the metrics registry.
Inline Supplier in SocketServer (use SAM).
Reduce redundant time.milliseconds and time.nanosecondscalls.
Use monotonic clock in ThrottledChannel and simplify compareTo method.
Simplify TimerTaskList.compareTo.
Consolidate the number of places where we update apiLocalCompleteTimeNanos
and responseCompleteTimeNanos.
Added toString to ByteBufferSendandMultiRecordsSend`.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

2 jobs passed, 1 unrelated flaky test failed:

org.apache.kafka.streams.integration.QueryableStateIntegrationTest.shouldAllowConcurrentAccesses


Input multiple color hex?

The material theme builder tool allows you to specify the HEX color for primary, secondary, tertiary, error and extended colors. Looking at the material color utilities library I didn't see anyway to specify these when generating the theme.
Does this library support inputting these multiple colors to generate the theme instead of it automatically picking secondary and tertiary?

Seems like this should be possible using:
const colors = CorePalette.fromColors({
  primary: argbFromHex("#B4D1FC"),
  secondary: argbFromHex("#987BFF"),
  tertiary: argbFromHex("#2692FF"),
  error: argbFromHex("#ED4545"),
  neutral:  argbFromHex("#D1D1D6"),
  neutralVariant:  argbFromHex("#D1D1D6"),
});

Theres no theme utility for using multiple colors, but I think it could be easily added. The built in forColors does not generate missing colors, so both palettes would need to be generated and combined.
export function themeFromSourceColors(
    source: CorePaletteColors, customColors: CustomColor[] = []): Theme {

  // Generate palette based on custom input colors
  const customPalette = CorePalette.fromColors({ ... });

  // Generate palette based on primary color
  const autoPalette = CorePalette.of(source.primary);

  // Combine into one palette
  const palette = { ...autoPalette, ...customPalette };

  return {
    source,
    schemes: {
      light: Scheme.light(source),
      dark: Scheme.dark(source),
    },
    palettes: {
      primary: palette.a1,
      secondary: palette.a2,
      tertiary: palette.a3,
      neutral: palette.n1,
      neutralVariant: palette.n2,
      error: palette.error,
    },
    customColors: customColors.map((c) => customColor(source, c)),
  };
}

None of this is tested, just from what I gathered looking through the source code.

Seems nice to have somthing like that @joezappie


Make ssh plugin work with disabled native OpenSSH client and no admin rights.

Make the ssh plugin work with an installed but disabled native OpenSSH client.

Add a configuration option to not try the native OpenSSH client
start the ssh-agent.exe and capture the environment variables through an agent.env file
save the SSH_AUTH_SOCK and SSH_AGENT_PID to the user environment so they can be picked up by other processes. This is required for using the git client in visual studio.
update the Readme

Looks cool but I don't have time to test. Does this break anything?!

Not intentionally. I have picked the defaults to maintain current behavior.

What if... could this functionality be moved to a separate plugin, say plugins/rogue-ssh.ps1 instead of mixed in with the default plugins/ssh.ps1? So people could explicitly opt in to this if they find themselves in this situation. We could have a separate wiki page about how it works (linked from the README). No need to change the default.json theme either that way.
I'm just trying to figure out how we can get this in there but minimise potential disruption. Looks like a very handy workaround for people that need it.

There would be quite a bit of code duplication in a separate plug-in. But I am not against it.
The change in default.json not strictly needed. I tested that the plugin behaves that same with and without the settings in default.json. People with a different theme should not notice anything.
I struggled finding out where to configure the plugin, therefore I added it as an example.
I can revise the pull request.

I created a new pull request. #119


Pydantic to 2.6

Bumped a lot of versions. Still sorting out langchain_community, langchain_core.

📚 Documentation preview 📚: https://swarms--385.org.readthedocs.build/en/385/

Added black reformatting


refactor: added setStatusBar TS declaration

Description

This pull request addresses the issue described here: https://github.com/wix/Detox/issues/4191

In this pull request, I have added TS declaration for missing method.

For features/enhancements:

[x] I have added/updated the relevant references in the documentation files.

For API changes:

[x] I have made the necessary changes in the types index file.

Thanks for review @asafkorem !
I'll address all your comments tomorrow 👍
By the way it seems that setStatusBar is not working on modern iOS simulators (iOS 16.4+). Are you aware about that? It seems like it's a problem with simctl itself and it's not a Detox problem, but just want to ask whether you are aware about this problem or not?

Hi @kirillzyusko, thank you, I wasn't aware of that. I just found some issues online, and indeed, it seems that there is a bug with Simctl 🙁

I just found some issues online, and indeed, it seems that there is a bug with Simctl 🙁

Got it! I've fixed all the problems you mentioned - would you mind to review PR again? 👀

@kirillzyusko see my review.

@noomorph I'm not sure I understood you correctly, so I'll ask more questions before making any changes, if you don't mind :)
Do you want me too add device.setStatusBar() call in beforeAll? Or do you want me to write a separate test with assertType function?

I'd suggest just adding two calls to device.setStatusBar in beforeAll

To be on the same page - those tests are not actually executed, they are just to check the typings

@noomorph I fixed your comment. Would you mind to review it again, please?

Thanks @kirillzyusko !!!


feat(cli): reload config on changes, closes #115

@antfu Does that look good to you? Am still a little unsure to reluctant directly to main, thus the PR. 🙂

Looks great, thanks. It's better to send PRs so I can have a quick review of them.


drawMesh and others should not modify glBlendFunc().

I was trying to erase the drawn shapes in an FBO and found out I could not as drawMesh() would reset my setting of:
    glBlendFunc( GL_ZERO, GL_ONE_MINUS_SRC_ALPHA );

to:
  glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);

Created my own drawMesh function to work around it.  Thanks.


Tab updates

Fix tab indicator rendering when the phone's orientation changes.
Fix TabBarView scrolling when the selected tab changes during a selection-change animation.

Fixes: https://github.com/flutter/flutter/issues/2977
Fixes: https://github.com/flutter/flutter/issues/1807
Fixes: https://github.com/flutter/flutter/issues/1955
[N.B. 2977 and 1807 are duplicates, 1955 had already been partially fixed, thanks @mpcomplete]

LGTM


MacOS (x86) binary not in PATH after restart

VS Code Version:
OS Version:

Steps to Reproduce:

Run the command "Install code in PATH" from the command palette.
Check the terminal, it should find the 'code' binary.
Restart the computer and check again - "command not found: code"

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

It looks like running that command palette operation mentioned above adds a symlink to the /usr/local/bin directory, which is what I would expect it to do. However, the target of the symlink is a bit odd. I just checked and I see the following in /usr/local/bin:
code -> /private/var/folders/mt/fmkgwzfj1kvg8s105b7znrfh0000gn/T/AppTranslocation/A37EC2BC-124B-4A46-A943-464286617262/d/Visual Studio Code.app/Contents/Resources/app/bin/code
Once rebooting, it looks like the symlink target is no longer a valid path. In my case, when I rebooted, the path segment after AppTranslocation changed to a different UUID, but the rest of the PATH looked correct.
A workaround for now was for me to manually add the following entry to my PATH env var: /Applications/Visual\ Studio\ Code.app/Contents/Resources/app/bin which of course persists over a reboot, as it is a direct link to where the application lives.
OS Platform: MacOS 11.2.3 (x86 architecture)
VS Code version: 1.55.0 (Universal)
Feel free to contact me for more information/details<EMAIL_ADDRESS>

macOS seems to have put VS Code into quarantine in your system. Here's how you can solve it: https://github.com/microsoft/vscode/issues/7426#issuecomment-425093469


Initial cohost server implementation

First part of https://github.com/dotnet/razor/issues/9519
Razor side of https://github.com/dotnet/roslyn/pull/70819
This PR consumes the new Cohost server from Roslyn, and when enabled, moves DocumentColor requests from being handled by our existing Razor Language Server, to instead be handled by the new server. There is currently no sharing of project system data or anything like that yet. A few TODOs for follow ups, ~can't be merged, and indeed won't build, until a Roslyn package is available etc. but~ it's reviewable while we keep working, and more important its shippable in its "off" state.
~Ignore the last commit, obviously won't be merged :)~

Integration test run passed: https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=8830201


[BUG] how to solve a multi step problem by langchain4j tools? Why my response seems can use just one tool instead of two ?

As we can see, the final answer is not given. why the second tool was't used?
Thanks so much for help!!!!

After trying different model， I seems to know the reason.
The process works just fine if I use gpt-4.
This kind of problem occures when I use GLM4-flash.
So I wonder if this problem is caused by model ability ?
Or I use the langchain4j in a wrong way? Thanks

Hi @flatLying

So I wonder if this problem is caused by model ability ?

Seems so, different LLMs cave different abilities with tools.
I would probably merge these 2 tools together to have a "get file content vby file id" tool. This should work muchmroe reliable and avoid having an extra round trip to the LLM


Add network_access_control as an optional configuration to the module to create workspace with restricted access

Is your request related to a new offering from AWS?

No

Is this functionality available in the AWS provider for Terraform? See CHANGELOG.md, too.

Yes ✅: please list the AWS provider version which introduced this functionality
aws provider 4.58.0. https://registry.terraform.io/providers/hashicorp/aws/4.58.0/docs/resources/grafana_workspace

Is your request related to a problem? Please describe.

No, its not a problem , its just that currently the module will always create a workspace with open public access , and I have to manuallly assign the vpc endpoint I have created to it.

Describe the solution you'd like.

I saw that the resource grafana_workspace can optionaly accept network_access_control and I was wondering if this can be requested as a feature request to the managed_grafana_module to accept it optionally so that I can initialize the workspace with restricted access instead of manually changing it.
Describe alternatives you've considered.

I have been tried only one alternative solution and that is to assigning a VPC endpoint manually to the grafana workspace after I create it with terraform with open access.
Additional context

I have been playing around with the module and I came to the use case where I want to create the workspace with a vpc endpoint so that upon creation it is in a private subnet behind a VPN

This issue has been resolved in version 2.0.0 :tada:


Shift to APP DIR in NextJS 13 with optimizing fetching

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 4 committers have signed the CLA.:white_check_mark: Palanikannan1437:x: Chiranjeev-droid:x: anjy7:x: KapadiaNaitikYou have signed the CLA already but the status is still pending? Let us recheck it.


Added a source code analyzing GitHub repository in Chinese

Hi,
I'm so happy to make my first contribution to the open source community. For my first contribution, I added an online GitHub repository for Android source analysis in Chinese with Pull Request #7429. I would appreciate that if you could help me to review and merge it, which I would think may help other people in this community. I am looking forward to building this community together to make it more robust.

Resolved by #7429


All these tables look the same more or less

I get that you can't just use DataGrid because of all the custom controls but surely you could write custom shared components for tables so that you aren't repeating the same structure over and over

An attempt was made and it was determined that the custom tables are a little too custom for their functionality to be abstracted to a shared component. The amount of things that would need to be passed would mean that the restrictiveness would substantially outweigh the reduction in line count

Not to mention all the sacrifices in type safety


TestG1OldAllocationPendingStackTrace.java run timeout because of can't trigger GC on some machine

Describe the bug
TestG1OldAllocationPendingStackTrace.java run timeout because of can't trigger GC
To Reproduce
jtreg -va -nr -w tmp-temurin jdk/test/jdk/jfr/event/gc/stacktrace/TestG1OldAllocationPendingStackTrace.java &> temurin.log
Expected behavior
run pass on every machine.
Screenshots
Additional context
When I change -Xmx128M to -Xmx30M, this testcase run success.

temurin-manual-Xmx30M.log
temurin-manual-Xmx128M.log.zip
temurin.log
I have two machine run this testcase fail.

alios:

ubuntu 20.04

I tried reproducing this problem on x86_64 linux with JDK8, but was unable to after 5 iterations: https://ci.adoptium.net/view/Test_grinder/job/Grinder/8839/testReport/jdk_jfr_event_gc_stacktrace_TestG1OldAllocationPendingStackTrace/java/
I'm pretty confident this is a limitation with the JFR test, not the actual JDK/hotspot code.  The test makes a best effort to trigger a GC by repeatedly doing allocations. A GC may be triggered by causes other than the test itself (this is undesired). If after 5 attempts, a GC due to the test allocations hasn't happened, the test fails).
https://github.com/openjdk/jdk8u/blob/master/jdk/test/jdk/jfr/event/gc/stacktrace/AllocationStackTrace.java#L389
I'm not sure what the best course of action is here. We could try to increase test allocation size or decrease heap size to increase the chances that a GC is caused by specific test code. I don't know how easy it would be to  upstream this though considering I wasn't able to reproduce it.


fix: console error with ghostLoadingIndicator

What kind of change does this PR introduce? (check one with "x")

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
Throws console error when rows set to undefined and ghostLoading is enabled
What is the new behavior?
Removes console error.
Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

@timowolf Could you please review this PR.
Thanks,
Chintan


Apps not included Packing a solution containing apps

Hi Wael,
There seems to be an issue with this. When we use the solution packager directly this doesn't happen

@JezNorton I presume you are comparing this to the task? Are you using the same version of the solution packager in both?

@WaelHamze  Hi, yes when packed manually using the solutionpackager it's fine. I selected SDK version 9 on the task. I did think perhaps it's using an outdated version of the packager, IDK.

@JezNorton when the task runs it will output the version of the solution packager to the console. Did you compare that with version that works fine locally?

@WaelHamze Hi, I'm using assembly file version <IP_ADDRESS> locally, the task is using <IP_ADDRESS>


Abandoned?

Should this be used or is it considered and abandoned project?

I would say abandoned, unfortunately -- 3 years without commit to the master branch. I tried to take this project over (see #4) but never heard back from the maintainer about some maintenance questions i had. Luckily, the Tauri ecosystem should eventually reach and surpass feature parity with Capacitor thanks to the Tauri team's work towards mobile (see e.g. https://tauri.app/blog/2023/03/20/tauri-2-0-0-alpha-4/#native-mobile-functionality-for-tauri-plugins)


Install Digital Certificate

https://attack.mitre.org/techniques/T1608/003

https://www.fireeye.com/blog/threat-research/2012/08/hikit-rootkit-advanced-persistent-attack-techniques-part-2.html

certmgr.exe

I think this technique is focused on the attacker infrastructure vs. infected hosts.

Installation of digital certificates may take place for a number of server types, including web servers and email servers.

Willi's reference and features are a good fit for https://attack.mitre.org/techniques/T1553/004/

Closing this and tracking now in #373.


Make the base selector configurable

Right now, the base selector is hardcoded -- .form-checkbox. I would like to style the checkbox element globally, since I'm working with a platform that's extensible and extensions aren't written with this plugin in mind.
It might be possible to achieve this by using input[type=checkbox] { @extend .form-checkbox }, but I think having a configurable base selector is a more elegant solution.

Have you figured out a way to be able to have the custom forms styles applied globally?
I'm new to tailwind and would like to have form inputs styled globally across my site, but not really sure the easiest way to accomplish that.


Adding support for Milestone Events

Since GitLab 13.1 there is the Resource milestone events API, which is currently missing in this package: https://docs.gitlab.com/ee/api/resource_milestone_events.html
It's primarily used to keep track of which milestone is associated with issues and merge requests.

Fixed by PR #1474


Cypress integration test block focusing issue

https://gist.github.com/sneridagh/75212d81d8bbb28a49540bb5c69f2574

https://gist.github.com/sneridagh/75212d81d8bbb28a49540bb5c69f2574

@tiberiuichim @sneridagh See https://github.com/eea/volto-slate/commit/98e7617ce3e3e7bfc866f9a9044ea28e20fbccd9
Not sure if it was supposed to fail but I adapted it to pass :laughing:


x/pkgsite: trim the non text elements from subdirectories synopsis

synopsis here (I believe this is the line https://github.com/golang/pkgsite/blob/master/content/static/html/pages/subdirectories.tmpl#L19) looks really awkward

as opposed to the

may I suggest rendering that synopsis to html and removing all non text html elements from that section? something like this might work
depth := 0
summary := bytes.NewBuffer(nil)
for {
	tt := z.Next()
	switch tt {
	case html.ErrorToken:
		break
	case html.TextToken:
		summary.Write(z.Text())
}
return summary.String()

@sevki - thanks for the feedback!

may I suggest rendering that synopsis to html and removing all non text html elements from that section? something like this might work

Feel free to send us a CL!

@julieqiu sure. Will do, let me check Monday if we have your corp CLA signed :)


Dynamic update data for chart in canvas

Hello, thank you for your great work.
I am able to connect fabric.js with chart.js by using your library. The chart is rendered pretty smooth.
Now I want to update the chart's data dynamically by user input, but the chart is not re-rendered again.
I have checked the chart's chart property and can see that all the data are updated correctly, but for some reason it is not displayed in the canvas.
Here is the function I am using for update data.  Is there anything missed, thank you for your help 😄
updateChartData: function(dataset, labels){
    const canvas = CommonHandlers.getCanvas();
    const currentChart = canvas.getActiveObject();
    if (currentChart) {
      currentChart.set('chart', {
        data: {
          labels: labels,
          datasets: dataset.map(row => ({
            label: '# of Votes',
            data: row.map(item => parseFloat(item.value)),
            backgroundColor: [
              'rgba(255, 99, 132, 0.2)',
              'rgba(54, 162, 235, 0.2)',
              'rgba(255, 206, 86, 0.2)'
            ],
            borderColor: [
              'rgba(255, 99, 132, 1)',
              'rgba(54, 162, 235, 1)',
              'rgba(255, 206, 86, 1)'
            ],
            borderWidth: 1
          })),
        }
      });
    }
    canvas.renderAll();
  }

Hi! Glad you like the package. Can you simply try to add a currentChart.set('dirty', true) after setting the data? The package does not force a re-render after updating the data unless you specifically request it.

Hi @yassipad 😄
I have tried to set the dirty property to true after setting the data but the chart is not auto-update until I drag it. And by setting the dirty property, I cannot interact with the chart as before (hover to see the value, ...).

Hi @yassipad,
Thank you for your tip 🤩  It can render new data smoothly now 👍🏻
By the way, I also have faced a problem with chart scaling.
When I try to scale the chart, it sometimes renders a blank object. I think it relates to the responsive and maintainAspectRatio options, I have to set these options to true but not working properly. Do you have any idea? 😄
https://user-images.githubusercontent.com/11147999/113080906-a88fe000-9212-11eb-82ac-c9da2b8244f7.mov

This is a bit odd but unfortunately i think it's probably related to the number of rendering requests when scaling... Is cache disabled ?

I have set the objectCaching: true when creating the chart but still face the same problem.

public initialize(options?: fabric.IChartConfiguration) {
	super.initialize(options);
	this.__createChart();
	this.__bindChartEvents();

	this.on('scaling', debounce(this.__setChartSize.bind(this), 5));
+	this.on('scaled', _ => {
+		const config = {
+			width: Math.ceil(this.width * this.scaleX),
+			height: Math.ceil(this.height * this.scaleY),
+			scaleX: 1.0,
+			scaleY: 1.0,
+		};
+		Object.keys(config).forEach(k => this.set.apply(this, [k, config[k]]), this);
+		this.__setChartSize.bind(this);
+	});

	return this
}

Thank you @QJesus,  it works amazingly! 😄

@QJesus @pnbao Hi, Can you tell me what exactly does the above code by @QJesus does? I am facing the same blank object issue on scaling.

after scaled, resize width, height and scaleX/Y equals 1, chat cannot be scaled
can change Object.keys(config).forEach(k => this.set.apply(this, [k, config[k]]), this); to this.set(config)


r128gain: 1.0.1 -> 1.0.3

Motivation for this change
Aside from the update, override the updateScript set by buildPythonApplication as it doesn't support GitHub sources. Manual intervention might still be needed for the patch file.
Things done

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

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer


Ability to install automatically binaries to /usr/local/bin

This would help would with integrations for IDES tremendously, instead of having to specify mise's shim path. or creating symlinks to the shims, it would be nice if it installed binaries to /usr/local/bin automatically by default for root users and $HOME/bin for non-root users ,and or allow to users to select a custom path. like ubi does

I'm not sure I follow. This would only support global installs of tools, you wouldn't be able to use project-versions which is a fundamental feature in mise.

I'm not sure I follow. This would only support global installs of tools, you wouldn't be able to use project-versions which is a fundamental feature in mise.

it would be more so for the tools that could be installed with asdf like  delta, fzf, lazygit, etc. and I find being able to install languages and tools globally with mise keeps my env very clean and organized. Though I guess making it default install to global for people wouldn't make much sense, but allowing the option  would be nice.


Ensure the stability of Nix CI

In https://github.com/MinaProtocol/mina/pull/10898, we propose a prototype of a Nix-based CI. To ensure that it works correctly, we need to run it in parallel with the current CI system. For that, a suitable webhook needs to be added, and some buildkite setup has to be changed.

I think it's safe to close this issue. The current state is that the nix CI is running side by side with the current CI (in a non-blocking manner), and it's working smoothly.
Switching over to the nix CI will of course require us to check that everything is still working, but we don't need to track that with this issue anymore.
Nice work @balsoft!


Using current ELF in Ghidra instead of file on disk?

Right now, ghidra2dwarf.py relies on having the original ELF on disk. It ends up breaking one of my automated workflows, since my tooling cleans up samples off disk after they've been imported to Ghidra.
https://github.com/cesena/ghidra2dwarf/blob/650597d7553654cd0c643c4bfbffaf9c8d9e1b4a/src/ghidra2dwarf.py#L69-L73
https://github.com/cesena/ghidra2dwarf/blob/650597d7553654cd0c643c4bfbffaf9c8d9e1b4a/src/ghidra2dwarf.py#L515
We should use the ElfExporter class instead to get the original ELF file. https://github.com/NationalSecurityAgency/ghidra/blob/master/Ghidra/Features/Base/src/main/java/ghidra/app/util/exporter/ElfExporter.java
Here's some examples of how to do that:
https://github.com/sengi12/GhidraScripting-Basics/blob/fa6181bf0634a12015296fed54068bff0585acc5/examples/exportLocalCopy.py#L22-L27
https://github.com/riverratz/ghidraheadless_binexport/blob/b69966cc6ece8820588355c5f5b893b6d0d1c613/sample_functions_cpy.py#L17-L32

I'll do this myself, just wanted to make a ticket so I don't forget.