[$250] Web - Login - Hitting "Enter" key does not save the password

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the App on device 1 (Android App for me)
In the email field, enter an email address for a new account
Verify there’s a message saying that a Magic Link has been sent to the email address
Go to another device 2 (Web for me), follow the next steps on device 2!!!
Navigate to the email inbox of the account and open the validation email
Copy the link and change it to staging - > Navigate to the staging link
Set a password that follows the password requirements
Click "Enter" key

Expected Result:
Enter key should work to save the password
Actual Result:
Enter key does not work to save the password
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/193017901-fc456e15-c7d4-4bdc-bd11-6a8e7dd00c9b.mp4
https://user-images.githubusercontent.com/93399543/193017904-5de6fc72-e647-4f2b-bc7b-79ebbb40005e.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Proposal
The issue is caused due to empty onSubmitEditing in NewPasswordForm
https://github.com/Expensify/App/blob/a0340070ebf94fe3acde872c8c0c4222edd984c3/src/pages/SetPasswordPage.js#L113-118

We just simply need to set onSubmitEditing  to onSubmitEditing={this.validateAndSubmitForm} and it fixes the issue of FormSubmission on enter key press as onSubmitEditing is triggered when you click the text input submit button (from keyboard).
Solution:-

After Implementing Solution :-
https://user-images.githubusercontent.com/81307212/193086697-182a1eb7-1f96-4f1b-8f98-cd72ae5f9135.mov

I noticed we have the key enter behavior on the login page, so we should also have it in this page. Thanks @syedsaroshfarrukhdot for the proposal. Applying the External label.

Hi @sobitneupane could you review this proposal?

Posted
Internal - https://www.upwork.com/ab/applicants/1575600235539378176/job-details
External - https://www.upwork.com/jobs/~016a8ee45661b0d7d7
Invited @sobitneupane

Should I take part in it?

like I need to be assigned this task on Upwork and will resolve this issue as I was also going through the same bugs because of  a few things now I've sorted them out and looking forward to offering my services here as well
Let me know how can I do that

@syedsaroshfarrukhdot onSubmitEditing={() => {}} was intentionally added in https://github.com/Expensify/App/pull/9445 PR to solve https://github.com/Expensify/App/issues/9058 issue. We have pressOnEnter prop set on <Button> in FormAlertWithSubmitButton.js. So, SetPassWord should ideally be pressed.
And the other thing, I was not able to consistently reproduce this issue. Few times it worked as expected.

@sobitneupane  In PR https://github.com/Expensify/App/pull/9445 details it is mentioned the multiple submits was causing validateAndSubmitForm() to be ran multiple times, and the second time it ran, it would have this.props.userSignUp.authToken from the first request. This caused changePasswordAndSignIn() to be triggered from an invalid state and put the UI into a position where it couldn't fetch all the reports. Since the reports couldn't fully load, it was stuck at a loading spinner.
So it was added this way intentionally to fix that issue since changePasswordAndSignIn() has been deprecated and we are not using it anymore so adding onSubmitEditing={this.validateAndSubmitForm} is safe and it  fixes this issue also.
I tested it for the issue https://github.com/Expensify/App/issues/9058 after adding onSubmitEditing={this.validateAndSubmitForm} it doesn't cause any problem again of Endless loading spinner.
Video After Adding   onSubmitEditing={this.validateAndSubmitForm} and it fixes the enter key save password issue also.
https://user-images.githubusercontent.com/81307212/193362932-416d0129-b415-4302-b587-837f4f27d8ee.mov
We are also using onSubmitEditing to handle Enter Key in PasswordForm.js
https://github.com/Expensify/App/blob/main/src/pages/signin/PasswordForm.js#L155

And if we still prefer not to use onSubmitEditing to handle enter key we need to pass prop enterKeyEventListenerPriority={1} on button in FormAlertWithSubmitButton.js along with pressOnEnter
This fixes the issue

After enterKeyEventListenerPriority={1} in FormAlertWithSubmitButton.js
https://user-images.githubusercontent.com/81307212/193364319-d0055d86-c3ef-42f2-a510-37c65616af94.mov
I have been able to consistently reproduce this issue.

Updated Proposal
@sobitneupane as recommended by you I digged a bit more in Button component and figured that the issue

the issue is with this.props.isFocused in Button.js

@syedsaroshfarrukhdot Your finding here looks promising.
Since the screen is focused at the start, why is the Set Password button not triggered?

the issue is with this.props.isFocused in Button.js

@syedsaroshfarrukhdot Your finding here looks promising.
Since the screen is focused at the start, why is the Set Password button not triggered?
I dug further and found that a button with text Continue is also being rendered. But this.props.isFocused for the button is false.
This might be the cause of the issue.

@sobitneupane  Yes, that is the reason the SetPassoword page loses screen focus. I looked in more depth and figured out that not only the Continue button but complete Login From is being rendered in the SetPassoword page.

Not sure yet, but I believe it is something to do with Stack Navigator in PublicScreens.js which is rendering SignInPage somehow in SetPasswordPage
https://github.com/Expensify/App/blob/e5d766115babfe4fed51b97e8b7f65b6bec64f21/src/libs/Navigation/AppNavigator/PublicScreens.js#L1-L44
Will be digging a bit more in depth to figure it why login form is being rendered in SetPasswordPage.js

Waiting for proposals.

Waiting for proposals.

Changing it to Weekly

Waiting for proposals.

Hey @jliexpensify, just a reminder to keep the pressure on to find a contributor and get this one closed out!

@syedsaroshfarrukhdot - how did you go with your digging?

Have also doubled the price.

Will be digging a bit more in depth to figure it why login form is being rendered in SetPasswordPage.js

Hi @syedsaroshfarrukhdot, were you able to find something out? Don't hesitate to let us know if you found a dead-end! :)

Sorry for not replying after mention somehow missed the notification earlier
@marcochavezf Yeah wasn't able to find any solution to it the problem it was pretty much a dead end. As whole Login Form is being rendered in SetPassword Screen and I wasn't able to find  why it is happening. Maybe a bug in StackNavigator causing higher level screens to rendered in the rest of the components.

Oh I see, thanks @syedsaroshfarrukhdot, something I would like to understand is why the SetPassword is different from the PasswordForm, because we have the "Enter" key behavior there, but not in the SetPassword. Would it require a different structure to avoid conflicts with the StackNavigator? cc @sobitneupane

I believe we can close this issue, we are closing issues related to password as we will be moving towards #passwordless https://github.com/Expensify/App/issues/11403#issuecomment-1303955452

@marcochavezf - do you mind confirming we can close? If so, I'll get the Upworks job removed. Thanks!

Keeping open for payment to C+ as discussed: https://expensify.slack.com/archives/C02NK2DQWUX/p1668132124601409

New job post for payment: https://www.upwork.com/ab/applicants/1592242364089561088/job-details

Paid @sobitneupane and job closed.


fixup! Toggle for disabling autofocus on debugger stop

The settings item debug.autoFocusEditor was renamed to
debug.focusWindowOnBreak on the settings definition, but not on the
code using it.
Corrects #77213

Cc @isidorn, @joaomoreno.

Sorry I have already tackled this via 1fa8c2f111
Thanks for the PR anyways


Allow RegExp getters in getStaticValue

This allows getStaticValue to evaluate RegExp getters. Only the safe readonly getters are allowed, those are source, flags, and the getters for individual flags.
lastIndex is not included because its result depends on previous uses of the RegExp.
I also added tests for String and Array length and property access and for the RegExp constructor.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.14 :tada:

Comparison is base (57693be) 96.70% compared to head (dacf5e5) 96.85%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
+ Coverage   96.70%   96.85%   +0.14%
==========================================
  Files          13       13
  Lines        2003     2032      +29
  Branches      380      386       +6
==========================================
+ Hits         1937     1968      +31
+ Misses         65       63       -2
  Partials        1        1

Impacted Files
Coverage Δ

src/get-static-value.mjs
97.16% <100.00%> (+0.49%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Type issue while onHover and onClick triggers.

Hello, I am facing the issue of gl geocoder disable after adding to clicks on MapGL instance, on hover, and onClick
Let me know how I can resolve this.

Hi @I1mran, is it possible for you to provide a Code Sandbox (https://codesandbox.io/) that demonstrates this issue?

I don't have code right now for sharing.


[Request] Custom FTL backgrounds

What do you think is missing or what do you think can be done better
I am hoping to make the FTL animated screen resemble the below image. If this is doable in a texture pack can you please let me know? Otherwise, consider adding a customisation?
This is for StargateMC.com.
https://i.ytimg.com/vi/LiMSPK23kIQ/maxresdefault.jpg

I'm afraid that with it's current implementation, there's no way to texture the animation since it's hardcoded to generate a series of squares, which is how MC renders stars.  I think it'd be possible to implement with a relatively low amount of effort, but it's probably not gonna happen until the 1.16 update of the mod.

Thanks for putting on the board @zmaster587 ... I'll backport whatever you come up with .... Im sure I can manage that, its just a little over my head understanding how to modify it from scratch :)


APIView BreakUp Swagger Token Files

Large token files (e.g. swagger) need to be post processed into smaller files.
This will help improve performance on UI side as the files can be loaded in sections.

@chidozieononiwu Do we still need this? I thought we decided to go with partial page load.

This can still be used to track that work.


Artwork based on Based on: Untitled 7 by Belfort Group, 1976

Originally published in "Computer Graphics and Art" v1n3, 1976
ReCode project url: http://recodeproject.com/translation/jan-vantomme-v1n3untitled-7-direct-untitled-7-belfort-group

thanks so much !  going to merge this shortly....

merged!
closing


New version v3.0.0 - Make it more easy and simple to use

Hello Grafana team !
When I looking for a plugin to display the state of a healthcheck, on my dashboard, I came across this plugin but I fond it complicated to use and to match whith my simple need. I decided to fork it and make it simpler.
Now, users can define their own thresholds (background color, severity name and value). Threshold activate when the query value is in range. Only works with number values.
Alert and multiple pane view was removed.
This version are breaking changes. Older versions are not supported.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: Benjamin Fourmaux:x: BenjaminFourmauxBenjamin Fourmaux seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello,
This version doesn't support multi metrics in the same panel, right ?
I use Status Panel to show many metrics, each panel have 10/15 metrics. But with you version, i can only use one metric.
And I can't use thresholds color in override option, So this feature would be very interresting :)
Thanks

Hello, This version doesn't support multi metrics in the same panel, right ? I use Status Panel to show many metrics, each panel have 10/15 metrics. But with you version, i can only use one metric. And I can't use thresholds color in override option, So this feature would be very interresting :)
Thanks

Hi, thank you for your feedback !
I thought that not many people use multi metrics in one panel feature. I will try to implement it.

Hello, This version doesn't support multi metrics in the same panel, right ? I use Status Panel to show many metrics, each panel have 10/15 metrics. But with you version, i can only use one metric. And I can't use thresholds color in override option, So this feature would be very interresting :)
Thanks

