TTS doesn't work on Slimproto Group

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

[X] Yes

The problem
Using TTS targeted to a Slimproto group doesn't seem to work. The music stops playing as if TTS is playing but no sound.
How to reproduce
Play a TTS message targeted to a Slimproto group
Music Providers
Spotify
Player Providers
Slimproto Group
Full log output
2024-03-27 17:17:39.977 ERROR (MainThread) [music_assistant.webserver] Error handling message: players/cmd/play_announcement: list indices must be integers or slices, not float
2024-03-27 17:17:51.697 INFO (MainThread) [music_assistant.players] Playback announcement to player Kitchen Player (with pre-announce: False): http://<IP_ADDRESS>:8123/api/tts_proxy/8c2bbe03546e470d11cf791840d6b1c8de0a6098_en-ie_b51835fb6e_cloud.mp3
[mp3 @ 0x55d50c874440] Estimating duration from bitrate, this may be inaccurate
Input #0, mp3, from 'http://<IP_ADDRESS>:8123/api/tts_proxy/8c2bbe03546e470d11cf791840d6b1c8de0a6098_en-ie_b51835fb6e_cloud.mp3':
  Metadata:
    Text            : Cloud
  Duration: 00:00:03.98, start: 0.000000, bitrate: 50 kb/s
  Stream #0:0: Audio: mp3, 24000 Hz, mono, fltp, 48 kb/s
Stream mapping:
  Stream #0:0 -> #0:0 (mp3 (mp3float) -> flac (native))
Press [q] to stop, [?] for help
[flac @ 0x55d50c89f8c0] encoding as 24 bits-per-sample, more is considered experimental. Add -strict experimental if you want to encode more than 24 bits-per-sample
Output #0, flac, to 'pipe:':
  Metadata:
    Text            : Cloud
    encoder         : Lavf60.3.100
  Stream #0:0: Audio: flac, 44100 Hz, stereo, s32 (24 bit), 128 kb/s
    Metadata:
      encoder         : Lavc60.3.100 flac
size=       0kB time=-577014:32:22.77 bitrate=  -0.0kbits/s speed=N/A
[flac @ 0x55d50c884180] unable to rewrite FLAC header.
size=     263kB time=00:00:04.00 bitrate= 538.0kbits/s speed=16.9x
video:0kB audio:255kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 3.183712%
2024-03-27 17:18:00.334 INFO (MainThread) [music_assistant.players] Playback announcement to player Kitchen Player (with pre-announce: False): http://<IP_ADDRESS>:8123/api/tts_proxy/8c2bbe03546e470d11cf791840d6b1c8de0a6098_en-ie_b51835fb6e_cloud.mp3
2024-03-27 17:18:00.335 ERROR (MainThread) [music_assistant.webserver] Error handling message: players/cmd/play_announcement: list indices must be integers or slices, not float
2024-03-27 17:18:04.195 INFO (MainThread) [music_assistant.players] Playback announcement to player Kitchen Player (with pre-announce: False): http://<IP_ADDRESS>:8123/api/tts_proxy/8c2bbe03546e470d11cf791840d6b1c8de0a6098_en-ie_b51835fb6e_cloud.mp3
[mp3 @ 0x55fd3a421440] Estimating duration from bitrate, this may be inaccurate
Input #0, mp3, from 'http://<IP_ADDRESS>:8123/api/tts_proxy/8c2bbe03546e470d11cf791840d6b1c8de0a6098_en-ie_b51835fb6e_cloud.mp3':
  Metadata:
    Text            : Cloud
  Duration: 00:00:03.98, start: 0.000000, bitrate: 50 kb/s
  Stream #0:0: Audio: mp3, 24000 Hz, mono, fltp, 48 kb/s
Stream mapping:
  Stream #0:0 -> #0:0 (mp3 (mp3float) -> flac (native))
Press [q] to stop, [?] for help
[flac @ 0x55fd3a44c8c0] encoding as 24 bits-per-sample, more is considered experimental. Add -strict experimental if you want to encode more than 24 bits-per-sample
Output #0, flac, to 'pipe:':
  Metadata:
    Text            : Cloud
    encoder         : Lavf60.3.100
  Stream #0:0: Audio: flac, 44100 Hz, stereo, s32 (24 bit), 128 kb/s
    Metadata:
      encoder         : Lavc60.3.100 flac
size=       0kB time=-577014:32:22.77 bitrate=  -0.0kbits/s speed=N/A
[flac @ 0x55fd3a431180] unable to rewrite FLAC header.
size=     263kB time=00:00:04.00 bitrate= 538.0kbits/s speed=  17x
video:0kB audio:255kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 3.183712%
2024-03-27 17:18:43.268 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue Kitchen Party Player - crossfade: True
2024-03-27 17:27:56.977 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue Kitchen Party Player - crossfade: True
2024-03-27 17:59:12.897 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue Kitchen Party Player - crossfade: True
2024-03-27 18:06:09.075 INFO (MainThread) [music_assistant.providers.spotify] Successfully logged in to Spotify as dezmond95
2024-03-27 18:14:42.052 INFO (MainThread) [music_assistant.players] Playback announcement to player Kitchen Player (with pre-announce: False): http://<IP_ADDRESS>:8123/api/tts_proxy/3992f8886671ba3331b1584ad82df2ae910459a3_en-ie_e09b5a0968_cloud.mp3
2024-03-27 18:14:42.052 INFO (MainThread) [music_assistant.players] Playback announcement to player Kitchen Party Player (with pre-announce: None): http://<IP_ADDRESS>:8123/api/tts_proxy/3992f8886671ba3331b1584ad82df2ae910459a3_en-ie_e09b5a0968_cloud.mp3
2024-03-27 18:14:42.070 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue Kitchen Party Player - crossfade: False
2024-03-27 18:14:42.137 WARNING (MainThread) [music_assistant.streams] Received unhandled GET request to /multi/syncgroup_rfddassw/cyUj6TpTt7vMGJy8q5JJfR/02:42:ac:14:00:02/http://<IP_ADDRESS>:8123/api/tts_proxy/3992f8886671ba3331b1584ad82df2ae910459a3_en-ie_e09b5a0968_cloud.mp3.flac from <IP_ADDRESS>
headers: <CIMultiDictProxy('Host': '<IP_ADDRESS>:8097', 'Connection': 'close', 'Accept': '*/*', 'Cache-Control': 'no-cache', 'User-Agent': 'VLC/3.0.9 LibVLC/3.0.9', 'Range': 'bytes=0-')>

2024-03-27 18:14:42.167 WARNING (MainThread) [music_assistant.streams] Received unhandled GET request to /multi/syncgroup_rfddassw/cyUj6TpTt7vMGJy8q5JJfR/d8:3a:dd:39:05:a2/http://<IP_ADDRESS>:8123/api/tts_proxy/3992f8886671ba3331b1584ad82df2ae910459a3_en-ie_e09b5a0968_cloud.mp3.flac from <IP_ADDRESS>
headers: <CIMultiDictProxy('Host': '<IP_ADDRESS>:8097', 'Connection': 'close', 'Accept': '*/*', 'Cache-Control': 'no-cache', 'User-Agent': 'VLC/3.0.9 LibVLC/3.0.9', 'Range': 'bytes=0-')>

2024-03-27 18:14:52.643 ERROR (MainThread) [music_assistant.streams] Abort multi client stream job for queue Kitchen Party Player: clients did not connect within timeout
Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/music_assistant/server/controllers/streams.py", line 239, in _stream_job_runner
    await self._all_clients_connected.wait()
  File "/usr/local/lib/python3.12/asyncio/locks.py", line 212, in wait
    await fut
asyncio.exceptions.CancelledError

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/music_assistant/server/controllers/streams.py", line 238, in _stream_job_runner
    async with asyncio.timeout(10):
  File "/usr/local/lib/python3.12/asyncio/timeouts.py", line 115, in __aexit__
    raise TimeoutError from exc_val
TimeoutError
2024-03-27 18:15:15.627 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue Kitchen Party Player - crossfade: True
2024-03-27 18:20:07.789 WARNING (MainThread) [async_upnp_client.profiles.profile] Failed (re-)subscribing to: uuid:79892097-b3f1-e476-6c2c-9481fc2262ad, reason: UpnpConnectionError('ClientConnectorError(ConnectionKey(host=\'<IP_ADDRESS>\', port=1703, is_ssl=False, ssl=True, proxy=None, proxy_auth=None, proxy_headers_hash=None), OSError(113, "Connect call failed (\'<IP_ADDRESS>\', 1703)"))', None)
2024-03-27 18:56:45.996 INFO (MainThread) [music_assistant.players] Playback announcement to player Kitchen Party Player (with pre-announce: None): http://<IP_ADDRESS>:8123/api/tts_proxy/0504cbae68c58937ad5a5dea1a66de58300d3504_en-ie_b51835fb6e_cloud.mp3
2024-03-27 18:56:46.556 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue Kitchen Party Player - crossfade: False
2024-03-27 18:56:46.614 WARNING (MainThread) [music_assistant.streams] Received unhandled GET request to /multi/syncgroup_rfddassw/5B3jhRTDMPtqkdzVrht4Rk/d8:3a:dd:39:05:a2/http://<IP_ADDRESS>:8123/api/tts_proxy/0504cbae68c58937ad5a5dea1a66de58300d3504_en-ie_b51835fb6e_cloud.mp3.flac from <IP_ADDRESS>
headers: <CIMultiDictProxy('Host': '<IP_ADDRESS>:8097', 'Connection': 'close', 'Accept': '*/*', 'Cache-Control': 'no-cache', 'User-Agent': 'VLC/3.0.9 LibVLC/3.0.9', 'Range': 'bytes=0-')>

2024-03-27 18:56:46.643 WARNING (MainThread) [music_assistant.streams] Received unhandled GET request to /multi/syncgroup_rfddassw/5B3jhRTDMPtqkdzVrht4Rk/02:42:ac:14:00:02/http://<IP_ADDRESS>:8123/api/tts_proxy/0504cbae68c58937ad5a5dea1a66de58300d3504_en-ie_b51835fb6e_cloud.mp3.flac from <IP_ADDRESS>
headers: <CIMultiDictProxy('Host': '<IP_ADDRESS>:8097', 'Connection': 'close', 'Accept': '*/*', 'Cache-Control': 'no-cache', 'User-Agent': 'VLC/3.0.9 LibVLC/3.0.9', 'Range': 'bytes=0-')>

2024-03-27 18:56:57.137 ERROR (MainThread) [music_assistant.streams] Abort multi client stream job for queue Kitchen Party Player: clients did not connect within timeout
Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/music_assistant/server/controllers/streams.py", line 239, in _stream_job_runner
    await self._all_clients_connected.wait()
  File "/usr/local/lib/python3.12/asyncio/locks.py", line 212, in wait
    await fut
asyncio.exceptions.CancelledError

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/music_assistant/server/controllers/streams.py", line 238, in _stream_job_runner
    async with asyncio.timeout(10):
  File "/usr/local/lib/python3.12/asyncio/timeouts.py", line 115, in __aexit__
    raise TimeoutError from exc_val
TimeoutError

Additional information
No response
What version of Home Assistant Core are your running
2024.2.5
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

This should be fixed in b130. Advise if not and we can reopen

I just had a very similar issue and could not find a way to fix this. Using TTS targeted to a Slimproto player or a RaspberryPi does not work. The music stops playing as if TTS is playing but no sound.
In my logs I found an issue: The IP-adress is referring to an old instance of my Home Assistant OS, when I used a different device as Home Assistant-Server in my network. But nowadays the IP and the port are wrong. How and where can I change those?

