Authentication user provider [adldap] is not defined

I already change provider  for web and add the provider in config\auth.php as instructed but getting the error.
ErrorException in CreatesUserProviders.php line 40:
Authentication user provider [adldap] is not defined. (View: C:\laravel5\blog\resources\views\layouts\app.blade.php) (View: C:\laravel5\blog\resources\views\layouts\app.blade.php)

Please advise.
i already add Adldap\Laravel\AdldapServiceProvider::class and the facade to the app.php

Did you add the Adldap\Laravel\AdldapAuthServiceProvider::class to your config/app.php file?

Yes I did as below.

However i put "adldap2/adldap2-laravel": "1.4.*" under "require-dev". should i put it under "require" in composer.json?

You haven't inserted the AdldapAuthServiceProvider in your config/app.php file according to your attached image.
You need both providers:
Adldap\Laravel\AdldapServiceProvider::class,
Adldap\Laravel\AdldapAuthServiceProvider::class

Also for your other question, if your project requires Adldap in production, you will need it in the require section of your composer.json file.

Thanks Steve. It is working after i add the Adldap\Laravel\AdldapAuthServiceProvider::class.
I guess I missed it since it is not in your intro writeup.

No problem. It's actually located in the Auth driver setup.

@stevebauman @faaazma I faced the same problem running L5.5 with adldap 2.1.3 via docker.
For some peculiar reason, adding the service providers in app.php didn't work out (although it works in other laravel application that I am running (but not via docker))
In addition, running php artisan package:discover returned all the packages that I have added in app.php except the adldap2 packages.
What I did to overcome the problem was to register the adlap providers in AppServiceProvider.php
public function register()
{
    $this->app->register('Adldap\Laravel\AdldapServiceProvider');
    $this->app->register('Adldap\Laravel\AdldapAuthServiceProvider');
}

When I did that the problem gone :confused:

Hi @iraklisg,

In addition, running php artisan package:discover returned all the packages that I have added in app.php except the adldap2 packages.

This is because v3.0 doesn't include the package discovery feature that was released with Laravel 5.5, which auto-registers the packages service providers for you. So you will have to register them manually for now.
v4.0 will have this feature included, but I will also update the v3.0 branch to include this as well.

Added: https://github.com/Adldap2/Adldap2-Laravel/releases/tag/v3.0.7

Hi, same issue.
adldap2/adldap2                       v8.1.3
adldap2/adldap2-laravel               v4.0.6
Config/auth.php
    'providers' => [
        'users' => [
            'driver' => 'ldap',
            'model' => App\Models\User::class,
        ],...

Config/adldap_auth.php published
And still this error:
InvalidArgumentException
Authentication user provider [ldap] is not defined.

/vendor/laravel/framework/src/Illuminate/Auth/CreatesUserProviders.php
     */
    public function createUserProvider($provider = null)
    {
        if (is_null($config = $this->getProviderConfiguration($provider))) {
            return;
        }

        if (isset($this->customProviderCreators[$driver = ($config['driver'] ?? null)])) {
            return call_user_func(
                $this->customProviderCreators[$driver], $this->app, $config
            );
        }

        switch ($driver) {
            case 'database':
                return $this->createDatabaseProvider($config);
            case 'eloquent':
                return $this->createEloquentProvider($config);
            default:
                throw new InvalidArgumentException(
                    "Authentication user provider [{$driver}] is not defined."
                );
        }
    }

"Authentication user provider [ldap] is not defined."

Hi @oksim14, your configuration is incorrect.
The driver name should be adldap, not ldap.

🥇 Thanks a lot... Love these 'bugs' :)

Hi I'm using laravel 5.7 and I got Authentication user provider [adldap] is not defined.
I added the following to AppServiceProvider.php inside register()
$this->app->register('Adldap\Laravel\AdldapServiceProvider');
$this->app->register('Adldap\Laravel\AdldapAuthServiceProvider');
And also tried in app.php with the following in provider array:
Adldap\Laravel\AdldapServiceProvider::class,
Adldap\Laravel\AdldapAuthServiceProvider::class,
And 'Adldap' => Adldap\Laravel\Facades\Adldap::class, in aliases array
I have 'provider' => Adldap\Laravel\Auth\NoDatabaseUserProvider::class, in ldap_auth.php
but still not work. Any idea what's the issue?

Same problem. Installed it today and got the same error and i dont find the right the error.

Hi @michaelachrisco, the driver name is 'ldap', not 'adldap'.
Open your config/auth.php file and change your driver option to ldap.
This is described in the upgrade guide:
https://adldap2.github.io/Adldap2-Laravel/#/upgrading?id=authentication-driver

@stevebauman I believe you meant @michaelcmundo not me. But its all good 👍

Shoot sorry about that @michaelachrisco! Should have checked the auto-correct 😅

Hi I changed to ldap and now I got this error instead. Any idea?

@michaelcmundo Is the input name of the username field in your login.blade.php file userprincipalname? Or is it something else?

In my login.blade.php, the name is username. I just noticed in config/ldap_auth.php that the usernames array has these:
'ldap' => [
'discover' => 'userprincipalname',
'authenticate' => 'distinguishedname',
]
Maybe my issue is because of these? What are those?

I just changed those 2 to have values 'username' as I don't really know what it is. Before I use laravel I just use php ldap_connect() and ldap_bind(). I don't need to specify the name of the username field, etc.

Yea I read it after but I don't know as previously I just use ldap_connect() and ldap_bind() which doesn't need to set those. So I tried change the config to have username instead of userprincipalname and when I tried log in I don't get the error but I always getting "These credentials do not match our records". The credentials I put in are correct but just not sure the exact name.
Is there a way I can turn on debug or logging so I can see exact name/values it's passing to the ldap?

I don't have login() in the LoginController. So I should create that function? I thought it's automatically doing the auth?


Expose Wait For Navigation

https://playwright.dev/docs/api/class-page?_highlight=waitfor#pagewaitfornavigationoptions
Is your feature request related to a problem? Please describe.
Really convenient when multiple redirects.. and only thing that works reliably.

Done


Requiring from Node returns {}

Animo seems to be just what I was looking for. However, it doesn't seem to be working for me. I guess it works when used with Webpack and ES6 imports? It doesn't for me with Node v9.11.1, bundling with Browserify. The require() just returns an empty object...

Hi, sorry, I've been traveling. Are you using babel to transpile? Maybe try sourcing the lib/index.js from the package?

I'm not transpiling.  And I can't see the file you mention in your repo...?
I solved the problem for me by just manually modifying the code and turning it into a CommonJS module.

The lib directory is in the built version, in the npm package. Yeah, ES6 import kind of handcuffs us to transpiling
Glad to hear you found a workaround. I'm going to close this out. Thanks!


i2c: add type Addr

The main rationale is that we could implement i2c.Addr.Set(s string) error method, which would implement flag.Var interface, which would reduce the amount of boiler plate in the executables in cmd/... In the meantime it's often specified as uint16 but it's annoying to process.

the code would be straight forward enough for the flag.Value
// Addr implements flag.Value
type Addr uint16

func (a *Addr) Set(s string) error {
	u, err := strconv.ParseUint(s, 0, strconv.IntSize)
	if err != nil {
		return errI2CSetError
	}
	if u > 0xffff {
		return errI2CSetError
	}
	*a = Addr(u)
	return nil
}

I presume we don't want to break api anywhere, so this would have to be an optional connivance, If maintainers wanted to accept and i2c.Addr into the New funcs for example.

closing since done by #362


Add isSatisfiedByAny() to Range and RangesList

These convenience methods check whether a range was created in such a way that it matches any version.

Many thanks 🙏
I'll review this when I'll be on PC.

Codecov Report
Base: 89.42% // Head: 89.31% // Decreases project coverage by -0.11% :warning:

Coverage data is based on head (262e372) compared to base (87d98ca).
Patch coverage: 66.66% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #108      +/-   ##
============================================
- Coverage     89.42%   89.31%   -0.12%
- Complexity      229      232       +3
============================================
  Files            19       19
  Lines           577      580       +3
  Branches         97       97
============================================
+ Hits            516      518       +2
  Misses           29       29
- Partials         32       33       +1

Impacted Files
Coverage Δ

src/main/java/org/semver4j/Range.java
75.00% <0.00%> (-1.93%)
:arrow_down:

src/main/java/org/semver4j/RangesList.java
75.00% <100.00%> (+1.92%)
:arrow_up:

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

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

@sschuberth gently ping :smile:

@sschuberth gently ping 😄

Sorry, I was busy at my day job.

@piotrooo I had to adjust the initialization order of the static ZERO field.

@piotrooo I had to adjust the initialization order of the static ZERO field.

Yet again 😒

@sschuberth thanks. I've merge it with errors but I need to fix it by my own.


Feature/third task corrections

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

155 of 162 (95.68%)  changed or added relevant lines in 10 files are covered.
32 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-3.3%) to 96.681%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

repository/IngSoft2-Model/Spaceship.class.st
9
11
81.82%

repository/IngSoft2-Model/SpaceshipCardHand.class.st
6
11
54.55%

Files with Coverage Reduction
New Missed Lines
%

repository/IngSoft2-Model/SpaceshipCardHand.class.st
4
74.29%

repository/IngSoft2-Model/Spaceship.class.st
11
81.43%

repository/IngSoft2-Model/UserInterface.class.st
17
77.33%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
1136

Relevant Lines:
1175

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

155 of 162 (95.68%)  changed or added relevant lines in 10 files are covered.
32 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-3.3%) to 96.681%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

repository/IngSoft2-Model/Spaceship.class.st
9
11
81.82%

repository/IngSoft2-Model/SpaceshipCardHand.class.st
6
11
54.55%

Files with Coverage Reduction
New Missed Lines
%

repository/IngSoft2-Model/SpaceshipCardHand.class.st
4
74.29%

repository/IngSoft2-Model/Spaceship.class.st
11
81.43%

repository/IngSoft2-Model/UserInterface.class.st
17
77.33%

Totals

Change from base Build<PHONE_NUMBER>:
-3.3%

Covered Lines:
1136

Relevant Lines:
1175

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

