Vue.js tutorial - doesn't do what is described

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x ] Correction or update

url: https://github.com/mdn/sprints/issues/new?template=issue-template.md&projects=mdn/sprints/2&labels=user-report
section "Adding the new todo into our data"
problem:
Up to the end of the previous section everything works fine - as is shown in the console. But when I replace the line "console.log('To-do added:', toDoLabel);" with "this.ToDoItems.push({id:uniqueId('todo-'), label: toDoLabel, done: false});" no item is added to the list. I checked my code many times, but I can't find anything that's different from the code in the tutorial. What went wrong?
ToDo code.zip

Hi @haenf, sorry to hear you are having trouble here.
Can you send me the correct URL of the tutorial part you are having trouble with?
and also send me a zip of your project's full src directory?
From there I should be able to track the error down.
Thanks!

Hi Chris,
The url:
https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Vue_methods_events_models
The part where it doesn't work (for me :- ) anymore: "Adding the new todo into our data". The first code snippet doesn't result in what is described: new ToDo item added to the list.
Thank you in advance,
Hans Nederbragt
enference  infologica  |  06 - 55 378 352  |  www.enference.nl
From: Chris Mills<EMAIL_ADDRESS>Sent: vrijdag 10 juli 2020 11:42
To: mdn/sprints<EMAIL_ADDRESS>Cc: haenf<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mdn/sprints] Vue.js tutorial - doesn't do what is described (#3477)
Hi @haenf https://github.com/haenf , sorry to hear you are having trouble here.
Can you send me the correct URL of the tutorial part you are having trouble with?
and also send me a zip of your project's full src directory?
From there I should be able to track the error down.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mdn/sprints/issues/3477#issuecomment-656586673 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AEGSFF6CA7EQAQMN322HQUTR23O53ANCNFSM4OWKEZVQ .  https://github.com/notifications/beacon/AEGSFF6WBH3FIG73IJAP3A3R23O53A5CNFSM4OWKEZV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE4RLPMI.gif

@haenf I've had a look, and the code in the article matches the code in the final app version (https://mdn.github.io/todo-vue/dist/).
Can you send in a zip of your project's src directory, for me to have a look at?

Hi Chris,
I send you  the src directory already last week, by direct reply to your mail. Didn’t you receive it?
Best,
Hans Nederbragt
enference gsm 06 55 378 352  |  https://enference.nl
From: Chris Mills<EMAIL_ADDRESS>Sent: Wednesday, 15 July, 2020 22:53
To: mdn/sprints<EMAIL_ADDRESS>Cc: haenf<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mdn/sprints] Vue.js tutorial - doesn't do what is described (#3477)
@haenf https://github.com/haenf  I've had a look, and the code in the article matches the code in the final app version (https://mdn.github.io/todo-vue/dist/).
Can you send in a zip of your project's src directory, for me to have a look at?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mdn/sprints/issues/3477#issuecomment-659003472 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AEGSFF73HFFNDA7SZHFBXZLR3YJJLANCNFSM4OWKEZVQ .  https://github.com/notifications/beacon/AEGSFF6YSBO46OUSI7MRTSLR3YJJLA5CNFSM4OWKEZV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE5DZQUA.gif

@haenf hi there! My apologies — I didn't see it the first time round, but I've found it now.
I've tried running your code, and I can see what the problem is. When you submit the form, it refreshes the entire page, and so you don't get the new todo item added — it just goes back to the initial state of the page. To fix this, you need to run preventDefault() in your event handler. This is done in Vue using the prevent event modifier on the submit handler, like this (this is inside ToDoForm.vue):
<form @submit.prevent="onSubmit">

You can read about it in this section: https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Vue_methods_events_models#Creating_a_method_binding_it_to_an_event_with_v-on

Hi Chris,
Yes, that was the thing I had wrong. It is working okay now, thank you for your help. And by the way: I like the tutorial very much, it is very clear! Together with the ‘official’ Vue.js manuals it is the best I have tried so far. When are you going to write a similar tutorial for Vuex ? :- )
Best,
Hans
enference gsm 06 55 378 352  |  https://enference.nl
From: Chris Mills<EMAIL_ADDRESS>Sent: Thursday, 16 July, 2020 12:55
To: mdn/sprints<EMAIL_ADDRESS>Cc: haenf<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mdn/sprints] Vue.js tutorial - doesn't do what is described (#3477)
@haenf https://github.com/haenf  hi there! My apologies — I didn't see it the first time round, but I've found it now.
I've tried running your code, and I can see what the problem is. When you submit the form, it refreshes the entire page, and so you don't get the new todo item added — it just goes back to the initial state of the page. To fix this, you need to run preventDefault() in your event handler. This is done in Vue using the prevent event modifier on the submit handler, like this (this is inside ToDoForm.vue):

You can read about it in this section: https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Vue_methods_events_models#Creating_a_method_binding_it_to_an_event_with_v-on
—
You are receiving this because you were mentioned.
Reply to this email directly,  https://github.com/mdn/sprints/issues/3477#issuecomment-659334807 view it on GitHub, or  https://github.com/notifications/unsubscribe-auth/AEGSFF6JA3VOPIACA7QDKT3R33L7BANCNFSM4OWKEZVQ unsubscribe.  https://github.com/notifications/beacon/AEGSFFYHPZYUM2LPS3WND73R33L7BA5CNFSM4OWKEZV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE5GKNFY.gif

And by the way: I like the tutorial very much, it is very clear! Together with the ‘official’ Vue.js manuals it is the best I have tried so far. When are you going to write a similar tutorial for Vuex ? :- )

I am really glad that you are finding them helpful!
We are currently researching what to do next in terms of improvement to our learning material. I'm not sure that we want to add too much more about specific frameworks, as this is only supposed to be a stepping stone to being comfortable enough with the framework to then go and find mroe information at the vendor's sites. It would be impossible for us to maintain all of the things! But we will certainly consider your feedback here.


Add Bootstrap 3 Pagination Template

Might be useful for bootstrap users.

Thanks, @mohitpanjwani. However, this will not be merged to vuetable-2 as the bundled VuetablePagination is already working with Bootstrap CSS. I see this as a customized component that each user will be used inside his/her own project. But I'll leave it in the PR anyway as other people might find it useful for them and can refer to your repo.
One suggestion though, you should use VuetablePaginationMixin instead of rewrite every methods again.

Current Component does not seem to be working with Bootstrap because Bootstrap requires you to have an Unordered list in markup instead of a parent div with anchored child elements . So this could be useful for users who do not wish to add extra styles specially for this component.
However I will take your suggestions into consideration and refactor the component to use the Mixin instead.
Thanks a lot 👍


Missing verification variant in PageProperty enum

Toying around, I happened upon Error: deserialization error: unknown variant 'verification', expected one of 'button', [...] when creating a page as a child to a wiki page: notion.create_page().page_id("<page in a wiki>").send().await? The page gets created, but the response can't get parsed.
A variant for the verification property is missing in PageProperty. Curiously there is an extra PageProperty::Button variant which doesn't appear in the api, unless the api documentation is just being weird?
The verification property type is documented here: https://developers.notion.com/reference/page-property-values#verification
Not yet familiar with PRs, could help out though.

Thank you for reporting this issue!
The problem arises because the Verification page property is not yet implemented, and this is not due to a discrepancy in the Notion API documentation.
To provide more details: when you use create_page, the page is successfully created, and the API call itself works fine. However, the issue occurs during the deserialization of the API response. Since the Verification variant is not implemented in the current library, deserialization fails, which results in an error even though the page is created successfully.
We will implement support for the Verification property in the next alpha release. Please wait a little longer, and thank you for your patience!

The bug has been fixed in the latest release, v1.0.0-alpha.21. Unit tests and integration tests passed without any issues (the verification property only appears in wiki pages, and this test case was missed). Please verify if it works correctly with the latest version.

Yes, it works as expected now with the new release. Thank you kindly.


GraphQL: isPuid query

Summary
As a User I would like to be able to query an id and determine if it matches the format of a PUID.
Related to DFCT0010065
Acceptance Criteria

Create a new isPuid query the adhere's to the following criteria

Accepts the following arguments:

id which is of type ID

Returns  a Boolean:

true when the id supplied matches the format of a PersisentUniqueID (refer to valid_puid? in lib/irida/persistent_unique_id.rb)

Possible object_class are Sample, Attachment, Group, and Namespace::ProjectNamespace

false if it doesn't match any known PUID types

Fixed in #625


Outsourced Distribution Computation Tutorial

Left a comment. Also, can you resolve the conflict in index.rst?

Done! Thanks for the comments.


Other types of mapping

Hey,
I am working on a project that depends on entities that are provided by third party bundles (in the vendor folder).
Is there any way to create the mappings without using the annotations? Some yml, json or xml mappings that will do the same as the annotations but without having to change the entities?
Thanks!

Only annotations are supported.

Well.. that is unfortunate.. Any plans in the pipeline to use anything else besides annotations?

no because there are no request for this features (excepting you)

Would this be something complicated to add? I can't change the way things are due to legacy issues and those entities I am dependent on being in the vendor folder.

I have never think about it how to implement a new mapping. So I don't know how complicated it would be.
To solve your problem: you can implement a DTO which uses the vendor-entity:
class WrapperDTP
{
    /** @Solr\Field() */
    private $vendorProperty1;

    /** @Solr\Field() */
    private $vendorProperty2;

