AppLocker prevents package installation

Brief description of your issue

our company has AppLocker enabled with an allow-list of directories that executables can be started
when installing a package (like Git.Git) it results in the following error (manually translated into english)

Installer-Hash successfully validated
Package installation is going to start...
Unexpected error during execution of command:
0x8007029c : An assert error has occured.

the applocker event log states

The execution from %OSDRIVE%\USERS\SOMEUSER\APPDATA\LOCAL\PACKAGES\MICROSOFT.DESKTOPAPPINSTALLER_8WEKYB3D8BBW              E\TEMPSTATE\WINGET\GIT.GIT.2.32.0.EXE was blocked.

Expected behavior
We should be able to set the package directory where the packages are downloaded and executed.
A option within winget settings should be added to override the default location.
Actual behavior
With AppLocker enabled, we can not use winget, because executables cannot be run inside the users profile path.
Environment
Windows Package Manager v1.0.11451
Windows: Windows.Desktop v10.0.19042.1052
Paket: Microsoft.DesktopAppInstaller v1.11.11451.0

We have a change coming in the next release to change the download location to %TEMP% https://github.com/microsoft/winget-cli/pull/1146.

This looks like it might be the same thing as #970.

We have a change coming in the next release to change the download location to %TEMP% https://github.com/microsoft/winget-cli/pull/1146.

This would not help us as the temp directory is the main directory for malicious scripts and programs and therefore also blocked by AppLocker.
Like I said, everyone with AppLocker needs a configuration option for the path from which the exe, msi and so on will be started.

Based on your comment, this does look like a duplicate of #970.

Duplicate of #970

@denelon I stubled across this issue as well. In my opinion this is not the same as #970. This issue is about the temp folder that is used by winget to download and run the installer. #970 is about the target directory of the software to be installed into. Both of these locations are blocked by AppLocker software, since all of them base in %LOCALAPPDATA%. The applications are installed in \Programs\ and the installer is run from \Temp.
As a target install directory I can image people might want to install into the C:\Program Files\ folder, or something else. This location a user will not want to use for the temporary data stored by winget during installation, which this issue is about. I can image that you'd might to set this folder to C:\Temp\ or something different.

@paalders I'll discuss this with the team. If there aren't any major blocking reasons associated with a user specified location, I'll re-open the Issue and mark the appropriate comments "No longer applicable". If not, I'll share the reasons so we can look for alternative solutions. One of the benefits we might lose is the ability to use the "Disk Cleanup" behavior associated with "Temporary Files".

We still have some concerns regarding long path names, but it looks like this is a different ask than what I initially understood.

@denelon Whats the status about this issue. We are still struggling with winget because we cannot set the temporary folder for the package content.

We will be tackling standalone .exe and .zip packages in 1.3. I'm highly expecting we will be looking at how we handle paths for those types of packages. It will likely make sense to bring this in with that effort. We will also need to include the information via winget --info as logs from some installers tend to get placed in the directory the installer runs from.

@denelon any update on this? We also use software restriction policy and %temp% is not and won't be in the allowed list.

This didn't make it in v1.4?

Sadly, no. We did get .zip support included in 1.4, but there wasn't enough time to look at supporting alternate paths for where we download and run installers from.

In the latest 1.5 preview I couldnt find any reference to this issue.
Will there be a fix for this issue in 1.5?
We really need an environment variable to give winget a hint for the temporary folder it should use.

@michha take a look at the Roadmap to see how we generally prioritize work. This Issue simply doesn't have enough "demand" yet to move up very high in the backlog. I've moved it into the v.next milestone rather than the backlog to see if there is more interest or need for this work.

[Policy] Area-Settings


Provide a variant of @Pattern annotation which accepts java.util.rexex.Pattern

Currently @Pattern annotation supports only String regex parameter, but sometimes it is handy to pass some predefined Pattern. For example android has an utility Patterns class which exposes a number of patterns as static Pattern instances.
It would be great to have an ability to just pass them to @Pattern annotation.
I could provide a PR for this, but I am not sure at how is it best to proceed (if you agree with this feature): by creating some new annotation, or by extending @Pattern with some new field or by using some other approach?

@dimsuz I don't see too many benefits that this PR / feature would add to the library. There are 6 patterns out of which 5 are already present in Saripaar. The only missing regex is phone which you can copy from the source code itself.
public static final Pattern PHONE
        = Pattern.compile(                      // sdd = space, dot, or dash
                "(\\+[0-9]+[\\- \\.]*)?"        // +<digits><sdd>*
                + "(\\([0-9]+\\)[\\- \\.]*)?"   // (<digits>)<sdd>*
                + "([0-9][0-9\\- \\.]+[0-9])"); // <digit><digit|sdd>+<digit>


[Event Request] CodeUnit 5763 "Whse.-Post Shipment" - OnInitSourceDocumentLines

local procedure InitSourceDocumentLines(var WhseShptLine: Record "Warehouse Shipment Line")
var
WhseShptLine2: Record "Warehouse Shipment Line";
begin
WhseShptLine2.Copy(WhseShptLine);
case WhseShptLine2."Source Type" of
DATABASE::"Sales Line":
HandleSalesLine(WhseShptLine2);
DATABASE::"Purchase Line": // Return Order
HandlePurchaseLine(WhseShptLine2);
DATABASE::"Transfer Line":
HandleTransferLine(WhseShptLine2);
DATABASE::"Service Line":
HandleServiceLine(WhseShptLine2);
else
OnInitSourceDocumentLines(WhseShptLine2);
end;
WhseShptLine2.SetRange("Source Line No.");
end;
You included an "else" event object in the other main work functions of this posting codeunit but not this one, I think you forgot one.  Please add it.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Add SOLID target support

This PR adds support for SOLID targets. See https://github.com/rust-lang/rust/pull/86191 for details.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @JohnTitor (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Hey, thanks for the PR! I'm holding off this PR until rust-lang/rust#86191 is approved.

@JohnTitor I think this can be unblocked now that https://github.com/rust-lang/rust/pull/86191 is approved?

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing a7932acd5279711161e2e3b4325ebd5422671c1c to master...


Update for A-Frame 0.6