155 of 155 (100.0%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1136

Relevant Lines:
1136

💛  - Coveralls


apply fill color defined in scattered options object

Apply fill color defined in scattered options object instead of plain white as currently defined.

Coverage remained the same at 87.42% when pulling befd50b1859f818a64e28d989db483c6c3bb693a on crowl:master into 4c2f7940227d9ce3b3a74c0a90cb603d49f2edbd on alfredkam:master.

publish @0.3.20


[BUG] attempt to index field 'Cfg' (a nil value)

Please confirm

[x] I am running the latest version of this plugin

Describe the bug
A clear and concise description of what the bug is.
For every Telekasten command I run I get an error about Cfg indexing. Even just running :Telekasten gives me the error.
To Reproduce
Steps to reproduce the behavior:

Start nvim
Add Telekasten to Astrovim via lazyvim /lua/plugins/telekasten.lua => return ...provided lazyvim code
Close and reopen neovim
Run any telekasten command
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Operating system (please complete the following information):

OS: MacOS 15.1 on M1

Additional context
Add any other context about the problem here.
This is my first time using this plugin, so I've likely just installed something incorrectly. But I've installed other neovim plugins via lazyvim and astrovim before, and have never encountered this error.

Is it still happening ? I do not have any issue on my side.

Just tried again and I get the same error, but I didn't disable all of my other plugins like I had before. See PR description for reproducible steps and system information. If you performed the reproducible steps with my same system information then I'd make sure that you're using AstroVim as I specified in the PR description.
Thank you for taking a look.

Using LunarVim, same issue.

Same issue here after an update of Astronvim (which broke a bunch of other stuff and I'm starting to suspect a trend there)

I also am having this issue. Redoing my nvim config from scratch at the moment and had the same issue with this.

EDIT: Well... i feel dumb, but I didn't save the file that called require('telekasten').setup{...} function, so it just was never getting called. Probably something like that for you too.

So, once you called the setup function properly everything started to work ? Is that right ?
@lcpichette , @mediapathic , @Docfips could you check ? I am not able to reproduce...

@lambtho12 Yes I do have the setup file. See step 2.

In my case, it actually did turn out to be the setup function; the Astronvim update messed up such that the file containing that was not actually being called. I apologize for the noise.

I've found zk which does what I want it to: https://zk-org.github.io/zk/config/config-group.html It also has its own neovim plugin https://github.com/zk-org/zk-nvim. Not to say it's better or worse then Telekasten, I just didn't face any odd bugs when trying to use it.
It sounds like others who had the same issue as me resolved it with some config tweaks -- so this is likely a PEBCAK issue. Closing this issue under that assumption.


Does it support Chinese data?

Does this work support Chinese data?
Can you give some sample tutorials in Chinese

Theoretically OpenAttack supports any languages. But currently the TextProcessor and Substitute modules are mainly designed for English. We are planning to support Chinese text preprocessing and substitution search.

Great! I hope it can be adapted to Chinese as soon as possible

Great! I hope it can be adapted to Chinese as soon as possible

Now, OpenAttack 2.x has incorporated the support of multilinguality. You can use datasets in any language with the help of the datasets library of Huggingface.


Run App: User Settings

As a user, I would like access to settings for my Robot and Run App.
Acceptance Criteria

User can view 'Advanced Settings' section on Robot Settings page
User can view 'Advanced Settings' section on App Settings page
User can toggle robot-level feature flags in the Robot 'Advanced Settings' section
User can toggle app-level feature flags in the App 'Advanced Settings' section

Design

TBD @pantslakz

@pantslakz for the moment, the only one we need for app is:

Show dev tools (requires restart)

@mcous when user flicks this on - what do they see in the app? Do we require additional UI within the app to represent these dev tools?

@pantslakz, no, this just switches on the normal Chrome/Electron devtools. This is the same option that is automatically enabled when you run the application in development mode from source, so no additional UI is required

from ben:
https://github.com/Opentrons/opentrons/blob/edge/api/opentrons/config/feature_flags.py

This looks fantastic. For the record, @pantslakz and I talked f2f, and the server is going to report a list of settings to expose through this panel in roughly the form:
{
  "settings": [
    {
      "title": "Calibrate to bottom of well",
      "description": "Perform labware calibration by moving the tip to the bottom-center of the well rather than the top-center (which is the default if this option is unset)",
      "value": true
    },
    {
      "title": "Other setting",
      "description": "Long description of next setting",
      "value": false
    }
  ]
}

@mcous do you have any thoughts/requests?

That response schema looks perfect

Ticket for the backend work: #1656

JK still need to do robot settings

@btmorr linking a ticket for blocker.


Error   PA3013: Property Value Changed: TopParent.Index

[ ] Please attach a sample .msapp to this Issue which demonstrates the issue you are experiencing. (Please do not attach any file which may contain sensitive personal or company data.)

Describe the bug
Starting: pac - Clone or Sync Solution
Task         : PowerShell
Description  : Run a PowerShell script on Linux, macOS, or Windows
Version      : 2.232.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/powershell
Generating script.
========================== Starting Command Output ===========================
"C:\Program Files\PowerShell\7\pwsh.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'D:\a_temp\3-5297-4360-9aeb-81745b3c0485.ps1'"
'***' authenticated successfully.
Validating connection...
Default organization: Dev
Connected to... Dev
Connected as ***
Authentication profile created
* DATAVERSE ppdev                          https:// XXXXX.crm.dynamics.com/    : ***     Public
Cloned solution available; Triggering Solution Sync
Pointing to cdsproj folder path -  D:\a\1\s\field-client-source\slnFieldClient\SolutionPackage
Triggering Sync - solution sync --processCanvasApps true --packagetype Both --async
Connected to... Dev
Connected as ***
Starting Solution Export...

SystemUser
Processing Component: Workflows
Field Client Initial Ticket Synch
xxxSync to Field Client
FieldClient Sync to xxxx
Field Client Wipe Tables
Processing Component: EntityRelationships
Processing Component: MobileOfflineProfiles
FieldClientOfflineProfile
Processing Component: CanvasApps
fc_fieldclientgallerycomponentlibrary_10cf9
fc_fieldclient_8d8f4
fc_fieldclientcomponents_530f1

-- unpacking Canvas app fc_fieldclientgallerycomponentlibrary_10cf9...
Unpack from D:\a\1\s\field-client-source\slnFieldClient\SolutionPackage\src\CanvasApps\fc_fieldclientgallerycomponentlibrary_10cf9_DocumentUri.msapp --> D:\a\1\s\field-client-source\slnFieldClient\SolutionPackage\src\CanvasApps\src\fc_fieldclientgallerycomponentlibrary_10cf9
source format version: 0.24
-- unpacking Canvas app fc_fieldclient_8d8f4...
Unpack from D:\a\1\s\field-client-source\slnFieldClient\SolutionPackage\src\CanvasApps\fc_fieldclient_8d8f4_DocumentUri.msapp --> D:\a\1\s\field-client-source\slnFieldClient\SolutionPackage\src\CanvasApps\src\fc_fieldclient_8d8f4
source format version: 0.24
Warning PA2001: Checksum mismatch. Checksum indicates that sources have been edited since they were unpacked. If this was intentional, ignore this warning.
Microsoft PowerPlatform CLI
Version: 1.29.11+g9e2b163
Online documentation: https://aka.ms/PowerPlatformCLI
Feedback, Suggestions, Issues: https://github.com/microsoft/powerplatform-build-tools/discussions
Error: Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3013: Property Value Changed: TopParent.Index
Error   PA3011: Roundtrip validation on unpack failed.
You have found a bug; this is not a specific bug, rather an indicator that some bug has been encountered.
Please open an issue and log the entirety of this error log at https://github.com/microsoft/PowerApps-Language-Tooling
To Reproduce
Steps to reproduce the behavior:

Unpacking the application

Screenshots

Additional context
This started happening this week after attempting to deploy an update to the application. We are using the ALM accelerator application.

You have this in you logs before the error:
Warning PA2001: Checksum mismatch. Checksum indicates that sources have been edited since they were unpacked. If this was intentional, ignore this warning.

Maybe some merge issues? does this error manifest with pac cli? what about older versions of pac cli?


Add Support for mat-form-field appearance

Would it be possible to add an @Input to mat-form-io or any other means to have different appearance to form-fileds as shown herehttps://stackblitz.com/angular/malkplgnkdb?file=src%2Fapp%2Fform-field-appearance-example.html. Our project heavily relies on Material form-io and we have all the fields with an outline appearance.

Hi any update on this?

@paramjeetnandal, @ArulRozario
Hey, we've done with this feature.
Now you can use it like this:
{
  "label": "Textfield",
  "mask": false,
  "type": "textfield",
  "input": true,
  "key": "textfield",
  "appearance": "standard"
}

Thanks. good work.

@paramjeetnandal, @ArulRozario
Hey, we've done with this feature.
Now you can use it like this:
{
  "label": "Textfield",
  "mask": false,
  "type": "textfield",
  "input": true,
  "key": "textfield",
  "appearance": "standard"
}

Could explain this briefly with full json


DUPLO-21171 hotfix: apply aurora perf insights validation only if performance insights is enabled

Overview
Ignore aurora type compatibility with performance insights if performance insights is not enabled
Summary of changes
This PR does the following:

Ignore aurora type compatibility with performance insights if performance insights is not enabled

Testing performed

[ ] Using unit tests
[x] Manually, on my local system
[ ] Manually, on a remote test system

Describe any breaking changes
No
Sanity check
Incompatible perf insights aurora db size and perf insights enabled

Incompatible perf insights aurora db size and perf insights disabled

Compatible aurora db size with perf insights enabled

Compatible aurora db size with perf insights disabled

Task linked: DUPLO-21171 RDS TF: Performance insight enable is not working for Cluster DB writer instance. it works when enabled through the API.


[eas-cli] add release:edit

While the graphql allows us to edit a release's name by either its ID, or the combination of projectID + releaseName, this command only implements the latter: projectId + releaseName. Adding both options seemed to add more clutter than utility.
Even though there is only one edit option: rename chose to require a special flag --rename.  This allows extensibility in the future and will also match other edit commands with more options, like the channel:edit command (which will have 2 possibilities).

Thanks, the command looks good, although I'm not a fan of --rename as a flag name. You mention extensibility and channel:edit having more options – however it will be difficult to match the same naming scheme for other field names.
What do you think of renaming the command to release:edit <release-name> --name <new-name>? Then the option name can typically match the name of the field to edit.

Or alternatively it could be release:rename --from <release-name> --to <new-name>, where the verb "rename" is in the command name itself.

Or alternatively it could be release:rename --from <release-name> --to <new-name>, where the verb "rename" is in the command name itself.

This makes sense! Changed it.


Missing Licence

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request/question

Current behavior

No Licence is defined.
Expected behavior

A type o open-source licence should be defined. NPM states that this project's licence is MIT, but Github has no visible licence.
Minimal reproduction of the problem with instructions (if applicable)

N/A
What is the motivation / use case for changing the behavior? (if applicable)

ng2-tag-input may not be used in many projects because of a missing licence - Licence Info
What do you use to build your app? (SystemJS, Webpack, angular-cli, etc.). Please specify the version

N/A
Angular version:

N/A
ng2-tag-input version:

^1.2.8
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

https://github.com/Gbuomprisco/ng2-tag-input/pull/403 :)

brilliant, thanks!


SqlPackage

SqlPackage or DacFx Version: <IP_ADDRESS>
.NET Framework (Windows-only) or .NET Core: .Net 8
Environment (local platform and source/target platforms): Windows Server 2022

Steps to Reproduce:

Connect Sql Server to Azure Arc
Try to export a bacpac
Get the following result:
One or more unsupported elements were found in the schema used as part of a data package.
Error SQL71627: The element User: [NT AUTHORITY\SYSTEM] has property AuthenticationType set to a value that is not supported in Microsoft Azure SQL Database v12.
Error SQL71627: The element Login: [NT AUTHORITY\SYSTEM] has property IsMappedToWindowsLogin set to a value that is not supported in Microsoft Azure SQL Database v12.

Did this occur in prior versions?  If not - which version(s) did it work in?
(DacFx/SqlPackage/SSMS/Azure Data Studio)

The solution to this issue depends a bit on what you want to do with a portable copy of your database. Are you looking to import it into an Azure SQL Database?
Bacpac creation includes a validation that the database elements are compatible with Azure SQL Database before export, so here's a few approaches to getting a portable copy:

If you're looking to import to Azure SQL Database - on a duplicate of the database on your SQL Server - you can remove the Windows users/logins before running the export
If you're not looking to import to Azure SQL Database - you can pursue extract/publish with the extract property /p: ExtractAllTableData=True included in the command.

@dzsquared 1. is such a common problem.,  wonder if there was a builtin way to fix this, like ignore the logins?

@dzsquared We cannot remove the logins, they are automatically set up by Azure Arc (SQL Extension) and mapped to all databases. In fact, the extension is automatically reinstalled even when removed on Azure Arc as long as the server is managed by Azure Arc.

And in fact even when the extension is removed, and the sql server is deleted from Arc management, the extension self-installs again and the problem starts all over again.

@ErikEJ I'm still "struggle" to understand the correct difference between "dacpac" and "bacpac". We are using "bacpac"s now in our application, but we only want to export/import the schema and data - that's why we used "bacpacs".
In one environment we had this issue and it seems this is "by design".
Maybe this is not the "right" place to ask, but would a "dacpac" + ExtractAllTableData:true solve this problem?

@robertmuehsig  The bacpac and dacpac formats are redundant in that they can both contain schema and data.  The bacpac does this by default, and as you noted you can opt to include data into the dacpac.
With a dacpac you have significant more flexibility of options with SqlPackage, but you cannot use the Azure portal service for import/export.

@dzsquared Is there a way to omit the "Security"-information in the bacpac? In an ideal world we have only schema+data and can import/export it from our application and work with it with first party tools like Azure Portal/SSMS/Azure Data studio.

Could possibly be done with the DacFX SDK, and packaged as a CLI.


Raspbian install corrections

Proposed changes
After implementing Pull Request #2965 the Raspbian section was inconsistent.
In addition the Raspbian section had several errors:

The given URL would install the Edge version of Docker
The package to install was incorrect (docker installs a KDE package, one needs to use docker-engine)
The docker-engine package contains the Bash completion setup, so the script installation does not bring any advantages. So I remove any mention of "recommendation" for the script as for Debian and Ubuntu this is also not the recommended installation.

These errors were corrected.
Related issues
Fixes #2098
Related PR #2965
It completes the related PR by correcting and making consistent the Raspbian section of the Debian installation instructions.

There are some minor English grammar issues in the directions. Additionally, I think the "recommend" notes are just opinions which can be rationalized all day, but choices should be left up to the user.

Hi @NonCreature0714
Thank you for your feedback. I agree with you about the "recommended" notes, so I removed it to make it unbiased.
I'm not a native English speaker, so I have attempted to correct some of my grammar. Sometimes I decided to rephrase the sentence. Hopefully this is now a bit improved. Fill free to correct me and/or to point me to the sentence I should improve.

I'm finding it a little hard to understand how this fits with #2965. Will they conflict with each other?

No they are complementary. Each PR is a change on a specific section, not overlapping.
The current PR is about the Raspbian specific section. It had a few things that could be improved or corrected, and I attempted to do so.
The other PR is only about the "pure" Debian section and it is enhancing and correcting a few things about running Debian on armhf.
I've made the 2 PR coherent with each other's. But they could be picked up independently.
I hope it clarifies my approach.

Hi @mstanleyjones
I have updated the file with your suggestion. Thank you for your feedback.
Note: in the coming 3-4 days I will be in a business trip with seldom time or access to internet. So if I do not reply quickly, please excuse me. I will resume any pending tasks at the end of this week probably.

@andrewhsu confirmed that the convenience script may be deprecated soon and so we should not add it to the docs for now. I'm not going to accept this PR because of that. Thanks for looking into this!


Reimplement with "cloudposse/terraform-aws-s3-bucket" to standardize parameters/features

Describe the Feature
I am wondering if there is a specific reason why this https://github.com/cloudposse/terraform-aws-tfstate-backend is not implemented with https://github.com/cloudposse/terraform-aws-s3-bucket? The parameters and features of both also differ.
It would be awesome if parameters (e.g. s3_object_ownership = "BucketOwnerEnforced" vs bucket_ownership_enforced_enabled = true) and features (e.g. lifecycle_configuration_rules) of both would be standardized. Reimplementing one with the other will likely prevent further drift.
Expected Behavior
Standardized parameters/features of similar TF modules.
Use Case
/
Describe Ideal Solution
/
Alternatives Considered
No response
Additional Context
No response

Not sure if this will ever be done as this module is largely stable and reimplementing would require a good deal of re-work by maintainers and consumers of this module.
cc @Nuru as he'll likely have a definitive opinion on whether or not to keep this open 👍


Contact form integration with formspree doesn't work

Hi - The formspree integration isn't working for me when I put my email address in on contact.md page (my contact page. It seems like the method is being phased out by formspree. Is there a way to connect to a specific form in Formspree?
Thanks!

You need to use the form ID from your form’s URL.
On the integration page for your form on Formspree you get a form endpoint URL that looks like this: https://formspree.io/f/myformid. The string that’s in the place of myformid here is what you want to put in the formspree_form_id field in contact.md.

@erinmr did this fix your issue?
This was fixed in https://github.com/hugo-apero/hugo-apero/pull/11
But looks we need to update the docs here to match: https://hugo-apero-docs.netlify.app/learn/built-in-contact-form/

Thanks for responding @mccarthy-m-g and @apreshill my issue was that I'd included the f/ with the form ID. I just took that out and everything worked like a charm!


My crate async-proxy fails to build in docs.rs

Crate name: async-proxy
Build failure link: https://docs.rs/crate/async-proxy/0.2.3/builds
Additional details:
My crate async-proxy started failing to build when it crossed the version of 0.2.3

INFO] [stderr] error: proc-macro derive panicked
[INFO] [stderr]  --> /opt/rustwide/cargo-home/registry/src/github.com-1ecc6299db9ec823/psl-0.4.1/src/list.rs:6:10
[INFO] [stderr]   |
[INFO] [stderr] 6 | #[derive(Psl, Copy, Clone, Eq, PartialEq, Ord, PartialOrd, Hash, Debug, Default)]
[INFO] [stderr]   |          ^^^
[INFO] [stderr]   |
[INFO] [stderr]   = help: message: failed to download the list: failed to lookup address information: Temporary failure in name resolution

It appears the Psl derive attempts to use the network, this is blocked by the docs.rs build sandbox.

It appears there is an upstream issue for this: https://github.com/addr-rs/addr/issues/6

This is a problem for the adblock crate as well, which also depends on psl. - https://docs.rs/crate/adblock/0.2.11

As Nemo said, psl (through psl-lexer) accesses the internet and this is impossible since docs.rs disallows network access to crates it's building for security reasons, but a solution would be to either use conditional compilation to disable the psl crates during docs.rs builds (Help with doing so can be found here) or to try and upstream changes to the psl crates that'll make them stop accessing the network in build steps on docs.rs

I saw that 0.2.5 built successfully, so I'm going to close this issue. Feel free to re-open if you're still having trouble!


qt6.qtbase: Could not find qmake spec 'linux-g++'

running qmake throws Could not find qmake spec 'linux-g++'
fix: install mkspecs files from qtbase source tree
--- a/pkgs/development/libraries/qt-6/modules/qtbase.nix
+++ b/pkgs/development/libraries/qt-6/modules/qtbase.nix
@@ -206,6 +206,7 @@ stdenv.mkDerivation rec {
   postInstall = ''
     mkdir -p $dev
     mv $out/mkspecs $out/bin $out/libexec $dev/
+    cp -r ../mkspecs/* $dev/mkspecs/
   '';

   dontStrip = debugSymbols;

Notify maintainers
@NickCao
Related
#186570

# ls /nix/store/fisg6zqjk144igbf8lcrkn2mhfc9xvi7-qtbase-6.3.1-dev/mkspecs/linux-g++
qmake.conf  qplatformdefs.h

Files now present, still seeing python3.10-PyQt6>   Could not find qmake spec 'linux-g++'.

I'm going to try porting the qmake-hook.sh for qt5.

It looks like the mkspecs files are already in dev output, just moved by the line before your patch.

I'm going to try porting qmake-hook.sh for qt5.

Maybe QMAKEPATH is the trick? qmakeFlags looks innocent.

{ lib
, stdenv
, fetchFromGitHub
, qtbase
, wrapQtAppsHook
}:

stdenv.mkDerivation {
  pname = "qarma";
  version = "unstable-2022-01-06";

  src = fetchFromGitHub {
    owner = "luebking";
    repo = "qarma";
    rev = "6ee1a72635d0f7974a4e38c348a511f12b593043";
    hash = "sha256-P4Hc2Q3id2M9XGexic4hoPTngdV/hYcaizUuc06LuHU=";
  };

  nativeBuildInputs = [ qtbase wrapQtAppsHook ];

  buildPhase = ''
    export QMAKEPATH=${qtbase.dev}
    qmake
  '';
}

This complains about Project ERROR: Could not find feature thread. instead of linux-g++, definitely some progress.

thread.prf is in $QMAKEPATH/mkspecs/features/unix, which should be searched by qmake, but it just ignores it, that's strange.

Tracked it down to these lines:
qarma-unstable> DEBUG 1: /nix/store/dn37lw8x5wl4f4xq6xzikx5fj4kdk8cf-qtbase-6.3.1-dev/mkspecs/features/qt_config.prf:3: QMAKE_QT_CONFIG := /nix/store/bpm1v3kgy3qkvy8fb1yfy3i39gm0wcca-qtbase-6.3.1/mkspecs/qconfig.pri
qarma-unstable> DEBUG 1: /nix/store/dn37lw8x5wl4f4xq6xzikx5fj4kdk8cf-qtbase-6.3.1-dev/mkspecs/features/qt_config.prf:4: calling built-in exists(/nix/store/bpm1v3kgy3qkvy8fb1yfy3i39gm0wcca-qtbase-6.3.1/mkspecs/qconfig.pri)
qarma-unstable> DEBUG 1: /nix/store/dn37lw8x5wl4f4xq6xzikx5fj4kdk8cf-qtbase-6.3.1-dev/mkspecs/features/qt_config.prf:5: calling built-in debug(1, "Cannot load qconfig.pri!")
qarma-unstable> DEBUG 1: Project DEBUG: Cannot load qconfig.pri!

qarma-unstable> DEBUG 1: /nix/store/dn37lw8x5wl4f4xq6xzikx5fj4kdk8cf-qtbase-6.3.1-dev/mkspecs/features/qt_config.prf:4: calling built-in exists(/nix/store/bpm1v3kgy3qkvy8fb1yfy3i39gm0wcca-qtbase-6.3.1/mkspecs/qconfig.pri)

out vs dev output ...

We fixed these lines in pkgs/development/libraries/qt-5/hooks/fix-qt-builtin-paths.sh for qt5, time to port yhem to qt6.

https://github.com/NickCao/nixpkgs/tree/qmake managed to build qarma just fine.

#186689 - ported most of the hooks, got PyQT6 sort of working without needing a hacky merged qtbase. Failing at the end of the build looking for libexec/moc in the wrong place, it's trying the non-dev output of qtbase.
 make[1]: *** No rule to make target '/nix/store/aqs2sib5wir0d3qdrivvq76zr9xm7vjm-qtbase-6.3.1/libexec/moc', needed by 'moc_qpycore_event_handlers.cpp'.  Stop.


It gives me en error.

When I want to show the Debug Menu, its is not showing!
Maybe its just me, or its an bug!

With some better description we could try to solve it together. Essentially, what error does it give you, etc.?

No only the FPS an this is my code, http://pastebin.com/P4HfZ0t6
I already try'd to fix the problem by my self, but it didn't work!

I'm not sure what you mean, and this probably isn't the place to discuss it. I would refer to the Cherno slack channel.

Okay.
Thanks anyway!


AsyncLock and HttpClient

In HockeySDK_Portable45 HockeyApp.Extensions.AsyncLock the following
return wait.IsCompleted ?                         m_releaser :                         wait.ContinueWith((_, state) => (IDisposable)state,                             m_releaser.Result, CancellationToken.None,             TaskContinuationOptions.ExecuteSynchronously, TaskScheduler.Default);
The option TaskContinuationOptions.ExecuteSynchronously can cause deadlocks with HttpClient. This should only be used with short running operations. See:
https://msdn.microsoft.com/en-us/library/system.threading.tasks.taskcontinuationoptions(v=vs.110).aspx

@mrsmitty do you have a unit test repro that cause the deadlock?


Drop indirect dependency on time 0.1

Removes chrono as a library dependency, since it wasn't being used anywhere, and reduces the features used by the build dependency to avoid bringing in time 0.1

/hold

@Luap99 @mheon @baude PTAL

/hold cancel
@paolobarbolini Thanks


Does decontX() gives out a correct count matrix?

Hi. I was trying to use decontX on my data and compare it to uncorrected data. While I see that each cell has been assigned a contamination score, in my analysis, I don't see any change in the UMI count of the cells. If there was some correction going on, shouldn't the count matrix change after running the function?

Hi @piyushjo15
The following commands should give you what you want.
If x is a matrix object:
dec <- decontX(x)
decontaminatedCounts <- dec$decontXcounts

if x is a SingleCellExperiment object:
dec <- decontX(x)
decontaminatedCounts <- decontXcounts(dec)

You can check the details in the documentation of decontX function by typing ?decontX in R. I'll paste some of the documentation here for your reference.

If x is a matrix-like object, a list will be returned with the following items:

decontXcounts:
The decontaminated matrix. Values obtained from the variational inference procedure may be non-integer. However, integer counts can be obtained by rounding, e.g. round(decontXcounts).

contamination:
Percentage of contamination in each cell.

estimates:
List of estimated parameters for each batch. If z was not supplied, then the UMAP coordinates used to generated cell cluster labels will also be stored here.

z:
Cell population/cluster labels used for analysis.

runParams:
List of arguments used in the function call.

If x is a SingleCellExperiment, then the decontaminated counts will be stored as an assay and can be accessed with decontXcounts(x). The contamination values and cluster labels will be stored in colData(x). estimates and runParams will be stored in metadata(x)$decontX. If z was not supplied, then the UMAPs used to generated cell cluster labels will be stored in reducedDims slot in x

Thanks!


cmd/go: test binaries ignore -short and -v options

go test -c produces binary that ignores -test.short and -test.v
What version of Go are you using (go version
go version go1.11.1 darwin/amd64
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/dobegor/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/dobegor/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/9h/8ghw84gx02l6__ryvxp4yzwh0000gn/T/go-build622504579=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
m_test.go:
package main

import (
	"fmt"
	"testing"
)

func TestBug(t *testing.T) {
	if testing.Short() {
		t.Skip("skipping test")
	}

	fmt.Println("this should not happen")
}

Then:
go test -c -test.v -test.short -o test
./test

What did you expect to see?
=== RUN   TestBug
--- SKIP: TestBug (0.00s)
    m_test.go:10: skipping test
PASS
ok  	temp	0.005s

What did you see instead?
this should not happen
PASS

You're misusing the -c flag - you should instead do:
go test -c -o test
./test -test.short -test.v

Perhaps the -c flag should give an error if any other test flags are given but take zero effect, since the tests aren't actually run.

I've forgot to replace the command in my steps to reproduce. I did use -test.short with same result. I'll update issue description.

Also, I don't think returning an error if -c flag is set is a right solution - there's no reason -test.short flag shouldn't work in a compiled binary.
That also breaks the use case of IDEs - in my case GoLand - it compiles the test via -c flag and runs it via go tool test2json to obtain a parse-able test output in order to correctly display it.

What I meant is that go test -c -short should be an error, since the -short doesn't actually do anything there. It should be used on the binary itself.
When you update the commands you ran to encounter the issue, please include a Go source file or repository to reproduce the problem too. I can't reproduce it on a package of mine.

I've figured out that passing -test.short to the compiled binary actually works. I'll close this issue as it's incorrect.
Thanks for pointing out!


SDL2

How I can use SLD2 function in my wr program?
How creating class graphics with .init .done .draw_line .draw_pixel .set_color etc.

take a look at https://domeengine.com - it does this already and you can look at the source code for some ideas.
You can get started by embedding the vm and making native functions you can call from wren, see the documentation here for how. https://wren.io/embedding/


Support interfaces for all intermediary types in polymorphic chains

This is needed because for the case A->B->C if we receive a response which contains an array of B's we won't deserialize correctly ( we will treat everything as B's so we will loose information for C's ).


Rule fails saying that "moduleA" which is imported by "moduleB" using as an aliased relative import is not imported

Label Type: question
Issue Type: Request for clarification
Environment
Linux Mint 21.3 (kernel 5.15.0-117-generic), running Python 3.10.12 using pytestarch == 3.0.1
Background

My project path is /home/developer/python/snakes-and-ladders-engine and it has the folder structure:

├── src
│   ├── game
│   │   └──  game.py
│   └── utils
│       └── gameutilities.py
└── tests
    └──  architecture
        └── test_architecture.py

The gameutilities.py module contains only methods - no classes are defined in the file.

game.py imports gameutilities.py as

from ..utils import gameutilities as GameUtilities

In test_architecture.py, a pytest fixture returns an evaluable architecture.

@pytest.fixture
def game_architecture_instance(project_root: Path, src_root: Path) -> EvaluableArchitecture:
    return get_evaluable_architecture(str(project_root), str(src_root))

Question
I am trying to define the rule "gameutilities should be imported by game" using a pytest test.
def test_utilities_invocation_rule(game_architecture_instance:  EvaluableArchitecture) -> None:
    utils_name: str = "snakes-and-ladders-engine.src.utils.gameutilities"
    game_name: str = "snakes-and-ladders-engine.src.game.game"

    utilities_invocation_rule: Rule = (
        Rule().modules_that()
        .are_named(utils_name)
        .should_only().be_imported_by_modules_that()
        .are_named(game_name)
    )

    utilities_invocation_rule.assert_applies(game_architecture_instance)

This fails with the result
AssertionError: "snakes-and-ladders-engine.src.utils.gameutilities" is not imported by "snakes-and-ladders-engine.src.game.game".

Other functional tests under the test tree work fine, and only the architecture test seems to be failing.
Is the rule misconfigured? What am I doing wrong here? Any support on how to fix this error is much appreciated - thank you!
I have attached the trace of the error as a text file for reference.
err_trace.txt

Hi,
thanks for the detailed setup description, that made it very easy to reproduce!
The issue lies with the import: With a statement like "from ..X import ..." --> the X is defined as the imported module.
So if you set utils_name = "test_games.src.utils", the test passes - but that's not quite what you want to test, I think.
To achieve what you're going for, I'd recommend specifying the import as absolute: import src.utils.gameutilities as GameUtilities. In my test setup, this results in a green test.
Does that solve your problem?

Hello @zyskarch
Thank you for your prompt and helpful response.
Changing the relative import to an absolute import (import src.utils.gameutilities as GameUtilities) fixed the problem.
Could the point about discouraging relative imports be added (maybe as a footnote or similar) to the README? It might be helpful for other users. Thank you for your consideration.
I'll close this query as resolved.


[Details to Grid View Converter] Does not work

Extension name
Details to Grid View Converter
Bug Description
I have done a fresh install of Playnite, Deleted all my settings, and I am unable to get Details to Grid View Coonverter to work with Harmony. I have gone into the extension settings and checkboxed it. The extension is also the ONLY  extension on my system. I wanted to make sure there was no possibilities of there being n incompatibility. As well as have restarted Playnite several times just in case.
To Reproduce
No response
Logs
playnite.log
extensions.log
Screenshots

You are in Details View. You need to switch to Grid View by pressing here:

Also make sure to increase the Grid View details Panel width in Playnite settings.


Qns about experiment result

Hi!
I noticed that you have an experiment log file: /data/stp_rl_pursuer_6_6_4_1_.txt.
This single target pursuit problem set up. world_size = 6, n_pursuers = 4 and n_evaders = 1. May i ask

And how many epoch you recommended for training? the default 1e5?
What is the fov_scope under this scenario?
What is the difference between stp_6_6_4_1_.txt and sop_40_40_4_1_.txt other than the size of map become bigger?

Thank you for your help!

Hi! I noticed that you have an experiment log file: /data/stp_rl_pursuer_6_6_4_1_.txt.
This single target pursuit problem set up. world_size = 6, n_pursuers = 4 and n_evaders = 1. May i ask

And how many epoch you recommended for training? the default 1e5?
What is the fov_scope under this scenario?
What is the difference between stp_6_6_4_1_.txt and sop_40_40_4_1_.txt other than the size of map become bigger?

Thank you for your help!

Hi,

First, The stp_rl_pursuer_6_6_4_1_.txt is an outdated result that is trained with A2C using a different reward function. This result is abandoned because we found that the reward function used is not suitable for the training with more than one evaders. Hence, we change that reward function to the current one, which performs good for the cases of multiple evaders.

As for the training epochs, for the simplest case of 4 pursuers vs. 1 static evader, 1000 training epochs are sufficient to observe the convergence. But for complex cases, such as with more evaders, stronger evaders, and in bigger environments, more epochs are needed. For example, for multiple static or random-walking evaders, 5000 epochs are sufficient for A2C.

The for_scope is always 11, the default value.

sop_40_40_4_1_.txt is the newest result with the current reward function.

Best regards!

Thank you for your answer! :) :+1:


Bug: Start menu covers the app switcher

If you have the start menu open, the App switcher appears beneath it:

There's no way to fix this. The windows don't allow any other window to appear on top of the start menu.


Fix: Fyle Redirect URL

Summary by CodeRabbit

Refactor

Cleaned up import statements in the application's base model.
Removed unused imports to streamline codebase.
Updated a URL template string for consistency and accuracy.

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (c06443b) 71.77% compared to head (75ba856) 71.77%.

Files
Patch %
Lines

apps/sage300/exports/base_model.py
80.00%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #121   +/-   ##
=======================================
  Coverage   71.77%   71.77%
=======================================
  Files          79       79
  Lines        3451     3451
=======================================
  Hits         2477     2477
  Misses        974      974

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[FLINK-24551][runtime] Read BUFFER_DEBLOAT_SAMPLES from task manager …

…configuration instead of task configuration

What is the purpose of the change
This PR fixes reading BUFFER_DEBLOAT_SAMPLES setting from the correct configuration
Brief change log

Read BUFFER_DEBLOAT_SAMPLES from task manager configuration instead of task configuration

Verifying this change
There are no tests for reading configuration
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

The motivation of the fix: Right now, you can read debloat configuration only from taskManagerConf the same as many other properties from TaskManagerOptions . So it is impossible right now to configure buffer debloat via taskConfiguration.
Unfortunately, our configuration is confusing. Some settings can be configured through StreamExecutionEnvironment or taskConfiguratioon while others are not and sometimes it is understandable but in most cases, I don't see any explicit reason why we can not do so.
Perhaps, it would be good to configure buffer debloat on the job level( StreamExecutionEnvironment) but it requires further investigation and separate ticket. This PR just fixes one setting in the same way as other buffer debloat settings to give an opportunity to configure it at least somehow.

thanks @akalash for clarification!

@akalash , do you need to put this fix into master as well?


Update package metadata

Summary of changes
This implements PEP 621, obviating the need for setup.py. The build backend hatchling (of which I am a maintainer) is quite stable and actively developed, the only reason the version is 0.x is because 1.0.0 will drop support for Python 2. It's also on the major distribution channels such as Debian, Fedora, Arch Linux, conda-forge, Nixpkgs, etc.
In comparison to setuptools, Hatchling has reproducible builds, supports the modern PEP 660 editable installation standard, has a more intuitive plugin system, and the codebase is an order of magnitude smaller thus making it more auditable and less prone to bugs.
Notes

As requested offline, the compact table format is used like https://github.com/tox-dev/pyproject-fmt/blob/main/pyproject.toml
In future, we should remove setup.cfg

black needed upgrading https://github.com/psf/black/pull/2966

Should I just remove the pyproject-fmt hook?

Sure :+1:


tpm0 is hard to use inside the container without a pre-profived abrm

Required information

Distribution: Ubuntu
Distribution version: 22.04
The output of "lxc info" or if that fails:
$ lxc info | grep version
api_version: "1.0"
driver_version: 5.0.1 | 7.1.0
kernel_version: 5.15.0-52-generic
os_version: "22.04"
server_version: "5.8"

Issue description
Using tpm in the container is hard. With VMs there is kernel provided abrm provided as /dev/tpmrm0.
Lxd sets up tpm0 without an access broker / resource manager. Which then leads down the rabbit hole of trying to create one inside the container.
Steps to reproduce

lxc profile device add default tpm tpm path=/dev/tpm0
lxc launch ubuntu:kinetic
lxc exec apt install tpm2-abrmd

Information to attach
tpm2-abrmd.service fails to start because of multiple issues:

/dev/tpm0 is root:root, whereas it is expected to be use tss:root
and dev-tpm0.device is not present inside systemd / udevd's world view

# systemctl list-units --no-pager --full --plain --no-legend '*.device'
dev-mapper-vg0\x2dvol0.device         loaded activating tentative /dev/mapper/vg0-vol0
sys-devices-virtual-net-eth0.device   loaded active     plugged   /sys/devices/virtual/net/eth0
sys-subsystem-net-devices-eth0.device loaded active     plugged   /sys/subsystem/net/devices/eth0

It it was inside container' udevd, then udev rule would have processed things correctly
# cat /usr/lib/udev/rules.d/60-tpm-udev.rules
# tpm devices can only be accessed by the tss user but the tss
# group members can access tpmrm devices
KERNEL=="tpm[0-9]*", TAG+="systemd", MODE="0660", OWNER="tss"
KERNEL=="tpmrm[0-9]*", TAG+="systemd", MODE="0660", OWNER="tss", GROUP="tss"

and start as normal.
Please provide abrm setup by the tpm, such that inside the container /dev/tpm0 and /dev/tpmrm0 are simply "there". Alternatively, please help to inject /dev/tpm0 into containers' udevd world view to allow running resource manager within the container.
I'm not sure how swtpm chardev thing works, and if it should be extended to provide chardev /dev/tpmrm0 as well?

Oooh, there is
dev-tpm0.mount                          loaded active     mounted   /dev/tpm0
i wonder if i can change tpm2-abrmd.service manager to respond to that as well.  and udev rules.

acutally, on the host I can see that there is host provided kernel tpmrm added with every container tpm device.
Can lxd please bind mount that into the container?
KERNEL[716987.515733] add      /devices/virtual/tpm/tpm3 (tpm)
KERNEL[716987.515767] add      /devices/virtual/tpmrm/tpmrm3 (tpmrm)
UDEV  [716987.517201] add      /devices/virtual/tpm/tpm3 (tpm)
UDEV  [716987.517844] add      /devices/virtual/tpmrm/tpmrm3 (tpmrm)


Add VR in Implicit Dataset?

Since we have the vrCallback now, would it make sense to add vr to the element within readDicomElementImplicit with the return value from the vrCallback (if provided)? I can do this.

I really want to keep this library focused on parsing DICOM P10 and returning exactly what was parsed without any changes to it.  The idea behind the src/util directory is to provide additional functionality on top of a parsed data set.  Would it meet your needs if you put the function to add VR's to implicit data sets in the src/util directory?

Nah, I can see this being more application logic then, so I'll keep it separate from the library.


Use NodeJS Action instead of TravisCI

Looks like Travis is a bit broken, and I don't have admin access to the repo over there. For now, as I have admin access to this repo here, let's try out GitHub Actions to replace the old CI.
TODO:

[ ] Re-add the Docker publishing
[ ] Create the npm run save-coverage script
[ ] Disable Travis

Coverage remained the same at 97.135% when pulling f7305393602a8fde33c385d0f95f32253855bb26 on dev/github-actions into 41f8444d17b2b282c437b5f871f0252968c03525 on master.


[wait for #1750] [layer] Support filter masking in mol attention

Add support for filter based masking in mol attention.
Add corresponding unittest.
Resolves #1713
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1758. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1758-202112021832100.44086790084839-6325e3634ed7c7db00c469b863a77236878933d0/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1758-202112022355250.66123604774475-afc237b15255c1b36efe5ddb78eb3f6cdb79f9cc/.

V2:
Added support for multiple outputs and corresponding unittests.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1758-202112031604050.18782591819763-66fd186ac5483ca17ac2a637640e1021f79c7d3a/.


Add tests and automated publishing the package to NPM

I created this PR on top of https://github.com/single-spa/import-map-overrides/pull/87
In the additional commit, I introduced automated publishing of the package. It uses the changesets/action action. And works in the following way:

As a contributor, when I open a PR I always have to remember to add the corresponding changeset (by using pnpm changeset)
The PR itself will run a job to build, lint and test the code. The maintainers of this repository can add this job as a required job to be able to merge the PR. In this way, if a PR does not pass any of the steps (build, check-format, lint, test) it won't be possible to merge it
When the PR is merged to the main branch, the changesets/action will find that there is a changeset file, and will open a PR to publish the package. This PR is the result of running pnpm changeset version on the code in the main branch.
If other PRs are merged from other contributors, the changesets files will accumulate and the PR opened at step 3 will be updated including all contributions
Once the maintainers are ready to release the package, they just have to merge the release PR. This time, the changesets/action, seeing that there is no changeset file anymore, will publish the package to NPM and create a release with the corresponding release notes in GitHub

For this to work it is necessary that:

The GITHUB_TOKEN is granted permissions to create and approve pull requests. This can be done in Settings -> Actions -> General -> Workflow permissions -> Allow GitHub Actions to create and approve pull requests
The maintainers create an NPM_TOKEN secret with an NPM token with write permissions to the organization (or at least the corresponding package, for publishing).

Closed in favour of https://github.com/single-spa/import-map-overrides/pull/96


Docker mkdir permission error

Describe the bug
Following the example docker run command in the README errors: "PermissionError: [Errno 13] Permission denied: '/label-studio/data/media'"
To Reproduce
Steps to reproduce the behavior:
docker pull heartexlabs/label-studio:latest
docker run -it -p 8080:8080 -v $(pwd)/mydata:/label-studio/data heartexlabs/label-studio:latest
Expected behavior
Shouldn't error
Environment (please complete the following information):

OS: Ubuntu
Label Studio Version: heartexlabs/label-studio:latest

Additional context
Current version on docker looks like release v1.7.0

Cannot reproduce

I have a possibly related issue when I use docker compose up in 1.7.0
label-studio-170-app-1    | => Postgres is up.
label-studio-170-app-1    | ./deploy/docker-entrypoint.sh: Launching /label-studio/deploy/docker-entrypoint.d/app/30-run-db-migrations.sh
label-studio-170-app-1    | => Do database migrations...
label-studio-170-app-1    | => Database and media directory: /label-studio/data
label-studio-170-app-1    | => Static URL is set to: /static/
label-studio-170-app-1    | Traceback (most recent call last):
label-studio-170-app-1    |   File "/label-studio/label_studio/manage.py", line 14, in <module>
label-studio-170-app-1    |     runserver.default_port = settings.INTERNAL_PORT
label-studio-170-app-1    |   File "/usr/local/lib/python3.10/dist-packages/django/conf/__init__.py", line 82, in __getattr__
label-studio-170-app-1    |     self._setup(name)
label-studio-170-app-1    |   File "/usr/local/lib/python3.10/dist-packages/django/conf/__init__.py", line 69, in _setup
label-studio-170-app-1    |     self._wrapped = Settings(settings_module)
label-studio-170-app-1    |   File "/usr/local/lib/python3.10/dist-packages/django/conf/__init__.py", line 170, in __init__
label-studio-170-app-1    |     mod = importlib.import_module(self.SETTINGS_MODULE)
label-studio-170-app-1    |   File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
label-studio-170-app-1    |     return _bootstrap._gcd_import(name[level:], package, level)
label-studio-170-app-1    |   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
label-studio-170-app-1    |   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
label-studio-170-app-1    |   File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
label-studio-170-app-1    |   File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
label-studio-170-app-1    |   File "<frozen importlib._bootstrap_external>", line 883, in exec_module
label-studio-170-app-1    |   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
label-studio-170-app-1    |   File "/label-studio/label_studio/core/settings/label_studio.py", line 6, in <module>
label-studio-170-app-1    |     from core.settings.base import *
label-studio-170-app-1    |   File "/label-studio/label_studio/core/settings/base.py", line 399, in <module>
label-studio-170-app-1    |     os.makedirs(MEDIA_ROOT, exist_ok=True)
label-studio-170-app-1    |   File "/usr/lib/python3.10/os.py", line 225, in makedirs
label-studio-170-app-1    |     mkdir(name, mode)
label-studio-170-app-1    | PermissionError: [Errno 13] Permission denied: '/label-studio/data/media'
label-studio-170-app-1 exited with code 1

No, it's not a good idea. Please check the changelog before upgrade to a new version:

No, it's not a good idea. Please check the changelog before upgrade to a new version:

Hmm, I didn't upgrade, this is my first time trying out label studio. I tried 1.6 after and docker compose had the same issue when running on Ubuntu 22.04 and using docker compose.

Got same problems after fresh install, but the command fixes it :+1:

Same

https://labelstud.io/guide/install.html#PermissionError-Errno-13-Permission-denied-label-studio-data-media

Cannot reproduce

I can reproduce so this issue is still relevant. So the main problem is you can't reproduce this issue because you doesnt read issue description?


Remove deprecated code

Bug/issue #, if applicable:
Summary

Remove unnessary available check
Clean up deprecated methods and declaration

Should SwiftDocC target be depended by other clients? It seems we have too much deprecated mark everywhere in that target.
If it should, I think maybe we can have some kind of version release, so that we can one day remove those deprecated code.
If it should only be intended for the repo's internal use, I can we could safely delete them.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

@swift-ci test

@swift-ci please test

@swift-ci Please test

Hi @Kyle-Ye, thanks for opening this!

Should SwiftDocC target be depended by other clients?

Yes, this allows for integrations with other tools and IDEs. For example, using these APIs: https://github.com/apple/swift-docc/blob/main/Sources/SwiftDocC/SwiftDocC.docc/SwiftDocC/Communication.md
Regarding the deprecated public APIs, I personally think it's still too early to remove them as some clients might be relying on them. I would say that we should wait at least a release before removing them.
PS: I see that this branch was pushed to this repository—I don't think we have this currently documented, but I think we should prefer keeping the branches in the apple/swift-docc repository to track releases and push development branches to forks, to align with the workflow of contributors that don't have write access.

Hi @Kyle-Ye, thanks for opening this!

Should SwiftDocC target be depended by other clients?

Yes, this allows for integrations with other tools and IDEs. For example, using these APIs: main/Sources/SwiftDocC/SwiftDocC.docc/SwiftDocC/Communication.md
Regarding the deprecated public APIs, I personally think it's still too early to remove them as some clients might be relying on them. I would say that we should wait at least a release before removing them.
PS: I see that this branch was pushed to this repository—I don't think we have this currently documented, but I think we should prefer keeping the branches in the apple/swift-docc repository to track releases and push development branches to forks, to align with the workflow of contributors that don't have write access.

Got it, I'll adjust the PR to only change the "doc" update and "available check" removal.
And sorry for interrupting the branches on the repo, I'll remove it and create a new PR from my fork branch.


Added a background switcher

This background switcher was made because someone changed the background color so I modified the dark mode generator to a background switcher.
Issue number: #4

@avinash201199 its done


Update Getting Started in Linux to reflect ANDROID_SDK bump

Updating Setting up the development environment in Linux documentation to reflect ANDROID_SDK version bump.
React Native is increasing the minimum ANDROID_SDK requirement from Android SDK Platform 31 to Android SDK Platform 33 for 0.72 onwards. Correspondingly changing the Android version. Updating our documentation to reflect the same as macOS & Windows
Found in https://github.com/facebook/react-native-website/pull/3605

Can we backport it also in the docs for 0.71?

Wasn't the whole point of this change that it's only needed 0.72 onwards ?

Can we backport it also in the docs for 0.71?

Wasn't the whole point of this change that it's only needed 0.72 onwards ?

We bumped to SDK 33 inside 0.71, so all the references has to be updated in .71 and on main (.72)


Bug: cannot target state in sibling parallel branch by ID

Bug or feature request?
Potentially a bug
Description:
When attempting to target a state by ID from one parallel branch in another parallel branch, the transition doesn't occur, but the onexit action does. Interestingly, the XState Visualizer draws an arrow correctly (but it does not work as intended).
(Bug) Expected result:
As far as I can understand in the SCXML spec (see 3.1.3 Parallel States, 2nd paragraph: https://www.w3.org/TR/scxml/#CoreIntroduction), it should be possible to target a state in a sibling parallel branch.
(Bug) Actual result:
Only the onexit action appears to occur. The intended transition does not work.
Link to reproduction or proof-of-concept:
https://codesandbox.io/s/festive-hamilton-0pr07

Click "Toggle Menu"
Click "Menu > Go to Dashboard"

This will log:
{"Pages":"About","Menu":"Closed"}
{"Pages":"About","Menu":"Opened"}
{"Pages":"About","Menu":"Closed"}

when what is expected is:
{"Pages":"About","Menu":"Closed"}
{"Pages":"About","Menu":"Opened"}
{"Pages":"Dashboard","Menu":"Closed"}

i.e., the event "go to dashboard": "#Dashboard" should be able to resolve the Dashboard state node, which has "id": "Dashboard".
As an aside, this works as intended in sketch.systems:
https://sketch.systems/matb33/sketch/eca5af52a7b0f8b6d3a2ed588708c817

Workaround: instead of "go to dashboard": "#Dashboard", fire an event:
import { send } from 'xstate';

// ...

"go to dashboard": {
  actions: send("TO_DASHBOARD")
},

// ...

Pages: {
  states: { /* ... */ },
  on: {
    TO_DASHBOARD: '.dashboard'
  }
}

