sh Windows format issue

Just info for Windows users. Cloning on Windows possibly saves plugins.sh (and other .sh) in Windows format. This produces "/bin/sh^M: bad interpreter: No such file or directory" error when build is started.
I've noticed this when using git clients (SourceTree).
Maybe install dos2unix ("yum install dos2unix") and "RUN dos2unix plugins.sh" after copying from dockerhost to container to prevent errors when editing these files on Windows (and saving in Windows format).
Or some script on dockerhost side..
http://stackoverflow.com/questions/14219092/bash-my-script-bin-bashm-bad-interpreter-no-such-file-or-directory

I should do more testing on Windows but yea, this is a definite challenge with cross platform checkins and shell scripts.  Thanks for the heads up! I'll take ac loser look and see if I can fix somethings.

That's an issue with Carriage Returns  in windows. By default git on windows adds carriage returns to line-endings. This works oke, when you run it on windows. Docker container, however, are linux images.
You can disable the carriage returns with:
git config --global core.autocrlf false

Note that this messes up the newlines when you open the file in notepad. It works fine in sublime though.


homebridge on docker can't find module './node_modules/eiscp/eiscp-commands.json'

I'm running homebridge on Docker and tried to install homebridge-onky plugin. Installation is successful using homebridge web interface

But a restart of homebridge raises an error:

But the module is there:

Any idea what I made wrong?

I have the same issue.

Problem solved: see above!

Not reproducing - issue should be closed

I upgraded homebridge to the 0.4.50 build of https://github.com/oznu/docker-homebridge and not the problem reoccurs for me, asking for the eiscp file in the path before the last fix.
I will submit a patch to fix this.

I commited the changes. @ToddGreenfield please push to npm.

the error persists still  at 0.6.5.
cf. above solution,  edit /homebridge/node_modules/homebridge-onkyo/index.js:
from:
var eiscpData = require('../eiscp/eiscp-commands.json');
to :
var eiscpData = require('./node_modules/eiscp/eiscp-commands.json');

The current version on Github is 0.6.6 - waiting for @ToddGreenfield to publish this version to npmjs.org as I do not have the access rights to do so.

Christian- I am not able to maintain this anymore. If you would like to be a maintainer on npm let me know your username and I can grant access to update.

@solowalker27 homebridge-onkyo can be add in HomeKit
https://github.com/solowalker27/homebridge-onkyo

New version was published on GitHub and on npmjs.org


prevent db connections during asset precompile

Problem
We are creating db connections during asset compilation
Solution
Do connections on load
Full disclosure: i have no idea what i'm doing :(
@kirs @rafaelfranca /cc @celsodantas @fw42

This is inside a railtie and then already depends on Active support. This
is the correct way to do this.
On Sat, Nov 26, 2016 at 3:18 AM Izaak (John) Alpert <
<EMAIL_ADDRESS>wrote:

@karlhungus commented on this pull request.
In lib/bugsnag/railtie.rb
https://github.com/karlhungus/bugsnag-ruby/pull/1#pullrequestreview-10208480
:

@@ -45,8 +45,10 @@ class Railtie < Rails::Railtie
ActionController::API.send(:include, Bugsnag::Rails::ControllerMethods)
end
if defined?(ActiveRecord::Base)

   require "bugsnag/rails/active_record_rescue"

   ActiveRecord::Base.send(:include, Bugsnag::Rails::ActiveRecordRescue)

   ActiveSupport.on_load(:active_record) do

@celsodantas https://github.com/celsodantas pointed out this is
probably unacceptable because it creates a dependency on active support :(.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/karlhungus/bugsnag-ruby/pull/1#pullrequestreview-10208480,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC66PgFXsmMwH-utWtIHA4k7dqH0FFgks5rB5cIgaJpZM4K8rSo
.

going to close this and point it at bugsnag/master -- i'll reping you all

We can't use bugsnag master right now because it has code that's not compatible with Shopify.
https://github.com/Shopify/shopify/pull/93214

Ok, it should be fine with Frenzy 👌

bugsnag/bugsnag-ruby version: https://github.com/bugsnag/bugsnag-ruby/pull/337

@kirs assuming it's ok :cool: if you merge this to bugsnag/bugsnag-ruby (i don't have write access)


Fix build of 'pyacoustid' under Python 3.

The added patch (source) ought to fix the Hydra failure for this module under Python 3.

Travis passed, thanks.


[range-v3] Update to 0.9.1

Whoops forgot about that, I'll update that when I get to my PC. However, is that not the latest? That's what I'm seeing here https://github.com/ericniebler/range-v3/releases

The release notes here say it contains gcc-9.x portability fixes. So should it be kept at 0.9.1?
https://ericniebler.github.io/range-v3/release_notes.html

LGTM, thanks @SuperWig for the PR and @ehsan-mohammadi for the review!


Fixed an issue with a wrong input view height after sending a multiline text message

Codecov Report

Merging #568 into master will decrease coverage by 0.1%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #568      +/-   ##
==========================================
- Coverage   59.97%   59.86%   -0.11%
==========================================
  Files          87       87
  Lines        3945     3952       +7
==========================================
  Hits         2366     2366
- Misses       1579     1586       +7

Impacted Files
Coverage Δ

...ChatController/Collaborators/KeyboardTracker.swift
61.68% <0%> (-2.94%)
:arrow_down:

Continue to review full report at Codecov.

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


add link to sizing guide documention for monitoring

On the page for enabling monitoring, we should link out to https://rancher.com/docs/rancher/v2.x/en/cluster-admin/tools/monitoring/#resource-consumption
so that people enabling know there is a sizing recommendation

with this, we should also improve the docs: https://github.com/rancher/docs/issues/1363

from @vincent99:
has to be hidden if ui-pl setting != rancher

