Generator stops working out of nowhere - Warning CS8785

Occasionally, the generator(s) stop working, which only can be solved by restarting the IDE / Editor of choice. F.e.:

Warning CS8785 Generator 'TypealizedClassSourceGenerator' failed to generate source. It will not contribute to the output and compilation errors may occur as a result. Exception was of type 'OperationCanceledException' with message 'The operation was canceled.'

Warning CS8785 Generator 'StringLocalizerExtensionsSourceGenerator' failed to generate source. It will not contribute to the output and compilation errors may occur as a result. Exception was of type 'OperationCanceledException' with message 'The operation was canceled.

Looks like the whole SOurceGenerator-SDK is not expecting OperationCanceledException, which currently are thrown when "CancellationRequested" by the infrastructure - so avoid that and instead return gracefully.

StackTrace for StringLocalizerExtensionsSourceGenerator
System.OperationCanceledException: The operation was canceled.
   at System.Threading.CancellationToken.ThrowOperationCanceledException()
   at Microsoft.CodeAnalysis.Host.WeaklyCachedRecoverableValueSource`1.GetValue(CancellationToken cancellationToken)
   at Microsoft.CodeAnalysis.RecoverableTextAndVersion.GetValue(LoadTextOptions options, CancellationToken cancellationToken)
   at Microsoft.CodeAnalysis.TextDocumentState.GetTextSynchronously(CancellationToken cancellationToken)
   at Microsoft.CodeAnalysis.Diagnostics.AdditionalTextWithState.GetText(CancellationToken cancellationToken)
   at TypealizR.Core.RessourceFile.<>c__DisplayClass22_0.<From>b__7(<>f__AnonymousType4`2 _)
   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()
   at System.Linq.Enumerable.<SelectManyIterator>d__17`2.MoveNext()
   at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
   at TypealizR.Core.ResxFileSourceGeneratorBase.<Initialize>b__0_3(SourceProductionContext ctxt, ValueTuple`2 source)
   at Microsoft.CodeAnalysis.UserFunctionExtensions.<>c__DisplayClass3_0`2.<WrapUserAction>b__0(TInput1 input1, TInput2 input2, CancellationToken token)

StackTrace for ``
System.OperationCanceledException: The operation was canceled.
   at System.Threading.CancellationToken.ThrowOperationCanceledException()
   at Microsoft.CodeAnalysis.Host.WeaklyCachedRecoverableValueSource`1.GetValue(CancellationToken cancellationToken)
   at Microsoft.CodeAnalysis.RecoverableTextAndVersion.GetValue(LoadTextOptions options, CancellationToken cancellationToken)
   at Microsoft.CodeAnalysis.TextDocumentState.GetTextSynchronously(CancellationToken cancellationToken)
   at Microsoft.CodeAnalysis.Diagnostics.AdditionalTextWithState.GetText(CancellationToken cancellationToken)
   at TypealizR.Core.RessourceFile.<>c__DisplayClass22_0.<From>b__7(<>f__AnonymousType4`2 _)
   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()
   at System.Linq.Enumerable.<SelectManyIterator>d__17`2.MoveNext()
   at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
   at TypealizR.Core.ResxFileSourceGeneratorBase.<Initialize>b__0_3(SourceProductionContext ctxt, ValueTuple`2 source)
   at Microsoft.CodeAnalysis.UserFunctionExtensions.<>c__DisplayClass3_0`2.<WrapUserAction>b__0(TInput1 input1, TInput2 input2, CancellationToken token)

current solution is not to pass down cancellationToken to AddtionalText.GetText() and instead fall back to string.Empty(), when cancellation is requested by the host.
This hopefully keeps the SourceGenerator alive.


Get URL of current tab when responseReceived is fired

Component
Version

Operating system
Ubuntu 16.04

Node.js
7.6.0

Chrome/Chromium/...
Chrome

chrome-remote-interface
0.25.1

Is Chrome running in a container? NO
Hello!
I'm using CDP and Chrome to catch redirects (301, meta, js redirect) by handling responseReceived event and checking params.type === 'Document' , but faced with problem :  iframes (and another widgets like YouTube videos) also loads as Document (and it's normal).  But I need handle url changes for tab only.  How I use this way:
    Network.responseReceived((params) => {
        if (params.type === 'Document') {
            this.redirects.push(params.response.url);
        }
    });

I thought about run js  window.location.toString() for each responseReceived event, but I think it's bad way.
Is there way to get URL for current tab when responseReceived is fired ?
Many thanks!

CDP({port : portCDP}, CDPHandler);

...

let CDPHandler = async function(client) {
    this.redirects = [];

    // Extract used DevTools domains.
    const {DOM, Emulation, Network, Page, Runtime, Security} = client;

    // Enable events on domains we are interested in.
    await Page.enable();
    await DOM.enable();
    await Network.enable();
    await Network.setCacheDisabled({cacheDisabled: true});
    await Security.enable();
    await Security.setOverrideCertificateErrors({override: true});

    // If user agent override was specified, pass to Network domain
    if (userAgent) {
        await Network.setUserAgentOverride({userAgent});
    }

    Network.responseReceived((params) => {
        if (params.type === 'Document') {
            this.redirects.push(params.response.url);
        }
    });

    Security.certificateError(({eventId}) => {
        Security.handleCertificateError({
            eventId,
            action: 'continue'
        });
    });

    await Page.navigate({url});
};

Hi! I needed something similar in chrome-har-capturer, the solution I came across was to keep track of the original request (Network.requestWillBeSent) and treat this one as a special kind of request. But this can help you only in the case of a simple 301 redirect; it's harder to discern the other two.
What you can do is to keep track of the frameId field of the original request (originalFrameId) and only consider requests (and responses) that have initiator.type === 'other' && frameId === originalFrameId.
You can also obtain the document URL via the documentURL field and consider only those requests that matches the tab URL (this should be the same as window.location.toString() ) but I guess you'll have to take into account the redirects themselves here.
Hope it helps!

Hi! Thanks for so detailed response ;)
I will try do it today to keep frameId, It can help