Thanks for the workaround idea. That was something we tried as well, but the problem is that when the "Menu" is "Closed", our design dictates that one shouldn't be able to emit the "go to dashboard" event. Only when the "Menu" is "Opened" should we be able to emit that event.


[Bug]: Multiple timelapse merged if printing same file multiple times

What happened and in what context?
printed same file multiple times
next time after finished printing old one
If needed, paste the relevant bot log contents here.
sent to private
If needed, paste the relevant bot journal contents here.
No response
What branch does this occur on?
development
Reports only on latest master or development branches!

[X] I have updated to latest development/master version before submitting the bug

Can not confirm, each print gets ms timestamp


Master switch

This has a breaking change, in that connect params are now a user configuration object.
This PR allows Reactotron.connect({enabled: __DEV__}) for allowing Reactotron to remain benign in shipped builds.

Thank you.  I wanted to add some more options at the client level to opt in to things.  This paves the way.
👍


New Technologies

"New Technologies" represents a groundbreaking advancement in artificial intelligence, poised to revolutionize industries and redefine human-machine interaction. At its core, the platform harnesses state-of-the-art natural language processing (NLP) and machine learning algorithms to enable seamless communication and intelligent decision-making. This capability not only enhances efficiency across sectors like healthcare, finance, and education but also sets new standards for AI-driven solutions.
What sets "New Technologies" apart is its commitment to ethical AI development, ensuring fairness, transparency, and accountability in every algorithmic decision. This focus not only builds trust but also positions the platform as a leader in responsible AI deployment.
Contributors to "New Technologies" have the opportunity to shape the future of AI by collaborating within a dynamic community of innovators and researchers. They can leverage the platform's scalability and integration capabilities to create impactful solutions that address complex challenges on a global scale. By joining "New Technologies," individuals contribute to advancing technology while gaining recognition as pioneers in AI innovation. Together, we are unlocking the full potential of artificial intelligence to benefit society and drive transformative change across industries.

