RTP Multicast Stream spit out errors

Hi again,
I'm currently using tsp to pipe an RTP multicast stream into ffmpeg (apparently using ffmpeg directly to take rtp stream produces more error?), however it constantly spits out mp2 Header missing and invalid data found at random intervals. Any ideas? 🍻
tsp command:
tsp -I ip -l <IP_ADDRESS> <IP_ADDRESS>:1234 -P fork "ffmpeg -i pipe: -vcodec copy -acodec aac -f flv rtmp://xxxxxxx"

ffmpeg stderr:
[mpegts @ 000002c0024d87c0] PES packet size mismatch0:47.08 bitrate=4821.4kbits/s speed=1.15x
[mp2 @ 000002c003f8a980] Header missing4kB time=00:00:47.54 bitrate=4810.9kbits/s speed=1.15x
Error while decoding stream #0:1: Invalid data found when processing input

ffmpeg summary:
Input #0, mpegts, from 'pipe:':
  Duration: N/A, start: 54733.380700, bitrate: N/A
  Program 101
    Stream #0:0[0x3f2]: Video: h264 (High) ([27][0][0][0] / 0x001B), yuv420p(tv, bt709, top first), 1920x1080 [SAR 1:1 DAR 16:9], 25 fps, 50 tbr, 90k tbn, 50 tbc
    Stream #0:1[0x3f3]: Audio: mp2 ([4][0][0][0] / 0x0004), 48000 Hz, stereo, s16p, 192 kb/s
Stream mapping:
  Stream #0:0 -> #0:0 (copy)
  Stream #0:1 -> #0:1 (mp2 (native) -> aac (native))
Output #0, flv, to 'rtmp://xxxx/stream?rtmpauth=xxxx:xxxx/metrotv_raw':
  Metadata:
    encoder         : Lavf58.33.100
    Stream #0:0: Video: h264 (High) ([7][0][0][0] / 0x0007), yuv420p(tv, bt709, top first), 1920x1080 [SAR 1:1 DAR 16:9], q=2-31, 25 fps, 50 tbr, 1k tbn, 90k tbc
    Stream #0:1: Audio: aac (LC) ([10][0][0][0] / 0x000A), 48000 Hz, stereo, fltp, 128 kb/s
    Metadata:
      encoder         : Lavc58.59.101 aac

If ffmpeg complains about invalid data, the received data must be invalid...
It does not mean that the data which is sent is invalid. But, depending on the transmission profile (bitrate, bursts), some packets may be lost. Since tsp has a large input buffer, there may be less packet losses than with ffmpeg directly.
I would suggest to increase the tsp buffer size and add a continuity plugin to detect continuity errors (a symptom for packet loss). Also, if you just need to feed ffmpeg with no additional processing of the stream, you should directly pipe the stream into ffmpeg.
The result is something like this (64 MB buffer):
tsp --buffer-size-mb 64 -I ip ... -P continuity | ffmpeg ....


Task - Matrix column coder - OTU filter

Make an option to display only unscored rows

Same for the Row Coder


Remove "ConsentiumThingsDalton"

Companion to https://github.com/arduino/library-registry/pull/4794

Done:
2024/07/29 13:00:51 Loaded 7189 libraries from DB
Removing ConsentiumThingsDalton
Success!


Could not find Charts.framework in the "Embedded Binaries"

I've clone from the git , drag the Charts.xcodeproj to my project explorer , but I Could not find Charts.framework in the "Embedded Binaries". My gesture is wrong?

Can you check ChartsDemo and what's the differences between yourrs.
BTW, you you target should >= 8.0
I checked they are all there for embedded binaries


Generated types should resolve to the same type when they are navigation properties

This is the start of a discussion that can unlock unlimited potential to build extensibility on top of some of our SDKs generated using the OData library.
When looking at building extensibility on top of our SDK, I am expecting to be able to add extension methods on the RequestBuilder to allow it to do what I want. Now, from what I understand, when an SDK is generated via CSDL to OpenAPI, we are losing this information and Kiota, for instance, is generating multiple types with the same structure.
In a Graph world, think about the following :
.Me should be the same as .Users["id"], which is not. So if I want to add an extension method to .Me, I need to add it to multiple RequestBuilder, making devs life harder and they need to understand fully the API.
I would love to be able to do the following :
client.Me.Colleagues.Get() and this should also work with client.Users["id"].Colleagues.Get()
Adding @vbiret to this conversation.

To add some context to the discussion, when generating from OData we know that /me and /users/{id} are the same exact thing (set of endpoints under those paths is the same) because of the way OData conventions work.
We kind of lose that information when working with OpenAPI after the conversion process goes through the metadata as it creates separate path items, and there's no way to tell they are the same.
If the conversion process could add some kind of extension we could read in to know "oh yeah, that path and under are the same as this other one", Kiota could greatly reduce code duplication.
@darrelmiller to provider his insights on that.

/me and /users/{id} are not the same thing, even in OData.  You can apply capability annotations to one that do not exist on the other.  Also, the only case that is the same is where the {id} is equal to the signed in user.  The path '/users/{id}' requires different scopes than '/me'.
I understand the appeal of trying to re-use the entity model that exists conceptually in CSDL, but the reality of the actual API implementation rarely actually means that the two paths have the same characteristics.

Fair enough. If models are the same, I think we can work with this, considering we wouldn't use extension methods to really fully extend, but using the Task construct to accomplish the same. We would decouple ourselves from the `RequestBuilder;  and would not run into this issue.

yeah models are deduped except for collection responses but this is tracked with #125.
Closing.


[9.x] uniqueIgnoreTrashed index

It's currently difficult to add a unique index to a table with SoftDeletes, as the unique index will prevent duplicate rows of a single column, even if records have been deleted.
Creating a composite unique index doesn't work as most databases ignore null columns for unique comparison. Further reading at https://www.dbi-services.com/blog/nulls-in-composite-keys/
This PR adds a shortcut method to add this functionality, via stored generated columns and a unique index. The generated column is set to NULL if deleted_at is set (allowing multiple NULLs) and to TRUE if deleted_at is not set (only allowing a single TRUE)
$table->string('username')->uniqueIgnoreTrashed();

or
$table->string('username');
$table->uniqueIgnoreTrashed('username');

This approach is supported by all databases, however SQL Server will prevent multiple null values
Naming inspired from #38124

Please note that draft PR's aren't reviewed.

@driesvints yep, ok to leave here and I'll mark ready when I've finished the todo's (happy to close and re-open if better for you)

@driesvints I've got the core DB engines supported now - except there are variations between versions of MySQL / Maria for this PR. Can I ask what's the preferred option here (answer will not guarantee merge 🤓):

Find the lowest common solution for all engines (+ less complexity, - not as performant)
Check engine / version numbers and use best solution (+ performant, - more complexity)

Description
I think what is missing from the description of this PR is the definition of the unique index on a table with softDeletes defined.
AFAIK, $table->unique() does not automatically add the deleted_at field at the end of the unique index definition - please correct me if I have this wrong.
If this is the case, then we have two distinct use cases:

$table->unique('username')
$table->unique(['username', 'deleted_at'])

Then we have at least the following 2 use cases:

Insert into this table when there are one or more existing records with the same username either all soft-deleted or one non soft-deleted. In the example link provided this seems to work consistently. It is inserting into a child table that varies between RDBMS'.
What happens when we try to insert a record into a separate table with a foreign key pointing to username when username is or is not defined in the first table and when it is or is not soft-deleted. TBH this is where the whole thing gets a bit hairy - probably the foreign-key should be [table2.username, null]->[table1.username, table1.deleted_at] so you can only insert the child record if the primary record is not soft-deleted. Then you have to deal with what happens to this relationship when the child is inserted and the parent record is subsequently soft-deleted.

I think that the real answer is to use an autoincrement column in the parent table as the foreign key, and to get the value of that primary key for matching username and deleted_at==null.
And once this is understood, then we can test against the various RDBMS' that Laravel supports.
What I would hope is that Laravel would hide the implementation differences between RDBMS' so that the developer can write code without worrying about such things and the code will work the same regardless.
So what is the problem?

Should Laravel automatically add deleted_at to unique indexes / primary keys because that would be an equivalent uniqueness to real deletes?
Should Laravel allow child inserts for a parent that is soft deleted?
What special handling code is needed inside Laravel to create consistent functionality regardless of RDBMS?
What documentation is needed to warn users of potential pitfalls?

@taylorotwell it's currently tricky to have a unique index on a column for a model with soft deletes and you want to ignore the deleted columns from the unique index.
Taking an excerpt from this article
This can be a common scenario if you use soft-deletes on your users table, where you have a unique constraint on the email field. With soft deletes this means if a user deletes their account they'll be unable to recreate the account at a later date because the email still exists in the table. Unless we intend to write user-hostile software, we probably want the user to be able to re-register with the same email.
To start off let's cover some basic assumptions about our application. We have a base Laravel user table migration, with the soft deletes attribute added.
Schema::create('users', function (Blueprint $table) {
    $table->id();
    $table->string('email')->unique();
    $table->softDeletes();
});

