Use Kolibri stable PyPI releases

Now that 0.16.0 has been released, we have no reason to use Kolibri pre-releases. Switch to using PyPI and specify that only stable (non-pre-release) versions should be used. Kolibri pre-releases aren't published to PyPI, but let's be safe in case they start doing that.

I'm going to merge this since I'm tired of the failed update checker emailing me every hour. Any post-hoc review would be appreciated.

Yep, that fixes the update checker.

Excellent! This should be QA'ed.


Notchmeister window opens unprompted on all desktops when multiple desktops are open

Summary:
If the user has multiple desktops open, the Notchmeister window will randomly open on all desktops; this happens even when the user has selected 'Assign to...All Desktops' or 'Assign to...This Desktop' in Notchmeister options.
Steps to Reproduce:

Open multiple desktops
Open Notchmeister
After a few minutes, swipe to other desktops

Expected Behavior:
Notchmeister window shouldn't open without user input; if it does open, it should be on assigned desktop only
Actual Behavior:
Notchmeister window opens unprompted on all desktops multiple times a day
App version:
v1.0 (15)
System Version:
macOS Monterey
Notes:
See attached screen recording
https://user-images.githubusercontent.com/70352484/148466625-a69d2591-7834-4886-81ff-8456cf5aaa21.mov

It's not clear from the video what is causing the window to open unprompted.
Notchmeister's window is configured to appear in all spaces (effectively ignoring any assigned desktop). The thinking here is that the notch appears in every space, so the window that configures it should as well.

I'm pretty sure the behavior described here "opening unprompted" is a natural consequence of the use of Spaces and Notchmeister's window being marked as having a "transient" collection behavior. With this collection behavior I think even if the window had previously been closed, activing a new Space will cause the window to be made visible again.

I think @danielpunkass's description most closely matches the behavior I'm seeing. I love Notchmeister as it is so much fun, but the window popping up pretty frequently is a bummer. Although I'm not sure how involved this is, it seems like it would be addressed by making a 'Preferences' menu item where the user could 'set it and forget it.' Or add an option to 'Don't show this window again' after which the user could change effects by secondary clicking on the app icon. There may be other solutions as well, but it seems the current behavior assumes that the user wants to change the effect every time they swipe to a new desktop, which I feel should be mitigated somehow.

@dmalachi I'm curious if the latest TestFlight build solves this problem. Part of the problem with transient windows is that they get "activated".
With the new "Hide Dock icon" feature, there's no app to activate. In theory, this will prevent the settings window from appearing as spaces are switched.
Here's the TestFlight: https://testflight.apple.com/join/Nj8aKRvl

This TestFlight is for an app called Tot...is that connected to Notchmeister in some way?

Also, after reading over what logs and stats TestFlight would automatically send to you guys, I don't think my company NDAs would allow for me to run a TestFlight build and send log info. So I might have to wait to test until it's a public release.

@dmalachi I'm curious if the latest TestFlight build solves this problem. Part of the problem with transient windows is that they get "activated".
With the new "Hide Dock icon" feature, there's no app to activate. In theory, this will prevent the settings window from appearing as spaces are switched.
Here's the TestFlight: https://testflight.apple.com/join/Nj8aKRvl

The TestFlight link that you provided is for your other Mac app! 😅

Yeah, whoops, wrong link. Here’s the correct one: https://testflight.apple.com/join/pIGViV74
@dmalachi Logs and other private info are not sent automatically. The only thing recorded is that you (anonymously) installed and launched the app. As with everything else Apple does, it very privacy oriented.

@dmalachi Does the most recent release on the App Store fix this problem? I'd like to close this issue.

Ah, very sorry. Haven't logged in on my laptop in a couple weeks. Let me update the app and get back to you shortly!

Yes, it seems the issue has been resolved with the latest release. Thank you!


fixed PropTypes import and pass modal's animationType as prop

@d-a-n could we merge this please? helps make it compatible with react-native 0.47


Empty files aren't downloaded

Empty files get added to metadata but never get transferred or downloaded.
Reproduce:

Run dat link in dat tests fixtures folder (with empty.txt)
Download the dat link to another folder.
folder and empty.txt are missing but metadata gets read (they appear in createEntryStream)

we need to create an empty file here instead of returning right away to fix this, https://github.com/mafintosh/hyperdrive/blob/master/index.js#L159

Fixed in 3.3.1

Same issue as before. They are added to metadata but never get created.

We should add a test for this then

It seems archive.download needs to special case where start == end.  I tried simply doing:
        self.options.storage.openAppend(entry.name, true)
        self.options.storage.closeAppend(cb)

but that doesn't seem to work as raf thinks it's noop.


fix(ecr): enable Docker outputs for Eliza Docker asset creation

#52  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Add support for YML migration files

Add support for writing migration files in yml
Abstracting the parameters for creating/updating tables to a simple yml file would make the code easier to read and maintain and reduces boilerplate code

The migrations are easily written in go itself as per the current implementation.
Reference: https://gofr.dev/docs/advanced-guide/handling-data-migrations
So, I believe adding an extra layer over migrations would not be a good thing to do.


List Orders By NextToken

I'm just wondering what would be the best approach to get orders from a entire year ?
I can get the proper NextToken value after the first request, but then Amazon returns a 503 response on the 2nd one:
error  { Type: {},   Code: 'RequestThrottled',   Message: 'Request is throttled',   Headers:    { 'x-mws-quota-max': 'unknown',      'x-mws-quota-remaining': 'unknown',      'x-mws-quota-resetson': 'unknown',      'x-mws-timestamp': '2019-04-17T10:10:30.346Z',      'content-type': 'text/xml',      'content-charset': 'unknown',      'content-length': '291',      'content-md5': 'unknown',      date: 'Wed, 17 Apr 2019 10:10:30 GMT' },   StatusCode: 503,   RequestId: 'a2b3a66e-c97c-4a0b-ba6f-deae85410aa0' }
I couldn't find any example regarding this issue. Do I have to use a loop ? A time out ?
Thanks !

@roasted-toast
http://docs.developer.amazonservices.com/en_CA/dev_guide/DG_Throttling.html
Please take a visit on this. It will help you to understand the error and proper solution for that.

@bhushankumarl
We have a large a mount of orders for 2018 (over 40k). I've finally decided to call for a report instead.
Less time consuming than using the Order API (that is useful for a < 100 orders request) since ListOrder and ListOrderByNextToken share a maximum request quota of six and a restore rate of one request every minute...
Thanks for the tip anyways!

@egdavid Can I ask you what type of report you were able to request this amount of orders? I'm facing similar issue. Thanks

@lincolnaleixo I know I've used the Amazon MWS Report API to get orders: https://docs.developer.amazonservices.com/en_ES/reports/Reports_WorkingWithOrderReports.html


MWPW-128722 Ability to autodetect caas country/lang

Adds new option in the Basics panel to have country and language automatically detected based off of the  attribute.
Resolves: MWPW-128722
The pages below all have a caas collection with autolang enabled.
Test URLs:

Before: https://main--bacom--adobecom.hlx.page/jp/drafts/cpeyer/caasautolang?martech=off

After: https://main--bacom--adobecom.hlx.page/jp/drafts/cpeyer/caasautolang?milolibs=c3-caasautolang&martech=off

Before: https://main--bacom--adobecom.hlx.page/ch_fr/drafts/cpeyer/caasautolang?martech=off

After: https://main--bacom--adobecom.hlx.page/ch_fr/drafts/cpeyer/caasautolang?milolibs=c3-caasautolang&martech=off

@JasonHowellSlavin A previous dev had committed code that hadn't been linted.


"Rerun" option causes a program error.

When running from the GUI, selecting "Rerun" will display the initial output but when it comes time for the GUI to reappear, the program abends.

Fixed in version 2.6.0, via use of psutil which enables the relaunching of the app (rerun).


ERROR: Cannot install -r requirement.txt (line 35), -r requirement.txt (line 39) and protobuf==3.20.0 because these package versions have conflicting dependencies.

INFO: pip is looking at multiple versions of tensorflow to determine which version is compatible with other requirements. This could take a while.
ERROR: Cannot install -r requirement.txt (line 35), -r requirement.txt (line 39) and protobuf==3.20.0 because these package versions have conflicting dependencies.
The conflict is caused by:
The user requested protobuf==3.20.0
tensorboardx 2.6.2 depends on protobuf
tensorflow 2.7.4 depends on protobuf<3.20 and >=3.9.2
To fix this you could try to:

loosen the range of package versions you've specified
remove package versions to allow pip to attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

$ python --version
Python 3.9.20


add: truncated transform

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)

[ ] Did you read the contribution guide?
[ ] Did you update the docs? We use Numpy format for all the methods and classes.
[ ] Did you write any new necessary tests?
[ ] Did you update the CHANGELOG?

Type of Change

[ ] Examples / docs / tutorials / contributors update
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Related Issue

Closing issues

closes #1201

Codecov Report

Merging #755 (00080f9) into master (88f4766) will decrease coverage by 32.81%.
The diff coverage is 43.33%.

@@             Coverage Diff             @@
##           master     #755       +/-   ##
===========================================
- Coverage   83.75%   50.93%   -32.82%
===========================================
  Files         122      122
  Lines        6714     6744       +30
===========================================
- Hits         5623     3435     -2188
- Misses       1091     3309     +2218

Impacted Files
Coverage Δ

etna/transforms/utils.py
51.42% <43.33%> (-48.58%)
:arrow_down:

etna/commands/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

etna/commands/backtest_command.py
0.00% <0.00%> (-96.43%)
:arrow_down:

etna/commands/forecast_command.py
0.00% <0.00%> (-93.94%)
:arrow_down:

etna/commands/__main__.py
0.00% <0.00%> (-87.50%)
:arrow_down:

etna/commands/resolvers.py
0.00% <0.00%> (-80.00%)
:arrow_down:

etna/analysis/outliers/density_outliers.py
22.44% <0.00%> (-75.52%)
:arrow_down:

etna/datasets/datasets_generation.py
27.02% <0.00%> (-72.98%)
:arrow_down:

etna/transforms/timestamp/time_flags.py
27.02% <0.00%> (-72.98%)
:arrow_down:

etna/transforms/timestamp/fourier.py
28.00% <0.00%> (-72.00%)
:arrow_down:

... and 71 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Crash On Return in Yak Title

When the return key is pressed in the Yak Title/Name area, the app moves to a blank screen with a nonfunctional app bar, which contains an add button and a greyed send button. When the add button is pressed, there's a brief pause, and then the app closes/crashes.

Fixed in f068542. The fix will be included in an upcoming release, which will be out...soon.


[Enhancement] Allow to programmatically enable/disable the Tags dropdown element

It would be nice to programmatically enable/disable the Tags dropdown element (simulating the appeal of a native disabled select element) to deal with situations where tag selection have a dependency on information required to be entered/selected on another element.

Fair point. Added in demo and available in https://github.com/lekoala/bootstrap5-tags/releases/tag/1.4.7


Update Travis badge and add EmberObserver badge

Tests?

How do I test this?


Controlling serverMaxWindowBits in uWebSocket

Hi, I'm using Websocket-sharp in my Unity game as client that is connecting to a uWebsocket backend server.
We want to enable Per-message Compression for this connection but the problem is that when I enable compression in client the server refuses the request from the client.
We also reproduced this issue with ws library in node js as backend with its provided example configuration, but the problem persists. When we set serverMaxWindowBits in ws library to default negotiated value, the client can connect to the server properly.
Now my question is, Is there any way that we can control this serverMaxWindowBits in uWebsockets?
uWebSocket 16.4
CentOS 7
Node js 10.17
Thanks.

This should ultimately be paired with the new compressors; specifying DEDICATED_COMPRESSOR_2KB should negotiate a smaller sliding window than 15 (in this case it should be 8).

Okay this is kind of fixed now, for most cases. But the server still decides with highest authority


Tech :: Add Coverage

Need express in issue: #73

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@552a9d2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #94   +/-   ##
=========================================
  Coverage          ?   77.23%
=========================================
  Files             ?       10
  Lines             ?      246
  Branches          ?       23
=========================================
  Hits              ?      190
  Misses            ?       49
  Partials          ?        7

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 552a9d2...c23394c. Read the comment docs.


GP: Add DDL generation for Greenplum's concept of external tables

Greenplum has a concept of external tables that store data outside of the primary Greenplum store.
DBeaver Greenplum plugin had no concept of DDL generation specifically for external tables
This commit adds basic external table DDL generation

Example external table DDL:
CREATE EXTERNAL TABLE ext_customer
   (id int, name text, sponsor text)
   LOCATION ( 'gpfdist://filehost:8081/*.csv' )
   FORMAT 'CSV' ( DELIMITER ',' );

Looks good 👍
Please add header with Apache license in new files. Add yourself and/or Pivotal in author area.
Notes:

We usually do not use static imports
As for me wide usage of String.format in combination is with StringBuilder is a bit odd and makes code less readable. I'd suggest to use StringBuilder and/or simple strings concatenations. String.format doesn't make much sense unless you are using some specific formatting options.

Thanks.
Commit added to address issues in the Greenplum plugin.

Awesome :)


Fixed #7060 -- Added a note about race conditions to the tutorial

This is a proposal for a note in the tutorial. Feel free to suggest any changes.

Please follow the commit message guidelines as noted in our patch review checklist and wrap lines at 79 characters.

I agree with your suggestion, updated it accordingly and fixed the commit message.

merged in 3e9b5bfd9c3e16c968278f7d050f52739690eb29, thanks!


Doesn't compile with 1.0.0 beta.

src/lib/protobuf.rs:2:1: 2:33 error: unstable feature
src/lib/protobuf.rs:2 #![feature(box_patterns, alloc)]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
note: this feature may not be used in the beta release channel
src/lib/protobuf.rs:5:1: 5:25 error: unstable feature
src/lib/protobuf.rs:5 #![feature(collections)]
^~~~~~~~~~~~~~~~~~~~~~~~
note: this feature may not be used in the beta release channel
src/lib/protobuf.rs:6:1: 6:21 error: unstable feature
src/lib/protobuf.rs:6 #![feature(convert)]
^~~~~~~~~~~~~~~~~~~~
note: this feature may not be used in the beta release channel
src/lib/protobuf.rs:7:1: 7:18 error: unstable feature
src/lib/protobuf.rs:7 #![feature(core)]
^~~~~~~~~~~~~~~~~
note: this feature may not be used in the beta release channel
src/lib/protobuf.rs:8:1: 8:22 error: unstable feature
src/lib/protobuf.rs:8 #![feature(str_char)]
^~~~~~~~~~~~~~~~~~~~~
note: this feature may not be used in the beta release channel
error: aborting due to 5 previous errors
Could not compile protobuf.

+1

Tried to fix to work with beta, and couldn't. But at least it works with nightly, thanks to #95.

+1


Using the library without winit::event_loop

Hi, I'm trying to use this library in MacOS Monterey 12.3.1.
Running examples/print_events doesn't work (no media event printed even when I press one, no metadata shown up in the media control). Running example/example.rs works perfectly but I'm not sure I understand the requirement of winit library here. winit and winit::event_loop seem to be the only difference between examples/print_events.rs and examples/example.rs
Does souvlaki work without winit, as I see that winit is only included as the devdependencies? Am I missing something here?

Media control is only available for window processes on MacOS.

Maybe it has to do with the thread being blocked. Can you try opening the media controls in another thread and/or reducing the sleep time?

Still not work AFAICT. I guess the reason is because MacOS needs an open window to listen to the events, as other mentioned.
Will close this issue for now. Thanks.

I'd appreciate if someone with access to MacOS could pin down the behaviour of media controls and window processes, so it can be stated in the crate's documentation and prevent misunderstandings. I'm going to put a warning in the docs for now.

From my experience with a NodeJS library I used with Electron (electron-media-service), it is likely that an application event-loop is required (i.e. Winit's event_loop), but a window itself is not strictly required (as it is common for Mac OS applications to continue running despite no open windows, the event loop would continue anyway)
I think the specific requirement would be a running AppDelegate, regardless of the existence of any windows.


[Feature Request]: Please provide authentication for client/server mode

Describe the problem
My two servers are located in different places. The Chroma client/server communication occurs through the public network. Although it is possible to encrypt the communication content via HTTPS, other people may be able to scan and connect to my server without any verification.
Describe the proposed solution
When setting up a Docker server, I can provide a key through environment variables. When a client remotely connects, it needs to verify this key.
Alternatives considered
No response
Importance
nice to have
Additional Information
No response

We now have this in 0.4.8 - landing in JS today as well.


Looks cannot update from sourceforge.net via aria2

I noticed recent update failed resources are all stored at Sourceforge and I cannot download it properly. There is anyone else as same as me?

PS C:\Users\fst> scoop info aria2
Name: aria2
Description: Lightweight multi-protocol & multi-source command-line download utility
Version: 1.35.0-1
Website: https://aria2.github.io/
License: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later.html)
Manifest:
D:\Scoop\buckets\main\bucket\aria2.json
Installed:
D:\Scoop\apps\aria2\1.35.0-1
Binaries:
aria2c.exe

