Set the Content-length when HTTP body is IO::Memory

Can you please give some context for this change? Doesn't this already work? HTTP.serialize_headers_and_body should take care of setting an approriate header.
I doubt this implementation is good. If I assign an empty IO and later append to it, the content length is incorrect.

@carlhoerberg are you trying to prevent the chunked encoding when a IO::Memory has the payload?
As @straight-shoota mentioned if this PR would lead to invalid content-length. So far setting the body with String or Bytes/Slice has the semantics that the body is all set. While setting an IO leads to chunked encoding.
If we set the content-length during serialize_headers_and_body to allow a chance to fill the IO, then we will need to introduce some way to state that chunked encoded is preferred if possible. (See the current logic in serialize_headers_and_body)
If in your usecase the IO::Memory is already filled, req.body = io.to_slice can be used and will set the content-length without duplicating memory. The underlying buffer of the IO::Memory will be reused.
Unfortunately the IO::Memory has not notion of closed for writtes, if that would be so, then we could safely choose if it's possible to set the content-length from the current buffer content.

What if in serialize_headers_and_body, if we have an IO and that IO is IO::Memory we just take a slice of it and write that to the underlying IO? But note that we have to be careful to do this correctly: the IO::Memory might have a pos different than zero (maybe).

I'm thinking that a lot of operations could "peek" into the underlying IO. For example IO.copy could check if the src is an IO::Memory, then write all the slice directly in one go.

IO are streams. Probably any intent to use them as a whole value will lead to confusion. Since String and Bytes are values defined as a whole, and the IO created there are owned by the request the api make sense to me.
I don't think is right to read the whole buffer to IO::Memory. Its current position should be used (and advanced) if there are optimization for IO.copy.

Ok, when's HTTP.serialize_headers_and_body run, after the before_request callback?  because this code:
require "http/client"
body = IO::Memory.new("aaaa")
http = HTTP::Client.new "localhost", 80
http.before_request do |r|
  pp r
end
http.post "/", body: body

outputs:
#<HTTP::Request:0x7f7137b4bf00
 @body=
  #<IO::Memory:0x7f7137b48ea0
   @buffer=Pointer(UInt8)@0x6639bc,
   @bytesize=4,
   @capacity=4,
   @closed=false,
   @decoder=nil,
   @encoder=nil,
   @encoding=nil,
   @pos=0,
   @resizeable=false,
   @writeable=false>,
 @cookies=nil,
 @headers=
  HTTP::Headers{"Accept-Encoding" => "gzip, deflate",
   "Host" => "localhost",
   "User-Agent" => "Crystal"},
 @method="POST",
 @query_params=nil,
 @remote_address=nil,
 @resource="/",
 @uri=nil,
 @version="HTTP/1.1">

before_request runs before HTTP.serialize_headers_and_body because that's when the request is actually "executed".
Maybe you could describe what you are trying to do and why the current code doesn't work for you?
For reference: https://en.wikipedia.org/wiki/XY_problem

You're right, pos has to be take in consideration, and yes, the IO::Memory can change between it's assigned and it's being sent. But at the time of serialization chunked encoding shouldn't be needed for IO::Memory, the size should be known at constant at that stage.

If you use body.to_slice you will have the content-length set and the buffer is not duplicated on runtime.
require "http/client"
body = IO::Memory.new("aaaa")
http = HTTP::Client.new "localhost", 80
http.before_request do |r|
  pp r
end
http.post "/", body: body.to_slice # <- here

Was posting to webserver (a AWS API) that didn't support keepalive when transfer encoding chunked was used, so pulled this one fast. But yes, it got several flaws in its current form. Don't know if it's worth working on, even though it can be done.

But then that slice will be wrapped in another io::memory, right? not optimal but i guess it's cheap enough.

With the latest commit problem is that the content-length still isn't available at before_request, which some middlewares, that eg. calculate authentication depeding on request headers, require. So nevermind, closing this

But then that slice will be wrapped in another io::memory, right? not optimal but i guess it's cheap enough.

I think it's cheap enough. Another alternative would be to allow the body to be directly IO | String | Slice. That would reduce allocating the intermediate IO::Memory. Maybe it would be worth in super heavy load, I'm not sure.


Lambda code doesn't respond appropriately when a key is missing

I ran into an error when trying to use this; I forgot to add a Priority key in the CloudFormation template, and the custom resource got stuck in creation. CloudWatch logs showed the following:
'Priority': KeyError
Traceback (most recent call last):
File "/var/task/alb_rule.py", line 19, in lambda_handler
Priority = int(event['ResourceProperties']['Priority'])
KeyError: 'Priority'

Because line 19 isn't in the try block, there wasn't an exception handler to send a 'FAILED' response. In order to get CloudFormation to clean up the resource, I had to edit the Lambda function to send 'FAILED', which allowed CloudFormation to see that creation had failed. Is there an easy way to change the code to handle missing keys, so CloudFormation can detect errors?

Thanks for the feedback. I've moved everything into the try block. All exceptions, including missing or misspelt parameters, should now be handled.
Can you test that it resolved the issue you had.

Looks like that mostly did the trick; I created a test template with the priority missing, and CloudFormation detected the error quickly and attempted to cancel the resource creation. The only problem I still see is that the Delete request CloudFormation sends also fails due to the key error. This shouldn't be too much of a problem, as the resource didn't get created in the first place, but it's mildly annoying. Not sure what you can do about it, though.
Thanks for writing this library, by the way. It's very helpful.

Delete requests will not come from Cloudformation unless there has already been a successful create. If you have an alternative way of handling delete's feel free to raise a pull request.
Thanks again for your feedback.


resolve input directory path in read blocks

attempt to fix symlink issue by resolving input directory path based on discussion with Alessio

@jtyoung84


using SSR strategy - updating from 0.3.5 to 0.4.1 breaks SSR

We went from:
"@apollo/client": "3.8.0-beta.6",
"@apollo/experimental-nextjs-app-support": "^0.3.5",

to:
"@apollo/client": "3.8.0-rc.1",
"@apollo/experimental-nextjs-app-support": "^0.4.1",
I removed the makeSuspenseCache={makeSuspenseCache} as instructed in the Release notes, however the behavior has completely degraded. In our stripped down example, before the update, if you reload the browser, you could see the data was fetched on the server, and no subsequent client side requests were made immediately after. That is my understanding of this working as intended, and that was great.
Now, after upgrading the packages, it renders cached data from the server and then performs a client side graphql query immediately upon loading (causing next to throw rehydration mismatch errors). I hope it's something I'm missing but I have reverted back to the old libraries and everything works. Hoping you all can shed some light on this situation.
Will provide more details if necessary. It's a bog standard test setup with a list component and a simple add form/button for the create post mutation. Again the only issue is with SSR on page load/hydration and all I had to do to introduce this bad behavior is update those packages (reverting works fine).
Thanks in advance.

Hi there,
I'm sorry that this happens! The patch in 0.4.1 is designed to restart the query in the browser in the case that the streaming connection from the server closed before data could be fully transported over - otherwise, the browser would hang in "loading" forever.
There seems to be something triggering that too soon for you, though. I would be very happy if you could get us a reproduction for that.
Maybe you could even add a failing integration test? That means you already have something to start on when building a reproduction.
We have the integration tests here: https://github.com/apollographql/apollo-client-nextjs/blob/main/integration-test/src/app/cc/dynamic/dynamic.test.ts
This is a test that makes sure that any requests happening client-side would fail the test (by destructuring blockRequest there).

Unfortunately we had to use the old fashioned useQuery until we get the useSuspenseQuery issues sorted out, I wonder if it's possible that the reason it was happening is because the server side query doesn't pass the cookies so it saw a "mismatch" when the client took over (related to my new issue) but we'll keep on pushing forward!

Without a reproduction there is unfortunately not much we can do to help you sort this out. Could you please take the time to create a small reproduction?
(Gonna take a look at your new issue now)

I believe this was fixed in 0.4.3, so I'll close this issue.


cni result should be versioned according to the one asked for

Signed-off-by: Rajat Chopra<EMAIL_ADDRESS>Or, supposedly if cniserver responds accurately to a 0.2.0 request with a 0.2.0 response, the cnishim will still return a current version result.
@dcbw Can you urgently take a look at this? I somehow get an error on Windows with an older kubelet where the response is expected to be 0.2.0 and the one given by ovn fails to translate properly (no IP addresses).

Looks good to me

The cniserver was sending the corrected version and cnishim was translating it back to 'current' version. It should be other way round - cniserver should send the 'current' and cnishim should convert according to the one asked by kubelet.

Tested ok.


Feature:  make the quiz section attractive

Feature description
I would like start my work by change the design of quiz section to be more attractive ,Please assign me

👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

I am a participant under GSSOC. I would like to work on the issue. Can you please assign it to me @mariam199911

@mariam199911  can i assign this issue to @Khushi25pandey  or would you start working on it.

you can. I started working on footer

@mariam199911 i assign this issue to you.

to me? I'm currently working on footer

hello, i want to deal with this issue.
Can i have it?
thank you


SPM package identifier conflict

