images in subfolders doesn't work

you may try this yourself to reproduce
I tried this solution by putting test.jpg in root of the bucket  - works perfect
eg. test123.com/test.jpg is the path location
test123.com/300x300/smart/diwali.jpg - works perfect
try the same image test.jpg in a subfolder of the same bucket - doesn't work, says keys not present
eg. test123.com/folder1/folder2/test.jpg is the path location
test123.com/300x300/smart/folder1/folder2/test.jpg - doesn't work
error in cloud watch
2019-06-23T09:05:46.801Z	f8642cb7-0584-4b30-9313-27aee17205c0	ImageRequest {
requestType: 'Thumbor',
bucket: 'xxx.xxxx.x.xxx',
key: 'test.jpg',
edits: { resize: { width: 300, height: 300 } },
originalImage: <Buffer ff d8 ff e0 00 10 4a 46 49 46 00 01 01 01 01 2c 01 2c 00 00 ff e1 00 d7 45 78 69 66 00 00 49 49 2a 00 08 00 00 00 01 00 0e 01 02 00 b5 00 00 00 1a 00 ... > }

looks like while retrieving the key:
    parseImageKey(event, requestType) {
        if (requestType === "Default") {
            // Decode the image request and return the image key
            const decoded = this.decodeRequest(event);
            return decoded.key;
        } else if (requestType === "Thumbor" || requestType === "Custom") {
            // Parse the key from the end of the path
            const key = (event["path"]).split("/");
            return key[key.length - 1];
        } else {
            // Return an error for all other conditions
            throw ({
                status: 400,
                code: 'ImageEdits::CannotFindImage',
                message: 'The image you specified could not be found. Please check your request syntax as well as the bucket you specified to ensure it exists.'
            });
        }
    }

it should fetch based on /smart/ context root instead of default / . what do you think ?

+1 We are experiencing the same issue. It should not assume return key[key.length - 1]; to be the key but for thumbor requests strip the known parameters? Or otherwise at least let us configure a base path, in our case all images are located in the same subfolder so that might work. For others it may not if they have varying image folders...

Hey @raaone7 and @MartijnKooij: Thanks for your input on this, since this seems to be a popular feature request (working with subfolders using the Thumbor request format), I've gone ahead and added this to our backlog for a future release. In the meantime, try giving the new JSON-based request format a try, since you can specify subfolders within the "key" property similar to how an s3.getObject({}) request would work. This may provide a workaround for you guys in the meantime while we work on this.

dear can you please fix this issue?

Yes, please put this high on the priority list!

@hayesry thanks for the acknowledgement. can you please advice us of an ETA when this issue would be fixed, and how would we update the existing setup with the fix you deliver. i suppose it would be delivered as a CFN update.

@hayesry
In the meantime, try giving the new JSON-based request format a try, since you can specify subfolders within the "key" property similar to how an s3.getObject({}) request would work.

but how, in the solution been deployed, i don't do anything, it adds the S3 bucket trigger to call the lambda and send the key automatically, i don't think so we have control to alter the key in JSON, unless we over ride your code, which i can, but i prefer not to and keep it native, since the change if i do, it would not be managed support in that case, and i would need to manage the code myself.

I am having similar issue, but strangely, I have one or two subfolder images that show, while the rest don't. Getting the error {"status":500,"code":"NoSuchKey","message":"The specified key does not exist."}. What is the solution for this?

Same here, just found out it's an issue related to the path. Really need this feature, but going to use JSON request format for now.

Can anyone please fix this issue i am not able to deploy this solution.

Due to the fact that AWS have updated the lambda execution environment, which caused our old thumbor implementation to break, we had to implement this ourselves. If anyone is interested, this is the code we have so far (this is not tested in production yet, so no guarantees...but maybe helpful to someone)

    /**
    * Gets the S3 key of the file from the request path
    * Please note: this function assumes that the dimensions will be in the request somewhere (eg /200x300/ or whatever)
    * @param {Object} event - The request body.
    */
    getFileS3Key(event) {
        this.path = event.path;
        const pathParts = this.path.split('/');
        var fileS3Key = '', sizeParameterFound = false;
        const matchSize = new RegExp(/^\d+x\d+$/);

        for (var i = (securityHashPathIndex + 1); i < pathParts.length; i++) {
            const pathPart = pathParts[i];
            if (fileS3Key.length > 0) {
                fileS3Key += '/';
            }
            else if (!sizeParameterFound && matchSize.test(pathPart)) {
                sizeParameterFound = true;
                continue;
            }
            else if (pathPart === ('fit-in') || pathPart.includes('filters:') || !sizeParameterFound) {
                continue;
            }

            fileS3Key += pathPart;
        }
        return fileS3Key;
    }

securityHashPathIndex is 1 (in my case at least).
and a unit test for that:
// ----------------------------------------------------------------------------
// getFileS3Key()
// ----------------------------------------------------------------------------
describe('getFileS3Key()', function () {
    describe('001/fileS3Key', function () {
        it(`Should pass if the file S3 key is being extracted correctly from the url path`, function () {
                // Arrange
                const event = {
                    path: "/somesecurityhash/fit-in/200x300/filters:grayscale()/some/filexfile/path/blah/test-image-001.jpg"
                }
                // Act
                const thumborMapping = new ThumborMapping();
                var result = thumborMapping.getFileS3Key(event);
                // Assert
                const expectedResult = "some/filexfile/path/blah/test-image-001.jpg";
                assert.deepEqual(result, expectedResult);
            });
    });
});

This solution is unusable for us without the ability to use subfolders.

I'm in exactly the same boat as @soupy1976 here.
Old version died due to environment changes. New version fails with anything outside the root of the bucket.
I would happily move to Sharp from Thumbor, but Sharp won't layer images based on absolute remote urls.
So here we are.
My fix was:

Go to the Lambda function
Select 'Actions' then 'Export Function'
Unzip the download and open 'image-request.js'
Update the parseImageKey method to the following:

        if (requestType === "Default") {
            // Decode the image request and return the image key
            const decoded = this.decodeRequest(event);
            return decoded.key;
        } else if (requestType === "Thumbor" || requestType === "Custom") {
            // Parse the key from the end of the path
            const pathParts = (event["path"]).split("/");

            for (let i = 0; i < pathParts.length; i++) {

                if (
                    pathParts[i] !== ''
                    && pathParts[i] !== 'fit-in'
                    && pathParts[i].match(/^\d+x\d+$/) == null
                    && pathParts[i].match(/^filters:/) == null
                ) {
                    const parts = pathParts.slice(i);
                    return (parts.join('/'));
                }
            }
        }

        // Return an error for all other conditions
        throw ({
            status: 400,
            code: 'ImageEdits::CannotFindImage',
            message: 'The image you specified could not be found. Please check your request syntax as well as the bucket you specified to ensure it exists.'
        });
    }```
* Save the file
* Zip the contents of the folder (not the folder itself)
* Back on your Lambda function page, select to upload a zip
* Choose the new zip file
* Once uploaded select 'save'

Not heavily tested, but works with image paths in the form: `https://[your path].cloudfront.net/fit-in/600x0/filters:blur(7)/prod/12345.jpg` Where the bucket path is `prod/12345.jpg`

@harrybailey v4.0 still uses sharpjs, even if you call it with the old Thumbor style request ... it simply translates that into a sharpjs request. So if you need functionality that Thumbor offers and Sharpjs does not, then you would need to use v3.0 still.
I noticed someone has just put a fork of v3.0 which is fixed to work in the new Lambda environment though, so you could be in luck:
https://github.com/awslabs/serverless-image-handler/issues/127
jb

You're spot on. It's not pulling in remote images.
I am however tempted to make v4 work with remote images, rather than jump back to v3.

@harrybailey  if you're thinking of making v4 work, then hopefully my post on stackoverflow will be helpful to you:
https://stackoverflow.com/questions/57178748/aws-serverless-image-handler-v3-x-broken-by-changes-to-aws-lamdba-execution-envi
jb

this also caught us by surprise, the version 3 (python) just stopped working. anyway, here's one-liner fix that we did for our implementation, hope it'll be of help to anyone:
https://github.com/awslabs/serverless-image-handler/pull/130

We have a mobile app in production which most of its links broke all of sudden. We manage our S3 assets by organizing them in sub-folders, which was the root cause of this problem.
We tried a lot of solutions, we were about to do huge modifications to our infrastructure which will delay us like a week or so. However, so many thanks for @harrybailey who has saved us a lot of time with his ready-to-use code.
For anyone who's searching for a solution for this issue, just follow @harrybailey's comment above, and you don't need to change anything in your current infrastructure.

@harrybailey it works here!

These edits seemsto make my URLs work - but it doesn't seem to be doing the resize when I'm using an image in a subfolder, IE
https://xxxxxxx.cloudfront.net/400x0/products/img/product1.jpg

for anyone that using @harrybailey solution and had a problem with resizing, i attach my workaround on @rpong's PR, hope it helps
https://github.com/awslabs/serverless-image-handler/pull/130#issuecomment-515665166

We have updated our solution, and I believe your issue has been fixed. If you still see the issue with the latest version (v4.2), please feel free to reopen the issue.
You can refer to the recent changes here


Wrong return type for union of overloaded functions

Bug Report

🔎 Search Terms

union, overload, function, return type
🕗 Version & Regression Information

This is the behavior in every version I tried (4.4, 4.8, 5.0, 5.1, 5.2beta)

⏯ Playground Link

Playground link with relevant code
💻 Code

interface Callable<Name extends string> {
	(): `${Name} without id`
	(id: number): `${Name} with id`
}

declare const f: Callable<"A"> | Callable<"B">
const result = f(123) //results in "A with id"|"B without id"

🙁 Actual behavior
result is inferred as "A with id"|"B without id".
"B with id" is completely missing and "B without id" should not appear since we used a parameter.
🙂 Expected behavior
result is "A with id"|"B with id".

