Fix Static emoji autosuggestion issues

@stitesExpensify
Details
Fix some issue missed during https://github.com/Expensify/App/pull/14686
Fixed Issues

Fix scoping issue : https://github.com/Expensify/App/pull/14686#issuecomment-1464609834
Fix invalid regular expression : https://github.com/Expensify/App/pull/14686#discussion_r1132972462

Tests

Test 1 :

Open any chat
Write a long chat untill the scrollbar is visible
Try to scroll

Test 2:

Open any chat
Type this message :+1:
Verify that auto suggestion menu appear with thumb emoji and the app doesn’t crash

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

Offline tests

Same Tests
QA Steps

Same 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 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 correct English and 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 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://user-images.githubusercontent.com/36869046/224452935-a3cb6ce7-1a6f-4226-8473-ccbff4e4c6a8.mov
https://user-images.githubusercontent.com/36869046/224452945-0ae4e630-c8b7-416f-978e-f753f056f3a0.mov

Mobile Web - Chrome
https://user-images.githubusercontent.com/36869046/224454414-cbad7c92-56b3-4750-a233-19664ba30991.mov

Mobile Web - Safari
https://user-images.githubusercontent.com/36869046/224453976-2bd43f8a-e276-41a3-b2d3-39470c15f15b.mp4

Desktop
https://user-images.githubusercontent.com/36869046/224454129-d8a2e7a9-6000-4e00-bb16-c5119230052c.mov

iOS
https://user-images.githubusercontent.com/36869046/224453798-a5b14d77-a79b-472a-a430-55ffe45f52a5.mp4

Android

@andrewgable Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

I am getting warning for one case (that we didn't catch earlier). ReportUtils.canShowReportRecipientLocalTime is returning undefined, and hence, shouldIncludeReportRecipientLocalTimeHeight prop for EmojiSuggestions is passed as undefined.
I think we should fix it in this PR @stitesExpensify @fedirjh? We can just cast the return value to Boolean, what do you think?
https://github.com/Expensify/App/blob/acc4984ce41d21e0f89c1913053232a567598d8c/src/libs/ReportUtils.js#L419-L425

https://user-images.githubusercontent.com/3069065/224541958-f46007cf-545d-4ebf-b57e-462467505154.mov

Reviewer Checklist

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

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

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

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

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and 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 verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

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

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

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

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

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

[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 reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://user-images.githubusercontent.com/3069065/224544596-cd43b1e7-98f1-4ccd-9dc5-997be1d40ae6.mov

Mobile Web - Chrome
https://user-images.githubusercontent.com/3069065/224544676-1180bdf0-db4c-49e6-8a7a-aba86191f5f7.mov

Mobile Web - Safari
https://user-images.githubusercontent.com/3069065/224544704-a94d2e60-d804-4dfc-93a3-80db851feba7.mov

Desktop
https://user-images.githubusercontent.com/3069065/224545098-6a8abace-cb18-4cf0-8197-00334986cdc1.mov

iOS
https://user-images.githubusercontent.com/3069065/224545105-3d7a6094-a4a2-4e66-b3d6-81ac451cfd6f.mov

Android
https://user-images.githubusercontent.com/3069065/224545115-ffd862ab-f812-43e0-9a1f-c72044a74205.mov

The current changes work well, but I raised another warning and checking if this can be a part of this PR.

Thanks for the comment @fedirjh.

I also think we should fix it here , we can just add a default prop value

I think instead of default prop we should retain the isRequired behaviour and pass the correct value.

For newly created accounts , the preferredSkinToneIndex has value default however the prop type is number.

I think this was fixed from the backend but @stitesExpensify can confirm.

For newly created accounts , the preferredSkinToneIndex has value default however the prop type is number.

I will fix it on the back end. Looks like we didn't update the API layer setting the default in onyx

LGMT and tests well. Gonna merge to fix the bugs.

🚀 Cherry-picked to staging by https://github.com/luacmartins in version: 1.2.83-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/roryabraham in version: 1.2.83-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@fedirjh issue is here, can you comment so that I can assign you please? https://github.com/Expensify/App/issues/15968

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Does current selenium2lib support selenium grid 2?

Any one has tried this?
Thanks!
Adam

s

How to control robot java class on remote machine (selenium-grid). Please advice to resolve this issue.


Context initialization failed error on jetty server startup post upgrade from Rhythmyx 7.3.2 to 8.1.1

server.log

Please find attached the server.log for errors

should be resolved


Consider Spark with Java

Hello,
Please consider adding “Spark with Java” to your awesome list.

Spark with Java - New book on Spark in Manning's Early Access Program (MEAP) with a very strong focus on Java, targeted towards software and data engineers. The author focuses on developing existing skills rather than learning Spark, Scala, and Hadoop at the same time, making Big Data easier to learn. You can find the accompanying GitHub repositories here.

Thanks!

Added as requested


fix: config in UI form values

Description

This PR attempts to clean up some tech debt related to the UI integration forms. It is a follow up to issues uncovered during #12277.

Removal of the spread operator when assigning the default form values. The current merge of default and passed through initial values doesn't make sense for integrations with a nested structure (the nested structure arises for integrations with credentials from an additional updatePassword field next to the integration payload). Also, the spread operator performs shallow copies, which may corrupt the original default values after subsequent assignments.
The current UI makes a couple of assumptions about the HTTP verb and the request structure which are not part of the API contract and no longer valid for all integrations. Some of the newer integration APIs (cloud sources, machine-to-machine auth) follow the new StackRox API guidelines, which recommend introducing a separate request proto message for all request types. Additionally, there is a mix between patch and put operations for integration edits. This PR leaves the old heuristic via the updatePassword field in place where it works, and makes the cases where it doesn't work more clear. I don't think this is perfect and the API assumptions should be removed completely, but it is already an improvement.

User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[x] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

CI and tested that creation and edit of integration works.

@stehessel
Question for visibility to UI team mates, not change request for this contribution:
Does your recent reorganization of integration forms into folders according to types imply:

it is easier for you as a frequent contributor
and therefore worthwhile to continue for the rest of the integration types early in 4.7 release

Does your recent reorganization of integration forms into folders according to types imply:

it is easier for you as a frequent contributor
and therefore worthwhile to continue for the rest of the integration types early in 4.7 release

For me yes, I prefer the separation into integration types and think it's easier to navigate. Otherwise I think it's easy to get lost in the files because there are so many different integration forms.


this phrasing is misleading

QnA Maker	In the QnA Maker portal, for the knowledge base, in the Manage settings, use the key value shows in the Postman settings for the Authorization header, without the text of EndpointKey.
it is not in the manage settings of the qna maker portal?  it is in the azure portal for the service.

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

ID: f68e3448-ea66-7aab-26d2-37de5f44527c
Version Independent ID: 30c9096c-926c-3b92-6ab0-cc49edf63842
Content: Use multiple LUIS and QnA models - Bot Service
Content Source: articles/v4sdk/bot-builder-tutorial-dispatch.md
Service: bot-service
GitHub Login: @diberry
Microsoft Alias: diberry

@xtianus79 - Thank you for creating this issue - can you be more specific with what you think is incorrect? #in-progress

@diberry Hi Dina :) Yes, the using of the phrase endpoint key is something it actually wants from the azure service as opposed to the authoring endpoint key that is given in the Azure portal.  That is what hung me up for a while.  Specifically it is where one should obtain the key that would fix all understanding from my perspective.

@xtianus79 Thank you for clarifying. The better explanation is in the QnA Maker docs - I'll link to that. https://docs.microsoft.com/en-us/azure/cognitive-services/qnamaker/how-to/set-up-qnamaker-service-azure#types-of-keys-in-qna-maker.

Fixed - #please-close

Can we get a recommendation on how to build a script file with CLI commands? I tried to build the regular file.bat and run it in command prompt, but it would stop executing on first dispatch command.

@dkonyayev which operating system and flavor of command prompt are you using?

@diberry I was able to resolve this. Added "call" command before "dispatch"

This second issue seems to be resolved. #please-close

@emgrol please-close


Regressions introduced by pull requests

@kylewm:
.....................F.F........
======================================================================
FAIL: test_parser.test_embedded_parsing
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/tom/.virtualenvs/mf2py/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/Users/tom/code/mf2py/test/test_parser.py", line 198, in test_embedded_parsing
    '\n
Blah blah blah blah blah.
\n
Blah.
\n
Blah blah blah.
\n')
AssertionError: u'\n
Blah blah blah blah blah.
\n
Blah.
\n
Blah blah blah.
\n  ' != '\n
Blah blah blah blah blah.
\n
Blah.
\n
Blah blah blah.
\n'

======================================================================
FAIL: test_parser.test_hoisting_nested_hcard
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/tom/.virtualenvs/mf2py/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/Users/tom/code/mf2py/test/test_parser.py", line 242, in test_hoisting_nested_hcard
    assert_equal([u'KP\nKP1'], result['items'][0]['properties']['name'])
AssertionError: Lists differ: [u'KP\nKP1'] != [u'KP\n    KP1']

First differing element 0:
KP
KP1
KP
    KP1

- [u'KP\nKP1']
+ [u'KP\n    KP1']
?        ++++

----------------------------------------------------------------------
Ran 32 tests in 0.654s

FAILED (failures=2)

The KP\nKP1 issue seems to have been introduced in March 2014 (!!) 32556ddeb713446a76ff98b3d3e144e8c063ba2f and I think the blah blah test goes back even farther. I'm guessing it depends on which backing html parser BS4 is using.
We should probably figure out what the output SHOULD be... It's not clear to me

Turns out it was a difference between html5lib (the default for mf2py) and lxml (what I had installed in my venv for some reason). Sent a pull request that puts the whitespace back how it was before. Sorry!


Correction of music in examples

Subject
Fix the popover in the application bar section of music so that only one popover is activated
Description
Fix the popover in the application bar section of music so that only one popover is activated.
https://github.com/user-attachments/assets/76f509ff-c302-45b5-b71e-31e2ce8146f4
Are you willing to participate in creating this documentation page and create a pull request?
None

Hi @illionillion I would like to try working on this issue, is there any guidelines on the documentation i should follow?

Hi @illionillion I would like to try working on this issue, is there any guidelines on the documentation i should follow?

Hi @mmmmaximus ,
Good luck!


Needs Kubernetes equivalent

Create the commands and files needed to run through these examples of updating an app with zero downtime testing on Kubernetes.

Do you need one line commands and yaml resources for Kubernetes examples? I am glad to create a PR for that

yes to both 👍


No response

I can't get this to work on Android or iOS. I can see that the M-SEARCH request is being made by using Wireshark but I never get an OK from any device. I receive a response with a list of devices but I believe these are just the devices that send out NOTIFY. I'm using the code in the example.

Hi, 'a list of devices' is the expected behaviour. This is only a (SSDP) discovery plugin.
Once you have the list of devices, you can communicative over the DLNA HTTP API to query, control etc


Add ability to generate the RCC-5 DIFF report

The listtools scripts are now able to generate RCC-5 reports that summarise the difference between two checklists, or subsets of checklists.
We'd like to introduce this as an additional DIFF tool allowing users to run a report by selecting the two source datasets, root taxa and then downloading the CSV output (i.e. similar to other diff tools).
This will require an API to take the configuration from the front end, run the linux process, and present the output file. It would be nice to be able to access previous runs, so perhaps the directory structure might store the input and output configurations within a directory per user, or similar.

I would probably integrate the request submission into the DatasetDiffResource similar to how a MatchingJob is submitted in NameUsageMatchingResource. And a new BackgroundJob subclass that does all the work.

@beckettws and I are together and had a call with Nico and Jonathan @jar398, who has a new output report.
We (I) believe this would be useful for the comparison of changes within chosen sectors between monthly releases.
@mdoering and I should find time to get this API and UI integrated in 2023 and separately, we propose to find time to present the report to the xcol data managers.
CC @sternerlab

Just merged my 'gbif' branch to main at https://github.com/jar398/listtools , as this is the code that produced the spreadsheet Beckett showed this morning.  Up next for me: make it easy for you to run it on some CoL example.

On a related note this may be of interest if it's not on people's radar https://github.com/gnames/gndiff.

I fixed a few bugs and updated the documentation.  It could certainly bear some more testing (have only done COL mammals) but what's in the listtools repo is a "release candidate" in the sense that I am not aware of any reasons not to have others try it.
My test is Mammalia in COL 2019 (COL19) vs. COL 2023 (COL23.1). Output for this comparison is in the doc/ directory of the repo.
Let me know what you think.

integrated into the UI for some time


signinfo support on addSignal handler

pcntl_signal support second parameter for callback (siginfo).
It's is a very interesting feature that is needed when developing multi-threaded daemons on PHP.

Hey @kikimor thank you for the PR. (Sorry for the late reply going through all my and react's open PR's and issues.) A few things, first of we always want new features tested. But first I would love to know what the use case is behind this feature. And thirdly, this only covers the stream_select loop but not the other loops we have. Could you look into those as well?


Add demo application to showcase all OpenAPI Pages that has implemented.

Add demo application to showcase all OpenAPI Pages that has implemented.

Where can we deploy this one?

Deta
Railway
Render


Verify unlogged files

Files too large to track at upload are off-loaded and therefore unlogged.
Detect these at the Echo side and add to logs.

non-issue


cast to Fragment Activity

in my project , run
FlutterLocker.save(SaveSecretRequest("passCode", "passCodeSecret", AndroidPrompt("Authenticate", "Cancel")));
but get exception under error
Unhandled Exception: PlatformException(error, com.bibtel.MainActivity cannot be cast to androidx.fragment.app.FragmentActivity, null, java.lang.ClassCastException: com.bibtel.MainActivity cannot be cast to androidx.fragment.app.FragmentActivity
how fix it.
me noob in the flutter.

Go to /android/app/src/main/kotlin/com/**/MainActivity and change:
import io.flutter.embedding.android.FlutterActivity to import io.flutter.embedding.android.FlutterFragmentActivity
and FlutterActivity to  FlutterFragmentActivity

Could this be added to the main source files?


Decision point: sheer to django transition

Hi everyone!
it's looking like the soonest we'll be able to switch beta over to Django will be mid-august. I think that gives us a few different options for how to proceed.
a) Wait until mid-august (or whenever) to start using Django for local development and the flapjack/refresh demo servers.
b) Make the switch soon, and freeze beta until it can be switched to the new stack
c) Make the switch soon, but take care to not break compatibility with Sheer (so, not taking advantage of the new abilities Django gives us, like more server side logic).
"c" seems like a bad idea to me, but I didn't want to take it off the table.

