legend not working when using samples argument in mafSurvival

Describe the issue
When using the 'samples' argument of the mafSurvival function the legend does not work properly. It keeps saying 'Mutant' and 'WT' (does not use the groupNames argument) and does not count the Ns [N = 0].
Command
laml.maf <- system.file("extdata", "tcga_laml.maf.gz", package = "maftools")
laml.clin <- system.file("extdata", "tcga_laml_annot.tsv", package = "maftools")
laml <- read.maf(maf = laml.maf,  clinicalData = laml.clin)
mafSurvival(maf = laml, samples = laml<EMAIL_ADDRESS>== 'M2'],
            time = 'days_to_last_followup', Status = 'Overall_Survival_Status',
            groupNames = c('M2', 'Other'))

Session info
sessionInfo()

R version 4.2.2 (2022-10-31 ucrt)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 22621)
Matrix products: default
locale:
[1] LC_COLLATE=Spanish_Spain.utf8  LC_CTYPE=Spanish_Spain.utf8    LC_MONETARY=Spanish_Spain.utf8 LC_NUMERIC=C
[5] LC_TIME=Spanish_Spain.utf8
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] maftools_2.14.0
loaded via a namespace (and not attached):
[1] lattice_0.20-45    digest_0.6.30      R.methodsS3_1.8.2  grid_4.2.2         evaluate_0.21      rlang_1.1.0        cli_3.4.1
[8] data.table_1.14.6  rstudioapi_0.14    R.oo_1.25.0        R.utils_2.12.2     Matrix_1.5-1       rmarkdown_2.21     splines_4.2.2
[15] RColorBrewer_1.1-3 tools_4.2.2        xfun_0.37          yaml_2.3.7         survival_3.4-0     fastmap_1.1.0      compiler_4.2.2
[22] htmltools_0.5.4    knitr_1.42         DNAcopy_1.72.3

Hi,
Thank you for issue and sorry for the late reply. I am following up on all the old issues now. In case of you are still using the tools, I have pushed a patch which should fix the bug . I hope this helps.


chore: prepare for first stable release

should close #752
Additionally we are:

changing the package name to @comapeo/core

We could also:

Edit readme to reflect current state of the project (I wouldn't lose much time on this now, just maybe removing the Early development disclaimer and correct npm command)
Edit authors? We can point different authors (probably can automate this so it reflects commits authors) or just put "Awana Digital" (simpler but hides individual authors)

This PR is also made so that we can test OPTICS for releasing a stable version.
We should have an NPM_TOKEN on the project before merging the OPTICS_RELEASE so we can actually publish the package.
On the other hand, I'm wondering if PRs merged after this, would be included in the release (I think yes?), but if not, we should make sure we're not missing anything (I think not)

This needs first the publishing of mapeo-schema v1 before merging


How to reference in code assets included in package

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

In SPFx v1.4 we have the ability to include assets in the package. If we were to include anything else than the main web part bundle (images, scripts that we'd like to dynamically load on runtime using SPComponentLoader), how would we refer to it in in code given that the URL is determined during deployment and the standard URL token (HTTPS://SPCLIENTSIDEASSETLIBRARY/) is replaced only in the manifest but not in code?

This has been fixed and will be released in SPFx v1.5.0.
Please re-open the ticket if it re-appears after that release.


Feature: define a custom user-class

Identify this iPXE ROM as "BlueBanquise". It allows to discriminate this
ROM from other iPXE ROM that may not comply with BlueBanquise's PXE
chain. Comply with RFC 3004 (https://tools.ietf.org/html/rfc3004).
Note: RFC 3004 defines the DHCP user class as a set of length-value
tuples, but iPXE treats it as a string. You can choose to manually
construct a value which conforms to RFC 3004 using the set command. This
is a long-standing bug both in iPXE and in the reference DHCP server
implementation (ISC dhcpd). There is a substantial amount of
documentation which suggests checking option 77 for the value 'iPXE' to
identify iPXE clients, and using 'set user-class ' to specify a
custom user class. To avoid breaking existing setups, iPXE will continue
to provide option 77 as a plain string for DHCPv4.
(Source: http://ipxe.org/cfg/user-class)

Dear @actatux ,
Many thanks for this feature. :-)
Merging right now, and will generate new iPXE rpms soon.
With my best regards
Ben


evince: 40.2 -> 40.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for evince is: "GNOME's document viewer"
meta.homepage for evince is: "https://wiki.gnome.org/Apps/Evince"
meta.changelog for evince is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4/bin/evince passed the binary check.

/nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4/bin/evince-thumbnailer passed the binary check.

/nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4/bin/evince-previewer passed the binary check.

/nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4/bin/.evince-wrapped passed the binary check.

/nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4/bin/.evince-thumbnailer-wrapped passed the binary check.

/nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4/bin/.evince-previewer-wrapped passed the binary check.

6 of 6 passed binary check by having a zero exit code.

2 of 6 passed binary check by having the new version present in output.

found 40.4 with grep in /nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4

found 40.4 in filename of file in /nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4

directory tree listing: https://gist.github.com/19abcaaab53494d3cfddb580ae814bc5

du listing: https://gist.github.com/32291c961cdd9b9ce25c7e5ca2a12792

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

21 total rebuild path(s)

7 package rebuild(s)

7 x86_64-linux rebuild(s)
7 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
7 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
denemo
evince
glom
gnome.evince
gnome.gnome-books
gnome.gnome-documents
gnome.sushi

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4 \
  --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 evince https://github.com/r-ryantm/nixpkgs/archive/edcce0c077133119445143345b2b409410267eae.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/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4
ls -la /nix/store/lap62a7jbcbvbr3lx84xgfcx76qfhwmg-evince-40.4/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 marked as broken and skipped:

gnome.gnome-documents

5 packages built:

denemo
evince
glom
gnome.gnome-books
gnome.sushi

Maintainer pings
cc @hedning @jtojnar @dasj19 @maxeaubrey for testing.

Result of nixpkgs-review pr 130600 at edcce0c0 run on aarch64-linux 1

1 package failed to build:

gnome.gnome-documents

5 packages built successfully:

denemo
evince (gnome.evince)
glom
gnome.gnome-books
gnome.sushi

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Newly added package not installing

A newly added module is not installing for some reason. Details below-
Module:
https://www.drupal.org/project/twitter_pane
composer show drupal/twitter_pane output:
name     : drupal/twitter_pane
descrip. : Provides a configurable pane for a Twitter widget feed.
keywords :
versions : 7.1.x-dev, dev-7.x-1.x
type     : library
source   : [git] http://git.drupal.org/project/twitter_pane.git d157902b1e6278e1f95e418cba9dcd3862ccc464
dist     : [zip] http://ftp.drupal.org/files/projects/twitter_pane-7.x-1.x-dev.zip
names    : drupal/twitter_pane

requires
drupal/drupal 7.*
drupal/panels 7.*

composer update output:
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - The requested package drupal/twitter_pane could not be found in any version, there may be a typo in the package name.

Potential causes:
 - A typo in the package name
 - The package is not available in a stable-enough version according to your minimum-stability setting
   see <https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussion> for more details.

Read <https://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.

I did git clone https://github.com/drupal-composer/drupal-project.git --branch 7.x foo && cd foo && composer require -vvv drupal/twitter_pane:7.1.x-dev and everything went fine.

I had an additional entry for drupal/twitter_pane. Sorry to bother.

I take that back... the package is going to /vendor instead of the type: drupal-module location. Also-
$ composer show drupal/twitter_pane
name     : drupal/twitter_pane
descrip. :
keywords :
versions : * master, 7.1.x-dev, dev-7.x-1.x
type     : drupal-module
source   : [git] http://git.drupal.org/sandbox/jaykali/2260089.git master
dist     : []
names    : drupal/twitter_pane

For some reason it's showing the (now defunct) sandbox for the source.

Something is wrong in your composer.json file. Maybe a custom repository (is suppose git.drupal.org/sandbox/jaykali/2260089.git)? The git url from composer show should be http://git.drupal.org/project/twitter_pane.git

Initial issue was related to cache, and then the module type was set to "library" instead of "drupal-module". Fixed by @webflo. Thanks!!


Visual Studio like team explorer

Need Visual Studio like team explorer, which is more useful in comparing the files from git and local repository. Code check-in and excluding files while code check-in are some great features which is missing in VS Code.

This month we're starting to look at adding an API that will allow extensions to create their own explorers (aka a tree view in a viewlet). You can check out the September iteration plan in #11917


Revert back IntelliJ .iml file to project root

In https://github.com/flutter/flutter/pull/7501 I made a number of changes to the IntelliJ files. One of them -- moving the .iml file into the .idea directory -- had an unforeseen bad consequence of leading do duplicated .iml files; see https://github.com/flutter/flutter-intellij/issues/689 for details.
Fixes https://github.com/flutter/flutter-intellij/issues/714

@pq @devoncarew can I have a review please?

👍 lgtm


xWebVirtualDirectory: Fails to create directory under application

As a proposed solution to the #331 issue I propose to change the way the resource checks Virtual Directory.

This Pull Request (PR) fixes the following issues

Fixes #331

Task list

[ x] Added an entry to the change log under the Unreleased section of the CHANGELOG.md.
Entry should say what was changed and how that affects users (if applicable), and
reference the issue being resolved (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

@regedit32 Do you have time to look over this too? You have a better knowledge around these cmdlets than me. 😄

@johlju solution looks good, but I will take another look after tests have been added.

We have renamed the resource, removing 'x', so please rebase this PR.


help with your awesome

how did you get your bar like this, i don't see it in your configs maybe i over looked something.

I only see this in where the spotify widget is showing
https://github.com/AllJavi/dotfiles/raw/master/.config/awesome/widget/spotify-widget/spotify-widget-custom-playing.png
I really love it and want it Thank you

It's because that is how the original widget looks, the init.lua file is modified to look like I have

oh OK that's already in your configs. Thanks. i'm not scripted i was just trying to understand how it worked. Awesome setup


Change trigonometric evaluation strategy

Math.sin appears to be more accurate near the origin than farther away. By shifting arguments into [-pi/2, pi/2] before applying the library function, we achieve more accurate results near the origin, especially at multiples of pi/2. In particular, now sin(pi) is exactly 0.
Compounding shifting error with the library function error means this strategy is less accurate as the shift gets large. However, the error is still comparable to that of the library function, within 1e-11 on [-40pi, 40pi].
Closes #9.

Because Math.PI is not equal to π, you would not expect Math.sin(Math.PI) to be equal to 0. It seems Math.PI ≈ π - 1.22e-16, so sin(Math.PI) = sin(π - 1.22e-16) = sin(1.22e-16) ≈ 1.22e-16. Indeed, that's very close to what Math.sin(Math.PI) gives me in Chrome right now. So I'm skeptical of changing this by "fixing" how trig functions are evaluated.

That's true, I shouldn't suggest that there's something wrong with Math.sin. However, if someone writes "sin(pi)" in a problem and that eventually gets passed to KAS, they probably don't intend to invoke Math.sin(Math.PI) and might expect to get back 0 rather than 1.22e-16.
Probably the cases where this difference would matter are rare, otherwise the issue would have received more attention. It's also admittedly a very weird choice to reimplement a library function, so I'm very sympathetic to the skeptical view on these changes.


glob pattern inside meta.xml

Description
Passing glob inside meta.xml (e.g. files: webui/build/**/* to recursively add all files contained in build folder and subfolders)
Is your feature request related to a problem? Please describe.
It will help to embed front-end frameworks for working with CEF (which generate many files)

https://github.com/mtasa-typescript/resource-boilerplate/issues/127


Transaction' object has no attribute 'description'

Behavior observed:
GUI threw the following error:Transaction' object has no attribute 'description'
Expected behavior:
Transactions are exported from Mint and tagged with Amazon purchase data.
2023-10-13_14-13-32.log

@preppietechie Sorry for the delayed response.  I've examined the stack trace and nothing jumps out at me. Considering mint (and this tool) are shutting down at the end of this year, is this still relevant to you? Thanks.

Please reopen if you are still having issue. Cheers


How to use vscode to debug the acceleration program with breakpoints? I checked a lot of information, but still didn't find a solution

How to use vscode to debug the acceleration program with breakpoints? I checked a lot of information, but still didn't find a solution

Hi! I've been debugging another project w/ VSCode and I had a similar launch.json file. In my case, I debugged the code in a docker container, but it should work with local environments. What error are you getting? Without testing your code, it seems ok. The module seems right and the args too. Are you including "justMyCode": false to also debug into accelerate & PyTorch code? Remember that you need to open the train_text_to_image.py in the editor, select the debug menu and start the debugging.


@OPENSSL_LIBS@ added into automake file

Re-opening PR regarding #2524

This is required prior https://github.com/apache/trafficserver/issues/1560 task which is started by me recently.

Is this related to the QUIC work? @masaori335 Do we want this on the QUIC branch, or master? Both?

[approve ci].

@zwoop not really related to QUIC but impacts overall build and codebase quality since doesn't allow to build 'test' target. What does it impact is std::vector support all across the code (instead of custom Vec implementation).

Hmmm, but we run the "make check" on all builds in the CI, and aren't running into this. I'm not saying your fix is wrong, just curious as to why it's not causing problems on the CI.

@zwoop that is very strange indeed. I've just checked 'make check' target on newly cloned repo from upstream and got the same error
/usr/bin/ld: libinknet.a(SSLNetVConnection.o): undefined reference to symbol 'X509_free@@OPENSSL_1.0.0' //lib/x86_64-linux-gnu/libcrypto.so.1.0.0: error adding symbols: DSO missing from command line collect2: error: ld returned 1 exit status Makefile:966: recipe for target 'test_UDPNet' failed

Could it be due to some different way how does project got configured on different platforms (Ubuntu vs FreeBSD)?

Checked CI target build log for Linux - it is OK except one thing - not enough verbosity to see how does test_I_UDPNet.cc does build.
Can we enable verbosity with V=1 for make check

@zwoop I want this on both branch to run the "make check". But quic branch is going to be rebased on master branch. So I don't want to cherry-pick this on quic branch right now.

Why are tests building and working without this change?  Is this change necessary?

@bryancall have no idea why they are building & working w/o these changes. But I am 100% confident that introducing these changes won't break anything.
BTW this is in regards of ^ - https://github.com/apache/trafficserver/pull/2547


Support multiple kinds of RudiContent

RudiContentFile - File-based content.
RudiContentRaw - Text and byte strings.
RudiContentTree - From HtmlWriter.
RudiContentVirtual - From RudiVirtual.

This was handled with commits:

9551c0a5b0a4c964efda6a7d7d24270c25b90a7d
02ff885022a7d8d622924c95e4b2844e7cf890cd
a5acf71925c92f0e60bdcd9bc65b4a28c95a71b5
89563e0990f23dd7136e1f9ac9580bb56eb577cd
0b4125c7d5b9cf594f0f3469ecc010bd1a95f091


[Issue] read_channel_status Method Returns Empty, Zero, or Default Values in Demo Script

Description -
When running the demo script, I've noticed that the read_channel_status method is returning empty, zero, or default values. I would like to understand why this is happening and if there are any known reasons for this behaviour.
Steps to Reproduce -

Execute the demo script.
Specifically, observe the output of the read_channel_status method for cell 3

Screenshot -

Question:
Additionally, I am interested in connecting the library to an external stimulator to obtain actual data. Is there any guidance or documentation available on how to achieve this integration?
Any assistance or insights into these matters would be greatly appreciated.
Thank you!!

Hello, in the demo_notebook.ipynb example, ArbinSpoofer is used to simulate an Arbin Cycler. You can refer to arbin_spoofer.py for more details. In the init method, all fields will have default values. Therefore, if you want ArbinSpoofer to return different values, you can refer to the following code to modify the voltage field:
arbin_spoofer.update_channel_status(0, {'voltage_v': 1.234})

@SyXuan, while the update_channel_status method allows us to update values, I'm actually looking to retrieve raw data from PyCTI. How can I achieve that?
I was previously using read_channel_status method but it seems like that the values provided by that method are null or empty. Why so ?
My primary objective is to obtain real data from PyCTI using spoofers. Could you provide guidance on that ?

Please follow the demo_notebook.ipynb with your cycler credentials to fetch the raw data

I am not using any cycler. I want to connect using spoofers. Is there any schedular file for that ?

@devanshiAppperfect generating data is not in the scope of pycti or pymacnet. We did write small implementation of cycler spoofer, but the intent was only to support development, testing, and debugging since hardware access cannot be guaranteed in remote work environments.
If you want mock data, then you can try a package like Faker (https://faker.readthedocs.io/en/master/index.html) to generate mock data. What the data format for different fields should be can be determined from the cycler output or CTI/MacNet docs.
To have the data look exactly like a real li-ion cell, you need a battery model. The better and more accurate the model, the closer the mocked or modeled data will be to a real cell.

@devanshiAppperfect this folder has a sample schedule and resulting data file. Let me know if you have further questions.

@chintanp  Could you please specify which folder you are referring to?


Add ACML Acceptance Rates?

Hi,
would it be possible to add the ACML acceptance rates: http://acml-conf.org/Print.aspx?Page=MainPage
I can also file a pull request with the required data, if requested.
Best regards!

I agree with @alexandertornede. ACML is growing in prominence with increasingly good research published there and prominent PCs (e.g., Masashi Sugiyama).

I'd argue that ECML should be even higher priority than ACML. I've attended both conferences a couple of times and ECML is noticeably bigger---around 1,000 attendees last year.

@henrygouk Agree.  ECML is more prestigious and established than ACML.

I agree with both of you that ECML is more established than the ACML. I am not sure if they also publish their rates in a collection as ACML does or if anyone has that data. If so, I guess both could be added.


At first very fast, but it get slower and slower

At the beginning, it is very fast. Less than 1s for a batch(256).
But it soon getting slower and slower. It is about 10s for a batch(256) after several epochs, which even slower than the original Pytorch dataLoader. The CPU-util is much lower than that of the begging.
Why does this happen, how can I fix it?

Hi, have you figured out why this happened?

Hi, have you figured out why this happened?

sorry I don't... I am no longer doing that job now


improve parser and server reload performance by indexing

small improvement by not using range in loops

Thanks for the PR. I understand the current implementation creates a copy and may be less efficient with large number of stubs.
I prefer readability over premature optimisation, so I’d  keep range over for i := 0; i < length; I++.
Perhaps we can go with for i := range schemas And access the slice using I

Thanks for the PR!


Warning: Can't perform a React state update on a component that hasn't mounted yet.

Hi there!
First of all, I am not quite sure if this is an issue with generouted or @tanstack/react-router, or even with Relay (the GraphQL client I use for data fetching) but I noticed that my page gives me the following error after a few refreshes:
Warning: Can't perform a React state update on a component that hasn't mounted yet. This indicates that you have a side-effect in your render function that asynchronously later calls tries to update the component.
I realised that the error disappears if I get rid of the lazy() import from the generated routes file and manually import the component in there.
Could this be an issue with TanStack router or perhaps with the way of how I fetch data in combination with React 18 / Suspense? I am not quite sure what's happening. Maybe any of you are able to help me out.
Thanks a lot, I appreciate it!

Hey @gino, thanks for reporting this.
I believe it's not related to generouted, and I was thinking about having routes lazy-loading/splitting optional.
If you still have this issue, could you please provide a StackBlitz repro?
Thanks!

Hey @oedotme, I realised that this is indeed an issue with lazy loading my routes. I am not quite sure why this would occur any issues but whenever I disable lazy loading, my component does not throw this error.
Not quite sure what's going on but I'll try to figure it out!


Added new CSS to avoid using card ID

Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

@kimsauce I have updated all of the available index pages.

@JV0812, the build-and-deploy failed (Details). Could you fix it? Then I'll give this PR a review.

@kimsauce I have updated all of the available index pages.

Thank you for clarifying! It looks like the last outstanding item is a minor merge conflict on one page.

@JV0812, are you ready to merge this PR now? I'm excited to see this one in production!

Yes John, I will merge it today.


Check if AHardwareBuffer is cpu readable in trim

Check if AHardwareBuffer has cpu read mask usage, before locking it for cpu read. This fixes some of the AHardwareBuffer_lock issues and stutters.
The same check is already in vulkan_capture_manager.cpp with similar code.

CI gfxreconstruct build queued with queue ID 248867.

CI gfxreconstruct build # 4755 passed.


Release 7.2.0 on the Unity Asset Store

Both packages on the Unity Asset Store are on 7.1.0, but 7.2.0 has been out for weeks. I'd really like to use the new object managing utilities in Unity, and was confused when I couldn't find methods that were described in the docs.

Hi @NomarCub,
Thanks for the issue. My fault, I forgot to click "Publish" button on assets, sorry.
Now it's done and the 7.2.0 version on the Assets Store. Thanks for using the library!

Thank you for the quick response!


Fancy zones broken

Microsoft PowerToys version
0.60.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Appears now it applies the same setting as the last monitor you select to every monitor.
✔ Expected Behavior
Should be able to slelect custom zones for each monitor
❌ Actual Behavior
When setting the zones on the last monitor is sets those zones to all monitors
Other Software
No response

PowerToysReport_2022-07-08-07-57-53.zip

/dup of https://github.com/microsoft/PowerToys/issues/19203
Working on a fix. Thank you.


Auto-deploy snapshots to Sonatype

Closes #68.

Coverage decreased (-0.13%) to 98.59% when pulling 33b77ef9ece52866b85450af69f36123cd13e47e on iss68 into af3748fd6a8a5f152c470b2e848b32d31d30da1d on master.


HSEARCH-2964 Add the JSR 352 modules to the distribution

https://hibernate.atlassian.net/browse/HSEARCH-2964
https://hibernate.atlassian.net/browse/HSEARCH-2778
https://hibernate.atlassian.net/browse/HSEARCH-2969
https://hibernate.atlassian.net/browse/HSEARCH-2968

Adding the Changes required label: Guillaume is working on making this faster (replacing the javadoc:jar goal with javadoc:javadoc in particular).

Merged, thanks a lot :)


Bundle size of the stacks package is really huge

Is your feature request related to a problem? Please describe.
I have a super simple next.js app, and the bundle size of the js files is in total 295 kB gzipped. Without stacks.js dependencies, the size is around 65 kB. So most of my bundle size is coming from the stacks dependencies as you can see in the following screenshot.

Describe the solution you'd like
Using stacks.js libs shouldn't make my app slow to load for my users.
Opening the issue to start the discussion around improvements that can be made in stacks.js

@zone117x As a starting point on this issue, to get rid of some of dependencies like bn.js and few others, is it allowed to use some source code from micro-stacks repo. I can see micro-stacks has GNU GENERAL PUBLIC LICENSE and stacks js is using The MIT License (MIT). I am asking in terms of copyright issue / permissions to use micro-stacks code, will it be allowed or not.

@zone117x Any thoughts on above question 🙏

Can you reopen the issue? I don't think this one is solved yet.

@reedrosenbluth

Moving this to the relevant milestone and sub-issues


about excptionless-spring-boot-starter

where can i find the excptionless-spring-boot-starter?
I'm really need this component @niemyjski @pc9795

I'm not sure what you are asking for. Can you please elaborate?

Hi @zen-arch correct me if I am wrong, but can't you declare exception-less client as a bean in one of your configuration classes like here: https://github.com/exceptionless/Exceptionless.Java/blob/4b3e2232ddf2d46e589dc21ee05e989f718b6549/samples/example-spring-boot-app/src/main/java/com/exceptionless/example/spring/boot/app/ExampleAPI.java#L15
And then use it anywhere like this:
https://github.com/exceptionless/Exceptionless.Java/blob/4b3e2232ddf2d46e589dc21ee05e989f718b6549/samples/example-spring-boot-app/src/main/java/com/exceptionless/example/spring/boot/app/api/ExampleResource.java#L24
There is an examples spring boot app in the repo - https://github.com/exceptionless/Exceptionless.Java/tree/main/samples/example-spring-boot-app
Let me know if it works for you. Thanks


fix links to point to invoke-ai.github.io #1117

hope I catched em all ;P
in favor of #1117

@lstein since I cannot request reviewers I just tagged you 🙈


Layout styles for write mode tab

These are just the changes to the layout and presentation of the write mode page as described in eregs/notice-and-comment#202.
The comment-context box still needs styling and show more/show less functionality.

Looks nice and works locally.


mylg user guide content

Hi,
Do you store the contents of http://mylg.io/doc/ on a github repo?

@jungle-boogie that's not under github. if you think we need to add/update/modify pls let me know then I can update there. I hadn't enough time to spend on the doc. Thanks!


j2objc - exception when deserialize enum

(Note that this was originally posted as a question on StackOverflow and answered by Tom Ball there. I'm posting here for completeness and tracking purposes.)
I am having trouble deserializing objects that contain an enum. The object serializes without complaint, but I get an InvalidObjectException when I deserialize the object. The exception message says that there is "No enum constant com.mypackagname."
I have isolated and reproduced the problem by creating some test code based on the testSerialization() method in SerializationTest.java.
public class SerializationTest {
    private static final String TEST_FILE_NAME = "serialization-test.bin";
    public enum Gender { MALE, FEMALE }

    public void testEnumSerialization() throws IOException, ClassNotFoundException {
        Gender gender = Gender.MALE;

        // Save the enum to a file.
        ObjectOutputStream out = new ObjectOutputStream(new FileOutputStream(TEST_FILE_NAME));
        out.writeObject(gender);
        out.close();

        // Read back the enum.
        ObjectInputStream in = new ObjectInputStream(new FileInputStream(TEST_FILE_NAME));
        Gender gender2 = (Gender) in.readObject();
        in.close();
    }
}

I have discovered that if I add a string value to the enum initialization in the generated Objective C code the deserialization works fine. The resulting initialize method in Obj C looks like this:
+ (void)initialize {
    if (self == [SerializationTest_Gender class]) {
        JreEnum(SerializationTest_Gender, MALE) = new_SerializationTest_Gender_initWithNSString_withInt_(@"MALE", 0);
        JreEnum(SerializationTest_Gender, FEMALE) = new_SerializationTest_Gender_initWithNSString_withInt_(@"FEMALE", 1);
        J2OBJC_SET_INITIALIZED(SerializationTest_Gender)
    }
}

Note that I added the @"MALE" and @"FEMALE", the default from the j2objc output is @"".
Thanks for all of your hard work on this really valuable tool!

What version of j2objc are you using? This looks to already be fixed, since the test passes (and the initialize method for Gender is completely different). The serialization-test.bin generated by the JVM and j2objc are byte-for-byte identical (as they should be). I verified this with release 2.0.5 as well as a build of the latest source.

I confirmed that I am using version 2.0.5.  Not sure if it makes a difference but here are the options I am using:  -l -d . --swift-friendly --no-package-directories -use-arc --static-accessor-methods

Thanks for the quick response. The issue only shows up with the -use-arc flag. Investigating...

Fixed in current sourc.


Fetch data from /servers

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3650

PR #3673

PR #3674   👈

This comment was auto-generated by Graphite.


Quick Play support

It's possible to add official Quick Play servers to the Server Browser.
I have a working prototype and am working on this under on the quickplay branch.
How it would work

Any player in a quick play lobby can choose to share it from the mod panel under Modifiers.
Rather than sharing a server code, the connection would happen by passing the host / server secret during the Matchmaking process.
The Quick Play lobby is shown in the browser as long as at least one Server Browser user is sharing it.

Task list
Core

[x] Update data model on API to include server type and secret
[x] Support sharing Quick Play lobbies
[x] Support connecting to Quick Play lobbies

Integration

[ ] Browser/API: Allow users to filter out Quick Play lobbies
[x] API: Handle multiple users potentially announcing the same game (merge)
[x] Detect and/or prevent connection failures

If we can't connect by secret for any reason, matchmaking will put us in a random Quickplay server. We should detect the secret not matching and abort the connection / show an error.

[x] Communicate Quick Play lobby difficulty correctly

The browser doesn't show the correct difficulty right now
The mod currently passes SongPackMask.All during the matchmaking process which may break things(?)

[x] API: Don't send Quick Play games to Server Browser <0.7.0

Quick Play support is now a core part of the mod because 1.16.4, closing this out.


Eciplse Project Launcher use Gradle dependency from first library project

BuildShip 2.0.1 and Gradle 3.4
If we add a library project for debugging to an eclipse project then the launcher resolve the project and external dependencies form the library project and not from the main project. The result are ClassNotFoundExceptions from libraries which are only in the main project.
This is also good visible in the classpath order of the launcher. The project and external dependencies follow the first library project.
Our library project look like:
<?xml version="1.0" encoding="UTF-8"?>
<classpath>
	<classpathentry kind="src" path="src"/>
	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.8/"/>
	<classpathentry kind="con" path="org.eclipse.buildship.core.gradleclasspathcontainer"/>
	<classpathentry kind="output" path="bin"/>
</classpath>

Our main project look like:
<?xml version="1.0" encoding="UTF-8"?>
<classpath>
	<classpathentry combineaccessrules="false" kind="src" path="/LibraryToDebug"/>
	<classpathentry kind="src" path="src"/>
	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.8/"/>
	<classpathentry kind="con" path="org.eclipse.buildship.core.gradleclasspathcontainer"/>
	<classpathentry kind="output" path="bin"/>
</classpath>

The library classpathentry must before the gradleclasspathcontainer else the library classes come from gradle and not from eclipse project and we can't debug it.
The expected behavior is that the project launcher use ever the dependencies from the current (main) project and not from the first project with Gradle dependencies. The Ivy plugin has solved this with adding then dependencies from all projects in the classpath. But I think this is not needed.

Can you please provide a reproducible example project?
Also you might want to use a composite build instead of manually adjusting Eclipse .classpath files.

The composite build based on settings in the gradle.settings will also have an effect to the build system.
Settings in the .project file and the *.lauch files are development only. A team can share this setting without any effect to the build system.
It is very important to have in the repository settings for

the build system
shared for build and dev systems
dev systems only.

I will create a eclipse workspace to demonstrate it and upload it.

The composite build based on settings in the gradle.settings will also have an effect to the build system.

That's the point, it properly tells Gradle to substitute dependencies, so you don't have to do classpath ordering stunts in Eclipse :)
You can do composite builds with an if-statement, so they don't affect your CI environment. We also plan to add some UI in Buildship in the future so you can do ad-hoc composites without changing a settings.gradle file.

You can do composite builds with an if-statement, so they don't affect your CI environment.
How can I do this if condition?

I have attached a sample workspace. Its contains:

a library project "JLessC" with extra debug output
its contain 2 main projects. The only difference is the order of classpathentry in the file .classpath and the main class was renamed to differentiate. Both haven a jcenter reference to 2 libraries.

The one main project does not show the extra debug output from the library (expected). The other show the ClassNotFoundException (the bug). The compiler can resolve the class. The launcher does not.
EclipseLauncherSample.zip
PS: Sounds also like a posible hack to differ between compile dependencies and runtime dependencies.

Thank you for the example. Turns out that JDT de-duplicates classpath containers when you don't explicitly tell it not to. I've created a PR for buildship to fix that.

FYI the Buildship PR has been merged.


Include polyfills for IE11 using polyfill-service?

Is this a bug report?
Nope! More of a feature request.
Is it possible to include some polyfills for a fallbacks for IE 11?
It could be as simple as adding a script tag to this library that provides all the polyfills in the public/index.html file.
The script tag is <script src="https://cdn.polyfill.io/v2/polyfill.min.js"></script>
And the github for the polyfill service is: polyfill-service
This would be nice feature for those who have to support internet explorer 11, with no cost to people who are using other browsers, as the polyfill service provides polyfills as they are needed!  :+1:

Never mind this is a duplicate.


Regression in latest nightly around dependencyInsight changes

Expected Behavior

ConfigurationContainer rules should be accounted for when searching for configuration by name. The suggested fix is to findByName first and if the result is null, then proceed with a name matcher using getAsMap(). This way the rules have a chance to trigger and create any of the required configurations. Then, if nothing is found, we may be using a short name which we can try to find among all configurations (current and generated by rules).
Current Behavior

The changes in https://github.com/gradle/gradle/pull/16142 don't account for rules anymore. It uses getAsMap() which doesn't trigger any rules.
Context

See https://github.com/gradle/gradle/pull/16142#issuecomment-859076187
Steps to Reproduce

Your Environment

Build scan URL:

I've reverted the changes and notified the contributor about the broken behavior: https://github.com/gradle/gradle/pull/17428


Support 182.x

New EAP 😄 Thanks for all your hard work!

Yes please, I have very large number of bash scripts which are now all displayed as plain text.  I have reverted to 2018.1 and/or VS Code in the meantime (hope a resolution is provided soon)

yes, pretty please :)

Should be possible next week.
Do you really use the first EAPs in production?

Awesome, thanks!
Yeah, I've never personally had any major issues with EAP versions and I happily use it as a daily driver.

Same experience here with EAP releases. This is actually the first time in as long as I can remember that I have had to “downgrade”.
Personally I do a lot of bash scripting, so this single issue had me downgrading :)

Thank you for all your hard work

There's an update now for 182.x. As far as I can tell the settings should be preserved. Let me know if you encounter an issue with the plugin and an EAP.
Thanks for your patience :) I'd love to work more on this, but it's not possible atm.

Although the plugin claims to support 182, it does not show up as supported in GoLand 2018.2 EAP Build #GO-182.2371.38, built on May 18, 2018

@jhvhs 182.2574 and later are supported. Is there a newer GoLand EAP available?


#3915 - Pasting does not work in Firefox browser at all

How the feature works? / How did you fix the issue?
(Screenshots, videos, or GIFs, if applicable)
Check list

[ ] unit-tests written
[ ] e2e-tests written
[ ] documentation updated
[x] PR name follows the pattern #1234 – issue name
[x] branch name doesn't contain '#'
[x] PR is linked with the issue
[x] base branch (master or release/xx) is correct
[x] task status changed to "Code review"
[x] reviewers are notified about the pull request

Pretested. Bug is fixed.


convert to onnx error

Thanks for the great work.
When I tried to convert the model to onnx to measure exec time, there is an error. It seems onnx does not support atan2?
How to solve this problem?
RuntimeError: Exporting the operator atan2 to ONNX opset version 11 is not supported. Please open a bug to request ONNX export support for the missing operator.

I am also having the same error when I am trying to convert the model into ONNX.

I am also having the same error when I am trying to convert the model into ONNX.

any progress?

OK, in our practice, we remove the (i)stft and atan part from the onnx part. Or you can use libtorch, this is also a good way for inference.

OK, in our practice, we remove the (i)stft and atan part from the onnx part. Or you can use libtorch, this is also a good way for inference.

Thanks.

OK, in our practice, we remove the (i)stft and atan part from the onnx part. Or you can use libtorch, this is also a good way for inference.

请问你导onnx的时候，用的pytorch和对应onnx什么版本的，以及对应的opset_version是多少。 谢谢


https://material.angular.io Don't work

The documentation don't work properly.
I always seen 404.
Someone can tell me why and if another documentation exists?

Is material.angular.io down?
It's just you. https://material.angular.io is up.
http://downforeveryoneorjustme.com/material.angular.io

@Necromancerx
Please, why is just for me and how to access documentation ?

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.

Site down for me too. :c


Consider an API for per-field addendums

We had a discussion with ILAO about long names that go past field limits. We decided that the addendum wasn't the best for the names, and you'd likely want something an auto-size font in the PDF field, or maybe an additional screen that lets the user know it might be too long?
However, to do that you'd have to turn off addendums for the whole document. Something like "excluded_fields" might be useful. Things get a bit more cumbersome to do that though, so we should think about that.

Ah, that's just me misunderstanding ALDocument then. Is "saying you want to send that specific field to the addendum" setting the my_doc.overflow_fields["my_field_name"].overflow_trigger to something? I missed that and only saw my_doc.has_addendum


MSVC compilation errors are not shown

When cl.exe fails to compile a file, the gcc crate only displays command did not execute successfully, got: exit code: 2, but does not print errors generated by cl.exe.
Running the full cl.exe command manually does show errors, but copying&pasting commands in cmd prompt is incredibly tedious and error-prone, so it'd be very helpful if Cargo did not eat cl.exe's output.

process didn't exit successfully: C:\Users\user\proj\target\debug\build\libtest-sys-9e3214e9b9252129\build-script-build (exit code: 101)
--- stdout
TARGET = Some("i686-pc-windows-msvc")
OPT_LEVEL = Some("0")
TARGET = Some("i686-pc-windows-msvc")
HOST = Some("i686-pc-windows-msvc")
TARGET = Some("i686-pc-windows-msvc")
TARGET = Some("i686-pc-windows-msvc")
HOST = Some("i686-pc-windows-msvc")
CC_i686-pc-windows-msvc = None
CC_i686_pc_windows_msvc = None
HOST_CC = None
CC = None
TARGET = Some("i686-pc-windows-msvc")
HOST = Some("i686-pc-windows-msvc")
CFLAGS_i686-pc-windows-msvc = None
CFLAGS_i686_pc_windows_msvc = None
HOST_CFLAGS = None
CFLAGS = None
PROFILE = Some("debug")
running: "C:\Program Files\Microsoft Visual Studio 14.0\VC\BIN\cl.exe" "/nologo" "/MD" "/Z7" "/DUSE_SSE=1" "/FoC:\Users\user\proj\target\debug\build\libtest-sys-96b972ed96e9414d\out\libtest.o" "/c" "libtest.c"
ExitStatus(ExitStatus(2))>

command did not execute successfully, got: exit code: 2>

--- stderr
thread 'main' panicked at 'explicit panic', C:\Users\user.cargo\registry\src\github.com-1ecc6299db9ec823\gcc-0.3.43\src\lib.rs:1127
note: Run with RUST_BACKTRACE=1 for a backtrace.

Out of curiosity, did you think that this crate is intentionally trying to swallow up these errors?
In any case, this is certainly not intentional. Is there any more debugging information you can offer to reproduce for example? I"ve never run into this locally at least when working with MSVC.

I don't want to imply intention, but I did assume Cargo and Command were capturing stderr to print it only under certain conditions, which probably didn't cover this one. Or maybe just Windows or msvc has stderr output done in a weird way.
To reproduce:

Windows 10 VM from here: https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/
Install VS studio tools recommended by rustup
rustup default nightly (because stable Rust doesn't work at all on this vm)
Cargo.toml:

[package]
name = "foo"
version = "0.1.0"
authors = ["IEUser"]
build = "build.rs"

[build-dependencies]
gcc = "0"

build.rs:

extern crate gcc;

fn main() {
     gcc::Config::new()
                .file("bork.c")
                .compile("libfoo.a");
}

echo bork > bork.c
cargo build -v

running: "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\amd64\cl.exe" "/nologo" "/MD" "/Z7" "/FoC:\Users\IEUser\foo\target\debug\build\foo-d1afa518aed92cd9\out\bork.o" "/c" "bork.c"
ExitStatus(ExitStatus(2))
command did not execute successfully, got: exit code: 2

But running the same from cmd.exe:
C:\Users\IEUser\foo> "C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\bin\\amd64\\cl.exe" "/nologo" "/MD" "/Z7" "/FoC:\\Users\\IEUser\\foo\\target\\debug\\build\\foo-d1afa518aed92cd9\\out\\bork.o" "/c" "bork.c"
bork.c
bork.c(2): fatal error C1004: unexpected end-of-file found

This crate doesn't capture the output of commands, and yes Cargo does. Cargo's been doing that for years with no issue, though.

I see something similar when running tests in AppVeyor:
     Running `C:\projects\cupid\target\debug\build\cupid-cdb2b23639ca70fa\build-script-build`
error: failed to run custom build command for `cupid v0.3.0 (file:///C:/projects/cupid)`
process didn't exit successfully: `C:\projects\cupid\target\debug\build\cupid-cdb2b23639ca70fa\build-script-build` (exit code: 101)
--- stdout
TARGET = Some("x86_64-pc-windows-msvc")
OPT_LEVEL = Some("0")
TARGET = Some("x86_64-pc-windows-msvc")
HOST = Some("x86_64-pc-windows-msvc")
TARGET = Some("x86_64-pc-windows-msvc")
TARGET = Some("x86_64-pc-windows-msvc")
HOST = Some("x86_64-pc-windows-msvc")
CC_x86_64-pc-windows-msvc = None
CC_x86_64_pc_windows_msvc = None
HOST_CC = None
CC = None
TARGET = Some("x86_64-pc-windows-msvc")
HOST = Some("x86_64-pc-windows-msvc")
CFLAGS_x86_64-pc-windows-msvc = None
CFLAGS_x86_64_pc_windows_msvc = None
HOST_CFLAGS = None
CFLAGS = None
PROFILE = Some("debug")
running: "C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\bin\\amd64\\cl.exe" "/nologo" "/MD" "/Z7" "/DMSVC" "/FoC:\\projects\\cupid\\target\\debug\\build\\cupid-e0d03e0505cc1475\\out\\src/arch\\shim.o" "/c" "src/arch/shim.c"
ExitStatus(ExitStatus(2))
command did not execute successfully, got: exit code: 2
--- stderr
thread 'main' panicked at 'explicit panic', C:\Users\appveyor\.cargo\registry\src\github.com-1ecc6299db9ec823\gcc-0.3.44\src\lib.rs:1140
note: Run with `RUST_BACKTRACE=1` for a backtrace.

I'm going to try and get a local MSVC install to see if there's any useful output outside of AppVeyor.

Strangely, the code compiles with a local MSVC (which is going to make debugging AppVeyor tricky). If I add #error "this is a bad thing" to my code, I do see the same behavior as before, and no sign of the text "bad thing".

I think the problem comes from the fact that stdout is dropped on the floor. It's assigned at line 1096, but we can fail the entire program at line 1123, line 1129 and line 1133.
Thus, I think the problem isn't specific to MSVC, but rather that MSVC is outputting errors to standard output and that any information from stdout is ignored in these (and other?) failure cases.

Using a quick hack to print out stdout in the 3 failure cases appears to be helping me in my case.
Any suggestions on how you'd like to see this kind of patch cleaned up for submission, @alexcrichton ?

Ah that'd do it, thanks for the investigation @shepmaster!
This was introduced in https://github.com/alexcrichton/gcc-rs/commit/0084d5e0ee691720d7c7bb3aaee3cbfc15cd5796, so I think we should just refactor to have that one precise endpoint capture stdout and otherwise nothing else should. I'll go ahead and do that locally real quick.

Thank you!


Allow different settings for pulls and issues. Add config 'limitPerRun'.

Hello, and thanks for the useful bot!
You recommended I make a PR, and I hope this makes the bot even better.
This PR does two things:

Closes #66: Feature Request: different stale settings for PRs and Issues

Most configs can be specified for just 'pulls' or 'issues' as overrides for the 'global' config values.
The config values for owner, repo, only, and logger all remain 'global' only.

Add config 'limitPerRun'

Limit per run is still capped at 30, but the config can now set it lower.
This will let me start chipping away at old issues at 1 per hour instead of getting a huge flood.

This should all be backward compatible, and examples for the new configs are added to the README.
~Unfortunately I was not able to easily find a way to test this out.~
~I have looked it over as carefully as I can, but if you can test it out or point me in the right direction I would really appreciate it!~

Updated with a unit test, and fixed some issues.

I'm planning to work with a friend this weekend to test things out on a real repo.

Hey @mezz, this is looking good! I won't have time to review closely until tomorrow or Saturday, but I like it at first glance.

Hi @bkeepers, do you have a chance to look this over?

Sorry @mezz, I have a milestone for work that's due this Tuesday, so I've been really busy. I'll check this out over the Thanksgiving holiday though.
/remind me to review this on friday

Hi @bkeepers, I am not in a rush at all but I will try to remind you of this occasionally :)
Hope you had a nice holiday.

Hey @bkeepers, do you have a chance to look this over?

Thank you for the review. I will make sure to create smaller PRs in the future, sorry.

I have commented out the new config example so that people don't accidentally use it by default.
The logger.debug issue was because webstorm supports console.debug but npm does not. I have changed these to info like all the other logs, which is supported everywhere.
This has been rebased to resolve conflicts. Github couldn't do it automatically, but git on my machine had no issue with it.

🎉  Thanks @mezz! This is deploying now and should be available by the time you read this.


tendermint: allow null in EventAttribute kv JSON

Fixes #1375
When deserializing EventAttribute values, allow null values in the key and value fields. This follows the behaviour in 0.34 RPC (where the key-value data were also base64-encoded), but technically breaks the serde schema of everything that includes ABCI event data.

[x] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[ ] Updated all code comments where relevant
[ ] Wrote tests
[x] Added entry in .changelog/

The coverage failure should be fixed by #1377.


Size by numerical attributes only

Just asking for a review of whether or not this is desired functionality, i think so but still would like a thumbs up or no from either of you.
This restricts sizing treemap cells by numerical attribute values.
i need to finish implementing this on the PI Vision side first.

Just asking for a review of whether or not this is desired functionality, i think so but still want a thumbs up or no from either of you.
Or if you think it's worth it, go ahead and merge! i think this is ready to go.

Looks great!  We can discuss if we want deal with AF enumerations


How to integrate pod player

We need to manually control whether to play
We need to preload the video

Please clarify that you need help with pod_player or this y_player ?


Don't use .getText() for module specifiers

Taking a cue from https://github.com/zerkalica/zerollup/commit/9f3cc5cf8fe2325e91cb15a8a23b6de88798f185, we won't call .getText() on module specifiers but just get the text property directly.

Fixes #9, hopefully.


Blocking rendering -> Webworkers?

As the rendering blocks the whole js thread currently, maybe I should use webworkers to do this in a non-blocking way.

That’s a great idea !

@yvanblanchard
I looked into it a bit, but I think it would need a bit more refactoring to add this feature.
However for now I found another way to avoid too long blocks:
I now just call setTimeout with 0 all 200 lines while rendering.
That way the browser gets a chance to handle its ui updates between rendering.
You can use v0.5.0 to use that feature.

Thank you very much for your work!

It is already😃


Understanding dependent rules

(This has been moved here from CodePlex)
Dear Jeremy,
I have been reading about dependent rules here:
https://github.com/JeremySkinner/FluentValidation/issues/65
From my understanding, a dependent rule will execute only if the preceding rules succeed. However, I cannot get this to work. Ex:
	internal class CustomerValidator : AbstractValidator<Customer>
	{
		internal CustomerValidator()
		{
			RuleSet("Add", () =>
			{
				RuleFor(o => o).NotNull()
					.DependentRules(d =>
					{
						d.RuleFor(x => x.FirstName).NotEmpty();
					});

				//this.BaseValidation();
			});
		}
	}

If I pass in a customer object which is not null, I expect to validate the first name. However, no validation is fired. Upon further investigation, it appears that DependentRules do not work inside the RuleSet. Is it possible to make it work inside the RuleSet?
Thank you for your help,
Michael

(This is a reply from Jeremy)
Hi Michael,
FluentValidation has moved to GitHub - in future please post your questions over there rather than on codeplex: https://github.com/jeremyskinner/fluentvalidation/issues
So at the moment Dependent Rules aren't compatible with rulesets - I'll get this fixed.
(Additionally, FluentValidation requires that the root-level object being validated is not-null, so your example using RuleFor(o => o).NotNull() won't actually work, but the concept is right)

Hi Jeremy,
Thank you for a swift reply. It would also be super useful for DependentRules to execute any code in inside that code block, not just rules. I've tried doing this yesterday, but it also did not work:
internal class CustomerValidator : AbstractValidator<Customer>
{
    /// <summary>
    ///
    /// </summary>
    internal CustomerValidator()
    {
		RuleFor(o => o)
			.NotNull().WithMessage("Customer request cannot be null")
			.DependentRules(o =>
			{
				RuleSet(RuleSets.Add, () =>
				{
					this.BaseValidation();
				});

				RuleFor(x => x.Id).NotEmpty().WithMessage("Customer id is required")
					.Must(BeAValidCustomer).WithMessage("This customer has been removed or does not exist. Please try navigating to this page again")
					.DependentRules(x =>
					{
						RuleSet(RuleSets.Update, () =>
						{
							this.BaseValidation();
						});

						RuleSet(RuleSets.Delete, () =>
						{
							this.BaseValidation();
						});
					});
			});
    }
}

So I've made the change in master that will enable support for rulesets.
Note that you can't put just any code inside a call to DependentRules, you can only invoke methods on the parameter itself. So this will work:
 .DependentRules(d =>                   {
                        d.RuleSet("Update", () =>
                        {
                            d.RuleFor();...
                            d.RuleFor();
                        });
                    });

But this will not:
 .DependentRules(d =>                   {
                        RuleSet("Update", () =>
                        {
                            RuleFor();...
                           RuleFor();
                        });
                    });

You must invoke the methods on the "d" parameter - otherwise there's no way for FV to know that the rule is dependent.

Jeremy, thank you for support! I haven't been expecting a turned around this quickly!
Is it possible to add support for a method call inside the RuleSet that is encapsulated by DependentRules? I am trying to avoid code duplication and my BaseValidation method holds all the rest of the rules needed for object validation.

No, sorry, the DependentRules implementation doesn't work that way - you can't just make normal method calls inside here- you must call methods on the parameter as I mentioned above.
What you can do though, is modify your BaseValidation method to take a parameter which you invoke rules on. Example:
public class CustomerValidator : AbstractValidator<Customer>
{
    public CustomerValidator()
    {
    RuleFor(...).Whatever().
       .DependentRules(d =>                   {
                        d.RuleSet("Update", () =>
                        {
                            BaseValidation(d);
                        });
                    });
    }

    public void BaseValidation(AbstractValidator<Customer> validator) {
     // Call v.RuleFor rather than just RuleFor
      validator.RuleFor(x => x.Foo).Whatever();
    }
}

Jeremy, that should work! Let me give it a try! One small bug that I found, when I do this:
		RuleSet("Add", () =>
		{
			RuleFor(o => o).NotNull()
				.DependentRules(d =>
				{
					d.RuleFor(x => x.FirstName).NotEmpty().WithMessage("First name cannot be empty");
				});
		});

It works as expected, but the validation error that I am getting is: " -- '' must not be empty " and it's ignoring my custom message.

Hmm, weird. It is working now. Must have been something on my end. I'll be testing more thoroughly today. Jeremy, thank you very much for the support!!

Hi Jeremy,
Trying to understand as why my BaseValidation() method will not fire off in the code below assuming first name is just an empty string. Is there a workaround for that?
internal class CustomerValidator : AbstractValidator<Customer>
{
	internal CustomerValidator()
	{

		RuleSet(RuleSets.Add, () =>
		{
			RuleFor(o => o)
				.NotNull().WithMessage("Customer cannot be null")
				.DependentRules(o =>
				{
					this.BaseValidation(o);
				});
		});

		RuleSet(RuleSets.Update, () =>
		{
			RuleFor(o => o)
				.NotNull().WithMessage("Customer cannot be null on update")
				.DependentRules(o =>
				{
					o.RuleFor(x => x.Id)
						.Cascade(CascadeMode.StopOnFirstFailure)
						.NotEmpty().WithMessage("Customer id is required")
						.Must(BeAValidCustomer).WithMessage("This customer has been removed")
						.DependentRules(y =>
						{
							this.BaseValidation(y);
						});
				});
		});

	}

	internal void BaseValidation(AbstractValidator<Customer> validator)
	{
		validator.RuleFor(o => o.FirstName)
			.Cascade(CascadeMode.StopOnFirstFailure)
			.NotEmpty().WithMessage("First name is required");
	}
}

I've also tried to cascade dependent rules directly. Everything works fine up the first name validation, but last name validation never fires off.
			RuleFor(x => x.Id)
				.Cascade(CascadeMode.StopOnFirstFailure)
				.NotEmpty().WithMessage("Customer id is required")
				.Must(BeAValidCustomer).WithMessage("This customer has been removed")
						.DependentRules(y =>
						{
							y.RuleFor(z => z.FirstName).NotEmpty().WithMessage("First name is required")
									.DependentRules(a =>
									{
										a.RuleFor(b => b.LastName).NotEmpty().WithMessage("Last name is required");
									});
						});
			}

I don't think nested dependent rules will work properly - it was really only designed for going 1 level down.
If you could put together a unit test that illustrates this, then I'll have a play with it and see how much work this'd be to implement.

Hi Jeremy, here are couple of unit tests that I wrote and tested them out in Visual Studio. Both of them are essentially the same and fail currently as they should. Nested dependent rules will be awesome to have. Unfortunately I do not understand fluent framework and a lot of hardcore c# syntax that's used well enough to do this on my own at this point.
	[Fact]
	public void Nested_dependent_rules_inside_ruleset()
	{
		var validator = new TestValidator();
		validator.RuleSet("MyRuleSet", () =>
		{

			validator.RuleFor(x => x.Surname).NotNull()
				.DependentRules(d =>
				{
					d.RuleFor(x => x.Forename).NotNull()
						.DependentRules(y =>
						{
							y.RuleFor(x => x.Address).NotNull();
						});
				});
		});

		var results = validator.Validate(new Person { Surname = "foo", Forename="foo" }, ruleSet: "MyRuleSet");
		results.Errors.Count.ShouldEqual(1);
		results.Errors.Single().PropertyName.ShouldEqual("Address");
	}

	[Fact]
	public void Nested_dependent_rules_inside_ruleset_inside_method()
	{
		var validator = new TestValidator();
		validator.RuleSet("MyRuleSet", () =>
		{

			validator.RuleFor(x => x.Surname).NotNull()
				.DependentRules(d =>
				{
					d.RuleFor(x => x.Forename).NotNull()
						.DependentRules(y =>
						{
							BaseValidation(y);
						});
				});
		});

		var results = validator.Validate(new Person { Surname = "foo", Forename = "foo" }, ruleSet: "MyRuleSet");
		results.Errors.Count.ShouldEqual(1);
		results.Errors.Single().PropertyName.ShouldEqual("Address");
	}

	private void BaseValidation(DependentRules<Person> validator)
	{
		validator.RuleFor(x => x.Address).NotNull();
	}

Hi Jeremy! Just wondering if you had a chance to play around with it and see if this is something that can be done without requiring too much effort. Thank you for all your help!

I haven't had a chance yet I'm afraid...I've got a pretty big backlog of FV issues to look at, I promise I will get to it though!

Thank you Jeremy! No worries, whenever you get a chance.

@JeremySkinner did You had time to look at this? I'm building complex validation class and multiple DependentRules would help me a lot. For now I'm using When but this way I don't get correct error messages.

Sorry, no- I'm not currently doing any work on FluentValidation as I'm busy with other commitments, so unless someone wants to contribute a pull request I'm afraid this will have to wait.

@JeremySkinner as I wrote I have a workaround for that. Sadly I'm not that advanced developer to change this.

This is actually implemented as of 7.4, but you must use the new/updated DependentRules syntax:
RuleSet("MyRuleSet", () => {
  RuleFor(x => x.Surname).NotNull().DependentRules(() => {
    RuleFor(x => x.Forename).NotNull().DependentRules(() =>	{
      RuleFor(x => x.Address).NotNull();
    });
  });
});

Note that I'm using a new overload of DependentRules that takes a simple action, rather than a DependentRulesBuilder. Inside the action, I now calling the root-level RuleFor method instead of calling d.RuleFor.
This is the recommended way of using DependentRules going forward.


Existing user info with mofh-client

This is a longshot - but is it possible to migrate over existing MOFH client information into my database using mofh-client? I have created my portal in Laravel, which lists users' hosting accounts, allows for creation of new hosting accounts, etc. To enable the ability of automatic cPanel login, I insert the chosen password on new account creation into my database (using an encryption algorithm and key). I also capture the mofh_username and currently logged-into Laravel user ID. These are inserted into my database as well.

mofh_username
laravel_user_id
password_enc
password_key

host_14307665 l
1
1igSThVvj4EfImIjWg==
e4d909cd0ca068ffaddf22cbd0

host_26834112
2
uIewK8nB42lOi9==
9e107d72bb68d81d35a419d6

When a user account is validated (logged-in laravel_user_id = mofh_username->laravel_user_id), the password_enc and password_key are grabbed from the database, and the decrypted password is injected into the login form which instantly POSTs through to the MOFH cPanel. Is there a better, more secure way of doing this than requiring direct PHP injection of the password into the form? My process works for new user accounts created using the portal (since I associate the MOFH username with the laravel_user_id), but the 150 existing accounts would not be able to have their account logged into from the client panel with this current method. Is there a better way to do this when using mofh-client besides closing all accounts and starting over, ensuring account creation is only done from the client panel in the future?

Is there a better, more secure way of doing this than requiring direct PHP injection of the password into the form?

Not really. You can generate login links to the control panel (by reverse engineering the login links in the reseller panel), but those seem to be intended for admins. And you're basically sending the password in the URL (albeit obfuscated), so the security benefits are negligible.

My process works for new user accounts created using the portal (since I associate the MOFH username with the laravel_user_id), but the 150 existing accounts would not be able to have their account logged into from the client panel with this current method. Is there a better way to do this when using mofh-client besides closing all accounts and starting over, ensuring account creation is only done from the client panel in the future?

Well, there are a couple of problems you run into when importing existing accounts:

You don't have user information in your database. If or how you can correct that is up to you and your application.
You don't have any records of the password. You could forcefully reset the passwords of all accounts from your application or urge users to update their own passwords.
The main domains from the old users are not accepted by the API. The API has hard coded validation rules which ensure that the custom username must be exactly 8 characters. The regular registration form can support usernames between 6 (?) and 15 characters, which the API will simply refuse to accept (I'm not sure if 8 character main domains actually works - I never bothered to test). So even with all the data in your database, you cannot manage the accounts through the API.

I can't say for certain if it's possible or feasible to import existing accounts. I can tell you that it's hard. Whether you want to try or just want to remove or forget these accounts, that's all up to you.

Thanks for the detailed reply @HansAdema. I mostly dabble in frontend design, so I havent done backend work in a long time. It takes a deep thought process to solve some of these issues. Lastly, could you inform me of the best way to generate this 'username' as part of account creation? Initially, I just used the chosen subdomain from the form post data as the username, since these subdomains must be unique anyway. But then I came across the custom domain names. I implemented that, too, but the problem is I have to manually specify a unique username in that case, since the user is not providing a subdomain. I could require a unique username be chosen only if a user also provides their custom domain, but do you also know of any other better way for this handling of the unique username identifier too? I notice the username is generated automatically on your host. Perhaps I am overthinking this:
If user chooses subdomain =>

User fills in subdomain field
This subdomain field must be unique, is used for the username, and stored in the database as unique_username

If user chooses custom domain =>

User fills in custom domain field
User is also then given an additional "username" field, which must be unique, and is stored in the database alongside the other unique_username values.

Basically, if registering with a subdomain, use the subdomain as the unique username. Or, if registering with a custom domain, require the filling in of an additional unique username input field.
But, this way, there might be conflicts when a user using a custom domain name chooses, for example, the username "bookstore", because then a user wanting the subdomain "bookstore" won't be able to register it. As I said, I might be overthinking it - I should just provide a username field for everyone - subdomain or custom domain sign-ups. However, I may be thinking something through incorrectly seeing as the username is generated automatically on your host. I've just been trying to find a way to not require manual entry of the unique username. Please let me know if this solution is logical and in line with what you would do or have done. Thanks again.

On InfinityFree, the username is randomly generated (using the Str::random() function IIRC).
Also, note that:

The domain field MUST be provided, and can be a custom domain or subdomain.
The provided username will be listed as the main domain, but will not actually be configured to host a website. IIRC, using the provided domain name for the username can cause weird issues, so I would not do that.

Also, in your example, the bookstore username would not be accepted, because it's 9 characters instead of 8. This is a result of the API having to much focus on replicating the cPanel API, rather than just offering compatibility with it.

Thank you, your client has worked great (within the limitations of MOFH’s API, which could use a number of improvements). All seems to be working perfectly on my end.

@HansAdema On InfinityFree, how do you store hosting account passwords? Do you store it as plain text or is there a way to encrypt it?
@javaskript I agree, the MOFH API needs a V2, preferably JSON based instead of this strange XML/plain text combo.

On InfinityFree, how do you store hosting account passwords? Do you store it as plain text

Passwords are encrypted, but only with a shared master key. Unlike @javaskript, I don't store the encrypt key next to the encrypted password, because that's about as secure as locking a door and hanging they key next to it.

or is there a way to encrypt it?

Your application stores the data. If you can encrypt stuff, you can encrypt the password as well.

Okay. I always just assumed it was stored as plain text because of it only being for the hosting account.


np.pad has not been implemented

The current version 0.21.0 of Tensorflow.NET does not implement np.pad()

@Banyc If possible, you need to get familiar with porting Unit Tests from native from google to Tensorflow.NET as discussed here.
Then if will have the "helicopter" view where are the missing parts and where the community can contribute in complementary way.

@Banyc Please use tf.array.pad instead of np.pad.


#anchors Slick removes padding/margin when receiving anchor from an external page

Here's my website: http://villamorapharmacy.com/vince/links/journals.html#art1 as you can see when clicking the link above the slick is too close to the jumbotron when I refresh it, it comes back to normal. And it's not an issue with bootstrap either since I tried removing/unslicking the article there is no issue with padding/margin.

Can you reproduce this in the JSFiddle link in CONTRIBUTING.markdown? I need it isolated to properly debug.

This should be the exact code I'm using now IDK how to reproduce it when loading the code with an anchor...

It seems like the adaptiveHeight feature is causing slick to offshoot from the container, I've tried removing it and it seems to work normally.

Hey I know this issue is super old, but it looks like it's working in this example, even with the adaptive height.
I'm going to go ahead an close, but if you see any issue we can look at reopening. Thanks!


Consolidate ExtensionMethods.cs and HtmlExtensions.cs in NuGetGallery

Right now, there are a lot of extension methods on HtmlHelper<TModel> in both ExtensionMethods.cs and HtmlExtensions.cs. I think all of these should be in HtmlExtensions.cs.

ping?


what time authorization type implements the password grant

what time authorization type implements the password grant

repeated submit

repeated submit


[fix] deliver_method must be symbol.

And authentication should be symbol

Coverage remained the same at 83.176% when pulling 05ae750640c51f0fbb8c62e77833a507b43a440a on sunny4381:fixes/webmail into 523ec2daa3793f8ef6774444cd48ec1a54bc67df on shirasagi:master.


add menu item for Events

Fix 11.8 Events link in http://jeffreykegler.github.io/Marpa-web-site/libmarpa_api/stable/api_one_page.html

I'm probably going to just let this sit until I fork Kollos's Libmarpa.
On Sat, Aug 15, 2015 at 5:44 AM, rns<EMAIL_ADDRESS>wrote:

Fix 11.8 Events link in
http://jeffreykegler.github.io/Marpa-web-site/libmarpa_api/stable/api_one_page.html
You can view, comment on, or merge this pull request online at:
https://github.com/jeffreykegler/libmarpa/pull/69
Commit Summary

add menu item for Events

File Changes

M work/dev/api.texi
https://github.com/jeffreykegler/libmarpa/pull/69/files#diff-0 (1)

Patch Links:

https://github.com/jeffreykegler/libmarpa/pull/69.patch
https://github.com/jeffreykegler/libmarpa/pull/69.diff

—
Reply to this email directly or view it on GitHub
https://github.com/jeffreykegler/libmarpa/pull/69.

Sure, that's fine with me.
On Sat, Aug 15, 2015 at 8:49 PM, Jeffrey Kegler<EMAIL_ADDRESS>wrote:

I'm probably going to just let this sit until I fork Kollos's Libmarpa.
On Sat, Aug 15, 2015 at 5:44 AM, rns<EMAIL_ADDRESS>wrote:

Fix 11.8 Events link in

http://jeffreykegler.github.io/Marpa-web-site/libmarpa_api/stable/api_one_page.html

You can view, comment on, or merge this pull request online at:
https://github.com/jeffreykegler/libmarpa/pull/69
Commit Summary

add menu item for Events

File Changes

M work/dev/api.texi
https://github.com/jeffreykegler/libmarpa/pull/69/files#diff-0 (1)

Patch Links:

https://github.com/jeffreykegler/libmarpa/pull/69.patch
https://github.com/jeffreykegler/libmarpa/pull/69.diff

—
Reply to this email directly or view it on GitHub
https://github.com/jeffreykegler/libmarpa/pull/69.

—
Reply to this email directly or view it on GitHub
https://github.com/jeffreykegler/libmarpa/pull/69#issuecomment-131408951
.

Thanks!  I decided that rather that it's probably easiest to tag my attempted fork point, and let git do what it does, rather than artificially hold back changes.

Great!
On Sun, Aug 16, 2015 at 12:20 AM, Jeffrey Kegler<EMAIL_ADDRESS>wrote:

Thanks! I decided that rather that it's probably easiest to tag my
attempted fork point, and let git do what it does, rather than artificially
hold back changes.
—
Reply to this email directly or view it on GitHub
https://github.com/jeffreykegler/libmarpa/pull/69#issuecomment-131448864
.


feat: implement the trello plugin

Summary
implement the trello plugin
Does this close any open issues?
Closes #1881
Screenshots

Other Information
This plugin collects data from the Trello API and transforms the raw data into the tool layer.
I didn't implement the task of converting the data to the domain layer because Trello is a flexible work management tool, and some fields cannot be converted to those in the issue with fixed rules, such as type and status.

ok, I will fix them

Hey @jakezhu9 , excellent work! We'll take a look at this PR. Are you in our community by the way?

Hi @Startrekzky, I've joined Slack. Also, the CI errors has been fixed! I'm hoping this pr will help improve DevLake even further :)

Hi @Startrekzky, I've joined Slack. Also, the CI errors have been fixed! I'm hoping this pr will help improve DevLake even further :)