PS C:\Users\fst> scoop update qbittorrent-portable
qbittorrent-portable: <IP_ADDRESS> -> 4.3.1
WARN  Scoop uses 'aria2c' for multi-connection downloads.
WARN  Should it cause issues, run 'scoop config aria2-enabled false' to disable it.
Updating one outdated app:
Updating 'qbittorrent-portable' (<IP_ADDRESS> -> 4.3.1)
Downloading new version
Starting download with aria2 ...
Download: 12/17 02:28:09 [ERROR] CUID#8 - Download aborted. URI=https://downloads.sourceforge.net/project/qbittorrent/qbDownload: Exception: [AbstractCommand.cc:351] errorCode=1 URI=https://downloads.sourceforge.net/project/qbittorrent/qbitDownload:   -> [SocketCore.cc:1021] errorCode=1 SSL/TLS handshake failure: Error: 鐢变簬鍚婇攢鏈嶅姟鍣ㄥ凡鑴辨満锛屽悐閿Download: Download Results:
Download: gid   |stat|avg speed  |path/URI
Download: ======+====+===========+=======================================================
Download: e2bd00|ERR |       0B/s|D:/Scoop/cache/qbittorrent-portable#4.3.1#https_downloads.sourceforge.net_project_qbittorrent_qbittorrent-win32_qbittorrent-4.3.1_qbittorrent_4.3.1_x64_setup.exe_dl.7z
Download: Status Legend:
Download: (ERR):error occurred.
Download: aria2 will resume download if the transfer is restarted.
Download: If there are any errors, then see the log file. See '-l' option in help/man page for details.

ERROR Download failed! (Error 1) An unknown error occurred
ERROR https://downloads.sourceforge.net/project/qbittorrent/qbittorrent-win32/qbittorrent-4.3.1/qbittorrent_4.3.1_x64_setup.exe#/dl.7z
    dir=D:\Scoop\cache
    out=qbittorrent-portable#4.3.1#https_downloads.sourceforge.net_project_qbittorrent_qbittorrent-win32_qbittorrent-4.3.1_qbittorrent_4.3.1_x64_setup.exe_dl.7z

ERROR & 'D:\Scoop\apps\aria2\current\aria2c.exe' --input-file='D:\Scoop\cache\qbittorrent-portable.txt' --user-agent='Scoop/1.0 (+http://scoop.sh/) PowerShell/7.1 (Windows NT 10.0; Win64; x64; Core)' --allow-overwrite=true --auto-file-renaming=false --retry-wait=2 --split=5 --max-connection-per-server=5 --min-split-size=5M --console-log-level=warn --enable-color=false --no-conf=true --follow-metalink=true --metalink-preferred-protocol=https --min-tls-version=TLSv1.2 --stop-with-process=22452 --continue --summary-interval 0 --all-proxy='<IP_ADDRESS>:7890'

Please try again or create a new issue by using the following link and paste your console output:
https://github.com/lukesampson/scoop-extras/issues/new?title=qbittorrent-portable%404.3.1%3a+download+via+aria2+failed
PS C:\Users\fst>

PS C:\Users\fst> scoop update winscp
winscp: 5.17.8 -> 5.17.9
WARN  Scoop uses 'aria2c' for multi-connection downloads.
WARN  Should it cause issues, run 'scoop config aria2-enabled false' to disable it.
Updating one outdated app:
Updating 'winscp' (5.17.8 -> 5.17.9)
Downloading new version
Starting download with aria2 ...
Download: 12/17 02:28:29 [ERROR] CUID#8 - Download aborted. URI=https://downloads.sourceforge.net/project/winscp/WinSCP/Download: Exception: [AbstractCommand.cc:351] errorCode=1 URI=https://downloads.sourceforge.net/project/winscp/WinSCP/5.Download:   -> [SocketCore.cc:1021] errorCode=1 SSL/TLS handshake failure: Error: 鐢变簬鍚婇攢鏈嶅姟鍣ㄥ凡鑴辨満锛屽悐閿Download: Download Results:
Download: gid   |stat|avg speed  |path/URI
Download: ======+====+===========+=======================================================
Download: d1c227|ERR |       0B/s|D:/Scoop/cache/winscp#5.17.9#https_downloads.sourceforge.net_project_winscp_WinSCP_5.17.9_WinSCP-5.17.9-Portable.zip
Download: Status Legend:
Download: (ERR):error occurred.
Download: aria2 will resume download if the transfer is restarted.
Download: If there are any errors, then see the log file. See '-l' option in help/man page for details.

ERROR Download failed! (Error 1) An unknown error occurred
ERROR https://downloads.sourceforge.net/project/winscp/WinSCP/5.17.9/WinSCP-5.17.9-Portable.zip
    dir=D:\Scoop\cache
    out=winscp#5.17.9#https_downloads.sourceforge.net_project_winscp_WinSCP_5.17.9_WinSCP-5.17.9-Portable.zip

ERROR & 'D:\Scoop\apps\aria2\current\aria2c.exe' --input-file='D:\Scoop\cache\winscp.txt' --user-agent='Scoop/1.0 (+http://scoop.sh/) PowerShell/7.1 (Windows NT 10.0; Win64; x64; Core)' --allow-overwrite=true --auto-file-renaming=false --retry-wait=2 --split=5 --max-connection-per-server=5 --min-split-size=5M --console-log-level=warn --enable-color=false --no-conf=true --follow-metalink=true --metalink-preferred-protocol=https --min-tls-version=TLSv1.2 --stop-with-process=22452 --continue --summary-interval 0 --all-proxy='<IP_ADDRESS>:7890'

Please try again or create a new issue by using the following link and paste your console output:
https://github.com/lukesampson/scoop-extras/issues/new?title=winscp%405.17.9%3a+download+via+aria2+failed
PS C:\Users\fst>

[SocketCore.cc:1021] errorCode=1 SSL/TLS handshake failure: Error: 由于吊销服务器已脱机，吊销功能无法检查吊销。

It means OCSP check failed so that aria2 cannot establish TLS connection. It's aria2 tls lib's determination, refuse continue without OCSP result.


New cross-compile failure with x86_64-w64-mingw32-g++

Zimg normally cross-compiles and works flawlessly when compiled under Linux (gcc-6.4.1) and running on Windows 10. However, as of this commit:
https://github.com/sekrit-twc/zimg/commit/b1eec67a1d31cddf2900c1d9d141025c4863d606
...several compilation errors appear in src/zimg/resize/resize_impl_avx512.cpp:
In file included from /mnt/e/Users/john/Documents/MultimediaTools-mingw-w64/sandbox/mingw-w64-x86_64/lib/gcc/x86_64-w64-mingw32/6.4.1/include/immintrin.h:29:0,
                 from src/zimg/resize/resize_impl_avx512.cpp:11:
src/zimg/resize/resize_impl_avx512.cpp: In instantiation of '__m512i zimg::resize::{anonymous}::resize_line16_h_u16_avx512_xiter(unsigned int, const unsigned int*, const int16_t*, unsigned int, unsigned int, const uint16_t*, unsigned int, uint16_t) [with bool DoLoop = false; unsigned int Tail = 0u; __m512i = __vector(8) long long int; int16_t = short int; uint16_t = short unsigned int]':
src/zimg/resize/resize_impl_avx512.cpp:728:20:   required from 'void zimg::resize::{anonymous}::resize_line16_h_u16_avx512(const unsigned int*, const int16_t*, unsigned int, unsigned int, const uint16_t*, uint16_t* const*, unsigned int, unsigned int, unsigned int, uint16_t) [with bool DoLoop = false; unsigned int Tail = 0u; int16_t = short int; uint16_t = short unsigned int]'
src/zimg/resize/resize_impl_avx512.cpp:828:53:   required from here
src/zimg/resize/resize_impl_avx512.cpp:589:36: error: invalid conversion from 'int' to '_MM_PERM_ENUM' [-fpermissive]
   c = _mm512_shuffle_epi32(coeffs, _MM_SHUFFLE(0, 0, 0, 0));
                                    ^
In file included from /mnt/e/Users/john/Documents/MultimediaTools-mingw-w64/sandbox/mingw-w64-x86_64/lib/gcc/x86_64-w64-mingw32/6.4.1/include/immintrin.h:45:0,
                 from src/zimg/resize/resize_impl_avx512.cpp:11:
/mnt/e/Users/john/Documents/MultimediaTools-mingw-w64/sandbox/mingw-w64-x86_64/lib/gcc/x86_64-w64-mingw32/6.4.1/include/avx512fintrin.h:3843:1: note:   initializing argument 2 of '__m512i _mm512_shuffle_epi32(__m512i, _MM_PERM_ENUM)'
 _mm512_shuffle_epi32 (__m512i __A, _MM_PERM_ENUM __mask)
 ^~~~~~~~~~~~~~~~~~~~
In file included from /mnt/e/Users/john/Documents/MultimediaTools-mingw-w64/sandbox/mingw-w64-x86_64/lib/gcc/x86_64-w64-mingw32/6.4.1/include/immintrin.h:29:0,
                 from src/zimg/resize/resize_impl_avx512.cpp:11:
src/zimg/resize/resize_impl_avx512.cpp:603:36: error: invalid conversion from 'int' to '_MM_PERM_ENUM' [-fpermissive]
   c = _mm512_shuffle_epi32(coeffs, _MM_SHUFFLE(1, 1, 1, 1));
                                    ^
In file included from /mnt/e/Users/john/Documents/MultimediaTools-mingw-w64/sandbox/mingw-w64-x86_64/lib/gcc/x86_64-w64-mingw32/6.4.1/include/immintrin.h:45:0,
                 from src/zimg/resize/resize_impl_avx512.cpp:11:
/mnt/e/Users/john/Documents/MultimediaTools-mingw-w64/sandbox/mingw-w64-x86_64/lib/gcc/x86_64-w64-mingw32/6.4.1/include/avx512fintrin.h:3843:1: note:   initializing argument 2 of '__m512i _mm512_shuffle_epi32(__m512i, _MM_PERM_ENUM)'
 _mm512_shuffle_epi32 (__m512i __A, _MM_PERM_ENUM __mask)
 ^~~~~~~~~~~~~~~~~~~~
In file included from /mnt/e/Users/john/Documents/MultimediaTools-mingw-w64/sandbox/mingw-w64-x86_64/lib/gcc/x86_64-w64-mingw32/6.4.1/include/immintrin.h:29:0,
                 from src/zimg/resize/resize_impl_avx512.cpp:11:
src/zimg/resize/resize_impl_avx512.cpp:617:36: error: invalid conversion from 'int' to '_MM_PERM_ENUM' [-fpermissive]
   c = _mm512_shuffle_epi32(coeffs, _MM_SHUFFLE(2, 2, 2, 2));

...and many similar.
When I try -fpermissive, this is output by the assembler:
/tmp/cc3qqecI.s: Assembler messages:
/tmp/cc3qqecI.s:138: Error: invalid register for .seh_savexmm
/tmp/cc3qqecI.s:140: Error: invalid register for .seh_savexmm
/tmp/cc3qqecI.s:142: Error: invalid register for .seh_savexmm
/tmp/cc3qqecI.s:144: Error: invalid register for .seh_savexmm

...etc.

Thank you very much for looking into this. I will try again at once.

Indeed, _MM_PERM_ENUM satisfies the compiler; but the "invalid register for .seh_savexmm" still occurs. It's Binutils-2.28 as x86_64-w64-mingw32-as that I'm using. Will check a few other things.

Maybe related: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65782

This could be very helpful. Thanks!

I added a workaround in configure to detect this MINGW bug.

Compiles cleanly here. Thanks for the fix!


Pass options through to GRPC.Stub.connect

Adds an optional keyword list argument to Authzed.Api.V1.Client.new so that additional GRPC.Stub.connect options can be configured.

Addresses: https://github.com/goodhamgupta/authzed_ex/issues/24

Thanks for your contribution @pwcsquared! Could you please bump up the version in the mix.exs file? I can then merge these changes 😄


Configure more VOs

Summary
Add configuration for several VOs:

Add mteam.data.kit.edu
Add EOServices-vo.indra.es
Add umsa.cerit-sc.cz
Add saps-vo.i3m.upv.es
Add cryoem.instruct-eric.eu
Add ops VO to Kharkov site

Related issue :

not sure why the tf is still being triggered. I will remove and re-create

not sure why the tf is still being triggered. I will remove and re-create


Issue with extracting var section

Thanks so much for providing this solution. I was able to run npm and create the metadata.json file, however as I want to pass on the pyLDAvis object to the pyldavis_dash.pyLDAvis output, I'm having some issues. How exactly should I extract the var section (as per your instructions)?
This is my code to create the pyLDAvis object and save it as html:
panel = pyLDAvis.gensim.prepare(lda_model, bow_corpus, id2word)
pyLDAvis.display(panel)
pyLDAvis_file = os.path.join(data_path,'plots/pyLDAvis.html')
pyldavis_figure = pyLDAvis.save_html(panel, pyLDAvis_file)
And then trying to pass it on to the dash output:
return [             html.Div(                 className='eight columns',                 children=[                     pyldavis_dash.pyLDAvis(                         id='lda_graph',                         data=pyldavis_figure                     )                 ])         ]
The html file was created successfully, and I can load it outside of my dash app in Chrome. Any help would be greatly appreciated.

Forgot to add: If i run the above and simply replace pyldavis_figure with your example 'data', it successfully displays it in the Dash app.

Hi @phrieder, please could you share how exactly you managed to run npm and create the metadata.json file? I am having trouble with this.

Hi @phrieder, please could you share how exactly you managed to run npm and create the metadata.json file? I am having trouble with this.

Just clone the repo and change directory to the repo and run npm install. It should download and install all dependencies for you.


[Required] rarely detects errors in top-level nodes

[Required] on top-level nodes works with string and few other types. This goes beyond ValueTypes, the usual [Required] / [BindRequired] discussion point. (That part's inherent in [Required]'s semantics.) [Required] also does not catch the expected errors on pretty much any collection or complex type if it's a top-level node.
Consider the following action and a <form> containing no <input/> fields:
[HttpPost("/Ints")]
public IActionResult Ints([Required] int[] ints)
{
    // ...
}

In this scenario, ModelState.IsValid will always be true.
Because ints is an action parameter, model binding considers it a top-level node and provides a non-null default value. In this case, the default value is an empty array. Importantly, this default is chosen before validation. So [Required] is passed the empty array and sees nothing amiss.
One possibility would be to move our IsTopLevelNode special cases out of model binding and into ParameterBinder and other model binding callers. This would allow providing default values after validation.

Hi @dougbu. Will this behavior repro for not only int[] but any other arrays/objects too?
In other words, will this also always result in ModelState.IsValid to be true:
[HttpPost("/Ints")]
public IActionResult Ints([Required] Customer[] ints)
{
    // ...
}

@mkArtakMSFT please see the first paragraph of the description above. int[] is just an example.

Closing this as this seems to be by-design and changing behavior will be a breaking change, which is undesirable.

Reactivating after a quick confirmation chat w/ @rynowak.
There's no breaking change if we fixed this bug. We haven't previously supported [Validation] attributes (including [Required]) on a top-level node.
Further, the current behaviour means [Required] behaves differently for top-level nodes than for a property further down the model tree. For example,

When the submitted form has no fields for a [Required] int[] (pick your favourite non-string, non-value type) parameter, no validation errors are detected
When the submitted form has no fields for a [Required] int[] property, the expected validation errors are added to ModelState

@mkArtakMSFT I'm clearing the milestone and assignment to make it clear this should be triaged again.

Clearing the rest of the labels and adding super-triage. Believe the idea is to add an MVC analyzer that detects [Required] on top-level, non-string nodes (parameters and controller or page model properties), perhaps on all parameters and properties that have collection or complex types.
(May make sense to edit the title if I understand the plan.)
Rationale
[Required] on a complex type can only mean "at least one of the properties in this POCO object must have a non-null default value or be model bound".  That's not particularly meaningful compared to specifying [Required] on the required properties or implementing IValidatableObject to ensure the correct property combinations are non-null.
[Required] on a collection can only mean "this collection must have a non-null default value or have at least one model bound entry". Since the default value may be empty, top-level collections are always non-null, and [MinLength(1)] always succeeds when a property is null, this gets pretty confusing pretty quick. Analyzer should recommend using [MinLength(1)] on top-level nodes ([Required] is irrelevant here) and [MinLength(1), Required] elsewhere.
Separately
Because [Required] is meant to be used together with many ValidationAttributes may also be worthwhile recommending adding [Required] where a nullable property is missing it but another ValidationAttribute exists.
/cc @rynowak @Eilon @mkArtakMSFT