AttributeError: 'StreamDetails' object has no attribute 'seconds_skipped'. Did you mean: 'seconds_streamed'?
2024-04-08 02:02:12.957 INFO (MainThread) [music_assistant.players] Playback announcement to player Squeezeplayer Moto G (5S) (Logitech Media Server (with pre-announce: True): http://192.168.178.XX:8123/api/tts_proxy/a94a8fe5ccb19ba61c4c0873d391e987982fbbd3_de-de_6e8b81ac47_cloud.mp3
2024-04-08 02:02:12.957 ERROR (MainThread) [music_assistant.webserver] Error handling message: players/cmd/play_announcement: unhandled errors in a TaskGroup (1 sub-exception)
2024-04-08 02:05:43.489 ERROR (MainThread) [music_assistant] Error doing task: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/local/lib/python3.12/site-packages/music_assistant/server/providers/deezer/init.py", line 494, in on_streamed
await self.gw_client.log_listen(last_track=streamdetails)
File "/usr/local/lib/python3.12/site-packages/music_assistant/server/providers/deezer/gw_client.py", line 182, in log_listen
"seek": 1 if last_track.seconds_skipped else 0,
^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'StreamDetails' object has no attribute 'seconds_skipped'. Did you mean: 'seconds_streamed'?
2024-04-08 02:05:XX.447 INFO (MainThread) [music_assistant.players] Playback announcement to player Squeezeplayer Moto G (5S) (Logitech Media Server (with pre-announce: True): http://192.168.178.XX:8123/api/tts_proxy/475f7ae642916bf5cb816dfbe99dee2e95593310_de-de_6e8b81ac47_tts.home_assistant_cloud.mp3
2024-04-08 02:06:02.489 ERROR (MainThread) [music_assistant.webserver] **Error handling message: players/cmd/play_announcement: Unable to retrieve info for http://192.168.178.XX:8123/api/tts_proxy/**475f7ae642916bf5cb816dfbe99dee2e95593310_de-de_6e8b81ac47_tts.home_assistant_cloud.mp3: Connection timed out
2024-04-08 02:06:32.286 INFO (MainThread) [music_assistant.players] Playback announcement to player Squeezeplayer Moto G (5S) (Logitech Media Server (with pre-announce: True): http://192.168.178.XX:8123/api/tts_proxy/475f7ae642916bf5cb816dfbe99dee2e95593310_sw-cd_-tts.piper.mp3
2024-04-08 02:06:34.613 ERROR (MainThread) [music_assistant.webserver] Error handling message: players/cmd/play_announcement: Unable to retrieve info for http://192.168.178.XX:8123/api/tts_proxy/475f7ae642916bf5cb816dfbe99dee2e95593310_sw-cd-tts.piper.mp3: No route to host
2024-04-08 02:08:32.445 INFO (MainThread) [music_assistant.players] Playback announcement to player raspberrypi (with pre-announce: True): http://192.168.178.XX:8123/api/tts_proxy/475f7ae642916bf5cb816dfbe99dee2e95593310_sw-cd-tts.piper.mp3
2024-04-08 02:08:46.710 ERROR (MainThread) [music_assistant.webserver] Error handling message: players/cmd/play_announcement: Unable to retrieve info for http://192.168.178.XX:8123/api/tts_proxy/475f7ae642916bf5cb816dfbe99dee2e95593310_sw-cd-_tts.piper.mp3: No route to host
2024-04-08 02:08:51.246 INFO (MainThread) [music_assistant.players] Playback announcement to player raspberrypi (with pre-announce: True): http://192.168.178.XX:8123/api/tts_proxy/475f7ae642916bf5cb816dfbe99dee2e95593310_de-de_6e8b81ac47_tts.home_assistant_cloud.mp3
2024-04-08 02:09:05.142 ERROR (MainThread) [music_assistant.webserver] Error handling message: players/cmd/play_announcement: Unable to retrieve info for http://192.168.178.XX:8123/api/tts_proxy/475f7ae642916bf5cb816dfbe99dee2e95593310_de-de_6e8b81ac47_tts.home_assistant_cloud.mp3: No route to host
2024-04-08 02:17:25.847 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue raspberrypi - crossfade: False
2024-04-08 02:21:21.799 ERROR (MainThread) [music_assistant] Error doing task: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/local/lib/python3.12/site-packages/music_assistant/server/providers/deezer/init.py", line 494, in on_streamed
await self.gw_client.log_listen(last_track=streamdetails)
File "/usr/local/lib/python3.12/site-packages/music_assistant/server/providers/deezer/gw_client.py", line 182, in log_listen
"seek": 1 if last_track.seconds_skipped else 0,
^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'StreamDetails' object has no attribute 'seconds_skipped'. Did you mean: 'seconds_streamed'?
2024-04-08 02:23:14.944 INFO (MainThread) [music_assistant.players] Playback announcement to player raspberrypi (with pre-announce: True): http://192.168.178.XX:8123/api/tts_proxy/1e0bc0a9623dd93bb05ca2e0d4b3862b6859e406_de-de_6e8b81ac47_cloud.mp3
2024-04-08 02:23:27.350 ERROR (MainThread) [music_assistant.webserver] Error handling message: players/cmd/play_announcement: Unable to retrieve info for http://192.168.178.XX:8123/api/tts_proxy/1e0bc0a9623dd93bb05ca2e0d4b3862b6859e406_de-de_6e8b81ac47_cloud.mp3: No route to host
2024-04-08 02:37:07.463 INFO (MainThread) [music_assistant.players] Playback announcement to player raspberrypi (with pre-announce: True): http://192.168.178.XX:8123/api/tts_proxy/1e0bc0a9623dd93bb05ca2e0d4b3862b6859e406_de-de_6e8b81ac47_cloud.mp3
2024-04-08 02:37:21.910 ERROR (MainThread) [music_assistant.webserver] Error handling message: players/cmd/play_announcement: Unable to retrieve info for http://192.168.178.XX:8123/api/tts_proxy/1e0bc0a9623dd93bb05ca2e0d4b3862b6859e406_de-de_6e8b81ac47_cloud.mp3: No route to host
2024-04-08 02:38:48.631 INFO (MainThread) [music_assistant.players] Playback announcement to player raspberrypi (with pre-announce: True): http://192.168.178.XX:8123/api/tts_proxy/1e0bc0a9623dd93bb05ca2e0d4b3862b6859e406_de-de_6e8b81ac47_cloud.mp3
2024-04-08 02:39:03.285 ERROR (MainThread) [music_assistant.webserver] Error handling message: players/cmd/play_announcement: Unable to retrieve info for http://192.168.178.XX:8123/api/tts_proxy/1e0bc0a9623dd93bb05ca2e0d4b3862b6859e406_de-de_6e8b81ac47_cloud.mp3: No route to host

Please go and read the troubleshooting section of the docs

Thank you!

Make sure the HA internal url is set correctly. HA SETTINGS>>SYSTEM>>NETWORK>>Home Assistant URL>>Local network (set to automatic or use your internal HA IP). If it is automatic you can try changing it to http://your.internal.ip:8123/

This was my problem. I changed to automatic and now the TTS-Sound is played!
On one of my players I now have an obviously not MA-related issue: After the tts-sound it cracks and then jumps to the next song. But on a slimproto-/Squeezelight-Player it works fine. Thank you a lot for your awesome work!!!

That might be an MA issue. Is it DLNA per chance? Either way if it is a problem for you open a new issue with all the info requested on the troubleshooting page and on the issue template


fix variables in clustercheck.epp

With EPP, they need to be full namespace, or local scope.
I choose to use full namespace for this fix

Merged, thanks!


Cmake build

Hi!
These days CMake became a de-facto standard for building and distributing C/C++ projects.
So i've add CMake build support for SOIL.
Cons:

clib and automake builds are probably slightly broken because of moving compile defs from actually compile defs to CMake-configured header file (passing string defs to compiler is broken on Windows).

Pros:

it's possible to build/install SOIL as static or shared libs and easlily discover with CMake
it works for *nix, Windows and Android (i personally use it for Android project) and will work (after i get mac device :) ) on iOs.
it's able to provide custom headers/libraries to SOIL build (needed since we have amount of different OpenGL versions these days)
doxygen documentation autogenerates while building if doxygen installed

Also i've added names to enums in SOIL.h just to slightly improve autodocs generation.
Thanx for your attention and criticism :)

unfortunately, I no longer have control of this repository so someone from littlstar will have to review, and merge. This repository should probably be archived as anyone who would maintain is no longer able to

Understood.
Thank You.


Inline fix

See #37

ah, of course, "or self". Thanks. WE should merge once travis completes.

and with "we" I mean "you".

I did it myself now.

Ok great :)


[en] update pkg.go link

upd

/close
The generated files are fixed from the upstream source code, not here.


which python version?

Hi,
I used Python 3.8. I will add this information to the repository.
Thanks.

docker is better.


adding .yaml to yml definitions block

Weird. This change was added in #2 and it's missing.

yeah, I remember you adding it and thought it weird.  Not a huge change so was easy to do again


Implement FromStr for Pixel

Uses av_get_pix_fmt() like the -pix_fmt option in ffmpeg.
One question: maybe it's better to return an Err if the returned pixel format is AV_PIX_FMT_NONE (which corresponds to unknown / not found pixel format)? Currently Ok(Pixel::None) is returned.

I'd say yes, return an error if it cannot be parsed, and I also honestly would just ignore the CString error, if the user is passing \u0000 he probably fucked up somewhere.

It's better to not ignore the null byte error because the user could be getting the &str from somewhere else out of their control.


Feature request: Can styled-components add support for nested properties like SCSS

for example:
const StyledDiv = styled.div`
  padding: {
    left: 20px;
    top: 30px;
  }
`

this would be great

Closing due to inactivity; we're also sticking to CSS-like behaviour with only nesting being added, so I don't think this would be a necessary or tremendously useful addition 😅  Sorry!


Add aws_codestar_notificaton_rule table

I recently discovered this project and it is a life saver! I was missing the ability to query CodeStar notification rules (used e.g. to set up notifications of CodePipeline build status) so had a go at implementing this.
This is my first time working with Go so I don't expect this PR is ready to be merged and at this point could use some input - e.g. what tests to add, perhaps how to reshape some of the columns (e.g. targets), and what to do about the tags.
Integration test logs
I haven't run these yet but would be happy to if you think this PR is worth developing further
Example query results

Results
select * from aws_codestar_notification_rule;

-[ RECORD 1  ]---------------------------------------------------------------------------
arn                     | arn:aws:codestar-notifications:eu-central-1:***:notificationrule/1725915dbd009519cd82707d18d8506e5f5d2926
name                    | some-pipeline-notification
resource                | arn:aws:codepipeline:eu-central-1:***:some-pipeline
detail_type             | FULL
status                  | ENABLED
event_types             | [{"EventTypeId":"codepipeline-pipeline-pipeline-execution-succeeded","EventTypeName":"Pipeline execution: Succeeded","ResourceType":"Pipeline","ServiceName":"CodePipeline"},{"EventTypeId":"codepipeline-pipeline-stage-execution-canceled","EventTypeName":"Stage execution: Canceled","ResourceType":"Pipeline","ServiceName":"CodePipeline"},{"EventTypeId":"codepipeline-pipeline-action-execution-failed","EventTypeName":"Action execution: Failed","ResourceType":"Pipeline","ServiceName":"CodePipeline"},{"EventTypeId":"codepipeline-pipeline-manual-approval-needed","EventTypeName":"Manual approval: Needed","ResourceType":"Pipeline","ServiceName":"CodePipeline"}]
targets                 | [{"TargetAddress":"arn:aws:sns:eu-central-1:***:my-topic","TargetStatus":"ACTIVE","TargetType":"SNS"}]
created_by              | ***
created_timestamp       | 2024-05-21T22:13:57+02:00
last_modified_timestamp | 2024-05-21T22:13:57+02:00
tags_src                | {}
partition               | aws
region                  | eu-central-1
account_id              | ***
sp_connection_name      | aws
sp_ctx                  | {"connection_name":"aws","steampipe":{"sdk_version":"5.10.0"}}
_ctx                    | {"connection_name":"aws","steampipe":{"sdk_version":"5.10.0"}}

