python37Packages.msldap: 0.3.24 -> 0.3.25

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.msldap is: "Python LDAP library for auditing MS AD"
meta.homepage for python37Packages.msldap is: "https://github.com/skelsec/msldap"
meta.changelog for python37Packages.msldap is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.3.25 with grep in /nix/store/s8hs6k07xjnhzc10h26yv2xn52d1q88y-python3.7-msldap-0.3.25

directory tree listing: https://gist.github.com/4ea03299dbd9107b34740a8a1e8b9df7

du listing: https://gist.github.com/da80a6b1b1e6c7ecf8b76ce602104015

Rebuild report (if merged into master) (click to expand)

36 total rebuild path(s)

9 package rebuild(s)

9 x86_64-linux rebuild(s)
9 i686-linux rebuild(s)
9 x86_64-darwin rebuild(s)
9 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python37Packages.lsassy
python37Packages.msldap
python37Packages.pypykatz
python38Packages.lsassy
python38Packages.msldap
python38Packages.pypykatz
python39Packages.lsassy
python39Packages.msldap
python39Packages.pypykatz

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s8hs6k07xjnhzc10h26yv2xn52d1q88y-python3.7-msldap-0.3.25 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.msldap https://github.com/r-ryantm/nixpkgs/archive/1571c8c983fcebcfa35f286426de7f194020fed4.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s8hs6k07xjnhzc10h26yv2xn52d1q88y-python3.7-msldap-0.3.25
ls -la /nix/store/s8hs6k07xjnhzc10h26yv2xn52d1q88y-python3.7-msldap-0.3.25/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

9 packages built:

python37Packages.lsassy
python37Packages.msldap
python37Packages.pypykatz
python38Packages.lsassy
python38Packages.msldap
python38Packages.pypykatz
python39Packages.lsassy
python39Packages.msldap
python39Packages.pypykatz

Maintainer pings
cc @fabaff for testing.


Rust lifetimes break highlighting in list view

Rust uses a single  ' to signify lifetime names and list view highlights everything between two lifetimes as a string (the block of yellow text in the screenshot from vim.lsp.buf.document_symbol()).

Should been fixed in latest version of guihua.lua. Please update your plugin

Looks good now; thank you!


Select data from .xlsx

Hi, sorry it's not an issue more a question, do you know how to select data based on some criteria, my question: https://stackoverflow.com/questions/60814823/select-data-from-excel-using-powershell/60817252#60817252, thanks in advance

Import-excel path | select Company , country | ? {$_.country -eq 'x'}
On Mon, Mar 23, 2020, 3:48 PM Miko998<EMAIL_ADDRESS>wrote:

Hi, sorry it's not an issue more a question, do you know how to select
data based on some criteria, my question:
https://stackoverflow.com/questions/60814823/select-data-from-excel-using-powershell/60817252#60817252,
thanks in advance
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/issues/808, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKW2GYP3VS2NHU3OHLGVWG3RI64I5ANCNFSM4LSECH3Q
.

Using this module
Import-Excel 'sample.xlsx' -worksheet sheet1 | where {$_.country -eq 'x'}

While I played with this question for a minute. I realized
Get-process | ? {$_.HasExited -eq 'False'}
Doesn't work since it's a bool property.
However it doesn't work with data imported from excel as well. Shouldn't be
the cell value true or false considered a string??
Sorry if its being novice.
On Mon, Mar 23, 2020, 3:53 PM Doug Finke<EMAIL_ADDRESS>wrote:

Using this module
Import-Excel 'sample.xlsx' -worksheet sheet1 | where {$_.country -eq 'x'}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/issues/808#issuecomment-602821544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKW2GYLETX67JYHQVFAISALRI644JANCNFSM4LSECH3Q
.

Not sure. This $false and 'false' work. PS is coercing the data.
$xlfile = "$env:TEMP\test.xlsx"
rm $xlfile -ErrorAction SilentlyContinue

$data = ConvertFrom-Csv @"
Region,Item,TotalSold,HasData
South,screws,43,true
North,apple,44,false
North,lemon,55,false
South,banana,2,false
North,screwdriver,86,true
East,saw,13,true
South,nail,46,true
West,peach,43,false
North,apple,4,false
North,avocado,93,false
"@

Export-Excel -InputObject $data -Path $xlfile

import-excel $xlfile | ? {$_.hasdata -eq $false}

Ok . Coercion.. I will study that:). Ty for your response.
On Mon, Mar 23, 2020, 4:12 PM Doug Finke<EMAIL_ADDRESS>wrote:

Not sure. This $false and 'false' work. PS is coercing the data.
$xlfile = "$env:TEMP\test.xlsx"
rm $xlfile -ErrorAction SilentlyContinue
$data = ConvertFrom-Csv @"Region,Item,TotalSold,HasDataSouth,screws,43,trueNorth,apple,44,falseNorth,lemon,55,falseSouth,banana,2,falseNorth,screwdriver,86,trueEast,saw,13,trueSouth,nail,46,trueWest,peach,43,falseNorth,apple,4,falseNorth,avocado,93,false"@
Export-Excel -InputObject $data -Path $xlfile
import-excel $xlfile | ? {$_.hasdata -eq $false}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/issues/808#issuecomment-602830139,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKW2GYKK73GK7CNCCVMD7Z3RI67CXANCNFSM4LSECH3Q
.

Import-excel path | select Company , country | ? {$_.country -eq 'x'}
…
On Mon, Mar 23, 2020, 3:48 PM Miko998 @.***> wrote: Hi, sorry it's not an issue more a question, do you know how to select data based on some criteria, my question: https://stackoverflow.com/questions/60814823/select-data-from-excel-using-powershell/60817252#60817252, thanks in advance — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#808>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKW2GYP3VS2NHU3OHLGVWG3RI64I5ANCNFSM4LSECH3Q .

Many thanks


Syllabus - Missing label "No other course runs"

Bug Report
Problematic Behavior
On syllabus page, when there is a single opened course run and closed course runs, the section "Other sessions" is displayed but the label "No other opened sessions" is not displayed.
Expected behavior/code
In all case, when there is a single opened course run, a label should be displayed below the Other sessions title.
Steps to Reproduce
N/A
Environment

Richie version: 2.17.0
Platform: N/A

Possible Solution
Fix the misbehavior within [course-detail.html] template
Additional context/Screenshots
Example of misbehavior
https://www.fun-mooc.fr/fr/cours/unlock-your-english/

Example of expected behavior
https://www.fun-mooc.fr/fr/cours/mooc-conseil-innovation-en-agriculture/

Fixed by #2015


Cannot set Steam online status properly

Contacts get reported of my activity dspite being in "Not visible" status. Any workaround?

This is definitely a Steam client issue. You can report it here:
https://github.com/ValveSoftware/steam-for-linux/issues


user canceled is 90, rather than 80

https://tools.ietf.org/html/rfc5246#appendix-A.3 has user_canceled as 90 (0x5a in hexadecimal) rather than 0x50.

Wow, good catch.


NetworkClient.connection: type changed from NetworkConnection to explicit NetworkConnectionToServer to ensure correct usage. C# has a great type system, we should use to prevent issues.

we need to use explicit types for safety.
this way C# type system can catch errors.
notice the unit test being wrong, this wasn't caught before.
TODO

[ ] 2 failing tests

commited and closing


REFPLTV-735: RDK Services: Controller.1.startdiscovery and Controller…

….1.discoveryresults returning Empty reply from server error
Reason for change: Crash observed when Controller.1.startdiscovery and Controller.1.discoveryresults are run.
Test Procedure: Execute Controller.1.startdiscovery and Controller.1.discoveryresults curl command.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I think tis is causing havoc in the tracking and tracing of the GitHub system, please use the commits already submitted to resolve this and do not create new pull requests this issue is already resolved by commit: 9a4a4ece02d0bba8f89db03f5aa85aba36ddc5ad just cherry pick it on the Rel1.5 branch if required.

I think tis is causing havoc in the tracking and tracing of the GitHub system, please use the commits already submitted to resolve this and do not create new pull requests this issue is already resolved by commit: 9a4a4ece02d0bba8f89db03f5aa85aba36ddc5ad just cherry pick it on the Rel1.5 branch if required.

Raising a new PR as suggested.

Raising a new PR as suggested.


Update to swift 2.

This includes migrating all of our code and also third party libraries.
We should do a better job of getting on the latest version of those
third party libraries once we drop iOS 7 support and we can pick up
frameworks.
Also, Alamofire got refactored so that might end up being some work. I just migrated the current version we're using directly.
I tried to clean up all the warnings too, but there were a couple
associated with TODOs that I wanted to collect as tickets before
cleaning them.

cc @ehmadzubair @mikekatz Reviewing this properly isn't really a sane option, but an eyeball or random sampling couldn't hurt.

@mikekatz @ehmadzubair  I'm making progress, but I can't get this to work. For now just keep working off master :(

This got messy. Closing


AppErrors: migrating Relationship conformance and general improvements

This PR improves the type Relationship in a few ways.

Establish Relationship.MissingRelationship error type
Improves how Errors are managed by the class
Improves Relationship.queryAndSortAllRelatedPopups to leverage swift Result type

Thank you @philium. I've applied your suggestion and am assuming the rest is okay. Merging.


Minor fixes to English Python tutorial

The changes made are:

Added the missing exercise for the Code Introspection tutorial.
Fixed minor bugs both in the Regular Expressions and the List Comprehensions tutorials.
Fixed typing and formatting errors in the Closures tutorial.

Holy *, you're fast.

:)

