Questions about integrating the package

Hi,
I'm currently trying to integrate the package (which I think is great by the way), however I'm running into a few things I can't find answers on in the documentation:

I would like to invoice at the first of the month. The so called 'anchoring the billing cycle'. As described here: https://github.com/laravel/cashier-mollie/issues/66. Is this currently possible?
I also would like to apply metered billing based on the resources used in the past month. I read on https://www.cashiermollie.com/12-faq.html#how-is-prorating-handled that "Cashier Mollie applies prorating by default. With prorating, customers are billed at the start of each billing cycle." So is it possible to change this default?
I develop on localhost. Is it possible to test on localhost? Maybe not even the connection with Mollie, but to generate some subscriptions and orders, so I can test for example how an invoice looks like or testing a page that lists the invoices?

Kind regards

Hi @michelterstege81,
sorry for taking so long to answer your questions!

I would like to invoice at the first of the month. The so called 'anchoring the billing cycle'. As described here: https://github.com/laravel/cashier-mollie/issues/66. Is this currently possible?

Anchor billing is not supported out of the box by laravel-cashier-mollie. However, you can create your own AnchoredIntervalGenerator class that implements the Laravel\Cashier\Plan\Contracts\IntervalGeneratorContract interface. In your cashier_plans.php file you can then configure your plan to use this generator. The next version of this package will likely contain this IntervalGenerator out of the box.
An example of an implementation for the AnchoredIntervalGenerator could look like the following:
<?php

use Carbon\Carbon;
use Illuminate\Support\Str;
use Illuminate\Support\Arr;
use Laravel\Cashier\Plan\Contracts\IntervalGeneratorContract;
use Laravel\Cashier\Subscription;
use Laravel\Cashier\Plan\BaseIntervalGenerator;

class AnchoredIntervalGenerator extends BaseIntervalGenerator implements IntervalGeneratorContract
{
    /**
     * @var array
     */
    protected array $configuration;

    public function __construct(array $configuration)
    {
        $this->configuration = $configuration;

        $this->useCarbonGivenDayOrLast();
    }

    /**
     * @param  \Laravel\Cashier\Subscription  $subscription
     * @return \Carbon\Carbon
     */
    public function getEndOfNextSubscriptionCycle(Subscription $subscription = null)
    {
        $cycle_ends_at = $subscription->cycle_ends_at ?? now();

        if ($this->isMonthly()) {
            return $cycle_ends_at
                ->addMonthNoOverflow()
                ->setDaysToGivenDayOrLastOfMonth($this->day());
        }

        $endOfNextSubscriptionCycle = $cycle_ends_at->month($this->month());

        if ($endOfNextSubscriptionCycle->isPast()) {
            $endOfNextSubscriptionCycle = $endOfNextSubscriptionCycle = $endOfNextSubscriptionCycle->addYearNoOverflow();
        }

        return $endOfNextSubscriptionCycle->setDaysToGivenDayOrLastOfMonth($this->day());
    }

    /**
     * @return int|string
     */
    protected function day()
    {
        return Arr::get($this->configuration, 'day');
    }

    /**
     * @return int
     */
    protected function month(): int
    {
        $month = Arr::get($this->configuration, 'month', 1);

        return $month > 12 ? 12 : $month;
    }

    /**
     * @return string
     */
    protected function period(): string
    {
        return Arr::get($this->configuration, 'period');
    }

    /**
     * @return bool
     */
    protected function isMonthly(): bool
    {
        return Str::startsWith($this->period(), 'month');
    }

    private function useCarbonGivenDayOrLast(): void
    {
        Carbon::macro('setDaysToGivenDayOrLastOfMonth', function (int|string $day) {
            if (in_array($day, ['end', 'last']) || $day > $this->daysInMonth) {
                $day = $this->daysInMonth;
            }

            return $this->day($day);
        });
    }
}

