Extension size causing lambda to exceed size limit

We are trying to use this extension for our lambdas, but getting a size limit error. The extension appears to include the folder listed below, which when I pull the code down locally, is showing 163MB. The entire code base extracted is only 164 MB, so this seems to be taking up the bulk of the size. Just going based on the name of the folder, but should this folder be included in the extension?
..\datadog-lambda-extension-main\datadog-lambda-extension-main\scripts\go-binsize-viz\example-data
We are using version 43 and 44 currently and seeing the same results in both.
Looks like this might have been reported before, but closed on issue #79 , but there was no resolution.

Hey @mbrmchll - thanks so much for reaching out!
Another user ran into this accidentally as they downloaded the release from github's zip archive, which contains all the source code, scripts, etc (and not the compiled version of the lambda extension which is published as a lambda layer).
Our extension is roughly 38mb when the compiled version is applied as a lambda layer, you can verify this yourself using the aws cli as documented here
aws lambda get-layer-version \
  --layer-name arn:aws:lambda:us-east-1:464622532012:layer:Datadog-Extension \
  --version-number 44

With 44, I see:

Can you clarify how you're installing the extension? Are you running into size errors when deploying to Lambda?
Thanks!

Hey @mbrmchll - thanks so much for reaching out!
Another user ran into this accidentally as they downloaded the release from github's zip archive, which contains all the source code, scripts, etc (and not the compiled version of the lambda extension which is published as a lambda layer).
Our extension is roughly 38mb when the compiled version is applied as a lambda layer, you can verify this yourself using the aws cli as documented here
aws lambda get-layer-version \
  --layer-name arn:aws:lambda:us-east-1:464622532012:layer:Datadog-Extension \
  --version-number 44

With 44, I see:
Can you clarify how you're installing the extension? Are you running into size errors when deploying to Lambda?
Thanks!

We are referencing the extension as a layer on our lambda as follows:
Layers:
        - !Sub arn:aws:lambda:${AWS::Region}:464622532012:layer:Datadog-Extension:44
        - !Sub arn:aws:lambda:${AWS::Region}:464622532012:layer:dd-trace-dotnet:8

We were understanding that we needed both the Datadog-Extension and the dd-trace-dotnet layers, but maybe the trace extension is causing the problem. Are both needed or do we only need the Datadog-Extension?

For APM tracing they are both needed, but the dotnet tracing layer is 28mb, so the total size is around 66mb:

Can you confirm that your own code plus 66mb is more or less than the 250mb limit?
Thanks!

For APM tracing they are both needed, but the dotnet tracing layer is 28mb, so the total size is around 66mb:
Can you confirm that your own code plus 66mb is more or less than the 250mb limit?
Thanks!

The biggest lambda we have in the project that's failing with the size limit error is 59.4 MB, so that including the layers should not be exceeding 250 MB.
The errors we are getting are similar to:
"Function code combined with layers exceeds the maximum allowed size of 262144000 bytes. The actual size is 270858104 bytes. (Service: Lambda, Status Code: 400..."

Ah that error seems familiar - are you using the Serverless Framework to build and package your service?

Ah that error seems familiar - are you using the Serverless Framework to build and package your service?

No, we are not using Serverless Framework to build and package.

Okay - something is off here, as 59.4 + 66 is still well under the limit, but somehow that project adds up to 270mb. I'm not sure I can help more in this issue as we can both confirm the size of these libraries are not in fact 163mb.
Can you open a support ticket with Datadog so that we can see the combined deployment package and identify what's contributing to this issue?
Thanks!


Make DAOs, services and web controller extendible

This is a major PR that makes all DAOs, services and web controllers extendible. This change is important to avoid code duplication in this project (or projects that extend SHOGun2). By using Java Generics, components like DAOs or services in subclasses will always have the correct type (and thereby provide the full API for their type), which is very nice because developers are not forced anymore to cast these components in certain situations.
Long story short: Defining a new (sub-)class (using Java Generics) is more complex than before, but using/developing the API will be more fun now.
On top of that one minor change has been introduced: Abstract parent classes (i.e. GenericHibernateDao, AbstractDaoService and AbstractWebController) now provide a LOGGER instance that can easily be used in all their subclasses, i.e. subclasses don't need to define their logger instance anymore as the Logger in the above mentioned parent classes is defined as:
protected final Logger LOG = Logger.getLogger(getClass());

where getClass() will always return the type of the concrete subclass...

Please wait with merging. I forgot the changes in the archetype... I will add some commits soon.

Great and impressive work. Very nice that there were nearly no changes to the tests involved. Do we have a readme for creating the sub-classes? Might well be worth adding one.
Please merge once the minor comments are adressed, the archetype is updated, and the lights a green again.
:clap:

We don't have a README that explains how to extend existing classes. But the result of the archetype contains demos for Model/DAO/Service/Controller...

… the result of the archetype contains demos for Model/DAO/Service/Controller

OK for me.

This can be merged now!


missing file ..gpu_fix

https://github.com/JDACS4C-IMPROVE/Singularity/blob/develop/definitions/DeepTTC.def#L15

found out the you need to run the command from Singularity folder.


host networking pods should be deployed with dnsPolicy set

Unless there's a specific reason not to, host networking pods should have their dnsPolicy set to dnsPolicy: ClusterFirstWithHostNet to allow the executables in the pods to leverage kube-dns.
The triggering example scenario for this issue is enabling Jaeger tracing in the cmd-forwarder-* pods.  Found during https://github.com/networkservicemesh/cmd-forwarder-vpp/issues/344

@denis-tingaikin Could we get someone on this?

I thought it should work by default. For example, our nsmgr is already using the registry service name instead of IP and it is just working without dnsPolicy https://github.com/networkservicemesh/deployments-k8s/blob/main/apps/nsmgr/nsmgr.yaml#L32
@tiswanso Could you provide steps to reproduce?

Kubernetes docs are pretty explicit on using ClusterFirstWithHostNet for host networking pods:
https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/#pod-s-dns-policy
To repro, you can test with the patch for networkservicemesh/cmd-forwarder-vpp#344 and the info in https://github.com/tiswanso/deployments-k8s/tree/fix-jaegerenv/examples/features/jaeger
The forwarder pod will throw the following error:
2021/09/23 14:53:18 [ERROR] ERROR: cannot init Jaeger: lookup simplest-agent.observability on <-node's default DNS server (redacted)->:53: no such host

Setting the dnsPolicy to ClusterFirstWithHostNet, fixes it.
As to why registry lookup works, I don't have time to look into it but I'm guessing the nsmgr (or lib) code defaults to use localhost when lookup fails.


problem

[Master HamsterKombat Bot][ERROR] 2024-08-16 18:18:00 [Account 1 the_pessage] Error: HTTPSConnectionPool(host='api.gamepromo.io', port=443): Max retries exceeded with url: /promo/login-client (Caused by ProxyError('Unable to connect to proxy', OSError('Tunnel connection failed: 403 Forbidden'))) [Master HamsterKombat Bot][ERROR] 2024-08-16 18:18:00 [Account 1 the_pessage] Error: HTTPSConnectionPool(host='api.gamepromo.io', port=443): Max retries exceeded with url: /promo/login-client (Caused by ProxyError('Unable to connect to proxy', OSError('Tunnel connection failed: 403 Forbidden'))) [Master HamsterKombat Bot][ERROR] 2024-08-16 18:18:00 [Account 1 the_pessage] Unable to get Merge Away key.

It's not a bot problem, it's a connection problem, possibly a proxy problem


Rename all 'pkcs8' to 'PrivateKeyInfo'

OpenSSL changed the structure name 'pkcs8' for encoders and decoders
to 'PrivateKeyInfo', the latter being the actual structure name.
(the reason for the name change is a bit more elaborate, please see
the OpenSSL commit log for details)

The LGTM analysis failure looks weird, please advise


Follow Up on Issue #10: Run Chrome Headless

Howdy Meteor testing world. I have an issue running headless when performing CI testing for my meteor app. I believe the issue is related to #10.
Problem
In #12, specifically in file chromedriver.js, the chrome driver always default to the headless view, but there is still an attempt to position the window off screen. I believe this might be the source of the issues I am facing.
Solution
Add another conditional to only move the window if there is not headless option.

@aldeed Tagging you here. If it is possible to create this quick fix, I would much appreciate it. I won't be able to make a PR until the end of the day.

I added a simple fix, at PR #22 . Tagging @aldeed to review and merge.

Closing issue. Using Puppeteer instead of chrome fixed my problems.


moveit_python: 0.2.17-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository moveit_python to 0.2.17-0:

upstream repository: https://github.com/mikeferguson/moveit_python.git
release repository: https://github.com/mikeferguson/moveit_python-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.5.21
previous version for package: null

moveit_python
* Merge pull request #9 <https://github.com/mikeferguson/moveit_python/issues/9> from mikeferguson/pyassimp_fix
  pyassimp is broken in 16.04, temporary work around so we can release
* Contributors: Michael Ferguson

The sourcedeb job for Jessie is failing since this release: http://build.ros.org/view/Ksrc_dJ/job/Ksrc_dJ__moveit_python__debian_jessie__source/
Please either release a patch version or blacklist the package for that platform: https://github.com/ros-infrastructure/ros_buildfarm_config/blob/9094ebf1348650185049d9a4e1df51b6b58b50e4/kinetic/release-xenial-arm64-build.yaml#L14

This is failing due to missing jessie tags. It is likely you did not allow bloom to generate the new track entries for debian?

