Update dependencies

Hello I tried installing this library but since it is dependant on old version of React (currently 17), I cannot install it into my React 18 project. Do you think it would be possible to update?

Hi, I'm also experiencing the same issue, have you been able to install it in react 18?

No I didn't manage to find a solution so I implemented it using standard KeyboarAvoidingVIew. Which of course has it's quirks. I'm planning to rewrite it using useAnimatedKeyboard from reanimated, so maybe you can try and use that too :)

omg, i never know that reanimated has useAnimatedKeyboard hook. Thanks for the info, I'll try it out

@iway1 any plans to update this?


Upgrade expfmt vendoring

@beorn7 @juliusv

:+1: otherwise (whether you keep the newline or not for now)

Filed kardianos/govendor#50


Add Darwin devdisk commands and put them in unixtools

Motivation for this change
Some new packages, cleanups, etc.

I think this (specifically 8287ba10fc7df5ae1cfcdd9057eb5e31f15a0824) broke something. unixtools.utillinux won't build on linux due to missing quota: https://hydra.nixos.org/build/72827257

I'm guessing the problem is the quota binary living in pkgs.quota not pkgs.utillinux. This makes it build for me on linux at least:
modified   pkgs/top-level/unix-tools.nix
@@ -92,7 +92,7 @@ in rec {
     darwin = pkgs.darwin.ps;
   };
   quota = singleBinary "quota" {
-    linux = pkgs.utillinux;
+    linux = pkgs.quota;
     darwin = pkgs.darwin.diskdev_cmds;
   };
   route = singleBinary "route" {


NonMaximum supression boxes

Please add NMSBoxes method to the DNN module.

+1

Works like a charm!! Thank you!!


Feature/inline external to bootstrap

Changed Inline and external Css to Bootstrap whichever was possible.
Fixes #122  (issue)
UI Enhancement
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules

Reviewer: Vinit Shahdeo

@vinitshahdeo  , Kindly review this PR. Thank you

@ashishnagpal2498 please squash your commits. There are so many redundant commits.

Done @jainpawan21 .

@ashishnagpal2498 Please make this PR to frontend branch

Hey @jainpawan21 , I have changed the base of PR as well as resolved the merge conflicts. Kindly review it.

Hi @vinitshahdeo , removed the deprecated and some unwanted tags. Also I have corrected the flexible movement of tanks, I would appreciate if you could upgrade the PR difficulty. Thanks

@vinitshahdeo , this one.


Add Unity simulator and interactive graph for Rocket Injector notebook.

Continuing #165
Simulator repository: https://github.com/FieryBlade-313/BusterinGaaz

Thanks again @FieryBlade-313!

My writing's a little off, if guys could suggest some changes (or other suggestions) @zoq @say4n @coatless

My writing's a little off, if guys could suggest some changes (or other suggestions) @zoq @say4n @coatless

Ryan left some comments on your blog post, I guess some things we should incoperate here as well. Other than that I think this is good.

View / edit / reply to this conversation on ReviewNB
say4n commented on 2021-07-28T17:20:44Z
Looks good to me!
I was wondering if we could have a mesh plot in addition to the scatter plot for the Pareto front (maybe in the same plot as above).
Not necessary at all, just a suggestion. :)

Easy peasy.

My writing's a little off, if guys could suggest some changes (or other suggestions) @zoq @say4n @coatless

Ryan left some comments on your blog post, I guess some things we should incoperate here as well. Other than that I think this is good.

Addressed.

@zoq  Lemme know if you have any further comments. I think this is good to go.

Looking at the comments, pretty clear that everyone likes the notebook, great job!


How to store mutations

