use MEMCPY instead of memcpy when the copied size is known

MEMCPY stands for __builtin_memcpy when that is supported by the compiler.
This patch systematically changes calls to memcpy to MEMCPY when the size of
the copied memory is known at compile time. This allows the compilers to
better optimize the code.

The change of the hardcoded builtin_memcpy is something I fully agree on, we should not hardcode the builtin.
The concept of changing from stock memcpy in general however..?
IIRC, GCC will auto-replace memcpy calls with its builtin if:

size is known
size is small enough to profit from it
optimization is enabled
compiler builtins are enabled

So I don't see why we hardcode the builtin? Does LLVM not do the same?
I made an experiment where i benchmarked this patch and one where I override MEMCPY and MEMSET in zutil.h to be the stock ones always, and even with several rounds of testing each I am unable to measure any performance change at all. If there is one, it is below 0.1% (that is approx what my benchmarks can vary from run to run).
Even when only looking at the fastest run out of the 490 of each, stock memcpy/memcmp is the winner (by an extremely minimal margin).

You are right.
I will send an amended patch that replaces MEMCPY with memcpy and that also fixes the __builtin_memcpy.


Add as_dict method to Document.

Add as_dict method to Document.
Add test.
Update docstring in extract_metadata to fix docs.
Fixes #298.

Codecov Report

Merging #306 (59394f3) into master (6d8fb8d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master     #306   +/-   ##
=======================================
  Coverage   96.01%   96.02%
=======================================
  Files          21       21
  Lines        3240     3242    +2
=======================================
+ Hits         3111     3113    +2
  Misses        129      129

Impacted Files
Coverage Δ