I vote for a, mostly because Kurt is on vacation next week and I'm gone the following week. Sprint 34 should start Aug 12 and would probably be perfect.

@jimmynotjim makes a good point, but I'm leaning b.  The current Django setup is pretty solid, and I don't think you guys missing some time will necessarily derail the switchover.

That works too, just wasn't sure what work was going to be necessary from the rest of us :+1:


Add a ConcurrencyPerPartition option to KafkaSource

Fixes #320
Proposed Changes

Add a spec.concurrencyPerPartition field to the KafkaSource CRD.

Copied from the 1st commit in this PR:
This field, if set, will limit that maximum number of concurrent sends (ie POSTS) of CloudEvent messages at a time for each partition. If left unset or set to 0, which is the default, the concurrency is unbounded.
When processing a large number of messages from a Kafka topic, the unbounded behavior leads to massive numbers of goroutines being created. This leads to problems with too many open sockets attempting to all POST to the sink at the same time.
So, in practice, we may want to actually default this to a reasonably high value that allows for a high throughput per topic partition without blowing up if processing a huge backlog of messages on a topic.
Note that, as the name implies, this bounds concurrency per partition. If a topic has many partitions and only a single receive adapter pod running (the default today) then you'll have up to ConcurrencyPerPartition * numPartitions concurrent HTTP POSTS to the sink.

I did not add a release note for this. But, if KafkaSource makes it into 0.5 and this doesn't make it until 0.6 we'll want to add a release note.
We should also discuss defaulting this to some reasonable value instead of being unbounded. I left it unbounded to match the current behavior.

cc @dubee @matzew

/assign matzew

/hold
After discussing this over Slack, @markfisher pointed out that there really shouldn't be a goroutine inside ConsumeClaim at all here. And that the knob to control consumer concurrency should be purely driven by the number of partitions in the Kafka topic, as is the usual Kafka consumer way. The goroutine inside ConsumeClaim, both with or without this PR, presents other problems as-written as far as committing of offsets and acking of events - the code assumes each offsets can be committed individually when in reality once you commit a specific offset it commits all messages with offsets less than or equal to that offset.
So, I'll take Mark's changes into account, removing the goroutine entirely, and see if that fixes the stability issues under load of the receive adapter.

FWIW, I've filed a pr here:
https://github.com/knative/eventing-sources/pull/322
feel free to close that, if you do update this :)

Closing - this is the wrong solution


Name of the account into txt

Is it possible to add the @name of the account that published the picture into the .txt?
It would be great to have that information to follow up with the person that posted the content.
Thanks.

--post-metadata-txt={owner_username}
It is explained in https://instaloader.github.io/basic-usage.html#metadata-text-files


Converting CamelCase to C style, adding _t for enums and messages

The Protocol Buffers guidlines recommend using CamelCase for Enums, Messages and Services.
The current version of nanopb does not convert them to C style, but rather leaves some strange looking C code like:
typedef struct _namespace_SomeMessage {
   namespace_Type some_value;
} namespace_SomeMessage;

This commit fixes this issue. It also uppercases the #define names for intt_defaults, tags, max size, and others.
The generated code look now like that:
typedef enum
{
    NAMESPACE_TYPE_FIRST = 0,
    NAMESPACE_TYPE_SECOND = 1
} namespace_type_t;
#define NAMESPACE_TYPE_MIN NAMESPACE_TYPE_FIRST
#define NAMESPACE_TYPE_MAX  NAMESPACE_TYPE_SECOND
#define NAMESPACE_TYPE_ARRAYSIZE ((namespace_type_t)(NAMESPACE_TYPE_SECOND+1))

