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/511 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target.git

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

File
Status
Preview URL
Details

:x:Error

Details

[Error-BackendEngineExceptionError] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in https://SiteHelp and include the URL of publishing report as well as below information: One or more errors occurred..

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Update livability score, add new features to get data endpoint

[Livability Score] (https://trello.com/c/Lh4R0aFh)
Location

ml.py
db.py

Details

updated get_data endpoints to include transit, bike, and school data
include this data in livability score

Committed Features

user gets updated livability score which includes new data points

 DS will create endpoints for both web and iOS, utlizing the new data sets.

 DS will update,livability score attached to each city to include added features


Strike Select - Need the ability to disable the showing of the placeholder

Currently the place holder always exists on strike select. If I want to set the select to a preselected value it'll cause a flicker during rerender cycles. It'd be nice to able to just not have the placeholder if we don't need one.

The default placeholder text has been removed and you should see the effect take place in out next release.


This project vs com.unity.nuget.newtonsoft-json

(Last updated: 2022-03-03)
Is this package compatible with com.unity.nuget.newtonsoft-json?
No. Sadly not. Reason for this is the combination of:

Cannot have multiple .NET assemblies with the same name in a project at the same time.

All files in Unity's and my package has the same GUIDs (in their .dll.meta, .md.meta, .asmdef, etc).

Plans on making it compatible with com.unity.nuget.newtonsoft-json?
There's no plan in making them compatible. There's too many techincal limitations in play here that makes it practically impossible.
There are however plans on resolving this with some Unity devs. I've had a years long discussion with some Unity developers on the topic almost since the dawn of this fork. Fun trivia: the same discussion that com.unity.nuget.newtonsoft-json@v2 was created from to begin with.
Different with this project and com.unity.nuget.newtonsoft-json?
Unity's "official" Newtonsoft.Json package, starting with version 2.0.0-preview.1, is a fork of this repo. They have practically "just" republished my fork as their own.
Just so there's no confusion: It's in their full right to do so. And in fact, one of their super kind developers reached out beforehand just to confirm it would be OK with me, something they didn't have to do, but I really appreciate that they did.
Is com.unity.nuget.newtonsoft-json safe to use?
Edit 2022-03-03: Since late February 2022, starting with v3.0.0 of their package, Unity has now committed to maintaining their fork to keep it up-to-date with the upstream JamesNK/Newtonsoft.Json, providing customer support, as well as publishing their package publicly inside Unity's official UPM registry.
They are just about to make it public, but until then, you can follow my guide on install the official Newtonsoft.Json UPM package over at my wiki.
Latest news on Unity adopting jilleJr/Newtonsoft.Json-for-Unity for real?
Edit 2022-03-03: It is now a reality! My fork can now be considered dead, in favor of Unity's package, starting with v3.0.0<EMAIL_ADDRESS>For the long run, Unity is working hard on the .NET 6 adoption and for the long run are looking for using .NET's built in System.Text.Json package instead, leaving end-users free to choose between Newtonsoft.Json or System.Text.Json. Read more: https://forum.unity.com/threads/unity-future-net-development-status.1092205/
What's the future of jilleJr/Newtonsoft.Json-for-Unity?
Edit 2022-03-03: I am officially deprecating this package in favor of Unity's package, as stated above.
I will stop providing:

Bug fixes
New features
New documentation

However, I will still provide:

Support from questions asked in discussions or issues. However, please refer to the https://forum.unity.com/ or their issue tracker for better support by Unity employees.
Package downloads for existing versions. No need for me to take down the Cloudsmith package or the branches here on GitHub. They will live on, as long as Cloudsmith and GitHub keeps living on.

I was using jilleJr for json in an old Unity project (year or so ago), and I've come back to the project recently. I was interested in updating the project's Unity editor version from 2019.4.12f1 LTS to 2020.3.26f1 LTS. However, I noticed com.unity.nuget.newtonsoft-json was now in play with the updated Unity editor.
Am I correct in assuming there's no way to remove com.unity.nuget.newtonsoft-json package since it's required by the latest editor?

Am I correct in assuming there's no way to remove com.unity.nuget.newtonsoft-json package since it's required by the latest editor?

Yeah, correct.
In detail: while the core Unity editor itself does not depend on com.unity.nuget.newtonsoft-json, there are so many of the basic packages that does depend on it.
If you were to empty out your Packages/manifest.json, then no com.unity.nuget.newtonsoft-json would be loaded. But that's extremely unpractical. I do not know which packages exactly that depend on it, but all I know is that there are multiple ones that do.

Very sad news. 😢
Unity requires Editor 2020.3+ for using their multiplayer Netcode solutions, so I'm in a hard spot.
From what I can tell, Unity's built in JSON utilities (aka UnityEngine.JsonUtility.FromJson) cannot deserialize many JSON strings that are supported by Newtonsoft.Json. For example, strings with newlines.
Sorry to ask about "competitors", but do you know of any other JSON tooling for Unity that may be alternatives?

@scott-lin hey don't worry! It's a valid question.
A few years ago I researched and created a list of alternatives. So mind you, it's probably quite out of date. But here they are:

OdinSerializer github.com/TeamSirenix/odin-serializer
UTF8Json github.com/neuecc/Utf8Json
FastJSON assetstore.unity.com/packages/tools/input-management/fastjson-27220
UltimateJson assetstore.unity.com/packages/tools/integration/ultimate-json-60845
SimpleJson wiki.unity3d.com/index.php/SimpleJSON
LitJson github.com/Mervill/UnityLitJson
FullSerializer github.com/jacobdufault/fullserializer

Or if you don't mind changing format, trying out XML might be a valid solution as .NET has had built in support for ages:

System.Xml docs.microsoft.com

Or if you want to try other binary formats, then maybe something like Cap'n Proto,  Google's protobuf, or Apache Thrift might be good:

Cap'n Proto capnproto.org
protobuf-net github.com/protobuf-net/protobuf-net
Apache.Thrift thrift.apache.org

I've not tried any of the above packages, so you'll have to verify their Unity and IL2CPP support yourself. Hope you find a solution!

Big news: Unity has now officially adopted the package
I've updated the issue description with more information


VTGate - Execute Batch V1

@sougou
This is an initial PR, contains ExecuteBatch API flow.
TODO (In next Commits):

Code Refactoring (Renaming to meaningful names with proper comments)
Test Cases
Remaining Implementation

This PR is open for suggestions.

This change is

What BeginExecuteBatch does?
https://github.com/youtube/vitess/blob/master/proto/query.proto#L454

Review status: 0 of 25 files reviewed at latest revision, 4 unresolved discussions, some commit checks broke.

proto/vtgate.proto, line 272 at r1 (raw file):
Previously, alainjobart (Alain Jobart) wrote…
Let's remove this field.

Done.

proto/vtgate.proto, line 274 at r1 (raw file):
Previously, alainjobart (Alain Jobart) wrote…
Maybe update this comment:
// default keyspace to target the queries to.

Done.

proto/vtgate.proto, line 279 at r1 (raw file):
Previously, alainjobart (Alain Jobart) wrote…
How about adding as_transaction in there too? Even if we don't support it in all cases just yet...

Done.

proto/vtgate.proto, line 286 at r1 (raw file):
Previously, alainjobart (Alain Jobart) wrote…
I think this field and the result should be combined in a new structure, and this should contain an array of it. And it should probably be defined in query.proto, as you'll need it there for the vttablet API anyway.
To be clearer, in query.proto, define:
message QueryResultOrError {
vtrpc.RPCError error = 1;
QueryResult result = 2;
}
(that message name is not very good!)
And this should be:
message ExecuteBatchResponse {
repeated query.QueryResultOrError results = 1;
Session session = 2;
}

We can do vttablet side changes in different PR as those changes are not part of version 1.
Also, we agreed that we will add execute_all flag in the existing ExecuteBatch in query.proto.

Comments from Reviewable

Gentle Reminder..

Review status: 0 of 28 files reviewed at latest revision, 7 unresolved discussions.

go/vt/vtgate/vtgate.go, line 384 at r11 (raw file):
Previously, sougou wrote…
Delete commented out section.

Done.

proto/vtgate.proto, line 260 at r11 (raw file):
Previously, alainjobart (Alain Jobart) wrote…
No sorry, it's ResultWithError that should be in query.proto, ExecuteBatchRequest and friends should stay in vtgateservice :)

Done.

proto/vtgate.proto, line 291 at r11 (raw file):
Previously, sougou wrote…
This name was too ambiguous. Let's go with ResultWithError instead.

Done.

Comments from Reviewable

Looks like you have to resolve a merge conflict after all.

LGTM


Support PACKAGE target/CPack integration

Brief Issue Summary
Currently cmake-tools doesn't show the PACKAGE target if CPack is enabled on the project, nor does it have any commands for running this target or CPack.
I'd like to be able to see and build the PACKAGE target from withing VS Code (apart from launching a terminal and using the command line).

maybe related to https://github.com/microsoft/vscode-cmake-tools/issues/543 ?


Fix path error in daemon mode

Codecov Report

Merging #30 into master will decrease coverage by 1.61%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
- Coverage   64.80%   63.18%   -1.62%
==========================================
  Files          29       29
  Lines        2563     2483      -80
==========================================
- Hits         1661     1569      -92
- Misses        902      914      +12

Impacted Files
Coverage Δ

turbo_tunnel/ssh.py
55.29% <71.42%> (-5.93%)
:arrow_down:

turbo_tunnel/utils.py
68.44% <0.00%> (-4.23%)
:arrow_down:

turbo_tunnel/socks.py
43.36% <0.00%> (-2.40%)
:arrow_down:

turbo_tunnel/conf.py
78.76% <0.00%> (-1.62%)
:arrow_down:

tests/test_ssh.py
98.91% <0.00%> (-1.09%)
:arrow_down:

turbo_tunnel/chain.py
65.62% <0.00%> (-1.05%)
:arrow_down:

turbo_tunnel/server.py
73.15% <0.00%> (-0.88%)
:arrow_down:

turbo_tunnel/https.py
55.46% <0.00%> (-0.74%)
:arrow_down:

turbo_tunnel/tunnel.py
64.68% <0.00%> (-0.52%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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


Update strings.xml

Updating subscription key URL.

Hi @bhansen3, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Birgit Hansen (Teksystems)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@bhansen3, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

@bhansen3, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

@bhanse3, Thanks for the contribution. The code has been merged.


Pass additional options to underlying Typhoeus client

My particular need was to use custom proxy option, but there are various other low-level options one may need to pass. This minor change allows passing custom request options straight to underlying Typhoeus client. You may then use it in the code like this:
config = AmzSpApi::SpConfiguration.new
config.request_extra_opts = {
  proxy: "http://user:pass@host:port",
  ...
}
...
client = AmzSpApi::SpApiClient.new(config)

oh wow indeed, I didn't realize this file is regenerated each time. Makes sense! I will close this for now


Df 376 tracking when a user sorts results

https://national-archives.atlassian.net/browse/DF-376
About these changes
Removing whitespace in the search-type attribute
How to check these changes
Perform a search
Switch to catalogue results
View page source and check that there is no whitespace in  search-type attribute
Where possible, provide guidance to help your reviewer
Before assigning to reviewer, please make sure you have

[x] Checked things thoroughly before handing over to reviewer.
[x] Checked PR title starts with ticket number as per project conventions to help us keep track of changes.
[x] Ensured that PR includes only commits relevant to the ticket.
[x] Waited for all CI jobs to pass before requesting a review.
[x] Added/updated tests and documentation where relevant.

Merging PR guidance
Follow docs\developer-guide\contributing.md
Deployment guidance
Follow docs\infra\environments.md

Hi @TNA-Allan , I can see from the console that the event details are being pushed to the dataLayer

Hi Allan,
It looks like these two have been merged and closed, I don’t think there is further action on these.
John
From: TNA-Allan @.>
Sent: 28 April 2023 09:00
To: nationalarchives/ds-wagtail @.>
Cc: Heery, John @.>; Mention @.>
Subject: Re: [nationalarchives/ds-wagtail] Df 376 tracking when a user sorts results (PR #757)
EXTERNAL: This email originates from outside of The National Archives.
@JohnHeeryTNAhttps://github.com/JohnHeeryTNA
There seems to be two related PR's that are merged

#779https://github.com/nationalarchives/ds-wagtail/pull/779
#572https://github.com/nationalarchives/ds-wagtail/pull/572
Is this PR still active for review? Can you check and close it if its no more needed?

—
Reply to this email directly, view it on GitHubhttps://github.com/nationalarchives/ds-wagtail/pull/757#issuecomment-1527150543, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A2L2C3GBERWFKVXBAOCT6C3XDN2JDANCNFSM6AAAAAAWKJPGXU.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>Please don't print this e-mail unless you really need to. ----------------------------------------------------------------------------------- National Archives Disclaimer This email and any files transmitted with it are intended solely for the use of the individual(s) to whom they are addressed. If you are not the intended recipient and have received this email in error, please notify the sender and delete the email. Opinions, conclusions and other information in this message and attachments that do not relate to the official business of The National Archives are neither given nor endorsed by it. ------------------------------------------------------------------------------------

Closing as out of date since subsequent work


adding new setup.py

This makes afp-cli pip+git installable:
e.g. for this branch/pull-request, you can try:
pip install<EMAIL_ADDRESS>

The pybuilder generated file seems buggy:
vhaenel@isdeblnnl086 ~/gw/afp-cli [afp-cli:feature/setup.py:★★] % pip install<EMAIL_ADDRESS>Downloading/unpacking<EMAIL_ADDRESS>  Cloning git://github.com/esc/afp-cli.git (to feature/setup.py) to /tmp/pip-hETXYo-build
  Running setup.py (path:/tmp/pip-hETXYo-build/setup.py) egg_info for package from<EMAIL_ADDRESS>    Traceback (most recent call last):
      File "<string>", line 17, in <module>
      File "/tmp/pip-hETXYo-build/setup.py", line 43, in <module>
        except FileNotFoundError as e:
    NameError: name 'FileNotFoundError' is not defined
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):

  File "<string>", line 17, in <module>

  File "/tmp/pip-hETXYo-build/setup.py", line 43, in <module>

    except FileNotFoundError as e:

NameError: name 'FileNotFoundError' is not defined

----------------------------------------
Cleaning up...
Command python setup.py egg_info failed with error code 1 in /tmp/pip-hETXYo-build
Storing debug log for failure in /data/home/vhaenel/.pip/pip.log

I can confirm that the fix submitted to pybuilder works, so I'll merge this here and now.


Want Close Folder button in Explorer control

It is unnecessarily tedious to Close a Folder, to have to perform: Alt > File > Ctrl+K F. This is not the easiest hotkey (or "chord") sequence, and the whole action sequence requires use of the keyboard. A pure mouse option would be easier.
When NO folder is open, the Explorer panel offers a prominent blue button labeled Open Folder. Good.

My Request:   When a folder IS open, offer a button labeled Close Folder.

Close a folder is very different than Hide the Explorer panel, obviously. I am asking for Close folder, not Hide.

This will make it easier to Close Folder when a folder is open.

Loosely Related Issue is 94755:   https://github.com/microsoft/vscode/issues/94755

https://github.com/microsoft/vscode/issues/94755
Title:   'Add "Close folder" on Welcome screen (if a folder is open)'

Thank you. Gene Milener (GeneMi at Msft = MightyPen) , 2020/May/01
. .

Please assign a custom keybinding to this command, via File > prefernces > keyboard shortucts
You can also write an extension to contribute this command whenever you want.
We will not change the default built in experience for now, thus closing this. Thank you


Components defined in a super type do not get initialized for the instance creation and instance creation fails

Components defined in a super type do not get initialized for the instance creation and instance creation fails

Was this fixed recently? Last night I've tested and it wasn't working.

Should be fixed in main now


Add Images to the Main Page of the Website

The main page of the Tic Tac Toe website currently lacks images, which could enhance the visual appeal and user engagement. Adding relevant images can help provide a more engaging and user-friendly interface.

Expected Behavior: The main page should include relevant images, such as a Tic Tac Toe board illustration or icons next to the game mode options, to create a visually appealing user experience
I would like to work on this issue
@iamparas0 please assign it to me under GGSOC'24EXTD, hacktoberfest'24 and level label

@iamparas0 please assign it to me under GGSOC'24 EXTD,hackofest'23,and level label


[WIP] Add links to the API in documentation

No code changes - just documentation

Looks great! :star2:
If we can have @see  marks then we can update the https://github.com/octokit/go-octokit/blob/master/TODO.md list just like https://github.com/octokit/octokit.rb :+1:
Elaborated here:
https://github.com/octokit/go-octokit/pull/79

As per a discussion in person, @see notifications have been deemed unnecessary as they are ruby documentation. The TODO.md file should be able to be updated automatically without them.

:+1:

@dannysperling Looks like we have a merge conflict on this one. Need help resolving that?

Missed your comment earlier. I've fixed the conflicts and this endpoint should be ready to merge now.


Flat cut case design

Would it be possible to get the stl from the japanese guy that did the acrylic cut case?

You should ask him directly, I do not have his files.
You should also note that he used https://github.com/mattdibi/redox-keyboard/blob/master/redox/case/redox_rev1-brd.dxf as a base... it shouldn't be hard to reproduce for your particular machine.
Hope this helps!

Closing as resolved.


Update examples link

Change the example link from archived repo (https://github.com/vulkano-rs/vulkano-examples) to the examples folder which has been updated

Updated README example URL

Thanks


logitech options

good evening
do all the options of hostinfo output work in the plugin?
so far i just added a few which seem to work, but some others dont work like 3D or Smart
"commands": [
{
"action": "{"command":"PowerOff","type":"IRCommand","deviceId":"xxxxxxxx"}",
"name": "PowerOff",
"label": "PowerOff"
},
{
"action": "{"command":"PowerOn","type":"IRCommand","deviceId":"xxxxxxxx"}",
"name": "PowerOn",
"label": "PowerOn"
},
{
"action": "{"command":"InputTv","type":"IRCommand","deviceId":"xxxxxxxx"}",
"name": "InputTv",
"label": "InputTv"
},
{
"action": "{"command":"InputHdmi1","type":"IRCommand","deviceId":"xxxxxxxx"}",
"name": "InputHdmi1",
"label": "InputHdmi1"
},
{
"action": "{"command":"InputHdmi2","type":"IRCommand","deviceId":"xxxxxxxx"}",
"name": "InputHdmi2",
"label": "InputHdmi2"
},
{
"action": "{"command":"InputHdmi3","type":"IRCommand","deviceId":"xxxxxxxx"}",
"name": "InputHdmi3",
"label": "InputHdmi3"
},
{
"action": "{"command":"3D","type":"IRCommand","deviceId":"xxxxxxxx"}",
"name": "3D",
"label": "3D"
}
also is it possible to get siri to action these? what are the commands

Hi @rossi2000, thanks for trying out the plugin! These are great questions.

do all the options of hostinfo output work in the plugin?

It depends on what you mean by “work”. None of the output of hubinfo.js should break the plugin, but some of it may not be supported by a. the plugin or b. HomeKit itself (yet).

some others dont work like 3D or Smart

I don’t own a 3D TV, so I can’t test this personally. Does your TV treat 3D like a separate input?
If so, you can make homebridge-harmony-tv-smockle treat it like an input by replacing
{
  "action": "{"command":"3D","type":"IRCommand","deviceId":"xxxxxxxx"}",
  "name": "3D",
  "label": "3D"
}

with
{
  "action": "{"command":"3D","type":"IRCommand","deviceId":"xxxxxxxx"}",
  "name": "Input3D",
  "label": "Input3D"
}

If your TV treats 3D as a toggleable feature that applies to all inputs, AFAIK that’s not supported by HAP-NodeJS or HomeKit itself.

is it possible to get siri to action these

According to this 9to5mac article, the answer is currently no:

Siri control for switching inputs outside of scenes is missing for now, but it seems like a feature we could easily see in the future. For now, Siri is limited to turning TVs on and off and activating pre-configured scenes that include HomeKit TVs.

Hi mate
I added input to 3D then it showed in the HomeKit app but it didn’t do anything.


Separate releases for 1.5.x and 1.6.x

#202 is the first change that does not apply to all NCS releases, it only applies to v1.5.x.
Update the release process of the documentation so it uses a branch based model instead of providing the documentation for all NCS releases from the same source code.

Solved via conditionals for now: c973402c1678a4d6da40d69aae9766fa6425c8bc


v3 host.Names use equivalent service and machine names

This change is

Pull Request Test Coverage Report for Build 1405
Details

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.009%) to 94.531%

Totals

Change from base Build 1377:
0.009%

Covered Lines:
2368

Relevant Lines:
2505

💛  - Coveralls


Android 4.1 Browser

It's incredibly important that this wallet works on older browsers. QR code scanning and everything.
Specifically Android 4.1
Right now I got it working past the SSL part but it loads a blank page.

This issue should be related to some breaking changes introduced in<EMAIL_ADDRESS>
Node 6 is no longer supported
Please upgrade to Node 8 (LTS) or later.

Polyfills for IE 9, IE 10, and IE 11 are no longer included by default (but you can opt in!)
We have dropped default support for Internet Explorer 9, 10, and 11. If you still need to support these browsers, follow the instructions below.

First, install react-app-polyfill:

npm install react-app-polyfill
Next, place one of the following lines at the very top of src/index.js:

import 'react-app-polyfill/ie9'; // For IE 9-11 support
import 'react-app-polyfill/ie11'; // For IE 11 support

https://github.com/facebook/create-react-app/releases?after=react-dev-utils%406.0.4
I experienced the same issue testing on Android 6.0
There is another restriction in package.json
  "browserslist": [     ">0.2%",     "not dead",     "not ie <= 11",     "not op_mini all"   ]

SSL is also an issue. Right now when we host the code in cloudfront, it uses SNI for SSL and that breaks old phones. It's much more expensive to have custom SSL so the solution might be to start running our own webservers to host the static content and use Let's Encrypt for SSL.


Update header.vim

support coldfusion or CFML.

Thank you!

I have the latest update on cf in my forked repository. Do I need to pull
the request ?
On Mon, Sep 6, 2021 at 7:16 PM H.Alper Tuna @.***>
wrote:

Thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alpertuna/vim-header/pull/84#issuecomment-913940979,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOT3BCB6BZTBF27ZT4UTH3UAVYXNANCNFSM4ZJZ656Q
.

If your fork works just fine, then you dont need to. But I recommend to pull for the latest other updates or you can also use this repo directly.


Parallel shards training

training shards in parallel for performance

Not necessary for unlearning task, since the key metric for unlearning is retraining time and HR/NDCG, rather than raw model training time.


Let ColorSettings Stub fail more gracefully if not present

Add better warning and allow hex text to be visible if user does not have color settings library installed and enabled.

Decided to just implement color library dependency instead.


project requires a paying subscription to openai

It maybe good to state that this project requires a paying subscription to openai to use this project.  This is stated in:
https://platform.openai.com/docs/models/gpt-4-and-gpt-4-turbo
Running the project I got the following error:
2023-12-23 11:19:49,095 - ERROR - OpenAI API error: The model gpt-4-1106-preview does not exist or you do not have access to it. Learn more: https://help.openai.com/en/articles/7102672-how-can-i-access-gpt-4.

Good point. This should be updated in the readme, agreed.
thank you for your feedback!

Addressed in the following pull request: PR #6 - Closing issue.


pip installation is failing

Collecting jiwer
  Downloading https://files.pythonhosted.org/packages/c7/fd/88639901195f2625941efdf2a1496c540b3390149
9a986fb271af28e4436/jiwer-1.3.2.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/home/sausharma/temp/pip-build-sMieMl/jiwer/setup.py", line 6, in <module>
        with open(path.join(this_directory, 'README.md'), encoding='utf-8') as f:
    TypeError: 'encoding' is an invalid keyword argument for this function

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /home/sausharma/temp/pip-build-sMieMl/
jiwer/

ooh so it required python3, dependency isn't mentioned anywhere
Closing the issue


Add doc_markdown clippy linting config to cargo workspace

Objective
Partially Addresses #10612
fix: add clippy::doc_markdown linting to cargo workspace
Rather than do all the warnings in tools/ci/src/main.rs in one-shot, just wanted to have an initial pr adding the first one to get the form correct as some may trigger build errors and require changes to get merged more easily.
Solution

adding the doc_markdown and removing it from the ci check as it'll now be a build error during normal compilation.

oh great! this works now with the ci tool locally. For some reason i thought the set of flags passed to clippy would overwrite the cargo config, but looks like they are merged with the cargo clippy lints.


[AutoPR logic/resource-manager] Bugfix for swagger/sdk not matching the underlying service API

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4769

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2782


Index Volumes screen limit issues

If I specify a limit of 1.9T, then the limit is shown as 1.8T

The issue seems to be that the conversion of an IEC byte size string <=> bytes is no reversible as when you convert back from bytes it is always flooring the value to 1 decimal point.  This leads to a potential  reduction of the value each time you convert.

Fixed in >7.0.3


fix(apps/prod/tekton/configs/tasks): update task create-pr-to-bump-tikv-version

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


Support squashfs as compression algorithm

I'm creating a draft PR to let you know that I'm working on #6273 (and also to get that hacktoberfest t-shirt ;)). The squashfs-tools-ng has trouble compressing/decompressing tarballs created by LXD. So, I'm trying to resolve that before I can continue with this.

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

I'm closing this as I've opened another PR #6356


Feat: Onboard covid19 jhu csse dataset

Description
covid19_jhu_csse.deaths
covid19_jhu_csse.confirmed_cases
covid19_jhu_csse.recovered_cases
Checklist
Note: If an item applies to you, all of its sub-items must be fulfilled

[x] (Required) This pull request is appropriately labeled
[x] Please merge this pull request after it's approved
[x] I'm adding or editing a dataset

[x] The Google Cloud Datasets team is aware of the proposed dataset
[x] I put all my code inside  datasets/covid19_jhu_csse and nothing outside of that directory

[x] I'm refactoring or cleaning up some code

@adlersantos @happyhuman - I am unable to set the reviewer in this PR for some reason.  It is ready for code review and I have closed Vasu's original PR.  Please review! Thanks.


Why do I throw SQLITE_BUSY exception when closing the database.

import { open } from 'sqlite';
import { Database } from 'sqlite3';

(async function (){
  const db = await open({
    filename: './base.db',
    driver: Database
  });

  const result = await db.prepare(
    "insert into user(nickname, email, age, gender, role) values (?, ?, ?, ?, ?);"
  ).then(statment=>{
    return statment.run(["zhanshang"<EMAIL_ADDRESS>18, 1, "ADMIN"]);
  });

  console.log('result=>', result);

  try{
    await db.close();
  }catch(e){
    console.error(e);
  }

})();

output
result=> {
  stmt: Statement { stmt: Statement { lastID: 5, changes: 1 } },
  lastID: 5,
  changes: 1
}
[Error: SQLITE_BUSY: unable to close due to unfinalized statements or unfinished backups] {
  errno: 5,
  code: 'SQLITE_BUSY'
}
Done in 1.62s.

only one process opened the file at the same time。

only one process opened the file at the same time。

You're mixing promises and await. You need to stick to using one or the other since close will execute before your statement in this case.


Revert env var change

Codecov Report

Merging #57 into master will not change coverage.
The diff coverage is 90.9%.

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


BUG: Relationships - set to List Setting - when Status changes to Former from Current, Type is set to Neutral value of row

Hub thread: https://powerofus.force.com/0D580000027HLFb
Replicated in test org. When changing status, on Save the Type is updated to the row's Neutral value.

Still can't replicate.   What is the gender set to?

Interesting...
Here's what I also put in the thread:
Gender field blank. Created 2 new Relationship rows, one duplicating Beth's use case (Public Health Nurse, Client, Client, Client). Updating Status changed the Type from Public Health Nurse to Client.
So I created a second relationship row with Original Name, Original Male, Original Female and Original Neutral to make it easier to see exactly what was happening. Confirmed that the type changed to Original Neutral on save.
All created values were added to the Relationship Type picklist.
Both contacts are Household contacts - NPSP 3.39 as vanilla as I can make it.
Account Model is Household Account.

Weird.  I have 3.39 vanilla.  Gender and salutation defined.  It works fine.
Tried just the gender per Beth's comment, and it still works as expected.  Lets get someone else from the dev team to try it out.

Okay, think we have it now:
If an update fires and the base relationship gender is unknown, the current behavior will update the type based on the new 'Neutral' gender.
Fix here is to simply ignore a change only to the status instead of reevaluating the relationships.

Hi guys! Glad this got figured out! Any ETA on the change/fix?

@bethbbc next release!

Awesome!!! Thanks!
On Thursday, June 11, 2015, Nicolas Campbell<EMAIL_ADDRESS>wrote:

@bethbbc https://github.com/bethbbc next release!
—
Reply to this email directly or view it on GitHub
https://github.com/SalesforceFoundation/Cumulus/issues/1564#issuecomment-111291494
.

Hi @bethbbc, this actually didn't make it in today as the fix I implemented didn't work correctly. Digging further it looks like quite a bit more of the relationships code isn't working as expected, which I'll be working on this release.

Thanks for the update, Nicolas!! Greatly appreciated. Let me know if you
want/need any help testing.
On Tue, Jun 23, 2015 at 9:01 AM, Nicolas Campbell<EMAIL_ADDRESS>wrote:

Hi @bethbbc https://github.com/bethbbc, this actually didn't make it in
today as the fix I implemented didn't work correctly. Digging further it
looks like quite a bit more of the relationships code isn't working as
expected, which I'll be working on this release.
—
Reply to this email directly or view it on GitHub
https://github.com/SalesforceFoundation/Cumulus/issues/1564#issuecomment-114556866
.

Included in beta release version 3.48 (Beta 4) and higher

Included in production release version 3.48


Look into odd behaviour of vein mining.

Reduce initial vein mining depth
Look into reports of instamine
Look into alternate algorithms for vein mining trees.

Hi, I'm Just adding to the reports of being able to instamine with ore mining pickaxe heads - tried oak, iron, diamond.
I'm using forgero-fabric-0.10.9+1.19.2

Fixed


Ucan-core refactor

Problem
Several problems are addressed by this refactor:

the ucan library required you to pull in several dependencies. This may be undesirable for high-security environments, the specific dependencies may not work in a certain environment (one-webcrypto in react-native for instance), and developers may want to choose a different crypto implementation
there was no was to do DID method resolution outside of did:key and the library only understood did:keys that had been hard coded in

Closes https://github.com/ucan-wg/ts-ucan/issues/77
Solution
Refactor the library into three pieces

@ucans/core: a 1-dependency (I didn't get rid of uint8arrays yet 😛) library that contains the core logic for working with ucans
@ucans/plugins: a set of basic default "plugins" for the ucan library: RSA, NIST P-256, and ED25519
@ucans/ucans (still workshopping the name on this one): Loads the default plugins into @ucans/core & re-exports core + plugins

Implementation notes
Take a look at core/plugins & each of the plugin files in the plugins package to get an idea of the semantics.
You'll notice that I handle did:key differently from the other methods, because it's such a common use case.
I ended up doing a global state variable for plugins. If you remember from the issue, this wasn't my original plan and I frankly still don't love it. But I was having to thread the plugins through a lot of functions, which mean re-exporting all those functions in the top-level package, and some of those it was difficult, because static methods on classes needed them, etc. I figured it was much more straightforward to do it in this manner. Anyone that wants to load in other dependencies would likely wrap their implementation similarly to our top-level.
I left uint8arrays in core. I'd like to make it 0-dependency, but I didn't want to go through re-creating all the encoding fns just yet. Also the library is pretty lightweight & comes from a trusted source. We can switch it out later if we feel so inclined.
I got rid of the KeyType type. These mapped 1:1 with jwtAlg and I figured that we could simplify by just using jwtAlg.
I also got rid of the BaseKey class, and just have each keypair implementing the interfaces directly. It was causing more headaches than good. I also removed the required publicKey on the Keypair interface. This can lead to some confusion with, for instance, P-256 keys. Is it the compressed or decompressed public key? We don't really need it anywhere in the code, so I removed it to avoid that confusion.
Tests of crypto functions are in the plugins library. I left the rest of the tests in core. Most of the tests don't make a ton of sense without being able to use some crypto functions, so I import @ucans/plugins as a devDependency and load it in for core tests. I don't love this approach, but it was the best I saw for now. Let me know if you have other ideas!
I mostly left the configs & build systems untouched. I know a bunch of trial & error has gone into them & my brain glazes over when I try to mess with build systems 😅. However, this did lead to a good bit of repeat code. There's a tsconfig.json, jest.config.js, and tsconfig.eslint.json in each package root and at the repo root. It would be nice to remove that duplication. Similarly, I just copied over the package.json to each package, with some minor edits. I left the package version where it's at on all of them (0.9.1, though I figure this refactor will bump it to 0.10.0). I'm very amenable to any suggestions/recommendations on any of this stuff.
I briefly considered splitting each cryptosystem out into its own library (instead of all in one plugins library), but decided not to proliferate the packages too much at this point. However, to mimic this separation of concerns, I split up the package by crypto system instead of function type (before it was crypto/, keypairs/, etc; now it is p256/, ed25519/, etc).

Thanks for helping plan this PR! Very excited to get it going. I hope the general direction I took makes sense, but happy for any feedback on it/changes requested.
If it's cool with yall, I'd like to publish an alpha version of core that we can start messing around with internally :)

Wohooo! Nice.
Some minor bikeshedding before I deeply look into this:
What do you think of naming it @ucans/base-plugins or @ucans/default-plugins?
Or perhaps something including the workd "did"? @ucans/base-did-support? Might be too wordy.
To outsiders I worry the first impression of @ucans/plugins would be that it's some kind of plugin system, not the actual plugins themselves.

Oh and - by the way - does<EMAIL_ADDRESS>still make sense? I guess it should be something @bluesky now.

ah yup thats a good point. i like @ucans/default-plugins 👍
ah right. hmm i mostly use my personal email even with work things lol. I'll switch it over to either bluesky or personal 👌

I'll take another look at the plugins abstraction. I'd also like for it not to be a global var. Reducing the API surface will certainly help. Big fan of eliminating the DID utility fns.
I think I can make threading the plugins through work. If it really looks like it's not shaking out, I'll give a shout & we can reassess.

The monorepo is giving my VSCode a hard time 😅 What setup are you using?
Is eslint working for you? For me it fails on the .eslintrc.js, because it links to ./tsconfig.eslint.json file which was moved.

Oops! That's on me. I had the same issue, but readded tsconfig.eslint.json at the root & it fixed it but forgot to push it!

VSCode shows me errors on import statements such as import ... from "@ucans/core". Apparently it can't resolve @ucans/core. Does that work for you?

This does work for me. It's a little bit fickle. Make sure you run yarn from the monorepo root to link the packages together. And then make sure you've done yarn build in @ucans/core. If you still have the red underline, then use the command prompt (cmd + shift + p) & type in "Restart TS Server".
Thanks for the feedback! Glad this roughly feels like the right direction. I'll get this patched up in the next day or two 👍

Make sure you run yarn from the monorepo root to link the packages together. And then make sure you've done yarn build in @ucans/core. If you still have the red underline, then use the command prompt (cmd + shift + p) & type in "Restart TS Server".

Ah! That was exactly it. That, and running yarn build in the plugins package. Perfect, thanks!

Started on simplifying tooling by moving jest config to the root: https://github.com/ucan-wg/ts-ucan/pull/82/commits/62d07c08e5a6cdf55168c165683363d470a59e03
I tried to do the same with tsconfig, but I'm running into hiccups because of the build system copying it over in dist & adding an extra level to the folder hierarchy 🤔

Sorry had a ton to do this week & just finally getting back to this. I'll get some fixes pushed up on monday :)

I think we should just move all tests that use DID stuff from packages/core into packages/ucans (which may be all tests).

ok yup this makes sense to me. I was feeling that same tension. (There's also a pseudo-circular dependency in here since @ucans/default-plugins is a dev dependency of @ucans/core that we could then eliminate).
The downside is the tests are further away from the code they're testing. So before running any tests, you have to build @ucans/core each time.
Although speaking of: I just did a big ol tooling/monorepo revamp on our main repo & learned some nice tsconfig tricks that can close that gap a bit. I'll try and bring them over.

Hm. I'm not sure what you mean by the types getting wonky if you parameterize the whole class 🤔

right so the issue I was running into here was that other files in the package use, for instance, the Store class. We wouldn't be exporting it anywhere until after the plugins are injected. Thinking through it again, we should be able to manage this with interfaces (which is probably a good rule of thumb anyway) 🤔
I'll poke it a bit more & see what I can't come up with

right so the issue I was running into here was that other files in the package use, for instance, the Store class. We wouldn't be exporting it anywhere until after the plugins are injected. Thinking through it again, we should be able to manage this with interfaces (which is probably a good rule of thumb anyway) 🤔

riiiight. Yes. Interfaces is the way to go I think :+1:

alrighty!
moved most of the tests to @ucans/ucans. I left behind the ones that don't rely on plugins: semver & capability.
wrapped Builder & Store in a mk function. Had to do a little bit of typescript massaging to get it to work, detailed that in the above comment.
I briefly tried to do some tsconfig monorepo tricks like config inheritance & a composite config with references to other packages. But the current build setup was preventing me from doing that. Namely, copying the tsconfig into the dist folder which would add an extra director to the file hierarchy & screw up relative paths. I think this should be doable, but we may need to chat through the current build process. & maybe belongs in a future PR?

I think this should be doable, but we may need to chat through the current build process. & maybe belongs in a future PR?

Sounds like a plan. Created an issue for this: #84

It didn't like my alpha version in one of the core package.json.
I also added in some workspace commands for building & testing all three packages from the workspace root.
Seems to all be passing now 👌


✨ New command to purge some tables

This adds a new command called purge that allows purging specific tables of a backup. The command can be useful when sharing a backup with someone else, while wanting to keep some data private.
Example: If I would like to remove all of my notes, tags, and tagMaps I can simply run:
go-jwlm purge backup.jwlibrary purged.jwlibrary --tables=Note,Tag,TagMap

Coverage decreased (-0.1%) to 86.089% when pulling 1de9895274338f02a078377cc7db4c88c1f83ecf on delete_from_db into 8c017b8c414c102da2516178ab1405b065262d68 on master.


Adding DoB to main branch parsers

Adds fields for date_of_birth, dob_year, dob_mont, dob_day.
If a date of birth is provided in the data, it uses this to populate all 4 fields.
If age is provided but not a date of birth, dob_year is calculated based on the admission date, plus dob_month if age is provided in months/days.
Concerns:

This doesn't really fix the initial issue of providing more user-friendly values for children/infants where the age is <1.
Date hierarchies don't work with apply functions, so there are instances where an age/year can't be calculated because the chosen date (either admission date or enrolment date) isn't filled

Uses functions from globaldothealth/adtl#79


Brightness support for 5x5 RGB Matrix Breakout

Hello,
I'm trying to use the Micropython library for the 5x5 RGB Matrix Breakout but I can't seem to find a way to set the brightness for an individual led. I saw in the examples that there's also no mention of this. Am I missing something?
I have found this other library that does have support for setting brightness but I'm having some issues turning on (some of) the leds on a RPi Pico.
Thanks!

The Python library just scales the individual colour channels by the brightness value, so you could do something like this:
brightness = 0.5
matrix.set_pixel(x, y, int(r * brightness), int(g * brightness), int(b * brightness))

To achieve the same result.
Brightness control is indeed currently unsupported in the Pico libraries.

That seems to work, thanks a lot @Gadgetoid !
I could try to make a PR to add brightness control (at least for the 5x5 matrix) but I'm not super sure where or how to start. Do you perhaps have any pointers for me?
Thanks again!

Most of our Python modules are implemented in C++ so it's frustratingly complex. The main thrust of the RGB 5x5 set_pixel is here:
https://github.com/pimoroni/pimoroni-pico/blob/6eb0f90e53e503790150028f2cf181961f589619/libraries/breakout_rgbmatrix5x5/breakout_rgbmatrix5x5.cpp#L27-L36
Which calls:
https://github.com/pimoroni/pimoroni-pico/blob/6eb0f90e53e503790150028f2cf181961f589619/drivers/is31fl3731/is31fl3731.cpp#L83-L85
So ideally here it would multiply the R, G and B channels by a brightness value from 0-255, scale it back to the range 0-255 and then apply gamma using the GAMMA_8BIT table from "common/pimoroni_common.hpp"
That brightness value would then have to be passed through from the Python bindings:
https://github.com/pimoroni/pimoroni-pico/blob/6eb0f90e53e503790150028f2cf181961f589619/micropython/modules/breakout_rgbmatrix5x5/breakout_rgbmatrix5x5.cpp#L49-L85
You can add brightness as an optional keyword arg with something like:
{ MP_QSTR_brightness, MP_ARG_INT, {.u_int = 255I} },

If you fork pimoroni-pico on GitHub, you can push changes up and - with any luck- it'll build you a new MicroPython without you needing to grapple with a local build environment.
Using C bindings for libraries like this was a choice we made to avoid having to write everything twice and it's definitely had its pitfalls. Your time might be better spent trying to get the pure Python library to work 😬

Thanks for the very detailed reply, @Gadgetoid ! I'm not a CPP kinda guy but I'll try!

Your time might be better spent trying to get the pure Python library to work

Trust me I spent far longer on this project than I should have and now that I've found a way to make it work I'm not sure I want to debug that further :p.
Some struggles I had:

Started with TinyGo on an Arduino Nano IoT 33. There was no lib for this 5x5 matrix so I copied the one you guys made into Go (which was a gigantic pain but I got there in the end)
For another device I have, I also couldn't find a library in TinyGo and tried to write my own, failed quite badly.
Changed to an Arduino ESP 32 because of the Micropython support hoping that I would be able to get these Python libraries to work. Failed miserably.
Now finally bought an RPi Nano W, flashed it with the Pimoroni Pico Micropython and finally I have both devices working nicely

Anyway, sorry for the rambling! I'll give the brightness thing a shot somewhere this week. Thanks again!

flashed it with the Pimoroni Pico Micropython and finally I have both devices working nicely

As contrived and troublesome as our libraries-included MicroPython build is, it's nice when it just has all the things good to go. And good to hear it was your solution!
No time like the present to dive into C++ 😆


Abbreviation of 'Science'

According to my current implementation of ISO-4, the correct abbreviation of the journal 'Science' would be 'Sci.', but Science is in fact not abbreviated at all in any publications I know of, so something is missing here?

Actually, seems the rule is to not abbreviate one-word journal titles at all, so adding that as a check would clear this up.


write CSI to csv file

The STA mode ESP32 can collect csi_data, but unfortunatelly cannot write csidata to csv file using cmd "findstr..." or "grep...".Looking forward to your apply!

I need more information. What is there error that you are seeing?
What operating system are you using? Windows uses findstr Linux/MacOS uses grep.
Run the following and report the results. :

idf.py monitor -p /dev/ttyUSB0 <- What is printed by STA mode?
idf.py monitor -p /dev/ttyUSB0 | grep "CSI_DATA" <- What is printed with this?
idf.py monitor -p /dev/ttyUSB0 | findstr "CSI_DATA" <- What is printed with this?

idf.py monitor

idf.py monitor | grep "CSI_DATA"

idf.py monitor | findstr "CSI_DATA"

Thank you, Steven

I am sorry, I have no knowledge about Powershell. From what I can gather, powershell does not use pipes as would be expected in something like bash on macOS or Linux.
I do not have a windows computer to solve this. If you can figure out a solution, please share it here so that others can benefit from your solution. Thank you.
Also, #18 deals with windows compatibility.

Closing issue in favor of keeping #18 open for any issues running on Windows machines


Is there any way to get the spec as json without calling doc/api.json?

Hi,
you can get the json-string from ApiSpec.get(PluginConfigDsl.DEFAULT_SPEC_ID) after its built by its id. By default, i.e. if you are not using multiple specs and the default configuration, the id is always PluginConfigDsl.DEFAULT_SPEC_ID, otherwise the one you assigned.
Another option is the "post-build"-hook in the plugin config that is called with the finished OpenAPI-object.
install(SwaggerUi) {
   postBuild = { openApiSpec ->
      // ...
   }
}


Wrap view store binding to avoid reading stale state

Attempt to fix #1800
The current method creates a binding with a location type of ObservableObjectLocation. By wrapping the binding the location type becomes FunctionalLocation. Bindings that use ObservableObjectLocation seem to have their getter called while evaluating if a views inputs have changed. This doesn't seem to be the case with FunctionalLocation. Which kind of makes sense.
I've confirmed this works for #1783 and #1798 as well as checked animations work as expected.
@stephencelis @mbrandonw @tgrapperon maybe you can stress test this a bit to see if there are any unwanted effects. I imagine there is going to be some sort of optimisation that can be applied to bindings that use ObservableObjectLocation, but don't really think that's relevant here.

"Yo Dawg, I heard you like bindings, so we put bindings into your bindings so you can bind while you bind!"
Great find! I think that most of the problems are caused by the fact that the ObservedObject we were using is not installed on the graph. So while it was fixing some animations issues, it likely wasn't participating as it should to the invalidation process. I would guess that function backed ones are likely eagerly evaluated, whereas ObservedObject based ones are maybe coalesced up to the next render pass. I haven't checked yet if a simple functional binding that would call objectWillChange() in its setter wouldn't have helped with animations (maybe it was like that at some point).
I'll try to give them a spin soon to check if I'm spotting issues.

Nice catch @iampatbrown! I'm curious about the 'transaction($1)' purpose here. Could you explain it, please? Because removing this part still fixes both issues.

Nice catch @iampatbrown! I'm curious about the 'transaction($1)' purpose here. Could you explain it, please? Because removing this part still fixes both issues.

@barabashd .transaction($1) allows any animations or transactions explicitly set on the binding to work correctly. eg:
Toggle("Toggle", isOn: $isOn.animation(.easeIn))

Using withTransaction(transaction) { base.wrappedValue = newValue } as an alternative has slightly different behaviour. It causes explicit animations/transactions to take precedence over other calls to withAnimation/withTransaction.

I think that most of the problems are caused by the fact that the ObservedObject we were using is not installed on the graph. So while it was fixing some animations issues, it likely wasn't participating as it should to the invalidation process.

@tgrapperon It's been a while since stumbling across the ObservedObject approach, can't remember having much success without it though. My guess was that something happened under the hood that tracked state changes and forwarded explicit animations/transactions correctly.

I haven't checked yet if a simple functional binding that would call objectWillChange() in its setter wouldn't have helped with animations (maybe it was like that at some point).

Pretty sure the main thing was forwarding explicit binding animations/transactions. Accessing the current Transaction to merge/override any explicit binding transaction could also work, though it's not really supported.


Support for sharp with bun in replit

I tried using sharp with bun on linux on replit with many of the latest versions, it's the same issue:
109 |   }
110 |   help.push(
111 |     '- Consult the installation documentation:',
112 |     '    See https://sharp.pixelplumbing.com/install'
113 |   );
114 |   throw new Error(help.join('\n'));
              ^
error: Could not load the "sharp" module using the linux-x64 runtime
undefined: libstdc++.so.6: cannot open shared object file: No such file or directory

Repl: https://replit.com/@7heMech/Sharp-falis
The bun version you see in the .replit file is replit's internal versioning which is a static reference to their latest bun version 1.0.23 currently.

You'll need to ask Replit.
My best guess would be that the nix packaging they provide for Bun at https://github.com/replit/nixmodules/blob/main/pkgs/bun/default.nix needs to add support for REPLIT_LD_LIBRARY_PATH like https://github.com/replit/nixmodules/pull/198


Goto not supported.

Steps to reproduce the behaviour
http://deck.net/46faf6e524f08b533ceed1b6aebd6b04
public class App
{
    public static void Main()
    {
        Block:
        var val = 7;
    }
}

Hi @michaelcheers,
Thanks for the suggestion. It can be useful in some scenarios.

Am I misunderstanding this issue in some way?
Given the C# in the links I expect this error.
/// <summary>
/// Creates a simple class
/// </summary>
public class App
{
    // Use the [Ready] attribute to automaically
    // run this method when the page is ready.
    [Ready]
    public static void Main()
    {
        Block:
        val = 7;
    }
}

C# Compilation Failed

Microsoft (R) Visual C# Compiler version 4.0.30319.34209

(12,9): error CS0103: The name 'val' does not exist in the current context

@Phrohdoh, thanks, I've changed it.

@vladsch, good news.


lxd/config: Allow modifying cluster.https_address

This allows modifying cluster.https_address when LXD isn't clustered yet.
Closes #6167
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

Testsuite passed


feat: add num iteration and start iteration to lightgbm model

add num iteration and start iteration to lightgbm model
set num iteration to be best iteration when early stopping criteria reached
resolves issues (https://github.com/Azure/mmlspark/issues/1006, https://github.com/Azure/mmlspark/issues/961)

/azp run

/azp run

/azp run


DO NOT MERGE: Showing problem when using API23/Marshmallow

I know this library is being deprecated for cwac-cam2, but I thought I would post this PR here for comment.
When consuming cwac-camera from an app with compileSdkVersion 23 the app runs fine on pre-API23 devices, but when run on an API23 device the following crash occurs when instantiating the subclass of CameraFragment:
10-13 11:09:03.801 13905-13905/com.commonsware.cwac.camera.demo I/System.out: waiting for debugger to settle...
10-13 11:09:04.002 13905-13905/com.commonsware.cwac.camera.demo I/System.out: debugger has settled (1367)
10-13 11:09:04.010 13905-13905/com.commonsware.cwac.camera.demo W/System: ClassLoader referenced unknown path: /data/app/com.commonsware.cwac.camera.demo-2/lib/arm64
10-13 11:09:04.022 13905-13905/com.commonsware.cwac.camera.demo I/art: Rejecting re-init on previously-failed class java.lang.Class<com.commonsware.cwac.camera.demo.DemoCameraFragment>
10-13 11:09:04.022 13905-13905/com.commonsware.cwac.camera.demo I/art: Rejecting re-init on previously-failed class java.lang.Class<com.commonsware.cwac.camera.demo.DemoCameraFragment>
10-13 11:09:04.022 13905-13905/com.commonsware.cwac.camera.demo I/art: Rejecting re-init on previously-failed class java.lang.Class<com.commonsware.cwac.camera.demo.DemoCameraFragment>
10-13 11:09:04.024 13905-13905/com.commonsware.cwac.camera.demo I/art: Rejecting re-init on previously-failed class java.lang.Class<com.commonsware.cwac.camera.demo.DemoCameraFragment>
10-13 11:09:04.058 13905-13905/com.commonsware.cwac.camera.demo I/art: Rejecting re-init on previously-failed class java.lang.Class<com.commonsware.cwac.camera.demo.DemoCameraFragment>
10-13 11:09:04.059 13905-13905/com.commonsware.cwac.camera.demo D/AndroidRuntime: Shutting down VM
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime: FATAL EXCEPTION: main
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime: Process: com.commonsware.cwac.camera.demo, PID: 13905
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime: java.lang.LinkageError: com.commonsware.cwac.camera.demo.DemoCameraFragment
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at com.commonsware.cwac.camera.demo.MainActivity.onCreate(MainActivity.java:36)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at android.app.Activity.performCreate(Activity.java:6237)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1107)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2369)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2476)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at android.app.ActivityThread.-wrap11(ActivityThread.java)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1344)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at android.os.Handler.dispatchMessage(Handler.java:102)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at android.os.Looper.loop(Looper.java:148)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at android.app.ActivityThread.main(ActivityThread.java:5417)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at java.lang.reflect.Method.invoke(Native Method)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
10-13 11:09:04.060 13905-13905/com.commonsware.cwac.camera.demo E/AndroidRuntime:     at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)
Any idea WTH is going on here?
Thanks!

