404 - Bad Request - On Creating Playlist

People seem to be getting a lot of 404 errors when the app tries to create a playlist on YTM...
I'm unable to replicate this... I was wondering if anyone would like to help me?
I don't suppose anyone getting this error and has landed here can copy me their HTTP request json body that gets sent to YTM when you create a playlist through YTM music?
I.e: Using a web browser (Firefox is good)...

Press F12 for the developer console
Within YTM, create a playlist (i.e. pick an uploaded song and use the 3 dots to add a new playlist)
After creating the playlist, I need the 'Request Payload' json string from the playlist/create endpoint.

I don't get the 404 error myself, so I'm wondering if the request payload might be different for different people?

Any help would be much appreciated.
Cheers,
J

Hey, Thank you for your great work; so about this issue; if it's any help, this worked just fine a couple of days ago, but now it seems to be failing, all the time; I followed what you said and this is the JSON it spits:
{"context":{"client":{"clientName":"WEB_REMIX","clientVersion":"0.1","hl":"en","gl":"SE","experimentIds":[],"experimentsToken":"","browserName":"Firefox","browserVersion":"84.0","osName":"Windows","osVersion":"10.0","platform":"DESKTOP","utcOffsetMinutes":60,"locationInfo":{"locationPermissionAuthorizationStatus":"LOCATION_PERMISSION_AUTHORIZATION_STATUS_UNSUPPORTED"},"musicAppInfo":{"musicActivityMasterSwitch":"MUSIC_ACTIVITY_MASTER_SWITCH_INDETERMINATE","musicLocationMasterSwitch":"MUSIC_LOCATION_MASTER_SWITCH_INDETERMINATE","pwaInstallabilityStatus":"PWA_INSTALLABILITY_STATUS_UNKNOWN"}},"capabilities":{},"request":{"internalExperimentFlags":[{"key":"force_music_enable_outertube_search","value":"true"}],"sessionIndex":"0"},"activePlayers":{},"user":{"enableSafetyMode":false}},"title":"Test 2","description":"","privacyStatus":"PRIVATE"}
Cheers.

Hey, Thank you for your great work; so about this issue; if it's any help, this worked just fine a couple of days ago, but now it seems to be failing, all the time; I followed what you said and this is the JSON it spits:
{"context":{"client":{"clientName":"WEB_REMIX","clientVersion":"0.1","hl":"en","gl":"SE","experimentIds":[],"experimentsToken":"","browserName":"Firefox","browserVersion":"84.0","osName":"Windows","osVersion":"10.0","platform":"DESKTOP","utcOffsetMinutes":60,"locationInfo":{"locationPermissionAuthorizationStatus":"LOCATION_PERMISSION_AUTHORIZATION_STATUS_UNSUPPORTED"},"musicAppInfo":{"musicActivityMasterSwitch":"MUSIC_ACTIVITY_MASTER_SWITCH_INDETERMINATE","musicLocationMasterSwitch":"MUSIC_LOCATION_MASTER_SWITCH_INDETERMINATE","pwaInstallabilityStatus":"PWA_INSTALLABILITY_STATUS_UNKNOWN"}},"capabilities":{},"request":{"internalExperimentFlags":[{"key":"force_music_enable_outertube_search","value":"true"}],"sessionIndex":"0"},"activePlayers":{},"user":{"enableSafetyMode":false}},"title":"Test 2","description":"","privacyStatus":"PRIVATE"}
Cheers.

Brilliant thanks for this.!!
Hmm, the only thing that seems to be different is the utcOffsetMinutes... I wondering if they're using that as some sort of security measure. I'll check.

Brilliant thanks for this.!!
Hmm, the only thing that seems to be different is the utcOffsetMinutes... I wondering if they're using that as some sort of security measure. I'll check.

I've uploaded a new version now. The utcOffsetMinutes in the request body will be populated with your local UTC offset. I'm pretty sure this won't fix the the issue, since I tried changing my own UTC offset to something invalid, and was still able to upload a playlist just fine... But the change is in anyway...
This version will however provide better diagnostic data, which will appear in your logs, and I'll get them if the 'Send Diagnostic Data' checkbox is checked in the application - or feel free to send them to me otherwise.
I'm keen to get to the bottom of this issue, as it's detrimental to the whole playlists feature... so watch this space.

I've uploaded a new version now. The utcOffsetMinutes in the request body will be populated with your local UTC offset. I'm pretty sure this won't fix the the issue, since I tried changing my own UTC offset to something invalid, and was still able to upload a playlist just fine... But the change is in anyway...
This version will however provide better diagnostic data, which will appear in your logs, and I'll get them if the 'Send Diagnostic Data' checkbox is checked in the application - or feel free to send them to me otherwise.
I'm keen to get to the bottom of this issue, as it's detrimental to the whole playlists feature... so watch this space.

Ah that's great! I tried to find where the actual log files are stored, but to no avail, but since you're getting them anyway, that sounds good, but if you want me to send you log files directly, I would gladly help ^^

Ah that's great! I tried to find where the actual log files are stored, but to no avail, but since you're getting them anyway, that sounds good, but if you want me to send you log files directly, I would gladly help ^^

Hey,
Thanks for your help @Aissasa... I believe I've fixed the issue now in this version:
https://github.com/jamesbrindle/YTMusicUploader/releases/tag/v1.6.6
Let me know how you get along (note, there was a mix-up somewhere in the caching process, setting the YTM entity ID as the video ID, so unfortunately, when you update to the newest version, it will automatically perform a full refresh of your library - Apologies for that but there was no other way around it).
Cheers,
J

Hey,
Thanks for your help @Aissasa... I believe I've fixed the issue now in this version:
https://github.com/jamesbrindle/YTMusicUploader/releases/tag/v1.6.6
Let me know how you get along (note, there was a mix-up somewhere in the caching process, setting the YTM entity ID as the video ID, so unfortunately, when you update to the newest version, it will automatically perform a full refresh of your library - Apologies for that but there was no other way around it).
Cheers,
J

It seems like it's working just fine, and I don't mind the refresh of the library ^^ Thank you so much for your hard work 💯 Much appreciated ^o^/

It seems like it's working just fine, and I don't mind the refresh of the library ^^ Thank you so much for your hard work 💯 Much appreciated ^o^/

Well, after spending a couple of hours trying to figure out what the issue was, I have determined that I'm using the correct 'videoIds' that YouTube / YouTube Music uses, and verified I'm create the request JSON properly...
Turns out though, that the site limits the amount of playlists created in a short amount of time to 10, and will then block subsequent playlists being created for 24 hours!
You can't even do this on the official site:

I believe this stems from other third parties trying to do something similar to what I'm doing and 'synching up' playlists over different platforms (like 'Spotify' to YTM').
What I'm going to do is implement a 'wait' mechanism... Waiting 10 seconds before each 'create playlists' and 2 seconds for each 'add to existing playlist'.... and see if that somehow works...
This may take a while to upload playlists consisting of many songs just to be aware, but if works around the issue it may be worth it.

Oh damn! Great catch 👍


Reworking decimal tests for DV Decimal compatibility

Most of the decimal specific tests aren't running when testing the SQL compiler because the range of decimal used was often 1.000000000000000000000001 which will round to 1 with a DV decimal.  Existing tests are duplicated and modified to be in a compatible range.  These tests are remaining here because they also work in C# (which the bigger range) and this ensures consistency.

✅ No public API change.

✅ No public API change.

✅ No public API change.


Pass params to layout

Hi,
As far as I searched there is no way to pass variables/params to layout.
By example the Email title should be included into layout. For this we need a way to pass it to layout.

All params passed to the template are passed to the layout as well. Just use those which apply on every place.

It doesnt seems to be the case.
		$result = $this->mailService->send('email/delivery/leave', [ 			'to' =><EMAIL_ADDRESS>			'template_params' => [ 				'user' => 7890, 				'title' => 'Comanda ta a plecat deja spre tine!' 			] 		]);
However, I am getting:
PHP Notice:  Undefined variable: title in /home/cristi/www/project/module/Frontend/view/layout/email.phtml on line 23
at that line is this:

<?php echo $title; ?>

Then it could be a bug. I'll need to check it

Overall a layout_params may be more appropriate, but your component is great enough that these are just very small improvements.

I'm affraid it cannot be implemented that way, because that would only make sense when using zend-view because of how it is implemented. If you are using any other renderer, that makes no sense.
I remember implementing the behavior which made params to be injected both in the layout and the template, but maybe there has been a regression.

Yep, it should be working.
Source: https://github.com/acelaya/ZF-AcMailer/blob/v7.3.1/src/View/MvcMailViewRenderer.php#L30
Test: https://github.com/acelaya/ZF-AcMailer/blob/v7.3.1/test/View/MvcMailViewRendererTest.php#L58
I'm going to do some manual testing and see what's going on.

I'm not sure what's going on, but it is working for me :thinking:
This is my layout (in application/mail/layout.phtml):

<?php echo $title ?>

<?php echo $content ?>

This is my template (in application/mail/template.phtml):
This is the content with user <?php echo $this->user ?>

This is my config:
return [
    'acmailer_options' => [

        /*
         * This is where you configure all the emails that can be sent by the application.
         * You can create new ones at runtime, but this eases predefining emails that do not need to change
         */
        'emails' => [
            'email/delivery/leave' => [
                'from' => '...',
                'encoding' => 'utf-8',
                'subject' => 'Testing layout params',
                'template' => 'application/mail/template',
                'template_params' => [
                    'layout' => 'application/mail/layout',
                ],
            ],
        ],

        'mail_services' => [
            'default' => [
               // ...
            ],
        ],
    ],
];

And this is how I send the email:
$result = $this->mailService->send('email/delivery/leave', [
    'to' => ['...'],
    'template_params' => [
        'user' => 7890,
        'title' => 'Comanda ta a plecat deja spre tine!'
    ]
]);

The resulting email contains the title inside the h1 tag.
Could you share your config with me? (without credentials). Just to make sure everything is in place and try to make a real test as similar as possible.

`<?php
return [
'acmailer_options' => [
    /*
     * This is where you configure all the emails that can be sent by the application.
     * You can create new ones at runtime, but this eases predefining emails that do not need to change
     */
    'emails' => [
		'base' => [
			'from' =><EMAIL_ADDRESS>			'from_name' => 'fsdfsdf',
			'bcc' =><EMAIL_ADDRESS>		],

		'email/delivery/leave' => [
			'extends' => 'base',
			'subject' => 'Comanda ta a plecat deja spre tine!',
			//'template' => 'email/delivery/leave',
			'template' => 'email/delivery/leave',
			'template_params' => [
				'layout' => 'layout/frontend/email',
			]
			//'template' => 'email/delivery-leave',
		],
    ],

    /*
     * This defines different services to send emails.
     * All of them will be capable of sending any of the previously configured emails.
     */
    'mail_services' => [
		'default' => [
			'transport' => 'smtp',
			'transport_options' => [
				'host' => 'mail.fdfdfsdsdsd.ro',
				'port' => 587,
				'connection_class' => 'login',
				'connection_config' => [
					'username' =><EMAIL_ADDRESS>					'password' => 'dgfgdfsdsdfsdfdsfds',
					'ssl' => 'tls',
				],
			],
		],

        ],
    ],

],

];`

