Add time zone support to datetime utilities

Airtable is adding more time zone formatting options to datetime fields. This pull request updates the datetime utilities to handle values with time zones other than UTC.

Why using pytz? Cant we just use ZoneInfo which is the way to go with timezones in the future?

@NicoHood honestly I typically use arrow for all of this, so I was just looking for a fallback but the change you suggest was easy to make. Thanks for the suggestion.

what is arrow? But the changes look nice, thanks!

Arrow has relieved so many date and time headaches for me.

Arrow is a Python library that offers a sensible and human-friendly approach to creating, manipulating, formatting and converting dates, times and timestamps. It implements and updates the datetime type, plugging gaps in functionality and providing an intelligent module API that supports many common creation scenarios. Simply put, it helps you work with dates and times with fewer imports and a lot less code.

hey similar to other prs look there are a few unrelated changes

@gtalarico just learning here... you'll probably want to squash the mess of commits I've made.
Thanks for your patience! Let me know if I need to do anything else.

I've left a lot of comments. I strongly recommend to accept my suggestions. If there are any questions, please ask. Timezones are a bit fuzzy.

Thanks, @NicoHood. I've incorporated your suggestions. Let me know if anything else is needed from me here, @gtalarico

Hi there, I'm going through older PRs in this repo and trying to merge or close them. I see that this PR uses the zoneinfo library (which is not present in Python 3.7 or 3.8) and that the tests don't seem to pass:
    def test_datetime_utils(datetime_obj, datetime_str):
>       assert utils.datetime_to_iso_str(datetime_obj) == datetime_str
E       AssertionError: assert '2025-12-31T2...:59.555-08:00' == '2025-12-31T23:59:59.555Z'
E         - 2025-12-31T23:59:59.555Z
E         ?                        ^
E         + 2025-12-31T23:59:59.555-08:00
E         ?                        ^^^^^^

In my limited experience and testing, the Airtable API will always return timestamps in UTC, regardless of how the base is configured to display them on the web. Given that, can I close this pull request? If there is a use case that I'm missing, can you share some steps to reproduce?

It may be the case, that the API outputs UTC, but the date that we put into the api from python might not be. Also this hack with hardcoding "Z" as timezone is not very smart. Its just more flexible.
I personally would not care too much about older python versions that are mostly end of life:
https://endoflife.date/python

It may be the case, that the API outputs UTC, but the date that we put into the api from python might not be.

I might've missed the intent here; I thought this was about parsing values from Airtable, not the values we pass in. If the goal is to allow an implementer to pass a timezone-aware datetime to the API, I think a patch to that effect (with passing tests showing what new library behavior it offers) would be very welcome. I'm guessing this is for the ORM?

I personally would not care too much about older python versions that are mostly end of life

This repo has no written compatibility policy (maybe it should) but my instinct is that a widely-used library should keep supporting all live Python versions until at least the official end-of-life. Plenty of organizations still have code running on Python 3.7, and Python 3.8 won't be end-of-life for another year and a half. Maybe we can use backports.zoneinfo here.

Hi, I'm doing another pass through old issues & PRs, and I'm wondering if we should close this one. If there's a legit bug or feature gap for us to address with respect to timezones, I think I could use help reproducing it. Even a code snippet against the Airtable API showing the expected vs actual behavior would be very helpful.

I think the PR should still get merged. The hardcoded Z might just crash. You can now set specific timezones in airtable. I am not sure if the API will always return UTC, but adding the correct %z identifier is just safe and future-proof. A must-have I'd say and a nobrainer.
About the isoformat I would say the +Z is also a bug and should not be there. The data should either always have a timezone set or at least be intepreted as a specific timezone. This can be UTC or the local timezone. But always adding +Z is wrong, if the input data has a timezone set. I guess the code will then crash.
I am not sure where those functions are currently being used. I remember only in ORM, but I am not sure. I am not using ORM so I guess I dont have those problems. Timezones are tricky, but we should treat it with care.
I would reopen/merge this. at least line 18 and 29

@NicoHood I'll be happy to review any future pull requests that follow the contribution guidelines. Right now this branch does not. In either case, without steps to reproduce, it's not clear to me there's a bug.


KAFKA-14729: The kafakConsumer pollForFetches(timer) method takes up a lot of cpu due to the abnormal exit of the heartbeat thread

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hi @guozhangwang @showuon
please help to review PR when available.
Thanks.

This is an interesting find, thanks @RivenSun2 . In general I think if the background thread dies for whatever the reason we should consider the following actions in precedence:

Make sure the consumer now falls into an abnormal state which would not return data, and would not try to tie up the caller thread. This is also for notifying the user.

Try to "selfheal" by re-creating the thread (we do not need to do it in this PR, just laying out the ground here), in order to bring the consumer back to normal state.

If we cannot selfheal the consumer and it simply becomes useless, let the consumer to throw an exception for any API calls so that the caller thread would then go ahead and recreate a brand new consumer.

As for this PR, I'm actually thinking if it makes sense to introduce a new state inside MemberState, e.g. "ABNORMAL" to indicate that the consumer is not usable at the moment --- even if it can still successfully fetch records since it does not rely on the heartbeat thread --- and under that state we should not proceed by returning any more data, in order to notify the caller of this consumer.
cc @kirktrue @philipnee

Hi @guozhangwang thank you for your reply.
Can we introduce a state like "ABNORMAL" in a new PR, this state may only appear on consumers with heartbeat threads (group.id!=null).
Introducing this state also needs to consider the value brought by this state and the impact on existing code logic. I think it's such a big change that it even needs a KIP?
In general, this PR should be a small change to solve the problem that the current pollForFetches method takes up cpu.
What do you think? Thanks.

In general I think if the background thread dies for whatever the reason we should consider the following actions in precedence:
1. Make sure the consumer now falls into an abnormal state which would not return data, and would not try to tie up the caller thread. This is also for notifying the user.
2. Try to "selfheal" by re-creating the thread (we do not need to do it in this PR, just laying out the ground here), in order to bring the consumer back to normal state.
3. If we cannot selfheal the consumer and it simply becomes useless, let the consumer to throw an exception for any API calls so that the caller thread would then go ahead and recreate a brand new consumer.

Good suggestion. I was thinking we can directly throw exception in timeToNextHeartbeat method when heartbeatThread is failed. That should be good enough. WDYT?

Hi @guozhangwang
could you give any suggestions?
Thanks.

Hi @RivenSun2 after some thought I think I also agree with @philipnee on throwing an exception directly form the foreground thread's callers. We can defer more complicated improvements such as adding new states or trying to categorize different reasons that kills the heartbeat thread into future works, as @kirktrue is revamping the consumer now.

Hi @showuon  @guozhangwang  @philipnee
Thank you for your suggestions.
If you guys fix this issue in the future, please help to update the jira status.
Thanks.

By the way, you can assign that jira to relevant experts.

Hey, @RivenSun2, thanks again for the PR and response - I think we still need to address this with the current client.  We are currently trying to refactor the consumer code, but that (the new client) will come later, so I think this jira is appropriate for this issue.

Sorry, maybe I didn't make it clear.
I also suggest to track this issue with KAFKA-14729, just hope you guys assign this jira to relevant experts. And after this problem is better solved in the future, will update the status of KAFKA-14729 to be resolved.
Thanks.

@RivenSun2 , I think what we're saying here, is we should throw exception in main consumer thread for a short-term solution. That is, throwing exception in timeToNextHeartbeat if heartbeat thread is failed. For your concern:
try {
               consumer.poll(duration);
           } catch (Exception e) {
               log.error("has error when consumer poll!", e);
           }

In this case, it'll be the consumer client's issue, not ours. The consumer should handle each kind of exception differently, not just catch all exceptions and keep retrying.
So, back to the original question, are you able to update the PR to throw exception in timeToNextHeartbeat if heartbeat thread is failed? If you don't have time, we can assign to other contributors. Thank you.

Hi @showuon Thank you for your reply.
Happy to submit a short-term solution. I will submit new code changes later.
Thanks.

Hi @showuon Thank you for your reply. Happy to submit a short-term solution. I will submit new code changes later. Thanks.

Great! Thank you!

update code and test case.

Hi @showuon  could you help review this PR?
Thanks.

Failed testcases should not be related to this PR.


[WIP] A little refactor to remove the need for nil parameters.

Codecov Report

Merging #222 into master will decrease coverage by 0.25%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #222      +/-   ##
==========================================
- Coverage    62.1%   61.85%   -0.26%
==========================================
  Files          49       49
  Lines        4059     4066       +7
==========================================
- Hits         2521     2515       -6
- Misses       1124     1135      +11
- Partials      414      416       +2

Impacted Files
Coverage Δ

store/memdb/memdb.go
53.67% <ø> (ø)
:arrow_up:

security/vault.go
70.71% <0%> (-7.86%)
:arrow_down:

handlers/hook.go
63.15% <100%> (ø)
:arrow_up:

workers/server/worker.go
50% <100%> (ø)
:arrow_up:

workers/scheduler/scheduler.go
65.43% <100%> (ø)
:arrow_up:

handlers/workers.go
48.12% <100%> (ø)
:arrow_up:

workers/pipeline/ticker.go
48.38% <100%> (ø)
:arrow_up:

workers/pipeline/git.go
70.96% <100%> (ø)
:arrow_up:

handlers/vault.go
57.37% <100%> (ø)
:arrow_up:

services/service_provider.go
74.6% <77.77%> (-0.4%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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

\o/
$ go version
go version go1.12.10 darwin/amd64


feat: ordered mention suggestions

Details

Implement mention suggestions with order of relevancy when in a group chat or a chat connected to a workspace:

People within the chat (members of the Report)
People within the Workspace
Other Contacts

Fixed Issues

$ https://github.com/Expensify/App/issues/42009
PROPOSAL: https://github.com/Expensify/App/issues/42009#issuecomment-2105413569
Tests

Type @ in a group chat with several participants
Verify the order of suggestions: group members >> other contacts, both in alphabetical order
Type in a letter
Verify the list is filtered to the individuals containing that letter with the same hierarchy preserved as described above

Type @ in a Workspace room, where only a fraction of the workspace members are in the room
Verify the order of suggestions: room members >> workspace members >> other contacts, all in alphabetical order
Type in a letter
Verify the list is filtered to the individuals containing that letter with the same hierarchy preserved as described above

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

Offline tests

NA
QA Steps

Same as Tests

[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

Android: mWeb Chrome

iOS: Native

https://github.com/Expensify/App/assets/153004152/da1cca36-c925-43d1-8524-c00f514797d5

iOS: mWeb Safari

MacOS: Chrome / Safari

Group chat

Workspace room

MacOS: Desktop

@gijoe0295 Can you check this in groups, in my case suggestion is not showing report members first

@gijoe0295 Can't add to an existing group

Looks like this is happening only with old groups. I couldn't reproduce the issue when creating a new croup even choosing the same members

I will complete the checklist

🚀 Deployed to staging by https://github.com/yuwenmemon in version: 1.4.84-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.4.84-3 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Transpile only test files during jest tests

To triple ensure we don't create another problem like #22, lets remove webpack, and rely on jest's babel transform specifically for the test files only.
Fixes #24
Note: This branch was branched BEFORE we landed the fix for #22, so the commit should error on tests in it's branch, but not in the merged version.

It feels weird to merge with the push test failing. But the fix is already in master and why the pr test passed.


Bad comparison when setting up default wallets

In WalletConnectSharp.Unity.WalletConnect::SetupDefaultWallet there is a comparison between the enum Wallets name and the supported wallets names in https://registry.walletconnect.org/data/wallets.json that don't match.
For example, "Trust" vs. "Trust Wallet"

This has been fixed in #32


enabled devcontainers

[Feature] Enabling DevContainers for Development
Fixes #199
STEPS

Dockerfile: Given there. put it inside .devcontainer
Port Forwarding: The port 8000 is forwarded for Vite
VS Code Extensions:
eslint for code linting.
prettier for code formatting.
gitlens for Git integration.
typescript extension (since you’re using Vite, this is useful for TypeScript support).
Post Create Command: yarn to install your dependencies after the container is built.

Major Steps:

Make  sure you have a Dockerfile inside your .devcontainer folder.
Run the command Dev Containers: Reopen in Container in VS Code to initialize the DevContainer setup.

Please let me know if any suggestions.


Minor ReportActionComposer code clean up

Details

Minor ReportActionComposer code clean up
Fixed Issues
$ https://github.com/Expensify/App/issues/22547
Apply this change: https://github.com/Expensify/App/pull/21962#issuecomment-1627763320
Tests

Open the app
Open any report
Copy report URL and paste in compose box
Add few numbers to it to make it invalid and send
Click on the link, it should open 'Hmm...it's not here' page
Click on back to observe that app does focus back on compose box

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

Offline tests

N/A
QA Steps
Same as tests

[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 / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

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

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If 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 author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/13113013/d7f5cd89-15b5-4c98-affd-63953e74c36a

Mobile Web - Safari
https://github.com/Expensify/App/assets/13113013/5e7f2d07-902a-4b98-9ae2-dc5195a1c621

Mobile Web - Chrome
https://github.com/Expensify/App/assets/13113013/73d1c7c4-2d55-42d0-844a-19ab2dbc63fd

Desktop
https://github.com/Expensify/App/assets/13113013/c32c5691-4c28-4cf3-a531-1cf3d4bbde06

iOS
https://github.com/Expensify/App/assets/13113013/b8c8abcb-58d1-4cdb-98da-1fbbebb4ca1b

Android
https://github.com/Expensify/App/assets/13113013/03f6cca0-96bb-415c-bdae-03c9c7827a7b

🚀 Deployed to staging by https://github.com/Julesssss in version: 1.3.39-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.39-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Bot doesn't start after NuGet update

This is moving issue from doc repo to dotnet repo. See initial issue here, but will copy main text to this issue. V4, issues running echo bot after updated NuGet
First comment
After updating all packages to the latest version the bot project does not start.
To reproduce this issue just create a empty "Echo Bot" from the template and start it in Visual Studio 2017 with the "Start" Button (debugging, IIS Express). Every thing should work as expected. Now go to "Manage Nuget Packeges" and update all packages. When you try to start the project now it fails with the message "HTTP Error 502.5 - ANCM Out-Of-Process Startup Failure". Changing the debug settings does not have any impact.
Second comment
User doesn't believe its related to any issues we linked
"To reproduce the Issue just create an empty Echo Bot and update to .net core 2.2. (everything is working..)
Now add the Microsoft.Bot.Builder.AI.QnA nuget package - here there are some issues with possible downgrading of nuget packages and the project does not start anymore."
[bug]

I've been trying to duplicate this so far. Following the steps outlines:

Created an Echo bot, using the 4.2.2 VSIX that was published yesterday.
Updated the resulting project to Net Core 2.2

<PropertyGroup>
    <TargetFramework>netcoreapp2.2</TargetFramework>
</PropertyGroup>

Fixed the compile error by updating Microsoft.Extensions.Logging.AzureAppServices from 2.1.1 to 2.2.0.
Added in the QnAMaker Nuget package.

The resulting package set look like this:
  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.All" />
    <PackageReference Include="Microsoft.Bot.Builder" Version="4.2.2" />
    <PackageReference Include="Microsoft.Bot.Builder.AI.QnA" Version="4.2.2" />
    <PackageReference Include="Microsoft.Bot.Builder.Integration.AspNet.Core" Version="4.2.2" />
    <PackageReference Include="Microsoft.Bot.Configuration" Version="4.2.2" />
    <PackageReference Include="Microsoft.Bot.Connector" Version="4.2.2" />
    <PackageReference Include="Microsoft.Bot.Schema" Version="4.2.2" />
    <PackageReference Include="Microsoft.Extensions.Logging.AzureAppServices" Version="2.2.0" />
  </ItemGroup>

So far, all runs.
Added a line of code to the Bot, that will force-load the QnA Package:
var foo = new Microsoft.Bot.Builder.AI.QnA.Metadata();
Still, all runs.
I'm not able to see the same bug(s) here that you're seeing. The emulator works find, and the bot is happily running.
Are there more detailed repro steps you could provide?

@cleemullins, do you still want Support assistance? I see you self assigned 1hr ago

Can you please do a dotnet --info at the command line and paste the output here? I've seen this before when a person had a preview version of the SDK installed. Everything will work fine locally, but when you publish to the Azure servers they will not contain the preview version of the runtime so it will fail to initialize in this way.
Another thing you can try to do is open the Advanced Tools (aka Kudu portal) and then use the "Debug Console" and navigate to /LogFiles/ and then open the eventlog.xml file there which should contain much more detail about why the process is failing to start up.

@drub0y  I don't think this involves Azure. The repro steps are all local.

Oh jeez, that's totally my bad... terrible reading on my part. I saw something that looked familiar and my brain just went with it. Apologies.
Ok, so even easier to get details to help further diagnose:

Repro the issue again
Go into Event Viewer
Go to the Windows Logs / Application Log
Look for a an Error from the IIS Express AspNetCore Module
Copy paste any/all details here

Alternatively, for an even easier debugging experience, stop debugging it under IIS Express and change to debug the application directly. You can do this by clicking the little drop down arrow next to the debug button and changing from the IIS Express profile to the  profile instead:

Alternatively you can right click on your project, choose property and then under the Debug tab choose the profile from the Profile: drop down.
Either way, this will run the executable directly in a console instance and it should dump out the start up issues right to that console.

Closing this.  The customer has confirmed they are unable to reproduce with the latest VSIX template. https://github.com/MicrosoftDocs/bot-docs/issues/834#issuecomment-463504942


minify graphql

Hello! We are using your product in our company in pair with laravel-graphql
(https://github.com/Folkloreatelier/laravel-graphql)
We have a great pleasure!
However. Can you add a minify button to graphiql near to prettify button?
Our programmers use get queries based on graphiql url and share them with each other, however after you format query you need to minify it back manually or it will be %%%%%horrible-whitespace-uri-encoded%%%%%.
Please can you add a minify option

Hi! This seems like a pretty niche use case so I don't think it's something we'll build in the foreseeable future. We're trying to keep built in features in graphiql as simple as possible.
You can add this functionality in by adding a custom <GraphiQL.Button> into the toolbar. I'm going to go ahead and close this issue since it isn't something on our roadmap.

How do you guys share graphql between multiple developers?

@drmax24 I'm having troubles understanding what you need "minification" for sharing queries between developers. Is it solely because of the formatting of the URL?
In Facebook we have a number of solutions to share queries between developers. We use a URL shortener to create a shorter/simpler version of GraphiQL url with query as a URL parameter. We also have a classical "export" feature that allows people to save queries/variables/responses in some sort of a web document (imagine something similar to Gist). I'm not sure if you're familiar with a concept of "persisted queries" - that's another way that we can share a query amongst others, by providing a persisted query ID for others to load in the GraphiQL tool.
I'm not sure what your situation allows for, but my suggestion is to try one of the above ways instead of sharing a query using an URL param. Even with the URL shortener, queries and variables tend to get bigger in size, and you will hit the URL length limit per browser specifications.

@asiandrummer I think he's referring to the ability to minify a query to remove unnecessary whitespace and comments from it (I might be wrong though but that's how I understand it).
@drmax24 You can checkout https://github.com/imolorhe/altair if it helps.


Submitting a comment

When I submit a comment, all is fine. However when I go to refresh that page I get a dialogue box open up asking for resubmission of form details.
Anyway to be able to refresh without this occuring?

This is a generic warning message that is displayed by many web-browsers when re-requesting a page that has received HTTP-Post data. If you are reloading (or “refreshing”), then the browser sends the HTTP-Post data again to the server. Kirby Comments can detect such a situation and stops you from re-submitting your comment (i.e. clicking “Continue” in the dialog window will not post the comment twice).
The dialog window is a browser feature; I don’t think Kirby Comments can prevent it.

When posting comments on other blogs/forums etc. Using similar comments plugins i dont get such submission issues on refresh.

Forms that use JavaScript/AJAX to submit the form data do not encounter this situation, as the data is not re-submitted on page reload. If you know of any example form system where plain HTTP requests are used and Google Chrome is not showing the dialog window when reloading, please let me know.


Update lodestar-params with medalla params

Once medalla testnet is live, we need to update lodestar/packages/lodestar-params/src/presets/mainnet.yaml and lodestar/packages/lodestar-params/src/presets/mainnet.yaml with the latest params from spec
mainnet (with diff from previous testnet) can be found here:
https://gist.github.com/q9f/941b2621214d05ef8759abde30e4f9ef
General medalla testnet info:
https://github.com/goerli/medalla/tree/master/medalla

Resolved with #1242


Slapp direct_mention/mention not responding back

Hi I was looking at https://github.com/BeepBoopHQ/slapp/issues/60 and I added that to my "Subscribe to Bot Events":

But I can't get a response from: @slapptest hello in my #general channel. I can however get a response from direct_message.
This is my Slapp app:
const Slapp = require('slapp');
const express = require('express');

const {
  SLACK_CLIENT_VERIFICATION_TOKEN: verify_token,
  SLACK_OAUTH_ACCESS_TOKEN: app_token,
  SLACK_CLIENT_ID: app_user_id,
  SLACK_BOT_OAUTH_ACCESS_TOKEN: bot_token,
  PORT: port
} = process.env;

const slapp = Slapp({
  context(req, res, next) {
    req.slapp.meta = Object.assign({}, req.slapp.meta, {
      verify_token,
      app_token,
      app_user_id,
      bot_token
    });
    return next();
  },
  log: true,
  colors: true
});

const app = slapp.attachToExpress(express());

slapp.message(
  '.*',
  ['direct_mention', 'mention', 'direct_message'],
  (msg, text) => {
    msg.say('How are you');
  }
);

console.log(`Listening on port: ${port}`);
app.listen(port);

Am I missing something?

I was missing bot_user_id which has to be the slack id from the bot user.


Imported tables have huge filesize difference than pg_relation_size

Fixes #2204

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)


Cant't download in nobrowser mode - use normal browser interaction to download

Hi,
I try to download file in nobrowser mode.

First try

My script have a one line:

https://doghouse.ru/pricelist/100sp_full.csv

Result:
There is no file in the directory

Second try:

My script:

download https://doghouse.ru/pricelist/100sp_full.csv to 100sp.csv

wait 5
Result:
File exists, but it size is zero!
To run script i'm use command on my Ubuntu machine:

tagui test.tag -n

Help please.

ERROR - for headless and visible Chrome, download file using normal webpage interaction
You should get the error above. Currently, you will need to download the file through normal webpage interaction (clicking).

I'm use normal browser interaction, see - first try
It's work in normal mode, but in nobrowser mode it's doesnt work.
File doesn't download.

For normal browser interaction, you would need a link to click on to download the file. Visiting the URL won't work.

Ok, i create simple page with DOWNLOAD link https://doghouse.ru/test
My script:
https://doghouse.ru/test
click DOWNLOAD
wait 10

My command:
tagui test.tag -n

START - automation started - Wed May 27 2020 14:01:16 GMT+1000 (+10)
https://doghouse.ru/test -

click DOWNLOAD
wait 10

https://doghouse.ru/test -
FINISH - automation finished - 10.5s

My result - there is now file in the directory,
P.S.: it's work with normal mode ( tagui test.tag -h)

Rocket - it's cool)
What should I do?

What's about problem? You solve it?

Oh I thought it was solved already.
chrome_step('Page.setDownloadBehavior',{behavior: 'allow', downloadPath: 'c:\\Users\\userid\\Downloads'});
Try this before clicking download.

It's doesn't work. It's freeze on start:
gorcer@www2:~/tagui/doghouse$ tagui test.tag -n

START - automation started - Thu May 28 2020 17:08:49 GMT+1000 (+10)

My script:
chrome_step('Page.setDownloadBehavior',{behavior: 'allow', downloadPath: '/home/gorcer/tagui/doghouse/'});

https://doghouse.ru/test
click DOWNLOAD
wait 5

NOBROWSER mode

tested
https://doghouse.ru/test
click DOWNLOAD

run with tagui my_flow.tag -h
-h headless is to not show a browser.
-n nobrowser cannot download the file.

Okay (((
Headless mode does't work on some my servers and if it's work it take to 3x system resource more.

Hi Egor, sorry I was away the past few months - https://github.com/tebelorg/RPA-Python/issues/144
I see.. Yes -headless mode runs Chrome using invisible mode, and Chrome takes up a lot of memory. However, it is better for automation because it is more accurate than the -nobrowser mode. That mode uses PhantomJS web browser which is an old browser and more likely to encounter issues during automation with modern websites.


add indication for failes status of report (#4245)

Pull request opened by the merge tool on behalf of #4245

Merged in 031ee3f64106dbfa317715ae3d3c906175a7c30a


Specify time-zone in calendar

Pages with times (e.g. https://www.chilconference.org/calendar.html#tab-thu), should specify that times are EDT.

Eastern Time (GMT-5)

@corcra / @tasmie check to see that this is sufficient and close if so.

@tnaumann I like how EDT is under Time on the Thursday schedule instead of beside it on the Friday schedule


Fix capturemacs script with general boot device

Left all surroundings the same and added the capturemacs (fcc & ign)
@phoracek @RamLavi

I will try to set a call on this, feels like to much of a mess to me

It is more work for the author of the PR but it makes reviewing so much more efficient, reviewers would get  better idea of what are you changing and why, and it protects us from introducing regressions.
From my experience on other projects it also makes PRs get merged faster compared to big monolithic PRs with huge diffs.


Add forgot password function for local users

Users forget passwords, so give them my way to reset them.  Send an email to reset.

Excellent! Thanks!


Mark Modrinth release to support 1.19.4

Hey, I've noticed that this mod supports 1.19.4 release but it only marked 1.19.4-pre1 as supported, can you mark the release setting to support 1.19.4 release?
P.S. For some reason your modrinth page does not have link to the repository, it would be great if you could add link to this repository as well.

done


include! macro not being expanded

I'm trying to use rust-project.json and I have proc macros mostly working, except that the include! macro does not seem to be being expanded properly. I'm using the sysroot_src parameter, so maybe it's something to do with librustc_macros.so not being loaded properly? Is it expected to be at a specific location relative to sysroot_src or even the rust toolchain?
The exact way I'm testing this is by using tonic to generate a proto definition, and then using include!("/absolute/path/to/file.rs"), but the types are not resolved in other files (e.g. {unknown} on hover). When I manually replace the include! with the actual contents of the generated file the types are all resolved as expected, so it seems to be the include! macro.

librustc_macros.so is a set of proc macros used during compilation of rustc itself. That it can be found in lib/ is just an artifact of the way the build system works. include! is directly implemented in rustc cq rust-analyzer.

If you're using an absolute path, that's probably it. We don't support including files outside the workspace, and I think we just don't support including absolute paths in general.

Yes, I'm using an absolute path, and reading over the code it does look like it include_expand only deals with relative paths. This makes me curious if include!(concat!(env!("OUT_DIR"), "/foo.rs")) works since OUT_DIR appears to me to be 1) an absolute path and 2) not required to be inside the workspace (e.g. when passing --target-dir).
Are there technical reasons for the limitation to relative paths and requiring files to be inside the workspace?

Actually, now I think absolute paths should work as long as they are in the workspace. The file just needs to be inside the same source root as the including file. I'm not 100% sure how the rust-project.json paths map to source roots, but probably both paths should be inside the same path defined using include inside the same Crate, and the included path should not be excluded.

Are there technical reasons for the limitation to relative paths and requiring files to be inside the workspace?

Rust-analyzer simply doesn't know about files outside the workspace. All files are added to the VFS in advance before parsing any single file using set_file_text. Due to the query architecture it is not possible to read arbitrary files without completely avoiding the query architecture. Avoiding it would prevent any changes to the read file to be propagated to the analysis results.

Thanks! These were the hints I needed to get it working. I had to set both the normal crate root as well as the OUT_DIR in the include array of the project config. With that set up, the VFS knows about the files and can resolve them from absolute paths.


Handle contingency matrices correctly

use verbose option properly
rename contingency matrix attribute

This PR does not apply any more due to removal of knowledgegraph.py as part of moving to ICEES KG cleanup, so closing the PR.


Trade not taking

Press any key to start  after that trade not taking

Its work?

The project is out of date. If u want to fix it just fork and make your changes.


Tidy up waiting for futures in snippets

As @coryan pointed out in #2571, the polling loops are long and hard to read - we should do something about it.

I started fixing these while fixing the tests to run against production.


Upload to hackage

@lpsmith We've got a pull request to upgrade to GHC 8 in a backwards incompatible way. Would you mind merging that and uploading to hackage? Thanks!

I would like to :+1: this.
I'm also thinking about using this library but I'm not sure if it's still maintained given https://github.com/lpsmith/linux-inotify/commit/b441753a199e0034a42b1f6b9fc8499c673063cb was merged quite some time ago but new version was still not published.

I think I'm actually a maintainer now, lol. I could upload it.

Oh wait, I did upload new stuff! I'm pretty sure <IP_ADDRESS> has everything.

@turboMaCk If there's something more you're looking for, then I'll definitely take what you got (or implement what you need) and push it to hackage.


formatGraphQuery Tests

What does this PR do and why?
While working on #312 , I noticed a couple oddities with the graph queries created by the formatGraphQuery util function, in particular that they would often include a redundant "id" key, e.g. project { id id }.
While the redundant key doesn't break things, it was an easy fix to make while still keeping the current behavior of always including at least id, so I did that. I also took this as an opportunity to write a couple of first tests, using Jest since it's already bundled in with react-scripts.
In the process of writing those tests, I noticed little quirks with formatting, so I addressed those as well. Example of a change:
# before:
projects(where: { id: "1" ,creator: "123"  }) { id }
                          ^               ^

# after:
projects(where: { id: "1", creator: "123" }) { id }

One extra thing, slightly unrelated to the above (if you'd prefer I split this lmk, I included it since it was a 2 line change in the same file): I updated WhereConfig to have context of the type it's filtering on, so that key is type aware, and only supports keys that exist on the entity being queried.
Screenshots or screen recordings
Not applicable.
Acceptance checklist

[x] I have evaluted the Approval Guidelines for this PR.

great fixes! thanks for paying close attention.


[QT-Wallet] Wallet setting Title need to be renamed

Description:
In Settings-->"Wallet Configuration" page Title need to be renamed from "mwc713 settings" to "MWC QT wallet Settings".
Please refer screenshot for more details.

Actually "mwc713 setting" need to be deleted.
In the beginning there was mwc713 settings only. Now it is not relevant.
The page name displayed in the App title.
Also that will make this page less crowded

Verified with the latest build, hence closing this ticket.


Update scala-compiler, scala-library, ... to 2.13.5

Updates

org.scala-lang:scala-compiler
org.scala-lang:scala-library
org.scala-lang:scala-reflect

from 2.13.2 to 2.13.5.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

labels: library-update, semver-patch

Superseded by #479.


HSEARCH-2709 Add support for converting MatchNoDocsQuery to Elasticsearch

https://hibernate.atlassian.net/browse/HSEARCH-2709

Jenkins, retest this please

Jenkins, retest this please

Merged, thanks!


Unable to change price selling/items to the sell option

We have implemented the hunting script, unable to sell new items nor able to change price, are you able to advise where this is?

Hi, first colored parameter is the selling price of items but if it's not defined it will be ignored for example second parameter will be ignored and players won't be able to sell that item.

if your issue still remains contact me by discord.
I probably need more information/logs/config etc.

Thank you tested and working, got two more issues to raise with you but I added you on discord to chat about if that's easier?
Should be quick fixed tbf I cant figure them out.

no problem happy to help. discord is faster to talk about bugs. I mostly write the reason in commits after every debug.


Absolute position not supported

figma,Absolute position not supported

What is an absolute position in figma. Please share an example and explain
how it should work in your opinion.
zlt110168 @.***> schrieb am Fr., 7. Apr. 2023, 08:58:

figma,Absolute position not supported
—
Reply to this email directly, view it on GitHub
https://github.com/KlausSchaefers/figma-design-system-example/issues/2,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWHOQGCYSLY47WRBX4BMWTW77CMHANCNFSM6AAAAAAWWJ6224
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

/


Externalize ReactiveStreams.h

Now have a dependency on https://github.com/ReactiveSocket/reactive-streams-cpp which can be installed into /usr/local/include
SmartPointers etc stay here, now in /streams/ as they are an implementation details, not part of the interface definition
We can eventually pull out the streams utilities if we create a concrete implementation library (such as ReactiveX-cpp/RxCpp) that has a type implementing Reactive-Streams

Will figure out what's wrong a little later. Off to meetings right now.

A permissions problem of some kind:
CMake Error at cmake_install.cmake:36 (file):
  file INSTALL cannot copy file
  "/home/travis/build/ReactiveSocket/reactivesocket-cpp/working/reactive-streams-cpp/include/ReactiveStreams.h"
  to "/usr/local/include/ReactiveStreams.h".

SmartPointers etc stay here, now in /streams/ as they are an implementation details, not part of the interface definition

I think this should change, per https://github.com/ReactiveSocket/reactivesocket-cpp/issues/11. Better now than later.

Sudo?
From: Ben Christensen<EMAIL_ADDRESS>Reply-To: ReactiveSocket/reactivesocket-cpp<EMAIL_ADDRESS>Date: Tuesday, 24 May 2016 at 14:59
To: ReactiveSocket/reactivesocket-cpp<EMAIL_ADDRESS>Cc: Mateusz Machalica<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [ReactiveSocket/reactivesocket-cpp] Externalize ReactiveStreams.h (#48)
A permissions problem of some kind:
CMake Error at cmake_install.cmake:36 (file):
file INSTALL cannot copy file
"/home/travis/build/ReactiveSocket/reactivesocket-cpp/working/reactive-streams-cpp/include/ReactiveStreams.h"
to "/usr/local/include/ReactiveStreams.h".
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHubhttps://github.com/ReactiveSocket/reactivesocket-cpp/pull/48#issuecomment-221413138

Sudo?

Yes :-) I just had to go to another meeting so haven't fixed this yet.

sudo fixed it.

The commits in this should be squashed when merging. The messages of all but the first commit can be deleted when squashing.

Are we mixing 2 styles already? Did I miss that? I thought we have been using FB style so far. And honestly the FB style is not bad at all. I think it is very similar to google style and other github libs. There are only very few things where we differ.

I'll open a new pull request after updating the naming convention to /reactive-streams/ReactiveStreams.h

New pull request: https://github.com/ReactiveSocket/reactivesocket-cpp/pull/50


[SPARK-47771][PYTHON][DOCS][TESTS][FOLLOWUP] Make max_by, min_by doctests deterministic

What changes were proposed in this pull request?
Make max_by, min_by doctests deterministic
Why are the changes needed?
https://github.com/apache/spark/pull/45939 fixed this issue by sorting the rows,
unfortunately, it is not enough:
in group department=Finance, two rows ("Finance", "Frank", 5) and ("Finance", "George", 5) have the same value years_in_dept=5, so min_by("name", "years_in_dept") and max_by("name", "years_in_dept") is still non-deterministic.
This test failed in some env:
**********************************************************************
File "/home/jenkins/python/pyspark/sql/connect/functions/builtin.py", line 1177, in pyspark.sql.connect.functions.builtin.max_by
Failed example:
    df.groupby("department").agg(
        sf.max_by("name", "years_in_dept")
    ).sort("department").show()
Expected:
    +----------+---------------------------+
    |department|max_by(name, years_in_dept)|
    +----------+---------------------------+
    |   Consult|                      Henry|
    |   Finance|                     George|
    +----------+---------------------------+
Got:
    +----------+---------------------------+
    |department|max_by(name, years_in_dept)|
    +----------+---------------------------+
    |   Consult|                      Henry|
    |   Finance|                      Frank|
    +----------+---------------------------+
    <BLANKLINE>
**********************************************************************
File "/home/jenkins/python/pyspark/sql/connect/functions/builtin.py", line 1205, in pyspark.sql.connect.functions.builtin.min_by
Failed example:
    df.groupby("department").agg(
        sf.min_by("name", "years_in_dept")
    ).sort("department").show()
Expected:
    +----------+---------------------------+
    |department|min_by(name, years_in_dept)|
    +----------+---------------------------+
    |   Consult|                        Eva|
    |   Finance|                     George|
    +----------+---------------------------+
Got:
    +----------+---------------------------+
    |department|min_by(name, years_in_dept)|
    +----------+---------------------------+
    |   Consult|                        Eva|
    |   Finance|                      Frank|
    +----------+---------------------------+
    <BLANKLINE>
**********************************************************************

Does this PR introduce any user-facing change?
no
How was this patch tested?
ci
Was this patch authored or co-authored using generative AI tooling?
no

thanks @LuciferYang and @HyukjinKwon
merged to master


Fix mobile CSS styling

On mobile screens, landing page subtitle runs off on multiple lines
Post landing page has cut off subtitle text
Sidebar is acting a bit glitchy with the phone viewport, will check this out and do more testing


Add note on suppressing trimming warnings

Inspired by the suppression justification from https://github.com/dotnet/linker/pull/2891.
Cc @vitek-karas @sbomer @tlakollo @eerhardt

I'm still terrible at thinking about trimming the right way - I think what the linker does today, not what it "Should do" in theory.

The more aggressive constant propagation in .NET 7 already puts the IL linker into these territories today. It doesn't kick in often, but when it does, statically used properties and methods get removed.

@adegeo is this okay to merge?


CompileTimeTracker fails to compile in Unity 5.6.1f1.

Does anyone know how to compile CompileTimeTracker? These two compilation error show when I try compiling CompileTimeTracker in Unity 5.6.1f1.

Hi DucRP,
I'm also using Unity 5.6.1f1 and I don't see this compilation error. Lines 8 and 9 of EditorApplicationCompilationUtil.cs should look like:
7: public static class EditorApplicationCompilationUtil {
8:     public static event Action StartedCompiling = delegate {};
9:     public static event Action FinishedCompiling = delegate {};

Is that what you're seeing? From the error it sounds like maybe they got changed to UnityEvent instead of Action possibly?
Here's a link to the EditorApplicationCompilationUtil.cs from latest master.

Closing due to inactivity.


[FEATURE] : Component to display the list of entrypoints of the project. 💡

Description:
This issue proposes implementing a feature to improve user experience by introducing a component to display the project's entrypoints list. Additionally, it includes functionality to navigate directly to a selected entrypoint node in the asset tree with the click of a button.
This enhancement will greatly assist users in navigating projects more efficiently, particularly in cases where projects contain multiple entrypoints and numerous directories before reaching the entry file or folder. Users frequently move between these entrypoints, making this feature very beneficial.

I discussed this with @lrasmus, and recieved few valuable inputs for the same.
I'm looking forward to take this up. I've developed a draft component for the same, will soon open a PR. 😃


Fold All Regions feature doesn't work

From @suadev on October 18, 2018 8:42

VSCode Version: 1.28.2
OS Version: Windows

"Fold All Regions" feature doesn't work neither by ctrl+k, ctrl+8 nor by F1 menu.
Copied from original issue: Microsoft/vscode#61220

From @vscodebot[bot] on October 18, 2018 8:42
(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

[folding] show folding actions at the end of folding range as well (#60670)
[folding] navigateToLastEditLocation doesn't expand folded regions (#59383)

Please add a code sample. I tried with the folloing sample and it worked
foo.ts
//#region
Test
//#endregion
//#region
Test2
//#endregion

ctrl + k / ctrl + 8: folds all regions
ctrl + k / ctrl + 9: unfolds all regions

From @suadev on October 23, 2018 10:43
@aeschli sorry, i should have specified the language.
It looks,  it's ok for .ts file.
But for .cs file, it doesn't work.
public class foo
{
    #region newregion
    int i = 0;
    #endregion
}

Do you use an extension for c# ?
Out of the box you get indentation based folding with region markers sand your example works the same way as my example.
But maybe an extension adds language specific folding and doesn't mark the regions correctly.

From @suadev on October 23, 2018 14:14
Yes, i use c# extension that powered by omnisharp.  My whole extension list;

The problem is with the C# extension's foldingRangeProvider: Ranges originating from #region/#endregion need be marked with Kind: FoldingRangeKind.Region.

@suadev What are the keys you are using to fold the region. I tried the combination specified here https://code.visualstudio.com/docs/editor/codebasics#_folding and it seems to work fine for me. Note that in using  Fold Level X (Ctrl+K Ctrl+2 for level 2) folds all regions of level X, except the region at the current cursor position the region at the current cursor position will not be folded.

@akshita31  as i said in my first post;
"Fold All Regions" feature doesn't work neither by ctrl+k, ctrl+8 nor by F1 menu.

@suadev As per the vscode documentation here : https://code.visualstudio.com/docs/editor/codebasics#_folding, "Fold All" is Ctrl K+ Ctrl 0 and that seems to work fine for me. Can you try that once ?

However I do agree that Fold All regions through F1 doesnt work here.

@suadev Apologies I got a bit confused here. I am totally able to reproduce the problem.

On further investigation, I realised that we are using the "BlockStructureService" from roslyn to provide the spans for CodeFolding. The "BlockType" defined in roslyn doesn't have an explicit type for a "Region" - http://source.roslyn.io/#Microsoft.CodeAnalysis.Features/Structure/BlockTypes.cs,863b31bd5e041d18. For regions it returns the base type called "NonStructural".
In omnisharp-roslyn, we are comparing the type and checking if the type == "Region" which is not returned by roslyn as of now, hence we return the null type to the extension and the given region is not marked as a "Region"

Same issue here -- folding works flawlessly for languages such as JS but fails for CS.
Keep up the great work =)

Any news on the issue?

@akshita31: anything we can do to help with the issue?

@rchande: anything I can do to help you with the issue? Would be really great to have this working again :) Thanks in advance!

@zeroskyx The issue is that OmniSharp isn't properly marking "region" folding units as having the "region" kind. Thus, when you issue the command in VS Code, the C# extension doesn't think it has any regions to fold.

@rchande: thank you for the clarification -- what would be the steps to fix the issue? It would be really great to have it working again with .NET Core 3 coming up :)
Keep up the great work!

Have you given up on this issue? With region support in almost every language now (even Bash) it's sad to see that this feature only does not work in C# 😕

This issue is fixed in 1.21.17

@JoeRobich  it doesn't work. Between, when i start vscode, i see this exception;
System.InvalidCastException: Specified cast is not valid.
at Roslyn.Utilities.EventMap.GetRegistries_NoLock[TEventHandler](String eventName)
at Roslyn.Utilities.EventMap.AddEventHandler[TEventHandler](String eventName, TEventHandler eventHandler)
at lambda_method(Closure , LifetimeContext , CompositionOperation )
at System.Composition.Hosting.Core.LifetimeContext.GetOrCreate(Int32 sharingId, CompositionOperation operation, CompositeActivator creator)
at System.Composition.Hosting.Providers.ImportMany.ImportManyExportDescriptorProvider.<>c__DisplayClass3_21.<GetImportManyDescriptor>b__4(ExportDescriptor e)    at System.Linq.Enumerable.WhereSelectArrayIterator2.MoveNext()
at System.Linq.Buffer1..ctor(IEnumerable1 source)
at System.Linq.Enumerable.ToArray[TSource](IEnumerable1 source)    at lambda_method(Closure , LifetimeContext , CompositionOperation )    at System.Composition.Hosting.Core.LifetimeContext.GetOrCreate(Int32 sharingId, CompositionOperation operation, CompositeActivator creator)    at System.Composition.Hosting.Providers.ImportMany.ImportManyExportDescriptorProvider.<>c__DisplayClass3_21.b__4(ExportDescriptor e)
at System.Linq.Enumerable.WhereSelectArrayIterator2.MoveNext()    at System.Linq.Buffer1..ctor(IEnumerable1 source)    at System.Linq.Enumerable.ToArray[TSource](IEnumerable1 source)
at System.Composition.Hosting.Core.CompositionOperation.Run(LifetimeContext outermostLifetimeContext, CompositeActivator compositionRootActivator)
at System.Composition.Hosting.Core.LifetimeContext.TryGetExport(CompositionContract contract, Object& export)
at System.Composition.CompositionContext.GetExport(CompositionContract contract)
at System.Composition.CompositionContext.GetExports(Type exportType, String contractName)
at System.Composition.CompositionContext.GetExports[TExport](String contractName)
at OmniSharp.Stdio.Host.Initialize() in D:\a\1\s\src\OmniSharp.Stdio\Host.cs:line 57
at OmniSharp.Stdio.Host..ctor(TextReader input, ISharedTextWriter writer, IOmniSharpEnvironment environment, IServiceProvider serviceProvider, CompositionHostBuilder compositionHostBuilder, ILoggerFactory loggerFactory, CancellationTokenSource cancellationTokenSource) in D:\a\1\s\src\OmniSharp.Stdio\Host.cs:line 51
at OmniSharp.Stdio.Driver.Program.<>c__DisplayClass0_1.b__1() in D:\a\1\s\src\OmniSharp.Stdio.Driver\Program.cs:line 66
at OmniSharp.HostHelpers.Start(Func`1 action) in D:\a\1\s\src\OmniSharp.Host\HostHelpers.cs:line 29

It seems, OmniSharp doesn't start properly on my machine.  The exception is added on above. Any idea? @aeschli  @filipw

do you have a repro project?
also, can you post full Omnisharp log?

@filipw i am working on a dotnet  core 3.1 web api project. Tried on aother project and got same exception. The whole log:
Starting OmniSharp server at 4/14/2020, 9:31:30 AM
Target: c:\source\Pms.Backend\PMS.sln
OmniSharp server started.
Path: C:\Users\skose.vscode\extensions\ms-dotnettools.csharp-1.21.17.omnisharp\1.34.5-beta.28\OmniSharp.exe
PID: 15168
    Starting OmniSharp on Windows 6.2.9200.0 (x64)

    DotNetPath set to dotnet

    Located 2 MSBuild instance(s)
        1: Visual Studio Enterprise 2019 16.4.29806.167 - "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\Bin"
        2: StandAlone 16.0 - "C:\Users\skose\.vscode\extensions\ms-dotnettools.csharp-1.21.17\.omnisharp\1.34.5-beta.28\.msbuild\Current\Bin"

    Registered MSBuild instance: Visual Studio Enterprise 2019 16.4.29806.167 - "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\Bin"

System.InvalidCastException: Specified cast is not valid.
at Roslyn.Utilities.EventMap.GetRegistries_NoLock[TEventHandler](String eventName)
at Roslyn.Utilities.EventMap.AddEventHandler[TEventHandler](String eventName, TEventHandler eventHandler)
at lambda_method(Closure , LifetimeContext , CompositionOperation )
at System.Composition.Hosting.Core.LifetimeContext.GetOrCreate(Int32 sharingId, CompositionOperation operation, CompositeActivator creator)
at System.Composition.Hosting.Providers.ImportMany.ImportManyExportDescriptorProvider.<>c__DisplayClass3_21.<GetImportManyDescriptor>b__4(ExportDescriptor e)    at System.Linq.Enumerable.WhereSelectArrayIterator2.MoveNext()
at System.Linq.Buffer1..ctor(IEnumerable1 source)
at System.Linq.Enumerable.ToArray[TSource](IEnumerable1 source)    at lambda_method(Closure , LifetimeContext , CompositionOperation )    at System.Composition.Hosting.Core.LifetimeContext.GetOrCreate(Int32 sharingId, CompositionOperation operation, CompositeActivator creator)    at System.Composition.Hosting.Providers.ImportMany.ImportManyExportDescriptorProvider.<>c__DisplayClass3_21.b__4(ExportDescriptor e)
at System.Linq.Enumerable.WhereSelectArrayIterator2.MoveNext()    at System.Linq.Buffer1..ctor(IEnumerable1 source)    at System.Linq.Enumerable.ToArray[TSource](IEnumerable1 source)
at System.Composition.Hosting.Core.CompositionOperation.Run(LifetimeContext outermostLifetimeContext, CompositeActivator compositionRootActivator)
at System.Composition.Hosting.Core.LifetimeContext.TryGetExport(CompositionContract contract, Object& export)
at System.Composition.CompositionContext.GetExport(CompositionContract contract)
at System.Composition.CompositionContext.GetExports(Type exportType, String contractName)
at System.Composition.CompositionContext.GetExports[TExport](String contractName)
at OmniSharp.Stdio.Host.Initialize() in D:\a\1\s\src\OmniSharp.Stdio\Host.cs:line 57
at OmniSharp.Stdio.Host..ctor(TextReader input, ISharedTextWriter writer, IOmniSharpEnvironment environment, IServiceProvider serviceProvider, CompositionHostBuilder compositionHostBuilder, ILoggerFactory loggerFactory, CancellationTokenSource cancellationTokenSource) in D:\a\1\s\src\OmniSharp.Stdio\Host.cs:line 51
at OmniSharp.Stdio.Driver.Program.<>c__DisplayClass0_1.b__1() in D:\a\1\s\src\OmniSharp.Stdio.Driver\Program.cs:line 66
at OmniSharp.HostHelpers.Start(Func`1 action) in D:\a\1\s\src\OmniSharp.Host\HostHelpers.cs:line 29

according to your log, you are on the old version of omnisharp
Path: C:\Users\skose.vscode\extensions\ms-dotnettools.csharp-1.21.17.omnisharp\1.34.5-beta.28\OmniSharp.exe
it should be 1.35.0.
check if you have any omnisharp.path setting in VS Code config and if you do, remove it.
Then try removing the extension and installing it again.

@filipw  done :)  Thank you !


[terraform] Reduce c5.2xlarge mem limit

Otherwise cluster-test refused to start on c5d.2xlarge, because available mem was 15463:
service cluster-test-validator-2cc0e46f was unable to place a task because no container instance met all of its requirements. The closest matching container-instance<PHONE_NUMBER>0c4f25964925a1b22aca20 has insufficient memory available. For more information, see the Troubleshooting section.

@andll can you review https://github.com/libra/libra/pull/920 while you're here?

@bors-libra r+

:pushpin: Commit 4208e19 has been approved by bschwab

:hourglass: Testing commit 4208e197328aa656ed9e6a945d6aa5aa7123411d with merge 11e00c82eef8160c98480c7a3038e69439f8238b...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: bschwab
Pushing 11e00c82eef8160c98480c7a3038e69439f8238b to master...


[Suggestion} Please add xvideos gallery support

add xvideos gallery support please

Would like to have that too


Add a method to get the current value of progress bar

While it is possible to manually set the value to progress bar. If you need to increment the value, you need to do the following.
var progress_bar_width = $('#jspsych-progressbar-inner')[0].style.width.slice(0,-1);
jsPsych.setProgressBar(progress_bar_width*0.01+0.02);
It would be convenient for users if there is a method that returns the current value of progress bar.

Agreed.
If you haven't manually altered the progress bar you can get the current value like this:
jsPsych.progress().percent_complete


Feature:  Random Joke Generator

Is your feature request related to a problem? Please describe.
I want to contribute in your repo , Kindly Merge it and Merge   it As #2559

Describe the solution you'd like

Describe alternatives you've considered

Additional context

pls merge it and give hacktober tag

Hey I can make this project in JavaScript Please Assign me this issue

hey can you please assign to me

sir.
can you assign to me

working on it. @vikastripathi707

@vikastripathi707 please review init #4595 and merge my pr


Jest Testing: Cannot read property 'style' of undefined

Fails white testing with jest.
"react": "16.2.0",
"react-native": "0.53.0",
"jest": "22.3.0"
 FAIL  src/components/AutoComplete/AutoCompleteComponent.test.js
  ● Test suite failed to run

    TypeError: Cannot read property 'style' of undefined

      1 | // @flow
    > 2 | import Autocomplete from 'react-native-autocomplete-input';
      3 | import React, { Component } from 'react';
      4 | import {
      5 |   Platform,

      at Object.<anonymous> (node_modules/react-native-autocomplete-input/index.js:186:450)
      at Object.<anonymous> (src/components/AutoComplete/AutoCompleteComponent.js:2:35)
      at Object.<anonymous> (src/components/AutoComplete/AutoCompleteComponent.test.js:5:28)

Did you try the setup for jest described in the readme?

My bad sorry for the trouble..


script to transfer tokens

Link to JIRA: https://objectcomputing.atlassian.net/browse/TE-811
Install
npm i

Transfer Tokens
npm run transfer-token -- <network> <from-account-id> <from-account-private-key> <to-account-id> <token-id> <serial-number>

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


Minor CSS updates and initial FAQ

Made sure that all pages had the down arrow
Fixed governance page blue to match the other pages
Added initial version of FAQ

👉   Need help with
Why does /docs show the full Main Menu in the left nav bar but /blog does not? The latter is the desired state. Screenshots of each attached.

Oh my god that was it. Thank you. ❤


[Request deployment] New Hub: must.af.catalystproject.2i2c.cloud

The GitHub handle of the community representative
@bkmust
Hub important dates

Target Date: Nov 1 - Nov 14, 2024 sprint cycle

Hub Authentication Type
Google suite email (@must.ac.mw is a Google backed email domain)
First Hub Administrators

Bennett Kankuzi<EMAIL_ADDRESS>
[GitHub Auth only] How would you like to manage your users?
None
[GitHub Teams Auth only] Profile restriction based on team membership
No response
Hub logo image URL
https://www.must.ac.mw/wp-content/themes/mustwebsite/images/must-logo.png
Hub logo website URL
https://www.must.ac.mw/
Hub user image GitHub repository
jupyter/scipy
Hub user image tag and name
jupyter/scipy
Extra features you would like to enable

[ ] Dedicated Kubernetes cluster
[ ] Scalable Dask Cluster

(Optional) Preferred cloud provider
None
(Optional) Preferred cloud region
No response
(Optional) Billing and Cloud account
None
Other relevant information to the features above
This deployment is part of the Catalyst project to be deployed on the African cluster.
Anyone with a @must.ac.mw email addresses may have access to the hub.
Tasks to deploy the hub

[x] 1. Deploy information filled in above
[x] 2. Engineer who will deploy the hub is assigned
[x] 3. If using GitHub Orgs/Teams Auth, Engineer is given Owner rights to the org to set this up.
[x] 4. Initial Hub deployment PR
[x] 5. Administrators able to log on -> Hub now in steady-state

Super, thanks for the ping! I will assign @AIDEA775 to this one for the next sprint.

@bkmust the hub was deployed at must.af.catalystproject.2i2c.cloud. Can you test it? Thank you!

HI Alejandro
Yes, its working as in the attached screenshot. Many thanks.
Kind regards
On Tue, Nov 21, 2023 at 11:21 PM Alejandro Ismael Silva <
@.***> wrote:

@bkmust https://github.com/bkmust the hub was deployed at
must.af.catalystproject.2i2c.cloud. Can you test it? Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/2i2c-org/infrastructure/issues/3305#issuecomment-1821709476,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDJWULTT76NFIW5C5XJMZU3YFULNVAVCNFSM6AAAAAA6M3H3A6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRRG4YDSNBXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Bennett Kankuzi, PhD Computer Science (UEF, Finland)
Senior Lecturer in Computer Science & Head of Department
Dept. of Computer Science and Information Technology
Malawi Institute of Technology
Malawi University of Science & Technology
P.O. Box 5196, Limbe, Malawi.
Mobile: +265 884 1888 93
Google Scholar: https://scholar.google.com/scholar?hl=en&q=kankuzi
DLBP Computer Science Bibliography:
http://dblp.uni-trier.de/pers/hd/k/Kankuzi:Bennett
LinkedIn: https://www.linkedin.com/in/bkankuzi
--

Hi Alejandro
I am able to access Jupyter Notebook. How can I also access RStudio?
Kind regards
On Tue, Nov 21, 2023 at 11:53 PM Bennett Kankuzi @.***>
wrote:

HI Alejandro
Yes, its working as in the attached screenshot. Many thanks.
Kind regards
On Tue, Nov 21, 2023 at 11:21 PM Alejandro Ismael Silva <
@.***> wrote:

@bkmust https://github.com/bkmust the hub was deployed at
must.af.catalystproject.2i2c.cloud. Can you test it? Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/2i2c-org/infrastructure/issues/3305#issuecomment-1821709476,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDJWULTT76NFIW5C5XJMZU3YFULNVAVCNFSM6AAAAAA6M3H3A6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRRG4YDSNBXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Bennett Kankuzi, PhD Computer Science (UEF, Finland)
Senior Lecturer in Computer Science & Head of Department
Dept. of Computer Science and Information Technology
Malawi Institute of Technology
Malawi University of Science & Technology
P.O. Box 5196, Limbe, Malawi.
Mobile: +265 884 1888 93
Google Scholar: https://scholar.google.com/scholar?hl=en&q=kankuzi
DLBP Computer Science Bibliography:
http://dblp.uni-trier.de/pers/hd/k/Kankuzi:Bennett
LinkedIn: https://www.linkedin.com/in/bkankuzi

--
Bennett Kankuzi, PhD Computer Science (UEF, Finland)
Senior Lecturer in Computer Science & Head of Department
Dept. of Computer Science and Information Technology
Malawi Institute of Technology
Malawi University of Science & Technology
P.O. Box 5196, Limbe, Malawi.
Mobile: +265 884 1888 93
Google Scholar: https://scholar.google.com/scholar?hl=en&q=kankuzi
DLBP Computer Science Bibliography:
http://dblp.uni-trier.de/pers/hd/k/Kankuzi:Bennett
LinkedIn: https://www.linkedin.com/in/bkankuzi
--

Hi @bkmust, sorry for the delay.
The user image specified in the issue is "jupyter/scipy," which is Python-only. If you need RStudio, we can change the image to "rocker/binder," which has JupyterLab and RStudio.
I already made the change. Let me know if it is okay!

Dear Alejandro
This is much appreciated. I have also accessed RStudio.  So fantastic!
Once again, many thanks.
Kind regards
On Friday, November 24, 2023, Alejandro Ismael Silva <
@.***> wrote:

Hi @bkmust https://github.com/bkmust, sorry for the delay.
The user image specified in the issue is "jupyter/scipy," which is
Python-only. If you need RStudio, we can change the image to
"rocker/binder," which has JupyterLab and RStudio.
I already made the change. Let me know if it is okay!
—
Reply to this email directly, view it on GitHub
https://github.com/2i2c-org/infrastructure/issues/3305#issuecomment-1825991439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDJWULSH5F5YAHKE7SWW4PDYGDTCXAVCNFSM6AAAAAA6M3H3A6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRVHE4TCNBTHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Bennett Kankuzi, PhD Computer Science (UEF, Finland)
Senior Lecturer in Computer Science & Head of Department
Dept. of Computer Science and Information Technology
Malawi Institute of Technology
Malawi University of Science & Technology
P.O. Box 5196, Limbe, Malawi.
Mobile: +265 884 1888 93
Google Scholar: https://scholar.google.com/scholar?hl=en&q=kankuzi
DLBP Computer Science Bibliography:
http://dblp.uni-trier.de/pers/hd/k/Kankuzi:Bennett
LinkedIn: https://www.linkedin.com/in/bkankuzi
--

@bkmust, are you OK with the current experience or do you prefer the server to start with RStudio UI instead of JupyterLab UI? cc @AIDEA775

Its very okay. We start with JupyterLab UI. Many thanks.
On Tuesday, November 28, 2023, Damian Avila @.***>
wrote:

@bkmust https://github.com/bkmust, are you OK with the current
experience or do you prefer the server to start with RStudio UI instead of
JupyterLab UI? cc @AIDEA775 https://github.com/AIDEA775
—
Reply to this email directly, view it on GitHub
https://github.com/2i2c-org/infrastructure/issues/3305#issuecomment-1830358777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDJWULXWAFBNUBTAHNNOQRLYGYOAPAVCNFSM6AAAAAA6M3H3A6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZQGM2TQNZXG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Bennett Kankuzi, PhD Computer Science (UEF, Finland)
Senior Lecturer in Computer Science & Head of Department
Dept. of Computer Science and Information Technology
Malawi Institute of Technology
Malawi University of Science & Technology
P.O. Box 5196, Limbe, Malawi.
Mobile: +265 884 1888 93
Google Scholar: https://scholar.google.com/scholar?hl=en&q=kankuzi
DLBP Computer Science Bibliography:
http://dblp.uni-trier.de/pers/hd/k/Kankuzi:Bennett
LinkedIn: https://www.linkedin.com/in/bkankuzi
--

Closing this one now, any further requests should be tunneled thought support.


Add support for HCN v2 endpoint and add unit tests

switch to HCN v2 endpoint API instead of HNS v1 endpoint API
Support parsing routes in GCS when we setup the network interfaces
[breaking] update gcs bridge LCOW network adapter type with new fields that better align with v2 endpoint
[breaking] REMOVE support for policy based routing and EnableLowMetric in the GCS network config code
Add unit tests for new GCS side changes

The motivation for this change was to support adding custom routes on interfaces. However, the v1 endpoint implementation does not have this support. Therefore, we needed to move to the V2 implementation.

Closing in favor of https://github.com/microsoft/hcsshim/pull/2343


[MLIR] Changes for tuning API v2, MLIR grid layout changes.

This PR constitutes the MIGraphX-side changes needed to not break the build in the presence of
https://github.com/ROCmSoftwarePlatform/rocMLIR/pull/1136 , and updates what data is sent in to MLIR during the kernel generation and tuning process.
Specifically, as of
https://github.com/ROCmSoftwarePlatform/rocMLIR/pull/1145 (which hasn't landed yet but is likely to) we will want to know the number of CUs on the card we're targeting in order to efficiently tile our matrix multiplications. This commit therefore updates the tsv parser (since we'll be adding a new field) and updates the compilation process to pass num_cu into MLIR.
This commit also removes a legacy "xdlopsv2" attribute that is no longer used as we have a more robust architecture feature lookup in MLIR and no longer require it.
We also get rid of a rather annoying "failed to set param" notice when there's no tuning DB loaded so we don't scare everyone with those.

@krzysz00 can you check the failures on the MLIR debug pass?

Enum for tuning space

@pfultz2 Is there anything else?

@krzysz00 Need to fix cppcheck warnings.


alpha-20 has tslib removed from dependencies

I created a repository for this
https://github.com/carlovirtucio/redoc-bug-test

Run create-react-app
Add redoc to dependencies
Import redoc in App.js

Error occurs: Module not found: Can't resolve 'tslib' in '/Users/carlovirtucio/Desktop/Lab/bug-test/node_modules/redoc/bundles'
In package.lock.json, tslib does not seem to be added to redoc's dependencies. When downloading the latest .tar.gz from the Redoc releases and checking its package.json, tslib is not included.
I was able to replicate this while cleaning up my repo by deleting package-lock.json and node_modules then running nom install again for an audit.

I think this was already fixed, but a new release to include it would be great.
https://github.com/Rebilly/ReDoc/commit/8e1a5cb7a186f78584681370ef14209a3e58ded4#diff-b9cfc7f2cdf78a7f4b91a753d10865a2

Just released 2.0.0-alpha.21

Thanks!


Add persistent environment support to lua neovim client

As title. The current approach let user specify which is the desired behavior. By default it use debug tools to store locals across evaluation. This can be improved with other approaches. I think multiple approaches are preferred because each one has its own drawback.

I'm not sure how you achieved this via coroutine because I didn't make a demo code that works... But it would be great if we can add that too.

I just refactor the code a bit and add support to input prompt, but it seems problematic. And I guess this has something to do with repeat.vim because whenever I tried io.read() I got this:
':silent! call repeat#set(",e", 1)\n'

I guess this is an issue. Directly call (extract.prompt "Input required: ") gives me the same result.

@Olical I'm pushing it but did you see this PR?

So the input prompt issue is something I'll solve with some unrelated work soon. Don't worry about that, it's a weird issue caused by hacks I had to do before Neovim had the proper Lua APIs to create mappings into Lua functions. Once I update how I perform a few of the mappings that issue will go away.
I've made small tweak to the output to make it valid Lua in the log buffer, another step on from your "return" lines you were inserting. I see why you had to do it, I'm prefixing results with res = for the same result but no syntax errors.
Seems better than what I had before! A little worried that overriding the io module will break a bunch of filesystem integration but we'll see. Still seems like an improvement! Merging!

Merged to develop, will release soon.

Sorry for my bad indentations, do you have a lint tool or something to fix them automatically?

Yeah, it would be good if we can somehow redirect it. But we have to write some codes at neovim's side. I made a issue for it.


The names of the localities can be found without diacritics

Description:
Localities containing diacritics can be found without using diacritics.

Preconditions:
The website is up and running.
The database contains valid information.

Step 1
Write the name of a localty that contain diacritics without diacritics.

Expected results
The name of the mentioned locality appears in the drop-down written with diacritics.


HTTPCORE-690   -Implement H2StreamMultiplexerFactory fluent builders

@arturobernalg These are internal classes not intended for use outside of the library. Do they really need a builder?

@arturobernalg These are internal classes not intended for use outside of the library. Do they really need a builder?

HI @ok2c probably not. I close the PR.
TY


SNT and ETH. wait 15 minutes -> submit PR, merge, confirm

Transaction: 0xa8cf675d7b316050dcb374e5b380f056d902d183cb3bab526299964d248eac26

eth sent first


Improved Material Conversion

Hoping to resolve #34 and resolve #3
I pass down Rhinos linear workflow to the material converter. This handles gamma correction on the textures for #3.
I filter out objects with no meshes in the sanitize objects method. There's a quirk with the new glTFLoader that doesn't like empty arrays. When the mesh is created the primitive list is empty and throws an exception. It's not logged as a bug but it makes the export fail.
I try to fix most of what I wrote in #34. The gist of it is using the base color and metallic roughness if there's no textures. I also combine the PBR alpha map into the alpha channel of the base color texture. I'm using the texture evaluators just incase the textures aren't all the same size. I also prefer getting out the Color4f from them. Then I can use the Color4f.L (Luminosity) for the metallic/rougness/alpha textures intensity.

dfdaf3b resolves #32

I get this result with a model that uses these textures: color, metallic, roughness, and normal map
glb Export in ModelViewer:

Rhino:

It looks like metallic and roughness are treated correctly, but all the others are inverted in the UV directions.
I've committed the Rhino sample scene I use for testing, along with textures, so we can test on the same model.
https://github.com/Stykka/glTF-Bin/tree/master/sample_scenes/ball
There's also something with the gamma correction going on, but we should probably save that for a separate PR?

There's also something with the gamma correction going on, but we should probably save that for a separate PR?

I was wrong. It seems to behave sort of correct: Rhino seems to ignore the "Treat as Linear" option for base color textures. sRGB base color textures are exported correctly as far as I can tell.


Add lang="typescript" to .svelte files

See here for a template that does it: https://github.com/babichjacob/sapper-typescript-graphql-template

@benmccann could you please describe a reasons to use lang-property?
Because SENT-template by default supports typescript and javascript or I have mistake?

The lang property is necessary to use Typescript in the templates. I thought the templates should use Typescript since this project is supposed to demonstrate how to use Typescript

@benmccann could you make acceptance criteria?

Oh, I just noticed you're not using the official preprocessor. I had assumed you would be
With the latest version you can set the language just once in the config instead of setting it in each template. https://github.com/sveltejs/svelte-preprocess/blob/master/docs/getting-started.md#31-setting-default-languages
But anyway this would only makes sense if using the official preprocessor


Remove SecurityMonkey

SecurityMonkey is in maintenance mode and will be EOL in 2020. It shouldn't be recommended.

I don't think this list recommends any of these tools.  It's just listing them.  If you want to get rid of, or maybe mark as deprecated, tools that no longer are maintained, there are some others:

Krampus (is based on Security Monkey)
probably half of the S3 bucket finding tools, if nothing else these are just duplicating each other
cloudjack and cloudfrunt as the issue they work with has been fixed
weirdAAL (questionable as to if it is maintained still)

I think it would be useful to at least mark them as deprecated.

Or maybe move to a different section?

Thanks for your suggestion @philipmeadows, I agree with @0xdabbad00, this list doesn't recommend any tool but it definitely would help others to say that is deprecated. There are other deprecated tools as well but I'm not 100% sure of that so I think the best way to list them is probably adding some metadata like: stars, last commit date, contributors, open issues, etc. Just to show how active each project is. WDYT?

That sounds like a good longer term plan, maybe just add [DEPRECATED] (or something like that) to the ones that we know are deprecated would be a good start?

Changed title for obvious reasons, not removing it.


Security login in Firefox does not work

This issue is imported from pivotal
When trying to perform a Fido/webauthn login with Firefox on Mac on test2, the login does not work. On the page \'Sign in with security key\', when clicking on the button \'Log in with a security key\', nothing happens, see screenshot. Firefox does not prompt me for a key.. This works OK in Safari and Chrome.

(Peter Clijsters - Jul 11, 2023)

This is a firefox-on-apple issue. Only hardware keys can be used (and work fine)
(Peter Havekes - Jul 11, 2023)

https://webauthn.me/browser-support (Peter Havekes - Jul 11, 2023)


Add Ricardian contracts to eosio.amend

Adding Ricardian clauses for actions and contract.

Wrong repo I assume?

Yeah, very sorry about that. Doing too many things, not enough sleep. Please delete.
Thanks for posting those examples of Ricardians under CDT. There are no references.
Douglas
From: arhag<EMAIL_ADDRESS>Reply-To: "EOSIO/eosio.contracts"<EMAIL_ADDRESS>Date: Monday, December 10, 2018 at 2:19 PM
To: "EOSIO/eosio.contracts"<EMAIL_ADDRESS>Cc: Douglas Horn<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [EOSIO/eosio.contracts] Add Ricardian contracts to eosio.amend (#147)
Wrong repo I assume?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Improve the performance of updateExportedFilesMapFromCache

https://github.com/microsoft/TypeScript/pull/45238 mitigates performance problems in updateExportedFilesMapFromCache but doesn't address the underlying problems.

Why is updateExportedFilesMapFromCache called so many times during the course of building a single projects?
If not making any updates is the common case, would another representation be more appropriate (e.g. storing a delta, rather than a copy)?
If we continue to update the "real" map from a full cache, can the update share code with clone?  (The PR doesn't because clone appears to make a shallow copy which (a) seems wrong and (b) wasn't what the code being replaced did.)

clone appears to make a shallow copy

This was intentional.. After state is formed, the referenceMap (the way it was normal map before) can only be set with New set of values.. doesnt modify existing values..

Fixed by https://github.com/microsoft/TypeScript/pull/48579


[NUXT3] Deploy to Vercel, Failed: [nuxt] [request error] [unhandled] [500] ENOENT: no such file or directory, open '/vercel/path0/.nuxt/i18n/config-cfcb45fa.js'

Environment
"devDependencies": {
"@nuxtjs/i18n": "^8.0.0-beta.11",
"nuxt": "3.3.3"
},
Reproduction
"devDependencies": {
"@nuxtjs/i18n": "^8.0.0-beta.11",
"nuxt": "3.3.3"
},
nuxt.config.ts
modules: ['@vueuse/nuxt', '@unocss/nuxt', '@nuxt/content', '@nuxtjs/i18n'],
i18n: {
vueI18n: './i18n.config.ts',
},
i18n.config.ts:
export default defineI18nConfig(nuxt => ({
legacy: false,
locale: 'en',
messages: {
...
}}))
vercel
Describe the bug
[nuxt] [request error] [unhandled] [500] ENOENT: no such file or directory, open '/vercel/path0/.nuxt/i18n/config-cfcb45fa.js'
Additional context
No response
Logs
No response

I have the same error when deploying on AWS Lambda:
[nuxt] [request error] [unhandled] [500] ENOENT: no such file or directory, open '/Users/laurensdhondt/Sites/xxx/xxx/.nuxt/i18n/config-0b4859c1.js'
Note how it refers to a path on my local machine.

Same here - deployments on Vercel fail right now:
[nuxt] [request error] [unhandled] [500] ENOENT: no such file or directory, open '/vercel/path0/.nuxt/i18n/config-cfcb45fa.js'

and
[nuxt] [request error] [unhandled] [500] ENOENT: no such file or directory, open '/vercel/path0/.nuxt/i18n/config-cfcb45fa.js'

[nuxt] [request error] [unhandled] [500]  (500  (/__i18n__/precompile))
  at async precompileMessages (./chunks/app/server.mjs:3122:29)
  at async i18n_options (./chunks/app/_nuxt/i18n.options-9b43ac01.mjs:33:23)
  at async vueI18nConfigLoader (./chunks/app/server.mjs:2765:81)
  at async resolveNuxtI18nOptions (./chunks/app/server.mjs:2767:19)
  at async Object.setup (./chunks/app/server.mjs:3397:116)
  at async Object.callAsync (./node_modules/unctx/dist/index.mjs:72:16)
  at async applyPlugin (./chunks/app/server.mjs:135:33)
  at async applyPlugins (./chunks/app/server.mjs:144:5)
  at async createNuxtAppServer (./chunks/app/server.mjs:4377:7)
  at async Object.renderToString (./node_modules/vue-bundle-renderer/dist/runtime.mjs:176:19)
[nuxt] [request error] [unhandled] [500] ENOENT: no such file or directory, open '/vercel/path0/.nuxt/i18n/config-cfcb45fa.js'

I had the same problem.
The secret is very simple. It was the file system permissions to write files to .nuxt directory, because the file .nuxt/i18n/config-cfcb45fa.js was created after running node .output/server/index.mjs.
And simply fixed by adding file write permissions to the .nuxt directory for the node process.

I had the same problem.
The secret is very simple. It was the file system permissions to write files to .nuxt directory, because the file .nuxt/i18n/config-cfcb45fa.js was created after running node .output/server/index.mjs.
And simply fixed by adding file write permissions to the .nuxt directory for the node process.

You mean every time you build, change  permission per files manually on a server? If you are using vercel like cd/ci deployment, it cannot be possible. What is the root issue?

I've fixed this issue.
Please check edge channel 8.0.0-beta.11-28059008.d1499b6 version.
Thanks!

I've tried to deploy the following repository with vercel and netlify.
I could not reproduce this issue.
https://github.com/kazupon/nuxt-i18n-hosting-repro-1
vercel: https://nuxt-i18n-repro-1.vercel.app/
netlify: https://harmonious-treacle-7e0aaa.netlify.app/
Could you give us your minimal repo please? 🙏

@kazupon I think this is related with a reproducible repo https://github.com/nuxt-modules/i18n/issues/2089


Add ResourceExtensions.HasAnnotation methods

Description
Adds two new methods:

ResourceExtensions.HasAnnoationOfType<T>
ResourceExtensions.HasAnnotationIncludingAncestorsOfType<T>

Also modify TryGetAnnotationOfType and TryGetAnnotationsIncludingAncestorsOfType to reduce allocations and remove stack-based recursion.
Expand test coverage.
Some XML doc tweaks.
Would be helpful in #6209, where TryGet* is used and the out param discarded.
Checklist

Is this feature complete?

[x] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[x] Yes
[ ] No

Did you add public API?

[x] Yes

If yes, did you have an API Review for it?

[ ] Yes
[x] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[x] Yes
[ ] No

[ ] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

This is post 9.0? It doesn't seem like an important bug fix.

The idea was that the API would be available for @adamint to consume in #6209, but that's merged now anyway.


Add parallelization for processing policies / authorities.

Signed-off-by: Ville Aikas<EMAIL_ADDRESS>
Summary
Since processing policies / attestations can be semi time consuming and there can be
multiple CIP that matches a single image, parallelize them.
Ticket Link

Fixes
Release Note

Codecov Report

Merging #1795 (1635269) into main (8cac645) will increase coverage by 0.29%.
The diff coverage is 80.95%.

@@            Coverage Diff             @@
##             main    #1795      +/-   ##
==========================================
+ Coverage   32.16%   32.46%   +0.29%
==========================================
  Files         146      146
  Lines        9208     9262      +54
==========================================
+ Hits         2962     3007      +45
- Misses       5898     5904       +6
- Partials      348      351       +3

Impacted Files
Coverage Δ

pkg/cosign/kubernetes/webhook/validator.go
76.14% <80.95%> (+1.70%)
:arrow_up:

pkg/cosign/tuf/client.go
61.68% <0.00%> (-0.82%)
:arrow_down:

Continue to review full report at Codecov.

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


Updating the Gradle examples

Updating the examples to make use of the updated Thorntail plugin for Gradle.
Please merge this PR only after the base PR has been merged.

Base PR for Thorntail: https://github.com/thorntail/thorntail/pull/1149

@sberyozkin
All the examples are now under a folder called gradle-examples. Inside this folder there's a gradle folder, which is required for the wrapper to work. I guess I didn't want to confuse myself about which gradle folder I was in.
Please go ahead and merge this PR. I'll need to raise another PR for the Arquillian adapter work anyways. I would like your inputs on what type of examples do we want to share with the broader community. Right now these are super simple ones. I can address them as part of the best PR.