Hi, thank you for your feedback ! I thought that not many people use multi metrics in one panel feature. I will try to implement it.

Hi, I would call support for multiple metrics the main feature of this plugin. There is a similar plugin where this is implemented, but it was also abandoned https://github.com/serrrios/Status-Overview-Panel

Yes, I use this main feature with this plugin, but this PR#6 (versionned 3.0) doesn't support it.
Check this :

And each panel is configured to show ~15 metrics by panel. It is mandatory to have this main feature to me.

Now support Multi pane !
I adding the support of multi pane in one panel. Just add more queries.
I adding template string to add variable in text fields (title, subtitle and url)
I wanted to make each pane customizable but that would have made the plugin complicated and cumbersome to use. Which is contrary to v3

Now support Multi pane !
I adding the support of multi pane in one panel. Just add more queries. I adding template string to add variable in text fields (title, subtitle and url)
I wanted to make each pane customizable but that would have made the plugin complicated and cumbersome to use. Which is contrary to v3

Hmmm. And, if you add a global plugin option like

Button -> concatenate all metrics in a panel (yes/no).

And then we can override the options by fields

Status panel - thresholds

Is it possible to add an option to show or hide the value/label if it is not in OK state?
My need is to have, in one panel, many metrics, to monitor many computes/nodes/storages. And if one of these metrics exceeds the threshold, my panel changes color and according to the thresholds configuration.

I can create a request on your github :)

@BenjaminFourmaux we are currently not looking to introduce breaking changes as we are mostly trying to offer a react version of this plugin working for existing customers. You are however free to fork and publish an alternative version of this panel and submit it for review

Hey @tolzhabayev thank you for you reply.
I think i have a stable version, prod-ready of my version, i will definitely propose to submit an alternative version of this plugin.


Cannot import typescript operation definition file with 'near-operation-plugin' and document: external