select rule.name as rule_name, rule.resource, target->>'TargetAddress' as target, topic.title as topic
from aws_codestar_notification_rule as rule
cross join jsonb_array_elements(rule.targets) target
left join aws_sns_topic as topic on target->>'TargetAddress' = topic.topic_arn;

-[ RECORD 1  ]---------------------------------------------------------------------------
rule_name | some-pipeline-notification
resource  | arn:aws:codepipeline:eu-central-1:***:some-pipeline
target    | arn:aws:sns:eu-central-1:***:my-topic
topic     | my-topic

Excellent, thanks for the comments! I have some travel coming up but will try to update the PR when I can.

I rebased the original commit to main and implemented the suggestions. I hope the choice of "Name" for title and "Arn" for akas makes sense.

I'm also wondering because it is quite natural to want to do joins on the "targets" - is it possible for the targets to be exposed as a secondary table somehow? The JSON join works but is quite inconvenient.

I'm also wondering because it is quite natural to want to do joins on the "targets" - is it possible for the targets to be exposed as a secondary table somehow? The JSON join works but is quite inconvenient.
I added another example query to the documentation that at least makes this more manageable.

According to our table development standards, we can create a separate table if the API supports pagination for retrieving the column value.
In our case, if there is a specific API call to get the targets only and it supports pagination.
Otherwise, we will leave them as they are.

The example query looks great to me. However, I have a few minor suggestions:

Please include the example query in both PostgreSQL and SQLite formats, as we have for other tables.
Please fix the indentation of the example queries. We can use the tool https://codebeautify.org/sqlformatter

Thank you!

* According to our table development standards, we can create a separate table if the API supports pagination for retrieving the column value.

Although there is an endpoint for listing targets it doesn't quite do what we'd want here so I'll leave it as is.

The example query looks great to me. However, I have a few minor suggestions:
* Please include the example query in both PostgreSQL and SQLite formats, as we have for other tables.

I couldn't find instructions for building the plugin for SQLite. How do I do that?

Hmm, It is difficult to build the plugin to work with SQLite in our local.
Can we use online tools like pg-to-sqlite and RebaseData to format our existing query into SQLite format?
On another note, the changes look good to me. We can merge them into a feature branch in our plugin and then make the remaining changes in that branch.
Thanks!

Awesome, thank you! :+1:


hand_physics_bone.gd errors when used in a physics hand scene

Hey, I'm running into an issue with XRTools when using the *_physics_hand.tscn and *_physics_* files. I run into the Invalid get index 'global_transform' (on base: 'Nil'). error when using these hands and these hands alone. It happens at line 113. This error doesn't appear in the Script Editor so I can see how this error got by. I'm using 4.1.rc2, but I believe this same issue occurred in 4.0.3.

This doesn't cause any issues in runtime, so I guess it's an editor specific bug?
Removing @tool fixed the spam of errors, so that's a plus.

Could indeed be some code that is running because this is a tool script, will have to investigate.

Seconding that I'm having similar impact.

Same issue here. For me, it's not just a spam of errors, the hands also do not work in runtime. I'm testing this on Oculus Quest 2.

Was having the same issue. I'm a newbie here, but it looks like it's related to there not being a "BoneAttachment3D" node as a child of the "Skeleton3D" node in the hand armature, since the script is an extension of that. Once I added one to each of the physics hands, and tried running the app, the errors stopped spamming...


updeps

@Valexr please, let's not PR just the only updates of deps, this can be done by robots!

This can be done by robots 🤖

SvelteKit in beta now & I'm testing updates locally before push them to repo.
How we can testing robots updates? 😁


Update to java:8u92-alpine upstream image

Includes Alpine 3.4 & security fixes in Java.

thanks!
are you sending a pull request to update the official link?

Yes I'll do that today.

Wes

On Jun 23, 2016, at 1:17 AM, Paul Lam<EMAIL_ADDRESS>wrote:
thanks!
are you sending a pull request to update the official link?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Add network to fedimint invite code

It is non-obvious how to check for the network when joining a federation. The current way requires creating a wallet client and asking what the network is after you've connected to the federation. Would be nice if you check up font without needing to connect.
I imagine this was never done because the network is really at the module level rather than the federation level but would be nice to be able to have some indication.

There is FederationInfo, which can be constructed using an invite code and contains the federation's config. You have to construct it anyway to build a client.
When it comes to the Bitcoin network: there has always been the tension how tightly to couple Fedimint to Bitcoin. In principle you could use it to build an independent transaction system, so we try not to assume anything about Bitcoin on the global level, it's delegated to modules. The two modules that are aware of their network/currency are the wallet and the LN module. You find it in their respective configs.
Since you probably don't want to talk to Federations without these modules anyway you can just require them to be present and read from their config.

Note that the FederationInfo functionality might be merged with ClientConfig soon: https://github.com/fedimint/fedimint/pull/4297

Actually, reviewing #4326 this morning I noticed that ClientBuilder doesn't require FederationInfo anymore, but takes ClientConfig directly again. We should really get #4297 merged then, I didn't know that API has such bad usability.

https://github.com/fedimint/fedimint/pull/4297 is merged


Allow for scrolling in day/week views

Currently, the entire calendar scrolls. I'm suggesting that the calendar header to be fixed and have the actual time table area scroll. This would mimic iCalendar as well as Google Calendar and allow for future addition of "all-day" events that should remain fixed to the top of the screen.
Thoughts?

@jamesdixon Definitely.
I think we just need to wrap .as-calendar-timetable__row into some container and give it styles:
container {
  height: 75vh;
  overflow: auto;
}

@jamesdixon "all-day" events are very nice addition.
Those events have to be contained in the row under day names row. They will always stay on top if days container is scrollable.
I think it should have separated issue.

Agreed. Should be relatively simple.


Update stuck on branches documentation

Update the documentation about being stuck on branches or working with a team that is stuck on branches to highlight how you can use the updated version of the branch command to make working in that way simpler.
https://book.git-ps.sh/tool/stuck-on-feature-branches.html

This has already been done. Therefore, I am closing this issue.


📝 Tutorial study functionality

What do these changes do?
Add a first tutorial for the studies entry.

NB: #59 should go in first. This one is based on that one.
WARNING: The e2e tests will fail once this one gets merged.

Related issue/s

https://github.com/ITISFoundation/osparc-simcore-clients/issues/50

How to test

@bisgaard-itis please do not forget to fill the PR issue attributes. e.g.
* assignee

* select project

* add labels

* select milestone

Also, the gh issue links render better if you use them in enumerations, ie.g. issue #1 renders this is how it renders a flat #1
* but this is how it renders in enum [Upgrade v0.2 #1](https://github.com/ITISFoundation/osparc-simcore-clients/pull/1)

Sorry, I thought I had already filled in all the PR attributes. I will be more careful next time.


⚠ BeyondHD has degraded performance

In 01f0ab6, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 1756 ms

Resolved: BeyondHD performance has improved in fdb2e58 after 8 minutes.


Network topology

This PR adds flexible functionality for dealing with typical topology problems described in #623.
The creation of a Network object now allows the user to optionally remove duplicate road segments and apply minor rounding to shapefile node locations.  This facilitates the process simplifying the road network to a simple graph of intersections and endpoints.   Work done with @jGaboardi.

This might need some cherrypicking? this is a ton of unrelated commits.

Should this be closed since #627 has the cherry picking?


Update addDuplicateSetter.ts

Summary by CodeRabbit

Tests

Added test cases for duplicate setter methods in TypeScript properties.

This makes the tests more comprehensive but I've yet to test it out

Have we checked the security aspect?

Yes, the security aspect has been validated properly


Email Input Validation is not available

User can put anything in the email and password entry data filed. We need to validate it and return to the user that the email is not valid and password should carry a character special symbols. I can do it. Assign this to me under gssoc-ext @GVishnudhasan
https://github.com/user-attachments/assets/ffd53599-4aad-47b8-9450-7709a8f0a70c

@lomash-choudhary Assigned, Good luck.

Can you assign this issue to me?

@rebornstar1 it is already assigned to me I am on it.

Ok

@GVishnudhasan if it is still open kindly assign me.

hey @lomash-choudhary  i'm ready to solve this issue so assign this to me.


Can't find this module

I can't find it when I search for it in Manage palette. When downloading it using npm, it isn't shown anywhere?

How are you trying to install it? with npm i ethernet-ip --save?

Yeah, it seems to be downloaded successfully, but I can't see the module anywhere in Node-RED

This is a node-red module, it is a nodejs module.  I am largely unfamiliar with node-red, but I think one could certainly build a node-red module using node-ethernet-ip.

We're in progress of creating a Node-RED node for this, but it's in very early stage of development. We have already the node-red-contrib-s7 and the node-red-contrib-pccc, and it'll work in a very similar way.
I'll post here when we have something public. Testing will be much appreciated.

Awesome! :grin:

Hi guys,
we've published the node-red-contrib-cip-ethernet-ip node, it's already available for installation through Node-RED's interface. Please let me know if you have any problems when using or if you have any suggestion, they'll be much appreciated :)

Is it working with Omron PLC's as well? It is written that it is mainly for
Allen Bradely PLC's?
Den tors 23 aug. 2018 kl 19:25 skrev Guilherme Francescon <
<EMAIL_ADDRESS>
Hi guys,
we've published the node-red-contrib-cip-ethernet-ip node, it's already
available for installation through Node-RED's interface. Please let me know
if you have any problems when using or if you have any suggestion, they'll
be much appreciated :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cmseaton42/node-ethernet-ip/issues/23#issuecomment-415501450,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVLMF4Q2_Gu1dd_YvX7PCRRPvE-WQJbXks5uTuWZgaJpZM4UtASV
.

--
Med Vänliga Hälsningar / Best Regards
Dhiaa Eddin Anabtawi
Högskoleingenjör/ elektronik och telekommunikations ingenjör
+46 (0) 72 783 87 22
Södergatan 56 Lgh 1101, 252 25 Helsingborg


Allow PLATFORM_SCHEMA_BASE modify platform name

Description:
This is a change prepared for #23012. After this merged, we can move google/tty.py to google_translate/tty.py without breaking change.
Changes