We also need to be able to add mutations to a graph such as that in #2. In tskit we add mutations above nodes, and the "site" at which a mutation occurs contains the genomic position. With a GIG, the site position must be in the local coordinate system, so we can't have a global "sites" table. My suggestion is that we encapsulate the site position into the mutations table.
If we don't have a sites table, we also need to think about what this means for the ancestral state. FWIW I don't think we should define an ancestral state, but instead place mutations above the MRCA nodes where necessary.
Finally, we need to decide on whether mutations are associated with a node or an edge. There was some discussion about this on tskit, before coming to the conclusion that the right decision had been made to associate a mutation with  node (https://github.com/tskit-dev/tskit/issues/668) mainly because edges are rather ephemeral objects that can be merged and split etc, and so tracking which edge a mutation is associated with is hard (and we can find it from the node with reasonable efficiency)

Also, I would be keen to avoid the tskit convention of associating an ancestral state with a specific site. Better would be a requirement to place a mutation above the root at each site to indicate the current ancestral state. If we add more distant samples, we can then decide whether that mutation should be shifted up.
We would need to work out how to represent a "reference sequence" without creating all the mutations at each position, though.

We also need to decide if alleles can only be a single letter, or whether we allow short indels. If we allow indels, it means that the actual sequence length will not be given by the max_child_position


Bedrock textures do not load with the mod

title.
vanilla Minecraft

Minecraft with the Changed Mod

a lot of people seem to have this issue, but i don't think it can be fixed

What is your mod list? Does this happen with just the mod?
@spacedan228

it had happened with other people, something to do with loading textures, as sometimes reloading textures helps, at least i heard so

origin of the issue is quite mysterious


.Net Core / .Net 6.0 Support

The subject says it all.

Hey @piotrzaremba, would you mind upvoting or commenting on #596? Appreciate the interest!


Generate PCA

Sorry Charlie, this was opened automatically by Asana and I can't close it now or it'll complete the Asana task too! Please Close when the Asana task is complete


Create pull request for assignment4 of sampling unit

@FarzaneHashemii
The links are provided and file is updated. Thanks.

发件人: Farzaneh Hashemi @.>
发送时间: 2024年11月23日 21:06
收件人: TinaToronto/sampling @.>
抄送: Tina Lin @.>; Author @.>
主题: Re: [TinaToronto/sampling] Create pull request for assignment4 of sampling unit (PR #4)
@FarzaneHashemii requested changes on this pull request.
Please provide links to sources used.
―
Reply to this email directly, view it on GitHubhttps://github.com/TinaToronto/sampling/pull/4#pullrequestreview-2456478778, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BIRJUZEVZPKPXGETJM5CY3D2CDU5DAVCNFSM6AAAAABR2CUKHOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDINJWGQ3TQNZXHA.
You are receiving this because you authored the thread.


Implement App Bar

Bumped to next sprint to align with the sprint planning that David and I did. @kaitmore, what is this blocked by?


server: support tls for the status server

What have you changed?
This PR is trying to support TLS by using rustls for the status server.
What is the type of the changes?

New feature

How is the PR tested?
For now, it is temporarily tested manually. I will add some tests to cover it.
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
Yes
Does this PR affect tidb-ansible?
Yes
Refer to a related PR or issue link (optional)
Close #5340.

Any update?

@rleungx Any progress on this?

/merge


Bug in get_rank method of Hyperedge View

What
There were a couple of bugs w.r.t get_rank method in Hyperedge View.
Details
The relevant code is present below
    def get_rank(self, e):
        """Get rank.

        Parameters
        ----------
        e : Iterable, Hashable or HyperEdge

        Returns
        -------
        int, the rank of the hyperedge e
        """
        if isinstance(e, Iterable):
            if len(e) == 0:
                return 0
            else:
                for i in list(self.allranks):
                    if frozenset(e) in self.hyperedge_dict[i]:
                        return i
                raise KeyError(f"hyperedge {e} is not in the complex")

        elif isinstance(e, HyperEdge):

            if len(e) == 0:
                return 0
            else:
                for i in list(self.allranks):

                    if frozenset(e.elements) in self.hyperedge_dict[i]:
                        return i
                raise KeyError(f"hyperedge {e} is not in the complex")

        elif isinstance(e, Hashable) and not isinstance(e, Iterable):

            if e in self.hyperedge_dict[0]:
                return 0
            else:
                raise KeyError(f"hyperedge {e} is not in the complex")

        elif isinstance(e, str):

            if e in self.hyperedge_dict[0]:
                return 0
            else:
                raise KeyError(f"hyperedge {e} is not in the complex")

Bugs

The hyperedge case in the above if-else block is never triggered because hyperedge itself is Iterable.
The str case in the above if-else block is never triggered because str itself is Iterable.
The cases where e is int or str never evaluates to True, because the int/str are not present in hyperedge_dict, but rather frozenset({e}) is

please open a pull request and fix the above errors


Multible questions

What is your question?

Who is Sylvest?
Why 0.5.2h instead of 0.6.3 or 0.7.3?

A friend of mine
I wanted to keep this mod minimal in terms of size, so I decided to use an older version of psych that didn’t have as much features as the later.


[alerting] Implement OpsGenie notifier

OpsGenie is also quite popular alerts/incident management system. It will be good to have OpsGenie notifier option.

#570 implements it.


Malformed gARP packet

Hello,
I would like to use this Go arping implementation in a containerized (k8s) environment for CNI purposes.
I started a Pod(1), which has eth1 interface for packet capturing.
I started another Pod(2), which has eth1 as follows:
105: eth1@if2: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 9000 qdisc noqueue state UNKNOWN group default
    link/ether fa:16:3e:41:8e:6b brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global eth1
       valid_lft forever preferred_lft forever
    inet6 fc00:caa5:1:a:f816:3eff:fe30:16bf/64 scope global
       valid_lft forever preferred_lft forever
    inet6 fe80::fa16:3e00:541:8e6b/64 scope link
       valid_lft forever preferred_lft forever

When Pod(2) was created, our custom CNI sent gARP packet using this Go implementation. That is the first packet in the capture below. Inside Pod(2) I executed "arping -c1 -A -I eth1 <IP_ADDRESS>" (arping binary from centos repo) as a reference, that is the last packet in the capture file.
The problem is that the gARP packet send by this Go implementation is malformed (as it is reported by Wireshark v3), so I'm not sure whether it is really working.
The packet capture is attached here:
garp.zip
Can you please check what's wrong?
Thanks!

I can't reproduce your problem on my side.
can you please test the arping executable from this repository on Pod(2)?
go get -u github.com/j-keck/arping/cmd/arping
sudo $GOPATH/bin/arping -v -U -i eth1 <IP_ADDRESS>

if you don't like to use sudo for a executable from the internet, you can set the needed raw socket permission at first per sudo setcap cap_net_raw+ep $GOPATH/bin/arping and then use it without sudo.
Thanks.

arping binary is also good:
garp2.zip
Here you can find the code which produces malformed gARP packet:
https://github.com/TothFerenc/danm/blob/garp/pkg/danmep/ep.go#L121
Do you have any idea how it can be fixed? Thanks!

Ok, i can reproduce a malformed arp packet when i use a v6 address.
I think this is a bug in this library - it should return a error, if someone try to use a v6 address.
But i need to read more about IPv6 / NDP at first (https://en.wikipedia.org/wiki/Neighbor_Discovery_Protocol).
Is it possible that you use a v6 address? Your test was with a v4 address.
Can you please log you ip to verify this?
Thanks

I figured out what was the problem: our ip string contains prefixed IPv4 address, so I have to use net.ParseCIDR() instead of net.ParseIP().
Anyway, arping.GratuitousArpOverIfaceByName() needs input validation. Better to report error instead of sending malformed ARP packets.
Thanks for your help.

Yes, i need better input validation for all api calls (verify it's a v4 address and the ip is not nil).
I didn't know that an net.IP can have "an illegal state":  https://play.golang.org/p/Rv2r13IyaVM
Thanks for your reponses


Search results don't include terms from a block's drop-down

Describe the bug
When searching, drop down values are not considered.
To Reproduce
Steps to reproduce the behavior:

Go to arcade.makecode.com/beta
Search for "stay in screen"
Notice there is no good match

Expected behavior
This match should appear:

Fixed


647 Adding all ansii character string types

Adding support for CHAR VARYING, CHARACTER VARYING, and CHARACTER data types, differentiating them from VARCHAR and CHAR, which were mistakenly considered the same (1).
The precision is optional to support PostgreSQL's syntax (2).
Also, adding tests to start resolving https://github.com/sqlparser-rs/sqlparser-rs/issues/2, but that's a WIP.
[1] : https://jakewheat.github.io/sql-overview/sql-2016-foundation-grammar.html#character-string-type
[2] : https://www.postgresql.org/docs/current/datatype-character.html
Resolves: https://github.com/sqlparser-rs/sqlparser-rs/issues/647

@alamb ready for review :D

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

26 of 28 (92.86%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 85.587%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/parser.rs
17
19
89.47%

Files with Coverage Reduction
New Missed Lines
%

src/ast/data_type.rs
1
91.57%

src/parser.rs
1
82.9%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
10202

Relevant Lines:
11920

💛  - Coveralls

Thank you @AugustoFKL

This PR appears to have a conflict -- can you please resolve that @AugustoFKL ?

@alamb done


Warning DataTransfer $this->parameter_bag is null

Stack trace
E_WARNING: array_key_exists() expects parameter 2 to be array, null given
in array_key_exists called at /bla/bla/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php (93)

Hi @vardumper
unfortunately we couldn't reproduce this behavior. Could you give us more insight into this issue? Are you using the current master? How can we reproduce this issue?

Hi guys. We are using the 1.4.0 version called wpackagist-plugin/bs-payone-woocommerce in our composer-based stack. (@see https://wpackagist.org/search?q=payone&type=any&search= with last commit date 2018-12-13 15:34:44).
Unfortunately I don't have that trace anymore, but I'll append the surrounding lines, hopefully that helps clarify what happened.
I did not manage to use your repo from GitHub since there's no composer.json in place. (I'll open a separate request for that).
We use Nginx and PHP7.2 fpm
HP message: PHP Notice:  post wurde fehlerhaft aufgerufen. Order properties should not be accessed directly. Backtrace: include('wp-admin/edit-form-advanced.php'), do_meta_boxes, WC_Meta_Box_Order_Data::output, do_action('woocommerce_admin_order_data_after_order_details'), WP_Hook->do_action, WP_Hook->apply_filters, wc_boniversum_api->show_order_meta, WC_Abstract_Legacy_Order->__get, wc_doing_it_wrong Weitere Informationen: Debugging in WordPress (engl.) (Diese Meldung wurde in Version 3.0 hinzugefügt.) in /var/www/whatever/releases/20190514121609/web/wp-includes/functions.php on line 4773
PHP message: PHP Notice:  post wurde fehlerhaft aufgerufen. Order properties should not be accessed directly. Backtrace: include('wp-admin/edit-form-advanced.php'), do_meta_boxes, WC_Meta_Box_Order_Data::output, do_action('woocommerce_admin_order_data_after_order_details'), WP_Hook->do_action, WP_Hook->apply_filters, wc_boniversum_api->show_order_meta, WC_Abstract_Legacy_Order->__get, wc_doing_it_wrong Weitere Informationen: Debugging in WordPress (engl.) (Diese Meldung wurde in Version 3.0 hinzugefügt.) in /var/www/whatever/releases/20190514121609/web/wp-includes/functions.php on line 4773" while reading response header from upstream, client: <IP_ADDRESS>, server: whatever.com, request: "GET /wp-admin/post.php?post=2669&action=edit HTTP/2.0", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "whatever.com"
2019/05/14 12:46:59 [error] 4395#4395: *180 FastCGI sent in stderr: "PHP message: testlog" while reading response header from upstream, client: <IP_ADDRESS>, server: whatever.com, request: "GET /wp-admin/admin.php?page=payone-settings-account HTTP/2.0", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "whatever.com"
2019/05/14 12:46:59 [error] 4395#4395: *180 FastCGI sent in stderr: "PHP message: testlog" while reading response header from upstream, client: <IP_ADDRESS>, server: whatever.com, request: "POST /wp-admin/admin-ajax.php HTTP/2.0", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "whatever.com"
2019/05/14 12:47:01 [error] 4395#4395: *180 FastCGI sent in stderr: "PHP message: testlog
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releas
2019/05/14 12:47:01 [error] 4395#4395: *180 FastCGI sent in stderr: "/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2 to be array, null given in /var/www/whatever/releases/20190514121609/web/wp-content/plugins/bs-payone-woocommerce/src/Payone/Payone/Api/DataTransfer.php on line 93
PHP message: PHP Warning:  array_key_exists() expects parameter 2
2019/05/14 12:47:01 [error] 4395#4395: *180 upstream sent too big header while reading response header from upstream, client: <IP_ADDRESS>, server: whatever.com, request: "GET /wp-admin/admin.php?page=payone-transaction-log HTTP/2.0", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "whatever.com"
2019/05/14 12:47:04 [error] 4395#4395: *180 FastCGI sent in stderr: "PHP message: testlog" while reading response header from upstream, client: <IP_ADDRESS>, server: whatever.com, request: "GET /wp-admin/admin.php?page=payone-settings-account HTTP/2.0", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "whatever.com"
2019/05/14 12:47:05 [error] 4395#4395: *180 FastCGI sent in stderr: "PHP message: testlog" while reading response header from upstream, client: <IP_ADDRESS>, server: whatever.com, request: "GET /wp-admin/admin.php?page=payone-api-log HTTP/2.0", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "whatever.com"
2019/05/14 12:47:09 [error] 4395#4395: *257 FastCGI sent in stderr: "PHP message: testlog" while reading response header from upstream, client: <IP_ADDRESS>, server: whatever.com, request: "GET /Widerrufsrecht HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "whatever.com"

Hi @vardumper
yep it seems like we fixed that in our master. Adding a composer.json seems reasonable, so I'll look into that and close this one here.


Change PlantTrait to Trait ready for future migration of COMADRE

Change PlantTrait to Trait...
Will need to change the model, relationships, meta table foreign keys, as well as migration commands and templates.
Might be a simple find & replace job...

@simonrolph I think I edited this for all of the views and relevant templates, but seeing as though you've been working on these quite a lot, please could you cast an eye and double check? Then I can close this issue. Cheers!


merge or obsolete GO:1902505 , no annotations

this would only apply to 'within' pathway negative regulation.
No annotations.
GO:1902505 negative regulation of signal transduction involved in mitotic G2 DNA damage checkpoint

I'll merge into GO:0007095 mitotic G2 DNA damage checkpoint signaling

already done in https://github.com/geneontology/go-ontology/issues/20935


telegram killed

@solderzzc 47e1ddd1e0a173cdade790981891a360d5cba42b kill telegram :-)
in dev new version with worked telegram

Traceback (most recent call last):
  File "pokecli.py", line 781, in <module>
    main()
  File "pokecli.py", line 139, in main
    bot.tick()
  File "PokemonGo-Bot/pokemongo_bot/__init__.py", line 658, in tick
    if worker.work() == WorkerResult.RUNNING:
  File "PokemonGo-Bot/pokemongo_bot/cell_workers/telegram_task.py", line 58, in work
    if not re.match(r'^[0-9]+$', "{}".format(self.config['master'])): # master was not numeric...
NameError: global name 're' is not defined

Nice . . .

@askovpen  Dev merged to master, the issue should be fixed.
https://github.com/PokemonGoF/PokemonGo-Bot/pull/4961


Remote istio-reader needs more cluster permissions

Fixes #32244
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@frankbu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
689e152488b11f491b6302c882054b0ba11de1e1
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@frankbu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
689e152488b11f491b6302c882054b0ba11de1e1
link
/test release-notes_istio

integ-security-multicluster-tests_istio
689e152488b11f491b6302c882054b0ba11de1e1
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@frankbu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
689e152488b11f491b6302c882054b0ba11de1e1
link
/test integ-security-multicluster-tests_istio

release-notes_istio
175a36f57513c2320df1a29b5735ac58838dc4f8
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@frankbu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
27fa92ecbcca62836b849c1b098872709214893e
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@frankbu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
520221764f2ec4f8a6666f409933cb7a1c92a2b9
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #32651


Allow deps, classpath, local repo to be specified in config

@arichiardi Does this look like what you want?

Maybe nesting it's better, yeah.  Note I haven't released yet.

Yeah I saw then the commit was in a PR. Nesting feels idiomatic as well and you'd avoid that warning in Readme


is there an equivalent of vscode.workspace.textDocuments ?

hi all,
I'm trying to get all my currently opened files using your language-server-procol. But there is no equivalent of vscode.workspace.textDocuments that I could find.
I didn't want to use the vscode as a dependency, since i want to be able to port my server to other text editors properly.
So it there any equivalent at all? or any work around?
my repo - https://github.com/ganezdragon/perl-language-server
Thanks,
Ganesan

Opened documents are signaled to the server using the textDocument/didOpen notification. It is up to a server implementor to decide how he surfaces this to possible clients.


Put const in each member that should not be modified

Fixes #35
@VictorLamoine Ready for review

saveDilatedMeshes(std::string path);

path should be const.
 void Bezier::displayNormal(std::vector<Eigen::Affine3d,
                            Eigen::aligned_allocator<Eigen::Affine3d> > way_points_vector,
                            std::vector<bool> points_color_viz,
                            ros::Publisher &normal_publisher)

points_color_viz and normal_publisher should be const. Same for displayTrajectory.
   void
   displayMesh(ros::Publisher &mesh_publisher,
               std::string mesh_path,
               float r = 0.6, float g = 0.6, float b = 0.6, float a = 1.0);

Everything should be const.
   void
   setTranslationMode(bool use_translation);

Const too.
   bool
   dilation(double depth,
              vtkSmartPointer<vtkPolyData> &dilated_polydata);

depth should be const.
   bool
   translation(double depth,
               vtkSmartPointer<vtkPolyData> poly_data,
               vtkSmartPointer<vtkPolyData> &translation_poly_data);

depth and poly_data should be const.
And so on!

@VictorLamoine Done

@VictorLamoine:
Apparently it was the only one that I missed, it should be OK now.


Placement Constraint Aware Sequence Pair Perturbation

Sequence pair generation might yield infeasible solutions which waste time during Simulated Annealing. The perturbation could be enhanced to eliminate infeasible candidates when there are order and align constraints. Below is a set of rules that can be used to quickly eliminate infeasible candidates:
order A, B, C left_to_right
Legal solutions: A, B or C cannot be shuffled to arbitrary indices
( .. A .. B .. C .. ), (.. A .. B .. C ..)

order with abut A, B, C left_to_right
Illegal solutions: A fourth block cannot be between an ordered pair
( .. A .. D .. B .. C .. ), (.. A .. D .. B .. C ..)
( .. A .. B .. D .. C .. ), (.. A .. B .. D .. C ..)

order A, B, C top_to_bottom
Legal solutions: A, B or C cannot be shuffled to arbitrary indices
( .. A .. B .. C .. ), (.. C .. B .. A ..)

order with abut A, B, C top_to_bottom
Illegal solutions: A fourth block cannot be between an ordered pair
( .. A .. D .. B .. C .. ), (.. C .. D .. B .. A ..)
( .. A .. B .. D .. C .. ), (.. C .. B .. D .. A ..)

align A, B, C bottom (lly_a = lly_b = lly_c)
Legal solutions: A is either right or left of B
( .. A .. B .. ), ( .. A .. B .. )
( .. B .. A .. ), ( .. B .. A .. )

align A, B, C left (llx_a = llx_b = llx_c)
Legal solutions: A is either top or bottom of B
( .. A .. B .. ), ( .. B .. A .. )
( .. B .. A .. ), ( .. A .. B .. )

@854768750 , @srini229 fyi.

When a single element of the sequence pair is perturbed, then the ordering constraint restricts the position the element can be moved. For e.g. in a horizontal ordering constraint with A to the left of B.  (....A....B) (......A..B), A can only move to any position before B in both the sequences. This can be enforced when changing A's position.
I have routines to do this locally in a separate branch. You can find these changes in

Accidentally closed the issue. You can find my changes in enhancement/constr_aware_seq_pair

Let's edit #823 to capture the logic with examples?

Good idea.

This issue is addressed in #823.


Text-only-mode for parsing

Would there be any interest in a flag for text-only parsing? Basically, it would be a flag that turned off readElement (and probably readHtmlComment) in the parser. I've found that this would come in handy for support modules around a Ractive app, like building pages in a CMS or exporting data using a template that may have <, >, & in it. Reversing a parse has been mentioned before (#1998), but I don't think that that would entirely suit these use-cases, unless...
If there were a method to reverse a parse that included a flag to only do so partially, this would be covered. Something like Ractive.unparse(someParsedTemplate, { only: 'elements' }) would work, but looks a bit nasty from where I sit. Comments and alternate suggestions are certainly welcome!
For reference, my primary use-case is a CMS that uses Ractive to assemble bits of Ractive template into whole pages. I'm lazy and didn't want to write my own parser or require users to understand the differences between template flavors, so I have Ractive munge the templates one time through using different delimiters and spit out the result with .toHTML() for use on the actual page. This works great except for a few corner cases, like < and > in expressions and elements with directives.
TL;DR double parsing can lead to tears and dirty regex hacks

Turns out my brain wasn't fully enabled when I was thinking that an unparse would work for expressions that won't parse unless they're in a current mustache.
I was also thinking about ways to implement progressive enhancement, and it occurred to me that skipping the element parse would be good for the server-side render. I haven't done any sort of benchmarking, but it seems reasonable that not parsing would be faster than parsing and unparsing to get directives and whatnot back.


Fix rest metrics instantiation - auto update to onflow/cadence v0.40.0

Description
Fixes issues with rest metrics initialization discovered by #461

For contributor use:

[ ] Targeted PR against master branch
[x] Linked to GitHub issue with discussion and accepted design OR link to spec that describes this work
[x] Code follows the standards mentioned here
[x] Updated relevant documentation
[x] Re-reviewed Files changed in the GitHub PR explorer
[x] Added appropriate labels

Codecov Report

:exclamation: No coverage uploaded for pull request base (auto-update-onflow-cadence-v0.40.0@02a44bf). Click here to learn what that means.
The diff coverage is n/a.

@@                          Coverage Diff                          @@
##             auto-update-onflow-cadence-v0.40.0     #462   +/-   ##
=====================================================================
  Coverage                                      ?   54.85%
=====================================================================
  Files                                         ?       28
  Lines                                         ?     3617
  Branches                                      ?        0
=====================================================================
  Hits                                          ?     1984
  Misses                                        ?     1473
  Partials                                      ?      160

Flag
Coverage Δ

unittests
54.85% <0.00%> (?)

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

Great! Thank you 🙏


[Markup] Print Tags in documentation comment XML

This information needs to be picked up through SourceKit. It might be
useful as both metadata for sorting/filtering as well as presentation,
so it makes sense to print it in the normal XML inside CommentParts.
rdar://problem/32877771

@swift-ci Please smoke test


Customize it for other languages

Hello,
how can I customize it for other language?
For example change the point "experience" to the german word "Erfahrungen" etc.
Thx

You can change the language code here:
https://github.com/salomonelli/best-resume-ever/blob/master/resume/data.yml#L69
If you want "Erfahrungen" instead of "Berufserfahrung" you could change it locally in this file:
https://github.com/salomonelli/best-resume-ever/blob/master/src/lang/de.js


there is no "tool" file in "opt/go/pkg", what should I do?

There is no "tool" file in "opt/go/pkg", what should I do ?

You should find the path which golang installed. For example, my install path is /usr/local/Cellar/go/1.16.3/, so the darwin_amd64 path would be /usr/local/Cellar/go/1.16.3/libexec/pkg/tool/darwin_amd64.

echo "$(cd which go/../../pkg/tool/darwin_amd64 && pwd)"
Through this command, you can locate the directory.


Support FUSE_READDIRPLUS

See: https://github.com/zargony/fuse-rs/pull/152/commits/d3bf0a8029ed219d5cba4b32b32fcfe420cd8e39


fix RETextClassificationWithIndicesTaskModule for NaryRelation

NaryRelation was not yet allowed in .encode_target()

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

Project coverage is 95.89%. Comparing base (558ea40) to head (eea9cc5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #87   +/-   ##
=======================================
  Coverage   95.89%   95.89%
=======================================
  Files          55       55
  Lines        3926     3926
=======================================
  Hits         3765     3765
  Misses        161      161

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


RSS Link

Hi, I just tried the RSS. The link seems to be not functioning properly.
Upon investigating the issue, I discovered that the current RSS link is: example.com/hello-world
However, it appears that the correct link should be: example.com/posts/hello-world
the code link: post.slug, should the code looks something like this link: `/posts/${post.slug}/`, ? in rss.xml.ts

Hi @vegetablesB, thanks for picking this up and your help, hopefully it should be fixed now


Dynamic notices fix for table detail page

Fix - This PR fixes the following issues in Amundsen:

Issue # 2204 Dynamic notices are not updated/changed if we use the "nav bar" search at the top to open another table.
Issue # 2205 Dynamic notices showing stale values from other tables and/or not updating correctly.

Description
This PR updates the table detail page to initiate a new request for the Notice API every time we transition over from the "TableDetailPage" over to the "TableDetailPage" for a different table. Furthermore, it makes use of the isLoadingNotices flag to hide stale / previously loaded values from the component while a new one is being fetched via the Notice API.
Motivation and Context
Issue # 2204 and Issue # 2205 prevented us from using a critical feature and depreciated the overall user experience of our users. This change helps address those issues.
How Has This Been Tested?
This has been tested locally as well as on the staging environment.
Documentation
The documentation did not require any further changes, beyond what's already mentioned.
CheckList

[x] PR title addresses the issue accurately and concisely
[ ] Updates Documentation and Docstrings
[ ] Adds tests
[ ] Adds instrumentation (logs, or UI events)

Please rebase with master to fix the checks!


Add MIMIC-IV as health domain dataset

https://www.nature.com/articles/s41597-022-01899-x

@vankesteren I have checked and I think they use a different definition of freely available than we do, it seems it is only available on request and cannot be republished. See the PR #330 for an alternative.


Data Events are not compatible with Cross-Company Data Sharing

As per the feedback we received in the support ticket<PHONE_NUMBER>001722 Data Events are not compatible with Cross-Company Data Sharing. This need to be added to the list of Limitations.

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

ID: 07ba5d8f-e335-9d6f-0418-7529f19d558f
Version Independent ID: 1ff0ba1d-54f0-8f59-49d3-c8ab44470cd9
Content: Cross-company data sharing - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/sysadmin/cross-company-data-sharing.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @Peakerbl
Microsoft Alias: peakerbl

Hi @abylkhatov,
Thanks for reaching out.
There was a bug with cross-company data sharing which blocked tables from being shared when the business events were enabled on tables or vice versa. This bug is fixed now. The only restrictions with cross-company data sharing are with dual write, but it's documented already.
Thanks again.


Disable replacement of the bag bar when BT4 is in control of it

I tested clicking each bag button, bag key bind, and bank access. I have not been able to get the Blizzard Bag UI to open with this in place.
The current replacement process destroys the existing buttons and Bartender4 controls the visibility and placement of them when it is installed, this leads to the placement being way off.
The current process leads to the button being off before:

after:

I added a message in the options, and disabled the "Show Blizzard Bag Button" option since with this that call is never made as BT4 handles that.

@Cidan  Thoughts on this? It's driving me nuts.


[Tabs] Tab children should have auto width

When using Tabs with Tab children, each Tab child has a fixed-set width. Instead, the width of each Tab should be set to 'auto'.
It'd be nice to have a prop to toggle 'fixed' vs 'auto', as there are also situations where we want fixed Tab widths.

Dupe of #3807.


Using twaindotnet in a thread...

I am really stuck on utilizing this codebase because in my case, interaction with the scanner must be on a separate thread from my MDI style windows form application.
My form app that starts a background thread (BGthread).  The BGthread creates a System.Collection.Concurrent.BlockingCollection and processes the commands that get added to the collection.  This BGthread uses the TwainDotNet classes to interact with the downstream scanner.
My MDI app puts a command into the collection monitored by the BGthread.  For example, an instance of command would be a command to instantiate the TwainDotNet.Twain class.  A follow-up instance of command would be to execute TwainDotNet.Twain.StartScanning().  And so on…
My issue is that the TwainDotNet.WinFroms.WinFormsWindowMessageHook.PrefilterMessage() method is not ever executed. The scanner behavior at this point is that the motor starts up as if to engage the document feeder.  However, the actual scan behavior never starts up.  The scanner holds in this state for a few seconds and then the motor shuts off and goes into an idle state.  Additionally, the MDI app is in a non-responsive state (maybe because the ImageHasTransferred/ScanningComplete events are not posted).
Suspect that this behavior is because TwainDotNet.Twain.StartScanning() is executed on my BGthread -not on my MDI app thread.  When I remove the threading, things are working successfully.
Very grateful for any help, suggestions and especially answers!

You could try using the "newer" method of registering directly with TWAIN to receive a callback, instead of hooking the message pump. See page (2-9) of the twain spec, under the heading Using DAT_CALLBACK for Messages from the Source to the Application.


Refactor str() implementation

I was looking through some of the source code for various elements and it seems like you're doing loads of repetitive work. For example, here's the __str__() for the div element.
def __str__(self):
        output = [ "<div" ]

        if self.ident:
            output.append(" id=\"")
            output.append(self.ident)
            output.append("\"")

        if self.cl:
            output.append(" class=\"")
            output.append(self.cl)
            output.append("\"")

        if self.style:
            output.append(" style=\"")
            output.append(self.style)
            output.append("\"")

        if self.attrs:
            for k in self.attrs.keys():
                output.append(' ' + k + '="' + self.attrs[k] + '"')

        output.append(">")

You could probably make your life a bit easier by using getattr() and friends. So something like this:
def __str__(self):
    output = ['<div']

    attributes = [
           # tuple of (attr_name_in_self, attr_name_in_html)
            ('ident', 'id'),
            ('cl', 'class'),
            ('style', 'style'),
    ]

    for name, as_html in attributes:
        # Grab the value of the `name` attribute from `self`
        value = getattr(self, name)
        if value:
            output.append(' {}="{}"'.format(as_html, value))

    output.append('>')

It looks pretty complicated, but I'm sure it wouldn't be difficult to extend/abstract out something like so the generation of most html elements can be done automatically.
This is an awesome project by the way, I only came across it the other day but I've already got plenty of ideas for places to use it. Keep up the great work!

I agree, there's a lot of repetitive work, after I had the first dozen or so widgets I started to look into refactoring, especially compositing some widgets on top of others, and I ran into a few issues with slight differences in how the HTML needed to be put together that made things a less legible, so for the moment I've chosen to flesh it all out and do a refactoring sweep once things are further along.
I like what you're suggesting, may make sense to stick that in the Element class to have a default way of generating the HTML and then overriding it wherever it makes sense to do something different. I'll stick it in the queue.

You could even use the beautifulsoup module to generate all the text for you.
https://www.crummy.com/software/BeautifulSoup/

I thought of a new Element like the one below but it would be to code something already existing:
`
class Element(object):
"""Base HTML tag element"""
def __init__(self, name, cls=None, ident=None, style=None, paired=True, **attrs):
    """Create a base element where cls is a space separated class attribute,
       ident is the element id and style is a CSS style string"""

    self.name = name

    if cls:
        self.cls = set(cls.split())
    else:
        self.cls = set()

    self.ident = ident
    self.style = style
    self.attrs = attrs
    self.paired = paired

    if paired:
        self.children = list()

def __repr__(self):
    return self.__str__()

def __str__(self):

    output = ['<' + self.name]
    if self.ident:
        output.append(' id="' + self.ident + '"')
    if self.cls:
        output.append(' class="' + ' '.join(self.cls) + '"')
    if self.style:
        output.append(' style="' + self.style + '"')
    if self.attrs:
        output += [' ' + k + '="' + v + '"' for k, v in self.attrs.items()]

    output.append('>')

    if self.paired:

        output += [str(child) for child in self.children]
        output.append('</' + self.name + '>')

    return ''.join(output)

def addclass(self, cls):
    """Add one or multiple classes to this element"""

    self.cls |= set(cls.split())

def removeclass(self, cls):
    """Remove one or multiple classes from this element"""

    self.cls -= set(cls.split())

def addchild(self, elem):
    """Add a child element to this tag"""

    self.children.append(elem)

def removechild(self, elem):

    self.children.remove(elem)

`
With this class you could implement Select like this:
`
class Select(Element):
"""Implements a boostrap select tag"""
def __init__(self, **kwargs):
    super().__init__(name='select', **kwargs)

def addOption(self, opt):

    # opt is an Option instance
    if isinstance(opt, Option):
        self.addchild(opt)
        return opt

    # opt is string
    result = opt.split()

    for option in result:
        self.addchild(Option(option))

    return result

def removeOption(self, opt):

    if isinstance(opt, Option):
        self.removechild(opt)
    else:

        opt_set = set(opt.split())

        self.children = [
            c for c in self.children
            if not isinstance(c, Option) or c.name not in opt_set
        ]

def setdefault(self, opt, default=True):

    for child in self.children:
        if isinstance(child, Option) and child.value == opt:
            child.setDefault(default)

class Option(Element):
"""Implements  tag for select"""
def __init__(self, value, **kwargs):
    super().__init__(name='option', **kwargs)
    self.value = value
    self.addchild(value)

def setDefault(self, default=True):
    if default:
        self.attrs['selected'] = 'selected'
    elif 'selected' in self.attrs:
        del self.attrs['selected']

`

@ugy, I like the idea of leveraging BeautifulSoup to do all the html construction. I've used it a lot for web scraping and html manipulation and it gives you a really nice API to work with.
Automating the html construction sounds like a lot nicer way to do things than manual string manipulation.

On BeautifulSoup. It's a good idea, haven't used it in a while, I'll look into it a bit more. However, I'm trying to keep dependencies to a minimum.
While string manipulation sounds ugly / low level, we're not pulling data from a webpage or processing the HTML in any way. In fact we're done with it as soon as it's sent over the socket, anything we might need from the page, we already have in the form of class instances and variables.


build as library error,This dependency was not found:* ~entry  in<EMAIL_ADDRESS>

Version
4.4.1
Reproduction link
https://github.com/zhouhuan327/awesome-wheels
Environment info
Environment Info:

  System:
    OS: Windows 10 10.0.18363
    CPU: (8) x64 Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz
  Binaries:
    Node: 12.16.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 6.13.4 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: 44.18362.449.0
  npmPackages:
    @vue/babel-helper-vue-jsx-merge-props:  1.0.0
    @vue/babel-plugin-transform-vue-jsx:  1.1.2
    @vue/babel-preset-app:  4.2.3 (4.4.1)
    @vue/babel-preset-jsx:  1.1.2
    @vue/babel-sugar-functional-vue:  1.1.2
    @vue/babel-sugar-inject-h:  1.1.2
    @vue/babel-sugar-v-model:  1.1.2
    @vue/babel-sugar-v-on:  1.1.2
    @vue/cli-overlay:  4.4.1
    @vue/cli-plugin-babel: ~4.2.0 => 4.2.3
    @vue/cli-plugin-router:  4.4.1
    @vue/cli-plugin-unit-jest: ~4.3.0 => 4.3.1
    @vue/cli-plugin-vuex:  4.4.1
    @vue/cli-service: ^4.4.1 => 4.4.1
    @vue/cli-shared-utils:  4.4.1 (4.2.3, 4.3.1)
    @vue/component-compiler-utils:  3.1.2 (3.1.1)
    @vue/preload-webpack-plugin:  1.1.1
    @vue/test-utils: 1.0.0-beta.31 => 1.0.0-beta.31
    @vue/web-component-wrapper:  1.2.0
    jest-serializer-vue:  2.0.2
    vue: ^2.6.11 => 2.6.11
    vue-hot-reload-api:  2.3.4
    vue-jest:  undefined (3.0.5)
    vue-loader:  15.9.2
    vue-router:  3.3.1
    vue-server-renderer:  2.6.11
    vue-style-loader:  4.1.2
    vue-template-compiler: ^2.6.11 => 2.6.11
    vue-template-es2015-compiler:  1.9.1
    vuepress: ^1.5.0 => 1.5.0
    vuepress-html-webpack-plugin:  undefined (3.2.0)
    vuepress-plugin-container:  2.1.4
    vuepress-plugin-smooth-scroll:  0.0.3
  npmGlobalPackages:
    @vue/cli: Not Found

Steps to reproduce
run   vue-cli-service build --target lib --name myLibName src/index.js
What is expected?
build success
What is actually happening?
i got error tips three times
 ERROR  Failed to compile with 1 errors                                                                                4:47:43 PM

This dependency was not found:

* ~entry in<EMAIL_ADDRESS>
To install it, you can run: npm install --save ~entry

run  " vue-cli-service build" was  successful
but  build with "target lib" can not

Delete these lines:
https://github.com/zhouhuan327/awesome-wheels/blob/2ae067c1245ebbb6f86b7d09777718898f8efafd/vue.config.js#L16-L27
These are all configured by default, and by calling Object.assign, you are overriding the default configuration fields such as the additional aliases required for the lib target.
(And upgrade @vue/cli-plugin-babel to ~4.4.0, seems there's a bug with the old version)


IR emitter blink only with specific resolution

Hi everyone,
I had write an issues few month ago: https://github.com/boltgolt/howdy/issues/19
I was a bit curious about why emitter emit when I used fswebcam but not when I used Cheese or Howdy. I play with it today and notice something:
By default fswebcam adjust the resolution from 384x288 to 340x340
I had try to use the parameter -r to set another resolution and it adjust resolution from 1280x720 to 640x480 and the emitter didn't work.
Where you aware of this issue ?
I also try to set Cheese in 340x340 and got this as a result, has I never used IR camera, is this a valid output ? http://webmshare.com/play/QNzyr
P.S: I'm on Ubuntu 18.04

Since your first issue i've seen a few of these weird resolution problems. For this reason i introduced the frame_width and frame_height config options in the latest release, you might want to check those out. For some people only the largest resolution works (like for me), for some you need to smaller one. There's not great default here unfortunately.
Your output is completely valid. The flashing if the LEDs are never in sync and produce this effect where about half of the frames are dark. Howdy automatically ignores these dark frames, as you can see with sudo howdy test.

Perfect, thanks for the answer.
I will try to make it work :)

I just tried to use howdy on my Thinkpad X1 Tablet Gen3 and had the same issue,
fixed it using the resolution 340x340.
But however, it took too long to discover this issue.
@boltgolt Would you please add a good description of this issue into the README file,
so that people having the same issue / using the same devices as we can find this fix more easily?

I've added a wiki page for common issues and also liked to it in the readme


Quick Start fails during first vagrant up

When I try the Quick Start the first vagrant up fails with the below error messages.
When I then ssh into the box and try to run sbt it says sbt does not exists. I guess thats because the vagrant provision failed.
Here is the output from vagrant
==> default: Checking connectivity... done. ==> default: git fetch https://github.com/snowplow/ansible-playbooks.git ==> default: === ==> default: Traceback (most recent call last): ==> default:   File "/usr/local/bin/peru", line 9, in <module> ==> default:     load_entry_point('peru==0.4.0', 'console_scripts', 'peru')() ==> default:   File "/usr/local/lib/python3.4/dist-packages/peru/main.py", line 345, in main ==> default:     async.run_task(command_fn(params)) ==> default:   File "/usr/local/lib/python3.4/dist-packages/peru/async.py", line 25, in run_task ==> default:     return asyncio.get_event_loop().run_until_complete(coro) ==> default:   File "/usr/lib/python3.4/asyncio/base_events.py", line 316, in run_until_complete ==> default:     return future.result() ==> default:   File "/usr/lib/python3.4/asyncio/futures.py", line 275, in result ==> default:     raise self._exception ==> default:   File "/usr/lib/python3.4/asyncio/tasks.py", line 236, in _step ==> default:     result = coro.send(value) ==> default:   File "/usr/local/lib/python3.4/dist-packages/peru/main.py", line 91, in do_sync ==> default:     params.runtime, params.scope, params.imports, params.runtime.sync_dir) ==> default:   File "/usr/local/lib/python3.4/dist-packages/peru/imports.py", line 18, in checkout ==> default:     previous_index_file=index) ==> default:   File "/usr/local/lib/python3.4/dist-packages/peru/cache.py", line 414, in export_tree ==> default:     yield from session.checkout_files_from_index() ==> default:   File "/usr/local/lib/python3.4/dist-packages/peru/cache.py", line 189, in checkout_files_from_index ==> default:     yield from self.git('checkout-index', '--all') ==> default:   File "/usr/local/lib/python3.4/dist-packages/peru/cache.py", line 77, in git ==> default:     raise GitError(command, process.returncode, stdout, stderr) ==> default: peru.cache.GitError: git command "['git', '--git-dir=/vagrant/vagrant/.peru/cache/trees', '--work-tree=/vagrant/vagrant', 'checkout-index', '--all']" returned error code 128. ==> default: stdout: ==> default: stderr: ansible/lib/ansible/module_utils/redhat.py already exists, no checkout ==> default: ansible/library/cloud/ec2_scaling_policy already exists, no checkout ==> default: ansible/library/source_control/hg already exists, no checkout ==> default: oss-playbooks/leiningen.yml already exists, no checkout ==> default: oss-playbooks/roles/packer/meta/main.yml already exists, no checkout The SSH command responded with a non-zero exit status. Vagrant assumes that this means the command failed. The output for this command should be in the log above. Please read the output to determine what

Hi @pj-moviestarplanet , I've added a recovery guide here: http://discourse.snowplowanalytics.com/t/problems-with-vagrant-and-our-github-repositories/122


[P2P] TCP Connection Pooling (Earliest Recently Used, Last Recently Used Push Queue)

Objective
Achieve an efficient connection pool to minimize TCP roundtrips that come with establishing new connections.
Origin Document
When a connection pool is full, and we need to pool new connections, some space has to be made so some existing pooled connections must be "evacuated".
In order to efficiently use the connections' pool, we need some pool re-ordering logic. A suggestion is to use something similar to the RAM's LRU (Last Recently Used) algorithms, that will evacuate the oldest unused connection in the pool, by always queueing first the most recently used connection, making the last position in the queue, the oldest unused connection in the pool, the one to be evacuated.
Deliverables

[ ] Implement a limited-size connection pool with re-ordering logic:

[ ] The pool must provide methods to retrieve specific questions by address
[ ] The pool must provide evacuate unused connections when new connections are wanting to be pooled to make space for them.
[ ] The pool must not interrupt an on-going pooled connection in favor of a to-be-made connection.

Owners

author/owner: @derrandz

Closing for housekeeping but may reopen in the future.


DllNotFoundException

hi.
i just want to know about below Log Error when i push the shot button.
Log Error>
2021-07-15 01:23:35.236 6353 7026 Error Unity DllNotFoundException: Unable to load DLL 'AirarColorMap': The specified module could not be found.
so file that, libAirarColorMap.so , libopencv_java3.so  path are Assets/Plugins/Android/
please let me know about the solution.. thanks.

Hi,
Set the CPU to ARM64 in the platform settings of the libAirarColorMap.so file and the libopencv_java3.so file.
thanks!


Issues obtaining user flair

Hi!
I've been trying to scrape /r/dailyprogrammer, but I've been having trouble receiving flair text. From this json, it looks like author_flair_text ought to equal "-9 8", but it is null instead. I've provided example code to replicate the issue.
public void minimalCase() {
        RootCommentNode comments = client.submission("8g0iil").comments();
        comments.loadFully(client);
        Optional<CommentNode<Comment>> commentNode = comments.getReplies()
                .stream()
                .filter(comment ->
                        comment.getSubject()
                                .getId()
                                .equals("dy7xbrm"))
                .findFirst();
        commentNode.ifPresent(node -> {
            assert node.getSubject().getAuthorFlairText() != null;
            // should equal "-9 8"
        });
    }

I might be using the API wrong, as I only have a few days of experience with it. If so, what is the right way of getting user flair?
Thanks,
Connor

This is actually a bug. I had Moshi look for a property called authorFlairText instead of author_flair_text.
Can you confirm this is fixed?
repositories {
    maven { url 'https://jitpack.io' }
}
dependencies {
    compile 'com.github.mattbdean:JRAW:4942855'
}


duplicate entries in make dependency file generation

When using the -M dependfile option in rgbasm, the resulting dependency file has duplicate entries. For example
foo.asm
INCLUDE "A.inc"
SECTION "code", ROM0
nop

A.inc
INCLUDE "B.inc"

B.inc (empty file)

Assemble:
rgbasm -M foo.d -o foo.obj foo.asm

foo.d:
foo.obj: foo.asm
foo.obj: A.inc
foo.obj: B.inc
foo.obj: A.inc
foo.obj: B.inc

The above file will also be generated if A.inc is empty and INCLUDE "B.inc" is added to foo,asm. The correct file should be the same as the above without the last two lines (kinda looks like it got written twice somehow).
While this file does list all dependencies for foo.obj and will work fine in make, it could be an issue for projects with a large number of includes.

It has been a problem for me, causing recipe overrides in Make (which is undefined behavior, btw), and required piping the output through sort | uniq
I have ~600 such lines already, especially after being passed to a sed script to simulate -MP

Does this still happen on current master? The parser was rewritten
semi-recently to use only one pass.

I haven't been able to find a duplicate entry using latest master.

For that reason, closing unless the issue resurfaces.


Remove Ignore tag from flowShouldFailWhenConditionalParameterDoesntExist

Remove Ignore tag from flowShouldFailWhenConditionalParameterDoesntExist and make it run

Pull Request Test Coverage Report for Build 7046

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.07%) to 45.477%

Files with Coverage Reduction
New Missed Lines
%

azkaban-common/src/main/java/azkaban/jobcallback/JobCallbackRequestMaker.java
3
80.21%

Totals

Change from base Build 7038:
0.07%

Covered Lines:
18263

Relevant Lines:
40159

💛  - Coveralls


error running command

Error running command hanaExplorerView.bindLocal.hdiContainer: Failed to chdir to /local/source/openvscode-server: No such file or directory Something unexpected happened. This is a bug in cf. Please re-run the command that caused this exception with the environment variable CF_TRACE set to true. Also, please update to the latest cli and try the command again: https://code.cloudfoundry.org/cli/releases Please create an issue at: https://code.cloudfoundry.org/cli/issues Include the below information when creating the issue: Command cf curl /v3/service_offerings?space_guids=830c4854-133b-4f35-ba1c-bae1867ee947&per_page=297 CLI Version 8.5.0+73aa161.2022-09-12 Error template: Display Text:1: function "app_router_url" not defined Stack Trace goroutine 1 [running]: code.cloudfoundry.org/cli/util/panichandler.HandlePanic() /home/runner/work/cli/cli/util/panichandler/handler.go:19 +0x57 panic({0x102e080, 0xc0003b80f0}) /opt/hostedtoolcache/go/1.18...
What happened
in SAP BusinessApplicationStudio the HANA Tooling offers an assignment of a BINDING. This fails with this error message
Expected behavior
no error message

I can't find any reference or app_router_url in the cli or capi code. This seems SAP-specific.
I'm resolving this issue as fixed for now. Please don't hesitate to reopen it if you have any more questions or comments.


weechat-matrix: 0.2.0 -> 0.3.0

Motivation for this change
The newest version fixes a bug preventing non-SSO connections.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Tested non-SSO logins, but unable to test SSO logins.

Result of nixpkgs-review pr 129186 at f4336e48 run on aarch64-linux 1

1 package built successfully:

weechatScripts.weechat-matrix

1 suggestion:

warning: python-include-tests
Consider adding a checkPhase for tests, or if not feasible, pythonImportsCheck.
Near pkgs/applications/networking/irc/weechat/scripts/weechat-matrix/default.nix:80:0:
   |
80 |     description = "A Python plugin for Weechat that lets Weechat communicate over the Matrix protocol";
   | ^


[WIP] Update Curation Concerns to 1.7.1

Coverage decreased (-7.08%) to 92.925% when pulling 3d0a87ed113b0236a61a2f11c2849879b039e31a on 920-cc-1-7-3 into 145e868ad09af731aa8fba99e048918b733c01f4 on master.


add references for kulprit and model selection

Codecov Report

Merging #52 (ea77996) into main (aa53673) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #52   +/-   ##
=======================================
  Coverage   81.22%   81.22%
=======================================
  Files          13       13
  Lines         522      522
=======================================
  Hits          424      424
  Misses         98       98

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


MQTT broke?

I flashed latest version (0.8.1).
I decided to do a clean update, so erased the flash, flashed both files of the 0.8.1 using the command from the wiki.
Logged into wifi, configured wifi.
Rebooted and logged into web and configured mqtt and the others.
I can see the webinterface,  under device info it indicates it is connected to mqtt.
I don't see anything published to mqtt, also otmonitor cannot connect ( connection refused).
I can connect to port 23, and I see messages going by.
This is device info:
Developer                                       Robert van den Breemen
NodeMCU Firmware Version          0.8.1+92f10cb (14-03-2021)
OTGW PIC Firmware Version         4.2.5
Compiled on (date/time)                 Mar 14 2021 14:56:26
HostName                                      OTGW
IP address                                     <IP_ADDRESS>
MAC address                                **
Free Heap Memory (bytes)          18328
Max. Free Memory (bytes)           16568
Unique Chip ID                             *****
Arduino Core Version                   2_7_4
Espressif SDK Version                 2.2.2-dev(38a443e)
CPU speed (MHz)                       160
Sketch Size (bytes)                      504.781
Free Sketch Space (bytes)         1540
Flash ID                                      00164020
Flash Chip Size (MB)                  4
Real Flash Chip Size (MB)         4
LittleFSsize                                 2
Flash Chip Speed (MHz)           40
Flash Mode                               DIO
Board Type                                ESP8266_NODEMCU
Wifi Network (SSID)                  ***
Wifi Receive Power (dB)            -55
MQTT Connected                     true
NTP Enable                              true
NTP Timezone                         Europe/Amsterdam
Uptime since boot                    0(d)-00:03(H:m)
Last Reset Reason                  Power On
Nr. Reboots                               3

I am a bit lost how to debug this.

@DaveDavenport could you downgrade to version 0.8.0?

@DaveDavenport also to what port are you connecting for OTmonitor?

aah that works. did that change?
I can see in my mqtt server logs it does connect to the server.
It also pushes the homeassistant settings (I don't use)
homeassistant/sensor/otgw-2CF4327817F3/ASF_oemfaultcode/config

but nothing under OTGW/#
Ill try downgrade.

@DaveDavenport yes it changed in v0.7.1
What version where you using before?  The way it works has changed in version v0.8.0. That has been an breaking change. The OTGW now published its topics under it's device. By making it unique it can actually work with multitple OTGW's on a single hub.
So what are you using as your home automation platform? Are you depending on the toptopic ('/OTGW'). If you explore your MQTT broker, you will notice that the published sensors are all still there, they just moved.

The format for the mqtt topic is now:

/OTGW/value/otgw-/
/OTGW/set/otgw-/command/

I just realized that I need to update the documentation to match.

aah that works. did that change?

homeassistant/sensor/otgw-2CF4327817F3/ASF_oemfaultcode/config
So the auto discovery templates are pushed...

but nothing under OTGW/#
That's weird, you should see values under the value topic, except that now you have the unique identifier in between.

Ill try downgrade.
You have to drop to a release below v0.8.0 to not have the unique id's.

Please let me know if that worked for you.

Yeah, I was many releases behind.
That and I saw the breaking change, so I did a complete clean flash.
As I mentioned I did not see anything on `OTGW/#'.  But I will look further down the chain.

You should see topics published under /OTGW/ though. So I wonder what's going on.
But I cannot explain it;  I saw that I was getting weird sockets errors in my mosquitto daemon.. realized was on an old version, so upgraded the daemon..
now it works.
Weird thing is I have everything going via mosquitto, and everything worked fine (my log files confirm this) except otgw.
I am confused af, but will take it as a win.. Sorry for wasting your time (again).

@DaveDavenport well no problem. Better you come and reporting an issue, than having a broken release. You are welcome to join the Discord to chat realtime next time. Just go here: https://discord.gg/gRVUQBKHbp

I bought you a few beers! thanks.


Add trailing slash to link to test #2614

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f85798425c4c6b47b540f3dc83d836e098948992/

Just a test.


Question: [FCB for 1.19.2] How to impregnate a roller with paint?

Question.
I wonder if anyone figured out how to saturate the roller with paint? On the new version 1.19.2.
How the mixer works was guessed in just 2 hours with the help of a hint from my niece. :)

How did you get the mixer to work??

Add redstone signal.


Split screen issues

Issue Type: Bug
Whenever I split view, I can't type anything in the new, split tab.
-I have an html and a css file open in my window.
-I then initiate split view by either:
a. dragging one tab to the right side of the window
b. click the split view icon on the top right
-I can't type anything in that new tab.
I have already restarted VS a couple of times. I can still type in single view, but would definitely like to have the option available.


expect(1).toBeGreaterThan(0); throws exception: undefined is not a constructor

🐛 Bug Report
expect(1).toBeGreaterThan(0); throws exception: undefined is not a constructor
To Reproduce
use typescript 2.4.2
install react 15.4
install jest with jest types for typescript 2.4.2
install enzyme with enzyme adapter 15.4
in tsconfig compileroptions use target: es5
Write a simple test which does only following:
describe('Enzyme basics', () => {
beforeEach(() => {
}
));
});
afterEach(() => {
});
it('should be ok', () => {
expect(1).toBeGreaterThan(0);
});
});
Expected behavior
this test should run and should be ok.
Link to repl or repo (highly encouraged)
envinfo

@pedrottimark could you verify if this is fixed with your latest matchers adjustments?

we can close this. I had a wrong configuration which let into this problem.


WIP: update to latest istio/api changes with MCP enhancements

This is a non-functional change that incorporates the latest MCP API enhancements. Some fields and protos have been renamed which is a backwards incompatible change. However, Istio w/MCP has not been released yet and our integration tests upgrade Galley/Pilot/Mixer at the same time.
Depends on https://github.com/istio/api/pull/741

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ayj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ayj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
383031a6b7ff80088857d193e153a028ad2a6ec1
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
383031a6b7ff80088857d193e153a028ad2a6ec1
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
383031a6b7ff80088857d193e153a028ad2a6ec1
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
383031a6b7ff80088857d193e153a028ad2a6ec1
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
383031a6b7ff80088857d193e153a028ad2a6ec1
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
a86d35f254c74fed14de5e09ab93ae17531450b1
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
383031a6b7ff80088857d193e153a028ad2a6ec1
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
a86d35f254c74fed14de5e09ab93ae17531450b1
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
a86d35f254c74fed14de5e09ab93ae17531450b1
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a86d35f254c74fed14de5e09ab93ae17531450b1
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
a86d35f254c74fed14de5e09ab93ae17531450b1
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
a86d35f254c74fed14de5e09ab93ae17531450b1
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
a86d35f254c74fed14de5e09ab93ae17531450b1
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
288d842b5f135979ee783a53c25ba92af14356a8
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
288d842b5f135979ee783a53c25ba92af14356a8
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
288d842b5f135979ee783a53c25ba92af14356a8
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

not stale - depends on https://github.com/istio/api/pull/741


Error: Specified module does not exist

Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.4.2
node: 8.2.1
os: linux x64
@angular/animations: 4.3.6
@angular/common: 4.3.6
@angular/compiler: 4.3.6
@angular/core: 4.3.6
@angular/forms: 4.3.6
@angular/http: 4.3.6
@angular/platform-browser: 4.3.6
@angular/platform-browser-dynamic: 4.3.6
@angular/router: 4.3.6
@angular/cli: 1.4.2
@angular/compiler-cli: 4.3.6
@angular/language-service: 4.3.6
typescript: 2.3.4

Repro steps.

15:38 $ ng new test-project
15:38 $ cd test-project
15:38 $ ng g m modules/admin
  create src/app/modules/admin/admin.module.spec.ts (267 bytes)
  create src/app/modules/admin/admin.module.ts (189 bytes)
15:38 $ ng g c components/cookie -m modules/admin
  Error: Specified module does not exist
  Specified module does not exist
15:38 $ ng g c components/cookie -m modules/admin/admin.module.ts
  Error: Specified module does not exist
  Specified module does not exist
15:39 $ ng g c components/cookie -m src/app/modules/admin/admin.module.ts
  Error: Specified module does not exist
  Specified module does not exist

As you can see I am trying to add a component to a module instead of the default app.module.ts, but since a recent update it doesn't seem to let me, any more.
ng g c components/cookie -m modules/admin // this used to work :(
The log given by the failure.

Desired functionality.

Mention any other details that might be useful.

Same issue. Any fix?

I held back my version and then updating it now to the latest seemed to fix the issue


Webex Teams startPrivateConversation error: TypeError: Cannot read property 'id' of undefined

What are you trying to achieve or the steps to reproduce?
In Webex Teams, after a group chat message, I am trying to initiate a private conversation with the user that initiated the chat using await bot.startPrivateConversation(message.user)
What was the result you received?
On doing this, I receive the following error:
TypeError: Cannot read property 'id' of undefined
It is this line in conversationState.js that is throwing the error:
const conversationId: string = Object.keys(activity.conversation).sort().map((key) => activity.conversation[key]).filter((val) => val !== '' && val !== null && typeof val !== 'undefined').join('-') + '-' + activity.from.id;

This seems to be because activity.conversation essentially just contains { id: 'temporary-value' } and this.changeContext is returning an empty object.
And in botworker.js, the BotWorker.changeContext function never gets past this line:
const dialogContext = yield this._controller.dialogSet.createContext(turnContext);

What did you expect?
I expected a direct conversation with the user.
Context:

Botkit version: 4.6.2
Messaging Platform: Webex Teams
Node version: v10.16.0
Os: macOS Catalina 10.15.2
Any other relevant information:

It seems this may be a known issue because the documentation for startPrivateConversation says:

Change the context of the next message Due to a quirk in the Webex API, we can't know the address of the DM until after sending the first message. As a result, the internal tracking for this conversation can't be persisted properly. USE WITH CAUTION while we try to sort this out.

This seems to imply that it should work at some level?
Thanks.

Seeing the same issue.  Perhaps an approach could be to go ahead and accept a required message parameter in startPrivateConversation (as well as startConversationInRoom).  The function could then go ahead and send the message, await the response, and extract the roomId/from etc. so that context creation works correctly...

@darrenparkinson what do you think of the above proposed solution, that the startPrivateConversation and related features take a first message parameter that is used to create the conversation ID?

not stale

Is this issue still open? i'm having exactly the same issue in botkit for webexteams.

Issue still exists and developers are still running into it...humbly bumped.

hey @dstaudt thanks for the bump.
I think we should go ahead and implement your suggested fix - require a message to begin the conversation.
I can do this for the next release.

Sounds good, happy to help with testing/validation if useful.

Bump to keep-alive, in hopes of a fix in 4.11

Bump to keep-alive, in hopes of a fix in 4.11

Still present, bump in hopes of a fix

Can anyone share a workaround for the time being?

@seansbox this workaround works for me, replace toPersonEmail with roomId for room messages.
let bot = await controller.spawn();
let message = "The message";
let user_email =<EMAIL_ADDRESS>
bot.api.messages
   .create({  // This already sends the message
       toPersonEmail: user_email,
       markdown: message,
   })
   .then((message) => maybe_do_something_after(message))

What is the current status on this?


Support tracerequest response header to correlate with traces

Description
We want too connect traces with client side activity. Currently we need to start traces and they're very verbose.
As an alternative to client-side tracing, we can use the upcoming OTel RUM events and use more or less flat events.
Proposed solution
Use traceresponse headers to communicate server span context back to client and use Faro to pick it up.
Context

@cedricziel should we pursue this for Q3? or does the OTel RUM event spec change change our plans here?

@eskirk I have linked it to a parent ticket. I dont think the spec changes anything necessarily


issue in protein graph

it cannot use networkx, since nx.set_node_attributes arguments, name and attributes are in inverse position
Another issue is that:
138 def add_edge(G, n1, n2, kind_name):
139     if G.has_edge(n1, n2):
--> 140         G.edges[n1, n2]["kind"].add(kind_name)
141     else:
142         G.add_edge(n1, n2, kind={kind_name})
TypeError: 'method' object is not subscriptable

Hi @etheng-jiao, I suspect this is due to version drift in the dependencies. Could you please share the output of pip list or conda list

Closing due to inactivity. Feel free to reopen.


gateway should implement function close()

now it is :
// Close the gateway connection and all associated resources, including removing listeners attached to networks and
// contracts created by the gateway.
func (gw *Gateway) Close() {
	// future use
}

after connect gateway ,get network and submit Transaction, I cannot break the tcp connection.
gw, err := gateway.Connect(
		gateway.WithConfig(config.FromFile(filepath.Clean(ccpPath))),
		gateway.WithUser(userid),
	)
...
network, err := gw.GetNetwork(channel)
...
connector.UseContract(..., ...)
funcName = "QueryAllRecords"
result, err = connector.Contract.EvaluateTransaction(funcName)

I can read the leadger by doing this ,the var result is expected, but I notice the connection is still active, I cannot close the gateway connection, after  invoking chaincode some times ,the connection is getting more
root@hecs-ubuntu-18:/home/localrootchain/service_proxy# netstat -ap | grep serviceProxy
tcp        0      0 <IP_ADDRESS>:60070      peer0.cn:8050           ESTABLISHED 1249/./serviceProxy
tcp        0      0 <IP_ADDRESS>:34686      peer0.cn:8050           ESTABLISHED 1249/./serviceProxy
tcp        0      0 <IP_ADDRESS>:41730      peer0.cn:7050           ESTABLISHED 1249/./serviceProxy
tcp        0      0 <IP_ADDRESS>:41282      peer0.cn:7050           ESTABLISHED 1249/./serviceProxy
tcp        0      0 <IP_ADDRESS>:41050      peer0.cn:7050           ESTABLISHED 1249/./serviceProxy
tcp        0      0 <IP_ADDRESS>:59808      peer0.cn:8050           ESTABLISHED 1249/./serviceProxy
tcp6       0      0 [::]:8001               [::]:*                  LISTEN      1249/./serviceProxy

more and more memory and cpu is occupied ,and I cannot close the sdk in gateway because it is not leaked, it is in lower case
type Gateway struct {
	sdk        *fabsdk.FabricSDK
	options    *gatewayOptions
	cfg        core.ConfigBackend
	org        string
	mspid      string
	peers      []fab.PeerConfig
	mspfactory api.MSPProviderFactory
}

after some time, the gateway will write log as "error from discovery request...."so many times ,I think it is reported byhttps://github.com/hyperledger/fabric-sdk-go/blob/main/pkg/client/common/discovery/dynamicdiscovery/chservice.go#L120
besides ,I can find many tcp connection and I can hardly query ledger ,peer will response like "context deadline exceeded".
nothing i can do, i just restart my client, and it will work fine for some time ,and the same problem will appear again.
How can I Close the gateway connection and all associated resources?

I was intended to write like
gw, err := gateway.Connect(
		gateway.WithConfig(config.FromFile(filepath.Clean(ccpPath))),
		gateway.WithUser(userid),
	)
defer gw.Close()

but as we know ,it wont be closed, it hasnt been implement ,and nothing will happen


NEW:  generate reverse and clipped trajectories AFTER main trajectory generation

Generate reverse and/or clipped trajectories via commands in Trajectory.  Previously, if reverse, clipped trajectories were wanted they had to be calculated at time of original trajectory generation (pysplit.generate_bulktraj()).   Particularly if only a small portion of generated trajectories are used, this new feature may save a lot of computing time.  Also can fetch names of meteorology files used to calculate trajectories.

Now, can create clipped versions of trajectory files and generate reverse trajectories AFTER main trajectory generation.  Advantageous for when large numbers of trajectories are generated but will not necessarily all be used in analysis- see example .


Is there a public installer for Rhino Inside AutoCAD?

Or is it manually installed via GitHub clone?

Unfortunately there is no installer.
Right now the only way to go is cloning and build.
The project is a bit outdated since it only builds for AutoCAD 2017-2020, but I can help in that area if there is interest.

Hey @kike-garbo we're about to start a large development project for a client using AutoCAD. It's looking highly likely qell use Rhino Inside for it so we'll need RiR running on AutoCAD 2020 ~ 2024 if that's possible?

Let's start updating the project to your needs in order to enable you to develop on top of it.
We can figure out latter the installer.

Sounds good, so long as it works inside AutoCAD, that's the main priority. Project is kicking off within the next 4 weeks so I'll keep this thread updated if we hit any road blocks.


🐞[Bug]: Submit Button

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Submit button is not good it require some padding and some hover effect to change

Expected behavior
i can solve this issue
Add ScreenShots
No response
What browsers are you seeing the problem on?
Chrome
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Anishkagupta04  please assign this issue to me under GSSOC

@varshith257 please assign this issue to me


Fix errors of solution not loading if only one UnitOperation was returned

For high-throughput simulations I removed all returns except for my UO of interest. Then, with the DLL runner I was getting empty solutions. This PR fixes that by ensuring the length of the solution is larger than 0, instead of larger than 1, before adding it to the full solution.
@schmoelder  do you remember if there was a reason for > 1 ?

Puh, good question. It just might well be a brain fart. Do you have a test / MRE for this issue?


Design pattern for Selecting Entrypoints, Builders

Describe your idea/feature/enhancement
Currently AWS SAM CLI uses some direct if elif else branches for determining entrypoints and selecting appropriate builders.
https://github.com/awslabs/aws-sam-cli/blob/develop/samcli/local/docker/lambda_container.py
https://github.com/awslabs/aws-sam-cli/blob/develop/samcli/lib/build/app_builder.py#L48
Proposal
The proposal is to change this to a pattern of encapsulation within dictionaries, and appropriate handlers defined that would compute the necessary state. This makes the code cleaner and less procedural.
Things to consider:

Will this require any updates to the SAM Spec

No

This is partially addressed for selecting workflows :)

This has been addressed.


no preprocessing code in repo

Hoping to get started with some preprocessing steps, perhaps larger scale task scheduler work. I'll need the latest stable version of the preprocessing pipeline.

Hi Jordan,
Thanks for submitting an issue. Please have a look at the two links below.
First, to get an idea of the folder structure for the project, have a look at the directory structure page in our Overview Wiki.
The bash script that I used to run the data for the first time is in the mitolin/src folder. Note that this script is a couple years old. So, I expect a lot will change, especially since it should all be upgraded to gatk4.
Feel free to respond here and/or ping me through Slack as questions arise. (Join the Slack group)

Hi Jordan,
I added a mitolin plan tutorial to the wiki. It outlines the steps that are needed for pre-processing, with links. It's basically my summary of what I got from reading gatk's website.
The tutorial is here: https://github.com/deepcelllineage/overview/wiki/mitolinplan-v0
I'm using the word 'tutorial' loosely ;)