Which packages are impacted by your issue?
@graphql-codegen/near-operation-file-preset
Describe the bug
Hello!
When using 'near-operation-file' with 'typescript-react-apollo' (but I guess it's the same with other plugins), we cannot import typescript files containing queries/mutations, only .graphql files work
Your Example Website or App
https://codesandbox.io/p/devbox/youthful-bose-wxjhxz
Steps to Reproduce the Bug or Issue
Using this config:
import { CodegenConfig } from "@graphql-codegen/cli";

const config: CodegenConfig = {
  schema: "schema.graphql",
  documents: "document.ts",
  emitLegacyCommonJSImports: false,
  generates: {
    ".": {
      plugins: ["typescript-react-apollo"],
      preset: "near-operation-file",
      presetConfig: {
        baseTypesPath: "./types.ts",
      },
      config: {
        importDocumentNodeExternallyFrom: "near-operation-file",
        importOperationTypesFrom: "Types",
        documentMode: "external",
      },
    },
    "types.ts": { plugins: ["typescript", "typescript-operations"] },
  },
};

export default config;

this config makes sense: the goal is to import queries/mutations defined in typescript files into local mtFile.generated.ts
Expected behavior
in the reproduction repository, line 25 of document.generated.ts should be:
`return Apollo.useQuery<Types.UserQuery, Types.UserQueryVariables>(Operations.userQuery, options); ``
and not:
return Apollo.useQuery<Types.UserQuery, Types.UserQueryVariables>(Operations.user, options);
userQuery is the name of the typescript variable, whereas user is the name of the graphql query
Screenshots or Videos
No response
Platform

OS: MacOS
NodeJS: 18
graphql  16.3.0
@graphql-codegen/* version(s): latest

Codegen Config File
import { CodegenConfig } from "@graphql-codegen/cli";

const config: CodegenConfig = {
  schema: "schema.graphql",
  documents: "document.ts",
  emitLegacyCommonJSImports: false,
  generates: {
    ".": {
      plugins: ["typescript-react-apollo"],
      preset: "near-operation-file",
      presetConfig: {
        baseTypesPath: "./types.ts",
      },
      config: {
        importDocumentNodeExternallyFrom: "near-operation-file",
        importOperationTypesFrom: "Types",
        documentMode: "external",
      },
    },
    "types.ts": { plugins: ["typescript", "typescript-operations"] },
  },
};

export default config;

Additional context
remotely related to https://github.com/dotansimha/graphql-code-generator/issues/3272#issuecomment-1427850754

Unfortunately its the same for typescript-react-query, it simply does not use the correct types inside each operation file.


Run schema merge tests on more platforms

Fixes #3515
Fixes #3465
Description
In rare instances, dialog:merge tests could pass when they shouldn't. This PR addresses that as well as opens up the dialog:merge tests to non-Windows platforms.

Pull Request Test Coverage Report for Build 709815266

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.003%) to 84.93%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

libraries/adaptive-expressions/src/triggerTrees/trigger.ts
2
62.79%

Totals

Change from base Build 709412078:
-0.003%

Covered Lines:
18941

Relevant Lines:
21240

💛  - Coveralls


[Doc] flavorDimensions is not marked as required in the documentation

In README.md, all samples set the flavorDimensions property:
flavorizr:
  app:
    android:
      flavorDimensions: "flavor-type"

but the table say that this parameter is not required
However if the property is not defined, flavorizr fails to run:
$ flutter pub run flutter_flavorizr
The following fields were missing: [android]. Read the docs for further details

The documentation should reflect that this property is required (and the error message was not very helpful here)

Hello @nicopico-dev , did you define the android entry? The flavorizer is reporting that you didn't.
flavorizr:
  app:
    android:
    ios:

  flavors:
    apple:
      app:
        name: "Apple App"

      android:
        applicationId: "com.example.apple"

      ios:
        bundleId: "com.example.apple"

    banana:
      app:
        name: "Banana App"

      android:
        applicationId: "com.example.banana"
      ios:
        bundleId: "com.example.banana"

Hi @AngeloAvv , yes I had the android entry under app and under each flavor.


WIP: introduce test failures for tls and sike proofs to check CI integrations

Issue # (if available):
Description of changes:
This pull request should not be merged, it is temporary to test CI infrastructure.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request head (ci-test@3fa578d). Click here to learn what that means.
The diff coverage is n/a.


assist is is incompatible with new >1.0 / 2.0-alpha web3.js versions

https://medium.com/@samuel_91690/1-0-release-web3-js-ddd23d3c8f62
1.0.0-beta.37 --> v1.0
1.0.0-beta.38 --> v2.0-alpha
and they're making minor version releases (eg v1.1, v1.2) which don't work in assist.js as it is right now. I don't think it would be too much work to add support (:crossed_fingers:).

Closed by afbf565dda006158f6f1397d8ea645fa6d307eff


Issue with rendering building with building parts

There seems to be an issue with rendering buildings that have building parts, at least the "Stadthaus" in Mainz doesn't render correctly.
This issue appears in tangram 0.14 and tangram-es 0.8.1. I am not sure if this is perhaps already fixed or a problem with the style,
because the building looks fine on your demo map:
http://tangrams.github.io/tangram/#18.58383433024089/50.002429190542124/368.260960742287
but on this map (tangram 0.14), it's missing most of it
https://map-data.de/#map=18.07618459065755/50.002007361945026/8.261264716010315&style=streetcomplete-light

Hi @westnordost – thanks for the report! I checked your datasource on tangram.city with a basic scene file, and compared it to the same scene file with the nextzen datasource. As far as I can tell, those building parts aren't in your tiles.
Just to be sure, I checked all of our styles with all Tangram versions that support MVT (everything from 0.12 on) and didn't see any trouble. I also checked your datasource against those Tangram versions and it looks like those building parts are just missing.
Maybe there was some kind of glitch when your datasource was created?

This may very well be the case. I am checking now in my app (that uses tangram-es 0.8.1, where the problem was also observed) what it displays for the same building and the same style...

Alright, the building shows using the nextzen tile source, so it is a glitch in the tile server I use

Sorry for the noise, and thank you for looking into it! :-)


file： file:///data/data/com.mhc.qccenter/files/HGH-18123.pdf

TypeError: Cannot read property 'querySelector' of nul
TypeError: Cannot read property 'querySelector' of null", source: file:///android_asset/www/build/main.js (178928)

43:01.342 13353-13475/com.mhc.qccenter E/AndroidProtocolHandler: Unable to open asset URL: file:///android_asset/www/assets/fonts/roboto-bold.woff2
11-27 16:43:01.412 13353-13474/com.mhc.qccenter E/AndroidProtocolHandler: Unable to open asset URL: file:///android_asset/www/assets/fonts/roboto-bold.woff
11-27 16:43:01.422 13353-13475/com.mhc.qccenter E/AndroidProtocolHandler: Unable to open asset URL: file:///android_asset/www/assets/fonts/roboto-bold.ttf

https://stenciljs.com/docs/angular
According to the integrated way of this document, why do you report these errors after integration? Is there a problem?


Add filesystem support under tab completion

Thank you for this wonderful package. The following is an enhancement request. There are cases where we need to pass in the relative file path under certain commands.
Example:
>>>mycmd file ../../file.txt
A bash style tab support that will validate and autocomplete file path will enhance the user experience.

+1
I wanted to do a cd type of tab completion, I want not able to do it using the completer.


Color schemes support

Description
User will be able to change color schemes on user profile modal implemented in #115

Can grab a few themes from MonkeyType


ES6 features in NodeOS

Hey, i want to know if we can use es6 features like template strings and such things because nodejs.org (https://nodejs.org/en/docs/es6/) and kangax (https://kangax.github.io/compat-table/es6/#node4) shows that in node 4.4.3 we can use them.
And if we can make use of them, should we use it in strict mode?

I don't see why not... what do you want to work about?
El 26/4/2016 6:12 PM, "Philipp Czarnetzki"<EMAIL_ADDRESS>escribió:

Hey, i want to know if we can use es6 features like template strings and
such things because nodejs.org (https://nodejs.org/en/docs/es6/) and
kangax (https://kangax.github.io/compat-table/es6/#node4) shows that in
node 4.4.3 we can use them.
And if we can make use of them, should we use it in strict mode?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232

Currently im writing tests and the documentation of the nodeos-mount-* modules.
And isnt it better to set every script to strict mode because most of the es6 functions need strict mode

You are right, so if you want to add strict mode you are welcome :-)
El 26/4/2016 6:28 PM, "Philipp Czarnetzki"<EMAIL_ADDRESS>escribió:

Currently im writing tests and the documentation of the nodeos-mount-*
modules.
And isnt it better to set every script to strict mode because most of the
es6 functions need strict mode
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232#issuecomment-214802178

Cool thanks, i think we should create Guidelines for creating modules for NodeOS
Because if another person creates a core module for NodeOS and this module is then in Non strict mode but every other Module is in strict mode, this module can create errors but they are not registered as errors and this is crucial for stability

I agree, or better than that, a checker tool similar to 'standard' so it's
just needed to exec it to know if it's valid. One of the first things, 100%
tests coverage with my module 'coverdeeps' and full strict mode, do you
agree? :-)
El 26/4/2016 6:36 PM, "Philipp Czarnetzki"<EMAIL_ADDRESS>escribió:

Cool thanks, i think we should create Guidelines for creating modules for
NodeOS
Because if another person creates a core module for NodeOS and this module
is then in Non strict mode but every other Module is in strict mode, this
module can create errors but they are not registered as errors and this is
crucial for stability
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232#issuecomment-214804650

I like the Idea but wouldnt it be to much effort to write such thing?

One step at a time ;-) Let's start finishing coverdeeps and we'll move
later to the guidelines checker.
I like the Idea but wouldnt it be to much effort to write such thing?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232#issuecomment-214820596

I like this idea. I can write a simple tool to do the validation after I
get this boot logo task completed.
On Tue, Apr 26, 2016, 11:31 AM Philipp Czarnetzki<EMAIL_ADDRESS>wrote:

I like the Idea but wouldnt it be to much effort to write such thing?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232#issuecomment-214820596

There you have :-)

Faaaaantastic!

As I currently don't have access to my usual dev computer, I'll be working on this validation tool for now. I've dubbed it NSV for (node script validator) and it'll be up on Github ^_^

Cool :-) Take a look on standard module, seems it would be needed only to tune-up it a little bit, there's no need to write it from scratch, just config it :-)

Cool :-) Take a look on standard module, seems it would be needed only to tune-up it a little bit, there's no need to write it from scratch, just config it :-)

Ah, I will check this out!

@piranna do you wan't leave the 'no semicolons' rule too? I'm a fan of semicolons but I could easily have it rip them from the code if you want.

It's 2016 AND node.js takes lots of ideas from Python, so no semicolons
here except for smileis ;-)
El 30/5/2016 18:48, "Lite McFish"<EMAIL_ADDRESS>escribió:

@piranna https://github.com/piranna do you wan't leave the 'no
semicolons' rule too? I'm a fan of semicolons but I could easily have it
rip them from the code if you want.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232#issuecomment-222526860, or mute
the thread
https://github.com/notifications/unsubscribe/AAgfvp7WQIKhWEY9PG9dTqulr7LfrjR8ks5qGxTkgaJpZM4IQHcs
.

Roger that :P
Mind if I make it auto correct the code?

How do you plan to do it? I have started to use eslint but didn't have time
to add it everywhere...
El 30/5/2016 9:11 PM, "Lite McFish"<EMAIL_ADDRESS>escribió:

Roger that :P
Mind if I make it auto correct the code?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232#issuecomment-222542674, or mute
the thread
https://github.com/notifications/unsubscribe/AAgfvup3ZG2eKKFg0gUIFlK2HM0eby24ks5qGzZrgaJpZM4IQHcs
.

@piranna simply correct the formatting errors that it can find, dump them to a temporary file as it parses the script. When done, I'll have it replace the old file with the new updated file, and report any changes it couldn't make on it's own. I'll probably have this as an optional feature.

Eslint has an option to auto-correct some of the most trivial and common cases, so maybe it's not needed to create a new tool :-) Anyway I think it's better to don't do the autocorrection automatically by default, I have been forced to do it in the past and it works, but you mostly feels the code as it has been written by other if you don't config it correctly.

I'm a fan of Eslint! I'm also a fan of Gulp. Why not: https://github.com/adametry/gulp-eslint ? using https://github.com/feross/eslint-config-standard ?

we could also use the package.json scripts

Package.json with eslint totally works for me. So is the issue about code style or ES6 support?

Package.json with eslint totally works for me. So is the issue about code style or ES6 support?

I think the original issue was about "upgrade" NodeOS source code to use ES6 features... I'm -0 here, I agree on use ES6 features on new source code (I'm already doing so partially) now that we use Node.js 6.x or update the code when we add new features or do some bug fixes or add documentation since "we are there", but I think it's a bit of waste of time to upgrade old code that's working to ES6 just for the shake of it, we could use that time to add tests for example (and upgrade the code ES6 once "we are there" ;-) ).
Regarding eslint I don't like too much standard, specially the K&R brackets in the same line instead of the Allman ones on a new one... :-P

@piranna but for the clarity, is it ok from now on to use es6?

Regarding eslint I don't like too much standard, specially the K&R brackets in the same line instead
of the Allman ones on a new one... :-P

Until this sentence i didnt knew the indentation styles had names :D
Im more the K&R type

@piranna but for the clarity, is it ok from now on to use es6?

Yes, it's ok :-)

Regarding eslint I don't like too much standard, specially the K&R
brackets in the same line instead
of the Allman ones on a new one... :-P

Until this sentence i didnt knew the indentation styles had names :D
Im more the K&R type

It's what more people is use to due to legacy systems and technology
heritage, but it's not the better one...

I'm cool with using any code style template. ESLint has a ton of presets. I like the Airbnb preset a lot. But honestly, it doesn't matter to me.
I'm not the biggest fan of some of the standard rules just cause... I don't know it's very strange haha. But if everyone has their heart on using Standard it's totally cool for me :)
I can totally work on adding ESLint if you want. Just need to know the rules everyone likes :)

I started to work on some of them but don't remember where :-P
El 7/8/2016 17:42, "Tyler James Leonhardt"<EMAIL_ADDRESS>escribió:

I'm cool with using any code style template. ESLint has a ton of presets.
I like the Airbnb preset a lot. But honestly, it doesn't matter to me.
I'm not the biggest fan of some of the standard rules just cause... I
don't know it's very strange haha. But if everyone has their heart on using
Standard it's totally cool for me :)
I can totally work on adding ESLint if you want. Just need to know the
rules everyone likes :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232#issuecomment-238089603, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgfvgN4TQ2xwU8tq9IEkP0ceIu1YrQeks5qdfz7gaJpZM4IQHcs
.

So I found this: http://noeticforce.com/best-javascript-style-guide-for-maintainable-code it's a year old though...
I'm fine with just picking a style guide and going for it unless everyone else has strong opinions on code style. Or we can take a style guide and modify it.
We could close this issue and open a new one to specifically talk about code style

We could close this issue and open a new one to specifically talk about code style

Good idea.

@piranna: should I do that? haha
I can't actually close this issue :(

@tylerl0706 I close for you

Create the new one then :-)
El 8/8/2016 7:57, "John Green"<EMAIL_ADDRESS>escribió:

Closed #232 https://github.com/NodeOS/NodeOS/issues/232.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232#event-747951098, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgfvvcBVHz1qYItZArWdAACPRwxHX48ks5qdsVegaJpZM4IQHcs
.

I'll do that today! Starting at my new job today! (I'm in Seattle, now :-) )

Congrats! :-)
El 8/8/2016 15:42, "Tyler James Leonhardt"<EMAIL_ADDRESS>escribió:

I'll do that today! Starting at my new job today! (I'm in Seattle, now :-)
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/232#issuecomment-238240946, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgfvjCouuVtNGSD91qXTTXWQvy0WyOyks5qdzJPgaJpZM4IQHcs
.


Rework wildcards matching all nodes

Re-enable ansible_ssh_pipelining as expected for the cluster.yml
Do not use 'all' wildcasts for hosts, limit only to k8s-cluster, etcd,
calico-rr groups instead. Other nodes in inventory are out of Kargo
scope and it's up to users how to manage them.

Closes https://github.com/kubernetes-incubator/kargo/issues/714
Signed-off-by: Bogdan Dobrelya<EMAIL_ADDRESS>

@bogdando Was the ansible_ssh_pipelining fact left with the False value from the previous play? I assumed play level vars do not affect the following plays...

@codablock AFAICT, play level gather_facts does. Not sure how to check with pipelining


migrate erc1155 contracts

thanks! @Amxx will be working on it. feel free to review the PR as soon as it's created :)

I wanted to wait until tests are ready to create the PR, but if we can get early review: #594

I would like to help on it with the new components release

@cloudvenger that's great! Do you think you can have a complete PR (component, tests, docs) within 3~4 weeks?
I'm asking this because we otherwise have time already allocated for this, so we don't step into each other's toes.

@cloudvenger said to me
:

it should be good to finish it in 3 weeks, starting next week.


Fixed Slow Hover Transition Effect on Trip Cards issue #433

Title and Issue number

Title :Slow Hover Transition Effect on Trip Cards
Issue No. : #433
Code Stack : CSS, HTML
Close #433

Video/Screenshots (mandatory)

https://github.com/apu52/Travel_Website/assets/120176158/27be9793-2296-4701-b9f3-dfb0b625841c
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
Are you contributing under any Open-source programme?
GSSOC'24

I have resolved the merge conflicts. Try to merge it know

I have resolved the merge conflicts. Try to merge it know

@RohithChitrada some checks are not checked yet.

Know you can merge it.


fix IE's script70 error

Whenever we scroll Readium tries to find the next visible media
overlay element to play.
The problem arises when the user changed the page and had scrolled
before. Some funny combination of things triggers the scroll handler to
run again and because it has a stale reference of the iframe's document
IE gives out with the script70 error message.
This commit unbinds the scroll handler before that can even happen,
i.e., when we are about to open a new page and rebinds it once its done.
May fix #162.

@nbeach, this might sort out https://github.com/readium/readium-js-viewer/issues/295, would you mind giving it a go?

@dariocravero
Thanks for this. I can give it a go.

Good stuff, let me know how it goes.
On Mon, Oct 5, 2015 at 5:03 PM Juan Corona<EMAIL_ADDRESS>wrote:

@dariocravero https://github.com/dariocravero
Thanks for this. I can give it a go.
—
Reply to this email directly or view it on GitHub
https://github.com/readium/readium-shared-js/pull/222#issuecomment-145651984
.


[FEAT]: Cite source when using RAG

What would you like to see?
The main problem with LLMs are their hallucinations. The best and the easiest way to have it  under control is to double check results ourselves. That would be much easier if LLM will cite the exact source the information was retrieved from. For this I would suggest instead of using [CONTEXT #] to use something like [CONTEXT SRC="Attention Is All You Need"] blah, blah. [CONTEXT END]. Then in the LLM Prompt tell it directly to show the source that it has used and provide few examples like that:
Given the following conversation, relevant context, and a follow up question, reply with an answer to the current question the user is asking. Return only your response to the question given the above information following the users instructions as needed.
[CONTEXT SRC="Training strategies for improving listeners' comprehension of foreign-accented speech" page=5]
blah, blah.
[CONTEXT END]
[CONTEXT SRC="Teaching native speakers to listen to foreign-accented speech" page 23]
blah, blah.
[CONTEXT END]
ASSISTANT:
A few researchers in the linguistics field have developed training programs designed to improve native speakers' ability to understand accented speech (source 1). Their training techniques are based on the research described above indicating that comprehension improves with exposure to non-native speech. Other researchers conducted their training with students preparing to be social workers, but note that other professionals who work with non-native speakers could benefit from a similar program. (source 2)
References
Source 1: "Training strategies for improving listeners' comprehension of foreign-accented speech", page 5.
Source 2: "Teaching native speakers to listen to foreign-accented speech", page 23.
For htmls I think a relevant link will be sufficient.

You cannot trust an LLM to do the above task, if it hallucinates a response even with valid context, it will hallucinate a validation for it as well. The reason the citation exists is that the only trustful way to verify information from a response is by directly viewing the context used by the LLM. This is why we include it in citations since you cannot, nor should you, trust an LLM to always give you perfect recall and answers.
When a citation is present we not only show the text snippets but also name the source document it was derived from. You get both of these details in the UI currently. I am not sure what more there is to accomplish on this detail that does not end with a black box evaluating a black box

Thank you for the answer, missed Citations part. Happy that you have it, although there is no exact snippet and what is very important page number in the current reply:

Btw, just recently stumbled upon your two competitors Kotaemon and Ragflow. May be you can draw some inspiration from them. I especially like that you can immediately see relevant page of PDF in the UI in Kotaemon. Still haven't time to check Ragflow though.

Btw, I am really curious how do you know what source has LLM used? I mean, you know what snippets you gave to it, but it is unclear to me how one can guess what exactly was used in the answer.

I would like to move this to a discussion, but...
I understand that "hey where was your attention inside the loaded context" is the responsibility of the language model.
But the embedder decided to load a specific context from the available documents.
I get that the context window isn't going to line up with a page or paragraph, but it would be nice to spit out the part of the document that was loaded, even if it needs to be output as text.  If we have some sort of "start of two pages worth of topic ... and that's why foo was chosen as a metasyntactic variable" citation mode, then we can look at how we could implement turning that into some sort of link that highlights relevant info.
I get that it's not what we really want (exact location citation), but it would make reviewing citations of large documents like books possible.


feat(nginx): merge existing ingress controller config with generated one

What this PR does / why we need it:
Add your description
Which issue(s) this PR fixes (optional, close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
/cc @your-reviewer
Release note:

feat(nginx): merge existing ingress controller config with generated one

/lgtm


Operators "span" tags enclose whitespace characters at the beginning of a line

I have a particular setup where operators have a different font than the rest of my source code. I use the following rule in my css to apply the right style :
atom-text-editor::shadow{
    .entity.other.attribute-name {
        font-style: italic;
    }

    .lines {
      .punctuation.definition.comment,
      .keyword.operator.assignment,
      .keyword.operator.arithmetic,
      .keyword.operator.comparison,
      .keyword.operator.relational,
      .keyword.operator.spread,
      .keyword.operator.logical,
      .type.arrow.function,
      .constant.numeric {
        font-family: "Fira Code Retina";
      }
    }
}

This made me realize the following behavior when showing whitespace chars :

As you can see, the whitespaces before the spread operator don't have the same font as the others.
This is easily explained by the markup :

Is this on purpose ? I understand that it makes basically no difference in most cases, but with my setup it's a bit annoying.

Yes I am a bit lazy in the grammar and also scope some prefixed space with the scope of the following code!  I have been meaning to fix this but..... i'll try and fix sometime this week.

Awesome, thank you!
There's no rush if you have more important things to do :)

Should be fixed in this commit https://github.com/gandm/language-babel/commit/c519a889cd2ce7b99e9acaf554bd2e1fe21f8209
I will roll out before the weekend.

Can you check 2.32.3 and see if it fixes this?

It seems fixed ! Thanks a lot, that was a quick correction :)


Finding out what packages are exploding the build

Here's a way to start analyzing what packages are causing the build to explode.
import json
from pathlib import Path
import pandas as pd

pkg_files = Path("/opt/conda/conda-meta/").glob("*.json")

paths = []

for pkg_file in pkg_files:
    with pkg_file.open() as f:
        pkg = json.load(f)
        paths += pkg["paths_data"]["paths"]

df = pd.DataFrame(paths)
df = df.drop(
    [
        "path_type",
        "sha256",
        "sha256_in_prefix",
        "no_link",
        "file_mode",
        "prefix_placeholder",
    ],
    axis=1,
)
df = df.dropna()
df = df.sort_values("size_in_bytes", ascending=False)

 $ df.head(20)

                                                    _path  size_in_bytes
37480   lib/python3.9/site-packages/tensorflow/python/...    271301744.0
25649                                    lib/libavcodec.a    152533588.0
25258                                   lib/libLLVM-11.so    105929424.0
74839   x86_64-conda-linux-gnu/sysroot/usr/lib64/local...     99188496.0
85596                                    lib/librsvg-2.so     97415432.0
85597                                  lib/librsvg-2.so.2     97415432.0
85598                             lib/librsvg-2.so.2.47.0     97415432.0
91707                                   lib/libLLVM-10.so     95685352.0
50318                        lib/libQt5WebEngineCore.so.5     92408776.0
50320                   lib/libQt5WebEngineCore.so.5.12.9     92408776.0
50317                          lib/libQt5WebEngineCore.so     92408776.0
50319                     lib/libQt5WebEngineCore.so.5.12     92408776.0
122433                                         bin/pandoc     76341040.0
25661                                   lib/libavformat.a     47442464.0
33070   site-packages/compliance_checker/tests/data/ma...     42981152.0
40321                                      lib/libgdal.so     35682192.0
40323                               lib/libgdal.so.28.0.1     35682192.0
40322                                   lib/libgdal.so.28     35682192.0
80393                                lib/libclang.so.11.1     35233816.0
80392                                     lib/libclang.so     35233816.0

Caching some of the build with #23
@ocefpaf

Removing tensorflow only slims things down another half gig
➜ docker images
REPOSITORY             TAG       IMAGE ID       CREATED          SIZE
ohw-no-py-tensorflow   latest    200d425d469f   52 seconds ago   5.3GB
ohw-cache-apt          latest    c646a0031f14   9 hours ago      5.81GB
ohw-cache              latest    274e85773a32   9 hours ago      5.81GB
ohw                    latest    d2014651c42b   10 hours ago     8.27GB

Archive.zip

causing the build to explode.

What do you mean by exploding? We are not able to upload that?
PS: let's remove tensorflow!

I meant size in this case, but also didn't have permissions for uploading to Docker Hub.

I'll remove tensorflow in #23

no_link is no longer in the dataframe, so it should now be
import json
from pathlib import Path
import pandas as pd

pkg_files = Path("/opt/conda/conda-meta/").glob("*.json")

paths = []

for pkg_file in pkg_files:
    with pkg_file.open() as f:
        pkg = json.load(f)
        paths += pkg["paths_data"]["paths"]

df = pd.DataFrame(paths)
df = df.drop(
    [
        "path_type",
        "sha256",
        "sha256_in_prefix",
        # "no_link",
        "file_mode",
        "prefix_placeholder",
    ],
    axis=1,
)
df = df.dropna()
df = df.sort_values("size_in_bytes", ascending=False)
df

It's also useful to include mamba in the environment for mamba repoquery to get dependencies.


toolbar.html.twig overrides core classes

With the new Openfed version, we'll have to drop toolbar_themes module, because it is not compatible with Drupal9 and is poorly maintained.
As such, we'll start using Drupal core Toolbar and its template.
I see that Kiso overrides this template (templates/toolbar.html.twig), maybe to make it more accessible, but it also overrides important classes, and without them the style is broken.
Since templates/toolbar.html.twig is based on core toolbar.html.twig can you make that any "toolbar_" class is converted to "toolbar-" like in core: https://api.drupal.org/api/drupal/core!modules!toolbar!templates!toolbar.html.twig/8.9.x
I don't see a reason to override these classes.

Indeed, I made some markup improvements regarding unexpected <h2-3> heading elements which are used in the core toolbar.html.twig template but appear before the <h1> first-level heading element giving the document a wrong structure for Assistive Technologies (AT).
Concerning the core classes override, Kiso stylesheets are based on the Drupal CSS coding standards and best practices (BEM Methodology) regarding the formatting of class names:
/* BEM for Block, Element & Modifier */
.block { ... }
.block__element { ... }
.block__element--modifier

/* Administrative toolbar class names formatting */
.toolbar { ... }
.toolbar__bar { ... }
.toolbar__tab { ... }
.toolbar-menu { ... }
.toolbar-menu__item { ... }
.toolbar-menu__item--expanded { ... }
.toolbar-menu__item--collapsed { ... }

However, I noticed that I didn't create the stylesheet which is supposed to apply the Kiso "Lool & Feel". I gonna fix this oversight! :-\

Regarding the overridden classes, I gonna revert to the original ones for this issue in priority. For a next release, I will upgrade them in the following issue #46 to meet the Kiso stylesheets guidelines.

This enhancement will be available with the next release 2.7.


Check for movement when trying to sit. Fix sitting animation in 3rd person view

Try to press a chair while running.

A terrible bug? How stopping? What to do? Where do I fly?

This PR fixes this.

https://github.com/minetest-mods/ts_furniture/issues/3

This PR fixes this, too.

Thanks for your contribution!


Test Issue created at Fri, 28 May 2021 11:46:41 GMT

Test issue body  Fri, 28 May 2021 11:46:41 GMT

Comment on issue at Fri, 28 May 2021 11:46:42 GMT


fix: add tests

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

PowerShell Gallery
GitHub release

Your semantic-release bot :package::rocket:


optional buffer-face-mode call

I've been using darkroom-mode but it has a few bugs that I'm hoping writeroom-mode can address. Specifically, darkroom-mode uses font scaling, which is incompatible with pretty much everything (e.g. company-mode).
As a workaround, I use buffer-face-mode like this:
    (set (make-local-variable 'buffer-face-mode-face) '(:height 200))
    (buffer-face-mode 1)

but without a writeroom-mode-hook I have no way of attaching this. Can you please consider adding a hook (that gets called both on load and unload) or adding a new writeroom-face-height variable that (if non-nil) will trigger a load of buffer-face-mode.

Well, a writeroom-mode-hook is defined automatically, because I use the macro define-minor-mode to define writeroom-mode. Isn't that sufficient for your use case?

aah, indeed it probably is. Sorry I was reading the source and didn't grep / grok that. Does that provide an unload hook as well?

I'm actually not sure whether minor mode hooks are rerun when the mode is disabled, though I suspect they're not. (There doesn't seem to be a function for it.)
Let me know if you feel you could use a variable writeroom-local-effects, which get undone when writeroom-mode is deactivated (similar to writeroom-global-effects).


download angular-material.js

Is it possible to download angular-material.js like it could be done with angular.js - for cases where browser app is on local network with no internet access?

Certainly, see #262.

Hm - I could not find anything about where to download from in that issue ..
Could you be more specific - provide a link if you have it?
Much appreciated

@zoplex - This doesn't pertain to 'crouton' as such (hence the irrelevant link), you might want to try this  - http://goo.gl/gyo85g


Current gcr.io/buildpacks/builder:v1 broken

Surprised nobody has mentioned this, I'm the only person crazy enough not to 'pin' versions? Noticed this a couple days ago:
Canceled build for *my image*
validating stack mixins: buildpack<EMAIL_ADDRESS>does not support stack 'google'

Can confirm it occurs for Skaffold 1.34 and 1.35.2
Is this a bug or a feature?
Whilst pondering the above question I built the sample-go app in the readme. It 'builds' (the little there is to build)
(pack build --builder gcr.io/buildpacks/builder:v1 sample-go)
So to add more info, I'm running a pretty standard Go(lang) build, apart from settings ENVs
GOOGLE_BUILDABLE
GOOGLE_CLEAR_SOURCE
GOOGLE_DEVMODE

Yeeep, our CI builds are also failing now.

@xSAVIKx  my 'fix'
gcr.io/buildpacks/builder@sha256:386d9463cbab60718d1f9e213df9e3ccf5e166190696be69bdd16d7d466875bd
(picked a pretty random SHA, but works for me)

You can pick some latest one from here as well: https://console.cloud.google.com/gcr/images/buildpacks/GLOBAL/builder

@matthewrobertson @MontyCarter do you maybe know what can the the issue here and when this can be addressed?

@xSAVIKx my 'fix' gcr.io/buildpacks/builder@sha256:386d9463cbab60718d1f9e213df9e3ccf5e166190696be69bdd16d7d466875bd
(picked a pretty random SHA, but works for me)

Also, this image fails for me with the following error:
Digest: sha256:386d9463cbab60718d1f9e213df9e3ccf5e166190696be69bdd16d7d466875bd
Status: Downloaded newer image for gcr.io/buildpacks/builder:17cbb932fff1a1487d75bfee65c0f0e21a0abc2c
ERROR: failed to build: fetching lifecycle image: resolve auth for ref buildpacksio/lifecycle:0.13.1: error getting credentials - err: exit status 1, out: ``

I'm building a container in the local env. Just FYI, I have another build that uses gcloud builds submit --pack and that one works fine in the cloud.
I may attempt to move my current build to Cloud Build env and check.

Sounds like this is a regression caused by the update to use wildcard stacks in: https://github.com/GoogleCloudPlatform/buildpacks/commit/3ce58348bc60686a2a2051cc438ac59350ee4ced
Sorry for the inconvenience. I am not able to reproduce the issue on my end. I suspect it is caused by some conflicting version of pack that doesn't support wildcard stacks.
@xSAVIKx  or @DGollings could you tell me more about how your are building your applications? If possible could you share a cloudbuild config that reproduces the error?

@briandealwis maybe you have more info about what could be happening here? Is skaffold using an old version of pack?

 It will take a few hours for the CI to publish a new builder.

I can confirm that the bc039ad change fixes the builder (resulted in the new builder image: https://console.cloud.google.com/gcr/images/buildpacks/global/builder@sha256:4a9460c52d511b306acdd5a939c5607f08654750fd4c10e00a5fad296b4eccd2/details)
I basically have a pack build running in GitHub Actions environment, smth like:
export GCP_PROJECT="${GCP_PROJECT}"
export IMAGE_NAME="gcr.io/${GCP_PROJECT}/<image-name>"

GITHUB_SHA=$( git rev-parse HEAD )
BUILDER_IMAGE_NAME="gcr.io/buildpacks/builder:v1"
RUNTIME_IMAGE_NAME="gcr.io/${GCP_PROJECT}/<runtime-image-name>"
IMAGE_VERSION="0.0.1"
pack build "${IMAGE_NAME}" \
  --builder "${BUILDER_IMAGE_NAME}" \
  --run-image "${RUNTIME_IMAGE_NAME}" \
  --tag "${IMAGE_NAME}:latest" \
  --tag "${IMAGE_NAME}:${GITHUB_SHA}" \
  --tag "${IMAGE_NAME}:${GITHUB_SHA:0:7}" \
  --tag "${IMAGE_NAME}:v${IMAGE_VERSION}" \
  --publish

And this setup builds a Python application based on Functions Framework.

@matthewrobertson
Good news, I can reproduce. Normally I build and deploy everything with Skaffold, but sometimes I do a hotfix using a simple CLI oneliner
Using image gcr.io/buildpacks/builder@sha256:4d69800df46491214311eb590a3a213e02d6e04fafd26232067afc4a0943e749
This works (identical to Skaffold)
export SERVICENAME=foo;export TAG=$(date +%s); pack build eu.gcr.io/our-shield-303914/${${SERVICENAME}}-go:$TAG  --builder gcr.io/buildpacks/builder@sha256:4d69800df46491214311eb590a3a213e02d6e04fafd26232067afc4a0943e749 --env=GOOGLE_BUILDABLE="./api/${${SERVICENAME}}/cmd/${${SERVICENAME}}" --env=GOOGLE_CLEAR_SOURCE=true --env=GOOGLE_DEVMODE=true

But, I checked the Skaffold go.mod and they use version 0.18
Downloaded 0.18 manually, ran the exact some command (against the old binary of course)
❯ export SERVICENAME=foo;export TAG=$(date +%s); ./pack build eu.gcr.io/our-shield-303914/${${SERVICENAME}}-go:$TAG  --builder gcr.io/buildpacks/builder@sha256:4d69800df46491214311eb590a3a213e02d6e04fafd26232067afc4a0943e749 --env=GOOGLE_BUILDABLE="./api/${${SERVICENAME}}/cmd/${${SERVICENAME}}" --env=GOOGLE_CLEAR_SOURCE=true --env=GOOGLE_DEVMODE=true

results in
gcr.io/buildpacks/builder@sha256:4d69800df46491214311eb590a3a213e02d6e04fafd26232067afc4a0943e749: Pulling from buildpacks/builder
Digest: sha256:4d69800df46491214311eb590a3a213e02d6e04fafd26232067afc4a0943e749
Status: Image is up to date for gcr.io/buildpacks/builder@sha256:4d69800df46491214311eb590a3a213e02d6e04fafd26232067afc4a0943e749
v1: Pulling from buildpacks/gcp/run
Digest: sha256:1004d44153a76910b05f9bc060682172452ffc05eee1331aeb69e03e3a9f01ea
Status: Image is up to date for gcr.io/buildpacks/gcp/run:v1
ERROR: failed to build: validating stack mixins: buildpack<EMAIL_ADDRESS>does not support stack google
❯ ./pack version
0.18.0+git-e00ee4a.build-2328

:)

Sorry I have a patch for Skaffold to update to a later version of pack-as-a- library. I’ll try to get that in ASAP.

I think most open source customers will be using the buildpacks with the google stack so we can keep that one in the toml files and add the wildcard for the product specific ones.

It's committed to HEAD (https://github.com/GoogleContainerTools/skaffold/pull/6979) and will be in the next release.  Anybody needing this now can use the "bleeding edge" binaries from https://skaffold.dev/docs/install


Add Import Validation for Emissions Data Codes

When a user imports an emissions file where a code (spanScaleCode, testResultCode, gasLevelCode, etc.) is not valid,
an import check validation message should appear, directing them to enter a valid code.
Should the response message include a list of valid codes?
Should this be covered under an existing Import check? (17?)

duplicate of #5096


Unable to get any data from Response::text()

Minimal example:
use std::io::{Write, Cursor};
use std::fs::File;

pub type Result<T> = std::result::Result<T, Box<(dyn std::error::Error + Send + Sync)>>;

fn main() -> Result<()> {
    let client = reqwest::Client::builder().build()?;

    let mut res = client.get("https://www.google.com")
        .send()?
        .error_for_status()?;

    //let mut cursor = File::create("response.bin")?;
    let mut cursor = Cursor::new(vec![]);
    let s = res.copy_to(&mut cursor)?;
    dbg!(s); // contains the data?!
    dbg!(&res.text()?); // empty?
    Ok(())
}

Terminal Output:
   Compiling test_reqwest v0.1.0
warning: unused import: `Write`
 --> src/main.rs:1:15
  |
1 | use std::io::{Write, Cursor};
  |               ^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

warning: unused import: `std::fs::File`
 --> src/main.rs:2:5
  |
2 | use std::fs::File;
  |     ^^^^^^^^^^^^^

    Finished dev [unoptimized + debuginfo] target(s) in 3.23s
     Running `target/debug/test_reqwest`
[src/main.rs:21] s = 11531
[src/main.rs:22] &res.text()? = ""

Cargo.toml:
reqwest = "0.9"

I attached the compiled linux debug binary as zip.
test_reqwest.zip

The Response only streams the body. Once you've read it once, reading again will just return empty.

hmm, I totally missed the Note.
Is there any reason, to not return an &str instead of a String?

It needs to create a String to push all the data into. After doing that, it cannot return a &str of that String since then the String would drop and deallocate when the function returns.


切换VLESS_TLS和xtls都会错误

切换VLESS_TLS和xtls都会错误
错误提示
xxx.cf:Verify error:Invalid response from http://xxx.cf/.well-known/acme-challenge/Jn43dtIWrPOQsOx4iXK0ulQAB4Fj06kss36QczA_4wM [<IP_ADDRESS>]:
Please add '--debug' or '--log' to check more details.
[Tue 13 Apr 2021 07:58:48 AM UTC] See: https://github.com/acmesh-official/acme.sh/wiki/How-to-debug-acme.sh

It is recommended to use a paid domain.


Get the current font.

Description
Following on from https://github.com/typst/typst/discussions/998: there doesn't seem to be any way to get the current font.
Use Case
So that another font can be appended to the font list, without needing to manually know which font is currently being used. (In the above link, when looking to add a fallback font that supports emojis.)

Also related to #763. It would in general be nice if there were some generalized mechanism to get the current value of settable parameters of functions.

It would be also useful to have some way to get font metrics. It would give a way to (for example) know where overline and underline is by default and offset it from there (as for now overline offset sets it from baseline so we can't just "move overline 2pt up")
This is more than just get previously set rule, with metrics it's also some evaluation

Closing in favor of #763. Retrieving font metrics could be a separate feature request.


Update the upgrade part of the installation guide

Proposed changes
To be complied with the change made by #1466
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[x] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

This is the follow-up task of #1466 (changing debian packages maintainer scripts). The rendered doc is here.


Added heapify file

Have you read the Contributing Guidelines on Pull Requests?
yes
(Write your answer here.)
Description
Heapify is the faster method to create heap. In this process, all elements are not processed.
i.e. Leaf elements are not processed. This makes the process faster.
(Write your answer here.)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#5187
(Write your answer here.)

@anush @ankitaggarwal23 can you please review my PR .

@anushkrishnav  @ankitaggarwal23 can you please review PR

I have updated required changes. Please review my pr.
@anushkrishnav  @bhav09 @gat786 @atarax665 @Apoorve73  @siddharth25pandey @ankitaggarwal23

I have updated required changes. Please review my pr.
@anushkrishnav @bhav09 @gat786 @atarax665 @Apoorve73 @siddharth25pandey @ankitaggarwal23


custom parameter types support

Hi,
I want to use cucumber's custom parameter type definition feature, so I implemented it in cytorus.
This PR contains 2 commits:

1st commit adds and exports defineParameterType function with the same interface as cucumber-js

I export it from src/Registry.js, but would it be better to export from src/Globals.js ?

2nd commit adjusts the loading order of the .js (or .ts) files to allows users to define their own parameter types anywhere under the cypress/integration directory.

Please have a look.

@koichirok Yes, it'll be good to have common place to import library functions. So Global.js is recommended.

@koichirok Yes, it'll be good to have common place to import library functions. So Global.js is recommended.

Done. changed defineParameterType to export from Global.js and moved its type definitions to Global.d.ts

Can you please also merge the code to pr-cucumber-type branch? So I can check it offline and merge into master.

Merged, but could not push because it did not seem to be allowed.
$ git push upstream pr-cucumber-type
remote: Permission to NaturalIntelligence/cytorus.git denied to koichirok.
fatal: unable to access 'https://github.com/NaturalIntelligence/cytorus.git/': The requested URL returned error: 403

Thanks. Let me verify the changes in local


Refine README doc of Adaptive-tool

Fixes #2714
Refine README doc of Adaptive-tool
Proposed Changes

[x] Fix Gif link issue
[x] Casing issue and typo fix
[x] Add Quick Start section
[x] Add general introduction\summary of this extension
[x] Add setting/command section
[x]  Introduce each feature more
[x] Introduce debugging features more
[x] move the "how to use" section

Very close and this is a big improvement on the current documentation.  Thanks!


add new external link filter

Describe your changes
Use a custom filter to determine if the link is external. The filter is very generic which makes it possible to keep the site-url: https://openproblems.bio
Issue ticket number and link
Closes #158
Checklist before requesting a review

[x] I have performed a self-review of my code

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality
[ ] Major changes
[ ] Minor changes
[x] Bug fixes

[x] Proposed changes are described in the CHANGELOG.md

[x] CI Preview succeeds and looks good!

the biblio ref links are opening in a new tab on the released website too #180

#180 is intentional so the PR can be reviewed

Is it possible link-external-newwindow: true shouldn't be removed? None of the links on the main page have target blank.

you are totally correct. forgot to test the external links ofcourse ...
I also added the external link icon.

hmm the results card are not fixed now :|

Do we want to keep link-external-icon: true? I could go either way on this. Also, looks like navbar docs links still don't open in new tab.

the alternative regex seems to fit the bill.
Wanted to add the icons to see what it looked like. I'm either way, maybe it is not necessary as the links will open in a new window anyway.

Also, looks like navbar docs links still don't open in new tab.

It's really strange that the links in the navbar indeed don't open in a new tab.

Looks like this is a feature that was added to quarto 1.3. I opened a separate PR for this (#188).


routing is not working

routing does not work when I add a new relic browser script but without the new relic script it works properly
I copied and past script from the new relick dashboard and removed

Hi @nisarg2023 have you by chance look at https://github.com/vercel/next.js/discussions/53746

@patrickhousley  Yes with next js 13.4.20-canary.5. Work properly

@nisarg2023 The SPA feature is not compatible with that version of NextJS. You could disable that feature by creating a custom Agent instance instead of BrowserAgent. If you need help with that, let me know and please provide the code you are using to instantiate the NR browser agent. Otherwise, you will need to upgrade the version of NextJS you are using.

Closing. This is a known issue in the NextJS router version 13.3.2-canary.2 and fixed in version 13.4.20-canary.5. Please reference the docs for more information.


Add SaveAsFile with URL and Bump to version 1.0.0

Coverage decreased (-1.2%) to 81.197% when pulling 54027d3aeff08c8e402031a0d8230a234b87b286 on dev into 3a6785deb6e2967139f3ac654a6c5c5c314d53cf on main.


NTR	normal global transcription during vegetative growth in response to heat stress

Figure 8A and B  PMID:20547592

Given that in GO, ( a) there are no separate terms for "global" transcription and ( b) "cellular response to heat stress" is an exact synonym of "cellular response to heat", the requested term boils down to
normal transcription during cellular response to heat during vegetative growth

normal transcription during cellular response to heat FYPO:0005740
edit file: c1e05ea3b8989eba0ed96dee128c7a554cf6fde9
release: aed22ae32d4bad9af58aadf470a791c96e3db8e5


Migrate CI builds from TeamCity to GitHub Actions

CI builds are currently executed by the public TeamCity server provided by JetBrains for free for OSS. Let's move everything to GitHub Actions for better integration (with GH) and maintainability, potentially also adding PR builds.
Current builds are structured as follows (each of them restore NuGet packages and then call a specific target on Orchard.proj):

Main builds

Continuous build - Major version: dev branch on every commit, "Build" target.
Continuous build - Major version: 1.10.x branch on every commit, "Build" target.
Nightly build - Major version: dev branch at midnight, "Spec" target.
Nightly build - Major version: 1.10.x branch at midnight, "Spec" target.
Release: master branch on every commit, "Build" target, zip packages (Source and Web distribution) published as artifact.

Major version (dev branch at midnight):

Build and SpecFlow Tests: "Compile" target, then calling NUnit on *.Specs.*.dll.
Build and Unit Tests: "Compile" target, then calling NUnit on *.Tests.*.dll.

Minor version: Same as the previous, but 1.10.x branch.

There are a few dozen unit tests that are failing to initialize due to braking changes as well as a dozen or so that are failing the assertion, so I'm working on those too.

I've paused the whole project in TC for now, because due to the failing tests, those builds are useless at the moment.

I've paused the whole project in TC for now

May as well delete it, it just doesn't work and we'll never use it again

Sure, we don't need it anymore when the GitHub Actions migration is complete. I'm just keeping it as a backup for now - it would be better to have something to compare the results with, but we probably won't be able to get it working again.

Fixed as part of #8686 and #8684, see this comment: https://github.com/OrchardCMS/Orchard/issues/8686#issuecomment-1985507899


IE compatibility?

G'day, I tried this polyfill and it worked great in chrome but did not seem to work in PhantomJS or IE11 - two environments we need to ensure our unit tests run in.  Is this polyfill intended only to support Chrome?  Thank you!

Have you tried using path-data-polyfill.es5.js? Unlike path-data-polyfill.js it does not rely on newly introduced ECMAScript features.

Thanks for your reply, I hadn't tried that yet, my mistake!  I'll give 'er a whirl right now!
Cheers!

The ES5 version still doesn't work in IE and seems like it also doesn't work in Chrome.  Hmmm.

I haven't tested the polyfill on IE myself, but I recall other users reporting it was working fine. Are there any errors printed in the console?

I get a "TypeError: Object doesn't support property or method 'getPathData'" in IE11.
It I log out what I'm trying to access, it is indeed a path element.  Not sure what's going on  :(

Are you getting the same error on Chrome? Are you sure you are loading the polyfill before executing any other scripts on the page?

Dude sorry please disregard my last comment, the ES5 version it working in IE, Chrome, and PhantomJS, I just made a typo!!  Thanks for replying nonetheless and sorry to be a bother!


Add VIP fields to portMappings in user mode

Report:
https://www.dropbox.com/s/czcjqqa7wxucz3f/Screenshot%202016-06-24%2017.27.09.png?dl=0
This generates:
https://www.dropbox.com/s/79l69cymf0h34ko/Screenshot%202016-06-24%2017.27.31.png?dl=0
Which is missing the VIP.

Note that the first screenshot shows host mode, I believe this to be an error in the report based on the most recent requirements I have of the modal -> JSON transformation. I have therefore fixed VIP addition in User mode.
New behaviour:

User mode is correctly detected (previously required networkType=user, now requires networkType≠bridge|host
VIP label is added in user mode.

Limitation:
It's still not clear where the VIP IP address is added, so I've set it to <IP_ADDRESS> in a constant.

Reviewer: @aldipower

LGTM

LGTM lets ship 'tis.


Is the --spirv-debug-info-version=ocl-100 needed for debugging.

@akharche @reazulhoque
The "--spirv-debug-info-version=ocl-100" in spirv_generator.generate is not used in any place. Is the flag essential for us to support GDB? If not the dead code should removed.
Originally posted by @diptorupd in https://github.com/IntelPython/numba-dppy/pull/481#r658251601

The flag is not essential in GDB supporting, it just fixes 1 error in spirv-validation. The flag is used again here: https://github.com/IntelPython/numba-dppy/pull/412


min-height not working as expected

Hi,  Thanks for great auto-expanding plug-in for textarea.
I have an issue in FF latest version (35.0) on Windows. I set rows="1" for my textarea and it is calculated as min-height:51px for single row.
The textarea element \has short word value like "Apr", "Save", "Cancel", etc
browser: FF 35.0
OS: Windows 7.0
placement: The  elements are nested in  tag.
value: Short single word (less than 10 characters)
plugin version: 0.2.0
It works correctly on Chrome (min-height:34px), but on FF it does not.
Thanks,
Paul.

Hi Paul,
What happens when you visit: http://bgrins.github.io/ExpandingTextareas/test/
Are there any failures?

Hi Dom,
Thanks for your reply.   The tests all passed on both browseres

ExpandingTextareas: Ensures the clone is at least as tall as the textarea (0, 1, 1)Rerun42 ms
Textarea wrapper CSS min-height set to textarea outer height

Well... this can be said true because textarea itself could be that tall by default if rows="1", but I think the auto-expanding plug-in should make the textarea look good even though how tall the default textarea element looks like.
Oh, I forgot one thing to mention.  I use bootstrap 3 and added the class form-control, but it doesn't matter.
So, if there are only few characters in the textarea, it should have min-height:34px so that there are no unneeded extra space vertically inside textarea.
Can you try yourself to test this case?
Thanks,
Paul.

@pzheng64 could you link to a demo page showing the problem?  You can use this jsfiddle as a starting point: http://jsfiddle.net/bgrins/wfsg5mbe/.

This looks like it could be a bug with the way Firefox 35 renders <textarea rows=1> on Windows (with out without the expanding behaviour:

The bug is noted on the mozilla bug tracker, along with a possible solution—apply the following:
textarea {
  overflow-x: hidden;
}

(unless you need a horizontal scroll bar)
This may be suitable for including in the plugin, too

This looks like it could be a bug with the way Firefox 35 renders  on Windows (with out without the expanding behaviour:

Wow, good find - that's an old bug.  I'm hesitant to add that CSS by default to the plugin to add a workaround for this edge case since it could cause some side effects.  And the comment following that one indicates that it may not work, but I haven't tested it.
Assuming this is the same issue that is being reported here, we should close this issue since we don't have control over how the browser renders that textarea.  @pzheng64 the CSS in https://github.com/bgrins/ExpandingTextareas/issues/59#issuecomment-70565900 may fix your issue, although you should probably be a bit more specific with the selector - something like:
textarea.expanding {
  overflow-x: hidden;
}

This does seem to fix the issue on Windows Firefox, however I think we should set a overflow:hidden or overflow-x:hidden on the textarea when possible (as in https://github.com/bgrins/ExpandingTextareas/issues/8#issuecomment-33510480)

if textarea has no max-height or wrap settings: overflow: hidden
else if textarea has max-height, but no wrap settings: overflow-x: hidden
else overflow: scroll


forc template implementation

closes #1380
This PR introduces forc template and removes forc init --template. forc template currently takes the following inputs:

--url : git repo URL
--project-name : name of the project
 --template-name : (optional) if specified provided name is searched in the repo. (for example counter example can be searched in sway repo)

The current implementation fetches the HEAD of the default branch of the repo URL.
Example Usage
forc template --project-name test_project --url https://github.com/FuelLabs/sway --template-name counter
forc template --project-name test_project --url https://github.com/owner/repo

Testing Steps
cargo run --bin forc template --project-name test_project --url https://github.com/FuelLabs/sway --template-name counter : should result in a new folder created "test_project" with counter example inside it. Also, the forc.toml will be edited.
cargo run --bin forc template --project-name test_project --url https://github.com/FuelLabs/sway : should result in an error since there is no Forc.toml in the root. (Because there is no Forc.toml in the root and no template name is provided which means the template should be in the root directory).
I am leaving this as a draft since there might be some other desired behavior.

I haven't looked closely at this PR just yet, but just a heads up that you'll need to add a file stub to the forc commands section of the book for the new template command. See here. The mdbook-forc-documenter plugin should then be able to automatically build the docs for the command at mdbook build time.
cc @binggh just a heads-up it looks like even though we have nice error messages when building the book locally, they're still hidden in CI: https://github.com/FuelLabs/sway/runs/6515239006. I've opened an issue https://github.com/FuelLabs/sway/issues/1617.

Nice work @kayagokalp. Pulled down and tested and seems to work nicely. Agree with @mitchmindtree comments above, especially with shortening the cli arguments down.

@mitchmindtree @JoshuaBatty Thanks for the great comments! With current changes, the following commands are valid:
forc template --template-name counter test_counter_project  (Automatically fetches sway repo and searches for counter)
forc template --url custom_url_here --template-name counter test_counter_project  (Automatically fetches the custom_url_here repo and searches for counter)


Filters' width

In docs :

we can see, that filters have set up width. What is method to archive it?

I don't understand your question, you wanna have your filters in 2 columns? if so that`s already the default, if you have 2 filters it will by default show like the screenshot above...
but if you want 3 columns for example you could do something like this:
NovaDashboard::make()
  ->addView('Main', fn (View $view) => $view
      ->addWidget(...)
      ->addFilters([...]))
  ->withMeta([ 'columns' => 3 ]),

I don't understand your question, you wanna have your filters in 2 columns? if so that`s already the default, if you have 2 filters it will by default show like the screenshot above...
but if you want 3 columns for example you could do something like this:
NovaDashboard::make()
  ->addView('Main', fn (View $view) => $view
      ->addWidget(...)
      ->addFilters([...]))
  ->withMeta([ 'columns' => 3 ]),

Thank you :-)


