Need help with ceph. Today all PVC unmounted and cant be create/mount again.

Describe the bug
Today our dev product stop working. All PVC cant mount, new cant create (stay in Pending state)
No any changes in CEPH/Kubernetes clusters.
There is a log from csi-provisioner
I0929 17:13:10.289585       1 controller.go:1317] provision "default/csi-cephfs-pvc" class "cephfs": started
I0929 17:13:10.289791       1 controller.go:573] CreateVolumeRequest {Name:pvc-d373ab9b-1497-4718-b9f4-783d1ef5a5d4 CapacityRange:required_bytes:1073741824  VolumeCapabilities:[mount:<mount_flags:"debug" > access_mode:<mode:MULTI_NODE_MULTI_WRITER > ] Parameters:map[clusterID:d26925a3-b527-4eb4-9b3a-23625d055104 csi.storage.k8s.io/controller-expand-secret-name:csi-cephfs-secret csi.storage.k8s.io/controller-expand-secret-namespace:ceph-csi-cephfs csi.storage.k8s.io/node-stage-secret-name:csi-cephfs-secret csi.storage.k8s.io/node-stage-secret-namespace:ceph-csi-cephfs csi.storage.k8s.io/provisioner-secret-name:csi-cephfs-secret csi.storage.k8s.io/provisioner-secret-namespace:ceph-csi-cephfs fsName:cephfs pool:cephfs_data] Secrets:map[] VolumeContentSource:<nil> AccessibilityRequirements:<nil> XXX_NoUnkeyedLiteral:{} XXX_unrecognized:[] XXX_sizecache:0}
I0929 17:13:10.290027       1 event.go:282] Event(v1.ObjectReference{Kind:"PersistentVolumeClaim", Namespace:"default", Name:"csi-cephfs-pvc", UID:"d373ab9b-1497-4718-b9f4-783d1ef5a5d4", APIVersion:"v1", ResourceVersion:"43895367", FieldPath:""}): type: 'Normal' reason: 'Provisioning' External provisioner is provisioning volume for claim "default/csi-cephfs-pvc"
I0929 17:13:10.296946       1 connection.go:182] GRPC call: /csi.v1.Controller/CreateVolume
I0929 17:13:10.296994       1 connection.go:183] GRPC request: {"capacity_range":{"required_bytes":1073741824},"name":"pvc-d373ab9b-1497-4718-b9f4-783d1ef5a5d4","parameters":{"clusterID":"d26925a3-b527-4eb4-9b3a-23625d055104","fsName":"cephfs","pool":"cephfs_data"},"secrets":"***stripped***","volume_capabilities":[{"AccessType":{"Mount":{"mount_flags":["debug"]}},"access_mode":{"mode":5}}]}
I0929 17:13:10.313760       1 connection.go:185] GRPC response: {}
I0929 17:13:10.313821       1 connection.go:186] GRPC error: rpc error: code = InvalidArgument desc = failed to get connection: connecting failed: rados: ret=-13, Permission denied
I0929 17:13:10.313862       1 controller.go:645] CreateVolume failed, supports topology = false, node selected false => may reschedule = false => state = Finished: rpc error: code = InvalidArgument desc = failed to get connection: connecting failed: rados: ret=-13, Permission denied
I0929 17:13:10.313931       1 controller.go:1084] Final error received, removing PVC d373ab9b-1497-4718-b9f4-783d1ef5a5d4 from claims in progress
W0929 17:13:10.313954       1 controller.go:943] Retrying syncing claim "d373ab9b-1497-4718-b9f4-783d1ef5a5d4", failure 6
E0929 17:13:10.313983       1 controller.go:966] error syncing claim "d373ab9b-1497-4718-b9f4-783d1ef5a5d4": failed to provision volume with StorageClass "cephfs": rpc error: code = InvalidArgument desc = failed to get connection: connecting failed: rados: ret=-13, Permission denied
I0929 17:13:10.314060       1 event.go:282] Event(v1.ObjectReference{Kind:"PersistentVolumeClaim", Namespace:"default", Name:"csi-cephfs-pvc", UID:"d373ab9b-1497-4718-b9f4-783d1ef5a5d4", APIVersion:"v1", ResourceVersion:"43895367", FieldPath:""}): type: 'Warning' reason: 'ProvisioningFailed' failed to provision volume with StorageClass "cephfs": rpc error: code = InvalidArgument desc = failed to get connection: connecting failed: rados: ret=-13, Permission denied

Aditional. Mount from any node with command:
mount -t ceph IP:6789:/ /mnt/cephfs -o name=,secret= is success

hey @mixMugz , Can you please add
ceph version:
cephcsi version:
Logs from

csi-cephfsplugin-provisioner pod -> csi-cephfsplugin container
csi-cephfsplugin pod(on the nodes with problem) -> csi-cephfsplugin container

Make sure,

ceph cluster is healthy
verify ceph fs commands work with the particular id and keys used for cephcsi

And check whether https://docs.ceph.com/en/latest/security/CVE-2021-20288/ is the cause of the issue.

Thanks for reply. Tonight it was fixed just changing all images to v3.4.0. Was v3.2.0


Adobe.Acrobat.Reader.32-bit - Changed installer to multi language installer

Changed installer to MUI version so it detects OS language and installs correct Adobe Reader in correct language

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

SEE >:O

@OfficialEsco I know that is in 32-bit version I already wrote that in 64-bit thread

Ahhhh i only saw this comment, not the edit

@OfficialEsco I did same change to 32 bit reader and that was just fine.

but when I try to create new package name like "Adobe.Acrobat.Reader.32-bit.MUI" it says:

ill wait for @ItzLevvie to share her thoughts, i'm not sure if there is a difference between the locale and MUI installer, but if there is we can't just overwrite their install

OK, no problem

Well just tried this myself but when I manually try to install MUI version over non-MUI version it says that I already have installed this product.


Getting quicker imagery

Is there any way of taking pictures quicker? E.g. for video recording? I don't need much fps. Like 5-10 would be fine. However one frame every 3 seconds in svga is simply not enough.

Have you looked at the benchmark? For jpeg and fb_count =2 you can achieve around 25fps on a S3 Chip

Alright, just tested it on another chip and it didn't have any kind of delay. Seems like a malfunctioning 2.4 Ghz antenna


Docs.MS Release Updates for azure-ai-textanalytics

Update docs metadata and targeting for release of azure-ai-textanalytics

Docs Build status updates of commit 3b149f9:
:clock10: Full build: the average full build time is 352 min(s) 36 sec(s), this is based on the last 1000 build(s) for this repository.


Not able to set "Content-Type" header as "text/plain" in traefik-forward-auth

Hi,
We are trying to set "Content-Type" header as "text/plain" before redirection.
For the following location
https://github.com/thomseddon/traefik-forward-auth/blob/4ffb6593d569801cf7c4571542c0ff03afcb1a0f/internal/server.go#L235
Although we are setting this header it's not reflected in the redirection call made to the server.
All of the following ways we have tried.
//[1]
w.Header().Set("Content-Type", "text/plain")
//[2]
r.Header.Set("Content-Type", "text/plain")

Any guidance for this is appreciated.
Thanks

What's your use case?


Relative neighborhood of a point cloud

https://en.wikipedia.org/wiki/Relative_neighborhood_graph

Closing in favor of #114 because apparently they're super close, and it appears as though the Urquhart graph might be cheaper to generate.


how can i use this with a up node and wallet?

i have running a node and wallet using docker and dont want to shut them down. how can i connect this with them ?

Hi @armagg. If you already have a way of starting the node and wallet, then you don't need cardano-launcher.

Oh! I thought that thisis additionally  an api for them. Thanks


add support for literal multi-line strings

Added support for the literal multi-line strings as described in the yaml specification:
https://yaml.org/spec/1.2.2/#812-literal-style.
Closes #31

Ok, thank should be fixed now.

Very nice @lovebug356 . Thank you. Sorry it took me so long to get back to this.


fix to handle if Tab has no href attribute

when tag of Tab has no href attribute, error thrown
Uncaught TypeError: Cannot read property 'charAt' of null

I signed it!

In what use-case would you not provide a reference?

In my case, I handle the tab being clicked by onClick, I didn't add a href and having this error Uncaught TypeError: Cannot read property 'charAt' of null

Hello there! We're currently doing some housecleaning on the repository to prepare for the big changes to v2, and part of this is to close the mdl-1.1 and mdl-1.2 branches, and move all 1.1 and 1.2 PRs to the mdl-1.x branch. Since it's not possible to target PRs at a new branch, we're closing this PR and asking you to reopen against mdl-1.x if you think it's still valid.
Thank you!


Display Ryuk logs when it fails to start

In an environment with multiple Docker daemons available (typically Jenkins running on a Kubernetes infrastructure), the Ryuk container can pick the wrong Docker socket. This can be fixed by using the TESTCONTAINERS_DOCKER_SOCKET_OVERRIDE environment variable.
However when such an error occurs - which was our case when we upgraded from version 1.14.2 to 1.16.2 of Testcontainers - it can be difficult to debug and understand what the root cause is, as the logs provided only refer to a 404 - No such container error.
We propose to display the logs of the Ryuk container in such a scenario - which are explicit about the docker socket error - so that users can understand faster how to fix their configuration.

Hey @vincz7777, good point (I regularly see users confused by this error message), and thanks for providing an improvement in this PR 👍

@vincz7777 @kiview I just applied a tiny change to make try {} catch {} smaller and catch more exceptions (just in case), should be good to go once CI is green 👍