Ron

[image: Facebook] http://www.facebook.com/ron.reiter [image: Twitter]
http://twitter.com/#!/ronreiter [image: LinkedIn]
http://www.linkedin.com/profile/view?id=4436607&locale=en_US&trk=tab_pro
On Wed, Sep 28, 2016 at 1:39 PM, Hildo Bijl<EMAIL_ADDRESS>wrote:

Holy *, you're fast.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ronreiter/interactive-tutorials/pull/204#issuecomment-250132345,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArFkBgO6X88axHNcoWEj9aiGzCBsvimks5qukPggaJpZM4KIqRe
.


[Bug]: As per Zeplin, since SAAS integration configs are not accounted for, there has to be a tool tip in the Manage Env screen to indicate the same

Is there an existing issue for this?

[X] I have searched the existing issues

Description
As per Zeplin, since SAAS integration configs are not accounted for in the DS configuration count, there has to be a tool tip in the Manage Env screen, next to the Configurations text, to indicate the same.
Steps To Reproduce

Login to appsmith as an admin user with rights to manage environments page
Create an application, and configure a google sheets DS
Create a page with a widget bound to the GS DS query and deploy it
Go to the Manage Env page
Note that though we have created a Datasource and configured it, Configurations column shows 0/0 configured DS

Now, this is because of the decision that SAAS integration configs will not be accounted for. However, as per discussion, and as per Zeplin designs, there should be a tooltip to indicate this to the user.
Zeplin screen -

DP screenshot -

Public Sample App
No response
Environment
Deploy Preview
Issue video log
No response
Version
Cloud

@btsgh this is not possible today due to ADS limitations


Tests

It would be good to have some tests, at least for the code which does not involve the twitter api.

All the non-trivial code involves the API…
On 06/03/2015 18:50, Tom Cornebize wrote:

It would be good to have some tests, at least for the code which does not involve the twitter api.

Reply to this email directly or view it on GitHub:
https://github.com/ProgVal/ppp_twitter_bot/issues/3

You could do a function which takes as input three strings (the sentence typed by the user on twitter, the name of the user, and the name of the bot), and outputs a string (the sentence answered on twitter).
With this, you could write tests which do not involve the twitter API, but are non-trivial.

An alternative which may be cleaner: using mock.


HDFS-14907. [Dynamometer] DataNode can't find junit jar when using Hadoop-3 binary

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
131
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
0
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1533
trunk passed

+1
mvnsite
34
trunk passed

+1
shadedclient
967
branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1
mvninstall
32
the patch passed

+1
mvnsite
27
the patch passed

-1
shellcheck
1
The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
1090
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
unit
26
hadoop-dynamometer-infra in the patch passed.

+1
asflicense
42
The patch does not generate ASF License warnings.

4025

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1680

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs

uname
Linux 4e61defae604 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 7be5508

shellcheck
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/1/artifact/out/diff-patch-shellcheck.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/1/testReport/

Max. process+thread count
306 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-infra U: hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-infra

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
78
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
0
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1299
trunk passed

+1
mvnsite
27
trunk passed

+1
shadedclient
824
branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1
mvninstall
28
the patch passed

+1
mvnsite
26
the patch passed

-1
shellcheck
0
The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
906
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
unit
23
hadoop-dynamometer-infra in the patch passed.

+1
asflicense
31
The patch does not generate ASF License warnings.

3371

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1680

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs

uname
Linux e6805440b6f9 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 7be5508

shellcheck
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/2/artifact/out/diff-patch-shellcheck.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/2/testReport/

Max. process+thread count
367 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-infra U: hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-infra

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/2/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
75
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
0
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1250
trunk passed

+1
mvnsite
27
trunk passed

+1
shadedclient
842
branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1
mvninstall
29
the patch passed

+1
mvnsite
24
the patch passed

+1
shellcheck
1
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
909
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
unit
24
hadoop-dynamometer-infra in the patch passed.

+1
asflicense
33
The patch does not generate ASF License warnings.

3340

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1680

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs

uname
Linux f99695c42aee 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / d5e9971

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/3/testReport/

Max. process+thread count
333 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-infra U: hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-infra

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/3/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

Thanks for your reviews, @xkrogen! Updated the PR.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
36
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
0
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1083
trunk passed

+1
mvnsite
34
trunk passed

+1
shadedclient
776
branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1
mvninstall
30
the patch passed

+1
mvnsite
26
the patch passed

+1
shellcheck
1
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
814
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
unit
28
hadoop-dynamometer-infra in the patch passed.

+1
asflicense
38
The patch does not generate ASF License warnings.

2999

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1680

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs

uname
Linux 63476cb388e5 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 30ed24a

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/4/testReport/

Max. process+thread count
447 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-infra U: hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-infra

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1680/4/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

LGTM! Thanks @tasanuma !


Update install_rocm.rst

It fixes confusing explanation in "Install CuPy from Source".
There's no pre-built wheels for ROCm environments for now, so it's better to mention it and remove the descriptions about NVIDIA libraries here.

Jenkins, test this please

Jenkins CI test (for commit 6b2d0f9da3564679989513b314687dd56d169a25, target branch master) succeeded!


Difference between Processed and Exported

I am running osxphotos on my photo library which is about 200Gb to back up to my external drive. After the end of the program I get the report saying:
Processed: 27776 photos, exported: 39613, updated: 9, skipped: 1, updated EXIF data: 39616, missing: 125, error: 254

What is the difference between Processed and exported? Does that mean some photos were not transferred to my drive?
These are the options I've used, as stored on my config.toml file
[export]
directory = "{folder_album|lower,camera roll}"
download_missing = true
edited_suffix = "_edit"
exiftool = true
exiftool_option = [ "-m",]
filename_template = "{created.strftime, %Y%m%d-%H%M%S%f}"
keyword_template = [ "{label}",]
report = "export.csv"
theme = "dark"
update = true

Processed is the number of assets in the library that OSXPhotos evaluated for export. This should match the total assets in the library (OSXPhotos includes hidden, shared and syndicated images that might not appear in the total shown at the bottom of the Photos library view though). Exported is the number of files exported. An asset might have more than one file (live images or raw+jpeg for example). So exported might be greater than processed.
You can use the osxphotos exportdb  command to produce a report of the last export that gives much more detail (same as if you had used --report with the export). See osxphotos help exportdb

Need to find a way to make the reporting more clear so users understand the numbers reported by osxphotos.


uuid feature pulls in database specific code

