Switch to flake8 for static code analysis

flake8 catches a wider net of mistakes than pycodestyle and is more
commonly used by the larger community. For example, it catches unused
imports, a few of which existed. These have since been removed.
Two "noqa" comments were added. One ignores the _compat.py file as it
has a large amount of Python version specific code. The second is in
utils.py which intentionally does not use an import.
Pull Request check-list
Please make sure to review and check all of these items:

[x] Does $ tox pass with this change (including linting)?
[x] Does travis tests pass with this change (enable it first in your forked repo and wait for the travis build to finish)?
[x] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Codecov Report

Merging #1328 into master will decrease coverage by 0.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1328      +/-   ##
==========================================
- Coverage   92.88%   92.70%   -0.18%
==========================================
  Files          20       20
  Lines        6545     6377     -168
==========================================
- Hits         6079     5912     -167
+ Misses        466      465       -1

Impacted Files
Coverage Δ

redis/_compat.py
87.80% <ø> (+0.70%)
:arrow_up:

redis/client.py
86.24% <ø> (-0.03%)
:arrow_down:

redis/connection.py
82.30% <ø> (-0.05%)
:arrow_down:

redis/utils.py
66.66% <100.00%> (-2.09%)
:arrow_down:

tests/test_multiprocessing.py
77.89% <0.00%> (-0.90%)
:arrow_down:

tests/conftest.py
96.62% <0.00%> (-0.25%)
:arrow_down:

tests/test_encoding.py
97.53% <0.00%> (-0.09%)
:arrow_down:

tests/test_connection_pool.py
97.77% <0.00%> (-0.07%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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

Ya, should have done this sooner. I use flake8 in my dev environment already :)


Wishlist: expose GestureDetector's TapDownDetails

GestureDetector exposes a TapDownDetails object onTapDown. I want to display a menu at the position of the tap on long press, but I can't do that at the moment since I need the coordinates of the tap. Those are in the TapDownDetails object.
It would be nice if flutter_calendar_carousel could expose this object, either by letting me add a listener onTapDown, or providing TapDownDetails to the already existing onLongPress handler.
Cheers for a nice library!

Still valid


Listhubs Improvements

[ ] Implement form validation for controls
[ ] Link To Homepage at end of form input
[ ] Clear form state of user session

Minor issues to fix on v1 iteration:
-Allow freeform + additional amenities
https://www.figma.com/file/lSLF4RzNimKet5YrupZBht/dApp?node-id=290%3A2257
-Make first set of 3 rules default.  (If we want, we can have the person making the reservation check all three of the default rules before reserving)
-Availability section to just copy format from Calendly (see product_mvp channel)
-Minor spelling fixes:
"Air Conditioning"
"Equipped"

@angelo-build  do you think we need a name or title property on the hub model? I believe its a human friendly way of identifying hubs example "Beautiful 3 bedroom on the beach", a further description can be added in the description field to elaborate

Yes, We should allow them to give a title to the hub, great call.  For example the Idaho hub will be called "The Den" most likely so that should show up as the title of the hub


KMail Renders Blank Messages

Description
The title says it all. Here is the corresponding upstream bug report:  KDE Bug Tracker .
The workaround of QT_WEBENGINE_DISABLE_WAYLAND_WORKAROUND=1 solves the problem.
Maintainers
@ttuegel @vandenoever @nyanloutre

Where do you put the env var? I have it in ~/.profile and ~/.bashrc contains source .profile, but when starting KMail from the task manager it still shows the bug, while it works fine when started from a terminal.