Working too...
The only thing that comes to my mind is that there's some external element modifying the params somehow.
Do you have any mail listener configured? Do you have any listener on the template renderer that could be interfering?
I would recommend you to do an isolated test just to make sure. I have been using a fresh skeleton application, where I have only installed this module. Then in the IndexController I have injected the mail service and sent the email just before returning the ViewModel.
namespace Application\Controller;

use AcMailer\Service\MailServiceInterface;
use Zend\Mvc\Controller\AbstractActionController;
use Zend\View\Model\ViewModel;

class IndexController extends AbstractActionController
{
    /**
     * @var MailServiceInterface
     */
    private $mailService;

    public function __construct(MailServiceInterface $mailService)
    {
        $this->mailService = $mailService;
    }

    public function indexAction()
    {
        $result = $this->mailService->send('email/delivery/leave', [
            'to' => ['...'],
            'template_params' => [
                'user' => 7890,
                'title' => 'Comanda ta a plecat deja spre tine!'
            ]
        ]);
        var_dump($result->getEmail()->getTemplateParams());

        return new ViewModel();
    }
}

The call to var_dump should print all the params, including the title.
If it works for you there, then we can keep investigating which external element is breaking this.

Thanks, I'll check.
Thank you for your time!

By the way, just came to my mind, but this was something which used to be broken and it was fixed in v7.2.1.
Make sure you have that version at least, because previous to that version, this didn't work.

Almost certainly this is/was the issue. I had 7.0 set on composer.json. Will confirm later today.

Ok, perfect. Let me know when you have checked.
Also, if you want to update, you should be good to do it to the latest stable version (I try to follow semver, so no BC breaks have been introduced). You can use the ^7.0 constraint in combination with the composer.lock file.
If you see that something does not work after the update, follow the CHANGELOG to see everything which has been changed.
And of course, if something does not work as expected, let me know.

Hey @cbichis, wad this finally the issue?

I can't actually confirm as the project is stuck on PHP 7.0 so I cant upgrade to version compatible with 7.1....

PHP 7.0 is getting out of support in a couple weeks. You should probably try to convince whoever can make the decision to update to PHP 7.2 ;-)

I'm closing this for now. If you get to update to PHP ^7.1 and still face this issue with a newer AcMailer version, just let me know.


Search column too

G'day Mate,
Great extension use it every day since I installed ADS, any chance you'd make it so I can search for colum names too? Would help heaps with my workload
Thanks

Hey Ben,
Do you mean that when you type a column name, in the search bar you want to see all tables that contains this column?

100% yeah you got it. If it's got a negative impact on the speed making it a configurable item could fix that.

Hey Ben,
I have added a setting that allows you to include column names to Tables.  You could try it in the new release 1.1.

When switching between database groups i get this error
The first group I select works no problems, change to another group get that error but if I switch back to the first group works fine.

Hey Ben,
It seems that there is some error during the query.
Can you run this at target database and back with results?
`-- Create a temporary table to store the results
CREATE TABLE #TableColumns (
SchemaName NVARCHAR(128),
TableName NVARCHAR(128),
ColumnList NVARCHAR(MAX)
)
-- Declare variables for dynamic SQL
DECLARE @schemaName NVARCHAR(128)
DECLARE @tableName NVARCHAR(128)
DECLARE @sql NVARCHAR(MAX)
-- Cursor to iterate through each table
DECLARE tableCursor CURSOR FOR
SELECT TABLE_SCHEMA, TABLE_NAME
FROM INFORMATION_SCHEMA.TABLES
WHERE TABLE_TYPE = 'BASE TABLE' -- Exclude views
-- Loop through each table and generate dynamic SQL
OPEN tableCursor
FETCH NEXT FROM tableCursor INTO @schemaName, @tableName
WHILE @@FETCH_STATUS = 0
BEGIN
SET @sql = 'INSERT INTO #TableColumns (SchemaName, TableName, ColumnList) ' +
'SELECT ''' + @schemaName + ''', ''' + @tableName + ''', ' +
'STUFF((SELECT '', '' + COLUMN_NAME ' +
'FROM INFORMATION_SCHEMA.COLUMNS ' +
'WHERE TABLE_SCHEMA = ''' + @schemaName + ''' ' +
'AND TABLE_NAME = ''' + @tableName + ''' ' +
'FOR XML PATH('''')), 1, 2, '''')'
EXEC sp_executesql @sql

FETCH NEXT FROM tableCursor INTO @schemaName, @tableName

END
-- Close and deallocate the cursor
CLOSE tableCursor
DEALLOCATE tableCursor
SELECT o.name AS name
, 'TABLE' AS type
, tableColumns.ColumnList AS DEFINITION
, ss.name AS schemaName
FROM sys.objects AS o
JOIN sys.schemas AS ss ON o.schema_id = ss.schema_id
JOIN #TableColumns as tableColumns on tableColumns.TableName = o.name
AND tableColumns.SchemaName = ss.name
WHERE type = 'U'
UNION ALL
SELECT DISTINCT o.name AS name
, o.type_desc AS type
, m.DEFINITION AS DEFINITION
, ss.name AS schemaName
FROM sys.sql_modules m
JOIN sys.objects o ON m.object_id = o.object_id
JOIN sys.schemas AS ss ON o.schema_id = ss.schema_id
-- Drop the temporary table
DROP TABLE #TableColumns`

Morning,
Yep as soon as I saw your code, my mind went straight to database collation. Something we have learnt to deal with on our end very quickly with our whitebox DBs. I had to add COLLATE DATABASE_DEFAULT to 3 places in your code.
Cannot resolve the collation conflict between "Latin1_General_CI_AS" and "SQL_Latin1_General_CP1_CI_AS" in the equal to operation.
`-- Create a temporary table to store the results
CREATE TABLE #TableColumns (
SchemaName NVARCHAR(128)
, TableName NVARCHAR(128)
, ColumnList NVARCHAR(MAX)
)
-- Declare variables for dynamic SQL
DECLARE @schemaName NVARCHAR(128)
DECLARE @tableName NVARCHAR(128)
DECLARE @Sql NVARCHAR(MAX)
-- Cursor to iterate through each table
DECLARE tableCursor CURSOR
FOR
SELECT TABLE_SCHEMA
, TABLE_NAME
FROM INFORMATION_SCHEMA.TABLES
WHERE TABLE_TYPE = 'BASE TABLE' -- Exclude views
-- Loop through each table and generate dynamic SQL
OPEN tableCursor
FETCH NEXT
FROM tableCursor
INTO @schemaName
, @tableName
WHILE @@FETCH_STATUS = 0
BEGIN
SET @Sql = 'INSERT INTO #TableColumns (SchemaName, TableName, ColumnList) ' + 'SELECT ''' + @schemaName + ''', ''' + @tableName + ''', ' + 'STUFF((SELECT '', '' + COLUMN_NAME ' + 'FROM INFORMATION_SCHEMA.COLUMNS ' + 'WHERE TABLE_SCHEMA = ''' + @schemaName + ''' ' + 'AND TABLE_NAME = ''' + @tableName + ''' ' + 'FOR XML PATH('''')), 1, 2, '''')'
EXEC sp_executesql @sql

FETCH NEXT
FROM tableCursor
INTO @schemaName
    , @tableName

END
-- Close and deallocate the cursor
CLOSE tableCursor
DEALLOCATE tableCursor
SELECT o.name AS name
, 'TABLE' AS type
, tableColumns.ColumnList AS DEFINITION
, ss.name AS schemaName
FROM sys.objects AS o
JOIN sys.schemas AS ss
ON o.schema_id = ss.schema_id
JOIN #TableColumns AS tableColumns
ON tableColumns.TableName = o.name COLLATE DATABASE_DEFAULT
AND tableColumns.SchemaName = ss.name COLLATE DATABASE_DEFAULT
WHERE type = 'U'
UNION ALL
SELECT DISTINCT o.name AS name
, o.type_desc AS type
, m.DEFINITION COLLATE DATABASE_DEFAULT AS DEFINITION
, ss.name AS schemaName
FROM sys.sql_modules m
JOIN sys.objects o
ON m.object_id = o.object_id
JOIN sys.schemas AS ss
ON o.schema_id = ss.schema_id
-- Drop the temporary table
DROP TABLE #TableColumns
`

Hey Ben,
I have updated the request. Should work now, please, try it out.

That works great thanks mate.


Upcoming software update on Casper

Description
CISL will  update the Casper software stack during the week of October 23-27 - we'll have to redo the entire spack-stack 1.5.0 (or 1.5.1, whichever is current) afterwards ...

HPC Systems Downtime October 23-27 to Refresh Casper and Perform Filesystem Migrations

September 25, 2023
All CISL HPC systems will be down for scheduled maintenance the week of October 23-27 to refresh software operating systems and perform filesystem migrations in preparation for Cheyenne retirement at the end of this year. CISL engineers will relocate a number of filesystem datasets during this outage - including /glade/work - to new hardware. We will update the operating system on Casper nodes to an OpenSUSE installation for better compatibility with Derecho.  Users are encouraged to evaluate the test Casper deployment through direct ssh login to a demonstration node, casper01.hpc.ucar.edu, to evaluate the new operating system environment.  For additional information see the #casper-users channel in our NCAR HPC Users Group Slack workspace.During this time, all Cheyenne, Casper, and Derecho compute nodes will be unavailable. Login nodes will be unavailable at the beginning of the outage window in order to allow for filesystem migrations. Scheduler reservations will be put in place to ensure that all user jobs have completed by October 23 as the downtime begins. Any jobs that are queued when the downtime begins will be retained for execution when the systems return to service.  Progress will be communicated through the Notifier system throughout the course of the outage.

https://arc.ucar.edu/articles/526

I updated the site config for<EMAIL_ADDRESS>and it was merged back into develop.


Hi, How to run maskcam with csi camera instead of USB Webcam?

Hi @HongDavis,
you need to add the --env MASKCAM_INPUT=argus://0 parameter if you're running on a docker container.
If you're running from Balena, add this as a variable from the dashboard.
Thanks for your interest!


code incompleted on npmjs.com/package/claude-ai

use "npm install claude-ai" will get a incomplete code project
dont have index.d.ts
startConversation dont call init

I'll release a new version with these fixes soon

I also found index.d.ts is not matched with index.js in main branch.
startConversation should return Conversation which include method sendMessage, index.d.ts dont have


Update events.md

Fixed blur and focus typo, descriptions are now correct.
@blur & @focus descriptions were originally the wrong way around.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove vertical pepper in the background

Hello,
I just started reading the lombok documentation. The vertical pepper in the background of every doc page hurts my eyes. I humbly suggest that it be removed.
As a workaround, I currently save the doc pages using Firefox and reopen the saved copy which does not display the pepper.

Pepper is indeed dangerous to the eyes. Be really careful with them. As a simpler workaround, you can use FF development mode and disable the rule #pepper {background-image: ...} or block https://projectlombok.org/features/lombok-background.png.


Notes are not synchronize with playnite

Since several versions, notes are no longer synchronized with playnite.
Thanks for all your hard work on LanCommander !

Fixed in latest version


errors + warnings if lib has foldername with underscore