We also have the base Laravel User model, with the SoftDeletes attribute added.
class User extends Authenticatable
{
    use Notifiable, SoftDeletes;
    ...

With this setup we can create uses and delete users, but creating a new user with the same email results in the "Integrity constraint violation" error.

Create User

User::create([
    'email' =><EMAIL_ADDRESS>]);

Users Table:

id
email
deleted_at

1
<EMAIL_ADDRESS>

Delete User

User::find(1)->delete();

Users Table:

id
email
deleted_at

1
<EMAIL_ADDRESS>2020-07-29 16:40:56

Recreating user produces an error

User::create([
    'email' =><EMAIL_ADDRESS>]);

Warning: Illuminate/Database/QueryException with message 'SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry<EMAIL_ADDRESS>for key 'users.users_email_unique' (SQL: insert into users (email) values (sam@example.com))'

The most obvious solution to try is to make an index with $table->unique(['email', 'deleted_at']) however this doesn't people would expect due to this and would allow a table to look like this.

id
email
deleted_at

1
<EMAIL_ADDRESS>

2
<EMAIL_ADDRESS>

This PR creates a unique index that works as expected.

@morrislaptop Craig

We need tests on the existing code that shows that this problem can actually occur as you just described. (I don't disbelieve you, but the purpose of tests is to show that there is a problem and then to show that your solution actually fixes it.) So what we probably need are 2 tests:

A table without soft deletes and a unique index. Add a record, attempt to add the same record again and get an exception, delete the record and add the record again successfully, then attempt to add it a second time and get an exception.
The same but with soft deletes.

For softDelete to work as advertised i.e. it is equivalent to a delete unless you explicitly request that soft-deleted records are included, IMO your code needs to be added to unique rather than creating a whole new method. However, @taylorotwell should probably comment on this as we should try to avoid backwards incompatibilities.
If we are going to have a separate method, how about calling the new method uniqueSoftDelete so it is clear that this should be used with soft-delete tables.

We need tests on the existing code that shows that this problem can actually occur as you just described. (I don't disbelieve you, but the purpose of tests is to show that there is a problem and then to show that your solution actually fixes it.) So what we probably need are 2 tests:
A table without soft deletes and a unique index. Add a record, attempt to add the same record again and get an exception, delete the record and add the record again successfully, then attempt to add it a second time and get an exception.
The same but with soft deletes.

Cool happy to show this, this is just showing how the database index works? Would you be happy with SQL Fiddle links?

For softDelete to work as advertised i.e. it is equivalent to a delete unless you explicitly request that soft-deleted records are included, IMO your code needs to be added to unique rather than creating a whole new method. However, @taylorotwell should probably comment on this as we should try to avoid backwards incompatibilities.

I initially explored $table->string('email')->unique()->ignoreTrashed(), but I couldn't quite crack the implementation. Happy to take direction here.

If we are going to have a separate method, how about calling the new method uniqueSoftDelete so it is clear that this should be used with soft-delete tables.

Naming taken from https://github.com/laravel/framework/pull/38124

Would you be happy with SQL Fiddle links?

No - these should be phpunit tests.

Naming taken from #38124

Well, perhaps with hindsight ignoreTrashed() should have been called ignoreSoftDeleted().
IMO consistency of conceptual naming is important - and this is a Schema method and Schema uses softDelete.
But as previously stated, IMO this should simply be part of the existing unique method.

Thanks for your working on this with me @Sophist-UK !

No - these should be phpunit tests.

I've added those tests.

Well, perhaps with hindsight ignoreTrashed() should have been called ignoreSoftDeleted().
IMO consistency of conceptual naming is important - and this is a Schema method and Schema uses softDelete.

I'd prefer to keep it ignoreTrashed() so it's consistent with the rest of the framework (there is $model->trashed() as well). Leaving this to @taylorotwell

But as previously stated, IMO this should simply be part of the existing unique method.

I wouldn't assume that soft deleted records in a unique index should be ignored by default - some applications may disallow "used" values even if those records have been deleted. Better to be explicit on the behaviour with a new method. Leaving this to @taylorotwell

@morrislaptop faced this issue recently while performing validation 'email' => ['email', Rule::unique('users', 'email')->withoutTrashed()]. This validation passed through but throw database error due to the unique index. Well done.

I think the idea of softDelete is that when you soft delete records, all further changes should behave exactly as you would expect if the record had been hard deleted - except if you explicitly ask that softDeleted records are included.
This is why IMO deleted_at (or your calculated column) should be added to unique indexes automatically for tables with softDelete defined.
P.S. On reflection, I think that your calculated column needs to be changed too, because you might have 3 records:
|Value|deleted_at|Calculated_field|
|Name|null|true|
|Name|2022-07-04|false|
|Name|2022-07-03|false|
Would your calculated field allow there to be two deleted versions in the unique index?

I think the idea of softDelete is that when you soft delete records, all further changes should behave exactly as you would expect if the record had been hard deleted - except if you explicitly ask that softDeleted records are included.
This is why IMO deleted_at (or your calculated column) should be added to unique indexes automatically for tables with softDelete defined.

This isn't the current behaviour of the framework - see Unique and Exists validation rules.

P.S. On reflection, I think that your calculated column needs to be changed too, because you might have 3 records:
|Value |deleted_at|calc |
-------------------------
|Name  |null      |true |
|Name  |2022-07-04|false|
|Name  |2022-07-03|false|

Would your calculated field allow there to be two deleted versions in the unique index?

Yes, there is no reason to prevent this from happening. The application can enforce stricter rules if needed.

This isn't the current behaviour of the framework - see Unique and Exists validation rules.

But IMO this is how it should behave. And now that you mention it, the unique and exists validations should also operate on softDeleted records as if they had been hard deleted i.e. deleted_at should be included automatically in detect or exists validations.

P.S. On reflection, I think that your calculated column needs to be changed too, because you might have 3 records:
|Value |deleted_at|calc |
-------------------------
|Name  |null      |true |
|Name  |2022-07-04|false|
|Name  |2022-07-03|false|

Would your calculated field allow there to be two deleted versions in the unique index?

Yes, there is no reason to prevent this from happening. The application can enforce stricter rules if needed.

So your calculated field would produce false whenever deleted_at was not null, and you would need to include both deleted_at and your calculated field in the index to check for uniqueness?

But IMO this is how it should behave. And now that you mention it, the unique and exists validations should also operate on softDeleted records as if they had been hard deleted i.e. deleted_at should be included automatically in detect or exists validations.

This is a little outside the scope of this PR - perhaps discussions would be a good place for this.

So your calculated field would produce false whenever deleted_at was not null, and you would need to include both deleted_at and your calculated field in the index to check for uniqueness?

the calculated field produces NULL whenever deleted_at was not null and since null is not tested in a unique index, it works as expected. I've expanded the test to cover this, thanks for flagging.

This is a little outside the scope of this PR - perhaps discussions would be a good place for this.

Actually, this is very much in scope for this PR. This PR is attempting to fix issues where softDeleted records do not operate as intended because a softDeleted record should not impact any other operations. The whole point of Eloquent / Schema is that it hides these sorts of complexities (and differences in how different RDBMs behave) from the user and gives a consistent developer experience.

the calculated field produces NULL whenever deleted_at was not null and since null is not tested in a unique index, it works as expected. I've expanded the test to cover this, thanks for flagging.

In other words, you are relying on an eccentricity of a database to fix and issue caused by the same eccentricity. IMO this is NOT a recipe for a reliable long-term solution which will work identically on every supported RDBM. An alternative fix might be to create a calculated field which copies the deleted_at timestamp but if it is null then we set it to the earliest allowed timestamp - and this would result in a non-null field that will work regardless of the idiosyncrasies of any specific RDBM.

@Sophist-UK this PR is not fixing any issues, both soft deleted and the unique index work as intended. This PR adds a feature allowing unique values in a table which ignores soft deleted records whilst hiding the complexities to make this possible. This is not eccentricity, it is the SQL standard.
I think this PR has some had healthy discussion which has contributed to the design and shape. At this point I will leave it to the maintainers to merge or close as is. Thank you for feedback.

@morrislaptop Craig - Sorry, but it is my belief that softDelete is supposed to mimic a hard delete but be recoverable, and therefore my own view is that it is not currently working as intended.
The SQL standard AFAIK is NOT to ignore null fields when checking a unique index for uniqueness. The definition in the SQL standard is apparently a bit open to interpretation, and different RDBMs implement it differently. So it is indeed a quirk.
But even if it is not a quirk, my point about fixing it is still relevant - implementing a fix in a way that is completely unambiguous is IMO a good thing as it makes it less likely that some quirk will create problems with it at a later date.
(I should add that I do not see my comments as a criticism of your views or the work you have done - anyone who takes the time and effort to create a PR deserves recognition for their contribution. I just have a different perspective. But the purpose of these reviews is to try to ensure that changes are done right, so that they don't require later fixes and don't create backward compatibility issues, and on that basis I believe that these sorts of debates are exactly what should be happening in the interests of quality.)
I guess we need @taylorotwell to adjudicate on this one. I have made my case, and if Taylor disagrees with me and agrees with you, that is still a good result either way because the debate has helped ensure that the right decision is made.

Thanks for that @Sophist-UK, a rising tide lifts all ⛵ 🛥

No plans to modify or add any behavior around this at the moment.

@morrislaptop Craig
As per my comment in #41007, and as stated above, I believe that this behaviour is actually a bug.
I am wondering whether you might get more traction creating a PR for [10.x] along the lines of #41007?
The idea here is not to do an extra field, but instead to use a non-null value for deleted_at when the record is not soft-deleted. To avoid backwards compatibility if existing code checks for deleted_at == null, we might want the Model class to continue to deliver the value as null even though it is stored in the database differently. The Schema code to make the field nullable() would also need to be replaced with a `default('1970-01-01 00:00:01'). Field resets and record restores would need to be handled.
A other considerations:

Migration from v9 to v10 would require some SQL statements to be run:

UPDATE table_name SET `deleted_at` = 1970-01-01 00:00:01 WHERE `deleted_at` IS NULL;
ALTER TABLE table_name MODIFY `deleted_at` DATA_TYPE NOT NULL;

Obviously it gets more complicated if the `deleted_at` field is part of a primary or foreign key.

It might be sensible to include code to detect if the deleted_at field is NULL and raise an exception or warn the user or run the above statements automatically (yes a performance hit - but it would be a one off and it would make the thing self-repairing).

If this is not considered a bug, then it would be likely that such a PR would be rejected, and then the alternative is to create a small "Fix softDelete" package that would modify the behaviour as above.

Hi @Sophist-UK , I'm just doing something application / database specific now!
The example from https://github.com/tpetry/laravel-postgresql-enhanced#partial-indexes works perfectly for this use case. Hope this helps other people arriving at this thread.

Timely news for anyone interested https://blog.rustprooflabs.com/2022/07/postgres-15-unique-improvement-with-null

@morrislaptop Craig - IMO the spec was wrong or it was wrongly interpreted. null should never have been "considered non-equal". I think most of us would expect null == null to be true, and we would expect a Unique index to ensure that items are actually unique, and so treating nulls as non-equal is anti-intuitive and TBH I struggle to envisage a real-life use case.
However IMO the key point is that one of the key selling points of frameworks like Laravel and ORMs like Eloquent is that they should hide the inconsistencies from users. So whilst we should welcome this Postgres initiative, until it becomes a de-facto standard across all SQL RDBMs it is IMO Eloquent's job to hide these inconsistencies from users and make Laravel behave consistently across DBs.


Signing in with another account?

Once I have signed in with one account, triggering the flow again (even after I call forgetTokens), will show me
<EMAIL_ADDRESS>Are you trying to sign in to App [cancel] [continue].
There are no ways for me to sign in with another account.
Google's corresponding screen will give me an option to "Use another account", is there a way to accomplish this??

Yes you can. Azure OAuth is customizable as to that behavior. If you add the prompt=select_account query parameter you can force the selection screen. See https://docs.microsoft.com/azure/active-directory/develop/v2-oauth2-auth-code-flow#request-an-authorization-code


Kernel from a notebook is not used when starting up

Dont install the python extension
Open a notebook that is pointing to a custom kernel
Run the notebook
The first kernel on the machine is used instead of the one in the notebook.

Validated. I remember having this issue before. Both my julia and java notebooks correctly used the right kernels, when before they used to pull some random kernel that happened to be first in the list. Python extension not installed.


Update api-operation.md

Add missing comma, to avoid issues when copy-pasting

Hi, thanks for the PR, the API markdown files are auto-generated from JSDocs in the source code. Please can you update the following source and use npm run docs-build to regenerate.
https://github.com/lovell/sharp/blob/e40a881ab4a5e7b0e37ba17e31b3b186aef8cbf6/lib/operation.js#L208

Hi, thanks for the PR, the API markdown files are auto-generated from JSDocs in the source code. Please can you update the following source and use npm run docs-build to regenerate.
https://github.com/lovell/sharp/blob/e40a881ab4a5e7b0e37ba17e31b3b186aef8cbf6/lib/operation.js#L208

Sure, done here

Coverage remained the same at 100.0% when pulling 296e12689d5a7aca9a92dea79a4b4b98b8858961 on elrumordelaluz:patch-1 into dd56a9699eebcec0cbbcc9f6563e5dd6cf4066ad on lovell:eagle.

Thank you!


Correct format when I try to use both col and data?

{cols: Array(2), data: Array(2)}:
    cols:
        Array(2)
            0: {name: "Anne", key: 0}
            1: {name: "x", key: 1}

    data:
        Array(2)
            0: (7) ["mz", "mz/H", "66.4", "mz", "mz", "57.4", "California"]
            1: (3) ["89.9", "78.9", "89.8"]

I have this format (object of two arrays) in my react component.
I tried to use this state to download the data, doing
        /* convert state to workbook */
        var ws = XLSX.utils.aoa_to_sheet(this.state.formattedLabels.data);

        const wb = XLSX.utils.book_new();
        XLSX.utils.book_append_sheet(wb, ws, "SheetJS");

        /* generate XLSX file and send to client */
        XLSX.writeFile(wb, "labels.xlsx")

When I just passed in this.state.formattedLabels, it gave me an error and when I used this.state.formattedLabels.data, it works but there is no cols information in my downloaded excel file.
Is this the correct usage?

In the original demo, the cols was used to cache the array columns -- it is not part of the export.
To add the names to the export, create a row with the column headers:
var headers = this.state.formattedLabels.cols.map(function(col) { return col.name; });
/* headers will be ["Anne", "x"] */

and create a new array of arrays by concatenation:
var aoa = [ headers ]; // e.g. [ ["Anne", "x"] ]
aoa = aoa.concat(this.state.formattedLabels.data);
/* aoa will be
[
  ["Anne", "x"],
  ["mz", "mz/H", "66.4", "mz", "mz", "57.4", "California"],
  ["89.9", "78.9", "89.8"]
]
*/
var ws = XLSX.utils.aoa_to_sheet(data);


Custom rights statement on collection view page should not have quotation marks around it.

This is how it looks on the collection view page currently.

It should look the same as this screenshot from the item view page, which is the preferred display:

closed by #3320 (merged to feature branch), will get merged to main by #3295, part of #3093


[snappy] fix correct lib name in snappy.pc file

Describe the pull request

What does your PR fix?
Fixes https://github.com/microsoft/vcpkg/issues/28675
Note: The lib name generated by the port is snappy.lib, fix the wrong lib name in the pc file.
https://github.com/microsoft/vcpkg/blob/0f719b3fdfc36f8dd45229370a0df180fdbf68f6/ports/snappy/snappy.pc.in#L9
No feature needs to test.

Is there any other distribution which uses the snappy.pcfilename?
Should pc files which only exist in vcpkg use name prefixes?

Currently, only the current port uses the pc file in vcpkg.


PostCSS plugins are not applied correctly during vite build

Describe the bug
When running vite build with a postcss.config.cjs file in the project, I would expect postcss plugins to run before the final style.css file is created in dist/.
Example:

final dist/style.css

.btn {
  align-items: center;
  color: #ff0;
  display: flex;
  justify-content: center
}

.modal {
  align-items: center;
  color: red;
  display: flex;
  justify-content: center
}

.box {
  align-items: center;
  color: blue;
  display: flex;
  justify-content: center
}

after manually running postcss dist/styles.css -o dist/styles.css

.btn {
  color: #ff0
}

.btn,
.modal {
  align-items: center;
  display: flex;
  justify-content: center
}

.modal {
  color: red
}

.box {
  align-items: center;
  color: blue;
  display: flex;
  justify-content: center
}

Reproduction
https://github.com/floroz/vite-css-module-treeshakable
Steps to reproduce

pnpm install
pnpm build
inspect dist/style.css output
run pnpm postcss (will invoke postcss CLI on the same dist/style.css)
inspect dist/style.css output (optimisations are applied)

System Info
- node 18.12
- sass 1.61
- postcss 8.4
- vite 4.2.1

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I wanted to extend a bit my initial description as I noticed also an issue coming from the dart-sass compilation, where %placeholders are expanded into individual style declaration, rather than collapsing class selectors together.
So there seems to be two issues with this pipeline:

sass compilation of %placeholders within *.vue (or *.scss file consumed via *.vue) create duplicate style declaration;
postcss plugins are not fully applied until postcss-cli is run manually on final putput

This is because style in every vue files is transformed individually and so does cssnano. So not possible for merging styles from different files by cssnano. This seems to be a bug for me as we may perform postcss on renderChunk in build mode.

Thanks for the reply @fi3ework.

This is because style in every vue files is transformed individually and so does cssnano.

Is this by design? If I create .scss files, shouldn't they be combined before being transformed so that pre-processors can apply their optimizations? (i.e. placeholder collapsing?).

So not possible for merging styles from different files by cssnano. This seems to be a bug for me as we may perform postcss on renderChunk in build mode.

Not clear on this point, is this a genuine bug that requires fixing, and therefore the expected behaviour is to have postprocessor optimization run after all stylesheets have been merged together?
Do you know if there is any workaround to apply transformations after styles are combined together?

@floroz
Similar to https://github.com/vitejs/vite/issues/7504 https://github.com/vitejs/vite/issues/12400, and the workaround is quite similar to https://github.com/vitejs/vite/issues/12400#issuecomment-1468740126.

#7504

Thank you @fi3ework for the resources. Will check and verify those workarounds but it seems they still target the ability to apply postprocessing transformation, which would still invalidate the initial transformation done by sass?
Also, using a enforce: "post" isn't that different from manually executing postcss CLI after the vite build, as presented in my reproduction. (unless there are benefits I am not seeing?).
Could I tap into your knowledge (and slightly outside the boundary of this issue) to ask whether using rollup directly without vite could potentially resolve this issue (I suppose at that point the question is with the rollup-vue-plugin)?

Any update on this? Will this be fixed in v4?

I've experienced such a problem too. I've decided for the time being just to create vite plugin which runs postcss after bundle was generated. Put this into your vite config plugins section:
      {
        "name": "vite-bundle-postcss",
        async generateBundle (_, bundle) {
          for (const output of Object.values(bundle)) {
            if (output.type !== "asset" || !output.fileName.endsWith(".css")) {
              continue;
            }
            output.source = (await postcss([
              mediaQueryPacker(),
              postcssMergePlus(),
              cssnano()
            ]).process(output.source)).css;
          }
        }
      }

> Any update on this? Will this be fixed in v4?


Update container-instances-mounting-azure-files-volume.md

Fixed $ in assignment
Removed hard coded resource group and storage account names

@bureado,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@bureado : Thanks for your contribution to the Azure documentation! The author, @seanmck, has been notified to review your proposed change.

Hello again, @bureado. This PR currently cannot be merged due to conflict. Please resolve, and we can look at merging this. It does look, however, like the changes might already (again) have made it into the published doc, so perhaps this can be close. Let me know, thanks. -MM

The conflict has been resolved and merged into private repo in PR # 25232.
@bureado, thanks again for your contribution. Your commit is tied to the PR noted above and will be included in the file history. I'm now requesting close on this PR as the commit has been merged in the private repo.
#please-close
Cc: @tysonn


Keybase for mac is preventing sleep

Keybase GUI Version: 2.3.0-20180710171051+64615d6258

Bumping this issue as I'm still seeing it happening in Version 5.2.0-20200130011203+cf82db8320 (5.2.0-20200130011203+cf82db8320). I'm running macOS 10.14.6 and have reports of it happening to others running 10.13.6, but suspect it's across the board. I'm not sure if this is still related to #12822 - cc @maxtaco & @mmaxim - but I'm happy to open a new issue if needed.

FWIW - this may be triggered by /giphy ... I find when I'm searching for or just display a gif in a channel/chat activity monitor shows keybase preventing sleep. It stops showing preventing sleep if I do any of the following:

stop using /giphy to search for a giphy
switch to another channel/chat that isn't displaying a giphy
collapse the giphy in the channel/chat


Strongly type FieldList

Many methods on FieldList will silently fail if you give them bad types.
e.g.: $fields->addFieldsToTab('Root.Main', $field1, $field2, $field3) will only add $field1 to the tab.
Acceptance criteria

All methods in FieldList are strongly typed and have an explicit return type.
addFieldsToTab hard fails if the second parameter is not an array.
All of kitchen sink is shown to pass CI before any PRs are merged

FieldList is not in a vacuum - it's the subclass of ArrayList which is a subclass of ViewableData, uses a trait, and implements several interfaces.
The ACs should specify whether any of that is in scope, or if this card is only for methods defined directly in FieldList, or if it's mostly about methods defined directly in FieldList but the implementor can choose to strongly-type other methods if they feel it's relevant to the goal of the card (i.e. making it clearer what can/cannot be passed into FieldList).

PRs merged


Add support for ROUTE

I was trying to decode some ROUTE (RFC 9223) which is pretty similar to FLUTE, but hit the lack of support for Codepoint 8. I'm wondering if you have any thoughts about adding ROUTE support to this crate?

You’re correct, this library currently does not support ROUTE.
Since ROUTE is based on FLUTE, I’m open to adding some support for decoding ROUTE sessions to this FLUTE library.
However, I don’t have an immediate timeline for implementing this feature.
codepoint field has a different meaning in ROUTE compared to FLUTE:

In FLUTE, the codepoint represents the FEC Encoding ID.
In ROUTE, the codepoint indicates the type of payload (e.g., Media Segment).

Would you be able to share a pcap file with ROUTE so I can use it for testing ?

Thanks for the sharing,
I did have a look to the short capture,
I could not find a way to detect that ALC packets are ROUTE and not FLUTE, so from my understanding it will necessary to configure the decoder with a parameter informing it's a ROUTE session.
I'm afraid that today I don't know the ROUTE protocol well enough to add the support. So I prefer to keep the focus on FLUTE (part of DVB-NIP)

I'm testing a few fixes in my local build to check if I can successfully decode your stream, I'll let you know about the result

After a more detailed review, I have decided not to integrate ROUTE support into this FLUTE library. There are several reasons for this decision:

Incompatibility: ROUTE and FLUTE have slightly different definitions, making them not directly compatible.
Detection Issues: There is no way to detect if an ALC/LCT packet is ROUTE or FLUTE.
Decoding Differences: There are several differences in decoding noCode objects, such as Source Offset vs. SBN/ESI.
Media-Aware Packetization: ROUTE uses media-aware packetization, whereas I prefer to keep this library application-independent, as it is for FLUTE.

Thanks again for you sharing, that was helpful,
I believe that having an open-source ROUTE implementation in Rust would be great. However, it should be developed as a separate crate.


No sé como funciona >_<

¿Como puedo probarlo?

Hello @livrasand, thanks for reaching out!
Are you trying to use it in the browser, or for a nodejs app?

Sure! I guess you’ve already installed it using the npm install jw-epub-parser command.
The next step would be to import it into your project:

For client application, you would simply use:

import { loadEPUB } from 'jw-epub-parser'

For Node app, use:

import { loadEPUB } from 'jw-epub-parser/dist/node/index.js'

The loadEPUB is an asynchronous function can accept one of these three arguments: the path to the EPUB file (node only) or EPUB file's blob (browser only) or url to download the EPUB file (you need to get the correct url from jw.org).
Example:
const result = await loadEPUB({ url: 'https://download-a.akamaihd.net/files/media_periodical/84/mwb_E_202307.epub' })

You will get the meeting workbook data, as explained in this section if everything goes well.

You’re welcome! I will close this issue then, but feel free to reopen it or create a new one if you have questions.

Hola @livrasand! ¿Cómo estás? Would you be interested in participating in development a new version of the Organized app? It's the app that uses this epub parser tool to make it possible to screate meeting schedules in only a few seconds.
We are working on the redesign and then plan to add new useful features, therefore looking for enthusiastic brothers who would like to help us with the frontend development.
We will be very glad if you have interest to contribute ☺ Of course, only as your time allows you to.
So, if you're interested, let me know how I can reach you out. Thanks in advance!


android 6 Digital Asset Links

So, basically, you just need to add a config on your web site and that's it: no code update is needed? Have you tested this?

Yes, I used it for my project inscoop. Before I uploaded that configurational file to the server, I had browser as a second option to open any link, and after I did this the app opens straight. But the app should be signed. So simple cordova run android will not work, have to sing it also

Great, thanks!


Can this model used for summarization?

Can this model used for summarization? Is there anything I should modify if I want to use this model in summarization tasks?

I think so. Summarization is a special case of machine translation, where the source language and the target language are the same.

@da03  Have you ever tried a summarization task? Is there anything different from translation tasks?

It should be basically the same, but I think in this case the -replace_unk option would be helpful. (http://opennmt.net//Guide/#translating-unk-words). Besides, since the target and source language are the same, their vocabularies may be shared and their word vectors may be tied. However, from our experience on the CNN/Daily Mail dataset (we used that for summarization), that does not help, and the current code does not support doing that.
References:

A work applying seq2seq to summarization task: Nallapati et al. Abstractive Text Summarization using Sequence-to-sequence RNNs and Beyond. arxiv:1602.06023
A summarization dataset: https://github.com/harvardnlp/sent-summary

BTW, we have posted some results on this page, including a summarization result on Gigaword: http://opennmt.net//Models/

@da03:  I may be missing something but I am getting "access denied" for the summarization files listed on your link.

I encountered the same problem.

Sorry, the model might be still uploading. Will fix that soon.

Fixed now.

Perfect.  Thanks.  Was the same code used for the summarization or is there another repo?

It's the same code.

i have take few amount of data after translation it will be give unk sentences how to resolve problem in OpenNMT.


I need remove money

I need remove money from currency.
Example :
Balance :  500 cash
after use command remove 100 cash
new balance :  400 cash
Please.  make command remove.  I need remove money because easy to manage money.
Thank you very much.

Just use /adminpay with a negative amount. That should get you there.


The games i have are mario 64, mario kart 64, pokemon white, pokemon pearl, pokemon heartgold, platinum, diamond, and soulsilver

sure, send an email to<EMAIL_ADDRESS>


decrypt_agile is undefined

0.12.5 xlsx.js
When use this version lib to read password protected file
after debug and see this:

thus I got "File is password-protected" error.
what does the variable come from?

BTW , I searched in whole project and find nothing.

The decrypt_agile and decrypt_std76 hooks are intentionally leaked so that our encryption module (part of the Pro compendium) can hook into the read and readFile functions (at a high level, Excel generates a plain unencrypted file then encrypts that file in a new package).   The checks are typeof-guarded so it shouldn't trigger an error in execution.


add Event to onKeyReleased callback

react-simple-keyboard version - 3.5.17
When the option physicalKeyboardHighlightPressUsePointerEvents={true} is used, onKeyPress callback contains 2 arguments: button and event. Event is present if user clicks on virtual keyboard and event is undefined on physical one. Is it possible to add event as argument to onKeyReleased so that it would be easy to tell the difference between where the event comes from?

Done. Thanks!

Can the documentation please be updated to reflect above change?


System.DllNotFoundException: Unable to load DLL 'sqlcipher'

Hi,
I think, this is related to #194. But this is happening with SQLitePCLRaw.bundle_sqlcipher with the RID win-x64.
I have following dlls in my win_x64 folder:

SQLitePCLRaw.batteries_sqlcipher.dll
SQLitePCLRaw.batteries_v2.dll
SQLitePCLRaw.core.dll
SQLitePCLRaw.provider.e_sqlite3.dll
SQLitePCLRaw.provider.sqlcipher.dll

Is there some kind of workaround? Because I need to stay to 1.1.7, because of this issue.
Are there maybe precompiled dlls for win-x64? Can I maybe use the compiled dll from win7-x64?

Can you try version 1.1.10-pre20180223200113 ?
The 1.1.10 prerelease has fixes for the win7 RID problem.
I am also interested in the (perhaps small) possibility that the 1.1.10 prerelease might fix the problem you are seeing with aspnet/Microsoft.Data.Sqlite#480.

Ok, this is fixed in 1.1.10-pre20180223200113.
Thank you very much!


Move artifacts from k/community to k/sig-security

What?
Create the following set of directory structure:
k/sig-security:

docs:

Refactor this to here: https://github.com/kubernetes/community/blob/master/sig-security/sig-security-docs/OWNERS

third-party-audit:

Move all of this here: https://github.com/kubernetes/community/tree/master/sig-security/sig-security-external-audit
Move all of this here: https://github.com/kubernetes/community/tree/master/sig-security/security-audit-2019
Move all of this here: https://github.com/kubernetes/community/tree/master/sig-security/security-audit-2021

tooling:

Move this to here: https://github.com/kubernetes/community/blob/master/sig-security/sig-security-tooling/OWNERS

How?
This will need two set of PRs, one to remove stuff from k/community and one to add the same stuff to k/sig-security
Why do this?
This will unblock following PRs: https://github.com/kubernetes/community/pull/6140 and https://github.com/kubernetes/community/pull/5853. More importantly, it will allow allow sig-security artifacts to have a single landing page.
Anything else you would like to add @tabbysable @IanColdwater @reylejano @savitharaghunathan ?
/sig security
/kind cleanup
/label good first issue
/label help wanted

/assign @chetak123

@PushkarJ when I was reviewing https://github.com/kubernetes/sig-security/pull/12 I asked for the directory structure to be matched to the old one from k/community, but I see here you requested it specifically.
I think that because our subproject names are fairly general terms on their own (docs and tooling especially) that it could be a little misleading to have the directory names as just those short words. So, I like the old names that match the slack channel names better. I don't want to dictate that, though-- WDYT about this idea of keeping the directory names the same as before?

Hi Tabby, thank you for reviewing!! I did not think about mapping the directory names to channel names for consistency earlier but I like the idea. Changed the original description as per your suggestion now.
@chetak123 please go ahead and make the renaming changes @tabbysable suggested in the PR for adding new files and then we should be good to go

With https://github.com/kubernetes/sig-security/pull/12 and https://github.com/kubernetes/community/pull/6176 merged, closing this ticket. Thank you again @chetak123 for working on this!!


style(icon): Update to new twitter logo

Fixes: #3633

@venkyvajrala why did you update docsy in this PR?
Also, updating docsy is way more complex than just updating the submodule.

@thesuperzapper
it has dependency on docsy  font-awesome package version. Twitter logo doesn't appear if we don't update the package version of font-awesome to 6.4.2 .I made the change in docsy repo and added the commit which has 6.4.2 package version for font-awesome library.
Please let me know if I am missing anything.

#3633

@thesuperzapper  to review. thanks.

@venkyvajrala it is not possible to update docsy without a crazy amount of work.
If people really care about the twitter logo, they will need to include it manually (not sure how), or we will need to create a forked version of docsy (starting at our current version) with a newer font-awsome version.

/hold


update changelog

API change check
API changes are not detected in this pull request.

/azp run python - cosmos - ci


Update the icons appeared on ''supported by" of scidrive login page

At the moment there are by  Sloan foundation, VAO, NSF, SDSSIII CasJobs/MyDB, SQLShare.
It may require Alex's input.

Updated the old SDSS-III logo to be the new SDSS logo.
HOWEVER, still need to check on funding to make sure all logos are up to date. @mikear69 or @thakar, do you know?
Link: http://preprod.scidrive.org/scidrive/ (bottom of the page)


Gt 672

Created mixpanel event management sending mechanism. Uses "mixpanel.log" property for each service to determine if we have to log specific mixpanel calls.
The keys involved on this first version are:

process.start,{{processid}}.end logs when a process is completed
process.var.{{varName}}.change logs when a process variable is changed
process.node.{{nodeName}}.trigger logs when a process task is started
process.node.{{nodeName}}.left logs when a process task is completed
rules.{{ruleName}}.fire logs when a business rule fires

I think the expectation that identify is called before events are sent needs to be removed. AFAIK, that's only useful in single-identity use cases.

Darshan asked to add it. It is the way it is being currently used on sapphire.


[All Hosts] (sso) add allowConsentPrompt option to walkthroughs

Docs Build status updates of commit 2a6d577:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 2a6d577:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/create-sso-office-add-ins-aspnet.md
:white_check_mark:Succeeded
View

docs/develop/create-sso-office-add-ins-nodejs.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Use SizeProvider for all sizings in the app

As a developer I should use the SizeProvider from Style lib for all sizing.
Remove Sizer library from app.
Acceptance criteria:

App running without sizer and use Style SizeProvider.

https://user-images.githubusercontent.com/8357343/166322300-501f1b02-a18d-4cc1-9acb-adc7f774647e.mp4


Allow multiple ACL's inside haproxy_lb provider

https://github.com/sous-chefs/haproxy/blob/master/providers/lb.rb#L13
This line makes it impossible to have an array of ACL's.
Is it possible to change the line to incorporate 'sub'-array's / hashes?
I would like to be able to define multiple 'acl' lines and multiple 'use_backend' lines to incorporate header switching.

does this look like it would fix your problem: #106

can you test against branch 106_MERGE


Update PowerBI Overview

Updates per feedback from Adam Saxton. Tagging @CamSoper for merge. I will have Power BI team review and provide feedback for sign-off.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/powerbi-embedded.md
:white_check_mark:Succeeded
View

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

Tagging @guyinacube for review.

Made the changes per @guyinacube request. Adam, can you please re-review and use the sign-off hash tag if it looks good? Thanks!

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/powerbi-embedded.md
:white_check_mark:Succeeded
View

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


Fix for issue 5932, adding default options to action(s)

Apologies for any missteps, first time doing a pull for a open source project :).
Issue: https://github.com/storybooks/storybook/issues/5932
What I did
I noticed that both action and actions defaulted their options to {}, hence the reported issue. To remedy this, I imported config from configureActions.ts and set it as the new default for ActionOptions.
How to test
To test, you can play around with the “with text” and “with some emoji” stories in the cra-kitchen-sink example story (https://github.com/storybooks/storybook/tree/next/examples/cra-kitchen-sink).
Sample test flow:

Navigate to “Button” -> “with text” in the cra-kitchen-sink storybook
Click the “Hello Button” a couple times to generate some action logs.
Switch to the “with some emoji” story. Note that the action logs aren’t wiped since that action doesn’t have clearOnStoryChange: true
Click the emoji button in the “with some emoji” story. Note that a new action is logged
Change to a different story. Note that the action log is wiped clean since the action associated with that button doesn’t have any options, thus using the default config from configureActions.

Additional testing scenarios can be done by adding configureActions with various settings in the config.js of the cra-kitchen-sink storybook.

Is this testable with Jest or Chromatic screenshots?
Does this need a new example in the kitchen sink apps?
Does this need an update to the documentation?

If your answer is yes to any of these, please make sure to include it in your PR.

@ndelangen I not certain how to assign reviewers, is it supposed to be an automatic process?


Change logging to use uber zap

As proposed in #247 we have to change fmt.println logging to log but at the same time suggest to use uber/zap since it has better performance compare to logrus which is used in master

There are more important advantages imo:

zap can be disabled/enabled per component
zap can be hierarchical (one can create child loggers based on parent ones, all fields will be inherited)

Now that the DBFT library is using it it's more important to fix this issue for logging consistency.


autoinject not working

This test fail with Expected Logger({  }) to be undefined.
https://github.com/unional/dependency-injection/blob/master/test/injection.spec.js
I was trying to use it in my TypeScript file with the same result.
Tried @autoinject() too.

The typescript files are transpiled at runtime through systemjs/jspm.

You can't inject logger. It's constructor cannot be called. You must use the getLogger function instead.

The Logger is an empty class, not the actual Logger, it can be just ClassA. It can be injected through inject(Logger)(App1) or @inject, but not @autoinject.

You probably don't have your TypeScript build setup correctly then. You should have something like this:
{
	"compilerOptions":{
		"target": "ES5",
		"module": "system",
		"emitDecoratorMetadata": true,
    "experimentalDecorators": true
	}
}

Ar, yes! That's what it is. :) Thanks!
Is there a way to do the same on babel? Would like to contribute a PR and add that test in.

There's no way to use autoinject with Babel because it doesn't have the notion of type metadata. However, there is a community member that wrote a babel plugin that would generate the type metadata so that you didn't need to use inject or autoinject at all, but could just put the types in the constructor. I'm not sure how solid that is though...but it's something that can be done as a Babel plugin.

I see. Might take a look at it.
I know there is a typescript preprocessor for karma, but I couldn't get it to work in the past.


my life is bad

If you found a bug, want to propose a feature or feel the urge to complain about your life, feel free to visit the issues page.


Create helper for adding use-flags to package from Buildconfig.sh

Basically what topic says. Should manage:

creating directory structure if it doesn't exist
adding use flags
possibly checking duplicity and/or overwriting current variables ( for instance USE="foo" would overwrite a previous USE="-foo", not just add to file for readability)

You may want to check out app-portage/flaggie for this purpose.

@steveeJ thanks for the tip!


Element Ambiguous Error when publishing

Power BI Version: 2.75.5649.861 64-bit (November 2019)
Windows 10
When getting to publishing, there's an elementambiguouserror with pywinauto that show two Publish to Power BI options.
Error and screenshot below.
pywinauto.findwindows.ElementAmbiguousError: There are 2 elements that match the criteria {'title': 'My workspace', 'top_level_only': False, 'parent': <uia_element_info.UIAElementInfo - 'Publish to Power BI', WindowsForms10.Window.20008.app.0.3c73ab4_r6_ad1, 988888>, 'backend': 'uia'}

I have the exact same issue here...

I think this can be fixed by adding 'found_index = 0' to line 77:

That worked, thanks!


wowza: scalabilty

Part of https://github.com/cargomedia/roadmap/issues/15

Draft of wowza scalability approaches

@njam @tomaszdurka a few initial thought about wowza scaling approaches

So you confirmed the second approach is possible?, with:

Using "load balancer" on publisher's end
Connecting "edge" servers to multiple "engines"

Alternatively as discussed we could multiply the first schema ("per event"), and register all of these clusters in CM, which supports round robin for multiple servers.

Well, there are options which I am testing and evolving now...

Using "load balancer" on publisher's end

I will confirm one more time but I don't see any downside for doing this. It is kind of basic behaviour of balancing. Wowza balancer works similar to round robin with extension to measure load on end-points.

Connecting "edge" servers to multiple "engines"
afaics it is impossible to do it easy. There is option that every wowza engine / publisher could have another application name so wowza edge instances could bind to all of them. The downside are restarts of edge instances every time the publisher instance come up. Also this approach require some advanced CM logic for controlling.

Per Event Scaling feels less complicated and better scalable but it is not my final point of view:)

@tomaszdurka please have a look on updated "head comment" and abstract requirements for CM app
@tomaszdurka cloud formation could be done as separate service/api
let's pair on it a bit? and discuss on stand-up

Did you consider using Terraform instead of CloudFormation?

I didn't think about Terraform but in this case it could make sense. I will have to investigate.
AWS/SDK/CloudFormation is pretty robust I would say but Terraform sounds exciting:)

As discussed the following might be a good solution:

Terraform to define and deploy infrastructure "bundles"
Ansible to run Terraform "apply"
Ansible Tower to trigger and control Ansible tasks. Can be accessed via REST API (similar to pulsar-rest-api).

:+1: Agree, this is very good replacement of CM_AWSCloudFormation_* which would limit us in the future.
Still the question is should we build this stuff as separate service/api :)?

The ansible tower could run on a dedicated cargomedia-wide machine, which provides a REST API to launch and control infrastructure. The application could then just have a job to trigger some launch-new-sk-wowza-publisher task on that API.

so then:

where would you store the runtime configuration to measure the load (number of publishers) on the specific server-group and also to store IPs, channelName etc?
how would you centralised authentication RPC calls (single point on www3 or any in App farm)?

Runtime wowza publisher config: Currently stored in flat config file. But the whole implementation allows for multiple wowza servers. We would just need to move parts of this configuration into mysql, shouldn't be too hard.
RPC: Currently go to https://www.fuckbook.com/rpc/null, so load balanced by the HTTP proxy. Could stay as is

I meant rather that if we start new group of servers then each streamName would be assigned exactly to one group of servers? We need layer which could detect where the streamName belongs to? Just to provide some informations for subscribers etc? I assume that each group has own load balancer etc.
agree

That mechanism is mostly built-in afaik. If you look at CM_Stream_Adapter_Video_Wowza::synchronize(), you can see that status information is enriched with server hostname:
$publish['serverHost'] = $wowzaServer['privateIp'];

And on the publisher side (SK_Component_VideoStreamPublish) we use CM_Stream_Video::getServer() which picks a random server to publish to:
        if (null === $serverId) {
            $serverId = array_rand($servers);
        }

Just draft of concept for the records cm-ansible-teraform-aws-puppet

Depending on how complex that terraform apply command to launch a new wowza publisher is we could also just use pulsar and pulsar-rest-api!

Hmm I did some test already and I believe we could do it with pulsar-rest-api and pulsar tasks.
Would work:

at least terraform apply by defining pulsar task

Not sure:

about e.g terraform output ip etc. to get runtime configuration; how to do it easily via pulsar-rest-api?
also where to store the *.tf files; CM or pulsar-conf-cargomedia?

Ahh I see, we have blocking requests to pulsar-rest-api; very cool!

Regarding pulsar-rest-api: You can either call the job "blocking", then the result will be returned, or can use "non-blocking" and use WebSocket to retrieve result.
See: https://github.com/cargomedia/pulsar-rest-api#create-job
Not sure about TF-files either :)

Closing for the time being. Pleas reopen if you disagree.


Switch tabs doesn't work on and to non pinned tab

tabsSwitchRight doesn't go to last active non-pinned tab.
Same, while being in last non pinned tab command tabsSwitchLeft doesn't work.
joplin v.1.5.7

Its a limitation in the current implementation. I've already planned to change this and so the behavior will be changed in the upcoming release.

Its a limitation in the current implementation. I've already planned to change this and so the behavior will be changed in the upcoming release.
Good to know, closing this then.


Rake setup:cluster failed - delivery-cli RPM package not signed

Using commit 2c37c08ea5e45279d4114aee69fa7653f97842d6, and ChefDK 0.10.0 on OSX 10.11.1.
Ran rake setup:generate_env and opted for a Vagrant-based cluster initially. Using the suggested default of opscode-centos-6.6 for the VMs.
When running rake setup:delivery_build_nodes, it ultimately fails on chef_ingredient[delivery-cli] action upgrade with the following underlying reason:
[...]
yum -d0 -e0 -y --disablerepo=* --enablerepo=chef-stable  install delivery-cli-20150811030818+20150811030818-1.el6 returned 1:
                                 STDOUT:
                                 STDERR:

                                 Package delivery-cli-20150811030818+20150811030818-1.el6.x86_64.rpm is not signed

I've also submitted an issue on chef-ingredient repo regarding the test-kitchen suites, because I wanted to add a kitchen suite to demonstrate this problem more easily:
https://github.com/chef-cookbooks/chef-ingredient/issues/63

Switching to the bento ubuntu-14.04 vagrant box, I get the following during the build-node provisioning:
apt_package[push-client] (/var/chef/cache/cookbooks/chef-ingredient/libraries/debian_handler.rb line 66) had an error: Chef::Exceptions::Package: opscode-push-jobs-client has no candidate in the apt-cache

There is an update in the delivery_build cookbook that allows a '--nogpgcheck' for installing the delivery-cli. The real fix would be to generate a new build of delivery-cli RPM and sign it.

@shanesveller: As Stephen was saying, what you need to do as a workaround is to modify your environment file you generated with rake setup:generate_env which is located at delivery-cluster/environments. Add the following options parameter to the builders block:
 "builders": {
    <- extra options depending on your driver ->
    "delivery-cli": {
       "options": "--nogpgcheck"
    }
  }

@shanesveller I have verified the second problem you listed here and I can confirm that it is a current problem we have with chef-ingredient and the rename of our product push-jobs to push-client.
============================================================================
Error executing action `install` on resource 'chef_ingredient[push-client]'
============================================================================

Chef::Exceptions::Package
-------------------------
apt_package[push-client] (/var/chef/cache/cookbooks/chef-ingredient/libraries/debian_handler.rb line 66) had an error: Chef::Exceptions::Package: opscode-push-jobs-client has no candidate in the apt-cache

There is already some issues/PRs opened here to track down the work done:
https://github.com/chef-cookbooks/chef-ingredient/issues/59
https://github.com/chef-cookbooks/chef-ingredient/pull/66
I am going to close this issue since the first problem you described has a current workaround. Please open another issue for the second one. Thank you for you feedback! :+1:

@afiune Thanks for the response!

@shanesveller You are very welcome.. Thank you for the feedback. :smile:


feat(lang): add maxInlayHintLength for vtsls to resolve inlay hint to…

Description
feat(lang): add maxInlayHintLength for vtsls to resolve inlay hint too long
Related Issue(s)

https://github.com/neovim/neovim/issues/27240
https://github.com/yioneko/vtsls/pull/173

Screenshots
before

after

Checklist

[ x ] I've read the CONTRIBUTING guidelines.

ty!


Updated some Debian scripts to support arm

This PR updates some (not all) scripts in script-library to add fixes so they can be used to build arm64 containers too. Note that the PR doesn't add full arm64 support for the repo (see #558), but it does offer some help in that direction.
How this came to be: I wanted to be able to use a devcontainer on a M1 Mac. The base images that we currently publish don't support ARM, so I resorted to creating my own containers but I used the scripts in the script-library folder to make things easier for me.

There's actually a branch with all the scripts updated to support this along with more secure use of GPG keys for apt. I'm waiting on a M1 mac to arrive to allow me to test, but if you want to help verify, it is here:  https://github.com/microsoft/vscode-dev-containers/tree/clantz/apt-key-and-architecture

@Chuxel I had missed that :) Looks like it already includes everything I did in this PR and then more. Closing this PR!


Test build

Codecov Report

:exclamation: No coverage uploaded for pull request base (v0.2.0@32f1875). Click here to learn what that means.
The diff coverage is 76.19%.

@@           Coverage Diff            @@
##             v0.2.0     #24   +/-   ##
========================================
  Coverage          ?   67.1%
========================================
  Files             ?      27
  Lines             ?     687
  Branches          ?       0
========================================
  Hits              ?     461
  Misses            ?     226
  Partials          ?       0

Impacted Files
Coverage Δ

webservice/pybonjour_service/pybonjour_service.py
54.83% <ø> (ø)

webservice/handler/abstract_request_handler.py
85.18% <ø> (ø)

webservice/pybonjour_service/test/server.py
0% <ø> (ø)

webservice/pybonjour_service/test/discover.py
0% <ø> (ø)

webservice/websocket/web_socket_connections.py
58.33% <ø> (ø)

webservice/pybonjour_service/test/client.py
0% <ø> (ø)

...service/websocket/web_socket_connection_handler.py
55.55% <100%> (ø)

webservice/handler/param_handler.py
71.42% <100%> (ø)

webservice/handler/effect_handler.py
68.75% <100%> (ø)

webservice/util/handler_utils.py
100% <100%> (ø)

... and 17 more

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 32f1875...2104fc6. Read the comment docs.


Fix attach() method

this.attached.concat(coral) returns a new array and does not change the original one.

Looks like we pushed some changes at the same time, sorry! Just fixed this with v7.6.5.
Demo: https://codepen.io/cferdinandi/pen/YzGLMdG

Looks like we pushed some changes at the same time, sorry! Just fixed this with v7.6.5.
Demo: https://codepen.io/cferdinandi/pen/YzGLMdG


None of the form controls like Buttons, Checkboxes etc are keyboard operable or have basic accessibility

Describe the bug
None of the form controls like Buttons, Checkboxes etc are keyboard operable or have basic accessibility
To Reproduce
Steps to reproduce the behavior:
I am talking basic accessibility here, like

Correct Semantics, E.g Checkboxes are rendered as
Nothing is keyboard focusable, operable.
Links are
and a ton more...

Expected behavior
When you publish a component library it should at-least be accessible to a basic level.

I would like to work upon this issue

Description not precise, could you be more clear please. Steps to reproduce unclear as well


Added overlay_protocol

@Eirenliel before I continue, just wanted you to give this a quick glance over. I decided that it might be a bad idea to block the overlay on the advanced union support for flatc-rust, so I introduced a temporary protocol in the meantime. The flatbuffer schema is finished I think, but the OverlayBridge is still WIP.

all checks have failed

idk i'm not to fond of the idea of pusing something temporarry in. there is nothing more permanent than temporarry. overlay is not the blocking feature for reviews, imo.

Okay then I don't mind.

Is this still needed ? Shoud we close it. @TheButlah ?

No longer needed now that the main protocol is compatible with rust


Support Cabal flags in stack.yaml

Parse Cabal flags section from stack.yaml file, and produce appropriate Nix overrides.
flags:
  package-name:
    flag-name: true

https://docs.haskellstack.org/en/stable/yaml_configuration/#flags
https://docs.haskellstack.org/en/stable/GUIDE/#cabal-flag-management

I'm so needing that. But that's OK, writing overrides by hand is not a big problem.
However, I have .cabal files with conditional flags in them (like static, or production). Using stackage2nix for now ignores those and pick the dependencies from the default value of those flags.
For instance, production adds a postgresql dependency. The generated default.nix doesn't have it. So if I override my package and I add a -fproduction, then nix-build complains of a missing dep, namely postgresql.
Can I go around this?

You definitely can. Check out Nixpkgs example <IP_ADDRESS>. How to build projects that depend on each other
Suppose, project foo is missing from generated Haskell packages, so you add derivation manually:
cd ~/src/foo && cabal2nix . > default.nix

Project bar lacks dependency on foo, so you override it with:
bar = addBuildDepends super.bar [ foo ];

Eventually, it should look something like this:
haskellPackages = super.haskellPackages.override {
      overrides = self: super: rec {
        foo = self.callPackage ./src/foo {};
        bar = addBuildDepends super.bar [ foo ];
      };
    };

I'm having trouble picking persistent-postgresql out of the packages.nix, so that I can add it as an extra build dependency.


bower ready, with d3 dependency

This solve problem with d3 dependency when rickshaw is installed via bower. Specifically when installed angular-rickshaw, because bower can't resolve dependencies.
Problem:
Uncaught ReferenceError: d3 is not defined

Looks legit to me. :+1:


SCons: Bump required C++ standard to C++17

SCons: Bump required C++ standard to C++17
As per #36436, we now need C++17's guaranteed copy elision feature to
solve ambiguities in Variant.
Core developers discussed the idea to move from C++14 to C++17 as our
minimum required C++ standard, and all agreed. Note that this doesn't
mean that Godot is going to be written in "modern C++", but we'll use
modern features where they make sense to simplify our "C with classes"
codebase. Apart from new code written recently, most of the codebase
still has to be ported to use newer features where relevant.
Proper support for C++17 means that we need recent compiler versions:

GCC 7+
Clang 6+
VS 2017 15.7+
Xcode 10+

Additionally, C++17's std::shared_mutex (conditionally used by
vk_mem_alloc.h when C++17 support is enabled) is only available in
macOS 10.12+, so we increase our minimum supported version.

Travis: Use Ubuntu 18.04 (bionic) as base image
It's now available and allows us to have a better default environment,
with GCC 7.4.0 and Clang 7.
We now need GCC 7+ for C++17 support so it's more efficient to upgrade
the image than to install it on Ubuntu 16.04 (xenial).
Also fixes a couple -Wdeprecated-declarations warnings on macOS now
that we build against macOS 10.12.

Actually, I'll just update our Travis script to run on Bionic (18.04 LTS) instead of Xenial (16.04 LTS) so we get GCC 7 as default.

Additionally, C++17's std::shared_mutex (conditionally used by
vk_mem_alloc.h when C++17 support is enabled) is only available in
macOS 10.12+, so we increase our minimum supported version.

Note that according to https://gs.statcounter.com/macos-version-market-share/desktop/worldwide, there's still ~6% of macOS users running 10.11 (OS X El Capitan). MoltenVK also seems to support 10.11+ for now: https://github.com/KhronosGroup/MoltenVK#building-moltenvk
So if we want to keep support for El Capitan, we could also disable vk_mem_alloc.h's use of std::shared_mutex with a compile flag: https://github.com/godotengine/godot/blob/master/thirdparty/vulkan/vk_mem_alloc.h#L3615-L3627
WDYT @godotengine/macos?

So if we want to keep support for El Capitan, we could also disable vk_mem_alloc.h's use of std::shared_mutex with a compile flag: https://github.com/godotengine/godot/blob/master/thirdparty/vulkan/vk_mem_alloc.h#L3615-L3627

Here's an alternative branch that implements this: https://github.com/akien-mga/godot/tree/c++17-keep-macOS-10.11
It seems to build fine, so if we want to keep macOS 10.11 support for now, we can use that instead. It might restrict our own use of some C++17 features that macOS 10.11 doesn't support, but we can reassess then. Or just go ahead with dropping 10.11 support now - it will likely be less significant in ~1 year when 4.0 is released.

Note that according to gs.statcounter.com/macos-version-market-share/desktop/worldwide, there's still ~6% of macOS users running 10.11 (OS X El Capitan). MoltenVK also seems to support 10.11+ for now: KhronosGroup/MoltenVK#building-moltenvk

By the time Godot 4.0 will be released, that will likely drop even lower. Not to mention people using macOS 10.11 are typically using old Macs, with hardware too slow for modern gaming.

Note that according to gs.statcounter.com/macos-version-market-share/desktop/worldwide, there's still ~6% of macOS users running 10.11 (OS X El Capitan). MoltenVK also seems to support 10.11+ for now: KhronosGroup/MoltenVK#building-moltenvk

Only 2012+ Mac (and 2010 MacPro with upgraded GPU) are Metal capable, and all of the Metal capable Macs should be able to run 10.14. Macs still running 10.11 are probably 2007-2008 models without any Metal support.

And one last data point in favor of dropping 10.11 support: the latest Steam hardware survey counts 3.01% macOS users in the Steam userbase, and 0.10% macOS 10.11 users. So that's 3.3% of 10.11 among macOS users on Steam.
Let's drop it :see_no_evil:

As per https://github.com/godotengine/godot/pull/36436#issuecomment-589999282, another change was missing in this PR, which is that full C++17 support on macOS requires Xcode 10+. So Travis needs a change too since it defaults to 9.4.1. (Adding it to #36346 directly.)

hrmm, is this change going to break support on SteamOS? For building on SteamOS I currently use Ubuntu 16 LTS as they both use the same version of GLIBC which is very old... 2.19 I believe.

No, as long as you have a recent enough compiler. Ubuntu 16's default GCC version is too old, but you can install a newer GCC or Clang and use it just fine.
Official Godot binaries are built on Ubuntu 14.04 with GCC 9.


add support for other calculation grouping in history attribute

If there is not a default string representation for an OCGIS calculation grouping, convert it to a string by default.

Thanks!


Can I edit docs added by someone else?

The Discord.js docs are out of date by several versions and basically obsolete. Is there a way to upload updated ones or does the original creator have to do it?

I think this is the same question as in #6. Might be a good one to answer and add to the FAQ.

yes

Thanks @choovin. Can you clarify what you're saying yes to? :)

Yeah honestly I have never used readthedocs till now, but it's frustrating cause it's one of the first results on google when searching, but also way out of date so I figured I would update what I could if possible.

Stale question closing.

@Blendify I'm not sure that this is a stale issue. @Ev1dentFir3's question wasn't answered, which is on us, not him.

We have a policy for abandoned projects now, check it out at http://docs.readthedocs.io/en/latest/abandoned-projects.html

Thanks @humitos. @Ev1dentFir3 - if this doesn't answer you, let us know!

This issue has not been resolved as is the case with the discord.js issue.

the discord.js docs not being updated doesn't count as my question not being answered.
They answered my original question so his closing the ticket is valid.


Update v1.4 docs

Update screenshots
Update add command

Codecov Report

Merging #225 (da80d18) into master (fbc7380) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #225   +/-   ##
=========================================
  Coverage     70.95%   70.95%
  Complexity      574      574
=========================================
  Files            97       97
  Lines          1887     1887
  Branches        211      211
=========================================
  Hits           1339     1339
  Misses          490      490
  Partials         58       58

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 fbc7380...da80d18. Read the comment docs.

Added Slideshow Class Diagram


New test samples/ffi/http/test/http_test failing after reland.

There are new test failures on [ffi] Reland the iNativeCallable.listener example test..
The tests
samples/ffi/http/test/http_test RuntimeError (expected Pass)
samples/ffi/http/test/http_test Crash (expected Pass)
samples/ffi/http/test/http_test RuntimeError (expected Pass)

are failing on configurations
vm-aot-android-release-arm64c
vm-aot-android-release-arm_x64
vm-aot-win-debug-x64c
vm-reload-linux-debug-x64
vm-reload-linux-release-x64
vm-reload-rollback-linux-debug-x64
vm-reload-rollback-linux-release-x64
vm-win-debug-x64
vm-win-debug-x64c

/cc @liamappelbe

Example logs:
vm-reload-linux-debug-x64:

--- Command "vm" (took 01:47.000933s):
DART_CONFIGURATION=DebugX64 out/DebugX64/dart --hot-reload-test-mode --sound-null-safety -Dtest_runner.configuration=vm-reload-linux-debug-x64 --ignore-unrecognized-flags --packages=/b/s/w/ir/cache/builder/sdk/.dart_tool/package_config.json /b/s/w/ir/cache/builder/sdk/samples/ffi/http/test/http_test.dart

exit code:
255

stdout:
00:00 +0: httpGet

00:22 +1: httpServe

00:29 +1 -1: httpServe [E]

  Bad state: Future already completed

  dart:async/future_impl.dart 43:31             _AsyncCompleter.complete
  samples/ffi/http/test/http_test.dart 21:38    main.<fn>.<fn>
  samples/ffi/http/lib/http.dart 46:14          httpServe.onNativeRequest
  samples/ffi/http/lib/http.dart                httpServe.<fn>
  dart:async/zone.dart 1415:13                  _rootRunUnary
  dart:async/zone.dart 1308:19                  _CustomZone.runUnary
  dart:async/zone.dart 1217:7                   _CustomZone.runUnaryGuarded
  dart:async/zone.dart 1254:26                  _CustomZone.bindUnaryCallbackGuarded.<fn>
  dart:isolate-patch/isolate_patch.dart 184:12  _RawReceivePort._handleMessage

  This test failed after it had already completed.
  Make sure to use a matching library which informs the test runner
  of pending async work.

  dart:async/future_impl.dart 43:31             _AsyncCompleter.complete
  samples/ffi/http/test/http_test.dart 21:38    main.<fn>.<fn>
  samples/ffi/http/lib/http.dart 46:14          httpServe.onNativeRequest
  samples/ffi/http/lib/http.dart                httpServe.<fn>
  dart:async/zone.dart 1415:13                  _rootRunUnary
  dart:async/zone.dart 1308:19                  _CustomZone.runUnary
  dart:async/zone.dart 1217:7                   _CustomZone.runUnaryGuarded
  dart:async/zone.dart 1254:26                  _CustomZone.bindUnaryCallbackGuarded.<fn>
  dart:isolate-patch/isolate_patch.dart 184:12  _RawReceivePort._handleMessage

01:00 +1 -1: Some tests failed.

Consider enabling the flag chain-stack-traces to receive more detailed exceptions.
For example, 'dart test --chain-stack-traces'.

stderr:
Unhandled exception:
Dummy exception to set exit code.

--- Re-run this test:
python3 tools/test.py -n vm-reload-linux-debug-x64 samples/ffi/http/test/http_test

vm-aot-android-release-arm_x64:
...
Executing adb -s HT7590205810 push /b/swarming/w/ir/out/ReleaseAndroidARM_X64/generated_compilations/vm-aot-android-release-arm_x64/samples_ffi_http_test_http_test/out.aotsnapshot /data/local/tmp/precompilation-testing/test/out.aotsnapshot
Stdout:
/b/swarming/w/ir/out/ReleaseAndroidARM_X64/generated_compilations/vm-aot-android-release-arm_x64/samples_ffi_http_test_http_test/out.aotsnapshot: 1 file pushed, 0 skipped. 2815.5 MB/s (2294364 bytes in 0.001s)
ExitCode: 0
Time: 0:00:00.197666

Executing adb -s HT7590205810 shell export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/data/local/tmp/precompilation-testing/test;export TEST_COMPILATION_DIR=/data/local/tmp/precompilation-testing/test;/data/local/tmp/precompilation-testing/dart_precompiled_runtime --android-log-to-stderr --sound-null-safety -Dtest_runner.configuration=vm-aot-android-release-arm_x64 --ignore-unrecognized-flags --packages=/b/swarming/w/ir/.dart_tool/package_config.json /data/local/tmp/precompilation-testing/test/out.aotsnapshot ; echo AdbShellExitCode:  $?
Stdout:
00:00 +0: httpGet

00:00 +0 -1: httpGet [E]

  Bad state: No element

  dart:_internal/iterable.dart 117                          ListIterable.firstWhere
  b/swarming/w/ir/samples/ffi/http/lib/dylib_utils.dart 20  dlopenPlatformSpecific
  b/swarming/w/ir/samples/ffi/http/lib/http.dart 65         dylib
  b/swarming/w/ir/samples/ffi/http/lib/http.dart            nativeHttpGet
  b/swarming/w/ir/samples/ffi/http/lib/http.dart            httpGet
  b/swarming/w/ir/samples/ffi/http/test/http_test.dart 15   main.<fn>
  package:test_api/src/backend/declarer.dart 215            Declarer.test.<fn>.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/declarer.dart 213            Declarer.test.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/invoker.dart 258             Invoker._waitForOutstandingCallbacks.<fn>

00:00 +0 -1: httpServe

00:00 +0 -2: httpServe [E]

  Bad state: No element

  dart:_internal/iterable.dart 117                          ListIterable.firstWhere
  b/swarming/w/ir/samples/ffi/http/lib/dylib_utils.dart 20  dlopenPlatformSpecific
  b/swarming/w/ir/samples/ffi/http/lib/http.dart 65         dylib
  b/swarming/w/ir/samples/ffi/http/lib/http.dart            nativeHttpServe
  dart:ffi-patch/ffi_patch.dart                             _NativeCallableBase._pointer=
  b/swarming/w/ir/samples/ffi/http/lib/http.dart            httpServe
  b/swarming/w/ir/samples/ffi/http/test/http_test.dart 21   main.<fn>
  package:test_api/src/backend/declarer.dart 215            Declarer.test.<fn>.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/declarer.dart 213            Declarer.test.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/invoker.dart 258             Invoker._waitForOutstandingCallbacks.<fn>

00:00 +0 -2: Some tests failed.

Consider enabling the flag chain-stack-traces to receive more detailed exceptions.
For example, 'dart test --chain-stack-traces'.

AdbShellExitCode: 255
Stderr:
WARNING: linker: Warning: unable to normalize ""
WARNING: linker: /data/local/tmp/precompilation-testing/dart_precompiled_runtime: unsupported flags DT_FLAGS_1=0x8000001
Unhandled exception:
Dummy exception to set exit code.
ExitCode: 255
Time: 0:00:00.136782

--- Re-run this test:
python3 tools/test.py -n vm-aot-android-release-arm_x64 samples/ffi/http/test/http_test

vm-win-debug-x64:
--- Command "vm" (took 15.000692s):
set DART_CONFIGURATION=DebugX64 & set DART_SUPPRESS_WER=1 & set DART_CRASHPAD_HANDLER=C:\b\s\w\ir\out\DebugX64\crashpad_handler.exe & out\DebugX64\dart.exe --sound-null-safety -Dtest_runner.configuration=vm-win-debug-x64 --ignore-unrecognized-flags --packages=C:\b\s\w\ir\.dart_tool\package_config.json C:\b\s\w\ir\samples\ffi\http\test\http_test.dart

exit code:
-1073740940

stdout:
00:00 +0: httpGet

--- Re-run this test:
python3 tools/test.py -n vm-win-debug-x64 samples/ffi/http/test/http_test


Connection is lost after 60 seconds

Hi!
After playing a game, it stops sending UDP after about 60 seconds. We have confirmed with Wireshark that it is transmitting UDP for a certain amount of time after playing the game.
When it stops transmitting, the following error occurs.
"UDP address is invalid XXXX:5000"
XXXX is garbled.
What to do in this case?

If it's dying after 60 seconds it's suggesting that whatever you're using to hold the uobject is being garbage collected, do you have more information on your setup?


feat: PTY output parser

Properly parsing the ANSI escape codes

fix #1
fix #2
fix #3

You're a madlad for doing this and I respect it all the way 🫡

@Moosems i have only added the templates so far and the base handler. I'd love to receive some help with the implementations though ❤

I will certainly try, but theres no guarantee I'll be much help. I'd say the easiest character to start with is the Carriage Return, though we will likely have to keep track of the visible string and actual string separately:
x = "xy\r\rza"
print(x)
# Prints "za"


no module named readline

i have a problem when i tap (python brutespray.py)
and it display (
Traceback (most recent call last):
File "brutespray.py", line 4, in
import readline, glob
ImportError: No module named readline        )

pip install -r requirements.txt

I had this issue on Kali linux.
sudo apt-get install libncurses5-dev
then
sudo -H pip3 install readline

just do:
apt install brutespray


Changing README

Fixing the readme to get rid of old information. Feel free to modify to suit, but want to make it clear this is not confused with the origin repo.

This is fine with me. @runcom PTAL

LGTM indeed


Improve logging of callback endpoint

Logging should be checked to be more verbose in the log stream and potentially add a message to the process log as well when a callback has been received.

logging is improved