@vincz7777 @kiview I just applied a tiny change to make try {} catch {} smaller and catch more exceptions (just in case), should be good to go once CI is green +1

Hi @bsideup  it seems the CI fails on the toxyproxy check. I'm not sure to understand how this could be related to the modifications of the PR. Could you help me on this one? Thanks in advance!

Hi @vincz7777! Seems to be unrelated (flaky test?), I just triggered a re-run 👍

The toxiproxy tests are failing on master too (and I've rerun), so I think we have a problem...


Use common pull request titles

[x] Have you signed the Contributor License Agreement?
[x] Are you working against an Issue?

Resolves #387

I also modified the default branch format to help find the branch more easily through git checkout <> (Tab completion) / git branch -a. This helps when a mod requests changes on a PR and the author wants to find the local branch and add changes

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


How do I configure my controller to work with cen64-qt?

I have a SteelSeries 3GC controller that I want to use to play an N64 game, but when I go into "Settings > Configure > Controllers" I do not see any way to assign button mappings for a controller.
I tried attaching a Controller Pak file and it didn't work. Is there something I'm missing there?

CEN64 doesn't yet have the capability to map buttons. It's keyboard only currently unless I've missed something.
If you're wanting an emulator that's currently stable enough to play games, I'd suggest Mupen64Plus.


Fix the method to wait to the next monkey

Before is done using Thread.sleep Now the delay is using the scheduller

Changes Unknown when pulling a3dd16abbaac71152dd1551c152e2aaf2134699b on fix/changeToScheduller into ** on master**.


Any plans to upgrade package to upgrade this software for umbraco 14?

Hi I wonder how long it is taking to update this software to update to Umbraco Version 14 which is currently in RC4 ...
Is there a plan to support it?
I'm someone what likes to try the newest version but at the moment when I upgrade to the new Umbrao version 14 I get the error that his package isn't available yet.
Any plans when ca. this update will come?
Kind regards,
Daniel

I am really curious as wel. Any news?

I've been struggling to find the time to move it to V14 so far, but I should have some time in the coming days/weeks. But because V14 is quite a big change in the backoffice, it might take a bit longer to move everything over.

It's awesome plugin and in Umbraco V14 have big improvements in GUI .
I"m hope that did plugin will support this version.
Thank you for awesome plugin .

Hey Patrick - how is the upgrade going?

Hey @patrickdemooij9 Can you tell me when we will get a new release that will be compatible with Umbraco 14?

Hi @patrickdemooij9 can we help?

Hi all!
Sorry for the late responses. I've just pushed a new branch: dev/main. If you are able to help out, that I would be greatly appreciated as it is quite a lot of work to get everything going especially with documentation lacking (in my opinion) a little bit.

Hi all!
Just wanted to give an update on the progress so far. Currently 2 modules have been moved over:

RobotsTxt
ScriptManager

I am currently working on the redirects and that should be done soon. After that, there are still 3 modules to be picked up:

SiteAudit
Sitemap
Meta Fields

Meta Fields will probably take the most time, but I am hoping the experience of the other modules will allow me to work a lot faster on those ones.
Thank you for your patience with this!


🐛🐛  audio error

wechaty==0.8.34
wechaty-grpc==0.20.19
wechaty-plugin-contrib==0.0.13
wechaty-puppet==0.4.0
wechaty-puppet-service==0.8.8
        if msg.type() == MessageType.MESSAGE_TYPE_AUDIO:
            audio_file = await msg.to_file_box()

receive error info <{'code': 9999, 'msg': 'call already closed'}>

This error is from puppet-server which is related to your token service. So, from the above description, we can't find the actual reason of error.

fixed


added the onRequestClose property for the modal

Using expo, I noticed that I got a yellow box warning regarding that a Modal should have the onRequestClose property. This PR sets the onRequestClose property equal to onCancel, so when the back button is pressed the onClose function is being called.

Thanks for this, see my comment.


Don't re-trigger the Plaid process if we already have a selected Plaid account

Context: https://github.com/Expensify/App/issues/15202#issuecomment-1453484477
Details

Fixed Issues

$ https://github.com/Expensify/App/issues/15202
$ https://github.com/Expensify/App/issues/15197
PROPOSAL: GH_LINK_ISSUE(COMMENT)
Tests

Adding a PBA via Settings > Payments

Sign in with any account that does not have an existing PBA
Navigate to Settings > Payments > Add payment method > Bank account
Add a PBA via Plaid
On the PBA success page click Continue
Observe that the Plaid flow is not triggered again

Adding a PBA via Sending Money

Have a user account under the wallet beta, with a silver wallet and with no PBA set up
Attempt to send money to another user
Select Pay with Expensify > Bank Account
Add a PBA via Plaid
On the PBA success page click Continue
Observe that the Plaid flow is not triggered again

Adding a PBA via deep link

Sign out if already signed in
Open https://localhost:8080/settings/payments/add-bank-account
Sign in with any account that does not have an existing PBA
Complete the plaid process and add a new account
On Add bank account page, select the bank account and click Save & continue
On the success page click Continue and verify the Plaid process is not triggered again

Offline tests

Not applicable
QA Steps

Same as test steps except they should be performed on staging
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
Via Settings > Payments
https://user-images.githubusercontent.com/12268372/223132991-319de55c-16af-4f8a-afd7-864e1def5cdc.mov
Via sending money
https://user-images.githubusercontent.com/12268372/223133045-607060e0-b2c1-4e7c-95a7-0704ba89ba0c.mov
Via deep link
https://user-images.githubusercontent.com/12268372/223134075-a0a7089f-eb2b-42e0-b261-6aad98e5c049.mov

Mobile Web - Chrome

Unable to test this platform as the redirect from Plaid leads to opening the native app

Mobile Web - Safari

Unable to test this platform as the redirect from Plaid leads to new.expensify.com

Desktop

https://user-images.githubusercontent.com/12268372/223136086-7df93420-1d06-4dd5-838e-91695dd89c43.mov

iOS

https://user-images.githubusercontent.com/12268372/223137943-32cd85c3-2683-449e-b87a-50a2e770f2c1.mov

Android

https://user-images.githubusercontent.com/12268372/223145694-71a07b23-0a38-40ef-84ac-379724eaa691.mov

@puneetlath @Santhosh-Sellavel One of you needs to 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]

@MariaHCD I'll investigate this again, if no luck lets merge this tomorrow!

bump @MariaHCD @nkuoch

Apologies for the delay here! Updated and re-tested!

Console errors on native while adding account

https://user-images.githubusercontent.com/85645967/226204749-4feb43b2-c416-4ee5-bd17-4435484a6639.mov

Reviewer Checklist

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

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] 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)
[ ] 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)

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

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

[ ] 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
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] 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).
[ ] I verified that comments were added to code that is not self explanatory
[ ] 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.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] 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.
[ ] 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.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] 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)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] 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
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] 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)
[ ] 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)
[ ] All JSX used for rendering exists in the render method
[ ] 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

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

[ ] 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

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

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

[ ] 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)
[ ] 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.
[ ] 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.
[ ] 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.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web & Mobile Web - Chrome & Mobile Web - Safari
Unable to test due CORS

Desktop
https://user-images.githubusercontent.com/85645967/226204848-26c1c222-8823-4b8c-a8bb-3555227128d8.mov

iOS & Android
Uploading Screen Recording 2023-03-20 at 1.04.06 AM.mov…

Adding a PBA via Settings > Payments -  ✅ Pass
Adding a PBA via deep link - Fail ❌
Uploading Screen Recording 2023-03-20 at 1.37.26 AM.mov…

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

🚀 Deployed to staging by https://github.com/nkuoch in version: 1.2.88-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.2.88-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


ffs: cid activity log

When you click on a file or folder, you should be able to see Activity relating to the file

@jsign, are the logs for a cid persisted? If so, could we provide a way to query the log items from the past? This could be a typical request response thing or a channel with a buffer that gets filled immediately with item from the past in the current WatchLogs.

@jsign, are the logs for a cid persisted? If so, could we provide a way to query the log items from the past? This could be a typical request response thing or a channel with a buffer that gets filled immediately with item from the past in the current WatchLogs.

Yes, they're persisted so past logs are recoverable.
Sure, we can make a new API for that.

This is now supported through the ffs.watchLogs api. You need to pass the withHistory(true) option and the listener will immediately receive all previous log events, and then will continue to receive any new log events


API User can create a Deposit Request

This PR adds a StellarClient::Client.deposit method that implements Stellar's 006 Anchor/Client Deposit feature.

🚀


Editor field close button doesn't close editor.

Issue description:
As the title says the close button of the editor field doesn't close the editor, thus the only option is to close the customizer panel. Everything else about the editor works fine.
Seems like a js bug in the button.
Version used:
Using latest Version 3.0.11, although i had the same problem with Version 3.0.10
Using theme_mods or options?
theme_mods
Code to reproduce the issue (config + field(s))
add_action( 'after_setup_theme', 's_tests_fields' );
function s_tests_fields()
{

	Kirki::add_field( 'my_config', array(
		'type'			=> 'editor',
		'settings'		=> 'block-about-content',
		'description'	=> __( 'Write your bio or other content for this section.', 'andy' ),
		'section'		=> 'tests',
		'default'		=> 'Default text',
		'priority'		=> 10,
		'sanitize_callback'	=> 'wp_kses_post',
	));
}

Tests using your _s theme and a WP fresh install.