Just wondering if the npm package is going to be updated to allow installation if a-frame 0.6 is present?
I get the following error when trying to install:
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>-> true
└─┬<EMAIL_ADDRESS>  ├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>(git://github.com/donmccurdy/cannon.js.git#022e8ba53fa83abf0ad8a0e4fd08623123838a17)
    └──<EMAIL_ADDRESS>

Fixed, thanks!


Keyboard layout cahnger

Description of the new feature / enhancement
The keyboard shortcut changes the written text to a different keyboard layout
Scenario when this would be used?
Example:
i have 2 keyboard layout (rus and eng)
Situation:
I want to write on eng, but keycoard lauout set on rus
In result it's "ифв лунщфкв дфнщге". But i need "bad keyboard layout"
What i want:
After this situation using keyboard shortcut (example: ctrl + shift + p) change typed text on other kryboard layout
("ифв лунщфкв дфнщге" >> use ctrl + shift + p >> "bad keyboard layout")
Supporting information
No response

Doesn't the "Windows+Space" shortcut from Windows does this for you already?


PHP 8.1: trim(): Passing null to parameter #1 ($string) of type string is deprecated

Laravel Version: 9.3..1
PHP Version: 8.1.3
Database Driver & Version:

Description:
A PHP warning is emitted when trim() is passed a null. trim() is used extensively throughout the framework,
but specifically in src/Illuminate/View/Compilers/Concerns/CompilesStacks.php the framework sets $id to null
as part of default behavior when processing a standard Blade pushOnce directive. There may be other instances
in the framework where this happens, but I have only triggered this particular instance.
Steps To Reproduce:
This was discovered by running tlint in a repository where the pushOnce directive was used.
Recommendation
Replace null with the empty string '' when setting a default 'empty' value that will be passed on to trim().

Apparently a lot of the string functions, not just trim() have more strongly-typed parameters now. It's probably a decision better made by the maintainers whether to fix everything in on big PR (probably just casting with (string) before parameter #1 is sufficient where appropriate), or to spread the fixes over many PRs in more easily reasoned-about chunks.

Apparently a lot of the string functions, not just trim() have more strongly-typed parameters now. It's probably a decision better made by the maintainers whether to fix everything in on big PR (probably just casting with (string) before parameter # 1 is sufficient where appropriate), or to spread the fixes over many PRs in more easily reasoned-about chunks.

How would casting (string) resolve this on a null parameter?  Would the value just be "null"?

As a follow-up thought, this deprecation seems like an indication that PHP is becoming more strongly typed, and it's likely worth having a long think about whether or not casting null values will be enough in the long-term. There's probably a good case to be made that adding type declarations wherever possible (function arguments, return values, class properties) should be a part of the fix, in order to better align with the direction that PHP is headed. There's probably a lot of other things I'm not considering here in terms of knock-on effects of a stronger type system or a move away from ad hoc type juggling.

@selfsimilar Thank you for those links, and I strongly agree that the fix should be future proof for the direction PHP is heading.   Probably starting off multiple small PRs is the way to go to start enforcing strictly typed variables in Laravel.

A PR was sent in for this, thanks.


Fix E2 functions to return default (non-nil) value (Fixes #1374 #1420)

Fixes #1374.
Fixes #1420.
 Fixed return values of many (broken) E2 functions, so now they will always return their default (and correct) values instead of a nil value...

There are probably more E2 functions to be fixed as said by @Divran (in issue page).
This is work in progress. Please reopen it.

Working as intended

@Divran Please reopen this since @TomyLobo doesn't know that Lua nil value is not the same as NULL entity. (see the issue page for details)

Fix the conflicts and then I'll review it.

Fix the conflicts and then I'll review it.

Right.....

It is gone. I do still have a backup of the repository on old hard drive tho.. I don't know if I have enough motivation to ditch this. rip...


inbox swipe actions

I had fun working on the swipe-to-reply feature, so I thought it would be cool to add something I thought would be useful, which is swipe left on inbox items to reveal actions. This patch adds it for "small team" rows (anything in time sorted list), and reveals "Mute" and "Hide" which are the same options available from long press. Here is a little demo of it in action (feel free to suggest different colors for the buttons):
cc @cecileboucheron @adamjspooner

Nice.
I don't know that we should use orange for the Mute/Unmute. Apart from that, lgtm!

Let's play with the colors later.


Allow importing an ebook from the menu

Drag & drop is not sufficient because 1. it's not obvious that it works 2. people with visual disabilities can't use it.
Therefore we need to add a Add to Library (cmd A) to the ReadiumDesktop menu.

done


Need to turn TUI off

I want to be able to run dagger run sh to run commands against an open session. In this case the command I’m running needs to be interactive so I need control over stdin, stdout and stderr.
This is also related to https://github.com/dagger/dagger/issues/5181 but I’ve wanted to turn it off for some reason multiple times when running our test suites. During tests you get a lot of failed pipelines but it’s okay because we’re making assertions on that, but it adds a lot of noise to the output. Sometimes when I’m debugging, I just want to Printf some things and skip the whole TUI.
I tried to use a pipe or redirection so that useTUI is false:
https://github.com/dagger/dagger/blob/1d902d3f2e3bcc20765683602dbdff214ed012ac/cmd/dagger/engine.go#L20-L21
But it seems to swallow it 🐞:
$ dagger run env | grep DAGGER
# TUI shows up and all env vars are returned

$ dagger run env > out.txt
# TUI shows up and out.txt is empty

\cc @vito

I think that this will be partially fixed by https://github.com/dagger/dagger/pull/5197. Would that be sufficient for you @helderco?


💡[Feature]:  Monkey Pox Confirmed Cases

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
To cleanse and analyse the course data to understand and draw insights from the given dataset.
Use Case
To cleanse and analyse the course data to understand and draw insights from the given dataset.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@TAHIR0110   can you please assign this to me?


feat: improve the styling of Test Tools

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature

What is the current behavior? (You can also link to an open issue here)

The various Test Tools tables can be difficult to read at-a-glance.

What is the new behavior (if this is a feature change)?

The styling of these pages has been improved.

Other information:

I was only able to test this with the Timeline tool. Someone should test with the 3 other tools before merging.
Before:

After:

Status

[x] Code documentation for the relevant parts in the code have been added/updated by the PR author
[x] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report
Patch coverage has no change and project coverage change: +8.65 :tada:

Comparison is base (a4a6867) 56.05% compared to head (d695c16) 64.70%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@              Coverage Diff              @@
##           release50     #951      +/-   ##
=============================================
+ Coverage      56.05%   64.70%   +8.65%
=============================================
  Files            487       14     -473
  Lines          77299     2930   -74369
  Branches        3777      222    -3555
=============================================
- Hits           43332     1896   -41436
+ Misses         33916     1033   -32883
+ Partials          51        1      -50

see 473 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Make additionalFields configurable for JUL

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #118 opened

Start Time: 2020-12-16T14:36:54.759+0000

Duration: 8 min 16 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
144

Skipped
4

Total
148


Simple border for non-rectangular shapes

Feature Description
Enable the border panel for non-rectangular shapes with the possibility to add a simple border that supports:

One border-width value only. If different border values exist before and the user changes that element to a shape, the left border value is used.
Allows choosing color, including opacity.
Supports outside border only (as the rest of the elements).

Alternatives Considered

Additional Context

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief
For adding border to the shape, we could use an additional svg with stroke and stroke-width, in a similar way that the drop-targeting blue line is currently implemented when hovering over a shape / starting dropping into a shape:
      <svg
        viewBox={`0 0 1 ${1 / mask.ratio}`}
        width="100%"
        height="100%"
        preserveAspectRatio="none"
      >
        <path
          vectorEffect="non-scaling-stroke"
          stroke="yellow"
          strokeWidth="5"
          fill="none"
          d={mask?.path}
        />
      </svg>

Since adding a border directly to the existing element would create an inner border instead, we should add a separate element for the border.
We can add a new wrapper (or separate element) with the same clip-path, and increase and position this new element based on the border width (e.g. if border-width is 5px, increase it by 10px and move it to maintain visually the same position, too). We can then add the border svg (described above) into the wrapper with increased size. The currently existing mask could also be inside the new wrapper element, or could be separately as well.
This needs to be done in the DisplayLayer, note that we are already doing similar positioning for the outer border with the rest of the elements as well.
The structure could be something like this:
<IncreasedSizeWrapper clipPath=... width="calc(100% + 10px)" height="...">
	<svg .... ><path stroke="yellow" strokeWidth="5px" .... /></svg>
	<CurrentlyExistingMaskElement .... />
</IncreasedSizeWrapper />

or this:
<svg .... ><path stroke="yellow" strokeWidth="5px" .... /></svg>
<NotIncreasedSizeWrapper clipPath="...">
	<CurrentlyExistingMaskElement ... />
</NotIncreasedSizeWrapper />

See the conversation and considered alternatives in more detail here.

cc @o-fernandez Pinging you here for prioritization and reviewing the feature description.

@miina the description looks good. w.r.t. prioritization, I'll mark it as a P2 for now, something we should try to do but not urgent if there are other P1s/roadmap items.

Tagging @gajasinski just in case to confirm moving forward with the feature and to review the feature description, since this issue has been stale for quite some time.

@choumx @gajasinski For implementing the border for non-rectangular shapes, I have tried some different solutions all which work to a certain degree but with some shortcomings. Since I have exhausted all of the ideas that I had about implementing this feature without finding a suitable solution, wanted to know if you have any thoughts about how to proceed or if you have any other alternative solutions that I might have missed. Here is a summary of my findings:

Solution
Features
Issues

1
Current solution: separate border SVG element with stroke
- supports transparent border  - simplest solution of the three options
- Supports only for non-transparent shapes, otherwise the following issues would appear:   1. Leaves some spaces between  the border and element in some shapes  2. Border and the element overlap making elements with transparency have some 'ghost borders'

2
SVG-stroke on the element, needs testing out
- support border for all shapes including shapes with transparency
- This would require changing the implementation of the shapes from using clipping to using a mask  - The element would not be cut off/clipped and would be clickable outside of the visible shape  - Needs more time to replace using clip-paths with masks  - SVG draws the stroke at the center of the element's boundary with half the stroke width "inside" the shape and no option to specify the stroke position. This will need some "hacks" to get around.

3.
CSS Paint API
- Most robust and supports most/all shapes & transparency
- Experimental and low browser support (caniuse)  - Uses more performance and resources

cc. @miina @barklund

@Trapsta thanks for laying this out. Can you show me an examples of what #1 would look like?

@gajasinski #1 is basically whats implemented in #9851. Note the spaces between the heart shape and the blob as well as the border showing through the triangle  shape with opacity.

@choumx @Trapsta thoughts of how much effort it would be to implement #2? The above experience looks unprofessional.

@gajasinski @choumx this will obviously need more testing out as noted in the table and might have some overlap with #5325  but I believe around 6-8 points give or take. Also might  need to first try it out on a small scale as a PoC to see if any unexpected issues come up from changing the shape's implementation from clipping to masking.

@Trapsta Let's try it out as a quick proof of concept to see what kind of issues come up that might require extra effort / bring new blockers. No need to make it work perfectly for the proof of concept, just need to confirm it works well for the shape border, and which issues need to get addressed (e.g. clicking "through" the empty area of the shape might be tricky).

Okay, noted @miina , I will look into that

@miina @gajasinski from testing out #2, my conclusion is largely negative after encountering a potential show-stopper. Issue is  that  SVG considers the stroke which we want to use as the border as part of the mask. Basically how masking works is you have an element and a mask placed on top of it. The mask hides, shows or partially shows the underlying element based on the pixels (color) of the mask. If the pixels at a particular point are say white, it displays the element , if black it hides and if some other color - allows the elements pixel to be visible, but with a certain level of transparency. The deal breaker is that if the border is considered part of the mask , it will be impossible to have some borders in a particular color and also the color of the border won't be always accurate due to interference from the mask.
This is demonstrated below where if we have a green border(stroke), it is visible albeit with some transparency but if we have a white border, it doesn't show since the mask reveals the underlying element where the pixels are white.

Green Border
White Border

A more interactive example is this codepen https://codepen.io/sdras/pen/d2a9315e310901a3d43ba3bdf8413c65 . Changing the stroke="#888" attribute inside the polygon in the mask element to stroke="#fff" and might help to remove the stroke-dasharray="10, 5" but not necessary. This doesn't result in a white border because the mask reveals the pixels underneath the stroke.
With that said , we are left with  either #3 or  going with #1 but  without transparency unless someone has some alternative ideas we could try out.  @gajasinski , do you think allowing non-transparent shapes to have border and otherwise not is an option?
cc. @barklund @choumx  @bmattb

adding @agingoldseco and @aaskedall for feedback

Hi miina,
Was trying to comprehend all of the above. It seems like the current way we have paths or .svgs implemented is limiting to what we can do with the stroke.
Are we only talking about using these strokes for the 12 "Shapes" or any SVG added to the work area?
I know it's a completely different stack but any ideas on how Figma is able to render these on the fly in the browser? Do we need to think of our shapes differently?

Two new approaches to try out (in prioritized order):

Add the border layer on top of the element (as originally implemented in this PR), but rather than increase the shape size and render a bordered path on both sides of that, instead render a border at the original shape size but at double the width, and then use the original shape as a mask to "punch out" the inside of the border leaving only the outside border (at half double-width, thus correct width)
Define the border as a path defined by the original shape on the inside and a separately defined shape on the outside and simply render this using <path fill /> without a stroke. Because this requires extra work (defining the outside shape of the border), this is a secondary option.

@barklund I'm temporarily assigning this issue to you since as far as I understand we're currently waiting for Codepen with an example of the suggested approach nr 4.
@Trapsta I know you were trying out something different as well so please leave your final notes and findings here as well.

Sure @miina.  I haven't yet extensively looked into these pens that Morten was very gracious to spin from the suggestions above but I was able to loosely validate that they  fit into the general solution that we were looking for which is support transparency for both border and shape as well as support for image masks.
Here are the codepens; (beware of creepy Nicholas Cage placeholders courtesy of Morten 😉)

https://codepen.io/barklund/pen/BawPpZw - Renders shape and border in a single svg. Works but not very ideal since incase of an image mask for the shape, image will be an svg element and this might have some side effects.

https://codepen.io/barklund/pen/MWEBpxj - This is no. 4 in the suggested options. Border is added as a separate svg. Uses a mix of both clipping and svg mask to implement shape and border.  More ideal and least disruptive since it's accommodates how we currently implement the shape and image masks.

https://codepen.io/Trapsta/pen/ZEXjEew  - Also tried using the Zero Width Tunnel technique more as an experiment but found it a bit too complex and tasking for the application since it would involve updating the path commands every time the border was updated and also didn't fit very with some shapes (non-regular polygons).

In conclusion I think we can proceed working with #2. This is what it looks like after a quick test:

Basic Shape
Shape transparency
Border transparency

I did find a few minor issues here and there that I shared with Morten e.g. Some shapes had their vertices cut off but I got them to display correctly after playing around with the scaling. This might be because we haven't factored in the mask path transform ratio which is what we currently use to get the viewbox when rendering the svg but I haven't tested this out so it's only speculation.

Tested against https://stories-qa-wordpress-amp.pantheonsite.io/wp-admin/, using PR #9851.

Added a heart shape
Increased border width value
Decrease border width value
Updated layer opacity


Remove theme doc from components documentation

"To learn more about how to customize the appearance of components, please see the Theme docs." where are theme docs? How can I use it?

"To learn more about how to customize the appearance of components, please see the Theme docs." where are theme docs? How can I use it?

We're working on the themeing for the components.
Theme doc will be available when we introduce dark mode.
Currently, we're removing that from that documentation.
Good finding, thanks @asynctushar


Sharing Text Files (.txt) Doesn't Work

Seems like the plugin tries to interpret .txt files as normal shared text, so the content is null.
You can reproduce this using the share menu in the Markor app. Try creating a test file then using Share -> File.

Yo @ImranR98, thanks for brining issue to my attention. I tested it on both Android and iOS and it seems only text file are not being shared due to some bug.
Will fix it after I get some free time.

@ImranR98, hi can you try adding below code in the AndroidManifest.xml file and see if it works on Android?
<queries>
    <intent>
        <action android:name="android.intent.action.PROCESS_TEXT"/>
        <data android:mimeType="text/plain"/>
    </intent>
</queries>

I cannot seem to get any solution for iOS yet.

Sharing .csv file as file, from Files App, doesn't work on iOS as well, works on Android.

@pingdong  from ShareViewController.swift inside handleFiles method change newPath with:
let newPath = FileManager.default
.containerURL(forSecurityApplicationGroupIdentifier: this.appGroupId)!
.appendingPathComponent("(fileName)")
this should append the file correctly and it should work. In my case solved problem with json files

@ultormagister  thanks for help, unfortunately, it doesn't work.
Here is the log
2024-04-08 21:50:30.590267+1200 File Uploader[7189:919692] [core] SLComposeServiceViewController got NSURL file:///private/var/mobile/Library/Mobile%20Documents/com~apple~CloudDocs/Downloads/test%202.txt for public.file-url error: (null)
2024-04-08 21:50:30.646189+1200 File Uploader[7189:919491] [core] SLComposeServiceViewController got attachment coarseType 0
2024-04-08 21:50:30.646325+1200 File Uploader[7189:919491] [core] SLComposeServiceViewController made no attachment for itemProvider conforming to public.file-url
2024-04-08 21:50:30.881340+1200 File Uploader[7189:919491] [MC] System group container for systemgroup.com.apple.configurationprofiles path is /private/var/containers/Shared/SystemGroup/systemgroup.com.apple.configurationprofiles
2024-04-08 21:50:30.883238+1200 File Uploader[7189:919491] [MC] Reading from public effective user settings.
2024-04-08 21:50:31.672428+1200 File Uploader[7189:919491] [Common] BSMachError: port 7903; (os/kern) invalid capability (0x14) "Unable to insert COPY_SEND"
[ERROR] Error loading data!
2024-04-08 21:50:31.837708+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:31.838322+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:31.838525+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:31.838984+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:31.958461+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:31.958660+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:31.994617+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:31.994787+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:32.010750+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:32.010942+1200 File Uploader[7189:919697] [Assert] Cannot be called with asCopy = NO on non-main thread.
2024-04-08 21:50:32.022697+1200 File Uploader[7189:919697] [Animation] +[UIView setAnimationsEnabled:] being called from a background thread. Performing any operation from a background thread on UIView or a subclass is not supported and may result in unexpected and insidious behavior. trace=(
0   UIKitCore                           0x00000001c86e17f0  + 116
1   libdispatch.dylib                   0x000000019b2bd7d4  + 16
2   libdispatch.dylib                   0x000000019b260eb8  + 28
3   UIKitCore                           0x00000001c86e1778  + 100
4   UIKitCore                           0x00000001c86e18e4  + 92
5   UIKitCore                           0x00000001c80f3db8  + 408
6   UIKitCore                           0x00000001c80defb0  + 88
7   UIKitCore                           0x00000001c80e349c  + 168
8   UIKitCore                           0x00000001c7cbe114  + 1232
9   UIKitCore                           0x00000001c7cbfccc  + 4300
10  UIKitCore                           0x00000001c7cc23a8  + 104
11  UIKitCore                           0x00000001c7cc28a4  + 508
12  UIKitCore                           0x00000001c7cc2300  + 176
13  UIKitCore                           0x00000001c7cc2560  + 160
14  File Uploader                       0x0000000100f7a2e4 $s13File_Uploader19ShareViewControllerC16dismissWithError33_C3E789C08EE88B77150BB6DED3E748A9LLyyF + 540
15  File Uploader                       0x0000000100f7698c $s13File_Uploader19ShareViewControllerC10handleText33_C3E789C08EE88B77150BB6DED3E748A9LL7content10attachment5indexySo15NSExtensionItemC_So14NSItemProviderCSitFySo14NSSecureCoding_pSg_s5Error_pSgtYbcfU + 1496
16  File Uploader                       0x0000000100f76a84 $sSo14NSSecureCoding_pSgs5Error_pSgIeghgg_ABSo7NSErrorCSgIeyBhyy_TR + 140
17  Foundation                          0x000000019c3135d0  + 140
18  libdispatch.dylib                   0x000000019b2bca38  + 24
19  libdispatch.dylib                   0x000000019b2bd7d4  + 16
20  libdispatch.dylib                   0x000000019b266324  + 592
21  libdispatch.dylib                   0x000000019b266e40  + 428
22  libdispatch.dylib                   0x000000019b26f4ac  + 596
23  libsystem_pthread.dylib             0x000000019b49e114 _pthread_wqthread + 304
24  libsystem_pthread.dylib             0x000000019b4a0cd4 start_wqthread + 4
)
2024-04-08 21:50:32.059129+1200 File Uploader[7189:919697] *** Assertion failure in -[UIKeyboardTaskQueue waitUntilAllTasksAreFinished], /BuildRoot/Library/Caches/com.apple.xbs/Sources/UIKitCore/UIKit-3698.140/Keyboard/UIKeyboardTaskQueue.m:453
2024-04-08 21:50:32.147830+1200 File Uploader[7189:919697] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: '-[UIKeyboardTaskQueue waitUntilAllTasksAreFinished] may only be called from the main thread.'


add dora

Hi, I need help to add our chain. @schnetzlerjoe

Hey! Thanks for this. We recommend using the following to add chains on the fly!
import { suggestChain } from "@cosmsnap/snapper";
let chain: Chain = {};
await suggestChain(chain);

or
const chainInfo: ChainInfo = {}
window.cosmos.experimentalSuggestChain(chainInfo)

Let us know if you need any help :)


Investigate & upgrade next-mdx-remote to v4

Important: we can't upgrade to v4 with the current code as it is. See #18 for more details about why we downgraded to v3.
Investigate how to fix the issues we have with next-mdx-remote v4 and upgrade it https://github.com/hashicorp/next-mdx-remote
We could potentially replace gray-matter since the new version can parse frontmatter as well and simplify a bit the code.
const mdxSource = await serialize(source, { parseFrontmatter: true })

@pettinarip took time to check this out. Did a bump to<EMAIL_ADDRESS>which looks to be the minimum version to jump to minimize any needed changes.

With heading ids, the beginning curly brace needs to be escaped so it is not interpreted as an expression syntax. See parse expression error troubleshoot.
## Official Ethereum support {#official-support} => ## Official Ethereum support \{#official-support}
Regarding text content inside components: I think wrapping the text content only in an expression would be the most natural

<DocLink>
  { "Ethereum security and scam prevention" }
</DocLink>

As of now, have not investigated further on the removal of gray-matter.

I am not seeing anything else to address at that version to prevent breaking.


Async generators for python 3.5

This packages provides async generators for Python3.5:
https://pypi.org/project/async_generator/
Is there interest in using this? Let's say if I open a PR, would that make a change of being merged?

Either that, or python_requires in setup.py should be changed to not advertise Python 3.5 support. The jinja2.asyncfilters and jinja2.asyncsupport contain syntax errors when imported in Python 3.5.

Jinja does support Python 3.5. Async support is Python 3.6+. While the library is interesting, I don't want to add a required external dependency.


[5.3] Added username, icon and channel options for Slack Notifications

This PR allows you to set a custom username, icon (emoji) and channel override for incoming webhooks in Slack.
This is a totally supported feature in Slack, so why not take advantage of it?
<?php

class FooNotification extends Notification
{
    public function toSlack($notifiable)
    {
        return (new SlackMessage)
                    ->from('Ghostbot', ':ghost:')
                    ->to('#testytest') // `@username` is also supported
                    ->content('This is a message from a ghost posted in #testytest!');
    }
}

.. will give you this:

As the new methods are optional, this PR is not a breaking change.
Thoughts?
--
Edit: Renamed methods

I guess as cannot be used as a method name 😭  Ideas, anyone?

@kalfheim from('Ghostbox', ':ghost:')?

@crynobone Yup! Sounds good to me.
I also renamed in to to. Makes more sense when sending a notification to a user instead of a channel (by typing @username as opposed to #some_channel)

Has merge conflicts.

Thanks, @taylorotwell! Was just about to try and fix the merge conflict, but you beat me to it. 😄

Why can't I use icon_url not icon_emoji ?


az network vnet create is not idempotent

Describe the bug
running the second time results in an output status of 1, first time it's 0.
so second time it fails.
To Reproduce
Execute the following twice:
az network vnet create -g $RESOURCEGROUP_VNET_NAME -n $VNET_NAME --address-prefix <IP_ADDRESS>/8 -l $LOCATION
echo status: $?
az network vnet subnet create -g $RESOURCEGROUP_VNET_NAME --vnet-name $VNET_NAME -n aks-subnet --address-prefix <IP_ADDRESS>/16
executing the first command results the second time in:
Subnet aks-subnet is in use by /subscriptions/xxxxxx/resourceGroups/playground-rg/providers/Microsoft.Network/networkInterfaces/xxxxx/ipConfigurations/ipconfig1 and cannot be deleted.
Expected behavior
Twice exactly the same behavior and result status of 0. Second time it just already exists.

I just tried your repo steps and it worked without issue. The second call to vnet create wipes away the subnet, but that is the expected behavior.
From the error message it seems like after creating the subnet but before you tried to run vnet create again, you associated that subnet with a network interface. This might be due to a call to vm create which, if a vnet is not specified, looks for a vnet in the resource group and uses that.

Hmm. You are correct about my repo steps. I will have a second look what is happening exactly in my situation.
Are you serious about the wiping of the subnet? My opinion would be that it is not allowed to delete  subnets. The commands asks to create the vnet. If the vnet exists already, it should leave it as is. Why is it expected that the command deletes subnets?

Nearly all create commands in the CLI are PUT requests. If you make a PUT request for a resource that already exists, you overwrite it. vnet create certainly follows this pattern. Doing an existence check would result in the command not being idempotent because the second call would fail.

This is confusing. "az webapp create" is not a PUT command becuase it gives this error:
az : ERROR: The site with name testsite already exists in resource group testrg.
So I have to do existence check myself.
Can someone explain why some commands work different than others?

@corveen the issue with webapp create was someone inserted validation logic that broke the idempotency of the command. That has been fixed and will be in the next release.


To CMYK

What are you trying to do?
It would be nice to add a toCmyk filter to the plugin. CMYK conversion appears to be supported in the Iris package as of October '23
What's your proposed solution?
Add a new Twig filter for toCmyk
Additional context
No response

Happy to add, but from what I can tell they haven't tagged CMYK support in a release. Their latest release is 3.1.1 with no CMYK support.

Happy to add, but from what I can tell they haven't tagged CMYK support in a release. Their latest release is 3.1.1 with no CMYK support.

Thanks for looking into it Josh! Unless I'm mistaken, it looks like CMYK was added, but never documented. See https://github.com/ozdemirburak/iris/pull/44

Yep, I can see it in their master branch (and documented), but it's not included in any of their releases


Add native support for jdk8 jsr310 date types

Fix #1544

Should we also do the same for joda and threetenbp ?

Im cool with pulling those in as well, only the'll require provided dependencies for those libraries and checking if those libraries exist in the class path

Is this ready to be pulled in? or should I wait for joda/310bp?

I can do the joda and threetenbp now. Shouldn't take long.
Since, there are only 5 mappings to do, it seems easier to integrate the libs by reflection than runtime dependency. What do you think ?

Arf ! There is a problem: when I add the rules for joda, I get
springfox.documentation.schema.alternates.AlternatePropertiesSpec > Nested properties that have alternate types defined are rendered correctly FAILED

Condition not satisfied:

modelProperty.type.erasedType == String
|             |    |          |
|             |    |          false
|             |    class java.sql.Date
|             java.sql.Date
springfox.documentation.schema.ModelProperty@5abf774f

	at springfox.documentation.schema.alternates.AlternatePropertiesSpec.Nested properties that have alternate types defined are rendered correctly(AlternatePropertiesSpec.groovy:52)

Apparently, if there is a default rule, the user can't override with a new one.
@dilipkrish Do you confirm ?

I also made a test in a Spring boot app using  .directModelSubstitute(java.time.LocalDate.class, String.class) and this time it worked: the swagger field was a regular "string" and not a "date-time" !
So I'm not sure what's wrong. Do the directModelSubstitute rules apply before the default rules ?

I think its talking about nested types... you want to push your changes? I can look

I just pushed. The CI should show the test error.

Its been pulled in thanks for putting this together! 🙇

Great ! :+1:

While this might be a step in the right direction, it still doesn't obey the configuration of the ObjectMappers configured by the user's ApplicationListener<ObjectMapperConfigured>. For example, even if https://github.com/FasterXML/jackson-modules-java8/tree/master/datetime is in use, Springfox describes Instant as serializing to, eg., "2016-11-07T23:04:55.623Z". However, it should be serializing to an integer.
Perhaps if Swagger used the ObjectMapper configured by Spring, this code would be unnecessary and the corresponding issues would be resolved?

@rehevkor5 The currency to determine swagger representations of types is either real types or alternate type via rules. Reaching into the object mapper to determine how a type is serialized is not in scope of this project.

@dilipkrish I see. I guess I misinterpreted the purpose of the "Configuring the ObjectMapper" section in the docs http://springfox.github.io/springfox/docs/current/#configuring-the-objectmapper What would be a valid use for that option?

The purpose of that is to match the object mapper with expectations of how serialization/deserialization is done. Sometimes the object mapper used for springfox may not be the same one used by spring-hateoas. This is to sync the configurations for e.g. camel cased properties names


[DOCS] - Documentation unclear in creating transaction

External
Summary
Documentation is unclear when creating a transaction. In the line --tx-out $(cat payment2.addr)+0, it is unclear where payment2.addr came from and what address the file contains.
Here's the page/section: https://docs.cardano.org/projects/cardano-node/en/latest/stake-pool-operations/simple_transaction.html#draft-the-transaction
Here's the full block:
cardano-cli transaction build-raw \
--tx-in 4e3a6e7fdcb0d0efa17bf79c13aed2b4cb9baf37fb1aa2e39553d5bd720c5c99#4 \
--tx-out $(cat payment2.addr)+0 \
--tx-out $(cat payment.addr)+0 \
--invalid-hereafter 0 \
--fee 0 \
--out-file tx.draft

Update
This is explained further down on the page. It's just very unclear at the time that you reach the section referenced above.

Closing this. If this is still relevant please reopen.


Profile page UI tweaks

[ ] description (short field of 255 chars) should be centered.
[ ] Clarify in the UI that the field can't be longer than 255 chars (right now it hits the server on every character change and database field length errors keep popping up continuously on the frontend. We could add a hard-limit to the html?

obsolete by v2


fixing CI pcells testing

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@FaragElsayed2 , ready for testing and review


Update functions / remove duplicates from BaseCommands and APPIUM_CORE_FUNCTIONS

Added/Updated following functions in BaseCommands:

getElementScreenshot
QueryAppState

Added comments for following functions, whose duplicates are generated in BaseCommands:

getWindowHandle
getWindowHandles
execute
executeAsync
Reference: https://github.com/facebook/php-webdriver/wiki/JsonWireProtocol-vs.-W3C-WebDriver

Updated files for APPIUM_CORE_FUNCTIONS and BaseConstants

PhpStorm Reformatting

LGTM


Duplicated database and table names

Hey guys, my company deploys databases in several countries and I was wondering how to solve the problem of duplicated database/table names?
For example, the MySQL database crm and table customers exists in both countries - Singapore (SG) and Malaysia (MY).
2 Methods that I can think of to differentiate them:

Prepend the country code in front of the dataset identifier

urn:li:dataset:(urn:li:dataPlatform:mysql,sg.cms.customers,PROD)
urn:li:dataset:(urn:li:dataPlatform:mysql,my.cms.customers,PROD)
However, this is not possible when using Metadata Ingestion MySQL helper to ingest metadata from MySQL.

For the dataset urn, add another field for country_code behind origin

urn:li:dataset:(urn:li:dataPlatform:mysql,cms.customers,PROD,SG)
urn:li:dataset:(urn:li:dataPlatform:mysql,cms.customers,PROD,MY)
This requires alteration of the DatasetKey.
Appreciate if anyone can assist me on this!

Hi there! We totally agree we need a better way to model "domains" or bounded contexts of data. Ideally, we'd want to do this in a way that generalizes across all asset types (datasets, dashboards, data pipelines, etc). Explicitly modeling domains in DataHub is on our roadmap.
Let us see if we can do something quickly to unblock your integration, and we'll loop back with a more detailed estimation of when and how we will accomplish a the more robust solution detailed above. cc. @hsheth2

Hi folks, this is currently being addressed by @jjoyce0510's Domain & Conainter work in our Q1'22 roadmap; please refer to that source to stay up to date with progress. We'll close out this issue in the meantime!


script for pre- and post-deployment doesn't work anymore

The script provided at the bottom of the page throws below error at the start trigger task.
##[error]TriggerEnabledCannotUpdate: Cannot update enabled Trigger; the trigger needs to be disabled first.
I see this issue is report on the below page as well.
https://feedback.azure.com/forums/270578-data-factory/suggestions/37608829-disable-and-enable-data-factory-triggers-for-devop
It would be nice to have a working script to avoid manual work.

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

ID: 1e22e4d6-366f-fbf8-e141-8905ca7ab7ae
Version Independent ID: 5be2acf2-66e7-cadf-6666-5a85a65b9771
Content: Continuous integration and delivery in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/continuous-integration-deployment.md
Service: data-factory
GitHub Login: @dcstwh
Microsoft Alias: weetok

@naga-pakalapati  Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Hi @naga-pakalapati - Thanks for the feedback. It has been called out in the document that Deployment can fail if you try to update active triggers. To update active triggers, you need to manually stop them and then restart them after the deployment as explained in this section of the doc: https://docs.microsoft.com/azure/data-factory/continuous-integration-deployment#updating-active-triggers

Hope this info helps.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply and reopen the issue. We will gladly continue the discussion.
Thank you


adding extras/editor/telescope-frecency.lua

Hi,
I'm migrating my nvim config to lazyvim and created an extra plugin file for telescope-frecency. You can
press [ key to jump between frequency and oldfiles and vice versa.
cheers
Arndt
Example config:
{
    "nvim-telescope/telescope-frecency.nvim",
    opts = {

      integration = {
        frecency = {
          -- key to switch from frecency to oldfiles and vice versa.
          lhs_switch_to = "[",

          -- remap default oldfiles mapping if you like (default is false)
          enable_oldfiles_mappings = true,

          -- telescope picker opts
          pickers = {
            oldfiles = {
              -- use theme = false to disable themes
              theme = require("telescope.themes").get_ivy,
            },
            frecency = {
              theme = require("telescope.themes").get_ivy,
            },
          },
        },
      },
      -- telescope extension config (see docs)
      extensions = {
        frecency = {
          show_filter_column = { "nvim", "telescope.nvim", "CWD" },
          workspaces = {
            ["nvim"] = vim.fn.expand("~/.config/nvim"),
            ["telescope.nvim"] = vim.fn.expand("~/.local/share/nvim/lazy/telescope.nvim"),
          },
        },
      },
    },
  },

Not a fan of this. Closing...
Thank you anyway!


TRPO does not work as the OpenAI baselines

Here are two pieces of code running on the OpenAI Hopper environment. The first one uses the openAI baseline implementation of TRPO, while the second one uses tensorforce - both with the same parameters as far as I could figure (I manually changed the MLP layer size to two internally).
The baseline implementation works really well, while the tensorforce implementation does not. Their internal code is a bit of a mess, so there might be some hidden setting somewhere that helps. Do you have any experience with this? It might be worth looking into to understand where the performance difference is coming from.
https://gist.github.com/befelix/517887234139a5d9ebb5654161fa0b54
Might be related to #26

2 things:

They do adaptive step sizes, for which we have not implemented any heuristics yet, which I think has a very significant performance impact, and also means just using the same starting configurations is not very indicative
They do a shared value function so the baseline computation is slightly different

I think it's more the adaptive stepsizing

Looking at your code in more detail, I think since our value function is a separate MLP, it needs way more updates (100 as set before), whereas they share the same network (if I recall correctly), so with the parameters given the baseline in tensorforce would be way, way off

It looks like they use two different networks to me https://github.com/openai/baselines/blob/master/baselines/pposgd/mlp_policy.py#L27

So it seems that with the last commit TRPO performance is better and much more stable, at least according to first test runs on CartPole. Hence I will close this issue for now. If the more in-depth experiments that we will run in the next few days still don't work well, we'll reopen it.


Bump version to rc.4

(Just for future reference, can we please hold off on bumping the version until we're ready to release? Thanks! 😁)


Allow running inspection with different build configuration

I have a solution which requires to be build with a different build configuration than the default debug x64 configuration. It would be great if there was support to pass the build configuration as parameters to the action, that would enable this action to be used for more diverse projects.

@koosvanw
Hi.
Thank you for creating issue.
You want the --properties parameter like following?
https://www.jetbrains.com/help/resharper/InspectCode.html#project-references-possible-problems-and-solutions
--properties:Platform=x64;Configuration=Debug
Is my understanding correct?

HI @muno92,
Thanks for getting back on this!
Yes, if it would be possible to pass in the Platform and Configuration values as parameters for the GitHub Actions that would be great! I can already pass NoBuild, but that is not enough for the analysis to be executed correctly.

@koosvanw
I added properties parameter in version 1.10.0.
Could you try it out?

That's quick!
I've done some testing but I'm running in a couple of issues which I failed to notice before:

Passing the argument should be --properties:Platform=... while now after properties a = is used. Changing the equals sign to a colon makes the inspection run. Otherwise it fails because it thinks multiple solutions have been passed,
In my case the Platform contains spaces. When I add that, I have to escape it somehow. If I escape with double quotes like Platform="My Platform", the entire --properties line is wrapped in double quotes in the command
Because of the quoting it is not possible to pass in both Platform and Configuration in 1 go. If I run it manually on the commandline I can pass multiple instances of --properties but I'm not sure that's a way forward you'd want to take. In my case I could just pass the Platform, that would be enough, since Configuration will for me always be Debug.

@koosvanw
Oh, sorry.
I mistaken and I fixed it.
How about it?

Quoting is still off. I have the following sample usage:
- name: Inspect code
  uses<EMAIL_ADDRESS>  with:
    solutionPath: mySolution.sln
    solutionWideAnalysis: false
    noBuild: true
    properties: Platform="Platform with spaces"

In order to be working, this should be passed in the command as
jb inspectcode ... --properties:Platform="Platform with spaces"
However it results in:
jb inspectcode ... "--properties:'Platform=Platform with spaces'
The double quotes are removed from the value, instead wrapping the full argument. The single quote is now being interpreted as part of the Platform value, resulting in the error that there is an invalid character in the Platform argument.
The quoting of the passed in parameters should be respected, but somehow it gets changed. I don't know why/where that happens.
I've tried working around it using env vars in the GH Action for example, but I can't get that to work.

It's caused by actions/toolkit and I'm trying.
Wait a while.

@koosvanw
I fixed code and succeeded to pass double quote with following yaml.
properties: 'Platform=\"Any CPU\"'

https://github.com/muno92/resharper_inspectcode/actions/runs/6945237569/job/18894129443
/home/runner/.dotnet/tools/jb inspectcode --output=result.xml --absolute-paths TestSolution/TestSolution.sln --severity=WARNING --build --properties:Platform="Any CPU"

But, this job was failed.
Is it works on your environment?

@muno92
Thanks for your time on this.
If I use your yaml code I'm getting the following output:
jb.exe inspectcode --output=result.xml --absolute-paths "test solution.sln" --no-swea --severity=HINT --no-build "--properties:Platform=\"Any Cpu)\""

I notice that you're running the workflow on an Ubuntu runner, while I'm running on a private Windows runner. I suspect that could be where the difference comes from. Unfortunately I can't run the action on an Ubunty/Linux based runner due to prerequisites which are unavailable on Linux.
Would it be possible to get this working on a Windows based environment?

@koosvanw
I debugged on Windows Runner (windows-latest).
And, maybe, you don't need to use double quote because actions/toolkit wrap quote around the arg which contains space.
I run following job and MSBuild recognized platform Mixed Platforms.
  e2e_test:
    runs-on: windows-latest
    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-dotnet@v3
        with:
          dotnet-version: '8.0'
      - uses: ./
        with:
          solutionPath: TestSolution/TestSolution.sln
          properties: Platform=Mixed Platforms;Configuration=Debug

https://github.com/muno92/resharper_inspectcode/actions/runs/6958995989/job/18935239058

The specified solution configuration "Debug|Mixed Platforms" is invalid.

Could you try this?

@muno92
That works! I didn't try the last time without additional quoting apparently.
This makes the action usable for me, thanks for your time!

I'm glad to hear that!


OnMembersAdded not thrown when entering Web Chat in AzurePortal

GitHub issues should be used for bugs and feature requests. See the Support section to get support related to Bot Framework and Web Chat.
Screenshots

Describe the bug
The official CoreBot project template has a welcome message implemented which should be sent in OnMembersAddedAsync. When using the "Test in Web Chat" feature in the Azure portal this event is not thrown. Reason is that the recipient.id equals the membersAdded.id in the first conversationUpdate message being sent (see attached screen shot).
To Reproduce
Steps to reproduce the behavior:

Go to the Channel registration of your bot in Azure portal
Click on "Test in Web Chat"
You wont see any welcome message sent in OnMembersAddedAsync

Expected behavior
Configured welcome message should be displayed.
Additional context

[Bug]

For reference: This is the reason why the event is not triggered if recipient.id equals the added members id.

So the question is why does web chat sends the bot id as membersAdded in the first conversation update when entering web chat? When i send the first message a correct conversationUpdate is sent with my id as memberAdded. But this makes it complicated to test the real dialog flow.

@nk111 the Bot Framework protocol sends two ConversationUpdates, one for the Bot when it enters the conversation and one for the User after they have sent their first message. This is not governed by WebChat, but built-in to the protocol events themselves itself. As a group we are discussion how to change this behavior for a better user experience, but this is what we have in place today.

@nk111, Web Chat has it's own recommended method for providing a welcome message, which you can take a look at on https://github.com/microsoft/BotFramework-WebChat/tree/master/samples/15.d.backchannel-send-welcome-event, our welcome message sample.

Thanks for the heads up. But this doesn't solve the problem in the Azure
portal. I doubt this implementation sends this action?? Instead it sends
this first somewhat weird conversation update where the the recipient
equals the sender. I'd still call this a bug (on the portal side).
Corina<EMAIL_ADDRESS>schrieb am Fr., 31. Mai 2019, 21:18:

@nk111 https://github.com/nk111, Web Chat has it's own recommended
method for providing a welcome message, which you can take a look at on
https://github.com/microsoft/BotFramework-WebChat/tree/master/samples/15.d.backchannel-send-welcome-event,
our welcome message sample.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/BotFramework-WebChat/issues/2049?email_source=notifications&email_token=AABF5OVIXKIHBYHVSZPAWV3PYF2ZPA5CNFSM4HRZNY72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWWETGA#issuecomment-497830296,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABF5OTAX6YJJIHYTFI24ZLPYF2ZPANCNFSM4HRZNY7Q
.

@nk111, unfortunately these scenarios follow current protocol and there are no changes imminent.
If you want to add a suggestion on how to improve implementation, I recommend filing an issue on https://github.com/Microsoft/BotFramework-Services/issues for ABS or Connector Services, or a DCR on the BotBuilder sdk repo https://github.com/microsoft/botframework-sdk/issues

Hi,
unfortunately I am also facing the same issue. Do you have an resolution?
@nk111

I'm in the same situation... and I don't know what to do.
Works fine on the local machine, but not on the azure server.

@nk111, Web Chat has it's own recommended method for providing a welcome message, which you can take a look at on https://github.com/microsoft/BotFramework-WebChat/tree/master/samples/15.d.backchannel-send-welcome-event, our welcome message sample.

The link is down...

we have documentation on welcome messages. Could you go through these docs and verify your implementation? Please let me know your results.

we have documentation on welcome messages. Could you go through these docs and verify your implementation? Please let me know your results.

Yeah, now works fine, but it's a little bit tricky. I'm currently working on a port from v3 to v4.


KalDbDistributedQueryService.java: Use loops instead of streams

Refactor getSearchNodesToQuery to use loops instead of streams

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Landing page KSM price is different from actual NFT price

Is your feature request related to a problem?
the Newly listed NFTs show a price but if you click on the NFT it then displays a much lower price.  This price difference may cause NFTs to be overlooked and passed over due to a much higher cost than the NFT is actually listed for on the site.
Describe the solution you would like
Sync the prices
Describe alternatives you have considered
none
Screenshots

Are there opened related issues?
No response

Yeah, I can confirm that probably all New Listed NFTs on the https://kodadot.xyz/ landing page that are RMRK2 have the wrong prices.

The prices in detail are correct

I guess it's issue that Singular shows prices calculated with their fee and not real on-chain prices?


Docker Added

No longer working due to today's source-code release. We plan on releasing docker support ourselves eventually.


Map not scanning

Whenever i leave the map open for a while it stops scanning randomly.
I have scanned locations on, along with 10 accounts running.
Pretty frequently all the map will turn red, where its usually green.
The status at the top remains saying its still operational, but it stops scanning completely.
This happens once every hour or so, and i can fix it by logging out and restarting it.
Does anyone have a fix for this?
Thanks!
Coolykid

Lower number of accounts or increase range. DB running over itself. I run a 10 range with around 10 accounts and it works. If I go above those 10 accounts same problem. Issue with db thread

I was running 10 with 7 range, ill try now and check if 10 works, thanks

Nope, same thing happened. About an hour in it just stops and quits. it still says operational but the squares are all turning yellow and red

once the scan circles are going red
open developer tools and go to console
we might see the problem there
I think this might also be linked to 50 db threads

Will do when it happens again

What helped for me, is that I stopped scanning for pokestops and gyms.
I did this by editing the config.ini file, which for me is located at C:\Users\MyName\AppData\Local\pokemon\app-0.3.4\resources\app\map\config.
I removed the # before 'no-gyms' and 'no-pokestops and typed 'true' after the :.
The config.ini file has to be edited in Notepad++.

https://gyazo.com/b55e47f3d53e29385a3e0ec1c601ecfb @Burbankian a picture of the console
It keeps saying "database is locked"

I'm not sure about the "db is locked" issue
but you can solve the "db queue > 50" by disabling pokestop and/or gym detection in config.ini
you can also increase your db threads instead via command line (I think), but I don't know how to, thats why I settled with disabling pokestop/gym detection


[BEAM-6144] Add support for the autoscaling flags to the GO SDK.

Add support for the autoscaling flags to the GO SDK.
--autoscaling_algorithm
--max_num_workers
Inline with what's documented here: https://cloud.google.com/dataflow/docs/guides/specifying-exec-params
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

Please take another look.

I didn't see this until today. Thanks for the contribution!
LGTM.
R: @aaltay Please merge.

R: @aaltay I typo'd a function in a previous commit, breaking top.  I've filed https://issues.apache.org/jira/browse/BEAM-6682 to improve this.


Does anyone has some pre-trained model which we can download it?

My computer is not very powerful and I don't have the resources available in my university, can someone or could the readme.md give us a link where we can download a pre-trained model?

@pfeodrippe please find attached:
wavenet_models.zip
Training information:
GPU used - Titan X last year's model (12 GB VRAM)
Runtime  - Not measured, but less than an hour.
BATCH_SIZE = 1
Trained on - 'VCTK-Corpus'
NUM_STEPS = 2000
LEARNING_RATE = 0.03
"filter_width": 2,
"quantization_steps": 256,
"sample_rate": 16000,
"dilations": [1, 2, 4, 8, 16, 32, 64, 128, 256, 512, 1, 2, 4, 8, 16, 32, 64, 128, 256, 512],
"residual_channels": 32,
"dilation_channels":16

@adroit91 were you able to generate an output with this? if yes could you also share the generated file

No, same issue as you. Still trying to trace it.

@adroit91 Thanks, man! Now let's attack the generating problme

First sounds with the current default settings! :)
generated_18Sep_1714.wav.zip

It would be nice to have these models and examples in a centralized place, like the repo wiki or a Releases page.

Yes, It'd be fine to have a centralized repository where to upload all examples.

How much was loss?

I have tried to compile the generated audio in a wiki:
Generated audio samples wiki
We can observe how the training is converting gibberish into natural sounding voice.

@adroit91 Could you share the different models or at least the last one? Is it the same zip you sent before?
Great work! I'll listen to it after my morning classes

@lemonzi

It would be nice to have these models and examples in a centralized place, like the repo wiki or a Releases page.

Are you referring to just the model description, or do you mean the trained model's checkpoint file too?
Github doesn't want to host large binary files (if I recall the TOS).  I was thinking about getting a dropbox or some such account to put the ckpt file on, but something that anyone could contribute to would better. Is there some ready solution for this of which I'm just ignorant?

@jyegerlehner We may post large binaries as 'releases' which github supports. :)

Thanks @nakosung that's complete news to me. I'm reading about github releases now :)

@jyegerlehner I meant both! Releases would be great for an "official" checkpoint with the default model settings and a decent number of iterations, but are not world-editable.

@adroit91 , I have limited computation resources, so I run this experiment with currently default config on only one person's waves (p225, about 231 waves).
I have run experiment by 35K steps, but the loss still more than 2, and generated waves are noisy.
Is there any issue about my experiment?
Thanks very much.

@weixsong Try decreasing silence threshold to 0.1-0.2 or lower. Yet be careful, with too low value it may start generating mostly silence. Also decreasing learning rate by 1-2 orders of magnitude might help.
After lowering silence threshold your loss would drop to circa 2. I'm not sure, but it looks like quality of generated sounds and loss are not directly related. Waiting for 10-20k more steps might help even if loss stays on relatively same level.

@lemonzi, is this https://soundcloud.com/user-731806733/tensorflow-wavenet-500-msec-88k-train-steps
wave generated by latest code that support global condition, people ID?,

@akademi4eg , thanks very much for your help.
Know i could generated some sound that sounds make sense.

@weixsong No I think that one was trained on only audio from the single speaker. That model preceded conditioning on speaker id.

@jyegerlehner , thanks very much. Do u have any wave that generated by model trained with latest code? I mean some wave generated with people ID as global condition.

I am a complete newbie here. Can there be an implementation of the wavenet trained model in C#/.NET ?
I am using System.Speech.Recognition module of C#, which is misinterpreting the speech provided with something else (Eg. Hello = Well oh )

@jyegerlehner did you find a way to upload the trained model's checkpoint?

@ebadawy sorry that model is long gone. At the time I was ultimately trying to get local conditioning on text characters (i.e. TTS) to work with and didn't succeed. Didn't think the GC stuff was very interesting by itself.
I don't remember it being difficult to train;  the PR has some discussion and links to samples I generated for different speakers. If I recall correctly I had a machine with a Titan GPU training for a few weeks to produce those samples so shouldn't be too hard to reproduce that if someone really wanted to.
Also, aren't there some pretty good open source implementations of TTS models (e.g. tacotron?) out there now, so I'd guess this wavenet implementation is not very interesting to most.

First sounds with the current default settings! :)
generated_18Sep_1714.wav.zip