trafilatura/metadata.py
98.41% <100.00%> (+0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Thanks!


"Running the contributed command:'gitlens.diffWithPrevious' failed." with keybinding not working

GitLens Version: 8.2.4
VSCode Version: Version 1.24.0-insider (1.24.0-insider)
OS Version: 10.13.4

Steps to Reproduce:

Disable all extensions, run it with the insiders build with only Gitlens enabled.
In your settings.json only keep the "gitlens.keymap": "alternate", setting.
Try to run the keyboard shortcut alt+,
See the bug

Thanks for the report, it looks like a recent change to the insiders build broke me: https://github.com/Microsoft/vscode/issues/42851

FYI, I've released a beta version of the upcoming 8.3 release here with this fixed: https://github.com/eamodio/vscode-gitlens/releases/tag/v8.3.0-beta


Helidon MP VSCode extension

It makes sense adding it as a part of MicroProfile Extension Pack
https://github.com/MicroShed/vscode-microprofile-pack

Glad to add Helidon to the list.  Feel free to submit a PR directly at the repo mentioned above or file an issue when ready.


New Sparse format 'dps'

Create a New Sparse format 'dps' : Dense Pretending to be Sparse
The first commit adds 2d dps array that passes tests.
The goal of this PR is to extend that to a 1-D sparse array.
Bigger goal is to make a 1-D coo sparse array.

Thanks @dschult. I'm missing a bit of context and maybe it's too early to comment since this is still draft (if so, just let me know). Seeing what's here now, the main question I have is whether this is mostly useful for testing, or for anything else. It's hard to imagine wanting to use this in production code over regular numpy arrays. So if it's mostly testing, consider keeping dps private.


Wrap LLVM Intrinsics

The infrastructure to generate Calls is all here in this patch.  However, the interface makes no guarantees of type safety, and LLVM overloads its intrinsics, so we have a real chance to provide just the subset of commented-out calls, do some type checks, and a little introspection to dispatch to the right intrinsic overload.
Basically, what should the typesafe API look like, if any?

Instead of using precondition it might be best performance-wise to use assert since it is omitted in release optimized code.

Good catch.  I'm also considering writing a tool that generates these definitions from llvm/include/llvm/IR/Intrinsics.td.

trill-lang/TableGenSwift

I'm hoping it won't come to reimplementing TableGen.  e.g. Huon has a little tool in Rust that parses out the different definitions from the td files.

I mean, I think it’d be fine to write a TableGen backend in C++ as a utility here.

The first draft of the tool has been committed.

Oh wow, of course I'm having trouble.  Nobody took the time to properly expose the getters for these types.

Submitted https://reviews.llvm.org/D38485 to track work getting these getters in.  In the mean time I can @_silgen_name("LLVMTokenTypeInContext") but we're SOL for LLVMMetadataType at the moment.

#112 Disables these in the mean time and will allow me to remove these hacks.

Does this mean you won't be able to use intrinsics including LLVMSwift via swiftpm?

In the sense that it won't be automatic, yeah.

Revision just got accepted.  I think this can be parked until the next point release so this API is actually usable.

5.0.1 is out and I’m told we have metadata and token type initializers in which officially unblocks this patch.

We missed the boat for 5.0.1.  Holding for 6.0 in the spring.

Build will be restarted once #127 gets scheduled.


No way to skip CUDA installation in a CUDA capable unit

There is no way to skip CUDA installation here are a couple of reasons you would want to do that:

Deploying k8s on LXD on a local machine. CUDA pull in few GBs of dependencies times the number of workers. This makes deploying k8s on a local machine rather impractical.
Deploy k8s in a network restricted environment. The nvidia-cuda layer tries to access the web.
For some reason the external dependency of CUDA fails, leaving us broken.

The only valid reason, imo, is the first one. The other two are known issues and should be addressed independently.
I think an enable-gpu flag with an auto value and false/off/no for not would be the right path to address this. We should then have conjure-up set that to off for localhost.

Closed via https://github.com/juju-solutions/layer-nvidia-cuda/pull/4


withEntitiesRemoteScrollPagination doesn't clear cache correctly

The clearEntitiesCache function in withEntitiesRemoteScrollPagination only resets total and hasMore.
Whereas the clearEntitiesCache function in withEntitiesRemotePagination resets all of the pagination including the currentPage and requestPage.
As a result, when the filter or remote sort changes and the entities are removed, the pagination has the incorrect state.

will sort this soon

@Toeler you can use 17.9.1 it has the fixes you need

Thanks, you didn't have to but it is most appreciated!


bug: Overflow Scroll Div - can't scroll on ionic rc0

Type: bug
Platform: android 4.4 webview
In beta 14 all works fine but now i can't scroll.

....Content....

Greetings @do-web!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

hey @do-web is your app by chance a side menu project?

I noticed the same problem. Our app is not a side menu project.

I think this is a duplicate of issue 3228. Reverting the changes of commit a5881eaeba7d12d016fe3712c77261e75e863cf3 solves this issue.

no its just a simple div container that i have created

@do-web it seems like it could still be the same underlying issue even if you don't use ion-content. I would check if your scroll event hits the ev.srcEvent.preventDefault(); line added in commit a5881eaeba7d12d016fe3712c77261e75e863cf3 (in which case it is a duplicate of issue #3228).

Closing this as a duplicate, that commit needs to be refactored. It broke too much.


Godot doesn't show any error when dividing by 0

Godot version:

4.0.dev.custom_build.93b50a62e
Issue description:

After GDScript rewriting 24 / 0 doesn't show any error about division by 0, neither in script editor and in runtime.


Add zp no degrad dequant

Type of Change
feature or bug fix or documentation or others
API changed or not
Description
detail description
Issues: xxx
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

you should raise the PR to IPEX too to avoid a huge PR again

you should raise the PR to IPEX too to avoid a huge PR again

That's why I made #311 and its corresponding IPEX PR to split the BF16 part. The "zp no degrad" part is WIP and blocked by dependencies anyway...


feat: Improve error message for typeError

Description
This PR modifies the TypeError message (cannot read property of undefined), replacing it with a much easier-to-understand message.
PR fixes following issue(s)
Fixes #27753
Media
Case 1: TypeError in property pane binding

Case 2: Cascaded error (JSObject.myFun2 executes JSObject.myFun1, and JSObject.myFun1 has a typeError )

Case 3: Better error when action is executed in a data field

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test

/build-deploy-preview skip-tests=true

/ok-to-test

/build-deploy-preview skip-tests=true

/ok-to-test

/build-deploy-preview skip-tests=true

/ok-to-test

/ok-to-test


feat: add slack service

Codecov Report

Merging #67 (4332b33) into master (46cb930) will increase coverage by 1.33%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
+ Coverage   89.70%   91.04%   +1.33%
==========================================
  Files          19       20       +1
  Lines         408      469      +61
  Branches       67       80      +13
==========================================
+ Hits          366      427      +61
  Misses         42       42

Flag
Coverage Δ

unittests
91.04% <100.00%> (+1.33%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/services/slack-service.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Decryption key format

Hi!
For those who get problems when decrypting:
when your key is 32 digits like D658345DF620A2839E5AD6A6D1B85135
then you have to fill each pair of the key into the 0x00 format... like:
0xD6, 0x58, 0x34, 0x5D, 0xF6, 0x20, 0xA2, 0x83, 0x9E, 0x5A, 0xD6, 0xA6, 0xD1, 0xB8, 0x51, 0x35
Have fun!

ok looks like the data is truncated as there is no "16" (Mbus Stop) at the end.

@valki2
So your code would be...
...
uint8_t key[] = {0xD6, 0x58, 0x34, 0x5D, 0xF6, 0x20, 0xA2, 0x83, 0x9E, 0x5A, 0xD6, 0xA6, 0xD1, 0xB8, 0x51, 0x35};
dlms_meter->set_key(key, 32; // Pass your decryption key and key length here
...
?

@ert666 no, it should be dlms_meter->set_key(key, 16); because there are 16 bytes.


Confusing deploy status

I noticed the following whilst merging a few pull request this morning:

This seems to happen when we have created a Heroku review app for a given pull request. We end up with a message from our Shields bot saying it hasn't yet been deployed (i.e. not live on our production servers) and a message from GitHub saying it has been deployed (i.e. only Heroku).
Any thoughts on this matter? Does anyone have ideas on how to solve or improve this?

Does anyone have ideas on how to solve or improve this?

GitHub has to be 'aware' of the heroku deploys and has no way to know if we've SSHed into a server somewhere, so there probably isn't a great solution to this with our current setup.
Just for lolz, next time we merge one we've deployed to staging, lets try removing the review app in heroku before we merge it and see what happens when we merge it..

Having done a bit more investigation on this, I don't think this is very important to worry about because the deployment status is only visible to maintainers. If I view a PR like https://github.com/badges/shields/pull/2192 and I'm either logged out or logged in as a user who can't push to this repo, the GH deployment status card isn't visible. As such I don't reckon we need to care to much..

Indeed I didn't realise that it only showed to us maintainers. I agree that we don't need to care much, therefore I'll close this issue. 😉


sentence-transformers not installed

I just followed up the quickstart, created the venv, and installed the requirements.txt file
As soon as you launch it and start prompting you get an error with sentence-transformers not installed.
So I installed it with: pip install sentence-transformers but ended with a plethora of dependencies installed that are irrelevant to the case such as CUDA (I don't even have an NVIDIA GPU), safetensors, scikitlearn and huggingface-hub. If the models run on Bedrock and this is just APIs, there is something wrong somewhere.
Thanks!
pip install sentence-transformers
...
...
...
Successfully installed huggingface-hub-0.22.2 joblib-1.3.2 mpmath-1.3.0 networkx-3.2.1 nvidia-cublas-cu12-<IP_ADDRESS> nvidia-cuda-cupti-cu12-12.1.105 nvidia-cuda-nvrtc-cu12-12.1.105 nvidia-cuda-runtime-cu12-12.1.105 nvidia-cudnn-cu12-<IP_ADDRESS> nvidia-cufft-cu12-<IP_ADDRESS> nvidia-curand-cu12-<IP_ADDRESS> nvidia-cusolver-cu12-<IP_ADDRESS> nvidia-cusparse-cu12-<IP_ADDRESS> nvidia-nccl-cu12-2.19.3 nvidia-nvjitlink-cu12-12.4.99 nvidia-nvtx-cu12-12.1.105 safetensors-0.4.2 scikit-learn-1.4.1.post1 scipy-1.12.0 sentence-transformers-2.6.1 sympy-1.12 threadpoolctl-3.4.0 tokenizers-0.15.2 torch-2.2.2 transformers-4.39.2 triton-2.2.0

And after that, it asked me to install chromadb.

Hey @carlosbravoa , do you mind sharing what sample you are trying to run when you encountered this issue?

i tried the amazon-bedrock-chat-poc, it works smoothly on M1 with my miniconda env. you'd better to point out which example you tried.  @carlosbravoa

Hey @carlosbravoa can you try pulling down the latest version of the repo, I believe I addressed this issue :)
Just to clarify the reason you need sentence transformers and chromadb is for two similar but different things. Some of the PoCs are using a dynamic prompting technique, leveraging a semantic similarity search based on a database of stored prompts.
When the app is initially started, it uses sentence-transformers and pulls down an embeddings model from hugging face. This model is then used to create embeddings of all your sample prompts, and stores them in a vector database (ChromaDB) in memory.
Each time you make a query, an embedding of your query is created, you do a KNN search against chromaDB, and it returns the three most semantically similar prompts, and injects those into the final prompt along with your question into Bedrock.
If you wanted, you could replace the hugging face embeddings model with an amazon bedrock based embeddings model, but since the one provided is free, we thought it was the best fit.
Hopefully this clarifies things!


Decode script does not generate 1Best.ctm

Hi, thank you for open sourcing this project.
I am trying to run the nnet3_online tdnn model. On running the decode.sh script, I get the following output
Argument filename.wav is a sound file, using it as audio
Diarization completed in 0:00:47 (CPU: 0:00:40), Memory used: 117 MB
Split 1 source file into 103 segments
Duration of speech: 0h:10m:23s
NNet3 decoding completed in 0:00:00 (CPU: 0:00:00), Memory used: 4 MB
Rescoring completed in 0:00:00 (CPU: 0:00:00), Memory used: 0 MB
Rescoring completed in 0:00:00 (CPU: 0:00:00), Memory used: 2 MB
Done

However  does not contain 1Best.ctm results file. Any help is appreciated.

same problem here, I'm using WSL, everything seems to be fine, but no Best1.cms
command
./decode_PR.sh /mnt/e/filtered_short.mp3 output/3 Argument /mnt/e/filtered_short.mp3 is a sound file, using it as audio [                                                  ] Diarization (0/1) Diarization completed in 0:01:13 (CPU: 0:01:34), Memory used: 589 MB Split 1 source file into 361 segments Duration of speech: 0h:50m:0s NNet3 decoding completed in 0:00:00 (CPU: 0:00:01), Memory used: 5 MB Rescoring completed in 0:00:00 (CPU: 0:00:00), Memory used: 0 MB Rescoring completed in 0:00:00 (CPU: 0:00:00), Memory used: 1 MB Done
log
./decode_PR.sh /mnt/e/filtered_short.mp3 output/3 cat: 'output/3/intermediate/data/*.stm': No such file or directory fix_data_dir.sh: kept all 361 utterances. fix_data_dir.sh: old files are kept in output/3/intermediate/data/ALL/.backup steps/make_mfcc.sh --nj 8 --mfcc-config output/3/intermediate/mfcc.conf output/3/intermediate/data/ALL output/3/intermediate/data/ALL/log output/3/intermediate/mfcc Usage: utils/validate_data_dir.sh [--no-feats] [--no-text] [--non-print] [--no-wav] [--no-spk-sort] <data-dir> The --no-xxx options mean that the script does not require  xxx.scp to be present, but it will check it if it is present. --no-spk-sort means that the script does not require the utt2spk to be  sorted by the speaker-id in addition to being sorted by utterance-id. --non-print ignore the presence of non-printable characters. By default, utt2spk is expected to be sorted by both, which can be  achieved by making the speaker-id prefixes of the utterance-ids e.g.: utils/validate_data_dir.sh data/train steps/compute_cmvn_stats.sh output/3/intermediate/data/ALL output/3/intermediate/data/ALL/log output/3/intermediate/mfcc steps/compute_cmvn_stats.sh: no such file output/3/intermediate/data/ALL/feats.scp steps/online/nnet3/decode.sh --nj 8 --acwt 1.2 --post-decode-acwt 10.0 --skip-scoring true models/AM/graph_PR output/3/intermediate/data/ALL output/3/tmp/tmp.Xzb6VJ5zsx filter_scps.pl: warning: some input lines were output to multiple files [OK if splitting per utt]  run.pl: 8 / 8 failed, log is in output/3/tmp/tmp.Xzb6VJ5zsx/log/decode.*.log steps/lmrescore_const_arpa.sh --skip-scoring true models/Lang_PR models/LM/Const_PR output/3/intermediate/data/ALL output/3/intermediate/decode output/3/intermediate/rescore ls: cannot access 'output/3/intermediate/decode/lat.*.gz': No such file or directory steps/lmrescore_const_arpa.sh: No lattices input directory output/3/intermediate/decode

Sorry for the long delay! I'm investigating these issues now.

I have just released Kaldi-NL v0.3 which should hopefully fix this problem. Please let me know if it still persists.


Enhacement: Add a new option to not show the "_text" option at all

We can introduce an option to directly show the corresponding XML value instead of showing it under _text.
Eg.
XML:
<key1>value1</key1>
<key2>value2</key2>

COMPACT JS-OBJECT:
{
    key1: {_text: "value1"},
    key2: {_text: "value2"}
}

With the new option:
{
    key1: "value1",
    key2: "value2"
}

For my own usage, I used this recursive workaround for the same, hence thought for the enhancement : https://github.com/noveens/js-owncloud-client/pull/19/files#diff-b9b823e719730e39048dffbb512e743cR777
Thanks.

@noveens What would be the result if the xml is:
<key1>
    value1
    <key2>value2</key2>
</key1>

This kind of structure is common in html.

Similarly, what would be the output if the xml is:
<key1 attr1="yes">
    value1
</key1>

I wanted to support your syntax when I first designed this library but I could not come up with a clear answer for these situations. I welcome any suggestion on these.

@nashwaan ,
Agreed, there is no workaround for these situations.


fix iOS build that was broken by validateBuildEnvironment.js

add ANDROID_HOME check
update validateBuildEnvironment.js to handle crashes better

This change is

PR is added to the queue for testing as 2. task. (0805991)

PR is added to the queue for testing as 3. task. (d664ac4)

Test (Fail) 91175308 build is completed (0805991)
See https://github.com/ThaliTester/TestResults/tree/911753080805991_fix_iOS_build_that_was_broken_by_validateBuildEnvironment_js_larryonoff/ for the logs

Test (Fail) 91175308 build is completed (d664ac4)
See https://github.com/ThaliTester/TestResults/tree/91175308d664ac4_fix_iOS_build_that_was_broken_by_validateBuildEnvironment_js_larryonoff/ for the logs

@yaronyg please review.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Possible battle optimization on first Mash fight.

Buster -> Empty -> Quick should save one turn over left->right.

Fixed in master by @birdievera.


JCF Select Doesn't work for GUID as ID

We are using the JCF control as part of a project and the value is a GUID not an integer?  The control doesn't set the selected item?  How can this be corrected to work with GUID values?

Hi, @ntegral !
Can you provide more information about which control was used (select, checkbox, radio, etc) and GUID value example?

The select. For example the id= 6ea01f73-0532-4700-a351-831fab201a58 and the value = USA. Each country would have a different guid I'd.
Sent from my iPhone
On Jul 9, 2015, at 2:50 AM, Sergey <notifications<EMAIL_ADDRESS>wrote:
Hi, @ntegralhttps://github.com/ntegral !
Can you provide more information about which control was used (select, checkbox, radio, etc) and GUID value example?

Reply to this email directly or view it on GitHubhttps://github.com/w3co/jcf/issues/8#issuecomment-119843733.

Component source code

Sample Json data:
{"stateProvinces":[{"label":"Alabama","value":"Alabama","id":"{EEF3C45E-72F9-DE11-BC84-0018FE280F2B}"},{"label":"Alaska","value":"Alaska","id":"{EFF3C45E-72F9-DE11-BC84-0018FE280F2B}"},{"label":"American Samoa","value":"American Samoa","id":"{F0F3C45E-72F9-DE11-BC84-0018FE280F2B}"},{"label":"Arizona","value":"Arizona","id":"{F1F3C45E-72F9-DE11-BC84-0018FE280F2B}"},{"label":"Arkansas","value":"Arkansas","id":"{F2F3C45E-72F9-DE11-BC84-0018FE280F2B}"},{"label":"Armed Forces Americas","value":"Armed Forces Americas","id":"{62F5B3F4-F338-DF11-B6BB-0018FE280F2B}"},{"label":"Armed Forces Europe","value":"Armed Forces Europe","id":"{63F5B3F4-F338-DF11-B6BB-0018FE280F2B}"}]}
The selected item on the select is not being updated.  The selected text is being updated.  However the selected attribute is not being updated.
<EMAIL_ADDRESS>Assistance on how to get this working would be greatly appreciated.
Dexter
From: Sergey<EMAIL_ADDRESS>Sent: Thursday, July 9, 2015 2:51 AM
To: w3co/jcf
Cc: Dexter Hardy
Subject: Re: [jcf] JCF Select Doesn't work for GUID as ID (#8)
Hi, @ntegralhttps://github.com/ntegral !
Can you provide more information about which control was used (select, checkbox, radio, etc) and GUID value example?
—
Reply to this email directly or view it on GitHubhttps://github.com/w3co/jcf/issues/8#issuecomment-119843733.

Ok. I've created plnkr to test the issue, but I don't see any problems. Do you use jcf.angular.js directive and jcf attribute on <select> tag? Or you're trying to customize form elements with jQuery and jcf.replaceAll() method?
To make JCF work properly with Angular you'll have to use a directive, not jcf.replaceAll() method (see my example link above).

@ntegral if that doesn't work for you please post more information here.


Improve scrolling behavior

Auto-scrolling is a bit broken since it keeps moving as tokens get streamed which prevent users from reading the text at the beginning of the message if streaming is fast.
Investigate what other platforms are doing (chatGPT, Claude, Anthropic), then define and implement a better UX.
Our current belief is that the best experience would be:
As the new agent message appear we scroll to it at a distance from top roughly equal to a 2 liner user message
Let the model generate but don't auto-scroll
it's more challenging than it looks to properly implement.

@spolu Down to pick up that one. Should we resume work from this PR: https://github.com/dust-tt/dust/pull/5690 ?

Yessir !


Search: Inconsistent results between local and Heroku ElasticSearch

Comparision between local dev machine and Heroku:
Local development machine, with ElasticSearch installed with Homebrew. No special settings.
In [11]: from haystack.query import SearchQuerySet, Raw
In [12]: SearchQuerySet().all().count()
Out[12]: 17

# 'Diver' wouldn't find diversity
In [16]: SearchQuerySet().filter(content=Raw('diver')).count()
Out[16]: 0

# I explicitely need to add the *
In [17]: SearchQuerySet().filter(content=Raw('diver*')).count()
Out[17]: 3

Heroku:
In [2]: SearchQuerySet().all().count()
Out[2]: 17

In [3]: SearchQuerySet().filter(content=Raw('diver')).count()
Out[3]: 9

In [4]: SearchQuerySet().filter(content=Raw('diver*')).count()
Out[4]: 9

I think it's either a simple setting, that bloats results up on Heroku, or its related to the EdgeNgramField we use. This field tokenizes the search text and allows partial matching. That might not work on my local machine, and the fallback is more restrictive.

Alright, the Field change fixes it and gives me the same results as locally:
https://aashe-hub-dev.herokuapp.com/browse/?search=diver
https://aashe-hub-dev.herokuapp.com/browse/?search=diver*

Nice work! :)


Support ‘Sodium’ versions higher than 0.5

Describe the Suggestion
Sodium version above version 0.5 are not supported
Screenshots and Videos

Additional Context
version: 1.20.1
mods:

create 0.5.1
sodium 0.5.2
indium 1.0.25
fabric-api 0.87.0
iris 1.6.8

#1039 - duplicate 4


[Suggestion] Pigeon used to send individual items to far distances

I don't see them scaring massive spiders... though it would be nice if another mob in general had that perk... but I think them being able to travel far distances to deliver either a single item or book would be neat feature.
Perhaps if Bundles were to be reenabled (by this or another mod), it would be a perfect way of sending more than just one item.
In very late game you could send Shulker Boxes using Pigeons too!
They could be sent to specific blocks which can store items and could be interacted with Redstone.

I think I've figured out how to do this. I'm going to add a letter item that players can right-click with to open a GUI. In this GUI, players can put an item, a message, and the coordinates where you want the pigeon to go. As long as the letter has coordinates and an item or a message, the letter can be sealed. Right-clicking on a pigeon with a sealed letter will make them go to the coordinates and drop the letter. When a pigeon drops a letter, the letter will become "dropped" and can be opened by right-clicking it. After the pigeon drops the letter, it will fly back to its owner. (Pigeons will only take a letter if it is from their owner.)
What's going to make this hard is pigeons flying to the coordinates, and the letter GUI. However, the rest of it shouldn't be too hard.

You could make it much easier coding-wise / lighter resources-wise by only allowing the pigeon to begin its journey when it has open sky overhead, then pathing it up to reach a certain altitude (eg server max render distance to nearest player) before simply teleporting above the destination (with a random time delay before arrival at destination based on naive distance). The unreliable time would be very pigeon-like. To avoid exploiting any imbalance and for flavour you could also add maybe a 5% chance the pigeon loses the letter during transit.

While I would love to add this feature, it is quite annoying to do so. There are many issues with my implementation that break the system, and I don't even know where to start when fixing them. Developing this feature has not been fun. Because of this, I'm making no plans to continue to work on this, but I might come back to it occasionally. All development is going to be shifted over to 1.18.2-x/letters.
This doesn't mean that we'll never have carrier pigeons, but don't expect them, either.


chore: update platform service framework

to fix snyk issue

@tim-mwangi do you mind linking the issue or relevant docs?

@tim-mwangi do you mind linking the issue or relevant docs?

Is that necessary for small fixes like updating dependencies? Or rather should we create an issue for every fix?


Refactor ES5 → ES6?

After writing tests, we could transform the code into ECMAScript 6 (the new JavaScript version). This requires:

[x] Minor rewrites in code
[x] Use babel-node (or node --harmony) in Docker container and npm start script

It will make the codebase modern and much cleaner.

Resolved since cf59eb6.


Add auto-clean functionality

One of the best features about Tup is its ability to automatically clean artifacts that are no longer configured outputs (whether it be through file changes or rule changes).
This should happen; ninja doesn't currently do this.

Going to have to wait on #17 and #9 for this first. This will require a list of outputs on last-run.


Thanks for sharing, helped me setup sops, multiuser multi nodes :)

Sent a small donation to say thanks for sharing your nix-config :)
It's helped me get multiple machines on nix more quickly, many thanks.

