[Feature Request] Make restart_backend from pyasic available

I use hass-miner mainly to control my Miner with Braiins Firmware (BOS) to make use of my excess Solar Energy, which works great so far. Thanks for the great ha integration.
My Problem is as follows:
I monitor the hashrate and pause and resume the miner if the hashrate drops below e certain level.
Sometime pause and resume does not restart the mining process, when the pool is "dead" according to BOS:

To fix this situation, a restart of bosminer is required, but that is currently not possible via hass-miner. It would be nice to make restart_backend() from pyasic available as a ha service for example, to fix this problem (https://docs.pyasic.org/en/latest/miners/functions/#pyasic.miners.base.MinerProtocol.restart_backend).
Additionally, having the pool status available in HA would be awesome, but also cumbersome?

I like the restart backend idea, I'm not 100% sure how that would work regarding home assistants UI, but I assume it would be pretty simple.
Regarding the pool status, there is currently a Summer of Bitcoin project going on that is adding pool metrics to pyasic, so once that is done it will be a lot easier to show the active pool.  This is a bit more complicated because there can be multiple active pools (in the case of quota splitting between groups), but I think it could be displayed as Active Pool(s): 1, 4

I'm not 100% sure how that would work regarding home assistants UI, but I assume it would be pretty simple.

My suggesten would be a service like miner.restart_backend , because HA UI still has no momentary button. The service could be called by automation or script (button).

This is essentially what I'm attempting, but sadly the dev docs for Hass are terrible.  I think I have something close to working, but it will have to take in an IP address, rather than allowing you to select a device.

Done!  Try out v1.1.10.

Awesome, thx a lot! That was fast.
I will test it today and report here.
readme.md should to updated to include the new service calls.

Will do.  Gonna leave this open for the pools request.

Hm, sadly it does not work as expected:

reboot does trigger a ssh session to the miner ip, but does not reboot the miner. Executing /sbin/reboot manually via ssh does work.
restart_backend does trigger a ssh session to the miner ip, but does not restart the bosminer. Executing/etc/init.d/bosminer restart manually via ssh does work.

I did not find a way to capture the commands send via ssh from HA / pyasic. All I can see is the incoming ssh connection with tcpdump on the miner from HA.
Tested with Braiins OS+ Version: 2022-09-27-0-26ba61b9-22.08.1-plus on Antminer S9.

That's really weird, I was able to get it working with my own S9, same firmware version and everything.  Is it possible the SSH connection is failing?  Did you change the SSH password or something?  Usually it should login with root no password...

Yes, changed the root password.

Try removing and re-adding the miner.  It should prompt you for an SSH user and password, try changing it there...

Readded the miner, entered ssh password in HA config flow. SSH connection still did not work.
# tail -f /var/log/syslog
Fri Jun 21 23:07:22 2024 authpriv.warn dropbear[3704]: Bad password attempt for 'root' from <IP_ADDRESS>:56266
Fri Jun 21 23:07:22 2024 authpriv.info dropbear[3704]: Exit before auth from <<IP_ADDRESS>:56266>: (user 'root', 1 fails): Exited normally

Changed root to default password "root". Now it works.
Looks like the ssh password from the config flow is not used, instead pyasic probably falls back to the default password "root"?

Interesting.  That will almost certainly be an issue with pyasic, lemme do some testing.

Yes, seems to be a bug in current pyasic. I have tested it with pyasic 0.57.6 again Braiins OS with non default password. With default password it works.

BTW, you forgot to update custom_components/miner/manifest.json for 1.1.10 release.

Interestingly, the error you sent is not SSH related, it's web related.  If you comment out the web password setting, does that work?  It's possible the LUCI endpoints just use the default info, since technically they were removed a few versions before latest for the S9, but there is a special case where they return data manually.

You are right, I forgot.
bosminer_restart is done via ssh. Data is gathered via LUCI API.

Interestingly, the error you sent is not SSH related, it's web related. If you comment out the web password setting, does that work? It's possible the LUCI endpoints just use the default info, since technically they were removed a few versions before latest for the S9, but there is a special case where they return data manually.

I have only tested to get data via LUCI API, not ssh.
If the default password root is set on the miner firmware and settings.update("default_bosminer_web_password", "changeme") is commented out, LUCI API query works. If I change the password of user root on the miner to changeme, LUCI query fails with APIError.

I am not able to create a quick test for restart_backend(), as I am not very deep into python.
import asyncio
from pyasic import get_miner

#settings.update("default_bosminer_ssh_password", "changeme")

async def do_restart_backend():
    miner = await get_miner("<IP_ADDRESS>")

    # call control function
    result = await miner.restart_backend()

if __name__ == "__main__":
    asyncio.run(do_restart_backend())

Error
Traceback (most recent call last):
  File "z:\xxx\braiinsos\pyasic-test-03.py", line 13, in <module>
    asyncio.run(do_restart_backend())
  File "C:\Users\xxx\AppData\Local\Programs\Python\Python311\Lib\asyncio\runners.py", line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "C:\Users\xxx\AppData\Local\Programs\Python\Python311\Lib\asyncio\runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\xxx\AppData\Local\Programs\Python\Python311\Lib\asyncio\base_events.py", line 654, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "z:\xxx\braiinsos\pyasic-test-03.py", line 10, in do_restart_backend
    result = await miner.restart_backend()

Can you help me?

I see nothing wrong with that?  I think I understand what the issue is though.  First thing I should do is fix the LUCI password issues, I wasn't aware they did that...

Looks like there are still some issue left with hass-miner 1.1.12
I have setup two miner in a test ha container:

Neither miner: reboot or miner: restart_backend service let me select any one miner either as a device or entitiy:

Looks like the filter for the service does not work as expected or I am doing it wrong?
It did work with hass-miner 1.1.11, because one had to specify the ip address of the miner instead of selecting a device.

Ok, yeah. I'm not super familiar with the internals of HASS, so I can try to figure out how to do that.

Neither miner: reboot or miner: restart_backend service let me select any one miner either as a device or entitiy:

My PR should fix the service call target filter.
The Error with the Service call itself remains. I get the following error when executing miner: restart_backend with a miner device selected:
Traceback (most recent call last):
File "/usr/local/lib/python3.12/site-packages/homeassistant/components/websocket_api/decorators.py", line 28, in _handle_async_response
await func(hass, connection, msg)
File "/usr/local/lib/python3.12/site-packages/homeassistant/components/websocket_api/commands.py", line 793, in handle_execute_script
script_result = await script_obj.async_run(
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/script.py", line 1770, in async_run
return await asyncio.shield(create_eager_task(run.async_run()))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/script.py", line 465, in async_run
await self._async_step(log_exceptions=False)
File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/script.py", line 529, in _async_step
self._handle_exception(
File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/script.py", line 559, in _handle_exception
raise exception
File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/script.py", line 527, in _async_step
await getattr(self, handler)()
File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/script.py", line 764, in _async_call_service_step
response_data = await self._async_run_long_action(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/homeassistant/helpers/script.py", line 727, in _async_run_long_action
return await long_task
^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/homeassistant/core.py", line 2731, in async_call
response_data = await coro
^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/homeassistant/core.py", line 2774, in _execute_service
return await target(service_call)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/miner/services.py", line 36, in reboot
miner = await get_miner(call)
^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/miner/services.py", line 28, in get_miner
if miner_id is None or miner_id not in miners:
^^^^^^^^^^^^^^^^^^^^^^
TypeError: unhashable type: 'list'

Should be fixed by implementing the changes from your PR (along with my fix for the above) in v1.1.14


backgrid.d.ts: added some class and exported the module

backgrid/backgrid.d.ts
to author(@jlujan). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

The actual changes look fine.


Fresh tfs clone: The path is already mapped in workspace

Hi. I try to make a fresh clone of a tfs repo and get following error:
$ git tfs clone http://vstfserver:8080/tfs/defaultcollection $/Petter.Letter/Portal
Initialized empty Git repository in c:/Users/lubosz.sarnecki/workspace/git/Portal/.git/
C3876 = 84555cfabef2b40f6d6c5d935a611bf32ba84b4c
...
C11023 = 08616af83ba617d5f8f90feeb498c833c6a268e8
The path c:\Users\lubosz.sarnecki\workspace\git\Portal\.git\tfs\default\workspace is already mapped in workspace git-tfs-755c038d-527a-4eb6-b623-639209a092ae.
You may be able to resolve this problem.
- Run 'git tfs cleanup-workspaces' to remove the workspace.

When i try the recommendation, it fails:
$ git tfs cleanup-workspaces
StructureMap Exception Code:  207
Internal exception while creating Instance 'b2d9da19-c914-4e22-abdf-d88389ecbc2e' of PluginType Sep.Git.Tfs.Core.IGitRepository.  Check the inner exception for more details.
Path '.git' doesn't point at a valid Git repository or workdir.

Using v0.19.2 on Windows 8 with VS Team Explorer 2012
Edit: The error did not occur when I tried it a second time, with Visual Studio turned off.

I get this issue 100% of the time cloning a large TFS repo. Issued somewhat sloppy pull request that solves issue for me.

could you describe your "somewhat sloppy pull request" that solved the issue for you? I am still stuck with a large TFS repo that consistently fails on cloning

@aroden-salesforce @sbarne3
I also have problem with clone for my HUGE TFS repo. (it has many branches and full trunk, source code could be 30G or more, it has a big 3rdparty libs included. Even without 3rdparty, the source code could easily be more than 10G). I tried to clone the trunk for many times, it took me several days every time, and then terminated because of all different kinds of reasons, e.g. network connection issue, server down issue, or client machine down issue, then I lost all I got. even after the break, I tried to use git tfs fetch, and have the same issue, takes forever to finish. I don't know why it is so slow. If I check them out with TFS, it only takes me about 1 hour or couple hours. But with git tfs never have success.
Is it possible, that we checkout the source tree all together with TFS, and then calculate the hash locally. If it can be done, then it'll be much safer, I believe.

Confusing…. You shouldn’t be losing work. Sounds like a very different issue, consider getting more precise steps to recreate and opening a new issue.
Could some of your issues have some solutions in the manuals. First I’d suggest init then pull. Server down times and network timeout should not impact you. Resuming worked great for me by just issuing the pull again, but do not ctrl-c the process as they might leave the git repo in an un-recoverable position.
To have full history all changesets have to have a hash calculated and yes takes a long time. Doing the hash only once like your suggesting is quick-clone and is already supported?
From: nogains-nopains<EMAIL_ADDRESS>Sent: Tuesday, February 10, 2015 1:56 PM
To: git-tfs/git-tfs
Cc: Andrew Roden
Subject: Re: [git-tfs] Fresh tfs clone: The path is already mapped in workspace (#570)
@aroden-salesforcehttps://github.com/aroden-salesforce @sbarne3https://github.com/sbarne3
I also have problem with clone for my HUGE TFS repo. (it has many branches and full trunk, source code could be 30G or more, it has a big 3rdparty libs included. Even without 3rdparty, the source code could easily be more than 10G). I tried to clone the trunk for many times, it took me several days every time, and then terminated because of all different kinds of reasons, e.g. network connection issue, server down issue, or client machine down issue, then I lost all I got. even after the break, I tried to use git tfs fetch, and have the same issue, takes forever to finish. I don't know why it is so slow. If I check them out with TFS, it only takes me about 1 hour or couple hours. But with git tfs never have success.
Is it possible, that we checkout the source tree all together with TFS, and then calculate the hash locally. If it can be done, then it'll be much safer, I believe.
—
Reply to this email directly or view it on GitHubhttps://github.com/git-tfs/git-tfs/issues/570#issuecomment-73759563.

Yes. It did delete all my checked out stuff in the folder .git~w, actually it deleted the whole folder when it failed in the middle and then I tried to use git tfs fetch, complains about workspaces already used by ...., suggest to use git tfs cleanup-workspace. then all gone. Quick-clone work for the trunk too? I tried quick-clone to a branch, it does work, but has no history information, so sometime cannot unshelve a shared set from other TFS users. Thanks.

Solved with #717


[Feat] Support raw strings in Embed.footer

Describe the bug
When you set the footer in the embed class you get an erorr.
To Reproduce
The code you used to produce this bug, must be in a code-block:
@dis.message_command("embed")
async def embed(ctx: dis.MessageContext):
    await ctx.send(embed=dis.Embed(title="test", description="test", footer="test"))

Expected behaviour
Send the embed
Traceback
Traceback (most recent call last):
  File "D:\Dis-Snek\dis_snek\client\client.py", line 1418, in _dispatch_msg_commands
    await self._run_message_command(command, context)
  File "D:\Dis-Snek\dis_snek\client\client.py", line 1289, in _run_message_command
    return await command(ctx)
  File "D:\Dis-Snek\dis_snek\models\snek\command.py", line 102, in __call__
    await self.call_callback(self.callback, context)
  File "D:\Dis-Snek\dis_snek\models\snek\command.py", line 146, in call_callback
    return await callback()
  File "D:\Dis-Snek\bot.py", line 10, in embed
    await ctx.send(embed=dis.Embed(title="test", description="test", footer="test"))
  File "<attrs generated init dis_snek.models.discord.embed.Embed>", line 16, in __init__
    _setattr('footer', __attr_converter_footer(footer))
  File "D:\Dis-Snek\dis_snek\client\utils\converters.py", line 56, in optional_converter
    return converter(val)
  File "D:\Dis-Snek\dis_snek\client\mixins\serialization.py", line 67, in from_dict
    return cls(**cls._filter_kwargs(data, cls._get_init_keys()))
  File "D:\Dis-Snek\dis_snek\client\mixins\serialization.py", line 36, in _filter_kwargs
    return {k: v for k, v in kwargs_dict.items() if k in keys}
AttributeError: 'str' object has no attribute 'items

LOL IM SO DUMB
footer has its own class cause of the icon thing XD
I will leave now

Pretend this is spoiler message
||I was gonna say something like this but I was too embarrassed.||


Add XDG home directories and variables

This script will only add the variables and directories that would default into the home directory. More specifically:

$XDG_CACHE_HOME (default: "$HOME/.cache"): user-specific non-essential data files.
$XDG_CONFIG_HOME (default: "$HOME/.config"): user-specific configuration files.
$XDG_DATA_HOME (default: "$HOME/.local/share"): user-specific data files.

Operation: For each variable ...

If the variable exists, check if the directory specified would be in the home directory.

If the directory specified would be in the home directory, check if it exists.

If the directory exists, then do nothing. We are done.
If the directory specified does not exist, then create it.

If the directory specified would be outside of the home directory, then do nothing. We are done.

If the variable does not exist, then set it a its default and check if that directory exists.

If the directory exists, then do nothing. We are done.
If the directory specified does not exist, then create it.

Reference:
XDG Base Directory Specification

Addressed in commit 13932b7e89092e4d74a91ffca18947a64fc04641.


[do not merge] test image-spec with embedded Platform

Testing the "embed_platform" branch from: https://github.com/opencontainers/image-spec/pull/949
Relates to https://github.com/containerd/containerd/pull/7376#discussion_r966193773

Ah, yes, some changes needed if struct literals are used;
# github.com/containerd/containerd/integration/client [github.com/containerd/containerd/integration/client.test]
Error: .\import_test.go:386:3: unknown field 'OS' in struct literal of type "github.com/opencontainers/image-spec/specs-go/v1".Image
Error: .\import_test.go:387:3: unknown field 'Architecture' in struct literal of type "github.com/opencontainers/image-spec/specs-go/v1".Image

I've rolled the updated version of this change into #7376, so unless @thaJeztah objects to what I've done, this is probably safe to close :eyes:

superseded by https://github.com/containerd/containerd/pull/8498


feat(NFT invite rewards): support NFT transactions in home feed

Description
** This should be merged after https://github.com/valora-inc/blockchain-api/pull/182
Added Received NFT item and Sent NFT in the home feed. When clicking the NFT transaction item, it opens Valora NFT page.

Other changes
N/A
Tested
With iOS emulator and locally running blockchain-api
How others should test
With iOS emulator and blockchain-api
Related issues

https://github.com/valora-inc/blockchain-api/pull/182
https://linear.app/valora/issue/RET-329/support-nfts-in-the-home-feed

Backwards compatibility
Yes

Hey @MuckT  as discussed in weekly meet this is needed to be tested once NFT minting is made available on Alfajores build
Thanks.!


Xcode 10, Swift 4.2 support

It's giving warning for swift 4.2.

My PR #422 was merged this morning. Shouldnt be long at this point. Just needs to be released.

@mduttondev thanks!

Would you please release it so it updates in cocoapods? Thanks.

Hello, I'm using cocoapods and It's giving warning for swift 4.2


Export project's skill metrics table

on /administrator/projects/<project-id>/metrics/skills add ability to export project's skills metrics to csv or excel; export the following data/columns:

skill name
skill id
num user of achieved
num of users in progress
date last reported in GMT
date last achieved in GMT

PR: #2837


Using requirements.txt to fill in setup.py install_requires field. Mi…

…ght not be best practice, but it is more convenient and will result in fewer errors related to inputting data twice.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #193   +/-   ##
=======================================
  Coverage   95.43%   95.43%
=======================================
  Files          45       45
  Lines        2256     2256
=======================================
  Hits         2153     2153
  Misses        103      103

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 abd8a27...c063bd2. Read the comment docs.


Write A SQL query example of Count of people that reach the Certificate page by Department

Write A SQL query example of Count of people that reach the Certificate page by Department

Blocked by #238

SELECT dept.name, count(proj.id) AS Projects
FROM approver_project as proj
JOIN approver_person as pers
ON pers.id = proj.owner_id
LEFT JOIN approver_clinicaldepartment AS dept
ON pers.department_select_id=dept.id
WHERE proj.approval_date IS NOT NULL
GROUP BY dept.id


How to use the "Secure Token" option

I'm integrating recaptcha into my cordova application and since the domain being used will be the users phone, I need to use the "Secure Token" option.
Any idea how to implement this with your angular module?

+1

+1

Server side token generation issues, plus poor reCaptcha UX lead us to remove reCaptcha from our API, so I'm taking back my +1 :)

+2

+1

+1

I just opened a PR to support secure tokens: #100
If you want to use it before it's merged, use
bower install --save 'https://github.com/fkammer/angular-recaptcha.git#feature-stoken-support'


[CLI] Added taskListCheck

What changed?

Why?

How did you test it?

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 01899430-aad8-4169-be5a-2fda590c0064

0 of 123 (0.0%)  changed or added relevant lines in 1 file are covered.
59 unchanged lines in 14 files lost coverage.
Overall coverage increased (+0.03%) to 57.027%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tools/cli/domainCommands.go
0
123
0.0%

Files with Coverage Reduction
New Missed Lines
%

common/peerprovider/ringpopprovider/config.go
2
81.58%

common/persistence/executionManager.go
2
78.94%

common/persistence/sql/common.go
2
50.65%

common/persistence/sql/sqlplugin/postgres/db.go
2
85.0%

common/persistence/statsComputer.go
2
95.71%

common/task/parallelTaskProcessor.go
2
93.06%

common/util.go
2
53.44%

service/history/execution/mutable_state_builder.go
2
68.6%

common/task/fifoTaskScheduler.go
3
84.54%

service/history/task/fetcher.go
4
91.24%

Totals

Change from base Build 01897a1d-f7f4-46e6-a6bc-58f203866256:
0.03%

Covered Lines:
87297

Relevant Lines:
153080

💛  - Coveralls


update nexusUrl

Codecov Report

Merging #7 (07be461) into master (ab11927) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #7   +/-   ##
=======================================
  Coverage   88.95%   88.95%
=======================================
  Files          33       33
  Lines         534      534
  Branches       82       82
=======================================
  Hits          475      475
  Misses         30       30
  Partials       29       29

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 ab11927...07be461. Read the comment docs.


Bump release to Rust 1.81 once sort changes shake out

Rust 1.81 came out today, with a rather prominent change to the implementation of sort. Part of this is that sort may now panic if Ord is not a total order.
While this change is overall quite good (probably will expose latent issues), let's temporarily pin cargo-nextest binaries to 1.80 while ecosystem issues are worked out.

Done, in https://github.com/nextest-rs/nextest/commit/8bd85d89cbc8d5997b98e99601a24fd0bf32b985.


Make pattern following entry an advanced panel option

I will add a little message to the wizard, at the top, but going to leave the existing progression of the wizard so that it can live without callbacks


Double Auth Request leading to errors

When clicking on the auth "square" in the Navbar and upon returning from the OAuth page, an DeliverAuthCode request is incorrectly sent twice. At least the server executes it twice.
As auth codes are single-use, this second call will return an error and shows an error on the frontend.

Fixed using a cors interceptor.


Nodejs scripts

Didn't know if I could fork and make a PR for a nodejs set of scripts that Im working on, based on your templates. For now I did a repo on my profile specifying that this is the main repo from which it originated.
Could you tell me how I could contribute the scripts once they are done? Thanks!
Repo here, just in case https://github.com/404answernotfound/script-to-rule-them-all-nodejs

Nice work on your fork! We're not interested in merging back here just because we wouldn't use that internally.

Nice work on your fork! We're not interested in merging back here just because we wouldn't use that internally.

Thank you for the answer Mike! Have a great day :D


jsonschema metadata error

python -m nuitka --version
0.6.5
Python: 3.7.5 (tags/v3.7.5:5c02a39a0b, Oct 14 2019, 23:09:19) [MSC v.1916 32 bit(Intel)]
Executable: C:\Users\IEUser\AppData\Local\Programs\Python\Python37-32\python.exe

OS: Windows
Arch: x86

How did you install Nuitka and Python

pip
python -m pip freeze
aiofiles==0.6.0
aiohttp==3.7.4.post0
aiomysql==0.0.21
aioodbc==0.3.3
aiopg==1.2.1
aiosqlite==0.17.0
alabaster==0.7.12
altgraph==0.16.1
appdirs==1.4.4
async-timeout==3.0.1
asyncio==3.4.3
attrs==20.3.0
auto-py-to-exe==2.6.5
Babel==2.9.0
bottle==0.12.17
bottle-websocket==0.2.9
certifi==2019.9.11
cffi==1.13.2
chardet==3.0.4
colorama==0.4.1
commonmark==0.9.1
Cython==0.29.23
distlib==0.3.1
docutils==0.17
Eel==0.10.4
filelock==3.0.12
future==0.18.2
gevent==1.4.0
gevent-websocket==0.10.1
greenlet==0.4.15
idna==2.8
imagesize==1.2.0
importlib-metadata==3.10.1
influxdb==5.2.3
iso8601==0.1.14
isort==4.3.21
Jinja2==2.11.3
jsonschema==3.2.0
lazy-object-proxy==1.4.3
MarkupSafe==1.1.1
mccabe==0.6.1
multidict==5.1.0
nose==1.3.7
Nuitka==0.6.5
numpy==1.17.3
outcome==1.1.0
packaging==20.9
paho-mqtt==1.5.1
pandas==0.25.2
pefile==2019.4.18
pluggy==0.13.1
psycopg2-binary==2.8.6
ptvsd==4.3.2
PuLP==1.6.10
py==1.10.0
pycparser==2.19
Pygments==2.8.1
PyInstaller==3.5
pylint==2.4.3
pymodbus==2.5.1
pyModbusTCP==0.1.10
PyMySQL==0.9.3
pyodbc==4.0.30
pyparsing==2.4.2
Pyro4==4.77
pyrsistent==0.17.3
pyserial==3.5
pyserial-asyncio==0.5
python-dateutil==2.8.0
pytz==2019.3
pywin32==225
pywin32-ctypes==0.2.0
recommonmark==0.7.1
Represent==1.6.0.post0
requests==2.22.0
rinoh-typeface-dejavuserif==0.1.3
rinoh-typeface-texgyrecursor==0.1.1
rinoh-typeface-texgyreheros==0.1.1
rinoh-typeface-texgyrepagella==0.1.1
rinohtype==0.5.2
serpent==1.28
six==1.15.0
slugify==0.0.1
snowballstemmer==2.1.0
Sphinx==3.5.4
sphinxcontrib-applehelp==1.0.2
sphinxcontrib-devhelp==1.0.2
sphinxcontrib-htmlhelp==1.0.3
sphinxcontrib-jsmath==1.0.1
sphinxcontrib-qthelp==1.0.3
sphinxcontrib-serializinghtml==1.1.4
SQLAlchemy==1.4.7
sqlalchemy-aio==0.16.0
SQLAlchemy-Utils==0.37.0
stringify==0.1.1
toml==0.10.2
tox==3.23.0
typed-ast==1.4.0
typing-extensions==<IP_ADDRESS>
urllib3==1.25.6
virtualenv==20.4.3
websockets==8.1
whichcraft==0.6.1
wrapt==1.11.2
xlrd==2.0.1
yarl==1.6.3
zipp==3.4.1

I am using jsonschema  pip module. This module needs some json files that install in C:\Users\IEUser\AppData\Local\Programs\Python\Python37-32\Lib\site-packages\jsonschema\schemas
Nuitka standalone (follow imports) compilations ends with no error and I get compilation directory (main.dist).
However, I get an error when I run main.dist/main.exe beacuse it does not found json files from jsonschema module.
If I copy jsonschema\schemas directory to main.dist directory, main.exe found json files but there is still another poblema: I get a no package metadata was found for jsconschema error.

Does't the --include-package-data option work for it here?
Regarding the metadata not found, for pkg_resources there is now a plugin, and I think it ought to be possible to expand it in a way that it works for metadata and its backport as well, I will look into that. But basically so far, that is not yet implemented and in terms of reporting it's a duplicate in that sense. Btw, strange how there is a typo in the error message quoted for the package name?

@kayhayen sorry for the typo, It was my own typo, I did not copy/pasted but I wrote from my memory with my hands ;-)
I get no such option: --include-package-data error when I use --include-package-data option.
How can I tracking new metadata feautre?
Thank you very much

You are using an impressively old version of Nuitka there.

I just pushed to factory branch improvements for this, the data files for jsonschema are now copied automatically. Also the pkg-resources plugin will recognize the metadata version code and compile time resolve when enabled.
https://nuitka.net/doc/factory.html

I just pushed to factory branch improvements for this, the data files for jsonschema are now copied automatically. Also the pkg-resources plugin will recognize the metadata version code and compile time resolve when enabled.
https://nuitka.net/doc/factory.html

It works with factory branch and --include-package-data=jsonschema --plugin-enable=pkg-resources options.
Thank you very muc, @kayhayen
It will be merge in master branch in the future, right?

@solanu the package data option ought to be unnecessary, and only duplicate some hard coding in Nuitka, it's on develop now and will be in 0.6.15 release, but not as a hotfix, because I did structural changes. But develop is pretty stable too usually. We keep the issue open until it's released as stable, so people find this until then.

Part of 0.6.15 release.

Part of 0.6.15 release.

Thank you very much!


Inconsistency in the Engagement report doc

The Engagement report default value is Off. This is mentioned in the notes of View and download meeting attendance reports in Teams and in Microsoft Teams meeting attendance report. The option is turned off when a new Teams meeting policy is created from Teams admin portal, but Enabled when a new Teams meeting policy is created in PowerShell.

Thank you for the review, @JohanFreelancer9. I'm not sure I'm doing it right, as I only have the 'Close pull request' button available.

Looks like I can't commit. I don't have write access to the repository.

There is a problem with this PR.  The change is inconsistent with the paragraph following the change.
@robertpp Can  you please clarify?
Proposed change says the policy is off by default:
This policy is off by default. If you turn this on, your organizers can see who registered and attended the meetings and webinars they set up. To turn it on in the Teams admin center, go to Meetings > Meeting policies, and set the Engagement report setting to Turn on.
But two paragraphs down still says the parameter is Enabled by default.
By default, the AllowEngagementReport parameter is set to Enabled in PowerShell. To prevent a meeting organizer from downloading the meeting attendance report, set the AllowEngagementReport parameter to Disabled.

If the following is true, then we need to clarify.  Because current change request doesn't clarify the following:  The option is turned off when a new Teams meeting policy is created from Teams admin portal, but Enabled when a new Teams meeting policy is created in PowerShell. I would also follow up with the PMs.  Is this the expected behavior?

Hello @CarolynRowe . This request was due to an incorrect understanding on my side based on incomplete information and erroneous assumptions. There are more details about this on another request.
In summary, the policy is no longer off by default in the Global policy. According to the references mentioned in the first post and internal documentation, this was supposed to be off by default, due to privacy concerns. Something must have changed on how PG regards this feature.
There is also a dependency between the AllowEngagementReport value in the Global policy and the AllowEngagementReport in new meeting policies. All new policies follow the setting in the Global policy if not set. This is not documented anywhere and became the scope of the second GitHub item.
Finally, I made a confusion regarding the 'Turn on' visible in TAC when a new meeting policy is created. I interpreted it as an option to turn on the feature, meaning it is currently turned off, when it actually means the feature is turned on. Probably the best here would be to change 'Turn on' with 'Enabled' and 'Turn off' with 'Disabled', which would match the PS implementation and would prevent confusions, but this is not within the scope of this request.
To conclude, the actual purpose of this item remains alignment of the public documentation with respect to the actual default value of the policy. It turns out the needed change is to update the 2 references in the first post, which continue to state the default is off. At least one of them should probably mention also why such default is not regarded as a concern for privacy, since such default might raise a brow with our GDPR-compliant customers.

@CarolynRowe


db.type() is deprecated

Description
Basically I started having this warning on my console:
db.type() is deprecated and will be removed in a future version of PouchDB
I think this is related to the current version of usePouch, and I just wanted to give a heads up to see if this can be updated.
Steps to Reproduce
just using usePouch will show this warning on the console

db.type() in not used in this package.


DS1823xs+ with E10M20-T1 isn't recognised in GUI

Again, many thanks for all the hard work you do here, it's much appreciated and very useful.
I've been running DSM 7.2-64570-2 for ages with no issues, however this morning I upgraded to DSM 7.2.1-69057-1 and although the script appears to have run without issue and found the card, none of it's functionality appears to be present i.e. the two M2 SSD drives and/or the 10Gb network port.
Any help would be appreciated.
Task: Check HDD Database
Start time: Wed, 01 Nov 2023 08:59:38 GMT
Stop time: Wed, 01 Nov 2023 08:59:45 GMT
Current status: 0 (Normal)
Standard output/error:
Synology_HDD_db v3.1.65
DS1823xs+ DSM 7.2.1-69057-1
Using options: --showedits --ram --autoupdate=2
Running from: /volume1/homes/AndrewTapp/Scripts/syno_hdd_db.sh

HDD/SSD models found: 2
WD181KRYZ-01AGBB0,01.01H01
WD221KRYZ-01BBAB0,01.01H01

M.2 drive models found: 1
KINGSTON SFYRD4000G,EIFK31.6

M.2 PCIe card models found: 1
E10M20-T1

Expansion Unit models found: 1
DX517

 [0;33mWD181KRYZ-01AGBB0 [0m already exists in  [0;36mds1823xs+_host_v7.db [0m
 [0;33mWD181KRYZ-01AGBB0 [0m already exists in  [0;36mds1821+_host_v7.db [0m
 [0;33mWD181KRYZ-01AGBB0 [0m already exists in  [0;36mds1823xs+_host_v7.db.new [0m
 [0;33mWD181KRYZ-01AGBB0 [0m already exists in  [0;36mds1821+_host_v7.db.new [0m
 [0;33 mWD181KRYZ-01AGBB0 [0m already exists in  [0;36mdx517_v7.db [0m
 [0;33mWD221KRYZ-01BBAB0 [0m already exists in  [0;36mds1823xs+_host_v7.db [0m
 [0;33mWD221KRYZ-01BBAB0 [0m already exists in  [0;36mds1821+_host_v7.db [0m
 [0;33mWD221KRYZ-01BBAB0 [0m already exists in  [0;36mds1823xs+_host_v7.db.new [0m
 [0;33mWD221KRYZ-01BBAB0 [0m already exists in  [0;36mds1821+_host_v7.db.new [0m
 [0;33mWD221KRYZ-01BBAB0 [0m already exists in  [0;36mdx517_v7.db [0m
 [0;33mKINGSTON SFYRD4000G [0m already exists in  [0;36mds1823xs+_host_v7.db [0m
 [0;33mKINGSTON SFYRD4000G [0m already exists in  [0;36mds1821+_host_v7.db [0m
 [0;33mKINGSTON SFYRD4000G [0m already exists in  [0;36mds1823xs+_host_v7.db.new [0m
 [0;33mKINGSTON SFYRD4000G [0m already exists in  [0;36mds1821+_host_v7.db.new [0m
 [0;33mKINGSTON SFYRD4000G [0m already exists in  [0;36mds1821+_e10m20-t1_v7.db [0m
 [0;33mKINGSTON SFYRD4000G [0m already exists in  [0;36mds1823xs+_e10m20-t1_v7.db [0m

 [0;33 mE10M20-T1 NIC [0m already enabled for  [0;36mDS1823xs+ [0m
 [0;33mE10M20-T1 NVMe [0m already enabled for  [0;36mDS1823xs+ [0m
 [0;33mE10M20-T1 SATA [0m already enabled for  [0;36mDS1823xs+ [0m
 [0;33mE10M20-T1 [0m already enabled in  [0;36mmodel.dtb [0m

Support disk compatibility already enabled.

Support memory compatibility already disabled.

Max memory already set to 64 GB.

M.2 volume support already enabled.

Drive db auto updates already enabled.

"WD181KRYZ-01AGBB0": {
"01.01H01": {
"compatibility_interval": [
{
"compatibility": "support",
"not_yet_rolling_status": "support",
"fw_dsm_update_status_notify": false,
"barebone_installable": true
}
]
},
"default": {

"WD221KRYZ-01BBAB0": {
"01.01H01": {
"compatibility_interval": [
{
"compatib ility": "support",
"not_yet_rolling_status": "support",
"fw_dsm_update_status_notify": false,
"barebone_installable": true
}
]
},
"default": {

"KINGSTON SFYRD4000G": {
"EIFK31.6": {
"compatibility_interval": [
{
"compatibility": "support",
"not_yet_rolling_status": "support",
"fw_dsm_update_status_notify": false,
"barebone_installable": true
}
]
},
"default": {

DSM successfully checked disk compatibility.

You may need to  [0;36mreboot the Synology [0m to see the changes.

What does the following command return:
cat /run/adapter_cards.conf
Can you run this script and reply with what it outputs:
https://github.com/007revad/Synology_HDD_db/blob/test/m2_card_check.sh

These return as follow.
root@DS1823xs: cat /run/adapter_cards.conf
M2D20_sup_nvme=no
E10M20-T1_sup_sata=no
E10M20-T1_sup_nic=no
M2D17_sup_sata=no
E10M20-T1_sup_nvme=no
M2D18_sup_sata=no
M2D17_sup_nic=no
M2D18_sup_nic=no
M2D20_sup_sata=no
M2D17_sup_nvme=no
M2D18_sup_nvme=no
FX2422N_sup_nic=no
FX2422N_sup_nvme=no
FX2422N_sup_sata=no
M2D20_sup_nic=no
root@DS1823xs:
root@DS1823xs:/volume1/homes/AndrewTapp/Scripts# ./m2_card_check.sh
DSM 7.2.1-69057 Update 1
2023-11-01 10:54:11
Checking support_m2_pool setting
/etc.defaults/synoinfo.conf: yes
/etc/synoinfo.conf:          yes
Checking supportnvme setting
/etc.defaults/synoinfo.conf: yes
/etc/synoinfo.conf:          yes
Checking permissions and owner of libsynonvme.so.1
Which should be -rw-r--r-- 1 root root
-rw-r--r-- 1 root root 58314 Jun 21 11:28 /usr/lib/libsynonvme.so.1
Checking permissions and owner of synonvme
Which should be -rwxr-xr-x 1 root root
-rwxr-xr-x 1 root root 17273 Jun 21 11:28 /usr/syno/bin/synonvme
Checking permissions and owner of model.dtb files
Which should be -rw-r--r-- 1 root root
-rw-r--r-- 1 root root 5482 Sep 23 16:12 /etc.defaults/model.dtb
-rw-r--r-- 1 root root 3515 Oct 18  2021 /etc/model.dtb
-rw-r--r-- 1 root root 5482 Nov  1 08:57 /run/model.dtb
Checking power_limit="100,100,100,100" is in model.dtb files
Missing in /etc.defaults/model.dtb
Missing in /etc/model.dtb
Missing in /run/model.dtb
Checking E10M20-T1 is in model.dtb files
Missing in /etc/model.dtb
Checking M2D20 is in model.dtb files
Missing in /etc/model.dtb
Checking M2D18 is in model.dtb files
Missing in /etc.defaults/model.dtb
Missing in /etc/model.dtb
Missing in /run/model.dtb
Checking permissions and owner of adapter_cards.conf files
Which should be -rw-r--r-- 1 root root
-rw-r--r-- 1 root root 3418 Nov  1 08:57 /usr/syno/etc.defaults/adapter_cards.conf
-rw-r--r-- 1 root root 2652 Sep 22  2021 /usr/syno/etc/adapter_cards.conf
-rw-r--r-- 1 root root 283 Nov  1 08:57 /run/adapter_cards.conf
Checking /usr/syno/etc.defaults/adapter_cards.conf
E10M20-T1_sup_nic NOT set to yes
E10M20-T1_sup_nvme NOT set to yes
E10M20-T1_sup_sata NOT set to yes
M2D20_sup_nvme NOT set to yes
M2D18_sup_nvme NOT set to yes
M2D18_sup_sata NOT set to yes
Checking /usr/syno/etc/adapter_cards.conf
E10M20-T1_sup_nic NOT set to yes
E10M20-T1_sup_nvme NOT set to yes
E10M20-T1_sup_sata NOT set to yes
M2D20_sup_nvme NOT set to yes
M2D18_sup_nvme NOT set to yes
M2D18_sup_sata NOT set to yes
Checking /run/adapter_cards.conf
E10M20-T1_sup_nic NOT set to yes
E10M20-T1_sup_nvme NOT set to yes
E10M20-T1_sup_sata NOT set to yes
M2D20_sup_nvme NOT set to yes
M2D18_sup_nvme NOT set to yes
M2D18_sup_sata NOT set to yes
Checking synodisk --enum -t cache
************ Disk Info ***************

Disk id: 1
Disk path: /dev/nvme2n1
Disk model: KINGSTON SFYRD4000G
Total capacity: 3726.02 GB
Tempeture: 19 C
************ Disk Info ***************
Disk id: 2
Disk path: /dev/nvme3n1
Disk model: KINGSTON SFYRD4000G
Total capacity: 3726.02 GB
Tempeture: 20 C

Checking syno_slot_mapping
System Disk
Internal Disk
01: /dev/sata1
02: /dev/sata2
03: /dev/sata3
04: /dev/sata4
05: /dev/sata10
06: /dev/sata11
07: /dev/sata12
08: /dev/sata13
Esata port count: 2
Eunit port 1 - DX517
01: /dev/sata5
02: /dev/sata6
03: /dev/sata7
04: /dev/sata8
05: /dev/sata9
Esata port 2
01:
USB Device
01:
02:
03:
Internal SSD Cache:
01: /dev/nvme2n1
02: /dev/nvme3n1

Checking udevadm nvme paths
nvme0: /devices/pci0000:00/0000:00:01.1/0000:01:00.0/0000:02:04.0/0000:06:00.0/nvme/nvme0
nvme1: /devices/pci0000:00/0000:00:01.1/0000:01:00.0/0000:02:08.0/0000:07:00.0/nvme/nvme1
nvme2: /devices/pci0000:00/0000:00:01.3/0000:0f:00.0/nvme/nvme2
nvme3: /devices/pci0000:00/0000:00:01.4/0000:10:00.0/nvme/nvme3
Checking devicetree Power_limit
14.85,9.075
Checking if nvme drives in PCIe card are detected with synonvme
nvme0: Not M.2 adapter card
nvme1: Not M.2 adapter card
nvme2: Not M.2 adapter card
nvme3: Not M.2 adapter card
Checking if nvme drives in PCIe card are detected with synodisk
nvme0: E10M20-T1
nvme1: E10M20-T1
nvme2: Not M.2 adapter card
nvme3: Not M.2 adapter card
Checking PCIe slot path(s)
[pci]
pci1="0000:00:01.1"
Checking nvme drives in /run/synostorage/disks
nvme2n1
nvme3n1
Checking nvme block devices in /sys/block
nvme0n1
nvme1n1
nvme2n1
nvme3n1
Checking synostgd-disk log
Current date/time:   2023-11-01 10:54:11
date: invalid date ‘@’
Last boot date/time:
date: invalid date ‘@’
2023-11-01T08:53:55+00:00 DS1823xs synostgd-disk[12931]: smartctl/smartctl_ata_smart_info_get.c:539 Failed to get smart info on /dev/sata9, 0xA00
2023-11-01T08:53:55+00:00 DS1823xs synostgd-disk[12931]: smartctl/smartctl_ata_smart_info_update.c:49 Failed to update smart info on /dev/sata9, 0xA00
2023-11-01T08:53:55+00:00 DS1823xs synostgd-disk[12931]: smartctl/smartctl_total_status_cache_update.c:46 Failed to update smart info on device /dev/sata9, 0xA00
2023-11-01T08:53:55+00:00 DS1823xs synostgd-disk[12931]: disk_monitor.c:57 Failed to update smart cache on /dev/sata9
2023-11-01T08:53:55+00:00 DS1823xs synostgd-disk[12913]: smartctl/smartctl_ata_smart_info_get.c:539 Failed to get smart info on /dev/sata6, 0xA00
2023-11-01T08:53:55+00:00 DS1823xs synostgd-disk[12913]: smartctl/smartctl_ata_smart_info_update.c:49 Failed to update smart info on /dev/sata6, 0xA00
2023-11-01T08:53:55+00:00 DS1823xs synostgd-disk[12913]: smartctl/smartctl_total_status_cache_update.c:46 Failed to update smart info on device /dev/sata6, 0xA00
2023-11-01T08:53:55+00:00 DS1823xs synostgd-disk[12913]: disk_monitor.c:57 Failed to update smart cache on /dev/sata6
root@DS1823xs:/volume1/homes/AndrewTapp/Scripts#

Just for information, how it looks in the GUI.

Try running these 4 commands then rebooting:
sudo set_section_key_value /usr/syno/etc.defaults/adapter_cards.conf E10M20-T1_sup_nic DS1823xs+ yes
sudo set_section_key_value /usr/syno/etc.defaults/adapter_cards.conf E10M20-T1_sup_nvme DS1823xs+ yes

sudo set_section_key_value /usr/syno/etc/adapter_cards.conf E10M20-T1_sup_nic DS1823xs+ yes
sudo set_section_key_value /usr/syno/etc/adapter_cards.conf E10M20-T1_sup_nvme DS1823xs+ yes

That has worked, after rebooting the NVME cache and storage pool, plus network adapter are showing as expected.  Many thanks.
On a side note, I see on the logs from running syno_hdd_db.sh that there are a number of references to DS1821+ config files, left around when I migrated from that DiskStation to my current DS1823xs+  Do you know if these now required and should they be removed?

That has worked

Excellent.
To delete the old ds1821+ db files run this command:
cd /var/lib/disk-compatibility && for db in ./ds1821+*; do rm -i $db; done
You will need to enter y to delete each file. This just gives you the chance to confirm it is going to delete the correct file.

Once again many thanks for your help, take care.


Improve drt performance by inlining dbTechLayerDir constructors

getIdx() continually calls into odb code to determine the layer direction.
Create a helper isZDirHorizontal() which caches this.
Detailed routing of the Microwatt register file is 8.5% faster with this
applied.
Signed-off-by: Anton Blanchard<EMAIL_ADDRESS>

FlexGRGridGraph does something similar already.
The other users of getZDir() error out if the direction is neither horizontal nor vertical. Perhaps we could move that error into the initialisation code and make getZDir() match FlexGRGridGraph and return direction == HORIZONTAL

Is this something that would be fixed by LTO? Would it be possible to add LTO only to DRT? e.g.
set_property(TARGET drt
             PROPERTY INTERPROCEDURAL_OPTIMIZATION TRUE)

I don't think you can LTO only one part of the program afaik but I'm not certain.

This stack overflow post indicates it can be done: https://stackoverflow.com/questions/31355692/how-do-i-enable-link-time-optimization-lto-with-cmake
It might be worth a quick check. I might have time later this evening.

I suspect the target has to be an executable not a library.  Fromhttps://gcc.gnu.org/wiki/LinkTimeOptimization
This expands the scope of inter-procedural optimizations to encompass the whole program (or, rather, everything that is visible at link time).

LTO just keeps all the .o files in LLVM IR. This should be more than possible.

Perhaps someone wants to try this out.  In general I see no real reason to limit LTO to drt.  The main concern is the compile time cost of LTO.

@rovinski great idea - there are a lot of small functions at the top of the drt profile, so LTO should help quite a lot.


Improve responsive design on mobile devices

@lin-lu Can you please describe (and possibly print screenshots of) the problem you're seeing that you're trying to fix?

The first problem is that when the screen width is between 720px and 900px, the hamburger menu button on the home page does not work. You can reproduce this bug by resizing the browser.
The second problem is that the search text box is displayed on top of the vue logo, when the width is around 800px.

I changed the breakpoint of the media queries from 720px to 900px, so the mobile version will be displayed when the width is smaller than 900px.
After I changed that, I noticed that the max-width of the content is fixed at 600px, so when the screen width is larger than that and smaller than 900px, the content will appear on the left side of the page, leaving some empty space on the right side.

To make it symmetry, I also centered the content for mobile devices.

Thanks! This looks good. 👍


Support vertical pagination

In-article vertical paging, horizontal for between articles.

Flagged for deletion. This goes against the core ideology of Treesaver by breaking narrative. Unless a compelling argument can be given for keeping it.


error...

Uncaught ReferenceError: require is not defined
at VASTPlayer.js:3

Hello and sorry for the delay! You either need to use browserify to build the package, OR use one of the prebuilt hosted scripts:
<script src="https://cdn.jsdelivr.net/vast-player/0.2/vast-player.min.js"></script>

You could also build the script yourself by cloning the repo and running
$> npm install && npm run build


Remove rendezvous from juttle-service.

This functionality is handled in juttle-engine now. I'll add the unit
tests to juttle-engine in a separate PR.
@demmer @go-oleg @mnibecker

lgtm


Using "_lookupFactory" is deprecated. Please use container.factoryFor instead

Hi, there's a new deprecation:
https://emberjs.com/deprecations/v2.x/#toc_migrating-from-_lookupfactory-to-factoryfor
It seems to come from here:
https://github.com/Vestorly/torii/blob/master/addon/lib/container-utils.js#L21
I tried setting up Torii to try and fix it but couldn't get it installed locally (windows pc atm).

@oskarrough Thanks for reporting. I'll fix this deprecation.


chore: release 1.0.1

:robot: I have created a release *beep* *boop*
1.0.1 (2019-05-24)
Bug Fixes

types: use Metadata type instead of r.Response (#266) (cf32552)

Release is at https://github.com/googleapis/nodejs-dns/releases/tag/v1.0.1

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Filter to route some requests through an external proxy

I have been trying to find solution for this for days now,
My problem
zuul:
  routes:
    test:
      path: /test/**
      serviceId: test
      url: http://my.api.server.com
    test2:
      path: /test2/**
      serviceId: test2
      url: http://my.api.server.com // but through an external proxy

I could find this https://github.com/spring-cloud/spring-cloud-netflix/issues/2320 at the best. But that would requiring extending the default SimpleHostRoutingFilter  with 2 different CloseableHttpClient  and is too much code change, this is probably unavoidable in my setting. any suggestions pls?

There is no way to do this automatically for particular routes without writing a custom filter.

@spencergibb , I am willing to write a custom filter, which is better than extending SimpleHostRoutingFilter completely, Please do give me hints how this can be achieved?

I think it is going to be best to extend SimpleHostRoutingFilter in most cases to reuse as much code as possible.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

got that. Thanks, @ryanjbaxter for the response.


Bugfix/UNR-5226

Description

Bugfix for UNR-5171
The fix it to change the parent class of AComponentUpdateEventTracingTest from AEventTracingCrossServerTest to AEventTracingTest. This means that the test correctly expects 1 worker rather than 2.
This, in turn, necessitated the creation of ComponentUpdateCrossServerEventTracingTestm which expects two workers as you need at least two workers for CrossServer to be an applicable concept/.

Release note
N/A: This is an issue with a test as opposed to a feature, it doesn't impact customers.
Tests
I ran both tests effected 25x prior to pushing this PR. All tests passed.
Documentation
N/A

Corresponding JIRA ticket: https://improbableio.atlassian.net/browse/UNR-5171

This was a typo. This PR actually refers to: UNR-5226


Disable profile stuff by default

Profile is used quite rarely IME and it results in automatic http calls (which is annoying if working locally offline).
Could we consider having profile support off by default?

We will achive it in https://github.com/frictionlessdata/datapackage-js/issues/43

@roll i'm not quite sure #43 this is the same thing. Simple disabling is different from bundling. Bundling results in a whole bunch of issues (e.g. bundled schema can go out of date etc).

@rufuspollock
What do you mean by disable profile - disable validation?

@roll 2 options:

disable validation by default - if i want to do that then i need the profile (or rather just validate you have valid json)
or just have base and bundle that in

The key thing is that we have to be able to run offline and without http calls from the lib ...

@rufuspollock
Thanks. We will support 100% offline (as a user story) no doubts.

just a note:

yes, 100% offline support, by building in the schemas (same as python lib)
no, not actually disabling validation - will always be possible to validate against the base dp schema

@pwalsh i note the problem of getting out of date with schemas but happy to go with built in (and you can't build all in, right ...?)
For me, right now it is just a question of getting MVP asap ;-) - for my use generic profile support is not on that MVP

so in that case, you are not blocked now that https works in schemas.frictionlessdata.io. it will be nice to not have the http requests for the schemas, but it does not block any map functionality while the lib still makes those requests.

@rufuspollock
Every version of jsontableschema/datapackage is packaged with some version of schemas. So e.g. if you use<EMAIL_ADDRESS>you use builtin<EMAIL_ADDRESS>So it can't be outdated it's just a characteristic of concrete package version. To update to new datapackage-profile schema a client should update datapackage package itself. It would say it's a good way to handle it because of the predictability to clients (strict map between datapackage version -> schemas versions).
Also we're injecting mechanism across libs to be sure we're releasing with the latest version of schemas (otherwise tests just fail).


[baremetal] Avoid narrow loop counters etc

Use uint_fast8_t instead of unsigned char in various loop-type situations. This avoids the need for a 16 or 32-bit system to insert explicit narrow-to-8-bit instructions.
Not the result of an exhaustive source analysis, rather inspecting the disassembly output for a cut-down Cortex-M0+ build looking for UXTB etc instructions, so there could well be more in the complete configuration.

There are some subtleties - unsigned char will promote to signed int when used in an expression, whereas uint_fast8_t will often be equivalent to unsigned int.
That could maybe cause some different results in certain weird scenarios. I had to make doublings be int_fast8_t to avoid a mixed-sign-comparison warning with an int renegotiation counter.
Others all seem fine though - they're being compared with simple constants for loop counters, or with size_t, which is a bigger-or-same-size unsigned value.


Prepare changelog for 0.1.5 release

Summary
Prepare for the 0.1.5 release.
Metadata

Question
Answer

Did you use a meaningful pull request title?
yes

Did you apply meaningful labels to the pull request?
yes

Did you perform a self review first?
yes

Documentation reflects changes?
no

CHANGELOG reflects changes?
yes

Unit/Functional tests reflect changes?
n/a

Did you perform browser testing?
no

Risk level
Low

Relevant links
rig-6

/approved


GH-2442 improve performance for sorting literals with datatype by introducing an enum for known types

GitHub issue resolved: #2442
Briefly describe the changes proposed in this PR:

introduce enum for known datatypes with helper methods that return precomputed data for things like isNumber()
reduce use of instanceof by introducing helper methods on Value like isLiteral() and isBnode()

PR Author Checklist (see the contributor guidelines for more details):

[ ] my pull request is self-contained
[ ] I've added tests for the changes I made
[ ] I've applied code formatting (you can use mvn process-resources to format from the command line)
[ ] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change
[ ] every commit has been signed off

Note: we merge all feature pull requests using squash and merge. See RDF4J git merge strategy for more details.

@jeenbroekstra this was my general idea. It's not helping as much as I had though. I was expecting to increase the performance by maybe 2x, but it's only around 25%.
I'm wondering it the instanceOf usage in the value comparator is what is actually slowing things down. Might look at bit more at that.
You can take a look at this PR though. It gives a good idea of the solution that I was thinking about. We don't need to consider merging it yet though. Since the performance isn't what I was hoping for I think we would need to find more performance in other aspects of the code (like maybe the memory store or maybe the sparql engine) in order to justify the new code.

Btw. I had initially written this code to use optionals, but the profiling showed that it was a big bottleneck. Removing the use of optional didn't actually make a difference though, so we could easily go back to using optionals instead of returning null when the xml datatype is unknown.

Performance wise I'm seeing in the order of 20-30% performance improvement, which is far away from the 2x I was hoping for. Should we still go for this @jeenbroekstra ?

That's still significant, I think. Though I don't quite know what the absolute numbers are and how much of a performance issue this was in the first place.

It's the hottest code when doing sorting. So it will make all sorting of literals faster.

Original
Benchmark                                       Mode  Cnt    Score    Error  Units
ValueComparatorBenchmark.sortManyDeepEquals     avgt    5  282.831 ±  5.465  ms/op
ValueComparatorBenchmark.sortManyPointerEquals  avgt    5  128.369 ±  3.106  ms/op
ValueComparatorBenchmark.sortObjects            avgt    5  147.940 ± 14.908  ms/op
ValueComparatorBenchmark.sortPredicates         avgt    5    0.111 ±  0.028  ms/op
ValueComparatorBenchmark.sortSubjects           avgt    5   42.832 ± 12.686  ms/op
Current
Benchmark                                       Mode  Cnt    Score   Error  Units
ValueComparatorBenchmark.sortLiterals           avgt    5   22.420 ± 8.165  ms/op
ValueComparatorBenchmark.sortManyDeepEquals     avgt    5  198.964 ± 2.486  ms/op
ValueComparatorBenchmark.sortManyPointerEquals  avgt    5   16.796 ± 0.338  ms/op
ValueComparatorBenchmark.sortPredicates         avgt    5    0.052 ± 0.005  ms/op
ValueComparatorBenchmark.sortSubjects           avgt    5   20.751 ± 9.934  ms/op

@jeenbroekstra I figured out the biggest issue was the use of instanceof in the comparator. I've introduced helper methods on Value like .isBnode() with default implementations which makes everything considerably faster!
The subjects benchmark (IRIs and BNodes) is 2x faster, only because of this change.
Sorting objects (mostly literals) is now 6x faster!

There are two things to still consider:

reintroduce Optional instead of returning null when getting the enum based datatype from a literal
remove the datatype IRI comparison pre calculation for known datatypes

Now that everything is as fast as I was expecting originally we can consider if the above two things will affect performance noticeably. If not then we should prefer readable code over highly optimised code with low impact.

Followup from this would be to reduce use of instanceof in the rest of the codebase and use the new helper methods instead (eg. value.isLLiteral() instead of `value instanceof Literal´).

Would really appreciate if you could take a look at this @jeenbroekstra . I'll test out optional today and then I would like to merge this PR.

Use of optionals did not change the performance significantly and it makes the code more intuitiv.


Fix environment export button

Fix undefined prefix
Improve encapsulation of URL gen in models vs views

Coverage remained the same at 76.808% when pulling b7a58222ed5900c2bb8d28a7dd74bbf0ddf8c5ad on parente:fix-env-export into 3364053f09f07d77bc3c97a84f853d7c2986f7cb on Anaconda-Platform:master.

LGTM and tested successfully, merging...
And sorry for the long time to prodive feedback on this one.
Thanks @parente!!


Removing the dormant category.

I do not personally see any reason behind the dormant phase of a help channel. Once a question is answered, or the channel is closed, why should that help channel not then return to being available? Currently as of writing this there are only three available help channels with twenty-seven channels waiting an arbitrary amount of time to be re-opened.
Maybe I am missing the purpose behind the phase, but to me it just limits the available channels, requires more help channels as most are sitting in the dormant phase and unusable.

Moving all of the channels into the available category would push the channel list down for a lot of users

Moving all of the channels into the available category would push the channel list down for a lot of users

So, have less help channels. Currently there are only so many to account for the large amount in the dormant category. Without the dormant category significantly less channels would actually be needed.

Currently there are only so many to account for the large amount in the dormant category.

We don't want to delete help channels because of the archival benefits, so at peak we have seen every channel you see today in use at once.

Without the dormant category significantly less channels would actually be needed.

This isn't so, no matter where the channels are at peak times all will be in use or all will be in available.

it just limits the available channels

This isn't how the system works, once an available channel is claimied one of the dormant channels is revived from the dormant category and moved to the available category, ready to be entered into circulation.


fix ws:xxx fail

What changes were proposed in this pull request
Issue Number: close #2984

@Al-assad

cc @wangsizhu0504 PTAL , thx


Control opacity feature

Added menu and control buttons to manage overlay chat opacity.

Thanks! :+1:
There's just some cleanup I want to do, but I can do that locally. I'll try ship this within the next few days.


Will the model weights and testing code be open-sourced

Hello,
I recently came across your project and I must say, it's an amazing piece of work. I was wondering if there are any plans to release the model weights and testing code as open source. This would be extremely helpful for those of us who would like to test the model's capabilities directly. With the weights and testing code available, we could simply input a video and see the results without having to train the model ourselves.
Thank you for your hard work and I look forward to your response.

Hi, could you refer to this answer? https://github.com/qianqianwang68/omnimotion/issues/37#issuecomment-1765825949. I'm sorry things are a bit disorganized. I will try to clean things up once I have some cycles. But I'd like to make it clear that this method is a per-video optimization-based method, which means that given a new input video, one would have to optimize on that video anyways to see the results.


Enable dynamic zone generation for bind9

Reduce the amount of copy-paste required to add a new zone

Done


(chore): a theme, maybe!

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut-icons@7.3.2-alpha.c2b91e.0@codecademy/gamut-illustrations@0.27.2-alpha.c2b91e.0@codecademy/gamut-kit@0.6.130-alpha.c2b91e.0@codecademy/gamut-labs@31.13.2-alpha.c2b91e.0@codecademy/gamut-patterns@0.8.6-alpha.c2b91e.0@codecademy/gamut-styles@13.1.2-alpha.c2b91e.0@codecademy/gamut-tests@4.0.5-alpha.c2b91e.0@codecademy/gamut@47.3.3-alpha.c2b91e.0@codecademy/markdown-overrides@0.9.3-alpha.c2b91e.0@codecademy/monaco@0.6.46-alpha.c2b91e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6287cc60153ecc0a050880aa--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@7.3.2-alpha.399e7f.0@codecademy/gamut-illustrations@0.27.2-alpha.399e7f.0@codecademy/gamut-kit@0.6.130-alpha.399e7f.0@codecademy/gamut-labs@31.13.2-alpha.399e7f.0@codecademy/gamut-patterns@0.8.6-alpha.399e7f.0@codecademy/gamut-styles@13.1.2-alpha.399e7f.0@codecademy/gamut-tests@4.0.5-alpha.399e7f.0@codecademy/gamut@47.3.3-alpha.399e7f.0@codecademy/markdown-overrides@0.9.3-alpha.399e7f.0@codecademy/monaco@0.6.46-alpha.399e7f.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6287ce5204d23f0be81a8399--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@7.3.2-alpha.79c8de.0@codecademy/gamut-illustrations@0.27.2-alpha.79c8de.0@codecademy/gamut-kit@0.6.130-alpha.79c8de.0@codecademy/gamut-labs@31.13.2-alpha.79c8de.0@codecademy/gamut-patterns@0.8.6-alpha.79c8de.0@codecademy/gamut-styles@13.1.2-alpha.79c8de.0@codecademy/gamut-tests@4.0.5-alpha.79c8de.0@codecademy/gamut@47.3.3-alpha.79c8de.0@codecademy/markdown-overrides@0.9.3-alpha.79c8de.0@codecademy/monaco@0.6.46-alpha.79c8de.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6287e710d7a3682200858c05--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@7.3.2-alpha.6e3966.0@codecademy/gamut-illustrations@0.27.2-alpha.6e3966.0@codecademy/gamut-kit@0.6.130-alpha.6e3966.0@codecademy/gamut-labs@31.13.2-alpha.6e3966.0@codecademy/gamut-patterns@0.8.6-alpha.6e3966.0@codecademy/gamut-styles@13.1.2-alpha.6e3966.0@codecademy/gamut-tests@4.0.5-alpha.6e3966.0@codecademy/gamut@47.3.3-alpha.6e3966.0@codecademy/markdown-overrides@0.9.3-alpha.6e3966.0@codecademy/monaco@0.6.46-alpha.6e3966.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6287f1607eb6522a4f7188ed--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@7.3.2-alpha.201f02.0@codecademy/gamut-illustrations@0.27.2-alpha.201f02.0@codecademy/gamut-kit@0.6.130-alpha.201f02.0@codecademy/gamut-labs@31.13.2-alpha.201f02.0@codecademy/gamut-patterns@0.8.6-alpha.201f02.0@codecademy/gamut-styles@13.1.2-alpha.201f02.0@codecademy/gamut-tests@4.0.5-alpha.201f02.0@codecademy/gamut@47.3.3-alpha.201f02.0@codecademy/markdown-overrides@0.9.3-alpha.201f02.0@codecademy/monaco@0.6.46-alpha.201f02.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6287fd640b60f900585f27a2--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@7.3.2-alpha.eff133.0@codecademy/gamut-illustrations@0.27.2-alpha.eff133.0@codecademy/gamut-kit@0.6.130-alpha.eff133.0@codecademy/gamut-labs@31.13.2-alpha.eff133.0@codecademy/gamut-patterns@0.8.6-alpha.eff133.0@codecademy/gamut-styles@13.1.2-alpha.eff133.0@codecademy/gamut-tests@4.0.5-alpha.eff133.0@codecademy/gamut@47.3.3-alpha.eff133.0@codecademy/markdown-overrides@0.9.3-alpha.eff133.0@codecademy/monaco@0.6.46-alpha.eff133.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://628808cd618f47060511dbab--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@7.3.2-alpha.35f59c.0@codecademy/gamut-illustrations@0.27.2-alpha.35f59c.0@codecademy/gamut-kit@0.6.130-alpha.35f59c.0@codecademy/gamut-labs@31.13.2-alpha.35f59c.0@codecademy/gamut-patterns@0.8.6-alpha.35f59c.0@codecademy/gamut-styles@13.1.2-alpha.35f59c.0@codecademy/gamut-tests@4.0.5-alpha.35f59c.0@codecademy/gamut@47.3.3-alpha.35f59c.0@codecademy/markdown-overrides@0.9.3-alpha.35f59c.0@codecademy/monaco@0.6.46-alpha.35f59c.0@codecademy/styleguide@57.5.2-alpha.35f59c.0


increase test cov of reducers test helpers and observable return

summary
not so important, but I was running some tests and saw these warnings.

for some tests, like for dispatchInMiddle, I saw some helpers unreachable lines (cause exceptions were being tested) and led to these uncovered warning.
another helper, id, was function with an unused branch, that was an unused default value for a param
and finally, there was an uncovered function on store observable return

before

now

(there's still an uncovered line for a warning shown on redux.js bundle)

Can you run Prettier on your code? That will get it to pass CI.

Thx, CI is passing now!

Thanks!


Remove gasless transactions page

This PR removes Gasless Transactions page from the docs. As per Andrey Falaleev, this feature wasn’t implemented, because there are bridges and hence, no clients/users for the functionality.

Thanks @sukanyaparashar AFAIK part of the vp of this feature was to allow dApps to pay gas fees on behalf of users -- this may be covered now by the alt gas fee payment option >> will need to verify with @afalaleev and @YYYurchenko

Putting discovery on this ticket


OpTestKernelTest: Add support for automated boot bisection v2.0

This code is modifed and develped on top of the previous code (2f103 : OpTestKernelTest: Git bisect automation..)
With version 2.0 these are the new changes added:

Added support to handle nested bisection loop, which was not working with previous code
Modified code to work for end to end CI server
Better handle all cases of good flow and bad flow
Add json store of good boot commit for good run
Fix the previous code blocker to properly manage the console thread

TODO:
Add boot json generation after bisection with bad commit
Email format for boot bug report

bootbisectionlogs.txt

@PraveenPenguin all concerns adopted, Please merge this


Expose content_id in frontend representations.

This includes the content_id in the frontend representation of content
items - both in the main item, and in the expanded links.
It is becoming necessary for some of the frontend applications to know
about content_ids (eg email_alert_frontend needs the topic content_ids
in order to send them to email-slert-api to create the link to
govdelivery topics).
Trello: https://trello.com/c/7K6FySvv

Code looks good. Is there already consensus about the intent?

One thing to note:
I've made the content_id field mandatory in the links expansion. This seemed reasonable given that items are linked by content_ids, and therefore it shouldn't be possible to have a linked item without a content_id.
There's a snag with this though.  Currently the available_translations expansion returns only self for an item without a content_id. I'm not sure this behaviour is relied on anywhere. Nor does it seem correct. I think we should therefore update content-store to not return available_translations for an item without a content_id.

I think we have sufficient consensus: at a minimum, @bradwright and @elliotcm have been involved in discussions where we've said we need to do this at some point.
Regarding the issue with translations; I agree that behaviour doesn't seem correct.  I don't think that should block merging this PR, though, but it would be good to address soon.
Also, we should "soon" not have any content in content-store without a content ID


Error building docs for private repositories

When merging the skeleton into a private repository, the docs fail to build with:
Extension error (pydata_sphinx_theme):
Handler <function update_config at 0x7f5421bd8160> for event 'env-updated' threw an exception (exception: HTTPSConnectionPool(host='https', port=443): Max retries exceeded with url: //github.com/dls-controls/dls-launcher-app.git.github.io/dls-launcher-app/switcher.json (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f54201b25c0>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution')))
ERROR: InvocationError for command /opt/hostedtoolcache/Python/3.10.7/x64/bin/sphinx-build -EWT --keep-going docs build/html (exited with code 2)

Inside of docs/conf.py, we define a 'github_user' with the repo url and use this to set up the theme:
# The theme to use for HTML and HTML Help pages.  See the documentation for
# a list of builtin themes.
#
html_theme = "pydata_sphinx_theme"
github_repo = project
github_user = "https://github.com/dls-controls/dls-launcher-app.git"

# Theme options for pydata_sphinx_theme
html_theme_options = dict(
     logo=dict(
         text=project,
     ),
     use_edit_page_button=True,
     github_url=f"https://github.com/{github_user}/{github_repo}",
     icon_links=[
         dict(
             name="PyPI",
             url=f"https://pypi.org/project/{project}",
             icon="fas fa-cube",
         )
     ],
     switcher=dict(
         json_url=f"https://{github_user}.github.io/{github_repo}/switcher.json",
         version_match=version,
     ),
     navbar_end=["theme-switcher", "icon-links", "version-switcher"],
     external_links=[
         dict(
             name="Release Notes",
             url=f"https://github.com/{github_user}/{github_repo}/releases",
         )
     ],
 )

Removing the github_user and theme options will allow the build to be successful, so I'm thinking that the build is having trouble connecting to a private repository.
Is this the best way to combat this error? is there a better workaround? or do you think a fix could be implemented to support this interaction?

What version of the pydata theme are you using? Does setting check_switcher=False in html_theme_options make it work?

Worked through it together, check_switcher=False worked

https://github.com/DiamondLightSource/python3-pip-skeleton/pull/91


[BUG] Discord bot not saving config in the txAdmin panel

Config won't save
---------------------
When I try to save the discord bot config in the txAdmin panel, the one that the server status command is supposed to show (see attached image), the save button doesn't work. The discord bot is still online and responds if you put in the status command, but it only shows what was saved before the most recent update since you can't save new configuration.
Hope this gets resolved in the next update <3

Please see this info here.
Issue can be closed.


Allow retrieval of StopIteration value from C# code

Actually, I'm not sure why I had this change. I don't currently have any need, so I think I will close it for now.


Enable controlling local stream mirroring by introducing isMirrored-prop

This is my suggestion to give people a way to avoid the problem described in #29
If somebody would like to add a checkbox to control this prop into the live example app, that'd be great (i don't have an access token at hand to test this).

@davidzhao my pleasure 😇
Maybe somebody could add a checkbox to control this prop to the example app?


Issue/61 pre process before print

Fix #61
Introduces a preprocessor that runs before pretty-printing. This thing performs the various hacks required for the pretty-printing to look nice. Any further hacks for pretty-printing should go into the PrinterPreprocessor, and the SporkPrettyPrinter can then rely on them being there.

Codecov Report

Merging #64 into master will increase coverage by 0.25%.
The diff coverage is 93.93%.

@@             Coverage Diff              @@
##             master      #64      +/-   ##
============================================
+ Coverage     81.42%   81.68%   +0.25%
- Complexity      267      274       +7
============================================
  Files            24       25       +1
  Lines          1066     1081      +15
  Branches        167      169       +2
============================================
+ Hits            868      883      +15
  Misses          113      113
  Partials         85       85

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/se/kth/spork/spoon/Spoon3dmMerge.java
90.05% <ø> (-0.34%)
28.00 <0.00> (-1.00)

...ain/java/se/kth/spork/cli/PrinterPreprocessor.java
92.59% <92.59%> (ø)
9.00 <9.00> (?)

src/main/java/se/kth/spork/cli/Cli.java
34.42% <100.00%> (+2.22%)
8.00 <5.00> (ø)

...main/java/se/kth/spork/cli/SporkPrettyPrinter.java
89.74% <100.00%> (+1.37%)
21.00 <3.00> (-1.00)
:arrow_up:

Continue to review full report at Codecov.

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


Add Solana support to ZetaChain client and browser deposits

Add Solana Wallet to ZetaChainClient
Update solanaDeposit to support browser

Related issue #182

Summary by CodeRabbit
Release Notes

New Features

Added support for Solana wallets alongside existing Ethereum wallet capabilities.
Introduced new methods to check Solana wallet connection and retrieve public keys.
Enhanced the solanaDeposit function for improved transaction handling and error reporting.

Bug Fixes

Improved error handling for wallet capabilities and transaction failures.

Chores

Updated project dependencies to include the Solana wallet adapter.

For Cli:
npx hardhat solana-deposit --amount 0.1 --recipient 0x1b02C3079c039DF9E20DC68427cb87bf821BffeF --types '["address", "bytes", "bool"]' 0x05ba149a7bd6dc1f937fa9046a9e05c05f3b18b0 0x4955a3F38ff86ae92A914445099caa8eA2B9bA32 true

For Browser:
I wrote a front-end test case here.

Final call result: 4CX34eMBpKcSPSjWfcuVa8whPsG1bk7vtrnN5PqvgrM5AZQa9UCTNF6gzKx5KuYfetNSwbhMRw3Gg69Q31Wis3Q7
Note: If you use the Phantom Wallet, you need to set it to developer mode.

@fadeev pls review


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/1046 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit 33aca6c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_main_2022-02-17-01-39-39 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Building with NGC : Can't resolve all parameters for AngularFirestore

Console from ionic build --prod :
ionic build --prod
[INFO] Running app-scripts build: --prod

[15:38:33]  build prod started ...
[15:38:33]  clean started ...
[15:38:33]  clean finished in 6 ms
[15:38:33]  copy started ...
[15:38:33]  deeplinks started ...
[15:38:33]  deeplinks finished in 234 ms
[15:38:33]  ngc started ...
Warning: Can't resolve all parameters for AngularFirestore in /Users/thomas/Documents/Dev/Meet/Meet-Ionic/node_modules/angularfire2/firestore/index.d.ts: ([object Object], ?). This will become an error in Angular v5.x
Warning: Can't resolve all parameters for AngularFirestore in /Users/thomas/Documents/Dev/Meet/Meet-Ionic/node_modules/angularfire2/firestore/index.d.ts: ([object Object], ?). This will become an error in Angular v5.x
[15:38:41]  ngc finished in 7.53 s
[15:38:41]  preprocess started ...
[15:38:41]  copy finished in 7.86 s
[15:38:41]  preprocess finished in 79 ms
[15:38:41]  webpack started ...
[15:39:17]  webpack finished in 36.01 s
[15:39:17]  uglifyjs started ...
[15:39:17]  sass started ...
[15:39:19]  sass finished in 1.79 s
[15:39:19]  cleancss started ...
[15:39:27]  cleancss finished in 8.04 s
[15:39:43]  uglifyjs finished in 26.10 s
[15:39:43]  postprocess started ...
[15:39:43]  postprocess finished in 40 ms
[15:39:43]  lint started ...
[15:39:43]  build prod finished in 70.11 s
> ionic cordova prepare
> cordova prepare

Version info

Angular:
4.4.3
Firebase:
4.5.2
AngularFire:
5.0.0-rc.3
Other (e.g. Ionic/Cordova, Node, browser, operating system):
ionic info

cli packages: (/usr/local/lib/node_modules)

    @ionic/cli-utils  : 1.13.1
    ionic (Ionic CLI) : 3.13.1

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.0.0
    Cordova Platforms  : ios 4.5.2
    Ionic Framework    : ionic-angular 3.7.1

System:

    Android SDK Tools : 26.0.2
    ios-deploy        : 1.9.0
    ios-sim           : 6.0.0
    Node              : v8.1.4
    npm               : 4.6.1
    OS                : macOS High Sierra
    Xcode             : Xcode 9.0 Build version 9A235

Misc:

    backend : legacy

How to reproduce these conditions
Failing test unit, Plunkr, or JSFiddle demonstrating the problem

Steps to set up and reproduce

Sample data and security rules
<-- include/attach/link to some json sample data (or provide credentials to a sanitized, test Firebase project) -->
Debug output
** Errors in the JavaScript console **
** Output from firebase.database().enableLogging(true); **
** Screenshots **
Expected behavior

Actual behavior

?

Same'ish error in Angular 5:
Warning: Can't resolve all parameters for AngularFirestore in ./node_modules/angularfire2/firestore/index.d.ts: ([object Object], ?). This will become an error in Angular v6.x
Version: webpack 3.8.1

Looks like a duplicate of #1206


Proposal: Sequences

I'm thinking of Immutable.js here.
When native Map and Set support became widespread there wasn't so much point in working with them, because there weren't tools (like iter-tools) to help out.
What I personally did instead, when possible, was just to use Immutable.js data structures and very well rounded standard library. Sometimes though I did actually need a data structure which could be mutated, so I'd use one. Now I had(have) mutable and immutable objects mixed together sometimes, and that sucks because both require different syntaxes to work with.
Iterators are the shared API surface between native Array/Map/Set and Immutable.js List/Map/Set. Both implement .keys, .values, and .entries iterators for all their objects.
I propose to make things even a little simpler. In immutable, Seq is pretty much just an OO wrapper around an iterator. It keeps track of which sort of data it represents (indexed, keyed, or set), a particular iterator instance, and returns a chaining object on which you can call important methods (which in turn also have their results wrapped in Sequences).
I'm thinking of implementing sequences -- I don't think the base functionality should be too hard, at which point it would be substantially easier to operate in environments in which both mutable and immutable data was necessary.

Can you write an example?

A sequence might look like:
Seq(range(3)).map(x => x+1).filter(x => x > 2).toArray()
Behind the scenes iterators are being used to prevent storage allocation on intermediate steps.
You can obviously write the same thing in iter-tools already, but it looks like:
Array.from(filter(x => x > 2, map(x => x+1, range(3))
Each style on its own is not too hard to read, but needing to read logic expressed in both styles is a total headache.

Also Sequences work much better in the land of pretty-printing, where they can flow downwards without repeated assignments.

I had some thought about having that in the past.
I am not 100% sure this is something for iter-tools or for an external library that depends on iter-tools

Yeah it could be an external library. In particular there's a conflict with iter-tools' pay-only-what-you-need philosophy. Sequences need all the methods to be defined on their chaining prototypes.
My main uncertainty if it doesn't go in this library is what to call it. seq, sequence, and mutable are all taken already : (.

Haha, dunno iter-tools seems like a fine name to me. It describes what the library does perfectly. I think it's interesting that the limited availability of good names has to be one of the driving forces against wanton forking in the ecosystem.
I think maybe "sequins". Brb gotta snap that up.

I had an idea on how to implement this while retaining the pay-only-what-you-need nature of iter-tools.
What if you have to build the library like:
import { map, filter, buildSequence } from 'iter-tools'

export default Sequence = buildSequence({ map, filter })

and then
import Sequence from './sequence'
const seq = new Sequence(iterable)

seq.map(..).filter(...)

The only additional step is that you have to "assemble"  the library at run time.

It's an interesting thought, but I'm not sold. For one it would be hard to type correctly, and it would be relatively critical that a type system understood the entire construct so that it could warn of issues, like calling a method which wasn't actually defined, or being clever by passing non-iter-tools functions to buildSequence.
Secondly I think it's hard to code. One of the big benefits of sequences is that a number of the APIs are context dependent. map as a method means something different when run on an indexed sequence (a flat iterable) vs when you run it on a entry sequence (an iterable of [key, value] pairs). In that second case it should actually transform [key, value] => [key, mappedValue]. The power of Sequences is that they can keep state internally so that they know which map logic was intended. It also means that the user couldn't just feed in a generic map function.
If you did want to make Sequence a normal part of iter-tools, it wouldn't incur it's require-the-whole-library costs unless the developer actually imported Sequence. I think then everyone might be reasonably served. The authors of libraries and modules would not use sequences in order to keep their code footprint lightweight. Those creating applications would have a straightforward choice as to whether the increased weight was worth it for the syntax.

By the way, I did settle on sequins for this, and I've got a quite a bit of it written. Repo is here.

I think this can be close now


some little fixes, improvements and tests

testAddToExistingEmptyList might be failing for BTree on the matrix test, we could remove the Btree for now from the setup,
(SoilIndexDictionaryTest class>>#testParameters)
I will then have a look independedly (the Btree code is not yet working for real with the changes)


Add support for the VBLUno51 board

Add support for the VBLUno51 board
Sarah Marsh (mbed) said that the VBLUno51 board could use board ID is C006
The slug: VBLUNO51
The VBLUno51 board was added to mbed-os 5.5.2 released
https://github.com/ARMmbed/mbed-os/pull/4629
https://github.com/ARMmbed/mbed-os/pull/4719
Signed-off-by: iotvietmember<EMAIL_ADDRESS>

@c1728p9 Could you please review this commit? Thank you very much

/morph test

@c1728p9 Thank you for your support!


Nothing happens when sending offers to the bot

Hi there,
We are using 100% original SteamBot (commit: 1cd2bd5b51ad3687aad45acbc1490ffa7297ebd4)
When we try to use the TradeOfferUserHandler class to send an offer to the bot nothing happens. For some reason the trade polling doesnt work over here. (Ubuntu 14.04 64-bit, mono 4.2.3)
I was wondering if other people had this too? I'm continuing to try and find the solution, will post it here if I do find it!
My Config:
{

  "Admins": [

  ],
  "ApiKey": "xxx",
  "mainLog": "syslog.log",
  "UseSeparateProcesses": "false",
  "AutoStartAllBots": "true",
  "Bots": [
  {
      "Username": "xxx",
      "Password": "xx",
      "DisplayName": "xx",
      "ChatResponse": "Hi there bro",
      "logFile": "betmyskins1.log",
      "BotControlClass": "SteamBot.TradeOfferUserHandler",
      "ApiKey": "xxx",
      "MaximumTradeTime": 180,
      "MaximumActionGap": 30,
      "TradeOfferPollingIntervalSecs": 30,
      "DisplayNamePrefix": "[Bot] ",
      "TradePollingInterval": 800,
      "ConsoleLogLevel": "Debug",
      "FileLogLevel": "Info",
      "AutoStart": "true",
      "SchemaLang": "en"
    }
  ]
}

Here is the log with some debug logging set for testing:
offers.AllOffers.Count 4
url http://api.steampowered.com/IEconService/GetTradeOffers/v1/?key=&get_sent_offers=1&get_received_offers=1&get_descriptions=0&language=en_us&active_only=1&historical_only=0&time_historical_cutoff=1462567598
offers.AllOffers.Count 4
[bms 1 2016-05-06 22:52:38] DEBUG: SteamBot.SteamNotifications+NotificationCallback
url http://api.steampowered.com/IEconService/GetTradeOffers/v1/?key=&get_sent_offers=1&get_received_offers=1&get_descriptions=0&language=en_us&active_only=1&historical_only=0&time_historical_cutoff=1462567628
offers.AllOffers.Count 5
url http://api.steampowered.com/IEconService/GetTradeOffers/v1/?key=&get_sent_offers=1&get_received_offers=1&get_descriptions=0&language=en_us&active_only=1&historical_only=0&time_historical_cutoff=1462567659

Update: think I got it!
https://github.com/Jessecar96/SteamBot/compare/master...peterwilli:patch-1


[qtkeychain] Support to build with less dependencies

[qtkeychain] Support to build with less dependencies

All features are tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static
x64-linux

Thanks for the feature!

Did you also build without the feature?
It seems to me without the new feature the dependency on the qt packages is completely removed (untested here).


The license for Visual Studio has expired.

Description
Azure DevOps hosted agent shows error message when building solution:
Microsoft Visual Studio 2022 Version 17.11.5.
Copyright (C) Microsoft Corp. All rights reserved.

The license for Visual Studio has expired.

The evaluation period for this product has ended."====================> Done"

Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] macOS 15
[ ] macOS 15 Arm64
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
Agent name: 'Hosted Agent'
Agent machine name: 'fv-az888-10'
Current agent version: '3.246.0'
Operating System
Runner Image
Runner Image Provisioner
Current image version: '20241021.1.0'

Is it regression?
worked before
Expected behavior
Build should succeed.
Actual behavior
Build fails.
The license for Visual Studio has expired.
Repro steps
Start a build

We're working on this , we will update you ASAP.

Hi @StefH ,
latest version of visualstudio is now available with 20241125.1  , couod you please check and confirm from your end.

It seems that the 20241125.1 is now used all the time, so no issues anymore.


Stash's Name

Every other Hashery class or module has a fairly explicit name. Stash on the other hand has a rather cryptic "playful" name. Should it be changed to something more literal? If so what?

Stash has been renamed to KeyHash for the next release (v1.5.0). Stash will stick around as an alias for the time being though.


Fix : activeIndex not changed when more than one slidePerView at the end of the swiper

When you have more than one slidePerView in the swiper and when you are at the end of it, if you click on one of the last visible slides, you don't need to slide to see them. However, it is good to change the activeIndex (to have the activeClass for example) on the clicked slide.
The initial slideIndex were changed when we check if we need to slide to see the clicked slide, therefore, the activeIndex were always the first visible slide.

Closing as outdated and incompatible with current version of Swiper


Adding OpenAPI 3.0 Support?

Hi. Any plan to support OpenAPI 3.0 specification and the new Swagger UI?

Duplicate of #419


Hint developers on how to enable debugging output

When running the generated code from Model Builder, there's not much output about what's going on, as per default the running code is not verbose.
You might want to generate the code so that  in ModelBuilder.CreateModel()  one would have the option to comment out something like
// debug ouput
// mlContext.Log += MlContext_Log;

/*
private static void MlContext_Log(object sender, LoggingEventArgs e)
{
    Console.WriteLine(e.Message);
}
*/

Thanks for reporting! That's a great suggestion!
Can you share how you used the logging or what you found helpful?

@codeclash I'm investigating adding this feature. I'd love to learn more about how you're using the logging.

Closing this due to inactivity. @codeclash if you have more details to share we would love to hear more! Please comment here or open a new issue with some more info. Thanks!


Cloud sample not working

I am trying to generate the native image from the cloud sample, but I always run into this error:
[INFO]     [creator]     Error: Classes that should be initialized at run time got initialized during image building:
[INFO]     [creator]      org.springframework.boot.context.properties.bind.Bindable was unintentionally initialized at build time. To see why org.springframework.boot.context.properties.bind.Bindable got initialized use -H:+TraceClassInitialization
[INFO]     [creator]
[INFO]     [creator]     Error: Use -H:+ReportExceptionStackTraces to print stacktrace of underlying exception
[INFO]     [creator]     Error: Image build request failed with exit status 1
[INFO]     [creator]     unable to invoke layer creator
[INFO]     [creator]     unable to contribute native-image layer
[INFO]     [creator]     error running build
[INFO]     [creator]     exit status 1
[INFO]     [creator]     ERROR: failed to build: exit status 1
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  11:53 min

I have just tested it and it seems to work on master, maybe it was someting we fixed since.


Add ability to set a responseDelay value.

Obviously more documentation and testing is needed, but I was wondering whether there would be any want for adding this feature to the main repository.
This provided to be useful when I wanted mocked calls to hang in order to test our loading icon.
mock([{
    request: { },
    response: { },
    responseDelay: 1000 // milliseconds
}]);

Hi @danieljin . It's an interesting problem. I do see some value in it, but I also see a bit of potential problems with it given the inexact nature of the setTimeout and just UI testing itself.
How was your experience with it? Did you find it worked reliably enough that you were able to verify your loading state consistently?

@atecarlos It's been working well for me. Our protractor tests that checks for the loading modal during the response delay and then checks that it's hidden afterwards is passing consistently.

Hi @danieljin . I made a minor schema change to your PR. Basically, instead of responsdeDelay, I added it to the response object. Therefore, now you define it as response.delay.
Here is the commit
This is now live in version 0.6.0

This is awesome. Thanks @atecarlos


fix(core _images.scss): revert height: auto update

using img:not([height]) adds specificity to img which can't be overridden by a class with low specificity

:tada: This PR is included in version 3.2.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Pass in the line number and keep the input stream available (for finding out the read file)

I added

sending through of the line number as well
keeping the input stream available in the object so that it's easy for listeners to read the path of the streamed file

thanks for your pull request, taking a look at it today


Failed at async committing ... null

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Ⅰ. Issue Description
When TC send branchCommit request to RM, RM log "Failed at async committing ... null"
Ⅱ. Describe what happened
AsyncWorker failed to do branchCommit, and left undolog.
Ⅲ. Describe what you expected to happen
AsyncWorker deletes undolog.
Ⅳ. How to reproduce it (as minimally and precisely as possible)
I find there are two way to instantiate the Class DataSourceManager

DefaultResourceManager uses EnhancedServiceLoader to instantiate DataSourceManager
EnhancedServiceLoader call DataSourceManager() and DataSourceManager.init(). Then register DataSourceProxy to this DataSourceManager.

DataSourceManager's member AsyncWorker call DataSourceManager.get() to instantiate  another DataSourceManager.

AbstractRMHandler uses the first DataSourceManager instantiation to do branchCommit.

The first DataSourceManager pass BranchCommit to its AsyncWorker.

While AsyncWorker uses the second DataSourceManager instantiation to get DataSourceProxy ,which no DataSourceProxy is registered.

AsyncWorker.doBranchCommit throw NPE at line158.

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

JDK version : JDK 1.8
OS : linux
Others: Fescar Version 0.4.0

@VainEffort please dingtalk ping me.

do u  use Tcc which contained At ？

mark as bug

can u provide a pr to fix it ？


Feature Request: Add Dash Line

Dash line is very useful in many cases.

duplicate of https://github.com/excalidraw/excalidraw/issues/1129


Ignore existing option does not work

🐛 Bug Report
When converting svgs to react-native components using a config file, the ignoreExisting flag is not working correctly.
Link: Ignore Existing Option
To Reproduce

Create a config file (.svgrrc.js) that includes the ignoreExisting: true option.
Place svgs in source folder
Create destination folder
Run npx @svgr/cli src -d dest
In the resulting output files, make any kind of change to the js.
Run npx @svgr/cli src -d dest again.
Observe your changes from step 4 are now gone since the file was regenerated.

Expected behavior
If the ignoreExisting flag is set to true then any changes I make to an outputted file should not be overwritten by subsequent executions of npx @svgr/cli src -d dest
Link to repl or repo (highly encouraged)
https://github.com/SamMcDuffie/SVGRIssueRepo
## System:
 - OS: macOS 11.6
 - CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
 - Memory: 270.25 MB / 32.00 GB
 - Shell: 5.8 - /bin/zsh
## Binaries:
 - Node: 16.10.0 - /usr/local/bin/node
 - Yarn: 1.22.15 - /usr/local/bin/yarn
 - npm: 7.24.0 - /usr/local/bin/npm
 - Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
## npmPackages:
 - @svgr/cli: ^5.5.0 => 5.5.0

Fixed in v6.


only show enemy power when hovering nodes

What do you mean by "only energy?"
If you are talking about stored energy, it is only displayed if it is greater than 0.

If you are talking about that energy table prevents you from seeing resources in generators and factories, I just fixed it


feat(carflag/webhook): add carLabel

Get the label of the car in the webhook instead of the hash

I'm curious, would this change work for the weapons too? changing the hash value into a text value like "WEAPON_"

I'm curious, would this change work for the weapons too? changing the hash value into a text value like "WEAPON_"

Since this uses the native GetDisplayNameFromVehicleModel, it isn't possible, as far as I know.

Tested working? 😄

Tested working? 😄

Of course! 😛


Any way to move the horizontal scrollbar below the footer row??

Not currently, no.  If we could have put it below the footer we would have.

Is it a big issue achieving that? I was trying my hands but was not successful. Has anyone else achieved that?

I agree with himsraw. Why is this so technically challenging? Most tables I've used have the scroll below the footer.


[IRGen] Validate that compiler crashes while building PythonKit

Reports SR-15871. This crash is affecting the build process for compiling philipturner/swift-for-tensorflow in release mode, although I can work around it by removing PythonKit.

This crash also blocks a planned refactoring to Swift-Colab. Until the bug fix is incorporated into a Swift release toolchain, I may not be able to utilize lambdas and subclassing to remove my "SwiftPythonBridge" module.
However, if this crash disappears on -Onone, I should be fine. I use release with -Onone internally because it's the fastest compilation path.