@enginne, I discovered the shortfix for it, but @aristath should be aware of it, I tested and it worked.
https://github.com/aristath/kirki/blob/develop/controls/editor/editor.js:76
        /**
	 * Toggle the editor.
	 */
	toggleEditor: function() {

		var control = this,
		    editorWrapper = jQuery( '#kirki_editor_pane' );

		var isHidden = editorWrapper.hasClass('hide');

		editorWrapper.removeClass();
		if ( isHidden ) {
			editorWrapper.addClass( control.id );
		} else {
			editorWrapper.addClass( 'hide' );
		}
	},

Regards,

fixed in #1596

v3.0.12 refactored the editor field and now implements the WP-Core control which is added in WordPress 4.9 (to be released in a few days).


Class Extends Function: Extra Newness

I present this as an oddity for your amusement.
class Foo extends Function {
  constructor(val) {
    super();
    this.prototype.val = val;
  }
}

new new Foo(':D')().val // => ':D'

Constructors in Javascript are of course just functions with some special treatment.  by extending Function using the ES6 class syntax you create a class that, when instantiated, is now a function, which you can then additionally instantiate.
While not exhaustively tested, I believe the last JS statement can be analyzed thus:
(new (new Foo(':D'))()).val
(new newFooInstance()).val
veryNewFooInstance.val
':D'

As a tiny addendum, doing new Function('return "bar";') of course creates a function with the body return "bar";.  Since super() in the constructor of our Foo class is calling Function's constructor, it should come as no surprise now to see that we can additionally manipulate things in there.
class Foo extends Function {
  constructor(val) {
    super(`
      this.val = arguments[0];
    `);
    this.prototype.val = val;
  }
}

var foo = new new Foo(':D')('D:');
foo.val // -> 'D:'
delete foo.val; // remove the instance prop 'val', deferring back to the prototype's 'val'.
foo.val // -> ':D'

Added to the list! Thanks a lot!


List data type field FromJson not getting correctly generated

Input file
import 'package:genq/genq.dart';

part 'genq_test.genq.dart';

@Genq(json: true)
class User with _$User {
  factory User({
    @JsonKey(name: 'full_name') required List<String> name,
  }) = _User;
}

Current generated code

User $UserFromJson(Map<String, dynamic> json) {
  return User(
    name: List.of(json['full_name']).map((e) => e as String).toList(),
  );
}

As could be seen json['full_name'] data type would be dynamic, so we can't loop over it, we would need to specifically cast it be iterable. The dart analyzer will give error
The argument type 'dynamic' can't be assigned to the parameter type 'Iterable<dynamic>'. dart[argument_type_not_assignable](https://dart.dev/diagnostics/argument_type_not_assignable)
Expected generated code

User $UserFromJson(Map<String, dynamic> json) {
  return User(
    name: List.of(json['full_name'] as Iterable).map((e) => e as String).toList(),
  );
}

There were some more improvement which could be done by taking inspiration from freeze, not reporting right in this for now.
Btw, this is great initiative to improve the code gen experience for classes. Today came across the package thought to give it try, as currently when using the freeze with build running is taking ~3 min per build and can go up to ~5-8 min some times.
While macros in dart around the corner, expected to be released by early 2025 and all build runner related package expected to move to macros. Still it will good to have alternate.
Will try my best to contribute to it and improve it.
Thank you again.

Hello, sorry for the long delay.
Likely this is caused due to the following setting in the analysis_options.yaml:
 include: package:lints/recommended.yaml
 analyzer:
   language:
>    strict-casts: true

The generated code by genq heavily relies on passing the dynamic values along without any casting. This makes code generation simpler and faster, as we do not have to lookup the types of any method parameters during codegen. While we could cast the json['full_name'] as Iterable in your example, the above rule would apply in other cases as well where we would not really know the type to cast to.
For example, a user is able to specify a custom fromJson method, where he can basically specify an arbitrary type as the parameter. genq does not have the ability to figure out this parameter type, so we cannot generate the correct casting. Thus, we would need to broaden the type to something more general, like Object, and move the actual casting into the functions (in this example, the user would need to do the casting in his custom fromJson).

TLDR: I would suggest you exclude **.genq.dart from analysis. genq won't have to change its architecture, and you will be able to happily generate the code.
include: package:lints/recommended.yaml
analyzer:
  exclude:
    - '**.genq.dart'

I think we should document this in the README though, as this is a common enough setting. You shouldn't be the only one bumping into this problem.

Thank you for the response, and this is great to know that there is work around.
While earlier was trying to migrate from the old project to new. Due to earlier shared case, didn't proceed with that plan.
In couple of months planing to start new project in that project will certainly going to give it another try.


Fix "Om Samfundet"-page overflow-x

Possible CSS issue, where it is possible to scroll to the right.

@Snorre98 I believe this is fixed, could you confirm?


"New release" dialog should not close when clicking outside its frame

Describe the solution you'd like
The dialog that slides in when a new release is to be submitted should not close when clicking outside the frame of said dialog.
Describe alternatives you've considered
Alternatively, it should keep a draft of what has been edited so far, so it can be recovered when it the dialog was closed by accident.
Additional context
It is irritating that all progress (e.g. extensive release notes) is lost when the user accidentally clicks outside the frame and the dialog closes.

This is a design pattern across all App Center. Will discuss with our design team to improve the user experience around this.

pls, current behavior annoying. You need to upload and write description again after random click outside.


Please document details of response to GetMeetingDetails

For example:

What are the possible values of meeting.type
What is the ID in msGraphResourceId-- which Graph APIs can this ID be used for?

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

ID: 805dc242-7b5a-cc81-ee0e-6acf0078a653
Version Independent ID: 11c0e023-2514-9639-6052-2cbbbef50663
Content: Meeting apps API references - Teams
Content Source: msteams-platform/apps-in-teams-meetings/API-references.md
Product: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

@aosolis  - We are looking into this I will get back to you soon.

@aosolis  - Please have a look at this doc- https://docs.microsoft.com/en-us/graph/api/onlinemeeting-get?view=graph-rest-1.0&tabs=http
https://docs.microsoft.com/en-us/graph/api/resources/onlinemeeting?view=graph-rest-1.0
Let us know if you have any query.

@aosolis - Could you please confirm if your issue has resolved with above suggestion or still looking for any help?

Thanks for the pointer!
Please update the documentation to make this clear.
Also, how does one get the ID of the corresponding calendar event?

@aosolis - for Getting id of calendar  event use the following Graph API:-
1.GET /me/calendar/events
Please have a look at this doc-https://docs.microsoft.com/en-us/graph/api/user-list-events?view=graph-rest-1.0&tabs=http

@aosolis - Could you please confirm if your issue has resolved with above suggestion or still looking for any help?

@aosolis - We will check as per suggestions and ask doc team to update it accordingly.

@aosolis - The possible values of meeting type is "Private Meeting , Channel Meeting".
https://docs.microsoft.com/en-us/MicrosoftTeams/tutorial-meetings-in-teams?tutorial-step=2
for the msGraphResourceId-  we are check it internally ,we will inform you once we get any update.
Will raise a bug to Document Team for the necessary changes to be made in the document.

@v-abirade, please look into this.

@surbhigupta12 , I will update the doc and keep you posted.


Null check operator used on a null value.

I received the folllowing error,while loading the example.
`════════ Exception caught by widgets library ═══════════════════════════════════
The following _CastError was thrown building BabylonJSViewer(dirty, state: _BabylonJSViewerState#21ec1):
Null check operator used on a null value

The relevant error-causing widget was
BabylonJSViewer
lib/ui/sample.dart:23
When the exception was thrown, this was the stack
#0      _BabylonJSViewerState.build
package:babylonjs_viewer/src/babylonjs_viewer.dart:61
#1      StatefulElement.build
package:flutter/…/widgets/framework.dart:4691
#2      ComponentElement.performRebuild
package:flutter/…/widgets/framework.dart:4574
#3      StatefulElement.performRebuild
package:flutter/…/widgets/framework.dart:4746
#4      Element.rebuild
package:flutter/…/widgets/framework.dart:4267
...
═══════════════════════════════════════════════════════════════════════════════`

Which version of dart sdk and flutter are you using, and which version of this library?

The error comes from this _proxy variable

My SDK versions:
Flutter 2.2.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision d79295af24 (6 days ago) • 2021-06-11 08:56:01 -0700
Engine • revision 91c9fc8fe0
Tools • Dart 2.13.3

thanks for the #2 merged, I'm trying to fix issue same time


There is a large difference between the memory estimated by Accelerate and the actual memory consumption during runtime inference.

System Info
1. memory estimated
 trust_remote_code=True accelerate estimate-memory neuralmagic/Meta-Llama-3.1-8B-Instruct-FP8 --trust_remote_code
2. runtime-infer memory
lm_eval \
  --model vllm \
  --model_args pretrained="neuralmagic/Meta-Llama-3.1-8B-Instruct-FP8",dtype=auto,add_bos_token=True,max_model_len=4096,tensor_parallel_size=1 \
  --tasks mmlu \
  --num_fewshot 5 \
  --batch_size 1

Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction

memory estimated
trust_remote_code=True accelerate estimate-memory neuralmagic/Meta-Llama-3.1-8B-Instruct-FP8 --trust_remote_code
runtime-infer memory
lm_eval
--model vllm
--model_args pretrained="neuralmagic/Meta-Llama-3.1-8B-Instruct-FP8",dtype=auto,add_bos_token=True,max_model_len=4096,tensor_parallel_size=1
--tasks mmlu
--num_fewshot 5
--batch_size 1

