Unable to bind Ctrl-J, Ctrl-M

I would like these to assign Ctrl-J and Ctrl-M to location & navigation dialogs respectively.
If I open clean session of elvish and do:
edit:insert:binding[Ctrl-M] = $edit:navigation:start~

After hitting Ctrl-M my caret goes to the new line with new prompts (as when we hit ENTER).
If I open clean session of elvish and do:
edit:insert:binding[Ctrl-J] = $edit:location:start~

Hitting Ctrl-J works as expected. The problem is however that ENTER itself gets mapped to $edit:location:start~ as well.
Could you help me to understand why these keys cannot be assigned as I would expect?

thats the limitation of terminals. bytes corresponding to Ctrl-M and Ctrl-J are used for carriage return and newline.

In case there is still any confusion I find it's often useful to use the od command to understand what particular keys send. In the example below I've pressed the following keys:  Tab, Ctrl-I, Enter, Ctrl-M, Ctrl-D.
> od -tx1z

0000000 09 09 0a 0a                                      >....<
0000004

Notice that Tab and Ctrl-I send the same character as does Enter and Ctrl-M. You could, in theory bind Ctrl-M to $edit:navigation:start~ but then you'd need to bind some key other than Enter to the $edit:smart-enter~ command.
P.S., You generally can't bind Ctrl-J and Ctrl-M to separate functions because of "icrnl" mode being enabled in the tty driver which makes them equivalent.

elvish could implement Kitty terminal support...

elvish could implement Kitty keyboard protocol support...

Thanks, I filed #1797 for that.


enum or literal js dictionary for type

sup , i have type issue with the type property, am new with this libs.

can you provide a enum or if your not ts friend,  maybe a literal js object dictionary for all types, or just a collection of literal string ?
because it look like we code blind here ! and the ide cant etablis relation

thanks

i also cant found all type list in the doc !
i ask gpt and it give me this, i cant confirm they are all valide because they are not in doc!
if someone can confirm ?

The type properties are all strings.
I think the current list is 'body', 'composite', 'constraint', 'mouseConstraint'. These are mostly intended for internal use in Matter.Composite rather than strict typing.
The TypeScript bindings are not native to this project, so you may want to check with the developers who maintain them if that's what you need.


Documentation of status?

Comment:
In response to Issue #14, on May 17th, 2023 @ocefpaf commented

Folks, this package is unmaintained and we recommend tectonic or miktex instead, both available on conda-forge.

If that's the case, it would be useful to update the README and the description as seen on https://anaconda.org/conda-forge/texlive-core to clearly indicate that fact.

AFAICT this is maintained. There are migration PRs getting merged:

https://github.com/conda-forge/texlive-core-feedstock/pull/85
https://github.com/conda-forge/texlive-core-feedstock/pull/83
https://github.com/conda-forge/texlive-core-feedstock/pull/82
...

Also this was bumped to the 2023 release: https://github.com/conda-forge/texlive-core-feedstock/pull/77
It is also used in several places