Found the problem in a Fork:
https://github.com/tictail/tictail-cwac-camera/commit/f3193320436f570e2d46d05a635c8668c4058b38
On behalf of tictail, I will clean up this PR and make it ready for merge.

Please do not submit a revised pull request. Yes, the problem appears to be in getHost(). However, this is a breaking change to the public API, and as such I need to handle this carefully.

@commonsguy ready for review/merge?

@commonsguy just caught your comment. I will remove the PR.

@paulpv This is addressed in v0.7.0

The issue is closed, but would like to get a bit more clarification.

If using compileSdkVersion < 23, is that all fine?
Also why changing a few function call would solve it (I just took a quick look at the changed files in the PR)?

@ggua5470 Please use Stack Overflow for questions, as is mentioned in the project documentation.

compileSdkVersion of 19 seems to be OK with v0.6.12, insofar as there is no immediate crash in the demo apps, but if something in the framework uses getHost(), you will likely have problems.

getHost() was added to Fragment in API Level 23. I needed to rename my getHost() method to avoid the name and usage collision. I elected to rename setHost() to match. See this blog post for more.


upgrade elm version to 0.19

support for elm 0.19 is added
example with play framework now can be run on java9+

closing this due to dirty commits. will create pull request from squash commit


builtin: pwd

Should support a variable PWD and a builtin for getting the present working directory