Let's discuss about the new technologies.


Update AUTHORS list

Add all authors from git log, sorting them alphabetically and preferring work email.

Long list:-)

Yes it's a well deserved update that was pending for too long :)


Unable To Export PDFs with Receipts

QuestPDF.Drawing.Exceptions.DocumentLayoutException: Composed layout generates infinite document. This may happen in two cases. 1) Your document and its layout configuration is correct but the content takes more than 250 pages. In this case, please increase the value QuestPDF.Settings.DocumentLayoutExceptionThreshold static property. 2) The layout configuration of your document is invalid. Some of the elements require more space than is provided.Please analyze your documents structure to detect this element and fix its size constraints.
   at QuestPDF.Drawing.DocumentGenerator.<RenderPass>g__ThrowLayoutException|6_1[TCanvas](<>c__DisplayClass6_0`1&)
   at QuestPDF.Drawing.DocumentGenerator.RenderPass[TCanvas](PageContext pageContext, TCanvas canvas, Container content, DebuggingState debuggingState)
   at QuestPDF.Drawing.DocumentGenerator.RenderDocument[TCanvas](TCanvas canvas, IDocument document)
   at QuestPDF.Drawing.DocumentGenerator.GeneratePdf(Stream stream, IDocument document)
   at QuestPDF.Fluent.GenerateExtensions.GeneratePdf(IDocument document, Stream stream)
   at QuestPDF.Fluent.GenerateExtensions.GeneratePdf(IDocument document, String filePath)
   at NickvisionMoney.Shared.Models.Account.ExportToPDF(String path, ExportMode exportMode, List`1 filteredIds, String password) in /run/build/org.nickvision.money/NickvisionMoney.Shared/Models/Account.cs:line 1561