An example configuration to anchor the billing cycle to the first of the month would be
// config/cashier_plans.php
[
    // ...

    ‘plans’ => [
        ‘example_1’ => [
            // ...

            ‘interval’ => [
                'generator' => AnchoredIntervalGenerator::class,
                'day' => 1,
                'period' => 'month',
            ],

            // …
]

This will only ensure that the billing date is anchored to the first of the month, but it will not adapt the price for an incomplete first interval.
—

I also would like to apply metered billing based on the resources used in the past month. I read on https://www.cashiermollie.com/12-faq.html#how-is-prorating-handled that "Cashier Mollie applies prorating by default. With prorating, customers are billed at the start of each billing cycle." So is it possible to change this default?

Prorating refers to the calculation of charges based on partial usage, and it does not determine when a user will be billed. In other words, prorating ensures that customers are charged only for the resources they have used in a billing cycle, regardless of when the billing cycle begins or ends.
You can implement metered billing by configuring the order_item_preprocessors in your cashier_plans config. A MeteredBillingPreprocessor could look something like the following:
use Laravel\Cashier\Order\BaseOrderItemPreprocessor;
use Laravel\Cashier\Order\OrderItem;
use Laravel\Cashier\Order\OrderItemCollection;

class MeteredBillingPreprocessor extends BaseOrderItemPreprocessor
{
    /**
     * @param  \Laravel\Cashier\Order\OrderItemCollection  $items
     * @return \Laravel\Cashier\Order\OrderItemCollection
     */
    public function handle(OrderItemCollection $items)
    {
        // You can add, remove or modify order items here.
        // Each order item relates to 1 line on the invoice.

        // For metered billing, you typically:
        // 1. fetch metered data from within your app
        // 2. apply it to the order items
        // 3. if necessary, notify the meters that the costs were incurred/reset the meters

        $owner = $items->owners()->first();

        $meters = $owner->getMeters();
        $meterItems = $meters->map(function ($meter) {
            return OrderItem::make([
                'process_at' => now(),
                'description' => $meter->description,
                'currency' => $meter->currency,
                'unit_price' => $meter->unit_price,
                'quantity' => $meter->quantity,
                'tax_percentage' => $this->tax_percentage,
            ]);
        });

        $items = $items->concat($meterItems);

        $owner->resetMeters(); // optionally put this in a separate ResetMetersPreprocessor right after the PersistOrderItemsPreprocessor

        return $items;
    }
}

Note that at this time it is not possible to add preprocessors to the first payment. This means you still have to reimburse the user for the first billing cycle for the amount he overpaid.
—

I develop on localhost. Is it possible to test on localhost? Maybe not even the connection with Mollie, but to generate some subscriptions and orders, so I can test for example how an invoice looks like or testing a page that lists the invoices?

To test the integration with Mollie and Cashier, you can use a test API key provided by Mollie and make your webhook URL available to the Mollie app.
To obtain a test API key, you can navigate to the Developers section of your Mollie account dashboard and select API keys. From there, locate the key with the test_ prefix, copy it, and paste it into your .env file under MOLLIE_KEY.
Expose your app to mollie by running the valet share command (Laravel Valet). Otherwise, you can use tools like Expose or ngrok.
The process is then to

create a subscription like shown in the documentation example
go to the returned url and execute the payment of the subscription (it will show that testmode is activated)
wait for the webhook request for your payment
find the order that was created for your subscription and get the invoice

$order = $user->orders()->first();

$user->findInvoice($order->number);

If you want only to see how the design of an invoice looks, I would recommend you to just manually create an order Item and use it to setup the order to retrieve the invoice:
Route::get(‘/invoice', function () {
    $user = User::first();

    $orderItemsCollection = $user->orderItems()->create([
        'process_at' => now(),
        'description' => 'test',
        'currency' => 'EUR',
        'unit_price' => 2499,
        'tax_percentage' => 0.19,
        'quantity' => 1,
    ])->toCollection();

    $order = Order::createFromItems($orderItemsCollection);

    return $order->invoice()->download();
});

—

I am currently gathering all the necessary information to answer your second question, and will get back to you as soon as possible.

Thanks for the very comprehensive answer!
I'm currently working on another project, but I'll definitely be working on this one soon.

Closing this for now, let me know if it should be reopened.


When starting a chal: Submission request failed: JSON.parse: unexpected character at line 1 column 1 of the JSON data

Tracking down what's going on here

It's a 504

we should probably check the status code before trying to parse JSON, and if it doesn't parse display what it is


Cow: Watch fajshop.ae

Cow requests the watch of the watch_keyword fajshop\.ae. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

fajshop\.ae has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cocomac in Charcoal HQ


chore: Template upgrade

Don't autoescape when rendering changelog
Generate credits on-the-fly, when building docs
(for this we're using mkdocs-macros-plugins)
Update docs deploy action to point to org pages
Add cleancov argument to test action,
to allow combining coverage for all tested Python versions
Reorganize docs navigation
Add coverage HTML report into docs
Update dev-dependencies, remove ipython

Yes I already have the fix for it, will push tomorrow.


Documentation

please write it...

Documentation is contingent on @dloman fixing the ESP code to actually Do The Needful™.

wut does that mean?


linux glibc/android memory policies flags

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit cd8a8722b6745634eaa2b376b82f6602ad2023f0 has been approved by Amanieu

:hourglass: Testing commit cd8a8722b6745634eaa2b376b82f6602ad2023f0 with merge add1f3db6ec5d327d09b453f7fbfe85a47698932...

:broken_heart: Test failed - checks-actions

Emscripten pretends to be Linux but isn't really linux... It should be excluded here.

@bors r+

:hourglass: Testing commit 510c45896a6987af77fa15cc2c37c049b6d375be with merge c55c52d35f8710e323cd5fe91aa3d73f67f054ad...


Make sure autosuggest handles bold formatting

Want HTML bold/strong tags to be accepted in suggestions dropdown.

Also adjust the SuggestComponent to use AnalyzingInfixLookupFactory instead of FuzzyLookupFactory.

Done!


Data loader fixes

Making a bugfix PR for the data loader and sorting issue for the 0.15.0 release
Due to changes that have already been merged into master and the changes that are coming in https://github.com/apache/incubator-druid/pull/7643 it does not make sense to merge this into master.
Technically porting this fix or merging https://github.com/apache/incubator-druid/pull/7643 will have to be a release blocker for the 0.16.0 release.
Fixes here:

Misc copy adjustments to align better with Druid concepts (thank you @gianm for the feedback)
The ability to go to the raw spec dialogs from the data loader init screen (see screenshot)
A fix for the same problem as one identified in https://github.com/apache/incubator-druid/pull/7657
A button to edit the last added transform if it created an error

Ok now that the mentioned PRs were merged this is essentially straight up a backport


Serve command loads server in wrong environment

Laravel Version: 5.8.3
PHP Version: 7.2.15
Database Driver & Version: sqlite locally (irrelevant I believe)

Description:
When using the artisan serve command the correct environment is not loaded.
Steps To Reproduce:

run php artisan serve --env=testing with a .env.testing file with APP_ENV=testing
put a {{ env("APP_ENV") }} in any view or anything
Instead of testing, see local

It seems the environment is correctly loaded for artisan commands, but not for the resulting server process. Contrast the above with:

run php artisan tinker --env=testing with a .env.testing file with APP_ENV=testing
run env("APP_ENV")
result is as one would expect: testing

Did you try resetting the configuration cache: php artisan config:clear

This works for me. Can you first please try one of the following support channels? If you can actually identify this as a bug, feel free to report back.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Try resetting the configuration cache: php artisan config:clear

Thanks. I have done this.
I do believe this to be a bug. Steps to reproduce based solely on laravel/laravel:

composer create-project --prefer-dist laravel/laravel test
copy .env to .env.testing
change in .env.testing APP_ENV to testing
add <h1> {{ env('APP_ENV') }} </h1> to welcome.blade.php
run php artisan serve --env=testing

The page will show local, not the expected testing. From within tinker however the correct testing will be shown.
If still unconvinced this is a bug I'll go to one of the support channels.

I can reproduce this.

I can reproduce this. It works in Laravel 5.7.

Correct. I ran into this in 3 apps upon upgrade to 5.8(.3).

That's very odd. I just tested again and I can indeed reproduce it. Not sure why it didn't the first time I tried.

I really can't see what changed it. Nothing really has changed to the ServeCommand.

Maybe this is caused by #27462 and the related PRs. /cc @GrahamCampbell

isnt the env helper meant to only be used in config files?
(not sure if actually related or not)

@rs-sliske We are only using it to demonstrate the issue. config('app.env') doesn't work either.

@rs-sliske We are only using it to demonstrate the issue. config('app.env') doesn't work either.

Correct. Problem is not in the env helper as far as I can see. If I drop a psyshell in index.php it seems that after constructing the $app the actual environmentFile() and related are off. Something in booting the app seems to be wrong, causing the env to be set to the .env instead of the correct file.
To unfamiliar with the inner workings of laravel to help much I'm afraid.

This is caused by the different ways Laravel 5.7 and 5.8 set environment variables.

5.7 sets the variables in $_ENV and $_SERVER and calls putenv().
5.8 only sets the variables in $_ENV and $_SERVER.

Adding the putenv() call to Laravel 5.8 fixes the issue: https://github.com/staudenmeir/framework/commit/1412636087d517842991c1817b230c3faf83b9cd
All the tests still work. Can you approve this @GrahamCampbell?

UPDATE
Apparantely, it's not that simple: Now removing E from the variables_order setting in php.ini completely breaks environment handling.

I tried upgrading Lumen, and it broke my app for me. The problem was that it wasn't seeing a required env variable set with export before starting the app.
I gather that Dotenv switched from using getenv to $_ENV due to threading issues with the former, the problem is that $_ENV only gets populated with the existing environment if E is part of variables_order which isn't standard. Standard PHP install on Ubuntu doesn't include it (I believe it's a security measure). A bit of asking around shows that  and at least some OSX installs doesn't either, and for docker containers it's a tossup (pretty much everything based on Ubuntu will probably have the problem).
I think that not using put/getenv was an explicit choice, but considering the amount of people that'll get hit by this, whether some compatibility fix can be done. Maybe preloading $_ENV from getenv if the former is empty when starting up. @GrahamCampbell is probably the person to talk to.

@staudenmeir PutenvAdapter was specifically removed to make the env helper read-only. Can we fix this any other way?

Why is it adding putenv fixes this/

@GrahamCampbell it's not putenv that fixes it, but the fact that PutenvAdapter uses getenv to read env variables.

@xendk You need both getenv() and putenv() to fix the issue.

@driesvints Why is getenv() not read-only?
@GrahamCampbell I don't fully understand it. Somehow, php -S treats getenv() and $_ENV/$_SERVER differently.

@staudenmeir You do? Why? The problem with PutenvAdapter is that it shadows EnvConstAdapter for setting too so all variables is then set with putenv() and doesn't show up in $_ENV anymore.

@xendk php artisan serve --env only works for me when I add the PutenvAdapter to LoadEnvironmentVariables::createDotenv() and env().

@StanAngeloff yeah, and that completely shadows EnvConstAdapter. You might want to try and hack EnvConstAdapter::get to try using getenv if the key was not found in $_ENV instead, and see if that doesn't fix things too.

I'm suffering the same issue here:
https://github.com/laravel/framework/issues/27949
Removing the PutenvAdapter  adapter breaks loads of third-party composer libraries too.

Here's another issue related to this:
https://github.com/laravel/framework/issues/27913

This has been fixed in the latest release 5.8.7.

This has been fixed in the latest release 5.8.7.

Thanks a lot!

Thanks for everyone's feedback and help with this.

I have liberated Laravel 5.7's env() from the dustbin of the git repo and made it its own package. What's more, using some real composer hacks, I have ensured that it will always be autoloaded first, overriding whatever Laravel's current env() does.
Benefits:

Works with any package, even non-Laravel ones (it was my secondary reason)
You no longer have to fear Laravel just going one day "Oops! We're not going to read from the environment anymore! Too bad!" on a non-major release.

https://github.com/phpexpertsinc/Laravel57-env-polyfill
To install: composer require phpexperts/laravel-env-polyfill.

It is now Augustus 2022 and this is still a issue! This hasnt't been fixed yet!! Does anybody have a solution?

I'm having an issue with this as well.

I created a solution for this years and years ago, back in 2019.
I ported Laravel 5.7's env() to be usable by any app. It even overrides Laravel's implementation, if the package is included:
composer require phpexperts/env

Then just env('MY_VAR'); like you would in pre-5.8...


[units/urb-example-ensayo/webapp] Server rendering of Ensayo does not open side menu

When navigating to a specific essay through menu the nav menu remains open:

However, when navigating by URL like http://<IP_ADDRESS>:4444/ensayo/item/RW5zYXlvOjMxMDE2YzYyLWIyMjQtNDhmNC1iZWY0LThiNjg0OTE5OWVhZA== the nav menu remains closed:

Navigation will be completely revamped. If the issue occurs in new navigation, this can be re-opened.


bug:  ascii export fails when baking animation and using default material.

Hello, I am trying to install the glTF2.0 exporter and have followed the instructions to the letter.
when I try to export a simple cube scene, I get the following error:
(I also note in the install that the version numer is 2.78, which is cool, but since so many things are broken still I can't use it.  Does this exporter work with 2.79?)

Traceback (most recent call last):
File "/Users/bay/Library/Application Support/Blender/2.79/scripts/addons/io_scene_gltf2/init.py", line 376, in execute
return gltf2_export.save(self, context, export_settings)
File "/Users/bay/Library/Application Support/Blender/2.79/scripts/addons/io_scene_gltf2/gltf2_export.py", line 85, in save
generate_glTF(operator, context, export_settings, glTF)
File "/Users/bay/Library/Application Support/Blender/2.79/scripts/addons/io_scene_gltf2/gltf2_generate.py", line 2824, in generate_glTF
generate_images(operator, context, export_settings, glTF)
File "/Users/bay/Library/Application Support/Blender/2.79/scripts/addons/io_scene_gltf2/gltf2_generate.py", line 2110, in generate_images
create_image_file(context, blender_image, path, file_format)
File "/Users/bay/Library/Application Support/Blender/2.79/scripts/addons/io_scene_gltf2/gltf2_create.py", line 351, in create_image_file
copyfile(src_path, dst_path)
File "/Applications/Blender/blender.app/Contents/Resources/2.79/python/lib/python3.5/shutil.py", line 120, in copyfile
with open(src, 'rb') as fsrc:
IsADirectoryError: [Errno 21] Is a directory: '/Users/bay/Google Drive/Foo Foo/Foo/Items/Foo/../../../../../../../'
location: :-1

ok so I've found out a little more about what's going on.
if I bake the action before hand and use "pose" instead of object and disable export materials (I was using default blender material) the glTF file will export without error.

adding a blend file that will fail to export .gltf
( exporting the same file as .glb works fine)
the blender scene is it's a simple smoothed cube skinned to a single bone with a curve type set to bounce instead of bezier.
if the animation gets baked manually before export and export materials is false, it will export as .gltf without error.
cube.blend.zip

This is now working correctly on the new exporter: https://github.com/KhronosGroup/glTF-Blender-IO.


Update jsdoc-default.css

Fixed overflow bug.

Q
A

Bug fix?
yes

License
Apache-2.0

This was my problem at first.

Hope this helps.
Mattia.

Now, the whole left side of the UI gets a scrollbar when 1 class has a long name.
a {
    overflow-wrap: break-word;
}

This works much better since it simply breaks the line

@Cratyz
I know it but sometimes breaking words can lead to some ugly behaviour.
Since jsdoc is not responsive ready, having scrollbar can be a little bit better.

In response to the suggestion targeted at links, this overflow problem also occurs for other elements (I have a table with fairly long descriptions getting made unreadable without this).


Relationship required-but-non-localized field validation error when updating global through API

Link to reproduction
https://github.com/qbeauperin/payload/tree/bug/global-localization-required-field
To Reproduce

run payload with this config ^
go to the "Required Relationship Global" global, and add content in English
run the following code to try and update Spanish through the API:

const response = fetch("http://localhost:3000/api/globals/required-relationship-global?locale=es", { method: 'POST', headers: { 'Content-Type': 'application/json', }, cache: 'no-cache', body: JSON.stringify({text: "some different Spanish text"}) });

notice how it returns a validation error: The following field is invalid: nonLocalizedRequiredRelationship

Describe the Bug
On the other hand, the exact same config has been duplicated in a collection and work as expected:

create a new item in the "Required Relationship Collection" collection and add content in English
run the following code to try and update Spanish through the API:

const response = fetch("http://localhost:3000/api/required-relationship-collection/[yourdocidgoeshere]?locale=es", { method: 'PATCH', headers: { 'Content-Type': 'application/json', }, cache: 'no-cache', body: JSON.stringify({text: "some different Spanish text"}) });

notice how the update went through properly

Payload Version
1.8.3

Hey @qbeauperin! This has been reported (just linking for clarity, not like you would have known) here. You can install 1.8.4-canary.2 to get the fix, or wait until 1.8.4 is released 👍

Wow. Long day, closed the wrong issue. 😵💫

Oh sweet, I'll pull 1.8.4 right now and test it out. Thanks Jarrod! 😄

Issue still occurs unfortunately, same repro steps. 😢

@qbeauperin wow good catch here. THANK YOU for your detailed report, too. I was able to clone down your repository and easily recreate the bug. I've got the fix in PR https://github.com/payloadcms/payload/pull/2776 which will go out in the next  release.


Unrecognized option --module-path with JDK 13

I've tried all the things said in this post: https://stackoverflow.com/questions/55855335/javafx-and-maven-in-intellij-java-home-set-but-unrecognized-option-module-pa
However, not even using <executable> works for me.

You are running with Java 1.8, that doesn't recognize modules nor the module-path option.
Make sure you don't have it set in the Windows Path environment variable

@jperedadnr , do you mean we shouldn't set the jdk 8 in windows path variable??


add file add README.md2. delete ambari_plugin/common-services/DOLPHIN/2.0.0

Tips

Thanks very much for contributing to Apache DolphinScheduler.
Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds checkstyle plugin.)
Brief change log
(for example:)

Add maven-checkstyle-plugin to root pom.xml

Verify this pull request
(Please pick either of the following options)
This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added dolphinscheduler-dao tests for end-to-end.
Added CronUtilsTest to verify the change.
Manually verified the change by testing locally.

Codecov Report

Merging #2546 into dev will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #2546      +/-   ##
============================================
- Coverage     35.96%   35.95%   -0.02%
+ Complexity     2409     2408       -1
============================================
  Files           424      424
  Lines         19595    19595
  Branches       2390     2390
============================================
- Hits           7048     7046       -2
- Misses        11924    11926       +2
  Partials        623      623

Impacted Files
Coverage Δ
Complexity Δ

...he/dolphinscheduler/api/service/LoggerService.java
82.60% <0.00%> (-8.70%)
9.00% <0.00%> (-1.00%)

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 4f9d8ca...3b81829. Read the comment docs.

chunyang,
1.3.0 will be the next version number after the vote from the dev@mailing list

The image with the wrong version number has been updated


Adding "Information Card" component to Storybook

Would love if someone could help create this "Information Card" component to add to our Storybook (storybook.backstage.io)! 🙏
Usage
The information card is one of the most basic UX components that will exist in Backstage. Its purpose is to provide informational content to users. There's the option of adding a footer that includes a link to another page/site/etc. There's also the option of adding a subhead to the header.

Specs

Let me know if you have any questions!!

@mateusmarquezini you seem to enjoy these kinds of challenges 😜

@mateusmarquezini you seem to enjoy these kinds of challenges

haha yeah, in fact! Feel free to assign this issue to me! :smile:

Closer in #400


AI Transcript acceleration by FFmpeg GPU support

What is the business background? Please provide a description.
AI transcript task is slow, especially ffmpeg encode part, so the transcript live stream with subtitle delay a lot.
Is your feature request related to a problem? Please describe.
Large delay and poor user experience
Describe the solution you'd like
AI transcript support GPU for ffmpeg encode, for example NVIDIA GPU for accelerate.
Describe alternatives you've considered

Use a better CPU
Use low-resolution video streaming

Additional context

ffmpeg -hwaccel cuda -i input.ts -vf "subtitles=subtitles.srt:force_style='Alignment=2,MarginV=20',hwupload_cuda" -c:v h264_nvenc -profile:v main -preset medium -rc:v vbr -cq:v 19 -bf 0 -acodec aac -copyts -y output.ts

You are now able to setup the video codec parameters for transcript.

Strongly recommand setting the -bt 0 to be compatible with both live stream and WebRTC.

Fixed in v5.14.11


[FEATURE REQUEST] Implement NumericNonOverflowingMod in Interval domain

Description
It seems that NumericNonOverflowingMod always returns top when using the interval domain.
Motivation
This kind of operator in particular is also used in rust-lisa; also further expansion of arithmetic operations should improve some analysis.

Instead of returning TOP, we could return the interval [0, +Inf].


Add has method to permissions

Summary
Add a has method to the permissions enum. It's confusing for beginners to do:
bool(permissions_1 & permissions_2)

and this should make that easier.
Checklist

[x] I have run nox and all the pipelines have passed.
[ ] I have made unittests according to the code I have added/modified/deleted.

Reason for closure was discussed in Discord, but just wanted to leave this here in case anybody stumbles upon this and is confused about it
There is already an extensive documentation on how to do these actions in the Permissions docstring as well as in the IntFlag docstring. Furthermore, IntFlag provides similar methods like .all() and .any(). If anybody is still confused and needs help, make sure to drop by the Discord and we will try to help you to the best of our abilities


clock is not updating

Hello, I noticed today that clock is not updating:

ZSH_THEME="powerlevel9k/powerlevel9k"
P9K_MODE='awesome-patched'

P9K_PROMPT_ON_NEWLINE=true
P9K_RPROMPT_ON_NEWLINE=false
P9K_MULTILINE_FIRST_PROMPT_PREFIX_ICON=""
P9K_MULTILINE_LAST_PROMPT_PREFIX_ICON=""
P9K_PROMPT_ADD_NEWLINE=true
P9K_TIME_FORMAT="%D{\ue184  %d/%m/%y}"
P9K_TIME_ICON=''
P9K_WHITESPACE_BETWEEN_RIGHT_SEGMENTS='  '
P9K_VCS_SHOW_CHANGESET=true
P9K_VCS_CHANGESET_HASH_LENGTH=6
P9K_BATTERY_VERBOSE=true
P9K_BATTERY_LOW_THRESHOLD=30
P9K_BATTERY_LOW_ICON_COLOR="red"
P9K_BATTERY_CHARGING_ICON_COLOR="springgreen1"
P9K_BATTERY_CHARGED_ICON_COLOR="white"
P9K_BATTERY_DISCONNECTED_ICON_COLOR="cyan"
P9K_BATTERY_LOW_ICON="\UF240"
P9K_BATTERY_CHARGING_ICON="\UF240"
P9K_BATTERY_CHARGED_ICON="\UF1E6"
P9K_BATTERY_DISCONNECTED_ICON="\UF240"
P9K_COMMAND_EXECUTION_TIME_THRESHOLD=0
P9K_MULTILINE_LAST_PROMPT_PREFIX_ICON="%K{cyan}%F{black}   `date +%H:%M` %f%k%F{cyan}%f "
P9K_STATUS_CROSS=true
P9K_PHP_VERSION_ICON=$'\uE63D'

P9K_LEFT_PROMPT_ELEMENTS=(os_icon root_indicator dir dir_writable vcs)
P9K_RIGHT_PROMPT_ELEMENTS=(status command_execution_time background_jobs node_version php_version time battery)

From what I saw on other's scripts in show off your config page, it normally should work, and it used to work, I'm afraid it stopped working after latest zsh upgrade, but I'm not really sure because I noticed it just now...
Thanks in advance!

Are you sure this ever worked or did you just notice this now?
You fill P9K_MULTILINE_LAST_PROMPT_PREFIX_ICON with the evaluated expression which means it is filled with the date when you sourced your .zshrc. P9K never sees any date command and probably never could unless you used single quotes before. Maybe is was single qutes before you switched to next?
For a solution for your problem, you could try the following:
P9K_LEFT_PROMPT_ELEMENTS=(os_icon root_indicator dir dir_writable vcs newline time)
P9K_PROMPT_ON_NEWLINE=false

Also, note that the config site is from P9K-users and anybody can write to it afaik. Config might be for older versions of P9K. I don't know if there is any checking done if it actually works.
PS: I'm a bit confused why you use the time segment in the P9K_RIGHT_PROMPT_ELEMENTS as well, but you can display as many watches as you want ;)

@Syphdias Indeed you are right... I totally missed that...
My current settings are:
ZSH_THEME="powerlevel9k/powerlevel9k"
P9K_MODE='awesome-patched'

P9K_PROMPT_ON_NEWLINE=false
P9K_RPROMPT_ON_NEWLINE=false
P9K_MULTILINE_FIRST_PROMPT_PREFIX_ICON=""
P9K_MULTILINE_LAST_PROMPT_PREFIX_ICON=""
P9K_PROMPT_ADD_NEWLINE=true
P9K_TIME_FORMAT="%D{%H:%M}"
P9K_TIME_BACKGROUND='cyan'
P9K_DATE_FORMAT="%D{\ue184  %d/%m/%y}"
P9K_DATE_ICON=''
P9K_WHITESPACE_BETWEEN_RIGHT_SEGMENTS='  '
P9K_VCS_SHOW_CHANGESET=true
P9K_VCS_CHANGESET_HASH_LENGTH=6
P9K_BATTERY_VERBOSE=true
P9K_BATTERY_LOW_THRESHOLD=30
P9K_BATTERY_LOW_ICON_COLOR="red"
P9K_BATTERY_CHARGING_ICON_COLOR="springgreen1"
P9K_BATTERY_CHARGED_ICON_COLOR="white"
P9K_BATTERY_DISCONNECTED_ICON_COLOR="cyan"
P9K_BATTERY_LOW_ICON="\UF240"
P9K_BATTERY_CHARGING_ICON="\UF240"
P9K_BATTERY_CHARGED_ICON="\UF1E6"
P9K_BATTERY_DISCONNECTED_ICON="\UF240"
P9K_COMMAND_EXECUTION_TIME_THRESHOLD=0
P9K_STATUS_CROSS=true
P9K_PHP_VERSION_ICON=$'\uE63D'

P9K_LEFT_PROMPT_ELEMENTS=(os_icon root_indicator dir dir_writable vcs newline time)
P9K_RIGHT_PROMPT_ELEMENTS=(status command_execution_time background_jobs node_version php_version date battery)

And it looks like that:

The reason I did it with multiline previously is that I don't like that my right prompt is not aligned with my left prompt. I wanted my left and right prompt to be aligned in order to give the feel of the header and below them is where I type my commands :)

would be cool if we could use segments in multiline :)

@panosru I just briefly looked at the code how P9K_RPROMPT_ON_NEWLINE works...and it's a bit hacky...

I wanted my left and right prompt to be aligned in order to give the feel of the header and below them is where I type my commands :)

I think @dritter is working on making newline also available on RPROMPT and some other stuff in #856. Looking at his code, I think he is aware of this since he dynamically adds RPROMPT_PREFIX and RPROMPT_SUFFIX.
If you want to hard code it for now, you could apply this patch. I've done very little testing though...
diff --git a/generator/default.p9k b/generator/default.p9k
index e115954..b46344f 100644
--- a/generator/default.p9k
+++ b/generator/default.p9k
@@ -395,8 +395,8 @@ ${__P9K_ICONS[MULTILINE_LAST_PROMPT_PREFIX]}"
       # the RPROMPT, we advise it, to go one line up. At the end of RPROMPT, we
       # advise it to go one line down. See:
       # http://superuser.com/questions/357107/zsh-right-justify-in-ps1
-      RPROMPT_PREFIX='%{'$'\e[1A''%}' # one line up
-      RPROMPT_SUFFIX='%{'$'\e[1B''%}' # one line down
+      RPROMPT_PREFIX='%{'$'\e[1A''%}%{'$'\e[1A''%}' # two line up
+      RPROMPT_SUFFIX='%{'$'\e[1B''%}%{'$'\e[1B''%}' # two line down
     else
       RPROMPT_PREFIX=''
       RPROMPT_SUFFIX=''

@Syphdias first of all, thank you for the effort!
Your patch indeed allows newline segment to work on RPROMPT, but that does not solve the aesthetic issue I was talking about.

While by putting newline on RPROMT brings it on the same level with left, the cursor is under the time instead of next to it like it is in my previous screenshots.
I can live with having the RPROMPT at the same level with my time and cursor if it is not possible to have a segment in multiline, but it will be really great fi multiline could also support segments in order to have this:
P9K_PROMPT_ON_NEWLINE=true
P9K_RPROMPT_ON_NEWLINE=false
P9K_MULTILINE_FIRST_PROMPT_PREFIX_ICON=""
P9K_MULTILINE_LAST_PROMPT_PREFIX_ICON=(time)

P9K_LEFT_PROMPT_ELEMENTS=(os_icon root_indicator dir dir_writable vcs)
P9K_RIGHT_PROMPT_ELEMENTS=(status command_execution_time background_jobs node_version php_version date newline battery)

instead of this:
P9K_PROMPT_ON_NEWLINE=false
P9K_RPROMPT_ON_NEWLINE=false
P9K_MULTILINE_FIRST_PROMPT_PREFIX_ICON=""
P9K_MULTILINE_LAST_PROMPT_PREFIX_ICON=""

P9K_LEFT_PROMPT_ELEMENTS=(os_icon root_indicator dir dir_writable vcs newline time)
P9K_RIGHT_PROMPT_ELEMENTS=(status command_execution_time background_jobs node_version php_version date newline battery)

Did you take a look at the PR I referenced? There would be no need to treat P9K_MULTILINE_LAST_PROMPT_PREFIX_ICON like P9K_LEFT_PROMPT_ELEMENTS.
For your individual problem:
I guess you could pull the prefix and suffix down to be also added if P9K_PROMPT_ON_NEWLINE is false. Play around a little for yourself ;)