unexpected characters changed to encoding when tried with the tool

scenario: tried to translate parts of text in source (html/go/js). encoding for '=' didn't change back in output and converted to u003d
command used:
trans -brief zh-CN:en file://example.txt -o example-out.txt

input example file content:
          <el-form-item label="字典名称" prop="dictType">

actual output file:
<el-form-item labelu003d"Dictionary name" propu003d"dictType">

expected / output using web browser:
<el-form-item label="Dictionary name" prop="dictType">

i tried with -no-ansi but i guess it's not right option
here is output with -D
  16 bytes > HTTP/1.1 200 OK
  46 bytes > Content-Type: application/json; charset=utf-8
  32 bytes > X-Content-Type-Options: nosniff
  31 bytes > Access-Control-Allow-Origin: *
  62 bytes > Cache-Control: no-cache, no-store, max-age=0, must-revalidate
  17 bytes > Pragma: no-cache
  39 bytes > Expires: Mon, 01 Jan 1990 00:00:00 GMT
  36 bytes > Date: Tue, 10 Aug 2021 10:33:01 GMT
  80 bytes > Content-Disposition: attachment; filename="json.txt"; filename*=UTF-8''json.txt
 101 bytes > Content-Security-Policy: require-trusted-types-for 'script';report-uri /_/TranslateApiHttp/cspreport
 192 bytes > Content-Security-Policy: script-src 'report-sample' 'nonce-5TXzZFq8qxe3qGstrORw6g' 'unsafe-inline';object-src 'none';base-uri 'self';report-uri /_/TranslateApiHttp/cspreport;worker-src 'self'
  43 bytes > Cross-Origin-Resource-Policy: cross-origin
  40 bytes > Cross-Origin-Opener-Policy: same-origin
  12 bytes > Server: ESF
  20 bytes > X-XSS-Protection: 0
  28 bytes > X-Frame-Options: SAMEORIGIN
  20 bytes > Accept-Ranges: none
  22 bytes > Vary: Accept-Encoding
  18 bytes > Connection: close
  27 bytes > Transfer-Encoding: chunked
   1 bytes >
   4 bytes > 352
 823 bytes > [[["\u003cel-form-item label\u003d\"Dictionary name\" prop\u003d\"dictType\"\u003e","\u003cel-form-item label\u003d\"字典名称\" prop\u003d\"dictType\"\u003e",null,null,3,null,null,[[]],[[["51171d70a5b6c2485a24361038890141","zh_en_2020q2.md"]]]],[null,null,null,"\u003cEl-form-item label\u003d\"zìdiǎn míngchēng\" prop\u003d\"dictType\"\u003e"]],null,"zh-CN",null,null,[["\u003cel-form-item label\u003d\"字典名称\" prop\u003d\"dictType\"\u003e",null,[["\u003cel-form-item label\u003d\"Dictionary name\" prop\u003d\"dictType\"\u003e",0,true,false,[3]],["\u003cEl-form-item label \u003d \"dictionary name\" prop \u003d \"dictType\"\u003e",0,true,false,[0]]],[[0,43]],"\u003cel-form-item label\u003d\"字典名称\" prop\u003d\"dictType\"\u003e",0,0]],1,[],[["zh-CN"],null,[1],["zh-CN"]],null,null,null,null,null,null,null,null,null,[null,2]]
   2 bytes > 0
   1 bytes >