You can also obtain the document URL via the documentURL
Where can I find documentURL ?  (in requestWillBeSent or responseReceived ? I didn't find this field)

You're welcome. It's in Network.requestWillBeSent.

@OlegBabakov I used this

Network.responseReceived((params) => {
if (params.type === 'Document') {
this.redirects.push(params.response.url);
}
});
Then outside in a functio when i try and fetch `this.redirects` it returns empty array. Have you faced something like this?

@shashidharus this will fail when you have iframe requests on the page. I agree with https://github.com/cyrus-and/chrome-remote-interface/issues/278#issuecomment-335135613

    Network.requestWillBeSent((params) => {
        if (params.initiator !== undefined && params.initiator.url !== undefined && params.documentURL !== undefined &&
            this.redirects.indexOf(params.initiator.url) !== -1 && //If initiator page exist in redirects => it's iframe loads
            this.ignoredDocuments.indexOf(params.documentURL) === -1
        ) {
            this.ignoredDocuments.push(params.documentURL);
        }
    });

I check every request by this code on existing of params.initiator.url in redirects array. This helps me to avoid counting iframe requests as redirects

I used this
... var cp = await CDP(); ... var infos = await cp.Target.getTargets(); var myUrl=infos.targetInfos[0].url;


Tilde operator treats values other than true and 1 as false

I tested vals.#(b==~false)#.a with the example provided:
{
  "vals": [
    { "a": 1, "b": "true" },
    { "a": 2, "b": true },
    { "a": 3, "b": false },
    { "a": 4, "b": "0" },
    { "a": 5, "b": 0 },
    { "a": 6, "b": "1" },
    { "a": 7, "b": 1 },
    { "a": 8, "b": "true" },
    { "a": 9, "b": false },
    { "a": 10, "b": null },
    { "a": 11 }
  ],
  "anything": "else"
}

And it gives the desired result: [3,4,5,9,10,11]
However, when we have values other than true or 1, they are treated as false:
{
  "vals": [
    { "a": 1, "b": "something" },
    { "a": 2, "b": "else" },
    { "a": 3, "b": false },
    { "a": 4, "b": "0" },
    { "a": 5, "b": 0 },
    { "a": 6, "b": "1" },
    { "a": 7, "b": 1 },
    { "a": 8, "b": "true" },
    { "a": 9, "b": false },
    { "a": 10, "b": null },
    { "a": 11 }
  ],
  "anything": "else"
}

Gives: [1,2,3,4,5,9,10,11]
Try: https://go.dev/play/p/ey6dS3zFk3p
I don't this is the desired behavior.
Is there a way to get the fields, for which the data is either null or missing?

In your case, gjson is simply treating ~false as all values that are not true-ish.
I agree that this is probably not desired.
Rather, we likely want to more explicit, where all true-ish values are true, and all false-ish are false.
I made some updates moments ago to the syntax that addresses the ~false issue.
Now vals.#(b==~false)#.a returns [3,4,5,9,10,11].
I also added two more tilde types:
~null      Converts null and non-existent values to true
~*         Converts any existing value to true

Is there a way to get the fields, for which the data is either null or missing?

Yes, using ~null or ~*:
vals.#(b==~null)#.a   => [10,11]
vals.#(b!=~null)#.a   => [1,2,3,4,5,6,7,8,9]
vals.#(b==~*)#.a      => [1,2,3,4,5,6,7,8,9,10]
vals.#(b!=~*)#.a      => [11]

Great! Just tried and it works like a charm! ~* is a handy addition too. Thank you!


Delete Transitions

It should be possible to delete Transitions:

select
press [Del]

Note that you need to delete everything that is related with the transition (except the connected States):

Transition
Connected PreCondition
Regenerate Source Code for Plan of deleted Transition in order to delete the Method Stubs

Please write it in a way, that >undo< is possible, including regenerating the old method stubs with old content.

Done :+1:


Add recent-logic-pro-projects extension

Description
Inspired by Raycast's recent-excel extension, I wanted to have the same thing for Logic Pro since you can be so much faster opening Logic Pro projects using raycast then via the Logic Pro UI.
The extension lets you choose your Logic Pro working dir in the beginning and then returns all projects from that dir including last accessed time. Using enter you can directly open the selected project.
The searchbar lets you filter the shown entries.
Screencast
Setup

Action

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

closed in favor of #15110


Add Cameron Blake to Contributors list

Thanks for the guide!

I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://goo.gl/PM1Jdh


03-Css-Waves

Complete the following REQUIRED checkboxes:

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this PR follows the location of change: brief description of change format, e.g. 01-flex-center: Update self check

Complete the following checkbox ONLY IF it is applicable to your PR. You can complete it later if it is not currently applicable:

[x] I have ensured that the TOP solution files match the Desired Outcome image

1. Because:

Closes #XXXXX
2. This PR:

3. Additional Information:

Please do check this out & let me know if there's any mistake by my side. ✌🏼

Hey can you please merge if there's no mistake?


Inaccessible Event Modal

When an event is clicked, two Modal pops up where one overlaps the other thus the first modal is not accessible at all.

The two modals have been combined into a single modal. The description and maps are combined in a single modal.


Various size optimizations.

Combination of various size optimizations to make room in the engine so we can then not use that to add Elo gainers.

Thanks!


I18N – add English and Swedish to the UI

Is your feature request related to a problem? Please describe.
Finland is a multilingual  country, and Finnish is not the easiest language to understand.
Describe the solution you'd like
An easy way to select the preferred language in the UI. Store the selected language in local storage or cookie for use on later visits. Optionally add different languages as routes in order for direct linking by URL.
Describe alternatives you've considered
Alternatively, just keep the UI in Finnish but display the reasons for interruptions in all 3 languages that the HSL API already provides.

@olpeh I'd like to pick this for english. any hint for where to start?

Hi @farskid
My idea for approaching this has been the following:

Modify the backend to return all languages instead of just picking the finnish version
Add a language selector to the UI, and displaying the UI and the traffic alerts in the selected language, possibly following this blog post or any other elmy way of doing I18N. My idea was to not do it in the easiest way, but rather do it the "correct" way
Save the language selection locally for next visits (also keep in mind that saving the selected theme would be nice)
Add language to routing, for example onkometrorikki.fi/en etc.

We could of course split this into smaller chunks and implement one step at a time or something
One challenge in implementing this is that the feed is the live data, I don't know if there is a test feed available that would always contain some service alerts. Testing this with live data would require the metro have some interruptions while developing.

@olpeh Sounds good to me. I'm on it.

And also, there is no way at the moment to run it using a local backend easily. Should probably make that possible. Added as an issue #10


During restart Zwavejs drop packets

Guys, during the restart of my zwavejs instance I am getting the following message when trying to set the power level resulting  for this to drop some packets only during the restart
However, I cannot change my power level on my zwave stick
Any thoughts?

04:01:00.082 CNTRLR   querying version info...

04:01:00.094 CNTRLR   received version info:

                        controller type: Static Controller

                        library version: Z-Wave 6.07

04:01:00.095 CNTRLR   querying protocol version info...

04:01:00.113 CNTRLR   received protocol version info:

                        protocol type:             Z-Wave

                        protocol version:          6.7.0

                        appl. framework build no.: 97

04:01:00.114 CNTRLR   querying serial API setup capabilities...

04:01:00.126 CNTRLR   supported serial API setup commands:

                      · GetSupportedCommands

                      · SetTxStatusReport

                      · SetPowerlevel

                      · GetPowerlevel

                      · GetMaximumPayloadSize

04:01:00.127 CNTRLR   Querying configured powerlevel...

04:01:00.136 CNTRLR   Current powerlevel 0 dBm (1 dBm) differs from desired powerlevel 0 dBm (0 dBm)

                      , configuring it...

04:01:00.145 CNTRLR   The controller response indicated failure after 1/3 attempts. Scheduling next

                      try in 100 ms.

04:01:00.249 CNTRLR   The controller response indicated failure after 2/3 attempts. Scheduling next

                      try in 1100 ms.

04:01:01.355 CNTRLR   Changing the powerlevel failed!

04:01:01.356 CNTRLR   querying controller IDs...

04:01:01.363 CNTRLR   received controller IDs:

                        home ID:     0xfbc85469

                        own node ID: 1

04:01:01.381 CNTRLR   Performing soft reset...

04:01:01.387 CNTRLR   Waiting for the controller to reconnect...

04:01:02.888 CNTRLR   Re-opening serial port...

04:01:03.897 CNTRLR   Waiting for the Serial API to start...

04:01:04.204 CNTRLR   Serial API started

04:01:04.297 CNTRLR   supported Z-Wave features:

                        · SmartStart

04:01:04.297 CNTRLR   querying controller capabilities...

04:01:04.304 CNTRLR   received controller capabilities:

                        controller role:      primary

                        is the SUC:           true

                        started this network: true

                        SIS is present:       true

                        was real primary:     true

04:01:04.305 CNTRLR   Enabling TX status report...

04:01:04.311 CNTRLR   Enabling TX status report successful...

04:01:04.312 CNTRLR   finding SUC...

04:01:04.318 CNTRLR   This is the SUC

04:01:04.318 CNTRLR   querying additional controller information...

04:01:04.424 CNTRLR   received additional controller information:

                        Z-Wave API version:         8 (legacy)

                        Z-Wave chip type:           ZW050x

                        node type                   Controller

                        controller role:            primary

                        controller is the SIS:      true

                        controller supports timers: false

                        nodes in the network:       1, 2, 3, 5, 9, 15, 16, 21, 22, 27, 30, 33, 37, 4

                      6, 49, 50, 51, 57

Please make another driver log, this time on loglevel debug and attach it here as a file (drag & drop into the text field).


Leap Motion Documentation Update

Overview
Updated the Leap Motion documentation:

Added instructions on integrating the new 4.5.0 Leap Core Assets.
Added intro gif at the top of the page
Updated images
Updated the Getting the Hand joints example script for both hands instead of one
Added the copying assembly error to Common Errors
Added Editor Workflow section

Changes

Fixes: #7689

/azp run mrtk_docs

/azp run mrtk_docs

Looks like mrtk_docs is failing because of missing images in the experimental documentation

Looks like mrtk_docs is failing because of missing images in the experimental documentation

Ah, I'll revert that for now and re-enable after Yoon's changes go in.

/azp run mrtk_docs


New ReCaptcha

I've added the new ReCaptcha support and implemented an asynchronous API for it (but it is not finished, there's more work to be done with it).
It should close #58 and there's a small amount of work left to close #44.

Looks good. Let's merge it!


Add failing test regarding missing input

Sadly this reopens #75
Given no configuration and no CLI input, Symfony will happily validate the InputDefinition.
Caused by removing the array_filter using is_null while fixing the InputOption>config issue.

Added my attempt of fixing it.

Pondered some more, the obvious solution is to check if the option is VALUE_NONE and handle that.
See the latest revision. It should make sense, the tests are passing and InspectableArgvInput can be removed.

The following works as well. The issue with the bug is that we set all arguments and options in the Input (in our case InspectableArgvInput) even if the configuration doesn't have them set. So we basically have the same problem we had with Input, but with the configuration class, that is, being able to determine if a key is set, and only then using that value.
Here's a fix that makes your test pass without having to use the InputDefinition. If you don't see anything wrong with this fix, I'll take your test in your commit above and submit a fix.
	public function merge(InputInterface $input, Configuration $config)
	{
		foreach ($input->getArguments() as $argument => $value) {
			if ($input->hasArgumentSet($argument)) {
				$config->set($argument, $value);
			} else {
				$value = $config->get($argument);
				if ($value) {
					$input->setArgument($argument, $value);
				}

			}
		}

		foreach ($input->getOptions() as $option => $value) {
			if ($input->hasOptionSet($option)) {
				$config->set($option, $value);
			} else {
				$value = $config->get($option);
				if ($value) {
					$input->setOption($option, $value);
				}
			}
		}
	}

The 2nd commit is OK, however the 3rd should be more clear. It's checking for the actual root cause instead of using InspectableArgvInput to get around the existing behaviour.
IMO you should clearly define the desired behaviour first. "Overriding VALUE_NONE options" is tricky in itself. IMO they should simply not be used as it's confusing: I can NOT override a VALUE_NONE from the CLI. Not the first time there's confusion surrounding symfony/console arguments/options :/

I'm not sure I'm following you. Can't a InputOption::VALUE_NONE be set to true/false, thus allowing us to override them via the configuration?


base_cam.py file

Want to know about base_cam.py , is this implementation of the earlier version of grad_CAM , I mean the normal Class Activation Maps based on paper "Learning Deep Features for Discriminative Localization" ?

Want to know about base_cam.py , is this implementation of the earlier version of grad_CAM , I mean the normal Class Activation Maps based on paper "Learning Deep Features for Discriminative Localization" ?

Hi, no, it's just a common abstraction for tall the cam methods.
The original CAM isn't implemented here, since it's relevant only for certain architecture (with a GAP + classifier at the end, like resnet).


Update the polyfills to their latest versions

Why would we need that? What's wrong with current polyfills?


Document some of the optimizations

First pass at documenting the built-in optimizations. Did not proof read. So feel free to nitpick typos
cc @sw-yx
Closes #8.

very good writeup! if we were to do the fb minified code thing, we may have to fork babel-plugin-dev-expression so that it doesnt quite strip away the arguments, but minifies and appends them to a file instead somehow (probably based on a hash). do i have that intuition right?


Large memory leak

There appears to be a large memory leak caused by sending notifications.
Here is a graph of the memory usage when running on AWS. When restarted it uses ~10mb, but at sending the first few notifications it jumps to ~120mb and never goes back down. I looked through the code and didn't see anything obvious, would appreciate help tracking down this issue.

FCM: 2.7
Docker Image: swift:5.2-bionic-slim

@grahamburgsma is it growing in time?

@grahamburgsma is it growing in time?

I think so, that first spike is sending about 20 notifications. If it was only one at a time more spread out I think it would be more gradual. Although it doesn't seem linear, that second spike from 110 to 120 was the same amount of notifications, but only jumped 10mb. The next time it tried sending notifications the process was killed though.

I've checked RAM for one last hour. If I disable my Vapor app then RAM is used for 200Mb (it is Ubuntu 20.04, nginx, postgres), then after Vapor app launch RAM usage is about 220Mb, after sending a bunch of notifications it grows up to 255Mb and never goes higher. So my app's RAM usage is around 60Mb overall.
@grahamburgsma How your logic for sending pushes look like?

I've checked RAM for one last hour. If I disable my Vapor app then RAM is used for 200Mb (it is Ubuntu 20.04, nginx, postgres), then after Vapor app launch RAM usage is about 220Mb, after sending a bunch of notifications it grows up to 255Mb and never goes higher. So my app's RAM usage is around 60Mb overall.

Regardless, sending notifications should not balloon the memory usage that much. Even if it spikes, it should be released and go back down to what it was before, no?

@grahamburgsma How your logic for sending pushes look like?
Pretty straight forward, just application.fcm.send(message). Sometimes it is in a loop which get's flattened, but that shouldn't be a problem. I've tried it from route handlers and using a job with queues, both have the same issue.

Looks good in profiler 🙂

if to look at the same thin in debug navigator or activity monitor it is only growing, while at the same time in profiler it shows that everything is ok and it is allocating. It looks like maybe it is by design? Like ARC works well but RAM is still reserved by the app.


Update MappingProcessor.java

