fix: Scan - Clicking outside of the allow location modal submits the scan expense.

Details

Fixed Issues
$ https://github.com/Expensify/App/issues/47857
PROPOSAL: https://github.com/Expensify/App/issues/47857#issuecomment-2304589085
Tests

Create a workspace
Make sure the location permission is denied on the browser settings
Go to workspace chat and click on plus > Submit expense
Go to scan tab
Upload an attachment to scan and click on submit
After the allow location modal opens up click somewhere else outside the modal
Verify allow location modal is closed and the app doesn't submit the scan expense

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

Offline tests

Create a workspace
Make sure the location permission is denied on the browser settings
Go to workspace chat and click on plus > Submit expense
Go to scan tab
Upload an attachment to scan and click on submit
After the allow location modal opens up click somewhere else outside the modal
Verify allow location modal is closed and the app doesn't submit the scan expense

QA Steps

Create a workspace
Make sure the location permission is denied on the browser settings
Go to workspace chat and click on plus > Submit expense
Go to scan tab
Upload an attachment to scan and click on submit
After the allow location modal opens up click somewhere else outside the modal
Verify allow location modal is closed and the app doesn't submit the scan expense

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

PR Author Checklist

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

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

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

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

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

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

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

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/e5d96d99-f1b3-4e03-ab90-9318394f77d4

Android: mWeb Chrome
https://github.com/user-attachments/assets/e4393745-06df-483a-8e5f-12843b565cd4

iOS: Native
https://github.com/user-attachments/assets/68201782-2c11-49a6-b8d8-48ee06f84eb8

iOS: mWeb Safari
https://github.com/user-attachments/assets/7972447c-ff29-4e93-8b78-310e19c916a2

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/543b97ad-c44a-4990-b449-fe0242a9088e

MacOS: Desktop

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

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

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/4a2eb312-f7a7-4206-946b-8e04a82b52c6

Android: mWeb Chrome
https://github.com/user-attachments/assets/649aef1a-7a59-4ef2-a255-7de22509baef

iOS: Native
https://github.com/user-attachments/assets/7bbdabac-993c-46f0-a62c-a5c022809930

iOS: mWeb Safari
https://github.com/user-attachments/assets/87a8ed41-362e-4871-8ddf-a3627fe7fd95

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/f1773561-1098-4cee-844c-24d73757cd00

MacOS: Desktop
https://github.com/user-attachments/assets/860c2103-31d3-456b-bccc-a06d136a8df8

I'm unable to reset the location permission data to get the modal in the Electron app. Could you please help with that if you know anything about it?

@Krishna2323 I was also not able to have the location permission modal show-up on Desktop, even tried building staging app locally but failed. Given that the implementation works as expected on Web, I think it's safe to assume that the fix will work the same on the Desktop app, therefore we're good to move on without testing Desktop.

I will 🟢 Approve as soon as the Requested changes are addresed.

Ignoring I included screenshots or videos for tests on [all platforms](https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#make-sure-you-can-test-on-all-platforms)

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


has no exported member 'OpaqueToken'

I update my ionic application for angular5 but since then I have this problem with the library

Are you using the latest version of the library? (v8.0.0)

"@ngx-translate/core": "^6.0.1"

Then update to 8.0.0 and it'll be fixed.

since v4.0.0 because it does not support type information, use InjectionToken<?>


Fix missing gitgraph css import

The filter was wrongly excluding the gitGraph.css file. Need to clean this up later so that imports are always relative to the source file (which is not the case for fonts right now).
Regressed by: https://github.com/go-gitea/gitea/pull/11997

Mentioned import cleanup done in https://github.com/go-gitea/gitea/pull/12114.


Add normalizers

• Lowercase
• NFD
• NFC
• NFKD
• NFKC
Related #4

Nice! Let me know when you want a review :)

Yes please, I will also give another look at the tests.


Guide to enable Xdebug on WSL system

Hello @davidalger
The warden env is working just perfect with the brew setup and other configuration, quick easy and fast. Kudos to it.
I am struggling to enable and use Xdebug with PHPStorm, it would be great if you or any one other can help me with setting this up.
Thanks,

Have you checked this configuration setup for PHPStorm?


[HOPS-1562] ACL mask modification failed due to ClassCastException

Make sure there is no duplicate PR for this issue

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added and passed (for bug fixes / features)
[x] HOPS JIRA issue has been opened for this PR
[x] All commits have been squashed down to a single commit
[x] The commit message has the following format: [HOPS-XXX] message

Post a link to the associated JIRA issue
https://hopshadoop.atlassian.net/browse/HOPS-1562

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Can one of the admins verify this patch?

Submitted as part of https://github.com/hopshadoop/hops/pull/787


I encountered an issue

While developing with ngx-admin, I encountered an issue where after opening one nbDialog and then another, upon closing the last opened nbDialog, a light gray blank space appears at the bottom of the page, and something resembling a vertical scrollbar appears on the right side. However, when I close the first opened nbDialog, this phenomenon disappears. What is the cause of this, is it a bug in Nebular, and why does it not always occur?

😧😧😧😧😧😧😧😧😧😧


Property Field Number round value on construction

Hello,
The property field number is loosing decimal value on futher edits of the properties.
Category

[ ] Enhancement
[X] Bug
[ ] Question

Version
Version in use is 2.4.0
Expected
According to documentation, using the component PropertyFieldNumber without setting property "precision" let the decimal unchanged and not rounded.
Observed Behavior
I'm using last version (2.4) of the components and as I was using the Property Field Number, I encountered an issue. Not setting the precision value and setting the value as decimal work perfectly when editing the web part. But when saving page, and coming back on the options the decimal is lost.
Steps to Reproduce
Create a web part with a number property and use a PropertyFieldNumber to set it's value in the web part property pane, do not set precision property. Set the value to a decimal number, save refresh the page and edit back the properties. Example of property configuration :
PropertyFieldNumber('DecimalValue', {
                  label: "Decimal",
                  minValue: 0,
                  maxValue: 100,
                  key: "decimalvalue",
                  value: this.properties.DecimalValue,
                })

Submission Guidelines
The issue is due to the file PropertyFieldNumberHost where, in the constructor line 20 the state of this component is set :
this.state = {
      value: this.props.value ? this.props.value.toFixed(props.precision || 0) : null,
      roundedValue: props.value
    };

When using this, a round is done between precision set by user, and 0. This means that when we do not set any precision, the component will consider that it's equal to precision = 0, resulting in losing of the decimal information. Only workaround for now is to set a prevision with high value (like 10) but might not fit all scenarios.
Thanks!

I'm adding a point, if you set value to 0, the code initializing the component do a wrong test :
value: this.props.value ? this.props.value.toFixed(props.precision || 0) : null,

The test on this.props.value is false if value is equal to 0, as a result an error message appear that my field is not a number.

Thank you @TheThor59 for reporting this one!
The fix has been implemented and will be included in the next release.
In a meanwhile you can test it in beta version

Thanks for the help, I was working on a fix and got a conflict juste before pushing PR and didn't notice your message.
Thanks for the quick fix.
Regards.

Hello I created a new pull request as half of the issue was not fixed

The fix is in dev and should be soon available in beta version.


Adds react-bootstrap responsive navbar

I had to work on the responsive navbar for this project today so in the meantime I made the modifications here too.
It is fully based on react-bootstrap and uses react-router-bootstrap which makes the router working with the react-bootstrap components.

Looks good.

Awesome work, @benoitvallon! :+1:

:+1:

@claym @erikras @snackycracky thanks


[mgmt] datafactory release

https://github.com/Azure/sdk-release-request/issues/4744

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-arm-datafactory


feat(core): add support for serialization groups

Every property can specify its serialization groups, which are then used with explicit serialization.

Properties without the groups option are always included.

Let's consider the following entity:
@Entity()
class User {

  @PrimaryKey()
  id!: number;

  @Property()
  username!: string;

  @Property({ groups: ['public', 'private'] })
  name!: string;

  @Property({ groups: ['private'] })
  email!: string;

}

Now when you call serialize():

without the groups option, you get all the properties
with groups: ['public'] you get id, username and name properties
with groups: ['private'] you get id, username, name and email properties
with groups: [] you get only the id and username properties (those without groups)

const dto1 = serialize(user);
// User { id: 1, username: 'foo', name: 'Jon', email<EMAIL_ADDRESS>}

const dto2 = serialize(user, { groups: ['public'] });
// User { id: 1, username: 'foo', name: 'Jon' }

Hello,
This is a really good enhancement.
But, can we make it work with Embedabbles properties  too ?


Article on the physics system has moved

