[DRAFT] Point propeller to Plugin metric branch

TL;DR
Please replace this text with a description of what this PR accomplishes.
Type

[ ] Bug Fix
[] Feature
[X] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
https://github.com/lyft/flyte/issues/
Follow-up issue
NA
OR
https://github.com/lyft/flyte/issues/

Codecov Report

Merging #120 into master will not change coverage.
The diff coverage is n/a.


Handling of // protocol resolving

Hi
Seems that similar to https://github.com/addyosmani/grunt-uncss/issues/13 error appeared again in grunt-uncss 0.4.0 and higher versions, which definitely has something to do with uncss itself:
###
Uncss
https://github.com/addyosmani/grunt-uncss
Remove unused CSS
###
module.exports = ->
  @config 'uncss',
    build:
      options:
        htmlroot: 'build'
      files: [
        src: 'build/{,**/}*.html'
        dest: build/assets/styles/style.tidy.css'
      ]

	<!-- Fonts -->
	<link rel="stylesheet" href="//fonts.googleapis.com/css?family=Open+Sans:300italic,400italic,600italic,700italic,400,300,600,700&subset=latin,cyrillic">

results in:

Please note that there is nowhere can be found path build\css in project or microtemplate's variables, while actuals project's css stored in build\assets\styles
Using of full protocol name http://fonts.googleapis.com/css?family=Open+Sans:300italic,400italic,600italic,700italic,400,300,600,700&subset=latin,cyrillic fixes that issue, but it should work with // too I guess
In grunt-uncss 0.3.8, which works with lower than 0.10.0 uncss version all works fine

Tested with 0.12.1 — issue still persists.

Just tested it with lastes version, problem is still present. Any comment on this?

Confirming this is still an issue.

Is this still the case with the latest release?

Upgrading to latest release gets me Fatal error: write EPIPE error.

@ArmorDarks @addyosmani Is this still an issue? Is anyone using // protocols anymore?

Seems like issue was resolved. At least I'm not getting any problems with //

@mikelambert Can you close this?


Make the wallet available on chrome extension

Motivation
majority of user will be scared of downloading random file into there pc and installing the extension manually due to a lot of scams and hacking occurring lately.
Usage example
I have a friend I introduced fuel beta testing to he said this is scam immediately he discovered the process of installing the extension  as result of his past experience. I need to take my time and explain everything to him.
Possible implementations
Make the wallet extension available on goggle chrome, Thank You

@HermesXchange, thanks for the issue. As the current version is beta and with some main features missing. Because of this, for now the installation is just via zip file, but the release on the web store is soon. Keep track here #125.


Support for dropping other dragsources

Any plans on supporting other dragsources. Was looking at this wonderful project to allow files to be moved from another view than the tree-view.

Did this feature get attention?


add Resource Manager resource generator

Since the deployment method for Azure uses a Resource Manager template, it would be nice to be able to easily add new resources to the template, like a container registry or storage account.
Could we add a Buffalo generator to create Resource Manager resources in the user's template? We should make it generic so it can take any Resource Manager "type" and collection of "properties", and demonstrate adding a Storage account, a CosmosDB/MongoDB account and an Event Hubs namespace.

I don't mind that idea at all, let's see if we can garner any community support for doing this.


Unclear how to store RateLimiter in struct

Minor thing I encountered while updating a small project from rate_limiter to governor. It's using it in a very basic way, but roughly like:
struct Example {
    rate_limit: DirectRateLimiter<GCRA>,
}

However on first glance, it's not clear how to go from the example in the governor docs:
use std::num::NonZeroU32;
use nonzero_ext::*;
use governor::{Quota, RateLimiter};

let mut lim = RateLimiter::direct(Quota::per_second(nonzero!(50u32))); // Allow 50 units per second
assert_eq!(Ok(()), lim.check());

to something along these lines:
use std::num::NonZeroU32;
use nonzero_ext::*;
use governor::{Quota, RateLimiter};

struct Example {
    limiter: RateLimiter<???>,
}

let eg = Example {
    limiter: RateLimiter::direct(Quota::per_second(nonzero!(50u32))),
}
assert_eq!(Ok(()), eg.limiter.check()); // although these are in methods on Example

All the examples I could find just create the limiter in a function (so just simple let x =  binding), and working out the generic parameters is slightly unclear
Is this an "expected" use of the API, or is there a more conventional approach which avoids needing to specify the generic parameters?

Hi, thanks for your interest! How to store a rate limiter in structs is definitely a thing that the docs ought to cover better, right on the cover page. Also, the types aren't super ergonomic for people that can't rely on type inference at the moment - even if you only use the defaults, as a person putting rate limiters on structs, you have to type out all the type args and that's tedious.
To answer your question specifically, I think I need to start with the type signature you're trying to form: It's not the way a governor RateLimiter should look - it looks like one from ratelimit_meter (the preceding crate, which had many design flaws, one of them being that type signature)! The concrete RateLimiter type signature is this:
RateLimiter<KeyType, StateStore, Clock>
A direct rate limiter that uses the default real-time clock would have this type signature:
RateLimiter<NotKeyed, InMemoryState, DefaultClock> - you can find that signature in the docs for to the return type of RateLimiter::direct.
Now, I should also note that in the next release, 0.4.x, there'll be a new generic parameter with a default, which will still require that your struct type sig changes; I will leave this issue open to ensure the next release has a more convenient type definition that remains backwards-compatible (and doesn't require people to spend ages digging through docs!)
Hope that helps!

I don't get autocomplete for the RateLimiter constructor like methods besides of new.

That's curious (but may be related to the near-extreme levels of genericness going on)... are you using rust-analyzer? What version? With rust-analyzer 2021-09-06, I get the following:

Switching my vscode extension from rust to rust-analyzer made the autocomplete work! Thanks!

The next release, 0.6.0, will have DefaultKeyedRateLimiter and DefaultDirectRateLimiter type aliases. Hope they will cut down on the type typing we have to do!


fix(ecma/transform/compat): private getter/setter

Description:

BREAKING CHANGE:

Related issue (if exists):
closes #3618

Did you close this intentionally?

Did you close this intentionally?

No, I rename the branch, and somehow this pr closes. A new pr is in #3695


v2.3.4

Change js convention

Codecov Report

Merging #73 into master will decrease coverage by 0.51%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
- Coverage   98.38%   97.86%   -0.52%
==========================================
  Files          31       24       -7
  Lines         618      469     -149
==========================================
- Hits          608      459     -149
  Misses         10       10

Impacted Files
Coverage Δ

src/utils/chooseBestURL.js
94.11% <100%> (-0.33%)
:arrow_down:

src/utils/loadHTML.js
100% <100%> (ø)
:arrow_up:

src/utils/logger.js
100% <100%> (ø)

src/parsers/extensions/youtube.js
90% <100%> (-0.63%)
:arrow_down:

src/parsers/extensions/vimeo.js
88.88% <100%> (ø)
:arrow_up:

src/utils/index.js
100% <100%> (ø)
:arrow_up:

src/parsers/extractWithReadability.js
100% <100%> (ø)
:arrow_up:

src/utils/getTimeToRead.js
100% <100%> (ø)
:arrow_up:

index.js
100% <100%> (ø)
:arrow_up:

src/utils/removeUTM.js
81.81% <100%> (ø)
:arrow_up:

... and 15 more

Continue to review full report at Codecov.

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


Eclipse .classpath paths and loose application paths are inconsistent

One of the biggest devexp features of Liberty is that your IDE can compile code for you and Liberty can "hot update" the app in a few milliseconds, without needing to restart the app or server.
When make a standard java gradle project, my .classpath looks like this:
<classpath>
	<classpathentry kind="src" output="bin/main" path="src/main/java">
		<attributes>
			<attribute name="gradle_scope" value="main"/>
			<attribute name="gradle_used_by_scope" value="main,test"/>
		</attributes>
	</classpathentry>
	<classpathentry kind="src" output="bin/test" path="src/test/java">
		<attributes>
			<attribute name="gradle_scope" value="test"/>
			<attribute name="gradle_used_by_scope" value="test"/>
		</attributes>
	</classpathentry>
	<classpathentry kind="src" output="bin/test" path="src/test/resources">
		<attributes>
			<attribute name="gradle_scope" value="test"/>
			<attribute name="gradle_used_by_scope" value="test"/>
		</attributes>
	</classpathentry>
	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.8/"/>
	<classpathentry kind="con" path="org.eclipse.jst.j2ee.internal.web.container"/>
	<classpathentry kind="con" path="org.eclipse.buildship.core.gradleclasspathcontainer">
		<attributes>
			<attribute name="org.eclipse.jst.component.dependency" value="/WEB-INF/lib"/>
		</attributes>
	</classpathentry>
	<classpathentry kind="output" path="bin/default"/>
</classpath>

And my generated loose application looks like this:
<?xml version="1.0" encoding="UTF-8"?>
<archive>
    <dir sourceOnDisk="/Users/aguibert/dev/git/basic-liberty/src/main/webapp" targetInArchive="/"/>
    <dir sourceOnDisk="/Users/aguibert/dev/git/basic-liberty/build/classes/java/main" targetInArchive="/WEB-INF/classes/"/>
    <file sourceOnDisk="/Users/aguibert/dev/git/basic-liberty/build/tmp/META-INF/MANIFEST.MF" targetInArchive="/META-INF/MANIFEST.MF"/>
</archive>

So, Eclipse is compiling my application classes into bin/main, but my loose application is pulling application classes from build/classes/java/main. For hot updates to work, these two locations need to be the same.
For now, I have to work around this issue by noticing that my app updates aren't taking effect, and then go hand-edit my .classpath. But any time I re-sync my gradle project (e.g. add/update a dependency) then the .classpath gets overwritten.

Interesting, I'll take a look at this. Is your project using the eclipse plugin in its build file? If so, we can just pull the Eclipse classpath from there.

no, my project is not using the eclipse plugin. I'm just using these:
apply plugin: 'war'
apply plugin: 'liberty'

I pull the project into Eclipse by doing Import -> Existing gradle project. I think the reason this broke is perhaps an older version of Gradle wrote class files into build/classes/java/main, but the newer versions of Gradle may write to bin/main.
The repo I am referring to is publicly available here: https://github.com/aguibert/basic-liberty

@aguibert I added some classpath configuration for the Eclipse plugin inside of our plugin. It ends up setting the Eclipse output directory to build/classes/java/main instead of bin/main. From what I've read this isn't recommended, but it works. Would configuring this in the loose app file be better or do you think changing the .classpath file is a valid solution?


mmt evaluate occasionally gets stuck (in getting MateCat scores?)

Occasionally, mmt evaluate gets stuck. The engine has translated all 1000 test sentences as evidenced by node.log. After tens of minutes, I lose patience and kill it. But I think it can hang for longer.
This is a bummer for running batteries of automatic tests.
I suspect the MateCat scoring: there should be a reasonable timeout on this (is 1 minute OK?)

yes, absolutely!

I also have the same problem.
I cannot reproduce it, but if I increase stack e cube pruning from 1000 to 5000 it happens very often. So I assumed it was a translation request stalled and not the Matecat API.
Anyway I totally agree in setting a timeout. But 1 minutes for all request may be too small.
@luca-mastrostefano can you suggest a good timeout?


Unclosed

