feat: demo wallet

Description

This PR implements the Demo Wallet feature, which is a loose combination and variation on the following specifications:
This PR implements the Demo Wallet feature.

Related issue
Product specification

Note:
From the current spec, using the type property of the useWallet hook is not feasible, since this property is mutable in the context of the "actual connected wallet" and is currently (mostly) used to determine the initial state of the "Connect A Wallet" modal.
As a result, and with some valuable insight from @majorhayes (much appreciated), it was decided that using the existing isConnected property would cover almost all scenarios where we need to differentiate between an "actually connected" wallet, and a wallet that is not connected, or a demo wallet. The few exceptions for this could also be accounted for by checking the deviceId in walletInfo`.
Still outstanding:

Production Demo Wallet xPubs: to be created and added
E2E tests: underway and will push and update soon

Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

Resolves #1621
Risk

TODO
Testing

TODO
Screenshots (if applicable)
TODO

suggestion for @DiggyDiggy2 @reallybeard @SS-FOX-McCloud - if demo wallet is connect, we change the default timeframe on the chart to the all timeframe - otherwise we land on a zero balance chart. thoughts?

suggestion for @DiggyDiggy2 @reallybeard @SS-FOX-McCloud - if demo wallet is connect, we change the default timeframe on the chart to the all timeframe - otherwise we land on a zero balance chart. thoughts?

I'd say that is fine, better than showing them a flat line.

can we get a demo wallet connected here that has assets in it for testing? is this the one that Ops made?

@0xf11nix w.r.t this comment
Still outstanding:

Production Demo Wallet xPubs: to be created and added
E2E tests: underway and will push and update soon

does this open hdwallet PR address it? https://github.com/shapeshift/hdwallet/pull/519

does this open hdwallet PR address it? https://github.com/shapeshift/hdwallet/pull/519

@0xdef1cafe I believe this is the issue related to generating the list of xpubs for the "official" demo wallet
https://github.com/shapeshift/security/issues/30
@mrnerdhair Is this still the case?

does this open hdwallet PR address it? shapeshift/hdwallet#519

It does now... shoved the code to generate the xpub list in there, and put the demo wallet's xpub list in a comment

A few things I've found, maybe because this is still a draft and it's being tackled on but leaving it here just as a reference as a non-aggressive comment:

It's possible to click the "Receive" button on account/asset page - which brings the "Connect Wallet" Modal
Accounts link is still on the nav
Accounts link is still on the asset page
Balance button is still there
Disconnect button still takes back to the "Connect Wallet" splash screen

@gomesalexandre Thank you for your review.
Unfortunately, there was a little misunderstanding between the original github issue and the most recent product specification. After discussing with @DiggyDiggy2 and @0xdef1cafe, we settled on something much closer to the latter.
To try and clear things up, I have created a new issue to try and better encapsulate the actual desired outcome. Note: the previously hidden elements, as well as forcing the using back to the "connect wallet" screen, are no longer required as part of this new spec.
Again, thanks for taking the time to review these changes, I do hope this new spec better explains some of the design decision taken.

@0xf11nix few things here

let's get the updated demo wallet xpubs committed
let's set the default timeframe on the charts to Timeframe.All if the demo wallet is connected
can we add a feature flag so we can get this merged/iterate without a goat?

@0xdef1cafe I have addressed these requested changes in the following commits:

94d0ea775f946cb294188c3a45c2e6a86109e075
cc9e95a44ab3493b752be5adce17305d7d255b13
6fb2555bb63af63abd3b89e65bb8f145deecef3d

The feature flag might be a little annoying for testing, since you need to connect a wallet in order to first enable the Demo Wallet flag, but I guess we can work around that.
Now that the feature is behind a flag, my Cypress tests are a bit broken, so I'll push those once I find a work-around.

@0xf11nix this looks dope AF. i've turned the flag on in preview cloudflare environments and will do the same for fleek.
two things i noticed while testing this with @DiggyDiggy2

Bug fix - defi/earn deposit and withdraw - the "Sign & Broadcast" button should open the connect wallet modal
Bug fix - "Confirm and Trade" button on trade widget white screens then refreshes page to /?#/dashboard
Let's change the default timeframe to 1 month on the portfolio charts - so we're not seeing a long period of flatlined zero balance

let's get these small fixes made and this PR merged behind the flag, and we're probably good to turn it on, after another mini goat.
additionally, we created another ticket that product will fully flesh out for some additional enhancements. this is scope creep but all really valuable stuff
https://github.com/shapeshift/web/issues/1637

Bug fix - defi/earn deposit and withdraw - the "Sign & Broadcast" button should open the connect wallet modal
Bug fix - "Confirm and Trade" button on trade widget white screens then refreshes page to /?#/dashboard
Let's change the default timeframe to 1 month on the portfolio charts - so we're not seeing a long period of flatlined zero balance. **let's make this

@0xdef1cafe I have addressed the three issues above.
After getting the wallet topped up with some ETH, we can now properly test some of the edge cases.
There does appear to be a new issue related to the approval step, before "Confirm and trade", so I'm working on that now.
I will also address the merge conflict before I mark these changes as ready for review.

Bug fix - defi/earn deposit and withdraw - the "Sign & Broadcast" button should open the connect wallet modal
Bug fix - "Confirm and Trade" button on trade widget white screens then refreshes page to /?#/dashboard
Let's change the default timeframe to 1 month on the portfolio charts - so we're not seeing a long period of flatlined zero balance. **let's make this

@0xdef1cafe I have addressed the three issues above.
After getting the wallet topped up with some ETH, we can now properly test some of the edge cases. There does appear to be a new issue related to the approval step, before "Confirm and trade", so I'm working on that now.
I will also address the merge conflict before I mark these changes as ready for review.

awesome. also when i was reviewing with @DiggyDiggy2 in person - let's just change the default timeframe for all charts to 1 month, day is too noisy, month gives enough context, and will have a full history for this demo wallet.

awesome. also when i was reviewing with @DiggyDiggy2 in person - let's just change the default timeframe for all charts to 1 month, day is too noisy, month gives enough context, and will have a full history for this demo wallet.

@0xdef1cafe I think we should be good to go now. I have made the above change, as requested, and addressed a number of other small bugs that were raised over the last couple of days.
As per our discussion, and noted in the description above, we will hold off on the E2E tests for now and include them in a follow-up PR once the feature flag is removed.


more test coverage

Codecov Report

Merging #640 into master will decrease coverage by 4.58%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #640      +/-   ##
==========================================
- Coverage   88.85%   84.27%   -4.59%
==========================================
  Files          47       47
  Lines        3348     3357       +9
==========================================
- Hits         2975     2829     -146
- Misses        373      528     +155

Impacted Files
Coverage Δ

tests/test_menu.py
61.6% <66.66%> (-38.4%)
:arrow_down:

vent/menus/logs.py
21.73% <0%> (-69.57%)
:arrow_down:

vent/menus/editor.py
52.77% <0%> (-36.12%)
:arrow_down:

vent/menus/add.py
80.95% <0%> (-15.88%)
:arrow_down:

vent/menus/inventory_forms.py
85.71% <0%> (-14.29%)
:arrow_down:

vent/menus/main.py
65.29% <0%> (-11%)
:arrow_down:

vent/api/actions.py
87.09% <0%> (-8.22%)
:arrow_down:

vent/menus/tutorials.py
93.75% <0%> (-6.25%)
:arrow_down:

vent/menus/tools.py
60.36% <0%> (-4.88%)
:arrow_down:

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 d96682e...fc3932d. Read the comment docs.


Sub 1000ms, with secondsDecimalDigits set to 0, shows ms

Reproducible example:
import prettyMilliseconds from 'pretty-ms';
const timeStamp = prettyMilliseconds(999, {
  secondsDecimalDigits: 0,
  colonNotation: true,
});
console.log(timeStamp);

Expected: 0:00
Result: 0:00:999
This appears to happen for any sub 1000ms input.
Setup: pretty-ms(v7.0.0), ReactJS (v16.13.1) in development mode running in Google Chrome (v85.0.4183.102)

// @villebro

Thanks for raising the issue @Mageer and the ping @sindresorhus , I will look into this shortly.

Reproduced the error, what a weird bug!


Metrics temp

Ⅰ. Describe what this PR did
upgrade metrics branch and resolve conflicts

Codecov Report

Merging #589 into metrics_temp will increase coverage by 2.21%.
The diff coverage is 30.66%.

@@                Coverage Diff                 @@
##             metrics_temp     #589      +/-   ##
==================================================
+ Coverage           30.57%   32.79%   +2.21%
- Complexity            670      825     +155
==================================================
  Files                 184      209      +25
  Lines                7139     8111     +972
  Branches              848      946      +98
==================================================
+ Hits                 2183     2660     +477
- Misses               4733     5188     +455
- Partials              223      263      +40

Impacted Files
Coverage Δ
Complexity Δ

...n/src/main/java/com/alibaba/fescar/common/XID.java
93.75% <ø> (ø)
9 <0> (?)

...m/alibaba/fescar/tm/api/TransactionalTemplate.java
68.42% <ø> (ø)
2 <0> (ø)
:arrow_down:

...main/java/com/alibaba/fescar/common/Constants.java
0% <ø> (ø)
0 <0> (?)

...ibaba/fescar/core/protocol/RegisterRMResponse.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...a/fescar/core/rpc/netty/TransportProtocolType.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...a/fescar/rm/datasource/PreparedStatementProxy.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...libaba/fescar/server/AbstractTCInboundHandler.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...n/java/com/alibaba/fescar/config/ConfigFuture.java
63.82% <ø> (ø)
11 <0> (ø)
:arrow_down:

...m/alibaba/fescar/server/session/SessionHelper.java
73.07% <ø> (ø)
4 <0> (ø)
:arrow_down:

...libaba/fescar/tm/api/DefaultGlobalTransaction.java
68.96% <ø> (ø)
15 <0> (ø)
:arrow_down:

... and 181 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 bdc4740...5256eb8. Read the comment docs.


Scripts stopped working...

Hi Ken,
We are using TagUI to automate one of the use case where we want to launch chrome browser -> enter url -> perform login -> launch Citrix app.
We are executing this on windows VM machine. When I log into machine and execute scripts, its working fine. However, when I jump off from RDP (VM) with below script where session remain active and it also render visual. My scripts also executed, it able to launch chrome -> enter url and then stopped.
Any idea or suggestions or pointer for the above issue?
Regards,
Tejas

Hi Tejas, can you paste the log files here to see if there are more clues what is going on. The useful log files will be the log file for your script, the tagui\src\tagui_chrome.log, and if you are using visual automation, then the logs in tagui\src\tagui.sikuli folder.
There might be some behaviour that happens when the  VM is disconnected that results in this outcome, but there's little other reference of this issue and I hear that some users are able to run VM in background without issues.

Hi Ken,
Attached logs and also script for the ref.
I am executing this command on VM, this will kick me off from the RDP but keep the session active and it also keep rendering display.
for /f "skip=1 tokens=3" %%s in ('query user %USERNAME%') do (
%windir%\System32\tscon.exe %%s /dest:console
)
tagui_files.zip

Hi Ken,
Did you get chance to look into this?
Regards,
Tejas

From the SikuliX log, below is the error. It looks like you have OpenCV on your VM. Try running this on a VM without OpenCV installed to see if the error happens. It may be a conflict between the OpenCV on your computer and the one used by SikuliX.
If after uninstalling your OpenCV or using another VM without OpenCV, it might be the display is somehow no longer rendered for SikuliX computer vision engine to consume and lead to the error. I'm afraid I do not have further details on SikuliX being used on a VM that is running in the background to advise further.
[tagui] ACTION - present C:/TagUi_app/tagui/src/chrome_is_max.png
[error] script [ tagui ] stopped with error at line --unknown--
[error] Error caused by: Traceback (most recent call last):
  File "C:\TagUi_app\tagui\src\tagui.sikuli\tagui.py", line 417, in <module>
    intent_result_value = parse_intent(tagui_intent)
  File "C:\TagUi_app\tagui\src\tagui.sikuli\tagui.py", line 369, in parse_intent
    return visible_intent(script_line);
  File "C:\TagUi_app\tagui\src\tagui.sikuli\tagui.py", line 300, in visible_intent
    if exists(params):
]
	at org.opencv.imgproc.Imgproc.matchTemplate_1(Native Method)
	at org.opencv.imgproc.Imgproc.matchTemplate(Imgproc.java:2612)
	at org.sikuli.script.Finder$Finder2.doFindMatch(Finder.java:898)
	at org.sikuli.script.Finder$Finder2.doFind(Finder.java:868)
	at org.sikuli.script.Finder$Finder2.find(Finder.java:617)
	at org.sikuli.script.Finder.find(Finder.java:280)
	at org.sikuli.script.Region.runFinder(Region.java:2958)
	at org.sikuli.script.Region.doFind(Region.java:2894)
	at org.sikuli.script.Region.access$300(Region.java:23)
	at org.sikuli.script.Region$RepeatableFind.run(Region.java:3175)
	at org.sikuli.script.Region$Repeatable.repeat(Region.java:3126)
	at org.sikuli.script.Region.exists(Region.java:2442)
	at org.sikuli.script.Region.exists(Region.java:2466)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
org.opencv.core.CvException: CvException [org.opencv.core.CvException: cv::Exception: OpenCV(3.4.2) C:\build\3_4_winpack-bindings-win64-vc14-static\opencv\modules\imgproc\src\templmatch.cpp:1107: error: (-215:Assertion failed) _img.size().height <= _templ.size().height && _img.size().width <= _templ.size().width in function 'cv::matchTemplate'
]

Hi Ken,
Based on your input, I have checked VM machine... but I did not find any opencv installed...
attached few snapshots .... am i missing something here?

org.opencv.core.CvException: CvException [org.opencv.core.CvException: cv::Exception: OpenCV(3.4.2) C:\build\3_4_winpack-bindings-win64-vc14-static\opencv\modules\imgproc\src\templmatch.cpp:1107: error: (-215:Assertion failed) _img.size().height <= _templ.size().height && _img.size().width <= _templ.size().width in function 'cv::matchTemplate'
]

Oh Tejas, it is on the last line of the Sikuli log, see above. The error is pointing to this OpenCV file below. This is not the way SikuliX uses OpenCV as it has its own OpenCV packaged within itself. So not sure why there is below OpenCV, which may be the cause of error.
OpenCV(3.4.2) C:\build\3_4_winpack-bindings-win64-vc14-static\opencv\modules\imgproc\src\templmatch.cpp
Uninstalling or removing may still not fix the issue, because SikuliX I don't think is tested to work for VMs running in background mode.

Right Ken... even When I am running 4-5 times, success ratio is 70%.... 30% it fails...
and for automation, I cant keep VM to be logged-in 24x7, that's also biggest issue...
Anyways, thank you so much Ken for your time and share feedback....
Do you suggest any other similar tool which can work on VM which is in background?
Regards,
Tejas

I haven't heard of other tools that are designed to run on VMs and in the background. Your best bet will be trying some community free versions of commercial tools like UiPath or Automation Anywhere. If those enterprise tools can't support the use case running in VM in background, it is unlikely other open-source tools have the capability to meet this uncommon use case.
If the reason for not running 24 x  7 is the cost, then maybe a workaround is automate / schedule the logging in and out so that during the times when automation is needed VM will load, and after that VM will close. Microsoft is partnering with Automation Anywhere and UiPath to have their apps running on Azure cloud servers, maybe they'll have a solution that works for this use case.

Closing issue for now pending further new inputs!


Demonstration dashboard for Versity gateway metrics

This is a Grafana-based dashboard and docker-compose demonstration for the Versity Gateway built around influxdb.
See README.md in the metrics-exploration folder for usage details.

This won't automatically run in continuous integration without approval. A member of the Versity organization must allow it.


Amend call out data text

Fiona has shared some text to add to the call out data page

Picked up here: https://github.com/open-innovations/leeds-2023/issues/76


Wait 5 s after creating release before uploading assets

Fix #67

Oops. I think this was resolved by #66

I take that back. I just installed the master branch locally and encountered the same issue. I think the problem is that the solution in #66 will retry the request if there is an HTTPError, but here, no HTTPError is raised. The request for releases returns fine, but it does not include the release that was just created less than a second before. Therefore, in get_release_info, releases is None and this exception is raised: https://github.com/j0057/github-release/blob/4ad1a66a34620ace9e881d9ce0fcc14994cdab30/github_release.py#L339

Not the maintainer but probably the fix should try to re-use the retry mechanism introduced in the previous branch instead of hard coded sleep.

Done. Thanks for the feedback!

Thanks @rly and @owais

I will get a release out tomorrow :rocket:

@jcfr do you have any updates on a release for this? thanks!

@jcfr any idea on a release?

The plan is to finalize tmrw morning.

@owais @rly @mavaylon1 Thanks for your patience :pray: Release 1.5.9 has now been uploaded :rocket:

Thank you @jcfr!


driver issue on toDate function inconsistent result with native cli

Describe the bug
when using toDate function the result is not the same as not running with toDate or with the result of running on Clickhouse-client
Steps to reproduce
Expected behaviour
returning 10-20 not 10-19
Error log

t = clickhouse_query("""select toDate(published_at) from divar.posts where toDate(published_at)='2023-10-20' limit 10""") print(t.result_rows) toolbox:clikhouse# venv/bin/python command.py  [(datetime.date(2023, 10, 20),), (datetime.date(2023, 10, 20),), (datetime.date(2023, 10, 20),), (datetime.date(2023, 10, 20),), (datetime.date(2023, 10, 20),), (datetime.date(2023, 10, 20),), (datetime.date(2023, 10, 20),), (datetime.date(2023, 10, 20),), (datetime.date(2023, 10, 20),), (datetime.date(2023, 10, 20),)]
Configuration
Environment

metabase-clickhouse-driver version: 1.2.1
Metabase version: v0.47.4

ClickHouse server

ClickHouse Server version: <IP_ADDRESS>

@AfrouzMashayekhi, are Metabase and ClickHouse timezones the same?

we have the same problem too.
select now() and select today() show different date. it is interesting that when i use select toString(today()) the output is the same as select now()

toDate(now())  function's output is the same as today() function
Environment

metabase-clickhouse-driver     version: 1.2.3
Dockerized Metabase                version: 0.47.2      timezone: utc
clickhouse server                       version: 22.6.1.     timezone: utc

@AfrouzMashayekhi, are Metabase and ClickHouse timezones the same?

Yes, both in UTC timezone

@AfrouzMashayekhi, are Metabase and ClickHouse timezones the same?

Yes, both in UTC timezone.

@AfrouzMashayekhi @niloo-sh
In your instances, what a query such as
WITH arr AS (SELECT arrayMap(x -> format('2023-12-12 {}:08:00', leftPad(toString(x), 2, '0')), range(0, 24)) AS a),
     cte AS (SELECT arrayJoin(a) AS s FROM arr AS s)
SELECT s, toDateTime(s) AS dt, toString(dt) AS dts, toDate(dt) AS d, toString(d) AS ds FROM cte;

prints if executed from the Metabase query editor?
today/now (tied to ClickHouse time, which is tied to Docker time, which is tied to the system time) are quite challenging to test appropriately, so we have to be creative here.
Mine (the system is GMT, CH 23.11 is UTC, MB 0.48.1 is UTC, driver version 1.3.0) gets the following (i.e. no difference between any of these):
[
{"s":"2023-12-12 00:08:00","dt":"2023-12-12T00:08:00","dts":"2023-12-12 00:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 01:08:00","dt":"2023-12-12T01:08:00","dts":"2023-12-12 01:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 02:08:00","dt":"2023-12-12T02:08:00","dts":"2023-12-12 02:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 03:08:00","dt":"2023-12-12T03:08:00","dts":"2023-12-12 03:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 04:08:00","dt":"2023-12-12T04:08:00","dts":"2023-12-12 04:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 05:08:00","dt":"2023-12-12T05:08:00","dts":"2023-12-12 05:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 06:08:00","dt":"2023-12-12T06:08:00","dts":"2023-12-12 06:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 07:08:00","dt":"2023-12-12T07:08:00","dts":"2023-12-12 07:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 08:08:00","dt":"2023-12-12T08:08:00","dts":"2023-12-12 08:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 09:08:00","dt":"2023-12-12T09:08:00","dts":"2023-12-12 09:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 10:08:00","dt":"2023-12-12T10:08:00","dts":"2023-12-12 10:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 11:08:00","dt":"2023-12-12T11:08:00","dts":"2023-12-12 11:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 12:08:00","dt":"2023-12-12T12:08:00","dts":"2023-12-12 12:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 13:08:00","dt":"2023-12-12T13:08:00","dts":"2023-12-12 13:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 14:08:00","dt":"2023-12-12T14:08:00","dts":"2023-12-12 14:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 15:08:00","dt":"2023-12-12T15:08:00","dts":"2023-12-12 15:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 16:08:00","dt":"2023-12-12T16:08:00","dts":"2023-12-12 16:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 17:08:00","dt":"2023-12-12T17:08:00","dts":"2023-12-12 17:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 18:08:00","dt":"2023-12-12T18:08:00","dts":"2023-12-12 18:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 19:08:00","dt":"2023-12-12T19:08:00","dts":"2023-12-12 19:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 20:08:00","dt":"2023-12-12T20:08:00","dts":"2023-12-12 20:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 21:08:00","dt":"2023-12-12T21:08:00","dts":"2023-12-12 21:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 22:08:00","dt":"2023-12-12T22:08:00","dts":"2023-12-12 22:08:00","d":"2023-12-12","ds":"2023-12-12"},
{"s":"2023-12-12 23:08:00","dt":"2023-12-12T23:08:00","dts":"2023-12-12 23:08:00","d":"2023-12-12","ds":"2023-12-12"}
]

Closing this due to inactivity; if this issue persists even after 1.50.1, please feel free to re-open/create a new one.


NBN Technology: undefined error for Tasmanian premises

Hey Luke,
Thanks for making this extension!
Over the last few weeks, I have noticed when looking at premises located in Tasmania, they all come up with an NBN Technology: undefined error message. I will leave an example below and a link to a home for sale.
For reference, I also use another extension for checking NBN technology types. https://chrome.google.com/webstore/detail/nbn-availability-checker/dphlehoebkdjgennbalpfpjnhjojcceo
https://www.realestate.com.au/property-house-tas-lenah+valley-139096555

Thanks for the report I'll have a look into what's causing this and see if I can find a solution.

Hi Luke, Great extension.
Is there a way to display the NBN status on the initial realestate.com list page, rather than only on the individual listing?

Hi @Hongie84,
I'm glad you like the extension. I currently support the saved property pages for real-estate.com.au and domain.com.au. Is this what you were looking for on the homepage?

Hey I wasnt expecting a reply that quick.
I'm talking about on the search list. I will attach a screenshot to show
you where.
I had used a different extension that was displaying it on this page, and
it has stopped working now, I imagine realestate.com has changed something?
regards
Brad
[image: image.png]
On Mon, 30 May 2022 at 13:07, Luke Prior @.***> wrote:

Hi @Hongie84 https://github.com/Hongie84,
I'm glad you like the extension. I currently support the saved property
pages for real-estate.com.au and domain.com.au. Is this what you were
looking for on the homepage?
—
Reply to this email directly, view it on GitHub
https://github.com/LukePrior/nbn-availability-extension/issues/2#issuecomment-1140636849,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZMROZQQCCAK5DRYZ3UMXALVMQWINANCNFSM5URCVLWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok, I will look into adding that.
Notes to self:

fix realestate.com.au single page display
use list view for search page


crio is crashing doe to duplicate config in /etc/crio/crio.conf

Description of problem
kata-deploy.sh (executed inside of kata-deploy container) is touching /etc/crio/crio.conf
It writes two entries with [crio.runtime.runtimes.kata-qemu] and then crio refuses to parse the config file with the error:
"unable to decode configuration /etc/crio/crio.conf: Near line 304 (last key parsed 'crio.runtime.runtimes'): Key 'crio.runtime.runtimes.kata-qemu' has already been defined."
logs of the observed issue:
https://gist.github.com/hnanni/c130bf4d961456500900dca79457dfde

@hnanni - do you have some details of which version of kata/kata-deploy you were using? I thought this issue had been seen and fixed many months ago :-) /cc @egernst @amshinde

@grahamwhaley  It is happening with the latest docker image, reverting back to 1.8.2 fixed it for me.
With latest I was getting this in my crio.conf
Path to the Kata Containers runtime binary that uses the QEMU hypervisor.
[crio.runtime.runtimes.kata-qemu]
runtime_path = "/opt/kata/bin/kata-qemu"
Path to the Kata Containers runtime binary that uses the QEMU hypervisor with virtiofs support.
[crio.runtime.runtimes.kata-qemu]
runtime_path = "/opt/kata/bin/kata-qemu-virtiofs"

@CraigSterrett yeah, seems that L98 needs to be fixed.
Let me open a PR to fix this.


Fixed rkt typo in Makefile

Makefile accidently mentioned 'rkt' in a comment in the 'calico-kubernetes' section.

LGTM


Add STEER_TORQUE to car kalman filter

Some cars have bad angle sensors. If we switch the PID loop to use this, they will be improved.
I even wonder if this will make the angle offset more accurate across the board, since it feels the torque. But I don't really know.
Need to fix the hardcoded constants and do large scale stability tests.

Hmm, this model isn't really right at all. It needs to be velocity dependent.

Code to run the filter for testing:
https://gist.github.com/geohot/7c387246949e343c1ce21315de844fd2

I think the deeper question is:
What units are STEER_TORQUE_EPS and STEER_TORQUE_CMD in (they are the same)? At low speeds, the torques are way too low to be correct.
And this would correspond with observed behavior. The same torque command at lower speeds turns the wheel more than that command at higher speeds.

Proposing the torque unification project.
We have all these controllers that output "torque", but what does that mean. Let's unify it, and write a function that understands what the torques mean per car.

After measuring steer delay from some ZSS data I'm getting 0.6 seconds as mean, even though it should be 0.12 right? I'm measuring from steer torque cmd to change in zss angle


DockerHub rate limit

Do you want to request a feature or report a bug?

Feature
What did you expect to see?
https://github.com/traefik/traefik/issues/7566
Failed to pull image "traefik:2.4.8": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/library/traefik:2.4.8": failed to copy: httpReaderSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/library/traefik/manifests/sha256:08d8a7759f5fffa2441488151cedcd4d556c1f124c097f929f469c1f7b82c16f: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

Not this. It has been almost a year since DockerHub rate limits have been imposed. This is a huge pain point to have your ingresscontroller be rate limited causing downtime. According to previous issues traefik was supposed to not be rate limited. This is not the case. Is it possible to push to another registry in CI? Having another registry helps incase DockerHub suffers downtime.

@anthr76 Hello. Having the image hosted officially anywhere else will take take some time (if ever), as we would have to study all the alternatives, and the related pros and cons (including hosting and egress costs, etc), before we can actually do it.
Our recommendation for now is to pull while being authenticated with dockerhub, as it should give you a more lenient rate limit.

@mpl I wouldn't call an additional 100 pulls per 6 hours more lenient while authenticated. Why can traefik use GHCR?

@mpl I wouldn't call an additional 100 pulls per 6 hours more lenient while authenticated. Why can't traefik also use GHCR?

we can't just also use , because any alternative that has the best rate-limiting for the users will de facto become the solution that everyone wants to use. Therefore it basically becomes our official image(s), and that is why we have to study them and choose the best one we can. Otherwise we'll basically end up in the same situation, just with another provider.

Traefik maintainers can also apply for the Dockerhub Open source developer license which will remove the pull limits for Traefik.
https://www.docker.com/blog/expanded-support-for-open-source-software-projects/

AFAIK (but I could be wrong), we already did that but never got any response so far. But I'll ask again where we are on that.

Hi @mpl It's seems this isn't something traefik or you are really interested in doing for one reason or not. Perhaps things are a bit hectic and you can't be bothered for a few google searches.
This is somewhat of a big deal. Authenticated dockerhub pulls are great until your within a corporate network and share one public IPv4. If you have clusters behind that you quickly reach a rate-limit authenticated or not. Like @onedr0p eluded the quickest and best way to fix this issue is to actually follow through with https://github.com/traefik/traefik/issues/7566 a locked issue from almost a year ago.
It seems like Traefik+Dockerhub doesn't have the capacity to make the " expanded support for open source software projects" work. That's okay. Considering the source code for Traefik lives on GitHub in my opinion it makes total sense to push to both Dockerhub and GHCR. Traefik has (a long time ago) outgrown traefik. It's deployed in OCI and non infrastructure. Limiting your image to only DockerHub would have made sense in 2017 but not now. It didn't even before rate limits were imposed considering your supported platforms.
I can't find one thing on the internet for egress limits with GHCR. With about 30 added lines https://github.com/traefik/traefik-library-image/blob/master/.github/workflows/main.yml trafeik can prevent a production ingress outage.
I totally agree we can't predict to the future and GitHub can turn just as evil as DockerHub but just sending a issue to the do never backlog isn't great and can loose potential customers...

Just ran into this issue as well. Traefik is one of very few containers I run in my cluster that still hasn't either completely switched away from dockerhub or provides an official mirror on another registry.

For those coming across this issue I've pushed the image to a public GHCR repo.
https://github.com/k8s-at-home/container-images-mirror
It can be used in the traefik helm chart like so:
  values:
    image:
      name: ghcr.io/k8s-at-home/traefik

Any update on getting traefik onboarded to the Open source developer license on Dockerhub? It seriously can't be that hard, can it? Intel, on the below repository was able to do this within 8 months.
https://github.com/intel/intel-device-plugins-for-kubernetes/issues/633

I've personally reached out multiple times to docker already on that subject. So let's say, it's at least not fast ;-)

@emilevauge has mentioned that Traefik doesn't meet the qualifications for the Docker open source program.
https://twitter.com/emilevauge/status/1518637216632815620
With that said would it be possible to mirror Traefik to Quay or on this repo under GHCR? GHCR is free for open source projects.

@onedr0p indeed we're investigating this possibility and the effort required to integrate with our build / CI tools

For anyone running into this issue since Traefik folks will not prioritize this annoying issue (who likes to be rate limited when rolling out changes to your ingress?), people can use mirror over at:
https://gallery.ecr.aws/docker/library/traefik

@ddtmachado It seems like the most obvious solution that costs you nothing is to publish your images on GitHub Container Registry (i.e., ghcr.io).  Docker is attaching licensing requirements for pulling images from Docker Hub (e.g., you can use such images for commercial purposes as long as you have a valid Docker Desktop license).  I suppose we could simply fork your repo and do it ourselves but this seems like a waste for something so simple.

@robertpatrick indeed using GHCR is simple enough technically speaking, but not future proof as they might change the policy and start charging for usage in the future.
For context I can give you the main reasons why publishing on it as an alternative to Docker Hub was not prioritized so far:

Publishing the official Docker image is part of our full release process, meaning we only announce a release once that image is available and that would not change.
Current Docker Hub rate limiting policy looks fair to us. 100 pulls per 6 hours per IP address even without a login, and it allows the free use of the images for commercial purposes for small businesses.
There are many workarounds that readily available and well documented, like using a mirror or creating a local registry to cache your images.

Having said that we're still open to the possibility of adding GHCR or other alternatives, but prioritizing the time to do that over improving Traefik Proxy for the broader user base is not in the cards for the foreseeable future.

Given the recent dockerhub announcement of removing free team orgs, I'm wondering if the current org will be affected? And if so perhaps this could finally have a high priority?
https://github.com/docker/hub-feedback/issues/2314

The official Traefik is not affected by this because it's an official image handle by Docker it-self.


Async requests handling

Does it handle case when my code will call checkMayDispatchRequest() asynchronous, let's I start 1000 requests, before any completeRequest() is executed? Or I need to make all the requests synchronously one by one?

You can still run multiple parallel request. Important is; if checkMayDispatchRequest() returns a value >0 do not send the request, otherwise you will hit a rate limit. The value returned by checkMayDispatchRequest() is the milliseconds you should wait to retry this request. If you retry a request call checkMayDispatchRequest() again to
if limiter.checkMayDispatchRequest() allows a request (return value 0) it will increase the pendingUsage usage in all relevant counters for a given request. So the counters can decide for further request if they still fit into the rate limits.
limiter.completeRequest() will then analyze the response headers and if the used weight returned from the server is higher update each counters current usage.
If you are close to the ip weight limits, checkMayDispatchRequest() might return a value >0 for a request with a high ip weight, but still return 0 for a request with a low ip weight.
If you want to log the current usage the limiter has a getUsage() method.
Hope this answer helps.

Yes, thank you. It works! Do you have any plans to implement fapi besides sapi?

I currently don't have the time to implement fapi. But a PR is always welcome.
As the limits on fapi look very similar to the ones in spot, it should probably doable with these 3. steps

copy the spot config and adapt it to fapi
create the fapi limiter (copy and rename spot limiter)
create the fapi queue (copy and rename spot queue)

Not the spot limiter is only for /api endpoints. The /sapi endpoints have limits for each single endpoint, whereas /api endpoints share the limits. According to docs /fapi works very similar to /api.

BTW, I think it is still can go out and be blocked in parallel requests, because before we have response with the first request and count weights from headers, it can be second request and weights from the first request will not be counted

Not the spot limiter is only for /api endpoints. The /sapi endpoints have limits for each single endpoint, whereas /api endpoints share the limits. According to docs /fapi works very similar to /api.

So your librari is only for /api and doesn't implement /sapi rate limits?

@skliarovartem

So your library is only for /api and doesn't implement /sapi rate limits?
correct. There is currently no plan on my side to implement /sapi as it is a custom rate limit that does not affect /api. But feel free to open a Pull Request to implements it.

BTW, I think it is still can go out and be blocked in parallel requests, because before we have response with the first request and count weights from headers, it can be second request and weights from the first request will not be counted

It takes the X-MBX-USED-WEIGHT-(intervalNum)(intervalLetter) from the header, which is always the used weight in a given window. The window resets at the second 0 of the given interval. If the used weight in the header is smaller then the one tracked in the counter, the used weight from the counter is not overriden by the smaler used weight from the header.


PowerToys V0.43 causes delayed keyboard commands in Windows Universal Remote Desktop

Microsoft PowerToys version
0.43.0
Running as admin

[X] Yes

Area(s) with issue?
General
Steps to reproduce
Open up a remote session with Windows 10 Universal Remote Desktop application.  Observe the typing speed in notepad.  Turn on Windows PowerToys in administrator mode.  Observe that the typing speed in notepad in RDP is significantly delayed.
PowerToysReport_2021-08-06-09-49-01.zip
✔ Expected Behavior
No typing delay.
❌ Actual Behavior
Typing text is delayed in RDP.
Other Software
No response

@KyleLeMarbe to confirm, this performance gap only occurs when PowerToys is run as an administrator? Otherwise it behaves normally?

This issue happens regardless of administrator status.  It also occurs when running in standard user mode.

@KyleLeMarbe can we work with you to see if maybe one particular utility is causing this?  Looking at the JSON, 8 utilities are enabled.  Also your response to Deondre is seems to contradict what is above.  Is it when PT is run in admin, it is worse?
{
  "startup": true,
  "enabled": {
    "Awake": false,
    "ColorPicker": true,
    "FancyZones": true,
    "File Explorer": true,
    "Image Resizer": true,
    "Keyboard Manager": true,
    "PowerRename": true,
    "PowerToys Run": true,
    "Shortcut Guide": true
  },
  "is_elevated": true,
  "run_elevated": true,
  "download_updates_automatically": true,
  "is_admin": true,
  "theme": "system",
  "system_theme": "dark",
  "powertoys_version": "v0.43.0"
}

I believe this was resolved in the .51 release, but will need to test it to be sure.  There was a duplicate case submitted near the same time I submitted this one.  I believe it was included in the release notes.

Similar to #12697

Would love to know if it is fixed or if we need to dive deeper


Support Drag-and-Drop for Order of Execution

As a part of the User Story "Drag-and-Drop for Order of Execution" (https://github.com/almighty/almighty-core/issues/498), the UI should support drag-and-drop for Order Of Execution of workitems.

@DhritiShikhar why did you create this issue? Why did you assign it to the next sprint before we voted on the stories of the that sprint?

@joshuawilson i removed the milestone. sorry for the confusion.


Refactor Festobionic to new Visitor / Worker structure

Festobionic should as well be refactored to use the new visitor / worker structure.

Implementation Notes to this issue: This should be fairly simple since Festobionic only uses the old ArduinoUsedHardwareCollectorWorker and ArduinoUsedMethodCollectorWorker. It also has no validator yet. This would also change in the refactoring, the ArduinoValidatorAndCollectorVisitor could be used as a visitor for all.
Of course there is some testing need, that's why I wrote a new issue for that instead of fixing it myself in #1163


./OpenCV-4-8-0.sh  build issue Jetpack6

Device:Orin Nx 16GB
Jetpack 6
Cuda 12.2
Cudnn 8.9
python 3.10
./OpenCV-4-8-0.sh - when i ran this command . Im facing issue. I have attached issue screenshot for reference.

It is a well known issue https://github.com/opencv/opencv/issues/23893.
And it is fixed in https://github.com/opencv/opencv/pull/24104.
Please use a later OpenCV version, like 4.10.0

Thanks for your response. I'm using Isaac ROS, and ROS Humble comes with OpenCV version 4.8.0. If I want to install OpenCV version 4.10.0 means I need to uninstall the existing OpenCV version first.when I uninstall OpenCV, the ROS packages also get uninstalled.
I managed to install OpenCV 4.10.0 successfully, but I still need Isaac ROS as well. When I tried to fix the broken packages by running the sudo apt install ros-humble-desktop command, the OpenCV 4.10.0 installation was reverted back to version 4.8.0.

In that case I would modify the OpenCV 4.8.0 code according to https://github.com/opencv/opencv/pull/24104 before compiling.
It's just a few lines.


⚠ PrivateHD has degraded performance

In 3f928d2, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1580 ms

Resolved: PrivateHD performance has improved in d06a2f2.


[PE-D] Many Error messages missing examples

Many error messages are missing examples making it hard for the user to correct the errors
E.g.

Should have its message usage and command format there for user friendliness.
i.e.
echo: Adds a person to folder in UNIon. Parameters: INDEX >> FOLDERNAME
Example: echo 3 >> CS2103

Labels: severity.Low type.FeatureFlaw
original: ruthpohrp/ped#20

KIV - We will make a new user story to re-structure our error messages.


Fix for issue #1259

Updated file with MultiMoyaProvider class definition
Fixes #1259

          1 Warning

:warning:
Consider also updating the Chinese docs. For Chinese translations, request the modified file(s) to be added to the list here for someone else to translate, if you can’t do so yourself.

          3 Messages

:book:
iOS: Executed 253 tests, with 0 failures (0 unexpected) in 11.393 (11.536) seconds

:book:
tvOS: Executed 253 tests, with 0 failures (0 unexpected) in 11.381 (11.563) seconds

:book:
macOS: Executed 253 tests, with 0 failures (0 unexpected) in 12.027 (12.150) seconds

  Generated by :no_entry_sign: Danger


Generic/SpreadOperatorSpacingAfter: move parse error test to its own file

Description

Suggested changelog entry

N/A
Related issues/external references
See #143
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

[ ] This change is only breaking for integrators, not for external standards or end-users.

[ ] Documentation improvement

PR checklist

[x] I have checked there is no other PR open for the same change.
[x] I have read the Contribution Guidelines.
[x] I grant the project the right to include and distribute the code under the BSD-3-Clause license (and I have the right to grant these rights).
[ ] I have added tests to cover my changes.
[x] I have verified that the code complies with the projects coding standards.
[ ] [Required for new sniffs] I have added XML documentation for the sniff.

Thanks @jrfnl. I have a couple more on their way for tonight (I started with comments that indicated they needed to be the last in the file).

@biinari That's awesome! I may not get to them all tonight, but will review those tomorrow in that case. Thanks for your willingness to help the project!


ImportError: cannot import name 'UniversalSentenceLanguage' from 'whatlies.language'

Hi Team! Facing the following error on UniversalSentenceLanguage import:
import pathlib
import numpy as np
from whatlies.language import CountVectorLanguage, UniversalSentenceLanguage, BytePairLanguage, SentenceTFMLanguage
from whatlies.transformers import Pca, Umap
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-4-6ecfd2b2c4c1> in <module>
      1 import pathlib
      2 import numpy as np
----> 3 from whatlies.language import CountVectorLanguage, UniversalSentenceLanguage, BytePairLanguage, SentenceTFMLanguage
      4 from whatlies.transformers import Pca, Umap

ImportError: cannot import name 'UniversalSentenceLanguage' from 'whatlies.language' (c:\users\rsiddiqui\anaconda3\envs\newenv\lib\site-packages\whatlies\language\__init__.py)

Edit: above error occurs on whatlies version 0.4.3 - if I upgrade to 0.5.7, this error goes away, but a new one arises, which does not get solved with proposed remedy:
lang_cv  = CountVectorLanguage(10)
lang_use = UniversalSentenceLanguage("large")
lang_bp  = BytePairLanguage("en", dim=300, vs=200_000)
lang_brt = SentenceTFMLanguage('distilbert-base-nli-stsb-mean-tokens')
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-2-77ddbd9218cf> in <module>
      1 lang_cv  = CountVectorLanguage(10)
----> 2 lang_use = UniversalSentenceLanguage("large")
      3 lang_bp  = BytePairLanguage("en", dim=300, vs=200_000)
      4 lang_brt = SentenceTFMLanguage('distilbert-base-nli-stsb-mean-tokens')

c:\users\rsiddiqui\anaconda3\envs\newenv\lib\site-packages\whatlies\error.py in __call__(self, *args, **kwargs)
     20
     21     def __call__(self, *args, **kwargs):
---> 22         raise ModuleNotFoundError(self.msg)

ModuleNotFoundError: In order to use UniversalSentenceLanguage you'll need to install via;

pip install whatlies[tfhub]

See installation guide here: https://rasahq.github.io/whatlies/#installation.

Resolved by installing tensorflow==2.4.0 and tensorflow_text along with it. Closing this issue.


Form patterns - single and multiple questions

Originally raised by Susannah as part of the suggestions on Umbraco components, I thought this might be better suited to a new pattern?   I think we could look at the best way of doing it, not sure about h1s inside legends?
Copied from Ticket:
Headings: If you are asking just one question per page as recommended, you can set the contents of the  as the page heading. This is good practice as it means that users of screen readers will only hear the contents once. Read more about why and how to set legends as headings at https://design-system.service.gov.uk/get-started/labels-legends-headings/
If you’re asking more than one question on the page, do not set the contents of the  as the page heading. Read more about asking multiple questions on question pages: https://design-system.service.gov.uk/patterns/question-pages/#asking-multiple-questions-on-a-page

Sounds good.
H1s in legends is fine (valid html5). In the ui-core PRO component the question was an h1, if the fields required a fieldset we put the h1 in a legend otherwise just an h1 in a form.


fix: ESLint issues in bobuilder-ai-orchestrator

Addresses # 4684
#minor
Description
This PR fixes the new errors and warnings incoming in the latest version of ESLint packages in bobuilder-ai-orchestrator.
Specific Changes

Removed eslint.config.cjs file.
Formatted with required spaces and comment lines required by ESLint.
Added some skips in rules that can be applied to the current logic.

Testing
The following image shows the lint command running without problems in botbuilder-ai-orchestrator.

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

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 84.355%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-ai-orchestrator/src/orchestratorRecognizer.ts
1
68.83%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.3%

Covered Lines:
20433

Relevant Lines:
23064

💛  - Coveralls


Onboarding of new validateers

With the addition of the direct invocation api, a worker's state can't be constructed with onchain data only. Hence, when a new worker onboards, it needs to fetch data from other workers. Two obvious solutions exists.

A worker fetches the current state from another worker, which consists of both, the chain_relay_db and the STF's state. This would be the simple onboarding, which leverages the trusted nature of TEEs.
A worker syncs layer 1 blocks in the chain relay, whenever it observes a sidechain block confirmation extrinsic. It fetches the corresponding sidechain block from another worker. (A worker could also fetch sidechain blocks batch-wise in beforehand.)

Eventually, we should implement both ways:

Because it is simple and essentially provides all the trust regardless.
Because we always need to be able to reconstruct the history (even if it is only for auditability).

@haerdib has already implemented state snapshotting AFAIR. This way new workes have to request the latest state from an active validator (I suggest not to use the term worker for sidechain validators. Maybe validaTEErs? 8-P )

I believe it should be your (1) only, not (2). Scanning the chain takes very long and it isn't necessary in our threat model

Yes, I agree. Came to the same conclusion yesterday after some thinking.

connected to #345

@mullefel can you please review this task and think about edge cases to tackle?

Here we briefly discussed two different syncs depending on state recency:

https://github.com/integritee-network/worker/issues/214

Please let me know if I got something wrong, or missed an important part. Much appreciated 🙏
Feature Requirements
Based on the proposed solution (1), I identified some feature requirements:

State syncing using a similar channel, or even the same, as sharing of state encryption key (i.e. RPC)
As a newly spawned validateer:

determine if there is a discrepancy in state, and need to fetch an updated state. This is done using the side chain (?)
determine from which 'mature' validateer we fetch the state. If validateers registered in the pallet teerex are guaranteed to be 'mature', i.e. up-to-date, we can choose randomly from those?
As discovered in the edge cases, we might need multiple ways of syncing, to mitigate some of the latency issues. So it might make sense to be able to:

sync state directly, fetching a full state
sync state incrementally from side-chain blocks

Edge Cases
Scenario multiple new validateers spawned at once

setup: We have 1 up-to-date validateer and we launch multiple new validateers at once.
Need to make sure these new validateers sync a) from the correct up-to-date validateer (and not one of the other newly spawned ones) and b) that multiple concurrent requests for state updates can be processed in a consistent manner and don't deadlock.

Scenario validateer is offline for a short period

starting point: Two validateers V1 and V2 are running and are up-to-date. Validateer V2 goes offline for a short while and comes back online, during which the state in V1 changes. At the same time as V2 comes online again, a new validateer V3 is spawned that needs to sync its state.
Ensure that newly spawned validateer V3 syncs correct state. Either:

Guarantee that V3 syncs from the unimpaired V1, because we have some way of knowing that only V1 but not V2 has the latest state.
Multi-stage sync, where we sync from V2, but then notice we are missing some state, and then sync again, possibly from V1

Scenario onboarding latency

setup: We have an up-to-date validateer V1 from which a newly spawned validateer V2 syncs. The state to sync is large, so it takes enough time that in the meantime new state updates are applied to V1. So at the end of the sync, V2 is still not fully up-to-date.
Similar to the scenario above, we probably need multi-stage syncing, to ensure we get to a fully up-to-date state in V2 (similar issue to #214)

Continuing my work of commenting on everything: I think the feature requirement sound about correct IMO.
The Edge cases I don't think we need such strict requirements. As long as the fetched state is not too old, the "missed" updates can be fetched by requesting the stored sidechainblocks. So we have somewhat of a "larger" timewindow:
As an example in scenario Scenario validateer is offline for a short period: In case V3 gets it state from V2 (which is outdated, but not very) and notices that it's missing some blocks, it can just request these missing blocks from V1. This should not require an additional implementation, because every validateer needs to be able to rerequest sidechain blocks in case it missed some due to whatever reason.
What need to keep in mind is that sidechain blocks will be deleted after some (configurable) time. So in case the state syncing latency gets too high, it might really end up in a scenario where a validateer syncs for some time and after it has synced, it only the nnotices that its state is too oudated and it can not refetch the missing blocks.

Please split all these learnings and decisions into tasks and schedule them. All in one epic


cannot read property 'replace' of undefined

I am attempting to write a client-side test that involves some database manipulation. I am able to run a db.find({}) command without problems, but when I try to interact with the db in any way (remove, insert etc) I get the error: "Cannot read property 'replace' of undefined". The Meteor Guide mentioned to use your package when testing the db on the client side, but so far it doesn't seem to be making a difference for me.

Are you able to share some of your code showing how you're using this package? For a few small usage examples, check out the tests.

Absolutely:
I have a collection call "SchedShifts" that I need to test - I need to remove everything in the collection before each test, and insert some data. In my beforeEach function, when the 'SchedShifts.remove({}) line runs, I get the "Cannot read property 'replace' of undefined" error.
I have tried stubbing the collection outside of the beforeEach function and some other places in the test, but so far have been unsuccessful
`
    beforeEach(function(){
        StubCollections.add([SchedShifts])
        StubCollections.stub()

        SchedShifts.remove({})
        SchedShifts.insert({data})
        console.log(SchedShifts.find({}).count())

});