    public function __construct($vendorEntity)
    {
        $this->vendorProperty1 = $vendorEntity->getProperty1();
        $this->vendorProperty2 = $vendorEntity->getProperty1();
    }
}

I can't see how I am going to load this when I run the populate command.
I see that the entity will be passed in the construct but how do I wire this up? As a service? How will your bundle read this exactly?
Sorry I am just not understanding fully your solution.

You can't use the existing populate command. You have to implement your own. It would be something like this:
// fetch all vendor-entities
foreach ($vendorEntity as $vendorEntity) {
    $dto = new WrapperDTO($vendorEntity);
    $this->get('solr.client')->updateDocument($dto);
}

Great! I was already implementing a "hook" on your populate command so that I could populate solr with the missing entity.
Thanks again for all your help.


Skins again not working?

Fix fast pleaseeee, when i join game and press tab nothing happens, dont even hear sound

still nothing?

Thank you, this worked <3


Leaderboard: submission ranking while multitasking

Dear @Logan-Ruf @Tthomas63 @NehzUx @ckcollab ,
I was wondering how the leaderboard deals with several tasks while rannking the participants.
It is unclear to me in the competition #218 how the submissions are sorted. For instance, the submission ranked [2] displays n/a everywhere...
Here is a screenshot of the leaderboard:

And a screen shot of the corresponding leadreboard set up:

I understand how the submissions are sorted using the 'primary' column, but I don't know how they are sorted in mutlitasks phases.
In the wiki page, the multitask case is not described.
And in this wiki's multitask example, all scores are the same :)
If possible, it would be nice for the user to have the option to select the task used to sort the submissions (for instance using a click button directly on the leaderboard table). Maybe this functionality already exists but I couldn't find it.
What are your thoughts about this?

The primary column is used to rank participants. I've updated the documentation to make it clearer.
Computation columns can be used to average scores over several columns. We plan to implement other ranking functions such as "average rank", etc.

#406


Update broken project links

Guess you've moved the project namespaces recently and forgot to update the template...

thanks!


99% Detection rate on bare metal Windows

Using Acer Laptop running Windows 10 22H2 and I get the following output:
`C:\Users\Lion\Downloads>vm --disable-hyperv-host
[NOT DETECTED] Checking VMID...
[NOT DETECTED] Checking CPU brand...
[NOT DETECTED] Checking CPUID hypervisor bit...
[NOT DETECTED] Checking CPUID 0x4 leaf...
[NOT DETECTED] Checking hypervisor brand...
[NOT DETECTED] Checking RDTSC...
[NOT DETECTED] Checking sidt null byte...
[NOT DETECTED] Checking processor count...
[  DETECTED  ] Checking MAC address...
[NOT DETECTED] Checking temperature...
[NOT DETECTED] Checking systemd virtualisation...
[NOT DETECTED] Checking chassis vendor...
[NOT DETECTED] Checking chassis type...
[NOT DETECTED] Checking Dockerenv...
[NOT DETECTED] Checking dmidecode output...
[NOT DETECTED] Checking dmesg output...
[NOT DETECTED] Checking hwmon presence...
[NOT DETECTED] Checking cursor...
[NOT DETECTED] Checking VMware registry...
[NOT DETECTED] Checking VBox registry...
[NOT DETECTED] Checking users...
[NOT DETECTED] Checking DLLs...
[NOT DETECTED] Checking registry...
[NOT DETECTED] Checking Sunbelt CWSandbox directory...
[NOT DETECTED] Checking Wine...
[NOT DETECTED] Checking VM files...
[NOT DETECTED] Checking hw.model...
[NOT DETECTED] Checking disk size...
[NOT DETECTED] Checking VBox default specs...
[NOT DETECTED] Checking VBox network provider match...
[NOT DETECTED] Checking computer name...
[NOT DETECTED] Checking hostname...
[NOT DETECTED] Checking low memory space...
[NOT DETECTED] Checking VM processes...
[NOT DETECTED] Checking default Linux user/host...
[NOT DETECTED] Checking VBox window class...
[NOT DETECTED] Checking gamarue ransomware technique...
[NOT DETECTED] Checking 0x4 leaf of VMID...
[NOT DETECTED] Checking Parallels techniques...
[NOT DETECTED] Checking RDTSC VMEXIT...
[NOT DETECTED] Checking loaded DLLs...
[NOT DETECTED] Checking QEMU CPU brand...
[NOT DETECTED] Checking BOCHS CPU techniques...
[NOT DETECTED] Checking VirtualPC motherboard...
[NOT DETECTED] Checking BIOS serial number...
[NOT DETECTED] Checking Hyper-V registry...
[NOT DETECTED] Checking Hyper-V WMI output...
[NOT DETECTED] Checking VirtualBox shared folders...
[NOT DETECTED] Checking MSSMBIOS...
[NOT DETECTED] Checking MacOS hw.memsize...
[NOT DETECTED] Checking MacOS registry IO-kit...
[NOT DETECTED] Checking IO registry grep...
[NOT DETECTED] Checking MacOS SIP...
[NOT DETECTED] Checking KVM registries...
[NOT DETECTED] Checking KVM drivers...
[NOT DETECTED] Checking KVM directories...
[NOT DETECTED] Checking HKLM registries...
[NOT DETECTED] Checking Audio device...
[NOT DETECTED] Checking qemu-ga process...
[NOT DETECTED] Checking MSR validity...
[NOT DETECTED] Checking QEMU processes...
[NOT DETECTED] Checking QEMU directories...
[NOT DETECTED] Checking VPC processes...
[NOT DETECTED] Checking VPC invalid instructions...
[NOT DETECTED] Checking SIDT...
[NOT DETECTED] Checking SGDT...
[NOT DETECTED] Checking SLDT...
[NOT DETECTED] Checking Offensive Security SIDT...
[NOT DETECTED] Checking Offensive Security SGDT...
[NOT DETECTED] Checking Offensive Security SLDT...
[NOT DETECTED] Checking VirtualPC SIDT...
[NOT DETECTED] Checking Hyper-V motherboard...
[NOT DETECTED] Checking Extra VM files...
[NOT DETECTED] Checking /proc/iomem file...
[NOT DETECTED] Checking /proc/ioports file...
[NOT DETECTED] Checking /proc/scsi/scsi file...
[NOT DETECTED] Checking VMware dmesg...
[NOT DETECTED] Checking STR instruction...
[NOT DETECTED] Checking VMware IO port backdoor...
[NOT DETECTED] Checking VMware port memory...
[NOT DETECTED] Checking SMSW instruction...
[NOT DETECTED] Checking mutex strings...
[NOT DETECTED] Checking uptime...
[NOT DETECTED] Checking unusual thread count...
[NOT DETECTED] Checking Intel thread count mismatch...
[NOT DETECTED] Checking Intel Xeon thread count mismatch...
[NOT DETECTED] Checking VM memory regions...
[  DETECTED  ] Checking VMware devices...
[NOT DETECTED] Checking Hyper-V CPUID...
[NOT DETECTED] Checking Cuckoo directory...
[NOT DETECTED] Checking Cuckoo pipe...
[NOT DETECTED] Checking Hyper-V Azure hostname...
[NOT DETECTED] Checking general VM hostnames...
[NOT DETECTED] Checking screen resolution...
[NOT DETECTED] Checking bogus device string...
[NOT DETECTED] Checking mouse device...
[NOT DETECTED] Checking BlueStacks folders...
VM brand: Hybrid Analysis
VM likeliness: 99%
VM confirmation: true
VM detections: 2/97
====== CONCLUSION: Very likely a Hybrid Analysis VM ======`
Is there a way we can just have detections base on VM detections only? Seems to be much more reliable in my case.

ok so 2 things:

for the MAC address check, what the technique does is compare your actual MAC address to those of various VM brands. Hybrid Analysis VMs usually have the address of XX:XX:X:0A:00:27, and i'm not sure why you got that result if i'm being honest. It's either an oversight on my part or it was some kind of rare coincidence where your mac address has been set to what Hybrid Analysis would've by sheer chance. This is the first time I received this kind of problem. I'll remove the Hybrid Analysis mac check.

For the VMware devices check, that's a very new technique that's been added right when the 1.6 version of the library was made. I didn't expect all new checks to be perfect but i'll remove this technique in particular seeing that you've received a false positive.

fixed here, let me know if you need anything else.
Also, it would be appreciated if you could test it again at the latest release section

fixed here, let me know if you need anything else.
Also, it would be appreciated if you could test it again at the latest release section, but thanks for reporting this issue, appreciate it!

No detections on baremetal now. Thanks.
After debugging, it seems that my VirtualBox Host-Only Ethernet Adapter matched the MAC address...

fixed here, let me know if you need anything else.
Also, it would be appreciated if you could test it again at the latest release section, but thanks for reporting this issue, appreciate it!

No detections on baremetal now. Thanks.
After debugging, it seems that my VirtualBox Host-Only Ethernet Adapter (on my baremetal machine) matched the MAC address...

Thanks! That’s actually very helpful, I appreciate the effort too :)


prevent unused variable error

A previous commit that changed the ACK pkg/resource templates
introduced an error for controllers that have CREATE or UPDATE
implementations where the Output shape from the operation sets no fields
on the CR.
In these cases, attempting to build these controllers was resulting in a
build failure, like this for the RDS controller:
pkg/resource/db_parameter_group/sdk.go:205:6: resp declared but not used

This corrects that by using a blanking technique immediately after the
variable declaration:
var resp *SomeSDKShape; _ = resp;

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Remove alpha badge of remote-config and remote-update from stable docs and docs-v0.24.x