When switching between two linkding instances where I exported all bookmarks and imported them in the second instance I noticed the following issue:

When there is a


[react-native-payments] Google Pay with Checkout.com error "token_data_invalid"

@vitalyiegorov Hi again!
Google Pay doesn't work with Checkout.com when signedMessage is parsed to JS here. Checkout.com needs signedMessage to be a string, not an object, so when I stringify the signedMessage field from Android token, it still doesn't recognize the token. When I remove the JSON.parse(rawToken.signedMessage) as AndroidSignedMessage, and then don't stringify it, everything works correctly. I'm guessing that checkout.com is expecting an exactly raw string, not a parsed, then stringified string.
Is there a reason why it is parsed here?

Could you please show two outputs to compare on the character basis the difference? I mean original and after json.stringify

@vitalyiegorov
These are examples from GPay in test environment, so tokens won't work :)
Raw singedMessage field from GPay Android SDK without parsing or stringifying:
{
  "encryptedMessage": "LaU8+f5ka+IYrkagK4K47OcrWu4EBfy77vGFHBy0gk0urMjWA03jHVF/sB5l22wm0FYbar/7wsJp8B0k3ZYo6tBhZrMPaL9H2msR+TONEZdnXx1udggsbLMUSD1JohCakSyQCdjEMygQv0PnD2sp8oGCjC2aHTSbpnP4GarwZOXz08ExuVBbU0753ncp376Nve+zBiCJGDJoXVQr4Qhw9Qf0JmlmTae9Kv1cUG7ah9qBsapgs33MYtdUgVGEoisgcHhDiEEOiDS59Hk9L76quWLiB8KY3Qq7wvfdKbcFF0pXfB/H5ygg1ncnhNsnOatIe5Iu3mpuRPwk0vcfWQWF6/DAo6Zw+och8CRQGBOnwI+G/Q8AY8P8u7JiQhaHXcy1+qolo3IY51nCzciYTt1OKx/k4udQ708mGfXAYIuOt3Y03pw55MTbrq8ZcpsU8z9OccJGhStH039IUlodH5WCUjqBfPTfsgNlEQ\u003d\u003d",
  "ephemeralPublicKey": "BOfl+EvwJSWg6mdFTqZyA0L3VcF03py9nYqIZOqcg1wkgjNW/j0r74pVVMDD8pSOI7IOGA9q/Yz7ZPBZfNjG4OU\u003d",
  "tag": "RDGYe24aMZ/xSPFGXF3ZVRPkF+3fhn/EiuLYeF3VxdA\u003d"
}

After react-native-payments JSON.parse()s it, the JSON becomes:
{
    "encryptedMessage": "LaU8+f5ka+IYrkagK4K47OcrWu4EBfy77vGFHBy0gk0urMjWA03jHVF/sB5l22wm0FYbar/7wsJp8B0k3ZYo6tBhZrMPaL9H2msR+TONEZdnXx1udggsbLMUSD1JohCakSyQCdjEMygQv0PnD2sp8oGCjC2aHTSbpnP4GarwZOXz08ExuVBbU0753ncp376Nve+zBiCJGDJoXVQr4Qhw9Qf0JmlmTae9Kv1cUG7ah9qBsapgs33MYtdUgVGEoisgcHhDiEEOiDS59Hk9L76quWLiB8KY3Qq7wvfdKbcFF0pXfB/H5ygg1ncnhNsnOatIe5Iu3mpuRPwk0vcfWQWF6/DAo6Zw+och8CRQGBOnwI+G/Q8AY8P8u7JiQhaHXcy1+qolo3IY51nCzciYTt1OKx/k4udQ708mGfXAYIuOt3Y03pw55MTbrq8ZcpsU8z9OccJGhStH039IUlodH5WCUjqBfPTfsgNlEQ==",
    "ephemeralPublicKey": "BOfl+EvwJSWg6mdFTqZyA0L3VcF03py9nYqIZOqcg1wkgjNW/j0r74pVVMDD8pSOI7IOGA9q/Yz7ZPBZfNjG4OU=",
    "tag": "RDGYe24aMZ/xSPFGXF3ZVRPkF+3fhn/EiuLYeF3VxdA="
  }

The main difference that I see is that GPay returns "equals" symbol (=) as unicode char ("\u003d"). Then JSON.parse() parses it into an actual equals sign (=). And then when stringifying it again, it's still left as an equals sign (=).
So in the end, the token returned from GPay SDK and the one sent to server differ and that's probably why it fails.
So I think that adding rawSignedMessage field would work, but maybe rawToken would work even better. It would hold the whole non-parsed GPay token, not only signedMessage field.

The same thing is happening to me.
"signature" prop gets a = signal attached at the end.
Any way to solve it?
Thanks
Valid token
{"signature":"MEYCIQCuRhDFy5mMzlPYFT0rDPe4LhYcNEqizOPLivQufxiNKgIhALJzxbdxPyz6krRxAjXPBc3U3Hl6YuZWDvZhMM9XqJdt","protocolVersion":"ECv1","signedMessage":"{\"encryptedMessage\":\"TlT6ZECX3LA/bMiHYQN/dWsSHfilUUkNeKnWZHfY47XkXIEQoPqhjg0Ts8E6SJk1c8NK1ROBWUzKBA7cMbpZTXo+rAt4h0YXolrLUE5LfcVnUQRKjBJ3KLcZjhB8P83dRZjnhHTG7Eo+YIHmPZPSi4YwHCP5z9eq4EkPPEyocKg+WdSSEox6gEwRUuYO78ehMwoHQqSNiH4YCV3dxgKEetfMrJqF3nlRSwX/llQ3q2/UZOiEX9KNrsu5pd3CXnMBX6Vmt7wEby+o9a+XtwuHRAxzP6M1jl1L9M+RSeAnbe/yKnzRH8/tbgTO9fekBWler/44UY5Q127L+r/1lvDUtuuATUZWQPFXUWaN46cZQoTy8XQPBbciwu0gncfIbjDXh++TMoKJNEYr43MSOl19W5WtYtzf4+xGJjyIPKklgciHSetlRwiwrbXbdVkDVgodcDtbJP/AbkE2DJ9i2dBiohmQSzi8E0AV+wbSHiET0N8Wu0y/JX1L5uSLVF+OyGU1Wzv40P7WrXjcSvErLsX9X7Nm0bWx3MgRdeVu03CVCKU8QHFNNWzT\",\"ephemeralPublicKey\":\"BC52d6btsSQpB6knNnVIpUl/bc24uJbpUb08oA2nMCPXYETpkzUQIjF145+lfGxII9m5Jql7t7vD16Nl7Bs0OdA\\u003d\",\"tag\":\"yUCeJAz1prlWdM3OuG0bgEkrwp2d2lgHg/aozvi+pq4\\u003d\"}"}
Failed token
{"protocolVersion": "ECv1", "signature": "MEUCIF0k6iwA/kGbYVa7cndAx63tmvYs6rACvbXWFHAAcuYKAiEAsmUW4jtl/vIhha3IE2tP6tSWQtWO8HfWrjEJRVfbUmQ=", "signedMessage": "{\"encryptedMessage\":\"QN7kDIvuKHd8/WOg9Aqm+FJJEJ2iVkg/ZEgs75POeNskmG5TdUFzHgNQIZj6CODzMOZlyI5fjStJyVu+fq1Z+a2edAMLOoTsASQp9UdXR2IfhZbxNPCUIyxpz7ZJ6qLkWmhIxw8BmJxaEVWeadv2htdco+Nx7mtyQ3CCZtt8In5YnsdjFYeThIGua9b6daatOURnVvUJLhnG+XNMcPXQ2koyy7AGOjvk3aFHeWzP17vv0W94BnTNjQ1XAuNL5SwV3FXCI4wfwAI5AATVsLDlOPtje7Q4I4eRyfhKZkzU7OgUbcs4jHqZt+2oo8TOIwBBaZSjqk/7NbI9lGhWIbOaOQmfqUs6Zd5k/qQxAKuLBUrabTUGdEXORY3jffsMtTvrDCIa5Ck6xS97OLKdJUSEDEpN+jGnZMDpDkLApTQTjBm4QA4lu1dNqdAifwXynJDyuVHgjk9ZBTpXqQp3A3R0juTOXU4dJY3WfDZvoEwfVW/XyxqphdY+moRTbFA+Hrm8TpCv5nNm9rq2fPblMv/y6Uiu08R8QntcQ9UdxCjsGJcVyUIpDTWWNs0kfJwX49KPH7CXJgP+wRQZUuSl\",\"ephemeralPublicKey\":\"BIZaFqOoXRVHpoxe0RYCQZJgBlVILsqjfHLDYkb/zNUSTR2Q7dRW3q9R/zcTSrJiMV3MclCFY7RvHO+0d93hms4\\u003d\",\"tag\":\"XJ0qSSXY/m1J04IIhumGakUdt4IdkBbF9g9wOnY1ze0\\u003d\"}"}

thanks for the information, #149 will fix it, please use new rawToken field for the android.
Waiting for your feedback! Btw are you using this lib in production? Could you please describe your use case and what payment gate are you using, we plan to add supported payment gates to the docs for other users.

Hi @vitalyiegorov I'm using WorldPay as a payment gate. I tried the fix on a local branch and doesn't seem to be fixed. Any chance I might be doing something wrong by adding my own merchant?

Have you tried using new rawToken field for sending to the WorldPay? Can
you provide a docs link so I can help?
пт, 13 жовт. 2023 р. о 18:19 Filippo Barcellos @.***>
пише:

Hi @vitalyiegorov https://github.com/vitalyiegorov I'm using WorldPay
as a payment gate. I tried the fix on a local branch and doesn't seem to be
fixed. Any chance I might be doing something wrong by adding my own
merchant?
—
Reply to this email directly, view it on GitHub
https://github.com/rnw-community/rnw-community/issues/147#issuecomment-1761693689,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEPGPXCJJ5CTM7YKLSPOM3X7FLZNAVCNFSM6AAAAAA5YLUN32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRRGY4TGNRYHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@filippobarcellos As this issue is closed, could you please create a new one with the proper description and title


heartbeat topic is in publish event

Caught '$SYS/' + that.id + '/heartbeat' topic in publish event, is it a design?
the packet is as follows when client is null:
 message from client  null Packet {
  cmd: 'publish',
  brokerId: 'tHMNcxXrm',
  brokerCounter: 1,
  topic: '$SYS/tHMNcxXrm/heartbeat',
  payload: <Buffer 74 48 4d 4e 63 78 58 72 6d>,
  qos: 0,
  retain: false,
  messageId: 0 }

Seems related to https://github.com/mcollina/aedes/issues/7 and https://github.com/mcollina/aedes/commit/9bca79c9572774f332435fbc391d7932f39b2ae7, but they do not mention what the packet is when client is null on publish event.

This is by design. It's a published packet, so it ends up in the publish event. client is null, because there is no client. It would be fantastic if you could update the docs.

Update README.me in PR https://github.com/mcollina/aedes/pull/245


HUE-7712: [spark] Livy-batch not available in HUE

https://review.cloudera.org/r/13870/
https://issues.cloudera.org/browse/HUE-7712

Review please

LGTM
We introduced extra states ('starting', and 'success') before and it complexifies a bit but above looks good.


Compare web resources content before update