`

You should be able to remove the SchedShifts.remove({}) line; When you call StubCollections.stub() your SchedShifts.find({}).count() should be 0, before adding any data in (so no need to remove anything).

While that is true, I have attempted this without the remove() as my DB should be empty when the tests run for the first time; regardless, without the remove() my count is still 0 after the insert and the insert causes the "cannot ready property 'replace' of undefined" error to appear in my browser console.

Could the aldeed:simple-schema package cause any issues? I just noticed your latest commit.

@jaskinn unfortunately the code that i'm speaking about here is for a different application than the fantasy football project that i've been committing to on github. You bring up and interesting point, however, as the simple-schema package is more strictly used on the app in question. @hwillson - do you know of any conflicts?

Hmm - I just accepted PR #9 which relates to simple-schema. I'm out of time today unfortunately, but I'll run some more thorough tests tomorrow morning. I just published version 1.0.2 with the latest simple-schema changes; maybe update and try that version out. Otherwise, I'll post back with more details tomorrow.

Will do, I'll let you know how it goes! Thanks!

@tonymckendry Looking at the following:
SchedShifts.insert({data})

Can you give me an example of what data looks like, along with your schema config?

Certainly - I was just about to update and try the new version out
Data:
    SchedShifts.insert({
      start: faker.date.past(),
      finish: faker.date.recent(),
      company: {
        shiftId: faker.random.number(),
        unitId: 34,
        personId: 4,
        jobId: 2
      },
      state: {
        geo_confirmed: false,
        django_has_it: true,
        status: shStatus.delivered
      }
    })

Schema:
`
export const ShiftSchema = new SimpleSchema({
    start: {
        type: Date
    },
    finish: {
        type: Date
    },
    company: {
        type: companySchema
    },
    state: {
        type: StateSchema,
        optional: true
    }
});