I wonder what is the point of this sample it sounds just like a noise. In original demo page they provide samples with voice https://deepmind.com/blog/wavenet-generative-model-raw-audio/

In issue #307 @DiyuanLu has provided a pre-trained model for 12000 steps.


ULP source fails to compile when including both GPIO and I2C headers (IDFGH-13547)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.1.1
Operating System used.
Linux
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
None
What is the expected behavior?
I expect the main.c file for a ULP project to compile when including ulp_riscv_i2c.h and ulp_riscv_gpio.h.
What is the actual behavior?
Many redefinition errors around various typedefs.
Steps to reproduce.
I've included a repo using VSCode and devcontainers for easy reproduction of this issue. Pull the repo and checkout the redefinition_bug branch.

Launch the project in VSCode
Reopen project in container development
Run idf.py build from the command line in VSCode

Build or installation Logs.
ections -MD -MT CMakeFiles/ulp_main.dir/workspaces/rtc_i2c_bugs/main/ulp/main.c.obj -MF CMakeFiles/ulp_main.dir/workspaces/rtc_i2c_bugs/main/ulp/main.c.obj.d -o CMakeFiles/ulp_main.dir/workspaces/rtc_i2c_bugs/main/ulp/main.c.obj -c /workspaces/rtc_i2c_bugs/main/ulp/main.c
In file included from /workspaces/rtc_i2c_bugs/main/ulp/main.c:7:
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:18:5: error: redeclaration of enumerator 'GPIO_NUM_0'
   18 |     GPIO_NUM_0 = 0,     /*!< GPIO0, input and output */
      |     ^~~~~~~~~~