Version: Master (4/15/19) ImageID: 0d755e769e77
Confirmed issue. Now, when you first try to enable cluster monitoring, you will see a message at the top suggesting to review the docs. This properly links to https://rancher.com/docs/rancher/v2.x/en/cluster-admin/tools/monitoring/#resource-consumption
If, in the settings, you set the ui-pl to non-default (not "rancher") then the message does NOT show up as expected (see the comment above, https://github.com/rancher/rancher/issues/19392#issuecomment-479986997).
Here is a screenshot of what this looks like (assuming ui-pl == rancher)


Add clarification for hook execution order

Summary
Add clarification for hook execution order. Existing problem may include:

Misconception that the top-level before hooks are ran before each describe or test block, and not necessarily every test block (although this is addressed in the docs with the sentence "By default, the before and after blocks apply to every test in a file.", this illustration makes it more explicit)
Confusion about the execution order of nested beforeAll hook in relation to top-level beforeEach
Likewise, confusion about the execution order of nested afterAll hook in relation to top-level afterEach

Test plan
I could not test the website / formatting is correct due to #4911 and #4912, but it is valid Markdown and fits within 80 character line-length limit.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #4913   +/-   ##
======================================
  Coverage    59.9%   59.9%
======================================
  Files         197     197
  Lines        6549    6549
  Branches        4       4
======================================
  Hits         3923    3923
  Misses       2626    2626

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 3d84d75...0850780. Read the comment docs.

That's useful. Thanks for adding the clarification.


[Addon] [dex] upgrade chart version and add support of oauth connector

Description of your changes

Upgrading the chart version to v0.15.3 in order to use the latest version of dex (v.2.37.0) (see https://artifacthub.io/packages/helm/dex/dex/0.15.3)
Adding the support of the [OAuth connector] (https://dexidp.io/docs/connectors/oauth/)

I set arbitrary the new number version of the addon to 0.7.0 but feel free to change it accordingly to your policy
How has this code been tested?
This version has been deployed and tested internally in our environment
Checklist

I have:

[x] Title of the PR starts with type (e.g. [Addon] , [example] or [Doc]).
[ ] Updated/Added any relevant documentation and examples.
[ ] New addon should be put in experimental.
[x] Update addon should modify the version in metadata.yaml to generate a new version.

Verified Addon promotion rules
If this pr wants to promote an experimental addon to verified, you must check whether meet these conditions too:

[ ] This addon must be tested by addon's e2e-test to guarantee this addon can be enabled successfully.
This addon must have some basic but necessary information.

[ ] An accessible icon url and source url defined in addon's metadata.yaml.
[ ] A detail introduction include a basic example about how to use and what's the benefit of this addon in README.md.
[ ] Also provide an introduction in KubeVela documentation.
[ ] It's more likely to be accepted if useful examples provided in example dir.

Please sign the DCO.


How can I Add Cache Plugin?

Would you care to elaborate your question?
To use plugins and create custom ones, you can see the doc here
You may also be interested in the issue #1679.


Figure out the intended link for https://developers.flow.com/concepts/transaction-lifecycle

In https://developers.flow.com/nodes/access-api#block, there's a link pointing to "block formation guide" that's broken. This might be referring to the white paper in https://flow.com/technical-paper but this needs to be confirmed.

The original link was created by @psiemens, reaching out on slack to figure out the correct replacement

static builds https://github.com/onflow/docs


Make CreateFinalResult actually save the result

The mutated result is not saved to datastore before returning.

Fixed in #47 (@pbochis :smirk:)


Get rid of "format-all-ensure-formatter"

Hi!
Is there really a good use case for format-all-ensure-formatter? To me, it seems like like a superfluous option that only does what a normal user would expect anyway. But maybe I'm just being blind here.
Still, I think that you should

get rid of format-all-ensure-formatter (and call it behind the scenes), or
call it by default and let power users disable it by setting an option such as format-all-do-not-ensure-formatter

Thanks for your package – now that I call format-all-ensure-formatter from a hook it works great! It seems that I skipped that paragraph when reading the manual... :smile:
Martin

The point of not using a formatter automatically is that many projects do not use code formatters (or they use a formatter for some languages, but not for others). If you have a default formatter, and use it to format code belonging to those projects, the formatting will likely be incorrect according to the style guide of that project.
I agree that it's confusing that you have to remember to call format-all-ensure-formatter. I can't think of an ideal way to do this. Would be nice to hear ideas from more people.

Also, setting format-all-formatters buffer-locally to nil is supposed to disable formatting for that buffer. These are quite logical semantics, but nil is currently the default value, and as you say, that may not be a logical default setting.

Thanks for your fast response!
You are right, the semantics make sense and are pretty logical. But maybe you should add your explanation to the README as it currently isn't very clear on that point. :smile:
Also, I find that adding a hook to a package in order to run that package slightly counter-intuitive. My idea is to remove the format-all-ensure-formatter hook and exchange it for a customizable variable format-all-enabled (declared as safe) that is set to non-nil by default. This could easily be customized using local variables, and should run just as fast as the current implementation.
Just my two cents.


Pool exhausted, Java 21 and Virtual Threads

Hi, I am getting Pool exhausted when I pass a ClientResources which does use an executor with virtual threads.
java.util.concurrent.CompletionException: java.util.NoSuchElementException: Pool exhausted
	at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(Unknown Source) ~[?:?]
	at java.base/java.util.concurrent.CompletableFuture.uniApplyNow(Unknown Source) ~[?:?]
	at java.base/java.util.concurrent.CompletableFuture.uniApplyStage(Unknown Source) ~[?:?]
	at java.base/java.util.concurrent.CompletableFuture.thenApply(Unknown Source) ~[?:?]
	at io.lettuce.core.support.AsyncConnectionPoolSupport$1.acquire(AsyncConnectionPoolSupport.java:189) ~[lettuce-core-6.3.2.RELEASE.jar:6.3.2.RELEASE/8941aea]

This is my java code,
var executor = Executors.newThreadPerTaskExecutor(Thread.ofVirtual().factory());
EventLoopGroup customEventLoopGroup = new NioEventLoopGroup(0, executor);
var clientResources =
        ClientResources.builder().eventExecutorGroup(customEventLoopGroup).build();
RedisClient.create(clientResources, ...);

how can I fix this?
I am using lettuce-core-6.3.2.RELEASE.jar running on a linux container.
Thanks!

I am also interested in this. I could not find a clear documentation on how to use virtual threads in lettuce. Appreciate a good reference.

Hi, I am getting Pool exhausted when I pass a ClientResources which does use an executor with virtual threads.
java.util.concurrent.CompletionException: java.util.NoSuchElementException: Pool exhausted
	at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(Unknown Source) ~[?:?]
	at java.base/java.util.concurrent.CompletableFuture.uniApplyNow(Unknown Source) ~[?:?]
	at java.base/java.util.concurrent.CompletableFuture.uniApplyStage(Unknown Source) ~[?:?]
	at java.base/java.util.concurrent.CompletableFuture.thenApply(Unknown Source) ~[?:?]
	at io.lettuce.core.support.AsyncConnectionPoolSupport$1.acquire(AsyncConnectionPoolSupport.java:189) ~[lettuce-core-6.3.2.RELEASE.jar:6.3.2.RELEASE/8941aea]

This is my java code,
var executor = Executors.newThreadPerTaskExecutor(Thread.ofVirtual().factory());
EventLoopGroup customEventLoopGroup = new NioEventLoopGroup(0, executor);
var clientResources =
        ClientResources.builder().eventExecutorGroup(customEventLoopGroup).build();
RedisClient.create(clientResources, ...);

how can I fix this?
I am using lettuce-core-6.3.2.RELEASE.jar running on a linux container.
Thanks!

Hey I'm not quite sure if I know the solution, but you might want to look into
newVirtualThreadPerTaskExecutor()

Hey @jkvargas ,

        Executor executor = Executors.newThreadPerTaskExecutor(Thread.ofVirtual().factory());
        EventLoopGroup customEventLoopGroup = new NioEventLoopGroup(0, executor);
        var clientResources =
                ClientResources.builder().eventExecutorGroup(customEventLoopGroup).build();

        try (RedisClient clusterClient = RedisClient.create(clientResources, redisURI)) {
            StatefulRedisConnection<String, String> connection = clusterClient.connect();
            connection.sync().set("key", "value");
        }```

I tried your code and it worked for me, could this be some environment issue with your JVM / JDK?

Hey I'm not quite sure if I know the solution, but you might want to look into newVirtualThreadPerTaskExecutor()

Both newVirtualThreadPerTaskExecutor() and newThreadPerTaskExecutor()

redisTemplate.getRequiredConnectionFactory().getConnection();
org.springframework.data.redis.connection.lettuce.LettuceConnectionFactory.SharedConnection#getConnection,   getNativeConnection()  is stucked,   there is synchronized , after upgrade to springboot 3.3.2 which is using lock,  it's still stucked.

See #3085


Hunter 1.20 fix header installation

Released: https://github.com/hunter-packages/leveldb/releases/tag/v1.20-p1


Provide a general option to set the shell to be used in for Run Command or when a path is opened in Console

Description of the new feature / enhancement
The default shell used for Run Command or when opening a path in a Console is cmd.  I'd like to use my preferred shell, which is TCC from JP Software.  Others may want to use a shell like bash or PowerShell.
Scenario when this would be used?
The principle scenario in which I'd like this feature is when I type a path name in the Run box and elect to open the path in a Console window.  I want to open that path in TCC, not cmd.  Someone else may wish to open it in bash or PowerShell.  We should have the choice.
Another scenario is when executing a shell command using > in Run.  Some scripts or commands might be sensitive to the shell they are executed within.  The shell startup might configure the environment a particular way, for example.  Or the execution of subshells could be affected.
These two cases might necessitate two different options.  Of the two, the first may be more useful than the second, since the shell remains open after opening a path whereas when executing a command it closes immediately upon command completion.  (Maybe leaving the shell open should be an option too!)
Supporting information
No response

/dup #29892

I have de-duplicated this against #29892 because the first part (Shell plugin setting) does exist now and the other issue is more precise for the second part that is requested.


Ink console

There are still two problems that I was not able to get working.

the commands which I talked about on the slack chat
closing the console pane

Closing the console pane is specially problematic given that there is currently no method on ink's console which emit that event upwards so that the repl process can be closed. The argument from ink's creators is that their (julia) client is always processing evals and the console is always receiving the output, but remains hidden until the user decide to actually open the console. Their approach (and my opinion as well) is closely related to Lighttable's way of doing things; display everything inline unless you need to check stdout or stderr.
I think it should be possible to add methods to the console such that it emits an event when it is closed but I am not familiar with the atom events so I tackle it.
I am biased in this case as I contributed before with Lighttable and I like it a lot ... so your call ;)
Let me know if you get stuck at some point with the ink console because it is not fully documented so it can get tricky.
Hope it helps :)

This was already merged so I'm closing this PR.


net: add IP.IsPrivate

Adds a new method IsPrivate to check if an IP is
private according to RFC 1918 & RFC 4193.
Fixes #29146

creddits to @MaxSem as I used your pull (#30278) as template :)
I would mention you in the comment but it does not look like souch information is wantend in the commit history so I leave it out...

@googlebot I signed it!

Any progress on the fantastic feature??

@bl-ue feature-freeze will go into 1.17

@bl-ue feature-freeze will go into 1.17

Thank you @6543!  I love this feature and can't wait for 1.17.

Thank you @6543!  I love this feature and can't wait for 1.17.

@6543 please take a look at, and address the feedback that I provided, then we’ll be ready to go. Thank you.

just had no time to allocate it to this :)
give me few minutes :)

@odeke-em hope, I got it right this time :rocket:

@6543 you got it, congrats and thank you!

🎊 🎉 🚀


[HUE-7919] oozie error 'NoneType' object has no attribute 'is_superuser'

Here are my reappearance steps:

create a shell or spark workflow ,(type as Pig/Sqoop/Dist/Shell/MapReduce/Java/Spark , also met issues)
create a schedule of this workflow.
submit schedule
would get HUE-7919 problem.

In Class Coordinator of hue/apps/oozie/src/oozie/models2.py ,
@property
def workflow(self)

initialize a Workflow without 'user' parameter, then the workflow.user=None, when passing to check can_read/write_or_exception(), would  raise  'NoneType' object has no attribute 'is_superuser'
--
ps: i'm not sure its exactly correct

Thanks!


Blog page

I'd like to develop the blog page, if there are any designs share it, and can you assign me for this

Hey, thanks for taking the decision to apply, I am thinking of desitning the blogging app like below :

Ok @ItsRoy69 , I'll develop the react page and raise PR as soon as possible. Could you give more info on the contents of the page or just random texts?

Keep lorem ipsum for now, thinking of making an admin section in future


Custom QueryString parser support

Bug Report
Hey Samchon,
we are having issue processing extended querystring using @TypedQuery decorator since you are using URLSearchParams for parsing querystring
Summary
When sending complex querystring like foo[0][type]=a&foo[0][value]=foo&foo[1][type]=b&foo[1][value]=bar nestia is unable to parse it using @TypedQuery decorator
Write detailed description in here.
In src/decorators/TypedQuery.ts you are parsing request.query using URLSearchParams however it does not support extended query strings.
I suggest changing a functionality to using already parsed QueryString from request.query if already parsed by middleware querystring parser and then change the validation logic accordingly to accept object instead of URLSearchParams instance
const params: object = request.query && typeof request.query === 'object' ? request.query : new URLSearchParams(tail(request.url)).searchParams;

I'm also having the same issue.
Would be nice getting this addressed.

If you write a query string "person.address.id=2", it means obj["person.address.id"] = 2 in the query string spec.
Also, URL path has length limit in most browsers, and nested query is dangerous in the length reason.

Instead, I recommend to use request body of POST/PATCH/PUT/DELETE methods.
The request body does not have any length and depth limit.

The main issue is that if somebody is already using its own querystring parser that works in nestjs, it stops working once nestia is installed since you are not using already parsed query value that Express/Fastify framework already handled.

In that case, it would better to use @Query() of original NestJS.
Also, in that case, how serialize the query string in the frontend application?
I need to update the SDK generator only for @Query() decorator case.

PHP and similar languages are supporting nested associative arrays natively when parsing querystring therefore quite a lot of people used nested objects - NodeJS ecosystem offloaded support to 3rd party libraries like qs.
Origins of our problem started few weeks ago when swagger client library was updated and it started generating querystring using indexed values for arrays instead of comma separated values and backend started throwing exceptions for those queries - therefore we started using qs library for parsing querystring but it was ignored by nestia.
qs.stringify({ a: ['b', 'c'] }, { arrayFormat: 'indices' })
// 'a[0]=b&a[1]=c'
qs.stringify({ a: ['b', 'c'] }, { arrayFormat: 'brackets' })
// 'a[]=b&a[]=c'
qs.stringify({ a: ['b', 'c'] }, { arrayFormat: 'repeat' })
// 'a=b&a=c'
qs.stringify({ a: ['b', 'c'] }, { arrayFormat: 'comma' })
// 'a=b,c'
qs.stringify({ a: { b: { c: 'd', e: 'f' } } });
// 'a[b][c]=d&a[b][e]=f'

If it helps i might have a look and create a merge request

Oh my god, looking at the format option, it seems terrible and why could not be standard.
Have you to use it? It seems not easy to support from SDK generator.

We have been using deep nested object serialization for past 20 years, since my backend origins were PHP.
Nonetheless swagger does offer deepObject serialization type and it is the only way to make it generic and universal enough so every backend can parse it

There is an option native to expressJS for parsing nested objects
const app = express();
app.set('query parser', 'extended');

I will study how native nestjs (platform of both express and fastify), and will determine how.


Unreadable Docs - Using Helm Section

Issue:
In the main docs, the section Using Helm has styles that are completely broken. The text is shifted to the right and it gets hidden under the left hand side panel.
Here are the pictures of how it looks with different browsers
Hardware:

macOS High Sierra Version 10.13.4

Google Chrome Version 67.0.3396.99

Safari Version 11.1

Firefox Quantum 61.0

Oh interesting! I managed to hit this after clicking a few links on the site too.
cc @flynnduism, any insight into this particular issue?

I've rebuilt the docs, the layout is corrected now: https://docs.helm.sh/using_helm/
This layout bug is a regression, I'm investigating it to determine why the previous build went awry build.

Thanks! @bacongobbler @flynnduism should I close this issue?

@sebasalvarado Looks like it's only partly fixed. Most anchors in the menu don't work and if you scroll down to the bottom, the layout is still screwed up. Looks like a part of the page is appended again to the end of the document.

@maetthu @sebasalvarado I've fixed and pushed a fix to the site: https://docs.helm.sh/using_helm/
The glitch arising from a fresh site build happened again this morning; I'm working on the CI script today to resolve this and to stop it occurring. Apologies for the inconvenience.

The sidebar doesn't scroll (especially command reference). I'm on Linux Mint, with Chrome (also tried Opera).

Same ^^ the sidebar does not scroll, any ideas? @bacongobbler

I've opened https://github.com/helm/helm-www/issues/130. Please feel free to follow up there/contribute fixes!

Thanks!


CHG: support -s option for console

Allow bundle exec cap proction rails:console sandbox=1.
'sandbox=' or 's=' are also OK.

Thanks - released the new version.


[Docs] Dark mode for meshery documentation

Current State:
Right now the documentation for meshery is available in light mode only

Desired State:
An option for a dark theme in the documentation would be more useful and practical as most developers love dark mode.

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

If approved, I would like to work on this issue.

Hey @Kanika637, It's great that you are interested in this, We really need a dark mode, But yes, This issue can take much time to tackle and need focus as we can have merge conflict day by day, hence not a beginner-friendly issue, We have many contributors who assign themself issue but does not stand with that, If you are really into this we can go ahead, I am the one who created a dark theme for layer 5, so definitely, I would love to help you here. Now, will you take this issue?

Surely I would love to work on this @Harshit-Dandriyal @eeshaanSA ✌

Great!! 🎉

@Kanika637 @eeshaanSA can i help out in this feature too?

Sorry, @Kanika637 @eeshaanSA This plan for a dark theme is a little delayed from our current focus, I would love to invite you when we will start working on this..

Hey! can I work on this?


Connecting to server or DB in Azure view does not allow navigating to the Manage view. Must connect via Server connections

Azure Data Studio Version: The very latest insider build. I honestly can't find how to check the version number, but I updated to the latest only this morning. (I suggest providing instructions for how to check version number in the above instructions.)

Steps to Reproduce:

Open Azure Data Studio - Insiders
Connect to Azure account
In Connections viewlet, expand Azure section in object explorer
Expand my subscription and select my server
Enter connection information for my server (I used AAD)
Once connected to server, right click on server and click 'Manage'
Nothing happens...
Right click on server and click Connect
Enter same connection credentials
Once connected to server a second time (it appears in the Servers section of object explorer now), right click on server (in Azure section) and click Manage
Manage view for server appears on right side

I encountered this same exact issue for a specific SQL database in an Azure SQL server. Connecting in the Azure view did not allow me to open the Manage view for the database. I had to go to the Servers section and open a new connection for that database before I could enter the Manage view for the database.

To provide more info, I am having this same issue for all of the right click operations/actions that I try to execute in the Azure view versus the Servers view. All of the options in the right click menu in the Azure view for a database don't work for me. Info for each action I've tried below:

Manage - works in Server view, does nothing from Azure view
New query or New notebook - works in Server view, does nothing from Azure view
Backup or Restore - helpful info message given from Management screen (on right side), and unhelpful error message given from Azure view. (screenshot attached)

@ninarn we're in the process of redesigning the Azure view experience.  That refactoring should address this type of issue.

This problem should be fixed.


Repatriate changelog updates from release/v2.4 to master

Cherry-pick 091be77 to pull forward changelog updates for v2.4 back into master.
Idk if it makes sense to also bring forward 12a0e77. That would certaintly cause a merge conflict and would need to undo changes since V2 doesn't exist on master anymore.

Going to clean up first before re-opening

@haq204 just saw your comment

Idk if it makes sense to also bring forward https://github.com/emissary-ingress/emissary/commit/12a0e77a88750087085ce2390cefb0ad6eea4119. That would certainly cause a merge conflict and would need to undo changes since V2 doesn't exist on master anymore.

I would say any v3 changes made in v2 should get pulled back into the master branch. We would have to then resolve the conflicts and omit the v2 changes.

https://github.com/emissary-ingress/emissary/commit/12a0e77a88750087085ce2390cefb0ad6eea4119 doesn't need to be moved forward because the contents of the change are already in master. When they were backported they need to be changed to match the v2 series due to it supporting multiple Envoy configurations.

Yea this is only because I forgot to update the 2.4 release notes in master originally so this is just to make sure the changelog between master and release/v2.4 are in sync.


fix: limit the max amount of concurrent network requests

Without limiting the amount of concurrent network requests, installation fails with too many open files error

Is this related to the error on macOS?

Is this related to the error on macOS?

This is one of the errors on macOS. The other one is failed to lookup address information: nodename nor servname provided, or not known. I don't have a fix for that one yet.
Also, this PR is not ready yet. Looks like this change is causing a deadlock.

Maybe related? https://superuser.com/questions/433746/is-there-a-fix-for-the-too-many-open-files-in-system-error-on-os-x-10-7-1

Looks like this change is causing a deadlock.

I don't get deadlock on my machine. What is your package.json?

I am using this package.json

The error is not unique to macOS, it seems.

This is a different one. I also get this one. pnpm solves this by using graceful-fs, which retries the file system operations on these type of errors.

Looks like this change is causing a deadlock.

I am using this package.json

After updating the package.json and switching branch to network-concurrency (this branch), what I get is not a deadlock, but instead (it seems) am infinite loop that logs endlessly.
You may try it with TRACE='pacquet::install,pacquet::download' to see for yourself.

If I set the semaphore number to 1000, this error appears:
Error:   × Main thread panicked.
  ├─▶ at crates/cli/src/package_import.rs:38:26
  ╰─▶ expected no write errors: CreateLink { from: "/home/khai/Temp/test-pacquet/pacquet-home/store/7c/a8bebf69ba6047dd7be009132ae6a30992062657634c1573fa04f35cb9de7b31d4316606d61f748300eddc0cc91220ccaa8562814d430d218a6bd0a13471e2", to:
<EMAIL_ADDRESS>error: Os { code: 24, kind: Uncategorized, message: "Too many open files" } }

But if I don't, the install process never stops.

I also created a more "polished" version here: https://github.com/pnpm/pacquet/commit/df491eada17de7d4022f3428ec2fac331982cb4b
Same result.

I think the issue might be that pacquet doesn't support circular dependencies yet.

I think the issue might be that pacquet doesn't support circular dependencies yet.

It would helps a lot if you can reduce your package.json to only circular dependencies (for testing and implementing purposes).

As I have mentioned in https://github.com/pnpm/pacquet/issues/124, there are tests in pnpm related to circular dependencies. And there are small mock packages for testing: https://github.com/pnpm/registry-mock/tree/main/packages/circular-deps-1-of-2


Support for GitHub module from platform

Needs changes to be able to add dependency injection from the platform and use Github module in the platform.

Needs to be released before releasing this feature:
https://github.com/UffizziCloud/uffizzi_app/issues/127

For QA:
This task shouldn't break the existing flow. Please test from CLI.

Add a compose file that specifies a build (like this)
Should get an error that the 'build' option is not supported

Checked - OK

Compose file

services:
  hello-world:
    build:
      context: https://github.com/moklidia/hello-world-1
      dockerfile: Dockerfile
  nginx:
    image: nginx

x-uffizzi:
  ingress:
    service: hello-world
    port: 80
  continuous_previews:
    deploy_preview_when_pull_request_is_opened: true
    delete_preview_when_pull_request_is_closed: true
    delete_preview_when_image_tag_is_updated: true
    deploy_preview_when_image_tag_is_created: true
    share_to_github: true

Ran bundle exec uffizzi project compose set --file 'test/compose_files/test_compose_success.yml'

Result: 'build' option is not implemented


feat(button): add md-icon-button styling.

Added md-icon-button component / styling
Showcased the md-icon-button in the button demos.

I noticed that the icon-button is like a clone of a min-fab, maybe we can make the fab mixin more generic, to let it use by the icon-button too.
Fixes #188

cc @kara

LGTM

I wonder why this isn't included in the documentation examples :(

@matheo it will be soon. Here is the example https://github.com/angular/material2/pull/5628

@willshowell awesome! thanks


Numlock on Startup half working

Describe the bug
Numlock gets enabled in sddm but gets disabled as soon as login into KDE Plasma (works for DWM though!).
To Reproduce
Steps to reproduce the behavior:
0. System: Debian (Testing), sddm & KDE X11

Run LinUtil "NumLock on Startup"
Reboot
Login to KDE
See Numlock getting disabled.

Expected behavior
It should stay enabled when get to the Desktop.
Screenshots
Not applicable.
Additional context
Not needed.
Checklist

[x] I checked for duplicate issues.
[x] I checked already existing discussions.
[ ] This issue is not included in the roadmap.
[x] This issue is present on both stable and development branches.

Some DE have their own num lock configuration in settings. For KDE you need to enable it in the settings. Go to System Settings > Input Devices > Keyboard, in the Hardware tab, in the NumLock on Plasma Startup section, choose the desired NumLock behavior

/close
solved

OK, Thanks for the Explanation! I would only suggest changing the description of NumLock to really highlight this. (Which it already does, but without this knowledge you don't really think it only works on boot.)


Introduce integration test for checking environment setup

requirements to be defined by @mierzwid

is no longer a part of GAP


Deprecating IntelliWait in favor of Polly

Issue(s) closed
#26
Description
Obsoleting IntelliWait in favor of Polly.

@Keboo  I am not authorized to merge this PR. :)


Heavy flickering: Notes become inaccessible after rotating the device

See video attached.
The last frame seems insightful: Although the note opened is from February, the date shown below the note says "just now". The word count is obviously moving either from or to zero, and there are no labels at all, though the folder the note is stored in has some, see first frame. This all looks as if rotating the device creates a new folder & article and inserts the previous text and undos and redos etc. The whole flickering can only be stopped by swiping Noto out of the app switcher.
This happens only in notes' edit mode, BTW, not in preview nor in lists or other places. Also, I could not witness that effect on my old Android 6 Motorola G4, same Noto version.
Noto v2.1.3, Android 10 (Fairphone 3), Webview v102.0.5005.78
https://user-images.githubusercontent.com/35634829/180997539-6a38a4ef-16c8-4f59-a743-b26170a33a10.mp4

This bug happens when changing the system theme too. It is kinda a known bug, and should've been fixed completely in version 2.1.3. However, I guess it isn't the case yet.
Thanks for reporting it!

This should be fixed in 2.1.6.

Great, Fixed it, Thanks

Fixed, here too, for rotation as the trigger. Thanks! See also https://github.com/alialbaali/Noto/issues/109#issuecomment-1207271022 for another trigger. I'll report back.

Oh, I didn't see that comment. Does it happen with 2.1.6? I think it should be fixed too.

I'll keep my eyes open. There might be more preconditions (note opened in foreground / background / whatnot) so might take some days to be sure.

Any update on this @uli-on?

Ah, didn't encounter flickering any longer. Can be closed, but I can't do it, I'm logged in with Octodroid, which has no such option yet.

Great to know.


Build opentitan bitstream

This PR enables building ibex and opentitan bitstream using vivado and uploads bitstream to artifacts.
It also reverts opentitan to last know working version (with working bitstream on HW) and prepares to do automatic bisect in CI.
Signed-off-by: Kamil Rakoczy<EMAIL_ADDRESS>

Failing tests/CastInFunctionToParameterWidth test is random network problem with ubuntu servers and it is not related to this PR.


Broken Cache Logic, as hash of string is not unique

Hello, we're using Sourcery on our project (dozens of difficult templates), so thank you for your hard job
Currently our all generations took up 80 seconds and I'm trying to find how to improve it, checking cache I found that it sometimes it doesn't work for the same 1-5 files in each module.
After research I found that Sourcery use 'hash' for strings, but hash logic for strings is not unique.
(I replaced NDA words with XXXXX)
E.g.
let line1 = "/Users/pavel.trafimuk/Developer/Viber_iOS/Classes/Assembly/Modules/ViberXXXXManager/Dependencies/LastXXXXXXNotificationProcessorDependency.swift"
let line2 = "/Users/pavel.trafimuk/Developer/Viber_iOS/Classes/Assembly/Modules/ViberXXXXManager/Dependencies/NewXXXXXXeNotificationProcessorDependency.swift"
In both cases we receive<PHONE_NUMBER>062482058
In results Sourcery doesn't find valid cache (unarchived file shows another modification date) and replace it with another classes. On the next run will be the same

hey, where do you think it use's string hash for cache invalidation? we use modified date or sha256 unless I missed some pr change that would do something else

Sorry, forgot to insert the code
        let artifactsPath = cachesPath + "\(path.string.hash).srf"
As I understand that you use .hash instead of .hashValue to have the same values during multiple running of the app. But seems like .hash uses a small parts of the string to calculate itself

I guess we can just replace that with sha256 as well

sha256 of filename or of content? I'm afraid about performance)

the bug you are talking about is the filename just, we don't do hashing on contents because modified date is faster and more convenient, if you can run local sourcery build on your codebase and see if it fixes your issue that would be great, you could send the pr with that line change too if that works

as quick solution I use:
extension String {     var uniqueHashValue: String {         stride(from: 0, to: count, by: 40).map {             let start = index(startIndex, offsetBy: $0)             let end = index(start, offsetBy: 40, limitedBy: endIndex) ?? endIndex             let substing = String(self[start..<end])             return "\(substing.hash)"         }.joined(separator: "-")     } }
I know that it doesn't look great, just reduce collision)
BTW, quick question, I assumed that if template was not modified, source files were not modified too -> rendering is obsolete and we can skip it. What do you think?

let artifactsPath = cachesPath + "\(path.string.sha256()?.addingPercentEncoding(withAllowedCharacters: .alphanumerics) ?? "\(path.string.hash)").srf"
helped, problem was gone, thanks! will prepare PR


Non-thunked Ghosts

I cannot remove the dummy unit argument in:
https://github.com/FStarLang/FStar/blob/cd7474186ad473038d59ac09968f16ba3cccc8e5/examples/extraction/ghostTest.fst#L20 (this is the same example as the one in #295 )
I wanted to write it as follows:
val aSizedList  : sizedList
let aSizedList = let h2 = (hide 2) in MkSList h2 [1]
Ghosts have no effects. So, why is the additional unit is needed?
For example, it was not needed here:
https://github.com/FStarLang/FStar/blob/cd7474186ad473038d59ac09968f16ba3cccc8e5/examples/extraction/ghostTest.fst#L14 (just 6 lines above)

Using computationally irrelevant terms is an effect.
Consider:
val x : erased nat

val y : nat
let y = reveal x

This program is not sound. The definition of y has a ghost effect that is incorrectly forgotten. How would we compile y, given that x is erased to a unit?
Note, you cannot write:
val y: GTot nat

Computation types like GTot nat are not types.
Hence, you must write:
val y : unit -> GTot nat
let y () = reveal x

I agree. However, I have a related confusion.
How does F* determine that the following is not ok
val aSizedList : sizedList
but, the following is ok:
val aSizedListNG :  sizedListNonGhost
Do types carry some kind of a ghostness qualifier? I don't remember any such thing in the F* AST.
More importantly, why is sizedList a ghost type? Because it uses reveal only in a pattern that is only visible to the SMT solver, I think it does not suffer from the issues you mentioned above.
Definitions of sizedList and sizedListNonGhost can be found in:
https://github.com/FStarLang/FStar/blob/cd7474186ad473038d59ac09968f16ba3cccc8e5/examples/extraction/ghostTest.fst

There are no qualifiers on types.
We infer a Ghost effect for the definition. At the top level, we do not allow the Ghost effect.

So, I think it is suboptimal to infer a ghost effect for aSizedList:
type sizedList =
| MkSList: maxsize:(ghost nat)->  cont:(list int){length cont < (reveal maxsize)} -> sizedList

It boils down to the type of hide. As discussed, this will now be in Tot.


Publish Temporalite to a public container registry

Is your feature request related to a problem? Please describe.
Users should be able to quickly spin up Temporalite instances in container environments like Kubernetes, docker-compose, and various CI providers.
Describe the solution you'd like
Configure goreleaser to build & push container images for Temporalite to a temporalio owned container repository.
Describe alternatives you've considered
Docker compose users can already leverage Temporalite by building from source, eg.
# docker-compose.yaml
version: '3.9'

services:
  temporalite:
    build: https://github.com/temporalio/temporalite.git#main
    ports:
      - 7233:7233
      - 8233:8233

Additional context

I am building a multi-arch temporalite image here https://github.com/slamdev/temporalite-container-image
Let me know if you are open for contribution of this solution to temporalite repo.

Is there any chance of an official docker image being pushed to Docker Hub please?

In the meantime, until an official temporalite docker image is available for local dev, I use the following Dockerfile as an alternative.
It is fast to build (just download latest temporal CLI, no need to build go modules), and starts temporalite via the new temporal CLI.
FROM curlimages/curl as builder

WORKDIR /
RUN curl -sSf https://temporal.download/cli.sh | sh

FROM gcr.io/distroless/base-debian11

COPY --from=builder /home/curl_user/.temporalio/bin/temporal /bin/temporal
EXPOSE 7233
EXPOSE 8233

ENTRYPOINT ["temporal", "server", "start-dev", "--ip" , "<IP_ADDRESS>"]

this would be super helpful!

Also waiting for official image to use in testcontainers library 🙏


Block access after 10 wrong login attempts

The login should be blocked after 10 wrong login-attempts for at least 10 minutes.

A value blocked should be added to lychee_settings. 0 when unblocked and filled with a timestamp when blocked.
Add to the settings.md and faq.ms how to unblock
Add error to the front-end

I disagree with this being added as a feature of this software. What is required from this software's end, is only to generate clear log events to the system log indicating successful and failed login attempts. It should be up to the system to monitor the logs and control access, using something to the effect of fail2ban.

This feature won't be part of Lychee. Lychee will only log failed log in attempts as suggested by @lbdroid and @qligier: #382.


$connection->quote() converts JSON into String

I am trying to use executeUpdate() on PostgreSQL 9.6 with Doctrine 2.6 with Symfony3 on a jsonb column.
The problem is that when I use $connection->quote() function, with Type::JSON it seems to convert whole JSON to a string. Documentation is not very clear on what it should be doing .  I understood from documentation that it should result in valid JSON, no? is this a bug?
$connection = $this->_em->getConnection();
$connection->quote('{"name": "product"}', Type::JSON);

Results:
"{\"name\": \"product\"}"

Pass in ['name'=>'product']
On Mon, 26 Mar 2018, 18:42 yurtesen<EMAIL_ADDRESS>wrote:

I am trying to use executeUpdate() on PostgreSQL 9.6 with Doctrine 2.6
with Symfony3 on a jsonb column.
The problem is that when I use $connection->quote() function, with
Type::JSON it seems to convert whole JSON to a string. Documentation is
not very clear on what it should be doing
http://docs.doctrine-project.org/projects/doctrine-dbal/en/latest/reference/data-retrieval-and-manipulation.html#quote.
I understood from documentation that it should result in valid JSON, no? is
this a bug?
$connection = $this->_em->getConnection();
$connection->quote('{"name": "product"}', Type::JSON);
Results:
"{"name": "product"}"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/doctrine2/issues/7159, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakCq7MYcT1raTmEPycCa64d4mjVTMks5tiRp0gaJpZM4S7dsk
.

But that is not JSON? Isn't the type should represent the input type?
I tried this print $connection->quote("['name'=>'product']",Type::STRING); which prints '[''name''=>''product'']'

I wanted to use parameter binding but I am using jsonb_set in PostgreSQL and trying to build the query ends up with an error Error: Expected known function, got 'jsonb_set'" at
If I use native query then doctrine does not recognize parameters. I have a query like this UPDATE cart c SET data = jsonb_set( coalesce(data,'{"product": {}}'), '{"product",":id"}','{"name":":name"}') WHERE cart.id = :cart'; it does not recognize :id and :name are parameters, only the :cart parameter is replaced in final query.
So as a last resort, I had to go totally low level. Now I figured out this works with json_decode()
$connection->quote(json_decode('{"name": "product"}'), Type::JSON);
Not sure if this improve security (eg. against sql injection) at all in cases where name "product" is given online by users?
Also shouldn't the quote() function work with JSON as string?

Quoting will do the type conversion. A string and a hashmap are very different things, so you need to pass in the correct data structure to be quoted as a value.

I didn't quote understand exactly what you mean. I was also not sure of what you meant earlier when you said "you are using quoting". You mean the quote() function?
I am giving in a standard json object. Isn't a json object technically a string and also consist of key/value pairs so a hash map?
Let me ask it in a more direct way. I feel like I am doing something wrong. Lets say I have the json object { "name":"John", "age":30, "car":null } and I am about to assign it to a jsonb field. What is the correct way to pass this to $connection->quote() function for quoting?

A string is a string, a key/value map is a key/value map. Doctrine converts what you have in memory. If what you have in memory is a string, you will store a JSON string, not a JSON object.

Sorry, but that really did not make things clear at all :( also did not quite answer my question.
The definition of JSON. -> JavaScript Object Notation (JSON) is a lightweight, text-based, language-independent data interchange format <- notice text-based, if it is not text, it is not JSON.
Can you give an example to a JSON object, with PHP code, which is NOT a string? Keep in mind, it is not a JSON object if it is a PHP object or associative array. I am trying to understand what you mean, perhaps the most direct way is with a code example?

Can you give an example to a JSON object, with PHP code, which is NOT a string?

Yes:
$jsonString = '{"foo":"bar"}'; // this is a string. It might look like json, but it is a string
$jsonObject = json_decode($jsonString); // this an actual object
var_dump(gettype($jsonString));
var_dump(gettype($jsonObject));

This is the difference here. If you pass around a string, PHP (and DBAL) will interpret it as a string. If you pass around an object (or anything compatible with json_serialize()) then you will get a JSON-object saved in your DB value.
Doctrine DBAL performs a json_encode() on its own.

@Ocramius your $jsonObject is a PHP object. json_decode() manual Takes a JSON encoded string and converts it into a PHP variable. also says Returns the value encoded in json in appropriate PHP type..
PHP converts JSON Object into PHP stdClass() object. So your $jsonObject variable has nothing to do with JSON.  Yes, your code prints out object as type, but it is the PHP object type. Same result will be from:
$stdObject = new stdClass();
var_dump(gettype($stdObject));

The $stdObject is also NOT a JSON object.
When I pass the JSON encoded string to quote() I clearly tell it that it is JSON $connection->quote('{"name": "product"}', Type::JSON); yet, it screws it up. Actually, on the contrary, if I tell it is Type::STRING then it actually returns '{"product","name"}' which is correct. I would say this is a bug.
Also from Doctrine json type manual entry.  If you know that the data to be stored always is in a valid UTF-8 encoded JSON format string, you should consider using this type. Well my data is a valid JSON string!?
Anyway, thanks for your time. But I won't dwell on this further...time constraints... :(

@yurtesen I've already explained it in the best way I can. Strings are strings: objects are objects. A json string is still a string: trying to encode it as json leads to double-encoding.


⚠ ETF has degraded performance

In 471d8f8, ETF (https://markets.hankyung.com/equity-product/etf) experienced degraded performance:

HTTP code: 200
Response time: 6079 ms

Resolved: 하락종목 performance has improved in cefa6ad.


[bug] Unexpected Comportament in Note

I'm using the same template of your README file, and when I link the currently note in the app, it takes control of the scroll movement, you can see that even when I openned the command menu it was still moving.

When I disabled the Mermaid Gantt it worked "just fine".

Same happened to me. I had to disable Mermaid Gantt and also disable Auto check options, since is not a great experience to use it so

Same happened to me. I had to disable Mermaid Gantt and also disable Auto check options, since is not a great experience to use it so

Yeah it revert to normality when I disabeled Mermaid Gantt here too

Should be fixed in 0.7.0.


fix: Make Git parser handle meta lines in chunk

Fixes #560

Looks like there's a merge conflict. I'll rebase

Just realized this includes some nonsense changes to the Cargo.toml that appear to be holdovers from the rebase. I'll remove them

@mchernicoff can I get a re-review? Just updated to no longer include the mistaken Cargo.toml changes.


Rails 6.1 support

Fixes #977
This PR is basic Rails 6.1 support. There were a couple of changes necessary.
First, instead of requiring all Rails frameworks in the dummy app, only the necessary frameworks are required. This is because Rails started raising some errors when it couldn't locate config for Active Storage, which Camaleon doesn't use anyway.
Second, there is a change to one of the model associations in post_type.rb. Apparently the :source option is only valid if there is also a :through option, and Rails 6.1 raises an exception on what was previously a no-op. I removed the option (since it apparently didn't do anything anyway), but I wonder if maybe that relationship should be scoped? Please check this part carefully before merging. See the issue linked above for more info.

Update: I've been running this PR in production without issue for a few days now without issue. The change to the PostType model doesn't appear to have broken any draft functions.

Super!!! @brian-kephart


Added unit tests in pkg/es/config

Which problem is this PR solving?
partially fixes: #5068
Description of the changes

This commit adds tests for the pkg/es/config package.

How was this change tested?

make test

Checklist

[x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[x] I have signed all commits
[x] I have added unit tests for the new functionality
[x] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: yarn lint and yarn test

Nit: avoid using description like partially fixes: #5068 - github reads it as fixes #5068 and closes the corresponding issue. Instead you can say "Part of: #5068`

looks like your new tests are leaking goroutines. Make sure you always close ES clients properly.

looks like your new tests are leaking goroutines. Make sure you always close ES clients properly.

done now
sorry didn't close config.TLS D:
but it's closed and all running play now

Thanks!


support qwen2.5 models

blog: https://qwenlm.github.io/blog/qwen2.5/
github: https://github.com/QwenLM/Qwen2.5

As more and more new models enter the market, we have prepared comprehensive instructions for TRT-LLM developers on adapting to new models of interest. We encourage our community developers to expand the range of supported models, fostering an open ecosystem with rapid iterations.
Please try following these instructions and let us know if you encounter any issues during the adaptation process. We greatly appreciate your dedication.

Has anyone been able to successfully compile qwen2.5 for tensorrt?
It's by far the best opensource model we've tested, would love to see it working on tensorrt.

Qwen 2.5 requires developers to compile it themselves？ as I see that the Qwen 2.5 series models are not supported in the list of supported Qwen models.

@anonymousleek @a2382625920
qwen2.5's layer structures and configs are almost identical to qwen2 except for some special tokens.
I built it with qwen2's instructions, and it works.
I recommend you to build the model within the nvidia container that matches your setup. since different tensorrt-llm versions require different dependencies from nvidia.


Can't get RTM connection from within kubernetes

I'm trying to get a connection from an RPI kubernetes cluster and can't. With debugging enabled I get to this point:
nlopes/slack2017/12/24 09:21:15 slack.go:92: Using URL: wss://mpmulti-s314.lb.slack-msgs.com/websocket/<b64 yadda yadda>

And it stops there. I'm seeing some retransmissions at the TCP level but that's it.
Things that do work:

Run the same container directly with docker on the same host.
Run a pure amd64 linux container on the same network.

I'm mostly opening this issue as a tracker to debug the problem. My approach will be to throw a lot of logging at this to understand where the problem is coming from. It all points to (flannel) networking, but it's simply a black box, even with debugging enabled.
I will be thankful for any pointer to debug this.

Ok, I just submitted a PR to improve visibility and to enable websockets to fail with a context timeout so they can retry.
This particular problem seems to be a networking one as I can't get a TLS handshake done with the Slack url through curl (it just locks up), so I doubt that this problem is about this library in particular.
Thus, I'm closing this issue as non-actionable.


SSH Key Login?

There doesn't seem to be any way to authenticate with an SSH key rather than username/password.

SSH keys are only for pushing and pulling from git repos, you can't sign in to GitHub with an SSH key.


[1.x] Added snapshot check for build.gradle

Description
Added snapshot check for build.gradle
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Lets create 0.1 branch. and bump 1.x to 0.2.


Improvements to GCloudVolume and lib.py

Just Gcloud stuff, I haven't added new Storage support yet

If you make the title slightly shorter it would be easier to read.
fix(GCloudVolume):  fixes handling of offsets and limits.

Info files can be created for new volumes.

Look at https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
specifically :
Any line of the commit message cannot be longer 100 characters! This allows the message to be easier to read on GitHub as well as in various git tools.

Explaining what CLOUD_COMPUTING is used for would also be useful.

create_new_info looks pretty useful, I'm wondering how it can combined with https://github.com/seung-lab/neuroglancer/blob/master/python/neuroglancer/pipeline/task_creation.py#L101

@tartavull w.r.t. create_new_info, I've already adjusted that function in submit_tasks. I saw elsewhere that you were talking about making some kind of Info class. That could be a very good idea.


[FEATURE] Set neural-search plugin 3.0.0 baseline JDK version to JDK-21

Is your feature request related to a problem? Please describe
Set neural-search plugin 3.0.0 baseline JDK version to JDK-21
Describe the solution you'd like
See please https://github.com/opensearch-project/OpenSearch/issues/10745
Related component
Build
Describe alternatives you've considered
N/A
Additional context
See please https://github.com/opensearch-project/OpenSearch/issues/14011

[Catch All Triage - Attendees 1, 2, 3, 4, 5]

following PRs related to adding JDK21, they were done before/separately from this campaign:
https://github.com/opensearch-project/neural-search/pull/565
https://github.com/opensearch-project/neural-search/pull/801
https://github.com/opensearch-project/neural-search/pull/837
https://github.com/opensearch-project/neural-search/pull/801

@reta is there anything left for neural search plugin in scope of this campaign? We have done multiple small PRs that added jdk21 to different plugin areas, list is here https://github.com/opensearch-project/neural-search/issues/775#issuecomment-2239551341.

@reta is there anything left for neural search plugin in scope of this campaign? We have done multiple small PRs that added jdk21 to different plugin areas, list is here #775 (comment).

Yes, we need to update baseline here https://github.com/opensearch-project/neural-search/blob/main/build.gradle#L158
allprojects {
    ...
    targetCompatibility = JavaVersion.VERSION_21
    sourceCompatibility = JavaVersion.VERSION_21
    ...


blender exporter addition

I'm starting to update blender exporter documentation.
Images should now be placed in the right folder, thanks Deltakosh ;)

This is so cool! Thanks a lot


ci: avoid runs if rs files weren't changed

ci: avoid runs if rs files weren't changed
We don't need to run linting tasks if rust files are same with HEAD.
GithHub Actions provide ability to skip task by conditions. Was added
check that there are changed rust files. If check returns True we
execute linting and others workflows, otherwise, GitHub Actions
should skip them.
Close: pyrsia/pyrsia#479
Runs links with example
when we have to skip jobs: this
when job should be triggered: that
PR Checklist

[x] I've read the contributing guidelines.
[x] I've read "What is a Good PR?"
[x] I've included a good title and brief description along with how to review them.
[x] I've linked any associated an issue.
[X] I've requested a review  from pyrsia/collaborators.

Code Contributions

I don't do next, because it is workflow changes

[ ] I've built the code cargo build --all-targets successfully.
[ ] I've run the unit tests cargo test --workspace and everything passes.

And I need some help to understand with what branches shoud we compare current. Is it good approach for your team?
Any other points?
Are these jobs chosen correctly?

 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.

How can I add the pyrsia/collaborators to review?

How can I add the pyrsia/collaborators to review?

Do you have pyrsia/collaborators in the list when you select the Gear icon by the reviewers?

How can I add the pyrsia/collaborators to review?

Do you have pyrsia/collaborators in the list when you select the Gear icon by the reviewers?

I see only next

It seems I can't add reviewers

Yes, I think the issue is at our github org level.  I reached out to the org admins to check out the settings.  Keep a look out for a github invite.

@VitaliyaIoffe I added them manually for you right now. But we will fix it in the permissions so that you can add them in the future.
Thanks for your interest and PR!

Example when we skip jons: https://github.com/VitaliyaIoffe/pyrsia/actions/runs/2105219557
Else (I used yml pattern): https://github.com/VitaliyaIoffe/pyrsia/actions/runs/2105159416

@VitaliyaIoffe your changes look good.  Can you please remove the "Draft" from the PR?  Also, make sure the CLA is signed.

@VitaliyaIoffe your changes look good. Can you please remove the "Draft" from the PR? Also, make sure the CLA is signed.

done and done

Am I right that I also should add the same change for coverage job from rust.yml?

@erwin1 is off so @tiainen picked up this assigned PR (lots of vacations)


MTU issues seen between an OnPrem cluster vs Public cloud

While running e2e tests in an OCP Cluster where one of the cluster is on OpenStack (OnPrem) and the other is on Public cloud (AWS), it was seen that the initial connection goes through, but there are errors during data transmission.
      Expected
          <string>: listening on <IP_ADDRESS>:1234 ...
          connect to <IP_ADDRESS>:1234 from <IP_ADDRESS>:46183 (<IP_ADDRESS>:46183)

      to contain substring
          <string>: b5539269-607a-4893-bd05-3981b1a7d985

As it can be seen above, the client was able to connect to the service but the data transmission failed. Submariner e2e tests validate if the connection is successfully established and it also verifies if the data transmission includes the unique UUID from the client, hence the e2e tests are failing.
Setup details:
OpenShift version: 4.10.2
Cloud platform: OnPrem (OSP) vs Public Cloud (AWS)
CNI: OpenShift SDN
Submariner Globalnet Enabled
Submariner version: 0.12.0
Additional notes:
So it is clear that its an MTU related issue and on debugging it further, it turns out that its because of iptables vs nft differences.
Submariner MTU driver in route-agent programs iptable rules to clamp tcp mss to PMTU in the Mangle table. Since the underlying platform is running nftables, the automatic translation layer should ideally translate them to corresponding nftable rules, but it appears like this conversion is not happening properly.
IPTable rules programmed on the Gateway node (mangle table):
Chain SUBMARINER-POSTROUTING (1 references)
target     prot opt source               destination
TCPMSS     tcp  --  <IP_ADDRESS>/0            <IP_ADDRESS>/0            match-set SUBMARINER-LOCALCIDRS src match-set SUBMARINER-REMOTECIDRS dst tcp flags:0x06/0x02 TCPMSS clamp to PMTU
TCPMSS     tcp  --  <IP_ADDRESS>/0            <IP_ADDRESS>/0            match-set SUBMARINER-REMOTECIDRS src match-set SUBMARINER-LOCALCIDRS dst tcp flags:0x06/0x02 TCPMSS clamp to PMTU

Translated nft rules dumped using the nft binary (nft -a list table mangle) on the same node:
chain SUBMARINER-POSTROUTING {
        meta l4proto tcp # match-set SUBMARINER-LOCALCIDRS src # match-set SUBMARINER-REMOTECIDRS dst tcp flags & (syn|rst) == syn counter packets 114 bytes 6840 tcp option maxseg size set rt mtu
        meta l4proto tcp # match-set SUBMARINER-REMOTECIDRS src # match-set SUBMARINER-LOCALCIDRS dst tcp flags & (syn|rst) == syn counter packets 42 bytes 2520 tcp option maxseg size set rt mtu
}

If we look at the nft rules carefully, the ....# match-set SUBMARINER-REMOTECIDRS... is commented out as the iptables mtu target/expressions are not supported by the automatic translation. The problem is described further in this issue - https://unix.stackexchange.com/questions/672742/why-mss-clamping-in-iptables-nft-seems-to-take-no-effect-in-nftables

To fix the issue for Globalnet deployments, we require two changes.

To ensure that SUBMARINER_LOCALCIDRS ipset is updated with the local PodCIDRs as SNAT (with globalIP) is performed in NAT Postrouting table which is hit after MANGLE POSTROUTING table.

Currently, the IPsets have only GlobalCIDRs
[root@asuryanarhos2-d5v94-worker-0-tjkr5 submariner]# ipset list
Name: SUBMARINER-REMOTECIDRS
Type: hash:net
Revision: 6
Header: family inet hashsize 1024 maxelem 65536
Size in memory: 440
References: 2
Number of entries: 1
Members:
<IP_ADDRESS>/16

Name: SUBMARINER-LOCALCIDRS
Type: hash:net
Revision: 6
Header: family inet hashsize 1024 maxelem 65536
Size in memory: 504
References: 2
Number of entries: 2
Members:
<IP_ADDRESS>/16

Note: The above change is necessary only for Globalnet deployments.

Identify the implementation used on the node and if its nftables, use the nft binary to program the tcp-mss-clamp rules.

Two more observations about the current implementation :

Current implementation enables PLPMTU for all connections on node, by setting /proc/sys/net/ipv4/tcp_mtu_probing = 2 .

We might have a conflict with another component that managed this file (like MCO in OCP)
We force PLMTUD for all TCP connections originated on this node.

MSS clamping rule configured on GW node, I think that GW node doesn't maintain pmtu for TCP connections originated on non-GW nodes

@mkimuram FYI

I haven't caught up completely yet.  However, I tested on clean installation of RHEL9 beta and confirmed that translation fails for https://github.com/submariner-io/submariner/blob/devel/pkg/routeagent_driver/handlers/mtu/mtuhandler.go#L79-L94 rule.
# dnf install -y iptables-nft
# iptables-translate -V
iptables-translate v1.8.7 (nf_tables)

# modprobe xt_set

# dnf install -y ipset
# ipset create SUBMARINER-LOCALCIDRS  nethash
# ipset add SUBMARINER-LOCALCIDRS <IP_ADDRESS>/16
# ipset create SUBMARINER-REMOTECIDRS  nethash
# ipset add SUBMARINER-REMOTECIDRS <IP_ADDRESS>/16

# iptables-translate -t mangle -A POSTROUTING -m set --match-set SUBMARINER-LOCALCIDRS src -m set --match-set SUBMARINER-REMOTECIDRS dst -p tcp -m tcp --tcp-flags SYN,RST SYN -j TCPMSS --clamp-mss-to-pmtu
nft # -t mangle -A POSTROUTING -m set --match-set SUBMARINER-LOCALCIDRS src -m set --match-set SUBMARINER-REMOTECIDRS dst -p tcp -m tcp --tcp-flags SYN,RST SYN -j TCPMSS --clamp-mss-to-pmtu

On the other hand, https://fossies.org/linux/iptables/extensions/libxt_TCPMSS.txlate seems working in the same environment
# iptables-translate -A FORWARD -p tcp --tcp-flags SYN,RST SYN -j TCPMSS --clamp-mss-to-pmtu
nft add rule ip filter FORWARD tcp flags & (syn|rst) == syn counter tcp option maxseg size set rt mtu

Is it due to failure in translating rules including ipset?
Because,

The results including ipset are commented out:

# iptables-translate -A POSTROUTING -m set --match-set SUBMARINER-LOCALCIDRS src -m set --match-set SUBMARINER-REMOTECIDRS dst -p tcp -m tcp --tcp-flags SYN,RST SYN -j TCPMSS --clamp-mss-to-pmtu
nft # -A POSTROUTING -m set --match-set SUBMARINER-LOCALCIDRS src -m set --match-set SUBMARINER-REMOTECIDRS dst -p tcp -m tcp --tcp-flags SYN,RST SYN -j TCPMSS --clamp-mss-to-pmtu
# iptables-translate -A FORWARD -p tcp -m tcp --tcp-flags SYN,RST SYN -j TCPMSS --clamp-mss-to-pmtu
nft add rule ip filter FORWARD tcp flags & (syn|rst) == syn counter tcp option maxseg size set rt mtu

But the result not including ipset isn't commented out:

# iptables-translate -t mangle -A POSTROUTING -p tcp -m tcp --tcp-flags SYN,RST SYN -j TCPMSS --clamp-mss-to-pmtu
nft add rule ip mangle POSTROUTING tcp flags & (syn|rst) == syn counter tcp option maxseg size set rt mtu

Note that there seem libxt_SET.c and libxt_TCPMSS.c, but not sure they work correctly if used together.
https://git.netfilter.org/iptables/tree/extensions

In my centos8.2 environment, below also succeeds, so I guess the rule with --clamp-mss-to-pmtu could be translated even in RHEL8 (not sure if it is true for all minor releases, though).
# iptables-translate -V
iptables-translate v1.8.4 (nf_tables)
# iptables-translate -t mangle -A POSTROUTING -p tcp -m tcp --tcp-flags SYN,RST SYN -j TCPMSS --clamp-mss-to-pmtu
nft add rule ip mangle POSTROUTING tcp flags & (syn|rst) == syn counter tcp option maxseg size set rt mtu

Is it due to failure in translating rules including ipset?

Thanks @mkimuram for looking into it and sharing your observations. As you know, we use ipsets even in Globalnet, so today in another similar setup, i verified Globalnet use-case that uses ipsets in the rules. I see that Globalnet Egress rules for Namespace (where we use ipsets) are also commented out on the node [iptables v1.8.7 (nf_tables)] when using nft, but the rules are getting hit and working as expected.
Output of nft list ruleset
	chain SM-GN-EGRESS-NS {
		# match-set SM-GN-J3JGDJY5EHAINM244WDEGL64S src mark and 0xc0000 == 0xc0000 counter packets 0 bytes 0 snat to <IP_ADDRESS>
	}

Output from iptables list (you can see the packet count of 4). Our e2e test cases are passing too.
Chain SM-GN-EGRESS-NS (1 references)
num   pkts bytes target     prot opt in     out     source               destination
1        4   240 SNAT       all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0            match-set SM-GN-J3JGDJY5EHAINM244WDEGL64S src mark match 0xc0000/0xc0000 to:<IP_ADDRESS>

In the following link, it is mentioned that Whatever nft displays back here is only for the display and mustn't be taken as the actual rules. Along with this, it also mentions the following
While there's no automatic translation [available](https://manpages.debian.org/nft.8#EXTENSION_HEADER_EXPRESSIONS) currently for -m tcpmss --mss , the feature is available: tcp option maxseg size which can be used either as an expression (the equivalent of the match -m tcpmss --mss) or with set as a statement (the equivalent of the target -j TCPMSS). Below would be the result of such translation (and might be in the future once the translation engine is improved):

I checked (OCP on OSP env) iptables counters with route-agent image including the SUBMARINER_LOCALCIDRS ipset fix [1].
The counters values of the outbound iptables rule seem to be growing when I'm trying to curl IP address from remote cidrs range, though match-set on relevant nft rule is still commented out [2].
Do you think pmtu rule is being applied?
[1]
https://github.com/submariner-io/submariner/pull/1805
[2]
        chain SUBMARINER-POSTROUTING { # handle 21
                meta l4proto tcp # match-set SUBMARINER-LOCALCIDRS src # match-set SUBMARINER-REMOTECIDRS dst tcp flags & (syn|rst) == syn counter packets 74 bytes 4440 tcp option maxseg size set rt mtu # handle 36
                meta l4proto tcp # match-set SUBMARINER-REMOTECIDRS src # match-set SUBMARINER-LOCALCIDRS dst tcp flags & (syn|rst) == syn counter packets 20 bytes 1200 tcp option maxseg size set rt mtu # handle 37
        }
}

The counters values of the outbound iptables rule seem to be growing when I'm trying to curl IP address from remote cidrs range, though match-set on relevant nft rule is still commented out [2].
Do you think pmtu rule is being applied?

When a rule has multiple match expressions and an action, sometimes one of the match expressions might be commented out. The counters are one way for us to know if the rule is getting applied (even partially). Please check if e2e is passing.


docs(PopupBase): docs

Close #110

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


How to produce PDF/UA with LibreOffice

LibreOffice supports the creation of PDF/UA (accessible PDF documents) when the input docx file contains appropriate accessibility information (e.g. alternative text for images and so on).
Is it possible with Gotenberg to pass the appropriate conversion option to LibreOffice?
When exporting from the LibreOffice GUI the relevant options are those highlighted in the picture below. Based on the documentation, I belive that Gotenberg allows requesting a PDF/A output, but does not provide an option for PDF/UA.

It looks like the appropriate export options to pass to LibreOffice are:
EnableTextAccessForAccessibilityTools=true
UseTaggedPDF=true
so until gotenberg provides a way to pass such options, I created a derived docker image that replaces unoconv with a shell script that invokes unoconv and always specifies such options:
FROM gotenberg/gotenberg:7

USER root
RUN mv /usr/bin/unoconv /usr/bin/unoconv2
RUN echo '#!/bin/bash\n/usr/bin/unoconv2 -e EnableTextAccessForAccessibilityTools=true -e UseTaggedPDF=true "$@"' > /usr/bin/unoconv && \
    chmod 755 /usr/bin/unoconv
USER gotenberg

@giamma thanks for the details!

One additional comment: if you need your accessible PDFs to be in PDF/A format, make sure to use the "nativePdfFormat" option (which I believe uses LibreOffice), instead of "pdfFormat" (which I believe uses unoconv) because the latter strips some accessibility information from the PDF document produced by LibreOffice (e.g. alternative text for pictures).


Merge several IR nodes into UnaryOp.

ArrayLength, GetClass, Clone, IdentityHashCode, WrapAsThrowable, UnwrapFromThrowable and Throw are now ops of UnaryOp, rather than having their own IR nodes.
They all follow the general contract of how a UnaryOp must behave: first evaluate the argument, then perform an operation that only depends on the value of the argument (and not, for example, on the scope).
It makes the JS FunctionEmitter a bit messier. However, everything else gets simpler, as we get rid of dedicated paths for those 7 IR nodes.
In order to better fit with the other UnaryOps, notably the ones for jl.Class operations, we demand non-nullable arguments. An explicit CheckNotNull must be used for arguments that are nullable.

In this commit, we do not update the compiler yet. It still emits the old IR nodes, so that we can test the deserialiation hacks. That also means the case classes still exist for now.

FWIW: IMO JSDelete and JSImportCall can benefit from a similar treatment. Did you deliberately decide against these?

JSImportCall will need an additional argument once we had support for those with things. Also IMO it uses the scope. It's the module scope, not a function scope, but still. Its behavior is not only dependent on the argument.
For JSDelete, it's mostly because it would then be the only JS thing that we merge this way. I'd rather keep them separate. Also they have divots behavior with evaluation order, and I don't want to pollute BinaryOp with that weirdness.

(rebased to address conflicts in BinaryIncompatibilities)


422 Client Error: Unprocessable Entity for url: https://api.tavily.com/search

Hey, I just cloned the repo and got it up and running on Mac OS(Ventura 13.4) with Python version 3.9.6
I tried making a request with the words: 'Martin Luther King' and 'super bowl' and got this error:
HTTPError: 422 Client Error: Unprocessable Entity for url: https://api.tavily.com/search
And In the console of the browser, I got this error message:
POST http://localhost:8000/generate_newspaper 500 (INTERNAL SERVER ERROR) produceNewspaper @ scripts.js:36

Same here

It worked here, got out of the corner. I omitted the image parts and it seems to be pulling

Me too

Hi all, can you make sure you entered the API key?

In LangChain, if the parameters are not reasonable, for example, if they are of type 'dict' or too short, a 422 error code will be returned directly. This issue is more likely related to the input parameters rather than a problem with the service.

In my case, the same error occurred when I typed in Japanese, but it worked fine when I typed in English.

i also have this error. Seemed to happen randomly as my code worked well before and now it doesn't and spews this error.

In my case, the same error occurred when I typed in Japanese, but it worked fine when I typed in English.

currently sending out a fix!


Can it possible make a PR merge it into Flashatten?

flash attn has more mutual environments.

Thank you very much for your suggestion. We will consider it and give it a try!


add gcc library

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 45.45%. Comparing base (313eed2) to head (b26081c).
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #22   +/-   ##
=======================================
  Coverage   45.45%   45.45%
=======================================
  Files          13       13
  Lines          44       44
=======================================
  Hits           20       20
  Misses         24       24

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Some potential may exist whitin this project

Thanks for replying previous issue report it had been a great help.
While I was developing my projects additional potential vulnerabilities has been detected within this project.
So I was wondering if you could look into this issue as my prior knowledge is very limited on this subject.
The suspicious files are

MD5Digest.java : CWE-916: Use of Password Hash With Insufficient Computational Effort ( The MD5 hash (used in org.bouncycastle.crypto.digests.MD5Digest) is insecure. Consider changing it to a secure hash algorithm )

Dump.java : CWE-23:Path Traversal (Unsanitized input from a command line argument flows into java.io.FileInputStream, where it is used as a path. This may result in a Path Traversal vulnerability and allow an attacker to read arbitrary files.)

Best Regards

So both these are okay (MD5Digest is the actual MD5 digest implementation, it's not just a password hash, it's still used in some places like TLS), but looking at it I am inclined to drop Dump.java - it's a wrapper calling the ASN1Dump class - as it doesn't really need to be there, we've got a few utility classes like that spread through the API now, it's probably time to move them all into a single executable style jar.

No further comment.


Deal with bad filenames

Need something to deal with bad filenames, optionally.
So that if someone wants to fix their broken filenames it can do that, otherwise they'll need to deal with it in their own front-end.

I think the scope of this is too broad to implement in a reliable way which will cover all cases for all people.


DropdownV2 Fixes + Input/Select.css.js

DropdownV2 Fixes + Input/Select.css.js

DropdownV2 gets better!
This (lovely) update fixes some implementation details with DropdownV2, making it more flexible + robust. Additional stories were added to manually test interactions and data flow.
Input/Select is now css.js
This is a huge change (and a major accomplishment). The Input component is one of the oldest components in HSDS:React. It's also one of the core components that's heavily relied upon. The decision to convert .scss -> .css.js was to ensure a smoother implementation of DropdownV2, specifically the variant that renders a searchable Input.
Rendering .scss code.. for a portal'ed Dropdown, potentially within a (non HSDS: React)Modal context, is not fun.
Ensuring that Input/Select UI work out-of-the-box means it'll work with the implementation described above (and more).
These changes were heavily E2E tested with the main application through a series of beta releases. A huge thanks to @knicklabs for working with me on this one!
Bug fixes
AvatarList was fixed to ensure that the items render correctly (inline).

Pull Request Test Coverage Report for Build 1444

63 of 63 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 1443:
0.0%

Covered Lines:
5678

Relevant Lines:
5678

💛  - Coveralls


Autobuyer pause

The autobuyer doesn't resume after a pause

I cant run it :(

Latest script should fix both the issue


analog drift auto calibration

Automatically compensate for drifting analog thumb sticks based on center drift at start

Thanks for the review @mthiesen , I went ahead and removed some other things I had added to the legacy from previous additions.

Looks good to me now! Thanks @NickGuyver!
Still, somebody with an actual analog stick device should try this out before we merge.


Problem using meshcat

I try to run the visualization demo for solo12 using meshcat 1. For me this results in the following UserWarning:
/usr/local/Cellar/python@3.9/3.9.0_4/Frameworks/Python.framework/Versions/3.9/lib/python3.9/runpy.py:127: RuntimeWarning: 'meshcat.servers.zmqserver' found in sys.modules after import of package 'meshcat.servers', but prior to execution of 'meshcat.servers.zmqserver'; this may result in unpredictable behaviour
  warn(RuntimeWarning(msg))
You can open the visualizer by visiting the following URL:
http://<IP_ADDRESS>:7000/static/
/usr/local/Cellar/pinocchio/2.5.2/lib/python3.9/site-packages/pinocchio/visualize/meshcat_visualizer.py:147: UserWarning: Error while loading geometry object: base_link_0
Error message:
No to_python (by-value) converter found for C++ type: boost::shared_ptr<hpp::fcl::CollisionGeometry>
  self.loadViewerGeometryObject(visual,pin.GeometryType.VISUAL,color)

This is on macOS where I installed pinocchio using homebrew.
Any idea how this could be fixed?

You did not install Pinocchio with the hpp-fcl support ;)

You did not install Pinocchio with the hpp-fcl support ;)

OK. If it is related to homebrew, I will have a look.

OK. If it is related to homebrew, I will have a look.

In between, you can also rely on conda.

In between, you can also rely on conda.

Thanks a lot, the pointer to hpp-fcl helped!
It looks like hpp-fcl was installed but not linked. So after running
jviereck@rams:demos$ brew install hpp-fcl
Warning: gepetto/gepetto/hpp-fcl 1.6.0 is already installed, it's just not linked
You can use `brew link hpp-fcl` to link this version.
jviereck@rams:demos$ brew link hpp-fcl

things are working fine now for me. Thanks a lot!

Thanks a lot, the pointer to hpp-fcl helped!
It looks like hpp-fcl was installed but not linked. So after running
jviereck@rams:demos$ brew install hpp-fcl
Warning: gepetto/gepetto/hpp-fcl 1.6.0 is already installed, it's just not linked
You can use `brew link hpp-fcl` to link this version.
jviereck@rams:demos$ brew link hpp-fcl

things are working fine now for me. Thanks a lot!


chore(master): release 2.7.18

:robot: I have created a release beep boop
2.7.18 (2024-11-15)
Dependencies

deps: update actions/checkout action to v4.2.2 (332f496)
deps: update ansible/ansible-lint action to v24.10.0 (#100) (f30b822)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.7.18
:sunflower:


chore: assertThrowsAsync is deprecated

Ref<EMAIL_ADDRESS>

Nice catch!


Implemented querying collections with Enumerable.All and Enumerable.Any

Implemented feature request in issue #112.

Hi @franzalex, nice works. Thanks... But, I why not use QueryIn for Enumerable.Any and implement another QueryAll for Enumerable.All?

The limitation with the current implementation is that QueryIn uses only the Object.Equals() method exclusively while QueryAll returns all the records in a collection with no exceptions.
With this implementation, the the limitation I described above is removed, allowing you to match a field with any method you desire (i.e. Contains, StartsWith, Equals, >, <=, etc). The following code shows what one had to do before and what can be done now.
var col = db.GetCollection<Person>("people");
col.EnsureIndex(x => x.Name, true);

// Test 1:  find people whose names start with John, Jane, Joe or Judy
var anyNames = new[] { "John", "Jane", "Joe", "Judy" };

// Before my submission (i.e. no Enumerable.Any)
var query1 = anyNames.Select(name => Query.StartsWith("Name", name))
                     .Aggregate((lhs, rhs) => Query.Or(lhs, rhs));
col.Find(query1);

// With my submission
col.Find(person => anyNames.Any(name => person.Name.StartsWith(name));

// Test 2: Find persons whose names contain all of the following John, W. and Smith
var allNames = new[] { "John", "W.", "Smith" };

// Before my submission (i.e. no Enumerable.All)
var query2 = allNames.Select(name => Query.Contains("Name", name))
                     .Aggregate((lhs, rhs) => Query.And(lhs, rhs));
col.Find(query2);

// With my submission
col.Find(person => allNames.All(name => person.Name.Contains(name));

Yes, it is true, you can do the same things using both methods. I believe though that my additions provide more convenience through the use of lambda expressions.

You are right. I never used Any or All enumetarion method, that why I didn´t undersand.. my Linq skill are terrible; Again, thanks for code and explanation. I will merge this weekend.


Mistakes in Wiki (API Documentation)

-[Persistent / JSON] Store content
--Sample Request:
 $ curl -X POST \
    'http://localhost:8040/ipfs-store/json/store' \
    -H 'Content-Type: application/json' \
    -d '{
      "field1": "val1",
      "field2": 10,
      "field3": {
          "test": true
      }
    }

' is missing in the above query
-[Query] Get content
curl \
    'http://localhost:8040/ipfs-store/fetch/QmWPCRv8jBfr9sDjKuB5sxpVzXhMycZzwqxifrZZdQ6K9o' \
    -o hello_doc.pdf

/query is missing before the hash so the corrected query is -
curl \
    'http://localhost:8040/ipfs-store/query/fetch/QmWPCRv8jBfr9sDjKuB5sxpVzXhMycZzwqxifrZZdQ6K9o'\
    -o hello_doc.pdf

-Sample Request:
curl -X POST \
    'http://localhost:8040/ipfs-store/search?index=documents' \
    -H 'content-type: application/json' \
    -d '{"query":[{"name":"title","operation":"contains","value":"Hello"},{"name":"author","operation":"equals","value":"Gregoire Jeanmart"},{"name":"votes","operation":"lt","value":"5"}]}'

/query is missing, the corrected query is
curl -X POST \
    'http://localhost:8040/ipfs-store/query/search?index=documents' \
    -H 'content-type: application/json' \
    -d '{"query":[{"name":"title","operation":"contains","value":"Hello"},{"name":"author","operation":"equals","value":"Gregoire Jeanmart"},{"name":"votes","operation":"lt","value":"5"}]}'

Thanks for your feedback. I fixed the wiki!


Can't download in HACS

HA core version
2024.3.3
Intergration version
0.3.22
Device type and model
Not relevant
Used App
None
The description of problem
Can't install from HACS. Throws error 'Unknown Error'. The cause is in the logs below. The URL is incorrect 'mieda' instead of 'midea'
The logs
ERROR (MainThread) [custom_components.hacs] Download failed - Got status code 404 when trying to download https://github.com/georgezhao2010/midea_ac_lan/releases/download/v0.3.22/mieda_ac_lan.zip

I also cannot download it. It says unknown error:

Same here

Same. HACS seems to want to download mieda_ac_lan.zip.

This repository is abandoned. I found this one instead: https://github.com/wuwentao/midea_ac_lan

Same here. Logs says
Traceback (most recent call last):   File "/config/custom_components/hacs/base.py", line 730, in async_download_file     raise HacsException( custom_components.hacs.exceptions.HacsException: Got status code 404 when trying to download https://github.com/georgezhao2010/midea_ac_lan/releases/download/v0.3.22/mieda_ac_lan.zip
well, the download url is wrong, there is a type in filename "mieda_ac_lan.zip" which should be "midea_ac_lan.zip".


DimensionProperty should modify the property names to dismiss the use of prefix

Need to recreate the context to resolve several issues in the context generated from the schema.


[google_sign_in] remove execute bit from a lot of files

I'd revert the @Dart=2.9 for now, until the tests are fully migrated, running and supported in stable.

ack

PTAL!

Are the windows bots just flaky?

Are the windows bots just flaky?

@kevmoo I've seen them fail in flutter/engine today too, not entirely sure how bad it is.

I think this is generally good, but happy to drop it. I need to get some other content going RE documentation here.

I think this is generally good, but happy to drop it. I need to get some other content going RE documentation here.

@kevmoo we're just waiting for the testing infra to recover to merge.

I think this is generally good, but happy to drop it. I need to get some other content going RE documentation here.

@kevmoo we're just waiting for the testing infra to recover to merge.

should I rebase?

@kevmoo no need, I think these builds are getting retried without much manual intervention (?)
https://ci.chromium.org/ui/p/flutter/builders/try/Windows Plugins/4026/related-builds


Javascript Enabed but I get error

In my browser javascript enabled. I check twice.

However, I get this warning.

"Staticshield checks for the valid token, verifies it and redirects to login page using Javascript. All this is done by the script that is loaded externally. Please enable network requests in your browser."
I try to add this function in docusaurus.

This seems like an issue with Blocking of network requests, not with executing javascript.
https://user-images.githubusercontent.com/69138026/128324269-95e2cc25-7a32-4da9-8e81-832512545c5d.mp4
In that video, you will be redirected to the Request Blocked page only when the network requests are blocked for the staticshield.vercel.app domain
Can you give some steps to reproduce the issue? Or any example on CodeSandbox or StackBlitz would help too

Hello @iozcelik, I've figured out the problem. There seems to be an issue with CORS with StaticShield. You will probably have to do nothing extra to get it working properly
I'll fix it as soon as possible.

Fixed it 🎉
If the problem persists, please reopen the issue


Added iOS 10 ITC rejection warning

Merge after a solution is found ?

I think it's worth bringing this in now, it will direct users to the right issue at the very least.


Made a change on the Spanner Java library

The pull request does following steps:

Made the batchClient library receive timestamp such that the query is repeatable.
Added retry settings for the spanner java library.
Translated the SpannerException error message into SpannerConnectorException.

/gcbrun


Only 24 photos.

Im trying to get photos from album which contains more than 300 photos and i always get 24 first photos

Hey @uraganas 👋,
Sorry for late response. Thank you for your issue. I fixed the issue on this commit https://github.com/yasinatesim/facebook-album-downloader/commit/e9c340859a73af5ded796ae38bd5324a0c94110e
Can you please make git pull origin master and try again

Now i get 0 of them 🗡


chore(main): release 0.1.0

:robot: I have created a release beep boop
0.1.0 (2024-08-12)
Bug Fixes

doc creation (a1f4fe0)
docs (fd0ee3f)
docs (b70caba)
environments (22ec8d5)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.1.0
:sunflower: