Faster media start up

Our application stream Live media.
Media Start up time is 4 to 5 sec.
I am looking for ways to reduce media start time up time in order to provide best user experience.
Following are the list of call during media start up.

You can observe that the call in the list
MPD
init.mp4 for audio
1st audio segment
init.mp4 for video
1st video segment
get License call
2nd audio segment
3rd audio segment
2nd video segment
Media start only after downloading 2nd video segment. Segment duration is 6 sec.
Following are the abr and streaming configurations
mPlayer.configure({
            abr: {
                defaultBandwidthEstimate: 1000000,
                switchInterval: 6
            },
            streaming: {
                bufferBehind: 6,
                bufferingGoal: 6
            }
        });

Following is the mpd,
<?xml version="1.0" encoding="utf-8"?>
<MPD xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:mobitv="urn:mobitv" xmlns="urn:mpeg:dash:schema:mpd:2011" xsi:schemaLocation="urn:mpeg:dash:schema:mpd:2011 DASH-MPD.xsd" profiles="urn:mpeg:dash:profile:isoff-live:2011,urn:com:dashif:dash264" maxSegmentDuration="PT6S" minBufferTime="PT2S" type="dynamic" timeShiftBufferDepth="PT20M" minimumUpdatePeriod="PT8760H" availabilityStartTime="1970-01-01T00:00:19Z" suggestedPresentationDelay="PT11S" id="5062" publishTime="2019-12-11T05:23:54Z">
   <ProgramInformation>
      <Title>Media Presentation Description by MobiTV.</Title>
   </ProgramInformation>
   <Period mobitv:publishTime="2019-12-11T05:23:54Z" id="precambrian" start="PT0S" mobitv:thumbnails="150_T;640_T">
      <AdaptationSet contentType="video" mimeType="video/mp4" mobitv:qualityLevel="SD" segmentAlignment="true" startWithSAP="1" par="16:9" minWidth="640" maxWidth="1280" minHeight="360" maxHeight="720" maxFrameRate="60/2">
         <Accessibility schemeIdUri="urn:scte:dash:cc:cea-608:2015"/>
         <Role schemeIdUri="urn:mpeg:dash:role:2011" value="main"/>
         <SegmentTemplate startNumber="0" initialization="$RepresentationID$/init.mp4" duration="6" media="$RepresentationID$/$Number$.m4s"/>
         <ContentProtection schemeIdUri="urn:mpeg:dash:mp4protection:2011" value="cenc"/>
         <ContentProtection schemeIdUri="urn:uuid:edef8ba9-79d6-4ace-a3c8-27dcd51d21ed" value="Widevine: SD"/>
         <ContentProtection schemeIdUri="urn:uuid:9a04f079-9840-4286-ab92-e65be0885f95" value="Playready: SD"/>
         <Representation id="V2200_W" codecs="avc1.4d401f" bandwidth="2384800" mobitv:average-bandwidth="2205862" mobitv:peak-bandwidth="2361517" width="960" height="540" frameRate="60/2" sar="1:1"/>
         <Representation id="V5000_W" codecs="avc1.640020" bandwidth="5120000" mobitv:average-bandwidth="5001378" mobitv:peak-bandwidth="5284876" width="1280" height="720" frameRate="60/2" sar="1:1"/>
         <Representation id="V864_W" codecs="avc1.4d401e" bandwidth="884736" mobitv:average-bandwidth="871321" mobitv:peak-bandwidth="944708" width="640" height="360" frameRate="60/2" sar="1:1"/>
      </AdaptationSet>
      <AdaptationSet contentType="audio" mimeType="audio/mp4" lang="eng" mobitv:qualityLevel="SD" segmentAlignment="true" startWithSAP="1">
         <Role schemeIdUri="urn:mpeg:dash:role:2011" value="main"/>
         <SegmentTemplate startNumber="0" initialization="$RepresentationID$/init.mp4" duration="6" media="$RepresentationID$/$Number$.m4s"/>
         <ContentProtection schemeIdUri="urn:mpeg:dash:mp4protection:2011" value="cenc"/>
         <ContentProtection schemeIdUri="urn:uuid:edef8ba9-79d6-4ace-a3c8-27dcd51d21ed" value="Widevine: SD"/>
         <ContentProtection schemeIdUri="urn:uuid:9a04f079-9840-4286-ab92-e65be0885f95" value="Playready: SD"/>
         <Representation id="A96_ENG" codecs="mp4a.40.5" bandwidth="98304" mobitv:average-bandwidth="101359" mobitv:peak-bandwidth="102064" audioSamplingRate="48000">
            <AudioChannelConfiguration schemeIdUri="urn:mpeg:dash:23003:3:audio_channel_configuration:2011" value="2"/>
         </Representation>
         <InbandEventStream schemeIdUri="urn:mobitv:event" value="id3"/>
      </AdaptationSet>
   </Period>
</MPD>

How to start the media right after LICENSE response? First audio and video segments are downloaded before the License call . why to wait till 2nd video segment download ?

What happens if you set the bufferingGoal a bit lower, like 5 seconds? Or 5.5 seconds?

i tried setting bufferingGoal lower than 6 sec. That didn't help.

I'm not sure if I can do much more without actually being able to try out your content. Could you please provide the information necessary to reproduce this (manifest URI, DRM keys, etc)? If anything is confidential, you can send the details to our mailing address<EMAIL_ADDRESS>

This might help?
Ignore your minBufferTime="PT2S" by setting:
player.configure('manifest.dash.ignoreMinBufferTime', true);
player.configure('streaming.rebufferingGoal', 0);

@raskri i have tried this. Didn't help.

@mraavi, if you can't provide us with a way to reproduce your problem, we will not be able to diagnose the issue.  A copy of your MPD text is not useful without the content.  If you can send an accessible manifest URL to<EMAIL_ADDRESS>we can help you further.
Alternately, if the issue is not specific to your content, you could provide us with repro steps using the content in our demo app, which is all public.

@joeyparrish Here is the mpd
https://live-integration-msp-edge.connectcdn.net/mm/dash/live/67812/LIVESERVICE_1234/group=TG_WIFI_HD.mpd
The issue can be reproduced with Shaka demo player as well
https://shaka-player-demo.appspot.com/demo/#audiolang=en-US;textlang=en-US;uilang=en-US;panel=CUSTOM CONTENT;build=uncompiled

When we start playing, we'll start playing in the middle of a segment, so although we download the first segment, it won't contribute the whole time to the bufferingGoal/rebufferingGoal.  So after we download the first segment, we may have only buffered 2 seconds.  So if you set the rebufferingGoal to 6 seconds, it will require us to always download two segments.  You should set the rebufferingGoal to a small number like 2 seconds and the bufferingGoal to a larger number like 30 seconds.
By playing your asset and throttling the network, I see us playing after we download the first segment with the default settings.  So try reducing the rebufferingGoal.

@TheModMaker I have tried reducing rebufferingGoal to 1. It didn't help.
Did the configuration as below
mPlayer.configure('manifest.dash.ignoreMinBufferTime', true);
mPlayer.configure('streaming.rebufferingGoal', 1);
In fact this can be observed with Demo Shaka player and with the mpd i have provided.
I have uploaded a video with screen recording.
https://youtu.be/KnEKneQNigU

@mraavi, can you confirm what version of Shaka Player you're using?  I don't see that anywhere in your original report.

Also, can you please provide your Shaka Player version and the complete configuration you're applying to it?  It's fine to censor license server URLs and such, but we need to see any non-standard config values you're using to completely assess the situation.