In file included from /home/dev/esp/idf/components/ulp/ulp_riscv/include/ulp_riscv_i2c.h:15,
                 from /workspaces/rtc_i2c_bugs/main/ulp/main.c:6:
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:169:5: note: previous definition of 'GPIO_NUM_0' with type 'enum <anonymous>'
  169 |     GPIO_NUM_0 = 0,     /*!< GPIO0, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:19:5: error: redeclaration of enumerator 'GPIO_NUM_1'
   19 |     GPIO_NUM_1 = 1,     /*!< GPIO1, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:170:5: note: previous definition of 'GPIO_NUM_1' with type 'enum <anonymous>'
  170 |     GPIO_NUM_1 = 1,     /*!< GPIO1, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:20:5: error: redeclaration of enumerator 'GPIO_NUM_2'
   20 |     GPIO_NUM_2 = 2,     /*!< GPIO2, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:171:5: note: previous definition of 'GPIO_NUM_2' with type 'enum <anonymous>'
  171 |     GPIO_NUM_2 = 2,     /*!< GPIO2, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:21:5: error: redeclaration of enumerator 'GPIO_NUM_3'
   21 |     GPIO_NUM_3 = 3,     /*!< GPIO3, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:172:5: note: previous definition of 'GPIO_NUM_3' with type 'enum <anonymous>'
  172 |     GPIO_NUM_3 = 3,     /*!< GPIO3, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:22:5: error: redeclaration of enumerator 'GPIO_NUM_4'
   22 |     GPIO_NUM_4 = 4,     /*!< GPIO4, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:173:5: note: previous definition of 'GPIO_NUM_4' with type 'enum <anonymous>'
  173 |     GPIO_NUM_4 = 4,     /*!< GPIO4, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:23:5: error: redeclaration of enumerator 'GPIO_NUM_5'
   23 |     GPIO_NUM_5 = 5,     /*!< GPIO5, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:174:5: note: previous definition of 'GPIO_NUM_5' with type 'enum <anonymous>'
  174 |     GPIO_NUM_5 = 5,     /*!< GPIO5, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:24:5: error: redeclaration of enumerator 'GPIO_NUM_6'
   24 |     GPIO_NUM_6 = 6,     /*!< GPIO6, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:175:5: note: previous definition of 'GPIO_NUM_6' with type 'enum <anonymous>'
  175 |     GPIO_NUM_6 = 6,     /*!< GPIO6, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:25:5: error: redeclaration of enumerator 'GPIO_NUM_7'
   25 |     GPIO_NUM_7 = 7,     /*!< GPIO7, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:176:5: note: previous definition of 'GPIO_NUM_7' with type 'enum <anonymous>'
  176 |     GPIO_NUM_7 = 7,     /*!< GPIO7, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:26:5: error: redeclaration of enumerator 'GPIO_NUM_8'
   26 |     GPIO_NUM_8 = 8,     /*!< GPIO8, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:177:5: note: previous definition of 'GPIO_NUM_8' with type 'enum <anonymous>'
  177 |     GPIO_NUM_8 = 8,     /*!< GPIO8, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:27:5: error: redeclaration of enumerator 'GPIO_NUM_9'
   27 |     GPIO_NUM_9 = 9,     /*!< GPIO9, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:178:5: note: previous definition of 'GPIO_NUM_9' with type 'enum <anonymous>'
  178 |     GPIO_NUM_9 = 9,     /*!< GPIO9, input and output */
      |     ^~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:28:5: error: redeclaration of enumerator 'GPIO_NUM_10'
   28 |     GPIO_NUM_10 = 10,   /*!< GPIO10, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:179:5: note: previous definition of 'GPIO_NUM_10' with type 'enum <anonymous>'
  179 |     GPIO_NUM_10 = 10,   /*!< GPIO10, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:29:5: error: redeclaration of enumerator 'GPIO_NUM_11'
   29 |     GPIO_NUM_11 = 11,   /*!< GPIO11, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:180:5: note: previous definition of 'GPIO_NUM_11' with type 'enum <anonymous>'
  180 |     GPIO_NUM_11 = 11,   /*!< GPIO11, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:30:5: error: redeclaration of enumerator 'GPIO_NUM_12'
   30 |     GPIO_NUM_12 = 12,   /*!< GPIO12, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:181:5: note: previous definition of 'GPIO_NUM_12' with type 'enum <anonymous>'
  181 |     GPIO_NUM_12 = 12,   /*!< GPIO12, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:31:5: error: redeclaration of enumerator 'GPIO_NUM_13'
   31 |     GPIO_NUM_13 = 13,   /*!< GPIO13, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:182:5: note: previous definition of 'GPIO_NUM_13' with type 'enum <anonymous>'
  182 |     GPIO_NUM_13 = 13,   /*!< GPIO13, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:32:5: error: redeclaration of enumerator 'GPIO_NUM_14'
   32 |     GPIO_NUM_14 = 14,   /*!< GPIO14, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:183:5: note: previous definition of 'GPIO_NUM_14' with type 'enum <anonymous>'
  183 |     GPIO_NUM_14 = 14,   /*!< GPIO14, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:33:5: error: redeclaration of enumerator 'GPIO_NUM_15'
   33 |     GPIO_NUM_15 = 15,   /*!< GPIO15, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:184:5: note: previous definition of 'GPIO_NUM_15' with type 'enum <anonymous>'
  184 |     GPIO_NUM_15 = 15,   /*!< GPIO15, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:34:5: error: redeclaration of enumerator 'GPIO_NUM_16'
   34 |     GPIO_NUM_16 = 16,   /*!< GPIO16, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:185:5: note: previous definition of 'GPIO_NUM_16' with type 'enum <anonymous>'
  185 |     GPIO_NUM_16 = 16,   /*!< GPIO16, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:35:5: error: redeclaration of enumerator 'GPIO_NUM_17'
   35 |     GPIO_NUM_17 = 17,   /*!< GPIO17, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:186:5: note: previous definition of 'GPIO_NUM_17' with type 'enum <anonymous>'
  186 |     GPIO_NUM_17 = 17,   /*!< GPIO17, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:36:5: error: redeclaration of enumerator 'GPIO_NUM_18'
   36 |     GPIO_NUM_18 = 18,   /*!< GPIO18, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:187:5: note: previous definition of 'GPIO_NUM_18' with type 'enum <anonymous>'
  187 |     GPIO_NUM_18 = 18,   /*!< GPIO18, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:37:5: error: redeclaration of enumerator 'GPIO_NUM_19'
   37 |     GPIO_NUM_19 = 19,   /*!< GPIO19, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:188:5: note: previous definition of 'GPIO_NUM_19' with type 'enum <anonymous>'
  188 |     GPIO_NUM_19 = 19,   /*!< GPIO19, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:38:5: error: redeclaration of enumerator 'GPIO_NUM_20'
   38 |     GPIO_NUM_20 = 20,
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:189:5: note: previous definition of 'GPIO_NUM_20' with type 'enum <anonymous>'
  189 |     GPIO_NUM_20 = 20,   /*!< GPIO20, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:39:5: error: redeclaration of enumerator 'GPIO_NUM_21'
   39 |     GPIO_NUM_21 = 21,   /*!< GPIO21, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:190:5: note: previous definition of 'GPIO_NUM_21' with type 'enum <anonymous>'
  190 |     GPIO_NUM_21 = 21,   /*!< GPIO21, input and output */
      |     ^~~~~~~~~~~