much faster deployment, if only few scripts changed, otherwise it always deploys everything and for big solutions it takes a lot of time

Thanks for this - awesome feature. Can you rebase onto the spkl_dev branch so it can be included in the next beta.


stream_typing_indicator: Remove the dev environment guard in settings.

CZO Discussion
This PR removes the page_params.development_environment guard for the setting send_stream_typing_notifications and makes it available outside the development environment.

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Question: Is it intentional that we don't show these two settings (DM and stream typing notifications) in SETTINGS / DEFAULT USER SETTINGS

Added a commit to make the wording more parallel to the direct messages version, and marked to merge once CI passes.
And yes we should add to default user settings as discussed in #design, thanks for noticing that!


chore: Make JIT into a feature that can be enabled or disabled

This way we can upgrade CKB with latest CKB-VM containing bug fixes, but don't need to worry that JIT related code could bring potential problems for now.

The additional CI items shown here seem to be a problem with Travis itself, the actual CI tests have passed.


[POC] Secret Providers

[NOTE: for discussion only, not meant to be merged]
This PR introduces the concept of "secret providers".
Example:
{
  core {
    image(ref: "alpine") {
      exec(input: {
        secretEnv: {name:"SECRET", id:"env://MY_SECRET"},
        args: ["printenv", "SECRET"]}
      ) {
        stdout
      }
    }
  }
}

In this basic example, env://MY_VARIABLE can be passed in place of any SecretID.
How does it work?

Secrets are retrieved just in time, when buildkit requests them (through the secrets gRPC "callback")

Cloak's secret provider turns around and issues a query against the API to resolve the secret, like so:

query {
  core {
    secrets {
      # in our example, this is going to be `env(name: "MY_SECRET")`
      plaintext: $PROTOCOL(name: $name)
    }
  }
}

e.g. For a given <protocol>://<key> SecretID, the query becomes core { secrets { <protocol>(name: <key>) }
Extensions
Let's imagine a hypothetical vault extension.
All the extension would have to do is extend Secrets with the new provider, like this:
extend type Secrets {
  vault(name: String!): String!
}

Then, automatically, vault://my_token will be a valid SecretID out of the box.
Cloak has a few built-in providers (env, file, ...) and extensions can add more on top.
Caveats

Caching needs to be disabled, otherwise all those secrets end up on disk

I have a potential solution in mind

The "interface" (extend type Secrets) is quick and dirty