Nope, it was actually a missing rosdep key and I manually skipped releasing debian to rush things out for world moveit day. Added key here: https://github.com/ros/rosdistro/pull/12954.
@tfoote -- might I suggest that we add a "recommended support" section for the debian/redhat/etc distros we provide build support for under the REP-003: http://www.ros.org/reps/rep-0003.html. I'm not entirely sure if there are other debians we are building for, I only added the key for jessie for the moment.


Cesium 1.58.0 package on npmjs is missing Build directory

I just pulled the latest Cesium version and it appears to be missing the Build directory that was included with previous versions. Was this intentional or a bug?

Thanks @skaughtx0r, looking into this now

I just published 1.58.1 on npm, and this includes the Build directory.  Thanks again for catching this so quickly @skaughtx0r!


Can I please find a collaborator for website project

https://github.com/open-source-ideas/open-source-ideas/issues/new?template=new-idea.md


Create "floatEncoding" field

Create "floatEncoding" field to "PropertyValue" and "ValueDescriptor"
struct in Mongo db models
fix #1188
Signed-off-by: Cloud Tsai<EMAIL_ADDRESS>

add @iain-anderson for awareness

also added MediaType into ValueDescriptor


Proposal: make Let's Encrypt a first-class entity in Workflow

From @jmspring on January 19, 2017 20:31
At the Azure / Deis hackfest at in December, there was time to work on additional enhancements.  One such was a PoC for adding support for Let's Encrypt certificates (to enable TLS) when new applications are deployed to Workflow.
The initial PoC takes on two parts:

An application that makes periodic checks for new applications:  https://github.com/jmspring/deis_certificate_manager

A worker process that the (1) spawns to do the actual Let's Encrypt/ACME handshake:
https://github.com/jmspring/deis_cert_letsencrypt_worker

This is just a PoC and real support should directly work with the ACME protocol and not necessarily shell out another process to do the handshake.
Further, some research into limitations in/around Let's Encrypt certificate issuance policies (limits per domain, etc) should be considered.
Also, the PoC only retrieves the initial certificate, Let's Encrypt certs have ~90 day life time so renewal as well as revocation as apps are removed should be considered as well.
@mboersma - Jason Hansen mentioned you were Workflow lead.
Copied from original issue: deis/workflow#708

From @jmspring on January 24, 2017 22:44
@bacongobbler, @mboersma - thoughts on moving forward on this?  I have cycles to work on it.

From @bacongobbler on January 19, 2017 20:43
related: https://github.com/deis/router/issues/134

From @MartinodF on January 26, 2017 15:28
We're investigating deis and one of the main issues we found is the lack of Let's Encrypt integration. Our current solution on Kubernetes relies on kube-lego together with nginx-ingress.
kube-lego monitors the kubernetes cluster for ingresses with a specific annotation. It then reads the spec.tls.hosts field from those and compares the result with the available certificates. When some are missing or near the expiry date, it requests a new certificate, creating a new ingress rule which matches the acme challenge path and intercepting the Let's Encrypt validation request. It then saves the new certificates in a secret, ready to be used by the application.
I believe such an approach would work for deis too, wouldn't it?

From @jmspring on January 24, 2017 23:52
Part of the Let's Encrypt Handshake (for issuance) currently requires some form of validation of the domain.  The PoC was one quick hack for such.  I'll dive into the protocol a bit more this week and solidify my understanding both of issuance and renewal -- the certs are only 90 day certs, so renewal (as well as revocation will be important).
My concern if it is CLI only and requires said handshake, what the best approach is.  As for updating nginx (per the thread you linked to -- the hash of a cert makes sense to trigger the update).
I'll get some more info by Thursday.

From @bacongobbler on April 5, 2017 22:53
Now with the experimental native ingress feature in v2.13.0, I've been able to get a Workflow cluster up and my apps being backed by letsencrypt running using a combination of kube-lego and the nginx ingress controller. I'll hopefully get to writing a blog post about the entire setup so others can learn about the internals and see where we can write a feature around it.

From @bacongobbler on March 28, 2017 19:51
@MartinodF just following up on this comment, but with #732 in place, yes. You could just run ingress and kube-lego on the cluster as usual.

From @johnschultz on June 22, 2017 7:2
@bacongobbler really interested in how you got that to work. Is that blog post up somewhere or can you point me in the right direction? Thanks!
edit: apologies, didn't realize commenting would re-open the issue.

From @mariusmarais on June 22, 2017 8:1
@johnschultz Native ingress via Traefik with its built-in Let Encrypt support is working quite well for us too. Default app names work perfectly, but deis domain:xxx needs manual changes to the ingress for the app, which I'm assuming will be handled by https://github.com/deis/controller/issues/1277. This is probably true for nginx too.

From @ccatlett2000 on July 5, 2017 0:38
Any progress on this? Yes, I can use native ingresses, but using Deis's router is preferable.

From @bacongobbler on July 5, 2017 4:56
No progress. The "hack" I did was merely just manually editing ingress routes and installing the nginx ingress controller, along with kube-lego. No code to contribute at this time.

From @ccatlett2000 on July 5, 2017 19:25
Does anyone who's setup native ingresses know if Deis adds the annotations for TLS (Or if there is a way to configure them)? Because the way that kube-lego works is that it looks for annotations to indicate which domain to pick.

From @bacongobbler on July 5, 2017 19:27
It does not; I had to manually edit the ingress resources to add those annotations in.


Signature help pops up and blocks the UI when no method/class has been chosen yet

I haven't even chosen a 'User' object yet and the signature already pops up.
It should only start showing the signature window when you have chosen the object e.g. new User();

I think it relates to triggerCharacters of your LSP. What LSP you are using and what is the triggerCharacters setting?

I think it relates to triggerCharacters of your LSP. What LSP you are using and what is the triggerCharacters setting?

I have no idea. I'm using LazyVim.


DEV-1646: Move model_version to ML backend - add versions endpoint

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@05bec1a). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master     #87   +/-   ##
========================================
  Coverage          ?   2.41%
========================================
  Files             ?       8
  Lines             ?     828
  Branches          ?       0
========================================
  Hits              ?      20
  Misses            ?     808
  Partials          ?       0

Continue to review full report at Codecov.

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


EventStream empty in Chrome Dev Tools?

Does anyone have any idea why Chrome isn't showing any events in the EventStream (Chrome DevTools/Network) when using POST with SSE? It might have to do with the non-standard use of POST of course, but it does actually show the EventStream tab, so it did correctly detect that we're dealing with a stream.
It's not important, because everything works just fine, but I was just curious if anyone was seeing the same thing and if there was some kind of fix.

It's unfortunately a bug in Chrome. See #39.

Sorry, I did look into the closed issues to see if anybody else had asked this before, I promise. Obviously I didn't look hard enough :-)


Add max retry request

Details

When we receive 500 or 404 response, the app retries to call the API forever. So we need to add the max retry request to remove the request when exceeding the limit retry requests
Fixed Issues

$ https://github.com/Expensify/App/issues/27886
PROPOSAL: https://github.com/Expensify/App/issues/27886#issuecomment-1732817535
Tests

Precondition: To test easier locally, we can edit an API to a non-existent API command i.e. DeleteMembersFromWorkspace -> DeleteMembersFromWorkspace1

Go to Settings > Workspace
Select a workspace that has some member
Open the member page of the workspace
Select some members and remove
See the log and verify that the app retries to call the API 10 times

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

Offline tests

None
QA Steps

Make an API that will receive 500 or 404 response
See the network and verify that the app retries to call the API 10 times

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

PR Author Checklist

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

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

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

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

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

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

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

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

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

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

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If 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
Screencast from 28-09-2023 11:38:44.webm

Mobile Web - Chrome
https://github.com/Expensify/App/assets/129500732/ae83d0d2-3123-4a23-b4c8-f29e58374d2e

Mobile Web - Safari
https://github.com/Expensify/App/assets/129500732/4a4bbd23-bd5e-4800-a122-06c3deca7f6d

Desktop
https://github.com/Expensify/App/assets/129500732/9365e482-d70c-4a7d-8a29-64c622e2964d

iOS
https://github.com/Expensify/App/assets/129500732/06c02439-5b88-4227-ad55-9d8ce4e6391c

Android
https://github.com/Expensify/App/assets/129500732/4d1ac1af-40cc-4d2d-a296-e01d373db8c1

@dukenv0307 @sobitneupane I see that both of you used the workspace page for your videos, but that page doesn't have any feedback.
What happens if I do something like adding a comment and it fails after the 10 tries. Will the UI update for the failure? From my testing it seems like no, but maybe I'm testing it wrong.

: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/puneetlath in version: 1.3.78-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@puneetlath @sobitneupane How do we trigger 404 and 500 api calls?

@mountiny I am not sure if we can trigger 404 and 500 api calls. Could you help us?

Ah sorry, this is not testable. So you can skip it for QA.

🚀 Deployed to production by https://github.com/Beamanator in version: 1.3.78-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/puneetlath in version: 1.3.79-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.79-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.79-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.79-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Security.md

This was a secyrity issue because ii did not keep up with the versioning

so i perfer the command line way, come from being a linux gut i suppose.


Clean installation fails on "yarn install" errors

Following the instructions in https://github.com/alan2207/bulletproof-react/blob/master/docs/application-overview.md I got the following errors in the console log (Mac OSX):
yarn install v1.22.10
[1/4] 🔍  Resolving packages...
warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>[2/4] 🚚  Fetching packages...
error An unexpected error occurred: "/Users/craigmcc/Library/Caches/Yarn/v6/npm-upper-case-1.1.3-f6b4501c2ec4cdd26ba78be7222961de77621598-integrity/node_modules/upper-case/.yarn-metadata.json: Unexpected end of JSON input".
info If you think this is a bug, please open a bug report with the information provided in "/Users/craigmcc/Git.Examples/bulletproof-react/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