https://github.com/NickvisionApps/Denaro/releases/tag/2023.5.0-rc1


Question: the maximum length for header fields.

From https://github.com/flashmob/go-guerrilla/wiki/Configuration-example:-save-to-Redis-&-MySQL:
CREATE TABLE IF NOT EXISTS `new` (
  `mail_id` BIGINT(20) unsigned NOT NULL AUTO_INCREMENT,
  `message_id` varchar(256) character set latin1 NOT NULL COMMENT 'value of [Message-ID] from headers',
  `date` datetime NOT NULL,
  `from` varchar(256) character set latin1 NOT NULL COMMENT 'value of [From] from headers or return_path (MAIL FROM) if no header present',
  `to` varchar(256) character set latin1 NOT NULL COMMENT 'value of [To] from headers or recipient (RCPT TO) if no header present',
  `reply_to` varchar(256) NULL COMMENT 'value of [Reply-To] from headers if present',
  `sender` varchar(256) NULL COMMENT 'value of [Sender] from headers of present',
  `subject` varchar(255) NOT NULL,
  `body` varchar(16) NOT NULL,
  `mail` longblob NOT NULL,
  `spam_score` float NOT NULL,
  `hash` char(32) character set latin1 NOT NULL,
  `content_type` varchar(64) character set latin1 NOT NULL,
  `recipient` varchar(255) character set latin1 NOT NULL COMMENT 'set by the RCPT TO command.',
  `has_attach` int(11) NOT NULL,
  `ip_addr` varbinary(16) NOT NULL,
  `return_path` VARCHAR(255) NOT NULL COMMENT 'set by the MAIL FROM command. Can be empty to indicate a bounce, i.e <>',
  `is_tls` BIT(1) DEFAULT b'0' NOT NULL,
  PRIMARY KEY  (`mail_id`),
  KEY `to` (`to`),
  KEY `hash` (`hash`),
  KEY `date` (`date`)
) ENGINE=InnoDB  DEFAULT CHARSET=utf8mb4;