Expected behavior
The memory estimation is not much different.

Could you please report what number you're getting and how much memory is actually being used? Remember that this calculation is for the minimum amount of memory used and does not account for the total required memory (e.g. from hidden states/activations).


Adding a SDS scenario to our existing upgrade tests

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities/
To ask questions about how to use Istio, please visit https://discuss.istio.io
)
Bug description
As per  @duderino 's comment in #15020 , we should have a SDS scenario in our upgrade tests.
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[X] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Version (include the output of istioctl version --remote and kubectl version)
1.2/1.3

Hi Jimmy, could you add this to your Code Mauve doc, and let's prioritize and delegate the work.

Hi Jimmy, could you add this to your Code Mauve doc, and let's prioritize and delegate the work.

Sure! Will do.


Define paths via options

Are there any plans to implement support for defining paths via options in the the proto definition - like grpc gateway does it?
It would essentially mean Twirp would provide a single server for grpc and what grpc gateway does and also support json - all at once.  Even swagger would then work with twirp(like it does with grpc gateway).
In current state, what Twirp produces is unusable for public consumption, in my case, but it has good potential.

No, we have no plan to do this. Twirp's fixed route scheme makes it much easier to build generators in different languages, and makes it simple to cURL Twirp endpoints. I'm pretty happy with the fixed routing scheme.
Could you say more on why "what Twirp produces is unusable for public consumption?" Why does the public care about your paths?


chore(*): update CoreOS to 766.4.0

See https://coreos.com/releases/#766.4.0

Would it be possible to make Vagrant grab the COREOS_VERSION from contrib/utils.sh?

@sgoings I think it's possible but not totally required for this to be merged. Would be a nice feature so we have one less version number to bump, though.

Would it be possible to make Vagrant grab the COREOS_VERSION from contrib/utils.sh?

That would be great; I'll put an issue in to track that.


[aksel.nav.no] Fix bug when clicking on active code example chip

https://nav-it.slack.com/archives/C04C1G7UWG7/p1730287562264449
Turn on "Hide whitespace" when reviewing.

Btw, onLoad on the iframe doesn't fire on page load in dev, any idea why?

Btw, onLoad on the iframe doesn't fire on page load in dev, any idea why?

@HalvorHaugan Works on my machine


Use erb templates for additional commodity code outcome titles

I've introduced support for using ERB templates to render Outcome titles, and updated the additional-commodity-code Smart Answer to use an ERB template.
There's currently some duplication between OutcomePresenter#title and OutcomePresenter#body but I plan to address that separately.

Did you consider implementing this using something like ActionView::Helpers::CaptureHelper#content_for?

Did you consider implementing this using something like ActionView::Helpers::CaptureHelper#content_for?

I certainly considered it but assumed it would be more work than the approach I've gone with at the moment. Do you think it's OK to go with my approach for now and consider a switch to something like content_for later?

I found the commit title confusing. I would've written "Add support for ERB templates for Outcome titles".

I'll update it.

Yes, I think your approach is fine for now. I just thought it would be good to record if we'd discarded the idea.

LGTM. :+1:

We just spoke about this, but I thought it was worth recording that the motivation behind making this change was so that we could remove all the outcome-related keys from the locale file.


Add pep621_pyproject.toml

Add pep621_pyproject.toml

This TOML file complies with PEP517 & PEP518 & PEP621

This link is for the PYPI package generated by this TOML file.

https://pypi.org/project/locust-je-test-pep621/2.15.2.dev8/

I don't want to overwrite old pyproject.toml so i call this file pep621_pyproject.toml
PEP621

https://peps.python.org/pep-0621/

Nice. But go ahead and add it to pyproject.toml, otherwise it will not be tested before merging etc. We definitely dont want to have both at the same time. Is this compatible with Python 3.7 btw?
You'll need to merge this with latest master to get the build working (because of this https://github.com/locustio/locust/pull/2337)

Is this compatible with Python 3.7 btw?
Sure, but we need upgrade build & setuptools (setuptools>=61) first

PEP621 allows pyproject.toml to generate the package on its own, but CI/CD requires setup.cfg so I didn't delete it.

Hm. If we'd have to maintain duplicate dependencies in setup.cfg I dont think this is worth it.

Look like we don't need  setup.cfg & setup.py anymore

That just ensures pip cache uniqueness, an optimization for faster (repeated) builds. I think you can replace 'setup.cfg' with 'pyproject.toml'

Makefile needs to be adjusted too, as this doesnt work anymore:
build:
	rm -f dist/* && python3 setup.py sdist bdist_wheel

While you are at it, you can nuke examples/vagrant/vagrant.sh.
It contains a reference to setup.py which will stop working now. And the file hasnt been updated in 11 (!) years :)

Hmm...

We need "pip install --upgrade build" first

We need "pip install --upgrade build" first

Yea, I was in a virtualenv and forgot to install it. Oops. Do you wanna remove vagrant.sh or should I?

Good stuff, thanks!


enable keyboard cursors in suggest

it should be possible by default to use the up/down cursors to navigate inside suggest

Just a little update after some years :)

The first problem with the cursor keys movement should be already solved. At least it worked for me when I tried this some days ago.
For the second problem with the enter key I just filed a pull-request: #96 - It's a simple fix on suggest.js

So when you merge the PR this issue most likely can be closed.

:+1:


Footer 1 dark.

One needs to make this component using HTML and Tailwind CSS. I would suggest using Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

After building the component please raise a PR with a screenshot of the component and add the component in the path of the file.
If you need to use any icon please use it from Hero Icons
Good luck.

hey @Rajdip019, I am interested in this particular issue. Kindly assign it to me!


fix for low percentage of outliers

Hello Bryan,
Thanks for this great tool! I just started to run it yesterday for the first time and ran into this small issue. So here is a quick fix so that truncation of sample array to zero does not occur anymore.

Awesome! I'm not at a computer right now but it looks good to me.


Graph node crash on Rinkeby subgraph indexing after some period of time

Report a bug?
We just deployed ocean-subgraph v3 for rinkeby and also rewrite start block to be sync faster.
sed -i 's/startBlock:.*/startBlock: 10762423/' subgraph.rinkeby.yaml

It started to sync and finished after some hours. Then it worked fine ~ 1 day and after that go into restart-loop.
We used Erigon and then moved to Infura to check if it relates to the RPC endpoint, but nothing changed.
It is repeated the second time in our configuration.
What is the current behavior?
Index node crashes after some period of time - on second day after it was already synched.
/usr/local/bin/start: line 52:    28 Aborted                 (core dumped) graph-node --node-id "${node_id//-/_}" --postgres-url "$postgres_url" --ethereum-rpc $ethereum --ipfs "$ipfs"

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.

Rewrite subgraph start block
Deploy subgraph
Follow the progress
Get the error after some period of time

What is the expected behavior?
Index node should continue to index subgraph
Workaround
We rolled-back to the graph-node v.0.25.2 and it started to work - let's wait some days, because v.0.26.0 also crashed only after some period of time. But at least it was able to start, unlike the latest version.

Hi - what OS are you running graph node on?
On Wed, 15 Jun 2022, 20:07 air3ijai, @.***> wrote:

Report a bug?
We just deployed ocean-subgraph
https://github.com/oceanprotocol/ocean-subgraph/tree/v3 v3 for rinkeby
https://github.com/oceanprotocol/ocean-subgraph/blob/v3/subgraph.rinkeby.yaml
and also rewrite start block to be sync faster.
sed -i 's/startBlock:.*/startBlock: 10762423/' subgraph.rinkeby.yaml
It started to sync and finished after some hours. Then it worked fine ~ 1
day and after that go into restart-loop.
We used Erigon and then moved to Infura to check if it relates to the RPC
endpoint, but nothing changed.
It is repeated the second time in our configuration.
What is the current behavior?
Index node crashes after some period of time - on second day after it was
already synched.
/usr/local/bin/start: line 52:    28 Aborted                 (core dumped) graph-node --node-id "${node_id//-/_}" --postgres-url "$postgres_url" --ethereum-rpc $ethereum --ipfs "$ipfs"
[image: Screenshot 2022-06-15 at 21 05 22]
https://user-images.githubusercontent.com/88528265/173895074-c894f056-21e1-4356-bbcf-cd35e89ca64e.png
If the current behavior is a bug, please provide the steps to reproduce
and if possible a minimal demo of the problem.

Rewrite subgraph start block
Deploy subgraph
Follow the progress
Get the error after some period of time

What is the expected behavior?
Index node should continue to index subgraph
Workaround
We rolled-back to the graph-node v.0.25.2 and it started to work - let's
wait some days, because v.0.26.0 also crashed only after some period of
time. But at least it was able to start, unlike the latest version.
—
Reply to this email directly, view it on GitHub
https://github.com/graphprotocol/graph-node/issues/3668, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACJK5LGYOQSWFJ45EGAE6P3VPILUTANCNFSM5Y4F37EA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hello, we are running in Kubernetes using AWS EKS in US East (N. Virginia).
AMI ID: ami-02da89ddc4cc9ed18
AMI name: amazon-eks-node-1.22-v20220526
Source: amazon/amazon-eks-node-1.22-v20220526
Architecture: x86_64
Description: EKS Kubernetes Worker AMI with AmazonLinux2 image, (k8s: 1.22.6, docker: 20.10.13-2.amzn2, containerd: 1.4.13-2.amzn2.0.1)

Amazon Linux 2 is a Linux operating system from Amazon Web Services (AWS). It provides a security-focused, stable, and high-performance execution environment to develop and run cloud applications.