The referenced yarn-error.log file is attached below:
yarn-error.log

Hi @craigmcc , which yarn version are you using? I am using yarn as well, and the installation completes without any issues.

I am using yarn 1.22.10 -- usually use npm so no idea if this is current or not.  But the complaints in the logs are all about dependency version incompatibilities, so I don't expect that would be an issue.

Have you tried to install with npm? Does it still fail?

It fails with npm as well.  Messages are slightly different, but there's "dependency of children" issues somewhere.

I inspected where the upper-case package is being used and it is used in plop which is not crucial in order to run the application. I would suggest you uninstall plop and try again. I know it is not an ideal solution, but if you want to run the app, it might work. I am also wondering if anyone else experienced this issue?

That is great, enjoy! :)


[SR-7612] Add host target triple for arm64/aarch64 machines.

Added host target triple for arm64/aarch64.
Fixed incorrect armv7 triple.
Refactored bootstrap "compute the build paths"
Refactored Triple.swift "os and machine" dif's
New OS's and processor/machine types are currently being added in an addhoc manner. Refactoring the Utilities/bootstrap and Triple.swift addresses [SR-7612]. While this solution is still hardcoded it provides conformity between machines and a consistent naming type for triples.
i.e [machine][OS] - x86Linux, arm64Linux.
This will provide an easy path for adding new machines and OS's in the future.

@swift-ci please smoke test

Great, thanks!

Is it possible to add this PR to the "swift-4.1-branch" ?

@futurejones Nope, Swift 4.1 is already released! https://swift.org/blog/swift-4-1-released/


maximum size for tensor at dimension 1 is 1088 but size is 88200