It assumes Message-ID, From, and To have at most 256 characters, but I see no maximum limits here in the header-parsing step. What happens if it exceeds 256?
https://github.com/flashmob/go-guerrilla/blob/b803e062a62e3103cbf238486f02c275c886f26a/mail/envelope.go#L116-L146

How do I set the field size limits so that it could “validate” incoming emails and throw the invalid out to begin with?

If the headers are folded (using white-space-folding) then there's really no limit.
However, there is an overall limit for how long a header block can be - that's hard-coded to 4kb here: https://github.com/flashmob/go-guerrilla/blob/master/mail/envelope.go#L30
Ideally, This should be changed to be a configurable option.


[WIP] SCF Potential

Found a small bug, but not yet where it lives. Elements of the S T matrices are 10^5 times larger than in gala. Their ratios are correct, just with that additional large scaling.


Question about composer.json

First of all, thank you for creating this image, it's super handy!
Secondly, I have a question about composer.json, I'd like to use my own composer.json file.  I've looked at dependencies.sh and I think the best approach for me would be to just create my own Dockerfile based on your image.
There are some things I need (VCS repo definitions for example) which I think will only work if I run composer against my own file.

Thank you!
You can add dependencies from other git repos as shown in the README:
https://github.com/urbantrout/craftcms#plugins
If you need further modifications on your compser.json you could add this file as a volume in your docker-compose.yml. maybe that helps.
    volumes:
      - ./composer.json:/var/www/html/composer.json