MappingProcessor.getExistingValue returning null when destination is a Collection #187
In the check to make sure that we are not in the list by checking the destFieldType it should be added that the destFieldType can be a Collection. e.g. CollectionUtils.isCollection.
Otherwise, this method returns null, when it should return the collection. (As it does for the Lists)

@Paueti ; can you rebase onto latest to force travis retest please

@Paueti ; any update?

Closing due to inactivity.


authenticationCodeTypeFirebaseAndroid - can be used for registration in tg or just for log in existing account already registered in telegram?

//@description An authentication code is delivered via Firebase Authentication to the official Android application
//@nonce Nonce to pass to the SafetyNet Attestation API
//@length Length of the code
authenticationCodeTypeFirebaseAndroid nonce:bytes length:int32 = AuthenticationCodeType;

authenticationCodeTypeFirebaseAndroid
authenticationCodeTypeFirebaseIos
can be used for registration in tg or just for log in existing account already registered in telegram?

Apps can't "use" them. They can be chosen by the server in the official mobile apps as the way to authenticate the user. The types are irrelevant for all other apps.

way to authenticate the user

for registration or only log in?

It is up to the server.


Release on gemcutter

Could you update the version on gemcutter to 0.3.x ?
Thanks

The current version is released on rubygems.org (formerly gemcutter.org when this issue was open). Subsequent releases will also be released there.
Issue closed.


React Bootstrap Beta - TypeError: Cannot read property 'current' of undefined

The code runs fine when building for dev, however when running tests on a project using react-bootstrap at version 2.0.0-beta.4, they fail with TypeError: Cannot read property 'current' of undefined. The following output is from creating a default application app with create-preact-app, adding<EMAIL_ADDRESS>adding a component to the header file and running tests, see repro here :
>npm run test