What this PR does / why we need it:
The status change of those features has already been announced in the latest release but the docs are still not being updated.
https://github.com/pipe-cd/pipecd/releases/tag/v0.24.5
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

nice catch
/approve


Fixed an uninitialized bool in the gallery FAB demo

This was causing the devicelab flutter_gallery_instrumentation_test to fail.

LG


Export a few internal symbols to make lua-sec compile

This patch is applied in the Debian package, could you please merge it?

This is complicated. We should probably change the names of these symbols to something prefixed by "luasocket", "ls", or something else. Otherwise, when linked with other libraries, the chance for collisions is too big. In fact, I recall somebody complaining about collisions before (he was compiling LuaSocket wrong, and all symbols were exported).

this is superseded by #346 , closing this.


fix (ui): add invoice list params

invoice list gen client and add params to admin ui API call.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 9.651%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
10836

Relevant Lines:
112283

💛  - Coveralls


[question] i'm trying to port the image to arm32v7 and it's crashing the host

Hi I know this isn't exactly an issue pertaining your image, but i'm just asking you because you might have a particular insight into the build .
i'm using arm32v7/ubuntu as base which is already locally available (does that make it :ro ?)
i added apt-utils (for debconf bug) and useradd to create and name a user
namely lampuser with uid & gid 1000 .
i did a  chown mysql:mysql /var/lib/mysql .. an issue i came across here on github regarding mounting db data volume ..
and changed php's TZ to CET .
it's the third time this week that it brings down the host and consequently my swarm services ..it's a pain restarting the lot over and over .. it seems like a perl-base upgrade found a read only file system and crashed ..
this is some output
$ docker build -t arm32v7/lamp .

Removing intermediate container 6f808b541cb3
---> 611391c4c734
Step 3/35 : RUN apt-get upgrade -y
---> Running in b92ad08095b9
Reading package lists...
Building dependency tree...
Reading state information...
Calculating upgrade...
The following packages will be upgraded:
apt gpgv libapt-pkg5.0 libgcrypt20 libncurses5 libncursesw5 libtinfo5
ncurses-base ncurses-bin perl-base
10 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Need to get 4082 kB of archives.
After this operation, 10.2 kB of additional disk space will be used.
Get:1 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf ncurses-bin armhf 6.1-1ubuntu1.18.04 [155 kB]
Get:2 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf perl-base armhf 5.26.1-6ubuntu0.1 [1290 kB]
Get:3 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf ncurses-base all 6.1-1ubuntu1.18.04 [17.8 kB]
Get:4 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf libapt-pkg5.0 armhf 1.6.2 [728 kB]
Get:5 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf libgcrypt20 armhf 1.8.1-4ubuntu1.1 [364 kB]
Get:6 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf gpgv armhf 2.2.4-1ubuntu1.1 [167 kB]
Get:7 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf apt armhf 1.6.2 [1121 kB]
Get:8 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf libncurses5 armhf 6.1-1ubuntu1.18.04 [74.8 kB]
Get:9 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf libtinfo5 armhf 6.1-1ubuntu1.18.04 [68.2 kB]
Get:10 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main armhf libncursesw5 armhf 6.1-1ubuntu1.18.04 [95.3 kB]
debconf: delaying package configuration, since apt-utils is not installed
Fetched 4082 kB in 1s (6456 kB/s)
(Reading database ... 4029 files and directories currently installed.)
Preparing to unpack .../ncurses-bin_6.1-1ubuntu1.18.04_armhf.deb ...
Unpacking ncurses-bin (6.1-1ubuntu1.18.04) over (6.1-1ubuntu1) ...
Setting up ncurses-bin (6.1-1ubuntu1.18.04) ...
(Reading database ... 4029 files and directories currently installed.)
Preparing to unpack .../perl-base_5.26.1-6ubuntu0.1_armhf.deb ...
Unpacking perl-base (5.26.1-6ubuntu0.1) over (5.26.1-6) ...
dpkg: error processing archive /var/cache/apt/archives/perl-base_5.26.1-6ubuntu0.1_armhf.deb (--unpack):
unable to make backup link of './usr/lib/arm-linux-gnueabihf/perl-base/unicore/lib/SB/SC.pl' before installing new version: Input/output error
dpkg: error while cleaning up:
unable to remove backup copy of '/usr/lib/arm-linux-gnueabihf/perl-base/unicore/lib/SB/LO.pl': Read-only file system
...... ......
rm: cannot remove '/var/cache/apt/archives/ncurses-bin_6.1-1ubuntu1.18.04_armhf.deb': Read-only file system
rm: cannot remove '/var/cache/apt/archives/perl-base_5.26.1-6ubuntu0.1_armhf.deb': Read-only file system
E: Sub-process /usr/bin/dpkg returned an error code (2)
thanks for any assistance

done fixed


Conditionally show imported need only if linked by another need

Intro
My needs are "distributed" modular over some documents and using/referencing this needs the needimport directive is useful.
Depending on the :hide: option ALL imported needs are shown or not.
If i want complex filtering, some directives do support filtering in the content of the directive but the needimport directive does not.
What i want to achieve
Show only the imported needs, which are linked in this document.
The links should be "useable", so that i can see the content of the linked need
What i have tried

Import the needs with needimport and :hide:

Tried to needextract but get the error : Content for requested need not found. [needs.needextract]

Import the needs with needimport and :hide:

Show the needs in a needtable but cannot follow the link shown in the requirement that references this need (Error ERROR: No anchor #REQ_EXAMPLE for internal URI reference

Import the needs with needimport

extend the needs to "delete" or "hide" them - does not work cause cannot use filter functions or filter content and simple filter does not consider tags.

Possible solution
~needextract should support imported, but not shown needs~
needtable could get an option to provide link targets instead of reference link targets, maybe also needlist
Any help appreciated.
Version used
~3.0.0~
4.1.0

This is related to #722. But the difference is that now NOT all imported/tagged requirements should be shown.
I would like to use it for reference of common requirements, that COULD, but not MUST be implemented.
It would be possible to generate a filter string to only import these requirements, but that would be lead to duplicate maintenance of the issues: in the requirements/tests itselfs and the "import" mechanism.
Therefore a simpler solution should be possible.

Also saw the possibility to export needs by id (separate json file for id). This has the same drawback like the configuration via filter, but may be also helpful to get the feature implemented.

With filtering only the "required" needs could get shown but requires defining the ids as filter.
needtable should get an option to generate a linkable target instead of using a links with an nonexisting reference.

to delete/needs needs after import is not possible via needextend, cause needextend cannot use filters from content or filter_functions which could to complex filtering with links, therefore the simple filters seems not help here.

If I miss something, maybe you can write more explicit what you have and what you want to get.
What i want to achieve, that only the import requirements are shown and linked in the document, that are referenced (by links). This could be done by duplication (filter the needs linked to in the document or import every need by id and adapt the import every time you add or remove links).
What i want here is to import the needs from the json file and then only show the onced referenced by links.

filter during import for referenced needs is not possible (logically, you first must import and then check all needs for links).
show them in a need table is possible cause the content could be filtered by searching for linked issues. But the needs shown here are not linkable in html output, so that the link in the need referencing this need could be clicked, but the need is then not shown
needextend to hide/show needs could not be used, cause the filter in the arguments could not search for links and the directive does not support complex filters by :filter_func: or content filter.
needextract could not be used, filtering works, but the content could not be shown if the "original" content was hidden.

To dynamicall hide or delete the needs with needextend is also not possible cause the delete or hide options do not accept the result of a dynamic filter like this:
in conf.py:
def not_implemented(app, need, needs):
    results = []

    for id in needs.keys():
        n = needs[id]
        if n['type'] == 'req':
            for links_id in n['implements']:
                print(f"found linked id {links_id}")
                results.append(links_id)

    if need['id'] in results:
        print(f"given need {need['id']} is linked!")

    return need['id'] not in results

needs_functions = [not_implemented]

and in rst:
.. needextend:: 'imported' in tags
    :hide: [[not_implemented()]]

nor is filter-func an valid option
.. needextend:: 'imported' in tags
    :hide:
    :filter-func: not_implemented

Solved it by setting style to invisible with needextent and dynamic function. CSS works for HTML and simplepdf builder.


Move code into langchain/src

@sullivan-sean what do you think? The current setup makes the tsconfig "invalid" after yarn build is run, because of the generated top-level entrypoints, if we could scope tsconfig to src/ would be easier?

The generated entrypoints should all be .js or .d.ts, can we just ignore those in the tsconfig? Open to introducing src too but I like having things not too deeply nested.

I tried that, but it didn't fix it

Can we actually move to a monorepo structure with turborepo & pnpm with that I believe we can solve the tree shaking much more smartly for edge code.

We're currently using turborepo and have multiple yarn workspaces in the monorepo as it stands -- is there anything more you'd like to see besides pnpm in place of yarn?

Oh sorry didn't notice the turbo.json as it's a bit of an odd structure right now IMO. perhaps my main point is breaking every service to it's own package.json to be able to isolate it's deps.
so community prs adding new functionality with it's deps won't make pnpm add langchain adding 3 gb node_modules
more modular

Oh yeah! Right now we're including any optional external modules in peerDepedencies with peerDependencyMeta set to optional. This makes sure things like openai aren't installed unless the consuming package installs it alongside langchain.
I don't think langchain should be adding anywhere near 3gb of node_modules (I just installed in an empty project and it's about 3.5 mb)