Yes, the dependency handling is cool, but I have some additional config in my composer.json file.  However, I didn't see any place in your existing code where composer install is being executed.
For example, if I add the volumes line you mentioned, what will actually consume it?  My idea was to create a new Dockerfile based off of yours which do exactly that.

Maybe I should add the possibility to execute shell scripts after start.
Similar to the db dump but instead of import the file to a DB they just get executed.

Hey 👋
I’m also trying to add additional changes to composer.json. Also I’m not quite sure how to add dependencies with versions:
From the README:
    volumes:
      - ./path/to/plugin:/plugins/plugin # If you want a local plugin for composer

    environment:
      DEPENDENCIES: >- # additional composer packages (must be comma separated)
        craftcms/redactor,
        [vendor/package-name:branch-name]https://url-to-the-git-repo.git # Branch name should be prefixed with dev-${branchname}
        [vendor/package-name:version]/path/to/volume # Version as 1.0.0

Does this mean this is how I’d add a package with a version?
    volumes:
      - ./path/to/plugin:/plugins/plugin # If you want a local plugin for composer

    environment:
      DEPENDENCIES: >- # additional composer packages (must be comma separated)
        craftcms/cms:3.0.11,
        craftcms/redactor:2.0.1

@maxhoffmann yes, you can install a specific version of a composer package with the syntax vendor/package-name:version. But this does not seem to work with craftcms/cms. This is due to the fact that the image comes preinstalled with Craft CMS. You can upgrade the CMS in the Control Panel but now downgrade it.

I'd like to add a question here if I may?
I'm working on a base install and I have the following in the docker-compose file.
...
environment:
  DEPENDENCIES: >- # additional composer packages (must be comma separated)
    yiisoft/yii2-redis,
    craftcms/redactor:2.1.6,
    craftcms/aws-s3,
...

Trying to run this from scratch I get the following error.
craft_1     | ==> ✅  All dependencies successfully installed.
craft_1     | ==> Activating Plugins.
craft_1     | *** installing aws-s3
postgres_1  | 2018-11-09 15:05:44.412 UTC [27] ERROR:  relation "plugins" does not exist at character 117
postgres_1  | 2018-11-09 15:05:44.412 UTC [27] STATEMENT:  SELECT "id", "handle", "version", "schemaVersion", "licenseKey", "licenseKeyStatus", "settings", "installDate"
postgres_1  | 	FROM "plugins"
postgres_1  | 	WHERE "handle"=$1
postgres_1  | 	LIMIT 1
craft_1     | *** failed to install aws-s3: SQLSTATE[42P01]: Undefined table: 7 ERROR:  relation "plugins" does not exist
craft_1     | LINE 2: FROM "plugins"
craft_1     |              ^
craft_1     | The SQL being executed was: SELECT "id", "handle", "version", "schemaVersion", "licenseKey", "licenseKeyStatus", "settings", "installDate"
craft_1     | FROM "plugins"
craft_1     | WHERE "handle"='aws-s3'
craft_1     | LIMIT 1
craft_1     |
craft_docker_craft_1 exited with code 1

Am I overlooking something obvious?

The docker image now updates all the composer dependencies on startup. This includes Craft CMS itself.

@matthewcummings If you add a volume for a plugin you can install this plugin via the docker-compose.yaml by adding this line to the DEPENDENCIES array:
[vendor/package-name:version]/path/to/volume # Version as 1.0.0


Add simple mail submit

I added mailho, it is extremely useful for local development (running on port 8025):

I can confirm it works like it was, without any code changes required.
Trick was in reading documentation, digging in code and finding out what is correct MAILER_DSN value :-)
MAILER_DSN=mailgun+api://<apiKey>:mail.getrector.org@default?region=eu

@TomasVotruba everything is prepared on production - if you merge this, it will work on getrector.org


Fixed no survey chat engagement end

During converting from if statements to when statements it seems that one close condition got modified a little that changed the logic in case of no surveys and manual end engagement on android side.

During converting from if statements to when statements it seems that one close condition got modified a little that changed the logic in case of no surveys and manual end engagement on android side.

@Pelkar I checked Git history and there was no viewCallback?.destroyView() inside the onSurveyLoaded() function, it seems. So I don't think the reason is identified correctly. That's why I'm concerned if we don't brake anything with this change. Could you elaborate on how you came up with this conclusion?

During converting from if statements to when statements it seems that one close condition got modified a little that changed the logic in case of no surveys and manual end engagement on android side.

@Pelkar I checked Git history and there was no viewCallback?.destroyView() inside the onSurveyLoaded() function, it seems. So I don't think the reason is identified correctly. That's why I'm concerned if we don't brake anything with this change. Could you elaborate on how you came up with this conclusion?

There needs to be a case that works outside shouldHandleEndedEngagement, because it is true when we need to show engagement ended dialog. However when the user triggers end engagement there is no need to the dialog. In our case, the when block was entered, but nothing was done about the view being showed and thus an empty chatview remained behind.
The possible cases are:
Has survey -> open survey
Is not user triggered and needs to handle ended engagement and no survey -> show ended dialog
Is user triggered and no survey -> close view

!squash


Can't 'rkt trust' the CoreOS public key on a Mac with Vagrant

I have a problem during my first steps with rocket.
I Installed vagrant-ubuntu-trusty-64 on a Mac and Successfully downloaded and ran the rkt binary:
vagrant@vagrant-ubuntu-trusty-64:~/rkt-v0.5.3$ sudo ./rkt version
rkt version 0.5.3
appc version 0.5.1

No walking through first steps at https://github.com/coreos/rkt#rkt-basics. But the rkt trust commands fails:
vagrant@vagrant-ubuntu-trusty-64:~/rkt-v0.5.3$ sudo ./rkt trust --prefix coreos.com/etcd
Error determining key location: --prefix meta discovery error: Get https://coreos.com?ac-discovery=1: dial tcp: i/o timeout

BTW: sudo ./rkt --insecure-skip-verify fetch coreos.com/etcd:v2.0.4 and run runs fine.

Does curl https://coreos.com work from this host? It is very odd that you are getting a dial timeout.

Yes curl https://coreos.com works.
BTW: specifying the key directly also seems to work:
https://github.com/coreos/rkt/blob/master/Documentation/commands.md#trust-a-key-from-specific-location
vagrant@vagrant-ubuntu-trusty-64:~/rkt-v0.5.3$ sudo ./rkt trust --prefix coreos.com/etcd https://coreos.com/dist/pubkeys/aci-pubkeys.gpg
Prefix: "coreos.com/etcd"
Key: "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg"
GPG key fingerprint is: 8B86 DE38 890D DB72 9186  7B02 5210 BD88 8818 2190
	CoreOS ACI Builder<EMAIL_ADDRESS>Are you sure you want to trust this key (yes/no)? yes
Trusting "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg" for prefix "coreos.com/etcd".

Very strange. @luebken is this reproducing consistently?

So I set up a new vagrant box with hashicorp/precise64 and it worked:
vagrant@precise64:~/rkt-v0.5.3$ sudo ./rkt trust --prefix coreos.com/etcd
Prefix: "coreos.com/etcd"
Key: "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg"
GPG key fingerprint is: 8B86 DE38 890D DB72 9186  7B02 5210 BD88 8818 2190

But I could reproduce the problem by setting another new vagrant box with ubuntu/trusty64 and it didn't work.
Is there anything I can help with? If not I would close this issue since I have a workaround for fooling around with rkt.  And future Mac / Vagrant users at least have this issue if they run into the same problem.

@luebken hmm, I wonder if this might be another symptom of https://github.com/coreos/rkt/pull/538#issuecomment-75123977 ? Could you try making that change to your Vagrantfile?

Yes that worked! \o/

Thanks for the pointers. A small section about the Vagrantfile would have helped me a lot. Added it here: https://github.com/coreos/rkt/pull/720