Bug Description
When including sqlx with no features but uuid, it will pull in database specific dependencies (e.g. sqlx-postgres, sqlx-sqlite and sqlx-mysql
Minimal Reproduction
Create an empty crate with only the following dependency, build/check and look into Cargo.lock:
sqlx = { version = "0.7.4", default-features = false, features = ["uuid"] }

I get the following entry in my Cargo.lock:
[[package]]
name = "sqlx"
version = "0.7.4"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "c9a2ccff1a000a5a59cd33da541d9f2fdcd9e6e8229cc200565942bff36d0aaa"
dependencies = [
 "sqlx-core",
 "sqlx-macros",
 "sqlx-mysql",
 "sqlx-postgres",
 "sqlx-sqlite",
]

Info

SQLx version: 0.7.4
SQLx features enabled: uuid
Database server and version: not applicable
Operating system: Linux
rustc --version: 1.77.2

Known issue, see #3211

Sorry I missed this and didn't look further for other issues...


👷 combine workflows

Generated summary (powered by Graphite)

Pull Request Description
TL;DR
This pull request removes the publish.yaml workflow and integrates its functionality into the release.yaml workflow. This change simplifies the process of creating a release and publishing a package to PyPI by combining them into a single workflow.
What changed

The publish.yaml workflow file was deleted.
The release.yaml workflow file was updated to include the steps previously in publish.yaml.
The create-release job in release.yaml was renamed to create-release-and-publish.
The Install dependencies step was replaced with Install Poetry and Install dependencies with Poetry steps.
The Determine Version Change step was updated to use Poetry to get the current version.
The Create Release step was updated to only run if the version has changed.
New steps Configure Poetry for PyPI and Build and publish to PyPI were added to publish the package to PyPI if the version has changed.

How to test
To test this change, you can create a new release on the main branch. The create-release-and-publish job should run and if the version has changed, it should create a new release and publish the package to PyPI.
Why make this change
This change simplifies the release and publish process by combining them into a single workflow. This reduces the number of workflow files and makes the process easier to manage. It also ensures that a new release is only created and published to PyPI if the version has changed, preventing unnecessary releases.

Current dependencies on/for this PR:

main

PR #70   👈

PR #71

This stack of pull requests is managed by Graphite.

Merge activity

Dec 31, 11:52 AM: @mingi3314 started a stack merge that includes this pull request via Graphite.


Configure webhook for mce v1

Webhook wasn't functioning because it was configured for v1alpha1 rather than the new v1 MCE apiversion
Signed-off-by: Jakob Gray<EMAIL_ADDRESS>

/lgtm


[Improvement][db operation] mybatis mapper xml dynamic sql should not use too much "select *"

Now, DS use mybatis as orm framework. There's too much "select *", which will affect database query performance .

I will optimize mybatis dynamic sql.

please check the pr, https://github.com/apache/incubator-dolphinscheduler/pull/3940. @dailidong

close by #3940


events: Fix buggy update starred event.

Fixes #10111.

Merged, thanks @lonerz!  It'd be nice if we had node tests for this code path, I suppose.


client-go/testing: Can't list deployments after scaling

What happened:
I wanted to test my method to scale deployments. But after changing the replicas by using updateScale I was not able to list deployments anymore.
What you expected to happen:

create fake api
create mock deployment
scale it
list it

How to reproduce it (as minimally and precisely as possible):

create fake api
create a mock deployment
scale the deployment
list the deployment

Anything else we need to know?:
package scaling

import (
    "testing"

    v1api "k8s.io/api/apps/v1"
    autoscalingv1 "k8s.io/api/autoscaling/v1"
    metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
    "k8s.io/client-go/kubernetes/fake"

    "github.com/stretchr/testify/require"
)

func TestScaling(t *testing.T) {
    initialReplicas := int32(3)
    cs := fake.NewSimpleClientset()
    _, err := cs.AppsV1().Deployments("test-namespace").Create(&v1api.Deployment{
        ObjectMeta: metav1.ObjectMeta{
            Name: "test-deployment",
            Labels: map[string]string{
                "app": "test-app",
            },
        },
        Spec: v1api.DeploymentSpec{
            Replicas: &initialReplicas,
        },
    })
    require.NoError(t, err)
    scale := &autoscalingv1.Scale{
        ObjectMeta: metav1.ObjectMeta{
            Name:      "test-deployment",
            Namespace: "test-namespace",
        },
        Spec: autoscalingv1.ScaleSpec{
            Replicas: 5,
        },
    }
    _, err = cs.AppsV1().Deployments("test-namespace").UpdateScale("test-deployment", scale)
    require.NoError(t, err)
    _, err = cs.AppsV1().Deployments("test-namespace").List(metav1.ListOptions{FieldSelector: "app=test-app"})
    require.NoError(t, err)
}

Run like this:
go test -run TestScaling ./scaling

This will lead to an error like the following:
item[0]: can't assign or convert v1.Scale into v1.Deployment

Environment:

go version: go1.13.5 linux/amd64
client-go version<EMAIL_ADDRESS>related issue(s)/pr(s):

80354
410
579

I'm also encountering this issue with Statefulsets.
@lynx-coding have you had any luck with tracking down a fix or work around?
From digging around a bit, it looks like Scale is a psuedo resource rather than a true subresource which may be the cause for issues?


Converting tardis data results in error

date = '2023-08-08'

pair = 'ltc_usdt'
tick_size = tick_and_lot[pair]['tick_size']
lot_size = tick_and_lot[pair]['lot_size']

path = f'data/{pair.split("_")[0]}_tardis_npzs/{pair.replace("_", "")}_{date}_tardis.npz'

# Convert automatically cals validate data no need for us to do so
tardis.convert(["/".join(path.split("/")[:-1]) + "/tmp_ob.csv.gz","/".join(path.split("/")[:-1]) + "/tmp_trades.csv.gz"], output_filename=path)

Error:
    156 ss_ask = ss_ask[:ss_ask_rn]
    157 # Clear the ask market depth within the snapshot ask range.
    158 tmp[row_num] = [
    159     DEPTH_CLEAR_EVENT,
--> 160     ss_ask[0, 1],
    161     ss_ask[0, 2],
    162     -1,
    163     ss_ask[-1, 4],
    164     0
    165 ]
    166 row_num += 1
    167 # Add DEPTH_SNAPSHOT_EVENT for the ask snapshot

IndexError: index 0 is out of bounds for axis 0 with size 0

in file tardis.py
Here is a screenshot from the Debugger (when the ss_ask is empty):

Maybe a useful note will be that the timestamp and local_timestamp sometime may be a bit shifted wrt what tardis gives (at maximum - one microsecond (sometimes)).

Could you please check if this fix resolves the issue? It would be even better if you could verify whether the snapshot is indeed provided only for one side.
Additionally, could you provide more details regarding the local_timestamp issue?

I have three snapshot events (for both sides) at times:
2023-08-08 00:00:00.023, 2023-08-08 05:31:00.545 and 2023-08-08 12:00:02.582 , (I get 1001 values per side per timestamp).
With the solution that you provided now - it works. Thank you very much.
Regarding the timestamp issue, it comes due to some pandas conversions (since it doesn't have microsecond support iirc), mapping to and from datetime objects sometimes causes a discrepancy in the magnitude of a microsecond (at max). It is totally from my side.
Thanks again!

Should this fix be merged to the master branch?

@pepi99  By any chance, can you tell me what this means IIRC?

iirc = if I remember correctly

Sure. It has been merged. Thanks for checks up.


Documentation site

Creating a placeholder to keep track of documentation status. There is no other obvious place.

@dywsjtu What's the current status?

@dywsjtu Please try our website repo. I created an individual page for this.

I think the docs source folder should be part of code repo, no? The compiled thing can be in the website repo.

I am still developing the documentation locally, and haven't posted it on the website yet. I will finish the first version in two days and we can discuss it. Yes, I think the docs source folder should be part of the code repo.

Close this one? @fanlai0990 @dywsjtu


UnsupportedError: Unsupported operation: Illegal character in path: assets\google_fonts

Flutter crash report.
Please report a bug at https://github.com/flutter/flutter/issues.
command
flutter --quiet assemble --depfile D:__Projects__Mobile\ecommerce\build\app\intermediates\flutter\debug/flutter_build.d --output D:__Projects__Mobile\ecommerce\build\app\intermediates\flutter\debug -dTargetFile=D:__Projects__Mobile\ecommerce\lib\main.dart -dTargetPlatform=android -dBuildMode=debug -dTrackWidgetCreation=true --DartDefines=flutter.inspector.structuredErrors%3Dtrue debug_android_application
exception
UnsupportedError: Unsupported operation: Illegal character in path: assets\google_fonts
#0      _Uri._checkWindowsPathReservedCharacters (dart:core/uri.dart:1734:11)
#1      _Uri._toWindowsFilePath (dart:core/uri.dart:2748:7)
#2      _Uri.toFilePath (dart:core/uri.dart:2720:38)
#3      _parseAssetsFromFolder (package:flutter_tools/src/asset.dart:749:27)
#4      _parseAssets (package:flutter_tools/src/asset.dart:689:7)
#5      ManifestAssetBundle.build (package:flutter_tools/src/asset.dart:163:53)
<asynchronous suspension>
#6      copyAssets (package:flutter_tools/src/build_system/targets/assets.dart:48:44)
#7      AndroidAssetBundle.build (package:flutter_tools/src/build_system/targets/android.dart:65:42)
#8      _BuildInstance._invokeInternal (package:flutter_tools/src/build_system/build_system.dart:801:27)
<asynchronous suspension>
#9      _BuildInstance.invokeTarget.<anonymous closure> (package:flutter_tools/src/build_system/build_system.dart:744:35)
#10     new Future.sync (dart:async/future.dart:223:31)
#11     AsyncMemoizer.runOnce (package:async/src/async_memoizer.dart:43:45)
#12     _BuildInstance.invokeTarget (package:flutter_tools/src/build_system/build_system.dart:744:21)
<asynchronous suspension>
#13     FlutterBuildSystem.build (package:flutter_tools/src/build_system/build_system.dart:566:36)
#14     AssembleCommand.runCommand (package:flutter_tools/src/commands/assemble.dart:221:58)
#15     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:977:18)
<asynchronous suspension>
#16     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:836:33)
<asynchronous suspension>
#17     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart)
#18     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
#19     _rootRun (dart:async/zone.dart:1190:13)
#20     _CustomZone.run (dart:async/zone.dart:1093:19)
#21     _runZoned (dart:async/zone.dart:1630:10)
#22     runZoned (dart:async/zone.dart:1550:10)
#23     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#24     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:825:20)
#25     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
#26     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:335:21)
<asynchronous suspension>
#27     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart)
#28     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
#29     _rootRun (dart:async/zone.dart:1190:13)
#30     _CustomZone.run (dart:async/zone.dart:1093:19)
#31     _runZoned (dart:async/zone.dart:1630:10)
#32     runZoned (dart:async/zone.dart:1550:10)
#33     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#34     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:281:19)
<asynchronous suspension>
#35     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:112:25)
#36     new Future.sync (dart:async/future.dart:223:31)
#37     CommandRunner.run (package:args/command_runner.dart:112:14)
#38     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:230:18)
#39     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:63:22)
#40     _rootRun (dart:async/zone.dart:1190:13)
#41     _CustomZone.run (dart:async/zone.dart:1093:19)
#42     _runZoned (dart:async/zone.dart:1630:10)
#43     runZonedGuarded (dart:async/zone.dart:1618:12)
#44     runZoned (dart:async/zone.dart:1547:12)
#45     run.<anonymous closure> (package:flutter_tools/runner.dart:61:18)
<asynchronous suspension>
#46     run.<anonymous closure> (package:flutter_tools/runner.dart)
#47     runInContext.runnerWrapper (package:flutter_tools/src/context_runner.dart:67:18)
<asynchronous suspension>
#48     runInContext.runnerWrapper (package:flutter_tools/src/context_runner.dart)
#49     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
#50     _rootRun (dart:async/zone.dart:1190:13)
#51     _CustomZone.run (dart:async/zone.dart:1093:19)
#52     _runZoned (dart:async/zone.dart:1630:10)
#53     runZoned (dart:async/zone.dart:1550:10)
#54     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#55     runInContext (package:flutter_tools/src/context_runner.dart:70:24)
#56     run (package:flutter_tools/runner.dart:46:10)
#57     main (package:flutter_tools/executable.dart:77:9)
#58     main (file:///D:/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#59     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299:32)
#60     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

flutter doctor
[✓] Flutter (Channel stable, 1.22.5, on Microsoft Windows [Version 10.0.19042.685], locale en-US)
    • Flutter version 1.22.5 at D:\flutter
    • Framework revision<PHONE_NUMBER> (5 weeks ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\tarek\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: D:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[!] Android Studio (version 4.1.0)
    • Android Studio at D:\Program Files\Android\Android Studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] VS Code (version 1.52.1)
    • VS Code at C:\Users\tarek\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.18.1

[✓] Connected device (1 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86 • Android 11 (API 30) (emulator)

! Doctor found issues in 1 category.

@LebToki  can you include the pubspec.yaml for the project that crashed this way?

@LebToki  can you include the pubspec.yaml for the project that crashed this way?

sorry for deleting the project and starting afresh with a standard font waiting for a resolution to switch :-)

sorry for deleting the project and starting afresh with a standard font waiting for a resolution to switch :-)


Created Search page of spotify

@TaufeeqRiyaz

@TaufeeqRiyaz i have created the search page of spotify. kindly label this PR with hactoberfest and hacktoberfest-accepted

Great!
Merged, Thank you for contributing :)