content = '''
352
[[["\u003cel-form-item label\u003d\"Dictionary name\" prop\u003d\"dictType\"\u003e","\u003cel-form-item label\u003d\"字典名称\" prop\u003d\"dictType\"\u003e",null,null,3,null,null,[[]],[[["51171d70a5b6c2485a24361038890141","zh_en_2020q2.md"]]]],[null,null,null,"\u003cEl-form-item label\u003d\"zìdiǎn míngchēng\" prop\u003d\"dictType\"\u003e"]],null,"zh-CN",null,null,[["\u003cel-form-item label\u003d\"字典名称\" prop\u003d\"dictType\"\u003e",null,[["\u003cel-form-item label\u003d\"Dictionary name\" prop\u003d\"dictType\"\u003e",0,true,false,[3]],["\u003cEl-form-item label \u003d \"dictionary name\" prop \u003d \"dictType\"\u003e",0,true,false,[0]]],[[0,43]],"\u003cel-form-item label\u003d\"字典名称\" prop\u003d\"dictType\"\u003e",0,0]],1,[],[["zh-CN"],null,[1],["zh-CN"]],null,null,null,null,null,null,null,null,null,[null,2]]
0

'''
tokens = ["352", "[", "[", "[", "\"\\u003cel-form-item label\\u003d\\\"Dictionary name\\\" prop\\u003d\\\"dictType\\\"\\u003e\"", ",", "\"\\u003cel-form-item label\\u003d\\\"字典名称\\\" prop\\u003d\\\"dictType\\\"\\u003e\"", ",", "null", ",", "null", ",", "3", ",", "null", ",", "null", ",", "[", "[", "]", "]", ",", "[", "[", "[", "\"51171d70a5b6c2485a24361038890141\"", ",", "\"zh_en_2020q2.md\"", "]", "]", "]", "]", ",", "[", "null", ",", "null", ",", "null", ",", "\"\\u003cEl-form-item label\\u003d\\\"zìdiǎn míngchēng\\\" prop\\u003d\\\"dictType\\\"\\u003e\"", "]", "]", ",", "null", ",", "\"zh-CN\"", ",", "null", ",", "null", ",", "[", "[", "\"\\u003cel-form-item label\\u003d\\\"字典名称\\\" prop\\u003d\\\"dictType\\\"\\u003e\"", ",", "null", ",", "[", "[", "\"\\u003cel-form-item label\\u003d\\\"Dictionary name\\\" prop\\u003d\\\"dictType\\\"\\u003e\"", ",", "0", ",", "true", ",", "false", ",", "[", "3", "]", "]", ",", "[", "\"\\u003cEl-form-item label \\u003d \\\"dictionary name\\\" prop \\u003d \\\"dictType\\\"\\u003e\"", ",", "0", ",", "true", ",", "false", ",", "[", "0", "]", "]", "]", ",", "[", "[", "0", ",", "43", "]", "]", ",", "\"\\u003cel-form-item label\\u003d\\\"字典名称\\\" prop\\u003d\\\"dictType\\\"\\u003e\"", ",", "0", ",", "0", "]", "]", ",", "1", ",", "[", "]", ",", "[", "[", "\"zh-CN\"", "]", ",", "null", ",", "[", "1", "]", ",", "[", "\"zh-CN\"", "]", "]", ",", "null", ",", "null", ",", "null", ",", "null", ",", "null", ",", "null", ",", "null", ",", "null", ",", "null", ",", "[", "null", ",", "2", "]", "]", "0"]
ast = {
"0,0,0,0"	"\"\\u003cel-form-item label\\u003d\\\"Dictionary name\\\" prop\\u003d\\\"dictType\\\"\\u003e\""
"0,0,0,1"	"\"\\u003cel-form-item label\\u003d\\\"字典名称\\\" prop\\u003d\\\"dictType\\\"\\u003e\""
"0,0,0,2"	"null"
"0,0,0,3"	"null"
"0,0,0,4"	"3"
"0,0,0,5"	"null"
"0,0,0,6"	"null"
"0,0,0,8,0,0,0"	"\"51171d70a5b6c2485a24361038890141\""
"0,0,0,8,0,0,1"	"\"zh_en_2020q2.md\""
"0,0,1,0"	"null"
"0,0,1,1"	"null"
"0,0,1,2"	"null"
"0,0,1,3"	"\"\\u003cEl-form-item label\\u003d\\\"zìdiǎn míngchēng\\\" prop\\u003d\\\"dictType\\\"\\u003e\""
"0,1"	"null"
"0,2"	"\"zh-CN\""
"0,3"	"null"
"0,4"	"null"
"0,5,0,0"	"\"\\u003cel-form-item label\\u003d\\\"字典名称\\\" prop\\u003d\\\"dictType\\\"\\u003e\""
"0,5,0,1"	"null"
"0,5,0,2,0,0"	"\"\\u003cel-form-item label\\u003d\\\"Dictionary name\\\" prop\\u003d\\\"dictType\\\"\\u003e\""
"0,5,0,2,0,1"	"0"
"0,5,0,2,0,2"	"true"
"0,5,0,2,0,3"	"false"
"0,5,0,2,0,4,0"	"3"
"0,5,0,2,1,0"	"\"\\u003cEl-form-item label \\u003d \\\"dictionary name\\\" prop \\u003d \\\"dictType\\\"\\u003e\""
"0,5,0,2,1,1"	"0"
"0,5,0,2,1,2"	"true"
"0,5,0,2,1,3"	"false"
"0,5,0,2,1,4,0"	"0"
"0,5,0,3,0,0"	"0"
"0,5,0,3,0,1"	"43"
"0,5,0,4"	"\"\\u003cel-form-item label\\u003d\\\"字典名称\\\" prop\\u003d\\\"dictType\\\"\\u003e\""
"0,5,0,5"	"0"
"0,5,0,6"	"0"
"0,6"	"1"
"0,8,0,0"	"\"zh-CN\""
"0,8,1"	"null"
"0,8,2,0"	"1"
"0,8,3,0"	"\"zh-CN\""
"0,9"	"null"
"0,10"	"null"
"0,11"	"null"
"0,12"	"null"
"0,13"	"null"
"0,14"	"null"
"0,15"	"null"
"0,16"	"null"
"0,17"	"null"
"0,18,0"	"null"
"0,18,1"	"2"
"0"	"0"
}