Trying to run train.py and it doesn't work. Looks like neural network architecture is incompatible with some constrains? Is a specific version of torch library required?
[2024-10-04 17:46:10,588][training][ERROR] - Couldn't run Trainer
Traceback (most recent call last):
  File "/dockerx/Stochastic-Restoration-GAN/train.py", line 94, in main
    Trainer(args)
  File "/dockerx/Stochastic-Restoration-GAN/train.py", line 24, in Trainer
    train_dataset = AudioFolder(args.dset.datapath,
  File "/dockerx/Stochastic-Restoration-GAN/src/data/dataset.py", line 88, in __init__
    waveform_chunks = torch.squeeze(waveform_mono.unfold(1, seg_len * self._sample_rate, overlap * self._sample_rate))
RuntimeError: maximum size for tensor at dimension 1 is 1088 but size is 88200

Something went wrong with my transcoding process, the file was very short, ffprobe reported Duration: 00:00:00.02, bitrate: 1436 kb/s. After removing it training started.


python310Packages.pywlroots: 0.15.18 -> 0.15.19

Description of changes
https://github.com/flacjacket/pywlroots/releases

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 182852 run on x86_64-linux 1

3 packages failed to build:

python310Packages.pywlroots
python39Packages.pywlroots
qtile

Result of nixpkgs-review pr 182852 run on x86_64-linux 1

3 packages failed to build:

python310Packages.pywlroots
python39Packages.pywlroots
qtile

Result of nixpkgs-review pr 182852 run on x86_64-linux 1

3 packages failed to build:

python310Packages.pywlroots
python39Packages.pywlroots
qtile

I got a 404 when trying to update this package earlier, but it seems to have resolved itself. Works for me (qtile still runs as expected).

Can you test again?

Result of nixpkgs-review pr 182852 run on x86_64-linux 1

3 packages failed to build:

python310Packages.pywlroots
python39Packages.pywlroots
qtile

Sorry, my mistake for not mentioning https://github.com/NixOS/nixpkgs/pull/182845 is required. I've tested this PR with that change already applied.

Merged #182845 . Please, rebase.

@arjan-s Can you squash commits?

@arjan-s Can you squash commits? (I mean, I asked for a rebase, you did a merge. Now there is 2 commits. I suggest you drop the merge, and do a rebase. So there is only a single commit.)

Yes, apologies! I tried the rebase from my phone last night, but Github website only did a merge. So I planned to fix it this morning. Thanks for picking it up!


Add Changelog and release notes for v0.51.0

Hey Pino, is this done? Could you update this ticket and the 0.51 milestone?

Done in #1849


Fixed broken URL present in DATASETS.md

I have fixed all the broken urls present inside DATASETS.md.
ISSUE : #117
Here is a screenshot

Please add hacktoberfest label and merge this PR @Rajdip019 @Debajyoti14

Hey @Rancho2002 | This is not a bug. Actually, the folder name is collage_names that's why it is written over there too. Fixing it will actually break the URL


Tapo C320SW Motion Detection

Is your feature request related to a problem? Please describe.
the Tapo C320SW camera does not have an entity to change the motion detection mode
Describe the solution you'd like
option to switch motion detection mode (OFF, low, normal, high)
Describe alternatives you've considered
Additional context
I have two Tapo cameras. The Tapo C210 has a switch between the entities to select the Motion Detection mode. The Tapo C320WS camera does not have this switch. Although the camera supports changing modes and can be used in the default application. Please add. Thank you

Fixed in 4.0.7.


Return headers for HEAD request

References
Fixes #9247
Description
Remove logical branch from HTTPHeadersInitializer.initialiseHeaders that returns null and another logical branch in BitstreamRestController.retrieve endpoint handler that returns ResponseEntity with headers and no response body.
Instructions for Reviewers
Issue HTTP HEAD request to API bitstream content endpoint and confirm the anticipated response headers inform what the GET request will return in the response body. i.e. Content-Type matches bitstream.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[ ] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[x] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

Thanks @wwtamu. I tested this on DSpace 7.6.1 by uploading a PDF to an item's ORIGINAL bundle.
Before the patch I saw the following relevant headers for a HEAD request:
$ curl -I 'localhost:8080/server/api/core/bitstreams/c0fa95ff-87c4-418b-a481-5fb4af71be3a/content'
...
Content-Type: application/json;charset=UTF-8
Content-Length: 760398
Date: Wed, 10 Jan 2024 08:07:45 GMT

The Content-Type is wrong as you noticed, but the Content-Length is correct. After the patch I see:
$ curl -I 'localhost:8080/server/api/core/bitstreams/c0fa95ff-87c4-418b-a481-5fb4af71be3a/content'
...
Content-Type: application/pdf;charset=UTF-8
Content-Length: 0
Date: Wed, 10 Jan 2024 08:13:45 GMT

Now the Content-Type is correct, but the Content-Length is 0. Would it be possible to address that?

@alanorth I'll check it out.

Thanks @alanorth for the review. I have pushed a commit to add the Content-Length header for a HTTP HEAD request and an update to an integration test. I am assuming Spring is deriving the Cntent-Type from the body added to the ResponseEntity. However, the request scoped singleton HttpServletResponse may be modified elsewhere. Which this and the HTTPHeadersInitializer helper class both modifying headers and response makes me a bit uneasy and certainly difficult to know where the response changes are occurring.
Additionally, I have discovered that the integration tests for the bitstream content endpoint appears to be asserting that the HTTP HEAD and GET requests are being counted the same for Solr statistics. Do you know if the statistics are differentiating between an actual download?

Was reading the specifications regarding HTTP HEAD requests. I like the cache benefits stated here.
A later standard mentions '...Such a response to GET might contain Content-Length and Vary fields, for example, that are not generated within a HEAD response...'.
Haven't been able to distill the latter down to determine if there is any difference.

Haven't tried the updated patch yet, but found your comments interesting. By my reading of the two RFCs I think it's OK to include the Content-Length here.

The server SHOULD send the same header fields in response to a HEAD request as it would have sent if the request method had been GET. However, a server MAY omit header fields for which a value is determined only while generating the content.

I think this part is more about dynamic pages or content that is generated upon request. In our case the content already exists and we know the size, so we can send it. I did some HEAD requests for other stuff on some other servers and found the Content-Length being sent.

Additionally, I have discovered that the integration tests for the bitstream content endpoint appears to be asserting that the HTTP HEAD and GET requests are being counted the same for Solr statistics. Do you know if the statistics are differentiating between an actual download?

Yes, in the past I wondered this as well. I just found my notes from 2019 where I confirmed that yes, HEAD requests on bitstreams are counted in Solr statistics (in DSpace 5 at least).

Thanks @wwelling. I've tested the new patch and it works as advertised. The Content-Length header in the HEAD response matches the expected size of the file on disk.
This is passing all tests and I'm ready to merge it. It is a welcome fix. The fact that HEAD requests are counted as statistics events is a topic for another discussion (perhaps was intentional?).


Develop the 2017 Solution: TypeScript

Check-List of all the solution that should be coded for the different days of the 2017 edition:

[x] Implement solution for day 1:

[x] Part 1
[x] Part 2

[x] Implement solution for day 2:

[x] Part 1
[x] Part 2

[x] Implement solution for day 3:

[x] Part 1
[x] Part 2

[x] Implement solution for day 4:

[x] Part 1
[x] Part 2

[x] Implement solution for day 5:

[x] Part 1
[x] Part 2

[x] Implement solution for day 6:

[x] Part 1
[x] Part 2

[ ] Implement solution for day 7:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 8:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 9:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 10:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 11:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 12:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 13:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 14:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 15:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 16:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 17:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 18:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 19:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 20:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 21:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 22:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 23:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 24:

[ ] Part 1
[ ] Part 2

[ ] Implement solution for day 25:

[ ] Part 1
[ ] Part 2

For day 7 the first part is developed on my computer. I still need to make part 2 of these problems, comment it and push it.


fix and tests

Fixes https://github.com/countries/countries/issues/863

The main branch of the fork is throwing an error locally:
Failures:

ISO3166::Country subdivisions #states should be deprecated
Failure/Error: expect { country.states }.to output.to_stderr
expected block to output to stderr, but did not
./spec/country_spec.rb:254:in `block (3 levels) in <top (required)>'

The reason for this failing is not obvious and does not seem to be coming from the changes I have introduced

Thanks for the PR!
As for the test failure, if memory serves me, you'd need to run rspec with -w to get the deprecation warning specs to work

No problem, this gem has been super helpful so far.  The least we can do is contribute back.
also, TIL about -w, Thx


PBot Explore: Collection Search Results

Here is a screenshot of what the "Quick Results" would look like for a collections search. We envisioned this in table format, but this may not be phone/tablet compatible. So, I could also see the information we want to show for each collection just listed underneath the collection name.

For min and max ages: for standardization on this quick view results, converting everything to international epoch would be preferred. However, if that is too difficult, we could display whatever timescale was entered for min and max by the user.
When people click on a collection, they would see the full search results, and we love the way it is set up now.
The Show me more data button is a placeholder to allow for data downloads.

Some of the summary results is implemented in
paleobot/pbot-client@11aeab475d9bb0daeb1236e32c813a3c7683cab0
merged to master and running on dev

Location (lat/lon) search is in
paleobot/pbot-api@652479b8a184ba3ba1a6b66e56dc413c58c874a8
paleobot/pbot-client@
paleobot/pbot-client@
paleobot/pbot-client@

Location (lat/lon) search is in
https://github.com/paleobot/pbot-api/commit/652479b8a184ba3ba1a6b66e56dc413c58c874a8
paleobot/pbot-client@1e362dc9dd9d57d4ac6bb18fbb2d0c4a304ccf71
paleobot/pbot-client@dc09931443d71efcf4a0979c95ff72235679552d
Merged to master and running on dev.
Note 1: To create the required location property in the Collection nodes, this cypher was run before turning this loose:
match (c:Collection  )
set
    c.location = Point({latitude: c.lat, longitude: c.lon})
return c

Note 2: The api changes were made in a way that should maintain continuity with the previous. In other words, you can access the deprecated lat and lon properties as well as the new location.latitude/location.longitude The lat and lon properties accessed through the api are generated on the fly from those inside location.
Note 3: Ideally, we would also delete the old lat and lon properties in the nodes. I'm leaving them around for now in case there are any latent problems with the conversion cypher in note 1. However, they are no longer accessible through the api (see note 2).

Note: 4: The search radius is currently set to 1500 meters. Let me know if we want it bigger/smaller.

Ah hah! I started playing about with what the radius was. We definitely want this bigger but Dori, Claire, Andrew, and I should consult on this. Maybe a whole degree? In v2, it might be nice to allow the user to specify a radius- I could see wanting a small radius for some questions but a big one for others. To some extent, you can get around the big radius now by doing a search by country or state.

Also, I think the quick search results look great. I'd prioritize getting other things done over standardizing min and max age or getting the number of specimens or OTUs at a site. Those would be icing on the cake.

My comments regarding location search were meant for #283. I've moved them there for clarity. Ellen, you might want to move your relevant comment as well so we track the discussion there.

In the collection record page, under references: when multiple references, there is nothing separating the different references, not even a space. Possible to have each reference as separate lines, like for specimens?

This may be too much work for the amount of time we have left, but it would be very helpful to add an accordion for OTUs to be able to show all the OTUs present at the site.

@ecurrano, I agree - I like that there is a list of linkable specimens, but a list of the OTUs identified at the site would be really useful as well.
I thought we could get that info a different way by searching OTUs and just refine by collection, but I just tried and realized that you can't search by a specific collection, just location/time.

There's no way to search by collection id? That's bad.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: doricon @.>
Sent: Wednesday, September 25, 2024 9:23:09 AM
To: paleobot/pbot-dev @.>
Cc: Zaffos, Andrew - (azaffos) @.>; Assign @.>
Subject: [EXT] Re: [paleobot/pbot-dev] PBot Explore: Collection Search Results (Issue #287)
External Email

@ecurranohttps://github.com/ecurrano, I agree - I like that there is a list of linkable specimens, but a list of the OTUs identified at the site would be really useful as well.
I thought we could get that info a different way by searching OTUs and just refine by collection, but I just tried and realized that you can't search by a specific collection, just location/time.
—
Reply to this email directly, view it on GitHubhttps://github.com/paleobot/pbot-dev/issues/287#issuecomment-2374548946, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACPQSQ325QRY5EII4LS4FHLZYLPO3AVCNFSM6AAAAABELLYJ6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZUGU2DQOJUGY.
You are receiving this because you were assigned.Message ID: @.***>


Add more examples to README

Instead of just and[not], there could be more examples that shows what you can do with Poi.

Here are two examples:
and[id x id -> not][not x not -> id]

and[not x not -> id][id x id -> not]

Here is also one that is interesting, since it shows equivalent expressions:
and[id x not -> id][not x id -> not]

This outputs:
and[id ⨯ not → id][not ⨯ id → not]
<=>  and[(not, id, not) · (id, not, id)]		( f[g][h] <=> f[h · g] )
<=>  and[id ⨯ not → id][(not, id, not) → id][id → (not, id, not)]		( f[g] <=> f[g → id][id → g] )

By copy-pasting and[(not, id, not) · (id, not, id)] it reduces to or:
and[(not, id, not) · (id, not, id)]
and[(not · id) ⨯ (id · not) → (not · id)]			( (x, y, z) · (a, b, c) => (x · a, y · b, z · c) )
and[not ⨯ (id · not) → (not · id)]			( x · id => x )
and[not ⨯ not → (not · id)]			( id · x => x )
and[not ⨯ not → not]			( x · id => x )
and[not]			( f[g ⨯ g → g] => f[g] )
or			( and[not] => or )

There could be a link to a tutorial from the README.
The tutorial could be markdown files in a folder, which is easier to edit than the wiki.

Some simple examples could also be added to help in Poi Reduce.

Here is another example: neg . mul[neg].
neg · mul[neg]
neg · (neg · mul)			( mul[neg] => neg · mul )
<=>  (neg · neg) · mul		( f · (g · h) <=> (f · g) · h )

By copy-pasting (neg · neg) · mul it reduces to mul:
(neg · neg) · mul
id · mul			( neg · neg => id )
mul			( id · x => x )


fix 'compute_ERC_h2'

          Hey Renzo, I used this recently and found a few bugs in the `compute_ERC_h2` function. I have made the corrections on my local repository. It might be helpful to change it here as well

Originally posted by @DamilolaDecarls in https://github.com/bonifazi/Integration_EBV_and_GEBV/issues/1#issuecomment-1775518813

Fixed and merged.


Gzip support for HTTP(S)Client/HTTP(S)Server

We should investigate how difficult it would be to add gzip support, as that can save huge amounts of bandwidth (and time).
(I'll take a look into possibly creating a lightweight Swift wrapper for it)
//Update: I created a zlib fork that builds as a SwiftPM package: https://github.com/czechboy0/zlib
Now we need to create a new package, probably something like Zewo/Gzip and make it Swifty to compress and uncompress passed data (presumably C7 Data), so that it can be used in other libs.
//Update2: I started working on a gzip library for Swift, just FYI. Will add link once it's out.

Ok, I created a self-contained (no external installation required) gzip library: https://github.com/czechboy0/gzip, which can work with either NSData or C7.Data, so let's integrate it into HTTP(S)Client and Server (but Client is probably more important, as Server can run behind a proxy which takes care of gzipping everything).

Another update: Now gzip has a GzipStream, which can be plugged into the S4.Body, meaning you can read/uncompress the body stream seamlessly. https://github.com/Zewo/gzip#usage
All that's left to do is find the right place in the stack to add this. (HTTPSClient/HTTPClient/...) or create a HTTPMiddleware repo where we can add this step so that it doesn't need to be duplicated in all 4 projects. @paulofaria thoughts?


deepgram: endpointing support number

https://github.com/jambonz/jambonz-feature-server/issues/329


Text search Spotify

Turns out you can actually search the Spotify API (with spotipy). Actually strikes me as weird to think it wasn't possible.
Anyway, this changes a lot regarding the behaviour of the app. Especially free text search could be a lot more powerful.
I want to be able to search Spotify now from the commandline.
Desired behaviour
There's multiple input modes:
Type-constrained
You can specify the input type as the first word. album, song, artist should be the available keywords. Will have to see if playlist is supported. May only be possible to query public playlists or the ones accessible to your user...
The query should be then forwarded to the spotify search, which returns the URI. The URI can then be used to costruct the objects as before and save as per usual.
Long term a interactive dialogue would be nice to select which specific song / album to choose, but for now just take the first.
Free form
Free form I think it would be great if it would not directly query, but would act similar to get_ids. It should take the top results and print them to the terminal {uri: (name, type, whatever)}.
Open question: Should it do that or should it only do that if you pass get_ids. Without get_ids it could just take the top result.
It should also take into account API query keywords.
spotify.search("artist:Frank Ocean track:Not Just Money")

IDEA: Maybe without --get_ids it prints a suggestion to use get_ids or format the string according to the format above.
API Reference: https://developer.spotify.com/documentation/web-api/reference/#/operations/search (could maybe link that in the print?).
-> This would also solve the multi-argument search problem #32
I could just make it a single argument that needs to be formatted in a specific way.
Get IDs
--get_ids is ambiguous as it currently only refers to genius and I'm shifting more to Spotify with this. Maybe make it --spotify_id and --genius_id?
This would help in two ways:

Differentiate between spotify and genius queries
Making it singular works better since you're normally just looking for a single resource.

Would be nice if I had a single function _print_search() which would handle pretty printing for --spotify_id and --genius_id
They should both be formatted the same way {ID: Rest}. The rest might be different, but otherwise should be the same.
Handling requests
By default freeform requests should be of type search.
How are search requests currently handled?
Currently: I would call Music.request(), which in turn would call the Genius or Spotify search function. Once I have searched, I would create the music album using the same Music.request() function.
How to best handle search requests?
It would be better to have another class function `Music.search(query, type="spotify","genius") that would search Spotify/Genius.
First: Make sure that query keywords are handled correctly (e.g. "artist:artist-name")
Then:
If requesting id:
    Print ids and quit()
else:
    Search and then return the best matching ID.

After calling Music.search, I have the id which can then be passed onwards to Music.request()

Will have to revert main to an earlier version where "nargs=+" isn't in it. Maybe not revert, but cherry-pick? Or just fix this individual thing?
I think I made some relevant changes after, but overall don't want to include changes that were made based on the nargs+ decision

Implemented split_search(query)
Takes a query and splits it up into keyword:value -> returns it as dict.
e.g.:
split_search("artist Frank Ocean track:Not Just Money")
--> {'artist': 'Frank Ocean', 'track': 'Not Just Money'}

The query is indifferent to colons being used. Currently doesn't support uppercase keywords as re.I didn't work as expected, but that's something to do at a later point in time
TODO: make split_search() capitalisation-independent

Implemented a first version of the search function!
It works really well if the search is accurate, i.e. if you perfectly format the query such as "artist: Frank Ocean track:Not Just Money".  It takes the first result from the search and then proceeds from there.
This is implemented for songs, albums, playlists, artists and queries without keywords.
Issue
The search always only takes the first result. This is not desirable, especially not for playlists. Users should have some say in which item gets selected.
Solution
Create a function which pretty prints the results from the search to the command line. It should have numbers associated with each of the items e.g.:
[0] Not Just Money    Channel Orange    Frank Ocean
[1] ....                           ....                           ....

The user can then type the number of the item they want and the system function will return the uri associated with that item. Maybe the user can type >, next or n to get the next row of results (or <, previous, p) respectively
Will need to look up how to print in columns (if possible).
This can likely be generalised and made to work for all types of queries. The question is how to select the attributes as they do depend on the type of the query.

Implemented a pretty print and allowed interactive input for tracks. Works well and looks pretty solid.
Hard to actually make it look good without knowing how big the terminal is, but that's a luxury problem.
Noticed that lyrics and most info aren't saved without --overwrite, even if the song/album doesn't exist yet. I suspect it has something to do with initialising empty files (don't need that for .json files anyway...).
Could be related to issue #32 ...
TODO: Fix --overwrite issue

Fixed the issue! Next step is adding interaction to the rest of the searches.
while looking for track:Reservations Wilco I found a sort of silent error. The only results that spotify returns are the remastered versions titled Reservations (2022 remaster) or smth like that. Passing that name onto genius returns the wrong song (ofc it does 🙄).
I could improve on the regex, but that also might remove parts of the title that are needed.
Probably just need to work on the --get_ids or --get_genius

Implemented the rest of the full text search for albums, playlists and artists.
"Only thing" left is the param for retrieving ids, but that's honestly secondary if you can just retrieve things interactively.
I will create a new ticket for that as it's really not high priority, just more of a "would be nice to have". Would prefer working on something else before implementing that.

Tried pulling Drake using the full-text search and spotify finds 10 other artists named Drake, but not the man himself.
Implementing a page feature so you can gather the next set of items until you get the right one.


OpenSUSE Leap Ruby2.6 Package broken

OpenSUSE package repository

The Ruby package I tested is obs://devel:languages:ruby/openSUSE_Leap_15.1 ruby (not obs://devel:languages:ruby/openSUSE_Leap_15.0 ruby)

Details
Hello OpenSUSE Team, I have recently worked a bit on Portus, and :

Other Community users, and I, have reproduced an issue using container image opensuse/protus:2.5 : see https://github.com/SUSE/Portus/issues/2241#issuecomment-593370789
I have provide one fix, that is not suitable for OpenSUSE distribution model, and anyway, not satisfying, it is more a temporary work around.
I released a container image definition providing that fix, here : https://github.com/pokusio/opensuzie-oci-library/releases/tag/0.0.2
In my release notes, I give you details :

explaining why my temporary fix is neither suitable for production, nor for OpenSUSE distribution policies,
explaining details about the only fix I see proper for OpenSUSE Teams, you can read it here : https://github.com/pokusio/opensuzie-oci-library/releases/tag/0.0.2 (Ctrl + F the string DEBIAN_)

I'll read with care any anwser / remarks / question, and would appreciate being informed of OpenSUSE Team opinion on that. (I understand there's recently been significant repository migrations going on last months).
Thank you for opening source code of Portus Source Code,
Regards

We have nes on this, see https://github.com/SUSE/Portus/issues/2279#issuecomment-596110419
Thanks to @JosuaSR

hi again @Josua-SR , After writing my sum up and questions above, I am almost sure that it is the Portus OpenSUSE package that has a bad design :
* in the `make_spec.sh`, hard-coded, and impossible to override
  https://github.com/SUSE/Portus/blob/53bd63e299f47f1761f55ba2bc53b9cb804c1616/packaging/suse/make_spec.sh#L6

Update
Yet, as of today, we don't have a final explanation for why and how it can happen that running portus requires a specific version of bundler :  this behavior does not make sense, since bundling an application, is not running it.  Bundling dependencies (even if clumsily managed), should not ever be Runtime dependencies. Any rails app should run without any dependency to a specific version of bundler. We, as of today, have for suspicions towards both the portus zypper package (for it may "pull" undesired old ruby gems, packaged inside the linux opensuse zypper package, see the spec file ), and the portusctl executable:  either one, or combination of both is responsible for


[question] is it possible build for ia32 on x64 linux?

I need to do it inside Travis-CI. Is it possible, how do you think?

Hi, if I remember correctly, Ubuntu doesn't come with i386 runtime and tools, but that's installable with apt-get. Then you gotta install i386 build-essentials, then node i386. If you run your build with that, then it builds ia32 automatically. Well, that's the theory actually. But by my understanding that should work that way.

Or maybe there is an option for Travis to use i386 container. Then you'll have tools for ia32 instantly.

Understand, thanks for explaining. I'll try.


7 chosen cards

Navigation bar will be removed later
Will implement a function where you can click on the card to change it
Some bugs that will be fixed

Needs more comments


Bad UTF-8 encoding

After upgrading my Visual Studio 2022 to latest stable release, I cannot build windows application.
PS X:\Projects2\flutter2\tech_control_app> flutter upgrade
Upgrading Flutter to 3.0.1 from 3.0.0 in X:\flutter_sdk\flutter...
Checking Dart SDK version...
Downloading Dart SDK from Flutter engine caaafc5604ee9172293eb84a381be6aadd660317...
Expanding downloaded archive...
Building flutter tool...
Running pub upgrade...
Upgrading engine...
Downloading android-arm-profile/windows-x64 tools...               315ms
Downloading android-arm-release/windows-x64 tools...               103ms
Downloading android-arm64-profile/windows-x64 tools...             118ms
Downloading android-arm64-release/windows-x64 tools...              99ms
Downloading android-x64-profile/windows-x64 tools...               106ms
Downloading android-x64-release/windows-x64 tools...                82ms
Downloading android-x86 tools...                                   636ms
Downloading android-x64 tools...                                   576ms
Downloading android-arm tools...                                   751ms
Downloading android-arm-profile tools...                           215ms
Downloading android-arm-release tools...                           164ms
Downloading android-arm64 tools...                                 547ms
Downloading android-arm64-profile tools...                         255ms
Downloading android-arm64-release tools...                         166ms
Downloading android-x64-profile tools...                           245ms
Downloading android-x64-release tools...                           174ms
Downloading android-x86-jit-release tools...                       312ms
Downloading Web SDK...                                           1 120ms
Downloading CanvasKit...                                           645ms
Downloading package sky_engine...                                   62ms
Downloading flutter_patched_sdk tools...                           166ms
Downloading flutter_patched_sdk_product tools...                   153ms
Downloading windows-x64 tools...                                 1 081ms
Downloading windows-x64/windows-x64-flutter tools...             2 292ms
Downloading windows-x64/flutter-cpp-client-wrapper tools...         16ms
Downloading windows-x64-profile/windows-x64-flutter tools...      1 877ms
Downloading windows-x64-release/windows-x64-flutter tools...      1 695ms
Downloading windows-x64/font-subset tools...                        72ms
Flutter 3.0.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision fb57da5f94 (35 hours ago) • 2022-05-19 15:50:29 -0700
Engine • revision caaafc5604
Tools • Dart 2.17.1 • DevTools 2.12.2
Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.0.1, on Microsoft Windows [Version 10.0.19044.1706], locale ru-RU)
[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc4)
[√] Chrome - develop for the web
[☠] Visual Studio - develop for Windows (the doctor check crashed)
X Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about this issue at https://github.com/flutter/flutter/issues.
X Exception: Bad UTF-8 encoding (U+FFFD; REPLACEMENT CHARACTER) found while decoding string: [
{
"instanceId": "adf64271",
"installDate": "2021-12-04T13:56:55Z",
"installationName": "VisualStudio/17.2.1+32516.85",
"installationPath": "C:\Program Files\Microsoft Visual Studio\2022\Community",
"installationVersion": "17.2.32516.85",
"productId": "Microsoft.VisualStudio.Product.Community",
"productPath": "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\devenv.exe",
"state":<PHONE_NUMBER>,
"isComplete": true,
"isLaunchable": true,
"isPrerelease": false,
"isRebootRequired": false,
"displayName": "Visual Studio Community 2022",
"description": "��魠� ��⥣�஢����� �।� ࠧࠡ�⪨, ��ᯫ�⭠� ��� ��㤥�⮢, ���⭨��� �஥�⮢ � ������ ����� � �⤥���� ���짮��⥫��.",
"channelId": "VisualStudio.17.Release",
"channelUri": "https://aka.ms/vs/17/release/channel",
"enginePath": "C:\Program Files (x86)\Microsoft Visual Studio\Installer\resources\app\ServiceHub\Services\Microsoft.VisualStudio.Setup.Service",
"installedChannelUri": "https://aka.ms/vs/17/release/channel",
"releaseNotes": "https://docs.microsoft.com/en-us/visualstudio/releases/2022/release-notes-v17.2#17.2.1",
"thirdPartyNotices": "https://go.microsoft.com/fwlink/?LinkId=661288",
"updateDate": "2022-05-17T20:12:23.1558052Z",
"catalog": {
"buildBranch": "d17.2",
"buildVersion": "17.2.32516.85",
"id": "VisualStudio/17.2.1+32516.85",
"localBuild": "build-lab",
"manifestName": "VisualStudio",
"manifestType": "installer",
"productDisplayVersion": "17.2.1",
"productLine": "Dev17",
"productLineVersion": "2022",
"productMilestone": "RTW",
"productMilestoneIsPreRelease": "False",
"productName": "Visual Studio",
"productPatchVersion": "1",
"productPreReleaseMilestoneSuffix": "1.0",
"productSemanticVersion": "17.2.1+32516.85",
"requiredEngineVersion": "3.2.2148.26540"
},
"properties": {
"campaignId": "1870145771.1592411815",
"channelManifestId": "VisualStudio.17.Release/17.2.1+32516.85",
"nickname": "",
"setupEngineFilePath": "C:\Program Files (x86)\Microsoft Visual Studio\Installer\setup.exe"
}
}
]
.  Doctor found issues in 1 category.