Using AddVersionedApiExplorer() with AddMvc()

It seems the AddVersionedApiExplorer() call is always used together with AddMvcCore(). It does not seem to be available with the standard AddMvc() builder. Is this correct? Do I have to convert my application to use the AddMvcCore() builder instead if I want the versioned version?

Thanks for your explanation. I think I understand fairly well the difference between the two ways of setting up MVC (I have used both in different projects).
Right now I stand with a project using UseMvc(). My question is, do I have to "convert" my project to use UseMvcCore() instead, to be able to then call AddVersionedApiExplorer()? It might not seem like a big change, but it's still a bit out-of-scope for the feature request "Add a V2 version of a certain controller".

You are of course correct. I'm sorry, my previous post was incorrect. I meant AddMvc(), not UseMvc(). I will reformulate it here with correct terminology:
Right now I stand with a project using AddMvc(). My question is, do I have to "convert" my project to use AddMvcCore() instead, to be able to then call AddVersionedApiExplorer()? (This method call is not available on the builder returned from AddMvc().)
Thanks for your patience.

No problem. You can, and will ultimately need, to call both. AddMvc() calls AddMvcCore() internally. There's nothing wrong with calling them both, even multiple times.
Since you're using the full MVC stack, you should expect your setup to update to:
public void ConfigureServices( IServiceCollection services )
{
    // the api explorer doesn't require the UI and you might not be using it,
    // which is why it's on AddMvcCore
    services.AddMvcCore().AddVersionedApiExplorer();
    services.AddMvc();
    services.AddApiVersioning();

The Swagger Sample shows a more complete example. I hope that helps and clears things up. Don't hesitate to keep asking questions.

Great, thanks! Excellent support! 🌟

I hope you don't mind if I jump in on this closed question. I ran into the same situation where I was using AddMvc() for my API project and noticed I can only use AddVersionedApiExplorer() on AddMvcCore().
I thought hey, for a pure API project, it would make sense to not have Razor pages and all wired up, so that would be a logical change anyhow - and so I switched from AddMvc to AddMvcCore. But while the API was still working, Swagger UI did not. It showed up, but always had empty definitions.
I finally noticed in the Swagger sample project that not only AddMvcCore() is called, but afterwards also AddMvc(). @commonsensesoftware also mentioned that we will and have to call both. That seems a little odd and counter-productive, but ultimately I want to understand why. What is it that AddMvc() adds that is needed by the Swagger UI to actually show any definitions?
Because I tried to manually chain everything that is done inside AddMvc() manually to my AddMvCore() call - but it still does not work:
            services.AddMvcCore()
            .AddAuthorization()
            .AddFormatterMappings()
            .AddViews()
            .AddRazorViewEngine()
            .AddCacheTagHelper()
            .AddVersionedApiExplorer(o =>
            {
                o.AssumeDefaultVersionWhenUnspecified = true;
                o.DefaultApiVersion = new ApiVersion(2, 0);
            })
            .AddDataAnnotations()
            .AddJsonFormatters()
            .AddCors()
            });

I even added everything inside AddDefaultFrameworkParts to my code but still no luck.
So I'd really be interested on what exactly is happening behind the scenes when calling both, AddMvcCore() and AddMvc(), that makes Swagger UI suddenly work as expected (as of: showing actual API definitions).
It really bugs me ... so if anyone can shed some light on this, that would be great.

There is likely a combination of things happening. Registering services is more tolerant than registering routes when it comes to order. The key thing for services is any service that might be replaced. It would appear that you've looked at the source code so you've seen that AddMvc() ultimately calls AddMvcCore() on your behalf.
That said, this could a case of inversed registration. First, AddVersionedApiExplorer() does not call AddApiVersioning(). Maybe it should? Second, the options you are specifying for AddVersionedApiExplorer() are not really meant to be explicitly set by you and have completely different meaning than when you use them in AddApiVersioning(). These option values are automatically copied over from the ApiVersioningOptions. Their use is purely related to API exploration in this context and have zero effect on routing, etc. The only reason they are visible at all is for extensibility purposes. The missing call to AddApiVersioning() could be what's missing here.
Another issue could be the route configuration, which you didn't show (e.g. the UseXXX methods). The order of these do matter and Swashbuckle (interchangeably called Swagger) does register it's own middleware to hook up the Swagger-related routes.
Sorry I don't have anything more specific, but that should put you on the path.

Thank you, that was more than specific enough. I called AddApiVersioning() later on, just hadn't it included here. For the options to AddVersionedApiExplorer(), i removed them, but it did not make a difference. I will keep them removed, thanks for your advice! But having GroupNameFormat is ok there, right? As it's also there in the sample project.
But based on the information you gave me, and the pure fact that you expect it to work without AddMvc() put me indeed on the right path: It's AddApiExplorer that needs to be called! It's called in AddMvc() and I didn't include it in my setup on AddMvcCore() as I jumped to the conclusion that AddVersionedApiExplorer() replaces AddApiExplorer() completely, but that seems not to be the case.
So what I have now is:
           services.AddMvcCore()
            .AddJsonFormatters()
            .AddDataAnnotations()
            .AddVersionedApiExplorer(options =>
            {
                options.GroupNameFormat = "'v'VVV";
            })
            .AddApiExplorer()
