[BUG] Error while running Chat with video

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
When running the code in Chat with video
https://github.com/OpenBMB/MiniCPM-V/blob/ab1141ee450ee1a3b45b3005eb1f5d4e8811b7eb/README.md?plain=1#L1416-L1469
using the video v_4LF0hL-mgks.mp4 and question: "is the person first appears in the video outdoor" in VCGBench, I have got the following traceback:
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Traceback (most recent call last):
  File "/**/chat_with_video.py", line 41, in <module>
    answer = model.chat(
  File "/~/.cache/huggingface/modules/transformers_modules/openbmb/MiniCPM-V-2_6/4e4be000cd81feda8b96d14b53f1791b4010b038/modeling_minicpmv.py", line 348, in chat
    inputs = processor(
  File "/~/.cache/huggingface/modules/transformers_modules/openbmb/MiniCPM-V-2_6/4e4be000cd81feda8b96d14b53f1791b4010b038/processing_minicpmv.py", line 67, in __call__
    return self._convert_images_texts_to_inputs(image_inputs, text, max_slice_nums=max_slice_nums, use_image_id=use_image_id, max_length=max_length, **kwargs)
  File "/~/.cache/huggingface/modules/transformers_modules/openbmb/MiniCPM-V-2_6/4e4be000cd81feda8b96d14b53f1791b4010b038/processing_minicpmv.py", line 165, in _convert_images_texts_to_inputs
    input_ids, image_bounds = self._convert(final_text, max_length)
  File "/~/.cache/huggingface/modules/transformers_modules/openbmb/MiniCPM-V-2_6/4e4be000cd81feda8b96d14b53f1791b4010b038/processing_minicpmv.py", line 121, in _convert
    image_bounds = torch.hstack(
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 124 but got size 123 for tensor number 1 in the list.

Could you please help me to solve the problem? Thaks!
期望行为 | Expected Behavior
Successfully return the anwser.
复现方法 | Steps To Reproduce

In the environment of MiniCPM
Run the code in https://github.com/OpenBMB/MiniCPM-V/blob/ab1141ee450ee1a3b45b3005eb1f5d4e8811b7eb/README.md?plain=1#L1416-L1469
change the video file to [v_4LF0hL-mgks.mp4](https://drive.google.com/file/d/1osAM0waEP23fvHtrIcgsdqfFkUgO
hlq9/view?usp=sharing) (can be downloaded clicking the link) and question: "is the person first appears in the video outdoor"
See error:

Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Traceback (most recent call last):
  File "/mnt/afs_intern/shuyan/experiments/MiniCPM-V/video_chat.py", line 41, in <module>
    answer = model.chat(
  File "/mnt/afs/shuyan/.cache/huggingface/modules/transformers_modules/openbmb/MiniCPM-V-2_6/4e4be000cd81feda8b96d14b53f1791b4010b038/modeling_minicpmv.py", line 348, in chat
    inputs = processor(
  File "/mnt/afs/shuyan/.cache/huggingface/modules/transformers_modules/openbmb/MiniCPM-V-2_6/4e4be000cd81feda8b96d14b53f1791b4010b038/processing_minicpmv.py", line 67, in __call__
    return self._convert_images_texts_to_inputs(image_inputs, text, max_slice_nums=max_slice_nums, use_image_id=use_image_id, max_length=max_length, **kwargs)
  File "/mnt/afs/shuyan/.cache/huggingface/modules/transformers_modules/openbmb/MiniCPM-V-2_6/4e4be000cd81feda8b96d14b53f1791b4010b038/processing_minicpmv.py", line 165, in _convert_images_texts_to_inputs
    input_ids, image_bounds = self._convert(final_text, max_length)
  File "/mnt/afs/shuyan/.cache/huggingface/modules/transformers_modules/openbmb/MiniCPM-V-2_6/4e4be000cd81feda8b96d14b53f1791b4010b038/processing_minicpmv.py", line 121, in _convert
    image_bounds = torch.hstack(
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 124 but got size 123 for tensor number 1 in the list.

运行环境 | Environment
- OS:Ubuntu
- Python: 3.10.14
- Transformers: 4.40.0
- PyTorch:2.1.2
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`): 12.1

备注 | Anything else?
No response

hi, I had the same error. Change params["max_slice_nums"] = 2 to 1 seems to work. It is suggested by the author that # use 1 if cuda OOM and video resolution > 448*448
It is also mentioned in a similar issue that the video resolution should be limited. https://github.com/OpenBMB/MiniCPM-V/issues/509

Yes, it's true that change max_slice_nums to 1 avoid the error.

marked


throws an error because of "import .makedocco import *"

Whether you want import pylode into Python, or execute pylode (in this case, under Linux), it throws an error.
Python 3.7

Thanks for the note @businessabstraction (Alex!). I've really struggled with getting Python 3 imports correct for both the CLI scripts and the Python modules and clearly haven't solved everything yet.
I'll try and fix this ASAP.
Would love a Pull Request if you can suggest a working formula...

I have pushed a new release of this code both here to GitHub and also to PyPI (2.2 and hopefully now all import issues are solved. Please check and, if they are, close this Issue.

Seems related, but after installing pylode with python3.7 and running pylode:
$ python -m venv .venv
$ source .venv/bin/activate
$ pip install pylode # Installs pylode-2.2
$ pylode
Traceback (most recent call last):
  File ".../.venv/bin/pylode", line 6, in <module>
    from pylode.cli import main
  File ".../.venv/lib/python3.7/site-packages/pylode/cli.py", line 7, in <module>
    from makedocco import MakeDocco
ModuleNotFoundError: No module named 'makedocco'

@koslambrou OK, I've just replicated that and will try and solve now. So many import issues...

I'll try to help with a PR

I've overcome all the import issues but haven't implemented the __all__ directives - would love a hand with that! Will close this now though as the main import issues are done.


model: Add initial HubertWithKmeans module and extract tool.

For use in extracting discrete HuBERT tokens from audio files.
Adapted from https://github.com/lucidrains/audiolm-pytorch

Hi there, welcome to the first PR and thank you for your consideration and steering as we explore together! A few notes as I begin to find my way:

initially I had a PR with only the hubert.py model. However, it felt bare without the tool to show usage. Open to breaking apart if you'd prefer.
have you seen https://github.com/Delgan/loguru and are you open to trying it out? If you'd prefer I can swap in logging instead (we've been using loguru for a while and much prefer it, by default for example it will log the source file/method and line number for log statements). It's only in the tool, so you can try running it to see what it looks like (loguru not yet included in requirements.txt, you'll need to install it).
I put the new deps in requirements-dev.txt since , in our explorations we can allow for cpu-based token extraction from dataset samples at runtime (e.g. not precomputed) which isn't really dev/tools at that point. Not sure we'd want to allow that, though, as it'd slow the datasets and adds complexity. So, just mentioning as placement reqs-file-wise isn't 100% clear.
testing: I started a unit test for HubertWithKmeans but, lol -- how best to test an nn.Module, one might ask. All the unit test really did was mock out the torch, fairseq, etc. and allow the ctor to pass, which isn't particularly useful beyond validating the ctor signature. More useful would be something like a trivial integration test that runs it with the tool on a few files, or at least the model on a fixture audio file, etc. There isn't infra for this today and given it's experimental and so forth it felt like overkill, so I backed out the test work and figured we'd discuss. Similarly, the tool is pretty simple and I'm not sure it warrants tests yet. I'm open to adding whatever you'd like, and can help set up fixture/etc. bits as needed, now or later.
the tool has to download the HuBERT checkpoint and kmeans data and store it locally. I didn't see clear precedence for this, so I added a new models dir to repo root and to .gitignore, but open to thought. I know typically one uses HF to download pretrained models and the HF cache handles it, however I don't believe the exact files we want here are set up for this on HF today (which doesn't mean we couldn't put them there, so again, open to feedback).

The HF thing was the first thing that jumped out at me - I tend to think we'll be happier if we have a unified way of hosting and downloading models. If this turns into a bear we can of course revisit.
Minor nits:

please add mypy types
please follow Google Python style guide and import only packages, not individual classes

Actually I know Justin said to separate out the PR, but I'd really like to see the entire changes first.
Just the diff (maybe with your main and ours) could be good enough.

The things that I'd like to know:

what datasets were used and with what prompts
how long did training take and it would be nice to see some training curves
what are the material changes in training required other than the addition of HuBert tokens for output

Let's follow up soon and happy to have a discussion. We are not yet as far as you describe above though we aim to move very quickly -- we are actively working on proper training next. What we've done so far is roughly:

add audio_hubert_tokens to voice samples
increase the model token space to allow for hubert tokens
repurpose the audio tag to populate it with said hubert tokens
trivial training with a straightforward test prompt that produces said tokens, so as to:
ensure that the ultravox model can ingest hubert tokens and produce them at inference time
ensure that our acoustic/vocoder models can take those tokens and produce quality speech

There are various directions we can go in editing/adding training/datasets with the goal to teach the model to be able to speak HuBERT for responses while preserving existing functionality. I expect it could take time to find the one we like best, and so that we'll be playing around with things here quite a bit. We're optimistic that we can quickly identify some initial training run/dataset changes that are not large in scope and support speech output, and we'd value your input and steering on approach!
Similar to my other comment, with the above as the planned direction, having a model/tool in repo to allow generating the tokens (and iteratively cleaning up and landing some of the bulleted work above, e.g. behind a flag or otherwise generalized/abstracted) seems useful to support a team effort for what's likely long-lived exploration.
Maybe another way to put it all is -- we have several of the likely foundational pieces together or in mind (semantic speech tokens, voice model/vocoder), we're learning the Ultravox internals/tooling and starting to fit them together, and we've done work to de-risk/validate and make sure the end to end flow is feasible and roughly operational. Another key upcoming work item re: foundational pieces is to also include our models for inference purposes.
I will put together a PR to sketch the above bullet points (done-so-far) soon but I don't expect all of it is yet something we'd want to land. This tool/model's case was clearer (but I understand your point re: using existing HF model if it fits).

Thanks for the explanation.
I think I understand the disconnect here. I thought you had training working already (which was totally surprising to me).
Repeating to make sure I understand, what you did was some sort of "overfitting" experiment just to make sure it was possible to get the tokens out. Not really a full-fidelity trained model that could respond in voice as it saw fit.
I'm aligned with the direction now.
Production vs experimentation and how to handle them:
Until now I've generally tended to do highly experimental features in a branch and only push them to main after getting more confident, but that becomes less viable as more people want to collaborate on the same path.
In some projects we had an experimental or projects directory under main for each person/project idea. This directory had looser constraints (e.g. no/less testing, possibly different Python requirements). However I don't think that's necessary for us yet.
I guess my rule of thumb is: highly experimental features related to training/model need to be proven at least somewhat first before being merged, but in your case you've already done the due diligence so we're good.
And as a rule, I'm open to experimental features as long as the existing paths still remain viable and not highly impacted.

Agree with @farzadab. This is definitely part of the roadmap so I don't think we need to worry too much about exactly what directory this lives in, but I do want to be cautious about taking on large dependencies and also align on core concepts like how we load models.
I definitely know that the PR process isn't always the easiest way to communicate, so don't hesitate to post/DM if we're giving unclear/conflicting advice.

Still working on this, also getting familiar with poetry, lol! Removing fairseq would be yet further helpful as it depends on hydra-core which breaks testing for some reason. More tomorrow.

Repeating to make sure I understand, what you did was some sort of "overfitting" experiment just to make sure it was possible to get the tokens out. Not really a full-fidelity trained model that could respond in voice as it saw fit.

Yes, that's right. Working on the trained model as well in parallel.


Adds nullcheck Connects to #168

Retrieving the same data twice caused a nullpointer exception, to I addes a nullcheck

All is well.


Renaming a file case should show up in git changes

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.73.0
OS Version: MacOs Ventura 13.0

Steps to Reproduce:

Rename a file but only change the case of something (Eg readme.md to README.md)

There will be no change registered in git - and this is a known problem in git -which can be mitigated by using the git mv command to rename the file instead of the file system standard rename. https://git-scm.com/docs/git-mv

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Pure JDK Vault Client for Vault Config Source

Fixes https://github.com/quarkiverse/quarkus-vault/issues/226

How are you setting up the proxy configuraiton for the JDK client? Also, wouldn't it be better to just merge #215?
The only outstanding issue I ran into for using the JDK for configurtion in the new client was setting SSL and proxy to match the Vert.x implementation exactly.

Merging 215 has the added advantage that we can close all but 1 PR 👍

this started as a POC. I would like to fix https://github.com/quarkiverse/quarkus-vault/issues/226, as we are spending a lot of time trying to chase situations where dev mode does not work.

How are you setting up the proxy configuration for the JDK client?

not done at this point.

Also, wouldn't it be better to just merge https://github.com/quarkiverse/quarkus-vault/pull/215?

yes. definitely if it is ready. is it? you could grab the ssl stuff from my PR? not sure about the proxy configuration (I did not do it).

are we talking about this?
https://openjdk.org/groups/net/httpclient/recipes-incubating.html#proxy

For me #215 is ready, there are a couple feedback items unresolved that you might want to close or comment further on.
I can move your SSL configuration into #215 easily and we can use the JDK client for configuration request.
The proxy configuration is what was troublesome to me for using the JDK with Quarkus's configuration.

are we talking about this? https://openjdk.org/groups/net/httpclient/recipes-incubating.html#proxy

Kind of. I saw no way of building a proxy selector using Vert.x proxy configuration items.

I like better what you did to be honest. we need it to go through.
did you check that it supports dev mode restart?
I am missing something. we have everything we need for vertx? don't we? i.e. io.vertx.ext.web.client.WebClientOptions
it is actually for the http client that we are missing a way to configure the non-proxy-hosts (is it supposed to be in the proxy selector?)

one thing to consider is that we do not need to support an entire jdk vault client for the config source. if we do, good. but it is not a requirement. we just need login and read secret (v1 and v2).
so if that makes it difficult to reuse the same approach between the config source and the vault api use cases, I would rather completely separate them.

The issue is, as I found it (I may be wrong), configurign a JDK HttpClient to work nearly identicaally to the Vert.x WebClient/HttpClient. I found most of this to be easily transferrable or not required to produce similar functionality (e.g. timeouts just need to be correct in aggregrate). The two issues I ran into were configuring the JDK's SSLContext and ProxtySelector. I knew the SSLContext configuration was doable but stumbled at the ProxySelector; given the proxy issue I stopped at that point and returned to using Vert.x clients even for the configuration requests. The specific issue is that the JDK's DefaultProxySelector doesn't support the same configuration as Vert.x.
Give this we have two options:

Configure the proxy as well as possible using the available Vert.x settings
Write our own, or find an available, ProxySelector implementation that supports all the options that Vert.x does.
Ignore the proxy configuration until some point in the future where we implement 1 or 2.

Given that you have tackled the SSL configuration. The proxy configuration is the only remaining hurdle for both this PR and #215.

one thing to consider is that we do not need to support an entire jdk vault client for the config source. if we do, good. but it is not a requirement. we just need login and read secret (v1 and v2).

#215 does implement the entie thing and as noted is only block by the proxy configuration problem. There is no other reason not to use it's JDK client in Quarkus.

Implemted via #215


Initial Update for BSidesRDU 2019

Please update the site to reflect the new 2019 Information by May 4th.
This year's date is October 18, 2019
Remove last year's schedule, and mention that the CFP is currently open. Link to CFP is https://sessionize.com/bsidesrdu-2018/  (opens on May 4)
Remove the Hands On Village from the Side Events
Remove the registration section under buy tickets and replace with a message that tickets will be on sale "soon".
Add a message above the sponsor logos that "Last year's sponsors included ...". TBH, I'd remove the "Click here to become a sponsor" link as well because we're still working through the rates and benefits of sponsorship.

Hi! Glad to help on this! FYI: when I go to the CFP site (which the URL still says 2018) I get the following warning:

"This event is in trial mode and can be used just for testing. Number of submissions is limited"

Would you like me to still link it?

Can you make that update on May 4th? :)
On Mon, Apr 22, 2019, 11:41 AM Omar Santos<EMAIL_ADDRESS>wrote:

Hi! Glad to help on this! FYI: when I go to the CFP site (which the URL
still says 2018) I get the following warning:
"This event is in trial mode and can be used just for testing. Number of
submissions is limited"
Would you like me to still link it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BSidesRDU/website/issues/12#issuecomment-485453625,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABX44MVBRINPKSMHMNE3TTLPRXMDBANCNFSM4HHQLDVQ
.

absolutely... ;-) I am at least cleaning up the old stuff and just updating the date, etc...

It said, "by May 4th" (be the 4th be with you) ;-) so I was at least cleaning stuff early.

These are the updates from me:

This year's date is October 18, 2019

Done.

Remove last year's schedule, and mention that the CFP is currently open. Link to CFP is https://sessionize.com/bsidesrdu-2018/ (opens on May 4)

I added just a text saying "coming soon!" Will edit on May 4th.

Remove the Hands On Village from the Side Events
Done.

Remove the registration section under buy tickets and replace with a message that tickets will be on sale "soon".
Done

Add a message above the sponsor logos that "Last year's sponsors included ...". TBH, I'd remove the "Click here to become a sponsor" link as well because we're still working through the rates and benefits of sponsorship.

Done

Thanks! That was fast ;)
Hands on Village still appears for me?
On Mon, Apr 22, 2019, 11:56 AM Omar Santos<EMAIL_ADDRESS>wrote:

These are the updates from me:
This year's date is October 18, 2019
Done.
Remove last year's schedule, and mention that the CFP is currently open.
Link to CFP is https://sessionize.com/bsidesrdu-2018/ (opens on May 4)
I added just a text saying "coming soon!" Will edit on May 4th.
Remove the Hands On Village from the Side Events
Done.
Remove the registration section under buy tickets and replace with a
message that tickets will be on sale "soon".
Done
Add a message above the sponsor logos that "Last year's sponsors included
...". TBH, I'd remove the "Click here to become a sponsor" link as well
because we're still working through the rates and benefits of sponsorship.
Done
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BSidesRDU/website/issues/12#issuecomment-485457841,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABX44MVJ53W4RRZJPTC6FWTPRXNZHANCNFSM4HHQLDVQ
.

The change should be there now...

Updated website with CFP information. Sorry for the delay.


Add count for current users online

Why is this change being made?
Adds #79
How is this being accomplished?
It adds a counter based on the amount of users accessing the website. This was done using Presence from Phoenix to handle the whole behavior.
Every LV module on the mount will trigger the Presence and a component on the home page will render the proper value.

Testing

Run the project
Access the home page
[ ] Must show the value of total online users
Open a anon browser and access a new page
[ ] It must update the value for the count

Take aways

UI is not my strongest skill so any suggestion or commit would be helpful
There is a couple of copy and paste code, I do not see as a problem now, but in future that can be improved

@danielbergholz all accepted, running locally it keeps working fine.

Impressive work @lcezermf! Thanks for spending the time to implement such a cool feature 🔥

Close #79


There is a plan to support multi type database connection pool ?

Why you need it?
Now, rm-datasource module is dependent on druid, not support others database connection pool.
How it could be?
rm-datasource module is dependent on druid, AbstractDataSourceProxy is hardcode DruidDataSource, it means only can use druid.Threr is a plan to support multi type database connection pool ? eg. hikariCP、cp30 ... etc.
Other related information

already support


Duration fields should use timedelta

          @saskiad @mekhlakapoor The duration fields in the pydantic models should probably be `timedelta` types, which is python's equivalent of Java's Duration class. The `time` class is supposed to represent a time of day, which has hard limits like hour < 24 etc. We can also avoid writing workarounds like the ones in the utils package here.

Originally posted by @jtyoung84 in https://github.com/AllenNeuralDynamics/aind-metadata-service/pull/82#pullrequestreview-1346288332

does this require start/end times be reported? I think the duration field we have (and @mekhlakapoor can correct me if I'm wrong here) are provided in the SWR as durations, rather than start/end times. Or is it just a different type of time unit?

We can use the reported durations. The issue is that we're converting the durations into a time object instead of a timedelta object. This can cause issues in the future. For example, if the duration lasted more than a day, then the time object would raise an error because it's expected to be less than 24 hours.


Add new pipeline helpers and deprecate old paths

Motivation:
As part of #214 we need to introduce new pipeline helpers to configure
the channel pipeline with the new handlers. This PR adds those helpers
and deprecates the old paths.
Modifications:

Add pipeline helpers with the new streamID-less initializer
Switch old pipeline helpers which provided codecs to use the new path
and codecs
Duplicate SimpleClientServerTests and ConfiguringPipelineTests to add
frame payload based counterparts
Deprecate older helpers and multiplexer init, delete tests which don't
hit deprecated paths (note: these still exist in the payload based
counterparts)
Update Bench1Conn10kRequests to be payload based

Result:

More 'FramePayload' based Tests
Paths for creating an 'HTTP2Frame' based stream via
'HTTP2StreamMultiplexer' are now deprecated

Oops. Looks like I missed some tests.

The API breakage is expected: we changed the argument name and removed the default value of an init which is yet to be released.
11:35:34 Constructor HTTP2StreamMultiplexer.init(mode:channel:targetWindowSize:outboundBufferSizeHighWatermark:outboundBufferSizeLowWatermark:inboundStreamStateInitializer:) has parameter 5 type change from ((NIO.Channel) -> NIO.EventLoopFuture<Swift.Void>)? to ((NIO.Channel, NIOHTTP2.HTTP2StreamID) -> NIO.EventLoopFuture<Swift.Void>)?
11:35:34 Constructor HTTP2StreamMultiplexer.init(mode:channel:targetWindowSize:outboundBufferSizeHighWatermark:outboundBufferSizeLowWatermark:inboundStreamStateInitializer:) has removed default argument from parameter 3
11:35:34 Constructor HTTP2StreamMultiplexer.init(mode:channel:targetWindowSize:outboundBufferSizeHighWatermark:outboundBufferSizeLowWatermark:inboundStreamStateInitializer:) has removed default argument from parameter 4


[SPARK-9153][SQL] codegen StringLPad/StringRPad

Jira: https://issues.apache.org/jira/browse/SPARK-9153

Can one of the admins verify this patch?

Jenkins, add to whitelist.

Can one of the admins verify this patch?

Test build #1117 has started for   PR 7527 at commit 3840c6b.

LGTM

Test build #1117 has finished for   PR 7527 at commit 3840c6b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Thanks - I've merged this.


[docs] add else statement to introducing jsx documentation

Hi, I think that in the code below at:
https://github.com/reactjs/reactjs.org/blob/master/content/docs/introducing-jsx.md#jsx-is-an-expression-too
function getGreeting(user) {
  if (user) {
    return
Hello, {formatName(user)}!
;
  }
  return
Hello, Stranger.
;
}

should be a else statement, because if user is true, the function will return two h1 elements.

A function can only return once. The pattern of conditionally returning is called an "early return" or "short-circuit". The rest of the function isn't executed.


feat: add support for custom step timeout

Add support for passing in a config param, matching the signature of cucumber-js, to set the step
timeout in cypress.
e.g.
Given('a long running step', { timeout: 60000 }, () => { ... });

It is possible to do this by calling cy.then({timeout: 60000}, () => {...}) inside the implementation function, but that's inconsistent with the signature for cucumberjs, which feels more intuitive.

Looks good, thanks!

:tada: This PR is included in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

It looks like this MR broke TypeScript for us at this line:
https://github.com/TheBrainFamily/cypress-cucumber-preprocessor/pull/385/files#diff-c3cd9d7865893aca768bfb5cecbcfd9bR50
export function defineParameterType(): void;

Is the change in the MR correct? Because we use the following snippet in our code:
import { defineParameterType } from 'cypress-cucumber-preprocessor/steps';
import * as faker from 'faker';

defineParameterType({
  name: 'faker',
  regexp: /.*/,
  transformer(s) {
    return faker.fake(s).replace(/^"|"$/g, '');
  },
});

(As you can see defineParameterType takes 1 argument here.)

sorry, you are definitely right. I wonder if it makes sense to keep some of the types here since the types are available here: https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/cypress-cucumber-preprocessor/steps/index.d.ts ?
I'm trying to decide between fixing the types here, or deleting them altogether. Having two sources of truths bothers me though.

Thank you for the fast response and trying to solve this.

do you have any opinion about the way to move forward with this problem? :)

My personal preference would be:

Have all types inside cypress-cucumber-preprocessor.
Have all types inside @types/cypress-cucumber-preprocessor.
....
Have a mix :)

For now I've updated the types inside the cypress-cucumber-preprocessor, as per your preference. ;-)
Thanks for the report here and sorry it took a bit. Trying to dig out from my busy todolist..

Thank you!


Revert "Update to Kotlin 1.9.0"

Reverts slackhq/slack-gradle-plugin#364

Closing as we're fixing forward


[Feature Request] Add an option to Tesseract to use external cloud OCR APIs

Tesseract is good for non-handwriting text, but it isn't accurate at all for handwritten text.
Websites like Google are very accurate. Is it possible to add the possibility to upload and do the OCR in external websites and then "paste" the results in the PDF to make text searchable like tesseract already does?

Your question is not related to Tesseract and would be better asked on the Tesseract user forum. But the answer is yes, it is possible to add searchable text to images or PDF files, no matter where this text comes from as long as there is also information on the text position (bounding boxes, baselines) available. Here is an example which has images and text transcriptions made with eScriptorium: http://nbn-resolving.de/urn:nbn:de:bsz:180-digad-31525.

We can split this to two feature requests:

Make Tesseract able to use the API of some cloud OCR (from Amazon, Google, Microsoft).
Make Tesseract's PDF renderer able to read hOCR/ALTO format and use the data instead of running and using Tesseract own OCR engine.

这是来自QQ邮箱的自动回复邮件。
您好，邮件已收到，我尽快给您回复。             祝您工作顺利、身体健康！

IMHO that's a workflow or utility problem, thus outside the scope of Tesseract.
There are still utilities creating searchable PDF from hOCR or other OCR markup formats. Maybe a ready to use convenience wrapper to combine results makes sense for some users. But this can and should be completely independent from Tesseract.
As I understand https://readcoop.eu/transkribus/?sc=Transkribus or similar other projects they still provide searchable PDFs for handwritten OCR. No need to reinvent the wheel.

@All3xJ : so you should ask support at Google directly (and so on).
How I read you request:
You ask opensource project to provide support for closed source project. You asked free project to provide free support for commercial (and  at the end of day paid) solution. This does not sound reasonable for me. I see no benefit for tesseract.

@All3xJ

yeah but Transkribus has not the amount of data and informations given by AI that Google (and so on) has.

We even don't know, what Google, Amazon or Microsoft provides as "data and information", let alone in which format and if they provide an exact technical specification for this format. I tried to find out and found nothing (1 hour wasted time).
You have the same problem that most contributors to Tesseract also have, as they are involved solving workflow problems of OCR in their professional life at universities, libraries or archives: If not available, develop it yourself.
Technically you have two alternatives:

convert the results to the same data format, e.g. hOCR, ALTO, Page-XML, ABBY-XML, or your own custom one. For hOCR there exist tools to create searchable PDF from it.
merge the PDF results, or convert the PDFs to hOCR, then see above.

Whatever your favourite programming language is, it should be a more or less easy task. For Python or Java you will find libraries doing the basic things.

https://github.com/tesseract-ocr/tesseract/issues/3754#issuecomment-1036309861
I also think that we should not support the first feature request.

We even don't know, what Google, Amazon or Microsoft provides as "data and information", let alone in which format and if they provide an exact technical specification for this format. I tried to find out and found nothing (1 hour wasted time).

I found this: https://cloud.google.com/vision/docs/handwriting

Whatever your favourite programming language is, it should be a more or less easy task. For Python or Java you will find libraries doing the basic things.

Maybe it is, but I have 0 experience in these kind of things, and even though I can code, I don't know where to begin with this.

We don't plan to implement this feature request.


Skill: DO-NOT-MERGE:Teaching a model to rhyme.

These provided examples demonstrate how to rhyme.

This PR was opened for instructlab UI testing purposes. Closing the PR.


Add cell defining agents

The definition must include the number of agents (implicit or explicit), labels for the agents (to use in the plots), and the associated units.

Solve with Issue #1


Debugger feature: Find active references

Don't know how difficult this would be to implement in practice (perhaps it could be built on top of mark-and-sweep?)  The idea is for the debugger to be able to tell you, for any object, where that object is being referenced.  That would help you to debug memory leaks, etc.

Most likely it'd be an operation similar to mark-and-sweep but with parallel code, I'm not sure if it could be fit cleanly into mark-and-sweep.
Maybe an easier, and more generic, way would be just to ensure that the DumpHeap command provided all the necessary references so that the debug client could find the references from there. This would mean dumping value stacks, activations, etc. For the target this is not a problem because all the debug data is streamed with a fixed overhead in this case.


Is there a request queue metric?

Title:  Is there a request queue metric?
Description:
I noticed that if my upstream can handle X requests per second, and I load test with say, 2X, envoy still reports X~ for upstream_rq, which makes me think it has a request queue for the active connections or something
Is there any metric exposed that indicates how many requests are queued?
EDIT: The motivation is to scale my service according to ACTUAL rps envoy gets, rather than perceived upstream request, and load X, and 2X seem to have the same _rq value..

https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_conn_man/stats#statistics
There are a number of metrics prefixed with downstream_rq that should help. The downstream_rq_active gauge is incremented at the start of each downstream request and decremented when the response is completed.

Thanks for the tip!
The rq_active metric seems to be more relevant as it represents to load accurately.
Does rq_total only count completed requests? why is the rate of rq_total and raw rq_active are different?

Thanks for the tip! The rq_active metric seems to be more relevant as it represents to load accurately. Does rq_total only count completed requests? Why is the rate of rq_total and raw rq_active are different?

@aclowkey I might be late with this answer, but I think the difference between two of those comes with the format.
rq_total is a counter metric, which (by Prometheus format) means that it can be only incremented. Thus, rq_total gets incremented each time when the new downstream request starts. This metric represents the total amount of requests since the Envoy process start.
rq_active is a gauge metric, which represents the current amount of requests, i.e. amount of active requests processed at the time of metrics scrape.
Keep in mind that it's still going hard to catch small 5/10-seconds spikes as the default interval of metrics scraping is 60 seconds.


Additional endpoints for tx notes

Coverage increased (+0.3%) to 94.802% when pulling 5a890c036e367b24655c3c54d2855dcdf2e9f9b7 on isocolsky:feat/tx-notes into 9b11eca313cdd2711ecf1abccd7a31ccbcd913e9 on bitpay:master.

Coverage increased (+0.1%) to 95.08% when pulling 7600654ba7f40bd83422d2bc023202d9c2b4e242 on isocolsky:feat/tx-notes into d4c97ff649dcec50496e851eee46bab71945d9c4 on bitpay:master.

Coverage increased (+0.1%) to 95.091% when pulling 602d67c94181fdf485e54dbeb0e32987db586e8c on isocolsky:feat/tx-notes into d4c97ff649dcec50496e851eee46bab71945d9c4 on bitpay:master.

Nice!

Thanks for the thorough review!


Generic Chart - not possible use queries from extension

It is not possible to make a Generic Chart based on a query from an extension.

Here is the list of queries:

Hi @peikba . Which version are you running on? Is this on prem or saas?

I'm running 0.14.17461 against NAV 2018 CU2

@peikba could you please provide us with code that reproduces this error? A zipped small extension would be great!

@DominikDitoIvosevic  Here it is
Default publisher_Test Charts With Queries_<IP_ADDRESS>.zip
Import the Extension.
Create a new Generic Chart

Select the Query from the  Extension (It is visible)

and I get the Error

@DominikDitoIvosevic
It is still a problem in Dynamics NAV 2018 CU3

A few side notes:
Al Language 0.14 should not be used with NAV 2018. See the FAQ: https://github.com/Microsoft/AL/wiki/Frequently-Asked-Questions#which-version-of-the-al-language-extension-should-i-use
This GitHub repo tracks issues only for the latest version of Dynamics 365. See this FAQ entry for more details about NAV 2018 process: https://github.com/Microsoft/AL/wiki/Frequently-Asked-Questions#nav-2018-issues

@peikba this issue is fixed in the latest developer preview and this issues page is for tracking issues for the Developer Preview. If you would like to have your bug backported please open a request through the NAV 2018 process: https://github.com/Microsoft/AL/wiki/Frequently-Asked-Questions#nav-2018-issues


New schnorr

This hasn't been tested and isn't in a working state! There are todos throughout that I think should all be resolved before merging too, and I'll comment on other bits that are also confusing

Curve.Scalar.to_bits doesn't exist (or at least isn't given to Signature_lib.Checked.Schnorr in lib/coda_base/schnorr.ml?) and so I removed it from the type signature and so this doesn't currently work -- I either need to implement it here instead ? Curve.Scalar.Checked.to_bits does exist though and is happily used :)
this isn't tested


From private repo

@PRMerger14 : Thanks for your contribution! The author, @aczechowski, has been notified to review your proposed change.


send message: Have no rights to send a message

I use this lib to send messages to Group on telegram .and I often take a message like that 'Have no rights to send a message' when I send a message to a group.so any one can help me?

You receive this error when toy have no rights to send messages to the group.

I want to get the time which until after times I can send message.I saw the PC client has that

If you have no rights to send messages, then this is likely a permanent restriction.

not yet,I saw that tips on PC clients.and this message will be showed on message input area.and  some time after.I can send message continue.

the admin of this group restricted you from writing message here until 21.06.01

this is tips

You can check supergroup.status.restricted_until_date, but in most cases it will be 0, i.e. the restrictions are permanent.


Add directions to enable prom style metrics reporting

Add comments to worker options
Verify metrics functionality with prom style metrics

@mfateev at this time, the only other solution I can think of is, dual emission of metrics to be prom compatible. I feel passing in a sanitized scope is neater though..


Question: Alias/resolver mappings with '*' supported?

Is there a way to use alias configuration similar to the paths of tsconfig.json or do I need to add it for each alias?
This possible?
{
  alias: {
    '@scope/*': '/packages/*/src'
  }
}

Current approach:
{
  alias: {
    '@scope/pkg1': '/packages/pkg1/src',
    '@scope/pkg2': '/packages/pkg2/src'
  }
}

And how would I do that with the resolvers?
Thanks!

See type definition of Resolver
You can do something like:
const scopeRegex = /^\/@scope\/(\w+)\//

const myResolver = {
  requestToFile(req) {
    const match = req.match(scopeRegex)
    if (match) {
      const pkg = match[1]
      return path.resolve(__dirname, 'packages', pkg, 'src', req.replace(scopeRegex, ''))
    }
  },
  fileToRequest(file) {
    // reverse logic
  }
}

module.exports = {
  resolvers: [myResolver]
}

Also note: path aliases must start with / and end with /. Otherwise it's considered a package name alias.

@yyx990803 Thank you that really helped me!

I've published vite-tsconfig-paths to solve this problem:
https://github.com/aleclarson/vite-tsconfig-paths
Also, retweet this if you'd like:
https://twitter.com/alecdotbiz/status/1291095881262010371

Also note: path aliases must start with / and end with /.

This conflicts with tsconfig.json's paths. It does not allow any absolute paths. This limits the usage of alias within the limits of Javascript.

Also note: path aliases must start with / and end with /.

This conflicts with tsconfig.json's paths. It does not allow any absolute paths. This limits the usage of alias within the limits of Javascript.

Try this.
// vite.config.ts
alias: {
    '/@/': path.resolve(__dirname, 'src'),
},
resolvers: [
    {
        alias(id: string) {
            return id.replace(/^@\//, '/@/') // add slash to particular id, then vite won't resolve it as a module
        },
    },
],

// tsconfig.json
"paths": {
    "@/*": [
        "./src/*"
    ],
},

This trick can allow you to use relative path as alias.

@RoB8080 thanks

Work on 2.0, see https://vitejs.dev/config/#alias

Work on 2.0, see https://vitejs.dev/config/#alias


x/crypto/ssh: publicKeyCallback cannot handshake using ssh-rsa keys signed using the ssh-rsa-sha2-256 algorithm

This relates to:
x/crypto/ssh: cannot sign certificate with different algorithm #36261
x/crypto/ssh: support RSA SHA-2 host key signatures #37278
What version of Go are you using (go version)?
1.14.2
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/mmarsh/Library/Caches/go-build"
GOENV="/Users/mmarsh/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/mmarsh/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/mmarsh/go/src/golang.org/x/crypto/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/8t/0yb5q0_x6mb2jldqq_vjn3lr0000gn/T/go-build245084723=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Here's a gist containing code to reproduce this issue, provided you have an instance to connect to that's similarly set up to mine. I've got full steps to create such an instance in the gist's README.md.
https://gist.github.com/SwampDragons/8b913208add452f1b50f6f47426ac329
What did you expect to see?
I expected to be able to connect to the instance using a custom AlgorithmSigner.
I am able to connect to an instance before the crypto policy on that instance is updated to deny keys signed using the "ssh-rsa" algorithm. Note that the policy being applied does accept the ssh-rsa keys if they are instead signed with the "rsa-sha2-256" algorithm. Theoretically, I should be able to create my own AlgorithmSigner to apply this algorithm to my key.
What did you see instead?
I get denied access to the instance with an authentication error.
I have traced this to the validateKey and publicKeyCallback methods in ssh/client_auth. These methods assume that the algorithm is always the same as the key type, which is not the case in my situation.
Possible Solutions
Create a new interface, AlgorithmSignerWithAlgo, which has the method Algorithm(). When called, Algorithm() will return the type of algorithm used, so that publicKeyCallback can set this field accurately. We will also need to update the validateKey method to not return an error if the algorithm used to sign the validation request doesn't match the key type.
Here's a diff of a lightweight solution that enables users to implement their own signers that the default publicKeyCallback  can use to correctly handshake: https://github.com/SwampDragons/crypto/pull/1/files

/cc @hanwen @FiloSottile per owners.

I've opened a patchset at https://go-review.googlesource.com/c/crypto/+/240717

Is there anything else the maintainers need from me in order to make this ready for a review?

Hi, I know it's annoying having someone nagging at an issue, but it's been a while and having some kind of solution to this is a blocker for some people using Packer. Is there any way @hanwen or @FiloSottile could take a look at the patch I've submitted?

Hi, I know it's annoying having someone nagging at an issue, but it's been a while and having some kind of solution to this is a blocker for some people using Packer. Is there any way @hanwen or @FiloSottile could take a look at the patch I've submitted?

Would love to see this reviewed, it is a blocker for me with packer.
Thanks!

If I understand correctly, I think in this case the ssh client should be responsible for the signing algorithm negotiation. Because the user may not know if the server configuration will accept the legacy format or not, I don't think think we can place the burden on the user to determine the signing algorithm, plus it's kind of cumbersome since changing the algorithm for most keys isn't valid anyway. The AlgorithmSignerWithAlgo also has the drawback of not working with agent signers, which have been the most common in my experience.
It looks to me like we should be able to easily negotiate the signing algorithm while verifying the allowed public keys in the server, and pass the discovered algorithm along. We probably need to add SignWithAlgorithm to the agent signer, and have some special handling of Certificates to deal with the nist names they have wrapped around the public key types, but I think that might be a better approach to solve the issue transparently for users.

With Fedora 32 gone EOL a couple months ago (the last release Packer worked with in AWS EC2), this issue has become more critical for me.  Happy to help test.

Hey,
I'm on the Git Protocols team at GitHub, and we're responsible for the service that terminates and serves Git connections, including connections over SSH.  We're in the process of planning some changes to the algorithms we support over SSH. Specifically, we're looking into getting rid of most support for SHA-1, since it's dangerously weak.  This includes, tentatively, prohibiting newly added RSA keys from using SHA-1, but allowing existing RSA keys to continue using it.
The Go SSH library is one of the largest clients we see still using RSA with SHA-1, and it would be great if it were possible for it to support RSA with SHA-256 and SHA-512.  I realize that the Go SSH client offers other algorithms which are secure, but users may wish to use RSA keys in the future.
We're still looking at options and we'll announce plans more openly when we're more definitive on them, but we wanted to reach out on this issue and inform you of our current intentions.  Feel free to let me know if you have any questions.

Update on my earlier comment re SHA1 deprecation timelines.
OpenSSH 8.7 was released on 2021-08-20. The release notes indicate:

OpenSSH will disable the ssh-rsa signature scheme by default in the next release.

x/crypto/ssh is failing on a growing list of machines. The next OpenSSH release will make this worse again.

OpenSSH 8.8 was released on 2021-09-26. RSA + SHA-1 has now been disabled by default (release notes):

This release disables RSA signatures using the SHA-1 hash algorithm
by default. This change has been made as the SHA-1 hash algorithm is
cryptographically broken, and it is possible to create chosen-prefix
hash collisions for <USD$50K

SHA-256 and SHA-512 have been supported by SSH servers for many years now. The Go SSH package should default to using one of them to avoid ongoing connectivity failures.

Fundamentally this is an issue with the ssh.Signer type and/or its use in handshake.go.
The issue is that the type ssh.Signer:
type Signer interface {
	// PublicKey returns an associated PublicKey instance.
	PublicKey() PublicKey

	// Sign returns raw signature for the given data. This method
	// will apply the hash specified for the keytype to the data.
	Sign(rand io.Reader, data []byte) (*Signature, error)
}

Doesn't provide a mechanism to say which type of algorithm is supported by this Signer - and its assumed incorrectly that that is the PublicKey type and although the AlgorithmSigner does provide a different signing method it's still not saying which type of Signer it is:
type AlgorithmSigner interface {
	Signer

	// SignWithAlgorithm is like Signer.Sign, but allows specification of a
	// non-default signing algorithm. See the SigAlgo* constants in this
	// package for signature algorithms supported by this package. Callers may
	// pass an empty string for the algorithm in which case the AlgorithmSigner
	// will use its default algorithm.
	SignWithAlgorithm(rand io.Reader, data []byte, algorithm string) (*Signature, error)
}

And so it is not wired into handshake.go.
The Signer interface would actually need to be something like:
type TypedSigner interface {
	Signer

	// Type returns the algorithm type for this signer
	Type() String
}

This would then allow backwards compatibility with the current system and allow handshake.go to be changed to:
		for _, k := range t.hostKeys {
			if typed, ok := k.(TypeSigner); ok {
				msg.ServerHostKeyAlgos = append(
					msg.ServerHostKeyAlgos, typed.Type())
				continue
			}

			msg.ServerHostKeyAlgos = append(
				msg.ServerHostKeyAlgos, k.PublicKey().Type())
		}

A couple of utility types and associated functions would need to be provided - the most simple being one that takes an AlgorithmSigner and creates a TypeSigner for a given algorithm.

A MultipleAlgorithmSigner could be created as below but it would be more complex to wire in:
type MultipleAlgorithmSigner interface {
	AlgorithmSigner

	// Algorithms returns the available algorithms
	Algorithms() []string

	// SignWithAlgorithm is like Signer.Sign, but allows specification of a
	// non-default signing algorithm. See the SigAlgo* constants in this
	// package for signature algorithms supported by this package. Callers may
	// pass an empty string for the algorithm in which case the AlgorithmSigner
	// will use its default algorithm.
	SignWithAlgorithm(rand io.Reader, data []byte, algorithm string) (*Signature, error)
}

My suspicion is that this would be better used as a way of an AlgorithmSigner advertising which TypeSigner they could become rather than being used directly in handshake.go. The idea being that it would be good if there was some programmatic way of listing all of the things could be supported. This could at first be used with some utility function to provide Signers for everything but later cut down to only support specific Signers.
At present there is no programmatic way of determining which algorithms are supported.

@pete-woods:
Mind sharing your patched x/crypto? I have been paying close attention to https://github.com/golang/go/issues/49952 but the progress has been slowed to fully address the issue. I'm planning to use a working forked x/crypto for now for my projects.

Thanks! The test code in https://github.com/golang/go/issues/39885#issuecomment-835233610 now works after upgrading to:
golang.org/x/crypto v0.0.0-20220314234724-5d542ad81a58.

From what I see this is still not working:
$ ssh-add -l
4096 SHA256:m+Hthc93TjF0wcAoq8OyrKZjDl8LE5ddhQwzwnBA02c /home/XXX/.ssh/id_rsa (RSA)
4096 SHA256:m+Hthc93TjF0wcAoq8OyrKZjDl8LE5ddhQwzwnBA02c /home/XXX/.ssh/id_rsa (RSA-CERT)
$ ssh <IP_ADDRESS> uptime
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
 10:47:52 up 423 days, 20:25,  8 users,  load average: 1.15, 0.97, 1.00
$ ssh <IP_ADDRESS> uptime
 10:48:03 up 157 days,  1:37,  0 users,  load average: 1.40, 1.02, 0.73
$ cat 001-ssh-test.go
package main

import (
        "log"
        "net"
        "os"
        "os/user"

        "golang.org/x/crypto/ssh"
        "golang.org/x/crypto/ssh/agent"
)

func logFatal(err error) {
        if err != nil {
                log.Fatal(err)
        }
}

func main() {

        sock, err := net.Dial("unix", os.Getenv("SSH_AUTH_SOCK"))
        logFatal(err)

        signers, err := agent.NewClient(sock).Signers()
        logFatal(err)

        u, err := user.Current()
        logFatal(err)

        cfg := &ssh.ClientConfig{
                User: u.Username,
                Auth: []ssh.AuthMethod{ssh.PublicKeys(signers...)},
                HostKeyCallback: ssh.InsecureIgnoreHostKey(),
        }

        log.Printf("Connecting to %s\n", os.Args[1])
        client, err := ssh.Dial("tcp", os.Args[1], cfg)
        logFatal(err)

        _, err = client.NewSession()
        logFatal(err)

        log.Println("We've got a live session!")
}
$ cat go.mod
module test

require golang.org/x/crypto v0.0.0-20220722155217-630584e8d5aa

require golang.org/x/sys v0.0.0-20220722155257-8c9f86f7a55f // indirect

go 1.18
$ go run 001-ssh-test.go <IP_ADDRESS>:22
2022/07/23 10:48:52 Connecting to <IP_ADDRESS>:22
2022/07/23 10:48:52 We've got a live session!
$ go run 001-ssh-test.go <IP_ADDRESS>:22
2022/07/23 10:49:02 Connecting to <IP_ADDRESS>:22
2022/07/23 10:49:05 ssh: handshake failed: ssh: unable to authenticate, attempted methods [none publickey], no supported methods remain
exit status 1
$

This looks like an issue in the agent package. Do you mind opening this as a new issue with details on the agent and server versions, and tag me in it?

@FiloSottile done: https://github.com/golang/go/issues/54027


Flutter Package?

Hello, just learned about MediaPipe today! It's amazing how many developer resources are open-source. It's also crazy to realize that Google has empowered so much of it. With that being said, I was surprised to see the MediaPipe team has not created a package for Flutter developers since both were founded by Google and the MediaPipe is marketed as cross-platform
System information

MediaPipe Solution: All solutions available on iOS and Android
Programming language: Dart
Are you willing to contribute it: Yes, but my team cannot prioritize this over other tasks.

Will this change the current API?
No. Creating a Flutter package will extend and merge the iOS/Android API such that the new, extended API can be used natively in the Dart language. Nothing about the current API will need to be changed as far as I am concerned.
Who will benefit with this feature?

Developers and teams using the Flutter framework.
Developers who want to build application to multiple platforms with one codebase
Developers who want to do great things with great Google products
Myself 🫶🏼

Hi @charlieforward9,
Thanks for raising this feature request. This looks like an interesting request. We will discuss this feature internally and prioritize it among others.

I have integrated mediapipe into my flutter app but it was really cumbersome. It would be great at least have C api that can be called directly from dart, instead of channel communication separate for each platform.
There is https://github.com/homuler/MediaPipeUnityPlugin/tree/master/mediapipe_api, originally created for Unity but could be repurposed. The problem is - it's not pure C, so generating ffi bindings is not so straightforward,  and it is not official.

@szotp-lc

It would be great at least have C api that can be called directly from dart, instead of channel communication separate for each platform.

If it were done this way, would it work on all the mobile and desktop platforms that Flutter supports? If so, then the only separate implementation would be the web. Asking to learn. Thanks

@charlieforward9 Correct.

Hi! We are having some discussion about this, but cannot provide a timeline at this point. We certainly recognize that better Flutter support is very important to large part of our community. We will provide updates if and when we can.

@schmidt-sebastian Thank you for the response. Please do keep us updated in this thread, there seem to be a few who tagged along with interest based on the emoji response.

@szotp-lc do you mind emailing me at<EMAIL_ADDRESS>I would appreciate hearing more about how you integrated into your Flutter app (unless you didn't mind keeping the conversation in this thread). Thank you!

Honestly I would recommend waiting for official support or using a different library (like ML Kit). It has been a painful process.

Honestly I would recommend waiting for official support or using a different library (like ML Kit).

I just learned all about and built an iOS PoseDetection MethodChannel using MLKit to compare it with my existing pose_detection_full.tflite file embedding using flutter-tflite. The iOS MethodChannel inference delivered slower than the .tflite file embedding with my implementation.
I am sharing this to save others time and keep this issue relevant.
Goal

Actual
https://github.com/google/mediapipe/assets/62311337/26d8be7b-6db4-423e-b54e-04dc3144cc29
My implementation uses Dart Isolates to keep the model processing off of the main thread.

Thanks to Google and their Mediapipe team, we are now able to witness development of the flutter-mediapipe library. It's great to follow along and see how Google would build a library from the ground up.
I have become a watcher of the repository to get a firsthand sight of the progress and changes, so if anyone else lands here and is interested in the progress of the package. I recommend doing the same.

so as of now is it possible to have a mobile application that is developed using flutter that has mediapipe ?

Yes, there is just a lot of manual setup. I recommend using this sample repo to help get started with it.
I didn't need Android config and I am actually moving to native Swift now for better performance. If you manage to get it up and running with all updated dependencies, feel free to create a PR to Open Source it.. I am waiting for the original creator @Jaeheee to merge it, so we can keep my fork current till it merges.
Happy Coding!

As a humbled developer who requested this over a year ago. I have subscribed to the  construction and have been nothing short of impressed.
I am particularly excited to see Pose Detection begin its development as an included feature at https://github.com/google/flutter-mediapipe/issues/51. It was the Mediapipe solution I was having trouble running performantly that prompted me to open this issue.
Thank you Google AI

@charlieforward9 I have been implementing mediapipe-pose using JS but now I want my application to built for multiple platform on one codebase so I have been trying to migrate to flutter, just wanted to ask whether using platform channel solve that problem as of now or is there some alternate

You can see the performance of Method Channels above.
I am really hoping for the official repo to perform closer to the "goal".
I am getting started on the vision package for the team.
This is my first large scale contribution, so I cannot provide a timeline at the time. However, I will continue to update as I proceed.
https://github.com/google/flutter-mediapipe/

Method Channel doesn't seem to have that great performance as compared to the native code

Hi @charlieforward9,
The official Flutter plugin, Flutter Mediapipe, is now available, and the feature has been added. Please let us know if we can consider this issue resolved and close it internally. If you encounter any further issues while using the Flutter Mediapipe plugin, we recommend opening a new issue in the respective repository. We would be happy to assist you with any further concerns.
Thank you!!

Thank you, @kuaashish, for following up. The scope of this issue has not been fully addressed yet. As outlined above, I am requesting that the vision package be made available through Flutter, which remains incomplete.
I’ve been in contact with @craiglabenz regarding contributing to this effort, and my team member, @DrBubbles42, has started preparing to assist. However, there is currently no timeline for achieving the desired functionality. Raising the priority of building out the vision task internally would be incredibly helpful for us.

I was also looking into using flutter-mediapipe, however, there was no pose detection made available, and I am not sure this plugin is still being actively maintained. Is there any other Mediapipe solution for pose that can be used with Flutter?
Thank you.

Hi @charlieforward9,
The official Flutter plugin, Flutter Mediapipe, is now available, and the feature has been added. Please let us know if we can consider this issue resolved and close it internally. If you encounter any further issues while using the Flutter Mediapipe plugin, we recommend opening a new issue in the respective repository. We would be happy to assist you with any further concerns.
Thank you!!

@richardhfj See https://github.com/google-ai-edge/mediapipe/issues/3922#issuecomment-1972492798

@richardhfj See #3922 (comment)

Thank you for the followup. I looked into this, but I see that tflite helper is now deprecated so I am not exactly sure how to use this.

@richardhfj it still works, just not getting new updates, I use it in my projects while I wait for the new package. If youre a real G, build the vision package from the ground up in flutter_mediapipe and use that! I need it too but am too overemployed to get it done rn.


config.schema.json wrong defaults

Hi,
thanks for this great tool! till date, used FFMPEG plugin for my cameras to be integrated into homekit, now with HSV in BETA 1.4.0, tried out your plugin and it's great!
In total, I work with 3 Cameras, of which 2 work under RTSP and integrate easily. Remaining camera operates with MJPEG and this cannot be changed in my setup. FFMPEG allows for this integration and it worked ok on there.
When I try to enter the MJPEG camera into camera.ui on Homebridge, the camera won't surface properly but throw errors on Camera.ui & Homebridge log, speficially regaring -stimeout and -rtsp_transport.
Took a look at the config.json with the assumption that these entries would simply not be recognizes for MJPEG streams so wanted to delete them so they would not be tried to read. That worked for me (so delting -stimeout and -rtsp_transport).
In the plugin config though, rtsp_transport is a mandatory field and -stimeout automatically repopulates with 10.
Would it make sense to make these entries conditional to RTSP streams only and for MJPEG (which works with FFMPEG), do not make these entries to the config.json?
Let me know if you need more/if this helps. Again, great work!
Environment

Node.js Version: v16.13.1

Homebridge Version: v1.4.0. beta 7

Camera UI Version: v5.0.9

Operating System: Raspbian

Hey yes post your config.json pls

Hi,
thanks for this great tool! till date, used FFMPEG plugin for my cameras to be integrated into homekit, now with HSV in BETA 1.4.0, tried out your plugin and it's great!
In total, I work with 3 Cameras, of which 2 work under RTSP and integrate easily. Remaining camera operates with MJPEG and this cannot be changed in my setup. FFMPEG allows for this integration and it worked ok on there.
When I try to enter the MJPEG camera into camera.ui on Homebridge, the camera won't surface properly but throw errors on Camera.ui & Homebridge log, speficially regaring -stimeout and -rtsp_transport.
Took a look at the config.json with the assumption that these entries would simply not be recognizes for MJPEG streams so wanted to delete them so they would not be tried to read. That worked for me (so delting -stimeout and -rtsp_transport).
In the plugin config though, rtsp_transport is a mandatory field and -stimeout automatically repopulates with 10. Would it make sense to make these entries conditional to RTSP streams only and for MJPEG (which works with FFMPEG), do not make these entries to the config.json?
Let me know if you need more/if this helps. Again, great work!
Environment

Node.js Version: v16.13.1
Homebridge Version: v1.4.0. beta 7
Camera UI Version: v5.0.9
Operating System: Raspbian

these fields arent required, if you dont need these field, set "stimeout": 0 in config and remove "rtspTransport" from config

Hi seydx,
true and that's what I did - however when you do the configuration through the homebridge UI and not directly in the config.json:

RTSP Transport shows as mandatory (has the '*') next to the field and overrides my manual deletion if I refresh with the UI config
Socket Timeout seems to override my manual deletion in the config as well. Trying to save with an empty field will add "                        "stimeout": 10," to the config

In short, using the ui to config vs. directly editing in config.json will let the values resurface. and then cause the issues again.

Hi seydx,
true and that's what I did - however when you do the configuration through the homebridge UI and not directly in the config.json:

RTSP Transport shows as mandatory (has the '*') next to the field and overrides my manual deletion if I refresh with the UI config
Socket Timeout seems to override my manual deletion in the config as well. Trying to save with an empty field will add "                        "stimeout": 10," to the config

In short, using the ui to config vs. directly editing in config.json will let the values resurface. and then cause the issues again.

Ah you re right, thank you for the report, i will fix it :)

And thanks to you for the great work :)

fixed in v5.0.10-beta.5

v5.0.10 is online


Total size of repos always 0

I migrated my rootless container podman yaml from tgbyte to your repo just fine.
But checking the logs I see
Mär 22 15:04:37 server borg-backup-server-pod[723937]: Following borg repos are present:
Mär 22 15:04:37 server borg-backup-server-pod[723987]: total 0
Mär 22 15:04:37 server borg-backup-server-pod[723987]: drwx------    1 borg     borg            98 Feb 20 12:58 name-a
Mär 22 15:04:37 server borg-backup-server-pod[723987]: drwx------    1 borg     borg           110 Mar 21 15:03 name-b
Mär 22 15:04:37 server borg-backup-server-pod[723987]: drwx------    1 borg     borg           104 Mar 21 09:31 name-c
Mär 22 15:04:37 server borg-backup-server-pod[723937]: Size of all backups combined: 356.0G        /home/borg/backups

"total 0" :D

Hey there, the total 0 output is part of the ls command. It should AFAIK show the total block count. I will have a look!

:tada: This issue has been resolved in version 1.7.0-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Wallet is constantly Crashing on Windows 2012 R2 64bit

Running Solo mining using core wallet on windows results in constant crashing where we need to restart the wallet to continue mining. Looking for some guidance on how to get past this issue.

Thanks we are in final testing of 10.5.1 https://github.com/BTA-BATA/BATA-SOURCE/releases which addresses the issue.

Solo mining meaning using the wallet itself to mine coins?
This is no longer supported as the computer CPUs will not compete with ASIC mining.

https://beta.hashwiz.io/pool/BTA/
No - I've setup a pool and I am trying to Mine with an ASIC Antminer L3+

OK, good.  Not really solo mining then.
10.4 is tested on mining pool (by me)

Ok cool thanks I will try 10.4 and move back for now see how that goes. I appreciate it!

I have a pool running 10.4 at pool.midnightminer.net

@midnight-miner seems like they don't have a windows binary on 10.4 from what I can tell.

Sorry, try 10.4.1
https://github.com/BTA-BATA/BATA-SOURCE/releases

@midnight-miner I already had tried 10.4.1 and it had given me same issue, but it did work much better than latest version. I've moved back but can't really mine in any capacity because of this.

You can point your miner to pool.midnightminer.net to mine.  only 1% fee.

It's ok, I am working on coding this Mining pool at in the meantime I will Mine another coin to do my testing. I appreciate it!

Are you using a Windows based mining pool?  Most pools and wallets run on Linux.  I have not seen much support for Windows.

Yes, I've taken the source code from a Linux/Windows C# project and improved it. It runs on IIS/.NET Core/MVC/REDIS and has a Windows Service based Stratum Agent that connects to the Pools Portal. I also noticed the same thing with Windows, Figured I would take a stab at changing that fact and bring some Windows Support to the Table. It's based on a source code originally called CoiniumServ. I've probably replaced 70% of the code so far on cleaning it up.

this is not completed, we have same issue with 10.4.1

This is interesting.
Windows testing I only have Win 7, Win 8.1, and Win10.  I am attempting to resurrect a 2008 server for testing, but it is still probably a week away.

@midnight-miner with Windows 2012 R2, it crashes every 12-20 hours on 10.4.1 - with 10.5.1 it doesn't last 20 minutes.

I had issues similar when testing on Mac.
We have committed many changes to my Github account for testing before committing it to the BTA-BATA repo.
I could sent you a test build to try out if your interested.

Happy to do so if that was available.
Sent from my iPhone
On Aug 17, 2017, at 12:44 AM, MidnightMiner <notifications<EMAIL_ADDRESS>wrote:
I had issues similar when testing on Mac.
We have committed many changes to my Github account for testing before committing it to the BTA-BATA repo.
I could sent you a test build to try out if your interested.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/BTA-BATA/BATA-SOURCE/issues/45#issuecomment-322967500, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMHp5pAQyoNL2_zgYo5KN2NB5yX2VYF0ks5sY8UZgaJpZM4O4WzV.

Pre-release binaries are available here:
https://github.com/midnight-miner/BATA-Development/releases/tag/v<IP_ADDRESS>rc1

@midnight-miner So far on this version seems like it's working good! No crash so far in 7 hours I will keep you posted.

@midnight-miner was just looking at your changes, that's actually quite funny that they hadn't done that up to this point lol I guess it's that easy to clone and go a coin. I guess the problem I was having could stem from also having the LTC wallet core running on the same server? My stratum is running multiple Wallets on it so this conflict makes a lot of sense. Good catch!

Closing - Let us know if you experience anymore issues and I will reopen. Thanks.


WIP: ENH: improve _count_paths_outside_method via pythran

Could speed up stats.ks_2samp(rvs1, rvs2, alternative='less', mode='exact') >20x times on my computer if went well with Pythran master
from scipy import stats
import numpy as np
from timeit import timeit

print(stats.__file__)
rng = np.random.default_rng(1234)
n1 = 200
n2 = 300

rvs1 = stats.norm.rvs(size=n1, loc=0., scale=1, random_state=rng)
rvs2 = stats.norm.rvs(size=n2, loc=0.5, scale=1.5, random_state=rng)
stats.ks_2samp(rvs1, rvs2, alternative='less', mode='exact')

CI failures can be ignore, they're due to Pythran master being needed (0.9.12 will be released soon, and it has support for scipy.special.binom that's needed here).
Right now this PR is also blocked by the macOS crash in gh-14315.

Wow, I'm seeing 90x speedup. Once gh-13265 merges, I'll merge main into this PR.
Otherwise, is there anything else holding up this one @rgommers? Looks like the things you mentioned above have been resolved.

This is closely related to gh-13957 so let's continue this work there.

@rgommers @mdhaber Thank you so much!


Charts are sporadically rendering blank

This library has been working well for various chart types (bars, waterfall, scatter) but every now and then, the chart doesn't render. This occurs for both canvas and svg renderers as shown below. I'm not sure if this is specific to the code echarts library and / sveltekit. There are no console errors / warnings either. Occurs in both Safari and Chrome. Any insights on this is appreciated.  It sounds like some reactivity stuff and we'll need to pinpoint where that is happening.

Thank you for your quick reply. I was on echarts 5.5.0 and svelte-echarts 1.0.0-rc1 but have since upgraded to 5.5.1 and 1.0.0-rc3. I'm actually using the example on this repo's readme. Have you by chance encountered this issue before?

I haven't seen this yet in my testing. What browser are you using and do you have any motion preferences? I think the demo has some animations when the data updates every second which could be causing the issues you're seeing, this is the default behavior of echarts. Maybe try disabling animation on the chart.

Thanks, it's the same issue unfortunately in that even with animation set to false, the graph only renders sometimes. This is occurring on the latest versions of Safari and Chrome with and without Incognito / Private modes. All client side libraries are loading too. I do see a warning in the console sporadically now stating: Instance ec_1724027982161 has been disposed but that returns even when the chart does render successfully.

I had a similar issue, but discovered that it was due to how I was binding the echarts instance. I have no idea if that's relevant for you as well, but figured it might be worth explaining what was wrong in my case.
My code looked something like this:
<script lang="ts">
  import { Chart } from 'svelte-echarts';
  import type { EChartsType } from 'echarts';

  export let showChart: boolean;

  let instance: EChartsType;
</script>

{#if showChart}
  <Chart {init} {options} bind:chart={instance} />
{/if}

Now, when we first render everything goes well - the chart is initialized and the echarts instance is bound to the instance prop.
The problem shows when we hide the chart (showChart = false) and then show it again. This time, since instance is already set, and since we're using a two-way binding, we are actually providing the Chart component an echarts instance that has already been disposed, meaning it won't render.
My solution was to avoid the two-way binding. If the svelte-echarts component emitted an event on init, providing the chart instance, it would be even easier to do so. Or perhaps also resetting the chart instance after the dispose call.


Remove lang files

The problems this is trying to solve is laid out here: https://github.com/quisquous/cactbot/issues/1221#issuecomment-602005392
I broke these up into separate commits, which hopefully should be easier to follow.  Probably the most interesting commit is 0d69d57 which creates a new resources/translations.js file (and might be a good home for some new LocaleText in the future) and combines all of the lang regexes in both Regex and NetRegex varieties.
The only thing really left to do is to remove the zones, and that's waiting on #1037.  I moved them out of the lang files just so that I could remove all the includes and variables.
Comments / nits / opinions / bikeshedding welcome.
(27 files changed, 526 insertions(+), 1219 deletions(-) isn't too shabby either.)

@JLGarber @Akurosia @valarnin FYI if you have any thoughts about this

Nothing from me, no. I'm very happy to see these going away. NetRegexes have made oopsy work significantly less frustrating.

Fine from my side as well :)

This looks good to me in general. The only thing I'm not really a fan of is this syntax:
let x = langArray[displayLang] || langArray['en'];

I wonder if it would be better to use a Proxy and have a default return, like this:
const defaultLangHandler = {
  get: function(obj, prop) {
    return prop in obj ?
      obj[prop] :
      obj['en'];
  }
};

const langArray = new Proxy({
  'en': ...,
  'fr': ...,
}, defaultLangHandler);

And then it could be invoked directly as this instead:
let x = langArray[displayLang];

The Proxy stuff is a great suggestion, thank you!.  I didn't know about that.  I think there are a million places in the code where that pattern occurs, so I should probably add some TranslateProxy in a common place and have everything use it.  I'll look at doing that in a follow-up.

Ok, fixed @panicstevenson's thoughts about an unnecessary global in 66a6170.
Thanks for all the feedback.  I wanted to make sure folks who cared about translations knew about this and thought it was good.  :+1:


Chapter 7: cloning GitHub in Jupyter

Hi, I'm on Chapter 7 and this sentence doesn't really make sense to me:

We can then start an SSH tunnel, a Chrome session via the network proxy, and browse to port 8080 on the master node of the cluster. In Jupyter, we then can clone the GitHub repository and start off a new notebook.

I previously went through Chapter 6 without problems, but it isn't clear to me what we're meant to do here. I know how to sign into the master node using SSH in Chrome, which I assume is the "Chrome session via the network proxy." But do you mean that we're supposed to browse to port 8080 within that SSH session somehow? That doesn't quite make sense to me because 8080 isn't an SSH port. What am I missing?
I tried just doing git clone within a Jupyter notebook that I accessed through Web Interfaces as explained in Chapter 6, but git clone isn't valid syntax within a Jupyter notebook. So I'm a bit lost. Thanks for any help you can provide!

This workflow has now become simplified.  Go to https://console.cloud.google.com/ai-platform/notebooks
and start a Dataproc Hub instance to your Dataproc cluster.


MP4Box support

It would be nice to have muxing via MP4Box other than just FFmpeg and avconv, usage is simple using the "-add" option.

Does it support webm videos?
I prefer using a single tool, what are the advantages of mp4box?

MP4Box supports DASH wich I don't think ffmpeg/avconv support unless it was added recently I was under the impression MP4Box supports webm but I just tried and It doesn't seem to.  I guess it wouldn't make sense to add support if it doesnt support webm.  but I do have a hardware media player from japan that sometimes errors out from muxed mp4 videos from ffmpeg untill I remux them with mp4box but thats probably the hardware's fault.

ffmpeg/avconv seem to support well dash files (for example Quicktime can't play the DASH video only file, but if you process it with ffmpeg it can play the final file). You may be interested in the --exec option, which would allow you to remux the video with mp4box.

LibAV/Ffmpeg seems like a better choice, but I figured I'd chime in here with a couple of things.

MP4Box allows similarly formatted videos (e.g. same framerate, resolution) to be concatenated with the -cat option, and videos can be added with separate chapter metadata. E.g.

MP4Box -cat video1.mp4:chapter="First Part" -cat video2.mp4:chapter="Second Part" final.mp4

MP4Box supports embedding subtitles (at least from *.srt) in MP4 streams. For example, to add iTunes-compatible subtitles:

MP4Box -add video1.srt:lang=eng:hldr=sbtl video1.mp4

Concat: ffmpeg supports that for more formats (flv ...), there has been some PR for them but they haven't been merged.

Subtitles: already supported by ffmpeg (not libav), see the --embed-subs option.


[AutoPR datafactory/resource-manager] Modify SecureString to SecretBase in SDK for ExecuteSSISPackage activity in pipeline

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7466

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/6012


Add constructors for UnitMatrix

The construction of Matrix instances with units is cumbersome, and not intuitive. Currently, constructing involves code like:
double[][] data = new double[1000][1000];
for (int i = 0; i < 1000; i++) {
  for (int j = 0; j < 1000; j++) {
    data[i][j] = 9 * i + 2 * j * 0.364;
  }
}
LengthMatrix lengthMatrix = new LengthMatrix(
  DoubleMatrixData.instantiate(data, IdentityScale.SCALE, StorageType.DENSE), LengthUnit.CENTIMETER);

The code could assume that the StorageType is DENSE, since we offer the data as an array. In case we would offer the data as a Map of indices to values, the code could assume that the StorageType is SPARSE. So, the last line should be replaced by:
LengthMatrix lengthMatrix = new LengthMatrix(data, LengthUnit.CENTIMETER);

This would be much more intuitive. The analogous implementation for Vector has already been completed successfully.

All new Matrix constructors have been added and tested. Test coverage is up to 97% again.


React Table Column Resize Broken for columns that specify Cell propery

Is this a general question about implementation, support, api, or anything else?
No This is a bug. A very very frustrating bug.
What version of React-Table are you using?
"react": "^16.4.1", "react-table": "^6.8.6",
npm update react-table results in 6.8.6.
What bug are you experiencing, or what feature are you proposing?
I have a table with several columns.  For some columns I am simply using the accessor to get data by name from the row but for other columns I have some processing which is implemented in Cell: which renders content with some basic decisions about the row data.  Nothing fancy , no resize events being triggered or anything but simply selecting correct icons for row type etc.  I am specifying width on all columns because I had read that there were issues when mixing and matching autosize columns with fixed width.
At any rate, when I resize the columns which use the custom Cell processing the resize event affects ALL columns not just the column I am trying to resize.  When I resize columns which have no Cell processing the table behaves correctly it only resizes the column being resized.
Use https://codesandbox.io/s/X6npLXPRW (by clicking the "Fork" button) to reproduce the issue.
https://codesandbox.io/s/91xm372kxy
What are the steps to reproduce the issue?

Go to the above link: https://codesandbox.io/s/91xm372kxy
Resize the name or created columns (resizing works as expected)
Resize any of the other columns and note that all columns resize not just the column you want to resize.

Also experiencing this issue, or one very similar to it.  In my case, the issue only happens when I attempt to resize the second to last column.  Some of my columns have fixed widths and resizable: false, others have minWidths and are resizable.  It's a mix.  I also am using a custom Cell renderer.
However, even when I take out the custom cell renderer, I am still able to get incorrect resize behavior.  I have pasted an example below that causes this weird behavior.  It's a very simple example and you don't even need to pass in any data to reproduce.
<ReactTable
    data={[]]}
    columns={[
    {
      width: 30,
      resizable: false,
    }, {
      Header: 'Field A',
      width: 55,
      resizable: false,
    }, {
      Header: 'Field B',
      minWidth: 30,
    }, {
      Header: 'Field C',
      minWidth: 30,
    }, {
      Header: 'Field D',
      minWidth: 55,
    }, {
      Header: 'Field E',
      resizable: false,
      width: 90,
    },
  ]}
    minRows={5}
    showPagination={false}
    sortable={false}
    NoDataComponent={() => null}
  />

This feature/issue has been tagged as one that will likely be fixed or improved in the next major release of React-Table. This release is expected to be completed in the next 6 - 8 months. Please see #1143 for more detail. If you believe this feature can be easily fixed, we invite you to fork the project, attempt the fix, and use it until the latest version is available. If the fix is simple enough, we will also consider it as a PR to the current version.

for anyone stumble upon here through google, the issue OP has is that he not specify id for some column, and has duplicated id for others. Set id uniquely for each will resolve the problem

hey, a year passed, I found this bug is still here.

Yeah we don’t use it anymore!
On Wed, Jun 19, 2019 at 9:49 PM lqforgithub<EMAIL_ADDRESS>wrote:

hey, a year passed, I found this bug is still here.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/issues/1084?email_source=notifications&email_token=AAACQXKY7FMQLWTQI4GH3LDP3LV4RA5CNFSM4FODX2LKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYD25UA#issuecomment-503819984,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAACQXLOTGYCVRBMG3ZBYCDP3LV4RANCNFSM4FODX2LA
.

--
Ruby: http://blog.rubygeek.com - http://www.twitter.com/rubygeek
http://www.linkedin.com/in/nolastowe - my linkedin profile
http://github.com/rubygeek - my code

fixed it by adding correctly the ids of every column

fixed it by adding correctly the ids of every column

How? Can you add code snippet

fixed it by adding correctly the ids of every column

How? Can you add code snippet?

i dont have the project anymore, in the columns definition i added the id prop to every column, i guess it uses the id to calculate the columns width on resize

fixed it by adding correctly the ids of every column

How? Can you add code snippet?

i dont have the project anymore, in the columns definition i added the id prop to every column, i guess it uses the id to calculate the columns width on resize

thanks!


TextNode replace space to entity

when text has multi spaces, html only shows one space.

\u00A0 - is non-breaking space. That means this PR will break text wrapping. I have merged another fix to master.


Would you release C-Net model?

Thank you for your amazing job~ Would you have plan to release the C-Net model weight or training script?

Hi, currently we do not have a plan for the C-Net part. In fact, C-Net only provides minor improvement upon the naive average scheme for multi-image reconstruction.

Thank you for your quick reply~

Hi @YuDeng , Thank you for sharing your great work.
I'm just new to this area. How can I do model inferencing for the multi-image case? If you could provide some example code would be much appreciated.
Thanks


iRODS client socket timeouts on writes (jargon)

Lately several ingests have failed due to socket timeout. The jargon client inside of Fedora is waiting for a response from the server, but fails to get one within the normal 2 minutes timeout. There are any number of reasons for why the server might be slow, from replication to tape, to other kinds of IO blocking. I am fairly certain that the delay is not due to load/cpu on the iRODS server itself.
I am testing a patch to low-level storage which will increase this timeout to 5 minutes. I'd expect iRODS to finish long before then in most cases and reply. I'm also adding a special catch clause for socket timeout on iRODS output stream close. This is a particular operation that seems to take a long time. If it times out, there is still a chance that we can proceed. Since we do checksum comparison against iRODS for all write operations, any problems will still be detected.
This is in testing this afternoon on DEV.

Example stack trace:
Caused by: java.io.IOException: JargonException caught in constructor, rethrow as IOException
at org.irods.jargon.core.pub.io.IRODSFileOutputStream.close(IRODSFileOutputStream.java:140) [jargon-core-3.0.1-20111012.172326-5.jar:na]
at java.io.FilterOutputStream.close(FilterOutputStream.java:143) [na:1.6.0_29]
at fedorax.server.module.storage.lowlevel.irods.IrodsIFileSystem.stream2streamCopy(IrodsIFileSystem.java:110) [fcrepo-irods-storage-2.1.jar:na]
at fedorax.server.module.storage.lowlevel.irods.IrodsIFileSystem.write(IrodsIFileSystem.java:356) [fcrepo-irods-storage-2.1.jar:na]
... 62 common frames omitted
Caused by: org.irods.jargon.core.exception.JargonException: java.net.SocketTimeoutException: Read timed out
at org.irods.jargon.core.connection.IRODSCommands.readHeaderLength(IRODSCommands.java:954) [jargon-core-3.0.1-20111012.172326-5.jar:na]
at org.irods.jargon.core.connection.IRODSCommands.readHeader(IRODSCommands.java:825) [jargon-core-3.0.1-20111012.172326-5.jar:na]
at org.irods.jargon.core.connection.IRODSCommands.readMessage(IRODSCommands.java:636) [jargon-core-3.0.1-20111012.172326-5.jar:na]
at org.irods.jargon.core.connection.IRODSCommands.readMessage(IRODSCommands.java:624) [jargon-core-3.0.1-20111012.172326-5.jar:na]
at org.irods.jargon.core.connection.IRODSCommands.irodsFunction(IRODSCommands.java:258) [jargon-core-3.0.1-20111012.172326-5.jar:na]
at org.irods.jargon.core.connection.IRODSCommands.irodsFunction(IRODSCommands.java:192) [jargon-core-3.0.1-20111012.172326-5.jar:na]
at org.irods.jargon.core.pub.IRODSFileSystemAOImpl.fileClose(IRODSFileSystemAOImpl.java:1414) [jargon-core-3.0.1-20111012.172326-5.jar:na]
at org.irods.jargon.core.pub.io.IRODSFileImpl.close(IRODSFileImpl.java:1337) [jargon-core-3.0.1-20111012.172326-5.jar:na]
at org.irods.jargon.core.pub.io.IRODSFileOutputStream.close(IRODSFileOutputStream.java:136) [jargon-core-3.0.1-20111012.172326-5.jar:na]
... 65 common frames omitted
Caused by: java.net.SocketTimeoutException: Read timed out
at java.net.SocketInputStream.socketRead0(Native Method) [na:1.6.0_29]
at java.net.SocketInputStream.read(SocketInputStream.java:129) [na:1.6.0_29]
at java.io.BufferedInputStream.fill(BufferedInputStream.java:218) [na:1.6.0_29]
at java.io.BufferedInputStream.read1(BufferedInputStream.java:258) [na:1.6.0_29]
at java.io.BufferedInputStream.read(BufferedInputStream.java:317) [na:1.6.0_29]
at org.irods.jargon.core.connection.IRODSConnection.read(IRODSConnection.java:730) [jargon-core-3.0.1-20111012.172326-5.jar:na]

Ultimately this issue is best resolved via async replication. The timeout does not occur during normal writing of a file to iRODS. It happens after the series of write operations complete, during the close operation. The replication to tape on the iRODS server-side happens during close and there are no incremental replies to the client while it happens. So if replication takes long enough, the client will timeout waiting for a reply..
Periodic replication is better anyhow, since reattempts can be made automatically.
For now this issue is resolved by catching the timeout on the client side and moving on. Checksum verification ensures that complete files are written.


Text is not rendered vertically centered on each line, but is rendered too high up

Describe the bug
Text is not rendered at the vertical center of each line, but is instead rendered near the top:

Here's how it looks in xfce4-terminal:

Font used in both pictures: Liberation Mono (size 10 in terminal, h14 in Neovide)
Desktop (please complete the following information):

OS: Arch Linux
Neovide Version: main branch (b0222d206e6c0775d0f75a470bdca355251e7713)
Neovim Version master branch (v0.5.0-dev+1459-g384f9870f)

Please run neovide --log and paste the contents of the .log file here:
https://gist.githubusercontent.com/Hubro/d53c69b04bb71035accc021790ffe1db/raw/066de6b886f69fa1bac5642660787bee5bfcb8f8/neovide_rCURRENT.log

This is a known issue. I haven't quite figured out what combination of metrics gets the desired effect from swash.

https://github.com/Kethku/neovide/pull/795 Could folks try this branch? It seems to act closer to what I expect, but I'm not sure if its the right fix.

Yes, that is much better but not perfect yet:

(Look at the parentheses)

Can you grab a comparison screen cap? It's hard to tell what the difference is

Yeah, the difference is smaller than I made it out to be ;) But at least the top of the parentheses no longer touch the border of the cell. Here's the same on main:

Ah sorry I meant a comparison to a terminal ground truth

@Kethku
Terminal ground truth (Alacritty, Liberation Mono 10pt):

Neovide gd2e33b8 (Liberation Mono h14):

Seems to consistently draw everything 1px too high up on the line.
Neovide vertical-positioning-fix (Liberation Mono h14):

Neovide now draws the text positioned more or less identical to Alacritty! I tested this and got the same results with Roboto Mono and Source Code Pro as well. It definitely seems to me like this patch fixes the vertical positioning issue.

This is merged, so I think this is fixed.


Asset Service: Generate icons for assets without an icon.

IMPORTANT: NO BOUNTIES WILL BE PAID FROM 2/5 - 2/14 - IF YOU COMPLETE THIS BOUNTY DURING THIS TIMEFRAME WE WILL REMIT PAYMENT ON 2/14
Background
We want to display icons for assets that do not have a remote icon resource. In the asset-service package, we go out to third parties (coingecko, coincap, etc.) to collect a list of assets and store those assets in a json file within the package. During this asset generation, we store a URL for an asset's icon, then on display, we reach out to that remote resource to download the icon into the app.
For assets without a remote resource, we want to be able to generate an icon for display with that asset. We can use identicon, or gravatar or a library to generate the icon.
AC:

Add a module to asset-service to deterministically generate an icon based on the caip19 of an asset.

During rendering in web, if the asset does not have a remote resource icon (property icon on the Asset type), use the new module to generate the icon on the fly and display that icon.

The display of the icon should include up to 5 characters of the asset's symbol.

Example assets currently in the app that do not have an icon.
https://etherscan.io/token/0x30ceCB5461A449A90081F5a5F55db4e048397BAB
https://etherscan.io/token/0xd51e852630debc24e9e1041a03d80a0107f8ef0c
https://etherscan.io/token/0x90528aeb3a2b736b780fd1b6c478bb7e1d643170
Bounty Documentation
Please check out our docs for how to get in touch with us or ask questions.

Do we really need a module for this?
It could be a React component with a simple SVG, and an url props, if the url is set => display a simple image, else => use 5 first letter of the asset as a prop inside the SVG
If you feel I'm right, this issue can be moved to the web repository

And of course, I would love to work on this issue :D

I would like to work on this, the original issue for this (https://github.com/shapeshift/web/issues/959) was also assigned to me

We would like to have the icon generation module in lib/asset-service so that it can be reused outside of web. There will be web code needed to implement the module.

We would like to have the icon generation module in lib/asset-service so that it can be reused outside of web. There will be web code needed to implement the module.

@theoboldfrazier we can do this using something like canvas then, I can implement that

closed in https://github.com/shapeshift/lib/pull/400


Gcp start stop

@doktoric - review
GCP start/stop functionality - upgraded gcc-init script for handling issues on restart (upgrade /etc/hosts in containers, restart them, register ambari for consul, restart ambari server/agents)

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/737/

for this, need to upgrade disk_mount.sh in the image:
https://gist.githubusercontent.com/oleewere/405dd6cdc9b08e7d22e7/raw/ead85ba2eb63e271d12625b8405c791ee38f9e7d/gistfile1.txt

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/743/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/744/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/755/

When the new image is ready I will merge it.

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/778/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/779/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/780/