Duplicate of https://github.com/flutter/flutter/issues/102451


[WIP] Send graphql errors to sentry

Resolves: #952 ??

Not sure why testing locally it sent 2 errors to sentry 🤔
https://sentry.io/organizations/iasql/issues/3416731240/?project=6327420&referrer=webhooks_plugin
https://sentry.io/organizations/iasql/issues/3416731206/?project=6327420&referrer=webhooks_plugin

No longer using graphql in the engine


How to acquire the data? The link is invalid

Hello, I tried to acquire the data and pretrained model for "Image Caption" through the links you offered, but I was unable to access to the website. Would you please reupload the data and model?  Thank you!
The links you offered are as follows:
Pretrained model: https://www.dropbox.com/s/ne0ixz5d58ccbbz/pretrained_model.zip?dl=0
Vocab File: https://www.dropbox.com/s/26adb7y9m98uisa/vocap.zip?dl=0

I got the data through the link you offer. Maybe something went wrong with my internet.
Thankyou!


KAFKA-13590: rename InternalTopologyBuilder#topicGroups

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

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

@sayantanu-dey hi, I wonder where the name came from, it's really a good name in context.

@sayantanu-dey hi, I wonder where the name came from, it's really a good name in context.

@Kvicii hi, thanks, I just saw that the function was returning the Map of subtopology and TopicInfo, so I thought of having that as the name :)