let companySchema = new SimpleSchema({
    shiftId: {
        type: Number,
        label: "Shift Id",
        optional: true
    },
    unitId: {
        type: Number,
        label: "Unit Id"
    },
    jobId: {
        type: Number,
        label: "Job Id",
        optional: true
    },
    personId: {
        type: Number,
        label: "Person Id",
        optional: true
    }
});

let StateSchema = new SimpleSchema({
    geo_confirmed: {
        type: Boolean,
        label: "The user was confirmed as being at the unit",
        defaultValue: false
    },
    django_has_it: {
        type: Boolean,
        label: "Is this shift stored on django",
        defaultValue: false
    },
    status: {
      type: Number,
      label: "Status of the shift state",
      defaultValue: 2
    },
    group_time: {
      type: Boolean,
      label: "Shift was generated in group time mode",
      defaultValue: false,
      optional: true
    }
});

`

@hwillson - with the updated package, no difference. Same error

Thanks @tonymckendry - arg, caught in meetings; I'll get back to this shortly. I'll put together a small repro mirroring your data/schema, and see what I can find out. More details shortly.

@hwillson No worries, you're the man for helping me out and being so responsive

Hi @tonymckendry - I've wired up a small attempted repro here, using your example schema and data. Everything seems to work properly, so I could be missing something. Take a look and let me know.

Sorry @hwillson I havent gotten a chance to review this yet, but am really going to try tomorrow.   StubCollections uses this code:
new collection.constructor(null, options)
I'll change my own subclass to work better with the standard (name, [options]) signature.

Sorry for the delay, I started looking at this again today and began by changing over to the practicalmeteor:mocha package rather than the dispatch:mocha-browser package to try to emulate exactly what you were doing in that example repo @hwillson -but my client tests seem to be running on the server side for some reason and I have yet to figure out why.

Hi guys,
for me the simple-schema will NOT work, when using the transform-approach for collection-helpers, like:
TheCollection = new Mongo.Collection('thecollection', {
  transform(doc) {
    return new CollectionSuper(doc)
  },
})
TheCollection.attachSchema(TheSchema);

Is there any workaround? I'd love to avoid changing the whole collection-helper-approach in a large project to be able to use this. ?

@tonymckendry Have you tried upgrading to v1.0.6?
I encountered the same error and it turns out it was masking an issue with the meteor sinon package that stub-collections used in v1.0.3. The issue is from an old version of sinonjs: https://github.com/sinonjs/sinon/issues/710


Possible finddom node fix

@zackify That looks like it should work. I'll want to definitely pull it down and test it before merging. If that isn't calculating the size on the proper element, it's going to cause all sorts of problems.

It's the same as doing a finddomnode on the component. A ref returns it's
dom node. Just make sure tho!!
On Mon, Mar 20, 2017 at 16:41 Darin Haener<EMAIL_ADDRESS>wrote:

@zackify https://github.com/zackify That looks like it should work.
I'll want to definitely pull it down and test it before merging. If that
isn't calculating the size on the proper element, it's going to cause all
sorts of problems.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Legitcode/scheduler/pull/21#issuecomment-287891039,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbacOs4_0v3ud4FBu3azySsCO8KoW6Qks5rnuR4gaJpZM4Mi-id
.

@zackify Yeah but that cell is wrapped in a HOC, so......not 100% certain we will be referencing the same DOM node.


ARROW-9191: [Rust] Do not panic when milliseconds is less than zero as chrono can handle…

… these formats

I've merged this, I've been having computer issues so haven't been able to do much work


It's not possible to return Vec<T> if T only implements IntoWasmAbi

Describe the Bug
It is not possible to return Vec<T: IntoWasmAbi>, but it is allowed to return just T.
Steps to Reproduce
Given this code
use serde::{Deserialize, Serialize};
use tsify::Tsify;
use wasm_bindgen::prelude::wasm_bindgen;

#[wasm_bindgen(js_name = TestClass)]
pub struct TestClass;

#[wasm_bindgen(js_class = TestClass)]
impl TestClass {
    #[wasm_bindgen]
    pub fn get_dummy_vec(&self) -> Vec<Dummy> {
        vec![Dummy {
            data: "abc".to_string(),
        }]
    }
}

#[derive(Tsify, Serialize, Deserialize)]
#[tsify(into_wasm_abi, from_wasm_abi)]
pub struct Dummy {
    data: String,
}

error[E0277]: the trait bound `Dummy: JsObject` is not satisfied
   --> src/test_bindgen.rs:8:1
    |
8   | #[wasm_bindgen(js_class = TestClass)]
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `JsObject` is not implemented for `Dummy`
    |
    = help: the following other types implement trait `JsObject`:
              Array
              ArrayBuffer
            and 71 others
    = note: required for `Dummy` to implement `WasmDescribeVector`
    = note: required for `Box<[Dummy]>` to implement `wasm_bindgen::describe::WasmDescribe`
    = note: 1 redundant requirement hidden
    = note: required for `Vec<Dummy>` to implement `wasm_bindgen::describe::WasmDescribe`
note: required by a bound in `ReturnWasmAbi`
   --> /Users/bes/.cargo/registry/src/index.crates.io-6f17d22bba15001f/wasm-bindgen-0.2.90/src/convert/traits.rs:177:26
    |
177 | pub trait ReturnWasmAbi: WasmDescribe {
    |                          ^^^^^^^^^^^^ required by this bound in `ReturnWasmAbi`
    = note: this error originates in the attribute macro `wasm_bindgen::prelude::__wasm_bindgen_class_marker` (in Nightly builds, run with -Z macro-backtrace for more info)

But, if we wrap the Vec in something that itself implements IntoWasmAbi everything works:
use serde::{Deserialize, Serialize};
use tsify::Tsify;
use wasm_bindgen::prelude::wasm_bindgen;

#[wasm_bindgen(js_name = TestClass)]
pub struct TestClass;

#[wasm_bindgen(js_class = TestClass)]
impl TestClass {
    // #[wasm_bindgen]
    // pub fn get_dummy_vec(&self) -> Vec<Dummy> {
    //     vec![Dummy {
    //         data: "abc".to_string(),
    //     }]
    // }
    //
    #[wasm_bindgen]
    pub fn get_dummy_collection(&self) -> DummyCollection {
        let dummies = vec![Dummy {
            data: "abc".to_string(),
        }];
        DummyCollection(dummies)
    }
}

#[derive(Tsify, Serialize, Deserialize)]
#[tsify(into_wasm_abi, from_wasm_abi)]
pub struct Dummy {
    data: String,
}

#[derive(Tsify, Serialize, Deserialize)]
#[tsify(from_wasm_abi, into_wasm_abi)]
pub struct DummyCollection(Vec<Dummy>);

Expected Behavior
I expect it to be possible to return Vec<T> where T: IntoWasmAbi.
Actual Behavior
It isn't possible to return Vec<T> where T: IntoWasmAbi, but it is perfectly possible to return the Vec when a wrapper type is added.
Additional Context
See also

https://github.com/rustwasm/wasm-bindgen/issues/111
https://github.com/rustwasm/wasm-bindgen/pull/3554
https://github.com/rustwasm/wasm-bindgen/discussions/3697
https://github.com/madonoharu/tsify/issues/49

I realized too late that tsify has been inactive for about 9 months. There is a maintained fork here, but unfortunately no issue tracker (yet?).

Returning a Vec requires VectorIntoWasmAbi.
FWIW the whole wasm_bindgen::convert module is unstable.
Let me know if this doesn't solve your issue!


[Deep Learning] Add Text Classification Scenario

ML.NET now supports Text Classification powered by TorchSharp. Add Text Classification as one of the supported scenarios in Model Builder.

Supported Training Environments

Local CPU
Local GPU
Azure

Supported Data Formats

File
SQL

@luisquintanilla to add details / priority.

@zewditu here are additional details about this scenario. Let me know if you have any questions.
UI
Scenario

Add a new tile for Text Classification as one of the supported scenarios

Tile Details:

Title: Text Classification
Description: Assign a label or category to raw text. e.g. sentiment analysis on customer reviews
Icon: TBD
Environment Tag: Local

Update Data classification description:

Current: Classify tabular data into 2+ categories, e.g. predict if comments are positive or negative sentiments.
Update: Classify tabular data into 2+ categories, e.g. categorize flowers based on petal size and width measurements.

Environment

Local CPU
Local GPU

Data

Data Source Type:

File
SQL

Column to Predict (Dropdown)
Text Column (Dropdown)

Train
Same as other scenarios
Evaluate
Same as other scenarios
Consume
Same as other scenarios
Requirements
Environment

Users should be able to train locally on CPU
Users should be able to train locally on GPU

Data
Data for this scenario is expected to be tabular. This includes the already supported File and SQL formats like other scenarios. Like other scenarios, users should be able to select a single column as the label column. However, one distinction for this scenario is, users can only choose a single column as the feature. This column should be the raw text containing. The Yelp dataset is a good example for this scenario.
Dependencies / Non-Goals

For the initial implementation, since the Text Classification API is not currently implemented in the ML.NET CLI, it's not a priority to enable training on Azure as an environment.


Exclude tests failure on Max_XL

The first time running on Max_XL and failed
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

related with #830


Links, documentation and repos are all over the place

I was trying to find some documentation, but the stuff is complete mess.
The links from readme.md of https://github.com/boonproject/boon point to http://richardhightower.github.io/site/Boon/Welcome.html as the home page, but that page
... links back to different github repo (https://github.com/RichardHightower/boon)
... when clicking getting started goes to 'new page' editor
... when clicking docs goes emty wiki at https://github.com/RichardHightower/boon/wiki
Some links point to one more (third!) repo at https://github.com/advantageous/boon, some links go blog posts in various sites.. info in the blog posts partially outdated / doesn't work.. the source doesn't have any javadocs...
Yes yes i understand that this is best effort open source project and work in progress etc, but you really aren't making things easy for users...

It is. :(
On Thu, Jun 4, 2015 at 9:43 PM, henrify<EMAIL_ADDRESS>wrote:

I was trying to find some documentation, but the stuff is complete mess.
The links from readme.md of https://github.com/boonproject/boon point to
http://richardhightower.github.io/site/Boon/Welcome.html as the home
page, but that page
... links back to different github repo (
https://github.com/RichardHightower/boon)
... when clicking getting started goes to 'new page' editor
... when clicking docs goes emty wiki at
https://github.com/RichardHightower/boon/wiki
Some links point to one more (third!) repo at
https://github.com/advantageous/boon, some links go blog posts in various
sites.. info in the blog posts partially outdated / doesn't work.. the
source doesn't have any javadocs...
Yes yes i understand that this is best effort open source project and work
in progress etc, but you really aren't making things easy for users...
—
Reply to this email directly or view it on GitHub
https://github.com/boonproject/boon/issues/322.

--
Rick Hightower
(415) 968-9037
Profile http://www.google.com/profiles/RichardHightower

I agree. The docs are spread all over.
And I have been spending most of my time on qbit, which uses a different (smaller) version of boon.
Guilty.
I do plan on spending most of my effort in finishing qbit and documenting it. Perhaps even writing a book on it.

I started today looking for an efficient JSON library, as I wan to to move away from org.json. I'm considering Boon, Jackson and GSON. It's so hard to find proper information on Boon, that I don't know if I should just skip it and decide between Jackson and GSON. But if Boon would have turned out to be the better pick, it would be sad that it wasn't chosen just because there is not a single, easy-to-find "Document", ie a blog post, which tells people like me what's up with boonproject/boon and advantageous/boon, why these two exist and appear to be worked on in parallel, where the differences reside, and which one should get chosen under which circumstance.
Could you please be so kind and shed some light on it? Like take an hour of your time and explain it in one or two paragraphs, with links to the post from both GitHub projects?
I've seen such good benchmarking results with Boon, but I don't know which one to use and why not the other, so it's kind of sad of not being able to consider using it instead of Jackson/GSON.
Thanks, and have a nice day.

boonproject/boon
On Tue, Aug 11, 2015 at 2:00 PM, Daniel Faust<EMAIL_ADDRESS>wrote:

I started today looking for an efficient JSON library, as I wan to to move
away from org.json. I'm considering Boon, Jackson and GSON. It's so hard to
find proper information on Boon, that I don't know if I should just skip it
and decide between Jackson and GSON. But if Boon would have turned out to
be the better pick, it would be sad that it wasn't chosen just because
there is not a single, easy-to-find "Document", ie a blog post, which tells
people like me what's up with boonproject/boon and advantageous/boon, why
these two exist and appear to be worked on in parallel, where the
differences reside, and which one should get chosen under which
circumstance.
Could you please be so kind and shed some light on it? Like take an hour
of your time and explain it in one or two paragraphs, with links to the
post from both GitHub projects?
I've seen such good benchmarking results with Boon, but I don't know which
one to use and why not the other, so it's kind of sad of not being able to
consider using it instead of Jackson/GSON.
Thanks, and have a nice day.
—
Reply to this email directly or view it on GitHub
https://github.com/boonproject/boon/issues/322#issuecomment-130073802.

--
Rick Hightower
(415) 968-9037
Profile http://www.google.com/profiles/RichardHightower

A recent tutorial is here:
http://tutorials.jenkov.com/java-json/boon-objectmapper.html
On Thu, Aug 13, 2015 at 1:20 AM, Rick Hightower<EMAIL_ADDRESS>wrote:

boonproject/boon
On Tue, Aug 11, 2015 at 2:00 PM, Daniel Faust<EMAIL_ADDRESS>wrote:

I started today looking for an efficient JSON library, as I wan to to
move away from org.json. I'm considering Boon, Jackson and GSON. It's so
hard to find proper information on Boon, that I don't know if I should just
skip it and decide between Jackson and GSON. But if Boon would have turned
out to be the better pick, it would be sad that it wasn't chosen just
because there is not a single, easy-to-find "Document", ie a blog post,
which tells people like me what's up with boonproject/boon and
advantageous/boon, why these two exist and appear to be worked on in
parallel, where the differences reside, and which one should get chosen
under which circumstance.
Could you please be so kind and shed some light on it? Like take an hour
of your time and explain it in one or two paragraphs, with links to the
post from both GitHub projects?
I've seen such good benchmarking results with Boon, but I don't know
which one to use and why not the other, so it's kind of sad of not being
able to consider using it instead of Jackson/GSON.
Thanks, and have a nice day.
—
Reply to this email directly or view it on GitHub
https://github.com/boonproject/boon/issues/322#issuecomment-130073802.

--
Rick Hightower
(415) 968-9037
Profile http://www.google.com/profiles/RichardHightower

--
Rick Hightower
(415) 968-9037
Profile http://www.google.com/profiles/RichardHightower

Ok, thank you. I had seen that tutorial, but some people still think that it's not that important to add a publishing date to their publications, so I couldn't tell if it was outdated, should it have been the case that boonproject/boon got replaced by advantageous/boon.

I see.
Boon is an underpinning of QBit.
I am actively growing QBit.
I did not want to change core Boon which is stable.
I opted to fork boon via advantageous/boon.
Currently most of my time is on QBit core which uses Boon.
On Thu, Aug 13, 2015 at 1:39 AM, Daniel Faust<EMAIL_ADDRESS>wrote:

Ok, thank you. I had seen that tutorial, but some people still think that
it's not that important to add a publishing date to their publications, so
I couldn't tell if it was outdated, should it have been the case that
boonproject/boon got replaced by advantageous/boon.
—
Reply to this email directly or view it on GitHub
https://github.com/boonproject/boon/issues/322#issuecomment-130575340.

--
Rick Hightower
(415) 968-9037
Profile http://www.google.com/profiles/RichardHightower

Ok, it's just that the code in advantageous/boon (or at least the file structure) looked so heavily refactored (and cleaned up) that it gave the appearance that this was your new Boon (I was a bit puzzled when I saw that both project were yours).
But what made me wonder was your not-so-old Google+ post about that tutorial from Jakob, which was refering to boonproject/boon. Then looking at the commits (from where I saw that advantageous/boon was a newer project) I was confused to see that both projects seemed to be somewhat active.
I then thought  advantageous/boon was a rewrite of projectboon since there was no up-to-date information on boonproject (the tutorial wasn't timestamped). This felt like boonproject wouldn't be a good pick compared to Jackson or GSON since the future of it was quite doubtful. Ultimately I opted to skip Boon, until I found this Issue where I decided to add my comment.
Luckily I found out that org.json wasn't the bottleneck, so I'll be delaying that migration to when it's really needed.
Have a good time and good luck with QBit.

advantageous/boon will one day be the new boon.
But the reality is.. I work for money to buy food for five kids.
Also college is not free in the USA and I have a daughter who wants an
education and a career.
Kids these days.
If I had an infinite level of capacity, I would keep refactoring
advantageous/boon,
and it would become the new boon.
Since I only have bandwidth to focus on one thing, I am focusing on QBit at
the moment.
There is code in QBit which I want to move into advantageous/boon
specifically with some rather elegant handling of generics. But then I have
clients who want stuff. And it turns out the stuff they want is actually
very good for QBit, and it is fun to code.
So the agenda of QBit and Boon are often driven as much by what clients
want as it is what I want to do.
Sometimes... through some miracle of the stars being aligned.. the projects
become aligned.
Quite a bit of QBit and Boon were written just because I wanted to write
something like them.
But the larger part was because I had clients who wanted stuff. And the
larger part pays for the rest.
--Rick
On Thu, Aug 13, 2015 at 9:16 AM, Daniel Faust<EMAIL_ADDRESS>wrote:

Ok, it's just that the code in advantageous/boon (or at least the file
structure) looked so heavily refactored (and cleaned up) that it gave the
appearance that this was your new Boon (I was a bit puzzled when I saw that
both project were yours).
But what made me wonder was your not-so-old Google+ post about that
tutorial from Jakob, which was refering to boonproject/boon. Then looking
at the commits (from where I saw that advantageous/boon was a newer
project) I was confused to see that both projects seemed to be somewhat
active.
I then thought advantageous/boon was a rewrite of projectboon since there
was no up-to-date information on boonproject (the tutorial wasn't
timestamped). This felt like boonproject wouldn't be a good pick compared
to Jackson or GSON since the future of it was quite doubtful. Ultimately I
opted to skip Boon, until I found this Issue where I decided to add my
comment.
Luckily I found out that org.json wasn't the bottleneck, so I'll be
delaying that migration to when it's really needed.
Have a good time and good luck with QBit.
—
Reply to this email directly or view it on GitHub
https://github.com/boonproject/boon/issues/322#issuecomment-130746823.

--
Rick Hightower
(415) 968-9037
Profile http://www.google.com/profiles/RichardHightower


create associations among models

A Product can have many OrderItems
An Order belongs to a user, a user can have many Orders.
An Order can have many OrderItems
An OrderItem belongs to a Product


Always cast errors to strings

Without this change, errors tend to print as {}, or whatever the
exported fields on the concrete implementation of the error are.
This code borrowed from logrus.JSONFormatter:
https://github.com/Sirupsen/logrus/commit/8287db793
(It would be nice if logrusly could just use JSONFormatter and call
client.Write instead of client.Send, but they make slightly different
choices for names for the timestamp and message fields, as well as the
capitalization of the level.)

Thanx David!


[Issue]: Add bolierplate to the top of code

Contact Details(optional)
No response
What is the issue you are having?
Add bolierplate
// coding=utf-8

// Copyright [2024] [SkywardAI]
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at

//        http://www.apache.org/licenses/LICENSE-2.0

// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

<code>

Relevant log output
No response

.


More options to STORE

Can You add some features to store?

How often integrations/plugins were downloaded
Ratings by users
Sorting by last modification/rating/nuber of downloads
Screenshots maybe?

This will not be possible without a centralized server that every instance push data to, I would not use an integration that did that.
The developer can add screenshots the the info.md file.
https://custom-components.github.io/hacs/developer/general/#enhance-the-experience


[Bug] Home : giant thumbnail take all the place

Describe the bug
Using default settings or custom settings.
When the css is enabled, there is one giant thumbnail by row in home. No issue in subscription.
Screenshots
When CSS enabled

When disabled

What browser are you using?
Firefox : 115.0.2 (64 bits)
Stylus : 1.5.33
What version of AdashimaaTube are you using?
v2.3.4

Did you install other extensions related to YouTube? If so, try disabling YouTube related extensions. please let me know if the issue still exits.

I tested in Edge too with only this style and stylus extension. The issue is still there. Some variables are not defined in your style.
--ytd-rich-grid-item-margin, --ytd-rich-grid-item-max-width. It could be the problem.


Unable to extract uploader id

Checklist

[x] I'm reporting a broken site support issue
[X] I've verified that I'm running youtube-dl version 2021.12.17
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar bug reports including closed ones
[X] I've read bugs section in FAQ

Verbose log

C:\Users\Ole>C:\Users\Ole\Downloads\youtube-dl.exe -U
youtube-dl is up-to-date (2021.12.17)

C:\Users\Ole>C:\Users\Ole\Downloads\youtube-dl.exe --extract-audio --audio-format mp3 -o "%(title)s.%(ext)s" "https://www.youtube.com/playlist?list=PLeVXhjd6cOIUAeR3NgR8pvPCTezXbWxMT" --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--extract-audio', '--audio-format', 'mp3', '-o', '%(title)s.%(ext)s', 'https://www.youtube.com/playlist?list=PLeVXhjd6cOIUAeR3NgR8pvPCTezXbWxMT', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: none
[debug] Proxy map: {}
[youtube:tab] PLeVXhjd6cOIUAeR3NgR8pvPCTezXbWxMT: Downloading webpage
[download] Downloading playlist: DnB
[youtube:tab] Downloading page 1
[youtube:tab] Downloading page 2
[youtube:tab] playlist DnB: Downloading 251 videos
[download] Downloading video 1 of 251
[youtube] XU1x3o8DbTQ: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\youtube.py", line 1794, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 1012, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

Command is in the log above. Reading the last few instances of this error does not have any result, just crosslinks to other issues where nothing was resolved.

Nonetheless, study #30839 as asked, which actually lists the master issue for this problem, and which is also the top issue when searching the tracker for Unable to extract uploader id -label:duplicate in:title as suggested in #30839.


Do I need to download Tesseract 5.0 in order to fix inword spacing?

I have the same problem described here
https://github.com/tesseract-ocr/tesseract/issues/781
My problem is that that issue was solved it seems to me in Tesseract 5.0 here
https://github.com/tesseract-ocr/tesseract/issues/781#issuecomment-329702881
That was in 2017 and still there is no stable version of Tesseract 5.0, so I'm a little skeptical that  that is the real solution. If that is the real solution then I'm a little worried that installing Tesseract 5.0 on my mac will be too hard since one cannot now install tesseract 5.0 using brew.  Whenever I have to install something manually without brew or pip, it is always an enormous struggle.

The fix was applied years ago and should be available in the latest 4.x version.
If it's not working for you in that version, it won't work in 5.0 either.

Please don't close this thread.  I still have some questions I want to ask but have not yet had time to ask them.

Please respect guidelines for posting issue: use tesseract user forum for asking questions/support


Difference between C++ and Go output

I've been running both the C++ and this version of Murmur3 side by side and i've noticed that in cases where a null byte occurs in the middle of the input data, the C++ version will output a different hash.
Now I guess this is a bug in the C++ implementation, as Go will iterate over the correct array length, and not just stop at a null byte. But, given that the C++ version is the "official" one, should this version implement the bug as a feature?

Well, I doubt there's a bug in the original implementation ;), but I could use a reproducer (with the C++ version you're using).
I would suspect an API misusage given the C/C++ null byte specific signification.
Thanks.


multi-select interactive tag hover NEEDED FOR V9

When hover over the interactive tag the color changes to brand-02, it should be switching to the new class hover-primary which has a hex value of #30588c

Issue moved to carbon-design-system/carbon-components #829 via ZenHub


The constructTransaction endpoint doesn't contain certificates in response

Just checking...

[X] This is a cardano-wallet bug.
[X] I am using the latest cardano-wallet release.
[X] I am using the correct cardano-node version for that release of cardano-wallet.

Version
v2024-11-18
Platform
macOS 14
Installation method
Nix
Network configuration
preview
Context
No response
Description
Steps to Reproduce

Call the constructTransaction endpoint with { vote: 'abstain' } payload
Expected behavior
The response contains a coin_selection.certificates with appropriate certificates.
Actual behavior

no coin_selection.certificates property.
the cbor received under transaction property has the proper certificates.

Again, the description of the bug quite terse. @szymonmaslowski Could you please elaborate on the circumstances? A minimal reproducible test case on a testnet would be the most useful here.

@abailly I can reproduce it no matter the circumstances. The coin_selection.certificates property is just not present in the response.

@paweljakubas your insights would be much appreciated here

we should have it from here https://github.com/cardano-foundation/cardano-wallet/blob/master/lib/wallet/src/Cardano/Wallet/Transaction.hs#L234

I see, so what you mean is that we need to take into account both delegation and voting actions when constructing certificates in the coin selection?

Plus check the presence of certificates somewhere here :
https://github.com/cardano-foundation/cardano-wallet/blob/master/lib/integration/scenarios/Test/Integration/Scenario/API/Shelley/TransactionsNew.hs#L3696
Please remember that we are expecting the following upon voting:

no delegation to stake pool, no voting => both registration and voting certificates
delegating to stake pool, no prior voting=> voting certificate
delegating to stake pool,  voting=> voting certificate

it is already taken into account, but we have not expose it in this place, ie. in response of tx when constructing

Plus check the presence of certificates somewhere here : https://github.com/cardano-foundation/cardano-wallet/blob/master/lib/integration/scenarios/Test/Integration/Scenario/API/Shelley/TransactionsNew.hs#L3696
Please remember that we are expecting the following upon voting:

no delegation to stake pool, no voting => both registration and voting certificates
delegating to stake pool, no prior voting=> voting certificate
delegating to stake pool,  voting=> voting certificate

This looks like a great test to me :)

Also when quitting you should see stake key deregistration certificate

Plus Test.Integration.Scenario.API.Voting might be even better place to put tests imho


fix:pg数据库querySysCementListByUserId执行异常

fix:pg数据库querySysCementListByUserId执行异常
2024-04-21 19:40:30.314 [http-nio-8080-exec-4] ERROR o.jeecg.common.exception.JeecgBootExceptionHandler:78 -
### Error querying database.  Cause: org.postgresql.util.PSQLException: ERROR: operator does not exist: character varying = integer
  Hint: No operator matches the given name and argument types. You might need to add explicit type casts.
  Position: 216
### The error may exist in file [/Users/widder/IdeaProjects/jeecg-boot/jeecg-module-system/jeecg-system-biz/target/classes/org/jeecg/modules/system/mapper/xml/SysAnnouncementMapper.xml]
### The error may involve org.jeecg.modules.system.mapper.SysAnnouncementMapper.querySysCementListByUserId-Inline
### The error occurred while setting parameters
### SQL: SELECT COUNT(*) AS total FROM sys_announcement WHERE send_status = '1' AND del_flag = '0' AND msg_category = CAST(? AS varchar) AND id IN (SELECT annt_id FROM sys_announcement_send WHERE user_id = ? AND read_flag = 0)
### Cause: org.postgresql.util.PSQLException: ERROR: operator does not exist: character varying = integer
  Hint: No operator matches the given name and argument types. You might need to add explicit type casts.
  Position: 216
; bad SQL grammar []; nested exception is org.postgresql.util.PSQLException: ERROR: operator does not exist: character varying = integer
  Hint: No operator matches the given name and argument types. You might need to add explicit type casts.
  Position: 216
org.springframework.jdbc.BadSqlGrammarException:
### Error querying database.  Cause: org.postgresql.util.PSQLException: ERROR: operator does not exist: character varying = integer
  Hint: No operator matches the given name and argument types. You might need to add explicit type casts.
  Position: 216
### The error may exist in file [/Users/widder/IdeaProjects/jeecg-boot/jeecg-module-system/jeecg-system-biz/target/classes/org/jeecg/modules/system/mapper/xml/SysAnnouncementMapper.xml]
### The error may involve org.jeecg.modules.system.mapper.SysAnnouncementMapper.querySysCementListByUserId-Inline
### The error occurred while setting parameters
### SQL: SELECT COUNT(*) AS total FROM sys_announcement WHERE send_status = '1' AND del_flag = '0' AND msg_category = CAST(? AS varchar) AND id IN (SELECT annt_id FROM sys_announcement_send WHERE user_id = ? AND read_flag = 0)
### Cause: org.postgresql.util.PSQLException: ERROR: operator does not exist: character varying = integer
  Hint: No operator matches the given name and argument types. You might need to add explicit type casts.
  Position: 216
; bad SQL grammar []; nested exception is org.postgresql.util.PSQLException: ERROR: operator does not exist: character varying = integer
  Hint: No operator matches the given name and argument types. You might need to add explicit type casts.
  Position: 216

改下数据库类型就行
ALTER TABLE sys_announcement_send
MODIFY COLUMN read_flag int(10) NULL DEFAULT NULL COMMENT '阅读状态（0未读，1已读）' AFTER user_id;


Support React 17 with new lifecycle methods.

To support react 17 in future some development needs to be done.
Migration path:
https://reactjs.org/blog/2018/03/27/update-on-async-rendering.html#gradual-migration-path
"The primary purpose of the upcoming version 16.3 release is to enable open source project maintainers to update their libraries in advance of any deprecation warnings. Those warnings will not be enabled until a future 16.x release."
A polyfill for backwards compatibility for open source maintainers:
https://reactjs.org/blog/2018/03/27/update-on-async-rendering.html#open-source-project-maintainers

Suggestion: Consider wrapping your app with <StrictMode>*
React will now log warnings about deprecated lifecycle methods to the console.

React 16.3 shipped with a component intended to make this process easier.
It lives in React.StrictMode, e.g. you could do

class App extends React.Component {
  render() {
    return (
      <React.StrictMode>
        <App />
      </React.StrictMode>
    )
  }
}

is redux-form compatible with react 16.4?  redux-form is amazing  and i will not change to formix or the new redux form, for very complex form is the only solution, hope the support will keep at least for react 16 versions

And now we have:
index.js:nnnn Warning: Using UNSAFE_componentWillReceiveProps in strict mode is not recommended and may indicate bugs in your code. See https://fb.me/react-async-component-lifecycle-hooks for details.

* Move data fetching code or side effects to componentDidUpdate.
* If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state

Please update the following components: Field, Form(AForm)

Same issue for me.
react v16.13
redux-form v8.3.6

We are also using strict mode in app and having the same error.

I have the same issue
"react": "^17.0.2",
"react-dom": "^17.0.2",
"react-redux": "^7.2.6",
"react-router-dom": "^6.0.2",
"react-scripts": "4.0.3",

Warning: Using UNSAFE_componentWillMount in strict mode is not recommended and may indicate bugs in your code. See https://reactjs.org/link/unsafe-component-lifecycles for details.

Move code with side effects to componentDidMount, and set initial state in the constructor.

Please update the following components: SideEffect(NullComponent)
I have the same issue


feat: support github markdown syntax

Support Github Flavoured Markdown, https://github.com/remarkjs/remark-gfm
Fixes rendering of markdown table on the site.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1240

Relevant Lines:
1724

💛  - Coveralls


Add new rule comment-style

This rule enforces eslint-disable-line and eslint-disable-next-line comments to be line comments instead of block comments.

As requested in https://github.com/mysticatea/eslint-plugin-eslint-comments/pull/53#issuecomment-1762994978 :)


fixed sharing invite on a virtual drive

Description

We fixed sharing invite on a virtual drive
Related fix https://github.com/cs3org/reva/pull/4568
Related Issue

Fixes https://github.com/owncloud/ocis/issues/8495

Motivation and Context

How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[X] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

@rhafer Already reworked. Here is a reva changes https://github.com/cs3org/reva/pull/4568


Add source column to merged context CSVs

Closes #51
This PR adds an additional field to the PrefixExpansion class that can be optionally annotated with the source from which the class was instantiated. This is then used during the Context.combine() function such that when the merged and merged.oak contexts are generated, each record gets annotated with the simple context from which it came.
This PR also runs the ETL pipeline to regenerate the merged prefix maps, now with the sources annotated in the merged files. The new source column does not affect reading in any way.
Interestingly, this sheds some light on #49 - now we know that the WIKIDATA prefix comes from the Bioregistry and wd from Prefix.cc.

@sierra-moxon please let me know if there's anything you'd like changed here / what the way forward is

@sierra-moxon can we please merge this? It will also solve @glass-ships' problem in #55


Very light on iOS Device Debugging

I can get VS for Mac to deploy to a simulator but I cannot get it to deploy or even show my usb connected iphone in the list of deployment targets. I am able to deploy to the iphone using XCode so its clearly a problem with VS for Mac configuration. But there is no documentation on how to do this!

Hi @markti, this documentation is covered in the Xamarin.iOS documentation, which you find here: https://docs.microsoft.com/en-us/xamarin/ios
Make sure that the device is properly provisioned to deploy to a device: https://docs.microsoft.com/en-us/xamarin/ios/get-started/installation/device-provisioning/
You can also use Wireless deployment for Visual Studio for Mac: https://docs.microsoft.com/en-us/xamarin/ios/deploy-test/wireless-deployment?tabs=macos
Let me know if you're still having issues.


Clean architecture

After reading about the clean architecture for the project structure, I find this interesting and something I would like to incorporate into the project.
Without having read the book, the implementation will be based on this article: https://www.c-sharpcorner.com/article/clean-architecture-in-asp-net-core-web-api/

This issue will be closed with the push of the grocerylist branch.