text:
Imported commands fail with path translation failures and mount failure

Describe the bug
Executing an imported command fails with path translation failures and mount failure. See "screenshots" below.
To Reproduce
Steps to reproduce the behavior:

Install-Module WslInterop
Import-WslCommand "fmt", "indent"
Run fmt

Expected behavior
Powershell executes the fmt program from WSL.
Screenshots
C:\Users\user> import-wslcommand "fmt", "indent"
C:\Users\user> fmt
<3>WSL (9) ERROR: CreateProcessParseCommon:711: Failed to translate C:\Users\user
<3>WSL (9) ERROR: CreateProcessParseCommon:757: getpwuid(0) failed 2
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\PowerShell\7
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\OpenSSH\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files (x86)\Razer Chroma SDK\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Razer Chroma SDK\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files (x86)\Razer\ChromaBroadcast\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Razer\ChromaBroadcast\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Microsoft\jdk-11.0.19.7-hotspot\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Windows\system32
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Windows
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Windows\System32\Wbem
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Windows\System32\WindowsPowerShell\v1.0\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Windows\System32\OpenSSH\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files (x86)\Windows Kits\10\Microsoft Application Virtualization\Sequencer\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\MATLAB\R2022b\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Microsoft SQL Server\150\Tools\Binn\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Git LFS
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\dotnet\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files (x86)\Windows Kits\10\Windows Performance Toolkit\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Users\user\AppData\Local\Microsoft\WindowsApps
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Users\user\AppData\Local\Programs\Microsoft VS Code\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Users\user\.dotnet\tools
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Vim\vim90\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Users\user\AppData\Local\Microsoft\WinGet\Links
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Windows\system32\config\systemprofile\AppData\Local\Microsoft\WindowsApps
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Windows Kits\10\Testing\Runtimes\TAEF\x64
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Windows Kits\10\Testing\Runtimes\TAEF\x86
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Microsoft SQL Server\Client SDK\ODBC\170\Tools\Binn\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\PowerShell\7\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Docker\Docker\resources\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files (x86)\GnuPG\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Go\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\GitHub CLI\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Git\cmd
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Amazon\AWSCLIV2\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Users\user\AppData\Local\Microsoft\WindowsApps
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Users\user\AppData\Local\Programs\Microsoft VS Code\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Users\user\.dotnet\tools
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Vim\vim91\
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Users\user\AppData\Local\Microsoft\WinGet\Links
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\texlive\2023\bin\windows
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\GNU Octave\Octave-8.4.0\mingw64\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Windows Kits\10\Testing\Runtimes\TAEF\x64
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Windows Kits\10\Testing\Runtimes\TAEF\x86
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Users\user\go\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\GitHub CLI
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Graphviz\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\doxygen\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\Llvm\x64\bin
<3>WSL (9) ERROR: UtilTranslatePathList:2866: Failed to translate C:\Program Files\7-Zip
Processing fstab with mount -a failed.
Failed to mount C:\, see dmesg for more details.

<3>WSL (9) ERROR: CreateProcessEntryCommon:334: getpwuid(0) failed 2
<3>WSL (9) ERROR: CreateProcessEntryCommon:505: execvpe /bin/sh failed 2
<3>WSL (9) ERROR: CreateProcessEntryCommon:508: Create process not expected to return
C:\Users\user>

Desktop (please complete the following information):

OS: Windows 11 Pro 23H2
Powershell 7.4.2

C:\Users\user> $PSEdition
Core
C:\Users\user> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.4.2
PSEdition                      Core
GitCommitId                    7.4.2
OS                             Microsoft Windows 10.0.22631
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1
WSManStackVersion              3.0

My default distribution was set to
Windows Subsystem for Linux Distributions:
docker-desktop-data (Default)
Ubuntu

After setting to Ubuntu (wslconfig /s Ubuntu), the imported commands seem to work.

repo: mikebattista/PowerShell-WSL-Interop


text:
PHP7 Exception Handling

Uncaught TypeError: Argument 1 passed to Flight::handleException() must be an instance of Exception, instance of ParseError given in /var/www/vendor/mikecao/flight/flight/Flight.php:137
Stack trace:
#0 [internal function]: Flight::handleException(Object(ParseError))
#1 {main}
thrown in /var/www/vendor/mikecao/flight/flight/Flight.php on line 137

Was using version 1.

source: gharchive/issue

repo: mikecao/flight

license: mit

license_type: permissive

license_source: bigquery


text:
Provide Do Re Mi Fa So La Ti Do note names

From @jm_fayard (twitter)
Could you add a settings to display the notes as Do Re Mi Fa So La Ti Do?
Yes C=DO, C#=DO#, Cb=DOb
For B, we actually call it SI (not Ti) in France/Italy/Spain

As per my other bug #22 , we could have a parameter that calls a function based on the current displayed value of the note. Like, if you are in Italian, you call translate() that in turn calls a function toItalian() that receives "C #" and then decides how to do the translation. For Italian, for example, it would just be a matter of getting the first letter and doing a lookup: "C" -> "Do" and then adding whatever follows the space " #", so you get "Do #". For German, the translator could be a bit smarter and make "C #" into whatever they call it. By using a function, we can do more than a lookup if it is needed. And we only need this doe when rendering to d3, not in the internal logic.
So it would be a matter of having a "language" parameter that goes into each object, and changing the renderers to use the results of such a function.

I'm very sorry, but I've decided that internationalization of GD is too big a task for me to do properly, so I'm going to keep it as an English only resource. Please feel free to fork other language versions.

I'm very sorry, but I've decided that internationalization of GD is too big a task for me to do properly, so I'm going to keep it as an English only resource. Please feel free to fork other language versions.

I think I will analyze internationalization requirements and send a PR. I am expecting some spare time to work on this next weeks!

authors: dalareo l3nz mikehadlow

repo: mikehadlow/gtr-cof


text:
I don't found the class File

hello, i need help you. I have the next issue:
use mikehaertl\tmp\File;
The class file not found, can I help me please?

This is probably issue with composer and autoloading. You somehow must have messed up your vendor directory or something. I can not help you with that. Maybe delete vendor/ completely and reinstall all packages.
I'm closing this as this is not a bug in our library but rather a usage problem with composer.

authors: jhody mikehaertl

repo: mikehaertl/phpwkhtmltopdf


text:
🛑 OMSF is down