Added missing import to init.py

from fastapi_versionizer import api_version_remove did not work, because api_version_remove was not imported in __init__.py

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Crash Report<PHONE_NUMBER>

Link:
https://wowsims.github.io/cata/druid/guardian/#eJzjEuVgdGDMYGxgZJzAyNjAxLiBifECE6MTpwCjBaMH4w1GRismAUYhBqknbLOY2QJyEitTizi4BRil9jNy8QususasxHZsBvOkGcwafYxcLAIHe1i4eASmHWNWgomxC5w/BmbwC2w7D1Q9CaJ6MSMXl8CpncxKjAwaUxm5eAUePmdWYgHKQLk3gVIsfSBuO0hr/weg1j64RTwCn05ArZgKsvYA0Fp2gTs7oNwb25mB3Mk7wNawC8xaB9bEANQ2/ydcmxIXB7OBZABTBFMCq9ZRJq4fjAwjFAh5UMskKwYnhlXM3FKcghC+nsMuJkcDLEDXyNjAwNjIyNgQiIB8QyMjA2NDQ10DIAUWQAaHmFQ4Fj5hEpjTzyJxZD+TwsH9TBpn9zMZHNjPZDHrKpPDsUtMHocvMZ1gYr3AxHmLiVNw1kwQuGn/iKmdUYJJqYmRq7BaKTcxM68EiN0SU4syU90yi1KVrEqKSlN14DIuqbn5RYk5mVWZeelB+YlFMPkUoHhIZi5QvYmOUkFpThnQgKpUhEhRKlAwJ7gkMbcgNQVqaq0QLxcTFwPcKS+YUn4wMS5khoRLpEMXswhnKAjM3+sADSwmBwU2pTNMnGBOQxpMWMRBEmLISXtLiMgFe8U0MLhm73j2DAi8sTeazKQQIcGqReNUrNHpQIryhpTjJKmnBEACZ+97B0cI/4d9AiMAe5a8YQ==
RNG Seed:<PHONE_NUMBER>
simulation failed: When validating reforging for item 63503, the reforging could not be validated. 142
Stack Trace:
goroutine 13 [running]:
runtime/debug.Stack()
/opt/hostedtoolcache/go/1.21.11/x64/src/runtime/debug/stack.go:24 +0x6
github.com/wowsims/cata/sim/core.runSim.func1()
/home/runner/work/cata/cata/sim/core/sim.go:124 +0x10
panic({0x108ec0, 0x334f1a0})
/opt/hostedtoolcache/go/1.21.11/x64/src/runtime/panic.go:914 +0x2c
github.com/wowsims/cata/sim/core.NewItem({0xf80f, 0x0, 0x0, {0x330fa90, 0x2, 0x2}, 0x8e})
/home/runner/work/cata/cata/sim/core/database.go:362 +0x30
github.com/wowsims/cata/sim/core.NewEquipmentSet({{0xeb2a, 0x0, 0x0, {0x330fa60, 0x2, 0x2}, 0x8e}, {0x10641, 0x0, 0x0, ...}, ...})
/home/runner/work/cata/cata/sim/core/database.go:420 +0x6
github.com/wowsims/cata/sim/core.ProtoToEquipment(0x33b9000)
/home/runner/work/cata/cata/sim/core/database.go:427 +0xd
github.com/wowsims/cata/sim/core.NewCharacter(0x31b5c20, 0x0, 0x33be380)
/home/runner/work/cata/cata/sim/core/character.go:118 +0x39
github.com/wowsims/cata/sim/core.NewAgent(0x31b5c20, 0x0, 0x33be380)
/home/runner/work/cata/cata/sim/core/agent.go:172 +0x1e
github.com/...

What tool did you use to import your gear? If none, could you tell us what steps you took to replicate the error?

I used the weakaura export button, that show up when viewing ur gear.

The weakaura is not supported or created by the wowsims team. The creator needs to update it to handle random suffix items. I would suggest using the addon.


E-MTAB-9841 - Transcriptional changes in the mammary gland during lactation revealed by single cell sequencing of cells from human milk

Project short name: MammaryLactactingCells
Primary Wrangler: @arschat
Secondary Wrangler: @idazucchi

Associated files

Google Drive: https://drive.google.com/drive/folders/1k5_XmKEKVrvb5Bm7bsNU3d05htI0oqhH?usp=share_link
Ingest: project

Published study links

Paper: https://www.nature.com/articles/s41467-021-27895-0

Accessioned data: arrayexpress_batch1
arrayexpress_batch2
arrayexpress_batch3

Key Events

[x] Convert published metadata to HCA spreadsheet
[x] Manually curate dataset to meet HCA metadata standard
[x] Collect any matrix and cell-type annotation files
[x] Upload sheet to validate metadata
[x] Transfer raw files to ingest to validate data files
[x] Check linking using ingest graph validator
[x] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Submit dataset to Production
[ ] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

Hi Arsenios, I'm done with the secondary review! You did a great job with this dataset!
I have some comments on fields you can fill in and a few about best practices, let me know if you have any questions.
Project

some contributors have England listed for the country, it would be better to swap it for United Kingdom because England is a region within the United Kingdom

Sepcimen from organism

