introduce uORB::SubscriptionBlocking and use in navigator and simulator

This PR introduces a new uORB Subscription class that has a blocking update. This can be used in place of the traditional orb subscriptions and polling.
Also contained in this PR are the other minor uORB changes needed for logger. https://github.com/PX4/Firmware/pull/12123
Example
Existing orb + poll usage
        int local_pos_sub = orb_subscribe(ORB_ID(vehicle_local_position));

        /* wakeup source(s) */
        px4_pollfd_struct_t fds[1] = {};

        /* Setup of loop */
        fds[0].fd = local_pos_sub;
        fds[0].events = POLLIN;

        /* rate-limit position subscription to 20 Hz / 50 ms */
        orb_set_interval(local_pos_sub, 50);

        while (!should_exit()) {

                /* wait for up to 1000ms for data */
                int pret = px4_poll(&fds[0], (sizeof(fds) / sizeof(fds[0])), 1000);

                if (pret == 0) {
                        /* Let the loop run anyway, don't do `continue` here. */

                } else if (pret < 0) {
                        /* this is undesirable but not much we can do - might want to flag unhappy status */
                        PX4_ERR("poll error %d, %d", pret, errno);
                        px4_usleep(10000);
                        continue;

                } else {
                        if (fds[0].revents & POLLIN) {
                                /* success, local pos is available */
                                orb_copy(ORB_ID(vehicle_local_position), local_pos_sub, &_local_pos);
                        }
                }

                // DO THE WORK
        }

        orb_unsubscribe(local_pos_sub);

Simplifies to this (using PR)
        uORB::SubscriptionBlocking<vehicle_local_position_s> local_pos_sub{ORB_ID(vehicle_local_position), 50};

        while (!should_exit()) {

                // wait for data
                if (!local_pos_sub.updateBlocking(_local_pos)) {
                        continue;
                }

                // DO THE WORK
        }

One point to discuss is the desired timeout behavior. There might be exceptions that I haven't come across, but in possibly all our orb related poll usage there's a timeout that does nothing but a fresh poll setup. So the question is if we need to preserve the ability to do anything else on a timeout, and if that timeout even needs to configurable.

@AuterionWrikeBot keeps helpfully clobbering the formatting in my example.

Out of flash on fmu-v2 again.

@dagar ok what do you need me to do here? Review or do we need to disable something else for v2?

@dagar ok what do you need me to do here? Review or do we need to disable something else for v2?

I'll ping you again once it's been resolved.

One point to discuss is the desired timeout behavior. There might be exceptions that I haven't come across, but in our orb related poll usage there's a timeout that does nothing but a fresh poll setup. So the question is if we need to preserve the ability to do anything else on a timeout, and if that timeout even needs to configurable.

Yes. Keep it.

One point to discuss is the desired timeout behavior. There might be exceptions that I haven't come across, but in our orb related poll usage there's a timeout that does nothing but a fresh poll setup. So the question is if we need to preserve the ability to do anything else on a timeout, and if that timeout even needs to configurable.

Yes. Keep it.

The timeout was kept, but now internal to SubscriptionBlocking.

Tested on Pixhawk 2 Cube V3:
Modes Tested
Position Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.
Mission Plan Mode (Automated): Good.
RTL: Good.
- Procedure
Arm and Take off in position mode, after flying for approximately one minute, switched to altitude then stabilized mode proceed to switch to mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoin activate RTL and see landing behaviour.
Notes:
No issues noted, good flight in general.
Log:
https://review.px4.io/plot_app?log=1d20e799-3a16-4505-b799-02967de22099
Tested on PixRacer V4:
Modes Tested
Position Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.
Mission Plan Mode (Automated): Good.
RTL: Good.
- Procedure
Arm and Take off in position mode, after flying for approximately one minute, switched to altitude then stabilized mode proceed to switch to mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoin activate RTL and see landing behaviour.
Notes:
No issues noted, good flight in general.
Log:
https://review.px4.io/plot_app?log=fdc055f9-7cf9-4fba-be03-9cd821fef4a4
Tested on CUAC+ V5:
Modes Tested
Position Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.
Mission Plan Mode (Automated): Good.
RTL: Good.
- Procedure
Arm and Take off in position mode, after flying for approximately one minute, switched to altitude then stabilized mode proceed to switch to mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoin activate RTL and see landing behaviour.
Notes:
No issues noted, good flight in general.
Log:
https://review.px4.io/plot_app?log=1646fbd1-46f3-4fce-bd87-0269965a0728

Tested on Pixhawk 4 v5 f450
Modes Tested
Position Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.
Mission Plan Mode (Automated): Good.
RTL: Good.
Notes:
No issues noted, good flight in general.
https://review.px4.io/plot_app?log=8ea521c4-94f3-47f8-be21-09c9bf402681

Tested on Pixhawk 4 mini v5:
Modes Tested
Position Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.
Mission Plan Mode (Automated): Good.
RTL: Good.
Procedure
Arm and Take off in position mode, after flying for approximately one minute, switched to altitude then stabilized mode proceed to switch to mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoint activate RTL and see landing behaviour.
Notes:
No issues noted, good flight in general.
Log:
https://review.px4.io/plot_app?log=3f0c6907-615f-47f9-a2f5-45f449e53962
Tested on Pixhawk Pro v4:
Modes Tested
Position Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.
Mission Plan Mode (Automated): Good.
RTL: Good.
Procedure
Arm and Take off in position mode, after flying for approximately one minute, switched to altitude then stabilized mode proceed to switch to mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoint activate RTL and see landing behaviour.
Notes:
No issues noted, good flight in general.

The estimators (attq and lpe) might as well transition straight to a WQ with callbacks. https://github.com/PX4/Firmware/pull/13287

Moving camera_feedback to a WQ. https://github.com/PX4/Firmware/pull/13044


Compiling samples by arm-none-eabi-gcc 6.3.1 could not be worked well

details: https://github.com/mROS-base/asp3-f767zi/issues/5

From mros2 v0.2.1 released at Dec 13 2021, we were finally able to switch the default host environment to Ubuntu 20.04 LTS and ROS 2 Foxy. Building sample1 for ASP3 kernel could build and work fine even if we use arm-none-eabi-gcc 9.2.1 which is installed as apt pkg. However, another issue happens to build mros2 resource.
So I decide to close this issue, and create new issue to #50 .
Also see https://github.com/mROS-base/asp3-f767zi/issues/5#issuecomment-998531084


Update to JupyterLab 3.0.0

Update to support JupyterLab 3.0.0 (rc.9).
It gets rid of the need to patch the reconnect procedure for the enterprise gateway which previously resulted in significant compile times for the jupyter-server service image.
This PR also removes the need for token based authorization of the JupyterLab instance since we authenticate all requests through the nginx proxy. This simplifies the jupyter-server spawning by just starting JupyterLab through the CLI instead of directly calling into it's Python classes (they provide a more stable interface on the CLI compared to direct Python initialization).
It also includes some minor changes to the front end (ctrl click for multi-step selection, and visibility checking for the fitAddon of XTerm)

As discussed we will try a larger rewrite where we no longer require the JupyterLab to be wrapped in a Flask API. Basically converting the into a Dockerfile with an ENTRYPOINT.


SetPassword logic needs improvement

Problem:

When we call API.SetPassword() we are creating a temp login when the response is 200
Otherwise we set the generic server error
We also have a catch() that is handling a reponse... but it's not clear where that response would come from.
SetPassword itself doesn't have any specific handling to throw a response
There's also a check for a Validate code failed message that is unlikely to be attached to an error since errors have no title (unless we throw the response itself - which we don't do)
Mostly, it just looks like a misunderstanding about how catch() even works or it could be possible we were throwing an error that we weren't catching in createTemporaryLogin() at some point (unlikely)

Solution:
I think at a minimum this logic needs to be migrated out of this catch() block as we are never going to hit it with the expected parameters and the logic makes no sense at all.
https://github.com/Expensify/App/blob/6c53be1ea7a022e8a5e788cc7e7efd62c4573474/src/libs/actions/Session/index.js#L319-L332
There are some side effects being performed in the catch like navigating back to home and making sure the resend validate code logic works, but unclear which flows that code was intended to fix. Will check git blames and try to piece it together 😄
catch() in general seems to be poorly understood throughout the app, but that is a separate issue...

We also just do this when the validate code expires and a user tries to set a password which should be updated to use FormAlertWithSubmitButton


New navbar

We need a new navbar that can redirect the user to nav links: /home, /account, /login, and /logout

I saw you already did this, right?


Remove compilation error when -Wshadow flag is set

Hi,
This PR fixes compilation error which is caused when -Wshadow flag is set.
Thanks
Stainlee

Hi,
Thank you for your contribution.
Would it be possible to rename count_local to count_ as done for hopscotch-map?
Thanks,
Thibaut

Yes. Definitely.
Will send a PR soon.

Thank you, I merged the changes.

Use n instead may be better


Daily update

Automated changes by create-pull-request GitHub action

Build failed because of:
no headquarter addresses for ulak-haberlesme-a-s at https://www.crunchbase.com/organization/ulak-haberlesme-a-s
no headquarter addresses for stc-solutions at https://www.crunchbase.com/organization/stc-solutions
No cached entry, and Ministry of Transport, Communications & Information Technology has issues with logo: ministry_of_transport_communications__information_technology.svg, SVG file embeds a png. Please use a pure svg file
No cached entry, and ATB Ventures has issues with logo: atb_ventures.svg, SVG file embeds a png. Please use a pure svg file
No cached entry, and Open Sourcing Mental Illness, LTD has issues with twitter: https://twitter.com/osmihelp, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
No cached entry, and Sollers College has issues with twitter: https://twitter.com/SollersEdu, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for CSIRO/DATA 61: https://twitter.com/CSIROnews
Empty twitter for Zhejiang Dahua Technology Co., Ltd: https://twitter.com/dahuasecurity
Empty twitter for TYNYBAY Inc: https://twitter.com/tynybayinc
Empty twitter for Truepic, Inc.: https://twitter.com/truepicinc
Empty twitter for Tectonic Labs Ltd.: https://twitter.com/cloud_ridge
Empty twitter for SVA System Vertrieb Alexander GmbH: https://twitter.com/SvaGmbh
Empty twitter for Surge: https://twitter.com/surgetech
Empty twitter for Rancher Federal: https://twitter.com/RancherFederal
No cached entry, and Mellanox Technologies, Ltd has issues with twitter: https://twitter.com/mellanoxtech, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
No cached entry, and JVC KENWOOD Corporation has issues with twitter: https://twitter.com/JVCKENWOOD_SR, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for eBaoTech International Pte Ltd: https://twitter.com/Ebaotech
Empty twitter for China Telecommunications Corporation: https://twitter.com/chinatelecom1
No cached entry, and ATS (a KGPCo Company) has issues with twitter: https://twitter.com/KGPCo1, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Solutions by STC either has no crunchbase entry or it is invalid
ULAK HABERLESME A.S. either has no crunchbase entry or it is invalid