@Syphdias based on  your example, I made the following change instead:
diff --git a/generator/default.p9k b/generator/default.p9k
index e115954..d94e723 100644
--- a/generator/default.p9k
+++ b/generator/default.p9k
@@ -407,6 +407,11 @@ ${__P9K_ICONS[MULTILINE_LAST_PROMPT_PREFIX]}"
     RPROMPT_SUFFIX=''
   fi

+  if [[ $P9K_LEFT_PROMPT_ELEMENTS == *"newline"* ]]; then
+    RPROMPT_PREFIX='%{'$'\e[1A''%}' # one line up
+    RPROMPT_SUFFIX='%{'$'\e[1B''%}' # one line down
+  fi
+
   if [[ "${P9K_DISABLE_RPROMPT:-}" != true ]]; then
     __p9k_unsafe_RPROMPT="${RPROMPT_PREFIX}%f%b%k$(__p9k_build_right_prompt)%{${reset_color}%}${RPROMPT_SUFFIX}"
     RPROMPT='$__p9k_unsafe_RPROMPT'

my config:
P9K_PROMPT_ON_NEWLINE=false
P9K_RPROMPT_ON_NEWLINE=false
P9K_MULTILINE_FIRST_PROMPT_PREFIX_ICON=""
P9K_MULTILINE_LAST_PROMPT_PREFIX_ICON=""