Cool 👍🏻 all the best.


Circulair option

Loving this library.
It's easy and working like a charm.
But i'm missing shapes or images that could be occur in a skeleton.

I think this can be closed, given #13 is merged in :)

Agree!


Tulip Leverage farming transactions

Hello,
I have been doing leverage farming with Tulip, and it doesnt seem to be able to handle all of the transactions. Will you be adding further support on this?

No plans but if you can provide sample txid (can DM @staketax or email support@stake.tax), I can take a look.


Autocomplete of Literal value only takes last overload into account

Describe the bug
In our code base we have a function which returns different objects based on a string provided as parameter. To provide good auto complete, our stubs make use of Literal and overload.

However, without the quotation marks, it does show all the possible outputs, but you can't use these to automatically fill the parameter:

To Reproduce
Create a function example and overload it with different Literal values. When calling the function now you can start typing quotation marks to see the possible strings to type, but you'll only be offered the last possible overload.
Expected behavior
All overloads should be offered to the autocompletion
Screenshots or Code
from typing import overload, Literal

@overload
def example(parameter: Literal["int"]) -> int: ...

@overload
def example(parameter: Literal["bool"]) -> bool: ...

@overload
def example(parameter: Literal["str"]) -> str: ...

def example(parameter):
    # some logic here
    return parameter

example("")

VS Code extension or command-line
Using PyLance as a VSCode extension (which includes Pyright (?))

As a sidenote, having an additional overload where parameter isn't a Literal gives no auto-completion:

Snippet
from typing import overload, Literal

@overload
def example(parameter: Literal["int"]) -> int: ...

@overload
def example(parameter: Literal["bool"]) -> bool: ...

@overload
def example(parameter: Literal["str"]) -> str: ...

@overload
def example(parameter: type[int]) -> int: ...

def example(parameter):
    # some logic here
    return parameter

example("")

A workaround is to actually put the non literal first, but would be great if both are supported.

The behavior is in fact inconsistent, whether you have extra required arguments and if you are defining the function in a class or not:

Plain function, no extra required argument: only last overload is suggested (same as OP use case)
Plain function, extra required argument: all overloads are suggested

@overload
def example(parameter: Literal["int"], arg: Any) -> int: ...

@overload
def example(parameter: Literal["bool"], arg: Any) -> bool: ...

@overload
def example(parameter: Literal["str"], arg: Any) -> str: ...

example("")  # suggests "int", "bool" and "str"

In a class, no extra required argument: only last overload is suggested

class A:
    @overload
    def __init__(self, parameter: Literal["int"]): ...

    @overload
    def __init__(self, parameter: Literal["bool"]): ...

    @overload
    def __init__(self, parameter: Literal["str"]): ...

A("")  # suggests "str"

In a class, extra required argument: no suggestion at all. Nothing is shown as a popup, even argument names aren't provided. If you type A(arg=..., parameter=", you get "str" as a suggestion.

class A:
    @overload
    def __init__(self, parameter: Literal["int"], arg: Any): ...

    @overload
    def __init__(self, parameter: Literal["bool"], arg: Any): ...

    @overload
    def __init__(self, parameter: Literal["str"], arg: Any): ...

A()

The latest(s) version(s) of Pylance seem to have fixed some use cases! I've updated my comment to reflect this. The auto-completion popup still seems to not be updated:

but I guess it's a different issue related to https://github.com/microsoft/pylance-release/discussions/4148

Any update on this? Still present now, and It's clearly a bug that needs to be fixed, all Literal options must be suggested


On the /F (Set Stack Size) page, the /STACK hyperlink points to the EDITBIN option instead of the linker option

Using the /STACK linker option. For more information, see /STACK.

The quoted link points to the EDITBIN /STACK option, but the surrounding context makes it clear the intent is to link to the linker option. The correct link should be https://docs.microsoft.com/en-us/cpp/build/reference/stack-stack-allocations?view=msvc-160 which is the reference for the /STACK linker option.

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

ID: cf6298fd-5773-69eb-c9e0-0c496310b537
Version Independent ID: b3d903ea-1584-f503-8079-94e87b432c8a
Content: /F (Set Stack Size)
Content Source: docs/build/reference/f-set-stack-size.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

@jagunter
Good catch. Since EDITBIN is really the linker in disguise, it's not a different command, but the context does matter.

Closing as fixed.


Added else if and else snippets.

Looks like this PR mixes up things a lot with your fork of the package, so I'm going to close this here.


Group sync with Zitadel

Describe the problem
I've been trying to configure self-hosted Netbird with Zitadel as IDP to use user roles which I've created under the Netbird project in Zitadel, but without any success. I've tried to use the claims as it is described  in Zitadel's manual, but without any success. I've tried to use these claims:

urn:zitadel:iam:org:projects:roles
urn:zitadel:iam:org:project:{projectid}:roles
urn:zitadel:iam:org:project:id:{projectid}:aud

To Reproduce
Steps to reproduce the behavior:

Create user roles in Zitadel under Netbird's project.
Add described claims to Settiings -> Groups -> Enable JWT group sync -> JWT Claim
Get error in management logs: JWT claim "urn:zitadel:iam:org:projects:roles" not found

Expected behavior
I expect to get user Roles from Zitadel to assign different groups in Netbird.
Are you using NetBird Cloud?
Self-host NetBird's control plane.
NetBird version
0.26.3
Screenshots