Hi @jakezhu9 .I have tried to use your plugin but failed to test connection at first.

and I have checked the document of oauth.
It seems you don't use the full way for OAuth.

I can only get Consumer Key and Consumer Secret.
So I don't know how you config the connection to run it.
There are some codes from the document about how to use OAuth1.0 for Trello.
https://glitch.com/edit/#!/trello-oauth?path=server.js%3A37%3A0
Maybe It can help you.

Hi @mappjzc. You're absolutely right, I didn't fully implement oauth because it's a bit more complicated to set up.
Instead, I've been using a combination of an API key and token for authentication. You can find instructions for configuring this in the README and on the Trello developer site.
One thing I should mention is that I haven't found a great way to store the API key and token in the plugin helper. Right now, I'm using helper.BasicAuth, which means you'll need to put the API key in the username field and the token in the password field. I know this isn't a good design and could cause some confusion.
If you have any suggestions for how to improve this, please let me know and I'll work on making it better!

@jakezhu9 @mappjzc I see, I think it would be better to use the AppKey instead of the BasicAuth.
Note that we can modify the req.Query in the SetupAuthentication method as well to avoid confusion regarding which authorization method we opted for.
Keep in mind we should add a document to https://devlake.apache.org/ at the end of the development to explain the authorization etc.
Thanks for your hard work, appreciated it. 💪