typedef struct {
  namespace_type_t some_value;
/* @@protoc_insertion_point(struct:namespace_some_message_t) */
} namespace_some_message_t;

// etc.
#define NAMESPACE_MESSAGE_SIZE                            4

I've tested it with messages, enums, oneofs, extensions, required and optional, repeated, max_length and max_size defined in an options file, default values (numeric or enums). The C na H files compile every time without problems with pb.h version 30.
Example proto file:
package namespace;

enum Type {
  FIRST = 0;
  SECOND = 1;
}
message SomeMessage {
  required Type some_value = 1;
}

Note: The commit also removes the _enum_name and _struct_name from enums and structs definitions, but to add it again you can just uncomment it.
The commit also includes the hardcoded _t for enums and structs names.
This commit will break compatibility with previously generated code as type names and defines will change.
Also, I didn't try to build an .exe file from it.
Changes made by me and @hakonfam due to #199.

One sec, I'll have a look at the bug.

Way nicer. Is that ok to break code compiled before your patch ?

This should probably be an option at first, otherwise existing code will break.
Also, just a heads up, but this does not handle well acronyms: "GetHTTPMessage" for instance gets transformed to "get_h_t_t_p_message" instead of "get_http_message".

Yes... And also I've just noticed that when the enum value in not written in capitals the generated name in small letters.
I need to spend more time on this one...

Ok, now I see there is a bug in reading the options:
# Generate all fields as callbacks.
AllTypes.*              type:FT_CALLBACK
SubMessage.substuff1    max_size:16
AllTypes.oneof          no_unions:true

All fields were changed to pb_callback_t, not only those in AllTypes. But at least the first alltypes test passed :)
I will continue on Monday.

It works when I change line 1457 to
file_options = nanopb_pb2.NanoPBOptions()#get_nanopb_suboptions(fdesc, toplevel_options, Names([filename]))
I'm not sure why but it read initial global options as type: FT_CALLBACK and applies type CALLBACK to SubMessage.substuff1, instead of STATIC.

Hmm, yeah, this should definitely be enabled with a separate option, and I think it should be off by default. I think it makes most sense to make it a .proto option, e.g. convert_camelcase = true.
Other than that, take your time :) I'll take a closer look once you are satisfied yourself, and I can of course help if you get stuck with some problem.

I think the C style should be always enabled. The only thing that could be customizable is whether to add _t to enums and structs names, as POSIX doesn't have it. But using mixed Camel Case and C style in C looks very odd...

Oh course there is a problem with applications that already use nanopb :) This is an argument to make it disabled by default.

No, the name style conversion cannot be enabled by default. It would break too much existing code for little benefit.. yeah.

Could it be enabled by default for proto3 ?

Possibly.. or even nanopb-0.4.0. Perhaps it is not such a burden to add one option for legacy code. But no breaking changes before 0.4.0, so it makes sense to keep it disabled at first.

Hi, I've looked at the options in nanopb, trying to add one as you wrote. It looks like currently you have only per-field options. The new one (or two) should be kind of a global one? So I tried to add new message with global options to the nanopb.proto file, compile it.. but then things started to get very complicated with parsing. My Python experience is around 4 days long, including weekend, so I must give up :(
We have now a version of the script that works for our need, but we hope someone, if not you, will continue with C-styling the generated files. I can't spend more time on it right now.

Looking back, why would you add the POSIX-reserved _t suffix to the identifier of every type? This is just being non-conforming for the sake of being non-conforming.

Hehe, yes. But this is actually a requirement in our C code standards. As I wrote, this should be also an option, default false. Of course you can just remove it completely, but it would be perfect for Nordic if such option could be enabled somehow without creating a private repo.

This should probably be in a "suffix" option, imo. I've seen people use weird suffixes like _s as coding standard, so it should probably be configurable.

Yeah, perhaps it would make sense to have this more widely configurable. Something like:
string MessageNameTemplate = "msg_%s_t";
CaseType MessageNameCase = CamelCase;
etc.

Considering that the current pull-request is non-workable (breaks existing code and tests), I've moved this into feature request #217.