We could do better, especially since cloak is "privileged" (e.g. don't need for the provider to be reachable through top-level Query

For instance, this might work:
# Provided by Core
interface SecretsProvider {
  protocol: String!
  plaintext(key: String!): String!
}

# An extension could look like this:
type VaultAlthoughTheTypeNameDoesntMatter implements SecretsProvider {
  protocol: String!
  plaintext(key: String!): String!
}

cloak can then scan the schema for types that implement SecretsProvider, query them for their protocol, and just-in-time resolve plaintext when a secret is needed.

Moved to https://github.com/dagger/dagger/pull/3124


Improve frame filtering

Improve frame filtering
Description
Expand the checks in eConsiderFrameForProcessing to drop invalid frames earlier
Test Steps

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run formatting


Add empty function

Like the project.
While this is my small contribution. To check the string length standard means you can not use the expression ${#myVar}.

The opposite is also possible to add a function. Or operator "!".

Doesn't it work to simply say if (str == "") { ... }?

@tdenniston Yes, perhaps, is the best option.

@tdenniston However, this function can be changed to obtain the length of the string, which is very convenient in different cases.

@tdenniston You can close this PR.


laser publisher still sending message if the sensor was disabled.

if the laser sensor was not launched, this topic publisher shouldn't send any topic, but now i got empty message. This problem caused error when i trying to slam by RGBD camera. thanks

@ahuizxc
Sorry for the late response. To give us a clearer idea of the issue you have, you might want to fill in the below form that we ask for as a default.
Regards,
Ryan

ISSUE TEMPLATE ver. 0.2.0

Which TurtleBot3 you have?

[ ] Burger
[ ] Waffle
[ ] Waffle Pi

Which SBC(Single Board Computer) is working on TurtleBot3?

[ ] Raspberry Pi 3
[ ] Intel Joule 570x
[ ] etc (PLEASE, WRITE DOWN YOUR SBC HERE)

Which OS you installed in SBC?

[ ] Ubuntu MATE 16.04 or later
[ ] Raspbian
[ ] etc (PLEASE, WRITE DOWN YOUR OS)

Which OS you installed in Remote PC?

 The laser publisher keep sending even if the sensor is not launched.

@ahuizxc
We only support the basic tutorials introduced on our e-manual to help users with the initial setup and basic functions we provide. If the question is about your own program, we cannot give any help. If you encountered the issue while following our tutorial, please fill in the form and let us figure out the reason you are having that issue.
Kind regards,
Ryan

This issue will be closed since there were no actions for a while. You can reopen this issue to show this issue to the users whenever. Thanks.


Allow configuration of confirmation prompt comparison via StringComparer

Based on suggestion from @0xced https://github.com/spectreconsole/spectre.console/pull/1151#issuecomment-1426768481 👍

@patriksvensson Sorry for the delay, made adjustments 👍

@MartinZikmund No worries. No SLA for open source contributions 😉

Merged! Thank you for your contribution. Much appreciated! 👍


WordPress.Files.FileName.InvalidClassFileName when theme functions contains a class

Hey there,
I'm getting a WordPress.Files.FileName.InvalidClassFileName error when my theme's functions.php contains a class.

Yes, because each PHP class should be contained within a separate file by itself.


fix(docker): missing dependency

Add missing libgmp3-dev when building on OSX
When building on OSX with docker build . -t ludwig i had this error. this fix adds the necessary libgmp3-dev to the docker container.
  Running setup.py install for gmpy: started
    Running setup.py install for gmpy: finished with status 'error'
    ERROR: Complete output from command /usr/bin/python3 -u -c 'import setuptools, tokenize;__file__='"'"'/tmp/pip-install-vlfrjt08/gmpy/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-8lu49ntg/install-record.txt --single-version-externally-managed --compile:
    ERROR: running install
    running build
    running build_ext
    building 'gmpy' extension
    creating build
    creating build/temp.linux-x86_64-3.6
    creating build/temp.linux-x86_64-3.6/src
    x86_64-linux-gnu-gcc -pthread -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -I./src -I/usr/include/python3.6m -c src/gmpy.c -o build/temp.linux-x86_64-3.6/src/gmpy.o
    In file included from src/gmpy.c:251:0:
    src/gmpy.h:30:10: fatal error: gmp.h: No such file or directory
     #include "gmp.h"
              ^~~~~~~
    compilation terminated.
    error: command 'x86_64-linux-gnu-gcc' failed with exit status 1
    ----------------------------------------
ERROR: Command "/usr/bin/python3 -u -c 'import setuptools, tokenize;__file__='"'"'/tmp/pip-install-vlfrjt08/gmpy/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-8lu49ntg/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-install-vlfrjt08/gmpy/
The command '/bin/sh -c git clone --depth=1 https://github.com/uber/ludwig.git     && cd ludwig/     && pip install -r requirements.txt     && python -m spacy download en     && python setup.py install' returned a non-zero code: 1

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


Do not get location if there is not source to get it from

Solving a NPE setting the geoLocationRequest into the CasEvent when there is no HttpServletRequest.
Just adding some null checks.

Is this something you may want to post to 5.1.x as well?

Would be nice to have this ASAP


Allow font size changes in Debug Terminal

Allow user to change the font size for the Debug Terminal.

Preference setting
Focused Debug Terminal and Ctrl+MouseWheel up/down (similar to edit control font size changes).

This will naturally work for newly arriving text; however, updating the existing text to the new font size may take more work depending on how that data is rendered in the Debug Terminal.


Fix hardcoded commit message: 'PAC' => 'crate'

Addresses https://github.com/atsamd-rs/atsamd/pull/340#issuecomment-733929859

the file should be renamed too.

Ooof XD


Allow disabling shadow in all components

Currently the 'shadow' prop had to be set to 'false' in order to turn off the shadow in individual component. Find a way to opt out of using shadow in all components.

hi! can i work on this?

@gabecar this is no longer needed but I forgot to close the issue. sorry for that!


KAFKA-5965: Remove Deprecated AdminClient from Streams Resetter Tool

Removed usage of deprecated AdminClient from StreamsResetter
No additional tests are required.
Committer Checklist (excluded from commit message)

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

\cc @bbejeck @vvcephei
@fedosov-alexander Thanks for the PR -- test failure seems to be related. We can review after those a fixed. Let us know if  you need any input.
Btw: you can trigger a new build, by putting a comment "Retest this please" on Github. Not need to do a "dummy commit"

Dear @guozhangwang @bbejeck @vvcephei thank you for your recommendations.
I've made requested changes, please,review.

Thanks @fedosov-alexander ! merged to trunk.


Leetcode Question Number 342

Leetcode Question Number 342 soultion needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

I want to contribute. Please assign me.


Rectified bugs

The two extra commits here cause the 2.x tests to fail in addition to the 3.x. Gonna close this for now. Happy to reopen if you want to revisit and get tests passing.


Usage of PagingMenuController inside of UINavigationController with translucency

I want to use PagingMenuController, but I have one issue. I add PagingMenuController inside of UINavigationController with true translucency value. Inside of PagingMenuController I use UIScrollView with vertical insets. That way my menu is hidden behind NavigationBar. How can I set some kind of vertical inset for MenuView only?

Could you show me your codes if possible?

@kitasuke of course!
//MARK: Initializing - viewDidLoad
pagingMenu_ = PagingMenuController(viewControllers: [UIViewController()])
addChildViewController(pagingMenu_)
view.addSubview(pagingMenu_.view)
pagingMenu_.didMoveToParentViewController(self)

I'm using Cartography extension as AutoLayout assistant
constrain(pagingMenu_.view) { view in
let superview = view.superview!
    view.leading == superview.leading
    view.bottom  == superview.bottom
    view.trailing == superview.trailing
}

But I as know I can't use topLayoutGuide with Cartography explicitly. That's why I set the constraint manually
topLayoutGuide.bottomAnchor.constraintEqualToAnchor(
	pagingMenu_.topLayoutGuide.topAnchor).active = true

Everything looks fine, MenuView locates below NavigationBar. But content of my inner controller is clipping on top during vertical scrolling. Maybe I do it wrong, I'm developing on iOS not so long.
I would be really appreciate for your help.

This could be workaround for that case.
let pagingMenuController = PagingMenuController(viewControllers: viewControllers, options: options)
pagingMenuController.view.frame.origin.y += 64
pagingMenuController.view.frame.size.height -= 64

Could you try this one?

@kitasuke Your workaround sometimes may not work perfectly. Have you found the real solution for this? THANK YOU.


Fix for LinkWithMesh causes some flickering on button creation

Since currentMeasure isn't calculated until the render and the update is happening before the render I call the computeLayout which will calculate the currentMeasures.
https://github.com/BabylonJS/Babylon.js/issues/11014

The problem is these two lines:
    var newLeft = ((projectedPosition.x + this._linkOffsetX.getValue(this._host)) - this._currentMeasure.width / 2);
    var newTop = ((projectedPosition.y + this._linkOffsetY.getValue(this._host)) - this._currentMeasure.height / 2);

_currentMeasure.width/height are not calculated until the first render frame. But the function that calls this is called before this. So the values are 0;

Another solution I thought about was just using w/h flat out without the _currentMeasure. But that might have other consequences.

Yeah this is a very slow solution. We cannot render a text per update per mesh to get a measure

Yeah this is a very slow solution. We cannot render a text per update per mesh to get a measure

What do you think of the second solution I wrote above?


DOC: Code/twitter-the-algorithm-analysis-deeplake not working as written

Issue with current documentation:
I followed the documentation @ https://python.langchain.com/docs/use_cases/code/twitter-the-algorithm-analysis-deeplake.
I replaced 'twitter-the-algorithm' with another code base I'm analyzing and used my own credentials from OpenAI and Deep Lake.
When I run the code (on VS Code for Mac with M1 chip), I get the following error:
_ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (1435,) + inhomogeneous part.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/catherineswope/Desktop/LangChain/fromLangChain.py", line 37, in
db.add_documents(texts)
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/langchain/vectorstores/base.py", line 91, in add_documents
return self.add_texts(texts, metadatas, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/langchain/vectorstores/deeplake.py", line 184, in add_texts
return self.vectorstore.add(
^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/deeplake/core/vectorstore/deeplake_vectorstore.py", line 271, in add
dataset_utils.extend_or_ingest_dataset(
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/deeplake/core/vectorstore/vector_search/dataset/dataset.py", line 409, in extend_or_ingest_dataset
raise IncorrectEmbeddingShapeError()
deeplake.util.exceptions.IncorrectEmbeddingShapeError: The embedding function returned embeddings of different shapes. Please either use different embedding function or exclude invalid files that are not supported by the embedding function._
This is the code snippet from my actual code:
import os
import getpass
from langchain.embeddings.openai import OpenAIEmbeddings
from langchain.vectorstores import DeepLake
from langchain.document_loaders import TextLoader
#get OPENAI API KEY and ACTIVELOOP_TOKEN
os.environ["OPENAI_API_KEY"] = getpass.getpass("OpenAI API Key:")
os.environ["ACTIVELOOP_TOKEN"] = getpass.getpass("Activeloop Token:")
embeddings = OpenAIEmbeddings(disallowed_special=())
#clone from chattydocs git hub repo removedcomments branch and copy/paste path
root_dir = "/Users/catherineswope/chattydocs/incubator-baremaps-0.7.1-removedcomments"
docs = []
for dirpath, dirnames, filenames in os.walk(root_dir):
for file in filenames:
try:
loader = TextLoader(os.path.join(dirpath, file), encoding="utf-8")
docs.extend(loader.load_and_split())
except Exception as e:
pass
from langchain.text_splitter import CharacterTextSplitter
text_splitter = CharacterTextSplitter(chunk_size=1000, chunk_overlap=0)
texts = text_splitter.split_documents(docs)
username = "caswvu"  # replace with your username from app.activeloop.ai
db = DeepLake(
dataset_path=f"hub://caswvu/baremaps",
embedding_function=embeddings,
)
db.add_documents(texts)
db = DeepLake(
dataset_path="hub://caswvu/baremaps",
read_only=True,
embedding_function=embeddings,
)
retriever = db.as_retriever()
retriever.search_kwargs["distance_metric"] = "cos"
retriever.search_kwargs["fetch_k"] = 100
retriever.search_kwargs["maximal_marginal_relevance"] = True
retriever.search_kwargs["k"] = 10
from langchain.chat_models import ChatOpenAI
from langchain.chains import ConversationalRetrievalChain
model = ChatOpenAI(model_name="gpt-3.5-turbo")  # switch to 'gpt-4'
qa = ConversationalRetrievalChain.from_llm(model, retriever=retriever)
questions = [
"What does this code do?",
]
chat_history = []
for question in questions:
result = qa({"question": question, "chat_history": chat_history})
chat_history.append((question, result["answer"]))
print(f"-> Question: {question} \n")
print(f"Answer: {result['answer']} \n")
Idea or request for content:
Can you please help me understand how to fix the code to address the error message?  Also, if applicable, address in the documentation so that others can avoid as well.  Thank you!

@casWVU What version of LangChain are you on?

Hi.  I'm on
Name: langchain
Version: 0.0.223
Thanks!
On Sun, Jul 9, 2023 at 12:04 PM Devin Stein @.***>
wrote:

@casWVU https://github.com/casWVU What version of LangChain are you on?
—
Reply to this email directly, view it on GitHub
https://github.com/hwchase17/langchain/issues/7435#issuecomment-1627759343,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3YSMTRYHKLMXD22OLAKVPTXPLJAXANCNFSM6AAAAAA2DSOAYI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @casWVU! what DeepLake version are you using? This problem is related to the documents stored in the folder. Could you pls filter the files that you don't use. When files of unsupported format comes inside of the OpenAI embedding it sends back an empty list. Appending this empty list is causing the issue. In the newer version of the deeplake, the exception should provide you more details, but overall that's the issue.

Hi!  I'm on the latest version:
Name: deeplake
Version: 3.6.8
Thanks so much for the insight.  Do you know which file types aren't
supported by OpenAI embeddings?  I'm reading OpenAI documentation and
searching the web but not finding anything.
On Tue, Jul 11, 2023 at 4:07 AM Adilkhan Sarsen @.***>
wrote:

Hey @casWVU https://github.com/casWVU! what DeepLake version are you
using? This problem is related to the documents stored in the folder. Could
you pls filter the files that you don't use. When files of unsupported
format comes inside of the OpenAI embedding it sends back an empty list.
Appending this empty list is causing the issue. In the newer version of the
deeplake, the exception should provide you more details, but overall that's
the issue.
—
Reply to this email directly, view it on GitHub
https://github.com/hwchase17/langchain/issues/7435#issuecomment-1630351570,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3YSMTVLBRIMQBKFXADETWTXPUCVBANCNFSM6AAAAAA2DSOAYI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Not sure which files are not supported, but I had faced similar issues before. I would suggest excluding files that doesn't bring any context info to the model, let's say like: .lock or .DS_Store

But I saw that LangChain has updated the open ai related code, so now it should raise this kind of exception:
    raise openai.error.APIError("OpenAI API returned an empty embedding")
openai.error.APIError: OpenAI API returned an empty embedding

Just update langchain in you repo till the latest version

Thanks, I'll check it out.
On Wed, Jul 12, 2023 at 3:20 AM Adilkhan Sarsen @.***>
wrote:

But I saw that LangChain has updated the open ai related code, so now it
should raise this kind of exception:
raise openai.error.APIError("OpenAI API returned an empty embedding")

openai.error.APIError: OpenAI API returned an empty embedding
Just update langchain in you repo till the latest version
—
Reply to this email directly, view it on GitHub
https://github.com/hwchase17/langchain/issues/7435#issuecomment-1631982346,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3YSMTXSP7OKVDH4OLSTADLXPZF5XANCNFSM6AAAAAA2DSOAYI
.
You are receiving this because you were mentioned.Message ID:
@.***>


run doc error

after run npm install, run gulp and the error logs are as follows:
[21:13:09] Starting 'setWatch'...
[21:13:09] Finished 'setWatch' after 47 μs
[21:13:09] Starting 'jshint'...
[21:13:09] Starting 'markup'...
[21:13:09] Starting 'fontIcons'...
[21:13:09] Starting 'css'...
[21:13:09] 'jshint' errored after 26 ms
[21:13:09] Error in plugin 'gulp-shell'
Message:
Command ../node_modules/.bin/jsxhint --harmony "../src/**" "./src/app/**" - -exclude ../src/utils/modernizr.custom.js failed with exit code 1
[21:13:09] Finished 'css' after 27 ms
[21:13:09] Finished 'fontIcons' after 42 ms
[21:13:09] Finished 'markup' after 51 ms

are you on windows? if so, see #845 for a workaround

Getting the same error on Mac. Possible because of the recent merged PR.
https://github.com/callemall/material-ui/commit/5ab2e6ec5f322e4004bc1a47b35a425c5c83de17
./src/app/components/pages/components/icons.jsx: line 115, col 11
Empty array elements require elision=true.

@mairh Sorry that was my bad - fixed with 8158e253aa38f2016c15bb45d4a10510f6bd1875

Thanks @chrismcv @mairh


webpack config for Jetbrains Webstorm

Webstrom has an option to set path to webpack config. But when I set it to nuxt's webpack config it throws error:

Can't analyse base.config.js: coding assistance will ignore module resolution rules in this file.
Possible reasons: this file is not a valid webpack configuration file or its format is not currently supported by the IDE

I tried to specify webpack/base.config.js, webpack/client.config.js,  webpack/server.config.js with no luck.
Maybe you extract parts of webpack config to separate file which Webstorm can analyze?
I want it to teach webstorm to understand ~/ aliases

I set webpack.config.js on the project root for WebStorm.
That file is invalid for Nuxt and it is valid only for WebStorm.
module.exports = {
  resolve: {
    // for WebStorm
    alias: {
      '@': path.resolve(__dirname),
      '~': path.resolve(__dirname)
    }
  }
};

Does this work for .vue files?
JetBrains say it won't:

https://intellij-support.jetbrains.com/hc/en-us/community/posts/360000927179-Is-there-a-canonical-way-to-set-up-prefixed-aliases-?page=1#community_comment_360000149560


XS✔ ◾ Updated logging docs

❌ Add a description.

@hussnain612 Image is available for testing. docker pull stakater/workshop-exercise:SNAPSHOT-PR-113-1dd47ccb


Nightly: IndexOutOfBoundsException when inserting a single record using the bulk interface

CrateDB version
nightly, 5.10.0-SNAPSHOT, build 08d84f8, date 2024-10-29
CrateDB setup information
Docker standard.
docker run --rm -it --publish=4200:4200 crate/crate:nightly -Cdiscovery.type=single-node

Problem description
When using the bulk interface to insert a single record, CrateDB nightly started failing on a downstream CI job 5 days ago. Maybe c7fb24dd is related?

CrateDB Version:    version[5.10.0-SNAPSHOT], pid[1], build[08d84f8/NA]
First Occurrence:   Tue, 29 Oct 2024 03:13:34 GMT

INSERT INTO t01 (name, value) VALUES (?, ?);

bulk_parameters=[('temperature', 42.42)]

When adding a second record, the procedure succeeds.
bulk_parameters=[('temperature', 42.42), ('humidity', 84.84)]

Steps to Reproduce
pip install crate sqlparse

"""
Error:              IndexOutOfBoundsException[Index: 1, Size: 1]
Problem:            ... when inserting a single record using the bulk interface.
CrateDB Version:    version[5.10.0-SNAPSHOT], pid[1], build[08d84f8/NA]
First Occurrence:   Tue, 29 Oct 2024 03:13:34 GMT

- https://github.com/crate/cratedb-toolkit/actions/runs/11566597884/job/32195528974#step:6:1362
- https://github.com/crate/cratedb-toolkit/actions/runs/11566597884/job/32195528974#step:14:16
"""
from pprint import pprint

import crate.client
import sqlparse

def insert_bulk():
    connection = crate.client.connect("http://localhost:4200/?schema=testdrive")

    # DDL.
    cursor = connection.cursor()
    for statement in sqlparse.parse("""
        DROP TABLE IF EXISTS t01;
        CREATE TABLE t01 (
            name STRING,
            value FLOAT
        );
        """):
        cursor.execute(str(statement))

    # DML.
    cursor.execute("""
    INSERT INTO t01 (name, value) VALUES (?, ?);
    """, bulk_parameters=[

        # Leave this alone to make it trip.
        ('temperature', 42.42),

        # Add this to make it work.
        # ('humidity', 84.84),
    ])

    cursor.close()
    connection.close()

if __name__ == "__main__":
    insert_bulk()

Actual Result
crate.client.exceptions.ProgrammingError: IndexOutOfBoundsException[Index: 1, Size: 1]

Expected Result
No error.
References

https://github.com/crate/cratedb-toolkit/actions/workflows/main.yml
https://github.com/crate/cratedb-toolkit/actions/runs/11566597884/job/32195528974#step:6:1362
https://github.com/crate/cratedb-toolkit/actions/runs/11566597884/job/32195528974#step:14:16

@amotl Thank you for reporting, it is indeed a regression introduced by https://github.com/crate/crate/commit/c7fb24dd00e0f14c8a2ab6f9c99ef5c663e85e8b. We are working on a fix.


plonk: benches: collaborative-proof: Add benchmark with stats collection

Purpose
This PR adds a benchmark for multiprover end-to-end latency and a baseline for the same circuit proved as a single-prover circuit.
Testing

All tests pass

lgtm, just curious if we're going to do the multiple mocked latencies as we did before? i.e. the 10ms, 100ms, 1000ms or whatever it was

We will do that in the renegade repo. Mocking latencies isn't of much use to us here since we want a compute bound profile to optimize it. Mocking with latencies is less about collecting a profile and more about load estimation


Dataset Splits

Great Work!
Can you release the splits used for the 6 folds for each dataset?
In figure 2 you mention that some were random and some were scaffold, but which was which was not discussed in either dataset section.  The splits would be especially helpful the Estrogen datasets as grabbing data hitting a protein from CHEMBL is a tricky process and hard to do exactly the same way twice.

PS For those looking for the MetStab Dataset I went through the original and cleaned it into a single csv here.
metstab.txt

Thank You!
We used random split for FreeSolv, ESOL, and Met- Stab. For all the other datasets we used scaffold split (this information is included in section 4.1 - paragraph "Evaluation").
We will release our datasets and splits this weekend.

Datasets and splits are in data folder.


Decouple mn payments from version.h, drop MIN_MNW_PEER_PROTO_VERSION

Changes:

decouple payments from from version.h, move MIN_MASTERNODE_PAYMENT_PROTO_VERSION_* to masternode-payments.h
drop MIN_MNW_PEER_PROTO_VERSION in fav of GetMinMasternodePaymentsProto(), imo  when spork10 is switched to "new only" state there is no reason to receive MNWINNER from outdated peers or to allow them to vote for new winner

EDIT: Any cases where having separate MIN_MNW_PEER_PROTO_VERSION could be helpful but I'm missing it?

So it looks like the substantive change here is to put the minimum version for receiving MN payments under control of SPORK_10.
The code looks good to me but I wonder if it wouldn't make more sense to leave all version constants in version.h ?  Why do you think it's a good idea to move these ?

re version.h: yeah, we had all versions there but imo it's not ideal solution at all. imo it's better to have module specific version right in this module's header and not somewhere outside. Also if you take a closer look at original (non-dash) variables, only PROTOCOL_VERSION and MIN_PEER_PROTO_VERSION are a subject to change sometimes and they are basically global for each and every module. Other *_VERSION variables are kind of static "points" in "version time" when some feature was implemented (or a bug happened).

Needs a rebase

Rebased but I still have some concern about this piece https://github.com/dashpay/dash/pull/960/files#diff-27f99aaf2c6f267cd36ff731a1409af3R559 actually. Probably should use MIN_MASTERNODE_PAYMENT_PROTO_VERSION_1 so that in the time of migration winners signed by non-upgraded masternodes for old blocks (or should I instead add some additional check for that?) still were legit and were able to pass IsValid()? New winners sent by non-upgraded masternodes should be rejected by ProcessMessage() anyway of course. Thoughts?

IsValid for old blocks will accept winners signed by old masternodes now.


Auto restart app consecutive ?

I create a App launcher and add CustomActivityOnCrash be like sample.
But when i get error, this app restart consecutive.
Not show the layout error .

Hi @AndroidLTG, I have actually used the library in a launcher application and it works as expected.
However, I imagine that your case is also one I have met before: if you do something that crashes in a background thread in your Application class, the app will enter a restart loop with the current version of the library (1.4.0). I am trying to fix this for 1.5.0, by setting a shared preference and defaulting to the default Android error screen if the last crash occurred less than 2 seconds ago. This should solve most  restart loops, but everything that is invoked in a background thread from the Application class is likely to cause restart loops in case of crashes...

Fixed in 1.5.0 with a shared preference. When a crash occurs, the current timestamp is saved into it. If a new crash occurs in less than 2 seconds, the library assumes that it's a restart loop and just defaults to the system exception handler ("App X has stopped.").
If you encounter this bug again and can provide a little sample to reproduce it, please let me know!

It save my day ! Thanks very much !


Laravel / Stripe - how to pass billing address to Stripe

billing address which we are collecting through booking form // stripe is asking us to pass that billing address to them //
how can I do it
I have a file strip gateway.php

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Error forwarding 1 message 2 times

Everything is fine except for one error: forward 1 message 2 times.
My source is a public channel (join only, no admin rights)
The destination is the group that I am an admin for
When the source has a message, it will be forwarded twice at the same time
I don't know how to fix it, please help, thanks!
https://i.imgur.com/E0qVNaB.jpg

Run tgcf live --loud and provide the log when the msgs are send twice.

Hi!
Yellow is a forwarded message from my channel to my group.
The other two messages are from another private channel (not me admin) to my group. It is duplicated.
https://imgur.com/undefined


Find out how to use pixel data as a texture

[ ] find out how to render to texture
[x] contact Jonas Haller regarding writing pixels to texture<EMAIL_ADDRESS>

Copy texture from main memory and import into existing texture with function Blt (of texture class)
Create Writable Texture and set as RenderTarget with RenderContext

To determine which objects should be rendered and which should not: RenderLayer component in objects

About 30 different layers

The RenderLayer property of the camera component can then be used to determine for each camera which of the layer(s) it should render.


Unable to install swift

Describe the bug
Recent support for swift is not working, when we try to install swift we getting error
~/.local/share/mise/installs/swift/.tmpfVQQU9/pkg/swift-5.8-RELEASE-osx-package.pkg/Payload -> ~/.local/share/mise/installs/swift/5.8

Mise expects that pkgutil --expand-full /Users/myuser/.local/share/mise/downloads/swift/5.8/swift-5.8-RELEASE-osx.pkg /Users/myuser/.local/share/mise/installs/swift/.tmpfVQQU9  creates a pkg folder but in reality unpacked swift is in ~/.local/share/mise/installs/swift/.tmpfVQQU9/swift-5.8-RELEASE-osx-package.pkg/Payload
To Reproduce

add .swift-version with value 5.8  (tested also with 5.10 and 6.0)
mise install
mise fails to install swift

Expected behavior
Mise is able to install swift.
mise doctor output
version: 2024.11.33 macos-arm64 (2024-11-28)
activated: yes
shims_on_path: no

build_info:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS
  Built: Thu, 28 Nov 2024 23:11:08 +0000
  Rust Version: rustc 1.82.0 (f6e511eec 2024-10-15) (Homebrew)
  Profile: release

shell:
  /bin/zsh
  zsh 5.9 (x86_64-apple-darwin23.0)

dirs:
  cache: ~/Library/Caches/mise
  config: ~/.config/mise
  data: ~/.local/share/mise
  shims: ~/.local/share/mise/shims
  state: ~/.local/state/mise

config_files:
  ~/.config/mise/config.toml

backends:
  aqua
  asdf
  cargo
  core
  gem
  go
  npm
  pipx
  spm
  ubi
  vfox

plugins:
  pnpm         https://github.com/jonathanmorley/asdf-pnpm.git#305baff
  sourcedocs   https://github.com/tuist/asdf-sourcedocs.git#ff81305
  tuist        https://github.com/cprecioso/asdf-tuist.git#3ecc489

toolset:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>      (missing)

env_vars:
  MISE_SHELL=zsh

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf_compat = false
  cache_prune_age = "30d"
  ci = false
  color = true
  debug = false
  default_config_filename = "mise.toml"
  default_tool_versions_filename = ".tool-versions"
  disable_backends = []
  disable_default_registry = false
  disable_hints = []
  disable_tools = []
  exec_auto_install = true
  experimental = true
  fetch_remote_versions_cache = "1h"
  fetch_remote_versions_timeout = "10s"
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  http_timeout = "30s"
  idiomatic_version_file = true
  idiomatic_version_file_disable_tools = []
  ignored_config_paths = []
  jobs = 4
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  libgit2 = true
  lockfile = true
  log_level = "info"
  not_found_auto_install = true
  override_config_filenames = []
  override_tool_versions_filename = []
  paranoid = false
  pin = false
  plugin_autoupdate_last_check_duration = "7d"
  quiet = false
  raw = false
  task_run_auto_install = true
  task_skip = []
  trace = false
  trusted_config_paths = []
  unix_default_file_shell_args = ["sh"]
  unix_default_inline_shell_args = [
      "sh",
      "-c",
      "-o",
      "errexit",
  ]
  use_file_shell_for_executable_tasks = false
  use_versions_host = true
  verbose = false
  windows_default_file_shell_args = [
      "cmd",
      "/c",
  ]
  windows_default_inline_shell_args = [
      "cmd",
      "/c",
  ]
  windows_executable_extensions = [
      "exe",
      "bat",
      "cmd",
      "com",
      "ps1",
      "vbs",
  ]
  yes = false

  [aqua]
  cosign = true
  slsa = true

  [cargo]
  binstall = true

  [node]

  [npm]
  bun = false

  [pipx]
  uvx = false

  [python]
  default_packages_file = "~/.default-python-packages"
  pyenv_repo = "https://github.com/pyenv/pyenv.git"
  uv_venv_auto = false
  venv_auto_create = false
  venv_stdlib = false

  [ruby]
  default_packages_file = "~/.default-gems"
  ruby_build_repo = "https://github.com/rbenv/ruby-build.git"
  ruby_install = false
  ruby_install_repo = "https://github.com/postmodern/ruby-install.git"

  [rust]

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false

  [swift]

1 problem found:

1. new mise version 2024.11.36 available, currently on 2024.11.33

Thx 🎖


fix description about kubectl apply

The related code is here:
https://github.com/kubernetes/kubernetes/blob/0207a090743398608b357264ab4c27aeea864abc/pkg/kubectl/cmd/apply.go#L503

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.10 Features: set Milestone to 1.10 and Base Branch to release-1.10
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

/cc @mikedanese

/approve (for writing)
Still need /lgtm for technical accuracy.

/assign @pwittrock  for lgtm

/assign

/lgtm


missing signature key

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'git repo'
Click on 'run 'docker-compose up''
Scroll down to '....'
See error
[+] Running 0/8
⠇ redis-cache Pulling                                                              5.9s
⠇ celery-beat Pulling                                                              5.9s
⠇ django Pulling                                                                   5.9s
⠇ celery-worker Pulling                                                            5.9s
⠇ db Pulling                                                                       5.9s
⠇ nginx Pulling                                                                    5.9s
⠇ maintenance Pulling                                                              5.9s
⠇ rabbitmq Pulling                                                                 5.9s
missing signature key

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Squest version or GIT commit number:
Ansible Tower/AWX version:

Hello,
We are not able to reproduce.
What is your Docker version? Seems that this kind of issue is related to too old docker version.

 You should upgrade.

Yes @Sispheor , it is working. Thank you.
BTW, I am in HPE and deployed in our cluster sucessfully.
http://<IP_ADDRESS>:8088/ui/

Nice! Don't hesitate to ping me on teams then !
Have fun with squest!

Sure. Thank you.


Upgrade to spring-data-elasticsearch v3.0.10

Looks like they fixed a number of dependencies on apache, jackson, and others and the primary reason is that I want to get access to DATAES-402.
See Compare v3.0.5.RELEASE to v3.0.10.RELEASE

Thanks for your help, it's released in 3.1.5.


terraspace: 2.2.8 -> 2.2.17

Description of changes

terraspace pkg is currently broken for aarch64-darwin on master(ca15aff5f428ae9d6cd76f741068864d9e962e47) and nixos-24.05(53e81e790209e41f0c1efa9ff26ff2fd7ab35e27). It hasnt been updated in about a year.
Bumping to the latest release fixes the pkg.
I also had to bump the package using the following method due to some existing issues with bundle lock including
architecture specific pre-built ruby packages (nokogiri specifically). Relates to https://github.com/nix-community/bundix/issues/88:
$ rm pkgs/applications/networking/cluster/terraspace/Gemfile.lock
$ export BUNDLE_FORCE_RUBY_PLATFORM="true"
$ nix-shell maintainers/scripts/update.nix --argstr package terraspace

Reproducing the build failure on aarch64-darwin:
nix-repl> :lf .
warning: Git tree '/Users/robert.hernandez/workspace/cloudinfra' is dirty
Added 20 variables.

nix-repl> :b inputs.nixpkgs.legacyPackages.aarch64-darwin.terraspace
warning: Nix search path entry '/nix/var/nix/profiles/per-user/root/channels' does not exist, ignoring
error: builder for '/nix/store/0jr1749dk5ddar9x2n69ysagdj068245-ruby3.1-nokogiri-1.15.3.drv' failed with exit code 1;
       last 10 log lines:
       > /nix/store/h7xrhgdyy68lxxgybxjcnjg71r2xwqv9-libxml2-2.12.7-dev/include/libxml2/libxml/xmlerror.h:898:29: note: passing argument to parameter 'handler' here
       >                                  xmlStructuredErrorFunc handler);
       >                                                         ^
       > 2 errors generated.
       > make: *** [Makefile:247: html4_document.o] Error 1
       >
       > make failed, exit code 2
       >
       > Gem files will remain installed in /nix/store/p69v1hf4ykqvaf92h1mb1jhxazv7cs70-ruby3.1-nokogiri-1.15.3/lib/ruby/gems/3.1.0/gems/nokogiri-1.15.3 for inspection.
       > Results logged to /nix/store/p69v1hf4ykqvaf92h1mb1jhxazv7cs70-ruby3.1-nokogiri-1.15.3/lib/ruby/gems/3.1.0/extensions/arm64-darwin-23/3.1.0/nokogiri-1.15.3/gem_make.out
       For full logs, run 'nix log /nix/store/0jr1749dk5ddar9x2n69ysagdj068245-ruby3.1-nokogiri-1.15.3.drv'.
error: 1 dependencies of derivation '/nix/store/kigmgvma1gic7jvngv1klim9g9jcl9p7-terraspace.drv' failed to build
error: 1 dependencies of derivation '/nix/store/c77q25lil7cm9pf3bvf1g1yc97gbgmq3-terraspace-2.2.8.drv' failed to build

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[x] Fits CONTRIBUTING.md.

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

fixes https://hydra.nixos.org/build/265294894 and https://hydra.nixos.org/build/265369065


bugfix: prevent TypeError in the bug detectors

The path-traversal and command-injection bug detectors now don't throw a TypeError upon receiving arguments of the wrong type. Instead, they forward their arguments as-is to their hooked functions that can give meaningful errors.

I do wonder if this should apply to URLs as those should also be valid paths to open. Would we be able to detect path traversal on a URL?
Yes, good point, working on it :smile:


Using color schemes in syntax-specific settings

Hello,
linters are working, meaning that I have a piece of code underlined and a mark on the gutter. The problem is that these have no colors.
The underline is white (I'm using a dark theme color for syntax highlight) and the gutter colors is sometimes white, sometimes blue.
I'm using customized syntax colors, they live in (for example) /Packages/User/MyColors/Flatland/Flatland Dark.tmTheme and I usually have different color schemes for different syntaxes (by modifying Preferences/Settings More/Syntax Specific -User`).
I tough that gutter might be conflicting with modific, a plugin that puts marks on the gutter when a tracked file is modified. I've disabled it and the the gutter marks turned white instead of blue..

Sorry about the late response, the problem likely exists with the Syntax Specific settings. SL doesn't check those files for syntax color schemes so it doesn't get to do it's typical additions to the color schemes if they don't exist. Since it can't find the color, it comes out as white. Try a language that doesn't have its own color scheme and uses the one in your User Settings.

Is it possible to require this as a new feature? I have defined custom colors for almost any language I use. Or at least could you point me where to start from for contributing?

So the best place if you wanted to take a crack at it would be to hack at lint/utils.py#L216. You'd want to search for syntax specific preference files as well and if it has a color scheme property, take that over the base preference. Not sure this truly fits #104 but it's a step in the right direction, where a color schemes or users can dictate or change the color of SL markers.

FYI, we're working on a new approach to the entire color scheme thing for Sublime Text > 3148 (i.e. current dev builds), which will fix this: #666

SL4 no longer messes with your color schemes, check it out!  https://github.com/SublimeLinter/SublimeLinter3#sublimelinter-4-beta


Alamofire impossible to create throw inside .responseJSON

I have this method to contact my REST webservice. I want to generate an exception when or connection fails. Xcode tells me that I can not use a function throw VendingMachineError.InvalidSelection because the method .responseJSON {(request, response, JSON) in ecc ... can not do the conversion. How can I fix this? thanks for your help.

You cannot do this with asynchronous calls.
The easiest way to explain it would be to have you create a playground where you dispatch_async to one of the global concurrent queues. Then try to throw inside the closure of the dispatch_async. This is the underlying issue with attempting to throw with async operations. Instead, you need to pass the error off to your completionHandler.

Thanks for the reply .. Unfortunately I made many attempts but I always get the error.
You have a sample code?
thank you

Hi there, i'm having the same issue... i think that if you would allow the responseJSON to throws, we could do rethrows and it would works just fine, or i'm missing something?

Yes, you are missing the core problem. Try to throw inside a dispatch_async call in a playground and you'll start to see what the problem is.

cnoon i'm sorry you right, you think that apple will change that in a near future? or in terms of design is just fine and we need to do a workaround?

I spent 10 days to find a solution to this problem without results. the only solution that I could find and to remove and add the function throws homes .Failure
completionHandler (element: nil, error: ErrroType)

I cannot speak for Apple @kevinbelter but it is certainly a design flaw with the error handling system which is why we use the alternative concept of a Result type in Alamofire.


SOQL Datatable: Single quotes in query string not actually escaped

It's possible to pass in whatever kind of query string you'd like when using the SOQL datatable, but there is also a line that would escape quotes from being used. Because that value isn't assigned, it's still possible to pass in values with extra dynamism, like in the test_query_has_advanced_where_filter_should_return_response test:
String queryString = 'SELECT Name FROM User WHERE Name LIKE \'%BlahBlahBlah%\' LIMIT 1';

It wouldn't be possible to do such powerful lookups using SOQL datatable without these previously unescaped strings being supported, but this is also an avenue for potential SOQL injection.
The support for escaped strings should either be documented in the Readme (with the SOQL injection caveat) or the feature could also be removed entirely.

This is muscle memory biting me in the butt - just adding the escape after the fact.
I'd like to remove it (since effectively, if you are passing SOQL in through a public prop, you presumably know what you are doing).
Will take a look!

This is muscle memory biting me in the butt - just adding the escape after the fact.
I'd like to remove it (since effectively, if you are passing SOQL in through a public prop, you presumably know what you are doing).
Will take a look!

Sounds good to me!

Sounds good to me!


Add Goodtables badge

CI is failing because #11 was not merged when it ran

Can you rebase? I can’t merge from mobile with failing tests ;-)

Rebased ✨

Oh yeah


How do i "Enable" the plugin?

I've just added this asset in the 'addons' folder (using the inbuilt AssetLib) and I am getting an error:

I've quit Godot and reopened the project. However, the editor still cant seem to find the Log singleton? Do you have any suggestions on what I can do to get the editor to find the Log Object?

Ah!
Project > Project Settings > Plugins > Click on 'Enable' plugin checkbox.
I would suggest adding this to the installation instructions!


OP-1820: Correcting bugs with paths for upgrade_build.

Typos in paths that caused issues with the upload script.

bors r+


Rejection not caught in PoolMaster._flushErrors

https://github.com/neumino/rethinkdbdash/blob/master/lib/pool_master.js#L94
On this line the promise is rejected but never caught leading to the following error output:
Possibly unhandled ReqlDriverError: None of the pools have an opened connection and failed to open a new one.

followed by a huge stack trace
There is no way for the user to hook into this error, and no way to silence it

I'm not sure that his error is in the driver. Do you have the stacktrace for this error?
If a pool becomes empty, all the queries that are waiting for a connection from this pool will be rejected. We push handlers here:
https://github.com/neumino/rethinkdbdash/blob/master/lib/pool_master.js#L126
And the driver uses _line only for getConnection, and we always have an error handler when getConnection is called.

The stack trace is on the line I linked above, I can include the full trace if you'd like but I wanted to keep any company specific details out of the issue. To be clear, it's not an actual error, it's that you reject that promise (on the line linked above) and have nothing that actually catches it, as far as I can tell.
I'm able to get this error to occur immediately when adding an invalid/unreachable server to my configuration for rethinkdbdash (or thinky for that fact, since it just forwards it).

Typically if you do run this script and kill rethinkdb just after, I believe you will see this error
var r = require('./lib/index.js')({ // rethinkdbdash for RethinkDB 2.1
  max: 1,
});

r.js('while (true) {}').run();
r.js('while (true) {}').run();

This is because the second queries doesn't have an error handler.
Theses rejections are called with _flushErrors. The property _line is just a queue and the place where we push something inside is in getConnection.
getConnection is called in a few places by the driver

In run where it bubbles up the error to the promise return by run.
In getConnection where we just loop over all healthy pools
In the new-connection handler where it does bind a error listener (the one given in getConnection.

So getConnection always bubbles up the errors to the promise returned by run. The driver runs a query only when discovery is true here and does bind an error handler.
The only place left where this could happen is if the user call run and doesn't add an error handler.

Please run this code and tell me what you see on the command line, just to make sure I'm not going crazy here:
var thinky = require('thinky')({
  host: 'this.totally.does.not.exist',
  port: 28015,
  db: 'some-non-existant-db',
  timeoutError: 5000,
  silent: true
});

var Test = thinky.createModel('test', {
  name: thinky.type.string()
});

Or is the implication that createModel should have an error handler attached to it?

Oh I see, this is actually a bit more complicated and is a thinky issue. I opened https://github.com/neumino/thinky/issues/289 to discuss more about that. Basically when you call createModel, we return a model. not a promise even though we do run a few queries to create the table and indexes. I thought about returning a promise, but it makes using thinky quite painful.
The current behavior in case we fail to create the table/indexes is kind of undefined. We catch the error in a few places but not everywhere. The intended behavior was to throw an error and let the process crash (I think). We may want to change that.


4 frames input to AdaCof

Hello, thank you very much for your excellent contribution, I am very interested in your work.
Can you please share the code file about the AdaCof method to change to 4 frames as input, thank you very much!

Hi, we just used the 2-frame version for AdaCoF without modification.


Draft: Added github action to perform cross builds

Jesus christ this is taking longer than I expected

Isn't emscripten available on ubuntu natively?

sudo apt-get install emscripten didn't work. Not sure what kind of ubuntu version they are using. Also: It is apparently generally not recommended to use the emscripten version from the ubuntu repos, as that one seems to be quite outdated

What exaclty is this error supposed to mean? (Happened on this action)
[37/39] Compiling C++ object libOxygenEngine.a.p/src_types_OE_World.cpp.o
em++: warning: linker setting ignored during compilation: 'WASM' [-Wunused-command-line-argument]
em++: warning: linker setting ignored during compilation: 'EXPORT_ALL' [-Wunused-command-line-argument]
em++: warning: linker setting ignored during compilation: 'USE_WEBGL2' [-Wunused-command-line-argument]
em++: warning: linker setting ignored during compilation: 'SINGLE_FILE' [-Wunused-command-line-argument]
em++: warning: linker setting ignored during compilation: 'FULL_ES3' [-Wunused-command-line-argument]
em++: warning: linker setting ignored during compilation: 'NO_DISABLE_EXCEPTION_CATCHING' [-Wunused-command-line-argument]
em++: warning: linker setting ignored during compilation: 'ALLOW_MEMORY_GROWTH' [-Wunused-command-line-argument]
em++: warning: linker setting ignored during compilation: 'PTHREAD_POOL_SIZE' [-Wunused-command-line-argument]
[38/39] Linking static target libOxygenEngine.a
[39/39] Linking target OE_Test.js
FAILED: OE_Test.js
em++  -o OE_Test.js OE_Test.js.p/examples_oe_main_test.cpp.o -s ERROR_ON_UNDEFINED_SYMBOLS=1 -s EXPORT_ALL=1 -s USE_SDL=2 -s USE_WEBGL2=1 -s SINGLE_FILE -s FULL_ES3=1 -s NO_DISABLE_EXCEPTION_CATCHING -s ALLOW_MEMORY_GROWTH=1 -s PTHREAD_POOL_SIZE=2 -s USE_PTHREADS=1 --preload-file=../assets@/assets -Wl,--start-group libOxygenEngine.a -Wl,--end-group -s USE_PTHREADS=1 -s PTHREAD_POOL_SIZE=4
em++: warning: USE_PTHREADS + ALLOW_MEMORY_GROWTH may run non-wasm code slowly, see https://github.com/WebAssembly/design/issues/1271 [-Wpthreads-mem-growth]
Nothing to do!
em++: error: '/home/runner/emsdk/upstream/emscripten/tools/file_packager OE_Test.data --from-emcc --export-name=Module --preload ../assets@/assets' failed (returned 1)
ninja: build stopped: subcommand failed.
Error: Process completed with exit code 1.

FYI the cross_web script has to be run from the top level directory and not some random other directory.
emscripten also needs to package the .csl files in the assets directory else they won't be available.

It needs access to the *.csl files? There is the problem

Ah yes, i knew this was gonna happen.

Any idea what this is about?
[208/213] Compiling C++ object libOxygenEngine.a.p/src_types_OE_World.cpp.obj
[209/213] Linking static target libOxygenEngine.a
[210/213] Compiling C++ object CSL_Test.exe.p/examples_csl_main_test.cpp.obj
[211/213] Compiling C++ object OE_Test.exe.p/examples_oe_main_test.cpp.obj
[212/213] Linking target CSL_Test.exe
[213/213] Linking target OE_Test.exe
cp: cannot stat '/usr/x86_64-w64-mingw32/bin/libwinpthread-1.dll': No such file or directory
Error: Process completed with exit code 1.

You can comment out the last line on cross_linux2windows (but do not remove it).
What it does is copy the only needed .dll so one can package the engine easily. It was used for share_oe.zip before.

The path to the .dll was hardcoded for Arch Linux

Also a good idea would be to split the cross_build jobs into one for emscripten and one for windows.

That might be a good idea. The reason I have not split them yet, is because I am not entirely sure which option would run faster. Splitting them into two different jobs would basically require another ubuntu installation with required compilers, libraries, etc. to be created. But running them in parallel might speed it up actually. Will test in a new commit

Separate jobs are faster, keeping the commit


path.extname is deprecated.

Argument to path.extname must be a string
path.extname (/usr/share/atom/resources/app.asar/src/electron-shims.js:20:10)
ATPOutputView.getCwd (/home/sudhanshu/.atom/packages/atom-terminal-panel/lib/atp-view.coffee:1874:17)
BuiltinVariables.parseFull (/home/sudhanshu/.atom/packages/atom-terminal-panel/lib/atp-builtins-variables.coffee:188:77)
BuiltinVariables.parseHtml (/home/sudhanshu/.atom/packages/atom-terminal-panel/lib/atp-builtins-variables.coffee:111:16)
ATPOutputView.parseTemplate (/home/sudhanshu/.atom/packages/atom-terminal-panel/lib/atp-view.coffee:664:36)
ATPPanel.updateStatusBar (/home/sudhanshu/.atom/packages/atom-terminal-panel/lib/atp-panel.coffee:190:50)

atom-terminal-panel(1 deprecation)
i have the same deprecation

same problem

same problem

same here

+1

same

same here.

I have the same problem here... 😢

So I was able to temporarily fix this issue... it could also just be a workaround so please follow this at your own risk. It did however remove the "deprecation" message from atom for me.
On the deprecated call, click on the second path:
ATPOutputView.getCwd (/home/sudhanshu/.atom/packages/atom-terminal-panel/lib/atp-view.coffee:1874:17)
In the atp-view.coffee file, for the getCwd method, you will want to make the following edits to the extFile and projectDir variables:
extFile = extname atom.project.getPaths()[0] if typeof atom.project.getPaths()[0] is 'string'
projectDir = dirname atom.project.getPaths()[0] if typeof atom.project.getPaths()[0] is 'string'
these are lines 1318 and 1331 respectively.
getCwd should look like this:
`getCwd: ->
if not atom.project?
return null
extFile = extname atom.project.getPaths()[0] if typeof atom.project.getPaths()[0] is 'string'
if extFile == ""
  if atom.project.path
    projectDir = atom.project.path
  else
    if process.env.HOME
      projectDir = process.env.HOME
    else if process.env.USERPROFILE
      projectDir = process.env.USERPROFILE
    else
      projectDir = '/'
else
  projectDir = dirname atom.project.getPaths()[0] if typeof atom.project.getPaths()[0] is 'string'

cwd = @cwd or projectDir or @userHome
return @correctFilePath cwd`

I hope this helps. I was glad I no longer saw the deprecation message 😆

I did as what @AuggieMBP had said.And the problem was solved.
Pay attention to the lines to be modified.

Created a Pull Request for this (#118) based on @AuggieMBP's comment

Same problem... When an updated to the package will be released?  I like much this package but don't like to have deprecation alert in my atom :(

same problem

same issue

same issue

same issue

same issue

same issue

same issue

same issue

same issue

same issue

Solution from @AuggieMBP solve problem

same issue

I have an issue the charset codification of the console, look this text:
"El n�mero de serie del volumen es: ECA4-13CE"
How can i upgrade this charset tu UTF-8?
Thanks!

Yup, same here....

FYI here is a Deprecated calls I am getting
Argument to path.extname must be a string
Report Issue
path.extname - /Applications/Atom.app/Contents/Resources/app.asar/src/electron-shims.js:20:10
ATPOutputView.getCwd - /Users/stigadmin/.atom/packages/atom-terminal-panel/lib/atp-view.coffee:1874:17
BuiltinVariables.parseFull - /Users/stigadmin/.atom/packages/atom-terminal-panel/lib/atp-builtins-variables.coffee:188:77
BuiltinVariables.parseHtml - /Users/stigadmin/.atom/packages/atom-terminal-panel/lib/atp-builtins-variables.coffee:111:16
ATPOutputView.parseTemplate - /Users/stigadmin/.atom/packages/atom-terminal-panel/lib/atp-view.coffee:664:36
ATPPanel.updateStatusBar - /Users/stigadmin/.atom/packages/atom-terminal-panel/lib/atp-panel.coffee:190:50

+1

+1

+1

+1

same problem

Up

+1

+1

+1

I did as what @AuggieMBP  had said.And the problem was solved.

+1

+1

+1

+1

I think this is dead

+1

@Encr1pt3d : please find a patch for charset issue https://github.com/styczynski/atom-terminal-panel/pull/175

@AuggieMBP  Thanks man

+１

o.O3  ---> any update ?

+1

I have the same issue

Still not fixed?

still not fixed


azure-storage-file-datalake not able to work with Append Blobs

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I am trying to work with Append Blobs on Data Lake Gen 2 because my case requires me to write incoming telemetry into files stored on Data Lake storage account. Although, I am able to create Append Blobs manually from the portal, and this post (https://azure.microsoft.com/en-us/updates/append-blob-support-in-azure-data-lake-storage-is-now-generally-available/) says that they have recently become generally available, I cannot find a method in the DataLakeFileClient class from the python azure-storage-file-datalake SDK to be able to create or append blocks to Append Blobs. By default, the create_file() method from the DataLakeFileClient class only creates Block Blobs, and there doesn't seem to be a parameter to make it create Append Blobs.
Describe the solution you'd like
A clear and concise description of what you want to happen.
It would be good to have a property in the create_file method of the DataLakeFileClient class to specify the type of blob that will be created.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
I have tried using the python azure-storage-blob SDK, however, it does not seem to be able to locate specific folder paths from containers in my Data Lake Gen2, since its only expecting the container's name.
Additional context
Add any other context or screenshots about the feature request here.

Hi @marincon-sarmiento, sorry for the long delay in response and thanks for question.
Unfortuantely, the datalake SDK does not support Append Blobs. Datalake files are backed by Block Blobs as Block Blobs can support the append + flush operations that datalake files need and therefore there is not usually a need for an Append Blob. So, you have a couple of options for your scenario.

Use the datalake SDK and files without worrying about the backing blob type. You should be able to use append_data and flush_data to append data to your file. We also have a feature coming within the next couple of releases that will allow you to append and flush in a single operation.

Your other option is to use the blobs SDK to support append blobs. The Blobs SDK will support some operations on an ADLS Gen2 account and you should be able to still create an Append Blob and use append_block on your Datalake account. This can tricky if you need to integrate your file into an existing datalake filesystem though and is not recommended in most cases.

Hi @marincon-sarmiento, I just wanted to touch base with you and see if this issue is persisting? If not we shall continue with the closure of this issue shortly.
Thanks!


if user deletes all tasks then show There's nothing todo!

Resolved with showHideNothingClass in app.js


No module named tensorFlow when trying to import it through Neptune

System information

I am following the instructions found here in order to create a roi pooling using Neptune and tensorflow:
Linux Ubuntu 16.04
TensorFlow installed from binary, using virtual environment
TensorFlow version (use command below):1.7
Python version: 3.5.2
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: CUDA
GPU model and memory: working with CPU
Exact command to reproduce:
neptune run             --parameter im_folder:$PWD/../data/images             --parameter roidb:$PWD/../data/roidb             --parameter pretrained_path:$PWD/../data/vgg16-20160129.tfmodel

Describe the problem
When trying to run the previous command, I get the ImportError: No module named tensorflow error while the environment is activated. If I try to import it with a  normal python command it works perfectly.
Source code / logs
(tensorflow3) ignacio@ignacio-Swift-SF314-52:~/Escritorio/nacho/AVIVA/roi-pooling/code$ neptune run             --parameter im_folder:$PWD/../data/images             --parameter roidb:$PWD/../data/roidb             --parameter pretrained_path:$PWD/../data/vgg16-20160129.tfmodel
>
> Experiment enqueued, id: SAN-43
>
> To browse the experiment, follow:
> https://app.neptune.ml/dashboard/experiment/d28dc701-8477-4dee-9281-def81feddc8f?getStartedState=folded
>

Calculated experiment snapshot size: 44.82 kB
Sending sources to server: 100%|█████████████████████████████████████████████████████████| 44.8k/44.8k [00:00<00:00, 170kB/s]
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/deepsense/neptune/job_wrapper.py", line 107, in <module>
    execute()
  File "/usr/local/lib/python2.7/dist-packages/deepsense/neptune/job_wrapper.py", line 103, in execute
    execfile(job_filepath, job_globals)
  File "main.py", line 4, in <module>
    from trainer import Trainer
  File "trainer.py", line 8, in <module>
    import tensorflow as tf
ImportError: No module named tensorflow
Calculated experiment snapshot size: 0 Bytes
Process exited with return code 1.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution

Hi, I have updated the issue. Thanks!

Any hints? I tried also with the native pip installation but I get the same output

Nagging Assignee @tatatodd: It has been 22 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 36 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 66 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 81 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 96 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


[Fix] nvm_strip_path: Preserve leading/trailing colons

Preserve leading & trailing colons in awk invocation.

This makes sense for MANPATH, but I'm not sure it makes sense for PATH.
As noted in the bug, empty directory names in path do have some meaning, even if it seems less useful. So colons should be preserved there too.

Also, what happens if the thing being removed is at the end? We wouldn't want to leave behind a double colon.
What this awk invocation does it put a colon before every unstripped directory name apart from the first. And then checks the end of the input to see if it needs to add a trailing colon. So I don't think there's a way to end up with a double colon, unless there's one in the input - unless you can see a case where it would?

I think the existing tests already cover this case anyway, they have entries in $NVN_DIR at the end of $TEST_PATH.

The other potential problem case is when it's not at the beginning or end.

The other potential problem case is when it's not at the beginning or end.

This works fine I think - with the patch I get:
$ nvm_strip_path /a/bin:$NVM_DIR/b/bin:/c/bin /bin
/a/bin:/c/bin

because awk continues to the next record when it finds a path entry to remove, and skips the part that prints anything.
Note that when the MANPATH or PATH documentation talks about a trailing colon, they mean when a colon is the last character in the input env variable, not about every colon used as a record separator. I feel like there's a bit of subtlety here, and half the challenge is to explain it well in the commit log.

Hmm, it's also possible that I'm stuck on my mental model from the original implementation, which did not use awk - maybe the awk-based solution doesn't have this fragility.


feat: generate Markdown from OpenAPI spec

Command Palette:

Tree View:

Codecov Report
Attention: Patch coverage is 6.19469% with 106 lines in your changes are missing coverage. Please review.

Project coverage is 28.84%. Comparing base (795ac9f) to head (9ff3f72).

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #178      +/-   ##
==========================================
- Coverage   29.33%   28.84%   -0.50%
==========================================
  Files          70       72       +2
  Lines        4305     4389      +84
  Branches       88       88
==========================================
+ Hits         1263     1266       +3
- Misses       3042     3123      +81

Files
Coverage Δ

src/constants.ts
100.00% <100.00%> (ø)

src/uiStrings.ts
100.00% <100.00%> (ø)

src/utils/openApiUtils.ts
70.90% <100.00%> (ø)

src/commands/detectBreakingChange.ts
0.00% <0.00%> (ø)

src/extension.ts
0.00% <0.00%> (ø)

src/commands/generateHttpFile.ts
82.08% <20.00%> (+0.72%)
:arrow_up:

src/utils/fsUtil.ts
28.12% <25.00%> (-0.73%)
:arrow_down:

src/utils/apiSpecificationUtils.ts
0.00% <0.00%> (ø)

src/commands/generateMarkdownDocument.ts
0.00% <0.00%> (ø)


VST3 not recognized in Ableton on Mac M1

Hi, thank you for making free plugins, I'd like to try them. I found a link from a youtube video and tried to install on my MacBook Pro M1 with Ableton Live 11 and Ableton Live 10 and neither one showed the plugins in Ableton's browser. I followed Ableton's help instructions and none of them worked, do you know what might be wrong? Thanks

Hi,
The plugins up on the CI are compiled for x86_64. Does Ableton Live have no way to use those plugins (through Rosetta?). I know the native ARM version of Bitwig Studio can do that using its sandboxing features. I don't have a Mac so I also have no way to test the Mac versions, but they should work fine.
If anyone wants to help set up ARM macOS builds on the CI then that would be very welcome, but I'd rather not overcomplicate things and I also have no way to test this without buying a Mac. Cargo doesn't support universal binaries, and there is also no ARM macOS GitHub Actions runner.
If you compile the plugins yourself (you need to install rustup and then run a command in a terminal, there are instructions on each plugin's page) then those should work just fine, but I don't have ARM-native binaries up on the CI right now. Sorry!

hmm, I think I might know how to do this, so I may work on it if this is something that is wanted?

@goggledefogger you could try to open ableton using rosetta, by right clicking on the application and clicking "get info", and checking the rosetta box before opening it:

I would also suggest cloning the code and building it on the mac itself, which would build for apple silicon and probably show up in ableton, but I just tried doing that and it has a mistake in the code (will PR a fix in a moment). Also for some reason it doesn't want to bundle it "because the package does not export any plugins", so must try to figure it out >~<

hmm, I think I might know how to do this, so I may work on it if this is something that is wanted?

@autumnblazey If you know how to set up another build for macOS that compiles to AArch64 macOS from x86_64 macOS (since that's what GitHub's CI runners are running) then that would be great! Rust currently doesn't have a built in way to do universal builds, and I would very much prefer not having to make the xtask bundle implementation more complicated by manually approximating that with lipo or something like that. So just having two separate macOS builds would be nice.

Also for some reason it doesn't want to bundle it "because the package does not export any plugins", so must try to figure it out >~< Some pointers on why this is may be helpful

I'll assume that's a native build on an M1 Mac? No idea, but the answer must be in here:  https://github.com/robbert-vdh/nih-plug/blob/bc2b3410891c2b15c9869bd2f5debee689dc78f2/nih_plug_xtask/src/symbols.rs#L5-L34
That's what NIH-plug's bundler currently uses to check which symbols are exported by a plugin. This works fine on x86_64 macOS, so the only reasonable explanation is that this is either a bug in the used goblin binary parsing library, or the library is somehow a fat multiarch Mach-O binary with the AArch64 version being the second architecture. That would sound very strange, but somehow it also sounds like a very Apple thing to do. You should be able to add a dbg!(&obj); before line 19 there to find out.

Oh and the third option would be that the symbol on AArch64 binaries doesn't have the underscore prepended to it. Still don't know why that's a thing. So if the dbg!(&obj); doesn't show anything interesting, look through the obj.exports to check which symbols are actually exported.

Okay, I added AArch64 builds for macOS @goggledefogger! You can find them separately from the x86_64 builds on both the GitHub Actions pages and on the nightly.link page.
@autumnblazey No idea why it didn't work for you, but the symbol detection worked fine when cross compiling to aarch64-apple-darwin on the macos-11 GitHub CI runner.

Hi, I'm currently working on M1 macOS and am running into an issue where the .vst3 files I build from the plugins/ folder are not showing up in Ableton Live 11, while the plugins downloaded from the nightly builds are working fine. To build, I am running the following:
cargo xtask bundle buffr_glitch --target aarch64-apple-darwin --release

I also tried running this:
cargo xtask bundle-universal buffr_glitch --release

but was met with a significant number of compiler errors.
Based on the CI changes here, I would think that the builds should be the same, but maybe I'm missing something.

@aponplugins Not sure why the plugins downloaded from GitHub would be fine (they're also unsigned), but AArch64 macOS seems to be much stricter about code signing and may refuse to run anything that isn't signed. Try running codesign -f -s - 'target/bundled/plugin-name.vst3'. Apparently there's an issue where Cargo doesn't always do that for you (it's supposed to). This may have something to do with Live's use of the hardened runtime. This supposedly isn't an issue in other hosts like Bitwig Studio. I'll add this step to the bundler, hopefully that helps.

That worked perfectly, thanks! Yes I'm not sure why the downloaded versions worked, there aren't any _CodeSignature files in them.
Also, FYI, I had to run this:
xattr -cr 'target/bundled/plugin-name.vst3'

before re-signing the project files. This is because I got this error trying to re-sign:
resource fork, Finder information, or similar detritus not allowed

I found the solution here:
https://stackoverflow.com/questions/39652867/code-sign-error-in-macos-monterey-xcode-resource-fork-finder-information-or
Based on these responses, maybe my compilation of the executables is creating local resource forks and my OS can't work with that.

Any idea what triggered that to be necessary that? The linked docs talk about a change from macOS Sierra, and I haven't heard about having to use xattr -cr before.

I think I might have the answer for you. I'm not a plugin dev (yet?) or even a mac software dev, so maybe this is already well known, but I had the same problem on both intel and m1 macs where the plugins just wouldn't show in any DAW. The above didn't seem to help me because I had two problems.
The main issue solved above is that macs "quarantine" unknown executables for security, such as those downloaded in browser. With an exe you can just allow it in Security settings, and the same thing appeared for plugins but it didn't do anything. I found in a random reddit post that you can remove it manually like this:
xattr -rd com.apple.quarantine <vst3 path>
Which explains why the above xattr -cr works, it's just removing all attributes. I guess they're given those attributes as part of the build. So you'll need to do that locally or make it part of your build on mac.
That worked once I copied them into files in /Library/Audio/Plug-Ins/VST(3) or the ~/Library equivalent. My other problem was I was trying to load them directly from my build folder under ~/src. I guess the DAW doesn't have access to those folders and silently ignored it. So copy them into ~/Library/... if your DAW supports that. Symlinks may also work.
With that solved, I can now become an audio plugin dev! Thanks for this great framework!