Try to install this little sample Dart-App: https://cdn.rawgit.com/MikeMitterer/dart-material-design-lite/master/site/downloads/simple.zip and pub update
This gives you all kind of errors and warnings. The problem is the .pub/deps/debug/mdl/lib/_templates (probably the underscore) - after deleting this folder the errors are gone.

Atom 1.0.15, dartlang 0.4.6, SDK 1.12.1 running on macos.

Thanks for the report! This is a known issue in the analysis server - https://github.com/dart-lang/sdk/issues/24212. You should also be able to remove the spurious issues by running the atom command 'reanalyze sources'.

(will try and repo with the sample just to be sure)


Re-introduce the possibility the refer the scope in filters

Starting from 1.3.0-beta19, this does no longer refer to the corresponding scope in a filter implementation, see this commit and the CHANGELOG. The possibility to get a reference to the scope inside a filter should be re-introduced; whether implicitly or by other means is something that can be discussed (within this issue). I will explain this in the following.
I see the point in getting towards pure functional filters (though they aren't, as explained below); i.e., making all its input explicit rather than allowing for implicit input by the possibility to refer state such as the scope, which results in the possibility of side effects). Nevertheless, the possibility to (implicitly) refer the scope as the context of a filter, irrespective of whether this can be done by means of this, is a very valid use case. Consider a scope-specific filter. Such a filter depends on state in the scope (as an implicit input) that cannot be moved somewhere else; e.g., into a service, which is the only other possibility for filters to make them depending on something different than its input and optional arguments. As an example, consider private state that you want in the scope only. Also, consider the case where you need to access a scope-specific property; i.e. a property that might exist in many scope instances with different values. How would you externalize that property and make it identifiable without knowing the scope? The only possibility is to make it an argument of the filter (which may get a cumbersome lengthy list if a filter needs to access many properties of a scope).
In contrast to the commit comment, there is significant code that depends on this behavior. For instance, the translate filter of angular-translate. In fact, it already suffers from this change, see this recent issue. In addition, we have also developed a filter the needs to reference a scope-specific property, and we could no longer use it with 1.3.0.
Proposal (more of an idea than thoroughly thought through):
Introduce a new modifier such as ~ as an optional prefix for filter expressions. If present then this refers to the scope inside the filter. If not, then this is undefined.
Example:
{{ 'foo' | ~blah }} - this refers to the scope within blah filter.
{{ 'foo' | blah }} - this is undefined within blah filter.
Using a modifier (such as ~) makes the fact that we depend on scope explicit and it is also more concise than writing {{ 'foo' | blah:this }} all the time.
Btw, with the possibility to inject services into filters, they are not pure functional (because services can have state). Therefore, one could argue, why bother with the scope being implicitly available if they are not pure functional anyway? Just because of 10% performance gain measured in Chrome only?

This was a conscious modification. Even when this breaks filters that have a strong dependency on $scope, I doubt that this is going to be reverted


PyPI upload/release test

Tag a 4.1.0a1 release to test uploading new PyPI packages

Uploading PyPI releases work now (swig project name had been name squatted by a "security researcher"). Decided against automated release uploads to ensure the wheels are okay before they get uploaded.


Learn C#: Parallel programming

Issue tracking new topic at https://github.com/dotnet/core-docs/blob/master/docs/languages/csharp/parallel.md

This has to be done with communication around the Task Parallel Library, and async, and await.

New loc: https://github.com/dotnet/docs/blob/master/docs/csharp/parallel.md

not applicable anymore. remove the wrench topic: https://github.com/dotnet/docs/blob/master/docs/csharp/parallel.md


Danbooru Premium Account Bug

kurohai:
After entering my account info while setting up Danbooru, I am unable to search for more than two tags. The two tag search limit applies to anon and free accounts only. I have a Platinum account which allows six tags.
I haven't tried to log in with an account with any other site. Danbooru is the only site I have an account for.

Fixed in 64779a7ae1f6b524f6e16e7f43b85648a3eb57b0.


Mapperly won't include renamed collection property

Describe the bug
I have two classes, each containing a collection property. The name of the property changes so, according to the documentation, I need to specify a partial method and apply the MapPropertyAttribute to it. This causes the property mapping not to be included in the generated class mapping.
Declaration code
[Mapper]
public static partial class Mapper
{
    public static partial ChildDto MapToChildDto(Child child);

    public static partial ParentDto MapToParentDto(Parent parent);

    [MapProperty(nameof(Parent.Children), nameof(ParentDto.ChildrenList))]
    private static partial ICollection<ChildDto> MapToChildrenList(ICollection<Child> children);
}

Actual relevant generated code
public static partial class Mapper
{
    [global::System.CodeDom.Compiler.GeneratedCode("Riok.Mapperly", "<IP_ADDRESS>")]
    public static partial global::MapperlyWithCollectionWrapper.ChildDto MapToChildDto(global::MapperlyWithCollectionWrapper.Child child)
    {
        var target = new global::MapperlyWithCollectionWrapper.ChildDto();
        target.GivenName = child.GivenName;
        target.FamilyName = child.FamilyName;
        return target;
    }

    [global::System.CodeDom.Compiler.GeneratedCode("Riok.Mapperly", "<IP_ADDRESS>")]
    public static partial global::MapperlyWithCollectionWrapper.ParentDto MapToParentDto(global::MapperlyWithCollectionWrapper.Parent parent)
    {
        var target = new global::MapperlyWithCollectionWrapper.ParentDto();
        target.GivenName = parent.GivenName;
        target.FamilyName = parent.FamilyName;
        return target;
    }

    [global::System.CodeDom.Compiler.GeneratedCode("Riok.Mapperly", "<IP_ADDRESS>")]
    private static partial global::System.Collections.Generic.ICollection<global::MapperlyWithCollectionWrapper.ChildDto> MapToChildrenList(global::System.Collections.Generic.ICollection<global::MapperlyWithCollectionWrapper.Child> children)
    {
        var target = new global::System.Collections.Generic.List<global::MapperlyWithCollectionWrapper.ChildDto>(children.Count);
        foreach (var item in children)
        {
            target.Add(MapToChildDto(item));
        }
        return target;
    }
}

Expected relevant generated code
public static partial class Mapper
{
    [...]

    [global::System.CodeDom.Compiler.GeneratedCode("Riok.Mapperly", "<IP_ADDRESS>")]
    public static partial global::MapperlyWithCollectionWrapper.ParentDto MapToParentDto(global::MapperlyWithCollectionWrapper.Parent parent)
    {
        var target = new global::MapperlyWithCollectionWrapper.ParentDto();
        target.GivenName = parent.GivenName;
        target.FamilyName = parent.FamilyName;
        target.ChildrenList = MapToChildrenList(parent.Children); // <--- THIS IS MISSING
        return target;
    }

    [...]
}

Environment (please complete the following information):

Mapperly Version: 3.6.0
Nullable reference types: enabled
.NET Version: .NET 8.0.302
Target Framework: .net8.0
Compiler Version: 3.9.0-6.21124.20 (db94f4cc)
C# Language Version: 12
IDE: Rider 2024.1.4
OS: macOS 14.5 (23F79)

Additional context
I created an unit test here.

Your trying to map the Children property of the ICollection, this does not work as the ICollection does not have such a property. There should several diagnostics be reported during compilation, are there?
If you apply the MapProperty attribute to the MapToParentDto method it should work as expected.

Also no information? No RMG* diagnostics at all? 🤔
Is the mapping now generated as expected?

Also no information? No RMG* diagnostics at all? 🤔
In "additional request", I put the link of a GitHub repository I prepared with a unit test. When building no warnings pop up.

Is the mapping now generated as expected?
Yes, the mapping is generated correctly as per your explanation. Thank you.


Mark more texts for localization

This is about some localizations that were not included in #2440.

Localize time period strings in time.ts. The now should be localized, and depending on the language, the h, m, s, d, mo may also need to be localized.
Localize date strings in time.ts and number format strings in format.ts. It works, but I'm not sure if it's good as is.
Localize any other strings.

Thank you!

        <Trans>
          <Text type="sm" style={[{flexWrap: 'wrap'}, textStyles]}> and </Text>
        </Trans>

This one shows both whitespaces correctly,
        <Trans>
          <Text type="sm" style={[{flexWrap: 'wrap'}, textStyles]}>
            {' '}
            and{' '}
          </Text>
        </Trans>

But this removes the left whitespace.
Why is this and what can I do about it?


issue205 skipws propagation (regression tests)

Here, we add some tests to check and demonstrate the propagation of skipws/noswkips across rules.
We should consider to add this tests, once the Arpeggio fix for the Arpeggio issue 61 is done.

Code review checklist

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[n/a] Commit messages are meaningful (see this for details) - squash
[x] Tests have been included and/or updated
[n/a] Docstrings have been included and/or updated, as appropriate
[x] Standalone docs have been updated accordingly
[n/a] Changelog(s) has/have been updated, as needed (see CHANGELOG.md).

@goto40 Arpeggio bugfix release 1.9.1 is out with a regression test for the issue.
I don't think that we also need regression tests in textX as well as it was solely an Arpeggio issue but I suggest, since you have spent time to investigate skipws behavior, to move some of these tests (I would prefer a non-redundant subset) into standard battery of tests to cover skipws better.

OK. Great. I will rework the test cases.

I reworked the tests and the documentation alongside the test cases.
I added some explanation to the docs (grammar.md)

Feel free to comment...
However: I installed the previous version of Arpeggio (git co aff1bb8) and all tests pass. I cannot (re)produce an error showing the new Arpeggio version has an effect... Any idea, what to do to demonstrate the old Arpeggio version was a problem for textx?

@goto40 Created a regression test that fails with the unfixed version of Arpeggio and pass with fixed version. I think that this single test is all what we need. I've reused your docstrings for explanation in comment. If you agree with the change I would merge/cherry pick just this single commit and your addition to the docs.

Very good! Good work.
OK for me, go ahead as you described.


cpr ssl broken for new cloudflare certs(discord etc)

Describe the bug
Cloudflare now issue new kind of ssl certs with no tls pin support.
While doing https request to any cloudflared website with new certs, you will get internal error 4
More details in cpr issue: https://github.com/libcpr/cpr/issues/1069
This issue only appears on vcpkg version (confirmed by me as well using this example: https://github.com/libcpr/example-cmake-fetch-content, NOTE: upgrade it to latest commit)
The issue author tested it for discord.gg, I have the same issue for discord and many more cloudflare websites.
Environment

OS: Windows 11
Compiler: MSVC 22

To Reproduce
const cpr::Url url = cpr::Url{ "https://discord.com" };
const auto response = cpr::Get( url);

Expected behavior
successful ssl verify and complete the request
Failure logs
Connected to ******************************** port 443
schannel: disabled automatic use of client certificate
schannel: connection hostname () validated against certificate name ()
Subject: CN=********************************
Issuer: C=US, O=Let's Encrypt, CN=E6
Version: 2
Serial Number: 03:cf:f5:40:86:7e:2e:f9:b6:d2:e6:24:17:ff:eb:b9:58:c9:
Closing connection
schannel: shutting down SSL/TLS connection with t******************************** port 443
Additional context

Which vcpkg port did you use during this process that caused this issue?

cpr:x64-windows-static                            1.10.5#2

https://github.com/curl/curl/issues/13845
maybe it is related with this issue
cpr uses curl-8_7_1 while vcpkg bugged v8.8.0 as I understand?

I call this example: https://github.com/libcpr/example-cmake-fetch-content, and then call the following usage in cmakelists.txt, and the result is displayed normally.
find_package(cpr CONFIG REQUIRED)
 target_link_libraries(main PRIVATE cpr::cpr)

I call this example: https://github.com/libcpr/example-cmake-fetch-content, and then call the following usage in cmakelists.txt, and the result is displayed normally.
find_package(cpr CONFIG REQUIRED)
 target_link_libraries(main PRIVATE cpr::cpr)

I've pulled and upgraded vcpkg rn and testing for discord.gg and github as from example.
Vcpkg cpr::x64-windows-static does not work for me as you see in the screenshots above.
while it works from the cpr repository(clang-llvm cmake)
Not working:
vcpkg: latest(pulled today),
cpr:x64-windows-static                            1.10.5#2
cpr[ssl]:x64-windows-static
curl:x64-windows-static                           8.8.0#3
OS: Windows 10  (tested my binaries on 10+ machines in 5 different regions RU\EU\USA)
compiler: Visual Studio 2022(17.10.4), c++ 20, build tools v143
example of code not working: https://github.com/Chrys4lisfag/vcpkg-cpr-issue

Probably related
https://github.com/libcpr/cpr/issues/1068

My workaround is to install both curl and cpr as dependencies and fixing curl version to 8.6.0.
(btw, I am on macOS, so not sure this will help if you are on Windows or Linux)

{
  "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg-tool/main/docs/vcpkg.schema.json",
  "name": "app",
  "version-string": "1.0.0",
  "dependencies": [
    "cpr",
    "curl"
  ],
  "overrides": [
    {
      "name": "curl",
      "version": "8.6.0"
    }
  ]
}

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.

I'm having the same issue on windows, Anyone find a solution?


score_func in path.lua is sometimes nil

My testing minimal init.vim
Default configuration, making sure the path source is configured to be used.
How to reproduce
Try typing in a context that should trigger the path source. Note that you get errors complaining that in path.lua line 39, there's an attempt to call score_func, which is nil.
Expected behavior
The path source returns without errors.

Should be fix now, please update and check:)

Seems to be fixed, thanks!


Handle stack pending_operations

[ ] Test pending_operations scenario
[ ] Export stack state in the event a pending_operations scenario occurs
[ ] Create any supporting documentation needed to debug this scenario.
[ ] Log potentially leaked resources in the update
[ ] Reflect the log of potentially leaked resources in CR spec.status field

Related to (or possibly dependent on) https://github.com/pulumi/pulumi/issues/4265.

Note the follow-up: https://github.com/pulumi/pulumi/pull/10394


ci: switch to hosted agents

GitHub Issue: #

Proposed Changes

[ ] Bug fix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes, no api changes)
[x] Build or CI related changes
[ ] Documentation content changes
[ ] Other, please describe:

What is the current behavior?

What is the new behavior?

ci: switch to hosted agents
Impact on version

[ ] Major (Public API was modified.)

Public constructs (class, struct, delegate, enum, etc.) were removed or renamed.
Public members were removed or renamed.
Public method signatures were changed or renamed.

[ ] Minor (Public API was extended.)

Public constructs, members, or overloads were added.

[ ] Patch (Public API was unchanged.)

A bug in behavior was fixed.
Documentation was changed.

[x] None (The library is unchanged.)

Only code under the build folder was changed.
Only code under the .github folder was changed.

Checklist
Please check that your PR fulfills the following requirements:

[ ] Documentation has been added/updated.
[ ] Automated Unit / Integration tests for the changes have been added/updated.
[ ] Updated BREAKING_CHANGES.md (if you introduced a breaking change).
[x] Your conventional commits are aligned with the Impact on version section.

Other information

@carlh98 I changed the impact on version from Patch to None in the PR description.


Add domain constructor for given p4est

This PR adds a new domain constructor that constructs a domain for a given p4est. This function will be required for the ForestClaw restart functionality because we must construct a domain given a p4est that was read from a file.

Do we require any conditions on the data_size and the user_pointer of the p4est that we pass to the new function?
Shall we document the ownership more explicitly?

Do we require any conditions on the data_size and the user_pointer of the p4est that we pass to the new function?

The user_pointer of the p4est must be NULL. I added this to the documentation and to be more explicit I also added an assertion to the function fclaw2d_domain_new_p4est although p4est_wrap_new_p4est already asserts on p4est->user_pointer == NULL. The data_size of the p4est is not touched by p4est_wrap_new_p4est and therefore there should be no conditions on the data size.

Shall we document the ownership more explicitly?

After the latest push we document  the user_pointer condition explicitly and on the ownership in general we still have the line:
This function takes ownership of the passed pest and its connectivity.

Do you think that all in all this is clear enough?

Do we require any conditions on the data_size and the user_pointer of the p4est that we pass to the new function?

The user_pointer of the p4est must be NULL. I added this to the documentation and to be more explicit I also added an assertion to the function fclaw2d_domain_new_p4est although p4est_wrap_new_p4est already asserts on p4est->user_pointer == NULL. The data_size of the p4est is not touched by p4est_wrap_new_p4est and therefore there should be no conditions on the data size.

Cool, this is helpful I think.  Would it make sense to call p4est_reset_data
with size 0 on the passed forest?  It seems that in wrap we only do this in
p4est_wrap_new_copy, while also with p4est_wrap_new_p4est we access
p.user_int etc, which will not play nicely with nonzero data_size.  There
are no assertions on this currently.  This may be an oversight in the design
of the wrap, or at least warrants a study of that code.

Shall we document the ownership more explicitly?

After the latest push we document  the user_pointer condition explicitly and on the ownership in general we still have the line:
This function takes ownership of the passed pest and its connectivity.

Do you think that all in all this is clear enough?

Ah, it's right there!  Depending on what we do with the p4est passed, we may
be a bit more wordy on the p4est parameter description itself.

Do we require any conditions on the data_size and the user_pointer of the p4est that we pass to the new function? The user_pointer of the p4est must be NULL. I added this to the documentation and to be more explicit I also added an assertion to the function fclaw2d_domain_new_p4est although p4est_wrap_new_p4est already asserts on p4est->user_pointer == NULL. The data_size of the p4est is not touched by p4est_wrap_new_p4est and therefore there should be no conditions on the data size.

Cool, this is helpful I think. Would it make sense to call p4est_reset_data with size 0 on the passed forest? It seems that in wrap we only do this in p4est_wrap_new_copy, while also with p4est_wrap_new_p4est we access p.user_int etc, which will not play nicely with nonzero data_size. There are no assertions on this currently. This may be an oversight in the design of the wrap, or at least warrants a study of that code.

In p4est_wrap_new_copy we create a deep copy of the given p4est and I think that this may be an additional justification for resetting the quadrant data since we may want to prevent having two p4est both pointing to the same quadrant data. However, in p4est_wrap_new_p4est we do not copy the given p4est but take its ownership. In that sense I think it may be sensible to not reset the quadrant data of the p4est because otherwise the quadrant data will be lost.
I agree that an access of p.user_int in p4est_wrap_new_p4est would cause problems with non-zero quadrant data but I could not find the access of p.user_int or in general of the union p in the function p4est_wrap_new_p4est.

Shall we document the ownership more explicitly? After the latest push we document the user_pointer condition explicitly and on the ownership in general we still have the line: This function takes ownership of the passed pest and its connectivity. Do you think that all in all this is clear enough?

Ah, it's right there! Depending on what we do with the p4est passed, we may be a bit more wordy on the p4est parameter description itself.

We could elaborate to what the user_pointer of the p4est is set, i.e. the created p4est_wrap.

In p4est_wrap_new_copy we create a deep copy of the given p4est and I think that this may be an additional justification for resetting the quadrant data since we may want to prevent having two p4est both pointing to the same quadrant data. However, in p4est_wrap_new_p4est we do not copy the given p4est but take its ownership. In that sense I think it may be sensible to not reset the quadrant data of the p4est because otherwise the quadrant data will be lost.
I agree that an access of p.user_int in p4est_wrap_new_p4est would cause problems with non-zero quadrant data but I could not find the access of p.user_int or in general of the union p in the function p4est_wrap_new_p4est.

I'd be suspecting that subsequent modiications of the wrap by refine and
coarsen assign to p, which loses any pointer to the user data stored
previously.  It gets deallocated ok eventually by the mempool, but it may
stop being accessible during the lifetime of the wrap already.

In p4est_wrap_new_copy we create a deep copy of the given p4est and I think that this may be an additional justification for resetting the quadrant data since we may want to prevent having two p4est both pointing to the same quadrant data. However, in p4est_wrap_new_p4est we do not copy the given p4est but take its ownership. In that sense I think it may be sensible to not reset the quadrant data of the p4est because otherwise the quadrant data will be lost. I agree that an access of p.user_int in p4est_wrap_new_p4est would cause problems with non-zero quadrant data but I could not find the access of p.user_int or in general of the union p in the function p4est_wrap_new_p4est.
I'd be suspecting that subsequent modiications of the wrap by refine and coarsen assign to p, which loses any pointer to the user data stored previously. It gets deallocated ok eventually by the mempool, but it may stop being accessible during the lifetime of the wrap already.

Ah, I see. The callback functions refine_callback, replace_on_refine, coarsen_callback, replace_on_coarsen and replace_on_balance use p.user_int. Therefore, I would propose to reset the data size to 0 in the function p4est_wrap_new_p4est and assert in the function fclaw2d_domain_new_p4est p4est->data_size == 0 since this seems to be more consistent to the handling of the data size in p4est_wrap_new_copy.

In p4est_wrap_new_copy we create a deep copy of the given p4est and I think that this may be an additional justification for resetting the quadrant data since we may want to prevent having two p4est both pointing to the same quadrant data. However, in p4est_wrap_new_p4est we do not copy the given p4est but take its ownership. In that sense I think it may be sensible to not reset the quadrant data of the p4est because otherwise the quadrant data will be lost. I agree that an access of p.user_int in p4est_wrap_new_p4est would cause problems with non-zero quadrant data but I could not find the access of p.user_int or in general of the union p in the function p4est_wrap_new_p4est.
I'd be suspecting that subsequent modiications of the wrap by refine and coarsen assign to p, which loses any pointer to the user data stored previously. It gets deallocated ok eventually by the mempool, but it may stop being accessible during the lifetime of the wrap already.

Ah, I see. The callback functions refine_callback, replace_on_refine, coarsen_callback, replace_on_coarsen and replace_on_balance use p.user_int. Therefore, I would propose to reset the data size to 0 in the function p4est_wrap_new_p4est and assert in the function fclaw2d_domain_new_p4est p4est->data_size == 0 since this seems to be more consistent to the handling of the data size in p4est_wrap_new_copy.

Ok! sounds good, and re-checking the doxygen content would be splendid.

Ok! sounds good, and re-checking the doxygen content would be splendid.

Alright, I created the p4est PR https://github.com/cburstedde/p4est/pull/252 to adjust the handing of the data size of the given p4est. I also updated the doxygen documentation in both p4est and ForestClaw. Moreover, I added an assertion for the data size in ForestClaw.
This ForestClaw PR should be not merged until the p4est PR https://github.com/cburstedde/p4est/pull/252 is merged and the corresponding p4est submodule update is committed in ForestClaw since otherwise the function fclaw2d_domain_new_p4est will not behave as documented.

This ForestClaw PR should be not merged until the p4est PR cburstedde/p4est#252 is merged and the corresponding p4est submodule update is committed in ForestClaw since otherwise the function fclaw2d_domain_new_p4est will not behave as documented.

Since the PR https://github.com/cburstedde/p4est/pull/252 is now merged into p4est develop and I updated the p4est submodule of ForestClaw, from my side this PR is now ready to be merged.

This ForestClaw PR should be not merged until the p4est PR cburstedde/p4est#252 is merged and the corresponding p4est submodule update is committed in ForestClaw since otherwise the function fclaw2d_domain_new_p4est will not behave as documented.

Since the PR cburstedde/p4est#252 is now merged into p4est develop and I updated the p4est submodule of ForestClaw, from my side this PR is now ready to be merged.

This PR is not yet ready to merge since first I want to recheck if there are still some submodule related problems with linking jansson that are unrelated to this PR but that we expericenced before while using the current p4est develop branch in ForestClaw.

This ForestClaw PR should be not merged until the p4est PR cburstedde/p4est#252 is merged and the corresponding p4est submodule update is committed in ForestClaw since otherwise the function fclaw2d_domain_new_p4est will not behave as documented.

Since the PR cburstedde/p4est#252 is now merged into p4est develop and I updated the p4est submodule of ForestClaw, from my side this PR is now ready to be merged.

This PR is not yet ready to merge since first I want to recheck if there are still some submodule related problems with linking jansson that are unrelated to this PR but that we expericenced before while using the current p4est develop branch in ForestClaw.

This PR can not be merged until the jansson related linking problems on some systems (cf. in particular https://github.com/cburstedde/libsc/pull/140 and https://github.com/cburstedde/libsc/pull/141) are resolved.

We have updated the p4est and libsc develop branches. These have all the latest CMake code in them. @scottaiton please let us know if anything is awry with these versions. The autoconf build should be working fine as usual.

The CMake builds work on my systems. I'll let you know if anyone runs into issues.

The CMake builds work on my systems. I'll let you know if anyone runs into issues.

I think the CMake build problems were architecture dependent and it may be required to update p4est and libsc to current develop since this PR only contained an update of the p4est and libsc to an older develop version.


Publish One Device Operator in Operator Hub

Publish One Device Operator in Operator Hub

I have to pause this task due to https://github.com/operator-framework/operator-lifecycle-manager/issues/1315#issuecomment-692597350

The progress can be tracked at https://github.com/operator-framework/community-operators/pull/2750 now.

@rojkov - could we ping operator-framework people on this to get some traction?

I've managed to find to this instruction on how to reproduce the CI tests locally. But I'm busy with a QAT engine issue ATM. Will try it later.

@avalluri has added CI testing of the PMEM-CSI operator. He can provide further pointers and help in case that you need it.

Closing as https://github.com/operator-framework/community-operators/pull/2750 got merged.

Closing as https://github.com/operator-framework/community-operators/pull/2750 got merged.


Option to prevent linking of last crumb to current url

Enhanced contribution instructions

Is there any reason why this PR was not merged?

THIS ! should be merged !

Who knows?! :) gretel is still great though!

Sorry guys. Haven't been so active Rails-wise lately. Not forgotten though. I will merge when I get the time.

Please, merge this PR. Needed!

Moving work on this feature to feature/last-crumb-link.

Thanks @WilHall, good to see some progress on it :)


Postgres Export Error: type modifier is not allowed for type "text"

When exporting from Postgres to Postgres, the text datatype is given a modifier (interpreted from the unknownLength driver setting). This causes an error in Postgres.

Can't create target table:
CREATE TABLE public.table_name (
row_id serial NOT NULL,
varchar_value varchar(256),
comments text(2147483647),
PRIMARY KEY (row_id)
)
SQL Error [42601]: ERROR: type modifier is not allowed for type "text"
Position: XXX
org.postgresql.util.PSQLException: ERROR: type modifier is not allowed for type "text"
Position: XXX

This is fixed by editing the column mappings, which is how I've been getting around this for now. Additionally, setting unknownLength = 0 in the driver settings for the source table seems to alleviate this issue, too, but I'm not sure what other impact that might have in DBeaver.

Yep, cross-database type mapping has to be  improved.

cannot reproduce on 6.0.2

Reproduced on 6.2.2 (MySQL remote server exported to PostgreSQL running on same remote server) @uslss @serge-rider :
Error starting data transfer
  Can't create target table:
CREATE TABLE public."2grams"(
	id int NOT NULL,
	frequency int NOT NULL,
	wordone varchar(255) NOT NULL,
	wordtwo varchar(255) NOT NULL,
	posone varchar(10) NOT NULL,
	postwo varchar(10) NOT NULL,
	lastupdated timestamp NOT NULL,
	lastviewed timestamp NOT NULL,
	description text(65535),
	wikilink varchar(2083),
	PRIMARY KEY (id)
)
  Can't create target table:
CREATE TABLE public."2grams"(
	id int NOT NULL,
	frequency int NOT NULL,
	wordone varchar(255) NOT NULL,
	wordtwo varchar(255) NOT NULL,
	posone varchar(10) NOT NULL,
	postwo varchar(10) NOT NULL,
	lastupdated timestamp NOT NULL,
	lastviewed timestamp NOT NULL,
	description text(65535),
	wikilink varchar(2083),
	PRIMARY KEY (id)
)
    SQL Error [42601]: ERROR: type modifier is not allowed for type "text"
  Position: 276
    SQL Error [42601]: ERROR: type modifier is not allowed for type "text"
  Position: 276
      ERROR: type modifier is not allowed for type "text"
  Position: 276
      ERROR: type modifier is not allowed for type "text"
  Position: 276


Component class names should be namespaced

Consider juiced-alert instead of alert, since the latter could easily be re-defined elsewhere in larger applications.

Keeping this open.  I like the idea and may move towards namespaced class names for components.

+1


Custom Legend for Bar Charts

Is it possible to draw a custom Legend for a bar chart?
I have many bars but only two conditions and want to place only those two conditions in a legend below the chart.
What I get is a Legend with one Item for each bar.
In the Docs it says that legends is part of BarChartData but I can't add id there.

Custom legends are not possible. They get generated based on the data provided.
You could look at Grouped Bar Charts? https://github.com/willdale/SwiftUICharts#grouped-bar-chart

Setting up a Grouped Bar Chart with only one group did the trick :)
Thanks!


Add dontRequireTtyForSudo option to locations

When using the dontCreateUser option on locations jclouds initialization won't run which means it won't overwrite /etc/sudoers leaving in the defaults. For CentOS that means having requiretty enabled. This will lead to all sudo calls from Brooklyn failing because we are running without a session.
It becomes even bigger problem because the installDevUrandom will fail (silently) and the machine will be unresponsive for no apparent reason.
The changes

run the /etc/sudoers fix if explicitly requested or if dontCreateUser is set.
convert commands executed during location obtain to tasks so they are visible in the acitivty tab
fails location obtaining on a non-0 exit code

Jenkins failure is unrelated. We should mark that test as @Broken:
2016-05-31 12:32:22,255 INFO  TESTNG INVOKING CONFIGURATION: "Surefire test" - @BeforeMethod org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicyConcurrencyTest.before()
2016-05-31 12:32:22,413 INFO  No Camp-YAML parser registered for parsing catalog item DSL; skipping DSL-parsing
2016-05-31 12:32:22,414 WARN  Deprecated use of unmanaged location (SimulatedLocation{id=xp55pmac6m, name=loc}); will be managed automatically now but not supported in future versions
2016-05-31 12:32:22,428 INFO  TESTNG PASSED CONFIGURATION: "Surefire test" - @BeforeMethod org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicyConcurrencyTest.before() finished in 173 ms
2016-05-31 12:32:22,429 INFO  TESTNG INVOKING: "Surefire test" - org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicyConcurrencyTest.testSimplePeriodicWorkrateUpdates()
2016-05-31 12:32:22,465 INFO  org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicy$2@3978b46a emitted COLD (suggesting 0): {pool.current.size=1, pool.current.workrate=0.0, pool.low.threshold=10.0, pool.high.threshold=30.0}
2016-05-31 12:32:23,090 INFO  org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicy$2@b5a4d43 emitted COLD (suggesting 3): {pool.current.size=19, pool.current.workrate=21.0, pool.low.threshold=190.0, pool.high.threshold=570.0}
2016-05-31 12:32:23,129 INFO  org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicy$2@2e8f0466 emitted COLD (suggesting 5): {pool.current.size=19, pool.current.workrate=41.0, pool.low.threshold=190.0, pool.high.threshold=570.0}
2016-05-31 12:32:23,168 INFO  org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicy$2@2c958fbf emitted COLD (suggesting 9): {pool.current.size=19, pool.current.workrate=81.0, pool.low.threshold=190.0, pool.high.threshold=570.0}
2016-05-31 12:32:23,190 INFO  org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicy$2@5105e270 emitted COLD (suggesting 11): {pool.current.size=19, pool.current.workrate=101.0, pool.low.threshold=190.0, pool.high.threshold=570.0}
2016-05-31 12:32:23,215 INFO  org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicy$2@59f4ce19 emitted COLD (suggesting 12): {pool.current.size=19, pool.current.workrate=120.0, pool.low.threshold=190.0, pool.high.threshold=570.0}
2016-05-31 12:32:23,227 INFO  org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicy$2@519824b4 emitted COLD (suggesting 14): {pool.current.size=19, pool.current.workrate=139.0, pool.low.threshold=190.0, pool.high.threshold=570.0}
2016-05-31 12:32:23,255 INFO  org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicy$2@4fd7e4bb emitted COLD (suggesting 16): {pool.current.size=19, pool.current.workrate=158.0, pool.low.threshold=190.0, pool.high.threshold=570.0}
2016-05-31 12:32:23,287 INFO  org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicy$2@1aea8c8d emitted COLD (suggesting 18): {pool.current.size=19, pool.current.workrate=178.0, pool.low.threshold=190.0, pool.high.threshold=570.0}
2016-05-31 12:32:33,566 INFO  succeedsEventually exceeded max attempts or timeout - 29 attempts lasting 10002 ms, for RunnableAdapter(org.apache.brooklyn.policy.loadbalancing.AbstractLoadBalancingPolicyTest$2@d2cd563)
2016-05-31 12:32:33,566 INFO  failed succeeds-eventually, 29 attempts, 10002ms elapsed (rethrowing): java.lang.AssertionError: actual=[18.0, 18.0, 18.0, 21.0, 18.0, 21.0, 0.0, 21.0, 36.0, 21.0, 20.0, 21.0, 21.0, 21.0, 21.0, 19.0, 18.0, 19.0, 20.0, 20.0]; expected=[20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0] expected [20.0] but found [0.0]
2016-05-31 12:32:33,570 INFO  TESTNG FAILED: "Surefire test" - org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicyConcurrencyTest.testSimplePeriodicWorkrateUpdates() finished in 11139 ms
org.apache.brooklyn.util.exceptions.PropagatedRuntimeException: failed succeeds-eventually, 29 attempts, 10002ms elapsed: AssertionError: actual=[18.0, 18.0, 18.0, 21.0, 18.0, 21.0, 0.0, 21.0, 36.0, 21.0, 20.0, 21.0, 21.0, 21.0, 21.0, 19.0, 18.0, 19.0, 20.0, 20.0]; expected=[20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0] expected [20.0] but found [0.0]
	at org.apache.brooklyn.util.exceptions.Exceptions.propagate(Exceptions.java:164)
	at org.apache.brooklyn.util.exceptions.Exceptions.propagateAnnotated(Exceptions.java:144)
	at org.apache.brooklyn.test.Asserts.succeedsEventually(Asserts.java:963)
	at org.apache.brooklyn.test.Asserts.succeedsEventually(Asserts.java:854)
	at org.apache.brooklyn.policy.loadbalancing.AbstractLoadBalancingPolicyTest.assertWorkratesEventually(AbstractLoadBalancingPolicyTest.java:136)
	at org.apache.brooklyn.policy.loadbalancing.LoadBalancingPolicyConcurrencyTest.testSimplePeriodicWorkrateUpdates(LoadBalancingPolicyConcurrencyTest.java:79)
Caused by: java.lang.AssertionError: actual=[18.0, 18.0, 18.0, 21.0, 18.0, 21.0, 0.0, 21.0, 36.0, 21.0, 20.0, 21.0, 21.0, 21.0, 21.0, 19.0, 18.0, 19.0, 20.0, 20.0]; expected=[20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0, 20.0] expected [20.0] but found [0.0]
	at org.apache.brooklyn.policy.loadbalancing.AbstractLoadBalancingPolicyTest.assertWorkrates(AbstractLoadBalancingPolicyTest.java:122)
	at org.apache.brooklyn.policy.loadbalancing.AbstractLoadBalancingPolicyTest$2.run(AbstractLoadBalancingPolicyTest.java:138)
	at org.apache.brooklyn.test.Asserts$RunnableAdapter.call(Asserts.java:1277)
	at org.apache.brooklyn.test.Asserts.succeedsEventually(Asserts.java:930)
	... 31 more

@neykov LGTM. Is there an appropriate test we should add for this? Happy for you to merge, depending on what you think about the ease of including a test.

Addressed comments. I could've created a recording location for testing, but that wouldn't be very useful I think.


unify LRU cache limits, make flow verification cacheable.

Codecov Report

Merging #23 (722d833) into master (08d2697) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #23   +/-   ##
=======================================
  Coverage   92.30%   92.30%
=======================================
  Files          47       47
  Lines        8896     8899    +3
=======================================
+ Hits         8211     8214    +3
  Misses        685      685

Impacted Files
Coverage Δ

faucet/valve_of.py
93.80% <100.00%> (-0.05%)
:arrow_down:

faucet/valve_packet.py
95.54% <100.00%> (+0.01%)
:arrow_up:

faucet/valve_pipeline.py
95.51% <100.00%> (+0.05%)
:arrow_up:

faucet/valve_switch_standalone.py
93.85% <100.00%> (+0.01%)
:arrow_up:

faucet/valve_table.py
96.13% <100.00%> (+0.05%)
:arrow_up:

faucet/valve_util.py
83.52% <100.00%> (+0.18%)
:arrow_up:

Continue to review full report at Codecov.

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


Will you write some blogs for high level design？

like the difference between polardb and pg....

We are working on it. You can reference https://github.com/alibaba/PolarDB-for-PostgreSQL#architecture--roadmap for more information.


reorganise lesson

full-cycle collaborative workflow is now an optional variant of a standard solo type-along exercise. Motivation is that the focus should be on testing and not pull requests, but the lesson should still show how CI works.
Also fixing a few smaller issues along the way

this new CI episode still uses PRs in a single-person type-along session, because that's good practice and learners can then more easily translate the workflow to own collaborations. Drawback is that it's somewhat artificial to raise PR's and merge them by oneself, but my feeling is that it's worth it. But would be interesting to hear other opinions


Percent sign in path causes error

When you have a path with a % in it, it causes error: failed to build archive: No such file or directory.

Thanks for the report! Could you describe the steps needed to reproduce this? I was unable to reproduce this unfortunately

Closing due to inactivity and inability to reproduce. If this is still a problem, please reopen with more details! Thanks!!

For future readers like me: this is not a bug in cargo, it's a bug in LLVM, see:

https://github.com/rust-lang/rust/issues/79567
https://github.com/llvm/llvm-project/issues/49094


Fix multiwallet transaction notifications

Currently vQueueNotifications holds transactions of any wallet, but the queue is dispatched on a given wallet and it assumes notifications are of that wallet.
This means that some transactions can be missed if multiple wallets are loaded.
Fix this by having a queue for each wallet.

@jonasschnelli @fanquake this is a bugfix.

utACK 241434200ec2067673d8522fee4f1228abfd8247


Lower text is not visible in "Configurator" Page

The page :

The portion of text which is not visible there:

Please assign me to this issue.

Go for it @imdebamrita

I want to work on this issue under gssoc'23

This Issue has been resolved @imdebamrita , thank you raising the issue


Ministry User Testing Sessions

Description
Conduct research sessions with Ministry participants. Each session will have a note taker.
Synthesize results is a separate ticket.
Dependencies
Availability of Ministry clients, we have asked Ministries to select a time that works best for them from Sept 20-24th
DOD (list sessions as they get booked)

[x] Session 1 - Alexandra Kavadas - ENV - Tues. Sept 14, 8 AM PST
[ ] Session 2 - Carrie Preziuso - EDUC - Tues. Sept 14, 11 AM PST
[ ] Session 3 - Darlene Grieve - HTH - Tues. Sept 14, 1 PM PST
[ ] Session 4 - Tara MacNair - MUNI - Wed. Sept 15, 11 AM PST
[ ] Session 5 - Danny Courcy & Luke Rioux - MUNI - Wed. Sept 15, 1 PM PST
[ ] Session 6 - Nicole Forward - MUNI - Thurs. Sept 16, 10 AM PST
[ ] Session 7 - Vicki Hudson - MUNI - Thurs. Sept 16, 1 PM PST
[ ] Session 8 - Anne Sum - FIN - Mon. Sept 20, 11 AM PST

Session 1 Complete - Recording Below
https://bcgov.sharepoint.com/teams/03521/Shared Documents/General/User Research/Sprint 9 & 10 - Recording/Session 1 - Alexandra Kavadas - ENV - Tues. Sept 14.mp4

@m-prodan @arielleandrews
Session 2 - Carrie Preziuso - EDUC needs to be rescheduled as she is away. Just wanted to track in a comment here so we have a record. Please comment when we have a rescheduled time

Session 2 - Complete - Recording Below
https://bcgov.sharepoint.com/teams/03521/Shared Documents/General/User Research/Sprint 9 & 10 - Recording/Session 2 - Darlene Grieve - HTH - Tues Sept 21.mp4

Session 2 - Complete - Recording Below
https://bcgov.sharepoint.com/teams/03521/Shared Documents/General/User Research/Sprint 9 & 10 - Recording/Session 3 - Tara MacNair - MUNI - Wed Sept 22.mp4

Session 4 - Complete - Recording Below
https://bcgov.sharepoint.com/teams/03521/Shared Documents/General/User Research/Sprint 9 & 10 - Recording/Session 4 - Danny Courcy, Luke Rioux - MUNI - Wed Sept 22.mp4

Session 5 - Complete - Recording Below
https://bcgov.sharepoint.com/teams/03521/Shared Documents/General/User Research/Sprint 9 & 10 - Recording/Session 5 - Nicole Forward - MUNI - Thurs Sept 23.mp4

Session 6 - Complete - Recording Below
https://bcgov.sharepoint.com/teams/03521/Shared Documents/General/User Research/Sprint 9 & 10 - Recording/Session 6 - Vicki Hudson - MUNI - Thurs Sept 23.mp4

Session 8 - Complete - Recording Below
https://bcgov.sharepoint.com/teams/03521/Shared Documents/General/User Research/Sprint 9 & 10 - Recording/Session 7 - Carrie Preziuso - EDUC - Fri Sept 24.mp4


Improved productivity through standardized configurations and testing of Trilinos on advanced platforms

Ready to merge

@maherou, should be ready to merge.


Write out "total" instead of abbreviating it to "tot"

There is plenty of space in the character sheet there:

In my native language German, "tot" means "dead", which is always weird to see as the abbreviation there 😅

Modified that via Weblate, will be fixed in next release


Add simple react front-end using create-react-app

This is just a simple implementation of this chat using React.
Features:

Create and join channels
Choose username (by channel)
Timestamped messages
Up arrow key to retrieve last message

For now this is completely silo'ed in a client folder.
It would be nice to have a single script to run both server and client.


If a prisma migrate dev fails, prisma suggests for you to reset your database

Bug description
First run prisma migrate dev command
npx prisma migrate dev
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database "development", schema "public" at "xxx.xxx.xxx"

Error: P2002

Unique constraint failed on the fields: (`attribute`)

Then run it again:
npx prisma migrate dev
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database "development", schema "public" at "xxx.xxx.xxx"

✔ The migration `20210905678234_make_attribute_unique` failed.

We need to reset the PostgreSQL database "development" at "xxx.xxx.xxx".

Do you want to continue? All data will be lost. y / no

Why would someone want to delete their entire database because a migration failed? Couldn't you show the same message that's shown with prisma migrate deploy or prisma migrate status when this happens?

@pantharshit00 Is there any progress on this?
It seems like a pretty dangerous issue that could lead someone to accidentally delete their database.
It's also a very quick easy fix.

@jeremygottfried Hey, what are you doing in your migration there? Is this a generated one from Prisma or did you modify it? Otherwise we can not explain why you get Unique constraint failed on the fields: (attribute) for a normal migration.

Internal note for reproduction:

migrate dev --create-only
Modify generated SQL to trigger a unique constraint error
Run migrate dev and observe failure
Run migrate dev again
Does it look like the issue reported above?

I think for me it was this, but your steps probably do the same thing:

add @unique to an attribute that has duplicate values so it fails.
run migrate dev and observe failure
run migrate dev again


Support mock avatars

Allow FE to pass a mock avatar into Users.
To do this, we need to also support explicit avatar url on LazyStaticImage construction


No ability to upload Model with multipart File

Hi
I'm using the last (and only)
 <dependency>
            <groupId>io.springfox</groupId>
            <artifactId>springfox-boot-starter</artifactId>
            <version>3.0.0</version>
 </dependency>

with spring boot
<parent>
    <groupId>org.springframework.boot</groupId>
    <artifactId>spring-boot-starter-parent</artifactId>
    <version>2.4.1</version>
    <relativePath/> <!-- lookup parent from repository -->
 </parent>

I would like to have ability to upload file and model in one controller, so I created one:

in this case on swagger-ui I see only file uploading without model:

I tried this case in postman and it works fine

Is it a bug? Or should I add some other annotations to make it work?

After exploring I found that defining a Docket Bean is providing this bug

After exploring I found that defining a Docket Bean is providing this bug


Every time a new window or tab column is opened,plugin's  function falis

Hello,I found a problem.When I open a new window or a new tab column,my zsh has not suggestions on shell.Unless I  executed an order: source ~/.zshrc
Why?How can I resolve this problem.I can't find the answer from google and baidu

Hi, sorry for the late answer.  Could you please show the output of typeset -p ZSH_VERSION ZSH_PATCHLEVEL ZSH_HIGHLIGHT_REVISION ZSH_HIGHLIGHT_VERSION?  I'd like to see whether you use master.  You probably do use master, but if you use the feature/redrawhook branch, then you might be running into #579.

Ping?

Feedback timeout.


Impossible to deserialize into DeserializeOwned

With update from serde 0.9 to serde 1 there were added trait lifetimes for Deserialize trait and another trait called DeserializeOwned. The library supports only the first trait but not the second one, so it is impossible to write such code:
fn load_config<T: std::fmt::Debug + serde::de::DeserializeOwned>(file_name: &str) -> T {
    let config_toml = match fs::read_to_string(file_name) {
        Ok(s) => s,
        Err(e) => panic!("Unable to read {}: {}", file_name, e),
    };
    let parsed = toml::de::from_reader::<T>(&config_toml);
    if let Ok(c) = parsed {
        return c
    } else {
        panic!("Unable to parse {}: {}", file_name, parsed.unwrap_err());
    }
}

This will never compile because there are only two functions which take string slice with explicit lifetime and so we will never be able to return an owned type.

There isn't a toml::de::from_reader, but your code with toml::from_str seems to work:
extern crate serde;
extern crate toml;

use std::fs;

fn load_config<T: std::fmt::Debug + serde::de::DeserializeOwned>(file_name: &str) -> T {
    let config_toml = match fs::read_to_string(file_name) {
        Ok(s) => s,
        Err(e) => panic!("Unable to read {}: {}", file_name, e),
    };
    let parsed = toml::from_str::<T>(&config_toml);
    if let Ok(c) = parsed {
        return c
    } else {
        panic!("Unable to parse {}: {}", file_name, parsed.unwrap_err());
    }
}

Thanks, my fail.


Open file error when files named as "positon.html" | "scorrl.html" ...

I named the file: position.html、scroll.html...
When i opened these file with mune "Open in  Browser", found the error below:
Error: Expected a target
Error: Expected a `target`
    at module.exports (/Users/sunhaibin/.atom/packages/open-in-browser/node_modules/opn/index.js:9:25)
    at Object.open (/Users/sunhaibin/.atom/packages/open-in-browser/lib/open-in-browser.coffee:22:5)
    at Object.openTreeView (/Users/sunhaibin/.atom/packages/open-in-browser/lib/open-in-browser.coffee:19:6)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:259:29)
    at CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:59)
    at CommandRegistry.module.exports.CommandRegistry.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:160:19)
    at AtomEnvironment.module.exports.AtomEnvironment.dispatchContextMenuCommand (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:1185:28)
    at EventEmitter.outerCallback (/Applications/Atom.app/Contents/Resources/app.asar/src/application-delegate.js:325:25)
    at emitThree (events.js:116:13)
    at EventEmitter.emit (events.js:194:7)

Perhaps these file name is used by system so that lead to conflict.

@seasidesun thank you for reporting, going to close this, because seems to be same issue as #43. May you please add in issues #43, what OS you're on.


Add Text Selections to the :players Command

Allowed text in the :players command to copy and pasted. (selectable)
Proof of Functionality:
https://github.com/user-attachments/assets/921d0cda-8a7c-45d4-9917-3cae917a03eb

does it work on mobile with no issues


Add 1.27 feature flag

Issue #, if available:
Description of changes:
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


🚧Create unit tests for extension methods

Complete The Item Below

[X] I have updated the title without removing the 🚧 emoji.

Description
Create unit tests for the following extension methods:
ReleaseExists()
MilestoneExists()
GetByTitle()
GetHtmlUrl()
CloseMilestone()
UpdateMilestoneDescription()
UploadTextFileAsset()
GetLogText() - This is overloaded.  Create tests for both.
Acceptance Criteria

[ ] Unit tests created for the following methods:

[x] ReleaseExists() method unit tests created
[x] MilestoneExists() method unit tests created
[x] GetByTitle() method unit tests created
[x] GetHtmlUrl() method unit tests created
[x] CloseMilestone() method unit tests created
[x] UpdateMilestoneDescription() method unit tests created
[x] ~UploadTextFileAsset() method unit tests created~
[ ] GetLogText() method unit tests created for both overloads

ToDo Items

[X] Change type labels added to this issue.  Refer to the Change Type Labels section below.
[X] Priority label added to this issue.  Refer to the Priority Type Labels section below.
[X] Issue linked to the correct project (if applicable).
[ ] Issue linked to the correct milestone (if applicable).
[x] Draft pull request created and linked to this issue (only required with code changes).

Issue Dependencies
No response
Related Work
No response
Additional Information:
Change Type Labels

Change Type
Label

Bug Fixes
🐛bug

Breaking Changes
🧨breaking changes

New Feature
✨new feature

Workflow Changes
workflow

Code Doc Changes
🗒documentation/code

Product Doc Changes
📝documentation/product

Priority Type Labels

Priority Type
Label

Low Priority
low priority

Medium Priority
medium priority

High Priority
high priority

Code of Conduct

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

Did not create unit tests for the UploadTextFileAsset() method.
Reason: This is because it does file operations using the System.IO.File class.  This is not ideal for testing unless they are abstracted away using System.IO.Abstractions NuGet package.
This will eventually make it into a milestone service for the issue below:

#74


Microsoft docs not working in iframe

WEB PART NOT LOADING: If you are getting the error: "Microsoft 365 learning pathways has a configuration issue. Ask your administrator for assistance". Please review the Diagnosing Pre-Check guidance before submitting your bug.

Submit your question
What is the context of your question

[] Installation
[] Administration
[] Custom Content
[] M365 learning pathways web part

What is your question
Is It possible to link to another page not in SharePoint when creating assets in Learning pathways?. I try to link to Microsoft docs for the power platform content but it keeps on showing a grey space in infoHub.

Provide related screen-shots to help us understand your question

I checked to make sure the domain was present in assets and then i checked the developer tools and console. This was the output below.

So the answer is yes, in that docs.microsoft.com is a host that Microsoft includes, but I know there are some issues with some of the sites, and I know they are working on it as there are more docs content that they want to include.  Anyway, you can follow the patterns that you'll find int he assets.json file
i.e.
https://docs.microsoft.com/en-us/office365/customlearning/dayinthelife?context=context%2Fchromeless&linktarget=true

Should i add the domain in the way you mentioned in assets then?

Ok, I checked, yep, it looks like one of the pages docs doesn't allow iFraming.  Sadly this is mostly out of our hands as the docs team needs to fix their site to allow us to iFrame all of their content.


ResponseTransformers should be able to set the Content-Type header

It'd be nice if my json ResponseTransformer could also set the Content-Type header anywhere it gets used.
Right now I can either:

Create a filter matching the same URLs as my routes that sets the header
Set the header in every one of my handlers

The first one is the better option, but if new routes are created the pattern for the filter has to be updated.

Agree. I've already opened an issue about this moths ago: https://github.com/perwendel/spark/issues/230

Duplicates #230


repeat-count in nested repeats not working

@yanokwa broke Enketo again: https://forum.opendatakit.org/t/xlsform-online-breaking-repeats/16730/8 :)

It's @lognaturel's form! She should get the blame ;)


[documentation] Mobile Nav doesn't work after navigation

The menu burger button seems to be broken after navigation, which is probably a side effect of the experimental Astro page transitions. @jerzakm I assigned this to you since you have the most knowledge about the matter, I can also look into it but probably not before end of next week.

Nevermind, I think #545 is about that already.


Redirect

Visiting http:\approach.xyz  doesn't redirect to https:\approach0.xyz\search instead it downloads a file called download with the following content (Chrome)
<?php
header("HTTP/1.1 301 Moved Permanently");
header("Location: https://approach0.xyz/search/");
?>

While in IE it just displays the page with plaint text from above.

@kingW3  Thank you for reporting.
Issue fixed. Recently I am doing some experiment on my VPS, and http to https rewrite rule is removed on purpose, the problem you see is because my / to /search/ redirection page is in PHP and no PHP handler rule is written in http section of NGINX configuration file.


Add synth script and update CI

Codecov Report

Merging #230 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #230   +/-   ##
======================================
  Coverage    97.8%   97.8%
======================================
  Files          11      11
  Lines        2093    2093
  Branches      128     128
======================================
  Hits         2047    2047
  Misses         25      25
  Partials       21      21

Continue to review full report at Codecov.

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


Docker version Works perfectly fine, but Go build from master branch only works on No TLS

the certificate verification seems good (tried both skip verification and provide CA certification), but dial state stuck in transient failure, reporting error "connection closed".
The docker version is good though with the same config.

This could be related to the go version you are using to build your binary.
Go dropped NPN support in 1.14, rightfully so. ALPN should be used.
Some routers TLS implementations still use NPN, that's why gnmic is still built with go1.13

You are absolutely right, I changed the building environment to go 1.13 and it fixed.
I did notice the difference from tcpdump, but should use the wireshark to check the details.
This makes GNMIC a viable tool to communicate with these "old" routers and push the results to its subscribers which are built in latest Go version.
Great thanks!


Gen2 battery charging rates - new values.

Just had gen2 hybrid installed.
The appropriate values in sensor.py for 3600 battery limits are below: Works for me perfectly alligned to webportal with local edit. Obviously needs some logic based on gen1/gen2 to be usable for all.
class BatteryPowerLimitSensor(InverterBasicSensor):
    """Battery power limit sensor for charging or discharging."""

    def convert_raw_power_to_watts(self, raw_value: int) -> int:
        """Convert an inverter register value to a power value in Watts."""
        # Here be dragons.
        # Interprets the raw value in the same way as the GivEnergy web portal.
        # Step values are 95W up to 38*95=3610W.
        # Tested on a Gen 2 inverter with +/- 3.6kW max battery power
        return 3600 if raw_value > 37 else raw_value * 95

And thanks a lot for this excellent add on !

Thanks for the data. For completeness, and so I've got it on record, it would be very useful if you could just double check some values using the GE web portal. On the Remote Control page, use one of those power sliders to send the 4 highest possible values, and for each one record:

The value the slider let you set
The value that appears under the "Command History" section down the page
The raw value that's displayed when you check "Show Raw Values"

e.g. for my Gen1:
2600W slider = 2600W command history = 50 raw
2511W slider = 2540W command history = 31 raw
2430W slider = 2458W command history = 30 raw
2349W slider = 2376W command history = 29 raw
Also, please confirm the first 2 letters of the inverter serial number.

As requested:
3600W slider = 3600W command history = 50 raw
3420W slider = 3428W command history = 36 raw
3325W slider = 3333W command history = 35 raw
3230W slider = 3238W command history = 34 raw
First 2 letters of the inverter serial number. = ED


Fuzz - Fuzz tests for sharding

Summary

Fuzz tests for sharding package.

Once this is merged in I will create open a PR with https://github.com/google/oss-fuzz to get it integrated.
Release Note
Fuzz tests for sharding
Documentation
NONE