Refactor "/signup" Route for Improved Code Structure

Description:
This GitHub issue aims to refactor the "/signup" route within the packages/server/src/api_auth/index.ts file. The main focus is on improving the code structure by utilizing res.locals.reqClientData as the source for client data, removing direct access to req.body, req.query, and req.params. Additionally, the refactoring should implement error handling using a try-catch block, taking inspiration from the error handling approach in the "login" route within the same folder.
Tasks:

Code Refactoring:

[ ] Review the existing "/signup" route in packages/server/src/api_auth/index.ts.
[ ] Replace direct accesses to req.body, req.query, and req.params with res.locals.reqClientData.
[ ] Ensure that client data is retrieved from res.locals.reqClientData consistently.

Error Handling:

[ ] Implement a try-catch block to handle errors gracefully.
[ ] Customize error messages and responses to provide meaningful feedback.

Testing:

[ ] Test the refactored "/signup" route to ensure it functions as expected.
[ ] Verify that error handling is effective in different scenarios.

Documentation:

[ ] Update any relevant documentation to reflect the changes made to the "/signup" route.

Acceptance Criteria:

The "/signup" route is successfully refactored to use res.locals.reqClientData.
Direct accesses to req.body, req.query, and req.params are removed.
The refactored code includes a try-catch block for error handling.
Testing confirms the correct functionality of the refactored route.
Documentation accurately reflects the changes made to the "/signup" route.

Additional Information:

Utilizing res.locals.reqClientData improves code structure and reduces dependencies on direct request access.
Effective error handling enhances the robustness of the route.

@prasenjeet-symon
I want to work in this issue. will u assign it to me?

hi @prasenjeet-symon
i have a question.
i think try-catch block is already Implemented in "/signup" Route
need other error handle ?

@prasenjeet-symon
Sorry for the delay.
What I didn't understand was that
The history of this task.
I need to rewrite res.locals.reqClientData because I want to use the value that has already been validated by the applicationRequestAuthSignupValidator and passed the validation.
Need to branch on validation errors (400) and 500 errors.
If you still have a chance, I would appreciate your confirmation.
238


Dark mode changes

What's the style idea?
We need to improve the dark mode. Unlike the light mode the dark mode has no wallpaper or design to it. It is just a grey background which doesn't look very good.
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[X] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Like we can add this image as a dark mode background

Dark mode cards is changed now, so can you pls tell what changes you want to do now after looking the new changes in the website again @SplinterSword

Okay proceed, let's see if it looks good @SplinterSword


Compression in EDIFACT messages

I believe the current serializer does not implement the EDIFACT specs on compression: https://web-beta.archive.org/web/20161004192113/www.unece.org/trade/untdid/texts/d422_d.htm#p7
The sample edifact.01.edn message contains the following line (here)
UNB+UNOC:3+1234567891123:14+7080005059275:14:SPOTMARKED+101012:1104+HBQ001++++1'
The unit test expected value has an extra ':' behind the 14, creating an ':+' combination (here)
UNB+UNOC:3+1234567891123:14:+7080005059275:14:SPOTMARKED+101012:1104+HBQ001++++1'
I believe the specs describe that the : should be omitted.
I will see if I can change this behavior and create an PR

I have made some changes to the serializer that fix this issue for all of my message implementations: https://github.com/KoosBusters/EDI.Net/commit/63ba14f73885f2a95a88bce81274588dab875023
All unit tests are still ok, but I think this behavior should be behind an configuration boolean because it might cause side effects if all messages are suddenly compressing. Maybe it should be part of the grammar in a way because it's the EDIFACT specs that require the compression?
@cleftheris it couldn't really find the place where to put in the current structure of the serializer, as a result it feels a bit as an add-on to the ediwriter instead of being an integral part of it. Could you have a look at this and see if there is a better way?

I agree it should be part of some kind of settings. Not sure it belong to the grammar though.
I think I prefer the commit.

@KoosBusters I tried to leave comments on the commit itself. Have not done this before though. Did you get the feedback? https://github.com/KoosBusters/EDI.Net/commit/63ba14f73885f2a95a88bce81274588dab875023

@cleftheris I can see a total of 3 comments on this commits. I will respond inline and see how that works :)

@cleftheris I left some comments in the commit as well. Could you take a look if you find some time? I will correct everything and update the PR with another commit. Resharper now follows the editorconfig after updating it to the latest version.

@KoosBusters I will try to find some time this week and close this.

Thanks!

Hi @cleftheris ,
I working on other projects the past few weeks but I'm about to get working on this again.
Any updates on this?


C3: Use Wide Road Camera as Light Sensor for Screen Brightness

Description
Using the narrow camera as a light sensor meant that tail lights and head lights of opposing vehicles and even the reflection of our own headlights dominated the sensor value at night even though the broader overall scene is very dark.  Personally, I have felt that the screen brightness at night can be too bright, and I have also seen a number of comments in discord about how bright the C3 is at night.
This has very little, I dare say no, effect during daylight hours.  Even when driving at dusk, the resulting brightness doesn't change much at all. The change affects nighttime driving the most, resulting in a dimmer screen when it is dark, but still turning brighter when in a bright intersection.
I think this makes more sense as the wide angle camera's field of view is much closer to the human eye's field of view than the narrow road camera.
Verification
I drove around during daylight, dusk, and nighttime to view the screen brightness.  I also plotted a comparison of the calculated screen brightness for both the wide and narrow camera for all drives.
Routes
I defined two Custom Series in Plot Juggler to plot the resulting brightness values calculated from the narrow and wide cameras, the only thing I wasn't able to do, was to incorporate the integration over time, so the values are more spiky than they would actually be.  I can share the plot juggler template used to plot this if it helps:

Nighttime - d50727925674b294|2022-01-18--18-00-44

As you can see, the resulting brightness is much lower.  The segment where the wide camera is at 10% and the road camera averages 30% looks like below, this seems pretty dark to me:

The narrow camera sees the scene as this:

Dusk - d50727925674b294|2022-01-21--17-12-28

There is a slight difference here, but really only in the darker scenes.

Daytime - d50727925674b294|2022-01-20--14-35-59

There is essentially no difference during daylight hours.

This looks good to me. @pd0wm what do you think?


Set up readthedocs

Need to improve overall documentation of the package and set up read the docs.

Not the greatest looking documentation in the world, but at least it a start!


Removed simulation inconsistency and other fixes

This branch was used to fix issue #94, and was called remove_interpolation, as I thought that was the problem. In the end, it was a problem with the thread safety of some operations. I also fixed some bugs and added some improvements to the plotting functions, like the ability to plot the simulation blocks and sequence blocks in plot_seq and plot_signal.

Oops, deleted Supressor and it was being used in the Tests. This will be fixed shortly.

The solution was to create a test environment (more info). Also, now we are using ThreadsX to do fast and thread-safe operations.

Codecov Report
Base: 62.22% // Head: 62.39% // Increases project coverage by +0.17% :tada:

Coverage data is based on head (c256ce8) compared to base (158db0f).
Patch coverage: 65.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
+ Coverage   62.22%   62.39%   +0.17%
==========================================
  Files          28       28
  Lines        1644     1710      +66
==========================================
+ Hits         1023     1067      +44
- Misses        621      643      +22

Impacted Files
Coverage Δ

src/ui/PulsesGUI_seq.jl
0.00% <0.00%> (ø)

src/ui/DisplayFunctions.jl
79.50% <38.09%> (-4.95%)
:arrow_down:

src/datatypes/sequence/RF.jl
63.63% <100.00%> (ø)

src/simulation/SimulatorCore.jl
87.42% <100.00%> (-1.02%)
:arrow_down:

src/simulation/other/DiffusionModel.jl
9.09% <100.00%> (+2.11%)
:arrow_up:

src/datatypes/sequence/Grad.jl
56.36% <0.00%> (-1.05%)
:arrow_down:

src/ui/PhantomGUI.jl
0.00% <0.00%> (ø)

src/ui/ReconGUI_absI.jl
0.00% <0.00%> (ø)

... and 8 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


FEATURE REQUEST - Show Groups in Sidebar

It would be awesome to have a Firefox "sidebar" hosting the Groups / Pages. It would show the groups, with the pages under them just like the toolbar button popup. Clicking on the group or any of it's pages would switch to that group.

I like the idea, it won't be too difficult as I will reuse the UI from the popup.
I have just a few features on the list before :)


Allow enabling departments when DisabledByDefault is true

When DisabledByDefault is true, only cops explicitly enabled in user configuration are run. This should work for departments too.

I just realized that I'm enabling all cops in the department, including the ones from disabled.yml. This is probably not what we want. I'll have to rework this.

There. I think it's all in order now.


Accuracy of LeakyBucketStrategy

I've discovered the algorithm implemented in LeakyBucketStrategy is far from accurate. Using limits of 3/10s, 10/1h, and 25/day all seem to function correctly.  However anything quicker than 1/s is very inaccurate.
I'm struggling to identify the issue. Can you provide any reference for where this algorithm came from?  It appears that there are significant floating point rounding errors when trying to implement faster rates.  I suspect the following line is the issue:
https://github.com/beheh/flaps/blob/master/src/Flaps/Throttling/LeakyBucketStrategy.php#L153
$timestamp = $time - ($rate - $unfinishedSeconds);

How to reproduce:
    public function testMake110RequestsAndExpect100Failures()
    {
        $fill = 10;

        $storage = new DoctrineCacheAdapter((new ArrayCache()));
        // same issue with redis:
        //$storage = new PredisStorage(new Client());

        $flapFactory = new Flaps($storage);
        /** @var Flap $flap */
        $flap = $flapFactory->getFlap('limit');
        $flap->pushThrottlingStrategy(new LeakyBucketStrategy($fill, '10s'));
        $flap->setViolationHandler(new PassiveViolationHandler());

        $uuid = uniqid();
        $violations = 0;

        for ($i = 0; $i < $fill; $i++) {
            $ok = $flap->limit($uuid);
            $violations += $ok !== true;
            $this->assertTrue($ok, "flap ok on iteration ${i}");
        }

        for ($i = 0; $i < 100; $i++) {
            $ok = $flap->limit($uuid);
            $violations += $ok !== true;
        }

        // on my hardware, only 10-30 requests trip instead of all 100, which is incredibly inaccurate.
        // curiously, setting $fill to 11 yields a consistent 89 of 100 failures
        $this->assertEquals(100, $violations);
    }
}

