rustfmt skip does not work on comments that exceed line width limit

Example:
#[cfg_attr(rustfmt, rustfmt_skip)]
fn foo() {
  // some very looooooooooooooooooooooong comment
}

produces errors of the form:
error: line exceeded maximum width (maximum: 79, found: 88)

It looks like it's not just comments, any line that exceeds the limit gets picked up by format_lines, which doesn't have attr checks.


[SAGE-203] Button update

Description

[x] remove raised prop
[x] update button styles to new sage
[x] add neutral variation
[x] verify spinner variation
~[ ] add spinner to react variation~ SAGE-278
[x] update docs

Breaking changes

remove the raised prop from the Rails and React buttons

Screenshots

Before
After

Testing in sage-lib

Visit the Button page:

Rails
React

Testing in kajabi-products

(HIGH) Update styles for the Button component across the application.

Related

Closes SAGE-203

Looking very good Quinton! One minor thing. The font weights for this font are a bit weird. They're listed as medium in Figma when medium should be semibold. If you see anything labeled as Book, that would be the correct medium weight.

🔬 💯


Mutators cannot be used for non-idempotent operations

Mutators can be called on the same data multiple times, making them unsuitable for operations that are not idempotent, such as mathematical calculations. The following is a basic reproducer:
const Vue = require("vue");
const Vuex = require("vuex");
const VuexORM = require("@vuex-orm/core");
const Model = VuexORM.Model;

Vue.use(Vuex);

class User extends Model {
    static entity = "users";

    static fields() {
        return {
            id: this.attr(null),
            aNumber: this.number(0, value => value + 1),
        };
    }
}

const database = new VuexORM.Database();
database.register(User);

const store = new Vuex.Store({
    plugins: [VuexORM.install(database)],
});

User.insert({
    data: {
        id: "abc123",
        aNumber: 41,
    }
}).then(() => {
    const user = User.find("abc123");
    console.log(user.aNumber);
});

The expectation is that 42 is logged to the console. However, you will find that 43 is logged instead.
Here is the package.json file used to reproduce the issue:
{
  "dependencies": {
    "@vuex-orm/core": "0.34.1",
    "vue": "2.6.11",
    "vuex": "3.1.2"
  }
}

It's worth noting that in real-world code, I've observed mutators being called 3-4 times on the same data, not just twice. I haven't been able to work out exactly why this happens though.

@djmattyg007 Ahh yeah this would happen. Because mutator will run every time data is getting instantiated. And I see your problem here...
Not sure what is the best solution for this. maybe we should ignore mutator after data has been persisted in the store...?

It feels like a symptom of the larger issue highlighted by #525. It looks like an architectural issue that wouldn't be easy to solve. I'm not familiar with the internals of the library so I'm not sure what the best fix would be.

Hmmm... I'll think about this. I think the ideal way is to only execute mutator when persisting record to the store, and ignore when fetching from the store.

@kiaking this is trivial. The documentation states the mutation takes place on instantiation of a model without implying that the mutation is being persisted to the store (yet it is). Introducing the concept of mutating only persisting records makes this a breaking change for anyone who's implementation utilises mutators in this way.
My two cents, considering attributes are declared for schematics I think it would be beneficial to have mutations on persisting records only and the documentation needs to reflect this.

@cuebit Yea, I agree too, I think mutation should only happen on the persistent phase. However, one idea is, maybe we should remove the current mutation feature.
We should implement some way to transform incoming data. It is a long-requested feature. Mostly for transforming API response into Vuex ORM model schema.
Mutation can be used for this purpose at some level. For example, you can change the value being saved. However, we need little more than that for the real transform feature, such as mapping field names.
For example, maybe your server response is all in snake_case, but being able to convert it to camelCase could be very nice.
// Map this
{
  first_name: 'John'
}

// To this
{
  firstName: 'John'
}

If a mutation is only going to be executed during the persistent, then I guess almost identical feature is required anyway in the transform feature.
So maybe, we could mark mutation as deprecated, add documentation, then focus on implementing transform feature. I think it would be OK after releasing the v1.0.0.
What do you think?


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 992 ms

Resolved: Bofrost.at performance has improved in 938836e.


App not opening

Steps to reproduce

Install arm64-v8a lastest version.
Open the app

Expected behavior
It should show the mai menu but keeps on the opening screen
Actual behavior
It should start
Crash logs
No response
TachiyomiSY version
1.10.5
Android version
Android 14
Device
Google Pixel 8
Other details

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.10.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Thx seems like the 1.10.2 runs the rest still bug out.


Improve QR code scanning when multiple codes are in view