waypoint: 0.3.0 -> 0.3.1

Motivation for this change
Bump waypoint to 0.3.1
They changed the output of version so that also needed changing.
I've noticed there's a -autocomplete-install flag but it just adds complete -C /nix/store/blah-0.3.1/bin/waypoint waypoint to your .bashrc
I tried running /nix/store/blah-0.3.1/bin/waypoint waypoint but that just gave me the same output as help
If there's a way to extract that out to a completion file let me know
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

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

1 package built:

waypoint


Use java 8

Some projects using this may require java 8, however this does not work in java 8.

I will not use older Java versions. I make new projects compatible with the latest LTS version, nowadays Java 17, and Java 11 at the time of making this library.
Many systems have moved to newer Java versions, and Android desugars Java 11 neatly. Please use an alternative (e.g. Gson) if you really need to use Java 8.


Reminders

Proposal:
Proposal written by Paquimex
Is it a proposal for the website or for the bot?: bot
What is the title of your proposal?: Reminders
Explain your proposal more accurately (It's best to give as much information as possible, so that we can implement the proposal better): Being able to set up reminders and even birthdays to be sent at specific times/dates on a channel in the server
Why should we add this feature?: Because it's a nifty little feature that will help many players with MMO events or social events
ReportID: 184

➤ LenoxBot commented:
Approve:
nice
Nickkel#6515

➤ LenoxBot commented:
Approve:
nice
SaVaGeCrEePeR28 | Furkan#3532

➤ LenoxBot commented:
Approve:
agree with nickkel and furkan
Profxy#5396

added


Add bool as valid array dtype, fix dtype validation

See: #77, #79

Current coverage is 98.07% (diff: 71.42%)

Merging #83 into dev will decrease coverage by 0.18%

@@                dev        #83   diff @@
==========================================
  Files             7          7
  Lines          1036       1040     +4
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1018       1020     +2
- Misses           18         20     +2
  Partials          0          0

Powered by Codecov. Last update 6e24d34...9bbee88


Fix for 'previous definition of xDrop was here' breaks Liquid.

Immediately after updating to a commit of Huginn that incorporated the 'previous definition of xDrop was here' fix (commit 3bb7f7a06f5cc8d5ffa47a689d7f3387f9284823) Liquid templating broke across all Agents in all Scenarios.  Manually reverting to commit a6dbb9c787e0c814921ef7001491346b148d35e9 which preceeded that fix repaired Liquid templating functionality.  Please re-examine patch.

You definitely restarted Huginn after checking out that version?

(I'm trying it now.)

I just updated my Huginn installation to the newest version and Liquid is still working.  That's VERY odd.

I did - it was running in Screen so I ^C'd it, then went through the process list to kill the other processes still hanging around before checking out that commit and trying again.  Events injected with the ManualAgent set off TriggerAgent as expected, and then forcing a run of the agent networks that fed them started resulting in events being transmitted.

@dsander you haven't seen anything like this, right?

@cantino No, but maybe the liquid formatting we use is too basic?
@virtadpt Do you have an agent/scenario that fails and you can share?

I can share any of my scenarios - they all broke.  Let me pick out part of one for you...
Type: Twitter User Agent
{
  "username": "insoc",
  "include_retweets": "true",
  "expected_update_period_in_days": "365"
}

Type: Twitter User Agent
{
  "username": "priyom_org",
  "include_retweets": "true",
  "expected_update_period_in_days": "3650"
}

Type: Twitter User Agent
{
  "username": "thecruxshadows",
  "include_retweets": "true",
  "expected_update_period_in_days": "3650"
}

Type: Jabber Agent
{
  "jabber_server": "{% credential primary_xmpp_server %}",
  "jabber_port": "5222",
  "jabber_sender": "{% credential primary_xmpp_login %}",
  "jabber_receiver": "{% credential primary_xmpp_destination %}",
  "jabber_password": "{% credential primary_xmpp_password %}",
  "message": "Twitter alert from Lifeline: @{{user.screen_name}} - {{text}}",
  "expected_receive_period_in_days": "365"
}

Hmm that is working without issues on my installation, I have a Twitter agent sending my a message via XMPP. Do you get failed jobs or an exception of any kind?

Neither.  Just empty alerts via XMPP of the form
Twitter alert from Lifeline: @ -

That is SO strange.  And nothing in the logs?

Not that I can tell.  I can run some tests on Saturday, depending on how things go this morning...

I re-ran my tests - there is nothing in the logs that would suggest that an agent was throwing exceptions or anything like that.  I also updated my checkout to the latest commit from master/ and the same thing happened - Liquid templating stopped working, even when I injected crafted events into agent networks with ManualAgent.  I then stepped backwards in commits and tested each time, and it wasn't until I got back to commit a6dbb9c787e0c814921ef7001491346b148d35e9 that Liquid templating started working again.

@virtadpt I finally got the same bug! Twitter alert from Lifeline: @ - the strange thing is the events look almost the same: working, not working

Yes!  That's what I'm seeing over here!

Just tested.

Factoid - the event gets written into the database in a broken fashion:
19:29:58 web.1  |   SQL (0.7ms)  INSERT INTO `events` (`user_id`, `expires_at`, `payload`,
`agent_id`, `created_at`, `updated_at`) VALUES (1, '2015-04-30 19:29:58',
'{\"message\":\"The number of people listening to the BART Police Dispatch radio network
has peaked to !  You may wish to listen to it here: \\n\\n
http://www.radioreference.com/scripts/playlists/1/2776/0-5708229104.mp3\",
\"peak\":9.0,\"peak_time\":1429816768.0,\"grouped_by\":\"no_group\"}', 30, '2015-04-23
19:29:58', '2015-04-23 19:29:58')

@virtadpt you are right, 3bb7f7a is the cause for the issues you are having. I am working on a solution that both works in development and production.
As a workaround you can run in production or set config.cache_classes = true in config/environments/development.rb

Awesome!
When you're done, mind if I pick your brain on Gitter about how that breaks Liquid templating?  Is it because it's already been pulled in ("class Drop < Liquid::Drop") and pulling it in again puts it into an inconsistent internal state?

To be honest, I have no idea why it broke stuff.  3bb7f7a was created to fix the 'previous definition of xDrop was here'  warnings which happened when you changed files in development and rails (tried?) to reload the classes.
I noticed the same issue you are having yesterday night for the first time and was able to reproduce it today by re-emitting an event twice after restarting huginn. I guess the first time loaded the required liquid classes and the second time something was reloaded (due to cache_classes being set to false in development), but because AgentDrop wasn't redefined the liquid interpolating failed. Why there wasn't any exception is another mystery to me.
Maybe @knu has more insight why #770 broke liquid templating in development?

Oh by the way, you could try #771 which should fix the issue.

After another scheduler crash last night, I'm trying it on a new checkout right now.

Applying the suggested changes in #771 while running HEAD on a new checkout definitely fixed Liquid.

Fixed by cfe8482c0d6eca95e7324aa266caacccc4ffd46f

Looks good!


App Bridge v4 prevents the app from being installed

Hello, I updated the app bridge to version 4 and added new html tags to _document.js.
import { Head, Html, Main, NextScript } from "next/document";

export default function Document() {
  return (
    <Html lang="en">
      <Head>
        {/* Needed for Shopify app bridge v4 */}
        <meta
          name="shopify-api-key"
          content={process.env.NEXT_PUBLIC_SHOPIFY_API_KEY}
        />
        <script src="https://cdn.shopify.com/shopifycloud/app-bridge.js"></script>
      </Head>
      <body>
        <Main />
        <NextScript />
      </body>
    </Html>
  );
}

Unfortunately, it prevents me from installing the app on my dev store. I get the 404 page.

Once I remove the

I need some more info on this - logs, etc. I don't see how it would be possible for CDN to throw a 404 here.

Hold up. This is user error. This is not even remotely how I've implemented AppBridge CDN into the repo.
_document.js: https://github.com/kinngh/shopify-nextjs-prisma-app/blob/main/pages/_document.js
next.config.js: https://github.com/kinngh/shopify-nextjs-prisma-app/blob/main/next.config.js
If you're running your own migrations that's away from the scope of this repo, I recommend moving over to Shopify Community.


Merge after 0.21.33

I didn't realize the backport was from 0.23 instead of 0.22.  Mostly had to change some message.rest to finalBuffer.

I still haven't internalized where website should build and not these days.

I still haven't internalized where website should build and not these days.

wdym?

WTF.  I thought I saw a build failure on an undefined VERSION_0_23 reference.

You did :) it was in the CI job that ran on your fork, since you are probably missing the tags there.
https://github.com/rossabaker/http4s/runs/5608364123


Content not pausing

When another content starts playing, such as another audio stream or incoming voip call, the LBRY content is not paused.

@michaeltintiuc Thanks. At the moment, issues for the mobile app should be filed in the https://github.com/lbryio/lbry-android repository. The plan is to start using lbry-react-native for issue filing once we have releases for both iOS and Android.


WEBEU2 - Anuoluwapo Apiti - Alex E. - Week 1 Sprint Challenge

This PR contains:
Project Set Up

[x] Create a forked copy of this project.
[x] Add your project manager as collaborator on Github.
[x] Clone your OWN version of the repository (Not Lambda's by mistake!).
[x] Create a new branch: git checkout -b <firstName-lastName>.
[x] Implement the project on your newly created <firstName-lastName> branch, committing changes regularly.
[x] Push commits: git push origin <firstName-lastName>.

Minimum Viable Product
Your finished project must include all of the following requirements:
Home Page
Review the provided design file for the home page.  Notice the navigation and header images are missing.

[x] Build the HTML and CSS to create the missing navigation and header.
[x] Link the About navigation item to the about.html page

You will also notice there are 10 boxes on the home page that need background colors.  Use this list below to correctly style each box:

[x] box1: teal
[x] box2: gold
[x] box3: cadetblue
[x] box4: coral
[x] box5: crimson
[x] box6: forestgreen
[x] box7: darkorchid
[x] box8: hotpink
[x] box9: indigo
[x] box10: dodgerblue

About Page
Review the provided design file for the about page. You have been provided the HTML wrapper, footer, and page content for the about page. Create the rest of the missing HTML and CSS to match the design file.

[x] Copy and paste your home page navigation and header into the about page
[x] Update the header image with the about page image
[x] Link the Home navigation item back to the index.html page.
[x] Build the rest of the about page layout to match the design

Thank for all the feedback, and your amiable remarks 👏🏾. I hope to keep up with the pace 🙏🏾, and continually improve. 🔥Gracias!


[nnstreamer/apptest] script to build examples

To run app-test in nnstreamer repo, clone nnstreamer-examples and install the examples.
Signed-off-by: Jaeyun Jung<EMAIL_ADDRESS>

:memo: Version: 1.20190225. Thank you for submitting PR #475. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.