I have already switched to using the AppKey. I will add a document to explain the authorization process at the end of development. Thank you for your help and support, it is greatly appreciated.

@jakezhu9 @mappjzc I see, I think it would be better to use the AppKey instead of the BasicAuth. Note that we can modify the req.Query in the SetupAuthentication method as well to avoid confusion regarding which authorization method we opted for. Keep in mind we should add a document to https://devlake.apache.org/ at the end of the development to explain the authorization etc. Thanks for your hard work, appreciated it. 💪

Thanks for your help.
I had already succeeded to test your connection. It works.
And I try to run a blueprint for it, But here have some problems.

As you see, It sends out the error that "boardId is required".
This error comes from PrepareTaskData as follows.

It decodes from the map option to TrelloOptions.

And in TrelloOptions, It required BoardId with json field "boardId"
But in the blueprint, there is only filed "scopeId" in the JSON string.

To see the makePipelinePlanV200 of your code, you use the filed "scopeId"  in the options JSON string.

Maybe you need to unify them in the way you like.
After I fixed this on my locally, this plugin can work successfully.
And another little thing. the CollectCheckItem and CollectChecklist Collect the same data.
It is enough only one.

@mappjzc Thank you so much for reviewing my work carefully! I just realized that I made a mistake in the boardId and scopeId, but I'll get it fixed right away. I'm also working on adding blueprint_test to prevent similar issues from happening in the future.
As for CollectCheckItem and CollectChecklist, my original idea was to keep them separate since they are two different tasks. After giving it some thought, I agree with you that merging them into one task would be a better approach. I plan to keep only CheckItem and include the Checklist data within each item, as Checklist has only a few fields.