Closing in favor of https://github.com/aspnet/Mvc/issues/7415, which will track the only outcome from this issue.

In case someone refers to this issue later: Removed the earlier Separately section because "optional but, if it exists, …" validations are just fine.


[ADD] Clean parcel configuration.

Added configuration for clean.

@geoperez this PR si ready for your review.


sheet_to_json (break after empty row?)

Hi there,
I want to stop reading the file after finding an empty row because sometimes my users save the Excel including all rows and columns, and the process cannot be completed because it takes too long.
For example, the excel is 6 rows but since they saved the file sheet_to_json wrongly it tries to read all the rows and columns of the excel.
I noticed that the problem is related to the property !refs into worksheet object. So I created a hack to fix that
function fixWorksheetReference(worksheet) {
    // get all cells ranges
    const cellRanges = Object.keys(worksheet).filter(x => x.match(/[A-Z]+[0-9]+/g));

    // get the last one cell range
    const maxCellRange = cellRanges[cellRanges.length - 1];

    // update the reference because before of that, it could be A1:AMJ1048576
    worksheet['!ref'] = `A1:${maxCellRange}`;
}

👉 My hack re-calculate the range reference of the property !ref using the cells mapped into the worksheet property.
This an example of its usage.
const worksheet = workbook.Sheets[workbook.SheetNames[sheetNumber]];

fixWorksheetReference(worksheet);

return xlsx.utils.sheet_to_json(worksheet, { raw: true, defval: null });

What do you think about that?

Have the same problem. The fix works.

update_sheet_range does something similar.
The underlying problem is that the !ref key uses the self-reported range, and some writers cheat by including more rows than exist in the file.  The parsers could (and probably should) rewrite the ranges.


Implement loop and dominator tree analyses

In order to convert Miden IR to Miden Assembly, we require two analysis passes over the IR which give us the necessary information to transform low-level control flow (primitive branches/jumps) into the higher-level control flow (if/while/repeat) used by Miden Assembly.

[ ] The dominator tree analysis is the foundation on which we build our control flow analyses. This analysis is critical for determining what blocks are involved in a given control flow path, and how control enters/exits that path. It is a low level analysis used by others we will build.
[ ] The loop analysis builds on the dominator tree to recognize loops and their various forms. This analysis will be used as the basis for our transformation of loop-like control flow to Miden Assembly.

There are other analyses/passes related to these that we will implement or may want to implement, but these two are critical for other work in the pipeline.

@bitwalker is this closed by https://github.com/0xPolygonMiden/miden-ir/commit/45e1b2265fb4916d7126d63ac0e581b2679e2ddd?

@grjte Yes, thanks for the reminder!


Add more examples

Hi, this PR adds more examples:

zk_cluster - start test cluster
zk_write - create or overwrite value
zk_watch - watch and re-read value or its children
zk_tree - traverse tree

Coverage increased (+0.2%) to 80.772% when pulling 78fb4e1bce004a0547021e8924fca19f1ce459cc on gavv:master into e64db453f3512cade908163702045e0f31137843 on samuel:master.

Coverage decreased (-0.4%) to 80.152% when pulling 78fb4e1bce004a0547021e8924fca19f1ce459cc on gavv:master into e64db453f3512cade908163702045e0f31137843 on samuel:master.

Coverage decreased (-0.4%) to 80.152% when pulling 56f0bf84f3d2207c63edd74794c3db8bceaebb1b on gavv:master into e64db453f3512cade908163702045e0f31137843 on samuel:master.

Hm, it seems that go test -race fails randomly on travis build.

Coverage decreased (-0.4%) to 80.152% when pulling 528d14053ce60f65e891dbb585dcd46925996b7f on gavv:master into e64db453f3512cade908163702045e0f31137843 on samuel:master.


Backport "Properly dealias tuple types when specializing" to LTS

Backports #18724 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-18723.
Reference


Upgrade bk version to resolve the BouncyCatle issue

Fixes #10937
Motivation
The previours bk version was introduce an incompatible issue
with BouncyCastle FIPS. BookKeeper already fix this and we
should upgrade the version to resolve the issue.
For more information: #10937
Modifications
Upgrade bk to 4.14.2