hello @sisumara, it seems like the claim is not the correct one. To discover the correct claim, please open your browser's development tools, and get one request to the management API, e.g., /api/users, there you will find an authentication header with a Bearer token, which you can copy and paste into the https://jwt.io/ website, then you will see the claims parsed in the payload view. Choose the one that maps to the roles.

hello @sisumara, it seems like the claim is not the correct one. To discover the correct claim, please open your browser's development tools, and get one request to the management API, e.g., /api/users, there you will find an authentication header with a Bearer token, which you can copy and paste into the https://jwt.io/ website, then you will see the claims parsed in the payload view. Choose the one that maps to the roles.

I've tried that approach, but this the only one Bearer token which I've found in there and there is no any mentions about groups. Here is how it looks like:

What am I doing wrong?

I've found that Zitadel includes information about the roles if you enable "Assert Roles on Authentication" option in Project settings.

Then the token includes information about user roles like this:
"urn:zitadel:iam:org:project:256448444641310477:roles": {
    "netbird-admin": {
      "356409606912278797": "my-idp-domain"
    },
    "netbird-user": {
      "356409606912278797": "my-idp-domain"
    }
  },
  "urn:zitadel:iam:org:project:roles": {
    "netbird-admin": {
      "356409606912278797": "my-idp-domain"
    },
    "netbird-user": {
      "356409606912278797": "my-idp-domain"
    }
  }
}

But when I try to admit claim urn:zitadel:iam:org:project:256448444641310477:roles or urn:zitadel:iam:org:project:roles to the dashboard I see this error in log - DEBG management/server/account.go:1677: JWT claim "urn:zitadel:iam:org:project:roles" is not a string array

Hi,
I'm also having trouble syncing the groups via JWT. The claims exist in the access token like @sisumara already described but NetBird somehow cannot read them correctly.
With the claim urn:zitadel:iam:org:project:<netBirdProjectId>:roles which exists inside the access token, I also get the following error with NetBird v0.26.6:
DEBG management/server/account.go:1721: JWT claim "urn:zitadel:iam:org:project:<netBirdProjectId>:roles" is not a string array

Hello, u can do the following steps for configure IdP(Zitadel) auto-groups with Netbird:
Ziradel configuration:

Netbird need list of groups for auto-group working when zitadel default claims return objects, because of that u need to create custom action in Zitadel with following content:

function flatRoles(ctx, api) {
  if (ctx.v1.user.grants == undefined || ctx.v1.user.grants.count == 0) {
    return;
  }

  let grants = [];
  ctx.v1.user.grants.grants.forEach(claim => {
    claim.roles.forEach(role => {
        grants.push(role)
    })
  })

  api.v1.claims.setClaim('groups', grants)
}

Select Flow: Complement token, Triggers: Pre Userinfo creation, Pre access token creation
Note that name of action need to be the same like function in this action
2) Create Role in Zitadel: Project -> Netbird project -> Roles -> + New
3) Go to the Authorizations -> Select user -> Select Netbird project and choose the correct roles for the user
Netbird configuration

Go to the Settings -> Groups -> Enable JWT group sync and JWT claim should be the same like in action (In this example groups)

After that configuration users should sync roles with Zitadel. but we still fight with SYNCING because in our setup Netbird only add groups from IdP but not delete groups when whe delete it in IdP


Binary data not supported

Hi，I was using the example from (https://github.com/cloudevents/sdk-javascript/blob/master/examples/express-ex) and got this error.
{ message: 'content type not allowed',
  errors: [ 'application/octet-stream' ] }

https://www.iana.org/assignments/media-types/application/octet-stream

The "octet-stream" subtype is used to indicate that a body contains
arbitrary binary data.

Code to send the CloudEvent message.
package cloudevents_test

import (
	"context"
	"testing"
	"time"

	"github.com/cloudevents/sdk-go"
	"github.com/cloudevents/sdk-go/pkg/cloudevents/client"
	cehttp "github.com/cloudevents/sdk-go/pkg/cloudevents/transport/http"
	"github.com/cloudevents/sdk-go/pkg/cloudevents/types"
	"github.com/google/uuid"
	"github.com/stretchr/testify/assert"
)

func TestTransportHTTP(t *testing.T) {
	encodeType := "application/octet-stream"
	event := cloudevents.Event{
		Context: cloudevents.EventContextV02{
			ID:          uuid.New().String(),
			Type:        "example",
			Source:      *types.ParseURLRef("/test"),
			Time:        &types.Timestamp{Time: time.Now()},
			ContentType: &encodeType,
			Extensions:  map[string]interface{}{"a": 1, "b": 2, "c": 3},
		}.AsV02(),
		Data:        []byte("Hello text."),
		DataEncoded: true,
	}

	c, err := newEventClient("http://<IP_ADDRESS>:3000")
	assert.NoError(t, err)

	resp, err := c.Send(context.Background(), event)
	assert.NoError(t, err)
	t.Log(resp)
}

func newEventClient(target string) (client.Client, error) {
	t, err := cehttp.New([]cehttp.Option{
		cehttp.WithBinaryEncoding(), cehttp.WithTarget(target),
	}...)
	if err != nil {
		return nil, err
	}
	c, err := client.New(t, client.WithUUIDs(), client.WithTimeNow())
	if err != nil {
		return nil, err
	}
	return c, nil
}

@pengsrc,
Nice PR!

@pengsrc
Released: https://www.npmjs.com/package/cloudevents-sdk/v/0.2.6


Move development configuration settings from custom file

In commit 6f677a5b2fcc8e1bc94a2d7cf217dcfbc140dbe6 configuration option was added in a custom file config.development.json in root of repository that is meant to be used for local settings, as is mentioned in .gitignore file:

# ignore all custom json files for config
/config.*.json

This PR reverts 6f677a5b2fcc8e1bc94a2d7cf217dcfbc140dbe6 and adds enableDeveloperExperiments to core/server/config/env/config.development.json to preserve functional change introduced in 6f677a5b2fcc8e1bc94a2d7cf217dcfbc140dbe6.

This change would enable developer experiments for anyone using the ghost-cli or other install when running in development mode - which (AFAIK) isn't what want :thinking: .
The original commit would only enable developer experiments when working directly with the source code.

I understand.
But then there are 2 development modes, right?
Shouldn't be development mode only one, regardless if it is installed/activated from source or from ghost-cli?
First reason for creating this change was to stop seeing my local config as unstaged changes and to prevent accidentally changing local config with various checkouts, hard resets etc ...
Another reason was following existing logic in config code, as root config.development.json is just one of the config sources in layered config. Same is valid for production or any other environment.
Last reason was mentioned entry in .gitignore that was set to ignore local configuration changes.
Maybe ghost-cli should be changed to disable developer experiments in custom/local config on install?

But then there are 2 development modes, right?

There's only one development mode, this turns off things like caching, and is used for developing on the project, or themes for Ghost.
Then there is the enableDeveloperExperiments configuration flag - this isn't a development mode thing, it can be used in production. You might be better thinking of it as an enableAlphaFeatures flag.

First reason for creating this change was to stop seeing my local config as unstaged changes and to prevent accidentally changing local config with various checkouts, hard resets etc ...

Yeah I can see why that would be annoying, I don't have the same problem because during development I tend to use the defaults - you might consider using git stash to store your config, so if you ever lose it, you can git apply it.

Maybe ghost-cli should be changed to disable developer experiments in custom/local config on install?

I don't think this is correct, as I mentioned above, it's within reason that someone would want to enable this flag on an install with ghost-cli

@dpensa Going to close this now - thanks for your contribution tho! If you want to help out with anything else, here is a place to start :ghost:

@dpensa https://medium.com/@igloude/git-skip-worktree-and-how-i-used-to-hate-config-files-e84a44a8c859 This article might help you with your git frustrations :relaxed:

So it seems that it is not possible to define custom development configuration settings in a file that is not tracked by git?
Thanks for the article, but I do not have any frustrations with git. ;)

The article shows you how to remove that file from being tracked :)

That is just a trick.
I still do not believe that cleaner and logically proper solution is not possible.

Setting enableDeveloperExperiments to true in a proper place (core/server/config/env/config.development.json).
Add in ghost-cli to override that value to false when doing local install by defining enableDeveloperExperiments to false in generated config json.

If user wants to enable experiments it just needs to change that setting to true (now it has to define it and set to true).
Do you agree that this is a cleaner solution? Proper if you will? No git tricks, no stash/apply? Local config file is ignored by .gitignore as it should be, all default settings are in one place (json files in core/server/config/) tracked by git, etc ...

