CI changes

Retry conformance test run with detailed logs at : https://github.com/googleapis/google-cloud-dotnet/actions/runs/4410137035/jobs/7727278176

Open for LEX team review

LGTM

Looks good - please rebase to one commit, and ideally rename the PR to have a conventional commit name as well.
Is it ok if I keep both the commits in this one. The previous one has logs set that might come in handy to quickly debug if we come back to the skipped test cases later or get the storage test bench working without the workarounds. Can also compare it the existing logs in the past run of this PR which is kept for record. I will rename the PR to have conventional commits.

Hmm. I'm in two minds about this - if we want logging to be available, I'd prefer it to be available with cleaner code, and ideally in a way that lets us enable it without having to go back to an old commit. On the other hand, the cost of keeping the commit is pretty low.
On the balance, I'm okay with keeping it - but if we find we actually use it (even once) I'd like us to put a little more time (half an hour would be enough I suspect) to add logging in a less ad-hoc manner.

Hmm. I'm in two minds about this - if we want logging to be available, I'd prefer it to be available with cleaner code, and ideally in a way that lets us enable it without having to go back to an old commit. On the other hand, the cost of keeping the commit is pretty low.
On the balance, I'm okay with keeping it - but if we find we actually use it (even once) I'd like us to put a little more time (half an hour would be enough I suspect) to add logging in a less ad-hoc manner.

Thanks @jskeet


Block Button sizes?

Do you guys plan to add block level button .btn-block (spans the entire width of the parent element)?

This class is already available as bttn-block here.


extract deleted curl arguments

This makes extract_curl_args.py also extract the arguments for each tagged release in curl's git repo's history (going back to 2001) and compare them over time, printing out which ones have disappeared and when.
Unlike extracting curl's current args, figuring out why (and if) certain arguments are deleted and whether or not to add them to the parser needs to be done by hand.

If we wanted to be more correct, we would extract an args object for each curl release (possibly we would have to modify the actual parsing code for certain releases, not just the object with the args), then on the client, run through all of them in newest-to-oldest release order, generating output for each release. If any of the generated code differs we would report that by showing buttons above the input box that would let the user select a curl version. So for example if curl 6.5 to 7.15 can't parse the input at all, parsing with curl 7.15 - 7.19 arguments generates the same code, and parsing with the arguments from curl 7.19 - 7.77 (the current release) generates a slightly different output, we would show the 7.77 result in the output but also put the buttons [7.19] and [latest] over the input box which would switch between the two possible outputs. We could even display some sort of timeline of curl version and color 6.5-7.14 in red to show that 7.14 and below doesn't parse the code at all too.


Add setting to change the increase/ decrease of stock.

Right now your app is showing the percentage change in the stock. Give a preference to change it to numerical value.

added in #17


Allow single Detector sign

Is there a way to have the detector sign work with a single one, or are two always required?
I would like to have a single one to detect, if a train is above it, to use it in combination with a station sign.

Use an unpowered switcher sign for a single detector sign.


Console print problem with INFO log

Hi,
I have a little problem when I attempt to print INFO log on the console. i try to make INFO log like this:

During a function in the test library, in call my function to log a message:

I expected this type of result:

But unfortunately I have two problems:
-The name of the test case is automatically written before the first INFO log message.
-I get a point at the beginning of every log message.
Here is my result:

I have no problem with the log file generated automatically at the end of the process. Just with the console print!
any ideas? Thanks in advance for your assistance!

The dots are generated by  monitor markers option.
See the commandline help:
-K --monitormarkers auto|on|off  Show . (success) or F (failure) on
console when top level keywords in test cases end.
Values have same semantics as with --monitorcolors.
In general if you dont want any printouts from robot on the console, just use --console quiet.
This isnt really a bug report. These sort of issues should be discussed in the users group. If you have further questions about logging and console, please ask them in the google users group.

I agree with everything Jussi wrote. Notice that --console option is new in RF 2.9, though.

Ok, it's noted! Sorry for my mistake.
Thanks for your answer.


Release package version 1.4.0