Already fixed. There is an error about the commit message in ci, but it doesn't seem to be my reason, just ignore it.

Already fixed. There is an error about the commit message in ci, but it doesn't seem to be my reason, just ignore it.
@jakezhu9

Thank you so much.
It is a great pull request.
Can we have the privilege of obtaining more contact information about you for more convenient communication?
Like WeChat or others that you prefer to use?

@mappjzc Absolutely! My WeChat ID is jakezhu9, and you can also reach me at<EMAIL_ADDRESS>


feat(module:type): support string literal union and auto complete

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
不能出现智能提示，原因是联合类型将父子类型合并只保留了最高类型:

What is the new behavior?
支持智能提示:

Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
主要是实现一个类型。
这个类型的功能是，将字符串字面量类型和字符串类型进行联合，而不是合并简化：
type NzStringLiteralUnion<T extends string> = T | (string & {});

Reference: https://github.com/Microsoft/TypeScript/issues/29729

另外你们是不是没有精力维护了：

要不是 VSCode 牛逼，我到死估计都不知道是什么问题导致我启动不了项目。。。


How to use ArangoDB as a data source on Appsmith

Please cover the following topics in this guide -

Integrating the datasource into Appsmith.
Fetching data from the datasource connected.
Binding the data to a widget.
For documentation guidelines, click here.