❯ trans -V
Translate Shell       <IP_ADDRESS>-release

platform              Linux
terminal type         xterm-256color
bi-di emulator        [N/A]
gawk (GNU Awk)        5.1.0
fribidi (GNU FriBidi) 1.0.10
audio player          mpg123
terminal pager        less
web browser           xdg-open
user locale           en_CA.UTF-8 (English)
home language         en
source language       auto
target language       en
translation engine    google
proxy                 [NONE]
user-agent            Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.138 Safari/537.36
ip version            [DEFAULT]
theme                 default
init file             [NONE]

Report bugs to:       https://github.com/soimort/translate-shell/issues

Thank you for peeking into the issue.

Thanks for reporting. Fixed now.


Fixed icons

Quick fix to icons:
File Manager finally has a plus
Added icon to review and replaced settings cog

Also deleted sidebar duplicate as it wasnt connected to anything


It cannot work

If i input the following command:
(interfuser) cai@cai:~/InterFuser$ bash ./leaderboard/scripts/run_evaluation.sh
then output the result and get out and no other things happen!
pygame 2.1.2 (SDL 2.0.16, Python 3.7.16)
Hello from the pygame community. https://www.pygame.org/contribute.html

Registering the global statistics

This is the first running:
...
========= Preparing RouteScenario_41 (repetition 0) =========