Include the Sendable conformances (https://github.com/apple/swift-system/pull/115)
Include Mach.Port (https://github.com/apple/swift-system/pull/129)
Bump up toolchain requirement to 5.6, which supports concurrency (https://github.com/apple/swift-system/pull/141)

Clean up any conditional compilation branches that are no longer needed.
Move the Sendable conformances to the base type declarations.

https://github.com/apple/swift-system/releases/tag/1.4.0


Use v2 as default for Docker Compose version

Resolves https://github.com/devcontainers/features/issues/620

Switches to use Docker Compose v2, v1 is deprecated
Installs binary from GitHub instead of using Pip (source)
Bump to v3

@samruddhikhandale I wasn't sure if there was an automated process for bumping versions, updating the README, so I did it manually

Regarding tests:

PR - Test Updated Features / test (docker-in-docker, debian:12)

I can see bookworm in https://github.com/microsoft/vscode-dev-containers/blob/main/script-library/shared/settings.env#L57

Is there any reason why this was not merged?
it seems that docker-outside-of-docker already defaults to compose V2.
https://github.com/devcontainers/features/blob/0cf690ee14279d9e7ae5e1308950d6067d3aba81/src/docker-outside-of-docker/devcontainer-feature.json#L23-L32


Segmentation fault (core dumped)

Require from composer.json:
        "php": ">=7.4.0",
        "barryvdh/laravel-cors": "^0.11.0",
        "botman/botman": "^2.5",
        "botman/driver-facebook": "^1.10",
        "brozot/laravel-fcm": "^1.2",
        "certifiedwebninja/caroline": "1.0.1",
        "doctrine/dbal": "^2.6",
        "egulias/email-validator": "^2.1",
        "firebase/php-jwt": "^5.0",
        "google/apiclient": "^2.0",
        "guzzlehttp/guzzle": "6.3.3",
        "illuminate/mail": "^6.0",
        "illuminate/redis": "^6.0",
        "intervention/image": "^2.4",
        "laravel/helpers": "^1.1",
        "laravel/lumen-framework": "^6.2.0",
        "league/flysystem": "^1.0",
        "nesbot/carbon": "^2.0",
        "phpseclib/phpseclib": "^2.0",
        "predis/predis": "^1.1",
        "sentry/sentry": "2.2.5",
        "sentry/sentry-laravel": "1.2.0",
        "snipe/banbuilder": "^2.3",
        "vlucas/phpdotenv": "^3.3",
        "webpatser/laravel-uuid": "^3.0",
        "zytzagoo/smtp-validate-email": "dev-master"

When I tried throw Exception I catch: Segmentation fault(core dump)
For example on "sentry/sentry": "^2.0" and "sentry/sentry-laravel": "1.2.0" I catch these errors:
Deprecated: Function ReflectionType::__toString() is deprecated in /var/www/html/vendor/sentry/sentry/src/Serializer/AbstractSerializer.php on line 331

Deprecated: Function ReflectionType::__toString() is deprecated in /var/www/html/vendor/sentry/sentry/src/Serializer/AbstractSerializer.php on line 331

Deprecated: Function ReflectionType::__toString() is deprecated in /var/www/html/vendor/sentry/sentry/src/Serializer/AbstractSerializer.php on line 331

Deprecated: Function ReflectionType::__toString() is deprecated in /var/www/html/vendor/sentry/sentry/src/Serializer/AbstractSerializer.php on line 331

Deprecated: Function ReflectionType::__toString() is deprecated in /var/www/html/vendor/sentry/sentry/src/Serializer/AbstractSerializer.php on line 331
Segmentation fault (core dumped)

For example on "sentry/sentry": "2.2.5" and "sentry/sentry-laravel": "1.5.0" I catch only:
Segmentation fault (core dumped)
I don't know what happened maybe somebody caught this and resolved?
Upd:
While It's only in scheme migration:
class AddIndexToUsers extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::table('users', function (Blueprint $table) {
            $table->index('id');
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::table('users', function (Blueprint $table) {
            $table->dropIndex([
                'id',
                'wront_value'
            ]);
        });
    }
}

Here I included wrong index and after it I caught Segmentation fault
Upd:
PHP Version: 7.4

Hmm, could you also share the exact version of PHP you are using when the segmentation fault occurred?

@stayallive 7.4

I am experiencing the same issue.

@acrossoffwest are you able to consistently trigger this using the php artisan migrate:rollback command?
When trying your composer.json I am unable to trigger this on OS X (with your composer requirements and just a clean Lumen with Sentry configured):
PHP 7.4.0 (cli) (built: Nov 29 2019 16:18:44) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
    with Zend OPcache v7.4.0, Copyright (c), by Zend Technologies

@sidvanvliet are you also on Lumen and PHP 7.4? Could you share your composer.json.
Could you both tell me a bit more about the machine you are running into this like the OS & output from php -v?

@stayallive I am using using Laravel Framework 6.4 with PHP 7.4.0 on a MacBook Pro running version 10.15 (Catalina)
My composer.json looks like this:
{
    "name": "laravel/laravel",
    "type": "project",
    "description": "The Laravel Framework.",
    "keywords": [
        "framework",
        "laravel"
    ],
    "license": "MIT",
    "require": {
        "php": "^7.4",
        "ext-json": "*",
        "facade/ignition": "^1.4",
        "fideloper/proxy": "^4.0",
        "guzzlehttp/guzzle": "^6.3",
        "intervention/image": "^2.5",
        "laravel/framework": "6.4.*",
        "laravel/horizon": "^3.4",
        "laravel/slack-notification-channel": "^2.0",
        "laravel/telescope": "^2.1",
        "laravel/tinker": "^1.0",
        "owen-it/laravel-auditing": "9.3.*",
        "predis/predis": "^1.1",
        "sentry/sentry-laravel": "1.3.*",
        "spatie/laravel-translatable": "^4.1"
    },
    "require-dev": {
        "barryvdh/laravel-debugbar": "^3.2",
        "beyondcode/laravel-dump-server": "^1.0",
        "filp/whoops": "^2.0",
        "fzaninotto/faker": "^1.4",
        "mockery/mockery": "^1.0",
        "nunomaduro/collision": "^3.0",
        "phpunit/phpunit": "^7.5"
    },
    "config": {
        "optimize-autoloader": true,
        "preferred-install": "dist",
        "sort-packages": true
    },
    "extra": {
        "laravel": {
            "dont-discover": []
        }
    },
    "autoload": {
        "files": [
            "app/Helpers/GeneralHelper.php"
        ],
        "psr-4": {
            "App\\": "app/"
        },
        "classmap": [
            "database/seeds",
            "database/factories"
        ]
    },
    "autoload-dev": {
        "psr-4": {
            "Tests\\": "tests/"
        }
    },
    "minimum-stability": "dev",
    "prefer-stable": true,
    "scripts": {
        "post-autoload-dump": [
            "Illuminate\\Foundation\\ComposerScripts::postAutoloadDump",
            "@php artisan package:discover --ansi"
        ],
        "post-root-package-install": [
            "@php -r \"file_exists('.env') || copy('.env.example', '.env');\""
        ],
        "post-create-project-cmd": [
            "@php artisan key:generate --ansi"
        ]
    }
}

And my bootstrap/app.php looks like this (default):
<?php

/*
|--------------------------------------------------------------------------
| Create The Application
|--------------------------------------------------------------------------
|
| The first thing we will do is create a new Laravel application instance
| which serves as the "glue" for all the components of Laravel, and is
| the IoC container for the system binding all of the various parts.
|
*/

$app = new Illuminate\Foundation\Application(
    $_ENV['APP_BASE_PATH'] ?? dirname(__DIR__)
);

/*
|--------------------------------------------------------------------------
| Bind Important Interfaces
|--------------------------------------------------------------------------
|
| Next, we need to bind some important interfaces into the container so
| we will be able to resolve them when needed. The kernels serve the
| incoming requests to this application from both the web and CLI.
|
*/

$app->singleton(
    Illuminate\Contracts\Http\Kernel::class,
    App\Http\Kernel::class
);

$app->singleton(
    Illuminate\Contracts\Console\Kernel::class,
    App\Console\Kernel::class
);

$app->singleton(
    Illuminate\Contracts\Debug\ExceptionHandler::class,
    App\Exceptions\Handler::class
);

/*
|--------------------------------------------------------------------------
| Return The Application
|--------------------------------------------------------------------------
|
| This script returns the application instance. The instance is given to
| the calling script so we can separate the building of the instances
| from the actual running of the application and sending responses.
|
*/

return $app;

@sidvanvliet ah, I wondered why your bootstrap/app.php looked like that but as you said you are on the full framework, in that case it's not relevant but thanks for sharing :)
Can you also share what exactly you do or what happens in your app when the segfault occurs?

And one more moment I'm using Docker with image: php:7.4.0-fpm-buster

@stayallive It happened when a job failed.

Can any of you two share or extract a reliable reproduction out of where it fails for you because I'm unable to make it happen still.
My best guess is this is a 7.4 bug and is triggered by our error tracking or trying to access something from the shutdown function (wouldn't be the first time) but it's extremely tricky to find a reproduction so we can try and figure out if we can fix this.

I tried to repeat this issue on new lumen project and it worked fine.

If by any chance you run into it again please do let us know if you have more info so we can investigate further!


Stricter experience type error with utility functions

Bug report
Describe the bug
In short, there is a type error when trying to use the utility functions with a theme token.
To Reproduce
I have made a repo:
https://github.com/alii/stitches-bug-repro
Expected behavior
There is no type error
Screenshots

System information

OS: n/a, typescript issue
Browser (if applies) n/a
Version of Stitches: 1.2.5 (latest at time of writing)
Version of Node.js: 16.10.0

Additional context
For those in the Modulz discord server, there's a little more conversation here: https://discord.com/channels/752614004387610674/752795957674115092/897312708452552706

I'm facing the same issue. I had logged in https://github.com/stitchesjs/stitches/issues/1077


Use heroku.com domain for SSL-Doctor

herokuapp.com address has been deprecated.

@dickeyxxx can we please get this looked at ASAP? We are planning on doing the switchover within a couple weeks.
cc @heroku/secdev @da3da1us

this code shouldn't be in use anymore cc @ransombriggs

@rdsharma (quote from @ransombriggs) "that has been ported to v3 and fixed in v3, so unless someone has a setup that has not transitioned at all from the Ruby code, they should not hit this." Do we need to address this in the v2 code as well?
/cc @da3da1us

@garyposter if you look at this search you can see that there are a bunch of people who have imported this code:
https://github.com/search?utf8=✓&q=https%3A%2F%2Fssl-doctor.herokuapp.com&type=Code&ref=searchresults
I haven't actually looked through all the results to determine which ones are using the v2 code, but I can tell you that I can see requests still coming in against the herokuappp.com subdomain:
Dec 05 00:51:51 ssl-doctor heroku/router:  at=info method=POST path="/resolve-chain-and-key" host=ssl-doctor.herokuapp.com request_id=f08d6a55-dcf2-4d95-9e8a-b4819e63cbed fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=33ms status=200 bytes=5433
Dec 05 00:52:06 ssl-doctor heroku/router:  at=info method=POST path="/resolve-chain-and-key" host=ssl-doctor.herokuapp.com request_id=33f52d42-26ef-4c80-bd70-0b0004e4c871 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=32ms status=200 bytes=5433
Dec 05 01:35:49 ssl-doctor heroku/router:  at=info method=POST path="/resolve-chain-and-key" host=ssl-doctor.herokuapp.com request_id=ee27f8f5-efcb-40ff-9b88-1dd72f46ab22 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=73ms status=422 bytes=295
Dec 05 06:08:37 ssl-doctor heroku/router:  at=info method=POST path="/resolve-chain-and-key" host=ssl-doctor.herokuapp.com request_id=92289fe5-1de7-4fb7-8792-5635cc6865e1 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=48ms status=200 bytes=6011
Dec 05 08:07:56 ssl-doctor heroku/router:  at=info method=POST path="/resolve-chain-and-key" host=ssl-doctor.herokuapp.com request_id=94996e87-309c-4a57-b643-9268b75bc47b fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=48ms status=200 bytes=6031
Dec 05 09:57:29 ssl-doctor heroku/router:  at=info method=POST path="/resolve-chain-and-key" host=ssl-doctor.herokuapp.com request_id=214cfc53-a748-4467-8e96-a93b6cbd0c10 fwd="<IP_ADDRESS>" dyno=web.1 connect=1ms service=184ms status=200 bytes=5387
Dec 05 09:57:38 ssl-doctor heroku/router:  at=info method=POST path="/resolve-chain-and-key" host=ssl-doctor.herokuapp.com request_id=317980f5-02e6-4116-a42e-03308de500bf fwd="<IP_ADDRESS>" dyno=web.1 connect=2ms service=194ms status=200 bytes=5387

While of course updating the v2 code won't explicitly fix this, I feel like it's easier for support to tell our customers that they can just update their Heroku dependencies and their code will start working again, rather than support telling them they have to switch to v3 entirely? The fix is only a one line string constant change, but I can understand why you would rather avoid updating an abandoned legacy library. Please let me know what you think.

Thanks for the data @rdsharma.
On the one hand, I think we're cool with merging and releasing. I'm verifying with @ransombriggs and @dickeyxxx that we can just go ahead and do this.
On the other hand, IMO we actually really don't want Support encouraging people to stick with v2. Keeping them on the old codebase will cause more support problems down the road, when we could remind them to migrate now.
The reason I'm cool with merging and releasing is because I want customers who encounter troubles migrating to be able to have a way to stick with the old code in an emergency. In that case, Support should emphasize that the affected customers are using old, unsupported code, and they should make an effort to migrate ASAP.
Do you buy that? If so, will you be the one advising support on this, or should we reach out?
I believe Ransom and Jeff are out for the day, and I'm out now, so this may not wrap up till tomorrow.

@garyposter sounds good to me. I was planning on talking to support soon to give them a heads up of the sunset of the herokuapp.com subdomain anyways, I'll make sure I mention the v2 vs v3 part as well. Thanks!

do you guys mean CLI v3/ CLI v5? or do you mean the api version? If you mean the CLI, this won't likely fix anything since those requests are most likely coming from users that haven't updated their CLIs at all, so shipping a new version won't result in anything changing for them. Definitely fine with releasing, but I don't expect anything to change.

@dickeyxxx yes, we are talking CLI, but this is about support.
AIUI, Security is going to kill the old URL within a week or two. That will stop the problem!
But Security wants Support to be able to have a good answer to give customers who write in to complain about any problems. Our strongly preferred answer is that they upgrade the CLI to the Node version. If they can't for some reason, perhaps because they are rarely but importantly using an old local Ruby plugin, then Security wants Support to have a short-term, unsupported option of using this old code, while they migrate.
Thanks to @ransombriggs who has now merged and is making a release.

@rdsharma like @dickeyxxx this may not actually fix those issues, but went ahead and released because it would at least put them on an easy path to a fix when it is fully deprecated.

Thanks @ransombriggs @garyposter @dickeyxxx !


fedimint-cli spend should have a flag to give exact change or error if it can't

Currently fedimint-cli spend <amount> will give us <amount> if it has those notes on hand, and give more than we ask for if it doesn't.
I was just writing a test and this behavior caused my test to be flaky. It would be nice for this reason (and probably in general) to be able to request exact amounts from fedimint-cli spend.

That should be the default behavior.


Distributed training

Thanks for open sourcing the code for this awesome paper!
I’m wondering if you used distributed training of the different GAN models during experimentation. If so, could you share an example of how to launch a distributed training job using compare_gan code?

@jppgks comparing multiple experiments in parallel is nothing like distributed training, unless hyper-parameter optimization is the end goal. Is this what you mean by multiple tasks?

Note: We have updated the framework in the meantime and it now supports distributed training (single run on multiple machines) for TPUs.

@Marvin182 where can I find this in the code?

https://github.com/google/compare_gan


Can't build /w v0.2 (can't find distDir) or v0.3.1 (can't find artifacts) (Ubuntu)

v0.2 of this action returns Error Unable to find your web assets, did you forget to build your web app? Your distDir is set to ""./ui"".: https://github.com/TheDiscordian/disco-chat/actions/runs/3857576476/jobs/6575122299
However, if I run "pwd" and "ls" before this action, it's in the correct directory (and it builds locally).
I also tried v0.3.1, but that can't find artifacts: https://github.com/TheDiscordian/disco-chat/actions/runs/3857160737/jobs/6574236201
    Finished release [optimized] target(s) in 6m 33s
    Bundling disco-chat_0.1.0_amd64.deb (/home/runner/work/disco-chat/disco-chat/target/release/bundle/deb/disco-chat_0.1.0_amd64.deb)
    Bundling disco-chat_0.1.0_amd64.AppImage (/home/runner/work/disco-chat/disco-chat/target/release/bundle/appimage/disco-chat_0.1.0_amd64.AppImage)
    Finished 2 bundles at:
        /home/runner/work/disco-chat/disco-chat/target/release/bundle/deb/disco-chat_0.1.0_amd64.deb
        /home/runner/work/disco-chat/disco-chat/target/release/bundle/appimage/disco-chat_0.1.0_amd64.AppImage

Expected artifacts paths:
/home/runner/work/disco-chat/disco-chat/target/release/bundle/deb/disco_chat_0.1.0_amd64.deb
/home/runner/work/disco-chat/disco-chat/target/release/bundle/appimage/disco_chat_0.1.0_amd64.AppImage
/home/runner/work/disco-chat/disco-chat/target/release/bundle/appimage/disco_chat_0.1.0_amd64.AppImage.tar.gz
/home/runner/work/disco-chat/disco-chat/target/release/bundle/appimage/disco_chat_0.1.0_amd64.AppImage.tar.gz.sig

What version should I be using, and how?

Try using @dev. v0.3 can't handle - in app names on Linux

@FabianLars thank you so much! This is resolved


TypeScript

at some point down the line, this project should be moved to typescript because as the project develops it'll be a lot more solid if there was type checking
here's an article about how to make that move

this might not actually be a good idea,
1 it'll take forever to implement
2 if its purpose is to stop bugs, I'm the only one working on this repo Im able to find bugs quickly because i wrote everything
3 i need to code the basics and get it out there to see if anyone will use the app, not spend time perfecting it
if it works out, and there's more people working on this then sure it makes sense but for now no


Optimisations for knn search

Hi,
I've been using this KDTree implementation for a simple KNN classifier, and made a small optimisation to speed up the KNN search.
I noticed in KDNode._search_node(), you sort the nodes by distance, which you already calculate and store in the results dict. The first commit changes the sorting operation to use the already calculated values, rather than recalculating the point-point distance for each item.
I ran it with my data set for my classifier, which reads a file, performs 500 inserts and then 268 KNN searches, and got my total runtime down from 9.8s to 2.8s(!!!!!!)
The second commit avoids doing this sort altogether if no nodes have been added to the list, and uses the previous values of bestNode and bestDist, as the values will not have change. This optimisation is a lot smaller, but could be useful for some search edge cases. In my particular data set it shaved an extra .5s off my already reduced search time, bringing me down to 2.3s.
Hope this speeds up your implementation of whatever you use this for :)
Denbeigh

Thanks a lot for your contribution!

I've just released 0.12 which contains your changes.

Thanks mate! :)


in docker-compose set hostname and ports from .env settings

with these settings the user can run multiple dev environments and decouple the hostname from the service name inside the network.
I'm a newbie approaching your project and it seems great!
Thank you

I understand the logic of different ports for different projects, is it that common, big inconvenience to shut down one stack and bring up the other one? Can you give a bit more detail how this became an issue for you?
With the ports, at least one thing i see that breaks with this pr is this: I wanted the user to be able to swap the database by changing the .env settings (and comment the db section) and have the rest of the components connect to his db (wherever it is, probably on localhost). With the changes in this PR, it's assumed the database will always be at port 5432.
I've never used hostname. How is this setting helpful?
Thank you

Thanks now I understand the motivation behind the DB_PORT env.
You can easily pull off the  PR.
For the hostname, inside the network, each service have the hostname set by default as the service name...
looking around the code, i see that the hostnames are not used, so seems not necessary to decouple .env hostname from service name.
again you can trash the PR.


Allow for NA in processing days from index and histologic morphology code

Processing Days from Index should allow NA for specimens that are not yet processed
Histologic Morphology Code should allow NA
Split from #92

The Histologic Morphology Code fix is related to #38 which I reopened on the basis of #116

@elv-sb are you able to take a look at this one for this sprint?

Histologic morphology code should be ok already.
Processing days from index currently has regex match \d+$|Not\sapplicable$|unknown$ changed to match \d+$|Not\sapplicable$|unknown$|NA$ which allows for NA

This was a request from SRRS
Processing days from Index currently is controlled by a regex
regex match \d+$|Not\sapplicable$|unknown$

This should allow for any integer, or Not applicable or unknown
On reflection (and sorry this is so late in the process) I think we should ask they submit Not applicable rather than add a feature to also allow NA
For completeness I checked in BigQuery to see what non-integer values have been submitted to date (before this control was put in place)
SELECT Processing_Days_from_Index, COUNT(Processing_Days_from_Index)
FROM `htan-dcc.combined_assays.Biospecimen`
WHERE NOT REGEXP_CONTAINS(Processing_Days_from_Index, r'^\d+$')
GROUP BY Processing_Days_from_Index

Processing Days from Index
Count

Not available
162

unknown
1606

Not Recorded
279

Not recorded
9

htapp-783
1

Not reported
8

Unknown
14

Not Reported
34

Histologic morphology code
I concur with Elvira above: This currently has no valid values and no validation rules but is a required field.
I'm not sure why SRRS are requesting to allow for NA - its not clear that they specifically experienced error on submitting NA

@elv-sb and @aclayton555
My proposed actions emerging from this issue and review of #131. Please react with 👍 if you agree.

[ ] Close #131 without merging
[ ] Contact SRRS to ask to submit using Not applicable rather than NA - this is already in data model and in prod DCA for Processing days from index. Histologic morphology code remains uncontrolled
[ ] New issue to add RPPA Level 4 attribute description
[ ] Implement valid values for Morpholgy correctly per #23
[ ] New issue to implement valid values for Histologic Morphology code including unknown and Not applicable
[ ] Adam to review valid values as part of Metadata QC in progress

Agree with these actions. Action plan:

Elvira
Adam
Ashley
Ashley
Next sprint
Ashley
Adam already on this

Clarifications with SRRS to be done following hand back of SRRS data liaison actions to ISB. Issue here to be closed


Scala 2.13.3 removed -Xlint:nullary-override

Attempting to use this plugin in projects with scala 2.13.3 will result in this error message
[error] 'nullary-override' is not a valid choice for '-Xlint'

Seems like the plugin doesn't have a way to disable an option in a patch release of scala
    ScalacOption("-Xlint:nullary-override", addedIn = Some(11), removedIn = dotty),           // Warn when non-nullary `def f()' overrides nullary `def f'.

Any guidance on how this can be resolved?

Ah I see. I think the best thing to do would be to filter out this option for now:
scalaOptions ~= { opts => opts.filterNot(Set("-Xlint:nullary-override")) }

I will cut a new release shortly that removes this option for 2.13 as I expect most people will probably be able to use the most current point release without any issues.
I generally don't make efforts to support old point releases of Scala in this plugin as I imagine that would be a pretty involved software archaeology project.

Thanks for the clarification, I made #25 based on your suggestion. Please take a look


Zone.js causing rubberbanding when dragging map?

Gif of issue was too large so here it is on imgur.
https://imgur.com/a/dwzLgvX
[ x] I've looked at the documentation to make sure the behavior is documented and expected
I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[ x] I've searched through the issues to make sure it's not yet reported
Expected behavior
When dragging across the leaflet map, the map moves.
Current behavior
Dragging the map causes the map to return to the initial position.
Environment
Leaflet version: 1.7.1
Browser (with version): Chrome
OS/Platform (with version): Windows 10 1109
Additional context
This same issue has occurred before. If I remember correctly, back in leaflet pre v1.0.0 there used to be an animate boolean we could disable when creating the map. Only the Zoom Control and Pan Control have the animate option. This issue is caused by zone.js in angular as specified in the issue below.
https://github.com/Leaflet/Leaflet/issues/4588#issuecomment-221424763
In this library, it is mentioned that zone.js causes these issues due to the cancelation callback. And that this library runs outside of zone.js so my question is, looking at the gif above. Does this seem to be an issue caused by zone.js's callback cancelation?
https://github.com/xavidram/LeafletRubberband
You can clone that sample to reproduce the issue.

Figured out the issue. this was due to the use of mapbox-gl-leaft
The use of L.mapbox() to create the tile layer was causing zone.js to constantly try and update the dependency tree as it was doing everything within zone.js rather than outsize like the ngx-leaflet does.


Hardcode debris orbit update when moving the sun station or orbital probe cannon

Feature
Right now these things are moved as if they orbit the satellites but actually they share the same orbits. Just put them in the same orbit and change the true anomaly by some amount.
Context
No response

Nebula fixed this that one time


[FEATURE REQUEST] Adding Generic Constraints

hi, Thank you for this tool by the way its fantastic. I was wondering if you could the type safety features that most other iterations and improvements of unity events. I find it very useful to have betterEvent betterevent<T1,T2> etc. when designing my games with event systems to ensure the correct response is wired to the correct event given the generic constraints

I would like to comment on this a bit (although it's been there since 2019).
This would also allow for 'Dynamic' type calling which I don't know if it is exactly possible as is... I lately when prototyping try to get an 'aggregator' class which detects changes on data all over the place and dispatches events for wiring to: Text.text, Image.Color, Score, etc... it's often UI related things though, granted and mostly only for very dynamic things.
This is glued together via an UnityEvent?.Invoke(labelText); for example. Don't think this isn't possible just yet with BetterEvents.


Remove the git api calls in the devfile library to avoid being rate limited

Which area this feature is related to?

/area library
Which functionality do you think we should add?

Why is this needed? Is your feature request related to a problem?
Detailed description:
When running unit tests from HAS, there were many test failures that were caused by calls in this validateToken function which resulted in 403 errors when calling the API endpoints due to the rate limits being exceeded.  Since these are unauthenticated calls, the rate is pretty low.  I would rather avoid calling these endpoints given the rate limiting issues we've had in the past with HAS and allow the git clone to fail if we get a bad token instead.
Describe the solution you'd like

Remove these calls and rely on the presence of a set token to determine whether the repo is public or private.  It'll be up to the user or client to set this up correctly
Describe alternatives you've considered
We can allow authenticated calls but I don't want to run the risk of rate limiting an end users token

Additional context

Implementing this in issue #1216


SQLite: "too many terms in compound SELECT" when inserting lots of records at once

SQLite has a hard limit of 500 terms in a compound SELECT (see: https://stackoverflow.com/a/9527898). The limit can be configured by the user to be lower, but not greater.
Laravel doesn't take that into account, resulting in an "too many terms in compound SELECT" error when trying to insert more than 500 entries at once.
https://github.com/laravel/framework/blob/fe68aa9ee8e6356a95259cfe984a204940ed3b22/src/Illuminate/Database/Query/Grammars/SQLiteGrammar.php#L146-L187
A possible solution would be to split up big datasets into multiple queries (at most SQLITE_MAX_COMPOUND_SELECT) in SQLiteGrammar's compileInsert function. Another one would be to simply wrap many inserts into a single transaction regardless of the dataset size, which should perform similarly performance-wise.
The current limit on the database can be checked with sqlite3_limit(db, SQLITE_MAX_COMPOUND_SELECT, -1) function, as described in https://www.sqlite.org/c3ref/limit.html

Just a regular batch insert of ~700 entries. The exact context can be seen here: https://github.com/dos1/movim/commit/306de37a5bcc02440381128f872ccfe1dc8da81c - where Roster::insert comes from Illuminate\Database\Eloquent\Model. The linked commit successfully workarounds this issue - splitting it to SQLITE_MAX_COMPOUND_SELECT entries per batch would work too.
(btw. it's not my code, I know about it only as much as I debugged, which led me to filling this bug).

We could fix it by using the standard syntax: INSERT INTO "table" ("column") VALUES (?), (?)
SQLite has been supporting it since 3.7.11 (6.5 years ago).
Ideally, we would detect the SQLite version and use the legacy syntax if required. But the problem is that we don't have access to the database version inside the grammar.
Replacing the old syntax completly would be a breaking change. And you can bet that some hosting services still use ancient SQLite versions...
This is another use case for version-specific grammars (https://github.com/laravel/ideas/issues/870).

@staudenmeir That syntax you mention is implemented in SQLite as just a syntax sugar and falls under the same limit as the old one.

@laurencei Any info on why has it been closed? I just looked at the SQLiteGrammar.php file on master branch and I don't see any relevant update there.

@laurencei Oh, no, it was a reply to the previous comment referring to the alternative syntax to the one used now by Laravel (before 3.8.8 it was just a syntax sugar, so changing it in Laravel won't fix the issue for the older ones). So, if Laravel is fine with requiring 3.8.8+, the syntax can simply be changed, otherwise it should do a version check and construct multiple INSERT queries on older ones (or just ignore that and leave this bug for the users of older SQLite versions).
To sum up, this is a Laravel bug that happens on the newest SQLite. Any specific SQLite versions were only mentioned when discussing a way to fix it.

(if you're fine with requiring SQLite 3.8.8 [or 3.7.11 if you don't mind this bug still happening for some people], I should even manage to make a PR with the fix :P)

...it kind of feels like a real edge case, that if someone runs into it - the easiest/obvious solution is to update to SQLite 3.8.8?
Changing the SQL statement just to handle this single case might have a performance impact for other users not affected by it?

Uhm, no, please read the whole discussion again. This happens for every SQLite version right now, it's just a fix proposed in https://github.com/laravel/framework/issues/25262#issuecomment-414836191 that will require SQLite 3.8.8 to actually fix anything.

Ok... now I see.
Well - seems like the best option is submit a PR for this and see what Taylor thinks.

Submitted :)

Please close the issue.


Do not specialize dead apply/partial_apply.

What's in this pull request?

Fix a bug where we end up with a validation issue due to specializing a known dead partial_apply.
Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

Do not specialize an apply/partial_apply that we've already added to the
set of dead instructions. Doing so can result in creating a new
instruction which we will leave around, and which will have a type
mismatch in its parameter list.
Fixes rdar://problem/25447450.

@swift-ci Please smoke test


Sections of minified CSS ignored due to over-eager matching in KEYFRAMES regexp

I found that the polyfill stopped working when it was operating on minified css.
I tracked the problem down to line 287 where the comments, imports, keyframes etc are stripped from the input css, and the problem was that the regex for keyframes matched entire chunks of css between the @ and the word keyframes. An example string that it matched in my project was:
@font-face{font-family:FontAwesome;/*... (many, many other rules here) ...*/}@-webkit-keyframes fa-spin{0%{-webkit-transform:rotate(0);/* ... */}}

I fixed it by excluding the { character from the characters that may appear between @ and keyframes.

Thanks.
Having a quick look at http://caniuse.com/#feat=css-animation I think the regex
KEYFRAMES = new RegExp('((@(-webkit-)?keyframes [\\s\\S]*?){([\\s\\S]*?}\\s*?)})', 'gi'),

would be more suitable.


How to loop thru win_updates until no updates left

I've been struggling with this for quite some time now,
In my organization, we carry out windows updates (using ansible) every quarter year and it usually requires 2-3 playbook executions to finish it off.
There's no looping support in Blocks yet, Include is not a module, more like a preprocessing macro so loops don't work here either.

You can do this with a dynamic include.

Hi autogun, i've just published some playbooks around windows patching; see if this helps with your issues.
Regards

@ikunduraci I looked through https://github.com/ikunduraci/Ansible/blob/b3d50cf7bd067443bdf0c6735cce7a9b412b0de3/Windows/install-updates/install-updates.yml and didn't see how that method did an exhaustive loop for all the available updates.


Reimplement Sixel text file tests as shell scripts

This is a reimplementation of my initial Sixel text files tests as shell scripts. As part of the conversion to shell scripts, I've also now made them use yes to generate the background fill in place of DECALN, and I've updated a number of them to better reflect the way the VT340 actually works.

Now that I know how the cursor positioning works, I've updated the cursor position tests so all the final positions line up, and it's clearer when the test is working correctly. I've also added some test cases covering large aspect ratios, which I'm hoping I've interpreted correctly.
I've fixed the test case for an invalid macro parameter, which I had initially thought would be interpreted as 2:1, but is actually 1:1.
I've fixed some of the raster aspect ratios test cases which weren't working the way I had expected. I've also added an additional test case (0:1) of which I'm unsure of the correct interpretation. If block 3 doesn't match the other blocks, then I've got it wrong, and this test will need a slight adjustment.
I've updated the display mode test again in another attempt to get the blocks to line up. I think I've got it right this time, but I'm not completely sure how scrolling works. I think I really need another test that just deals with scrolling offsets.
Given that the VT340 doesn't support the home command, I've updated that test so that it looks "correct" when home isn't supported, rather than the other way around. I think it's still worth keeping the test, just as a record of the VT340 behavior.
In the image movement test, I've rewritten the ICH and DCH test cases to reflect the fact that they erase the image content rather than moving it.
I've adjusted the margin clipping test to reflect the fact that images output below the margin are not displayed at all.
For the line attribute tests, I'm still not exactly sure how the VT340 will behave in some cases, but I've now rewritten the tests in a way that I think will produce some meaningful output.

Screenshots are up. Some minor glitches in the png files due to my use of the VT340 Media Copy control sequence, but I think you can get the gist.

Thanks. Using the media copy to take screenshots is a brilliant idea. And for my tests the little glitches aren't a big deal - they don't need to be pixel perfect, as long as they give a reasonably accurate representation of what's on screen.
It looks like there are still a bunch of issues with some of my tests, but I'm learning something new every time which is great. Some of the fixes are easy, but I may have to ask you you to do some manual testing to figure out a few things that aren't obvious from the current results (I'll raise another issue explaining what I mean).


Virtual html support in rollupOptions.input

Clear and concise description of the problem
vite.config.ts:
rollupOptions: {
  input: {
     // These html files must exist
     main: resolve(__dirname, 'index.html'),
     nested: resolve(__dirname, 'nested/index.html')
  }
}

In webpack we use html-webpack-plugin to make multiple pages, so we only need to provide a template.html, in vite we need to generate a lot of html files locally for vite to parse them as entry points.
Suggested solution
Support virtual html as input files in vite
In vite.config.ts we could provide following implementation
Example:
rollupOptions: {
  input: {
    index: {
        // This can be dynamically generated and does not exist locally
        path: resolve(__dirname, 'index.html'),
        content: '<html> ...'
      },
      login: {
        path: resolve(__dirname, 'login.html'),
        content: '<html> ...'
      },
    }
}

Alternative
Plugin API for manipulating input can be provided
Additional context

vite already supports this, just write a plugin and you can handle it in resolveId and load hook.

I forgot about it... thanks @lovetingyuan

@wyhaya you can use https://github.com/IndexXuan/vite-plugin-html-template


xterm.js source code contains "CR" line ending for some files

xterm.js source code contains "CR" line ending for some files. It produces issue: after clone git repository on the machine with linux based OS command git status displays stash:
modified:   logo.png
modified:   src/test/escape_sequence_files/t0005-CR.in
modified:   src/test/escape_sequence_files/t0011-RI_scroll.in
modified:   src/test/escape_sequence_files/t0014-CAN.in
modified:   src/test/escape_sequence_files/t0015-SUB.in
modified:   src/test/escape_sequence_files/t0023-CUU_scroll.in
modified:   src/test/escape_sequence_files/t0024-CUD.in
modified:   src/test/escape_sequence_files/t0026-CNL.in
modified:   src/test/escape_sequence_files/t0033-VPB_scroll.in
modified:   src/test/escape_sequence_files/t0034-VPR.in
modified:   src/test/escape_sequence_files/t0051-IL.in
modified:   src/test/escape_sequence_files/t0052-DL.in
modified:   src/test/escape_sequence_files/t0073-DECSTBM_RI.in
modified:   src/test/escape_sequence_files/t0101-NLM.in
modified:   src/test/escape_sequence_files/t0500-bash_long_line.in
modified:   src/test/escape_sequence_files/t0503-zsh_ls_color.in
modified:   src/test/escape_sequence_files/t0504-vim.in
Details

Browser and browser version: it doesn't matter.
OS version: Ubuntu 16.04, Fedora 25.
xterm.js version: 2.7.0

Steps to reproduce

Type: git clone https://github.com/xtermjs/xtermjs.org.git
Type "cd xterm.js"
Type command: git status.
Actual result:  git output contains information about not empty stash
Expected result: git status should display info about git status is up to date.

Another solution can be changing * text=auto eol=lf to the  * text=auto
From docs(https://help.github.com/articles/dealing-with-line-endings/): text=auto Git will handle the files in whatever way it thinks is best. This is a good default option.


bug : Anchor IDL address misalignment.

When I was testing my code with TS, I found that my Anchor test code runs fine initially, but after a while, when I run anchor test --skip-deploy (without --skip-build), it throws an error.

At this point, when I check the address in target/idl, I find that the address is not the address of my program, but instead the address of the admin in a mod defined in the lib.

When I manually change the address in the IDL back to the correct pay... address, and then test using anchor test --skip build --skip-deploy, everything works fine.
Therefore, I think this is a bug with anchor build.

This doesn't look like a bug to me because your program should only use declare_id! once. For declaring other public keys, like the one used in the admin module above, you should use the pubkey! macro instead (see https://www.anchor-lang.com/release-notes/0.30.1#pubkey-macro).


Managed Apps Upgrades (Month of June 2021)

Aim to focus on 30DU and get them done within 2 days.
NGINX Ingress Controller @giantswarm/app-squad-nginx  // @ubergesundheit
https://github.com/giantswarm/nginx-ingress-controller-app

Link to latest release:  https://github.com/giantswarm/nginx-ingress-controller-app/releases/tag/v1.17.0
If won't upgrade, why?

[ ] tick this box if there is no new upstream release and no need for upgrade
[x] upgrade https://github.com/giantswarm/nginx-ingress-controller-app/pull/204
[x] release

External DNS @giantswarm/app-squad-external-dns  @webwurst
https://github.com/giantswarm/external-dns-app

Link to latest release: https://github.com/giantswarm/external-dns-app/releases/tag/v2.4.0
If won't upgrade, why?

[ ] tick this box if there is no new upstream release and no need for upgrade
[x] upgrade https://github.com/giantswarm/external-dns-app/pull/91
[x] release

Cert Manager @giantswarm/app-squad-cert-manager  @ubergesundheit
https://github.com/giantswarm/cert-manager-app

Link to latest release:
If won't upgrade, why?

[x] tick this box if there is no new upstream release and no need for upgrade
[ ] upgrade
[ ] release

EFK @giantswarm/app-squad-efk  // @webwurst
https://github.com/giantswarm/efk-stack-app

Link to latest release:
If won't upgrade, why?

[ ] tick this box if there is no new upstream release and no need for upgrade
[ ] upgrade
[ ] release

Kong @giantswarm/app-squad-kong  @ubergesundheit
https://github.com/giantswarm/kong-app

Link to latest release:
If won't upgrade, why?

[ ] tick this box if there is no new upstream release and no need for upgrade
[x] upgrade https://github.com/giantswarm/kong-app/pull/134
[x] release

Check to see if these issues are solved by the upgrade

[ ] https://github.com/giantswarm/giantswarm/issues/17165
[ ] https://github.com/giantswarm/adidas/issues/713#issuecomment-851558480

Prometheus Operator @giantswarm/app-squad-prometheus  @webwurst
https://github.com/giantswarm/prometheus-operator-app

Link to latest release: https://github.com/giantswarm/prometheus-operator-app/releases/tag/v0.8.2
If won't upgrade, why?

[ ] tick this box if there is no new upstream release and no need for upgrade
[x] upgrade
[x] address CVE https://github.com/giantswarm/giantswarm/issues/17281
[x] release

Loki @giantswarm/app-squad-prometheus  @giantswarm/app-squad-loki @ubergesundheit
https://github.com/giantswarm/loki-app

Link to latest release: https://github.com/giantswarm/loki-app/releases/tag/v0.2.0
If won't upgrade, why?

[ ] tick this box if there is no new upstream release and no need for upgrade
[x] upgrade https://github.com/giantswarm/loki-app/pull/37/files
[x] release

Aqua - we upgrade when they say there is sth in the Aqua meeting w the customer.

Notes

Team divides the app upgrades amongst ourselves. Please add @ name beside the app
30 Day Upgrade Promise: By default, we upgrade our apps within 30 days of an upstream release.
Engineer can override this default and decide not to upgrade with good reason.

Request to include external-dns in >15.0.0 releases https://github.com/giantswarm/releases/pull/741

All done!! :heart:


[Feature Request] Ability to send SMS via  a single press "Request for help" interface

Subject of the issue
Incorporate the ability to request help via SMS  with a single button press, from the submit request interface in https://keralarescue.in/request
Your environment

python3
all
Ubuntu

Steps to reproduce
Go to https://keralarescue.in/request/.
Expected behaviour
A user in distress will not have the ability to enter all the information and choose from the drop down menu.
Having the ability to trigger an SMS  to a helpline, with a single click, that will collect the gps location, and send an message to the nearest fire station or rescue center who can be dispatched to help.
Future addition:  A panic button alert in the home page, which will  receive the phone number and auto populate the gps coordinates, owner info and alert the rescue crew for help.
Actual behavior
User in distress has to enter all details manually. This will not help people who are in the middle of an emergency situation. Automation is the key

This is being tracked by #463. Duplicate issue.


RPC system error when calling get balanced on governance addresses

Don't know if this is something worth looking at or fixing as it is an edge case, but when calling:
{
    "jsonrpc": "2.0",
    "method": "icx_getBalance",
    "id": 1234,
    "params": {
         "address": "hx0000000000000000000000000000000000000000"
    }
}

Or on hx0000000000000000000000000000000000000001, I get the following error:
Response={"jsonrpc":"2.0","error":{"code":-31000,"message":"SystemError: E1002:InvalidAddressPrefix(valid=cx0000000000000000000000000000000000000000)"},"id":1234}

I built in an exception handler in my logic for these addresses but figured I'd raise this as an issue to take note.

ID part of hx0000000000000000000000000000000000000000 is 0x0000000000000000000000000000000000000000. And currently, 0x0000000000000000000000000000000000000000 is system contract. So, it should use cx0000000000000000000000000000000000000000 instead of hx0000000000000000000000000000000000000000.
So, it's an expected behavior.

Thanks @mksong-icon - Yeah I didn't think this was a big issue but figured I'd raise just in case. Thanks for clarification.


[glib] Build failure on x64-windows

Package<EMAIL_ADDRESS>Host Environment

Host: x64-windows
Compiler: MSVC 19.38.33135.0
vcpkg-tool version: 2024-02-07-8a83681f921b10d86ae626fd833c253f4f8c355b
vcpkg-scripts version: 7f9f0e44db 2024-03-13 (27 hours ago)

To Reproduce
vcpkg install glib
Failure logs
-- Using cached glib-2.78.4.tar.xz.
-- Cleaning sources at C:/programs/compa_libs/vcpkg/buildtrees/glib/src/glib-2-322e03f702.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/programs/compa_libs/vcpkg/downloads/glib-2.78.4.tar.xz
-- Applying patch use-libiconv-on-windows.patch
-- Applying patch libintl.patch
-- Using source at C:/programs/compa_libs/vcpkg/buildtrees/glib/src/glib-2-322e03f702.clean
-- Found external ninja('1.11.0').
-- Getting CMake variables for x64-windows
-- Configuring x64-windows-rel
-- Using cached msys2-mingw-w64-x86_64-pkgconf-1~2.1.0-1-any.pkg.tar.zst.
-- Using cached msys2-msys2-runtime-3.4.10-4-x86_64.pkg.tar.zst.
-- Using msys root at C:/programs/compa_libs/vcpkg/downloads/tools/msys2/fdbea3694fb5c0d4
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
    Command failed: C:/programs/compa_libs/vcpkg/downloads/tools/python/python-3.11.8-x64/python.exe -I C:/programs/compa_libs/vcpkg/installed/x64-windows/tools/meson/meson.py -Dselinux=disabled -Dlibmount=disabled -Dgtk_doc=false -Dinstalled_tests=false -Dlibelf=disabled -Dman=false -Dtests=false -Dxattr=false --backend ninja --wrap-mode nodownload -Dbuildtype=plain -Doptimization=plain --libdir lib --native C:/programs/compa_libs/vcpkg/buildtrees/glib/meson-x64-windows-rel.log -Ddebug=false --prefix C:/programs/compa_libs/vcpkg/packages/glib_x64-windows -Dcmake_prefix_path=['C:/programs/compa_libs/vcpkg/installed/x64-windows','C:/programs/compa_libs/vcpkg/installed/x64-windows/debug','C:/programs/compa_libs/vcpkg/installed/x64-windows/share'] C:/programs/compa_libs/vcpkg/buildtrees/glib/src/glib-2-322e03f702.clean
    Working Directory: C:/programs/compa_libs/vcpkg/buildtrees/glib/x64-windows-rel
    Error code: 1
    See logs for more information:
      C:\programs\compa_libs\vcpkg\buildtrees\glib\config-x64-windows-rel-meson-log.txt.log
      C:\programs\compa_libs\vcpkg\buildtrees\glib\config-x64-windows-rel-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_meson.cmake:436 (vcpkg_execute_required_process)
  ports/glib/portfile.cmake:42 (vcpkg_configure_meson)
  scripts/ports.cmake:175 (include)

C:\programs\compa_libs\vcpkg\buildtrees\glib\config-x64-windows-rel-out.log
WARNING: Recommend using either -Dbuildtype or -Doptimization + -Ddebug. Using both is redundant since they override each other. See: https://mesonbuild.com/Builtin-options.html#build-type-options
The Meson build system
Version: 0.62.1
Source dir: C:\programs\compa_libs\vcpkg\buildtrees\glib\src\glib-2-322e03f702.clean
Build dir: C:\programs\compa_libs\vcpkg\buildtrees\glib\x64-windows-rel
Build type: native build

..\src\glib-2-322e03f702.clean\meson.build:1:0: ERROR: Value "plain" (of type "string") for combo option "Optimization level" is not one of the choices. Possible choices are (as string): "0", "g", "1", "2", "3", "s".

A full log can be found at C:\programs\compa_libs\vcpkg\buildtrees\glib\x64-windows-rel\meson-logs\meson-log.txt

C:\programs\compa_libs\vcpkg\buildtrees\glib\config-x64-windows-rel-meson-log.txt.log
Build started at 2024-03-15T03:25:50.974798
Main binary: C:\programs\compa_libs\vcpkg\downloads\tools\python\python-3.11.8-x64\python.exe
Build Options: -Dselinux=disabled -Dlibmount=disabled -Dgtk_doc=false -Dinstalled_tests=false -Dlibelf=disabled -Dman=false -Dtests=false -Dxattr=false -Dbuildtype=plain -Doptimization=plain -Ddebug=false '-Dcmake_prefix_path=['"'"'C:/programs/compa_libs/vcpkg/installed/x64-windows'"'"','"'"'C:/programs/compa_libs/vcpkg/installed/x64-windows/debug'"'"','"'"'C:/programs/compa_libs/vcpkg/installed/x64-windows/share'"'"']' -Dprefix=C:/programs/compa_libs/vcpkg/packages/glib_x64-windows -Dlibdir=lib -Dbackend=ninja -Dwrap_mode=nodownload '--native-file C:/programs/compa_libs/vcpkg/buildtrees/glib/meson-x64-windows-rel.log'
Python system: Windows
The Meson build system
Version: 0.62.1
Source dir: C:\programs\compa_libs\vcpkg\buildtrees\glib\src\glib-2-322e03f702.clean
Build dir: C:\programs\compa_libs\vcpkg\buildtrees\glib\x64-windows-rel
Build type: native build

..\src\glib-2-322e03f702.clean\meson.build:1:0: ERROR: Value "plain" (of type "string") for combo option "Optimization level" is not one of the choices. Possible choices are (as string): "0", "g", "1", "2", "3", "s".

Duplicate of https://github.com/microsoft/vcpkg/issues/37393, this issue has been fixed by PR https://github.com/microsoft/vcpkg/pull/37396. I can't reproduce this issue locally.
Did you run command git pull before installation?

Well, the fix must have arrived simultaneous to my attempts because I had done a git pull in day before or something like that.
But it works now, thanks.


Allow workflow_dispatch jobs to target a specific repo

Dispatch actions run the whole set of repositories.  Add an input so it can target just one or a reduced set.

https://github.blog/changelog/2021-11-10-github-actions-input-types-for-manual-workflows/


bazel: Use hermetic JDK 11

Description: Using hermetic JDK 11 with Bazel. This improves caching, while also removing the need to manage a local JDK.
Risk Level: Low.
Testing: Built locally.
Docs Changes: N/A
Release Notes: N/A
I feel the risk is low, as this only changes the Java Runtime used, but not the toolchain used for compiling (which would still be target version of 8).

Failures seem to be related to the Java 8 ClassLoader change. Looking into a fix.

:clap: @brentleyjones next up hermetic NDK??? :)

I do have an idea for that, it's just less out of the box.

I do have an idea for that, it's just less out of the box.

 :)


Add the "hacktoberfest" topic

If this is a feature request:
Hacktoberfest now requires repositories to include the "hacktoberfest" topic for PRs to count. Assuming we want this repository to count for Hacktoberfest we should add this topic. See https://hacktoberfest.digitalocean.com/details#details. Alternatively the label hacktoberfest-accepted could be added to PRs.

Done! 🎉


Let's make a WebTorrent sticker!

Calling everyone with graphical design skills!
We need a logo for webtorrent.io, as well as a hex sticker design for our laptops!
Hex stickers are awesome because you can tile them.

I really like the Node School international day sticker:

So I tried to make something based on that:

This one gets rid of the satellite and uses a rocket instead. And it's connected to the rest of the webtorrent network!

Still looks a bit too similar to the NodeSchool international day sticker.

There's a bunch of inspirational hex designs on Hexbin and Hexdex. Use to get ideas!

The logo with the rocket looks fantastic.

Here's another attempt:

I want some too! Shape is no issue for me.

I would love a sticker for my laptop :)
Idea: a rocket with a heart leaning out the widow; you could imagine it similar to the Github logo

Replaced the round window in the rocket with a heart-shaped one.

Finally learned how to use an "erase layer mask", something I've wanted to do for years, but could never adequately describe and find on google.

An amateurish attempt at an option..

@bitanath Cool, hadn't considered that circuit boards look kinda like peer-to-peer networks!

@feross Just putting it out there... anyway, one with the heart inlay :)

Hi Guys, just to say I'm completely fond of that last one with the circuit
board. It looks totally awesome !
On 25 January 2016 at 05:57, bitanath<EMAIL_ADDRESS>wrote:

[image: webtorrent]
https://cloud.githubusercontent.com/assets/7344942/12542814/1f025068-c34e-11e5-90b2-3adb83b4876e.png
@feross https://github.com/feross Just putting it out there... anyway,
one with the heart inlay :)
—
Reply to this email directly or view it on GitHub
https://github.com/feross/webtorrent/issues/586#issuecomment-174398888.

I was thinking about adding some water to represent the bittorrent ecosystem but I am not able to make a design on my own ;)
I like your stickers but in my opinion, the rounded typo is not that great.
Good job!

Latest iteration, lots more love:

@yciabaud got a suggested font?

No rocket version:

Okay, so I just printed 500 of the last sticker!
Look out for me at conferences and meetups to pick one up for free! If you've contributed to WebTorrent, send a message to my email (on my github profile) and I'll send you one for free!
If you just want to buy one for yourself, here's a link to get one from StickerMule: https://www.stickermule.com/en/marketplace/10200-webtorrent
(I added no markup, so the price is just what StickerMule charges to print the stickers and it goes entirely to them)

@feross   at the end of the rocket you can add the heart rather then the smoke

The WebTorrent stickers just arrived!

Look out for me at conferences and meetups to pick one up for free! You can also just buy one for yourself right now from StickerMule (lowest price, no markup).

I want some! But i'm now in NYC :( -- i'll be back to SF soon.

@jbenet sounds good! sorry i missed you this time :(

Wow, they look awesome!!
I'd love some, but I live in Argentina, so I don't think it would be possible.
Also, I don't think I deserve them! My contributions (if any) were so small that I don't even remember them :flushed:

@feross sent you my address on IRC :)

@ivantodorovich that's alright! if you want some stickers, i'm happy to send you some! i can send to argentina!
@whyrusleeping got it!


Update catalog deployment template

The catalog deployment template only sets the
readiness and liveness probes to HTTPS Scheme if the
values.catalog.tlsKeyPath and values.catalog.tlsCertPath
configurations are provided. These values should be set
when the values.catalog.tlsSecret or
values.catalog.clientCASecret configurations are set as
it is done in the olm deployment template.

@benluddy addressed with the latest changes. Updated commit message.

/retest

/lgtm


Error: Kubernetes cluster unreachable: the server has asked for the client to provide credentials

Description
I had this same issue when I was just taking the Karpenter example out and using registry image for EKS, but since it wasn't working I decided to try with the exact modules the example is using, so when I cloned the repo I got the same error.
Error: Kubernetes cluster unreachable: the server has asked for the client to provide credentials
with helm_release.karpenter
on main.tf line 149, in resource "helm_release" "karpenter":
resource "helm_release" "karpenter" {
Forked Repo with small changes: https://github.com/crystal-koii/terraform-aws-eks
My changes to the original example are:

Adding the cloud block
changing region to us-east-1 in main.tf locals
Adding the var for AWS_EKS_ADMIN_ROLE_ARN to variables.tf
Adding assume role to AWS providers in main.tf

Steps to reproduce:

clone the repo
make sure you're logged in to Terraform HCP cloud service from the console
Make a role with appropriate permissions and add the arn to the cloud variables
Make sure you have an AWS user who can assume the role added to the cloud variables
Change the terraform cloud block to be your organization
Go into the workspace settings and add the relative path: examples/karpenter
Run commands

Commands
cd examples/karpenter
terraform init
terraform plan
terraform apply
``
- Takes a long time to build some resources, so wait like 20 minutes.
- Everything will build except helm_release.karpenter and 3 kubectl_manifest ones. See picture:
![image](https://github.com/terraform-aws-modules/terraform-aws-eks/assets/170098609/e2dcfe95-c48a-481a-bcc1-f35af43bbd36)

- [x] ✋ I have searched the open/closed issues and my issue is not listed.

## ⚠ Note

Before you submit an issue, please perform the following first:

- [x] 1. Remove the local `.terraform` directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): `rm -rf .terraform/`
- [x] 5. Re-initialize the project root to pull down modules: `terraform init`
- [x] 6. Re-attempt your terraform plan or apply and check if the issue still persists

## Versions

- Module version [Required]: same as in repo

- Terraform version:
Terraform v1.8.5 ...
~~I noticed it's out of date so I'm going to update to 1.9.1 and try again ~~
I updated to 1.9.1 and the issue persists

- Provider version(s):
As defined in repo

## Reproduction Code [Required]

https://github.com/crystal-koii/terraform-aws-eks

Steps to reproduce the behavior:

<!-- Are you using workspaces? -->
cloud workspaces
<!-- Have you cleared the local cache (see Notice section above)? -->
Yes
<!-- List steps in order that led up to the issue you encountered -->
see above

## Expected behavior
terraform apply to build everything correctly

## Actual behavior
Error: Kubernetes cluster unreachable: the server has asked for the client to provide credentials
with helm_release.karpenter
on main.tf line 149, in resource "helm_release" "karpenter":
resource "helm_release" "karpenter" {

### Terminal Output Screenshot(s)

<!-- Optional but helpful -->

I figured it out with help from the senior dev supervising me, I was missing the role arn as an arg in the helm provider exec block.
Solution:
If you need a role arn so that the EKS isn't tied to the account that makes it so multiple users can assume the role:
Change the args to:
# args = ["eks", "get-token", "--cluster-name", module.eks.cluster_name]
args = ["eks", "get-token", "--cluster-name", local.cluster_name,  "--role-arn", var.AWS_EKS_ADMIN_ROLE_ARN, "--output", "json"]


No provider for CompleteTestService!

Hello,
I've followed your instruction but i've faced some issues.
I'm working on a lazy loading page, so i've imported AutoCompleteModule on my lazypage.module.ts file. Then i imported CompleteTestService on my lazypage.ts file.
I'm getting the following error:

Uncaught (in promise): Error: StaticInjectorError(AppModule)[LazyPage -> CompleteTestService]:
StaticInjectorError(Platform: core)[LazyPage -> CompleteTestService]:
NullInjectorError: No provider for CompleteTestService!
Error: StaticInjectorError(AppModule)[LazyPage -> CompleteTestService]:
StaticInjectorError(Platform: core)[LazyPage -> CompleteTestService]:
NullInjectorError: No provider for CompleteTestService!

My CompleteTestService file looks like the one you provided in the instruction page.
Any help will be appreciated!

i've tried this on a new project. I got this error TypeError: Cannot read property 'getResults' of undefined

I've tried this on a new empty project. I got this error TypeError: Cannot read property 'getResults' of undefined
I'm working on ionic 3 btw

I've used another component (ng2-completer) to finish up with my application, but this need to be fixed !

It seems like I am having the same issue. I am getting this error:

ERROR Error: Uncaught (in promise): Error: StaticInjectorError[CompleteTestService]:
StaticInjectorError[CompleteTestService]:
NullInjectorError: No provider for CompleteTestService!
Error: StaticInjectorError[CompleteTestService]:
StaticInjectorError[CompleteTestService]:
NullInjectorError: No provider for CompleteTestService!

I'm getting this error too when ionic is ran in development mode, haven't tried production though. What seems to temporarily fix the issue is when you modify/save the provider class and ionic rebuilds the components.

@karlerikounapuu if you mean that you found a workaround, can you explain what is it ? what do you mean by modify / save the provider ?

I ran into the same issue. Here is how I solved it:
import { CompleteTestService } from '../providers/CompleteTestService'; into your app.module.ts file.
Add CompleteTestService to the providers in app.module.ts :)

Since this project is not actively maintained I have fixed this issue and others in a forked repository.
Ionic4-auto-complete


Issue/3

Set up the data pipeline for an RNN (with some extensibility to other models); RNN can now train with no problem. Still, however, may need some future work done in the realm of manipulating the (ra, dec) point coordinates into better representations of position (PointNet?) or processing them for specific network architectures (graph net, for example).

Seems like you went well beyond data loading! The code looks clear and very well documented. I had some minor comments. I haven't run the training myself yet, but it'd be very interesting to see how the multiple objects (right now, simply concatenated) are processed. Let's talk soon. Great work!


added target velocity functionality and tests

This will need to be merged at some point when full simulations are added, but I wanted to test out CI stuff on github.

@AnvayPradhan @nzamb222 Can yall try to figure out why the automatic tests cant file the lapacke function? It worked last night but after I added the psuedo inverse function in the code, it won't work now.

is it the cube example or something else? Nice work

@nzamb222 its supposed to just be simple fold but something has gone seriously wrong haha

@nzamb222 If you get a chance, can you make sure I am making the projection matrix correctly in the simulation method of the structure class. it was really hard to make the animations and they do not look good. It could just be because the projection method isn't the best for specifying specific motions, but it could also be something is programmed wrong. I tried to write a test to verify that I was projecting correctly, but I could not get the test to pass.


Fix-broken-jar

jar was not fixed after introduction of yazl; not  it should; added another proper test with actual files for good measure

Thanks again for fixing on such short notice !

Thanks again for fixing on such short notice !

I break it, I fix it.


Count reopened connections in Active Connection Tracking

Fixes https://github.com/cilium/cilium/pull/32584
Part of https://github.com/cilium/cilium/issues/31752
Reopened connections are now correctly counted towards opened in Active Connection Tracking

/test

/test

/test


Add support for "preserve" argument for addons

Is your request related to a new offering from AWS?
Is this functionality available in the AWS provider for Terraform? See CHANGELOG.md, too.

Yes ✅: 4.10.0

Describe the solution you'd like.
Add support for preserve argument in resource "aws_eks_addon", similar to other arguments.
Describe alternatives you've considered.
Managing addons out of the module.
Additional context
I understand that AWS v4 has significant breaking changes and not all people can or want to upgrade (though v4.9.0 has improvements in this regard), so for now just probing whether setting min version of aws provider to 4.10.0 in this module is something that would be considered.

marking as duplicate of #1908 - when that is added this functionality will be supported as well

actually keeping it open since its not same source 😅


[hot fix] Set upper version for pandas

While digdag-python:3.7 uses numpy 1.16.14, pandas 1.2 drops it. For
stability, will set upper version of pandas for while.

Confirmed it works with digdag-python:3.7 image:
root@69d5095d2f3d:/opt# pip install -e .
Obtaining file:///opt
Requirement already satisfied: urllib3<1.25,>=1.21.1 in /usr/local/lib/python3.7/site-packages (from pytd==1.4.2) (1.24.3)
Requirement already satisfied: presto-python-client>=0.6.0 in /usr/local/lib/python3.7/site-packages (from pytd==1.4.2) (0.7.0)
Collecting pandas<1.2,>=0.25.0 (from pytd==1.4.2)
  Downloading https://files.pythonhosted.org/packages/fd/70/e8eee0cbddf926bf51958c7d6a86bc69167c300fa2ba8e592330a2377d1b/pandas-1.1.5-cp37-cp37m-manylinux1_x86_64.whl (9.5MB)
     |████████████████████████████████| 9.5MB 7.3MB/s
Requirement already satisfied: td-client>=1.1.0 in /usr/local/lib/python3.7/site-packages (from pytd==1.4.2) (1.2.1)
Requirement already satisfied: pytz>=2018.5 in /usr/local/lib/python3.7/site-packages (from pytd==1.4.2) (2019.1)
Requirement already satisfied: numpy<1.20 in /usr/local/lib/python3.7/site-packages (from pytd==1.4.2) (1.16.4)
Requirement already satisfied: six in /usr/local/lib/python3.7/site-packages (from presto-python-client>=0.6.0->pytd==1.4.2) (1.12.0)
Requirement already satisfied: click in /usr/local/lib/python3.7/site-packages (from presto-python-client>=0.6.0->pytd==1.4.2) (7.0)
Requirement already satisfied: requests in /usr/local/lib/python3.7/site-packages (from presto-python-client>=0.6.0->pytd==1.4.2) (2.22.0)
Requirement already satisfied: python-dateutil>=2.7.3 in /usr/local/lib/python3.7/site-packages (from pandas<1.2,>=0.25.0->pytd==1.4.2) (2.8.0)
Requirement already satisfied: msgpack>=0.6.2 in /usr/local/lib/python3.7/site-packages (from td-client>=1.1.0->pytd==1.4.2) (1.0.2)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in /usr/local/lib/python3.7/site-packages (from requests->presto-python-client>=0.6.0->pytd==1.4.2) (3.0.4)
Requirement already satisfied: idna<2.9,>=2.5 in /usr/local/lib/python3.7/site-packages (from requests->presto-python-client>=0.6.0->pytd==1.4.2) (2.8)
Requirement already satisfied: certifi>=2017.4.17 in /usr/local/lib/python3.7/site-packages (from requests->presto-python-client>=0.6.0->pytd==1.4.2) (2019.6.16)
Installing collected packages: pandas, pytd
  Found existing installation: pandas 1.2.0
    Uninstalling pandas-1.2.0:
      Successfully uninstalled pandas-1.2.0
  Found existing installation: pytd 1.4.2
    Uninstalling pytd-1.4.2:
      Successfully uninstalled pytd-1.4.2
  Running setup.py develop for pytd
Successfully installed pandas-1.1.5 pytd
WARNING: You are using pip version 19.1.1, however version 21.0.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.


Add new files to the 'data/examples' directory

From https://zenodo.org/records/10647894
close #76

Could we keep only DNA_start.gro and RNA_start.gro? All other structures are kind of redundant.

We don't need DNA_start_nowat.gro and RNA_start_nowat.gro

I already remove them

OK. Ready for merge?

Yes ready to merge


Scraper Fails to Run for Company Deloitte

Environment:
Production URL: https://admin.peviitor.ro/
Browser: Chrome
Device: PC
OS: Windows 10
Steps to Reproduce:

Open the URL https://admin.peviitor.ro/ in the browser.
Navigate to the "Cont" section.
Attempt to run the scraper "deloitte.py".

Expected Result:
The scraper runs successfully.
Actual Result:
The scraper does not run.
Attachments:

Issus fixed

retested, all works


⚠ Bofrost.be has degraded performance

In 64a0f84, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1139 ms

Resolved: Bofrost.be performance has improved in f9111f7.


Issue while encoding decoded strings with ~ character

If we encode dwfedhgwefdf~12gehgdhegd and then decode it, then what i am getting is 'dwfedhgwefdf�12gehgdhegd'

I could not reproduce your claim:
> var Base64 = require("js-base64");
undefined
> var str='dwfedhgwefdf~12gehgdhegd';
undefined
> Base64.decode(Base64.encode(str))
'dwfedhgwefdf~12gehgdhegd'
> Base64.decode(Base64.encode(str)) === str;
true

Would you tell me your environment more in detail?


It seems createMany not generating.

It seems you should start reading release notes before asking:
https://github.com/MichalLytek/typegraphql-prisma/releases


Non-idempotent repeatedly growing indentation in doc attribute containing multi-line concat

Minimized from https://github.com/rust-lang/rust/blob/1.74.0/library/core/src/num/nonzero.rs#L341-L342.
macro_rules! repro {
    () => {
        #[doc = concat!("let var = ",
                        "false;")]
        fn f() {}
    };
}

As of current master, rustfmt applies the following diff. This is from cargo run --bin rustfmt -- /path/to/repro.rs.
-                        "false;")]
+                                "false;")]