Check if you're not running out of memory or disk space.

During next 5 hours we saw more than 40 pod restarts.
Increased pod memory
          resources:
            limits:
              cpu: 1000m
              memory: 2000Mi
            requests:
              cpu: 1000m
              memory: 2000Mi

And it started to work. Then we decreased resources slightly - 500/1500. It at least started and continued to index. Let's see the progress.
Not clear why pod restarted if at the moment of restart we see < 200 MiB usage and limit is 1000 MiB

During last ~ 47 hours we have ~ 138 restarts and pod can't start with same error.
Increased CPU and Memory limits
          resources:
            limits:
              cpu: 1500m
              memory: 2500Mi
            requests:
              cpu: 1500m
              memory: 2500Mi

It at least started and we can continue indexing, let's see the progress.

We notes just 1 restart during last 4d6h and all looks good. So, the solution is to provide more resources to the container.
In our case, minimal working resources configuration is the following
v.0.25.2
cpu: 500m
memory: 1000Mi

v.0.26.0
cpu: 1500m
memory: 2500Mi

Let's consider this as a solution or maybe a workaround.
Is there any special requirements for 3 x CPU / 2.5 Memory increase in the latest version?


Please invite me to the community

Name
tadekdev
Additional Context
I hope to join this community

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is a remarkable open-source solution tailored for serverless platforms, aimed at empowering creators to craft their infrastructure solutions. 💻 We're thrilled to welcome you into our vibrant community! ✨ We eagerly anticipate your valuable contributions! 🤓 Don't forget to explore our community on Github and connect with us on LinkedIn 💜 Feel free to spread the word about our community to like-minded individuals interested in development, DevOps, technology, and collaborative contributions. 👉💜👈


Snapshot version not working

While using the snapshot version, gradle build fails:
dependencies {
compile 'org.florescu.android.rangeseekbar:rangeseekbar-library:0.4.0-SNAPSHOT'
}
It says:
Error:(51, 13) Failed to resolve: org.florescu.android.rangeseekbar:rangeseekbar-library:0.4.0-SNAPSHOT
Show in FileShow in Project Structure dialog
Please update the doc if already fixed..

Sample builds on CI so it should be fine: https://travis-ci.org/anothem/android-range-seek-bar/builds/113207989
Have you added the snapshots repository?
https://github.com/anothem/android-range-seek-bar/blob/master/rangeseekbar-sample/build.gradle#L3-L5


Windows XP Support ?

if is supported?

.Net 4.5 cannot be installed to windows XP. So its not supported.

Thanks


Remove NB_UID ENV command

The NB_UID environment variable doesn't need to be set anymore as we are
handling the UID for the running user in the corresponding helm chart now. This
is better because setting the NB_UID in the docker file causes the upstream
image to chown the home directory to this uid, but this is slow and gets
slower as the users have more files in their home directory.
We are now making the cluster launch the pod with the running user as 1001 so
this chown is not required

will be merged after helm chart is updated

related helm chart PR: https://github.com/ministryofjustice/analytics-platform-helm-charts/pull/262/files


feat: query string field with IN

Adds feature to query the string field using in:
allPost(where: {
  title: {
    in: ["Crimes in Concrete", "Fast Compilers for Fast Programs"]
  }
}) {
  edges {
    ...
  }
}

Also adds neq to the operators object, it was missing before

Thank you very much @pippinthegreat !
Very nice work, I will merge and then clean up some inconsistency with the keys used to filter.

Sorry for the ever long wait


Issue Opening .hdf5 file

Hi,
I have a scRNA-seq .csv file that I have converted to .hdf5 format that is fairly large that is causing an error with scDeepCluster. I had to take a custom function from here in order to convert it to .hdf5 format and I believe it is causing the error I am seeing. You mention in the documentation to include an X component and I am not sure how to do that within my setup. Happy to share my conversion code and data. Thanks for such a cool tool! A screenshot of the error is attached:

Andrew

Hi,
According to the error message, it seems you didn't create a "X" object in the h5 file. Here is my R sample code to save h5 file:

h5createFile('data.h5')
h5write(counts, "data.h5", "X")
h5write(rownames(counts), "data.h5", "genes")
h5write(colnames(counts), "data.h5", "cells")
h5write(labels, "data.h5", "Y")

hi，i'm still confused,I only have the expression datafile .how to get the labels？Thanks

hi，i'm still confused,I only have the expression datafile .how to get the labels？Thanks

You don't need the labels. Labels are only used for evaluating the clustering performance (ARI, NMI, etc). Just store expression as "X" matrix in the h5 file.
Thanks.

hi，i'm still confused,I only have the expression datafile .how to get the labels？Thanks

You don't need the labels. Labels are only used for evaluating the clustering performance (ARI, NMI, etc). Just store expression as "X" matrix in the h5 file. Thanks.

oh！that's right.thank you for the reply.

excuse me，hi！when i run the code in my own data,the error said
Make sure that the dataset (adata.X) contains unnormalized count data,
so is there any suggstion to solve the problem? only the unnormalized count data could be used?
Thanks!!

excuse me，hi！when i run the code in my own data,the error said
Make sure that the dataset (adata.X) contains unnormalized count data,
so is there any suggstion to solve the problem? only the unnormalized count data could be used?
Thanks!!

Yes, only discrete counts are accepted as input. scDeepCluster uses ZINB distribution to characterize scRNA-seq data. I think the error message is given by scanpy package. You can trim you data to discrete counts.

I'm sorry to bother you again. When I  trim my data later,the error is attached.
TypeError: void() takes exactly 1 positional arguments (0 given)
it drives me crazy now. I am grateful for your kind assistance.
Thank you again！！

I'm sorry to bother you again. When I trim my data later,the error is attached.
TypeError: void() takes exactly 1 positional arguments (0 given)
it drives me crazy now. I am grateful for your kind assistance.
Thank you again！！

What line generates the error message? Could you also present a sample of your data?

Thank you for the reply！！there are the pictures about error message and my data.may them could help you to understand.if i need to do something else,please tell me.Thanks!!!!!

and if I just store expression as "X" matrix in the h5 file. There are noting wrong, everything  goes well.

Thank you for the reply！！there are the pictures about error message and my data.may them could help you to understand.if i need to do something else,please tell me.Thanks!!!!!

The error message comes from scanpy package. Could you tell me the version of your scanpy package?

The scanpy versin is 1.9.3.which version of scanpy could be OK?Thank you for the reply.

i see，it's all my fault. I will change the version to 1.7.2.Thanks

i have changed the scanpy version to 1.7.2,but it didn't work.so sad.

i have changed the scanpy version to 1.7.2,but it didn't work.so sad.

I can't see any problem of your screenshot. Could you share a sample data with me? Meanwhile, I noticed that your data is so small, there are only 90 cells. It may not a good idea to apply deep learning to a so small dataset.

Sure，my email count<EMAIL_ADDRESS>could you tell me your e-mail?my data is small cause I just want to test my datafile is OK or not. I will change the dataset if everything goes right, Thanks.

i have changed the scanpy version to 1.7.2,but it didn't work.so sad.

I can't see any problem of your screenshot. Could you share a sample data with me? Meanwhile, I noticed that your data is so small, there are only 90 cells. It may not a good idea to apply deep learning to a so small dataset.

My email address is on my GitHub page

Sure，my email count<EMAIL_ADDRESS>could you tell me your e-mail?my data is small cause I just want to test my datafile is OK or not. I will change the dataset if everything goes right, Thanks.

I find your problem. You store "y" as an array of vectors. Here is your "y" vector:
array([(1.,), (1.,), (1.,), (1.,), (1.,), (1.,), (2.,), (2.,), (2.,),        (2.,), (2.,), (2.,), (3.,), (3.,), (3.,), (3.,), (3.,), (3.,),        (3.,), (3.,), (3.,), (3.,), (3.,), (3.,), (4.,), (4.,), (4.,),        (4.,), (4.,), (4.,), (4.,), (4.,), (4.,), (4.,), (4.,), (4.,),        (4.,), (4.,), (4.,), (4.,), (4.,), (4.,), (4.,), (4.,), (5.,),        (5.,), (5.,), (5.,), (5.,), (5.,), (5.,), (5.,), (5.,), (5.,),        (5.,), (5.,), (5.,), (5.,), (5.,), (5.,), (6.,), (6.,), (6.,),        (6.,), (6.,), (6.,), (6.,), (6.,), (6.,), (6.,), (6.,), (6.,),        (6.,), (6.,), (6.,), (6.,), (6.,), (6.,), (6.,), (6.,), (6.,),        (6.,), (6.,), (6.,), (6.,), (6.,), (6.,), (6.,), (6.,), (6.,)],       dtype=[('V1', '<f8')])
However, a preferred "y" should be a 1-D vector such as [1, 1, 1, ...]


changed request url builder key dict

Not sure why the travis check failed here. Anyone have an idea?


Use profile to enable logging output for gradlew test

@mfussenegger Please review
We can also do it with a new task instead of profile.

lgtm


Update build script to pack '@tensorflow/tfjs-node-gpu'

This change is

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Sample App not working

Describe the bug
Running Sample Project crashes when searching for devices in iOS sample.
To Reproduce
Steps to reproduce the behavior:
Download and run sample to iOS divide with iOS 16. Search fro devices
Expected behavior
A list of BT devices to appear
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Mac OS 13
Browser Safari
Version N/A

Smartphone (please complete the following information):