I confirmed (by looking at string /proc/${kmail-pid}/environ) that QT_WEBENGINE_DISABLE_WAYLAND_WORKAROUND=1 is set. Still KMail does not render the contents of emails when started from Plasma, but does so when started from a terminal (the envvar is set in both cases).
To further debug this, I compared environment variables from the different processes and used export and unset to get both environments to match as closely as possible.
In the end I found that if NIX_PROFILES=/run/current-system/sw /nix/var/nix/profiles/default /etc/profiles/per-user/dschridde /home/dschridde/.nix-profile /var/lib/flatpak/exports /home/dschridde/.local/share/flatpak/exports was set, KMail would render emails, but if it was not set, it would output:
grantlee.template: "Plugin library 'kde_grantlee_plugin' not found."
org.kde.pim.messageviewer: "Plugin library 'kde_grantlee_plugin' not found." . Searched in subdir mimetreeparser/themes/default in these locations ("/home/dschridde/.local/share", "/nix/store/r0iadjy142dwjv8shn709b2b596lcf80-kmail-22.08.2/share", "/nix/store/hbh1vssyd03kipgza9jhq16ll01mfgdc-pim-data-exporter-22.08.2/share", "/nix/store/hi803yfn16fxkxan3flyw4sbbxad81q3-kleopatra-22.08.2/share", "/nix/store/1n3nsj89invp1kcj5jhjcgbz2g08mz8z-kaddressbook-22.08.2/share", "/nix/store/jbrydf95y7c7nadi0srl5dz9dhgh4m2z-akonadi-import-wizard-22.08.2/share", "/nix/store/3d7skyq2r9aqqy69amhmjvafb3mizxs7-pim-sieve-editor-22.08.2/share", "/nix/store/5a3k6k06abh6wzpgll6hadvsnxd9gc5z-attr-2.5.1/share", "/nix/store/nx8hvigvlyzm6h59md4gi5bxpsw3panc-xz-5.2.7/share", "/nix/store/fa1slp7raq5a0wrygmcs3nig1ysi69pd-libkleo-22.08.2/share", "/nix/store/5gn5s91x87jn57fvw91ap2f20r4vpdwz-libgpg-error-1.45/share", "/nix/store/2111axif7wdziy1czxh6hvbackgjn45y-kidentitymanagement-22.08.2/share", "/nix/store/xk8kkp48czp1gh9riglz5k1kp9qsms7v-mailcommon-22.08.2/share", "/nix/store/pqvfzcri5pncybs5q4p1mx8855fkkagj-messagelib-22.08.2/share", "/nix/store/1j6pfmxrrh13655lvmraqdbpyf56nj12-libsecret-0.20.5/share", "/nix/store/jwzbvzazhsrh3kv4h1cvhsfpwb3rg8by-libksieve-22.08.2/share", "/nix/store/lcn1cllv8ccncarcbc9zhv7cza6gy0vb-libgravatar-22.08.2/share", "/nix/store/7rwszv3aq0l177zcsa5qqmg9hm35lbw4-pimcommon-22.08.2/share", "/nix/store/145hp36ckr61lbnrrh9ngzpp5byj433k-purpose-5.99.0/share", "/nix/store/2v2w2hzaf7nffqc9ibjacwbmk8qv3chd-libkdepim-22.08.2/share", "/nix/store/vqga6q96a4icc5a949candpb638l3qh9-purpose-5.99.0-bin/share", "/nix/store/f2h17swsqyb3rsw5664fq6cknnyn4sqw-kimap-22.08.2/share", "/nix/store/3wbzjr50v42r0g592a49yy0f2k6aacyw-akonadi-contacts-22.08.2/share", "/nix/store/mbp13p0g3k94qf53224ynlbj36k8jrpg-ktnef-22.08.2/share", "/nix/store/yq41xrshprwpabx3396rn26410lgj02b-kpty-5.99.0/share", "/nix/store/kli0yrdv666i2vl67hprfm6v675k1x7m-kontactinterface-22.08.2/share", "/nix/store/8j2adsskmz5fz6j7r93wb4yzid03qv7w-knotifyconfig-5.99.0/share", "/nix/store/ggnpm72fv59xvc2n33n9n0p87arqwryp-kmailtransport-22.08.2/share", "/nix/store/ny449rka6d673cvczj4sq8ihid2ar941-kmail-account-wizard-22.08.2/share", "/nix/store/di7w7l3ajrls2rjfvappdxdp1pp20ckf-kldap-22.08.2/share", "/nix/store/yc627hhpzli854q6w19y6fffn8q9da6z-kinit-5.99.0/share", "/nix/store/495ckkq4h559h5c87dmwpwrsn968fdg5-knewstuff-5.99.0/share", "/nix/store/4w1dkfv83xn1w09rlq33wllfriby2bfq-grantleetheme-22.08.2/share", "/nix/store/df66vl80h3j6a21wrpddf52g97khwy9i-kdepim-addons-22.08.2/share", "/nix/store/k1qpxmd5zscgg5qvk0lfwg27i5hxyivs-kcalutils-22.08.2/share", "/nix/store/slb05rfzm0rkffimy1ayc2c8laqq56zf-syntax-highlighting-5.99.0/share", "/nix/store/43xsp7km042jmham8panc7q6qacr6i78-kparts-5.99.0/share", "/nix/store/n4s4q7r6xkzyxgwcspgnf5g48zxyci6m-kparts-5.99.0-bin/share", "/nix/store/jkvqq7y1vm7r96i567ydj6jyv97zwi4g-kpimtextedit-22.08.2/share", "/nix/store/c8mzpv2wi3g35j93aq9qw6xzniwcv6vk-kemoticons-5.99.0-bin/share", "/nix/store/xyc11g56m2664gnkndj6hp08i50mqc10-akonadi-search-22.08.2/share", "/nix/store/g38s6d5v1qnxxwaxl4g14rl9s9g1pfvp-plasma-framework-5.99.0/share", "/nix/store/9hgqpag9cs88qazi4akpasdl4iyfamis-kcmutils-5.99.0/share", "/nix/store/mvcdwxdn3j7paw6gdml50wj4m1z15gqc-kdeclarative-5.99.0/share", "/nix/store/74mik9ihisjcbx8y99fhdlkw96yqy970-kpackage-5.99.0/share", "/nix/store/4w8c9wnag95ps5hm711xdw3rs93kxcpf-kpackage-5.99.0-bin/share", "/nix/store/176h84wv23plyhsigyindgr48riny4jh-krunner-5.99.0-bin/share", "/nix/store/vi59gqkzwccf0w8gz3ssnzsc61ivlzda-plasma-framework-5.99.0-bin/share", "/nix/store/bkbb813g132crm9239wj339qml5bxibh-kirigami2-5.99.0/share", "/nix/store/knf5ni6r3pwlhw1y8ji8zx0ciaggw8hg-kcmutils-5.99.0-bin/share", "/nix/store/9si2nl38v1nx7wa22f0hk4wc2p3awlmn-kmime-22.08.2/share", "/nix/store/0lz3dyiwlgsi2s1f3r1d81jd241115pa-kcontacts-5.99.0/share", "/nix/store/f97fn9d66i2s4nmrdr5f22wiwjrfrn65-iso-codes-4.11.0/share", "/nix/store/46dhnd4cxlva85xdaxsjhhvgxcvwnnl2-akonadi-mime-22.08.2/share", "/nix/store/7pgqndmag1zab75ysqqvrbpk0n51pvy5-akonadi-22.08.2/share", "/nix/store/ds2fkslmgppm38sidqzlqkm7shwhahg9-kdesignerplugin-5.99.0/share", "/nix/store/dh4c0wcf6mwbr41gbjpzlm3gg6zkjdas-kwallet-5.99.0/share", "/nix/store/s8l63hcldsc0awxzgssicylixnbk7kpg-kwallet-5.99.0-bin/share", "/nix/store/sklbxhf2qsb5qm283bkhavvdi0fmr7f7-knotifications-5.99.0/share", "/nix/store/vhq7l3n4asmxb61m8dpinz77hgs42flz-kded-5.99.0/share", "/nix/store/g69i1ify8k664fcwgbl85l54phhvlvpz-solid-5.99.0/share", "/nix/store/kbmhv929ip0qchi4z6j84gpx3m9f78vp-kjobwidgets-5.99.0/share", "/nix/store/vz5yrpzyqpv2hb0hi6rxj1q13ym0155q-kjobwidgets-5.99.0-bin/share", "/nix/store/s5w1gpd5rdvw37h87kb8kny9ichs1hv2-kbookmarks-5.99.0/share", "/nix/store/c52xgznk0pwm5rmpwcrxa4k47wm00q67-kxmlgui-5.99.0/share", "/nix/store/ha793l4psb3r9h1gy51xkwzqy0bwj7nc-ktextwidgets-5.99.0/share", "/nix/store/sacljksyq9py4pv57a4mjpa214lwcnwh-sonnet-5.99.0/share", "/nix/store/7wf67sv0z533v9k5j6h5s7sy1446zq2h-kservice-5.99.0/share", "/nix/store/8xc0zr1qfyjirysf82zx4pkpzy0mj3i4-kservice-5.99.0-bin/share", "/nix/store/cipsmxl8765m803p9p8sqkjl0wsksgqg-kiconthemes-5.99.0/share", "/nix/store/jlxql53xndizl75px7cvhpg6p84125i2-kdbusaddons-5.99.0/share", "/nix/store/8x3n01sfhxc83qcnryqlwbm2gajnwbby-kguiaddons-5.99.0/share", "/nix/store/93whh6j9f618pqp77b90cszbp2840xsb-kcodecs-5.99.0/share", "/nix/store/ilhhj2ndal9wjqpvdnwq6hrsfq8yx7ck-kcoreaddons-5.99.0/share", "/nix/store/8rk0rd64igkf09ayqjpl6fs00a93p918-kcoreaddons-5.99.0-bin/share", "/nix/store/b6ijj02mlapbynbmwl2zyryipr8ghwy5-kconfig-5.99.0/share", "/nix/store/xlay3lx621q881axgldslj8xqv7h5vrb-kio-5.99.0/share", "/nix/store/bziwmqb7877q4a3qn07ja28zapi4jpck-knotifications-5.99.0-bin/share", "/nix/store/5ky8v4md26pmy3773jcclxrnw7h5k54i-phonon-4.11.1/share", "/nix/store/jx03nwiw2dad8vi5nbyq1prsjmqjjm1y-kglobalaccel-5.99.0/share", "/nix/store/9lb2clw5wkyyn5n3zbscra3sg0iagvy1-kitemviews-5.99.0/share", "/nix/store/z6sz6qa084kw119jfwi7avgvz4i5qf9c-kwindowsystem-5.99.0/share", "/nix/store/hlg9mwiyj2kz7fsif11nfp9x2l1825s6-kconfigwidgets-5.99.0/share", "/nix/store/slv1a7g2ms2rqdpr0my8q78f02kpjh00-kauth-5.99.0/share", "/nix/store/6x12s7mcfpms268k16fs00nm5vsxhjhp-kcompletion-5.99.0/share", "/nix/store/45iwbc558klp4q3jzqh38pslv4gcyk24-kwidgetsaddons-5.99.0/share", "/nix/store/izvarj1m0mkgwazcd4fdf1591hjd8cqk-kaccounts-integration-22.08.2/share", "/nix/store/rcphlfsz29vlcn30lfa5v1i1kg4zcd1h-flex-2.6.4/share", "/nix/store/vmx659n6v4hwsaa1srip54v812kicql6-gnum4-1.4.19/share", "/nix/store/hirgcsy1dlgsvk0fzz7g5zpkf2pavrl8-bison-3.8.2/share", "/nix/store/86f0dfvl6g1jbfmrw2ciphzh4vpqvnr4-kdoctools-5.99.0/share", "/nix/store/bhyzhr5116pjmjlilpsdysq6yvz34w4p-ki18n-5.99.0/share", "/nix/store/8ij41s8wskpjr3vydz1zqgpcdvs01vy6-fontconfig-2.14.0-lib/share", "/nix/store/z5qg6m93yv2wsij772rsljah3ahw1m8b-systemd-251.5/share", "/nix/store/phl7g6rn4qhl2pkm76167c5k6kp3jyzs-glib-2.74.0/share", "/nix/store/x9cfvkh726wqgc1rj63vyz9ycdjsl5ca-gettext-0.21/share", "/nix/store/ch1wps03skraiz6b0jrvmbg2rsj7ha1p-dbus-1.14.4/share", "/nix/store/2izis5n0s1ajddlgzc6q1hxa8g7y7f2y-plasma-workspace-5.26.2/share", "/nix/store/16kvx3l6shnygzxs0ni66dd05x3g80j0-pipewire-0.3.59-lib/share", "/nix/store/gqjxarr1d6ymgqvp7kxdq622dpypljvr-exiv2-0.27.5/share", "/nix/store/nng516kni2psw204dzs9jsw0bg3fxi03-kpipewire-5.26.2/share", "/nix/store/dlc32p5vnm2shfy3rdpfzvdzsqw0pv5m-appstream-qt-0.15.5/share", "/nix/store/lfjhshx5j330rwjyyhkx2s7ki866mj0w-kholidays-5.99.0/share", "/nix/store/1jscm3l28sdydz4p6bw563x56p778mm2-networkmanager-1.40.2/share", "/nix/store/h3gwbs4jxsgi756lhmxh26746r3mbza3-gnutls-3.7.8/share", "/nix/store/vy21zhmp6v8p35qwxlk3nnyaamljgszc-libqalculate-4.4.0/share", "/nix/store/jnhjnnq34xjrrndag0lsga36276fd313-libksysguard-5.26.2/share", "/nix/store/145iba1mhxjxs1dibq79vxjfb0payjxq-libksysguard-5.26.2-bin/share", "/nix/store/x0h72ycfbkrk25r86846yik0g9z55bcz-libkscreen-5.26.2/share", "/nix/store/j9gywkz88h126cdp3mq0iig3anz79jxq-kxmlrpcclient-5.99.0/share", "/nix/store/vy5ff8hsnxhzrlz71msx66461ihqg2rz-kwin-<IP_ADDRESS>/share", "/nix/store/yn7afxxxqwk8741w7pjz7mhbf9am9ij9-breeze-qt5-5.26.2/share", "/nix/store/g268dmqc057vss31fa02w2rbq4cdpq0m-kdecoration-5.26.2/share", "/nix/store/0snk0b09b2k9md9gas7cf7qbna5l6yf5-breeze-qt5-5.26.2-bin/share", "/nix/store/gl0mkyf9q4l3in5ki7k22jcrssx2mxl3-frameworkintegration-5.99.0-bin/share", "/nix/store/rzz7fw1x17iyv0xscxmksj7l8fqpbzx2-ktexteditor-5.99.0/share", "/nix/store/f4ga159gh7q80q5hb1a7w12nwqifr8da-ktexteditor-5.99.0-bin/share", "/nix/store/agj19gkncbcyh91mh01ry8sv722rswzi-kscreenlocker-5.26.2/share", "/nix/store/f7kr4hfp5a0xbq8jrc3r9dbnr68yn2jm-kpeople-5.99.0/share", "/nix/store/p3dclwajhmddajnlssn8yjkpmr3lphwb-kjsembed-5.99.0/share", "/nix/store/g18z0lkcv29jvfh8x2kam9mf7f70jssr-kdesu-5.99.0/share", "/nix/store/g50br00r84z0kga1wd740xzxbiv3h5rp-kdelibs4support-5.99.0/share", "/nix/store/qihs7q7j1b332z0jyhp78xnny9ymzzsc-kunitconversion-5.99.0/share", "/nix/store/w74h3v0dshky01xl09j2cbqmcblsiri0-baloo-5.99.0/share", "/nix/store/64vjhw2jb9fplwc5czfcisp7159iwww1-kfilemetadata-5.99.0/share", "/nix/store/c2iwcm67jp2zsvznvhi6bzw0kx2i1xl3-linux-pam-1.5.2/share", "/nix/store/27qkkgv5a5xf1pqbz3r90vfxvnkyhin5-desktops/share", "/home/dschridde/.local/share/flatpak/exports/share", "/var/lib/flatpak/exports/share", "/home/dschridde/.nix-profile/share", "/etc/profiles/per-user/dschridde/share", "/nix/var/nix/profiles/default/share", "/run/current-system/sw/share")

This bug is probably a duplicate of #96255 and #41426 and maybe somehow related to #129442.

I gave kde a try again after 5 years and was surprised this was still not fixed :)
With some strace I was able to find a way to make it load. I don't understand the whole problem space but here is a patch that fixes the issue: https://github.com/NixOS/nixpkgs/pull/268893


colored STL files - enable showing binary colors of STL file

I added the ability to read the binary colors of the file.
now, if the file has colors, it will be seen.
example for a colored file, from three.js official site: https://threejs.org/examples/models/stl/binary/colored.stl.
note: in that case, when there is binary colors, the modelColor parameter does not needed and will not affect.

@mkantz84  I think you've changed the built file. You should change source codes instead. It's the file src/Paint.js at line 85.

@mkantz84 I think you've changed the built file. You should change source codes instead. It's the file src/Paint.js at line 85.

Thank you for that. Updated the PR

You guys rock! Will review and merge this week :)


[Feature Request]Display XML content

Hi, Is it possible to display XML structure content, like Ray?
check it is an XML structure or Node, it have the display.

Thanks