P9K_LEFT_PROMPT_ELEMENTS=(os_icon root_indicator dir dir_writable vcs newline time)
P9K_RIGHT_PROMPT_ELEMENTS=(status command_execution_time background_jobs node_version php_version date battery)

So basically I check if newline is present on left prompt, and adjust the right prompt acordingly, that worked perfectly :)
Thank you very much for the idea, now my prompt looks great again!

I did a small change, in case someone has more than one newline segment in their left prompt, I count the newline segments and adjust the lines accordingly.
diff --git a/generator/default.p9k b/generator/default.p9k
index e115954..6b85d99 100644
--- a/generator/default.p9k
+++ b/generator/default.p9k
@@ -407,6 +407,21 @@ ${__P9K_ICONS[MULTILINE_LAST_PROMPT_PREFIX]}"
     RPROMPT_SUFFIX=''
   fi

+  # Check if `newline` segment is present in Left Prompt
+  if [[ $P9K_LEFT_PROMPT_ELEMENTS == *"newline"* ]]; then
+    # Get how many times it present
+    TIMES=$(echo $P9K_LEFT_PROMPT_ELEMENTS | tr " " "\n" | grep -c "newline")
+    # Reset RPROMT Prefix and Suffix in case they where modified previously
+    RPROMPT_PREFIX=''
+    RPROMPT_SUFFIX=''
+    # Loop based on `newline` instances
+    for INDEX in $(seq 1 $TIMES)
+    do
+      RPROMPT_PREFIX="$RPROMPT_PREFIX"'%{'$'\e[1A''%}' # add line up
+      RPROMPT_SUFFIX="$RPROMPT_SUFFIX"'%{'$'\e[1B''%}' # add line down
+    done
+  fi
+
   if [[ "${P9K_DISABLE_RPROMPT:-}" != true ]]; then
     __p9k_unsafe_RPROMPT="${RPROMPT_PREFIX}%f%b%k$(__p9k_build_right_prompt)%{${reset_color}%}${RPROMPT_SUFFIX}"
     RPROMPT='$__p9k_unsafe_RPROMPT'

This is how my terminal looks now:

In case someone have more than one newline segments:
P9K_LEFT_PROMPT_ELEMENTS=(os_icon root_indicator dir dir_writable vcs newline newline newline time)
P9K_RIGHT_PROMPT_ELEMENTS=(status command_execution_time background_jobs node_version php_version date battery)


Root Blade view is not resolved relative to the architecture.root path

Describe the bug
Hybridly cannot locate the root Blade view when the architecture.root option is set to an alternative path.
Current behaviour
Hybridly always attempts to locate the root Blade view at resources/application/root.blade.php.
Expected behaviour
Hybridly should attempt to locate the root Blade view at {architecture.root}/application/root.blade.php
Reproduction
https://github.com/nhedger/hybridly-root-view-issue
Steps to reproduce

Setup a new Laravel application: laravel new hybridly-root-view-issue
Apply the preset to the new installation: pnpm dlx @preset/cli apply hybridly/preset
Publish Hybridly's configuration file: php artisan vendor:publish --tag hybridly-config
Change the architecture.root option from resources/ to frontend/
Move the contents of the resources/ folder to the frontend/ folder
Run php artisan serve and attempt to load the page.
Assert that an View [root] not found. error is shown.

System information
System:
    OS: macOS 13.3.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
    Memory: 2.67 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.13.0 - ~/Library/Caches/fnm_multishells/22511_1687367505328/bin/node
    Yarn: 1.22.19 - ~/Library/Caches/fnm_multishells/22511_1687367505328/bin/yarn
    npm: 8.19.3 - ~/Library/Caches/fnm_multishells/22511_1687367505328/bin/npm
  Browsers:
    Chrome: 114.0.5735.133
    Edge: 114.0.1823.51
    Safari: 16.4
  npmPackages:
    hybridly: ^0.4.0 => 0.4.0

Used package manager
pnpm
Logs
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Hybridly issue and not an issue related to something else (Vite, Vue...). For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A, join our Discord server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Should be fixed in https://github.com/hybridly/hybridly/commit/b07f13c7786a47da26def40fd56ec139d5864933, can you try using https://github.com/hybridly/hybridly/tree/0.x locally and tell me if that works for you?

Yep, that fixed the issue. Thanks for looking into it.


Mac binaries

Mac prebuilt binaries when

Good afternoon! Your request is currently being worked on. 😄
To build MacOS binaries, I have to set up GitHub Actions on this repository so I can automate the creation of the binaries for all three major OSes (up to this point I was manually building the releases, but since I do not own a Mac I am not able to build it for Mac).
I attempted to create an Action for this yesterday, but seems like I found an issue in the release step action that prevents it from finding any built binaries. I've opened an Issue in the official repository, so we have to wait for them to explain to me what is happening. After this issue is fixed, I'll be copying the newly created Action pipeline to this repository and rebuilding the latest release to include the MacOS binary.
This is the repository I'm using to create and test the build automation if you are curious.

The issue was solved, hurray! If you have a MacOS, could you please this test this binary and let me know if it works?
ZipSource.x64.MacOS.zip

I'll go and get my MacBook soon

Works on my machine

Glad to hear that! Since everything worked out just, I've opened a PR creating a new deployment pipeline that will build a binary for all these OSes:

Windows x32
Windows x64
Linux x64
MacOS x64

I also atempted to have it build for MacOS and Linux ARM based systems too, but since I am no expert in Rust in general I wasn't able to make the build for ARM work while in an x64 system (which is what GitHub Actions use). If someone opens a new issue asking for this I might consider investing more time into that, but for now, I think that what was done is enough.
Thanks for opening an Issue and helping improve the project!


Refactor Toastr

Toastr logic shouldnt be present on redux. A new react component is needed so that the states pass through.


How to undo Notifications.registerRemoteNotifications() ?

I do not see any API to undo Notifications.registerRemoteNotifications(). How to do that?

Once we've registered once, seems that await Notifications.isRegisteredForRemoteNotifications(); will always return true then...

I'm really surprised no one else has needed this at some point 🤔...

I've just skipped this package because of this.

I also need that. I though Notifications.ios.abandonPermissions();  would to the job for iOS, but it does not seem too.
It is quite annoying, when developping an app, because one would certainly test the registration procedure before having set the entire flow to send the token to the backend and deal with it.
In my case I just wanted to see what this token looked like, and especially its type :/
I am trying actually to clean everything and build again the project, I hope it will unregister 😅


add cluster control plane health status

Signed-off-by: x893675<EMAIL_ADDRESS>
What type of PR is this?

/kind optimize
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:

/milestone v1.3.1

/cc @lixd @zhuzhenfan


Gas limit when using multicall