Setting up the agent

Could not set up the required agent:

Unknown model (interfuser_baseline)

Traceback (most recent call last):
File "leaderboard/leaderboard/leaderboard_evaluator.py", line 269, in _load_and_run_scenario
self.agent_instance = getattr(self.module_agent, agent_class_name)(args.agent_config)
File "/home/cai/InterFuser/leaderboard/leaderboard/autoagents/autonomous_agent.py", line 45, in init
self.setup(path_to_conf_file)
File "leaderboard/team_code/interfuser_agent.py", line 204, in setup
self.net = create_model(self.config.model)
File "/home/cai/anaconda3/envs/interfuser/lib/python3.7/site-packages/timm/models/_factory.py", line 110, in create_model
raise RuntimeError('Unknown model (%s)' % model_name)
RuntimeError: Unknown model (interfuser_baseline)

Registering the route statistics
Registering the global statistics

More question:How to modify the scenario I want?
e.g. I want to test the interfuser_agent in the intersection scenario by setting some NPC vehicles controlled by RL, the NPC's initial position and velocity can be defined anyway

Hi!

e.g. I want to test the interfuser_agent in the intersection scenario by setting some NPC vehicles controlled by RL, the NPC's initial position and velocity can be defined anyway

If the scenario that you want to test includes some agents controlled by RL, i think it's a very difficult thing. You may need to rewrite the whole benchmark framewoke.
If the scenario that you want to test includes some agents controlled by the special rule, you may need to create a new scenario file in the existing framework. You can refer to my other repo: DOS, i have created four types of scenario in that repo.