[DBZ-336: Do not mark offsets for commit log files with error.

Do not mark offsets for commit log files with error and need to be re-processed.

@gunnarmorling Merging this PR as well since it's just some nit change :)

Hey @bingqinzhou, just seeing this, sorry for the super-long delay. Ideally, there isn't any merge commit, for the sake of a linear, and thus easy to follow history. For that, just use "Rebase and Merge" instead of "Merge" in the GH UI.


Support method references in the cucumber-java8 to cucumber-java migration

🤔 What's the problem you're trying to solve?
When migrating from cucumber-java8 to cucumber-java, method references are not supported.
✨ What's your proposed solution?
Support method references in the cucumber-java8 to cucumber-java migration recipes.
⛏ Have you considered any alternatives or workarounds?
Replace the method references with lambda's before conversion, through an inverse of Use method references in lambda.
📚 Any additional context?
Likely more important only if we do indeed deprecate cucumber-java8.

This text was originally generated from a template, then edited by hand. You can modify the template here.

Continued here https://github.com/openrewrite/rewrite-cucumber-jvm/issues/3


kernel_shape is wrong

in kernel shape, dim of group_num is missing .
I correct it as https://github.com/fangwudi/Group-Depthwise-Conv3D-for-Keras

Can you make this into a merge request so I can have a look?
Best,


Set up git-secrets/maven integration

The intention of this PR is to help keep passwords and keys from ending up in the source tree using git-secrets. git-secrets itself must be installed and available in the path for this to work. This helps avoid the need to manage whether the git hooks have been installed, and whether the correct providers are set up.
A .gitallowed file is added that will allow all the matches discovered while hooking this up.
Once this is merged, developers will have to have install git-secrets to the their path, but this PR will manage setting it up for the repo.
You can try it out by running mvn initialize. Then try adding a credentials file to a commit.
[default]
aws_access_key_id = AKIAZHZQVUBBBBBBBBBB
aws_secret_access_key = 5AHiqowZsCCCCCCCCCCBBBBBBBBBBCCCCCCCCCC0

You should see an error like:
credentials:2:aws_access_key_id = AKIAZHZQVUBBBBBBBBBB
credentials:3:aws_secret_access_key = 5AHiqowZsCCCCCCCCCCBBBBBBBBBBCCCCCCCCCC0

[ERROR] Matched one or more prohibited patterns

Possible mitigations:
- Mark false positives as allowed using: git config --add secrets.allowed ...
- Mark false positives as allowed by adding regular expressions to .gitallowed at repository's root directory
- List your configured patterns: git config --get-all secrets.patterns
- List your configured allowed patterns: git config --get-all secrets.allowed
- List your configured allowed patterns in .gitallowed at repository's root directory

And your changes will not have been committed. The suggested method is to use .gitallowed if you are confident it is a false positive for a file that has a password-like string in it.
In a future PR we could consider adding the scan to a CI stage. https://ucsc-cgl.atlassian.net/browse/SEAB-1531

Need to add git-secrets to CI now, too, though it won't perform a scan (yet)

Sorry @denis-yuen I know you already approved I accidentally clicked it

Will need to follow up by updating https://wiki.oicr.on.ca/display/DOC/Git+Secrets
Also, build success

Should also add a mention in the local set up primer.


inference error: KeyError: 'audio_model.model.encoder.pos_conv_embed.conv.parametrizations.weight.original0'

hi, when I run the inference code, raised following error:
Traceback (most recent call last):
File "/home/jovyan/wangzairan/projects/DEEPTalk/DEEPTalk/demo.py", line 240, in
main(args, DEMOTE_config)
File "/home/jovyan/wangzairan/projects/DEEPTalk/DEEPTalk/demo.py", line 123, in main
original_weights = DEMOTE_ckpt[name]
KeyError: 'audio_model.model.encoder.pos_conv_embed.conv.parametrizations.weight.original0'
how can I solve this problem

Hi wzr1201,
We updated the inference and added the rendering code.
If the issue persists, leave an additional comment! Thanks for the interest!

Hi wzr1201, We updated the inference and added the rendering code. If the issue persists, leave an additional comment! Thanks for the interest!

Hi there, I downloaded the code two days ago, and got the same error.


setup not working?

I got client token from the server and passed it to BTClient.setup(token)
but I got this error:
I'm using android
Could not invoke Braintree.setup
null
No static method newInstance(Landroid/app/Activity;Ljava/lang/String;)Lcom/braintreepayments/api/BraintreeFragment;
in class Lcom/braintreepayments/api/BraintreeFragment; or its super classes
(declaration of 'com.braintreepayments.api.BraintreeFragment' appears in /data/app/com....

Same error here, any fix?


Cannot run the action since 1.6.2 release: action.tar.gz not found

Since v1.6.2 was released, we are not able to run this action, failing on docker build:
  #0 building with "default" instance using docker driver

  #1 [internal] load .dockerignore
  #1 transferring context: 2B done
  #1 DONE 0.0s

  #2 [internal] load build definition from Dockerfile
  #2 transferring dockerfile: 607B done
  #2 DONE 0.0s

  #3 [auth] library/alpine:pull token for registry-1.docker.io
  #3 DONE 0.0s

  #4 [internal] load metadata for docker.io/library/alpine:3.17.2
  #4 DONE 0.5s

  #5 [1/3] FROM docker.io/library/alpine:3.17.2@sha256:ff6bdca1701f3a8a67e328815ff2346b0e4067d32ec36b7992c1fdc001dc8517
  #5 resolve docker.io/library/alpine:3.17.2@sha256:ff6bdca1701f3a8a67e328815ff2346b0e4067d32ec36b7992c1fdc001dc8517 done
  #5 extracting sha256:63b65145d645c1250c391b2d16ebe53b3747c295ca8ba2fcb6b0cf064a4dc21c 0.1s
  #5 sha256:ff6bdca1701f3a8a67e328815ff2346b0e4067d32ec36b7992c1fdc001dc8517 1.64kB / 1.64kB done
  #5 sha256:e2e16842c9b54d985bf1ef9242a313f36b856181f188de21313820e177002501 528B / 528B done
  #5 sha256:b2aa39c304c27b96c1fef0c06bee651ac9241d49c4fe34381cab8453f9a89c7d 1.47kB / 1.47kB done
  #5 sha256:63b65145d645c1250c391b2d16ebe53b3747c295ca8ba2fcb6b0cf064a4dc21c 3.37MB / 3.37MB 0.1s done
  #5 extracting sha256:63b65145d645c1250c391b2d16ebe53b3747c295ca8ba2fcb6b0cf064a4dc21c 0.1s done
  #5 DONE 0.3s

  #6 [2/3] RUN wget -q -O- https://github.com/srvaroa/labeler/releases/latest/download/action.tar.gz | tar xzvf -
  #6 0.341 wget: server returned error: HTTP/1.1 404 Not Found
  #6 0.342 tar: invalid magic
  #6 0.342 tar: short read
  #6 ERROR: process "/bin/sh -c wget -q -O- $ASSET_URL | tar xzvf -" did not complete successfully: exit code: 1
  ------
   > [2/3] RUN wget -q -O- https://github.com/srvaroa/labeler/releases/latest/download/action.tar.gz | tar xzvf -:
  0.341 wget: server returned error: HTTP/1.1 404 Not Found
  0.342 tar: invalid magic
  0.342 tar: short read
  ------
  Dockerfile:12
  --------------------
    10 |     WORKDIR /
    11 |     ARG ASSET_URL=https://github.com/srvaroa/labeler/releases/latest/download/action.tar.gz
    12 | >>> RUN wget -q -O- $ASSET_URL | tar xzvf -
    13 |     ENTRYPOINT ["/action"]
    14 |
  --------------------
  ERROR: failed to solve: process "/bin/sh -c wget -q -O- $ASSET_URL | tar xzvf -" did not complete successfully: exit code: 1
  Warning: Docker build failed with exit code 1, back off 1.424 seconds before retry.

This looks caused by 404 error returned by GitHub, downloading the source code archive.
  #6 [2/3] RUN wget -q -O- https://github.com/srvaroa/labeler/releases/latest/download/action.tar.gz | tar xzvf -
  #6 0.341 wget: server returned error: HTTP/1.1 404 Not Found
  #6 0.342 tar: invalid magic
  #6 0.342 tar: short read
  #6 ERROR: process "/bin/sh -c wget -q -O- $ASSET_URL | tar xzvf -" did not complete successfully: exit code: 1

Releases before 1.6.2 have action.tar.gz as their assets, but 1.6.1 does not have one.
1.6.1

1.6.2

Getting the same error for v1.6.1.

Submitted a PR to fix this issue.
@srvaroa Could you please review the PR?

v1.6.3 should be fine


Can i set Condition Breakpoint ?

and i also want to get the bplist

Not at the moment, sorry. I believe @mrexodia is re-working the breakpoint API at the moment which will also allow callbacks per breakpoint enabling much more flexible conditions.

and can i get this string that jump is not taken

Relevant pull request: https://github.com/x64dbg/x64dbg/pull/3317 (not ready yet, but it should be a good base)


Alignment of start event off in initial diagram

Describe the bug
As I'm working off the default BPMN diagram template I notice that the alignment of the start event is off.
If I continue to model, i.e. a sub-process with a contained start event, then I cannot align both events with each other:

Steps to reproduce

Create a new BPMN file
Add a Sub-Process
Place a Start Event into the Sub-Process and try to align it with the incoming Sequence Flow

Expected behavior

[ ] Start event in sub-process and start event outside are aligned

Environment

OS: Linux
Camunda Modeler Version: v5.26.0
Execution Platform: Any
Installed plug-ins: None

Additional context
Related, if I change the start event position manually (in the XML) and use the align (vertical) tool, then the start event snaps back to the original (not aligned) position:

Raised in internal discussion

Moving to spring cleaning.

Closed by https://github.com/camunda/camunda-modeler/pull/4642


[WIP] Remove portal

Please don't merge before the styleguide has been updated here
This is a master<-feature-branch<-update-feature-branch pull request. See above.

Updated and published styleguide, now you need to update to 0.5.11


Allow use of comment instead of tag

Sometimes using a tag to trigger the benefits of this plugin on a template string isn't feasible, and it would be nice to be able to substitute a comment instead. For example, instead of doing this:

var template = html`

Hello World!

`

...it would be nice to be able to do this:

var template = /*html*/`

Hello World

`

This would solve https://github.com/mjbvz/vscode-lit-html/issues/14.

but i want to understand why a comment be desirable over of using a dummy-noop html tag function?
using a noop has the advantages of (a) already working, and (b) making for a good point of refactorability, ie, validation or santization
what motivates you to prefer a comment for this purpose?

Just wanted to throw a +1 on this — I've been using that es6-string-html plugin and it works fine visually, but it doesn't implement the wonderful IntelliSense support and other things that this plugin has. Sometimes I'm just throwing some stringified HTML into a <template> via JS and would love to have this stuff come along with it!


Overhaul the types

Rewritten the type declaration using also the native mongodb types.
Linked issue: https://github.com/Automattic/monk/issues/312

Awesome! Forgot about those. Thanks for doing a quick job :) had other repos where it takes ages 🙈
If you have problems with types or so I'd be happy to fix them in the future :)


419 status with laravel breeze and nuxt

Describe the bug
I'm trying to use HyperFetch in one my projects. When I use the code below trying to login I get 419 unknown status. But it is working fine with axios and nuxt's useFetch.
To Reproduce
Steps to reproduce the behavior:
async function handleLogin() {
const client = new Client({ url: "http://localhost:8000" });
	const cookie = client.createRequest()({
		endpoint: "/sanctum/csrf-cookie",
		method: "GET",
		options: { withCredentials: true },
	});
	await cookie.send();

	const token = useCookie("XSRF-TOKEN");

	type ResponseType = { token: string; refreshToken: string };
	type RequestType = { email: string; password: string };

	const postLogin = client.createRequest<ResponseType, RequestType>()({
		method: "POST",
		endpoint: "/login",
		headers: {
			["X-XSRF-TOKEN"]: String(token.value),
		},
	});

	const { data, error, status } = await postLogin.send({
		data: {
			email: loginForm.email,
			password: loginForm.password,
		},
	});

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Not a hyper-fetch issue I think.
You don't need to call /sanctum/csrf-cookie every login (submit).

subsequent requests to the Laravel application may receive 401 or 419 HTTP error response
@see https://laravel.com/docs/10.x/sanctum#logging-in

Thanks @johngerome for help! 🚀
I will close the issue, if it still exist - I will reopen thread.


Encrypted bytecode Error

There is an error in changing the encrypted bytecode, although jbyteedit does this fine, but the Label and Line function does not work in it, so I had to use your bytecode editor, please fix this problem

There is not enough information here for me to figure out what the problem is. I will need you to share some more information with me so I can help you out.
What code are you trying to edit? Can you share that? Also Recaf generates a file rclog.txt, can you please share that?

rclog.txt

I am changing the link connection string

A window should pop up when the export fails that shows the verification analysis of the current jar. Is the message about L the only thing showing up in the verification window or is there more?

Only "L"

Well, the method instructions have to be valid to properly execute. So this leads me to believe some debug information is improperly formatted and ASM is failing to verify that.
Can you try checking the Skip Debug option and re-importing your jar and making your changes?

Ok, so something is still bad. Uncheck the Verify option.
The sample looks obfuscated so the obfuscator probably has tampered with the bytecode in such a way that the modifications to the class are technically allowed by the file format, but isn't actually representing valid data, and this is why ASM fails to verify the data.

It will be necessary to test
If everything worked or not
I will answer tomorrow

@Col-E Thx

:+1:


Databox does not start

I have built all of components with master branch in hub. docker.com ,and then databox-start run,cannot start the docker container .

In following command, there was no container manager,arbiter,and exportservice ,etc
[vagrant@databox ~]$ docker ps -a
CONTAINER ID        IMAGE                                COMMAND                  CREATED             STATUS                   PORTS                              NAMES
6578ae017b7b        databoxsystems2/app-server           "npm start"              20 minutes ago      Up 20 minutes            8080/tcp, <IP_ADDRESS>:8181->8181/tcp   databox_app-server_1
be116ce75022        databoxsystems2/core-network:0.3.2   "./start.sh"             20 minutes ago      Up 20 minutes            8080/tcp                           databox_databox-network_1
d257c2d72cf9        e35aaaa4ce38

I solved it.


Route table and NSGs for the new VNET integration

Could you clarify on the final status for route tables and NSGs for the new VNET integration?

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

ID: a7a98803-1438-b1b5-f543-7dd88bc4294e
Version Independent ID: 37ff1d0f-ed8e-5e4d-1f4c-1b9f6cffb938
Content: Integrate app with Azure Virtual Network - Azure App Service
Content Source: articles/app-service/web-sites-integrate-with-vnet.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@nelroosevelt , The new VNET integration is in preview. Route tables and global peering are not yet available with the new VNet Integration. There is currently no ETA on this.

@nelroosevelt , I will go ahead with the closure of this issue. Feel free to re-open this if you have further queries.


Chain state database corruption likely

What happened: the chain start getting corrupt see debug file
2021-03-25T15:06:59Z UpdateTip: new best=48664522bf6408560f80fa153a8e1cbe5485e33b04f15776ae6fd634a94eee3d height=733230 version=0x20000000 log2_work=79.646165 tx=2039000 date='2021-03-25T11:34:51Z' progress=0.998852 cache=0.0MiB(0txo)
2021-03-25T15:06:59Z UpdateTip: new best=2b4d4a765707b6543956a8619b7fa418b5cd5eee5311a2ee0afa77285fd07a1c height=733229 version=0x20000000 log2_
What you expected to happen: running the app
How to reproduce it (as minimally and precisely as possible): running the app

What are your environment parameters?

DeFi Wallet Version: v.2.3.2
Operating System (OS):WINdows server 2009

Anything else we need to know?:

Hi @supervolot , thanks for your feedback, there might not anything wrong with the binary log portion you sent. Please send full app log, binary log and app config so it'll help us investigate.

If you're stuck on block 733229 / 733230 you can try to reindex and see if it syncs to 100%.

Hi @supervolot , regarding the issue Sync stuck at 98%+ / blockHeight 733594, we can further discuss on the problem there. We would like to keep same issues in one thread. Thank you.


Json converter should support ICollection<object> type

@TheHadiAhmadi is this issue active, or we have resolved that?

@pournasserian this is resolved


IOS-7287: Update the various file and folder icons

Verified that @boxtcarpel has signed the CLA. Thanks for the pull request!

+2


⚠ Bofrost.fr has degraded performance

In b4c1c86, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1174 ms

Resolved: Bofrost.fr performance has improved in 5f0888f.


attempt to parse JSON when GM_xmlhttpRequest is not supported

Hello,
First off, thanks loads for the userscript and the work you've put into it.
Also, I've not touched JavaScript until today so I very likely will sound like a noob, and I'm not sure how you'd prefer your versioning done, so I hope you don't that I've incremented it with the change.
Qutebrowser has builtin 'greasemonkey support' as they call it. Unfortunately as of now GM_xmlhttpRequest is not implemented. All in all this is not a problem as this script very nicely seems to fall back to XMLHttpRequest, the issue that follows though is that the attempt to parse the response as JSON doesn't happen when XMLHttpRequest is used. This causes the script to error out when checking for res[".meta"].match as res is not a JSON object and has no properties.
All this diff does is duplicate the the JSON.parse() stuff from the GM_xmlhttpRequest block.
Thanks,

Thanks for opening this PR and alerting me of this. I haven't tested the fallback in a while, admittedly. Great work on these changes; I'll merge this.


Add a Stream type and make iterate return a stream.

 It can also be dropped to NonEmpty and to the usual list free of charge. Unfolds without a base case are the class of functions that produce streams, iterate being the immediate example, and they could be typed as such.
I think this type would be a nice addition to Relude.

The problem with adding something like a new type (e.g. Stream) is that it won't have all the instances as the original type. This will result in code duplication, and code reuse actually won't be possible since the Stream type in relude will be different from Stream type in another library. Since we don't want to depend on more non-boot libraries according to the goals of this project, reexporing Stream is not an option unless it happens to be on one the boot libraries.


Only one GPU shows usage while quantizing

Hi, while quantizing large models (qwen 72b) on 5x A40 GPUs, I noticed that only the first GPU seems to show high (80-90%) utilisation, while the rest sit at 0%. Is this something normal, or am I missing some config flags? The process works ok and the model works fine afterwards, just wondering if this is normal, or if I can increase the speed somehow. Thanks!

Hi! Could you share the script you're running to quantize the model?

Sure, I'm using a pretty straightforward script that matches the examples:
from llmcompressor.modifiers.quantization import GPTQModifier
from llmcompressor.modifiers.smoothquant import SmoothQuantModifier
from llmcompressor.transformers import oneshot
from datasets import load_dataset
from llmcompressor.transformers import SparseAutoModelForCausalLM
from transformers import AutoTokenizer

NUM_CALIBRATION_SAMPLES=1024
MAX_SEQUENCE_LENGTH=4096

recipe = [
    SmoothQuantModifier(smoothing_strength=0.8),
    GPTQModifier(scheme="W8A16", targets="Linear", ignore=["lm_head"], dampening_frac=0.1),
]

model = SparseAutoModelForCausalLM.from_pretrained(
    MODEL_ID, device_map="auto", torch_dtype="auto",
)
tokenizer = AutoTokenizer.from_pretrained(MODEL_ID)

ds = load_dataset("AI-MO/NuminaMath-TIR", split='train')
ds = ds.shuffle(seed=42).select(range(NUM_CALIBRATION_SAMPLES))

def preprocess(example):
    return {"text": tokenizer.apply_chat_template([{"role":"system", "content":"Please integrate natural language reasoning with programs to solve the problem above, and put your final answer within \\boxed{}."}, *example["messages"]], tokenize=False,)}

ds = ds.map(preprocess)

def tokenize(sample):
    return tokenizer(sample["text"], padding=False, max_length=MAX_SEQUENCE_LENGTH, truncation=True, add_special_tokens=False)

ds = ds.map(tokenize, remove_columns=ds.column_names)

oneshot(
    model=model,
    dataset=ds,
    recipe=recipe,
    output_dir=OUTPUT_DIR,
    max_seq_length=4096,
    num_calibration_samples=NUM_CALIBRATION_SAMPLES,
)

Where MODEL_ID is qwen2.5-math-72b-instruct.

For reference, this is what I see in nvtop

Hi @gmonair
Instead of passing in "auto" for the device map, could you pass in the output of calculate_offload_device_map?
Example:
from llmcompressor.transformers.compression.helpers import calculate_offload_device_map

device_map = calculate_offload_device_map(
    MODEL_ID,
    reserve_for_hessians=True,
    num_gpus=torch.cuda.device_count(),
    torch_dtype=torch.bfloat16,
    trust_remote_code=True,
)

This is will ensure that cpu_offloading is applied properly during oneshot.


Null Job Output in Dashboard

Describe the Issue
Null job output in dashboard for Hello World example
To Reproduce
Run the code sample provided below and view the job output in the dashboard.
Expected behavior
Expected "Hello World" in the output
Script Sample
$Script = New-UAScript -Name "My First Script" -ScriptBlock {Start-Sleep -Seconds 5; Write-Host "Hello World"}
Invoke-UAScript 1

Version Information

Windows Server 2019:
5.1.17763.592:
2469:
2.8.1:
PowerShell behind an NGNIX reverse proxy
Am testing everything both through and around the reverse proxy

Possibly related to #3 ?

This is resolved in tonight's build.


Fix/execute with retry wrapper exec

Description

Refactors executeWithRetry to no longer block  but rather poll the executeCallback for its completion which allows the Timeout to successfully invoke and short circuit operations.
Reduces the default retry attempts from 3 to 1

Type of change

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

How Has This Been Tested?

Added new unit tests to misc.spec.ts for exercising different retry situations that where not covered.
Used local testing

command to test: DEBUG=false NETWORK=manzano yarn test:local --filter=testDelegatingCapacityCreditsNFTToAnotherWalletToExecuteJs
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Closing to reimplement with the following design

Reduce operations that are retried
Remove retry from top level LitNodeClient
Add selective retry at the sendRequest level such that we only retry single requests rather than groups of requests.


Fix most-recent remark at building when empty

This should fix #304. We now respond with an empty list when there are no remarks when the most-recent parameter has been given.

@TiboStr don't forget to review this one :smile:


Keypunches Portraits i and ii

Took my best guess at the Greek(?) in Portraits II.

Oh dang, I'm realizing this PR has a weird first commit where I pulled from your repo. I don't quite know enough Git to fix that. If you can, go for it. If not, I can re-do this PR.

Looks like we've got a conflict, so, yeah, if you could redo that would be great.


Add wrapper for running client methods

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#983
#982
#981  👈
#976
#975
main

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

Join @carolineechen and the rest of your teammates on  Graphite


show the command center chat icon in the text

There was a recommendation to have the icon in the text so users know what to look for in standup. We don't currently support icons in welcome view content, so I had to add it.

@sbatten this commit seems to be missing from https://github.com/microsoft/vscode/commits/release/1.95/

Will backport.


Feature Request: Support file association properties in forge.config.js

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Problem description
Other electron build systems allow the developer to specify custom file extensions for their built apps. For instance, if I wrote an app that was a editor for "Foo" files, I would want to be able to launch a file with the extension .foo in the system that would then open the application with that file. As a developer I could then handle the file's contents via the app open-file event handler. In addition, .foo files should have their own specific icon.
Currently providing this information via platform-specific configuration files (such as for Windows, OSX, and Linux) is cumbersome and the method for doing so is not always well documented, making cross-platform applications that much more difficult to create.
Proposed solution
I would suggest a set of forge configuration properties such as:
module.exports = {
  packagerConfig: {
    "fileAssociations": [
      {
        "ext": "foo",
        "name": "Foo File",
        "icon": "assets/icon/foofile.icns"
      }
    ],
  }
}

...where one or more file associations can be specified.

ext is the name of the file extension to associate with one's Electron application
name is the label that the platform presents to users in the user interface
icon is the path to the icon for this type of file

Refer to the fileAssociations section of https://www.electron.build/configuration/configuration.html for more information. Note that their set of configuration properties allows this feature to be used in OSX, Windows, and Linux applications.
Alternatives considered
Currently in order to implement this in OSX, a separate Info.plist file must be provided that contains the correct metadata to enable one or more file associations to work. In Windows a separate file containing file association properties must be generated as well. This is cumbersome for developers, who must currently deal with many sets of documentation to implement this feature on multiple platforms.
Additional information
No response

Im waiting for this too!
It was so easy to do with electron-builder..

Can you explain how to setup file association with a info.plist file?

For Info.plist, you can use the extendInfo option to add anything, like this:
module.exports = {
  packagerConfig: {
    extendInfo: {
      CFBundleDocumentTypes: [
        {
          CFBundleTypeName: "MyTextEditor",
          CFBundleTypeRole: "Editor",
          LSTypeIsPackage: true,
          LSHandlerRank: "Default",
          CFBundleTypeExtensions: ["txt"],
        },
      ],
    }
  }
};

Windows would be tricky because "regular" win32 applications have to register these in the registry while Microsoft Store applications have to declare them in the appx manifest.

For Windows if you are using the Wix-Maker you can do the following:
interface ExtendedMakerWixConfig extends MakerWixConfig {
  // see https://github.com/electron/forge/issues/3673
  // this is an undocumented property of electron-wix-msi
  associateExtensions?: string;
}
...
{
  config: {
     icon: path.join(process.cwd(), 'assets', 'icon.ico'), // I needed this path to be absolute
     associateExtensions: 'ext',
  },
  beforeCreate: (creator: WixCreator) => {
    // manually convert app icon to some other icon
    // manipulate the fileAssociationTemplate
  },
} satisfies ExtendedMakerWixConfig,


Data Import/Export

Importing/Exporting Data
This issues discusses the possibility of exporting card data from the app as a local file as well as importing files containing card data in the app. This could be really useful when you wish to share your kanban boards with someone else or to import other's boards or even if you wish to transfer your boards between devices as Kards does not have a backend.
Implementation
The app data is already stored as a JavaScript object which is easy to serialize into JSON format, so the obvious choice is to use .json files for the export and import of data files. Another format like .csv could have also been considered, however it is quite difficult to store multiple boards  along with all the settings of the app. Perhaps in the future, there could be a way to export/import individual boards as/from .csv files.
Importing
Importing can be implemented using the File Reader API. Perhaps a button in the settings menu of the app could prompt the user to select a JSON file, which will then be parsed and loaded by the app. There could be additional options whether to overwrite the existing boards or to load the boards from the file as additional boards.
Exporting
Exporting can be implemented using the Blob API. The app data will be JSON stringified and downloaded locally.

I will work on it.


Provide the List of Personal Admin and Time Admin Groupings

#Privacy of Information and Protection:
Your opinion is your personal information. Please do not include any information which identifies you or others in your response.
#Description:

Identify and list any time admin groups that do not follow the division/CRD section/work unit hierarchy.  SuccessFactors calls this dvision/department Eg. Personal Admin: Information Technology & GIS Services Time admin: Technical Services.
Determine a path to correct these anomalies to fit the structure.  These can be fixed on #509

#Outcome:

[ ]  Provide a clean list by Division and Department (CRD Section/Work Unit) to Colleen for pick lists in SF.
[ ]
[ ]

#To Complete this ticket:

[ ]
[ ]
[ ]

@ctimlick  Looks like we cannot provide a clean list as there are too many issues.  There are 252 entries.  I also think we did not capture it when we need to pick the time admins.  When the hiring  manager creates a job requisition, at that point they should assign the time admin, sender cost centre and activity type and that should flow through to EC.  I think.


displayName:Null for android and ios

Device
Android 9.1 ,iOS 13.1.2
console image
Problem
Contact name is not displayed properly

See https://github.com/apache/cordova-plugin-contacts#ios-quirks-1 for iOS
Sadly, this plugin is deprecated
Please follow the issue template

Thanks

This problem has already been solved.


[ngtcp2] Update to 1.5.0

Disable Testing (via munit)
Remove build cunit parameter
Update no export patch

Not yet ready for merge (no yet 1.5.0 release yet)

@alagoutte Thank you for your update!
Why don't you export ngtcp2 in cmake official? instead to change the patch here?
Also, any plan to add openssl support, now it release on 3.3.0, and if not, what is missing?

@alagoutte Thank you for your update! Why don't you export ngtcp2 in cmake official? instead to change the patch here? Also, any plan to add openssl support, now it release on 3.3.0, and if not, what is missing?

I'm not author of ngtcp2 ;-)
I'm only update the package...
I can look for see to include export on ngtcp2
and for openssl, it is not planned -> https://github.com/ngtcp2/ngtcp2/issues/990

Updated with new release

and for openssl, it is not planned -> ngtcp2/ngtcp2#990

I wonder if that answer also covers OpenSSL 3.3  and later version.

@alagoutte
1.5.0 is declared with c529fdf592d945696a24e6cb0f7f169aea71e0c2, but the local port has a different SHA e84533ed07e21e33b38553c3b75abd9a65536e76.
Please update the port's version fields and then run:
vcpkg x-add-version ngtcp2 --overwrite-version
git add versions
git commit -m "Update version database"
to add the new version.
To attempt to resolve all errors at once, run:
vcpkg x-add-version --all


bash-env-json: fix path issue

Things done
Fixed bash-env-json breaking if it tries to process a file that sets PATH.
This program cannot rely on PATH, therefore full paths must be used in this program all the time.

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@Aleksanaa can you please read this issue through and tell us how to go about this package in the long run?
https://github.com/tesujimath/bash-env-json/issues/14
I want to know if there are any other nixpkgs packages that use nix code from the repo they are downloading to build the package and if it is even possible.
And if it is, is it allowed due to the fact that nix reviewers will have to review outside code that is imported before merging the updated derivation.
I think I already know the answer but I don't think I should be the one to say the final verdict to the maintainer of the program because I am just a package maintainer in nixpkgs who doesn't know all the rules, not an authority.

There's actually a tool called resholve here: https://github.com/NixOS/nixpkgs/tree/nixos-unstable/pkgs/development/misc/resholve
it correctly identifies executables in scripts and replaces them with those provided in packages, and fails if the provided list isn't sufficient. However, it depends on Python2 thus is discouraged to use. But I still recommend you use it here, if possible.

And if it is, is it allowed due to the fact that nix reviewers will have to review outside code that is imported before merging the updated derivation.

This is probably unrealistic. We will read the diff, but this does not mean that we can always fully understand them...

And we have no responsibility to enforce a standard in other people's projects, which is needed in Nixpkgs from time to time. To put it bluntly, the writing format of most out-of-tree derivation is completely substandard (although some in-tree derivation is also like this)

@Aleksanaa Thank you for your explanation.
The main concern for us is, that if the maintainer adds another dependency to the script it will also have to be added to the derivation along with the update.
I didn't know about resholve and it sounds great except for the fact that it uses python2, but I think it won't exactly solve the issue because it still requires changes to inputs and package call parameters if new dependencies are introduced.
Also resholve failed with if ! source "$_path" >/dev/null 2>&4; then Can't resolve dynamic argument in 'source' on bash-env-json so it's out of the question if this error is not a result of my incompetency with resholve
here is what I tested it with:
{
  resholve,
  bash,
  lib,
  fetchFromGitHub,
  coreutils,
  gnused,
  jq,
  nix-update-script,
}:

resholve.mkDerivation rec {
  pname = "bash-env-json";
  version = "0.9.2";

  src = fetchFromGitHub {
    owner = "tesujimath";
    repo = "bash-env-json";
    rev = version;
    hash = "sha256-EYro4pMILnQFpXpFjdzSDuudhqC2EvysYMUmIOvesgo=";
  };

  installPhase = ''
    runHook preInstall

    install -Dm755 bash-env-json -t $out/bin

    runHook postInstall
  '';

  solutions = {
    default = {
      scripts = [ "bin/bash-env-json" ];
      interpreter = lib.getExe bash;
      inputs = [
        coreutils
        gnused
        jq
      ];
    };
  };

  passthru.updateScript = nix-update-script { };

  meta = {
    description = "Export Bash environment as JSON for import into modern shells like Elvish and Nushell";
    homepage = "https://github.com/tesujimath/bash-env-json";
    mainProgram = "bash-env-json";
    license = lib.licenses.mit;
    maintainers = with lib.maintainers; [ jaredmontoya ];
    platforms = lib.platforms.all;
  };
}

But as the docs say:

Fair warning: resholve does not aspire to resolving all valid Shell scripts. It depends on the OSH/Oil parser, which aims to support most (but not all) Bash. resholve aims to be a ~90% sort of solution.`

so it's expected.
I think I will just have to check a small section of the source code for new dependencies before updating the package which is not a big deal.

@Aleksanaa what is happening with https://github.com/NixOS/nixpkgs/pull/358950/checks?check_run_id=33519605117
Why does it take more than 11 days?

Finally, it passed

@teto Can you merge this please?


Added "Publish-time Pre-render for Blazor Wasm" into Tooling

Added "Publish-time Pre-render for Blazor Wasm" into Tooling.
An output of "dotnet publish" before installing this package:

And after installing this package:

Interesting, thanks!

@AdrienTorris Thanks!


Help message should not be written to stdout for sh- commands

rbenv shell --help is broken since it tries to write help messages on stdout. stdout for sh- commands should be shell commands though.
% rbenv shell --help
(eval):2: parse error near `\n'

As a workaround, this patch will redirect help messages to stderr instead of stdout.

Good catch! This could be one workaround, but I'm not happy with the fact that going forward, rbenv shell --help will be mysteriously output to stderr, while all others go to stdout. An explicit call for --help should always go to stdout, so that a program could capture help text (in theory) and pass it on somewhere else; such as to a pager (e.g. less).
Maybe the workaround belongs instead in the rbenv() shell wrapper function. But that might make the function more complex, which is tricky to pull off since the function needs to work for a variety of shells (including fish, which has a totally different syntax).

You're right. I didn't want to make rbenv() more complex.
I reconsider about the implementation and prepared 26ac59fd1da0a28695697a03bd575810bc962d90. Now it should work fine with stdout. Please review it.


doc: EVP_PKEY_get_utf8/octet_string_param() clarify NULL buffer behavior

Checklist

[x] documentation is added or updated

This pull request is ready to merge


chore: update qg-checklist

Description

remove colapsible items and remove all sub-tasks
add link to our automatic checks
add back on top and back on section links

updates #14
Visual preview

Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

Missing update to 4.02 in my opinion: It's also allowed to (maybe even encouraged) to have a dedicated .md file for DockerHub. Title could therefore be misleading

updated the Checklist on @almadigabor suggestions so that now sub-task is show on the template but for earch TRG a available link is present what to check in there... only thing we can still discuss is the wording.


Water Sensors: ZJH NH

Site is completly offline. No apparent issue in string to me.

asking for download again. no data for a while.

all available data downloaded at the end of the season.


script not working

script not working....

When I test your Preferences file, Edge says it's corrupted:

So, I manually added, the only IE Mode page I see in your Preferences file, to my IE Mode pages:

Then I ran the script and the expiry date was correctly updated:

I don't see any issue with the script. I only see an issue with your Preferences file.
However, the site http://bpm.diodes.com does not exist. Is that the page you wanted to add?
Did you run the script as is? If you edited the script in any way, please send me your edited version.

i did'nt change the script,my step is add the web site and double click the script.
then get same message...

please help me to cover any about diodes’s url, thank you
LesFerch @.***>於 2023年6月27日 週二，上午12:23寫道：

When I test your Preferences file, Edge says it's corrupted:
[image: image]
https://user-images.githubusercontent.com/79026235/248865283-f7f37657-1746-4021-b638-e263f04d4ae3.png
So, I manually added, the only IE Mode page I see in your Preferences
file, to my IE Mode pages:
[image: image]
https://user-images.githubusercontent.com/79026235/248865519-a1c64ef8-7294-44af-8979-6e01915a28f5.png
Then I ran the script and the expiry date was correctly updated:
[image: image]
https://user-images.githubusercontent.com/79026235/248865643-47d0ab18-71c4-4f92-9da0-476e4beace59.png
I don't see any issue with the script. I only see an issue with your
Preferences file.
However, the site http://bpm.diodes.com does not exist. Is that the page
you wanted to add?
Did you run the script as is? If you edited the script in any way, please
send me your edited version.
—
Reply to this email directly, view it on GitHub
https://github.com/LesFerch/IEModeExpiryFix/issues/2#issuecomment-1607819231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP6MHOLDI2GAAK7SGUTM6VDXNGZRFANCNFSM6AAAAAAZTSDKG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

I don't know why your Preferences file is getting corrupted. I tested with the same Edge version on Windows 11 build 22621.1848. Everything works fine for me. I could try setting my system language and regional settings to match your system, in case that makes a difference. Can you post a screenshot of your Region control panel and your Language and region setting screen?

Taiwan and Traditional Chinese

Taiwan and Traditional Chinese

Thanks. 🙂

It still works fine for me even with everything set to Taiwan and Traditional Chinese.
Have you tried deleting the Preferences file so that Edge creates a new one?

Did you try deleting the Preferences file?

您是否嘗試刪除首選項文件？

Yes,i try delete the preferences file.

I see a "sitelist_data_2" section in your Preferences file. It looks like your Edge settings are centrally managed. If so, the expiry can be set by your IT department, as per the documentation here.
Please confirm and let me know. Thanks.

yes it is managed.so i can't use the script to extend date?

That's my conclusion. Your IT department would have to add that page to the site list with an extended expiry. If they can't, or won't, do that, I can only suggest an alternate solution...
You can save a simple three line script that will open that page directly in IE. It's really no different then opening in IE Mode because that mode also uses IE to open the page (you may have noticed iexplore.exe in your task list when the page is open). Just change about:blank in the script to the URL you need. There's no added risk accessing the page that way, as long as you don't use IE as a general purpose browser.


libffi / ffi.h issue on alpine-python:3.6-onbuild

I could be doing something wrong but I get the following with a fairly simple requirements.txt and alpine-python:3.6-onbuild.  This could be just an issue with upstream pip however?
Short summary
  Installing build dependencies: finished with status 'error'
  Complete output from command /usr/lib/python3.6.6/bin/python3.6 /usr/lib/python3.6.6/lib/python3.6/site-packages/pip install --ignore-installed --no-user --prefix /tmp/pip-build-env-c3okpqft/overlay --no-warn-script-location --no-binary :none: --only-binary :none: -i https://pypi.org/simple -- setuptools wheel "cffi>=1.4.1; python_implementation != 'PyPy'":

Full log
      Running setup.py install for cffi: finished with status 'error'
      Complete output from command /usr/lib/python3.6.6/bin/python3.6 -u -c "import setuptools, tokenize;__file__='/tmp/pip-install-labi968q/cffi/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-record-t4nbxvbv/install-record.txt --single-version-externally-managed --prefix /tmp/pip-build-env-r9ugyqb3/overlay --compile:
      Package libffi was not found in the pkg-config search path.
      Perhaps you should add the directory containing `libffi.pc'
      to the PKG_CONFIG_PATH environment variable
      Package 'libffi', required by 'virtual:world', not found
      Package libffi was not found in the pkg-config search path.
      Perhaps you should add the directory containing `libffi.pc'
      to the PKG_CONFIG_PATH environment variable
      Package 'libffi', required by 'virtual:world', not found
      Package libffi was not found in the pkg-config search path.
      Perhaps you should add the directory containing `libffi.pc'
      to the PKG_CONFIG_PATH environment variable
      Package 'libffi', required by 'virtual:world', not found
      Package libffi was not found in the pkg-config search path.
      Perhaps you should add the directory containing `libffi.pc'
      to the PKG_CONFIG_PATH environment variable
      Package 'libffi', required by 'virtual:world', not found
      Package libffi was not found in the pkg-config search path.
      Perhaps you should add the directory containing `libffi.pc'
      to the PKG_CONFIG_PATH environment variable
      Package 'libffi', required by 'virtual:world', not found
      running install
      running build
      running build_py
      creating build
      creating build/lib.linux-x86_64-3.6
      creating build/lib.linux-x86_64-3.6/cffi
      copying cffi/vengine_cpy.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/model.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/commontypes.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/error.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/backend_ctypes.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/cparser.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/ffiplatform.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/pkgconfig.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/vengine_gen.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/api.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/lock.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/__init__.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/setuptools_ext.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/cffi_opcode.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/verifier.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/recompiler.py -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/_cffi_include.h -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/parse_c_type.h -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/_embedding.h -> build/lib.linux-x86_64-3.6/cffi
      copying cffi/_cffi_errors.h -> build/lib.linux-x86_64-3.6/cffi
      running build_ext
      building '_cffi_backend' extension
      creating build/temp.linux-x86_64-3.6
      creating build/temp.linux-x86_64-3.6/c
      gcc -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -fPIC -DUSE__THREAD -DHAVE_SYNC_SYNCHRONIZE -I/usr/include/ffi -I/usr/include/libffi -I/usr/lib/python3.6.6/include/python3.6m -c c/_cffi_backend.c -o build/temp.linux-x86_64-3.6/c/_cffi_backend.o
      c/_cffi_backend.c:15:17: fatal error: ffi.h: No such file or directory
       #include <ffi.h>
                       ^
      compilation terminated.
      error: command 'gcc' failed with exit status 1

      ----------------------------------------
  Command "/usr/lib/python3.6.6/bin/python3.6 -u -c "import setuptools, tokenize;__file__='/tmp/pip-install-labi968q/cffi/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-record-t4nbxvbv/install-record.txt --single-version-externally-managed --prefix /tmp/pip-build-env-r9ugyqb3/overlay --compile" failed with error code 1 in /tmp/pip-install-labi968q/cffi/

Here's my requirements.txt
cherrypy
pyyaml
mongoengine
requests
ipdb
python-wordpress-xmlrpc
python-git
flake8
paramiko
pexpect

Here's my Dockerfile
FROM jfloff/alpine-python:3.6-onbuild

RUN apk upgrade --update
RUN apk add git
WORKDIR /opt

Have you tried install libffi-dev ?

Hi there, I recently updated the images and libffi-dev is now included. Could you double check if your previous image now works?
Thank you!

@jfloff with apk add libffi-dev it works perfectly

apk add libffi-dev works great on 3.7-alpine too.

RUN apk add --no-cache libffi-dev


Use TurboSnap feature from Chromatic to reduce quota usage

Summary
The goal for this PR is to reduce our Chromatic quota usage by leveraging TurboSnap, which should only compare snapshots for code that has changed.
Changes
See https://www.chromatic.com/docs/turbosnap#enable
The PR just adds the relevant option to Chromatic.
Review checklist

[ ] Description is complete
[ ] 2 reviewers (1 if trivial)
[ ] Tests coverage has improved
[ ] Code is ready for a release on NPM

Well, unfortunately, TurboSnap won't work for us :( https://www.chromatic.com/docs/turbosnap#specify-external-files-to-trigger-a-full-re-test-when-they-change
Basically, it can only detect direct dependencies to stories. In our case, most changed we want to detect happen in visualization, and these changes are imported through node_modules and Webpack can't trace beyond it. This means that changes are not detected. And if we add these files with --externals, this mean any change in there will trigger a full re-test, which means TurboSnap is useless here.


EKF: Add feature to use the mag to only update the mag states

FYI @priseborough

There could be unintended interactions between this PR and the recently added functionality to switch in and out of 3-axis fusion. Replay testing on flight logs is required.

That looks better. I will do reversion testing on this changes, but am not expecting any issues as the functionality is off by default.

We also need to think about how to handle reporting of the control status which is currently via the get_control_mode get_control_mode(uint16_t *val) accessor. At the moment the accessor is only copying the first 16 bits.
The accessor in estimator_interface.h will need to be updated and this will also require a corresponding change to ekf2_main.cpp when the upstream library reference is updated.

God point. I updated the function in ecl

I've flight tested this using the following test bench which hardwired the feature on: https://github.com/priseborough/Firmware/tree/test_ecl_pr276-wip
Flight characteristics were unchanged and the log showed that filter operation was as expected.
log001.ulg.pdf

This PR will require the PX4/Firmware estimator status topic to be modified to accomodate the increased size of the control mode flag, however we were already out of bits and the increased size is going to be necessary longer term as we add more measurement types.


Implement Azure Search Index Management Operations

Here are a couple of quick links to help with that first SearchServiceClient.GetStatistics method to get the client hierarchy in place:
Source: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/search/Azure.Search/src/SearchServiceClient.cs#L201
Single Test ported from Track 1: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/search/Azure.Search/tests/SearchServiceClientTests.cs#L133

See https://github.com/Azure/azure-sdk-for-python/pull/10571

Fixed in #10571


Linux build failed

Linux build is failing with an error:
39.33 /app/.build/checkouts/Actomaton/Sources/Actomaton/MainActomaton.swift:54:19: error: actor-isolated property 'state' can not be referenced from the main actor
39.33         actomaton.state
39.33                   ^
39.33 /app/.build/checkouts/Actomaton/Sources/Actomaton/Actomaton.swift:11:29: note: property declared here
39.33     public private(set) var state: State
39.33                             ^
39.33 /app/.build/checkouts/Actomaton/Sources/Actomaton/MainActomaton.swift:54:19: error: actor-isolated property 'state' can not be referenced from the main actor
39.33         actomaton.state
39.33                   ^
39.33 /app/.build/checkouts/Actomaton/Sources/Actomaton/Actomaton.swift:11:29: note: property declared here
39.33     public private(set) var state: State
39.33                             ^
39.34 /app/.build/checkouts/Actomaton/Sources/Actomaton/MainActomaton.swift:54:19: error: actor-isolated property 'state' can not be referenced from the main actor
39.34         actomaton.state
39.34                   ^
39.34 /app/.build/checkouts/Actomaton/Sources/Actomaton/Actomaton.swift:11:29: note: property declared here
39.34     public private(set) var state: State
39.34                             ^

I have created a simple fix for this issue here


D6 - 3.2 Acceptance Tests

An acceptance test is a test that verifies the correct implementation of a feature from the user interface perspective. An acceptance test is a black box test (the system is tested without knowledge about its internal implementation). Provide the following information:

Test framework you used to develop your tests (e.g., Selenium, Katalon Studio, Espresso2, Cucumber, etc.)
Link to your GitHub folder where your automated acceptance tests are located
An example of an acceptance test. Include a GitHub link to the test and an explanation about the tested feature in your answer.
A print screen/video showing the acceptance test execution

Grading criteria (2 points): adequate choice of a test framework, coverage of the tests, quality of the tests, adequate example of an acceptance test, print screen/video showing successful test execution.

Fixed in #185


Overriding messages

Love the work you're doing with this...
I would love to override the 'pattern' message as it's not that user-friendly. I can't expect my users to understand the constraints of a regex! :)
Anyways, I have not been successful doing this as per the docs here. Could you please provide a pointer or a little help getting this working? I assume the custom error message resolver is the way to go? Is this necessary for overriding just 1 message?

This works for overwriting the error message for an input with 1 type of pattern, but usually my forms have multiple inputs with regex pattern attributes and I would like user-friendly error messages for each of them. Is there any way to get this to work?
Examples:
Phone Number:
Soc Security Number:
Zip Code:
Account Number:


No Python v3.12.7 support?

Why it does not support the latest stable Python build? I got error in ComfyUI log saying it wants WAY OLD Python version =<3.11 only!

After I re-installed it once again via manager it seems to work now.


problem while setting css (top,left) on load means setPosition

problem while setting css (top,left) on load means setPosition #

Could you provide sample code about that issue.

i have solved that issue thanks for replying me.
Regards,
Shaikh Shafay Shahid
Web Designer & Programmer
On Thu, Dec 7, 2017 at 9:46 PM, Atanas Atanasov<EMAIL_ADDRESS>wrote:

Could you provide sample code about that issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/atatanasov/gijgo/issues/212#issuecomment-350016675,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXeiBXhGFISG3Vsy2ar35ow8ERxDND9Zks5s-A9jgaJpZM4Q5liy
.


Is that possible add a feature of cli for studio?

The origin prisma tool can use studio command to show the database GUI.
https://www.prisma.io/studio

It needs work from the core Prisma team, as it currently only works with JS projects unfortunately.
I created an issue in the studio repo, feel free to upvote it.


fix: Expose DataAsset class publicly

Description
This pull request addresses ApiView review feedback that pointed out that an experimental model was not exposed publicly.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

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


last update/checked

Looked into this interesting data for the first time via the search on the website.
What I'm missing in the shown results (within site and in popup), is an information when the data was last updated/checked/synced against official resources.
This would give a much better feeling about the quality of the data.
If one think this even further, there could be 3 dates within each data set for

added to opensanction
last update/change
last check/sync

without this, one may be afraid, that there may be some pretty old datasets with db like in other projects... see:

Thanks, that's a really good prod: we have these three fields in the data but aren't showing any of them on the site :) Easy fix.

awsome! looking forward to it!

Added this to the entity page, definitely not winning any design prizes with that but it's visible now...


Create issue forms

At the very least, a "question" issue should point to making a Q&A discussion, instead.
docs

Closed by 7b0a559ace7af7fc758b9cc1d0acf9bb01d6b9c9, 2b199a4b471b0f59944f0612d8d2434ac86da2b7, 97dd220c50bb7337aa5c359f438cf400198425cb, and 2979c739daf093df218690677b728cf1ed8ab074.


Last ~10 seconds of each track not played

 Issue by jsopenrb
Tuesday Apr 18, 2017 at 16:59 GMT
Originally opened as https://github.com/plietar/librespot/issues/175

Right before the end of each track, the controlling device is sending Load command which makes the player jump right to the next track without waiting for the current track to finish. This happens when controlling from both Android and Windows.

As there are no other reports of this and there is no one with the bug to troubleshoot with, I'm closing this issue/ Feel free to open a new one if its still there.