I believe it's the same as: https://github.com/microsoft/TypeScript/issues/55101#issuecomment-1646025879

The contextual type for the return type of the function expression in this case comes from the first overload, since it's the first one that appears to match on arity.

The first overload that matches the arity is the first one, without any arguments (providing too many arguments is fine).
You can fix your code by changing the order of overloads from most specific to least specific:
interface Callable<Name extends string> {
	(id: number): `${Name} with id`
	(): `${Name} without id`
}

There's no contextual type or widening here. Definitely better to write it with the overloads in the other order, though


docs: add subscription section

Fixes https://github.com/daytonaio/enterprise-docs/issues/162

@stefanicjuraj  I think we should give instructions how to upgrade/downgrade/cancel plans here?

@ivan-burazin Not entirely sure that this documentation belongs here. cc @gdraganic

same as AI agent. this needs a a "cloud" only label, and all such pages should stay commented.
Thanks


refactor(rating): update to latest conventions

This component still needs sweet love, but let put it pretty first.

LGTM

LGTM

So what convention you decide ?


Add bybit full implementation of Order logic for interface version V3

Added logic for  limited order, trade histrory, cancel order , ....

Hello, can you please update these for V5 instead of V3. They will make it deprecated. Thank you


Triggering edit on the current cheat file.

Hi!
Is it possible to implement a way to trigger own editor on the current or even just a certain navi cheat file based on a keybinding?
Now feels quite cumbersome that it needs to separately navigate to those files before editing.

I'm not sure I'm following. Could you elaborate or explain it with different words?
Do you mean you would like a keybinding to directly edit a .cheat file?

I meant if when the navi prompt has being started then I can get the output from .cheat file.
It would be nice to get a keybinding that can triggger from the prompt to directly edit a predefined .cheat file.

Hi! Sorry for the late reply.
Being 100% honest here: this is a feature it would be nice to have but unfortunately I haven't had much free time lately, so I have no plans to implement this in the near future.
If anyone could submit a PR for this, I'll gladly accept it, though! 👍
One workaround in the meantime for this is to set $FZF_OVERRIDES (please check README.md) with --bind and something like vim $(_find {}) where _find is some command such as rg or find that's able to find a filename given the contents of the selected line.

Hi, I wanted to solve this but I just need some clarification:
So you want to bind a key to open one specific file
or a global binding to open the .cheat file of the selected entry?
I will start to implement the latter unless you have objections.

Hi,
global binding to open the .cheat file of the selected entry could make this tool more flexible but I guess u've implemented it already

Yes, it's implemented already. It's available on master but if you want to use it you'll need to build navi. In any case, I'll probably release a new version in December still

Is it possible to add keybinding reminders on the navi screen? Like ctrl+o is easy but could be nice if there is on top or bottom a quick tip bar that can be enabled.

Otherwise this is super nice feature :D and quick. Thanks!!!!


Find Details page's text is too big

The text on the Find Details page is slightly bigger than the Hunt Details which makes it look off.

Also, the "overview" icons do not match

Fixed!


⬆ upgrading to actix-web@4

Codecov Report

Merging #200 (ec398b7) into master (b21180c) will increase coverage by 0.17%.
The diff coverage is 0.00%.

:exclamation: Current head ec398b7 differs from pull request most recent head d3ef7e7.

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


replace string matcher (linux related)

org.hamcrest.Matchers.isEmptyString is deprecated and is to be replaced by org.hamcrest.Matchers.emptyString().
Whatever the issue is on Linux, this is not the way to fix it.


Angular: make a call on using the dev-override pageloader

Seems SUPER hacky.

