Added new climate component from Daikin

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

[x] Documentation added/updated in home-assistant.github.io; see docs PR

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hello @rofrantz,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

3871c13fdd42f81667a796ef90ae26c270cb779e - This commit has something that looks like an email address (frantz@buadc3573500500628.rom.europe.techteam.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Hello @rofrantz,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

f407af7adb4037c6eb457be70c87d2c1824ab0e9 - This commit has something that looks like an email address (frantz@buadc3573500500628.rom.europe.techteam.com). Maybe try linking that to GitHub?.
3871c13fdd42f81667a796ef90ae26c270cb779e - This commit has something that looks like an email address (frantz@buadc3573500500628.rom.europe.techteam.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Hello @rofrantz,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

f407af7adb4037c6eb457be70c87d2c1824ab0e9 - This commit has something that looks like an email address (frantz@buadc3573500500628.rom.europe.techteam.com). Maybe try linking that to GitHub?.
3871c13fdd42f81667a796ef90ae26c270cb779e - This commit has something that looks like an email address (frantz@buadc3573500500628.rom.europe.techteam.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Hello @rofrantz,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

f407af7adb4037c6eb457be70c87d2c1824ab0e9 - This commit has something that looks like an email address (frantz@buadc3573500500628.rom.europe.techteam.com). Maybe try linking that to GitHub?.
3871c13fdd42f81667a796ef90ae26c270cb779e - This commit has something that looks like an email address (frantz@buadc3573500500628.rom.europe.techteam.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Hello @rofrantz,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

3871c13fdd42f81667a796ef90ae26c270cb779e - This commit has something that looks like an email address (frantz@buadc3573500500628.rom.europe.techteam.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

First of all thanks @tinloaf for your feedback.
I've did now some small of your comments and I would also like to add the fact that I'm now to both HA and python development
I'll try to do the rest tonight.
Once again, thanks for your feedback

@tinloaf all comments should be solved now, let me know if I missed anything

There are at least two different hardware systems Daikin is using for remote access to a unit. The one in this pull request is through SkyFi with Daikin's BRP15A61 Wi-Fi Interface. Another one is through Daikin's BRP069A42 Wi-Fi interface, exposing a different API. I have made an out-of-tree component supporting the latter (https://github.com/robhuls/home-assistant).
What would be the way forward, adding both components, or integrating them? I have looked at the code in this pull request and at first glance, an integrated component seems well possible, I would mainly need and if/then/else construct for the API calls. The configuration.yaml file would then need a specification for which of the two systems is used (beside the host name). My proposal is to wait until this pull request is part of the main code, then start a new development to merge the second system and when that is done, do a new pull request. Would that be ok?

It woukd be perfect @robhuls. That's my main goal also.
In addition to what u've said can tell u that api already provides us additional info from where we can extract de device capabilities.
I'll fix now the conflicts with master and we can merge it. What do u think @tinloaf, @andrey-git ?

I'll close this PR and open a new one since travis seems to be going crazy :)

Created https://github.com/home-assistant/home-assistant/pull/10983


Allow customization of JAVA options for root Marathon, and other changes

High-level description

post-installation environment variable configuration can now be placed in /var/lib/dcos/marathon/environment
JVM parameters can be specified via MARATHON_JAVA_ARGS

Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-3556 - Marathon DCOS packaging needs revision

Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:

Manual acceptance criteria is specified on the ticket. I've run through the steps and verified that the change works.

[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

@mesosphere-mergebot bump-ee

I believe changes need to be re bumped

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label  ready for review


IRC connection management

This might not be an issue with discord-irc itself, but it's where i saw it. Console output while the bot is running below. This problem is a bit peculiar as the bot shouldn't ever have to reconnect to IRC in this instance. The number of discord reconnects seems kind of high to me, but i don't have any reference value for those (the following logs are for a roughly three day long run).

The IRC-Server runs on the same machine/environment as the discord-irc bot.
No clients other than this bot disconnected in this period of time, most of them are connected from IRCCloud.

Output from the bot:
runner@cortex:/srv/discord-irc$ npm start -- --config ../discord-irc.conf

><EMAIL_ADDRESS>start /srv/discord-irc
> node dist/index.js "--config" "../discord-irc.conf"

info: Connected to Discord
info: Connected to IRC
info: Connected to Discord
info: Connected to Discord
info: Connected to IRC
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
(node:4618) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: getaddrinfo ENOTFOUND discordapp.com discordapp.com:443
(node:4618) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): Error: getaddrinfo ENOTFOUND discordapp.com discordapp.com:443
info: Connected to IRC
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
/srv/discord-irc/node_modules/irc/lib/irc.js:936
    if (!this.conn.requestedDisconnect) {
                  ^

TypeError: Cannot read property 'requestedDisconnect' of null
    at Client.send (/srv/discord-irc/node_modules/irc/lib/irc.js:936:19)
    at Timeout.dequeue [as _onTimeout] (/srv/discord-irc/node_modules/irc/lib/irc.js:966:22)
    at ontimeout (timers.js:365:14)
    at tryOnTimeout (timers.js:237:5)
    at Timer.listOnTimeout (timers.js:207:5)

npm ERR! Linux 3.16.0-4-amd64
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "start" "--" "--config" "../discord-irc.conf"
npm ERR! node v6.9.4
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node dist/index.js "--config" "../discord-irc.conf"`
npm ERR! Exit status 1

--- CUT LOTS OF COMPLAINING ABOUT BEING UNABLE TO WRITE LOGFILE ---

Opers channel on the server (no lines removed, only changed hostnames/ips):
00:00:00	<─>	Sunday (January 15, 2 days later)
11:20:58	<irc.abc.xyz>	CONNECT: Client connecting on port 6697 (class local): b!b@abc.xyz(<IP_ADDRESS>) [b]
00:00:00	<─>	Tuesday (January 17, 2 days later)
05:13:42	<irc.abc.xyz>	CONNECT: Client connecting on port 6697 (class local): b!b@abc.xyz(<IP_ADDRESS>) [b]
00:00:00	<─>	Wednesday (January 18)
23:34:07	<irc.abc.xyz>	CONNECT: Client connecting on port 6697 (class local): b!b@abc.xyz(<IP_ADDRESS>) [b]

In any case, thanks to the team for the continued support!

I encountered the same errors. My bot stayed logged into IRC, but didn't post Discord messages to IRC It continued to post IRC messages to Discord, though
vincent@rateless:~$ discord-irc --config discord-irc-config.json
info: Connected to Discord
info: Connected to IRC
dinfo: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
(node:30127) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Forbidden
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
info: Connected to Discord
(node:30127) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): Error: Internal Server Error
(node:30127) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 3): Error: Internal Server Error

Experienced this twice.
IRC -> Discord worked
Discord -> IRC did not work
my IRC server (Debian 7 ircd-hybrid (1:7.2.2.dfsg.2-10+deb7u1) is not on the same host as discord-irc, but the network between should be stable. Can't reproduce by interrupting the connection.
Console output
$ cd ~/node_modules/discord-irc; npm start -- --config ~/config.json

><EMAIL_ADDRESS>start /home/discordbot/node_modules/discord-irc
> node dist/index.js "--config" "/home/discordbot/config.json"

info: Connected to Discord
info: Connected to IRC

Turned on debug log in case it happens again

Might be related to my new ticket, #210 - the opers on the IRC net that happened on stated that frequent reconnections is a common problem with Node bots using the irc module

Having the same issue. Currently trying to figure it out but no luck yet. If I find a work around; I'll post it here.

Bump?

For the original issue report here, the requestedDisconnect issue should be fixed now, but that was a problem with the upstream IRC library, and occurred when the connection between the bot and the server is interrupted. The reconnecting with Discord would appear to indicate that it was a problem with your internet connection, in the original instance, as both the IRC and Discord sides of the bot were having trouble?
I'm not sure why you would be frequently disconnected from the Discord side of things and not the IRC side, but it could be a problem with the upstream discord.js library, or another internet connectivity issue.
I have noted that sometimes the bot disconnects from one side, most typically with Discord → IRC mirroring functioning but IRC → Discord not, and if I recall correctly the bot shows as offline. I'm not sure why this is and haven't managed to reproduce it on a device I'm able to debug, though.

All issues but the repeated Discord connection should now be fixed. Is that issue still present? I haven't managed to reproduce it and I expect it would be a problem with the upstream discord.js library, like I said in the last message.

(Closing for inactivity. If the issue persists, please comment in response to this with more information to have it reopened.)


[Spotify Player] ...

Extension – Spotify Player
Author: @mattisssa

Description
Steps To Reproduce

run Like Current Song or Dislike Current song

The current behavior
Getting an API error: Bad Request
The expected behavior
Executes successfully
Raycast version
Version: 1.42.2

Same here. Tried re-login to Spotify to refresh authentication, but still same error. This is the main feature I got Raycast in first place. Could you please look into it?

Also getting this.

Looks like they've updated the extension so that you have to create and use your own developer account on Spotify. But there's no place to even configure the extension with your API key and secret as far as I can tell. Will be taking a look


Allow pillow to float version in conda builds

Closes #2778

Looks good. We should have a larger conversation about unpinning all our conda deps, but I can unblock you for now :)

Will probably be a day or two before it gets deployed.

Thanks @ericholscher.

Please notify here when this gets deployed (apparently not the case yet).  Thanks!

Should be deployed now.
On Mon, Apr 10, 2017 at 11:25 AM, Antony Lee<EMAIL_ADDRESS>wrote:

Please notify here when this gets deployed (apparently not the case yet).
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rtfd/readthedocs.org/pull/2779#issuecomment-293036580,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABjprdY9Rct-3jzGyrfE0jc04fIsoQ1ks5runQjgaJpZM4MyKHm
.

--
Eric Holscher
Maker of the internet residing in Portland, Oregon
http://ericholscher.com

Well, you apparently also need to unpin sphinx_rtd_theme at least (https://readthedocs.org/projects/mplcursors/builds/5271935/) :-)

Hi @anntzer,
You should be able to modify your environment.yml:
https://github.com/anntzer/mplcursors/blob/master/environment.yml#L4
to install sphinx_rtd_theme such as we do here:
https://github.com/jupyterhub/jupyterhub/blob/master/docs/environment.yml#L14

Thanks for the pointer, it is helpful as a workaround.  But really this should be fixed at the level of RTD, no?

Actually, looking at it closer there will still likely be a mismatch depending on Python version used. Probably does make sense to unpin. Overall, we should have a discussion with the conda folks to come up with sensible defaults. cc @ericholscher

Yea, I'd really like to have more folks involved with conda on our core team, so that we can make sure we handle this properly. Nobody on our core team actively uses conda, so it's hard for us to support it.

That makes sense @ericholscher. I'll see who I can round up and worst case we will find them at PyCon ;-) Bummed to be missing WTD but my daughter is graduating from college :tada:


[Github Actions] actions/checkout@v3 should be upgraded to v4

The actions/checkout@v3 action uses node16 under the hood, which is deprecated by github.
We should switch to actions/checkout@v4, in the docs and for generating the file.
=> I'll shortly add a PR for this.

Resolved by https://github.com/denoland/fresh/pull/2286


Reload command does not use the SHELL

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf (0.42.0 (d471067))
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
From reading the man page and looking at https://github.com/junegunn/fzf/issues/1750 I got the idea that one could use FZF_DEFAULT_COMMAND in the reload action, but that does not work like advertised:
export FZF_DEFAULT_COMMAND='echo "test"' SHELL=bash

fzf  --bind 'ctrl-r:reload:$FZF_DEFAULT_COMMAND' --preview 'echo $FZF_DEFAULT_COMMAND'

When run, this displays test in the result list. When hitting ctrl-r it displays "test" (including the quotes).
So in order to use the FZF_DEFAULT_COMMAND in reload you would have to pass it to the SHELL explicitly:
export FZF_DEFAULT_COMMAND='echo "test"' SHELL=bash

fzf  --bind 'ctrl-r:reload:$SHELL -c "$FZF_DEFAULT_COMMAND"' --preview 'echo $FZF_DEFAULT_COMMAND'

When run, this displays test in the result list. When hitting ctrl-r it displays "test" (including the quotes).

try with eval

ref: bash manual - eval
ref: https://github.com/junegunn/fzf/issues/2887#issuecomment-1199258039 (29/Jul/22)

fzf  --bind 'ctrl-r:reload:eval $FZF_DEFAULT_COMMAND'

So in order to use the FZF_DEFAULT_COMMAND in reload you would have to pass it to the SHELL explicitly:
export FZF_DEFAULT_COMMAND='echo "test"' SHELL=bash

fzf  --bind 'ctrl-r:reload:$SHELL -c "$FZF_DEFAULT_COMMAND"' --preview 'echo $FZF_DEFAULT_COMMAND'

seems to be superfluous, because fzf already does that by using $SHELL -c COMMAND

man fzf
# FZF_DEFAULT_COMMAND
#   Default command to use when input is tty. On *nix  systems,  fzf runs the command
#   with $SHELL -c if SHELL is set, otherwise with sh -c, so in this case make sure
#   that the command is POSIX-compliant.


New cell types (based on term suggestions)

Based on the term suggestions, there is a number of cell tyes that we should consider adding:
aromatase expressing neuron
calbindin expressing neuron
calretinin expressing neuron
choline acetyltransferase expressing neuron
cortical neuron
cortical postmitotic cell
cortical progenitor cell
D2 receptor expressing neuron
excitatory neuron
hippocampal neuron
inhibitory neuron
layer 6 neuron
neuropeptide Y expressing neuron
nitric oxide synthase expressing neuron
@UlrikeS91 @lzehl Related to #149, let me know what naming convention you would like to use and I can prepare a PR.

@MaaikevS thanks for preparing this. There are a few that I would not register, because they are semantic combinations with brain regions and not represent necessarily their own class (but feel free to disagree, and discuss). I think we should avoid these where possible (which we did not in the past). @tgbugs your insight here?
Here the once I would not register:

cortical neuron
cortical postmitotic cell (I would only register postmitotic cell)
cortical progenitor cell (I would only register progenitor cell)
hippocampal neuron
layer 6 neuron

For the rest: my impression is that the x expressing neurons are typically written as "x-expressing neuron", if the x is one word. Not sure if that is true. The decision of going with ... neuron now instead of ... cell is good I think.
All of those decisions we make now would require us to clean up what we already have in for cell types (in particular the combination of brain region with a general cell type). I think it would be worth it.
@MaaikevS @UlrikeS91 @tgbugs what do you think?

Cell types that are now registered (and cleaned up in the KGE):
aromatase expressing neuron
calbindin expressing neuron
calretinin expressing neuron
choline acetyltransferase expressing neuron
postmitotic cell
progenitor cell
D2 receptor expressing neuron
excitatory neuron
inhibitory neuron
neuropeptide Y expressing neuron
nitric oxide synthase expressing neuron
Further discussion is need to sort out cell types that refer to an anatomical region

this will be solved with https://github.com/HumanBrainProject/openMINDS_controlledTerms/issues/257


added RobustScaler for KMeans Algorithm

Hey virat,
Try to keep the code template generalized for multiple dataset. the currently your code is specialized for single dataset. Refer the reference code available on standardScaler kmeans.

I've made changes. Would you mind taking a look at them

If you want, you can add the seasborn pair plot at the bottom

for the pair plot let the seaborn library use default markers. since currently code will throw error on different number of clusters
and create individual pull request for each files


Instances exec: exit with cmd status code

Didn't see it! But it makes sense, I'll integrate it in a subsequent PR


Add an action to "Play Next Song"

Investigate adding an action to play next song.

What music player wiuld that action control? YouTube Music? I'm am not that familiar with music apps and if there is a generic command for Android to start a default player and play music.

hi.wow that was fast.
for me it is spotify but i guess generic music control would work too as the player would anyway run in the background.

for me it would be enough to just jump to the next track when music is already playing

OK, I will experiment with  it. It needs to be a realible solution, otherwise people will get frustrated and will complain.
I think the "official" music app for android now is "You Tube Music", so this is what I will work with.

thank you in advance for your efford

No problem. Do you think you can test it if I sent you an APK?
On Tue., Jan. 16, 2024, 1:17 p.m. futtman, @.***> wrote:

thank you in advance for your efford
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/CasioGShockSmartSync/issues/67#issuecomment-1894270036,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37QJMDLPG3HUCXA5FQTYO273JAVCNFSM6AAAAABB43HP7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJUGI3TAMBTGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

@futtman
I have added this action:

Note I called int "Skip to next track", because this should work for videos, like YouTube videos, not only songs.
Here is an APK file to try:
https://drive.google.com/file/d/1TnscPpSVJgN9P1R94-5qJxDzM_k7XbJC/view?usp=sharing
It generally works for me, but sometime cannot call the action, and I need to restart to app to get it to work again. I think there is a requirement to complete the action and go mack to the initial screen before starting the actions again.
Please play with it for a day or so and let me know your experience.
Thanks

i tested your app and over all it works flawlessly. thank you for that it makes the bluetooth connection of my watch worth.
just to let you know what is not "perfect" yet: the app needs to be the last one opened - means i need to start the music, then open the watch app and then turn off the screen. is there a possibility that the app could run in the background and does not need to be reopened all the time? i am on a14 by the way on sony 5v - could be also my phones problem i know....

Hey @futtman , thanks for your feedback and the beer! Yes, running actions work best when this app is in the foreground, which is true for actions. I will try to see if I can get it to work better while running the app in the background, and will create a release 11.9 soon.

looking forward to it...thank you again

@futtman Could you try this APK? I think this will work better. Thanks.
https://drive.google.com/file/d/1zY5Lem805sIcGyMwDQ2Ie4upALeVfv1v/view?usp=sharing

hi it works with no issues

Great! This will be in version v11.9 on the Playstore and F-DROID. It takes
about a week to appear on F-DROID.
Thanks for suggesting this feature and help to test it.
On Tue, Jan 23, 2024 at 5:59 AM futtman @.***> wrote:

hi it works with no issues
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/CasioGShockSmartSync/issues/67#issuecomment-1905797382,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37WTRJQMIZWVYDX6JD3YP6JXRAVCNFSM6AAAAABB43HP7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBVG44TOMZYGI
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

thank u


Broken Datapreview

It looks like I need to re-write the Dataframe Preview.  An update to VSCode broke it.  Apparently, workspace.rootPath is now deprecated.  The folder path now is a property of an object inside workspaceFolder[].
C'est la vie.  I guess it's a good excuse to re-factor it.
On the flip side, I think large dataframes will be viewable, given they've updated the open file size to 4gb.

Hi @Ladvien, is the issue here that R: Data Preview always results in the error 'This does not appear to be a dataframe'?
If so, I think that this is because getSelection() now returns an object (with properties linesDownToMoveCursor and selectedTextArray) whereas its use in previewDataframe() assumes it will return a string.
workspace.rootPath might be deprecated but I don't get any errors in makeTmpDir() where it is used.
I have a working fix for previewDataFrame(), so I can put together a PR if that would solve this issue (until workspace.rootPath is removed, anyway).

Hey @andycraig,
Sadly, I've started a dev job and don't have much time to work on vscode-R.  Apologies.
But I'm really glad to hear you've got a fix.  I say PR and let's get it resolved.  And hey, thank you.

@Ladvien Congrats on the job! :) No worries, I should have it ready in the next couple of days.

The problem appears again.

@ohmlovesampere Oh dear, I thought this was fixed.
It's working for me on Linux and Windows, so can you provide the following?

The version of vscode-R you are running. (Needs to be at least 1.0.3.)
The operating system you are running.
A short example that I can follow to reproduce the error.
A description of what happens when you run the example. If it produces an error message, provide that.

Thank you.

@ohmlovesampere Any update on this? It's hard to investigate further without a bit more information, so I'll probably close this issue again in a couple of weeks if I don't hear anything further.

I'm seeing this too. Installed R extension, then Excel Viewer extension (original error wasn't telling me to, so it was a bit hard to find that extra dependency). Now I get "Dataframe failed to display" whenever I try to run Ctrl+Shift+P: R: Preview Environment. I can run individual lines with Ctrl+Enter.
This is on VS Code 1.33.1 running Windows 10 10.0.17763

Hi @micwebnet, thank you for posting these issues. I've reopened a separate issue for the unclear Excel Viewer dependency error message (#96). This particular issue is about Preview Dataframe, and it sounds like you're having trouble with Preview Environment. Could you create a new issue for the Preview Environment problem? Please include a short reproducible example, that someone can copy straight into an R file and run to produce the same error.
Thank you!

When I try to preview a data frame, I get the message: Visual Studio Code currently limits opening files to 20 MB.
Using VS code 1.40.0
Ubuntu 18.04
R extension 1.1.8
R tools extension 0.0.6
@Ladvien you seem to have the same desire as me to check out the df.


update sendTimeAsDateTime default property to false.

After merging this pr, the default behavior of the driver would be to send Time as time.
Reference for the current behavior in here.
Will fix issue #559

Codecov Report

Merging #572 into dev will decrease coverage by 0.1%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##                dev     #572      +/-   ##
============================================
- Coverage     46.56%   46.46%   -0.11%
+ Complexity     2221     2217       -4
============================================
  Files           108      108
  Lines         25340    25340
  Branches       4178     4178
============================================
- Hits          11800    11774      -26
- Misses        11505    11545      +40
+ Partials       2035     2021      -14

Flag
Coverage Δ
Complexity Δ

#JDBC41
46.21% <100%> (-0.13%)
2202 <0> (-11)

#JDBC42
46.36% <100%> (-0.08%)
2212 <0> (-1)

Impacted Files
Coverage Δ
Complexity Δ

.../com/microsoft/sqlserver/jdbc/SQLServerDriver.java
77.22% <100%> (ø)
25 <0> (ø)
:arrow_down:

...m/microsoft/sqlserver/jdbc/SQLServerException.java
75.2% <0%> (-1.66%)
28% <0%> (-1%)

...n/java/com/microsoft/sqlserver/jdbc/Parameter.java
61.37% <0%> (-1.08%)
63% <0%> (ø)

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
33.11% <0%> (-0.33%)
244% <0%> (-3%)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
53.33% <0%> (-0.3%)
0% <0%> (ø)

...oft/sqlserver/jdbc/SQLServerPreparedStatement.java
51.11% <0%> (-0.25%)
160% <0%> (ø)

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
62.87% <0%> (-0.13%)
0% <0%> (ø)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.58% <0%> (-0.07%)
240% <0%> (+1%)

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
45.73% <0%> (-0.04%)
276% <0%> (-2%)

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
59.75% <0%> (+0.1%)
134% <0%> (+1%)
:arrow_up:

... and 3 more

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 9fe90f7...f1674af. Read the comment docs.


added module initialization

Hi, thank you for saving my time with observers.
I just added some AMD support for this stuff

Hi @dzharikhin, thanks for the PR. However, I am not yet sure if I like to add AMD support. Doesn't the AMD ecosystem has build tools that wrap libraries automatically?
Additionally, For your interest jquery-observe may send the onAdded callback for one element multiple times. selector-observer has a fix for this (but is therefore a little bit more complex).


[DOCU-1427] Update Nginx injecting directives docs

Summary

Make sure two NGINX links are rendered correctly on versions 2.6.x-3.0.x

2.8.x is fine, 2.7.x and 2.6.x need to be updated

Add a link to the supported locations
Explain how to add the same directive multiple times

For reviewers: Is it okay that the list of supported namespaces in the "Injecting individual Nginx directives" section is copy/pasted from "NGINX Injected Directives section", but I didn't remove it in that second section? I don't think there is any way we can make this an _include file because the content from "NGINX Injected Directives section" lives in the kong/kong repo.
Reason

DOCU-1427
Testing

https://deploy-preview-4116--kongdocs.netlify.app/

@lena-larionova Could you take a look at this? It looks good to me, but I don't want Diana's work to be overwritten the next time we generate auto docs.


Fixed a bug with multiple admin bundles

In case there were two bundles on the page, all the subsequent ones were
overriding CMS namespace, which could contain widget information, which
would in turn result in errors.
forwardport of https://github.com/divio/django-cms/pull/5637

Coverage remained the same at 82.766% when pulling c7617f3150e3fddc37ed2345aa2d38b6f0956ee1 on vxsx:bugfix/widget-init-fp into 59d303bb0b07beecbbe3f18a2498e382d6022430 on divio:develop.


Extracts ApmServerRule for use in Servlet integration tests

What does this PR do?
Before we migrate to jupiter (e.g. via openrewrite), we should have code organized as typical JUnit 4 as possible. This extracts out static initialization in servlet ITs as a class rule while keeping all other logic the same.
Next steps will attempt to make TestApp more normalized, possibly via rules or nested or something similar, I'm not quite sure yet, but the goal is to be able to have tests be normal junit tests. Currently, they are loops which have complicated state because they need to peek back and clear apm logs between themselves.
Checklist
Left alone as this is test infra only

[ ] This is an enhancement of existing features, or a new feature in existing plugins

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Added an API method or config option? Document in which version this will be introduced
[ ] I have made corresponding changes to the documentation

[ ] This is a bugfix

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that would fail without this fix

[ ] This is a new plugin

[ ] I have updated CHANGELOG.asciidoc
[ ] My code follows the style guidelines of this project
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have updated supported-technologies.asciidoc
[ ] Added an API method or config option? Document in which version this will be introduced
[ ] Added an instrumentation plugin? Describe how you made sure that old, non-supported versions are not instrumented by accident.

[ ] This is something else

[ ] I have updated CHANGELOG.asciidoc

trying to figure out if this is a flake or not. locally so far so good

abandoning this as there's some weird issue in jboss related to (no idea how) on this cd893419137fd80981505c754ecea5cce6fb0bbd


feat: add offline bundle signature verification

Signed-off-by: Asra Ali<EMAIL_ADDRESS>
Adds cryptographic bundle verification following the Sigstore client spec. I expect this will be replaced by a direct call to another library in the long term
Removed a test case that was no longer relevant. That test was checking our code against the Rekor server update that returned a new return response. Now all code tests the new server response anyway.
Pipes out trusted root info outside the verification calls
Updates cosign to v2

qq: does the verification also verifies the SCT (ie the fulcio CA cert was recorded in the TLS CT log)?
something we need to do is unable this for the TLS connection as well, correct?

qq: does the verification also verifies the SCT (ie the fulcio CA cert was recorded in the TLS CT log)?

Yes! By default now the check is on

Are the leaf cert also appended to the PKI CT logs? Or only CA certs are?

Hmm not sure I totally understand this. The leaf cert is in the CT log.

qq: does the verification also verifies the SCT (ie the fulcio CA cert was recorded in the TLS CT log)?

Yes! By default now the check is on

+1

Are the leaf cert also appended to the PKI CT logs? Or only CA certs are?

Hmm not sure I totally understand this. The leaf cert is in the CT log.

Are CA certs (ie Fulcio cert) in CT log as well?
How about rekor cert (which is a leaf cert)?
Which are verified?

Are CA certs (ie Fulcio cert) in CT log as well? Do we care?

Oh, yes the CA cert is in the CT log. SCT verification being enforced now (regenerating test cases, this is why they are failing) checks the offline promise of that. We don't check them online, but neither does normal CT. So this part is now enforced/verified.

How about rekor cert (which is a leaf cert)? Do we care?

This I do not understand. Rekor signs with a public key that is retrieved from the TUF root and does not have a leaf cert.

Are CA certs (ie Fulcio cert) in CT log as well? Do we care?

Oh, yes the CA cert is in the CT log. SCT verification being enforced now (regenerating test cases, this is why they are failing) checks the offline promise of that. We don't check them online, but neither does normal CT. So this part is now enforced/verified.

How about rekor cert (which is a leaf cert)? Do we care?

This I do not understand. Rekor signs with a public key that is retrieved from the TUF root and does not have a leaf cert.

OK. I thought maybe there was a cert issued as well. Ignore my comment then.

LGTM to merge.


How to insert values into a table with optional relationship in ef?

Describe what is not working as expected.
I have those two entities, PatientRegistry and PatientAccount. The relationship between them is 1-0..1. I am trying to seed in my primary table PatientRegistry but I keep getting null errors for required fields in my optional table PatientAccount, How can I insert values in my primary table without the need to seed in my optional table as well?
Exception message:
> Unhandled Exception: Microsoft.EntityFrameworkCore.DbUpdateException:
> An error occurred while updating the entries. See the inner exception
> for details. ---> System.Data.SqlClient.SqlException: Cannot insert
> the value NULL into column 'Password', table
> 'ArtCoreDb.dbo.AspPatientsAccount'; column does not allow nulls.
> UPDATE fails.

Steps to reproduce
public class PatientRegistry {
            [DatabaseGenerated (DatabaseGeneratedOption.Identity)]
            [Display (Name = "Record Id")]
            public long RecordId { get; set; }

            [Key, DatabaseGenerated (DatabaseGeneratedOption.None)]
            [Display (Name = "Patient File Number")]
            public long PatientFileId { get; set; }

            [Required, StringLength (50)]
            [Display (Name = "First Name")]
            public string FirstName { get; set; }
            public virtual ICollection<PartnerRegistry> Partners { get; set; }
            public virtual PatientAccount PatientAccount { get; set; }
    }

    public class PatientAccount {
            [Key, DatabaseGenerated (DatabaseGeneratedOption.Identity)]
            public long RecordId { get; set; }
            [Required]
            public string Password { get; set; }
            [Required, StringLength (15)]
            public string MobileNo { get; set; }
            public bool IsConfirmedPhoneNumber { get; set; } = false;
            public bool IsConfirmedEmailAddress { get; set; } = false;
            public bool IsLocked { get; set; } = false;
            public int? FailedAttempts { get; set; }
            public DateTimeOffset? LastLoggedIn { get; set; }
            public DateTimeOffset DateCreated { get; set; }
            [Timestamp]
            public byte[] RowVersion { get; set; }
            public long? PatientFileId { get; set; }
            public virtual PatientRegistry PatientFile { get; set; }
        }

And my fluent API,
    protected override void OnModelCreating (ModelBuilder builder) {
            base.OnModelCreating (builder);
            builder.Entity<PatientRegistry> ()
                    .HasOne (a => a.PatientAccount)
                    .WithOne (b => b.PatientFile)
                    .HasForeignKey<PatientAccount> (c => c.PatientFileId)
                    .OnDelete (DeleteBehavior.Cascade);

    }

Further technical details
EF Core version: 2.0.2
Database Provider: (Microsoft.EntityFrameworkCore.SqlServer)
Operating system:
IDE: (VS CODE)

Share the code you are using to seed the table or insert data into the table.

      ```
      if (!context.PatientsRegistry.Any ()) {

                    context.PatientsRegistry.AddRange (
                        new PatientRegistry {
                            PatientFileId = 1111,
                                FirstName = "John",
                                SecondName = "M",
                                LastName = "Doe",
                                GenderId = context.Genders.Where (g => g.Name == "Male").SingleOrDefault ().Id,
                                StatusId = context.Statuses.Where (g => g.Name == "Active").SingleOrDefault ().Id,
                                NationalityId = context.Nationalities.Where (g => g.Name == "Jordanian").SingleOrDefault ().Id,
                                MaritalStatusId = context.MaritalStatus.Where (g => g.Name == "Single").SingleOrDefault ().Id,
                                OccupationId = context.Occupations.Where (g => g.Name == "MD").SingleOrDefault ().Id,
                                EthnicityId = context.Ethnicities.Where (g => g.Name == "B").SingleOrDefault ().Id,
                                CountryId = context.Countries.Where (g => g.Name == "Jordan").SingleOrDefault ().Id,
                                StateId = context.States.Where (g => g.Name == "Amman").SingleOrDefault ().Id,
                                CityId = context.Cities.Where (g => g.Name == "Abu Nusair").SingleOrDefault ().Id,
                                PatientDob = DateTime.Parse ("01/25/1985"),
                                SocialSecurityNo = "1212121SSN",
                                PatientAccount =

                        }

);
context.SaveChanges ();
                }

works only if I add ,,

new PatientAccount {
PatientFileId = 1111,
CountryCodeId = context.Countries.Where (g => g.Name == "United States of America").SingleOrDefault ().Id,
AreaCode = 424,
Email =<EMAIL_ADDRESS>MobileNo = "3244990",
IsConfirmedEmailAddress = false,
IsConfirmedPhoneNumber = false,
IsLocked = false,
Password = "213123",
},
in reality, I want to be able to insert in `PatientsRegistry` table without the need to insert into `PatientAccount`

If you are adding PatientRegistry with PatientAccount navigation set, EF will insert both the records in the database. Using Add (or AddRange) marks all reachable entities in the graph in added state, which will be inserted into database at SaveChanges. If you want to add only PatientRegistry without adding associated PatientAccount then use entity entry APIs for individual entities.
Further, the Password property is marked as required (hence non-nullable in the database). The exception message suggests that you are inserting new PatientAccount without password set. Which is not what is happening in your insert code you shared. Make sure you are only marking entities as added which you want to insert. And all the entities which are supposed to be inserted has required columns value provided (or generated)


redo go async processor

redo processor task. Need to identify it in logging
redo goroutines id. Provide processor context(something like logrus)

Use logrus.WithField for processor context


Allow additional sources, sort orders & filters to be defined for the list component

https://github.com/Adobe-Marketing-Cloud/aem-core-wcm-components/blob/be656c09ea43ea80b37e0727399aa48de3e43c8a/bundles/core/src/main/java/com/adobe/cq/wcm/core/components/models/impl/v1/ListImpl.java
As you can see, the options are now hardcoded. In most cases you will definitely want to reuse the existing options, you will probably only want to add a few more sources. And probably also a few more ways of sorting the list.
I think we can easily facilitate this with the OSGi framework - They can be separate components that are gathered and used by the List-component.

Also, the List component currently contains a lot of specifics that are related to rendering a specific list item.
I.e. showDescription, showModificationDate, getDateFormatString, linkItems, etc.

Henry, that's because until now the List only provided Pages as items. All those optional specifics make sense for Pages.
We've discussed a bit your comment and we understood this: you'd like a SPI for collecting items that could dynamically be selected from the list's configuration dialog; to that extent I think this could be achieved with a data source that would return collector types, bound to SPI implementations.
The concern I have with this approach is documenting the collectors, since the returned result will greatly differ from what you can document when you know the possible collection types.
Could you please provide more details / a minimal patch that would provide the functionality you'd like to see implemented?

Indeed:
Collectors
A collector would consist of, at a minimum, a class that defines a way to return the contents of the list (i.e. from a CAConfig, based on an inherited page property, based on a date, related articles, .... could be anything) + a way to describe it (i.e. a name and a title)
It could also contain a dialog definition that allows the author to further configure any specifics regarding the collector.
I.e. if tags are the source, then the dialog partial would contain a field that allows the author to pick the tag.
If a search query would be the result, then the dialog partial would contain a text field where the author can enter the query.
If a CAConfig is the source, then it could be a list of all configs/properties that would be allowed to function as a list source (this could be custom code, as this doesn't sound like a very generic use case - It's more about being able to do this, without having to adjust/override almost the complete list component).
Renderers
In most cases, the result would be pages (I assume letting the List-component support Assets is a whole other use case?).
Given the outcome of the collector, a list of pages, the renderer would be responsible for rendering out the resulting pages.
It can simply be a component, with again a dialog partial that allows the author to customize the rendering.
Examples:
Simple list
This would be the rendering that we have right now, the most basic rendering that you can do. The dialog could contain the various properties (showDescription, showModificationDate, linkItems, etc).
Other renders
The above renderer would be a perfect starting point. Any website that wants to show a basic list, can use the 'simple list' renderer to at least show the outcome of a certain list configuration.
However, with this approach you can add different custom renderers. Such as:

Cardview
Mosaic grid
Article
Teaser
Carousel slide
etc. etc.

The renderers could be documented on their own. They don't necessarily need to be documented in the List-documentation itself.
If any more examples are appreciated, I can share some with you. We have lots of different implementations for all our customers and it feels kind of clumsy to have to do that all the time, while we now have the chance to share our ideas and make the "ootb" list-component ready for more complex use-cases.
So, I'm not saying that the List-component itself should become very specific and very complex, I'm just saying that it would be great to have this flexibility ootb. :)

Makes sense @henrykuijpers, list component is a component that almost always returns somewhere in your project and is really hard to extend right now (without copying all of the code). This should be more pluggable indeed.

(CQ-4218127)


Limit editing area to bounding box via url api

Idea here is that task managers could pass an option to the iD url api to limit the user from editing outside a bounding box.

For projects like http://tasks.hotosm.org it'd be awesome to see the task boundary within iD, rather than having to guess.
The initial window is great, but if I need to fix the offset of the imagery i'll end up moving around and guessing the tasks boundary.

For projects like http://tasks.hotosm.org it'd be awesome to see the task boundary within iD, rather than having to guess.

Can you share a screenshot?  You should be able to see the task boundary in iD.
The only reason I'm aware of where you would not see the task boundary is if your browser is rejecting the gpx file because it is mixed content (e.g. iD is loaded with https and the gpx task boundary is loaded with http).   You should see a browser warning about this I think, or at least something in the developer console.

The only reason I'm aware of where you would not see the task boundary is if your browser is rejecting the gpx file because it is mixed content (e.g. iD is loaded with https and the gpx task boundary is loaded with http). You should see a browser warning about this I think, or at least something in the developer console.

Ah bingo, that's the issue, using Chrome so nothing I can do about that.


Error message reporting should show overall progress to be less demoralizing

 Issue by larsbergstrom
Monday Feb 10, 2014 at 22:06 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/12166
This issue was labelled with: I-wishlist in the Rust repository

Right now, rustc will report error messages only up through the highest phase it could complete. That leads developers to think they nearly have worked through all of the compilation errors only to be hit by another huge lump of them after they get to the next phase. One of our external Servo contributors described this as, "demoralizing because it looks like the number of errors is decreasing, but in reality the parser just hasn't gotten past the second line."
Ideal would obviously be to report an overall number that was more representative of the total number of errors, but even showing a measure of the overall progress (cmr suggested a counter like "125 errors (stage 3/N)") would be a huge improvement.

Closing in favor of https://github.com/rust-lang/rust/issues/12166.


My custom views are deleted every restart

DBSync 12.0.0 on Postgres 14.1 with Ubuntu 20.04.
I have added some custom views into the cexplorer schema for my application, but these views are removed when I restart the dbsync after patching the OS.  I tried adding my views to the schema files, but this causes dbsync to not start up.
How can I integrate my own custom views such that they get recreated when dbsync is restarted?

There is currently no way to do this.

How do I put in a feature request for this?

I do not consider this something that should be in db-sync. In my opinion you should figure out a way to manage this outside of db-sync.

I disagree with your opinion. Software should be extendable.

You can add a seperate schema to extend with secondary views as desired, it will not be touched :-)
Koios does something similar for views and RPCs, also helps if you're collaborating together instead of individual parallel streams

I just modified the migration scripts at the source and recompiled, now the migration scripts include my views and the hashes will align so it works.

Modifying the migration scrips and recompiling is more difficult than it should be.
Currently, migrations are applied in an order specified by the alphapbet sort of the schema files. Currently the schema directory looks like:
> ls -1 schema/
migration-1-0000-20190730.sql
migration-1-0001-20190730.sql
migration-1-0002-20190912.sql
migration-1-0003-20200211.sql
migration-1-0004-20201026.sql
migration-1-0005-20210311.sql
migration-1-0006-20210531.sql
migration-1-0007-20210611.sql
migration-1-0008-20210727.sql
migration-1-0009-20210727.sql
migration-2-0001-20211003.sql
migration-2-0002-20211007.sql
migration-2-0003-20211013.sql
migration-2-0004-20211014.sql
migration-2-0005-20211018.sql
migration-3-0001-20190816.sql
migration-3-0002-20200521.sql
migration-3-0003-20200702.sql
migration-3-0004-20200810.sql
migration-3-0005-20210116.sql
migration-3-0006-20210116.sql
migration-3-0007-20211022.sql
migration-3-9999-20200728.sql

and currently the db0-sync executable checks the md5sum of the files on disk against the ones that were present when db-sync was compiled.
The current code could be modified to only check schema files of the form migration-X-yyyyyyyy.sql where X <= 3
That means anyone could add whatever extra migrations they like as long as they use stage 4 or above.  If it breaks, whoever added the extra migrations gets to keep all the pieces.
Does that make sense? Would it help?

Yes that would make sense, and yes that would help!  Would save me the 20 minutes it takes for me to compile from source when I add a new view.
thanks


Submit KlimaDAO App section as TWA to App Stores to gain SEO Traffic and start competing

Summary
Update the site manifest.json to include a valid TWA serviceWorker.js so that we can submit it as a TWA to Google Playstore/iOS Store/Microsoft Store without having to build native apps. or refactor into react-native.
Motivation
With the advancement of the App in its current form and the ability for Fiat to Crypto ramps with wallets that enable SSO via Web2 means. The app is in a state that can be shipped onto the App stores as a TWA. Benefits of this are

TWAs update in realtime as the webApp does so no need to maintain an app CI/CD, same build pipeline works
Can start to compete against Mobile Native offsetting apps. (ex Klima.com, Earth Hero, Decarbon)
Additional SEO and Backlinking to KlimaDAO
More exposure to non-native crypto users and ability to garner feedback and refine the user journey

Cons of this proposal

TWAs are technical and prone to sometimes breaking if devs are unaware of its existence (will need an monitor setup)
Will require having a Dev account on google, microsoft and ios
App stores have little experience with dApps as TWAs and in short sometimes prematurely ban them
TWAs are still a new experience, and few devs know how to work with them

Use this issue to discuss

I love this idea. Let me know how I can help.
Maybe we could identify super stable old commits to safely base the app off of so that it's not generated from the latest? That might give us the ability to move quickly on web and selectively push out new versions to the app stores.
Also, maybe coming up with a regular release schedule would help us book reliable QA for a specific time.

@Atmosfearful this is old, maybe it was done.


⚠ Bofrost.be has degraded performance

In eeac949, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1413 ms

Resolved: Bofrost.be performance has improved in 2008bb5 after 13 minutes.


Add FACEIT Enhancer to the extensions using it

First of all, thank you very much for this great CLI.
I'd like to propose to add my browser extension FACEIT Enhancer to the list of chrome-webstore-upload-cli users. FACEIT Enhancer is using chrome-webstore-upload-cli since the very beginning and has over 390k users and growing. chrome-webstore-upload-cli has helped FACEIT Enhancer to seamlessly upload and release to the web store for the past 2 years and is still powering the deployment process. The intention of adding FACEIT Enhancer to the list is to showcase how another extension is using chrome-webstore-upload-cli.

@fregante


Specify compatible eslint version

This package should specify the versions of ESLint that it is compatible with in the peerDependencies field in package.json.

Good point. Released as 2.0.1.


Pattern()

Hi!
I am following your instructions to set up the workload for matlab r2023b.
I however receive the following error and do not find a way around this (I receive the error after the duplication finding phase):
Error using pattern
Conversion to pattern from double is not possible.
Error in pave_atomcount (line 17)
P=pattern(rep);
Error in PAVE_main (line 16)
pks = pave_atomcount(M,pks,settings,rep); %atomcount sp01
Error in PAVE_ini (line 39)
PAVE_main
Could you help me out if I am doing something wrong?

Thanks for reporting this error!
Since 2020, Matlab introduced a new internal data type called "pattern", which accidently has the same name and overwrites my defined function pattern().  Instead of installing an earlier version of matlab, an easy way to get around this is to rename pattern() in pattern.m and line 17 in pave_atomcount.  (for example: change 'pattern' into 'mypattern')
Please let me know if it works. I will update it after testing.

Hi,
I solved the matlab issue with changing the filename and also all the 'pattern'names in those files with 'mypattern'.
Then it works.
Another issue which comes up is the read table which does not read in the readtable(fullfile(pathname,'Peaklist-yeast-neg.csv')) line.
If you take it out and run this line in the code directly after the .ini files then it loads the table already on beforehand.
After that the atomcount worked again.
Yours,
Reinier Groeneveld
RU

Van: Xi Xing @.>
Verzonden: donderdag 23 november 2023 19:08
Aan: xxing9703/PAVE @.>
CC: Fakesight @.>; Author @.>
Onderwerp: Re: [xxing9703/PAVE] Pattern() (Issue #2)
Thanks for reporting this error!
Since 2020, Matlab introduced a new internal data type called "pattern", which accidently has the same name and overwrites my defined function pattern(). Instead of installing an earlier version of matlab, an easy way to get around this is to rename pattern() in pattern.m and line 17 in pave_atomcount. (for example: change 'pattern' into 'mypattern')
Please let me know if it works. I will update it after testing.
—
Reply to this email directly, view it on GitHubhttps://github.com/xxing9703/PAVE/issues/2#issuecomment-1824790879, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQSMNDKH5ABDAUXLQKEN65TYF6GI7AVCNFSM6AAAAAA7YCWTOSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRUG44TAOBXHE.
You are receiving this because you authored the thread.Message ID: @.***>


Warning about system python in MacOS but correctly uses virtualenv

Environment data
VS Code version:  1.20.1
Python Extension version:  2018.1.0
Python Version:   3.5.4 and 2.7.10
OS and version:   MacOS 10.13.3
Actual behavior
VSCode shows a warning: "Selected interpreter is macOS system python which is not recommended. Please select different interpreter" but the venv version (3.5.4) is correctly selected (see screenshot) and seems to be correctly used (the system version is 2.7.10).

Expected  behavior
The warning shouldn't be shown
Steps to reproduce:

cd to the project folder
activate the virtualenv
code .

Logs
Output from Python output panel
##########Linting Output - flake8##########
Traceback (most recent call last):
  File "/usr/local/bin/flake8", line 11, in <module>
    load_entry_point('flake8==3.5.0', 'console_scripts', 'flake8')()
  File "/usr/local/Cellar/flake8/3.5.0/libexec/lib/python2.7/site-packages/flake8/main/cli.py", line 16, in main
    app.run(argv)
  File "/usr/local/Cellar/flake8/3.5.0/libexec/lib/python2.7/site-packages/flake8/main/application.py", line 396, in run
    self._run(argv)
  File "/usr/local/Cellar/flake8/3.5.0/libexec/lib/python2.7/site-packages/flake8/main/application.py", line 383, in _run
    self.initialize(argv)
  File "/usr/local/Cellar/flake8/3.5.0/libexec/lib/python2.7/site-packages/flake8/main/application.py", line 361, in initialize
    self.parse_preliminary_options_and_args(argv)
  File "/usr/local/Cellar/flake8/3.5.0/libexec/lib/python2.7/site-packages/flake8/main/application.py", line 133, in parse_preliminary_options_and_args
    opts, args = self.option_manager.parse_known_args(args)
  File "/usr/local/Cellar/flake8/3.5.0/libexec/lib/python2.7/site-packages/flake8/options/manager.py", line 307, in parse_known_args
    self.parser.largs.append(err.opt_str)
AttributeError: 'OptionValueError' object has no attribute 'opt_str'
##########Linting Output - pylint##########
No config file found, using default configuration
##########Linting Output - mypy##########

Output from Console window (Help->Developer Tools menu)

If it is annoying you, you can set disableInstallationChecks to true in settings

well, thanks for the suggestion, but it's not a matter of be annoyed 😀
It's that the message is wrong, the editor acts differently from what it says

Please could you install the Insiders build of the extension (see here under Insiders Build).
Please install that and re-test. It may be fixed in the master branch (soon to be released).

Yes, we are working on the fix. I just meant temp workaround.

@DonJayamanne unfortunately the issue persists with the insider build (2018.2.0-alpha)

why not add an interpreter menu in the IDE itself to select the interpreter from the system?

@rpranshu I think you're talking of the menu you get if you click on the python version in the bottom bar.

Found the problem, will create fix asap.
We need to perform the check after seeing the interpreter.
  const pythonInstaller = new PythonInstaller(serviceContainer);
    pythonInstaller.checkPythonInstallation(PythonSettings.getInstance())
        .catch(ex => console.error('Python Extension: pythonInstaller.checkPythonInstallation', ex));

    // This must be completed before we can continue.
    await interpreterManager.autoSetInterpreter();

Yes, It keeps appearing for me as well as "Linting is not installed.....install" and when I clicked on install "Linting could not be found"

add this key-value line in vscode settings:
"python.pythonPath": "/usr/bin/python"

@heshiweij after add it, still appears message

Configuring Python environments :

⇧⌘P
input the words: python: s, and choose [python: Select Interpreter]
select a appropriate version you need in your project


Anonymous types must have locally-defined names in order to be stable

Suppose I have the following schema:
$ion_schema_2_0

type::{
  name: Foo,
  fields: {
    a: { fields: { x: int, y: int } },
    b: int,
  }
}

type::{
  name: Bar,
  type: list,
  element: { fields: { u: float, v: float } }
}

As it is currently implemented, the code generator might generate something like this. (I'm using Kotlin, rather than Java, for conciseness here, but the principle will still apply.)
class Foo(val a: AnonymousType1, val b: Int)

class AnonymousType1(val x: Int, val y: Int)

class Bar(val value: List<AnonymousType2>)

class AnonymousType2(val u: Double, val v: Double)

Now, suppose I add a new field to Foo, like this:
type::{
  name: Foo,
  fields: {
    a: { fields: { x: int, y: int } },
    b: int,
    c: { fields: { id: string, value: int },
  }
}

The code generator has no way of knowing how to keep the generated code stable, and would likely end up generating these classes:
class Foo(val a: AnonymousType1, val b: Int, val c: AnonymousType2)

class AnonymousType1(val x: Int, val y: Int)
class AnonymousType2(val id: String, val value: Int)

class Bar(val value: List<AnonymousType3>)

class AnonymousType3(val u: Double, val v: Double)

In this scenario, AnonymousType2 and Bar have been redefined in a way that will break the code of anyone who is using either one of these classes in any non-trivial way. This cannot be solved by sorting the anonymous types because the only sort attribute that would not result in types being moved/redefined would be the timestamp that the inline type was added to the schema for the first time.

There are a few things that I think we must do in order to fix this problem, but I'm certainly open to hearing other solutions.

All inline types must be placed inside a namespace corresponding to the parent type. In Java, that would mean e.g. org.example.AnonymousType1 would instead be org.example.Foo.AnonymousType1. In Rust, it's a little trickier because you could end up with naming clashes where you create a module called foo to hold children of Foo when there's already another module you're generating called foo, but this is solvable.
Inline types, if they have a $code_gen_name field, should use that name; otherwise...

Inline types in a fields constraint should be named after their respective fields. I.e.AnonymousType1 would be named Foo.A in Java/Kotlin.
Inline types in an element constraint should be named Element. I.e. AnonymousType2 would be named Bar.Element in Java/Kotlin.
Inline types in type or all_of should have their constraints flattened into the containing type.
Inline types in one_of constraint have two possible solutions

They should either be required to have a code-gen name (so that we can add a variant-specific annotation) OR
They should be named Variant0..VariantN, and every one_of constraint must have its own distinct counter (i.e. not a globally incremented counter). E.g. Car.Variant0, Car.Variant1, etc.

Inline types in ordered_elements should be named Element0..ElementN, and every ordered_elements constraint must have its own distinct counter.

I might be missing a case somewhere, but I think that mostly covers them all.

As per an offline discussion, all the inline type definitions must have a name defined for the initial implementation. This simplifies later adding support to infer names based on above suggested option 2(e.g. field names).
Also, as part of a 1.0 release of code generation we must add a $code_gen_name field to allow user to specify a name for inline types.

There can be few alternatives to $codegen_name in order to extend it in future to hold more details about the inline type like namespace, different names in different programming languages etc. Writing it out here for consideration,
Option 1: Use an Ion struct
We can use an Ion struct here which can hold more information about the name of inline type, like different names in different programming languages, a namespace to consider for when there is a collision of same inline names possible etc.
e.g.
type:: {
  name: uses_inline_type,
  type: {
       type: int,
       $isl_codegen: {
          java: {
              name: FooType,
          },
          rust: {
             name: Foo,
          }
          ...
      }
  }
}

Option2: Use annotations
We can use annotations for a compact syntax then the struct in Option 1.
e.g.
type:: {
  name: uses_inline_type,
  type: {
       type: int,
        $isl_codegen: [
          java::name::FooType,
          rust::name::Foo,
        ],
        ...
  }
}

If we do decide to stick with $codegen_name then we can add more constraints like $codegen_name in future and on the implementation side require all/some of them being present based on usage.
e.g.
type:: {
  name: uses_inline_type,
  type: {
       type: int,
       $codegen_name_java: FooType,
       $codegen_name_rust: Foo,
       ...
  }
}

I did some thinking about this a while ago, here is what I had come up with...
In theory, all code gen properties are one of (click on them for examples):

Language-agnostic fields that just accept the data as the field value without any language specific cases
type::{
  name: foo,
  $codegen: { ignore: true }
}

 Language-specific fields that accept a struct with language-value pairs
type::{
  name: special_type,
  $codegen: {
    native_type: {
      rust: "my_crate::types::MyType",
      kotlin: "org.example.MyType",
    }
}

Language-optional fields that can go either way and accept a language agnostic value and/or language-value pairs.

type::{
  name: foo,
  $codegen: { name: foo_object }
}

type::{
  name: bar,
  $codegen: {
    name: {
      kotlin: bar_class,
      rust: bar_struct,
      '*': bar_type, // Optional default value for other languages
    }
  }
}

It’s a backwards compatible change for a codegen config property to go from language-agnostic to language-optional or to go from language-specific to language-optional, but it’s breaking to change a field from language-optional to anything else. Therefore, we should try to make fields language-agnostic or language-specific until we have a compelling case to make it language-optional.
This pattern is especially useful because it's easy for a codegen tool to find the right values. First, find the $codegen field. Then, if you need to access a specific value, such as name, you get the name field. If the value is not a struct, then you've got the value. If it is a struct, you use the language name as the field to get the actual value.
Whatever we do, the configuration hierarchy should be codegen signifier -> property name -> language specifier so that we can avoid repetition for language-agnostic properties, and so that we can make it possible to have language specific values and a fallback value.
We could choose from many different ways to structure the keys. One alternative to the above proposal is a flatter set of keys, like this:
type::{
  name: foo,
  $codegen_name: { java: FooClass, rust: FooStruct },
  $codegen_type_hint: tuple,
}

^^ That was some from of my notes from nearly 18 months ago.
I don't think there's too much difference between different structures, but I would suggest that we don't rely on constructing field names to form something like $codegen_name_java. Having to build strings in order to create the right field names at runtime seems like a bit of an anti-pattern, and if we e.g. want to find all of the java related fields, we need to see all the fields and check for partial matches that include "java". Let's make the IonReader do that parsing work for us.
I would also suggest that we don't use a list of annotated values. I'd rather we be opinionated about a structure for the property values and force values that are similar in some way to be grouped together. I also think that the double annotations are going to be unwieldy to work with and unintuitive for users.

Single annotations could be okay. (Click to expand examples)
I don't love it, but I'm also not opposed to something like either of these:
type::{
  name: foo,
  $codegen_name: [
    java::kotlin::FooClass, // Applies to both Java and Kotlin
    rust::FooStruct,
    FooType,         // Language not specified, so this is the default value for other languages
  ]
  $codegen_type_hint: tuple,
}

type::{
  name: foo,
  $codegen: {
    name: [
      java::FooClass,
      rust::FooStruct,
    ],
    type_hint: tuple,
  }
}

Thanks for the detailed explanation Matthew! I think from your examples, language-specific sounds like the best solution to start with and infact having that specific syntax rather then the flatten one will make it much easier for user to specify everything in one place/one struct, rather then having different properties(e.g. $codegen_name, $codegen_type_hint). I also realized from your examples that this $codegen struct can be used for all inline types as well as top level types(e.g. to use something like type_hint which is useful for both top level and inline types).
So following example from your ideas on language-specific syntax, sounds like the best approach to me:
type::{
  name: foo,
  $codegen: {
    name: {
      java: "FooClass",
      rust: "FooStruct",
    }
    type_hint: {
       java: class,
       rust: struct
    }
}


[Docs] How to plot text in "axis coordinates"

Hi,
MWE:

In the bottom left corner of each axis I want to plot "s = ..." where s is the slope of the red line. It becomes unpleasant to specify the location of this text in data coordinates. I would like to specify it in "axis coordinates".
This means: I give location = (0.1, 0.1) and this plots the text in the 10% x and 10%y location of the axis. Does this make sense? In PyPlot I do this by providing a transformation, not sure if it is possible in makie. If not, consider this a feature request instead of docs. If it is possible, then please add this info here: https://makie.juliaplots.org/stable/examples/plotting_functions/text/index.html or just tell me so I do the PR.

We currently don't have this, no. For vlines! etc I do this manually by lifting off the limits observable, you could do the same. I sometimes envy matplotlib for their transform system, because it's so flexible. I'm not sure if we should try to emulate something like it as well, or if it makes sense in the overall architecture.
For your specific issue, you could just use Labels that you put in the same cells as the axes, with halign = :left and valign = :bottom as well as tellheight/width false.

Thanks, the Label solution is okayish, but certainly not perfect:

I need to pad my text with spaces for it to be clear. (and also it is very useful to be able to specify with more freedom axis proportions).
So I rename this to [FR] and let's see how it is best implemented in the future. If I had more time I would also lift the axis observable, but as this is "normal science work", I need to focus on getting the slope there and thus writing as little makie code as possible!

Label also has a padding keyword, so you could do padding = (5, 0, 5, 0) for example (left, right, bottom, top).
Or halign = 0.1 etc would also work if you'd rather have it relative to axis size.


Demo DAG and multiple progress bars

Idea is to show the connection between array names on the DAG visualization, and the progress bars for each array during a computation. Probably best done in a Jupyter notebook.

Done in https://github.com/tomwhite/cubed/blob/main/examples/lithops-add-random-local.ipynb


object has no attribute 'length' SDK v3.1

Copied from https://github.com/MicrosoftDocs/azure-docs.es-es/pull/748
object has no attribute 'length' SDK v3.1
Changed file: articles/cognitive-services/text-analytics/includes/quickstarts/python-sdk.md

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

ID: 8f57e416-a37f-e719-4a3e-58a4e6390818
Version Independent ID: dbb3821b-5793-b41b-9601-559b00e4096a
Content: Quickstart: Text mining using the Text Analytics client library - Azure Cognitive Services
Content Source: articles/cognitive-services/text-analytics/quickstarts/client-libraries-rest-api.md
Service: cognitive-services
Sub-service: text-analytics
GitHub Login: @aahill
Microsoft Alias: aahi

Thanks for reaching out, I'm thinking was a temporary bug, as I tested the code and was able to get the 'length' parameter in the output, using the code in in the article.
#please-close


[BUG]Vulnerable shared libraries might make azure-communication-calling vulnerable. Can you help upgrade to patch versions?

Hi, @anuchandy , @vcolin7, I'd like to report a vulnerability issue in com.azure.android:azure-communication-calling:2.1.0-beta.1.
Issue Description
com.azure.android:azure-communication-calling:2.1.0-beta.1 directly or transitively depends on 23 C libraries (.so) cross many platforms(such as x86-64, x86, arm64, armhf). However, I noticed that some C libraries are vulnerable, containing the following CVEs:
llibskypert.so from C project openssl(version:1.1.1i) exposed 2 vulnerabilities:
CVE-2021-3711, CVE-2021-3712
libxeengine.so from C project libpng(version:1.6.16) exposed 4 vulnerabilities:
CVE-2017-12652, CVE-2015-8472, CVE-2016-10087, CVE-2016-3751
Suggested Vulnerability Patch Versions
openssl has fixed the vulnerabilities in versions >=1.1.1l
libpng has fixed the vulnerabilities in versions >=1.6.37
Java build tools cannot report vulnerable C libraries, which may induce potential security issues to many downstream Java projects.
Could you please upgrade the above shared libraries to their patch versions?
Thanks for your help~
Best regards,
Helen Parr

Hi @HelenParr, thank you for bringing this to our attention. Could you take a look @jsaurezlee-msft? Thanks :)


Payments via turbo channel not forwarded

Description
Payments are not forwarded via turbo channel. Direct payments work correctly.
Environment

Client (A) BLW freshly installed from Google Play
Eclair fork with turbo support on server (B) side with channel opened to the payee (C).
Turbo channel opened from B to A with sufficient push amount. Opening done using lnch-vekslak

Steps for reproducing the issue

Payee (C) generates an invoice (QR code)
A scans QR code - can't pay it with message that the channel isn't opened.
Direct payment from A to B works.
Direct payment from B to C works.

The last three points were tested in that order.
Expected behavior
Forwarding should work because A can't cheat B. Actually, in the worst case A loses money, so it makes sense for A to try to spend it.

Probably B - C channel is not visible to BLW for some reason. Doubt it's specific to turbo nature of A - B channel, otherwise we'd be hearing a lot of complains from users.
Do payments to other places work? (yalls for example)
How old is B - C channel, what is the capacity there?

Strangely B - C is not visible at 1ml. The capacity should be 1500000 sat. Didn't try others yet.

Strangely B - C is not visible at 1ml

This is most likely the reason, it seems somehow channel is not announced yet.
Is the funding for B - C at least 6 blocks deep?
Do you control both B and C nodes? Both of their signatures are required for an announce, perhaps a restart would help.

The B - C channel has > 450 confirmations.
I restarted both nodes, now the channel is visible on 1ml.
Unfortunately, it still doesn't work. :( However, the message is "Failed and returned" now.

Propagation may take some time occasionally, should definitely normalize within a few hours unless this is something new, please let me know.

Also it may in fact work already, but B - C channel maybe returning an error. Detailed bug report can be obtained by tapping a failed payment and then "Share debug info" (with a caveat: debug info is not persisted, will be erased after app restart).

So we tested it after the channel opened and it worked. (Although considerably slower than D - C direct payment, where D is also BLW - if you have any tips to speed this up, it'd be very helpful.)
We tried second time on another two devices - and they worked. Closing this now, will reopen if something comes up.
Thanks a lot!


with-react-relay-network-modern - non-functional example

Examples bug report
with-react-relay-network-modern
Describe the bug
At first the data is there but then it disappears and is null thus throwing this error.
TypeError: Cannot read property 'modern' of undefined
    at getNode (/Users/devionvillegas/Developer/NEXTGEN/with-react-relay-network-modern-app/node_modules/relay-runtime/lib/RelayModernGraphQLTag.js:25:71) ...

To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

download the example with npx create-next-app
run npm run schema
run npm run relay
console.log props inside the BlogPosts.js component
run npm run dev
error is thrown along with two console.log outputs

Expected behavior
You can see that on the first render, the data is there which means something else is triggering a re-render and making the array null. I can't exactly figure it out as I'm used to using apollo and this was my first time trying to see what relay was about. Hopefully the other examples work. Was hoping it would be trivial to use relay with nextjs.

We welcome a motivated contributor to fix this example (or debug what's wrong here). 😄


Parallelize distance field creation

With 90% of CPU time, the biggest bottleneck so far is the computation of the distance fields. By generating multiple distance fields in parallel with OpenMP we can achieve a significant speedup
Some work has already been done, see #59. So far, these changes have roughly halved the runtime on Linux, however they have not been tested on MacOS and Windows yet.
Also, the Travis tests for Clang on Linux currently fail because of travis-ci/travis-ci#8613.

Tested on Windows. It works and performance seems better (didn't run a benchmark, so I don't have any numbers).

Travis issue is fixed in PR #69


Support Marathon 0.15.0+ JSON definition for 422 error responses.

The errors poperty key for 422 HTTP error codes was renamed to details in Marathon 0.15.0. (See the diff on Github.com.)  In order to support both old and newer Marathon versions, we duplicate the
details JSON sub-structure in the overall JSON structure under both the old and new JSON property key.
As a minor cleanup along the way, we remove spaces from a space/tab-mixed intentation and align properly in related tests.

@gambol99, if you have time could you PTAL? Thanks!

:+1:  LGTM


Add curl extension to composer.json

The curl extension is not installed by default (on the popular platforms).
Please add it as a requirement to composer.
All other changes are typo fixes and PHPDoc corrections.

Thanks @SpacePossum!


Skip server rebuilds for change sets that don't resolve to any file mount

I'm not sure if this is a bug or an enhancement... or perhaps there's a good reason for the current behavior.
[[module.mounts]]
source = 'content'
target = 'content'
excludeFiles = ['posts/*']

With the above, when I edit content/posts/post-1.md ...
DEBUG Received System Events: [WRITE         "/home/jmooring/code/hugo-testing/content/posts/post-1.md" WRITE         "/home/jmooring/code/hugo-testing/content/posts/post-1.md"]

Change detected, rebuilding site (#5).
2024-09-19 10:43:59.783 -0700
DEBUG cachebuster: Matching "content/posts/post-1.md" with source "(postcss|tailwind)\\.config\\.js": no match
Source changed /posts/post-1.md
DEBUG
INFO  build:  step process substep resolve page output change set changes 0 checked 0 matches 0 duration 104.933µs
INFO  build:  step process substep gc dynacache duration 338.639µs
INFO  build:  step process substep collect files 0 files_total 0 pages_total 0 resources_total 0 duration 460.679µs
INFO  build:  step process duration 1.727645ms
INFO  build:  step assemble duration 314.482µs
INFO  build:  step render substep pages site en outputFormat html duration 95.511µs
INFO  build:  step render substep pages site en outputFormat rss duration 92.611µs
INFO  build:  step render pages 0 content 0 duration 449.541µs
INFO  build:  step render deferred count 0 duration 4.045µs
INFO  build:  step postProcess duration 16.526µs
INFO  build:  duration 3.17658ms
Web Server is available at http://localhost:1313/ (bind address <IP_ADDRESS>)
Total in 3 ms
INFO  livereload: build changed 0 files

I can see from the last line that no files have changed, which is good, but it seems like we shouldn't be doing the other stuff.
Tested with hugo v0.135.0-DEV-a619deec1827ea2d601a9b3708d1375c1eca3c5c+extended linux/amd64

Note that using the ignoreFiles setting in the root of the site configuration produces the desired results, but we've been recommending the use mount exclusions instead.

The issue title here isn't correct. We do honor mount ignore files setting for all builds, meaning: We don't build them. We trigger a noop build when those files changes, which I guess is a difference between the global setting, but I would say that that is undocumented behaviour. We could and should probably skip builds for change sets that don't resolve to any files on the file system.

Note that using the ignoreFiles setting in the root of the site configuration produces the desired results

ignoreFiles only works for content files, which isn't the desired result.

Thanks for the clarification. This obviously isn't a high priority.


about  run error

ConnectionResetError: [WinError 10054] 远程主机强迫关闭了一个现有的连接。

Please provide a more detailed description of how to reproduce this issue, including the following:

Steps taken on the GUI.
The version of Python and Gradio being used.

Please provide a more detailed description of how to reproduce this issue, including the following:

Steps taken on the GUI.
The version of Python and Gradio being used.

: windows 10
[environmental information]:
python: 3.9.16
Gradio: 3.25.0
gradio-client:0.1.3
torch:  2.0 cuda 11.7
mmcv:2.0

Please provide a more detailed description of how to reproduce this issue, including the following:

Steps taken on the GUI.
The version of Python and Gradio being used.

 : windows 10 [environmental information]: python: 3.9.16 Gradio: 3.25.0 gradio-client:0.1.3 torch: 2.0 cuda 11.7 mmcv:2.0

Regarding ConnectionResetError, it may be due to an issue with the version of Gradio. You can try updating Gradio using pip install gradio -U.
As for the inpaint error, kindly monitor the progress of issue #22.


Geo-Partitioning Doc Update

Re: Geo-Partitioning
https://www.cockroachlabs.com/docs/stable/training/geo-partitioning.html#writes
Issue Description
The write performance was 2x network RTT of the nearest datacenter after partitioning.   Below was used to setup the cluster.
roachprod create $f --gce-zones us-east1-b,us-east4-a,us-west1-a --geo --nodes 3
Suggested Resolution

write expected performance for a known topology that can be reproduced as a part of training
@robert-s-lee to investigate why write is 2x

Below is actual figures
oot@localhost:26257/movr> INSERT INTO users VALUES (gen_random_uuid(), 'new york', 'New Yorker', '111 West Street', '9822222379937347');
INSERT 1

Time: 28.961306ms

root@localhost:26257/movr> select id,city from users where id='112360c6-4a73-489f-84af-4a7d8bf37fb8' and city = 'new york';
                   id                  |   city
+--------------------------------------+----------+
  112360c6-4a73-489f-84af-4a7d8bf37fb8 | new york
(1 row)

Time: 15.00465ms

@robert-s-lee, since the training module uses a local cluster, I think your suggestions fits better for the geo-partitioning tutorial I'm writing, which will use a cluster distributed across 3 regions.

Incorporated into https://github.com/cockroachdb/docs/issues/4461.


Problem with PictureBox object and ImageLocation

I am using the opensource report designer to create an frx file. This file has a PictureObject as shown here:
   <TableCell Name="Cell9" Border.Lines="All" Border.Width="0.5" Text="[dtList.Surname]" VertAlign="Center" Font="Arial, 9pt"/>
      <TableCell Name="Cell25" Border.Lines="All" Border.Width="0.5" Font="Arial, 10pt" RowSpan="4">
        <PictureObject Name="PicBxPhoto" Width="75.6" Height="75.6" ImageLocation="[dtList.Photo]" ShowErrorImage="true"/>
      </TableCell>

dtList is a datatable and the Photo column contains the file path of an image, e.g. C:\img\pic1.png. This is done in C# (ASP.NET)
The problem is while the report  displays all 10 customers surnames, the picturebox  is empty. The photo (png) file path or image location is correctly formed.
I have also tried setting the dtList.Photo with the byte array of the image and used Image="dtList.Photo" in the frx file,  that does not display any of the photos either.
Can anyone shed some light on this? Can the Picture Object be used in the way that I am trying?
Many thanks in advance.

You need to set the DataColumn property of the Picture object if you want it to get pictures from the registered datasources. Like in one of our demo reports:


[WIP] Vortex impl changes to enable multiple stream append clients per vortex-stream and per JVM

incorporated changes from https://github.com/apache/beam/pull/17417/files, removed sync blocks when accessing the cache (since its guaranteed to works as a ConcurrentHashMap) and added a list of StreamAppendClients to be held by the cache so we can scale throughput that way.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?