If you run rustfmt again, it keeps going for 16 levels of indentation before reaching an idempotent state.
-                        "false;")]
+                                        "false;")]

-                        "false;")]
+                                                "false;")]

-                        "false;")]
+                                                        "false;")]

-                        "false;")]
+                                                                "false;")]

-                        "false;")]
+                                                                        "false;")]

-                        "false;")]
+                                                                                "false;")]

-                        "false;")]
+                                                                                "false;")]

-                        "false;")]
+                                                                                "false;")]

@rustbot claim

Had this issue happen to me as well, with the following (on rustfmt 1.7.1-nightly (fda509e 2024-06-25)):
macro_rules! my_macro {
    () => {
        // Breaks
        #[doc(concat!(
        "Multi\
        Line"
        ))]
        pub struct Struct;
        // Incorrect but not broken
        #[doc(
        "Multi\
        Line",
        )]
        pub struct Struct;
    };
}

// Normal
#[doc(concat!(
"Multi\
Line"
))]
pub struct Struct;

https://github.com/user-attachments/assets/c84fdf8f-bc49-4fc0-8aba-ebba8d0f8038
I noticed that there seems to be a difference when I don't use the concat!() macro call. Also, I'm using a single string, just split over multiple lines.
Possible related to #6161?


Pt

Please ensure you have given all the following requested information in your report.
8034Issue details
Please provide the details of your issue
3700Reproduction steps/code
Even if you think your issue is trivial to reproduce, please supply a SSCCE that demonstrates your issue. This saves time on our end, and makes it much more likely that your issue will be fixed.
You can find barebones templates here
Version of libGDX and/or relevant dependencies
Please provide the version(s) affected.
Stacktrace
//Please provide the stacktrace if applicable