NMC1B_L1, NMC1B_L2 , NMC1B_L3 are lane replicates but the original sample is the same for all of them and it was processed in the same way. You can use only NMC1 to represent the specimen.
you can add preservation cryopreservation, other for all breast specimens
you can add preservation cryopreservation, other for all milk specimens marked as viably frozen
LMC2 and LMC2B`  are the same sample but they were stored in two different ways, you can describe that using the storage/processing fields

Cell suspension

the milk tissue should have an enrichment protocol applied to it density gradient centrifugation to explain how the cells were isolated from their matrix. Dissociation protocols are only applied to a solid tissue that has to be broken down
the experiment accessions is not needed here, they are only needed for the sequence files to represent the sequencing experiment

Sequencing protocol

you can remove the library preparation form the description
you can add the fastq creation method and version

Sequence file

you could fill in the read length
files MUC9105.R1.fastq.gz, MUC9105.R2.fastq.gz should have v2 library preparation and sequencing
you can group the file in process based on their experiment accession, the lane parameter can be used to group them 2-by-2 or 3-by-3

Analysis protocol

the computational method field was used by our pipelines team but they are no longer workinf with us, so the field is now used to capture the software used for analysis, like Seurat, AnnData ...

Analysis file

NMC4_cell_suspension doesn't have any barcode file, the file is not available in ArrayExpress. It would be a good idea to get in touch with the authors and ask for the missing file
for the matrix file Count matrix is the preferred term, gene expression matrix is more specific for microarrays
for feature files we use feature table
you can add the cell count for the matrices
the information on reference genome is conflicting: the paper says hg19 (so GRCh37) but the ArrayExpress accession say either GRCh38 or genome ensemble release 94, which I think is GRCh38

Contacted authors for genome version, and missing NMC4_cell_suspension barcode file.

Authors provided the genome version (the arrayExpress) as well as the missing file (although it was sent privately it contains cell barcodes). Will proceed with a new complete submission and export.

Surgically removed duplicated files in the terra staging area (file descriptor and supplementary files.json)


[vmla] xla_hlo.reverse is not working properly

test/e2e/xla/reverse.mlir
Actual:
2x3xf32=[6 5 4][3 2 1]
2x3xf32=[6 5 4][3 2 1]
2x3xf32=[6 5 4][3 2 1]
Expected:
// CHECK: 2x3xf32=[4 5 6][1 2 3]
// CHECK-NEXT: 2x3xf32=[3 2 1][6 5 4]
// CHECK-NEXT: 2x3xf32=[6 5 4][3 2 1]

This was fixed by one of Ben's PRs (allocate/clone were being treated as pure, allowing them to be garbled by passes like CSE)


Document best practices for static sites built with Node.js

This seems to be a common use of the image - to create static single-page apps that are built from React or Angular codebases. Until we create a separate builder image tuned for this kind of usage, we need to document best practices.
See: https://github.com/bucharest-gold/centos7-s2i-nodejs/issues/94

Closing this as work is ongoing to develop a dedicated static site s2i builder image: https://github.com/bucharest-gold/centos7-s2i-web-app


Getting Promise implementation for async methods from a property of the current class

As per @rbuckton in #3364:

For an async function, the return type annotation must be both a type and a value in the same scope.

It's nice as it allows using any compatible Promise implementation. However, it requires that this implementation should be assigned to a variable in the scope. Unfortunately, this doesn't always work well with AngularJS (Angular 1.x), whose approach discourages globals in favour of dependency injection for the sake of testability. Consider this code of an AngularJS controller:
type $q<T> = Promise<T>;
class MainController {
    public name: string;
    constructor(private $q: PromiseConstructor, private myService: MyService) {}
    async fn1(): $q<void> {
        this.name = await this.myService.fn1();
    }
}
module.angular('myApp').controller('MainController', MainController);

The promise implementation $q is injected to the constructor by AngularJS here. Of course, it doesn't work as TypeScript looks for $q, not for this.$q. So as for now, the only solution for this situation is saving $q to a global variable. Please consider adding the property lookup for async methods.

You mean being able to write the following?
    async fn1(): this.$q<void> {
        this.name = await this.myService.fn1();
    }

I don't think a new syntax is needed. Now when the TypeScript compiler sees this:
    async fn1(): $q<void> {
        this.name = await this.myService.fn1();
    }

it looks for the variable $q. I propose that it should look first for the property $q within the current class, and then for the variable if the property isn't found. Not having a special syntax for this would, in particular, allow omitting the type declaration if $q is assigned to the property Promise.
class MainController {
    public name: string;
    constructor(private Promise: PromiseConstructor, private myService: MyService) {}
    async fn1() { // the return type is inferred as Promise<void>
        this.name = await this.myService.fn1();
    }
}
module.angular('myApp')
    .factory('Promise', $q => $q)
    .controller('MainController', MainController);

would be transpiled to
    // ...
    fn1() {
        return __awaiter(this, void 0, this.Promise, function* () {
            this.name = yield this.myService.fn1();
        });
    }
    // ...

Angular users would really appreciate such clean code.
However, there is a risk with this code that the developer can forget to inject Promise to the constructor so that the global variable Promise would be silently used by the TS compiler instead of the property as there is an ambient declaration of this variable in lib.es6.d.ts. Is there a way to 'undeclare' it somehow?

What if in place of the current logic, we had some kind of mapping in tsconfig:
async return type -> promise constructor expression
?
"promiseConstructors": {
    "ng.IPromise": "this.$q",
    "QQPromise": "this.$qq"
}

After a successful lookup, the found expression should be type-checked, so that in the example above, we have a compiler error if the current class doesn't have the $q property.
IMHO such a mapping looks more explicit and less magic than the current logic.

Solved this by overriding the __awaiter helper. Fortunately, the context (this) is passed to __awaiter, so this.$q can be accessed:
const __awaiter = (
	thisArg: { $q: PromiseConstructor },
	_arguments: any,
	P: any,
	generatorFunction: GeneratorFunction
) => {
	P = thisArg.$q;
	if (!P) throw Error('$q not defined in ' + thisArg.constructor.name);
	return new P<any>((resolve, reject) => {
		// ... the rest in unchanged
	});
};


Nuget and DNU not working properly with Asp5-Mvc6

http://stackoverflow.com/questions/33919834/nuget-not-working-properly-with-asp5-mvc6
Cant fint any issue creater for Nuget

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Failed to decode the unity prefab due to an error

i am trying to import a vrchat avatar but i cannot import the .unitypackage file because it says "Failed to decode the unity prefab due to an error" and at the bottom it says:
"instanciated 817 unity objects/components/meshes for prefab! Now cleaning up"
altough i couldn't follow the install steps as you cannot access the website where the zip file is contained
it just gives back the error: {"message":"Artifact not found or access denied."}

Thanks for the issue report, can you please attach your latest log here?

Also, if you could attach the unity package here that would be helpful

as the one who wrote the code, it is impossible to debug without the package. thousands of things can go wrong

REMOTEHUN - 2<IP_ADDRESS> - 2024-05-26 18_41_00.log
Here is the log and
https://drive.google.com/file/d/1nH6Ks3Mav-3NLWzPiCOGZEgJ2NRf4AI7/view?usp=sharing
here is the unity package file (i had to use google drive because of the size limit)

apparently it's missing the fbx model
i placed it in and it works now


node-canvas does not have prebuilt binaries for node 22

Before proceeding, is there an existing issue or discussion for this?

[X] I have done a search for similar issues and discussions.

OS and version
Ubuntu 24.04
Open-RMF installation type
Source build
Other Open-RMF installation methods
No response
Open-RMF version or commit hash
main
ROS distribution
Jazzy
ROS installation type
Binaries
Other ROS installation methods
No response
Package or library, if applicable
No response
Description of the bug
The version of node-canvas (v2) used does not have prebuilt binaries for the latest node lts (node 22). It can still be built from source but you will need to install several system dependencies. We should update to node-canvas >= 3 which uses the newer NAPI api.
Steps to reproduce the bug
pnpm install on node 22.
Expected behavior
No response
Actual behavior
No response
Additional information or screenshots
No response

This is causing nightly to fail.

I have the same problem when I install the dependencies on ros2-jazzy. After I remove the canvas from package.json, it seems work well.


https://jsconsole.com get a net::ERR_CONNECTION_CLOSED

Hello,
as first i must say is a very nice tool, i have test it for few weeks.
But now i will debug again and it work not more :(
i get this errors on my browser:
GET https://jsconsole.com/remote/5d3bcc15-a686-4250-bd87-cc1d8e09777f/run net::ERR_CONNECTION_CLOSED
https://jsconsole.com/remote/5d3bcc15-a686-4250-bd87-cc1d8e09777f/log Failed to load resource: net::ERR_CONNECTION_CLOSED

i hope you can fix or help.
Best regards,
Stefan

Something causes jsconsole to break, but I'm not sure what it is. If you can replicate consistently, please let me know.

@remy It happens for me at the moment.
I just used :listen and pasted the script tag into my page. It worked well yesterday.

@andertun I'm seeing this as well.

This is back online for the time being.


Correct TEST.ROOT copy if logic

Signed-off-by: andrew-m-leonard<EMAIL_ADDRESS>Original change used "openjdk1?-openj9" but it seems that is not supported.
Changed it to use  operation.

@smlambert Shelly, please can you review this? it seems original fix didn;t work...

My original suggest is matches   not not equals  <matches string="${JVM_VERSION}" pattern="openjdk1?-openj9"> .   Though or is good too.


problem saving file names with colons

I don't see this reported elsewhere, but my students working on PCs are unable to use the -sav switches in many PmagPy programs because the output files all have colons in them which is not permitted by the Windows operating system.  For example, dayplot_magic.py or eqarea_magic.py.

Interesting. It seems like this should be a pretty straight-forward fix. @ltauxe, any issue with getting rid of the colons in these output file names from your perspective?

It should be an easy fix but we need to make sure we don’t break things for Rupert’s use on the server.  I believe we just need to use the isServer flag.  This is something we discussed a bit before the workshop, but couldn't get to.  I can take this one.

Thank you!!
Julie

On Feb 14, 2017, at 12:49 PM, moonshoes87<EMAIL_ADDRESS>wrote:
It should be an easy fix but we need to make sure we don’t break things for Rupert’s use on the server. I believe we just need to use the isServer flag. This is something we discussed a bit before the workshop, but couldn't get to. I can take this one.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/143#issuecomment-279798373, or mute the thread https://github.com/notifications/unsubscribe-auth/AYNj8CbkHeIqs6G0pQQLuQJzwmsRwmmPks5rcfdDgaJpZM4L_5Z9.

I just released pmagpy/pmagpy-cli 3.8.7.  The colons will now be replaced with underscores, so the auto-generated filenames should no longer be illegal in Windows.  I am not closing this issue yet because I want to go through the programs and output a more sensible, readable name for each file, but for now this should prevent things from breaking.
Thanks for reporting this issue, @jbowles100!

no problem at all.  Rupert wanted them, but they are not essential.
lisa

On Feb 14, 2017, at 9:11 AM, Nick Swanson-Hysell<EMAIL_ADDRESS>wrote:
Interesting. It seems like this should be a pretty straight-forward fix. @ltauxe https://github.com/ltauxe, any issue with getting rid of the colons in these output file names from your perspective?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/143#issuecomment-279771080, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKrDbUzznZReVqQuJwkf-HaeEHBqwlyks5rceAugaJpZM4L_5Z9.

Thank you for fixing this so quickly!
Julie

Julie A Bowles
Assistant Professor
Department of Geosciences
University of Wisconsin - Milwaukee
PO Box 413, Lapham Hall 366
Milwaukee, WI  53201
bowlesj<EMAIL_ADDRESS>414-229-6110
http://people.uwm.edu/bowlesj/

On Feb 14, 2017, at 6:02 PM, moonshoes87 <notifications<EMAIL_ADDRESS>wrote:
I just released pmagpy/pmagpy-cli 3.8.7. The colons will now be replaced with underscores, so the auto-generated filenames should no longer be illegal in Windows. I am not closing this issue yet because I want to go through the programs and output a more sensible, readable name for each file, but for now this should prevent things from breaking.
Thanks for reporting this issue, @jbowles100https://github.com/jbowles100!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PmagPy/PmagPy/issues/143#issuecomment-279876353, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYNj8I7kLSahmbjMQFXPjgKuWTBjBBZMks5rckCxgaJpZM4L_5Z9.

No problem!
@ltauxe, I am preserving the old style of naming for on the server, so this shouldn’t affect Rupert at all.

On Feb 15, 2017, at 7:42 AM, jbowles100<EMAIL_ADDRESS>wrote:
Thank you for fixing this so quickly!
Julie

Julie A Bowles
Assistant Professor
Department of Geosciences
University of Wisconsin - Milwaukee
PO Box 413, Lapham Hall 366
Milwaukee, WI 53201
bowlesj<EMAIL_ADDRESS>414-229-6110
http://people.uwm.edu/bowlesj/

On Feb 14, 2017, at 6:02 PM, moonshoes87 <notifications<EMAIL_ADDRESS>wrote:
I just released pmagpy/pmagpy-cli 3.8.7. The colons will now be replaced with underscores, so the auto-generated filenames should no longer be illegal in Windows. I am not closing this issue yet because I want to go through the programs and output a more sensible, readable name for each file, but for now this should prevent things from breaking.
Thanks for reporting this issue, @jbowles100https://github.com/jbowles100!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PmagPy/PmagPy/issues/143#issuecomment-279876353, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYNj8I7kLSahmbjMQFXPjgKuWTBjBBZMks5rckCxgaJpZM4L_5Z9.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/143#issuecomment-280046116, or mute the thread https://github.com/notifications/unsubscribe-auth/ABwGIo59-v_0DcADIL4yhzuBLtlwpI1lks5rcxz3gaJpZM4L_5Z9.

The following commits make more readable plot names when not used on the server: 95bbf0995069b2c5ee15647550648ec966121ba4, dca344068dee9db4392f5eb45c0e21f23348cdcf,  167d8d7955eb6445b3f6762abe355ee82a472143.


Gate.io - add market and limit_maker order type

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Based on the Technical Roadmap, we are going to support all order types.
In this case, this PR will allow sending market&limit_maker orders with gateio.
Tests performed by the developer:
unittest
Tips for QA testing:

PR update:

Ran test on PMM and AMM-arb (CEX connectors) all ok

Pending:

Looking for scripts to use to test the added market and limit_maker


[$250] Chat - Chat history screen starts to shake when 'New Messages' badge appears

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

Version Number: 1.4.74-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/vifew/4565233
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Open iOS App
Login with any account with long chats
Open any chat with long history
Clear any unread messages if needed.
Scroll up a few screens in the chat
Send a message to this chat from the second device

Expected Result:
The "New Message" badge appears at the top of the screen without any distortion
Actual Result:
Chat history screen starts to shake when 'New Messages' badge appears
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/06bc02cd-ec58-45ab-bbf2-832d9700a37d
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011f7777bd257c3722
Upwork Job ID:<PHONE_NUMBER>125892096
Last Price Increase: 2024-05-27

Issue OwnerCurrent Issue Owner: @jayeshmangwani

@lschurr  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

Hello,  @jayeshmangwani
Let me try to solve this issue.
I think the one of these are listed below can be the cause.

The app might be trying to adjust the scroll position automatically when a new message arrives.
Inserting new messages might cause a full layout pass, leading to the shake.
Improper constraints might cause the entire view to re-layout in an unexpected way.
Frequent updates to the "New Message" badge can cause rapid layout changes.
I will find more.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: Don't have.

@emilkaminski1022 Please check our Contributing Guidelines to understand the process

Okay, thanks.

No proposals yet, Waiting on proposal

Still waiting on proposals.

Not overdue.

@jayeshmangwani can you confirm this bug is still happening?

@lschurr I just tested in latest app store build(v1.4.77-11) and issue is not happening for me, I can see the New Message badge and there no screen flickering.
https://github.com/Expensify/App/assets/35371050/c88b545c-12bc-4a1d-9e2f-4584994f4e75

Alright, thanks! I'm going to close this one since it's not happening anymore.


No available formula with the name "eigenpy-python3"

Running brew install pinocchio currently fails due to eigenpy now using python3 by default (there is no eigenpy-python3 formula anymore).
This is the output:
==> Installing pinocchio from gepetto/gepetto
Error: No available formula with the name "eigenpy-python3" (dependency of gepetto/gepetto/pinocchio)
==> Searching for a previously deleted formula (in the last month)...
Warning: homebrew/core is shallow clone. To get complete history run:
  git -C "$(brew --repo homebrew/core)" fetch --unshallow

Error: No previously deleted formula found.
==> Searching for similarly named formulae...
Error: No similarly named formulae found.
==> Searching taps...
==> Searching taps on GitHub...
Error: No formulae found in taps.```

