Reimplement String.erase() as immutable method

Reimplements String.erase() as an immutable method that was removed in #54869 as it was mutable in 3.x
See godotengine/godot-proposals#6598 for more information.
Production edit: closes https://github.com/godotengine/godot-proposals/issues/6598

Should length default to 1, an often used case

Also I think there should be checks added to prevent confusing error messages coming from left and substr if the bounds are incorrect

Also while it was present in the original implementation you can just use -1 for the length of substr as that gives the exact same result

With p_chars negative you get no change, so not an error as such but still wrong

Like so:
"ABCDEF".erase(2, -1) gives "ABBCDEF"

Yes since substr will start before p_pos. As I'm not sure how this should be fixed and it's a bug in 3.5 as well I wasn't sure if it shouldn't be it's own issue + PR that can then be cherry-picked for 3.6.

No it's not a bug in substr you just need to make sure p_chars is positive

it's not a bug in substr but in erase that also existed in 3.x. That's what I'm getting at.

How about this then:
String String::erase(int p_pos, int p_chars) const {
	const int start = MAX(p_pos, 0);
	return left(start) + substr(MAX(p_pos + p_chars, start));
}

That works I think

(also my bad the comment you quote first I deleted as I realised it wasn't the case)

Don't think so, it's not broken just useless

For posterity, erase() was removed in https://github.com/godotengine/godot/pull/54869 as it conflicted with the new binding system (it needed to be made immutable), we opted to remove it as it was not commonly used in the engine. Clearly it is still needed by users, so this is the way to bring it back :)


Initialized vocabs, mappings and docs folders

For now yes. Workbench modules is code and thus licensed using Apache 2. All others modules as well as mappings and vocabs are content and thus licensed CC-BY-4.0.
We have mainly one license when the workbench will be externalised.
Please propose another way to arrange the licenses between code and content.

Root README file has a typo in the link to Apche 2.0 license, can you fix it in this commit as well? Otherwise it works for me, it is one of the standard practices to put the license file in the corresponding subfolder and info in the Readme file.

done.


AntiForgeryToken is reusable. Is there any configuration to make it unique on every request?

AntiForgeryToken is reusable. Is there any option/configuration to make it unique on every request and invalidate the previous ones?

Can you create an issue from the doc page? That will connect the doc to the issue. Go to the bottom of the page and select This page .


Choose license

We need to choose a license for this repo. and mention it in readme too, I would vote for MIT. @PeerHerholz  what do you say?

Hey @ofgulban,
thx for starting this topic.
Usually I would say: as open as possible, but as we're going to use data from others (e.g. the extracted ROIs from different publicly available atlases) I guess we have to adapt to that. Hence, checking their licenses and see from there.
That being said, does MIT also cover data, etc. ?
Furthermore it could be useful to have different license types for different parts of the repo:
the analyses scripts, ROI and tutorials are going to be included in the docker image. The stimulation scripts maybe, maybe not. For example thinking of FSL and it's license (info here). Kinda the same with FreeSurfer. As the docker image/toolbox will use both quite prominently we need to include those. I'll talk to some peeps with more experience in licenses, etc. and will post the answer here.
Would you be okay with that?

I am not sure whether MIT covers data. Probably it would be good to have a script to download the atlas data from their sources instead of including them directly. We might consider using duecredit at some point too.

it could be useful to have different license types for different parts of the repo
+1

I am ok with any decision, I think any license is better than having none.


Add .env to Node.gitignore

dotenv is sufficiently popular in the Node world (over 1200 dependents in the npm registry, over 2400 stars on GitHub), along with other tools that can utilize .env files like node-foreman, to make this a valuable, security-focused convenience. As a precedent, .env is already included in the Python, Ruby/Rails and Laravel .gitignore files.

@noinkling thanks!


Possibility of overriding the default test directory location on Windows

Hey,
I was wondering if it's possible to control the location of the test directory.
From what I can tell, setting the --output-dir sets the parent directory for all build-related environments and work directories.

As shown in the figure below, the work directory and build_env/host_env environments are present underneath the --output-dir/bld/<package-name-id>

But for some reason only the test environment is located under the --output-dir/bld/<package-name-id>/work

And the tests are moved to a local temporary directory %AppData%\\Local\\Temp\\.tmpAUNvtX\\conda_build_script.py.
In certain windows enterprise configurations, the user might not have the privileges to run code inside %APPDATA%. So I was wondering if it's possible to get more fine-grained control over the test directory location?
Let me know if you need more information to make sense of my request :)
Best Regards
Kristoffer

Suggestions / PRs very welcome! :)

@wolfv I can give it a shot. I guess it boils down to how we'd like to manipulate tmp_dir in https://github.com/prefix-dev/rattler-build/blob/main/src/package_test/run_test.rs#L118 either through using a env var passed as a flag to the cli.

@wolfv so I made some extremely ugly hacks that at least managed to move the test directory to a subdirectory inside --output-dir in https://github.com/prefix-dev/rattler-build/compare/main...Krande:rattler-build:fix/windows-test-directory.
I'll try to find time between other responsibilities to look into this. But if you see any immediate opportunities based on this, feel free to take a stab at it. I can also make a PR with what I have if that's a more convenient place to continue to the conversation :)


In ubuntu no such command "TBD" please update your repo. or write its compatible to windows development only....

TBD is for 'To be decided/ To be done'


BLOB shared key authentication

Implemented shared key authentication scheme for BLOB service.
Authentication scheme described here:
https://docs.microsoft.com/en-us/rest/api/storageservices/authentication-for-the-azure-storage-services
To enforce signature verification one should start the emulator with '-a' option.

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

Hi @vitaly-goot ,
many thanks for your patience.
Sorry, after fat fingering the review, I found that the issue is actually in our documentation of the API.
The some framework milddleware normalizes the headers for us.
In https://docs.microsoft.com/en-us/rest/api/storageservices/authenticate-with-shared-key, we specifically state that we should include 0 when evaluating content length, however this causes both the storage explorer and storage SDK tests to fail.
I'm figuring this out internally, and will update the pull request and then merge it.

Manually merged with changes.


Bluetooth state of iOS device is 'Unknown' for first time access