Please select the affected platforms

[ ] Android
[ ] iOS
[ ] HTML/GWT
[ ] Windows
[ ] Linux
[ ] macOS

Please ensure you have given all the following requested information in your report.
8034Issue details
Please provide the details of your issue
3700Reproduction steps/code
Even if you think your issue is trivial to reproduce, please supply a SSCCE that demonstrates your issue. This saves time on our end, and makes it much more likely that your issue will be fixed. You can find barebones templates here
Version of libGDX and/or relevant dependencies
Please provide the version(s) affected.
Stacktrace
//Please provide the stacktrace if applicable

Please select the affected platforms

[ ]  Android
[ ]  iOS
[ ]  HTML/GWT
[ ]  Windows
[ ]  Linux
[ ]  macOS

https://github.com/libgdx/libgdx/issues/6768#issue-1105105448


⚠ Bofrost.nl has degraded performance

In 0b7e3fc, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 3422 ms

Resolved: Bofrost.nl performance has improved in 946ed46.


Implement : Joystick support in the custom mode

This is the joystick support, using SFML.
All you need is to add the json key "joyCodes" in the same place of "keyCodes" in the custom section.
This json key is not require in every key object, just add this when you want to have a joystick support.

It come with a joystick debugger, to show the information about joystick.
Toggle by key "Control + D"

