[BUG] SetInputFilesAsync() allocates extremely large amount of memory

Context:

Playwright Version: 1.35
Operating System: Windows
.NET version: .NET 6 and .NET 7
Browser: Chromium, Firefox, Webkit
Extra: Also occurs with Selenium Grid integration (using environment variable)

Code Snippet
[TestFixture]
public class PlaywrightPOCTest
{
    private static IPlaywright playwright;
    private static IBrowser browser;
    private static IBrowserContext context;
    private static IPage page;

    [OneTimeSetUp]
    public static async Task DownloadBrowsers()
    {
        var exitCode = Microsoft.Playwright.Program.Main(new[] { "install" });
        if (exitCode != 0)
        {
            throw new Exception($"Playwright exited with code {exitCode}");
        }

        playwright = await Playwright.CreateAsync();
    }

    [OneTimeSetUp]
    public static async Task EdgeDriverInitialize()
    {
        browser = await playwright.Chromium.LaunchAsync(
            new BrowserTypeLaunchOptions
            { Channel = "msedge", Headless = true, /*SlowMo = 2000 */}
            );

        context = await browser.NewContextAsync();

        page = await context.NewPageAsync();
    }

    [Test]
    public static async Task PlaywrightFileUpload()
    {
        var frame = page.MainFrame;

        await frame.GotoAsync(@"http://www.fileconvoy.com/");
        Console.WriteLine($"Memory usage before upload: {System.Diagnostics.Process.GetCurrentProcess().PrivateMemorySize64}");
        await frame.Locator("id=upfile_0").SetInputFilesAsync(@"C:\Users\testuser\Downloads\19588.pdf");
        Console.WriteLine($"Memory usage after upload: {System.Diagnostics.Process.GetCurrentProcess().PrivateMemorySize64}");
        await frame.Locator("id=readTermsOfUse").CheckAsync();
        await frame.GetByRole(AriaRole.Button).GetByText("Upload").ClickAsync();
        Console.WriteLine($"Memory usage after page navigation: {System.Diagnostics.Process.GetCurrentProcess().PrivateMemorySize64}");
    }

    [OneTimeTearDown]
    public static async Task EdgeDriverCleanup()
    {
        await context?.CloseAsync();
        await browser?.CloseAsync();
        playwright.Dispose();
    }
}

Describe the bug
Steps
-Download this ~30MB PDF from here: https://research.nhm.org/pdfs/19588/19588.pdf
-Update path in SetInputFilesAsync() method to your file location
-Run PlaywrightFileUpload() test
Expected Results:
I would expect memory usage to increase temporarily by the uploaded filesize (~30MB) and then go away or be cleaned up fairly soon after page navigation occurs.
Actual Results:
Memory usage jumps from about 40MB to >700MB just from the file upload and persists for quite a long time in other tests which do not involve file uploads. Sometimes I can get it to drop by forcing a GC using GC.Collect() several minutes later, but I definitely should not be doing that.
This has had an impact on long running scenarios where we are doing a file upload as the first dependency and then running several tests on a web app after the upload. It also affects concurrent runs in our framework due to out of memory issues. Fwiw, I took a memory snapshot inside VS, and it shows up as a large number of dead char[] objects which are outside of my code.

An update after more code research...
If I use a PDF that is above the 50MB filesize limit like with a 60MB PDF, it skips saving a base64 copy to a buffer which is where the memory leak occurs. I have tested the >50MB limit with both codepaths, local and remote (Selenium Grid), and the memory  issue did not occur as well as being faster since it skips the byte[] conversion to base64. Memory jumped to 55MB around the upload and moved back to ~35MB after ClickAsync() to navigate/submit the form.
Was there a reason for this 50MB limit? Maybe it makes since more around 1 or 2MB?

I was able to reproduce. Memory went up to 2 GB for me

Closing for now and waiting for more user reports! Since on macOS it seems fine and Node.js seems to collect memory after a few seconds.


Authorization header missing in request payload from swagger UI

Problem : Authorization header is missing in request payload from swagger UI
django-rest-swagger==2.2.0
djangorestframework==3.8.2

schema_view = get_swagger_view(title='Pastebin API')

urlpatterns = [
    url(r'docs', schema_view),
]

SWAGGER_SETTINGS = {
    'SECURITY_DEFINITIONS': {
        'api_key': {
            'type': 'apiKey',
            'in': 'header',
            'name': 'Authorization'
        }
    },
    'JSON_EDITOR': True,
}

I see the authorize button immediately in the swagger UI
api_key  (apiKey)
Name: Authorization
In: header
Value:
(textbox)
I typed my token and authorized.
Now I would expect all my requests from swagger getting attached with an Authorization header with the value I provided. But in the request payload, I don't see any header attached. Is there anything that I've missed? I have also gone through many other issues (related to this) and couldn't find a working solution.

Hi!
I've faced with the same issue and found this note https://github.com/marcgibbons/django-rest-swagger/issues/757#issuecomment-397527350.
Looks like it works now for django-rest-swagger==2.1.2 only.

Can somebody tell me whether we can pass authorization headers in django-rest-swagger==2.2.0

Nope. Use 2.1.2

Hi everyone, have you enabled CORS on your servers?
If not, take a look at https://enable-cors.org. In order for your browser to be allowed to send Authorization headers, your server needs to be watching for OPTIONS requests and sending back Access-Control-Allow-Headers: *. This is a security mechanism enforced by your browser, so there's no way for Swagger UI (or any web app) to get around it.

Also - sharing your Swagger JSON/YAML would be helpful here.

It's definitely not a CORS issue. It works perfectly in 2.1.2. Just not in 2.2.0
There will be no JSON/YAML file generated when using django-rest-swagger

Hi !
@marcgibbons it seems something related with SECURITY settings.
I think this is a required settings from Swagger UI OpenApi, along side with SECURITY_SETTINGS
After some investigation i notice that is something already merged in previous release
refers to this PR #636
something went wrong because in 2.2.0 CHANGELOG
https://github.com/marcgibbons/django-rest-swagger/blob/master/CHANGELOG.md

Add security #636 (dimitrismakris)

it seems that it should be included ... but it's not.
After include this PR it works.
For all that want to try the fix, please pay attention to SECURITY settings format
from openapi 2 spec it's an array :-D
example
'SECURITY': [
        {
            'api_key' : []
        }
    ]

Wow in that case just include that msising pull request and plz release a version. This is very much needed. @marcgibbons


Attendance page stuck on spinner when all group participants have graduated [4h]

After digging into this, found that root cause is at logic for generating data for editable session.
When we have participants (even when all of them have graduated), we expect the data to have at least one person loaded. However, in this case, there're none.

@adamhstewart
Haven't heard of this problem for 6 months.  I think we can close this issue.


Documentation of webpack plugin

I can't use make to generate the manual.html file, looks like my source-highlight not configured correctly, and I don't know how to make it works.
source-highlight: missing feature: language inference requires input file
source-highlight: could not find a language definition for application/x-json
asciidoc: WARNING: manual.txt: line 504: filter non-zero exit code: source-highlight -f xhtml -s application/x-json: returned 1
asciidoc: WARNING: manual.txt: line 504: no output from filter: source-highlight -f xhtml -s application/x-json

I've noticed there're some break change in enhanced-resolve v3.0.0, I think it's better to wait for webpack use it instead of upgrade too early.

Done: c752a24


Possible typo in style helper

https://github.com/ryansolid/dom-expressions/blob/ad095a10233455439bb72625a4f889b67990eedf/packages/dom-expressions/src/client.js#L97-L101
In various places in the helper you use prev[s], suggesting that the key of prev should be a css property name. However, in line 100 you're using the property value as key. To me, that looks like a typo...

Yep completely.. I can see why this one got through since missing from previous just means extra work, but still works. Thanks.


The encoding (CMap) specified by a font is missing for copied PDF pages from this specific PDF

When merging this specific PDF using the copy page methods discussed in #134, it ends up garbling all text, and giving the following error code in Adobe Reader: "The encoding (CMap) specified by a font is missing."
This happens both in the branch 1.0.0 and the 0.x.x branch.
The input PDF before copying passes qpdf --check with no errors, and is this file:
a6d6d730-a2ec-11e9-9500-7b856429f3db.pdf
The output PDF also passes qpdf, but contains no fonts for the pages from the input PDF:
2019-01-30-Tenåsen Regnskap-1644.00-NOK.pdf
Interestingly enough the output PDF is fully readable in Google Chrome, but broken in Adobe Reader and Mac OS Preview.
Might be fixed for my use case with #121 , but unsure what's going on.