That said, think the expectations of this package might be out-of-step with how it is used in practice. Am guessing folks are looking for a fully functional TeX Live distribution based on this package (or the ability to get one started from here). This package is very far from that
Despite the occasional discussion about adding tlmgr, this hasn't really gone anywhere. This in part because this has proved to be a hard problem. This issue ( https://github.com/conda-forge/texlive-core-feedstock/issues/3 ) has been open basically since this package was added and recent attempts haven't been sufficient ( https://github.com/conda-forge/texlive-core-feedstock/pull/78#issuecomment-1504346375 ). This has largely been due to a lack of build options that would support what we would want (hence attempts to graft on other pieces unsuccessfully)
Instead this package has provided the most minimal starting point of a TeX Live distribution and nothing more
Completely understand this is disappointing, but don't really have anything better to offer 😞

So where to go from here...

We could document the status quo (maybe in a README in the recipe and/or the package description)
We can suggest alternatives that are more feature complete like Tectonic
Welcome attempts to finish the tlmgr, but note this has proven to be a hard task
?

cc @conda-forge/core

We could document the status quo (maybe in a README in the recipe and/or the package description)
We can suggest alternatives that are more feature complete like Tectonic

Let's do this for now: document the status quo and suggest alternatives for those looking for something else. If tlmgr gets added at some point, we'll modify the description accordingly.
How does that sound?

BTW, my comment in #14 may be influenced some burnout in maintaining this package. I just couldn't handle the constant barrage of complains. So, by unmaintained I meant: I won't add new features/options to it. Just the bare minimum maintenance stuff, like merging migration PRs.
Also, help to add tlmgr is very welcomed!

Completely understandable. Hoping a clear statement about what this package does and does not do will help calibrate expectations (and associated discussions). Or if nothing else, we will have a document to point to when we get questions like these


Adds Latvian Wikipedia (lvwiki) support and Latvian translation

@googlebot I signed it!

This is awesome @papuass. Just one thing, could you consolidate #154 and #155 into one PR?
The reason I ask you this is that  we expect there will be more than several dozens of PR for adding language support. Therefore, we hope each language support PR to be self-contained and can be used as a reference commit.

@xinbenlv I have consolidated everything into this pull request.

@papuass  I think you need to update a few more files, e.g. to specify the domain name of Latvian Wikipedia. See https://github.com/google/wikiloop-battlefield/pull/133/files for a recent example.
Also, you can use the "Deploy to Heroku" button in your branch (https://github.com/papuass/wikiloop-battlefield/tree/patch-1) to have your own instance of the website and test whether you patch have worked.

Thank you @papuass for this Pull Request. I have deployed a Heroku Review App for this PR (it should have been deployed automatically, I am pinging Heroku support team currently.). You can check the review app for a demo of your change.
Like @hrasyid said, it seems there are a few changes in #133 that this PR lacks. For example the changes in  layouts/default.vue is not there in this PR
Therefore I assume based on the code's behavior, the new language will not yet
show up in the language switch dropdown.

I think I have completed the changes.
Also managed to set up Heroku account (need to authorize it with credit card to launch with Mongo plugin) and deployed my branch.

Good job.

Also managed to set up Heroku account (need to authorize it with credit card to launch with Mongo plugin) and deployed my branch.

By the way you don't have to create a Heroku account / authorzie with your credit card, This project has been setup to create Heroku Review App so you don't have to, but it's good that you created one, just like @hrasyid
Again, thank you for the contribution and welcome to the contributors of WikiLoop, @papuass


Update README.md

Added vue.js library to community libraries section
Microsoft Reviewers: Open in CodeFlow

@DeeJayTC what's the .vue library again @DeeJayTC ?

@DeeJayTC what's the .vue library again @DeeJayTC ?

Found answer here -- very interesting!

@paulcam206 can you help shepherd this in? Thanks


Add text from the structs/unions RFC to the guide

Also renames the original RFC to match the RFC naming policy. I think we should remove the existing RFCs (including #53) in favour of just using the guide, but I wasn't sure what to do with the rationale. I think we discussed adding a rationale section to the guide. Thoughts?

Any more comments on this?

No more comments, merging...


Capcha issue

Hello. I followed all the steps to download the project, but I found a problem with the capcha when trying to login (I don't know what to do with it). Can someone help me please?

Hi @Zikiri7i !
Can you try to open the IMG url to check if prints some error?


the input channel order

I am not sure about the input channel order.the input size is 3,16,112*112(three channels),but channel order should be RGB?or BGR?

I used RGB order for training from scratch, but I'm sorry I couldn't find time to complete the TO-DO's on the readme yet. The models I will release will be RGB order. If you are training from scratch, it doesn't matter. Here, I put an option for using pre-trained models with BGR order. C3D model is converted from caffe and there, it should be BGR. By default opt.bgr is false. See here.

https://github.com/gulvarol/ltc/releases/
Thank you for responding.Is this the model you released?So what's the channel order?I am a little confused..........

Yes I released this upon request after the issue #1. The order for this model should be BGR. I am planning to release instructions how to use it etc. once I have time.

well,I input some data to the network,and find that if the channel order is set to RGB rather than BGR,the prediction accuracy  does not change,but the probability do increase.If the correct channel order is RGB,this is strange.
Besides,now I want to finetune this model based on torch on ActivityNet and Thumos,would this be
feasible？

Please be more precise.
What do you mean by 'prediction accuracy does not change'? Do you train an SVM on top of last layer? In that case it is normal that you get similar prediction accuracy if you use RGB or BGR.
What do you mean by 'probability do increase'? Do you mean the output of last layer is higher when you input BGR? As I said in my previous reply, the correct channel order for this particular model should be BGR.
What do you mean by 'feasible'? In terms of coding, yes it is trivial. But I cannot help you further with this.

I mean,the output of last layer is higher when I input RGB,which makes me confused,because you said that the correct order should be BGR

I found that the released C3D model is with dropout.how can I disable dropout of fc layers?

Yes because C3D architecture has dropout layers. You can remove them if you want. I am closing this issue as this question is unrelated to the title.


Update default control plane and worker instance type to r6i.large

What this PR does / why we need it
Towards https://github.com/giantswarm/giantswarm/issues/30671
Towards https://github.com/giantswarm/giantswarm/issues/30682
This PR updates the default instance type for control plane and worker nodes to be more cost effective. r6i.large is around 1/2 the price of r6i.xlarge, and can adequately provide enough resources for a default cluster.
Checklist

[x] Updated CHANGELOG.md.


inverted instance number

i have get latest version from master branch
In the ViewportOverlay i get fully inverted instance_number value of current slice
For example:
if i placed on the first slice of 112 that i get instance number equals to 112
if it is second slice that i get instance number equals 111
and so on
if i try get image info on the first slice in series via cornerstone.metaData.get('generalImageModule', imageId) that i get wrong instance_number inside

i found separate function for revert instance_number value in array
Does anyone know why?

Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing
Please try and re-open if the issue persists


Consider option to centrally remove (COMPACT) from beginning of messages

Original: https://github.com/Mic-M/ioBroker.logparser/issues/36
Describe desired Enhancement !
Log-Parser already has options to remove the pid and to remove script.js.scriptname.
Please consider an additional option to remove (COMPACT) at the beginning of all messages which are output by an instance running in compact mode. See example:
snmp.0 | 2022-08-08 13:43:50.287 | debug | (COMPACT) handleConnectionInfo

Added option to remove only (COMPACT). Will be available in next release.

Added in 2.1.0


Update avl-tree.c

Why you are not freeing the tree. You have to free it because it is also a pointer and we have to free it if you does not free it it will become a dangling pointer
DO NOT SUBMIT PULL REQUESTS HERE.
Because of spam, an automated bot will automatically close all pull requests
opened on this repository. If you've found a legitimate bug in the source code,
you can open an issue to report it.
This project isn't a dumping ground for random bits of C code you've written.
That goes double if you're being instructed to send pull requests as part of a
competition or a school class.

Not a bug.

No, you have not understood the code. Your PR introduces a bug. To demonstrate, please run make check on your change to invoke the unit tests, or you can check out this continuous integration run to see.


More precise completions and namespace member completions

Feature overview
This PR enables:

Much more precise keyword completions where we only show the relevant syntax at a particular cursor location. For example:

within { } |

At the cursor location indicated by |, we should only get the keyword apply .

More precise completions for names where we take the syntactic context into account. For example:

operation Foo(x : | )

Here, we should only get type names.

Namespace member completions. For example:

import Std.Diagnostics.|

The completions here should return only items that exist under Std.Diagnostics.
Code changes
Parser completions module
Introduce the completion module in the parser.

possible_words_at_offset_in_source - a new API to return completions at a particular offset . Uses the parser directly as the best source of knowledge about what tokens to expect.
Introduce the WordKinds flags to define the different kinds of word tokens that the parser can expect during parsing. (See compiler/qsc_parse/src/completion/word_kinds.rs for longer description)
See compiler/qsc_parse/src/completion/collector.rs for a description of how this all works.
Sprinkle expect()s at all possible locations in the parser where we expect a word.

Error recovery - AST changes

Introduce PathKind::Ok and PathKind::Err .

This new approach allows for a valid AST node to exist even when path parsing fails for a partial path. This is good for syntax such as "open Std." where we need:

The identifier nodes preceding . to be valid, so we can use them to look up possible completions.
The open statement to also be a valid node, so we can tell that the context requires a namespace (and not, say, a type name).

Most existing code will continue to only process valid paths.

Remove Idents struct and replace it with a trait implemented by various types such as Path and Ident slices. This should make the convenience methods easier to use without cloning in/out of the different types we need.

Error recovery - parser changes

Recovering struct parsing, so that e.g.  new Foo.  is still parsed as a valid struct expression node, allowing us to access the Foo ident from the AST
Recovering open statement parsing so that open Foo. can be recognized as an open item in the AST.

Language service
In language_service is essentially a complete rewrite of completions.rs, so I suggest reviewing the new code only and ditching the diff.
Other parser changes

Move implicit namespace parsing a bit so that it works better with the completion  mechanism.

This also fixes the  spurious "expected item, found eof" error when document only contains one token

Delete the dead path  path_field_op
Add some missing keywords to item parsing recovery (export, import)

Other notes

Locals completions exclude imports from locals (they conflict with globals already added to completion) (fix in resolve.rs)
language-configuration.json : and removed wonky word pattern in playground and VS Code because we don't need it anymore now that we have proper . completions! Also update trigger characters.
Only include the samples snippets in empty documents, because now with the more concise completion list, it gets quite intrusive for them to show up at every word when typing code.
Update integration test to account for the fact that completions are more accurate now.

Not in this PR (coming next)

Struct and UDT field access completions.

Change in memory usage detected by benchmark.
Memory Report for 6fbbec5
Test	This Branch	On Main	Difference
compile core + standard lib	17940700 bytes	17932132 bytes	8568 bytes

I ran some tests to narrow it down. This increase is attributable entirely to the changes in the AST. A bunch of Path or Idents fields have been replaced with PathKind, a Span field has been added to ImportOrExportItem. It adds up to a net increase in memory.
I can't think of a way to collapse the AST any further, we are storing more information in there after all. So I think the 0.04% increase in memory usage is a worthwhile tradeoff for the feature.

@idavis found a bug:
Trying to complete
import QIR.in;

causes a panic since in is a keyword and that's a parse tree I did not expect to have to handle....
Working on the fix.

@idavis found a bug:
Trying to complete
import QIR.in;

causes a panic since in is a keyword and that's a parse tree I did not expect to have to handle....
Working on the fix.

Fixed with my latest commit.

Fixed with my latest commit.

Okay, I regressed things pretty badly with this fix, as @ScottCarda-MS discovered through testing (thanks!).
The following piece of code should produce a mostly valid AST (save for the error on line 3).
namespace Sample {
    operation BellStates() : (Result, Result)[] {
        Std.
        let bellStateTuples = [];

        for (label, prepare) in bellStateTuples {}
        return measurements;
    }

    /// doc comment
    operation PreparePhiPlus() : Unit {
    }
}

With my latest commit, there's no AST at all.
cc reviewers @idavis  @swernli . I'm going to have to go back to the drawing board with the "keyword after dot" fix that handles QIR.in and similar paths. Expect more parser changes :/

I pushed what I think is my final update. The parser change to fix the last issue ended up being minimal!
Please sign off if no more concerns.


ZSTD snapshotting compression

Description
This PR adds support ZSTD compression for snapshoting metadata. ZSTD compression for indexes is out of scope since such compression must be supported by Lucene.
There is a PR in Lucene https://github.com/apache/lucene/pull/439 with support of ZSTD, but they do not want to merge it so far.
Issues Resolved
https://github.com/opensearch-project/OpenSearch/issues/2192
Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

A cursory look at this looks good to me.
How does one use it?
I'd like @nknize to take a look please.

During creation of a repository set compression type (default is deflate, how it works now):
{
  ...
   "settings": {
      "compress": true,
      "compression_type": "zstd", // `deflate`, `lz4`
   }
}

Looking pretty good! Thanks for doing this. I left some comments.
I think we should add a none compression_type option to the DSL API that nulls out the compressor instead of adding a NullCompressor:
{
  ...
   "settings": {
      "compress": true,
      "compression_type": "zstd", // `deflate`, `lz4`, `none`
   }
}

We also need to widen the test coverage to include LZ4 and no compression.

Yes will none much better than Null will change

@willyborankin Hey, would you like to address the gradle check failure and merge this PR?

@willyborankin Have you ever benchmarked this to measure the actual impact on snapshotting and restore?

Hi @scrawfor99, fixed diff. AFAIR I addressed all changes @nknize asked to do a long time ago. After that I was waiting for the plugin for index compression to be merged. I fixed all conflicts.

FYI, there is only one plugin (outside core) that implements repository for OCI [1], it may need minor adjustments (primarily, to use the settings centralized in BlobStoreRepository), otherwise the change should not break anything.
[1] https://github.com/opensearch-project/opensearch-oci-object-storage/

FYI, there is only one plugin (outside core) that implements repository for OCI [1], it may need minor adjustments (primarily, to use the settings centralized in BlobStoreRepository), otherwise the change should not break anything.
[1] https://github.com/opensearch-project/opensearch-oci-object-storage/

good point should I connect with them?

FYI, there is only one plugin (outside core) that implements repository for OCI [1], it may need minor adjustments (primarily, to use the settings centralized in BlobStoreRepository), otherwise the change should not break anything.
[1] https://github.com/opensearch-project/opensearch-oci-object-storage/

good point should I connect with them?

Open an issue in that repo so we don't forget.

@nknize mind to resolve your comments as well (may be something pops up)? (the merge is still blocked)

I would love to. But apparently you can't resolve outdated (missing) conversations. And the alleged "workaround" doesn't work on mobile view. I'm happy to merge as admin to override these pearly github gates.

Thanks @willyborankin for your contribution and to everyone for your tenacity on this long running PR.

@willyborankin mind sending manual backport against 2.x? thank you

@willyborankin mind sending manual backport against 2.x? thank you

sure will do


fix(runtime-core): When computed is executed early, the update method of instance in setupRenderEffect should be called using callWithErrorHandling

close #11286

This is no longer necessary after ee0248acc


Fix float, fix constructor tests

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

Project coverage is 84.30%. Comparing base (c2c3012) to head (d0240a6).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #110      +/-   ##
==========================================
+ Coverage   81.10%   84.30%   +3.19%
==========================================
  Files          18       18
  Lines         757      758       +1
==========================================
+ Hits          614      639      +25
+ Misses        143      119      -24

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


Calculate width using "tput cols" command

This isn't portable across platforms. If you'd like to use tput than it should be done as a shell expansion to gcalcli's width argument.

Totally agree. Closing out


Have the node name use the Source for Files tag

For example:
<folder source="RandomFolder\131\ELECTRIFY No Hands Glow" destination="" priority="1"/>
would have the name "ELECTRIFY No Hands Glow"

The same with files minus the file extension

Duplicate but added some info. Refer to #3, #5.


 tag throwing aria prop warning

react-dom.development.js:506 Warning: Invalid aria prop `aria-role` on <input> tag.
    in input (created by Context.Consumer)
    in StyledComponent (created by InputStyles)
    in Input (created by ForwardRef)
    in div (created by Context.Consumer)
    in StyledComponent (created by CurrentValue__DefaultWrapper)
    in InnerReference (created by Context.Consumer)
    in Reference (created by Downshift)
    in Manager (created by Downshift)
    in div (created by Downshift)
    in Downshift (created by Select)
    in Select (created by InfluxDbPage)

Seems like it comes from Downshift. Might need to look into why they apply that.


SHA-256 is fixed

In the XML:
<Item Key=" XmlSignatureAlgorithm">Sha256</Item>
But in the ADFS configuration, step 12:
"select the Advanced tab and change the Secure hash algorithm to SHA-1 or SHA-256, and click Ok."
It should say "Use SHA-256". SHA-1 won't work.

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

ID: e14a5d4c-5125-8ed2-7fda-38aee1296b76
Version Independent ID: 054d298b-937e-9f01-ae56-7025e47e4eec
Content: Add ADFS as a SAML identity provider using custom policies in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-custom-setup-adfs2016-idp.md
Service: active-directory
GitHub Login: @davidmu1
Microsoft Alias: davidmu

@PRMerger18 Thanks for your feedback! We will investigate and update as appropriate.

Removed the reference to SHA-1.
#please-close


Updating Credentials

The instructions state that we should update the credentials if we delete a Flow owner but they don't state how to do this task. Please add this information

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

ID: a50a5424-6bb4-c34c-bb82-991a5dd3e0df
Version Independent ID: ea8ac9ee-e014-ad33-59f5-5d15f8a38ad8
Content: Learn how to add other owners to a flow and create team flows - Microsoft Flow
Content Source: flow-docs/create-team-flows.md
Service: flow
GitHub Login: @MSFTMan
Microsoft Alias: deonhe

@DeanGross Thanks for using Microsoft Flow, and for reporting this issue. I'll take a look at it. - Deon


Can't connect to local MySQL server through socket '/var/lib/mysql/mysql.sock' (2)")"

When i tried to install zabbix server 3.2 in centos7.3 ,the mysql database cannot be created.
The error message is:
TASK [dj-wasabi.zabbix-server : MySQL | Create database] ***********************
fatal: [zabbix.paastest.cn -> zabbix.paastest.cn]: FAILED! => {"changed": false, "failed": true, "msg": "unable to find /root/.my.cnf. Exception message: (2002, "Can't connect to local MySQL server through socket '/var/lib/mysql/mysql.sock' (2)")"}
to retry, use: --limit @/etc/ansible/roles/main.retry
The DB has been installed in the host:
Package 1:mariadb-5.5.52-1.el7.x86_64 already installed and latest version

Same error on Ubuntu 16.04 LTS and mysql
fatal: [master -> None]: FAILED! => {"changed": false, "failed": true, "msg": "unable to find /root/.my.cnf. Exception message: (2002, \"Can't connect to local MySQL server through socket '/var/run/mysqld/mysqld.sock' (2)\")"}

Which Ansible role do you use for installing MySQL|MariaDB?

I'm using mysql. the default that comes with your playbook

@dj-wasabi any ideas?

This role will only install Zabbix Server and depending on a property, MySQL or PostGreSQL is used and a database is configured and data is installed. You'll have to find a suitable Role for installing the database, in this case mysql. The geerlingguy.mysql is a very good one.

oh so, I though its contained in it, so I just have to install that from ansible-galaxy and it will work? or do I have to add something? @dj-wasabi

I can confirm that this work, sorry, I thought that this is handled internally by your role.

Iam getting error to create database while ansible playbook
PLAY [database] ***************************************************************************************************************************
TASK [Gathering Facts] ********************************************************************************************************************
ok: [<IP_ADDRESS>]
TASK [mysql : install mysql] **************************************************************************************************************
ok: [<IP_ADDRESS>]
TASK [mysql : install Mysql-python install] ***********************************************************************************************
ok: [<IP_ADDRESS>]
TASK [mysql : create database] ************************************************************************************************************
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "msg": "unable to find /root/.my.cnf. Exception message: (2002, "Can't connect to local MySQL server through socket '/var/lib/mysql/mysql.sock' (2)")"}
[WARNING]: Could not create retry file '/etc/ansible/site.retry'.         [Errno 13] Permission denied: u'/etc/ansible/site.retry'
PLAY RECAP ********************************************************************************************************************************
<IP_ADDRESS>               : ok=3    changed=0    unreachable=0    failed=1

Hi @irparaja
Do you already have a MySQL service running on the host?

Hi,
i have a small doubt here that is i am running five docker containers with ubuntu kernel, if the ubuntu kernel is went something wrong then what about the five docker container, at the time containers going down or still running.


Use DotNetStaging service connection credentials to gather drop

Summary
Validation build: https://dev.azure.com/dnceng/internal/_build/results?buildId=2552320&view=logs&j=9c2ccffd-8e7d-5b34-ccf9-d3f1053322e9&t=4fd9d253-d592-541b-cb24-178afb820382

Release Notes Entry

/backport to main

/backport to release/8.0

/backport to release/9.x


Use batch requests for actions on multiple entities

When I select a lot of clients and delete them, Uchiwa dispatches one XHR request for each delete operation, which is very slow.
Using a batch request to post a JSON "delete" command to a batch URL would speed this up.
If Sensu doesn't have batch requests, let's get an issue opened there as well.

Unfortunately I don't think the Sensu API offers such capability, according to this documentation: https://sensuapp.org/docs/latest/api-events

Here is the issue in Sensu to implement a batch API.

:+1:

I'm going to close this issue since it will not be something available before Sensu 2.0!

Why not just tag this issue "v2.0"?
We're going to lose track of it if you close it.


Table-question-answering Task Page

This is the tracker for completion of task page for table-question-answering task. The page is here: https://huggingface.co/tasks/table-question-answering. You can contribute parts of a task or the whole task! (There's no single assignee to a task, these are wikipedia-like pages)
You can refer to https://github.com/huggingface/hub-docs/issues/369 if you want to contribute! 🤗
List of items to contribute:

[ ] Task description
[ ] Subtasks (if there is any)
[ ] Most used dataset for the task
[ ] Most used model for the task
[ ] Metrics that are used to evaluate the task
[ ] Libraries used for the task
[ ] Use cases
[ ]  Small snippet for inference that demonstrates the task

Hi Merve, I want to contribute to this task.

@merveenoyan @haotieu2001 hello are Libraries used for the task and  Subtasks (if there is any) still left? I would like to contribute to theses tasks

@haotieu2001 you can 🙂 there's no open PR for this from what I see.
@congaco yes you can contribute them 🙂


Auct 463 registration modal

This PR is https://github.com/artsy/force/pull/4346 with a fresh lockfile (copied from the staging branch).

Warnings

:warning:
The staging version of Metaphysics has breaking changes, this might not affect your build, but could mean your queries don't work the way you expect. See below for breaking changes:

:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceOrder.lastApprovedAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceOrder.lastSubmittedAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceOrder.stateExpiresAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceOrder.stateUpdatedAt'
:warning: Field 'CommerceLineItem.createdAt' changed type from 'CommerceDateTime!' to 'String!'
:warning: Field 'CommerceLineItem.updatedAt' changed type from 'CommerceDateTime!' to 'String!'
:warning: Field 'CommerceFulfillment.createdAt' changed type from 'CommerceDateTime!' to 'String!'
:warning: Field 'CommerceFulfillment.estimatedDelivery' changed type from 'CommerceDate' to 'String'
:warning: Field 'CommerceFulfillment.updatedAt' changed type from 'CommerceDateTime!' to 'String!'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceBuyOrder.lastApprovedAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceBuyOrder.lastSubmittedAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceBuyOrder.stateExpiresAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceBuyOrder.stateUpdatedAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceOffer.submittedAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceOfferOrder.lastApprovedAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceOfferOrder.lastSubmittedAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceOfferOrder.stateExpiresAt'
:warning: Argument 'convert_to_utc: Boolean' was removed from field 'CommerceOfferOrder.stateUpdatedAt'

  Generated by :no_entry_sign: dangerJS against f023ef071358bec0df597f894c926c947ebfedc6


truly random names using uuid

customers pointed out that sometimes the random names were being repeated. This happened because they were setting seeds in their code, which would cause the random generator to reset and return the same name.
UUIDs are not affected by the seed in the same way (tbh not sure how thats possible), but this prevents the issue from occuring. We tac on 5 characters from a uuid to make sure the name is random in the case of a seed being set

Codecov Report

Merging #594 (b9df456) into main (91bc93a) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:exclamation: Current head b9df456 differs from pull request most recent head 222a5de. Consider uploading reports for the commit 222a5de to get more accurate results

@@            Coverage Diff             @@
##             main     #594      +/-   ##
==========================================
- Coverage   84.87%   84.86%   -0.01%
==========================================
  Files         161      161
  Lines       12587    12590       +3
==========================================
+ Hits        10683    10685       +2
- Misses       1904     1905       +1

Impacted Files
Coverage Δ

dataquality/integrations/ultralytics.py
86.63% <100.00%> (ø)

dataquality/utils/name.py
100.00% <100.00%> (ø)

tests/utils/test_name.py
100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Unable to extract video description

Checklist

[ ] I'm reporting a broken site support issue
[ x] I've verified that I'm running youtube-dl version 2019.11.28

$ pip freeze | grep youtu
youtube-dl==2019.11.28

[x ] I've checked that all provided URLs are alive and playable in a browser

$ cat LISTAMUSICA
https://www.youtube.com/watch?v=3fxq7kqyWO8
https://www.youtube.com/watch?v=DT61L8hbbJ4
https://www.youtube.com/watch?v=Gc3tqnhmf5U
https://www.youtube.com/watch?v=U-ZNfo_NlIU
https://www.youtube.com/watch?v=B7xai5u_tnk

[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped

[x] I've searched the bugtracker for similar bug reports including closed ones
This #23331    has been closed without comments

[x] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE
$ youtube-dl LISTAMUSICA --verbose -i
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['LISTAMUSICA', '--verbose', '-i']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.11.28
[debug] Python version 3.6.9 (CPython) - Linux-5.0.0-36-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] LISTAMUSICA: Downloading webpage
[youtube] LISTAMUSICA: Downloading video info webpage
WARNING: Unable to extract video title
WARNING: unable to extract description; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
ERROR: This video is unavailable.
Sorry about that.
Traceback (most recent call last):
  File "/home/obiwan/Music/youtubedl/lib/python3.6/site-packages/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/home/obiwan/Music/youtubedl/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/home/obiwan/Music/youtubedl/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 2148, in _real_extract
    raise ExtractorError(error_message, expected=True)
youtube_dl.utils.ExtractorError: This video is unavailable.
Sorry about that.

Description

WRITE DESCRIPTION HERE
Latest youube dl is failing to download videos from list.
I guess a new release might be needed?
Tested and I am able to play each video on my Firefox browser

incorrect option order(-i option should be followed by batch file name).


Fails to start my chain

I am trying to utilize the tests with my cosmos-sdk chain, but it can't start it. It fails with
?panic: validator set is empty after InitGenesis, please ensure at least one validator is initialized with a delegation greater than or equal to the DefaultPowerReduction

What may it be? My current chain config is
[]*interchaintest.ChainSpec{
		{Name: "dymension", Version: "latest", ChainConfig: ibc.ChainConfig{
			Type:           "cosmos",
			ChainID:        "dymension_100-1",
			Images:         []ibc.DockerImage{{Repository: "dymension", UidGid: "1025:1025"}},
			Bin:            "dymd",
			Bech32Prefix:   "dym",
			Denom:          "udym",
			CoinType:       "60",
			GasPrices:      "0udym",
			GasAdjustment:  0,
			TrustingPeriod: "168h0m0s",
		}},
		{Name: "osmosis", Version: "v7.2.0", ChainConfig: ibc.ChainConfig{ChainID: "osmosis-1001"}},
	}

Dymension fails, osmosis works.

Hey! Thanks for reaching back to me. I don't think this is the problem, as I am pointing to a local docker image and it is being fetched successfully. The problem happens when the test try to start the container chain, which throws the error I have been written below: ?panic: validator set is empty after InitGenesis, please ensure at least one validator is initialized with a delegation greater than or equal to the DefaultPowerReduction

Are you familiar with this error by any chance?

@ItayLevyOfficial Gotcha - could you make it []ibc.DockerImage{{Repository: "dymension", Version: "latest", UidGid: "1025:1025"}},
(make sure version is latest and not local. Some will build as dymension:local)
if this does not fix it, please send me your entire log via a pastebin (or gist) and I will dive in further

Hey! I've made the changes, but the error persists. I believe I've pinpointed the issue. Our chain, "Dymension," utilizes 18 token decimals, while the default in Cosmos is 6. We adopted 18 decimals for compatibility with Ethereum. The validator delegation specified by interchaintest in the genesis configuration is<PHONE_NUMBER>000, likely assuming 6 decimals. This amount is beneath the hardcoded minimum delegation in our chain's code. Do you know if there's a straightforward way to adjust the chain's decimal settings?

Apparently there is a ModifyGenesisAmounts parameter to modify those, I will try and see if it solves the issue

@ItayLevyOfficial I just checked and interchaintest does not yet have ethermint support. This should become easier to add after the Avalance support PR is added. But for now due to the use of hex addresses, I am unsure if interchaintest can handle this properly.

Ye. Also the genesis amounts are limited to less then 10 tokens as long as you use 18 decimals due to this line. Is there an open issue on this? We want to utilize your framework for our chain to test our IBC connection with dymension rollapps.

I started utilizing the sdk's Int type, which is a wrapper around big.Int, in a few places some months ago to get support for Penumbra off the ground since they use int128 iirc. I'm not sure if there is an open issue for the limitation that you have pointed out but I personally am not opposed to the proliferation of the sdk.Int type throughout the framework given we have a few chains in the pipeline who need support for values greater than what a 64bit integer can represent.
#679

Completed. Ethermint support is added in the main branch

Hey, thanks for the update. When trying to upgrade the dependency and run my test I get ```itaylevy@Itays-MacBook-Pro dymension_interchaintest % go test ./...
github.com/cosmos/cosmos-sdk/types
../../.gvm/pkgsets/go1.21.0/global/pkg/mod/github.com/cosmos/cosmos-sdk@v0.50.0-rc.1.0.20231026141021-0469fc17e158/types/context.go:64:29: undefined: comet.BlockInfo
../../.gvm/pkgsets/go1.21.0/global/pkg/mod/github.com/cosmos/cosmos-sdk@v0.50.0-rc.1.0.20231026141021-0469fc17e158/types/context.go:91:36: undefined: comet.BlockInfo
../../.gvm/pkgsets/go1.21.0/global/pkg/mod/github.com/cosmos/cosmos-sdk@v0.50.0-rc.1.0.20231026141021-0469fc17e158/types/context.go:299:48: undefined: comet.BlockInfo
FAIL	dymension_interchaintest [build failed]
FAIL
itaylevy@Itays-MacBook-Pro dymension_interchaintest ```
Are you familiar with with this error? @Reecepbcups

@ItayLevyOfficial Were you using the v7 branch before? Can you share me a gist / pastebin of your code? I did not run into this issue in my test on build

Nop, v8. I just copied the learn_ibc_test.go file to a new repo i created and I try to make it work with our chain. You can take a look here --> https://github.com/dymensionxyz/dymension_interchaintest/tree/itay-games

I looked a bit more into this – you have github.com/cosmos/cosmos-sdk v0.50.0-rc.1.0.20231026141021-0469fc17e158 as a dependency in interchaintest, and this version doesn't compile. There is a closed issue on it here, but there hasn't been a new release with a fix yet. Do you have any idea how I can get my code to compile?

@ItayLevyOfficial can you confirm if the latest ictest main commit solves this for you? (sdk v0.50.1, seems the have had many issues tagging upstream)
if not, replacing the cosmossdk.io/core => v0.11.0 in your go.mod should resolve it

Confirmed. Thanks a lot buddy!

Now I am getting this error
Error: error creating clients: failed to create client on src chain{dymension_100-1}: failed to send messages on chain{dymension_100-1}: no concrete type registered for type URL /ethermint.types.v1.EthAccount against interface *types.AccountI

When trying
	require.NoError(t, ic.Build(ctx, eRep, interchaintest.InterchainBuildOptions{
		TestName:  t.Name(),
		Client:    client,
		NetworkID: network,
		// BlockDatabaseFile: interchaintest.DefaultBlockDatabaseFilepath(),

		SkipPathCreation: false},
	),
	)

Do you have any idea why this may happen?

@ItayLevyOfficial You need to register the ethermint Interface type(s) for your test since Ethermint is it's own Module.
ibc.ChainConfig example: https://github.com/CosmosContracts/juno/blob/main/interchaintest/setup.go#L81-L110

Tried to run go get github.com/evmos/evmos/v12/crypto/codec to get the Ethermint interface types but got
go: dymension_interchaintest tested by
	dymension_interchaintest.test imports
	github.com/evmos/evmos/v12/crypto/codec imports
	github.com/evmos/evmos/v12/crypto/ethsecp256k1 imports
	github.com/evmos/evmos/v12/ethereum/eip712 imports
	github.com/cosmos/cosmos-sdk/simapp/params<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: cosmossdk.io/simapp
	        but was required as: github.com/cosmos/cosmos-sdk/simapp

Do you know how it can be solved? I think this is because evmos is using a different cosmos-sdk version

@ItayLevyOfficial likely due to changes in v8 vs v7. I need to see if we can backport math.Int to v7 branch since ethermint is not using that hmm

Cool ping me when there's an update on this

Can you re-open this issue pls?

Ahh, looks like Ethermint on SDK v46 is causing some problems in relation to all imports. The v7 branch is SDK 47, which uses the new cosmossdk.io/ namespace instead of github.com/cosmos/cosmos-sdk/simapp. This has cascading effects through IBC-Go, Tendermint/CometBFT, Ethermint, and the Cosmos-SDK.
So it looks like this has to be ported to the v6 interchaintest branch instead of v7 as originally hoped.

@ItayLevyOfficial I have the interchaintest case passing now for evmos

Currently I use a slightly modified relayer (to register the extra ethermint codec for accounts) but will be modifying this in interchaintest to allow for standard relayers :)

Thanks a lot man!


View title doesn't update after modal dismiss

Short description of the problem:
When you are on a view, and you open a modal, the page title (only visible on web) updates to the modal title (ion-title)
When you dismiss the modal, the title (from the modal) remains, even though the modal is no longer in view
What behavior are you expecting?
I expect that when you dismiss the modal, that the page title to revert back to using the title from the now visible view, not the previously shown modal
Which Ionic Version? 1.x or 2.x
2
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 5.4.1
Ionic Framework Version: 2.0.0-beta.10
Ionic CLI Version: 2.0.0-beta.31
Ionic App Lib Version: 2.0.0-beta.17
ios-deploy version: 1.8.5
ios-sim version: 5.0.4
OS: Mac OS X El Capitan
Node Version: v5.3.0
Xcode version: Xcode 7.3.1 Build version 7D1014

Hello, thanks for opening an issue with us! Would you be able to provide a plunker that demonstrates this issue? After testing and looking at our source, a modal should not be updating the title at all. Thanks for using Ionic!

Hello Justin
You mean a modal, when brought INTO view, should not update the title? So there was not an expectation that when the modal dismisses, that it shout revert the title...... I see.
Let me see what I can create on Plunkr......
Not a show stopper this.

The title of a view, in plunkr, is plunkr...... so this isn't going to be possible

Hmm, would you be able to provide a repo that demonstrates this issue? Thanks!

After reviewing this issue and the modal source i am going to close as i cannot reproduce this issue and the modal source does not have the bit of code code to update the title. Thanks for using Ionic!

FYI, this fixed itself in beta 11....

this issue seems back again
Ionic Framework: 3.2.1
Ionic App Scripts: 1.3.7
Angular Core: 4.1.0
Angular Compiler CLI: 4.1.0
Node: 6.9.5
OS Platform: macOS Sierra
Navigator Platform: MacIntel
User Agent: Mozilla/5.0 (iPhone; CPU iPhone OS 9_1 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Version/9.0 Mobile/13B143 Safari/601.1

it is reproducible on the 'super' template when you go to 'New item' page and after closing it.
cli packages: (/usr/local/lib/node_modules)
@ionic/cli-utils  : 1.10.2
ionic (Ionic CLI) : 3.10.3

local packages:
@ionic/app-scripts : 2.1.4
Ionic Framework    : ionic-angular 3.6.1

System:
Node : v6.11.2
npm  : 5.4.1
OS   : macOS Sierra


[FIZZ-50] Retry logic and exception handling - DRAFT

Arguably still needs unit tests for the new facade.py - although since those lines of code came from main.py and we accepted them before, I think there's a reasonable argument to keep progressing on the sprint by accepting that they are not unit tested, and creating a ticket to address that. There are additional file types to create and we should not skip the facade testing when we get to those additional tests.
I created a quick-and-dirty API test harness in JavaScript - took me only a few minutes after struggling to get a Python-based simple service running. Can use that in combination with the SCHOOLOGY_BASE_URL environment variable to inject different HTTP status codes into the response. Although the requests module handles these status codes in standard ways, it was useful to see the results in action and for testing that our code now handles these different statuses.

For doing a fake API server, take a look at Pretenders.  It's Python, and the example shows how you can spin up and configure a server right in your PyTests.  It looks like the project hasn't been updated in a while, but I don't see that as an issue for this kind of testing tool.


Support for merging into existing assembly

With unity assemblies changing the order of fields on classes breaks the game. I would like to merge into an existing AssemblyDefinition instance, in the hopes that the metadata for the original assembly stays the same, and only merge another assembly into it.
I figure something along the lines of setting ILRepack.TargetAssemblyDefinition could provide an easy solution

Just to confirm, "Unity assemblies" = Unity3d?
Do you have a sample of such breakage? Seems weird as we don't change the order or fields (unless cecil does it)

Yes I ment Unity3D. Issue may have had different cause, you can close the issue. If I find a solid repro i'll comment here again


emoji support

Mac OS has a feature where you can bring up an emoji keyboard with Control-Command-Space. I know Android and iOS also have this, and I'm sure windows does too.
My users want to be able to use emoji's in their prototype games. The problem is that PTT seems to cut off the top of the emoji. Not sure why. Here's an example:

I'm not able to reproduce this issue.
Here's my code and output...

@rizen Can you send me some sample code that is causing the issue or check to see if something else is clipping the text?

@rizen Are you still seeing this issue? If not, I'm ready to close it.

BTW, I'm pretty sure you can test this with windows using an onscreen keyboard (or by copy pasting emoji from another program)

I've figured out how to replicate it. The unfortunate bit is that I think this may be something broken at the browser level.
I've created a fiddle showing how it's broken: https://jsfiddle.net/plainblackguy/6r9hzgef/4/
It has to do with the base font that is loaded, in this case Bebas Neue from Google Fonts. However, I'm sure it will happen with some others too.
On Safari it looks like this:

The top is cut off.
On Firefox it looks basically the same as Safari:

On Chrome it instead looks like:

The top isn't cut off, but instead there's an extra gap between the first word and the emoji
Not sure why the browsers look different.

@rizen Well, this is not good. One of my concerns going into this was the need to generate print-ready bitmaps that  to match between the version the users create and the version you print. I (perhaps naïvely) hoped to avoid any cross browser issues and I had assumed that the best way to do this was to render the text into a canvas / pixi. But unfortunately, this proves that the way different browsers render into canvas, or at least how they render text and non-system fonts, is not 100% consistent.
I expect ensuring the image matches what the client sees is critical for your app. To my knowledge and gut instinct,  this is not something we will be able to fix completely, but I can think of some ways it might be managed (more on that later).
I created a detailed table of results on different browsers.
https://airtable.com/shrvTQjAM5kx3Agkv
(most of them are using this slightly different fiddle as a starting point https://jsfiddle.net/sqa5uz3e/)
Notes:

Windows, Mac, & Android (not in table) use different emojis
Text all seems to work much better with system fonts (arial, segoe ui) than with custom font loaded at runtime
I needed to reload JSFiddle each time to render the correct font since pixi's text was being created before the font loaded. In Chrome only I saw issues where the font loaded but had spacing issues until I loaded it again.
This demo is running under Pixi 6.1
I saved the images straight from the canvas in most cases but firefox and safari don't support this. I'll need to redo some of the firefox ones. The output from the canvas buffer seems to be slightly different from the screenshot version of the same browser.
Some browsers render exactly the same (e.g. edge & chrome). Firefox appears to look almost exactly the same when i overlay a screen shot of firefox on a screenshot of edge.
Firefox looks blurry to me until I bump up the resolution to 2.

Some more things to test to help get a better idea of what's going on:

Test with a plain Text object (instead of PTT)
Test with Pixi v5
Test with other custom fonts
Test with a delay to make sure custom font is loaded before creating the pixi app and rendering

Some ideas for possible fixes:

After doing a little research into which is the most stable / predictable browser, or which one is the most widely used by your users, require that they use, or at least proof the images pre-print on, a specific browser.
When your users save their files, generate a PNG from their local version, save it and make that the canonical version. Likewise, you may want to record what version of browser they use to create their files and warn them if you detect that it has changed at any point.
An extreme solution could be to render the image using an instance of a renderer running on the server and then serving that image to the user. Or I suppose you could generate them only before printing and show a proof (similar to number 1.)
We could search for emojis and automatically wrap them in special tags that would render them with a system font.
You could search for emojis and automatically swap them out with png icon tags.

Let me know what direction you think might be best for handling this one.

Update:
Tested chrome with PIXI.Text instead of Pixi-Tagged-Text and it has the same issue with cutting off the top of emojis with Bebas Neue, looks good with Arial.

I would like to eliminate as much difference between browsers as possible, but ultimately I'm not terribly concerned about it. I knew going into this that using different browsers would likely produce slightly different results. So ultimately users will just have to stick with whatever browser they designed in first to continue doing renders. As long as it stays consistent for them, that's all that really matters. That said, if there's something we can do to mitigate some of the browser differences we should do that.
Given that I'll answer your questions.

I think what I said above covers this.
This was always the plan. Their design view will generate an image and upload to the server which will then be processed into the printable PDFs.
Yeah, that's a no go. The whole point was go get away from server side rendering.
Ooh. That would be awesome if it's possible? If that's not possible, having a special tag that we could offer users to manually wrap them, or even if I just tell them to use Helvetica or whatever when wrapping emoji's would be fine. This is a good idea if that solves the problem.
I don't want to do that. They are forever coming out with new emoji's and that sounds like a maintenance nightmare.
Yeah, some fonts just have screwy parameters. I ran into this in my old version of CS too. Giving people tools to override problems when practical / possible was how I solved it. So it sounds like a reasonable way to go here too.

OK. So I'm hearing that we would do:

Not require / suggest a specific browser because...
For your app, the client will generate an image and that would be the one used for printing.
No server side rendering.
I'll try to wrap emojis in a tag making them a system font.
 But this is only a major need if you need cross-platform parity.
I'll create a new ticket for adding fontProperties: { ascent: number, descent: number, fontSize: number } to the stylesheet.

I've implemented the tag wrap thing in v2.2.4. Emojis are wrapped in a <__EMOJI__> tag and given a style {fontFamily: "sans-serif"}. However, it's not clear to me whether this actually fixes anything. The most notable thing it does is make the emojis move to the correct baseline while Bebas stays below it (on chrome / brave). https://jsfiddle.net/jr2mxvn8/
You can turn this feature on and off with wrapEmoji: boolean in the options object.

Just wanted to add that the JS Fiddle (or how it's loading the font and stuff) could be to blame for some of this?
Here's how Bebas Neue looks in JSFiddle in Brave on Mac. Notice the baseline position:

This is the same font, same browser in my demo code. Looks fine:

I won't be able to look at this until Thursday. Just wanted to give you the heads up on the timeline.

Checking in on the status of this one.
Main thing that concerns me is that the JSFiddle and the demo code I'm running locally look very different. https://github.com/mimshwright/pixi-tagged-text/issues/71#issuecomment-878681720
That could point to something to do with loading the fonts and creating the code for the PTT before the font is ready. Or possibly with how the code is all loaded through script tags or some other such thing.

Works perfectly in 2.2.7!


Is there anyway to get access token without firebase-admin

Hello, i want to get access token to call REST API get remote config. Is there anyway to get access token without firebase-admin because we can not use firebase-admin in react native :D

@huy-lv this is working as intended.  The Remote Config admin API is meant to be used from trusted (server) environments only.  It's not safe to put admin credentials in React Native.
If you want to use Remote Config client APIs in your RN app see these instructions:
https://rnfirebase.io/remote-config/usage

hey @samedson , i can not use that lib because i use Expo framework, @react-native-firebase/app does not include in expo. Please help me find a way to generate access token so i can get remote config through REST API instead of some lib else.


Improve core/domain/enums and tests

There was a panic if trying to convert to string some invalid values.
Added tests to AdminState and OperatingState
protocol.go is not used anywhere, should be removed
database.go is only used by metadata, should be moved to core/metadata/

recheck

recheck

@tsconn23
Should I remove protocol.go and move database.go to metadata?

@feclare Removing protocol.go and the related test file should be fine. Personally, I'd probably just leave database.go where it is for now.


Instant speedmine and grim option

we can use instant mine on cc, and blocks can be mined faster.
Also limit BlockDestructionService to a maximum of (n.o. players * 2) blocks. It also should remove positions if no player is in a 6 block radius of that block.

I will add instant mine and faster mining, but limiting blocks amount in BlockDestructionService can be easily overlooked and should not be implemented as it might cause issues.


Sync 2201.10.0 stage with 2201.10.x

Purpose

$title

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

2201.10.0-stage branch is up to date with 2201.10.x. Hence closing this.


Default to including all three communities

Originally created by Tom Smyth at 2015-08-13 14:07, migrated from redmine-#2842

Priority changed from 4 to 3

By Tom Smyth at 2015-08-19 07:19

Status changed from New to Closed

By Tom Smyth at 2015-08-23 07:44


How strong is your commitment to actually maintain apg

Hi,
I have been maintaining apg in Debian GNU/Linux for more than two decades, and of course I was also hit by upstream disappearing. There is a (small) number of repositories on github that have recovered apg, with yours being the most "active" one that has at least seen an occasional commit.
Would you be okay with me regarding your repository as "upstream" for the Debian packaging? This does not include any obligation to fix bugs or continue development, but of course I would appreciate if somebody would work with the actual code.
Feel free to get in touch with me, the package mail address is<EMAIL_ADDRESS>- thanks!
Best regards,
Marc Haber

Sadly, there was no response up to today.

In Mageia we already use this repo as the upstream for the package.


Markdownlint file relative

Closes: #2273
What's Changed

Adds a new markdownlint custom rule to ensure relative links are using the file-relative instead of directory-relative path.

This obsoletes #2274, correct?

This obsoletes #2274, correct?

It is based on the work done in #2274, so if you want to combine, that would be fine.


Error when running export_svg.py

I get the following error in my Binary Ninja log when running the export_svg.py plugin provided in the python/examples folder:
Traceback (most recent call last):
  File "C:\Software\Binary Ninja\plugins\..\python\binaryninja\plugin.py", line 172, in _function_action
    action(view_obj, func_obj)
  File "C:\Users\Me\AppData\Roaming\Binary Ninja\plugins\export_svg.py", line 46, in save_svg
    outputfile = outputfile.replace('/', '\\')
TypeError: a bytes-like object is required, not 'str'

The error occurs right after I click the save button in the Windows Explorer window that pops up when I run the plugin.
Running Windows 10 x64, using Python 3.

Should be resolved whenever the next stable goes out or you can download it manually now: https://github.com/Vector35/binaryninja-api/blob/dev/python/examples/export_svg.py
As an added bonus, it now supports IL forms of graphs. :-)
Note, that it is also ONLY supports Python3 for now. Slowly starting to remove Py2 compatibility where it hurts testing as was the case with some of the edge-cases around file system operation on windows in particular.


App crash on mobile-messaging initialization

After creating the widget on the dashboard and following the infobip-mobile-messaging-react-native-plugin installation guide, the Android app fails to build with this error
Execution failed for task ':app:mergeDebugResources'.
> [string/google_app_id] /android/app/src/main/res/values/strings.xml	[string/google_app_id] /android/app/build/generated/res/google-services/debug/values/values.xml: Error: Duplicate resources

If I should remove this line of code below in /android/app/src/main/res/values/strings.xml  the app builds successfully.
<string name="google_app_id">your google app id</string>
We have Firebase set up in the app with the google-services.json file.
When I get to build the app successfully for development, after initialization configuration,  the app crashes,

Hi @kostap13 @tjuric,
What we did was to add key/values from google-services.json to resource strings (strings.xml) and removed this line apply plugin: 'com.google.gms.google-services' from the build.gradle file.
It works!
Thanks

Hi @vokecodes if I remove apply plugin: 'com.google.gms.google-services' then I am getting a different error. Can you please elaborate on the process of setting up infobip in android?

@vokecodes Thank you for your response. Can you let me know the keys which will be used from google-services.json in these strings element in the place of *** as  I am not aware of all the keys in the above answer?

Hi @ahadtechcarrot  you can find this information here - https://developers.google.com/android/guides/google-services-plugin

Thank you @riskpp but I am facing errors in both the scenarios now :
Scenario 1 : removing apply plugin: 'com.google.gms.google-services' and adding all the values in strings.xml I am getting the error message :
> Task :app:fabricGenerateResourcesStagingGoogleDebug FAILED
ERROR - Crashlytics Developer Tools error.
java.lang.IllegalArgumentException: Crashlytics found an invalid API key: null.

Scenarios 2 : Using apply plugin: 'com.google.gms.google-services'
Caused by: java.lang.ClassNotFoundException: Didn't find class "com.google.firebase.iid.FirebaseInstanceId" on path: DexPathList[[zip file "/data/app/~~HqJrbxlqvpKu-fCiGQ_E2Q==/com.reelCinema.Screens.stage-FIAC9pxSrE844PRXDZOPtg==/base.apk"],nativeLibraryDirectories=[/data/app/~~HqJrbxlqvpKu-fCiGQ_E2Q==/com.reelCinema.Screens.stage-FIAC9pxSrE844PRXDZOPtg==/lib/x86, /data/app/~~HqJrbxlqvpKu-fCiGQ_E2Q==/com.reelCinema.Screens.stage-FIAC9pxSrE844PRXDZOPtg==/base.apk!/lib/x86, /system/lib, /system_ext/lib]]
@riskpp @vokecodes can provide any help regarding this?

@ahadtechcarrot could you pls check compileSdkVersion, targetSdkVersion on your build.gradle, and which version of com.google.firebase:firebase-messaging are you using?
I could only guess that you probably have compileSdkVersion or targetSdkVersion = 31 or using newer version of com.google.firebase:firebase-messaging, installed by some other plugin.
If it's the case: we've updated Android SDK to support Android 12, but for plugins this task isn't yet done.

@riskpp I am using react-native third party plugins for firebase :
    "@react-native-firebase/analytics": "^12.0.0",
    "@react-native-firebase/app": "^12.0.0",
    "@react-native-firebase/auth": "^12.0.0",
    "@react-native-firebase/crashlytics": "^12.0.0",

I have no setup done for firebase-messaging as there is no mention of that in your documentation and the above firebase libraries we are using already before this infobip setup. Also, please find my configurations in build.gradle file :
        buildToolsVersion = "29.0.3"
        minSdkVersion = 21
        compileSdkVersion = 30
        targetSdkVersion = 30
        supportLibVersion = "28.0.0"
        playServicesVersion = "15.0.1"
        androidMapsUtilsVersion = "0.5+"
        googlePlayServicesAuthVersion = "16.0.1"

@riskpp please let me know if I am missing out on something for Android for Version 5.0.1. As per the documentation in GitHub, I have only written the key/value pairs in the strings.xml file and updated the .bash_rc file.

Just to have a follow-up here as well,
@ahadtechcarrot your issue has been moved to a new thread https://github.com/infobip/mobile-messaging-react-native-plugin/issues/39 and solved.


Unable to records bytes produced to topic errors logged in repository meta analyzer

Whenever repository-meta-analyzer produces a result, Kafka Streams logs an error like the following:
2023-01-30 12:04:38,195 ERROR [org.apa.kaf.str.pro.int.RecordCollectorImpl] (kafka-producer-network-thread | hyades-repository-meta-analyzer-7308f986-fa3c-43a0-a365-5abc59f77ea9-StreamThread-3-producer) stream-thread [hyades-repository-meta-analyzer-7308f986-fa3c-43a0-a365-5abc59f77ea9-StreamThread-3] task [0_2] Unable to records bytes produced to topic dtrack.repo-meta-analysis.result by sink node produce_npm_result_to_dtrack.repo-meta-analysis.result_topic as the node is not recognized.
Known sink nodes are [produce_empty_result_to_dtrack.repo-meta-analysis.result_topic].

Records are produced as expected, so this does not appear to be a show stopper. We should investigate on this anyway and try to resolve it.

This may have been fixed automatically by updating kafka-streams to 3.3.2 (included in Quarkus 2.16.1.Final): https://github.com/apache/kafka/pull/12836

Resolved by upgrade to Quarkus >=2.16.1.Final.


Ignore: Dummy PR to play with github actions.

Description
Ignore this PR. It's just a draft and only exists to play with some github actions.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Yup. It passed. Good to know.


dataset

As mentioned in the paper: "MediaPipe Hands: On-device Real-time Hand Tracking", what is the commercial 3D
hand model tool did you use to generate your synthetic data?

I refer to the hand media pipe. Please refer to this paper: https://arxiv.org/abs/2006.10214. In the data generation step, the Google team used a synthetic dataset generated using the commercial 3D hand model tool. I wanted to know what tool/software did the team use to generate a synthetic hand dataset? I am planning to make the same synthetic dataset for my ML project. thx.

Hi @pythonsql21, As mentioned the output results with 21 3D landmarks you can generate the same with any platform your are familiar with by using  world landmarks.  You can try building one such with Three.js – JavaScript 3D Library if you wanted to target it on JavaScript. Same if you are planning to build on Python you can refer mplot3d library. This https://github.com/google/mediapipe/issues/851#issuecomment-651990259 gives you more information about the synthetic images used.

@sgowroji Thank you for your prompt response.


#2907 dataset swapping behavior change

Description
Currently, we could change the upstream I.DS or a W.DS to another I.DS in the same dataflow.
Now, we can bring any datasets in the system.
The original upstream stays in the dataflow.
Related Issue : #2907
How Has This Been Tested?
Ran locally.
Ran all test cases in dataprep.
Need additional checks?
Yes, please.
Types of changes

[x] 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 change)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have added tests to cover my changes.

deploy to 4


license site down

The server is temporarily unable to service your request due to maintenance downtime or capacity problems. Please try again later.

Try again @skyzhp https://licensing.meteorite.bi/login


Multi-GPU direct mode support broken during refactoring

Commit in question is here: https://github.com/sensics/OSVR-RenderManager/commit/20e8e2445fa81faca5fa4dcede69cbe7434f8ebc
The underlying concern is that there is a bunch of processing that needs to happen (in particular, enumeration of which devices support direct mode) in subclasses that support direct mode, before we can reliably have a IDXGIAdapter in the code around https://github.com/sensics/OSVR-RenderManager/commit/20e8e2445fa81faca5fa4dcede69cbe7434f8ebc#diff-c1563522d9fcfcc122b7be39a675d87eR166 - because on a multigpu system, the one that Windows hands you by default might not, and probably isn't, the right one for direct mode. Thus, direct mode subclasses, who can actually find the right adapter to correspond to the display, call setAdapter() before calling this base class openDisplay().
I agree that the class design here is a bit funky and could use refactoring, (and I am a bit surprised there wasn't a warning that you had a "conditional expression is constant") but that particular move was a regression.

This should be fixed in the fixing-d3d-issues branch, which I plan to use to fix other issues on the D3D path and the do a pull request for.

I think this got fixed in that pull request.  Re-open if not.


Switch to async mutex

This PR adapts usage of async locks by switching from parking_lot mutex to tokio::sync::async mutex.

General thought: How do the sync and the async benchmarks compare? Is the level of performance identical?

General thought: Now that we have both a sync and an async version of the same benchmark, how do they compare? Is the level of performance identical?

@emmanuel-keller Here is the difference between existing main branch vs async (without using async locks)
(async)
insert key/value lengths 10/10
                        time:   [15.230 µs 15.474 µs 15.729 µs]
sequential inserts      time:   [180.34 ns 180.76 ns 181.21 ns]
sequential gets         time:   [2.8562 ns 2.8595 ns 2.8629 ns]

(sync)
insert key/value lengths 10/10
                        time:   [5.6125 µs 5.8483 µs 6.0650 µs]
sequential inserts      time:   [49.604 µs 50.466 µs 51.194 µs]
sequential gets         time:   [504.90 ps 518.60 ps 536.32 ps]

Don't see any improvements as such, except for the sequential insert. Need to check if it is an implementation issue or not.

General thought: Now that we have both a sync and an async version of the same benchmark, how do they compare? Is the level of performance identical?

@emmanuel-keller Here is the difference between existing main branch vs async (without using async locks)
(async)
insert key/value lengths 10/10
                        time:   [15.230 µs 15.474 µs 15.729 µs]
sequential inserts      time:   [180.34 ns 180.76 ns 181.21 ns]
sequential gets         time:   [2.8562 ns 2.8595 ns 2.8629 ns]

(sync)
insert key/value lengths 10/10
                        time:   [5.6125 µs 5.8483 µs 6.0650 µs]
sequential inserts      time:   [49.604 µs 50.466 µs 51.194 µs]
sequential gets         time:   [504.90 ps 518.60 ps 536.32 ps]

Don't see any massive improvements as such, except for the sequential insert. Need to check if it is an implementation issue or not.


Still not a 100% sure on how to download this...

Im not a tech guy and im still having trouble figuring out how to download and get this running. Could you make a video or something to make it more clear on what I have to do. Thanks.

No problem!
So you have two options...the simplest is to go to the latest version on the Releases page, here:
https://github.com/majikat768/HuntStatsLogger/releases/tag/v0.4.2
and download either of the HuntStatsLogger.rar files.  You'll need something like winrar ( https://www.win-rar.com/start.html ) to open it.  Extract HuntStatsLogger.exe from the .rar file to your desktop, then you can just open it by double clicking HuntStatsLogger.exe.
However, sometimes Windows incorrectly thinks my program is a virus, and won't let you open it.  I believe it's possible to add an exception in your firewall or Windows Defender, I'd recommend looking up on google how to do this.
Your other option is just running it as a python script... to do this you'll need to download and install python:  https://www.python.org/downloads/
When installing, there'll be an option saying something like, "add Python to system PATH".  make sure that's checked.
Then you'll download my entire code repository; from the main page ( https://github.com/majikat768/HuntStatsLogger ) click on the box on the top right that says 'Code', and click Download Zip.  Extract the zip file to some folder, then in the folder, you'll need to run the start.ps1 file...sometimes, just double clicking this will work.  But you may need to right click it, and tell it to run with Windows Powershell.
Let me know if you need more assistance.

Ok, I tried to open it with WINrar but It only worked once. Then I tried the python download and I went through the steps and then it just kept crashing or not allowing me to use the file but every time I opened the start.ps1 file it opens the powershell terminal the disappears immediately.

Interesting- so when you open it with WinRar, do you click on "Extract to..." and choose to put the .exe on the Desktop?  After that, you'll start it by opening the HuntStatsLogger.exe file.
Also, I just figured out the issue with python!  I'll push a change asap, but basically, if you open start.ps1 in notepad, remove the # from the first line, so it just says "pip install -r requirements.txt" and try again.
If there are still issues, try right clicking in the folder that start.ps1 is in, and choose Powershell > open Powershell Here.  Then type start.ps1 into the console window and click enter.  If it doesn't work, an error message should display, that should be helpful.

Hmm, you're sure you downloaded and extracted the entire code directory?  It should contain all of the files listed in the repository:
https://github.com/majikat768/HuntStatsLogger
, including requirements.txt.

Directory should look like this when you execute start.ps1:

How do I change the resolution (I don't have a second monitor)

ahh, that may be an issue, you're the second person who's mentioned the window being too large.  Unfortunately it's not resizable currently, but I'll work on getting that fixed asap!

Just for reference, what is the resolution of your screen currently?

1920x1080

I've pushed a quick fix, so that the window can be resized, and the main content area is scrollable.

So do I just reinstall the software?

Also mmr not showing up properly

In settings you'll need to set your steam username before your MMR will show up.
After you set your username in settings, exit the program then restart, then it should work.


vllm multi gpu support

Hi team, I'm reading https://github.com/triton-inference-server/tutorials/tree/main/Quick_Deploy/vLLM and it mentions that it currently does not support multi gpu. Is there any reason for this?

Our vLLM example uses the Python backend, which is not set up to support multi-GPU inference. The only backend that currently supports multi-GPU inference is the FasterTransformer backend. We are working on increasing support for multi-GPU LLM inference.


Warning message for Build Cache during Settings script compilation should be removed

Issue type
Polishing
Problem description
When running a basic test with the Build Cache enabled, the following warning occurs:
Caching disabled for Kotlin DSL script compilation (Settings/TopLevel/stage2) because:
      Build cache is disabled

Because:

This warning should be removed as it is confusing for users.
Context (optional)
https://gradle-community.slack.com/archives/CAHSN3LDN/p1729000371866449

The issue is in the backlog of the relevant team and is prioritized by them.


You can't add a user if email login is set in security

Describe the bug
You can't add a user if email login is set in security settings
To Reproduce
Steps to reproduce the behavior:

Set email login in control panel > security
Add user
if put the email in the user name field you will get an error saying it needs to be left blank. If you leave it blank you get an error that it's required.

Expected behavior
No login name field is displayed  when adding a user and a user can be added
Screenshots
If applicable, add screenshots to help explain your problem.
Software (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Volto Version 16
Plone Version 6
Plone REST API Version [e.g. 7.0.1]

Additional context
Add any other context about the problem here.

might get fixed as part of #5952 ?

A quick fix might be to remove the clientside validation on the username field if there is currently no easy way to know the security settings when that form is rendered?

The form already fetches the usergroup control panel settings. I haven't looked at the details but I would guess it's not hard to make it also fetch the security settings.

if you leave it blank you get an error that it's required.

This shouldn't happen see: https://github.com/plone/plone.restapi/blob/5cf1418e928f143ef1b42fdc3e4ed72a3126e9dd/src/plone/restapi/services/users/add.py#L69-L71
What version of plone.restapi are you using?

It appears that this is mostly a Volto 16 issue, as in Volto 17 we use the userschema to display the fields where the username field isn't required so can be skipped over. I've included screenshots below showing the difference in an out-the-box Volto experience
Add user form in Volto 16

Add user form in Volto 17

However, it still isn't ideal that this field is display. I did look into adding the use_email_as_login option to the userschema, but I don't this makes sense as the userschema can be used elsewhere and the option isn't really linked to users. I think it would make sense to include the use_email_as_login as part of the @site endpoint as it's more of a global setting which could make sense to be used elsewhere (for example, in the Login form as mentioned in the issue description). This would also be a suitable place for the other public security settings like enable_self_reg.
@djay @davisagli Any thoughts on this?


Provide better guidance to users when an analysis is shared to avoid unintentional changes being made

Users need clear guidance about the consequences of editing an analysis after it has been shared.
We can only assume that if the share link has been copied, then it has been shared. Obviously we can't spy on the user to see if they have sent it to someone. The shared-ness needs to be preserved between visits to the analysis.
There are two parts to this:
Back end
TO DO: make a service-eda ticket
Add a new isShared property as a sibling to isPublic and displayName and apiVersion etc (e.g. here?)
This should return a default value of false if there is no data actually stored in the database. But see next paragraph...
TO DO: Does this require a new database column? I think it will. What's the process for patching databases? Would this patching fill with the default value false? Where does this ticket go?
What's the order of roll out? 1. database patching, 2. service, 3. front end?  Would appreciate @ryanrdoherty's input here.
Front end
Preliminaries
The new analysis property will go here
https://github.com/VEuPathDB/web-monorepo/blob/b2458f0600b2ba6d2e77743f229f0cd24aa8966c/packages/libs/eda/src/lib/core/types/analysis.ts#L62
Based on the discussion above about the back end, this property can probably be mandatory and old studies will contain false values even if they have been shared. We will have to be careful to roll out the back end first.
The useAnalysis hook will need to return a setIsShared setter.
Upon sharing

In the share dialogues for both EDA and Map, we need to do analysis.setIsShared(true) when the copy button is pressed or if the text is selected
Add the bold danger/safe zone messages to the share dialogue. See the mockup here https://www.figma.com/design/zUxPFmqMhOAyDbhZITj6s9/Share-mechanism-overhaul?node-id=2009%3A44&t=UwHTqvqW5ng3ixU2-1
Also remove or disable the (x) from the share dialogue (EDA only)
The red/orange danger button will close the dialogue and continue in the same analysis
The green button will create a copy and switch to the copy (which will be new and unshared) - make sure the copy is isShared=false!!
Colours should come from our regular palette: coreui/src/definitions/colors.ts - warning and success

Indicating isShared state
Two options to run by UX ASAP:

Shared badge in the map header and in the analysis header (same row as the Share button)
:red_circle:

EDA: Share button gets a red dot (tooltip explains that the analysis has been shared, verbiage to be refined later)
Map: Share item in left hand menu gets a red dot and same tooltip

Note that we may move some Map functions into the header, and this will likely include the Share button, so the red dot would be more visible there.
Upon opening an isShared=true analysis
Map and EDA
Show a modal that contains only the red and green warnings with the same functionality as above (red=stay, green=copy)

Two followups:

are there any other analysis schema changes we want to roll out at the same time
do we want a "never show this again" (for this analysis) checkbox? This would require another analysis property "userHasDismissedShareWarning". Probably overkill?

In the share dialogues for both EDA and Map, we need to do analysis.setIsShared(true) when the copy button is pressed or if the text is selected

We should set it if the user copies the selected text (see copy event). In the copy event handler, we can use getSelection to determine if anchorNode is the node containing the text of the share url.

Show a modal that contains only the red and green warnings with the same functionality as above (red=stay, green=copy)

Can we reuse the share dialog, but alter the title of the dialog to something like "Share details". This can be based on isShared, or maybe we can find something that works in both cases.

We've had some discussion on Slack about having a button to reveal and copy the share link. The link would be revealed without pressing the button if the analysis has already been shared. (The button basically sets isShared to true and the link visibility depends on isShared.)
I think this is probably the best solution. Does anyone have any other suggestions?

I think probably option 1 (share "badge") is better at this point. Even though I don't personally like the proliferation of big buttons/badges, it's better for the map for the badge to be in the header.


measure image if we don't have node attributes to use

not sure this is the ideal way to do this, but was playing around with another skin and it was trying to load images from nodes that didn't define w/h/x/y

LGTM!


Error channel naming enhancement

Please add support for something like a binding name alias so that different destinations can be used but they can always be set to the same error channel bean name.
This enhancement request is being requested after the suggestion posted in this issue. There it was suggested that the enhancement requested originally opened against Spring Integration would be better suited with Spring Cloud Stream.

I think the problem is in the AbstractMessageChannelBinder where we register an error channel bean:
	String errorChannelName = errorsBaseName(destination, group, consumerProperties);
...
	protected String errorsBaseName(ConsumerDestination destination, String group,
			C consumerProperties) {
		return destination.getName() + "." + group + ".errors";
	}

So, the name of the bean is really based on a destination.
However I feel like it has to be based on the binding name since already this one is closer to the application logic rather than environment this application is deployed in.
In that registerErrorInfrastructure() method we an access to this consumerProperties.getBindingName().
From here I wonder why the choice was really about a destination name, but not the binding we do in the application...

I believe this issue has been addressed with the upcoming functional error handling. It's already available in SNAPSHOTS for both 4.x and 3.2.x. Also, 3.2.5 is coming out very shortly (few days), so it will be available to you
Here is the link to the updated ref doc with more details - https://docs.spring.io/spring-cloud-stream/docs/3.2.5-SNAPSHOT/reference/html/spring-cloud-stream.html#_handle_error_messages
Please let us know if that is what you are looking for

I am going to close it as I believe the issue has been addressed with functional error handling approach discussed earlier.
Feel free to reopen or raise a new one if the issue persists.


Upgrade Saxon from version 9.7 to version 9.9

Certain aspects of the XSLT 3.0 spec, like xsl:map are currently not supported with the version of Saxon (9.7, release in 2017).
These features would be very handy in XSL filters and could improve maintainability and performance.
Recommend updating the Saxon library to >= 9.8 in order to enable support for xsl:map and other features.

This change is known to cause issues with certain translations.  Retesting is strongly recommended!
Comments attached to this issue refer to particular aspects that have been observed to be especially problematic.

Behaviour of current-group() has changed.  In addition to the documented fact that it is now no longer used within xsl:merge operations, it appears to have changed in other more subtle respects too.


Could not get instance for TraceUploaderProxy when using StructureMap as DI container

I followed the steps as described in the ReadMe. But it seems that when trying to instantiate classes from Application insights profiler (using services.AddServiceProfiler()), StructureMap throws an exception when trying to instantiate TraceUploaderProxy.
Bellow the exception I get:
Unable to create a build plan for concrete type Func<String, String, Task<Int32>> new Func<String, String, Task<Int32>>(Object, IntPtr method) Object = **Default**  IntPtr method = Required primitive dependency is not explicitly defined 1.) Attempting to create a BuildPlan for Instance of Func<String, String, Task<Int32>> -- Func<String, String, Task<Int32>> 2.) new TraceUploaderProxy(*Default of IUploaderLocator*, *Default of IProfilerFrontendClient*, *Default of IFileWrapper*, *Default of IOutOfProcCaller*, *Default of IServiceProfilerContext*, *Default of ILogger<TraceUploaderProxy>*, *Default of IOptions<UserConfiguration>*, *Default of Func<String, String, Task<Int32>>*, *Default of Func<String>*) 3.) ServiceProfiler.EventPipe.Client.TraceUploader.TraceUploaderProxy 4.) Instance of ServiceProfiler.EventPipe.Client.TraceUploader.ITraceUploader (ServiceProfiler.EventPipe.Client.TraceUploader.TraceUploaderProxy) 5.) new ServiceProfilerProvider(*Default of TraceConfiguration*, *Default of IServiceProfilerContext*, *Default of ITraceControl*, *Default of ITraceSessionListenerFactory*, *Default of ITelemetryTrackerFactory*, *Default of ICustomEventsTracker*, *Default of ITraceUploader*, *Default of ITraceValidatorFactory*, *Default of IMetadataWriter*, *Default of IOptions<UserConfiguration>*, *Default of IAppInsightsSinks*, *Default of ILogger<ServiceProfilerProvider>*) 6.) ServiceProfiler.EventPipe.Client.ServiceProfilerProvider 7.) Instance of ServiceProfiler.EventPipe.Client.IServiceProfilerProvider (ServiceProfiler.EventPipe.Client.ServiceProfilerProvider) 8.) new OrchestratorEventPipe(*Default of IServiceProfilerProvider*, *Default of IOptions<UserConfiguration>*, *Default of IEnumerable<SchedulingPolicy>*, *Default of IDelaySource*, *Default of ILogger<OrchestratorEventPipe>*) 9.) Microsoft.ServiceProfiler.EventPipe.Orchestration.OrchestratorEventPipe 10.) Instance of Microsoft.ServiceProfiler.Orchestration.IOrchestrator (Microsoft.ServiceProfiler.EventPipe.Orchestration.OrchestratorEventPipe) 11.) new ServiceProfilerStartupFilter(*Default of IOrchestrator*, *Default of IOptions<UserConfiguration>*, *Default of ICompatibilityUtility*, *Default of ILogger<ServiceProfilerStartupFilter>*) 12.) ServiceProfiler.EventPipe.AspNetCore.ServiceProfilerStartupFilter 13.) Instance of Microsoft.AspNetCore.Hosting.IStartupFilter (ServiceProfiler.EventPipe.AspNetCore.ServiceProfilerStartupFilter) 14.) All registered children for IEnumerable<IStartupFilter> 15.) Instance of IEnumerable<IStartupFilter> 16.) new GenericWebHostService(*Default of IOptions<GenericWebHostServiceOptions>*, *Default of IServer*, *Default of ILoggerFactory*, *Default of DiagnosticListener*, *Default of IHttpContextFactory*, *Default of IApplicationBuilderFactory*, *Default of IEnumerable<IStartupFilter>*, *Default of IConfiguration*, *Default of IWebHostEnvironment*) 17.) Microsoft.AspNetCore.Hosting.GenericWebHostService 18.) Instance of Microsoft.Extensions.Hosting.IHostedService (Microsoft.AspNetCore.Hosting.GenericWebHostService) 19.) All registered children for IEnumerable<IHostedService> 20.) Instance of IEnumerable<IHostedService> 21.) Container.GetInstance(IEnumerable<IHostedService>)
I wanted to force StructureMap to use the default constructor if it exists, but unfortunately I can't, source code isn't available in the repo.