Joystick have a map range.
keycode 0 - 31 is for the joystick button.
and keycode 32 - 45 is for the joystick axis, follow by this order

Looks pretty promising! I will take a closer look soon.

Looks pretty promising! I will take a closer look soon.


[api-docs] Update and alphabetize accepted locales

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/727077eb028a56982fa95bc9e91fcd2d56a3e270

ncn


Add NPM distribution

Closes #561
Check the following

[x] Keep 100% code coverage
[x] Be properly formatted
[x] Documentation changes are included
[x] Include a change file if expected

https://www.npmjs.com/package/changie ran the commands locally and published the package, the npx and install commands seem to be working for me on linux, not sure if you want to give this a try before I merge it.

yes, that works - sorry, it took a while to install from Australia as it hadn't reached the CDN end nodes yet 😅

yes, that works - sorry, it took a while to install from Australia as it hadn't reached the CDN end nodes yet 😅

haha, that explains the timing of all your comments :smirk:
Alright, lets see how this goes. :laughing:

hehe yeah I'm sure some of my activity is at weird hours for you 😆
I don't think it's a huge problem, but you seem to have used the main branch to build and release that rather than the 1.14 tag. This build includes the swap from changie merge --project to auto-merging all projects.

hehe yeah I'm sure some of my activity is at weird hours for you 😆
I don't think it's a huge problem, but you seem to have used the main branch to build and release that rather than the 1.14 tag. This build includes the swap from changie merge --project to auto-merging all projects.