Hi!

e.g. I want to test the interfuser_agent in the intersection scenario by setting some NPC vehicles controlled by RL, the NPC's initial position and velocity can be defined anyway

If the scenario that you want to test includes some agents controlled by RL, i think it's a very difficult thing. You may need to rewrite the whole benchmark framewoke.
If the scenario that you want to test includes some agents controlled by the special rule, you may need to create a new scenario file in the existing framework. You can refer to my other repo: DOS, i have created four types of scenario in that repo.

I am interesting to create the waypoints file (.xml) to guide the agent to follow, how do i make one route waypoints I want?

Hi！
Here is the website to view the xodr map file:
https://odrviewer.io/
You can use it to load carla map files and choose the waypoints sequence that you want the car to follow.

But i remember the coordinate system of the carla map and the website is differnet, maybr (x, y) -> (y, -x)? You can try it.

Hi！ Here is the website to view the xodr map file: https://odrviewer.io/
You can use it to load carla map files and choose the waypoints sequence that you want the car to follow.

this website seems to output .obj file only , I don't know how to use it. Your .xml files in ./leaderboard/data are made by this website?

We do not use the outpur of the website directly. We upload the carla map file (*.xodr, which can be dumpped from the carla python api) and we can get the visualization of carla map. Finally, we choose the positions as the start point and end point (maybe have some other passing waypoints), and write them into the xml files.

We do not use the outpur of the website directly. We upload the carla map file (*.xodr, which can be dumpped from the carla python api) and we can get the visualization of carla map. Finally, we choose the positions as the start point and end point (maybe have some other passing waypoints), and write them into the xml files.

thanks! I have met another problem: the interfuser agent vehicle is set to cruise from A point to B point, the first time it can work properly, but when I use the CARLA 'set_transform' function to move the agent from the its location when it's on its way, it don't work properly. e.g. If the first time it choose to turn left at a intersection, after 'set_transform' to the initial location before the agent complete its route, the second time the agent may will not turn left and don't work normally. I am wondering the route completion remark or other things is not reset but I dont know how to adjust. So how to solve this problem

Hi!
The function 'set_transform' is always used in the position initialization of the backgroung agents (like the walkers, running-light cars). The agent (ego-car) will cruise the route which is generated by the A* algorithm when we start the route. Then if you change the agent's poisition, it can not follow the previous generated navigation route (which may have been passed part o it ) . If you want the agent follow the accurate route, you can design more dense navigation points in the xml files. You can also refer the code related to the scenario, like https://github.com/opendilab/DOS/blob/main/scenario_runner/srunner/scenarios/cut_in_active_object.py


Drop down menu for integer sliders on flow tab show decimals

The drop-down menu for the integer sliders in the flow tab (e.g. number of X, Y, and Z seeds) display an option to set the number of decimals. These are integer values and there are no decimals.
To reproduce

Load Duku.nc
Create a flow renderer.
control-right click on one of the seed sliders.

possible solution: only show decmials when scientific notation used

possible solution: only show decmials when scientific notation used


I should be able to turn off the "pasting multiple lines" warning.

Description of the new feature/enhancement

When I paste into the terminal and my copied text includes a newline, I get a warning stating

You are about to paste text that contains multiple lines. If you paste this text into your shell, it may result in the unexpected execution of commands. Do you wish to continue?

Since I perform this action many times a day as part of my work, I'm well aware of the risk inherent to pasting commands with newlines in them. I would love a way to turn this warning off completely in the app's settings.
Proposed technical implementation details (optional)

You can do that in the global section by using:
 "multiLinePasteWarning": false,
Check the release notes of v1.2.2022.0

Yup.

This is also in the documentation at https://docs.microsoft.com/windows/terminal!

I tried adding the above to my settings but I still get prompted. I also don't see this in defaults.json?
Select-String -Pattern "Paste" -Path "C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.3.2651.0_x64__8wekyb3d8bbwe\defaults.json"

C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.3.2651.0_x64__8wekyb3d8bbwe\defaults.json:335:        {
"command": "paste", "keys": "ctrl+shift+v" },
C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.3.2651.0_x64__8wekyb3d8bbwe\defaults.json:336:        {
"command": "paste", "keys": "shift+insert" },

Thanks

@divsyd Put it in the "root" of the settings.

Working example
    // This file was initially generated by Windows Terminal 1.3.2651.0
    // It should still be usable in newer versions, but newer versions might have additional
    // settings, help text, or changes that you will not see unless you clear this file
    // and let us generate a new one for you.

    // To view the default settings, hold "alt" while clicking on the "Settings" button.
    // For documentation on these settings, see: https://aka.ms/terminal-documentation
    {
        "multiLinePasteWarning": false,

        "$schema": "https://aka.ms/terminal-profiles-schema",

        "defaultProfile": "{837e823b-6f8e-5b96-be1e-a29c2a12456a}",

        // You can add more global application settings here.
        // To learn more about global settings, visit https://aka.ms/terminal-global-settings

        // If enabled, selections are automatically copied to your clipboard.
        "copyOnSelect": true,

        // If enabled, formatted data is also copied to your clipboard
        "copyFormatting": true,

        // A profile specifies a command to execute paired with information about how it should look and feel.
        // Each one of them will appear in the 'New Tab' dropdown,
        //   and can be invoked from the commandline with `wt.exe -p xxx`
        // To learn more about profiles, visit https://aka.ms/terminal-profile-settings
        "profiles":
        {
    [...]

i can only recommend to keep the warning on even when its annoying:
https://www.wizer-training.com/blog/copy-paste

"I  use a "scrap" file in a text editor to test what I copied from a web site, since with browsers you can never be sure what exactly you just copied."
https://superuser.com/questions/355383/how-can-i-prevent-or-interrupt-accidental-copy-paste-in-the-terminal

Why don't you make a checkbox option in that warning? This is annoying

We'd love to add a checkbox! See #6641. If you'd like to add them yourself, we'd love the contribution! Now that we've got state.json, adding the checkboxes should be a lot easier!


Cannot load plugin–jar does not contain Plugin.yml

Running into this issue on a Nodecraft server running Paper 1.17* and Bits & Chisels 2.6.0. Any insight?

This mod is for the fabric mod loader and will not load on a paper server. This mod fundamentally cannot exist on a paper server.