><EMAIL_ADDRESS>test
> jest

 FAIL  tests/header.test.js
  ● Test suite failed to run

    TypeError: Cannot read property 'current' of undefined

      2 | import { Link } from "preact-router/match";
      3 | import style from "./style.css";
    > 4 | import { Container } from "react-bootstrap";
        | ^
      5 |
      6 | const Header = () => (
      7 |   <Container>

      at jsxDEV (node_modules/react/cjs/react-jsx-runtime.development.js:860:73)
      at jsxWithValidation (node_modules/react/cjs/react-jsx-runtime.development.js:1155:19)
      at jsxWithValidationDynamic (node_modules/react/cjs/react-jsx-runtime.development.js:1209:12)
      at Object.<anonymous> (node_modules/react-bootstrap/cjs/Carousel.js:57:46)
      at Object.<anonymous> (node_modules/react-bootstrap/cjs/index.js:69:40)
      at Object.<anonymous> (src/components/header/index.js:4:1)
      at Object.<anonymous> (tests/header.test.js:2:1)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.873s
Ran all test suites.
Exception: npm exited with 1
[tty 17], line 1: npm run test

Thanks for filing an issue! Looks like bootstrap is using the newer jsx functions from jsx-runtime. I've just published<EMAIL_ADDRESS>to resolve that issue.
To add that fix, update the following version ranges in your package.json and run npm install again.
-    "jest": "^24.9.0",
-    "jest-preset-preact": "^1.0.0",
+    "jest": "^27.0.6",
+    "jest-preset-preact": "^4.0.5",


Add support for image building for Oracle Cloud

There is a need for enabling Wardroom to build base images on Oracle Cloud. It looks as if HashiCorp Packer (as of 1.2.0 or later) includes a builder to build base OS images for Oracle Cloud, so this should be a fairly straightforward addition.

@scottslowe Do we have access to an account to use for testing?

@detiber Not yet, working on it. As soon as I get an account, I'll start working on a PR.

@detiber I've managed to obtain an Oracle Cloud account I can use for testing, so I'll get started on a PR in the next day or two.

Closed via #95.


Allow access to the backlog state of the disruptor to improve program…

…atic reconfiguration of async appenders.
This change is to help support programmatically switching async appenders. In cases where it is important that logged messages go to the appender that was configured at the time of writing the log event, the flushing the appender buffer to disk is not sufficient, as the log events may still be in the disruptor. This change provides access to test if the disruptor has items in its backlog. This becomes useful in an application that is controlling its logging config and logging events to ensure that no new logging events are generated between removing one appender and adding another, so that it can also ensure the change does no happen until all pending log events are processed.

I am not sure if the build errors are due to my changes. I get the same errors if i do a clean master build. However, last Travis master build was OK. I cannot kick off a new build to verify. Does the PR build process use different build settings? I can see  PR https://github.com/apache/logging-log4j2/pull/256 has the same issue, and they only changed a document.

The problem appears to be the revapi.json, or the reference version. The deprecated methods have been removed on master, but I think the revapi plugin is comparing against the last release, and so it thinks they have just been deleted. Not sure how the last master build manged to work though. Was the check skipped as the methods were actually supposed to be deleted?

Have raised a Jira, if that helps... LOG4J2-2552

Hi @ptogher, sorry I haven't had a chance to look at this earlier.
The information you are looking for is already available in current versions of Log4j, although in slightly less convenient form, if you do:
RingBufferAdmin admin = asyncLoggerContext.createRingBufferAdmin();
boolean hasBacklog = admin.getRemainingCapacity() != admin.getBufferSize();

Please be aware that if you have an application that is logging from multiple threads, and one of these threads asks the question "do we have a logging backlog", the answer may be stale immediately. This is simply the nature of multi-threading. The answer is only reliable if the application logs from a single thread, or if you can somehow stop all threads from producing log events.

Ooh, many thanks, That's really useful. I missed this when i was looking for a solution. Completely understand about the threading. We have very specific thread controls in place to manage this.


Over-the-spot implementation for IME preedit

demo:

Fixed incorrect commit message and title.

Fixed test.

Could you update screenshot in description? I can't look at the link.

I could see the screencast in the linked page. Looks great. Let me try this patch before merging it in order to see it works in other environments.

Reply to https://github.com/rhysd/neovim-component/pull/46#issuecomment-327717556.
I can reproduce the problem neither on Windows 10 nor on Linux. I've changed "font-size" of  component but I can't see. I don't have Mac so I can't test it.

OK, I'll try to fix the issue on Mac.

Below diff fixed the problem. Could you apply it to this branch?
diff --git a/src/neovim/input.ts b/src/neovim/input.ts
index 182ed4a..f04be79 100644
--- a/src/neovim/input.ts
+++ b/src/neovim/input.ts
@@ -435,9 +435,7 @@ export default class NeovimInput {
     }

     updateFontSize() {
-        const {specified_px} = this.store.font_attr;
-
-        const font_size = specified_px * (window.devicePixelRatio || 1);
+        const {specified_px: font_size} = this.store.font_attr;
         this.element.style.fontSize = font_size + 'px';
         this.fake_element.style.fontSize = font_size + 'px';
     }


Update docstring for Conv3DTranspose

As mentioned in #29841, dillation_rate is not a supported feature for Conv3DTranspose as of yet. Until this feature is supported, it is misleading to show dilation_rate as an available parameter in the docstring.

Thanks for the PR. This is fixed, i.e., conv3d_transpose should support dilation_rate. Closing it for now. Let us know if any other issues come up


React-Track

... is taken.

Closing, because we use a private registry now and it doesn't look like publishing OSS stuff is a priority.


When I load More Data,the _onPaginate function was called twice and page add twice,so some data can not be request(usually request page 1,2,4,6....)

I have the same problem. How do you solve it ？ Can you tell me?


Update free-courses-ml.md

Ordered playlist in alphabetic order
Added a new Malayalam playlist of

Android
Angular
Bootstrap
C
CSS
CPP
C#
Flask
Flutter
Game Development
Git
HTML
JAVA
JavaScript
Linux
Laravel
MYSQL
Next JS
Node JS
PHP

What does this PR do?
Added resources and improved repo
For resources
Description
Why is this valuable (or not)?
This open-source contribution is valuable because it allows Malayali students to learn coding in their native language, Malayalam. By making coding more accessible to students in their native language, this open-source contribution can help to increase access to coding resources and potentially open up a new world of opportunities for Malayali students.
NB: Malayalam is the native language of Kerala. Kerala is situated in INDIA
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
This is an online course
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

🎉 Added playlist in alphabetic order

Thanks, and welcome to free-programming-books!


Restore valid behavior of superuser User objects.

When a site admin has no organization role, they need a valid role description
or graphql will (correctly) complain that the object is invalid.
Related Issue or Background Info
A snap fix last night caused a test to fail, and the test is actually testing a valid use case: super-user who has no role in an organization.
Changes Proposed

Return "Super Admin" if a super admin is logged in with no other role
return a badged version of the organization role (e.g. "Standard user (SU)") if a user is both a site admin and an organization user

This doesn't cover the case where somebody manages to log in with neither organization role nor admin rights, which is unlikely but possible.

This doesn't cover the case where somebody manages to log in with neither organization role nor admin rights, which is unlikely but possible.


logging.info(f'last_percent loaded with a value of {last_percent}')

Hello,
i tried run script on my MacBook-Pro$ python bmw.py
and it shows:
File "bmw.py", line 101
logging.info(f'last_percent loaded with a value of {last_percent}')
What i doing wrong? Thnx.

@tandrysek What version of Python?

MacBook-Pro$ python --version
Python 2.7.10

@tandrysek There's your problem. F-strings only work in Python 3. You can either use Python 3, or you can modify the code to remove the f-strings and use older methods of formatting your output. There may be other things that break in Python 2, though.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 234 ms

Resolved: ClamAV performance has improved in 4651989.


Allow multiple file patterns per filter in Windows file dialogue

Fixes #81781
Allows for multiple file types to be used in the same filter in Windows file dialogues. This allows for a consistent syntax with the built-in FileAccess and now allows filters like *.png,*.jpg,*.jpeg;Supported Images to work as intended.
windows_file_dialogue_test.zip

I would update the docs to describe the expected filter syntax, but I have no way to know if this does or doesn't work on other platforms, so the docs would be specific to Windows.

Is this change compatible with other platforms? Since you mentioned this as a windows only feature, either it should be documented as such or it should be flexible and support all platforms. I'd much prefer the latter in my opinion.

FYI, multiple filters support for all platforms is implemented in #81218.

For reviewers, see:

https://github.com/godotengine/godot/pull/81218
https://github.com/godotengine/godot/pull/81785

@bruvzg does your PR fix the issue in question?

Is this change compatible with other platforms? Since you mentioned this as a windows only feature, either it should be documented as such or it should be flexible and support all platforms. I'd much prefer the latter in my opinion.

This specific PR only implements the fix for Windows. I didn't realize there was a PR open that seems to be fixing this issue for all platforms.

Superseded by #81218. Thanks for the contribution!


Add Data Interpreter RAG integration Example

please add an example of best approach in integrating the RAG module into Data Interpreter without hindering the current architecture of the agent.

Examples are under development and will be provided later.

Hi! Thanks for your great work. I'm also wondering whether you could add this example for integrating RAG into Data Interpreter. It seems that there are only examples for RAG Retrieval and Rerank, but I have no idea how to utilize this ability into the current Data Interpreter Machine learning modeling procedure. Thanks!


How to disable background playing?

Hi, I'm trying to effectively disable backgorund playing. So when I navigate, the player just stops.
What's the way I should approach this? I was thinking of maybe creating a big new AudioRecorderPlayer() in index and relate to that instance of the module or what should I do?
I tried something like this
<Stack.Screen name="Home" options={{
          headerShown: true,
          title: '',
          headerStyle: {
            backgroundColor: '#212529'
          },
          headerLeft: () => (
            <HeaderBackButton
              label=""
              onPress={() => {
                audioRecorderPlayer.stopPlayer().catch((e) =>
                    console.log(`error while stopping player, ${e}`)
                );
                navigate('Home')
              }}
              pressColorAndroid="#00000000"
              backImage={(props) => <Image source={require("../../graphics/medinav_logo_no_bg.png")}
                                           style={{ width: 100, height: 40 }}/>}
            />
          ),

it doesn't work!

You may try to use AppState and stop player when it is on background


RootElement option is a function when calling appendChild

Hi,
I am trying to use the rootElement option, but the problem is that this gets transformed to an anonymous function, and therefore when appendChild gets called, an error is returned instead. I am using just by passing the DOM element to the method (and as shown in the examples), so I suppose I am not doing something wrong.
Happy to submit a PR if it helps, thanks!

Sorry, I just had a look at https://github.com/Caged/d3-tip/pull/175/files, this will probably fix the issue.

@Gbuomprisco There's still some issues that need to be resolved in that PR, if you have any advice about it that could help out a lot


PLEASE add cdist

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): 2.0b1
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
calculating distance matrices efficiently with tensorflow is a huge pain involving reading tons of stack overflow threads and re-implementing the same stuff. the solutions on stack overflow only cover euclidean distances and give MxM matrices even if you want city-block distance and MxMxD tensors ... it is extremely frustrating to experiment with optimal transport theory with tensorflow when such an obvious thing as CDIST is missing.
Will this change the current api? How?
Yes, it will be easy to calculate distance matrices
Who will benefit with this feature?
anyone doing molecular dynamics, optimal transport, wasserstein, shape matching, graph theory, etc etc
Any Other info.
PDIST is also potentially useful
https://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.distance.cdist.html
https://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.distance.pdist.html
Block-wise version would be IDEAL as huge distance matrices often overflow gpu memory

best i can do right now ... but it's impossible to do big problems ... im looking into tensor decomposition / factorization / compression to solve memory issues. ugh
@tf.function(input_signature=[tf.TensorSpec(shape=(None, D_FEATURES), dtype=tf.float16),
                              tf.TensorSpec(shape=(None, D_FEATURES), dtype=tf.float16)])
def get_distances(a, b):
    return tf.math.abs(tf.expand_dims(a, axis=1) - tf.expand_dims(b, axis=0))

https://github.com/ftonolini45/Variational_Sparse_Coding/blob/master/models/vae_utils.py
def cartesian(arrays, out=None):
    """
    Generate a cartesian product of input arrays.
    FROM http://stackoverflow.com/questions/1208118/using-numpy-to-build-an-array-of-all-combinations-of-two-arrays/1235363#1235363
    Parameters
    ----------
    arrays : list of array-like
        1-D arrays to form the cartesian product of.
    out : ndarray
        Array to place the cartesian product in.
    Returns
    -------
    out : ndarray
        2-D array of shape (M, len(arrays)) containing cartesian products
        formed of input arrays.
    Examples
    --------
    >>> cartesian(([1, 2, 3], [4, 5], [6, 7]))
    array([[1, 4, 6],
           [1, 4, 7],
           [1, 5, 6],
           [1, 5, 7],
           [2, 4, 6],
           [2, 4, 7],
           [2, 5, 6],
           [2, 5, 7],
           [3, 4, 6],
           [3, 4, 7],
           [3, 5, 6],
           [3, 5, 7]])
    """

    arrays = [np.asarray(x) for x in arrays]
    dtype = arrays[0].dtype

    n = np.prod([x.size for x in arrays])
    if out is None:
        out = np.zeros([n, len(arrays)], dtype=dtype)

    m = n / arrays[0].size
    out[:,0] = np.repeat(arrays[0], m)
    if arrays[1:]:
        cartesian(arrays[1:], out=out[0:m,1:])
        for j in xrange(1, arrays[0].size):
            out[j*m:(j+1)*m,1:] = out[0:m,1:]
    return out

def get_distances(xyz):
    xyz = tf.squeeze(xyz, 0)
    d = tf.reshape(tf.reduce_sum(xyz * xyz, 1), [-1, 1])
    return d - 2 * tf.matmul(xyz, xyz, transpose_b=True) + tf.transpose(d)

docs for desired method have link to source:
https://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.distance.cdist.html

these both work for us now, might require some tf.squeeze action for other applications
# @tf.function
# def get_distances(a, b):  # L1
#     print("tracing get_distances")
#     return B.sum(B.abs(tf.expand_dims(a, 0) - tf.expand_dims(b, 1)), axis=-1))

@tf.function
def get_distances(a, b):  # L2
    print("tracing get_distances")
    return B.sqrt(B.sum(B.square(tf.expand_dims(a, 0) - tf.expand_dims(b, 1)), axis=-1))

highly cool if fast distances were supported by TF team, also Sinkhorn / OT and Higher Order SVD, Normalized Compression Distance could be good loss for many many applications, but there's not a clear way to compress tensors for this metric in TF 2.0

Maybe this belongs in tf probability of tf addons, but unclear if it belongs in core TF given how easy it is to implement with TF.

@alextp note that this is not easy to implement at all. The version provided by bionicles uses much more memory than necessary to compute cdist as for pairwise distances between two sets of N and M objects in needs an array NxM. This is quadratic memory scaling, and it blows up very easily.
 If you wanted to compute the CDIST of the images in the sequence, the forward + backward pass of this operation would not fit on your GPU. However, it is in fact only necessary to store 100 x 256 x 256 x 20 values, which is well below a gigabyte. This is what scipy implements, and it far from easy for an average user.

I believe cdist can be implemented as a composition of existing TF operations (efficient cdist amounts to a batch matmul under the hood, right?), so it can live in addons.
@rmlarsen do you know if I'm being confused here?

I think that's right, for L2 distances a batchmul will do (but for other common distances it can be more tricky). I just wanted to note that it is rather non-trivial to implement for an average user and it would be helpful to have it since this thread seems to have concluded that the implementation is trivial.

One related question is if XLA could fuse reductions with cwise operations to avoid the blowup of memory.

This is the feature I've been waiting for years... Even if it could be easily implemented by users, although I didn't find it very easy, to include this feature in the official library will definitely benefit a lot of average users like me. I'm working on graphs and calculating similarity matrices using TF gave me quite a bit of pain...

I don't know where else to put this so I'll post my finding here. First I did find a pairwise distance function (I think by following linked issues here):
def pairwise_dist(A, B):
    na = tf.reduce_sum(tf.square(A), 1)
    nb = tf.reduce_sum(tf.square(B), 1)
    na = tf.reshape(na, [-1, 1])
    nb = tf.reshape(nb, [1, -1])
    D = tf.sqrt(tf.maximum(na - 2 * tf.matmul(A, B, False, True) + nb, 0.0))
    return D

If you want something that is maybe more memory friendly then I imitated what scipy does with this setup:
def distance(a, b):
    return tf.sqrt(tf.reduce_sum(tf.math.squared_difference(a, b), axis=-1))

def cdist(a, b, metric=None):
    a_i = tf.range(tf.shape(a)[0])
    b_i = tf.range(tf.shape(b)[0])
    indices = tf.concat([tf.expand_dims(x, -1) for x in tf.meshgrid(a_i, b_i, indexing='ij')], axis=-1)
    indices = tf.reshape(indices, [-1, 2])
    distances = tf.map_fn(lambda i: metric(a[i[0]], b[i[1]]), indices, fn_output_signature=tf.float32)
    distances = tf.reshape(distances, [tf.shape(a)[0], tf.shape(b)[0]])
    return distances

I highly doubt my solution is optimal but maybe somebody can improve it.

All the tools are already there imo:
Same as subtracting two vectors with shapes (n,1) - (1,m) is an (n,m) matrix,
subtracting two matrices X,Y with shapes (m,k,1)-(1,k,n), gives sqeuclidean (m,n) distance matrix by doing:
tf.reduce_sum((X  - tf.einsum('ijk->kji', Y))**2, axis=1)


Fix for ImportFileUpload model is tightly coupled to UI Issue 13792

This fixes #13792 and needs to be combined with this classic-ui PR.

@chessbyte Whoops, you're right, I completely missed the recommended suggestions somehow. Will fix.


Documentation Update

Can you please update the documentation here to include all the latest features of version 2.0.1, and can you specifically add to the documentation, the details regarding usage of jvmRoute in the server.xml.  Not only how/when to use it, but also how to generate unique jvmRoute when scaling up/down in ElasticBeanstalk.  Please update references to the JAR version of this package, and also any updates to the version of Tomcat supported.
Thank you!

This project has been archived. Please see the README.md for more detail on alternative projects.


minikube installed with amd64 binary instead of arm64 on M1

First, thanks for the great work behind arkade, this is awesome!!
I have just installed minikube with arkade on an M1 and the version installed was the amd64 on instead of the arm64. Here are the installation logs:
≻ arkade get minikube
Downloading: minikube
2022/06/11 15:34:11 Looking up version for minikube
2022/06/11 15:34:11 Found: v1.25.2
Downloading: https://github.com/kubernetes/minikube/releases/download/v1.25.2/minikube-darwin-amd64
68.67 MiB / 68.67 MiB [--------------------------------------------------------------------------------------------------------------] 100.00%
/var/folders/13/rk64jr55189989bqpkf7c53r0000gn/T/minikube-darwin-amd64 written.
2022/06/11 15:38:33 Looking up version for minikube
2022/06/11 15:38:33 Found: v1.25.2
2022/06/11 15:38:33 Copying /var/folders/13/rk64jr55189989bqpkf7c53r0000gn/T/minikube-darwin-amd64 to ...

Expected Behaviour

Download should come from https://github.com/kubernetes/minikube/releases/download/v1.25.2/minikube-darwin-arm64 for M1 chips.
Current Behaviour

When launching minikube version, I get the following message:
minikube version
╭──────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│                                                                                │
│    You are trying to run the amd64 binary on an M1 system.                                               │
│    Please consider running the darwin/arm64 binary instead.                                              │
│    Download at https://github.com/kubernetes/minikube/releases/download/v1.25.2/minikube-darwin-arm64    │
│                                                                                │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────╯
minikube version: v1.25.2
commit: 362d5fdc0a3dbee389b3d3f1034e8023e72bd3a7

Are you a GitHub Sponsor yet (Yes/No?)

[ ] Yes
[X] No

Possible Solution

I downloaded the binary manually
Your Environment

What Kubernetes distribution are you using?

kubectl version

Client Version: v1.24.1
Kustomize Version: v4.5.4

Operating System and version (e.g. Linux, Windows, MacOS):

MacOS
uname -a

Darwin XXX 21.5.0 Darwin Kernel Version 21.5.0: Tue Apr 26 21:08:29 PDT 2022; root:xnu-8020.121.3~4/RELEASE_ARM64_T8101 arm64

What arkade version is this?

arkade version

Version: 0.8.26
Git Commit: 613830dba8a5a8a529ce9ff2390b054b20fa6929

See: https://github.com/alexellis/arkade/pull/699.
Your assistance and input has been requested on that PR.  I hope you'll be able to oblige.

@rgee0 thank you for that. Sorry I came up late to help you

No problems. The change is in:
https://github.com/alexellis/arkade/releases/tag/0.8.27


Exp fix upload files error

Warnings

:warning:
Please include a description of your PR changes.

  Generated by :no_entry_sign: dangerJS against 7e35c13467774b8486be664a70ebd1e992ebe8c5


Create DefaultUrlResolver.java

Add ContextPath to url if application run with ContextPath value

You can't do that. You cannot cast to JEEContext in the DefaultUrlResolver.
Though, you can use the getPath() method from the WebContext interface.


Added verbose errors for "parsing error", uncompressed reader for algo=0

When a parsing error occurs there is no verbosity on what the actual error is, this makes the error message more verbose.
And when a packet has algo=0 use a regular io.reader interface without decompression.

We found this because a client of ours was sending us uncompressed encrypted files. Our clients usually have a lot of legacy systems preventing them from doing something like compressing files. Would love for this use-case to be merged. I think RFC 4880 says this MUST be implemented, but I'm not sure if ProtonMail has other reasons for not implementing this:
9.3.  Compression Algorithms

       ID           Algorithm
       --           ---------
       0          - Uncompressed
       1          - ZIP [RFC1951]
       2          - ZLIB [RFC1950]
       3          - BZip2 [BZ2]
       100 to 110 - Private/Experimental algorithm

   Implementations MUST implement uncompressed data.  Implementations
   SHOULD implement ZIP.  Implementations MAY implement any other
   algorithm.

Hi :wave: Thanks for the PR.
Re. the uncompressed algorithm, it's interesting that you ran into that. Typically, when creating uncompressed encrypted messages, you wouldn't create a compressed data packet at all, and just put the literal data packet there directly. My interpretation of RFC4880 is that that must be supported, not that a compressed data packet with uncompressed data packet must be supported (however, I'm not sure what the original intention was). That interpretation is also codified in the crypto-refresh. I also never encountered such packets before. But, if they're out there, we should probably add support for it, yeah.
Instead, regarding the error messages, the reason why they're called sensitiveParsingErr is because we shouldn't expose the error message in certain cases, to avoid decryption oracles. But, we could think about adding a "debug mode" someday that exposes them. But I would separate that from the rest of this PR.

@twiss I created a separate MR for just the algo=0 with an example. https://github.com/ProtonMail/go-crypto/pull/121

As for the sensitiveParsingErr being more verbose, it would be advantageous to have because we commonly get strange and archaic pgp payloads from clients who are using all manner of GPG libraries. In order to track down this specific issue with the algo=0 I had to recompile a checking tool that printed more verbose errors, as all we would see was  parsing error".


Installation Issues

installing on parrot OS
[+] Node is already installed.  Checking version...
[-] Current Node Version: v12
[!] Node 18 is NOT already installed.  Installing now...
[!] This can take 30+ minutes depending on your machine.
[!] Something went wrong!  Please try to run the installer again or open an issue on the repo...

This framework has only been tested on Kali.


Tutorial and publish this lib to pub.dev

Hi,
It's a great lib. Any chance you provide a tutorial, and publish this lib to pub.dev?
Thanks.

Don't know about a tutorial, don't think projects should document that much, instead i can write docs like dart/flutter ones (embedded with code). For pub.dev upload instead i don't know, should discuss it with the team


lib: add findFirstIndex

Motivation for this change
Can be useful (see tdesktop).
Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

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

Hi, did you try using lib.getVersion instead? If yes, what where the limitations?

My usecase was to separate string 1.2.3-4, taking 1.2.3 from it. I accomplish it now by finding -'s index and taking a substring.

I believe you can achieve what you want using this builtin:
nix-repl> builtins.parseDrvName "3.4.5-3"
{ name = "3.4.5"; version = "3"; }

stringsToCharacter is very inefficient so we are trying to avoid it. It seems like other packages depending on qtbase are also doing the same sort of things so it might make sense to export that mainVersion in the qtbase passtrough instead.

+1 for @zimbatm solution

Thanks for the idea! I've implemented this now.


[GLUTEN-1474][VL][Fix] Align with spark's behavior by allowing decimal in casting string to int

What changes were proposed in this pull request?
Set a config for velox to correct the cast behavior. The config is newly introduced in a velox patch.
How was this patch tested?
UT.

@jackylee-ch


Feature/mob 2732 company name ignores remote base locals value

Jira issue:
https://glia.atlassian.net/browse/MOB-2732
What was solved?
BaseLocale value of general.company_name is an empty string and this should be skipped.
This way the Company name fetch priority order becomes the following:

Custom locales
Local SDK configuration
Hardcoded string resource from local strings.xml

Release notes:

[x] Feature
[ ] Ignore
[ ] Release notes (Is it clear from the description here?)
[ ] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Double approve was by accident )


Handle terminal line wrap to avoid new line

Terminal usually wrap line in case it can't fit the current terminal width.
This behavior is not taken in account by indicatif and create some visual artifacts.
The issue was reported as #144 (closed with a workaround) and #520.
Please find below a demo without the patch:

Demo with the patch:

Using #144 example:

Seems like a nice improvement to me! @chris-laplante what do you think?
Would be nice to have some tests for this.

Makes sense to me too! I agree some tests would be nice. @RDruon are you able to implement a render test or two for this?

Seems like a nice improvement to me! @chris-laplante what do you think?
Would be nice to have some tests for this.

Makes sense to me too! I agree some tests would be nice. @RDruon are you able to implement a render test or two for this?

I added 2 render tests, one with ProgressBar and one with MultiProgress

@chris-laplante could I get another review? It seems several projects are interested in this change.

I think this is looking good, thanks! @chris-laplante feel free to follow-up if you have more thoughts.

Sorry for the delay! Looks good to me as well


Cost 300 navigation

Description of changes: Navigation for 300 labs
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Looks good @natbesh


GRAPHICS_API switch for Win32/Android/Linux

Select Graphics API this way:
cmake .. -G "Visual Studio 16 2019" -DGRAPHICS_API=D3D12

Need https://github.com/BabylonJS/BabylonNative/pull/950 to be merged before this one.
Vulkan status:
Current state: Playground runs and displays a green cube. Uniforms for VS are properly bound. For FS, it looks like uniforms are shuffled.

Cube appears green because vAmbientColor is full green. No idea why it happens.

@bghgary @darraghjburke Regarding D3D12/D3D11 shader compilation. I've put everything in 1 file and just added an #include per platform. Not great but does the job. Do you see anything better that can be done?

gltf2 samples (models and generated) are passing with mandatory BABYLON.MorphTargetManager.EnableTextureStorage = false;


Nice repo !

The whole "injector" source code is stored in plain text in the binary so finding IOC shouldn't be complicated

Hey, vercel/pkg packing was fun to reverse since it used brotli compression and created a virtual file system - I got a nice tool out of it and it took like an hour. Otherwise yea, it's easy to use signature based detection on your new version.
In any case, if you want to discuss anything I'd be happy to take it over another channel that is more appropriate than a GitHub issue. But to be honest, I think you've seen what my thoughts are on if it's "educational" or not, but who am I to judge. I'll just leave it at saying that your time is better spent on at least learning how to use Git properly.


[Hypershift] Arg validation for create cluster

* All private hosted clusters are privatelink clusters
* All hosted clusters are STS cluster

Fixes SDA-6873

@vkareh @pvasant Could you please take a look?

LGTM

/lgtm


Handle event types seperately

Current test for server being started correctly and running fails if handling an event type, such as "push" and it throws an exception (such as cloning repo without auth).
Create a function for each event to handle them seperately, and use a switch case to select it using the event type from the header X-Github-Event as input.

We already kind of do this, if this needs improvement, we need to make a new issue for refactoring it a bit.


update picture for elena

Docs Build status updates of commit 7b08d5a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

rd/media/profiles/elena-sinel.jpg
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Microsoft.Web/connections: Missing kind with V2

Resource Type
Microsoft.Web/connections
Api Version
2016-06-01
Issue Type
Type is unavailable
Other Notes
The property "kind" does not exist in the resource or type definition, although it might still be valid. If this is an inaccuracy in the documentation, please report it to the Bicep Team.
Bicep Repro
  resource LogicAppOffice365Connection 'Microsoft.Web/connections@2016-06-01' = {
    name: LogicAppOffice365ConnectionName
    location: Location
    properties: {
      displayName: 'Office 365 Outlook - Insight System Email User'
      statuses: [
        {
          status: 'Connected'
        }
      ]
      customParameterValues: {}
      nonSecretParameterValues: {}
      api: {
        #disable-next-line use-resource-id-functions
        id: '/subscriptions/${SubscriptionId}/providers/Microsoft.Web/locations/${Location}/managedApis/office365'
        type: 'Microsoft.Web/locations/managedApis'
        name: 'office365'

      }
    }
    tags: Tags
    kind: 'V2'
  }

Confirm

[X] I have read the troubleshooting guide and looked for duplicates.

Also missing properties.parameterValueType, properties.parameterValueSet


Migrated to MUI v5

This migrates the Demo site App to use MUI v5 but there are still dependencies that use Material-UI v4 like @backstage/core-components.
Note: this migration was pretty easy the place where I ran into the most issues was hitting and "theme.spacing is not a function" TypeError. The fix was making sure that the @mui/material and @mui/styles versions matched what we were using upstream. This comment on this issue was key to helping me solve this: https://github.com/mui/material-ui/issues/30611#issuecomment-1075090922

Closing for the time being based on some aspects still needing refinement.


Mention example Vagrantfile

Hello!
I apologize if external links are not preferred. I can also contribute this directly to this repo if that is preferred.
I thought it would be useful to point to an example Vagrantfile for use by casual users such as me :-)
Regards,
Swaroop

@swaroopch this is probably not in scope for the readme.
Quick note on your install.sh script: I think that it is preferable to use a Miniconda rather than a full anaconda distribution. Also, anaconda comes with its own libzmq, which has some binary compatibility issue.

@SylvainCorlay Thanks for replying.

I am not sure that this is in scope for the readme.

Okay, sure :)