oh yea... let me delete v1.14 from NPM as its not accurate then.

@TheSpyder ok it seems like v1.15.0 is up on NPM properly with your auto deployment. Had some issues with winget but all good for now, will have to come back to that one later.

Ah yeah NPM doesn't really support deleting versions, the closest they have is deprecate
But nobody is using it yet so I'm sure they won't notice 🙂


Add comment about comment

Comments might get added part of: https://github.com/atom/language-json/pull/29.

Aha okay.
Official specs for json doesn't allow comments.

Yeah.. I guess it's more like the unofficial // in CSS.
/* real CSS comment */
// fake CSS comment


'/home/pi/rhasspy/profiles/de/sentences.ini' does not start with '/home/pi/.config/rhasspy/profiles/de'

I just did a fresh installation (venv method) on a clean and new raspbian stretch installation (on a pi 3), and I get the following error in the web GUI:
 '/home/pi/rhasspy/profiles/de/sentences.ini' does not start with '/home/pi/.config/rhasspy/profiles/de'

Also, when going to sentences, it looks pretty empty:
https://imgur.com/a/M0whJmE
So I can see that I have both directories:
pi@henrietta:~/rhasspy $ ll ~/.config/rhasspy/profiles/de/
total 15M
drwxr-xr-x 6 pi pi  4.0K Dec 17 20:37 .
drwxr-xr-x 4 pi pi  4.0K Dec 17 21:05 ..
drwxrwxr-x 2 pi pi  4.0K Dec 17  2017 acoustic_model
-rw-rw-r-- 1 pi pi 1021K Dec 17  2017 base_dictionary.txt
-rw-r--r-- 1 pi pi    47 Dec 17 20:37 .doit.db.bak
-rw-r--r-- 1 pi pi   721 Dec 17 20:37 .doit.db.dat
-rw-r--r-- 1 pi pi    47 Dec 17 20:37 .doit.db.dir
drwxr-xr-x 2 pi pi  4.0K Dec 17 20:35 download
drwxr-xr-x 2 pi pi  4.0K Dec 17 20:37 fsts
-rw-r--r-- 1 pi pi   14M May 14  2019 g2p.fst
drwxr-xr-x 2 pi pi  4.0K Dec 17 20:37 grammars
-rw-r--r-- 1 pi pi  2.5K Dec 17 20:37 intent.fst
-rw-r--r-- 1 pi pi  5.9K Dec 17 20:37 intent.json

