MRG, MAINT: Use context manager for tests

@GuillaumeFavelier I think that in theory this should be equivalent to what is in master but cleaner because it uses a contextmanager.

I think that in theory this should be equivalent to what is in master

I totally agree, I had something similar in mind. It's just that I would like to avoid to manipulate directly pyvista.OFF_SCREEN somehow and maybe use a new fixture like renderer_interactive

Also, it's an item of #7162

The tests pass now, which is nice. I wonder if it's random variation, or maybe something actually wasn't getting set properly in master but is now ...?
I can try turning it into renderer_interactive, should be doable

The tests pass now, which is nice. I wonder if it's random variation, or maybe something actually wasn't getting set properly in master but is now ...?

Where did it fail?

Ahh I thought master had been failing, but maybe it was just the TimeViewer PR(s)

@GuillaumeFavelier feel free to merge if you're happy

... and then you could rebase / merge into #7247


3.0.4 Crash from my app's Glide

I get crash message :
java.lang.ClassCastException: com.github.irshulx.GlideRequests cannot be cast to com.my.app.GlideRequests at com.funliday.app.core.GlideApp.with(GlideApp.java:88)
And get some information:
Avoid AppGlideModule in libraries

Libraries must not include AppGlideModule implementations. Doing so will prevent any applications that depend on the library from managing their dependencies or configuring options like Glide’s cache sizes and locations.

In addition, if two libraries include AppGlideModules, applications will be unable to compile if they depend on both and will be forced to pick one or other other.

This does mean that libraries won’t be able to use Glide’s generated API, but loads with the standard RequestBuilder and RequestOptions will still work just fine (see the options page for examples).

from
http://bumptech.github.io/glide/doc/configuration.html#avoid-appglidemodule-in-libraries

And I found
Libraries

Libraries that do not register custom components do not need to perform any configuration steps and can skip the sections on this page entirely.

Libraries that do need to register a custom component, like a ModelLoader, can do the following:

Add one or more LibraryGlideModule implementations that register the new components.
Add the @GlideModule annotation to every LibraryGlideModule implementation
Add a dependency on Glide’s annotation processor.
An example LibraryGlideModule from Glide’s OkHttp integration library looks like this:

@GlideModule
public final class OkHttpLibraryGlideModule extends LibraryGlideModule {
  @Override
  public void registerComponents(Context context, Glide glide, Registry registry) {
    registry.replace(GlideUrl.class, InputStream.class, new OkHttpUrlLoader.Factory());
  }
}
Using the @GlideModule annotation requires a dependency on Glide’s annotations:

compile 'com.github.bumptech.glide:annotations:4.9.0'

from
http://bumptech.github.io/glide/doc/configuration.html#libraries

Dear irshuLx
Can you help me?

I had the same problem

There's never been a solution to this problem?


How can I use image drag-drop (or upload) like in teachable machine site instead of just webcam feed?

will a little of googling i found this https://gist.github.com/chris-chris/8e41e4431e808b55ba2691171f13591b


Fix composer in mWeb safari is always auto focused

Details

In mWeb/native, the composer will auto focus if the chat is not empty, but on mWeb Safari, the composer will always be auto focused.
Fixed Issues

$ https://github.com/Expensify/App/issues/31708
PROPOSAL: https://github.com/Expensify/App/issues/31708#issuecomment-1823197048
Tests

Same as QA Steps

[x] Verify that no errors appear in the JS console

Offline tests

Same as QA Steps
QA Steps

A

Open a chat with existing messages
On web/desktop, verify the composer is auto-focused and the cursor is at the end of the input/text
On mWeb/Android/iOS, verify the composer is not auto-focused

B

Open an empty chat without existing messages
Type anything in the composer
Close the chat/switch to another chat
Reopen the empty chat
Verify the composer is auto-focused and the cursor is at the end of the input/text

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

A
https://github.com/Expensify/App/assets/50919443/4bb71ad7-bdac-47a7-8a17-907280982c2e
B
https://github.com/Expensify/App/assets/50919443/7e0f8217-6d9d-4ccd-b2de-6edac903d9e2

Android: mWeb Chrome

A
https://github.com/Expensify/App/assets/50919443/0b8f74d4-6481-48b8-adcb-e9213d09e84e
B
https://github.com/Expensify/App/assets/50919443/33473b20-2a83-44cb-9deb-15c3e26acb6a

iOS: Native

A
https://github.com/Expensify/App/assets/50919443/e795a45c-b3a7-4bef-8c79-afbed152d595
B
https://github.com/Expensify/App/assets/50919443/047d59bb-6cee-42d8-a6ae-6c7da7b8fef0

iOS: mWeb Safari

A
https://github.com/Expensify/App/assets/50919443/eef4d4f2-875d-4d8a-86aa-586abe77c678
B
https://github.com/Expensify/App/assets/50919443/92b18b37-3f5f-4a7e-8a78-55639b870623

MacOS: Chrome / Safari

A
https://github.com/Expensify/App/assets/50919443/3c7516f5-5cc2-4814-a2f3-052255c89a47
B
https://github.com/Expensify/App/assets/50919443/cc018dd2-5929-4e3c-bb73-d76e510f15b6

MacOS: Desktop

A
https://github.com/Expensify/App/assets/50919443/84e37c6b-f176-4974-932f-b8cf6dd80924
B
https://github.com/Expensify/App/assets/50919443/eebaa842-b743-4fe3-af99-577ad30c3c0b

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/dangrous in version: 1.4.7-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.4.7-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Sorting option that separates directories and files

Is your feature request related to a problem? Please describe.
The sorting method of the file browser currently sorts directories and files in a combined manner. It also displays the listing in descending order. This is different to the general conventions adopted by other code editors and file browsing tools.
Currently, files and directories are all sorted as follows:
C-file.md
› C-directory/
B-file.md
› B-directory/
A-file.md
› A-directory/
› ../

However, In vim/neovim's built-in netrw plugin, the same listing is sorted as follows:
A-directory/
B-directory/
C-directory/
A-file.md
B-file.md
C-file.md

In VS Code's explorer, it is listed as:
> A-directory
> B-directory
> C-directory
A-file.md
B-file.md
C-file.md

In GitHub's repo view:
A-directory
B-directory
C-directory
A-file.md
B-file.md
C-file.md

The same is true with many other file browsers out there. Whilst I appreciate that the Telescope file browser is intended to be navigated using fuzzy searches, it can be difficult to visually get your bearings whilst browsing a project. Especially if you have been browsing the same directory using a different tool.
Describe the solution you'd like
The option to sort directories and files separately and in ascending order (so that it matches the conventions of other file browsers).
So that the listing is as follows:
› A-directory/
› B-directory/
› C-directory/
A-file.md
B-file.md
C-file.md
› ../

Describe alternatives you've considered
Not sure if there are any alternative approaches I could try.
Additional context
n/a
Thanks!

As per :h fb_picker.file_browser
To have initial sorting as desired, :Telescope file_browser grouped=true or grouped = true to the lua function calling the picker or extension setup.
Closing as I'm pretty sure that's it. Let me know in case it's not.

Doh! You're correct, this can already be controlled via the options. To get the behaviour described above:
require('telescope').setup({
    extensions = {
      file_browser = {
	grouped = true,
	sorting_strategy = 'ascending',
      },
    },
})

Thanks for your help!


npm was not the lastest code

https://www.npmjs.com/package/react-native-svg?activeTab=explore
android/build.gradle code is not for rn-0.71.x

Yes, since the release has been made before adding changes to android/build.gradle. It is available soon in v13.8.0 so I will close this issue. Feel free to comment here if something is wrong and we can always reopen it then.


Update puppet scripts for OB200

Purpose
To Run 200 obiam, obam , obbi for UK,Berlin, AU.
Goals
To run and test OB-2.0.0 products.
Approach
Compared OB200 scripts and configs and updated the changes.
Tested with all specs.
User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

WUM updates for UK MI Reporting siddhi files for UK 3.1.5 were also released. Shall we create a issue to track this requirement to add UK 3.1.5 siddhi-files also to puppet.

Please do the changes required for the readme file and @isharailanga please review that

Hi @DivyaPremanantha @DoryFazna,
Ack. Please keep me posted.

WUM updates for UK MI Reporting siddhi files for UK 3.1.5 were also released

Created the issue. Thanks

Please do the changes required for the readme file and @isharailanga please review that

Updated. Thanks.

Please do the changes required for the readme file and @isharailanga please review that

Hi @DivyaPremanantha @DoryFazna,
Ack. Please keep me posted.

Updated the README file. Please check.

Please do the changes required for the readme file and @isharailanga please review that

Hi @DivyaPremanantha @DoryFazna,
Ack. Please keep me posted.

Updated the README file. Please review.

Line 26 in Readme:
Add file and use specification instead of Spec as follows:
Update the <puppet_environment>/modules/ob_common/manifests/params.pp file of Puppetmaster according to the Open Banking specification (UK, AU, Berlin). The default value is set as UK.

There is few more updates needs to be done to the following template file.
https://raw.githubusercontent.com/wso2/puppet-open-banking/master/modules/obam/templates/carbon-home/repository/resources/api_templates/velocity_template.xml.erb
There are few new changes in WUM related to "cds-admin" in the velocity_template.xml. Please add those changes as well.

There is few more updates needs to be done to the following template file.
https://raw.githubusercontent.com/wso2/puppet-open-banking/master/modules/obam/templates/carbon-home/repository/resources/api_templates/velocity_template.xml.erb
There are few new changes in WUM related to "cds-admin" in the velocity_template.xml. Please add those changes as well.

Updated. Thanks.


Added a TFDS and Croissant notebook

@marcenacp a first version of the notebook. I will add a further section with a working example of training a model in a subsequent PR.


Stabilize

There is function change.
Changes are for makefile, create deb folder and use submodule so that can make deb package

Are we updating this to gom based dependency management similar to https://github.com/ooyala/atlantis-router/pull/26.


godot: convex shapes are sometimes incorrect when using blenders mesh