Hi,
We access the bluetooth state of iOS device using CrossBluetoothLE.Current.State.
Bluetooth is turned on in iOS device. When we access the bluetooth state using the above mentioned API, it returns 'Unknown' for the first time.(but iOS device's bluetooth is turned on).
When we access the same API again, it returns proper state 'On'.
Is this the proper way of accessing the bluetooth state of the device.?
or, are we missing anything here?
When does the bluetooth state get updated to On ?
Can you please help us?

hi,
We provided a issue template...
https://github.com/xabre/xamarin-bluetooth-le/blob/master/ISSUE_TEMPLATE.md
Pls provide the Information like device, version, os, ...
For the next time: fill it out please.

@smstuebe sure.
Steps to reproduce
IBluetoothLE _bluetoothLe = CrossBluetoothLE.Current;

Turn on bluetooth in iOS device. Use _bluetoothLe.State API to know the bluetooth status of iOS device. Register for bluetooth state change event _bluetoothLe.StateChanged

It returns the bluetooth state as 'Unknown', though bluetooth is turned on in the iOS device.

Now, bluetooth state changed event gets fired. the state is on.

Call the API CrossBluetoothLE.Current.State again. It returns the state as 'On' properly.

Expected behavior
When we access the bluetooth state for the first time, it should return the proper value as 'On'
Actual behavior
Bluetooth state is not reflected properly for the first time.
Configuration

Version: 9.3.1
Platform: iOS
Device: iPhone 5S.

Version: 10.1.1
Platform: iOS
Device: iPhone 6S.
We are using latest version of Plugin.BLE

I've been struggling with this as well without using a plugin.
What I think happens is that the CBCentralManager will return the Unknown state if the initialization hasn't finished yet.
By the look of it (code of the plugin you're using) they don't init() CBCentralManager so this explains the unknown state:
protected override void InitializeNative()
{
_centralManager = new CBCentralManager(DispatchQueue.CurrentQueue);
_centralManager.UpdatedState += (s, e) => State = GetState();
}

Okay, so I checked the Plugin.BLE source code and played with it a bit and found an inconsistency between Android and iOS state detection that Plugin.BLE does not fully compensate.
While on Android we can get state immediately, on iOS initial state is unknown and delivered later via state update event. This is alike CBCentralManager which also delivers states via a delegate.
I wrote a simple workaround which is a solution for issues #188 and #112. I tested it on both iOS and Android, using iOS 11 IP7 and Android 7 Samsung Galaxy S7.
/// <summary>
/// Checks the bluetooth adapter if it is on or off.
/// </summary>
private async Task CheckBluetooth() {
    // iniial state check
    var state = await GetBluetoothState(Plugin.BLE.CrossBluetoothLE.Current);
    if(state == BluetoothState.Off)
    {
        ShowDisabledBluetoothAlert();
    }

    // tracking state in case if user disable his bluetooth
    Plugin.BLE.CrossBluetoothLE.Current.StateChanged += (o, e) =>
    {
        if(e.NewState == BluetoothState.Off)
        {
            ShowDisabledBluetoothAlert();
        }
    };
}

/// <summary>
/// Get current bluetooth state
/// </summary>
/// <param name="ble">Crossplatform bluetooth implementation object</param>
/// <returns>BluetoothState</returns>
private Task<BluetoothState> GetBluetoothState(IBluetoothLE ble)
{
    var tcs = new TaskCompletionSource<BluetoothState>();

    if (ble.State != BluetoothState.Unknown)
    {
        // If we can detect state out of box just returning in
        tcs.SetResult(ble.State);
    } else
    {
        // Otherwise let's setup dynamic event handler and wait for first state update
        EventHandler<Plugin.BLE.Abstractions.EventArgs.BluetoothStateChangedArgs> handler = null;
        handler = (o, e) =>
        {
            Plugin.BLE.CrossBluetoothLE.Current.StateChanged -= handler;
            // and return it as our state
            // we can have an 'Unknown' check here, but in normal situation it should never occur
            tcs.SetResult(e.NewState);
        };
        Plugin.BLE.CrossBluetoothLE.Current.StateChanged += handler;
    }

    return tcs.Task;
}

/// <summary>
/// Show alert to user that his bluetooth is disabled
/// </summary>
private void ShowDisabledBluetoothAlert()
{
    App.ShowAlert("Bluetooth disabled", "Bluetooth required to work with BLE devices.");
}


Draft / WIP: Add latest Airbyte loader (do not merge)

This is still a work in progress...

Hey @aaronsteers  How are you?
We moving away from the config file for 1.0. We might come back with another one until the main release but this one won't work for new components.
We could get to a solution together since Airbyte is a very powerful tool.
First thing is that all new components should be CustomComponents (we'll update the docs on this in the coming days).
We could add the loader in the src/backend/base/langflow/components/documentloaders, or, depending on the amount of dependencies it could add, we could create a generic file/data loader using airbyte and decide somewhere else to put it in.

Hi, @ogabrielluiz! Thanks very much for this info. Once the docs and new process are updated, let me know and I'll be happy to support setting up the new integration.

Ok. I can probably get it written by tomorrow.
Essentially, you'd add your component, add the dependency to the outer pyproject.toml, and test it.


Error: Cannot find module './match'

When deploying to AWS ElasticBeanstalk, Node 0.12.2, React-Router 1.0.3, I am getting the following error when starting the server:
Error: Cannot find module './match'
at Function.Module._resolveFilename (module.js:336:15)
at Function.Module._load (module.js:278:25)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at Object. (/var/app/current/node_modules/react-router/lib/index.js:96:15)
at Module._compile (module.js:460:26)
at Module._extensions..js (module.js:478:10)
at Object.require.extensions.(anonymous function) [as .js] (/var/app/current/node_modules/babel/node_modules/babel-core/lib/api/register/node.js:214:7)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
Interestingly enough, I do not encounter this issue locally when using NVM 0.12.2. However, it does seem like the require('./match') is case-sensitive on my AWS instance (but not OX X) and is therefore not finding lib/Match.js because it is looking for lib/match.js
Seems pretty minor but definitely causing some headaches over here!

Thanks for your question!
We want to make sure that the GitHub issue tracker remains the best place to track bug reports and feature requests that affect the development of React Router.
Questions like yours deserve a purpose-built Q&A forum. Could you post this question to Stack Overflow with the tag #react-router? https://stackoverflow.com/questions/ask?tags=react-router
We also have an active and helpful React Router community on Reactiflux, which is a great place to get fast help with React Router and with the rest of the React ecosystem. You can join at https://discord.gg/0ZcbPKXt5bYaNQ46.

UPDATE: This is indeed a bug with v1.0.3, on Linux. Linux being case-sensitive, unlike OS X, causes an error when trying to require './match' because the file name is Match.js.
Renaming Match.js to match.js should fix this problem on Linux.
This is especially problematic on ElasticBeanstalk with AWS, which by default re-installs all node modules for each deployment.

There's something messed up in your environment. It is in fact package/lib/match.js and package/es6/match.js in the tarball, with that exact casing.

When installing via npm it is es6/match.js and lib/Match.js, haven't checked the tarball

Again, I'd suggest double-checking your deployment. Unless you think that you are literally the only person in the world running React Router on Linux (hint: you're not), this is an error on your side. Please reach out elsewhere for support – I've attached a number of links upthread.

After hoping that 2.0 would just put this to rest, I am still having an issue and there has to be something wrong with the package. After running npm install locally on OS X I get lib/match.js, when npm install is running on Linux (AWS ElasticBeanstalk) it is lib/Match.js and triggers an error when running npm start, saying the module could not be found.
After terminating the environment, rebuilding and even deploying a new test environment I am still getting the same behavior. I SSHed into the instance, simply change index.js to require('./Match') instead of './match' and it works.
What I am most confused by is why I'm getting different filenames in different environments after confirming both are react-router 2.0.
Truly baffled by this one, and is a major problem due to the inability to deploy without manually SSHing into my instance and change the require statement. Any ideas what could be happening?

What version of node/npm do you have installed? You should see the lowercase match.js in https://registry.npmjs.org/react-router/-/react-router-2.0.0.tgz

I'm also getting this same issue on AWS EC2 (linux) where match.js is located at node_modules/react-router/lib/Match.js. I'm not really sure how or why. One quirky thing I've done is I installed the module on OS X, committed it to my git repo, then pulled it on my linux server. This is probably not the recommended way of doing things.
I did the following and this solved my issue:
rm -rf node_modules/react-router
npm install


Improve UX with Confirming Payments

Look into ways to improve the process while keeping it just as secure.

Closing due to inactivity


feat: Create MEAN booster

EDIT Tasks added by @lance
Once the initial repository is created in step 1, many of these tasks can be added as individual GH issues to that repo.
The goal with this task is to make it very simple for Node.js users trying out Kubernetes or OpenShift to be able to get a multi-node cluster up and running quickly.
Initial tasks:

[x] create a new repository for the booster
[x] copy existing booster project (probably the CRUD booster) to the new repo
[x] before moving on, build this and ensure it works in minishift
[x] add a node to the project which exposes mongodb (see the circuit breaker booster for a multi-node booster example)
[x] test against minishift
[ ] add documentation to the openshiftio/appdev-documentation repo
[ ] add project to the openshiftio/booster-catalog
[ ] review and approval from @lance, @lholmquist and/or @gastaldi
[ ] submit pull requests for both of the above projects

/cc @balunasj

closing this and keep discussions about coolstore-microservices migration to Node.js.


adds emailautomator task and whenever gem to automate database scrubb…

…ing, abstract fetching, and emailing on a schedule

that config file is unimportant, leftover from mailgun test


fix(website): remove symlinks in translated about pages

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug
Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area documentation
What this PR does / why we need it:
Fixes the about page being redirected to another language:

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
Once this PR has been reviewed and has the lgtm label, please assign vjjmiras for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

content/OWNERS

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

LGTM label has been added.  Git tree hash: 97f2aa849c0be9347ab72e335be0ef356cf8a270

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [leogr]

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


Godep fails to parse dependencies when ._ files are present.

Running godep on Linux, on a path that has shared over sshfs with OSX, sometimes a ._foo.go is created by OSX for various reasons.
When this happens, godep save will report zero dependencies for the entire package.
No error is reported, just godep save -v will show an empty list of dependencies.

It's probably an invalid go file, likely a temporary editor file. Godep should skip parsing files that start with '.' and it should return parsing errors to the user. I'll see about fixing at least the former a little later today.
Sent from my iPhone

On Mar 27, 2016, at 23:03, Paul Querna<EMAIL_ADDRESS>wrote:
Running godep on Linux, on a path that has shared over sshfs with OSX, sometimes a ._foo.go is created by OSX for various reasons.
When this happens, godep save will report zero dependencies for the entire package.
No error is reported, just godep save -v will show an empty list of dependencies.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub


For #10540: status can be a dict or a str.

Consider both cases, where status is a dict or where status is a str. Add a unittest to test both.

@gplsteph as suggested I checked, and there's no reason to have this as a dict.


Generate new Discogs sensors + fix scan interval

Closes #12511
Description:
This PR addresses and closes some issues in #12511 and creates a new want list sensor and a random record sensor. https://github.com/home-assistant/home-assistant.io/pull/8214
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

Can be merged when build passes.


New version: Alex313031.Thorium.AVX2 version 120.0.6099.235

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run


一个不错的干掉360的方法，star了

360 has removed this drive whitelist when this repository code is released ( quickly removed, seemingly with threat detection )

360 has removed this drive whitelist when this repository code is released ( quickly removed, seemingly with threat detection )

no....


Disable auto quality updates

Motivation and context

Removed functionality to run quality report updates automatically after task changes

How has this been tested?

Checklist

[ ] I submit my changes into the develop branch
[ ] I have created a changelog fragment
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Summary by CodeRabbit
Release Notes

New Features

Enhanced documentation for automated quality assurance (QA) processes, including clearer instructions for managing Ground Truth jobs and validation modes.

Bug Fixes

Removed automatic quality report updates, streamlining the quality management process.

Documentation

Updated the document on automated QA to improve clarity and functionality, including expanded sections on quality estimation and management.

Chores

Removed deprecated settings related to quality check job delays in various configuration files.

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

Project coverage is 73.91%. Comparing base (f098ee5) to head (d291ab7).
Report is 10 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8790      +/-   ##
===========================================
- Coverage    73.96%   73.91%   -0.05%
===========================================
  Files          409      408       -1
  Lines        43942    43885      -57
  Branches      3985     3986       +1
===========================================
- Hits         32500    32436      -64
- Misses       11442    11449       +7

Components
Coverage Δ

cvat-ui
78.36% <ø> (+0.02%)
:arrow_up:

cvat-server
70.08% <100.00%> (-0.13%)
:arrow_down:


error: could not SSL_read crypto error:0A000126:SSL routines::unexpected eof while reading

Describe the bug
When Unbound is configured for DoT/DoH and some clients don't send the mandatory close_notify alert on shutdown. This causes unbound to log about this error if it was compiled against OpenSSL 3 (I think OpenSSL < 3 ignored this by default):
# journalctl -fu unbound
Feb 14 13:32:36 fw unbound[135]: [135:0] error: could not SSL_read crypto error:0A000126:SSL routines::unexpected eof while reading
Feb 14 14:06:39 fw unbound[135]: [135:0] error: could not SSL_read crypto error:0A000126:SSL routines::unexpected eof while reading
Feb 14 15:09:14 fw unbound[135]: [135:0] error: could not SSL_read crypto error:0A000126:SSL routines::unexpected eof while reading
Feb 14 15:36:21 fw unbound[135]: [135:0] error: could not SSL_read crypto error:0A000126:SSL routines::unexpected eof while reading
Feb 14 15:40:04 fw unbound[135]: [135:0] error: could not SSL_read crypto error:0A000126:SSL routines::unexpected eof while reading
Feb 14 15:45:51 fw unbound[135]: [135:0] error: could not SSL_read crypto error:0A000126:SSL routines::unexpected eof while reading

System:

Unbound version: 1.17.1
OS: Ubuntu 22.04.1 x86_64
unbound -V output:

# unbound -V
Version 1.17.1

Configure line: --build=x86_64-linux-gnu --prefix=/usr --includedir=${prefix}/include --mandir=${prefix}/share/man --infodir=${prefix}/share/info --sysconfdir=/etc --localstatedir=/var --disable-option-checking --disable-silent-rules --libdir=${prefix}/lib/x86_64-linux-gnu --runstatedir=/run --disable-maintainer-mode --disable-dependency-tracking --with-pythonmodule --with-pyunbound --enable-systemd --with-libnghttp2 --with-chroot-dir= --disable-rpath --with-pidfile=/run/unbound.pid --with-libevent --enable-tfo-client --with-rootkey-file=/usr/share/dns/root.key --enable-tfo-server
Linked libs: libevent 2.1.12-stable (it uses epoll), OpenSSL 3.0.2 15 Mar 2022
Linked modules: dns64 python respip validator iterator
TCP Fastopen feature available

BSD licensed, see LICENSE in source package for details.
Report bugs to<EMAIL_ADDRESS>or https://github.com/NLnetLabs/unbound/issues

On the client side, it's a mix of systemd-resolved (249.11) using DoT and Firefox (109.0.1) using DoH.
Additional information
I don't think DNS over TLS/HTTPS is subject to truncation attacks as messages lengths are verified using the length field for DoT or the content-length for DoH.
If you can confirm the above, the recommended workaround is to use SSL_OP_IGNORE_UNEXPECTED_EOF to ignore  buggy clients.
Additional information about TLS truncation attacks:

https://github.com/curl/curl/issues/7800#issuecomment-1183212116
https://security.stackexchange.com/questions/91435/how-to-handle-a-malicious-ssl-tls-shutdown

Thanks for the complete information!


Validate email after blur

Hi!
Is there best way to run validate input after blur in some cases?
Or validation runs strictly after onChange?

You can pre-verify value or trigger validation (this.setValue(this.getValue());) in your onBlur handler.

Yup, would suggest setting the value on the blur event, instead of the change event.
var Comp = React.createClass({
  mixins: [Formsy.Mixin],
  onInputBlur: function (event) {
    this.setValue(event.target,value);
  },
  render: function () {
    return <input onBlur={this.onInputBlur}/>
  }
});

Closing this as it seems to be solved?

What if originally i have a value for the input how would i add it to the input and keep it editable without change event with setValue()?

The tricky thing here is that you can no longer modify the field after the blur. You have to also setState onChange.

I just inserted an initialBlur flag which was set to true on first blur then checked if initialBlur was true before executing this.getErrorMessage() in render.

Got this example from someone too: https://github.com/christianalfoni/formsy-react/wiki/validate-on-blur
When next release is out we should start the work on some "default elements", where this validate on blur input could be one of them :-)


Fixed stage, since it crashed

wha

Dude On html it crashes i fixed it

?? it literally works fine lmao, i just everything from the original mod

also, why did you change the jsons? the stages are hardcoded

when it was ported to normal html5 the stage made the game crashed

normal html5?

yeah, since this version is supporting moblie-html5
for norml html5 stage it crashed my game and i looked in to it and found that was the reason why

what does normal html5 mean?

no moblie shit

oh, this is meant to be ran only on psych 0.5.1

it is , Something was breaking the stages

oh, alright then

but now it should work! Im gald to help with anything

nice, thanks

np!


[Diagnostics] Diagnose type with mismatching generic arguments in ternary expr

Fixes a missing diagnostic related to types with mismatched
generic arguments in one of both ternary branches.
Resolves: rdar://98862079

A one line fix, barely FC.

@swift-ci please smoke test


Implement the 3D Touch Home Screen Quick Actions

Expected Behaviour
Implement 3D touch home screen quick actions to instantly access some feature from 3D touch
Screenshots of the issue
N/A
Would you like to work on the issue?
Yes

Hey, @jogendra I would like to help on this issue and work on it.

Sure @AnuritaS. Go ahead :)