Allow platform name read from the validated config by component level platform schema.
Use validated config by component level platform schema as the input to execute platform level platform schema validation. (It actually revert an unintentional change made in #20743)
Add bunch of tests

This PR need revise after #23082 merged.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

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

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Thought about it more, this is not something that we should add.

Close it as I use other way to resolve the google tts migration issue.
I will open a new pull request to bring in some tests in this PR.


[Question/Bug] Raw strings in function

I have a suspicion that the module crashes when parsing a raw string r# in a function parameter.
It crashes for the flutter_rust_bridge in this code piece, only if an r# function exists
syn::parse_file(&source_rust_content).unwrap(),

Which I strongly assume is an issue here

Backissue in the bridge https://github.com/fzyzcjy/flutter_rust_bridge/issues/1103

Sorry for the horrible issue format, I just don’t know how exactly the module works so it’s more of a question then a bug report

I’m sorry you were right, the issue was/is that we use an old version of syn (1.0.77)


Remove the API call for returning active extensions

See the discussion on #939
The consensus is that we don't really need that API. @SahilTikale I am tagging you again, please let us know if you think there's a reason to keep it.

so @naved001 @Izhmash @xuhang57 we did have a discussion about it. Here is what I feel about this.
the heart of the problem is How do I know what arguments to send when I am trying to register a node or a switch
currently the options are hard-coded in the cli.py which makes for bad design.
we do not want the cli to be burdened with knowing argument requirements for each driver.
@naved001 suggested that once a call for node_register or switch_register fails the corresponding api error message should respond with what arguments are needed for registering a particular type of node or switch. I liked the idea but the problem with this approach is that the client library still needs to built a correct dictionary for which it needs to know the keys apriori. How do we manage that programmatically ?
I suggested the following:
When the call to register a switch or node fails, the api will respond with what dictionary of arguments it requires to register the hardware. We send a complete json object (dictionary of arguments) from cli which is treated as payload by client library. That way client side does not have to know about any drivers available on the server side. Thus we can remove api call that queries the active extensions in hil.cfg
Alternatively, lets not have any support for node_register and switch_register in client library.  Since their usage requires admin privileges and assumptions that the user knows the infrastructure being registered with HIL, we can also assume that these calls will be done only on the server itself. May be direct rest calls. Just an idea.
What do you guys think @zenhack

My thoughts:
We have to issues here.

Do we want to keep the list_active_extensions
How should we handle the hardcoding problem in switch_register (as Sahil talked above)

My humble opinions:

For issue 1, we should determine who should use this function and if no one is going to use it, we should remove it. The idea is not to introduce an unused chunk of code to the code base. My personal opinion on this is to remove the call since we don't have any usage of it for now or in the near future.

For issue 2, I am thinking maybe we should introduce some sort of new models about Switches. The pro is we can specify how many arguments it needs to construct the object so that we can make sure the user pass the proper numbers of arguments. I don't have implementation details in my mind yet. But this should be like:

new brocade = Brocade(arg1, arg2, arg3, arg4)

I agree with @xuhang57 on 1.
On 2, what I had in mind was. just send a sorted list {"key1": <hostname>, "key2": <other attribute>}. The API server will check for the  number of arguments, and if it matches the required number then it will map it to respective fields (in order). The downside is, the order here will matter and I don't know if it's a bad design idea or not. And will also require a change in API.
Will dig more into it on how can this be handled.

I agree on (1) as well; the complaint was the api call doesn't have a clear use (that actually works anyway; as discussed it doesn't solve the problem it was designed for).
Somewhere (not sure which PR issue it was on, and can't find it now) I outlined a way to write the switch_register calls where we basically hard-code knowledge of certain drivers in the client library, but allow a user to just pass the raw values to be sent to the server, so the library is still usable if you're working with a server that has drivers the library doesn't know about. I still think this is a decent solution.
We could also look into using a more principled way of validating the format of our messages, e.g. json schema. We could build an api call to fetch the schema for the drivers it understands, and the client library could use those to validate the arguments. This solves the problem in a way that its somewhat similar to what the existing API call was trying to do, but it does so without needing to hard-code what each extension name "means," and it avoids exposing the implementation detail. The latter also means it wouldn't be hard to use this approach in OBMd as well, even though it doesn't re-use HIL's extension mechanism at all.

@zenhack https://github.com/CCI-MOC/hil/pull/950 you wrote something here in this PR.

Yep, that's what I was thinking of, thanks for digging it up.

the heart of the problem is How do I know what arguments to send when I am trying to register a node or a switch

Node registration will now happen in OBMd; and the cli can now pass either subtype dependent info or raw info the client library. So I guess it's safe to remove the API in question.


How to skip verification on layout update?

Verification uses to much time how to disable it? I am pretty sure my install dir is not broken, I was not touching it, Why I should wait half an hour before installer will load new components?

@yatagarasu25 -- Igor, thanks for your feedback; we appreciate it. To make sure we have all the information needed to triage your issue, please let us know which topic you’re referencing, or provide more context around the general task.

Interesting system you have created - people can open and create issues from your documentation pages, but you can not track which one it belong to.
https://docs.microsoft.com/en-us/visualstudio/install/update-a-network-installation-of-visual-studio?view=vs-2019
I am talking about updating network installer.
If i will launch layout update with command like this "vs_Community.exe --layout "E:\Visual Studio\Visual Studio 2017\Layout" --lang en-US" it will be verifying all my already downloaded packages. for no useful reason - because all downloaded packages are ok. To download 200mb of updates it will check all my 30gigs. It takes much more time then downloading, especially on convenient hdds.

@yatagarasu25  It looks like you’re having an issue with installation. If your Visual Studio installation fails, see Troubleshooting Visual Studio installation and upgrade issues for step-by-step guidance.
We also offer a Live Chat (English only) support option for installation-related issues.
Here are a few more support options:

-Report product issues to us via the Report a Problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.

Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.

Use your GitHub account to talk to us and other Visual Studio developers in the Visual Studio conversation in the Gitter community.

I don't have issues with installation, I ask you to add flag to visual studio installer.

@yatagarasu25 -- Igor, thank you for clarifying. It looks like you have product feedback rather than documentation feedback. If you are looking for functionality that isn’t currently provided by the product, please use our “Suggest a feature” option in the Visual Studio Developer Community. We’d be happy to have this on record, where others can view it and upvote it, too. For more information about this works in the dev community, see our Suggest a feature page.


Databrary

New York, NY
Hiring for now - https://www.databrary.org/about/jobs/back-end.html , not sure when that post will expire.

Added, thanks!


⚠ Bofrost.es has degraded performance

In ae99b39, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1370 ms

Resolved: Bofrost.es performance has improved in 0287da1 after 6 minutes.


Email notifications

Feature request

[ ] I confirm that this is a feature request rather than a question.

What problem does this feature solve?
Whilst the notification profiles already supplied do allow for users to get notified of issues/maintenance, I find that alot of users wish to be notified by email.
What does the proposed API look like?
Not sure
How should this be implemented in your opinion?
Allow an admin to define a email server to use when sending notifications, an email address to send the email notifications from. Users would then need to input their email address when subscribing. It would also be nice to allow a user to only subscribe to particular services in which they want to be notified on issues/maintenance
Are you willing to work on this yourself?**
I unfortunately don't have the skills required

I would like this feature too

Hi @pcrosthwaite. We're thinking how can we achieve this feature on Static Generated Mode. On SSR will be quietly easy with an API endpoint.

@juliomrqz that's great news!  Can't wait to use it.  Thanks!

I'd like to have this feature too.

This actually can be done using third parties such as Mailgun or Mailchimp. On Static Generated Mode we usually use CI/CD, so we can send email to the mailing list (ref: Mailgun) seamlessly. We can store the token on Secret Variables on the CI Settings.
What do you think?

@chez14 It is not as easy as it sounds, the Automatic Notification On Static Generated Mode. How could you differentiate between a Statusfy update and an incident update to know if a notification should be sent?, that is, you need to store somewhere the latest status of the application to determine if a notification should be sent automatically, and the answer to that is to use a database which is not currently supported.

Hello. There are big changes coming (mostly internal) for the next major version of Statusfy (v1). For more details go to https://github.com/aceforth/statusfy/issues/551.
We are evaluating how to implement this functionality with the new changes. We appreciate your comments or suggestions.

I would also like to request this functionality


Pretrained checkpoint

Hi,
Thanks for sharing the code. I am wondering if it is possible for you to also release the pretrained checkpoint for ShapeNet planes and chairs?
Thanks

Sure, however, it's been a while and it might take me a few days to dig around.

Much appreciation!

Also looking for the pretrianed checkpoints!

Hi,
Is there a way to get the pretrained models?
Thanks,
Mika


Testing Attack Spawn: Bug summary

In 3 attacks from the AAF at 3 different locations the resutls are:

One attack lasted about 1 min and then was completed (successfully).
The other one transport heli spawned with troops and moved to location. Transport heli unloaded the troops about 500m from the location but they did not move to the task (stood still in formation where they were drooped).
In the last one the transport heli spawned together with the troops inside but did not took off the airfield (checked with debug).

P.S. Tested with ACE in the feature/spawn branch in MP
Arma3_x64_2017-10-30_10-24-48.txt

I believe that this has been fixed in 3e4a4588614c5b9099ae494cd9b16477486f6470.


Fix a bug with enum type mapping from explicit schema

Fix the mapping bug related to enum database types which are located in the schema not in the search path (see this change)
Add additional logging and integration tests for the use case when the schema is not in the search path.
Cleanup the Java 8 code
Fix typo in the class name.

Fixes #69

:+1:

👍


Composite key cursor pagination

Fixes https://github.com/SeaQL/sea-orm/issues/1209

@jacob-pro this might be all it takes?

Would appreciate some input

Any thoughts? @jacob-pro @Sytten

I don't fully understand how the code works (I'm new to this library) - but the generated SQL looks correct!.
Thank you for fixing this!


ARROW-5050: [C++] Specify dependencies of grpc_ep

I encounter the following build error when I built the tarball of 0.13.0 RC.
[38/597] Performing configure step for 'grpc_ep'
FAILED: grpc_ep-prefix/src/grpc_ep-stamp/grpc_ep-configure
cd /Users/mrkn/Downloads/apache-arrow-0.13.0/cpp/build/grpc_ep-prefix/src/grpc_ep-build && /opt/brew/Cellar/cmake/3.13.3/bin/cmake -P /Users/mrkn/Downloads/apache-arrow-0.13.0/cpp/build/grpc_ep-prefix/src/grpc_ep-stamp/grpc_ep-configure-RELEASE.cmake && /opt/brew/Cellar/c
make/3.13.3/bin/cmake -E touch /Users/mrkn/Downloads/apache-arrow-0.13.0/cpp/build/grpc_ep-prefix/src/grpc_ep-stamp/grpc_ep-configure
CMake Error at /Users/mrkn/Downloads/apache-arrow-0.13.0/cpp/build/grpc_ep-prefix/src/grpc_ep-stamp/grpc_ep-configure-RELEASE.cmake:16 (message):
  Command failed: 1

   '/opt/brew/Cellar/cmake/3.13.3/bin/cmake' '-DCMAKE_BUILD_TYPE=RELEASE' '-DCMAKE_PREFIX_PATH=';/opt/brew;/Users/mrkn/src/github.com/apache/arrow/cpp/build/gflags_ep-prefix/src/gflags_ep;/Users/mrkn/Downloads/apache-arrow-0.13.0/cpp/thirdparty/cares_ep-install;'' '-DgRPC
_CARES_PROVIDER=package' '-DgRPC_GFLAGS_PROVIDER=package' '-DgRPC_PROTOBUF_PROVIDER=package' '-DgRPC_SSL_PROVIDER=package' '-DgRPC_ZLIB_PROVIDER=package' '-DCMAKE_CXX_FLAGS= -Qunused-arguments -fcolor-diagnostics -O3 -DNDEBUG -O3 -DNDEBUG -fPIC' '-DCMAKE_C_FLAGS= -Qunused-arguments -O3 -DNDEBUG -O3 -DNDEBUG -fPIC' '-DCMAKE_INSTALL_PREFIX=/Users/mrkn/Downloads/apache-arrow-0.13.0/cpp/thirdparty/grpc_ep-install' '-DCMAKE_INSTALL_LIBDIR=lib' '-DProtobuf_PROTOC_LIBRARY=/opt/brew/lib/libprotoc.dylib' '-DBUILD_SHARED_LIBS=OFF' '-GNinja' '/Users/mrkn/Downloads/apache-arrow-0.13.0/cpp/build/grpc_ep-prefix/src/grpc_ep'                                                                                See also                                                                                /Users/mrkn/Downloads/apache-arrow-0.13.0/cpp/build/grpc_ep-prefix/src/grpc_ep-stamp/grpc_ep-configure-*.log