Device: iPhone 13
OS: iOS 16
Browser Safari
Version [e.g. 22]

Additional context
Crashes in Main.cs

Where can we get this sample app?

Added you to the repo @nissaba

@nissaba any luck?


Canceled codes in multiscale_saliency_grad.m

I've been told that there are many canceled codes in multiscale_saliency_grad.m . Now I've updated a new code and this issue has been fixed.

As the corresponding problem has been fixed, I'll close this issue. Please open a new issue if you have any other questions.


Is there a way to get the most 24h_change

I wonder how easy is to request the top 10 of 24_change or other category like that, recursively.

You'd have to loop over all the currencies and order 24h_volume_usdwith sorted(24h_volume_usd, key=int, reverse=True). Hope this helps.


Filter order

Hi,
I am aplying a custom filter to a texture.
let rawSprite = new PIXI.Sprite(texture);
//....
var dicomShader = new PIXI.Filter('',
    window.warp.resources.frag.data,
        {
            photometric_interpretation: photometric,
            pixel_representation: representation,
            bytes_per_pixel: bytes,
            window_center: frame.windowCenter,
            window_width: frame.windowWidth,
            rescale_intercept: frame.rescaleIntercept,
            rescale_slope: frame.rescaleSlope,
        }
    );
rawSprite.filters = [dicomShader];

let actualWidth = rawSprite.width;
let actualHeight = rawSprite.height;

rawSprite.width = 146;
rawSprite.height = actualHeight * (146/actualWidth);

I split, from the server, 2 byte grayscale pixels into Red channel and Green channel (RGB). It works well, but I need to avoid resize image before the filter. I noticed that PIXI first resize image and then apply the filters. Is there an option to revert the order. If it is not possible to do, how I can apply filter "offline" and the use the filtered image to resize.
Thanks!

Are you sure you need a filter? Filter is applied to result of rendering of whole container. If you dont need that, you can use mesh-shaders, not filter: https://pixijs.io/examples/#/mesh-and-shaders/triangle-textured.js , its easier. But even then, it will be applied to result of texture filtering (linear, clamp, whatever). You have to store result of your mesh-shader or filter in renderTexture if you want to apply any shader on top of it, not dealing with scaling

@idria was Ivan's suggestion sufficient for you?

Looks like we can close this issue


Is there an open source plan for the Megatron framework witch used on aria's training

The work of using megatron to implement expert parallelism on multimodal models is fantastic, and I am confident that the community greatly needs technical support in this area. Do you have any plans to open source this part of the code.

@ghntd At this time, we do not have plans to open source this part of the code.


[fix](dynamic partition) fix create too many dynamic partitions #30994

cherry pick from #30994

run buildall


Okhttp canceled the request on its own

I use okhttp 4.12.0 to upload images to K8S. It took 40 minutes to upload the file when more than half of the network bandwidth was occupied. According to the Kubernetes logs, one of the image upload requests was canceled by okhttp. The error stack of Okhttp is TimeoutException.
Sorry, this is an occasional question. So no failed test cases are provided.

com.xxx.ThrowableFacade:timeout
at okio.SocketAsyncTimeout.newTimeoutException(JvmOkio.kt:146) ~[okio-jvm-3.6.0.jar:?]
at okio.AsyncTimeout.access$newTimeoutException(AsyncTimeout.kt:161) ~[okio-jvm-3.6.0.jar:?]
at okio.AsyncTimeout$sink$1.write(AsyncTimeout.kt:342) ~[okio-jvm-3.6.0.jar:?]
at okio.RealBufferedSink.emitCompleteSegments(RealBufferedSink.kt:256) ~[okio-jvm-3.6.0.jar:?]
at okio.RealBufferedSink.write(RealBufferedSink.kt:147) ~[okio-jvm-3.6.0.jar:?]
at okhttp3.internal.http1.Http1ExchangeCodec$KnownLengthSink.write(Http1ExchangeCodec.kt:279) ~[okhttp-4.12.0.jar:?]
at okio.ForwardingSink.write(ForwardingSink.kt:29) ~[okio-jvm-3.6.0.jar:?]
at okhttp3.internal.connection.Exchange$RequestBodySink.write(Exchange.kt:223) ~[okhttp-4.12.0.jar:?]
at okio.RealBufferedSink.emitCompleteSegments(RealBufferedSink.kt:256) ~[okio-jvm-3.6.0.jar:?]
at okio.RealBufferedSink.writeAll(RealBufferedSink.kt:195) ~[okio-jvm-3.6.0.jar:?]
at okhttp3.RequestBody$Companion$asRequestBody$1.writeTo(RequestBody.kt:167) ~[okhttp-4.12.0.jar:?]
at okhttp3.MultipartBody.writeOrCountBytes(MultipartBody.kt:157) ~[okhttp-4.12.0.jar:?]
at okhttp3.MultipartBody.writeTo(MultipartBody.kt:93) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.http.CallServerInterceptor.intercept(CallServerInterceptor.kt:62) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.connection.ConnectInterceptor.intercept(ConnectInterceptor.kt:34) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.cache.CacheInterceptor.intercept(CacheInterceptor.kt:95) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.http.BridgeInterceptor.intercept(BridgeInterceptor.kt:83) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.http.RetryAndFollowUpInterceptor.intercept(RetryAndFollowUpInterceptor.kt:76) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.connection.RealCall.getResponseWithInterceptorChain$okhttp(RealCall.kt:201) ~[okhttp-4.12.0.jar:?]
at okhttp3.internal.connection.RealCall.execute(RealCall.kt:154) ~[okhttp-4.12.0.jar:?]

This is a normal timeout which occurs due to the timeout values on the OkHttpClient which are either the default values or your overridden ones. The solution is to bump the values, assuming you want to tolerate a longer delay before being notified of a congested/absent network, or to re-enqueue the work to happen at a later time using the next layer up in your application architecture.

This is a normal timeout which occurs due to the timeout values on the OkHttpClient which are either the default values or your overridden ones. The solution is to bump the values, assuming you want to tolerate a longer delay before being notified of a congested/absent network, or to re-enqueue the work to happen at a later time using the next layer up in your application architecture.

Is there a recommended timeout value?

I'd say our defaults are the generally-recommended values. You know more about your application than we do. Some mobile applications want higher timeouts to tolerate slower mobile networks and intermittent radio connectivity loss like tunnels and such. Some server applications want lower timeouts as they want to respond to their caller quickly. Additionally, who you're talking to also matters (which again only you know about). If you're talking to a server halfway around the world then you need larger timeouts to accommodate the fact that it will simple take a long time for the many roundtrips of the underlying TCP connection to complete. If you're talking over fiber to a server in the same datacenter then you don't need to wait to detect link failures.

Closing - no clear action to take.


Support alternative filtering mode in recursive edge traversals

When applying filters in a scope marked @recurse, order of operations matters. There are at least two options for it:

Fully recurse first, then filter — this results in reaching child vertices even if intermediate steps did not satisfy the filtering condition.
Filter at each level of recursion — this results in pruning the recursion as soon as the filtering condition is no longer satisfied, and ensures all returned results were reachable along paths where each step satisfied the filtering condition.

The compiler's current choice of semantics is option 1. However, the option 2 has its uses as well, and is not implementable (not even as a post-processing step) as a function of the first, so an additional query-language-level feature appears to be warranted here.
Option 2 may not be supported by all backends (e.g. MATCH). However, it is definitely implementable e.g. in SQL.

One way to support both options is to add an enum-typed field to @filter directives to set the recurse filtering mode. This directive field would only valid to use immediately within a scope marked @recurse, since that's the only scope that is evaluated within recursion steps.


Expand on "Migrating existing steps to a pipeline.yml"

This adds some extra details to the migration part of the "defining steps" doc.
Previously, we did not include a "wait" step to ensure other steps couldn't run in concert and thus be preserved.
This also adds a screenshot showing the described configuration.

I’m all for changing names, I think an additional explanation would be awkward. Are you happy with “existing step 1”?

Yeah agreed! Yep, that would be great 😄 thanks!


Produce more helpful Difference.ObjectXValue for objects without .ToString() override

This would especially help with simple cases where one of the objects is a null (other cases are mostly fine because then the library goes down to the primitive-valued properties). It's especially of issue for IEnumerables.
Currently, what the library outputs is a very lengthy and hard to understand piece of text:
Types [WhereSelectEnumerableIterator`2,null], Item expected != actual, Values (System.Linq.Enumerable+WhereSelectEnumerableIterator`2[MyOrganization.MyApp.SomeFeature.Core.Model.Blah.SomeEntity,MyOrganization.MyApp.SomeFeature.Core.Model.Blah.SomeEntity],(null))

A perfect value to choose in these would be a serialization of the object (would handle IEnumerables and named and anonymous objects alike), e.g. to JSON.
If an extra dependency is a problem here, perhaps provide a way for users to plug this in, .e.g. ComparisonConfig.GetNoToStringFallbackValue. A simple function to detect that there's no ToString override is:
bool IsToStringMethodFromObject(object value) => value != null && value.GetType().GetMethod("ToString", types: new Type[0]).DeclaringType == typeof(object);

If, for some reason, this solution is not an option, what could be considered at least is adding support for IEnumerables (e.g. display [].Count=5 instead of the System.Linq.Enumerable+WhereSelectEnumerableIterator``2[...]).

I don't want to introduce a JSON.NET dependency.  The other two options are fine.  Please implement and I will merge in the pull request.