@chrisleavoy I updated the isViolator function a bit. Try this -
public function isViolator($identifier) {
    if ($this->storage === null) {
        throw new LogicException('no storage set');
    }

    $toBlock = false;
    $time = microtime(true);
    $timestamp = $time;

    $rate = (float) $this->requestsPerTimeSpan / $this->timeSpan;
    $identifier = 'leaky:'.sha1($rate.$identifier);
    $oldRequestCount = $this->storage->getValue($identifier);
    $newRequestCount = $oldRequestCount;

    if($oldRequestCount == 0) {
        $oldTimestamp = $time;
    }
    else {
        $oldTimestamp = $this->storage->getTimestamp($identifier);
    }

    $secondsSince = $time - $oldTimestamp;
    $maxToLeak = floor($this->requestsPerTimeSpan * ($secondsSince / $this->timeSpan));
    $newRequestCount = max($newRequestCount - $maxToLeak, 0);

    if($newRequestCount + 1 > $this->requestsPerTimeSpan) {
        $toBlock = true;
    }

    $newRequestCount++;
    $newExpiryIn = $this->timeSpan - $secondsSince;

    $this->storage->setValue($identifier, $newRequestCount);
    $this->storage->setTimestamp($identifier, $timestamp);
    $this->storage->expireIn($identifier, $newExpiryIn);

    return $toBlock;
}


Pushing blog changes and OSCI changes to prod

Pushing blog changes and OSCI changes to prod
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

@dtaivpp @nateynateynate Could you review this PR please?


The "published" attribute of the work model should be the workCollection's one

You sould not need to have a "published" attribute on the work model. Instead, take the associated workCollection's "published" attribute.

You're right ! Now the published attribute comes from the collection associated to the current work


Nag for deprecation of TaskExecutionGraph beforeTask/afterTask

This issue is about adding nagging by default. As in, even when the STABLE_CONFIGURATION_CACHE feature preview is not enabled.
This should happen in 8.1 with a message that says it will become an error in 9.0.

@mlopatkin This is being addressed in the context of #30000.

What is missing is to remove documentation samples for these endpoints. See: https://github.com/gradle/gradle/tree/master/platforms/documentation/docs/src/snippets/buildlifecycle/taskExecutionEvents
Context:
| Do we already have an issue about removing beforeTask and afterTask? We should have one and include a note about removing these samples as well.
Originally posted by @bamboo in https://github.com/gradle/gradle/pull/30000#discussion_r1692018150

There is no replacement for beforeTask at the moment, and afterTask can be replaced with OperationCompletionListener, but only for some use cases.


change signature of 'filter'

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
yes/no

Tests pass?
yes/no

Fixed issues
#16442

Won't be implemented in 2.0.

Is there a reason why the PR wasn't allowed? This PR seems like a very handy update.
Otherwise one have to create a custom validation rule to achieve the same. Which isn't such a clean solution.


include symbols and src from referenced projects

include source files for referenced projects in symbols nupkg's
ensure pdb's for referenced projects are included in nupkg's
add command line option for pack (include-referenced-projects)
create some tests for some of the above

cool

Sorry had to revert. Something is wrong with the paths. We I try to release Paket can't pack iteself anymore.
Please resubmit the PR and try to run "build.cmd NuGet" on the repo.

ok, Probably be tomorrow now.  I'll see what the problem is and fix.

Thanks.
On Jan 11, 2016 5:28 PM, "Paul Saunders"<EMAIL_ADDRESS>wrote:

ok, Probably be tomorrow now. I'll see what the problem is and fix.
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/pull/1373#issuecomment-170605586.


Container Access Level not working

Hi
Is the container access level currently supported by azurite?
When creating a new container with the node-sdk the default access level should be private. However, the files within this container are public accessible without any SAS token.
When changing the AZURE_STORAGE_CONNECTION_STRING to a real azure blob storage account, everything is working like expected. The files are not public accessible.

Azurite is more focusing on positive features right now. Negative cases are not fully aligned yet.

Hi @yss14 please try with Azurite v3 which has fully container access level support. Will close this issue!


Kitematic Early Linux support - Boot2Docker Error: Cannot connect to the Docker daemon. The daemon is not running: connect ENOENT /var/run/docker.sock

Hi,
I've installed Kitematic Early Linux support  on my Ubuntu where I also have configured a boot2docker VM. I've all the env vars configured:
export DOCKER_HOST=tcp://<IP_ADDRESS>:2376
export DOCKER_CERT_PATH=/home/maq/.docker/machine/certs
export DOCKER_TLS_VERIFY=1
export DOCKER_IP=<IP_ADDRESS>
docker-machine & docker-compose work fine and I can interact with my VM.
Kitematic for linux is always asking for docker daemon running on my host and because of that I suspect it doesn't take into account docker env vars and it doesn't manage docker in the VM.
I would like to get Kitematic running against docker in my boot2docker VM with my Ubuntu host.
Any ideas?

@miguelqvd The Early support for Linux is actually made to NOT create a VM and why the above error exists.
You should be installing the Docker daemon directly on your ubuntu workstation, which Kitematic will then connect to.
The above export commands are useless as the early Linux support doesn't look for a VM at all.
You can checkout the Ubuntu install how-to at the following url: https://docs.docker.com/installation/ubuntulinux/