Quick note on your install.sh script: I think that it is preferable to use a Miniconda rather than a full anaconda distribution. Also, anaconda comes with its own libzmq, which has some binary compatibility issue.

Aah, will look into this. Thanks for letting me know!


[BUG] async BackgroundTasks run in the foreground

Describe the bug
async BackgroundTasks are not running as background tasks. My response blocks until the task finishes.
To Reproduce
Steps to reproduce the behavior:

Create a file run.py with contents

from fastapi import FastAPI, BackgroundTasks

import time
import asyncio

app = FastAPI()

async def sleep_for_10_seconds():
    start = time.time()
    while True:
        now = time.time()
        if now - start > 10:
            break
        asyncio.sleep(1)
    print(time.time(), 'slept for 10 seconds')

@app.get("/")
async def read_root(background_tasks: BackgroundTasks):
    print(time.time(), 'read_root')
    background_tasks.add_task(sleep_for_10_seconds)
    return {"Hello": "World"}

Run with uvicorn run:app --reload
Run curl http://localhost:8000

Expected behavior
{"Hello": "World"} returns immediately. Instead it takes 10 seconds.
Environment:

macOS
fastapi==0.27.0
 uvicorn==0.8.4
Python 3.6.6 :: Anaconda, Inc.

Additional context
I originally thought this problem was confined to asyncio.ensure_future() in #403, but I'm getting the same behavior with fastapi.BackgroundTasks.
sync tasks return immediately (i.e. don't define with async).

try with def sleep_for_10_seconds(): instead of async def sleep_for_10_seconds():

@euri10 That may be a reasonable short term workaround, but I don’t think you should have to pay the threadpool overhead just to get background tasks to finish after returning the response.. If this is the behavior on the most recent version of FastAPI I think it merits a fix.
@ebanner I’ll look into this as soon as I can (hopefully tomorrow), but in the mean time could you confirm that it isn’t fixed by updating to the most recent version of fastapi?

/agree
you can also use it the Starlette way buy have to declare your JSONResponse and won't benefit from the docs goodies

import time
import asyncio

app = FastAPI()

async def sleep_for_10_seconds():
    start = time.time()
    while True:
        now = time.time()
        if now - start > 10:
            break
        asyncio.sleep(1)
    print(time.time(), 'slept for 10 seconds')

@app.get("/")
async def read_root():
    print(time.time(), 'read_root')
    background_tasks = BackgroundTasks()
    background_tasks.add_task(sleep_for_10_seconds)
    return JSONResponse({"Hello": "World"}, background=background_tasks)

@euri10 @ebanner I think I found the problem:
async def sleep_for_10_seconds():
    start = time.time()
    while True:
        now = time.time()
        if now - start > 10:
            break
        await asyncio.sleep(1)  # <-- HERE (was missing the await)
    print(time.time(), 'slept for 10 seconds')

After making this change, it seems to behave properly.

evil :+1:

Ah, my mistake!
Thank you for pointing it out.

Thanks @dmontagu and @euri10 for the help!
Thanks @ebanner for reporting back and closing the issue.

I realised if you replaced asyncio.sleep with time.sleep it blocks the request anyone has a reason why async background tasks does not run in separate thread and how to potentially solve this problem


bad video url leads to server error

Affected Version
SS 4.4.4
Description
Our client had a live video: https://vimeo.com/event/15595 of an event. Once the event is over, the video is no longer available. This leads to a server error!
Steps to Reproduce

add to source code:

RESULT ...

Thanks for raising this issue. I'm going to close it as a duplicate of https://github.com/silverstripe/silverstripe-framework/issues/9073 (given that has some comments already)

Thank you


error running the demo

I have clone the repository as is, however getting error:
Failed. card1.jpg could not be rectified :(
cv2.error: OpenCV(4.6.0) /io/opencv/modules/imgproc/src/shapedescr.cpp:280: error: (-215:Assertion failed) count >= 0 && (depth == CV_32F || depth == CV_32S) in function 'arcLength'

the fix I have implemented is to replace the line #204 in rectify.py
cnts = cnts[0] if imutils.is_cv2() else cnts[1]
with
cnts = cnts[0] if imutils.is_cv2(or_better=True) else cnts[1]

thx. i'll fix it later.

the fix I have implemented is to replace the line #204 in rectify.py
cnts = cnts[0] if imutils.is_cv2() else cnts[1]
with
cnts = cnts[0] if imutils.is_cv2(or_better=True) else cnts[1]

it worked, Thanks


Deployment on server

I have decided to use Google Colab for the Deployment Stage

Here I'm attaching the link
https://colab.research.google.com/drive/1YJYMOFNcoE1vZMzz0PiFSKvxfdQILY_o?usp=sharing


Provide a way to enable Default Point Aggregation

Part of https://github.com/CartoDB/product/issues/30.
We want to enable Default Point Aggregation in Builder/CARTO.js. Deafult Point Aggregation only works as long as the map dataset contains points. Maps API returns an error otherwise. The challenge here is that CARTO.js doesn't know anything about geometry types so it's not really possible to know when default point aggregation can be enabled.
The goal of this issue is to implement a way to enable Default Point Aggregation without without knowing the underlying geometry of a layer.
See Point 2 in this comment for more text.
Also, quoting @rochoa (https://github.com/CartoDB/product/issues/30#issuecomment-384020382):

Although it's far from ideal, I'm OK with your proposal about telling the Maps API to try to aggregate a layer —this requires changes in the Maps API— when not knowing in advance the geometry type.

cc: @dgaubert @jgoizueta @ivanmalagon

Wrong repo, sorry!


Update install_lightbits.sh - added 3.6.1b1503 userspace / rhl

Tested it that it works to configure and install.
Sidenote: my installation failed at the very end but its unrelated to the script so this script still works. it just got to the part where nodes load failed to start (this is related to my servers or previous LB install)

@kostia-lb looks good, can you add your update to the notes at the top of the script? Thanks!

LGTM


Consider renaming "Withdraw" to "Claim rewards"

Yesterday @Zheng-Shilin and I talked with a layman Desmos investor who was having some troubles with X.
He was trying to send some tokens but he didn't realise they were locked so we explained him that
he should have claimed the rewards and send those instead.
Here was when I realised that maybe it's not that easy to spot the way to collect the rewards if you're new to crypto-world. Plus, if you come from other wallets you would be more familiar with the "claim" word.
Probably we should consider to rename the "withdraw" with something more explicit like "Claim rewards".
What do you guys think?

I have been thinking about this as well. I wonder if this was a mistake for the wallets to the word Claim at the very beginning. @Rooty2020 has asked me when she first designed the buttons. I personally think this is not a claiming process as the rewards are already in the pool and that belongs to the delegator. The action is actually withdrawing my belongs from the pool but not claiming something so that I own such thing.
However, if the term would create confusion, we can change it to Claim if the end users don't care about the original meaning. This is similar to inflation which is wrong as the increase of supply doesn't mean the ecosystem is expanding but the number is for diluting the stake of non-staked DSM hodlers.

@kwunyeung The reasons you expressed are correct, in fact in the SDK it's called withdraw and not claim.
Personally it doesn't bother me much, but I would like to know what other people think.
@RiccardoM @ryuash @calvinkei

Personally I think it would be much better to use the complete "Withdraw rewards" (or "Claim rewards") phrase instead of just "Withdraw" or "Claim":

Withdraw is used inside CEX/DEX to mean the withdraw of the funds from the CEX into your personal wallet. Using this term might make the users think that X is a CEX/DEX
Claim is used when there are some airdrop. Using this term might make the users think that X is doing some airdrops

I would personally prefer seeing "Withdraw rewards" since this is also the same notation that is used inside the SDK CLI.

I've asked @rachelhox if "Withdraw" was intentional before as I found it hard to find the "Withdraw Rewards" button in the beginning.
Would agree we should be more direct in the wording over aesthetics

We can try to replace [withdraw] with [withdraw rewards].
I think it will be easier for our users to spot where they can claim rewards.

Please add your planning poker estimate with ZenHub @rachelhox


now ls error if no deployments exists

now ls with no deployments on that team
 Error! Unknown error: TypeError: Cannot read property 'deploymentId' of undefined
TypeError: Cannot read property 'deploymentId' of undefined
    at /usr/local/lib/node_modules/now/build/bin/now-list.js:117:48
    at Generator.next (<anonymous>)
    at c (/usr/local/lib/node_modules/now/build/bin/now-list.js:254:99)
    at process._tickCallback (internal/process/next_tick.js:109:7)

now version 5.0.0

Duplicate of #471


[rails6] Fix evm:dbsync:resync_excludes

In Rails 6.0, the following error was caused when trying to run migrations:
rake aborted!
PG::ConnectionBad: connection is closed
pg-1.2.3/lib/pg/basic_type_mapping.rb:115:in `server_version'
pg-1.2.3/lib/pg/basic_type_mapping.rb:115:in `supports_ranges?'
pg-1.2.3/lib/pg/basic_type_mapping.rb:119:in `build_coder_maps'
pg-1.2.3/lib/pg/basic_type_mapping.rb:402:in `initialize'
pg-logical_replication-1.0.0/lib/pg/logical_replication/client.rb:316:in `new'
pg-logical_replication-1.0.0/lib/pg/logical_replication/client.rb:316:in `typed_exec'
pg-logical_replication-1.0.0/lib/pg/logical_replication/client.rb:158:in `subscriptions'
manageiq/app/models/pglogical_subscription.rb:177:in `subscriptions'
manageiq/app/models/pglogical_subscription.rb:182:in `find_all'
manageiq/app/models/pglogical_subscription.rb:19:in `find'
manageiq/lib/acts_as_ar_model.rb:156:in `search'
.gem/ruby/2.6.6/gems/query_relation-0.1.1/lib/query_relation.rb:48:in `block in initialize'
.gem/ruby/2.6.6/gems/query_relation-0.1.1/lib/query_relation.rb:181:in `call_query_method'
.gem/ruby/2.6.6/gems/query_relation-0.1.1/lib/query_relation.rb:145:in `to_a'
manageiq/lib/tasks/evm_dbsync.rake:26:in `block (3 levels) in <top (required)>'
manageiq/lib/tasks/evm_dbsync.rake:35:in `block (2 levels) in <top (required)>'
rake-13.0.1/exe/rake:27:in `<top (required)>'
bundler-2.1.4/lib/bundler/cli/exec.rb:63:in `load'
bundler-2.1.4/lib/bundler/cli/exec.rb:63:in `kernel_load'
bundler-2.1.4/lib/bundler/cli/exec.rb:28:in `run'
bundler-2.1.4/lib/bundler/cli.rb:476:in `exec'
bundler-2.1.4/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
bundler-2.1.4/lib/bundler/vendor/thor/lib/thor/invocation.rb:127:in `invoke_command'
bundler-2.1.4/lib/bundler/vendor/thor/lib/thor.rb:399:in `dispatch'
bundler-2.1.4/lib/bundler/cli.rb:30:in `dispatch'
bundler-2.1.4/lib/bundler/vendor/thor/lib/thor/base.rb:476:in `start'
bundler-2.1.4/lib/bundler/cli.rb:24:in `start'
bundler-2.1.4/exe/bundle:46:in `block in <top (required)>'
bundler-2.1.4/lib/bundler/friendly_errors.rb:123:in `with_friendly_errors'
bundler-2.1.4/exe/bundle:34:in `<top (required)>'
bin/bundle:23:in `load'
bin/bundle:23:in `<main>'
Tasks: TOP => evm:dbsync:resync_excludes

This seems to be caused by the following Rails commit:
https://github.com/rails/rails/commit/0f0aa6a275876502e002c054896734d6536ba5cd
In which the tasks which are triggered after a db:migrate (db:dump in our case), caused the connection to be reset, rendering the cached pglogical connection as invalid.

The fix here simply updates the methods being called that this task relies on to do some error handling if the connection has been reset (in other words, raises the aforementioned error), and retry if it fails, but attempt a retry once before raising the original error.  If the connection has been severed for other reasons and isn't resolved with the first retry, we probably have bigger issues.
Links

Rails 6 Epic:  https://github.com/ManageIQ/manageiq/issues/19977
Current Cross Repo (includes these changes): https://github.com/ManageIQ/manageiq-cross_repo-tests/pull/179

Steps for Testing/QA
See https://github.com/ManageIQ/manageiq-cross_repo-tests/pull/179
(or rather, see that they are passing now)

@miq-bot assign @jrafanie
@miq-bot add_label rails6

Awesome, thanks for fixing this @NickLaMuro 👍

Just an FYI:  This seems to not be working in Rails6 with this fix in place:
https://travis-ci.com/github/ManageIQ/manageiq-cross_repo-tests/builds/205462258
So going to be looking into this further to see what is up... might have just mucked something up when doing the refactoring...
Regardless, a followup will be inbound for this.


v6web: mobile-friendly scrolling

Hello, love this project. It's super cool!
Here's a minimal fix for the tty scrolling issue on mobile. It's not perfect but it should make it useable for mobile browsers.

Hidden input changes:

Make #input fully transparent instead of offscreen, fixes mobile browsers scrolling left
Move #input inside of #bottom so default focus behavior gets us close to where we want to be

Scrolling changes:

When focusing input, use preventScroll: true, this prevents iOS Safari from zooming in
During tty switch scroll after a 200ms timeout, which gives the onscreen keyboard time to pop up. The magic number is gross and I tried a bunch of alternatives but this is the only approach I could get to work consistently 🥲.
Give #bottom a bit of negative left margin to that scrolling to it will leave some margin space on screen
Move #bottom to after the tty selection buttons; this keeps them in view and gives mobile browsers a bit of a vertical space buffer so that even if scrolling overshoots the y axis (seems to happen rarely, you can see it at the end of the video below) it will still keep the terminal cursor in view.

Video from iOS Safari:
https://github.com/rsc/unix/assets/131059/542083d3-8e33-4093-b073-3ef76d573830

Thank you!


Support for Google Chat

By @aslafy-z: https://github.com/argoproj-labs/argocd-notifications/issues/243
It would be great to see notifications to Google Chat directly integrated into Argo CD Notifications

argocd notification to google chat get error "Failed to notify recipient {googlechat om-argocd} defined in resource argocd/testing-new-app: error creating client to webhook: no Google chat webhook configured for recipient om-argocd using the configuration in namespace argocd" resource=argocd/testing-new-app" although have done everything in this docs https://argo-cd.readthedocs.io/en/stable/operator-manual/notifications/services/googlechat/ (already config trigger)


New version: DevelopedMethodsLLC.playit version 0.15.25

Pull request has been created with komac v2.6.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-09-23 19:51:15.640 [FAIL] Installer failed security check. Url: https://github.com/playit-cloud/playit-agent/releases/download/v0.15.25/playit-windows-x86-signed.msi Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 896.)

@wingetbot run


armnnTfParser does not support tf.Variable, so what could be alternative?

Hi, I'm trying to run tensorflow model on cortex-a53 board so trying to use armnn.
But I found tf.Variable method doesn't supported in armnnTfParser.
(from https://github.com/ARM-software/armnn/blob/master/src/armnnTfParser/TensorFlowSupport.md)
I'm not in professional at tensorflow but almost examples that I found are using tf.Variable method.
So my question is these.

What could be an alternative for tf.Variable in armnn?  And is there any plans for support tf.Variable?
Or do I have to use another way for armnn works?
For example, 1. make a graph for training 2. and save parameters after training done. 3.And have to
make another graph for armnn?

This is my self answer.
I missunderstood purpose of ARM_NN and tensorflow model export behavior.
ARM_NN is only for inference.
(https://developer.arm.com/technologies/machine-learning-on-arm/developer-material/how-to-guides/improving-your-machine-learning-workflow-using-the-arm-nn-sdk/single-page)
So all models from tensorflow which want to run with ARMNN have to exported as 'freezed_graph'; loading checkpoint and change all Variables to Const.
With this focus, ARM_NN don't need to support Variable, right?
I think now I'm understand  about ARM_NN concept.  But if there is any miss-understand or tips for me please comment me.
Thank you.

Hi @leeanzel that is right, ArmNN is only for inference (at least at the moment) so for Tensorflow models you need to freeze it, and probably run some "optimise for mobile inference" steps.
We're gradually increasing our support for Tensorflow models but these steps will likely be required for the foreseeable future.
Good luck with your project and please post back here with any news of success or further problems!

Hi @leeanzel,
As per Matthew comment, support for this feature will be added in the future. I'm marking this issue as resolved. Let us know if you have any other issues.
Telmo


Feat/#113

types define file development

good job!
If you don't mind, i recommend you to check "type"s.
I think you should use "interface" instead of "type", because types are duplicated.


Fix entry action search in saiapi.cpp.j2

The refactoring introduced in https://github.com/sonic-net/DASH/pull/438 only looks for a proper action attribute in SAI objects, not entries. In case of SAI entry, it iterates through all the attributes and matches the values regardless of attribute ID.
This PR aligns action search for both objects and entries, and adds action ID attributes to missing create_entry calls in tests.

hi Marian, do you mind to summary a bit on the background of this PR?

hi Marian, do you mind to summary a bit on the background of this PR?

Done.


fix: respect existing trailers in commit messages

Use git interpret-trailers (probably requires a fairly recent git version) to see if the original commit message already contains trailers and do not add an empty line before adding metadata if it does (as suggested in https://github.com/nodejs/node-core-utils/issues/602#issuecomment-1015172790). This should fix how git node land treats Co-authored-by lines.
No reordering happens. The generated metadata is appended to the existing trailer. My understanding is that the order of trailers is irrelevant, but if that's not true, we might want to revisit that part.
This particular part of the code base does not seem to be covered by any tests, so please review carefully.
Fixes: https://github.com/nodejs/node-core-utils/issues/602

I'm going to try it with some backports and V8 update

@targos Have you had a chance to test this yet? Looks like it'll be difficult to write tests for this part.

Have you had a chance to test this yet?

It's still on my todo list.


Fire widget.onChanged on EditableText value change

The value of an EditableText is composed of the text value as well as
other editing-related data such as selection-related information.
Previously, widget.onChanged() was only called for updates via
updateEditingValue(). For pastes via a TextSelectionOverlay, updates are
signalled via _handleSelectionOverlayChanged(), which only ever
triggered widget.onSelectionChanged(), but not widget.onChanged().
Both updateEditingValue() and _handleSelectionOverlayChanged() perform
the value update via _formatAndSetValue(), which is where this patch
moves the widget.onChanged() call.

Needs a test. @xster any thoughts on best way to test this? TextSelectionOverlay isn't publicly exposed at the moment, so a little tricky to shim in there. I don't think the clipboard is either for that matter, though we could fake out the bus like we do with some of the other services tests.

Fixes #11671

BTW, pastes supposedly triggered form validation on Android but not iOS.  Donno if this change risks causing double-validation on android.

@eseidelGoogle this patch ensures that FormFieldState.onChanged is called by the framework on paste, but is independent of the validation, which happens either on submit (in general) or on build (when autovalidate is true).
Originally this change did result in onChanged being called twice on paste on Android though. I've updated to avoid triggering onChange unless the text content has actually changed. I'll take a look to see if we can eliminate the restartInput call that currently happens for Android in the engine, which triggers an extra TextInputClient.updateEditingState platform message today (details on #11671).

@cbracken Are you still working on this one? It looks like Travis found a test problem.

@Hixie Definitely want to land this. The failing test uncovered a legitimate bug in form reset(): #11713. Once that's fixed I'll land this.

This was landed in #12273.


Update offline versions file on successful list-remote calls

Why? 🤔
We need this change because the offline versions file will be outdated after some time since it is a static file. See #35
What? :hammer_and_wrench:
Update the offline version file when listing remote versions is successful.
Additional Links 🌐

@HeikoNeblung Have you been tested this changes? Works?
@iokiwi Are you see some possible issue if merge?

@weyderfs I've tested the changes i've made successfully. Every time the listing of the terragrunt repo tags is successfull, the offline file will updated.


saveWebViewAsPdf doesn't generate the PDF

I tried this first in a Swift project. I added the library via CocoaPods.
platform :ios, '8.0'
use_frameworks!

pod 'BNHtmlPdfKit', :git => 'https://github.com/brentnycum/BNHtmlPdfKit'

Here's the code.
var htmlPdfKit: BNHtmlPdfKit!

override func viewDidLoad() {
    super.viewDidLoad()

    webView.loadRequest(NSURLRequest(URL: NSURL(string: "https://www.google.com/")!))
}

@IBAction func saveButtonTapped(sender: UIBarButtonItem) {
    let documentsDirectoryPath = NSSearchPathForDirectoriesInDomains(.DocumentDirectory, .UserDomainMask, true).first!
    let documentsDirectoryPathURL = NSURL(string: documentsDirectoryPath)!
    let pdfPath = documentsDirectoryPathURL.URLByAppendingPathComponent("Google.pdf")
    print(pdfPath.absoluteString)

    htmlPdfKit = BNHtmlPdfKit()
    htmlPdfKit.delegate = self
    htmlPdfKit.pageSize = BNPageSizeA4
    htmlPdfKit.saveWebViewAsPdf(webView, toFile: pdfPath.absoluteString)
}

When the save button is tapped, the PDF isn't generated. Nor any of the BNHtmlPdfKitDelegate methods get called.
However other methods like saveUrlAsPdf and saveHtmlAsPdf do work. I tested this in Xcode 7.1 with iOS 8.4 and iOS 9.1 simulators.
I've uploaded a demo project too.
Later I tested this in a Objective-C project as well. And it didn't work either. So I guess this isn't a language issue.

@Isuru-Nanayakkara did you get to fix it?

@vinbhai4u Nope, I had to resort to saving via a HTML string.

@Isuru-Nanayakkara even that doesn't work for me,
    generateBNHtmlPDF(){
    let pdfKit = BNHtmlPdfKit()
    pdfKit.delegate = self
    let toFile = Path.documentsDir["file.pdf"].path_string
    let htmlString = Path.documentsDir["BSM.html"].readString()
    pdfKit.saveHtmlAsPdf(htmlString, toFile: toFile)
    print("open " + toFile)

is there something wrong?

i also can not generate pdf from an webView, follow the code, i add the save method in the - (void)saveWebViewAsPdf:(UIWebView *)webView toFile:(NSString *)file;
And it work now. i am not sure if this is correct ,but the doc has no use case on this .
-(void)saveWebViewAsPdf:(UIWebView *)webView toFile:(NSString *)file {
[[self class] cancelPreviousPerformRequestsWithTarget:self selector:@selector(_timeout) object:nil];
self.outputFile = file;
webView.delegate = self;
self.webView = webView;
//add for test
[self _savePdf];
}

Hey, I have solved the issue while generating pdf file from webview --saveWebViewAsPdf(webView, toFile: pdfPath.absoluteString) and now delegates are being called properly with the path of saved pdf file.
Actually I have done my changes after forking the project. Infact I have make a test project for this but unfortunately I don't know how to push my changes.


Add sounds to playback

At the moment you can see which notes are played (both on the score and on a drum kit) but you can't hear any. It'd be great if you could hear drum sounds at the right times.

Code updated to use phaser.io so that sounds can be mixed together. There's an option to enable audio playback:


Changes are being accepted when shouldn't

I have added buttons to accept or reject changes in a toolbar:
<button
  onClick={() => editor.chain().focus().acceptChange()}
  disabled={!editor.can().chain().focus().acceptChange()}
  >
  accept
</button>

this is a main approach to construct toolbar with commands.
The issue: when this button is there, inserted content goes accepted right after typing, but the deleted content — on putting caret in the deleted area.
I have also tried to do like this
<button
  onClick={() => editor.commands.acceptChange()}
  disabled={!editor.commands.acceptChange()}
  >
  accept
</button>

Same result, also throws an error:
Uncaught RangeError: Applying a mismatched transaction

The Demo doesn't have an example of usage of these functions.

btw there is a typo in documentation, should be acceptAllChanges and rejectAllChanges, "-s" is missing.

AcceptChange is a method call, not a state. Maybe you put it in your disable status cause it was caculated(invoked) sometime?
It seems we need expose a computed status to outside. I'll try it latter.
Thanks

I'm working on my own track changes plugin, so I cannot check the issue again.


Remote validation does not work when wrapped in a custom directive

Hello,
Firstly, thank you for this wonderful plugin! It has truly been very helpful!
We have a custom directive that wraps your plugin so we don't need to repeat the implementation details on every page and Angular controller. The problem is that we are trying to use your remote validation feature to validate that a certain value is valid before the user clicks save, but it's not working from within our directive. The validation call seems to be called multiple times from different scopes, and only one of them has access to the remote validation function.
I have replicated the issue in this Plunker. See the console to see the errors we're encountering.
Please, if you have some time, could you help me figure out if we're misusing your plugin or if there's something else that's the cause of this?

You might have to use Wiki - isolatedScope for that to work.
Not sure where the error comes from but you could troubleshoot with the unminified code, that would help finding what is throwing the error. So if I replace the call to Angular-Validation with the following unminified code.
<!-- angular-validation -->
<script type="text/javascript" src="https://rawgit.com/ghiscoding/angular-validation/master/src/validation-directive.js"></script>
<script type="text/javascript" src="https://rawgit.com/ghiscoding/angular-validation/master/src/validation-service.js"></script>
<script type="text/javascript" src="https://rawgit.com/ghiscoding/angular-validation/master/src/validation-common.js"></script>
<script type="text/javascript" src="https://rawgit.com/ghiscoding/angular-validation/master/src/validation-rules.js"></script>

After doing that, I can see that your Promise seems invalid, since it throws an error on line 1338 of validation-common.js file. The line of code is this:
if (typeof previousPromise.abort === "function") {
    previousPromise.abort(); // run the abort if user declared it
}

In my code, I forgot to check that the Promise isn't undefined (funny enough, I did that check on line 1345 but forgot line 1338). However that won't fix your problem, but at least you know what throws the error.
if (!!previousPromise && typeof previousPromise.abort === "function") {
    previousPromise.abort(); // run the abort if user declared it
}

I think that Angular-Validation tries to find the function and the Promise in the current scope but fails to do so. I can correct the missing undefined try-catch, but you still have to fix your issue. I believe that for this to work, you have to find the way to pass the scope of the parent to the directive, or at least the scope that contains the remote function should be the same scope called by the validation.

I appreciate you taking the time to look into this issue.
I made that change you mentioned (checking for an undefined Promise) locally until the source code is changed. I have been trying all day to utilize the isolated scope you mention, but I still end up getting the same "Remote Validation requires a declared function (in your Controller) which also needs to return a Promise, please review your code" error.
I'm pretty sure it's just a scoping issue, but I don't know where/how to ensure that your call to validate always has access to the proper scope. Do you have any additional suggestions?
Again, thank you very much.

Ok, I don't want to push any new revision with such a small code change (and that doesn't bring any good to the table anyway).
As for the scoping issue, I do believe that is the issue. Using isolatedScope is a way to make sure that Angular-Validation uses the same scope for everything, that is helping doing validation in Modal window for example. Looking at the code, I can guide you to do some troubleshooting, but I would rather not do it myself if possible. Here's first thing you can check, put a breakpoint on line 1332 which is this line of code var promise = runEvalScopeFunction(self, fname);, I would expect the self, that is self.scope, (but I bet that it's not) to hold the same scope as your parent scope (hopefully also the same scope as where you defined your remote function). The function runEvalScopeFunction() does an $eval to transform a simple string into a javascript object/function depending on what it finds and in your case that function probably returns null because it didn't find it. Start with that and give me the result then we'll go further.

So I've added the $scope.$validationOptions { isolatedScope: $scope }; call to our controller, as seen in this updated Plunker. The call to runEvalScopeFunction(self, fname); is now being consistently called from our custom directive scope (though it's still being called twice, which is not ideal because this validation fires AJAX calls to our server).
The $validationSummary and call to checkFormValidity() are now not working though.
We have decided to not worry about getting client-side AJAX validation for this particular field to work and simply provide our server side validation.
Thank you for your time and help--you may close this issue.


added underscore as valid symbol for domain

Thank you for making this!
There are sites with the underscore in the name, so I think it should be correct pattern
Examples:
http://jurnalsda_pusair.pu.go.id
http://contributii_botanice.reviste.ubbcluj.ro
http://nova_scientia.delasalle.edu.mx

Thanks! This is a great catch, and you are absolutely correct that underscores fully compliant with RFC 2181 for domain names. I appreciate the fix, and I'll be reviewing it for inclusion in the next release.

Cool!
When can I expect a release with this fix?

I'm hoping to get the new version released before Monday - there's still one bug showing up in Python 2.7 for a different bug fix related to domains and URLs in #26 which I need to resolve.

@mastak : Just an FYI in case you're not subscribed to notifications, v.1.3.2 with the underscore fix included has just been released.

Excellent, thank a lot!


@mui/material: incorrect responsiveFontSizes return type when using CssVarsTheme as input

Steps to reproduce
No response
Current behavior

Expected behavior
responsiveFontSizes should return the same type of theme that it was given
Context
Switching to the CSS vars theme provider (primarily for dark mode), but still wanting to use the responsive fonts helper.
For now I am simply casting the return type back to CssVarsTheme, but the below change to packages/mui-material/src/styles/responsiveFontSizes.d.ts fixes the issue:
import { Breakpoint } from '@mui/system';
import { Typography } from './createTypography';

export interface ResponsiveFontSizesOptions {
  breakpoints?: Breakpoint[];
  disableAlign?: boolean;
  factor?: number;
  variants?: Array<keyof Typography>;
}

export default function responsiveFontSizes<T extends { typography: Typography }>(
  theme: T,
  options?: ResponsiveFontSizesOptions,
): T;

Your environment

npx @mui/envinfo
  System:
    OS: macOS 11.6.5
  Binaries:
    Node: 20.10.0 - ~/.nvm/versions/node/v20.10.0/bin/node
    npm: 10.2.3 - ~/.nvm/versions/node/v20.10.0/bin/npm
    pnpm: 8.15.4 - ~/.nvm/versions/node/v20.10.0/bin/pnpm
  Browsers:
    Chrome: 126.0.6478.116
    Edge: Not Found
    Safari: 14.1.2
  npmPackages:
    @emotion/react: ^11.11.4 => 11.11.4
    @emotion/styled: ^11.11.5 => 11.11.5
    @mui/base:  5.0.0-beta.40
    @mui/core-downloads-tracker:  5.15.20
    @mui/icons-material: ^5.15.20 => 5.15.20
    @mui/material: ^5.15.20 => 5.15.20
    @mui/material-nextjs: ^5.15.11 => 5.15.11
    @mui/private-theming:  5.15.20
    @mui/styled-engine:  5.15.14
    @mui/system:  5.15.20
    @mui/types:  7.2.14
    @mui/utils:  5.15.20
    @types/react: ^18 => 18.3.3
    react: ^18 => 18.3.1
    react-dom: ^18 => 18.3.1
    typescript: ^5 => 5.5.2

Search keywords: theme, css, responsive

@jxdp That looks good to me, do you want to submit a PR?

@jxdp That looks good to me, do you want to submit a PR?

Sure thing: https://github.com/mui/material-ui/pull/42786


feat: Make audio playlist div full-width

Modified the main container to use w-full for full-width display
Removed margin classes to allow the div to span the entire page width
Enhanced list item styles for better hover effects and user experience
Added a play/pause button with consistent styling
-https://deploy-preview-25--lisbook.netlify.app/

Okay sure @2MJ-DEV