@ialong I'm currently regenerating the images. In 10 minutes, it should be available.

great thanks! (amazing response speed :))

Done! The binaries are now available.
Best,
Justin

It seems I'm still getting the same issue (after running brew update)...

I would advise you to proceed as follows:
brew untap gepetto/geptto
brew tap gepetto/geptto
brew install pinocchio

or, for Python 2:
brew install pinocchio@2

@jcarpent unfortunately still the same error.

OK. I've found the bug.

I just need 5 mins to solve it and regenerate the correct image

Done. Thanks @ialong for your quick feedback.

It worked! @jcarpent thank you very much for fixing this so quickly

You're welcome.


Only error types defined as 'public' can be used in the catch statement

import ballerina/io;

type CustomError record {
    string message;
    error? cause;
    string extraField;
};

function main(string... args) {
    try{
        error e = {message: "builtin error"};
        throw e;
    } catch (CustomError e) {
        io:println("Catching custom error");
    } finally {
        io:println("Finally");
    }
}

Above program gives the below error:
error: ./testMain.bal:13:14: incompatible types: expected 'error', found 'CustomError'

If the public keyword is added to the CustomError record definition, then the above program works as expected.

This is no longer valid, as try-catch is no longer supported by the language.


Image in  is not rendered correctly

This is a funny one. :smile:
An image in <stepresult> is only rendered in PDF2, if it appears at another position in the same topic. Please have a look at the test files. Tested with dita-ot2.2.5.
2411.zip

I was able to reproduce this with the current develop branch with FOP, but not with AHF.
Looks like a FOP bug then, since topic.fo does look right:
<fo:block font-size="10pt" start-indent="25pt">An image in <fo:inline line-height-shift-adjustment="disregard-shifts" font-family="Courier New, Courier, Arial Unicode MS, Tahoma, Batang, SimSun"><stepresult></fo:inline> is not rendered.</fo:block>
<fo:block font-size="10pt" start-indent="25pt">
  <fo:list-block provisional-distance-between-starts="5mm" provisional-label-separation="1mm" space-after="0.6em" space-before="0.6em">
    <fo:list-item relative-align="baseline" space-after="3pt" space-before="3pt">
      <fo:list-item-label end-indent="label-end()" keep-together.within-line="always" keep-with-next.within-line="always">
        <fo:block font-weight="bold" text-align="start">
          <fo:inline/>
        </fo:block>
      </fo:list-item-label>
      <fo:list-item-body start-indent="body-start()">
        <fo:block>
          <fo:block>
        </fo:block>
          <fo:block>
            <fo:inline id="unique_1_Connect_42_2">
              <fo:external-graphic src="url('file:/Users/eerohele/Downloads/2411/image.png')"/>
            </fo:inline>
          </fo:block>
        </fo:block>
      </fo:list-item-body>
    </fo:list-item>
  </fo:list-block>
</fo:block>

Quite bizarre indeed.

I'm no longer able to reproduce this with latest develop code. We've updated FOP since the last time there was a comment here - I'm guessing FOP 2.1 has resolved this issue. @xephon2 I think we can mark this closed now?

Hi @robander, I have tested this and also cannot reproduce this anymore.