(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

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

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
For contributor
For this PR, do we need to update docs?

If yes, please update docs or create a follow-up issue if you need help.

If no, please explain why.

For committer
For this PR, do we need to update docs?

If yes,

if you update docs in this PR, label this PR with the doc label.

if you plan to update docs later, label this PR with the doc-required label.

if you need help on updating docs, create a follow-up issue with the doc-required label.

If no, label this PR with the no-need-doc label and explain why.

@zymap license check failed, PTAL


About pth

hi, where can i find 'control_sd15_landmarks.pth'

you can find it here: https://huggingface.co/georgefen/Face-Landmark-ControlNet/blob/main/models/control_sd15_landmarks.pth


In the background, after a while elapsed time proceeds slowly

In the background, after a while elapsed time proceeds slowly. For example, we set the time to 15 min. After a 10 min. we need to see 5 min, but I see 11 min. So there is a timing or state problem. I also look the app permissions, but it is not for that situation, because I tested lots of android devices.
I calculate the remaining time with this code:
final notificationDuration = (timerController!.duration! - state.spentFocusedTime).inSeconds;
Actually, it works properly, but after a while, it is not.

The problem is simple_timer widget. So there is no solution except rewriting codes. Therefore, I close this issue, and I will open an enhancement issue for rewriting the timer.


https://developer.mozilla.org/en-US/docs/Web/API/Document/querySelector

Request type

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

Details

In regards to page:
https://developer.mozilla.org/en-US/docs/Web/API/Document/querySelector
Near the bottom under specification there are listed the to level of api 1 & 2 as obsolete with no
link to the current apis 3 & 4
https://www.w3.org/TR/2018/REC-selectors-3-20181106/
&
https://www.w3.org/TR/selectors-4/
It is confusing as a novice to read that the supplied documentation is obsolete.  Can there be updated the Specification table to read with the addition of the current api?
Thank for you time, William Warren

Hi @somestuff,
Thanks for reporting this; yes, we have quite a lot of obsolete spec links, and are soon hoping to replace the manual tables with a more automated solution.
In this case, I have just replaced the obsolete spec links with a link to querySelector()'s definition in the WHATWG DOM Living Spec. I think that's the most relevant one now, especially since the W3C is stopping publishing snapshots of the relevant WHATWG stuff in this area.


Support force tty of nodeConsole

Feature request

What is the expected behavior?
Provide some environment variable or someway to support force set tty of nodeConsole.
What is motivation or use case for adding/changing the behavior?
To use with Jenkins.
I make builds using Jenkins, and it supports colors by AnsiColor plugin.
Since Jenkins shows 'Console Output' as non-TTY environment, I need to set force color options many place.
So I want something like https://github.com/webpack/webpack-cli supporting force color via .npmrc color=always.
How should this be implemented in your opinion?
https://github.com/webpack/webpack/blob/263c950952ca6aad298f2b7651c06e072fbf3e62/lib/node/nodeConsole.js#L11
Change above to below:
const tty = (process.stderr.isTTY && process.env.TERM !== "dumb") || process.env.TERM === "tty";

Are you willing to work on this yourself?
yes

const tty = (process.stderr.isTTY && process.env.TERM !== "dumb") || process.env.TERM === "tty";

it is breaking change and potential break terminal for other developers

I see, then how about using a new environment variable, something like process.env.WEBPACK_FORCE_TTY ?

I think we need logic here:
const tty =
  !("NO_TTY" in process.env) &&
  ("FORCE_TTY" in process.env ||
    (process.stderr != null &&
      process.stderr.isTTY &&
      process.env.TERM &&
      process.env.TERM !== "dumb"))

So you can force enable and disable it

Yeah but not process.env.TERM && since the property may not exists.

So could I make a PR?

Yes, let's send a PR

Cool, let me study awhile for write js test code.

Is using env vars the best idea for that?
What do you think about using config options for that infrastructureLogging.colors: false/true and infrastructureLogging.appendOnly: false/true?

Ooh that's cool also.

BTW if we use something like infrastructureLogging.colors we should also provide force value to address this issue.

@KENNYSOFT Byt with this option, you can do it in webpack.config.js

@alexander-akait Do you mean it's already implemented? I cannot find :(

@KENNYSOFT No, it is not implemented

we should also provide force value to address this issue.

Why you need force here?

Ah, may not. I've seen some programs ignore color flags even manually set when using in non-TTY environment. So if I implement without ignoring when TTY then it's not needed.

We can provide option to disable/enable colors for infrastructureLogging and you can implement any logic on own side

Yup it's great. So someone will want:
infrastructureLogging: {
  colors: process.stderr.isTTY && process.env.TERM !== "dumb",
  ...
}

just like now, or I will want:
infrastructureLogging: {
  colors: true,
  ...
}

to force set colors everywhere!


fixing the way we install the gateway in standalone mode

@balda-rdx Just so we can track it later. Can you change to the title of this PR to: Add option to install gateway in standalone mode? The feature never existed before.


Filter/Unfilter and breakpoints not functioning properly

Here is a full demo of the issue: https://jsfiddle.net/jasonsan3/ocfL0q2j/2/
Slick works well before the breakpoint. Filtering and Unfiltering are triggered predictably. At breakpoint, the filtering/unfiltering stops working correctly. It seems that there is a failure to adjust the 3d transform settings, but I can't quite figure out what is happening.
Also, in the jsfiddle, I have used a mobileFirst setting. However, if I take that out, the issue persists, just in the opposite direction. That is, if the break is met from moving large to small, then it is the same error. It seems that something is set when the slick is first initialized that fails after a breakpoint.
Thank you.

I have a similar issue. Demo: http://codepen.io/CAYdenberg/pen/ZGqbzw
It looks like this was referenced in #366 and a partial solution was submitted by @smk1 but it seems to have forgotten.

Hi guys, check this https://github.com/kenwheeler/slick/pull/1569 this fixed for me.

Thanks @ramon-villain this fix worked like a charm. Startling to see something work on my first try. Much appreciated! Just in case someone googles this later...
Edit slick.js (version 1.5.6) lines 627 and 634. Just delete those lines, or comment them out. The lines are as follows.
Line 627:
_.refresh(initial);

and Line 634:
_.unslick(targetBreakpoint);

Thanks again to @ramon-villain in reference to fix #1569 as described above.

I think the issue is not closed.
lopping out code for your specific use-case is not the same as fixing an issue :P
there's a million options for slick. I'll leave this open until it has been investigated properly.

Great, thank you @simeydotme.

Thank you @ramon-villain! In my hands, this PR fixes #366 in my project and in the codepen: http://codepen.io/CAYdenberg/pen/NqOpEG.
The ONLY issue I see is if: you select a filter, resize outside of the breakpoints (ie, so that responsive mode is not being used), and then select a different filter. In that case, it doesn't behave properly.
Still, this is a relatively minor use-case, so I'll be using the updated code on my project.

+1 on this issue - I'm having the same problem.
The suggested fix does seem to solve the problem - but I also encounter the issues reported by @CAYdenberg .

+1

I have the same issue.

Afaics the issue originates in the not-so-well-written refresh function.
Slick.prototype.refresh = function( initializing ) expects initializing to be boolean and does not validate that assumption.
If you like to hack that particular one away, change https://github.com/kenwheeler/slick/blob/master/slick/slick.js#L1540
if( !initializing ) { to
if (typeof initializing !== "boolean" || !initializing ) {

Hey I know this issue was created a while ago, but if you’re still interested can you recreate in a fiddle with the master? Thanks and apologies for the dormancy.
Closing pending updated fiddle

The same issue.
JSFidle: http://jsfiddle.net/kq17qknt/
There we can see that slick permanently removes some items from dom and ufliter method not working properly.


HTTP error 403 since 5.7.0b1

Since version 5.7.0b1, the integration can no longer register with the API. I now always get an HTTP Error 403 in the log. Reinstalling the integration and registering again via the Chrome plugin does not work either. I'm attaching a screenshot.

I also got an API error. For me it helped to change the name of the user agent in the configuration dialog.

Where did you change what name? Thanks for your help.

similar problem for me. using  chrome extension ends up with error "The connection to the Bosch Indego API failed". it started after updating to v 5.7.0b1

Where did you change what name? Thanks for your help.

You can change the User-Agent in Devices & services --> Bosch Indego Mower Integration --> Configure (right side next to your mower)
For me the default User-Agent didn`t work too so i just changed it to "The name of my mower Indego"


Remove unused variable

@meziantou thank you for opening all these PRs. I just wanted to confirm that you're building the source and testing these changes?

Yes, I build the code and start the app. Maybe I do not test all the paths as I do not know the app correctly yet. I hope you'll add automated tests, so we can avoid regressions when changing the code.

Yes, I build the code and start the app. Maybe I do not test all the paths as I do not know the app correctly yet. I hope you'll add automated tests, so we can avoid regressions when changing the code.

@meziantou we'd like to add automated tests at some point, but we've been focusing primarily on general stability and optimizations like these.


Add simple gh-pages template

I was looking into a really simple Jekyll template to pull together markdown repositories like this. Here's a first pass at it. Has an index page that lists the markdown pages and shows them with a link on the title to edit on Github.
Thought it may be worth trying here.
See it in action here: http://dhcole.com/API-Program/
If you like this idea, I'd suggest merging this into gh-pages and ditching the master branch.

Also, added yaml headers to the markdown files with the following shell script:
for f in *.md; do
  echo -e "---\n---\n" | cat - $f > temp && mv temp $f;
done

Thanks for spurring this.  I went with what seems to be the 18F microsite template - http://18f.github.io/api-program/.

:+1:


[GAL-2340] Add "Worldwide" pill to "Latest" feed

Changes

Added worlwide pill in latest page

Demo
https://user-images.githubusercontent.com/4480258/217786401-37f1950a-cd16-4f03-bf5f-947088ba2983.mp4

looks great!!


to_camel improvement (fix?) to properly convert field names with a single trailing underscore

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
def to_camel_corrects_trailing_underscore(field_name: str) -> str:
    """Great for working around python reserved words and (VSCode + Pyright)'s dataclass alias= understanding"""
    return to_camel(field_name[:-1] if field_name[-1] == "_" else field_name)

Fixes field names like: "from_" -> "from".  Currently it does: "from_" -> "from_"
Perhaps the matching should be improved with regex to something like r'[A-Za-z0-9]_$' or r'[a-z]_$'?
This would cause it to match only when the field name ends in a single underscore.
I think I prefer the 2nd pattern as it works fine for all the reserved words it is likely to be used with.
I had to use this alias generator recently because a model that is understood properly in PyCharm w/ Pydantic Plugin is incorrectly understood by VScode. Without this special alias generator VScode couldn't be made happy when trying to populate by field name.
# From (Original with problem in VScode)
class Relation(BaseModel, alias_generator=to_camel, populate_by_name=True, frozen=True):
    from_: EntityRef = Field(..., alias="from")
    to_: EntityRef = Field(..., alias="to")
    type_: str = Field(..., alias="type")
    type_group: RelationTypeGroup

# To (Both VScode and PyCharm are happy)

class Relation(BaseModel, alias_generator=modified_to_camel, populate_by_name=True, frozen=True):
    from_: EntityRef
    to_: EntityRef
    type_: str
    type_group: RelationTypeGroup

### Affected Components

- [ ] [Compatibility between releases](https://docs.pydantic.dev/changelog/)
- [ ] [Data validation/parsing](https://docs.pydantic.dev/concepts/models/#basic-model-usage)
- [ ] [Data serialization](https://docs.pydantic.dev/concepts/serialization/) - `.model_dump()` and `.model_dump_json()`
- [ ] [JSON Schema](https://docs.pydantic.dev/concepts/json_schema/)
- [ ] [Dataclasses](https://docs.pydantic.dev/concepts/dataclasses/)
- [ ] [Model Config](https://docs.pydantic.dev/concepts/config/)
- [ ] [Field Types](https://docs.pydantic.dev/api/types/) - adding or changing a particular data type
- [ ] [Function validation decorator](https://docs.pydantic.dev/concepts/validation_decorator/)
- [ ] [Generic Models](https://docs.pydantic.dev/concepts/models/#generic-models)
- [ ] [Other Model behaviour](https://docs.pydantic.dev/concepts/models/) - `model_construct()`, pickling, private attributes, ORM mode
- [ ] [Plugins](https://docs.pydantic.dev/) and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

Hi @skewty,
Thanks for reporting this. We won't change this in a minor version, as this would be a breaking change. For now, I think logic to strip the trailing underscore should reside in user code, but I've tagged this with our V3 tag as something we could revisit down the line for a major release.

Hi @sydney-runkle,
Could you please add similar functionality (in the future) to the to_pascal() function?


How could one implement onWIllPop for a form in a wolt sheet?

I have a form as a page I want to show the normal all unsaved data will be lost before the user goes back. how could I do this?

Okay I just figured you will have to manually implement it in the leadingNavBarWidget and set the pageIndexNotifier correctly

But how would you implement it if user dismiss the bottom sheet by dragging?

If you know the  index of the form (which you most probably do) you can use the onModalDismissedWithBarrierTap and onModalDismissedWithDrag to intercept based on the notifier index value and based on the result do something -> pop or remain

same problem...


Issue with time.Duration

This works fine:
package mytest

type MyDuration int64

//proteus:generate
type MyType struct {
        Sleeping MyDuration
}

$ proteus -p github.com/alban/mytest -f . ; echo $?
0

But this does not work:
package mytest

import (
        "time"
)

//proteus:generate
type MyType struct {
        Sleeping time.Duration
}

$ proteus -p github.com/alban/mytest -f . ; echo $?
generated.proto: warning: Import google/protobuf/duration.proto but not used.
ERROR: field MyType.Sleeping is a native type and in proto3 syntax with nullable=false there exists conflicting implementations when encoding zero values--gofast_out: protoc-gen-gofast: Plugin failed with status code 1.
error generating Go files from "github.com/alban/mytest/generated.proto": exit status 1
1

I don't understand what that means. time.Duration is basically a int64, so what's the difference between the two code snippets?
/cc @asymmetric

@alban I just made a new release with a few fixes. Thank you for the report!

@Serabe thanks!


docs: update TasksDate reference

Types of changes

Changes visible to users:

[ ] New feature (prefix: feat - non-breaking change which adds functionality)

Issue/discussion:
WARNING: If the link is absent, the PR may be closed without review, due to the workload that such reviews usually require.

[ ] Breaking change (prefix: feat!! or fix!! - fix or feature that would cause existing functionality to not work as expected)

Issue/discussion:

[ ] Bug fix (prefix: fix - non-breaking change which fixes an issue)
[x] Documentation (prefix: docs - improvements to any documentation content for users)
[ ] Sample vault (prefix: vault - improvements to the Tasks-Demo sample vault)
[ ] Contributing Guidelines (prefix: contrib - any improvements to documentation content for contributors - see Contributing to Tasks)

Internal changes:

[ ] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[ ] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)
[ ] Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Description
Commit ff8616370012a0023e8dca16b5bed31a7d51c13e moved the location of TasksDate.ts. This PR adjusts sources to changes.
Motivation and Context

How has this been tested?

Screenshots (if appropriate)
Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] My change has adequate Unit Test coverage.

Terms

[x] My contribution follows this project's contributing guide
[x] I agree to follow this project's Code of Conduct

Good spot - thanks very much for fixing it @emmanuel-ferdman!

I've published the update:
https://publish.obsidian.md/tasks/Scripting/Task+Properties#Values+in+TasksDate+Properties


Refactor post_installation playbook to use new convention -> import|include_roles

Since Ansible 2.4, we can adopt this new convention as we can :

Define the task to be used from the role and then have one role to by example, install, uninstall

Pass condition

Old

---
- hosts: webservers
  roles:
    - { role: foo, tags: ["bar", "baz"] }

New

---
- hosts: webservers
  tasks:
  - import_role:
      name: foo
    tags:
    - bar
    - baz

See: http://docs.ansible.com/ansible/latest/playbooks_reuse_roles.html#using-roles


[AL Language VSC Extension] Debugging in Codespaces fails to show the code when hitting breakpoint

Please include the following with each issue:
1. Describe the bug
When using GitHub Codespaces and a breakpoint is hit in the AL code, the execution seems to stop correctly (variables and database statistics are available in debug window), but source code isn't shown. Error:

2. To Reproduce
Steps to reproduce the behavior:

Open an AL-Go repo inside a codespace
Setup your development environment and launch.json
Set a breakpoint within your app code
Publish the app and do the according action in the web client to hit the breakpoint
Verify that the execution stopped and the debugger shows variables + database statistics but an error shows that the source code file can't be openend although it exists under the exact same path

Example Hello World App: https://github.com/lippertmarkus/MyAlGoRepoAppSource/blob/main/MyApp/HelloWorld.al#L9
3. Expected behavior
Source code where the breakpoint was hit should be shown.
4. Actual behavior
Error is shown:
The editor could not be opened due to an unexpected error: Unable to read file
'/workspaces/MyAlGoRepoAppSource/MyApp/HelloWorld.al'
(Unavailable (FileSystemError): Error: No file system handle registered (/workspaces))

Video:
https://github.com/microsoft/AL/assets/1463140/e1537cdd-45aa-4720-a73f-01894816d76b
5. Versions:

AL Language: 13.0.921843
Visual Studio Code: 1.85.2
Business Central: sandbox-23.3.14876.15532-us
List of Visual Studio Code extensions that you have installed: AL Language, COSMO Alpaca Preview

Final Checklist
Please remember to do the following:

[x] Search the issue repository to ensure you are reporting a new issue

[x] Reproduce the issue after disabling all extensions except the AL Language extension

[x] Simplify your code around the issue to better isolate the problem

Note: It also doesn't work with https://marketplace.visualstudio.com/items?itemName=StefanMaron.allanguagelinuxpatcher from @StefanMaron

I noticed that it only works when accessing the codespace with local vs code running on Linux when you have a copy of the repository at the exact same location locally as it is in the codespace

Thanks for the feedback @StefanMaron. Maybe we can get a built-in fix which works everywhere from the AL-Team :)

We do not have proper support for debugging within codespaces.  One of the issue being  the same file canonicalization issues you have seen.  What I can tell you is that it is in the timeline. ETA: "Good question"


Interact with Entities

Unable to open minecart chest
it's an entity not a block

Edit:
Interacting with all entities needs to be implemented, including interacting with villagers

Changed by @nevercast

related to #179

Is this working in the latest version?

interacting with villagers

This certainly isn't implemented.

There could be a clear checklist of things that still need work.

done !


[Feature]: Add readiness/liveness probes for components of the icinga-stack chart

Affected Chart
icinga-stack
Please describe your feature request
We should introduce proper health checks for components of the icinga-stack chart.
For Icinga2, we could hit the API at https://<host>:<api-port>/v1, once we receive a 401 Unauthorized, Icinga2 and its API should be up and running (there's no health endpoint, unfortunately).
For Icingaweb2, we could just hit the webpage.
Director and IcingaDB remain to be evaluated.

icinga2 chart
Regarding the icinga2 chart we can use an httpGet probe, passing a header for basic auth.
We'd need to fill it with credentials (static icingaweb + .Values.global.api.users.icingaweb.password) and pass it base64 encoded.
It could look something like this:
(charts/icinga-stack/charts/icinga2/templates/statefulset.yaml)
readinessProbe:
  failureThreshold: 3
  httpGet:
    scheme: HTTPS
    path: /v1
    port: {{ .Values.service.port }}
    httpHeaders:
      - name: Authorization
        value: Basic {{ printf "%s:%s" "icingaweb" .Values.global.api.users.icingaweb.password | b64enc }}

For the liveness probe we could just pick a higher failureThreshold. I think it's fair to say that we can just assume the chart to work if the icinga2 API answers our request.
icingaweb2 chart
I've already noticed a problem regarding this chart. Simply checking the base URL of that container does not work (using an httpGet probe).
The problem is twofold:

probes follow redirects (return code 302 eventually leads to return code 403)
Icinga Web 2 does not allow for basic auth (as far as I know)

Within a probe we also can't do an equivalent of curl -L "USER:PASSWORD@DOMAIN:PORT/".
This means that we can't authenticate against its main page.
We could however try to check for static images such as /img/favicon.png. This would only tell us that an image of that name can be found though.
Any other ideas are appreciated here :)
Possible example:
(charts/icinga-stack/charts/icingaweb2/templates/deployment.yaml)
readinessProbe:
  failureThreshold: 3
  httpGet:
    scheme: HTTP
    path: /img/favicon.png
    port: {{ .Values.service.port }}

Also, would probes in charts/icinga-stack/templates/internal-databases.yaml be usefull?

Regarding icinga2: I like the proposition, feel free to implement it this way.
Regarding icingaweb2: That's probably the best way to do it right now. As you mentioned, Icingaweb2 doesn't support BasicAuth and the getHttp probes are somewhat limited in their configurability.
Regarding internal-databases: Those use mariadb:latest by default, which comes with its own healthcheck script preinstalled: https://github.com/MariaDB/mariadb-docker/blob/master/healthcheck.sh
We'd need to invoke this properly for our use case. Feel free to take a look at the linked script and write an draft an exec probe :)


Rescan takes many hours when importing existing wallet

Expected behavior
When importing an existing wallet, the rescan process should finish within a reasonable timeframe
Actual behavior
When importing an existing wallet,  the rescan process does not finish within a reasonable timeframe. The rescan process has been running for multiple hours now and have not finished yet.

Steps to reproduce the problem

Create new wallet via Jam
Uninstall Jam
Install Jam again
Import the wallet that you just created

Specifications

Version: v0.2.0
Platform: UmbrelOS 1.1
Browser: Firefox

Additional context
When creating the wallet i wanted to make sure, that I could recover it on via Jam. I unistalled Jam to remove the wallet info. When importing the wallet it has now been stuck rescanning for many hours.

I've tried restarting the Jam app to redo the import, but I get the same problem
I've tried uninstalling and installing Jam app againg to redo the import, but I get the same problem

Hey @theborakompanioni, thanks replying so fast!
I chose to write "reasonable timeframe" because I don't know the actual process of the rescan and what is a realistic timeframe. For reference on a raspberry pi 4 with my current internet connection and SSD drive it took 3 or 4 days to install and load a full node in the beginning of the year, which is the hardware that i'm running Umbrel on.
Regarding the "reasonable timeframe" when I have tried recovering a wallet using sparrow on an old laptop, it usually doesn't take more than a few minutes at most. But maybe it's completely different than with Jam?
If the process take much longer than a few minutes or even hours I think it would be nice to see the process, otherwise it's easy to think that something has gone wrong.

Looking at JoinMarket Wallet RPC API currently, I see two things that could be improved (besides feedback of progress).

There is no way to specify rescan start date. If user knows when wallet was initially created, he could specify that date and rescan would be faster.

For segwit wallets there is no point of rescaning before segwit activation block, so that could be default instead of genesis.

Looking at JoinMarket Wallet RPC API currently, I see two things that could be improved (besides feedback of progress).
1. There is no way to specify rescan start date. If user knows when wallet was initially created, he could specify that date and rescan would be faster.

:+1:

2. For segwit wallets there is no point of rescaning before segwit activation block, so that could be default instead of genesis.

481824 is the default (in Jam).

Does Jam currently ask user for rescan start date when recovering wallet? Guess not and some API that converts date to block height would be helpful?

Yes :upside_down_face:

Does Jam currently ask user for rescan start date when recovering wallet? Guess not and some API that converts date to block height would be helpful?

Currently, it is plain block height input.
But a datepicker is on the roadmap.

https://github.com/JoinMarket-Org/joinmarket-clientserver/pull/1696


[FAIL] Error: Unknown assertion type: contains-some

The full error:
[FAIL] Error: Unknown assertion type: contains-some Error: Unknown assertion type: contains-some at runAssertion (/Users/me/.npm/_npx/1ddbeaccabfe00a4/node_modules/promptfoo/dist/src/assertions.js:256:11) at runAssertions (/Users/me/.npm/_npx/1ddbeaccabfe00a4/node_modules/promptfoo/dist/src/assertions.js:43:30) at Evaluator.runEval (/Users/me/.npm/_npx/1ddbeaccabfe00a4/node_modules/promptfoo/dist/src/evaluator.js:101:74) at async /Users/me/.npm/_npx/1ddbeaccabfe00a4/node_modules/promptfoo/dist/src/evaluator.js:277:25
I was just changing the initial test YAML
# This configuration runs each prompt through a series of example inputs and checks if they meet requirements.
#removed prompts.txt,
prompts: [prompts-v1.txt]
providers: [openai:gpt-3.5-turbo-0613]
tests:
  - description: Test
    vars:
      var1: first variable's value
      var2: another value
      var3: some other value
    assert:
      - type: contains-some
        value:
          - "something "
          - "other "
          - "The other"
      - type: javascript
        value: 1 / (output.length + 1)  # prefer shorter outputs

... everything else is the same

I think this is an example of (human) hallucination on my part.  The correct type is contains-any, not contains-some.  Give that a try and see how it works.
I'll get the docs/example fixed :sweat_smile:


How to limit the CPU usage of mindspore-lite?

When we use Mindspore Lite 2.0 for inference, its CPU usage has reached 200%. How can we limit its CPU usage?

Hello, your question has been received and will be answered as soon as possible.

Can you provide more detailed infomation? your senario and running config, etc.

If you set one thread, the max cpu usage is 100%. Maybe the thread number that you set does not come into effect.
If there's no other process running, then the cpu usage of inference process will be very high. You can use cpulimit command to limit cpu usage.
Are you using 2.0.0rc1 version? If not can you tell me the commit id? btw, your arm64 hardware is a phone or server?  so i can reproduce this issue.

We have three models, and attach each model  with one thread;
Sure we use 2.0.0rc1. Our arm64 is a whiteboard with android system.

So you have three models in one process and call predict function for each model?


IC2 Advanced Circuit

Fix autocrafting issue.
-> maybe add placeholder item
-> replace calculator circuits with placeholder
-> craft placeholder with any calculator circuit

removed calculator
adapted new recipes


disable CHTMLpreview.js on IE<8

Fixes #1137

I want to look at IE < 8 and see what the problem is. It might be something small that can be worked around rather than disabling entirely.

Thanks! Look forward to seeing the fix.

See my comment on issue 1137.
Should the patch (either yours here or the one I suggest) be a hot-fix?  If so, then it should branch from master and be named hotfix-2.5.2, and the pull request would be to master.  You could still use an issue1137 branch if you want, but it should be branched from hotfix-2.5.2 and the pull should be targeted to hotfix-2.5.2 (with a separate pull from hotfix-2.5.2 to master).  That would allow us to add more fixes, if needed.  (There are a couple of measurement issues in HTML-CSS that would be good to fix in a hot-fix as well.)

OK, now that issue 1137 is in the v2.5.2 milestone, which is a hotfix, this pull request will need to be made from the hotfix-2.5.2 branch (now available).  I can rebase this one to that branch if you like (I've already been doing that with my branches that needed to be moved).

Sounds good to me.

Closing this in favor of #1165 (hotfix pull request).


Clear explanation missing on boxens puppet descriptions

It's great that there is a readme, but unless you use Boxen on a daily basis you have no clue how to apply most of it.
As an example
# Set the global default ruby (auto-installs it if it can)
class { 'ruby::global':
  version => '1.9.3'
}

# ensure a certain ruby version is used in a dir
ruby::local { '/path/to/some/project':
  version => '1.9.3-p194'
}

This is great, but where do I put these? Trial and error is the only answer I have come up with which is likely to be quicker than running through the boxen tutorials.
Boxen is very nicely done, and seems like a good structure, but it feels like vodoo.
I have no clue where to use what syntax and why, so all I know how to do is copy-paste to the right place. It seems to me that extensive examples and docs are key to success with boxen unless you are a master Ruby sys admin.

@thepian is this still an issue or can it be closed?

Closing this since been a while since it was originally reported. If it's still an issue, feel free to reopen.


Relax redundant-key check in _check_dsk

Possible solution for the test_setitem_hardmask component of https://github.com/dask/dask/issues/10672
It seems that we are knowingly adding redundant keys for array setitem operations (https://github.com/dask/dask/blob/main/dask/array/slicing.py#L2064).  It seems like we are only catching this in Windows, because other systems seem to tokenize distinct np.ma.masked objects to different values. I added a new test (test_setitem_slice_twice) that also fails on Linux without the proposed change.
Question: Should we relax the redundant-key check in _check_dsk (as this PR currently does). Or, should we avoid doing things like this?
cc @charlesbluca

thanks @rjzamora and @charlesbluca !