There are extra triangles sometimes generated with the mesh.
Some faces on the collision shape have "missing" or bugged collisions where there seems to be an extra vertex making a cross over an existing face.
Previously, I would save a lot of data in the gltf by recycling the mesh, and surely this mesh should generate a convex mesh quite fine as its using the same vertexes.
it could be one of these reasons:

Godot is adding the extra vertex, as a bug.

The blender mesh is not suitable for a collision shape, maybe the vertexes are being generated in some incorrect order, leading to an extra vertex to try and "fill" the shape (basically not godot's fault)

The BoxShape character controller I'm using for tests possibly might have issues against this kind of shape.

~4) The mesh is somehow incorrect because I did something wrong with the file and it can be fixed in blender.~

https://github.com/godotengine/godot/issues/95690
I need to provide a MRP / .gltf example
still have no idea about this, but I'm leaning more towards blenders shapes being "incorrect" for convex shapes. possibly to do with vertex clockwise vs counter clockwise ordering? will have to test more.

it seems like blender is making funky shapes when using the convex hull geometry node. more testing needed.


missing epilepsy warning :-)

blink/flash effect removed with a few modifications, rendering is a bit slower though.
let me know if you want a PR for this 😊
https://github.com/armel/DRADIS/assets/1893754/26bde9b6-ab8a-4460-8676-0e41591e24ea

Hi :)
I was sure you would find a way to solve this problem :) I'm interested by your PR of course :) Thank you.
Armel.


Add license information for LibreJS compatibility.

These commits

