Firestore connection delays when enables WiFi in background

[REQUIRED] Step 1: Describe your environment

Xcode version: 11.5 Build version 11E608c
Firebase SDK version: 6.25.0
Firebase Component: Firestore
Component version: 1.14.0
Installation method: CocoaPods

[REQUIRED] Step 2: Describe the problem
After changing a connection from 4G to WiFi in the background, it seems that it spent long time (about 20 seconds) to reconnect to the server at when the app has become active.
Some documents are created while the app is in the background, then makes the app active:

snapshotListener isn't called until about 20 seconds after
query.getDocuments with FirestoreSource.server is failed with the error:

Error Domain=FIRFirestoreErrorDomain Code=14 "Failed to get documents from server. (However, these documents may exist in the local cache. Run again without setting source to FirestoreSourceServer to retrieve the cached documents.)"

If it is called with FirestoreSource.default, it returns results from the cache.
Steps to reproduce:

Activates an app with WiFi disabled. (4G is used)
Makes the app background.
Enables WiFi.
Creates some documents on firestore in order to make snapshotListener called.
Makes the app active
-> After about 20 seconds, snapshotListener is called

@shosokawa Thanks for reporting this.
We have seen similar reports for Transactions (https://github.com/firebase/firebase-ios-sdk/issues/4905) and believe that its fix (https://github.com/firebase/firebase-ios-sdk/pull/4985 ) should apply to all of our networking logic.
It will take me a couple of days to look at this in more detail, which should allow us figure out why the code changes for #4905 don't apply in this case.  Please do bear with us.


Problem with otbr-agent status. Init() at spinel_driver.cpp:82: Failure.

Hi,
I've come across with a problem regarding the connection of my nRF52840-DK board to my Raspberry Pi 4B to start the otbr-agent service. I followed the instructions indicated here: https://openthread.io/codelabs/openthread-border-router#1, but to no avail.
Are there any solutions to this? I can't run the succeeding commands, e.g., "sudo ot-ctl dataset init new".
Thanks!

Have you flashed the latest ot-rcp firmware for your nRF52840-DK? I think it may be an incompatibility between the host and the RCP.
cc @zhanglongxia @Irving-cl

Yes, I followed the same steps here in flashing the device. Is there a much newer version than this?
Thanks!

Could you provide logs of the otbr-agent?  You can see the log via tail -f  /var/log/syslog | grep otbr.

I'm getting the same error message, looking at the line the error message is referencing my best guess is there's a version mismatch between the border router and the RCP?

Here are the logs too:
Aug 05 21:15:47 apollo otbr-agent[2373]: [NOTE]-AGENT---: Thread interface: wpan0
Aug 05 21:15:47 apollo otbr-agent[2373]: [NOTE]-AGENT---: Radio URL: spinel+hdlc+uart:///dev/ttyACM0
Aug 05 21:15:47 apollo otbr-agent[2373]: [NOTE]-AGENT---: Radio URL: trel://eth0
Aug 05 21:15:47 apollo otbr-agent[2373]: [NOTE]-ILS-----: Infra link selected: eth0
Aug 05 21:15:47 apollo otbr-agent[2373]: [INFO]-RCP_HOS-: OpenThread log level changed to 4
Aug 05 21:15:49 apollo otbr-agent[2373]: 49d.17:06:09.397 [W] P-SpinelDrive-: Wait for response timeout
Aug 05 21:15:49 apollo otbr-agent[2373]: 49d.17:06:09.397 [I] P-SpinelDrive-: co-processor self reset successfully
Aug 05 21:15:51 apollo otbr-agent[2373]: 49d.17:06:11.399 [W] P-SpinelDrive-: Wait for response timeout
Aug 05 21:15:53 apollo otbr-agent[2373]: 49d.17:06:13.401 [W] P-SpinelDrive-: Wait for response timeout
Aug 05 21:15:53 apollo otbr-agent[2373]: 49d.17:06:13.401 [C] Platform------: Init() at spinel_driver.cpp:82: Failure

Hello,
Seems to be the same issue for us.
Here is our setup :

Sonoff ZBDongle-E (OpenThread (RCP) SL-OPENTHREAD/<IP_ADDRESS>_GitHub-7074a43e4)

Linked on /dev/ttyACM1 on host and on container

Latest docker image for otb (updated 3 days ago on Docker Hub)

ot-agent are crashing and daemon isn't running.
Logs :
Aug  5 20:48:22 2a45fc7d7e9e mDNSResponder: Default: mDNSResponder (Engineering Build) (Aug  2 2024 16:28:17) starting
Aug  5 20:48:23 2a45fc7d7e9e rsyslogd: rsyslogd's groupid changed to 101
Aug  5 20:48:23 2a45fc7d7e9e rsyslogd: rsyslogd's userid changed to 101
Aug  5 20:48:23 2a45fc7d7e9e rsyslogd:  [origin software="rsyslogd" swVersion="8.32.0" x-pid="98" x-info="http://www.rsyslog.com"] start
Aug  5 20:48:24 2a45fc7d7e9e otbr-web[172]: [INFO]-WEB-----: Running 0.3.0-da1f867
Aug  5 20:48:24 2a45fc7d7e9e otbr-web[172]: [INFO]-WEB-----: Border router web started on wpan0
Aug  5 20:48:24 2a45fc7d7e9e otbr-web[172]: [ERR ]-WEB-----: OpenThread daemon is not running.
Aug  5 20:48:24 2a45fc7d7e9e otbr-agent: [NOTE]-AGENT---: Backbone interface: eth0
Aug  5 20:48:24 2a45fc7d7e9e otbr-agent[199]: [NOTE]-AGENT---: Running 0.3.0-da1f867
Aug  5 20:48:24 2a45fc7d7e9e otbr-agent[199]: [NOTE]-AGENT---: Thread version: 1.3.0
Aug  5 20:48:24 2a45fc7d7e9e otbr-agent[199]: [NOTE]-AGENT---: Thread interface: wpan0
Aug  5 20:48:24 2a45fc7d7e9e otbr-agent[199]: [NOTE]-AGENT---: Radio URL: spinel+hdlc+uart:///dev/ttyACM1
Aug  5 20:48:24 2a45fc7d7e9e otbr-agent[199]: [NOTE]-ILS-----: Infra link selected: eth0
Aug  5 20:48:24 2a45fc7d7e9e otbr-agent[199]: [INFO]-RCP_HOS-: OpenThread log level changed to 5
Aug  5 20:48:24 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:42.935 [D] P-SpinelDrive-: Sent spinel frame, flg:0x2, iid:0, tid:0, cmd:RESET
Aug  5 20:48:24 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:42.935 [D] P-SpinelDrive-: Waiting response: key=0
Aug  5 20:48:26 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:44.937 [W] P-SpinelDrive-: Wait for response timeout
Aug  5 20:48:26 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:44.937 [I] P-SpinelDrive-: co-processor self reset successfully
Aug  5 20:48:26 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:44.937 [D] P-SpinelDrive-: Sent spinel frame, flg:0x2, iid:0, tid:1, cmd:PROP_VALUE_GET, key:PROTOCOL_VERSION
Aug  5 20:48:26 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:44.938 [D] P-SpinelDrive-: Waiting response: key=1
Aug  5 20:48:28 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:46.940 [W] P-SpinelDrive-: Wait for response timeout
Aug  5 20:48:28 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:46.940 [D] P-SpinelDrive-: Sent spinel frame, flg:0x2, iid:0, tid:1, cmd:PROP_VALUE_GET, key:PROTOCOL_VERSION
Aug  5 20:48:28 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:46.940 [D] P-SpinelDrive-: Waiting response: key=1
Aug  5 20:48:30 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:48.942 [W] P-SpinelDrive-: Wait for response timeout
Aug  5 20:48:30 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:48.942 [C] Platform------: Init() at spinel_driver.cpp:82: Failure
Aug  5 20:48:30 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:48.943 [D] P-SpinelDrive-: Sent spinel frame, flg:0x2, iid:0, tid:1, cmd:PROP_VALUE_GET, key:PROTOCOL_VERSION
Aug  5 20:48:30 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:48.943 [D] P-SpinelDrive-: Waiting response: key=1
Aug  5 20:48:32 2a45fc7d7e9e otbr-agent[199]: 58d.21:22:50.945 [W] P-SpinelDrive-: Wait for response timeout

I do have the same problem while using a ESP32-C6 with the most recent rcp image connected via USB
ot_rcp
2024-08-05T22:33:06.572147+01:00 otbr2 otbr-agent: [NOTE]-AGENT---: Backbone interface: wlan0
2024-08-05T22:33:06.572485+01:00 otbr2 otbr-agent[9110]: [NOTE]-AGENT---: Running 0.3.0-thread-reference-20230710-339-gda1f867030
2024-08-05T22:33:06.572623+01:00 otbr2 otbr-agent[9110]: [NOTE]-AGENT---: Thread version: 1.3.0
2024-08-05T22:33:06.572718+01:00 otbr2 otbr-agent[9110]: [NOTE]-AGENT---: Thread interface: wpan0
2024-08-05T22:33:06.572795+01:00 otbr2 otbr-agent[9110]: [NOTE]-AGENT---: Radio URL: spinel+hdlc+uart:///dev/ttyACM0?uart-baudrate=460800
2024-08-05T22:33:06.572877+01:00 otbr2 otbr-agent[9110]: [NOTE]-AGENT---: Radio URL: trel://wlan0
2024-08-05T22:33:06.572948+01:00 otbr2 otbr-agent[9110]: [NOTE]-ILS-----: Infra link selected: wlan0
2024-08-05T22:33:06.573046+01:00 otbr2 otbr-agent[9110]: [INFO]-RCP_HOS-: OpenThread log level changed to 4
2024-08-05T22:33:08.576571+01:00 otbr2 otbr-agent[9110]: 49d.19:00:21.954 [W] P-SpinelDrive-: Wait for response timeout
2024-08-05T22:33:08.576760+01:00 otbr2 otbr-agent[9110]: 49d.19:00:21.954 [I] P-SpinelDrive-: co-processor self reset successfully
2024-08-05T22:33:10.578648+01:00 otbr2 otbr-agent[9110]: 49d.19:00:23.956 [W] P-SpinelDrive-: Wait for response timeout
2024-08-05T22:33:12.581322+01:00 otbr2 otbr-agent[9110]: 49d.19:00:25.958 [W] P-SpinelDrive-: Wait for response timeout
2024-08-05T22:33:12.581681+01:00 otbr2 otbr-agent[9110]: 49d.19:00:25.958 [C] Platform------: Init() at spinel_driver.cpp:82: Failure
2024-08-05T22:33:14.583655+01:00 otbr2 otbr-agent[9110]: 49d.19:00:27.960 [W] P-SpinelDrive-: Wait for response timeout

Perhaps you can refer to this link to try adding flow control parameters.

@ihidchaos Thanks, I can confirm that this works! TLDR: edit /etc/default/otbr so that it includes the following or similar:
OTBR_AGENT_OPTS="-I wpan0 -B eth0 spinel+hdlc+uart:///dev/ttyACM0?uart-flow-control trel://eth0"

I cannot bring it to work with a Sonoff ZBDongle-E, too?
Log: with an error on spinel_driver.cpp:83(!)
2024-12-09T18:42:54.433987+01:00 dspi otbr-agent[29507]: 49d.17:58:08.106 [I] P-SpinelDrive-: co-processor self reset successfully
2024-12-09T18:42:56.436555+01:00 dspi otbr-agent[29507]: 49d.17:58:10.108 [W] P-SpinelDrive-: Wait for response timeout
2024-12-09T18:42:58.438888+01:00 dspi otbr-agent[29507]: 49d.17:58:12.110 [W] P-SpinelDrive-: Wait for response timeout
2024-12-09T18:42:58.439326+01:00 dspi otbr-agent[29507]: 49d.17:58:12.110 [C] Platform------: Init() at spinel_driver.cpp:83: Failure
2024-12-09T18:43:00.440255+01:00 dspi otbr-agent[29507]: 49d.17:58:14.112 [W] P-SpinelDrive-: Wait for response timeout
2024-12-09T18:43:01.184773+01:00 dspi systemd[1]: otbr-agent.service: Main process exited, code=exited, status=1/FAILURE
On a raspi 4. Adding the ?uart-flow-control did not make any change.
Does anybody know a solution, or is the (newer)  Sonoff ZBDongle-E not usable with otbr?
Thanks and greetings!

I had the same problem when updating to GSDK4.4.5 using multi-protocol firmware.
For multi-protocol, the "adding flow control parameters" method has no effect.
ubuntu@ubuntu:~$ sudo otbr-agent -I wpan0 -B eth0 "spinel+cpc://cpcd_0?iid=2&iid-list=0" -v otbr-agent[1634]: [NOTE]-AGENT---: Running 0.3.0-bff5d377-dirty otbr-agent[1634]: [NOTE]-AGENT---: Thread version: 1.4.0 otbr-agent[1634]: [NOTE]-AGENT---: Thread interface: wpan0 otbr-agent[1634]: [NOTE]-AGENT---: Radio URL: spinel+cpc://cpcd_0?iid=2&iid-list=0 otbr-agent[1634]: [NOTE]-ILS-----: Infra link selected: eth0 otbr-agent[1634]: [INFO]-RCP_HOS-: OpenThread log level changed to 4 otbr-agent[1634]: 49d.17:16:59.025 [W] P-SpinelDrive-: Wait for response timeout otbr-agent[1634]: 49d.17:16:59.025 [I] P-SpinelDrive-: co-processor self reset successfully otbr-agent[1634]: 49d.17:17:01.026 [W] P-SpinelDrive-: Wait for response timeout otbr-agent[1634]: 49d.17:17:03.026 [W] P-SpinelDrive-: Wait for response timeout otbr-agent[1634]: 49d.17:17:03.026 [C] Platform------: Init() at spinel_driver.cpp:82: Failure otbr-agent[1634]: 49d.17:17:05.026 [W] P-SpinelDrive-: Wait for response timeout ubuntu@ubuntu:~$

@Mrz-creator : I could get it to work, after I inserted both parameters, the "flow control" and the correct "baudrate"!
OTBR_AGENT_OPTS="-I wpan0 -B wlan0 spinel+hdlc+uart:///dev/ttyUSB0?uart-flow-control&uart-baudrate=460800 trel://wlan0"
OTBR_NO_AUTO_ATTACH=0

@StrippgenHTW  Thank you for your reply, but I am using multi-protocol mode and do not have uart. I am using cpcd. My configuration must be changed to the following：
-I wpan0 -B eth0 spinel+cpc://cpcd_0?iid=2&iid-list=0

@StrippgenHTW Thank you for your reply, but I am using multi-protocol mode and do not have uart. I am using cpcd. My configuration must be changed to the following： -I wpan0 -B eth0 spinel+cpc://cpcd_0?iid=2&iid-list=0

Silabs' cpcd program also has a cpcd.conf configuration file to set up flow control, perhaps you can refer to this file to modify your device's parameters. https://github.com/SiliconLabs/cpc-daemon/blob/main/cpcd.conf#L42

@ihidchaos Thanks for your answer，I tried single protocol thread RCP with success. I want multi-protocol to work too, I modified cpcd.conf as you said and configured the firmware to support flow control, but the result is still frustrating.


Clarify URL when creating the lable

Please update the documentation when creating the label itself in the compliance portal.
You need to include the KEY name you have provided within "TestKeys" section of the appsettings.json into the url. For example:
https://pocdke1.azurewebsites.net/**KEYNAME**

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

ID: 484b1a6d-dfd0-1da6-5405-7826ed4c088f
Version Independent ID: 6e0ba9d2-aaa6-528a-daab-0e75c80f7135
Content: Double Key Encryption (DKE) - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/double-key-encryption.md
Service: information-protection
GitHub Login: @KCCross
Microsoft Alias: krowley

@alschneiter Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/4873. Once the author approves, the document will be udpated.
Thanks
Sri

@alschneiter and @yogkumgit
To me, it looks like this information might be incorporated in the article now:

@alschneiter - can you please confirm whether your issue is now resolved? If it is, we can close this related PR.  Please @ mention me in your reply.
Thank you!

@msbemba can you please take a look and see where we're at with this one? I've merged the PR. Please continue to test. Closing this comment in the meantime.


Proper location of schema import

Right now we use a type that isn't quite correct and produces an error in server.ts. I need to drill down again and place it.

Linked to hdresearch/nolita#44


How to get the ordered elements  manually?

I insert dom into gridly dom and call method gridly, but it did not trigger callback.How do I deal with it?
BTW, can I change some grid's position with code?

these are my dirty hack to solve problems above


Grammar: Add struct expression.

This includes some rework and expansion of the contents.

💟 Thanks!


Condition with String fails I believe

Note for me to investigate a situation like this

resulting in

I've been trying some conditions and they are a bit broken. They also get transformed weird to text, but I'll try to report it better when I have a proper timeline to test against.


[UNDERTOW-2422] Return the protocol field of the HttpServerExchange into the status line

Issue: https://issues.redhat.com/browse/UNDERTOW-2422
Currently, no matter what you set the HttpServerExchange.protocol field to, the value "HTTP/1.1" will always be used. This change will use the value of the protocol field (or "HTTP/1.1" if the field is empty) in the HTTP response's status line.
There is a test included in this PR. It confirms the default behavior and the new behavior.

@ropalka The two test failures on JDK17-core, Mac and Windows, are "Buffer Leaks", which are listed as intermittent in the known test issues ticket. The failure in testRstOnPush in JDK11-ipv6 on ubuntu is, I think, another intermittent. I believe in the prior CI run, this same test failed in a different environment, and succeeded in this run. I believe that this CI run should be treated as having succeeded and that testRstOnPush should be added to the known issues list.

I have squashed the 4 commits into 1.

@fl4via I have made the requested changes (and squashed the commit).


Exception in a loop body is not reported when expectations are not met

Consider the following example:
foreach (var _ in EventualAssertions.Attempts(4000, 50))
{
    false.Should().BeTrue();
    throw new Exception();
}

The exception thrown leads to the underlying AssertionScope being disposed. And since there is a failure in the scope, a corresponding exception is instantly thrown.
The expected behaviour - to report the explicitly thrown exception

Not solved for the F# version


[Question] Trace/Debug events missing from Seq when using Seq extension

For local environment / debugging I prefer to have Debug events to be logged.
Unfortunately, I was unable to configure service (I assume problem lies it is on a sending side) to send Debug events.
I tried:
builder.Logging.SetMinimumLevel(LogLevel.Debug);

and:
builder.Logging.AddFilter(level => true);

with no success.
`tye.yaml' is:
extensions:
  - name: seq
    logPath: "./.volumes/seq"

Is there a way to fix it, am I doing something wrong?
I understand, that I can host Seq as a container (not extension) and configure Serilog.Sink.Seq but I maybe there is something simpler / quicker I just missed.

When I've used the Seq extension I get mixed results depending on the service type (Container, function or project)
I've always opted to use a Sink for serilog just to be sure, and switch out my sinks for my target environment!
I hope this helps.

When I've used the Seq extension I get mixed results depending on the service type (Container, function or project)
I've always opted to use a Sink for serilog just to be sure, and switch out my sinks for my target environment!
I hope this helps.

That's what I did in then end. I mean just configured Seq as container and added Seq sink manually.
Thanks.


Shaka and not correct availabilityStartTime interpretation №2

Hi! Issue is reated to https://github.com/google/shaka-player/issues/1853
I made a test stream (link#1 sent to<EMAIL_ADDRESS>with 5-sec segments (gop=1 second) and burned-in timecode.
-encoder started at "Initial timecode: 08:53:45:01" (MSK timezone)
-first segment appeared Modify: 2019-06-11 08:53:50.000000000 +0300
-second segment appeared Modify: 2019-06-11 08:53:55.000000000 +0300
In MPD file i see availabilityStartTime="2019-06-11T05:53:55Z" (by standard it have to be 2019-06-11T05:53:50Z, it is encoder bug and fixed in newer versions, but anyway it does not matter now).
When i watch stream (link#1) then i see this timeline

I made an mpd file with changed availability start time (minus 10 seconds) (link#2) and now player timeline is correlated with burned-in timeline.

So to get correct time line we have to manipulate availabilityStartTime in mpd file or make some constant with player config to shift back timeline.
I want to understand if shaka doing all properly and if not will be it fixed?

I tried out the assets you sent, and for both of them the baked-in time is significantly off of the video's currentTime. Like, about 9-10 hours off. If you try again now, do you see a large time discrepancy like that? Or does the second stream look correct to you, still?

"Like, about 9-10 hours off. If you try again now, do you see a large time discrepancy like that? " - seems to be that some time zone issues. I see all correctly.

I tried setting my timezone to MSK. Now I see an even bigger time difference. I think I may have misunderstood what you are expecting.
At the moment, it looks like the burned-in time is the current time of day in the MSK timezone, correct?
And you are expecting the video's currentTime to be equal to the burned-in time, and thus the current time of day. But the video element's currentTime is the time since the broadcast began. So I'm not sure why your screenshots show 8:57:17 instead of the ~4 minutes it was since the broadcast began.
So I guess I should ask... What browser are you using? Do you have a custom UI with different time drawing behavior? That specific UI you are showing is kind of familiar looking, but I can't quite place it. Am I missing something obvious here?

"it looks like the burned-in time is the current time of day in the MSK timezone, correct?" - yes
"What browser are you using?" - ff, chrome latest
"Do you have a custom UI with different time drawing behavior?" - seems to be yes, i will try to find out more about it in several days

Here it is
var playheadTime = shakaPlayer.getPlayheadTimeAsDate()
return Math.floor(playheadTime / 1e3 - opts.liveTimeDelta)
where liveTimeDelta - is our constant 10 seconds

@theodab ?

availabilityStartTime represents the wall-clock time the live stream started.  So the presentation (or media) time of 0 is the same as the availabilityStartTime.  So:
video.currentTime = 726163;
availabilityStartTime = Date.parse('2019-06-11T05:53:55Z') / 1e3;  //<PHONE_NUMBER>;
currentTime = video.currentTime + availabilityStartTime;  //<PHONE_NUMBER>
timeAsDate = new Date(currentTime * 1e3);  // Wed Jun 19 2019 08:36:38 GMT-0700

But the burned in time shows 18:36:28.  So my guess is your encoder is off by 10 seconds.

No, encoder is ntp synchronised totally.
"availabilityStartTime represents the wall-clock time the live stream started" - not fully correctly:
From ticket https://github.com/google/shaka-player/issues/1853
"Segment availability start time
latest time instant in wall-clock time at which a Segment becomes an available Segment"
So when we use 5-sec segmens and begin to make mpeg-dash, AST = time of when first segment was writed totally and player can play from only ther second segment and time in time line is shifted ahead for 5 seconds, It means that shaka player takes AST for timing on timeline do it not so correctly, thats why we have to put some shift in config."
And from the beginning of this ticket:
"In MPD file i see availabilityStartTime="2019-06-11T05:53:55Z" (by standard it have to be 2019-06-11T05:53:50Z, it is encoder bug and fixed in newer versions, but anyway it does not matter now)"
Shaka interprets avaailablity start time not correctly, not by standard

The @availabilityStartTime attribute is not the same as the Segment availability start time.  The former is the anchor point specifying the start of the presentation (DASH table 3):

For @type='dynamic' this attribute shall be present. In this case it specifies the anchor for the computation of the earliest availability time (in UTC) for any Segment in the Media Presentation.

The Segment availability start time is the start time that a specific segment becomes available.  When a specific segment is available is when the live edge of the stream passes its end time.  (DASH section <IP_ADDRESS>.3):

For services with MPD@type='dynamic', the Segment availability start time of a Media Segment is the sum of

the value of the MPD@availabilityStartTime,
the PeriodStart time of the containing Period as defined in <IP_ADDRESS>,
the MPD start time of the Media Segment, and
the MPD duration of the Media Segment.

"In this case it specifies the anchor for the computation of the earliest availability time (in UTC) for any Segment in the Media Presentation."- does it mean to be the very beginning of the event start?
So it seems to be means encoder soft does not correctly put availabilityStartTime in mpd

availabilityStartTime is the wall clock in Z when the MPD manifest is made available and the first segment has been written.   The segment duration might not match the segment duration in the Dash Settings if the GOP aligned boundary doesn't match.  This can typically happen if the framerate, GOP, fragment length and segment length do not align.   You are probably seeing a 10 second difference because the duration of that segment was automatically adjusted and is not 5 seconds, but is now 10 seconds.  If you cat the init mp4 and first video mp4 and run media info you will probably see this.  Sometimes this happens if frame rate = follow input and the user isn't in control of the output framerate.  In this case the muxer will adjust as needed.  If you set the fragment lengh = 3, segment length = 6, gop =90 and frame rate = 30, this will produce 6 second segments.  The first segment is created exactly 6 seconds into the encode and the AST= wall clock when that first segment and first instance of the manifest was written.

It seems to be we speak about different things, as TheModMaker says AST and Segment AST has different meanings. And i just want to find out what is AST in manifest file. Now i think that it must be time of when encoding started. But in standard it is described not so obviously:
"For @type='dynamic' this attribute shall be present. In this case it specifies the anchor for the computation of the earliest availability time (in UTC) for any Segment in the Media Presentation."

AST is not when encoding started.  It is when the  first segment is available as it says in the last sentence of your message.
The segment start time refers to the start time of each arbitrary segment in the playlist.  Each segment has its own segment start time.
You are getting an AST of 10 seconds after the encoding started because the segment is 10 seconds long and isn’t available until the entire segment has been written.
Brian

On Jun 24, 2019, at 12:29 PM, kuznetcoff777<EMAIL_ADDRESS>wrote:
It seems to be we speak about different things, as TheModMaker says AST and Segment AST has different meanings. And i just want to find out what is AST in manifest file. Now i think that it must be time of when encoding started. But in standard it is described not so obviously:
"For @type='dynamic' this attribute shall be present. In this case it specifies the anchor for the computation of the earliest availability time (in UTC) for any Segment in the Media Presentation."
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@dudeskidaddy if you are right about "AST is not when encoding started. It is when the first segment is available as it says in the last sentence of your message.", then shaka does not interpretate correctly availabilityStartTime. But @TheModMaker says that AST and Segment AST is different things.
Look at my issue description carefully, i described everything with details.

@dudeskidaddy As I stated in my comment above, the AST is just used as a reference point.  It doesn't have to be anything.  It is just used to determine availability and to align the presentation timeline with the wall clock.
The AST is the same as presentation time 0.  So a segment starting at 30 seconds has a wall-clock time of AST + 30.  It doesn't matter what the value of AST is so long as it is consistent with the presentation timeline.


GH-1912 User defined SPARQL queries in SHACL constraints and targets

GitHub issue resolved: #1912
Briefly describe the changes proposed in this PR:

PR Author Checklist (see the contributor guidelines for more details):

[ ] my pull request is self-contained
[ ] I've added tests for the changes I made
[ ] I've applied code formatting (you can use mvn process-resources to format from the command line)
[ ] I've squashed my commits where necessary
[ ] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

Renamed branch


Cargo format

I ran cargo fmt to get up to date formatting.
I suggest we merge this after the clippy branch has been merged (and this one has been rebased)

Because between the origin of this branch and the current state of the repo a lot has changed. Mainly a bazillion clippy errors that have been removed. These are gonna conflict badly with touching every other line with the formatter :stuck_out_tongue:. We basically have to run fmt again on the new code, force push and merge.

Haha, OK, fair enough - let's see if I can do that


[7.0.1] Delete MODULE.bazel.lock

Forked from #446

Cherry-picked in https://github.com/bazelbuild/bazel/pull/21318


Enable MLIR by default in migraphx builds

what's the purpose behind having it enabled by default ?
I see that we already have Jenkins job that builds MIGraphX with MLIR.

The tidy error requires a fix in rocm-cmake: https://github.com/RadeonOpenCompute/rocm-cmake/pull/140


Fixed #15826 : JavaScript compareValidator for attributes not in rules

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#15826

Thanks!


Update Probe-Matching-Verification_FULL.csv

Updated the patchlist to 95% stimulus to align with recent ColourSpace changes for probe matching.

Cool @BlackJoker90!
Would you care about updating the HOWTO.md (https://github.com/peschee/probe-match-verifier/blob/main/public/HOWTO.md) as well?

Done 😎


proposal: add a handful of acquire/release atomics (make the internal runtime versions public)

Proposal Details
I asked for this a decade ago, and I want to start this discussion up again.
I'm not saying Go should add the full gamut of relaxed, acquire, release, etc orderings available in other languages.
But I am saying it's useful to have some basic acquire and release loads and stores for 32bit, 64bit, and uintptr.
To see that this is useful one need look no further than the Go runtime which use exactly these.
uintptr:
internal/runtime/atomic.LoadAcquintptr
internal/runtime/atomic.StoreReluintptr
uint32:
internal/runtime/atomic.LoadAcq
internal/runtime/atomic.StoreRel
uint64:
internal/runtime/atomic.LoadAcq64
internal/runtime/atomic.StoreRel64
I would like that we make these 6 functions available in the atomics package and a LoadAcq and StoreRel on each of the atomic types. I can make a PR.
It's not that these are so advanced and so dangerous that only Go runtime authors should have them. We deserve advanced tools too. For a language with concurrency as a strength, it's lacking in some important primitives to develop performant concurrent data structures. And it's not that we can't have that in Go, it's already implemented, it just isn't public.

Related Issues

proposal: sync/atomic: add more relaxed atomics #35639 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

As far as I can tell the current *Rel and *Acq versions only differ from the standard versions on ppc64.
Is that your understanding as well?

My understanding is the the standard Go atomic promise sequential consistency, which require a lock prefix or fence on x86/64 processors. The documentation states "This definition provides the same semantics as C++'s sequentially consistent atomics". Since x86 mov instructions have acquire/release semantics, you need a fence or stronger instruction (I believe it's common to abuse xchg instruction for this).
I could do some digging in the Go internals if I had an idea where to look.

The intrinsic are only implemented for PPC64:
https://github.com/golang/go/blob/caee788a48f19814bd778c1bd2422cb6f60ad810/src/cmd/compile/internal/ssagen/intrinsics.go#L243-L256
https://github.com/golang/go/blob/caee788a48f19814bd778c1bd2422cb6f60ad810/src/cmd/compile/internal/ssagen/intrinsics.go#L289-L300
Looking through the .s and .go impl files here are the edge cases:

Load Aqcuire on amd64 is implemented without atomic instructions, but so does any atomic load on amd64 because of it's strong memory model.
wasm's impls use non atomic go primitives, but it also lack SMP.

note: the compiler internally use theses functions calls to disable certain optimizations and various other things, because the implementation of atomic uses non atomic operators does not mean it is safe to do so outside of theses exact functions.
If anything I guess this is because no one found a compelling use for theses, we can implement relaxed store and acquire store on most architectures people would care about if this proposal required it, for others fallback to sqcst impls as we currently do.

Interesting, so go just uses a plan mov on x86/64 for atomic load, which actually is the same as what C++ does.
It implements the Store with XCHG, which is also what C++ does.
So then the question is, if we implement the release store with mov instead of xchg on x86/x64, does it matter?
Go has an extensive benchmark suite, and it uses StoreRel in some parts of the runtime. So I guess we could test this for the Go runtime pretty easily.
It'd be nice to test it for a synthetic benchmark too, to see how much of a difference there is in theory. I could start with that.

I created a very simple benchmark in Rust, so I can have different levels of atomics, to see if the difference between SeqCst and Release stores matter at all, on x86. This is on an Intel Core™ i9-14900HX, the difference will matter more on multi-socket systems and more relaxed architectures.
The best time for sequential consistency was 9.925s, the best for release store was 0.452s. Roughly a 20x difference in performance on this very contrived microbenchmark. The timings vary depending which cores I pin it to (this CPU has both E cores and P cores), but the gulf between them remains wide.
I think that proves the difference will matter for some problems, and that in itself is enough to convince me that it would be good to have support for weaker atomics in Go.


yahm-network -w create_bridge Ganzzahliger Ausdruck erwartet

Trying to create new bridge yahmbr0 with physical interface eth0
ATTENTION: Network configuration will be changed, you may no longer login to the system!!!
No bridge with choosen name found, continue...
Interface eth0 is physical and valid, continue...
Found manual interface configuration on eth0
Preparing network interfaces file
Creating new interface yahmbr0
Please review new network configuration
# interfaces(5) file used by ifup(8) and ifdown(8)

# Please note that this file is written to be used with dhcpcd
# For static IP, consult /etc/dhcpcd.conf and 'man dhcpcd.conf'

# Include files from /etc/network/interfaces.d:
source-directory /etc/network/interfaces.d

auto lo
iface lo inet loopback

iface eth0 inet manual

allow-hotplug wlan0
iface wlan0 inet manual
    wpa-conf /etc/wpa_supplicant/wpa_supplicant.conf

allow-hotplug wlan1
iface wlan1 inet manual
    wpa-conf /etc/wpa_supplicant/wpa_supplicant.conf

auto yahmbr0
iface yahmbr0 inet manual
    bridge_ports eth0
WARNING: Writing network changes. To cancel this operation type CTRL+C you have 5 seconds...
... to late ;)
Backuping /etc/network/interfaces to /etc/network/interfaces.1464638853
Writing changes to network configuration
WARNING: Your system is configured now, restarting network to apply the changes. You may get new IP address from DHCP
. To cancel this operation type CTRL+C you have 5 seconds...
... to late ;)
/usr/sbin/yahm-network: Zeile 399: [: is_static: Ganzzahliger Ausdruck erwartet.
Clean up

ist gefixt danke ;)


Cannot install on Windows 10

I've tried to install 1.3.12 library version on windows 10 but the installation gives the following error:
fatal error C1047: The object or library file 'C:\Users\guidr\AppData\Local\Temp\pip-install-vb4p8xkh\xmlsec_c83a2796c6b84c61ae947515cf2ad097\build\tmp\libs\iconv-1.14.win64\lib\iconv_a.lib' it was created with a different compiler version than other objects, for example 'build\temp.win-amd64-cpython-310\Release\Users\guidr\AppData\Local\Temp\pip-install-vb4p8xkh\xmlsec_c83a2796c6b84c61ae947515cf2ad097\src\constants.obj'; Rebuild all objects and libraries with the same compiler
LINK : fatal error LNK1257: code generation failed
error: command 'C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.32.31326\bin\HostX86\x64\link.exe' failed with exit code 1257
Anyone could help me for this issue?
Thanx

The 1.3.13 has prebuilt windows wheel for Python 3.10, try those instead.


Vanilla bunny and bee suits have floating headbands when worn by dog model players

I don't want to support MoreCompany cosmetics on dog bodies since there's so much variability with the cosmetics, but I am interested in supporting the vanilla suits!

Write postfix for UnlockableSuit.SwitchSuitForPlayer

Hmm I looked into adjusting the headbands to sit on the dog model, but the way that they are being positioned makes it quite clunky to update their position on the fly.
For example, I can adjust their position when the suit is changed, but then what about when the player toggles between human and dog? Then I would have to update the position of the cosmetics for each toggle, making sure to remove the cosmetics from the previous position. And I think that tulip snakes also attach to the same head container, so I would have to be careful not to accidentally delete the tulip snakes when doing this... It seems a little annoying and fiddly.
It's a lot easier to just hide them again, like with the MoreCompany cosmetics. That's unfortunately the solution I'm landing on since I don't figure that too many people are using the bunny/bee suits anyways!


Linear Guassian blur effect

@XeduR and I were discussing the current Gaussian blur effect, which can be pretty expensive, especially on many mobile platforms. I implemented the idea from this RasterGrid post and it seems to work quite well, albeit with slightly different characteristics.
The stock Gaussian effect is able to do quite a large loop and must be able to branch out of that as well, probably explaining the cost in large part. This variant, on the other hand, will do five samples on each directional pass, all straight-line code.
The new effect is available as "filter.blurGaussianLinear".
I'm not sure how useful it is, but I made the offsets available as parameters offset1 and offset2, defaulting to those listed at the end of the article.
Here's the test project.
I'm having a little difficulty at the moment working out the incantations to launch iOS and tvOS test builds, but confirmed it works on Windows, Mac, and Android. I've not yet built anything on web or Linux, but also made an attempt to put the new files in place in their respective projects, from eyeballing what other effects are doing.

TODO:

[ ] add docs: https://github.com/coronalabs/corona-docs/blob/master/markdown/guide/graphics/effects.markdown
[ ] add to samples: https://github.com/coronalabs/samples-coronasdk


Issue/5558 local auth login

Description
Recreated to create clean view as the flow got changed (closed PR #5599)
closes #5558
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[ ] Changelog entry
[ ] Code is clear and sufficiently documented
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Processing this pull request

Pull request rejected by merge tool. The tests for this branch did not succeed.

Processing this pull request

Merged into branches iso7, master in 6c284970cfe126f85ef08daee5612052de0bb2ae


Provide a transpiled version

It would be nice if you could provide a transpiled dist version and
point your main to it, for your module won't be included by laravel-mix
when in development  environment.
Jeffrey Way, the author of laravel-mix, explicitly excludes all modules
residing in node_modules to speed up development/watch builds.
Thanks in advance
Sam

I second this. I don't use laravel-mix, but symfony encore witch is using webpack 3. In order to use this component I had to load webpack-uglify-js. Or to tell babel to transpile this module (currently cant find how to do this) to ES5.

Thanks for the proposal, you can update to 2.0.0. Injoy and welcome more suggestion.

Thank you @bsdfzzzy works lika a charm 👍


More cleanup for failures on missing commands.

Currently in podman if a user specifies a command that does not exist
the tool shows the help information.  This patch changes it to show
information like:
$ ./bin/podman foobar
Error: unrecognized command 'podman foobar'
Try 'podman --help' for more information.
$ ./bin/podman volume foobar
Error: unrecognized command podman volume foobar
Try 'podman volume --help' for more information.
$ ./bin/podman container foobar
Error: unrecognized command podman container foobar
Try 'podman container --help' for more information.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

@baude @mheon @edsantiago @TomSweeneyRedHat @giuseppe @vrothberg PTAL

Closes:https://github.com/containers/libpod/issues/2530

Okay... I have no idea why, but am throwing this out in case it's obvious to anyone: this change slightly breaks the Usage format:
before:
    $ podman system --help
    Usage:
        podman system [command]
after:
    Usage:
      podman system [flags]
      podman system [command]

This breaks all the scripts I've written that rely on '[command]' to indicate 'this is a podman command with further subcommands'. I can find a workaround, I'm sure, but first: does anyone understand why this is happening, and is there an easy fix for it? (my guess is that if {{.Runnable}} in common.go is now triggering, but what tdo I know).
Oh, and I think you missed pod.go. (Found by my recursion scripts)

Update: this edit to common.go resolves the double-usage-line issue without, I think, introducing any unexpected behavior (but I'm still checking):
--- a/cmd/podman/common.go
+++ b/cmd/podman/common.go
@@ -525,7 +525,7 @@ func scrubServer(server string) string {
 // This blocks the desplaying of the global options. The main podman
 // command should not use this.
 func UsageTemplate() string {
-       return `Usage:{{if .Runnable}}
+       return `Usage:{{if (and .Runnable (not .HasAvailableSubCommands))}}
   {{.UseLine}}{{end}}{{if .HasAvailableSubCommands}}
   {{.CommandPath}} [command]{{end}}{{if gt (len .Aliases) 0}}

This is really exciting. Thank you for finding a way to address this.
I hope you'll accept my refactored PR (and the warning comment therein)

@rhatdan I need to head out soon but just want to make sure you saw my refactoring PR? There's a lot of unnecessary duplication here, and also a serious index dereferencing bug (which can be fixed in only one place, if you agree with my PR approach)

Sure lets finish this up tomorrow, Hopefully your idea works.

/retest
bot, retest this please

/lgtm
I'd still encourage you to consider a more descriptive name than commandRunE, but otherwise this is really great and very much appreciated. Thank you.


[2.0-wip] Pager

Add hover effect.

Refined the initial styles and improved the hover effect. Thanks!


Extend the Time Delay grep type with more options

Now, the GrepTypeTimeDelay is verified with the function defined at https://github.com/BountySecurity/gbounty/blob/main/internal/match/match.go#L216-L220.
However, it would be very nice if we could extend that support to not just apply a static margin of 2s, but to let the user determine if they want the response time to be greater/lower of the given value, between two values, with a different margin, etc.
Specific keywords are TBD.

Hello @joanlopez, any news here?


bip32 HierarchicalKeyFactory::fromExtended xpub xprv error

Hello,
When I uncomment the following line
//$master = HierarchicalKeyFactory::fromExtended("xprv9s21ZrQH143K4Se1mR27QkNkLS9LSarRVFQcopi2mcomwNPDaABdM1gjyow2VgrVGSYReepENPKX2qiH61CbixpYuSg4fFgmrRtk6TufhPU");
from this example:  https://github.com/Bit-Wasp/bitcoin-php/blob/master/examples/bip32.php
I receive this error:
PHP Fatal error:  Uncaught Error: Using $this when not in object context in /home/bip32/bitcoin-php/src/Key/Factory/HierarchicalKeyFactory.php:93 Stack trace: #0 /home/bip32/bip32.php(14): BitWasp\Bitcoin\Key\Factory\HierarchicalKeyFactory::fromExtended('xprv9s...') #1 {main}   thrown in /home/bip32/bitcoin-php/src/Key/Factory/HierarchicalKeyFactory.php on line 93
How to fix it and use xpub or xprv key?
Thanks

Found the answer, must  be $hdFactory->fromExtended


Minor edits to Cloud Check Docs

fixed some missing words / typos
re-phrased some long sentences

Thanks!


CARLA takes forever to start when run from a network folder

Is this still happening?

I don't think we have ever tried again, this was reported by @dosovits.

I'm going to close this since we cannot replicate. Please reopen if you still observe this.


Message Analyzer is an effective tool for troubleshooting network issues.

"Message Analyzer is an effective tool for troubleshooting network issues."
But Microsoft Message Analyzer (MMA) is being retired and its download packages removed from microsoft.com sites on November 25 2019.

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

ID: b27f7338-33e6-d006-2038-068767ec0ea3
Version Independent ID: f82bca63-1a2e-4cdf-6564-9eb6d281b892
Content: Network and migration planning for Office 365
Content Source: Enterprise/network-and-migration-planning.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@richard-artes -  Can you please share what specific issue you have with this doc? Or are you just giving a general user feedback?
If you have some issues with this doc, we would really appreciate if you are able to help by providing more details as to what should be edited and/or included.
Thank you!

Yes, references to Microsoft Message Analyzer are no longer relevant and should be removed. If you click on the link to https://technet.microsoft.com/library/jj649776.aspx then you see a message saying this tool is no longer supported. The tool is no longer available for download so there is no point in mentioning it in the article. The whole sentance should be removed:
: Message Analyzer is an effective tool for troubleshooting network issues. Message Analyzer captures, displays, and analyzes protocol-based messaging traffic and system messages. Message Analyzer also lets you import, aggregate, and analyze data from log and trace files.

@richard-artes - Thank you for submitting feedback.
From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/OfficeDocs-Enterprise/commit/e1116229cb2a8ee2ede2e3e06204fc1d1ddf19c6.
Thank you for your contribution to make the docs better! Much appreciated!


Is v1.0.7 the latest release?

Hi! Homebrew maintainer here. We noticed that v1.0.7 was tagged, but is not yet marked as latest on the repo's releases page. I'm filing this issue to confirm if the release is ready to be marked as latest, so that we can go ahead and ship it. Thanks!

v1.0.7 only updated the API, and the terminal tool has not changed, so homebrew package does not need to be updated.
Thanks for asking

发件人: Ruoyu Zhong @.>
发送时间: 2023年4月9日 13:59
收件人: HDT3213/rdb @.>
抄送: Subscribed @.***>
主题: [HDT3213/rdb] Is v1.0.7 the latest release? (Issue #14)
Hi! Homebrewhttps://github.com/Homebrew/brew maintainer here. We noticed that v1.0.7 was tagged, but is not yet marked as latest on the repo's releases pagehttps://github.com/HDT3213/rdb/releases. I'm filing this issue to confirm if the release is ready to be marked as latest, so that we can go ahead and ship it. Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/HDT3213/rdb/issues/14, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADCFRKWHMUIU5RAT3RY7D7LXAJF43ANCNFSM6AAAAAAWX473QA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks @HDT3213. However, in that case, I don't see any reason against shipping this latest release. (Although the CLI tool will probably be the same.) So we decided to update rdb to 1.0.7 anyway.


Move MiqApache from manageiq-gems-pending

MiqApache is only used by the "evmserver" parts of MiqServer, so it
should live here.
@carbonin Please review.

gems/pending removing in https://github.com/ManageIQ/manageiq-gems-pending/pull/235

keep this PR a straight move

Updated the brakeman.ignore


Disable autocomplete in column filters

What about disable autocomplete on column filters fields?

[x] Enhancement Request

@billmn not sure what you mean... there's no autocomplete on filters fields. you sure you're not talking about chrome's autocomplete?

Yes, I’m talking about browser’s autocomplete. Sorry, I didn’t specify it.

hey @billmn, just out of curiosity how would you stop the browser based autocomplete on inputs? I've been researching a bit and haven't found a reliable way, autocomplete="off" is ignored now by chrome too. Have you done this before?

Yep, autocomplete="off" has been the standard for years ... but seems there's a bug regression on Chrome: https://bugs.chromium.org/p/chromium/issues/detail?id=914451
Seems that autocomplete="new-password" does the trick


PyList::as_tuple() -> PyList::to_tuple()

As discussed as a follow-up to #3042

(I also simplified the inner implementation a bit.)

bors r=mejrs


Docs updates

This PR

Updates the Documentation

Transform.rotation. Fixes #5020
BaseSoundManager etc. (#4953)
Arcade.Group, Arcade.StaticGroup
Vector2#normalizeRightHand
Easing (#5062)

These are some nice looking updates to the docs, thank you @samme. Post here when you're happy with this branch and I can test them against the docs website and get them merged.

@hexus you can go ahead with these, thanks.


goflow2: 1.3.4 -> 2.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/netsampler/goflow2/releases.
meta.description for goflow2 is: High performance sFlow/IPFIX/NetFlow Collector
meta.homepage for goflow2 is: https://github.com/netsampler/goflow2
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.0.0 with grep in /nix/store/p5d67frnz5zzll4hzn4k4dxcc5gr4mq7-goflow2-2.0.0
found 2.0.0 in filename of file in /nix/store/p5d67frnz5zzll4hzn4k4dxcc5gr4mq7-goflow2-2.0.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
goflow2

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/p5d67frnz5zzll4hzn4k4dxcc5gr4mq7-goflow2-2.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A goflow2 https://github.com/r-ryantm/nixpkgs/archive/cbe3b1e50dcc9bf59382f045557d11ab27683f1d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/p5d67frnz5zzll4hzn4k4dxcc5gr4mq7-goflow2-2.0.0
ls -la /nix/store/p5d67frnz5zzll4hzn4k4dxcc5gr4mq7-goflow2-2.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

goflow2

Maintainer pings
cc @yu-re-ka for testing.

Result of nixpkgs-review pr 248649 run on x86_64-linux 1

1 package built:

goflow2

@onny for future reference: please don't merge breaking changes without consulting with the maintainer. In this case I still need to keep around the 1.x version.


Path jump problem

issue: I am using version 1.6.5, jumping from http to https, the following issue
Bug:
When I visit http:/test.a.com/ui/dashboard/, I jump to http:/test.a.com/dashboard/ , which causes page 404, which removes /ui from my path. How to solve it?
Ingress:
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
annotations:
kubernetes.io/ingress.class: traefik
traefik.backend.circuitbreaker: NetworkErrorRatio() > 0.5
traefik.frontend.entryPoints: http,https
traefik.frontend.passHostHeader: "true"
traefik.frontend.rule.type: PathPrefixStrip
traefik.ingress.kubernetes.io/frontend-entroy-points: http,https
traefik.ingress.kubernetes.io/load-balancer-method: drr
traefik.ingress.kubernetes.io/preserve-host: "true"
traefik.ingress.kubernetes.io/rule-type: PathPrefixStrip
traefik.protocol: https
name: traefik-ingress-web
namespace: kube-system
spec:
rules:

host: test.a.com
http:
paths:

backend:
serviceName: traefik-ingress-service
servicePort: admin
path: /ui
backend:
serviceName: api-gateway
servicePort: api-gateway
path: /api
backend:
serviceName: traefik-ingress-service
servicePort: web
path: /

host: test.b.com
http:
paths:

backend:
serviceName: discovery01
servicePort: eureka1
path: /

This has been fixed in https://github.com/containous/traefik/pull/3631 and should released soon in 1.7.0 (you can try the fix in 1.7.3-rc3).
Original issue: https://github.com/containous/traefik/issues/1957

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Træfik!
We dedicate the issue tracker to bug reports and feature requests only.
If you have questions and need community support, you can join our Slack workspace and reach out to us on the #support channel.
Another option is to use Stack Overflow and tag your question traefik.
Of course, double checking Træfik's documentation is always a good idea :sweat_smile:


Different string obfuscation

I don't remember if I already suggested it, but there is another fancy way to mess with strings. Currently I cannot afford using any of the string methods since they all use a patchable functions. But I found alternative! I split the functions by chars with obfuscator.io
"example" -> "e"+"x"+"a"+"m"+"p"+"l"+"e"
and then I apply options that stores all primitives in array.
It turns out pretty confusing to me and without patchable functions :p

Still weak as deobfuscators can undo pretty easy. Safe functions are planned, potentially with #152


Parallel test hang

This week's recurring test suite failed - looks like it hung on one of the parallel tests and timed out after almost 6 hours.
test/test_adapt.py::test_adapt_parallel_3d_np3[False]

https://github.com/mesh-adaptation/animate/actions/runs/9239789581/job/25419352820

It looks like https://github.com/mesh-adaptation/animate/actions/runs/9282388099/job/25540301333 was hanging again on the same test so I cancelled it.

More recently we have
*** buffer overflow detected ***: terminated

 Unexpected error:  *** potential lack of memory.

Looks like there's a memory leak.


No full-screen function on companion app

Did non have a chance to test it on android. Full-screen function working fine on web interface, button is missing on companion app, while sound and play/pause buttons are present and working

I have the same problem with IOS, with ui: true
There is no full screen button.

even unable to unmute. it gets muted again if i try to unmute it

@ammaralfarsi same for me. Workaround in my case is to hold the mute/unmute button for few seconds.

@ammaralfarsi same for me. Workaround in my case is to hold the mute/unmute button for few seconds.

Just tried it and it works when i hold if for few secs.
Thx

Updated to HA 2022.7 and WebRTC 2.3.1. Cameras work fine with full-screen option on iPhone until I set ui: true
Mute on/off does not work even if I press for seconds.

For me fullscreen still not working.
Home Assistant 2022.12.6
WebRTC Camera 2.3.1

Unfortunately, the iPhone has the worst browser in the world today.
It does not support a lot of modern web technologies. Including MSE and Full screen:
https://caniuse.com/mediasource
https://caniuse.com/fullscreen
PS. No problem on desktop Mac or iPad. Only iPhones...

Unfortunately, the iPhone has the worst browser in the world today. It does not support a lot of modern web technologies. Including MSE and Full screen: https://caniuse.com/mediasource https://caniuse.com/fullscreen
PS. No problem on desktop Mac or iPad. Only iPhones...

Hi, I have very same problem also on iPad. When "ui: true" all buttons are gone. I can see them just for short time before the video is loaded.
Home Assistant 2023.1.1
Supervisor 2022.12.1
Operating System 9.4
Frontend 20230104.0
WebRTC-3.0-beta.3
iPadOS 16.2
Home Assistant app on iOS 2022.10.1

Full screen not showing for me on ios iphone 13 pro, ios 16.1.2
controls never show, not when UI: true is set, and not when it is not set
webrtc version: v3.0-beta4
ued to work in non v3 versions

The iPhone browser does not support this feature

The iPhone browser does not support this feature

WebRTC cards in iPhone Safari do show the normal video controls (play, stop, Airplay, full screen), and native Home Assistant video streams in the iPhone companion app also have the video controls, so videos can show the controls. I don't know off the top of my head what the requirements are for doing so, though.

@jason0x43 problem with custom UI. When native controls disabled

Odd that the issue only seems to apply to the iPhone companion app, and not the iPad companion app or Safari on any device.
Interestingly, with WebRTC v3.0.0 I get the custom UI (the PTZ controls, at least) and the native controls in the iPhone companion app, and all of them work.

@MStcz custom UI should be fixed in latest release

@MStcz custom UI for iPad should be fixed in latest release

Hi @AlexxIT, thank you for the update. I can see custom shortcuts, speaker button at the bottom right but not full screen button at the bottom left. Tested on iPad with "ui: true".
Home Assistant  2023.1.7
Supervisor 2023.01.1
Operating System 9.4
Frontend Frontend 20230110.0
WebRTC v3.0.1
iPadOS 16.2
Home Assistant app on iOS 2022.10.1

I saw that this issue is closed but wanted to advise that the Full Screen Button is missing from iPAD IOS 16.2.  Iam running WebRTC version 3.0.2.  The Full Screen button appears and works on a PC web browser.
Home Assistant 2023.2.0
Supervisor 2023.01.1
Operating System 9.5
Frontend 20230201.0 - latest

Just for comparison, I'm also running HA 2023.2.0 and WebRTC 3.0.2, and the full screen button does show up on my iPad running iOS 16.2.

I cleared the cache in the companion app and that did not fix the issues.
Do you have "ui: true" in your webRTC card.  I do and the full screen indicator [ ] shows up in the bottom left corner of my PC but nothing for the app. Maybe that is the difference.   If I set "ui: false" then I see the arrows that allow me to stretch  the screen to full but then other items appear that I do not wish to see.

I have ui: true, and if I tap the video I see both the native controls and the WebRTC UI (the PTZ controls, at least).

I have ui: true, and if I tap the video I see both the native controls and the WebRTC UI (the PTZ controls, at least).

It is strange. I am able to achieve very same result as you but just with ui: false. In that case each time I open the stream can see  pause icon in the middle of the screen and partial opacity over the whole screen - I don 't like it. That 's why I prefer ui: true but there is no full screen mode button/icon.
iPad ui: false after click:

Zive Vysilani = Live Broadcast
iPad ui: false:

iPad ui: true:

Laptop ui: true:

Laptop ui: false:

Would it be possible to have an option of "ui: supported" and not set custom UI for iOS browsers?


feat(v2): Allow configuring babel via docusaurus.config.js

Motivation
Close #2772.
Implementation
Unfortunately, we cannot exactly follow the approach by next.js, since docusaurus's existing babel config is not static: it's behavior is determined by isServer. Therefore, there is no way that we can let user specify a .babelrc or even babel.config.js without breaking the babel config semantics, since the isServer logic is unknown to Babel.
The above constraint suggests that we must at least use a function of type (isServer: boolean) => BabelConfigObject to configure babel. You might think getBabelLoader is a good candidate that can be used out-of-box, but it's not true. getBabelLoader merges in the second optional babelOptions is a dumb way. (e.g. presets in babelOptions won't be merged with existing presets, but simply replace them.) It's possible to write our own logic to handle that, but there is a better way to leverage the existing config merging mechanism: babel preset.
Therefore, I moved the main content of getBabelLoader to docusaurus-core/src/babel/base, and separately exposed client preset and server preset in two different files. This would allow user to start with our preset like this:
  configureBabel: (isServer) => ({
    presets: [
      isServer
        ? require.resolve('@docusaurus/core/lib/babel/server-preset')
        : require.resolve('@docusaurus/core/lib/babel/client-preset'),
    ],
  }),

Then I exposed the configuration API in docusaurus.config.js
Potential Concern
This approach breaks the current implicit contract that docusaurus.config.js contains a plain JS object. If we really want that, we can do something like customBabelClientConfig and customBabelServerConfig, but it looks more ugly and less maintainable.
Another approach is to expose it as a lifecycle API in plugins. However, it might make the feature less discoverable and create a steeper learning curve for people who want to customize babel configs.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
No custom babel config
The current repo doesn't use custom babel config, and the preview site can still build.
With custom babel config

yarn workspace docusaurus-2-website add @babel/preset-flow
Add the following code to the end of docusaurus.config.js:

  configureBabel: (isServer) => ({
    presets: [
      '@babel/preset-flow',
      isServer
        ? require.resolve('@docusaurus/core/lib/babel/server-preset')
        : require.resolve('@docusaurus/core/lib/babel/client-preset'),
    ],
  }),

Add the following code with flow specific syntax to the top of website/src/pages/index.js

// @flow

type FlowTypeExample = { optional: ?string };

const a: FlowTypeExample = {};
const b: FlowTypeExample = {optional: ''};

yarn start, and observe that the site can correctly load

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Thanks Sam, will leave to @slorber to review!

Hi @SamChou19815
Your current solution works but I'm not totally fan of the API.

Unfortunately, we cannot exactly follow the approach by next.js, since docusaurus's existing babel config is not static: it's behavior is determined by isServer. Therefore, there is no way that we can let user specify a .babelrc or even babel.config.js without breaking the babel config semantics, since the isServer logic is unknown to Babel.

Actually, I was surprised by what you said here, as I've seen many isServer or isProduction things in many fwk codebases (Next, Gatsby, Expo, RN/Metro...) codebase.
I think the approach used by Next is suitable for our usecase. Here's what I would try:

Create a Docusaurus preset

Use the caller api to know if the preset is used in server/client mode, dev/prod mode, and create the appropriate babel config.
  const isServer = api.caller((caller: any) => !!caller && caller.isServer)
  const isDev = api.caller((caller: any) => !!caller && caller.isDev)

Next does this: https://github.com/vercel/next.js/blob/31b3e46b8f08f3ae8c16d95f4379f3129324a9be/packages/next/build/babel/preset.ts#L66
Alternative, the babel loader already injects a caller.target === "node" / "web" so that you know if isServer is true in the preset
https://github.com/babel/babel-loader#customize-config-based-on-webpack-target

Use the preset in the Babel webpack loader

The loader option should tell the Docusaurus preset in which mode it is
      babelLoaderOption.caller.isServer = isServer
      babelLoaderOption.caller.isDev = development

Next does this:
https://github.com/vercel/next.js/blob/canary/packages/next/build/webpack/loaders/next-babel-loader.js#L136

Add a .babelrc with the Docusaurus preset:

{
  "presets": [
    [
      "docusaurus/babel",
      {presetOptions: 42}
    ]
  ],
  "plugins": ["my-other-babel-plugin"]
}

This is the public api that most users would expect.
Next does this:
https://github.com/vercel/next.js/blob/master/docs/advanced-features/customizing-babel-config.md

I think we could by default add a babel.config.js with this new Docusaurus preset on every new D2 project. For retrocompatibility we could default to a config with just the preset.
What do you think @SamChou19815 @yangshun @lex111 ?

I've also looked at other solutions

Gatsby has a dedicated lifecycle api to update the webpack/babel config directly at the plugin level. Probably more advanced than we want here.
The Gatsby setup is quite complicated, with multiple babel stages (one for example is dedicated to extract the static queries from the code).
It does not seem to use the caller api but rather build the config and then call the different babel stages one by one, with custom preset options.
Also, in my experience trying to use a .babelrc in it, it didn't work great ^^
https://github.com/gatsbyjs/gatsby/issues/21157
Not sure we should be inspired by Gatsby for these reasons.

Expo
Expo uses the  caller api so that the preset can know for which platform it builds (ios/android/web) and adapt the preset accordingly
https://github.com/expo/expo-cli/blob/master/packages/webpack-config/src/loaders/createBabelLoader.ts#L204
https://github.com/expo/expo/blob/master/packages/babel-preset-expo/index.js#L8

@slorber updated my implementation!

Do you think it could be useful to publish the preset as a separate package? I don't think it would be useful, but others are doing this so... don't know the reasons

It's definitely easier to keep it within @docusaurus/core. Next.js also doesn't publish its preset separately.

@slorber feel free to merge it, you're also a maintainer and you're familiar with the code already :)

I'm afraid to forget squashing now ;)


fix(batch): always use pure epoch in query

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
The meta backup test failed because it used an arbitrary epoch in batch query over backup and failed an assertion in storage layer debug_assert!((epoch & EPOCH_SPILL_TIME_MASK) == 0);
This PR ensures batch query always uses a pure epoch.
#15600
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

There's another issue found in the failed meta backup test: the failed query is stuck rather than responding with error. Will look into it later.


vNext - feat:Scissors Rectangle Tool

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
New feature Rectangle Scissors Tool for segmentation data.

What is the current behavior? (You can also link to an open issue here)
Does Not Apply.

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Does Not Apply.

Other information:

Moved to the main Scissors PR.


Feature: tab indentation

For the purposes of programming:

The tab key should increase the indentation level
The shift+tab combination should decrease the indentation level

I'm not sure this is feasible without replacing the standard browser component for editing.


Add a way to create elements of elements

Right now there is a chance the subj won't work.

Not sure we need this ATM, so I'm closing until I'd really need it.


separator option for .js concat

Hi, just made a small mod to respect the separator option passed to your task. Tested and working. Hope you merge, it'll save me from fixing sloppy history.js termination by hand on every freaking project ;)

Thanks! Could you also update the docs?

Ok, I cleaned up the verbosity and declared default option value, updated the Readme, ran the linter, and added a test case. Should be good to go. Thanks!

Thanks!

You bet. Thanks for writing the module :)


DXF parsing and drawing

Hello! Is there an example of WinForm or WPF program parsing DXF elements and drawing?

No, all available samples are in the TestDxfDocument project.

Thanks!


Initial setup in HA vs Simulator

Hi Gazoodle! Thanks for creating the component.
Unfortunately, I do not get it working in my HA setup (newest HA version). It does not discover the SPA, I just get one visual component called "Gecko Update", and nothing else is discovered.
I did check your geckolib project and run the commands in the shell on the same host the HA is running, and I am able to see the SPA being discovered and giving correct output with temp. etc in the python console.
I tried to installed HA on raspberryPI on docker, doing all setup with hacs and your repo there, and the result is the same. Currently, HA is running on VM, new setup, same behaviour. I got latest version from GitHub both on main and latest release branch.
Logging enabled, not producing any logs for Gecko component.
Unless this is related ?
00:55:11 – (FEL) /usr/local/lib/python3.9/site-packages/aiohttp/web_protocol.py - Meddelandet inträffade först 00:54:18 och har hänt 2 gånger
Thanks for any advice,
Dariusz

00:55:11 – (FEL) /usr/local/lib/python3.9/site-packages/aiohttp/web_protocol.py - Meddelandet inträffade först 00:54:18 och har hänt 2 gånger

I don't think that is related, the web_protocol.py is not used by the geckolib library.
Did you try using the specific IP address to try to discover the spa that way?

hi,
i am having the same issue.
Fresh HA, fresh HACS, latest Gecko plugin and i only see the update function.
And i had half a year ago a working setup. So its not the Gecko Hardware.
Do you think it might be a DNS issue?
br
Michael

i did connect via terminal and using GeckoShell.run, that shows me the name IP and all looks correct.

ok, i found the reason:
On HA 8 when you install Gecko it automatic adds the Gecko update to the integrations, but you still need to add the integration to have it fully available

hi, i am having the same issue. Fresh HA, fresh HACS, latest Gecko plugin and i only see the update function. And i had half a year ago a working setup. So its not the Gecko Hardware. Do you think it might be a DNS issue?
br Michael

Great it was not just me:)

ok, i found the reason: On HA 8 when you install Gecko it automatic adds the Gecko update to the integrations, but you still need to add the integration to have it fully available

Cool, I will give it a go first with the shell. Can you explain a little bit more what do you mean by "still need to add the integration to have it fully available" ? On the list of devices in hacs, I get only gecko update, so not sure what else could I add.
thanks for helping and sharing

00:55:11 – (FEL) /usr/local/lib/python3.9/site-packages/aiohttp/web_protocol.py - Meddelandet inträffade först 00:54:18 och har hänt 2 gånger

I don't think that is related, the web_protocol.py is not used by the geckolib library.
Did you try using the specific IP address to try to discover the spa that way?

Thanks for your reply, I will try a shell first as mf76130 suggested.

I tried with the gecko shell on that raspberry pi I am running HA.
It discovers it without problems.

But in GUI, just Gecko Update.
mf76130 I would appreciate your help with claryfying how did you manage do add the rest of the sensors of gecko - for me is just this gecko.update that is available.
Thanks in advance!

You just go to "Settings" - "Devices & Services", click on "Add Integration" and there search for "Gecko", click it and follow the setup.

Thanks for advice. Well, I was adding other integrations from HACS this week and this extra step was not required that's why I even did not try to do that. It works now as expected.


Error Message mostly in Ship causing lag

Hello,
I've seen the below error message when playing with friends and everytime this comes up I also see frame drops in the ship.
Don't appear everytime but often but also goes away after we finished the moon and taken off.

Hello, I've seen the below error message when playing with friends and everytime this comes up I also see frame drops in the ship. Don't appear everytime but often but also goes away after we finished the moon and taken off.

The mod doesn't patch EnemyAI
it may be from another mod
they may be incompatibles

@Fabimilo I've got the same issue, if we compare mods we may find the culprit.

@Fabimilo I've got the same issue, if we compare mods we may find the culprit.

yes, with more info i could see if the issue is from LE and fix it


A query about theoretical total input of spike-ins

Hi,
For the code:
cms_095046 <- read.delim("~/cms_095046.txt", stringsAsFactors = FALSE)
SpikeInfo = data.frame(ERCCID=cms_095046$ERCC.ID, count=cms_095046$concentration.in.Mix.1..attomoles.ul.)
SpikeInfo = SpikeInfo[SpikeInfo$ERCCID %in% rownames(ERCC_dat),]
rownames(SpikeInfo) = SpikeInfo$ERCCID
SpikeInfo[,2] = SpikeInfo[,2]/1000

May I ask can I calculated the theoretical total input of spike-ins as sum(SpikeInfo$count)? If so, is it the same for both CELL_seq2 and Sort-seq datasets?
I would like to divide the total number of observed spike-ins by that number to obtain capture efficiencies which should range between 0 and 1.
Thank you very much!
Best wishes,
Wenhao

Hi Wenhao,
I have added the ERCC annotation: https://github.com/LuyiTian/CellBench_data/tree/master/misc.
It should be used with care though. We have noticed the calculated capture efficiency is much lower than we expected and we thought it might be caused by ERCC degradation because there are a couple of freeze-thaw cycles and dilution. Also, we used a very high dilution factor, so as you might see many ERCC genes have a theoretical input lower than one, so cannot be detected even with 100% capture efficiency. personally, I found the methods that use ERCC to separate technical variation do not perform very well, such as BASiCS.
The last thing is that the RNA mixture data were generated from pure RNA instead of cells, where ERCC spike-in behaved well. But the ERCC have more variation in real single-cell data, which might be related to cellular content, as ERCC have short polyA and no secondary structures or attached proteins, making them more vulnerable than cellular mRNA.

Hi @LuyiTian, I see, thank you so much!
Best wishes,
Wenhao


SessionAffinity timeout max value

Describe the bug
According to the K8s API documentation, the maximum value for service.spec.sessionAffinityConfig.clientIP.timeoutSeconds is 86400, which corresponds to 1 day. See https://godoc.org/k8s.io/api/core/v1#ClientIPConfig.
In Antrea, the maximum value is actually 65535, since this timeout is implemented using an OpenFlow hard timeout, which is a uint16.
I am not sure there is too much we can do about this, except for documenting this limitation somewhere.
Versions:

Antrea version: v0.11 and ToT

Since #3609, we do the following when the user-provided timeout exceeds the maximum supported value (65535):

Log a warning
Truncate the user-provided timeout to the maximum supported value


Live HLS stream crashing on IOS

I am playing a live HLS stream URL which works fine on Android and on IOS emulator, but when I play on iPhone player gets crashed
Stream url: http://capcobroadcaststream.in:1935/capco/ad24/playlist.m3u8

@paraswatts did you find a workaround? Experiencing same issue, working well inside iOS simulator, crashing after a second on a device.


DITAVAL with conditions for group default values

Expected Behavior
If we have a conditional attribute group such as groupA(a b) and we set the group to exclude in a ditaval file using <prop action="exclude" att="audience" val="groupA"/> it should set the default values for a and b to exclude and the entire group should evaluate to exclude, according to DITA Specification.
Actual Behavior
The default values of a and b are not changed and the group evaluates to include.
Steps to Reproduce

Download the following testcase: test-condition.zip
Publish main_map.ditamap to PDF
Open the PDF. Chapter1 contains the word product1, but the keydef element containing that keyword should have been excluded from the ditamap and instead we should see product2

Environment

DITA-OT version: DITA-OT version:4.1
Operating system and version:
(macOS Big Sur Version 11.6)
How did you run DITA-OT?
(oXygen)
Transformation type:
(PDF)

@TroacaDenis - when you use a group, treat the group name as the att name:
<val>
    <prop action="exclude" att="groupA"/>
</val>

and I think this will give you the behavior that you want.

@chrispy-snps Denis is one of our interns, he was just testing to see if what the specification states works in the published output and according to the specs this should work as well:
  <prop action="exclude" att="audience" val="groupA"/>

Example in the linked spec page_

<p product="appserver(A B)"> is filtered out, because there is no explicit rule for A or B, and values in the "appserver" group inside of @product default to exclude.

Yes, I think @TroacaDenis 's interpretation comes from this example in the specification:
 <prop action="exclude" att="product" val="appserver"/>

 The first <prop> element excludes the value "appserver" when used within the @product attribute. It also sets a default of "exclude" for values within any appserver() group inside of the @product attribute.


Fully commit to CF compliance?

It seems the design philosophy here is that the core functionality is based on CF-compliant datasets.
If a dataset is not CF compliant, then we provide utilities to make it CF compliant (and/or require the provider to do that themselves first).
I think this simplifies the code, and makes it more robust -- if the heuristics used to find something fail, it will fail at the "make it CF compliant" stage, not the "use it" stage.
I haven't audited the code to see where this might not be done already -- but one place that needs updating is the data_vars code.
(and of course, the assign_ugrid_topology utility, which is not complete -- in particular, there's no addition of metadata to the data variables
https://ugrid-conventions.github.io/ugrid-conventions/#data-defined-on-unstructured-meshes
which should have mesh and location attributes:
double Mesh2_waterlevel(time,nMesh2_face) ;
    Mesh2_waterlevel:standard_name = "sea_surface_height_above_geoid" ;
    Mesh2_waterlevel:units = "m" ;
    Mesh2_waterlevel:mesh = "Mesh2"
    Mesh2_waterlevel:location = "face" ;
    Mesh2_waterlevel:coordinates = "Mesh2_face_x Mesh2_face_y" ;

In theory they should.... but almost no datasets include this so i targeted the middle ground in the beginning. We can add functions to add that metadata as an option though since we can already detect via xarray dimensions.

Yeah, as far as I know, there are pretty much no properly compliant datasets in the wild.
So we need to accommodate non-compliant files -- but where? So far, I"ve seen (and written) a lot of code that used heuristics / makes assumnptions in the middle of the code, where the information is used. But I regret that -- it:
a) makes it very hard to locate the errors / fixes when it doesn't work.
b) makes the code fail late, rather than early
c) provides not encouragement to others to make their files compliant.
So I'm thinking that any "make it compliant" or "guess what things mean" code should be outside of the core, and strictly a pre-processing step, that users opt-in to.

I think that we're heading that way now -- with the check for start_index in the subsetting code, I don't think there are any other heuristics/assumptions in the core code.
@mpiannucci: if you agree, we can close this.

NOTE: I've added #39 -- if we get to it.

 Then we can close this out. Also have to see what pieces are missing for schism

Moved to #39


Issue 116: Fix for missing script headers

Adds qw( get_template_and_user ) to C4::Auth uses
Adds qw( output_with_http_headers ) to C4::Output uses

Please see Issue 116

Added to PR #118 which includes rebuild of kpz.


Emphasize that ack is about directories, too

Usage: ack [OPTION]... PATTERN [FILE]
Even in usage, we say "file" even though it could be a directory.  Make it obvious and clear that ack need not take an actual filename.

Hi.
You can find a fix (at least partial) to this problem here:
https://github.com/shlomif/ack2 .
Regards,
-- Shlomi Fish

Can you give me a specific commit or file you're referring to?

https://github.com/shlomif/ack2/commit/1eda2955da61a83ffe08fba738762ed17fdd6422 - this one.


JSDoc?

Would specjs support JSDoc in the furure? Thanks, keep up the good work

Does it currently interfere with JSDoc?

Hey @kokujin and @DannyDelott, thanks for asking! The SpeckJS team currently does not have any plans to add support for JSDoc (nor have we tried using both in the same project), but if you happen to try it and see any funny behavior, we'd love to hear about it! Thanks


Add support for Github Actions

Add support Github actions for collector
Add test case

Travis build for PHP with version 7.1 failed
> php dev-tools/vendor/bin/phpmd src text build/config/phpmd.xml

/home/travis/build/php-coveralls/php-coveralls/src/Bundle/CoverallsBundle/Command/CoverallsJobsCommand.php:24
The class CoverallsJobsCommand has a coupling between objects value of 13. Consider to reduce the number of dependencies under 13.

Script php dev-tools/vendor/bin/phpmd src text build/config/phpmd.xml handling the sca event returned with error code 2

I don't know that it means

Do we need Github Actions ci file for this PR?

@keradus, can you review please?

Could the maintainers review / merge this please.

As I understand this repo has one member - @keradus and  judging by the status on the github he is on vacation

@ADmad  you can use this repo for sending events to coveralls.io

Thanks @Smolevich.

Unfortunately, this does not work for pull requests, because GITHUB_REF is set to refs/remotes/pull/123/merge instead of refs/heads/… (see this log). Then Coveralls will not associate the build with the actual pull request (see this build) and never report the status back to GitHub.

hey. exactly, i was on hols. thanks for understanding.
I took care of Travis build for PHP with version 7.1 failed issue on 2.1 branch, please merge/rebase the PR to see it.
@Smolevich, can you please look at @leofeyer concern? thanks!

hey. exactly, i was on hols. thanks for understanding.
I took care of Travis build for PHP with version 7.1 failed issue on 2.1 branch, please merge/rebase the PR to see it.
@Smolevich, can you please look at @leofeyer concern? thanks!

Ok, i see case @leofeyer. @keradus, What branch should merge in branch for PR?

@leofeyer, see build

@Smolevich What exactly do you want to show me with the build? According to the logs, you are using cedx/coveralls to send the coverage report and not php-coveralls/php-coveralls. I might have missed the point here though. 🙈

@Smolevich What exactly do you want to show me with the build? According to the logs, you are using cedx/coveralls to send the coverage report and not php-coveralls/php-coveralls. I might have missed the point here though. 🙈

Yes, service_branch will be set incorrectly for event pull_request if branch for PR is located in fork repo

@keradus, what I should to do for accepting this PR?

While testing this (I was trying to get our workflow working even before this PR is merged) I updated my composer.json to use this branch:
require-dev: {
        "php-coveralls/php-coveralls": "dev-add-support-for-github-actions",
}
...
"repositories": [
        {
            "type": "vcs",
            "url": "git@github.com:Smolevich/php-coveralls.git"
        }
]

But I'm getting the following message:
Collect git info
Read environment variables
Requirements are not satisfied.
  - GITHUB_ACTIONS='true'
  - GITHUB_REF='refs/pull/1482/merge'
  - CI_NAME='github-actions'
  - COVERALLS_REPO_TOKEN='********(HIDDEN)'

Set environment variables properly like the following.
For Travis users:

  - TRAVIS
  - TRAVIS_JOB_ID

For CircleCI users:

  - CIRCLECI
  - CIRCLE_BUILD_NUM
  - COVERALLS_REPO_TOKEN

For Jenkins users:
  - JENKINS_URL
  - BUILD_NUMBER
  - COVERALLS_REPO_TOKEN

For AppVeyor users:

  - APPVEYOR
  - APPVEYOR_BUILD_NUMBER

From local environment:

  - COVERALLS_RUN_LOCALLY
  - COVERALLS_REPO_TOKEN

Is a change on this line necessary? The error message should probably also be changed to include the new GitHub Actions variables required.

I will check it

join in case I can help

@emmetog i update exception message

@keradus can you review please?

@Smolevich I'd just like to say that the way you copy my code into yours does not preserve any of my attribution and I consider it a bit unfair. Especially when I prepared easy to merge PR for you.

I check your changes and if they are correct i move them in this PR

@simPod check your conversations, please. @renekorss review please

This comment is still valid, it won't run on GA without setting COVERALLS_RUN_LOCALLY=1
#275 (review)
@renekorss what about this case?

@keradus review please

This comment is still valid, it won't run on GA without setting COVERALLS_RUN_LOCALLY=1
#275 (review)

@keradus what about case?

I've tried to test this PR again (instructions here) but it's not working, I believe it's because coveralls doesn't yet handle the "github" service_name.
When I modify this PR branch to hardcode CI_JOB_ID and CI_NAME to null it generates a coveralls.json as follows:
    "service_name": "",
    "service_branch": "my-branch",
    "repo_token": "<<redacted>>",
    "git": {
    ...

this seems to work :point_up:
However the PR in its current state creates a coveralls.json like this:
    "service_name": "github",
    "service_job_id": "44727b6d6df5c15b11b26579f72d9b8edd8069a1",
    "service_branch": "my-branch",
    "repo_token": "<<redacted>>",
    "git": {
    ...

which does not work :disappointed: It fails with the following output:
...
Collect git info
Read environment variables
Dump submitting json file: /path/coveralls.json
File size: 968.15 kB
Submitting to https://coveralls.io/api/v1/jobs
Client error occurred. status: 422 Unprocessable Entity
Couldn't find a repository matching this job.
elapsed time: 2.054 sec memory: 10.00 MB

The official coveralls docs suggest that the service_name and service_job_id are optional, it seems that if these are present and valid then coveralls asks the relevant CI system for additional info to supplement it's analysis.
I suspect it's failing because coveralls doesn't yet recognise the "github" service_name or it's not working yet or something.
Should we set the service name and job ID to be null or not present for the moment until Coveralls handles them correctly?

This works well for me https://github.com/simPod/php-coveralls/commit/b9c9d4c8ffdf837c589aba5da3f83e660350bfea I reversed engineered their js implementation and api so I'm setting the same as they do.
 I don't have BRANCH set. Try it without it.

@emmetog can i show your test?

@emmetog you can see on work PR here https://github.com/jenssegers/laravel-mongodb/pull/1920/checks

The steps of my test are:

@Smolevich For my test I first set the env vars:
$ export CI_NAME='github'
$ export GITHUB_ACTIONS=1
$ export GITHUB_REF=refs/heads/my-branch
$ export GITHUB_SHA=44727b6d6df5c15b11b26579f72d9b8edd8069a1
$ export COVERALLS_REPO_TOKEN=<<redacted>>
$ export GITHUB_EVENT_NAME=pull_request

Then I run phpunit to generate the clover coverage xml and run coveralls to push them:
$ ./vendor/bin/phpunit --coverage-clover coverage/clover.xml
$ vendor/bin/php-coveralls --coverage_clover coverage/clover.xml --json_path coveralls.json -vvv

I'm using a private GitHub repo. The php-coveralls package in my vendor/ is using this exact PR which can be set up by following the instructions here.
@simPod I tried removing the branch as you suggested but still no joy :disappointed: Here's the coveralls.json that was generated:
    "service_name": "github",
    "service_job_id": "44727b6d6df5c15b11b26579f72d9b8edd8069a1",
    "repo_token": "<<redacted>>",
    "git": {

Could you add a step in your github-actions file to show the generated /home/runner/work/laravel-mongodb/laravel-mongodb/build/logs/coveralls-upload.json file in that repository that works? Something like this:
      - name: Debug converalls.json
        run: cat /home/runner/work/laravel-mongodb/laravel-mongodb/build/logs/coveralls-upload.json

@Smolevich For my test I first set the env vars:
$ export CI_NAME='github'
$ export GITHUB_ACTIONS=1
$ export GITHUB_REF=refs/heads/my-branch
$ export GITHUB_SHA=44727b6d6df5c15b11b26579f72d9b8edd8069a1
$ export COVERALLS_REPO_TOKEN=<<redacted>>
$ export GITHUB_EVENT_NAME=pull_request

Then I run phpunit to generate the clover coverage xml and run coveralls to push them:
$ ./vendor/bin/phpunit --coverage-clover coverage/clover.xml
$ vendor/bin/php-coveralls --coverage_clover coverage/clover.xml --json_path coveralls.json -vvv

I'm using a private GitHub repo. The php-coveralls package in my vendor/ is using this exact PR which can be set up by following the instructions here.
@simPod I tried removing the branch as you suggested but still no joy 😞 Here's the coveralls.json that was generated:
    "service_name": "github",
    "service_job_id": "44727b6d6df5c15b11b26579f72d9b8edd8069a1",
    "repo_token": "<<redacted>>",
    "git": {

Could you add a step in your github-actions file to show the generated /home/runner/work/laravel-mongodb/laravel-mongodb/build/logs/coveralls-upload.json file in that repository that works? Something like this:
      - name: Debug converalls.json
        run: cat /home/runner/work/laravel-mongodb/laravel-mongodb/build/logs/coveralls-upload.json

What is the version php-coveralls in composer.json?

Interestingly, setting COVERALLS_RUN_LOCALLY in my github-actions job seems to make everything work (my composer is still locked to this PR branch). in other words:
      - name: Upload coverage results to Coveralls
        run: vendor/bin/php-coveralls --coverage_clover coverage/clover.xml --json_path coveralls.json -vvv
        env:
            COVERALLS_RUN_LOCALLY: 1

From what I've seen this also sets the branch and pull request link details correctly in the coveralls UI. This suggests that the changes which are made when COVERALLS_RUN_LOCALLY=1 are the correct ones for github actions (they work):
        if (isset($this->env['COVERALLS_RUN_LOCALLY']) && $this->env['COVERALLS_RUN_LOCALLY']) {
            $this->env['CI_JOB_ID'] = null;
            $this->env['CI_NAME'] = 'php-coveralls';
            $this->env['COVERALLS_EVENT_TYPE'] = 'manual';
            // ....

Interestingly, setting COVERALLS_RUN_LOCALLY in my github-actions job seems to make everything work (my composer is still locked to this PR branch). in other words:
      - name: Upload coverage results to Coveralls
        run: vendor/bin/php-coveralls --coverage_clover coverage/clover.xml --json_path coveralls.json -vvv
        env:
            COVERALLS_RUN_LOCALLY: 1

From what I've seen this also sets the branch and pull request link details correctly in the coveralls UI. This suggests that the changes which are made when COVERALLS_RUN_LOCALLY=1 are the correct ones for github actions (they work):
        if (isset($this->env['COVERALLS_RUN_LOCALLY']) && $this->env['COVERALLS_RUN_LOCALLY']) {
            $this->env['CI_JOB_ID'] = null;
            $this->env['CI_NAME'] = 'php-coveralls';
            $this->env['COVERALLS_EVENT_TYPE'] = 'manual';
            // ....

See https://github.com/Smolevich/reindexer-client/runs/431760052?check_suite_focus=true

Hello, I have merged the PR into my forked repo, added parallel job support, and published a new release, feel free to use it.
https://github.com/twinh/php-coveralls#github-actions
Add a new step after phpunit generate coverage report.
- name: Upload coverage results to Coveralls
  env:
    COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
  run: |
    composer global require twinh/php-coveralls
    php-coveralls --coverage_clover=build/logs/clover.xml -v

@OndraM @keradus Could we somehow support GHA?
It is 1 year old.
https://github.blog/2019-08-08-github-actions-now-supports-ci-cd/

Anyone considering / already migrated to codecov and can share experience?

migrated to codecov

Coveralls is the supoerior service.
I use and advocate https://github.com/twinh/php-coveralls

Anyone considering / already migrated to codecov and can share experience?

they've got a github action for it, works fine.

Anyone considering / already migrated to codecov and can share experience?

They use the GH annotations API so that you get comments on the lines without code coverage on the pull request page itself.

I merged the extended version of this PR - the #296.
big thanks to work on that!
i kept your co-ownership when merging to master ;)


fix: remove core-anywhere reference

I can't test this locally anymore
Access to XMLHttpRequest at 'https://keedrop.com/api/secret/' from origin
'http://htpc:4000' has been blocked by CORS policy: Response to preflight request
doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on
the requested resource.

Closes #39

I'll use #32 as a new base for this and expose cors-anywhere as an option if defined in the env.

Alternatively you could check jekyll.environment == 'development' - https://jekyllrb.com/docs/configuration/environments/


Add segmented progress bar

Note: This does not handle showing the current page's progress (i.e. for auto-advance); it simply exposes the hooks to add this functionality in a future PR.

Note to self: this is diffed against the page-refactor-2 branch, but should just be merged into master after #181 has been merged.


Auth users (2)

Codecov Report

Merging #186 into master will increase coverage by 4.97%.
The diff coverage is 79.53%.

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

Replaced by 3.


Redis uses a single connection

There's a max 256 in the server available and force uses a single connection. Is this by design?

Would recommend looking into https://www.npmjs.com/package/redis-connection-pool


Stringified JASON and paeseInted it back to show the array of ckicked…

… data

pair programming with Nathon


Store all integration events i one central eventstore

Accept criteria

[] all integration events is stored in one central storage
[] one or more eventtypes can be requested replayed
[] It is possible to replay events from any given point in time
[] Replayed events is clearly marked as a replayed event

Some thoughts on this feature:

It should be possible to get all events from a specific domain replayed in the correct order regardless of event type.
The domain which is responsible for the event should be the one publishing the events, so the events gets published to the correct topics for consumption by subscribers.
All subscribers should implement event idempotency, so replayed events does not affect local data model constructed by already received events.