Hi! I'm interested in this issue. Can you assign it to me?

Hi @Pranay105, is the issue still open? I would like to work upon the issue.

Hi @Pranay105, is the issue still open? I would like to work upon the issue.


Release OCP <IP_ADDRESS>

Hi!
I see that OCP version <IP_ADDRESS> was released in May. I'd really like to use it's functionality (IMesh_Tools) in my meshing package https://github.com/stellarmesh/stellarmesh. Is there a plan or timeline for releasing this version on PyPI?
Thanks!

The timeline is as soon as possible. We've been fighting some build issues though. Hopefully within the next week or so we will be able to release <IP_ADDRESS>.

Fantastic @jmwright great to hear

All - Please download the correct wheel for your platform and Python version and do a test installation. Once these have been tested, I will publish them to PyPI. During the installation, cadquery-vtk will be downloaded from PyPI since it is no longer embedded inside the OCP wheel.
https://github.com/CadQuery/ocp-build-system/releases/tag/<IP_ADDRESS>

This has been completed.
https://pypi.org/project/cadquery-ocp/

There was an issue with the VTK dependency that is causing this wheel not to work properly for certain use cases, so I am reopening this issue.


update instructions for a minimal local setup

I ran into some errors when getting my local dev environment set up. Minimal.env.json seems to be more appropriate to copy for a quick local setup of all the services, rather than full.env.json.