and
pi@henrietta:~/rhasspy $ ll profiles/de
total 416K
drwxr-xr-x  3 pi pi 4.0K Dec 17 17:11 .
drwxr-xr-x 16 pi pi 4.0K Dec 17 17:11 ..
-rwxr-xr-x  1 pi pi   90 Dec 17 17:11 custom_words.txt
-rwxr-xr-x  1 pi pi  631 Dec 17 17:11 dictionary.txt
-rwxr-xr-x  1 pi pi  321 Dec 17 17:11 espeak_phonemes.txt
-rwxr-xr-x  1 pi pi 298K Dec 17 17:11 frequent_words.html
-rw-r--r--  1 pi pi  45K Dec 17 17:11 frequent_words.phones
-rwxr-xr-x  1 pi pi  22K Dec 17 17:11 frequent_words.txt
drwxr-xr-x  2 pi pi 4.0K Dec 17 17:11 kaldi
-rwxr-xr-x  1 pi pi 3.0K Dec 17 17:11 language_model.txt
-rwxr-xr-x  1 pi pi 1.1K Dec 17 17:11 phoneme_examples.txt
-rwxr-xr-x  1 pi pi 3.4K Dec 17 17:11 profile.json
-rwxr-xr-x  1 pi pi  326 Dec 17 17:11 sentences.ini
-rw-r--r--  1 pi pi 1.4K Dec 17 17:11 stop_words.txt

What should I do?
Weird thing is, I was having a clean installation on another pi like 2 days ago, following the exact same steps, and I ended up with a working sentences.ini. Now it no longer the case as it seems.