I agree that there is a cleaner solution, I don't think it involves setting the enableDeveloperExperiments flag inside of Ghost core though.
We do not want it set as a default for anyone downloading the Release zip or installing via the cli. (As you say we can add logic to the ghost-cli, but there are many ways to get Ghost installed, and this isn't a "sane" default for a release IMO)
I'm thinking now if we maybe add --enableDeveloperExperiments to the yarn dev command - or something like that, to preserve the flag for running from source - but remove the config file form being tracked :thinking:

Maybe enable enableDeveloperExperiments in grunt for dev so that is enabled regardless if it is run as grunt dev or yarn dev (with debugging enabled).
Additional --disable-experiments argument for grunt dev could disable enableDeveloperExperiments if needed.
Commit 6f677a5 can be reverted.
Local configuration file won't be tracked and changes will be preserved without any tricks.
enableDeveloperExperiments will be enabled by default through yarn dev / grunt dev.
No changes in ghost-cli are needed and by default development environment is left with disabled enableDeveloperExperiments.
I did quick test and it seems that this simple change in Gruntfile.js is enough.
diff --git a/Gruntfile.js b/Gruntfile.js
index a1f7608cc..1ec0e7985 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -560,6 +560,9 @@ const configureGrunt = function (grunt) {
     //
     // Note that the current implementation of watch only works with casper, not other themes.
     grunt.registerTask('dev', 'Dev Mode; watch files and restart server on changes', function () {
+        if (!grunt.option('disable-experiments')) {
+            process.env.enableDeveloperExperiments = true;
+        }
         if (grunt.option('client')) {
             grunt.task.run(['clean:built', 'bgShell:client']);
         } else if (grunt.option('server')) {


New Updated curl to 7.65.1

Updated curl to latest version 7.65.1

@shubham769 Could you please check failed build errors? Thanks!


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


remove data

Description
Dear, how can I remove data that I do not want to appear in the report, for example the complete list of object addresses and I tried to delete lines in Get-AbrFgtFirewall but even so when generating the report all the data appears.
Additional Context
No response
Before submitting

[X] I have read the documentation, and referred to the known issues before submitting this change request.
[X] I have checked for previously opened & closed issues before submitting this change request.

Hi,
do you have (re) import-module with -force after change ?
I known if is not possible for the moment for custom (option) each sub chapiter @tpcarman

@geraignacio , if you want to exclude firewall address objects from the report, you can:
In "Get-AbrFgtFirewall.ps1" change "if ($Address -and $InfoLevel.Firewall -ge 1) {" to "if ($Address -and $InfoLevel.Firewall -ge 2) {"

thanks and another question in case an authentication token is required, can it be added?

use API Token ? and not login/password ?

is MFA with fortitoken

is MFA with fortitoken

Ok, you need to modify Public/Invoke....ps1 file for add -token_prompt on Connect-FGT function
As build report Fortinet FortiGate use PowerFGT

I added what you told me but it doesn't ask me for the token
Connect-FGT -Server $System -Credential $Credential  -token_prompt
-SkipCertificateCheck -Port $Options.Port -vdom $Options.vdom | Out-Null
El mar, 3 dic 2024 a las 10:57, Alexis La Goutte @.***>)
escribió:

is MFA with fortitoken
Ok, you need to modify Public/Invoke....ps1 file for add -token_prompt on
Connect-FGT function
As build report Fortinet FortiGate use PowerFGT
—
Reply to this email directly, view it on GitHub
https://github.com/AsBuiltReport/AsBuiltReport.Fortinet.FortiGate/issues/102#issuecomment-2514631676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2R227YPMCIMZLTEVRLWNPT2DW2EJAVCNFSM6AAAAABS4GJBYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJUGYZTCNRXGY
.
You are receiving this because you were mentioned.Message ID:
<AsBuiltReport/AsBuiltReport.Fortinet.FortiGate/issues/102/2514631676@
github.com>

same error
ADVERTENCIA: [ 12:56:25:452 ] [ Document ] - Two-factor Authentication is
needed (use -token_code XXXXXX or -token_prompt)
New-AsBuiltReport : Not Connected. Connect to the Fortigate with Connect-FGT
En línea: 1 Carácter: 1

New-AsBuiltReport -Report Fortinet.FortiGate -Target <IP_ADDRESS> -User ...

  + CategoryInfo          : NotSpecified: (:) [Write-Error],

WriteErrorException
+ FullyQualifiedErrorId :
Microsoft.PowerShell.Commands.WriteErrorException,New-AsBuiltReport
PS C:\WINDOWS\system32>
El mar, 3 dic 2024 a las 12:52, Alexis La Goutte @.***>)
escribió:

May be try to remove | out-null :)
—
Reply to this email directly, view it on GitHub
https://github.com/AsBuiltReport/AsBuiltReport.Fortinet.FortiGate/issues/102#issuecomment-2514941922,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2R2274F4XPN2TPW6ROJNGD2DXHSFAVCNFSM6AAAAABS4GJBYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJUHE2DCOJSGI
.
You are receiving this because you were mentioned.Message ID:
<AsBuiltReport/AsBuiltReport.Fortinet.FortiGate/issues/102/2514941922@
github.com>

Sorry, never try with MFA

Ok, thank you very much for your time and help.
El mar, 3 dic 2024 a las 13:45, Alexis La Goutte @.***>)
escribió:

Sorry, never try with MFA
—
Reply to this email directly, view it on GitHub
https://github.com/AsBuiltReport/AsBuiltReport.Fortinet.FortiGate/issues/102#issuecomment-2515074395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2R227YP2URRFCITEQDV4E32DXNY7AVCNFSM6AAAAABS4GJBYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJVGA3TIMZZGU
.
You are receiving this because you were mentioned.Message ID:
<AsBuiltReport/AsBuiltReport.Fortinet.FortiGate/issues/102/2515074395@
github.com>


Sorry, newbie here!

Hi. Am trying out the codes. Understand that the raw files here mean genotype files from DTC companies. However, what I have is only vcf and gvcf files from in-house sequencing platform. May I know can these be used? How? I have zero knowledge in coding. Thanks a lot!

Computed cMs in shared_DNA_one_chrom.excel. Total cMs equals to 3600, in which DNA painter shows parent/child relationship. But the two persons are totally unrelated. Anyone could help?

Hi, thanks for the note. Yes, VCF and GVCF files should work if the SNPs are annotated with RSID. The files can be loaded like shown in the examples.
As for the issue with total shared cMs = 3600, was this for output generated by lineage?

Dear Apriha/Lineage,
Thank you so much for replying.
The output by lineage is an excel, one of the columns is the cMs (please
see attached). The total shared cMs is calculated by adding all the values,
no? That's how I obtained ~3600cMs. When I entered this value on
DNApainter, it showed parent/child relationship, when the two individuals
are actually husband and wife.
Can you please help?
Thank you.
Sincerely,
Chai San
On Tue, Feb 25, 2020 at 1:01 PM Andrew Riha<EMAIL_ADDRESS>wrote:

Hi, thanks for the note. Yes, VCF and GVCF files should work if the SNPs
are annotated with RSID. The files can be loaded like shown in the
examples
https://lineage.readthedocs.io/en/latest/readme.html#load-raw-data.
As for the issue with total shared cMs = 3600, was this for output
generated by lineage?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apriha/lineage/issues/75?email_source=notifications&email_token=AOTD7RQO3DPWPYOY376BGQ3RESQZ7A5CNFSM4KXUPKQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEM2SDVY#issuecomment-590684631,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOTD7RSQRGEUVIV2YZDOZRTRESQZ7ANCNFSM4KXUPKQA
.

Hi Chai, sorry, the file didn't come through. But yes, you're correct that the total shared cMs is calculated by adding all values in the cMs column of the shared_dna output files.
But I suspect that lineage is finding matches due to the way that the genotype is parsed by snps in the VCF / GVCF... You said that this was an in-house sequencing platform - is that also creating the VCF / GVCF? Thanks again.


Acmev2

Thank you for contributing to sewer.
Every contribution to sewer is important to us.
You may not know it, but you have just contributed to making the world a more safer and secure place.
Contributor offers to license certain software (a “Contribution” or multiple
“Contributions”) to sewer, and sewer agrees to accept said Contributions,
under the terms of the MIT License.
Contributor understands and agrees that sewer shall have the irrevocable and perpetual right to make
and distribute copies of any Contribution, as well as to create and distribute collective works and
derivative works of any Contribution, under the MIT License.
Now,
What(What have you changed?)

support acme version 2

Why(Why did you change it?)

its the future
fix; https://github.com/komuw/sewer/issues/68
fix; https://github.com/komuw/sewer/issues/69
fix; https://github.com/komuw/sewer/issues/24

TODO:

on merge send a PR to https://github.com/komuw/website/blob/master/content/docs/client-options.md
to show that sewer full integrates with acme v2 production endpoints

Hello @wilfriedjonker .
If you still use sewer as your letsencrypt client, would you mind testing that this branch and pull request will still work for you.
This Pull request adds support for ACME version2 and removes version 1 support.
Letsencrypt recently announced the availability of ACME v2(https://letsencrypt.org/2017/06/14/acme-v2-api.html)
And they have since created test endpoints(https://community.letsencrypt.org/t/staging-endpoint-for-acme-v2/49605)
I've also made a few other changes that I've been meaning to do for a while:

I've ported sewer to python3, it is now developed using python3.
I've also removed the logging library(structlog) that I was using in favour of the inbuilt stdlib logging library.


Helm Chart v1.10.3: template for k8ssandraclusters CustomResourceDefinition has uninterpolated strings / missing Helm templating

What happened?
After  helm install of k8ssandra-operator v1.10.3, the k8ssandraclusters CRD contains the following annotation:
cert-manager.io/inject-ca-from: $(CERTIFICATE_NAMESPACE_K8S)/$(CERTIFICATE_NAME_K8S)
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  annotations:
    cert-manager.io/inject-ca-from: $(CERTIFICATE_NAMESPACE_K8S)/$(CERTIFICATE_NAME_K8S)
    controller-gen.kubebuilder.io/version: v0.12.0

How to reproduce it (as minimally and precisely as possible):
helm install --version 1.10.3 k8ssandra-operator k8ssandra/k8ssandra-operator -n k8ssandra-operator --set global.clusterScoped=true --create-namespace

kubectl get crd k8ssandraclusters.k8ssandra.io -o jsonpath='{.metadata.annotations}'
# {"cert-manager.io/inject-ca-from":"$(CERTIFICATE_NAMESPACE_K8S)/$(CERTIFICATE_NAME_K8S)","controller-gen.kubebuilder.io/version":"v0.12.0"}%

┆Issue is synchronized with this Jira Story by Unito
┆Issue Number: K8OP-60

Hello I have the same issue, did you find a workaround?

Annotations can be added and removed. Although it shouldn't have any effect.

It does have an effect in this case, with this annotation the cert manager cainjector will print the following log in loop
E1016 08:59:10.670334       1 sources.go:106] "unable to fetch associated certificate" err="Certificate.cert-manager.io \"$(CERTIFICATE_NAME_K8S)\" not
found" logger="cert-manager" kind="customresourcedefinition" kind="customresourcedefinition" name="k8ssandraclusters.k8ssandra.io" certificate="$(CERTIF
ICATE_NAMESPACE_K8S)/$(CERTIFICATE_NAME_K8S)"

If the annotation is not needed I can create a quick pr to remove it

The annotation is only used in the Kustomize installation (as that's a kustomize replacement directive), but since the CRD is copied from there it's in the Helm chart also. To remove would require a modification to the prepare-helm-release.sh to remove it after it was generated. A sed command would probably be enough..

Thanks @burmanm I'll create a PR as soon as I have time


Add properties to user resource for certificate creation

Description
This change adds properties to the user resource of the openvpn cookbook. As a result, developers can specifically define attributes of the user certificate they want to create within a resource statement. Environment variables are updated in the server recipe to provide default values.

Released as: 6.1.0


Support exclamation point in front of internal links

Fixes #511 and fixes #512
Support ![[Link]] format by simply including an optional ! character in between the quotation marks. It's a very naive fix, but do let the banners work
A concern is, should images be able to show in views? For example, would there be a rendered image in a table cell with richtext format enabled? Related to #424

Thanks for the quick fix! We need a regression test for this to make sure this doesn't come back. I'll merge the PR and add the test afterwards, so that I can release a fix today.


[DOC]File upload labels are not marked as required.

https://github.com/pankod/refine/blob/master/examples/fineFoods/admin/src/components/product/create.tsx#L61-L72
This is how it's supposed to be.
<Form.Item label={t("products.fields.images.label")}>
  <Form.Item
    name="images"
    valuePropName="fileList"
    getValueFromEvent={getValueFromEvent}
    noStyle
    rules={[
      {
        required: true,
      },
   ]}
  >
 ・・・

It seems to show up like this.
  <Form.Item
    label={t("products.fields.images.label")}
    name="images"
    valuePropName="fileList"
    getValueFromEvent={getValueFromEvent}
    noStyle
    rules={[
      {
        required: true,
      },
   ]}
  >
 ・・・

Hey @nrikiji ,
Thank you for contacting us. Do you want to open PR? It would be great 🔥

I would fix it if I could, but it doesn't seem to work in my environment. Sorry.

If I'm not mistaken it's an Ant Design bug. We will check if it is fixed

This issue is on the side of ant Design. No solution is specified. I'm closing for now


Zero length span

Ideally, this PR would just be commit c340273 and c7cbb07[^1] stacked on top of PR #354, but Github doesn't support PR stacking, so this also contains the commits from #354.
The first is to add a "error line only" mode, where one does not want extra context line but still get the line with the error, which is necessary when using zero-length span without context.
The second adds support for the zero-length span at the end of the source or the context.
This is a API breaking change. It changes the usize to Option<usize> in SourceCode::read_span().
GraphicalReportHandler and NarratableReportHandler supports the new capability but in a non-API breaking way, by adding with_opt_context_lines instead of changing the signature of  with_context_lines. The latter's behavior is also unchanged (where 0 means None in the new function rather than Some(0))
[^1]: and commit 8e4849e to fix new miri errors.

... and mark the unused structs in the tests as dead_code

I don't know why the CI got aborted, I'm guessing because it failed on my fork. There it failed because of an update of the backtrace crate (rust-lang/backtrace-rs#597).

Sorry about that, backtrace 0.3.71 is out now!
Though uhh Windows backtraces may be... interesting, still, on the current stable. Sorry about that!

FYI, my branch has passed the checks with backtrace 0.3.71

I'll have to think about this one for a bit: it's a pretty significant API breakage, and I'm trying to limit things that change the API this significantly. The only thing that I know I want to change about miette that would be notably semver-breaking is that I have some plans for #242 that might involve replacing/deprecating Report.

Not the most significant API change I was thinking about 😜. A bigger one was to change the SpanContents::data to an array of lines, but I haven't dared yet 😓

Yeah there's a certain extent to which, considering how widely used Miette is, it needs to be "mostly done", and we don't really have the ability to make major changes to it. It's ok to make "breaking" changes that only affect the small minority of folks who use certain niche APIs, I think, but the upgrade path for 99% of people should be clean, even across semver-major boundaries. It can otherwise be a lot of work to upgrade.

Agreed, which is why I hesitated doing the "vector of lines", and why I chose to add with_opt_context_line rather than fix with_context_lines for the Option<usize> change.
And in that regard, I don't think this Option<usize> is a big change (I'm biased of course). For most people, nothing changes. Pure "users" (those who don't impl anything) won't be affected beside a recompile.
The main group of affected people would be the one having their own SourceCode implementation (it's hard to know how big that group is though, so I don't know if it's a minority or not). I think most implementations would be trivial to fix since it's not really a new feature is "just" a matter of renaming 0 to None and extending non-zero case to also cover 0. As an example, see my own fix for source_impl.rs, which is just a couple of is_some and unwrap_or (although it relies on some big prior cleanup I did for other unrelated reasons, see PR #354).
Another group of affected people would be the one implementing their own Report, but that group should be very small, and quite likely, the change should be very trivial too (see my changes to GraphicalReportHandler and NarratableReportHandler).
Maybe to you the change looks bigger to you than it to you is to you because, as mentioned earlier, this PR also includes some unrelated commits (PR #354) due to the lack of stacked PR support in GitHub? See commit d63b2bb for what actually changed to support Option<usize>.


Support for s390x for the Go operator

Feature Request
Is your feature request related to a problem? Please describe.
As per https://github.com/operator-framework/operator-sdk/blob/master/doc/dev/release.md the operator-sdk is not supported on s390x. To enable out operators to be fully portable we need to be able to test and ship our operators across multiple platforms.
Describe the solution you'd like
The operator-sdk support s390x and provide built binaries for the platform.

HI @IBMRob,
I flagged this one for we are able to check and discuss it internally. However, please feel free to collab with PR's as well if you wish in order to provide and proposal solution to attend this need.
@joelanford WDYT?

Is this not fixed by #2564?  Binaries will be available once v0.16.0 is released, otherwise I have already provided binaries from my fork from which the aforementioned PR originated.  If you need official upstream binaries soon, that depends on whether a /cherry-pick v0.15.x of #2564 (which would apply cleanly last I checked) would be accepted.

Hi @yselkowitz,
I also understand that yes it is fixed. I am not sure if in this case, we should cherry-pick since it is not a bug fix. So, I am closing this one as solved.
@IBMRob please, feel free to ping us if you think that it should be re-opened fo any reason.


Text not sanitized

Wikipedia will search anything

Placing call functions after retrieving country data means only valid inputs are recognized, great job!!!


Clear cache or write over it when there's been an error or version update

(We've been doing this manually for testing the CLI)
We ran into this problem when trying to get the mobility connector up and running - we hit a timeout on creating tiles/graph, and then any future tests would use the cached files and get an error message. Would be great to clear this out automatically when necessary, otherwise we will need to troubleshoot this 1:1 with cities.

Cache now announces if the graph is incomplete and provides instructions to delete the directory.


Update Date Validation to Handle Additional Scenarios

Problem Statement
We created a custom mapper to handle dates that come from a Sender submission as a string:
https://app.zenhub.com/workspaces/experience-607d9d5e68b95200150fec37/issues/cdcgov/prime-reportstream/2246
However, what we didn't know was that the CSV Serializer was doing date validation based on the field being typed as a date (often from the base COVID schema) before the custom mapper would run, thereby, nullifying the effects of the new mapper.
Acceptance Criteria

Remove the custom mapper that was created as part of this ticket:
https://app.zenhub.com/workspaces/experience-607d9d5e68b95200150fec37/issues/cdcgov/prime-reportstream/2246
Enhance the checkForError/toNormalized method to handle the DATE and DATETIME Element type normalization to include the following formats

"2008-09-05 0:00:00.000"
"2008-09-30 03:00 PM"

NOTE: have received both AM and PM

"8/1/2021 1:17:01 PM"

NOTE: have received both AM and PM

"8/1/2021 13:00"
"20210101"
"01012021"
"01/01/2021" and "1/1/2021"
"01-01-2021" and "1-1-2021"

Notes

What does Simple Report use for date validation?

reference back to this ticket for context
https://app.zenhub.com/workspaces/experience-607d9d5e68b95200150fec37/issues/cdcgov/prime-reportstream/3056
there are also attachments that will help debug/test the issue

i changed the pointing for this story to a 13 because:

the architecture of the application is not conducive to make changes in the area where this was needed. it was complex to navigate and write reusable/clean code
code that was put in before a few of the building actions were put in place does not pass the build requirements now - so i had to refactor a lot of code that i did not write to pass the build
local development builds were botched by pulling in master and attempting to rebuild the application... constantly


Feat: Group Associated Outputs

Description of change

Grouped associated outputs by output Id
Added a tooltip to display "additional" associations in AssociatedOutputsTable on mouse hover
Refactor

Type of change

Enhancement (a non-breaking change which adds functionality)

How the change has been tested
Manually
Change checklist

[x] My code follows the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[] I have made corresponding changes to the documentation
[] I have added tests that prove my fix is effective or that my feature works
[] New and existing unit tests pass locally with my changes

LGTM.


template not preview properly in Higher resolution phone.

Here is the template link
http://developerinfotek.com/demo/extragroup/outlook.html
this is working on most of browser but it not showing properly in high resolution small width phone.
Mainly this issue in SAMSUNG EMAIL client. Model SAMSUNG GALAXY S3 and above version.
Red Mark in image is the issue.
AS PER media query email Show Mobile View , but it show Desktop view.
What is solution for this issue.

This should be addressed in v2. We're not supporting v1 currently, however we're more than happy to merge in a PR for v1.0.0. Thanks!


Bug/allocator always pub

Fixes #48 and cleans up the allocator a bit. A lot of renames, using a unique pointer so we don't have to worry about delete, and some more stuff.

Never mind the tests...

@mortenfyhn Why ignore the tests?


Upload VRH token

This logo file is missing for VRH token profile upload. Refer to PR#7401.
为确保 Token Profile 的顺利通过，务必逐一确保以下检查项并补充项目资料：
.json 及 .png 文件检查项

[ ] logo 尺寸：120*120 px
[ ] logo 背景：透明
[ ] 代币有至少一笔成功的转账记录
[ ] 此项对于 ERC-20 项目方：合约地址有大小写（做 checksum）
[ ] .json 和 .png 文件名一致（包括大小写）

项目资料：
官方公告：
团队背景：
项目基本情况：
媒体报道：
收录的交易所：
转账所需 Gas Limit（对 ERC-20 项目方）：（默认为 60000）

Please check the following to ensure a successful verification of Token Profile
.json and .png document checklists

[ ] logo dimensions: 120*120 px
[ ] logo background: Transparent
[ ] At least one transaction on the token contract
[ ] For ERC-20 project teams: Upper and lower capitalized alphabets in token address (For checksum purposes)
[ ] Same document name for both .json and .png (Including capitalization)

Project background:
Official announcement:
Project team background:
Project basic information:
Media publications:
Tradeable exchanges:
Recommended Gas Limit (For ERC-20 project teams) for transaction: (Default is 60,000)

@zengbing15 As you requested before, I have to re-commit logo file to /images folders.  This LOGO file is for ERC20 VRH token and its contract address: 0xeAd482da0793B00bbAe0E34C8cfaE6DAf29a44b2


OpenGL GPU-accelerated terminal screen stutters with Vim, doesn't happen with neovim

Describe the bug
Vim screen stutters a lot with modern OpenGL-based terminals that offload screen rendering to GPU. AFAIK the philosophy of these terminals is different, they don't redraw part of the screen, and rather update entire screen as its fast & cheap. The same stutter behavior is not seen in old terminals that use CPU to do the rendering, and draw only part of the screen that needs update. But users are & will keep moving to new OpenGL-based terminals in future as they are super fast & low power (even compared to GVim GUI on macOS).
To Reproduce

Open one of the OpenGL-based terminals like Alacritty, Kitty, or WezTerm (problem happens with iTerm2 too, but not as apparent in this very short example)
Run vim --clean
Open a file :e ~/.vimrc
Run command :redraw! multiple times.

Expected behavior
There should never be screen stutters on openGL terminals when executing commands (like :redraw!). In this example it seems like :redraw! is the culprit, though redraw also does it when called from inside functions (like in plugin Startify [minimal example at bottom]). Note: NeoVim doesn't have such stutters, so it already is doing something different for redraws compared to Vim.
MP4 Videos (new GitHub feature works only on laptop/desktop for me)

Stuttering in Vim:

https://user-images.githubusercontent.com/71736629/112261708-979b0980-8c29-11eb-9cbe-cfcff2fb9bcf.mp4

No stuttering in NeoVim:

https://user-images.githubusercontent.com/71736629/112260239-c82d7400-8c26-11eb-913f-cd7219dbfbc5.mp4
The minimal .vimrc in case you want to try it with :Startify command in above videos:
set nocompatible
filetype plugin on
set noswapfile
syntax enable

call plug#begin('~/.vim/plugged')
Plug 'mhinz/vim-startify'
call plug#end()

set background=dark

let g:startify_lists = [
    \ { 'type': 'files',     'header': ['   MRU']            },
    \ { 'type': 'dir',       'header': ['   MRU '. '$PWD'] },
    \ { 'type': 'sessions',  'header': ['   Sessions']       },
\ ]

nnoremap <silent> <leader>] :Startify<CR>

The above setup should also make stutters clearly visible in  iTerm2.
Environment (please complete the following information):

VIM - Vi IMproved 8.2 (Included patches: 1-2576) (macOS version - arm64)
macOS 11.2.3 ARM64 (machine: Apple M1 based Mac mini)
OpenGL-based Alacritty, Kitty, or WezTerm

Additional context
Options like lazyredraw, and ttyfast have no effect on this.

This is not reproducible.  No idea what you are doing in those videos, I don't see a :redraw! command being typed.
Please provide a reproducible example without using a plugin.  E.g. editing the Vim source code.

@brammool Here we go... Below is a complete example with not plugins & me typing :redraw! visible... Sorry for the ambiguity in previous video (you couldn't see what I was doing because of this nnoremap <silent> <leader>] :Startify<CR>). I was pressing <leader>]
https://user-images.githubusercontent.com/71736629/112394997-d4631100-8cba-11eb-837c-9fb89433af1d.mp4

What I see here is perfectly normal: screen is cleared and then the text is drawn.
Perhaps neovim flushes less often, so that it's all done in one go?
That may reduce flicker, but delay output in other situations.
Investigation would require intercepting what exactly is sent to the terminal, possibly with timing.
I'll leave that to someone who wants to spend time on it.

@brammool: @bfredl from neovim team has some preliminary insight in his comment: https://github.com/neovim/neovim/issues/14225#issuecomment-809623524.
@bfredl: Any chance to see what's going on in Vim? Thanks!

@brammool: @bfredl from neovim team has some preliminary insight in his comment: https://github.com/neovim/neovim/issues/14225#issuecomment-809623524.
@bfredl: Any chance to see what's going on in Vim? Thanks!

I believe what is said here that Neovim doesn't clear the screen, but
draws over the text.  In many cases that is slower, but the advantage is
that the user sees less flicker.  A risk is that when some text was
drawn outside of Vim's control it might not be replaced.
The Vim :redraw command only has two modes: without !: redraw what is
outdated, with !: clear and redraw.  There is no way to redraw with
NOT_VALID, which would redraw everything but not clear.  Perhaps
":redraw all" could be added.
--
hundred-and-one symptoms of being an internet addict:
74. Your most erotic dreams are about cybersex
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Good day @brammool
Any updates on fixing this issue?
It's really annoying to scroll filetypes with heavy regex syntax (too much redraws equal too much flickering) or using such popular tool like fzf inside vim.
Sorry to be annoying and understandable that you personally do not have such issues.

The priority on this issue is very low, since I don't see why a user would type ":redraw!", other then when the screen is messed up and then the behavior is totally justified.
If screen redrawing is so fast, then why would there be any flicker?
Is there any normal use that is affected?

I guess OP missguide You or something. You don't need to invoke :redraw! to see flickering. My english isn't the best so I'm trying hard to explain. It flickers the sceen too much when you just scroll with <C-f> some filetypes which syntax has some heave regexes as example new vim9 syntax file by lacygoill https://github.com/lacygoill/vim9-syntax ( Just add it to plugins open some big vim9 file) and do some   enable 'number' and laststatus=2. And for me screen redraws on like every such keypress that will flicker expecially number column and statusline too. When at the same time neovim is smooth.
Or I'm not sure if you know, but fzf cli tool is very popular among linux and vim users, and when you use it in popup terminal window inside vim it flickers too. This is too usecases that bothers me personally, but i do believe there are much more.
And it's not GPU accelerated terminals related issue, for me it behaves the same in any terminal emulator i have tested.
What else I can provide gifs movs (that compare both editors) to ensure you that this problem is present in just normal usage, not only in a nonsense :redraw! spam?

I don't know whether it will help, but clearing 't_ut', and setting 'lazyredraw' might be worth a try:
set t_ut=
set lazyredraw

Perhaps it's also worth setting writedelay option to some value. That may help to identify problems with redrawing, because redrawing is slow downed. You should see if the screen is always cleared

When you say "it flickers" when you use CTRL-F, then it's normal that the screen is redrawn, since the text is updated.
Flicker means that some text is cleared and then the same text is drawn again.  Where do you see that? Only the number column and the status line?
I'm not sure a recording is much help showing what you see.  You can use the "script" command to record the output, comparing Vim and Neovim to see what is sent and finding the relevant parts.  Perhaps a "clear screen" code t_cl appears?

@lacygoill laready had lazyredraw and t_uu haven't change anything.
@chrisbra ok, i have played with with writedelay I'm not sure how it should look but it is even like with writedelay=2 very very slow line redrawing, while in neovim it is still almost instant. For me vim with writedelay=2 == neovim with writedelay=200
Why is it so?

You can use the "script" command to record the output, comparing Vim and Neovim to see what is sent and finding the relevant parts. Perhaps a "clear screen" code t_cl appears?

To record the output:
$ script --quiet --timing=/tmp/script_timing.log --command='vim -Nu NONE' /tmp/script_record.log

To replay the recording:
$ scriptreplay --typescript=/tmp/script_record.log --timing=/tmp/script_timing.log

On Ubuntu, the script command is provided by the bsdutils package:
$ sudo apt install bsdutils

The control sequences are documented here.  The xterm package might provide a local documentation:
$ vim $(locate ctlseqs.txt)

@lacygoill not sure really what is going on
vim output
[2;2R[3;1R[>1;4000;24c]10;rgb:b8b8/afaf/9696\]11;rgb:1f1f/1a1a/1717\P1+r436f=323536\P1+r6b75=1b4f41\P1+r6b64=1b4f42\P1+r6b72=1b4f43\P1+r6b6c=1b4f44\P1+r2332=1b5b313b3248\P1+r2334=1b5b313b3244\P1+r2569=1b5b313b3243\P1+r2a37=1b5b313b3246\P1+r6b31=1b4f50\P1$r1 q\[?12;0$yP1+r6b32=1b4f51\P1+r6b33=1b4f52\P1+r6b34=1b4f53\P1+r6b35=1b5b31357e\P1+r6b36=1b5b31377e\P1+r6b37=1b5b31387e\P1+r6b38=1b5b31397e\P1+r6b39=1b5b32307e\P1+r6b3b=1b5b32317e\P1+r4631=1b5b32337e\P1+r4632=1b5b32347e\P1+r2531=\P1+r2638=\P1+r6b62=7f\P1+r6b49=1b5b327e\P1+r6b44=1b5b337e\P1+r6b68=1b4f48\P1+r4037=1b4f46\P1+r6b50=1b5b357e\P1+r6b4e=1b5b367e\P1+r4b31=\P1+r4b33=\P1+r4b34=\P1+r4b35=\P1+r6b42=1b5b5a\

neovim at the same time
]11;rgb:1f1f/1a1a/1717\

I guess that some escapes are missing from the log; they should be represented with their caret notation (^[).
For example, this sequence:
[2;2R

Might actually be:
^[[2;2R

If so, it's probably how the terminal answers to a request about the cursor position.  Here, it means that the cursor is on row 2, column 2.

Anyway, about this question which was asked previously:

Perhaps a "clear screen" code t_cl appears?

What's the value of the 't_cl' option?
:new | put =&t_cl

For me, it's:
^[[H^[[J

Can you find this sequence somewhere in the log?

I posted the logs ? symbols are escapes as ^[

What's the value of the 't_cl' option?

^[[H^[[2J

Should I scroll for some time for relevent results?

Yes, I think you should.

I can't help much more, but here are 2 other suggestions.
You can attach the logfiles to a github issue by dragging and dropping the files.  This could help preserving special characters, such as raw escapes (and avoid the weird replacement characters �).
Also, you might separate the escape sequences, one per line, with this substitution command:
:% substitute/\ze\e/\r/g

Just to make the logs a little easier to read.

vim.log
neovim.log

@lacygoill @chrisbra Thank You both. At least to understand what is going on.

I looked at only part of the output, but it appears that Vim's strategy is to minimize the text sent to the terminal, while Neovim avoids using the 'screen clear' sequence and instead sends a "clear to end of line' after every line.
At some point the Vim output is "ESC[H" (cursor home) "ESC[2J" (clear screen) and then for each line position the cursor (if there is leading white space) and output the text.
While Neovim does "ESC[H" (cursor home) and prints spaces, then a line of text, then "ESC[K" (clear to end of line) and CR.
As an experiment, you could make a vertical split, which should Vim change strategy to avoid a clear screen.
I wonder what Neovim will do then.

As an experiment, you could make a vertical split, which should Vim change strategy to avoid a clear screen.

Make a split and do the same thing i did before?

Make a vertical split and do the same thing i did before?

I think so, yes.  Start Vim with a vertical split (e.g. $ vim -O /tmp/file1 /tmp/file2), then scroll a little to reproduce the issue.
Don't forget to attach new logs to check how the vertical split influences the issue.

vim.log 10 screen clears
neovim.log only 2
And this is how opening popup terminal window with fzf file fuzzy searching and set writedelay=2 looks like in both vim and neovim
vim https://user-images.githubusercontent.com/6261276/161395203-6a461162-a114-4994-9add-c5f8c6d09965.mp4
neovim
neovim.mp4
Don't say to me you that you don't see the difference @brammool :) and i guess this jumping curssor is what causing "flickering" to me if we skip slow rendering part.

I have the same problem. This happen almost daily when I use Fzf.

script logs for one vertical split of python file with -u NONE vimrc
vim
vim.log
neovim
neovim.log
some visual preview of terminal screen updates with one vertical split
It shows opening the editor itself and then one <C-f> keypress
vim with set writedelay=4
https://user-images.githubusercontent.com/6261276/161414418-a4215448-6291-41d1-a892-fd3bb9b11ea0.mp4
neovim with set writedelay=4
https://user-images.githubusercontent.com/6261276/161414426-bdbdaf39-533e-48ea-baba-d5fcba8a1764.mp4
neovim with set writedelay=150
https://user-images.githubusercontent.com/6261276/161414435-ffc2c56c-c011-4a00-b9a7-2d17b5bc8970.mp4
Which differences my eye can see:
Vim writes content of the first split and the border at the same time, then writes content of the second split
updates line character by character. Hides cursor when it start updating content, then it clears screen line by line, and then pasting new content line by line character by character, then it shows the cursor.
Neovim differ at how it draws border of the split, first it writes content of the first split, then the border, and then content of the second split. It doesn't hide cursor, but like places it at the end of the line it currently process, it doesn't clear the whole updated part it just pastes new content line by line (at least i don't see this teletyping that i see in vim).
@brammool  I'm not sure how else i can convince you that at leat 50x increase in speed of refreshing a screen is worth to implementing for such a greet editor as vim. My perspective is just from regular user, maybe neovim screen updates are broken on some old hardware or platforms, but maybe it can then be implemented as vim flag option or compiler flag?
I understand that its FOSS you don't own me anything, but human knowledge get outdated, how things used in the past  changes too (and if like 20 years didn't recongnize such thing as a problem, doesn't mean it isn't today). I just don't understand your reaction that you don't recognize such slow screen redraws as a problem at all. That's all. No offence.

It appears neovim uses "writedelay" differently, causing most of the difference in speed.
I notice a few things.  Vim clears part of the screen by outputting spaces, then draws text in that space, that looks wrong.
Neovim uses CSI nr X to clear the screen, Vim draws spaces.  What matters here is that there is no highlighting involved.  When there is it might work very differently.  Especially when there is a background color.  And that is much slower, that is what needs to be optimized for.
The original question was whether a vertical split would avoid the screen-clear.  It looks like it does.  So did this solve the flickering?

It appears neovim uses "writedelay" differently, causing most of the difference in speed.

Oh sorry, then my screencasts a almost useless :(

So did this solve the flickering?

Not really, not sure screencasts without writedelay with fps limits could show the difference but let me do some and see myself if it does, so i will then let you know.
Can you also explain a little what is going on with the cursor on my first screencast of popup terminal? Why it jumping around so much?

@brammool
with -u NONE and set nu rnu nowrap ls=2
only one window with <C-f> <C-b> presses or holddowns (for much more frequent flickers)

I can't reproduce flickering of statusline (i guess my custom statusline requires more refreshes or something).
I can't reproduce flickering of LineNr column

vertical split

statusline doesn't flicker
Here is intresting thing LineNr column on <C-f>presses it sometimes flicker (better visible on holddowns). But it doesn't flicker with <C-b> even on holddowns.

with custom vimrc and set nu rnu nowrap ls=2
single window
Visible flickering of both LineNr column for both scroll directions and statusline
vertical split

No flickering of statusline
Same behaviour as with -u NONE it flickers on <C-f>, but doesn't on <C-b>

Can provide logs for all the cases if you are intrested.

Another question. Do you need logs for opening external tool in terminal popup window that causes inadequate cursor jumps or you are not intrested to fix this issue, just once again using such tool in neovim doesn't cause such abnormal cursor jumps?

It appears neovim uses "writedelay" differently, causing most of the difference in speed.

Oh sorry, then my screencasts a almost useless :(

They are useful to see how the redrawing works.  You can't use it to
judge performance.

So did this solve the flickering?

Not really, not sure screencasts without writedelay with fps limits
could show the difference but let me do some and see myself if it
does, so i will then let you know.

The statusline should not be cleared and redrawn now.  I'm not sure how
serious this flickering is, I haven't really noticed it.  Avoiding the
screen clear may cause other redrawing to become slow.  I know most
terminal emulators can clear very fast, while drawing characters takes
much more time.

Can you also explain a little what is going on with the cursor on my
first screencast of popup terminal? Why it jumping around so much?

Probably the cursor is not disabled while redrawing.  For a terminal Vim
cannot know how the job in the terminal uses the cursor.  The cursor is
switched off when drawing, it's back on when it wasn't disabled in the
terminal.
--
ROBIN:  (warily) And if you get a question wrong?
ARTHUR: You are cast into the Gorge of Eternal Peril.
ROBIN:  Oh ... wacho!
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Didn't test this patch by myself yet, but i appreciate any work that you have done to at least reduce such flickering on some cases.

Probably the cursor is not disabled while redrawing.  For a terminal Vim
cannot know how the job in the terminal uses the cursor.

I will log this situation with script for better understanding a little bit later today, so not just to depend on some screencast. So your assumption it is this external tool that makes cursor jump around?
The weird part that this cursor jumps happens when nothing is going on on the screen nor lines clear nor line writing. It just happens for some reason before redraws.
So nothing can be done from vim side? Because neovim doesn't has this problem with the same tool.
I will collect more info and logs and then create new issue, maybe it would help you to trully decide if something can be done from vim's side.

@hungpham3112 should be fixed with the last fzf commit https://github.com/junegunn/fzf/commit/b3ab6311c587e18ee202c5205afc658a33f2a23c
At least it fixes this flying cursors for me, there is still sublte difference with other a little bit flickering parts compare to neovim, but it is now not annoying anymore. Thanks to Bram that points me out that vim is nothing to do with the cursor in a terminal window.

Is this working well enough now that this issue can be closed?

Hello, Bram. Not sure if the problem was addressed. Redraw is behaving as documented, so Vim is doing it jobs right. The problem is that a lot of plugins overuse redraw which is hard to address from the user expirience, and when on Neovim such redraws doesn't do any harm, on Vim they produce a lot of flickering. As linked issue of coc.nvim above.

A way to clear the command line is feedkeys(":\<BS>", 'xt')
Otherwise there is no specific issue remaining, let's close this.
If you can reproduce flicker with an example, feel free to create a new issue.

Another question. Do you need logs for opening external tool in
terminal popup window that causes inadequate cursor jumps or you are
not intrested to fix this issue, just once again using such tool in
neovim doesn't cause such abnormal cursor jumps?

There is support for showing and hiding the cursor in a terminal window,
is that external tool using it?
--
Sorry, no fortune today.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Add CVE severity icons to image page overview card

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #5073

PR #5116   👈
PR #5111

This comment was auto-generated by Graphite.

Looks like some of the e2e failures started cropping up 4 days ago, but aren't 100% consistent:
https://issues.redhat.com/browse/ROX-15888
https://issues.redhat.com/browse/ROX-15889
https://issues.redhat.com/browse/ROX-15934
https://issues.redhat.com/browse/ROX-15891
https://issues.redhat.com/browse/ROX-14757


confirm passing and canceling

Created by sdhar (サミア ダー)

Guessing happens too frequently and would be too annoying to confirm every time.
Passing seems pretty common too.
A 'Are you sure you want to cancel the game? Yes / No' Prompt might be good for cancelling though.
Implementation-wise, a yes/no prompt would need to be asynchronous.

Alternatively, a warning message is displayed the first cancel and a second cancel cancels the game.

Wont fix.


The fmlrc2 used with racon

Dear developer:
After we assemble our genome using raven, we want to use fmlrc2 and racon for genome correction, but I am not sure whether to use fmlrc2 or racon first. Can you give me some suggestions?
Best day!

Hello, I don't think we have any full test data to inform the decision. I would defer to published papers that using a multi-correction strategy to determine if one way is better than the other.

OK，Thank you!


Wowza push module

We are able to ping and telnet from our streaming server to akamai entry points. but  we are getting below msg in logs and stream is not getting pushed to akamai
2012-01-30      18:15:46        IST     comment server  INFO    200     -
PushPublisher#PingResult.onResult[rtmp://p.ep50414.i.akamaientrypoint.net:1935/EntryPoint/definst/a_b_c@50414]:  false
Thanks

Check your akamai URL. the a_b_c at the end looks suspect. are you trying
to run 3 streams?
On Mon, Jan 30, 2012 at 12:30 PM, bsvp <
<EMAIL_ADDRESS>
wrote:

We are able to ping and telnet from our streaming server to akamai entry
points. but  we are getting below msg in logs and stream is not getting
pushed to akamai
2012-01-30      18:15:46        IST     comment server  INFO    200     -
PushPublisher#PingResult.onResult[rtmp://
p.ep50414.i.akamaientrypoint.net:1935/EntryPoint/definst/a_b_c@50414]:
false
Thanks

Reply to this email directly or view it on GitHub:
https://github.com/telvue/WowzaAkamaiPushPlugin/issues/1

--
James Slocum
Software Engineer
Telvue Corporation
16000 Horizon Way, Suite 500
Mt. Laurel, NJ  08054
800-885-8886 ext 252
www.telvue.com

After looking into this a bit, we are getting the same thing with the
newest version of wowza push publish plugin. Using an older version might
solve your problem.
On Mon, Jan 30, 2012 at 12:30 PM, bsvp <
<EMAIL_ADDRESS>
wrote:

We are able to ping and telnet from our streaming server to akamai entry
points. but  we are getting below msg in logs and stream is not getting
pushed to akamai
2012-01-30      18:15:46        IST     comment server  INFO    200     -
PushPublisher#PingResult.onResult[rtmp://
p.ep50414.i.akamaientrypoint.net:1935/EntryPoint/definst/a_b_c@50414]:
false
Thanks

Reply to this email directly or view it on GitHub:
https://github.com/telvue/WowzaAkamaiPushPlugin/issues/1

--
James Slocum
Software Engineer
Telvue Corporation
16000 Horizon Way, Suite 500
Mt. Laurel, NJ  08054
800-885-8886 ext 252
www.telvue.com

i have configured the wowza 3. If i deploy the push publish plugin lder verion, will it work?

We have not tested with Wowza 3. We will hopefully be doing this in the coming weeks. This plugin was originally written for version 2.

Can you provide the wms-plugin-pushpublish for  Wowza Media Server 2 Perpetual 2.2.4 build27452

Hi
we have build the TelvueAkamiPushPlugin module using wowza3.0.3 libraries and deployed it in wowza 2.2.4, but still we are getting same message in logs
2012-01-30 18:15:46 IST comment server INFO 200 -
PushPublisher#PingResult.onResult[rtmp://p.ep50414.i.akamaientrypoint.net:1935/EntryPoint/definst/a_b_c@50414]: false
is  wms-plugin-pushpublish library different for wowza3  and wowza2?
or do i need to get the new plugin for wowza2.2.4
please help
thanks

@bsvp We have updated the plugin to work with the latest version of Wowza's PushPublishAddOn. Please give it a try and let us know how it goes. This should work both with Wowza 2 and Wowza 3.

with update code base i am getting below error, how to resolve this
The method isFlashVerionFMLE(String) is undefined for the type PushPublisherRTMP

thanks ... it is working now

Hi Brookemckim
has this module have any limitation rnumber of streams?  because i tested with one channel and it was working fine.. after that i deployed 8 more channels., but overall only 2 channels are working and for rest all publishing point im getting again same error like...
PushPublisher#PingResult.onResult[rtmp://p.ep50414.i.akamaientrypoint.net:1935/EntryPoint/definst/a_b_c@50414]: false
Thanks

To my knowledge, no. However, this is just built on top of the plugin that Wowza provides and does very little extra work. So it is very possible the limitation is there.