Thank you for your reply, which make really a lot of sense. However I need to ask how might I mange docker environments that are not installed in my local linux machine. I'm thinking for example about administrating digital ocean VMs. I think Kitematic has a lot of potential and it should support this feature which is already available for windows and mac through env vars.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain biomassiv.es has exceeded the max emails per hour (104/100 (104%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext5.iad.github.net ([<IP_ADDRESS>]:58664 helo=github-smtp2a-ext-cp1-prd.iad.github.net)
by server23.websitehostserver.net with esmtps (TLSv1.2:DHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.85)
(envelope-from<EMAIL_ADDRESS>id 1ZmS3u-003D7Z-Mo
for<EMAIL_ADDRESS>Wed, 14 Oct 2015 14:50:11 -0500
Date: Wed, 14 Oct 2015 12:50:09 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1444852209;
bh=xBHgliJikVM7NjBc/JAhsTxCKkpKFBjPaZMmBfAKb7o=;
h=From:Reply-To:To:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=1v4dHaCa6lPWQIGnnAiBg0ftU//Yo1cDU0frSfaB3gygxvaqc5ObwepXdCHqKrJNM
E8fJleVgYYsG9lF4qx2UOeHHcdaWOFg2Rg4pbFNZiochsvPAVDA28NaD6JOfWwORjl
rDU5UdLQo0i7/uYKP93rMVfBy26iy54vEWODz+sE=
From: miguelqvd<EMAIL_ADDRESS>Reply-To: kitematic/kitematic<EMAIL_ADDRESS>To: kitematic/kitematic<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [kitematic] Kitematic Early Linux support - Boot2Docker Error:
Cannot connect to the Docker daemon. The daemon is not running: connect ENOENT
/var/run/docker.sock (#1101)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_561eb1f1d6e59_580a3f8bbbae52a019802b";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: miguelqvd
X-GitHub-Recipient: biomassives
List-ID: kitematic/kitematic <kitematic.kitematic.github.com>
List-Archive: https://github.com/kitematic/kitematic
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AELU4iAlr6ly7cu-aqHDpn5CNwgzqtZBks5o7qlxgaJpZM4GMjrp
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-Spam-Status: No, score=0.0
X-Spam-Score: 0
X-Spam-Bar: /
X-Ham-Report: Spam detection software, running on the system "server23.websitehostserver.net",
has NOT identified this incoming email as spam.  The original
message has been attached to this so you can view it or label
similar future email.  If you have any questions, see
root@localhost for details.
Content preview:  Thank you for your reply, which make really a lot of sense.
However I need to ask how might I mange docker environments that are not
installed in my local linux machine. I'm thinking for example about administrating
digital ocean VMs. I think Kitematic has a lot of potential and it should
support this feature which is already available for windows and mac through
env vars. [...]
Content analysis details:   (0.0 points, 5.0 required)
pts rule name              description

0.0 URIBL_BLOCKED          ADMINISTRATOR NOTICE: The query to URIBL was blocked.
See
http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block
for more information.
[URIs: github.com]
-0.0 RCVD_IN_MSPIKE_H4      RBL: Very Good reputation (+4)
[<IP_ADDRESS> listed in wl.mailspike.net]
-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%
[score: 0.0000]
2.1 HTML_IMAGE_ONLY_12     BODY: HTML: images with 800-1200 bytes of words
0.0 HTML_MESSAGE           BODY: HTML included in message
-0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
domain
0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
-0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
-0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders
X-Spam-Flag: NO
----==_mimepart_561eb1f1d6e59_580a3f8bbbae52a019802b
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Thank you for your reply, which make really a lot of sense. However I need to ask how might I mange docker environments that are not installed in my local linux machine. I'm thinking for example about administrating digital ocean VMs. I think Kitematic has a lot of potential and it should support this feature which is already available for windows and mac through env vars.

Reply to this email directly or view it on GitHub:
https://github.com/kitematic/kitematic/issues/1101#issuecomment-148176425
----==_mimepart_561eb1f1d6e59_580a3f8bbbae52a019802b
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Thank you for your reply, which make really a lot of sense. However I need to ask how might I mange docker environments that are not installed in my local linux machine. I'm thinking for example about administrating digital ocean VMs. I think Kitematic has a lot of potential and it should support this feature which is already available for windows and mac through env vars.
—Reply to this email directly or view it on GitHub.

----==_mimepart_561eb1f1d6e59_580a3f8bbbae52a019802b--


ButtonToggle

@SamProf ,
Is this one still an issue? In not certain as there isn't much to go off of.

@SamProf ,
Just a bump on this one


Re-trigger gitstream after submitting a review

Is your feature request related to a problem? Please describe.
After submitting a review, I was thinking that gitstream would be re-triggered but that doesn't seem to be the case.
Describe the solution you'd like
After I submit a review, then gitstream should be re-triggered
Describe alternatives you've considered
I've looked at the other actions but I don't think they'll help me/I'd run in to a similar issue
Additional context
My use case for above was specifically for when a review is submitted with requested changes, I wanted an automation to run to add a "requested changes" label
Below is what I am trying to use but after submitting my review, I didn't see gitstream run again

automation snippet
requested_changes:
    if:
      - {{ pr.status | includes(term='requested-changes') }}
    run:
      - action: add-label@v1
        args:
          label: '🚨 Changes requested'
          color: '#EC560D'

@vim-zz perfect, thanks! Did you want me to leave this open for now or close and just track the progress of issue 18?

@vim-zz Actually, after submitting some more PRs today with requested changes, gitstream seems to be running again, but my automation check always evaluates to false.
I tried updating it to this earlier today just to see if my check was too specific but it's still evaluating to false.
updated automation snippet
requested_changes:
  if:
    - {{ pr.status | includes(list=['requested-changes', 'Changes requested', 'requested', 'changes', 'Changes']) }}
  run:
    - action: add-label@v1
      args:
        label: '🚨 Changes requested'
        color: '#EC560D'

Couple follow up questions:

Is it possibly just the case that the PR status isn't actually up to date when the check occurs? Or is something wrong with my automation check?
Is there a way to check the PR status when the automation is running?

@vim-zz any updates/thoughts on this? It always seems to be evaluating to false after submitting a PR review with requested changes

@matthewparavati we added this to the roadmap, I don't have definite timeline yet

@matthewparavati: Added:

Values
Type
Description

pr.status
String
The PR status: open, closed and merged

pr.requested_changes
[String]
List of users that requested changes

@vim-zz thanks for the update. should the requested_changes array be zero after changes are resolved or once the user approves the PR?
Basically, we aren't seeing the label getting removed once it's been added and wondering if this is expected?

@matthewparavati the list clears when all issues are resolved. Do you see gitStream triggers a new check after the changes are resolved? If not, this is a bug. If it tirggers bug doesn't change the resolution it's might be anoter problem

@vim-zz I was able to re-test and I did not see a new gitstream check after marking a requested change as resolved, I created #153 as a follow up to this


Add blockchain

This might be scope creep, but I see this as is a high-value addition that is in the spirit of the project.


Hub site navigation with extended header on homesite

Your Note:
When you apply the extended header layout to the site, you will no longer see the hub navigation.
This did not work as expected.
Our homesite is a hub-site with extended header and 'hub or global navigation' selected. The 'site navigation' disappears and is replaced with the hub-navigation.
When scrolling down, the extended header will collapse and the site navigation will appear replacing 'the hub navigation' on the header.
I already raised a ticket with Microsoft Support and they claim that this is 'By design'.
Their advice is to not use extended design.

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

ID: b2cb90e0-b8bb-32b2-d8b1-1710af6312f2
Version Independent ID: 9d7a64c4-a3b0-47b8-b209-16e257a9a23a
Content: Use the new SharePoint app bar and set up global navigation - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/sharepoint-app-bar.md
Service: sharepoint-online
GitHub Login: @Holland-ODSP
Microsoft Alias: hokavian

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Hi Manuela,
I did log a ticket, but then they said it was ‘By Design’ so the ticket was closed.
So how can it be ‘By design’ when it does not match with the documentation?
Documentation says that the hub-navigation disappears, but in reality the site navigation disappeared.
Please advise…
Kind regards,
Guy
Van: Manuela Scanu @.>
Verzonden: dinsdag 29 juni 2021 23:55
Aan: MicrosoftDocs/OfficeDocs-SharePoint @.>
CC: guywel @.>; Author @.>
Onderwerp: Re: [MicrosoftDocs/OfficeDocs-SharePoint] Hub site navigation with extended header on homesite (#2824)
Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/2824#issuecomment-870944756 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AB5HB3E6U3JDHQWQOWSVI7DTVI6M5ANCNFSM47PTWJPA .  https://github.com/notifications/beacon/AB5HB3BIOCLBKWR7OWQCQZ3TVI6M5A5CNFSM47PTWJPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGPUY75A.gif

Hey @guywel - You are right, there is an error in the article here. I'll get this updated. Thank you!


FF-1939 Update to UFC

Test case is failing because 123456789.0 is getting serialized as "1.2345679e+08" as discussed in slack.


[Tool] Support AutoMerge in workflow

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[x] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

@Mergify backport branch-2.5

@Mergify backport branch-2.4

@Mergify backport branch-2.3


Add callback property to FramingBehavior

Add a callback property to FramingBehavior that gets passed to zoomOnMesh when the target is changed, so that  we can more easily know when the animation has ended.
Forum: https://forum.babylonjs.com/t/how-can-i-see-when-a-framingbehavior-animation-has-finished/25086/6

PS, if an observable would be preferable to the callback,  let me know and I can rework it that way. :)

We have moved towards using observables instead of fixed callbacks. This way you also don't have to check if it exists - just trigger notification and the observable will do the rest for you.
Thanks!!

Okay, I changed the new property to be an observable instead of a callback. zoomOnMesh takes a callback, so I used an anonymous callback which calls notifyObservers on the observable. :)
PS, previously I was just passing the callback through, which is already being checked by onZoomMesh, so I think it just comes down to rather or not more than one callback will be needed by users and which is easier to set…

Hmm, also, on second thought, maybe onZoomToTargetEndObservable would be a better name than onTargetFramingAnimationEndObservable for this property?


Search for empty NPI causes BloomAPI Internal Server Error

e.g. http://localhost:3005/api/search?limit=50&offset=0&key1=npi&op1=eq&value1= causes an internal server error (result code 500).
This should return a 400 instead with description.

Also caused by requests such as:
/api/search?limit=1&offset=0&key1=npi&op1=eq&value1=:npi

First query fixed by ede0211b212217f69cf90077671959864f869079.
The last query was fixed by 2ebbff3d5982d9d28f13dc9a6d7aabb5c11a3631


Vg/add share replay

Adding a missing binding for shareReplay.

Hi Vaclav
Thanks for the PR!
Sorry for not seeing this sooner, will take a look Monday morning Pacific time

Hey, Alex 👋 Nice to see your reply. Originally I wanted just to add the shareReplay but I'm loading this branch in my project so I abused it for the distribution of my experiments into my other repo, sorry for that. I will close this PR and create new ones for each change separately, we can discuss the reasons and consequences in those PRs.


Pwm module,How to understand set_duty

i want to use pwm drive steering engine，but I  don't understand set_duty how to set value,
in c design formulas
Freq = CK_PSC/((PSC+1)*(ARR+1))
Duty = CCR/(ARR+1)
freq = 50Hz
Duty should how to set value

1 PWM cycle contains (ARR+1) ticks. You can take this number with .get_max_duty(). It is 100% of cycle. If you set 50Hz, it match 2ms.
Duty is part of cycle when pin is high.
For example .get_max_duty() gives you 8000.
If you pass .set_duty(4000) you take square wave with 50% duty cycle. .set_duty(2000) => 25%

1 PWM cycle contains (ARR+1) ticks. You can take this number with .get_max_duty(). It is 100% of cycle. If you set 50Hz, it match 2ms. Duty is part of cycle when pin is high. For example .get_max_duty() gives you 8000. If you pass .set_duty(4000) you take square wave with 50% duty cycle. .set_duty(2000) => 25%

thanks very much

 let mut pwm = dp
        .TIM2
        .pwm_hz::<Tim2NoRemap, _, _>(pins, &mut afio.mapr, 50.Hz(), &clocks);
let (mut c1) = pwm.split();
let max = c1.get_max_duty();

why max =53333.
then
c1.set_period(50.Hz());
let max = c1.get_max_duty();
max still is 53333

1 PWM cycle contains (ARR+1) ticks. You can take this number with .get_max_duty(). It is 100% of cycle. If you set 50Hz, it match 2ms. Duty is part of cycle when pin is high. For example .get_max_duty() gives you 8000. If you pass .set_duty(4000) you take square wave with 50% duty cycle. .set_duty(2000) => 25%

 let mut pwm = dp
        .TIM2
        .pwm_hz::<Tim2NoRemap, _, _>(pins, &mut afio.mapr, 1.kHz(), &clocks);
let (mut c1) = pwm.split();
let max = c1.get_max_duty();

the max is 8000

pwm_hz automatically select values of PRE and ARR registers to make ARR as much big as possible. So you can't control them.
Instead you could use .pwm::<Tim2NoRemap, _, SAMPLING_FREQ_YOU_NEED>(pins, &mut afio.mapr, 2.ms(), &clocks).
https://docs.rs/stm32f1xx-hal/0.9.0/stm32f1xx_hal/timer/pwm/trait.PwmExt.html#tymethod.pwm

1_000 is ~ lower limit
use 1_000_000 for start. In this case max_duty will be 2000

1_000 is ~ lower limit use 1_000_000 for start. In this case max_duty will be 2000

thanks,programe right

I have still some question:
i'm try to drive steering engine
steering engine requirement:

FREQ: 50Hz
Duty:  (0.5- route to -90°, 1 - route to -45°，1.5 - route to 0, 2 - route to 45°,2.5-route to 90°)

in C Project
Freq = 72M/((PSC+1)*(ARR+1))
Duty=CCR/(ARR+1)
ARR+1=20ms=20k
Freq = 50Hz
PSC=720000000-1
so CRR =500,Duty=1.5ms
CRR=2500,Duty=2.5ms
 let mut pwm =
        dp.TIM2
            .pwm::<Tim2NoRemap, _, _, 50_000>(pins, &mut afio.mapr,2.millis() , &clocks);
 let (mut c1) = pwm.split();

should i how to set this value will generate result to same c code

ARR+1=20ms=20k Freq = 50Hz PSC=720000000-1
Your C prject looks broken. You cannot get ARR+1=20k as you cannot get 10 MHz from 72 MHz on stm32f1 (you need other chip like stm32f411 or other oscillator).
Correct values are 9 MHz, 9.6 MHz (and maybe 10285714 Hz).
This is what you need for 10 MHz on TIM is set sys clock to 40 Mhz and APB1 to 20 MHz:

        let clocks = rcc.cfgr
            .use_hse(8.Mhz())
            .sysclk(40.Mhz())
            .pclk1(20.Mhz()) // bus with clocking for TIM2
            .freeze(&mut flash.acr);

.pwm::<Tim2NoRemap, _, _, 10_000_000> This will set PSC+1=20_000_000/10_000_000=2

ARR+1=20ms=20k Freq = 50Hz ~PSC=720000000-1~
Your C prject looks broken. You cannot get ARR+1=20k as you cannot get 10 MHz from 72 MHz on stm32f1 (you need other chip like stm32f411 or other oscillator).
Correct values are 9 MHz, 9.6 MHz (and maybe 10285714 Hz).
This is what you need for 10 MHz on TIM is set sys clock to 40 Mhz and APB1 to 20 MHz:

        let clocks = rcc.cfgr
            .use_hse(8.Mhz())
            .sysclk(40.Mhz())
            .pclk1(20.Mhz()) // bus with clocking for TIM2
            .freeze(&mut flash.acr);

.pwm::<Tim2NoRemap, _, _, 10_000_000> This will set PSC+1=20_000_000/10_000_000=2

thanks, i understand


User Deprecated: The "Sentry\State\HubAdapter::captureMessage()"

Environment
How do you use Sentry?
Sentry SaaS (sentry.io)
Which SDK and version?
sentry/sentry-symfony 4.0.3
Steps to Reproduce

Install package
configure like this

$containerConfigurator->extension('sentry', [
    'dsn' => '%env(SENTRY_DSN)%',
    'register_error_listener' => false
]);

Profit 💰

Expected Result
No deprecation notifications
Actual Result
[info] User Deprecated: The "Sentry\State\HubAdapter::captureMessage()" method will require a new "EventHint|null $hint" argument in the next major version of its interface "Sentry\State\HubInterface", not defining it is deprecated.

Could you please add some more information, like the stack trace? I can't find the origin of such deprecation message.

Ok found it: https://github.com/symfony/symfony/blob/0cc982c251f4e0a404f7df1f4521636d4ed4f2b3/src/Symfony/Component/ErrorHandler/DebugClassLoader.php#L675
It's from the Symfony DebugClassLoader, so it's only in local, not in prod due to debug set to false. What I do not understand is why it has been triggered, since the HubAdapter already implements that additional argument. What version of sentry/sentry are you running?

Hey @Jean85
Thanks you for your response. I am using de sentry/sdk version 3.1.0. I havent added it seperately in my composer so it came with the sentry/sentry-symfony package

Problem found, it's the HubAdapter that is missing the new parameter:

https://github.com/getsentry/sentry-php/blob/fb4f83e6e2d718d1e5fbfe3a20cced83f47f040f/src/State/HubAdapter.php#L109
https://github.com/getsentry/sentry-php/blob/fb4f83e6e2d718d1e5fbfe3a20cced83f47f040f/src/State/HubAdapter.php#L117
https://github.com/getsentry/sentry-php/blob/fb4f83e6e2d718d1e5fbfe3a20cced83f47f040f/src/State/HubAdapter.php#L125

I'll transfer this issue in the other repo.


Add associated taxons support

We have added a new link type associated_taxons which allows a taxon to be associated with another. The effect of this association is that content tagged to the associated taxon(s) is included in the content of the taxon that declares the association.
e.g.

usa taxon has child_taxons of [travelling-in-the-usa]
travelling-in-the-usa has associated_taxons of [travelling-abroad]

The taxon page for usa would display a 'Travelling in the USA' heading. The 'Travelling in the USA' heading would contain content tagged to travelling-in-the-usa and also any content tagged to travelling-abroad .
This PR adds support for this feature.
Trello
Discussed with @carvil how we can include this associated taxon content in the #most_popular_content results. We don't necessarily require this currently but it makes sense for it to be included in the results. Any suggestions welcome.

@bevanloon I've updated TaggedContent to accept an array if you could just give it another quick look.


SpriteFrames Editor: Fix Frame Duration applied to wrong frame when switching frame

Splitted from #79692.

Fix frame duration change applied to last frame when switching frame while editing value of Frame Duration spinbox.
Other changes:

Fix the UndoRedo merging mode of the Frame Duration field to be consistent with the FPS field (I think it was my copy-paste mistake).
Use *_no_signal() methods in _update_library() for Frame Duration control.

Thanks!


Delete cloud-architecture/oracle-apps-erp/cyber-recovery-solution dir…

…ectory

this directory can go right


No "www" in links to social media causes redirects

Please add to links generated from entries in _config.yml to include "www".
This would make it a little faster for users by avoiding a redirect.

This is from the report by the W3 validator at:
https://validator.w3.org/checklink?uri=http%3A%2F%2Fwilsonmar.github.io%2F&summary=on&hide_type=all&recursive=on&depth=1&check=Check
I'll create a PR on file author_profile.html (previously _author_bio.html) in the _includes folder within the develop branch.

A PR to develop should be good.

PR #293 filed.

Merged in PR #293


DES-13573 - upgrade to CEFv64

CEF upgrade to CEFv64.
It is needed to upgrade to VS2015 due to CEFGlue.
Some parameters from PopupFeatures got removed so changes had to be reflected on paragon code:
-minuet/Paragon.Runtime/Kernel/Windowing/ApplicationWindowManager.cs

lgtm


[WIP] Add DbgMemory RGB output format

> Read-DbgMemory -DefaultDisplayFormat RGBA -DefaultDisplayColumns 32 0x17EF1B10342 -LengthInBytes 0xC00

The code is more or less rough prototype currently, but I wanted to open the pull request to start discussing: how should 24-bit color output be handled? Currently, I'm using the two byte escape sequence to dodge the processing by AnsiColorWriter, but that's maybe not the best.

Thanks Zhentar! Looks cool. We are currently stuck with AnsiColorWriter because of the non-standard push / pop SGR codes, so using the two-byte escape sequence to sidestep it seems like a reasonable way around it.
(In the future I hope to be able to get rid of AnsiColorWriter by using the slightly-more-standard (but still non-standard) push / pop sequences introduced in xterm patch 334. I need to convince somebody to implement that sequence in conhost, though.)

The ADHD finally indulged me long enough to revisit this... I'm trying to figure out what I need to do to let ColorString know whether the VT sequences will work. I know that $host.UI.SupportsVirtualTerminal works to determine it within Powershell, but I'm trying to understand why Out-Default & friends set $HostSupportsColor to false during their processing and whether I need to do anything similar. Any hints?

why Out-Default & friends set $HostSupportsColor to false during their processing

IIRC, the formatting proxy commands temporarily turn off $HostSupportsColor only when they are proxying to the built-in formatting commands, because the built-in formatting commands don't support color.
"But why should they have to worry about color command sequences; isn't that handled by the underlying console control?"
It's true that the built-in formatting commands do not have to worry about interpreting the color command sequences; the problem is that they try to do formatting, such as truncating at a certain width, but they get their string length calculations all messed up because they don't know that escape sequences should be counted as "zero width", and they might even end up chopping a string in the middle of a sequence, which is no bueno.
So when DbgShell's Format-Table command determines that it is going to send the content on to the built-in Format-Table, it temporarily "turns off" color.
(All that said, I don't recall off the top of my head how "bullet-proof" that scheme is w.r.t. "pre-rendered" color; i.e. when the escape sequence characters are just characters in the string, rather than tracked in a ColorString. So it may be possible to sneak color content past $HostSupportsColor.)
(And of course ColorString objects will not be sent to the built-in formatting cmdlets.)
So... I don't think you need to do anything about $HostSupportsColor here.
However, I'm flip-flopping about the utility of the new RGB/RGBA formats (the AppendFgRgb addition on its own is totally good). On the one hand, I just think it's super neat. On the other, for it to really work, you have to know up-front the proper width (and pass that in), and that width has to be small enough to fit into a console window... (I guess if you have a wide monitor, that can be pretty wide, but still...)
It seems like it's a very niche sort of thing to do. Maybe it would be better packaged as a separate script command? I hesitated to say it, because one minute I think it's just so cool; but the next I feel like it unnecessarily clutters up the display formats, and would be the very least used... what do you think? What was your scenario that prompted you to write this?

In reply to: 500648001 [](ancestors = 500648001)

Thanks for the explanation! Very helpful!
I'm quite amenable to the argument that the RGB/RGBA display should be a separate command. The display width part isn't a particularly significant challenge (at least on the win 10 versions I've been using, you can set the console buffer to be much wider than your screen, and at the minimum 5pt font size you can see a lot of it at once, particularly with larger monitors), but figuring out the stride is definitely a major challenge/usability issue - I've done it through trial & error, which means I usually need dozens tries to get it right (even when I'm very close, my intuition is a pretty terrible guide on adjusting it). When I first submitted this, I had thought it could also be useful for detecting patterns in data, but after further experiencing difficulties just getting perfect vertical lines matched up right on simple multiple-of-8 sized image buffers, I'm skeptical of it being generically useful without some sort of automatic stride detection, which I don't see fitting in well with the standard Read-DbgMemory interface.
As far as what inspired me/what I've used it for, it's been identifying clearly image-y regions in memory; figure out what icons are or what a large buffer is buffer-ing. It's been valuable to me for that, but you have to get really deep into things before you even notice they exist, much less care (stay tuned, I'm preparing a pull request to help with a lot of the intermediate work there 😉 or just look at the commits on my fork).
So for the purposes of this PR, I'll focus on cleaning up the AppendFgRgb stuff (I think I've got a clear view now on how to make $HostSupportsColor sufficient for it as-is), and stick the RGB/RGBA output stuff into a separate command, for look-at-this-cool-shit-can-you-imagine-your-old-fashioned-debugger-doing-this purposes 😄

Still need to do the 'stick RGB output in a new command' part, but I've pushed my RGB support changes. There are some major changes to AnsiColorWriter, but we get a massive performance boost out of it - the Measure-Command here drops from 20-22 seconds for me down to 3*
 ntna
 $sym = (x ntdll!Rt* | Format-Table)
 Measure-Command { $sym | Out-Default }

*(If I minimize conhost or use the new Terminal app. If I leave conhost visible at my default console size, it goes from ~55 seconds to ~40 seconds... but that's all on conhost's GDI renderer.)
For PowerShell/PowerShell@2174dd8 note that the reduced string allocations aren't particularly significant (just a drop in the ocean of Powershell allocations), but it's got a substantial benefit in halving the number of GetMode, GetConsoleScreenBufferInfo†, and WriteConsole calls for a WriteLine.
†A call which I find particularly annoying because PowerShell is just grabbing the columnwidth, which it then immediately throws away because we aren't asking it to wrap 😑

Just 5 months later, I've successfully moved the RGB output to a separate cmdlet 😅

Thank you!
I think in general this looks good, but I have a few nits to pick. I'm excited to have a little time to put towards this over the holidays. Next big thing I'm working on is moving to .NET Core and PowerShell Core. I don't know if I'll make it--it's a gigantic change, and I think there's a good chance I'll run into significant, blocking problems, but I hope to at least make good progress. Toward that end, I'd love to get this in soon if you have the cycles to spare for it.

I went ahead and made the fixups myself (didn't know if I would have permissions to push to your branch like that, but apparently I do). Let me know if I messed anything up. Thank you!

Thanks! Your changes look fine to me.  The transcribeResult thing was definitely an oversight - however, checking against pwsh it looks like we should have added newline info: https://github.com/PowerShell/PowerShell/blob/d523d557391fdcc84c058b776fe8a4913ebb1713/src/Microsoft.PowerShell.ConsoleHost/host/msh/ConsoleHostUserInterface.cs#L594-L597
But since the transcribing logic in PostWrite is commented out, I don't think it matters (and I believe makes transcribing already non-functional).
As for the app.config thing: version 4.5 of the package contains an assembly with the AssemblyVersion 4.0.3. Why are they not the same? What is the purpose of bindingRedirects in app.config files? I assume there are good reasons, but in my experience thus far they only serve to generate unpredictable failures with unintelligible error messages and unclear resolution.


develop to main

Ideally, we don't want to squash merge this
these commits have already been squash / merge from PR to develop.

Codecov Report

Merging #63 (9561ebf) into main (8f5fba4) will increase coverage by 4.79%.
The diff coverage is 80.73%.

@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
+ Coverage   80.04%   84.84%   +4.79%
==========================================
  Files           8        8
  Lines         406      495      +89
  Branches       88      100      +12
==========================================
+ Hits          325      420      +95
+ Misses         44       40       -4
+ Partials       37       35       -2

Impacted Files
Coverage Δ

src/providers/ethereumProviderWrapper.ts
75.00% <70.00%> (-12.50%)
:arrow_down:

src/lib/provider-wrapper-utils.ts
88.88% <71.42%> (-3.42%)
:arrow_down:

src/lib/utils.ts
79.72% <74.41%> (+2.80%)
:arrow_up:

src/providers/web3ProviderWrapper.ts
86.95% <80.00%> (-13.05%)
:arrow_down:

src/index.ts
83.92% <81.25%> (+5.49%)
:arrow_up:

src/lib/constants.ts
100.00% <100.00%> (ø)

src/lib/encoder.ts
91.48% <100.00%> (+19.57%)
:arrow_up:

src/types/Method.ts
100.00% <100.00%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


Bump version to Mattermost 5.24.2

Summary
Bump Mattermost Server version to pull the latest plugin api docs.

Hm, actually there were no changes to the plugin api in that release. Anyway, I still think it makes sense to stay updated.

@justinegeffen I can't merge this PR without a second review but there isn't anything visually to review. Would you mind giving me a rubber stamp review?


adding isMobile

boolean, Returns true if the client uses mobile, otherwise returns false.
isMobile() does not affected by screen size, if your using laptop and your screen width 500px it still returning false.
isMobile() uses regex to check userAgent if it's mobile or not.

some information regarding the future of user agent:
https://medium.com/@galmeiri/get-ready-for-chrome-user-agent-string-phase-out-c6840da1c31e
I'm not a fan of this snippet because of the fact that it requires a known list of "mobile" clients, only interacts wit h the "declared" user agent string, and is not future compatible, and when attempting to code to a "mobile" client mobile can mean many things that this doesn't address.  This will not be a very adaptable snippet either and would require maintenance as time goes on.
If there were a more addressable way to detect what a mobile client were based on the criteria put forth of what the developer wanted I'd be all for this, but because "mobile" doesn't always mean "engine" and can commonly refer to "screen size" and is in all cases arbitrary I'm not certain in the current form this snippet can really be made more adaptable either.

@skatcat31 honestly I wasn't aware of deprecating UA. this a new info.

Thanks for contributing. User Agents are extremely unreliable, easy to fake and need to be constantly updated with any new information that comes out anytime there's an update. Apart from that, newer devices blur what is mobile and what is not (think about laptops with touchscreens or mobiles that have keyboards attached via a dock), so it's far less desirable to check for mobile UAs. Instead, checking for individual features, screen sizes etc. is much more useful information for most modern web apps. Based on that info, I am going to go ahead and close this PR. Thanks anyways and feel free to open a new PR if you have any other snippets you want to share.


issue 110

issue 110
Ticket #110
Description
Create frontend template for dashboard - for recent activity and upcoming grants columns. This PR contains only the frontend mockups with hard coded values NOT the full functionality of these tables.
Screenshots / Demo Video

Testing
Not a whole lot of testing since this is just the mocks but I did test different values in the table to make sure that the tables will scale will longer agencies and grants
Checklist

[x ] Provided ticket and description
[ x] Your PR title contains the ISSUE ticket number e.g "86: ..."
[ x] Provided screenshots/demo
[ x] Provided testing information
[ x] Provided adequate test coverage for all new code
[ x] Added PR reviewers
[ ] Ensure at least 1 review before merging

QA: LGTM. I would not merge this into main until the backend is hooked up though!


Insert string at the current cursor position

Hi there,
is there a way to insert a string at the current cursor position?
So I have a toolbar (a component) and there I have some buttons prepared for the user to add some pre-defined commands (like bold, underline, etc.) when clicking on each of them, a different type of strings should be inserted into the editor, where the cursor is set. how is this achievable with current configurations? is there any workaround?
many thanks

You can insert at cursor position by using
const doc = editor.getDoc()
const cursor = doc.getCursor()
doc.replaceRange('text', cursor)

You can get editor reference from onChange function arguments (editor, data, value)


python311Packages.aioesphomeapi: 15.1.15 -> 16.0.0

Diff: https://github.com/esphome/aioesphomeapi/compare/refs/tags/v15.1.15...v16.0.0
Changelog: https://github.com/esphome/aioesphomeapi/releases/tag/v16.0.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

40 packages built:

esphome
esphome.dist
home-assistant-component-tests.airthings_ble
home-assistant-component-tests.aranet
home-assistant-component-tests.bluemaestro
home-assistant-component-tests.bluetooth_adapters
home-assistant-component-tests.bluetooth_le_tracker
home-assistant-component-tests.bthome
home-assistant-component-tests.dormakaba_dkey
home-assistant-component-tests.esphome
home-assistant-component-tests.eufylife_ble
home-assistant-component-tests.fjaraskupan
home-assistant-component-tests.govee_ble
home-assistant-component-tests.homekit_controller
home-assistant-component-tests.ibeacon
home-assistant-component-tests.inkbird
home-assistant-component-tests.kegtron
home-assistant-component-tests.keymitt_ble
home-assistant-component-tests.ld2410_ble
home-assistant-component-tests.led_ble
home-assistant-component-tests.moat
home-assistant-component-tests.mopeka
home-assistant-component-tests.oralb
home-assistant-component-tests.qingping
home-assistant-component-tests.rapt_ble
home-assistant-component-tests.ruuvitag_ble
home-assistant-component-tests.sensirion_ble
home-assistant-component-tests.sensorpro
home-assistant-component-tests.sensorpush
home-assistant-component-tests.snooz
home-assistant-component-tests.switchbot
home-assistant-component-tests.thermobeacon
home-assistant-component-tests.thermopro
home-assistant-component-tests.tilt_ble
home-assistant-component-tests.xiaomi_ble
home-assistant-component-tests.yalexs_ble
python310Packages.aioesphomeapi
python310Packages.aioesphomeapi.dist
python311Packages.aioesphomeapi
python311Packages.aioesphomeapi.dist


Thumbnail Versions of Images

Some images such as digest front pages and user-created content can be large and take time to download - can smaller versions be delivered quickly while the full-size images are downloaded in the background and then replace the thumbnail?

Options to consider;

Downscale images (1920px, 1600px, 1366px, 1024px, 768px, 640px).
Use  to select images based on screen size.
User <picture> to specific multiple fallback sources.
Clicking image navigates to the full size image.
Lazily load images when the reader scrolls to them using .

https://developer.mozilla.org/en-US/docs/Learn/HTML/Multimedia_and_embedding/Responsive_images


Added flag to skip updating constraints

https://github.com/IllusionMods/HSPlugins/issues/51
If everything looks okay, please merge.
This is an implementation of a proposed modification to ShalltyB. Skip updating constraints in ApplyConstraint when flagged.

@ShalltyB

Hi, adding the new parameter to the AddConstraint method breaks BetterPenetration and ShalltyUtils (and maybe others that use NC)

I did it like this: NodeConstraints.cs - pastebin
That file also contains the modified UI buttons to move up and down the constraints, if you could also include it please 😋

Fixed. Thanks for trying!
I like the new move button. Added.
Reflection!
↓me

@ShalltyB @ManlyMarco
I've also committed fix #54. I'm sorry, but could you try it while you're at it?
The scope of impact of #54 overlaps with this pull request.
It is related to the order of NodeConstraints and Timelines.
#54 may be as sensitive as this pull request.
It's a separate fix and should be separated into a separate pull request, but I'd like to see them work together.

#54 and a summary of amendment.
Bug Summary
Objects that were previously tracked by NodeConstraints are no longer tracked.
Cause
The processing order of X (Expression_LateUpdate_Patches.Postfix @ Timeline) and Y (Expression_LateUpdate_Patches.Postfix @ NodeConstraints) was changed.
HarmonyBefore and HarmonyAfter were supposed to be processed in the order X -> Y, but GUIDs for Harmony were not specified and HarmonyBefore and HarmonyAfter did not work, resulting in the order Y -> X.
Fixes:
GUID set for Harmony.
https://github.com/IllusionMods/HSPlugins/pull/55/commits/15a16dcee7f7d6f71c48f7cbc4244c6c584edb76
As far as I searched in Github, it was only used to check for plugins. It looks safe. But I don't trust it anymore.
https://github.com/search?q=com.joan6694.illusionplugins.timeline&type=code
https://github.com/search?q=com.joan6694.illusionplugins.nodesconstraints&type=code

The DynamicBone fixes seems to be working well 👍
About  #54:
Tested in NodeConstraints 1.2.1 / 1.2.2 / 1.2.3.
I tested the linked scene and it worked with and without the GUID changes.
I also re-saved the scene in 1.2.1 and loaded it in 1.2.2 / 1.2.3 and it still works, I couldn't reproduce the bug shown in the user videos.

My load order is also NodeConstraints -> Timeline in the log and in the BepInEx_Manager 🤔
output_log.txt

Oops...
I guess the order in which the plugins are loaded has nothing to do with it...
No... in my environment, the order in which plugins are loaded seems to have something to do with the bug.
I added the following to the class definition of NodesConstraints so that NodesConstraints is loaded after the timeline.
[BepInDependency("com.joan6694.illusionplugins.timeline")]

Then the bug stopped happening without GUID modification.
After the bug was gone, I commented out BepInDependency in NodesConstraints and added BepInDependency to the timeline, and the bug occurred again.
//[BepInDependency("com.joan6694.illusionplugins.timeline")]
[BepInDependency("com.joan6694.illusionplugins.nodesconstraints")]]

With the bug in place, if the GUID modification is inserted, the bug will no longer occur.

By the way, if you add the following log with no GUID modification, I can confirm that Postfix in NodesConstraints comes first in my environment.
 Adding a BepInDependency changes the order that patches are added in, but it does not guarantee that they will stay in that order.

Before my fix, HarmonyBefore and HarmonyAfter were not working because the second argument, harmonyInstanceId, of CreateAndPatchAll was not set.
A random GUID was being given to the patch.
fix: https://github.com/IllusionMods/HSPlugins/commit/15a16dcee7f7d6f71c48f7cbc4244c6c584edb76
CreateAndPatchAll: https://github.com/BepInEx/BepInEx.Harmony/blob/0d24abfed64596ee88b4242b9c7e3daa849d6d28/HarmonyX2Interop/Harmony.cs#L313

You can still use HarmonyPriority since it doesn't rely on IDs.

What? Other plugins?
Isn't it dangerous?
I would suggest not to touch them unless there is a problem like this one.

There is no risk in switching from random ID to a known one. It's a bug because it was meant to use GUID in the first place.

I fixed this repository's.
There are so many, do you want to fix the others?
https://github.com/search?q=org%3AIllusionMods+CreateAndPatchAll&type=code
https://github.com/search?q=owner%3AManlyMarco CreateAndPatchAll&type=code

@ManlyMarco By the way, does your environment reproduce #54?

In most cases it's not really necessary to use GUID since nothing else will patch the same things, so I never bothered to fix it.
If you see any of this in IllusionFixes, BepisPlugins and similar common plugins then by all means do make PRs (be careful in case of shared GUIDs).
At the moment I don't have an environment to reproduce the issue, sorry.

I had the person who first reported the bug try it. In his environment, the GUID fix fixed the bug.
From what I see in the log, it is probably the same as my environment.
I wonder if there is something else affecting the order of the patches.
@ManlyMarco @ShalltyB
GUID fix should not be a problem in terms of content, so if you two don't mind, I would like to have this PR merged.
What do you think?

Thanks both for the reviews!


MMT Cluster - syncing not start

ubuntu@ip-172-30-0-35:~/mmt$ ./mmt start -e BIG --master-pem ../MyMemoryBot-1.pem -m<EMAIL_ADDRESS>
Starting MMT engine 'BIG'... OK

But nothing happened afterwords. Do syncing works in 0.11.1?
mmd status not providing any info

Various Problem found:

.pem files should have permission 600 -> We should notify the user with a warning
relative path for the pem do not work, absolute works. -> Fix
mmt status  should provide info about the sync.
mmd start --master user@ip is on README and  -m is in the cmd line help. Which one is correct?
Log file was hard to find.
mmd start said OK, but the sync failed.

Last merge from develop resolved all these issues:

mmt start checks pem permission before starting rsync
relative paths for pem files now works correctly
mmt start ends only when the models has been correctly synchronized
removed short version -m for option --master
in case of error, log file path is now printed on screen


How sam2_configs pushed in the {sys.path} in python?

By the segment-anything-2/INSTALL.md's comment
It says that

I got `MissingConfigException: Cannot find primary config 'sam2_hiera_l.yaml'`

This is usually because you haven't run the pip install -e . step above, so sam2_configs isn't in your Python's sys.path. Please run this installation step. In case it still fails after the installation step, you may try manually adding the root of this repo to PYTHONPATH via
export SAM2_REPO_ROOT=/path/to/segment-anything  # path to this repo
export PYTHONPATH="${SAM2_REPO_ROOT}:${PYTHONPATH}"

to manually add sam2_configs into your Python's sys.path.

But I can't find out
how sam2_configs are inserted by executing
pip install -e .

I know that acting means python setup.py
But in that file there is no indication that insert python.syspath

Hi @aza1200, do you encounter any failures after pip install -e . (or manually adding the repo root to PYTHONPATH) as in https://github.com/facebookresearch/segment-anything-2/blob/main/INSTALL.md?

I know that acting means python setup.py
But in that file there is no indication that insert python.syspath

Regarding pip install -e ., it would usually create a link to the SAM 2 repo path (in my environment, it adds a link in my Anaconda environment at
~/anaconda3/envs/base/lib/python3.10/site-packages/__editable__.SAM_2-1.0.pth) to allow editable installations. See details in https://setuptools.pypa.io/en/latest/userguide/development_mode.html.


Support for useFieldArray?

It's not clear whether useFieldArray is supported, this library doesn't export it.

Sorry for the delay, will add it, Thank you


Setup GitHub Workflow simulate Travis builds

https://www.reddit.com/r/ruby/comments/jt2uub/deep_dive_moving_ruby_projects_from_travis_to/
Travis will no longer be free for Open Source projects (And it's already slow for OS projects now)
This is to move to GitHub Action for CI
Jobs are split for Ruby & JS since there is no point running JS test for every ruby version / Gemfile

Pull Request Test Coverage Report for Build 691

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 68.337%

Files with Coverage Reduction
New Missed Lines
%

lib/i18n/js/utils.rb
1
97.3%

lib/i18n/js/dependencies.rb
2
50.0%

Totals

Change from base Build 687:
-0.2%

Covered Lines:
300

Relevant Lines:
439

💛  - Coveralls

Great! I'm migrating everything I have to GitHub Actions and would migrate i18-js eventually. 👍

Should I merge this PR anytime soon?
Or just merge it when you feel appropriate
Also need to remove Travis check from the project (which I am not sure how to)


A/B test report

Partially closes #292
Complements #296
Supersedes #294

In scope

Generate dashboard in PRs and upload it as an artifact
General refactoring of dashboard.py
Bar chart by test
Bar chart by runtime (absolute measures)
Bar chart by runtime / baseline runtime

Out of scope

Run additional runtimes on arbitrary git hashes (delivered by #296)
Display historical variance of baseline as a bracket overlay (read below)
Reduce noise (read below)

The plots highlight that the performance metrics are heavily affected by noise.
In some cases, this looks like a high variance between runs (which can be seen in the historical plots); in others, the difference is consistent throughout the history, but is not justified by a difference in runtime so I suspect there's some hidden warm-up cost that differs between the runs.
For example, latest and 0.1.0 only differ by the addition of pynvml, but the A/B charts comparing the two (top right plot above) show major differences all over the place - which are all noise. The memory usage plots look just as jittery. Ideally this chart should be used to take the informed decision to finalize a new release (all bars are negative or almost zero) and is also twitter gold - but at the moment, it's not usable due to the underlying data.
On one side, I can incorporate historical data into these plots to show brackets of variance for the baseline. I will in a successive PR. However, such brackets will just tell us what we can already see in the historical graphs: that most tests are swinging wildly and that a +/- 50% runtime is nothing to be surprised about.
We need to sit down and find the best way to actually reduce variance in runtime and memory usage.

@crusaderky it looks like you were hitting some permissions issues. Is it the same as #293? I've been unable to reproduce, but you might try reverting #240 to see if that helps

@crusaderky it looks like you were hitting some permissions issues. Is it the same as #293? I've been unable to reproduce, but you might try reverting #240 to see if that helps

No,  the issue was that

git@github.com:coiled/coiled-runtime.git works for git pulls from all forks and for git pushes to my own fork
https://github.com/coiled/coiled-runtime.git also works for git pushes to other forks, but annoyingly it doesn't work in gitkraken

Ready for review (assuming CI passes)

@crusaderky it looks like there is a legitimate error in the static site generation script

@jrbourbeau fixed now

@ncclementi @ian-r-rose I called my latest commit test-upstream; the setup job outputs "true" in its Check upstream task; however the upstream runtime is not there. Am I missing something?

I think things are working as expected. The upstream build should only install nightly versions of dask and distributed in the latest CI builds. For example, if you look at the linux Python 3.9 latest build on your latest commit (the test-upstream one), you'll see the the dev version of dask and distributed are installed from the dask conda channel. Does that help clarify, or am I missing something?

@crusaderky  this is a bit of a bad design in the name. What happened is that when you called test-upstream then "latest" will have the nightlies see that the nightlies where installed.  https://github.com/coiled/coiled-runtime/runs/8237542627?check_suite_focus=true#step:5:20
That logic is here https://github.com/coiled/coiled-runtime/blob/1614ab58128e3e3dcfa7d213db2c88e7c55ac6d4/ci/create_latest_runtime_meta.py#L41-L49

Re: noise of the samples
I believe if we have historic variances we can employ some statistical significance analysis on the new results. I believe this is something one typically uses a t-test for to verify whether the new measurement is in agreement to the old one. That's basically a very simple quantity t = (old_measurement - new_measurement) / sqrt(historic_variance / n_historic_data_points). If that value is large, this is true signal (depending on how we define our confidence interval) @ncclementi I believe you wrote the regression detection script. You might have more insight here than I do.
We can also look into running some tests multiple times. Particularly the ones that have very small runtimes (<30s) are likely more prone to noise and we can afford running them a couple of times.
Obviously, all of this is a follow up topic. I think this is already a great first step

I believe if we have historic variances we can employ some statistical significance analysis on the new results. I believe this is something one typically uses a t-test for to verify whether the new measurement is in agreement to the old one. That's basically a very simple quantity t = (old_measurement - new_measurement) / sqrt(historic_variance / n_historic_data_points).

A t-test is reasonable if your null hypothesis is that the data are normally distributed, if you don't have that guarantee (for example a long-tailed distribution due to outliers) then you're better off with a non-parametric test (such as Kolmogorov-Smirnov (available in scipy.stats.kstest)


GeometryWidget: Need to be able to save multiple single points in params

The current GeometryWidget CAN operate in a SinglePoint mode, where it simply writes the singlepoint coordinates to both min and max values in RendererParams.
But the ability to save multiple single points is desired, in tools such as Plot.

I think we've decided to move away from a heavy GeometryWidget and instead use multiple simpler widgets, for example, use widgets such as:
https://github.com/NCAR/VAPOR/blob/92e1d57f6b97676dc4a12ca5bb688f58f4dc0dd3/apps/vaporgui/QSinglePoint.h#L15


Added support for Microsoft Windows Media Meta Data (Xtra) and Quicktime GPS Coordinates (©xyz)

We use Windows Live Photo Gallery for maintaining our home movies and it uses a proprietary atom for storing tags, ratings, etc. This push adds support for those tags, as well as Quicktime GPS Coordinates.
I also wrote another example based on the ChangeMetaData example that allows for editing of the windows specific meta data. There's also a minor change to AppleNameBox to facilitate changing titles without hard coding the box type.

thank you. Looks good.


Windows paths with spaces not supported

PS C:\Users\Eric Peters> bloop about
C:\Users\Eric Peters\scoop\apps\python\current\python.exe: can't find '__main__' module in 'C:\\Users\\Eric'

Looks like something isn't quoted properly to handle paths with spaces.
original bloop.cmd
@echo off
python C:\Users\Eric Peters\scoop\apps\bloop\1.3.4\bloop %*

Changing to
@echo off
python "C:\Users\Eric Peters\scoop\apps\bloop\1.3.4\bloop" %*`

Fixes the problem, I'm not entirely sure where this file is coming from/how it is generated.
Version scoop install bloop - 1.3.4

Would you mind sending a PR to fix this @er1c ? Sorry, I didn't get to see this ticket.

@jvican I'm not actually sure howto fix it

Hmm looks like it's from: https://github.com/scalacenter/bloop/releases/download/v1.4.0-RC1/install.py

Added https://github.com/scalacenter/bloop/pull/1131


Rogers-Satchell volatility

Hello @nardew
Thanks for the hard work done recently on talipp.
See https://portfolioslab.com/tools/rogers-satchell
Adding such a volatity estimator should probably be considered.
Kind regards

Thanks @femtotrader , I will look into it

Thanks @nardew


initial commit

You should fork your own copy of the repo and make changes there, please

Yeah that PR was a mistake, my bad

From: Jen Looper @.>
Sent: Monday, June 19, 2023 8:48:16 PM
To: Digital-Humanities-Toolkit/timeline-builder @.>
Cc: George Lawrence @.>; State change @.>
Subject: Re: [Digital-Humanities-Toolkit/timeline-builder] initial commit (PR #5)
You should fork your own copy of the repo and make changes there, please
—
Reply to this email directly, view it on GitHubhttps://github.com/Digital-Humanities-Toolkit/timeline-builder/pull/5#issuecomment-1597678915, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJ3C7BR24KILLLBKVDD6WDTXMCUIBANCNFSM6AAAAAAZMJ7UY4.
You are receiving this because you modified the open/close state.Message ID: @.***>


Unit test generated by RoostGPT

Please pull these awesome changes in!
Test Name - go-unit-sample
AI Type   - Open AI
AI Model  - gpt-4
Test Type - Unit test
Use Type  - ui
Test Framework - gotest

Access Logs At - https://test.roost.ai/roostgpt/logs?trigger_id=e89e11fa-5c0c-4e4d-b0d0-2bff5faf84bb

Interface based structures and methods tests have issues.


Show bug in version 0.3.0

There is a bug in version 0.3.0 when comments directly after opening a child object will break the parser. Trying to demonstrate it with this PR.

Aye this should be working now. I'll release once I got #9 sorted.


Wrong bibliography order - first two digits of year ignored when sorting (using "alphanumerical" style only)

Description
For example, a bibliography entry dated 1902 is listed after one dated 2001. This only happens when using the "alphanumerical" citation style. Full example:
#cite("test1", "test2", style: "alphanumerical")
#bibliography("references.yml", title: "References", style: "ieee")

test1:
  title: Test1
  date: 1995
  author: Foo

test2:
  title: Test2
  date: 2003
  author: Bar

In the example above test2 is listed before test1. If I change from "alphanumerical" to "numerical" then test1 is listed before test2 as expected.
Discord discussion: https://discord.com/channels/1054443721975922748/1155551192752062705
Reproduction URL
No response
Operating system
Web app
Typst version

[X] I am using the latest version of Typst

It seems that the default ieee style defaults to using year for sorting, but it uses the yead by parsing form the tag Foo95 and interpret the last two digit as the numer 95 to do comparison, see here.
Maybe we should add an interface to access the true year field to do comparison instead of paring the last few digit to a number on which comparison is done?

This is now fixed.


[Update Request]: Git.Git to version 2.44.0

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Git.Git
Package Version
2.44.0
Please describe the changes you would like to see
Update Git.Git to version 2.44.0.
https://git-scm.com/download/win

https://github.com/microsoft/winget-pkgs/pull/140946


latency and memory consumption

ADD print info about latency and memory consumption to the training code.

https://github.com/QDelta/LQ-Nets/pull/4#issue-2229117158


How can I include other UI components with each image?

Suppose, other than the provided options, I want a toggle switch, a text box and check box for each image. Is it possible? and how?

I don’t understand your question. Can you provide some code to illustrate? Is there some reason you can’t render these elements alongside each “image”?

What in the library is preventing this? Seems like you should be able to render these elements with each thumbnail component and position everything with css.


Repeated timeouts in GitHub Actions fetching wheel for large packages

In the last few days since switching to uv, I have seen errors that I have not seen before with pip.
I see:
error: Failed to download distributions
  Caused by: Failed to fetch wheel: torch==2.2.1
  Caused by: Failed to extract source distribution
  Caused by: request or response body error: operation timed out
  Caused by: operation timed out
Error: Process completed with exit code 2.

I see this on the CI for vws-python-mock, which requires installing 150 packages:
uv pip install --upgrade --editable .[dev]
...
Resolved 150 packages in 1.65s
Downloaded 141 packages in 21.41s
Installed 150 packages in 283ms

I do this in parallel across many jobs on GitHub Actions, mostly on ubuntu-latest.
This happened with torch 2.2.0 before the recent release of torch 2.2.1.
It has not happened with any other dependencies.
The wheels for torch are pretty huge: https://pypi.org/project/torch/#files.
uv is always at the latest version as I run curl -LsSf https://astral.sh/uv/install.sh | sh. In the most recent example, this is uv 0.1.9.
Failures:

https://github.com/VWS-Python/vws-python-mock/actions/runs/8017894929/job/21902693117
https://github.com/VWS-Python/vws-python-mock/actions/runs/8000017693/job/21848747452
https://github.com/VWS-Python/vws-python-mock/actions/runs/8000017693/job/21848749557

Thanks for the feedback, I've opened an issue for your request

https://github.com/astral-sh/uv/issues/1921

Thank you @zanieb ! I don't know the value of having this issue open, but I'll leave it to you to close if desired.

In #1921 my co-worker noted that this might be a bug in the way we're specifying the timeout so I'll recategorize this one and leave it open.

Looking at the actions runs, all the passing actions take ~30s, while the failing ones error after 5min, which is our default timeout, so this looks like a network failure (in either github actions or rust)

I'm not seeing any timeouts anymore with the two most recent versions (https://github.com/konstin/vws-python-mock/actions). Could you check if this now solved?

I have not seen this issue since posting. Thank you for looking into this.

I'll close it for now, please feel free to reopen should it reoccur

@konstin I do not have permissions to re-open this issue. I can create a new one, but it is probably easier if you re-open this.
This failure has reoccurred:

https://github.com/VWS-Python/vws-python-mock/actions/runs/8134588970/job/22227737289
https://github.com/VWS-Python/vws-python-mock/actions/runs/8134588970/job/22227737596

I'm seeing very similar error message for non pytorch package that's also pretty large. It's ~400 MB wheel and consistently gives me,
(bento_uv2) pa-loaner@C02DVAQNMD6R training-platform % uv pip install --index-url=$REGISTRY_INDEX data-mesh-cli==0.0.66
error: Failed to download: data-mesh-cli==0.0.66
  Caused by: The wheel data_mesh_cli-0.0.66-py3-none-any.whl is not a valid zip file
  Caused by: an upstream reader returned an error: request or response body error: operation timed out
  Caused by: request or response body error: operation timed out
  Caused by: operation timed out

Package is company internal one though, but I think only notable thing is very large size (it vendors spark/java stuff).

I have changed the title of this to not reference torch. It recently happened with nvidia-cudnn-cu12, another large download.
As another example, https://github.com/VWS-Python/vws-python-mock/actions/runs/8262236134 has 7 failures in one run.

It can happen on Read the Docs as well, not only GHA https://beta.readthedocs.org/projects/kedro-datasets/builds/23790543/

Spotted it locally today inside a local Docker image running under QEMU
error: Failed to download distributions
  Caused by: Failed to fetch wheel: nvidia-cublas-cu12==<IP_ADDRESS>
  Caused by: Failed to extract archive
  Caused by: Failed to download distribution due to network timeout. Try increasing UV_HTTP_TIMEOUT (current value: 300s).

I encountered the problem when I used either uv or pip to download large wheels (for pip, the issue is like https://github.com/pypa/pip/issues/4796 and https://github.com/pypa/pip/issues/11153), so I think the root cause is the network. However, I am wondering if uv can be smarter to retry automatically, like something in https://github.com/pypa/pip/pull/11180.

Worth trying 0.1.35, which includes https://github.com/astral-sh/uv/pull/3144

It seems likely that this is resolved by #3144

I encountered the problem when I used either uv or pip to download large wheels (for pip, the issue is https://github.com/pypa/pip/issues/4796 and https://github.com/pypa/pip/issues/11153), so I think the root cause is the network. However, I am wondering if uv can be smarter to retry automatically, like something in https://github.com/pypa/pip/pull/11180.

that would be a great feature. we have our dev environments behind TLS inspection and some packages often run into a timeout due too slow inspection. we can reproduce this with a browser and the download gets stuck until a timeout. in the browser we can just click resume and the browser reconnects snd downloads the remaining part. with uv we don't have a retry with resume. so it starts from scratch and gets stuck again.
+1 for retry with resume

Going to close for now, but we can re-open if this comes up again post-changing the timeout semantics.


docs - identify from as optional when a template is in use

A quick doc update to clarify the optionality of 'from' when a template is used.

Thanks for the contribution @astletron.