Hello @DanielJackson-Oslo!
I just released a fix for this to the v1.0.0 beta branch (it's the latest @beta tag). You can install it with npm:
npm install --save<EMAIL_ADDRESS>
or yarn:
yarn add<EMAIL_ADDRESS>
It's also available on the unpkg CDN:

<EMAIL_ADDRESS><EMAIL_ADDRESS>
The issue was a slight oversight in the decode function for PDFNames. Only the 0-9 hex digits were being decoded. The digits A-F were not being decoded. The fix was just to add these digits to the regex: https://github.com/Hopding/pdf-lib/commit/d123b59e72b6c63567887b70934f5720d249938a.
I'll release a fix to the non-beta version (0.6.4) as well.

I just released the fix to 0.6.4-rc2. You can install it with npm:
npm install --save<EMAIL_ADDRESS>
or yarn:
yarn add<EMAIL_ADDRESS>
It's also available on the unpkg CDN:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

You're the best @Hopding . Will test as soon as I get a chance to.

@Hopding The<EMAIL_ADDRESS>build gives the following error:
Error: Cannot find module '@pdf-lib/fontkit'
The build works fine in -beta.1. I've flushed my node_modules folder and package-lock.json and tried it again, with no luck.
I cannot for the life of me understand how that error can come from this https://github.com/Hopding/pdf-lib/commit/d123b59e72b6c63567887b70934f5720d249938a though...

Are you embedding custom fonts? If so, you have to install Fontkit separately in v1.0.0: https://github.com/Hopding/pdf-lib/tree/Rewrite#fontkit-installation.

Ah, the problem is that for some reason<EMAIL_ADDRESS>does not in fact load<EMAIL_ADDRESS>The fix in d123b59 is not in the code in node_modules, so god knows which version it actually is.
The sha is: sha512-godVXwb4UIVBn7Z/ZGCPgOupyJgW6L+LRsysxbPP8eE2CQZ8y1xfNHWHa8nIkJACg0SgB5Y7d2336GmHw3mEDA==
Do you get the fix with<EMAIL_ADDRESS>on your end?
My package.json looks like this:
		"pdf-lib": "^1.0.0-beta.2",
and my package-lock like this:

		"pdf-lib": {
			"version": "1.0.0-beta.2",
			"resolved": "https://registry.npmjs.org/pdf-lib/-/pdf-lib-1.0.0-beta.2.tgz",
			"integrity": "sha512-godVXwb4UIVBn7Z/ZGCPgOupyJgW6L+LRsysxbPP8eE2CQZ8y1xfNHWHa8nIkJACg0SgB5Y7d2336GmHw3mEDA==",
			"requires": {
				"@pdf-lib/standard-fonts": "^0.0.4",
				"pako": "^1.0.10",
				"png-ts": "^0.0.3",
				"tslib": "^1.10.0"
			},
			"dependencies": {
				"tslib": {
					"version": "1.10.0",
					"resolved": "https://registry.npmjs.org/tslib/-/tslib-1.10.0.tgz",
					"integrity": "sha512-qOebF53frne81cf0S9B41ByenJ3/IuH8yJKngAX35CmiZySA0khhkovshKK+jGCaMnVomla7gVlIcc3EvKPbTQ=="
				}
			}
		},

(Not embedding any fonts)

Cloning and building the Rewrite branch works, and with it the fix also works 🎉  However, installing from npm does not work but gives a broken branch like described above :/

Looks like I forgot a prepublish step for the 1.0.0-beta.2 release. I just fixed this and released 1.0.0-beta.3. Please let me know if this one works!

This one works! Thank you @Hopding . Closing the issue. 🎉

I just released version 0.6.4. The full release notes are available here.
You can install it with npm:
npm install --save<EMAIL_ADDRESS>
or yarn:
yarn add<EMAIL_ADDRESS>
It's also available on the unpkg CDN:

<EMAIL_ADDRESS><EMAIL_ADDRESS>


add NTP access prerequisite

old doc work item: HPC Cache - Add documentation to allow outbound NTP traffic (8029174)

@femila Could you review this pull request? The article ownership can't be changed in the public repo; we'll need to either back out that change or move the commit to the private repo for publishing. #hold-off
Note that publishing in this repository is suspended until Tuesday, November 2. Merging will resume Tuesday morning. Authors can still review and sign off on PRs in the interim. Thanks.

I see that resolving the merge conflict will replace the author with @femila, so I'll resolve that conflict in the GitHub UI now.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Configure Kotlin support in Moshi

Without this Moshi will incorrectly decode null into non-nullable
fields which causes all kinds of problems.

Moshi started out as a Java library and this isn't strictly backwards-compatible due to differences in nullability between Java and Kotlin.
In Moshi 1.8 this is opt-in. In Moshi 1.9 we refuse to handle Kotlin types without being opted-in, so you can't really forget it.


Wrong sha256 sum for addons

Problem

Hash mismatch for addons

Solution

Correction of sha256 for addons in manifest.toml

PR Status

[ ] Code finished and ready to be reviewed/tested
[ ] The fix/enhancement were manually tested (if applicable)

Automatic tests
Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

Checking if https://github.com/YunoHost-Apps/streams_ynh/issues/21 is solved
!testme

Wokay, something probably changed in the controls during install. The error shows up during the initial install of versions that have the wrong sha256 sum. I'll have to remove the upgrade tests here.

Let's try this again.
!testme


ENH CI setup and mesa builds

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge/core this one is still WIP but comments welcome!

ok @conda-forge/core @conda-forge/help-cdts This PR is ready for review!

cc @mingwandroid

Very nice!

Thanks for the comments @jakirkham! I made two issues for the ones I'd to address later.

any more comments here @conda-forge/core ?

Thanks Matt! 😀


Using argo tunnel + loadbalancer with istio

I want to use the cloudflare loadbalancer with istio + argo tunnel, but I am not sure if this is possible, my current setup is Cloudflare DNS + SSL -> NLB (AWS) then origin certificate termination in Envoy.
I would love to get rid of NLB from AWS and use a cloudflare load balancer instead.
Do you guys have any ideas or examples?

Hey did you figure something out?

According to https://github.com/cloudflare/cloudflare-ingress-controller/issues/172#issuecomment-541230988 this project is dead, so the recommended setup I would think is:

Cloudflare Load Balancer
cloudflared sidecar that connects to your Cloudflare LB with argo tunnel
Traffic forwarded from cloudflared sidecar to Envoy for termination and further processing


Completion of Button and Icon

Rendering for Tag is a bit messy at the moment because Button and Icon are a bit lacking.

Some classes are missing. Icon looks troublesome because they are handled by a macro.
See relevant page in blueprint's doc. The remove buttons on tags don't have bp3-button class and cross icons are wrapped in a span with class bp3-icon-small-cross so I imagine each icon has its own class.

https://github.com/palantir/blueprint/blob/develop/packages/core/src/components/tag/tag.tsx#L152-L159
I think the problem is that you used the <Button> component of yewprint instead of the <button> html component.
See in the code here they don't use blueprint's  either


Fix link handling

egui now returns OpenUrl (since 0.11) instead of the raw url.
This implementation doesn't handle new_tab, the webbrowser crate doesn't support a wasm environment anyway.

LGTM


Rotting system exceptions

Seems to be if a body in a morgue gets deleted the flies try to reparent for reasons.
Looking at the code it just seems like a hack around ambient sounds so you're really just better off changing it to use that.

Also this one:


Receive following Error on Asp.net Core 5

TypeLoadException: Method 'Translate' in type 'EntityFrameworkCore.SqlServer.JsonExtention.JsonTranslator' from assembly 'EntityFrameworkCore.SqlServer.JsonExtention, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have an implementation.

@christopher192 I update project to .Net 6. I pass .Net 5 because of it is not long term support. After .Net 6, entity framework has native json support.


Upstream documentation issue: AutoScalingGroupNamesType as used in DescribeAutoScalingGroups API call

I understand that service-2.json files are auto-generated, so I haven't opened a PR here.
The issue is that a DescribeAutoScalingGroups call with more than 50 group names provided in the AutoScalingGroupNames list results in an exception:
botocore.exceptions.ClientError: An error occurred (ValidationError) when calling the DescribeAutoScalingGroups operation: The number of group names that may be passed in is limited to 50
Ideally, this should be documented as a restriction.

Thanks for the feedback! Looks like it isn't in the service api docs either. I've forwarded this on to the service team.


UI freezes when skipping tracks

OS: Windows 10 Update 1607
Issue Description: The UI completely freezes when skipping from one track to the next by any mechanism, either automatic or manual. Media keys also fail to function. This occurs when the player is launched with the --disable-gpu option as well (although I'm not sure whether that has an effect on Windows). The only solution is to kill the program via task manager.
Steps to Reproduce: Launch the player, skip a song.

This was happened to me with 3.6 version, today I upgraded to 4.0 and all is working perfect.
Try to update GPMDP 👍

I can't replicate this issue. Can you download and run these commands from Command Prompt and tell me the output.
set ELECTRON_ENABLE_LOGGING=true
set ELECTRON_ENABLE_STACK_DUMPING=true
%LOCALAPPDATA%\GPMDP_3\app-4.0.0\Google%20Play%20Music%20Desktop%20Player.exe --dev

@MiguelCTorres I've reproduced it on both versions.
@MarshallOfSound It seems to have spontaneously resolved itself since this morning, after declaring it was loading the music library after the most recent launch. (The computer wasn't restarted or otherwise suspended between then and now, although a restart earlier today didn't resolve it) Not sure if this was caused by a remote issue on Google's end, but I imagine this can be closed for now (I'll leave that decision to you). I'll reopen if it occurs again.

Google's been messing with things in their end for the last week or so.  J had a cool issue yesterday where songs wouldn't play if they had a dash in their name.  Fixed in a few hours... :/
Gonna close this out


remove misleading sentence on from where the API can be called

What type of PR is this?

documentation

Fixes #71

Rather than resolve conflicts (caused by YAML file name change), I incorporated the agreed text into PR #85. Closing this PR.


feat: seed client

https://github.com/starcoinorg/starcoin-explorer/issues/79

#79

What's REACT_APP_SEED_API_URL?

What's REACT_APP_SEED_API_URL?

like https://main-seed.starcoin.org

What's REACT_APP_SEED_API_URL?

like https://main-seed.starcoin.org

so how do you set the ENV for different networks?

@tarvos21 fixed


Fixup build on qt6

I guess this is obsolete by https://github.com/sailfishos/nemo-qml-plugin-time/pull/5


Use project.version for internal dependencies

Codecov Report

Merging #9 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop       #9   +/-   ##
==========================================
  Coverage      60.73%   60.73%
  Complexity        87       87
==========================================
  Files             64       64
  Lines            787      787
  Branches          27       27
==========================================
  Hits             478      478
  Misses           295      295
  Partials          14       14

Continue to review full report at Codecov.

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


netbsd adding few more ptrace queries types.

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit 0e82ece4eb1a4b357c5097563c050a1d5f283507 has been approved by Amanieu

:hourglass: Testing commit 0e82ece4eb1a4b357c5097563c050a1d5f283507 with merge 1725ca0cbb8ccc66171fb2647e6373cc8c5ece2f...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 1725ca0cbb8ccc66171fb2647e6373cc8c5ece2f to master...


32 update pipeline metro map on the readme page

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs- [ ] If necessary, also make a PR on the nf-core/spatialxe branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

For the gene panel json file and all other inputs the color coding is gray but for the output files its white. Also, the legend has an input and output key. I tried getting it on the other side to fix the height, but it does not look very good.


This fails under the latest sunpy...

...needs to be updated to match with the latest API

I haven't checked to see what API changes have affected your code, but let me know if you're having difficulty figuring anything out.
Also, with v0.8, SunPy's coordinate frames can now be transformed back and forth with Astropy's coordinate frames, and there is also a more accurate calculation of the position angle of the Sun (see https://github.com/sunpy/sunpy/pull/2186).  Leveraging the transform code may make your code cleaner, but I'm not positive about that since you want astrometric coordinates rather than apparent coordinates.

It seems to be two separate problems in making maps from NuSTAR data.
One is that mapmeta() no longer exists but I don't think this is a big deal as we can just remove the line
https://github.com/NuSTAR/nustar_pysolar/blob/1e782cc2457eb16a3d2921972a6f21e291a265cb/nustar_pysolar/map.py#L91
And use the dictionary/header directly in making the sunpy map.
The bigger issue is that the info you need to pass to make a working map has changed and not sure what it is we are missing now. I have the same problem with my code to make GOES/SXI maps in sunpy. There are warnings about there being more accurate ways of getting the solar ephemeris in v0.8 but I don't think that's it. I'm guessing something is missing and/or we don't have the units correct.

@ayshih I think since we want to make sure that we also deal with the orbital parallax (which Astropy didn't used to do, but I haven't been following this lately) and make sure that we're doing right coordinate transformation, we'll probably stick with what we've got. I'm getting back into this again, so I'll take a look at what's been going on.

Sorry, I've been distracted, but I finally spent some time looking into this.
Regarding MapMeta, yes, sunpy.map.MapMeta was renamed to sunpy.util.MetaDict, but as you noted, you can just use the dictionary directly.
Regarding making a working map, I don't encounter any errors when running make_sunpy() or when plotting the Map object it creates.  I've tried it on both 0.8.x and the development branch with a random event file from the eclipse.  Maybe there's some library-version dependency that hasn't been captured?  Can you be more specific about the difficulties you're having?
Regarding coordinate transformations, I think you could actually account for orbital parallax using an "observer" construct that we have in SunPy, but that will still be afflicted by the aberration correction (I believe).  That said, there isn't really a reason for you to muck with your code that already works.

Closed in PR #45


http stdlib module

Describe the problem you are trying to solve
Make dealing with http in tremor easier, especially the http connectors: https://www.tremor.rs/docs/edge/reference/connectors/http
Describe the solution you'd like
An http module in the standard library.
It should contain:

a submodule with common header names as constants: std::http::headers.
a submodule with method names as constants: std::http::methods
a submodule with common content-types as constants.
a submodule with common status numbers and messages as constants.

And whatever you think is useful.

Hi @mfelsche , I'd like to work on this issue. Could you brief me more about it ?

Heya,
we try to supply utility functions as part of the tremor standard library.  The goal is to have have http and related constants as part of the standard library. Matthias did detail some of the things that'd be nice to import.
To give an example or two.
instead of using "content-type" as a header it'd be nice to have a constant http::headers::CONTENT_TYPE that expands to it, instead of 200 it'd be nice to have a constant http::status::OK and so on.
we have something similar for google cloud logging: https://github.com/tremor-rs/tremor-runtime/blob/main/tremor-script/lib/google/cloud/logging/severity.tremor

Thanks @Licenser! Yes, having http and related constants as part of the standard library would definitely make dealing with http in Tremor easier. However could you help me knowing where should i add the module in the tremor-runtime?

Heya,
I'd start in https://github.com/tremor-rs/tremor-runtime/tree/main/tremor-script/lib we have google, cncf and some other high levels consts there, so I'd follow the same structures so something like lib/http/methods.tremor and so on :)


Unable to include incidentSeverity.name in workflow

I want to include the name of the incident severity in a workflow message I send to slack. I was able to send the title and description (and even the incident severity entity object) to slack using this in the select field of the "on create incident" component:
{"title": true, "description": true, "incidentSeverity: true}
incident severity shows as [object Object] in the slack message.
In the logs, I see the incidentSeverity object with its _id field.
How can I select the incidentSeverity name and then include it in the slack message (unless its currently not possible)?

Try:
{"title": true, "description": true, "incidentSeverity: {"name": true}}

That worked perfectly, thanks!


pull request for e2e test at 12/06/2016 22:56:03

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


chore: bdd-spring-1583872437 to 0.0.1

chore: Promote bdd-spring-1583872437 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


We cannot find that module webpack/lib/chunk

https://github.com/webpack-contrib/extract-text-webpack-plugin/blob/8de6558e33487e7606e7cd7cb2adc2cccafef272/src/index.js#L3
Which version of webpack would have that module?

@vytch
master: https://github.com/webpack/webpack/blob/master/lib/Chunk.js
3.6.0: https://github.com/webpack/webpack/blob/v3.6.0/lib/Chunk.js
2.7.0: https://github.com/webpack/webpack/blob/v2.7.0/lib/Chunk.js
Please remove package.lock (if exists), remove node_modules and try to run npm cache clean and again npm i, thank you!


Add warning formspec, misc changes

Don't squash and merge! - It's best to keep the commits separate as they do different things. Some commits may be squashed if you think they're too similar however.
Sorry for not making these separate PRs, I'm developing to place directly on my server

Add mute function which will be used in future for variable length mutes.
Add warning formspec on first violation.
Add callback support for custom violation handlers.
See commits

This is now done, just needs testing on my server

$ luacheck .
Checking init.lua                                 OK

Total: 0 warnings / 0 errors in 1 file

I'd like to rewrite the formspec stuff to use fsc if you don't mind, at some point.

I do mind, that doesn't make sense as no data is submitted and just adds unneeded code


Enhancement Request: Export QRDA without a db connection

The MADiE Team is working on our QRDA Export feature for QDM measures and was hoping to use cqm-reports exposed as a web service.
We tried out a PoC and got stuck when we realized its export feature has a runtime dependency on the Bonnie database. Such that, without an active connection to retrieve measures, it will error out.
When an Array of CQM::Measure is passed in as the measures argument into:
Qrda1R5.new(patient, measures, options).render

We get an error originating from qrda-export/helper/view_helper.rb on line 7, with the following message:
undefined method `only' for [#<CQM::Measure ...

Since only comes from the Mongoid library, replacing the Array of CQM::Measure with Mongoid::Critiera results in a successful QRDA render.
This creates a dependency on the QRDA Exporter to have an active connection with Bonnie's db. In order for MADiE to utilize the QRDA Export functionality, we request a feature be added that enables QRDA Export without any database dependency.

@jkotanchik-SB
Can you see if this branch suits your needs
https://github.com/projecttacoma/cqm-reports/tree/measure_objects
This branch is currently based upon another branch (recent_updates) that will merge in recent changes we've made to a cypress specific branch into the master branch.

@dczulada
Thanks for the tip! That branch has been working well for my little proof-of-concept.
I was able to successfully export a QRDA with measure and patient objects, though I did need the mongoid dependency and an active mongo connection to make the QRDA exporter work. I'm guessing that's more to do with cqm-models as having a connection open to my local madie mongo was enough to generate the QRDA export.
I didn't quite follow whether the measure_objects branch will be merged to master or the branch it's based on, recent_updates, would be merged to master. And since it worked well, I'm curious to know, will the option of using measure objects be supported longterm?

@jkotanchik-SB Hearing that the branch worked for you, I will merge recent_updates and measure_objects into master and will tag and publish a gem.

Apologizes for the long delay! I was able to run the export successfully in my PoC using the 4.1.0 Gem. Going to close the gem. Should we encounter any issues during implementation, I'll open another issue.
Thanks again!


[Question]In Nodify.Calculator, the difference between static TabItem and dynamic TabItem in TabControl

Hi Miroiu.
In the TabControl, static TabItem and dynamic TabItem, when the Tab page is switched, all nodes in the dynamic TabItem seem to be initialized and re-rendered, so it will take more time to complete the Tab page switch (the number of nodes in the test is 150/page, and the more nodes, the more obvious the time difference).
Is this a problem with the processing mechanism of TabControl itself? Can the processing logic of dynamic TabItem switching be the same as that of static TabItem?
Thanks for any replies and suggestions.
Static tab item video
https://github.com/user-attachments/assets/575caa62-49b5-4de5-90bb-0353df32fac9
Dynamic tab item video
https://github.com/user-attachments/assets/28594096-452d-443c-be26-ab9a5456f471

I missed some information, the supplement to this question is as follows.
Static tab item code:
shared:TabControlEx
shared:TabControlEx.ItemContainerStyle

</shared:TabControlEx.ItemContainerStyle>
<shared:TabItemEx Header="EditorView 1">
<local:EditorView DataContext="{Binding EditorViewModel1}" />
</shared:TabItemEx>
<shared:TabItemEx Header="EditorView 2">
<local:EditorView DataContext="{Binding EditorViewModel2}" />
</shared:TabItemEx>
</shared:TabControlEx>
Dynamic tab item code:
<shared:TabControlEx ItemsSource="{Binding Editors}"
SelectedItem="{Binding SelectedEditor}"
AddTabCommand="{Binding AddEditorCommand}"
AutoScrollToEnd="{Binding AutoSelectNewEditor}">
shared:TabControlEx.ItemContainerStyle

</shared:TabControlEx.ItemContainerStyle>
</shared:TabControlEx>

I missed some information, the supplement to this question is as follows.
Static tab item code: shared:TabControlEx
shared:TabControlEx.ItemContainerStyle

</shared:TabControlEx.ItemContainerStyle>
<shared:TabItemEx Header="EditorView 1">
<local:EditorView DataContext="{Binding EditorViewModel1}" />
</shared:TabItemEx>
<shared:TabItemEx Header="EditorView 2">
<local:EditorView DataContext="{Binding EditorViewModel2}" />
</shared:TabItemEx>
</shared:TabControlEx>
Dynamic tab item code:        <shared:TabControlEx
AddTabCommand="{Binding AddEditorCommand}"
AutoScrollToEnd="{Binding AutoSelectNewEditor}"
ItemsSource="{Binding Editors}"
SelectedItem="{Binding SelectedEditor}">
shared:TabControlEx.ItemContainerStyle

</shared:TabControlEx.ItemContainerStyle>
</shared:TabControlEx>

Hi @LaiHT1211 , this is a known issue with the default TabControl implementation. When binding to the ItemsSource property, the TabControl discards visuals for unselected tabs, causing them to be rebuilt entirely when re-selected. This can be costly, especially for complex templates.
The linked article provides an excellent explanation and a potential solution: https://groups.google.com/g/wpf-disciples/c/bzUxoXICUt0
Key points from the article:

Unfortunately, if you're using a typical MVVM approach with a binding on the ItemsSource property of the TabControl, the entire tree must be rebuilt each time a tab item is selected.  This is usually a very expensive operation.

The discarding of items is really only a problem when using a TabControl in ItemsSource mode.  In Direct mode, the visuals will still be unloaded from the tree, but they won't be discarded.

However, the performance hit may also be caused by the complexity of the DataTemplate of the node, but that's only for the initial load. Try replacing the DataTemplate with a simple control like this to see the difference:
<DataTemplate DataType="{x:Type local:ExpandoOperationViewModel}">
    <TextBlock Text="This is a node" />
</DataTemplate>

Miroiu,Thanks for your reply.

Miroiu,Thanks for your reply.


the answer about ch08 ex8.12

I have tested in-class initializers as follows:
struct PersonInfo{
	string name = "init";
	vector<string> phones {"10","2"};
};

int main(){
	PersonInfo person;
	cout << person.name;

	system( "pause" );
}

It will cause an error：
![](http://i.imgur.com/x9pRdC1.png error)
But it will be ok if I write as follows:
struct PersonInfo{
	string name = "init";
	vector<int> phones {10,2};
};

I don't know why, can someone else tell me?
In addition, why can't a a aggregate class have in-class initializers.

It seem a bug of Visual studio compiler. The code works well with gcc/clang.
Have a look at : http://cpp.sh/8pm7k.

It seems that Microsoft has fixed this "bug"(I am not quite sure if it is) at the latest version of VS(Visual Studio 2015 with Update 1). It works on my laptop.

Many thanks~ @wtlusvm @pezy


fix: predicate error message

Summary of Changes
We are addressing an issue with proof requests involving multiple credentials. Previously, all proof items were marked in red as errors, even if only one of them was incorrect. Now, we are applying a filter based on the credDefId so that only the credential with the unsatisfied predicate is displayed with an error layout.
Related Issues
N/A
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[X] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[X] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[X] Updated documentation as needed for changed code and new or modified features;
[X] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

@bryce-mcmath I was able to rebase in main.


Unable to Launch Edge

Not able to launch Edge , edge version 94.0.992.47 and driver version are same. But still get the below are the error.
Do we need to use any specific capabilities ?
Caused by: java.lang.RuntimeException: org.openqa.selenium.SessionNotCreatedException: session not created: No matching capabilities found
Build info: version: '3.141.59', revision: 'e82be7d358', time: '2018-11-14T08:17:03'
System info: host: 'LNAR-5CG9322585', ip: '<IP_ADDRESS>', os.name: 'Windows 10', os.arch: 'x86', os.version: '10.0', java.version: '1.8.0_261'
Driver info: driver.version: EdgeDriver

Do we need to use any specific capabilities ?


feat: change FingerprintJsProAgent constructor interface

BREAKING CHANGE:
In previous version FingerprintJsProAgent constructor had several position arguments. In new version it is one params argument of type FingerprintJsProAgentParams. Type FingerprintJsProAgentParams will be used instead of type FingerprintJsProProviderOptions.

This PR is for main, isn't it supposed to be for the test branch? @ilfa

@necipallef I'm trying too keep a bit another process before it fails. (I merge main to test and keep it a little bit cleaner without pre-release commits
)

:tada: This PR is included in version 2.0.0-test.4 :tada:
The release is available on:

npm package (@test dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Request for action: Can you add the action force-detach for the resource EBS

We need to force detach and delete few attached volumes. Can you add force-detach volumes to actions of the resource EBS?
We determine which volumes to be detached and deleted by a whitelist.

its already an option for ebs action delete to force which will detach


Reuse an Utterance for the Description component

In order to prevent the utterance queue from building up tons of Utterances every time there is a component change, the description component should reuse a single Utterance.

This is working pretty well for existing description examples. Ready to merge into main.

This has been merged into main.


[Bug]: Menu location on home page is wrong when updating slug

What happened?
I have created the default location of a restaurant and edited the slug to read default. On the home page, bottom of the page the View Menu has a location of http://mysite.com/local/menus and the Reservations button has a location of  http://mysite.com/default/reservations
This creates an issue where the menu cannot be displayed when the user clicks on the View Menu buttons on the home page.
What did you expect to happen?
The menu should be displayed
Version
3.x
What browser are you seeing the problem on?
Firefox
Relevant log output
No response

This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. If this issue is still relevant or you would like to see it actioned, please respond and we will re-open this issue. If this issue is critical to your business, consider the Dedicated Support Service where a Service Level Agreement is offered.

@hanserasmus have you set the Menu Items Page option under System > Settings > General

Closing this. If you're still experiencing the issue, you can start a discussion on our forums. You may also find solutions to similar issues by searching the forums.


Try out the text-buffer prerelease

atom/text-buffer#303 fixes what looks an awful lot like a bug in TextBuffer, but it could have non-obvious ramifications beyond TextBuffer code. Let's see if the test suite catches anything 🤞
cc @maxbrunsfeld

Green on Windows on VSTS 👍


Move getHeader into Asserter, update Rollup/WebPack plugins

I first developed the Rollup plugin and added this function to it:
https://github.com/kungfooman/RuntimeTypeInspector.js/blob/5f8e1e09a5ac4ee660641e2c0c144e08d69706bb/plugin-rollup/index.mjs#L10-L19
It works, but we need the same code in the other plugins - hence we need to move the code into the transpiler (specifically into Asserter class)
ToDo list:

Move getHeader into Asserter
Update Rollup plugin
Update WebPack plugin

This means the import declarations header will also appear in REPL and for that I already made another PR to support ESM code.

Newly published packages to reflect the changes:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>


Add new Monokai-Dev theme

Please review it :)

Done, to review.

Done, to review.

Done, to review.

thanks


Improve error message when email address is incorrect

Previously npm didn't care if my email address didn't match when authenticating - it only looked at my username. So I had my username set right to "msergeant" and on one of my macs, my email was set to<EMAIL_ADDRESS>but NPM had me listed as<EMAIL_ADDRESS>This was harder to debug what the problem was than it should have been because the error given was:
$ npm publish
npm ERR! Error sending version data
npm ERR! Error: forbidden _npmUser must be a current maintainer: Haraka/1.0.1/-tag/latest
npm ERR!     at Request.callback (/usr/local/lib/node_modules/npm/lib/utils/npm-registry-client/request.js:181:12)
npm ERR!     at Request.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/request/main.js:341:18)
npm ERR!     at Request.emit (events.js:64:17)
npm ERR!     at IncomingMessage.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/request/main.js:309:16)
npm ERR!     at IncomingMessage.emit (events.js:81:20)
npm ERR!     at HTTPParser.onMessageComplete (http.js:133:23)
npm ERR!     at Socket.ondata (http.js:1213:22)
npm ERR!     at Socket._onReadable (net.js:684:27)
npm ERR!     at IOWatcher.onReadable [as callback] (net.js:180:10)

It would be nice if it mentioned the email address instead.

Updated registry to print better error message in this case.


add package plugin

Codecov Report

:exclamation: No coverage uploaded for pull request base (v2.6@777b5c5). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             v2.6      #13   +/-   ##
=======================================
  Coverage        ?   76.49%
  Complexity      ?      151
=======================================
  Files           ?       22
  Lines           ?     1468
  Branches        ?      228
=======================================
  Hits            ?     1123
  Misses          ?      206
  Partials        ?      139

Continue to review full report at Codecov.

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


Showing events in the editor based on the month

Is there an 'out of the box' way to show the events in the event editor only when they are in the month you are currently viewing to avoid a big list?

The event editor isn't part of this library, it exists solely for the demo, so WYSIWYG. You could easily just use the built in angular filter with a function as the expression value to just include the events in the current month.


Add Yandex support

Add Yandex support

Duplicate of https://github.com/iorate/uBlacklist/issues/91

As @Semro pointed out, your issue is a duplicate of #91, which was closed because nobody contributed to it. I didn't (and don't) have the time or motivation to do it either.
Do you plan to contribute to this issue? If so, please implement the Yandex support yourself and create a pull request. See also #63.


Adds Spring java configuration

This is just for the sake of clearing all XML config from our projects ;)

Fixed ;)


responsive design

Description
Enhanced Website Responsiveness and Icon Integration

Improved the responsiveness of the website for better user experience across devices.
Integrated icons using Bootstrap to replace certain text elements, making the interface more visually appealing.

Changes Made

Responsiveness:

Utilized Bootstrap grid system for a fluid layout.
Adjusted CSS for media queries to optimize viewing on mobile devices.

Icons:

Replaced text labels with Bootstrap icons in key areas of the site to enhance visual engagement.
Ensured proper alignment and spacing of icons for a cleaner look.

Screenshots

Include Before and After screenshots side by side to visually demonstrate the changes.

Before:

After:

Benefits

Improved User Experience: The responsive design ensures users can navigate easily on any device.
Visual Appeal: Icons make the website more engaging and easier to understand at a glance.

Additional Notes

Please review the implementation and provide feedback or suggestions for further improvements.

Issue Solved In This PR

#1
#2
#4
#5

@YadavAkhileshh, Sir I am from GSSoC 2024 Extd and this issue has no tags can you please add tags for that issues


Feature: docker metrics

had some issues with loki, needed to reduce the number of labels to get it working again, loki was overloaded. Doing queries in grafana now


Icon Request

Notion - notion.id
today weather - mobi.lockdown.weather
tachij2k - eu.kanade.tachiyomi.j2k
openweather - uk.co.openweather
tick-tick - com.ticktick.task
taskito - com.fenchtose.reflog
kuro - com.deceptions.no
neko - tachiyomi.mangadex
pulse music - com.hardcodecoder.pulse

TickTick is waiting to be merged :) #275

Hello! Weʼre switching to a new system for icon requests in order to be able to better manage them. From now on, icon requests can be submitted here: https://forms.gle/Fx8vZAiWdW1Tyjo57. If your icon request hasnʼt yet been fulfiled, it has been copied to our new icon request database, meaning no action is required. Consequently, this issue will be closed.


Update to next dom-chef

Follows https://github.com/vadimdemedes/dom-chef/pull/62 and https://github.com/vadimdemedes/dom-chef/pull/75

Closing because that PR is inactive


Unable to open private folders

Was unable to see my private folders display when I opened from the widget.  Was the same result from the Keybase app.
my log id: ecc7d9460702cff41b37361c

Restarted keybase and the folders displayed. However, I wasn't challenged to re enter my passphrase.  As a result, unable to open folders.

The latest versions of Keybase don't require your password on a restart, we cache it securely locally until you explicitly log out.
It looks like you can't access your folders because of an excess of traffic hitting our servers.  We're working on it, sorry!

Excess traffic is good, right?
I was able to access my folder this morning from Windows.
On Mon, Sep 26, 2016 at 1:59 AM, Jeremy Stribling<EMAIL_ADDRESS>wrote:

The latest versions of Keybase don't require your password on a restart,
we cache it securely locally until you explicitly log out.
It looks like you can't access your folders because of an excess of
traffic hitting our servers. We're working on it, sorry!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/4419#issuecomment-249429611,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ9TfBdiUCSfy5xiGhNd6m2jE-4deAsZks5qtppvgaJpZM4KF0_n
.

this can be closed


Regression with hipSYCL >= 0.8.2 for hand-rolled tree reduction

Bug summary
We found a bug with hipSYCL > 0.8.2 where our existing hand-rolled tree reduction in CloverLeaf produced the wrong answer.
This was not the case for versions <= 0.8.2 or DPC++ and ComputeCpp.
To Reproduce/Expected behavior
To facilitate testing with other SYCL implementations, I've create a separate repo.
Describe your setup
Programs are compiled using CMake, this bug has only been verified on the CPU.

Thanks for reporting!
I could reproduce the issue, and I think I have found a race condition in your code (see comment in code snippet):
queue.submit([=](sycl::handler& h) mutable {
    cl::sycl::accessor<double, 1, sycl::access::mode::read_write,
                       sycl::access::target::local>
        local(cl::sycl::range<1>(dotWgsize), h);
    auto drain(result.get_access<sycl::access::mode::read_write>(h));
    h.parallel_for<class reduce>(
        sycl::nd_range<1>(dotNumGroups * dotWgsize, dotWgsize),
        [=](sycl::nd_item<1> item) {
          size_t globalId = item.get_global_id(0);
          size_t localId = item.get_local_id(0);
          size_t globalSize = item.get_global_range()[0];

          local[localId] = DOUBLE_MAX;
          for (; globalId < size; globalId += globalSize) {
            local[localId] = sycl::fmin(local[localId], drain[globalId]);
          }
          size_t localSize = item.get_local_range()[0];
          for (size_t offset = localSize / 2; offset > 0; offset /= 2) {
            item.barrier(sycl::access::fence_space::local_space);
            if (localId < offset) {
              local[localId] =
                  sycl::fmin(local[localId], local[localId + offset]);
            }
          }
          if (localId == 0) {
            // This is where I think the problem is: You are writing to memory at indices [0, dotWgsize).
            // Since we are not guaranteed that groups are synchronized between each other,
            // these data elements may be read simultaneously by your first group trying
            // to initialize its local memory buffer at the beginning of the kernel.
            drain[item.get_group(0)] = local[0];
          }
        });
  });

If I change your code such that partial results for group i are stored at  i*dotWgsize (which then guarantees that each group only writes to the memory that it processes itself, and no other group reads), the error no longer reproduces for me.
Note: I have only looked at the non-generic version as the error also reproduced there and it was much easier to understand and follow.

Nice catch!
I wrote that generic reduction back in July 2019 and it's been a source of many bugs for CloverLeaf.
With SYCL2020 officially released, I'll probably use the newer reduction once all SYCL implementations implement it.
I can confirm your fix does indeed resolved the issue for the non-generic version.
However, while back porting the fix to CloverLeaf, which uses the generic version, I've notice basically the same failure.
Digging further, I was able to recreate the failure in the non-generic version:
sycl::buffer<double, 1> result = ...
queue.submit([=](sycl::handler& h) {
    auto source(result.get_access<sycl::access::mode::read>(h));
    auto drain(result.get_access<sycl::access::mode::read_write>(h));
    h.parallel_for<class bar>(sycl::nd_range<1>(42), [=](sycl::nd_item<1> item) {
        // do reduction
        // use drain only, or use both drain and source, doesn't matter
    });
});
// queue.wait();
queue.submit([=](sycl::handler& h) {
    auto source(result.get_access<sycl::access::mode::read>(h));
    auto drain(result.get_access<sycl::access::mode::read_write>(h));
    h.parallel_for<class bar>(sycl::nd_range<1>(42), [=](sycl::nd_item<1> item) {
        // do group reduction
        // use drain only, or use both drain and source, doesn't matter
    });
});
// queue.wait();

The unusual structure (i.e creating accessors but not using them) is an artefact of the generic reduction where I reuse lambdas.
If we then access result  at the end, we get the wrong answer most of the time.
The issues only occurs iff:

The source accessor in the foo and bar kernel are both set to read (any other access mode combination passes verification), whether source is actually used in the kernel is not important.
The absence of queue.wait() or equivalent between the two kernels or at the very end. If we uncomment queue.wait() at either location, the issue goes away.

I've updated the reproducer repo with your fix and the condition described above for both the generic and non-generic version.

That was very helpful, thank you!
I think I know what the issue is: hipSYCL does various optimizations to the DAG, in particular it elides DAG edges between two operations that only read the same memory, such that the operations may be executed in parallel. I think that in your case, this optimization goes overboard and causes a missing dependency between the two kernels.
What should happen is that it detects that there is still a read-write access and therefore still a conflict between the two kernels that requires synchronization. However, we have this innocent looking if statement here:
https://github.com/illuhad/hipSYCL/blob/213c683c4fcd3b107d03a6a866c952f7962b0794/src/runtime/dag_builder.cpp#L60
which prevents the update of the users of a memory buffer for conflict detection if there is already an entry for the node. As a consequence, the node remains registered as a read-only access and the DAG edge is removed.
I suspect that probably it might even work if you swap the declaration order of source and drain in one of the kernels.
What we need to do is remove that if-statement and instead add logic to promote the access mode of existing users if we need stronger guarantees (e.g. read to read-write).
I will verify/come up with a PR tomorrow. Too late already ;)

Fix in PR #466.

Thanks for fix!
Can confirm on my side as well, CloverLeaf also passes verification with the PR.


Clarify help for  Install and Update-MaesterTests

Install-MaesterTests and Update-MaesterTests is expected to be executed in tests-folder or pointed to it using -Path.
https://github.com/maester365/maester/blob/ffe36539a86170c287ebf1414bd97010f14381d9/powershell/public/core/Update-MaesterTests.ps1#L11-L15
The examples for both commands use -Path maester-tests which can be confusing for new users following the docs and cloning the template repo with the same name.
Suggestion: Update help to -Path ./tests to make it more obvious and consistent with the installation guides.

Hi, @fflaten! I'm curious whether you're suggesting a different folder name (tests vs maester-tests) or if you're suggesting the use of a relative path indicator (./) in front of the folder name?

Different folder name in help example. maester-tests is the template repo name. The expected folder for -Path is the parent folder of the custom, maester, eidsa and cisa folders which out of the box is called tests.
The command examples and parameter help doesn't make it clear which folder you expect the path to be.
See step one here which is correct: https://maester.dev/docs/updating-tests#step-1-change-to-the-folder-with-your-tests

You're right, the folder naming instructions are inconsistent. The 'tests' folder really feels redundant to me.
Instructions in the repo state:
md maester-tests
cd maester-tests
Install-MaesterTests -Path .\tests

Instructions at maester.dev state:
md maester-tests
cd maester-tests
Install-MaesterTests

Since the tests folder is never created by any of the above steps, the simplest resolution may be to submit a fix for the README.md in the repository. It should be updated to use Install-MaesterTests -Path .\ or simply Install-MaesterTests like the example from the web site.
Alternatively, you don't need to pre-create the folder if you specify a folder name for the command below:
❯ Install-MaesterTests -Path .\maester-tests
Maester tests installed successfully!
Run Connect-Maester to sign in and then run Invoke-Maester to start testing.

❯ ls

        Directory: C:\Tools\Maester

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
d----          7/9/2024   3:43 PM                
                                                 maester-tests

❯ ls .\maester-tests\

        Directory: C:\Tools\Maester\maester-tests

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
d----          7/9/2024   3:43 PM                  CISA
d----          7/9/2024   3:43 PM                  Custom
d----          7/9/2024   3:43 PM                  EIDSCA
d----          7/9/2024   3:43 PM                  Maester
-a---          7/7/2024   1:36 PM           1528 󰪷  README.md

That was a typo in the readme. I've removed the ./tests in the readme.md
I'm hoping that should resolve the issue.

Tx @fflaten I followed @SamErde suggestion and updated the readme.md in the tests repo with the installation and update instructions.

Thanks. There's still room for improvement IMO.
Both CI guides (https://maester.dev/docs/monitoring/github) import the maester-tests template repo where the test folder is named tests. Fortunately the guides explicitly specifies that update should be done inside maester-tests/tests.
However a user will likely use the command help when updating months later which use maester-tests as the test folder. So there's still a unnecessary name conflict (template repo root vs manual install tests-folder). Maybe the template repo could've been renamed so the included tests folder could follow the manual maester-tests name suggestion?
Consider this a nit and feel free to ignore though. Just a UX opinion :)


Extension Request: convert hex color to name

There's a tool I always use that converts hex/rgba colors to a unique color name:
http://chir.ag/projects/name-that-color/#6195ED
I'd love to have an extension in popclip that can take a hex or rgba string and replace it with the color name! 🎨

Hi @mesqueeb , see #1135 for a suggestion on my end! :-)

Done thanks to @oxxie. Quick download link: https://github.com/pilotmoon/PopClip-Extensions/blob/master/extensions/NameColor.popclipextz?raw=true

@oxxie @invariant it doesn't seem to come with an icon. Is it possible to have it use an icon like other extensions?

The functionality is great though!

@mesqueeb This is certainly possible! Do you have any suggestions of what this icon should look like? Some example pictures would be nice.

Maybe some kind of color picker? Like this one:
<svg xmlns="http://www.w3.org/2000/svg" style="color:rgb(171,146,240);" width="20" height="20" viewBox="0 0 20 20"><g fill="none" fill-rule="evenodd"><path fill="currentColor" d="M7.536 15.236c-1.56.617-4.257 2.371-4.728 1.9-.306-.305.323-1.546.972-2.805h4.66l-.905.905z"/><path stroke="currentColor" stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M16.51 6.262l-8.974 8.974c-1.56.617-4.257 2.371-4.728 1.9-.472-.471 1.283-3.169 1.9-4.728l8.974-8.975m0 0h0a2 2 0 0 1 2.828 0h0a2 2 0 0 1 0 2.829h0m-6.366-2.124l5.657 5.657"/></g></svg>

Nice suggestion! See #1141 :-)


[Sub-versioner] - Bug in prod

Received this bug in prod
2016/09/14 05:22:31 Could not repsond to request with non-public error: Error, could not rename archive folder to target folder. rename /construction-zone/go.uuid-v1.1.0 /construction-zone/6satori7go.uuid: no such file or directory

I find it very odd that when I run a request that breaks prod, it doesn't break dev. The outputs aren't even the same?
Here was the request go get -v<EMAIL_ADDRESS>

Here is prod output

Here is dev output

Notice that the archive and target folders have different names. Dev uses refs which is correct and known issue that we fixed in the past, specially to fix this problem. The other users version tags.

Idk how this could be possible but my first inclination is that these two samples are using different versions of the code base. The prod output seems to be using an outdated copy of the sub-versioner. If you look at the dev output towards the bottom after /target. There is only once instance of the line /construction-zone/.... in dev and two in prod (both of which are incorrect in prod).

The short of it is. This bug is not currently possible with our current setup. We changed our router code and our sub-versioner code long ago, so that we never have an instance where we're not passing refs in the sub-versioner.
Maybe we need to rebuild the router images and recycle them?

I still think that we should write tests to ensure that we catch something like this at the PR stage - even if it is just a deployment mistake.

Absolutely, I'm totally with you on that.
Also, I just tested this again and now it seems to have resolved it self and is working again. The tests that broke before now.

In response to the testing. I definitely want to make it more of a personal responsibility to test more as well.


[MB-1986] Add Xamarin shared project.

Add Xamarin shared project

I'm repackaging this as a PCL -- new PR will be up soon, the code is virtually identical.


Consider another CI approach rather than Azure DevOps

Building Atom isn't that hard in real life. Why should it be harder in a predictable environment such as CI?
I propose that we should investigate other tried and true methods to build and test Atom.  I am personally familiar with Travis CI, and I find Github Actions a reasonably promising alternative (but I'm much less familiar with it).
Azure DevOps has this benefit: Upstream uses it.
And in my mind it has this downside: It's extremely complicated.
Maybe there is a better way.
We could do this in stages: Start by building on another CI in parallel with Azure. Then move on to building and testing on two CIs. Then move to mainly building/testing with the Non-Azure CI but using Azure DevOps for releases. Then ultimately do releases from the other CI.
If at any point we're comfortable with the transition being permanently split (such as keeping the code for doing releases on Azure DevOps, but validating on another CI), then we can just leave it there. So long as we are deliberate about it.
Or I could investigate on my personal fork in spare time. And report back the good news... or not bring it up again. :laughing:

Considering that the focus of this fork is on the end-user side of the things (See them here), I do not want us to spend more time on reinventing the wheel.
We have a CI system that we are pretty happy with it. GitHub Actions is good for projects. All the other atom packages in this organization use that. See here for an example. But it does not have the capability and flexibility that Azure offers.
For example, the coaching system is not that developed, it lacks templates, and many other things.
Thanks for the suggestion, but for big projects like Atom, Azure is very suitable.

I will close this for now. I look forward to simplifying some other things like linting, but that is less related to CI. see here for more plans: https://github.com/atom-ide-community/atom/projects


Trying to replicate the basic install but not working

Hello,
I'm trying to replicate de basic install like in the doc reference, but i'm getting a error:
$provider = new \League\OAuth2\Client\Provider\GenericProvider([
'clientId'                => 'whatever',
'clientSecret'            => '',
'urlAuthorize'            => 'whatever',
'urlAccessToken'          => 'whatever',
'urlResourceOwnerDetails' => '',
'scopes'                  => 'openid',
]);
try {
// Try to get an access token using the resource owner password credentials grant.
$accessToken = $provider->getAccessToken('password', [
'username' =><EMAIL_ADDRESS>'password' => 'password',
'resource' => 'https://analysis.windows.net/powerbi/api'
]);
$token = $accessToken->getToken();

} catch (\League\OAuth2\Client\Provider\Exception\IdentityProviderException $e) {
// Failed to get the access token
exit($e->getMessage());
}
$client = new \Tngnt\PBI\Client($token);
print_r($client);
$response = $this->client->dataset->getDatasets('123');
print($response);
?>
This is the error:
Fatal error: Uncaught Error: Class 'League\Provider\GenericProvider' not found in C:\xampp\htdocs\sysportalbi\test.php:8 Stack trace: #0 {main} thrown in C:\xampp\htdocs\sysportalbi\test.php on line 8
I don't know what happens, the class exists in project, simply doesn't get...
my composer.json:
"require": {
"slim/slim":"2.0",
"rain/raintpl":"3.0.0",
"tangent-solutions/powerbi-sdk": "@dev",
"league/oauth2-client": "^2.7"
},
"autoload": {
"psr-4": {
"sysbi\": "vendor\sysbi\php-classes\src"
}
}
}

i forgot about autload.


Auto Dark/Light mode

Description
Hi !
I think that it could be good to add an auto dark mode !
Maybe we could configure it by using server time (E.G : From 11pm to 7am dark mode) ...
Thanks ! 😄
Priority
Low (Nice-to-have)

Duplicate of #273


multiple values for --extra-index-url aren't supported

As of v21.1.3 pip properly handles multiple values for --extra-index-url with credentials. This is needed when you have private package repos. pip-compile doesn't seem to work for multiple extra indexes however.
pip issue: https://github.com/pypa/pip/issues/3931
Environment Versions

OS Type: macos
Python version: 3.9.1
pip version: 21.1.3
pip-tools version: 6.2.0

Steps to replicate

and an --extra-index-url line to requirements.in with multiple values like:

--extra-index-url 'https://gitlab+deploy-token-17:xxx@gitlab.yyy.com/api/v4/projects/323/packages/pypi/simple,https://gitlab+deploy-token-18:xxx<EMAIL_ADDRESS>

run pip-compile

Expected result
The specified indexes are searched for packages
Actual result
Could not find a version that matches my-packages (from -r requirements.in (line 44))
No versions found
Were https://pypi.org/simple or https://gitlab%2Bdeploy-token-17:****@gitlab.yyy.com/api/v4/projects/323/packages/pypi/simple,https://gitlab+deploy-token-18:xxx<EMAIL_ADDRESS>reachable?

It appears that the URLs aren't being split on the , as it only masked the password of the first URL.

Hello @foozmeat,
Thanks for the issue!  Use a separate option for each URL, that's how click's @option(multiple=True) works:
echo | pip-compile - -qo- \
    --extra-index-url<EMAIL_ADDRESS>\
    --extra-index-url<EMAIL_ADDRESS>\
    --extra-index-url<EMAIL_ADDRESS>

Specifying separate --extra-index-url  entries at the top of requirements.in also worked fine. Thanks!


[silgen] Quick drive by thinko fix.

Just noticed this while reading over this code. Disheartening that our tests did
not catch it. = /.
rdar://30955427

I am sure this is correct (I made a refactoring mistake). I will try and add a test in a little bit.

@swift-ci Please smoke test and merge


[v0.1.0] Literature Pages

Criteria

[ ] Define model interfaces:

[ ] Note
[ ] Citation
[ ] Literature

[ ] Define api services:

[ ] Note: Read one (paginated), Upsert
[ ] Citation: Read all, Upsert
[ ] Literature: Count, Create, Update, Delete

[ ] Define routes:

[ ] Literature Create Page
[ ] Literature Notes Page
[ ] Literature Citations Page

[ ] Literature Create Page:

[ ] Create views
[ ] Create functionalities

[ ] Literature Notes Page:

[ ] Create views
[ ] Create functionalities

[ ] Literature Citations Page:

[ ] Create views
[ ] Create functionalities

Update: Won't be implemented in v0.1.0 due to resource constraint.


Invalid Request for Files.create_file

Hello :wave: and thank you for this neat library.
I ran into the issue below:
ExOpenAI.Files.create_file({"Test Document.pdf", testfile}, "assistants", openai_api_key: api_key)

{:error,
 %{
   "error" => %{
     "code" => nil,
     "message" => "Additional properties are not allowed ('openai_api_key' was unexpected)",
     "param" => nil,
     "type" => "invalid_request_error"
   }
 }}

I was able to fix locally with a modification to client.ex:
    multipart_body =
      {:multipart,
       params
+      |> Enum.into(%{})
+      |> Map.delete(:openai_organization_key)
+      |> Map.delete(:openai_api_key)
+      |> Map.to_list()
       |> Enum.map(&multipart_param/1)}

I'm happy to test further and contribute if appropriate.

Thanks for spotting this. Can you create a PR to fix this issue?
The behaviour of stripping the args should probably be aligned with the other methods so we can just re-use the same thing

Sure! Giving it a try now

Released in https://github.com/dvcrn/ex_openai/releases/tag/1.6.0


Update dependencies and workflows

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

Comparison is base (b9553ad) 100.00% compared to head (d310b87) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #316   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           41        41
  Lines         1633      1633
  Branches       333       333
=========================================
  Hits          1633      1633

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[GLUTEN--1491]Fix failed velox_windows_rank_test

Remove the changed RowNumber.cpp to sparksql

@zhejiangxiaomai Please help to review. Thanks.


Security considerations for public GitHub Actions

Hi,
I'm curious if there are some security considerations for the 1Password Connect server since it should be exposed publicly.

Hi @hazcod -
Yes, there are some security considerations if you choose to expose your Connect server publicly:

Use TLS: We have a guide for configuring the deployment to use LetsEncrypt certificates or a certificate you supply manually

Review some basic information about 1Password Connect security

You may want to put the Connect server behind a load balancer if you expect heavy load on the server. The public-facing API server can scale horizontally (i.e. many instances can run at once).

If you have any specific questions, I can try to get those answers for you.

Hi @verkaufer ! I have no issue discussing here FWIW.
Needing 1P secrets in GitHub Actions would mean exposing the Connect server to anyone (or at least the whole of GitHub).
We do not have any plans for running our own hosted GitHub Actions at this point.
Exposing the connect server would mean the need to implement rate limit controls against brute forcing, which is not mentioned at all in the current documentation. (e.g. what stops an attacker from endlessly trying out tokens?)

You're right, we don't make recommendations about rate limit. We can suggest high availability configurations, but it's ultimately up to the customer to decide the best setup for their infrastructure. We can't speak for every customer's set up, but we're happy to make suggestions and point you in the right direction.
As I mentioned earlier, setting up a load balancer or reverse proxy is one of the recommended approaches.


Out of date

[Enter feedback here]
This page needs updating as the information regarding setting up Github secret - cannot fid this on Github

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

ID: c9d44818-c536-ac44-327b-7f83584ce104
Version Independent ID: 86cdefdc-a6a6-8e3d-c4dc-faa7758ea170
Content: Reset deployment tokens in Azure Static Web Apps
Content Source: articles/static-web-apps/deployment-token-management.md
Service: static-web-apps
GitHub Login: @webmaxru
Microsoft Alias: masalnik

RegalTanks , Thanks for leveraging Azure docs feedback channel to raise this question. Since you have closed the thread I presume the issue is resolved, if it's not the case or you need further assistance kindly just let us know.
As a side note, this channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements.


Program crashes after logging in

Incredibly excited when we found this program but alas the program crashes at the same point every time. We can log in just fine, but when we click the "Test Connection" button it crashes. We've tried with Basic Auth, No Auth, different URLs, but same issue. Tried deleting the program and redownloading it. Tried running in different versions of Microsoft. Same issue.
Looking at the debug log the issue we get is:
_An unhandled exception of type 'System.IO.FileNotFoundException' occurred in PresentationCore.dll
Additional information: Could not load file or assembly 'Microsoft.Data.Edm, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The system cannot find the file specified._
Any help would be great as this program is exactly what we need. Thanks!

Thanks for reporting the issue, it looks like the application isn't finding a library that's required. let me take a look at it and see if I can get you an updated package.
Also, can you help me understand your use case in detail? Are you trying to connect to on-premises data using an OData API which you developed or are you using Progress DataDirect solution to connect to your data. Which version of  OData are you using? And any detail about your end to end use case would be helpful for us to learn more about the interest for this application.


About experience

The event ChangeEntityExperienceEvent had not fired when a player gains EXP.And never been fired.

Is there anyone who can help me with this question?

Patience doesn't appear to be a virtue of yours is it.

@5igm The only method to speed this up, is to provide test plugins demonstrating that it's not fired, as wel as debug logs to back up your claim.
Github Issues are not a support forum.
If you have happened across a bug, or something unimplemented, This is also the wrong GitHub repository.
SpongeAPI Repository contains only the API, reporting issues and bugs here is largely for Design issues in the API, or future feature requests.
SpongeCommon is shared code between Sponge Vanilla & Forge. If you can replicate the issue on both Sponge Vanilla, and on Sponge Forge, post an issue there. If the issue only occurs on one platform, post it there.


feature requests

Cool project man! @botolo78 tried to make something similar forking the recalbox-manager and this is the repository: https://github.com/botolo78/RetroPie-Manager
But it seems that he is very busy nowadays and don't have time to maintain it.
If you don't mind, I'll put here the features we have there as a feature request list here.

Monitor the system health and disk space
Edit the Emulation Station config file
Edit the RetroArch config file
Edit the autostart.sh script
View the Emulation Station log file
Manage your BIOS files

@meleu just now I've uploaded new version with the first of the requested features:
[ x ] Monitor the system health and disk space
Give it a try, let me know what you think ;)

Unfortunately, upon closer look, the CPU usage seems to be a little off. I'll work on it :(

As @CourierSS mentioned on the forum, the temperature is an important item to monitor.

As @CourierSS mentioned on the forum, the temperature is an important item to monitor.

@meleu agree. I've created an issue for it (#5) and I'll be working on it soon.


change error message to ensureActualIsNumber

Summary
Fixes #3020
Test plan
Given
it('test .toBeGreaterThan message', () => {
  expect("222").toBeGreaterThan(344444)
})

It prints

Seems like we don't have a test for this :O

Thank you!


Improve warning message when a field name shadows a field in a parent model

Change Summary
Added in #7193, a helpful warning message is displayed when a subclass contains a field with the same name as the parent class.
If you have many models that inherit from a parent class, it would be helpful to be able to see in the warning message which child model contains the duplicate field name - not just the parent class.
So I've simply tweaked the warning message text to allow the user to easily see which model they should inspect.
Before:
UserWarning: Field name "foo" shadows an attribute in parent "ParentModel";

After:
UserWarning: Field name "foo" in ChildModel shadows an attribute in parent ParentModel

I think this is clearer for cases where you have tens or hundreds of ChildModels.
I also removed the semi-colon because I'm not sure why it's there, but happy to put that back, I may have missed a convention or other warning message or code style rule.
Related issue number
None
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


Faisal fork waiting to be merged #hackoctober

Description
Hi, i have added my own readme file under #irfanmit (my user name) folder

[x] I have added a folder(Named it as my GitHub user id)
[x] I have added a Readme.md file inside the folder
[x] I understand the GitHub Forking Concept fully

hi @atapas sir
can you look at the PR
Thank you sir


Adding background illustrations and Glass effect on Contact section of Homepage

The current design of the contact page seems very simple. So can I work on the UI of the Homepage under Hacktoberfest 2022?
Before -

After-

Looks nice. Go ahead

Great! 👍


Add information about http clear-text traffic

@jamesmontemagno : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@davidbritch : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@davidbritch : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add links to progress bar

Add links to the progress bar to allow user to just from page to page

LGTM.


Platform Server fails to export

https://github.com/godotengine/godot/commit/787be50f1760be541e97b18a371abccc9c9b1ac9
x11
https://github.com/godotengine/godot/pull/21927
./../godot_server.server.tools.64 --path ./ --export "Linux/X11" ./../exports/test.x86_64
ERROR: instance: Class 'EditorSettings' can only be instantiated by editor.
   At: core/class_db.cpp:518.
ERROR: poll: /home/alexholly/.config/godot/editor_settings-3.tres:3 - Parse Error: Can't create sub resource of type: EditorSettings
   At: scene/resources/scene_format_text.cpp:560.
ERROR: load: Condition ' err != OK ' is true. returned: RES()
   At: core/io/resource_loader.cpp:155.
ERROR: _load: Failed loading resource: /home/alexholly/.config/godot/editor_settings-3.tres
   At: core/io/resource_loader.cpp:192.
WARNING: create: Could not open config file.
   At: editor/editor_settings.cpp:846.
ERROR: vformat: Condition ' error ' is true. returned: String()
   At: core/variant.cpp:3252.

I have noticed that when using '/opt/godot/godot --export-debug Android x.apk' in command line editor_settings-3.tres get changed all settings for android are reset and export does not export android.apk at all
export/android/adb = "" export/android/jarsigner = "" export/android/debug_keystore = ""

Hi,
Any news about this issue? I've migrated my projet from 3.0 to 3.1, and have this exact same problem on Windows and Linux. Same error, same config override. Exporting to Android from cli is now impossible since it removes important fields.

So I've been testing different things with the "normal" build and  the headless build, and I have more info:

If I don't give the path (I was calling the command from my project directory) or use --path: I have the error from this issue and my .tres is being re-written

examples:

cd FULL_PATH; /opt/Godot/Godot_headless --export-debug Android /tmp/project.apk
/opt/Godot/Godot_headless --path FULL_PATh --export-debug Android /tmp/project.apk

If I give the path to my .godot project, it works

examples:

cd FULL_PATH; /opt/Godot/Godot_headless project.godot --export-debug Android /tmp/project.apk
/opt/Godot/Godot_headless FULL_PATH/project.godot --export-debug Android /tmp/project.apk

I can confirm that fix by @groug works.

Seems to work fine in the current master branch (ae21664655).
It works both with a pre-existing .import folder and if it's missing (in the latter case, it reimports everything before exporting).
#34887 improved usability and documentation, so it should be clearer how to export from the command line now.


Added prereq directive to Makefile

Fixed #55. Added 4 directives: install-knative, install-helm, install-logging and predev that runs the previous 3.

Continuation of #58
/lgtm
/approve


add dartfix as a tool to /bin

We should add a script to launch dartfix (pkg/analyzer_cli/bin/fix.dart) into the bin/ directory of built sdks.
cc @danrubel

We don't believe this should be added to the SDK at this point.


Cannot find module 'bootstrap-icons/bootstrap-icons.svg' or its corresponding type declarations

Being able to import "bootstrap-icons.svg" in BIcon.vue in a Typescript project require the following config in the project file src/custom.d.ts:
declare module "bootstrap-icons/*.svg" {
  const path: string;
  export default path;
}

Without this file, the following error is generated:
TS2307: Cannot find module 'bootstrap-icons/bootstrap-icons.svg' or its corresponding type declarations.
I think that we need to specify this somewhere in the documentation!

Or we could declare this module inside BootstrapVue.d.ts, what do you think?

Declaring it directly on the lib would be indeed better.
However, adding the code in BootstrapVue.d.ts (or shims-vue.d.ts) is not working.
Do you have any idea?
Alternatively we could allow the //ts-ignore

solved by https://github.com/cdmoro/bootstrap-vue-3/pull/98 (TS compilation is done only when the /dist folder is not present)


The height to be rendered directly with the set values and not 'jump' to the new value from the 'default hard coded' ones
What is the current behavior?
The mat-expansion-panel-header height will jump to the newly set value (even when using the new MAT_EXPANSION_PANEL_DEFAULT_OPTIONS injection token) resulting in ugly visuals. This is very evident when switching between tabs for example
What are the steps to reproduce?
https://stackblitz.com/edit/angular-e2of2s
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular: 7.2.0
Material: 7.2.1
Browser: Chrome or Mozilla@latests
Is there anything else we should know?
In the StackBlitz example if you change the 'collapsedHeight' and 'expandedHeight' to the default value of '48px' they will not 'jump' anymore

any news regarding this problem?

I was having this same problem and found that disabling animation for the panel header addressed the issue.  I just made the change in your Stackblitz example and it seemed to work there as well:
<mat-expansion-panel-header<EMAIL_ADDRESS>expandedHeight="60px" collapsedHeight="60px">
        This is the expansion 3 title
</mat-expansion-panel-header>

So, as long as the desired behaviour is static header height, this workaround should work

@cmdickson Phew! This was driving me nuts. Thanks for pointing out that you can do this just on the header - so as to retain expand and contract animations for the panels' contents.

Anyone noticed that if you set<EMAIL_ADDRESS>when the expansion panel is within a tab, if you navigate to another tab and then back to the one with the expansion panel in, the animations are re-enabled?  Driving me nuts!

Looking at the markup, it appears that the ng-animate-disabled class is removed when you switch tabs

Any news or progress on this issue?


Add all exes to qemu

Nice. I also added autoupdate https://github.com/lukesampson/scoop/commit/801c342e105234880f85d8ba1c2ceaee71dc09f1

Tres cool!


WIP: initial duplicate bevy version lint

A first draft of a quick lint to check if multiple versions of bevy are defined. https://github.com/TheBevyFlock/bevy_cli/issues/15
todos:

[ ]  crate ui test
[x] better error message, perhaps use cargo metadata  to get information about the crate versions.
[x] fix the span ( this should point to the crate causing the issue in the Cargo.toml)

error: Multiple versions of `bevy` found
  --> /Users/didi/.cargo/registry/src/index.crates.io-6f17d22bba15001f/bevy-0.13.2/src/lib.rs:1:1
   |
1  | / #![allow(clippy::single_component_path_imports)]
2  | |
3  | | //! [![](https://bevyengine.org/assets/bevy_logo_d...
4  | | //!
...  |
51 | | #[allow(unused_imports)]
52 | | use bevy_dylib;
   | |_______________^
   |
   = note: `#[deny(bevy::duplicate_bevy_dependencies)]` on by default

example of the lint message
error: Mismatching versions of `bevy` found
  --> Cargo.toml:10:26
   |
10 | leafwing-input-manager = "0.13"
   |                          ^^^^^^
   |
help: Expected all crates to use `bevy` 0.14.2
  --> Cargo.toml:8:8
   |
8  | bevy = { version = "0.14.2", features = ["android_shared_stdcxx"] }
   |        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

One function you may find useful is was_invoked_from_cargo(), which detects whether we're being run from Cargo or not. We can use this to skip Cargo lints when bevy_lint_driver is called directly.!

One function you may find useful is was_invoked_from_cargo(), which detects whether we're being run from Cargo or not. We can use this to skip Cargo lints when bevy_lint_driver is called directly!

nice catch! Thanks will add this.


Add skipActionRuleSet to appgw waf definition

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-go

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Please rebase your branch, there are a lot of unrelated commits here

@jshaker  could you take a look at the error for spell check?

@number213 addressed commits
@lirenhe addressed spell check

@jshaker, please fix the problem for model validation.

These changes are being pushed to a later release cycle


“Failed to save credentials"

When trying to add credetials I get this error: “Failed to save credentials - Please report this error: set_credentials”
Also when running :SourcegraphLogin! and adding the token, nothing happens.

As a workaround, setting the environment variable "SRC_ACCESS_TOKEN" worked for me.


Dev Branch Starter Pull Request

Now I'm sending from dev branch.
Previous I've done with master branch by mistake

This was pushed into the master branch, so I am closing this PR


Can't drag the image if it's smaller than viewport.

As mentioned in #64 image is not centered when using in Materialize JS and it can't be dragged if smaller then viewport. I've changed minZoom = Math.max(minW, minH) line to minZoom = 0.1 in croppie.js to make the problem more visible.

This one would be easier to diagnose if we had that fiddle or bin from #64.  What's the ratio of the size of your image to your viewport?

Check this please http://jsbin.com/cirulonimi/edit?html,js,output

If you check out this modified bin - where the zoom out correctly centers on the image: http://jsbin.com/xepoxenobu/edit?html,js,output
When you're zoomed all the way out, I wouldn't expect to be able to drag it.  The image has to fill up the viewport, that's one of the limitations of croppie.  What behavior are you expecting?

For example lets say we have a viewport size 100x100 and image is 100x80
I think it would be better if croppie can fill the blank 20pixels with white bg or transparent bg so the image fits the viewport.

In the scenario you described, croppie sees that your image isn't as tall as your viewport so it sets the minZoom to 100/80=1.25.  Then the max zoom is arbitrarily set to 150% (or something like that) of the min value, so there isn't any blank space.
If you change the viewport.height in the latest bin to 120, you can see what I'm talking about.

I'm trying to achieve an option where minZoom is not 100/80=1.25 but 80/100=0.8 and after this we still can drag the image in viewport.

I understand, but I will say the chances of this being done are slim.  There's a ton of logic in croppie.js that is restricting you from dragging the image outside of the viewport, so that minzoom becomes pretty important.
Also, currently the image that you're dragging around is an actual  tag.  I could see this being a possibility if we drew the image to a canvas (like we're experimenting around with in a different branch), and you drag the canvas around.  Even then, I'm having a hard time understanding why you wouldn't want the image to span the entire viewport.
What if you had the ability to resize the viewport after knowing the dimensions of the image?

Let's say we need to get 100x100 image on the out. But user is uploading a rectangle image and he wants it to fit the 100x100 viewport. Is there any way I can resolve this?

Not with the stock croppie code, you might be able to change line 652 to Math.min(minW, minH).
But I'm not sure retrieving the result canvas type will work.  You can sure give it a try.

I guess this could be a great option if you can implement it later. Fitting an image into viewport is a common task. Anyway, thanks a lot for response and support, you are the best!!! )))