I'd qualify it as "manual" rather than "hacky" :).
It just occurred to me that the current pageloader package could actually allow us to test ACX apps, ... something that we could not do with the pageloader v2. (I haven't had the time to try that, but it has been on my to-do list for a little while.)

Just chatted w/ angular folks. We're going to remove the test bits that require this workaround. @kwalrath to run w/ this


New compare table

Overview
Resolves #209
This turned out to be quite a bit more difficult than expected. Why? Because you can't build a table by columns, only by rows. So you can't make a component for each query added to compare because the line wraps in each ones title get everything out of line. A table manages these line wraps. So this implements a component for each row. As in there's a component made to show the GPA info, the rmp rating, ...
Other than that the gradient implementation went well and the sorting works.
Also copies over that fix to the sorting bug from PR #207

To-do

[ ] Bottom padding (the horizontal scroll bar is a few pixels above the bottom of the components in my instance
[ ] Does there need to be a way to get back to the default sort (maybe a sort by color?)
[ ] Make sure the arrows point the correct way by default
[ ] Question: Should each component have the same width
@TyHil

Bottom padding (the horizontal scroll bar is a few pixels above the bottom of the components in my instance

Looks fine on my machine, can you send a screenshot?

Does there need to be a way to get back to the default sort (maybe a sort by color?)

You can click through the the sort options to deselect and it returns to default

Make sure the arrows point the correct way by default

Done

Question: Should each component have the same width

Done


Process lamp files, initial

Pending work on actually processing

Imports are weird, my local config may be messed up


test: de-duplicate plan definitions

Instead of maintaining plan definitions in the tests, they should be read from the source of truth - the .envrc file. This reduces duplication and the tendency for the definitions to drift apart.

Happy to approve, it's worth noting it also needs updating the upgrade tests README.md to indicate that .envrc needs to be also present in the directory.


Fix some snitches crashing the server

The problem with snitches crashing the server is that the MariaDB query planner decides that it is going to be smart, and actually do a query from the (very small) ja_snitch_actions table, and then join the (humongous) ja_snitch_entries table.
To do this, it has the genius idea of filtering the ja_snitch_entries table by the type_id column. This column represents the type of snitch entry, be it entering the snitch, breaking a block, placing a block, etc. Naturally, MariaDB thinks that it can reduce the 10M+ row database to just 1000 rows with this filter. But it turns out that there are LOTS of entries for each type_id, so it actually get about 600K columns, and has to linearly search through them.
Only a handful of snitches have been found to crash the server this way. I don't know how the MariaDB query planner works, nor do I want to, but I can speculate that only very heavily trafficked snitches would crash the server in this way, or perhaps those that show an unusual distribution of entry types.
Here is the MariaDB ANALYZE output for when the query planner thinks it's really smart:
+------+-------------+-------+-------+-----------------------------+---------+---------+------------------+------+-----------+----------+------------+-------------+
| id   | select_type | table | type  | possible_keys               | key     | key_len | ref              | rows | r_rows    | filtered | r_filtered | Extra       |
+------+-------------+-------+-------+-----------------------------+---------+---------+------------------+------+-----------+----------+------------+-------------+
|    1 | SIMPLE      | jsa   | range | PRIMARY                     | PRIMARY | 4       | NULL             | 17   | 1.00      |   100.00 |     100.00 | Using where |
|    1 | SIMPLE      | jse   | ref   | type_id,snitch_action_index | type_id | 5       | jukealert.jsa.id | 1241 | 671443.00 |     0.51 |       0.00 | Using where |
+------+-------------+-------+-------+-----------------------------+---------+---------+------------------+------+-----------+----------+------------+-------------+

As you can see, the query planner thinks it will get 1241 rows, but it actually gets 600K. Big miscalculation.
In contrast, here is a normal snitch:
| id   | select_type | table | type   | possible_keys               | key                 | key_len | ref                   | rows  | r_rows | filtered | r_filtered | Extra       |
+------+-------------+-------+--------+-----------------------------+---------------------+---------+-----------------------+-------+--------+----------+------------+-------------+
|    1 | SIMPLE      | jse   | ref    | type_id,snitch_action_index | snitch_action_index | 5       | const                 | 54054 | 10.00  |    50.00 |     100.00 | Using where |
|    1 | SIMPLE      | jsa   | eq_ref | PRIMARY                     | PRIMARY             | 4       | jukealert.jse.type_id | 1     | 1.00   |   100.00 |     100.00 |             |
+------+-------------+-------+--------+-----------------------------+---------------------+---------+-----------------------+-------+--------+----------+------------+-------------+

Interesting, the query planner drastically over-estimates the amount of rows it will scan. It thinks it will need to comb through 50K logs, but it actually ends up looking through 11.
Perhaps an alternative solution could be trying to update the data analysis the query planner gets - it may be severely inaccurate. However, this fix should be sufficient as STRAIGHT_JOIN forces the join to be done in a specific order (big table joins small table), instead of the wrong order (small table joins big table), which is ALWAYS wrong.
Database query planner bugs are just the best aren't they?

Question, to what extent, if at all, do ancient snitch logs (logs older than 30 days) play into this?

I don't think snitch creation time would have any impact on the query planner. Perhaps insertion order could, but I doubt it.

@RedDevel2 Code looks good and logic is sound, moved to ready for testing although testing may be somewhat difficult.

I've been able to manually test this by querying the bugged snitches directly with the old and new statements. Not on the server itself but it should be good enough to deploy straight to production @AngrySoundTech

The only thing, you may want to add a comment about specifically using STRAIGHT_JOIN to force a specific order of joining.


Issue : file not found.

Hello,
I keep running issue when executing the command.
Running FileProcessor: writing file .tmp/ios/firebaseScript.sh with nested IOSFirebaseScriptProcessor
Running ShellProcessor: Running script 'ruby' with arguments .tmp/scripts/ios/add_firebase_build_phase.rb, ios/Runner.xcodeproj, .tmp/ios/firebaseScript.sh
/Library/Ruby/Site/2.6.0/rubygems/core_ext/kernel_require.rb:85:in require': **cannot load such file** -- xcodeproj (LoadError)         from /Library/Ruby/Site/2.6.0/rubygems/core_ext/kernel_require.rb:85:in require'
from .tmp/scripts/ios/add_firebase_build_phase.rb:1:in `'
The rest is working fine, but all the Xcode config files are not generated :
ruby: No such file or directory -- .tmp/scripts/ios/add_file.rb (LoadError)
Could you help ?
here is the config file :
flavorizr:
ide: "vscode"
flavors:
roomvi_dev:
app:
name: "-----"
  android:
    applicationId: "------"
    firebase:
      config: "firebase/dev/google-services.json"

  ios:
    bundleId: "------"
    firebase:
      config: "firebase/dev/GoogleService-Info.plist"

roomvi:
  app:
    name: "-----"

  android:
    applicationId: "------"
    firebase:
      config: "firebase/prod/google-services.json"
  ios:
    bundleId: "-----"
    firebase:
      config: "firebase/prod/GoogleService-Info.plist"

Updates : I've updated gem, home-brew and Xcode
Now, I've got this error :
.tmp/scripts/ios/add_build_configuration.rb:23:in <main>': undefined method build_settings' for nil:NilClass (NoMethodError)
build_config.build_settings = base_config.build_settings.clone

Hey @AntoineWaes , what's the output of xcodeproj --version?

I same error
xcode 14.2
flutter 3.10

Hi. I had the same error and found solution for me.
 I  dont quite understand your Solution. What do you mean by get all the needed files? Your Command does not help in my case.

@devkirch
Hi. You can get all the needed files by setting assets:download process as below. Then, set some process you need like ios:xcconfig and so on. Lastly, you can delete downloaded files by setting assets:clean.
flutter pub run flutter_flavorizr -p assets:download assets:extract ios:xcconfig assets:clean
# (assets:clean is optional.)

@takekapp1990 Thank you, I already fixed my problem. I had custom instructions where the assets:clean was run right after the assets:download, which obviously deletes the files.

If none of the above solutions works for you (my case), I tried the following:
My problem started when I tried to add Firebase processor with the command flutter pub run flutter_flavorizr -p google:firebase. Then I got the error Unhandled exception: Exception: ruby: No such file or directory -- .tmp/scripts/darwin/add_file.rb (LoadError).  For solving this:

Generated the assets with flutter pub run flutter_flavorizr -p assets:download assets:extract -p google:firebase. I got the same error but a zipped file named assets.tmp.zip was generated in the root of the project.
I extracted this file and renamed the tmp folder to .tmp.
Run the command again: flutter pub run flutter_flavorizr -p google:firebase

Updates : I've updated gem, home-brew and Xcode Now, I've got this error :
.tmp/scripts/ios/add_build_configuration.rb:23:in <main>': undefined method build_settings' for nil:NilClass (NoMethodError)
build_config.build_settings = base_config.build_settings.clone

any luck in fixing this issue?


Pinecone examples updates

Description
There's an upcoming release of the Pinecone Python client which includes new functionality and breaking changes to the pattern for instantiating the Pinecone client.
The changes in this pull request update the Pinecone example Notebooks to reflect this new method of Pinecone client instantiation, while also including examples of how to use the previous format, for any users who prefer not to elect into the new functionality.
Fixes # (issue)
Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense
[x] I tested thoroughly in Google Colab

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

lgtm. pls check whether there are other examples using Pinecone

Thanks so much for the reviews! :bow:


Add MangaHen

Closes #3805
Checklist:

[x] Updated extVersionCode value in build.gradle for individual extensions
[ ] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[x] Added the isNsfw = true flag in build.gradle when appropriate
[ ] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[x] Have removed web_hi_res_512.png when adding a new extension

Extention needs update
https://github.com/keiyoushi/extensions-source/issues/3916


hermes create channel ibc-0 basecoin-0 --port-a transfer --port-b transfer fails

basecoin chain fails in ConnectionOpenTry part of the handshake:

IBC module error
Caused by:
ICS03 connection error
Caused by:
0: error verifying connnection state
1: tendermint error
2: ics23 commitment error
3: proof verification failed

backtrace:
thread '<unnamed>' panicked at 'not implemented: RevertibleStore doesn't support delete operations yet!', src/app/store/mod.rs:360:9
stack backtrace:
   0: rust_begin_unwind
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/core/src/panicking.rs:143:14
   2: <tendermint_basecoin::app::store::RevertibleStore<S> as tendermint_basecoin::app::store::Store>::delete
             at ./src/app/store/mod.rs:360:9
   3: <tendermint_basecoin::app::store::RevertibleStore<S> as tendermint_basecoin::app::store::Store>::reset
             at ./src/app/store/mod.rs:385:43
   4: <tendermint_basecoin::app::store::SharedStore<S> as tendermint_basecoin::app::store::Store>::reset
             at ./src/app/store/mod.rs:255:9
   5: <tendermint_basecoin::app::BaseCoinApp<S> as tendermint_abci::application::Application>::deliver_tx
             at ./src/app/mod.rs:259:25
   6: <A as tendermint_abci::application::RequestDispatcher>::handle
             at /Users/plafer/.cargo/registry/src/github.com-1ecc6299db9ec823/tendermint-abci-0.23.5/src/application.rs:132:69
   7: tendermint_abci::server::Server<App>::handle_client
             at /Users/plafer/.cargo/registry/src/github.com-1ecc6299db9ec823/tendermint-abci-0.23.5/src/server.rs:114:28
   8: tendermint_abci::server::Server<App>::spawn_client_handler::{{closure}}
             at /Users/plafer/.cargo/registry/src/github.com-1ecc6299db9ec823/tendermint-abci-0.23.5/src/server.rs:91:39

To reproduce

Set up basecoin chain & gaiad chain as indicated in README.md
run hermes create channel ibc-0 basecoin-0 --port-a transfer --port-b transfer

I am able to reproduce this with the phase6 branch. I have a feeling that this is caused due to a proto version mismatch, so currently working on updating basecoin's proto defs. I think I also figured out why this wasn't showing up on the CI and it seems the CI was always testing against a specific branch of ibc-rs which was previously required since basecoin depended on some ibc-rs code that wasn't merged at the time.


[Backend] GET API for answers

Deliverables-

GET API to get all the answers related to a particular questions sorted by the number of upvotes
question_id will be extracted from the request body and all the answers should be fetched based on the same.
It should be an open API(no auth middleware required)

Can i work on this ?

@arinmodi please go ahead with this issue.


Importing project on both eclipse and intellij gives error

I tried importing the project on both intellij and eclipse but lots of errors just pops up. No other projects gives these errors. What could be the reason. I have attached a snap below

Hi @vishnu-chalil,
I'm afraid that's lombok.
It requires a plugin both for eclipse and intellij
I hope that helps.
Best regards.

That did help. Thanks :)


Request for consistent release tag

Thank you for the awesome GitHub action, this solves a lot of common use cases for us. much appreciated!
I'm curious if the tag v1 is meant to track the latest unbreaking change, at this time i.e. v0.15.0? if yes, is there a way to automate this tag/release process?
It's risky to always follow the main branch as shown in examples and risk being stale if using a static tag like v0.15.0.

hm sorry, I t hink I created the tag by mistake. As there is no v1.x release yet.
I could provide an v0.1 tag or similar if this would help

I created the v0.1 tag which you can use.

Thank you!

I’m absolutely not suggesting you change this (given the v0.1 tag is in the wild and in use) but… I think a common way to do this would be a v0 tag, that points to the latest v0.x.y release.
I’d expect v0.1 to point to the latest v0.1.x release, but that’s not what it does currently.

right, v0.1 is not really intuitive but as mentioned is it now already in use. The next release will be v1.0.0 to start clean and be more consistent in tagging.


[IMPROVE] Need better sorting buttons

I visited https://gsoc.rocket.chat/ and clicked on this button randomly

Then I got to know that these buttons are for sorting the list in various ways. I think it could have been implemented better by may be adding some buttons so that people would understand what it could do. As of now the sorting feature seems hidden.

We do not need cosmetics improvement in this project.   Closing the issue.


Call to undefined method freezeTime()

Laravel Version
10.10
PHP Version
8.1
Database Driver & Version
No response
Description
I defined my test, where I need to stop the time to match the result of assertEquals()
However, I get an error because the freezeTime function is undefined.
Is it a bad implementation of mine or a bug?
<?php

namespace Tests\Unit\Services\Commands\Interfaces;

use Illuminate\Foundation\Testing\RefreshDatabase;
use PHPUnit\Framework\TestCase;
use App\Services\Commands\Interfaces\InterfaceService;
use Illuminate\Support\Carbon;

class InterfaceServiceTest extends TestCase
{
    use RefreshDatabase;
    public function testFormatFileName()
    {
        $this->freezeTime(function (Carbon $time) {
            $interfaceService = new InterfaceService();
            $time->setDateTime(2021, 8, 2, 15, 30, 0);
            $fixedDate = strtotime("2023-08-02 15:30:00");
            $this->assertEquals('20230802153000', $interfaceService->formatFileName("YYYYMMDDHHII"));

        });
    }

}

Steps To Reproduce

Define the unit test

Attempt to run the test

That method is only available on feature tests or in tests that extend Laravel's test case, which is usually present at ./tests/TestCase.php.


fix: set ingress loglevel v=3

What this PR does / why we need it:
Add your description
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @hicaistar

Release note:

NONE

/lgtm

/cherry-pick release-0.5

/cherrypick release-0.5


GitHub playground persistence

The google drive interface is in several ways better than the GitHub interface:

The file name is displayed (where "Google Drive" was shown before)
The file name and Google Drive icon are displayed in blue when the current buffer content is already saved
The file name and Google Drive icon are displayed in orange when the current buffer content is not saved
The GitHub interface has this additional dialogue, which is not necessary:

The PR #1799 has a problem: The file saving dialog box does not close.
Reproduce:

log into GitHub
"save as"
choose repo
type a NEW filename (say blurbel.js)
press "save"
The system responds by popping up "successfully created file!"  (that's good)
but the dialog box does not go away:


fix(model-server): configure websockets connections to be more robust

The existing timeouts sometimes caused disconnects when requesting larger amounts of data despite everything being functional.

see. #208
Make the LightModelServer in the model server behave more robust and more alike to the LightModelServer in MPS.

:tada: This PR is included in version 2.16.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Sftp remote file support

This is a PR to fulfill #4043 .  The sftp_provider cookbook has gotten at least one PR since creation and sftp usage is strictly required for my employer's security policy. I think there's enough community need to consider a PR into core now. Additionally the sftp_provider cookbook creates an awkward gem dependency race condition that is addressed if that dependency is delivered with the chef-client install. I structured the spec to be as similar to the ftp spec as possible where appropriate to ensure the interface is similar.
Notable gaps include:

network proxy capability
ssh key usage ( username and password in uri are enforced )

These might be addressable but I'm not sure at this time if they are inside the capability of the net-sftp gem.

John is right, the gem dependency would be great to be handled by the core chef & this would make things for us at NCR much more simple with dealing with SFTP resources.

Doh!  I'll get on those build failures shortly

As I say every time this shows up: put it in a cookbook, if it becomes popular then nominate it for core inclusion. With the new gem metadata there is literally no reason to add this to core as a first step.

Does the gem metadata support gem installation from local file system cache? I can't find anything that says so. Our PCI compliant products are still stuck in the same situation since they have to include gem dependencies in the cookbook payload and are restricted from network resources.
It feels like a no-brainer to expand remote_file access protocols, especially more secure protocols but I'm sure every PR you guys get looks like a no-brainer to the author

and the cookbook is here:
https://github.com/jkerry/sftp_provider

Adding any feature to Chef core is an increased management and maintenance load, so it always has to be weighed against the magnitude of the benefit to users. Given the limitations of this implementations (notably that it requires plain-text passwords be floating around in the recipe) I can't see this being a security win for most people and so the end benefit is going to be minimal. Using this effectively would require having an existing secrets management/distribution platform and in most cases you should just use that for access secure data instead of SFTP :) There are totally cases where you would want SFTP that aren't amenable to tools like Vault or KeyWhiz, like moving big files around, but that is a less common use case (most people would use HTTPS for that situation). Anyways, hope that explains my resistance.

Basically mutually authenticated file transfer under heavy automation is a Hard Problem and I don't think this fits the bill well as it requires a lot of support services Chef does not provide (well).

@thommay we should look at this in PR review

@lamont-granquist if there's any information I can provide let me know.  I closed it out for the time being but I can re-open a PR with the rubocop warnings cleaned up at a later date if there's need.

The rubocop failures are on files not touched in this PR.  What's the best practice for that?  I don't mind fixing them but I don't want to pollute the PR either.

could you rebase on top of master and force push?  the merge commits are making this PR pretty ugly...

Yessir, got caught up in holiday stuff.  working on it

Back to just my changes.  My apologies for the mess.

Known issue:  the copyright information on lib/chef/provider/remote_file/sftp.rb reads Jesse Campbell as it does in the ftp resource.  I'll switch that to my name when I get back to the desk.

other than the dead code i think this looks fine.  :+1:

Yeah. I'm :+1: after a bit of a spring clean.

It is used as a path validation mechanism (utilized directly in the tests).  I can probably validate the path more directly and fulfill the same interface

@thommay @lamont-granquist is this what you had in mind?

:+1: LGTM

:shipit:

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.


Fix absolute paths on validation errors

Currently some errors don't have a path that is absolute. With this PR we try to make all paths to be absolute. I.e.,
NotOneOf: #/data
{
  IntegerExpected: #/data.foo
  NoAdditionalPropertiesAllowed: #/data.bar
}
{
  NullExpected: #/data
}

or when a model contains a list
NotOneOf: #/data
{
  ArrayItemNotValid: #/participants.[0]
  {
    ObjectExpected: #/participants.[0]
    PropertyRequired: #/participants.[0].id
  }
  NullExpected: #/data.participants
}
{
  NullExpected: #/data
}

Looking forward your feedback.
Thanks,
Roger

Shouldn't it be ArrayItemNotValid: #/data/participants.[0]?

You were right. ValidateArray needed to be changed as well.

And while we're at it, we should also remove the . for an array item: array.[0] => array[0]
E.g. just call .TrimEnd('.') before assigning [0] in ValidateArray

Agreed. Now we would get an error like this
NotOneOf: #/data
{
  ArrayItemNotValid: #/data.items[0]
  {
    NullExpected: #/data.items[0].name
    StringExpected: #/data.items[0].name
    NoAdditionalPropertiesAllowed: #/data.items[0].gender
  }

  NullExpected: #/data.items
}
{
  NullExpected: #/data
}

Thanks!


[Physics] Fix typecast warnings generated by visual studio 2019

Now that all the samples are included in the visual studio projects, more parts of raylib are generating typecast warnings during a build.
This PR resolves the issues in physics and raymath that were found building the physics sample.

@JeffM2501 thanks for the review!


App restyling - App activity

Review of version 3.0.0-beta4

[ ] Fix / uniform padding as explained here

Review of version 3.0.0-beta5
Some times when I enter the activity area I get the "Amber isn't responding" popup.  Whether I select Wait or View details nothing changes and the popup appears again after a couple of seconds. The weird thing is that the view show a "No activities found" even if the app is actively using Amber.

[ ] On the top show the current app name, even if it's just the app ID
[ ] Check the "isn't responding" error


Fix sidebar flicker

This package's CSS rule for elements matching [x-cloak] conflicts with Filament's default of [x-cloak=""].
The reason this is a problem is because the admin sidebar now has x-cloak="-lg" which is only applied on non-lg screens to prevent a flicker on page load before Alpine initialises.
https://github.com/filamentphp/filament/blob/e1b3aea563a10ef1d094014c08043cd278ac94b8/packages/admin/resources/views/components/layouts/base.blade.php#L31
This PR tweaks the source spotlight.css file to ensure only elements with an x-cloak attribute without a value are matched and hidden.

Have also added /node_modules to the .gitignore file as for me (and perhaps others) this is configured on a per-project basis instead of by the IDE.

Thanks!


task2 completed

Issue: #3
completed task 2

Hey @07CalC
Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.
Thanks for contributing in OpenCode'24 ✨✨!

Hey @07CalC
Your PR has been merged 🥳🥳 and you have earned 10 points.
Thanks for contributing in OpenCode'24 ✨✨


Buttons can be placed on the top and on the bottom face of a block

Addesses issue #1756 .

Google-test failed again, like on #1734.

Is that a known issue? How can i fix it?

I've restarted the builds. No idea what happens in the Google test, for some reason there's no output from the tests. I question the need for such tests.

I think the meta values are wrong, at least according to the wiki: http://minecraft.gamepedia.com/Button#Data_values
There's no mention of meta value 6 or 7.

@alexandreguertin Any update on this?

I'll be working on it this weekend.

Oh, you should have dropped a message, I didn't see this was updated. I guess it's now good for a merge.


App Insights module needed

Hello, I am looking to utilize application insights resources but there is no module currently. Is it possible to generate a appinsights module to create the appinsights component and create web tests?

The App Insights has made their own client here:
https://github.com/Microsoft/ApplicationInsights-Python
Does this contains what you need, or there is missing part?

I did not even know that client existed. Let me give it a once-over and see if it has everything.

From the looks of it, that client is for adding into my application versus interacting with the resources in Azure. I need to create an App Insights component and then add web tests from https://docs.microsoft.com/en-us/rest/api/application-insights/. These features do not seem to be part of the Telemetry client or any contexts.

Gotcha. I create a client from the service metadescription:
https://github.com/Azure/azure-sdk-for-python/pull/1665
If you could try it, would be nice :)
Thanks!

@lmazuel Thanks for generating. I will give it a test run and let you know if there are any issues.

@lmazuel application insights module seems to be working fine. I havent run into any issues with the use cases I have. I would think it is fine to merge.

Thanks for the heads up! Will do on Monday


[C++][Parquet] Use arrow::MemoryPool for all heap allocations

We are using std::vector in many places for memory allocation; if we want to use SSE on this memory we may run into some problems.
Couple things we should do

Add an STL allocator for std::vector that ensure 16-byte aligned memory
Check user-provided memory for alignment before utilizing an SSE-accelerated routine (e.g. SSE hash functions for dictionary encoding) and decide whether to copy and use SSE or no-copy and use no-SSE code.

Reporter: Wes McKinney / @wesm
Assignee: Zherui Cao](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=czxrrr) / @czxrrr
PRs and other links:

GitHub Pull Request #5496

Note: This issue was originally created as PARQUET-517. Please see the migration documentation for further details.

Wes McKinney / @wesm:
I renamed the issue. Using arrow::MemoryPool for allocations will ensure alignment

Wes McKinney / @wesm:
I took a look through the codebase and didn't find any significant STL heap allocations. Closing

Deepak Majeti / @majetideepak:
The std::vector<int> buffered_indices_ inside DictEncoder can benefit using an arrow::MemoryPool. This is significant when there are many columns.

Wes McKinney / @wesm:
Issue resolved by pull request 5496
https://github.com/apache/arrow/pull/5496


no network connectivity information

no network connectivity information
port / egress and ingress

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5e1f5955-e5b8-9cff-873e-4a195a462e62
Version Independent ID: 12a87ce1-0e1b-f21c-bad3-09963e628d99
Content: On-premises data gateway - Power BI
Content Source: powerbi-docs/service-gateway-onprem.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @arthiriyer
Microsoft Alias: arthii

Hi, @bakerm00 -- thanks for asking! Assigning to the author to investigate.

Hi @bakerm00 ,
I'm cleaning up our Git Issues queue, and I noticed that this Git issue is a bit old, so I'm going to close it.  I hope that you were able to get your questions answered. If not, please let us know.  Another option is to submit your questions to the Power BI community (community.powerbi.com).
All the best,
the DOCS team


change back to angular 4 template

Hello,
when I open angular project with

yo aspnetcore-spa
and then I choose angular I get a new angular template.
I mean i never see app.module.server.ts , app.module.shared.ts, boot-client.ts
How can i downgrade it to old version template?

If you want an older version of the Yeoman templates, you could use npm install -g generator-aspnetcore-spa@version where version is whatever version you want.
However I recommend using the latest templates only. The latest Angular template is built into the .NET Core 2.0 SDK, and you can generate projects on the command line using dotnet new angular without installing any extra packages.
The older Yeoman templates are deprecated now and won't be maintained further.


Geo fixes

Closes #526
Closes #279
Possibly addresses #349
fix bug: cannot create annotations/shapes/images in non-cartesian traces

not sure the error messages really need to be localized, but I appreciate the thoroughness :)

no idea what percy is doing, I see all panels just fine locally

💃 on the code


Fixup CKS UI for external managed clusters

Description
This PR fixes #9291 #9293

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@vishesh92 a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 10445

@blueorangutan test alma9 kvm-alma9 keepEnv

[SF] Trillian test result (tid-10949)
Environment: kvm-alma9 (x2), Advanced Networking with Mgmt server a9
Total time taken: 48065 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9434-t10949-kvm-alma9.zip
Smoke tests completed. 132 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File


Add project details to README

Closes #13

As discussed in a group meeting, we'll get this merged first and improve structure & readability in #20


Parse language field in YAML

Parse language field in YAML to generate project(<PROJECT_NAME <PROJECT_LANGUAGE>) in CMakeLists.txt
Why?
Delayed enabling of languages within the cc.cmake, cxx.cmake and fc.cmake causes CMake to load C and C++ as default languages for a project. This is in principle unnecessary for a project that only needs one language. The "only one language" behavior can be enforced by setting the language when declaring the project.
Questions

[ ] This line uses a ternary if to check whether a list of languages or a single language was given. It might not be exactly the most readable piece of code.
[x] The enable_language in cc.cmake, cxx.cmake and fc.cmake is probably redundant.

Interesting. I did not know there was a difference between declaring a language with project or enable_language. I liked that the language was defined in one place and this is now lost but I think your suggestion is better although one could now declare Fortran in the YAML and then include C via the module.
The ternary is a bit terse but I can then expand it and add a comment.
Indeed the enable_language is now redundant.
Do the broken tests have anything to do with this PR? You mentioned over coffee that this might need more testing or was this something else?

I did not know either. It came as a big surprise that for a Fortran-only project both the C and C++ compiler were searched for before the Fortran compiler. I think this need documenting inside Autocmake, i.e. that the language is set in the YAML file and in a module. If you don't have time writing docs, hold off merging until I can.
By the way, the language field in the YAML is now compulsory.

The fc_omp test is broken, as it seems CMake checks for the C OpenMP flag even though the language is not defined. I am puzzled by the fact that the if(DEFINED CMAKE_<LANG>_COMPILER_ID) checks are in place and should avoid these scenarios...

I cannot reproduce the problem locally. I will check whether this is linked to the CMake version.

Can you please update your branch with latest changes on upstream master? Now testing CMake 3.8.0 instead of an arcane 2.8. Hopefully this will resolve the problem.

Apologies for slow response. Fortran tests cases are failing Mac OS X.

No problem - if somebody is slow then it's me. Just for my reference saving your good suggestion: "Try using a layer xcode image 7.3".

And it worked! I think you can ignore the older XCode version on Travis. I will rebase and squash some of the commits so you get a cleaner history.

Awesome! Thank you so much for this important change and also for your patience with this.


Can't Connect to Smart Bridge

Describe Your Problem:
I've been using your bridge for about a year and never had any problem with it until today. I changed my router (had been Linksys Velop, now switched to TP-Link XE75pro). I use Hoobs. The bridge could not log in with the new router. So I removed the bridge and tried to set it all up as new, and it gets stuck on the Searching for SmartBridges page. My Lutron smart bridge works fine, so that's not the problem. And my other Hoobs bridges are all fine too. Perhaps the problem relates to port forwarding? I don't know, but would sure like to get it working again!
Plugin is 2.4.5-beta1
Hoobs 16.20.2 (Homebridge 1.6.0)
Thank you.

Plugin Version:
Homebridge Version:
Node.js Version:
NPM Version:
Operating System:

The plugin (and, indeed, all of Lutron's connectivity) relies on mDNS and DNS-SD, which requires that broadcast packets arrive. If you can see the Lutron device with your phone, but not on Homebridge, you'll need to figure out what's different between your Homebridge install and your phone (different WLAN? firewall rules?). Unfortunately, that's impossible to diagnose remotely, sorry.


lazyload routes

Is there any way of adding additional routes without causing a routie.reload?
e.g.
current hash is #text
routie('text', textFn);
if (imagesExist()) {
  routie('images', imagesFn);
}

results in textFn being triggered twice

@christopherdarling This is indeed a duplicate. I'm going to close the issue but feel free to continue the discussion in #30.


Added images for docs and changed images path

See exercism/x-common#69 and exercism/rossconf#1.

Thanks!


Add option to affect operation flags with value fomatting

The original implementation of ITypeTranscoder does not allow full customization of encoded values. It still binds the encoding by hard wired formatting flags based on the generic type of operation. The new implementation allows for full encoding customization like unwrapping of values passed in as object generic type.

@Svizel -
I pushed this to gerrit, will review later this week.
Thanks for the submission!
-Jeff

@Svizel -
This is merged; can you please close the PR?
Thanks,
-Jeff


[E2E] HF amp_fp16 and amp_bf16 training accuracy have 2 new models crash issue

🐛 Describe the bug
According to the latest weekly tests, there are 2 models crash issue in AMP_FP16 and AMP_BF16 training accuracy tests. Refer https://github.com/intel/torch-xpu-ops/actions/runs/10278413083/job/28442046973
Model List:

[ ] BartForCausalLM
[ ] BartForConditionalGeneration

Failures log:
xpu  train BartForConditionalGeneration
ERROR:common:
Traceback (most recent call last):
  File "/home/sdp/actions-runner/_work/torch-xpu-ops/pytorch/benchmarks/dynamo/common.py", line 2846, in check_accuracy
    new_result = optimized_model_iter_fn(model_copy, example_inputs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_dynamo/eval_frame.py", line 464, in _fn
    return fn(*args, **kwargs)
  File "/home/sdp/actions-runner/_work/torch-xpu-ops/pytorch/benchmarks/dynamo/common.py", line 2550, in run_n_iterations
    self.model_iter_fn(mod, inputs, collect_outputs=False)
  File "/home/sdp/actions-runner/_work/torch-xpu-ops/pytorch/benchmarks/dynamo/huggingface.py", line 521, in forward_and_backward_pass
    cloned_inputs = clone_inputs(inputs)
  File "/home/sdp/actions-runner/_work/torch-xpu-ops/pytorch/benchmarks/dynamo/huggingface.py", line 522, in torch_dynamo_resume_in_forward_and_backward_pass_at_521
    self.optimizer_zero_grad(mod)
  File "/home/sdp/actions-runner/_work/torch-xpu-ops/pytorch/benchmarks/dynamo/huggingface.py", line 524, in torch_dynamo_resume_in_forward_and_backward_pass_at_522
    pred = mod(**cloned_inputs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/transformers/models/bart/modeling_bart.py", line 1731, in forward
    outputs = self.model(
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/transformers/models/bart/modeling_bart.py", line 1599, in forward
    encoder_outputs = self.encoder(
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/transformers/models/bart/modeling_bart.py", line 1207, in forward
    layer_outputs = encoder_layer(
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1736, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1747, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/transformers/models/bart/modeling_bart.py", line 643, in forward
    def forward(
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_dynamo/eval_frame.py", line 631, in _fn
    return fn(*args, **kwargs)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_functorch/aot_autograd.py", line 1048, in forward
    return compiled_fn(full_args)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_functorch/_aot_autograd/runtime_wrappers.py", line 316, in runtime_wrapper
    all_outs = call_func_at_runtime_with_args(
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_functorch/_aot_autograd/utils.py", line 121, in call_func_at_runtime_with_args
    out = normalize_as_list(f(args))
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_functorch/_aot_autograd/utils.py", line 95, in g
    return f(*args)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/autograd/function.py", line 574, in apply
    return super().apply(*args, **kwargs)  # type: ignore[misc]
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_functorch/_aot_autograd/runtime_wrappers.py", line 1503, in forward
    fw_outs = call_func_at_runtime_with_args(
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_functorch/_aot_autograd/utils.py", line 121, in call_func_at_runtime_with_args
    out = normalize_as_list(f(args))
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_functorch/_aot_autograd/runtime_wrappers.py", line 500, in wrapper
    return compiled_fn(runtime_args)
  File "/home/sdp/miniforge3/envs/e2e_ci/lib/python3.10/site-packages/torch/_inductor/codecache.py", line 1412, in __call__
    return self.current_callable(inputs)
  File "/tmp/torchinductor_sdp/fe/cfe4hqo23icogcgdufcrcpz5dmijb5oev2xdml6mhkz5adosc4wo.py", line 679, in call
    extern_kernels.addmm(buf20, reinterpret_tensor(buf18, (1024, 1024), (1024, 1), 0), reinterpret_tensor(buf19, (1024, 1024), (1, 1024), 0), alpha=1, beta=1, out=buf21)
RuntimeError: expected mat1 and mat2 to have the same dtype, but got: float != c10::BFloat16
TorchDynamo optimized model failed to run because of following error
fail_to_run

Versions
Failure On-demand Test on 2024-08-07, See: https://github.com/intel/torch-xpu-ops/actions/runs/10278413083

Torch-xpu-ops
PyTorch
Triton

fb365ac on main
527f104 on HEAD
1b2f158

Transformers
Timm
Torchbench
Torchvision
Torchaudio

243e186
730b907
03cde49
d23a6e1
b3f6f51

Device
OS
GCC
Python
Driver(DKMS)
Bundle(DPCPP)

pytorch-06
Ubuntu 22.04.2 LTS
11
3.10
<IP_ADDRESS>.231129.50
2<IP_ADDRESS>40604

Inputs
huggingface/amp_bf16,amp_fp16/training/accuracy

This issue is related to the wrong datatype infer in sdp kernel. Will investigate further.

The issue is related to pytorch main regression. This issue also occurs in cuda, if forcing to use math sdp.

@retonym will create a pytorch issue to track it

PyTorch issue for this crash: https://github.com/pytorch/pytorch/issues/133974

Due to the fix in pytorch master, the models can pass now.


Back to Malmo on fedora 25, with a Python ImportError: libTH.so.

Hi Malmo,
one testimony :) and one help-request
It had been my pleasure to test malmo en redhat/fedoar a few moths ago and then had to switch to other projects, sorry, now back.
To witness :
- source package compiles entirey well (just to check)
- binary distribution is sufficient for a normal user like me and
- C++ example runs well with one patch
# - Adds to ./MalmoPlatform/Cpp_Examples/CMakeLists.txt
# 65a66
#   ${CMAKE_THREAD_LIBS_INIT} # to avoid a link error related to fc fun…nny threads install

To ask for help
- i also need to run Python missions and i fail on
Traceback (most recent call last):
  File "run_mission.py", line 20, in <module>
    import MalmoPython
ImportError: libTH.so.0: cannot open shared object file: No such file or directory

(i am a python-dummy :( ) it seems that libTH.so.0 might be related to torch that we do need and that i can not install on my fedora-25 distribution (or may be not) -> any one can help ?
Sincerely.

Hi @vthierry,
The first thing would be to check if torch library can be found on the system or not :
locate libTH
If nothing appear, Torch is not installed on your system. You'll need to follow the setup for Torch
Let us know.
Morgan

Hi Morgan,
Thanks for your prompt answer.

i confirm torch is not installes (e.g. locate libTH return is empty)
i did try http://torch.ch/docs/getting-started.html but get stucked by:
torch>bash install-deps;
Only Fedora 20-24 are supported for now, aborting.
while :
torch>uname -a
Linux kiti 4.9.3-200.fc25.x86_64

Now i reinstall Malmo from source https://github.com/Microsoft/malmo/blob/master/doc/build_linux.md and it works the Python_Examples/MalmoPython.so does not ask for torch.
So, fine for me; your feedbak helped me to move in the right direction.
Sincerely
Thierry.

Hi @vthierry,
Glad to hear that I finally worked someway for you.
I'm closing this issue for now, feel free to reopen/create a new one if needed :)
Morgan


Add Javadoc to keystone/critical classes and modules (esp. if their purpose is obscure/unclear)

Use Case
I am a Heroic new joiner
Who wants to understand the codebase's critical constructs
So that I can ramp up to speed as fast as possible
Notes
Here's a list (or a link to a list) of classes and modules that, should they be Javadoc'd, would speed the onboarding of new team members. Each team member may add their own.
Once this is done, the list will be normalised (i.e. no repeat entries) and prioritized for Javadoc'ing and/or README.md's :
peterk
Classes

**/Module.java
core/src/main/java/com/spotify/heroic/consumer/schemas/Spotify100.java
heroic-core/src/main/java/com/spotify/heroic/consumer/schemas/spotify100/Version.kt
heroic-component/src/main/java/com/spotify/heroic/common/Groups.kt
heroic-component/src/main/java/com/spotify/heroic/ingestion/Ingestion.java
heroic-component/src/main/java/com/spotify/heroic/metric/QueryResult.kt
heroic-core/src/main/java/com/spotify/heroic/CoreQueryManager.java
heroic-core/src/main/java/com/spotify/heroic/metric/LocalMetricManager.java
metric/bigtable/src/main/java/com/spotify/heroic/metric/bigtable/BigtableBackend.java

Modules/directories

heroic-consumer-pubsub
heroic-core
heroic-discovery-simple
heroic-dist
heroic-metadata-elasticsearch
heroic-metric-bigtable
heroic-rpc-grpc
heroic-suggest-elasticsearch
metadata/elasticsearch
metric/bigtable
rpc/grpc
statistics/semantic
suggest/elasticsearch

Hi @sjoeboo , @malish8632 , @lmuhlha - when you have 2 mins, please scrawl any classes and modules here (that don't have good Javadoc/README.md already) that you think would really benefit a new joiner, should they get properly documented.
I will then tidy up the scrawls, collate the list and present at a later date for prioritization by the team. I.e. I'll do the heavy lifting :)
Cheers.


[Enhancement] Method makes literal string comparisons passing the literal as an argument [EventHandler]

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request

located at:
eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/EventHandler.java line 108,194,197,200
analysis and explanation:
This line is in the form of String str = str.equals("someOtherString");
A NullPointerException may occur if the String variable str is null. If instead the code was restructured to String str = "someOtherString".equals(str);that is, call equals() on the string literal, passing the variable as an argument, then this exception could never happen as both equals() check for null.
Describe the solution you'd like
Adjust the code to String str = "someOtherString".equals(str).
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

I like to work on this issue.


Re-export AppHandle from the crate root.

Previously, the type name was inaccessible and could not be used as e.g. a field in a struct.

CI is complaining that cargo fmt is failing. Would you mind fixing that?

Thanks!


No paired region for France Central

According to https://azure.microsoft.com/mediahandler/files/resourcefiles/microsoft-azure-france-en-us/Azure_France_Datasheet_EN.pdf, paired region seems to be France South.
Can you confirm and update this article please?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 09a788e4-0130-73f2-d985-6fc3aec77974
Version Independent ID: a9f4ec3f-f0a3-b050-020b-f487c74c5179
Content: Business continuity and disaster recovery (BCDR): Azure Paired Regions
Content Source: articles/best-practices-availability-paired-regions.md
Service: multiple
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@ggirard07 Thanks for the feedback! We are currently investigating and will update you shortly.

@ggirard07 I have assigned the issue to the content author to investigate further and update the document as appropriate.


pyvistaqt install

Not sure if this can be classified as an issue.
On a clean windows 10 install I installed Anaconda (Anaconda3-2020.11-Windows-x86_64.exe)
After install, pyvista or pyvistaqt cannot be installed via the anaconda prompt (conda install -c conda-forge pyvistaqt) nor the Anaconda Navigator.
PIP install gives an SSL error.
I can install other packages.
Over the same network, my laptop (which has a previously upgraded anaconda version) installs without problem (same on my work pc on a different network).
Does anyone have any idea what is wrong here,

Not sure if this can be classified as an issue.

Hi @zoner72 , I'll try to help anyway

PIP install gives an SSL error.

Could you share the log?
Are you behind a proxy? After a short search online, I found the following:
pip install --trusted-host pypi.org --trusted-host pypi.python.org --trusted-host files.pythonhosted.org <package>

Reference: https://stackoverflow.com/questions/49943410/pip-ssl-error-on-windows

@GuillaumeFavelier
thanks for your help
I will retry to reproduce the PIP error when Im back home (currently at work) and post the log.
I initially I thought it was my internal network (new router), but after I tried on the laptop and the install went perfect I realised it must be pc related. I tried again on my work PC without problem.
Since it is a clean install with only Anaconda and Pycharm on the PC, Im a bit surprised it did not work 'out of the box'.
I did try the link you posted yesterday
The only thing I can think off is that the other PC's have python 3.9.1 latest version installed in addition to the full Anaconda installation.
I was wondering if there is a procedure to manually install pyvistaqt

I tried again on my work PC without problem.

This is good to hear and is useful as baseline.

The only thing I can think off is that the other PC's have python 3.9.1

Could be because of this. Although I just realize I missed the update of https://github.com/pyvista/pyvista/issues/1001, AFAIK pyvistaqt is not tested yet on python=3.9.*.
I just created a quick test env on my linux with:
conda create -n test-py39 python=3.9

And then I did:
conda activate test-py39
conda install -c conda-forge pyvista pyvistaqt

I was surprised to see that my basic script works as expected but I can't guarantee the same result on Windows.
As a short term solution, you could try on a different version of python.
It's just my guess unfortunately, the other @pyvista/developers might have a different perspective.

I was wondering if there is a procedure to manually install pyvistaqt

You could also try to install the package from source?
git clone https://github.com/pyvista/pyvistaqt.git
cd pyvistaqt
python -m pip install -e .

The commands above will install the latest version still in development.

pip:
WARNING: pip is configured with locations that require TLS/SSL, however the ssl module in Python is not available.
WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pyvistaqt/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pyvistaqt/
WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pyvistaqt/
WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pyvistaqt/
WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError("Can't connect to HTTPS URL because the SSL module is not available.")': /simple/pyvistaqt/
Could not fetch URL https://pypi.org/simple/pyvistaqt/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='pypi.org', port=443): Max retries exceeded with url: /simple/pyvistaqt/ (Caused by SSLError("Can't connect to HTTPS URL because the SSL module is not available.")) - skipping
ERROR: Could not find a version that satisfies the requirement pyvistaqt (from versions: none)
ERROR: No matching distribution found for pyvistaqt

pip after stackoverflow solution has the same result (as above)
Anaconda:
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.

Manual Install:
great success!
I never realise this is how the manual install works, so will definitely bookmark this.
pyvista now shows in my Anaconda environment and can use pyvistaqt in my code.
However,  pyvistaqt does not show in my anaconda environment, not sure if this is how it is suppose to be.
Thanks again for your help, really appreciated.
I hope someone else can maybe solve the problem or solve their install problems


Too long at this step: electrify:  installing electrified dependencies

Hi, I follow the instructions on my Linux Mint 17.1
meteor create --example leaderboard
cd leaderboard
meteor add arboleya:electrify
meteor

The process stalled at this:
I20150924-21:59:15.216(2)? electrify:  installing electrified dependencies
I'm waiting more than 20 minutes.
And:
du -hs .electrify/
16K	.electrify/

if it is worth to mention, I did this previously:
sudo npm install electrify -g

You're on OSX, I suppose?
Which version of meteor?

@arboleya
I'm on Linux Mint 17.1

Hi, I'm experiencing this too. Meteor <IP_ADDRESS> running on OSX 10.10.3. After:
meteor --verbose

I got this error:
=> App running at: http://localhost:3000/
I20150928-07:31:25.866(8)? electrify:  launching electron
W20150928-07:31:25.871(8)? (STDERR)
W20150928-07:31:25.872(8)? (STDERR) fs.js:439
W20150928-07:31:25.872(8)? (STDERR)   return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
W20150928-07:31:25.872(8)? (STDERR)                  ^
W20150928-07:31:25.872(8)? (STDERR) Error: ENOENT, no such file or directory '/Users/jacquelyng/hobbi_games/meteor/leaderboard/.electrify/node_modules/electron-prebuilt/path.txt'
W20150928-07:31:25.873(8)? (STDERR)     at Object.fs.openSync (fs.js:439:18)
W20150928-07:31:25.873(8)? (STDERR)     at Object.fs.readFileSync (fs.js:290:15)
W20150928-07:31:25.873(8)? (STDERR)     at Object.<anonymous> (/Users/jacquelyng/hobbi_games/meteor/leaderboard/.electrify/node_modules/electron-prebuilt/index.js:4:21)
W20150928-07:31:25.873(8)? (STDERR)     at Module._compile (module.js:456:26)
W20150928-07:31:25.874(8)? (STDERR)     at Object.Module._extensions..js (module.js:474:10)
W20150928-07:31:25.874(8)? (STDERR)     at Module.load (module.js:356:32)
W20150928-07:31:25.874(8)? (STDERR)     at Function.Module._load (module.js:312:12)
W20150928-07:31:25.875(8)? (STDERR)     at Module.require (module.js:364:17)
W20150928-07:31:25.875(8)? (STDERR)     at require (module.js:380:17)
W20150928-07:31:25.875(8)? (STDERR)     at Object.<anonymous> (../meteor/leaderboard/.electrify/node_modules/electron-prebuilt/cli.js:3:16)

Hello @eahmedshendy @jacquelynjg
I've just released a complete revamp version of the project, now it has tests and I'm confident it's way more stable than before.
Can you please update and report back if you still are facing the problem you've mentioned?
Some notes on upgrading:
https://github.com/arboleya/electrify#upgrading
Thanks in advance.

Should be fixed in latest version - 2.0.0.
In case the problem persists, please re-open.


IR receiver support

Hi
Is there IR receiver support?
Great library to work with!
Thanks in advance

No direct support in HomeSpan for IR receiving, just transmitting.  However, receiving is possible using the ESP32 RMT peripheral - you might be able to use HomeSpan transmitting class (RFControl) as a starting point and modify as needed to implement receiving logic.


remote execution

i am thinking about using goreman as a controller for job executions.
for example this one would run two jobs locally -
j1: curl "www.google.com" | process
j2: curl "www.yahoo.com" | process

and this one would run the job on a remote machine:
j1: curl "www.google.com" | process
j2: ssh w1 'curl "www.yahoo.com" | process'

what's the best way to achieve that?
also, is it possible to add handler for "on proc done" ?
(so i can use that event in order to clean up / copy out the result of the proc)

Use a command and control system like NATS.
then write a nats client that can control Goreman. Deploy the nats client on your servers.
you can even embed the Goreman binary I side the nats client so that it spits out Goreman at startup time to disk .
Then you can start up 100’s of servers and via another nats client on your desktop of in GitHub action control how many instance of Goreman in what .
ypu can even send the process file across using Nat object storage !

Do you mean sequencial processes with j1 to j2 ?

it has been 5 years :)
i wrote this -
https://github.com/korovkin/parallel

the only reason I suggested nats is because it can run anything on remote servers.
you don’t event need to compile anything because the nats cli and a bash scriot can run Goreman prof file on other servers.


Coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@6c41c9b). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev      #76   +/-   ##
======================================
  Coverage       ?   71.59%
======================================
  Files          ?       41
  Lines          ?     1507
  Branches       ?        0
======================================
  Hits           ?     1079
  Misses         ?      428
  Partials       ?        0

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 6c41c9b...8e1cd3d. Read the comment docs.


problem in creating an account

Thanks but when I am trying to create an account of me in the server , its saying problem undefined.. PFA the screenshot of the above mentioned issue.

I think we can close this issue?

yes, probably a problematic installation


slic set command is not included in slic help

...

doh, it's not a main level command... this works: slic php-version set 8.1


Proxy settings not being used

Ripple Framework is not using proxy settings. Ripple fails to load the UI in environments where a proxy is being used for internet access.

This is a duplicate of issue #87. We should close this one.

Sorry. Closing this one.


Enhancements to user and group configuration

Added a flag to specify if the user home directory should be created
Added ability to specify user UID
Added ability to specify group GID

I intentionally didn't add any validation because I'm not sure how much of it made sense. There are some reserved UID/GIDs and negative numbers aren't allowed, but I feel like if someone is going to set these manually, it's on them to know and not on us to enforce. I'm open to discussing it if anyone has strong opinions.

I’m fine with this landing without the extra validation as is, but I definitely think that (way) down the line, the more stuff that we can catch for the user the better. But at the same time, I agree when you say that a lot of this stuff falls on the user.

Not a part of this PR but if we have to introduce a new script, isn't it better to just push back the rest instead of using a/b as a sorting mechanism?

I thought about this, especially since we haven't shipped 1.0 yet. I actually intentionally used a/b here since in my head the functionality is "users and groups" and there's a relationship there. But I could also see the argument that not others would see that rationale (for example, the reason you asked the question), so perhaps that's more confusing than it's worth.


2 Issues with Create PCF instructions.  Please help fix.

Hello, thank you for writing the instructions to create a PCF.
May I suggest the following updates?
I spent an hour trying to figure out why I cannot package my code component and create a solution zip file.
Turns out there are 2 issues with the instructions:

"Create a new folder Solutions inside the LinearInputControl folder and navigate into the folder."

meant create a new folder Solutions inside the ROOT folder called LinearInputControl rather than the subfolder called LinearInputControl.  Following the instructions from top to bottom, you get 2 folders BOTH called LinearInputControl.  This is quite confusing when following the instructions.
May I suggest to update the instructions so that the ROOT folder is called LinearInput while the folder inside of it is called LinearInputControl?  (and update the following instructions according to the new name change).

"To generate a zip file from your solution project, when inside the the cdsproj solution project directory, using the following command: msbuild /t:restore"

This is NOT enough.  After running msbuild /t:restore, we also need to run msbuild as well in order to get the zip folder.
Will you also add to run msbuild as well?
Thanks, the updates to the instructions will help future developers.
-Thuy ("twee")-

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3fc8bc19-4e5e-b6f4-34ef-77b004c2afd6
Version Independent ID: 847c467c-0015-1eb9-bdc7-720a40f1cdba
Content: Create your first component using Power Apps Component Framework in Microsoft Dataverse - Power Apps
Content Source: powerapps-docs/developer/component-framework/implementing-controls-using-typescript.md
Service: powerapps
Sub-service: pcf
GitHub Login: @Nkrb
Microsoft Alias: nabuthuk

Thanks for contacting us. We have updated the docs and pushed it live. Closing the issue. If you need further help, please reopen or create a new issue.


Fix :class:~.Tex not working properly with OpenGL

Changelog / Overview

Utilized the ~.set_submobjects function for both renderer in the ~.break_up_by_substrings function.

Motivation

Previously, as far as I know, parents were not being added to the ~.Tex mobject when OpenGL renderer is used, which raised several issues.
Few problems that were resolved with this PR:
class Test1(Scene):
    def construct(self):
        title = Tex(r"This is some \LaTeX")
        self.play(Write(title))
        self.wait()
        transform_title = Tex("That was a bad transform")
        self.play(Transform(title, transform_title))
        self.wait()

https://user-images.githubusercontent.com/11293921/125080133-1e86cb80-e0cd-11eb-9961-86ce7ac7b3b9.mp4
class Test2(Scene):
    def construct(self):
        title = Tex(r"This is some \LaTeX")
        self.play(Write(title))
        self.wait()
        transform_title = MathTex("01234567890123456789")
        title.become(transform_title)
        self.wait()

https://user-images.githubusercontent.com/11293921/125080154-25add980-e0cd-11eb-9b83-4ff12c6f05f7.mp4
Testing Status

All tests pass.
Checklist

[X] I have read the Contributing Guidelines
[X] I have written a descriptive PR title (see top of PR template for examples)
[X] I have written a changelog entry for the PR or deem it unnecessary
[X] My new functions/classes either have a docstring or are private
[ ] My new functions/classes have tests added and (optional) examples in the docs
[X] My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

[ ] The PR title is descriptive enough
[ ] The PR is labeled correctly
[ ] The changelog entry is completed if necessary
[ ] Newly added functions/classes either have a docstring or are private
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

One more problem:
class Test3(Scene):
    def construct(self):
        title = Tex(r"This is some \LaTeX")
        self.play(Write(title))
        self.wait()
        transform_title = MathTex("0123").next_to(title, UP)
        title.become(transform_title)
        self.wait()

https://user-images.githubusercontent.com/11293921/125080775-e16f0900-e0cd-11eb-9fb8-751d8e806f3f.mp4


Do you really need a personal access token?

This looks really useful although I'm very reluctant to create a PAT.
Have you experimented to see whether this can be replaced with a packages: write permission? i.e.
name: Delete old container images

on:
  schedule:
    - cron: '0 0 * * *'  # every day at midnight

permissions:
  packages: write

jobs:
  clean-ghcr:
    name: Delete old unused container images
    runs-on: ubuntu-latest
    steps:
      - name: Delete 'dev' containers older than a week
        uses: snok/container-retention-policy@v1
        with:
          image-names: python-dev, js-dev
          cut-off: A week ago UTC
          account-type: org
          org-name: my-org
          keep-at-least: 1
          untagged-only: true

This certainly lets you push and pull containers but I don't know if delete is included in write. I wondered if you knew one way or the other.

Things change all the time, but last time I checked a PAT was required. Definitely understand your hesitation though - not entirely comfortable with it either.

Thanks for the quick reply. I will experiment and feedback just in case it has changed.

Please do 🥇 And I guess we can leave this open for a while in case anyone else has relevant input 👍

It hasn't changed. A personal access token is still required 😢
Would be great to understand if packages/delete will make it to the GITHUB_TOKEN permissions anytime soon @NamrataJha et al.

I can confirm that, at the moment, an own personal access token with packages: delete permission is required.
GITHUB_TOKEN is not enough at the current moment.

This GitHub blog post states GITHUB_TOKEN is supported in delete operations

I just tried switching in one of my workflows, and it resulted in a 500 internal server error :thinking:

This GitHub blog post states GITHUB_TOKEN is supported in delete operations

The article is referring to a different type of API endpoint used by docker CLI to login, push, pull, rm images... that type of API is in fact supported by ${{ secrets.GITHUB_TOKEN }} and is used by Docker Login Action. However, afaik, docker cannot remove container images from private repositories.
To delete private container images, you need access via https://api.github.com/orgs/github/packages/container, which as of now is not supported by ${{ secrets.GITHUB_TOKEN }}.

It hasn't changed. A personal access token is still required :cry:

I've opened a community discussion on this topic over here:

https://github.com/community/community/discussions/50123

Feel free to comment on or upvote the discussion to boost it's visibility/priority.

Did anyone test fine-grained tokens https://github.blog/2022-10-18-introducing-fine-grained-personal-access-tokens-for-github/ for that?

Not sure if something has changed with the API, but the github provided action for cleaning up packages does work with the standard runner token.  As does this action.
Unfortunately neither provide the option for excluding an image from deletion by tag 😞 .

The other actions query the packages individually, and that operation works with the default token, as long as the repository can manage its own packages:

https://api.github.com/orgs/{organization}/packages?package_type=container 🚫
https://api.github.com/orgs/{organization}/packages/container/{name}/versions ✅

@0x2b3bfa0 this is also my observation. In case we can identify the special case of only completely specified package names, it should be possible to use without a PAT similar to what I did in https://github.com/Chizkiyahu/delete-untagged-ghcr-action/pull/27/commits/3202d0501dfc8d1e1e1f84b126219b9978eb803e.

The other actions query the packages individually, and that operation works with the default token, as long as the repository can manage its own packages:
* [https://api.github.com/orgs/{organization}/packages?package_type=container](https://api.github.com/orgs/%7Borganization%7D/packages?package_type=container) 🚫

* [https://api.github.com/orgs/{organization}/packages/container/{name}/versions](https://api.github.com/orgs/%7Borganization%7D/packages/container/%7Bname%7D/versions) ✅

This action doesn't have an easy migration path to the latter, because it also supports regular expressions for package names.

I have implemented this suggestion in the newest release. The action now supports:

Personal access tokens (classic ones)
secrets.GITHUB_TOKEN tokens, with some caveats - see details here
Tokens generated with a Github app

The new version is ready now, and the migration guide for v3 is included in the release post 👍
If you run into any issues, please share them in the issue opened for tracking the v3 release ☺


💡[FEATURE]: Add a favicon to the website

Is your feature request related to a problem? Please describe.
The website doesn't have a favicon. It needs a favicon for symbolic representation of the website
Describe the solution you'd like
I would like to add a favicon to the website which will represent the view of the website.
Additional context

hey @Abidsyed25 i would like to contribute the issue under gssoc'24.

@SnehansuBehera  Add any related icon


Bulk insert

From Gitter:
@petermattis: Peter Mattis @petermattis May 23 17:33
@aquarat For bulk loading data, you should insert lots of rows in a single statement. I’d recommend something in the 100-1000 range. Inserting a row at a time is going to be very slow in comparison. This recommendation applies to all SQL databases, not just CockroachDB.

Another question from Gitter:
WANG QIANG @wangqiang8511 03:39
I am new to cockroach db. Have couple of questions. What is the most efficient way to do the bulk insert? And the primary key is used as partition key?

Added to forum: https://forum.cockroachlabs.com/t/whats-the-best-way-to-do-bulk-insert/58

From gitter on 7/14/16:
Arnold Villatoro @Heyarrnolld_twitter 13:24
I'll double check
Also, is there an easier way to insert massive amounts data into the database?
Currently, I am doing this: cockroach sql --insecure --host localhost --port 26257 --user root --database databasename < file.sql
I have about 650 files, each around 400 MB of data
I know I can just put this command in a for loop and loop over the files, but one file alone takes quite a bit of time to actually insert all the contents into the DB

Response from @tschottdorf:
Tobias Schottdorf @tschottdorf 13:47
No, currently that’s it. We’ve got to make them faster, but have been working mostly on stability lately. Here’s an issue you may want to track about this: cockroachdb/cockroach#5981
There are also some ways to make bulk inserts faster today which we ought to document. They involve setting the replication factor to one and the maximal range size to infinity (so that you really load only a single node and don’t do much distributed systems stuff). Were you inserting against a multi-node cluster at all?
cc @jseldess about documenting this (I suppose we should also test it thoroughly on big datasets first)

Also from @mjibson:
Matt Jibson @mjibson 13:51
another minor improvement is to do ~100 rows per insert statement instead of 1-1

We answer this in faqs and on https://www.cockroachlabs.com/docs/import-data.html, so closing this.


Fahdkmsft/2022 06 01 preview registries

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following appy to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Please select the correct checkboxes in the Changelog section. This looks like a new api version rather than an update for an existing api version.
please follow the steps correctly if you are creating a new api version :
https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/208/OpenAPI-Hub-Adding-new-API-version
or if doing it manually :
https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/83/Manual-Process-Adding-new-API-version

Please respond to the comments and remove the "ARMChangesRequested" label when this PR is ready for review again .

Abandoned.


Fix noop assignment in HeliosSoloDeployment

@rohansingh @mattnworb

:+1:

Current coverage is 44.90%

Merging #812 into master will increase coverage by +0.02% as of d43c410

@@            master    #812   diff @@
======================================
  Files          259     259
  Stmts        10852   10852
  Branches      1511    1511
  Methods          0       0
======================================
+ Hit           4871    4873     +2
  Partial        432     432
+ Missed        5549    5547     -2

Review entire Coverage Diff as of d43c410

Powered by Codecov. Updated on successful CI builds.


Import issue

When following best-practises for tsconfig.json (here: https://www.totaltypescript.com/tsconfig-cheat-sheet), there is an import issue.
Import code:
import * as ftrackWidget from '@ftrack/web-widget';

Error:

Could not find a declaration file for module '@ftrack/web-widget'. '/workspaces/WATS/node_modules/.pnpm/@ftrack+web-widget<EMAIL_ADDRESS>implicitly has an 'any' type.
There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@ftrack/web-widget' library may need to update its package.json or typings.

Here is my full tsconfig:
{
  "compileOnSave": false,
  "compilerOptions": {
    "rootDir": ".",
    "strict": true,
    "sourceMap": true,
    "declaration": false,
    "allowSyntheticDefaultImports": true,
    "noUncheckedIndexedAccess": true,
    "moduleResolution": "Bundler",
    "esModuleInterop": true,
    "importHelpers": true,
    "target": "es2022",
    "module": "ESNext",
    "allowJs": true,
    "resolveJsonModule": true,
    "moduleDetection": "auto",
    "isolatedModules": true,
    "noEmit": true,
    "noEmitHelpers": false,
    "lib": [
      "ES2022",
      "DOM",
      "DOM.Iterable"
    ],
    "skipLibCheck": true,
    "skipDefaultLibCheck": true,
    "baseUrl": ".",
    "forceConsistentCasingInFileNames": true,
    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,
    "types": [
      "reflect-metadata",
      "jest",
      "node"
    ],
    "paths": {
    }
  },
  "exclude": [
    "**/node_modules",
    "tmp"
  ],
  "ts-node": {
    "require": [
      "tsconfig-paths/register"
    ]
  }
}

The offending setting is isolatedModules: true.
But that should be possible strictness-wise. The Ftrack JavaScript API does not have this issue.

It seems the offending setting is moduleResolution: "Bundler".
But that is recommended in our scenario.

Thanks for the report. I'll look into it!

That was quick! Thanks! 😍


Changed the heap size to the correct MCU heap size for SRAM1

The heap size was wrong for the mbed MICRO online builds. This should fix it.
L4 Nucleo and Disco versions.

@bcostm @adustm

Thanks for the update.
Yes for the HAL driver, we try to keep it updated but it required times as we need to verify at least the rebuild of all targets and perform some basic tests to verify nothing is broken... The update of the L4 HAL driver is actually on-going.