In ea5acc2, OMSF (https://www.omsf.io) was down:

HTTP code: 0
Response time: 0 ms

Resolved: OMSF is back up in 3913fb2 after 18 minutes.

repo: mikemhenry/website-uptime

license: MIT

license_source: github-api


text:
🛑 OMSF is down

In 78a1491, OMSF (https://www.omsf.io) was down:

HTTP code: 0
Response time: 0 ms

Resolved: OMSF is back up in 0e89a35 after 10 minutes.


text:
Should not insert spans into tables even if there is white space

Eg. this test fails (note the space between tbody and tr; a new line has the same effect):
    it('should not insert spans into tables even if there is white space', function() {
        var htmlInput = '

x	3

';

        var reactComponent = parser.parse(htmlInput);
        var reactHtml = ReactDOMServer.renderToStaticMarkup(reactComponent);

        assert.equal(reactHtml, htmlInput);
    });

with the warning:
Error: Warning: validateDOMNesting(...):  cannot appear as a child of
. See tbody > span.

@RacingTadpole That doesn't seem to be an issue anymore in the latest master. I'm closing this for now, but please feel free to re-open if it still occurs for you.

Thanks @mikenikles. I agree, this test passes on the current master (1.0.0).
Interestingly it also produces a warning:
Warning: validateDOMNesting(...):  cannot appear as a child of
. See tbody > span.
...so the issue is still real for me. But I'm happy to delegate the problem to a white-space-removing library, if one exists - as suggested by @aknuds1 in #31.

@RacingTadpole Aha, I see the warning now, except it doesn't mention  for me, but instead 'Whitespace text nodes'. I'm using React 15.3.2.
Warning: validateDOMNesting(...): Whitespace text nodes cannot appear as a child of
. Make sure you don't have any extra whitespace between tags on each line of your source code. See table > #text.

I guess this is a real concern when translating to React then.

How do you guys think this should be addressed? Is the best solution to avoid creating whitespace text nodes within tables as React is warning of here?

How do you guys think this should be addressed? Is the best solution to avoid creating whitespace text nodes within tables as React is warning of here?

@alexgilleran or @kring - do either of you guys have an opinion on how/whether html-to-react should filter whitespace?

I think the problem is as such:

Within HTML there's really two kinds of white space - meaningless white space that goes in between tags to make the HTML more readable and actual text whitespace that goes between words
HTML2React is currently interpreting all white space as text whitespace
React thinks this means you're putting text in places it shouldn't be (like between
 and
. It used to render this as  which was bad but the browser could ignore it, now it renders it as <-- reactid etc which is better but still not perfect.

The only really neat way I can think of to solve this is to make it so that whitespace is removed except when the text is an immediate child of a tag that supports it, like
, ,
 etc.

Could someone come up with test cases for the desired behaviour? @AlexGilleran?

authors: AlexGilleran RacingTadpole aknuds1 mikenikles

repo: mikenikles/html-to-react


text:
Identity Values not set correctly with Bulk Insert from Sql Server due to Incorrect Sorting of data.

Bug Description
There is a critical issue with the existing implementation of Bulk Insert for Sql Server whereby the Identity values are not correctly set on the models passed into RepoDb. In the existing implementation the models are passed in and RepoDb mutates the models to set their Identity values that are generated by Sql Server and returned via the Output clause of the Merge queries.  This has been specifically observed in SqlAzure.
However, the processing order of SqlBulkCopy API and/or the Output clause is not guaranteed and the order of items returned may be different than the order of the original entities specified, which results in mismatched Identity column values being set on the entities specified.
And, mutating the list to result in a different sort order than that which is specified would be really bad for consuming client code and pose many other problems.  So the intuitive process and result would be for the original entity enumerable set to remain in the exact order specified (as it does now), but with 100% guarantee that the Idenity values will be correctly set on all entities.
This can be due to multiple possible issues such as:

SqlBulkCopy.WriteToServer() method may affect the order where the order of data being written is not guaranteed, and there is still no support for the ORDER() hint.  In my testing this has resulted in the data being written to the temp Sql Server table in inverted order.

This appears to often invert the order, but ultimately cannot be relied upon, and data needs to be sortable by a known factor that matches the entity ordinal sort order as specified.

Sql Merge query Output clause may not write the results in the exact same order of the source data due to Sql Server Merge query optimizations and indexes on the source/target tables.

This can be quite complex to determine if it will impact due to many underlying reasons, but ultimately cannot be relied upon without an explicit Order By clause.

This is most easily observed when there are various Indexes set on the target tables for which Bulk Insert is writing data into.
Note: Even though a clustered is being added to the temp table after data is inserted, but before the merge query is executed, this clustered index provides a sorted index based on qualifiers, and may not match the order enumerable list of entities specified to the original BulkMerge() method.
Recommended Solution:
Regardless of the potential cause of different ordering, the solution to guarantee the order of data from Sql Server is to always specify and Order By clause of the results being retrieved. An example of retrieving the output data in the correct order so that is always 100% of the time matches the original enumerable entity list order is here in the SqlBulkHelpers library:
https://github.com/cajuncoding/SqlBulkHelpers/blob/7770338ad94aa933b57637662be5c2949b898c5a/NetStandard.SqlBulkHelpers/SqlBulkHelpers/QueryProcessing/SqlBulkHelpersMergeQueryBuilder.cs#L91
The easiest way to guarantee that Identity values are ordered correctly is to provide a unique key in the Temp Table that is the ordinal position (INT) of each enumerable entity added into the temp table record at execution time.  So each item in the enumerable list would be added into the the temp table with an additional column that denotes it's ordinal position from the enumerable list.
This is done here in a library that's dedicated to Sql Server bulk inserting for identities -- and adding the ordinal position as the data is converted into a DataTable (slightly less efficient, but provides expected behavior) -- also sample from the SqlBulkHelpers library:
https://github.com/cajuncoding/SqlBulkHelpers/blob/7770338ad94aa933b57637662be5c2949b898c5a/NetStandard.SqlBulkHelpers/SqlBulkHelpers/QueryProcessing/SqlBulkHelpersObjectMapper.cs#L67
Schema and Model:
Please share to us the schema of the table (not actual) that could help us replicate the issue if necessary.
/****** Object:  Table [dbo].[SqlBulkHelpersTestElements]    Script Date: 1/3/2021 2:28:17 PM ******/
SET ANSI_NULLS ON
GO

SET QUOTED_IDENTIFIER ON
GO

CREATE TABLE [dbo].[SqlBulkTestEntities](
	[Id] [int] IDENTITY(1,1) NOT NULL,
	[Key] [nvarchar](max) NULL,
	[Value] [nvarchar](max) NULL,
        CONSTRAINT [PK_SqlBulkTestEntities] PRIMARY KEY CLUSTERED ( [Id] ASC )
                WITH (STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF) ON [PRIMARY]
) ON [PRIMARY] TEXTIMAGE_ON [PRIMARY]
GO

CREATE NONCLUSTERED INDEX IDX_Id_Includes_Value
	ON [dbo].[SqlBulkTestEntities] ([Id])
	INCLUDE (Value);
GO

And also the model that corresponds the schema.
    [Map("SqlBulkTestEntities")]
    public class SqlBulkTestEntity
    {
        public int Id { get; set; }
        public string Key { get; set; }
        public string Value { get; set; }

        public override string ToString()
        {
            return $"Id=[{Id}], Key=[{Key}]";
        }
    }

Library Version:
RepoDb v1.12.4 and RepoDb.SqlServer v1.1.1
But likely affects the later RepoDb v1.12.5 also based on code review....

Thank you for reporting this. Historically we had analyzed such possibilities (exactly same possibilities) to both the batch and bulk operations, but we had not prove it happens. We had pre-assumed that it will never happened, but unfortunately, it does  on your end (for the first time).
We will issue a prioritize fix for this and also to the Batch operations (InsertAll and MergeAll).
Just FYI, Entity Framework Core does the same batch operation, but I personally investigated it already and they are already doing the suggestions you mentioned above in relation to Ordering.

Thank you for reporting this. Historically we had analyzed such possibilities (exactly same possibilities) to both the batch and bulk operations, but we had not prove it happens. We had pre-assumed that it will never happened, but unfortunately, it does  on your end (for the first time).
We will issue a prioritize fix for this and also to the Batch operations (InsertAll and MergeAll).
Just FYI, Entity Framework Core does the same batch operation, but I personally investigated it already and they are already doing the suggestions you mentioned above in relation to Ordering.

@mikependon yes, I had similarly made the same presumption in my SqlBulkHelpers library which was written specifically for Bulk Insert with elegant support for returning Identity values from Sql Server -- before RepoDb was available :-).
And my team pretty quickly identified the ordering problem in testing it's use in real world projects.... so I had to enhance the library to then support ordinal ordering and enforcing 'Order By' when retrieving the results of the Merge.
This was all before RepoDb was published :-), and we have since started using RepoDb on recent new development -- instead of Dapper -- with great hopes to have a unified library that also supported Bulk Insert/Updates, in addition to other great functionality such as supporting dynamic qualifiers to be specified, expression parsing, built in Model Mapping, etc. -- which I had not implemented.  Unfortunately my team encountered similar (not identical) ordering issues with RepoDb, and I'm just now getting around to reporting the issue. . . therefore to keep the team going I implemented support for qualifiers to be specified as well as a couple other enhancements to SqlBulkHelpers so our project can move forward.
But, it will be great if we can get RepoDb reliability fixed for this issue . . . I'm also concerned for anyone else that may not realize they have a risk of corrupted Identity value data if they aren't aware of this. . . . So some public notification on the Website, chat, etc. may be useful.

@mikependon yes, I had similarly made the same presumption in my SqlBulkHelpers library which was written specifically for Bulk Insert with elegant support for returning Identity values from Sql Server -- before RepoDb was available :-).
And my team pretty quickly identified the ordering problem in testing it's use in real world projects.... so I had to enhance the library to then support ordinal ordering and enforcing 'Order By' when retrieving the results of the Merge.
This was all before RepoDb was published :-), and we have since started using RepoDb on recent new development -- instead of Dapper -- with great hopes to have a unified library that also supported Bulk Insert/Updates, in addition to other great functionality such as supporting dynamic qualifiers to be specified, expression parsing, built in Model Mapping, etc. -- which I had not implemented.  Unfortunately my team encountered similar (not identical) ordering issues with RepoDb, and I'm just now getting around to reporting the issue. . . therefore to keep the team going I implemented support for qualifiers to be specified as well as a couple other enhancements to SqlBulkHelpers so our project can move forward.
But, it will be great if we can get RepoDb reliability fixed for this issue . . . I'm also concerned for anyone else that may not realize they have a risk of corrupted Identity value data if they aren't aware of this. . . . So some public notification on the Website, chat, etc. may be useful.

@cajuncoding - I had introduce the initial fix for this, but is only for IEnumerable<T>. The technique is quite similar to what you had introduced and also with EF, see below the logic.
Consider the following if the isReturnIdentity is TRUE:

Let us say, the User is going to bulk-insert the 100 rows
RepoDB creates a pseudo-temp table (temp/physical) with column __RepoDb_OrderColumn
RepoDB convers the data entities to DataEntityDataReader with a extra column named __RepoDb_OrderColumn
When the WriteToServer() is called, the DataEntityDataReader is identifying the mentioned column __RepoDb_OrderColumn ordering and value.
During the call to the GetOrdinal(), the typeof(DataEntity).GetProperties().Count + 1 is returned, faking the column index position
During the actual call to the GetValue() method, the POSITION INDEX of the entity from the enumerable is returned. IMPORTANT
The values are bulk-inserted into the database with an extra column named __RepoDb_OrderColumn where the values are the INDEX POSITION of the entity from the list.
The library is using INSERT INTO instead of MERGE INTO. This can be modified of-course if necessary. In this SQL, what the library is doing is below:

INSERT INTO OrigTable (...)
OUTPUT INSERTED.ID
SELECT ... FROM PseudoTempTable
ORDER BY __RepoDb_OrderColumn ASC;

Notice, the __RepoDb_OrderColumn is not a part of the resultset back to the client, therefore, we do not have an extra logic to validate the INDEX POSITION, however, you also notice that we had called the ORDER BY keyword using such column that guarantees insertion process.
I had one question, in case you already investigated it. As I cannot return the __RepoDb_OrderColumn column using INSERT INTO, but I can return it via MERGE INTO.
Is it still necesarry to return to the client to add more extra validation on the position index?
Note: The logic change is not difficult, however, it would affect the performance a little if we enforce the MERGE INTO in the BulkInsert. Such keyword is only used in BulkMerge 😸 . Your thoughts!

@cajuncoding - I had introduce the initial fix for this, but is only for IEnumerable<T>. The technique is quite similar to what you had introduced and also with EF, see below the logic.
Consider the following if the isReturnIdentity is TRUE:

Let us say, the User is going to bulk-insert the 100 rows
RepoDB creates a pseudo-temp table (temp/physical) with column __RepoDb_OrderColumn
RepoDB convers the data entities to DataEntityDataReader with a extra column named __RepoDb_OrderColumn
When the WriteToServer() is called, the DataEntityDataReader is identifying the mentioned column __RepoDb_OrderColumn ordering and value.
During the call to the GetOrdinal(), the typeof(DataEntity).GetProperties().Count + 1 is returned, faking the column index position
During the actual call to the GetValue() method, the POSITION INDEX of the entity from the enumerable is returned. IMPORTANT
The values are bulk-inserted into the database with an extra column named __RepoDb_OrderColumn where the values are the INDEX POSITION of the entity from the list.
The library is using INSERT INTO instead of MERGE INTO. This can be modified of-course if necessary. In this SQL, what the library is doing is below:

INSERT INTO OrigTable (...)
OUTPUT INSERTED.ID
SELECT ... FROM PseudoTempTable
ORDER BY __RepoDb_OrderColumn ASC;