Describe the bug
multicall sometimes exceeds eth_call gas limit and an out of gas error is raised.
To Reproduce
Do eth_call consuming the maximum gas allowed by the node. Library raises an exception. Example trace from the transaction-service:
2022-05-16 10:23:21,794 [WARNING] [MainProcess] https://safe-transaction.mainnet.gnosis.io/api/v1/safes/{address}/collectibles/ - Exception: NodeConnectionException - Data received {}
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/gnosis/eth/multicall.py", line 227, in _try_aggregate
    result = self.contract.functions.tryAggregate(
  File "/usr/local/lib/python3.10/site-packages/web3/contract.py", line 957, in call
    return call_contract_function(
  File "/usr/local/lib/python3.10/site-packages/web3/contract.py", line 1501, in call_contract_function
    return_data = web3.eth.call(
  File "/usr/local/lib/python3.10/site-packages/web3/module.py", line 57, in caller
    result = w3.manager.request_blocking(method_str,
  File "/usr/local/lib/python3.10/site-packages/web3/manager.py", line 198, in request_blocking
    return self.formatted_response(response,
  File "/usr/local/lib/python3.10/site-packages/web3/manager.py", line 171, in formatted_response
    raise ValueError(response["error"])
ValueError: {'code': -32015, 'message': 'VM execution error.', 'data': 'Out of gas'}

Expected behavior
Reverts to use batch call

@Uxio0 has a look at github.com/banteg/multicall.py if you want to fix it

Sorry @007vasy , what exactly do we have to take a look at?

you can set the maximum gas and also multicall.py does a backtrack and tries to do the multicall with less amount of calls

Yeah, I mean, our biggest problem here is how to do it without impacting performance. Fallbacking to use less amount of calls after a failure is something we want to prevent

my best guess is that some contracts are faulty and the functions are tagged incorrectly (a view function which is not tagged as view) therefore the node think it needs gas, but it does not (but that's just a guess)

Well, it's not just gas. For example, Erigon has an evm timeout of 5 seconds and gas does not matter too much. Gladly, in the new Erigon versions it can be configured


Make duck-typing checking in serialize_body method more restrictive

Thebody parameter in DynamicClient's most methods currently accepts both dict type and ResourceInstance type. This was achieved by calling serialize_body() before mading any requests.
But in current implementation, the duck-typing style checking in serialize_body was kind of meaning-less---ResourceInstance type has overwritten it's __getattr__ method, so basically it's instances can pass every hasattr(...) check.
>>> inst = ResourceInstance(None, {'kind': 'Pod'})
>>> hasattr(inst, 'to_dict')
True
>>> hasattr(inst, 'to_dict_whatever_name')
True

This PR makes the checking in serialize_body() method more restrictive and also add some related unit tests.

/assign @yliaog

/lgtm
/assign @fabianvf

/approve

/lgtm, needs rebase though

Rebased with upstream/master.

MR refreshed.

please squash the commits

please squash the commits

Done

/lgtm
/approve

@piglei just needs the release note and it should be good to go

@piglei just needs the release note and it should be good to go

Thanks for reminding me, the release note was now added.


Explanation of hysteresis

Hello Denis,
I was wondering if you could give an explanation of hysteresis in the context of a thermostat/heater arrangement, please?
I'm an audio engineer so I understand the concept of hysteresis in magnetic tape, but I'm not sure about it regarding a thermostat, hence I'm not sure how I should set it (or determine the best value for it from some tests).
Any help would be greatly appreciated,
Dax.
P.S. Sorry for posting this as an 'issue', I couldn't find any other way to contact you. All the best.

Actually, just after posting my question to you, I thought about Googling for it. This is a good reference, perhaps you might like to add that link to your project documentation?
https://www.omron-ap.com/service_support/FAQ/FAQ00549/index.asp
Cheers!
Dax.


NaN on arm server

Issue Description
Please describe our issue, along with:

expected behavior
encountered behavior
NaN occors in https://gist.github.com/liweigu/bfcbd2a6e612e02ad7b24fee3cfac235 for epoch=500.
It's ok for epoch=50.
The detail is at https://community.konduit.ai/t/nan-on-arm-server/320 .

Version Information
Please indicate relevant versions, including, if relevant:

Deeplearning4j version
1.0.0-SNAPSHOT
Platform information (OS, etc)
arm on Huawei
CUDA version, if used
NVIDIA driver version, if in use

Additional Information
Where applicable, please also provide:

Full log or exception stack trace (ideally in a Gist: gist.github.com)
pom.xml file or similar (also in a Gist)

Contributing
If you'd like to help us fix the issue by contributing some code, but would
like guidance or help in doing so, please mention it!

We've checked libnd4j and nd4j tests on ARM. All tests pass, now we're going to check dl4j tests there.
But so far i suspect there's some kind of underflow/overflow due to hardware specifics.

As expected, we see NaNs in couple of DL4J tests on ARM. However, it's just a few tests, and by the looks of it - it's really numeric stability issue. We'll keep looking into it.

so far it did not occur on arm32 builds. will test arm64 when I switch to it.

@liweigu
could you test it again on your arm server with the latest source codes?
I want to know if the issue is still relevant.
thanks

I shall try it out and reply recently.

Do i have to compile dl4j from source? Or, can i just use the SNAPSHOT by maven?

it would be better to use from the source. you can cross-compile to build it fast

After libnd4j and nd4j compiled, i got error on compiling datavec:
[ERROR] Failed to execute goal on project datavec-parent: Could not resolve dependencies for project org.datavec:datavec-parent:pom:1.0.0-SNAPSHOT: The following artifacts could not be resolved: com.jakewharton.byteunits:byteunits:jar:${byteunits.version}, net.ericaro:neoitertools:jar:${neoitertools.version}: Failure to find com.jakewharton.byteunits:byteunits:jar:${byteunits.version} in https://repo.maven.apache.org/maven2 was cached in the local repository
I'm compiling without openblas which might not be necessary i think.
The compiling command is:
export LIBND4J_HOME=/root/deeplearning4j/libnd4j
cd /root/deeplearning4j/datavec
mvn -Djavacpp.platform=linux-arm64 clean install -Dmaven.test.skip=true

How to solve it?

It sounds like you make be using an old version of Maven.
Make sure you're running the latest version:
https://maven.apache.org/download.cgi

I get the same error with apache-maven-3.6.3.

It's there though: https://github.com/KonduitAI/deeplearning4j/blob/master/pom.xml#L263

Right, it's in the deeplearning4j/pom.xml. Then where is com.jakewharton.byteunits in the pom?
The full log:
[INFO] Scanning for projects...
[INFO] Inspecting build with total of 19 modules...
[INFO] Not installing Nexus Staging features:
[INFO]  * Preexisting staging related goal bindings found in 19 modules.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] DataVec                                                            [pom]
[INFO] datavec-api                                                        [jar]
[INFO] datavec-data                                                       [pom]
[INFO] datavec-data-audio                                                 [jar]
[INFO] datavec-data-image                                                 [jar]
[INFO] datavec-data-codec                                                 [jar]
[INFO] datavec-arrow                                                      [jar]
[INFO] datavec-geo                                                        [jar]
[INFO] datavec-python                                                     [jar]
[INFO] datavec-local                                                      [jar]
[INFO] datavec-data-nlp                                                   [jar]
[INFO] datavec-hadoop                                                     [jar]
[INFO] datavec-spark_2.11                                                 [jar]
[INFO] datavec-spark-inference-parent                                     [pom]
[INFO] datavec-spark-inference-model                                      [jar]
[INFO] datavec-spark-inference-server                                     [jar]
[INFO] datavec-spark-inference-client                                     [jar]
[INFO] datavec-jdbc                                                       [jar]
[INFO] datavec-excel                                                      [jar]
[INFO]
[INFO] ---------------------< org.datavec:datavec-parent >---------------------
[INFO] Building DataVec 1.0.0-SNAPSHOT                                   [1/19]
[INFO] --------------------------------[ pom ]---------------------------------
[WARNING] The POM for com.jakewharton.byteunits:byteunits:jar:${byteunits.version} is missing, no dependency information available
[WARNING] The POM for net.ericaro:neoitertools:jar:${neoitertools.version} is missing, no dependency information available
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for DataVec 1.0.0-SNAPSHOT:
[INFO]
[INFO] DataVec ............................................ FAILURE [  0.334 s]
[INFO] datavec-api ........................................ SKIPPED
[INFO] datavec-data ....................................... SKIPPED
[INFO] datavec-data-audio ................................. SKIPPED
[INFO] datavec-data-image ................................. SKIPPED
[INFO] datavec-data-codec ................................. SKIPPED
[INFO] datavec-arrow ...................................... SKIPPED
[INFO] datavec-geo ........................................ SKIPPED
[INFO] datavec-python ..................................... SKIPPED
[INFO] datavec-local ...................................... SKIPPED
[INFO] datavec-data-nlp ................................... SKIPPED
[INFO] datavec-hadoop ..................................... SKIPPED
[INFO] datavec-spark_2.11 ................................. SKIPPED
[INFO] datavec-spark-inference-parent ..................... SKIPPED
[INFO] datavec-spark-inference-model ...................... SKIPPED
[INFO] datavec-spark-inference-server ..................... SKIPPED
[INFO] datavec-spark-inference-client ..................... SKIPPED
[INFO] datavec-jdbc ....................................... SKIPPED
[INFO] datavec-excel ...................................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  2.133 s
[INFO] Finished at: 2020-12-25T15:46:22+08:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project datavec-parent: Could not resolve dependencies for project org.datavec:datavec-parent:pom:1.0.0-SNAPSHOT: The following artifacts could not be resolved: com.jakewharton.byteunits:byteunits:jar:${byteunits.version}, net.ericaro:neoitertools:jar:${neoitertools.version}: Failure to find com.jakewharton.byteunits:byteunits:jar:${byteunits.version} in https://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

You need to build from the root of the repository, not in datavec.

Let me try it.

I tried the latest arm64 build on raspberry pi4 (raspios(debian buster) lite arm64).
it worked as well.

It worked well for me too.
By the way, a suggestion about compiling is to avoid accessing to googleapis durning maven building, because it was blocked on my side at < org.deeplearning4j:dl4j-spark_2.11 > with error:
Dec 25, 2020 8:46:19 PM org.apache.http.impl.execchain.RetryExec execute
INFO: I/O exception (java.net.SocketException) caught when processing request to {s}->https://maven-central.storage-download.googleapis.com:443: Connection reset
Dec 25, 2020 8:46:19 PM org.apache.http.impl.execchain.RetryExec execute
INFO: Retrying request to {s}->https://maven-central.storage-download.googleapis.com:443

(which should be ok on your side.)
I think this issue has been fixed. :-)

great.
I was told we will have a release. so I'm closing this issue


Make helm --create-namespace Parameter Optional

The Helm flag --create-namespace currently does not check if the namespace exists but instead attempts to create it. This action requires cluster permissions for the namespace creation API (reference).
In my case, I only want to grant permissions on github action to the namespace itself, not for the cluster-wide namespace creation API. Could you please make the --create-namespace parameter optional?

HI @stolendog! I'm looking into this now and will get back to you asap. Thanks for contributing!

Looks like the issue is here:
https://github.com/bitovi/github-actions-deploy-eks-helm/blob/016d790e0f780d468a99a4ded0d10636d94daad5/deploy.sh#L126-L132
I'll create a conditional on that parameter. @LeoDiazL

Hi @stolendog, i've got the change on a branch in PR #55. I haven't had time to test it out yet; if you want, you can test it by using the branch in your workflow:
name: Deploy Helm
uses: bitovi/github-actions-deploy-eks-helm@54-namespace-create-option

Thanks, I'm already using it. It works perfect.

Thanks, I'm already using it. It works perfect.

That's fantastic, thanks for letting us know! I've merged the PR into main as v1.2.10. I haven't deleted the branch yet so as to not disrupt your work.
Thank you for your contribution and use of our Action!

thanks, you can delete the branch


Google API Authentication stopped working today 2024-09-18

Important gapi.auth2 notice: gapi.auth2 has been deprecated and replaced with Google Identity Services. Please see https://developers.google.com/identity/sign-in/web/deprecation-and-sunset for more information. If you have questions related to authentication/authorization please look at the associated documentation or post questions on Stack Overflow with the google-oauth tag.
Summary
Google API Authentication stopped working today 2024-09-18
Browser(s)/Version(s)
Version 129.0.6668.59 (Official Build) (64-bit)
Expected Behavior
Should authenticate with Google sign-in popup and load the page
Actual Behavior
The Authentication popup appears, I perform sign-in, the popup closes, but the page is blank.
The website was working for years, including yesterday.
This morning it stopped working.
In the console, I see multiple error messages like:
Cross-Origin-Opener-Policy policy would block the window.closed call.
cb=gapi.loaded_0?le=scs:425
POST https://play.google.com/log?format=json&hasfast=true&authuser=0 net::ERR_BLOCKED_BY_CLIENT

Steps to Reproduce
Open a website, authenticate in the sign-in popup window.

It is likely this failure is due to iframe permissions-policy or Content Security Policy changes, please read Google Sign-in with FedCM APIs and let me know if you're able to resolve the issue.

For some reason it began working again after a couple of days.

Good. During the transition period that the client library is currently in, some percentage of requests will use the new sign-in flow and some will use the existing flow. So there may be differences between sign-in attempts, even if nothing changes with your code and app. If you do see failures again I'd recommend testing with use_fedcm:true  to confirm that it is not responsible.


fix: remove chart param from list function.

Couldn´t  get video by ID. Cause chart was defined. I removed this parameter.
https://developers.google.com/youtube/v3/docs/videos/list#filter-parameters
Greetings

Great! Thank you

New version 0.2.5 with that fix is available in npm from now.
npm install youtube-video-api --force


columns collapse

created a new story inside Columns - columnsGrouping where you can test

@rita-moreira the collapse and expand is working fine, but it's not possible to write inside the Name2 or City2 columns, if we do it it goes to the next column
(if I write on Name2 it goes to City, if I write in City2 it goes to Country)

@rita-moreira the collapse and expand is working fine, but it's not possible to write inside the Name2 or City2 columns, if we do it it goes to the next column
(if I write on Name2 it goes to City, if I write in City2 it goes to Country)

right!! I didn't test that sorry, I will check

@rita-moreira the collapse and expand is working fine, but it's not possible to write inside the Name2 or City2 columns, if we do it it goes to the next column
(if I write on Name2 it goes to City, if I write in City2 it goes to Country)

right!! I didn't test that sorry, I will check

notice now in Row Grouping example, that we can't change also the nested rows

@rita-moreira I think it works fine now!

@rita-moreira I think it works fine now!

nice!!


Add example of linkerd-await usage in a multi-stage build.

Also emphasizes the adding of execute access and linkerd-await being outside the PATH.
Signed-off-by: Naseem<EMAIL_ADDRESS>

My pleasure!


CI Memory Issues

Currently, CI only supports builds that are under 2GBs. This is an issue when building bigger configs such as BOOM or Hwacha. Is there a way to go around this properly rather than trying to create a smaller config (in Hwacha's case this is hard since a minified Rocket core with Hwacha still needs more that 2GB to verilate)?

Will be addressed by eventually adding nightly/weekly FPGA builds of current items. So CircleCI will still be a "small" set of regressions.


docs(protocol): fix typos

@lissstern actually, although this is fixing a typo, the changelog is correct. the commit message for this change is "fix an occasional error in test:tokenomics", so we should probably keep the changelog as is.


Switch to docker compose v2

closes https://github.com/inveniosoftware/invenio-cli/issues/348

:heart: Thank you for your contribution!
Description
As described in https://github.com/inveniosoftware/invenio-cli/issues/348 invenio-cli internally uses the deprecated version 1x of docker-compose to orchestrate containers. This forces users to install the old version alongside the newer version of docker compose. This pull request changes the underlying subcommands to use the new syntax.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[X] I'm aware of the code of conduct.
[X] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[X] I've NOT included third-party code (copy/pasted source code or new dependencies).

Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

Hi @fenekku I can't associate it with the "PR Community" project

Hi, I updated my fork with your latest changes. Is there anything blocking the merging that I could help with? Tbh I somehow lost track.

Everyone approved, tests pass and it's been out very long: I am merging. Thank you @karkraeg for all the effort, we appreciate it!


 These are just a few of the slurs used to describe this pull request.
And yet, here we are.
PROMPT: I was trying to track down where drag gets its threshold from and noticed window and document were getting fresh JQuery wrappers with every use. It made me curious more than anything so I began digging in to see what, if any, performance gains there were to referencing a single, global instance of these instead of rewrapping them every time.
CHANGES: I set const declarations at the top and used them to replace all instances of $(window) and $(document).
PAIN-POINT: JQuery does a great job of detecting if window is the object getting wrapped and, as far as I can determine, not rewrapping it. I can't tell if it's doing this for document or not. Either way, it has to do logic to make that determination. Logic that we can just do from our end to help it along.
Additionally, the performance gains that I could find seem more relevant to looped instances than seldom used one-offs, so really this comes down to a matter of style/opinion.
My enthusiasm isn't nearly as high as my curiosity was, so I'm not pushing this issue too hard. If I hadn't noticed that work was being done on a completely fresh version, I probably wouldn't be PR'ing at all, but this seems like a good time to make this update if it needs to get made at all.
I made this test with version 1.6.0: https://codepen.io/nominalaeon/project/editor/Dnnxzj/#0
I've also implemented the change in a microsite that is doing some heavy lifting slider stuff using angular-slick and nested sliders. Here are some screenshots:

In summation, gentlemen, I propose you accept this change and embrace it in your hearts. As true believers of JQuery it's our job, nay our Bob-given duty, to use it responsibly and set good examples for the children.
Or not, whatever.

Sidenote, if anyone can ELI5 drag's threshold, I would be forever indebted.

Hey thanks for putting in the effort on this. Not opposed to putting window/document in variables, but I think we're still trying to support es5, so gonna have to swap out the const for var.
I think touch threshold refers to the number of pixels required before a slide change occurs. See here
Thanks

Thank you for the direction on touch threshold, that is exactly what I was hunting for.
I used const to play along with the const $ = window.$ || window.jQuery; declaration above it. I haven't had a chance to poke around too much, but the Webpack config looks like it's using Babel, so the es6 stuff should get distilled into compatible biz in the end. Happy to change it if that's not the case.
Thanks for looking this over, legg!

Ohhhh, this is the v2 branch, cool in that case you're fine with const.

Nice, thanks! Updating version 1 with this seemed more risk than reward


Move RDS Module to its own Repo

Per this ADR, move the RDS module to its own repo to be tested / versioned independently of other modules
AC:

[ ] There is a product supported repo and we have updated our example to point to it
[ ] refactor as necessary

Same comment as in #171

Repo created: https://github.com/defenseunicorns/terraform-aws-uds-rds

Working on this now. Should have a PR out later today.


Warning messages if endpoint url is invalid

Hello,
I am working under Mac OS X and with python 3.7.1 in a conda environment. Using invalid endpoint argument (iot.us-east-1.amazonaws.com) results in strange error messages.

First case: I suggest checking the endpoint argument for the correct format first (abcd123456wxyz-ats.iot.us-east-1.amazonaws.com).
Second case: Also, I suggest throwing a warning if people do not use the ats endpoint (abcd123456wxyz.iot.eu-central-1.amazonaws.com).

The first case produces the following output currently. It seems like message->allocator is not set:
''' Connecting to iot.eu-central-1.amazonaws.com with client ID 'DemoDevice'...
Fatal error condition occurred in /Users/Shared/Jenkins/Home/workspace/aws-crt-python-build-wheels-osx/aws-crt-python/aws-common-runtime/aws-c-common/source/allocator.c:176: allocator != ((void*)0)
Exiting Application
'''
...
'''
################################################################################
Raw stacktrace:
################################################################################
1   _awscrt.cpython-37m-darwin.so       0x000000010281e5e3 s_print_stack_trace + 19
2   libsystem_platform.dylib            0x00007fff685ca5fd _sigtramp + 29
3   ???                                 0x0000000000000000 0x0 + 0
4   libsystem_c.dylib                   0x00007fff684a0808 abort + 120
5   _awscrt.cpython-37m-darwin.so       0x00000001028221db aws_fatal_assert + 75
6   _awscrt.cpython-37m-darwin.so       0x0000000102821074 aws_mem_release + 52
7   _awscrt.cpython-37m-darwin.so       0x0000000102840910 s_do_read + 336
8   _awscrt.cpython-37m-darwin.so       0x00000001028402dc s_on_readable_notification + 92
9   _awscrt.cpython-37m-darwin.so       0x000000010283edde s_on_socket_io_event + 350
10  _awscrt.cpython-37m-darwin.so       0x000000010282eecc s_event_thread_main + 956
11  _awscrt.cpython-37m-darwin.so       0x000000010282bf28 thread_fn + 88
12  libsystem_pthread.dylib             0x00007fff685d6109 _pthread_start + 148
13  libsystem_pthread.dylib             0x00007fff685d1b8b thread_start + 15
'''
The second case produces:
'''
Connecting to akkh9376u0e33.iot.eu-central-1.amazonaws.com with client ID 'DemoDevice'...
Traceback (most recent call last):
File "src/backgroundworker/sample.py", line 132, in
connect_future.result()
File "/Users/tobiasjacob/anaconda3/lib/python3.7/concurrent/futures/_base.py", line 432, in result
return self.__get_result()
File "/Users/tobiasjacob/anaconda3/lib/python3.7/concurrent/futures/_base.py", line 384, in __get_result
raise self._exception
awscrt.exceptions.AwsCrtError: AwsCrtError(name='AWS_IO_TLS_ERROR_NEGOTIATION_FAILURE', message='TLS (SSL) negotiation failed', code=1029)
'''

Aws-c-io is intended as a general async networking framework, so validating endpoint names against a narrow pattern isn't something that would really fit here.  Even at the iot SDK level its still a bit suspect since it would lead to backwards compatibility issues should the endpoint pattern change in the future.


Add new property not in the end

Discussed in https://github.com/CarlosNZ/json-edit-react/discussions/131

Originally posted by Set27 December 13, 2024
Is it possible?
I don't see ways how to implement it.

Will consider adding as an option.

Hi @Set27, you can now do this in v1.18.0
Use the insertAtTop prop, as detailed here

Hi @Set27, you can now do this in v1.18.0
Use the insertAtTop prop, as detailed here

Thank you for your work.
But do you consider option, to have custom field where I can choose between what elements I can insert new property?

But do you consider option, to have custom field where I can choose between what elements I can insert new property?

Can you give me an example?

But do you consider option, to have custom field where I can choose between what elements I can insert new property?

Can you give me an example?

Let's imagine I have an array of 5 elements.
I want to add a new element in between the third and fourth elements

Let's imagine I have an array of 5 elements.
I want to add a new element in between the third and fourth elements

Are you suggesting having that in the UI for the user to select? You can always use drag-n-drop to move it to where you want.
Or are you wanting to configure it in the code?


Failed to translate TritonGPU to LLVM IR

Hello,
Trying to implement a projected outer-product I ran into this crash:
loc("/tmp/ipykernel_1194073/3068465724.py":42:8): error:  size mismatch when packing elements for LLVM struct expected 8 but got 4
loc("/tmp/ipykernel_1194073/3068465724.py":42:8): error: invalid element type in packLLEElements. Expected 'i32' but got 'f32'
loc("/tmp/ipykernel_1194073/3068465724.py":42:8): error: invalid element type in packLLEElements. Expected 'i32' but got 'f32'
loc("/tmp/ipykernel_1194073/3068465724.py":42:8): error: invalid element type in packLLEElements. Expected 'i32' but got 'f32'
loc("/tmp/ipykernel_1194073/3068465724.py":42:8): error: invalid element type in packLLEElements. Expected 'i32' but got 'f32'
loc("/tmp/ipykernel_1194073/3068465724.py":42:8): error: 'llvm.insertvalue' op Type mismatch: cannot insert 'f32' into '!llvm.struct<(i32, i32, i32, i32, i32, i32, i32, i32)>'
Pass execution failedLLVM ERROR: Failed to translate TritonGPU to LLVM IR.

This is not a python Exception and completely kills the underlying process.
The code (simplified as much as I could)
Kernel:
@triton.jit
def load_input_vector(vector_ptr, vector_len,
                      block_id,
                      WEIGHT_BS:tl.constexpr,
                      BATCH_BS: tl.constexpr
                     ):

        vector_indices = tl.arange(0, WEIGHT_BS) + WEIGHT_BS * block_id
        batch_indices = tl.arange(0, BATCH_BS)
        final_indices = batch_indices[:, None] * vector_len + vector_indices[None, :]
        loaded_vector = tl.load(vector_ptr + final_indices)
        return loaded_vector, vector_indices

@triton.jit
def projected_outer_product(vec_a_ptr, projection_direction_ptr,
                    vec_length, n_projection_dims,
                    result_ptr,
                    BATCH_BS:tl.constexpr,
                    PROJ_BS:tl.constexpr,
                    WEIGHTS_BS: tl.constexpr):

    pid_i = tl.program_id(0)
    pid_j = tl.program_id(1)

    (vec_a, vec_a_indices
    ) = load_input_vector(vec_a_ptr, vec_length,
                          pid_i, WEIGHTS_BS, BATCH_BS)

    proj_indices = tl.ravel(
        vec_a_indices[:, None] * vec_length
        + vec_a_indices[None, :]
    )
    proj_dim_indices = tl.arange(0, PROJ_BS)
    projection = tl.load(projection_direction_ptr
                         + proj_dim_indices[None, :]
                         + proj_indices[:, None])

    # Computing the local outer product
    outer_product = tl.view(
        vec_a[:, :, None] * vec_a[:, None, :],
        (BATCH_BS, WEIGHTS_BS * WEIGHTS_BS)
    )

    # Computing the projection
    result = tl.dot(outer_product, projection)

    batch_indices = tl.arange(0, BATCH_BS)
    proj_result_indices = tl.arange(0, PROJ_BS)
    tl.store(result_ptr
             + batch_indices[:, None] * n_projection_dims
             + proj_dim_indices
             , result)

Caller
batch_size = 32
length = 32
n_projection_dims = 32
device = ch.device('cuda:0')

vec_a = ch.randn(batch_size, length, dtype=ch.float32, device=device)
projections_directions = ch.randn(n_projection_dims, length * length,
                                  dtype=ch.float32, device=device)

BATCH_BS = 32
PROJ_BS = 32
WEIGHTS_BS = 4

grid_size = (
    triton.cdiv(length, WEIGHTS_BS),
    triton.cdiv(length, WEIGHTS_BS),
)

result = ch.zeros(batch_size, n_projection_dims, dtype=ch.float32, device=device)

projected_outer_product[grid_size](vec_a, projections_directions,
                           length, n_projection_dims,
                           result,
                           BATCH_BS, PROJ_BS, WEIGHTS_BS)

I know that after simplification the code itself isn't mathematically correct but I would like to figure out why the whole compiler crashes the process (and maybe you might want to improve the error message, though It seems that the error is past your type checker so looks like a bug there not in my code)
Thanks!

Thanks for reporting. One general comment about reporting such bug, please share either a self contained repro or an IR dump (you can dump the IR by setting this env var MLIR_ENABLE_DUMP=1). This way there is no overhead at all reproducing.
If developers needs to put together a repo it makes it harder to look at the bug.

what hw are you running on? It passes for me on H100

Here is the dump.
log.log
From my understanding the bug comes from the .view().
I'm running locally on a 3090Ti. But I can quickly check on (V/A/H)100 and report

looks like your code is quite old from the IR. Can you try building from main top of tree? Most likely the bug has already been fixes

Will do! thanks for the help, appreciate it

Just to try, I tested on the latest pip version and this time I get another error you might find interesting (but I will compile from source now to check if that's fixed there)
getElemsPerThread is not supported for shared layout
UNREACHABLE executed at /project/lib/Dialect/TritonGPU/IR/Dialect.cpp:807!

Aborted (core dumped)

log.log

Fixed on 3.0. Thanks!


Replacement for Scrapy Contracts

@rennerocha recently raised the issue of Scrapy Contracts often making the test suite fail due to false positives.
The initial idea of using Contracts was to have a quick (for experienced developers) and beginner-friendly way of writing Scrapy tests, exempting developers from having to write test cases, creating fixtures and maintaining them up-to-date.
We could also keep using Contracts, knowing they will eventually fail in online tests, using them only as a way of monitoring the state of all crawlers (today we have 13 different spiders) until we have a better and more robust one.

I think that tests that depend on external resources (an external URL, the DOM of an external webpage etc.) is always troublesome when we try to make it part of our CI. The suite will eventually fail because of something nothing related to our code. However, this is gold IMHO:

The initial idea of using Contracts was to have a quick (for experienced developers) and beginner-friendly way of writing Scrapy tests, exempting developers from having to write test cases, creating fixtures and maintaining them up-to-date.

I think we could be less pragmatic about Scrapy Contracts: we know they will fail. We could use them not as part of the CI, but as part of an sort of integration monitor outside the CI.
This will avoid raising red flags in PRs (that might not be friendly for beginners), and make the PR/review/merge process more seamless. But we keep it running elsewhere and it will let us know when some Spider needs to be updated, or even tell us stories about cities publishing gazettes in months that are way past behind…

I have some discussions at the company on better ways to test spiders. One thing that we mostly agreed is that it is better to monitor the spider execution than try to unit test its code, as the pages changes and the content is not always reliable. We even have a tool (unfortunately not open-sourced yet) that we can include monitoring (checking the existence of errors, the structure of the data returned, etc). It is not ideal so if someone else have better ideas it will be great to hear!
One thing that could be tested are utilitarian code (like pipelines, processors, simple functions) that we use and it is not related to the content of the pages we are scraping. We could think about custom contracts (https://doc.scrapy.org/en/latest/topics/contracts.html#custom-contracts) to create better ways to validate if the spider is working well, but I have never used it.
But I agree with @cuducos that it'd be better if contracts failures didn't break the build, so we won't have a lot of red flags in the PRs.

Just changed the CI to run only unit tests, leaving the contracts for when running in the local environments.
Once we have automation on the data collection tasks, a monitoring system may be configured and the contracts removed.


Test code against memory leaks.

After all decoder features are in place looped test on IPs doesn't show any increasing memory consumption.


Snippet for Angular2 pipes.

Added snippet for simple ng2 pipes.

:+1:

The pipe should implements PipeTransform

@Gillespie59  Thx for the Hint - I don't know why it was missing in the API docs for custom pipes.

i just reported to the docs.
the interface should be here.


Fix broken link to old how-to url

🚨 IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.
I Assume the above doesn't really apply for a minor documentation update

Thanks @madorb !


Forced upgrade to django-staff-sso-client 3.1.1

This is because the way that a new version of pip resolves packages is more strict and doesn't allow wheel only packages anymore. The client team have issued a fix which requires a point version upgrade.
See also: https://ditdigitalteam.slack.com/archives/C1Q2EKZK3/p1633009272197500

Checklist

Requires migrations? No
Requires dependency updates? Yes

Codecov Report

Merging #390 (a0a54f6) into master (5ff5089) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #390      +/-   ##
==========================================
+ Coverage   91.23%   91.25%   +0.02%
==========================================
  Files         274      274
  Lines       16263    16263
  Branches     1317     1317
==========================================
+ Hits        14837    14841       +4
+ Misses       1192     1189       -3
+ Partials      234      233       -1

Impacted Files
Coverage Δ

common/forms.py
67.88% <0.00%> (+1.83%)
:arrow_up:

regulations/forms.py
65.65% <0.00%> (+2.02%)
:arrow_up:

Continue to review full report at Codecov.

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


Basic mouse pointers support

This PR covers basic mouse pointer support such as buttons and multi pressed states and scrolling

Thanks!


[WFCORE-2293] Add sensitivity classifications for attributes referencing Elytron capabilities.

:bangbang: Commits from this branch are used in numerous other topic branches so please do not cherry-pick or rebase. :bangbang:
Resolves https://issues.jboss.org/browse/WFCORE-2293

Retest this please


YAML incorrect highlighting

Hello,
I've been experiencing a problem with YAML files. Some files are not correctly highlighted:

Thanks.

Version: 1.39.0-insider (user setup)
Commit: c8252b63f547674d2faf361d4181bb76b0b5f81a
Date: 2019-10-04T09:24:45.690Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Windows_NT x64 10.0.18995

Can you please provide a copy/pastable example?
Does this reproduce with all extension disabled?

Yes, I disabled all extensions and the problem persists.
apiVersion: v1
kind: ConfigMap
metadata:
  name: varnish-vcl
  namespace: kickante-website-development
  labels:
    app: varnish
    release: kickante
data:
  vcl_synth_500.vcl: |
    // Custom error
    set resp.http.Cache-Control = "no-cache, max-age: 0, must-revalidate";
    set resp.http.x-sitename = "Kickante";
    set resp.http.x-sitetitle = "Kickante";
    synthetic( {"
      <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />

      <title>"} + resp.http.x-sitetitle + {" - Something went wrong</title>

            🤔

Something went wrong, but it's not your fault.

We are working on it, please check back shortly.

    "} );
    return (deliver);

Yes, I disabled all extensions and the problem persists.
apiVersion: v1
kind: ConfigMap
metadata:
  name: varnish-vcl
  namespace: test
  labels:
    app: varnish
    release: kickante
data:
  vcl_synth_500.vcl: |
    // Custom error
    set resp.http.Cache-Control = "no-cache, max-age: 0, must-revalidate";
    set resp.http.x-sitename = "test";
    set resp.http.x-sitetitle = "test";
    synthetic( {"
      <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />

      <title>"} + resp.http.x-sitetitle + {" - Something went wrong</title>

            🤔

Something went wrong, but it's not your fault.

We are working on it, please check back shortly.

    "} );
    return (deliver);

Yes, I disabled all extensions and the problem persists.
apiVersion: v1
kind: ConfigMap
metadata:
  name: varnish-vcl
  namespace: test
  labels:
    app: varnish
    release: test
data:
  vcl_synth_500.vcl: |
    // Custom error
    set resp.http.Cache-Control = "no-cache, max-age: 0, must-revalidate";
    set resp.http.x-sitename = "test";
    set resp.http.x-sitetitle = "test";
    synthetic( {"
      <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />

      <title>"} + resp.http.x-sitetitle + {" - Something went wrong</title>

            🤔

Something went wrong, but it's not your fault.

We are working on it, please check back shortly.

    "} );
    return (deliver);

VSCode gets its YAML grammar from Textmate:
https://github.com/microsoft/vscode/blob/master/extensions/yaml/syntaxes/yaml.tmLanguage.json
On the Textmate repo, they've discussed the same problem and determined that it's a problem with the parser and not the grammar:
https://github.com/textmate/yaml.tmbundle/issues/17#issuecomment-418602300
Hopefully this will help with tracking down the root of the issue.

Thanks for the context @cthrasher! Based on the discussion in textmate/yaml.tmbundle#17 (comment), this is not something that's solvable with a Textmate grammar without adding an exception to the single line parsing (and single line parsing is fundamental to textmate). The real solution is https://github.com/microsoft/vscode/issues/50140 or https://github.com/microsoft/vscode/issues/77140


CVCalendarView in TableView Header in width not stretching

Hello @MakeHui,
Please check out  develop  and issued #101 #104 #97. It should help!
Regards,
Eugene


add missing include

Just noticed this header didn't compile on its own.

thanks @wjwwood, I forward ported this fix to melodic-devel as well


username_block_signal send multiple times

When blocking a user a signal is send in defender/utils.py:block_username
https://github.com/kencochrane/django-defender/blob/master/defender/utils.py#L197
However, this happens on every call of defender/utils.py:record_failed_attemp in
https://github.com/kencochrane/django-defender/blob/master/defender/utils.py#L218
even though the user was already blocked before.
For instance, I want to send an email informing the user (if present) that she was blocked and this results into an email for each blocked login.
Is this on purpose?

@horida good question, I would suspect that we only need it the first time, but I'm not sure if there is a use case where someone wants to know every time.
@williamboman wrote that functionality, so let's ask them. @williamboman do you see any issue if we only send the blocking signal when it is blocked and not after every invalid login?
Maybe we need to have a different signal for invalid login that happens every time, and then only trigger the blocked username/ip when it is blocked.
Open to ideas/suggestions.

Ah that does seem like a flaw. That signal should indeed only be dispatched when a user conceptually becomes blocked. I would not oppose changing this behavior, although I no longer use this library.

Thanks both! I can create a PR these days.
I like that currently the signal is only sent once a user is actually blocked. I would not change this behavior.
On the other hand, we could check if the user is already blocked before blocking it. Then, afterwards the signal is only sent if the user was not blocked before. Though, I'd leave the set blocked in both cases to avoid race conditions.
Something like
already_blocked = is_user_already_locked(username)
key = get_username_blocked_cache_key(username)
if config.COOLOFF_TIME:
    REDIS_SERVER.set(key, 'blocked', config.COOLOFF_TIME)
else:
    REDIS_SERVER.set(key, 'blocked')
if not already_blocked:
    send_username_block_signal(username)

here you go: https://github.com/kencochrane/django-defender/pull/137

completed with #137 thanks @horida and @williamboman


Use I18n and add Japanese locale.

I made changes based on the work made by @joelmeyerhamme ( https://github.com/joelmeyerhamme/ice_cube/tree/international ).
https://github.com/seejohnrun/ice_cube/issues/68

What is the status of this? Will it be possible to use I18n?

Hey @emilkarl, this PR is working. I'm using that code in a side project, though it's not battle-tested.
I'm not sure why this PR is not getting attention.

Cool @dgilperez.

You can refer to a particular branch or commit in your Gemfile like this:
gem 'ice_cube', github: 'dgilperez/ice_cube', branch: 'pull-259'
# or
gem 'ice_cube', github: 'dgilperez/ice_cube', ref: '4926bf972f7c24c280e652dbac79348e5b9b9744'

Also, peeking at the network graph from my fork, I see the nice guys at gocardless have continued work on that PR, adding some locales and a few other goodies. It may a better replacement, what do you think @isaacseymour? Is your fork production ready?

Awesome. I'll take a look at that too then!

Yep, we've been using the gocardless fork in production, it's not hugely
different. Would be great to be back on the official RubyGems version
though!
On Thu, 1 Oct 2015 20:18 Emil Karlsson<EMAIL_ADDRESS>wrote:

Awesome. I'll take a look at that too then!
—
Reply to this email directly or view it on GitHub
https://github.com/seejohnrun/ice_cube/pull/259#issuecomment-144821479.

Thanks @isaacseymour !!! Giving it a try ...

@tomoyuki28jp Sorry, I have been away from this project for some time. I18n was added as an optional dependency in the meantime, and I would be happy to merge your contribution for Japanese translation.
If you still find it useful, please see the latest master branch and add your jp.yml on it. We can reopen this PR or make a new one.


Change ‘Resources’ to ‘Topics’ in hamburger menu

 (Not Launch Critical) CPG Next: Change "Resources" to "Topics"


support override operands images with annotation.

ref: https://github.com/stolostron/backlog/issues/24609
Signed-off-by: morvencao<EMAIL_ADDRESS>

/lgtm


Add C# version to the grunt build (and CI)

Currently the build does the following:

Runs static code analysis on the JS code
Runs the JS tests
Transpiles to Java and runs tests
Transpiles to C and runs tests
Packages / bundles all of the above.

The CI runs all of the above too.
The C# code is very much absent from this process!

Update: Thanks to #151 the C# code is now compiled and tested - the only thing missing from CI is the bundling, which in this case is the creation of a DLL.

Build process has changed so closing this. C# version will be integrated into buck build process as well.

Cool, I did notice that some big changes were on their way.
Can I just check, will the new build still wanted regular releases to npm? My project, and a number of others, depend on this!

If the community wants native binaries through npm then that is definitely something we will look into continuing to do. The JS version has been removed though. The reason behind removing the JS version was that it was not used internally and the web supports flexbox very well nowadays. css-layout targets native platforms so it makes more sense that the source of truth be C.

Actually it isn't the native binaries that I was wanting to have distributed through npm, it's the JavaScript code. While the web supports flex box, there are still a number of interesting / novel uses for this JavaScript implementation.
For example, we use it within d3fc to bring flex box to SVG, as described in this blog post. There are a few other projects that use this code too:
https://libraries.io/npm/css-layout/dependents
I presume the new implementation is C rather than JS? I can understand the reasoning for this.
I'd just suggest not withdrawing css-layout from npm (although I guess that as I was the one who published it, I'd be the one that would have to remove it!)

Yeah I don't see a great reason to remove what is already on npm :) It
would be interesting to look into using emscripten to generate a js version
from the c code for use in these kinds of projects. Nothing I am
prioritizing but I would definitely be interested in seeing a pull request
for that if you would find that useful :)
On Thu, 11 Aug 2016 at 22:29, Colin Eberhardt<EMAIL_ADDRESS>wrote:

Actually it isn't the native binaries that I was wanting to have
distributed through npm, it's the JavaScript code. While the web supports
flex box, there are still a number of interesting / novel uses for this
JavaScript implementation.
For example, we use it within d3fc http://d3fc.io to bring flex box to
SVG, as described in this blog post
http://blog.scottlogic.com/2015/02/02/svg-layout-flexbox.html. There
are a few other projects that use this code too:
https://libraries.io/npm/css-layout/dependents
I presume the new implementation is C rather than JS? I can understand the
reasoning for this.
I'd just suggest not withdrawing css-layout from npm (although I guess
that as I was the one who published it, I'd be the one that would have to
remove it!)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/facebook/css-layout/issues/142#issuecomment-239298818,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdIpCHfm1TKMkzIHqlUzZCtrX0l4Yvvks5qe5RRgaJpZM4GK24I
.


Add a FileSpecificDiagnosticConsumer

If a top-level diagnostic is in a particular source range, the FileSpecificDiagnosticConsumer will funnel it and any attached notes to a particular "sub-consumer" designated for that range (intended to be used with whole files). If it's not in a range designated for any sub-consumer, the diagnostic is passed to all registered sub-consumers.
This is intended to be used for batch mode, so that diagnostics that are definitely associated with a particular file can be emitted in that file's .dia output, while diagnostics that may be associated with other files (such as those that come from Clang) will still get presented to the user.

@swift-ci Please smoke test

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - a546ae3a58fe98414e5d2a5c4a13f13d4b4da709

Build failed
Swift Test OS X Platform
Git Sha - a546ae3a58fe98414e5d2a5c4a13f13d4b4da709

@davidungar: Still waiting for another round of review, or you could jump to #14900 and see how it's used (and the unfortunate but unavoidable switch to basing on file names).

(We moved to just doing it all at once in #14900. Closing.)


Optionally run mypyc on projects

This PR

allows Projects to run a custom command to verify if mypyc breaks on the project.
turns this feature on for Black

For Black this takes around 6 minutes on my laptop. Definitely worth gating off for typeshed. I can add a command line flag if that helps

A command line flag would be great!

I've addressed your comments in the new commit, going to add a command line flag now. Thanks for the blazing fast reviews :)

Ping; it would be great to have this for schema-salad and cwltool given the mypyc regressions we experienced in the last two releases

Hello from 2024, can someone take this over?


pca and data

Tested and it works

Looks good! Thanks for making the PCA changes we talked about.

Good update to text,


Issues/#196 workbench xsl fix

This PR addresses GitHub issue: #
Briefly describe the changes proposed in this PR:

...
...
...

Make sure you've followed the Contributor Guidelines. In particular (please tick to indicate you've taken care of it):

[x] RDF4J code formatting has been applied
[ ] tests are included
[x] all tests succeed

IP validation still fails, you will probably have to redo your changes from scratch (I don't think you can sign off on a commit after the fact). Btw if you are using a git client like Sourcetree: it has an option to make it automatically sign off every commit you do, per repository. Might make things easier :)
Apart from that: you based your issue branch on the wrong source branch: looks like you branched off from master (which is why this PR still includes changes by me). Instead, branch off from the release branch. See the developer guidelines for details.

Yeah, i got myself in a right mess with this gah.

No worries. I'm on irc if you wanna talk about it :)


Fix trpc route path

api route path for prod deploy

@lucavallin  Can you try once more trigger deploy ? Thanks

Yep, this works now, thanks! I merged it and the changes are live.
The API for preview deployments however is still broken (it always goes to verto.sh). Do you mind checking that to see what we need to do to fix it? Thanks!

Yep, this works now, thanks! I merged it and the changes are live.
The API for preview deployments however is still broken (it always goes to verto.sh). Do you mind checking that to see what we need to do to fix it? Thanks!
A few bugs I found:

It seems we are not updating the page title anymore, which might be bad for SEO.
Filters are not reset when going back to the homepage

I am going to revert the PR. It works, but we need to fix these issues first :)

Thanka. ill try fix main issue,
Another two bug is easy to fix


fix: lint failing on develop

Lint is failing on develop and causing other PR's to fail the validation checks

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (25a27b9) 90.54% compared to head (5d8488d) 90.54%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1327   +/-   ##
========================================
  Coverage    90.54%   90.54%
========================================
  Files          191      191
  Lines         5966     5966
========================================
  Hits          5402     5402
  Misses         564      564

Flag
Coverage Δ

py3.8-ubuntu-22.04-pandas
90.54% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/ydata_profiling/report/structure/report.py
92.03% <100.00%> (ø)

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


updated typescript config

typescript files can now be compiled using tsc

It took me a while to see that this adds a define into the file, I try to change that as well.