[Bug]: Incorrect api operation when working with multi-tenant

What happened?
Errors regarding token creation in multi-tenancy mode
How to reproduce the bug
Hello! I have been unable to cope with numerous errors related to the creation of the token for several days now. Namely, if I try to go to the page with the creation of tokens, I get an error :  The model [Rupadana\ApiService\Models\Token] does not have a relationship named [company]. You can change the relationship being used by passing it to the [ownershipRelationship] argument of the [tenant()] method in configuration. You can change the relationship being used per-resource by setting it as the [$tenantOwnershipRelationshipName] static property on the [Rupadana\ApiService\Resources\TokenResource] resource class.
But I fixed this error by changing the model Token.php and by changing the migration of create_personal_access_tokens . But now I'm facing a mistake. :
SQLSTATE[HY000]: General error: 1364 Field 'tokenable_type' doesn't have a default value INSERT INTO   personal_access_tokens(    name,     token,     abilities,     expires_at,     user_id,     updated_at,     created_at   ) VALUES   (     ssaddsa,     9a9284467652ea6ea8e33584f75de3bbfe764c4582120f0dc7b57b9b346c6bde,     [ "company-contact:create",     "company-contact:update",     "company-contact:delete",     "company-contact:pagination",     "company-contact:detail" ],     ?,     1,     2024 -01 -21 21: 31: 00,     2024 -01 -21 21: 31: 00   )
The redesigned version Token.php:

namespace Rupadana\ApiService\Models;

use...

class Token extends PersonalAccessToken
{
    use HasFactory;

    public function company(): BelongsTo
    {
        return $this->belongsTo(Company::class);
    }
    protected $table = 'personal_access_tokens';
}

api-service.php:

// config for Rupadana/ApiService
return [
    'navigation' => [
        'group' => [
            'token' => 'User',
        ],
    ],
];

Company:php:

namespace App\Models;

use ...

class Company extends Model
{
    protected  $fillable = [
        'user_id',
        'name',
        'short_name',
        'type',
    ];
    public static array $allowedFields = [
        'name'
    ];

    // Which fields can be used to sort the results through the query string
    public static array $allowedSorts = [
        'name',
        'created_at'
    ];

    // Which fields can be used to filter the results through the query string
    public static array $allowedFilters = [
        'name'
    ];
    public function token(): HasMany
    {
        return $this->hasMany(Token::class);
    }
    public function users(): BelongsToMany
    {
        return $this->belongsToMany(User::class);
    }
    //other functions ...

User.php:
<?php

namespace App\Models;

use ...

class User extends Authenticatable implements HasTenants
{
    use HasApiTokens, HasFactory, Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array<int, string>
     */
    protected $fillable = [
        'name',
        'email',
        'password',
    ];

    /**
     * The attributes that should be hidden for serialization.
     *
     * @var array<int, string>
     */
    protected $hidden = [
        'password',
        'remember_token',
    ];

    /**
     * The attributes that should be cast.
     *
     * @var array<string, string>
     */
    protected $casts = [
        'email_verified_at' => 'datetime',
        'password' => 'hashed',
    ];
    public function getTenants(Panel $panel): Collection
    {
        return $this->companies;
    }

    public function companies(): BelongsToMany
    {
        return $this->belongsToMany(Company::class);
    }

    public function canAccessTenant(Model $tenant): bool
    {
        return $this->companies->contains($tenant);
    }

    public function tokens(): HasMany
    {
        return $this->hasMany(Token::class);
    }
}

Package Version
3.0.2
PHP Version
8.1
Laravel Version
10.40
Which operating systems does with happen with?
Windows
Notes
Friends, your help is very important. I tried to contact the official filament discord, but to no avail. I really hope for you.

I never didnt try tenancy, maybe u can use version ^1.0

If its fixed by yourself, please make a PR

OK, friend.Due to my time field, I will address the gaps and try to do a PR in 8 hours.And now I'm going to sleep.😊

OK, friend.Due to my time field, I will address the gaps and try to do a PR in 8 hours.And now I'm going to sleep.😊

I tried to fix the problem, now I'm stuck on the error :

Now the keys are generated and written to the database.
Here is some piece of code where the error is located :
User.php:

namespace App\Models;

// use Illuminate\Contracts\Auth\MustVerifyEmail;
use Filament\Models\Contracts\FilamentUser;
use Filament\Models\Contracts\HasTenants;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Relations\BelongsToMany;
use Illuminate\Database\Eloquent\Relations\HasMany;
use Illuminate\Database\Eloquent\Relations\HasOne;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;
use Laravel\Sanctum\HasApiTokens;
use \Filament\Panel;
use \Illuminate\Database\Eloquent\Model;
use \Illuminate\Support\Collection;
use Rupadana\ApiService\Models\Token;

class User extends Authenticatable implements HasTenants
{
    use HasApiTokens, HasFactory, Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array<int, string>
     */
    protected $fillable = [
        'name',
        'email',
        'password',
    ];

    /**
     * The attributes that should be hidden for serialization.
     *
     * @var array<int, string>
     */
    protected $hidden = [
        'password',
        'remember_token',
    ];

    /**
     * The attributes that should be cast.
     *
     * @var array<string, string>
     */
    protected $casts = [
        'email_verified_at' => 'datetime',
        'password' => 'hashed',
    ];
    public static array $allowedFields = [
        'name'
    ];

    public static array $allowedSorts = [
        'name',
        'created_at'
    ];

    public static array $allowedFilters = [
        'name'
    ];

    public function companyContacts(): HasMany
    {
        return $this->hasMany(CompanyContact::class);
    }
    public function getTenants(Panel $panel): Collection
    {
        return $this->companies;
    }

    public function companies(): BelongsToMany
    {
        return $this->belongsToMany(Company::class);
    }

    public function canAccessTenant(Model $tenant): bool
    {
        return $this->companies->contains($tenant);
    }

    public function tokens(): HasMany
    {
        return $this->hasMany(Token::class);
    }
}

CompanyContact.php:
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Illuminate\Database\Eloquent\SoftDeletes;

class CompanyContact extends Model
{
    protected $fillable =
        [
            'address',
            'phone',
            'email'

        ];

    public function company() : BelongsTo
    {
        return $this->belongsTo(Company::class);
    }

    public function user(): BelongsTo
    {
        return $this->belongsTo(User::class);
    }
    use HasFactory, SoftDeletes;
}

Token.php:
<?php

namespace Rupadana\ApiService\Models;

use App\Models\Company;
use App\Models\User;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Laravel\Sanctum\PersonalAccessToken;

class Token extends PersonalAccessToken
{
    use HasFactory;

    public function user(): BelongsTo
    {
        return $this->belongsTo(User::class);
    }
    public function company(): BelongsTo
    {
        return $this->belongsTo(Company::class);
    }
    public static ?string $tenantOwnershipRelationshipName = 'company';
    protected $table = 'personal_access_tokens';
}

CompanyContactResource/Api/Handlers/PaginationHandler.php:
namespace App\Filament\Resources\CompanyContactResource\Api\Handlers;

use App\Models\CompanyContact;
use Illuminate\Http\Request;
use Rupadana\ApiService\Http\Handlers;
use Spatie\QueryBuilder\QueryBuilder;
use App\Filament\Resources\CompanyContactResource;

class PaginationHandler extends Handlers {
    public static string | null $uri = '/';
    public static string | null $resource = CompanyContactResource::class;