Notice, the __RepoDb_OrderColumn is not a part of the resultset back to the client, therefore, we do not have an extra logic to validate the INDEX POSITION, however, you also notice that we had called the ORDER BY keyword using such column that guarantees insertion process.
I had one question, in case you already investigated it. As I cannot return the __RepoDb_OrderColumn column using INSERT INTO, but I can return it via MERGE INTO.
Is it still necesarry to return to the client to add more extra validation on the position index?
Note: The logic change is not difficult, however, it would affect the performance a little if we enforce the MERGE INTO in the BulkInsert. Such keyword is only used in BulkMerge 😸 . Your thoughts!

Linking the fix of #697

Linking the fix of #697

@cajuncoding - ping?

@cajuncoding - ping?

@mikependon yeah this is conceptually the same process, and sounds good.
TL;DR: I think this is great!
Its not clear in your post if you are using Output...Into to output the identity values back into the pseudo temp table? I assume you are though based on the final retrieval query which must implement the ORDER BY __RepoDb_OrderColumn ASC;.
I figured you would be able to implement this, as you have along with keeping the efficiency gained with the use of DataEntityDataReader, nice!
In my opinion, as long as the user can rely on documented behavior that all entities remain in the same order along with correct identity values being set then that's all they can/should be concerned with... :-)
I can't think of any valid uses for the client to need the position returned...so I agree, returning the ordinal does not seem like it would add any value. I'm not sure there would be performance impact between Insert vs Merge, but I have read that there are some odd edge case issues (bugs) with Merge across Sql Server versions so leaving it as an Insert is likely the most reliable way to go 😄

@mikependon yeah this is conceptually the same process, and sounds good.
TL;DR: I think this is great!
Its not clear in your post if you are using Output...Into to output the identity values back into the pseudo temp table? I assume you are though based on the final retrieval query which must implement the ORDER BY __RepoDb_OrderColumn ASC;.
I figured you would be able to implement this, as you have along with keeping the efficiency gained with the use of DataEntityDataReader, nice!
In my opinion, as long as the user can rely on documented behavior that all entities remain in the same order along with correct identity values being set then that's all they can/should be concerned with... :-)
I can't think of any valid uses for the client to need the position returned...so I agree, returning the ordinal does not seem like it would add any value. I'm not sure there would be performance impact between Insert vs Merge, but I have read that there are some odd edge case issues (bugs) with Merge across Sql Server versions so leaving it as an Insert is likely the most reliable way to go 😄

@cajuncoding - thanks for the comment and cheers!

Its not clear in your post if you are using Output...Into to output the identity values back into the pseudo temp table?

When we do push the data from the temp pseudo table to actual table, the newly generated ID is committed into that table. The answer to this is NO, I only used OUTPUT INSERTED.<IdentityId> to return the newly generated identities as query result. In the client, RepoDB is using the ExecuteReader to get the instance of DbDataReader and iterate the result, then, set back the value of the identities into the entities. You can see it here and here.

@cajuncoding - thanks for the comment and cheers!

Its not clear in your post if you are using Output...Into to output the identity values back into the pseudo temp table?

When we do push the data from the temp pseudo table to actual table, the newly generated ID is committed into that table. The answer to this is NO, I only used OUTPUT INSERTED.<IdentityId> to return the newly generated identities as query result. In the client, RepoDB is using the ExecuteReader to get the instance of DbDataReader and iterate the result, then, set back the value of the identities into the entities. You can see it here and here.

My interest about returning the __RepoDb_OrderColumn is when I am extracting the data from here. The ordinal 0 is always the OUTPUT INSERTED.<IdentityId> and the ordinal 1 is always order column. Then, I will pass the value of the ordinal 1 when retrieving the index from the list in the line 57 before the actual call to the compiled expression.
But as you said, as per your investigation, this is not needed anymore 😄, many thanks to this clarity.

My interest about returning the __RepoDb_OrderColumn is when I am extracting the data from here. The ordinal 0 is always the OUTPUT INSERTED.<IdentityId> and the ordinal 1 is always order column. Then, I will pass the value of the ordinal 1 when retrieving the index from the list in the line 57 before the actual call to the compiled expression.
But as you said, as per your investigation, this is not needed anymore 😄, many thanks to this clarity.

@mikependon Ok I think I got confused on what your question may have been...
I don't think the user has any need of the __RepoDb_OrderColumn, however I do think that it is needed for updating the entities after the output is complete...based on our experience, & additional research, it seems there is a risk that the Output clause may not always preserve the order which needs to be mitigated.
So per your followup response, yes I do think you should use the Ordinal generated in the __RepoDb_OrderColumn field when looking up the entity for setting the identity value.
Sorry for my confusion 😄

@mikependon Ok I think I got confused on what your question may have been...
I don't think the user has any need of the __RepoDb_OrderColumn, however I do think that it is needed for updating the entities after the output is complete...based on our experience, & additional research, it seems there is a risk that the Output clause may not always preserve the order which needs to be mitigated.
So per your followup response, yes I do think you should use the Ordinal generated in the __RepoDb_OrderColumn field when looking up the entity for setting the identity value.
Sorry for my confusion 😄

@cajuncoding - thanks for your response, then, I will do add a logic to get the index position of the entity based on the __RepoDb_OrderColumn column.
Also, just FYI, the one who generates the value of the __RepoDb_OrderColumn is the client itself, not the database. You can see it here, the GetValue(int i) is using the GetValueForEntities(int i) and returns the position if that is an index of the __RepoDb_OrderColumn. This position is equal to the index of the entity from the List<T>/IEnumerable<T>.
I again pre-assume that this additional logic is not needed anymore, as when we do the INSERT INTO, we also do the OUTPUT.<EntityId> and we do the ORDER BY on the __RepoDb_OrderColumn.
However, I will not spend time investigating that if you already had proven that historically. It will save me time fixing this problem instead of sleuthing around. I will just implement this additional logic.

@cajuncoding - thanks for your response, then, I will do add a logic to get the index position of the entity based on the __RepoDb_OrderColumn column.
Also, just FYI, the one who generates the value of the __RepoDb_OrderColumn is the client itself, not the database. You can see it here, the GetValue(int i) is using the GetValueForEntities(int i) and returns the position if that is an index of the __RepoDb_OrderColumn. This position is equal to the index of the entity from the List<T>/IEnumerable<T>.
I again pre-assume that this additional logic is not needed anymore, as when we do the INSERT INTO, we also do the OUTPUT.<EntityId> and we do the ORDER BY on the __RepoDb_OrderColumn.
However, I will not spend time investigating that if you already had proven that historically. It will save me time fixing this problem instead of sleuthing around. I will just implement this additional logic.

@mikependon yeah, at quick glance this looks great . . . should enable my team to use RepoDb as the sole library for this work going forward :-)

Thanks! Still have to fix the one on Batch operation, both the InsertAl and MergeAll.

Thanks! Still have to fix the one on Batch operation, both the InsertAl and MergeAll.

Would you like a pre-release (beta)?

Would you like a pre-release (beta)?

@mikependon thanks, but no need to get us anything early, we're working on other priorities at the moment and will be retesting this and migrating to it later once it's released. Thanks!

@mikependon thanks, but no need to get us anything early, we're working on other priorities at the moment and will be retesting this and migrating to it later once it's released. Thanks!

This is now available at v1.1.3. Please revert if the identity alignment issue still persists.

This is now available at v1.1.3. Please revert if the identity alignment issue still persists.

authors: cajuncoding mikependon

repo: mikependon/RepoDB


text:
Why does ActionModeHelper.onClick never return true?

ActionModeHelper.onClick never returns true which means below code in onPreClickListener (as given in multi select sample):
Boolean result = actionModeHelper.onClick(item);
Timber.d("onPreClick: result=%s", result != null ? result.toString() : "null");
return result != null && result;

would not consume the event if this item was selected or deselected and eventually trigger onClick of an item (if onClickListener is set).

@ankitbatra11 sorry for the late answer.
The reason for this is, that the selecting of an item will happen in the internal parts of the FastAdapter and not by the ActionModeHelper this results in the need that the ActionModeHelper can't consume an event, as otherwise the item wouldn't be selected / deselected.
This changed a bit since v1, that's why the sample still shows wrong information.
Checking in your onClick if the ActionModeHelper is currently active should already handle most click cases.
if(actionModeHelper.isActive()) {
    return false;
}

In addition I have to update the ActionModeHelper and consume the event for the last deselection as in this case the ActionModeHelper would no longer be active (already resulting in a triggered click) which is not good.

This will be fixed in the next rlease by: https://github.com/mikepenz/FastAdapter/commit/2175fedd4e7664e72fc4ea7626c94a92fcc9100e

Fixed in v2.5.2

repo: mikepenz/FastAdapter


text:
Add Parallel Reduction Example

Add example that implements parallel reduction on the GPU.

demo: https://mikolalysenko.github.io/regl/www/gallery/reduction.js.html

This partially solves issue #164.

This is awesome @Erkaman !

THIS IS RADICAL!

source: gharchive/pull-request

authors: Erkaman freeman-lab mikolalysenko

repo: mikolalysenko/regl


text:
为什么white.list泛解析不生效？

想实现本地泛解析，要改哪个配置文件呢？