grpc_ep's build log is given below:
$ cat /Users/mrkn/Downloads/apache-arrow-0.13.0/cpp/build/grpc_ep-prefix/src/grpc_ep-stamp/grpc_ep-configure-*.log
CMake Error at cmake/cares.cmake:38 (find_package):
  Could not find a package configuration file provided by "c-ares" with any
  of the following names:

    c-aresConfig.cmake
    c-ares-config.cmake

  Add the installation prefix of "c-ares" to CMAKE_PREFIX_PATH or set
  "c-ares_DIR" to a directory containing one of the above files.  If "c-ares"
  provides a separate development package or SDK, be sure it has been
  installed.
Call Stack (most recent call first):
  CMakeLists.txt:139 (include)

(snip)

This build log says that grpc_ep couldn't find cares_ep.
In this case, grpc_ep was built before cares_ep.
This pull-request could fix this dependency error on my environment.

IMHO this isn't necessary for 0.13.0.


Update build.sbt to not duplicate compile/test aritfacts and get rid …

…of unnecessary delete statements in CI

Funny, it actually failed for the first time on this PR. I guess something is still up.

You will want to have a shorter commit message or take what you have and break into multiple lines. You will also need to add all issues to commit message by adding Closes #ISSUE_NUMBER on a separate line for each issues. Here is a sample commit from what you now:
Update build.sbt and CI.yml:

- Stop duplicating compile/test artifacts.
- Get rid of unnecessary delete statements in CI.

Closes #521

@arosien, @NolanMatt is having a difficult time investigating the problem since it works for him locally and intermittently in CI.  He indicated that you are able to reproduce issue locally, so I'll reassign it to you.

I really don't understand what's going on here. Since the original message was referring to schema files that existed in a path that were only meant to be a temporary location, and later moved, I removed the need to move any extracted schema files at all. However, this still failed the build--only under Node 16--with a timeout during the headless extension tests, which doesn't seem to have anything to do with TDML.
Running the same tests locally, including using Node 16, passes for me.
Rerunning the failed job makes it pass, but that doesn't reassure me. Trying another complete rerun to see if I can trigger any failures.

@NolanMatt I can't force push to your fork, so you'll have to squash everything to get to a single commit. (or give me perms somehow)


YouTube version of this part

Available with common creative license :
https://www.youtube.com/watch?v=P7iTtQnkrNs&feature=youtu.be

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

ID: 7a54c97f-ddd4-b041-5e5f-a2df009ace50
Version Independent ID: fa488196-1f98-08ea-adcc-17d9dac86727
Content: Razor Pages with Entity Framework Core in ASP.NET Core - Tutorial 1 of 8
Content Source: aspnetcore/data/ef-rp/intro.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@krchome can you look at this list of youtube issues

Make sure each issue you create has the following text "If you find this video helpful, leave a comment or 👍". For example, see #9608
Find the issues I created pointing to your video's. You need to create an issue for those so I can close my issue. You need to be the author of issues pointing to your videos.
I think customers love your videos. I'd like to get some feedback/metrics on that. For example, see #9766


[stable/homeassistant] Homeassistant fails to start with AppDaemon enabled

Describe the bug
Homeassistant fails to start with AppDaemon enabled.
A clear and concise description of what the bug is.
Version of Helm and Kubernetes:
Helm: v3.2.1
Kubernetes: v1.18.2
Which chart:
stable/homeassistant
What happened:
➜  ~ kubectl -n homeassistant logs homeassistant-home-assistant-7d5d949c68-zqjfd -c appdaemon
2020-06-10 18:45:03.650536 INFO AppDaemon Version 3.0.5 starting
2020-06-10 18:45:03.650939 INFO Configuration read from: /conf/appdaemon.yaml
2020-06-10 18:45:03.653758 INFO AppDaemon: Starting Apps
2020-06-10 18:45:03.657716 INFO AppDaemon: Loading Plugin HASS using class HassPlugin from module hassplugin
2020-06-10 18:45:03.724632 INFO AppDaemon: HASS: HASS Plugin Initializing
2020-06-10 18:45:03.725158 INFO AppDaemon: HASS: HASS Plugin initialization complete
2020-06-10 18:45:03.725450 INFO Starting Dashboards
2020-06-10 18:45:03.725692 WARNING ------------------------------------------------------------
2020-06-10 18:45:03.725788 WARNING Unexpected error during run()
2020-06-10 18:45:03.725884 WARNING ------------------------------------------------------------
2020-06-10 18:45:03.726918 WARNING Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/appdaemon/admain.py", line 82, in run
    self.rundash = rundash.RunDash(self.AD, loop, self.logger, self.access, **hadashboard)
  File "/usr/local/lib/python3.6/site-packages/appdaemon/rundash.py", line 130, in __init__
    dash_net = url.netloc.split(":")
TypeError: a bytes-like object is required, not 'str'

2020-06-10 18:45:03.727020 WARNING ------------------------------------------------------------
2020-06-10 18:45:03.727147 INFO AppDeamon Exited

What you expected to happen:
AppDaemon starts.
Homeassistant starts.
How to reproduce it (as minimally and precisely as possible):
appdaemon:
  enabled: true

  ## code-server container image
  ##
  image:
    repository: acockburn/appdaemon
    tag: 3.0.5
    pullPolicy: IfNotPresent

  ## Home Assistant API token
  # haToken:

  ## Additional hass-vscode container environment variable
  ## For instance to add a http_proxy
  ##
  extraEnv: {}

  ingress:
    enabled: false
    annotations: {}
      # kubernetes.io/ingress.class: nginx
      # kubernetes.io/tls-acme: "true"
    path: /
    hosts:
      - appdaemon.local
    tls: []
    #  - secretName: appdaemon-tls
    #    hosts:
    #      - appdaemon.local

  service:
    type: ClusterIP
    port: 5050
    annotations: {}
    labels: {}
    clusterIP: ""
    ## List of IP addresses at which the hass-appdaemon service is available
    ## Ref: https://kubernetes.io/docs/user-guide/services/#external-ips
    ##
    externalIPs: []
    loadBalancerIP: ""
    loadBalancerSourceRanges: []
    # nodePort: 30000

I have the exact same issue, thanks for reporting

@kifeo After further investigation is seems that stable/home-assistant is deprecated (see the README.md).
I tried running with https://github.com/billimek/billimek-charts/tree/master/charts/home-assistant, which is what stable/home-assistant's README.md directs users to. Unfortunately, I hit the same bug.
I've filed a defect against billimek, here. You might want to try with this chart and comment on my bug report if you still have the same problem, as I do.
https://github.com/billimek/billimek-charts/issues/248

thanks @rwlove


SSL through a custom port

Hi there, I've read different issues here and I think I have a different problem here.
I don't have any self signed certificate, I'm trying to make a connection to a third party that has his SSL connection in other port than 443.
When I try making a connection to his endpoint using https://hisdomain.com:hisport/his-endpoint I'm getting the "unable to verify the first certificate"
I read the posts that says about using a custom agent and tried creating a custom agent and passing it to the agent parameter in options but it's giving me the same error.
Can you help me how to solve this?
The third-party is playing hardball with me not wanting to go to the pattern of the standard SSL port (443)...
Your Environment

software
version

node-fetch
2.6.1

node
12.13.0

npm
6.14.4

Operating System
amazon AMI (linux)

Don't post comments like: +1, Same hereThey don't help, they spam and will only be removedPress the subscribe button instead if that is what you wantvote with reactions

Up

same issue here

I'm also having this issue

I'm having the same issue here

same here

Stop we get it... it's a problem
you don't have to spam it and say "yea same here"
it don't help the issue in any way

Do you have any example code?
what have you tried so far?
Maybe this could help?
import { Agent } from 'node:https'
import fetch from 'node-fetch'

const agent = new Agent({ rejectUnauthorized: false })

fetch(url, { agent })

That did it @jimmywarting ! Thank you so much for this. It's not clear in node.js docs that this is an option available to the constructors of Agent instances!


fix compilation error

fix issue #213
The problem seems to occur when sdl-gpu is installed, then CMakeList
was calling include_directories cmake function with SDL path before adding
project include/, So /usr/include/SDL had an higher priority than
sdl-gpu includes, and so it was using old versions,
which didn't have default_textured_vertex_shader_id.
Signed-off-by: matthias<EMAIL_ADDRESS>

Thanks for the fix!


Error: connect ECONNREFUSED <IP_ADDRESS>:8080

I setup the docker instance as documented, it seems to be running, though I cannot connect or run my own tests.

you need to expose the ports via the -p option
ie.
docker run -p 8080:8080 xxxxx
this tells docker to bind the 8080 port of the container to the 8080 port of host's localhost

Yea I did run with that too, same as the instructions, no?
On Sat, Oct 5, 2019 at 1:24 AM Jonathan Underwood<EMAIL_ADDRESS>wrote:

you need to expose the ports via the -p option
ie.
docker run -p 8080:8080 xxxxx
this tells docker to bind the 8080 port of the container to the 8080 port
of host's localhost
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bitcoinjs/regtest-server/issues/9?email_source=notifications&email_token=ADELLF6IGNW5OLE7KL3ELOLQM67BZA5CNFSM4I5TANM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEANBCGY#issuecomment-538579227,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADELLF5LN43ALS7WY33HOYLQM67BZANCNFSM4I5TANMQ
.

run
sudo netstat -tulpn
and see if anything is listening to localhost 8080 and what process it is.

With the full command I get: netstat: n: unknown or uninstrumented protocol
With partial commands I do not see any localhost.
If I try to run the docker run command again I get: docker: Error response from daemon: driver failed programming external connectivity on endpoint goofy_jang (<NUM>): Bind for <IP_ADDRESS>:8080 failed: port is already allocated.

Are you using the image from docker hub or did you build it yourself?

I built it myself after cloning this repo.

try using the image from docker hub.
let me know what happens


appmetrics have conflict with appmetrics-zipkin when enable trace

appmetrics have conflict with appmetrics-zipkin when enable trace
here is the error message:
/root/joyce/pets/node_modules/appmetrics/probes/trace-probe.js:57
  ret.__ddProbeAttached__ = function() {
                          ^

TypeError: Cannot create property '__ddProbeAttached__' on boolean 'false'
    at TraceProbe.attach (/root/joyce/pets/node_modules/appmetrics/probes/trace-probe.js:57:27)
    at /root/joyce/pets/node_modules/appmetrics/index.js:158:27
    at Module.newFunc (/root/joyce/pets/node_modules/appmetrics/lib/aspect.js:102:14)
    at Module.newFunc [as require] (/root/joyce/pets/ibmapm/appmetrics-zipkin/lib/aspect.js:101:26)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/root/joyce/pets/node_modules/cross-spawn/lib/parse.js:4:27)
    at Module._compile (module.js:653:30)
    at Object.Module._extensions..js (module.js:664:10)
    at Module.load (module.js:566:32)
    at tryModuleLoad (module.js:506:12)

code may like the following:
var appmetrics = require('appmetrics');
var zipkin = require('appmetrics-zipkin');
//composing zipkinOptions
...
//composed zipkinOptions
zipkin(zipkinOptions);
appmetrics.enable('trace');

Thanks, we will recreate this then let you know the resolution

I don't think these 2 play well together.   You probably need to disable the trace probe in app metrics if using zipkin as that has similar functions

Didn't find same problem in appmetrics 5.0.3


[BUG] Fails to check from latest tag

The issue has already been filled on cocogitto-action repository side. However, it's related to the binary, not the action directly as opposed to what was mentionned by the author of the initial issue.
And well, when reading the code it seems the problem is coming from there: https://github.com/cocogitto/cocogitto/blob/main/src/command/check.rs#L22-L23
        } else if check_from_latest_tag {
            self.repository
                .get_commit_range(&RevspecPattern::default())?
        }