Thanks.
As for implementation, this is only for quite a busy project at work - I don't think I can convince the stakeholders to allow me to familiarize myself with the codebase and fully implement this to contribution standards any time soon. Nevertheless, I felt it was a worthy suggestion to give. If this means that implementing the feature though other means won't happen, feel free to close the issue.

@slawomir-brzezinski-at-clarksons , I will leave open for a while in case someone wants to help.  I know what you mean by busy.  I have a day job, a side business, and three kids.

Closing inactive issue.


[Compute] az vmss update: Add new parameters --force-deletion to support force delete vmss

Description

Support forceDeletion of in VMSS ScaleIn Policy in Azure CLI  https://github.com/Azure/azure-cli/issues/20301
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

@zhoxing-ms, please help with this PR.

Compute


logger profile for full raw IMU logging

We should create a logger profile (or mode?) for logging all raw data from the new IMU drivers.

log a particular sensor_accel_fifo or sensor_gyro_fifo message at max rate
can logger keep up with a 1 or 2 kHz orb publication or do we need to queue?
do we need to optimize sensor_accel_fifo and sensor_gyro_fifo?

contains 32 element arrays of 16 bit data, but often only 4-16 samples are populated
if necessary for logging we could merge this data into another topic with all 32 samples populated
another alternative could be to create different sized messages (sensor_accel_fifo8, sensor_accel_fifo16, sensor_accel_fifo32, etc).

include helper script to unroll logged sensor_accel_fifo/sensor_gyro_fifo

Yet another possible alternative, we could log per axis with a new message of perhaps 128 entires (uint16).

When I tested this, logger was easily able to keep up with 1khz, depending on overall CPU load you want to increase its priority though.
The current fifo messages are definitely too large, we can do it if we reduce them to 8 elements @ 1khz (or combine in some other manner).
BTW flight review already shows the messages.

Done.


Virtual IP/CARP Settings are not synced to the secondary FWs

Hello,
I am using OPNsense 20.7.3-amd64. My observations is, that the CARP Settings are not sync'ed to the secondary. That might be since the "Virtual IPs" are a submenu item of "interfaces" instead of "Firewall".
I double checked that the XMLRPC sync of "Virtual IPs | Synchronize the CARP Virtual IPs to the other HA host." is activated. I changed Carp PW, Skew and description but the change never arrived on the secondary.
I adapted it manually to workaround the issue.
rgds,
derjohn

Did you force a manual sync after adding it?

Yes, I clicked the link "Perform synchronization" in the "Configuration Synchronization Settings (XMLRPC Sync) Perform synchronization" panel. There is button to do so and after clicking you're redirected to "System: High Availability: Status".

Hmmm ... I checked with firewall rules on the WAN interfaces. They aren't sycned, too. So the issues is probably somewhere else.

Now I clicked the the  "Perform synchronization", which brings me to "System: High Availability: Status" Panel, there I click on "Synchronize config to backup" and then it works.
Is it the intended URL of the the "Perform synchronization" Link? Furthermore "Synchronize config to backup" reads like I want to perform a backup to the cloud storage. Shouldn't the wording be something like "Synchronize config to secondary firewall" ?
--

And: Shouldn't the config sync occur on every "apply" I click? I re-tested: That doesn't happen. Only manual sync works. I cant find a button for "enable auto sync". Is it normal behavior?

no https://docs.opnsense.org/manual/hacarp.html

hahaha an RTFM, Thx :)
I checked the logs and see this one ONLY if I click manual sync
opnsense[27413] | /usr/local/etc/rc.filter_synchronize: Filter sync successfully completed with https://<IP_ADDRESS>:10443/xmlrpc.php.
If I add e.g. a  new WAN Firewall rule, nothing happens.

@AdSchellevis In the docs there is written "Configuration changes made on the primary system are synchronized on demand to the secondary firewall." Does that "demand" mean, I have to initiate that manually, i.e. it does not directly after clicking apply on the primary FW?