Add agent pause mode to NavigationServer

Adds agent pause mode to NavigationServer.
Fixes https://github.com/godotengine/godot/issues/78384.
Required so users that do not use NavigationAgent node but the NavigationServer directly can also pause agent processing.
Split from and builds on top of pr https://github.com/godotengine/godot/pull/73658.

Rebased pr and added pause mode for the newer obstacles as well.

Don't merge yet, need to push some smaller fixes.

Thanks!


checkov: 2.0.566 -> 2.0.568

Motivation for this change
Update to latest upstream release 2.0.568
Change log: https://github.com/bridgecrewio/checkov/releases/tag/2.0.568
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release 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.

Result of nixpkgs-review pr 145693 run on x86_64-linux 1

1 package built:

checkov


Only choose subprotocol if subprotocol list is non-empty

Addresses #39 to fix breaking change for websockets in Tornado 5.1
See http://www.tornadoweb.org/en/stable/releases/v5.1.0.html#tornado-websocket

The WebSocketHandler.select_subprotocol method is now called with an empty list instead of a list containing an empty string if no subprotocols were requested by the client.

Thanks for this!
Trying to work out in my head (and failing) if this will work also with tornado<5.1? Not sure if we need to be compatible with 5.0 but we should definitely modify setup.py so that it reflects whether or not this is compatible with 5.0.

This is backwards compatible with tornado 5.0, but it looks like there's still an issue after this - thought I was testing against 5.1 but an install of 5.0 was shadowing this. Getting this exception now:
[E 2018-07-30 10:31:25.835 SingleUserLabApp ioloop:801] Exception in callback functools.partial(<function wrap.<locals>.null_wrapper at 0x7f750ca00048>, <Future finished exception=TypeError("'>' not supported between instances of 'int' and 'NoneType'",)>)
    Traceback (most recent call last):
      File "/opt/conda/lib/python3.6/site-packages/tornado/ioloop.py", line 758, in _run_callback
        ret = callback()
      File "/opt/conda/lib/python3.6/site-packages/tornado/stack_context.py", line 300, in null_wrapper
        return fn(*args, **kwargs)
      File "/opt/conda/lib/python3.6/site-packages/tornado/ioloop.py", line 779, in _discard_future_result
        future.result()
      File "/opt/conda/lib/python3.6/site-packages/tornado/gen.py", line 1141, in run
        yielded = self.gen.throw(*exc_info)
      File "/opt/conda/lib/python3.6/site-packages/tornado/websocket.py", line 888, in _receive_frame_loop
        yield self._receive_frame()
      File "/opt/conda/lib/python3.6/site-packages/tornado/gen.py", line 1133, in run
        value = future.result()
      File "/opt/conda/lib/python3.6/site-packages/tornado/gen.py", line 1147, in run
        yielded = self.gen.send(value)
      File "/opt/conda/lib/python3.6/site-packages/tornado/websocket.py", line 934, in _receive_frame
        if new_len > self.handler.max_message_size:
    TypeError: '>' not supported between instances of 'int' and 'NoneType'

Will look into this.

Looks good to me. Let's see if we can attract the attention of one more person to look this over.
One day we will have automated tests, I promise ;)

Please try to find someone else to look this over; I'd love it to be merged.

Thanks for helping out, and it has been merged :)

Thanks!
The combination of nbserverproxy + jupyterhub + dask-kubernetes has been very useful to our setup :)


hyperrogue: 12.0u -> 12.1a

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/zenorogue/hyperrogue/releases.
meta.description for hyperrogue is: "A roguelike game set in hyperbolic geometry"
meta.homepage for hyperrogue is: "http://www.roguetemple.com/z/hyper/"
meta.changelog for hyperrogue is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 12.1 with grep in /nix/store/7m30a41plj7mvsl83dhq4fnbcapa7g5a-hyperrogue-12.1
found 12.1 in filename of file in /nix/store/7m30a41plj7mvsl83dhq4fnbcapa7g5a-hyperrogue-12.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
hyperrogue

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7m30a41plj7mvsl83dhq4fnbcapa7g5a-hyperrogue-12.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A hyperrogue https://github.com/r-ryantm/nixpkgs/archive/28ce3ae3935d772686b46b6e5dd2a0768c63a22a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7m30a41plj7mvsl83dhq4fnbcapa7g5a-hyperrogue-12.1
ls -la /nix/store/7m30a41plj7mvsl83dhq4fnbcapa7g5a-hyperrogue-12.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

hyperrogue

Maintainer pings
cc @rardiol for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/zenorogue/hyperrogue/releases.
meta.description for hyperrogue is: "A roguelike game set in hyperbolic geometry"
meta.homepage for hyperrogue is: "http://www.roguetemple.com/z/hyper/"
meta.changelog for hyperrogue is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 12.1a with grep in /nix/store/jzqm195vm9ppjg2qa20bwgaviris28bs-hyperrogue-12.1a
found 12.1a in filename of file in /nix/store/jzqm195vm9ppjg2qa20bwgaviris28bs-hyperrogue-12.1a

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
hyperrogue

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jzqm195vm9ppjg2qa20bwgaviris28bs-hyperrogue-12.1a \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A hyperrogue https://github.com/r-ryantm/nixpkgs/archive/0502d6ef51a9b222a59d89535914e0559ead3e7a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jzqm195vm9ppjg2qa20bwgaviris28bs-hyperrogue-12.1a
ls -la /nix/store/jzqm195vm9ppjg2qa20bwgaviris28bs-hyperrogue-12.1a/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

hyperrogue

Maintainer pings
cc @rardiol for testing.

Result of nixpkgs-review pr 191516 run on x86_64-linux 1

1 package built:

hyperrogue


Minimum .NET 8 SDK? Source generators does not seem to work with .NET 8.0.1x SDK versions

Hello
Can you please mention somewhere (like in the README) what minimum SDK version is required.
I'm trying this library out following the instructions here, but the source generators do not run at all using .NET 8 SDK 8.0.110 which is kind of a bummer because I'm using VS 2022 17.10 LTSC which doesn't use the latest SDK version (which I think is 8.0.4x)

I suspect you might have hit this issue https://github.com/eiriktsarpalis/typeshape-csharp/issues/18. I'll try to push a fix but in the meantime you could try using the latest VS 2022 Preview which is known to work.

Can you try again? I just pushed version 0.12.1 which should work with any .NET 8 SDK or Visual Studio version above 17.8.

Thanks, it works fine with 17.10


Maven pom packaging is set to bundle

Hello,
The released artifact's pom.xml file defines packaging as bundle. Usually the maven packaging values are pom, jar, maven-plugin, ejb, war, ear and rar.
We are creating simple maven model to obtain dependency information, but bundle packaging throws an exception as unrecognized packaging value.
Would it be possible to change packaging to one of the supported ones? Or what are the main reasons to use bundle packaging?
Thanks a lot in advance!

@morazow afaik it is because the publish artifact is a OSGi bundle, I am a bit surprise that the exception is thrown as unrecognized packaging value.  Some explanation about 'bundle' packaging type here:
https://stackoverflow.com/questions/5389691/what-is-the-meaning-of-type-bundle-in-a-maven-dependency

Hello @cheeseng,
Thanks for the update! We fixed the issue in maven pom dependency parser. I am going to close this issue.
Best


Remove redirect

Expected Behavior
Just load the page.
Actual Behavior
Right now, http://opensource.adobe.com/spectrum-css/ redirects to the latest version in a separate directory.
Potential solution
Generate the current version in the base directory and in the versioned directory.

This was closed by the new version of the site.


Added a command that prints installed KPOps version

Closes #184
The ideas were taken from:

_version.py file -> https://stackoverflow.com/questions/458550/standard-way-to-embed-version-into-python-package
Handling the version command -> https://github.com/tiangolo/typer/issues/52

Should be integrated in conjunction with the PR that changes bump action to bump the version in the new _version.py file (will paste it below in a moment)
[edit] the related ci-templates pull request can be found here https://github.com/bakdata/ci-templates/pull/95

@szymonsadowski3 Thanks for your contribution! It seems that your formatting is not correct, and it breaks the CI pipeline. Could you please install the pre-commit hook for the formatting?
https://github.com/bakdata/kpops/blob/main/.pre-commit-config.yaml

@szymonsadowski3 Thanks for your contribution! It seems that your formatting is not correct, and it breaks the CI pipeline. Could you please install the pre-commit hook for the formatting? https://github.com/bakdata/kpops/blob/main/.pre-commit-config.yaml

I just ran the pre-commit hooks and commited the changes accordingly :)

Thanks! I created an issue for that. We need a guideline on how externals should contribute to KPOps. https://github.com/bakdata/kpops/issues/201

during testing I've noticed that in local development environments reading the version dynamically from the package using importlib doesn't work, i.e. when I would run
kpops --version
poetry version patch
kpops --version

it would both times print the same version.
I've switched to the poetry bumpversion plugin and not it works as expected.


Ansible playbook field name is ambiguous in Add hook modal

When the Ansible playbook definition is selected but the playbook is not defined, the warning says, "Hook definition is a required field."
It would be clearer if this warning said "Ansible playbook is a required field" because the field is for the playbook, not the hook definition.

On second thought, I see that this field is used for both Hook definition options, so it might as well stay the way it is. Closing this issue.


Write breadboard prototype section of report