However I'm not familiar with Rust to determin what's happening exactly when calling the default() method.

This is the expected behavior. Cocogitto only interpret valid SemVer tags.
Tags in your repo are valid ones but prefixed with v (see: https://docs.cocogitto.io/guide/#tag-prefix)
In order to make cocogitto work in your repo you need to setup a cog.toml config like so:
cd my_repo
cog init

Add the tag prefix configuration:
sed -i '1s/^/tag_prefix = "v"\n/' cog.toml

Then confirm that cog check is working as expected
cog check --from-latest-tag

The action use the same commands so if that works locally it will work on your CI.
The documentation might be confusing about which tags are picked by cog, I will update the docs to avoid confusion.
PS: You might want to use the bot instead which requires not setup (see: https://github.com/apps/cocogitto-bot)


ImportError: DLL load failed while importing _SimpleITK: The specified module could not be found.

i had a problem with loading the image, so I uninstalled everything and followed the tutorial, but I get two different errors.
First, when I type
conda update anaconda
I get
PackageNotInstalledError: Package is not installed in prefix.
prefix: C:\Users\paral\anaconda3
package name: anaconda
Which can be bypassed with
conda install anaconda.
After that everything works fine until I open the notebook. When I try to import SimpleITK I get

ImportError                               Traceback (most recent call last)
Cell In[1], line 1
----> 1 import SimpleITK as sitk
2 from downloaddata import fetch_data, fetch_data_all
4 from ipywidgets import interact
File ~\anaconda3\envs\sitkpy\lib\site-packages\SimpleITK_init_.py:18
1 # ========================================================================
2 #
3 #  Copyright NumFOCUS
(...)
16 #
17 # ========================================================================
---> 18 from SimpleITK.SimpleITK import *
19 from SimpleITK.extra import *
21 from SimpleITK._version import version
File ~\anaconda3\envs\sitkpy\lib\site-packages\SimpleITK\SimpleITK.py:13
11 # Import the low-level C/C++ module
12 if package or "." in name:
---> 13     from . import _SimpleITK
14 else:
15     import _SimpleITK
ImportError: DLL load failed while importing _SimpleITK: The specified module could not be found.
I have re-installed anaconda python but nothing seems to work.
Edit: The first time import simpleITK worked fine. I have no clue what I did differently.

Hello @Ilias-Paralikas,
Possibly you activated the sitkpy environment before updating conda? If you follow the installation instructions, you will first install anaconda and update the base environment, conda update conda and conda update anaconda.
Please uninstall your anaconda installation and redo the setup process. If this does not resolve the issue, please provide the set of steps you took till encountering an error.

Hello,
so the steps after full uninstall of anaconda were:
download&run conda installer.
add C:\Users\paral\anaconda3 to environment variables.
Open anaconda prompt
conda update conda.  works fine.
conda update anaconda.
I get this error
PackageNotInstalledError: Package is not installed in prefix.
prefix: C:\Users\paral\anaconda3
package name: anaconda
that I solve by installing anaconda( this step is not in the tutorial)
conda install anaconda
conda update anaconda  works fine
conda install git
git clone https://github.com/SimpleITK/TUTORIAL.git
conda env create -f TUTORIAL/environment.yml
conda activate sitkpy
cd TUTORIAL
jupyter notebook 00_setup.ipynb
run the first cell and I get the error from above
I follow the tutorial except from the conda install anaconda part.

Hello @Ilias-Paralikas,
It appears you uncovered a fault in the anaconda distribution of SimpleITK 2.2.1 for windows. We will look into this shortly.
For a quick workaround, please do the following:

Activate the environment: conda activate sitkpy.
Uninstall the SimpleITK anaconda package: conda remove SimpleITK
Install the SimpleITK vanilla Python package: pip install SimpleITK
Run the tutorial.

Please let us know if this resolved your issue or not.

yeap, this worked fine.
Thank you very much!


Why making_data.ipynb mix up the train, validation and test splits of FiQA-SA?

While for the other 3 datasets, only the train split is used.
btw, I notice that in https://github.com/AI4Finance-Foundation/FinNLP/tree/main/finnlp/benchmarks, test_fiqa.py uses all train, validation and test splits while the others use only the test split. I wonder the reason and sincerely request your help. Thanks!

Hi, KYLN24. For the FPB and FiQA datasets, we have tried our best to do the same split as the BloombergGPT so that the comparison with it can be much more meaningful. In BloombergGPT's paper:  Like with FPB, we create our own random split combining both microblogs and news. After discretization, our training set contains 938 sentences with 576 positive, 287 negative, and 75 neutral sentences and our test set contains 235 sentences with 141 positive, 76 negative, and 18 neutral sentences. We select 5 shots and report weighted F1. However, we don't know the exact split of BloombergGPT, so we do it in this way:
# Train:
dataset = datasets.concatenate_datasets([dataset["train"], dataset["validation"] ,dataset["test"] ])
dataset = dataset.train_test_split(0.226, seed = 42)['train']

# Benchmark(Test):
dataset = datasets.concatenate_datasets([dataset["train"], dataset["validation"] ,dataset["test"] ])
dataset = dataset.train_test_split(0.226, seed = 42)['test']

thanks for your answering！


not catching MediaStream on ios

I am only getting trace message "Created remote peer connection object remotePeerConnection." without a mediastream after using this.
cordova.plugins.sharescreen.startScreenShare(function(mediaStream){console.log(mediaStream)}, function(error){});
Are there any NSExtension attributes that should be altered in order to make it work on cordova app ??

Did you end up figuring this out?


Load not called for child components

Describe the bug
Child components' load not called.
Logs
No errors, just nothing happens.
To Reproduce
index.svelte
<script lang='ts'>
  import Child from './_Child.svelte';
</script>

<Child />

_Child.svelte
<script lang='ts' context='module'>
  export async function load({}) {
    const val = 33;

    return {
      props: {
        val
      }
    };
  };
</script>

<script lang='ts'>
  export let val;
</script>

{val}

The output is undefined instead of the expected 33.
Let me know if making a sscce repo would be useful.
Expected behavior
Child components' load should be called.
Stacktraces
None.
Information about your SvelteKit Installation:

The output of npx envinfo --system --npmPackages svelte,@sveltejs/kit --binaries --browsers

  System:
    OS: Linux 5.4 Ubuntu 20.04.2 LTS (Focal Fossa)
    CPU: (8) x64 Intel(R) Xeon(R) CPU E3-1230 V2 @ 3.30GHz
    Memory: 2.76 GB / 7.73 GB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Node: 14.16.0 - /usr/bin/node
    Yarn: 1.22.10 - ~/.local/bin/yarn
    npm: 6.14.11 - /usr/bin/npm
  Browsers:
    Firefox: 86.0.1
  npmPackages:
    @sveltejs/kit: next => 1.0.0-next.61
    svelte: ^3.35.0 => 3.35.0

Your browser

Firefox.

Your adapter (e.g. Node, static, Vercel, Begin, etc...)

Node.
Severity
Low.
Additional context
The same code works if put inside index.svelte instead of _Child.svelte. load in _Child is not called at all, though the <script context='module'> is entered.

This is expected behavior.

As described in https://kit.svelte.dev/docs#loading in the red box, load only runs on page components, just like preload did in Sapper.

https://github.com/sveltejs/kit/blob/7166c5760615a94dd2b61613f8224f871f74e96c/documentation/docs/03-loading.md#L23

load  only applies to components that define pages, not the components that they import.

@babichjacob @Conduitry Thanks!


Forgot Password feature

The Issue
Currently if a user forgets their password they need to inform the Superset admin from outside the superset environment via email, slack. After which the admin either:

Runs a CLI command
superset fab reset-password --username<EMAIL_ADDRESS>--password the_new_password
OR
Deactivates the user from Superset UI with the need to change all mandatory fields to prevent referential integrity conflicts on the DB and then recreates the user.

Desired Solution
Adding a forgot password option/button on the login page that any user can access themselves to change their passwords would make it so much efficient for Admins to manage.
Note: The only change/update password feature that exist is after the user has successfully logged in.

yes this would be a great feature!

@jessicaxyl @stevensuting
We would love to provide Design resource if anyone wants to contribute the implementation. please do reachout to me on Slack :)


Disable apphost ilverify

Looks like it worked based on the project.assets.json:
        "downloadDependencies": [
          {
            "name": "Microsoft.AspNetCore.App.Ref",
            "version": "[3.1.2, 3.1.2]"
          },
          {
            "name": "Microsoft.NETCore.App.Ref",
            "version": "[3.1.0, 3.1.0]"
          }
        ],


withPropsOnChange: when shouldMap returns false, subsequent calls have outdated "prevProps"

When using withPropsOnChange with "shouldMapOrKeys" function first argument (a.k.a.shouldMap, signature (prevProps, nextProps): Boolean), prevProps is only updated when the previous call to shouldMap returned true.
I believe this is incorrect — shouldMap signals whether computedProps should be updated, and prevProps should always be updated, ready for the next call to shouldMap/
Repro steps

On first change, shouldMap is called with prevState.prevProps
Because this is within getDerivedStateFromProps, if shouldMap returns false, it returns null (does not update state)
Therefore the next call to shouldMap, it is passed not the "current" props but the original, on-mount props.

Or, after some changes, "prevProps" will actually be "props from the last time shouldMap returned true"

I have pushed PR #667 with this change :)

see PR comment

+1 on this. This change broke old behavior and I heavily rely on withPropsOnChange. But now prevProps are not reliable in certain cases. And this is causing a lot of flack from the anti recompose guys here also :(

It took me a day to realize the reason my oldProps are not up to date because withPropsOnChange changed it's behavior a bit and release notes didn't mention that. Release notes should state that behavior was changed.
But I think something might have been missed here. The problem not only happens if you "calculate something based on some propsA, and don't use that propsA in decision to recalculate".
I have a case with something like this:
withPropsOnChange(
  ({ auth }, { auth: nextAuth }) => !auth.hasError && nextAuth.hasError,
  ({ auth, onExpectationFailed }) => {
    if (_.get(auth, 'error') === httpStatus.EXPECTATION_FAILED) {
      onExpectationFailed();
    }
  }
),

Because my shouldMap returns false when transitioning from hasError=true to false, the next time I have an error my old props indicate hasError=true even though they are false. So if I'm looking inside a prop's fields for my comparison, this change breaks my code.
If you would keep the oldProps in derived state, it will be consistent with previous implementation.

I don't want to be consistent with previous, IMO previous implementation was wrong.
I wrote why I think so see links (PR, and link iside PR)
https://github.com/acdlite/recompose/pull/667
https://github.com/acdlite/recompose/issues/651#issuecomment-383163549

I understand that, but I thought my example was different than the other examples where this breaks, as I'm relying on the same prop for calculation and for my predicate. If you think this should be different I think I need to go to a lot of old code and make sure it's not going to break now, and this should be documented as a breaking change. This could break a lot of production code based on this HOC.

withPropsOnChange was never assumed to be a pattern like in your example, calling event on props change is not a good code practice. Idea was to use it as some kind of memoize function. Here in issues I wrote multiple time why this practice is bad and how to avoid. Now it become just nearer to memoize and its good that it eliminates some practises like above :-)

Ok I'll search to see what you said about alternatives :)
Should still be documented as a breaking change though, regardless of what it was meant to be used for.

Sorry for that,
summer in Russia is too short to spend my time documenting things ;-)

@istarkov Sorry to bother but I searched to see alternatives and couldn't find something that addresses what I'm doing as a bad pattern or suggests an alternative. A link would be great :)
I'm trying to understand, did you mean the way I use shouldMap is wrong (looking inside a field of a prop) or the way I use propsMapper (not actually mapping and setting a state)?
Because this breaking change is only breaking the way I use shouldMap AFAIK.