Matt's article has been updated and moved to Medium, and the original link has been invalidated. It would be nice to update Source/Player/Readme.md to point there instead.
It is a very inspiring design. I modelled a platformer game after this and it's been a delightful experience. Thank you.

Thanks, fixed!


requires std?

Hi, just discovered this lib. Seems like something very useful for storing data on an sd card on a microcontroller. When I try to add this library to a no_std binary I get errors with memchr, does it require std with an allocator?

   Compiling memchr v2.4.1
   Compiling generic-array v0.14.4
   Compiling regex v1.5.4
   Compiling delog v0.1.2
   Compiling serde_derive v1.0.130
   Compiling bare-metal v0.2.5
   Compiling cortex-m-rt v0.7.0
   Compiling cortex-m v0.7.3
error[E0463]: can't find crate for `std`
  |
  = note: the `thumbv7em-none-eabi` target may not support the standard library
  = note: `std` is required by `memchr` because it does not declare `#![no_std]`
  = help: consider building the standard library from source with `cargo build -Zbuild-std`

For more information about this error, try `rustc --explain E0463`.
error: could not compile `memchr` due to previous error

[package]
name = "sby-buoy"
version = "0.1.0"
edition = "2018"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
cortex-m = "0.7.3"
cortex-m-rt = "0.7.0"
littlefs2 = "0.3.1"
panic-halt = "0.2.0"

Hey, thanks for your interest. You don't need no_std (cf. https://docs.rs/memchr/2.4.1/memchr/#crate-features), we use this in https://github.com/solokeys/solo2/tree/main/runners/lpc55. I think if you link to https://github.com/solokeys/solo2/tree/main/components/c-stubs in your dependencies it might just work, if not let me know and I can try to reconstruct what exactly the dependencies are.

I remembered... In this commit: https://github.com/nickray/littlefs2/commit/cdcb99fea7a53f59e582d0b3702c5bae432a8db3 I removed a workaround for fact that the bindgen build dependency triggers the std feature of memchr. With a new cargo you can simply add a line resolver = "2" under [package] in your consuming crate. The link to a c-stubs is necessary at build time, otherwise you'll get a linker error then.
I'll try and improve the docs on this.

Thanks. I'll experiment a bit more when I get a bit further on the project

I release 0.3.2: https://docs.rs/littlefs2/0.3.2/littlefs2/#no_std

Great! That is a nice way to solve it.
On Wed, Sep 29, 2021 at 11:01 PM Nicolas Stalder @.***>
wrote:

I release 0.3.2: https://docs.rs/littlefs2/0.3.2/littlefs2/#no_std
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nickray/littlefs2/issues/14#issuecomment-930539943,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAN365LSNBL5UMLFNBRIDDUEN5CNANCNFSM5EXUU4VA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Long loading time on multi-project

We got ~140 more seconds on sbt reload when sbt-version-policy is added to our multi-project with 10 libraries under the root. The problem turned out to be that, the versionPolicyPreviousVersions setting is resolved at load time by searching the resolvers for previous versions of the project artifacts. It take ~14 seconds for each subproject. Typically we work on a single project at a time, that makes fetching previous versions for all the other projects a waste.
A potential optimization I'm thinking about is to make versionPolicyPreviousVersions a taskKey instead of settingKey, so the versions will be fetched only when actually needed. It would also need to make settingKeys depending on it taskKeys. I'm not familiar with this project to tell how much work there would be to do it. And is there anything else simpler could be done to improve?
Update:
Another issue I found is that it fails silently without trying sbt credentials for private repositories, so we waited a long time and got nothing.

Hello, thank you for explaining the issue. I agree that 140 seconds is way too long. A workaround could be to manually set versionPolicyPreviousVersions so that it is not computed at build-load-time.

It would also need to make settingKeys depending on it taskKeys.

Would that even be possible?

It take ~14 seconds for each subproject. […] Another issue I found is that it fails silently without trying sbt credentials for private repositories, so we waited a long time and got nothing.

It seems this is the heart of the issue. Unfortunately, because it is not possible to make a setting depend on task, we do not “inherit” from the standard sbt configuration to resolve dependencies, which would explain why your credentials are not taken into account. The only solution I see to this problem would be to introduce new settings to configure the dependency resolution engine here. From what I have seen in your draft PR, you only need to introduce a key to define the list of repositories to search in, which should have a default value corresponding to the current behavior. How does that sound? Please let me know if you would like more guidance.


Is there a way to not load documents in memory in order to be able to query an index?

Hello,
I am not sure if I understand the flow of the following program correctly, but suppose I have indexed thousands or millions of documents along with their embeddings in a VectorStore , Say an OpenSearchVectorStore. Why still do I still need to load the documents in the memory in order to to be able to access or query the index? I am thinking that in one call we do a semantic search to the OpenSearchVectorStore using a cosine similarity to get the right documents and then pass one or two of them as the context to the OpenAPI prompt as the subsequent call to do summarization or any other instructions we pass to the OpenAI. This is my program:
documents = SimpleDirectoryReader('../paul_graham_essay/data').load_data()
client = OpensearchVectorClient(endpoint, idx, 1536, embedding_field=embedding_field, text_field=text_field,auth=auth)
index = GPTOpensearchIndex.from_documents(documents=documents, client=client)
response = index.query("What did the author do growing up?")

what I intended to do is to ingest millions of document, get embedding for them, and store them in a vector store. Then in a separate program that is supposed to only query the index, get the relevant documents so that I can make subsequent calls to OpenAI models with prompts that has the relevant document as the context so that  I don't have to load the documents in the memory but just connect to the vector store? is there a way to access the index without having documents loaded in memory but rather only have them in the vector store? In other words, is there a way to get the GPTOpensearchIndex without passing the documents and only the client?
Alternatively, I suppose I can pass the cosine similarity script query myself? like
rdr = ElasticsearchReader(endpoint, idx,httpx_client_args ={"verify":False,"auth":("admin","admin")})
rdr.load_data(embedding_field,{
 "size": 4,
 "query": {
   "script_score": {
     "query": {
       "match_all": {}
     },
     "script": {
       "source": "knn_score",
       "lang": "knn",
       "params": {
         "field": embedding_field,
         "query_value": [vector for the question goes here],
         "space_type": "cosinesimil"
       }
     }
   }
 }
}
)

is that correct?
Thanks for your help?

Hey @rfarahmand, if you already have an existing vector store with data already inside, you can directly query data without loading documents.
You can do something like:
client = OpensearchVectorClient(endpoint, idx, 1536, embedding_field=embedding_field, text_field=text_field)
# initialize vector store
vector_store = OpensearchVectorStore(client)

And starting using that. Note that we changed our API in the latest version (0.6.0), so instead of GPTOpensearchIndex, you can just create the vector store and pass it to the GPTVectorStoreIndex

That did work, although I had to pass the embedding like this:
from llama_index.vector_stores import OpensearchVectorStore
from llama_index.vector_stores.types import VectorStoreQuery
from gpt_index.embeddings.openai import OpenAIEmbedding
from llama_index.vector_stores import OpensearchVectorClient

embed_model = OpenAIEmbedding()

auth = {"user": "admin", "pass": "admin"}
client = OpensearchVectorClient(endpoint, idx, 1536, embedding_field=embedding_field, text_field=text_field, auth=auth)
vector_store = OpensearchVectorStore(client)
res = vector_store.query(VectorStoreQuery(query_str="which projects do supermarkets?" ,query_embedding=embed_model.get_text_embedding("which projects do supermarkets?")))
print(res.nodes[0].extra_info["content"])

Thanks very much from a fellow UofT alumni ;)


Make config items of Function type in SWAGGER_UI_CONFIG work

fixes #381

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[x] Add or update relevant docs, in the docs folder and in code docstring.
[x] Add an entry in CHANGES.md summarizing the change and linking to the issue.
[x] Add *Version changed* or *Version added* note in any relevant docs and docstring.
[x] Run pytest and tox, no tests failed.

Could you add some tests to validate this feature?

Actually I've tried to add a test case to validate, but since all swagger-ui configurations will only take effect after HTML rendering, finally I found there is no way to check whether one certain configuration work as expected without a HTML rendering framework (such as requests-html, selenium).
And frameworks with HTML rendering capability are always built on top of a chromium core, which means we need to download a 100+mb file first before running test cases. So I think maybe more discussions on whether adding a HTML rendering framework to dependencies are needed.

Oh, I see. Thanks for the detailed explanation. I'll just merge this (tested locally & improved the code style).


add stream pallet api

does all need to be a package? Wondering if can be merge into one payroll etc

also CI deployment should be changed as well maybe if finally we decided to always use seperate package, I think maybe one package should be enough.
can also be named as sdk-payroll etc

we can check how acala.js is organizing their packages https://github.com/AcalaNetwork/acala.js/tree/master/packages/sdk-homa