@cothman Sorry to hear that it is not working for you. Could you please provide some information for trouble shooting:

What's your dotnet version? (Run dotnet --info at the project folder)
What is the package version that you referenced to?
What's your OS?

Is it possible for you to post a copy of your project here? I can take a look then.

@xiaomi7732, this is my dotnet version:
.NET Core SDK (reflecting any global.json):
Version:   3.0.100
Commit:    04339c3a26
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.18362
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.0.100\
Host (useful for support):
Version: 3.0.0
Commit:  7d57652f33
.NET Core SDKs installed:
2.1.801 [C:\Program Files\dotnet\sdk]
2.2.401 [C:\Program Files\dotnet\sdk]
3.0.100 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Referencing this packages:
 <PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.8.2" /><PackageReference Include="Microsoft.ApplicationInsights.Profiler.AspNetCore" Version="2.0.0-beta4" /> <PackageReference Include="StructureMap.Microsoft.DependencyInjection" Version="2.0.0" />
And in this repo I reproduce the exception https://github.com/cothman/AspNetStructureMap
Thank you for the help

@cothman I did some investigation and here's some of my findings:

You are trying to use StructureMap to replace the existing DI for the app. However, StructureMap is not working with ASP.NET Core 3.0 yet: Details here: active issue #47.
ConfigureContainer will be called only when StructureMap.AspNetCore is referenced due to it's documenation here: StructureMap GitHub repo.
That leas to the question: Is your intention to use StructureMap.AspNetCore or StructureMap.Microsoft.DependencyInjection ?
StructureMap seems try to injecting services from all referenced assemblies. Since AddServiceProfiler is called already, you shall be able to exclude it from the registry:    public void ConfigureContainer(Registry registry)
    {
        registry.Scan(x =>
        {
            x.Exclude(type => type.Name.Contains("ServiceProfiler"));
            x.TheCallingAssembly();
            x.WithDefaultConventions();
            x.LookForRegistries();
        });
    }