Is your feature request related to a problem? Please describe.
Currently as soon when a QR code is detected we interpret its data and directly afterwards close the QR code reader. Either a success message or an error is shown.
This is especially problematic in Nürnberg where we have a PDF which two different QR codes.
Describe the solution you'd like
I think this behavior can be improved. In the case of a parsing/decoding error the QR code scanner should stay active. This way when the QR code scanner returns invalid bytes (scanning error) or an unexpected card is read (static instead of dynamic code) the user can just continue to scan until it works (This actually was the behavior previously when scanning QR codes which contain binary data before #775).
This is way more convenient than pressing OK in the error dialog and start the scanner again.
We could actually never automatically close the scanner during verification, but just show a popup above the scanner. Then the verifying person can just continue with the next QR code, until the scanner is closed explicitly.
Describe alternatives you've considered
Keep the current state.
Additional context
None

Another alternative could be that the dialog that shows the verification result, shows the action buttons "Weiteren Code scannen" and "Fertig".

Another alternative could be that the dialog that shows the verification result, shows the action buttons "Weiteren Code scannen" and "Fertig".

But what would you do if a wrong QR code is scanned? e.g. when having the whole Nürnberg PDF in view or dragging the scanner over the page?
This happened to me yesterday. Even though I did not want to scan the static code I got multiple popups at the same time saying once the QR code is invalid and once the card could not be activated.

I believe there's not much we can do about that (if the scanner has multiple qr codes in view, I guess it is undeterministic behavior which qr code it chooses to scan). What would be your solution to that?

I believe there's not much we can do about that (if the scanner has multiple qr codes in view, I guess it is undeterministic behavior which qr code it chooses to scan). What would be your solution to that?
Edit: Also, if a wrong one is scanned, I would also show the action buttons "Weiteren Code einscannen" and "Fertig". Then, the user can at least try again more conveniently.

So my proposal in this issue is to just more silently ignore invalid QR codes. We could display a snackbar message or just display nothing. This is the same behavior when e.g. scanning a QR code which is damaged.
Else when just dragging your phone over the Nürnberg PDF, you will likely get a popup which says an error happened.
Also we neeed to disable scanning new codes while another popup is open. This is also a bug right now, as you will get multiple popups which cover each other.

Imo, displaying nothing would be bad UX. If the user tries to scan a wrong QR code, we should at least tell him (somehow) that we saw the QR Code, but could not do anything with it.
Snackbar sounds fine to me for wrong qr code types. I think we should not show a snackbar, if we have no internet connection, or if the qr code is of the correct type, but backend says it's invalid.

the improvement or invalid qr code handling will be implemented here #1173


Guards: Global Guards is not being checked

Hi guys,
I'm trying to set the Guards for the RouterDelegate to check all routes but enable the '/Login' route, and it's not working. How should I set up it properly?
final routerDelegate = BeamerDelegate(
    locationBuilder: BeamerLocationBuilder(
      beamLocations: [
        HomeLocation(),
        LoginLocation(),
        BooksLocation(),
      ],
    ),
    guards: [
      // Guard /books and /books/* by beaming to /login if the user is unauthenticated:
      BeamGuard(
        pathBlueprints: ['/login'],
        check: (context, location) =>
            context.read<AuthenticationNotifier>().isAuthenticated,
        beamToNamed: '/login',
      ),
    ],
  );

Hi @nagringapp
I think you might be missing guardNonMatching: true to guard everything except pathBlueprints, i.e. /login in your case


When using relationMode="prisma" (or MongoDB), provide an escape hatch to remove emulated checks from onUpdate and onDelete

Problem
We have a preview feature called referentialIntegrity
It allows users to set the datasource property relationMode to "prisma" or "foreignKeys"
When relationMode="prisma"

Prisma Client emulates the behavior of the referential actions, onUpdate, onDelete based on the behavior observed in SQL databases that use Foreign Keys.
Prisma Migrate does not create Foreign Keys,

For MongoDB provider, relationMode="prisma" is the default and cannot be changed, it makes it easier to work with relations.
In some cases, people maybe want to have an alternative to this emulated behavior of Prisma Client

In MongoDB world this is a new behavior

some people might prefer handling relations on their own

which I think make sense if the MongoDB database is accessed by another ORM/Client (which does not have the same behavior).

some people might find it slower because of additional queries

Suggested solution
Option 1 relationMode="none"

Valid for all databases
It would be a global setting where all referential actions would be invalid and show a schema validation error.
The behavior would be that the default onUpdate, onDelete does "nothing", no checks would be performed, which means that you could break the Referential Integrity of your data. (The referenced entity might not exist)

Option 2 onUpdate="Nothing" / onDelete="Nothing"

Valid for all databases
Only valid if relationMode="prisma"
It would be a Prisma "referential action" that only exist for Prisma
The behavior would be that onUpdate, onDelete does "nothing", no checks would be performed, which means that you could break the Referential Integrity of your data. (The referenced entity might not exist)

Note, as per what @Jolg42 said yesterday on Zoom, this use case would not only be of interest for mongodb users, but for SQL database users as well. In particular, there are users who are unaware of the semantic implications of NoAction, and confuse it for "a referential action that does nothing".

Is the desired behavior possible with relational databases when using relationMode="foreignKeys" (default, where relations are managed via foreign keys)?

Removing emulated checks brought by relationMode = "prisma" is indeed possible on relational databases, only if you do not use @relation but then you lose the Prisma Client API that comes with it.

Notes:

For MongoDB what people might want to disable is the emulation on update and delete Prisma Client queries, (create is not covered by the emulation).
Technically, we can do that by introducing Nothing, for onUpdate onDelete, on @relation, where both should be set?

When relationMode="foreignKeys" (default), on relational databases.
Nothing would be impossible (as long as those are real relations with foreign keys).
Which makes us think that something like a "weak" relation could be an option https://github.com/prisma/prisma/issues/7351

When relationMode="foreignKeys" (default), on relational databases.
Nothing would be impossible (as long as those are real relations with foreign keys).
Which makes us think that something like a "weak" relation could be an option Support (some) weak relations with no foreign key #7351

Just a note that using foreign keys in general, with the ability to opt out for some particular relations (aka "weak" relations) is still important for me. I'm still removing my weak relations for prisma migrate and adding them back for prisma generate.
So, it would be super helpful to have the migrate-time escape hatch for foreign keys, as well.

@janpio Yup, I just meant to reaffirm the need and agree that I see the connection to this. Sorry for the noise.

I’m unfortunately blocked from upgrading<EMAIL_ADDRESS>to a newer version. Because I need to reply on the “bug-as-a-feature” behaviour on v4.4.0 that “NoAction” actually does nothing, while "NoAction" becomes "Restrict" since v4.5.0. So I really need this "Nothing" feature.

Are you ok with your data getting invalid and you having orphan entries after? What is your motivation not to prevent such actions or removing the invalid valid after? (Just trying to understand)

Are you ok with your data getting invalid and you having orphan entries after? What is your motivation not to prevent such actions or removing the invalid valid after? (Just trying to understand)

For example, I have user and post tables. A user will post many posts. But there might be some case where a post is coming to the database while its owner hasn't been inserted into. Because the data was scrape from the internet, blockchain, random other places. I have no control or idea what data would come or lose first actually. So when inserting, updating, deleting, I need to only update the row itself, not to influence other tables. But when query, they do have relations, which needs the usage of relationMode="prisma".
I think this scene is pretty common? Orphan entries are totally acceptable. ORM doesn't need to assume that we must update or delete the related table's row if one row is changed.

Hi Team. May I know the status of this issue? This prevents me updating from 4.4.0 to enjoy the latest features ever (will get error 2014 "The change you are trying to make would violate the required relation 'ChildToParent' between the Child and Parent models."). Thank you.

It is not on the top of our list and hence not prioritized. We can not say if or when this will be implemented.

Are you ok with your data getting invalid and you having orphan entries after? What is your motivation not to prevent such actions or removing the invalid valid after? (Just trying to understand)

Yes, in many cases it's perfectly fine to rely on your transactions to be implemented correctly and application ensuring the referential integrity. In heavy load systems it is sometimes crucial to turn off any strong integrity constraints in order to scale.
Currently the only escape hatch is raw queries, and that becomes cumbersome for complex relationships very quickly.

This issue would probably be closed by https://github.com/prisma/prisma-engines/pull/4116


ci: use delvewheel for Windows wheels

Fixes #99.

Sample run: https://github.com/lidavidm/arrow-adbc/actions/runs/3526025766


Initial store tab

This is an initial implementation of the store
Depends on #169
TODO:

[ ] refreshing after connecting a watch/disconnecting a watch, so the url is right
[ ] handle file:// urls in android code

@hellcp Is this done? Otherwise, can you mark it as draft please?


How to compile on a computer with less than 4G memory

my laptop  is very old , Does not support replacing and adding memory
How to compile on a computer with less than 4G memory ？？？

/question


Add PandasDataFrameEqualityComparator

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (c78803e) 100.00% compared to head (0675a8a) 100.00%.

:exclamation: Current head 0675a8a differs from pull request most recent head 6277954. Consider uploading reports for the commit 6277954 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #398   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           65        65
  Lines         2306      2317   +11
  Branches       390       390
=========================================
+ Hits          2306      2317   +11

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


v5.2.0

Updated the minor version number due to the addition of icons.

We need to merge https://github.com/mui-org/material-ui/pull/29711 first


New version: IBM.Semeru.22.JRE version <IP_ADDRESS>

Created by 🥟 Dumplings.
Microsoft Reviewers: Open in CodeFlow

Verify:


 Make sure to specify a main property :+1:. Thanks.

i just released v0.9 which has a bower.json and registered url-pattern with bower.

Thanks :+1:


Customise Port forwarding UX

an issue has been opened regarding Customise Port forwarding UX to allow user to choose the port number.
https://github.com/rancher/dashboard/issues/6130

does this issue concern the 9080/9443 default port usage for the dashboard (and its lack of customization ability)?


Anuran Species (frog draft)

Anuran species draft. Adds a frog/toad race to ss14.  I don't see what's wrong with this. Especially since it's still in draft.

Needs a design doc backing it if someone gets time.


Fix SchedulerTypeTime distance(to:)

The new versions of RunLoop and OperationQueue (copied from the Apple implementations) have incorrect implementations of SchedulerTimeType::distance(to:).
The original Apple implementations rely on Date::distance(to:), which is only available on the 2019 and later systems. So CombineX replaces the use of Date::distance(to:) with Date::timeIntervalSince(_:).
The problem is that the calls to timeIntervalSince have date and other.date in the wrong places, so each call to distance(to:) returns a Stride with a flipped sign.
This pull request contains two commits.
The first commit adds a test case demonstrating the incorrect behavior. The test case succeeds when compiled with Combine and fails when compiled with CombineX. The test case is a little tricky because I was unable to make it compile with Combine using the RunLoop.CX.SchedulerTimeType type. I worked around it with type inference, so that it uses RunLoop.SchedulerTimeType when compiled with Combine and RunLoop.CX.SchedulerTimeType when compiled with CombineX.
The second commit fixes the two implementations of distance(to:).

I noticed how the other tests (in the same file, duh) were using CXWrappers and used it to abstract away the type names in the new test.

Looks great! I just fixed falling tests irrelevant to this pr. Would you mind rebase and run checks again?

@mayoff Hi, I noticed the CXShim solution is rather complicated for people new to CombineX. So I wrote a migration guide. Feedbacks are welcomed!


wlcs: 1.6.1 -> 1.7.0

Description of changes
https://github.com/MirServer/wlcs/releases/tag/v1.7.0
Tested building mir locally, worked fine.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@OfBorg build mir mir.passthru.tests

GCC 13 bump broke wlcs and mir. wlcs bump fixes wlcs, mir fix is included so biggest / only packaged wlcs user can be tested.


Lack of test on Travis for PHP 7.4 and tests don't run on upcoming PHP versions

Describe the bug
For now Travis configuration have not set test for PHP 7.4. Other problem is that tests on upcoming PHP version won't run.
CodeIgniter 4 version
RC2.1
Affected module(s)
none

This both an issue and a PR? We don't need both.
Why do we need to test PHP7.4 & nightly? The build takes a while as it is, and it is likely to be some time before we look at taking advantage of them.

For now tests are take for nightly only. I assume that initially at target to PHP 7.4. Now nightly is future version of PHP 8.
Other thing is that now tests for version > 7.3 do not run at all, because composer can not install all packages because of dependencies.
Because PHP7.4 will be released in about month, I think that it's reasonable to make test for this version and prepare to this.

We were just discussing this one, actually. The matrix is too unweildy to deal with if we don't have to. I think dropping nightly for now and keeping 7.4 as an allowed_failure is workable.
According to a comment on this post over at Travis, we can use 7.4snapshot might allow us to get that working.

I update my PR to work with PHP 7.4 (no snapshot) as it's avaliable right now on Travis.
@lonnieezell I wonder if there is a way to change tests to run all databases specified in some file on one pass. That way we will not need to run one travis instance for each driver. As @jim-parry point out, now test take a long time. When there will be more database drivers and more PHP versions to test, there will be easier to test all drivers in one run. In summary that approach should take less time.

7.4 is working from #2514 - a larger pass at the testing service and matrix framework would probably be involved in a refactor.


GSSoC: Design Profile Page

Currently there is no profile page on view profile. I would like to work in this issue under gssoc.

Prerequisites: #24 before claiming any issue.

Hi, I am interested in contributing to this project. Could you please assign this issue to me? Thank you!

I have completed it in tailwind , please review it

I have completed it in tailwind , please review it

Please mention your PR while commenting. This will help us to review your PR.

#81 profile page creation

can you assign this issue to me As I learned MERN stack development I implement various tech stacks using the tailwind material ui etx can you assign this iisue to me under gssoc'24 let me know what fields are required in profile page

@Swarnendu0123 can i work on this issue

@RamakrushnaBiswal go for it.

its good @Swarnendu0123

any discord channel to Communicate


[Trashbin] restoring a file in existing location gives incorrect status code

Describe the bug
when restoring a file to an existing location should overwrite existing file and give status code 204 - No content (this is what we get in owncloud 10) but gives 201
Steps to reproduce
Steps to reproduce the behavior:

Create a file lorem.txt
Delete lorem.txt
create another file lorem.txt
From traashbin restore original lorem.txt

Expected behavior
file is overwritten with status code 204
Actual behavior
file should be overwritten with status code 201
Setup
Please describe how you started the server and provide a list of relevant environment variables.

OCIS_VERSION=git
BRANCH=master
STORAGE_FRONTEND_UPLOAD_DISABLE_TUS=false

Additional context
Add any other context about the problem here.

Update: status code is as expected 204 and file also exists for the owner!

re-tested with local server oCIS version 1.11.0 and oC10 in docker(v.10.8)
Restore file in Oc10:  MOVE http://host.docker.internal:8080/remote.php/dav/trash-bin/admin/2133
and in oCIS: MOVE https://localhost:9200/remote.php/dav/trash-bin/admin/0316ab63-4076-4d51-bee8-ce2f18d69d0e
have same behavior
actual behavior: file is not overwritten. status code 412 Precondition failed
with response:
<d:error xmlns:d="DAV:" xmlns:s="http://sabredav.org/ns">   <s:exception>Sabre\DAV\Exception\PreconditionFailed</s:exception>   <s:message>The destination node already exists, and the overwrite header is set to false</s:message>   <s:header>Overwrite</s:header> </d:error>

@dpakach do you sure that file should be overwrite?

seems like the issue is fixed so closing it.


Train on custom dataset

How do I train it on a custom dataset?

Why not rewrite the data loader yourself?


The command "C:\WinObjC\msvc..\bin\Xib2Nib.exe" exited with code -1073741819

Hi all,
Having an issue trying to convert the Mattermost iOS app over to UWP through Islandwood. Getting the following error:
MSB3073 The command ""C:\WinObjC\msvc\..\bin\Xib2Nib.exe" "..\..\Mattermost\Base.lproj\Main.storyboard" "Debug\Base.lproj\Main.storyboardc"" exited with code -1073741819.
Not being an iOS developer by trade, I've not got a clue how I would even start going about resolving this. Can anyone point me in the right direction?
Thanks!
Curtis

The issue appears to have happened while copying files to output directory, clear the output directory or change permissions on the file and it should be good to go.


Broken link in the CONTRIBUTING.md

Link to the  "Becoming a contributor" blog post refers to the private blog. It couldn't be viewed without author's permission.

Fixed, thanks


eth/021: Holocene Protocol Versions task

Questions:

Looks like way back we used to do two of these tasks (one for recommended, one for required). Then we switched to just doing them at the same time. Are we still happy to do that?

I encourage the reviewer to perform a mutation test -- clear out the transactions from input.json and check that the validation fails.
TODO

[x] factor out a common validation contract for this to make it easier in future (build on Seb's new framework)

If we can add the CI check for this task using the same as Seb for Sepolia would be amazing (https://github.com/ethereum-optimism/superchain-ops/pull/421/files#diff-78a8a19706dbd2a4425dd72bdab0502ed7a2cef16365ab7030a5a0588927bf47R245)


Handle optional properties when flatten an alias

When there are optional properties in an alias, should flatten these optional properties as nuallable parameters.
e.g.
alias ThingAliasWithOptionalProps = {
  @path id: string;
  @doc("name of the Thing")
  name: string;
  @doc("optional property of the Thing")
  color?: string;
  @doc("age of the Thing")
  age?: int32;
  @header top: int32;
};
op spreadAliasWithOptionalProps( ...ThingAliasWithOptionalProps ): void;

The expected generated sdk
public virtual Response SpreadAliasWithOptionalProps(string id, int top, string name, string color, int? age, CancellationToken cancellationToken = default)

I believe optional properties should be flattened into optional parameters in the method signature. For instance, the cadl
alias ThingAliasWithOptionalProps = {
  @path id: string;
  @doc("name of the Thing")
  name: string;
  @doc("optional property of the Thing")
  color?: string;
  @doc("age of the Thing")
  age?: int32;
  @header top: int32;
};
op spreadAliasWithOptionalProps( ...ThingAliasWithOptionalProps ): void;

should be generated as
public virtual Response SpreadAliasWithOptionalProps(string id, int top, string name, string color = null, int? age = null, CancellationToken cancellationToken = default)

So that the users will not required to pass in a null value for those optional properties. But this might introduce issues around overloads of methods, or issues of breaking changes when new properties are added into the alias because we will have to sort the parameters


Update README.MD

The previous change of folding common events into the Event Tracing container makes sense. But for less intelligent folks like me, can be a non-intuitive way to find Security and other built in event log definitions. Suggest adding to README. Love this project btw.

Thank you very much @breakersall ! Makes sense. I like the additional details. Thank you for the feedback 🙏


Use dirname(@FILE) instead of Pkg.dir

This allows installing the package elsewhere.
Add testing against 0.5 to Travis - this runs the most
recent RC now, release once final tags are done.
Add CrossDecomposition to test/REQUIRE

Current coverage is 82.51% (diff: 100%)

No coverage report found for master at 623ec80.

Powered by Codecov. Last update 623ec80...6f41170


True logical layout

Describe the solution you'd like
Offer a third layout option, that lays out the pins grouped by their logical function as configured. So GPIO pins would be together (maybe inputs grouped and outputs grouped?), if I2C was used, those pins involved grouped together, UART, SPI etc etc...
Describe alternatives you've considered
The existing layouts
Additional context
For things like uart, it would also give a bit more vertical space and allow for nicer visualizations, alongside the group, instead of the space being scattered across pins.

Maybe the user can select a function for a group of pins all at once (e.g. SPI0) and then it changes them all en-masse.
We can then change the colors for pins to match the ALT function chosen, aligned with the colors in this doc
https://drive.google.com/file/d/14keVYmzA8FGI9p2b7R-4piGvmDs5MEX_/view
section 5.3


[backport] Remove the data member use_32bit_indexing from CArray

Backport of #5376 by @leofang

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit b2a50c8023d635e5a6bf638c0c9c37fbb5491256, target branch v9) succeeded!


One time key field allows for special char and more

Environment :
Pixel 3a, Android 10, v62
Scenario :

Navigate to the "Enter a one time key" screen
In the key field, enter special characters, even emojis, copy-paste content...

Expected :
The field should only allow alphanumeric characters
Issue :
The field currently allows for all types of content to be entered

was able to reproduce with iPhone XR.

Closing this - we have a PR in but need to handle some more cases.


Proposal: Unmarshal map keys into type

If I have a type that implements the json.Unmarshaler interface, it would be great if I could unmarshal map keys into that type.
Currently the below code will panic with panic: json: cannot unmarshal object into Go value of type map[main.Key]string.
type Key struct {
    value string
}

func (this *Key) UnmarshalJSON(data []byte) error {
    var s string
    err := json.Unmarshal(data, &s)
    if err != nil {
        return err
    }

    this.value = s
    return nil
}

func main() {
    d := `{"foo": "bar"}`

    var m map[Key]string
    err := json.Unmarshal([]byte(d), &m)
    if err != nil {
        panic(err)
    }

    fmt.Println(m)
}

You can already do that by implementing encoding.TextMarshaler / encoding.TextUnmarshaler

Oh ... thanks! Forget I ever said anything then. 👍


Fixed replicate() hook

replicate(): $args attribute was forgotten when calling next hook

Same fix as here: https://github.com/jarektkaczyk/eloquence/pull/187
But for 5.3 branch

Coverage remained the same at 93.594% when pulling 88fde3aa86cb493f239766c7e8a34a591ddd65c0 on uniconstructor:patch-2 into d058c6963204f42aef1830e60349268c3af77135 on jarektkaczyk:5.3.


Trouble with overwriting the aHrefSanitizationWhitelist

There was a previous issue about this at https://github.com/danialfarid/ng-file-upload/issues/1014 which was "fixed", seemingly by having the relevant lines commented out in https://github.com/danialfarid/ng-file-upload/commit/df94b03681fcce413d67fd5439ff045dbba9650d, but then they were un-commented in https://github.com/danialfarid/ng-file-upload/commit/9f5c4697bc5fa6b3f84bcfd3292b00053615e679 and now the same issue remains.
In our case, we use webcal:// links in the same app as ngFileUpload, so our own config to add "webcal" to the whitelist was being overriden by ngFileUpload's, which was being run afterward because it comes later in the dependency list.
Our current solution to this, which is not pretty, is the following module:
// ng-file-upload overrides this and does not include "webcal", so to get our webcal links to work on pages
// that use ng-file-upload, we need to use their pattern (https://github.com/danialfarid/ng-file-upload/blob/master/src/data-url.js)
// and add "webcal" to it. Furthermore, this module must be included in the dependency list AFTER the last module
// that depends on ngFileUpload so that this config function runs last.
angular.module('urlSanitizationOverride', []).config(['$compileProvider', function($compileProvider){
    $compileProvider.aHrefSanitizationWhitelist(/^\s*(https?|ftp|mailto|tel|local|file|data|blob|webcal):/);
}]);

Is there a better solution here? Perhaps some user-facing ngFileUpload config/setting that takes extra URL prefixes and adds them to the whitelist for you?

Should be fixed at 12.1.0


add Members with MD5 filter for Member list

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (03d7796) 75.91% compared to head (b538e20) 75.72%.

:exclamation: Current head b538e20 differs from pull request most recent head 51779c2. Consider uploading reports for the commit 51779c2 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@               Coverage Diff                @@
##           prep-release     #122      +/-   ##
================================================
- Coverage         75.91%   75.72%   -0.20%
================================================
  Files                46       46
  Lines              1557     1557
================================================
- Hits               1182     1179       -3
- Misses              375      378       +3

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


update containerd (v1.5.5), slirp4netns (v1.1.12), fuse-overlayfs (v1.7), RootlessKit (v0.14.3)

Seems regression in RootlessKit v0.14.3
$ go test -v -run TestPush ./cmd/nerdctl
test target: "nerdctl"
=== RUN   TestPushPlainHTTPFails
    push_test.go:78: hostIP="<IP_ADDRESS>", listenIP="<IP_ADDRESS>", listenPort=5000
    push_test.go:194: testImageRef="<IP_ADDRESS>:5000/test-push-plain-http-fails:3.13-org"
    push_test.go:199: result: exitCode=1, out="index-sha256:841830f4e8b16e3bfec87ffc70854211399bbe4324b4e109944f8ec3716af0e7:    waiting        |\x1b[32m\x1b[0m--------------------------------------| \nmanifest-sha256:e103c1b4bf019dc290bcc7aca538dc2bf7a9d0fc836e186f5fa34945c5168310: waiting        |\x1b[32m\x1b[0m--------------------------------------| \nlayer-sha256:ca3cd42a7c9525f6ce3d64c1a70982613a8235f0cc057ec9244052921853ef15:    waiting        |\x1b[32m\x1b[0m--------------------------------------| \nconfig-sha256:49f356fa4513676c5e22e3a8404aad6c7262cc7aaed15341458265320786c58c:   waiting        |\x1b[32m\x1b[0m--------------------------------------| \nelapsed: 0.1 s                                                                    total:   0.0 B (0.0 B/s)                                         \ntime=\"2021-08-04T15:04:39+09:00\" level=info msg=\"pushing as a single-platform image (application/vnd.docker.distribution.manifest.list.v2+json, sha256:841830f4e8b16e3bfec87ffc70854211399bbe4324b4e109944f8ec3716af0e7)\"\ntime=\"2021-08-04T15:04:39+09:00\" level=error msg=\"server \\\"<IP_ADDRESS>:5000\\\" does not seem to support HTTPS\" error=\"failed to do request: Head \\\"https://<IP_ADDRESS>:5000/v2/test-push-plain-http-fails/blobs/sha256:ca3cd42a7c9525f6ce3d64c1a70982613a8235f0cc057ec9244052921853ef15\\\": http: server gave HTTP response to HTTPS client\"\ntime=\"2021-08-04T15:04:39+09:00\" level=info msg=\"Hint: you may want to try --insecure-registry to allow plain HTTP (if you are in a trusted network)\"\ntime=\"2021-08-04T15:04:39+09:00\" level=fatal msg=\"failed to do request: Head \\\"https://<IP_ADDRESS>:5000/v2/test-push-plain-http-fails/blobs/sha256:ca3cd42a7c9525f6ce3d64c1a70982613a8235f0cc057ec9244052921853ef15\\\": http: server gave HTTP response to HTTPS client\"\n"
--- PASS: TestPushPlainHTTPFails (7.67s)
=== RUN   TestPushPlainHTTPLocalhost
    push_test.go:78: hostIP="<IP_ADDRESS>", listenIP="<IP_ADDRESS>", listenPort=5000
    push_test.go:86: assertion failed: expression is false: res.ExitCode == exitCode: time="2021-08-04T15:04:45+09:00" level=fatal msg="failed to create shim: OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: Running hook #0:: error running hook: exit status 1, stdout: , stderr: time=\"2021-08-04T15:04:45+09:00\" level=fatal msg=\"listen tcp <IP_ADDRESS>:5000: bind: address already in use\"\nFailed to write to log, write /home/suda/.local/share/nerdctl/1935db59/containers/nerdctl-test/a313f205f23c1e445dec0a3d6f38e631109687452c153e58458080e17864933d/oci-hook.createRuntime.log: file already closed: unknown"

--- FAIL: TestPushPlainHTTPLocalhost (0.68s)
=== RUN   TestPushPlainHTTPInsecure
    push_test.go:78: hostIP="<IP_ADDRESS>", listenIP="<IP_ADDRESS>", listenPort=5000
    push_test.go:86: assertion failed: expression is false: res.ExitCode == exitCode: time="2021-08-04T15:04:46+09:00" level=fatal msg="failed to create shim: OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: Running hook #0:: error running hook: exit status 1, stdout: , stderr: time=\"2021-08-04T15:04:46+09:00\" level=fatal msg=\"listen tcp <IP_ADDRESS>:5000: bind: address already in use\"\nFailed to write to log, write /home/suda/.local/share/nerdctl/1935db59/containers/nerdctl-test/b2be38fe0c6fdfe3dbc60a3571d593a0b26722c776339e5170e9159fa4ac3f25/oci-hook.createRuntime.log: file already closed: unknown"

--- FAIL: TestPushPlainHTTPInsecure (0.66s)
=== RUN   TestPushInsecureWithLogin
    push_test.go:106: hostIP="<IP_ADDRESS>", listenIP="<IP_ADDRESS>", listenPort=5000, authPort=5001
    push_test.go:166: assertion failed: expression is false: res.ExitCode == exitCode: time="2021-08-04T15:04:47+09:00" level=fatal msg="failed to create shim: OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: Running hook #0:: error running hook: exit status 1, stdout: , stderr: time=\"2021-08-04T15:04:47+09:00\" level=fatal msg=\"listen tcp <IP_ADDRESS>:5000: bind: address already in use\"\nFailed to write to log, write /home/suda/.local/share/nerdctl/1935db59/containers/nerdctl-test/6e0cdc73273e9153ab6d227574d3732bdd1af5770b4fbad7fccef24bfeedce8a/oci-hook.createRuntime.log: file already closed: unknown"

--- FAIL: TestPushInsecureWithLogin (1.51s)
FAIL
FAIL    github.com/containerd/nerdctl/cmd/nerdctl       10.543s
FAIL

Updated RootlessKit to v0.14.4

LGTM


[Secure] Local Image Execution

Is there any way to run local container images in secure mode if DIGEST of the image is <none>?

@t25kim In secure mode, we cannot run unknown containers (without a DIGEST)

@t25kim In secure mode, we cannot run unknown containers (without a DIGEST)

@tdrozdovsky Thank you for the prompt answer!
Do you think HomeEdge doesn't need to run local images for the developers?
I think it might be good for developers to have the method to enable or disable the verifier in secure mode.

@tdrozdovsky Thank you for the prompt answer!
Do you think HomeEdge doesn't need to run local images for the developers?
I think it might be good for developers to have the method to enable or disable the verifier in secure mode.

There is a normal mode (without security) for developers. We cannot allow containers to run without a DIGEST. But in the future, when we add a digital signature of containerWhiteList, we can consider this option.

There is a normal mode (without security) for developers. We cannot allow containers to run without a DIGEST. But in the future, when we add a digital signature of containerWhiteList, we can consider this option.

@tdrozdovsky I thought there might be a developer wants to test his dev container in local PC with HTTPS and RBAC. Just my 2 cents. 😁

@tdrozdovsky I thought there might be a developer wants to test his dev container in local PC with HTTPS and RBAC. Just my 2 cents. 😁


CMake components

PR Summary
Rewrites the CMake targets and installation to split up Interface and Library components.
Closes #323

PR Checklist

[ ] Adds a test for any bugs fixed. Adds tests for new features.
[ ] Format your changes by using the make format command after configuring with cmake.
[ ] Document any new features, update documentation for changes made.
[ ] Make sure the copyright notice on any files you modified is up to date.
[ ] After creating a pull request, note it in the CHANGELOG.md file.
[ ] LANL employees: make sure tests pass both on the github CI and on the Darwin CI

If preparing for a new release, in addition please check the following:

[ ] Update the version in cmake.
[ ] Move the changes in the CHANGELOG.md file under a new header for the new release, and reset the categories.

@dholladay00 looks like https://github.com/lanl/singularity-eos/blob/main/test/test_eos_modifiers.cpp is broken, since SINGULARITY_BUILD_CLOSURE was never passed on to the test code and therefore always ignored.

@mauneyc-LANL the CI fix was actually quite simple. All I needed was avoid the inclusion of the closure header.

@mauneyc-LANL the CI fix was actually quite simple. All I needed was avoid the inclusion of the closure header.

Looked simple, I will review once passes

@jhp-lanl @Yurlungur tested this in the SAP integration branch.

@rbberger a conflict popped up, I think with a change @jonahm-LANL made last week.


Drop python 2 support

Issue: #390

Remove support python 2 from the docs, requirements.txt, travis-ci, setup.py, tox.ini
Replace unicode with str
Remove dbfpy folder and rename dbfpy3 to dbfpy
Remove compat file and remove python2 packages from dependency

Thanks for your review.
I fixed these issues and some other linting errors and updated my pr

@peymanslh I'm sorry that recent commits broke your PR. Are you available to rebase it? Otherwise I can do it too.

The big changes (from https://github.com/jazzband/tablib/pull/398) are to move tablib into a src directory, and test_tablib.py into a tests dir.

@claudep @hugovk
Yes, I'm working on it.

Many thanks for your work, I just pushed the main part in master in b3c7145c407 (outside of this PR) and will finish by cherry-picking the linting fixes in a separate commit.

Done in bdc84255a83. Great work!


fixed table inconsistency

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/8080

Dear @JohanFreelancer9,
kindly do the copy editor review for this PR.
Dear @e0i, for your information
Thank you


Work thru goreleaser deprecation warnings

Bottom of https://github.com/stateful/runme/actions/runs/6249835863
Also the json schema balks at:

If no one else is looking at this yet I might take a crack at it tonight. I recently did a little work with github actions and go releaser for my terraform provider so it's fresh in my mind.

Be my guest, please!

@sourishkrout  well I got to where I can run the release on tags! https://github.com/catdevman/runme/actions/runs/6322034405
I was wondering what vscode extension you're using? Can we work on making a list of all the things that need updated.

[ ] goreleaser version
[ ] nfpms.replacements deprecated https://goreleaser.com/deprecations/#nfpmsreplacements
[ ] archives.rlcp deprecated https://goreleaser.com/deprecations/#archivesrlcp
[ ] archives.replacements deprecated https://goreleaser.com/deprecations/#archivesreplacements
[ ] set-output deprecated https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/ (this should be fixed with the goreleaser version from what I could tell)

Anything else?


forge-mtg: 1.6.53 -> 1.6.56

Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@eigengrau Sorry if this is rude; I'm new here. This package is causing issues for a Maven update (#229144) and I'm kind of hoping that a bump here might just solve those issues. Or alternatively, I'll probably need to mess around with the patches, which will cause conflicts. It would be great if this could be merged soon.


fix: error handing for row cursors

(*sql.Rows).Next() can return false due to an error. The error must be handled by checking (*sql.Rows).Err().
The transaction must be ended by commit or rollback to close the db connection. Calling rollback after commit is noop.
Don't call a query in the middle of cursor, as it may lead to a deadlock. The inner query may wait for a free connection, and the connection from the cursor can't be freed because it's stuck by the inner query waiting.

Oh, that's a good catch there ! Thank you!
I think @ccoVeille is right, all the errors are not being handled.


feat(rh-label): add

What I did

Created rh-label based on BaseLabel from pfe
Closes #398

Todo:

[ ]  integrate rh-icon when ready

Testing Instructions

npm start
npm run watch:docs
View deploy preview

Notes to Reviewers

i'm concerned about the use of the word label here.
Can we make a FACE that acts like <label>?
<rh-label for="switch">AT reads me as label for switch</rh-label>
<rh-switch id="switch"></rh-switch>

Until we're sure we can do that (perhaps with cross-root aria and element internals?) I think we should consider renaming this element.
cc @markcaron @hellogreg @nikkimk

i'm concerned about the use of the word label here.

rh-tag maybe to remove the confusion with a potential FACE element?
I'm trying to figure out why we'd need to build rh-label as a FACE element that mimics the standard <label> tag, but enforcement of styles maybe enough of a reason.

why we'd need to build rh-label as a FACE element that mimics the standard <label> tag

label means something specific on the web. it's critical that we preserve the semantic meaning of label as "element or attribute containing text which describes a control or other element".

@coreyvickery when you have a chance could you weigh in here?  We would need to revise the naming in the XDs and future docs on ux. so you have the veto power here on this.  We would like to resolve this as soon as we can so can move forward with this PR as it's been sitting for a while now.
The plan is rh-tag (or whatever we name this component) would still extend pfe-label, opening up rh-label namespace for a future new component that would be a Form Associated Custom Element, using element internals that more resembles the <label> HTML tag and use case.

@zeroedin Fine with that, as long as rh-tag can incorporate the styles seen in the link at the top.


Sorting tabs in most recently used order

Hi, I saw a closed issue for sorting tabs in most recently used order. https://github.com/microsoft/vscode/issues/113717. It looks like it was closed without any upvotes. I am restarting the same feature request. It's very frustrating to organize the tabs - instead it should just get arranged as a stack automatically.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

I am interested in working on this issue! Can I be assigned to this once it becomes backlogged/approved?

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!

I also suffered from not having this open editor auto sort feature, there's also another issue requesting 83692 this feature. Can you reconsider this?


add rules to correlations for correlation engine

Description
add rules to correlations for correlation engine
Issues Resolved
[List any issues this PR will resolve]
Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #423 (7e06e73) into main (2bdba83) will decrease coverage by 0.17%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #423      +/-   ##
============================================
- Coverage     28.35%   28.18%   -0.17%
+ Complexity      897      893       -4
============================================
  Files           231      231
  Lines          9280     9318      +38
  Branches       1071     1073       +2
============================================
- Hits           2631     2626       -5
- Misses         6413     6454      +41
- Partials        236      238       +2

Impacted Files
Coverage Δ

...arch/securityanalytics/correlation/JoinEngine.java
0.00% <0.00%> (ø)

...yanalytics/correlation/VectorEmbeddingsEngine.java
0.00% <0.00%> (ø)

...rch/securityanalytics/model/CorrelatedFinding.java
0.00% <0.00%> (ø)

...arch/securityanalytics/model/FindingWithScore.java
0.00% <0.00%> (ø)

...ics/transport/TransportCorrelateFindingAction.java
0.00% <0.00%> (ø)

...tics/transport/TransportListCorrelationAction.java
0.00% <0.00%> (ø)

...cs/transport/TransportSearchCorrelationAction.java
0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes


how to install? does it download every kiwify course?

i think the title its enough haha but i dont know how to use the codes/scripts/extentions and etc

Hi
You'll need ffmpeg and Nim in your machine. So you can install this app with:
nimble install kiwifyDownload

Now go to an lesson of your course with opened dev tools at network tab, locate the XHR request that returned the course data and put it's response into a file, such data.json. Generally it's the request with the larger response body.
Then run the app using:
kiwifyDownload -j data.json -o outDir


docs: integrations reference updates 5

Added missed references. Added missed provider pages.

@ccurme Can you review, please?

@ccurme Can you review, please?

@ccurme Thanks for review!


Vulkan: Shader compilation stutter when materials enter the view frustum for the first time (unless cached)

Bugsquad note: This issue has been confirmed several times already. No need to confirm it further.

Godot version
4.0.alpha8
System information
Fedora 36, GeForce GTX 1080 (NVIDIA 510.68.02)
Issue description
A noticeable shader compilation stutter occurs when materials enter the view frustum for the first time (unless cached). This is more visible on lower-end CPUs/GPUs which take a longer time to compile shaders.
Labeling regression, as ubershaders make it possible to avoid the initial stutter (on cold cache) in 3.5 with GLES3.
Steps to reproduce

Open the minimal reproduction project. If you can't notice shader compilation stuttering in that project, try https://github.com/godotengine/godot-demo-projects/pull/743 instead.
Run the project and wait for stuttering to occur due to shader compilation.
Run the project a second time and notice there won't be stuttering anymore.
Remove the shader cache folder (.godot/shader_cache/ within the project folder).
Run the project again, and notice stuttering occurring once again.

Minimal reproduction project
test_shader_compilation_stutter.zip

GD4 Beta4 still stutters quite a bit on load with a lot of assets. You need to force precompilation of shaders.
https://github.com/godotengine/godot/pull/53411#issuecomment-935537126
https://github.com/godotengine/godot/pull/46330#issuecomment-786280648

what now?

I think adding a true asynchronous shader compilation mode is the way to go – essentially mimicking dxvk-async behavior. It causes objects to be temporarily invisible until their shaders are compiled, but this is far preferable to stutter in 99% of situations.

Hopefully we can do most of it in time for the stable release.

Since VkGraphicsLibrary extension is available almost with every single gpu vendor, could that solution be considered for stuttering problem 🤔

Would be interesting to try and implement it since I've been doing multiple personal vulkan projects and I'd like to put it to use.

The problem is still present in 4.1 (Mobile at least).

Still an issue on desktop as well.  Able to reproduce by clearing the cache shaders in my project file and NVIDIA cache folder, then restarting the game.  Only happens the first time.

Is it not possible for Godot 4.x to use the asynchronous compilation technique 3.5 does? I'm having huge stutters whenever I turn a light on or add a light to the scene (even when using preload()) and I'm not sure what the current workaround is.

I don’t think that adding a light to the scene causes a pipeline compilation🤔 though someone more knowledgeable can correct me. Does the stutter happen every single time when you add a light?

I don’t think that adding a light to the scene causes a pipeline compilation🤔 though someone more knowledgeable can correct me. Does the stutter happen every single time when you add a light?

It happens the first time a new light is added or turned on, but then it's fine each time you do it after (as if it's become cached).

It is cached. Pipeline caching was introduced in 4.1 on top of shader caching. You can find it in user://vulkan folder. Deleting that folder will most likely trigger the stutter again unless your gpu driver has made its own cache.

I can retrigger the stutter by deleting specific cache files in my graphics card settings folders.  This resets the process of re-caching.  When a new shader or post processing effect is enables it happens then, too.  When the game is launched on a version update, it happens though not as severe. Would need to test further to provide feedback there.
I look longingly at Steam's  "precompile vulkan shaders" message which comes up when I launch certain games through that platform.  I'm wondering if there's something to be gleaned from that approach.

It is cached. Pipeline caching was introduced in 4.1 on top of shader caching. You can find it in user://vulkan folder. Deleting that folder will most likely trigger the stutter again unless your gpu driver has made its own cache.

Are lights considered shaders to be cached, though? I'm only having this issue with lights, I'm not even having trouble with changing materials it just seems to happen when lights are enabled or added to the scene enabled.

I look longingly at Steam's "precompile vulkan shaders" message which comes up when I launch certain games through that platform. I'm wondering if there's something to be gleaned from that approach.

This works by making users compile and upload shaders after they're done playing a game. It works because Steam has such a large network of users that you're almost guaranteed to be able to download a compiled shader that matches your OS, GPU and driver version (all 3 need to match exactly).
It is possible for Godot games on Steam to use this approach if they use a Vulkan-based rendering method.

Geaphics pipeline library could help with invisible meshes in async compilation by creating a fast linked pipeline first before doing final optimized version and by recycling pipeline stages as much as possible. But it would require first to have async compilation infrastructure in place if we want to take advantage of that extension.

At least that’s my understanding that I got after writing GPL draft pr.

Here's a clip someone sent me of them playing a game I'm beta testing through Steam (so it's built and run via Steam), you can see a huge lag when the flashlight goes on the first time and not subsequent times (the flashlight is already added to the scene on ready, this is enabling it):
https://github.com/godotengine/godot/assets/16231628/89dcd418-b0bc-45a8-a9c9-7b02844d1773
Here's another when a light is removed from the scene, and one after when a light behind the wall is instatiated (it's not seen in this clip):
https://github.com/godotengine/godot/assets/16231628/8f1bee39-1a5c-41c6-96af-3b0b792f3c67
This seems like an issue with compilation stutter, is this expected behavior with lights being added to the scene/enabled?

Shaders are compiled per lighting setup, or at least they were in gd3. The only reliable way we've found to reduce stutter is force precompilation. I linked instructions in my comment above.

Shaders are compiled per lighting setup, or at least they were in gd3. The only reliable way we've found to reduce stutter is force precompilation. I linked instructions in my comment above.

Essentially the 4.x version of this? https://github.com/Brandt-J/ShaderPrecompiler
I think for the time being a built-in node or even an addon like this would be a good immediate solution so people can avoid as much stutter as possible out-of-the-box (as you mentioned in your comment, 99% of the stuttering https://github.com/godotengine/godot/pull/46330#issuecomment-786280648)

I forked Brandt-J's to 4.x: https://github.com/elvisish/ShaderPrecompiler4x
It seems to work okay from what I can tell, but this should hopefully serve as a bandaid until something can be implemented in 4.x.


Add NFTLockBox contract

This PR introduces the NFTLockBox contract, formerly known as NFTAirDrop.
Please see the comments in NFTLockBox.cdc for an explanation of how the contract works.

Would it be possible to make the transactions so that they can be generic and not use the placeholders for the contract names?

@joshuahannan Unfortunately this isn't possible with the current claim_nft template because it has a block to initialize a collection (if one doesn't exist), and that requires a reference to the actual contract.
However, maybe the LockBox could expose a createEmptyCollection function that is configured to return the correct collection type required to claim. Do you think that would make sense?

@psiemens Actually, if the NFT contract has implemented the NFTCollectionData view implemented, then you don't need to import that contract because that view includes the createEmptyCollection method. So you could just use that view

Actually, if the NFT contract has implemented the NFTCollectionData view implemented, then you don't need to import that contract because that view includes the createEmptyCollection method. So you could just use that view

@joshuahannan Oh I would love to just do this. Is it valid for a NonFungibleToken.Collection to be a MetadataViews.Resolver (in addition to the NFT resource)? This would allow a LockBox to fetch the NFTCollectionData view from a collection. I know we talked about this in the past but not sure if it's been done yet.
Otherwise the caller will need a reference to an NFT from the collection, which is tricky.

@psiemens The collection has a resolver interface also that can be used instead of getting the NFT

@joshuahannan @satyamakgec As it stands now, I still need a reference to the actual contract in order to link a new collection. This was flagged in this issue: https://github.com/onflow/cadence/issues/1617
In particular it's the ExampleNFT.ExampleNFTCollectionPublic that I can't link. While it is possible to exclude that capability, I'm not confident that we can do that (yet) without breaking things for NFT projects that may depend on the concrete type in the future.
As such, I think it may make sense to leave out the NFTCollectionData view for the first version of this contract. Do you think that's valid?

Even if we can't use it completely now, that doesn't mean we should get rid of it. maybe just leave it in there but don't actually use it. then when we need to upgrade, it'll make that process easier

@joshuahannan Good point.
I'm thinking of adding it to the LockBox resource like this:
pub resource LockBox: LockBoxPublic {
  pub let collectionData: MetadataViews.NFTCollectionData

  // ...
}

The creator of the lock box then would resolve NFTCollectionData when creating the lock box (or set it manually).
The other option I'm considering is this (need to figure out the exact syntax):
pub resource LockBox: LockBoxPublic {
  pub let collectionData: MetadataViews.NFTCollectionData

  pub fun borrowResolverCollection(): MetadataViews.ResolverCollection? {
    if let collection =  self.collection.borrow() {
      return collection as? MetadataViews.ResolverCollection
    }

    return nil
  }
  // ...
}

Which do you think is better?

probably better to have it resolve the view on its own instead of storing the NFTCollectionData view, in case what the contract returns for the view changes at some point

@joshuahannan That makes sense.
I added a borrowCollection method here: https://github.com/packagelabs/freshmint/pull/81/commits/96293b547830c73096314e67cd1e43e483c511b9
LMK what you think!


⚠ GitLab Server has degraded performance

In 1b12238, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6533 ms

Resolved: GitLab Server performance has improved in b40ba10.


Improve CPU prover setup documentation

What ❔
This PR expands the documentation on how to run the stack with the CPU prover, also adding a small script that will automatically generate the setup data needed so it's more streamlined.
Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.
[x] Code has been formatted via zk fmt and zk lint.

LGTM - pinging @akash-chandrakar to PTAL on the changes to the witness binary.


Font Not Working on Windows

Hey there. I am trying to switch my default windows emoji font to the IOS emojis. But the font doesn't to seem to be supported by windows.

Hey @404Apollo, unfortunately that’s not supported right now. I do plan to make it work across all systems eventually, but I just don’t have the time right now (at least for the next few months).
If anyone’s interested in jumping in, I’d really appreciate the help!

Haven't tested it out, but I saw an abandoned fork that claims Windows compatibility...maybe the diff of changes would help? https://github.com/samuelngs/apple-emoji-linux/compare/ios-15.4...zhdsmy:apple-emoji:ios-15.4

i used the prebuilt font file attached in that person's repo, renamed it to Segoe UI Emoji using Typograf, and followed these instructions and it kinda worked! when i use the emoji viewer (Win + Period) they show up as apple emojis, albeit a bit pixelated. some things aren't working, like the github reaction emojis, but for me it's not a huge deal. but their fix is at least partially functioning and as of now not crashing my computer, so it would be awesome if this repo was updated too, esp with the latest emojis!


Feature request: Health check function call

Would it be possible for mail-time to create a health check function call or an isReady flag (for use on a /health endpoint).
This would allow us to run mail-time as a microservice and monitor its availability.
In our instance, we would use it on the AWS ECS infrastructure, but this nomenclature seems to be prevalent on other platforms eg.

https://docs.aws.amazon.com/Route53/latest/DeveloperGuide/monitoring-health-checks.html
https://learn.microsoft.com/en-us/azure/app-service/monitor-instances-health-check
https://cloud.google.com/load-balancing/docs/health-check-concepts

@justinlawrence thank you for the great suggestion. #ping() method is added in 📦 v3.0.0

Wow wow wow, you guys are on the ball!! 🫶🏻🤯🙌🏻👏🏻
Thank you so much.
Justin Lawrence
Co-founder
M2North
https://www.linkedin.com/in/justin-lawrence
https://twitter.com/justin_lawrence
+44<PHONE_NUMBER> <+44+7903060047>
@.***
www.m2north.com
uk +44<PHONE_NUMBER>usa +1<PHONE_NUMBER>za +27 11 856-2000
M2North disclaims liability for any loss, damage or expense however caused,
arising from use of this email. Full Disclaimer.
https://www.m2north.com/email-disclaimer
On Sun, 14 Apr 2024 at 21:40, dr.dimitru @.***> wrote:

@justinlawrence https://github.com/justinlawrence thank you for the
great suggestion. #ping() method is added in 📦 v3.0.0
https://github.com/veliovgroup/mail-time/releases/tag/3.0.0
—
Reply to this email directly, view it on GitHub
https://github.com/veliovgroup/mail-time/issues/42#issuecomment-2054178695,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALTMH7G2V2JKU2552DIFFTY5LSSXAVCNFSM6AAAAABFZK53PSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJUGE3TQNRZGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@justinlawrence Feel free to close it in case if the issue is solved after testing latest update on your end

@dr-dimitru apologies for the delay.
We have just implemented the ping() check into our AWS health checks, and it is working well.
Once again, huge thanks!! 🙌🏻


Github Source drops critical header

/kind bug
I am trying to use the github.com/google/go-github library to handle events received through the Github event source, and it seems that a key header is being dropped.
From my own logging:
ERROR: unable to parse webhook: unknown X-Github-Event in message:

What I'm doing is largely following the sample usage here, with the exception that I don't call ValidatePayload because that secret verification is managed by the github event source.
Here's the code printing the above error:
        event, err := github.ParseWebHook(github.WebHookType(r), payload)
        if err != nil {
                log.Printf("ERROR: unable to parse webhook: %v", err)
                http.Error(w, err.Error(), http.StatusBadRequest)
                return
        }

Making sure this sample usage works feels like it would be a great e2e test for this event source.
Note: I'm not sure if the "github source" or the "in-memory-channel" it proxies through is dropping the header.

It looks like it is the github event source translating this into cloud event headers.  If I add further instrumentation I see this header:
ce-eventtype: dev.knative.source.github.issue_comment

I believe this is the offending logic.
From a user perspective, this is confusing and frustrating.  I don't really care if it uses cloud events or not, I just want it to work (and with unmodified/existing libraries!).
Can the injection of cloud events headers be additive (so it is non-breaking)?  Why is it our responsibility to translate Github's events?  Shouldn't Github be responsible for adopting a different payload?

I just updated to 0.6 and tried reverting my hack and still get no github header.

cc @n3wscott


On-startup setting not working on Linux

System Details

OS: TUXEDO OS
Thorium 117.0.5938.157

Problem
I set Thorium to open one specific page right on start-up. It only persists showing a dashboard containing little squares of bookmarks. The “open a specific page” under the start-up section does not obviously work.
Thank you

It is recommended to uninstall and reinstall the latest version of the browser and then check again to see if the problem continues to occur


Many GET requests for every namespace in Releases page

There are a lot of requests for each namespace executed when namespace selector option changed to All namespaces.
UI completely freezes until all of the requests are fetched.
https://user-images.githubusercontent.com/9607060/107762645-6122c280-6d3e-11eb-88c6-cae72d462fb8.mp4
Related to #2125

I suggest that load/watch reworks touched only KubeObjectStore instances, but Helm Releases use their own store which doesn't extent KubeObjectStore with newest loadAll() changes.


New package: Survey v0.11.1

Registering package: Survey
Repository: https://github.com/xKDR/Survey.jl
Created by: @smishr
Version: v0.11.1
Commit: 72840f0be2bd4c50733e5cea07dec32fab52f0fc
Git reference: main
Description: Analysis of complex surveys
Release notes:

Registration of Survey.jl package

Please do not merge this. It will be closed soon, a different one has been opened.
@smishr can you please close this ASAP?


[BUG] Not displaying past value anomalies correctly on chart

What is the bug?
Anomaly detection results also display past values when we have cases where we detected late anomalies.
The past_values is the actual input that triggered an anomaly. If past_values is not null, then the attributions or expected values are from a past input. This however can cause some incorrect situation on our visualization because it would attribute an anomaly that happened two intervals ago (T-2) to the current time, it also would mean that if current value is also anomalous (T-0) we will see that anomaly displayed later on T+2 for example.
explanation of late anomalies can be found here: https://opensearch.org/docs/latest/observing-your-data/ad/result-mapping/
How can one reproduce the bug?
Do you have any additional context?
Add any other context about the problem.

[Catch All Triage - 1, 2, 3, 4]


Fillet Knife Dupe

Describe the bug
Shift-clicking the result of a fillet knife and a fish in an occultism storage results in the knife being duplicated.
To Reproduce
Steps to reproduce the behavior in screenshots:
Required things:

Getting ready to craft:

After shift-clicking result:

Both recipe and knife are from Aquaculture. I think the issue is that the knife is unbreakable, but the recipe may be trying to consume durability.
Expected behavior
For the amount of knives to remina constant :P
System (please complete the following information):

Occultism Version: occultism-1.16.5-1.13.2
OS: [e.g. Windows] Windows
Minecraft Version: 1.16.5
Modpack Link and Version, or list of mods:
Enigmatica 6 v0.5.11 in-dev

Additional context
Relayed from: https://github.com/NillerMedDild/Enigmatica6/issues/3293

That's a lovely one - let's see if I can get Aquaculture set up in my dev env :)

Dayum, nice work!


Error in rendering in Android WebView in some Samsung tablet

Issue Summary
The problem occurs in Galaxy Tab S6 and Tab A10.1 in WebView in android app. Most symbols are here but not at the right place.
I was not able to reproduce on a samsung phone, nor in chrome or in samsung browser. This mean, and I'm really sorry for this, that I really need to use a webview in an app to reproduce the problem. It's trivial to reproduce in the app,  and we'll assist you in any way we can if you accept to take time to help us. I understand that it would make debugging harder.
Essentially, our app generate some html to display in a webview. Up until know, we never had any trouble displaying web, and everything rendering properly in browser were rendered properly in the webview.
Steps to Reproduce:

Install the free app AnkiDroid on android
Download the attached file mathjax.zip
unzip it  and import it in the app (or just use "+" to create a card and put   \[\sigma_{1,2}=\frac{-\frac{B}{A} \pm \sqrt{\left(\frac{B}{A}\right)^{2}-4 \frac{C}{A}}}{2}\] in its front)
click on the deck in the main screen
Look at the rendered maths. You can see at least that the top of the square root is not correctly placed.

You can look at our investigation, in the ankidroid team, on https://github.com/ankidroid/Anki-Android/issues/7848
Technical details:

MathJax Version: Mathjax 3.1.2,
Client OS: Android 10
Browser: android.webkit.WebView

Supporting information:

If your issue is with the display of the mathematics produced by MathJax, include a screen snapshot that illustrates the problem, when possible.

Check your browser console window for any error messages, and in crude them here.

I fear I don't know how to do this one from a webview in an app

Include the MathJax configuration you are using, and the script tag that loads MathJax itself.
https://github.com/hgiesel/Anki-Android/blob/5decb2e877f063a496adae30a01ee362a42cb838/AnkiDroid/src/main/assets/mathjax/conf.js
and

<script src="file:///android_asset/mathjax/tex-chtml.js"> </script>

It looks to me that the scaling of the characters isn't right (they look enlarged compared to the positioning and sizing of the horizontal lines).  Could there be CSS that is interfering with MathJax's layout?  Anything with a * selector, or just a [...] selector, perhaps?  Or anything referring to mjx-*?
Have you tried switching to MathJax's SVG output rather than the HTML output?  CSS issues tend to interact with it less often.  Can you try switching to tex-svg.js instead of tex-chtml.js?

I just tried the very same formula with absolutely no css and it seems to produce exactly the same result, appart from the font color. So I doubt it's a CSS problem.
I tried with tex-svg.js (and of course adding the file to the app).
For some reason it seems to be stuck processing. No content appears on screen, not even the content that is not related mathjax

So, we decided in going on with SVG, as it solves our problem.
If you are interested in solving the bug with chtml and want our help, I'll be happy to contribute as much as possible and do all tests you want.
I updated the reproduction step to indicate which version of the app allow to reproduce the problem

I totally understand, thank you very much for the time you took to answer me.


@orta => 3.4 fixes

Christine asked for a 3.4 release for Monday, fixing these things.
No tests yet. Here's what I want to do:

[x] Write code for fixes
[x] Orta reviews code
[x] I write tests(ish)

After Orta review's the code, we at least have a build to test for the auction.

Looks good.

OK, so checks are in. Once merged, I'll make a build and get @alessandra-castillo's help testing.

Ready to merge :+1:


fixed the dependencies and set up some empty js files for the login view

Nothing huge, I had missed to change the requires in the html file. Now it should work. This is what it looks like:

:shipit:


Versions refactor

Description
This PR contains:

removal of old unused makefile targets, and bad CI jobs.
insertion of git-godescribe
refactoring of python/ambassador/VERSION.py to use git-godescribe
edits to Makefiles and shell scripts to support the above

TODO:

make sure charts and manifests generation is using new versioning
update CI and rewrite charts and manifests processes

Related Issues
List related issues.
Testing
Manual testing was done including

make env
make images
make deploy

CI needs to be tested.... and rewritten to be honest
Checklist

[ ] I made sure to update CHANGELOG.md.
Remember, the CHANGELOG needs to mention:

Any new features
Any changes to our included version of Envoy
Any non-backward-compatible changes
Any deprecations

[ ] This is unlikely to impact how Ambassador performs at scale.
Remember, things that might have an impact at scale include:

Any significant changes in memory use that might require adjusting the memory limits
Any significant changes in CPU use that might require adjusting the CPU limits
Anything that might change how many replicas users should use
Changes that impact data-plane latency/scalability

[ ] My change is adequately tested.
Remember when considering testing:

Your change needs to be specifically covered by tests.

Tests need to cover all the states where your change is relevant: for example, if you add a behavior that can be enabled or disabled, you'll need tests that cover the enabled case and tests that cover the disabled case. It's not sufficient just to test with the behavior enabled.

You also need to make sure that the entire area being changed has adequate test coverage.

If existing tests don't actually cover the entire area being changed, add tests.
This applies even for aspects of the area that you're not changing – check the test coverage, and improve it if needed!

We should lean on the bulk of code being covered by unit tests, but...
... an end-to-end test should cover the integration points

[ ] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently

Closed in favor of #4003 and #4013


deployment on a headless server problem??????

when i run
/opt/websockify/run 5901 --web=/opt/noVNC --wrap-mode=ignore -- vncserver :1 -securitytypes otp -otp -noxstartup
this command,i got the infomation like this:
_WebSocket server settings:

Listen on :5901
Flash security policy server
Web server. Web root: /opt/noVNC
SSL/TLS support
proxying from :5901 to 'vncserver :1 -securitytypes otp -otp -noxstartup' (port 33951)
Starting 'vncserver :1 -securitytypes otp -otp -noxstartup'

Desktop 'TurboVNC: unix:1 ()' started on display unix:1
One-Time Password authentication enabled.  Generating initial OTP ...
Full control one-time password: 23411271
Run 'vncpasswd -o' from within the TurboVNC session or
'vncpasswd -o -display unix:1' from within this shell
to generate additional OTPs
Log file is /root/.vnc/unix:1.log_
then, how can i run
_Run gibson with DISPLAY=:1 vglrun python xxxx
this command? that is not interactive any more, it is listening the connection, and print the log info.
anyone can help ?
thks

for this command
/opt/websockify/run 5901 --web=/opt/noVNC --wrap-mode=ignore -- vncserver :1 -securitytypes otp -otp -noxstartup
when i add --daemon  flag
the command window become interactive, but i can't find the password for vnc login!
and when i run  DISPLAY=:1 vglrun python  examples/demo/play_husky_nonviz.py
got the errors like this:
root@b35cb79bd062:~/mount/gibson# DISPLAY=:1 vglrun python  examples/demo/play_husky_nonviz.py
pybullet build time: Mar 20 2018 23:59:46
/root/mount/gibson/examples/demo/../configs/play/play_husky_nonviz.yaml
startThreads creating 1 threads.
starting thread 0
started thread 0
argc=2
argv[0] = --unused
argv[1] = --start_demo_name=Physics Server
ExampleBrowserThreadFunc started
X11 functions dynamically loaded using dlopen/dlsym OK!
No protocol specified
[VGL] ERROR: Could not open display :0.

For your first question, you can open a tmux session and put /opt/websockify/run 5901 --web=/opt/noVNC --wrap-mode=ignore -- vncserver :1 -securitytypes otp -otp -noxstartup in a panel and run in background. You can run DISPLAY=:1 vglrun python xxxx in another panel in tmux or in the shell.

@fxia22 thanks for your reply.
there is still some problem i not understand.
i'm not familiar about tmux.
i'm used the command like below:

tmux
docker run --runtime=nvidia -ti --rm -e DISPLAY -v /tmp/.X11-unix/X0:/tmp/.X11-unix/X0 -v /home/docker-map-folder/gibson/assets:/root/mount/gibson/gibson/assets -p 5901:5901 xf1280/gibson_server:0.2
create a new pane ( ctrl + b,  ")
4.in the first pane, i put /opt/websockify/run 5901 --web=/opt/noVNC --wrap-mode=ignore -- vncserver :1 -securitytypes otp -otp -noxstartup the shell command, and run it.
then i change to the second pane, but this pane was not in the docker container, so  i run
docker attach f4443771ecc4    the command into the container.
however ,these two pane are synchronizing show info.

i don't understand how run /opt/websockify/run 5901 --web=/opt/noVNC --wrap-mode=ignore -- vncserver :1 -securitytypes otp -otp -noxstartup  this command in background.
beg for your reply and answer, thanks again.

i use nohup  command & solved this problem, but i got the same problem with issue https://github.com/StanfordVL/GibsonEnv/issues/16


Billing RP Operations

Add Billing Document
Add Billing Cosmos operation : Create / Get / Delete
Create Billing Document on Create
Update Billing Document on Delete

Cosmos DB deployment will need to be re-apply in regions to add the Billing container

/azp run e2e

/azp help

/azp run e2e

@julienstroheker please can you rebase so this is just flat commits on top of master?  Otherwise this is ready to merge.

/azp run e2e

just changed e2e pipeline file to renamed e2e.yml one.
testing if it is working

/azp run e2e

/azp run e2e

sorry @julienstroheker, one more rebase

@jim-minter should be good now


Changed editor/format lsp feature requirement from textDocument/rangeFormatting to textDocument/formatting

A lot of lsp servers don't support textDocument/rangeFormatting but support textDocument/formatting(eg: rust-analyzer). The fallback formatter has some issues (like in rust-mode it won't respect the project's cargo files). I think it's better to use lsp as the default formatter as long as the lsp server support file formating.

Makes sense to me, thanks!


feat: add cursor_text_color

Cherry-pick of:

https://github.com/kdrag0n/base16-kitty/pull/12

from the "original" base16-kitty repo (ie. the one referenced from chriskempson/base16-templates-source).
Original description:

By default, Kitty is setting the cursor text color to #111111.
See https://sw.kovidgoyal.net/kitty/conf/#opt-kitty.cursor_text_color.
It's not rendering well when using vim with light theme.

Thanks for the PR. Can you add this change to the base24.mustache and base24-256.mustache file?

Thanks for the PR. Can you add this change to the base24.mustache and base24-256.mustache file?

Sure thing. Done in 4e9258dc8224810bcaede11ba0d13b4c049c16a9.

I’ve just run a build so the themes have been updated.


[🐞] loader$ yields TS4023: Exported variable has or is using name 'ServerLoader' from external module

Which component is affected?
Qwik City (routing)
Describe the bug
When using the new loader$ like:
import { loader$ } from<EMAIL_ADDRESS>
export const myContent = loader$(() => myData)

The exported data loader is yielding a typescript warning:
TS4023: Exported variable 'myContent' has or is using name 'ServerLoader' from external module

The cure for this is usually straightforward; it just needs the type declared:
export const myContent: ServerLoader<MyType> = loader$(() => myData)

However, this is impossible because ServerLoader is not exported out.
Reproduction
https://stackblitz.com/edit/qwik-starter-5zm2al?file=src/routes/index.tsx
Steps to reproduce
To see the error, you need to enable declarations in your tsconfig.
System Info
System (WSL):
- OS: Linux 5.15 Alpine Linux
- CPU: (12) x64 AMD Ryzen 5 3600 6-Core Processor
- Memory: 12.72 GB / 15.57 GB
- Container: Yes
- Shell: 5.9 - /bin/zsh

Binaries:
- Node: 18.12.1 - /usr/bin/node
- Yarn: 3.3.1 - /usr/bin/yarn
- npm: 9.1.2 - /usr/bin/npm

Additional Information
The missing type does not break the build but is reported as an error in Jetbrains and Vscode. A quick solution for this would be to export out the ServerLoader so that it is available to be used as a type.

For a potential use-case, with the declarations, one could create or potentially generate the loader$'s as a client and share it as a package with types pre-compiled to minimise the compilation burden.

Fixed in latest QwikCity beta!


Custom LLMs & APIs Endpoints

Will it support the local llm models like LlaMa and RWKV using the local APIs like oobobga Text Generation WebUI ?
This would solve the problems of insufficient OpenAI credits and limited tokens and the countries that ChatGPT access is still unavailable in yet

Yes – since we're using the ai SDK, you should be able to swap OpenAI out for other supported LLMs fairly easily!
Here are the docs: https://sdk.vercel.ai/docs


DOC: Add document description to Eventify

quick doc change

Codecov Report

Merging #92 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   97.54%   97.54%
=======================================
  Files           6        6
  Lines        2160     2160
  Branches      488      488
=======================================
  Hits         2107     2107
  Misses         25       25
  Partials       28       28

Impacted Files
Coverage Δ

shed/event_streams.py
93.9% <100%> (ø)
: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 2a6e7cf...a6d7845. Read the comment docs.

just need a rebase

rebased

thanks!


feat: create the home and the search components

The landing page is now the HomeComponent and the search component can lookup some mock repositories.


Remove hard coded switch on whether simplified rewards are used.

I will merge this now to move forward, even though it misses setting simplified rewards to regtest for easier test vector setup. I'll PR this together with test fixes I'll be working on, cause it is a test fix after all :)


Update getting-started.md

If we follow the getting started document, there is nowhere we have created a entity_type . If the entity_type is already available then we can use the reference using data block but here the entity_type is not created already hence we are getting entity_type not found error, to avoid I'm updating the document as below
Before:
data "itsi_entity_type" "terraform_itsi_etype"
entity_type_ids = [ data.itsi_entity_type.guide_itsi_host.id]
After:
resource "itsi_entity_type" "terraform_itsi_etype" entity_type_ids = [ resource.itsi_entity_type.guide_itsi_host.id]

@premadhas thanks for the PR.
Also, FYI, when you update the docs, also need to run make docs to actually render the final docs that will be published to the terraform registry. I'll do that in a separate PR since I can't push to your branch.


feat(ci): decommission old container-integrations e2e tests

What does this PR do?
Remove k8s-e2e-* gitlab jobs own by @DataDog/container-integrations.
This jobs were replaced by the new new-e2e-containers* gitlab jobs.
Motivation
Remove useless jobs since the new jobs cover more use cases and should be less flacky 🤞
Additional Notes
Monitors needs to be updated/remove to not trigger if we received "no data" information for these jobs execution.
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
No need to do QA for this change, the CI is handling it.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge -p low


Upload via form in rails 4.2

I'm using the following software
[git:master]% bundle show refile
/home/user/.gem/ruby/2.2.0/gems/refile-0.5.0
[git:master]% bundle show rails
/home/user/.gem/ruby/2.2.0/gems/rails-4.2.0

This is my model
class ValidationJob < ActiveRecord::Base
[...]
  attachment :source_file
[...]
end

I'm uploading the file using the form below
= form_for(@validation_job, html: { class: 'fe-form' }) do |f|
  - if @validation_job.errors.any?
    = render partial:'errors/saved_object', object: @validation_job

  .fe-form-field
    = f.attachment_field :source_file, class: 'fe-form-file-field'
  .fe-form-field
    = f.submit t('views.validation_jobs.new.link'), class: 'fe-button-new'

The params hash contains this:
[21] pry(#<ValidationJobsController>)> params['validation_job']['source_file']
=> #<ActionDispatch::Http::UploadedFile:0x000000041f8a98
@content_type="application/zip",
@headers="Content-Disposition: form-data; name=\"validation_job[source_file]\"; filename=\"blub.zip\"\r\nContent-Type: application/zip\r\n",
@original_filename="blub.zip",
@tempfile=#<File:/tmp/RackMultipart20150117-25550-28gxif.zip>>

When adding this to source_file
@validation_job.source_file = params['validation_job']['source_file']

This is the attachment after adding the file from from.
[29] pry(#<ValidationJobsController>)> @validation_job.source_file = params['validation_job']['source_file']
=> #<ActionDispatch::Http::UploadedFile:0x000000041f8a98
@content_type="application/zip",
@headers="Content-Disposition: form-data; name=\"validation_job[source_file]\"; filename=\"blub.zip\"\r\nContent-Type: application/zip\r\n",
@original_filename="blub.zip",
@tempfile=#<File:/tmp/RackMultipart20150117-25550-28gxif.zip>>

Now I'm gonna save the file.
[30] pry(#<ValidationJobsController>)> @validation_job.save!
(0.3ms)  BEGIN
SQL (0.3ms)  UPDATE "validation_jobs" SET "progress" = $1, "source_file_id" = $2, "updated_at" = $3 WHERE "validation_jobs"."id" = $4  [["progress", nil], ["source_file_id", "9456bb331274aca0e0ee66564e918aa18a47034404978ca142a3a2abb2ae"], ["updated_at", "2015-01-17 21:06:32.903457"], ["id", 9]]
(3.3ms)  COMMIT
=> true

What is weird, the size is zero.
[25] pry(#<ValidationJobsController>)> @validation_job.source_file.size
=> 0

But if I use #tempfile on params, then adding the file works differently.
[26] pry(#<ValidationJobsController>)> @validation_job.source_file = params['validation_job']['source_file'].tempfile
=> #<File:/tmp/RackMultipart20150117-25550-28gxif.zip>

Now it shows the correct size.
[27] pry(#<ValidationJobsController>)> @validation_job.source_file.size
=> 5754238

Anything I do/did wrong? Thanks a lot.

First thing, try upgrading to 0.5.2.
I'm not quite sure what's going wrong here.

Done. Not fixed by upgrade.
[1] pry(#<ValidationJobsController>)> params['validation_job']['source_file']
=> #<ActionDispatch::Http::UploadedFile:0x007f7f4b35e158
@content_type="application/zip",
@headers="Content-Disposition: form-data; name=\"validation_job[source_file]\"; filename=\"blub.zip\"\r\nContent-Type: application/zip\r\n",
@original_filename="blub.zip",
@tempfile=#<File:/tmp/RackMultipart20150118-32543-u5efdy.zip>>

[2] pry(#<ValidationJobsController>)> @validation_job.source_file = params['validation_job']['source_file']
=> #<ActionDispatch::Http::UploadedFile:0x007f7f4b35e158
@content_type="application/zip",
@headers="Content-Disposition: form-data; name=\"validation_job[source_file]\"; filename=\"blub.zip\"\r\nContent-Type: application/zip\r\n",
@original_filename="blub.zip",
@tempfile=#<File:/tmp/RackMultipart20150118-32543-u5efdy.zip>>

[3] pry(#<ValidationJobsController>)> @validation_job.source_file.size
=> 0

[git:master]% bundle show refile
/home/user/.gem/ruby/2.2.0/gems/refile-0.5.3

This is very strange. I have no idea how to help you though. This behaviour is quite well tested, and I can't replicated it locally. The very best would be if you were able to write a failing test case for this (see spec/refile/feature). Or failing that, provide an example application which shows off this failure. Try digging a bit deeper, into what the size method is doing and where it's going wrong. In pry, doing something like show-source @validation_job.source_file.size is a good starting point.

Ok. This is a "copied" rails application. I already remove tmp/. Is there any other directory which needs to be removed?

Err link?

I still can't understand what the problem is, unless you can specify it in more detail, I'll have to close this issue.

Ok. I will release the application as open source soon.

Closing this for now, please let me know when you have more information.

ok.

here's the application https://github.com/fedux-org/middleman-presentation-validator


Make builtins more discoverable

I think an easy way to do this would be to make a default module that contains all of the builtins.  We could name it something like builtin, and have it as a builtin itself.  Alternatively, it doesn't need to be named at all, and can just be used for building documentation.

Yes! I had considered doing this in the past. Another possible name is prelude, as that is used in some other languages (but builtin is more obvious). As part of the GC additions I may be reworking how the builtins interact with module loading (as part of adding the GC tracing), so I'll keep this in mind.


add immersive loading hud options

Progress dialog can now maintain Immersive mode

Hey @aritchie , are you reviewing this?

When I have time

Perfect - this should be out later on today or tomorrow.  Issues with devops at the moment.  Thanks for the PR

Hey, I made the required changes. Have a look...