🎊 Thanks for your contribution @nellshamrell!


Proposal Draft: Support Timestamps and Labels for Individual Events

Motivation
My team at Datadog is using pprof as our data collection format for profiling data (cpu, wall clock, lock contention, allocations, etc.) coming from non-JVM languages (which use JFR). This works very well for us, and we'd like to continue building on pprof. However, there are two use cases we are interested in that are currently difficult to implement efficiently using pprof.
This proposal outlines these use cases as well as potential changes to the profile.proto format that would support them. We believe that these use cases would also be useful to the OSS community. Additionally the proposal should meet the signal correlation requirements of the Profiling Vision OTEP, which might allow pprof to be adopted by OpenTelemtry.
Last, but not least, we hope that this proposal would ease the adoption of these features by the Go runtime, e.g. by adding timestamps to CPU profiles.
UC 1: Thread Timelines (FlameChart)
We would like to break down the self-time of distributed tracing spans using per-thread (or goroutine) flame chart timelines. The data for this would be collected using wall clock sampling, tracing techniques, or a combination of both. This is similar to what fgtrace does, but would only show the threads relevant to the span:

In order to support this use case, pprof would need the ability to efficiently store timestamps, thread ids and span ids for the individual profiling events that are being recorded. Additionally the pprof UI might be extended to show this data, or the command line could be extended to target perfetto or other flame chart visualization tools.
UC 2: CPU Heatmaps (FlameScope)
Another use case we are interested in is the display of of CPU Heatmaps pioneered by FlameScope which provide a more powerful way to understand application behavior.