StructureMap, if works, doesn't support to supply null to optional parameters on constructor that is a Fun<string, string, Task<int>>, that should be addressed if the filter in the last point is addressed.

On the side, it doesn't seem like StrcutureMap is sunsetted. Quoting from its official repro:

As StructureMap has been sunsetted, it's recommended to move to Lamar, StructureMap's successor, which is more compatible with ASP.NET Core's DI system.

Do you have a specific scenario that structure map is required?

@xiaomi7732 Thanks for the reply.
Actually our API uses StructureMap as DI. We are working on a migration to Lamar (it will take some time to implement).
I tried the exculde way in ConfigureContainer, it doesn't seem to be working.

@cothman Thanks for the update. Yes, exclude anything in ConfigureContainer will NOT work because it only works with StructureMap.AspNetCore. And in your project, StructureMap.Microsoft.DependencyInjection is used than StructureMap.AspNetCore. At this point, it doesn't matter what you do in ConfigureContainer - you can totally remove it and it still won't work.
It seems that you are already start to migrating to Lamar, I would suggest invest time on the new DI and try to make it work there.

Close this as there's nothing actionable from profile perspective. Feel free to open new issues.


mfa align

Excuse me!
What's the format of input parameter "input dataset"?
it doesn't work if using input as follows

input text should be split by space


Nucleotide Count: Remove unnessary class

The Nucleotide Count stub and exemplar solution currently contain a class definition that serves no purpose.
https://github.com/exercism/javascript/blob/0017b79c40521091794437fa0d0d9974de35c4d5/exercises/practice/nucleotide-count/nucleotide-count.js#L6-L7
The exercise should be changed so that the stub and exemplar only use a simple function definition, e.g.
export function nucleotideCounts(strand) {
   // ...
}

The following files need to be updated:

nucleotide-count.js
nucleotide-count.spec.js
.meta/proof.ci.js
root level config.json (remove "classes" from the prerequisites of the exercise)

You can find general information about the structure of practice exercises here: https://exercism.org/docs/building/tracks/practice-exercises

Hi! Can I work on this issue?

Sure, I'll assign you. Thanks Form helping out.