@luanfreitasdev, I think it would be better to not render HTML. This might be inconvenient when trying to debug an HTML response or even a XML string:
ds('
<?xml version="1.0" encoding="UTF-8" ?>
<Document xmlns="urn:iso:std:iso:20022:tech:xsd:pain.001.001.03">
    <CstmrCdtTrfInitn>
        <GrpHdr>
            <MsgId>Message-Id</MsgId>
            <CreDtTm>2024-05-10T16:10:02.017+00:00</CreDtTm>
            <NbOfTxs>1</NbOfTxs>
') //...

ds('red
line break');

Issue solved in https://github.com/laradumps/app/pull/353


Adapt to latest SCDF docs

Also, fix includes of SCSt docs
Fixes https://github.com/spring-cloud/spring-cloud-dataflow-server-kubernetes/issues/124

Rebased and merged as commit c8e56685a9b00feab37c1a0daa801e8d73955a90


[Chrome 100.0.4896.60 / Linux 0.0] Running testcafe headless on a default linux image (see gitlab pipelines f.e.) does not recognize any webelement

What is your Scenario?
Given I estabilish a new testcafe project
When I create an automation E2E on a webpage (e.g. www.google.com)
And I want to execute the flow on linux server
And I use the chromium or firefox which come by default on Chrome 100.0.4896.60 / Linux 0.0 (gitlab pipelines)
Then I will receive the error "The specified selector does not match any element in the DOM tree." for any element which i'm trying to perform an action.
Note: locally on macos or windows, the execution works fine headless, this happens only on gitlab pipelines or jenkins
What is the Current behavior?
I will receive the error "The specified selector does not match any element in the DOM tree." for any element which i'm trying to perform an action.
What is the Expected behavior?
The scenario is executed remote on Jenkins or Gitlab Pipelines on Firefox or Chrome
What is your public website URL? (or attach your complete example)
www.google.com
What is your TestCafe test code?
import {environment} from '../environments/environment';
import { testfilePage } from '../pageObjects/testfilePage';
const googlePage: testfilePage = new testfilePage();
fixture Getting Started
test('Reach google page proof of concept test', async t => {
await t
.navigateTo(environment.url)
.setTestSpeed(environment.testspeed)
.resizeWindow(1920, 1080);
//here the action occurs
await t.click(googlePage.agreeButton), {timeout: 3000};
await t.expect(googlePage.title.innerText).eql('Google');
});
Your complete configuration file
No response
Your complete test report
No response
Screenshots

Steps to Reproduce

TestCafe version
1.18.5
Node.js version
No response
Command-line arguments
testcafe chromium:headless fixtures/*
Browser name(s) and version(s)
Chrome 100.0.4896.60 / Linux 0.0 or Firefox
Platform(s) and version(s)
Linux 0.0
Other
No response

I've run into something like this before with the google homepage.
This could be a workaround for you; would you mind trying this with the privacy modal, please? :-)
In your test add this:
const privacyModal = '#xe7COe';
await removeElement(t, privacyModal);

add the function below to your test script:
async function removeElement(t, elementSelector) {
  await ClientFunction(elementSelector => {
    const element = document.querySelector(elementSelector);
    element.parentNode.removeChild(element);
  }).with(t)(elementSelector);
}

What the function above does, is remove the privacy modal entirely. Not ideal, but from there you should be able to continue with your test as normal

Hi folks,
It's not a TestCafe issue. The google.com website is rendered differently depending on the environment. You can record a video to determine which web element is not found and why it happens. After that, you can add conditional logic to remove or hide the required elements.

i'm using a private website, not google, maybe i've logged the wrong example. will add a new issue asap.


README.md improvement

A few fixes and some improvements for your README.md.

Sorry I had to change the backticks to tabs first. Now it can be merged.…

Thanks, merged

You're welcome.
On 06/27/2015 01:54 PM, Nikita Nazarenko wrote:

Thanks, merged

Reply to this email directly or view it on GitHub:
https://github.com/savant2212/xdr-rs/pull/2#issuecomment-116020018


Add a way to contain non-js files inside outputDir

What problem does this feature solve?
I want my folder public to contain some unrelated files (index.php, .htaccess, favicon.ico, robots.txt) and my vue-cli-service built files (/js/app.13314234.js and /css/app.13314234.css).
My vue.config.js looks like this
module.exports = {
    outputDir: '../../public/build',
    indexPath: 'some.index.html'
};

And js and css are built into /public/build/js/app.13314234.js and /public/build/css/app.13314234.css but, into the generated some.index.html, only paths /js/app.13314234.js and /css/app.13314234.css are injected (without the build) file.
I could specify
    outputDir: '../../public',

but then /public folder will be cleared of my files I want to keep.
I could use flag --no-clean, but then the /js and /css folders will contain older versions of built js and css, which I don't want.
Is there a way to add a prefix, to the webpack injecting js/css to inject it with a prepended path?
What does the proposed API look like?
Add an entry to the vue.config.js to add a prefixed path, which will be used to generate paths injected into indexPath

Unless I missunderstand you, all you have to do is to set the publicPath to /build when building for production in vue.config.js:
publicpath: process.env.NODE_ENV === 'production' ? '/build' : undefined

That would prefix paths in your built files to go to /build/css/app.13314234.css etc.
If that's not what you want, please clarify

I'll try it when I'm home.
Thank you for your answer :)
niedz., 3 lut 2019, 14:05: Thorsten Lünborg<EMAIL_ADDRESS>napisał(a):

Unless I missunderstand you, all you have to do is to set the publicPath
to /build when building for production in vue.config.js:
publicpath: process.env.NODE_ENV === 'production' ? '/build' : undefined
That would prefix paths in your built files to go to
/build/css/app.13314234.css etc.
If that's not what you want, please clarify
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vuejs/vue-cli/issues/3413#issuecomment-460050038, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMv5txoiQTlcV0d-Av6GtuUIGxBVfgxaks5vJt5-gaJpZM4agEVR
.

Yes, that was exactly it! :)
Sorry for bothering you :/


Easier way to stop processors on failure

It would be useful to be able to stop consuming events from the stream if some persistent problem is detected while processing events (database connection problem for instance) and be able to start consuming events again after certain condition is given.
To do that it is needed to stop a processor when its inner consumer detects an error.
The issue is there is no a direct way to map the failing consumer to the processor that created it, so when an error happens inside an observer there is no an easy way to figure out which processor it is needed to be stopped and recreated

This is probably going to need some design, possibly a carrier object that tracks processors and consumers.
In the meantime, there's been some changes to stop behaviour in 0.9.0 along with the option to throw a NonRetryableNakadiException from the observer to kill the consumer pipeline.

Would it also be reasonable to have a shutdown on the client to cleanly shutdown all processors. IMO at the moment it is still unclear how to approach a proper shutdown.


[docs] Pressing enter in docs search only refreshes current page

If this isn't the right place for the issue, I apologize! I couldn't find a docs repo and no one in Gitter channel knew of one.
What's happening
When typing something in the search at vuejs.org/guide/, a dropdown of results appear. I can click on items in those results, but if I press Enter, the list of results disappears and the page is refreshed.
What I expect to happen
When pressing Enter, it would make sense to me if the top result in the search were navigated to, instead of refreshing the current page.

The docs repo is at https://github.com/vuejs/vuejs.org
The search is using algolia's docsearch, maybe open an issue there instead?

@yyx990803 Thanks! I'll be sure to post there for future documentation issues.

@yyx990803 This should now be fixed in docsearch 1.1.1. :tada:

Sweet


Problems with history by using router.back() / back class

If I use router.back() or a link with back class for navigating to the previous page, I always get the same behaviour:
The page transists to the previous page but the page events deliver the page name (data-page) from the page before the previous page.
This happens also on smart selects.

Yes, same problem here. Because FW7 load previous of previous, all my page properties are override with previous.

You have to deactivate preloadPreviousPage
var mainView  = myApp.addView('.view-main', {
preloadPreviousPage:  false
});

Issue is closed because of outdated/irrelevant/not actual
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


"multi-domain" or "SAN" (Subject Alternative Names) SSL certificate

We might need to point multiple domains to a single ip. In such cases we can obtain a single multi-domain SSL certificate for all the domains with let's encrypt. That way we can have a consistent nginx config.
sudo certbot certonly --manual --preferred-challenges=dns --email<EMAIL_ADDRESS>--server https://acme-v02.api.letsencrypt.org/directory  --agree-tos --manual-public-ip-logging-ok -d "*.everyoneisdoingdrugs.com" -d "everyoneisdoingdrugs.com" -d "*.everybodyisdoingdrugs.com" -d "everybodyisdoingdrugs.com"
Nginx Config:
server {
    listen      80;

    #Rewrite all nonssl requests to ssl.
    return 301 https://$host$request_uri;
}

server {
    location / {
        proxy_pass http://localhost:3000;
        proxy_set_header Host $host;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header X-Forwarded-Proto $scheme;
    }
    client_max_body_size 4G;

    listen 443 ssl;
    ssl_certificate <PATH TO fullchain.pem >;
    ssl_certificate_key <PATH TO privkey.pem >;
}

this solves: https://github.com/restarone/violet_rails/issues/1534


New pipes are added with initial status set to CLOSED

In EN_addlink new pipes are added with initial status set to CLOSED and CVs set to OPEN.
I think both are should be OPEN as this is the most common case.
Thoughts?

👍 agree. What is the default status on pumps? I remember having some discussion about what the expectation is - like when you create a steady-state model, you would expect adding a pump to mean "adding a pump that is turned on"?

Currently, apart from CV all new links are assigned a CLOSED initial status
https://github.com/OpenWaterAnalytics/EPANET/blob/dev/src/epanet.c#L4345

Long overdue... so the change would be to mimic the logic of the EPANET GUI when adding links. Status is OPEN for pumps\pipes\CV and ACTIVE for valves.


ISSUE: Player does not revive after leaving

When the dead player leaves the server and comes back, they are not revived.

Fixed issue


How did you do MT

Hello, I want to know how did you do your machine translation so I can extend this model to more languages, which I cannot find in paper and this repo.
What I was asking are as follows:

What MT tools/interface (eg. baidu/google translation)
What version (eg. free/vip)
Thank you!

Hi,  I am using the machine translation API from Microsoft Azure. As the work is done during my internship at Microsoft, so it does not really cost any money to use them as an internal tool. There are some recent work that translates the CC into more languages: Multilingual Multimodal Learning with Machine Translated Text, which you might be interested to taking a look at before you translate CC to more languages.

Thanks for such comprehensive answer!


Lazy Rendering

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

I want to introduce experiments on lazy rendering.
How it works now:

We receive a HTTP request and translate it into a Request object
We send the Request through middleware to the controller
There may be some action on data change
Now we need to prepare the contents of the page. We designate the page and call render()
The page content is rendered and written to the buffer.
Then a layout with markers is rendered into which the page content is written. The layout content is also written to the buffer
on a calling WebView::endPage() the buffer is retrieved, the markers are replaced with CSS/JS.
The result is written as a string into stream of a Response object.
Response is piped back
Emitter receives a Response and emit content to the client

The idea of lazy rendering:

When we call the render() method, a generator is created. Rendering as such does not occur.
The generator is wrapped in a stream and placed in a Response object.
Response is piped back to Emitter
Emitter starts emitting the contents of the stream to the client, and here inside the generator, rendering starts
When ready, data is immediately sent to the client, not to the buffer

Visually the essence of lazy rendering in this picture:

Features

Page does not linger in buffer
Stream of the Response object does not know the length of the content
At the time of rendering, all the middleware is already “behind”

Benefits

From feature №1 follows less memory consumption, and you can also generate a page of any length
From item 1 follows a shorter waiting time to first byte (TTFB)
Now we are using PSR-7 from nyholm, which means we can avoid unnecessary I/O (Response from nyholm writes the buffer to a tmp file that uses as stream)

Disadvantages

From feature №3, difficulties may arise with middleware who write information related to rendered content in HTTP headers
Solution: we can force the rendering to run at any stage by simply requesting content from the stream
From feature №3, in the case of an uncaught error at the stage of rendering and lack of buffering, it will not be possible to change the page code to 500. At the same time, part of the page can already be generated and sent.
Current WebView is completely unsuitable for this approach. In addition, without markers that do not work without page prerendering, the Asset system may work incorrectly (some Assets may be declared while the page content is rendered, but we need to know about them earlier)

Implementation
I put the experimental code in a separate folder src/LazyRendering
You can find the start links on the index page.

I also made some demonstration pages, and added different rendering modes for them.
I had to replace the existing WebView for these pages with a streaming counterpart.
My results
Without I/O actions in a Controller

Questions

Do you see a benefit for yourself in this approach?
What do you think about making streaming WebView?
What do you think about lazy rendering?
Would you like to see the minimum TTFB on your site?

That's a good demo of current emitter capabilities. Potentially it could be converted into tutorial and/or parts of it could be made reusable. Need to check it in more detail.

I like your idea, but I don't understand the next disadvantages:

From feature №3, difficulties may arise with middleware who write information related to rendered content in HTTP headers
From feature №3, in the case of an uncaught error at the stage of rendering and lack of buffering, it will not be possible to change the page code to 500. At the same time, part of the page can already be generated and sent.

Could you explain it more please ?

Looks interesting :slightly_smiling_face:

While it's definitely interesting and allowed to test Yii 3 for edge cases, I don't think it should be merged into demo app.


Question regarding consolidated.fasta

Does consolidated.fasta includes all the sequences present in all the inclusion genomes?
How to determine that the found sequence is present and significant in all inclusion genomes?
Thanks

Does consolidated.fasta includes all the sequences present in all the inclusion genomes?

Yes. Please see this link for more information.

How to determine that the found sequence is present and significant in all inclusion genomes?

The signature score informs how discriminatory the signature is. Please see this link for more information. The higher the inclusion score, the more the signature is represented in all the inclusion genomes. The closer the exclusion score is to zero, the more specific the signature is, with respect to the input data. The total score is the sum of the inclusion and exclusion scores.

My problem is that consolidated.fasta no 1 sequence that is originating from first genome is not present in other genomes in sorted signatures.
Should the consolidated.fasta no 1 sequence be present with 100% similarity in all the other genomes in sorted signature fasta files?

The signatures in the consolidated.fasta file will exactly match at least one inclusion genome (the genome it was taken from). They are not guaranteed to exactly match any other inclusion genome, but high-scoring signatures will very close match sequences in other inclusion genomes (for example, 95% sequence identity matches with the other inclusion genomes).
This strategy is desirable because mutation means that it's unlikely that two sequences will remain exactly the same, given enough time.
If you are looking for long sequences that exactly match with 100% similarity in many genomes (no errors, mutations, etc.), you might want to use an approach that finds the longest common substring. A suffix tree approach may be helpful, but I don't know of any such tools.

Thank you for quick explanation!


Add templates for issues, pull requests and more

When users want to create an issue or pull request, it’s often unclear what information to include or which details are necessary. This pull request introduces templates designed to guide users through the process and help them provide the required information.

Duplicated


Secure binary serialization is not supported on this platform

I have an error like "Secure binary serialization is not supported on this platform."
I think this happens when I have #95 error again. I think when I used complicated queries I have this error!
Does have any idea?
I would be appreciated if you can help to fix it,
Than you!

Complicated queries like
select * from Customers inner join Cities on Id = CustomerId where IsActive = 1 /*1 means true. It comes from your server side linq query: Where(c => c.IsActive == true))*/

Bit version: v1.0.4
ASP.net core
dotnet versions :
dotnet-sdk-2.1.300-preview2-008533-win-x64
dotnet-sdk-2.1.300-preview1-008174-win-x64
corefx version:
4.5.0-preview3-26513-0 -> runtime.win-x64
OS: Windows 7, 8.1, 10

StackTrace: in version 1.0.5
 Bit.OData.Implementations.DefaultWebApiODataControllerActionSelector.SelectAction(HttpControllerContext controllerContext) in C:\\Users\\b.abbasi\\Downloads\\bit-framework-1.0.5\\bit-framework-1.0.5\\src\\Server\\Bit.OData\\Implementations\\DefaultWebApiODataControllerActionSelector.cs:line 106\r\n   at System.Web.OData.Routing.ODataActionSelector.SelectAction(HttpControllerContext controllerContext)\r\n   at System.Web.Http.Tracing.Tracers.HttpActionSelectorTracer.<>c__DisplayClass7_0.<System.Web.Http.Controllers.IHttpActionSelector.SelectAction>b__0()\r\n   at System.Web.Http.Tracing.ITraceWriterExtensions.TraceBeginEnd(ITraceWriter traceWriter, HttpRequestMessage request, String category, TraceLevel level, String operatorName, String operationName, Action`1 beginTrace, Action execute, Action`1 endTrace, Action`1 errorTrace)" | string

@behzad888 Sorry for my late response. I've not seen any notification for your reply in GitHub notifications center! Could you please copy/paste entire visual studio's output here? Thanks in advance.

It seems that you were running ODataSamples of bit on top of .net core. As you might know, entity framework 6 which is being used at that sample is not compatible with .net core (yet). I submit you that sample which works with entity framework core and asp.net core & .net core. Note that you've to install .net core 3 preview 1+
ODataSamples.zip


suggestion: testing instructions

Hi team, I'd like to contribute to this great module. However, there aren't any testing instructions. Is it possible to have this added so contributors can do so?

Gently pinging

I am gently pinging once more. I'd love to help make this repo better! 🙂
CC @cdschexnide and @mdwatson988

Hi @iuioiua. Sorry for the delayed response. I started a new job in October last year and we use GitLab so I'm rarely on GitHub anymore. I believe if you fork and clone the repo, you can locally make whatever changes you'd like to add. Then once you submit the pull request and one of us approves it, you'd be added as a contributor.

Great! Thank you 🙏🏾
However, I was more referring to testing. If I make a change what test can I run to ensure I haven't broken anything?

This is the test file: https://github.com/oslabs-beta/Deno-Redlock/blob/main/src/single.test.ts
Running this test file in the terminal will verify that your additions do not interfere with proper lock functionality.

Ah, I didn't realise there was a src/single.test.ts. I was using src/testing/single.test.ts. Thanks!


Y-axis scale on right side

The Line chart could have an option to display the legend, or scale, at the right side instead of the left side. when using it together with live updating, it's easier to read the current value if the scale is there where the latest data points are displayed. And in the western world, information commonly flows from the left to the right.

Huzzah! The first alpha of Chart.js 2.0 has landed and should fix this issue. Check out the release and try it out! We've got a lot of momentum right now, so please help us test so we can launch 2.0 Gold by the end of the month.
https://github.com/nnnick/Chart.js/releases/tag/v2.0-alpha


chore: upgrade data deps

Description of changes

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

Does this change supersede https://github.com/aws-amplify/amplify-cli/pull/13488?

Oops, I forgot we already had one open. Yes, this does supersede https://github.com/aws-amplify/amplify-cli/pull/13488.

We have discovered a bug in amplify-category-api 5.9.2 and 5.9.3. It does not affect 5.9.1 so the released CLI is OK.


Enhance effects button

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
The background blur option is currently hidden behind the three-dot menu, which is not intuitive. Users expect the blur option to be associated with the camera icon or directly visible on the main interface. This can cause confusion and make the feature harder to find.
Describe the solution you'd like
Move the blur option closer to the camera icon or add a dedicated button on the main screen. This would make it easier for users to locate and activate the feature without navigating through multiple menus.
Describe alternatives you've considered

Adding a tooltip or hint to the three-dot menu to make the blur option easier to discover.
Providing the blur option as a checkbox or toggle directly on the "Join Room" screen.

To be discussed from a product perspective.

one method consists of adding a button directly in the picture of its own camera return that points to the blur menu

one method consists of positioning it at the level of the camera settings

This is a preliminary mockup from the designer. When hovering over your camera tile, you'll have the option to blur your background.

I love also your proposition in the menu while selecting the camera, I will discuss it with our designer


@conda-forge-admin, rerender feedstock

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/python-gmail-feedstock#2.

@sujitmandal, I don't think you're supposed to directly edit like this:
https://github.com/conda-forge/python-gmail-feedstock/commit/8ad1b4696cc391273ff64b949c6c6faa5472d10a

So I re-rendered to show you that it would be reverted automatically next time you do a PR or update, see #2 or commit https://github.com/conda-forge/python-gmail-feedstock/pull/2/commits/393230b2b6f6f4eea8cab555dda7e504f8c2e4f5

Refer to the docs for more info https://conda-forge.org/docs/maintainer/updating_pkgs.html

@sujitmandal, I saw your merged #2. I only did this to show you since you seem to be new here; don't be discouraged or offended please. Welcome and thank you for your contributions :)

@ngam this my 7th package here but others packages i was did same after merge, no one told me that this is first time i amr hearing this..

Do you see the logic though? I am not telling you to change things (I have no power; I am not part of core or anything like that). I wanted to show you that your changes (in https://github.com/conda-forge/python-gmail-feedstock/commit/8ad1b4696cc391273ff64b949c6c6faa5472d10a) will be reverted with a "rerender" (as in https://github.com/conda-forge/python-gmail-feedstock/commit/393230b2b6f6f4eea8cab555dda7e504f8c2e4f5 in PR #2). Does that make sense?
I was told when I first came around here that nothing outside of a few files should ever be changed. If you're not convinced or if you think I am wrong, you could ping @conda-forge/core to confirm (they actually have the power to say yes/no, unlike me just noting things). Of course, you're in charge of this feedstock and you should feel empowered to do everything however you like :)

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/python-gmail-feedstock#3.

Ugh, please ignore #3

(I noticed that we may not have good documentation on what to edit in a feedstock, so maybe it should be added somewhere... some files have a comment atop them about being automatically generated, readme.md should be one of those)


Add first version of tile retriever.

A first working version of a tiler that will create tiles over a scene based on location and dates. The tiles are in proper numpy format and each has metadata like bounds, centroid, and resolution.
This is a basis for discussion, but I am quite happy about the tiling overall and the approach. This should be very scalable, and can be switched to a different STAC provider very easily.
Refs #10

Closing in favor of https://github.com/Clay-foundation/model/pull/27


[Feature]  Infer EthereumSigner automatically in the signer layer of the ProviderBuilder

Component
provider, pubsub, signers
Describe the feature you would like
Given
let signer = LocalWallet::random();

We currently require the signer to be cast into EthereumSigner::from(signer) when using the signer layer in the ProviderBuilder as follows:
let provider = ProviderBuilder::new()
    .with_recommended_fillers()
    .signer(EthereumSigner::from(signer))
    .on_http(rpc_url);

The proposed API:
let provider = ProviderBuilder::new()
    .with_recommended_fillers()
    .signer(signer)
    .on_http(rpc_url);

Additional context
No response

change would need to consider #638 and #535

sorry that should be #575

This seems simple but under the hood has a bunch of extra work needed to allow calling .signer(...) multiple times which would collect credentials in 1 EthereumSigner. Right now if you call .signer(...) multiple times it is not great behavior bc it might create a new SignerFiller or override the existing signer.


Combine all mixins into a single component mixin

This is in preview in the v6-trey branch.

It's neat you can compose stuff together, but in wanting to streamline the API a bit, I've found I really only ever use withComponent(). The rest of the mixins are really tiny, some offering little value. Here's what I propose:

Remove withLifecycle(). It doesn't really do anything except alias lifecycle names and make calling them slightly simpler.
Remove withContext() in favour of creating a separate context API as a new package. This has een discussed in #1485.
Merge withChildren() and withUpdate() together into withComponent() and just call it component().
Export a pre-executed version of component() and call it Component. This will be the 99% use case. The function exists to extend built-ins if you really want to, or give it a different base class but not sure why you wouldn't just extend Component in the latter case.
Renderers can now be just functions, instead of classes and you can mix them into Component by just setting it as a static prop.

Speaking from my personal experience, and preference, I never liked piecing together the mixins, especially when the one's outside of withUpdate() were so tiny and you opted into their behaviour anyways. It feels like there's a canonical way to use Skate if they're all in one and gives us an opportunity to start creating separate packages for these things, leaving Skate as just "core".

especially after realizing (through trial and error), we really just end up using withComponent

This has been my experience all of the time.

Also like the idea of still shipping component() and Component — that should make upgrading us over to this super straightforward!

It's definitely intended for this purpose! Also, if people want to use customised build-ins, it's for them, too, so I don't think it's something that will be removed at any point of time. At least not soon until we know exactly what will happen with built-ins over the years.

It's definitely intended for this purpose! Also, if people want to use customised build-ins, it's for them, too, so I don't think it's something that will be removed at any point of time. At least not soon until we know exactly what will happen with built-ins over the years.

Yeah... @_@ https://github.com/w3c/webcomponents/issues/509

We ended up not shipping a way to do mixins, at least yet. Happy to reintroduce at some point but wanted to get some solid use cases submitted first before pre-optimising for that use case.


Infinite loop when modifying Tree children

Tree:s (and ParentedTree:s) go into an infinite loop if you try to change or delete a child by using a string as key:
>>> t = nltk.Tree("(a b c (d e) g f r)")
>>> t["2"]
Tree('d', ['e'])
>>> t["2"] = 42
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/peter/Projekt/NLTK/nltk/nltk/tree.py", line 177, in __setitem__
    self[index[0]] = value
  (...)
  File "/Users/peter/Projekt/NLTK/nltk/nltk/tree.py", line 170, in __setitem__
    if isinstance(index, (int, slice)):
RuntimeError: maximum recursion depth exceeded in __instancecheck__

The reason is that neither __setitem__ nor __delitem__ tries to convert the key to an integer: self[index[0]] = value. __getitem__ does that on the other hand: return self[int(index[0])]
The solution is simple, but there are two possibilities:

Convert string keys to integers as is done in __getitem__: replace index[0] with int(index[0]) in __setitem__ and __delitem__
Disallow string keys: replace int(index[0]) with index[0] in __getitem__, and add a check isinstance(index, (list, tuple)) to all three methods

Personally I prefer the second, since it is what Python does for lists.  I don't see the reason why tree["123"]should be allowed, and have the same meaning as tree[1,2,3].  However, although I don't think the current behaviour is documented, the change might break some existing code.
What do the benevolent dictators think?

Is it typical to index into a tree using strings? That seems pretty bizarre. I like the second option as well.

I agree that the existing API is inconsistent and that the type conversion is not useful functionality.  I also prefer the second option.


Loop prop not working as intended on Vimeo Player

Current Behavior
The loop prop does not seem to currently be working for the Vimeo player. Specifically, I'm setting loop to false but the video element still has the loop attribute set.
// Video elm still has loop set to true
<ReactPlayer
	url={VIDEO_URL}
    ref={this.playerRef}
    loop={false}
/>

I've been able to work around this by setting the loop prop to true initially, then setting it to false once the video is ready, forcing setLoop to be called.
// constructor
this.state = { loop: true };

...

// renderPlayer
const onReady = () => {
	setTimeout(() => this.setState({ loop: false }));
}

<ReactPlayer
	url={VIDEO_URL}
    ref={this.playerRef}
    onReady={onReady}
    loop={loop}
/>

Expected Behavior
The loop prop should be respected if it is initially set to false. From looking at player.js it seems like setLoop needs to be called after the onReady callback fires
Steps to Reproduce

Set loop prop to false with vimeo url (haven't tested other platforms)
Play video
Inspect video element inside iFrame
Observe loop attribute is set on

Environment

Browser: Chrome Version 74.0.3729.169
Operating system: MacOS 10.14.5

Other Information
Thanks for maintaining this project, its been super helpful.

Do you have a jsFiddle example of this happening? I'm trying to reproduce this myself but I can't see loop being set anywhere if it's set to false.


python3Packages.osmnx: 1.3.0 → 1.9.1

Description of changes
https://github.com/gboeing/osmnx/compare/v1.3.0...v1.9.1
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

2 packages built:

python311Packages.osmnx
python311Packages.osmnx.dist

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3542

@ofborg build python311Packages.osmnx


Missing item in the cube

Hey,
if i want to import my non-season character, in-geom and gathering of elements is in the cube. the broken crown that I actually have missing in it. This seems to be currently only with one chara the error.

o.o very suspect o.o
i know, the charakter has 3 cube items, and this a long time


Allow nullable values

I would like to add a way to validate nullable values.  I thought that the format validation might allow me to do this, but it is separate from the type validation.   I don't mind submitting the code, but I would love a pointer on where to start.
The biggest potential issue is that swagger does not officially define nullable fields. In fact, null isn't allowed in the spec. So either a spec would need to be picked or it would need to be configurable. We have been using x-nullable: true in our specs.   For example,
properties:
  id:
    type: integer
    format: int32
  organization_id:
    type: integer
    format: int32
  email:
    type: string
  first_name:
    type: string
    x-nullable: true

As per https://github.com/swagger-api/swagger-spec/issues/229, x-nullable and x-isnullable seem to be popular choices for people with nullable fields.

You should be able to do the following.
properties:
  id:
    type: [integer, null]
    format: int32
  organization_id:
    type: [integer, null]
    format: int32
  email:
    type: [string, null]
  first_name:
    type: [string, null]

The type field in json-schema allows for either a string, or an array of strings.
http://json-schema.org/latest/json-schema-validation.html#anchor79
Feel free to re-open if this doesn't solve your problem.

That is awesome, for some reason I never realized that I could do that.  It works just as expected.  Thanks for the clarification @pipermerriam.
For the record, if anyone else looks for this, you must quote the null in the yaml
first_name:
    type: [string, "null"]

otherwise you will get an error, because swagger really doesn't allow nulls:

Running into the same issue. Swagger 2.0 does NOT support null in type array.
I could work around it by building two different Swagger schemas - one that has null, one that uses x-nullable but that sort of defeats the purpose of validating the schema.
I guess the question is - whether Flex validates Swagger 2.0 schema or 2.0+ (null is expected to be in future Swagger versions)?

Flex is intended to be Swagger 2.0.  The inconsistency in handling of the null type is a bug that was introduced a long time ago due to my not noticing the discrepancy between JSON-schema and swagger's types.
I'd be happy to help in a tertiary way to get this discrepancy fixed but I don't have the bandwidth to execute on it myself at this time.

@pipermerriam I have the fix for this, going to send PR.

Re-opening.  I think I'm comfortable with this being added via the #116 PR but I want to mull it over for an afternoon.

Would like to reopen this, since i need to get JSON as follows if I leave the "first_name" field empty,
{
id: 100
organization_id: 1
email<EMAIL_ADDRESS>first_name: null
}
Please suggest me to solve my issue.

I'd suggest you go speak with the swagger developers and get them to
include nullability into the spec.
On Mon, Feb 20, 2017, 12:34 AM praveen2512<EMAIL_ADDRESS>wrote:

Would like to reopen this, since i need to get JSON as follows if I leave
the "first_name" field empty,
{
id: 100
organization_id: 1
email<EMAIL_ADDRESS>first_name: null
}
Please suggest me to solve my issue.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/pipermerriam/flex/issues/91#issuecomment-281007188,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAyTgj_7X9CNwPBaHJJcggCwRCzHcsOQks5reUIRgaJpZM4G06cp
.


Make Wrap Paragraph to Column action constrain wrapping to selected lines

Hi, and thanks for this plugin. I made some changes, mostly to make the plugin work better for my own use cases.

Fix applying plaintext filetypes option
Make Wrap Paragraph to Column action constrain wrapping to selected lines

I've also made some changes to the gradle build file to make it all compile for me.

Oh my gosh, I totally missed your PR, @mikke89! Thanks so much for opening. I saw it tonight after finally getting tired of exactly the behavior you fixed and similarly fiddling with the gradle file. A lot. 😂
I ended up refactoring a few things in the plugin code to drop some deprecated usages that were throwing warnings. I'll compare your implementation with where I landed and see how they combine!

No worries, please feel free to do whatever you wish with this PR. I'm just happy to see this extension continue to get support, it's really a must-have tool! :)

Closing because I integrated this change into #60. Thanks again!


Document how Kap is better than the built-in Screenshot app

Maybe a FAQ section in the readme. It's a common question.
https://twitter.com/sindresorhus/status/1273238540604731393

Indeed a very common question. You already mentioned several features that we'd want to highlight, I've added them to this list:

Export to GIF, MP4, WebM and APNG
Record a specific window
Set aspect ratio
Record and export with customizable FPS
Community plugins for recording system audio, uploading to Giphy, editing playback speed, and more...
Highlight mouse clicks

Highlight mouse clicks

I think the native screen recorder does this too

@sindresorhus @skllcrn , can I work on. this issue ??
I would like to add features section on readme and list all the features that @skllcrn list and put some image/gif for each of them...
happy to help this issue 😁

@asyraf-labs If you're willing to put some effort into making it good, go for it.

great @sindresorhus , I'm working on it

Hello, do you think I could work on this issue? It appears not to have been touched for a while.


game.time.slowMotion should lerp

Love what you're doing with this framework. Noticed that slowMotion makes the game feel like it's lagging (see here). Would be cool if the positions of game objects lerped between the previous and current when slowMotion is used so you still get a nice 60fps feel.

When using slowMotion you should increase desiredFps such that
60 === game.time.desiredFps / game.time.slowMotion

Seems like a non-obvious api. Perhaps this should be in the documentation for game.time.slowMotion ?

I have hunch the lag (#2631) was a bug introduced by some other change in the game loop, and this is a non-obvious workaround. :)

Thank you for taking the time to open this issue. However, official support for Phaser 2 has now ended as we focus on preparing Phaser 3 for beta release. In a bid to keep the project tidy, and avoid confusion, we are closing all v2 related issues.
This does not mean your issue won't be looked-at. In November 2016 the Phaser CE (Community Edition) project began. The aim of Phaser CE was to allow the community to continue v2 development. Since then it has gained significant traction, with many updates and releases.
If you are still interested in having your issue investigated we strongly recommend testing the latest Phaser CE release. And only if the problem persists, opening an issue on the Phaser CE repo.


Fix Test Failures Part 1

Fixes few errors raised in tests.

Codecov Report

Merging #115 (b1b2180) into master (eca9657) will decrease coverage by 0.19%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
- Coverage   58.23%   58.04%   -0.20%
==========================================
  Files          60       60
  Lines        2112     2114       +2
==========================================
- Hits         1230     1227       -3
- Misses        882      887       +5

Impacted Files
Coverage Δ

KD_Lib/Pruning/lottery_tickets/lottery_tickets.py
41.44% <66.66%> (+0.11%)
:arrow_up:

KD_Lib/KD/common/base_class.py
79.35% <100.00%> (ø)

KD_Lib/KD/vision/DML/dml.py
69.36% <100.00%> (ø)

KD_Lib/KD/vision/BANN/BANN.py
58.33% <0.00%> (-6.67%)
:arrow_down:


Expand key press handling to emit event

We have a use case where we are handling the cell-click event to perform alternate functionality in some cases.  We want to make it accessible, but there wasn't any way to handle when a user pressed Enter on their keyboard.  With this additional event, we believe we can handle that case.


Add support for headers in logged mailables

This adds support for headers provided with a logged mailable #47

Very nice! Cool test!

Very nice! Cool test!


Fixes 3485. Update jline version to 3.24.0

This closes #3485 to update Jline past 3.21.0 to version 3.24.0.
Verified that stdout and stderr output is redirected to the desired files using uno and the following:
accumulo shell -e 'tables -np' 1>/tmp/ashell.stdout 2>/tmp/ashell.stderr

The output of the stdout file:
Loading configuration from [...]/fluo-uno/install/accumulo-2.1.3-SNAPSHOT/conf/accumulo-client.properties
accumulo.metadata
accumulo.replication
accumulo.root
tableA
tableB
tableC
tableD

The output of the stderr file:
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.jline.terminal.impl.exec.ExecTerminalProvider$ReflectionRedirectPipeCreator (file:[...]/fluo-uno/install/accumulo-2.1.3-SNAPSHOT/lib/jline-3.24.0.jar) to constructor java.lang.ProcessBuilder$RedirectPipeImpl()
WARNING: Please consider reporting this to the maintainers of org.jline.terminal.impl.exec.ExecTerminalProvider$ReflectionRedirectPipeCreator
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
Oct 26, 2023 4:18:20 PM org.jline.utils.Log logr
WARNING: Unable to create a system terminal, creating a dumb terminal (enable debug logging for more information)

@EdColeman - you have this marked for 2.1.3, but are merging into main.

Changed base to 2.1


lambda-sqs-worker-cdk: use aws-codedeploy-hooks

Would be good to avoid copy-paste (via template) with https://github.com/seek-oss/aws-codedeploy-hooks.

@AaronMoat, I was trying to take a crack at this to learn more about skuba but am confused about the pre and post hook required by LambdaDeployment construct.
Question 1
Why does the construct listen to two lambdas (aws-codedeploy-hook-BeforeAllowTraffic and aws-codedeploy-hook-AfterAllowTraffic)  that are account-wide? Should it not allow the construct user to pass in their own pre and post hook lambda with their own logic? Something along this line:
 const workerDeployment = new LambdaDeployment(this, 'workerDeployment', {
      lambdaFunction: worker,
      preHook: new aws_lambda.Function(this, 'preHookWorker', {
        architecture: aws_lambda.Architecture[architecture],
        code: new aws_lambda.AssetCode('./lib'),
        runtime: aws_lambda.Runtime.NODEJS_18_X,
        handler: 'preHook.handler',
        functionName: 'aws-codedeploy-hook-AfterAllowTraffic',
      }),
      postHook: new aws_lambda.Function(this, 'postHookWorker', {
        architecture: aws_lambda.Architecture[architecture],
        code: new aws_lambda.AssetCode('./lib'),
        runtime: aws_lambda.Runtime.NODEJS_18_X,
        handler: 'postHook.handler',
        functionName: 'aws-codedeploy-hook-AfterAllowTraffic',
      }),
    });

Question 2
I see that the two hooks are created here. I also noticed that the pre and post lambda uses the same index.handler here. Is there some ways for user to specify different logic for pre and post hook that I am missing?

Hi @aurelius0523! Happy to iterate on the design of aws-codedeploy-hooks, but there are some simplicity benefits to the deploy-once strategy. I could see a world where both are supported? cc @72636c
Inside the functions, it’s internally forking to different codepaths. We could point to seperate files as well, I think this was just convenience.

I have created a PR to move to aws-codedeploy-hooks as is. Happy to discuss further. No rush, I am just trying to explore libraries at seek 😄

Apologies for the late revert @AaronMoat @aurelius0523! How I was thinking this would work in practice is as follows:

Team looks to deploy their first Lambda function with the CDK
They deploy it in a seemingly self-contained manner, i.e. they would include both HookStack and LambdaDeployment in the CDK app
Team looks to deploy their second Lambda function with the CDK
They can now move HookStack to its own CDK app given it will be shared across the two functions

While I'm not fully convinced yet, I can definitely see some upside in providing the option for teams to continue to keep things separate in Step 4 🙂


BUG _Table classes are not generated with 4.0.0-beta3 but works with 4.0.0-beta2

WORKS:
dependencies {
	annotationProcessor 'com.github.Raizlabs.DBFlow:dbflow-processor:4.0.0-beta2'
	compile 'com.github.Raizlabs.DBFlow:dbflow:4.0.0-beta2'
}

ERROR:
dependencies {
	annotationProcessor 'com.github.Raizlabs.DBFlow:dbflow-processor:4.0.0-beta3'
	compile 'com.github.Raizlabs.DBFlow:dbflow:4.0.0-beta3'
}

i don't know how to see log of annotationProcessor

anyone has this issue?

do you have other annotation processor dependencies?

no, this is the complete list:
dependencies {
	compile 'com.android.support:appcompat-v7:25.1.0'
	compile 'com.android.support:support-v13:25.1.0'
	compile 'com.android.support:cardview-v7:25.1.0'
	compile 'com.google.code.gson:gson:2.8.0'
	compile 'com.squareup.retrofit2:retrofit:2.1.0'
	compile 'com.squareup.retrofit2:converter-gson:2.1.0'
	compile 'com.github.xanscale:localhostToolkit:-SNAPSHOT'
	annotationProcessor 'com.github.Raizlabs.DBFlow:dbflow-processor:4.0.0-beta2'
	compile 'com.github.Raizlabs.DBFlow:dbflow:4.0.0-beta2'
}

@agrosner this is the commit that breaks my build. i don't know why seams only test
https://github.com/Raizlabs/DBFlow/commit/d6381258908d36a1f00b3bc1ad3afe5b685b0879

Maybe the 2.2.2 to 2.2.3 is the issue?
Try adding this to your build.gradle:
gradle.projectsEvaluated {
  tasks.withType(JavaCompile) {
    options.compilerArgs << "-Xmaxerrs" << "500" // or a larger number
  }
}

And see if more issues pop up that describe what happened.

i found the problem but i don't know why.
Changing this line:
public static Indicatore querySingle(@NotNull Long id) {.......}
in this line
public static Indicatore querySingle(Long id) {.......}
works with both beta2 and beta3.
PS in my app i use
classpath 'com.android.tools.build:gradle:2.2.3'
why now i can't use NotNull Annotation in parameters?

i found the problem but i don't know why.
Changing this line:
public static Indicatore querySingle(@NotNull Long id) {.......}
in this line
public static Indicatore querySingle(Long id) {.......}
works with both beta2 and beta3.
PS in my app i use
classpath 'com.android.tools.build:gradle:2.2.3'
why now i can't use NotNull Annotation in parameters?

because the NotNull is either DBFlow not null and thats for fields only, or its the sun.istack.NotNull which doesnt actually exist in Android runtime...

I Confused
android.support.annotation.NonNull
with
com.raizlabs.android.dbflow.annotation.NotNull
@agrosner why use own annotation for this?

@NotNull is define a column via SQLite as not being null...

@agrosner why not use "standard" annotation for that? now if i want to define column NOT null and relative variable i need to use 2 annotation.


Constantly increasing CPU and memory usage after opening a directory with Atom

Reproduction:

Close every Atom window, if running.
Open a directory with atom by running  atom /path/to/directory/.  In my case atom /var/www/html/data/. It doesn't happen with other directories!

Expected behavior:
Atom should open with /var/www/html/data/ opened in tree-view.
Observed behavior: (See Screenshots)
Atom opens with /var/www/html/data/ opened in tree-view.
CPU and memory usage gets high.
And if i don't close project folder in tree-view, it occupies all memory and leads to frozen system.
But, if I close project folder in tree-view before it occupies all the memory, CPU and memory usage gets low and atom runs normally.
Screenshots
Opening with nautilus file manager or by running atom /var/www/html/data/ in gnome-terminal

Opens Following window

Ran for a minute with constantly increasing CPU and memory usage, system started becoming slow, I also left it for more than 3 minutes with this condition to confirm this issue and my system gets unresponsive/frozen. Tested it more than once and it occurs every time.

After closing Atom

Solution
I disabled fuzzy-finder package, and issue is gone.
Installed packages:
linter
linter-php
minimap
tool-bar
tool-bar-main
Additional information:
$ atom --version
Atom    : 1.7.3
Electron: 0.36.8
Chrome  : 47.0.2526.110
Node    : 5.1.1

$ lsb_release -a
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04 LTS
Release:	16.04
Codename:	xenial

$ uname -r
4.4.0-22-generic

Problem can be reproduced in safe mode: [Yes]
Problem started happening recently, didn't happen in an older version of Atom: [No]
Problem can be reliably reproduced, doesn't happen randomly: [Yes]
Problem happens with all files and projects, not only some files or projects: [No]

@sagarpanchal Can you please try the latest Atom 1.8.0 beta and see if that fixes your issue?

@50Wliu Yes

@50Wliu I can reproduce same issue with atom-beta [https://github.com/atom/atom/releases/download/v1.8.0-beta3/atom-amd64.deb] with fuzzy-finder package enabled.
And I can also solve it by disabling fuzzy-finder package.
$ atom-beta --version
Atom    : 1.8.0-beta3
Electron: 0.36.8
Chrome  : 47.0.2526.110
Node    : 5.1.1

@50Wliu I can reproduce same issue with atom-beta [https://github.com/atom/atom/releases/download/v1.8.0-beta3/atom-amd64.deb] with fuzzy-finder package enabled.
$ atom-beta --version
Atom    : 1.8.0-beta3
Electron: 0.36.8
Chrome  : 47.0.2526.110
Node    : 5.1.1

Can you give an example of a publicly accessible directory where we can reproduce this problem? What is unique about your /var/www/html/data directory that causes the problem?

@lee-dohm output of tree /var/www/html/data/
/var/www/html/data/
├── about.html
├── config
│   ├── color.conf.php
│   └── server.conf.php
├── cookie-manager.php
├── error.php
├── favicon.ico
├── filelinker-main.php
├── filelinker-router.php
├── images
│   ├── bg
│   │   ├── Blue.png
│   │   ├── checkbox-checked.svg
│   │   ├── checkbox-unchecked.svg
│   │   ├── Custom.png
│   │   ├── Gray.png
│   │   ├── Green.png
│   │   └── Orange.png
│   ├── checkbox-checked.svg
│   ├── checkbox-unchecked.svg
│   ├── go-home.svg
│   ├── go-previous.svg
│   └── icons
│       ├── css.png
│       ├── file.png
│       ├── folder.png
│       ├── image.png
│       ├── office.png
│       ├── php.png
│       ├── script.png
│       ├── sound.png
│       ├── video.png
│       ├── word.png
│       ├── xml.png
│       └── zip.png
├── menu.php
├── redir.php
├── scripts
│   ├── jquery.min.js
│   ├── list.min.js
│   ├── menu.js
│   ├── search.js
│   └── sorttable.js
├── styles
│   ├── animate.css
│   ├── color_converter.php
│   ├── colors.php
│   ├── css.php
│   ├── FiraCode.otf
│   ├── menu_css.php
│   ├── normalize.css
│   ├── tooltip_css.php
│   └── Ubuntu.ttf
└── tree.txt

6 directories, 48 files

/var/www/html/data$ du -ha
4.0K	./about.html
8.0K	./error.php
4.0K	./menu.php
4.0K	./redir.php
8.0K	./styles/css.php
4.0K	./styles/color_converter.php
4.0K	./styles/tooltip_css.php
8.0K	./styles/menu_css.php
4.0K	./styles/colors.php
8.0K	./styles/normalize.css
76K	    ./styles/animate.css
148K	./styles/FiraCode.otf
348K	./styles/Ubuntu.ttf
612K	./styles
4.0K	./images/go-home.svg
4.0K	./images/go-previous.svg
40K	    ./images/bg/Blue.png
32K	    ./images/bg/Custom.png
48K	    ./images/bg/Green.png
4.0K	./images/bg/checkbox-unchecked.svg
4.0K	./images/bg/checkbox-checked.svg
44K	    ./images/bg/Gray.png
44K	    ./images/bg/Orange.png
220K	./images/bg
4.0K	./images/checkbox-unchecked.svg
4.0K	./images/checkbox-checked.svg
4.0K	./images/icons/sound.png
4.0K	./images/icons/file.png
4.0K	./images/icons/xml.png
4.0K	./images/icons/word.png
4.0K	./images/icons/office.png
4.0K	./images/icons/folder.png
4.0K	./images/icons/php.png
4.0K	./images/icons/video.png
4.0K	./images/icons/image.png
4.0K	./images/icons/css.png
4.0K	./images/icons/zip.png
4.0K	./images/icons/script.png
52K	    ./images/icons
292K	./images
20K	    ./scripts/sorttable.js
136K	./scripts/jquery.min.js
4.0K	./scripts/menu.js
4.0K	./scripts/search.js
28K	    ./scripts/list.min.js
196K	./scripts
4.0K	./filelinker-router.php
24K	    ./favicon.ico
4.0K	./cookie-manager.php
12K 	./filelinker-main.php
4.0K	./config/server.conf.php
4.0K	./config/color.conf.php
12K	    ./config
1.2M	.

@lee-dohm
I tested the copy of /var/www/html/data and found a symbolic link to (parent directory) /var/www/html/ that must be causing the issue.
I deleted the symbolic link to parent directory and relaunched atom and everything worked fine.

Ah, thank you @sagarpanchal. I'm pretty sure this is a dupe then of another Issue. I'll track it down tomorrow.

This is a duplicate of https://github.com/atom/atom/issues/10286. You may want to subscribe there for updates.


Delete on upgrade

Is there an easy way to clear out the entire database and start fresh?
Some other libraries have a Delete On Upgrade setting in the configuration that is very useful when developing. Most of the time during pre-release development I don't care about upgrading a schema, I just want it deleted.

Currently onUpgrade will attempt to add missing tables/columns but this could be made configurable. You can do this currently extending DatabaseSource and overriding onUpgrade with something like:
@Override
public void onUpgrade(final SQLiteDatabase db, int oldVersion, int newVersion) {
    SchemaModifier schema = new SchemaModifier(getConfiguration());
    schema.createTables(TableCreationMode.DROP_CREATE);
}


Connect UI context cleanup

PR-Codex overview
This PR focuses on cleaning up the Connect UI context code by replacing the usage of useWalletConnectionCtx with useConnectUI.
Detailed summary

Replaced useWalletConnectionCtx with useConnectUI in multiple files for context code cleanup.

The following files were skipped due to too many changes: packages/thirdweb/src/react/web/wallets/shared/ScanScreen.tsx, packages/thirdweb/src/react/web/wallets/shared/ConnectingScreen.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/AnyWalletConnectUI.tsx, packages/thirdweb/src/react/core/providers/wallet-connection.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/screen.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/NetworkSelector.tsx, packages/thirdweb/src/react/web/wallets/shared/WalletConnectConnection.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/AllWalletsUI.tsx, packages/thirdweb/src/react/web/wallets/in-app/InAppWalletSocialLogin.tsx, packages/thirdweb/src/react/web/wallets/in-app/InAppWalletSelectionUI.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/screens/StartScreen.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/SmartWalletConnectUI.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Details.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectModalInline.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectModalContent.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectModal.tsx, packages/thirdweb/src/react/web/wallets/in-app/InAppWalletOTPLoginUI.tsx, packages/thirdweb/src/react/web/wallets/in-app/InAppWalletFormUI.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Modal/ConnectEmbed.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/WalletSelector.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/ConnectWallet.tsx, packages/thirdweb/src/react/web/providers/wallet-ui-states-provider.tsx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

/release-pr

changes make sense - need to do manual testing since we don't have tests for this yet

Yep - did manual testing. You can try it out on https://internal-test-next-git-mnn-test-connect-ctx-refactor.thirdweb-preview.com/


Quantity unit checking in some methods fails for equivalent, but not identical, units

For instance, if a method wants an angle argument, and the calculation expects it to be degrees, some methods (like _setup_new_region), will fail if the argument is not in degrees, even though an argument in arcseconds would be totally fine.

Sorted this by using the unit.is_equivalent method for astropy units.


Export or Import problem

When i export playlist it's save to "text/plain" type, not to "application/json" Link to export function
I thing the type specified in 	let file = new Blob([data], { type: "text/json" }); is wrong..
must be "application/json" ?
But  this it does not matter, because i tried to "application/json" and don't work..
And is show me "File is not supported."
I attached a screen photo of debug when i trying to import play list.

Seems like Chrome on windows determine mime type as "" (empty string), on linux as "text/plain".
Changed file type validation to some old school approach. e15978ec285e60431c7f162d3bca283b73d21909
Thanks for reporting, cheers.


Create Rake dist task

Create a Rake dist task to output:

client including index.html, JS bundle, version.txt
JAR file for API

Closing this as I think it's superfluous.


⚠ Bofrost.be has degraded performance

In 8c87259, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1563 ms

Resolved: Bofrost.be performance has improved in e0dd7c7.


Fix: removed entities were not synced to the engine

Hi.  I'm not 100% sure if I'm right because I'm didn't really know the Amethyst internals and the timing of the FlaggedStorage events but I had some issues when it comes to sync deleted entities back to the engine.  I saw they were never deleted.  Only if I insert an entity again, which got the id of a previously deleted entity, specs-physics detected it and deleted it.
I debugged a little bit and saw that it the join loop, where the sync calls are placed, the id of the deleted entities never appeared.  This makes sense since they are already deleted.  What I did is simply remove all the 'remove-ID's' outside of the loop.
Not sure if this can have any other side effects but it works perfectly on my end with this fix.

Oh, I think this probably fixes #13


How to get currentItem on render / if onSnapToItem not fired.

Is this a bug report or a feature request?
Neither both. This is a simple question.
Have you read the guidelines regarding bug report?
Yes I did. But couldn't find.
Have you read the documentation in its entirety?
Yes I did. But couldn't find.
Have you made sure that your issue hasn't already been reported/solved?
Yes I did. But couldn't find.
Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
Trying on iOS simulator.
Is the bug reproductible in a production environment (not a debug one)?
Nope.
Have you been able to reproduce the bug in the provided example?
I didn't tried. Should I?
Environment
Environment:
React: 16.3.1
React native: 0.55.3
react-native-snap-carousel: ^3.7.0
Target Platform:
iOS (11.3 15E217)
Expected Behavior
I'm expecting currentCard variable should be set on render.
I'm calling onCardActionActivateDeactivate() method without sliding(or snap?) cards and this.state.currentCardDetail is an empty object.
Actual Behavior
currentCard variable be set after "onSnapToItem" method is fired.
Reproducible Demo
class Cards extends React.Component {
  constructor(props) {
    super(props);
    this.state = {
      currentCardDetail: {}
    };
  }

  onCardActionActivateDeactivate() {
    this.props.navigator.showModal({
      screen: 'cards.activateCard',
      passProps: {
        cardDetail: this.state.currentCardDetail
      }
    });
  }

  renderCardItem ({item, index}) {
      return (
        <View>
            <Button onPress={() => onCardActionActivateDeactivate()}><Label>{I18n.t('Cards.appealButton')}</Label></Button>
        </View>
      );
  }

  renderCards() {
    let cardsWithRequestNewCard = this.props.cards.cards.slice();
    cardsWithRequestNewCard.push({
      isRequestNewCard: true
    });
    return (
      <Carousel
        ref={(c) => { this._carousel = c; }}
        data={cardsWithRequestNewCard}
        renderItem={this.renderCardItem.bind(this)}
        sliderWidth={555}
        sliderHeight={55}
        itemWidth={555}
        onSnapToItem={(index) => {
          this.setState({ currentCardDetail: cardsWithRequestNewCard[index] }) // This works
        }}
      />
    );
  }

return (
      <Base>
        {this.renderCards()}
      </Base>
    );

  }

Hi @tayfunyasar,
I think you should be able to use the currentIndex getter.

thanks this helped a lot.... ;)


Update setup.py

import elementary_math as package will be fail when package has install_requires and install from PyPI mirror(like https://pypi.tuna.tsinghua.edu.cn/simple)
The Error like this one
        import skimage
ModuleNotFoundError: No module named 'skimage'
   ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-uwfirk47/boxx/

So I change the code to avoid import package

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #1   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines          50     50
=====================================
  Hits           50     50

Continue to review full report at Codecov.

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


vdc helm chart - warning on chart deployment

when deploying a chart an error such as
➜  charts git:(helm22) ✗ helm install vdc -f ~/cloud.yaml                                                  git:(helm22|✚3
2019/03/07 15:02:12 Warning: Building values map for chart 'cp-kafka'. Skipped value (map[]) for 'image', as it is not a table.
Appears  - due to likely incorrect formatting in values.yaml
low priority but should fix

Seems to be an embedded error from the cp-kafka Helm chart we're re-using, I think?

yes that's my perception too - if nothing else I should report to that team so will retain the issue for that. Perhaps even a PR..

I think this is no longer an issue after moving to the Bitnami charts?

agreed


some of the icons are mapped wrong

I was loading a combo with the available icons and I noticed that some are bringing broken values, including duplicates.

Could you create example on https://blazorfiddle.com/ ?

@enginexon on the icons page itself, it already shows this.

@SamProf I confirm it, bug can be found in the device section on the icon's demo page device section.
https://www.matblazor.com/Icon

@drma-dev I could suggest a workaround until it will be fixed
static MatIconDataIcon[] GetValidMatIconDataIcons()
{
    var excludeIconIds = new HashSet<string>(new string[]
    {
        "play_circle_filled_white",
        "battery_20",
        "battery_30",
        "battery_40",
        "battery_50",
        "battery_60",
        "battery_70",
        "battery_80",
        "battery_90",
        "battery_charging_20",
        "battery_charging_30",
        "battery_charging_40",
        "battery_charging_50",
        "battery_charging_60",
        "battery_charging_70",
        "battery_charging_80",
        "battery_charging_90",
        "signal_cellular_1_bar",
        "signal_cellular_2_bar",
        "signal_cellular_3_bar",
        "signal_cellular_connected_no_internet_1_bar",
        "signal_cellular_connected_no_internet_2_bar",
        "signal_cellular_connected_no_internet_3_bar",
        "signal_wifi_1_bar",
        "signal_wifi_2_bar",
        "signal_wifi_3_bar",
        "signal_wifi_1_bar_lock",
        "signal_wifi_2_bar_lock",
        "signal_wifi_3_bar_lock",
        "",
    }, StringComparer.InvariantCultureIgnoreCase);

    return MatIconCategories.Data
        .SelectMany(x => x.Icons)
        .Where(x => !excludeIconIds.Contains(x.Id))
        .ToArray();
}


Replaced WebL10n with i18next in Abacus activity

Fixes #1378
Locales work, no errors in console.
Had to remove an eventListener to make it work after referring to qrcode.activity migration.

Nice. Thanks.


minify transform

div {transform:scaleX(0.5);}

div {transform:scaleX(.5);}

Also, scale and scaleY.
Also,
@keyframes rollIn{0%{transform:translateX(-100%) rotate(-120deg)}100%{transform:translateX(0px) rotate(0deg)}}

could be
@keyframes rollIn{0%{transform:translateX(-100%) rotate(-120deg)}100%{transform:translateX(0) rotate(0)}}

Plus 0%.
Another thing that seems to work is removing spaces before rotate, translateXYZ and scale.

foo {
-webkit-transform:translateX(100%) rotate(120deg)
}

foo {
-webkit-transform:translateX(100%)rotate(120deg)
}

@XhmikosR please feel free to give https://github.com/jakubpawlowicz/clean-css/tree/i/182-transforms a try.

Sure thing, I'll try it now with animate.css.

Perfect :+1:

Damn the branch names you choose cause issues with npm i :/
This fails for me
"clean-css": "jakubpawlowicz/clean-css#i/182-transforms",

Anyway, I did it manually, but, please try using sensible branch names so that we don't have such issues.
3.0.2: 53201 bytes
this:  53017 bytes

Seems to be doing the trick :+1:

On a side note, any reason why 2.x is still listed as the default one on npm?

Seems like a bug on npm site as I pushed 2.2.23 after 3.0.2 today. Lesson for the future not to.

@jakubpawlowicz: can you ping me when a new version is out? TIA.

Defintiely :+1:


merge and tsswitch

is-it possible to use a tsswitch just behind a merge. In fact I want to use the result of the switch to merge it in a MPTS.
Thanks

The merge plugin uses the output of the command it launches. So, to use the output of a tsswitch command as input of a merge plugin, the global command should look like this:
tsp ... -P merge "tsswitch -I ... -I ..." ...

Note the quote around the complete tsswitch command because the whole command is a parameter of the merge plugin.
Of course, if the tsswitch command is long, I would recommend to either write it in a shell script and run that shell script in the merge plugin or use the TSDuck partial command line redirection from a file (see the TSDuck user's guide, section 3.1.5).

Perfect! it works very well. (a quote forgotten in a quite long command )
Thanks!


Create MS Learn module: Cloud-native ASP.NET Core microservices

Plan, write, and publish an MS Learn module for cloud-native microservices.

[x] Module planning and scoping
[x] Create a proof-of-concept
[x] Create module proposal
[x] Gather feedback on module proposal and adjust as necessary
[x] Create Bash script to provision Azure resources
[x] Create the starter project
[x] Request module badge
[x] Create the code sample repo
[x] Write the module
[x] Write knowledge check
[ ] Obtain approvals and publish

PR is available at https://github.com/MicrosoftDocs/learn-pr/pull/9914

The module was published on 5/29.


Not training using GPU

Hello!
Thanks for sharing your implementation. However, the loss went to Nan on the latest commit so, following https://github.com/werner-duvaud/muzero-general/issues/40, I used this tree: https://github.com/werner-duvaud/muzero-general/tree/f5dd3d2a3fd2e0c354731112b23c2d4c55811914. However, when I check nvidia-smi, it shows that only around 100mb of my GPU is used:+-----------------------------------------------------------------------------+
| NVIDIA-SMI 440.64.00    Driver Version: 440.64.00    CUDA Version: 10.2     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX 108...  On   | 00000000:02:00.0  On |                  N/A |
| 25%   60C    P2    94W / 250W |   1553MiB / 11177MiB |     30%      Default |
+-------------------------------+----------------------+----------------------+
+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|    0      3234      G   /usr/lib/xorg/Xorg                           265MiB |
|    0     15010      G   ...uest-channel-token=17125473231410870467   115MiB |
|    0     15854      G   ...AAAAAAAAAAAACAAAAAAAAAA= --shared-files   404MiB |
|    0     18818      C   ray::Trainer.continuous_update_weights()     741MiB |
|    0     27228      G   ...charm-community-2018.1.4/jre64/bin/java    11MiB |
+-----------------------------------------------------------------------------+
and the program is heavily reliant on CPU. I noticed in the latest commit, this was fixed. Any ideas on how I can get the model to run on GPU?
Thanks,
Weichen

Turns out it was selfplay using up the CPU.


Added suport for encrypted private keys (by a passprase) in DKIM Signer

Problem
I noticed that when I want to use a private key, encrypted with a passprase, it was not possible to pass this passprase as a parameter, causing the openssl_get_privatekey() method to throw an exception.
This method accepts as an optional parameter a string passphrase
Documentation: http://php.net/manual/es/function.openssl-pkey-get-private.php
Added

passphrase attribute
passphrase parameter as optional
second parameter passphrase to openssl_get_privatekey()

Is there a problem with Travis and PHPUnit on PHP>7?
PHP Fatal error:  Class 'PHPUnit_Framework_Constraint' not found in /home/travis/build/swiftmailer/swiftmailer/tests/IdenticalBinaryConstraint.php on line 8

@dariogz Seems that a newer phpunit is used under php7x https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-6.0.0

Thank you @dariogz.


Merge Dev

Current coverage is 83.53% (diff: 81.94%)

No coverage report found for master at a3f30ba.

Powered by Codecov. Last update a3f30ba...4f9df72


Added raw parameter to androguard.misc AnalyzeDex and AnalyzeODex…

…` functions, to support raw data (not from disk)

Codecov Report

Merging #819 into master will decrease coverage by 0.07%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #819      +/-   ##
==========================================
- Coverage   76.83%   76.75%   -0.08%
==========================================
  Files          51       51
  Lines       16293    16269      -24
==========================================
- Hits        12518    12487      -31
- Misses       3775     3782       +7

Impacted Files
Coverage Δ

androguard/misc.py
60.78% <50.00%> (-1.47%)
:arrow_down:

androguard/core/__init__.py
82.30% <0.00%> (-7.76%)
:arrow_down:

androguard/decompiler/dad/decompile.py
53.41% <0.00%> (+0.22%)
:arrow_up:

Continue to review full report at Codecov.

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


Display objects that have both a child/parent relationship in EnsoGL and in the DOM, should be positioned correctly.

This task is automatically imported from the old Task Issue Board and it was originally created by Michael Mauderer.
Original issue is here.

Why
As Developer
I want to be able to parent DomElements with each other while also use the EnsoGL API to set their position.
So that I can create visuals based on DOM elements with ease.
Acceptance Criteria
Scenario:
GIVEN the demo scene pasted here: https://pastebin.com/G7Hc5qe6 with line 56 uncommented
WHEN looking at the demo scene
THEN the two square should move in sync.

Notes
How this should be fixed should be planned in more detail. See the discussions below for the current state of the discussion.
Resources
Discord thread with discussion about this issue: https://discord.com/channels/401396655599124480/1050037851376586812

EnsoGL is removed from repository.


Update required Rust version

According to the README, 1.53.0 should be enough but it seems that we require something between 1.60.0 (which fails) and 1.65.0 (which works) nowadays.

Yeah, we're using some features which were not stabilized back then. The current MSRV is 1.62 due to the addition of total_cmp. We might consider bumping this in the README, although I would suggest changing the guarantee to the latest stable version, since this a research stack after all. The compiler should always work with older code so we atleast don't have to worry about faulty documentation. Do you have any preference?

Recommending the latest version sounds good to me. Can we still add something to one of the TOML files indicating that at least 1.62.0 is needed so we get a useful error message instead of a failed build?

Yeah we can add to the Cargo.toml of betree the rust-version field which does exactly that (https://doc.rust-lang.org/cargo/reference/manifest.html#the-rust-version-field). Also, just saw that someone implemented a binary search for this https://github.com/foresterre/cargo-msrv :D They also have a verification command to see if the rust-version field is still valid. Since this is very little effort, we can put the verification in a small additional job in the CI.


fix printf when reaching end of printf buffer

If we do not have the space to store all the arguments, we should not store them. It leads to memory corruption from out-of-bounds access on platform like lavapipe.
I thought of merging the tests for the printf id and the args, but we need both of them.
Here is an example to understand:
Considering we have a printf with 1 argument and a buffer size of 8 bytes (2 words).
The first word is for the counter of number of words written. We are left with one word.
When performing the printf, we will execute the atomic add, which will set the counter to 2 (1 for the printfid and 1 for the arg). If we do not store the printf id, the runtime won't be able to know whether we are trying to print 2 string without argument (each one needing 1 word) or if we are printing 1 string with 1 argument.

Printf needs to store both the printf id and the args.
If we have space for everything, we store everything, but if we just have space for the printf id, we need to store it and not the args. So in the end we have 2 checks. One to know whether we have space for the printf id, and then one for the args.
We could rearrange that, but we would still have 2 checks in the end I believe.
Here is what we have at the moment:
if (more than 1 word left in the buffer) {
  store printf id
  if (enough space for the args in the buffer) {
    store args
  }
}

As explained in the commit message, we really want to store the printf id if we have the space for it even if we do not have the space for the args.
About the test added, we are not checking the same thing in the entryBB and headerBB. In entryBB we are comparing against atomicadd+1 and in headerBB against atomicadd+2.

I can reorganise the test so that we have only 1 test in the best case:
entry:
  cmp = enough space for everything
  br cmp, copy_args, test_printf_id
copy_args:
  store args
  br copy_printf_id
copy_printf_id:
  store printf_id
  br end
test_printf_id:
  cmp = enough space for printf id
  br cmp, copy_printf_id, end
end:
  return value

What do you think of that?

I can reorganise the tests so that we have only 1 test in the best case:
entry:
  cmp = enough space for everything
  br cmp, copy_args, test_printf_id
copy_args:
  store args
  br copy_printf_id
copy_printf_id:
  store printf_id
  br end
test_printf_id:
  cmp = enough space for printf id
  br cmp, copy_printf_id, end
end:
  return value

What do you think of that?

What is the value of printf if we can't fit the args. Does CTS cover that case? It seems unnecessary. If we don't have space for everything just don't print. If that is not allowed by the spec then your suggestion is an improvement.

What is the value of printf if we can't fit the args. Does CTS cover that case? It seems unnecessary. If we don't have space for everything just don't print. If that is not allowed by the spec then your suggestion is an improvement.

CTS does not cover that case. It is hard for it to do it because even so OpenCL has CL_DEVICE_PRINTF_BUFFER_SIZE, it does not specify what is in the buffer exactly, thus making it hard to know when the buffer should be full.
But the issue here is about the fact that any application could completely fill the buffer. Then the question is how does the runtime handle it?
In the current implementation, the runtime has the information of the number of words written in the buffer (stored in the first word of the buffer), the content of the buffer and the size of the buffer.
To know when to stop reading the buffer, the runtime is using the number of words written as long as it is smaller than the size of the buffer (meaning the buffer is not full).
But when the buffer is full, we have 2 possibilities:

the last printf stored its last word in the last word of the buffer. In that case, it is easy and the runtime knows there is nothing after and it stops.
the buffer is too small to store all the printf, thus we have a number of words written greater than the buffer size because whether or not we store something in the buffer, printf is atomicaly incrementing this counter (the number of words written). And we need to know when the last one is. As we do not have some kind of marker telling us that it is the end, as long as we did not reach the end of the buffer, we are looking for the printf id, and then we know how many argument we should expect. But if the first printf overflowing does not write its ID, it means that the runtime will read a random value in the buffer and will consider it as a printf id. Which can lead to various issues.

Please have a look at https://github.com/kpet/clvk/pull/585/files. There we have some tests for that. At the moment they are passing because we are lucky. Swiftshader zero allocates the buffer, thus allowing the test to pass. But running with lavapipe, it fails because we are missing this fix.

I think it also passes at the moment because the current implementation has an issue with the buffer size (fix here), making it write word out-of-bounds. It does not generate a bug at the moment, but could easily change if we change anything.


Testing if comment triggers ontobot

Irrelevant stuff

Hey ontobot! apply:

create narrow synonym 'Low-flow priapism' for MONDO:0004745

Hey ontobot!, apply:

create exact synonym "cortical visual impairment" for MONDO:0001385

Hey Ontobot! apply:

create narrow synonym 'Low-flow priapism2' for MONDO:0004745

Hey Ontobot! apply:

create narrow synonym 'Low-flow priapism3' for MONDO:0004745


training log file and curves

Hi @edgarschnfld,
Thanks for your paper and code, I was wondering do you have log file  and training curves,
I am interested in reconstruction loss and kld losses and Wasserstein distance curves while training, and their coefficient,
I would appreciate if you share your trained log files to us. It would be so useful,

Hey, I don't have log files or training curves, unfortunately. However, running the code takes about 5-10 minutes, so it should be easy to write the loss values to a file and compare. Sorry I couldn't help, but feel welcome to write me if you have more questions.
Best,
Edgar