add javascript sources and license information following the FSFs guidelines (https://www.gnu.org/software/librejs/free-your-javascript.html). This lets the users a) examine what code gets exectuted in their browser and b) learn what code is available under a free license, so as to facilitate copying. With the added sources and license information users of the GNU LibreJS add-on, which blocks non-free javascript by default, will also be able to display the website correctly (with the exception of Twitter's follow button which doesn't seem to be free).

change a few http URLs to https

fix indentation here and there.

By the way, "dist/bootstrap-decorator.js" does not appear to be the source file for "dist/bootstrap-decorator.min". (In "jslicense.html" I say it is.)

https://github.com/Textalk/angular-schema-form/blob/master/src/directives/decorators/bootstrap/bootstrap-decorator.js should be the source


Zeroes in hundredths decimal place are getting deleted

hundredths decimal are being deleted if it is a 0 with aPad: false
I need a currency format, something like this:
$1,000.10
If I set aPad: false I get this:
$1,000.1
Also I should omit decimals if the user doesn't type, the user just can type
$1,000
$1,000.10 and $1,000 both should be valid in the field
$(this).autoNumeric('init',{aSep: ',', aDec: '.', aSign: '$', aPad: false});

Please post a jsfiddle and I can see the details
Robert (Bob) Knothe

On Apr 13, 2015, at 6:10 PM, Luis Nicanor<EMAIL_ADDRESS>wrote:
hundredths decimal are being deleted if it is a 0 with aPad: false
I need a currency format, something like this:
$1,000.10
If I set aPad: false I get this:
$1,000.1
Also I should omit decimals if the user doesn't type, the user just can type
$1,000
$1,000.10 and $1,000 both should be valid in the field
$(this).autoNumeric('init',{aSep: ',', aDec: '.', aSign: '$', aPad: false});
—
Reply to this email directly or view it on GitHub.

I need these valid formats for a currency in the field
$1,000.10
$1,000
$1,000.11
https://jsfiddle.net/unoqa1mh/
Thanks for your time

Currently aN does support that but there maybe a work around.
Untested but try testing the value on keyup and use the update method to change the padding option.
Let me know if that works.
Bob
Robert (Bob) Knothe

On Apr 13, 2015, at 6:49 PM, Luis Nicanor<EMAIL_ADDRESS>wrote:
I need these valid formats for a currency in the field
$1,000.10
$1,000
$1,000.11
https://jsfiddle.net/unoqa1mh/
Thanks for your time
—
Reply to this email directly or view it on GitHub.

It is not recommended, but I have to change the regex:
var regex = (rDec === 0) ? (/(.(?:\d*[1-9])?)0*$/) : rDec === 1 ? (/(.\d(?:\d*[1-9])?)0*$/) : new RegExp('(\.\d{' + rDec + '}(?:\d*[1-9])?)0*$');
to this
var regex = (rDec === 0) ? (/(.(?:\d*[1-9])?0*)$/) : rDec === 1 ? (/(.\d(?:\d*[1-9])?)0*$/) : new RegExp('(\.\d{' + rDec + '}(?:\d*[1-9])?)0*$');
I will try to use the update method.
Thanks

You may consider the focusin an focusout events for the update method.
Robert (Bob) Knothe

On Apr 14, 2015, at 11:39 AM, Luis Nicanor<EMAIL_ADDRESS>wrote:
It is not recommended, but I have to change the regex:
var regex = (rDec === 0) ? (/(.(?:\d[1-9])?)0$/) : rDec === 1 ? (/(.\d(?:\d[1-9])?)0$/) : new RegExp('(.\d{' + rDec + '}(?:\d[1-9])?)0$');
to this
var regex = (rDec === 0) ? (/(.(?:\d[1-9])?0)$/) : rDec === 1 ? (/(.\d(?:\d[1-9])?)0$/) : new RegExp('(.\d{' + rDec + '}(?:\d[1-9])?)0$');
I will try to use the update method.
Thanks
—
Reply to this email directly or view it on GitHub.

$(selector).focusout(function(){
                if($(this).val().match(/(\.[1-9])0*$/g) != null){
                    $(this).autoNumeric('update', {aSep: ',', aDec: '.', aSign: '$', aPad: true});
                }
                else{
                    $(this).autoNumeric('update',{aSep: ',', aDec: '.', aSign: '$', aPad: false});
                }
});

Glad this worked - FYI - you only need to pass options that are changing
$(selector).focusout(function(){ if($(this).val().match(/(.[1-9])0*$/g) != null){   $(this).autoNumeric('update', {aPad: true});  } else{   $(this).autoNumeric('update',{aPad: false});  }  });
--------- Original Message --------- Subject: Re: [autoNumeric] Zeroes in hundredths decimal place are getting deleted (#128)
From: "Luis Nicanor"<EMAIL_ADDRESS>Date: 4/14/15 11:40 am
To: "BobKnothe/autoNumeric"<EMAIL_ADDRESS>Cc: "Bob Knothe"<EMAIL_ADDRESS>$(selector).focusout(function(){ if($(this).val().match(/(.[1-9])0*$/g) != null){ $(this).autoNumeric('update', {aSep: ',', aDec: '.', aSign: '$', aPad: true}); } else{ $(this).autoNumeric('update',{aSep: ',', aDec: '.', aSign: '$', aPad: false}); } });  -
Reply to this email directly or view it on GitHub.


Add JS release workflows based on maplibre-gl-js

As suggested in #177, we could start publishing prerelease versions of the JS module in order to use it downstream.
This PR:

Attempts to duplicate the relevant GitHub Actions workflows from maplibre-gl-js
Adds @maplibre/ to the name

@HarelM I attempted to copy over the relevant GitHub Actions work here as you mentioned in #177. Would this work for you? If so I think I'll mostly need the secrets to be configured so we can publish.

I just added NPM_ORG_TOKEN to this repo, you may need to restart CI

I guess the dry-run is intended, right?

For now yes, didn't want to inadvertently publish something before we're ready.

Looks good. You should be able to test bump pr action and after it is merged a new version should be created in main (dry run)

@HarelM Great! Should I leave --dry-run on for now as-is for testing?

I would also recommend creating a RELEASE.md file describing the steps to carve out a new release as this might be done in large time intervals, and one might forget how to do it.


File upload component - It will let you upload multiple files even "multiple is false" in case of drag and drop.

when multiple is false and you are trying to upload a file by clicking on the upload link its working fine, replacing the old file with the new one.
But if you drag a file to upload it will let you upload the multiple files.
i am using -
"carbon-components": "^10.9.3",
"carbon-components-angular": "^3.38.0",

Duplicate of #1045 ?

Looks like it ... closing in favor of #1045


Refactor/core delimited control

The following is a minimal example that segfaults in LLVM
effect Get() : Int

def run(n: Int) = {
  try {
    do Get()
  } with Get { () =>
    val x = 4;
    resume(x)
  }
}

def main() = println(run(5))

LLDB tells me it is a return-continuation (the one of the reset, I guess)
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=2, address=0x100003758)
    frame #0: 0x000000010000375c countdown`returnAddress_27 + 12
countdown`returnAddress_27:
->  0x10000375c <+12>: str    x9, [x2, #0x8]
    0x100003760 <+16>: ldur   x2, [x8, #-0x18]
    0x100003764 <+20>: br     x2

Maybe not surprisingly removing the resume from the example (which will drop the return-continuation of the reset) fixes the issue.
More surprisingly, changing it to try { 42 } ... also fixes the issue, while it needs to run the return continuation. Potentially shift-resume corrupts the stack and the frame on it...

Here is the generated machine code from your example
def main_3675 = {
  push { (run_3810 : Int) =>
    subst [v_r_3697_3772 !-> run_3810];
    let pureApp_3809 = show_14(v_r_3697_3772);
    let pureApp_3808 = println_1(pureApp_3809);
    subst [v_r_32232_3784 !-> pureApp_3808];
    return v_r_32232_3784
  };
  let p3_3788 = reset { (returned_3811 : Int) =>
    return returned_3811
  };
  push { (v_coe_3770117_3801 : Positive) =>
    let pureApp_3812 = unboxInt_297(v_coe_3770117_3801);
    return pureApp_3812
  };
  let k732_3804 = shift0p p3_3788;
  let longLiteral_3813 = 4;
  subst [x1065_3806 !-> longLiteral_3813];
  resume k732_3804;
  return x1065_3806
};
jump main_3675

It tries to unbox an integer that isn't boxed in the first place.

Ok, I think this is ready to go.
We should do two things in the future:

disable the optimizer by always returning tree here: https://github.com/effekt-lang/effekt/blob/dc9149737c9d713ce04a26591d77d8cdfccf50e7/effekt/shared/src/main/scala/effekt/core/Optimizer.scala#L25 and observe that many tests fail in LLVM (maybe @phischu can look into this?)
investigate why the two interesting use cases of bidirectional handlers still need to be commented out (maybe @serkm can look into this?)

However, this is still a big step forwards to supporting bidirectional handlers uniformly across backends! Thanks for the help @serkm, @marzipankaiser, and @phischu!


Need to setup automated tests

Express-autodocs has a need to improve the development environment for this project, I feel adding unit tests, setting up eslint would help.

Hey, I would like to work on setting up eslint. What kind of styling are you looking for? @Pika1998

Hey @rahil1304  ESLint has been already setup, checkout this PR4
Let me know if you would like to work on something else :)

Okay yeah my bad. Are there any pending issues that I could work on? @Pika1998

@rahil1304 yes, actually people are finding it difficult to pass JSON in the comments to define the API's it would be better to implement a JSDoc Like Parser Check this issue
Also I have thought of two future plans

Creating a workflow generator website (as a helper for those who are finding it difficult to integrate this action)
[ I am working on a Figma prototype for this which will be shared in the issues soon]
Adding more output formats for this action (e.g React documentation etc)

Okay, will check this issue out.
I was also wondering since you already have tests set up in your repo, why not integrate Travis CI to automate the testing. So every time somebody submits a PR, the tests will be run to make sure the code is not broken. @Pika1998

I don't have tests in the sense unit tests, I have added a github action for tests ( I don't know about Travis CI ),
Can you check this test workflow
I am not running npm run test (Because I have not yet written any custom tests using chai or mocha)

So basically Travis CI is a continuous integration service used to build and test software projects. So we can set up your project using Travis CI, so every time a change is made, it is built automatically to make sure the code is not broken. What I can do is add a travis.yml file that will contain the requirements for this project (similar to what is already done in the node.js.yml file). And then you can just link your repository to travis CI and that's it you're done. Your repository is setup with Travis CI. @Pika1998

So to give you some more details on Travis, we can add checks for the node version, checks for your other packages, run npm install, and build your project automatically. So basically before merging a PR, these tests will be checked to make sure there is no issue. If any of these tests fail, means the new feature is not working with your project, and it can automatically inform the contributor.

@rahil1304 I got it, but we can add similar tests using github action, so why travis CI, the workflow file I have can build the project run all the scripts we want like npm install npm run test etc
So is there any particular reason why we need to install Travis CI explicitly?

No particular reason. Just thought that in the future, when/if the code base becomes more complex it would be more useful.
Anyways, no issues. I would love to start off by working on this issue https://github.com/Pika1998/express-autodocs/issues/4, could you assign it to me?

Sure cool 😁

No particular reason. Just thought that in the future, when/if the code base becomes more complex it would be more useful.
Anyways, no issues. I would love to start off by working on this issue https://github.com/Pika1998/express-autodocs/issues/4, could you assign it to me? @Pika1998

Can you just comment on that issue, so that I can assign you 😁


how to close stdin

I'm trying to create a container, launch a process (let's say 'wc -c' for the example), send data to its stdin and read the response (number of bytes in this example).
It all works fine if I use:
      optsc.AttachStdin = true;
      optsc.AttachStout = true;
      optsc.AttachSterr = true;
      optsc.Tty = true;
      optsc.OpenStdin = true;

as long as the data written to stdin is less than 4096 bytes, after that it goes wrong, i.e. wc returns "1" when I send 4097 bytes. I also tried writing by smaller chunks using a bufferstream to respect the stream's readiness.
I tried not using tty=true, but in that case I can't figure how to send EOT to stdin and the command never exits. I tried using writing '\n\x04', using stream.end or stream.destroy as per #53 without success.
I guess I'm not doing this right, but not too sure how, the example uses the tty=true, the unit test does not use EOT (it uses bash's exit command).
sample code (based on the mocha unit tests) to reproduce the issue :
    it("should support attach with stdin enable", function(done) {
      this.timeout(120000);

      var length = 4096; //set to 4095 to see it working fine, note that stdin will be 1char longer because of the terminating `\nctrl+d`

      function randomString(length) {
        var result = '', chars = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
        for (var i = length; i > 0; --i) result += chars[Math.round(Math.random() * (chars.length - 1))];
        return result;
      }

      function handler(err, container) {
        expect(err).to.be.null;
        expect(container).to.be.ok;

        var attach_opts = {
          stream: true,
          stdin: true,
          stdout: true,
          stderr: true
        };
        container.attach(attach_opts, function handler(err, stream) {
          expect(err).to.be.null;
          expect(stream).to.be.ok;

          var memStream = new MemoryStream();
          var output = '';
          memStream.on('data', function(data) {
            output += data.toString();
          });
          memStream.on('end', function(data) {
            console.log('memStream.end');
          });
          stream.pipe(memStream);

          container.start(function(err, data) {
            expect(err).to.be.null;
            stream.write(randomString(length)+'\n\x04');

            container.wait(function(err, data) {
              expect(err).to.be.null;
              expect(data).to.be.ok;
              output = output.trim();
              console.log('received:'+output);
              expect(output).to.match(new RegExp('.*'+(length+1)+'$'));
              done();
            });
          });
        });
      }

      optsc.AttachStdin = true;
      optsc.AttachStout = true;
      optsc.AttachSterr = true;
      optsc.Tty = true
      optsc.OpenStdin = true;

      optsc.Cmd = ['wc', '-c'];

      docker.createContainer(optsc, handler);
    });

Any sample code that would allow me to run 'wc -c', sending more than 4096 bytes to stdin and reading the number of bytes from stdout would be great :)
Did

Check this out https://github.com/apocas/dockerode/issues/53

Thanks,
for the quick response, I knew about #53 and tried stream.end and stream.destroy without success (docker-modem@0.2.2). I guess I don’t understand #53, could you help me understand how I should use the destroy method ?
Thanks,
Did

On 18 May 2015, at 13:39, Pedro Dias<EMAIL_ADDRESS>wrote:
Check this out #53
—
Reply to this email directly or view it on GitHub.

Any other advice? with tty=true it fails after 4096 bytes sent via stdin and with tty=false it never receives stdin. what's the proper way to send EOT with tty=false?

Verify that you really have an HttpDuplex object and the end method should work.
No need to use destroy.

I disagree with you this is just a question, as far as I can tell these are 2 bugs. It is trivial to repro. Just try running the following code :
'use strict';
var util = require('util');
var assert = require('assert');
var Docker = require('dockerode');
var docker = new Docker();
var MemoryStream = require('memorystream');
var stdinSize = 4097;
var optsc = {
'Hostname': '',
'User': '',
'AttachStdin': true,
'AttachStdout': true,
'AttachStderr': true,
'Tty': false,
'OpenStdin': true,
'StdinOnce': false,
'Env': null,
'Cmd': ['wc', '-c'],
'Dns': ['<IP_ADDRESS>', '<IP_ADDRESS>'],
'Image': 'ubuntu',
'Volumes': {},
'VolumesFrom': ''
};
function randomString(size) {
var result = '', chars = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
for (var i = size; i > 0; --i) { result += chars[Math.round(Math.random() * (chars.length - 1))]; }
return result;
}
function handler(err, container) {
var attach_opts = {
stream: true,
stdin: true,
stdout: true,
stderr: true
};
container.attach(attach_opts, function handler(err, stream) {
var memStream = new MemoryStream();
var output = '';
memStream.on('data', function(data) {
  output += data.toString();
});
memStream.on('end', function() {
  console.log('memStream.end, received: '+output.length+' bytes');
});

stream.pipe(memStream);

container.start(function() {

  stream.end(randomString(stdinSize));
  container.wait(function() {
    output = output.trim();
    console.log('received:'+output);
    assert(new RegExp('.*'+(stdinSize)+'$').test(output));
  });
});

});
}
docker.createContainer(optsc, handler);
I am sure that the above stream is an instance of httpduplex. It would be great to get a useful suggestion.
Did

On 19 May 2015, at 15:39, Pedro Dias<EMAIL_ADDRESS>wrote:
Verify that you really have an HttpDuplex object and the end method should work.
No need to use destroy.
—
Reply to this email directly or view it on GitHub https://github.com/apocas/dockerode/issues/136#issuecomment-103498147.

I'm using end in production without any problems since it was implemented in https://github.com/apocas/dockerode/issues/53, although can't confirm Docker versions or anything right now.
When I have time I will try to reproduce your example. Don't know when it will be.
If you are in a hurry try to do some debug in https://github.com/apocas/docker-modem/blob/master/lib/http_duplex.js to get some details on the issue, since stdin is managed in here.

Confirmed by https://github.com/apocas/dockerode/pull/146

Check the tests now.
The was not being truncated it is sent into the container, was just being sent in multiple "chunks" into "wc" which mess things up.
I rewrote the test check it out, it even has the tty disabled like you wanted so it also answers your second doubt.
This use case is a bit... "tricky", dockerode is the simplest part in all this it just send the data. You have the data being sent, piped, processed, displayed...
I was seeing in the stdout stream the 5k bytes and when dumping them into a file I saw them inside the container filesystem, so I just rewrote your docker usage keeping the same philosophy of sending the data in the duplexstream :)


2.1.0

Thanks for a great plugin, @shornuk!!
This PR adds the following:

A Base URI setting, making it possible to connect to the API when self-hosting Plausible (resolves https://github.com/shornuk/craft-plausible/issues/11).
A Top Countries dashboard widget.

Awesome. Thanks @bencroker - adding this in now.


Create in-toto attestations based on the Pipeline definition

Feature request

An in-toto layout defines the steps of your software supply chain that you carry out in order to write, test, package and distribute your software. I feel the Tekton Pipeline could create an attestation with all the tasks defined in a Pipeline. This can be used to ensure the integrity of the triggered pipeline.
Use case

/kind question


Bot doesn’t remove listings when stops v3.3.3

Describe the bug
Bot doesn’t remove listings when stops
To Reproduce
Steps to reproduce the behavior:

Use pm2 on Windows
Stop the bot
Check listings on backpack.tf
See that they are not removed

Expected behavior
Listings removal
Screenshots and logs

Already closed but not fixed issue: https://github.com/Nicklason/tf2-automatic/issues/305
Version: 3.3.3, pm2, Windows

Don't open a new issue, just reopen the old one

Don't open a new issue, just reopen the old one

Didn't know you couldn't reopen an issue if you are not a contributor

https://github.com/Nicklason/tf2-automatic/issues/305

The problem is caused by PM2 not telling the process to stop, I'll look for a fix.


Dockerfile uses --openssl-legacy-provider flag for node

The dockerfiles recently got updated to node17.
This cut off support for old algorithms (see the release notes).
In our particular case, the problem is react-scripts that depends on an too-old version of webpack. See this discussion.
To make the app buildable with node17, the flag --openssl-legacy-provider was added to the client.dockerfile.
As soon as the new version of react-scripts is released, this flag should be removed again.

This also applies to the github action workflow.


Explore UI - Incorrect url

Affected module
UI
Describe the bug
When I click Explore -> Topics or Dashboards or Pipelines -> result title,
URL is incorrect.
To Reproduce
Screenshots or steps to reproduce

Expected behavior
URL is should be correct URL
not messagingService, messagingServices is correct. s is missing.
(dashboardService, pipelineService is same problems. But Tables url is correct.)
Version:

OS: [e.g. iOS]
Python version:
OpenMetadata version: 1.0.0
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

Duplicate issue, already being tracked here


🐛 BUG: When VsCode setting "organizeImports" is enabled, saving will result in "crashing"

Describe the Bug
In VsCode's settings.json its possible to automatically organize imports with this setting:
 "editor.codeActionsOnSave": {
      "source.organizeImports": true
    }

If this is enabled and the astro language support is enabled, saving will make vscode freeze in .astro files.
I think the bug only occurs when the order of import is actually changed automatically.
As soon as I add the following to my settings.json I can save the file without a problem:
 "[astro]": {
    "editor.codeActionsOnSave": {
      "source.organizeImports": false
    }
  }

Note: I am working on an M1 Mac
Steps to Reproduce

Install VSCode
Install Astro Language Support
Set organizeImportsto true
Create an .astro file that imports multiple components in an "incorrect" order
Try to save the file

Hmm, I am not able to reproduce. On my computer, imports automatically get reorganized on save as expected.
Do you have more information? Does this happens on specific files, specific amount of imports etc?

Interesting,  I mean maybe its a problem on my machine for some reason?
I suspected that it has to do with my project, which is a quite large monorepo. But I could also reproduce the Problem on this example project: https://github.com/mhyfritz/astro-landing-page
this is my complete settings.json
{
  "workbench.iconTheme": "material-icon-theme",
  "workbench.colorTheme": "Electron",
  "[typescript]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[javascript]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[typescriptreact]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "editor.formatOnSave": true,
  "javascript.updateImportsOnFileMove.enabled": "always",
  "editor.codeActionsOnSave": {
    "source.organizeImports": true
  },
  "diffEditor.ignoreTrimWhitespace": false,
  "[scss]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[jsonc]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[vue]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "cSpell.userWords": ["Fastify"],
  "[json]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "typescript.updateImportsOnFileMove.enabled": "always",
  "[html]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "files.associations": {
    "*.astro": "astro"
  },
  "[astro]": {
    "editor.codeActionsOnSave": {
      "source.organizeImports": true
    }
  }
}

I have a bunch of other extensions installed, but I think most of them should not be relevant. Here are the ones that seem relevant to me:

Prettier
Path Intellisense
IntelliCode
EsLint
Electron Color Theme

Also I want to point out that without the organizeImports setting I have no problems, so even for me this is not very important. I dont want to waste your time, if you feel like it I understand if you just close the Issue :D

Hi, same issue for me, I also tried a test with this:
{
	"astro.typescript.codeActions.enabled": false
}

And it also had the same result for not getting into a save-format loop. I also suspect the issue within the organizeImports setting.

Marking as a bug, but I still cannot reproduce 😕
Is there any output in the Output panel for the Astro extension when this happens?

Having same issue. Nothing in output tab

Seems like it's not only Astro related
microsoft/vscode#165326

I reported something similar here: #444

VS Code 1.73.1 was released with a fix for this. Thank you to everyone who reported this issue!


Improve support debugging

Currently debugging support questions is a bit of a pain. Requiring a lot of copy-paste of different IDs & events and searching through one-by-one until you reach the root cause.
Would be massively improved with:

[x] Integrate Posthog with Sentry to tie exceptions to session events
[x] Implement Correlation IDs in logging to tie all logging events to one particular request that can more easily be debugged
More logging. Ongoing effort vs. one-off fix here.

Funding

You can sponsor this specific effort via a Polar.sh pledge below
We receive the pledge once the issue is completed & verified

[ ] Integrate Sentry for Next.js SSR

Solved by the following PR:

#1079
#1075
#1079
#1083

Basically, I tweaked the setup of Sentry and Posthog so they report more data in the events. They're now tied together so errors are reported in Posthog (with a link to Sentry) and Sentry reports show a link back to the affected user in Posthog.
All logs now also show a correlation ID so they can be grouped easily in monitoring. I also took the opportunity to tweak the configuration so third-party loggers (namely uvicorn, sqlalchemy and arq) share the same one and adopt our formatting. Output is much cleaner this way!

So nice! This is going to be so helpful and an incredibly nice DX enhancement. Thanks @frankie567!


Problem running AutoCellSeg in MacOS

Dear Angelo,
I installed MATLAB_R2016b on my mac and when I tried to run the AutoCellSeg with the following command
./run_AutoCellSeg.sh /Applications/MATLAB/MATLAB_Runtime/v91
I got the following error messages and the program popped up a Download window and stuck at 0%. Do you know what can be problem?
Thanks,
Martin

Setting up environment variables
DYLD_LIBRARY_PATH is .:/Applications/MATLAB/MATLAB_Runtime/v91/runtime/maci64:/Applications/MATLAB/MATLAB_Runtime/v91/bin/maci64:/Applications/MATLAB/MATLAB_Runtime/v91/sys/os/maci64
2019-10-10 14:55:45.713 AutoCellSeg[79553:64753414] Unknown class 'Download', using 'NSObject' instead. Encountered in Interface Builder file at path /Users/martinwu/Downloads/AutoCellSeg-master/Mac/AutoCellSeg.app/Contents/Resources/en.lproj/MainMenu.nib.
2019-10-10 14:55:45.713 AutoCellSeg[79553:64753414] Failed to connect (downloadPercent) outlet from (NSObject) to (NSTextFieldCell): missing setter or instance variable
2019-10-10 14:55:45.713 AutoCellSeg[79553:64753414] Failed to connect (image) outlet from (NSObject) to (NSImageView): missing setter or instance variable
2019-10-10 14:55:45.713 AutoCellSeg[79553:64753414] Failed to connect (progress) outlet from (NSObject) to (NSProgressIndicator): missing setter or instance variable
2019-10-10 14:55:45.713 AutoCellSeg[79553:64753414] Failed to connect (window) outlet from (NSObject) to (NSWindow): missing setter or instance variable
d-172-25-48-254:Mac martinwu$ ./run_AutoCellSeg.sh /Applications/MATLAB/MATLAB_Runtime/v91

It looks like the issue has been raised before. Hopefully the error messages may help you figure out the issues. I tried executing AutoCellSeg.app directly as you suggested but it froze at the same Download window.

If you do this in terminal:
DYLD_LIBRARY_PATH=.:/Applications/MATLAB/MATLAB_Runtime/v91/runtime/maci64:/Applications/MATLAB/MATLAB_Runtime/v91/bin/maci64:/Applications/MATLAB/MATLAB_Runtime/v91/sys/os/maci64
export DYLD_LIBRARY_PATH

And then add execution primision to AutoCellSeg.app/Contents/MacOS/setup:
chmod u+x AutoCellSeg.app/Contents/MacOS/setup

The download and installation worked for me. And I am able to run the newly installed AutoCellSeg.app.

If you do this in terminal:
DYLD_LIBRARY_PATH=.:/Applications/MATLAB/MATLAB_Runtime/v91/runtime/maci64:/Applications/MATLAB/MATLAB_Runtime/v91/bin/maci64:/Applications/MATLAB/MATLAB_Runtime/v91/sys/os/maci64
export DYLD_LIBRARY_PATH

And then add execution primision to AutoCellSeg.app/Contents/MacOS/setup:
chmod u+x AutoCellSeg.app/Contents/MacOS/setup

Then run it:
AutoCellSeg.app/Contents/MacOS/setup

The download and installation then will start. Follow the instruction and install a new copy of AutoCellSeg.app.
After finishing the above, I was able to run the newly installed AutoCellSeg.app.

I tried this method. But it shows "zsh: segmentation fault  AutoCellSeg.app/Contents/MacOS/setup"
Not sure what happened. This tool sounds very attractive, but it seems to be a bit hard to install. any chance that there is a conda package for rookie user?

zsh: segmentation fault AutoCellSeg.app/Contents/MacOS/setup means that you are using the wrong binary file. But I guess you are on ARM Mac right? With M1 chip or higher. Try turn on rosetta for command line and see what you get.

Thank you. I'm using 2017 Mac with intel i5, not with M1. So I guess rosetta is not the problem? what did you mean by "wrong binary file"?

Thank you. I'm using 2017 Mac with intel i5, not with M1. So I guess rosetta is not the problem? what did you mean by "wrong binary file"?

I think it means that the program is not compiled using your architecture. But since you are using a 2017 Mac. The problem might be more complicated than that. Which I have no idea about.


migrating from jekyll-paginate to v2 with a bundle theme

Using minimal-mistakes gem theme and trying to move to jekyll-paginate-v2. This theme uses jekyll-data to populate defaults in _config.yml. I should be able to wipe out the theme's values with:
paginate: nil
paginate_path: nil

This does not seem to do the trick. Jekyll build throws the following:
Config Warning: The paginate key must be a positive integer or nil. It's currently set to 'nil'.
Default values keep  pushing through as if jekyll-paginate-v2 doesn't read the nil correctly.
Any workaround or fix?
Reference issue: https://github.com/mmistakes/minimal-mistakes/issues/1240

Hi Doug,
Sorry to hear that you're having issues with the gem. However the error message you mention doesn't seem to originate from the paginate-v2 gem.
The paginate-v2 gem listens for a non nil value in the legacy settings and runs the legacy code in that case.
https://github.com/sverrirs/jekyll-paginate-v2/blob/master/lib/jekyll-paginate-v2/generator/paginationGenerator.rb#L28
Try this:
If you run jekyll with the --verbose switch, do you see the message "Pagination: Starting" being printed? This is an indication that the  paginate-v2 code is being run. If the error is thrown before this line is seen then the issue lies with something else, if it shows up after then let me know and I will dive into the issue further.
Cheers!
Sverrir S.

The original error message I received was:
Pagination: You are running jekyll-paginate backwards compatible pagination logic. Please ignore all earlier warnings displayed related to the old jekyll-paginate gem.
Pagination: Detected legacy jekyll-paginate logic running. Please update your configs to use the jekyll-paginate-v2 logic. This compatibility function will stop working after Jan 1st 2018 and your site build will throw an error.

My attempt at disabling jekyll-paginate was to configure _config.yml:
paginate: nil
paginate_path: nil

Which is where I ended up flagging whether jekyll-paginate-v2 understood a nil setting.
Perhaps there is a way to 'force' jekyll-paginate-v2 to ignore the legacy config? That would address my issue as well.

Let me try this on my end, I'm really surprised that setting the values to nil don't trigger the new code. It might take me a few days to find time to spend on this. Hope you are alright with a few days wait :)

totally cool with waiting.
Thanks for taking a look

Looking at this closer this error is actually coming from jekyll itself https://github.com/jekyll/jekyll/blob/master/lib/jekyll/configuration.rb#L255 as it is trying to do validation on these pagination keys. I've asked the Jekyll core team to reconsider this code in the Jekyll core https://github.com/jekyll/jekyll/issues/6391

However regarding setting the old paginate values explicitly to nil
The paginate-v2 code already handles this case and the warning that is printed out can be safely ignored.
When setting my config file to include
# Explicitly turn off the old code
paginate: nil
paginate_path: nil

and then running jekyll build
I get this print out (includes the warnings) but the output in the resulting site\ folder looks good.
Configuration file: E:/git/jekyll-paginate-v2/examples/01-typicalblog/_config.yml
    Config Warning: The `paginate` key must be a positive integer or nil. It's currently set to '"nil"'.
    Config Warning: The `paginate` key must be a positive integer or nil. It's currently set to 'nil'.
            Source: E:/git/jekyll-paginate-v2/examples/01-typicalblog
       Destination: E:/git/jekyll-paginate-v2/examples/01-typicalblog/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
         AutoPages: Disabled/Not configured in site.config.
        Pagination: Complete, processed 1 pagination page(s)
                    done in 0.45 seconds.
 Auto-regeneration: disabled. Use --watch to enable.

I am afraid I need to hand you back over to the @mmistakes theme builder as it looks like this issue needs resolving on his end.

I'm closing this issue now but if there are further developments don't hesitate re-opening it and pinging me again :)

Thanks for the work on this. Nothing is as simple as it seems.


ValueError: Dimension 0 in both shapes must be equal,

Hello selfOcracle
Can you please fix this issue.
raise ValueError(str(e))
ValueError: Dimension 0 in both shapes must be equal, but are 153600 and 38400. Shapes are [153600,512] and [38400,512]. for 'Assign_42' (op: 'Assign') with input shapes: [153600,512], [38400,512].

Hi，
I faced the same bugs. Did you solved it?


Curve lending bot: coin info is NULL in the bot-info API

{"deposit_id":"0","token0":"0","token1":"0","amount0":"5000000000000000","amount1":"38287906260352484","depositor":"0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2","withdrawer":"0x944cF2acB0DB10d0863fE45C4916B2fd7005C6a4","dex_name":"curve","bot":"lending_curve","contract":"0xe0b6D0c1b41eFE8d1E5B3DDf6b503Db1CeA9553C","sent_to_mixpanel":"0","create_to_mixpanel":"0","created_at":"2023-10-04 21:16:14","coin_info":null,"coin_info1":null,"coin_info0":null}]

server didn't have latest code


[1.36.0] Android 14 - SecurityException

Hi.
"As discussed at Google I/O 2023, registering receivers with intention using the RECEIVER_EXPORTED / RECEIVER_NOT_EXPORTED flag was introduced as part of Android 13 and is now a requirement for apps running on Android 14 or higher (U+).
If you do not implement this, the system will throw a security exception."
So i get a SecurityException crash on Android 14 in few test-cases
To Reproduce

Start non short test-cases (my test contain 3-5 flow) on Android 14 device/emulator.
Crash SecurityException

Expected behavior
On previous android api (33-) everything works fine
Screenshots

Environment information (please complete the following information):

Maestro version [1.36.0]
Platform: [Android 14]
Framework: [Native Android]
Device model and OS version: [Clear android]
Simulator, emulator or physical device - both
Host [Windows WSL]

Additional context
I tried to register broadcast receiver like
registerReceiver(
   receiver,
   IntentFilter(Intent.ACTION_ALL_APPS),
   RECEIVER_EXPORTED/RECEIVER_NOT_EXPORTED
)

but it's give nothing, maybe i need to set some specific IntentFilter, but which?

Hey @SergeyKornyushin,

To Reproduce

Start non short test-cases (my test contain 3-5 flow) on Android 14 device/emulator.
Crash SecurityExceptio

This issue is not reproducible. We have a huge backlog, so it's important we can triage issues in an efficient manner.
Please provide a minimal, reproducible example that clearly explains how to reproduce the problem with the detailed, step-by-step instructions (that can be easily run by me or other maintainers).


remove unuse dependency

Description
I had some issue about grpc and node-gyp during installing the @zk-email/helper(with pnpm add). After analysis, pki doesn't be imported and I can't find the source code for the lib.
Type of Change
Remove unused dependency.

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

Checklist:

[ ] I have discussed with the team prior to submitting this PR
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings
[ ] New and existing unit tests pass locally with my changes

Sorry, didn't see the PR earlier.
Thanks for the debug :-) Merged and released v6.0.3


Updated Keycloak to 12.0.2

Updated Keycloak to 12.0.2

Have you checked the breaking changes from 10.x to 12.x ?

@mschulte-tsi  ?

Have you checked the breaking changes from 10.x to 12.x ?

@mschulte-tsi ?

Yes there are no breaking changes https://www.keycloak.org/docs/latest/release_notes/


No Module named stringify

Describe the bug
Unable to install pygame_gui.
To Reproduce
Steps to reproduce the behaviour:

python -m pip install pygame-gui

Expected behaviour
Installation of pygame_gui
Screenshots
None
Platform and software (please complete the following information):

OS: Windows 10 (64bit)
Pygame GUI version: 0.4.0
Pygame version: 1.9.6 / 2.0.0dev6

Additional context
Tried two versions of pygame.
Google suggested possibly a Path issue. Confirmed python's Path points to the project directory.
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Gabriel\AppData\Local\Temp\pip-install-56tyfvta\pygame-gui\setup.py", line 2, in
from stringify import stringify_py
ModuleNotFoundError: No module named 'stringify'
----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

I installed stringify via pip and I was then able install pygame GUI. I'm not sure if pygame GUI needs me to do so but I'll close the issue.

Thanks for the report! This is actually fixed in the master branch already, as it happens - I noticed it a few days ago when setting up continuous integration. You only need stringify if you are going to build the library from source yourself (and only really if you change the default data and are trying to make a PyInstaller executable). If you just do:
pip install pygame_gui
it should install for end use.

Using just pip install pygame_gui I get an error on module object is not callable, which is certainly a local issue.
Using python -m pip install pygame-gui gives the stringify error above. I just uninstalled them both to test. ;)

Ah, interesting I see a lot of people are having trouble with regular pip install calls online. Well, it should vanish when I do the next release on PyPi, but I might hurry that along now I've seen a few people having the same problem.


Remove pre-extract of bazel binary

fixes #90
pre-extraction causes many issues with transmitting the extracted bazel files to rbe (preserving timestamps, various files which aren't copied for some reason).
I think this might also fix #88 since I haven't seen it since running with this mode.
The failures are linting related :(

Wdyt about merging this and trying to iterate on the TreeArtifacts solution?
Also,
I think part of the problem is that Bazel fails if mtime of any of the
files is not in the future (years away) BUT RBE doesn’t preserve mtime
since that would make cache hits irrelevant.
We can stamp the mtime in the invocation but I’m not sure there aren’t
other caveats.
I want the pre extract benefit but I’ve spent many many hours iterating on
this issue and this solution at least works now.
On Fri, 28 Sep 2018 at 10:27 Damien Martin-Guillerez <
<EMAIL_ADDRESS>wrote:

@damienmg approved this pull request.
I guess that means you don't even want to try unbundling bazel in a
separate rule, anyway it's fine with me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazel-integration-testing/pull/91#pullrequestreview-159745427,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF2Orx2xi_PCEIuGZ2WVn9qZDMZUpks5ufc9PgaJpZM4W9cTU
.

I am fine with merging this one (that's what I meant earlier). I don't even
know if we need to do the treeartifact solution. Wix were the one hitting
the slowness iirc so it is up to you, if it's not a big pain it might not
be worth doing more.
On Fri, Sep 28, 2018, 9:46 AM Ittai Zeidman<EMAIL_ADDRESS>wrote:

Wdyt about merging this and trying to iterate on the TreeArtifacts
solution?
Also,
I think part of the problem is that Bazel fails if mtime of any of the
files is not in the future (years away) BUT RBE doesn’t preserve mtime
since that would make cache hits irrelevant.
We can stamp the mtime in the invocation but I’m not sure there aren’t
other caveats.
I want the pre extract benefit but I’ve spent many many hours iterating on
this issue and this solution at least works now.
On Fri, 28 Sep 2018 at 10:27 Damien Martin-Guillerez <
<EMAIL_ADDRESS>wrote:

@damienmg approved this pull request.
I guess that means you don't even want to try unbundling bazel in a
separate rule, anyway it's fine with me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/bazelbuild/bazel-integration-testing/pull/91#pullrequestreview-159745427
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ABUIF2Orx2xi_PCEIuGZ2WVn9qZDMZUpks5ufc9PgaJpZM4W9cTU
.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazel-integration-testing/pull/91#issuecomment-425351913,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADjHfyPLOwnSntUm_qPdDJT5yaE2VPNBks5ufdPugaJpZM4W9cTU
.


Add target to apply clang-tidy fixes

When I run the example with:
bazel build //example --aspects clang_tidy/clang_tidy.bzl%clang_tidy_aspect --output_groups=report

this silently succeeds. If instead I run
bazel build //... --aspects clang_tidy/clang_tidy.bzl%clang_tidy_aspect --output_groups=report

I get the expected lint warning.
I get tidy fixes applied with
bazel run :apply_fixes

or
bazel run :apply_fixes //...

any update?

@oliverlee
Hi, tried to use the fix patch, however, it seems to be confused by sandbox.
I tried to add back no-sandbox inside https://github.com/erenon/bazel_clang_tidy/commit/99ed470c25033baafe9ddd0634bc269f15b64b16, it succeed in fixing the std::string problem.
dongxu@MiDX01:~/ws/micah/tmp/bazel_clang_tidy$ bazelisk run :apply_fixes //...
INFO: Invocation ID: 2334cead-3d8a-4851-9396-e18a3046da53
INFO: Analyzed target //:apply_fixes (1 packages loaded, 1 target configured).
INFO: Found 1 target...
Target //:apply_fixes up-to-date:
  bazel-bin/clang_tidy.apply_fixes.sh
INFO: Elapsed time: 0.045s, Critical Path: 0.00s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
INFO: Running command line: bazel-bin/clang_tidy.apply_fixes.sh //...
Described file '/home/dongxu/.cache/bazel/_bazel_dongxu/7cea99f25702216cc5bfbca1e10eeac0/sandbox/linux-sandbox/11/execroot/bazel_clang_tidy/example/lib.cpp' doesn't exist. Ignoring...
Described file '/home/dongxu/.cache/bazel/_bazel_dongxu/7cea99f25702216cc5bfbca1e10eeac0/sandbox/linux-sandbox/11/execroot/bazel_clang_tidy/example/lib.hpp' doesn't exist. Ignoring...


Fix example

The class method decode does not exist for Base64.

This is a good opportunity to ask the question whether we think calling method is OK here. It causes the service not to start because the method doesn't exist, maybe it should always be a lambda?

I think it is good to support method because give devs more possibilities. It seems good to me as well the service not to start because the method does not exist, it is easier to point errors.

Merged


Breakout & beam to grouproom.io

The idea is from a kollokvium session enable users the possibillity to breakout / beam into an a new or ongoing grouproom.io meeting.  Myabe @chrismatthieu have drop a few lines ideas ?

We have recently added "Zoom" integration to GroupRoom (https://medium.com/grouproom/zoom-integration-685363fd38f0). This feature will actually work with any web based meeting platform such as Kollokvium.
The idea is that users can hangout in GroupRoom before a meeting and be automatically redirected to the meeting when it starts by passing in the meeting url and start time as query parameters into GroupRoom.
It would be really cool if the Kollokvium could redirect users back to GroupRoom after the meeting. Perhaps this could be done by checking the referrrer of the initial request. If rerferror === 'grouproom.io' redirect. Open to discussion and suggestions.

We have recently added "Zoom" integration to GroupRoom (https://medium.com/grouproom/zoom-integration-685363fd38f0). This feature will actually work with any web based meeting platform such as Kollokvium.
The idea is that users can hangout in GroupRoom before a meeting and be automatically redirected to the meeting when it starts by passing in the meeting url and start time as query parameters into GroupRoom.
It would be really cool if the Kollokvium could redirect users back to GroupRoom after the meeting. Perhaps this could be done by checking the referrrer of the initial request. If rerferror === 'grouproom.io' redirect. Open to discussion and suggestions.

@chrismatthieu is see a solution that allows users from i.e kollo do  breakout to a room in GroupRoom,  how we do it ? using referers, or API calls, to set up and autojoin?
I more see, the "integration" in a way , that "proper" meetings / gatherings , can he held on GroupRoom.   Coworking and smaller meetings works fine in a traditinal platform, but if we have gatherings, and vitrtual office spaces,
Maybe we need to think outside the box ?

@chrismatthieu is see a solution that allows users from i.e kollo do  breakout to a room in GroupRoom,  how we do it ? using referers, or API calls, to set up and autojoin?
I more see, the "integration" in a way , that "proper" meetings / gatherings , can he held on GroupRoom.   Coworking and smaller meetings works fine in a traditinal platform, but if we have gatherings, and vitrtual office spaces,
Maybe we need to think outside the box ?

The beam idea, i can explain futher, we may need a session online regaring this,  Beam is just very breif description  :)

The beam idea, i can explain futher, we may need a session online regaring this,  Beam is just very breif description  :)


docs(readme): correct command to run the CLI and only use npm i -g

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Changes to createNotificationConfig

Signed-off-by: Aditya Jindal<EMAIL_ADDRESS>Description
Makes the API call to index the Notification Config Asynchronous
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

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

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

Codecov Report

Merging #380 (65d2c30) into main (99baa27) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 65d2c30 differs from pull request most recent head 4b86858. Consider uploading reports for the commit 4b86858 to get more accurate results

@@           Coverage Diff           @@
##             main     #380   +/-   ##
=======================================
  Coverage   83.25%   83.25%
=======================================
  Files          66       66
  Lines        1893     1893
  Branches      493      493
=======================================
  Hits         1576     1576
  Misses        310      310
  Partials        7        7

Flag
Coverage Δ

dashboards-notifications
83.25% <ø> (ø)

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

Continue to review full report at Codecov.

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


Feature: compare accordion

An additional PR to Accordion feature which extends it to handle changing from normal accordion to the compare accordion. Visit http://localhost/ndcs/bra to view all features:

Button in header to switch between compare and regular
sync all compare and active country comparisons in the url
only render additional definition list data inside accordion if in compare mode

Note: this is to demo the compare function inside a single accordion component. The accordion could be adapted to accept children but the store model for this view is quite specific. Please let me know your thoughts. The reason I have made this way is to create the simplest set of components possible.
Comments: I think there must be a nicer way to manage these url changes outside of the component, maybe creating a helper to do so.

I'd prefer to have a new page to the compare feature because I think it makes the things easier to handle, because:

The header is different and we need to include the dropdown.
We can parametrize the Accordion and merge the requested data in this new component.

So for example to compare Brazil and Spain I'll go for this url /compare/bra/esp

We need to think about in a very different approach, so let's close this and use the other branch.


Fixed cookies parsing

correct handling of cookies values with equal sign (sid="c2lkDQo=")
improved default path parsing
updated tests

i'd like some eyes on this who've been in this code, I haven't spent enough time in in it.
@jhurliman @janjongboom @alessioalex

I've added comments

Hi.
I'm not sure if this is related but the new version of request does not appear to handle cookies correctly. Or rather, not as well as it used to. After spending  a morning try to work out why some previously working code had stopped functioning I narrowed it down the header information sent and saw that the cookie I was sending appeared to be undefined. I am now using an old version of request (not sure which version it is but I installed it 18/11/2011) and that works correctly.
var cookieStr = "sessionID=" + sessionId;

var headerData = {"Accept":"text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8",
				  "Accept-Language":"en-us",
				  "Origin":"http://www.xxxxxxxx.com",
				  "Referer":"http://www.xxxxxxxx.com/home",
				  "User-Agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_2) AppleWebKit/534.51.22 (KHTML, like Gecko) Version/5.1.1 Safari/534.51.22",
				  "Cookie":cookieStr};

var options = {uri:headingUri, headers:headerData};

request.get(options, function (error, response, body) {

latest release or master?

So what's up with this request? Its kind of fixes a major bug.

The latest release (the version one gets with "npm install request"). The weird thing is I tested this version with a simple php file that reads header data and all appeared fine, including cookies. It's just that the response coming back for the code mentioned above indicates that I am send an "undefined" sessionid (a cookie) but with the older version of request the response shows the correct sessionid sent.

pushed new version with this fix in it.

great, thanks!


Unexpected failure for otherwise successful copy due to lack of container perms

Which version of the AzCopy was used?  v10.
Which platform are you using? Linux.
What command did you run?
$ azcopy copy <source> <destination>

The source is a single file and the destination ditto. The destination container already exists.
What problem was encountered?
I get the following notices as part of the output:
Failed to create one or more destination container(s).
Your transfers may still succeed if the container already exists.

0.0 %, 0 Done, 0 Failed, 1 Pending, 0 Skipped, 1 Total,

Elapsed Time (Minutes): 0.0333
Number of File Transfers: 1
Number of Folder Property Transfers: 0
Total Number of Transfers: 1
Number of Transfers Completed: 0
Number of Transfers Failed: 1
Number of Transfers Skipped: 0
TotalBytesTransferred: 0
Final Job Status: Failed

That is, there actually is no problem here, but the job is reported as a failure.
How can we reproduce the problem in the simplest way?
Create a SAS token that does not have permission to create a container on the destination storage account.
Have you found a mitigation/solution?
No.

Hi there! Could I see a redacted SAS token as an example so I can reproduce this?

@adreed-msft Just to clarify, you mean a token where the "sig" value is removed, but everything esle is still present. Right?

Correct.

@adreed-msft, here is a SAS token: ?sv=2019-10-10&ss=b&srt=o&sp=wlc&se=2021-05-26T16:06:56Z&st=2020-05-26T08:06:56Z&spr=https&sig=[REDACTED].

Bump?
That is an issue for us because shell scripts fail with a non-zero exit code for no good reason. We can't just swallow the exit code because there might be a real error some day which is then swallowed, too.

Sorry for the slow reply, I notice your SAS token allows for only Object operations (judging by &srt=o). When creating your SAS token, if you wish for container creation permissions, you should add the Container level of operations.

@adreed-msft as I mentioned above, the "destination container already exists". That is, I'm not looking to create any containers here – they will always exist.
And this scenario is probably common to many users relying on container-scope RBAC assignments – the containers will be managed separately.

OK, so,
Failed to create one or more destination container(s).
Your transfers may still succeed if the container already exists.

and
Any empty folders will not be processed, because source and/or destination doesn't have full folder support

are more or less just warnings. They don't indicate a direct failure condition.
However,
0.0 %, 0 Done, 0 Failed, 1 Pending, 0 Skipped, 1 Total,

Elapsed Time (Minutes): 0.0333
Number of File Transfers: 1
Number of Folder Property Transfers: 0
Total Number of Transfers: 1
Number of Transfers Completed: 0
Number of Transfers Failed: 1
Number of Transfers Skipped: 0
TotalBytesTransferred: 0
Final Job Status: Failed

This was a failed transfer. Do you have the log files on hand for this transfer, in that case? They'd live under ~/.azcopy.

I have tried to dig a little deeper into this and haven't been able to consistently reproduce the issue, especially after I have looked through the code to realize that there's no connection to those logging messages and the failure. I'll close out this issue for now and will re-open if needed.


added animation

#38  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @leonchike and the rest of your teammates on  Graphite


:bug:, :blowfish: and :beetle: are animals too!

Added "animal" tag to :bug:, :blowfish: and :beetle:

🐹🐹🐹🐹👍 thanks!


Change the AudioStreamPlayer icons to be more clear.

closes https://github.com/godotengine/godot-proposals/issues/9688

You should optimize SVGs, use for example SVGOMG

See here for optimization instructions

Thanks!


consumer benchmark

Feature request: It'd be nice to have a consumer benchmark with some posted numbers that others could compare against. Does anything like that exist? This repo appears to only have producer benchmarks today.

In my application, I'm seeing BaseConsumer deliver messages at 5-6x the throughput of StreamConsumer. It'd be good to get both of these in the benchmark suite.

(I suspect the unbuffered channel here may be to blame: https://docs.rs/rdkafka/0.14.1/src/rdkafka/consumer/stream_consumer.rs.html#183 , but we should have a repeatable benchmark before proposing any changes.)

In some tests I did in the past the buffer didn't seem to have an effect on the consumer throughput (which I found surprising). Yeah a repeatable benchmark would be better. I'll write one (contributions are also welcome :) ).
Are you receiving small messages? That's where the extra overhead of the StreamConsumer might become noticeable.

@fede1024 I found a small buffer on that channel to help a lot (throughput went from about 100MBps to about 500MBps). The messages I'm testing with average around 2KB in size.
I don't think I'll have time to contribute a benchmark soon, but if you get one started I'd be more than happy to take it for a spin in my environment.

Very interesting! Thank you for running the test. I've started working on a consumer benchmark. I'll run similar tests and see if I can replicate your results.

I've added a simple consumer benchmark. Indeed, a small buffer can greatly reduce the number of context switches and make the consumer considerably faster. I'll change the default buffer size in rust-rdkafka, and also make it configurable.
The BaseConsumer is still considerably faster than StreamConsumer when dealing with very small messages (10 bytes). However in more realistic benchmarks (1KB for example), the performance becomes basically identical if a buffer is used.


cross compile for windows

Description
The cross compile for windows change has added the following new interface:
1) c++ side
added setup_engine() interface
moved base64_encode/decode from register_jit_hooks.cpp to runtime.cpp since it is being used by setup_engine also
2) python side
added torch_tensorrt.cross_compile_save_for_windows() interface
torch_tensorrt.dynamo.cross_compile_save_for_windows() interface
Type of change
Please delete options that are not relevant and/or add your own.

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

Checklist:

[ ] My code follows the style guidelines of this project (You can use the linters)
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[ ] I have added tests to verify my fix or my feature
[ ] New and existing unit tests pass locally with my changes
[ ] I have added the relevant labels to my PR in so that relevant reviewers are notified

One idea from Angela to avoid overhead of setting up engines on windows is to handle in torch_tensorrt.load(). This modifies and creates a new graph which makes the engines go live.

Proposal:

torch_tensorrt.dynamo.cross_compile - Input is exported program, output is an exported program.

Exporter:
a) Currently, for x86 compilation, we create a custom op execute_engine(inputs, TRTEngine) in the graph and TRTEngine gets registered
as a getattr node in the graph.
b) For cross compilation, pack the engine_info into this getattr node and introduce setup_engine nodes

if cross_compile:
    # Set the packed TRTEngine info as a graph attribute
    setattr(gm, engine_name, trt_module._pack_engine_info())
    packed_engine_node = gm.graph.get_attr(engine_name)
    engine_node = gm.graph.call_function(torch.ops.tensorrt.setup_engine.default,
                                                  (packed_engine_node),)

    # The usual workflow ...
    trt_node = gm.graph.call_function(
                torch.ops.tensorrt.execute_engine.default,
                (trt_module_node.args, engine_node),
            )

During torch_tensorrt.load(), we setup the engine and remove the setup_engine node
exp_program = torch_tensorrt.load("trt.ep")
gm = exp_program.module()
# Check if the engine is not set up yet
is_engine_live = not any(node.target == torch.ops.tensorrt.setup_engine.default for node in gm.graph.nodes)
if not is_engine_live:
     # Lowering pass which replaces setup_engine nodes into execute_engine nodes
     # The packed engine should be replaced with TRTEngine
     # packed_engine_info = getattr(gm, engine_name)
     # trt_engine = torch.classes.tensorrt.Engine(packed_engine_info)
     # setattr(gm, engine_name, trt_engine)
     # remove setup_engine node
     gm = torch_tensorrt.dynamo._exporter.setup_engine(gm)
     return exp_program

This ensures we support loading this via

torch.export.load() - This incurs the engine setup cost for every inference (has setup_engine + execute_engine nodes)
torch_tensorrt.load() - This does not have any overhead (only contains execute_engine nodes similar to how we do it on x86 platform)

What are you guys' thoughts on maybe just making the cross_compile load now?


What if I don't want to redirect the unauthorized user to a different page?

Hi! I'm new to Supabase and supabase-auth-helpers.
My goal is to fetch the Supabase user inside getServerSideProps, and pass it to the component I'm rendering. If the user is not authenticated, I don't want to redirect to a different page, I just want to pass a null object in place of a user.
That way I would be able to show the premium content to the authenticated users, and if the user is not authenticated, I'd be able to render a paywall into the page instead.
Is it possible to do using this library? Or am I always required to redirect the unauthorized user away from this page?

Yes, you can pass authRequired: false, see https://github.com/supabase-community/supabase-auth-helpers/blob/main/examples/nextjs/pages/profile.tsx#L26

Ah, thank you very much!
I have a similar question about the protected routes. The docs say:

If you visit /api/protected-route without a valid session cookie, you will get a 401 response.

But what if I don't want it to automatically send the 401 response, what if I would rather do something else instead?
Also, are there are docs where I could've learned this on my own? Since I couldn't find this information in the README.


Public Docs for Epochs and Staking Collection

Issue To Be Solved
Need docs to share with the public and partners about epochs and staking collection before we launch them
Suggest A Solution

[ ] Outline in a feature branch in onflow/flow
[ ] Epochs Technical Overview, scripts, and events
[ ] Update staking auction technical overview, scripts, and events
[ ] QC Technical Overview, scripts, and events
[ ] DKG Technical Overview, scripts, and events
[ ] Updated staking instructions to use the staking collection
[ ] Update old staking instructions

cc @jordanschalm @JeffreyDoyle @franklywatson

Overall not many issue or feedback to make. A few nits below:
In https://flow-docs-git-epoch-org-onflow.vercel.app/staking/
Sentence "These processes are called Cluser Quorum Certificate Generation (QC), and Distributed Key Generation (DKG)" has cluster spelt wrong.
Epochs link in the same page (paragraph above Rewards title) links to https://flow-docs-git-epoch-org-onflow.vercel.app/staking/epochs/ (currently 404s). Not sure if that will be fixed by deployment or not. Same for the rewards link below that
In https://flow-docs-git-epoch-org-onflow.vercel.app/staking/staking-scripts-events/
the sentence "Emitted by FlowIDTableStaking.Admin.moveTokens() at the end of an epoch if a node operator's staked tokens are being unstaked in response to a request from the node operator to do so" could probably drop the "to do so" at the end.
Similar comment on "Emitted by FlowIDTableStaking.Admin.moveTokens() at the end of an epoch if a delegator's staked tokens are being unstaked in response to a request from the delegator to do so."
let me know when the TODOs are added and will do a follow up review


gradle build fails to resolve dependencies

Input

Output
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'ribbon'.
> Could not resolve all dependencies for configuration ':classpath'.
   > Could not resolve com.netflix.nebula:nebula-bintray-plugin:2.2.0.
     Required by:
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10
      > Could not resolve com.netflix.nebula:nebula-bintray-plugin:2.2.0.
         > Could not get resource 'https://plugins.gradle.org/m2/com/netflix/nebula/nebula-bintray-plugin/2.2.0/nebula-bintray-plugin-2.2.0.pom'.
            > Could not GET 'https://plugins.gradle.org/m2/com/netflix/nebula/nebula-bintray-plugin/2.2.0/nebula-bintray-plugin-2.2.0.pom'.
               > Remote host closed connection during handshake
   > Could not resolve com.netflix.nebula:nebula-core:2.2.0.
     Required by:
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > com.netflix.nebula:nebula-project-plugin:2.2.1
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > com.netflix.nebula:nebula-publishing-plugin:2.2.2
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > com.netflix.nebula:nebula-release-plugin:2.2.7
      > Could not resolve com.netflix.nebula:nebula-core:2.2.0.
         > Could not get resource 'https://plugins.gradle.org/m2/com/netflix/nebula/nebula-core/2.2.0/nebula-core-2.2.0.pom'.
            > Could not GET 'https://plugins.gradle.org/m2/com/netflix/nebula/nebula-core/2.2.0/nebula-core-2.2.0.pom'.
               > Remote host closed connection during handshake
   > Could not resolve com.perforce:p4java-jfrog:2011.1.297684.
     Required by:
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > com.netflix.nebula:gradle-info-plugin:2.2.2
      > Could not resolve com.perforce:p4java-jfrog:2011.1.297684.
         > Could not get resource 'https://plugins.gradle.org/m2/com/perforce/p4java-jfrog/2011.1.297684/p4java-jfrog-2011.1.297684.pom'.
            > Could not GET 'https://plugins.gradle.org/m2/com/perforce/p4java-jfrog/2011.1.297684/p4java-jfrog-2011.1.297684.pom'.
               > Remote host closed connection during handshake
   > Could not resolve org.eclipse.jgit:org.eclipse.jgit:<IP_ADDRESS>312181205-r.
     Required by:
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > com.netflix.nebula:gradle-info-plugin:2.2.2
      > Could not resolve org.eclipse.jgit:org.eclipse.jgit:<IP_ADDRESS>312181205-r.
         > Could not get resource 'https://plugins.gradle.org/m2/org/eclipse/jgit/org.eclipse.jgit/<IP_ADDRESS>312181205-r/org.eclipse.jgit-<IP_ADDRESS>312181205-r.pom'.
            > Could not GET 'https://plugins.gradle.org/m2/org/eclipse/jgit/org.eclipse.jgit/<IP_ADDRESS>312181205-r/org.eclipse.jgit-<IP_ADDRESS>312181205-r.pom'.
               > Remote host closed connection during handshake
   > Could not resolve org.tmatesoft.svnkit:svnkit:1.8.5.
     Required by:
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > com.netflix.nebula:gradle-info-plugin:2.2.2
      > Could not resolve org.tmatesoft.svnkit:svnkit:1.8.5.
         > Could not get resource 'https://plugins.gradle.org/m2/org/tmatesoft/svnkit/svnkit/1.8.5/svnkit-1.8.5.pom'.
            > Could not GET 'https://plugins.gradle.org/m2/org/tmatesoft/svnkit/svnkit/1.8.5/svnkit-1.8.5.pom'.
               > Remote host closed connection during handshake
   > Could not resolve com.mycila.maven-license-plugin:maven-license-plugin:1.10.b1.
     Required by:
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > nl.javadude.gradle.plugins:license-gradle-plugin:0.11.0
      > Could not resolve com.mycila.maven-license-plugin:maven-license-plugin:1.10.b1.
         > Could not get resource 'https://plugins.gradle.org/m2/com/mycila/maven-license-plugin/maven-license-plugin/1.10.b1/maven-license-plugin-1.10.b1.pom'.
            > Could not GET 'https://plugins.gradle.org/m2/com/mycila/maven-license-plugin/maven-license-plugin/1.10.b1/maven-license-plugin-1.10.b1.pom'.
               > Remote host closed connection during handshake
   > Could not resolve com.google.guava:guava:17.0.
     Required by:
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > nl.javadude.gradle.plugins:license-gradle-plugin:0.11.0
      > Could not resolve com.google.guava:guava:17.0.
         > Could not get resource 'https://plugins.gradle.org/m2/com/google/guava/guava/17.0/guava-17.0.pom'.
            > Could not GET 'https://plugins.gradle.org/m2/com/google/guava/guava/17.0/guava-17.0.pom'.
               > Remote host closed connection during handshake
   > Could not resolve org.ajoberstar:grgit:1.3.0.
     Required by:
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > com.netflix.nebula:nebula-release-plugin:2.2.7 > org.ajoberstar:gradle-git:1.2.0
      > Could not resolve org.ajoberstar:grgit:1.3.0.
         > Could not get resource 'https://plugins.gradle.org/m2/org/ajoberstar/grgit/1.3.0/grgit-1.3.0.pom'.
            > Could not GET 'https://plugins.gradle.org/m2/org/ajoberstar/grgit/1.3.0/grgit-1.3.0.pom'.
               > Remote host closed connection during handshake
   > Could not resolve com.github.zafarkhaja:java-semver:0.9.0.
     Required by:
         project : > com.netflix.nebula:gradle-netflixoss-project-plugin:2.2.10 > com.netflix.nebula:nebula-release-plugin:2.2.7 > org.ajoberstar:gradle-git:1.2.0
      > Could not resolve com.github.zafarkhaja:java-semver:0.9.0.
         > Could not get resource 'https://plugins.gradle.org/m2/com/github/zafarkhaja/java-semver/0.9.0/java-semver-0.9.0.pom'.
            > Could not GET 'https://plugins.gradle.org/m2/com/github/zafarkhaja/java-semver/0.9.0/java-semver-0.9.0.pom'.
               > Remote host closed connection during handshake

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 6.593 secs

This happens if you use JDK 7. With JDK 8 it should work.
(Side note - JDK 12 has other issues)


ZEVA - Tab/Stage names inconsistency during loading in Complaints Reporting

Describe the task
On the Complaints reporting tab, when a user opens any report, there is a series of stages/ tabs (such as Supplier information, Complaints obligation, Summary, etc.) displayed at the top. Currently, when navigating between these stages/ tabs, outdated names such as Consumer ZEV Sales and Consumer Sales appear during the loading process. Once the page finishes loading, the correct, updated names are displayed.
Purpose
To improve user experience by ensuring consistent and accurate stage/tab names are shown at all times, including during page loading.
Acceptance Criteria

[ ] When a user navigates between stages/tabs (Supplier Information/ ZEVs Supplied and Registered/ Compliance Obligation/ Summary/ Assessment) in any report within Complaints Reporting, the correct names should be displayed, even during the loading phase.
[ ] Users should not see outdated or incorrect stage/tab names at any point in the navigation or loading process.

Additional context
Correct names: Supplier Information/ ZEVs Supplied and Registered/ Compliance Obligation/ Summary/ Assessment
Navigation path: Home page -  Complaints reporting tab - Open any report from the table below

Currently, when clicking on the Summary page or the Assessment page, the tabs will disappear during loading. When clicking on the other pages, the tabs stay appearing.
The tabs should keep appearing when clicking on the Summary page. Fixing this behaviour is included in this issue.
The current behaviour of clicking the Assessment tab remains unchanged.


tunnel dsdl

why [<=60]?
why not 59, or 66? Even 64 would be better frame usage than 60

protocol doesn't really make much sense, right. Especially if 0 is mavlink. 0 = default should be rather something like UNDEFINED/GENERIC/TRANSPARENT or whatever it is named. What instead would be seriously needed is some possibility to send meta data, such as baudrate, clock rate, etc. pp. See gps example in https://www.youtube.com/watch?v=Sw24wAIoemA.

my 2 c :)

The objective was to make the message always fit into one CAN FD frame; 60 bytes is therefore the maximum.
The meaning of 0 is debatable. The tunneling feature is not intended for the exchange of raw/unstructured data; as I said a number of times previously, UAVCAN should not be used for raw data tunnelling as that partially defeats the purpose of the protocol. Obviously, anybody is free to define their own data types and use the protocol for whatever purpose suits them, but this use case will not be supported in the standard data type set.


itest: add missing build tag

Avoid building itests when building for a release.

I don't think we should add build tags to each file, as that makes it a bit harder to use (for example, the itest files aren't compiled anymore when running the unit tests, which is sometimes handy to detect compilation errors early).
Because the release binary shouldn't have any reference to the itest package at all, it will not be built by default.
So something is wrong and the offender seems to be proof/aperture.go. That needs to be moved to the itest package, then we resolve the original issue.

Replaced by https://github.com/lightninglabs/taproot-assets/pull/378.


fix(Statistic): fix wrong inverted type

Best regards.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1909   +/-   ##
=======================================
  Coverage   99.77%   99.77%
=======================================
  Files         148      148
  Lines        2637     2637
=======================================
  Hits         2631     2631
  Misses          6        6

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 82259a7...c590b2f. Read the comment docs.

Released in<EMAIL_ADDRESS>


Add support for setting decorators for a reader

I found that to be super nice so I thought it could be part of ROM. This basically allows you to decorate mapped objects with additional behavior that can be anything, ie:

UI view presenters (for presenting values)
UI view renderers (for rendering html chunks etc.)
JSON serializers (for more higher level stuff like hypermedia etc.)
probably more...

I don't have a strong opinion if it should be part of rom core or an external plugin. My tactic right now is to put stuff into ROM and extract later once we see things clearly.

Closing in favor of #155


test output is too long

https://app.circleci.com/pipelines/github/statechannels/statechannels/11009/workflows/9885820f-7d33-4238-ace2-7e9466281db9/jobs/53817/parallel-runs/0/steps/0-106
Your output is too large to display in the browser.

Only the last 400000 characters are displayed.

This has been resolved ( probably due to someone cleaning up some console.log statements). See here for an example.

This has been resolved ( probably due to someone cleaning up some console.log statements). See here for an example.

This has been resolved ( probably due to someone cleaning up some console.log statements). See here for an example.


Improve the top level spice file names under models

The names of the top level spice files are currently;
./models
./models/ngspice
./models/ngspice/design.ngspice
./models/ngspice/sm141064.ngspice
./models/ngspice/smbb000149.ngspice
./models/xyce
./models/xyce/design.xyce
./models/xyce/sm141064.xyce
./models/xyce/smbb000149.xyce

The names design / sm141064 / smbb000149 don't really have useful meanings. I believe that @mkkassem has better names that should be used instead.

I think @mkkassem was suggesting something like this?

design --> gf180mcu.global_parameters
sm141064 --> gf180mcu.common
smb000149 --> gf180mcu.highvoltage

@mkkassem - Can you document what these files should be called and then assign the issue to the right person to fix?

@mithro I agree with the suggestion above. If @mkkassem approves, we could move forward.

@mithro Should I start to fix the issue with @mkkassem suggestions?