Add topic per message handling

#9

Also, watch out the failing PHP-CS rules and tests :)

Thanks for the review.
I'll try to make a new version for the beginning of next week.

Done!
Tell me if I can do something more.

Thank you @Gnucki.


Improve efficiency of CLI

Now the command needed to generated the licenses in run in parallel, so multiple licenses are simultaneously generated.

The default number of accounts also has been set to 0 since rarely someone wants accounts.
I tried generating 10 license keys.
Before it took around 8min, now it takes only 2min

Generating 10 accounts takes even shorter time.

We can reduce time by around one more minute of there is a command that only sets up the chrome/firefox driver if ran.

We can reduce time by around one more minute if there is a command that only sets up the chrome/firefox driver if ran. Eg: python3 main.py --driver-only --chrome. You can help me in that.

I figured that out.


Hide route pins after mission complete #89

Description

When a mission is completed the pins showing the dropoff and pickup location are still visible when the user routes to the homepage. This pull request fixes that
Related Issue

#89
Motivation and Context

Pins should only be visible when a mission is in progress.
How Has This Been Tested?

Complete a mission
Click on the Complete button that routes to the homepage
Notice the pins are no longer visible.

Screenshots (if appropriate):
Types of changes

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x ] My code follows the code style of this project.
[x ] My change requires a change to the documentation.
[x ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x ] All new and existing tests passed.

Great work @seyi-adeleke :tada:
And lots of thanks for your contribution!


[RFC] Interface design for rules

Currently rules have different modes, These modes make it easy to filter out which location they should go. This proposes the data structure be changed so multi checks can be defined in the same rule.
current structure
This end up requiring you to delcare multi top level rules for the same test at different stages
{
  regula.rules = {
    opensshForceEnable = {
      enable = true;
      mode = "nixosTest";  # determines what behavior occurs
    };
  };
}