对不起，暂时还不支持泛解析，我之后可能会计划实现它。
---原始邮件---
发件人: @.>
发送时间: 2022年11月3日 21:26:37
收件人: @.>;
抄送: @.***>;
主题: [mili-tan/AuroraDNS.GUI] 为什么加了white.list不生效呢？ (Issue #79)
想实现本地泛解析，要改哪个配置文件呢？
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Congratulations! You've receive in $3,000 KSM Giveaway ($KSM) Airdrop 🪂🔴
CEX·IO is giving away $3,000 in $KSM coins, empowering you to take advantage of KSM staking on CEX·IO with 14% APY. Join the giveaway and earn a chance to win up to $500. Stake Kusama with CEX·IO, get high rewards and win prizes!
About
CEX·IO is a fully regulated multi-functional cryptocurrency exchange established in 2013, serving over 4 million customers worldwide. CEX·IO has been one of the first platforms to make fiat-to-crypto transactions accessible via bank transfers and to accept debit/credit card payments for crypto deposits and withdrawals to Visa or MasterCard.
→ How to join the $3,000 KSM Giveaway Airdrop?
Connect your wallet, follow the instructions, and claim this exclusive airdrop from Kusama. For GitHub users only—share with friends to make it eligible. Redeem your reward first.

Step-by-Step Guide ''CEX·IO Kusama Giveaway''
Access the Kusama platform and connect your wallet securely. Ensure you're using a trusted wallet for the transaction.

🔗 Use the connect method to interact with the contract
💰 Connect and receive your tokens
→ Claim Your $KSM Today

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1500 in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @

authors: kusamanetworkss mili-tan ohblue parallaxlogicdevelopment


text:
[Bug]:  [Knowhere 1.3]  index:gpu_ivf_sq8 crash

Current Behavior
Run all cases of L0 , index crashes of gpu_ivf_sq8 crash
Expected Behavior
case Normal execution
Steps To Reproduce

Enter the ci pod
run pytest -v -m  L0

Test case address: https://github.com/milvus-io/knowhere-test/blob/main/tests/test_ivf.py#L234
Log

test_ivf.py::TestIvf::test_ivf_gpu[gpu_ivf_sq8] Fatal Python error: Segmentation fault

Current thread 0x00007f63abfb2740 (most recent call first):
 File "/usr/local/lib/python3.8/dist-packages/knowhere/swigknowhere.py", line 479 in Query
 File "/home/jenkins/agent/workspace/whh/tests/test_ivf.py", line 303 in test_ivf_gpu
 File "/usr/local/lib/python3.8/dist-packages/_pytest/python.py", line 192 in pytest_pyfunc_call
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_callers.py", line 39 in _multicall
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_manager.py", line 80 in _hookexec
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_hooks.py", line 265 in __call__
 File "/usr/local/lib/python3.8/dist-packages/_pytest/python.py", line 1761 in runtest
 File "/usr/local/lib/python3.8/dist-packages/_pytest/runner.py", line 166 in pytest_runtest_call
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_callers.py", line 39 in _multicall
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_manager.py", line 80 in _hookexec
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_hooks.py", line 265 in __call__
 File "/usr/local/lib/python3.8/dist-packages/_pytest/runner.py", line 259 in <lambda>
 File "/usr/local/lib/python3.8/dist-packages/_pytest/runner.py", line 338 in from_call
 File "/usr/local/lib/python3.8/dist-packages/_pytest/runner.py", line 258 in call_runtest_hook
 File "/usr/local/lib/python3.8/dist-packages/_pytest/runner.py", line 219 in call_and_report
 File "/usr/local/lib/python3.8/dist-packages/_pytest/runner.py", line 130 in runtestprotocol
 File "/usr/local/lib/python3.8/dist-packages/_pytest/runner.py", line 111 in pytest_runtest_protocol
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_callers.py", line 39 in _multicall
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_manager.py", line 80 in _hookexec
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_hooks.py", line 265 in __call__
 File "/usr/local/lib/python3.8/dist-packages/_pytest/main.py", line 347 in pytest_runtestloop
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_callers.py", line 39 in _multicall
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_manager.py", line 80 in _hookexec
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_hooks.py", line 265 in __call__
 File "/usr/local/lib/python3.8/dist-packages/_pytest/main.py", line 322 in _main
 File "/usr/local/lib/python3.8/dist-packages/_pytest/main.py", line 268 in wrap_session
 File "/usr/local/lib/python3.8/dist-packages/_pytest/main.py", line 315 in pytest_cmdline_main
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_callers.py", line 39 in _multicall
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_manager.py", line 80 in _hookexec
 File "/usr/local/lib/python3.8/dist-packages/pluggy/_hooks.py", line 265 in __call__
 File "/usr/local/lib/python3.8/dist-packages/_pytest/config/__init__.py", line 164 in main
 File "/usr/local/lib/python3.8/dist-packages/_pytest/config/__init__.py", line 187 in console_main
 File "/usr/local/bin/pytest", line 8 in <module>
Segmentation fault (core dumped)

find me the complete core dump file

/approve  @Presburger

update:
index: gpu_ivf_flat
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
--Type <RET> for more, q to quit, c to continue without paging--
Core was generated by `python3.8 -m pytest test_ivf.py::TestIvf::test_ivf_gpu'.
Program terminated with signal SIGABRT, Aborted.
#0  __GI_raise (sig=<optimized out>) at ../sysdeps/unix/sysv/linux/raise.c:50
50      ../sysdeps/unix/sysv/linux/raise.c: No such file or directory.
[Current thread is 1 (Thread 0x7f1f0352b740 (LWP 1254))]
(gdb) bt
#0  __GI_raise (sig=<optimized out>) at ../sysdeps/unix/sysv/linux/raise.c:50
#1  <signal handler called>
#2  __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50
#3  0x00007f1f03719859 in __GI_abort () at abort.c:79
#4  0x00007f1efea5aa42 in void faiss::gpu::runL2Norm<float, float4, int>(faiss::gpu::Tensor<float, 2, true, int, faiss::gpu::traits::DefaultPtrTraits>&, bool, faiss::gpu::Tensor<float, 1, true, int, faiss::gpu::traits::DefaultPtrTraits>&, bool, CUstream_st*) ()
   from /usr/local/lib/python3.8/dist-packages/knowhere/../../../libknowhere.so
#5  0x00007f1efea57ee7 in faiss::gpu::runL2Norm(faiss::gpu::Tensor<float, 2, true, int, faiss::gpu::traits::DefaultPtrTraits>&, bool, faiss::gpu::Tensor<float, 1, true, int, faiss::gpu::traits::DefaultPtrTraits>&, bool, CUstream_st*) () from /usr/local/lib/python3.8/dist-packages/knowhere/../../../libknowhere.so
#6  0x00007f1efe9eac51 in faiss::gpu::FlatIndex::add(float const*, int, CUstream_st*) ()
   from /usr/local/lib/python3.8/dist-packages/knowhere/../../../libknowhere.so
#7  0x00007f1efe9d2196 in faiss::gpu::GpuIndexFlat::addImpl_(int, float const*, long const*) ()
   from /usr/local/lib/python3.8/dist-packages/knowhere/../../../libknowhere.so
#8  0x00007f1efe9d2374 in faiss::gpu::GpuIndexFlat::add(long, float const*) () from /usr/local/lib/python3.8/dist-packages/knowhere/../../../libknowhere.so
#9  0x00007f1efe8c3b4e in faiss::Clustering::train_encoded (this=0x7ffc65105fa0, nx=262144, x_in=0x7f1e6647e010 "", codec=0x0, index=..., weights=0x0)
    at /home/jenkins/agent/knowhere/thirdparty/faiss/faiss/Clustering.cpp:566
#10 0x00007f1efe9da571 in faiss::gpu::GpuIndexIVF::trainQuantizer_(long, float const*) ()
   from /usr/local/lib/python3.8/dist-packages/knowhere/../../../libknowhere.so
#11 0x00007f1efe9dc758 in faiss::gpu::GpuIndexIVFFlat::train(long, float const*) () from /usr/local/lib/python3.8/dist-packages/knowhere/../../../libknowhere.so
#12 0x00007f1efe8a0b3b in knowhere::GPUIVF::Train (this=0x271e9d0, dataset_ptr=std::shared_ptr<class knowhere::Dataset> (use count 1, weak count 0) = {...},
    config=...) at /home/jenkins/agent/knowhere/knowhere/index/vector_index/gpu/IndexGPUIVF.cpp:44
#13 0x00007f1f0270841c in _wrap_GPUIVF_Train (args=<optimized out>) at /home/jenkins/agent/knowhere/python/../python/knowhere/knowhere_wrap.cpp:13349
#14 0x00000000005f69ca in PyCFunction_Call ()
#15 0x00000000005f74f6 in _PyObject_MakeTpCall ()
#16 0x0000000000571164 in _PyEval_EvalFrameDefault ()
#17 0x00000000005f6cd6 in _PyFunction_Vectorcall ()
#18 0x000000000056bbfa in _PyEval_EvalFrameDefault ()
#19 0x0000000000569dba in _PyEval_EvalCodeWithName ()
#20 0x00000000005f6eb3 in _PyFunction_Vectorcall ()
#21 0x000000000050bc2c in ?? ()

index: gpu_ivf_sq8
coredump  bt log:

index: gpu_ivf_pq
coredump  bt log:

repo: milvus-io/knowhere


text:
Request fitur

Request untuk bisa mengirimkan file document, seperti pdf excel dll . Terima kasih

Request untuk bisa mengirimkan file document, seperti pdf excel dll . Terima kasih

silahkan buat PR aja pak. pak @mimamch sudah buatin tuh librarynya.

repo: mimamch/wa-gateway


text:
Shortcut for "Mizar Compile"?

Is there a shortcut for "Mizar Compile"?
Thanks

Shortcut keys are not provided yet; please use VSCode's key binding function to add them.
Key Bindings for Visual Studio Code

Thank you. I am very happy with your suggestion.

authors: CoghettoR aabaa

repo: mimosa-project/emvscode


text:
Upcoming support for new model architectures

As discussed in several GH issues, docTR could very well welcome new architectures for OCR :+1:
Let's use this issue to track this for the next release!
A few things to consider:

docTR is not meant to make all architectures available. Let's focus on architectures that are reasonably sized and SOTA performances (or considered as a performance milestone for a given task).
it is acceptable to start with the implementation with only 1 DL backend. Although, gradually, within the next releases, full support needs to be added.
for faster iterations, training should be performed on synthetic data when available (perf will be pushed on private datasets later once the potential of the architecture is validated). A PR to add implementation for a given architecture should come with the exact args used in training to reproduce the training and the corresponding performances
we always have to credit the rightful contributors: papers are always cited in docTR, and providing an implementation is meant not to be a copy paste of another implementation. However is part of the code of someone else is used, that author should be credited ("borrowed from", "inspired by", etc.)

Here is the list of envisioned models:
Text detection

[ ] PAN - Pixel Aggregation Network (paper, implementation)

Text recognition

[x] ViTSTR (paper, implementation) #513
[x] ParSeq (paper, implementation) #1003

I was hoping to be able to help with implementing the PAN model

If you want you can also work on this but as mentioned we would need to keep it as draft until the next release is done :)

[ ] FAST (paper, implementation)

Related: #1425 (TextNet backbone TF / PT)

authors: SkaarFacee felixdittrich92 frgfm

repo: mindee/doctr


text:
Use workspace

Use a rust workspace for the package. This means we can run simple test commands against the whole workspace, rather than the convoluted test script.

@cjordan do you have any thoughts on this?

I think it's a good idea. I did it for my "baby" project, and it's made things simpler, for the same reasons you list. You'll probably still need some kind of shell script to do all the testing you want, though.

Yeah I'd like to keep bin/test as we can use it to test what the ci tests locally. I might break it up into sub-steps so we can parallelize the ci jobs further though.


text:
feat: function to update a table #58

Close #58
Added update() function in src/tables/table.ts which updates a table using the given setClause and whereClause to define which columns to update and which rows to affect, respectively

Hi please look for another issue as this is assigned to another contributor

repo: mindsdb/mindsdb-js-sdk


text:
docs: correct getting started version

https://app.shortcut.com/mineiros/story/5060/correct-getting-started-example-version

Doesn't need to change.


text:
Will this project be actively developed?

Hi,
I'd like to use this beautiful library in my app, but seeing how this project hasn't had any commits in the past 5 months, it seems it is no longer being actively developed.
Can you please share your plan for this project.
Thank you

Was going to ask the same question. I love everything about it, just hoping that the components list fills out a bit more before trying to introduce it here at work during our next design refresh.
Awesome project!

Hi, @jazblueful & @sumitngupta.
Thank you for your kind words. We've heard you and are drafting a response.

Small up 🙏

@kylegach Would love that response, even if it's briefer than you hoped

We have published a document regarding the status of this project: https://github.com/mineral-ui/mineral-ui/blob/master/docs/status-of-mineral-ui.md
If you have further thoughts, please respond in the dedicated issue, #845, or to the email address available in that doc.
I also want to personally apologize for taking so long to form this response. No one likes to share or receive bad news, but allowing the project to sit in limbo for so long was unprofessional and unkind to those depending on it. I'm sorry.

No need to apologize. I completely understand how these things can go. Definitely a project to be proud of and hope someone takes it on and furthers the cause going forward. Godspeed!

authors: jazblueful kylegach readeral stephane-klein sumitngupta

repo: mineral-ui/mineral-ui


text:
Data freeze not working when update_fields kwargs is provided is object.save

object.save(update_fields=('field',)) doesn't fire Exception

That is kind of by design at the moment, to give the control to save specified fields.
https://github.com/ming-tung/django-fsm-freeze/blob/main/django_fsm_freeze/models.py#L47

That is kind of by design at the moment, to give the control to save specified fields.
https://github.com/ming-tung/django-fsm-freeze/blob/main/django_fsm_freeze/models.py#L47

Ah got it! Sorry will close the issue then.

I will add something in the readme to clarify this - thanks for raising it.

Hey @loukou007, just FYI, I am working on a https://github.com/ming-tung/django-fsm-freeze/pull/13, which is a bit related to your question here, feel free to comment there if you have some thoughts as well.

repo: ming-tung/django-fsm-freeze


text:
🛑 MineVN Website is down

In bacd539, MineVN Website (http://minevn.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MineVN Website is back up in dd08898.


text:
🛑 [MineVN] Grafana is down

In 5f38040, [MineVN] Grafana (https://grafana.minevn.net) was down:

HTTP code: 502
Response time: 3959 ms

Resolved: [MineVN] Grafana is back up in 4af899e after 18 minutes.


text:
JSON Schema for configuration file

Is your feature request related to a problem? Please describe.
When working through defining and editing configuration, I need to constantly reference the docs and don't get in-editor validation.
Describe the solution you'd like
With a JSON schema defined for the configuration, I could get author-and-edit-time feedback about the configuration with several different editors, and even in-document help-on-hover and auto-complete.
Describe alternatives you've considered
Write my own JSON schema for the configuration for local use.
Additional context
Add any other context or screenshots about the feature request here.

I definitely think a generated json schema and / or yaml schema sounds good. Including these in the generated static site would be ideal we well. I have a prototype of a new config structure that I still need to finalize. Including schemas with that update would probably be the way forward.

authors: michaeltlombardi miniscruff

repo: miniscruff/changie


text:
fixing flyway db error in DB

What does this pull request do?

deletes one of the dependant rows from the referral desired outcomes table

What is the intent behind these changes?

The flyway db migration in dev was failing due to dependant row not being deleted. So we need to delete the row from the table

@sldblog I believe this will fail in prod as well. The problem here is we tried the delete the desired outcomes which are duplicates using the query

I don't understand, sorry. Does production have any referrals made for these new desired outcomes?
Or are these not new, made specifically for FBD-NW?

One of them 'af1d6c9c-bcdd-4e9f-a778-5c1a1e90163c' is not new and that is causing the issue

I have double checked this id 'af1d6c9c-bcdd-4e9f-a778-5c1a1e90163c' and this is new Id and not an old Id. This Id was used in dev only for testing purpose which we can get rid for the main build to pass, this  PR can be discarded


text:
New MP environment maat-cd-api

Environment details
I would like a new environment for the application MAAT (Means Assessment Administration Tool) Court Data API please.
Application Name
maat-cd-api
Description of application
This is a Java based Spring Boot Application which will be hosted on AWS Environment. The application is being deployed on to the AWS ECS container service. This is a Facade application to the existing LAA legacy Applications MAAT/MLRA. laa-maat-court-data-api orchestrates the traffics between Court Data Adaptor/Common Platform and MAAT/MLRA.
GitHub team slug
laa-aws-infrastructure
Environments

[X] Development
[X] test
[X] Preproduction
[X] Production

Environment access level Development
developer
Environment access level Test
developer
Environment access level Preproduction
developer
Environment access level Production
developer
application
Means Assessment Administration Tool Court Data API
business-unit
LAA
infrastructure-support
laa_ops@digital.justice.gov.uk
owner
laa-crime-apps@digital.justice.gov.uk
Subnet sets

[X] General
[ ] Isolated

How do users connect to the application
Over the public internet
Additional features
Please check any additional features required

[ ] Additional VPC Endpoints
[ ] Extended DNS Zones
[ ] Other - please specify

Other information
No response
Definition of Done
Definition of Done

[ ] account created
[ ] customer informed

Just to confirm @vc13837 on the basis that an environment rename will result in that being destroyed you're going to review this request & amend it once you've discussed it with your colleagues in the LAA Ops team. Thanks.

On hold pending discussions on if a new environment is needed.

https://mojdt.slack.com/archives/C06A07N525C/p1706100842757569?thread_ts=1705935733.430049&cid=C06A07N525C

Hi folks, we have made the decision to just go ahead and have MAAT and MAAT CD API in the same accounts on MP with the same Terraform states, so that we can begin work on MAAT migration immediately. We will have further considerations on the impact of this once we are at the stage of cutting over MAAT CD API.

authors: davidkelliott mikereiddigital vc13837


text:
LAWS-3284-extend-dns-zone: locals private application zones and priva…

local private application zones an private dns zone extend added to the vpc.tf file

Issue with module leave it with me will take a look


text:
Additional NAT Gateway for DNS

User Story:
As a DevOps engineer I want to ensure that my services provide as high a level resiliency as possible so that I can reduce the chance of downtime.
Context:
we need to update our route table and add an additional NAT gateway as the current configuration sites in one availability zone which would mean our service would go down

Ticket to be updated by Jahir

we need to update our route table and add an additional NAT gateway as the current configuration sites in one availability zone which would mean our entire service would go down

DNS and DHCP VPC private subnet in AZa and AZb routing traffic via a single NAT(nat-0137946977ae3f9ad).This risk service resilience in the event of an AZ failure.
Outcome of this ticket would be to update the AZb route table(rtb-0ff36ff3a27ae8756 / staff-device-production-dhcp-dns-private-eu-west-2b
) with NAT in AZb (nat-0b5f7950219dc295d)

https://github.com/ministryofjustice/staff-device-dns-dhcp-infrastructure/pull/295

rolled in the change in DEV and Pre-Prod, we will have to do PROD in a controlled way.

Executing this action in a production environment during regular work hours carries inherent risks. This stems from the necessity to manually delete and re-add the route table, as Terraform currently lacks the capability to update an existing route without the initial manual deletion. Consequently, a brief period of downtime is unavoidable, and it would be prudent to formally initiate a change request for this purpose. The most suitable timing for this would be during off-hours, either late at night or in the evening. Till will place this ticket in BLOCKED status.

Production stae modification commands
 terraform state rm 'module.servers_vpc.aws_route.pdns-route-2a-pdns-1["rtb-08908a4069cec43fa"]'
 terraform state rm 'module.servers_vpc.aws_route.pdns-route-2a-pdns-1["rtb-0ff36ff3a27ae8756"]'
 terraform state rm 'module.servers_vpc.aws_route.pdns-route-2a-pdns-2["rtb-08908a4069cec43fa"]'
 terraform state rm 'module.servers_vpc.aws_route.pdns-route-2a-pdns-2["rtb-0ff36ff3a27ae8756"]'`

IMPORT_ARGUMENT='module.servers_vpc.aws_route.pdns-route-2a-pdns-1["25.25.25.25"] rtb-08908a4069cec43fa_25.25.25.25/32' make import
IMPORT_ARGUMENT='module.servers_vpc.aws_route.pdns-route-2a-pdns-1["25.26.27.28"] rtb-08908a4069cec43fa_25.26.27.28/32' make import

IMPORT_ARGUMENT='module.servers_vpc.aws_route.pdns-route-2b-pdns-1["25.25.25.25"] rtb-0ff36ff3a27ae8756_25.25.25.25/32' make import
IMPORT_ARGUMENT='module.servers_vpc.aws_route.pdns-route-2b-pdns-1["25.26.27.28"] rtb-0ff36ff3a27ae8756_25.26.27.28/32' make import

Terraform plan for production after successful imports:
Terraform will perform the following actions:
  # module.dns.aws_ecs_service.service will be updated in-place
  ~ resource "aws_ecs_service" "service" {
      ~ desired_count                      = 3 -> 5
        id                                 = "arn:aws:ecs:eu-west-2:037161842252:service/staff-device-production-dns-cluster/staff-device-production-dns-service"
        name                               = "staff-device-production-dns-service"
        tags                               = {
            "Name"             = "staff-device-production-dns"
            "Namespace"        = "staff-device"
            "Stage"            = "production"
            "application"      = "dhcp-dns"
            "business-unit"    = "HQ"
            "environment-name" = "production"
            "is-production"    = "true"
            "owner"            = "NVVS DevOps Team: lanwifi-devops@digital.justice.gov.uk"
            "source-code"      = "https://github.com/ministryofjustice/staff-device-dns-dhcp-infrastructure"
        }
        # (15 unchanged attributes hidden)

        # (4 unchanged blocks hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy

repo: ministryofjustice/nvvs-devops


text:
VEGA-2020 create donor and correspondent split name fields

VEGA-2020 #minor

Codecov Report
Patch coverage is 100.00% of modified lines.

Files Changed
Coverage

internal/sirius/create_draft.go
ø

internal/server/create_draft.go
100.00%

:loudspeaker: Thoughts on this report? Let us know!.

authors: GemTay codecov-commenter


text:
edited python script to seed test user into database

Purpose
Seeds a test user account into the database to save dev's having to create a new account each time they bring up the containers
Checklist

[ ] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage increased (+0.6%) to 76.55% when pulling 6438caa8dd456f8097039fb67e5a0c3d7cba6a88 on seed-test-user-account into a3e0705cb8314f70d6369b0ca61f351829303e46 on master.

Just tested login in the UAT env - it works! Horray!

authors: GemTay coveralls

repo: ministryofjustice/opg-use-an-lpa


text:
More ie styling fixes

various IE7, 8 and 9 styling problems have existed for quite a while. This is the first of several
PRs to fix them. including:

[X] IE7 login/out alignment
[X] IE7 and 8 footer alignment
[X] IE9 main content width and horizontal centering

Assuming everything is visually OK in the browser, this looks fine to me. (You know, for values of "fine" that include "make IE play nice".) 😄
👍

authors: axemonkey jsugarman

repo: ministryofjustice/peoplefinder


text:
🛑 Matrix is down

In 89c4de6, Matrix (https://matrix.mint.lgbt:8448/_matrix/federation/v1/version) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Matrix is back up in 02e5e28 after 24 minutes.

repo: mint-lgbt/status


text:
🛑 Ralsei is down

In 0d944dd, Ralsei (ralsei.nightcore.monster) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Ralsei is back up in 909bbc4.


text:
🛑 Matrix is down

In 7d4c980, Matrix (https://matrix.mint.lgbt:8448/_matrix/federation/v1/version) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Matrix is back up in 359a72b after 39 minutes.


text:
rebuild with vue-cli

之前的文档已经跑不起来了，用 vue-cli 重构建了一下，把 cooking 干掉了，文档只有两个小变化：

Quick Start 中关于 Mint-Starter 和 自己规划文件结构的全部干掉了，改为建议使用 vue-cli，对小白友好些
加了个参与文档编辑的入口，省的其他人不知道该在哪贡献文档

@QingWei-Li @Leopoldthecoder 大佬们过目一下。

盖神开心就好

authors: QingWei-Li galenyuan

repo: mint-ui/docs


text:
帝国cms 整合包

暂时里面只有一个赞的功能

@pearlwang1106  请问现有没有百度站内搜索这个组件  还有就是审核是没周二与周四是吗

站内搜索暂时还不支持，审核是每天都有，上线是周二和周四

@pearlwang1106 你好 我想咨询一下 我现在用的是自适应的网站 然后不打算从弄一个mip的站 就用这个自适应的改  但是现在我的站内有个百度站内搜索 但是没有这个组件 我应该如何处理这块 需要提交一下吗 因为自适应就是pc+web 很多代码虽然在手机端不会显示 但是源码里都还在 请问很多类似的情况我该如何处理

@pearlwang1106   额 我这个审核为什么没有过啊 然后也没有告诉我哪里有问题....

有问题的我有在代码和pr中做评论

@pearlwang1106 是说的那个作者的问题吗 之前有添加啊 然后我刚才又改了一次

repo: mipengine/mip-extensions


text:
Support for new ShowRSS?

Do you have plans to support the new ShowRSS (http://new.showrss.info/)?

I did, Catch 1.9.3 supports it :) Eventually new ShowRSS with replace the old one, so even the "Launch ShowRSS" menu item will go to new ShowRSS. But the feeds work today!

authors: aaronsher mipstian

repo: mipstian/catch


text:
Added method to remove unused materials / elements / isotopes

https://github.com/mipt-npm/gdml.kt/issues/41
Code should work but it is not tested with geometry where material is composed of other materials (only tested when material is composed of multiple elements).
There are a few assertions commented in the test because I couldn't get them to work but if implemented they should provide a decent check.
I also updated the babyIAXO.gdml geometry for a new one with excess materials, in order to use in the test.

Thanks a lot. I will review and merge soon.

authors: altavir lobis


text:
Improve parallel search performance

This PR improves the parallel search performance by

Selecting a proper number of threads
Shuffling the orders for search tree exploration for each thread for a better reuse of existing z3 invocations

Another modification included in this PR is solving memory leakage in search. I do not use smart pointers because the operators should be removed in a reversed order in the graph, which is not considered by the pointer reference manager. So currently I free the pointers in the destructor of graphs and disable the copy construction.

repo: mirage-project/mirage


text:
Add Classes in CLASSPATH Automatically

It is counter-intuitive to Java developers needing to pass the current CLASSPATH as in "mirahc -cp $CLASSPATH" or "mirah -cp $CLASSPATH" to compile or run an application with dependencies.
You may want to consider adding classes in the CLASSPATH automatically.

Do you mean that it's weird that if you have CLASSPATH defined in your shell, that you'd expect mirahc to add it to the compile classpath, and mirah to add it to the runtime classpath?
I'll change it to follow java / javac's behavior, so the following happens:
mirah foo.mirah, uses default classpath: .
CLASSPATH=<my-cp> mirah foo.mirah, uses <my-cp>
CLASSPATH=<my-cp> mirah -cp <flag-cp> foo.mirah, uses <flag-cp>

Exactly.

Mirah's current default classpath is the destination path, maybe the classpath state machine should be
if cp flag, use flag
if CLASSPATH use CLASSPATH
else use destination
then if you do
mirahc -d classes a.mirah
mirahc -d classes b.mirah

where b depends on a, b will compile, even though -cp wasn't specified. I'll go with that, and see how I feel.

I think we should follow the same rules as java/javac, but your proposal also hits the mark.

I think 67d8898b00d6e2fe78305429933ca55193d660f8 should do it.

Thanks!

repo: mirah/mirah


text:
trailing comma should be ignored in method calls

in ruby its fine
irb(main):003:0> puts(1,2,3,)
1
2
3
=> nil

in mirah it causes error
test.vrb failed to parse. org.mirah.mmeta.SyntaxError: expected block_arg before ')' (at line: 1, char: 11)
Since ruby have no inline comments one can comment out some args by splitting the call into multiple lines and commenting out whole line
But then its simple to forget the comma and the code won't compile

Sounds reasonable to me
On Sep 26, 2016 9:21 AM, "guai" notifications@github.com wrote:

in ruby its fine
irb(main):003:0> puts(1,2,3,)
1
2
3
=> nil
in mirah it causes error
test.vrb failed to parse. org.mirah.mmeta.SyntaxError: expected block_arg
before ')' (at line: 1, char: 11)
Since ruby have no inline comments one can comment out some args by
splitting the call into multiple lines and commenting out whole line
But then its simple to forget the comma and then the code won't compile
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mirah/mirah/issues/442, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAc0ggF_AG_jRdVA0W-_sQJTMkOmvRWks5qt-L9gaJpZM4KGp7E
.

authors: baroquebobcat guai


text:
Implement Java-array .new(length) method

For example

byte[].new(6)

should be valid Mirah code and should create a byte[] array with a length of 6.

Currently mirah uses brackets as array constructor:
bytes = byte[6] # 6 element byte array
Here is macro implemented for array initializer: https://github.com/uujava/mirah/commit/1556c562d7a176ef828c4942923c3cec2d8eb055
Actually, it should be implemented differently:
byte[6] should create single element array, and byte[].new(6) - 6 element empty array
But where were a lot of legacy code in test and other places so here we are.

In my view,
byte[].new(6)

should create a 1-dimensional array of bytes with 6 elements (and zero values).
byte[][].new(6,9)

should create a 1-dimensional array of byte-arrays with 6 elements, each containing a 1-dimensional array of bytes with 9 elements (and zero values).
[1,5,4,2].as(byte[])

should create a 1-dimensional array of bytes with 4 elements (and the appropriate values).
byte[][].new(6) do |i|
  [i,i*2,i*4].as(byte[])
end

should create a 1-dimensional array of byte-arrays with 4 elements, each containing a 1-dimensional array of bytes with 3 elements (and the values [0,0,0],[1,2,4],[2,4,8],[3,6,12]).
Array.new(6) do |i|
  [i,i*2,i*4].as(byte[])
end

should create a 1-dimensional java.util.Array of byte-arrays with 4 elements, each containing a 1-dimensional array of bytes with 3 elements (and the values [0,0,0],[1,2,4],[2,4,8],[3,6,12]).
Unfortunately, the current legacy Mirah dialect and @uujava’s Mirah dialect are conflicting regarding the meaning of byte[6] (a byte array of zeroes with length 6? or a byte array containing a 6 with length 1?). That this has been possible at all is a sign of that expression not being intuitively understood. Maybe it would be good to deprecate and phase out byte[variable] expressions altogether.

We work with arrays mostly in performance or memory critical code.
This  [1,5,4,2].as(byte[]) has bad metrics in both memory and performance area.

We work with arrays mostly in performance or memory critical code.
This [1,5,4,2].as(byte[]) has bad metrics in both memory and performance area.

That’s not true. Note that ArrayList#as is a macro which does the equivalent of
byte[].new(4) tap do |b|
  b[0] = 1.as!(byte)
  b[1] = 5.as!(byte)
  b[2] = 4.as!(byte)
  b[3] = 2.as!(byte)
end

(note that as!(byte) is a no-operation at the JVM level)
and not, as you might have believed,
begin
  a = ArrayList.new(4)
  a.add(Integer.new(1))
  a.add(Integer.new(5))
  a.add(Integer.new(4))
  a.add(Integer.new(2))
  byte[].new(a.length).tap do |b|
    b.length.times do |i|
      b[i] = a[i].as!(Integer).intValue.as!(byte)
    end
  end
end

Thus, we can be perfectly efficient here.

Oh, that's nice mirah macros stuff. Sorry for misunderstanding.

Fixed by #450.

authors: felixvf uujava


text:
Update 'carlito.js' to respect guidelines

Update carlito.js to respect JavaScript guidelines.

LGTM! 👍

authors: Palleas loboulet remiprev

repo: mirego/carlito


text:
[Question] StartedCommand CanExecute is not working

Hi @miroiu. I am trying to restrict a connector to having only one outgoing connection by checking if it is already connected and returning a false from CanExecute.
StartConnectionCommand = new RelayCommand<ConnectorViewModel>(source => PendingConnection.Source = source!, model => !model.IsConnected);
But this is not working. Am I doing it the correct way?

Hi @trrahul ! The recommended method is to create or cancel the pending connection in the PendingConnection.CompletedCommand command handler. Because even if you don't start the connection from the "disabled" connector, you can still drop pending connections onto it.
In support of the recommended approach, there is the EnablePreview dependency property that works with a PreviewTarget, which you can use to show the connection status in real time. (pay attention to the tooltip text below)

However, you have a good point! Users should not be able to start a pending connection if that's not allowed.

This is now fixed, and it can be seen in the Calculator example app. Please let me know if that's what you wanted to achieve.

Yes, working as expected. Thanks!

authors: miroiu trrahul

repo: miroiu/nodify


text:
use defaultChannelSlug in queries and mutations

I want to merge this change because...

Store the channel slug in the storefront
Update queries that require channel slug and pass the value to these queries to get channel’s data.
Update featured products - use products from channel with featured-products slug

Screenshots

Pull Request Checklist

[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[x] Changes are mentioned in the changelog.

Test Environment Config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: SALEOR-458 Use Channel SLUG in storefront

⚠ This PR uses temporary non-released SDK version that should not be merged into master - commit 1fb1a01

Waiting for stage server update, then tests should pass

Task linked: SALEOR-1645 Drop deprecated navigation field from shop query

authors: AlicjaSzu jwm0 krzysztofwolski patrys

repo: mirumee/saleor-storefront


text:
"Add product" dropdown sends requests to API on every key press

The "Add products" autocomplete input sends requests to API on every key press which is very slow and causes half of them to fail.
Steps to reproduce the problem

In the new dashboard go to "Orders" > "Add order" > "Add products".
Start typing anything to search for products.
In the dev tools console observe how requests are sent on every key press.

Proposed solution
Use debouncing or any other solution that would add some delay before request is sent.
Screenshots

@maarcingebala I want to take this... can I ???

Hey @lokesh1729, it is unassigned yet so you can give it a try. Please provide a PR and we will review it.

Hi @lokesh1729, any luck implementing debounce on this widget?

@dominik-zeglen nope, not finding time... will spend this week

authors: dominik-zeglen lokesh1729 maarcingebala

repo: mirumee/saleor


text:
🛑 La bonne alternance is down

In 75bd365, La bonne alternance (https://labonnealternance.apprentissage.beta.gouv.fr/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: La bonne alternance is back up in 95183eb.

repo: mission-apprentissage/upptime


text:
Add 'hits' to counter kwargs, fix 'remaining' kwarg for downward counts

This PR adds a new kwarg hits to the LogicBlock.Counters hit events, and changes the calculation of the remaining kwarg.
Currently, the remaining value is always calculated as the complete_value - self.value, which works great for counting up but not for counting down. This PR changes the logic such that downward counters use self.value - complete_value for the remaining amount.
The new hits value is the balance between the total value and the remaining. For upward counters it's self.value - start_value and for downward counters it's start_value - self.value.

Builds failed on Python 3.5 platforms with the expected Yield inside async function, but passed on all other platforms.

repo: missionpinball/mpf


text:
Fix fast board counting

Thanks to @lamoraldus for finding this!

Coverage increased (+0.001%) to 91.889% when pulling f9946347e64ff26d2df63038b445eed98ee17f86 on fix_fast_board_counting into 6ba59203fed6e4dc723689a1fb1935c3d816727d on dev.


text:
Backpack and procedure drawer interact poorly

Describe the bug
From the forum: Storing and retrieving procedures in the backpack and across projects can lead to a situation where the procedure database contains uninitialized procedure definition blocks, causing an internal error.
Affects

[ ] Designer
[x] Blocks editor
[ ] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Debugging
[ ] Other... (please describe)

Expected behavior
Opening and closing the backpack should leave the procedure database unmodified.
Steps to reproduce
I was able to reproduce but I'm not 100% sure all of these steps were needed:

Create a new project and create a statement procedure.
Add the procedure to the backpack.
Delete the procedure.
Create a new value procedure.
Add the procedure to the backpack.
Create a new project.
Open and close the backpack.
Create a new value procedure.
Right click and add the procedure to the backpack.
Try opening the procedure drawer again. This should result in an internal error.

@BeksOmega I believe this is just an error in App Inventor, and I think it will probably need to be resolved in our definition of procedure blocks. My current hypothesis is that the procedure definitions are likely not flyout aware and so they're registering themselves as valid procedures when rendered in the backpack, but this registration should only occur when they appear in the main workspace.

repo: mit-cml/appinventor-sources


text:
Set screen bakcground color to none when there is a backgorund image

Background images do not show up unless the background color is none.   People always get confused by this -- they think the image is missing, or something.  So let's just set the background color to NONE when there is a background image.  This seems like a simple fix, although it might require some fancy footwork in the designer to keep the background color selection consistent with whether or not there is a background image.

Just an FYI, Justus figured out how to fix this issue in another way by making sure a BackgroundImage overlays a BackgoundColor and not the other way around.

Closing this, given Justus's better approach.

authors: William-Byrne halatmit


text:
Fix screen orientation flip-flop when screen opens

Change-Id: I7cf33778ae2fc0f08ba1e178d2d3c80d1b1f4026

To test, create a project where Screen1 has landscape orientation. Possibly add more screens with other orientations. Compile the APK and install it on a device. Hold the device in portrait mode and launch the app. It will start in portrait and then rotate to landscape in production. With this PR applied, the app should start in the designated orientation and not rotate.


text:
Textbox Validation while creating a new AI Project

While a user is typing a project name, an error message will pop up and the textbox will turn red if an invalid character has been typed. It will stay there until the problem is fixed.

@KeithGalli
How do I test this?  I must be missing something, because if  go to create a new project and start typing the project name with a bad first character, I don't see any different behavior than happened previously with App Inventor.

@halatmit

The above is what you should be seeing when there is invalid text in the textbox. The perimeter of the textbox is now red and there is an error message while the text is invalid

It looks like LabeledTextBox.java is not importing Validator.

Yea that's what the problem is. The import was working just fine when I originally made the change, but when I switched branches and switched back it stopped working. Is there anything special that I have to do when I add a new file to app inventor? Validator.java is a new class that I created for this change.

A new file is called an "untracked file" in git parlance.
You need to explicitly add it with the git add command.
== Hal Abelson
hal@mit.edu
Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Sun, May 31, 2015 at 9:50 PM, KeithGalli notifications@github.com
wrote:

Yea that's what the problem is. The import was working just fine when I
originally made the change, but when I switched branches and switched back
it stopped working. Is there anything special that I have to do when I add
a new file to app inventor? Validator.java is a new class that I created
for this change.
—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/402#issuecomment-107280193
.

Hmm, I already explicitly added it with the "git add" command. When I make additional changes to the validator.java file it shows up in my tracked files on git.

@KeithGalli When you indicate and error, you should also disable the "OK" button.

@jisqyv That makes sense, I'll try to add that functionality today.

@jisqyv I disabled the "OK" button in my most recent commit.

@jisqyv @afmckinney
I started reviewing this and it doesn't seem to be working.  As far as I can see, the validator is not being called.

@halatmit I'll poke at it.

@halatmit This code is working fine for me. I'm going to do a more complete review today with the intention if getting this code in tonight's release.

@KeithGalli @halatmit LGTM

@KeithGalli @halatmit MERGED

authors: KeithGalli halatmit jisqyv


text:
About ModuleNotFoundError: No module named 'config.shapenet'

Hi, it is wrong when i train the command:
python train.py config/shapenet/pvcnn/c1.py
And the error is like this:
==> loading configs from ['config/shapenet/pvcnn/c1.py']
Traceback (most recent call last):
  File "train.py", line 266, in <module>
    main()
  File "train.py", line 79, in main
    configs = prepare()
  File "train.py", line 24, in prepare
    configs.update_from_modules(*args.configs)
  File "/workspace/pvcnn-master/utils/config.py", line 113, in update_from_modules
    importlib.import_module(module)
  File "/opt/conda/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 961, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 961, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'config.shapenet

It should be python train.py configs/shapenet/pvcnn/c1.py (with configs instead of config).

authors: eine-kenta zhijian-liu

repo: mit-han-lab/pvcnn


text:
Update README.md

Looks like the UI has changed some. This is based on my experience setting this up a few minutes ago.

:+1:  LGTM @ericmand, thanks for the contribution!

authors: ericmand erjohnso

repo: mitchellh/vagrant-google


text:
Fix shielded api

There was a small issue introduced in #248 that I didn't notice, which resulted in following errors running any non-shielded VM image:
stderr: /Users/temikus/.rbenv/versions/2.6.6/lib/ruby/gems/2.6.0/gems/google-api-client-0.50.0/lib/google/apis/core/http_command.rb:228:in `check_status': invalid: Invalid value for field 'resource.shieldedInstanceConfig': '{  "enableSecureBoot": false,  "enableVtpm": false,  "enableIntegrityMonitoring": false}'. Shielded VM Config can only be set when using a UEFI-compatible disk. (Google::Apis::ClientError)

Seems like the API just looks at the presence of the container and doesn't actually look into any of its' fields ¯_(ツ)_/¯

I'm sorry for this issue, I didn't notice that the API still reject shieldedInstanceConfig in non-shielded VM, even if all the value are set to false

@lcy0321 Nothing to apologise about. That’s IMO incorrect API behaviour.


text:
The program requirement operation titles should not be used in logic

As a program administrator, when adding requirements to a program, defining the title of the operators in the requirements tree should be hard coded and not left up to the administrator.  This will ensure that requirements operator titles such as "Elective courses", "Required courses", and "One of" are reliably referenced throughout the code of our application.
Acceptance Criteria:

[ ] Remove dependency on program requirement and elective operator titles for logic in code.
[ ] Documentation is updated to reflect this.

I don't understand. "Elective" and "required" aren't operators, they are titles. The operators are "All of" and "Minimum # of" combined with a value.
@rhysyngsun @jkachel do we have documentation somewhere on configuring program requirements?

We enforce the root level ones in code; if you add an elective or a requirement through the Program model helper methods, it'll create either a "Elective Courses" or a "Required Courses" for you if they don't exist already. The ProgramAdminForm also enforces these if you open a program in Django Admin that doesn't have a requirements tree. There's nothing keeping someone from changing the titles later, though.

Those fields were supposed to be opaque values and up to the course team to decide on what's best, are we relying on them to be specific values?

@rhysyngsun https://github.com/mitodl/mitxonline/blob/56a164b3a25128cdb51aeac9ff9aba4a9d9204f3/frontend/public/src/containers/pages/records/LearnerRecordsPage.js#L93, frontend/public/src/containers/pages/records/LearnerRecordsPage.js, https://github.com/mitodl/mitxonline/blob/bc5a9a86364f1b30e2564360282e2a10dc7c4984/courses/models.py#L318, https://github.com/mitodl/mitxonline/blob/bc5a9a86364f1b30e2564360282e2a10dc7c4984/courses/models.py#L311

Well. I guess we broke that today. I named the titles to be descriptive of the courses, as intended. You can see it now in production.
What’s the impact on the front end?

@pdpinch, I would probably change them back to the defaults.

It looks like this affects the program record. Anything else?

Having a value for the operation title which is not the default will likely impact more than just the program record.  It appears that our application should only use this value for configuring the text shown in the badge for each course in the program record.

Ok, I reverted the original DEDP Program, and I can see that it affected the display of the program record.
https://mitxonline.mit.edu/admin/courses/program/1/change/
I left the new DEDP Program as is, since it's new to everyone anyway.
https://mitxonline.mit.edu/admin/courses/program/2/change/
We need to find a way to balance between UI text that we want to be customizable and functional settings. And we need to make the distinction clear in the code so we don't drift like this in the future.

authors: collinpreston jkachel pdpinch rhysyngsun

repo: mitodl/mitxonline


text:
upgrade button in open edX production

Steps to Reproduce

Enroll as auditor in https://courses.mitxonline.mit.edu/learn/course/course-v1:MITxT+JPAL102x+3T2022/home
Click the upgrade button on the course home page

Expected Behavior

If possible, we'd like to hide the upgrade button, until we turn on the ecommerce system early next month
When the learner clicks the button, it should take them to the mitxonline checkout page, with the correct course in their basket

Actual Behavior

The upgrade button displays, even though we're not ready to start taking payments through mitxonline
The button links to the open edX dashboard

While looking around this I didn't find any specific settings/feature flag to disable the upgrade of a button but there are other simpler ways that we can use as mentioned below:

If we don't have Verified course mode or the deadline of the Verified course mode is in the past the upgrade button will hide automatically from the frontend.

If possible, we'd like to hide the upgrade button, until we turn on the ecommerce system early next month

The upgrade button displays, even though we're not ready to start taking payments through mitxonline

Possible Solutions:

So if we are not ready to take payments for a course we should not have Verified course mode set up for that in edX or we should set the upgrade_deadline in the past if that is feasible.
If we can't delete the Verified course mode, it might be a case where we want two services e.g. Micromasters & MITxOnline to use the same edX instance where Micromasters needs Verified mode and MITxOnline doesn't. In this case even if there was a setting in edX that might not have been useful. In this case we might have to hide the upgrade button intentionally from our MITxOnline fork of https://github.com/mitodl/frontend-app-learning.

When the learner clicks the button, it should take them to the mitxonline checkout page, with the correct course in their basket

It will start working when we setup our external-checkout-plugin in production as mentioned in https://github.com/mitodl/hq/discussions/227#discussioncomment-3517802

Unfortunately, the verified course mode is a requirement because the course team needs to do some authoring with cohorts -- they want to start working on the exam which should only be seen by learners in the verified mode.
I'll try setting the date into the past.

Ok, setting the start date in the past hides the button, but I've been reminded that we also want the upgrade date to display in the course Important Dates.
@mitodl/devops Can we set MARKETING_SITE_CHECKOUT_URL to https://micromasters.mit.edu/dashboard/ only for the mitoxonline production open edX?
https://github.com/mitodl/ol-infrastructure/blob/8863a97df92b0ee777e975be9e8343f50620567b/src/bilder/images/edxapp/templates/edxapp/mitxonline/common_values.yml.tmpl#L340

@arslanashraf7 devops tried setting MARKETING_SITE_CHECKOUT_URL to https://micromasters.mit.edu/dashboard/ in production, but it didn't seem to work. They're not sure if it's set in the right place to be included in the Learning MFE build. Can you take a look?

@arslanashraf7 devops tried setting MARKETING_SITE_CHECKOUT_URL to https://micromasters.mit.edu/dashboard/ in production, but it didn't seem to work. They're not sure if it's set in the right place to be included in the Learning MFE build. Can you take a look?

Taking a look, Will keep you posted

MARKETING_SITE_CHECKOUT_URL key is not part of edX setting it was added to support our own plugin.
So i was looking at it and below are the possible solutions:
If external checkout plugin is configured on production:

We need to check the ecommerce configuration(admin/commerce/commerceconfiguration) - I suppose the value for Basket checkout page is set to dashboard in which case, to redirect this to external checkout plugin we need to set /checkout-external/ as value for Basket checkout page. After setting this, it should redirect the user to MARKETING_SITE_CHECKOUT_URL with a query param course_id e.g. https://micromasters.mit.edu/dashboard/?course_id=course-v1%3AMIT%2BMIT_RR_1%2BR1.

If external checkout plugin is not configured on production:

We need to set value for https://github.com/mitodl/ol-infrastructure/blob/d3e4cd09eeac92e704691a67517f0fd3894121ab/src/bilder/images/edxapp/templates/edxapp/mitxonline/common_values.yml.tmpl#L529 instead of MARKETING_SITE_CHECKOUT_URL. and the value would be https://micromasters.mit.edu. the dashboard would be appended from the ecommerce configuration value set through Django Admin which is already dashboard i think.

FYI, @pdpinch @blarghmatey

Follow up on my above comment:
@blarghmatey Just wanted to confirm something, This line means that the ol-openedx-external-checkout plugin is installed in production too, Could you please confirm?
If the answer to the above is YES, We will need to do the following (I don't have production access to do the below):

Visit admin/commerce/commerceconfiguration in production and add a configuration with value of Basket checkout page=/checkout-external/.
Visit the verified course mode for the course and make sure that it has a unique SKU value set

With the above changes, @blarghmatey 's This PR should start working and it should start taking the user to Micromasters with links that would look like https://micromasters.mit.edu/dashboard/?course_id=<course_id>.
FYI, @pdpinch

Thanks for those details @arslanashraf7, you are correct that the ol-openedx-external-checkout plugin is installed with the line that you linked.
@pdpinch let me know if you want me to do any of that admin setup, otherwise I'll assume that you're going to take it from here.

We fixed this today. The ol-infrastructure code will need to be updated again when we switch from micromasters to mitxonline for payment.

authors: arslanashraf7 blarghmatey pdpinch


text:
Add metadata to mediaconvert job for filtering, based on queue name

Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

[x] Settings

[x] New settings are documented and present in app.json
[x] New settings have reasonable development defaults, if applicable
[x] Opened issue for DevOps regarding necessary configuration changes to deployed environments

What are the relevant tickets?
Part of #995
What's this PR do?
Adds a new setting VIDEO_TRANSCODE_QUEUE, to match the queues specified in ol-infrastructure, and updates the MediaConvert job definition to include it.

Related: https://github.com/mitodl/salt-ops/pull/1501

Adds a UserMetadata.filter key to the MediaConvert job definition, which will be used for SNS event rule filtering on AWS (this will require a change to the ol-infrastructure code).
How should this be manually tested?

Copy RC AWS_* settings to your .env file, and set VIDEO_TRANSCODE_QUEUE=ocw-studio-mediaconvert-queue-qa
Create a Video object in a shell and then start a mediaconvert job for it.  There should be no errors.from videos.api import *
video = Video.objects.create(website=<your_website>, source_key="gdrive_uploads/11-382-water-diplomacy-spring-2021/1RxRDvHiKFR-QZ99GgZam2PiXRFCLJTLM/jellie3local1.mp4")
create_media_convert_job(video)

If you change VIDEO_TRANSCODE_QUEUE=fake_queue, restart the shell, and run create_media_convert_job(video) again, you should get an error about an invalid queue.

Codecov Report

Merging #1018 (b249fa2) into master (a5a926f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1018   +/-   ##
=======================================
  Coverage   91.02%   91.03%
=======================================
  Files         194      194
  Lines        7413     7415    +2
  Branches     1297     1297
=======================================
+ Hits         6748     6750    +2
  Misses        583      583
  Partials       82       82

Impacted Files
Coverage Δ

main/settings.py
94.25% <100.00%> (+0.02%)
:arrow_up:

videos/api.py
95.65% <100.00%> (+0.09%)
: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 a5a926f...b249fa2. Read the comment docs.

authors: codecov-commenter mbertrand

repo: mitodl/ocw-studio


text:
How do I expose deforum through this API?

Hi this API is amazing!
I'm looking to build a docker image for deforum specifically -- and I wanted to know how I can use this API to make deforum requests, assuming deforum has been installed. As there's a deforum plugin I assume it's possible but was looking for some sample code if it's possible…?

Deforum extension doesn't have API support AFAIK.
Here's how I automated deforum.

Make deforum text(json) file using python and wiggle library. (copy generated txt file and customize instead making it from scratch)
Use playwright to load text file and generate.

Deforum extension doesn't have API support AFAIK.
Here's how I automated deforum.

Make deforum text(json) file using python and wiggle library. (copy generated txt file and customize instead making it from scratch)
Use playwright to load text file and generate.