close due to no activity


[💡 FEATURE REQUEST]: Allow specifying import paths for protobuf

Plugin
GRPC
I have an idea!
We have to specify services path as:
grpc:
  proto:
    - proto/service.proto

However, all the imports in the service.proto  are seemingly resolved as relative paths to proto/service.proto itself which not exactly a bad thing per se. However, It would be convenient to have a config option for passing import paths:
grpc:
  import_paths:
    - proto
    - /other/protos
  proto:
    - proto/service.proto

That way, proto/service.proto could have all the imports without relative paths, but more like the usual protobuf way:
syntax = "proto3";

// no relative path, it will be resolved according to `import_paths`
import "my/package/service/v1/types.proto";

Hey @rauanmayemir 👋
You can already do that. You may use *, [, ? elements to specify files to import.
For example:
grpc:
  proto:
    - proto/*
    - proto/myf?e.proto
    - proto/m[yf].proto

It's not the same.
grpc.proto: loads protobuf files from the specified paths. But if those protos have import "my/package/service/v1/types.proto" and its path is not relative to the proto/, then it will fail.
What we want is to have this:
Use proto/service.proto and if it contains imports, resolve them with import_paths.

What's the difference between specifying import path in proto with the glob syntax vs using the same but in the new import_path configuration option?

The difference is that I need all the dependency types to be loaded lazily if needed from the import_paths I provided, but I don't need RR to load all of them, accidentally loading other services.
I need to be able to control which services exactly I want RR to serve.

Then put them into the separate folder 😃
In my humble opinion, this is not the case to add a somehow a controversial additional syntax to the configuration, sorry. All protofiles, used by the service should be properly put into the corresponding folders.

It is not controversial, it is precisely how protobuf is designed to work.

protoc has -I path option where you set your import paths, and then you specify a variadic list of proto file names themselves without relative path letting protoc discover it based on import_path from -I path (like, literally try out all of them)
grpcui has -import-path which you can specify multiple times (same as -I path) and then also specify a list of needed proto files via -proto service_a.proto

Complex protobuf-driven projects do not use standalone drag-and-drop .proto files you could arbitrarily copy around, they have dependencies and dependencies of the dependencies.
Consider this structure. My protobuf/buf module has its own hierarchy, so when I say:
grpc:
  proto:
    - proto/my/package/v1/service.proto

and then it turns out that it imports my/other/package/v2/types.proto, I will have to care and hand-pick all the imported types for this to work.
What's better here is that instead of assuming that import_path is the directory of the proto file itself, allow specifying the list of paths from the config which will be universal for the whole runtime.
So it would look like this:
grpc:
  import_paths:
    # path is absolute or relative to $CWD
    - /proto_root/myapis
    - /proto_root/otherapis
  proto:
    # path is relative to protobuf (so it will iterate over import paths and try to find the actual file in one of them)
    - my/package/v1/service_a.proto
    - my/other/package/v2/service_b.proto

For backward compatibility, it will be trivial to check that if import_paths is empty, then assume it is ['$CWD', '/included_proto_file_dir'].

Then put them into the separate folder 😃

Yeah, that's how I'm doing it now and it's pain to deal with, because now my buf modules workflow is different from my runtime workflow. I literally maintain a separate module.proto that looks like this:
import "./myapis/my/package/v1/service_a.proto";
import "./myotherapis/my/other/package/v2/service_b.proto";

Where ./myapis and ./myotherapis are symlinks to ../../proto/myapis and  ../../proto/myotherapis that I maintain as buf modules (so ../../proto is buf workspace). For production I then copy those module dirs for paths to stay the same. That's too many manipulations for something that could be configurable as a natural protobuf-oriented workflow.

protoc has -I path

This is what you can achieve with the glob syntax. Not exactly the same, but somehow similar. You may include all files in the directory, or separate files if you choose to do that.

Complex protobuf-driven projects do not use standalone drag-and-drop .proto files you could arbitrarily copy around, they have dependencies and dependencies of the dependencies.

Thus, use the glob syntax to import all at once. Or import all at once from the separate folders.

because now my buf modules workflow is different from my runtime workflow

You know, this isn't a common, but your use-case.

It is indeed my use case, but one way or another, people eventually do converge on commonly adopted conventions.  https://google.com/search?q=postman+import_path


Snapchat Filters using OpenCV

Add the filters which are not present in repo

A very fun issue to contribute.
We will create our own filters on our facial images.
This issue will not be closed so many pull request. are welcomed.
I hope we will get some different filter in every new pr.
This issue can be assigned to many.

@akshitagupta15june will try working on this issue

multiple people can work on it, no need to assign, go for it

@akshitagupta15june Hey 👋
I am interested in this issue and would like to contribute to this in the best with optimized way, Let's work on it!!

I'm interested in this issue and would like to work on it.

I want to work on this issue

I'm interested in this issue and would like to work on it.

Interested in this issue going to work on it.

@akshitagupta15june I am also interested and would like to contribute.

Hi @akshitagupta15june , iam a SSOC23 contributer and I would like to resolve this issue
Can you assign me this issue? Or any other issues you might have?
Heres My Resume which you might find useful: Resume
Here is my background in ML:

6 months plus experience in working under college professors for research project on medical image classification.
Got 228 global rank in Amazon ML Challenge 2023 out of 30000+ teams participating.
Experienced with trainig machine Learning Models. Including image preprocessing techniues such as histrogram equilization, augmentation, stationary wavelet transformation, image segmentation. Along with feature extraction from models like resnet50 inceptionv3 vgg16 etc

Looking forward to hear from you.

Hi! I would like to contribute to this issue under the GSSoC'23. It seems interesting and aligned with my skills. I would be happy to take it up and work on this project.

Hello. I would like to contribute to this issue. I am a GSSoC'23 contributor. I have overviewed code of other filters of this repo and I am confident that I can work on this issue. Can you please assign this to me.

Hi Iam a GSSOC'23 Contributor I would like to contribute to it .Can you please assign it to me

I would like to work on this issue, please assign this issue to me under GSSoC'23

Hi, I'm a GSSoC'23 Contributor and would like to work on this issue. Please assign this issue to me.

Hii , I’m a GSSoc’23 Contributor and  I am Interested in this issue going to work on it.

I would like to work on this issue. I'm a GSSoC'23 contributor

I would like to contribute in this issue as a part of GSSOc 23 contributor

@akshitagupta15june I am also interested and would like to contribute.

Hello! I'm really excited about this Snapchat Filters using OpenCV project. I would love to contribute and work on it. Could you kindly assign the project to me? I'm confident that I can make a valuable contribution.

@akshitagupta15june I am really excited as it's really a fun issue and would like to contribute in it.

@akshitagupta15june i would like to contribute in it.

I would like to contribute in it

I am GSSOC 23 contributor assign this to me i want to work on this

I would like to contribute to this as a GSSOC'23 contributor.
I have reviewed the codes of many of the existing filters and I think it would be a fun activity to add new cool snapchat filters such as vibe glasses, bald head etc.

I would like to contribute to this as a GSSoC CONTRIBUTOR

Seems fun, will be working on this

@akshitagupta15june I am GSSOC 23 contributor.I want to add 'mahabali' filter to this.He is a king associated with kerala celebration onam.He has got a big moustache, gold ornaments, chubby cheeks ,a crown and a big smile. Please assign this to me.

please assign this project to me. I want to contribute with the time wrap filter.


Uncaught exception: ReferenceError: _ is not defined

After i tried to send 1 BB to myself on multisig 2 of 3 wallet second device which should authorize transaction raises this error on every start and closes after pressing ok button.

Will be fixed in the next release
https://github.com/byteball/byteballcore/commit/2d7df7244f101518698c83ccf5191e9cfffe2041

Fixed in 1.1.0


[Theme Editor]: color theme umschalten

Checkbox zum umschalten zwischen light und dark theme über Editor
https://forum.iobroker.net/topic/39522/test-adapter-material-design-widgets-v0-4-0-beta/42

I've added this in a script:
setState('vis-materialdesign.0.colors.darkTheme', !isAstroDay()); // On script startup
schedule({astro: 'sunrise'}, () => setState('vis-materialdesign.0.colors.darkTheme', false));
schedule({astro: 'sunset'}, () => setState('vis-materialdesign.0.colors.darkTheme', true));

But you probably should make this user-configurable in your adapter config.

I've added this in a script:
setState('vis-materialdesign.0.colors.darkTheme', !isAstroDay()); // On script startup
schedule({astro: 'sunrise'}, () => setState('vis-materialdesign.0.colors.darkTheme', false));
schedule({astro: 'sunset'}, () => setState('vis-materialdesign.0.colors.darkTheme', true));

But you probably should make this user-configurable in your adapter config.

@cdellasanta:
Still had this idea too.
But this is not a 'classic' daemon adapter with an instance. Means there is no main.js file, where ein can listen to astro time changes.

@cdellasanta:
Still had this idea too.
But this is not a 'classic' daemon adapter with an instance. Means there is no main.js file, where ein can listen to astro time changes.


No cancelation for disk query operation

Hi,
I have noticed that if some UIImageView switches between remote images quite fast, the SDImageCache don't have enough time to complete the following operation:
[cacheOutQueue addOperation:[[[NSInvocationOperation alloc] initWithTarget:self selector:@selector(queryDiskCacheOperation:) object:arguments] autorelease]];
before UIImageView, which is the final delegate, asks cache for image located on another URL, if the second request to cache is processed faster than this operation, so finally you get UIImageView setup with the 1st image got from disk.
You can easily check it in some synthetic test with quick filling UIImageView with images from different URLs. Or just check cancelation code, which never stops the already running NSOperation.
I may suggest to not to use NSInvocationOperation, but create your own subclass from NSOperation with correct 'isCancelled' processing.

Great catch! However, cancellation alone will not solve this issue. cancel is called on a different thread from the operation itself, and according to the docs:

-(void)cancel
If the operation has already finished executing, this method has no effect.

Issuing cancel would race with the completion of the operation, so the race condition would merely be less frequent.
The proper solution to this is to include a URL check to make sure the UIImageView is still trying to show the same URL as originally requested when the operation was queued.

Hi,
In my opinion the best solution would be a combination of these two approaches, but you should definitely start with the one you've suggested.

That could be tricky. Actually I've already implemented the URL checking solution like @adamjernst 's suggestion, which you can have a look at following commit.
https://github.com/mystcolor/SDWebImage/commit/a5f9bb9d3bd91910c42dc17f95963fe7742d0fb6
However, that's breaks multiple fetching of images using a single delegate object. (For example using SDWebImagePrefetcher or delegating the SDWebImageManagerDelegate to your viewController, the images arrives in random sequences, and the following logic isn't able to know whether that delegate is interested on the last response only or all responses, and discarded all responses not equal to the expected URL.
 // The disk query returned, but sometimes it was an older request for this delegate which is supposed to be cancelled.
 // We compare the URL and the key and see if it was the actual request.
 if ( ! [[[urls objectAtIndex:idx] absoluteString] isEqualToString:key]) {
     return;
 }

So the main objective is to let SDWebImageManager to know whether the delegate only interested on the last response or all responses, we've to come up a solution for this use case.
Using NSNotificationCenter instead of the delegation pattern could probably solve this problem (which requires a huge refactoring), but I am sure @rs originally has reason to use delegation at the first point and needed to be take consider of, and I am thinking if there's any other  ways to do.

@mystcolor your solution is very similar to my original code (which never made it into trunk), but what's missing is that you need to also check that the URL is equal for that delegate index before removing the delegate from the array. You're removing all instances of the delegate when any one of them returns, but you should only remove the one that has a corresponding URL equal to the just-completed URL. This is a bit confusing, but see my code here:
https://github.com/adamjernst/SDWebImage/commit/04f96e7809e5777ac05c9e00401591268cbe5a2c
That should do it. Let me know if I'm missing something (which I very well might be). Thanks!


Whitespace fixes

Few petty whitespace fixes.

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.


More audit, rename and invariants

Description
More on audit feedback. Renaming as in the TODOs and invariants pulled at the entry of the function
Test Plan
Unit tests

If your changes are not user-facing and do not break anything, you can skip the following section. Otherwise, please briefly describe what has changed under the Release Notes section.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

I'll actually add tests for this in another PR given there are a ton of missing tests and I have a task to bring unit tests up to snuff

fwiw and not critical, the first commit is the real change, the second commit is snapshots/docs and things you may not care about


Support for icons in NeumorphButton

Describe the request you'd like:
Request to add support for icons in NeumorphButton on either left or right and it would be amazing if we could add tint over them.
Additional context: NA

NeumorphButton extends AppCompatButton.
So the button already has the ability to show an icon.
Please refer to the following codes.
<soup.neumorphism.NeumorphButton
    android:drawablePadding="8dp"
    app:drawableStartCompat="@drawable/ic_left"
    app:drawableEndCompat="@drawable/ic_right" />


Apdex score always 1 on your sample project.

I tried to add Apdex score calculation to my project the relaised that score never change.
Look at your sample project which name is "AspNetCore2.Api.QuickStart" and its Apdex score never change too.
Result is always 1.
--------------------------------------------------------------
TIMESTAMP:<PHONE_NUMBER>50522649
MEASUREMENT: [Application.HttpRequests] Apdex
TAGS:
                 server = THB-DEV-131
                    app = MetricsSandboxMvc
                    env = configoverride
                  mtype = apdex
                   unit = result
FIELDS:
                samples = 43
                  score = 1
              satisfied = 4
             tolerating = 8
            frustrating = 31

Can you help me with this issue?
Thanks,

Thanks for raising, looks like a new bug

Looks like an interesting bug there. If your sample request count below than 100, score never change but after 100 request score changed correctly.
Hope this help to you.
Thanks


Construction Stages

Discussion regarding a binding between a value that appear twice (.fdscript and .struxml)
The value in the .struxml is correctly deserialised in FemDesign and it can be seen in the calculation method.
Other analysis settings are usually set up in the .fdscript and it should be also for the ConstructionStages calculation.
https://strusoft.fogbugz.com/f/cases/121415/Ghost_method
Analysis can not be run


ci: build on windows-2022 runner

Related to failed run https://github.com/camunda/camunda-modeler/actions/runs/6723359618

Roger that! https://github.com/camunda/camunda-modeler/actions/runs/6723359618

Oh ok the action run is frozen to the outdated windows version. We will need to wait with this till tonight.


Alibaba Cloud TableStore state Component

What content needs to be created or modified?
Alibaba Cloud TableStore state Component
Additional context
https://github.com/dapr/components-contrib/issues/914

@AaronCrawfis Note that the corresponding components-contrib https://github.com/dapr/components-contrib/issues/914 and https://github.com/dapr/components-contrib/pull/915 are not currently being tracked for v1.3

Looks like this was done as a part of https://github.com/dapr/docs/pull/1778 - think we can close this one.


Correct minor grammatical and spelling issues

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Minor README grammar and spelling corrections.

What is the current behavior? (You can also link to an open issue here)

N/A

What is the new behavior (if this is a feature change)?

N/A

Other information:

I know this commit is a bit ironic given the purpose of the repository. I do not want a t-shirt and am not participating in Hacktoberfest.

Thank you very much @kylef000! 🤘


[stable/prometheus] Configure enableServiceLinks for server

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Prometheus 2.18 supports experimental Jaegar tracing which is configured by environment variables. However, if you have existing services called jaegar, this might cause issues because the Kubernetes injected environment variables for services are not the format that Jaegar client expects, leading to crashloops.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @zanhsieh

/ok-to-test

/lgtm


Implement a get question by ID for when a question is updated, we don't retrieve all questions at once

Might save some bandwith and what not and in case we do need a question by ID thing in the future.. Also update the mutate call

Closing, re-implementing questions / question sets in #401


BUG: Missing matmat coverage in scipy.sparse.linalg.aslinearoperator

Describe your issue.
When calling scipy.sparse.linalg.aslinearoperator with an matrix A that implements all of the methods matvec, matmat, rmatvec and rmatmat the resulting scipy.sparse.linalg.LinearOperator uses the provided methods for matvec, rmatvec and rmatmat but not for matmat where the default implementation is used instead of the user-provided implementation. This is annoying since the default implementation is inefficient and falls back to several matvec calls.
This should be changed such that the user provided implementation is used.
Reproducing Code Example
from scipy.sparse.linalg import aslinearoperator
class Matrix:
    shape = (1,1)
    dtype = float

    def matvec(self, x):
        print("matvec")

    def matmat(self, x):
        print("matmat")

    def rmatvec(self, x):
        print("rmatvec")

    def rmatmat(self, x):
        print("rmatmat")

A = aslinearoperator(Matrix())
A._matvec(1)
A._rmatvec(1)
A._rmatmat(1)
A._matmat(1)

Error message
matvec
rmatvec
rmatmat
Traceback (most recent call last):
  File *** in <module>
    A._matmat(1)
  File "/venv/lib/python3.9/site-packages/scipy/sparse/linalg/_interface.py", line 527, in _matmat
    return super()._matmat(X)
  File "/venv/lib/python3.9/site-packages/scipy/sparse/linalg/_interface.py", line 187, in _matmat
    return np.hstack([self.matvec(col.reshape(-1,1)) for col in X.T])
AttributeError: 'int' object has no attribute 'T'

SciPy/NumPy/Python version information
1.8.0

Is not it needed to return something meaningful in the definition of matmat so that it doesn't fall back to matvec?

If you call aslinearoperator with a object that is not a (sparse) matrix or a LinearOperator the following code is executed:
if hasattr(A, 'shape') and hasattr(A, 'matvec'):
    rmatvec = None
    rmatmat = None
    dtype = None

    if hasattr(A, 'rmatvec'):
        rmatvec = A.rmatvec
    if hasattr(A, 'rmatmat'):
        rmatmat = A.rmatmat
    if hasattr(A, 'dtype'):
        dtype = A.dtype
    return LinearOperator(A.shape, A.matvec, rmatvec=rmatvec,
                          rmatmat=rmatmat, dtype=dtype)

Obviously the matmul function is completely ignored. There are two major problems with that:

Your object must implement the matvec function. A object that only implements matmul is not accepted even though matvec could be implemented by simply falling back to matmul as the default implementation of LinearOperator already does anyways.
Even if your object supplies a efficient matmul implementation it will always fall back to a probably inefficient default implementation based on matvec.

I do not see this example as a proof of a bug. It's documented at https://docs.scipy.org/doc/scipy/reference/generated/scipy.sparse.linalg.aslinearoperator.html :
"An object with .shape and .matvec attributes"
It is reasonable to request that aslinearoperator takes objects with other attributes in which case the request is for an enhancement, rather than a bug report., - just like your already opened https://github.com/scipy/scipy/issues/16114


[fix] fix DynamicPillarFeaureNet bug

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Does this change affect the normal run or performance of current models?

Since DynamicPillarFeaureNet is inherited from PillarFeatureNet, original implementation will cause an error channel mismatch. This change will affect the performance of [dv_pointpillars_secfpn_6x8_160e_kitti-3d](https://github.com/open-mmlab/mmdetection3d/blob/master/configs/dynamic_voxelization/dv_pointpillars_secfpn_6x8_160e_kitti-3d-car.py) slightly according to experiments.

Does this change affect the normal run or performance of current models?

Since DynamicPillarFeaureNet is inherited from PillarFeatureNet, original implementation will cause an error channel mismatch. This change will affect the performance of dv_pointpillars_secfpn_6x8_160e_kitti-3d slightly according to experiments.

OK we will try your fixed version and update the model links accordingly.


Could someone please post a working code for using servcie account for sending mail?

I have used the below code, which i got, but gives the error as "Error:"invalid_client", Description:"The OAuth client was not found.", Uri:"""
public GmailService CreateGmailService1()
{
String serviceAccountEmail = "Test";
var certificate = new<EMAIL_ADDRESS>"notasecret", X509KeyStorageFlags.Exportable);
string userEmail = "testaccount";
        ServiceAccountCredential credential = new ServiceAccountCredential(
            new ServiceAccountCredential.Initializer(serviceAccountEmail)
            {
                User = userEmail,
                Scopes = new[] { "https://www.googleapis.com/auth/gmail.send" }

            }.FromCertificate(certificate)
        );

        var service = new GmailService(new BaseClientService.Initializer()
        {
            HttpClientInitializer = credential,
            ApplicationName = ApplicationName,
        });

ServiceAccount.cs
Note this will only work with a Gsuite account which has Domain-Wide Delegation setup for the service account.
You can not use a serivce account with normal user gmail accounts.

Thanks for that, so while using the send, should we be providing the service account email id?
service.Users.Messages.Send(message,sFrom).Execute();

You will want to use the email address of the user who you are impersonating.  A service account cant send emails on its own its not a true email address.    (I dont think it may be fun to try though)

"Error:"invalid_client", Description:"The OAuth client was not found.", Uri:"""
It errors out as the above messgae?

Check your service account settings in Google Developer console. Invalid client means its not set up correctly make sure you are creating service account credentials and downloading the file for it

I'm closing this issue as we're archiving the repository.


Assets & portfolio tables: improvements

What is the purpose of the change:

Refines style of asset and portfolio tables, including some updates to global styles.
Linear Task
Linear Task URL
Brief Changelog

Global: Update table header style, update table content text style
Fix width of sparkline chart if data is missing
Fix font size of price change
Refine asset name cell, note differences on portfolio and assets pages

Summary by CodeRabbit

New Features

Improved visual structure and alignment in several components, including PriceChange and AssetCell.
Enhanced search functionality in the AssetsInfoTable with expanded input field.

Bug Fixes

Added fallback display for empty price data in the HistoricalPriceCell.

Style

Updated various visual styles such as border properties, padding, and font weights across the platform. Adjusted default text classes in the PriceChange component for better readability.

Refactor

Replaced <span> with <div> in PriceChange for improved HTML structure.
Enhanced data passing in AssetBalancesTable for clearer asset representation.

Let's change the up down price icons to the one's nass said to change to.  The arrows


Compatibility for sidekiq 4.1.1

We added a wrapper class to use sidetiq with the new version of Sidekiq::Manager in sidekiq 4.1.1

Changes Unknown when pulling d8afb4c81a7239a2b77ca6428a33f71d71c688dd on platogo:master into ** on tobiassvn:master**.

:+1: tested with the sidekiq 4.1.2, sidekiq-pro 3.2.1 & it works for me

excited for this, my logs are being spammed similar to https://github.com/tobiassvn/sidetiq/issues/51#issuecomment-161876852

any chance for this to get merged @tobiassvn?

Hey @g0ody I'm seeing the following with your fork:
NoMethodError: undefined method present?' for #Sidekiq::Launcher:0x005632af957cd8`

This should be fixed with aa1d919637c8b174e3a3b291d6ad6777122910fe.


Option for verbose in OSQP

Currently verbose is always on. I believe that we should have an option parameter on the class PolynomialSolver such as bool verbose so we can set it to false.

So I understand, you want to turn off the OSQP solver output to terminal?

Yes, I will work on the pull request for this, looks like a simple update.

You want to change verbosity now, but what about future changes? I don't think it's scalable to append new flags to the Options or PathInfo structs every time we want new data.
I think we may want to put an instance of OSQPSettings in the PolynomialSolver::Options structure that then gets passed to the osqp solver at run time. That way, any end user can configure any osqp options in their own executable.
Similarly, we may want to put an instance of OSQPSolution, OSQPInfo, etc in the PathInfo structure. That way the end user can have whatever data they want from OSQP without further changes to P4.

I agree... This is the best solution!

Would you like to do it, or me?

I can do it!

Added osqp_settings to the PolynomialSolver::Options structure. Users now have complete and direct access to all osqp settings.


Turn off tst_mem1.c test as part of benchmarks

Fixes #2057
Turn off tst_mem1 as a benchmark test. It does not work reliably on very busy systems.

Agreed


firefox: 80.0 -> 80.0.1

Motivation for this change
https://www.mozilla.org/en-US/firefox/80.0.1/releasenotes/
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Waitin for local build + test

Result of nixpkgs-review pr 97106 1

3 packages built:
- adoptopenjdk-icedtea-web
- xulrunner (firefox-unwrapped)
- slimerjs

LGTM


[docs]: Update foreground service example

How to call the onTaskUpdate somewhere? Also this example doesn't even recognized Cannot find name 'onTaskUpdate' very confusing
notifee.registerForegroundService((notification) => {
  return new Promise(() => {
    // Example task subscriber
    onTaskUpdate(task => {
      if (task.update) {
        notifee.displayNotification({
          id: notification.id,
          body: notification.body,
          android: {
            ...notification.android,
            progress: {
              max: task.update.total,
              current: task.update.current,
            },
          },
        });
      }

      if (task.complete) {
        await notifee.stopForegroundService()
      }
    });
  });

});

onTaskUpdate is an example function. But agreed, may be confusing. Happy to take suggestions or merge a p/r if you can improve it.
Basically say you are downloading an image... onTaskUpdate would be for example :
myImageDownloader.listen(progress => {
notifee.displayNotification({
          id: notification.id,
          body: notification.body,
          android: {
            ...notification.android,
            progress: {
              max: task.update.total,
              current: task.update.current,
            },
          },
        });
})

onTaskUpdate is an example function. But agreed, may be confusing. Happy to take suggestions or merge a p/r if you can improve it.
Basically say you are downloading an image... onTaskUpdate would be for example :
myImageDownloader.listen(progress => {
    notifee.displayNotification({
          id: notification.id,
          body: notification.body,
          android: {
            ...notification.android,
            progress: {
              max: task.update.total,
              current: task.update.current,
            },
          },
        });

    if (progress === '100%') {
     await notifee.stopForegroundService()
    }
})

Hey @helenaford , please I would like to where you declared the  'myImageDownloader' that has the listen method...
Would really appreciate if there's a full example.

@rosnaib11 please do you understand this now? I am still having issues with this

@iamifechi can you give more details on what you are trying to achieve with the foreground service? What's your long processing task that you are trying to perform?

Ok thank you @helenaford,
I am trying to build a countdown time that updates the time left. I want this displayed on the notification body at the same time as the showChronometer ... updating the notification body has been a major problem. I also want this to run in background even when the app is closed or killed.
So far, I created a TimerNotification function I hope to call in my context file when the timer starts, and as the timer in my state counts down, I planned to update the notification via useEffect, this is where I get stuck.
I made a  gist of what my index.js and TimerNotifcation.ts file looks like : Countdown timer gist

@wardbeyens what is your use case? is it to download an image (for example)?

Why is the onTaskUpdate in the promise and not somewhere in the specific code with the new value that should be placed in the progress bar in the notification?

OK, after reading the documents several times, I think I get it.
Above we've updated the callback handler for the onTaskUpdate method.

OK, after reading the documents several times, I think I get it.
Above we've updated the callback handler for the onTaskUpdate method.

@wardbeyens could you please explain it better  if you've understood it? The stale bot keeps closing the issue but I believe a lot of people still need to understand this

Someone in the notifee community  has an example here:
Every 5 minutes, checking the location of the user. Does this help at all?
if you tell me you're exact use case and some sample code of how you're downloading your image... I will try my best and help you

See above, issue has been resolved.

Wish the document is updated to explain it better, i was confused until i found this thread.

Wish the document is updated to explain it better, i was confused until i found this thread.

What are you trying to implement?


Add support for Leap Motion Core Assets version 4.5.0

Overview
UltraLeap released a new version of the Leap Motion Unity Core Assets recently.
The current MRTK Leap Motion Data Provider only supported the Leap Core Assets version 4.4.0, this PR adds support for 4.5.0 as well.
The largest difference between 4.4.0 and 4.5.0 is the location of the tests directory.  If the 4.5.0 core assets are imported into MRTK, a new test assembly definition is added.
Both 4.4.0 and 4.5.0 will be supported.
Changes

Fixes: #7764

Verification

Download the Leap Motion Unity Core Assets 4.5.0
Import the package into either Unity 2019.3 or 2018.4

/azp run

/azp run


Wrap sudo access prompts in a mutex

Fixes #3910.

Thanks!

@homu r+

:pushpin: Commit fe06b05 has been approved by segiddins

:hourglass: Testing commit fe06b05 with merge ef26de9...

@segiddins I've updated the patch to work on Ruby 1.8 also, so I guess you could cancel https://travis-ci.org/bundler/bundler/builds/76799207.
Sorry for the inconvenience! :)

@homu r+

:pushpin: Commit 7bfd52d has been approved by segiddins

:hourglass: Testing commit 7bfd52d with merge 1f04cbe...

:sunny: Test successful - status


Seeking Detailed Information on the Model Architecture and Weight File Used in Basic AI for LiDAR-Camera Fusion Annotation

Hello, Basic AI Team
I started annotating on the datasets of LiDAR Fusion Trial. I used Basic LiDAR Object Detection model. The results are so good.
I'm wondering that how worked it. So I red your documents and I understood that Lidar-camera Fusion (Frame series) Annotation uses OpenPCDet. That annotates 3D point Cloud. However, I couldn't know which model (architecture) and weight file are used on Basic AI. Could I know the information.
Regards,
Silva

Hello! The detailed configuration of the model can be found in this file: https://github.com/xtreme1-io/point-cloud-object-detection/blob/main/pcdet_open/cfgs/nuscenes_models/cbgs_voxel0075_res3d_centerpoint.yaml

Hello! The detailed configuration of the model can be found in this file: https://github.com/xtreme1-io/point-cloud-object-detection/blob/main/pcdet_open/cfgs/nuscenes_models/cbgs_voxel0075_res3d_centerpoint.yaml

@ShirleySprite
Thank you for your answer I confirmed the source code and understand the algorithm.


Update OpenStack CCM and CSI to v1.25.3, v1.24.5, v1.22.2

What this PR does / why we need it:

Update OpenStack CCM and CSI to v1.25.3, v1.24.5, v1.22.2

Which issue(s) this PR fixes:
xref #2406
What type of PR is this?
/kind feature
Does this PR introduce a user-facing change? Then add your Release Note here:
- Update OpenStack CCM and CSI to v1.25.3, v1.24.5, v1.22.2

Documentation:
NONE

/test pull-kubeone-e2e-openstack-default-install-containerd-external-v1.22.15
/test pull-kubeone-e2e-openstack-default-install-containerd-external-v1.24.7

/test pull-kubeone-e2e-openstack-default-install-containerd-external-v1.22.15
/test pull-kubeone-e2e-openstack-default-install-containerd-external-v1.24.7

/test pull-kubeone-e2e-openstack-default-install-containerd-external-v1.23.13
/test pull-kubeone-e2e-openstack-default-install-containerd-external-v1.25.3

/test pull-kubeone-e2e-openstack-default-stable-upgrade-containerd-external-from-v1.24.7-to-v1.25.3

/retest

/lgtm
/approve


why is my tsconfig path cleared after startup, and how do I configure a path nickname?

why is my tsconfig path cleared after startup, and how do I configure a path nickname?
Before：
tsconfig.js
{
  "compilerOptions": {
    "target": "es5",
    "lib": [
      "dom",
      "dom.iterable",
      "esnext"
    ],
    "experimentalDecorators": true,
    "allowJs": true,
    "skipLibCheck": true,
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "strict": true,
    "forceConsistentCasingInFileNames": true,
    "noFallthroughCasesInSwitch": true,
    "module": "esnext",
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "isolatedModules": true,
    "noEmit": true,
    "jsx": "react-jsx",
    "baseUrl": "./",
    "paths": {
      "@asset": ["/src/assets"],
  }
  },
}

config-overrides.js
const { paths } = require('react-app-rewired');
const configPath = require(paths.scriptVersion + '/config/webpack.config')

const multipleEntry = require('react-app-rewire-multiple-entry')([
  // module1
  {
    entry: 'src/page/Order/index.tsx',
    template: 'public/order.html',
    outPath: '/order.html'
  },
  // module2
  {
    entry: 'src/page/Singup/index.tsx',
    template: 'public/singup.html',
    outPath: '/singup.html'
  }
  // module other.....
]);

module.exports = {
  webpack: function(config, env) {
    multipleEntry.addMultiEntry(config);
    // config = {...config, ...configPath}
    // config.resolve.alias = {
    //     "@": path.resolve(__dirname, "src")
    // };
    return config;
  }
};

** after**
tsconfig
{
  "compilerOptions": {
    "target": "es5",
    "lib": [
      "dom",
      "dom.iterable",
      "esnext"
    ],
    "experimentalDecorators": true,
    "allowJs": true,
    "skipLibCheck": true,
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "strict": true,
    "forceConsistentCasingInFileNames": true,
    "noFallthroughCasesInSwitch": true,
    "module": "esnext",
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "isolatedModules": true,
    "noEmit": true,
    "jsx": "react-jsx",
    "baseUrl": "./"
  },
  "include": [
    "src"
  ]
}

It's something that create-react-app's react-scripts package does - they explicitly delete anything that was in the paths variable when validating the tsconfig file:
https://github.com/facebook/create-react-app/blob/main/packages/react-scripts/scripts/utils/verifyTypeScriptSetup.js#L159
There have been a few ways to work around it - one being to try making your tsconfig file extend from another file and set it in the other file, the second being to use something like patch-package to patch the line in react-scripts so it doesn't delete the paths entry.
There's some discussion on this in an older issue: https://github.com/timarney/react-app-rewired/issues/375#issuecomment-474806096

Closing --- this given it's not a bug and no activity.


The Bash@3 task ignores the target: container step setting.

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: Bash@3,
Environment

Server: TFS on-premises

If using TFS on-premises, provide the version:  Version Dev18.M170.6

Agent - Hosted or Private:

If using private agent, provide the OS of the machine running the agent and the agent version:
Linux Agent, running in Container uname -a: Linux lnx-adsagent-1 3.10.0-1160.59.1.el7.x86_64 #1 SMP Wed Feb 23 16:47:03 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Agent Version: 2.170.1

Issue Description
This is an example of the pipeline, showing the issue:
resources:
  containers:
  - container: sqlserver
    image: mcr.microsoft.com/mssql/server:2019-latest
    env:
      SA_PASSWORD: supersecure1!
      ACCEPT_EULA: Y
    ports:
    - 1433/tcp
jobs:
- job: Tests
  services:
    sqlserver: sqlserver
  steps:
  - task: Bash@3
    target: sqlserver    ## here i specify the container target
    inputs:
      targetType: inline
      script: |

        for i in {0..10..1}; do
            /opt/mssql-tools/bin/sqlcmd -U sa -Psupersecure1! -Q"SELECT @@SERVERNAME"
            if [ $? -eq 0 ]; then
                exit 0
            fi
            sleep 1
        done

        echo Giving up after 10 retries, but it can work ...

The Bash@3 task doesn't execute in the target: sqlserver container, but in the host. It is not ignored completely. If I pass in an invalid name like sqlserver_error, an error is returned (A container resource with name sqlserver_error could not be found. The container resource does not exist...)
Checking the logs, there is no indication that the task actually tries to execute anything in the container. I expected the script to be execute as docker exec ID ....
As you see within the logs, the error /opt/mssql-tools/bin/sqlcmd: No such file or directory occurs. There is no sqlcmd on the agent.
Task logs
##[debug]Evaluating condition for step: 'Wait from sqlserver to accept connections'
##[debug]Evaluating: SucceededNode()
##[debug]Evaluating SucceededNode:
##[debug]=> True
##[debug]Result: True
Starting: Wait from sqlserver to accept connections
==============================================================================
Task         : Bash
Description  : Run a Bash script on macOS, Linux, or Windows
Version      : 3.163.3
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/bash
==============================================================================
##[debug]agent.TempDirectory=/azp/agent/_work/_temp
##[debug]loading inputs and endpoints
##[debug]loading INPUT_TARGETTYPE
##[debug]loading INPUT_FILEPATH
##[debug]loading INPUT_SCRIPT
##[debug]loading INPUT_WORKINGDIRECTORY
##[debug]loading INPUT_FAILONSTDERR
##[debug]loading INPUT_NOPROFILE
##[debug]loading INPUT_NORC
##[debug]loading ENDPOINT_AUTH_SYSTEMVSSCONNECTION
##[debug]loading ENDPOINT_AUTH_SCHEME_SYSTEMVSSCONNECTION
##[debug]loading ENDPOINT_AUTH_PARAMETER_SYSTEMVSSCONNECTION_ACCESSTOKEN
##[debug]loading SECRET_SYSTEM_ACCESSTOKEN
##[debug]loaded 11
##[debug]Agent.ProxyUrl=undefined
##[debug]Agent.CAInfo=undefined
##[debug]Agent.ClientCert=undefined
##[debug]Agent.SkipCertValidation=undefined
##[debug]noProfile=true
##[debug]noRc=true
##[debug]check path : /azp/agent/_work/_tasks/Bash_6c731c3c-3c68-459a-a5c9-bde6e6595b5b/3.163.3/task.json
##[debug]adding resource file: /azp/agent/_work/_tasks/Bash_6c731c3c-3c68-459a-a5c9-bde6e6595b5b/3.163.3/task.json
##[debug]system.culture=en-US
##[debug]failOnStderr=false
##[debug]workingDirectory=/azp/agent/_work/2/s
##[debug]check path : /azp/agent/_work/2/s
##[debug]targetType=inline
##[debug]script=for i in {0..10..1}; do
    /opt/mssql-tools/bin/sqlcmd -U sa -Psupersecure1! -Q"SELECT @@SERVERNAME"
    if [ $? -eq 0 ]; then
        exit 0
    fi
    sleep 1
done

echo Giving up after 10 retries, but it can work ...
Generating script.
##[debug]which 'bash'
##[debug]found: '/bin/bash'
##[debug]Agent.Version=2.170.1
##[debug]agent.tempDirectory=/azp/agent/_work/_temp
##[debug]check path : /azp/agent/_work/_temp
========================== Starting Command Output ===========================
##[debug]which '/bin/bash'
##[debug]found: '/bin/bash'
##[debug]/bin/bash arg: --noprofile
##[debug]/bin/bash arg: --norc
##[debug]/bin/bash arg: /azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh
##[debug]exec tool: /bin/bash
##[debug]arguments:
##[debug]   --noprofile
##[debug]   --norc
##[debug]   /azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh
/bin/bash --noprofile --norc /azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
/azp/agent/_work/_temp/0ff0d0a0-acc1-44e1-b399-4fb361dff64b.sh: line 2: /opt/mssql-tools/bin/sqlcmd: No such file or directory
Giving up after 10 retries, but it can work ...

##[debug]Exit code 0 received from tool '/bin/bash'
##[debug]STDIO streams have closed for tool '/bin/bash'
##[debug]task result: Succeeded
##[debug]Processed: ##vso[task.complete result=Succeeded;done=true;]
Finishing: Wait from sqlserver to accept connections

Can you tell what's wrong here?
Or am I doing something wrong?
Thank you!

Hi @gprossliner thanks for reporting! We are working on more prioritized issues at the moment, but will get back to this one soon.

Hi @Kozlov-Igor. thank you for the answer.

In the meantime I checked the code of the task. There is nothing related to container here. Maybe this should be handled by the agent itself?
I was checking the Agents source, and noticed this PR: https://github.com/microsoft/azure-pipelines-agent/pull/3035
By using the agent.containermapping var, i can run the exec itself, so I have a workaround
This was not release at 2.170.1, so I updated to the current version 2.202.1

This is the workaround:
  - script: |
        CID=$(echo "$AGENT_CONTAINERMAPPING" | jq -r ".sqlserver.id")

        for i in {0..10..1}; do
            docker exec $CID /opt/mssql-tools/bin/sqlcmd -U sa -Psupersecure1! -Q"SELECT @@SERVERNAME"
            if [ $? -eq 0 ]; then
                exit 0
            fi
            sleep 1
        done

But it would be nice if this could be fixed. Because it's documented to work this way.
Please also let me know if you need some help (tests, PR, ...)

Hi @gprossliner!
Sorry for the delayed answer.
This is not a problem with a task Bash itself, but a combination of issues with insufficient user permissions on the MsSQL container and usage of services alongside containers.
Using services in your template will initialize required containers in non-interactive mode, and further steps will be handled by the host machine instead of the target container. If you want to pass the task directly to the target container you should remove the additional services definition from your template.
Please, notice that in this case there will be an issue with the MsSQL container initialization job, due to the specific of the azure agent - to execute tasks directly on the container, the default container user should have root permissions, however on the MsSQL image default user is mssql. You can use options: --user 0:0 in the container definition to avoid this.
Example of the working template:
resources:
  containers:
  - container: sqlserver
    image: mcr.microsoft.com/mssql/server:2019-latest
    options: --user 0:0
    env:
      SA_PASSWORD: supersecure1!
      ACCEPT_EULA: Y
    ports:
    - 1433/tcp

jobs:
- job: Tests
  steps:
  - task: Bash@3
    target: sqlserver
    inputs:
      targetType: inline
      script: |
        for i in {0..10..1}; do
            /opt/mssql-tools/bin/sqlcmd -U sa -Psupersecure1! -Q"SELECT @@SERVERNAME"
            if [ $? -eq 0 ]; then
                exit 0
            fi
            sleep 1
        done

        echo Giving up after 10 retries, but it can work ...


Fix RegEx for export statements on denoify.ts

The RegExp im using assumes export * from ... so it does not work on export {...} from ...
The RegExp is /export \* from .+?;/gms so I need to change that.
So those exports were not fixed previously. See also #4 for other things I missed.
For now, a solution is to fix manually all ocurring conflicts :(

Fixed with commit 70c502dce83524de201e5d3fc54e54947e7d6189


Make the website responsive

Hello @thearcdatascience I reviewed your website and I am interested to contribute for the same and wanted to work on this issue.
Please assign this issue to me.


feat(words): add MISRA

This word appears in https://github.com/tier4/autoware.universe/blob/tier4/main/common/autoware_auto_common/include/common/types.hpp#L35

https://ja.wikipedia.org/wiki/MISRA_C


Erick/feat/ilithya_requests

This commit adds a new column "subtitle" to the "public.modules" table in the database. The "subtitle" column is used to store the subtitle for countable modules. The commit also updates the subtitles for specific countable modules using their IDs.
Co-authored-by: [Author Name]
Summary by Sourcery
Introduce a 'subtitle' column in the 'public.modules' table to store subtitles for countable modules and update the UI to display these subtitles. Add an 'is_countable' column to manage module countability, and update specific modules accordingly.
New Features:

Add a new 'subtitle' column to the 'public.modules' table in the database to store subtitles for countable modules.

Enhancements:

Update the UI to display the new 'subtitle' field for course modules in the course curriculum section.

Chores:

Add a new 'is_countable' column to the 'public.modules' table to indicate whether a module is countable, with default value set to true.

Falta actualizar el nombre de los capitulos
No hacer merge


Fixing Duplicate Key Error in AngularJS ngRepeat Directive

Error message
js: Error: [ngRepeat:dupes] Duplicates in a repeater are not allowed. Use 'track by' expression to specify unique keys. Repeater: addon in getAddons() track by addon.transportUrl, Duplicate key: https://a0da031547f5-hanime-stremio-main.baby-beamup.club/manifest.json, Duplicate value: {"manifest":{"id":"eurc24fi1a8wmoefxzmy3qhyo","version":"1.0.0","behaviorHints":{"adult":true},"catalogs":[{"type":"Adult","name":"Hanime: Newset","id":"newset","extra":[{"name":"search","isRequired":false},{"name":"skip","isRequired":false},{"name":"genre","options":
ChatGPT description
A JavaScript error occurred due to duplicates in a repeater. The error message suggests using 'track by' expression to specify unique keys. The repeater consists of various 'addons' from which data is fetched, and the error message indicates a specific 'addon' with the transport URL https://a0da031547f5-hanime-stremio-main.baby-beamup.club/manifest.json that has a duplicate key and value. The duplicate value is a JSON object with metadata about the 'addon' such as its version, catalogs, resources, etc. The 'addon' is related to adult content from Hanime.TV and offers various filters and options for searching and browsing. The transport URL for the 'addon' is also provided. The error message suggests resolving the issue by specifying a unique key for the repeater using a 'track by' expression.
This error is being caused by having duplicates in a repeater in AngularJS. To fix it, you need to add a "track by" expression to your ng-repeat statement. The "track by" expression specifies a unique identifier for each item in the repeater so that duplicates can be avoided.
For example, you can modify your ng-repeat statement to track by the "transportUrl" property of each item in the repeater like this:
<div ng-repeat="addon in getAddons() track by addon.transportUrl">

This should resolve the issue and allow the ng-repeat to function without errors

Where is the fix


fix: Ensure whitespaces is allowed between name and bracket

Fixes: https://github.com/argoproj/argo-workflows/issues/5169

If it is a regression, please merge. If not, then I think we should not do this.

We explicitly say this is allowed: https://github.com/argoproj/argo-workflows/blob/master/docs/variables.md

If it is a regression, please merge. If not, then I think we should not do this.

We explicitly say this is allowed: https://github.com/argoproj/argo-workflows/blob/master/docs/variables.md

We could correct the docs instead. But, this is not the only place we do this. Can you check other fasttemplate usage please?


[website] Add books to Resources page

Motivation
Many books on Apache Pulsar have been published, so I would love to add the book resource to the website to help community members to learn Apache Pulsar.
Modifications
Add books resources to the webpage.
Documentation

[x] no-need-doc

@tuhaihe thanks for your contribution!

Have you run a local preview before submitting changes? If so, can you please provide screenshots of your changes in the PR description? So that others know your changes clearly.
Steps of running a local preview: https://github.com/apache/pulsar/tree/master/site2#website

You delete the backticks (``) so that Bot can not recognize the info and then labels the PR with doc-info-missing.
[x] doc-required  ✅
[x] doc-required ❌
Instrctions of doc labels: https://docs.google.com/document/d/1Qw7LHQdXWBW9t2-r-A7QdFDBwmZh6ytB4guwMoXHqc0/edit

Hi @Anonymitaet @urfreespace Thanks for your guys's help. I update this PR and run it successfully in local. This is the screenshot. PTAL.


npm run build fails on frontend setup

I'm following the instructions in the Readme for the "Running From Source" -> "One Time Setup".
My environment is Node.js 18.19.0 and npm 9.2.0, both of which are current and stable as at today. I'm running Python 3.10.14. My uname -a is:
Linux f1cd504fc97e 6.8.0-39-generic #39-Ubuntu SMP PREEMPT_DYNAMIC Fri Jul  5 21:49:14 UTC 2024 x86_64 GNU/Linux
Here are the outputs of npm install and npm run build respectively:
npm_install.log
npm_run_build.log
Full context: I am trying to set up a Docker file to pull, build, and run Stellaru, which I plan to make available either as a PR here or as a separate repo. Here are the steps leading up to the error:
FROM python:3.10-slim

RUN apt-get update && apt-get install -y \
    build-essential git npm \
    && apt-get clean \
    && rm -rf /var/lib/apt/lists/*

RUN git clone https://github.com/benreid24/Stellaru.git /app

WORKDIR /app/frontend
RUN npm install
RUN npm run build

I am taking a look. A lot of the dependencies (both backend and frontend) were created a couple of years ago and haven't been updated since. I will try to upgrade everything and make sure the project still builds and works as expected. A Docker file would be appreciated!

Landed a bunch of changes which should unblock you. Everything works for me locally on Windows using the same node version. I also beefed up the PR checks to test all three environments so hopefully it should build for you now.

Hi. Thank you, I confirm the latest version proceeds beyond this step.
I'm now encountering an issue with the xdg package where XDG_DATA_HOME is not found in linux_finder.py.
When I have time, I will investigate further whether this is to do with my individual installation, or if this is a deprecated component of xdg. If so, I can raise another issue.

Regarding the above, have you seen: https://github.com/srstevenson/xdg-base-dirs/commit/3019d7dfe750965d332dd32f2aff6c59bf02b1d0
It looks like there was a collision between two different packages called xdg, so about 18 months ago one was renamed to xdg-base-dirs.


⚠ LPDM Handbook (hosted on Divio) has degraded performance

In aec3e4f, LPDM Handbook (hosted on Divio) (https://repohubdev.tk) experienced degraded performance:

HTTP code: 500
Response time: 5226 ms

Resolved: LPDM Handbook (hosted on Divio) performance has improved in 7298ff8.


Plugin label metadata in the attributes for some spans is not correct

Version report
LTS 2.289.1
Latest plugin version
Reproduction steps
Metadata for the plugin step is not correct for the googleStorageUpload step

While debugging the descriptor, it seems the descriptorId is coming from CoreSteps

While for some other plugin steps the information is populated correctly, see

Maybe the implementation in the google storage plugin is the reason and this is just a corner case

As far as I see this is caused when the step implements https://javadoc.jenkins.io/jenkins/tasks/SimpleBuildStep.html (most likely an old traditional support for the existing 1.x non-pipeline versions)


facing issue while updating json file in karate framework

Hi I am want to update a value in json file.i am using karate framework.
my json file is as below,
{ "unit": "percent",
"toleranceSeconds": 30,
"evaluationMode": "latest",
"alternateId": "alt_1312_instrument_air_flow_to_osbl_psb_0_25_mm",
"expression": "{1312.psb_0_25_mm} + 10",
"triggers": [
{ "assetId": "1312",
"channelKey": "anomaly-score" }
]
}
I want to update value of assetId and channelKey.please help me. I have tried
solution 1 - * set myReq.channelKey = "MyChannelKey" ,
Solution 2 - * def newChannel = myReq.update("$..channelKey", "NewChannel") but it does not worked.

@SayaliBankar1 please take time and read the documentation because you are missing some fundamentals. after that, please ask your question on stack overflow. but I'll provide one hint here below, the rest is up to you:
* def req = { a: 'foo' }
* req.b = 'bar'
* match req == { a: 'foo', b: 'bar' }


🚀Display (and enable filtering by) size / number of parameters

Is your feature request related to a problem? Please describe.
It is hard to tell how big a model is without going to the repo and seeing the size of the weights file.
Describe the solution you'd like
It might prove useful to be able to sort models by size or number of parameters, especially for people with more modest compute budgets. Perhaps even display that information alongside a model's popularity.
Describe alternatives you've considered
Alternatives would be prior knowledge of the distilled / small model landscape. This feature might also help people discover new models they hadn't heard of by enabling filtering queries like "most popular and smallest model trained using MLM on wikitext"
Additional context
The datasets tab has a similar feature with the Size category filter.

This is now done for all repos with safetensors files

(filtering is API-only for now)


AccessViolationException error

Hi, I get a System.AccessViolationException when I launch my test program, I'm using .NET Framework and C#, I downloaded ImGui.NET From NuGet. Here's the code I made:
using System;
using ImGuiNET;

namespace ImGuiNET_test
{
    class Program
    {
        static void Main(string[] args)
        {
            ImGui.ShowDemoWindow();
        }
    }
}

It's not that simple.. ImGui by itself does not have a rendering pipeline, so it doesn't know how to show the demo window on the screen. I recommend taking a look at the examples in this repo and on imgui itself

Yes, please take a look at the example projects here. ImGui by itself does not render anything, it just provides data for your own renderer to use.