Codecov Report

Merging #1147 (8bed43e) into main (3942a0e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1147   +/-   ##
=======================================
  Coverage   48.89%   48.89%
=======================================
  Files          82       82
  Lines        7620     7620
=======================================
  Hits         3726     3726
  Misses       3194     3194
  Partials      700      700

Flag
Coverage Δ

e2etests
48.89% <ø> (ø)

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

Impacted Files
Coverage Δ

pkg/sharding/sharding.go
48.27% <ø> (ø)

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

@asraa @bobcallaway  👀 Please


Rendering Stop when CZML data timeline is forwarded fast.

Rendering stops when a CZML data's timeline is being forwarded. We have observed this issue from v1.104 onwards.

Sandcastle example: https://sandcastle.cesium.com/index.html#c=bVLNjtowEH4VK6cgNfamUKDdLGpFpV6oemDVQxUJGXtSLBwb2RMQVH33Ok6gZHdzcma+P8+YMbJ0wBEI7oAswaumJkcFJ3ClOXLXn8kTMXDq+/RnrKVlIuL/0hrkygRGMnosTWnYTZQTcan1V458bRsnoNMc1obay0EvvQmuLJddxl/fV0QGBKmcrYNDpXTQHWpSHeAh4A7x4D8xxr0H9FTUnh50CMYb6gC4wx1Vtu8yUbGZmM6kzMRk/CGbwGSczUU+zaZyMoePsprOZ5yhqsGDU+A3+Th/eNig3Z/t+3G2bzbc8T0/8c32kE/mtI0URkJDapNWjRGorElH5E9pCAlX+iK7G70YB9pYvS6B9CcqbxBPuZTpkNYNKup+A4wKYFDhmQirNUTvbmKvLaNJ2EskhIuFjbwY57Xz32Tdmwheg+Nt6DbmzdcB8QIM3OW/WFs/2/Qq1Qb+O3pM3iWFx7OGRQttv8+qPliHpHE6pZQh1HFlnm0bsQekwkduCy3YPbWQ6kiUfHrjXRKhw5JDp2q0XqsLlMmiYAH/ito+HGV+/ziC0/zcwnb5YtUVKaUFC79vM9FaveXuTrlD/AM
Browser: Chrome Version v114.0.5735.133 / Firefox Version v114.0.1
Operating System: MacOS Ventura v13.4 / Windows 11 OS Build 22621.1848

Thanks @pyshx! I added a fix in https://github.com/CesiumGS/cesium/pull/11360. It should go out with the next release on July 3.

@ggetz Thanks for quick response and fix.


[Pebble Clock with distance] clock showing wrong time / frozen

Affected hardware version
Bangle 2
Your firmware version
2v19
The bug
I looked at my watch and time was off by 30 (or so) minutes. I thought something went wrong with system time, but after starting another clock, they showed correct time, and going back to "Pebble with distance" showed correct time.
I suspect clock just froze and did not show correct time for 30? minutes? I was charging my watch in relevant timeframe.
Not sure how to debug this, it only happened once so far.
Installed apps
notify (0.13), sched (0.22), widbt (0.09), widlock (0.08), widid (0.03), sixths (0.01), run (0.16), pebbled (0.04), mylocation (0.10), widclk (0.07), boot (0.59), health (0.27), setting (0.63), recorder (0.30), bikespeedo (0.06), clock_info (0.07), widadjust (0.02), astrocalc (0.06), orloj (0.01), altimeter (0.03), widbat (0.12), f9lander (0.03), a_clock_timer (1.00), folderlaunch (0.03), alarm (0.43), gipy (0.22), dragboard (0.09), waypoints (0.01), magnav (0.06), waypointer (0.09), waypoint_editor (0.02), spacew (0.02)

I imagine what happened is at some point during drawing, something had an error, and then it didn't manage to get to schedule the next draw.
You could just move the 'queue next draw' code to the top of the draw function, which would probably fix this: https://github.com/espruino/BangleApps/blob/master/apps/pebbled/pebbled.app.js#L105C3-L110
But if you want to find the actual issue I'd enable the debug log in settings, then if it happens again you can look at the debug log and find out what the stack trace was.
However: I'm really anxious about the BangeApps repo being full of a bunch of "This happened once and I can't reproduce" issues, as I can just see them piling up and nobody ever bothering to follow up and close if they never happen again... For instance when do you plan to close this issue if you never see the issue happen again?

Well, I consider showing stale time pretty serious. I guess moving the draw would make sense.
I'll take a look at the debug logging.
Would it make sense to have some kind of watchdog (if clock have not repainted for 2 minutes, display failure screen?) or if we should be killing applications that produce stack traces so we don't have this kind of silent fails?

I think that's a hard one to do - we don't know which timeout is for a redraw or something else.
Someone could make an app that caught any uncaught exceptions and restarted I guess. But my point is you've seen this happen once, with one app. It's not exactly a common occurrence.
When do you plan to close this issue if you never see the issue happen again?
Bugs that happen one time on a watch that I have no real idea of the configuration of, in code that I didn't write, with no stack trace and which are not reproducible are basically impossible for me to reliably pinpoint and fix.

I understand that your situation is not easy. OTOH showing wrong time on watch is quite critical failure and.... well... bugs should be reported.

I've added the suggested fix - @pavelmachek, if you do see the issue again and have debug logs turned on, if you could pop them here that'd be great, thanks.

Thank you!


Use shortcode for link to API reference

Follows on from PR #21271
Rather than always linking to v1.16 API docs, link to current release for this docs build.
/kind cleanup

Yes, agreed. Related PR #21216
/lgtm

/approve


patch for getdistreader

This addresses #171, which seems to suggest that anesthetic is behind GetDist, assuming the second column in the output format to be chi2 when it really is -logL.
Fixes #171
Checklist:

[x] I have performed a self-review of my own code
[x] My code is PEP8 compliant (flake8 anesthetic tests)
[x] My code contains compliant docstrings (pydocstyle --convention=numpy anesthetic)
[x] New and existing unit tests pass locally with my changes (python -m pytest)
[x] I have added tests that prove my fix is effective or that my feature works

However, in order to really isolate this, I think that the best way to prove we've fixed it is to create some explicit tests which load our test files in tests/example_data/gd.txt using getdist, and explicitly compares them.

Good idea, currently there is a sign difference between getdist and anesthetic when loading the tests/example_data/gd* data, which means that either the data is outdated or there is some inconsistency with the getdist documentation (according to the getdist docs the second column should be -log(L) which is how I treat it in this PR).

We should ideally also make sure that the example getdist is in the new format (e.g. using -logL rather than chi2).

The tests/example_data/gd* files indeed seem to be out of date. I expect they were generated using CosmoMC? Could you generate and push new ones, @williamjameshandley?

This is also the appropriate PR to do the same for cobaya and montepython chain reading

Yes, I'll start this once #167 is merged.

The tests/example_data/gd* files indeed seem to be out of date. I expect they were generated using CosmoMC? Could you generate and push new ones, @williamjameshandley?

Done in 706a6bc, and updated the automatic generation script.

This is on my to-do list, but I first want to verify that the new CosmoMC behaviour is indeed to put -logL in the second column. Since the new gd.txt data was generated with anesthetic's gen_data.py and not with CosmoMC itself, we should double check this manually.

Ok, I checked with CosmoMC, the second column indeed should be -logL, so this PR should take care of that.


typo: exsist -> exist

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/configure-visual-studio-team-services-to-deploy-biztalk-solutions-or-projects.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add SingleStoreDB to demo page

These are trivial changes, so I just go ahead and merge these right in.


Event will get wrong ProviderInfo when use reuses client transport.

Describe the bug
ClientSyncReceiveEvent may get the wrong ProviderInfo when use reuses client transport with the same remote address.
Expected behavior
When two consumer config (for example A and B) get the same address which provides this A and B two services. They will use same client transport.
And after invoke, It will post ClientSyncReceiveEvent with provider info to the event bus.
Actual behavior
Now in BoltClientTransport, code like this:
if (EventBus.isEnable(ClientSyncReceiveEvent.class)) {
    EventBus.post(new ClientSyncReceiveEvent(transportConfig.getConsumerConfig(),
         transportConfig.getProviderInfo(), request, response, throwable));
}

If this client transport is initial for A, transportConfig.getProviderInfo() is provider info of A, and will be reused by B. When invoke by B, transportConfig.getProviderInfo() will also return provider info of A, It needs fixing.
Environment

SOFARPC version: 5.3.2

one possible solution
when generate map key,we add consumerConfig interface and uniqId
com.alipay.sofa.rpc.transport.ReusableClientTransportHolder#getClientTransport

@Override
    public ClientTransport getClientTransport(ClientTransportConfig config) {
        String key = getAddr(config); //here we make key unique
        ClientTransport transport = clientTransportMap.get(key);
        if (transport == null) {
            transport = ExtensionLoaderFactory.getExtensionLoader(ClientTransport.class)
                .getExtension(config.getContainer(),
                    new Class[] { ClientTransportConfig.class },
                    new Object[] { config });
            ClientTransport oldTransport = clientTransportMap.putIfAbsent(key, transport); // 保存唯一长连接
            if (oldTransport != null) {
                if (LOGGER.isWarnEnabled()) {
                    LOGGER.warn("Multiple threads init ClientTransport with same key:" + key);
                }
                transport.destroy(); //如果同时有人插入，则使用第一个
                transport = oldTransport;
            }
        }


units: 2.21 -> 2.22

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/units/versions.
meta.description for units is: "Unit conversion tool"
meta.homepage for units is: "https://www.gnu.org/software/units/"
meta.changelog for units is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.22 with grep in /nix/store/6s4sx2af48fxq8z7vckhiivvrvwdqfm2-units-2.22
found 2.22 in filename of file in /nix/store/6s4sx2af48fxq8z7vckhiivvrvwdqfm2-units-2.22

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
octavePackages.miscellaneous
octavePackages.vrml
units

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6s4sx2af48fxq8z7vckhiivvrvwdqfm2-units-2.22 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A units https://github.com/r-ryantm/nixpkgs/archive/64e5fe651197f6c6f5fe9b8f40edd40b55dc1726.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6s4sx2af48fxq8z7vckhiivvrvwdqfm2-units-2.22
ls -la /nix/store/6s4sx2af48fxq8z7vckhiivvrvwdqfm2-units-2.22/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package marked as broken and skipped:

octavePackages.vrml

2 packages built:

octavePackages.miscellaneous
units

Maintainer pings
cc @vrthra for testing.

Release is just a bug fix release (see below for changelog)
Builds ok
Basic unit conversions ok
Can verify that definitions of Rinfinity and hartree are different, as listed in the changelogs

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

1 package marked as broken and skipped:

octavePackages.vrml

2 packages built:

octavePackages.miscellaneous
units

changelog
Version 2.22 - 4 September 2022 (bug fix release)

Fixed bug: segfault on inverse conversions of tables in verbose moden
Added support for ExchangeRate-API in units_cur
Minor unit definition fixes and updates, including updates to
planetary data and a change in Rinfinity and hartree to increase accuracy.
Do not update currency.units at build

This pull request has been mentioned on NixOS Discourse.