Nightly not updating despite #1997

Problem
⮞ rustup show
Default host: x86_64-unknown-linux-gnu
rustup home:  /home/lesur/.rustup

installed toolchains
--------------------

stable-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu (default)

active toolchain
----------------

nightly-x86_64-unknown-linux-gnu (default)
rustc 1.39.0-nightly (9b9d2aff8 2019-09-19)

⮞ rustup --version
rustup 1.19.0 (2af131cf9 2019-09-08)

⮞ rustup update
info: syncing channel updates for 'stable-x86_64-unknown-linux-gnu'
info: syncing channel updates for 'nightly-x86_64-unknown-linux-gnu'
info: latest update on 2019-10-09, rust version 1.40.0-nightly (2748a9fd9 2019-10-08)
error: component 'rustfmt' for target 'x86_64-unknown-linux-gnu' is unavailable for download for channel 'nightly'
If you require these components, please install and use the latest successful build version,
which you can find at <https://rust-lang.github.io/rustup-components-history>.

After determining the correct date, install it with a command such as:

    rustup toolchain install nightly-2018-12-27

Then you can use the toolchain with commands such as:

    cargo +nightly-2018-12-27 build
info: checking for self-updates

       stable-x86_64-unknown-linux-gnu unchanged - rustc 1.38.0 (625451e37 2019-09-23)
  nightly-x86_64-unknown-linux-gnu update failed - rustc 1.39.0-nightly (9b9d2aff8 2019-09-19)