The problem that you instead of returning state, calls some side-effect function which is possibly change the parent state etc.
One of answers on similar
https://github.com/acdlite/recompose/issues/545#issuecomment-340469880
Usually it breaks idea that View is a pure function of props, now view affects world around in some not clear cases. As a usual result it causes inconsistent data on some renders, few sources of truth,  parent components need to have knowledge about children internals etc, infinite loops and other fun issues.
It's somehow similar to what's written here
https://reactjs.org/blog/2018/06/07/you-probably-dont-need-derived-state.html#common-bugs-when-using-derived-state
React guys did a big job to prevent such kind of activity, deprecating old lifecycle componentWillReceiveProps and providing new static getDerivedStateFromProps
To solve, usually is enough to move state outside - see controlled components. As an example
you don't need calculate this on component level if (_.get(auth, 'error') === httpStatus.EXPECTATION_FAILED) { if you use any state management like redux you could do this calculation in middleware and set error:'error message' somewhere in global state or even calling setState({error}) in fetch callback. Now all your component need is to just render Error without any checks or lifecycles

The other problem in new async react that your method could be called multiple times, I dont think it will be what you expect ;-)

Ok I went through a lot of reading and consulted some office mates, I think I got the idea and we change what we did there. But I think there is still an issue here in a different way though.
You were referring to causing side effects from propsMapper as the issue with my code, which is also true, but you didn't respond to why shouldMap looking at inner fields should have been broken.
The way it's currently built (not saving oldProps if shouldMap returns false) breaks the option for shouldMap to be a custom function. Only the string array option is viable now, otherwise you could have unexpected results (as I have gotten).

Internally its just call to pick and shallowEqual and this has no difference if you call any other function. So please provide an example where you think that string array option is viable and any other function is not. And I respond few times why it have current behaviour - memoize is keyword.

Yes but memoize is for the propsMapper rather than the shouldMap isn't it?
The example I had up there would not work because it uses the function. I get that the default implementation for the string array is just a pick, but if you allow using a function you give the developer a lot of other options. Like this one:
({ auth }, { auth: nextAuth }) => !auth.hasError && nextAuth.hasError,
So in my case, I'm trying to catch a tranisition from false to true on hasError.
I'll break down what happens with each change on the previous implementation and the new one, so the difference will be clear:
Old implementation

Actual value
prevProps
nextProps
comment

false
false
false

true
false
true

false
true
false

true
false
true

New implementation

Actual value
prevProps
nextProps
comment

false
false
false

true
false
true

false
true
false

true
true
true
prevProps has true because we don't save the previous values in the change before

Notice how the prevProps is always correctly stating the last value sent to nextProps except for the last row in the new implementation.

Good catch,
let me think.
The problem I think with all that comparisons will arise when react become async, so it will be possible that some updates will be replayed multiple times, memoize will work, comparisons will not. BTW to support your case seems like for now will be better to revert. Tests and PR are welcome.

In the future code above will be a problem, as you could get multiple heavy recalc during render phase

Returning will be no different


Add Vector AbsDiffEq, RelativeEq, and UlpsEq trait impl for floating point approximate equivalence relation comparisons

Closes #10
TODO

[x] add docs

docs: https://github.com/chrissimpkins/vectora/pull/11/commits/c4aebc16b322dc821983cebfe5ed3178594178ca


fix: lock broccoli-funnel to prevent rebuild error

Temporary fix until we find the problem in broccoli-funnel. This fixes errors
of the following nature:
The Broccoli Plugin: [BroccoliMergeTrees] failed with:
Error: ENOENT: no such file or directory, lstat '/Users/iradchenko/sandbox/ember-admin-demo/tmp/funnel-output_path-TLi8yNpH.tmp'
    at Error (native)
    at Object.fs.lstatSync (fs.js:982:18)
    at symlink (/Users/iradchenko/sandbox/ember-engines/node_modules/symlink-or-copy/index.js:60:26)
    at symlinkOrCopySync (/Users/iradchenko/sandbox/ember-engines/node_modules/symlink-or-copy/index.js:55:5)
    at /Users/iradchenko/sandbox/ember-engines/node_modules/broccoli-plugin/read_compat.js:58:9
    at tryCatch (/Users/iradchenko/sandbox/ember-engines/node_modules/rsvp/dist/rsvp.js:538:12)
    at invokeCallback (/Users/iradchenko/sandbox/ember-engines/node_modules/rsvp/dist/rsvp.js:553:13)
    at publish (/Users/iradchenko/sandbox/ember-engines/node_modules/rsvp/dist/rsvp.js:521:7)
    at flush (/Users/iradchenko/sandbox/ember-engines/node_modules/rsvp/dist/rsvp.js:2373:5)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

The broccoli plugin was instantiated at:
    at BroccoliMergeTrees.Plugin (/Users/iradchenko/sandbox/ember-engines/node_modules/broccoli-plugin/index.js:7:31)
    at new BroccoliMergeTrees (/Users/iradchenko/sandbox/ember-engines/node_modules/broccoli-merge-trees/index.js:42:10)
    at BroccoliMergeTrees (/Users/iradchenko/sandbox/ember-engines/node_modules/broccoli-merge-trees/index.js:36:53)
    at CoreObject.treeForPublic (/Users/iradchenko/sandbox/ember-engines/lib/engine-addon.js:369:46)
    at CoreObject._treeFor (/Users/iradchenko/sandbox/ember-admin-demo/node_modules/ember-cli/lib/models/addon.js:371:33)
    at CoreObject.treeFor (/Users/iradchenko/sandbox/ember-engines/lib/engine-addon.js:624:23)
    at /Users/iradchenko/sandbox/ember-admin-demo/node_modules/ember-cli/lib/broccoli/ember-app.js:498:20
    at Array.map (native)
    at EmberApp.addonTreesFor (/Users/iradchenko/sandbox/ember-admin-demo/node_modules/ember-cli/lib/broccoli/ember-app.js:496:30)
    at EmberApp.publicTree (/Users/iradchenko/sandbox/ember-admin-demo/node_modules/ember-cli/lib/broccoli/ember-app.js:773:20)

when the parent app rebuilds from a change in an engine.
cc @nathanhammond @hyderali

@knownasilya Thank you for bisecting this! We'll figure out what the underlying problem is.
@stefanpenner https://github.com/broccolijs/broccoli-funnel/compare/v1.0.6...v1.0.7 introduces a bug in ember-engines. Unsure why or how, haven't begun bisecting this.


Rewrite requirements-dev.txt before installing in conda CI.

Even if packages are listed as pinned_packages, conda still tries to upgrade/downgrade them if the packages are in requirements-dev.txt without the version or with a version range.
pinned spec pyarrow==0.15.1 conflicts with explicit specs.  Overriding pinned spec.

The installed version satisfies the version range, though.

Codecov Report

Merging #1576 into master will decrease coverage by 1.66%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1576      +/-   ##
==========================================
- Coverage   94.56%   92.89%   -1.67%
==========================================
  Files          38       38
  Lines        8773     8675      -98
==========================================
- Hits         8296     8059     -237
- Misses        477      616     +139

Impacted Files
Coverage Δ

databricks/koalas/usage_logging/__init__.py
26.78% <0.00%> (-64.60%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50.00% <0.00%> (-50.00%)
:arrow_down:

databricks/koalas/__init__.py
84.31% <0.00%> (-9.81%)
:arrow_down:

databricks/conftest.py
94.33% <0.00%> (-5.67%)
:arrow_down:

databricks/koalas/utils.py
94.09% <0.00%> (-2.29%)
:arrow_down:

databricks/koalas/typedef/typehints.py
85.26% <0.00%> (-2.11%)
:arrow_down:

databricks/koalas/generic.py
94.71% <0.00%> (-1.95%)
:arrow_down:

databricks/koalas/testing/utils.py
78.98% <0.00%> (-1.81%)
:arrow_down:

databricks/koalas/frame.py
95.48% <0.00%> (-1.35%)
:arrow_down:

databricks/koalas/spark/accessors.py
91.66% <0.00%> (-1.08%)
:arrow_down:

... and 10 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 a02fe6e...01e1068. Read the comment docs.

I'd merge this now to unblock PRs since the tests started without any installation issues.


PR: Add "run selection" icon to run toolbar

Fixes #4020
This also removes the "Run settings" icon. The run toolbar looks like this now:

Nice, thanks!


Error when click in "Send new Issue"

I'm trying to click in "Send new Issue" button in http://www.codetriage.com, but I get a "We're sorry, but something went wrong"

When running local, the IssueAssignmentsController is trying to call a undefined_method

Closed by #308


postbox - ing.de: adapt to new layout (fix #6)

ING restructured the postbox table. This commit allows parsing of the updated layout.

Thank you @dominik-th for the fix!

I have to thank you. The project saved me a lot of time, glad I could give something back


⚠ Bofrost.lu has degraded performance

In 9ff71a7, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1114 ms

Resolved: Bofrost.lu performance has improved in 9f43d9a after 35 minutes.


Update to 1.18.1 and added spanish (Spain) language

I updated the mod to be compatible with 1.18.1 and Mod Menu 3.0.0.
Additionally, it now has a spanish translation.

Awesome! When I can get to a computer with a dev environment, I'll test it and merge it

No problem! although I'm not really sure about the mod_version jump that I did (1.2.3 to 1.3.0), since the only major thing it's the 1.18 update.

Ah, didn't catch that it. Yeah the version number shouldn't change from the 1.17 won't m one

gg me, I merged this on github, so now the 1.18 is in the 1.17 branch. Going to fix now, then do a release


[FR]

Is your feature request related to a problem? Please describe.
Ability to play spotify playlists?
Describe the solution you'd like
Same way how youtube playlists work but accept spotify playlist links!
Describe alternatives you've considered
I was going to work on grabbing every single song from a Spotify playlist link and then querying it thru YT to play thru voice chat but im sure others would also enjoy this feature as part of the module.

lol

lol

im a little slow 😔


sap.m.Dialog: <footer> content not affected by sapUiResponsivePadding--footer

URL (minimal example if possible):
https://jsbin.com/ruroquz/edit?js,output
Steps to reproduce the problem:

Open the above sample. As you can see in the code, the following sapUiResponsivePadding--* CSS classes are assigned to the dialog's list of style classes:

sapUiResponsivePadding--header
sapUiResponsivePadding--content
sapUiResponsivePadding--footer

In the UI, increase the dialog size by dragging the bottom right corner. --> Issue: the responsive footer paddings are not applied corresponding to the changed dialog width.
Close the dialog and open the other "Dialog Without Custom Footer". --> This time, the responsive paddings are all applied expectedly corresponding to the changed dialog width.

Hello @boghyon ,
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.

Thanks! :)

Hello @boghyon ,
Thank you for finding this, the issue is now fixed :)
Best regards,
Petar


Times when we fire a pulse seems wrong