created the pwd builtin and the test for it


Make check-names.sh accept any grep

check-names.sh works fine with GNU and with modern FreeBSD grep
so remove the check for GNU grep.
Signed-off-by: Dave Rodgman<EMAIL_ADDRESS>Backport of #4308

The backports are trivial so can be merged with a single approval, but I'd prefer a third pair of eyes on the change, so I'm keeping "needs: reviewer" on the main PR.


Sort elements inside pairs

Sorting { a, c, b, d } with :Sort if selected inside the block will be { a, b, c, d }
but if the whole block is selected itll be b, c, d }, { a

@olivertzeng This is the intended behavior. When you select the whole block it splits the input with ,, which would give you the following blocks:

{ a
b
c
d }

Sorting these blocks alphabetically will give you b, c, d }, { a.

@olivertzeng This is the intended behavior. When you select the whole block it splits the input with ,, which would give you the following blocks:

{ a

b

c

d }

Sorting these blocks alphabetically will give you b, c, d }, { a.

Yeah I was thinking that it would automatically exclude that but I should've just only select the elements from the start
Thanks though
Btw issues like this that is not intended to be fixed should be closed as "not planned won't fix" instead of completed


[discussion] Should be push private model neffs to the trainium cache

When you try to push NEFFs from compiled model with a private origin model it will fail since we have a safeguard in place. This might be suboptimal, e.g. we cannot cache llama.
cc @michaelbenayoun

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!


Adds 'list' subcommand family

These changes add a list subcommand family with two subcommands:

backplane-tools list available returns a list of tools which are available to install with the current backplane-tools binary
backplane-tools list installed returns the list of tools which have already been installed to the local filesystem

/unhold


refactor(cli): better gitignore management to make it easier when running the CLI on upstream backstage environments.

When running the export-dynamic-plugin CLI command on any backstage environment (like upstream backstage), for example using the npx approach, the 2 following behavior are painful:

with the --dynamic-plugins-root option, the specified dynamic plugin root folder would be created, possibly at the root of your backstage environment, but no gitignore is added inside, so that the files of the dynamic plugins generated inside this folder will be considered tracked by source control, which is obviously wrong.
when exporting a backend plugin, by default the package.json and yarn.lock are excluded in the gitignore file created in the generated dist-dynamic folder. These 2 files end up being considered added to to source control. Once again, running the dynamic plugin export on any backstage environment should not add files to the source control of this backstage environment by default. Having those 2 files under source control is a meaningful option in some case, but it should be an opt-in, and not by default.

This PR fixes those 2 behaviors by:

always ignoring the whole content of the created dynamic-plugins-root folder,
introducing a new --track-dynamic-manifest-and-lock-file option (false by default), to exclude those 2 files from the dist-dynamic/gitignore file.

So running export-dynamic-plugin --dev --dynamic-plugins-root <root of my backstage> doesn't end up proposing new files to be added to source control anymore.

/approve


Ternary operator not filtering out NilClass.

Input
    sig { returns(String) }
    def embargo_date
      work.embargo_date ? work.embargo_date.to_formatted_s(:long) : 'Immediately'
    end

    sig { returns(Work) }
    attr_reader :work

Observed output

srb tc
app/components/works/detail_component.rb:42: Method to_formatted_s does not exist on NilClass component of T.nilable(Date) https://srb.help/7003
42 |      work.embargo_date ? work.embargo_date.to_formatted_s(:long) : 'Immediately'
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Autocorrect: Use -a to autocorrect
app/components/works/detail_component.rb:42: Replace with T.must(work.embargo_date)
42 |      work.embargo_date ? work.embargo_date.to_formatted_s(:long) : 'Immediately'
^^^^^^^^^^^^^^^^^
Errors: 1

Expected behavior
The ternary should ensure that we're not looking at the NilClass.
Sorbet typechecker 0.5.5994 git 83fcb0939893f7a328dc75962a23479eca4e9422 debug_symbols=true clean=1

Changing the code to
    def embargo_date
      if work.embargo_date
        work.embargo_date.to_formatted_s(:long)
      else
        'Immediately'
      end
    end

does not change the result:
$ be srb
app/components/works/detail_component.rb:43: Method to_formatted_s does not exist on NilClass component of T.nilable(Date) https://srb.help/7003
    43 |        work.embargo_date.to_formatted_s(:long)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  Autocorrect: Use `-a` to autocorrect
    app/components/works/detail_component.rb:43: Replace with T.must(work.embargo_date)
    43 |        work.embargo_date.to_formatted_s(:long)
                ^^^^^^^^^^^^^^^^^
Errors: 1

See:
https://sorbet.org/docs/faq#why-does-sorbet-think-this-is-nil-i-just-checked-that-its-not
https://sorbet.org/docs/flow-sensitive#limitations-of-flow-sensitivity
Rewriting your example to:
def embargo_date
  date = work.embargo_date
  if date
    date.to_formatted_s(:long)
  else
    'Immediately'
  end
end

should work properly.

@froydnj Thank you for taking the time to explain it to me.  It is unfortunate that sorbet can't handle this.


Feedback based on DWX

Thanks for merging
I just tested this branch on the other laptop and it somehow didn't work there (black screen when I want to start the extension). I tried to reset the source code there and still got the same error, so it might not even be related to these changes.
But just to be sure, can you try to pull the changes and see if the custom mission starts for you?

Will do in a couple hours. On a train at the moment
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Marius Obert @.>
Sent: Thursday, July 28, 2022 1:18:01 PM
To: nokenwa/twilioQuestBoothExecution @.>
Cc: Nathaniel Okenwa @.>; State change @.>
Subject: Re: [nokenwa/twilioQuestBoothExecution] Feedback based on DWX (PR #1)
Thanks for merging
I just tested this branch on the other laptop and it somehow didn't work there (black screen when I want to start the extension). I tried to reset the source code there and still got the same error, so it might not even be related to these changes.
But just to be sure, can you try to pull the changes and see if the custom mission starts for you?
—
Reply to this email directly, view it on GitHubhttps://github.com/nokenwa/twilioQuestBoothExecution/pull/1#issuecomment-1198064977, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFQC2RK6PJG6BKVYW35QZKTVWJ27TANCNFSM544ZWRSA.
You are receiving this because you modified the open/close state.Message ID: @.***>

Hmm, Worked on my machine.

Any logs or anything I can take a look at?


improve loading inference api to reduce cognitive complexity

The revised extract_scores does not seem to be more readable for human. And as sonarlint cognitive complexity is not our top priority here, we close this PR.


Better Looking Dark mode

This looks more like Windows 11

This looks more like Windows 98

I think you get the idea

@Czekoloczek Well, feel free to give it a crack. You can define another style in this function:
https://github.com/Windows200000/TwitchDropsMiner-updated/blob/58b903516d31ed9e15c66baa316b7f7872fd94af/gui.py#L2195-L2198
If you find something, feel free to just add another elif and I will change the checkbox in settings to be a drop-down.
Keep in mind, that not all Tkinter themes are cross-platform. This list has a quick overview for that.
The default theme Tkinter uses on Windows, the one in your first picture, is called "vista". This theme is Windows-exclusive, as it isn't entirely defined in Tkinter, but rather uses some universal elements built into windows, such as the buttons, which as a consequence, cannot be recolored.


Remove asserts

Discussed here, https://github.com/annot8/annot8-common/commit/0f62a5295530a16af90f089589a27f656120cb8d.
Any uses of assert should be replaced with proper checks and throwing a catchable exception.

I've removed the assert statements, but haven't replaced them with Exceptions for now as if we choose to do this we ought to do it consistently across all constructors and I wasn't sure we wanted to do that?


feat: option to disable schema key conversion

Current dependencies on/for this PR:

main

PR #29   👈

This comment was auto-generated by Graphite.


v1.5 doesn't show time for step in report

any way to show the time for step in report with this version?
it only show the time stamp at which the step started, in the previous version it also displayed the time per step (and it's sb steps)
Regards,
Tsachi

@baev sorry that I write here, but i didnt see changes in report. in page code i see divs with this commit.
google chrome latest version/firefox latest version
I can create a new report if u wish

At our demo report, it works. Try to put cursor over the time and you will see duration

@just-boris yea, sry im expected this tooltip on my test step text. its ok now.


Create PostgreSQL performance test comparison

Is your feature request related to a problem? Please describe.
We don't have any data on performance of Rook Operator and any comparison data.
Describe the solution you'd like
We want to have comparision of performance between Rook Operator storage with normal storage with using
for comparison purpose PostgreSQL backend.
Test plan can be similar to the one below:

Have epiphany cluster with PostgreSQL database created in standard epiphany way
Create performance test running in Kubernetes pod using PostgreSQL in current setup (pgbench can be used)
Deploy Rook Operator and create Ceph cluster
Create PostgreSQL database running in Kubernetes pod using Ceph PVC (use some of those https://operatorhub.io/?keyword=postgresql)
Run performance test using Kubernetes PostgreSQL instance
Compare results

Describe alternatives you've considered
No alternatives.
Additional context
No additional context.

Test results look like the following:
WRITE NO PVC CURRENT SETUP:
transaction type: <builtin: simple update> scaling factor: 1 query mode: simple number of clients: 100 number of threads: 1 duration: 300 s number of transactions actually processed: 1198256 latency average = 25.052 ms tps = 3991.660980 (including connections establishing) tps = 3991.698811 (excluding connections establishing)
WRITE  PVC WITH CEPH (block storage):
transaction type: <builtin: simple update> scaling factor: 1 query mode: simple number of clients: 90 number of threads: 1 duration: 300 s number of transactions actually processed: 915886 latency average = 29.486 ms tps = 3052.266371 (including connections establishing) tps = 3052.285772 (excluding connections establishing)
WRITE  PVC WITH CEPH (block storage) INSTALLED WITH CRUNCHY PGO:
transaction type: <builtin: simple update> scaling factor: 1 query mode: simple number of clients: 90 number of threads: 1 duration: 300 s number of transactions actually processed: 741266 latency average = 36.440 ms tps = 2469.828167 (including connections establishing) tps = 2469.852819 (excluding connections establishing)
Test results look like the following:
READ NO PVC CURRENT SETUP:
transaction type: <builtin: select only> scaling factor: 1 query mode: simple number of clients: 100 number of threads: 1 duration: 300 s number of transactions actually processed: 5159306 latency average = 5.818 ms tps = 17188.423726 (including connections establishing) tps = 17188.528112 (excluding connections establishing)
READ  PVC WITH CEPH (block storage):
transaction type: <builtin: select only> scaling factor: 1 query mode: simple number of clients: 90 number of threads: 1 duration: 300 s number of transactions actually processed: 4749131 latency average = 5.686 ms tps = 15827.274870 (including connections establishing) tps = 15827.371305 (excluding connections establishing)
READ  PVC WITH CEPH (block storage) INSTALLED WITH CRUNCHY PGO:
transaction type: <builtin: select only> scaling factor: 1 query mode: simple number of clients: 90 number of threads: 1 duration: 300 s number of transactions actually processed: 3993386 latency average = 6.761 ms tps = 13311.008753 (including connections establishing) tps = 13311.180973 (excluding connections establishing)

@ar3ndt  can you create table with comparison "Rook (Ceph) | VM"?

write tps
read tps
replicas
vm/k8s

NO PVC
4200
18402
2
vm

NO PVC
3991
17188
1
k8s

PVC WITH CEPH
3052
15827
1
k8s

PVC WITH CEPH & PGO
2469
13311
1
k8s

PVC WITH CEPH & PGO
2382
12720
3
k8s

LGTM. Results in table are clear. Moving to DoD Check.

Notes for the future:
With pgbench you may test at the 3 levels of performance related to disk: in buffer, mostly in cache, and all on disk. You manipulate this by changing the scale factor, see Memory_vs._Disk_Performance.
While running read-only test cases (with the -S option), it's good practice to measure the database cache hit ratio (query from pg_stat_database). At table level we can use: EXPLAIN (ANALYZE, BUFFERS) SELECT * FROM <table_name>;
It seems PostgreSQL do not have any predefined functionality to clear the cache from the memory. To clear the database level cache, we need to shut down the whole instance then clear the operating system cache.
Steps:

Shut down PostgreSQL cluster
Clear OS cache as follows:

sync
echo 3 >/proc/sys/vm/drop_caches

Start PostgreSQL instance

It is also possible to collect timing statistics for database I/O activity, see track_io_timing.


Introduce a "Reply to" email for host accounts

When I am approving or rejecting a collective to join my fiscal host, I would like to get an email in Freshdesk from the message which I send using the platform. I believe this enables that functionality.
I am unsure about the replyTo being the no-reply email address, and whether or not HostCollectiveID is the right email. Ideally, this would be my email, as the host admin making the call. Is that possible?

Actually, unsure if this goes to Freshdesk at all - this might just go to the email of the Collective Host Admin?

Note: Ben is actively working on a better version of this.

Thank you, @Betree!
I really appreciate this merge.

@RichardLitt I thought I merged & deployed already, sorry for the delay :grimacing:

No worries. :)
Is there a way for me to test this? I'm not seeing emails in Freshdesk having used the forms.

@RichardLitt The form itself is not CCed to the email address in order to prevent flooding support. But the replies will come to the emails we've set (since they're used as the Reply to in the emails). They should include the original reply since -by default- replying includes a copy or it. We can iterate if we're not satisfied with the results.


Large python libraries slow down language server dependent features.

Issue Type: Bug
I'm using bitcoinlib and when working with bitcoinlib.keys everything depending on a language server slows down in my editor. Things that normally happen pretty instantly like syntax highlighting, jumping to a definition, autocomplete, etc. take seconds. The editor itself remains responsive. I can still type, open / close files, change tabs, etc.
To reproduce, install bitcoinlib and navigate to this file:
https://github.com/tylerchambers/flask-btc/blob/main/src/flask_btc/wallet.py
In the Wallet class try doing anything with self.HDKey.
e.g: declaring a variable and assigning to subkey.address(). Takes forever.
Things I've tried to fix it: fresh VSCode and fresh OS Install.
Any help appreciated!
EIDT: I ran the extension profiler, and reproduced the issue. It does look like the MS python extension is waiting on a reply from the language server. I'm no longer sure this is the appropriate place to file the bug, any pointers appreciated.
VS Code version: Code 1.60.1 (83bd43bc519d15e50c4272c6cf5c1479df196a4d, 2021-09-10T17:06:57.931Z)
OS version: Linux x64 5.11.0-34-generic
Restricted Mode: No

System Info

Item
Value

CPUs
AMD Ryzen 5 2600 Six-Core Processor (12 x 1550)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 1

Memory (System)
31.29GB (23.60GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

Extensions (11)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.1.25

gitlens
eam
11.6.0

prettier-vscode
esb
9.0.0

go
gol
0.28.0

vscode-docker
ms-
1.17.0

python
ms-
2021.9.1246542782

vscode-pylance
ms-
2021.9.2

jupyter
ms-
2021.8.2041215044

jupyter-keymap
ms-
1.0.0

cpptools
ms-
1.6.0

vscodeintellicode
Vis
1.2.14

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
pythonvsuse255:30340121
vscod805:30301674
pythonvspyt200:30340761
vscextlangct:30333562
binariesv615:30325510
vsccppwtct:30364498
pythonvssor306:30344512
bridge0708:30335490
pygetstartedt2:30360495
bridge0723:30353136
pythonrunftest32cf:30365367
pythonf5test824cf:30361778
javagetstartedt:30364666
pythonvspyt187cf:30365362
pydsgst2:30361792
vssid140cf:30363604
vssur157:30367808

Please try filing the issue https://github.com/microsoft/vscode-python first, thanks for filing the issue.


Relax dependencies, fix gemspec outdated error

This gemspec was forcing down my redis version and I need to stay up to date.
Followed the hiredis and sidekiq versioning recommendations.

LGTM
Thanks


What is a kinase?

This PR adds more details to the human-kinases notebook. It relies on human-supervised curation of the existing records using automated methods validated by extensive literature references.
More sources

[ ] DGIdb (api)

Possible explanation for discrepancies between UniProt queries and other kinase-specific (published) sources:

First, molecular function annotations do not necessarily correlate with family annotations.
Some atypical kinases fall under lipid kinases (PIK and friends)
We can use Function>Caution messages to guide some of the more dubious entries
"Keyword" and "Enzyme class" views in the side menu are useful!
Useful queries:

# Our initial query (wrong): 637 results
https://www.uniprot.org/uniprot/?query=keyword:%22Kinase%20[KW-0418]%22&fil=organism%3A%22Homo+sapiens+%28Human%29+%5B9606%5D%22+AND+reviewed%3Ayes

#Helge's query: 493 (too conservative)
family:"protein kinase superfamily" AND reviewed:yes AND organism:"Homo sapiens (Human) [9606]"

# Too restrictive (only ser/trh and tyr kinases): 473
(keyword:"Tyrosine-protein kinase [KW-0829]" OR keyword:"Serine/threonine-protein kinase [KW-0723]") AND reviewed:yes AND organism:"Homo sapiens (Human) [9606]"

# PIK and friends: 12 extra kinases
https://www.uniprot.org/uniprot/?query=ec:2.7.1.-%20family:%22pi3%20pi4-kinase%20family%22&fil=organism%3A%22Homo+sapiens+%28Human%29+%5B9606%5D%22+AND+reviewed%3Ayes

# How to query for caution messages
https://www.uniprot.org/uniprot/?query=annotation%3A%28type%3Acaution+kinase%29+reviewed%3Ayes+organism%3A%22Homo+sapiens+%28Human%29+%5B9606%5D%22+NOT+name%3Akinase&sort=score

Uniprot also lists all entries by family here: https://www.uniprot.org/docs/similar.txt

@AndreaVolkamer: Last commit fixes the issue we had with UniProt queries being too broad. Turns out we were querying for all kinases, not just protein kinases. @hlgvth correctly pointed out that nuance, so now the number went down from 700ish to 493, which is what we can observe in other datasets.
That said, some datasets add extra kinases to their lists. KLIFS for example adds phosphatidylinositol kinases, saccharide kinases, and other proteins that bind ATP (a total of 38 more).

Cleaned activities for kinases in ChEMBL 27 (preview before new data release).
activities-chembl27_2.7z.zip

Data for the new ChEMBL 28. activities-chembl28_2.zip
The dataset grew from 218101 / 174649 to 237830 / 187387 (raw / curated), so yay 10K extra data points!

Thanks @schallerdavid! Merging now and cutting a new release.


Publicity for XXX-ext-html-webpack-plugins

Replacement for PR 511
Updated README

Thanks @numical 👍


No Try now Button for quiz app on the main html file

The try now button for How old is you? quiz is not visible.
I'm a GSSOC Contributor, please assign this issue to me.

✅ I agree to follow this project's Code of Conduct.
✅ I checked the current issues for duplicate problems.
✅ I have gone through the project README.

I want to work on this under GSSOC2022. Here is my discord profile DHANANJOY DEY | Contributor#7903

I would like to work on this issue. Please assign it to me @ShamithaReddy


Add new paper & authors in Readme and Docs

New author email added in #181
New paper should be added once published.


Parallel playbook execution

Proposal: Enhance Ansible to allow playbooks to be run in parallel.
Author: Jon Hawkesworth @jhawkesworth
Date: 2016/10/07

Status: New
Proposal type: Enhancement to core (or possibly new CLI tool)
Targeted Release: To be decided
Associated PR: None
Estimated time to implement: weeks

Motivation
Describe the reasons for this proposal.
During complex deployments, in order to keep the period of change down to the smallest possible time period, it would be very helpful if Ansible would allow (included) playbooks to be run in parallel.
Problems
What problems exist that this proposal will solve?

Currently I am using Jenkins to wrap up playbooks so that I can simultaneously update middleware and computer components at the same time as updating web services and front end webservers.  Wrapping each playbook in a jenkins job is a tedious manual process which I would like to replace with a top-level playbook.

Solution proposal

One way would be to enhance playbook includes so that a list of included playbooks causes a fork and runs all the playbooks in the list simultaneously.

So may top level playbook could look like this
    - include: find_release_versions.yml
    - include: synchronize_components_to_datacenter.yml
    - include: pre-deployment-checks.yml
    - include: start-deployment-notifications.yml
    - include:
         - migrate-database.yml
         - update-compute-nodes.yml
         - update-dequeuers.yml
         - update-web-services.yml
         - update-file-transfer.yml
         - update-frontend-webservers.yml
    - include: end-deployment-notifications.yml
    - include: generate-change-report.yml

So everything from migrate-database.yml to update-frontend-webservers.yml would be executed simultaneously and end-deployment-notifications would only start once all the listed items in the previous include had completed.
Rather than overloading 'include' in this way, another keyword might be preferable.  Ideas I have had so far for alternative keyword are 'parallel', 'together', 'fork', or 'join'.
Another implementation might be a separate CLI tool, distinct from ansible-playbook which ran playbooks listed in its own file.  In keeping with the sports name I came up with the name 'league' for the command and 'league-table' as a name for the file format.  On balance I like this idea less as it seems like much more work.  However, you might want to add the ability to specify forks or other playbook command line options (--check perhaps) for each of the parallel playbooks.
Keeping the change within ansible-playbook would make integration with tower and other tools easier.
Here's what a playbook might look like with parallel and some options passed to individual playbooks.
    - include: find_release_versions.yml
    - include: synchronize_components_to_datacenter.yml
    - include: pre-deployment-checks.yml
    - include: start-deployment-notifications.yml
    - parallel:
         - { playbook: 'migrate-database.yml', forks: 1, check: True}
         - { playbook: 'migrate-database.yml', forks: 1 }
         - update-compute-nodes.yml
         - update-dequeuers.yml
         - update-web-services.yml
         - update-file-transfer.yml
         - { playbook: 'update-frontend-webservers.yml', forks: 20 }
    - include: end-deployment-notifications.yml
    - include: generate-change-report.yml

Dependencies
I can't think of any dependencies that aren't already in use in ansible-playbook
Testing
If passing playbook options is included in this functionality, there would be significant testing.
Possibly useful to enable testing, although this seems to be handled in Shipable at the time of writing.
Documentation
I can imagine that this feature could possibly cause ansible controllers to run out of resources as it might cause many forks to be generated, so it would need documenting to advise users not to go crazy with it.

This would be very useful to us.  One use case, we get to a point in our playbook when we need to spin up 4 different aws instances, configure them, and create ami's from the result.  Doing that in Parallel would save a lot of time.

This would save a lot of time by rolling out changes in common roles across our hundreds of servers :).

For the parallel invocation case
- parallel:
         - { playbook: 'migrate-database.yml', forks: 1, check: True}
         - { playbook: 'migrate-database.yml', forks: 1 }
         - update-compute-nodes.yml
         - update-dequeuers.yml
         - update-web-services.yml
         - update-file-transfer.yml
         - { playbook: 'update-frontend-webservers.yml', forks: 20 }

What is the general expectation for how the parallel playbooks interact with each other?

if update-compute-nodes.yml adds hosts to the inventory, how would the other playbooks react? Do they see it in inventory immediately? After some stage? never?

In this example, is each playbook targeting a different set of hosts?

if they target the same hosts, is there any ordering implied?
If a tasks ends up conflicting with another (say, one has 'file: name=/somefile, state=absent' and another has 'file: name=/somefile, state=present') how would/should/could that be resolved?

would handlers be expected to work across playbooks?

Granted, those scenarios could be consider as errors on the playbook authors part, so may just be 'dont do that then'.

Funny enough, I show how several ways to do this, without altering ansible http://www.slideshare.net/bcoca/more-tips-n-tricks (starts at slide 20)

@alikins Thanks, all good points.
My expectation would be that inventory would be determined as each playbook begins execution.  So in the example above you would want to have an
- include: update-inventory.yml
before the
parallel:
update-compute-nodes.yml would probably be too late.
Yes, my assumption is that each of the playbooks listed in the parallel: list would be targeting different hosts (this is based on my actual use case).  Targeting the same hosts from different playbooks sounds like a 'don't do that unless it really makes sense'.
I would not want any ordering implied - so a yaml list is probably a bit of a confusing way of notating it.
I would want handlers to have a scope that only exists within each invoked playbook (which I suspect is not the case at the moment) so that could be a deal breaker.  I haven't given this any significant thought as I don't use handlers in any of my roles, so I would welcome comment from anyone who uses handlers and playbook includes.
Another assumption in the above is all playbooks are targeting the same inventory which seems potentially inflexible (especially if inventory and handlers were scoped to the playbook level).
@bcoca - Thanks for the slides (wish I could have been there).  I will experiment and see if your 'trying really hard to avoid shell scripts v3' method meets my needs.

So to extend my view on this, there are plenty of external systems that allow you to do this w/o building it into ansible (my slides above show some easy/crude examples).
To add this feature we cannot do this with the current main object (play) it would have to be a 'super playbook' that both contains the inventory and list of playbooks and will require all new DSL to describe this object and deal with it having playbooks (or plays?), then there is the handling of output and error conditions ... and that is just from a first thought, i'm sure plenty of other issues will arrise.
To add this supra layer and complexity to Ansible itself does not seem warranted when there are a plethora of execution systems that can invoke Ansible in a parallel fashion, starting by Tower (shameless plug for the thing that pays my salary) but not limited to it as xargs, parallels, condor, jenkins, cron, at, celery, hadoop ... anything that can execute a command line program in parallel with different options exist already.
I don't see this as something Ansible should tackle and way outside it's purview and adds extra layers of complexity that are really not needed.

@bcoca Having experimented today with the examples from the slide above and I came across two immediate issues:

Because of that use, ad-hoc groups cannot be used.
Because of the invoking of the playbooks you introduce new requirement for paths and (heaven forbid) things like sudo prompts.

I also found that it's also tricky to deal with feedback, for example: I will still require output from the scripts to be captured, so I had to opt back to a poll time, breaking the parallelism.
Those two use-cases above were preventing me to allow for proper parallelism with complicated tasks such as coordinated cluster reboots within a small maintenance window. The solution currently is to simply schedule multiple task in an external task scheduler. But this proposal would perhaps bring a more native solution to the table.

@hyperized my slides are a hint, not a solution, specially the ones that use ansible to shell out to run ansible, that is a 'negative example'.
As mentioned above, there are plenty of other external schedulers that already can do this much better and I really see no need of a native solution.

I still think a native-to-ansible way of running playbooks in parallel which is declarative would enhance ansible's powers as a tool for orchestrating change.  When I say orchestrating, I'm talking in the 'conducting the orchestra' sense.  Like all good orchestral conductors, I think ansible should be able to get the members of the orchestra to play different stuff at the same time. :-)
However, I'm going to close this proposal because:
The time to do it is not now.
I have no time to create something.

This would be an really useful feature for us
in our case we have a playbook include many other playbooks run on different host group, parallel execution would make this a lot more efficient

I wound up using a bash script that calls the playbooks using gnu parallels.  This works ok, parallels doesn't return any output until the playbook has run through or failed, so we tend to wind up tailing the ansible log file or syslogs

Thinking about this I would tend to do include: using async.
The problem obviously is that include doesn't really return status information of the processed 'tasks'. So maybe that's what is missing currently. Make include/import report back failed/changed and allow async.
Maybe this is already possible, I haven't tested this, but IMO that would be the most in line with how Ansible currently works, and would offer a lot of flexibility.