This use case is simpler than the first use case and only requires timestamp information for individual CPU samples. Additionally the UI could add this visualization or the command line could output the data in a format understood by flamescope.
Example
Let's say we have a wallclock profile that contains 3 events:

time since start
duration
stack trace
trace id
span id
thread id

10ms
15ms
a;b;c;d;e;f;g;h
1
1
1

25ms
5ms
a;b;c;d;e;f;g;h
1
1
1

40ms
20ms
a;b;c;d;e;f;g;h
2
2
2

Note: For the purpose of this example the stack trace is the same for all 3 events.
How could this data be encoded into pprof?
Using Labels (Today)
With the existing profile.proto, one may use labels to store the data like this (shown in pseudo pprof format):
sample_type: [walltime/nanoseconds]

sample {
  location_id: a
  location_id: b
  location_id: c
  location_id: d
  location_id: e
  location_id: f
  location_id: g
  location_id: h
  value:<PHONE_NUMBER>
  label: {time_ms=10}
  label: {trace_id=1}
  label: {span_id=1}
  label: {thread_id=1}
}

sample {
  location_id: a
  location_id: b
  location_id: c
  location_id: d
  location_id: e
  location_id: f
  location_id: g
  location_id: h
  value: 500000000
  label: {time_ms=25}
  label: {trace_id=1}
  label: {span_id=1}
  label: {thread_id=1}
}

sample {
  location_id: a
  location_id: b
  location_id: c
  location_id: d
  location_id: e
  location_id: f
  location_id: g
  location_id: h
  value:<PHONE_NUMBER>
  label: {time_ms=40}
  label: {trace_id=2}
  label: {span_id=2}
  label: {thread_id=2}
}

Unfortunately this requires to encode the location_id list three times b/c the Sample message has no concept of referencing a stack trace by id and each unique combination of labels requires us to have its own Sample. In practice this leads to very large pprof files.
Using Breakdown (Proposed Change)
Using the proposed profile.proto change in this PR, the same data could be encoded like shown below. In practice, this should lead to much smaller pprof sizes.
sample_type: [walltime/nanoseconds]
tick_unit: ms

sample {
  location_id: a
  location_id: b
  location_id: c
  location_id: d
  location_id: e
  location_id: f
  location_id: g
  location_id: h
  value:<PHONE_NUMBER>
  breakdown: {
    tick: 10
    tick: 25
    tick: 40
    value: 15
    value: 5
    value: 20,
    label_set_id: 1
    label_set_id: 1
    label_set_id: 2
  }
}

labelSet {
  id: 1
  label: {trace_id=1}
  label: {span_id=1}
  label: {thread_id=1}
}

LabelSet 2 {
  id: 2
  label: {trace_id=2}
  label: {span_id=2}
  label: {thread_id=2}
}

Efficiency
This proposal can lead to significant size reduction of uncompressed pprofs compared to using the existing label mechanism when adding timestamps and trace ids. Workloads with less unique stack traces and longer profiling periods gain the most benefits from this proposal. E.g. in the examples below the gains are from 1.4x to 3.2x, but it's possible to construct examples that see bigger or smaller improvements.
That being said, the benefits mostly disappear after gzip compression. E.g. the same examples see only 1.01x to 1.22x compression gains. This is perhaps not surprising given that the duplicated stack traces are easy targets for compression.
For more details see felixge/pprof-breakdown and raw results (spreadsheet).

Compatibility
The proposed change is fully backwards compatible, but not fully forwards compatible when it comes to labels. The new format puts event specific labels into the new Breakdown message. This means that profiles taking advantage of the new profile.proto Breakdown labels won't be completely accessible to old versions of the pprof tool or other tools consuming pprof information. Those tools could still render basic flame graphs, but the label information would be invisible to them.
Next Steps
We're looking for feedback from the pprof maintainers to understand if "Support Timestamps and Labels for Individual Events" is a compelling feature or if the pprof project would rather stay focused on storing pre-aggregated data. Our suggest changes to profile.proto should mostly be seen as a demonstration of the technical feasibility of implementing such a proposal, but we're not too attached to the implementation details and happy to adjust them as needed. Perhaps the small gains on compressed pprofs are not sufficient justification for the added complexity, and a simpler proposal to standardize the usage of the existing label mechanism for the use cases outlined above would offer better tradeoffs.
Additionally we'd like to get feedback from the OSS community and potentially OTel group to make sure the outlined use cases are clear and useful to others.

Awesome! I say go for it :) The one thing that's not clear to me, is what you do if some ticks in a breakdown have labels and some don't? In other words, can a breakdown have a list of label set IDs, where some of them are 0 to indicate "no label set for this tick"? That should be OK since 0 is defined to not be a valid label set ID.

Yeah, when there is no label my current Go fork just puts label_set_id: 0 into the pprof 👍

Simple grammar adjustment in the proposal:

Our suggest changes to

Our suggested changes to

Closing this. Official discussion is here now: https://github.com/google/pprof/pull/728
@morrisonlevi will apply your suggestion there now.


Update IP table to contain organization uid

💡 Summary
Modify the IP address table to store organization ID as a FK
Motivation and context
This will allow linking organizations to individual IP addresses in the event they are not part of a CIDR block
Implementation notes

Individual IP addresses will be attributable to the organizational owners
Queries and lookups can pull directly from the IP table without having to map CIDR tables

Acceptance criteria

[ ] OrgID is a Foreign Key in the IPS table

Pushed to staging, and db is updated to have correct columns


[LPC11U68] fix Uart1-4 irq handling bugs, add Pins P0_14,P0_13 for use with Uart1

Hello,
thanks for the PR. Can you please provide some information for this change set? I'll add comments what is not clear. You can write a commit message longer where youc an provide more details to the change-set. There are few bugs fixed in this one, would be beneficial to know more details about those.

to clear any combination of bits in the INTENSET register you have to write a 1 to the corresponding bit position in the INTENCLR register. INTENSET can only be used for setting!
referring to UM10732 page 199
All interrupts are disabled if in INTENSET register no bit is set so only then its safe to turn off global NVIC interrupt, at the moment only TX and/or RX is getting set so old implementation would work too.


Did a different thing and unbroke git

Git was being bad... Clearly it wasn't user error, as I am perfect and
can do no wrong :neckbeard:

💯


hdf5widget: add import hdf5plugin before h5py

try importing hdf5plugin before h5py, so that some compressed files can be opened without being struck with the "can't open directory" error.
Maybe advise to add hdf5plugins if a dataset cannot be read with an OSError

It is more a responsibility of the application. We can put it in the examples.

Yes, I was thinking about adding an import to the hdf5widget.py in silx/examples:
try:
# This import must be done before h5py
import hdf5plugin
except:
print("You should install the hdf5plugin package [url] to support XX hdf5 compression")

Yes, it is exactly what it will be done

It seems HDF5 or h5py do not properly set the dataset attribute compression when an external filter has been used for compression.
At least I could found a way to know what filters have been used on a dataset:
import h5py
h = h5py.File("lz4.h5", "r")
d = h["/entry/data"]
for i in range(d._dcpl.get_num_filters()):
    print(d._dcpl.get_filter(i)

That would be very useful. Only catch, is that if they follow PEP, _dcpl is a non-public API. Would be nice to have a stable access.

Ok, let's be purists although I do not think they will change that.
I have taken a look at how _dcpl is generated and we can replace it by the public API
import h5py
h = h5py.File("lz4.h5", "r")
d = h["/entry/data"]
dcpl = d.id.get_create_plist()
for i in range(dcpl.get_nfilters()):
    print(dcpl.get_filter(i))


Windows goes to sleep causing Android machine memory to increase

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS: Windows
scrcpy version: 2.4
installation method: Windows exe
device model:rk3588
Android version: Android12、Android13

Describe the bug
I am using Windows to interact with rk3588 development and found that the memory usage is increased
Causes the memory to be full, terminating all running background programs
I think it may be caused by the data packet not being released.
Here is how I tested Reappear soon:First open scrcpy and enter the desktop, then open the system camera  --->stand still,Next, put your computer Windows into sleep mode.
Select Sleep on the power button in the lower left corner of Windows and wait for 1 minute before waking up.
Then open adb shell and execute top to find that the memory usage is more than 80%.  As shown

I am using 16GB of RAM
I have tried Android 12 and Android 13 and both have this problem
This happens only when the computer is shut down after sleeping and the adb connection is disconnected. I have also tried disconnecting the adb cable and this does not happen.
Normal memory is as shown in the figure

I have finished talking about the above questions. If you need testing, I am willing to test the software for you.
Thank you!

I used this software and it did not increase the memory capacity. https://www.vysor.io/

Thank you for the report.
Given the description, it looks like the device buffers input data indefinitely if it is not consumed by the client (applying a backpressure). But if I try to "freeze" (Ctrl+z on Linux) the client, I cannot reproduce on my device.

First open scrcpy and enter the desktop, then open the system camera --->stand still,Next, put your computer Windows into sleep mode.

I fail to reproduce: when I put a Windows into sleep mode, when I wake it up, scrcpy is disconnected (the adb socket is closed probably).
Can you reproduce with a different device (other than rk3588)?
Does it also happen with another encoder?

Also, does it happen if you capture the camera directly (instead of mirroring the screen and opening the camera app)?
scrcpy --video-source=camera

Sorry, I didn't make that clear.

First open scrcpy and enter the desktop, then open the system camera --->stand still,Next, put your computer Windows into sleep mode.

Open scrcpy software in Windows
Use the mouse to open the system camera on the scrcpy desktop
Put the computer to sleep
Wait for 1 minute
Wake up the Windows computer again
Use adb to connect to the Android device and execute top to check the memory usage to see if it has increased
This is how I did it, please try again

My Xiaomi phone 12GB also has a similar situation, which does cause the memory to increase

For Xiaomi, I do this: I use scrcpy on the computer to open the Xiaomi 12 mobile phone camera, and then the computer goes to sleep.
At this time, the mobile phone camera is always open and not closed.
The memory starts to increase slowly.
It is recommended to wait 1-2 minutes before waking up the computer during sleep, open adb shell and execute top to find that the memory is occupied a lot.

On the device, run top with these arguments:
top -o PID,USER,CMDLINE,%MEM`

It will list the process sorted by memory usage.

It seems that the application does not take up memory

hi is there any solution?
If I haven't explained it clearly, we can contact you via Google chat and I will tell you the steps to reproduce the problem.