    public function handler()
    {
        $model = static::getModel();

        $query = QueryBuilder::for($model)
        ->allowedFields($model::$allowedFields ?? [])
        ->allowedSorts($model::$allowedSorts ?? [])
        ->allowedFilters($model::$allowedFilters ?? [])
        ->paginate(request()->query('per_page'))
        ->appends(request()->query());

        return static::getApiTransformer()::collection($query);
    }
}

I really need your help

Mate, please try to remove this method in your Users table
public function tokens(): HasMany
{
return $this->hasMany(Token::class);
}


Upgrade to rails 6

Main changes needed for this included removing the jasmine-rails gem, using libsass, and removing auto_load config.

I'd suggest two minor changes:

As per RailsDiff, config/puma.rb could be updated to replace:
threads_count = ENV.fetch("RAILS_MAX_THREADS") { 5 } threads threads_count, threads_count
with
max_threads_count = ENV.fetch("RAILS_MAX_THREADS") { 5 } min_threads_count = ENV.fetch("RAILS_MIN_THREADS") { max_threads_count } threads min_threads_count, max_threads_count

There is a paragraph in the README about Cucumber and the bundler version which will be out of date once the updates in this PR are applied.

Hey @cdccollins this looks great, I've compared the pr changes with railsdiff and there are some changes that didn't happen with bundle exec rails app:update that could also be done:

Remove bin/bundle, bin/update
Update system! "bin/rails db:setup" with system! "bin/rails db:prepare"
In the config/environments/test.rb the config.cache_classes = true has changed to false and there is config.action_view.cache_template_loading = true
Remove config/initializers/new_framework_defaults.rb, config/initializers/new_framework_defaults_5_1.rb as per the reason in alphagov/travel-advice-publisher@8b0745e. I also think we can remove , config/initializers/rails_3_4_flash_compatibility.rb`
Update app/views/layouts/application.html.erbto include csp_meta_tag after csrf_meta_tags. Maybe it will be good to have a csrf_meta_tags, csp_meta_tag, stylesheet_link_tag and then javascript_include_tag
In the config/spring.rb the %w[] needs to be updated to Spring.watch()

Spring.watch(
  ".ruby-version",
  ".rbenv-vars",
  "tmp/restart.txt",
  "tmp/caching-dev.txt"
)


model/labels: Update exp/slices usage

It seems like they broke that experimental package's API in order to now be compatible to what the Go 1.21 built-in slices package uses.
This change updates the usage of the broken slices.SortFunc.
Once updating Prometheus to Go 1.21 as minimum the exp/slices can be replaced with the stdlib version and shouldn't break then.

Oops, should have checked for other PRs first. There is already https://github.com/prometheus/prometheus/pull/12650


terrascan: 1.13.2 -> 1.14.0

Description of changes
https://github.com/accurics/terrascan/releases/tag/v1.14.0

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package built:

terrascan


Cheats Folder path on Windows

Hi, Awesome plugin you have here, really usefull, thanks for creating it!.
I'm using Windows 10 insider's preview and cerebro-cheats didn't work at first, so this is what I did:
1- install cerebro-cheats
2- type cheats
3- cerebro shows autocomplete
4- nothing else, "open cheats directory" didn't show.
5- come read readme.md to search for folder path
6- no information on windows
7- look at the code
8- find: const CHEATS_DIR = path.join(remote.app.getPath('userData'), 'plugins', 'cerebro-cheats', 'cheats');
9- try: remote.app.getPath('userData')
10-find dir
The path that cerebro-cheats is looking for is:
C:\Users<user>\AppData\Roaming\Cerebro\plugins\cerebro-cheats\cheats
but it is installed in:
C:\Users<user>\AppData\Roaming\Cerebro\plugins\node_modules\cerebro-cheats
I don't know if this is intended.
When I created the first folder path the "open cheats directory" started to appear and everything is working fine.
Just reporting.

Hello @Sniperkillerut. Glad you found this plugin useful and thanks for reporting the issue.
I hadn´t the opportunity to test this in Windows, since I work mostly on Linux.
Ite seems I got the cerebro plugins path wrong. Still, the plugin should have created the cheats directory even if its not within Cerebro plugins folder. Maybe some permissions issue on Windows I will fix the plugin to use the correct path when I have some free time.
Thank you.


[Caloris] [Github Code] Tar download and processing

Periodic tar download and processing
Apply limits: 1GB / repo, 1mb / file
Pretty strict blacklist, maybe even whitelist
See https://github.com/dust-tt/dust/issues/2324

Run daily per workspace from temporal. One activity per repository.


SMTP: refactor and accept starttls :always and :auto

This PR is a refactored version of https://github.com/mikel/mail/pull/1515 (which is a master-rebased version of https://github.com/mikel/mail/pull/1435) and adds :always and :auto as acceptable values for enable_starttls.
While the original PR 1515 fixes the disabling of starttls, it also yields unexpected/faulty 'shortcuts' in the if-chain given some inputs, e.g.:

ssl: false ensures anyenable_starttls-flag is ignored.
enable_starttls: false would ignore the enable_starttls_auto-flag.

I refactored the logic to come to the right starttls value for smtp (i.e. :always, :auto or false). It also adds the option to directly pass :always or :auto as value for enable_starttls (separate commit).
The implementation is based on this decision table:
| tls/ssl   | enable_starttls | enable_starttls_auto | result                                     |
|-----------+-----------------+----------------------+--------------------------------------------|
| true      | *               | *                    | {smtp_tls?: true, smtp_starttls: false}^1  |
| - / false | -               | -                    | {smtp_tls?: false, smtp_starttls: :auto}   |
| - / false | - / false       | true                 | {smtp_tls?: false, smtp_starttls: :auto}   |
| - / false | true            | *                    | {smtp_tls?: false, smtp_starttls: :always} |
| - / false | - / false       | false                | {smtp_tls?: false, smtp_starttls: false}   |
| - / false | false           | -                    | {smtp_tls?: false, smtp_starttls: false}   |
| - / false | :always         | *                    | {smtp_tls?: false, smtp_starttls: :always} |
| - / false | :auto           | *                    | {smtp_tls?: false, smtp_starttls: :auto}   |

* = any value
- = not provided (i.e. key absent or nil-value)

^1: raise when truthy value for any enable_starttls* flag is provided (similar to IMAP).

Hey there @ahorek  and @eval does this also replace https://github.com/mikel/mail/pull/1526 ?

with old rubies (net/smtp), both options were disabled by default, so historically it used to make sense to enable them separately
enable_starttls: true
enable_starttls_auto: true

but since starttls is now :auto by default due to the upstream change in net/smtp, it's problematic to do the opposite and disable them with existing options
what about introducing a new setting, that accepts the same values?
starttls: [false, :auto, :always]

if enable_starttls or enable_starttls_auto are passed - raise an error with a hint of what the new setting is
this is already a breaking change, so I suggest simplifying it if we have a chance. @mikel @eval What do you think?

Hey there @ahorek and @eval does this also replace https://github.com/mikel/mail/pull/1526 ?

yes, but the related PR should still be merged for 2.8.x

@mikel @eval @ahorek Do you have an ETA for a release that includes this? Thank you!

@ron-shinall I'm not a maintainer.
only @mikel or @jeremy can push a new version, but none of them are active lately

@jeremy Would you mind releasing a new version with this fix? We're seeing a number of users not able to disable STARTTLS without this change.

@stanhu fyi it has already been released in 2.8.1

the main problem was starttls cannot be disabled with any settings, this should already be fixed.
this refactor also fixes a combination of
tls: false + enable_starttls: true (should ignore starttls)
also enable_starttls: true is now enabled by default (in net/smtp) and a simple tls: false doesn't disable it
just pass tls: false + enable_starttls: false that should do what you want. I agree there's still inconsistency, but a workaround is pretty simple.

a workaround is pretty simple.

Yes, the workaround is simple in principle, but we have potentially thousands of users that have set settings[:tls] and settings[:ssl] to false. We ship mail v2.8.1 now, but having to comment out existing settings that worked is not a good user experience. We will backport this patch for now.

well, this PR is merged for 2.9, but it isn't released. If that's important for you, you should open a new issue and ask maintainers if they're willing to backport it.


hello.js ,function Hello as controller does not work

<html ng-app="demo">

angular.module('demo', [])
.controller('Hello', function($scope, $http) {
    $http.get('http://rest-service.guides.spring.io/greeting').
        success(function(data) {
            $scope.greeting = data;
        });
});

Where's your ng-controller?

I mean that hello.js
function Hello($scope, $http) {     $http.get('http://rest-service.guides.spring.io/greeting').         success(function(data) {             $scope.greeting = data;         }); }
does not work and gets an error,when  should change to
angular.module('demo', []) .controller('Hello', function($scope, $http) {     $http.get('http://rest-service.guides.spring.io/greeting').         success(function(data) {             $scope.greeting = data;         }); });
and in index.html add ng-app="demo"

I know, but you haven't shown your whole index.html. It has to have an element with ng-controller="Hello" for your change to do anything useful.

sorry for delaying.
the index.html
<!doctype html>

Hello AngularJS

The ID is {{greeting.id}}
The content is {{greeting.content}}

the hello.js
angular.module('demo', [])
.controller('Hello', function($scope, $http) {
$http.get('http://rest-service.guides.spring.io/greeting').
success(function(data) {
$scope.greeting = data;
});
});
2016-09-04 1:43 GMT+08:00 Dave Syer<EMAIL_ADDRESS>
I know, but you haven't shown your whole index.html. It has to have an
element with ng-controller="Hello" for your change to do anything useful.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spring-guides/gs-consuming-rest-angularjs/issues/6#issuecomment-244559940,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APlm4fR5FZJazeQOHOXf2RUvqv8JPnzpks5qmbGwgaJpZM4J0Rs_
.

Works for me. What happens when you load the app in a browser?

if hello.js like that
function Hello($scope, $http) {     $http.get('http://rest-service.guides.spring.io/greeting').         success(function(data) {             $scope.greeting = data;         }); }
then loading  the app in a browser gets error
angular.js:12330Error: [ng:areq] http://errors.angularjs.org/1.4.3/ng/areq?p0=Hello&p1=not%20a%20function%2C%20got%20undefined     at Error (native)     at https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js:6:416     at Sb (https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js:22:18)     at Qa (https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js:22:105)     at https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js:79:497     at x (https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js:59:501)     at S (https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js:60:341)     at g (https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js:54:384)     at g (https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js:54:407)     at g (https://ajax.googleapis.com/ajax/libs/angularjs/1.4.3/angular.min.js:54:407)(anonymous function) @ angular.js:12330

I finally understood your issue (please try to be more explicit if you submit any more). Something changed in AngularJS since this guide was published (anonymous apps with controllers as functions apparently used to be a thing and they are no longer). We updated the Angular version but didn't check that it works, I think.
Should be fixed now.


Build Error in Mac OS

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
No
Source
source
Tensorflow Version
2.13
Custom Code
No
OS Platform and Distribution
macOS Monterey
Mobile device
No response
Python version
3.9
Bazel version
5.3.0
GCC/Compiler version
Apple clang version 13.1.6 (clang-13<IP_ADDRESS>.5)
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Running
`pip install /tmp/tensorflow_pkg/tensorflow-2.13.0-cp38-cp38-macosx_12_0_x86_64.whl`

Leads to

`tensorflow-2.13.0-cp38-cp38-macosx_12_0_x86_64.whl is not a supported wheel on this platform`

Built the code from source using https://www.tensorflow.org/install/source

Standalone code to reproduce the issue
No Code

Relevant log output
No response

Hey @synandi
I was actually trying to build from source with the intention of contributing to tensorflow. While I was not able to build the mac binary, I got around it by spinning up a  linux docker container, mounting my local directory there and building linux binary. Thanks for the response though :)
Closing the issue as I think mac binaries are unstable


P- Poly Resistors

I am experiencing a number of errors with P- poly resistors that are generated in Magic with the sky130A generators.
These issues seems to be caused by the precision resistors not being correctly detected and applying the correct rules.
I have attached and example GDS to demonstrate this.
res.gds.zip

Thank you for the test case ! I solve it.


Add credential provider support

Adds an abstract credential provider class that can be used to programmatically supply credentials at
authentication time, instead of just a static username and password.

Closes #246

@alisaifee Thanks for the review! Let me know if there are any other changes you'd like me to make.

@alisaifee Thanks for the review! Let me know if there are any other changes you'd like me to make.

Wow, amazing turnaround to address the suggestion! Thank you! Looks great to me. I wasn't able to verify the rendering of the docs since I've broken my local sphinx environment but that's a minor detail we can iron out in master.
Regarding the typing issues in CI with respect to boto - feel free to add it to the ignore list.

@alisaifee Hopefully that's the correct place to ignore them. Let me know if it's not.

@alisaifee Hopefully that's the correct place to ignore them. Let me know if it's not.

It is indeed. Needed a few tweaks to get past mypy. Since it's a recipe it's fine to be a bit relaxed on the type strictness IMO.

@trentbitterman unless you have anything else to add, I'll probably merge this once CI is done.

That works for me, I didn’t have any other changes planned. Thanks!


BUG: --config should be a flakeheaven global argument, not just a flakeheaven lint one

Checklist

[x] Same issue occurs when reproducing the MWE below: new virtualenv, clean folder, minimal additional libs, etc.
[x] Same issue occurs when running without cache (eg by setting the env var FLAKEHEAVEN_CACHE_TIMEOUT=0)

Bug Description

$ flakeheaven plugins --config=flakeheaven-base.toml
NAME                          | VERSION  | CODES            | RULES
flake8-bugbear                | 22.9.23  | B                | +*, -W503
flake8-class-attributes-order | 0.1.3    | CCE              | +*, -W503
flake8-docstrings             | 1.6.0    | D                | +*
flake8-type-checking          | 2.1.3    | TC               | +*, -W503
mccabe                        | 0.6.1    | C90              | +*
pycodestyle                   | 2.8.0    | E, W             | +*
pyflakes                      | 2.4.0    | F                |
pylint                        | 2.15.2   | C, E, F, I, R, W |
$ flakeheaven plugins -h
NAME                          | VERSION  | CODES            | RULES
flake8-bugbear                | 22.9.23  | B                | +*, -W503
flake8-class-attributes-order | 0.1.3    | CCE              | +*, -W503
flake8-docstrings             | 1.6.0    | D                | +*
flake8-type-checking          | 2.1.3    | TC               | +*, -W503
mccabe                        | 0.6.1    | C90              | +*
pycodestyle                   | 2.8.0    | E, W             | +*
pyflakes                      | 2.4.0    | F                |
pylint                        | 2.15.2   | C, E, F, I, R, W |

[tool.flakeheaven]
# Make output nice
format = "grouped"
show_source = true

# Black compatible settings
max_line_length = 120
extend_ignore = ["E203"]

# Suppress exceptions from plugin (the exception will be converted into E902 error)
safe = true

extend_exclude = [
    ".venv",
]

# list of plugins and rules for them
[tool.flakeheaven.plugins]
"flake8-*" = ["+*", "-W503"]
mccabe = ["+*"]
pycodestyle = ["+*"]

and
[tool.flakeheaven]
base = "flakeheaven-base.toml"

[tool.flakeheaven.plugins]
flake8-docstrings = ["+*"]

Expected behavior

Bug Repro

OS, arch
python --version
pyproject.toml (and / or all other config files that apply, eg .pylintrc, stc)
pip freeze
flakeheaven plugins
Contents of any additional files (eg sample .py file(s) to be linted)
Any additional env vars setup
[If available] contents of flakeheaven config -vvv

Steps to reproduce the behavior:

run flakeheaven ...
...

Additional context

Source: https://discord.com/channels/927757025193099375/927757025193099378/threads/1027501357080530954

because of the ways the flakeheaven commands are dispatched, this has to be tracked down and implemented for every command
tests should be added to ensure new commands also respect this. For example, a test with a dummy variable in the main config whould be available after command dispatching and app initialization, but before command execution.

because of the ways the flakeheaven commands are dispatched, this has to be tracked down and implemented for every command

😭


RFP-3: Chain-Key Signatures

Overview
The Internet Computer implements a novel threshold ECDSA (tECDSA) protocol to produce chain-key signatures. In this protocol, the private ECDSA key is held in a secret-shared manner by multiple parties, namely the replicas of a threshold-ECDSA-enabled subnet on the IC, and signatures are computed using those secret shares without the private key ever being reconstructed. Each replica of such subnet holds a key share that provides no information about the key on its own, and at least one-third of the replicas are required to generate a threshold signature using their respective key shares. This allows each canister on any subnet of the Internet Computer to control a unique ECDSA key pair and request signatures.
ECDSA signatures are widely used within and outside the Blockchain ecosystem. Platforms/Ecosystems like Ethereum, Cosmos, Filecoin, or Polkadot among many others support ECDSA (on the same curve secp256k1) for signing transactions.
Furthermore, JSON Web Tokens (JWTs) support ECDSA and are widely used for authentication and authorization. More recent standards such as Verifiable Credentials which are heavily used in the Self-Sovereign Identity (SSI) ecosystem can also be used with ECDSA signatures.
Lastly, Public Key Infrastructure (PKI) based on Certificate Authorities and x.509 certificates support ECDSA. However, the Web PKI underlying HTTPS and TLS uses another curve and is not directly compatible today.
Areas for Proposals
Novel applications and infrastructure
Integrations with other Blockchains
Chain-key signatures allow a canister to effect actions on other blockchains. The signed transactions can be either submitted using HTTPS outcalls or introducing (incentivized) relayers.
JWT AuthZ canister
Chain-key signatures allow canisters to issue JWTs. This allows the Internet Computer to be used as an authorization server for microservices and other resource servers outside the IC.
Decentralized Certificate Authority
Chain-key signatures allow canisters to issue x.509 certificates used in Public Key Infrastructures (PKIs). Hence, a canister can serve the role of a decentralized certificate authority. An interesting project would be to investigate if a canister using chain-key signatures and HTTP outcalls, potentially using a custom gateway, could serve as an ACME server similar to Let's encrypt.
Your unique idea
Of course, we are also excited to support you in the implementation of novel use cases that we haven't thought of.
Libraries and Tooling

Libraries to simplify the issuance of JWTs, Verifiable Credentials, and x.509 certificates in Motoko

Tutorials and well-documented Sample Applications
Tutorials in written and/or video form as well as well-documented example applications help new developers get started.
How to apply?
Please submit your application at https://dfinity.org/grants
References

Developer docs
Sample app
Specification of ecdsa_public_key and sign_with_ecdsa

@domwoe , wanted to check whether this RFP is still open. The idea of hosting an ACME server over Internet Computer looks interesting and promising to me. Would be coming up with a proposal soon :)

@Rishabh0712 Looking forward to your proposal.


Like Reactions

@msftdherron @jotrick @goelashish7 @MattSFT @aosolis @jialic
Added Like Reactions feature in NodeJs Sample App.

Like Reaction Feature in Sample App


Drop-down menu for clip length

For times, when one wants to deviate from the personal default clip length (which I suggested in another comment), it would be less fiddly if the clip length could be changed numerically via an integer or float input field or a drop-down menu or the like.

Will be available in next version!


README tweaks

Slight messaging / presentation tweaks in preparation for launch, since this will be what people see when they look up lit on npm.

Removed version number from heading (I couldn't find any comparable packages that included a version number)
Grabbed the more official tagline and 1-phrase description from the monorepo
Lightly edited intro text to de-emphasize the identity of lit-html as a separate thing, though resisted the temptation to make more edits further down for now

Can we come up with an alternative to "leads the pack", both here and on lit.dev? It seems like a bit too marketing-y, and it's not obvious whether Lit leads, compared to which library, and measured how. It would be better not to invite comparisons and disputes about this. Better would be to just communicate small size, fast, expressive.

Oh I see! I didn't parse it like that. How about "At Lit's core"?

Can we come up with an alternative to "leads the pack", both here and on lit.dev? It seems like a bit too marketing-y, and it's not obvious whether Lit leads, compared to which library, and measured how. It would be better not to invite comparisons and disputes about this. Better would be to just communicate small size, fast, expressive.

Sure, I'll add a commit to make these changes.

Will make a separate PR to tweak the same wording on lit.dev.


base64js.toByteArray is spoliling the actual value

when a base64 encoded string is input to this function it is producing a Arraybuffer which internally changes its original value. If we again convert the array buffer to base64 then a different base64 is generated. Hence When i am trying to implement signature verification for a hybrid app, then the digest value is getting differ and hence a mismatch between front-end and back-end.
Need an urgent help please. We are stuck at client project delivery. Earnest request.

It's not possible to debug this issue without additional information. If this is still an issue, please leave a comment or open a new issue.


Quick fix for problem with move_summary

This fixes a problem with MoveScheme.move_summary that @hejung found. I had used is in a case where I should have used == (to use UUID to check that the mover is the same). Was only a problem in some edge case (not fully identified, since it depended on the scheme) where the scheme was already in memory. For some reason, movers loaded from storage were re-instantiated (@jhprinz, any idea how that might have happened)? This meant that the mover in the change will be == to the one in the scheme, but not is.

Hmm, I can only imagine that there is a problem in the recreation of the movescheme that it will recreate part of it and hence these objects will be recreated and be identical. Not sure why == would work in that case if UUID is used. This should be different as well. I will have a look when I have some time.

also, if one of the object is a proxy then == will work, but not is

I will have a look when I have some time.

Might require the example from @hejung. It worked fine with one move scheme, but not with another. No idea why that was.

also, if one of the object is a proxy then == will work, but not is

These are PathMovers, so there shouldn't be any proxies. But the general point is important, and the reason that this should be == instead of is anyway.
In any case, I'll go ahead and merge this. It sounds like @jhprinz has seen enough to approach the (2-character) changes!

My example can be found in https://gitlab.e-cam2020.eu:10443/hejung/sr_shooter/blob/master/examples/OneWayShooting_vs_TwoWayShooting.ipynb
if i use scheme.move_summary(steps) for the twoway shooting scheme that still lives in memory on the storage that it used to write its data to it works.
if i use scheme.move_summary(steps) for the same object that still lives but on the newly created analysis storage (just reopened the store i closed before) it does not work.
if i also load the scheme from that reopened storage it works.


Connection lost on Docker container

We're running a ASP.NET Core 2.1 Web API which proxies outbound notifications to Azure SignalR, running on a Docker Container on AKS.
Sometimes throughout the day, we get the following exception:
fail: Microsoft.AspNetCore.Http.Connections.Client.Internal.WebSocketsTransport[15]
      Error while sending a message.
System.Net.WebSockets.WebSocketException (0x80004005): The remote party closed the WebSocket connection without completi
ng the close handshake. ---> System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values

And then the only way to recover is to kill the container and restart it.
Is there a way to make the HubContext recover and reconnect automatically?

Same issue here.
Anyone can help?

What is your version of the SDK?
The latest SDK changes the behavior to always reconnect https://github.com/Azure/azure-signalr/releases/tag/v1.0.3
Please have a try and let us know if there are any issues/concerns/suggestions

@vicancy Thanks, we'll look into take and update this issue ASAP

Looks like the latest release was at least beneficial, since we are not noticing so many exceptions.


New error: {'is_ctv': ['Unknown field.']}

Hello, in this week this error is appearing:
{'is_ctv': ['Unknown field.']}

I'm "bypassing" it using the data value in this error.

Created a PR: https://github.com/sann05/python-adjust/pull/3


Better handling of Huawei phones

A few people have reported a strange problem on Huawei phones, where the app initialization pauses a few seconds after it connects to the car. They see the car app in a weird state, mostly around the Music app: Spotify icon is showing but unresponsive, the other music AM apps never come in, but then everything resumes initializing when they unlock the phone and open the phone app #355
It might be possible to detect this situation using a Handler loop to watch for timing differences and show a hint recommending that they change their battery saving settings. It might also be appropriate to claim a Wake Lock while MainService is running and showing its Foreground Notification, so that Huawei doesn't kill it.

ive just come on to say, ive got a weird bug with the 17sept nightly build where AAID connect and spotify shows up, but no song information or music plays.
i thought it was to do with my battery settings but the app launch mentioned before is unchanged from the previous issue i had.
Cheers


Write in future work section about accumulative propagation

How to detect the need to learn new fix utility model (FUM) versus new failure propagation model (FPM)? In the FPM, mRubis will return a negative reward for a wrong fix. How many unsuccessful fixes (or some negative cumulative reward) from the same Agent until that Agent should learn a new FPM? Concerning the FUM, how much discrepancy between the predicted and actual utility would warrant the need to learning a new utilitty model for a given Shop? There must be a trade-off between impact of prediction errors and the cost of training.


Branch 0.6.2

Coverage remained the same at 100.0% when pulling 2a9e8a7236ae0501e315afabb35d174c67b75028 on branch-0.6.2 into 177b46410b4ad6ea9a0c444f03dbf0ad42c6262d on master.


Bug: Parameters field in the PathItem is ignored

Steps To Reproduce
https://himenon.github.io/openapi-typescript-code-generator-playground/v0.8.0/index.html?code=PYBwpgdghiCWBcACAzAOgAyuQKFhAZsPNoogG5gBOAzrMBEgIwYYmIAus7ANmEgCIBRALIB5NgBMw1AMaVYITvSRSAtsETVwMtt1gzI1Pm1LRVfRMICSAFWzYjlCjWKkAtIgCulbkgBEABbs7CDwAPRhUmSoMLCo7NLsqBLAqlB4YX4miFKy8op0DIj8YBTcoOYQ7IiCEGSwlPSV7Gwe3r6IgcEg1OFhsfGJyanpEJnZuXIKSkUACo0SnjIzNXUNTZAt2DKpIPSbvWyyAWBph6SkAIKe7AHAlK4XpOwAnuBIwABGAFZgy9mkShgACOngaYAkjye7kQsAkAIuHjMYARIEa4EonGkUOhcJx0I4bws1HY8ggAHMEaYoOZ8dDXu9NKS8JSCU9JvkZkgoDc7pREMjsgAhYDAADWdIZFi%2Bv3%2BBI500KSBF4sswCk3ARQNB4MhVMQHk4PBRbINiB5t3u%2Bo8VgAykKAHLWxAgTyfPTUAKXFqmtogCRQBISb2o9FULHnNlG3h0p5SpAksmstkWvmxi4AEiB%2BH8AGIwjtVHsIAcwsdTlBqGFrpbKFlTXbHennkSE8yKfrXe7YJ7vc3CYzEyz9aRCJQ0uwVIGTWzPP7p8HJyOB8T28nTWOJ1OEmwgVp6EZI6QVRKEQqCspEABxMDVE8InZVTbpmAgPQyQOFMLfajKZfltJ%2B1ILMwBzTp80LYtSwAyswhPLJ7BAQMAkjMJyVvMJPlFMUqwAbzhABfHEkMoGlbyoI8zWRJA4QRPAkCQ24tRBMEgUhDhKE8Gd2WkKYLyKE9ECsfgERg9N4yZJN9U3QMkE8MF4SedClwJUAqE-egrHY5STyFF4tOY-cIEPdMACZ0HQfsQLAvwIN2fYqirPdi0PODsOoLIgA
The current behavior
The parameters field contained in the Path Item Object is ignored.
The expected behavior
Processed in the same as the parameters included in the Operation Object.

Playground

Before
paths:
  /get/books/{id}:
    parameters:
    - name: id
      in: path
      required: true
      description: Book ID
      schema:
        type: string
        format: uuid
    get:
      operationId: getBookById
      responses:
        200:
          $ref: "#/components/responses/Books"

After
Since parameters is an Operation Object, it seems that it belongs to Paths differently.
paths:
  /get/books/{id}:
    get:
      operationId: getBookById
      parameters: // <----- here
        - name: id
          in: path
          required: true
          description: Book ID
          schema:
            type: string
            format: uuid
      responses:
        200:
          $ref: "#/components/responses/Books"

A list of parameters that are applicable for this operation. If a parameter is already defined at the Path Item, the new definition will override it but can never remove it. The list MUST NOT include duplicated parameters. A unique parameter is defined by a combination of a name and location. The list can use the Reference Object to link to parameters that are defined at the OpenAPI Object's components/parameters.

https://github.com/Himenon/openapi-typescript-code-generator/issues/47#issuecomment-827387401 is not wrong, but the implementation just doesn't seem to be faithful.

fixed: v0.9.4 higher version
Thanks !


zeit-token is still required

Is this intentional?

I also am seeing this, pretty sure it's not.

Use v25, not v20. Doc needs updating.


enable master_port selecting for DeepSpeed and MPI

What does this PR do?
This PR adds a parameter to set master_port for DeepSpeed and MPI, which is motivated by running multiple DeepSpeed tasks simultaneously with each task using various parts of the HPUs.

The code quality check failed, can you run the following from the root of the repo please?
pip install --upgrade ruff
make style

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Add workflow step labeler

Once a work package is done from the contributors side, issues need to be marked with a specialized label.
Maybe assignees need to be removed and/or added.
In general this needs to be configurable.

Addressed in 7ee5d938bbc8af0cd0c9720c93562f8802f7ddcb

Closed with Release v0.1.14


[Bug]: Minio API URL is not correctly configured

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Hi,
upon installing minio, I set the console url to https://console.domain.tld and the API url to https://api.domain.tld
When I try to login from the console gui, I get this error: S3 API Requests must be made to API port.
Looking at the logs inside coolify I see that the API's URL is wrong:
MinIO Object Storage Server
Console: https://console.domain.tld
API: https://console.domain.tld
Status: 1 Online, 0 Offline.
Documentation: https://docs.min.io

It seems like the API url setting is not properly passed during the install process.
Steps To Reproduce

add new minio service
set console and api url
start the service
try to connect from the console url

Version
3.10.4

The next version will fix this!


[bug]: download_with_resume occasionally fails to find filename

Is there an existing issue for this?

#5032 has the same traceback. But this happens when using the model manager from the web interface, not just at install-time, and it occurs with current URLs so it's not an issue of old links to no-longer-available models.

OS
Linux
GPU
cuda
VRAM
12GB
What version did you experience this issue on?
1c7ea57492802eaf19f81ada9ed545a2e618add8
What happened?
When installing a model from a civitai download URL such as
https://civitai.com/api/download/models/218017?type=Model&format=SafeTensor&size=pruned&fp=fp16

I sometimes get this error:
  File "invokeai/backend/install/model_install_backend.py", line 326, in _install_url
    location = download_with_resume(url, Path(staging))
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "invokeai/backend/util/util.py", line 290, in download_with_resume
    file_name = re.search('filename="(.+)"', resp.headers.get("Content-Disposition")).group(1)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "python3.11/re/__init__.py", line 176, in search
    return _compile(pattern, flags).search(string)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: expected string or bytes-like object, got 'NoneType'

but when I tried to run this in the debugger to see what HTTP headers it was getting to see why it might fail to parse them, the problem disappeared?
dunno if it's a race condition, or some rate-limiting that the debugger slowed down enough to pass through, or what.
waiting a few minutes and trying again didn't clear it up.

some thoughts about download_with_resume https://github.com/invoke-ai/InvokeAI/blob/1c7ea57492802eaf19f81ada9ed545a2e618add8/invokeai/backend/util/util.py#L285-L293:

how does stream=True work with headers? It doesn't seem to be defined as an async function. Is it guaranteed to have headers by the time requests.get returns? Or does it block when you access request.headers until the headers are complete?

docs say headers are expected to always be there and that streaming only applies to the body content: https://requests.readthedocs.io/en/latest/user/advanced/#body-content-workflow

why is there an except AttributeError here? Well-defined interfaces shouldn't occasionally run in to AttributeErrors in production.
maybe we should check the response status code before looking for the filename?

maybe we should check the response status code before looking for the filename?

This would make sense to me as a quick solution - the issue with the LoWRA URL was that it returned a 404


curl: command not found

The multirust install failed for me with:
==> default: /tmp/vagrant-shell: line 7: curl: command not found
==> default: /tmp/vagrant-shell: line 8: multirust: command not found

Maybe explicitly apt-geting curl would be a good call?

I met the same problem. #6 fixes this (:

yup! hoping it gets merged (:


Pin ACK runtime to v0.34.0

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Allow PHP 8.4

I don’t get it why this library forcefully disallows installation on higher / unsupported PHP versions.
It’s one of the few dependencies that I know that do this.
Can we please relax this so that people can start running their CI on PHP 8.4 already to spot issues and report / fix them here.

Handled in https://github.com/moneyphp/money/pull/796


Compatibility with RustCrypto

Hi!
I'm currently working on an implementation of the Balloon hashing algorithm for the RustCrypto organization: RustCrypto/password-hashes#232. In an effort to make this viable, I am seeking to provide compatibility between different implementations.
Only two changes are required to provide compatibility:

Change encoding of values to little-endian as is done in the prototype implementation
The ints_to_block function was interpreted as hashing these values and produce a "block" of bytes; the prototype implementation uses some sort of encryption here, I merely replaced it with the hash already used

Any input is welcome! I added test-vectors to assure compatibility. I am willing to contribute a CI setup, just say the word!
I have never programmed in Go before!
Another library I provided compatibility for: nachonavarro/balloon-hashing#2
Addresses #1.

Hi!
Thanks for your pull request, looks good to me!
I will only replace the slice inversion via more straightforward function - there should be one in math.big.


Add 'Pass Through Input' option to all nodes

These changes add the Pass Through Input option to the 9 remaining nodes (all except object-detector).
Changes include adding the functionality, the checkbox in the node UI, and documentation in the sidebar.

The missing label has been added, but the bounding boxes will come in a later PR.


FSDP meta-init for full and LoRA finetuning

I'm resubmitting the previously rejected changes in #351.
Makes the scripts compatible to run with meta-device init in FSDP via fabric.init_module(empty_init=True). Also enables fabric.load_raw() which uses lazy-loading internally. The combined changes make it possible to initialize very large models and load the checkpoint in an efficient way without using up all available CPU RAM.
I tested convergence on both scripts.

I can confirm that full finetuning with these changes and Llama 2 70B does not cause CPU OOM.

That's the idea yes. This should help single-GPU fintuners as well as FSDP users.

Is it possible to apply this to evaluate and generate?


Refered column creation more visible

Hello
By doing user tests, we realized that there is a not clear behavior. Users often don't understand / see that if a referenced data doesn't exist it must be explicitly created.

Instand of use the + button they click on Enter and not understand why the field is red, it is not clear that the referenced data have not been created

Suggestions :

We can bring the button to top of list (risk : the user could create a new referenced data before check if already exists)
Remove element not corresponding to search entry (so the + button will remain the only one visible)
Consider enter as a creation (maybe it's risky if there is a miss click, dirty data can be created)

I think I prefer the second suggestion. Want do you think ?

FWIW my vote goes to option number 2 though I suspect that the way reference fields search works is non-trivial 🤔

I agree that option 2 sounds best of the ones listed (to show only the "+" item when no other choices match).
Another option (not mutually exclusive) is to pop up a dialog when a user hits Enter on a value that doesn't match anything. It could offer the choices of

Create new record in Table1 with Field Name of "CMS New", and add a reference to it.
Store "CMS New" as text (not a valid reference).
Cancel (revert change).

@anaisconce It may require a little bit of UX/UI design, right ?

Option 2 is sensible!
As for Dmitry's suggestion, what if instead of a pop-up, there's a one-click way to add the reference and fix the broken link. I am picturing something like this.

Clicking the green + icon would be the same as double clicking into the cell to add it from the menu. On hover, the green icon in the red cell can darken to the darker green color, as it does when adding from the reference search list, in order to better communicate that this is a button and not just an icon.

A pop-up when hitting enter may be a bit jarring, it can feel like something is (disproportionately) wrong, or interrupt a workflow. The downside to using the small + button in the cell is that we don't seize the opportunity to teach a bit more about reference columns. But sometimes, for end users, that's the goal -- to make it easy and frictionless as if working in a custom app without having to parse too much information and make additional decisions.
Also I want it for myself because I often forget to click the + in the reference search list and then it's more clicks to fix that error. 😊

a user reported a UX issue to me about this,
when copy/pasting data in a table with references, it would be great to have a way to automatically add new entries in the reference table,
there are probably different ways to handle this, could be a button to transform all red entries of the column into new entries in the column settings maybe or change the popup behavior here to make it default (maybe it needs a separate issue for this...)


Ignore lost+found directories

The startup script incorrectly assumes bootstrap has been done already when it encounters any file in the data or config volumes. This obviously breaks when mounting in real volumes but might break in other cases as well.

Thanks !
merged in 1.1.3


Update gift card validation when adding to checkout

Check currency when adding the gift card to checkout.
Check used_by user when adding the gift card to checkout.
Add tests for adding a gift card to the checkout.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-3902 Update AddPromoCode mutation


fix code coverage badge to point to on-tftp

@benbp @rolandpoulter @davequick @RackHD/corecommitters

Is this local to this repo, or is this copy/paste error in every one?

this is just this one.

:+1:

+1

:+1:


update registry s3 storage driver docs to add skipverify flag

Proposed changes
Document useful flag that was missing from the docs

Doing a quick check if this flag is in the current (2.7.1) release; looks like this option was added in https://github.com/docker/distribution/pull/2632, and I see that change is in the v2.7.1 tag at least; https://github.com/docker/distribution/blob/v2.7.1/registry/storage/driver/s3-aws/s3.go#L95 (https://github.com/docker/distribution/blame/2461543d988979529609e8cb6fca9ca190dc48da/registry/storage/driver/s3-aws/s3.go#L95)

Thank you @eug48 for updating the documentation :)
Thanks @thaJeztah for verifying the update!


How can I write cluster.npy?

Hi,
I'm trying to restore a 10-mer trajectory, but I don't know how the .npy file format should be written. Could you please provide an example?
Thanks.

Dear user,
You do not need to specify any .npy. you only need to specify the protein selection with the selection flag. If your system contains more than just the 10mer (except solvents). It might be wise to add that to the selection too. In case you want to create a custom .npy for segmentation (more advanced but usually not required). The format is [nframes, natoms] where the values are the segment id per atom (integer).
Feel free to ask more questions if you still have issues,. However it would be useful to know the whole complete and the terminal input you specify.
Cheers,
Bart

If you are using an atomistic system you can run the following:
mdvwhole -f your.gro -x your.xtc -sel 'protein' -wa True
If you use a martini system of proteins in water use:
mdvwhole -f your.gro -x your.xtc -wa True
For more examples look at the examples in the readme of this repository.

Please let me know if the problem is resolved. Also, if it did work, feel free to share a snapshot of the system in this thread. I'd love to see things being whole :D

Hello, sorry for taking so long to reply.
I've tried the commands you've listed and haven't gotten much improvement.
In addition, I have some questions about the natoms option of ndp. Does it refer to the number of atoms? For example,
nframes, natoms
1, 2657
2, 2657
Thanks.

Could you please tell me what you system is and what your solvent is? Also what force field etc. I think you are either using a non-standard solvent or you are running atomistically. In which you have to change your selection. There is no ndp required. The matrix shape should be a binary .npy file generated by numpy in python which can be exported. However, like I said before you probably do not need it.
You can also send a single gro file or something similar to me, I will try it and give you the command to use.

An example .npy file shape for a 5 atom system with 6 frames:
clusters = np.array([
[1,2,1,3,4],
[1,2,1,3,1].
[1,2,1,3,1],
[1,2,1,1,1],
[1,1,1,1,1],
[1,1,1,1,1],
])
print(cluster.shape) => (6,5)
As you can see there is a cluster assigned to each atom in each frame. The story in this clustering is that over time everything clumps up.

The system is made up of ten cyclic peptides. Tip3p is used in the simulation, but the water is removed when dealing with the trajectory.
I have sent it to your email address by gro. Thank you for your help.

In the end mdwhole was working, the issue was that after the fragments are made whole. Betty would like to also make the trajectory continuous, (or as continoous as possible). This is currently not a feature of MDVWhole.


Created Parser Plugin for validation errors   issue#675

Ok. I will make changes.
Another thing I want to point out that we should have plugin for viewcell and form helper methods as well.
If you agree I would happy to help.

@jaynarayan89  Just pointing out that we would like to see contributions GPG-signed ... https://bcit-ci.github.io/CodeIgniter4/contributing/signing.html

Hi , I accidentally merged another branch with this and messed up commits. I tried to revert but not succeed. what should I do now?
I am learning github. sorry for the trouble.
@jim-parry ok. I will do it.

@jaynarayan89 Your last "good" (pre patch2) merge shows a commit hash tag starting with "17fa3d4". Assuming that is the "good" point you want, you should be able to revert to that with "git reset --hard 17fa3d4...", and then "git push -f origin patch-2" to fix your feature branch.

Thanks a lot @jim-parry
I need another help regarding Unit testing. when i run phpunit in my root folder it shows:
PHPUnit 3.7.21 by Sebastian Bergmann.
Configuration read from D:\xampp\htdocs\ci4.dev\CodeIgniter4\phpunit.xml
Time: 921 ms, Memory: 10.00MB
←[30;43m←[2KNo tests executed!
←[0m←[2K
Generating code coverage report in Clover XML format ... done

@jaynarayan89 I run unit tests with the following command from my project root...
phpunit --exclude-group DatabaseLive --colors --coverage-text=tests/coverage.txt --coverage-html=tests/coverage/

@jim-parry
Again same messages.
D:\xampp\htdocs\ci4.dev\CodeIgniter4>phpunit --exclude-group DatabaseLive --colo
rs --coverage-text=tests/coverage.txt --coverage-html=tests/coverage/
PHPUnit 3.7.21 by Sebastian Bergmann.
Configuration read from D:\xampp\htdocs\ci4.dev\CodeIgniter4\phpunit.xml
Time: 505 ms, Memory: 10.00MB
←[30;43m←[2KNo tests executed!
←[0m←[2K
Generating code coverage report in Clover XML format ... done
Generating code coverage report in HTML format ... done

@jim-parry
Regarding phpunit version:
I installed phpunit  via composer an hour ago. using this command
composer require --dev phpunit/phpunit ^6.2
however it installed old version. I tried but cant install latest version.

@jaynarayan89 Rather than clutter up this commit history, can you start a thread with me on our slack channel? thanks ... http://codeignitersignup.ciblox.com/

@lonnieezell
Please review changes

I will, but you have to give me more than an hour. :) It's the middle of my work day here, and, unfortunately, I don't make a living developing this so won't be until tonight (hopefully!) before I can check it. Though the way the last couple of weeks have been going, I'll probably be working really late.


v1.0.3 failing tests involving RunInTransaction

I upgraded the xUnitRevitUtils package in my solution to 1.0.3 from NuGet and all the tests that had an xru.RunInTransaction() are now failing.
I then went back to my build of the dll (which has the context and uiapp exposed but not the logic to suppress warnings), overriding the dll from NuGet, and it works fine.
Also, when I tried to use RunInTransaction from 1.0.3 I didn't set ignoreWarnings = true, but it was still failing.
You might want to have a look at that. Again, I'm not sure why this is happening but my suspects are on the logic to suppress warnings.

Mmmmh ok. It sounds odd because it all works with my build of the dll of the utils and the program works too, so I'm pretty sure my code is fine.
I'll debug and let you know if I find out something.

Aaaactually! Could it be that's happening because I'm using 1.0.3 from NuGet, but what's in Revit is still built upon 1.0.1?
I guess what's in Revit doesn't have the warning suppress logic so it freaks out?

Yes, the version of xru in use by xUnitRevit and by your test library has to be the same.
Closing this as we seem to have nailed it!

Yeah I just tried again by updating both xru and library and it works.
(It didn't work this morning when I updated both, but then I tried again and it worked...I suspect I messed up something initially 😆 )
Maybe worth writing that both versions need to be the same as a reminder in the docs? It might happen that someone updates the library but doesn't think of updating the Revit runner.
Bella vez! 🤘


[error]The runner has received a shutdown signal. This can happen when the runner service is stopped, or a manually started runner is canceled.

Description
Runner shutdowns in the middle of our conda env install step for Ubuntu and Windows.
"[error]The runner has received a shutdown signal. This can happen when the runner service is stopped, or a manually started runner is canceled."
No changes were made internally to our software to trigger this, but maybe there was to one of our dependencies/runners/Github resources?
Tried debugging it by clearing out my env file and adding one by one, stopped when the error first occured, removed that dependency and added the next one and it triggered the same error. So likely a memory issue stemming from conda conflict management, but it's difficult to debug as 1) the miniconda install addon doesn't appear to have verbosity option 2) the output logs only says it's being shut down for ubuntu and memory limit reached for windows (but what memory? ram/space/cpu?), and 3) Tried this https://github.com/marketplace/actions/workflow-telemetry tool to monitor memory usage during this step, but it doesn't output its report for Ubuntu/Windows because of the immediate shutdown (Mac report looks normal).
See our PR debugging this: https://github.com/axondeepseg/axondeepseg/pull/756
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
2.309.0
Failed build: https://github.com/axondeepseg/axondeepseg/actions/runs/6236507561/job/16927973102#logs
Last build that passed (5 days ago): https://github.com/axondeepseg/axondeepseg/actions/runs/6181019195/job/16787172425
Is it regression?
No
Expected behavior
Conda install goes through. Bug happened without changes to the software, it was detected on an overnight automated test.
Actual behavior
Linux and Windows tests now failing.
Repro steps
Github actions file: https://github.com/axondeepseg/axondeepseg/blob/master/.github/workflows/run_tests.yaml

It looks like you have a similar problem with this discussion (https://github.com/actions/runner-images/discussions/7188).

Hey @mathieuboudreau!
I doubt that the problem lies in the image - from the history of running tests in your repository, I see that red and green runs exist on the same image of ubuntu-22.04 (20230911.1.0), which means it cannot be related to updates of the image itself.
Let me try to collect as much information as possible that may help in finding the reasons. This is a very common problem. The discussion on this topic is perhaps the most popular.
From the runner history I may say next:

https://github.com/axondeepseg/axondeepseg/actions/runs/6231217029/job/16912426816 - ran out of RAM
https://github.com/axondeepseg/axondeepseg/actions/runs/6231217029/job/16912427056 - ran out of RAM
https://github.com/axondeepseg/axondeepseg/actions/runs/6193967674/job/16816263071 - ran out of CPU
https://github.com/axondeepseg/axondeepseg/actions/runs/6193967674/job/16816263277 - ran out of RAM
https://github.com/axondeepseg/axondeepseg/actions/runs/6181019195/job/16778360528 - random failure, didn't use all resources

We can only recommend to optimise build process or to ask community to help you with it.

@mathieuboudreau internal resources, unfortunately there is no information there that would allow us or you to localise the issue. You may ask us to provide info about failed runs of this type if you need it.

Ok no worries, just wanted to make sure I didn't miss something - appreciate it!

Resolved it on my end, documenting it here in case someone finds this thread in the future. Removing the conda-forge channel and moving the necessary packages that aren't in the default channel to pip in the environment file fixed it : https://github.com/axondeepseg/axondeepseg/pull/756/files
The CI runs through quickly and without memory problem now: https://github.com/axondeepseg/axondeepseg/actions/runs/6239345090/job/16937069377?pr=756


Uncaught TypeError: Bad argument

From @noahcawley on December 10, 2014 16:50
Uncaught TypeError: Bad argument
Atom Version: 0.154.0
System: linux 3.13.0-40-generic
Thrown From: linter package
Steps To Reproduce

Open atom with a haskell file (.hs postfix)
...

Stack Trace
At child_process.js:1134

TypeError: Bad argument
  at TypeError (native)
  at ChildProcess.spawn (child_process.js:1134:26)
  at Object.exports.spawn (child_process.js:993:9)
  at new BufferedProcess (/opt/atom/resources/app/src/buffered-process.js:40:37)
  at LinterHLint.Linter.lintFile (/home/ncawley/.atom/packages/linter/lib/linter.coffee:142:19)
  at /home/ncawley/.atom/packages/linter/lib/linter-view.coffee:145:18
  at Object.oncomplete (fs.js:93:15)

Copied from original issue: atom/atom#4514

[Enter steps to reproduce below:]

...
...

Atom Version: 0.199.0
System: Mac OS X 10.10.3
Thrown From: linter package, v0.12.3
Stack Trace
Uncaught TypeError: Bad argument
At /opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:260

TypeError: Bad argument
  at TypeError (native)
  at ChildProcess.spawn (child_process.js:1119:26)
  at Object.exports.spawn (child_process.js:977:9)
  at BufferedProcess.module.exports.BufferedProcess.spawn (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:188:44)
  at new BufferedProcess (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:52:14)
  at LinterScssLint.Linter.lintFile (/Users/kurtschlatzer/.atom/packages/linter/lib/linter.coffee:178:19)
  at /Users/kurtschlatzer/.atom/packages/linter/lib/linter-view.coffee:163:18
  at Array.forEach (native)
  at /Users/kurtschlatzer/.atom/packages/linter/lib/linter-view.coffee:162:18
  at FSReqWrap.oncomplete (fs.js:77:15)

Commands
     -2:45.1.0 core:backspace (input.hidden-input)
  3x -2:40 core:save (input.hidden-input)
  2x -0:32.7.0 tree-view:reveal-active-file (atom-workspace.workspace.scrollbars-visible-always.theme-base16-spacegray-syntax.theme-one-dark-ui)

Config
{
  "core": {
    "themes": [
      "one-dark-ui",
      "base16-spacegray-syntax"
    ],
    "excludeVcsIgnoredPaths": false,
    "followSymlinks": true,
    "disabledPackages": [
      "autosave",
      "autoflow",
      "background-tips",
      "autocomplete",
      "welcome",
      "language-c",
      "language-clojure",
      "language-csharp",
      "language-go",
      "language-java",
      "language-objective-c",
      "language-perl",
      "language-python",
      "language-toml",
      "metrics",
      "open-on-github",
      "pepper-autocomplete",
      "autocomplete-paths",
      "dash",
      "file-templates",
      "git-log",
      "linter-doiuse",
      "package-list",
      "timecop",
      "bookmarks",
      "styleguide",
      "feedback",
      "spell-check",
      "wrap-guide",
      "Remote-FTP",
      "cut-line",
      "lorem-ipsum",
      "w3c-validation",
      "linter-tidy",
      "linter-htmlhint",
      "less-autocompile",
      "language-mustache",
      "linter-less",
      "title-case",
      "pigments",
      "auto-indent",
      "deprecation-cop",
      "esformatter",
      "linter-phpcs",
      "linter-scss-lint",
      "linter-csslint"
    ],
    "projectHome": "/Users/kurtschlatzer/Projects"
  },
  "linter": {
    "lintOnSave": false,
    "statusBar": "Show error of the selected line",
    "showInfoMessages": true,
    "clearOnChange": true,
    "lintDebug": true
  }
}

Installed Packages
# User
Package-Upgrader, v0.3.1
atom-beautify, v0.27.1
autoclose-html, v0.16.0
autocomplete-php, v0.3.6
base16-spacegray-syntax, v0.2.0
editorconfig, v1.0.0
indent-guide-improved, v1.2.0
language-generic-config, v0.2.0
language-htaccess, v0.1.1
language-svg, v0.8.0
linter, v0.12.3
linter-jshint, v0.1.4
linter-php, v0.0.12
markdown-preview-opener, v0.1.1
open-last-project, v0.4.1
pretty-json, v0.3.3
synced-sidebar, v0.2.3
text-manipulation, v0.6.0
wordpress-api, v1.2.1

# Dev
No dev packages


skin tone support

🏻
Hi, this emoji not support
please add support this emoji and other skin tone support

@forPelevin
I would have sent a pull request a long time ago but:
// Code generated by generator.go ; DO NOT EDIT. in first line data.go
Have a look, please?)

Hi @mc-sim , thank you for the heads up. I got distracted and forgot to respond on your previous message, sorry for that.
I've researched the topic and couldn't find the skin tone emojis in the unicode data files. For this library, I use the files https://unicode.org/Public/emoji/15.1/ as emoji data source. Since your suggestion seems like a real emoji, can you raise a PR with the proposed change? Thank you.


[Bug] [1.20.1] [Forge] All ingredients and outputs are empty in the latest JEI

Description
All ingredients and outputs are empty in the latest JEI:

Step to Reproduction

Join a world.
Press 'U' over Smithing Table to show JEI's usages screen.
Select 'Armour Trimming' category.

Environments
Minecraft 1.20.1
Forge 47.3.7
Architectury 9.2.14
All The Trims 3.4.3
JEI <IP_ADDRESS>
Logs
No warn or error.

1.20.1 is no longer maintained. ATT has support for EMI/JEI and REI. Try another recipe browser


Updated spc700.asm to work with current bass

referenced here:
#6
I took a crack fixing it, it seems to compile OK and gives (I think) the same results as the compiled asm in spc700.h
I don't know if you want PR on this kind of stuff.. but I thought I'd give it a shot.
My workaround for the alignment isn't pretty, but it seems to get the job done.  Feedback welcome, I'm new at this.

That's actually not a bad idea to update the code for the latest version of bass. I'll use this as a test case since perhaps I can do some C++ programming to restore the align command...

Ahh, good catch!  I didn't notice that there were duplicate named setInstrument labels, should be easy enough to fix by renaming one.

I pushed the fixed version, and now it generates an identical binary!


Compatibility with aiofiles

aiofiles is a Python project to perform IO asynchronously. I am able to asynchronously read files from s3, but I cannot write files to s3. The following code example fails:
import asyncio

import aiofiles
import aiohttp
from cloudpathlib import S3Path

async def main():
    chunk_size = 1000

    path = S3Path("s3://my-bucket/image.png")

    async with aiohttp.ClientSession() as session:
        async with session.get("http://httpbin.org/image/png") as resp:
            async with aiofiles.open(path, "wb") as f:
                async for chunk in resp.content.iter_chunked(chunk_size):
                    await f.write(chunk)

asyncio.run(main())

The error is:
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/tmpq4ls9jhw/....../image.png'

I am using cloudpathlib==0.16.0 and aiofiles==23.2.1.
If I add a path.touch() before initializing the ClientSession, the code runs without error, but the file in s3 is zero bytes.
Thank you for your work on such a clean and easy to use library! It truly is a job!

Thanks @hwong557, the issue here depends entirely on what aiofiles.open(path, "wb") does within their method with the S3Path object. This is another instance of the kinds of issues in #128.
~If they see that path has an open method and they use that properly like a context manager, things could work. If not, all bets are off.~ I took a look, and they just pass it to the built-in open.
A few possible workaround:

It's possible that the experimental work in #322 makes this scenario work. That would be useful to know.
Write to a temporary file and then use your cloud path's upload_from method to write to the cloud.
Call path.open() before your async code and then path.close() afterwards to close and upload it (this likely breaks the async writing, but I'm not sure).


pip dependency conflicts on MacOS

Hello! In attempting to install this package on MacOS I have been running into a pip dependency conflict. Specifically, all versions of this package depend on ecmwflibs>=0.5.3 . After examining the available packages, I find that on MacOS the only version of ecmwflibs that seems to be available via pip is 0.5.0. By contrast, on a Linux machine that I have access to, there are several more versions of ecmwflibs available and so this conflict does not occur.

While awaiting formal word from ECMWF, my only suggestion is that you might have to install ecmwflibs 'manually', prior to your other builds.  For details: https://github.com/ecmwf/ecmwflibs/blob/master/HOWTO.md

Similar issue on Windows. Is there any guide on manually installing ecmwflibs>=0.5.3 on Windows?

For most issues related to ecmwflibs not found/not installed, you can try installing climetlab (which is currently a dependency of ai-models) differently.
Have a look here for more details : https://github.com/ecmwf/climetlab/blob/develop/climetlab-and-ecmwflibs-install.md

Thanks for the answer Florian! Do you have any information on what to do after the steps in that link? I'm able to install climetlab without ecmwflibs, but am unsure how to then get ai-models installed after that without running into the ecmwflibs>=0.5.3 error.


Fix ROC evaluation to properly access binary labels

ROC evaluation accesses uninitialized memory via CLabel::get_value() instead of the actual labels via CBinaryLabel::get_label().
I added the typecasts after checking the label type and replaced the function calls.

@lkuchenb thnx! there was an error actually in the evaluation_thresholds_modular script, which i think is actually an error that should be investigated as it just popped up now. just to be sure i'm re-running now the python modular test... let's see what's the outcome.

@lkuchenb ok so it's definitely an error caused by this change

I reverted parts of the fix. The predicted labels need to be accessed through get_value() as these are the confidence values returned from the machine and used for discrimination based on different thresholds for the ROC evaluation.
The ground truth labels however don't necessarily have confidence values. There is also no mechanism that ensures that if there are no confidence values, the labels are used instead.
The constructors
CBinaryLabels::CBinaryLabels(SGVector<int32_t> src)
CBinaryLabels::CBinaryLabels(SGVector<int64_t> src)

ensure that values and labels synchronously set. However, I was using
CBinaryLabels::CBinaryLabels(int32_t num_labels)

Commits should be squashed to one

Fine to merge apart from this.
Is ROC unit test covered?

Thanks for your comments, I integrated them and squashed the commits.
I don't know whether there is a ROC test case, but as I said, this is rather related to Labels coming with uninitialized confidence values. Possibly other features in shogun break when ground truth labels are passed with uninitialized confidence values.

OK, some minor things then we can merge.
About the unit test. If there was an uninitialised memory error earler, you should add a unit test that causes this error before your patch and that passes after your patch.

Merging, as travis failures were unrelated


Nested tables are not being exported correctly in @blade to Excel

Salam. I am trying to export my view with nested tables to excel but no success. The inner table gets distorted in excel file whereas it is correctly loaded in view in the browser. I think this library does not support exporting views with nested tables......or may be there is some workaround which I don't know. Any help will be appreciated.
Thanx

Found a solution. Append a random number in templateURL. This will not cause any harm and issue will be resolved. Here is my working code.
        var modalInstance = $modal.open({

            templateUrl: 'views/shopfloor/check-in-panels/check-in-item.html?temp='+Math.random(),
            controller: 'CheckInPanels.CheckInItemController',
            resolve: {
                workorder: function () {
                    return $scope.workorders.work_order;
                }
            }
        });

I experienced the same problem. I'm forced to do nested tables because of the complicated tabular data.


Add initial support fo guix

Hi,
I'd like to add another Linux distro. More information can be find here.
It's similar to NixOS, however I don't know how to integrate it correctly into VPSFree.
Could you provide guidance please?

Wow, that'd be cool! I tried to play with guix support myself, but I'm not familiar with the language and get lost in it...
Scripts in this repository have to build or download a root filesystem, which is then cloned to create VPS. build.sh should prepare this filesystem in directory at $INSTALL. Our VPS are containers, so there's no bootloader, we start the init system directly, e.g. /sbin/init.
The build scripts are run by osctl-image from vpsAdminOS, which you could download here: https://iso.vpsadminos.org. But maybe it'd be sufficient to run ./bin/runner manually, e.g.
./bin/runner image build 1234 <work-dir> <install-dir> guix

<install-dir> is where the result should be.
Otherwise feel free to ask concretely, you can also join us on IRC in #vpsfree or #vpsadminos on irc.libera.chat.

Based on the last time I looked at it, I think we'd need something like command guix system docker-image. guix system container unfortunately seems to be rely on the host's store, but now I see guix system init, which has a --no-bootloader option, that might be worth a try, although I think it didn't work for me.
If they at least supported LXC images like docker, we could fit into that.

Hi @aither64, thanks for the information. True, it takes time to learn all the parentheses ...
Well my idea was to follow the guide published for Linode
As you correctly said we probably need to use guix system docker-image as this is independent of the host which is probably Nix based, right? And it needs also separation for all the VPS running there which is what the guix system container does not provide.
Here's list of supported Guix images:
guix system image --list-image-types
The available image types are:

   - rock64-raw
   - pinebook-pro-raw
   - pine64-raw
   - novena-raw
   - hurd-raw
   - hurd-qcow2
   - uncompressed-iso9660
   - iso9660
   - efi-raw
   - docker
   - raw-with-offset
   - qcow2

I'll look into osctl-image.
Basically what is needed is to build the rootfs in correct format and then let the ./bin/runner to start Sheppard (the initd process in Guix), right?

I'll try to explain the process a bit better. The build scripts's job is to build a container image, which is essentially a tarball of its root filesystem, i.e. /. It's just directories and files that you place in directory $INSTALL.
osctl-image is a tool which can run the build script in a container on vpsAdminOS, this is the builder.  Builders are needed, because e.g. it is best to build ubuntu on ubuntu, fedora on fedora, etc. osctl-image executes ./bin/runner with proper arguments and creates the tarball of $INSTALL. What I meant is that you could avoid having to use vpsAdminOS and osctl-image by executing ./bin/runner directly, just for testing purposes.
The builder containers are created from previously built images, so as it is, it cannot be guixsd. That shouldn't be a problem afaik, because it should be possible to run guix from another distribution. Built images are put into a repository at https://images.vpsadminos.org.
When creating a VPS, the tarball is downloaded, extracted and then the init system is run, usually sbin/init.
The problem I see with guixsd is that it does not support building the kind of image that we need, i.e. a normal system without needing bootloader and a disk to mount root filesystem from. The container's filesystem is already mounted when we run its sbin/init.
I can demonstrate on NixOS. The build scripts calls make template from vpsAdminOS, which uses nixpkgs/nixos/lib/make-system-tarball.nix to create the tarball. So there's support for this kind of thing in nixpkgs itself. The question is if guixsd supports something like this :)
In theory, you could build the docker image, then extract it to $INSTALL... and maybe then it would be possible to create a VPS from it. But could such a system be managed by guix from the inside? I don't know.

I looked into creating the initial rootfs and the best option might be to download the guix installer and then run:
guix system init --no-bootloader server.scm /tmp/rootfs

I'll try to implement it and send the updated PR.


⚠ Bofrost.at has degraded performance

In abdae0b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 971 ms

Resolved: Bofrost.at performance has improved in 3dd292c.


Update line break regex for HTML->Markdown to support break tags with styles

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

More context in https://github.com/Expensify/Expensify.cash/pull/4009#issuecomment-879577943, this is part of https://github.com/Expensify/Expensify.cash/issues/3047, @parasharrajat will handle as a follow-up
Action Performed:

Copy the following text:

Bold text
italic text
~Strikethrough text~

Paste the text in e.cash

Expected Result:
Line breaks should be included in the pasted message
Actual Result:
Line breaks aren't included in the pasted message

Workaround:
The user has to manually insert line breaks
Platform:
Web
Version Number: 1.0.77-6
View all open jobs on Upwork

It is straightforward like we did for Italic.
<br(?:"[^"]*"|'[^']*'|[^'">])*>(?![^<]*(<\/pre>|<\/code>))\n?

We need to exclude pre and code tags and Allow any attribute between start tags, with one exception there is no end tag here.

Let us know when https://github.com/Expensify/Expensify.cash/issues/3047 is fixed so we can retest this PR :)

We're going to be doing https://github.com/Expensify/Expensify.cash/issues/3047 in pieces, so https://github.com/Expensify/Expensify.cash/pull/4009 is the latest PR (on staging) to test for this!

Paid!


no-response-body rule is triggering for some standard templates

Some templates shown in the samles trigger that rule which is confusing
reason this wasn't caught before it is that rule is only triggered on namespace starting with Azure. (legacy behavior from before the linter system)
Arm
  delete is ArmResourceDeleteWithoutOkAsync<Employee>;
  /** A sample HEAD operation to check resource existence */
  checkExistence is ArmResourceCheckExistence<Employee>;

Playground Link
Azure core
 createWidget is Operations.ResourceCreateWithServiceProvidedName<Widget>;

playground

est: 5

Copy core rule but exclude 202


[v2] I/O Exception when processing request due to connection reset

Babysat till the error occurred, the youtube livestream didn't go down at any point nor did the discord bot, stop command was not issued by any user.  Using Jar build 3005, Pip version <IP_ADDRESS> and Lavalink.py v2 branch.
Log:

Connection problems will inevitably happen. You might want to retry on abnormal track ending.

Looking like jar build 3065 has fixed this.

This should be resolved on Lavaplayer 1.2.54 (Lavalink Build 3066)

This hasn't been reported in a while so I'm going to assume what @Devoxin says is correct.


branch rename master -> main

changed workflows
changed all markdown files

Codecov Report

Merging #489 (4f126c0) into main (eeab6db) will decrease coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #489      +/-   ##
==========================================
- Coverage   37.39%   37.12%   -0.27%
==========================================
  Files          22       22
  Lines        9141     9141
==========================================
- Hits         3418     3394      -24
- Misses       5322     5352      +30
+ Partials      401      395       -6

Impacted Files
Coverage Δ

KubeArmor/monitor/syscallParser.go
78.24% <0.00%> (-1.74%)
:arrow_down:

KubeArmor/core/dockerHandler.go
52.12% <0.00%> (-1.55%)
:arrow_down:

KubeArmor/feeder/policyMatcher.go
39.14% <0.00%> (-0.24%)
:arrow_down:

KubeArmor/core/kubeUpdate.go
42.34% <0.00%> (-0.17%)
:arrow_down:

Continue to review full report at Codecov.

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


double "Allow remote administration"

It seems like there's something wrong with the configuration file it genereted for me (just used cups as it came installed on my very recent install of Manjaro KDE).
(I haven't modified the configuration manually yet I think)
The configuration has this:
# Allow remote access
Port 631
Listen /run/cups/cups.sock
# Allow remote administration...
<Location />
  Order allow,deny
  Allow @LOCAL
</Location>
# Allow remote administration...
<Location /admin>
  Order allow,deny
  Allow @LOCAL
</Location>

I think line 4 in that code block should say " # Allow remote printing" or something, right?
Because both shouldn't be saying 'remote administration' as only one has admin access?
EDIT:
CUPS 2.3.3

Maybe contact the Manjaro folks?

Doesn't seem like it's a problem anymore. Cups did switch to the openprinting version, so maybe that has something to do with it. But I did reinstall manjaro at the beginning of the year. But it's only in the config once now :+1:


Special coverage percolator scoring

Adds sponsored special coverage precedence rules. Additionally:

Moves special coverage percolator queries into bulbs
Special Coverages ALWAYS percolated (fixes recent regression where in-active SC never added to percolator after start_date becomes active until it is saved). Must update property site logic to filter start dates when upgrading.
Lint

This introduces breaking changes to Special Coverage percolator that will require property sites to switch to using these new bulbs-based percolator methods (because all special coverages now percolated and must be filtered on percolator GET queries). A migration step will automatically trigger percolator re-saves.
Extra effort required to make this all work under ES v1.4, which is missing various Percolator scoring and filtering features.
STD-2853

I need to sort out start_date/end_date field issues, otherwise rest of logic is in place.

@benghaziboy @MichaelButkovic @spra85
This is ready for review.
Everything now works on ES v1.4+, and I Travis now tests against multiple ES versions (so we're ready for a future upgrade).

:+1:

@spra85  Updated per our discussion. Merged both percolator queries into a single method, that will optionally omit Unsponsored.

:+1:

Updated query to sort Sponsored ahead of Unsponsored. This allows property reading lists to just call percolate_special_coverage() once,  instead of 2 separate calls   (#1-sponsored, #2-all)
https://github.com/theonion/django-bulbs/compare/973f593a29a286bab037312aa7d498be779f5cc7...cfedeeb1cd37a3cc4a3706ff9b1a0ad3dd8793e1


Documentation: Fix Hyper-V provisioning issue.

Fixes the Hyper-V provisioning issue in the documentation by providing relevant command.

@TheCelestial25 did you intend to close this?  It seems like something we'd want to clean up and integrate (sorry I somehow missed this PR when you first sent it)

Thanks @timabbott this was merged after editing by you in another PR.

Ahh, OK, great :)


feature: allowing manage lists Append / Pop

Implementation to allow manage lists / stacks using append and pop.
Get: returns a list or value
Set: accepts value/values separated by comma
Append: add the value/values to the end of the list
Pop: retrieve and delete the last item

@cristianoliveira I will take a look at it next week, if you feel like it merge into Master, seems ok :)

It is hard to tell if there is something wrong here. Your code is clean, well designed and functional.
Also the feature is worth these lines.  Good job!

Thanks guys.
I did this PR just because is kind of big change that modify the behavior of get and set commands. So I thought was a good idea to have a code review.

@cristianoliveira merged!

:)


Documentation for ensuring CUPTI for Profiling is Misleading

URL(s) with the issue:
https://www.tensorflow.org/guide/profiler#install_the_profiler_and_gpu_prerequisites
Description of issue (what needs changing):
The documentation says to do ldconfig -p | grep libcupti to check that CUPTI exists on the path, and to do export LD_LIBRARY_PATH=/usr/local/cuda/extras/CUPTI/lib64:$LD_LIBRARY_PATH to fix it if it is not on the path.  However, the documentation can be misleading in situations where an old install of CUDA 10.0 has been replaced with 10.1 (at least on my installation).
My output when checking the path is as below:
tyler@lambda2:/usr/local/cuda/bin$ ldconfig -p | grep libcupti
	libcupti.so.10.0 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libcupti.so.10.0
	libcupti.so (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libcupti.so

Reading the documentation, this suggested to me that I did indeed have a version of libcupti on the path, and that everything should work. However, when I trained my model with the profiler on I saw the following error logs in the console.
2020-05-13 15:49:23.364143: I tensorflow/core/profiler/lib/profiler_session.cc:163] Profiler session started.
2020-05-13 15:49:23.364212: I tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1365] Profiler found 1 GPUs
2020-05-13 15:49:23.364588: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'libcupti.so.10.1'; dlerror: libcupti.so.10.1: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/cuda-10.1/lib64
2020-05-13 15:49:23.364606: E tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1415] function cupti_interface_->Subscribe( &subscriber_, (CUpti_CallbackFunc)ApiCallback, this)failed with error CUPTI could not be loaded or symbol could not be found.

After double checking that I had CUDA 10.1 installed and not 10.2, I did the below
tyler@lambda2:~/$ export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/local/cuda/extras/CUPTI/lib64
tyler@lambda2:~/$ echo $LD_LIBRARY_PATH
/usr/local/cuda-10.1/lib64:/usr/local/cuda/extras/CUPTI/lib64:/usr/local/cuda/extras/CUPTI/lib64

This then allows the profiler to load CUPTI
2020-05-13 18:18:51.560268: I tensorflow/core/profiler/lib/profiler_session.cc:163] Profiler session started.
2020-05-13 18:18:51.560338: I tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1365] Profiler found 1 GPUs
2020-05-13 18:18:51.561266: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcupti.so.10.1

However, rerunning the command from the documentation for checking that CUPTI is on the path gives the same output as before
tyler@lambda2:~/$ ldconfig -p | grep libcupti
	libcupti.so.10.0 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libcupti.so.10.0
	libcupti.so (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libcupti.so

Desired fixes
After updating my path, I would expect that ldconfig -p | grep libcupti would update to show that usr/local/cuda/extras/CUPTI/lib64 with version 10.1 is available.
Additionally, I believe the documentation should explicitly state that running ldconfig -p | grep libcupti should show libcupti.so.10.1 or greater
Submit a pull request?
No, I'm not sure of what the best way to check for 10.1 or 10.2 would be

i think a better instruction is just to check if there's libcupti.so.10.x in /usr/local/cuda/extras/CUPTI/lib64 because ldconfig won't show CUPTI lib files in some cases even if CUPTI has been correctly installed.

how to fix this issue on windows10?

I met the same problem that ldconfig -p | grep libcupti  shows the link info of the old install of CUDA instead of the latest install of CUDA in my server. This is because I have no uninstalled the old version of CUDA completely. After following the commands from the official guidance https://docs.nvidia.com/cuda/cuda-installation-guide-linux/index.html#removing-cuda-tk-and-driver, I remove CUDA together with Nvidia driver and install CUDA 10.1 again, and the above problem is solved.
However,  when I confirm that the CUPTI path has been added to LD_LIBRARY_PATH, the program still reminds me that can not load libcupti.so.10.0.  Then, I execute the following  commands: sudo cp /usr/local/cuda-10.1/extras/CUPTI/lib64/libcupti.so /usr/local/lib/libcupti.so && sudo ldconfig sudo cp /usr/local/cuda-10.1/extras/CUPTI/lib64/libcupti.so.10.1 /usr/local/lib/libcupti.so.10.1 && sudo ldconfig.
Amazing, it solved.

Any progressing ？
My environment is :
echo $LD_LIBRARY_PATH

output is
/usr/local/cuda/lib64::/usr/local/cuda/extras/CUPTI/lib64

$ ll /usr/local/cuda/extras/CUPTI/lib64

output is :
drwxr-xr-x 2 root root     4096 10月 21 15:31 ./
drwxr-xr-x 6 root root     4096 10月 21 15:31 ../
lrwxrwxrwx 1 root root       16 10月 21 15:31 libcupti.so -> libcupti.so.10.1*
lrwxrwxrwx 1 root root       20 10月 21 15:31 libcupti.so.10.1 -> libcupti.so.10.1.208*
-rwxr-xr-x 1 root root  5700176 10月 21 15:31 libcupti.so.10.1.208*
-rw-r--r-- 1 root root 13516866 10月 21 15:31 libcupti_static.a
-rwxr-xr-x 1 root root  9716376 10月 21 15:31 libnvperf_host.so*
-rw-r--r-- 1 root root 14726370 10月 21 15:31 libnvperf_host_static.a
-rwxr-xr-x 1 root root  2349848 10月 21 15:31 libnvperf_target.so*

@TylerADavis,
The command to check if CUPTI exists on a particular path has been updated.
Could you please take a look at this link and let us know if this is still an issue? Thanks!

@TylerADavis,
The command to check if CUPTI exists on a particular path has been updated.
Could you please take a look at this link and let us know if this is still an issue? Thanks!

I don't know how to use the install_and_run.py script . It raises error :
usage: test.py [-h] --envdir ENVDIR --logdir LOGDIR [--port PORT] [--version VERSION]
test.py: error: the following arguments are required: --envdir, --logdir

When I run
/sbin/ldconfig -N -v $(sed 's/:/ /g' <<< $LD_LIBRARY_PATH) | grep libcupti ,
It gives :
/sbin/ldconfig.real: given more than once “/usr/lib/x86_64-linux-gnu”
/sbin/ldconfig.real: given more than once “/usr/lib”
        libcupti.so.11.0 -> libcupti.so.2020.1.1
/sbin/ldconfig.real: /lib/x86_64-linux-gnu/ld-2.31.so is the dynamic linker, ignoring

@DachuanZhao,
Could you please submit a new issue from this link and fill in the template, so that we can track the issue there. Thanks!

@amahendrakar It seems that the updated instructions do resolve the misleading output, addressing the first of the two desired fixes I listed in the original post.
I believe the accompanying text could be still be made more clear. For example, Ensure CUPTI exists on the path: could be replaced with something like Ensure the proper version of CUPTI for your version of TensorFlow, as described by the TensorFlow GPU requirements (https://www.tensorflow.org/install/source#gpu) , exists on the path:. This is because some versions of TF require CUDA (and cupti) 10.0 or 10.1 or 11.0, and language reminding users to double check the version number could be helpful.
Running the new commands does indeed change the output before and after editing the path. Before editing the path, observe that only libcupti.so.10.0 is printed:
tyler@lambda2:~$ /sbin/ldconfig -N -v $(sed 's/:/ /g' <<< $LD_LIBRARY_PATH) | \
> grep libcupti
/sbin/ldconfig.real: Path `/usr/local/cuda-10.1/targets/x86_64-linux/lib' given more than once
/sbin/ldconfig.real: Can't stat /usr/local/lib/i386-linux-gnu: No such file or directory
/sbin/ldconfig.real: Can't stat /usr/local/lib/i686-linux-gnu: No such file or directory
/sbin/ldconfig.real: Can't stat /lib/i686-linux-gnu: No such file or directory
/sbin/ldconfig.real: Can't stat /usr/lib/i686-linux-gnu: No such file or directory
/sbin/ldconfig.real: Can't stat /usr/local/lib/x86_64-linux-gnu: No such file or directory
/sbin/ldconfig.real: Path `/lib/x86_64-linux-gnu' given more than once
/sbin/ldconfig.real: Path `/usr/lib/x86_64-linux-gnu' given more than once
/sbin/ldconfig.real: /lib/i386-linux-gnu/ld-2.27.so is the dynamic linker, ignoring

/sbin/ldconfig.real: /lib/x86_64-linux-gnu/ld-2.27.so is the dynamic linker, ignoring

/sbin/ldconfig.real: /lib/x86_64-linux-gnu/ld-2.27.so is the dynamic linker, ignoring

/sbin/ldconfig.real: Cannot stat /usr/lib/x86_64-linux-gnu/libnvjpeg.so: No such file or directory
	libcupti.so.10.0 -> libcupti.so.10.0.130

Now I edit the path and rerun the command, observing that libcupti.so.10.1 is now present:
tyler@lambda2:~$ export LD_LIBRARY_PATH=/usr/local/cuda/extras/CUPTI/lib64:$LD_LIBRARY_PATH
tyler@lambda2:~$ /sbin/ldconfig -N -v $(sed 's/:/ /g' <<< $LD_LIBRARY_PATH) | \
> grep libcupti
/sbin/ldconfig.real: Path `/usr/local/cuda-10.1/targets/x86_64-linux/lib' given more than once
/sbin/ldconfig.real: Can't stat /usr/local/lib/i386-linux-gnu: No such file or directory
/sbin/ldconfig.real: Can't stat /usr/local/lib/i686-linux-gnu: No such file or directory
/sbin/ldconfig.real: Can't stat /lib/i686-linux-gnu: No such file or directory
/sbin/ldconfig.real: Can't stat /usr/lib/i686-linux-gnu: No such file or directory
/sbin/ldconfig.real: Can't stat /usr/local/lib/x86_64-linux-gnu: No such file or directory
/sbin/ldconfig.real: Path `/lib/x86_64-linux-gnu' given more than once
/sbin/ldconfig.real: Path `/usr/lib/x86_64-linux-gnu' given more than once
/sbin/ldconfig.real: /lib/i386-linux-gnu/ld-2.27.so is the dynamic linker, ignoring

	libcupti.so.10.1 -> libcupti.so.10.1.59
/sbin/ldconfig.real: /lib/x86_64-linux-gnu/ld-2.27.so is the dynamic linker, ignoring

/sbin/ldconfig.real: Cannot stat /usr/lib/x86_64-linux-gnu/libnvjpeg.so: No such file or directory
	libcupti.so.10.0 -> libcupti.so.10.0.130

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.