We are using Shaka player version v2.5.3.
Configuration applied,
```
mPlayer.configure('manifest.dash.ignoreMinBufferTime', true);
mPlayer.configure('manifest.dash.clockSyncUri', headerUrl);
mPlayer.configure({
abr: {
defaultBandwidthEstimate: 1000000,
switchInterval: 30
},
streaming: {
bufferBehind: 12,
bufferingGoal: 30,
rebufferingGoal: 1
}
});
**As i stated above in the thread the issue could be reproducible with demo Shaka player as well both with live and non-live urls.

@ismena

You are right about, it takes 2 media segments to finish download to start the media on Chrome PC/MAC browser.
But it doesn't not depend on the network condition. On any network it requires 2 Video segments to finish downloading to start Live Playback.
Uploaded a video, which shows Live playback starts after 2 video segment download finish - https://youtu.be/kq8ZH0fj_I0. You can see 2 fetched/appended logs for Video and 2 fetched/appended for audio  before video start.
Uploaded a Video, which show "Non-Live playback" starts after 1 audio and 1 video segment download finish - https://youtu.be/zHekZqP3x1s
Our main concern is, Why would Shaka player waits till the 2nd video segment finish downloading where it can start the playback right after 1st video segment download (like Non-Live)
On Samsung TV's (With Tizen OS), Live playback starts after 3rd Video segment download finish. - https://youtu.be/YELEfJi-i54
Full log from Samsung TV is sent to<EMAIL_ADDRESS>with Subject 'Faster media start up #2291'.

This is actually expected behavior for live.  The segments in the manifest are only approximations of the times of the segments.  The times in the manifest can be off by half a segment duration.  So we start buffering at the segment behind the current time so we make sure we get something buffered at the playhead.
As for buffering three segments on Tizen, your video is a whole 4 seconds long.  It only appears we "wait" because everything loads so fast.  We check whether we have buffered enough on a timer, so after we have buffered enough, there is a slight delay before we start playing.  There may also be a delay caused by the platform that causes the next segment to complete before you can see anything.

This is actually expected behavior for live. The segments in the manifest are only approximations of the times of the segments. The times in the manifest can be off by half a segment duration. So we start buffering at the segment behind the current time so we make sure we get something buffered at the playhead.

@TheModMaker you mean to say, For live, Shaka downloads the previous segment before it downloads the current segment? And the playback starts after both previous and current segment download finishes?

As for buffering three segments on Tizen, your video is a whole 4 seconds long. It only appears we "wait" because everything loads so fast. We check whether we have buffered enough on a timer, so after we have buffered enough, there is a slight delay before we start playing. There may also be a delay caused by the platform that causes the next segment to complete before you can see anything.

As for buffering three segments on Tizen, your video is a whole 4 seconds long. It only appears we "wait" because everything loads so fast. We check whether we have buffered enough on a timer, so after we have buffered enough, there is a slight delay before we start playing. There may also be a delay caused by the platform that causes the next segment to complete before you can see anything.

In case of Samsung (Tizen), i can confirm that it is (3 segment downloads) not because the segments downloaded too fast. We have done 10's of tests under different network conditions to confirm this.
Do you think it is a platform issue? Is there a way i can provide you more logs to confirm this?

We're adding more logs and more granular configuration for the "download previous segment" behavior.  After that, we'll ask you to try again with the new code and new configuration to see if things have improved.  The new logs will improve our ability to explain to you what's happening in more detail.

The latest code in the master branch should both provide better logs and be less conservative about the "download previous segment" behavior to deal with inaccurate manifests.  This behavior is also now configurable.
Can you please try again with the latest code and tell us if things have improved?
If not, can you please attach logs at level v2?  shaka.log.setLevel(shaka.log.Level.V2);  (This logging requires a debug or uncompiled build.)

@shaka-bot reopen
We are testing with latest shaka code. will share the logs soon.

Looks like Demo shaka player not available with the main branch.
We have integrated main dev brach in our app and we observe the issue reported still exist,
Logs with "shaka.log.setLevel(shaka.log.Level.V2)" sent to<EMAIL_ADDRESS>

Can you please be more specific?  I don't know what you mean by "main branch" or "main dev branch".  Can you specify the branch name or release version that you are using from our repo?
I also don't know what you mean by "Demo shaka player not available with the main branch".  Whichever branch you mean, our demo player is available in the master branch and all release branches (v2.5.x, v2.4.x, etc) of our repo.

Also those logs aren't with the log level set, or it was severely redacted.  Also be sure to check that you aren't filtering logs in DevTools; make sure you have selected "All Levels" for logs.  V2 logs should be very verbose.  Also you can configure when we fetch the previous segment with:
// Fetch previous segment when within 2 seconds of start of the segment:
player.configure('streaming.inaccurateManifestTolerance', 2);
// Never fetch previous segment:
player.configure('streaming.inaccurateManifestTolerance', 0);

Also note the default site is v2.5.9; you can see builds from the master branch on our nightly site: https://nightly-dot-shaka-player-demo.appspot.com/demo/.

@joeyparrish @TheModMaker
Thanks for the detail.
We have tested Live demo player (https://nightly-dot-shaka-player-demo.appspot.com/demo/) with inaccurateManifestTolerance as 0. Now the playback starts as soon as first segment downloaded.
Also we have integrated, Shaka master branch in our app and tested Live with the configuration (player.configure('streaming.inaccurateManifestTolerance', 0);), Observed the same - Playback starts as soon as first segment and license calls are downloaded.
Observed same on Samsung TV's.
Thanks for the fix.
When can we expect next version of Shaka player release?

We're working on cherry-picks for the next bug fix release (v2.5.10) now.  We will finalize v2.6.0 as soon as #1339 is complete.  This change will not appear in the bug fix release, as the previous behavior was intended and the new behavior might break some applications.

Since there doesn't seem to be any more work to do on this issue, I'm going to close it now.  Thanks!


Zooming

Hello,
Do you know how to use your library and allow for the user to zoom into the image before cropping?
Thanks,
Anshul

I'm gonna try to add auto zoom in/out depending on crop size relative to the image...

Wow that would actually be amazing. Please let me know when you want me to test. :) I can QA it properly for you.

update: will start working on it soon, but because its not a trivial feature and I don't have a lot of time it may take at least a month, sorry.

+1 I love this library and I've been wishing for auto zoom feature. It'll definitely be very useful. Hope to get it soon. Thanks, Arthur!

It took long enough but it's finally here v2.0.0, hope you will find it useful.
I marked it as beta because it required many changes so there might be a bug or two, let me know and I will fix them ASAP.


Artifact identity-server-parent version 5.11.0 missing in wso2-public

Describe the issue:
Maven Artifact
    <artifactId>identity-server-parent</artifactId>
    <packaging>pom</packaging>
    <version>5.11.0</version>

is not released to http://maven.wso2.org/nexus/content/groups/wso2-public/
Expected behavior:
Artifact deployed to http://maven.wso2.org/nexus/content/groups/wso2-public/

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


feat: suggestions module

Implements the Suggestion module, overruling the old PR since its commit history was broken.
Closes #709

Closing this as @cfanoulis has left the development team.

Resuming development of this


k3s: k3s_1_27 -> k3s_1_28

k3s: k3s_1_27 -> k3s_1_28
Tracking issue: https://github.com/NixOS/nixpkgs/issues/287344
ping @euank @Mic92 @yajo @wrmilling

@euank Please, review.

@ofborg build k3s.passthru.tests.single-node k3s.passthru.tests.multi-node

Pass through tests are broken in CI.

       >     exec(self.tests, symbols, None)
       >   File "<string>", line 4, in <module>
       >   File "/nix/store/xbnrwrrmjmjzb6wg6z56wdblvydfq358-nixos-test-driver-1.1/lib/python3.11/site-packages/test_driver/machine.py", line 469, in wait_for_unit
       >     retry(check_active, timeout)
       >   File "/nix/store/xbnrwrrmjmjzb6wg6z56wdblvydfq358-nixos-test-driver-1.1/lib/python3.11/site-packages/test_driver/machine.py", line 134, in retry
       >     raise Exception(f"action timed out after {timeout} seconds")
       > Exception: action timed out after 900 seconds
       > kill vlan (pid 7)
       For full logs, run 'nix log /nix/store/gv3y300mfhb3ph89bkfccl64vlwqjwm2-vm-test-run-k3s-1.28.6+k3s1-multi-node.drv'.
error: build of '/nix/store/gv3y300mfhb3ph89bkfccl64vlwqjwm2-vm-test-run-k3s-1.28.6+k3s1-multi-node.drv', '/nix/store/xr15wlq6bllyapxpajpl095iylalrgxb-vm-test-run-k3s-1.28.6+k3s1-single-node.drv' failed

It builds this way:
nix build .#k3s.passthru.tests.single-node
Won't build this way:
nix build .#nixosTests.k3s.single-node
Would OfBorg understand as that? I think, maybe we need to rethink this.
Idk yet.

Looking at ofborgs logs, the errors seem real:
server # [  928.439558] k3s[6224]: time="2024-02-10T16:32:43Z" level=fatal msg="Failed to validate golang version: kubernetes golang build version not set - see 'golang: upstream version' in https://github.com/kubernetes/kubernetes/blob/dev/build/dependencies.yaml"

(from this link).
I think that means that we want to land https://github.com/NixOS/nixpkgs/pull/287792 before we land this one so the default k3s attribute works.
We accidentally broke 1_28 when https://github.com/NixOS/nixpkgs/pull/286095 merged, so before swapping the default to 1_28 we should fix that.
Apart from waiting on your fix to 1_28 to merge, this looks good to me!

I think that means that we want to land #287792 before we land this one so the default k3s attribute works.
We accidentally broke 1_28 when #286095 merged, so before swapping the default to 1_28 we should fix that.
Apart from waiting on your fix to 1_28 to merge, this looks good to me!

Yes, you are right.
A commiter needs to merge: #287792
Then I need to rebase this PR.
Can you still do merges?

Thanks wegank!


621 add sample pages with components

I cant run this branch, because of this error
/Users/reza/workspace/fluentcms/FluentCMS/src/FluentCMS.Web.UI/Layouts/ComponentsLayout.razor(78,2): error RZ9981: Unexpected closing tag 'div' with no matching start tag. [/Users/reza/workspace/fluentcms/FluentCMS/src/FluentCMS.Web.UI/FluentCMS.Web.UI.csproj]

please run the branch at least once before applying a pull request.


Sway treats comments as arguments

When running sway, I get (amongst others) the following error message:
Handling command 'set $mod Mod4       # Logo key. Use Mod1 for Alt.'
Invalid set command.(expected 2 arguments, got 9

Command failed: set
It looks like it interprets the comment at the end of the line as arguments to the set command.
(commit 786bb87, using AUR package sway-git)

Thanks for letting us know!


Fix circular import loop

Motivation
While testing the python3-smart-open package in Debian, I ran into a circular import error.
Tests
If you're fixing a bug, consider test-driven development:

Create a unit test that demonstrates the bug. The test should fail.
Implement your bug fix.
The test you created should now pass.

If you're implementing a new feature, include unit tests for it.
Make sure all existing unit tests pass.
You can run them locally using:
pytest smart_open

If there are any failures, please fix them before creating the PR (or mark it as WIP, see below).
Work in progress
If you're still working on your PR, include "WIP" in the title.
We'll skip reviewing it for the time being.
Once you're ready to review, remove the "WIP" from the title, and ping one of the maintainers (e.g. mpenkov).
Checklist
Before you create the PR, please make sure you have:

[x] Picked a concise, informative and complete title
[x] Clearly explained the motivation behind the PR
[x] Linked to any existing issues that your PR will be solving
[x] Included tests for any new functionality
[ ] Checked that all unit tests pass

Workflow
Please avoid rebasing and force-pushing to the branch of the PR once a review is in progress.
Rebasing can make your commits look a bit cleaner, but it also makes life more difficult from the reviewer, because they are no longer able to distinguish between code that has already been reviewed, and unreviewed code.

Can you please show the error you encountered? It's not being picked up by any of our tests.

Weird, now I can't reproduce it :-( Sorry!


Avatar is always R15 when playtesting

When using RSMM, my avatar is always set to R15 when I playtest.
I've tried:

Setting the rig type to R6 in game settings.
Modifying the Humanoid Description
DataModel:SetUniverseId(1818)

This issue does not occur on vanilla studio.

RSMM Version v2024.03.02, Studio is Latest
I only have the AnimatorIsThrottledPropertyEnabled DFFlag modified.
Any help is appreciated.

Can you confirm this isn’t happening in regular studio? RSMM shouldn’t have any effect on this.


Update serverless-ai-tutorial.md

Made some changes to the post including couple of errors in the code that caused it to break:

max_tokens should be maxTokens
the /static static path was causing the file dynamic.js to not be found
Added the code for index.html and dynamic.js in the tutorial
the TS example said http-js so made the change the http-ts
ai_models = ["llama2-chat"] was missing in the spin.toml file
Changed the title to reflect the use case discussed in the post - similar to our other tutorials
@rajatjindal suggested adding a link to the repo for people who want to git clone the example directly. Added that too

Content must go through a pre-merge checklist.
Pre-Merge Content Checklist
This documentation has been checked to ensure that:

[ ] The title, template, and date` are all set
[ ] Does this PR have a new menu item (anywhere in templates/*.hbs files) that points to a document .md that is set to publish in the future? If so please only publish the .md and .hbs changes in real-time (otherwise there will be a menu item pointing to a .md file that does not exist)
[ ] File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep '^M' | wc -l and expect 0 as a result)
[ ] Has passed bart check
[ ] Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)
[ ] Headings are using Title Case
[ ] Code blocks have the programming language set to properly highlight syntax and the proper copy directive
[ ] Have tested with npm run test and resolved all errors
[ ] Relates to an existing (potentially outdated) blog article? If so please add URL in blog to point to this content.

Folks, closing this PR. When other edits were made to this tutorial the file name was changed and that has caused some merge issues. i've consolidated the changes mentioned in this PR to a new PR here #916


Fix failed to load profile's error message

It seems we need to drop Py3.3

@roll I agree. Even Python and Pytest itself dropped support already (see https://github.com/pytest-dev/pytest-mock/pull/96)

Would you like to commit?
I'll merge and release straight after.

@roll All good now :+1:


Add Hex as binary to text encoding option for HashPipe

Reason for PR: I have a system that depends on the hashes being encoded using Hex instead of Base64. This is currently handled in custom code based on the HashPipe. That custom code is merged into the HashPipe with this PR.


Pronunciation Evaluation

Hi,
Is there a function in this library to compare pronunciation of a given word and check how close it is to the ground truth.
This will be immense help to have if it is there.
Regards,
Ash

@easyash ,
not really. The library focuses on bringing different speech_recognition APIs together. It is not meant to some kind of "research" in speech recognition. For this purpose you'll need something heavier.


Stuck on "Initializing sumneko-lua"

So I just removed coc-lua because I couldn't figure out ignoring globals, and hear good things about coc-sumneko-lua.
Opening a bare minimum lua file, though, sumneko-lua never stops initializing.

I haven't adjusted any settings yet, my coc-settings.json is incredibly minimal and unrelated to lua:
{
  "suggest.autoTrigger": "none",
  "diagnostic.enable": true,
  "python.pythonPath": "/usr/bin/python"
}

So, curious, anything I should be configuring here? Could you point me to the relevant bits of any documentation, if so?
Cheers


Fix non-teaching load export bug

This fixes the issues specified in #142. To test, import a schedule and create many non-teaching load activities (more than 1 for a professor, some with no term, etc.). Then, export and reimport the schedule. The non-teaching loads should work exactly as they did before the export. I believe the issue with all the non-teaching loads importing with the same load hours was fixed with #151.

@Jonri2
I just tried this on the development branch and I'm now able to create/display/sum multiple non-teaching activities of different hour loads per faculty, but only the first activity is exported per faculty member, and if I manually add a CSV row back in for a second activity for that faculty member, only the first is imported.

@kvlinden I just tested it on develop and am not able to reproduce that issue. It could be that there are issues with the spreadsheet you are importing if it's an export from before the bug was fixed. Could you try again with a schedule that starts with no non-teaching loads?

@kvlinden, I believe in order to get the updates, you need to refresh your website once in order for the newest javascript to load. We could consider adding the commit hash to our website using GitHub actions so we can see which version is on develop so you can be sure it's on the latest. Or we could alert the user when they are on an older version.

@Jonri2 & @charkour : I just tried again and got the same problem.
On the development site (https://sharp-babbage-a45ee2.netlify.app/#/), I loaded a raw dump of the CS schedule, added two non-teaching activities for myself, which displayed/summed/exported properly. Here's the export file (converted to XLSX to make GitHub happy):
full_schedule_2021-03-18T07 19 15-04 00.xlsx
When loading (the CSV version), I only see the first activity.

@kvlinden Alright, I think I figured out what the bug is. It looks like only the 1st non-teaching load is taken for a professor if they are listed under the same term. I should have it fixed by the end of the day.

Thanks - maybe it's the semester setting. A while back, I found that I needed to set a semester to get this to save, so I've just always done that. Actually, these non-teaching assignments don't have semesters, so it would make sense to leave the FA/SP/... un-specified, if that would help.

I think the deans to like to have these things declared in a particular semester.  And some things like sabbaticals and CRFs do have a specific semester in which the work is to be done.


nightly only?

I notice this crate currently seems to be nightly only. It would be nice if this were not the case. I'll look into this on my own, but if there are any particular nightly features we're using maybe there are viable alternatives?

fixed in #28


Test fails when sending a large number of pages with fast dispatch on ethernet cores.

When using ethernet cores for fast dispatch, Sending131072Pages test fails consistently when sending pages greater than around 1834.
Modifying the test to send less than this number of pages passes.
It looks like the host command queue has some kind of bug as data for later pages overwrites earlier pages.
At the point of failure, data for page 1834 or its vicinity appears at page 786.
From experiments it looks like both EnqueueWriteBuffer and EnqueueReadBuffer are impacted.
The difference between Tensix fast dispatch and Ethernet fast dispatch is that ethernet core had around 132 KB of L1 for buffering while Tensix has > 1MB of L1 for buffering.
I tried combinations of Direct read/write to host memory and fast dispatch Enqueue commands to do the corresponding write/read.
Always Passes.
tt::tt_metal::detail::WriteToBuffer( bufa, src);
tt::tt_metal::detail::ReadFromBuffer(bufa, result);
Starts Failing at Page Id >= 1835.
tt::tt_metal::detail::WriteToBuffer( bufa, src);
EnqueueReadBuffer(cq, bufa, result.data(), true);
Starts Failing at Page Id >= 1902.
EnqueueWriteBuffer(cq, bufa, src.data(), true);
tt::tt_metal::detail::ReadFromBuffer(bufa, result);
Starts Failing at Page Id >= 1834.
EnqueueWriteBuffer(cq, bufa, src.data(), true);
EnqueueReadBuffer(cq, bufa, result.data(), true);
To Reproduce:
checkout branch: ucheema/fd-on-idle-erisc-debug
build
run test:
WH_ARCH_YAML=wormhole_b0_80_arch_eth_dispatch.yaml NUM_PAGES=1836 TT_METAL_DPRINT_CHIPS=0 TT_METAL_DPRINT_ETH_CORES="all"  build/test/tt_metal/unit_tests_fast_dispatch --gtest_filter=*.Sending131072Pages

found the issue to be missing clearing of sync registers in idle_erisc.cc

closing


Add a share button in a petition's page

We want to add on any petition's webpage a sharing wiidget, which allow an user to post the petition on a social media or to copy the petition's path on his clipboard by clicking on the appropriate button :

Thanks for your support !

For now I mark this one as closed. Please reopen it if something is missing.


Anyone has 6.35 opcode now?

Dupe of #587
After updating XivAlexander and checking for opcode updates, I do have the correct opcodes, and things seem to be working properly for me.


Issue building CARLA during make CarlaUE4Editor: cannot find xerces 3.2.3 since they upgraded to 3.2.4

CARLA version: 0.9.13
Platform/OS: Ubuntu 18
Problem you have experienced: Impossible to build the dependency xerces 3.2.3
What you expected to happen:

Cannot download the archived 3.2.3 version: https://ftp.cixug.es/apache/xerces/c/3/sources/xerces-c-3.2.3.tar.gz
This file doesn't exist since they released a new version on October 17th 2022

Steps to reproduce:

Build the Carla.Dockerfile using master or 0.9.13 branches.
This happens when running cd Util/BuildTools && make setup (bash Util/BuildTools/Setup.sh)

Other information (documentation you consulted, workarounds you tried):

We can replace https://ftp.cixug.es/apache/xerces/c/3/sources/ by https://archive.apache.org/dist/xerces/c/3/sources/

It has been fixed in the latest develop. Thanks!

3q


Router not matching wildcard as expected

see https://github.com/riot/riot/issues/1820

Closing this issue because it's related to an old router version. Please update to the latest @riotjs/route version if you can.


Data helper routines seem to be a bit verbose

According to https://github.com/Ulibka68/ydb-nodejs-sdk/blob/master/examples/basic-example-new-way/table-definitions.ts#L7 the following change is proposed: https://gist.github.com/tsufiev/221af742929ba806f1f46b43054e775e

The version 3.0.0 introduced quite a few improvements in that regards.


Status page links

feat: Add footer to search results page
feat: Add mention of page status in data loading error hint

Waiting until client approval.

LGTM 👍

[ ] Remove logo
[ ] Add LD Platform Status Page link (at the bottom) https://status.ldbar.ch/
[ ] Implement banner / error message with link to status page


NotImplementedError: Cannot copy out of meta tensor; no data!

Hi, I am trying to follow the notebook InteractiveManual.ipynb
I have downloaded both llama-13b-hf and vicuna-13b-delta-v1.1 locally.
I am using fschat-0.2.11 and transformers-4.28.1
GPU 2*48GB VRAM
While running this,
convert_llama_names("./decapoda-research/llama-13b-hf") !python -m fastchat.model.apply_delta --base ./decapoda-research/llama-13b-hf --target ./lmsys/vicuna-13b --delta ./lmsys/vicuna-13b-delta-v1.1
I am getting the following error while Loading checkpoint shards:
NotImplementedError: Cannot copy out of meta tensor; no data!
Any idea why this is happening?
Thanks for reading this.

I've never seen that error before.


Test newest DGL Version

Use the newest DLG version. Once we merge this in we should cut another espaloma release and get it onto conda-forge.

Codecov Report

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

Additional details and impacted files

This is ready for review and can be merged in before or after #187

I'd say merge this first; #187 should have people actually looking at the code.


Dynamic packet size

Allow dynamic allocation of buffer size so rebuilding library is not required if user needs greater than MQTT_MAX_PACKET_SIZE (128) byte packet size.
See discussion at https://github.com/knolleary/pubsubclient/issues/110

Hi, thanks for this. It is, on face value, a reasonable change. It's just needs me to have sufficient spare time to review in detail and go through the process of merging and releasing etc. Can't give you an outlook for that right now.

All understood.
Let me know if i can be of any assistance.
thanks,
dunk.
On Thu, May 4, 2017 at 2:06 PM, knolleary<EMAIL_ADDRESS>wrote:

Hi, thanks for this. It is, on face value, a reasonable change. It's just
needs me to have sufficient spare time to review in detail and go through
the process of merging and releasing etc. Can't give you an outlook for
that right now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/knolleary/pubsubclient/pull/282#issuecomment-299179337,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEzU8mWigKZxukuD3_bSeP5o0OqKzYalks5r2c06gaJpZM4NMh-Y
.

+1, also interested in dynamic packet size

Any news on this? Right now when using this library have to tell everyone to download a forked copy or hack the existing define in their library folder...

No news - will try to get to it in the next week.

+1, more than interested in dynamic packet size. Thx

+1

Interested as well.
Thanks

+1 Interested as well.
Thanks

yep +1 here, seems like some of my messages get lost if they are bigger than 128 bytes

+1 Please merge this! It's a completely unnecessary burden to ask people to adjust a compile-time setting to build a library when it's got a dependency on this library. Looking at the code it seems completely backward compatible.

Definitely interested in this. Thanks for the PR!

do i have to pull this code manually into my cloned repo.. not seeing this in the master branch

I'm curious if this is going to be merged? My project (http://github.com/ThisSmartHouse/CoogleIOT) relies on PubSubClient for it's MQTT and the lengthy topics the framework uses causes problems (specifically with my heartbeats of devices). Obviously we can side-step this issue with a compile-time flag but what can we do to actually get this dynamic allocation code into the library proper?

heavy plus for this feature (as well as for other similar configuration stuff)!..
forcing people to direct sources' modification means additional maintenance effort delegation to client code... so instead of just clicking "install" in a library manager to retrieve a new version, people have to keep an up to date overridden sources in their own libraries or frameworks... that might be quite disappointing... don't you think so?..
but anyway thanks for the great lib!..

Any news on this?
I am not very up to date on programming so using Arduino IDE  with a TTGO T-Display to display 240x135 pictures via MQTT using PubSubClient
Getting even this far has been a long winding slog for me so Please
Can anyone show me how to change the PubSubClient to allow #define  MQTT_MAX_PACKET_SIZE 64800
Thank you in anticipation.

At this point I don't understand why it hasn't been merged, other than the owner of the project just doesn't want to do it 👎

I agree this is a long overdue feature.
The challenge has always been finding the time and energy to do the work needed to accept a change like this - especially as it's a feature I have little personal need for. I've also been fairly preoccupied with one of my other OSS projects...
That said, I have recently started dusting off my hardware so I can test some of the proposed enhancements to the library. This one is high on the list... in fact, I may well have a play with it this evening...

What do you make to this possible alternative >
https://github.com/marvinroger/async-mqtt-client
Appreciate any comments

Well, it took me 3 years, but we got there eventually. Thanks for your patience @mrdunk.

Cool,  Thanks, AatVerzonden vanaf mijn mobiele telefoon-------- Oorspronkelijk bericht --------Onderwerp: Re: [knolleary/pubsubclient] Dynamic packet size (#282)Van: knolleary Aan: knolleary/pubsubclient Cc: Aat0304 ,Comment
Merged #282 into master.
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.

oh cool, just saw this made it in.
thanks knolleary!
i know it's hard to spend time maintaining stuff you don't have much use for.


Datagrid: column customiztion when header is React component

Package
Carbon for IBM Products
Description
I have one of the columns in datagrid as ReactElement e.g.
{
    id: 'info',
    Header: <ToggleTip />
}

I expect to have column listed in column customization. However it is not.
Component(s) impacted
Datagrid

https://carbon-for-ibm-products.netlify.app/?path=/docs/ibm-products-components-datagrid-columncustomization--docs
Browser
Chrome
@carbon/ibm-products (previously @carbon/ibm-cloud-cognitive) version
v2.34.0
Severity
Severity 2 = Aspects of design is broken, and impedes users in a significant way, but there is a way to complete their tasks. Affects major functionality, has a workaround.
Product/offering
Turbonomic
CodeSandbox or Stackblitz example
https://carbon-for-ibm-products.netlify.app/?path=/docs/ibm-products-components-datagrid-columncustomization--docs
Steps to reproduce the issue (if applicable)

create datagrid where one of the columns header is react component
open column customization

Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[ ] I checked the current issues for duplicate issues

related thing is happening with table cells and header, on hover if value is not plain text it shows [object Object]

I would expect ability to control on hover text

It's because some components provided to Header field might lack children prop.
They are filtered-out from the draggable list.
https://github.com/carbon-design-system/ibm-products/blob/de9c862ad77ef955ee7b57a113be4e832d2dca22/packages/ibm-products/src/components/Datagrid/Datagrid/addons/CustomizeColumns/DraggableItemsList.js#L47
https://github.com/carbon-design-system/ibm-products/blob/main/packages/ibm-products/src/global/js/utils/getNodeTextContent.js#L38
As a workaround, it's possible to turn off TypeScript error and provide children prop to React element anyways (even though it does not use/support it).
Before:
        return {
            id: 'unparkable',
            Header: (
                <SomeComponent
                    someProp={{
                        children: <Information size={20} aria-label='Info' />,
                        label: '',
                    }}
                    otherProp='bottom'
                />
            ),

After:
        return {
            id: 'unparkable',
            Header: (
                // @ts-expect-error - providing `children` prop is required to make column customization work
                <SomeComponent
                    someProp={{
                        children: <Information size={20} aria-label='Info' />,
                        label: '',
                    }}
                    otherProp='bottom'
                >
                    Label
                </TurboToggleTip>
            ),

We've been investigating TanStack Table, a third-party, open-source offering, which provides extensive data table capabilities surpassing what our Carbon Datagrid offers. It provides much more flexibility and customization. TanStack Table is headless which means it can easily be added alongside Datagrid component in your product or application. The benefits of more flexibility for product teams and less maintenance for Carbon makes it a win win. Lastly, it is available in multiple frameworks including React and Web Component so it provides an option to non-React product teams.
For these reasons, we have decided to transition from building our own custom table component to using an example-based approach with TanStack Table. Datagrid will still exist in our library for existing teams but we are announcing the deprecation* of the Datagrid component in v2.54.0 release so teams can begin to work through the transition. Details about how to use both Datagrid and TanStack together can be found here.
*Deprecation means that no new features will be added however sev 1 and sev 2 bugs will be supported.


Be able to ignore ports to expose in a katenary.io/ports list

Sometimes when declaring multiple ports with the  katenary.io/ports label, all of the list might not need to be exposed.
eg
We declare the following ports
katenary.io/ports: 80,443,8080,9000
In a PHP-Nginx configuration, the 9000 port shouldn't b exposed as it is only used internally.
We could ave a second label that is used to "declare" the internal ports, so it is not exposed

I wonder how we can declare this.
At this time we declare ports to be in "containerPort" in pods and "port" in services.
I propose:
katenary.io/ports: N
katenary.io/containerPorts: M

if "ports" is declared, it creates the containerPort in pods and the port in service
if "containerPorts" is declared, it only creates ports in pods but not in service
if both, so merge the definitions

Right ?


Please release 0.3.8 tag to be able to enter into official homebrew.

See: https://github.com/Homebrew/homebrew/pull/39987/files#diff-8c3e3bae4209ea806fdd9b449a582314R6

Hi @pr0d1r2
Tag 0.3.8 is now available.
Leave this issue open until you merge your pull request with the official homebrew repo.

Closing, since this issue is outdated.
@pr0d1r2 you can open a new issue and we can discuss the details there if you are still intrested.

@MrS0m30n3 sorry, no time for this right now :(


[feat] Async validate when using multiple validators

Description
When you are using multiple validators for an industry usecase, time is crucial. When you are using 5-6 validators, if the each validator takes a couple of seconds, the total reponse time might as well make this a response waiting game(30s to 1 min). Could you please add the async function to the validators so that we we can decrease latency
Why is this needed
[If you have a concrete use case, add details here.]
Customer facing chatbot which needs to be guardrailed on multiple validators (restricting topics, prompt injections, competitor checks etc)

Hi @Samartha27
Guardrails provides an interface for async validation execution via the AsyncGuard class and GUARDRAILS_PROCESS_COUNT environment variable for parallel processing of validators. Each validator does need to have run_in_separate_process set to true for it to be handled properly.  I've included an example code snippet below.
import os
from guardrails import AsyncGuard
from guardrails.hub import DetectPII, CompetitorCheck, ToxicLanguage

os.environ["GUARDRAILS_PROCESS_COUNT"] = 4 # default is 10
guard = AsyncGuard()
pii_validator = DetectPII(pii_entities=['PERSON'], on_fail="fix")
pii_validator.run_in_separate_process = True
competitor_validator = CompetitorCheck(competitors=["Apple"], on_fail="fix")
competitor_validator.run_in_separate_process = True
toxic_validator = ToxicLanguage(on_fail="fix", threshold=0.5)
toxic_validator.run_in_separate_process = True
guard.use_many(
    pii_validator,
    competitor_validator,
    toxic_validator
)
response = await guard.parse('this is a sentence about bananas and apples and Bill Gates. I love Apple.

We recently made some additional improvements in this area.
See the docs here: https://www.guardrailsai.com/docs/concepts/concurrency
And the announcements here:
https://www.linkedin.com/posts/guardrailsai_huge-updates-to-concurrency-in-guardrails-activity-7242933455297134592-f_Ws
https://x.com/guardrails_ai/status/1837165867991240736


Styling overhaul (Sheets & Foundry UI) - Ongoing effort / discussion

A holding place to discuss any ongoing styling overhauls for v1.2.

Something like this would be great as right now my custom UI clashes with the system, being able to choose color codes could make matching super easy.

I am always open if you wish to recover my personalised style.
It is up to date and perfectly functional, the French-speaking community uses it and helps me to keep it in good shape. In any case good luck and thank you for the latest improvements. 😊

Could we get the social skills make into a category and put between Combat and Knowledge, then the two columns could be even?

You can do that already by using a custom skill set, and adding new categories.  I think it's best if we stick to the format in the book but allow others to change it as they want.

We now has an alternate UI, and can add more.  Looking to set Mandaar's theme as the default.

We now has an alternate UI, and can add more.  Looking to set Mandaar's theme as the default.


pr-checker.yaml

Description of the Issue
This PR addresses the issue where the GitHub Action workflow fails with the error:
Error: cannot read a block mapping entry; a multiline key may not be an implicit key

Issue Reference
Fixes #228
The error occurs because the re.match regular expression in the Python code block is split across multiple lines, which YAML does not interpret correctly.
Fix Summary
The issue was fixed by modifying the re.match pattern in the YAML file, combining it into a single line within the run command. This ensures YAML can correctly parse and execute the Python script within the GitHub Action.
Changes Made

Updated the re.match pattern in the Check PR Description and Title step to be on a single line.
Verified that the YAML file now correctly processes the multiline Python command without errors.

@Mayureshd-18 I have linked the PR with the respective issue. Now you can check and verify.


Fix indentation mistake

Found this while tracking down another bug.
I suspect this has been masked by the scheduler only passing in one account at a time and python "leaking" variables out of for loops.
Not a big issue but could cause unexpected behavior.

Looks like a good catch.


Suggestion: Additional hopper types.

Blast furnace hopper: Dual output that snaps to the inputs on the blast furnace and automatically filters coal and not coal appropriately. Allowing a single chest to be placed above, holding both fuel and ore. I suggest it as coal and not coal to allow compatibility with Tasty Chicken Legs https://valheim.thunderstore.io/package/TastyChickenLegs/BlastFurnaceTakesAll/
Large hopper: A 2x1m hopper to make chicken egg farm egg collection more straight forward.
Dual output hopper: Dual outputs on opposite sides, with round-robin output to allow evenly distributed routing to multiple destinations.
Hope these make sense. :)

Thanks for the suggestions but it's rather unlikely there will be many other hopper types. The current models where made by Bento, a fellow modder who is no longer active in the Valheim scene. Also, from a game design perspective I don't necessarily want to add over complicated hoppers as I think a lot of the solutions can already be build with a bit of creativity.
Nevertheless, the next update will also feature hopper pipes and a better distribution system of items. This will hopefully be achieved with with the combination of behavior difference between hopper/pipes and perhaps a round-robin distribution by normal hoppers, if it works good.
We can keep this feature request open, maybe there are more arguments for additional hopper types that I don't see at the moment. But it will not be a priority at the moment.

Simple suggestion for pipes:
A square 4 meter pipe that would allow vertical or horizontal movement of things.
Could have one horizontal pipe and 2 vertical pipes  (one up and one down).
if you need help with models I could try and help.

To give a small update here: Update 1.0.0 introduced basic pipes along with an updated distribution system, though it does not include round-robin distribution.
I'm currently working with an artist on new pipe models, as well as implementing the round-robin distribution and filtering for pipes.

On round-robin distribution, how about a t-junction piece? A 2-metre pipe, where both ends are outputs and there are input snap point at the mid-section of the pipe on all faces. I am desperate for round-robin to allow for larger scale kiln/smelter/furnace automation.


Manage more cases in universal link

Added support of:

room aliases that need to be resolved by the HS
room ids not yet known by the app because it is not sync'ed yet. In this case, we wait for the sync before handling the link
the case where is not yet logged in or registered. Once done, the universal link will be opened
/register/: we do nothing as we suppose the app is already in the auth screen

Coming soon: email invitation link

LGTM, but please check my remark about public rooms


feat: add support for additional properties in object schemas

This PR adds additionalProperties keyword on the generated schema, when Joi object is created with unkown(false) option

Coverage decreased (-0.03%) to 99.404% when pulling<PHONE_NUMBER>11351900f2df1b1f68f371693cc3c1 on igorsechyn:add_support_for_additional_properties into c02dd0dc8632766006854d1a4ddcd88a77a411c8 on glennjones:master.

Any chance this PR can be reviewed?

Adding links to swagger spec for future ref:
https://swagger.io/docs/specification/data-models/dictionaries/

A dictionary (also known as a map, hashmap or associative array) is a set of key/value pairs. OpenAPI lets you define dictionaries where the keys are strings. To define a dictionary, use type: object and use the additionalProperties keyword to specify the type of values in key/value pairs. For example, a string-to-string dictionary like this:

type: object
additionalProperties:
  type: object
  properties:
    code:
      type: integer
    text:
      type: string

@igorsechyn i'll try and review asap.

https://swagger.io/specification/

additionalProperties - Value can be boolean or object. Inline or referenced schema MUST be of a Schema Object and not a standard JSON Schema. Consistent with JSON Schema, additionalProperties defaults to true.


After Winter 20 release Non-selective query error

Ever since the winter 20 release we randomly are getting System.QueryException:
Non-selective query against large object type(more than 200000 rows). Consider an indexed filter or contact salesforce.com about custom indexing... (The whole error message is attached)
I am not sure where to look for the issue because none of the filter fields are null on the executed record. Has anyone else run into this issue and have any advice?
Thank you.
Tami

Speaking with Andy,  Winter '20 would not cause this issue, but you could have just coincidentally gone over the limit around the same time.  If you can provide more details, about the object size and query that may help.

@tamilust let us know if indeed this was a progressive data growth issue? Per @JimBTek  message above, if you can provide some more info we can help you determine if so. Thanks. 👍


RethinkDB Connection Issue

I am using Linux and my console is getting spammed with this error
"Fail to create a new connection for the connection pool. Error:"msg":"Failed to connect to localhost:28015 in less than 20s","message":"Failed to connect to localhost:28015 in less than 20s.","isOperational":true"
even though rehinkdb is up and when I go to make a request through the DB it works
So that error makes 0 sense because it isn't an actual error.

@Jayden000 You can test if your Rethink instance is up and listening with netcat.
What does nc -zv localhost 28015 give you?
If anything is listening on localhost:28015 you should get: Connection to localhost 28015 port [tcp/*] succeeded!

Talk to a Dev on the Live support page so that your issue can be looked into. CLICK HERE


Export From Rviz

Hi, Thank you for your excellent work.
I am running a SLAM program using Rviz based on ROS to show the points cloud. I want to save the showed map in Rviz, is there a method to save the points showed directly?
Looking forward to hearing from you soon!

No, there is no saving option. rviz is a real-time visualizer. Which point in time should be saved?
If you want to save your data consider rosbag.

No, there is no saving option. rviz is a real-time visualizer. Which point in time should be saved?
If you want to save your data consider rosbag.

I'd like to save the final output of the result, that is, the showed points after running, could you recommend a rosbag method for it? Thank you very much!

Sorry, not sure what you are referring to. Are you talking about the rviz display accumulating points that are sent incrementally by your PCL publisher and you want to save the final result, i.e. all of them? I don't know of an option to do this. First question would be, which file format do you expect?
I suggest writing a simple python script accumulating the data and writing them to your preferred file format.

Sorry for the unclear question. Because I use '--img_fov' so there is only a few points in front of ego car, I want to view the whole lidar points, now I just delete the '--img_fov', it seems that the whole 360-degree view is presented.


Increase default QPS and Burst value to improve slow startup duration.

:bug: This patch increases default QPS and Burst value to improve slow startup duration.
Release Note
The default `kube-api-qps` and `kube-api-burst` are set to `200`. You can adjust them via env variable if necessary.

/cc @skonto @maschmid

/assign @skonto

Do you have a linked issue describing the slow startup?

There was originally  https://github.com/knative-sandbox/net-kourier/pull/879 , which was eventually closed as stale.

Out of curiosity which outbound calls to the K8s API server are being throttled?

I believe it was services and endpoints gets, done by the IngressTranslator

Is there a reason those calls aren't using informers?

I was not involved this discussion in https://github.com/knative-sandbox/net-kourier/pull/276 but the client correctly list all resources at startup.
https://github.com/knative-sandbox/net-kourier/blob/790358f73d01d37c908df5c7c54a7e275c22219e/pkg/reconciler/ingress/controller.go#L213-L214
I researched the context before as https://github.com/knative-sandbox/net-kourier/issues/968#issuecomment-1427260209 but it was a little bit complicated and still open https://github.com/knative-sandbox/net-kourier/issues/968 (it didn't intend to solve this informer vs client but I think related) for that.

Finding good default values is an open question to me (depends on the scale we support). If we can't avoid the direct K8s api server calls then I don't see many options here. In general this is a problem faced elsewhere too see: https://github.com/kedacore/keda/issues/3730 & the fix https://github.com/kedacore/keda/pull/3731.

I think it's not necessary to exert effort in finding the good default values as users can easily adjust them.
And we will actually turn off them in the future https://github.com/knative/pkg/issues/2756 so bigger value like 200 should be alright for now, I think.

@skonto @dprotaso
Could you merge this before 1.11 cut if you are alright?

Yes, that's correct. The one "the listing to fill the informer cache is the slow piece right now", I need to test and confirm it by myself but I believe there is no change since https://github.com/knative-sandbox/net-kourier/pull/276.

Afaik getReadyIngresses is the slow part that lists ingresses (with a big number things get stuck). This call access the api server directly https://github.com/knative-sandbox/net-kourier/blob/main/pkg/reconciler/ingress/controller.go#L317.
@nak3 could you confirm the last bits so we can merge.

getReadyIngresses also calls k8s client but the critical part is startupTranslator which lists services, endpoints and etc as @maschmid answered above https://github.com/knative-sandbox/net-kourier/pull/1066#issuecomment-1604316418
Please see the log in https://github.com/knative-sandbox/net-kourier/issues/840 or (SRVKS-1078 our internal issue). It prints Priming the config with ... so getReadyIngresses passed without stuck.

Yeah correct now I remember ;) Ok so should we merge? Are there any objections?

I'm confused why doesn't startupTranslator use an informer?

Hmm... I tested https://github.com/knative-sandbox/net-kourier/pull/1075. It works and the start speed is fast on 200 ksvcs.
I'm not sure what issue observed with the informer when https://github.com/knative-sandbox/net-kourier/pull/276 was created (the comment line clearly mentions The startup translator uses clients instead of listeners to correctly list all resources at startup. but it works on the current net-kourier.
Should we merge https://github.com/knative-sandbox/net-kourier/pull/1075 instead of here?


How to migrate easy from JFlex?

I am looking for JFlex successor, a project that would be maintained and where
performances will be significantly better the JFlex.

I have flex files for JFlex, how easy is to migrate to cookcc?

How mature the current version is?

Thank you!

Original issue reported on code.google.com by igor.spasic on 10 Dec 2013 at 12:42

CookCC is pretty stable.  I use it a lot to write utilities at work.
As for JFlex, I do not plan to write any tools for it.  There is no point and no reason and no point to do so.  The file format is archiac and difficult to read / maintain.
It would be far better for you to migrate to Java annotation based approach used by CookCC which is far superior in terms of maintenance and ease of use.


Feature/mezzanine admin order

Integrates mezzanine & harmonizes with stock grappelli (borrowing the same approach we've used for derrida & ppa), and then uses mezzanine's ADMIN_MENU_ORDER to customize admin list display.

@bwhicks I've adapted the mezzanine integration we've used twice already, but would appreciate thoughts on how to generalize this for future.  (Here's the ppa commit I referred to as I worked on it: https://github.com/Princeton-CDH/ppa-django/commit/46b088ef456bffe5726dbce69fc4e3dc822a45fa )
I thought Mezzanine's ADMIN_MENU_ORDER was a good choice for customizing the display order on the admin dashboard, but it does bother me that things are hard-coded and we'll have to remember to update them as we add new models - so would appreciate thoughts on that also.

Codecov Report

Merging #69 into develop will decrease coverage by 0.13%.
The diff coverage is 77.77%.

@@             Coverage Diff             @@
##           develop      #69      +/-   ##
===========================================
- Coverage    97.68%   97.55%   -0.14%
===========================================
  Files           69       69
  Lines         2721     2739      +18
===========================================
+ Hits          2658     2672      +14
- Misses          63       67       +4

Impacted Files
Coverage Δ

mep/urls.py
77.77% <60%> (-22.23%)
:arrow_down:

mep/settings.py
83.67% <84.61%> (+0.34%)
:arrow_up:

Continue to review full report at Codecov.

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

@bwhicks thanks for the review and catching those little things.  Just pushed some corrections.  I'm not sure what magic mezzanine does in the optional package handling, but seems to work fine if I remove all the other debug_toolbar configs and let mezzanine set it up.
I was also thinking that the mezzanine + grappelli set up probably makes sense as a django project template.  I'm not sure how soon we need it, since I'm not aware of any new projects we'll be starting soon, but it would be nice to have it ready when we want it, so I'll make a note.


After enable ipv6 in the docker, it works for only static ipv6 address, but not DHCP ipv6 address

Warning
libnetwork was moved to https://github.com/moby/moby/tree/master/libnetwork
libnetwork has been merged to the main repo of Moby since Docker 22.06.
The old libnetwork repo (https://github.com/moby/libnetwork) now only accepts PR for Docker 20.10,
and will be archived after the EOL of Docker 20.10.

Sorry to trouble all of you.
Recently, I follow the guidance to enable ipv6 for the docker in my linux machine
I directly edit the /etc/docker/daemon.json file for adding these parameters:
"ipv6": true,
"ip6tables": true,
"experimental": true,
"fixed-cidr-v6": "2001:db8:abc1::/64"
Then just restart the docker.
Then for example, here are two ipv6 addresses
static ipv6 address A
DHCP(dynamic) generated ipv6 address B
Case1:
When I use "ping6 A" in my linux machine, it works with response
Case2:
When I use "ping6 B" in my linux machine, it works but should have a zone identifier behind, like "B%eth0"
Case3:
when I use "docker exec -it myDocker bash" to get inside a docker, and then run "ping6 A", it works with response
Case4:
when I use "docker exec -it myDocker bash" to get inside a docker, and then run "ping6 B", it always throws the error message "ping6: sendto: Invalid argument"
Then I also try "ping6 B%eth0", it will be time out without any response.
For case4, I do not understand, for DHCP generated ipv6 address, why we cannot use it or ping6 it inside the docker? Why the docker cannot automatically assign a zone identifier to it? what is the problem here? how to fix it?
Thanks a lot. Really need your help.

Hi, could you please send the output of
ip -f inet6 addr show
From within the container and from the host?
Also, there is/was a problem with a /64, can you try again with a /80 ?
Kind regards,
Ed
On 3/24/23 09:52, Haochen Sui wrote:

*Warning*
libnetwork was moved to
https://github.com/moby/moby/tree/master/libnetwork

libnetwork has been merged to the main repo of Moby since Docker
22.06.

The old libnetwork repo (https://github.com/moby/libnetwork) now
only accepts PR for Docker 20.10,
and will be archived after the EOL of Docker 20.10.

Sorry to trouble all of you.
Recently, I follow the guidance to enable ipv6 for the docker in my
linux machine
I directly edit the /etc/docker/daemon.json file for adding these
parameters:
"ipv6": true,
"ip6tables": true,
"experimental": true,
"fixed-cidr-v6": "2001:db8:abc1::/64"
Then just restart the docker.
Then for example, here are two ipv6 addresses
static ipv6 address A
DHCP(dynamic) generated ipv6 address B
Case1:
When I use "ping6 A" in my linux machine, it works with response
Case2:
When I use "ping6 B" in my linux machine, it works but should have a
zone identifier behind, like "B%eth0"
Case3:
when I use "docker exec -it myDocker bash" to get inside a docker, and
then run "ping6 A", it works with response
Case4:
when I use "docker exec -it myDocker bash" to get inside a docker, and
then run "ping6 B", it always throws the error message "ping6: sendto:
Invalid argument"
Then I also try "ping6 B%eth0", it will be time out without any response.
For case4, I do not understand, for DHCP generated ipv6 address, why
we cannot use it or ping6 it inside the docker? Why the docker cannot
automatically assign a zone identifier to it? what is the problem
here? how to fix it?
Thanks a lot. Really need your help.
—
Reply to this email directly, view it on GitHub
https://github.com/moby/libnetwork/issues/2675, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPTWE4UTWU3YOEKML72E73W5VOGHANCNFSM6AAAAAAWGIS3KQ.
You are receiving this because you are subscribed to this
thread.Message ID: @.***>


Be sure that script is loaded

From your wiki, I've read:
Sometimes, on some browsers, scripts are not parsed yet when we want to use them (like calling a function from that script). To resolve that issue, toast provides a simple way:
toast(
    'scripts/jquery.js',
    function(){
        // The callback will be called until `$` is not set
        if(!window.$){
            return false;
        }
        // jQuery actions
        // .....
    }
);

When it could happen, I don't really understand. If loader provide callback function, it means, that callback will be called when loader has already load the script, than functions from this script are available.
So, I need if (!some_var_from_script){ return false;} protection in every script I load/

In fact, to know when a file is loaded we can use two functions :onreadystatechange, used by IE and Opera, and onload, used by IE>8 and all others.
And it begins!
Some browsers/versions hadn't parse code yet when the callback is called (based on onreadystatechange and onload states). Many libraries provide their own system to handle that behavior. With toast, I've chosen to let the developer handle it with a powerful way (but a bit redundant, I guess).
That system isn't always needed. It depends on what you want to load. And, even if we often just need to do a if(!window.lib) return false, sometimes that condition could be more complicated (like nest toast calls, verify multiple library load state, etc...).

Ok, I understand.
I've used loaders before, for example, I've used head.js and I had some  issue, that I couldn't fix:
Loader loads jquery, plugins, and other files (client-code). It works fine when I just open the site. But when I reload it (by pressing F5) - problems may occur - callback is executed early than $ is defined.
So, is it precisely this problem?
I think, I've try toast with if (!window.lib) return false protection some time later.

It could be the same problem, yes. Linked to the browser cache, per example. I don't know :)


stats: add tag extraction rules for ADS disconnections

Commit Message: adds tag extraction rules for envoy.ads_cluster.streams_closed.
Additional Description: as seen on #21595
Risk Level: low
Testing: removes skip_tag_extraction_rule_check_ from ADS integration tests.
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Maybe @adisuissa can help to review this PR, as xDS expert.

/wait

@adisuissa please let me know if you're happy with my changes. All CI checks passing now.


Add a Gitter chat badge to README.md

borisyankov/DefinitelyTyped now has a Chat Room on Gitter
@basarat has just created a chat room. You can visit it here: https://gitter.im/borisyankov/DefinitelyTyped.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Any objections? Desire showed up in https://gitter.im/Microsoft/TypeScript

Fine with me - merge away!


lab proposal for HL-StarterKit

Lab  proposal for HL Starterkit
Signed-off-by: Ravi Vasagam<EMAIL_ADDRESS>Checklist for Hyperledger Lab Proposal PR

[ ] Read README.md carefully
[ ] Make sure your changes are committed with the proper sign-off
[ ] Use the labs name as the PR title
[ ] Copy the short description of the proposal as the description of the PR
[ ] Remove this checklist section
[ ] Submit your PR and the labs stewards will review your proposal
[ ] Ask your sponsor to confirm sponsorship by stating so in a comment to the PR

I capitalized all project names,  Thank you

I capitalized all project names,  Thank you

@ravinayag : I notice that the branch trashmain on your repo is also missing some DCO signoffs. Can you correct and/or remove this branch and then transfer the HL-StarterKit GitHub repo to @tkuhrt, then I can transfer it to the Hyperledger Labs organization.

@ravinayag : I notice that the branch trashmain on your repo is also missing some DCO signoffs. Can you correct and/or remove this branch and then transfer the HL-StarterKit GitHub repo to @tkuhrt, then I can transfer it to the Hyperledger Labs organization.

https://github.com/hyperledger-labs/HL-StarterKit is now available. @ravinayag : Please accept the invite to the team for this lab.

https://github.com/hyperledger-labs/HL-StarterKit is now available. @ravinayag : Please accept the invite to the team for this lab.

@ravinayag : I notice that the branch trashmain on your repo is also missing some DCO signoffs. Can you correct and/or remove this branch and then transfer the HL-StarterKit GitHub repo to @tkuhrt, then I can transfer it to the Hyperledger Labs organization.

@tkuhrt : I removed that repo,(trashmain)  and  transferred it to @tkuhrt
@lehors : Could you confirm for me,  Do you still have a problem accessing it?

@ravinayag : I notice that the branch trashmain on your repo is also missing some DCO signoffs. Can you correct and/or remove this branch and then transfer the HL-StarterKit GitHub repo to @tkuhrt, then I can transfer it to the Hyperledger Labs organization.

@tkuhrt : I removed that repo,(trashmain)  and  transferred it to @tkuhrt
@lehors : Could you confirm for me,  Do you still have a problem accessing it?


The method authorizationRequestResolver(CustomAuthorizationRequestResolver) is undefined for   the type OAuth2LoginConfigurer.AuthorizationEndpointConfig

source code path : https://github.com/eugenp/tutorials/blob/master/spring-5-security-oauth/src/main/java/com/baeldung/oauth2/CustomRequestSecurityConfig.java
line no : 47
oauth2 maven dependencies:
https://mvnrepository.com/artifact/org.springframework.security.oauth.boot/spring-security-oauth2-autoconfigure/2.3.1.RELEASE
https://mvnrepository.com/artifact/org.springframework.security/spring-security-oauth2-client/5.3.3.RELEASE
https://mvnrepository.com/artifact/org.springframework.security/spring-security-oauth2-core/5.3.3.RELEASE
error : The method authorizationRequestResolver(CustomAuthorizationRequestResolver) is undefined for   the type OAuth2LoginConfigurer.AuthorizationEndpointConfig

Hi @PraveenBiradar I'm not able to reproduce the issue on our codebase. The method resolves correctly, as it's defined in the OAuth2LoginConfigurer.AuthorizationEndpointConfig class as shown here: https://docs.spring.io/spring-security/site/docs/current/api/org/springframework/security/config/annotation/web/configurers/oauth2/client/OAuth2LoginConfigurer.AuthorizationEndpointConfig.html#authorizationRequestResolver-org.springframework.security.oauth2.client.web.OAuth2AuthorizationRequestResolver-
This class is part of the spring-security-config dependency.


Update Colours Hindi .h5p file

This has the credits page added at the end that was previously missing.
https://drive.google.com/file/d/1trfs5RnyI9PGsKlyMsErxWZY3H59skE9/view?usp=drive_link
Send files to both development and prod S3 server buckets.

@burrage In content.json we don't have those meta data which will parse and renders the image at the end of slides, i have putted the images for now and we required to update the content.json file as well.

CC: @dz4va for extra insight into the process since he has done this many times before!
@ashishkrr I'm not sure I'm completely following.
The link in the original comment is an updated .h5p file. When we unpack that .h5p file, does the content.json file NOT have the meta data we need to add the final credits page?
Let me know!

@burrage As per my understanding, Yes content.json file NOT have the meta data we need to add the final credits page, i informed same to george, he is looking into it and verifying the same.

This file is added in dev version, but needs to be added in Prod version.


Create Framework

Plan out the design of the web application either using an online design tool or pencil sketch and take a picture

Made a sketch of the initial landing page design


Add opentracing-basic Java library

Adds a reference to the opentracing-basic bridge library/implementation that I spun up a while back.
Thanks!

@mcumings could you elaborate what you mean by "Basic OpenTracing bridge library"? Is this an equivalent of basictracer-go?

We generally used the term "basictracer" (https://github.com/opentracing?q=basic) to refer to a "reference" implementation that, for example, can be used to test real instrumentations. It is not performance, does not export spans anywhere (keeps them in memory), but it does provide on the wire context format.  Initially it was thought we could use it as the actual base implementation extensible by vendors, but at the time it was too early and there was no real interest from vendors in such impl.

In that case then it sounds like it matches pretty well.  The only delta being that the wire model is undefined in the opentracing-basic library as it is expected to be supplied by the consumer.

Maybe it makes sense to add the default codec that is compatible with Go implementation, then there will be parity in the two projects.
And after that, maybe it makes sense to move the repo to opentracing org

For now I'd prefer to keep it separate, if that's alright on your end.

Bump...


Different children with the same key in Datepicker

What:
Use generated ids as keys in Datepicker

Why:
When using the Datepicker component a warning appeared when the Datepicker opened due to multiple keys having the same value in the component that represents each day of the month. Check the screenshots to understand how the keys' values were.

How:

Generate an id for each day to ensure that the values are different

Media:

Checklist:

[ ] Ready to be merged

@martimalek do we merge?


Fix issue #3778 Profile picture and theme switch toggle button are overlapped.

Related Issue
None
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #3778
Description
Theme switch toggle button and profile picture were overlapped.
[Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.]
Issue no. #3778 Profile picture and theme switch toggle button are overlapped.

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
before

After

Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

Please merge this pr.

https://github.com/user-attachments/assets/567b4577-a8e6-46aa-8541-a54e4567018d
this page has many bugs and needs enhancement .
navbar alignment , the more dropdown is not visible ( u may remove it ) ,  fixing position of profile dropdown ,  explore new worlds section UI improvement .
u can work on these issues and create new pr .

Ok I will fix this.


Address many deprecation warnings

References

Fixes #2956

Description
Address many deprecation warnings.
Instructions for Reviewers
It may be easiest to review each commit separately, since I've tried to group all work by the deprecated code it addresses.
List of changes in this PR:

First, ...
Second, ...

Include guidance for how to test or review your PR. This may include: steps to reproduce a bug, screenshots or description of a new feature, or reasons behind specific changes.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[ ] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[ ] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[ ] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[ ] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

@mwoodiupui : Just a quick note...it looks like this has a single test failure in DiscoveryRestControllerIT.  There's also a minor, new alert that LGTM points out above in S3BitStoreService.  Once those are cleaned up though, I'll gladly give this a review.

@mwoodiupui : just a quick FYI...this needs rebasing on main as it has merge conflicts & also looks to have previously hit the Travis 50min build time limit

@mwoodiupui : Any plans to get this updated in the near(ish) future?  I'd love to include this in 7.0 Beta 5 (likely due in Dec or very early Jan), if it can get updated as the deprecation warnings are also a thorn in my side.  If you aren't able to get to this soon, let me know & maybe I can find time in the coming month to help move this work forward.

There are a few deprecations which are non-trivial and probably should be separate issues:

org.dspace.core.Context implements #finalize().  Current best practice seems to be to use a Cleaner instead.
org.dspace.eperson.service.EPersonService and .GroupService have #findAll methods which are deprecated in favor of paginated versions.  In the cases that I've examined, pagination would just clutter the caller's code because it simply goes straight down the List in one swoop.  May I suggest a findAll variant that returns an Iterable and manages the pagination internally.
org.dspace.browse.BrowseIndex has several deprecated #getTableName methods which seem to be used only in a single database migration.  I don't know why they are deprecated.  I think that this needs discussion.

I'm inclined to say this is finished, even though there are a few deprecations left.  As noted above, they are more intrusive or require discussion.

Cleaning up all of these warnings has exposed more complaints from errorprone, so the build output is still huge.  I've begun looking at those.


Missing Cache-Control Headers for Password-Protected Pages

Issue Summary
When serving password-protected pages in Wagtail, the application does not set appropriate cache-control headers to prevent these pages from being cached by browsers or intermediary proxies like Squid. This can lead to security issues where protected content may be inadvertently exposed to unauthorized users through cached content.
Steps to Reproduce

Start a new project with wagtail start myproject.
Create a new page (e.g., ProtectedPage) and publish it.
In the Wagtail admin interface, add a password restriction to the page:

Go to the page in the admin.
Click on "Settings" > "Privacy".
Select "Private, accessible with the following password" and set a password.

Access the password-protected page in a web browser.
Enter the correct password when prompted.
Use browser developer tools or a tool like curl to inspect the response headers.

Observed Behavior:

The response headers do not include Cache-Control, Pragma, or Expires headers to prevent caching.
As a result, the password-protected page may be cached by the browser or intermediary caches.

Expected Behavior:

The response should include headers that prevent caching, such as:

Cache-Control: no-store, no-cache, must-revalidate, max-age=0 Pragma: no-cache Expires: 0

This ensures that password-protected content is not cached and remains secure.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: Yes

Working on this
I am willing to work on this issue and submit a pull request. My proposed solution involves adding appropriate cache-control headers to the HTTP response to prevent caching.
Additional Notes:

The lack of cache-control headers on password-protected pages poses a security risk, as sensitive content might be stored in browser caches or intermediary proxies.
Implementing this fix will enhance the security of Wagtail applications by ensuring that protected content is not inadvertently exposed.

Proposed Solution:
from django.utils.cache import add_never_cache_headers

@hooks.register("before_serve_document")
def check_view_restrictions(document, request):
    """
    Check whether there are any view restrictions on this document which are
    not fulfilled by the given request object. If there are, return an
    HttpResponse that will notify the user of that restriction (and possibly
    include a password / login form that will allow them to proceed). If
    there are no such restrictions, return None
    """
    for restriction in document.collection.get_view_restrictions():
        if not restriction.accept_request(request):
            if restriction.restriction_type == BaseViewRestriction.PASSWORD:
                from wagtail.forms import PasswordViewRestrictionForm

                form = PasswordViewRestrictionForm(
                    instance=restriction,
                    initial={"return_url": request.get_full_path()},
                )
                action_url = reverse(
                    "wagtaildocs_authenticate_with_password", args=[restriction.id]
                )

                password_required_template = getattr(
                    settings,
                    "WAGTAILDOCS_PASSWORD_REQUIRED_TEMPLATE",
                    "wagtaildocs/password_required.html",
                )

                if hasattr(settings, "DOCUMENT_PASSWORD_REQUIRED_TEMPLATE"):
                    warn(
                        "The `DOCUMENT_PASSWORD_REQUIRED_TEMPLATE` setting is deprecated - use `WAGTAILDOCS_PASSWORD_REQUIRED_TEMPLATE` instead.",
                        category=RemovedInWagtail70Warning,
                    )

                    password_required_template = getattr(
                        settings,
                        "DOCUMENT_PASSWORD_REQUIRED_TEMPLATE",
                        password_required_template,
                    )

                context = {"form": form, "action_url": action_url}

                response = TemplateResponse(request, password_required_template, context)
                add_never_cache_headers(response)
                return response

            elif restriction.restriction_type in [
                BaseViewRestriction.LOGIN,
                BaseViewRestriction.GROUPS,
            ]:
                return require_wagtail_login(next=request.get_full_path())

This sounds like an issue to me, and something Wagtail should do better.
I think your proposed solution makes sense too. Would you be interesting in opening it as a PR?

@RealOrangeOne I have just created a PR.

Assigning to @cnk to take a look. Can you also consider #1111 and whether we might be able to work on that at the same time?


Initial Focus causes double requests when loading app (in web)

I found this code to not be true for us:
const focusCount = useRef(
  Platform.select({
    // react-navigation fire a focus event on the initial mount, but not on web
    web: 1,
    default: 0,
  })
)

We are using:
"@react-navigation/native": "^6.0.4",
"@react-navigation/stack": "^6.2.1",

This causes our initial load of the page on web to perform duplicate requests for each SWR hook.
I have confirmed that the duplicate request is due to the onFocus listener being called immediately on page load. I searched through react-navigation issues and could not identify anything that specifically spoke to this change being made (to fire a focus even on the initial mount on web), although there were some around re-ordering focus and blur events that could have caused this change.
At the very least, I am hoping this setting can become configurable, so we can continue to use the library. In the meantime I think we will have to fork.

Has anyone else had this issue?


Trying to help you with debugging AUTH on master branch

I see you made some changes to "fix AUTH", but for my example, AUTH is broken both on the release, and on master branch.  Here is a rough example code of the basics of what I'm attempting here:
// Top level constants
const nip07signer = new NDKNip07Signer();

const ndk = new NDK({
    signer: nip07signer,
    autoConnectUserRelays: false,
    enableOutboxModel: false,
});

// inside UseEffect
const ndkPool = ndk.pool;
            ndkPool.on("flapping", (flapping: NDKRelay) => {
                addToStatus("relay is flapping: " + flapping.url);
            });
            ndkPool.on("relay:auth", (relay: NDKRelay, challenge: string) => {
                addToStatus("auth: " + relay.url);
            });
            ndkPool.on("relay:authed", (relay: NDKRelay) => {
                addToStatus("authed: " + relay.url);
                const kind1Sub = ndk.subscribe({ kinds: [1], limit: relayLimit }, {closeOnEose: false});
                kind1Sub.on("event", (event: NDKEvent) => {
                    if(lookupProfileName(event.pubkey) == event.pubkey) {
                        const profileSub = ndk.subscribe({ kinds: [0], limit: 1, authors: [event.pubkey] }, {closeOnEose: true});
                        profileSub.on("event", (pevent: NDKEvent) => {
                            addProfile(pevent);
                        });
                    }
                    addPost(event);
                });
            });
            ndkPool.on("relay:disconnect", (relay: NDKRelay) => {
                addToStatus("disconnected: " + relay.url);
            });

            ndkPool.on("relay:connect", (relay: NDKRelay) => {
                addToStatus("connect: " + relay.url);

            });

            ndkPool.on("relay:connecting", (relay: NDKRelay) => {
                addToStatus("connecting: " + relay.url);
            });

            ndk.addExplicitRelay(nrelaydata, NDKRelayAuthPolicies.signIn({ndk}), true);

So, here's what I found:
First of all, just want to mention that the following did NOT work, although it seems like it should have.  Having the two different policies (one for the pool and one per relay) may just not apply to explicit relays or etc, I don't know but the if statement to select either or, is not working apparently:
ndk.relayAuthDefaultPolicy = NDKRelayAuthPolicies.signIn({ndk});
ndk.addExplicitRelay(nrelaydata);
await ndk.connect();

However, after some digging I managed to get farther by passing a policy and connect=true to addExplicitRelay.
Second of all, the nitty gritty debug.  AUTH was still not working, I could see the auth message come through on the relay but the relay itself inside NDK is stuck in "connect" (ie, it connected then doesn't proceed).  Here's why:
On line 82 of connectivity.ts this if statement was returning false when it should be true.  res when inspected, is not a bool, it is an object.  I naievely hotwired this to if (!!res === true) and this got farther down the code path.  At this point, the signed event had already arrived at the relay and should be clear to proceed.
The next error happens on line 83 of connectivity.ts it fails saying "No signer available for authentication".  This is not the case, it has already signed the AUTH event and sent it off to the relay at this point so clearly it has a signer.  This is NIP07 signer.  So I hotwired this as well by commenting it out and jumping straight to the if statement on line 85 of connectivity.ts changing it from an else to an if (this._status === NDKRelayStatus.AUTHENTICATING) which returns true.
This allowed NDK to continue and :boom: my code started working.  I am receiving reqs, and the pool is now emiting all the statuses I expected to see and subscribing etc.
I hope this helps you figure out what mistakes are happening here!  My hot-wiring is obviously not PR ready or anything as I still need to understand more about some of this.  I have a feeling that bunkers vs. nip07 and possibly just some bad if's are the culprit here.

could be that we're not setting the NDK here when addExplicitRelay is called:
ndk/src/ndk/index.ts line 327

       relay = new NDKRelay(urlOrRelay, relayAuthPolicy);

                  relay = new NDKRelay(urlOrRelay, relayAuthPolicy, this);

Ok, I think I am starting to understand what's going on here.  (Or at least guess..)
It appears to me that the first part of the auth handler was meant for NIP07 use.. and it's simply not setting the status to connected and emitting auth.  So when i back out my hotwiring, and just add these things, I think this is working also and that the second part of the authHandler, where it asks if res===true, that must be for bunker signing.

Like this:
                    if (res instanceof NDKEvent) {
                        this.relay.auth(async (evt: EventTemplate): Promise<VerifiedEvent> => {
                            return res.rawEvent() as VerifiedEvent;
                        });
                            this._status = NDKRelayStatus.CONNECTED;
                            this.ndkRelay.emit("authed");
                    }

Turns out I was just confused about all this stuff.  I have backed out all my changes to NDK, and things are working.  I even went back to latest released version 2.8.2 and it's also working.  I learned a LOT about the internals of NDK, so that's a win.  I am enjoying these grouped subscriptions and auth features.  Thank you.
Here's some example code of what finally seems like a pattern that works for me.  I have to delay the subscription until after the auth for it to work.  I also have to make sure to close the subscription on disconnect and re-create it.  Overall, I'm very happy with this now.  PV
const nip07signer = new NDKNip07Signer();

const ndk = new NDK({
    signer: nip07signer,
    autoConnectUserRelays: false,
    enableOutboxModel: false,
});

const ndkPool = ndk.pool;

export default function PostsPage() {

async function grabStuff(nrelaydata: string, auth: boolean = false) {
        var kind1Sub: NDKSubscription

        ndkPool.on("flapping", (flapping: NDKRelay) => {
            addToStatus("relay is flapping: " + flapping.url);
        });
        ndkPool.on("relay:auth", (relay: NDKRelay, challenge: string) => {
            addToStatus("auth: " + relay.url);
        });

        ndkPool.on("relay:authed", (relay: NDKRelay) => {
            addToStatus("authed: " + relay.url);
            wipePosts();
            kind1Sub = ndk.subscribe({ kinds: [1], limit: relayLimit }, {closeOnEose: false, groupable: false});
            kind1Sub.on("event", (event: NDKEvent) => {
                // do profile lookups on the fly
                if(lookupProfileName(event.pubkey) == event.pubkey) {
                    const profileSubAuth = ndk.subscribe({ kinds: [0], authors: [event.pubkey] }, {closeOnEose: true, groupable: true});
                    profileSubAuth.on("event", (pevent: NDKEvent) => {
                        addProfile(pevent);
                    });
                }
                addPost(event);
            });
        });

        ndkPool.on("relay:disconnect", (relay: NDKRelay) => {
            kind1Sub.stop()
            addToStatus("disconnected: " + relay.url);
        });

        ndkPool.on("relay:connect", (relay: NDKRelay) => {
            addToStatus("connected: " + relay.url);
            wipePosts();
            if(!auth) {
                kind1Sub = ndk.subscribe({ kinds: [1], limit: relayLimit }, {closeOnEose: false, groupable: false});
                kind1Sub.on("event", (event: NDKEvent) => {
                    // do profile lookups on the fly
                    if(lookupProfileName(event.pubkey) == event.pubkey) {
                        const profileSubAuth = ndk.subscribe({ kinds: [0], authors: [event.pubkey] }, {closeOnEose: true, groupable: true});
                        profileSubAuth.on("event", (pevent: NDKEvent) => {
                            addProfile(pevent);
                        });
                    }
                    addPost(event);
                });
            }
        });

        ndkPool.on("relay:connecting", (relay: NDKRelay) => {
            //addToStatus("connecting: " + relay.url);
        });

        ndk.addExplicitRelay(nrelaydata, NDKRelayAuthPolicies.signIn({ndk}), true);
    }

useEffect(() => {
        grabStuff(nrelaydata, useAuth == "true");
    }, []);
}

Thanks for all this! I am writing documentation on the docs directory with tutorials and stuff like that
I would absolutely love it if you could spend some time to contribute some docs on what could have helped you understand how all this works; I am very close to NDK's inténtale (I wrote them! 😂) so there's probably a lot of stuff that you can see that I can't because of that
What's your npub btw?

Thanks for all this! I am writing documentation on the docs directory with tutorials and stuff like that
I would absolutely love it if you could spend some time to contribute some docs on what could have helped you understand how all this works; I am very close to NDK's inténtale (I wrote them! 😂) so there's probably a lot of stuff that you can see that I can't because of that
What's your npub btw?

No problem, I'd be happy to contribute.  Let me know if you have a branch you're currently working on for the docs or just main branch.  Some of the things from code snippits in the code comments did not work for me, so I will reproduce and make sure those are addressed.  I'm still learning a lot (i filed a separate issue for I can't even seem to turn on debug haha).  It looks like I'm going to be rolling with ndk for my project so contribution is the name of my game and I'll keep flowing with the issues, PRs and etc. :)  I'll tag you on nostr so you have my npub.

I'm writing docs and tutorials on the /docs directory and they are deployed here https://nostr-dev-kit.github.io/ndk

Cool, while testing this morning I realized I had *not backed out all changes, and that auth actually is broken in master.  I will corral and test these changes and push them up.  PR incoming..

While this commit does un-stick authentication for a successful authentication.  When authentication fails (relay rejects the auth) an error is thrown:
index.js:773 Uncaught (in promise)
Error: auth-required: invalid auth received
    at Relay.handleNext (￼index.js:773:23)
    at Relay.runQueue (￼index.js:712:26)
    at Relay._onmessage (￼index.js:852:12)

I will need to think about this further, as I would like to be able to detect authentication failure and show a nice message to the user.  Any thoughts appreciated.  I think this error is coming from nostr-tools..

A bonus of catching this error is that it doesn't infinitely keep asking the user to sign these auth requests.  It fails once, and then doesn't ask again..


make locale storage path segments configurable

if you want to change the path or segments you can now add the following to the config.yml
sulu_media:
    storage:
        local:
            path: "%kernel.root_dir%/../uploads/media_test"
            segments: 10

Tasks:

[ ] test coverage
[ ] gather feedback for my changes
[ ] submit changes to the documentation
[x] added changelog line

Informations:

Q
A

Tests pass?
none

Fixed tickets
none

BC Breaks
no

Doc
none

ack


Crashing to Desktop with latest version of GW2

Describe the bug
After 5-15 minutes of play, the game crashes to desktop.
To Reproduce
Steps to reproduce the behavior:

Install the mod as usual
Play for a bit (I was hitting things, I didn't even use the mount menu)
Crash to Desktop, no error.

Expected behavior
Not crashing to desktop
Screenshots
My wallpaper is boring :)
System configuration (please fill in):

OS: Windows 10
GPU: GTX 1070 (latest nvidia drivers)
Was chainloading with arcdps's built-in chainloader, then removed arcdps to only use this (I'd rather have this).  Neither configuration worked.

Additional context
N/A

So I'm looking into these new build issues, but thus far I haven't had a crash using the latest dev build. Mind trying with that? Here it is: d3d9.zip


Go back to LoggerFactory temporarily

needed to unblock Hosting PR: https://github.com/aspnet/Hosting/pull/146
Service fallback currently is broken with generic services (ILogger<> no worky)

Hosting issue tracking: https://github.com/aspnet/Hosting/issues/180
cc @DamianEdwards

:shipit:

:shipit:

Merged ccb9f70545762abf48a7affa0a8494bb2d0a8248


doc: remove needless todo:

Required fix was merged into specinfra and released as 2.8.22.
https://github.com/mizzy/specinfra/pull/721

Why don't you remove serverspec/find_installed_gem.ps1?

When #721 is merged, we can remove serverspec/find_installed_gem.ps1

https://github.com/fluent-plugins-nursery/td-agent-builder/blob/master/serverspec/find_installed_gem.ps1 can be removed.

Oops, it should be also removed.


Implement Retry ObjectStreamer

Make ObjectStreamer retry able so that we can retry while error happened during next.

completed by #1304


tests: after a few days of no failures of test_mesh_network, drop the iterations back to 10 (from the current 50)

haven't seen a failure for some time. closing