new structure
To combat this we can change the structure where each version has its own sub attrset. This design would reduce excess code being written by the user so i am inclined to this design,
{
   regula.rules = {
    opensshForceEnable = {
      enable = true;
      nixosTest = {
         enable = true
         script = '''';
          vmConfig = ({...}:{});
      };
      perPackage = {
         enable = true;
         package = pkgs.nano;
      };
      topLevelCheck = ();
      assertion = false;
      warnings = false;
    };
  };
}

This also feels like it could improve readability as it allows various related stages to be grouped but differing info to be distinct.

After getting feedback with a few individuals I've concluded this to be a good decision going forward.


ui.router does not replace view

Hey everybody,
I have a small problem and do not understand where I made the mistake.
Here is a link to Codepen
The initial view, as well as the first state to be displayed properly, but when I click on the link of the first reloaded view the next state does not appear. The URL is modified but the new content will not be reloaded.
Any idea?

You're missing a <div ui-view></div> in your template for new
http://bit.ly/UIR-SOF


Fix division by zero in ADX calculation

Fix "RuntimeWarning: invalid value encountered in double_scalars" in lines 608 and 612

This didnt solve my problem getting NaN as a whole series occasionally. I'm testing a change I made, where every time we do a .shift(1), I set the first element from NaN to 0.0. for example:
trend.py:566
cs = self._close.shift(1)
cs[0] = 0.0
and also on dn and up, that seems to have solved my issue.


Unlinking Airplay players not working correctly

What version of Music Assistant has the issue?
2.0.0
What version of the Home Assistant Integration have you got installed?
2024.5.0
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Two  airplay players link correctly but fail to unlink correctly as shown in this video
https://github.com/music-assistant/hass-music-assistant/assets/19848947/6c3d1873-0489-440b-9d95-616a75b7ae63
Sometimes it takes a tab refresh to get the kodirumpus player to show the grouping was removed but the suffix remains

And then when I tried to group the other way (Kodirumpus to the amplifier) I get

How to reproduce
Start playback on Airplayer A
Link Airplayer B to Airplayer A
Unlink Airplayer B from Airplayer A
Stop playback on Airplayer A
Start playback on Airplayer B
Attempt to link Airplayer A to Airplayer B
Music Providers
All
Player Providers
Airplay
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.5.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Please re-test with 2.0.6 / 2.1.0b5

I see the fronted has changed but I can’t see how to unlink them?

I see the fronted has changed but I can’t see how to unlink them?

Just turn off or click the link icon again

But the link icon is no longer there. See picture.

OK I see the turn off works. Closing and can look at the missing link icon later (unless that is expected behaviour)


Contact Form / Integration with Mailchimp

Front-end is nearly done - need to work on back end and what happens when the user sends the message.

Currently, an 'enquiry' post type is created. Once we've created that, we should send the form to Mailchimp also.


Does 1.0 work with Kafka 0.10?

Hi, the README says that there is full support for 0.10 in the 0.5.x series. However, it is not 100% clear to me from the wording that future versions can't talk to Kafka 0.10.
I see in the CHANGELOG for newer versions of ruby-kafka things that seem generic improvements per se.
I need to consume from Kafka 0.10, should I stay with 0.5.5?

:wave:
Hello my friend @fxn :)
I came here with a similar question. The changelog for 0.7.0 states that support for kafka 0.10 was dropped there:

Drop support for Kafka 0.10 in favor of native support for Kafka 0.11.

BUT in a project I'm on at the moment and have a mixture of ruby-kafka versions running that should be incompatible from their official kafka version support but somehow work (0.5.x - 0.7.x) and are currently running against 1.0 and 2.3 but I believe they ran against 0.10 before (not sure).
So what I'm trying to say I'm also interested in the answer to this question but I suspect the answer is "it's not supported but it might work as long as you don't use certain features aimed at newer kafka versions".

Awesome @PragTob 😄 .


Gh pages

Resolving conflicts

Hello. Try to clone this repo to your profile/organization and make changes there. It's a original template ;)


Format of negative durations

var df = new Intl.DurationFormat("en", {style:"digital"});
console.log(df.format({hours: -1, minutes: -1, seconds: -1}));

This is currently spec'ed to return the string "-1:-01:-01", but I'm not sure if this matches user expectations.
This issue was previously discussed in #29:

https://github.com/tc39/proposal-intl-duration-format/issues/29#issuecomment-760530957 mentions that a single minus sign (i.e. "-1:01:01") should be used.
https://github.com/tc39/proposal-intl-duration-format/issues/29#issuecomment-763079333, https://github.com/tc39/proposal-intl-duration-format/issues/29#issuecomment-825983776, and https://github.com/tc39/proposal-intl-duration-format/issues/29#issuecomment-1113974576 mention that the output should be defined by CLDR resp. is implementation- and locale-defined (ILD).
https://github.com/tc39/proposal-intl-duration-format/issues/29#issuecomment-1171481635 closed # 29 by saying that mixed signs are no longer allowed in durations.

But I don't quite follow the discussion in #29:

I don't understand why disallowing mixed signs allowed to close #29.
The current Intl.DurationFormat spec doesn't allow any CLDR/ILD specific duration formatting, because the complete output is defined through Intl.NumberFormat and Intl.ListFormat. So mentioning CLDR/ILD is kind of moot, because we define how to glue the individual components.

Yeah, it is a bug if the spec requires "-1:-01:-01"
Issue #29 was closed when we decided to not allow mixed-sign units. However, bikeshedding the specific formats has not been done yet as far as I know. Hmm.

Seems there are two reasonable ways to display negative durations:

Only the first unit in display order gets the sign: "-3 days, 2 hours, 1:00"
All units get a sign except for trailing numeric units: "-3 days, -2 hours, -1:00"

We should also make sure that we don't allow mixed signs in the formattable object when it is not a Temporal.Duration.

TG2 discussion: https://github.com/tc39/ecma402/blob/master/meetings/notes-2023-10-12.md#format-of-negative-durations
No firm conclusion. A few more options were proposed, which I added to https://github.com/tc39/proposal-intl-duration-format/issues/169#issuecomment-1690712130. I have scheduled this for an upcoming CLDR design call.

Also, @justingrant volunteered to share some use cases here in this thread.

Here's a few use cases to help guide the decision, using plain conversational language rather than computer-ese, to better model how real language works.
Q: How much later, in local time, will it be when my flight arrives in arrive in San Francisco, compared to the local time when it departed Tokyo?
A: 6 hours and 35 minutes earlier, because of the time zone change.
Q: How long after this retail store opened was the first sale made yesterday?
A: Negative 2 minutes, because the store manager opened the door a few minutes early and and a customer walked in and bought something.
Q: (Manager) How long until you have a PR ready to fix this bug?
A: (Engineer) Negative 3 hours, I already fixed it! 😄
Q: How much later were you born than your twin sister?
A: 5 minutes before, I'm older!
A trucking company wants to encourage their drivers to arrive on time, or maybe a bit early. So they institute a bonus system where every month the delta between expected arrival times and actual arrival times are summed, and if the result is negative then they're awarded a $1-per-negative-minute bonus up to $100.
Q: (a lawyer's client, angry) You submitted that brief to the wrong judge, and now I lost my case and have to pay $5,000. Are you still going to charge me for writing it?
A: (lawyer) I'm so sorry. I'll not only not charge you, but I'll add a negative 10 billable-hour credit to your next invoice to cover my mistake.
The general theme to all of these examples is that negative durations are a possible but often unusual case where the people assumed that the duration would be positive, but sometimes they're negative so nonetheless must be accommodated.

Another use case I encountered a few days ago:
Q: When are the next departures from San Francisco to San José?
A: There are departures in -3 min, 17 min, and 37 min

I scheduled this for the CLDR design meeting at 10am on 2023-11-06.

The CLDR committee recommends option 1 as the default fallback behavior with the observation that it works better with narrow and numeric durations, like "-5d 3h 7m". However, CLDR wants to continue this in two lines of work:

Locales should be able to specify their own data for these formats. The form of that data is yet to be designed.
The committee feels that the more dominant use case is the "relative to fixed point in time" format, the third category of duration formating which has also been brought up in earlier threads on this repo.

Also worth noting that some folks on the committee were not convinced that we should support formating of floating negative durations at all, suggesting we throw an exception or similar.

Thank you so much for your work with the CLDR folks! Will proceed with option 1 for now, barring other guidance.
re: point 2, I'm increasingly convinced that negative durations, and formatting of negative durations, makes sense for the reasons that Justin gave up thread, and also because sometimes you want to do math involving durations. Negative durations are useful for the same reason that negative numbers are useful.
Justin's examples (particularly the trucking company one and the lawyer-and-client one) get at this, but also the contrived example that I keep coming back to is something like:

Anna is running errands. This will take some amount of time (say, 3 hours).
Brian is doing a series of housekeeping tasks, the sum total of which will take some amount of time (say, 2 hours). One of those tasks is "make lunch", which will take 30 minutes. If one of Anna's errands is "pick up carryout" and also she can get home before lunchtime, Brian no longer has to make lunch. As such, in that case we can subtract 30 minutes from the amount of time that Brian's doing housework.

There's a negative duration involved here, it's something that happens regularly in the world, someone could conceivably want to format it, and it's not directly related to any particular fixed point in time.


HDDS-6719. Verify build from source release tarball in CI

What changes were proposed in this pull request?
Change compile check to work against source tarball created by build check to ensure the contents of the tarball can be compiled.  The goal is to catch changes that become a problem only at release time.
https://issues.apache.org/jira/browse/HDDS-6719
How was this patch tested?
Regular CI:

build check uploaded source tarball: https://github.com/adoroszlai/hadoop-ozone/runs/6355813051#step:8:19
compile check used it instead of git repo: https://github.com/adoroszlai/hadoop-ozone/runs/6356144283#step:3:1
checks using binary tarball still work fine: https://github.com/adoroszlai/hadoop-ozone/runs/6356144521

Thanks @kerneltime and @szetszwo for the review.


[Question] - OpenId Url

I have been searching for the OpenId Url of a server running IdentityServer4
I have also search it in the docs.
So, is IdentityServer4 implementing this part of OpenId?

Not sure what you are looking for.  Identity Server 3 & 4 are certified here:
http://openid.net/certification/
Perhaps you mean that they are not an identity server?
That is your job.

Im asking for something like the habiltity to login to MyGet putting the following URL http://steamcommunity.com/openid

OpenID != OpenID Connect


test: Add testing for PodDisrputionBudgets in E2E testing

Fixes #N/A
Description
Add testing for ensuring PodDisruptionBudgets block drift and expiration in E2E testing
How was this change tested?
/karpenter snapshot
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.547%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4952

Relevant Lines:
5999

💛  - Coveralls


lpc55: Add external flash check

This patch adds a check that verifies that the external flash storage can be accessed and formatted with a littlefs2 filesystem.

looks very good to me, will merge soon


Get Started Page should not show if you are logged in

It should go straight to overview

@medensqui I think we should keep it this way. There is a "Go to Overview" button in the top right, and this way you don't have to sign out if you want to see the front page or about page when we make those.

@thatoddmailbox True!

@thatoddmailbox Get Started should link to overview when signed in though

@medensqui but it does though. (I haven't updated planhub.me yet, but I will)

@thatoddmailbox Ok. Do the updates go immediately on the azure server?

@medensqui No, I have to update it manually. I think what I will do is make it so it only updates to specifically marked releases. Like if we tag a release as "stable" then it automagically updates it, but otherwise ignores it.
But for now, no, I have to poke it.

YAY THIS IS FIXED

@thatoddmailbox OK THEN POKE IT
Smart idea for stable
YAY

@medensqui I DID POKE IT


X11 connection rejected because of wrong authentication.

Bug Report
Relevant information
When starting multiboard with yarn start, I get the following printed to STDOUT:
yarn run v1.3.2
$ webpack-dev-server --hot --config webpack/dev.config.js
(node:3438786) DeprecationWarning: Tapable.plugin is deprecated. Use new API on `.hooks` instead
(node:3438786) DeprecationWarning: Tapable.apply is deprecated. Call apply on the plugin directly instead
[JARVIS] Starting dashboard on: http://localhost:3333
X11 connection rejected because of wrong authentication.
X11 connection rejected because of wrong authentication.
X11 connection rejected because of wrong authentication.
X11 connection rejected because of wrong authentication.

Multiboard is not reachable on either localhost:2222 (as suggested in the readme) or localhost:3333 as suggested from the printed output.
Your Environment
trello multiboard built with yarn 1.3.2 on a raspberry pi running Debian 11 bullseye
Steps to reproduce:

clone the repo
install dependencies with yarn
modify config
start with yarn start

Observed Results:
see above
Expected Results:
Multiboard should be reachable on localhost:2222 or localhost:3333

Solved with export XAUTHORITY=$HOME/.Xauthority.


Can we force the necessity that Segments have a constructor with context and args?

Or are they really really required?

Perhaps with abstract Segment(context, args);

SegmentManager#rebuildSegment() should fall back to constructor without bundle if there is none with a bundle. If there is none, there cannot be any arguments anyways.

Done


Pony vim fix new keyword

I added the 'in' keyword, I've seen that one missing when writing a loop.

I messed up with my branches, I'll close that one.

Ah, you might just want to include the in syntax keyword now

Exactly


Unsatisfiable constraints for /bin/sh (virtual) with apk upgrade on apline:edge

Hi,
I am unable to upgrade the lasted alpine:edge docker image. It fails with unsatisfiable constraints on /bin/sh (virtual) during the upgrade:
$ docker run -ti alpine:edge
/ # apk --update upgrade
fetch http://dl-cdn.alpinelinux.org/alpine/edge/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/edge/community/x86_64/APKINDEX.tar.gz
Upgrading critical system libraries and apk-tools:
(1/1) Upgrading apk-tools (2.7.2-r0 -> 2.8.0-r1)
Executing busybox-1.27.0-r3.trigger
Continuing the upgrade transaction with new apk-tools:
fetch http://dl-cdn.alpinelinux.org/alpine/edge/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/edge/community/x86_64/APKINDEX.tar.gz
ERROR: unsatisfiable constraints:
  /bin/sh (virtual):
    provided by: busybox-1.27.2-r2
    required by: alpine-baselayout-3.0.5-r2[/bin/sh] alpine-baselayout-3.0.5-r2[/bin/sh]

Same for me!

I'm stuck in the same issue. Anyone come up with the solution yet?
Upgrading critical system libraries and apk-tools:
(1/5) Installing libressl2.5-libcrypto (2.5.5-r3)
(2/5) Installing libressl2.5-libssl (2.5.5-r3)
(3/5) Upgrading apk-tools (2.6.9-r0 -> 2.8.1-r0)
(4/5) Purging libssl1.0 (1.0.2k-r0)
(5/5) Purging libcrypto1.0 (1.0.2k-r0)
Executing busybox-1.24.2-r13.trigger
Continuing the upgrade transaction with new apk-tools:
ERROR: unsatisfiable constraints:
  /bin/sh (virtual):
    provided by: busybox-1.27.2-r3
    required by:
                 alpine-baselayout-3.0.5-r2[/bin/sh]
                 alpine-baselayout-3.0.5-r2[/bin/sh]
The command '/bin/sh -c apk upgrade --purge' returned a non-zero code: 2

I think this is just a pretty outdated alpine:edge image. I'm not having this issue on 631c9b7258b2. Please try pulling the image again.


Exception when using SciChartInteractionToolbar and Binding to YAxes/XAxes

An exception is thrown from this line when using binding on either the XAxes or YAxes properties of the ChartSurface.
This can be seen by uncommenting this line in the example "Axis Binding and Annotations Binding".
One way this can be resolved by changing the line to something like this...
var isPolar = surface != null &&               (surface.IsPolarChart || (surface.XAxes?.Any(x => x.IsPolarAxis) ?? false) || (surface.YAxes?.Any(x => x.IsPolarAxis) ?? false));
And probably adding something like this to the end of OnTargetSurfaceDependencyPropertyChanged...
scs.XAxesCollectionNewCollectionAssigned += (_, __) => toolbar.OnCreateModifiers(toolbar, scs); scs.XAxesCollectionNewCollectionAssigned += (_, __) => toolbar.OnCreateModifiers(toolbar, scs);

Fixed in SciChart v8. Thanks for reporting it!


Unsafe access on cls.getNameSpace()

In lib/context_utils.js, unsafe access in setSegment: Function:
cls.getNamespace(NAMESPACE).set(SEGMENT, segment)

@samhuk
It looks like this was edited a few times. Can you share more details on what steps to take to encounter an issue? Are you getting an error, or seeing unexpected behavior?

Ah yes of course. My initial issue was with another different npm package that used cls-hooked (correlation-id). It turns out that other packages that later initialise this local storage feature will overwrite the storage and cause this line to throw a fatal exception.

Are you using the experimental branch of the SDK which depends on cls-hooked then? Or the stable version (which is what you'd get if you didn't specify a tag when npm installing) that depends on continuation-local-storage?

Sorry for not clarifying. Both of the packages interact with the same storage. What originally motivated me creating the issue was that if one does an import ... from aws-xray-sdk-node and next line import ... from {a-cls-package}, the initialisation of a-cls-package could wipe the namespace from the storage, and then the unsafe access causes an error instead of gracefully failing ot reinitialisation ("recovering") the state.

@samhuk
Thanks for reporting this. I see now that the issue has something to do with continuation-local-storage and cls-hooked being imported in the same project. One overwrites the other.
It looks like we may have to update our code to create the namespace if it doesn't exist. If you switch to the experimental tag of the SDK (npm install --save aws-xray-sdk@experimental) does this issue go away?

Only just looking at this now. Good work, can confirm it works with other cls package.


US 07.01.01: As a borrower, I want to hand over a book I borrowed by scanning the book ISBN code to denote the book as available

User Story: As a borrower, I want to hand over a book I borrowed by scanning the book ISBN code to denote the book as available

closed via #137


Change tensorflow-addons version

Proposed changes:

Expand the allowed range of tensorflow-addon versions to make it possible to install Rasa on MacOS Mojave (0.8.2 doesn't support it: https://pypi.org/project/tensorflow-addons/0.8.2/#files)

Status (please check what you already did):

[x] updated the changelog (please check changelog for instructions)

@alwx Can you merge this in 1.8.x as well. Installation breaks everytime we try to install 1.8.0.


Enabling toggling of GPIOs besides LEDs

Need to be able to send a varying pulse width to the ESC every 20ms

Use GPIOs for simple first approach.

Only LEDs currently toggle - Figure out how to use any of the other GPIOs broken out 1.6?
Determine accuracy of pulse widths with this approach

https://github.com/thaletterb/BLDC_ESC/pull/2


Finalize foundation collections

Abstract
Polishes and finalizes the foundation collections in preparation for the 1.0 release.
Issues

[x] Fixes https://github.com/hydephp/develop/issues/1194
[x] Fixes https://github.com/hydephp/develop/issues/1196

Breaking/significant changes

Moves collection accessors to the facade classes to limit scope of the classes to focus on data discovery

With this, the Route facade and the Routes facade overlap. I think they can be merged.


Testing webhooks locally

I'm using expose to receive webhooks locally and it is working just fine in general.
For a new project, I use this package to create subscriptions, but I'm having a hard time getting started with the webhooks. Basically, they are not incoming.
In my config/cashier.php I set the following for the webhook_url:
'webhook_url' => (new GetWebhookUrlAction)->execute('webhooks/mollie'),

The action looks like this:
public function execute($routePath): string
    {
        if (config('app.env') === 'local' && env('WEBHOOK_BASEURL')) {
            $path = data_get(parse_url($routePath), 'path');

            return rtrim(env('WEBHOOK_BASEURL'), '/') . $path;
        }

        return $routePath;
    }

And in my env file, I set the expose url. The url is reachable from the outside.
Note: When just using webhooks/mollie in the config, Mollie complains that the webhook url is not reachable, thus the workaround for local development.
But when I want to create a subscription, I am directly to the Mollie Checkout, but when I am redirected back to my application, no webhook is incoming. Expose is running, as well as php artisan schedule:work. I am getting no errors or anything. Just the redirect from Mollie and then silence. No database records updated, nothing.
Just to be sure, Cashier should taking care of any incoming webhooks and there should be nothing for me to do to make this work, right? What could be the cause for this?

What happens if you use cashier's default webhook config?

 I just debugged $user->newSubscription($name, $user->plan_name)->create(); and noticed, that there was a / missing in the webhook url. Now it works just as expected.

No problem, enjoy!


vendor: upgrade dependencies except grpc-go

lgtm

@gyuho Excellent, I was a bit concerned with merging that into the balancer branch, thanks for testing it.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@88acced). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #9841   +/-   ##
=========================================
  Coverage          ?   69.67%
=========================================
  Files             ?      377
  Lines             ?    35315
  Branches          ?        0
=========================================
  Hits              ?    24607
  Misses            ?     8938
  Partials          ?     1770

Continue to review full report at Codecov.

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


error: future cannot be sent between threads safely -- dead_letters.rs and writer.rs

I cloned the project and am trying to run it locally, but I'm getting this error when I try to compile:
error: future cannot be sent between threads safely
   --> src\dead_letters.rs:283:43
    |
283 |       ) -> Result<(), DeadLetterQueueError> {
    |  ___________________________________________^
284 | |         let values: Result<Vec<Value>, _> = dead_letters
285 | |             .iter()
286 | |             .map(|dl| {
...   |
313 | |         Ok(())
314 | |     }
    | |_____^ future created by async block is not `Send`
    |
    = help: the trait `std::marker::Sync` is not implemented for `(dyn PageWriter + 'static)`
note: future is not `Send` as this value is used across an await
   --> src\writer.rs:401:26
    |
395 |         for (key, values) in self.divide_by_partition_values(values)? {
    |                              ---- has type `&writer::DataWriter` which is not `Send`
...
401 |                         .await,
    |                          ^^^^^ await occurs here, with `self` maybe used later
...
417 |         }
    |         - `self` is later dropped here
help: consider moving this into a `let` binding to create a shorter lived borrow
   --> src\writer.rs:395:30
    |
395 |         for (key, values) in self.divide_by_partition_values(values)? {
    |                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = note: required for the cast from `Pin<Box<[async block@src\dead_letters.rs:283:43: 314:6]>>` to `Pin<Box<(dyn futures::Future<Output = Result<(), dead_letters::DeadLetterQueueError>> + std::marker::Send + 'async_trait)>>`

I get that error about 10 times, with the only variation being what other types std::marker::Sync is not implemented for.
The other types it's not implemented for:

(dyn parquet::compression::Codec + 'static)
(dyn parquet::encodings::encoding::Encoder + 'static)
- and Int96Type, FloatType, DoubleType, and FixedLenByteArrayType
(dyn parquet::encodings::encoding::Encoder<deltalake_core::parquet::data_type::Int32Type> + 'static)
- and Int64Type, and ByteArrayType

I've been trying to fix it myself but I'm not making any progress. If anyone else knows how to fix this, I would really appreciate the help!
To reproduce this error, clone kafka-delta-ingest and run cargo build --release
I'm running KDI locally on Windows. To get it to run on Windows I replace the rdkafka dependency with the dependencies below, but otherwise did not make any changes to my clone of KDI.
openssl = { version = "0.10.45", features = ["vendored"] }
rdkafka = { version = "0.28", features = ["cmake-build"] }

Apparently my version of Rust was too old to get along with one of the updated crates. I updated it and now it compiles again. Sorry for the false alarm!


[Question] Is this project updated regularly?

Hi,
Hope you are doing well.
I'm an engineer at fabricwarehouse.com (one of your clients). For many reasons, we want to use this code as our main checkout app instead of the default checkout app.
Our question: Is this project updated regularly following the BigCommerce system's update? Is it fully tested?
Currently, we only use PayPal as a payment method.
Thank you!

Hi @iceleo-com from what I can see fabricwarehouse.com uses the native checkout. Is that correct?
Are you planning to change that to a custom checkout?
The project is regularly updated and maintained. The project has very good amount of testing in form of unit tests and we do run functional tests as part of the release pipeline.

Also can take a look at the commit history to see the activity on the project https://github.com/bigcommerce/checkout-js/commits/master

Hello, we release almost every working day. You can tell by looking at our release history. All BC customers will receive the new release in about three business days. In response to another question, we have 1400+ unit tests in this repo.
We also keep an eye out for new issues in this repo.

Hi @iceleo-com!
This is the default BigCommerce ("optimized one-page") checkout. Any changes made to this repository are applied to most BigCommerce stores (as long as they're on Optimized One-Page checkout, so not custom/multi-page/other checkouts), during a staggered rollout.
You're welcome to fork this code since it is open source, so you can create your own custom checkout. Keep in mind that it's up to you to update it. New functionality and security updates may not automatically work or be included on forked checkouts, since the old version may not have the required code for support this. We do aim to be as backwards-compatible as possible, so new changes of storefront APIs (which this repo - through the checkout-sdk-js uses) shouldn't break old versions of the checkout (so most custom checkouts).
Test coverage is on the unit test level in this repository. As a final check before deploying this to BigCommerce stores, we run many high-level (acceptance, functional, and contract) tests internally before deploying this to BigCommerce stores. People using this repository to create their own custom checkout can write their own high-level tests, which could perhaps run on their sandbox (or dev store) before deploying their version to their production store.
We're always interested in why people are building their own checkouts! Do you have particular requirements the default checkout doesn't cater for?

@animesh1987 , @bc-peng thank you so much for the clarification! I will close this question
@TomA-R oh yes, we need some custom to make it smarter, and easier to use for our customers


Change how Materialize is included

At the moment, the catalog application imports Materialize CSS and JS. Change it to only use CSS and let the bridge take control of Materialize's JS parts.

I disagree with this approach since we made the decision not to task the bridge with any UI responsibilities - leaving this intentionally to the application developer.

You're right, that was my intention. :smile:
But you explained it more thoroughly.


ISPN-5788 Add new numeric field to Address

Needed to test aggregation functions on numbers in embedded entities.

@jmarkos you need to fix some tests:
Tests in error:
testMarshallObject(org.infinispan.protostream.WrappingTest): Required field "sample_bank_account.User.Address.number" should have been written by a calling a suitable method of org.infinispan.protostream.MessageMarshaller$ProtoStreamWriter
testMarshallUserList(org.infinispan.protostream.WrappingTest): Required field "sample_bank_account.User.Address.number" should have been written by a calling a suitable method of org.infinispan.protostream.MessageMarshaller$ProtoStreamWriter
test(org.infinispan.protostream.ProtobufParserTest): Required field "sample_bank_account.User.Address.number" should have been written by a calling a suitable method of org.infinispan.protostream.MessageMarshaller$ProtoStreamWriter
testSerializationRoundtrip(org.infinispan.protostream.impl.UnknownFieldSetImplTest): Required field "sample_bank_account.User.Address.number" should have been written by a calling a suitable method of org.infinispan.protostream.MessageMarshaller$ProtoStreamWriter
testProtobufRoundtrip(org.infinispan.protostream.impl.UnknownFieldSetImplTest): Required field "sample_bank_account.User.Address.number" should have been written by a calling a suitable method of org.infinispan.protostream.MessageMarshaller$ProtoStreamWriter
testMarshallUser(org.infinispan.protostream.MarshallingTest): Required field "sample_bank_account.User.Address.number" should have been written by a calling a suitable method of org.infinispan.protostream.MessageMarshaller$ProtoStreamWriter
Tests run: 62, Failures: 0, Errors: 6, Skipped: 3

Sorry I didn't realize this had a testsuite - fixed.

Pushed thanks


Update layout to include more contrast

currently thinking to use more pink and yellow and light green since it looks really nice when combined
reference designs
non hovering

elements hovered

did this already, but might reference this for future use


Add Unstoppable Domains App

What kind of change is it?

[X] Addition
[ ] Removal
[ ] Edit
[ ] Other

Checklist

[x] I edited the /data directory instead of the README.md.
[X] This PR includes only one addition/removal/edit.
[x] I ran the make build command following my edits to the /data directory.
[X] I reviewed the content policy to ensure my submission meets the requirements.
[X] I have followed the CONTRIBUTING.md guidelines.

Details
Added the Unstoppable Domains app which is the ability to connect your blockchain domain with an IPFS hash to build a censorship resistant website. Also added unstoppable-domains-app.png file to /src/static/images folder.

@Stebalien @lidel
Hi Martin,
Please contact me via my profile if you require any details as part of your review for this pull request.
Thanks
Paul

@lidel what two steps do you require Martin?

@PaulUD  Unchecked items from your post at https://github.com/ipfs/awesome-ipfs/pull/309#issue-370717983, namely: adding your app to manifest in data/ directory and then running build which generates README and the website.
Let me know if you have any issues.

@lidel so it's not an option of amending the README.md OR the /data directory?

@lidel amended /data/apps.yaml reverted README.md to original and checked additional 2 items in the checklist.

Thank you @PaulUD,  do you mind moving it from apps.yml to services.yml ?
It sounds like its a better place for it.

@lidel moved from Apps to Services as requested.


[improve][java client] No need print exception stack when DLQ send failed.

Motivation
No need to print an exception stack when DLQ send failed. Because sending DLQ fails, it will be re-delivered.
Modifications

Only print exception message instead of exception stack.

Documentation

[x] doc-not-needed
(Please explain why)

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


UI Issue was closed by the original OP yesterday, but I'm reopening it because the version in the Chrome Store is outdated

Hi there, I see the issue related to the UI was fixed and you wrote it was published yesterday, but on the Chrome Web Store it still shows the last version on Sept 27th. I was concerned because the original OP said it was fixed (as did you), but I don't see how to get the latest version for Chrome. Is there a delay from the time that you publish to the point that we would see the updated version?

@Keith12345678910,
Sorry I should have said in my comment on #222 that the update was still pending approval on the chrome store. This normally takes just a couple of hours, but can take longer. I think the weekend probably slowed it down a little.
I can see now that it is still in pending approval. I'll update here when this changes.
Also note that the Firefox and Opera approval times are typically much fast, in the order a few minutes normally. The previous OP was likely using Firefox and had the update already.

Here is how the Chrome dashboard looks on my side, note the Pending review status in the top left.


The F# MVVM Template does not compile

Describe the bug
Upon creating a new project using the F# MVVM template, the resulting project does not compile. I did not make any changes to any files. If I instead use the C# MVVM template, it compiles and runs just fine.
To Reproduce
Steps to reproduce the behavior:
I tried to follow the instructions here: https://docs.avaloniaui.net/tutorials/todo-list-app/creating-a-new-project
Specifically I downloaded the VS 2019 extension, and created a new project using the F# MVVM template.
Expected behavior
Ideally the bootstrapped project should compile and run. Or if there are any additional steps that need to be taken by the user, they should be added to the tutorial.
Screenshots
Not applicable.
Desktop (please complete the following information):
Windows 10 home, 10.0.19041. Visual studio 2019 16.8.6. Avalonia for VS <IP_ADDRESS>.
Additional context
These are the compilation errors I am faced with:
ViewLocator.fs 10 - FS0859 No abstract property was found that corresponds to this override
Program.fs 9 - FS0501 The member or object constructor 'LogToDebug' takes 2 argument(s) but is here given 0. The required signature is '(extension) AppBuilder.LogToDebug<'T when 'T : (new : unit -> 'T) and 'T :> Controls.AppBuilderBase<'T>>(?level: Logging.LogEventLevel, [] areas: string []) : 'T'.
Program.fs 3 - FS0039 The namespace 'Serilog' is not defined.
MainWindow.xaml.fs 10 - FS0039 The type 'MainWindow' does not define the field, constructor or member 'AttachDevTools'.

These are the changes I made to the bootstrapped F# project to make it compile, by comparing and contrasting against a bootstrapped C# project.
Moved member __.SupportsRecycling with get() = false outside of the interface IDataTemplate with ... block. This is because this property is not a part of the implementation of the IDataTemplate interface.
Change line 11 of Program.fs to .LogToTrace(LogEventLevel.Warning). This is because F# does not understand having an optional argument in front of a params... argument. The template also defaulted to .LogToDebug which was marked deprecated.
Remove the .Serilog part of line 3 from program.fs: open Avalonia.Logging.Serilog -> open Avalonia.Logging
Install the Avalonia.Diagnostics nuget package, to add DevToolsExtensions for .AttachDevTools().

There is PR with a fix for the issue
https://github.com/AvaloniaUI/AvaloniaVS/pull/200
I've tried to bring more attention to this PR on gitter but was ignored once or twice :( Maybe you'll get some luck since I gave up here.

Ah I see, I didn't know there was a separate repository. That PR fixes the exact issue I have. I guess I will leave this open for posterity, but feel free to close it.

Sorry about that guys, I didn't see it. Will get it merged.


Update documentation to 5.0.1

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

elasticmachine spams a little bit.


Typo fix in searchlight/README.md

In the table's colume description:
Sometime first letter in capital, sometime not, except the proper noun. It's better use same format.

Thanks @AdamDang ! But we are not merging this . Please see https://github.com/kubernetes/charts/pull/4957


Why recastnavigation use its own array and vector instead of stl?

For example, in RecastCounter.cpp:
...
rcIntArray verts(256);
rcIntArray simplified(64);

for (int y = 0; y < h; ++y)
{
...

Why not use std::vector to replace it? Is it because when recast navigation born, the stl containers have not born(similar reason as TArray in ue4)?

std::vector long precedes recast.  Many games avoid using the STL for various reasons, so for maximum compatibility Recast also avoids using STL containers.


Pulumi import of a linode resource results in warnings

What happened?
I tried to import a resource using the Linode provider, which resulted in the following warnings:
    warning: One or more imported inputs failed to validate. This is almost certainly a bug in the `linode` provider. The import will still proceed, but you will need to edit the generated code after copying it into your program.
    warning: linode:index/instance:Instance resource 'tommy' has a problem: Conflicting configuration arguments: "swap_size": conflicts with disk. Examine values at 'tommy.swapSize'.
    warning: linode:index/instance:Instance resource 'tommy' has a problem: Conflicting configuration arguments: "disk": conflicts with swap_size. Examine values at 'tommy.disks'.
    warning: linode:index/instance:Instance resource 'tommy' has a problem: Conflicting configuration arguments: "config": conflicts with swap_size. Examine values at 'tommy.configs'.

It's pretty easy to fix, but I figured opening a bug report would be a good idea anyway.
Example
pulumi import linode:index/instance:Instance tommy

resulted in the following program:
	pulumi.Run(func(ctx *pulumi.Context) error {
		_, err := linode.NewInstance(ctx, "tommy", &linode.InstanceArgs{
			Alerts: &linode.InstanceAlertsArgs{
				NetworkIn:     pulumi.Int(5),
				NetworkOut:    pulumi.Int(5),
				TransferQuota: pulumi.Int(80),
			},
			BootConfigLabel: pulumi.String("My Debian 11 Disk Profile"),
			Booted:          pulumi.Bool(true),
			Configs: linode.InstanceConfigArray{
				&linode.InstanceConfigArgs{
					Devices: &linode.InstanceConfigDevicesArgs{
						Sda: &linode.InstanceConfigDevicesSdaArgs{
							DiskId:    pulumi.Int(<id>),
							DiskLabel: pulumi.String("Debian 11 Disk"),
						},
						Sdb: &linode.InstanceConfigDevicesSdbArgs{
							DiskId:    pulumi.Int(<id>),
							DiskLabel: pulumi.String("512 MB Swap Image"),
						},
					},
					Helpers: &linode.InstanceConfigHelpersArgs{
						DevtmpfsAutomount: pulumi.Bool(true),
					},
					Kernel:     pulumi.String("linode/grub2"),
					Label:      pulumi.String("My Debian 11 Disk Profile"),
					RootDevice: pulumi.String("/dev/sda"),
				},
			},
			Disks: linode.InstanceDiskTypeArray{
				&linode.InstanceDiskTypeArgs{
					Filesystem: pulumi.String("ext4"),
					Label:      pulumi.String("Debian 11 Disk"),
					Size:       pulumi.Int(50688),
				},
				&linode.InstanceDiskTypeArgs{
					Filesystem: pulumi.String("swap"),
					Label:      pulumi.String("512 MB Swap Image"),
					Size:       pulumi.Int(512),
				},
			},
			Label:  pulumi.String("tommy"),
			Region: pulumi.String("eu-west"),
			SwapSize: pulumi.Int(512),
		}, pulumi.Protect(true))
		if err != nil {
			return err
		}
		return nil
	})

The SwapSize here seems to conflict with the swap partition defined in the Disks struct.
Output of pulumi about
$ pulumi about
CLI
Version      3.86.0
Go Version   go1.21.1
Go Compiler  gc

Plugins
NAME    VERSION
go      unknown
linode  4.6.1

Host
OS       debian
Version  12.1
Arch     x86_64

This project is written in go: executable='/home/tgummerer/.nix-profile/bin/go' version='go version go1.20.8 linux/amd64'

Current Stack: tgummerer/pulumi-test/dev

TYPE                            URN
pulumi:pulumi:Stack             urn:pulumi:dev::pulumi-test::pulumi:pulumi:Stack::pulumi-test-dev
pulumi:providers:linode         urn:pulumi:dev::pulumi-test::pulumi:providers:linode::default_4_6_1
linode:index/instance:Instance  urn:pulumi:dev::pulumi-test::linode:index/instance:Instance::tommy

Found no pending operations associated with tgummerer/dev

Backend
Name           pulumi.com
URL            https://app.pulumi.com/tgummerer
User           tgummerer
Organizations  tgummerer
Token type     personal

Dependencies:
NAME                                    VERSION
github.com/pulumi/pulumi-linode/sdk/v4  4.6.1
github.com/pulumi/pulumi/sdk/v3         3.86.0

Pulumi locates its logs in /tmp by default

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

It's plausible this is related to https://github.com/pulumi/pulumi-terraform-bridge/issues/577.

Still having some issues here:
package main

import (
	"github.com/pulumi/pulumi-linode/sdk/v4/go/linode"
	"github.com/pulumi/pulumi/sdk/v3/go/pulumi"
)

func main() {
	pulumi.Run(func(ctx *pulumi.Context) error {
		inst, err := linode.NewInstance(ctx, "web", &linode.InstanceArgs{
			Label:  pulumi.String("simple_instance"),
			Image:  pulumi.String("linode/ubuntu22.04"),
			Region: pulumi.String("us-central"),
			Type:   pulumi.String("g6-standard-1"),
			AuthorizedKeys: pulumi.StringArray{
				pulumi.String("ssh-rsa AAAA...Gw== user@example.local"),
			},
			RootPass: pulumi.String("this-is-not-a-safe-password"),
			Tags: pulumi.StringArray{
				pulumi.String("foo"),
			},
			SwapSize:  pulumi.Int(256),
			PrivateIp: pulumi.Bool(true),
		})
		if err != nil {
			return err
		}
		ctx.Export("instId", inst.ID())
		return nil
	})
}

And:
anton@anton-mbp-m3> pulumi import linode:index/instance:Instance mylinode $(pulumi stack output instId)                                                                                ~/bugs/pulumi-linode-373
Previewing import (dev)

View in Browser (Ctrl+O): https://app.pulumi.com/anton-pulumi-corp/pulumi-linode-373/dev/previews/68e7c523-6f19-45e1-85c7-2a43e8f8dae8

     Type                      Name                   Plan       Info
     pulumi:pulumi:Stack       pulumi-linode-373-dev
 =   └─ linode:index:Instance  mylinode               import     8 warnings

Diagnostics:
  linode:index:Instance (mylinode):
    warning: urn:pulumi:dev::pulumi-linode-373::linode:index/instance:Instance::mylinode verification warning: The embedded config is deprecated and scheduled to be removed in the next major version.Please consider migrating it  to linode_instance_config resource.
    warning: urn:pulumi:dev::pulumi-linode-373::linode:index/instance:Instance::mylinode verification warning: The embedded disk block in linode_instance resource is deprecated and scheduled to be removed in the next major version. Please consider migrating it to be the linode_instance_disk resource.
    warning: One or more imported inputs failed to validate. This is almost certainly a bug in the `linode` provider. The import will still proceed, but you will need to edit the generated code after copying it into your program.
    warning: linode:index/instance:Instance resource 'mylinode' has a problem: Value for unconfigurable attribute. Can't configure a value for "config.0.id": its value will be decided automatically based on the result of applying this configuration.. Examine values at 'mylinode.configs[0].id'.
    warning: linode:index/instance:Instance resource 'mylinode' has a problem: Value for unconfigurable attribute. Can't configure a value for "disk.0.id": its value will be decided automatically based on the result of applying this configuration.. Examine values at 'mylinode.disks[0].id'.
    warning: linode:index/instance:Instance resource 'mylinode' has a problem: expected length of disk.0.root_pass to be in the range (11 - 128), got . Examine values at 'mylinode.disks[0].rootPass'.
    warning: linode:index/instance:Instance resource 'mylinode' has a problem: expected length of disk.1.root_pass to be in the range (11 - 128), got . Examine values at 'mylinode.disks[1].rootPass'.
    warning: linode:index/instance:Instance resource 'mylinode' has a problem: Value for unconfigurable attribute. Can't configure a value for "disk.1.id": its value will be decided automatically based on the result of applying this configuration.. Examine values at 'mylinode.disks[1].id'.

Resources:
    = 1 to import
    2 unchanged

Do you want to perform this import?  [Use arrows to move, type to filter]
  yes
  no
> details

About:
CLI
Version      3.117.0
Go Version   go1.22.3
Go Compiler  gc

Plugins
KIND      NAME    VERSION
language  go      unknown
resource  linode  4.21.0

Host
OS       darwin
Version  14.5
Arch     arm64

This project is written in go: executable='/Users/anton/bin/go' version='go version go1.22.2 darwin/arm64'

Current Stack: anton-pulumi-corp/pulumi-linode-373/dev

TYPE                            URN
pulumi:pulumi:Stack             urn:pulumi:dev::pulumi-linode-373::pulumi:pulumi:Stack::pulumi-linode-373-dev
pulumi:providers:linode         urn:pulumi:dev::pulumi-linode-373::pulumi:providers:linode::default_4_21_0
linode:index/instance:Instance  urn:pulumi:dev::pulumi-linode-373::linode:index/instance:Instance::web

Found no pending operations associated with dev

Backend
Name           pulumi.com
URL            https://app.pulumi.com/anton-pulumi-corp
User           anton-pulumi-corp
Organizations  anton-pulumi-corp, moolumi, pulumi
Token type     personal

Dependencies:
NAME                                    VERSION
github.com/pulumi/pulumi-linode/sdk/v4  v4.21.0
github.com/pulumi/pulumi/sdk/v3         v3.118.0

Pulumi locates its logs in /var/folders/gd/3ncjb1lj5ljgk8xl5ssn_gvc0000gn/T/com.apple.shortcuts.mac-helper// by default

Although the bridge made progress on heuristics to automatically resolve ConflictsWith warnings during import, issues remain with imported values not passing validators and perhaps there is more work to be done.

It looks like https://github.com/pulumi/pulumi-terraform-bridge/pull/2277 approach may be relevant here but may not fully fix this given that the violations are nested properties. Generalizing that could work.

The remaining issue could be mitigated by https://github.com/pulumi/pulumi-terraform-bridge/issues/2314 if it handled recursive values.


Sentence/document classification

Hi, congrats on the project! I'm finding it very useful.
Is there a script to also do whole document or sentence classification in the same way you have one for whole word classification?
Cheers.

Closing as it is stale. Feel free to re-open if the issue persists.


persistent cache resiliency

Taking to account that file locks are not completely reliable on non Win platforms, It would be nice to implement following features to be race conditions resilient:

Switch file cache accessor from non-atomic file writes to atomic file renaming (in order to prevent dirty reads + dirty writes - cache corruption)
Retry logic on KeyChainAccessor to deal with race conditions

details can be found here -> https://github.com/AzureAD/microsoft-authentication-extensions-for-java/wiki/Race-conditions

@bgavrilMS : Java and soon Python has these improvements. We should ensure to properly plan for these in .NET.


Cannot set property 'showKeyboardShortcutsButton' of undefined

What error is this? How to solve?
DayPickerKeyboardShortcuts.js:1 Uncaught TypeError: Cannot set property 'showKeyboardShortcutsButton' of undefined
    at ref (DayPickerKeyboardShortcuts.js:1)
    at detachRef (ReactRef.js:1)
    at Object.ReactRef.detachRefs (ReactRef.js:1)
    at Object.receiveComponent (ReactReconciler.js:1)
    at Object.updateChildren (ReactChildReconciler.js:1)
    at ReactDOMComponent._reconcilerUpdateChildren (ReactMultiChild.js:1)
    at ReactDOMComponent._updateChildren (ReactMultiChild.js:1)
    at ReactDOMComponent.updateChildren (ReactMultiChild.js:1)
    at ReactDOMComponent._updateDOMChildren (ReactDOMComponent.js:1)
    at ReactDOMComponent.updateComponent (ReactDOMComponent.js:1)

Set up:
class DP extends React.Component {
    render() {

        return (
            <div className={this.props.className}>
                <SingleDatePicker
                    date={this.state.date} // momentPropTypes.momentObj or null
                    onDateChange={this.onDateChange} // PropTypes.func.isRequired
                    focused={this.state.focused} // PropTypes.bool
                    onFocusChange={this.onFocusChange} // PropTypes.func.isRequired
                />
            </div>
        );
    }

    onFocusChange = (focused) => {
        console.log('[onFocusChange] focused', focused)
        this.setState(
            focused
        )
    }

    onDateChange = (date) => {
        console.log('[onDateChange] date', date)
        this.setState({ date })
    }

    constructor(props) {
        super(props);
        this.state = {
            date: moment(),
            focused: false
        };
        // this.onDateChange = this.onDateChange.bind(this);
        // this.onFocusChange = this.onFocusChange.bind(this);
        // this.onClick = this.onClick.bind(this);
    }

You have the three binding lines commented out; they're required for your code to work.

No, they are not.
Look at event handler signatures:
onDateChange = (date) => { ...  The code doesn't work either way.

Ah - you're right. They're required when you use prototype methods (which is superior), but not when you use instance properties (the stage 2 proposal).
I'm not sure what's causing your error. Any chance you could make a jsfiddle or repo that reproduces it?


plugin-interface: use extern "C" for plugin init function

Currently, smartvmi loads plugins dynamically by loading their shared object. From that shared object, the main entry function is searched. The problem with that is that the current approach depends on the hard-coded C++-mangled symbol name. This however depends on the toolchain (compiler version etc.). Specifically, when compiled on NixOS, the name didn't match the upstream source code version.
As this symbol name is only relevant when loading a shared object into memory as part of the shared object's metadata analysis, this name doesn't have to be unique across plugins. Therefore, to get rid of the hardcoded symbol name, we use "extern C" to make things easier and more portable.
The function was renamed from "init" to "init_plugin" to have a more expressive name.
The disadvantage is that the caller can no longer implicitly verify the function signature used when the plugin was compiled.
However, with a few more assertions, we can mitigate this mostly.

So, you didn't run into any demangling errors with the plugin api version? Should we export this as extern C nonetheless? :thinking:

I am not really happy with this solution because it makes plugin initialization less foolproof. There is not really a good way of reliably validating all input parameters on the plugin side, nor should this be the responsibility of the plugin.

Fair point. However, we should look at actual realistic use-cases of smartvmi. Correct me if I'm wrong but I don't see real use-cases in the past or the near or even far future, where smartvmi is not consumed as entire tree and built from that.
Although I understand the possible idea that smartvmi is distributed via a package manager and users create plugins on their own, VMI is much too niche for that to be actually used like that. Right? Or am I missing something? :)

Fair point. However, we should look at actual realistic use-cases of smartvmi. Correct me if I'm wrong but I don't see real use-cases in the past or the near or even far future, where smartvmi is not consumed as entire tree and built from that.

The university partners we have worked together with over the past two years mentioned that they might consider letting students write their thesis using this software and for this use case it might be handy to develop a plugin separately. I don't know how likely this is to happen though ;)

Regarding the formating issues: Most IDEs support formatting with clang-format, so you don't have to do it manually. Alternatively, you can also use the clang-format CLI tool: clang-format --style=file

Done. Are we good to go?

Done. Are we good to go?

There are still some formatting issues unfortunately.

Weird, I ran exactly the same commands locally and it worked.


[ci] Only run workflow tasks on djl repo

Change-Id: Ib0ba95795f7a7597be0733f2704e580311d1fc79
Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #1923 (e226c39) into master (bb5073f) will decrease coverage by 2.18%.
The diff coverage is 67.05%.

@@             Coverage Diff              @@
##             master    #1923      +/-   ##
============================================
- Coverage     72.08%   69.90%   -2.19%
- Complexity     5126     5681     +555
============================================
  Files           473      543      +70
  Lines         21970    25072    +3102
  Branches       2351     2713     +362
============================================
+ Hits          15838    17526    +1688
- Misses         4925     6208    +1283
- Partials       1207     1338     +131

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

.../cv/translator/StyleTransferTranslatorFactory.java
40.00% <ø> (ø)

... and 444 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


deps: Correctly consume audit-scanner v1.17.0

Description
v1.17.1 doesn't exist yet, it was incorrectly bumped by the update-cli automation PR.

Test

Additional Information
After merging this, the release for kubewarden-controller version 3.0.1 should succeed.
Tradeoff

Potential improvement

This is the fix for a known bug, see https://github.com/kubewarden/helm-charts/issues/500.


[BC-90] Add rpc timeouts

PR Description
Add timeouts to rpc requests.
Close outgoing request stream if:

Initial bytes are not received in time
Any response chunk takes too long to arrive

Close incoming request stream if:

Request bytes do not arrive in time after stream is initialized

As you mentioned elsewhere we need to follow up to ensure we don't penalise peers if we're slow reading responses because of block import times or other factors.

👍Yep - working on a follow-up PR to address this


Script cannot be launched.

Node: 7.7.0
NPM: 4.4.1
Returns error while running:
(node:<pid>) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: str.toLowerCase is not a function
(node:<pid>) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Need some more investigation


New builders without a suffix failing with an exception

Description
Like discussed in the idea #104, we would like to introduce experimental builders without a suffix, that should throw an exception in case of a failure, for all existing types.

These builders should be marked by the experimental annotation corresponding to their package.

Here's an example for the NonZeroInt type:
package kotools.types.number

import kotools.types.experimental.ExperimentalNumberApi

/**
 * Returns this number as a [NonZeroInt], which may involve rounding or
 * truncation, or throws an [IllegalArgumentException] if this number equals
 * zero.
 */
@ExperimentalNumberApi
fun Number.toNonZeroInt(): NonZeroInt = TODO("Not implemented")

println(1.toNonZeroInt()) // 1
println(0.toNonZeroInt()) // IllegalArgumentException

Dependencies
This issue is blocked by the following ones:

[ ] #114

Checklist

[ ] Implement and test it for the StrictlyPositiveInt type.
[ ] Implement and test it for the StrictlyNegativeInt type.
[ ] Implement and test it for the PositiveInt type.
[ ] Implement and test it for the NegativeInt type.
[ ] Implement and test it for the NonZeroInt type.
[ ] Implement and test it for the NotBlankString type.
[ ] Implement and test it for the NotEmptyList type.
[ ] Implement and test it for the NotEmptySet type.
[ ] Implement and test it for the NotEmptyMap type.
[ ] Refactor.
[ ] Check the generated documentation.
[ ] Check and update the public API binaries.
[ ] Update the Unreleased section in changelog.

The new builder for the StrictlyPositiveDouble type will be introduced in the issue #134.

Like discussed in #104, we will not add type converters that could fail with an exception.


‘User-defined symbols file not found’

I don’t want a symbols file, I don’t want the self-defined snippets functionality. I just want the normal autocomplete. I also don’t want to create an empty symbols.md file to clog up my vault. Is there some way to disable this functionality?

@greasycat


version update 0.22.9 -> 0.22.10

unify line endings and set them to CRLF
bump up version number to 0.22.10

:shipit:


WebviewPanel.visible doesn't seem accurate

VSCode Version: 1.23
OS Version: Windows 10 x64

It seems that WebviewPanel.visible behaves more as active rather than visible. For example, if you have an editor (file) in view column 1 and a webview panel in view column 2, I expected that visible would always be true for the panel (since it is visible), but it seems visible is only true if the panel is also active. This presented some challenges in detecting when a webview really wasn't visible, since messages won't get passed in that case.
IMO, it would be great if visible was split into two properties visible and active.
//cc @mjbvz

Moving out due to grid layout work

Verified using the webview-sample: output visible and active' in onDidChangeViewState`


[Question] Should code examples be self-contained?

@MightyPen Wondering if there is any style guideline with regards to code examples in sql-docs?
For example, https://github.com/MicrosoftDocs/sql-docs/blob/live/docs/relational-databases/system-dynamic-management-views/sys-dm-sql-referenced-entities-transact-sql.md has an Example E that extends Example D.  Example E requires clicking the copy button for Example D, pasting it into SSMS, then click the copy button for Example E, pasting that into SSMS, and then clicking execute.
In my 2 cents, I think examples that combine multiple concepts should become samples.  I realize overly restrictive rules like these can create burdens and be counter-productive, but as a developer, I just want to immediate copy-paste something and play with it.
Further, in this example, it is using AdventureWorks2012 database, but I don't see why it can't just create a database called Test instead, so that the example is immediately runnable.  Long-term, with C# moving to more interactive examples you can execute online in your browser (without an IDE), it would be nice if the SQL examples became self-contained so they could run in http://www.sqlfiddle.com/ or equivalent in the future.

@jzabroski Hi John, There is a perpetual debate about making small-to-medium size T-SQL examples dependent on AdventureWorks db (or dependent on WideWorldImporters db, etc). Certainly the majority of our SQL examples to Not rely on A.W. db, and that is very unlikely to change. As you implied, the use of A.W. db for routine SQL examples would present a barrier to quick testing by customers.
In the specific T-SQL examples 'D' and 'E' that you cited (and 'A-C'), it seems that their initial uses of the 'USE AdventureWorks2012' statement are pointless, because the rest of their T-SQL code uses nothing in A.W. db. Perhaps those two 'USE...' lines should simple be removed.
Example 'F' might be odd, because it might rely on A.W. db for the return data that would match the data shown in the article; yet 'F' lacks the 'USE AdventureWorks2012' statement.
At least those are my impressions from a quick first glance.
Our team discussed this exact question about the use of A.W. db on April/19, so we are aware of the situation. Feedback on this question, such as this feedback from you today, helps inform our discussions. I am forwarding your comments onward to the relevant documentation team.
And thanks for your mention of SqlFiddle.com.
So, you points are noted, well-taken, and have been sent to the whole SQL documentation team.  I'll think about whether/how to edit the article.
I am now Closing this Issue 2042.
(The @ notification system continues to work regardless of whether the Issue is Closed.)
CC: @stevestein

https://github.com/MicrosoftDocs/sql-docs-pr/pull/10100

@MightyPen Thanks for fixing this and all the additional context.  You're my docu-hero.

And thanks for your mention of SqlFiddle.com.

My personal favorite is actually SqlBolt.  If you go to, https://sqlbolt.com/lesson/select_queries_introduction , at the bottom of the page there is a live example.  If you add a "WHERE Year = 1995", the data grid instantly refreshes.  It's like typing on a piano, which, as Peter Norvig mentions in Teach Yourself Programming in 10 Years, is quite essential to mastery.


[SEVERE] when building flutter main();

I do basic example, expect that I do flutter app.
I get this warning on build
[SEVERE] TransformerBuilder[Reflectable] on reflection_test|lib/main.dart: Unable to resolve asset ID for "dart:ui"

Please provide more information.
How did you create the project?
What changes did you make (added dependencies)?
What is the output of flutter doctor -v?

This could well be because 'dart:ui' is a 'system library' that reflectable does not know. I've just tentatively added "dart:ui" to the set of names of known system libraries in code that will be reflectable 2.0.0-dev.3.0. This may create the danger that generated code can contain import 'dart:ui'; even outside Flutter, and it will then not compile, but that should only happen when the program already won't compile outside Flutter (so "that program is broken in this context, anyway").

In https://github.com/dart-lang/reflectable/issues/117 I asked for a setup that demonstrated the problem; it would be great if such a setup could also demonstrate this problem at the same time (maybe that comes for free?).

The issue reported initially still exists, but this reflectable & Flutter example repo allows for generating code using reflectable and running the example both from the command line and from Android Studio.

Here's another situation where the same symptom arises: https://github.com/dart-lang/build/issues/733.

Besides the dart:ui issue there is another problem occurring with version 0.6.0. Given the example repository https://github.com/eernstg/reflectable-flutter using the described steps to generate the reflectable it fails with the following Exception:
[SEVERE] reflectable on lib/main.dart:

NoSuchMethodError: The getter 'id' was called on null.
Receiver: null
Tried calling: id

Tracked in https://github.com/dart-lang/reflectable/issues/145

Fixed a null bug, published 2.0.4, and closed #145.
(But the SEVERE 'dart:ui' message that this issue deals with still exists, so this one remains open).

Updating to 2.0.4 fixed this problem for me. If the bug persist, try to upgrade Flutter and see if it fixes the problem.

Closing: I cannot reproduce the issue reported here any more. Please create a new issue if it still occurs for you.