As xCode SPM using last path component of the package URL as identifier, so, in case of statsig SDK the identifier is ios-sdk, which may conflict with other packages, for example with UXCam.
Not sure when Apple will update the identifier generation - I see there were some old discussions, but till now it works the same.
I would propose to rename the project to statsig-ios-sdk, for example, to avoid conflicts with other packages (https://github.com/statsig-io/statsig-ios-sdk)

Thanks for the feedback. At this point we have no plans to change for now, if and when this becomes problematic we can address it


Added in admin handling for auto annotation

This has only been lightly tested.

I believe this would allow the merging of this code into the auto annotation, correct?

Could probably get permissions straight from the owner object rather than adding a separate field to the AnnotationModel

This is bugged slightly, since there's no way for the AnnotationModel to be created/updated with the run_as_admin boolean value as anything other than false.
I'll probably rework this so that it pulls that value from the User object, rather than creating a new field in the AnnotationModel

@benhoff , could you please add information about the change into CHANGELOG.md file? The patch is good. Just need to resolve some minor issues and clarify behaviours with update_model. I hope to merge the patch tomorrow if we resolve all mentioned issues. Thank you very much for the contribution and patience.

@azhavoro , please have a look.


feat: Prepare for 0.13.2 release

Describe your changes :

[x] Bumped up Chart Versions
[x] Updated the repository image names to docker.getcollate.io/openmetadata/*

Type of change :

[x] Improvement
[x] New feature

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have performed a self-review of my own.
[ ] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

What about the new values added to openmetadata.yaml file? Do we have to update them somewhere?

Hello @nahuelverdugo , can you help me with the new values here? I'll provision them along with this PR. Thanks!

Hi Akash after running this command:
git diff 0.13.1 0.13.2 -- conf/openmetadata.yaml

We have changed the following lines:
Added
searchIndexMappingLanguage: ${ELASTICSEARCH_INDEX_MAPPING_LANG:-EN}

Uncommented:
eventMonitoringConfiguration:
  eventMonitor: ${EVENT_MONITOR:-prometheus}  # Possible values are "prometheus", "cloudwatch"
  batchSize: ${EVENT_MONITOR_BATCH_SIZE:-10}

So I think we need values for EVENT_MONITOR, EVENT_MONITOR_BATCH_SIZE, and ELASTICSEARCH_INDEX_MAPPING_LANG.

Hi Akash after running this command:
git diff 0.13.1 0.13.2 -- conf/openmetadata.yaml

We have changed the following lines: Added
searchIndexMappingLanguage: ${ELASTICSEARCH_INDEX_MAPPING_LANG:-EN}

Uncommented:
eventMonitoringConfiguration:
  eventMonitor: ${EVENT_MONITOR:-prometheus}  # Possible values are "prometheus", "cloudwatch"
  batchSize: ${EVENT_MONITOR_BATCH_SIZE:-10}

So I think we need values for EVENT_MONITOR, EVENT_MONITOR_BATCH_SIZE, and ELASTICSEARCH_INDEX_MAPPING_LANG.

Thanks @nahuelverdugo . I've provisioned a way in k8s helm values for the above said environment variables!


Module Is Not Displayed

When I wanted to add module as Menu, in Menu Type in mod dropdown list the mods aren't displayed, Please correct it.

The module menu is still in development. Will be added with new features and backend features.
thank You

Thanks :+1:

not yet. i'm still find a simple solutions for that.

done https://github.com/semplon/GeniXCMS/commit/d885eb20006099262c0278932b9f8aca3c1ac97f


I don't know how to get the settings right

i get how it works but i dont know how to set it up for myself. basically i have no idea what settings are right for my colorblindness and if it helps to know, im a strong Deutan.

If you have Hexaplex and Mod Menu installed, you can access Hexaplex's settings screen:

If you have strong Deutan colorblindness, I'd recommend using the Deuteranomaly setting and playing with the strength and skew sliders until it looks nice to you.
Deutan was the profile I saw the least change in (in all fairness, I'm not colorblind). However, I'm going to be working on potentially updating the color profile matrices for when 1.17 releases. So try it out and let me know if it helps and/or what you'd like changed. (And thanks for using it! 😊)

Thanks! This Helped abunch!


Outdated Google Play icons

The Google Play symbol and logo use the pre-2016 design.
Updated SVGs:

symbol
complete logo

Updated in aaf25561c9a8f102bcad7b778b2a681345939ea9


fix: reconcile queues scrambling Action callbacks

Description
When running uds run slim-dev (and friends) against<EMAIL_ADDRESS>then UDS Package CRs will occasionally (approx. 50% of runs) not complete deployment.
Description / analysis / troubleshooting steps detailed in related issue.
This PR is intended to fix the erroneous behavior.
Related Issue
Fixes #1115
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[ ] Unit, Journey, E2E Tests, docs, adr added or updated as needed
[x] Contributor Guide Steps followed

This is passing CI, it fixes the bug. We are also going to introduce a new part of CI that tests against uds run command.


Fix anonymous user can access report sub-page

Details

Anonymous users can access the report sub-page by deep link or through URL. This PR fix it.
Fixed Issues

$ https://github.com/Expensify/App/issues/46951
PROPOSAL: https://github.com/Expensify/App/issues/46951#issuecomment-2273506703
Tests

Same as QA Steps

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

Offline tests

Same as QA Steps
QA Steps

Login as anonymous user by opening a public room (e.g., https://staging.new.expensify.com/r/7369791765254619)
Edit the URL by appending /settings to it (or deep-link to it in Android/iOS)
Verify sign in modal is shown
Press back
Verify you are navigated back to the home

Note: the report behind the sign-in modal is blank in web, this is a different unrelated issue

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

PR Author Checklist

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

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

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

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

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

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

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

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is 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/f793912b-3d5f-4616-8943-e1fb95ecbe2f

Android: mWeb Chrome

iOS: Native

https://github.com/user-attachments/assets/e66dfd85-11fa-4db5-8b56-9ae324a17cd8

iOS: mWeb Safari

https://github.com/user-attachments/assets/5b01db01-c759-42f5-b9bc-f3dbfd08f3a6

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/569d1471-7b9f-41e0-9e49-2dce8e4e8951

MacOS: Desktop

https://github.com/user-attachments/assets/6b1a6be8-2801-40d0-b399-78554e2d49db

I still can't access public room from my android mWeb

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

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/68c258c8-d56b-4ad1-ad60-39c6fab1beb5

MacOS: Desktop

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

🚀 Deployed to staging by https://github.com/cristipaval in version: 9.0.22-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/cristipaval in version: 9.0.22-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/chiragsalian in version: 9.0.22-9 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Add new channel upgrade proto messages

This PR only picks the new channel upgrade messages introduced in the ibc-go branch 04-channel-upgrades

Closing this as it is not required anymore. The channel upgrade proto types are in the newest release v0.42.0 https://github.com/cosmos/ibc-proto-rs/pull/193


[workaround] Retrolambda is a compilation dependency when using libraries

I am using the plugin version 3.2.3. I just spent the whole day trying to figure out why I was getting java 8 class files when compiling despite being sure I setup the retrolambda plugin for each of my modules. The situation I had was:
moduleA
\--moduleB
   \--moduleC

Artifacts for B and C were put into my maven repo. I was trying to run my tests for A, and it was failing at the dex step because it was getting class files with CA FE BA BE 00 00 00 34 in the header. It was complaining about this class in particular:
https://github.com/orfjackal/retrolambda/blob/master/retrolambda/src/main/java/net/orfjackal/retrolambda/files/ClasspathVisitor.java
That was strange to me because that shouldn't be a part of the output of my compilation. Running gradle dependencies in A revealed:
A
\--B
   |--C
   \--net.orfjackal.retrolambda

And there's my problem. Somehow retrolambda was becoming a transitive dependency to my compilation! Thanks to some gradle configuration I have a working workaround:
compile('foo:bar:1.2.3') {
    exclude group: 'net.orfjackal.retrolambda'
}

Oddly, compiling B and running gradle dependencies did not have any issues.

Interesting, that certainly should not be the case. The only way retrolambda is added as a dependency is through a custom configuration retrolambdaConfig. There might be a wrinkle in how gradle handles transitive dependencies that I'm not aware about. What version of gradle are you using so I can do further testing?

I am using gradle version 2.2.1. This is in an android project where all referenced modules are android library modules. I am using version 1.0.0 of the android gradle plugin.

If you have a change, mind updating to the latest version (2.7) and seeing if the issue is still present?

I just tried gradle 2.7, and the bug still presented itself. I don't have much more time to look into it today, I can get back to it on Monday.

I created a minimal setup to see if it would reproduce the bug, but it did not. That tells me that something in my project setup is triggering this bug. I will keep looking.

I think I figured out how to reproduce this. It requires a very strange setup:
proj1/a/build.gradle:
dependencies {
    compile('module:b:1.0.0')
    compile project(':c')
}

proj2/b/build.gradle:
dependencies {
    compile 'module:c:1.1.0'
}

proj1/c/build.gradle:
dependencies {
    // ...
}

The bug only happens if A's dependency on C is a project dependency. If I change the dependency to an artifact dependency, the bug does not happen. It only seems to happen if:

A depends on B as an artifact
B depends on C as an artifact
A depends on C as a project dependency

I can fix this without the exclude group workaround by removing A's dependency on C since A will get C anyway transitively through B, or change it to an artifact dependency.

Ok this is very weird and smells like a gradle bug. Mind creating a sample project that reproduces it? Also instead of depending on retrolambda, try just creating a custom configuration in each of the projects.

Since github doesn't support attaching .zip files (wat), here's a repo that demonstrates the bug:
https://github.com/Wopple/retrolambda-bug
Dependency graph:
moda > modb (artifact)
moda > modc (project)
modb > modc (artifact)
To reproduce:
~/dir/proj1/modc$ gradle assemble publishReleasePublicationToMavenLocal
~/dir/proj2/modb$ gradle assemble publishReleasePublicationToMavenLocal
~/dir/proj1/moda$ gradle dependencies
You might need to temporarily comment out moda in settings.gradle in proj1 for the first command to run.
Below is sample output from running gradle dependencies in each of the three modules to demonstrate the bug. You can see the issue from module moda. I have also included the .pom files generated from modc and modb.
~/retrolambdabug/proj1/modc% gradle dependencies
:modc:dependencies

------------------------------------------------------------
Project :modc
------------------------------------------------------------

archives - Configuration for archive artifacts.
No dependencies

compile - Compile classpath for source set 'main'.
No dependencies

default - Configuration for default artifacts.
No dependencies

retrolambdaConfig
\--- net.orfjackal.retrolambda:retrolambda:2.0.5

runtime - Runtime classpath for source set 'main'.
No dependencies

testCompile - Compile classpath for source set 'test'.
No dependencies

testRuntime - Runtime classpath for source set 'test'.
No dependencies

BUILD SUCCESSFUL

Total time: 3.452 secs

~/retrolambdabug/proj2/modb% gradle dependencies
:modb:dependencies

------------------------------------------------------------
Project :modb
------------------------------------------------------------

archives - Configuration for archive artifacts.
No dependencies

compile - Compile classpath for source set 'main'.
\--- org.daniel:modc:1.0

default - Configuration for default artifacts.
\--- org.daniel:modc:1.0

retrolambdaConfig
\--- net.orfjackal.retrolambda:retrolambda:2.0.5

runtime - Runtime classpath for source set 'main'.
\--- org.daniel:modc:1.0

testCompile - Compile classpath for source set 'test'.
\--- org.daniel:modc:1.0

testRuntime - Runtime classpath for source set 'test'.
\--- org.daniel:modc:1.0

BUILD SUCCESSFUL

Total time: 3.559 secs

~/retrolambdabug/proj1/moda% gradle dependencies
:moda:dependencies

------------------------------------------------------------
Project :moda
------------------------------------------------------------

archives - Configuration for archive artifacts.
No dependencies

compile - Compile classpath for source set 'main'.
+--- org.daniel:modb:1.0
|    \--- org.daniel:modc:1.0 -> project :modc
|         \--- net.orfjackal.retrolambda:retrolambda:2.0.5
\--- project :modc (*)

default - Configuration for default artifacts.
+--- org.daniel:modb:1.0
|    \--- org.daniel:modc:1.0 -> project :modc
|         \--- net.orfjackal.retrolambda:retrolambda:2.0.5
\--- project :modc (*)

retrolambdaConfig
\--- net.orfjackal.retrolambda:retrolambda:2.0.5

runtime - Runtime classpath for source set 'main'.
+--- org.daniel:modb:1.0
|    \--- org.daniel:modc:1.0 -> project :modc
|         \--- net.orfjackal.retrolambda:retrolambda:2.0.5
\--- project :modc (*)

testCompile - Compile classpath for source set 'test'.
+--- org.daniel:modb:1.0
|    \--- org.daniel:modc:1.0 -> project :modc
|         \--- net.orfjackal.retrolambda:retrolambda:2.0.5
\--- project :modc (*)

testRuntime - Runtime classpath for source set 'test'.
+--- org.daniel:modb:1.0
|    \--- org.daniel:modc:1.0 -> project :modc
|         \--- net.orfjackal.retrolambda:retrolambda:2.0.5
\--- project :modc (*)

(*) - dependencies omitted (listed previously)

BUILD SUCCESSFUL

Total time: 3.41 secs

modc.pom:
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <modelVersion>4.0.0</modelVersion>
  <groupId>org.daniel</groupId>
  <artifactId>modc</artifactId>
  <version>1.0</version>
  <dependencies/>
</project>

modb.pom

<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <modelVersion>4.0.0</modelVersion>
  <groupId>org.daniel</groupId>
  <artifactId>modb</artifactId>
  <version>1.0</version>
  <dependencies>
    <dependency>
      <groupId>org.daniel</groupId>
      <artifactId>modc</artifactId>
      <version>1.0</version>
    </dependency>
  </dependencies>
</project>

I think i'm having a variation of this issue, but the workaround isn't working for me.
I have an Android project consisting of 1 application project (A), 1 android library project (B), and 1 java library (C).
the dependencies are basically
A -> B+C
B-> C
C is standalone
I'm getting errors on the  transformClassesWithDexForDebug task if I don't apply the retrolambda plugin on the java library project (C). While i intend to use it there anyhow, it smells like something is wrong.


Quick fix to declare an unresolved lifetime

With inband lifetimes going away (https://github.com/rust-lang/rust/issues/44524), I believe we should just fix the ergonomics at the ide layer.
Typing fn foo(&'a str) should emit an error for unresolved 'a, and a quick fix to declare a lifetime: fn foo<'a>(&'a str).
See how NoSuchField diagnostics is treated, which has a similar flow with a fix: https://github.com/rust-analyzer/rust-analyzer/search?q=NoSuchField.
@Veykril I believe you did name resolution for lifetimes, could you add a code link for where it is happening?

Since we don't use lifetimes in type inference and such yet we only actually resolve them in the ide layer(through Semantics).
So I guess we should first actually lower/resolve them in type inference(without using them afterwards, we can just keep the static lifetime hack) and then emit diagnostics from there which I think should happen somewhere around here? https://github.com/rust-analyzer/rust-analyzer/blob/master/crates/hir_ty/src/lower.rs#L162-L351


Read agency website info tags. Iss30

This PR takes two more tags, namely url and contact:website, into account in order to find agency URL.
Close #30

I just noticed a weird behavior after I tried o2g.hiposfer.com with a local file and dummy flag.
The agencies url were all set to hiposfer.com. That made me wonder, is the dummy flag overwriting the agency url?

@carocad does it produce some agency urls when you leave out the dummy flag?


Typescript

I got an error from typescript when i try to use it
Cannot find module 'bunrest'. Did you mean to set the 'moduleResolution' option to 'node', or to add aliases to the 'paths' option?
bun version: 0.1.13

I fixed by installing it on NPM and setting "moduleResolution": "node" in the tsconfig.json.


[Suggestion] sort dice rolls

Hi, I'd love to be able to roll One-Roll Engine style dice, which uses a system for matching sets of dice. To that end, it'd be great to sort dice, e.g.:
/roll 10d10s = 10,10,9,4,3,2,1,1,1,1

Hello, @melissasage.
In Sidekick most rolls generate a scalar that can be used in arithmetic computations.
I don't think it a good idea for a general-purpose sorting modificator to diverge from that practice, thus
/r 10d10s = (10+10+9+4+3+2+1+1+1+1) = 42

But I can probably implement an ORE-specific pseudo-function (like in #90) to the effect of
/r ore (10d10) = 10,10,9,4,3,2,1,1,1,1

Though it will be easier to implement this as repeat modificator:
/r repeat (d10, 10, sorted) = 10,10,9,4,3,2,1,1,1,1

(we already have a repeat modifier implemented in https://github.com/ArtemGr/Sidekick/issues/79#issuecomment-326776091).

I'd actually like this, as sort of a general use thing, that will sort any comma separated list of numbers.
So something like /r sort ( repeat (4d6k3, 6, brief ) )
Would give you a sorted list of ability scores for D&D, for instance. Or you could use it on arbitrary mathematical constructs:
/r sort ( (3+7), (2-2), (9^2) )
and so on.
Additionally, a keep/drop notation that worked on such arbitrary lists would also be handy.
/r sort ( repeat (4d6k3, 7, brief)k6 )
For instance, would generate 7 ability scores, drop the lowest, and then present them in order.

Have you heard about DiceParser ? It is a dice bot for discord and it provides a way to sort result.
To do it:
!10d10s
To add it to your server: https://discordapp.com/oauth2/authorize?&client_id=279722369260453888&scope=bot&permissions=0


PrimaryCareDPCApp

Describe the bug
"I notice there's no actual error shown in the file modifications. Let me help deploy the application"
I am asking the bot to: "Fix the error first. Then add additional code."
Thank you,
stephen
Link to the Bolt URL that caused the error
https://bolt.new/~/bolt-nativescript-js-pgmcqy
Steps to reproduce
not sure how to work with these "steps to reproduce."
Expected behavior
Fix the error and continue building the app. I also want to see the progress of the app. not sure if deployment will give me that view. Let me know if that helps with what I want to see.
Screen Recording / Screenshot
https://github.com/user-attachments/assets/47b35e88-268f-469a-b979-8e1738a4f3b9

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 140960350,
  "usedJSHeapSize": 134828962,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = CBA Advisors DPC
Chat ID = 955dd01eafdf

Additional context
Error:
An error occurred while streaming this message.

Hi @CBAAdvisorsDPC,
Appreciate the feedback and apologies for the issue here. It looks like there was an error during deployment. This is a known error the team is tracking here: #499.


CodexiaReviewRepository.java:24: CodexiaReviewRepository:...

The puzzle 186-f76ca369 from #186 has to be resolved:
https://github.com/iakunin/codexia-bot/blob/55842476b55ea8a4a1723a36925e3cd39eb1fed4/src/main/java/dev/iakunin/codexiabot/codexia/repository/CodexiaReviewRepository.java#L24-L24
The puzzle was created by iakunin on 17-May-20.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 186-f76ca369 has disappeared from the source code, that's why I closed this issue.


Ability to exclude files

Great project!   Is there a way to excluded files?   I have a bunch of generated protobuf files that create lots of noise.   I tried using extended glob exclude syntax but I could not make that work for some reason.

I would think not, anyhow -- this repo does not do anything remotely what you ask for.


Test SDK needs a Subscription Replacer

Test infrastructure currently scrubs the user's subscription but does not scrub secondary subscriptions that may be used to construct cross-subscription tests. We need a mechanism to enable this scenario and re-enable LiveOnly tests such as in SubscriptionLevelDeploymentTest and PolicyScenarioTest in the resource module (PR #7716).

@bim-msft  add to S162.

hi @qwordy could you pls share how cross-subs test supported in live test run?

https://github.com/Azure/azure-cli/pull/13592
Subscription replacer does have some problems when I try to add customized subscription support to preparer.


Update deps

Coverage remained the same at 100.0% when pulling a1627d380dd26f9816fa719d946085f092a7c091 on chore-update-deps into b1d176b091e147709b95f597e9d87fb2d634f2c1 on master.


Mailman module in NixOS requires postfix even though Mailman supports other MTAs

Describe the bug
Mailman module in NixOS requires postfix enabled and has no way to configure Mailman to work with other MTAs supported by both NixOS and mailman upstream.
To Reproduce
Steps to reproduce the behavior:

Disable postfix
Enable mailman
Fail the evaluation with an assertion

Expected behavior
The Mailman module allows to select an MTA of choice (or even adapts to the currently enabled MTA)
Screenshots
n/a
Additional context
Link to Mailman docs about MTA support
Notify maintainers
cc @lheckemann
Metadata

system: "x86_64-linux"
host os: Linux 4.19.150, NixOS, 21.03.20201011.24c9b05 (Okapi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 3.0pre20200829_f156513
channels(root): ""
channels(vika): ""
nixpkgs: /nix/store/vjj6vlbfq661l2zllgvyspdsdyvz8hl9-source

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module: ["mailman"]

It looks like exim's configuration should be RFC42-ised first if we want to inject Mailman-related settings into Exim config, which is a requirement (Mailman requires a custom Exim router and transport).

Yes, the module is currently quite opinionated. If you're willing to contribute exim support that's great.

I see how I could make it work, but the first version of it will require manual editing of Exim's config files (because RFC42-ising Exim will be a pain, even though I have some drafts on how to translate Nix into the syntax that's required), and I'm not sure if it will be merge-worthy enough :3

Exim config is complicated enough as is, I'm not sure adding an intermediate layer from nix will help that much.
I also don't really see this integrating well with existing or other exim config. I'm honestly kind of surprised this works for postfix.


fix(date-picker): undefined dayIndex

🔗 Linked issue

Resolves #2208
❓ Type of change

[x] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thanks 😊


CIV-6483 refactor transition state flow

https://tools.hmcts.net/jira/browse/CIV-6483
refactor transitionTo(PENDING_CLAIM_ISSUED_UNREPRESENTED_DEFENDANT) as it had happened twice
prexisting

removed duplicate state transition and moved transition predicates under the single .transitionTo

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x ] No

@drummondjm looks good to me


List all accounts in the service

As a [user]
I need [the list of all accounts in the service]
So that [every service is accounted for]
Details and Assumptions
* [if this is not achieved,it will result in mismanagement of funds ]

Acceptance Criteria
gherkin
Given [that we have a number of accounts under this service]
When [all accounts are listed]
Then [accurate accountability will be achieved]


Error running instructions in rag24.md Retrieval step

When I run the java code in the retrieval step
export ANSERINI_JAR=anserini-0.36.1-fatjar.jar
export OUTPUT_DIR="runs"
TOPICS=(rag24.raggy-dev rag24.researchy-dev)
for t in "${TOPICS[@]}"; do
    java -cp $ANSERINI_JAR io.anserini.search.SearchCollection \
        -index msmarco-v2.1-doc-segmented \
        -topics $t \
        -output $OUTPUT_DIR/run.msmarco-v2.1-doc-segmented.bm25.${t}.txt \
        -threads 16 \
        -bm25 \
        -hits 100 \
        -outputRerankerRequests $OUTPUT_DIR/retrieve_results_msmarco-v2.1-doc-segmented.bm25.${t}_top100.jsonl

I get
Index file already exists! Skip downloading.
Index folder already exists!
2024-07-21 01:53:23,970 INFO  [main] search.SearchCollection (SearchCollection.java:1008) - ============ Initializing Searcher ============
2024-07-21 01:53:23,974 INFO  [main] search.SearchCollection (SearchCollection.java:1009) - Index: /home/jay/.cache/pyserini/indexes/lucene-inverted.msmarco-v2.1-doc-segmented.20240418.4f9675.6ec4cd595c9fe1ad91b43eabb39a637c
Jul 21, 2024 1:53:23 AM org.apache.lucene.store.MemorySegmentIndexInputProvider <init>
INFO: Using MemorySegmentIndexInput with Java 21; to disable start with -Dorg.apache.lucene.store.MMapDirectory.enableMemorySegments=false
2024-07-21 01:53:24,166 INFO  [main] search.SearchCollection (SearchCollection.java:1012) - Threads: 16
2024-07-21 01:53:24,166 INFO  [main] search.SearchCollection (SearchCollection.java:1013) - Fields: []
2024-07-21 01:53:24,166 INFO  [main] search.SearchCollection (SearchCollection.java:1027) - MaxPassage: false
2024-07-21 01:53:24,167 INFO  [main] search.SearchCollection (SearchCollection.java:1032) - Hits: 100
2024-07-21 01:53:24,167 INFO  [main] search.SearchCollection (SearchCollection.java:1045) - Collection class: null
2024-07-21 01:53:24,168 INFO  [main] search.SearchCollection (SearchCollection.java:1332) - Using DefaultEnglishAnalyzer
2024-07-21 01:53:24,169 INFO  [main] search.SearchCollection (SearchCollection.java:1333) - Stemmer: porter
2024-07-21 01:53:24,169 INFO  [main] search.SearchCollection (SearchCollection.java:1334) - Keep stopwords? false
2024-07-21 01:53:24,170 INFO  [main] search.SearchCollection (SearchCollection.java:1335) - Stopwords file: null
2024-07-21 01:53:24,235 INFO  [main] search.SearchCollection (SearchCollection.java:1345) - ============ Launching Search Threads ============
2024-07-21 01:53:24,235 INFO  [main] search.SearchCollection (SearchCollection.java:1346) - runtag: Anserini
2024-07-21 01:53:26,241 INFO  [pool-3-thread-14] search.SearchCollection$SearcherThread (SearchCollection.java:904) - ranker: bm25(k1=0.9,b=0.4), reranker: default: 100 queries processed
2024-07-21 01:53:27,116 INFO  [pool-2-thread-1] search.SearchCollection$SearcherThread (SearchCollection.java:925) - ranker: bm25(k1=0.9,b=0.4), reranker: default: 120 queries processed in 00:00:02 = ~42.08 q/s
2024-07-21 01:53:28,158 INFO  [main] search.SearchCollection (SearchCollection.java:1418) - Total run time: 00:00:04
Index file already exists! Skip downloading.
Index folder already exists!
2024-07-21 01:53:28,939 INFO  [main] search.SearchCollection (SearchCollection.java:1008) - ============ Initializing Searcher ============
2024-07-21 01:53:28,942 INFO  [main] search.SearchCollection (SearchCollection.java:1009) - Index: /home/jay/.cache/pyserini/indexes/lucene-inverted.msmarco-v2.1-doc-segmented.20240418.4f9675.6ec4cd595c9fe1ad91b43eabb39a637c
Jul 21, 2024 1:53:28 AM org.apache.lucene.store.MemorySegmentIndexInputProvider <init>
INFO: Using MemorySegmentIndexInput with Java 21; to disable start with -Dorg.apache.lucene.store.MMapDirectory.enableMemorySegments=false
2024-07-21 01:53:29,112 INFO  [main] search.SearchCollection (SearchCollection.java:1012) - Threads: 16
2024-07-21 01:53:29,113 INFO  [main] search.SearchCollection (SearchCollection.java:1013) - Fields: []
2024-07-21 01:53:29,113 INFO  [main] search.SearchCollection (SearchCollection.java:1027) - MaxPassage: false
2024-07-21 01:53:29,113 INFO  [main] search.SearchCollection (SearchCollection.java:1032) - Hits: 100
2024-07-21 01:53:29,114 INFO  [main] search.SearchCollection (SearchCollection.java:1045) - Collection class: null
2024-07-21 01:53:29,114 INFO  [main] search.SearchCollection (SearchCollection.java:1332) - Using DefaultEnglishAnalyzer
2024-07-21 01:53:29,115 INFO  [main] search.SearchCollection (SearchCollection.java:1333) - Stemmer: porter
2024-07-21 01:53:29,115 INFO  [main] search.SearchCollection (SearchCollection.java:1334) - Keep stopwords? false
2024-07-21 01:53:29,115 INFO  [main] search.SearchCollection (SearchCollection.java:1335) - Stopwords file: null
Error: "rag24.researchy-dev" does not refer to valid topics.
2024-07-21 01:53:29,168 INFO  [main] search.SearchCollection (SearchCollection.java:1418) - Total run time: 00:00:00

was able to fix it after downloading the topic and then passing a topicReader


ERROR: unable to create encoder input surface (err=-38)

I got this on the android 4.4.2 (API 19) emulator. Can you please let me know if I did anything wrong?
ADB Recorder v0.1
Starting recording on device...
Press Ctrl+C to stop recording.
ERROR: unable to create encoder input surface (err=-38)
Traceback (most recent call last):
  File "record.py", line 53, in <module>
    recorder.send_signal(signal.SIGTERM)
  File "/usr/lib/python2.7/subprocess.py", line 1554, in send_signal
    os.kill(self.pid, sig)
OSError: [Errno 3] No such process

Unfortunately the screenrecord adb command doesn't work on the emulators (stock or Genymotion) because requires hardware acceleration / encoding support isn't implemented. Where's nothing much I can do about it, you'll need an actual device :(

Thanks :->


Add --limit flag

For testing purposes, it'd be good to be able to limit the number of feedstocks checked. That said, this might require a bit of rejiggering - taking the first ten feedstocks we find is easy, taking the first ten that need updates is more complicated. (Version checks currently occur after the feedstocks have been selected.)

Basic flag added! All feedstock teams are still checked, but can still drop out once limit updateable feedstocks have been found. There are some interesting long-term questions here: checks on dependencies and forkability occur after the limit feedstocks have been identified, so the limit is a maximum that may be reduced. I'm considering all of this behavior appropriate for now, and closing this up.

To elaborate on this point: it's not hard to imagine this sitation arising:

A, B, and C are feedstocks that need updating
A and B depend on C.
we run tick_my_feedstocks.py --limit 2
A and B are returned for updates
this causes some later complications because we need to fix C first.

This isn't an immediate issue because the process is going to always be hands-on and involve tweaking, but it does go back to the question of what limit is intended to provide. The basic thing it should help with, I think, is speed in test runs.  I want to be able to break out faster.


Add DigitalIO device configuration and data nodes

Adds support for device configuration
Introduce DigitalInputDataFrame to breakout the device to host payload
Add DigitalIO device to ConfigureBreakoutBoard hub device

Fixes #80

@jonnew The HasFlag operator can be used to pull the state of individual bits in the enums.

I'm trying to test this but its not working as I would expect. This is my workflow:

When I press buttons frames are received, and make to this call:

But in the workflow, no frames are show in the visualizer or checked with the HasFlag operator.

@jonnew This was my last minute attempt at refactoring gone wrong. This operator can be implemented using a simple projection, but when changing it didn't realize I was still inside Observable.Create. It should all work now.

I reviewed this and made some corrections. If you agree with these its good to go. There is logic for grouping the Power and Buttons into one enum: port state actually records the position of the power switches for the ports. So this enum encapsulates all switch/button input from the breakout.


Old Edited comments that are still open trigger the keyboard when the user is scrolling the conversation - Reported by:  @parasharrajat

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

Action Performed:

Open any chat.
Scroll back in the history.
Edit the message. Leave it as it is. Don't close the editor
Go back to LHN.
Open any other chat.
back to LHN.
Open back the previous chat.
Scroll back in the list slowly.
Screen will jump to the message being edited as soon as you reach closed to it.

Expected Result:
Old edited comment that are still open should not trigger the keyboard when the user is scrolling.
Actual Result:
Old Edited comments that are still open trigger the keyboard when the user is scrolling the conversation
Workaround:
Close or save the comments that are being edited.
Platform:

Where is this issue occurring?

iOS
Android

Version Number: 1.1.8-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/44479856/137535354-b90dfc55-59f2-4645-9ca9-d8441984030e.mp4
Expensify/Expensify Issue URL:
Issue reported by: @parasharrajat
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1633794986065800
View all open jobs on GitHub

Triggered auto assignment to @thienlnam (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Triggered auto assignment to @SofiedeVreese (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Posted on Upwork: https://www.upwork.com/jobs/~018f79ec6f97efcf87
Adding Exported Label.

Triggered auto assignment to @NikkiWines (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Hey @NikkiWines is this a dupe from https://github.com/Expensify/App/issues/4756  that has been reopened? Feels like this case would fall under that other GH?

mmm, I'd say they're related but not sure if they're exactly the same issue. In this one, it's more that navigating back to this chat and scrolling over an old open message will retrigger keyboard to pop up.

They are completely different.
This => explained above.
#4756 => has to do with draft messages.

Thanks for the explanation both, we'll keep this issue open then!

PR merged and QA ongoing.

PR deployed, adding a payment hold 7 days from deploy.

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.1.15-15 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/6138

If no regressions arise, payment will be issued on 2021-11-29. :confetti_ball:

Job paid out & contract cancelled. Closing GH.

@SofiedeVreese I think you missed the reporting bonus.

Yes I did, sorry @parasharrajat ! I'll pay that one out now.

Ok that bonus is now also paid out, sorry about that @parasharrajat !

It looks like I forgot to add the Company Offsite hold label to this issue.  I've added the bonus for the hold to his Upwork payment.

That makes sense to me, I was thinking it was ineligible for the n6-hold bonus but hadn't even considered the company offsite bonus and should've compared the dates as well. My bad and thanks for clearing things up @mallenexpensify


L'image buildée doit avoir le nom de la branche ou du tag

Jenkins will now build & push the docker image with tag or branch as prefix.
If branch is master, then Jenkins will also build & push  on latest

branch feature/stuff will result in the image intoo/imgpush:feature-stuff
tag v0.0.2 will result in the image intoo/imgpush:v0.0.2
branch master will result in the images intoo/imgpush:master and intoo/imgpush:latest

It builds https://jenkins.intoo.best/job/lib-imgpush/job/feature%252Ftags/


Artifactory PDF link Online

Hi I am unable to view Artifactory PDF link on quality thought website.
Getting 404 error. Could you please provide me..

Hi Aamukta,
Configuring-Artifactory.pdf
Thanks
Ravi


fix(ui): correctly show Include Node Info

Fixes #917

Pull Request Test Coverage Report for Build 675370470

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 18.209%

Totals

Change from base Build 675331800:
0.0%

Covered Lines:
2026

Relevant Lines:
11410

💛  - Coveralls


Provide the SPI to plug Opensearch / Kafka configurators based on configuration

Provide the SPI to plug Openseach / Kafka configurators based on configuration, closes https://github.com/aiven/opensearch-connector-for-apache-kafka/issues/128

@reta could you plz resolve conflicts and I will merge it

@reta could you plz resolve conflicts and I will merge it

@willyborankin done, thanks!


Need to modify kdl/transformers/keras_applications to be able to use resnet50

Hi,
Per  this overflow question one needs to modify /home/user/.local/lib/python3.8/site-packages/sparkdl/transformers/keras_applications.py
.  This happened in databricks using v 10.3.
Have to change
from keras.applications import inception_v3, xception, resnet50
to
from keras.applications import inception_v3, xception
from tensorflow.keras.applications import resnet50

did you find a way to import the library?


Make mongodb ObjectId (_id) available during run time.

Maybe this is already possible, but here is what I want:
How can I make use of the ObjectId, which will be used to store in MongoDB already during runtime? Ideally, I would like to print it during run time of the experiment and store associated files in a folder dependent on the ObjectId.

Printing the ObjectID is a good idea and is really easy to add. I'll do that soon.
For storing the related files in a directory I think the best solution would be to add a custom observer. The only problem is that this observer needs to know about the ObjectID assigned by the MongoObserver.
This usecase means we might need to add some communication between the observers such that they can cross-reference. That is an important point, but a bit more tricky and I need to think about it a bit.

You can point me at the variable, which stores the _id right now. I guess, the Observer needs to store it, to be able to put each subsequent update in the same location.
I would be fine to write additional file io independently of sacred. Right now I use pickle to store model parameters and just store the file path in sacred. Summing up: Having the _id available at run time would be sufficient.

On 2016May25, at 17:53, Klaus Greff<EMAIL_ADDRESS>wrote:
Printing the ObjectID is a good idea and is really easy to add. I'll do that soon.
For storing the related files in a directory I think the best solution would be to add a custom observer. The only problem is that this observer needs to know about the ObjectID assigned by the MongoObserver.
This usecase means we might need to add some communication between the observers such that they can cross-reference. That is an important point, but a bit more tricky and I need to think about it a bit.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/IDSIA/sacred/issues/107#issuecomment-221617454

It is stored in the MongoObserver in the run_entry dict. So if you did this:
mobs = MongoObserver.create(...)
ex.observers.append(mobs)

Then you can access it with mobs.run_entry['_id'].  (Or alternatively, if you have only one observer: ex.observers[0].run_entry['_id'])
But you know that you can also just store these files in the database by adding them as an artifact, right?

I store some files outside of MongoDB since further processing steps might not cooperate with MongoDB. I now simply use it like this:
@ex.automain
def sacred_main(_config):
    print('MongoDB ID:', ex.observers[0].run_entry['_id'])
    train(
        _id=ex.observers[0].run_entry['_id'],
        **_config
    )

The train function then uses the _id to generate paths.

This feature is added in the labwatch branch (see 2c05572bfd7b19a3bad7e1a3b09b60658140f2bb) and it will be part of the next release (0.7).

This feature is part of the current beta release 0.7b1:

The _id of the current run is printed through logging.
it is available through the _run object like this    @ex.automain
    def main(_run):
        _id = _run._id

the observers coordinate their _id such that they match between databases/filestorage/...

I hope that solves your usecase. Check it out and let me know if there are any problems.


Docs: examples missing whitespace

Many or all of the examples on https://gateway-api.sigs.k8s.io/reference/spec/ are missing leading whitespace on every line.  For instance, the example under https://gateway-api.sigs.k8s.io/reference/spec/#gateway.networking.k8s.io/v1.HTTPRouteRule appears as:
matches:
- path:
value: "/foo"
headers:
- name: "version"
value: "v2"
- path:
value: "/v2/foo"

but I think it's meant to be
matches:
- path:
    value: "/foo"
  headers:
  - name: "version"
    value: "v2"
- path:
  value: "/v2/foo"

btw, this is using Chrome 118.0.5993.117 on OSX 13 if that helps.

/assign

/kind documentation

@murthy95  are you working on it ?
if not i can push changes for this.

@kundan2707 please push your changes.

/assign

When is this expected to be fixed? The bad yaml is very confusing so it should be fixed ASAP

it seems its issue with doc generation. otherwise there is no issue.

it seems its issue with doc generation. otherwise there is no issue.

Yes, I understand. But most people see the generated doc and not the original type definitions so it will be good to fix the tooling

Looking through old issues, it looks like https://github.com/kubernetes-sigs/gateway-api/issues/1458 is covering the same thing, will close that one out in favor of this.
I've mentioned this in Slack before but apparently never on GitHub. I believe the issue here is with the tool we're using to generate CRD Reference docs (https://github.com/ahmetb/gen-crd-api-reference-docs). It's possible that moving to a more recently maintained tool like https://github.com/elastic/crd-ref-docs would either fix the issue or enable us to merge a fix into the upstream tool.
@kundan2707 are you still interested in working on this?

@robscott  yes i will work on it to fix it.

Thanks @kundan2707!


Property 'red' does not exist on type '{ enabled: any; }'.

I'm using chalk with vscode and getting these warning errors below:
Property 'red' does not exist on type '{ enabled: any; }'.
it's a pretty simple thing i'm building:
const chalk = require("chalk")
let cmd = 'something'
console.log(chalk.yellow(cmd))

I wonder if there is some definition/typings file (for JS) or some other magic that's going on that is confusing the VSCode typechecking?

To be clear, are you getting errors when running or using some type-checking static analysis tool?
If it's the latter, please take it up with them. Chalk is valid Javascript - if they're going to try to stomach a static analyzer for a highly dynamic language then this is their problem, not ours.

edit time, yes.
using with VSCode.
OK, I'll take it up with them I guess! But wondering where that '{ enabled: any; }'. could be coming from. is {enabled: {} } some type of object you return from initialization?
any is the default object type, but `{enabled: .. } looks like you're returning a specific field.

I have no idea what any means. I'd imagine it's something similar to Flow typing where VSCode can't determine the type of the enabled flag. It's either truth-y or false-y, anyway.
Yes, please open an issue over there as this is room for them to improve. Our API is a corner case in that we make liberal use of property prototypes. Perhaps they could start reading Typescript definitions even for non-typescript source. Just a thought :)
Thanks for the question!

Perhaps they could start reading Typescript definitions even for non-typescript source

they do exactly that:
https://code.visualstudio.com/Docs/languages/javascript#_automatic-type-acquisition
Most common JavaScript libraries have type declaration files available. You can search for a library's type declaration file package using the TypeSearch site.
https://microsoft.github.io/TypeSearch/
It looks like someone created them for chalk but it's just a stub
https://www.npmjs.com/package/@types/chalk
@types/chalk
This is a stub types definition for chalk (https://github.com/chalk/chalk). chalk provides its own type definitions, so you don't need @types/chalk installed!
VSCode seems to have a lot of momentum as a JS IDE, it would be nice to not have errors popup each time I open a file using chalk...

@dcsan we have typescript defs that were created and verified by several typescript peeps (I don't know TS therefore I cannot comment on how accurate they are). However, those people are pretty smart and I would wager they're probably correct with their definitions.
In which case, it's very much VSCode's job to improve their code hinting functionality. They're also welcome to contribute better definitions.
There's nothing actionable on our end, however. If I'm wrong, let me know.

any ideas how I could install those typedefs? It seems whatever is being pulled down by default, publicly registered is causing the problem.
➜  yarn install

warning<EMAIL_ADDRESS>This is a stub types definition for chalk (https://github.com/chalk/chalk). chalk provides its own type definitions, so you don't need @types/chalk installed!

if you are using import * as chalk from 'chalk';
then chalk.default.yellow(cmd) will work


Adding Kruskal's Algorithm in  Graph Algorithms Folder

@Lakhankumawat  Please assign this issue to me

Kruskal is already present in graphs and in minimum spanning tree folder please have a look , closing this issue


CAS-629: Token Required value does not display correctly for mobile

fixed by https://github.com/DapperCollectives/CAST/pull/605


Avoid email notifications

Depends on #271.
I kind of got tired of those email notifications... What you think?

Thank you. I got tired of them from the moment we started integrating Appveyor.
Just one question, though, why so many commits?

@ocaballeror As I mentioned, this depends on #271 (it includes all commits in that PR). Note that #271 depends on #270 too, which means you might want to review #270 first, #271 then and lastly this PR.
Otherwise just review this PR, but take into account it is the sum of 3 PRs.

@ocaballeror Here again, the only relevant commit is the last one, all the others are already in master.
It seems I am unable to change the "base" of this PR to the current master. And I do not want to close and reopen the PR just for that... :joy:

The tests failed in travis, though. The error is not related to these commits at all, but still worth noticing nonetheless.

Yeah, I noticed. It is the already familiar ns.shutdown() error... :sweat_smile:

@Peque Not this time. In this case it failed on agent.shutdown(), which is kinda weird.


Any support for scala?

Is there any support or examples to run gauge implementations in scala?

Not directly, there was a similar request to support clojure a while back.
But you might be able to get the execution bit working (not authoring). See the example for groovy here: https://github.com/getgauge-examples/groovy-selenium
If you get your scala code to build into a jar, you might be able to get gauge-java to read and execute implementations.
Note: this is purely coincidental, and we don't test out multiple jvm languages. I guess the fact that all JVM based languages compile to the same bytecode helps this case.

Currently, gauge doesn't have any support with scala. You can follow the above instruction to execute scala code.

If you get your scala code to build into a jar, you might be able to get gauge-java to read and execute implementations.

I'm closing this issue. If you have any question you can ask in gitter(https://gitter.im/getgauge/chat)


Autoplay pause that respects the remaining time of slide to be changed

Clear and concise description of the problem
I was a bit confused, found that there is no way to pause the progress of each slide autoplay.
Example:

We have autoplay setting for 5 seconds
After 2 seconds, user make some interaction, api call etc., that pauses the autoplay and then resume
After the resume I expect an option to play current slide with only the remaining time (3 seconds), but got slide timer starts from the beginning only

Suggested solution
I expect an option resumeAfterPause, that could be boolean and if set to true - autoplay of current slide will continue respecting the time before interaction/action. And also would be excellent to have a slide property of autoplayDuration, like we have progress property of each slide. And, of course, I expect to have some methods for it.
The usefull example of implimentation: instagram story-like slider, that have a pagination styled like separate timelines and on every slide you can tap the slide even on the end of it's time but after interaction it will continue just the rest of timeline.
Alternative
No response
Additional context
No response
Validations

[x] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Would you like to open a PR for this feature?

[ ] I'm willing to open a PR

Swiper v9 comes with fully reworked autoplay module. If you have similar issues in Swiper 9, open a new issue with a CodeSandbox showing the issue.


Execute Function Now return error due to url case issue for Custom

I followed the steps here to create a Rust Azure function.
When I try to run 'Execute Function Now' within vscode, I get a 404 response.
When I use 'Copy Function Url', what is copied to the clipboard is https://.azurewebsites.net/api/httptrigger1, but it should be https://.azurewebsites.net/api/HttpTrigger1.
The latter works using a browser or an API client, the former returns the same 404 response.
Environment, versions, etc:
macOS Catalina (10.15.7)
Azure Functions Extension: 3.0.3442
vscode:
Version: 1.55.2 (Universal)
Commit: 3c4e3df9e89829dce27b7b5c24508306b151f30d
Date: 2021-04-13T09:36:32.643Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


compatibility with inveniordm v12

make compatible with InvenioRDM v12
:package: release v2024.3.6

Expect to potentially fail until https://github.com/galterlibrary/galter-subjects-utils/pull/43 is merged and released

also @mattbaumann1


Make suspensions an optional feature

When implementing the new TS SDK, the runtime initiated suspension mechanism made the implementation harder. It would be simpler for a new SDK to be able to ignore this feature altogether if possible. One idea could be to let the SDK tell the runtime whether it supports runtime initiated suspensions (w/o it, the SDK would probably not work on AWS lambda, though).
Another simplification could be to allow the waiting_for_completed_entries field to be empty which denotes that the invocation will be resumed on any journal entry completion.
More details are needed for why the suspension mechanism caused trouble (Stephan, Giselle).

This task is not really actionable right now, will close this.


jsonpb: properly support ListValue well known type

golang/protobuf/jsonpb does not serialize/deserialize ListValue according to the proto3
JSON spec. ListValue is supposed to be translated into a JSON array. Without this change,
ser-des outputs ListValue as a generic protobuf type, e.g.
ListValue{Values: {Value: Value_NumberValue{NumberValue: 1}}}
=> {"Values": [1]}
Correct output should be
=> [1]

This change addresses https://github.com/golang/protobuf/issues/221

I signed it!

Fix already in 18c9bb3261723cd5401db4d0c9fbc5c3b6c70fe8.


Automatically build releases

I unpacked the already-built Workflow to have info.plist also unter version control and added a GitHub action that automatically builds the .alfredworkflow and creates a release, once a new version is tagged.

@birnstiel Thank you for this idea. I already use a custom tool I wrote called alfred-workflow-packager to export the packaged workflow. But this is an interesting alternative.
A few questions:

How does the version-controlled info.plist get updated if I make changes to the workflow objects in Alfred Preferences?
How does the version of the packaged workflow get updated? Is is set to the tag I just created? (if so, it would need to be without the v prefix)
Does the packaged workflow generated by the Github Action get committed as a second commit because the tagged commit was already created? I would prefer the exported workflow to be part of the same commit that is tagged, hence the reason why I export the workflow (with the desired version) before  creating/tagging the release commit

@birnstiel Thank you, but yes, I think I would prefer to stick with the system I have. However, if you're interested in building your own custom version of the workflow with my system, here's a rundown of the commands for setting up my alfred-workflow-packager tool:
# Optional; switch to the project directory if you're not already there
cd ./projects/
# Stand up virtualenv for project
pip3 install virtualenv
virtualenv --python=python3 ./.virtualenv
# Activate virtualenv in current directory (run `deactivate` to exit virtualenv context)
source .virtualenv/bin/activate
# Install project dependencies
pip install -r requirements.txt
# Copy any changed repository files to the installed workflow; run this
# whenever you have project scripts or assets that have changed
awp
# When you want to bump the version and re-export the workflow to the repository
awp -v 4.0.0-beta.1 -e

I may add a CONTRIBUTING.md file in the future to contain this info. But for now, what you have above is the basic know-how.
Also, here's a link to my alfred-workflow-packager repository, if you want to learn more about configuring it:
https://github.com/caleb531/alfred-workflow-packager


Replace occamy's memory macro

The file needs to modify is tc_sram.sv in tech_cells_generic
It is sufficient to cover 32 -> 64 -> 128 -> 256 -> 512 five cases

Let's start with what we need and we add more later 😄


Update Loggregator network paths

Update Loggregator network paths for PAS 2.4.
Related story: https://www.pivotaltracker.com/story/show/162497749
/cc @MasslessParticle

Thanks @bradylove !   What docs versions should this be on (based on PCF minor releases)?  Master branch will publish to upcoming 2.4 docs.

This is updated for PCF 2.4


IGAPP-946: Add ukrainian translations

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.

are we waiting for uk shelter translations?

I'd say so, yes.


Templatefactory

Please review the new system for template creation.
At the end of the it.py file there is a commented code that can be used to test the new code and to learn how to use it.

@jccosta this brought the test suite from 3mn to 45mn with some test (diags11) timing out. I'm reverting this. Please hold on until the test suite is fixed before submitting a new PR. At this point I will send instruction on how to run the suite which should always be ran and pass before submitting a PR.

@jccosta see: https://open.cdash.org/viewTest.php?onlyfailed&buildid=4259473
vs the one before your merge:
https://open.cdash.org/viewTest.php?onlyfailed&buildid=4259479

Thanks for point it to me, Charles.
I will take a look on what is happening and I will wait for your instructions.

Thank you! Hopefully in a few hours.


Enhacement: Use Firefox Multi-Account Containers in browser_command

Hi. Great plugin, use it a lot.
Would be great to be able to provide container name in firefox's Multi-Account Containers plugin
So the command in
~/.awsume/config.yaml might look like:
console:
  browser_command: /Applications/Firefox.app/Contents/MacOS/firefox -container-name "AWS-{profile}" -no-remote "{url}"

I was able to achieve this by using
https://github.com/honsiorovskyi/open-url-in-container
Mostly thanks to this comment: https://github.com/mozilla/multi-account-containers/issues/365#issuecomment-526965740
Currently my config looks like:
console:
  browser_command: /Users/username/.local/bin/firefox-container --orange --fingerprint --name={profile} "{url}"

@noizo   Thanks for following up and posting the solution you found.  It's working great for me.


I'm unable to submit your survey.

This isn't related to the extension; I got a notification about your extension, and proceeded to fill it out. However, when I got to the last page, I see this at the bottom:

When I click the "Next" button, I am redirected to the same page, and the bottom stays the same. I went back and counted my questions, and there are only 17 of them, so not sure why it insists there are 18. The bar in the picture looks like its full as well.
I don't want to lose my responses (because I'm unwilling to have to write them again), so I'm just letting you guys know about this.

This sounds like a bug with survey monkey.  We don't really have any control over that.  Feel free to follow-up with them.  I'm going to let our product manager know about this in case there's anything they can do about it.  FWIW, at this point I'm not aware of anyone else having problems with the survey.


Connection to azure.archive.ubuntu.com/ubuntu

azure.archive.ubuntu.com/ubuntu also need to be allowed so that user can update the security package on the cluster.

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

ID: f48eda35-f874-4ea3-27be-e49fd2767399
Version Independent ID: 8f683e01-020f-807a-624b-a4e971a6ce14
Content: Configure outbound network traffic restriction - Azure HDInsight
Content Source: articles/hdinsight/hdinsight-restrict-outbound-traffic.md
Service: hdinsight
GitHub Login: @JasonWHowell
Microsoft Alias: jasonh

@xujiongda Thank you for the feedback. We are actively investigating and will get back to you soon.

@xujiongda
Thanks for the feedback! We have assigned the issue to the content author to further review this and provide an update as appropriate.

@farukc @nicjohn79 do you know this subject by chance?

@JasonWHowell Yes, I think they are saying 'azure.archive.ubuntu.com' needs to be added to the Target FQDNs section. @xujiongda is the Target FQDNs section is where this endpoint needs to be added in the documentation?

@nicjohn79 Thank you for checking this. Yes, it needs to be added in the Target FQDNs section.


[BUG] Redis as memory not working

Describe the bug
No data is being stored in redis when configured as the memory.
To Reproduce
Steps to reproduce the behavior:
Use the following docker-compose file to launch services.
version: '3.1'

services:
    flowise:
        image: flowiseai/flowise
        restart: always
        environment:
            - PORT=${PORT}
            - PASSPHRASE=${PASSPHRASE}
            - FLOWISE_USERNAME=${FLOWISE_USERNAME}
            - FLOWISE_PASSWORD=${FLOWISE_PASSWORD}
            - DEBUG=${DEBUG}
            - DATABASE_PATH=${DATABASE_PATH}
            - APIKEY_PATH=${APIKEY_PATH}
            - SECRETKEY_PATH=${SECRETKEY_PATH}
            - LOG_LEVEL=${LOG_LEVEL}
            - LOG_PATH=${LOG_PATH}
        ports:
            - '${PORT}:${PORT}'
        volumes:
            - ~/.flowise:/root/.flowise
        command: /bin/sh -c "sleep 3; flowise start"
    cache:
        image: redis:6.2-alpine
        restart: always
        ports:
        - '6379:6379'
        command: redis-server --save 20 1 --loglevel warning
        volumes:
        - cache:/data
volumes:
    cache:
        driver: local

Expected behavior
When using a tool like redis-commander, you should be able to see the chat history stored in redis.
Screenshots

Flow
{"nodes":[{"width":300,"height":525,"id":"chatOpenAI_0","position":{"x":754.8942497823595,"y":-70.76607584232393},"type":"customNode","data":{"id":"chatOpenAI_0","label":"ChatOpenAI","version":1,"name":"chatOpenAI","type":"ChatOpenAI","baseClasses":["ChatOpenAI","BaseChatModel","BaseLanguageModel"],"category":"Chat Models","description":"Wrapper around OpenAI large language models that use the Chat endpoint","inputParams":[{"label":"Connect Credential","name":"credential","type":"credential","credentialNames":["openAIApi"],"id":"chatOpenAI_0-input-credential-credential"},{"label":"Model Name","name":"modelName","type":"options","options":[{"label":"gpt-4","name":"gpt-4"},{"label":"gpt-4-0613","name":"gpt-4-0613"},{"label":"gpt-4-32k","name":"gpt-4-32k"},{"label":"gpt-4-32k-0613","name":"gpt-4-32k-0613"},{"label":"gpt-3.5-turbo","name":"gpt-3.5-turbo"},{"label":"gpt-3.5-turbo-0613","name":"gpt-3.5-turbo-0613"},{"label":"gpt-3.5-turbo-16k","name":"gpt-3.5-turbo-16k"},{"label":"gpt-3.5-turbo-16k-0613","name":"gpt-3.5-turbo-16k-0613"}],"default":"gpt-3.5-turbo","optional":true,"id":"chatOpenAI_0-input-modelName-options"},{"label":"Temperature","name":"temperature","type":"number","default":0.9,"optional":true,"id":"chatOpenAI_0-input-temperature-number"},{"label":"Max Tokens","name":"maxTokens","type":"number","optional":true,"additionalParams":true,"id":"chatOpenAI_0-input-maxTokens-number"},{"label":"Top Probability","name":"topP","type":"number","optional":true,"additionalParams":true,"id":"chatOpenAI_0-input-topP-number"},{"label":"Frequency Penalty","name":"frequencyPenalty","type":"number","optional":true,"additionalParams":true,"id":"chatOpenAI_0-input-frequencyPenalty-number"},{"label":"Presence Penalty","name":"presencePenalty","type":"number","optional":true,"additionalParams":true,"id":"chatOpenAI_0-input-presencePenalty-number"},{"label":"Timeout","name":"timeout","type":"number","optional":true,"additionalParams":true,"id":"chatOpenAI_0-input-timeout-number"},{"label":"BasePath","name":"basepath","type":"string","optional":true,"additionalParams":true,"id":"chatOpenAI_0-input-basepath-string"}],"inputAnchors":[],"inputs":{"modelName":"gpt-3.5-turbo","temperature":0.9,"maxTokens":"","topP":"","frequencyPenalty":"","presencePenalty":"","timeout":"","basepath":""},"outputAnchors":[{"id":"chatOpenAI_0-output-chatOpenAI-ChatOpenAI|BaseChatModel|BaseLanguageModel","name":"chatOpenAI","label":"ChatOpenAI","type":"ChatOpenAI | BaseChatModel | BaseLanguageModel"}],"outputs":{},"selected":false},"selected":false,"positionAbsolute":{"x":754.8942497823595,"y":-70.76607584232393},"dragging":false},{"width":300,"height":385,"id":"conversationChain_0","position":{"x":1174.6496397666272,"y":311.1052536740497},"type":"customNode","data":{"id":"conversationChain_0","label":"Conversation Chain","version":1,"name":"conversationChain","type":"ConversationChain","baseClasses":["ConversationChain","LLMChain","BaseChain"],"category":"Chains","description":"Chat models specific conversational chain with memory","inputParams":[{"label":"System Message","name":"systemMessagePrompt","type":"string","rows":4,"additionalParams":true,"optional":true,"placeholder":"You are a helpful assistant that write codes","id":"conversationChain_0-input-systemMessagePrompt-string"}],"inputAnchors":[{"label":"Language Model","name":"model","type":"BaseChatModel","id":"conversationChain_0-input-model-BaseChatModel"},{"label":"Memory","name":"memory","type":"BaseMemory","id":"conversationChain_0-input-memory-BaseMemory"},{"label":"Document","name":"document","type":"Document","description":"Include whole document into the context window","optional":true,"list":true,"id":"conversationChain_0-input-document-Document"}],"inputs":{"model":"{{chatOpenAI_0.data.instance}}","memory":"{{RedisBackedChatMemory_0.data.instance}}","document":"","systemMessagePrompt":""},"outputAnchors":[{"id":"conversationChain_0-output-conversationChain-ConversationChain|LLMChain|BaseChain","name":"conversationChain","label":"ConversationChain","type":"ConversationChain | LLMChain | BaseChain"}],"outputs":{},"selected":false},"selected":false,"positionAbsolute":{"x":1174.6496397666272,"y":311.1052536740497},"dragging":false},{"width":300,"height":331,"id":"RedisBackedChatMemory_0","position":{"x":762.0517911197522,"y":478.6077943073576},"type":"customNode","data":{"id":"RedisBackedChatMemory_0","label":"Redis-Backed Chat Memory","version":1,"name":"RedisBackedChatMemory","type":"RedisBackedChatMemory","baseClasses":["RedisBackedChatMemory","BaseChatMemory","BaseMemory"],"category":"Memory","description":"Summarizes the conversation and stores the memory in Redis server","inputParams":[{"label":"Base URL","name":"baseURL","type":"string","default":"redis://localhost:6379","id":"RedisBackedChatMemory_0-input-baseURL-string"},{"label":"Session Id","name":"sessionId","type":"string","description":"If not specified, the first CHAT_MESSAGE_ID will be used as sessionId","default":"","additionalParams":true,"optional":true,"id":"RedisBackedChatMemory_0-input-sessionId-string"},{"label":"Session Timeouts","name":"sessionTTL","type":"number","description":"Omit this parameter to make sessions never expire","additionalParams":true,"optional":true,"id":"RedisBackedChatMemory_0-input-sessionTTL-number"},{"label":"Memory Key","name":"memoryKey","type":"string","default":"chat_history","additionalParams":true,"id":"RedisBackedChatMemory_0-input-memoryKey-string"}],"inputAnchors":[],"inputs":{"baseURL":"redis://<IP_ADDRESS>:6379","sessionId":"test","sessionTTL":"","memoryKey":"chat_history"},"outputAnchors":[{"id":"RedisBackedChatMemory_0-output-RedisBackedChatMemory-RedisBackedChatMemory|BaseChatMemory|BaseMemory","name":"RedisBackedChatMemory","label":"RedisBackedChatMemory","type":"RedisBackedChatMemory | BaseChatMemory | BaseMemory"}],"outputs":{},"selected":false},"selected":false,"positionAbsolute":{"x":762.0517911197522,"y":478.6077943073576},"dragging":false}],"edges":[{"source":"chatOpenAI_0","sourceHandle":"chatOpenAI_0-output-chatOpenAI-ChatOpenAI|BaseChatModel|BaseLanguageModel","target":"conversationChain_0","targetHandle":"conversationChain_0-input-model-BaseChatModel","type":"buttonedge","id":"chatOpenAI_0-chatOpenAI_0-output-chatOpenAI-ChatOpenAI|BaseChatModel|BaseLanguageModel-conversationChain_0-conversationChain_0-input-model-BaseChatModel","data":{"label":""}},{"source":"RedisBackedChatMemory_0","sourceHandle":"RedisBackedChatMemory_0-output-RedisBackedChatMemory-RedisBackedChatMemory|BaseChatMemory|BaseMemory","target":"conversationChain_0","targetHandle":"conversationChain_0-input-memory-BaseMemory","type":"buttonedge","id":"RedisBackedChatMemory_0-RedisBackedChatMemory_0-output-RedisBackedChatMemory-RedisBackedChatMemory|BaseChatMemory|BaseMemory-conversationChain_0-conversationChain_0-input-memory-BaseMemory","data":{"label":""}}]}

Setup

Installation docker
Flowise Version 1.3.5
OS: macOS
Browser chrome

Additional context
No errors in the logs.

this https://github.com/FlowiseAI/Flowise/pull/1025 should fixed the issue


Cursor changes position between consecutive calls to complete functions

Describe the bug
First and second calls to any of 'complete-function's (both omnifunc and completefunc) might have different cursor positions.
To Reproduce
Detailed steps to reproduce the behavior:

Save the following configuration file (for example, under 'completefunc-curpos.vim'):

let g:curpos_dict = {}
function g:CompleteFunc(findstart, base)
  let g:curpos_dict[a:findstart] = getpos('.')

  if a:findstart == 1
    return 2
  else
    return ['Jun', 'Jul']
  endif
endfunction

set omnifunc=g:CompleteFunc

Run vim -u completefunc-curpos.vim (or insert other name for configuration file)
In Insert mode type 'abcdefg'.
Type '<C-x><C-o>' to call omnifunc.
Type '<Esc>:echo g:curpos_dict<Cr>'. This should show {'0': [0, 1, 3, 0], '1': [0, 1, 8, 0]} meaning that cursor changed position from [0, 1, 8, 0] to [0, 1, 3, 0] (start of completion item) between calls.

Expected behavior
Cursor position doesn't change from first to second call to 'complete-function'. This issue is even more apparent, when using completeopt=noselect as cursor doesn't visibly change its position.
Screenshots
Animation of behavior:

Environment (please complete the following information):

Vim version [e.g. 8.1.1234] (Or paste the result of vim --version.)
OS: Ubuntu 18.04
Terminal: GNOME Terminal

Additional context
Cursor will not actually move after first call if it returned -1 because it "moved" to the same spot.
Change in cursor position is relevant to writing custom completion functions.

I believe what happens is this: On the first call of the omnicompletion function, you tell vim to start completing from column 2. So Vim removes the old text until column 2, meaning the cursor cannot be at at position 8 anymore, since the line is too short. That's why on the second call the cursor position is at 2. Because that is where you want the completion to be inserted. Perhaps we should just document that the cursor will be positioned on the column that is returned on the first call of the completion function.

Yes, this seems like a the most reasonable explanation of the current behavior. And at least documentation would be nice, certainly (it would have saved some time of my life :) ).
I would argue, though, that this behavior is not really intuitive. I don't see a reason why active buffer should be modified at all when computing completion matches on the second call. Especially, when having noselect in compleopt: currently on second call Vim basically removes all text subject to completion, computes matches, and then pastes back previous text to achieve the exact starting state.


How to create my .bin?

I have trained a model based on OpenPCDet, but I have no way to create a .bin of my model, thus I've got all 0 when I submit my result……
Help me, plz!!!

Have you read this FAQ?

Thanks for your reply.
I have try to submit the gt.bin, it can get all 1 result, my question is that I have used OpenPCDet to train a model, it get a result.pkl, but it unable to submit normally, maybe the dataformat is different……

Hi, Tim, I am in same situation. Just wonder if you found a solution yet?