```csharp

And with that, I do not need `AddMvc()` anymore.

So thank you very much - that's a relief :)

Just one question: I think the sample project should do this change too - remove `AddMvc()` and instead just call `AddApiExplorer()` in addition to `AddVersionedApiExplorer()`. It would make the sample more clear on what's needed to be done and my whole adventure started by overlooking the additional `AddMvc()` call in the sample. If you agree, I'd also do the pull request for this change if you like.

Typically, the only options you're ever interested in for the API Explorer is:

GroupNameFormat
SubstitutionFormat (URL segment style only)
SubstituteApiVersionInUrl (URL segment style only)
DefaultApiVersionParameterDescription

GroupNameFormat is generally the only thing you need to change from the out-of-the-box experience. If you're using the URL segment style, then most prefer SubstituteApiVersionInUrl = true as well. Everything else is choice, but is usually fine as is.
I see the confusion. The API Explorer model in ASP.NET Core is quite different (and more robust) than it was in Web API. The default API Explorer actual discovers everything, but there is no organization to it. The API Versioning provider runs at the end of the API Explorer pipeline and collates all the APIs into appropriate buckets by API version. Technically, AddVersionedApiExplorer() doesn't depend on AddApiExplorer(), but it also doesn't make sense without it so it probably should call AddApiExplorer() on your behalf. I'll consider adding that to the 3.0 work in progress.
Your suggestion is valid, but - unfortunately - I find a lot of people mixing and matching API versioning with UIs. I don't know why anyone needs API versioning if you own both the UI and API in the same application, but I see it quite often. For that reason, the current example should probably stay as is. It's useful for those mixing in with the full MVC stack and I use the samples quite often as the basis of for troubleshooting.
That said, I'm not at all opposed to an entirely new sample. Perhaps something called ApiOnlySample that has the setup you are describing. The controllers will need to change their base type to ControllerBase instead of Controller as well. There may be other changes required too, but that's what I can think of off the top of my head.

Thank you again for your extensive response, this was very helpful to me.
I agree that, considering people mixing their API projects with UIs, the current sample is the best option as it works in every scenario.
If you call AddApiExplorer() on our behalf in the future, which I would really love to see, I also think there is no need for an additional sample project for an ApiOnlySample as, should someone like me decide to leave AddMvc() out for a pure API project, it will still work as expected.


CLI rewrite

For #282.
At this point, none of the (CLI)test work (nor compile).

But now it should be pretty easy to add new functionality to the CLI.

Yup, but only in the case of how inlyne view is handled (before it would open a markdown file named "view", while now it runs the subcommand)

Oh, you're right.


[#130003999] Remove container size info from terraform README.

What
This no longer applies since we've upgraded to 0.7 (in fa630ad) as
terraform now has a single binary that includes all the built-in
providers.
How to review
Review update README and confirm it makes sense to remove these details.
Who can review
Anyone but myself.

Indeed.


Fixed Fahrenheit config option not being detected

The default case for units in clap makes the check for units later return true even if no unit arguments were supplied.

Hey @Piggggggggggy thanks for your contribution,
there was an issue with the fahrenheit unit arg not being detected which was fixed in #2. The fix wasn't contained in the crate until the 0.1.2 release which was just a few minutes ago.

Hey I just finished a release script for binary files and it would be great to have this issue resolved before creating them. I'll do a quick rebase to a fix branch to do those little edits and still keep your commit.


Add the yaml env load

Type of change
Note: Fill x in []

[ ] bug
[x] enhancement
[ ] documentation
[ ] dependencies

Description

Add environment variables load by yaml file.
For security reasons, all pull requests need to be approved first before running any automated CI

@athiruma, Can you pls add e2e test that using yaml file ?

Added


BBC micro:bit - detected but not working

Hi there,
I'm trying to get an Android phone to talk to a BBC micro:bit, which is similar to an ARM mbed. It presents a composite device to the host; a Mass Storage class for loading firmware, a HID class, and a Communications class with CDC Interrupt endpoint and a CDC Data endpoint. The micro:bit is recognised by Mac OS X and Linux without additional drivers and can be used with terminal software.
The micro:bit uses the same USB VID/PID as the NXP LPC1768 - 0d28:0204.
However, the micro:bit uses a nRF51822 as its main processor and an Kinetis KL26 as a USB controller.
Output of lsusb -vvv:
Bus 003 Device 076: ID 0d28:0204 NXP LPC1768
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               1.10
  bDeviceClass            2 Communications
  bDeviceSubClass         0
  bDeviceProtocol         0
  bMaxPacketSize0        64
  idVendor           0x0d28 NXP
  idProduct          0x0204 LPC1768
  bcdDevice           10.00
  iManufacturer           1 MBED
  iProduct                2 MBED CMSIS-DAP
  iSerial                 3<PHONE_NUMBER>864e45002e10060000005000000000cc4d28bd
  bNumConfigurations      1
  Configuration Descriptor:
    bLength                 9
    bDescriptorType         2
    wTotalLength          122
    bNumInterfaces          4
    bConfigurationValue     1
    iConfiguration          0
    bmAttributes         0x80
      (Bus Powered)
    MaxPower              500mA
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        0
      bAlternateSetting       0
      bNumEndpoints           2
      bInterfaceClass         8 Mass Storage
      bInterfaceSubClass      6 SCSI
      bInterfaceProtocol     80 Bulk-Only
      iInterface              7 USB_MSC
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            2
          Transfer Type            Bulk
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               0
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x02  EP 2 OUT
        bmAttributes            2
          Transfer Type            Bulk
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               0
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        3
      bAlternateSetting       0
      bNumEndpoints           2
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      0 No Subclass
      bInterfaceProtocol      0 None
      iInterface              6 MBED CMSIS-DAP
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.00
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      33
         Report Descriptors:
           ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x81  EP 1 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x01  EP 1 OUT
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         2 Communications
      bInterfaceSubClass      2 Abstract (modem)
      bInterfaceProtocol      1 AT-commands (v.25ter)
      iInterface              4 USB_CDC
      CDC Header:
        bcdCDC               1.10
      CDC Call Management:
        bmCapabilities       0x03
          call management
          use DataInterface
        bDataInterface          2
      CDC ACM:
        bmCapabilities       0x06
          sends break
          line coding and serial state
      CDC Union:
        bMasterInterface        1
        bSlaveInterface         2
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x83  EP 3 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0010  1x 16 bytes
        bInterval              32
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        2
      bAlternateSetting       0
      bNumEndpoints           2
      bInterfaceClass        10 CDC Data
      bInterfaceSubClass      0 Unused
      bInterfaceProtocol      0
      iInterface              5 USB_CDC1
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x04  EP 4 OUT
        bmAttributes            2
          Transfer Type            Bulk
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               0
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x84  EP 4 IN
        bmAttributes            2
          Transfer Type            Bulk
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               0
Device Status:     0x0000
  (Bus Powered)

I have purchased DroidTerm PRO, but haven't tried writing my own Java code yet, so I haven't got any additional debug logs yet.
On the micro:bit I have a short program that repeatedly sends a string to the USB serial output at 115200 baud every second. DroidTerm PRO recognises that there is an "LPC1768" attached, but no text appears. It works fine on a Linux PC or Mac.
I have another program that scrolls characters across the screen of the micro:bit when they are received from the computer over serial. Again, this works on the desktop, but not with DroidTerm PRO. However, the LED on the micro:bit blinks when I send a character from DroidTerm - so something is happening! :)
Do you have any ideas what might be wrong, or some hints for debugging this?
Thanks!

Hi @dwhinham
Could you post a screenshot from DroidTerm USB viewer? It would be help to see how DroidTerm is seeing this hardware

Hi @felHR85,
Did some more testing and found that it's actually a problem switching baud rates, so I've updated the issue title.
If I set the micro:bit firmware to use 9600 baud and DroidTerm to match, it works perfectly and I can send and receive data just fine.
If I change to any other baud rate in the firmware, it stops working (but works fine using a PC connection as already described).
I wrote a basic Android app using the library and watched the logs.
After serial.open() is called on the device:
I/CDCSerialDevice: Interface succesfully claimed
I/CDCSerialDevice: Control Transfer Response: -1
I/CDCSerialDevice: Control Transfer Response: -1

And then after serial.setBaudRate() is called:
I/CDCSerialDevice: Control Transfer Response: -1
I/CDCSerialDevice: Control Transfer Response: -1

...and the same for the setParity() and setFlowControl() functions.
Here are some screenshots of how DroidTerm Pro sees the device:


New version: Dogramming.Imaconverter version imaconverter2.0

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/113626&drop=dogfoodAlpha

Automatic Validation ended with:

(Inconclusive)

Manual Validation ended with:


Remove process.env.NODE_ENV completely in favor of development build for browser compatibility

process.env.NODE_ENV is not supported in browsers and causes an error, but of course this is known already and talked about in several closed issues. It's still a problem though for users of buildless dev setups.
I think a better dev experience, and definitely better browser compatibility, could be provided with the production and development builds already exposed with the "development" and "default" package exports. The development build should have the extra warnings, with no check for process.env.NODE_ENV necessary, and the default builds should not include the extra warnings, again with no check for process.env.NODE_ENV.
Because "development" is an officially recommended export condition, it should become increasingly common for development tools to use that condition when resolving import specifiers in the development modes, completely replacing and eliminating the need to check the browser-incompatible process.env.NODE_ENV at all.
With Node's native support for export conditions, this should also allow you to remove the browser builds and just product one default and one development builds.

What's wrong with the new "browser" export condition which has browser-ready default/development modules?


Add procedure to configure autoscaling nodes

Add procedure to configure autoscaling nodes on a cluster using the rosa CLI. This goes in the main doc set as it is written as a tutorial.
JIRA: https://issues.redhat.com/browse/OSDOCS-1701

QE and peer review needed. No preview available, please review in the PR.
@yuwang-RH  @vikram-redhat  @jeremyeder  @okashi18

Based on @okashi18 feedback, I made changes to the CLI reference guide too for rosa edit machinepool.


offline camera perception: lane dection

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 14.04):
Apollo installed from (source or binary):
Apollo version (1.0, 1.5, 2.0, 2.5, 3.0):
latest mater branch

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:

Supporting materials (screenshots, command lines, code/script snippets):
E0719 16:35:30.773737 29714 lane_detection_component.cc:308] [perception] InternalProc failed, error_code: 4001
E0719 16:35:30.786329 29719 transform_wrapper.cc:224] [mainboard] Can not find transform.<PHONE_NUMBER>.302338 frame_id: novatel child_frame_id: front_12mm Error info: canTransform: target_frame novatel does not exist. canTransform: source_frame front_12mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_12mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_12mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_12mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_12mm does not exist.
E0719 16:35:30.786394 29719 lane_detection_component.cc:602] [perception] failed to get camera to world pose, ts:<PHONE_NUMBER>.302338 camera_name: front_12mm
E0719 16:35:30.786429 29719 lane_detection_component.cc:308] [perception] InternalProc failed, error_code: 4001
E0719 16:35:31.033892 29721 transform_wrapper.cc:224] [mainboard] Can not find transform.<PHONE_NUMBER>.352347 frame_id: novatel child_frame_id: front_6mm Error info: canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.canTransform: target_frame novatel does not exist. canTransform: source_frame front_6mm does not exist.
E0719 16:35:31.033977 29721 lane_detection_component.cc:602] [perception] failed to get camera to world pose, ts:<PHONE_NUMBER>.352347 camera_name: front_6mm
E0719 16:35:31.034008 29721 lane_detection_component.cc:308] [perception] InternalProc failed, error_code: 4001

@chenjo
Does your transform module runs successfully?
You should adapt transform/conf/static_transform_conf.pb.txt before running it.

@xmyqsh, now I have this error: load camera extrisic file  error, YAML exception:bad file for offline camera_perception and offline lane detection. The following error for the traffic light detection.
E0722 16:24:51.829178 26336 hdmap_common.cc:183] [map] lane[id = id: "11267dup0_1_-1"
[perception_trafficlights]  ]. sampled_right_width_[0.991181] is too small. It should be larger than half vehicle width[1.05].
I am replaying a 100 sec recording from baidu without calibration files attached, just for camera perception evaluation. Can you help me find the right calibration files.  Other sensor calibration files are also missing. The recording dated 2019-04-24-16-26-08 along the sunnyvale big loop.

@chenjo Hi! Would you like to tell me with how you visualize the perception module? I would appreciate that.


Update to Angular 6

Would be good to update to Angular 6.

Have you tried it out with Angular 6?
Any specific problems?

I've tried but I dont remember the specific problems. I believe it was something about rxjs that has changed to other version that is not totally compatible with previous one.

Hey., Trying installing rxjs-compat library.

 Package "angular5-toaster" has an incompatible peer dependency to "rxjs" (requires "^5.0.0-beta.11", would install "6.2.2").

Hi, i have created a version for angular 6 and pushed it to my fork of this repo. Maybe you want to try it. https://github.com/MCangularGitHub/angular5-toaster
Maybe i would suggest it to be added to the official master branch, but i am new to github and have to find out how this works properly.

@MCangularGitHub how can I install your version for angular 6? Its the same command npm install angular5-toaster..?

@grabovacnem Hi! No, i am sorry. You can go to my fork and download the source and copy it to your project. The owner of this library didn't respond to my update so far.


Added activation offload

Have passed the unit test.

https://github.com/Oldpan/Pytorch-Memory-Utils
Can you use this tool to check current tensors in CPU and GPU.

https://github.com/Oldpan/Pytorch-Memory-Utils Can you use this tool to check current tensors in CPU and GPU.

Hi, sorry for the late reply. I have used this tool and found that all tensors were in their positions.


Can't create superuser

Tried to create a superuser in an Ubuntu and in a macOS environment getting this exception:
(.virtualenv) asterix:OpenSlides pingus$ python manage.py createsuperuser
Traceback (most recent call last):
File "/Users/pingus/Documents/Projects/OpenSlides/.virtualenv/lib/python3.5/site-packages/django/contrib/auth/hashers.py", line 129, in get_hasher
return hashers[algorithm]
KeyError: 'md5'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "manage.py", line 11, in
exit(main())
File "/Users/pingus/Documents/Projects/OpenSlides/openslides/main.py", line 37, in main
execute_from_command_line(sys.argv)
File "/Users/pingus/Documents/Projects/OpenSlides/.virtualenv/lib/python3.5/site-packages/django/core/management/init.py", line 367, in execute_from_command_line
utility.execute()
File "/Users/pingus/Documents/Projects/OpenSlides/.virtualenv/lib/python3.5/site-packages/django/core/management/init.py", line 359, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/Users/pingus/Documents/Projects/OpenSlides/.virtualenv/lib/python3.5/site-packages/django/core/management/base.py", line 294, in run_from_argv
self.execute(*args, **cmd_options)
File "/Users/pingus/Documents/Projects/OpenSlides/.virtualenv/lib/python3.5/site-packages/django/core/management/base.py", line 345, in execute
output = self.handle(*args, **options)
File "/Users/pingus/Documents/Projects/OpenSlides/openslides/users/management/commands/createsuperuser.py", line 13, in handle
created = User.objects.create_or_reset_admin_user()
File "/Users/pingus/Documents/Projects/OpenSlides/openslides/users/models.py", line 49, in create_or_reset_admin_user
admin.password = make_password(admin.default_password, '', 'md5')
File "/Users/pingus/Documents/Projects/OpenSlides/.virtualenv/lib/python3.5/site-packages/django/contrib/auth/hashers.py", line 79, in make_password
hasher = get_hasher(hasher)
File "/Users/pingus/Documents/Projects/OpenSlides/.virtualenv/lib/python3.5/site-packages/django/contrib/auth/hashers.py", line 133, in get_hasher
"setting?" % algorithm)
ValueError: Unknown password hashing algorithm 'md5'. Did you specify it in the PASSWORD_HASHERS setting?
Additional version infos:
(.virtualenv) asterix:OpenSlides pingus$ python --version
Python 3.5.2
(.virtualenv) asterix:OpenSlides pingus$ pip --version
pip 8.1.1 from /Users/pingus/Documents/Projects/OpenSlides/.virtualenv/lib/python3.5/site-packages (python 3.5)
(.virtualenv) asterix:OpenSlides pingus$ node --version
v6.5.0
(.virtualenv) asterix:OpenSlides pingus$ npm --version
3.10.3

In Django 1.10.x they removed our MD5PasswordHasher. We have to readd it to global settings. https://docs.djangoproject.com/en/1.10/ref/settings/#password-hashers

This issue is very urgent: You can't create a new database with current master because django 1.10.1 is installed by default!
@normanjaeckel Can you make a pullrequest please?
Workaround:  Install Django 1.9.9 manually (pip install django==1.9.9) and then start and create db (python manage.py start)


Relays don't show up in peer list for block producing nodes

Using cardano-node 1.26.2 and cardano-rt-view 0.3.0 the peer list is empty on block producing nodes.
Peers show up fine on relay nodes.
Same result with current master branch as of a couple days ago.
Note that the block producing nodes are working fine and receiving and sending to the relays.
Other tabs in rt-view work fine and blocks are being produced and propogated.

I'm experiencing the same issue


Automated hyperparameter Tuning for FS.

Description
see #133 which is manual hyperparameter by the user. This issue directly extends from #133 as FeatureEngineerer and Intents both may change the pipeline (# steps, # columns). Consequently, the proposed flow is as follows:

tune intents
tune engineerer
tune the rest.
Rinse and repeat.
This flow is described in the design doc sequence diagram.

Estimate: 3 days

Uses the combination section.


wide-deep AML Hyperdrive notebook link fix

Description
Fix broken links to script in markdown cell
Also, found bugs in azureml/wide_deep.py cause by the recent refactor. So fixed it.
TODO: Should add unit-test for azureml entry script, because it seems like the azureml/wide_deep.py script has been broken since the recent refactor and we didn't have the unit-test to catch that.
Related Issues
#655
Checklist:

[x] My code follows the code style of this project, as detailed in our contribution guidelines.
[ ] I have added tests.
[ ] I have updated the documentation accordingly.

@yueguoguo Can you specify which one? I tested three of them and all works. Are you getting 404 error?


Use dual license

It's just a suggestion, so if you disagree - feel free to close this PR.
I suggesting to use dual license to make this crate more accessible. This is what most crates do, like https://github.com/bevyengine/bevy

Sigh I honestly don't really care but I'm fine with it. If we do it, we should do it well. IMO that means https://reuse.software/ compliance, making sure every file has a license header and all that. While still making sure Github can identify the license as well, which means picking one as the "primary" license to put in LICENSE and all the REUSE licenses in a LICENSES folder.

making sure every file has a license header and all that.

I'm not a layer, but it looks like for MIT / Apache it isn't needed. At least all the crates I've seen was without license header in source files. Do you know a popular crate that does it?

While still making sure Github can identify the license as well, which means picking one as the "primary" license to put in LICENSE and all the REUSE licenses in a LICENSES folder.

GitHub recently received support of dual licensing and I just used the suggested names. This is also what other crates do. Should I move licenses into LICENSE folder instead?

I've done a little research on other crates, and they all seem to use the -MIT and -APACHE or .MIT and .APACHE suffixes for license files in the root folder, and don't put license headers in the source files:
https://github.com/bevyengine/bevy
https://github.com/gfx-rs/wgpu
https://github.com/SergioBenitez/Rocket
https://github.com/serde-rs/serde
https://github.com/seanmonstar/reqwest
https://github.com/rustsec/rustsec/tree/main/cargo-audit
https://github.com/EmbarkStudios/cargo-deny
So we probably doing it right :)

It is not a license requirement, it is simply good practice to add a SPDX identifier, so that if people copy paste random files around, license information is maintained. REUSE has a nice tool for adding them and verifying that all your files have the right info and your license files match the license of your files.
REUSE was recommended to my by some FSF lawyer people, so that's like... the pinnacle of whatever it is FSF is trying to accomplish.
The Github dual license option is indeed very recent, and very unfortunate that they chose a scheme that's incompatible with REUSE. You'd have to copy all the licenses around to comply with both. Ugh.
Well as I said I don't really care, so use your best judgement.

It is not a license requirement, it is simply good practice to add a SPDX identifier, so that if people copy paste random files around, license information is maintained. REUSE has a nice tool for adding them and verifying that all your files have the right info and your license files match the license of your files.

I agree with you, I also follow this practice for my GPL projects because it clearly described in the official GPL page.
I just assumed that since the other popular "Apache or MIT" Rust projects don't do this (see the previous message), that's fine.
I could add SPDX identifiers, but could you help me to automate it? I don't have enough experience with it.

The Github dual license option is indeed very recent, and very unfortunate that they chose a scheme that's incompatible with REUSE. You'd have to copy all the licenses around to comply with both. Ugh.

 GitHub in a nutshell :smile:

Well as I said I don't really care, so use your best judgement.

I would do the same as other projects. Their version looks easier to maintain :smile:
But if you want, we can try to automate the validation of SPDX and put the license files as recommended by REUSE by making symlinks.


Typos in code

I think I found a typo. tokens_a emptiness is checked twice.
https://github.com/maxbachmann/rapidfuzz-cpp/blob/4afd61ee3dcd2895362750308361b2e13ea24275/rapidfuzz/fuzz.impl#L389
Also here:
https://github.com/maxbachmann/rapidfuzz-cpp/blob/4afd61ee3dcd2895362750308361b2e13ea24275/rapidfuzz/fuzz.impl#L488

Yes this is a typo. Note that it still worked correctly, since this condition is only really required for the case len1 == len2 == 0. When only one of the lengths is 0 the subsequent implementation will result in 0 anyways. Fixed the typo in https://github.com/maxbachmann/rapidfuzz-cpp/commit/2ff2a6a19aec02557b66b4de05d3c57e698ae75d.


Feature/support better notebook

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed
[ ] Branch name follows type/descript (e.g. feature/add-llm-agents)
[ ] Ready for code review

ℹ Additional Information

continuing developement in #63 .


pkg.go.dev doesn't show our Go modules doc

https://pkg.go.dev/github.com/tillitis/tillitis-key1-apps/tk1 should show the documentation of the tk1 Go module but doesn't. Instead it says: "Documentation not displayed due to license restrictions". This is probably because it doesn't automatically detect our license, see
https://pkg.go.dev/license-policy

We now use a single LICENSE file on the top level and specific LICENSE files under imported sources, which are kept isolated in separate directories.
README has been updated with a copyright notice and references to these files.

Now visiible.


Remove reference to WithExit in README

This function was removed from the containerd package in da1b5470cd26e9e2b4b1cd0100c876ce8db87471
Signed-off-by: Felix Abecassis<EMAIL_ADDRESS>

But maybe you want to delay this change, since the README is a landing point and people might still using the older API.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2507   +/-   ##
=======================================
  Coverage   44.74%   44.74%
=======================================
  Files          93       93
  Lines        9559     9559
=======================================
  Hits         4277     4277
  Misses       4589     4589
  Partials      693      693

Flag
Coverage Δ

#linux
48.95% <ø> (ø)
:arrow_up:

#windows
41.04% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM
I think the README should reflect the current code. Not sure what replaces that functionality though

LGTM

Adding this here for posterity because I was curious:
It was replaced during the v2 runtime PR; now the client side implementation in ctr tasks (and test code) has a non-exported options helper: https://github.com/containerd/containerd/blob/master/cmd/ctr/commands/tasks/checkpoint.go#L77-L101


Feedback for Video Meetings for SuperOffice - Microsoft Teams

[It needs to be added, that MS Authenticator should be activated for SuperOffice login for MS Teams links to work as videomeeting provider.
MS business account is required for MS Authenticator - family account does not give access to Authenticator/MS Login/soazuread]

Document Details
⚠ Do not edit this section. It is required for docs.superOffice.com ➟ Docs Team processing.

Content Source: video_meetings_msteams

https://github.com/SuperOfficeDocs/superoffice-docs/blob/main/docs/en/admin/preferences/learn/video-meetings/index.md/#L1

@FBSOLT does this apply only to SuperOffice employees?

No - It was a costumer who wanted Teams as Video Meeting provider. We tried to run the process of setting it up, but it did not work with a family Microsoft Account. It needed to be a Microsoft Business / MS 365.
I am not sure, if the costumer needed a two step verification login with MS Authenticator.

I cannot test - I do not have access to change these things. If needed IT could be involved to test.
You could be right that a Personal could work.


fix: add newer version of cachet

https://github.com/CachetHQ/Cachet/commit/6e9d3563ef7f76e40e28d09aa2a81a8c889586ee
comes from cachet issue 3441... apparently it's now broken because that link is a redirect to twitter and not a rss feed anymore..........

or not? apparently it is a cache (i.e. redis) problem, and dashboard can only be seen after ./helper.sh --docker -- exec cachet php artisan cache:clear


Clean Up Scripts

Why: Because I would like the code-base of 2.0 to be relatively clean and well-done before release. This is not necessary before the beta tho.
Big Cleanup necesaary

[x] TimelineEditor.gd
[ ] EventNode.gd

Both have been copied from 1.x and are thus pretty unorganized and likely contain unnecessary code.
Cleanup for all scripts

[x] Event scripts
[x] Resource scripts
[x] Subsystem scripts
[x] Display Node scripts
[x] Editor scene scripts

What I suggest:

rename methods and properties to be as self-explanatory as possible
make use of the gdscript styling guide
remove unnecessary code
add an explanation at the top of each script on what it does
sort/organize methods into

main methods (those that are the main purpose of the script)
helper methods
ui methods

make clear what methods are for internal and external access (for example by consistently starting internal methods with _ or sorting them into specific sections)

Shall we close this? It became stale and should be reevaluated.


Linux OOM

GoReSym cli gets proc killed consuming 64GB of RAM on a fast modern laptop running Linux (Ubuntu 22.04) with this  tidb-server (large) Go binary found in a docker image, built from these sources: https://github.com/pingcap/tidb
To get the pre-built binary using docker CLI:
docker pull pingcap/tidb@sha256:375160102a038d452e6468321ac02cd58d8fdf1edf8890b942fded5488a12c1b
docker save pingcap/tidb@sha256:375160102a038d452e6468321ac02cd58d8fdf1edf8890b942fded5488a12c1b >pingcap.tar

tar -xf pingcap.tar
tar -xf a5d7053833d52f85d8f0be1156633cd75e32cd54a630c452cd24dc4a096d5f4f/layer.tar
ls -al tidb-server
-rwxr-xr-x 1 pombreda pombreda 228929528 Aug  2 14:04 tidb-server

We implement section caching logic to re-merge sections for more accurate scanning, there are definetely issues in some cases. I'd appreciate if the specific bug could be located

We implement section caching logic to re-merge sections for more accurate scanning, there are definetely issues in some cases. I'd appreciate if the specific bug could be located

I am not sure where to look first and where to instruments to catch things. A difficulty is that using the binary mentioned means that in a matter of a few seconds the laptop freezes running out of RAM until it eventually get proc killed by the kernel. There is little to observe in that short time.
Can you tell me where in the code is the caching taking place?

Here is one cache https://github.com/mandiant/GoReSym/blob/d7d9a98b02b95c2822d59994f632b1f76c4f4266/objfile/objfile.go#L227
Each file type is going to also have a DataAfterSection function such as https://github.com/mandiant/GoReSym/blob/d7d9a98b02b95c2822d59994f632b1f76c4f4266/objfile/pe.go#L179 which merged adjacent section data.
These two operations together are where I bet the issues arise

Also experiencing this no matter what flags are specified

Can you send your binary so I can see what blows up? There's no built in limit but with the in memory caches we have some binaries explode memory usage with certain layouts or assembly patterns.

That's a lot of sections. I bet our section caching is what's blowing up https://github.com/mandiant/GoReSym/blob/d7d9a98b02b95c2822d59994f632b1f76c4f4266/objfile/pe.go#L179

This is helpful, this section caching was originally an optimization. I'm learning through this issue and others we might need to remove it entirely and fall back to redoing operations more slowly. Breaking the tool entirely is a horrible trade-off I'm not willing to make for the sake of some speed.

Ah, so the pclntab raw data copy for each candidate is way too much. I bet this can be changed to not build the whole candidate array at once and instead send them via a channel like a generator function instead. That way they're at least not all held in memory at the same time.

@mazza-av can you try out https://github.com/mandiant/GoReSym/commit/54a6712945cd980dcf720c162c1347e14f078155 I've switched to streaming the candidates via go channels (like a generator function). This should significantly reduce the memory pressure.

From my tests this helped a ton with the OP tidb-server. Memory usage is now "only" ~23GB though processing takes 1-2 minutes. For such a huge binary I think this is much more acceptable than 64GB+ and crashing.

Out of curiosity of you profile it while you run it on your binary what dominates the runtime. Is there anything that stands out in particular?


Add interface definition for Bundler::GemHelper.

Motivation
An application I'm type checking with Sorbet depends on a gem that extends Bundler::GemHelper and I currently need to add a shim. Since this is part of the stdlib, this constant should resolve without a shim.
Test plan

Automated test coverage.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_QFK09vZJww0dgk
→ https://go/builds/bui_QFK0hM6Zv84WaX
→ https://go/builds/bui_QFK0xkarZpZIgh

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_QFZ3HaxkJ2uCId
→ https://go/builds/bui_QFZ3KF6ytDVQga
→ https://go/builds/bui_QFZ3bU7kOiYMZl


Delete License Template

This is not needed.

Hi @kristakhare thanks for your PR.  Our legal team requires that we have the LICENSE file as part of the produced artifacts.
There are also dependencies in the pom.xml referring to LICENSE.txt and LICENSE_TEMPLATE.txt as part of the build process that would need to be addressed.


Adding Aggregations

This PR implements the following aggr functions:
The only function missing from #1125 is the any (alias for first) because the Postgres parser reserves it to be used with subqueries. Not sure how (or even if I should dare to) change the parser.

corr
product
bool_and
bool_or
approx_count_distinct (Had to bring hyperloglog from redis back)
regr_avgx
regr_avgy
regr_count
regr_intercept
regr_r2
regr_slope
regr_sxx
regr_sxy
regr_syy
mode
argMin
argMax
variance (alias for var_samp)

This looks really good!
I have three minor comments:

I'm cool with many of those functions only being defined for DOUBLE, but for APPROX_COUNT_DISTINCT  you do define more types. Would that not also be applicable for things like PRODUCT?
What about DECIMAL types, they seem not handled. They are likely auto-cast to DOUBLE, but that would be counter-productive in many cases I think
Could you add some tests grouped aggregates perhaps?

The reason I only went for double only in the product was to diminish the overflow problem, if adding all types this would need to be checked and handled for each multiplication I think.
Good point, I'll extend them to Decimal and add grouped test cases for those that do not have that.
Any suggestions about "any"?


There shows an odd string ${args.targetDisplayName} on 'Manage Access' dialog

Storage Explorer Version: 1.9.0
Build: 20190823.5
Platform/OS: Windows 10/ Linux Ubuntu 18.04
Branch: master
Architecture: ia32/x64
Language: German/Russian/Chinese
Steps to reproduce:

Storage Explorer using the command 'StorageExplorer.exe --lang=zh-CN'.
Expand one ADLS Gen2 account -> 'Blob Containers' node.
Right click one blob container then select 'Manage Access...'.
Check the shown message next to the 'Managing permissions for:'

Expect Experience:
There shows the container name next to the 'Managing permissions for:'
Actual Experience:
There shows the string ${args.targetDisplayName}  next to the 'Managing permissions for:' on ACL dialog.

Looks translation averted to the using outdated argument. Fixed to current 'Управление разрешениями для: %1$s'

This issue has been fixed on the latest master build 20190827.11. So we close it.

Thanks for verification, @v-xuanzh

This issue reproduces on the build master/20190827.12. So we reopen it.


Replace part of images with original from docker hub

@davidalger I think it would be good to replace some images like redis, mariadb, mysql with original images from hub.
These images don't require any customizations. Currently, we need to wait for warden update to support new versions.

Can you not override them with .warden/warden-env.yml?

The ability to override is there are @adam-paterson pointed out. It was by-design that the images used by Warden switched to reference those in the wardenenv account on Docker Hub (originally, I think all 3 of these pointed at the original images) as there may be a need for an entrypoint script for these in the future to support features and so this keeps the door open for that. An example would be having the redis container not persist data on disk, something I've given thought to due to the better DevEx where restarting the container will flush what's in it, resetting it (something I've seen folks trip over, not realizing the redis container by default writes to disk on a volume, that's not re-created even if they delete and re-create the container)
Perhaps it would be useful to update the CI routines to fetch the major versions available, and build/push an image for each minor version available to eliminate the manual steps.


Interactive Plot.polar draft implementation (#544)

I'm not very familiar with either .NET ecosystem or F#, so I have no idea whether the workaround I used here is an acceptable (besides the fact that at least compiler is satisfied with it, since the types don't get narrowed as they would if one uses mere fun (r, phi) -> r * sin(phi)).

-- I comically missed the point considering the original usecase, because apparently getOnlyTwoVariables for "r * sin(phi)" returns them in order of (phi, r), which becomes obvious with plotting a circle (i.e., $r(\phi)=1$, but not for the $\phi + \cos \phi$ curve).

Well, the 3D extensions for polar coordinate system are usually called spherical and cylindrical, but the polarLinear makes sense.

Codecov Report

Merging #547 (1448ac4) into master (23319c5) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 1448ac4 differs from pull request most recent head dd313a8. Consider uploading reports for the commit dd313a8 to get more accurate results

@@      Coverage Diff      @@
##   master   #547   +/-   ##
=============================
=============================

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 23319c5...dd313a8. Read the comment docs.


Typos kubesimplfy's fixed in README.md

Screenshot

Note for Reviewers:-
Sorry I have not created an issue for this 😢

@AvineshTripathi take a look in this PR 😺

/lgtm
thank you for the contribution


Don't require helper scripts for objc extractor [phab:D2178]

Created by salguarnieri at 2018-02-09 01:44:17:
The objc extractor should be allowed to run without the helper scripts that tell us where Xcode is, where SDKs are located, etc.

shahms wrote at 2018-02-09 15:15:43:
Feel free to remove the extra logic in the Skylark rules related to this (or I can) :-)

shahms accepted this revision at 2018-02-09 15:15:43

salguarnieri closed via commit 39955b6cd405803b751db7a67f45f8443ba89e52 at 2018-02-09 17:27:21


Focus cursor on the file after moving it to a target dir

yazi --debug output
n/a

Please describe the problem you're trying to solve
After moving a file to a new location (focusing the cursor on the file, pressing x, then navigating to a different directory, and pressing p), the cursor will still be on the first file in the directory. In some situations, it would be helpful if the focus was placed on the file being moved.
Would you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
After pressing p the focus / cursor is placed on the new file.
Additional context
This is useful when some additional action with the moved file needs to be taken. For example, when xdg-desktop-portal-termfilechooser uses yazi (with --chooser-file argument) to save a file, it is often necessary to first move the file to a different location, after which the file is selected (with Enter), its path is then passed to termfilchooser, and the file is saved at that path.
With the current situation, after moving the file (pressing p), it is necessary to find the new file among all the other files already in the dir, put the cursor on it, and press Enter. Depending on how many files are already in the dir, and how distinctively the new file is named, this might be not effortless.
Perhaps there could be a switch / argument for it. Or it could be automatically enabled when --chooser-file is active. But offhand it seems like this might be convenient even enabled by default.
Somewhat relevant: https://github.com/sxyazi/yazi/discussions/1136 for other suggestions to making yazi more convenient to use with termfilechooser.
Related in case this is advised to be implemented with a plugin: https://github.com/sxyazi/yazi/issues/1713#issuecomment-2389952305 Tips on p action code.
Validations

[X] I have searched the existing issues/discussions
[X] The latest nightly build doesn't already have this feature

Duplicate of https://github.com/sxyazi/yazi/discussions/1706

Ah, I used different keywords when searching through the existing issues. Thanks!

Since the linked conversation appears to be locked, I'll ask here:
Would the following work?
After pressing p, an emtpty file is immediately created and the cursor / focus is placed on it. Then yazi (asynchronously) copies / movies the contents of the file.
There might be potential problems if performing certain actions on the file while it is still being copied (e.g. d should abort the ongoing copy, etc)

Can you explain your proposal in more detail? For example, are you suggesting creating it in Yazi's file system (file list) or actually creating it on the operating system's file system? How would cutting and copying work in each case? What happens if the creation of a new file fails? What if the file is created successfully, but copying or cutting fails? When dealing with multiple files, how would copying and cutting work? Would it create only the first file or all of them?

This is indeed a not-so-straightforward area of operating on partially transferred files. I don't know the internal mechanics of yazi, but from a UX perspective yazi appears to already have this type of user interaction:
When copying a large file or directory to a new location, the new file shows up while the transfer is still in progress. I propose to simply place the focus / cursor on the new file. So, to answer your first question: whatever way yazi currently handles partially-transferred files would be preserved, the only difference is that the focus would be on the new file right away.
In situations when yazi would remove the new file from the listing, due to e.g. unsuccessful transfer, the cursor could be moved to another file, e.g. returned to the position before the transfer start. This however might cause some UX issues if the user was in the middle of an attempt to perform an action on the new file (e.g. trying to delete it by pressing d), as now the action would be applied to the file the focus has jumped to. To overcome this kind of issue, one solution might be to not remove the file from the listing until the user manually moves focus to a different file.
When transferring multiple files, any reasonable option would be fine for the purposes of this issue. For instance, placing it on the first file that has started the transfer.

Added to the Feature requests


CRIMAP-101 Make corrs selector dynamic

Description of change
Dynamically changes the Correspondence Address selection on the contact details page
Link to relevant ticket
CRIMAP-101
Notes for reviewer
Screenshots of changes (if applicable)
Before changes:

After changes:

when no home address:

How to manually test the feature
Go through the form to the client details page, you should see all three options available to select a Correspondence address:

Same as home address
Provider's address
Somewhere else

Open rails console, run the following:
ha = HomeAddress.last
ha.address_line_one = nil
ha.save

Reload the page. You should see only the following options now:

Provider's address
Somewhere else

Submit the form - validations should work as before.

LGTM - all works as expected


fix naming error in readme and documentation

the readme and the first page of the API docs incorrectly use Parser.spaces, which does not exist, instead of Parser.whitespaces. This PR corrects that. Closes #10

Codecov Report

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

@@           Coverage Diff           @@
##           master      #11   +/-   ##
=======================================
  Coverage   95.51%   95.51%
=======================================
  Files          49       49
  Lines        1248     1248
  Branches      131      131
=======================================
  Hits         1192     1192
  Misses         56       56

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 2910978...dc106a2. Read the comment docs.


What is this?

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az iot hub list
Errors:
The command failed with an unexpected error. Here is the traceback:
APIVersion 2020-03-01 is not available
Traceback (most recent call last):
  File "/home/veksler/.local/lib/python3.8/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/home/veksler/.local/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/home/veksler/.local/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/home/veksler/.local/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/home/veksler/.local/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/home/veksler/.local/lib/python3.8/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/veksler/.local/lib/python3.8/site-packages/azure/cli/command_modules/iot/custom.py", line 463, in iot_hub_list
    return client.iot_hub_resource.list_by_subscription()
  File "/usr/lib/python3/dist-packages/azure/mgmt/iothub/_iot_hub_client.py", line 175, in iot_hub_resource
    raise NotImplementedError("APIVersion {} is not available".format(api_version))
NotImplementedError: APIVersion 2020-03-01 is not available

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az iot hub list

Expected Behavior
Environment Summary
Linux-5.8.0-53-generic-x86_64-with-glibc2.29
Python 3.8.5
Installer:

azure-cli 2.23.0

Extensions:
azure-iot 0.10.11
azure-devops 0.17.0

Additional Context

route to service team


Verify Blockscout urls for Klaytn network

looks like Blockscout don't respond for Klaytn anymore, and uses another explorer

@hboon it looks like the is not way for not to get transactions history for Klaytn chain

@oa-s let's leave it then