I wrote the breadboard prototype section of the report
Breadboard prototype and testing
The breadboard prototype was the first time the team combined the different subsystems together. This was done after each individual part was tested separately and confirmed to work exactly as expected. Because of this, there were already individual breadboards with each of the subsystems, so they were easily wired together to have a common ground and power. After this, the team combined all the designated GPIO pins and connected the ADC channels to the components. An image of the combined breadboards is shown below.
Figure 14: Breadboard Prototype
The microphone circuit breadboard is seen on the left of figure 14. The stereo decoder is on the right. The heart rate sensor and LED bar are in the middle. To test this prototype, demo.py was used to test each individual part.
First, the microphone was tested by calculating peak-to-peak value over a ten-second interval. The test would print out whether the voltage was above or below the threshold during this interval. This is supposed to test and detect whether a baby is awake or not. This test revealed some issues with the microphone baby detection. The main issue was that there were instances when the voltage was above the threshold despite there just being ambient noise in the room. The way we resolved this was by finding a more precise peak-peak value. However, if time would have allowed, we could have either added a voltage divider to improve the ADC resolution or analyzed the frequency at which a baby cries instead of the peak-to-peak value. After the microphone was tested, the stereo decoder proceeded. The stereo decoder was set to play the music, pause, and then play it again. After this was done, the volume was increased, decreased, and then the music was stopped. These tests were done to make sure all the functionalities of the stereo decoder still worked. This proved successful. The OLED was tested next in demo.py, the clock was first displayed to show the time. After this, random text was sent to the OLED. The OLED was then cleared. These tests proved that the OLED was functional and wired correctly. It also gave insight into the character limit for the screen. The heart rate sensor was up next. In demo.py, the test required the user to put their finger on the sensor. It called the get HR function and returned the user’s BPM and SPO2. This proved to be successful for what was expected of this sensor, as the readings have always been a bit skewed in value from individually testing the heart rate sensor and its abilities. Finally, the LED bar was tested with demo.py. This testing included running the breathe_in() and breathe_out() functions. The test revealed that two of the GPIO pins were switched by accident. A quick edit fixed the issue and the LED bar was successful in its guided breathing effort.
Overall, the breadboard prototype was successful and helped the team find what worked and what didn’t work together. Aside from a few microphone problems that were later fixed, the prototype was ready to be soldered onto the perfboard.


[MINOR] Moving some scala tests in hudi-spark to functional

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure

@hudi-bot run azure

Closing this since much has happened after.
@nsivabalan @xushiyan feel free to reopen pursue again ifyou think we need this


Optionally disable parallelism

Is there any way to optionally disable parallelism between specific tests? In my project I have some complex tests that use global state and therefor can't be run in parallel. One solution I have found is to move them into their own integration test files, but I'm trying to keep like-tests in the same files.

I see three options:

You can limit the amount of parallel tests: cargo test -- --test-threads 1
You can limit the mock server pool size using the HTTPMOCK_MAX_SERVERS environment variable (its somewhat hidden in the docs: https://docs.rs/httpmock/0.4.5/httpmock/?search=#parallelism). Be aware of the following though: If you set HTTPMOCK_MAX_SERVERS=1and one of your tests requires more than one MockServer, this test will be blocked forever waiting for a second MockServer to become available.
Using the serial_test crate (https://crates.io/crates/serial_test)

Thanks for the quick response!
Your first option works, but I don't want to slow down all tests -- most tests can run in parallel and it greatly speeds up all tests. Plus, I don't want tests to fail for someone that doesn't know this is required.
Your second option doesn't work for the reason you state: some of my tests require more than one MockServer.
Your third option solves my problem perfectly. I've added this as a dev-dependency. Thanks again!


Question about Ouster timestamp and GT pose timestamp

@Qingq-li
Thanks for your great work!
I think my question is similar to #7.
In my case, I want to execute the SLAM algorithm with Ouster0_128.
When I try to visualize my own trajectory with evo, I found that Ouster's timestamp and GT timestamp is absolutely different.
I used the Indoor02(medium) dataset.
I checked the Ouster timestamp with rostopic echo /os_cloud_node/points --noarr, I got the result below.
---
header:
  seq: 6067
  stamp:
    secs: 764
    nsecs:  38519990
  frame_id: "os_sensor"
height: 128
width: 2048
fields: "<array type: sensor_msgs/PointField, length: 9>"
is_bigendian: False
point_step: 48
row_step: 98304
data: "<array type: uint8, length: 12582912>"
is_dense: True
---

And I check the GT pose timestamp with rostopic echo /vrpn_client_node/UWBTest/pose, I got the result below.
---
header:
  seq: 11362
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 389417356
  frame_id: "world"
pose:
  position:
    x: 0.995311319828
    y: 1.77796709538
    z: 1.33014786243
  orientation:
    x: -0.528763532639
    y: 0.47708722949
    z: 0.639543950558
    w: -0.289448618889
---

The above results were captured at a similar time frame.
But, Their timestamps are absolutely different.
Is there anything I'm missing? I'd appreciate your help.
Thanks,

@Qingq-li Thanks for your great work! I think my question is similar to #7.
In my case, I want to execute the SLAM algorithm with Ouster0_128. When I try to visualize my own trajectory with evo, I found that Ouster's timestamp and GT timestamp is absolutely different.
I used the Indoor02(medium) dataset. I checked the Ouster timestamp with rostopic echo /os_cloud_node/points --noarr, I got the result below.
---
header:
  seq: 6067
  stamp:
    secs: 764
    nsecs:  38519990
  frame_id: "os_sensor"
height: 128
width: 2048
fields: "<array type: sensor_msgs/PointField, length: 9>"
is_bigendian: False
point_step: 48
row_step: 98304
data: "<array type: uint8, length: 12582912>"
is_dense: True
---

And I check the GT pose timestamp with rostopic echo /vrpn_client_node/UWBTest/pose, I got the result below.
---
header:
  seq: 11362
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 389417356
  frame_id: "world"
pose:
  position:
    x: 0.995311319828
    y: 1.77796709538
    z: 1.33014786243
  orientation:
    x: -0.528763532639
    y: 0.47708722949
    z: 0.639543950558
    w: -0.289448618889
---

The above results were captured at a similar time frame. But, Their timestamps are absolutely different.
Is there anything I'm missing? I'd appreciate your help.
Thanks,

Thanks for your interest in our work.
The  Ouster Lidar and LIVOX-ROS-Driver provided by the factory (https://github.com/Livox-SDK/livox_ros_driver) didn't use the ROS system timestamp. Each lidar messages kept the original timestamp which was the absolute time since lidar power-on during all the experiments. Please check the related discussion here (Livox-SDK/livox_ros_driver#2).
To compare the trajectory, you can utilize the ros::Time::now() as the timestamp for both the lidar and Optirack messages. The time offset between lidar odometry and Optitrack can be calculated by methods like (https://github.com/ethz-asl/lidar_align).
We are preparing a new dataset where the timestamps between all sensors are synchronized with precise timestamp protocol (PTP). The new dataset will be released in few weeks, but you can test it with this link (https://utufi.sharepoint.com/:u:/s/msteams_0ed7e9/EUGHyHbS4BJPn0buhIy_q_8B_LJU2Egv1Q0E2B0q6S1vyw?e=UFS9sP)

@Qingq-li
Thanks for your reply!
First of all, I'll test it with the given link. :smile:
I'll look forward to the Dataset that will be released soon.
Thanks,


Create landing page in React

The landing page needs to be created in React as per the design laid out (right now just in Discord, we should add these design docs in a directory in this repo somewhere perhaps)

Use the assets in /design created by @BrianHHough.

Anyone working on this? Also, in React or React Native?

Hey Roy! I had created this issue a while back, when we were first laying the groundwork for this product. As it stands now, the landing page is going to be made up of a bunch of components we've made / are making based off of the mockups/wireframes by @BrianHHough in /design/wireframes/corona_tracker_landing_page.png.
It looks like we're getting close to having all the components in place to start putting together this landing page, but there's still things that have yet to be done. I've been focusing pretty heavily on getting up and running with Blockstack on the backend/database side of things, but I know @tesla809 has been doing a great job keeping track of what's being worked on / where we're at on the frontend. There's definitely work to be done, but it might be better to break out a more refined issue and get going on that.
Thoughts?
Also, feel free to join the Discord in the README if you haven't already!

@roychowd check out #16 and its sub-issues. A great place to start that needs some help.

Closing this, as "Landing Page" is a little too vague and I think we have more issues surrounding this that make more sense.

Agreed


Sema: Fix soundness hole with variable initializers and opaque return types

Just because the type of the initializer expression is an opaque return type, does not mean it is the opaque return type for the variable being initialized.
It looks like there is a bit of duplicated logic and layering violations going on so I only fixed one caller of openOpaqueType(). This addresses the test case in the issue. For the remaining calls I added FIXMEs to investigate what is going on.
Fixes https://github.com/swiftlang/swift/issues/73245.
Fixes rdar://127180656.

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please smoke test

@swift-ci Please test source compatibility

https://github.com/swiftlang/swift-docc/pull/1116/
@swift-ci Please smoke test

@swift-ci Please smoke test


Colorize lane lines and path

Taken from stock_additions 082 and modified to work with SA-Master

Thanks for the PR!


Missing demopath from the repo

there is no directory named demopath which contains the 3d model of one piece luffy hat or any other models like wolverine face

https://linux.die.net/man/1/find


[BUG] some items in deviceProperties do not have value

What happened:
when checking status of device with kubectl get device xxx -o yaml, some of the items in status.deviceProperties do not have values while the response of EdgeX API contains value. Here is and example below:

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Just create a device and check the status.
Anything else we need to know?:
Environment:

OpenYurt version:  v0.5.0
Kubernetes version (use kubectl version): 1.18.8

others
/kind bug

Thanks for raising this bug. Are you willing to be responsible for fixing this bug? @wawlian

/assign @wawlian


Feature: spacebar to select file

What feature or improvement do you think would benefit Files?
We can chech multiplefiles with checkboxes. How about checking the with spacebar?
Requirements

This proposal will accomplish X
This proposal will accomplish Y
This proposal will accomplish Z

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.1413
Comments
No response

This is currently available in the preview version and will be included in the next stable release.


fix(typescript): cannot find module errors during compile

In the dark of the night, build-index-remove tags in exports.js, run deep into the woods and got lost. I found them and followed them safe back.

Coverage remained the same at 95.154% when pulling 35b5502b1571cd36c46c8be665ae3933682e7ffe on Ullfis:ts-definition-build-index-exports into b80ba34992442bd5ef86c21a39b30777eeef52f1 on aurelia-ui-toolkits:master.

You're very brave to venture out into the woods at night only to bring back orphans.. 😉
Does this not work?
https://github.com/aurelia-ui-toolkits/aurelia-materialize-bridge/blob/master/src/index.js#L26-L28

All contents from files are added to a stream. Then the content is transformed here. At that time both content from index and exports is present. Index has its tags. Exports not.
Source of scary things with many errors during compile:
https://github.com/aurelia-ui-toolkits/aurelia-materialize-bridge/blob/master/dist/aurelia-materialize-bridge.d.ts#L103-L161


Running the job in a different process

The FAQ says that it's not possible to run jobs in a different process, and recommends delegating to a service that runs in that different process. There are two problems with that.

Starting with Oreo, startService while in the background will fail. https://developer.android.com/about/versions/oreo/background
Unless I'm mistaken, you can actually run jobs in a process other than the app's main process. You just have to register android-job's services and broadcasts listeners with android:process=":another_process". What you can't do is run some jobs in one process and other jobs in a different process. Am I correct?

The problem is that there will be race conditions. Another technical limitation is that you can't run some jobs in process A and some in process B. The library already defines its services and you can't change that at runtime. However, you could change all services to run in another process and only use JobManager in this other process. This should work.

Closing because of inactivity.


Use code and headers in make_response()

I don't see any obvious reasons why this wouldn't work? (and it does for me at least, but that's using it in combination with Flask JSON as well)

thank @bivald, could you create a PR for develop branch instead?

I've cherry-picked your changes here https://github.com/teracyhq/flask-classy/commit/c33f035cbd084bccdc9dc9efc6e049b4943a45cb

Awesome!


add paraibanes with 3 dialects

Pre-Submission Checklist

[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command yarn test and yarn lint.

Description
Add paraibanes with dialects related with that new region (Paraíba).

@natarajanrodrigues nice job, thanks to help ups


Favicons updated for most Informa sites

https://www.pivotaltracker.com/story/show/171526948
Updated the favicon icons as allowable by the images provided by Alex. Unable to replace the android-chome icons at 192x192 or 512x512, as the images provided were all around 105x105. Alex has been informed.
Was able to create a 96x96 android chrome... wondering if I should remove that from the files, or update the site.webmanifest to reflect the size for the update icon? Decided it would be best at this time to add the file and leave the old icons as is, until a larger file is provided to replace them.

@Alixers505 once you get the high-res versions that can support 192x192 and 512x512, please include those and then open this. :)

@Alixers505 once you get the high-res versions that can support 192x192 and 512x512, please include those and then open this. :)

I'll let Alex know.

@Alixers505 closing due to age. Feel free to re-open if/when it's ready. :)


feat: Add page view instrumentation plugin

Part of the client sdk project, Implement PageView event instrumentation
#2372
This PR adds the page view instrumentation which sends an event when a page is loaded (as soon as the html is loaded) or when a route change happens (tracking the history push and replace states)
Tested on an example app with generic routes that use the history api

@Abinet18 I have approved the PR, but forgot to mention that also these files should be updated:

.github/component-label-map.yml
.github/component_owners.yml

@martinkuba , when can this be merged. I would fix the conflict just before the merge time so that I don't have to do it every time the main branch changes


compilerOptions.paths to support relative and absolute paths

TypeScript Version:  2.0.2
Code
{
  "compilerOptions": {
    "paths": {
      "./moment": ["node_modules/moment/moment.d.ts"],
      "/foo/bar/baz": [ "some/other/path/for/compiler"]
    }
  },

Expected behavior:
I expect (need) to be able to redirect the compiler for any module, not just 'non relative' paths. I'm deploying to a runtime environment that requires me to use relative or absolute paths, but I need the compiler to look to different locations when building locally.
Actual behavior:
The compiler seems to ignore any compilerOption.paths setting which starts with / or ./

It is important to the compiler to know the relationship between files when it is emitting with --outFile. relative names are generally treated as the file next to you. most loaders i am aware of behave the same way as well. only non-relative names that can be redirected.

can you elaborate on your use case?

Related to #10690 in the general goal of getting TypeScript working for devs compiling locally and still result in compatible ES5 code in our target runtime (AMD-ish) environment.
At its core I think the ask is further decoupling how the compiler finds types from where the implementation files live at runtime. The paths option currently does that for non-relative modules, if I understand it correctly?  I accept that existing loaders take relative paths literally, but in my naive view extending the paths option in this way would allow much more flexibility between compile time file layout and deployment layouts?
In our environment deployment is typically laid out similar to the following with all files under some parent folder (e.g. /FOO/BAR)
LIBA-1.0/
         lib.js
LIBA-2.0/
        lib.js
LIBFoo-1.1/
       lib.js
MainScriptFoo.js
MainScriptBar.js

Here, LIBA has two versions, 1.0 and 2.0, and MainScriptFoo and MainScriptBar can be considered separate applications altogether.
Any of the MainXXX scripts may depend upon either of the versions of LIBA. Since each main script is independent, I wouldn't mind using absolute paths to the LIB/ folders - I can guarantee their location.
If there's a better way that this can be done without adding a lot of extra complexity I'd love to know -there is often no local build step required - folks just author Main scripts in JS and push them out to this virtual filesystem. If those files refer correctly (either absolute or relative path) then it all works without an additional layer of mapping.
Maybe I should file another issue but I noticed absolute paths seemed to want to resolve relative not from my baseUrl or the project root (where the tsconfig file is located) but rather from my actual disk root directory. So my attempts at using absolute references (e.g. /FOO/BAR/LIBA-1.0) failed as well.


feat: add multi term support, add cli

ahh sorry.. fork was bad idea ._.

No worries, glad this hacky code is useful to someone else :).


fatal error: value failed to bridge from Swift type to a Objective-C type

I get a crash with the following line of code from ChartXAxisRenderer.swift:
let labelSize = widthText.sizeWithAttributes([NSFontAttributeName: xAxis.labelFont])
The crash is:
fatal error: value failed to bridge from Swift type to a Objective-C type
When I po xAxis.labelFont the output is <uninitialized>.
This crashes only when I use a custom font like so:
xAxis.labelFont = [UIFont fontWithName:@"SourceSansPro" size:12];
Using the system font does not crash.

Turns out the font name was bad. Perhaps Charts could handle an invalid font more gracefully? Fall back to a default font or throw an exception?


Multilingual db update

Reviewer: @amiraliemami
Estimate: 1h

Ticket
Fixes: AAQ-382
Description, Motivation and Context
Goal
This PR is the first official PR as part of the multilingual support. The goal is to add the language table as well as the routes to be able to run CRUD operations.
Changes
Added a  language submodule with the following files:

schemas
routers
models
Added tests for languages
Also, since we are not planning on building a frontend for the languages yet, added english as the default language in the migration script.

How has this been tested?
This has been tested by manually testing the endpoints one by one and updating the tests.
Checklist
Fill with x for completed. Delete any lines that are not relevant

[x] My code follows the style guidelines of this project
[x] I have reviewed my own code to ensure good quality
[x] I have tested the functionality of my code to ensure it works as intended
[x] I have resolved merge conflicts
[x] I have updated the automated tests (if applicable)
[ ] I have updated the requirements (if applicable)
[] I have updated the README file (if applicable)
[ ] I have updated affected documentation (if applicable)

Thanks for this - really nice code and logic and tests!
Two points re. default language:

Check out the comment on the "edit_language" router - I think that should be fixed to avoid causing a situation where there is no default language.
Maybe we also add some logic onto the "create_language" endpoint to ensure there is always a default language?

About checking for a default language, we are already adding a default language in the migrations, so it shouldn't be an issue. So I suggest we keep it like this now and when we will build the language for frontend we will add the logic, by for example creating the language at launch if there is no language in the DB.


Hide menu bar?

Is there a way to hide menu bar? This with "Edit", "View", "Window" and "Help".

I think this can be coupled together with #35 as a command line integration by setting the autoHideMenuBar option of browserWindow

If we start adding command line options we might do a mess as there'll be a lot of options to include https://github.com/atom/electron/blob/master/docs/api/browser-window.md. What about a json file with all the options availables?

Thank you @fearenales! The solution https://github.com/jiahaog/nativefier/pull/58 is working.

Good to know @kstawiski ! Thank your for testing that :)
Anyway, I'd let the issue open until the PR is fully merged into master, don't you think?


Signing transactions

@janek26 Feel free to add more context to this use case.
https://github.com/authereum/starkware-monorepo

Interesting notes from @martriay of @openzeppelin
Python implementation
https://github.com/OpenZeppelin/cairo-contracts/blob/main/test/Account.py
https://github.com/OpenZeppelin/cairo-contracts/blob/main/test/utils/Signer.py
Smart contract account abstraction:
https://github.com/OpenZeppelin/cairo-contracts/blob/main/contracts/Account.cairo
https://github.com/OpenZeppelin/cairo-contracts/blob/main/contracts/IAccount.cairo


French Translation of Night Witches

Changes / Comments

Translation in French of each lines based on licensed work of Edge
There is some mistakes on the original sheet. Like a double use of "regimental" id who define Regimental Honors title AND Regimental Commander title. The text below Command Staff is also a wrong copy of Role's legend, there is no text here on officials character sheets.

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

After reading the sheet code, I realized the core work isn't done. Night Witches is a PBTA and we can't get the choice on the archetype played (like Monsterhearts if you're looking to compare). So the changes between archetypes can't be made (like Hawk who get only 4 slots for Deepening Ties when Pigeon got 5).
In actual state, the sheet is unusable. It need at least some work to make it usable (adding slots and every marks). In this way, the sheet would work with a few of work by the GM. The perfect solution would be to do something like https://github.com/Roll20/roll20-character-sheets/tree/master/Monsterhearts-2.


Missing french translation

@al3xisb Could you sign the CLA?

/lgtm

@al3xisb Why did you closed it?

Reopen PR

/approve

/retest


Add tutorial for source code change

Codecov Report

Merging #25 (6efdc7c) into main (a15952b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   50.00%   50.00%
=======================================
  Files           1        1
  Lines           4        4
=======================================
  Hits            2        2
  Misses          2        2

Continue to review full report at Codecov.

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


HttpContext.Connection.RemoteIpAddress is null when using Nginx

We have a website developed by asp.net core rc2。We publish the website to linux server, use nginx as reverse proxy. We have added "proxy_set_header X-Forwarded-For $remote_addr;" config.The problem is HttpContext.Connection.RemoteIpAddress is null, LocalIpAddress,LocalPort is also null, we cannot get the value of these properties.
Nginx config:
server{
    listen 80;
    server_name  localhost;

    location / {
        proxy_pass http://unix:/var/aspnet/kestrel.sock;
        proxy_http_version 1.1;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection keep-alive;
        proxy_set_header Host $host:80;
        proxy_set_header X-Forwarded-For $remote_addr;
        proxy_set_header X-Original-For $server_addr;
        proxy_cache_bypass $http_upgrade;
    }
}

Program.cs(simplified)
public class Program
{
        public static void Main(string[] args)
        {
            IConfiguration configuration = new ConfigurationBuilder().SetBasePath(Directory.GetCurrentDirectory()).AddJsonFile("host.json").Build();
            var host = new WebHostBuilder()
                .UseConfiguration(configuration)
                .UseKestrel()
                .UseContentRoot(Directory.GetCurrentDirectory())
                .UseIISIntegration()
                .UseStartup<Startup>()
                .Build();

            host.Run();
        }
 }

Startup.cs(simplified)
public class Startup
{
        public void ConfigureServices(IServiceCollection services)
        {
        }

        public void Configure(IApplicationBuilder app, ILoggerFactory loggerFactory)
        {
            app.Run(async (context) =>
                {
                    try
                    {
                        StringBuilder requestBuilder = new StringBuilder();
                        var remoteIp = context.Connection.RemoteIpAddress;
                        string remoteIpstring = remoteIp != null ? remoteIp.ToString() : "-";
                        requestBuilder.Append(string.Format("remoteIp={0};", remoteIpstring));

                        var serverIp = context.Connection.LocalIpAddress;
                        string serverIpstring = serverIp != null ? serverIp.ToString() : "-";
                        requestBuilder.Append(string.Format("serverIp={0};", serverIpstring));

                        string serverPort = context.Connection.LocalPort.ToString();
                        requestBuilder.Append(string.Format("serverPort={0};", serverPort));

                        await context.Response.WriteAsync(requestBuilder.ToString());
                    }
                    catch (Exception e)
                    {
                        Console.WriteLine(e);
                    }
                });
        }
 }

OS：Centos 7.1
Nginx version：1.10.1
.NET Core version：RC2

@hamu01 I think you have to add the ForwardedHeadersMiddleware  to the pipeline.
   app.UseForwardedHeaders();

A) You need to specify ForwardedHeadersOptions.ForwardedHeaders = ForwardedHeaders.XForwardedFor in UseForwardedHeaders
B) Please show the headers so we can see if something is wrong.
C) We recommend updating from RC2 to RTM

@Tratcher Thx, it works now.
But the forward configuration really confuse me. I have following questions need to be answered:

If I use IIS as reverse proxy, asp.net core rc2 website do not need add the forward middleware to the pipeline. In that situation HttpContext.Connection.RemoteIpAddress is not null, I still can get the remoteip value. Why?It seems the behavior in IIS and Nginx is inconsistent
The context.Connection.LocalIpAddress and LocalPort is also inconsistent when use IIS as reverse proxy and when only use kestrel server, donot use the IIS as reverse proxy. If the asp.net core rc2 website get the real remoteip in IIS, then it also should get the real serverip, I think.
Only specify ForwardedHeadersOptions.ForwardedHeaders = ForwardedHeaders.XForwardedFor works, if specify ForwardedHeaders.All, HttpContext.Connection.RemoteIpAddress is still null, I think when specify ForwardedHeaders.All, it means forward the x-forward-for,x-forward-host and x-forward-protocol, so HttpContext.Connection.RemoteIpAddress shouldnot be null. The real case seems to forward one of them, not all of them.

(If any of the question fixed in RTM version, please ignore it, I have not upgraded to RTM version)
Bellow is the demo code and output：

Startup.cs(simplified)，display the HttpContext.Connection's property and all the headers

public class Startup
{
        public void ConfigureServices(IServiceCollection services)
        {
        }

        public void Configure(IApplicationBuilder app, ILoggerFactory loggerFactory)
        {
            //ForwardedHeadersOptions options = new ForwardedHeadersOptions();
            //options.ForwardedHeaders = ForwardedHeaders.XForwardedFor;
            //app.UseForwardedHeaders(options);
            app.Run(async (context) =>
                {
                    try
                    {
                        StringBuilder requestBuilder = new StringBuilder();
                        var remoteIp = context.Connection.RemoteIpAddress;
                        string remoteIpstring = remoteIp != null ? remoteIp.ToString() : "-";
                        requestBuilder.Append(string.Format("remoteIp={0};", remoteIpstring));

                        string remotePort = context.Connection.RemotePort.ToString();
                        requestBuilder.Append(string.Format("remotePort={0};", remotePort));

                        var serverIp = context.Connection.LocalIpAddress;
                        string serverIpstring = serverIp != null ? serverIp.ToString() : "-";
                        requestBuilder.Append(string.Format("serverIp={0};", serverIpstring));

                        string serverPort = context.Connection.LocalPort.ToString();
                        requestBuilder.Append(string.Format("serverPort={0};", serverPort));

                        foreach (string key in context.Request.Headers.Keys)
                        {
                            string value = context.Request.Headers[key];
                            requestBuilder.Append(string.Format("{0}={1};", key, value));
                        }

                        await context.Response.WriteAsync(requestBuilder.ToString());
                    }
                    catch (Exception e)
                    {
                        Console.WriteLine(e);
                    }
                });
        }
    }

use IIS, the output is:

remoteIp=<IP_ADDRESS>;remotePort=62242;serverIp=<IP_ADDRESS>;serverPort=30791;Cache-Control=no-cache;Connection=Keep-Alive;Accept=text/html, application/xhtml+xml, */*;Accept-Encoding=gzip, deflate;Accept-Language=zh-CN;Host=<IP_ADDRESS>:6000;User-Agent=Mozilla/5.0 (Windows NT 6.1; WOW64; Trident/7.0; rv:11.0) like Gecko;MS-ASPNETCORE-TOKEN=7c45ff68-2c0d-499e-bb53-f89ab8c33554;X-Original-Proto=http;X-Original-For=<IP_ADDRESS>:51137;

You can see that I can get the context.Connection.RemoteIpAddress value, but I donot add the forwardmiddle to the pipeline.
3.not use IIS, only use the kestrel server,listening on the 5000 port, the output is:
remoteIp=<IP_ADDRESS>;remotePort=63149;serverIp=<IP_ADDRESS>;serverPort=5000;Cache-Control=no-cache;Connection=Keep-Alive;Accept=text/html, application/xhtml+xml, */*;Accept-Encoding=gzip, deflate;Accept-Language=zh-CN;Host=<IP_ADDRESS>:5000;User-Agent=Mozilla/5.0 (Windows NT 6.1; WOW64; Trident/7.0; rv:11.0) like Gecko;

You can see in this situation, the context.Connection.LocalIpAddress isnot <IP_ADDRESS>, it is the server Intranet IP. But in previous situation the context.Connection.LocalIpAddress is <IP_ADDRESS>

UseForwardedHeaders is added by UseIISIntegration (https://github.com/aspnet/IISIntegration/blob/dev/src/Microsoft.AspNetCore.Server.IISIntegration/IISSetupFilter.cs#L23) with some default settings (https://github.com/aspnet/IISIntegration/blob/dev/src/Microsoft.AspNetCore.Server.IISIntegration/WebHostBuilderIISExtensions.cs#L47).
We could consider doing something similar for NGinx, but we haven't outlined what a standard NGinx deployment should look like yet.
There's a catch when you use All. See RequireHeaderSymmetry (https://github.com/aspnet/BasicMiddleware/blob/dev/src/Microsoft.AspNetCore.HttpOverrides/ForwardedHeadersOptions.cs#L34-L38)

@Tratcher Got it, Thx.
I upgrade to RTM now. But the other problem is still exists.

When use IIS as reverse proxy, I can't get the real localip(context.Connection.LocalIpAddress), I get the <IP_ADDRESS> instead.
If specify ForwardedHeaders.All, HttpContext.Connection.RemoteIpAddress is still null, why?

Don't use All, don't customize the settings for RTM.
Sharing your request headers as observed on the server may be the best way to understand the problem.

I already list the request header in previous comment, I list them again now.
Cache-Control=no-cache;Connection=Keep-Alive;Accept=text/html, application/xhtml+xml, */*;
Accept-Encoding=gzip, deflate;Accept-Language=zh-CN;Host=<IP_ADDRESS>:6000;
User-Agent=Mozilla/5.0 (Windows NT 6.1; WOW64; Trident/7.0; rv:11.0) like Gecko;
MS-ASPNETCORE-TOKEN=7c45ff68-2c0d-499e-bb53-f89ab8c33554;
X-Original-Proto=http;X-Original-For=<IP_ADDRESS>:51137;

Sorry, I had not read your posts thoroughly enough. LocalIpAddress is the address Kestrel listens on behind IIS. There's no way to get IIS's external IP. However you can get the host and port from the Host header.
It looks like you've gotten everything else working that's expected to work.

Looking forward to UseIISIntegration() equivalent for Nginx and an update to https://docs.asp.net/en/latest/publishing/linuxproduction.html  : )

@karldodd is there a specific bug open tracking your request for UseNginxIntegration? Can you add a link here?

@Tratcher I have no question, the issue can close for me.


raftstore-v2: offload checkpoint during split

What is changed and how it works?

Issue Number: Close #14711
What's Changed:

offload checkpoint during split

Compared to other apply tasks, rocksdb checkpoint is an expensive work in terms of CPU and IO where CPU works can block other tasks.
Although pre-flush mechanism makes it much more lightweight, it is only a best effort and can still cause performance drops.
This PR offloads it to a dedicated worker so that the apply thread can execute other regions' tasks during the creation of the checkpoint.
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test

Release note
None

Please add the issue# and the explanation how does the PR work

Please add the issue# and the explanation how does the PR work

Done

/merge


[feature request] break-even, max loss, max profit etc

for multi_plotter (multi-leg) , can we return a dictionary with break-even, max loss, max profit etc?
Would be great if you could update the example notebook too. Thanks!

:+1: for this


Events emitted on mount not captured

Events emitted immediately in the mounted hook are not captured by wrapper.emitted(). With something like the below

event something is not captured
event something-else is not captured on first emit in the immediate watcher, but is captured on subsequent watcher callbacks
event toggle is captured if the toggle method is called after mount, e.g. on user click

{
  mounted() {
    this.$emit('something');

    this.$watch('foo', (v) => {
      this.$emit('something-else');
    }, {
      immediate: true
    });

  },
  methods: {
    toggle() {
      this.$emit('toggle');
    }
  }
}

This happens, because the wrapper attaches to the component after the component was created and mounted.
I do not know if there is a way to change this, but I have the feeling if we do not attach the wrapper earlier there might be more edge cases where the wrapper is not ready when the component gets created.
One possibility I see is to install the wrapper earlier is via beforeCreate-mixin. It would be called by the Vue-framework after the vm-object is instiated but before the Vue-component is created and mounted, and so we could do the setup then already. We might have to split the actions done in vue-wrapper.js in several hooks.
But maybe there are other ways as well.

I was thinking we could add a listener object before mount and pass it to VueWrapper when it's created and merge that listener object with the current listener object.

Ok, so what worked for me was this (using the logEvents method in src/lib/log-events.js):
localVue.mixin({
  beforeCreate: function() {
    this.$_emitted = {}
    this.$_emittedByOrder = []
    logEvents(this, this.$_emitted, this.$_emittedByOrder)
  }
})

It does catch all events from beforeCreate on in a variable of the component (in this case $._emitted). In the constructor we could then copy these self-tracked events to the wrapper. The mixin could be added in every localVue instance when creating it.
This approach is quite naive and the first listener should be removed after the wrapper is created. It also seems a bit hacky and not too clean to use a field on the component.
Those are just my ideas. I would be interested if you know a better way to add the listener.

I can't think of a better way. Would you like to implement @wtho ?

@eddyerburgh I will submit a PR this weekend

Fixed in 1.0.0-beta.7, thanks @wtho!

I'm having similar issues when trying to test a component that has an updated method with an emit in it. Could it be that there is an issue with this as well?
component:
updated () {
    this.$emit('table-updated')
}

in my test:
console.log(wrapper.emitted('table-updated'))

result:
    console.log test/unit/specs/Table/Table.spec.js:73
      undefined

coverage documentation shows that the lines are being executed properly.

Ok, so what worked for me was this (using the logEvents method in src/lib/log-events.js):
localVue.mixin({
  beforeCreate: function() {
    this.$_emitted = {}
    this.$_emittedByOrder = []
    logEvents(this, this.$_emitted, this.$_emittedByOrder)
  }
})

It does catch all events from beforeCreate on in a variable of the component (in this case $._emitted). In the constructor we could then copy these self-tracked events to the wrapper. The mixin could be added in every localVue instance when creating it.
This approach is quite naive and the installed listener should be removed after the wrapper is created. It also seems a bit hacky and not too clean to use a field on the component.
Those are just my ideas. I would be interested if you know a better way to add the listener.

Hey, man.
I am trying to get this to work for me but to no avail.
I have a BaseModal component where I am emitting a close event when the esc key on the keyboard is pressed.
Let me represent it in code:
<script lang="ts">
import { Component, Vue } from "vue-property-decorator";
import { mapState } from "vuex";
import ModalContainer from "./ModalContainer.vue";
import ModalCard from "./ModalCard.vue";
import ModalShowUserId from "./ModalShowUserId.vue";
@Component({
  name: "BaseModal",
  components: {
    ModalContainer,
    ModalCard,
    ModalShowUserId,
  },
  computed: {
    ...mapState(["activeModal"]),
  },
})
export default class BaseModal extends Vue {
  activeModal!: string;
  mounted(): void {
    const invokeClose = () => {
      this.closeModal();
    };
    // Adding close on escape for keyboard users.
    document.addEventListener("keydown", function (event) {
      if (event.key === "Escape") {
        // Making use of invokeClose here because `this` and IIFE's are not available at this point.
        invokeClose();
        // Property 'closeModal' does not exist on type 'Document'
        // this.closeModal();
      }
    });
  }
  beforeDestroy(): void {
    document.removeEventListener("keydown", () => {
      console.log("component destroyed.");
    });
  }

  closeModal(): void {
    this.$emit("closeModal");
  }
}
</script>

<template>
  <modal-container>
    <modal-card>
      <div :class="$style['u-flex-end']">
        <base-button
          styleType="c-base-button--transparent"
          @click="closeModal"
          class="js-modal-button"
          >CLOSE or PRESS ESC</base-button
        >
      </div>
      <component :is="activeModal" />
    </modal-card>
  </modal-container>
</template>

<style module lang="scss">
.u-flex-end {
  display: flex;
  justify-content: flex-end;
}
</style>

The component works perfectly, but I am trying to test that it closes on escape.
I wrote this test for it, and modified it with your own:
import BaseModal from "./_base-modal.vue";
const localVue = createLocalVue();
localVue.mixin({
  mounted: function() {
    this.$_emitted = {}
    this.$_emittedByOrder = []
    logEvents(this, this.$_emitted, this.$_emittedByOrder)
  }
})
<EMAIL_ADDRESS>() => {
  it("Modal close button should trigger event when clicked.", () => {

    const wrapper = shallowMount(BaseModal, {
      computed: {
        activeModal () {
          return "ModalShowUserId";
        }
      }
    });
    wrapper.find(".js-modal-button").trigger("click");
    expect(wrapper.emitted().closeModal).toBeTruthy();
  });
  it("Modal should close when the escape key is triggered", () => {
    const wrapper = shallowMount(BaseModal, {
      attachTo: document.body,
      computed: {
        activeModal () {
          return "ModalShowUserId";
        }
      }
    });
    wrapper.trigger('keydown.esc');
    expect(wrapper.emitted().closeModal).toBeTruthy();
  });
});

But I am getting this error:
8:42 Property '$_emittedByOrder' does not exist on type 'VueConstructor<Vue> | ComponentOptions<Vue, DefaultData<Vue>, DefaultMethods<Vue>, DefaultComputed, PropsDefinition<...>, Record<...>>'.
  Property '$_emittedByOrder' does not exist on type 'VueConstructor<Vue>'.
     6 |     this.$_emitted = {}
     7 |     this.$_emittedByOrder = []
  >  8 |     logEvents(this, this.$_emitted, this.$_emittedByOrder)
       |                                          ^
     9 |   }
    10 | })
    11 |<EMAIL_ADDRESS>() => {

Can you tell me if there was something you did differently? or tell me how I could solve this issue.
Cheers!