As per this page, I cannot upgrade to the latest nightly version. However it also shows that 2019-10-04 should work. If I understand #1997 correctly, rustup should upgrade from 2019-09-19 (my current version) to 2019-10-04 (latest version including everything I am using).
Am I doing something wrong here or is it an actual bug?
Thanks :)

This is expected, we have not yet released a rustup containing that change.  Hopefully that'll change next week.


Fix slice unpad

Fixed slice_unpad, now it supports complex numbers

Ignore the formatting error.
It is fixed in another PR
Once you merge https://github.com/DifferentiableUniverseInitiative/jaxDecomp/pull/17 and https://github.com/DifferentiableUniverseInitiative/jaxDecomp/pull/12 and I wll rebase and this should be ok

@ASKabalan can you try to rerun your precommit locally to squash the formatting test error?

Yes all is green locally.
But at some point this formatting error correction slipped here and I think it fixes this issue.
Any ways this PR should come after the other two

rebased and fixed commit history


Exec task fails with Unicode characters

Run Exec task with non-ANSI characters in the command:
<Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
    <Target Name="Build">
        <Exec Command="创建\test.cmd"/>
    </Target>
</Project>

This doesn't work. Exec creates a temp cmd file and executes it.
Cmd file looks like:
setlocal
set errorlevel=dummy
set errorlevel=
??\test.cmd
exit %errorlevel%

This was done on purpose a very long time ago (Windows XP doesn't support this scenario).

Sounds like https://github.com/Microsoft/msbuild/issues/104 ?


refactor: data sources generate OTel data

Proposed changes
This is a refactor PR, which presents an alternative way to produce OTel data for our data sources.
In this approach, the data source transforms its source metrics into OTel data points depending on the type of source content. OTel metrics are then propagated forward to the OTel Producer using a callback, which then periodically sends it out to our exporter (in this case via gRPC).
Using this method we don't need to keep track of e.g. previous counter values, but instead define the temporality of our data as Delta (instead of Cumulative), so OTel should understand that they should not be added to previously received values.
Usage of our own internal message bus is completely removed in this implementation.
I will still test this out to ensure that it works the way we want it to. We can opt to not go down this route too, just wanted to make sure we think about our options before pulling the trigger.
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING document
[ ] I have run make install-tools and have attached any dependency changes to this pull request
[ ] If applicable, I have added tests that prove my fix is effective or that my feature works
[ ] If applicable, I have checked that any relevant tests pass after adding my changes
[ ] If applicable, I have updated any relevant documentation (README.md)
[ ] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

We opted to go with another approach than what was proposed here (although some elements were incorporated into PR-544), so I'll close this PR.