/home/dev/esp/idf/components/ulp/ulp_riscv/ulp_core/include/ulp_riscv_gpio.h:40:3: error: conflicting types for 'gpio_num_t'; have 'enum <anonymous>'
   40 | } gpio_num_t;
      |   ^~~~~~~~~~
/home/dev/esp/idf/components/hal/include/hal/gpio_types.h:216:3: note: previous declaration of 'gpio_num_t' with type 'gpio_num_t'
  216 | } gpio_num_t;

More Information.
To get the project to successfully build, simply remove the i2c references in the main.c ulp file which includes commenting out the include of the header file as well as the I2C Operations section of main.c
I also tried this in the v5.3 branch of IDF with the same results.

Hi @sidwarkd,
Thanks for reporting this bug and provide the helpful reproduction repo. Our fix will be deleting the redundant gpio_num_t enum in ulp_core/include/ulp_riscv_gpio.h and unify it with the definition in "hal/gpio_types.h". This fix will come soon on IDF v5.3, but may not be backported to IDF v5.1.1.
Thanks,
Xiaoyu

This fix will come soon on IDF v5.3, but may not be backported to IDF v5.1.1.

@louielauanu
I'm surprised you don't backport compile fix for v5.1 branch? (Isn't v5.1 brnach still in maintenance period?)

This fix will come soon on IDF v5.3, but may not be backported to IDF v5.1.1.

@louielauanu I'm surprised you don't backport compile fix for v5.1 branch? (Isn't v5.1 brnach still in maintenance period?)

This fix will be ported to v5.1, but may not on v5.1.1, because it's code freeze. To obtain it, user have to fetch the next v5.1 release version, i.e. v5.1.5.

@louielauanu Thanks for the info. I understand about it being in a bugfix release of 5.1.x. That makes sense. :+1:


Error creating Portaliases

1005:1008 https://docs.opnsense.org/manual/aliases.html?highlight=range

rtfm ... I'm sorry ;-(

@hsiewert no worries, but would you mind keeping the original description intact for others who will find this issue later on? an answer without a question is not as effective. thanks! :)


Can't Create CDS Authentication account through PowerApps CLI

Trying to create a CDS --kind Authentication through PAC and gives me this error
Error: Failed to connect to Dataverse
There was no endpoint listening at https://< organization-URL >.crm4.dynamics.com/XRMServices/2011/Organization.svc/web?SDKClientVersion=<IP_ADDRESS>82 that could accept the message. This is often caused by an incorrect address or SOAP action. See InnerException, if present, for more details.
The remote server returned an error: (407) Proxy Authentication Required.
Our environment is accessing dynamics through a proxy that is installed properly.
I am able to perform other tasks of PAC such as create --kind ADMIN or create a Backup of environment.
In Azure we have an app registration that is pointing at the enterprise application Dynamics CRM and it has user-impersonation with admin consent

@fadyyeta the error to not be able to connect to Dataverse is caused by your local network that requires proxy authN:
 The remote server returned an error: (407) Proxy Authentication Required.
The Dataverse service client that is used by pac CLI has no support for handling local proxy authentication; before trying to connect via pac CLI to your Dataverse environments, ensure that your PC has authenticated to your network's internet proxy, e.g. via browser.
This behavior is currently by design, with no backlog work items to address this

@fadyyeta I have also come across this issue, but when using pac via CLI.  To fix this I added the following to the configuration section of pac.exe.config file :
<system.net>
	<defaultProxy useDefaultCredentials="true" enabled="true"></defaultProxy>
</system.net>

The pac.exe.config for me was located in the Microsoft.PowerApps.CLI.1.9.4\tools subdirectory of the pac installation location.
Hope this helps.


Update paper to hallucination.md

added HallusionBench: An Advanced Diagnostic Suite for Entangled Language Hallucination and Visual Illusion in Large Vision-Language Models -- https://github.com/tianyi-lab/HallusionBench

Hi! Great work! We will add it in our next update.


Implement issue index synonym support

The Nest index engine supports built-in synonyms to enhance search results. To improve the user experience, we can map related terms (either one-way or two-way). For example, a search for "mern" could return issues related to JavaScript, TypeScript, React, and Node.js.
This task is focused on setting up basic synonym support for the issue index.

/assign


Can you confirm that the license for v1.0 is also MIT?

The master branch is MIT (https://github.com/webpack/tapable/blob/master/LICENSE)
v1.1 is MIT (https://github.com/webpack/tapable/blob/v1.1.0/LICENSE)

For some reason, v1.0 goes to "page not found" (https://github.com/webpack/tapable/tree/v1.0.0/LICENSE)

The LICENSE file wasn't added until v1.1.0. But the package.json in v1.0.0 does show MIT in its license field.
https://github.com/webpack/tapable/blob/v1.0.0/package.json#L6

yep it's MIT


CSV is poor file format that is half specd, json/xml is better

How do you do handle \n in csv ?!!

Put quotes around it. Json/xml is on the roadmap.


hasktags: add to all-packages

I had haskellPackages.hasktags installed in my nix profile, which caused some confusion when I switched to the NixOS 16.09 channel where the Hasktags package apparently produces a test executable that can shadow the test program from coreutils.
This PR adds a hasktags entry to all-packages.nix which removes bin/test and cleans up some unnecessary directories as well.
Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I don't think all-packages.nix is the right place to fix this. To me it looks like the test executable shouldn't be in there at all, so it's likely more a packaging issue.
Please don't remove .nix-support or share either. Random things like this cause more trouble than they're worth. (share won't be linked into your nix-profile or system at all except for whitelisted subdirectories. nix-profile won't be linked either)

Please don't remove .nix-support or share either.

Oh? I only did that because I was following the example of other things in all-packages

Putting this into all-packages was also my attempt to follow the example of how it's already done. stack, darcs, cabal2nix, and pandoc use this same pattern. Should those be changed as well?

Any way in mind to do that? I thought all the code in haskell.packages was generated.

You can't modify the hackage-packages.nix file because that's generated, but there's a bunch of other configuration for overrides. https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/haskell-modules/configuration-common.nix

Okay, I've updated the PR accordingly. Now it just adds an override to remove bin/test from hasktags.

@chris-martin FYI: https://github.com/NixOS/nixpkgs/commit/5fae046771a5166430669816a88f7c27dfbecf88 Looks like this was done with darcs to prevent the haskell dependencies from ending up in the propagatedBuildInputs because it's just a binary.

I agree that this should be fixed upstream for future versions, but we do support old stackage lts releases, so why not patch it here?

I'm not sure what you mean? I did apply your patch. It is fixed.

Oh, I'm sorry. I just saw the status of closed instead of merged and I didn't notice the commit. :+1:

Hey all. Sorry, I merged a PR to fix that issue last week and just haven't pushed to update to Hackage. I'll get that done tonight.


CompileError

command 'C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\x86_amd64\cl.exe' failed with exit status 2
What's going on here？

Look for the actual compiler error further up in the output.


Ignore files on exclusion list

It would be fantastic if the script could read a plaintext list of filenames and exclude them from the download. There are always a few wallpapers that are not wanted for whatever reason, and this would prevent having to delete them after every download.
(Thanks for creating a great utility!)

I would love this as well.


Investigate whether there is a point in running our benchmarks with --profile option

It is possible that there is no point in running the benchmarks with --profile option, since, cProfile results might not be portable. We need to verify that and turn off profiling if we cannot use its results.

@pared The data is not useful for the asv itself, hence why they mention abs paths in the doc. But it is still useful for us, that's why we ask people to provide profiler report and we have no trouble viewing it on our own machines.

@efiop
right, closing for now


Correctly check access control for a class's generic superclass

The previous code was too clever in trying to avoid work and missed the fact that ClassDecl::getSuperclass produces an interface type but the types in the inheritance clause are contextual types.
This actually successfully built in non-WMO builds with an internal base class, because the internal class symbol wouldn't get stripped out. However, it's highly likely that trying to import such a library would result in instability (read: compiler and debugger crashes), and it's clearly a mistake to allow this. (If you can't show your superclass to a user in a library's generated interface, something's definitely gone wrong.)
SR-6206 / rdar://problem/35119972

@swift-ci Please test

@swift-ci Please test source compatibility

Oh no, people are actually doing this. :-(

Oh, drat. Stage in via a warning I guess? This is going to cause consistency problems if we don't clamp down on it.

Yeah, a warning is fine.  Just extra hassle.
(Slava's new test cases are good anyway.)

@jrose-apple When you say "people are actually doing this" what do you mean exactly? They are actually declaring public classes with internal superclasses? Or they are doing the typealias thing @slavapestov mentioned (which I admittedly don't fully understand)
Because in the former case, even if they are doing it, it should be generating a linking error when it's actually built for release. That has been my experience at least. So if it's preventing them for building for distribution anyway, then what's wrong with actually throwing an error? Either that or the linking error needs to be fixed.

It's possible the only cases that are happening "in real life" are internal classes with private/fileprivate superclasses, like this one in RxSwift. In that case, the build should fail in non-WMO mode, but would succeed under WMO.
I don't plan to do any work to support this working other than to make it a warning rather than an error in pre-Swift-5 modes. That should be "not a regression".

@swift-ci Please test

@swift-ci Please test source compatibility

@slavapestov Mind taking another look?

@swift-ci Please test

@swift-ci Please test linux

LGTM!


Implement Cw20 reward

This contract bases Anchor/reward distribution and integrates cw20 for stake.
Not ready!

We should probably make a cw-tokens repo for a lot of these examples, to separate from the core ones (cw20, cw20-base, cw20-ics20)

@ethanfrey Indeed

Please move to https://github.com/CosmWasm/cw-tokens
You cannot transfer PR. You may be able to cherry-pick commits if you are a git super-star. Or simply apply the patch. I kept the same contract names and paths.

Moving to https://github.com/CosmWasm/cw-tokens


support non-numeric id in pgslice fill

First of all, I love the project and have been using this for quite some time. For me, the most common use case is to partition a table with a numeric id. But this time I want to partition a table that has a varchar type id.

pgslice fill table_name --batch-size 20000
INSERT INTO "public"."table_name_intermediate" ("id",.......) SELECT "id", ....... WHERE "id" > 11674306 AND "id" <= 11694306 AND "start_time" >= ........................................
PG::UndefinedFunction: ERROR:  operator does not exist: character varying > integer
LINE 4:     WHERE "id_" > 11674306 AND "id_" <= 11694306 AND "start_...

Hey @singals, thanks for the suggestion. I don't think it's common enough to support, but you could write a custom script that handles it. If you do, would be great to share it for others who might need it.


Missing icons in<EMAIL_ADDRESS>

Reproduction
NA
Describe the bug
Some icons are missing from<EMAIL_ADDRESS>  I am missing check, circle, spinner, star, in<EMAIL_ADDRESS>more info:
I am installing via Yarn 3.
Reverting to<EMAIL_ADDRESS>resolves the issue.
The playground for beta-15 does not have the same issue.
When I build formkit locally from the beta.15, it is Not missing the icons.
It appears that something went wrong when publishing to NPM.
Andrew Boyd suggested I create an issue for this:  https://discord.com/channels/793529058377072650/1025447518617735210/1080245549892128828
Environment
• OS: Mac 12.6.3
• Browser: NA - this issue is seen in the files downloaded via NPM
• Version<EMAIL_ADDRESS>

So looking at the actual files on npmjs.org it appears that the icons are all accounted for: https://www.npmjs.com/package/@formkit/icons?activeTab=explore
Possible this was a dependency resolution error? We've had reports of users installing beta.15 and getting beta.9 in their lockfiles for dependencies because the module resolution has a hard time telling that beta.1_ is greater than beta.9.  We're changing our versioning to 0.16.0 starting with the next release to help address this.
If you change your packaged.json to use the latest tag as your preferred version do you get the icons as expected?

I'm closing this, feel free to re-open if it's still an issue.


Krypton Navigator - outlook full Nav Mode

When you try to place krypton navigator in VS IDE, it crashes Visual Studio 2022. Krypton Navigator version: <IP_ADDRESS>. 70 version is working correctly, but get back to it it's a REAL nightmare.

@almendieta
Component factory does not go above V 4.5
Please delete this and move it to the Krpyton-suite github project

Thanks for the answer, I've just move it to Krypton Suite GH project. I can not delete this message, couldn't find a way to do it.


update(readme) Correction for sass-convert options (and find options)…

… to convert from scss to sass as described in the instructions in the readme.

Ah true!

Thank you.  Thank you for sharing your boilerplate! It was very helpful :)


docs: remove 0.1.0  from linkinator ignore list

For #3014

/retest

please wait for https://github.com/envoyproxy/gateway/pull/3017

/retest


Let user specify snapshot name

Right now, Snapshot-Lab uses a hard-coded snapshot name. It might be helpful to be able to specify a name which would also need to be supported with Refresh-Lab.

Made these changes in v4.1.0


Rendering in Jupyter Lab

Describe the bug
When drawing a graph in Jupyter Lab in 'SVG' format, the resulting plot does not cover the whole graph. If I change the format to 'PNG' or 'JPG,' then it plots only low-resolution images. The same behavior occurs when I save the graph to a file.
To Reproduce

from torchview import draw_graph
from torchvision.models import AlexNet
model_graph1 = draw_graph(AlexNet(), input_size=(1,3,224,224), save_graph=True)
model_graph1.visual_graph

Ubuntu 20.01
Firefox, Chrome
graphviz==0.20
torchvision==0.10.1+cu111
torchview==0.2.5
jupyter lab==3.0.0

Additional context
Needed for a teaching purpose.

There is a model_graph.resize_graph(x) method where x scale factor is essentially rescales the size of graph. In your case, it seems that the size of figure has to be bigger to contain the graph, so you can try
x = 2   # > 1
model_graph1.resize_graph(x)

Note that resize_graph might not be idempotent for some cases.

Thanks, now it works!