I too experienced this, but with the latest docker image as a fresh installation yesterday. Adding a sentences.ini with some content resolved it, however, previously a sentences.ini file would be present with some default config.

Okay well the sentences.ini is present, its in /home/pi/rhasspy/profiles/de/sentences.ini. So should this be put into ~/.config/rhasspy/profiles/de/ now instead? What about the other stuff in the /home/pi/rhasspy/profiles/de folder?

Sorry, fix for this has been pushed. Should be live soon!

Fix is now in the latest Docker image. Please re-open if issue continues!


Multiple transaction popup keeps returning after (x) it out to close

To recreate:
try sweeping an orderbook (take multiple trades at once) to trigger the notification pop-up in the order ticket.  Click the (x) in the top right of the message (DONT do the trade yet), wait 5 seconds and it returns

component doesn't rem that the user dismissed it. the component refreshes and loses all state.
That is current behavior. We'll need to hold state outside of component and reset when either trade is executed or form is cleared.
This won't be an issue with 0x trading.


GYB cannot restore non-ASCII labels

./gyb --email<EMAIL_ADDRESS>--action restore --service-account --local-folder<EMAIL_ADDRESS>
Using backup folder<EMAIL_ADDRESS>Creating label Приоритетные
400: Invalid label name - invalidArgument

--extra-reserved-labels "Приоритетные" don't help, got the same error. It looks like gyb cannot recognize non-ASCII characters in labels.
When I launched gyb in debug mode, I saw it trying to post Unicode characters but got 400 error from Gmail.
send: b'POST /gmail/v1/users/me/labels?fields=id&prettyPrint=true&alt=json HTTP/1.1\r\nHost: gmail.googleapis.com\r\naccept: application/json\r\naccept-encoding: gzip, deflate\r\nuser-agent: Got Your Back 1.70 | jaylee.us/gyb | Jay Lee -
<EMAIL_ADDRESS>| Python 3.10.5 64-bit final | google-api-client 2.51.0 | Linux-3.10.0-862.9.1.el7.x86_64-x86_64-with-glibc2.35 x86_64 (gzip)\r\nx-goog-api-client: gdcl/2.51.0 gl-python/3.10.5\r\ncontent-type: application/json\r\ncont
ent-length: 153\r\nauthorization: Bearer XXXXXXXX\r\n\r\n'
send: b'{"labelListVisibility": "labelShow", "messageListVisibility": "show", "name": "\\u041f\\u0440\\u0438\\u043e\\u0440\\u0438\\u0442\\u0435\\u0442\\u043d\\u044b\\u0435"}'

reply: 'HTTP/1.1 400 Bad Request\r\n'

See #227 for a workaround.

@krystof-k thank you for the link but looks like it's not applicable for backup via service account.


Semantic tokens should be type checked

Description
When I create a semantic token which references another token, I expect to have some kind of checking to prevent me from referencing tokens which don't exist when I have strictTokens: true,
  theme: {
    tokens: {
      colors: {
        myColor: { value: 'red' }
      }
    },

    semanticTokens: {
      colors: {
        mySemanticToken: { value: '{colors.myColor}' },
        mySemanticTokenTypo: { value: '{colors.invalid}' }
      }
    }
  },

In this example I can reference mySemanticTokenTypo in my code and I won't get any type errors (that makes sense, it's a valid token!) but the underlying CSS references a variable which doesn't exist:
--colors-my-semantic-token-typo: var(--colors-invalid);

.text_mySemanticTokenTypo {
  color: var(--colors-my-semantic-token-typo);
}

and I get no warning about this anywhere.
Link to Reproduction
https://play.panda-css.com/gfgL5aFu42
Steps to reproduce
No response
JS Framework
React (TS)
Panda CSS Version
0.27.3
Browser
No response
Operating System

[ ] macOS
[ ] Windows
[ ] Linux

Additional Information
I don't mind where I get an error, I would just like one somewhere so I know I've made a mistake.
At a minimum I think it would make sense for the compiler to report an error if it encounters the {} syntax inside a token which references something that doesn't exist.

regarding type-safety with TS, this is not really possible, see https://github.com/chakra-ui/panda/discussions/1776#discussioncomment-8198659
we will add warnings logs for these


shipping cost calculation

I think we need more flexibility to calculate shipping costs. Here are some ideas takes from past projects:

Country specific shipping costs
Non-linear prices
They should optionally have VAT. The global setting we currently have won't cover all situations as shipping can be done through a company charging you VAT and VAT free delivery
There should be a minimum and maximum amount / weight / dimension per shipping method as not all methods make sense
Additional cost should be absolute or relative (fix amount vs. percentage)

Here's a badly translated screenshot from our old system.
Please note that we've created geo-zones to avoid the necessity to select all the EU countries in various places throughout the store.

 That is a bit ticket item I'm currently working on to make modular in general to plug in different shipping types.
You bring up a good point about maximum weight for a shipping method. It just donned on me that obviously (for US especially) that the post office won't take anything over like 60 pounds or something. At that point, we rely on FedEx/UPS.

Also, here's some additional discussion on my thoughts on setting up shipping #3

@Remo This is nearly there now with the latest push (which you should check out). It's still missing weight restrictions. That said, that will be fairly easy to add.
Check out the FlatRateShippingMethod class, there's two important methods, isEligible() and getRate(). isEligible() is where you can determine your own logic of whether the cart is eligible for the shipping method. This could be anything from the weight of the cart, to the country their in, quantity...whatever.
On my to-do is to add a method to the Cart class to deliver the weight. Though there's enough provided technically that you could figure that out.

@remo can you school me in shipping with VAT? If you're a company that charges VAT on shipping, I assume you do so on all shipping methods. If that's the case, do the tax options available not work? They allow you to charge tax based on subtotal, or subtotal+shipping.

donesies.

@brucewyne I'm by no means a VAT expert, it was actually the topic where I've got my worst grades ;-) One example, the part I believe is the most complex: Since the beginning of the year the EU has changes their law in a way where you must pay the rate of the country of your customer.

This means for example that your Dutch company will need to pay 21% Dutch VAT for the fees collected from your Dutch private customers, 25% Swedish VAT from your Swedish private customers and 19% German VAT from your German customers, etc.
(http://www.foreigninvestments.eu/en/fdi/expert-topics/change-in-european-union-vat-rules-per-1-january-2015-for-telecommunications-broadcasting-and-electronic-services)

It's a bit different in Switzerland, he you either don't add VAT or add 2.5% or 8% depending on the kind of goods/sevices you're offering. Funny thing though is the fact that the VAT law defines the country of Switzerland in a slightly different way than everyone else. That's something I'd ignore for now though as it goes a bit too far - I assume other countries have such strange laws too.

I made it so it was pretty much that flexible and then realized I should probably take it the next step and make Tax Classes. Allowing the ability to group tax rates, and those tax rates could be applied to certain products.

Yes but those tax rates would have to depend on the country you select during the checkout too!

Yes - Each tax rate would allow you to pretty much select when to charge those rates. It's setup as a multiple version of the previous tax rate options.
So each time you add a tax rate, you can choose how you'd like to total the tax rates (pre or post shipping), if the customers shipping or billing address match which country etc etc...
Then the idea is you could group those into a class. Despite whether a product is associated with a certain tax class or not, it would be up to the Tax Rate settings to determine whether to actually charge tax or not.


Get pricepoint information

Is there any way for the server making this call to determine the pricepoint (and currency) at which the customer purchased a given product?

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

ID: 3b95f043-9483-170e-3aef-fa675bfc0fb7
Version Independent ID: 819f2b1e-a55b-7b05-bf54-111eed1a9916
Content: Query for products - UWP app developer
Content Source: windows-apps-src/monetize/query-for-products.md
Service: unspecified
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

I have confirmed with the engineering team that there is currently no way to get the price and currency information for the customer purchase using this API.


Add Croatian Sentiment Classification

Checklist for adding MMTEB dataset

Reason for dataset addition: Cover Croatian in Classfication

[x] I have tested that the dataset runs with the mteb package.
[x] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[x] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[x] intfloat/multilingual-e5-small

[x] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[x] I have considered the size of the dataset and reduced it if it is too big (2048 examples is typically large enough for most tasks)
[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.
[x] I have added points for my submission to the POINTS.md file.

Note to self: Add fix: when merging

@imenelydiaker you're too fast! Beat me to merging main.


Pot snap add option to remove former snapshot

add option to remove former snapshot

Add in 0.10.0
-r replace the oldest available snapshot with the new one


Get your Flask app running locally on your machine

Use the instructions in the README to get the virtual environment and Flask app set up.

Finished!


Can't install in Mac IOS

None of the solutions actually work in IOS. Can someone help, please?

You need to do the following things :

install python2 from Homebrew : brew install python2
install gcc6 from Homebrew : brew install gcc@6
set gcc6 as the compiler : export<EMAIL_ADDRESS>install the package with python2 from Homebrew : python2 -m pip install --no-cache-dir glove_python

@theravengod  I did what you said but now i receive this error:
<EMAIL_ADDRESS>-fno-strict-aliasing -fno-common -dynamic -arch x86_64 -arch i386 -g  -Os -pipe -fno-common -fno-strict-aliasing -fwrapv -DENABLE_DTRACE -DMACOSX -DNDEBUG -Wall -Wstrict-prototypes -W shorten-64-to-32 -DNDEBUG -g -fwrapv -Os -Wall -Wstrict-prototypes -DENABLE_DTRACE -arch i386 -arch x86_64 -pipe - I/System/Library/Frameworks/Python.framework/Versions/2.7/include/python2.7 -c glove/glove_cython.c -o build/temp. macosx-10.13-intel-2.7/glove/glove_cython.o -fopenmp -ffast-math -march=native     g++-6: warning: x86_64 conflicts with i386 (arch flags ignored)     g++-6: error: unrecognized command line option '-Wshorten-64-to-32'     error: command<EMAIL_ADDRESS>failed with exit status 1

I did the following and it worked for me

$ conda install llvm gcc libgcc
$ pip install glove_python

Hi @Binathi, while your specified commands worked in macos (glove_python is now installed), the following command is not working in my python project:
from glove import Corpus, Glove
I tried with "import glove.Glove" but no luck.  The error is either "module 'glove' has no attribute 'Glove'" or "cannot import name 'Corpus'"

I think you mean macOS. If so, you need to do the following things :

install python2 from Homebrew : brew install python2
install gcc6 from Homebrew : brew install gcc@6
set gcc6 as the compiler : export<EMAIL_ADDRESS>install the package with python2 from Homebrew : python2 -m pip install --no-cache-dir glove_python

also could apply to python3.6. Really thank u!

I think you mean macOS. If so, you need to do the following things :

install python2 from Homebrew : brew install python2
install gcc6 from Homebrew : brew install gcc@6
set gcc6 as the compiler : export<EMAIL_ADDRESS>install the package with python2 from Homebrew : python2 -m pip install --no-cache-dir glove_python

also could apply to python3.6. Really thank u!

really also work in python3.6 and gcc(9.3.0), thank u!

I think you mean macOS. If so, you need to do the following things :

install python2 from Homebrew : brew install python2
install gcc6 from Homebrew : brew install gcc@6
set gcc6 as the compiler : export<EMAIL_ADDRESS>install the package with python2 from Homebrew : python2 -m pip install --no-cache-dir glove_python

To anyone who's interested,
This won't work for python version 3.7 and above, switch your version to  < 3.7