Hi @rodolfocarobene,
I am testing the qibosoq interpolated version by sending different types of pulses and plotting them into the oscilloscope. I already did the same with qick and  things worked as expected. Now, using qibosoq, I see the duration of the pulses are Ok but the time when they are fired is weird. I am wondering if this is the way it was programmed. In addition, how can we send pulses at the same time?
The pulses are:
`pulse_fl_1 = Rectangular(
frequency = 0, #MHz
amplitude = -0.5,
relative_phase = 0,
start_delay = 0,
duration = 1.,
name = "flux_pulse",
type = "readout",
dac = 0,
adc = 0
)
pulse_fl_2 = Rectangular(
frequency = 0, #MHz
amplitude = -0.5,
relative_phase = 0,
start_delay = 0.0,
duration = 2.,
name = "flux_pulse",
type = "readout",
dac = 0,
adc = 0
)
pulse_drive_1 = Rectangular(
frequency = 100,    # float in MHz
amplitude = 0.5,     # float in [-1, 1]
relative_phase = 0,  # int in degrees
start_delay = 0.0,     # float in us
duration = 0.5,        # float in us
name = "id1",         # str
mixer_frequency = 150.0, # float in MHz
type = "readout",      # str in {"readout", "drive"}
#rel_sigma = 6,
dac = 5,             # int
adc = 0,          # optional int
)
pulse_drive_2 = Rectangular(
frequency = 100,    # float in MHz
amplitude = 0.5,     # float in [-1, 1]
relative_phase = 0,  # int in degrees
start_delay = 0.,     # float in us
duration = 0.5,        # float in us
name = "id2",
mixer_frequency = 150.0, # float in MHz        # str
type = "readout",      # str in {"readout", "drive"}
#rel_sigma = 5,
dac = 5,             # int
adc = 0,          # optional int
)
pulse_ro = Rectangular(
frequency = 200, #MHz
amplitude = 0.5,
relative_phase = 0,
start_delay = 0.,
duration = 2.,
name = "flux_pulse",
type = "readout",
dac = 14,
adc = 0
)
sequence = [pulse_fl_1, pulse_drive_1, pulse_ro,pulse_fl_2 , pulse_drive_2]`

Hi @JavierSerranoGarcia , this is quite weird. Pulses should start immediately when start_delay=0
https://github.com/qiboteam/qibosoq/blob/faa13a2bfb36c996fb8b24aa9dc04fa67376e84c/src/qibosoq/programs/flux.py#L159-L161
Can you share the working QICK code? Also, would be nice to know if this problem appears also with standard firmwares or is linked to the interpolated channels.
Moreover, does the frequency part work?

The qick program:
`(pynq-venv) xilinx@ZCU216b-pynq:~/logs$ cat program.log
INFO :: 2024-05-10 08:02:12 ::
// Program
    synci 700;
    regwi 0, $15, 0;
    regwi 0, $14, 0;

LOOP_J: synci 50;
regwi 0, $22, 0;                        //phase = 0 | freq = 0
regwi 0, $24, -1073676288;              //gain = -16383 | addr = 0
regwi 0, $31, 0;                        //mode3 = 0
regwi 0, $26, 590131;                   //stdysel | mode | outsel = 0b01001 | length = 307
regwi 0, $16, 16;                       //out = 0b0000000000010000
seti 7, 0, $16, 181;                    //ch =0 out = $16 @t = 0
seti 7, 0, $0, 191;                     //ch =0 out = 0 @t = 0
regwi 0, $27, 0;                        //t = 0
set 0, 0, $22, $24, $26, $0, $31, $27;  //ch = 0, pulse @t = $27
synci 530;
regwi 2, $12, 3810;                     //phase = 0 | freq = 3810
regwi 2, $14,<PHONE_NUMBER>;               //gain = 16383 | addr = 0
regwi 2, $21, 16777216;                 //mode3 = 16777216
regwi 2, $16, 590039;                   //stdysel | mode | outsel = 0b01001 | length = 215
regwi 0, $16, 16;                       //out = 0b0000000000010000
seti 7, 0, $16, 181;                    //ch =0 out = $16 @t = 0
seti 7, 0, $0, 191;                     //ch =0 out = 0 @t = 0
regwi 2, $17, 0;                        //t = 0
set 1, 2, $12, $14, $16, $0, $21, $17;  //ch = 5, pulse @t = $17
synci 530;
regwi 5, $12, 87381333;                 //freq = 87381333
regwi 5, $13, 0;                        //phase = 0
regwi 5, $15, 16383;                    //gain = 16383
regwi 5, $16, 591053;                   //phrst| stdysel | mode | | outsel = 0b01001 | length = 1229
regwi 0, $16, 16;                       //out = 0b0000000000010000
seti 7, 0, $16, 181;                    //ch =0 out = $16 @t = 0
seti 7, 0, $0, 191;                     //ch =0 out = 0 @t = 0
regwi 5, $17, 0;                        //t = 0
set 5, 5, $12, $13, $0, $15, $16, $17;  //ch = 14, pulse @t = $17
synci 700;
regwi 0, $22, 0;                        //phase = 0 | freq = 0
regwi 0, $24, -1073676288;              //gain = -16383 | addr = 0
regwi 0, $31, 0;                        //mode3 = 0
regwi 0, $26, 590438;                   //stdysel | mode | outsel = 0b01001 | length = 614
regwi 0, $16, 16;                       //out = 0b0000000000010000
seti 7, 0, $16, 181;                    //ch =0 out = $16 @t = 0
seti 7, 0, $0, 191;                     //ch =0 out = 0 @t = 0
regwi 0, $27, 0;                        //t = 0
set 0, 0, $22, $24, $26, $0, $31, $27;  //ch = 0, pulse @t = $27
synci 699;
regwi 2, $12, 3810;                     //phase = 0 | freq = 3810
regwi 2, $14,<PHONE_NUMBER>;               //gain = 16383 | addr = 0
regwi 2, $21, 16777216;                 //mode3 = 16777216
regwi 2, $16, 590039;                   //stdysel | mode | outsel = 0b01001 | length = 215
regwi 0, $16, 16;                       //out = 0b0000000000010000
seti 7, 0, $16, 181;                    //ch =0 out = $16 @t = 0
seti 7, 0, $0, 191;                     //ch =0 out = 0 @t = 0
regwi 2, $17, 0;                        //t = 0
set 1, 2, $12, $14, $16, $0, $21, $17;  //ch = 5, pulse @t = $17
synci 530;
waiti 0, 0;
synci 50;
synci 35;
mathi 0, $15, $15 + 1;
memwi 0, $15, 1;
loopnz 0, $14, @LOOP_J;
end ;`
Channel 5 is interpolated and seems it works but I have not checked yet if the right frequency is sent,
Regarding other firmwares I will try later

Yes, there is indeed always the synci instruction, but that should not be the case...
We have to check if this is something introduced by QICK (maybe with interpolated channel it is required to wait a bit?) or not. If this is the case with other firmware versions this should not happen.
Moreover, another possible source of error could be a mismatched version between the qibosoq client and the qibosoq server, I believe

Hi @rodolfocarobene, which version of qibosoq and qick we should use on both sides?

The interpolated branch of qibosoq, both for the client and the server.
QICK should be of version <=0.2.249

Hi Javier, version 0.2.230 should be supported. If I remember correctly I also have tested it when I was at TII.
I would try to debug the problem adding some "print" in the server code.
For example you could add print(obj_sequence) at the end of this:
https://github.com/qiboteam/qibosoq/blob/30aa3de9cb0b6431aa4154a7c1c51d393009a0b2/src/qibosoq/server.py#L23-L34
In this way we will be able to understand if the problem (not having a fixed start_delay) is in the communication protocol or later


(deadline) Repository constructe fails to install with MountableFsxLustre

The Repository construct fails to install Deadline Repository when the IMountableLinuxFilesystem provided to it is a MountableFsxLustre because the MountableFsxLustre class uses POSIX permissions, which causes the Repository construct to run the Deadline Repository installer as the Deadline repository user, which will fail because the FSx Lustre file system mount point is still owned and only writable to by root.
Reproduction Steps

Use the Repository construct with the MountableFsxLustre class.
Error Log

Deadline Repository installer logs:
Preferred installation mode : unattended
--
Trying to init installer in mode unattended
Mode unattended successfully initialized
This can take a few minutes
Preparing to Install
Preparing to Install
Error creating directory /mnt/efs/fs1/DeadlineRepository
There has been an error.
Unable to create directory

Environment

CDK CLI Version      : 2.51.1 (build 3d30cdb)
CDK Framework Version: 2.33.0
RFDK Version: 1.0.0
Deadline Version: <IP_ADDRESS>
Node.js Version:  v16.13.0
OS               : Linux
Language (Version):  Python 3.9.15

Other

One idea is to add a "mount owner user" for IMountableLinuxFilesystem, where, if a subclass .usesPosixPermissions(), then they must ensure the path the filesystem is mounted at is also owned/writable by the "mount owner user" as part of their .mountToLinuxInstance() implementation.

This is :bug: Bug Report

A temporary workaround for this would be to subclass MountableFsxLustre and add the "mount owner user" idea as a prop to it. Example in Python:
from aws_cdk.aws_fsx import LustreFileSystem
from aws_rfdk import (
    IMountingInstance,
    MountableFsxLustre as OriginalMountableFsxLustre,
    MountPermissions,
)
from constructs import Construct
from typing import Optional, Sequence

class MountableFsxLustre(OriginalMountableFsxLustre):
    def __init__(
        self,
        scope: Construct,
        *,
        filesystem: LustreFileSystem,
        extra_mount_options: Optional[Sequence[str]] = None,
        mount_owner_user: Optional[str] = None,
        **kwargs,
    ):
        super().__init__(
            scope,
            filesystem=filesystem,
            extra_mount_options=extra_mount_options,
            **kwargs
        )
        self.mount_owner_user = mount_owner_user

    def mount_to_linux_instance(
        self,
        target: IMountingInstance,
        *,
        location: str,
        permissions: Optional[MountPermissions] = None,
    ):
        super().mount_to_linux_instance(target, location=location, permissions=permissions)
        if self.mount_owner_user:
            target.user_data.add_commands(f"sudo chown {self.mount_owner_user} {location}")

Then, when creating the Repository construct, pass in the Deadline Repository user (which is hardcoded in the Repository construct...) to this subclass:
repository = Repository(
    # other props...
    file_system=MountableFsxLustre(
        # other props...
        mount_owner_user='1000:1000',  # The hardcoded repository UID & GID
    ),
)


Release90 apidocbranding

Update index file and last release date to get javadoc main pages and api changes with accurate date

Hi @vieiro do you think this would be acceptable with also your modification
https://github.com/apache/incubator-netbeans/commit/4f98d5b61eb81ea88f1e441a517dbb8864ff20ee
Asking also @neilcsmith-net for review.

Can we have such a PR for release100 branch as well?

Hi @ebarboni I don't think https://github.com/apache/incubator-netbeans/commit/4f98d5b61eb81ea88f1e441a517dbb8864ff20ee is worth it since this is a "release" javadoc. The timestamp trick makes sense if were to push the same javadoc multiple times (as this would reduce the number of diffs), but we're pushing the 9.0 javadocs just once.

Sorry, only just noticed this because of the Release100 one.  I know I made a few more changes to the output from export2html.xsl, both changing the title in head and a lot of the body text / links to refer to Apache.  https://github.com/apache/incubator-netbeans-docs/blob/asf-site/content/9.0/javadoc/overview-summary.html  I started editing export2html.xsl but then ended up editing the output file instead for some reason that escapes me now.  Obviously needs to go into the xsl file.
Longer term it might be good if we could switch on a release flag (or even the version number) so that release and dev text variants can remain in the file?
Also be good to get the Release100 one in at least before the vote so that checking out the tag for 10.0 can build the correct Javadoc.  Obviously not an option for this one.

Just one note guys. As long as the project in incubator phase we cannot use the name Apache NetBeans without mentioning "(incubating)". Please fix the headers!

Hi I ask @sdedic for review as it's more or less the same than for release100 branch.
Would be nice to merge if ok.

Hi @emilianbold, would just notify that I want to merge on release90 to have apidoc prepared for generation. (as you are the release manager for release90)
Best Regards

I have nothing against this PR, seems to only do some minor changes. We don't need to do another 9.0 release so all is well.
+1

merging apidoc branding