@derjohn you have to initialise it manually indeed, as explained in the document under workflow (https://docs.opnsense.org/manual/hacarp.html#workflow).

Did this change recently? I was under the impression that for example firewall rules were indeed XMLRPC-synced but not activated yet?

I'd call it a "fuck I kicked myself out from cluster - phew, wasn't synchronized to backup-unit"-feature

At least for the firewall rules there is a message after apply
"The changes have been applied successfully, remember to update your backup server in System: High availablity: status"
So: Works as designed. #closing


[Bug]: Cannot save docker compose service configuration

Description
First things first: Thank you for your great work on this 🙌
I'm trying to migrate services from v3 to v4.
I deploy via the Github app, using the docker compose build pack. I have successfully migrated other packages, which uses the docker build pack.
All the docker compose packs ends fails with the same error. When I add the path to the docker-compose file and try to save it, it spins for a few seconds, shows an error message, and then kicks back to the previous value.

Minimal Reproduction (if possible, example repository)
Here's the docker compose file
version: '3'
services:
  nordlynx:
    image: ghcr.io/bubuntux/nordlynx
    container_name: nordlynx
    cap_add:
      - NET_ADMIN
    ports:
      - 3002:3002
    environment:
		 - PRIVATE_KEY=***
		 - QUERY=***
    sysctls:
		- value_1=1
		- value_2=2
    restart: unless-stopped
  automation:
    build:
      context: .
      dockerfile: packages/automation/Dockerfile
      args:
	    - SERVICE_ACCOUNT=***
    network_mode: service:nordlynx
    volumes:
     - data:/data
    depends_on:
      - nordlynx
volumes:
  data:
    external: true

Exception or Error

generateEnvValue(): Argument #1 ($command) must be of type string, null given, called in /var/www/html/bootstrap/helpers/shared.php on line 1484

This is the LOC it points to: https://github.com/coollabsio/coolify/blob/07c9db9b54ea746ce65aa0b57f03eb7634ba44af/bootstrap/helpers/shared.php#L1484
Version
v4.0.0-beta.239

If I'm right the # REDACTED parts are not there, as it makes the yaml file invalid.
Can you please add the envs (ofc with different values) and all the REDACTED values (ofc with different values) to the example?
I've added a fix to the next version, but I would like to make sure.
If I'm right, one of them are starting with SERVICE.

If I'm right, one of them are starting with SERVICE.

You are totally correct 👏
  automation:
    build:
      context: .
      dockerfile: packages/automation/Dockerfile
      args:
	    - SERVICE_ACCOUNT=***
    network_mode: service:nordlynx

Next version will fix it.

It should be fixed now in v244. Can you please update and let me know if it still does not work?


build homebrew recipe for sqlitebiter on macos

https://brew.sh/

Thank you for your suggestion.
I had added the Homebrew installation for macOS: https://github.com/thombashi/sqlitebiter/blob/master/docs/pages/installation.rst#installation-for-macos-via-homebrew

wow, thank you so much!

I'll close the issue.
Thanks.


Enable Data Restrictions

This PR introduces data restrictions into the dev environment. The restrictions can be overridden by specifying a REACT_DISABLE_DATA_RESTRICTIONS=true environment variable.

Aha - I see - the production version of Page.tsx has it always-on.
That wasn't immediately obvious ;-)
On Thu, Jul 15, 2021 at 2:12 PM Jamie Long @.***> wrote:

@.**** commented on this pull request.
In src/index.tsx
https://github.com/VEuPathDB/web-eda/pull/226#discussion_r670447718:

@@ -54,8 +64,29 @@ initialize({
],
componentWrappers: {
SiteHeader: () => Header,

Page: (DefaultComponent: React.ComponentType) => {

 return function ClinEpiPage(props: Props) {

   useEffect(() => {

     if (process.env.REACT_APP_ENABLE_DATA_RESTRICTIONS === 'true') {

In fact, reverting my most recent commit will change the behavior to the
fail-safe approach you described! I'll go with my first instincts, because
the short-term pain isn't all that painful.
I should clarify that:

This component is the Page that's rendered by the dev server - the
production version of the Page currently lives here
https://github.com/VEuPathDB/ClinEpiWebsite/blob/master/Site/webapp/js/client/component-wrappers/Page.tsx
.
In this repo, assets which lie outside src/lib, such as this file,
aren't published to npm.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/VEuPathDB/web-eda/pull/226#discussion_r670447718, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACLLH72NDAMA34QIB7G4FLTX3NCNANCNFSM5AMDOGKQ
.

@jtlong3rd do we need to add a proxy for getting permissions, or does this come from the wdk rest service?

@jtlong3rd do we need to add a proxy for getting permissions, or does this come from the wdk rest service?

Offline, I clarified that we will eventually need to add this proxy, but not right away. The endpoint which fetches a user's permissions (maintained here) is only exercised for registered users, and currently, the local dev environment doesn't offer WDK login.


Releasing v1.2.0

Preparing to release v1.2.0
Releasing the following functionality:

Auto merge not requiring qualify for auto merge label #28
Action not running on PRs from Forked repos (excludes welcome message) #24
Support for key value lookup file on front matter reviewers #23
Auto merge filters (initial GLOB support) #15
Documentation improvements
Added docs on FrontMatter support

Bug fixes:

Bump lodash from 4.17.15 to 4.17.19 #25
Action not running on PRs from Forked repos (excludes welcome message) #24

Breaking changes:

Default prlabel-automerge label changed from auto-merge to qualifies-auto-merge. Use workflow config file to continue with your preferred label.

#pr-ready


[FEATURE] click to tray icon shall toggle player window

First click shall show window, if window is shown it shall hide it

I don't think this is a good idea, the tray is mostly used to give you the play/pause/next/previous features. If I am working in another program, I do not want museeks to pop-up.
On Win32 though, double clicking the tray shows the app, and right-click show the context menu.
https://github.com/KeitIG/museeks/blob/master/src/main-process/tray.js

Let's move everything to #389


Section with same name works wrong

We have 3 views. All three have section('content'): parent have one for all contents, middle have to: one override parent and own for children.
, example:
//master.blade.php

@section('content')
@show

//middle.blade.php

@extends('master')

@section('content')

This content will not be displayed.

    @yield('content')

This content will not be displayed too...

@endsection

//last.php

@extends('middle')

@section('content')

Hello! I override parent fully %)

@endsection

Invoke:
...
return view('last');
...

Middle context content ignoring and result of parse be:

Hello! I override parent fully %)

Expected:

This content will not be displayed.

Hello! I override parent fully %)

This content will not be displayed too...

Please look at the blade docs. You should use the @parent directive!


SP5 Cookbook: Duplicate line style examples

This is a very small deal, but I noted that in our enthusiasm to add an additional line style recently I think we ended up creating a duplicate recipe in the cookbook.
Under Styling Plots there is a Line Styles recipe.
Under Scatter Plot there is a Scatter Line Patterns recipe.
They are close to being identical, although the Scatter Line Patterns includes labels with the names of each line style which is nice.

Apologies, I added the bug label inadvertently.  More a suggested improvement.

Thanks for raising this point!
My inclination is to keep them both, and add even more similar examples for the other plot types that use them (line, axis line, etc.) to make these customizations easier to locate because people researching a specific plot type (e.g., scatter plot) and who may not think to look in the "styling" section. There will be some duplication, but I think it's okay in this case because my aim is to make it more likely for the reader to see it.
However, I'll work to simplify the example in the styling section and ensure it doesn't appear as a duplicate 👍

this new line pattern demo for the styling section is a lot simpler and I think it more clearly communicates what patterns are


Setting WORKDIR in dockerfiles breaks Kerblam!

Kerblam! assumes that the working directory in the docker is /, for running and packaging. If this is wrong (e.g. the user adds a WORKDIR ... directive), kerblam! just fails.
Parsing the dockerfile is against the philosophy of Kerblam!, but it is possible to add a kerblam.toml option to allow changing where kerblam! executes inside containers.
For example:
[execution.container]
workdir = "/app"

Will make kerblam! run docker run ... --entrypoint "bash /app/executor" instead of docker run ... --entrypoint "bash /executor".

Fixed in https://github.com/MrHedmad/kerblam/pull/58


Bump github-changes-maven-plugin dependency version to 1.1

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/cannoli-prb/203/
Test PASSed.

Coverage remained the same at 13.097% when pulling 49a9123f97e04b13856f4e16a798ba5661228706 on heuermh:github-changes into cd2dfe603fd77a151af4f1093ebe302d5e1695af on bigdatagenomics:master.


Git tool missing tooltips on Pull and Push buttons (when there are commits)

I.e. hovering the mouse above the Pull and Push buttons in the image below doesn't show a tooltip:

I think you fixed this @akgrant43, right?

Yes, I even put the Fixes message in the commit.  Not sure why this wasn't automatically closed.


TopImage

You added insert left and right image!
like this: cardStack.setLeftImage(R.id.left_image);
cardStack.setRightImage(R.id.right_image);
Can you add setTopImage and setBottom image?

I assume you want to be able to swipe up and down also?
currently swipedeck doesn't have that feature but i could add it.

yes when i am swiping up or down and image should display can you add this feature ??


Ignore non-seed images in scan

Ignore non-seed images when performing scans.  To test seed-ness, we could either check the image name or look for a seed manifest label.

To be clear, when checking the image name we would test for a valid name-version-seed:version format.


Two way audio

Since homebridge-camera-ffmpeg supports two way audio since v3.0.0 (https://github.com/Sunoo/homebridge-camera-ffmpeg/releases/tag/v3.0.0) it would be great to have that feature integrated into homebridge-dafang too

@roboterm I was unable to find a way to play audio stream over speaker in Dafang. For now, i guess only single wav file can be played from local sd card of the Dafang. I guess there could be some workaround to record chunked wav files on homebridge and then transfer those files to dafang sd card and then play those wav files. It will have quite high latency and error rate making it more like Walkie Talkie rather than real time.

@roboterm I was unable to find a way to play audio stream over speaker in Dafang. For now, i guess only single wav file can be played from local sd card of the Dafang. I guess there could be some workaround to record chunked wav files on homebridge and then transfer those files to dafang sd card and then play those wav files. It will have quite high latency and error rate making it more like Walkie Talkie rather than real time.

I already have a working Walkie-Talkie like solution if you want to try. Its quite buggy yet though.

I already have a working Walkie-Talkie like solution if you want to try. Its quite buggy yet though.


Implement IntoIterator for Offset and StringArray

Opening an issue to discuss and understand what this implementation would look like.
Is it correct to think that the Offset struct operates in the u8 realm? And the IntoIterator implementation for a StringArray with the type Item = String will have to deal with the complexity of converting the iterator of u8's the wrapped Offset produces into a String type using the offset values provided by offset_buffer()?
Pardon if I'm overlooking something obvious.

Offset does not restrict the data it wraps around, however to construct it from an iterator over T, T must implement IntoIterator + Length. If you look at the tests of the iter mod in the offset mod here, you can see that there are some example iterators for the Offset wrapper.
However, for the IntoIterator of StringArray we need a new Offset iterator that just yields the lengths (via the offset buffer). Then the StringArray iterator can wrap around that and use the underlying data (u8 buffer) to yield str slices (via https://doc.rust-lang.org/stable/std/str/fn.from_utf8_unchecked.html).


CI: use docker hub to setup kong in CI

Bintray is going away, which means the CI will always be red.
The current CI also uses debian packages to set up Kong.
The scope here is

update the CI script to not use Bintray
use Docker in the CI to run Kong

Maybe would it worth to try to define a github action for this :

https://docs.github.com/en/actions/creating-actions/creating-a-docker-container-action
https://docs.github.com/en/actions/creating-actions/creating-a-composite-run-steps-action

This way it could be reused by go-kong and deck, at least.

@mmorel-35 We could use Github Actions here to solve the problem in the CI but then we will need a separate script to run the tests locally. Instead, if we write a script, we can use that in CI and developer workflow, which helps with keeping different environments consistent.

Not relevant anymore. Bintray is not used for quite some time. Related: https://github.com/Kong/go-kong/pull/55


Load Java Maven Project automatically after creation

Environment
Operating System: Linux Debian on Codespaces

JDK version: 17+

Visual Studio Code version: 1.70.2

Java extension version: 0.25.1
Steps To Reproduce

[step 1] Create a Java Multi-Module (with parent POM) or microservices spring boot application (containing many miro-Java Maven projects).
[step 2] Refresh the Java projects extension tab.
[Please attach a sample project reproducing the error]
Please attach logs
Current Result Newly added project do not appear in the java extensions tab immediately after creation. This is not the case even after 'refreshing' the 'Java extension' tab.
Expected Result
I will expect the newly added Java Maven or Spring project to appear among the existing projects
Additional Information
Since the newly created project do not appear (immediately after creation), I have to either restart, close or open Vscode before the newly created Java Maven or Spring project can appear among the Java Maven/spring existing projects in the 'Java Extension' tab.

As a workaround, you can trigger the command: Java: Import Java projects into workspace

@jdneo Thanks. What do you think about adding tags/similar to indicate that a given child POM depends on a parent POM dependenicies? This might help in large projects.


MMIF annotation getter doesn't work on short ids

Bug Description
When using the native MMIF annotation getter mmif[anno.id], the SDK throws a KeyError. When using view-scoped long ids (mmif[anno.long_id]), it works as expected.
Reproduction steps
Using a MMIF file (version 1.0.4), try to access a given annotation via mmif[id], where id is a short annotation ID.
Expected behavior
>>> for anno in mmif["v_0"].annotations:
...     anno = mmif[anno.id]

Should successfully access a MMIF Annotation object in each iteration
Log output
>>> with open("spacy1.mmif", "r") as f:
...     mmif = Mmif(f.read())
...
>>> for anno in mmif["v_0"].annotations:
...     anno = mmif[anno.id]
...
Traceback (most recent call last):
  File "<stdin>", line 2, in <module>
  File "/home/hmccormick/.local/lib/python3.9/site-packages/mmif/serialize/mmif.py", line 590, in __getitem__
    raise KeyError("ID not found: %s" % item)
KeyError: 'ID not found: to_1'

>>> for anno in mmif["v_0"].annotations:
...     anno = mmif[anno.long_id]

>>> # Works without error on long IDs

Screenshots
No response
Additional context
No response

Not sure this is actually a bug. Although the documentation isn't super clear,
https://github.com/clamsproject/mmif-python/blob/2a5bffc36ba360e45715bd48c7af07e841717f7a/mmif/serialize/mmif.py#L564-L590
considering

it says it expects a view-scoped annotation ID (short ID)
it raises error when two objects with the same ID (when a document and a view have the same id, line 588)
annotation IDs are only scoped within a view, so it's much more likely to have ID conflict by querying annotation ID only.

I think an error when querying with a short ID is somewhat expected behavior. Easy workaround this limitation in the downstream is probably just using anno.long_id instead of anno.id.
https://github.com/clamsproject/mmif-python/blob/693972a2868e226453f55b9c42380061ce96d996/mmif/serialize/annotation.py#L131-L135