Not enough to restart docker, I had to disable and enable

File: config/daemon/systemd.md
For me on CentOS 8 and docker 19.3.12 it was not enough to do sudo systemctl restart docker.
I had to disable and enable for it to pick up the changes.

Did you run sudo systemctl daemon-reload first? Doing that is needed to load the updated configuration.

yes, I ran
sudo systemctl daemon-reload
sudo systemctl disable docker
sudo systemctl enable docker

What changes did you make to the unit files? Setting the proxy variables, or something else?

Yes, setting proxy variables.


Changelog - version 6.3 overwritten with 6.4

Commit - 0794958f126d050333bf068d006c94fd27639bb1
Version 6.3 with Validation Severity was changed to 6.4 instead of adding 6.4 as a new entry/line ?

There won't be a 6.3 release. This will be part of 6.4.

but there is 6.3.x-alpha nuget packages.

Yes. The 6.3.x version numbers were used incorrectly rather than incrementing the alpha suffix. I've  bumped the version to 6.4 to work around this.


Update Bootstrap to v4 from v3.3.7

Describe your problem and how to reproduce it:
Currently FCC is teaching Bootstrap v3.3.7, I think that it is better if the Bootstrap curriculum was updated to the current version v4.1.x. Also some of the like 'wells' are not there in v4.1.x.
Currently I have changed the CDN link and updated one challenge. The changes are in my fork - https://github.com/Nirajn2311/freeCodeCamp/tree/feature/update-bootstrap
Add a Link to the page with the problem:
https://learn.freecodecamp.org/front-end-libraries/bootstrap

Should the changes be visually similar, or should they follow the bootstrap migration page suggestions?
E.g. The old btn-default looks more like the new btn-light but the migration page suggests using btn-secondary, which is a gray/darker color.
I'm kind of new to github collaboration.
How would this be handled? Is it a one man job? If not, would we just work off our own fork/branch and do pull requests, challenge by challenge?

Hi @lasjorg, are you still working on this? I also would like to update the bootstrap curriculum v4 and make it more extensive with lessons on cards, alerts, jumbotrons, etc. It's really really short right now.

@dnntrn
No, I'm not really working on it. I didn't get what the plan was for the update. It felt wrong just doing a pure refactoring of the old curriculum to use Bootstrap 4.
Also, looking at the plan for v7 of the curriculum, I'm not sure if the plan is to ditch Bootstrap or if it was just unintentionally left out on the list.
https://www.freecodecamp.org/forum/t/help-us-build-version-7-0-of-the-freecodecamp-curriculum/263546
I know the curriculum update isn't exactly right around the corner but it still wouldn't be very cool to make a totally new Bootstrap 4 curriculum if it is just going to be dropped anyway.
I guess I should figure out what the plan is.

@Nirajn2311 Should we work on updating the current Bootstrap version to V4 + adding more Bootstrap challenges? It doesn't seem that Bootstrap will be in v7 of the curriculum, but it's also not too difficult to do, and I don't mind taking it on! @lasjorg

Closing at stale. I don't think we will be changing these challenge at the moment in favor of the project based curriculum. Thanks and happy coding everyone 🎉


ARROW-889: [Python/C++] Unify PrettyPrints between Python and C++

My main intention was only to have PrettyPrint for ChunkedArray instance but as the Python and C++ output was quite diverging, I have attempted to unify both. During that I came across many untested and unimplemented edge cases that I also fixed. Thus this turned out slightly larger than expected.

Wanted to look at this today but it's getting late; will endeavor to review tomorrow as early in the day as I can

OK, I made the changes that I mentioned -- this can be merged whenever the build is green. I added a pyarrow.NULL object to be more consistent vs. pyarrow.NA. I left NA for backwards compatibility and we can decide what to do with it later


Dialogue

Describe your bug here.
The dialogue for all songs one specific week keeps crashing on me. The only thing is, I used the dialogue editor to make it, and I don't see any JSON errors. I'm stumped, especially because I've been doing the same thing for all the other weeks.
Here's the first song's JSON, for instance.
{
	"dialogue": [
		{
			"expression": "confused",
			"text": "I really think if Aloe decided to flee she wouldn't be somewhere nearby like here.",
			"boxState": "normal",
			"portrait": "bf",
			"speed": 0.05,
			"sound": "dialogue/bf"
		},
		{
			"expression": "worried",
			"text": "I get that, but still...",
			"boxState": "normal",
			"portrait": "nene",
			"speed": 0.05,
			"sound": "dialogue/nene"
		},
		{
			"expression": "worried",
			"text": "I could at least leave a note here for the others to keep an eye out.",
			"boxState": "normal",
			"portrait": "nene",
			"speed": 0.05,
			"sound": "dialogue/nene"
		},
		{
			"expression": "oblivious",
			"text": "...where's a piece of paper? I gotta go find one.",
			"boxState": "normal",
			"portrait": "nene",
			"speed": 0.05,
			"sound": "dialogue/nene"
		},
		{
			"expression": "talk",
			"text": "This girl... I guess I'll help.",
			"boxState": "normal",
			"portrait": "gf",
			"speed": 0.05,
			"sound": "dialogue/gf"
		},
		{
			"expression": "talk",
			"text": "Man, I guess-",
			"boxState": "normal",
			"sound": "dialogue/bf",
			"speed": 0.05,
			"portrait": "bf"
		},
		{
			"expression": "talk",
			"text": "Hiiiiiiiiiiiiiiiiii",
			"boxState": "normal",
			"sound": "dialogue/sky",
			"speed": 0.05,
			"portrait": "sky"
		},
		{
			"expression": "confused",
			"text": "...you again?",
			"boxState": "normal",
			"sound": "dialogue/bf",
			"speed": 0.05,
			"portrait": "bf"
		},
		{
			"expression": "talk",
			"text": "No one's around!
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Nope

Nevermind, decided not to do it


Add the git+ssh rewrite config settings

What does this do / why do we need it?
I use multiple private repos, there are only acessable via SSH and I didn't find any hint in the documentation.
Thanks @barovic and @FQ400 (#1041)

I signed it!

thanks!


"This page could not be found" is seen when a user deletes a cluster on k8s 1.22 rke2 version

on 2.6-head commit id: 2a7857d

Deploy an RKE2 AWS Node driver cluster 1.22rke2 version
when the cluster is up and Active, select the cluster and click on delete.
"This page could not be found" is seen
User has click on Go Back too Home, to get out of this page.

@jordojordo to reproduce this. @nwmac can assist during UK hours if it gets tricky.

@sowmyav27
I could not replicate the issue. I tried on multiple cloud providers to see if it was an AWS issue but did not encounter the problem.
https://user-images.githubusercontent.com/40806497/146067322-2541b016-78e3-4387-a4b9-35e03c5aa9da.mp4

Here's my config for the cluster (Is this the correct config to test with?):

Version:

Happens only with an 1.22RKE2 version node driver cluster

On 2.6-head commit id: 39cecf0
Verified this issue is not seen when deleting a 1.22 RKE2 aws cluster


rename vars in mnsync to make more sense

nRequestedMasternodeAssets -> nCurrentAsset
nRequestedMasternodeAttempt -> nPeerCount

Rebased onto latest develop and renamed var to nTriedPeerCount per discussion, ready for (re-)review.

👍 Updated description


[decomp] Small fixes

Fixes issue where local-vars doesn't show up in the top-level (#498)
Fixes issue with return value casts missing (#489)

Pull Request Test Coverage Report for Build 891112760

33 of 45 (73.33%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 69.766%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

decompiler/IR2/FormExpressionAnalysis.cpp
3
4
75.0%

decompiler/config.cpp
0
2
0.0%

decompiler/ObjectFile/ObjectFileDB_IR2.cpp
0
3
0.0%

decompiler/analysis/final_output.cpp
0
6
0.0%

Totals

Change from base Build 887207602:
0.004%

Covered Lines:
35898

Relevant Lines:
51455

💛  - Coveralls

Codacy got confused and failed. After running it again manually it passed (https://app.codacy.com/gh/water111/jak-project/commit?cid=557437275&fileBranchId=23798640).  It seems like codacy fails more often than it finds something :)


new accessory in homekit: TV_accessory.js

Hello, could you add a new accessory?
https://github.com/KhaosT/HAP-NodeJS/blob/master/accessories/TV_accessory.js

Since it seems @kkapitan isn't supporting this repo anymore I couldn't wait anymore either and made a fork and published it on npm: https://www.npmjs.com/package/homebridge-homesung-tv
Enjoy :)

Since @kkapitan doesn't seem to support this package anymore I made my own fork to change exactly what @skop1986 said: https://github.com/dariusrosendahl/homebridge-homesung-tv it is installable via npm. Enjoy!
I'm not very into creating homebridge plugins yet but I'll take a look to add remote controls when I have the time.

@dariusrosendahl
thank you for your work, I hope you will continue to develop this plugin

Hi. Would you know how to add some basic steering? I.e. for left/right/up/down - where in the field to add some "send key" command?

how do i get input button


[docs] Revise the related projects page

[x] I have followed (at least) the PR section of the contributing guide.

Closes https://github.com/mui/material-ui/issues/31104
Deploy preview: https://deploy-preview-32180--material-ui.netlify.app/discover-more/related-projects/#main-content

[x] Remove not directly related projects: "directly" meaning that they do not immediately extend Material UI's capacities.
[x] Remove every outdated (still in v4) project.

To be honest, I don't think I really get the purpose of this doc. It feels cluttered and unfocused. Are these really "related" projects? How are they related to us exactly? It feels more like "a big list of stuff we recommend."

To put it another way: what value does this doc offer to the reader? What do I, as a user of Material UI, stand to gain by reading this doc?

Is this a list of complementary tools and libraries that I might want to use?
Is this a list of products/projects that use Material UI, that I might want to use?

It seems like this doc is both of these things. If we want to keep it that way, then I think it needs to be restructured somehow.
Either way—do I really need a link to Unsplash, for example?

Yeah, that's definitely a good observation. Though I really think that the biggest outliers here are the "Image and illustration" and "Application frameworks" categories.
I think that all the others fall nicely in the "list of complementary tools and libraries that I might want to use" bucket, extending the library to abilities it doesn't yet perform "natively".
In fact, I also believe the Design kits there are also a bit odd as it sits together with a bunch of third-party tools, not communicating that this is actually something we ourselves develop and sell.

I've removed three projects that, as far as I understood, haven't had updates to work with Material UI v5. Please let me know if I got any of them wrong!

Maybe the design resources should live on a different/their own page? I'm having a hard time formulating a description of this page as a whole because there are still multiple different things going on. The UI kits were created by MUI, right? Whereas all the rest of the links here are for third-party libraries and supplemental resources that were created by members of the community.

Yeah, did that! Removed the Kits from the "related projects" page and put them on a standalone dedicated page, which serves as a quick link to the Store, where more info on them is available.

I forgot that this page even existed 😁
I share Sam's point of view, there are too many unrelated things on this page. For example, I would remove the Calendar and Chart resources if we plan to develop our in house replacements as part of the X product line.
For the other components I would try to find a better fit for the on some of the components' pages, for example I would add the list with forms components in the FormField page (similar to how we have notistick example in the Snackbar page etc).

This page definitely makes a lot more sense now! And nice job on the Design kits page @danilo-leal !

This PR introduces a regression, searching for Figma in the docs does no longer return any results

@danilo-leal We removed the search on the body of the page. It's fixed now.


Ronin rated as insecure by bithound because of minimatch dependency

See https://snyk.io/vuln/npm:minimatch:20160620 please update dependencies.

Sorry, but this module is temporarily unsupported. Consider using meow or args.

Added a note to the top of readme to prevent confusion in future visitors.


fix set thread name fail with string more than 16 byte.

opening this one to PR on unstable branch.  https://github.com/KvrocksLabs/kvrocks/pull/495

Merged, thanks @patpatbear


⚠ Fortnite has degraded performance

In 2672192, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1556 ms

Resolved: Fortnite performance has improved in 69468f2.


The queue can accept several tasks

"The queue can accept several tasks at the same time."
Can I upload multiple files to the queue?
And how to do it?

It is in development, I have already proposed it to the developer to implement it in future updates.
For now you can only upload one file at a time.


AttributeError: Can't pickle local object (only on Mac M1)

This error only occurs on Mac M1, and the same code works on Linux (gitpod.io), so it's not really an issue for me, but just in case to leave it as a placeholder if anyone has the same error.
https://gist.github.com/louis030195/416fea6224b86e8cb993cd9d5c22b894
(or if you suspect that it's unrelated to the environment?)

Hi Louis, I am facing the same problem. Did you manage to solve it ?

Solution is to make all functions first level
On Fri, Oct 14, 2022, 12:50 Hadi Wazni @.***> wrote:

Hi Louis, I am facing the same problem. Did you manage to solve it ?
—
Reply to this email directly, view it on GitHub
https://github.com/rom1504/clip-retrieval/issues/142#issuecomment-1278842803,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR437TIMGTKAQL5IJIV2YTWDE3ITANCNFSM5VUTALPA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

once using the command
!clip-retrieval filter --query "cat" --output_folder "cat/" --indice_folder "indice_folder"
I am getting the following error :
Traceback (most recent call last):
  File "/usr/local/bin/clip-retrieval", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.7/dist-packages/clip_retrieval/cli.py", line 25, in main
    "parquet_to_arrow": parquet_to_arrow,
  File "/usr/local/lib/python3.7/dist-packages/fire/core.py", line 141, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
  File "/usr/local/lib/python3.7/dist-packages/fire/core.py", line 471, in _Fire
    target=component.__name__)
  File "/usr/local/lib/python3.7/dist-packages/fire/core.py", line 681, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/clip_retrieval/clip_filter.py", line 22, in clip_filter
    df = pd.concat(pd.read_parquet(parquet_file) for parquet_file in sorted(data_dir.glob("*.parquet")))
  File "/usr/local/lib/python3.7/dist-packages/pandas/util/_decorators.py", line 311, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/pandas/core/reshape/concat.py", line 304, in concat
    sort=sort,
  File "/usr/local/lib/python3.7/dist-packages/pandas/core/reshape/concat.py", line 351, in __init__
    raise ValueError("No objects to concatenate")
ValueError: No objects to concatenate


Add support for LoongArch.

Add support for new loongarch architecture.
Only 64-bit is supported at the moment.
The commit were tested on Loongson-3A5000 processor ,the kernel information is as follows：
# uname -a
Linux clfs 5.16.0-rc4 #1 SMP PREEMPT Wed Dec 8 09:07:12 UTC 2021 loongarch64 GNU/Linux

The test results is as follows:
# ./autogen.sh && ./configure && make -j`getconf _NPROCESSORS_ONLN` && make check
....
============================================================================
Testsuite summary for gperftools 2.9.1
============================================================================
# TOTAL: 49
# PASS:  49
# SKIP:  0
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
============================================================================
....

Thanks. Applied.


Support mapping document positions after the end of the file

See https://github.com/dotnet/razor/pull/8669#discussion_r1184646673 and https://github.com/dotnet/roslyn/pull/68081 for more context.
Essentially it is per LSP spec to send a document position for { Line: 10, Column: 0 } in a file that has only 9 lines, if there is no newline at the end. We are currently working around this in our OmniSharp extension for formatting (the bug the above PR is fixing) but what that is essentially doing is putting some work in TypeScript, in a specific endpoint, to avoid our C# code crashing. We should just make the C# code more tolerant. Also the LSP spec specifically calls out this behaviour as being allowed.

Did this in https://github.com/dotnet/razor/pull/9149.


Simplify a AND a and a OR a.

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Some features like https://github.com/apache/arrow-datafusion/pull/3859 may find / introduce seemingly nonsensical predicates like a AND a -- in this case the expression simplifier should remove them.
Describe the solution you'd like
Add rewrite rules to rewrite
a AND a --> a
a OR a --> a
Rewrite rules are https://github.com/apache/arrow-datafusion/blob/feff5dc/datafusion/optimizer/src/simplify_expressions.rs#L793

Marking as good first issue as there are many examples and existing tests to follow

I will take a look!

As pointed out by @src255 this is already implemented https://github.com/apache/arrow-datafusion/pull/3915#issuecomment-1287373139 🤔


Anerdw: Watch phenomanmegummiesch.blogspot

Anerdw requests the watch of the watch_keyword phenomanmegummiesch\.blogspot. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

phenomanmegummiesch\.blogspot has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Vickel in Charcoal HQ


Footer no longer sticks to bottom of page in 2.0.0-beta.22

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[x] I have read the console error message carefully (if applicable).

Description
It seems that the latest   2.0.0-beta.22 changed the footer behavior to no longer stick to the bottom of the screen as it did in previous iterations. Going through the changelogs I did not see anything that would indicate this behavior change (ofc I could have missed this)
This issue becomes more apparent with a shorter footer and short pages , however if you maximize some of the larger pages it'll happen too.
Examples in beta.22
My site:
https://62c8a564ff781b0008fdb9ec--lichdocs.netlify.app/test3/

https://docusaurus.io/docs/playground
Docusaurus Site:

Example in beta.21
my site: https://62c78fd7a6bde2000abc5216--lichdocs.netlify.app/test3/test-4/

https://stackblitz.com/edit/github-rcr1hm?file=package.json
Stackblitz Playground of Docusaurus beta.21

Reproducible demo
No response
Steps to reproduce

Go to site with beta.22 installed
Either go to a small page and see that the footer bar does not stick to the bottom or scroll out a few longer pages to see the same behavior. (longer pages seem to stick)

Expected behavior
Footer should not be contained with the page contents and stick to the bottom of screen.
Actual behavior
Footer wraps with page content instead of sticking to bottom of screen.
Your environment

Public source code: its private but this issue is on the main Docsusaurus Site as well
Public site URL: https://docs.lichcraft.games/ (2.0.0-beta.21 version) https://62c8a564ff781b0008fdb9ec--lichdocs.netlify.app/ (2.0.0-beta.22 version)
Docusaurus version used: 2.0.0-beta.22
Environment name and version (e.g. Chrome 89, Node.js 16.4): Node.js 16.5, Netlify
Operating system and version (e.g. Ubuntu 20.04.2 LTS):  Ubuntu 20.04 LTS

Self-service

[x] I'd be willing to fix this bug myself.

Setting Html and Body to height: 100%; seemed to fix this. Did it in my custom css
html , body {
	height: 100%;
  }

This is set in the Layout/styles.module.css , i noticed that there was updates to Layout and its css stylesheet in beta.22 .  Seems that this style isn't being applied correctly.

@LichLord91 Could you try removing the :global on line 1 and 14 locally? If that works you can send a PR.

@Josh-Cena  that worked... could have sworn I tried that 😓 maybe I didn't save the file before rebuilding haha. Okay pull request sent and CLA signed. Glad I could contribute back albeit the tiniest change ever 🤣 .


Add a healthcheck to the Universe package

Looks like the DC/OS package doesn't have a healthcheck. This would be an easy fix - currently there is no health reported.

Well, the only health check that could (maybe) work would be a command health check which looks for the actual Node.js process. It's not exposing/using a http API

A health check endpoint was added by https://github.com/tobilg/marathon-slack/commit/af88c18c77ed0563b90b660750df986c03b310fa. Closing.


Animation (e.g.,mp4)

📝 Description of the feature
Be able to add an animation (mp4) in pyADR (there is no like item_animation, like there is item_image, etc. - so we can not add an animation in pyADR natively at the moment)
💡 Steps for implementing the feature
NA
🔗 Useful links and references
NA

@ekostson  The feature you are requesting actually already exists, it is just not documented.  For example, the following:

import ansys.dynamicreporting.core as adr
adr_service = adr.Service(db_directory="D:/RJF/mytest")
id = adr_service.start(create_db=True)
animation = adr_service.create_item()
animation .item_animation = r"D:\foo.mp4"
adr_service.visualize_report()

Works for me.   Note that the file should be in "web compliant" mpeg4 (H.264 encoding) to be viewable.  ADR will not transcode the animation for you.   The open item here is to document the API.

@ekostson there is an example of this in the second example:
https://dynamicreporting.docs.pyansys.com/version/dev/examples/00-basic/01-connect.html#sphx-glr-examples-00-basic-01-connect-py


I'm having trouble running the plugin on my server

Can you make the plugin work for hybrid versions like mohist?

Can you make the plugin work for hybrid versions like mohist?

The plugin works correctly in Spigot/Paper, this must be their problem.

Can you make the plugin work for hybrid versions like mohist?

Fixed: PerWorldPlugins-1.4.9.zip


Integration of OpenTelemetry Tracing in PR Agent Execution

PR Type:
Enhancement

PR Description:
This PR introduces OpenTelemetry tracing into the PR Agent execution using the Traceloop SDK. The Tracer is initialized in the main PRAgent class and correlation IDs are set for tracking multiple processing of the same PR. Additionally, the Traceloop workflow decorator is added to various tools in the PR Agent for tracing their execution. A new section for tracing configuration is also added in the secrets template file and the Traceloop SDK is added to the project requirements.

PR Main Files Walkthrough:
pr_agent/agent/pr_agent.py: Initialized the Tracer from Traceloop SDK and set correlation ID for tracking multiple processing of the same PR.
pr_agent/tools/pr_code_suggestions.py: Added the Traceloop workflow decorator to the PRCodeSuggestions class.
pr_agent/tools/pr_description.py: Added the Traceloop workflow decorator to the PRDescription class.
pr_agent/tools/pr_questions.py: Added the Traceloop workflow decorator to the PRQuestions class.
pr_agent/tools/pr_reviewer.py: Added the Traceloop workflow decorator to the PRReviewer class.
pr_agent/tools/pr_update_changelog.py: Added the Traceloop workflow decorator to the PRUpdateChangelog class.
pr_agent/settings/.secrets_template.toml: Added a new section for tracing configuration, including the Traceloop API key.
requirements.txt: Added the Traceloop SDK to the project requirements.

Preparing pr description...

Preparing pr description...

Preparing pr description...

Preparing pr description...

Preparing pr description...

Preparing pr description...

Preparing pr description...


Revise README for GitHub Badges Action Implementation

Description

Major overhaul of the README.md to introduce the GitHub badges action.
Added detailed usage instructions, including inputs and outputs.
Removed obsolete content related to TypeScript actions.

Changes walkthrough 📝
Relevant filesDocumentation

README.mdRevamped README for GitHub Badges Action

README.md
Updated the README to reflect the new GitHub badges action.  Removed outdated sections and added new usage instructions.  Included examples and detailed input/output specifications.

+51/-216

Summary by CodeRabbit

New Features

Transitioned the README to focus on a GitHub badges action, enhancing clarity and usability.
Introduced new badges displaying repository status, including code size and last commit.
Added detailed usage instructions for the badges action, including input and output parameters.
Provided a new example workflow for scheduling and manual execution of the action.

Documentation

Streamlined content by removing outdated TypeScript action instructions and replacing them with relevant badge management guidance.

@gstraccini prettier


Implement My Images page

Purpose

Implement My Images page

Goals

Implement My Images page

Approach

Implement My Images page

User stories

N/A

Release note

N/A

Documentation

N/A

Training

N/A

Certification

N/A

Marketing

N/A

Automation tests

Unit tests

N/A

Integration tests

N/A

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Samples

N/A

Related PRs

N/A

Migrations (if applicable)

N/A

Test environment

N/A

Learning

N/A

Codecov Report

Merging #73 into master will decrease coverage by 1.93%.
The diff coverage is 8.44%.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
- Coverage   18.37%   16.44%   -1.94%
==========================================
  Files          47       46       -1
  Lines        1464     1569     +105
  Branches      260      274      +14
==========================================
- Hits          269      258      -11
- Misses       1077     1181     +104
- Partials      118      130      +12

Impacted Files
Coverage Δ

components/portal/src/components/SignIn.js
9.09% <0%> (-5.2%)
:arrow_down:

...ts/portal/src/components/sdk/FederatedIdpSelect.js
3.63% <0%> (-0.21%)
:arrow_down:

components/portal/src/components/Home.js
18.18% <0%> (ø)
:arrow_up:

...nents/portal/src/components/org/OrgCreateDialog.js
3.77% <0%> (-0.31%)
:arrow_down:

...mponents/portal/src/components/appLayout/NavBar.js
3.03% <0%> (ø)
:arrow_up:

components/portal/src/components/user/MyImages.js
2.38% <0%> (-21.15%)
:arrow_down:

components/portal/src/utils/api/httpUtils.js
86.76% <100%> (ø)
:arrow_up:

components/portal/src/utils/api/authUtils.js
28.57% <100%> (-19.05%)
:arrow_down:

components/portal/src/components/App.js
23.8% <11.11%> (-9.53%)
:arrow_down:

.../portal/src/components/common/state/stateHolder.js
71.42% <16.66%> (-18.16%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


163 set number of draws for each resampled control

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 1.36986% with 72 lines in your changes missing coverage. Please review.

Project coverage is 54.43%. Comparing base (c619463) to head (7cc71f6).

Files with missing lines
Patch %
Lines

src/resample.rs
0.00%
32 Missing :warning:

src/utils/logging.rs
3.03%
32 Missing :warning:

src/main.rs
0.00%
6 Missing :warning:

src/cli.rs
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #164      +/-   ##
==========================================
- Coverage   55.74%   54.43%   -1.31%
==========================================
  Files          29       29
  Lines        2935     3000      +65
==========================================
- Hits         1636     1633       -3
- Misses       1299     1367      +68

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Linemasking post-rescale

A good additional operation to have for rescaling might be to mask it to lineart. This is a fairly standard rescaling operation as it reduces damage to grain and the like. Ergo, it would be nice if descale did this internally. Maybe worth looking at once you finish your vsmask rewrite?

You can already do this, though it might not be super intuitive, through the mask param
You just have to pass an inverted mask

Doesn't this override regular credit masking, however?

Yeah
I will make the mask param a tuple with the first element being the error mask, the second being post-processing like linemasking


Work profile issue

The list shows the work profiles apps but doesnt launch any app from the work profile. Only launchs this apps if they are added to the main screen and launched from there. Great launche anyway.

Thank you for the feedback!  I am pretty sure this is a duplicate of https://github.com/jkuester/unlauncher/issues/11
There are some known issues with Work Profiles that need to be addressed.


Reorganize  components in the various subpackages

These are just wrong. Examples:

Wine libraries are used for running Steam Wine/Proton Games, it's not compute related.
Powerd/dbus interface is needed only for Dynamic Boost laptops (anything recent basically), it's not compute related.
nvidia-modprobe is required for creating all the necessary device files, so it should be installed in any case, as it covers corner cases as sandbox files, imex control files, etc.NGX is not compute, it's a graphical library.
ML/CFG are used by multiple components, they are not desktop or compute specific.
etc.

SUSE nvidia driver redesign.pdf
I've used the package breakdown of this document written by Kevin/Andy about 2 years ago. Apparently it needs some improvements.

Wine libraries are used for running Steam Wine/Proton Games, it's not compute related.

Ok. So libnvidia-ngx.so , _nvngx.dll, nvngx.dll and nvidia-ngx-updater to be moved to nvidia-gl-G06 instead.

Powerd/dbus interface is needed only for Dynamic Boost laptops (anything recent basically), it's not compute related.

Ok. Question is where to put this. nvidia-utils-G06, nvidia-video-G06 or nvidia-gl-G0X. I would suggest nvidia-gl-G0X because without that running a laptop with a desktop doesn't make much sense.

nvidia-modprobe is required for creating all the necessary device files, so it should be installed in any case, as it covers corner cases as sandbox files, imex control files, etc.

nvidia-modprobe package is now required by nvidia-compute-utils-G06 which is the minimal userspace package to be installed. See the package dependancy graphics in the design document in my comment above.

NGX is not compute, it's a graphical library.

Covered by 1. above

ML/CFG are used by multiple components, they are not desktop or compute specific.

Then libnvidia-ml should stay in nvidia-compute-G06 , since this is the minimal userspace package to be installed (see above).  But libnvidia-cfg.so should be moved to nvidia-compute-G06.

etc.

To be discussed.

@scaronni-nvidia See the document attached and comments added above. What do you think?

Wine libraries are used for running Steam Wine/Proton Games, it's not compute related.

Ok. So libnvidia-ngx.so , _nvngx.dll, nvngx.dll and nvidia-ngx-updater to be moved to nvidia-gl-G06 instead.

Powerd/dbus interface is needed only for Dynamic Boost laptops (anything recent basically), it's not compute related.

Ok. Question is where to put this. nvidia-utils-G06, nvidia-video-G06 or nvidia-gl-G0X. I would suggest nvidia-gl-G0X because without that running a laptop with a desktop doesn't make much sense.

nvidia-modprobe is required for creating all the necessary device files, so it should be installed in any case, as it covers corner cases as sandbox files, imex control files, etc.

nvidia-modprobe package is now required by nvidia-compute-utils-G06 which is the minimal userspace package to be installed. See the package dependancy graphics in the design document in my comment above.

NGX is not compute, it's a graphical library.

Covered by 1. above

ML/CFG are used by multiple components, they are not desktop or compute specific.

Then libnvidia-ml should stay in nvidia-compute-G06 , since this is the minimal userspace package to be installed (see above). But libnvidia-cfg.so should be moved to nvidia-compute-G06.

I've opened a pull request for this all now. Pull request  https://github.com/openSUSE/nvidia-driver-G06/pull/22

etc.

To be discussed.

This is still open.

I think this can be closed as well.

I think this can be closed as well.

Yes, that's true. Adressed by your pull requests.


Tutorial for Lightoj 1062 - Crossed Ladders

Tutorial for Problem: http://www.lightoj.com/volume_showproblem.php?problem=1062
Creating PR now.

Tutorial for Problem: http://www.lightoj.com/volume_showproblem.php?problem=1062
Creating PR now.

I am about to push to create a PR. it says permission denied. I have tried with my own repos. They are OK.

You need to create pull request. Learn about it from here: https://docs.github.com/en/free-pro-team@latest/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request

Check this PR: https://github.com/lightoj-dev/problem-tutorials/pull/42

Images are commented for further usage at makrdown pages.


SecContacts_L_Phone_FLAGSHIP_USA after compiling not working.

Decompile Orig file
I: Using Apktool 2.0.0 on SecContacts_L_Phone_FLAGSHIP_USA.apk
I: Loading resource table...
I: Decoding Shared Library (touchwiz), pkgId: 2
I: Decoding AndroidManifest.xml with resources...
I: Loading resource table from file: C:\Users\MAXIMATOR\apktool\framework\1.apk
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values / XMLs...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
Build FILE
I: Using Apktool 2.0.0
W: Could not find sources
I: Building resources...
warning: string 'guide_call_addcall' has no default translation.
warning: string 'guide_call_endcall' has no default translation.
warning: string 'guide_call_headset' has no default translation.
warning: string 'guide_call_mute' has no default translation.
warning: string 'guide_call_on_hold' has no default translation.
warning: string 'guide_call_record' has no default translation.
warning: string 'guide_call_screen_end' has no default translation.
warning: string 'guide_call_speaker' has no default translation.
warning: string 'guide_contacts_saved' has no default translation.
warning: string 'guide_dialing' has no default translation.
warning: string 'guide_speed_dial_complete_popup' has no default translation.
warning: string 'guide_videocall_capture' has no default translation.
warning: string 'guide_videocall_dualcamera' has no default translation.
warning: string 'guide_videocall_hide' has no default translation.
warning: string 'guide_videocall_switchcamera' has no default translation.
warning: string 'guide_voicecall_swap' has no default translation.
warning: string 'import_from_4sim2' has no default translation.
I: Building apk file...
Decompile just builded file
I: Using Apktool 2.0.0 on SecContacts_L_Phone_FLAGSHIP_USA.apk
I: Loading resource table...
I: Decoding AndroidManifest.xml with resources...
I: Loading resource table from file: C:\Users\MAXIMATOR\apktool\framework\1.apk
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values / XMLs...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
After rebuild no
I: Decoding Shared Library (touchwiz), pkgId: 2
and when package build W: Could not find sources
and package not working :(
orig files https://dl.dropboxusercontent.com/u/32328783/S6_problem_apk.rar

The warnings regarding sources is perfectly fine, because there is no sources. The output message from Decoding Shared Libraries is not a good indicator of an error. I simply output that text when I encounter a Shared Library reference, so it is a tad bit misleading.
Do you have a logcat adb logcat of the failure occurring?

Here is loggcat
https://dl.dropboxusercontent.com/u/32328783/1.rar

Galaxy Note N910P same proplem
Original apk
C:\Users\MAXIMATOR>adb shell
shell@trltespr:/ $ am start -n com.android.contacts/com.android.dialer.DialtactsActivity
rt -n com.android.contacts/com.android.dialer.DialtactsActivity               <
Starting: Intent { cmp=com.android.contacts/com.android.dialer.DialtactsActivity }
Only recompiled with nothing changes
shell@trltespr:/ $
C:\Users\MAXIMATOR>adb shell
shell@trltespr:/ $ am start -n com.android.contacts/com.android.dialer.DialtactsActivity
rt -n com.android.contacts/com.android.dialer.DialtactsActivity               <
Starting: Intent { cmp=com.android.contacts/com.android.dialer.DialtactsActivity }
Error type 3
Error: Activity class {com.android.contacts/com.android.dialer.DialtactsActivity} does not exist.

I don't see a failure of SecContacts_L_Phone_FLAGSHIP_USA anywhere in that logcat. Since its a system apk, you cannot install it using adb install. You must push it /system.

SecContacts_L_Phone_FLAGSHIP_USA.apk stored in
/system/priv-app/SecContacts_L_Phone_FLAGSHIP_USA
I dont install it. On both phones. Just replaced it in system folder.

Not apktool bug, - Samsung feature


Test Harness Expansion to cover AIP 1.0.0 Protocol 0160-connection-protocol

AS AN Agent, I WANT to connect to another Agent, SO THAT we can do secure transactions together beyond this initial connection
Acceptance Criteria
Feature: Aries agent connection functions RFC 0160
Scenario: establish a connection between two agents
Given we have two agents "Alice" and "Bob"
When "Alice" generates a connection invitation
And "Bob" receives the connection invitation
And "Bob" sends a connection response
And "Alice" accepts the connection response
And "Bob" sends a response ping
And "Alice" receives the response ping
Then "Alice" and "Bob" have a connection
Question: The RFC states "The inviter sends a connection-response message at the end of the share phase" How is this captured here? It seems there might be some confusion here. Not sure if the Protocol Implemented needs to change or just tests?
Here is my rewrite of the scenario above based on how I see the RFC.
Scenario: establish a connection between two agents
Given we have two agents "Alice" and "Bob"
When "Alice" generates a connection invitation
And "Bob" receives the connection invitation
And "Bob" sends a connection request
And "Alice" accepts the connection request by sending a connection response
And "Bob" sends a response ping
And "Alice" receives the response ping
Then "Alice" and "Bob" have a connection
Scenario: Inviter Sends invitations to multiple agents
Question: Are invitations generic or specific to the intended invitee?
Scenario: Inviter Sends invitations to multiple agents only one sends connection request
Scenario: Inviter Sends invitations to an agent and never gets a connection request from them
Does the invitee stay invited perpetually?
Scenario: Inviter gets connection request but invitee is offline for connection response
Scenario: Invitee sends connection response but inviter is offline
Scenario: Invitee sends new invitation after a preceding one
new invitation supersedes the old one. Does the old one no longer work?
Scenario: Establish a connection between two agents who already have a connection
Given we have two agents "Alice" and "Bob"
And "Alice" and "Bob" have an existing connection
When "Alice" generates a connection invitation
Then "Bob" sends an exception of some sort?
Scenario: Establish a connection between two agents who already have a connection initiated from invitee
Given we have two agents "Alice" and "Bob"
And "Alice" and "Bob" have an existing connection
When "Bob" generates a connection invitation
Then "Alice" sends an exception of some sort?
Scenario: send a trust ping between two agents
Given "Alice" and "Bob" have an existing connection
When "Alice" sends a trust ping
Then "Bob" receives the trust ping
Scenario: Send a trust ping between two agents that do not have a connection
Given "Alice" and "Bob" do not have an existing connection
When "Alice" sends a trust ping
Then "Bob" doesn't receive a response?
Can we check the states of the Agents through the backchannel?
Scenario: Inviter send an Invitation to Invitee
Given "Alice" and "Bob" do not have an existing connection
When "Alice" sends generates a connection Invitation
Then "Alice" is in the state of invited
Scenario: Invitee receives an Invitation
Given "Alice" and "Bob" do not have an existing connection
And "Alice" has generated a connection invitation
When "Bob" receives the connection Invitation
Then "Bob" is in the state of invited
References:
0160-connection-protocol from the Aries Interop Profile

The answer to your question is that yes, your rewrite is appropriate.
The use of the "trust ping" after the connection is not a requirement and is problematic. The need for it is because there is not an "ACK" back to the invitee to say that the connection is established. Further, "trust ping" itself was dropped from AIP 1.0.  I'm not exactly sure what we need to do about that.
For now we can leave it at that, but this is something we need to figure out as the use of this spreads out.

Ok, Thanks @swcurran, I'll remove the pings and call it connected after the connection response.
There was another question above asking if the State can be determined at any point through the backchannel or otherwise. I need to know if checking state can be accomplished in the tests. I'm assuming yes, and that there should be a set of test scenarios to check proper state after each interaction between agents.

Since you define the backchannel, you can make it so :-).  The agent has a
way to be queried to get the state, and the agent can send you events when
things just change. So in the case of ACA-Py, every change of status has a
webhook notification to the controller (and the backchannel is a
controller).
On Fri, Feb 7, 2020 at 2:25 PM Sheldon Regular<EMAIL_ADDRESS>wrote:

Ok, Thanks @swcurran https://github.com/swcurran, I'll remove the pings
and call it connected after the connection response.
There was another question above asking if the State can be determined at
any point through the backchannel or otherwise. I need to know if checking
state can be accomplished in the tests. I'm assuming yes, and that there
should be a set of test scenarios to check proper state after each
interaction between agents.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bcgov/aries-agent-test-harness/issues/7?email_source=notifications&email_token=AAHYRQVBAFBM5RJPVOQOXRDRBXNT5A5CNFSM4KQSDEU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELE26GQ#issuecomment-583642906,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHYRQTH33PLUM4VY4UVKKTRBXNT5ANCNFSM4KQSDEUQ
.

--
Stephen Curran
Principal, Cloud Compass Computing, Inc. (C3I)
Technical Governance Board Member - Sovrin Foundation (sovrin.org)
*Schedule a Meeting: *https://calendly.com/swcurran
https://calendly.com/swcurran

OK, perfect. That's what I thought. Thank you.
On Fri, Feb 7, 2020 at 5:46 PM Stephen Curran<EMAIL_ADDRESS>wrote:

Since you define the backchannel, you can make it so :-). The agent has a
way to be queried to get the state, and the agent can send you events when
things just change. So in the case of ACA-Py, every change of status has a
webhook notification to the controller (and the backchannel is a
controller).
On Fri, Feb 7, 2020 at 2:25 PM Sheldon Regular<EMAIL_ADDRESS>wrote:

Ok, Thanks @swcurran https://github.com/swcurran, I'll remove the
pings
and call it connected after the connection response.
There was another question above asking if the State can be determined at
any point through the backchannel or otherwise. I need to know if
checking
state can be accomplished in the tests. I'm assuming yes, and that there
should be a set of test scenarios to check proper state after each
interaction between agents.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/bcgov/aries-agent-test-harness/issues/7?email_source=notifications&email_token=AAHYRQVBAFBM5RJPVOQOXRDRBXNT5A5CNFSM4KQSDEU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELE26GQ#issuecomment-583642906
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAHYRQTH33PLUM4VY4UVKKTRBXNT5ANCNFSM4KQSDEUQ
.

--
Stephen Curran
Principal, Cloud Compass Computing, Inc. (C3I)
Technical Governance Board Member - Sovrin Foundation (sovrin.org)
*Schedule a Meeting: *https://calendly.com/swcurran
https://calendly.com/swcurran
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bcgov/aries-agent-test-harness/issues/7?email_source=notifications&email_token=AKSQXLPPB3OA3AYM5HBHRNDRBXQEPA5CNFSM4KQSDEU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELE4NCI#issuecomment-583648905,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKSQXLN44D7ALQFOZAXNEFDRBXQEPANCNFSM4KQSDEUQ
.


net/http: WriteHeader called with X-Content-Type-Options:nosniff but no Content-Type

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.11beta1 linux/amd64
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/agniva/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/agniva/play/go"
GOPROXY=""
GORACE=""
GOROOT="/home/agniva/sdk/go1.11beta1"
GOTMPDIR=""
GOTOOLDIR="/home/agniva/sdk/go1.11beta1/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build285380950=/tmp/go-build -gno-record-gcc-switches"
VGOMODROOT=""

What did you do?
Used 1.11 beta1 to test my web app.
Context: I use the "github.com/unrolled/secure" middleware which injects security headers in my responses. The issue is about the X-Content-Type-Options header.
What did you expect to see?
I expected my Content-Type header to remain unchanged when I use 1.11 beta1.
What did you see instead?
Found that my Content-Type in the response got changed from text/plain; charset=utf-8 to application/octet-stream.
Along with a warning from the app - http: WriteHeader called with X-Content-Type-Options:nosniff but no Content-Type
Repro -
package main

import (
	"log"
	"net/http"
)

func main() {
	http.HandleFunc("/bar", func(w http.ResponseWriter, r *http.Request) {
		w.Header().Set("X-Content-Type-Options", "nosniff")
		// w.Header().Set("Content-Type", "text/plain; charset=utf-8")
		w.WriteHeader(http.StatusCreated)
		w.Write([]byte("OK"))
	})

	log.Fatal(http.ListenAndServe(":8080", nil))

}

This gives Content-Type: text/plain; charset=utf-8 in 1.10.2 and gives Content-Type: application/octet-stream in 1.11beta1.
If you uncomment the line to explicitly set Content-Type, it goes away.
Wondering if this is an intentional change. Because sensitive http clients might break due to this behavior. And if it indeed is intentional, I will have to change all my apps to add this new line :sob:

/cc @bradfitz

Looks like intentional - https://github.com/golang/go/issues/24795.
@bradfitz - What is the expected action here ? Explicitly set content-type ?

The X-Content-Type-Options: nosniff header means "I set my Content-Type explicitly and I know it's correct, so please don't sniff it and change it, browsers."
But if you're not setting a content-type, it doesn't make sense to say that the value should not be sniffed by browser.
As such, in Go 1.11 we also don't sniff it, and issue a warning if you issued that header but forgot to set a Content-Type.
So, yes, this is intentional. Although I imagine this does break people who previously trusted Go's content-sniffing but didn't trust the more-aggressive browsers' content sniffing. In that regard this is a regression, but one I'm happy about if this is sufficiently documented in the release notes. https://go-review.googlesource.com/c/go/+/89275 is marked as RELNOTE=yes and I see https://tip.golang.org/doc/go1.11#net/http has a TODO to flesh out the docs on this.
So I think this can be closed, but cc @FiloSottile @mikesamuel if there are other opinions.


System.DllNotFoundException: Unable to load DLL 'libwkhtmltox' when deploying on Windows 2016

Hi,
I saw this use case was already mentioned in other issues but I found a strange workaround.
So first some information about my environment : I'm using DinkToPdf along with some fixes provided by #18 . Everything work natively with VS2017 in a .net core 2.0 (AspNet core), I can generate any PDF I want without any problem, any errors (and without loading explicitly any DLL).
I deployed it on a Windows 2016 Server and I got the infamous System.DllNotFoundException. I tried a lot of things but finally I added in the root of my project the libwkhtmltox.dll coming from the <IP_ADDRESS> windows 64bits package. And it worked perfectly fine. Swapping it with 0.12.4 does not work at all (DllNotFoundException). I tried to use the latest alpha and even if I did not have a System.DllNotFoundException, I didn't get any PDF output (did not spend any time debugging it).
If I read wkhtmltox changelog they switched from VS2013 to VS2015 with 0.12.4.
Can somebody with access to Windows server 2016 test and confirm ?

@seblucas Where is the <IP_ADDRESS> vrsion of the libwkhtmltox.dll ?..Can you share it?

Version 0.12.4 on Windows 2016 server (Azure VM) works for us.
See https://github.com/rdvojmoc/DinkToPdf/issues/5
Solution is to download and install the tool on your server
https://wkhtmltopdf.org/downloads.html

I am having the same issue any work around ?
@seblucas: Installed <IP_ADDRESS> from download site to server not working ...


guess_language-spirit replacement or alternate solution?

According to guess_language-spirit's PyPI page it hasn't been updated since March 11, 2017. Also, the home link returns a 404 https://bitbucket.org/spirit/guess_language.
Curious if someone else is maintaining the project somewhere else? maybe there is a better solution? I noticed both the Azure and GPC support detecting languages. Maybe this is a better route for solving this issue?

https://docs.microsoft.com/en-us/rest/api/cognitiveservices/translatortext/translator/detect
https://cloud.google.com/translate/docs/basic/detecting-language

Yes, if you look at the two leading translation services (Microsoft's and Google's) both have endpoints for language detection, so if I had to code this today that's what I would use.

@miguelgrinberg thanks for the response! Are you accepting PRs for this project? I have found this tutorial/series to be invaluable and would love to give back. What are your plans for "keeping up with the times"? It seems as if it's gotten one major overhaul already. Sorry if you have already answered this somewhere else.

You are welcome to submit a PR, but because the structure of this repository is directly related to the tutorial I cannot merge them, but just take them as suggestions that I eventually incorporate into the tutorial and the code.
I'm constantly making changes to the tutorial and the code, by the way, but this is not an area where I feel I need to rush and make a change. The guess-language package does work, and it is very convenient that it is a local package that does not require sending requests to an external service. Replacing it with a service will make posting articles much slower, and avoiding that added delay will require significant changes to how the feature is designed.

@miguelgrinberg before anything, thank you so much for this tutorial.
I'm having trouble making guess-language work for me. I started this tutorial with Python 2.7, then switched to 3.9 (on Windows). Once I switched, Python started having trouble with the unicode parentheses (u'' and ur'') in the guess_language init file. I'm also getting "NameError: name 'unicode' is not defined."

@m-nt-s-no you need to include the full stack trace of your error. There is no specific unicode in Python 3, all strings are unicode. You may be installing an old and abandoned version of the guess_language package, you did notice that the package is called guess-language-spirit, right?

@m-nt-s-no okay, then remove the one that is not used.

Replaced with langdetect package in the tutorial.


APT whitelist request for gnuplot-nox

Duplicate of #663.

Yeah, setuid is needed for the svga terminal, which is useless in travis anyhow, also we dropped the suid bit in Gentoo a long time ago and gnuplot works just fine.


Feature/4 add analytics

Fixes #4 for now. Allows to see the basic usage pattern.

Current coverage is 0.00% (diff: 100%)

Merging #87 into develop will not change coverage

@@           develop   #87   diff @@
====================================
  Files            1     1
  Lines           27    27
  Methods          0     0
  Messages         0     0
  Branches         0     0
====================================
  Hits             0     0
  Misses          27    27
  Partials         0     0

Powered by Codecov. Last update ad8d1ee...67b44b8


Multiple users working in the same env

I have several questions.

I sometimes get merge conflicts with src/graphql/schema.json or amplify/backend/backend-config.json, does it matter that instead of fixing the conflicts, I delete the file and do amplify push again to autogenerate a new ones instead. The conflicts I get usually relate to aws_oidc and aws_auth for the schema.json, which don't really make any sense to me.

Let's say Bob and Alice are working on their own feature branch.
a) Bob makes some changes to Lambda-A and amplify push call it UPDATE-A,
b) Bob decides to make more changes to Lambda-A, but DOES NOT amplify push UPDATE-B and merges the feature branch to master
c) Alice comes in to the master branch and runs amplify status and notice it can be updated.

2a) In this scenario if Alice amplify pull, I assume she would get UPDATE-A because that was the last thing amplify pushed to the cloud. Is that correct?
2b) Assuming Alice did amplify pull, that will override Bob's UPDATE-B changes meaning amplify push after the pull would be useless right?
2c) Assuming Alice didn't do 2a) and 2b), since the master git branch technically has the code for UPDATE-B, isn't the safest option for Alice to amplify push instead to always get the latest changes. I'd like to think that the git branch is always the source of truth.
Thank you.

@chantlong Generally speaking CLI team workflow works in concert with source control, but Amplify CLI only working with the cloud pieces.

in case of schema.json that is fine to "take theirs" to accept the changes or running codegen manually will recreate that file. In case of backend-config.json, that usually changes when you add/remove/update resources. Depending on the type of change that can be also be a "take theirs" type of merge. If you've FEATURE-1 and FEATURE-2 and both changing resources then you can end up with a need to merge, but it is not different in any way from other merges.

When Amplify pulls, then it downloads the cloud artifacts, and not source code as I mentioned before. So if you want updated resources in that env that is associated with master, then you've to do a push to make it up-to-date.

Review the team workflow docs and provide some suggestions on which sections could we enhance to make customers more confident when executing these Amplify commands.


Outdated Spotify Fingerprint

I believe the Spotify Fingerprint changed recently and it should be: 9F 3F 7B C6 26 4C 97 06 A2 D4 D7 B2 35 45 D9 AA 8D BD CD 4D
(would make a pull request but I'm not familiar enough with Github)

Fixed now!


borderColor function receives context with dataset, but dataset values behave weirdly.

I am using streaming plugin but not sure if it's related.
Every time the plugin redraws the graph the borderColor is calling a function to make a gradient.
I am using it to draw a gradient based on the data, however seems like something weird is happening with the data object:
data: {
  labels: graphDataTime,
  datasets: [{
    label: 'Data',
    borderColor: function (context) {
      console.log('new', context.dataset.data) // gives me the updated object each time
      console.log('new[0]', context.dataset.data[0]) // does not change from the first render!

I cannot understand why does context.dataset.data[0] remain the same.

I am trying to redraw the gradient on each data change:
if (!_.isEqual(oldData[0], context.dataset.data[0])) {
  drwaGeadient(context.dataset.data)
}

oldData = _.cloneDeep(context.dataset.data)

Yet it is always false; they are equal.
Seems like context.dataset.data[0] always points to some original object.

If you really need some help, please add a link to interactive example demonstrating the issue.

It's hard to say without a live sample to debug + knowing how the data updates. @ilyador are you able to provide something?

Closing due to lack of test case


Fix: save images as PNG with full quality

Small fix to ensure that images are saved in full quality and as PNG, such that the bytes will not change by any form of compression.

LGTM!


What must be dimensions of frame

What must be the size of each frame of the video?
Or what must be the dimensions of a batch input?

Hello,
The default sample size is 112 x 112 (check opts.py flag --sample_size).
The size of the batch input depends on the ram capacity, if using gpu, depends on your gpu ram capacity.
The temporal window is fixed on 16 frames, so every batch will have this format: batch_size x 16 (frames) x 3 channels (RGB) x 112 x 112.


search results are still pointing to versioned urls

e.g. http://kubernetes.github.io/v1.1/docs/getting-started-guides/docker-multinode/

Will request a re-index before launch.

Fixed.


Suggestion: keybinding to move between separators of the quick search view could move to "more..."

Testing #208367
This is just a suggestion as I was trying out the new keybinding, feel free to close it if it's out of scope or not a good suggestion :p
As I was playing around with the quick search list and it returned a large results list, I thought it'd ben nice if when navigating between separators using the new keybindings, I could also navigate to to "More..." directly:

I don't particularly wanna be in the business of giving quickpicks control of what cmd+up/down does.. because that will introduce inconsistencies... so in order to do this, the quick search would need to separate the More's perhaps with a blank inline separator.
@andreamah would make this decision.

Hmm, I wonder if there's a way to set another keybinding for this? One that's not cmd+up/down. Not sure which keybinding would be best in this case.

You do have the ability to set a context key for a specific picker FWIW... so you could have a unique keybinding. Not sure if it's worth it though.

I'm not sure whether this keybinding would be used a lot. If someone wants to see the results in the search view, they can click the action button beside one of the entries. I'll close this for now, unless there is another use case for this. Thanks!


WIP: Update to new version, add ability to add labels

Executor image has to be changed to jggoebel/terraform-executor:v1.0.10


Flatten optionality of regex captures

We currently nest optionals for regex literal captures, we should flatten them out to match the regex literal proposal.

@milseman do you recall what needs to be done here? Are we also proposing to flatten the dynamic captures of Regex.init (i.e only the DSL should have nested optional captures)?


setup-hooks/strip: silence 'File format not recognized' errors again

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Please fix the commit message to at least point to e4d17dc where this change was lost.

added

Please fix the commit message to at least point to e4d17dc where this change was lost.

added


Multi-tenancy

This change will allow for the hosting of multiple databases by a single Pathling server.
Each database will expose its own FHIR endpoint, for example:
myserver.com/database1/fhir
myserver.com/database2/fhir

@kaicode I'm working on a solution for this in v2, shouldn't be too far away.

I think we will put all our synthetic data in one database so we can demonstrate selection of records using the criteria. We have a workaround for loading from multiple source datasets. Thank you.


How to send ipv6 arp packets using gopacket, any examples?

https://github.com/google/gopacket/blob/32ee38206866f44a74a6033ec26aeeb474506804/examples/arpscan/arpscan.go#L148
arp := layers.ARP{Protocol: layers.EthernetTypeIPv4}
change to
arp := layers.ARP{Protocol: layers.EthernetTypeIPv6}


documentChanges isn't a good name for file management

From: https://github.com/Microsoft/vscode/issues/57484

Agree. The general problem is that any rename is breaking. So we need to decide carefully on this.


HOT RELOAD: Support reloading deferred libraries

We need to handle deferred libraries specially during reload. Currently all deferred library tests are marked skip in the status files.

I submitted a change to explicitly disallow deferred libraries when reloading, so at least the system will handle it more gracefully until we fix this.
77b99396e8c9251d816ed45eab8479dee812a397
There was one odd test that I still saw passing (spuriously) despite the change:
deferred_super_dependency_test: Pass, Crash # Requires deferred libraries

Probably could bear some investigation.

Closing as outdated.


Prevent deselection of selected text when click on the menu item.

Version
1.9.1
Steps to reproduce
So, I have been using the context-menu from many months now, I am using it at different places getting success with each and everything, Now, I have one common context-menu for entire website consists of undo,redo, cut,copy, paste.
This is my menu

 { text: UIPagesResources.CxBuilder.BUTTON_UNDO, value: "undo", category: "BrowserMenu", callingMethod: "BrowserMenu()", icon: "fa fa-fw fa fa-undo" },
{ text: UIPagesResources.CxBuilder.BUTTON_REDO, value: "redo", category: "BrowserMenu", callingMethod: "BrowserMenu()", icon: "fa fa-fw fa fa-redo-alt" },
{ text: "Divider", isDivider: true },
{ text: UIPagesResources.CxBuilder.BUTTON_CUT, value: "cut", category: "BrowserMenu", callingMethod: "BrowserMenu()", icon: "fa fa-fw fa fa-cut" },
{ text: UIPagesResources.CxBuilder.BUTTON_COPY, value: "copy", category: "BrowserMenu", callingMethod: "BrowserMenu()", icon: "fa fa-fw fa fa-copy" },
{ text: UIPagesResources.CxBuilder.BUTTON_PASTE, value: "paste", category: "BrowserMenu", callingMethod: "BrowserMenu()", icon: "fa fa-fw fa fa-paste" },

This is trigger when clicked on the menu item
switch (data.value) {
                    case "undo":
                        document.execCommand("undo");
                        break;
                    case "redo":
                        document.execCommand("redo");
                        break;
                    case "cut":
                        document.execCommand("cut");
                        break;
                    case "copy":
                        document.execCommand("copy");
                        break;
                    case "paste":
                        document.execCommand("paste");
                        break;
                    default:
                        break;
                }

The real problem is now, when user selects any text in the browser lets say 'abc' and then right click the menu opens and then he clicks on the copy. the copy command executes but selection of 'abc' disappears. so, in my clipboard I have empty string.  I am not sure this is duplicate issue but I can not find any hint from the previous issues.
Expected Behavior
It should copy the selected text to the clipboard.
Actual Behavior
It deselects the selected text and nothing happens.

@vkbansal Can you please check this?

I can't sorry. I'm on Vacation.
On Thu 6 Dec, 2018, 6:26 PM ankit<EMAIL_ADDRESS>wrote:

@vkbansal https://github.com/vkbansal Can you please check this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vkbansal/react-contextmenu/issues/249#issuecomment-444861904,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFp9b1FoNNjOjCXAXh0eh8zKR0wHFbAyks5u2RP2gaJpZM4ZAWCa
.

@vkbansal can you guide me who can help me to find out where is the problem?

@dholakiyaankit could you store the selected text in your components state? In the click handler, have it first save the text to state and then have the menu open:
Then when the user clicks the menu option copy, you can pass text stored in state -
    /* success / close menu / clear text in local state */
  }, function() {
    /* handle failure */
  });```


setting AccordionItem height ignores margin

I think due to using scrollHeight
demo of differences here

think this was actually fixed by #75


[backport]   Bluetooth: ASCS: Validate num_ases in CP requests

Missing validation  in CP requests fixed in #74976 must be backported.

@ceolin Can this be closed? https://github.com/zephyrproject-rtos/zephyr/pull/77958 is merged


dependencies: update several modules related to drafts work

Coverage remained the same at 0.0% when pulling 34cd55cf53be3b80e23939bf0f88927835a4df26 on ppanero:deps into eb37a285b3635664c95d48a776cb50d5263c6b52 on inveniosoftware:master.

Coverage remained the same at 0.0% when pulling 34cd55cf53be3b80e23939bf0f88927835a4df26 on ppanero:deps into eb37a285b3635664c95d48a776cb50d5263c6b52 on inveniosoftware:master.


Change providers strategy

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
✔

Suggestion.
Add support \Closure for providers.
This is can use for not yii-* packages with simple code in providers.
For example, config:
return [
    'yiisoft/csrf' => static function() {
    CsrfToken::initialize();
    return [];
  },
];

Need to try that on yii-demo and packages. @yiiliveext will do it.
Need to decide if that's a good way to go overall.

@yiiliveext seems https://github.com/yiisoft/widget/issues/12 will be broken after the change. How would solve it?

@yiiliveext seems yiisoft/widget#12 will be broken after the change. How would solve it?

Maybe it is BootstrapMiddleware in the main pipeline?

Yeah, that could be solved that way.

Here's a draft of bootstrapping: https://github.com/yiisoft/app/pull/76

Won't be merged in this variant.


chore: improve cypress happo screenshots (part 6)

FX-2850
Description
Cypress tests with happo screenshots are being improved. I separated commits to ease reviewing process
In this PR, I focused on 9 components:

Slider (cleaned up and improved with names)
Stepper (spec file deleted because of redundant tests with storybook)
Table (cleaned up and improved with names)
TableCell (improved with names)
Tabs (cleaned up and improved with names)
Tag (spec file deleted because of redundant tests with storybook)
Timeline (spec file deleted because of redundant tests with storybook)
Tooltip (improved with names)
Typography (spec file deleted because of redundant tests with storybook)

How to test

happo review and code review would be enough

Screenshots
happo reports are our screenshots
Development checks

Add changeset according to guidelines (if needed)
Read CONTRIBUTING.md and Component API principles
Annotate all props in component with documentation
Create examples for component
[ ] Ensure that deployed demo has expected results and good examples
Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[ ] Self reviewed
Covered with tests

Breaking change

codemod is created and showcased in the changeset
test alpha package of Picasso in StaffPortal

PR commands

List of available commands:

@toptal-bot run all - Run whole pipeline
@toptal-bot run build - Check build
@toptal-bot run deploy:documentation - Deploy documentation
@toptal-bot run package:alpha-release - Release alpha version

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

No need to keep separate table variant tests for cypress side. compact and striped and narrow and clear would be enough to cover table variants.


Issue 11833

closes #11833
Final Commits

@jreback  @jorisvandenbossche  it has passed the test time !!
Had to link with travis to run them locally
Please have a look at the commit

needs a note in whatsnew, with an example as this is techincally an API change.

@jreback   @jorisvandenbossche  do i have to update the what's new section in the documentation?
I have no experience with Pandas workflow, please guide me here?

http://pandas.pydata.org/pandas-docs/stable/contributing.html, specifically http://pandas.pydata.org/pandas-docs/stable/contributing.html#documenting-your-code for the whatsnew.

@TomAugspurger  @jreback
I did the changes in the documentation
With the recent commit

some comments
remove the html-styling file from the commits.
squash into a single commit.

@jreback  is there anything else that needs attention , for the issue #11833 , I did the docs changes as directed by you

you need to squash: http://pandas.pydata.org/pandas-docs/stable/contributing.html#combining-commits


Update Azure samples links.

Existing links to GitHub result in 404 not found.

#sign-off

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @


Libjpeg-turbo

Apparently there is now 12-bit support in libjpeg-turbo as of v3.0.x.
Perhaps it can be investigated as a replacement for / in addition to pylibjpeg-libjpeg, with a more permissive license (BSD-style).
According to the libjpeg-turbo README:

The libjpeg API implementation in libjpeg-turbo is both API/ABI-compatible and mathematically compatible with libjpeg v6b. It can also optionally be configured to be API/ABI-compatible with libjpeg v7 and v8 (see below.)

Which sounds to me (non-expert) like it could be almost a drop-in replacement in the existing libjpeg handler code.

Good find! Yes, if it has the same functionality and we are able to integrate it, I see no reason to retain the old version, as the other plugins already are under MIT or compatible licenses. I (also as a non-expert) have recently been told in another context that libjpeg can be completely replaced by libjpeg-turbo featurewise, and that both implement the same standard. And libjpeg-turbo should be faster...
So this sounds very promising to me. I may have a closer look at this, though that may take quite some time, as I'm really no expert here (wish @scaramallion would re-surface...). Or you may have a go, as you already have been diving into this.

(wish @scaramallion would re-surface...).

Same.

Or you may have a go, as you already have been diving into this.

I've looked some more out of curiosity.  Seems to me the key is in building the wheels, after that the drop-in should be okay.
But the source layout and make files etc look quite different, so that might not be completely straightforward, but I'll leep looking.
Given that it look like there will be a fair number of changes, I'll probably create a pylibjpeg-libjpegturbo repo (long name but matches the current style) and get that started at least.  I'll start locally and push up to create on github if it seems to be going okay.  That leaves the existing -libjpeg repo in place for anyone who needs it and we can just use the turbo from pylibjpeg.

Seems to me the key is in building the wheels

I think you can reuse the existing workflow to build the wheels without problems. I think the key is rewiting the _libjpeg.pyx and utils.py (though that won't probably change much) files - basically the interface to the library.

Given that it look like there will be a fair number of changes, I'll probably create a pylibjpeg-libjpegturbo repo

Yes, I have been thinking along the same lines after thinking about it a bit more, so I completely agree. Go ahead!

basically the interface to the library.

Hmmm ... I started digging into the Cython stuff and all that, then I remembered I had seen PyTurboJPEG - wraps the lib with ctypes calls, but it doesn't build libjpeg-turbo (up to the user to install themselves).  Probably enough and simplifies things a lot for us.  In fact, can start without even trying to build the wheels ourselves.
And... I was thinking that because pylibjpeg-libjpeg is GPLv3, then we should be careful not to possibly be interpreted as a derived work.
As I look a little more, PyTurboJPEG is really small and simple, we could even just copy the bits we need (it is MIT).  I'll start down this line and see how it goes.

Okay, I've created a structure and opened a public repo: https://github.com/pydicom/pylibjpeg-turbo  (I went with the shorter name, too much typing).  Still very much WIP, of course.

As I look a little more, PyTurboJPEG is really small and simple, we could even just copy the bits we need (it is MIT). I'll start down this line and see how it goes.

Yes, this is possible, though in general I prefer the sub-module approach - much less maintenance needed for moving to a new version. But if we need only a small part, it certainly makes sense.

I agree, I think I didn't write carefully, what I meant was to start with PyTurboJPEG as a dependency, but if we need something a little different we could just add it ourselves, (modeling on their code, and contribute it back to PyTurboJPEG if it might be of general interest).

Ah ok, I didn't indeed understand you correctly. This make sense, of course!


⚠ BCR Bot has degraded performance

In 55d0d64, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 1039 ms

Resolved: BCR Bot performance has improved in 10022da.


Add a missing space in WindowsGetStringRawBuffer page

This change adds a missing space between "an" and "HSTRING" in WindowsGetStringRawBuffer page.

@masaru-iritani : Thanks for your contribution! The author(s) have been notified to review your proposed change.

thanks!


fix: don't crash on drop highlighted text onto canvas

Closes #3971
Adds a missing check for if a file is present on drop. This is required manually due a typescript as any cast further inside the code block. Also manually adds a couple of type annotations to make clear that file may not be provided.
The behaviour is now to take no action, if a file is not found (rather than showing an error modal).

Thanks for the fast feedback - updated and squashed with your feedback (as it's such a small change)

Thanks for the fix! ❤

Thanks for the fix! heart

No problem, and thanks for the fast turnaround and maintaining such a great tool for everyone!


Adding departments and/or buildings to Jamf pro cloud

Im having difficulty finding the right options to do this, or maybe its not a supported option, but essentially, I would like to import a CSV list of departments and buildings into JSS en masse

Hello @grainofric3 , unfortunately MUT will not create these on-demand. MUT actually doesn't create users either--that's a Jamf Pro feature that MUT happens to be able to leverage--if a device is assigned to a User and that User does not exist yet, Jamf Pro creates it.
I know there are plenty of bash scripts and things like that out there, which can create Buildings/Departments en masse, but MUT is not able to do it.
I recommend you reach out to Jamf Support to see if they have any scripts for you, and if they're not able to find one, I may be able to dig one up somewhere.

Hello @grainofric3 , unfortunately MUT will not create these on-demand. MUT actually doesn't create users either--that's a Jamf Pro feature that MUT happens to be able to leverage--if a device is assigned to a User and that User does not exist yet, Jamf Pro creates it.
I know there are plenty of bash scripts and things like that out there, which can create Buildings/Departments en masse, but MUT is not able to do it.
I recommend you reach out to Jamf Support to see if they have any scripts for you, and if they're not able to find one, I may be able to dig one up somewhere.

understood, thanks Mike, if you happen to come across any scripts lmk, but ill keep digging to see.

understood, thanks Mike, if you happen to come across any scripts lmk, but ill keep digging to see.


issue

 Environment data

Language Server version: 2023.9.20
OS and version: darwin x64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: true
python.analysis.typeCheckingMode: off

 Code Snippet

XXX

 Repro Steps

XXX

 Expected behavior
XXX
 Actual behavior
XXX
 Logs

XXX

Thanks for the report. Unfortunately, this is not enough information for us to investigate this issue.
If this continues to be a problem, would you please reopen the issue and add the information described in the troubleshooting guide?


Support version 2.1.0 of Magit

This should be merged after I have released version 2.1.0 which will happen on the 1th of July, but not earlier. Do you have any questions regarding this?

Thanks for this! No guarantee I'll keep the suggested bindings... :-)


DnsMontioring not available in Master Slave Mode

One of our use case requires that if there is a failover, we reconnect automatically to another node. This is available in single server config as DnsMonitoring but not in Master Slave mode. Is there a specific reason for this? Can there be a support added? @mrniko

Yes, it's possible to implement

If that is the case, then dnsMonitoring can be added for both the master and the slave right?

Yes

Can this be implemented by you? I am trying to go through the code base as well.

@mrniko Any update on this issue?

I think it will be implemented in June's release

Has this been released yet?

Implemented


ENH: Add special.log_expit and use it in stats.logistic

Add the function special.log_expit that computes the logarithm of the logistic sigmoid function.  The function improves the numerical behavior of the logcdf and logsf methods of stats.logistic, and could be used in downstream projects that implement logistic regression, such as the Logit class in statsmodels.

Have we established a minimum standard version of C++ that is required to build SciPy?  For example, it would be nice to know that we require at least C++11, to be sure that #include <cmath> puts log1p in the std namespace.
If we haven't already, we should establish minimum standard versions for the C and C++ compilers, and add this information to the PREREQUISITES section of INSTALL.rst.txt.

The last commit, in which sf_error.cc is created as a C++ file that simply includes sf_error.c from within an extern "C" {} block, is a bit of a hack.  Without it, ufuncs_cxx_src = ['_ufuncs_cxx.cxx', 'sf_error.c', '_faddeeva.cxx', 'Faddeeva.cc', '_wright.cxx', 'wright.cc'] is a mix of C and C++ files.  With the _pre_build_hook for the extension module set to set_cxx_flags_hook set_cxx_flags_hook to ensure that -std=c++14 is used for the C++ files (to ensure that #include <cmath> puts log1p in the std namespace), the compiler on MacOS complains when that option is also passed to gcc (the C compiler) when sf_error.c is compiled.
The Main Lint failure is because of the indentation of the triple-quoted string in _add_newdocs.py:

add_newdoc('log_expit',
    """
    log_expit(x)

This is the same style used in 257 other calls of add_newdoc(), so I think we can ignore the linter in this case.

The ci/circleci: build_docs and ci/circleci: run_benchmarks checks are displayed as Pending, but the Details link of each one shows that they completed successfully.  Go figure.

I added this to the 1.8 milestone, but only because @person142 has already approved it. If it turns out that the PR requires more work, we can bump it to 1.9.

Could you add a release note @WarrenWeckesser?

Could you add a release note

Done.


MIME Type and UTI?

We should clarify the MIME Type and UTI that are appropriate for OTIO files.
Currently our documentation says:
TODO: Should we specify a MIME Type `application/vnd.pixar.opentimelineio+json` and UTI `com.pixar.opentimelineio` for OTIO, or just use MIME: `application/json` and UTI: `public.json`?

What is the right MIME / UTI to use? Is this something we need to register with a standards body?

It makes sense to register a mime type, although a generic binding to json is confusing, in the same way the binding to xml or txt might be. I'd recommend adopting .otio instead, in order that applications can filter directory views, and desktop environments like Apple's Finder can display an OTIO icon. As to the UTI, com.pixar.opentimelineio makes sense as these are usually bound to the originator. The alternative is com.aswf.opentimelineio.
For comparison, here is OpenEXR's MIME entry: https://www.digipres.org/formats/sources/pronom/formats/#fmt/1001 and  UTI: https://escapetech.eu/manuals/qdrop/uti.html
The process for registration is documented here: http://www.iana.org/assignments/media-types/media-types.xhtml

It makes sense to register a mime type, although a generic binding to json is confusing, in the same way the binding to xml or txt might be. I'd recommend adopting .otio instead, in order that applications can filter directory views, and desktop environments like Apple's Finder can display an OTIO icon. As to the UTI, com.pixar.opentimelineio makes sense as these are usually bound to the originator. The alternative is com.aswf.opentimelineio.
For comparison, here is OpenEXR's MIME entry: https://www.digipres.org/formats/sources/pronom/formats/#fmt/1001 and  UTI: https://escapetech.eu/manuals/qdrop/uti.html
The process for registration is documented here: http://www.iana.org/assignments/media-types/media-types.xhtml

If we someday change the internal format of .otio files to something other than JSON, should/would the MIME type or UTI change?

The mime/uti tell a compliant parser that it should be able to consume the file ~ so a change from json to binary wouldn't necessarily require new signatures. The extension isn't important either; an extension to mime-type mapping can be set, but it's not exclusive, one could declare the mime-type of foo.bar is image/x-exr and an exr parser would be expected to load and parse it. If you wanted to preserve and abandon old-otio files to old-otio-parsers such that new parsers would never see old-otio, that's the circumstance under which you'd introduce a new mime/uti signature, to avoid crossing the streams.

at first blush, txt/otio would seem to be mime type. Other suggestions welcome.

See also discussion here: https://github.com/AcademySoftwareFoundation/tac/issues/541#issuecomment-1887735866


sleep 30 seconds after installation completed, then to run the test case

https://github.com/open-cluster-management/backlog/issues/8697
for the cert renewal problem in observability, cause is that we started the test case before the installation is ready indeed. Add a "sleep 30 sec" logic after installation can resolve this problem.

discuss with @marcolan018, invalid PR, and closed.


Escape linebreaks properly in the difftool command

Implicit linebreaks in the backticked command line don't seem to work in all Ruby versions, so we should be explicit about escaping linebreaks.
This was making Whitehall builds hang.

:+1:


Web machine learning @ W3 - let's not let them spoil it!

Guys (and Girls? :-) ),
There is W3 community that tries to work out a standard for machine learning for web. It is great idea, but I'm strongly afraid that they will make another crappy standard that we all will have to live on later. I find Brain.js API very close to something I would expect from standards for web scripting - simple to use, simple to learn, doing its work. I'm worried also that W3 group will overcomplicate standard so we will have to wait for it couple of years to see in browsers. I think it would be much better to make firstly quick basic ML standard and then second version that extends it with all kind of "low level" and "super scientific" ML capabilities.
I want you all to get involved to avoid mentioned problems. You're doing great work with Brain.js! With your experience and expertise (or just voting power) we could push it forward so standard could be quickly adopted as something that is close to Brain.js. Something that developers could quickly adopt without tons of code and studying.
If you decide to get involved go to:
https://www.w3.org/community/webmachinelearning/
https://github.com/webmachinelearning/webnn/issues
I personally see ML api for www as something like that: https://github.com/DanielMazurkiewicz/WebAI, but anything close to Brain.js should be simply good.
Don't wait for the standard, make it! :-)
Cheers!

Hi @DanielMazurkiewicz!  This does sound awesome.  Have you looked at https://github.com/BrainJS/brain.js/wiki/Roadmap as well?  This is our 2.0 roadmap.  We're currently implementing it, and we're starting planning on v3, here: https://github.com/BrainJS/brain.js/issues/317.
I'm interested in this, but it will take more energy than just me.  If we are going to do this, we need to form a team of willing volunteers, that can help respect the ideas of both simplicity and the needs that such a standard would require.
Just curious, and wanted to start the discussion.

Hi @robertleeplummerjr !

Have you looked at https://github.com/BrainJS/brain.js/wiki/Roadmap as well? This is our 2.0 roadmap. We're currently implementing it, and we're starting planning on v3, here: #317.

Yes, I've seen it. I don't know how about you, but I would like NN to be easily exchangeable not only within JavaScript runtimes, but also with other programming languages, and for that JSON format of storing NN architecture (also for recurrent NN) is much better in my opinion. So as for the new versions of Brain.js I would prefer more JSON descriptive formats instead of any sort of callback functions or somewhat "layer objects". That is why I proposed to feed NN constructor with property "layers" that looks like that: layers: [8, 14, 8]. It gives an opportunity to introduce later named layers with sort of pipes for any sort of recurrent nets. For example:
https://github.com/DanielMazurkiewicz/WebAI/commit/a5578b115c1ac706ce1c2089c4420979d2713c3a
As for using OpenCL plans, that sounds great!
In couple of weeks I plan to start to work on vulkan compute module on top of NAPI for nodejs. With vulkan it should be easy to use all available GPUs simultaneously - even from multiple vendors. Coding NN directly in SPIR-V should give performance gains even over OpenCL. Moreover, I think it might be even bigger performance gain if making entire training process without need of involving CPU, SPIR-V should be able handle that. Moreover - intel offers spir-v to cpu efficient compiler that uses CPU accelerations. So once written same code could be used very efficiently on CPU and GPU.
So if I could wish myself something here it would be brain.js api ported to spir-v and available as one of brain.js backends (openCl can be another one) :-)

If we are going to do this, we need to form a team of willing volunteers, that can help respect the ideas of both simplicity and the needs that such a standard would require.

Totally agree! For me it is not a question "if" - I've already started supporting and promoting brain.js like api for W3 NN standard. I think the more people supporting brain.js will sign to W3 to that project, the more chances to get the standard falling into Simplicity and Performance First phrase. It doesn't require much to sign and sometimes a share opinion.
Is it ok for you to place sort of appeal to participate in standard preparations in read.me of Brain.js? So users of brain.js could also share their opinion?

Yes, I've seen it. I don't know how about you, but I would like NN to be easily exchangeable not only within JavaScript runtimes, but also with other programming languages

You share my desire, and I feel a large part of the community's as well.  I've always wanted to do a quick port to c++, c#, java, and php, just for the sake of offering an easy api to everyone.

So as for the new versions of Brain.js I would prefer more JSON descriptive formats instead of any sort of callback functions or somewhat "layer objects".

Short answer: I love this strategy!  This actually is already mostly adopted in the fromJSON method, but I agree, it should be very easy to invoke.  As for instantiation, that'd easily be part of the options.  The main thing is that GPU or CPU (or other?) be accessible quickly without much effort to get at or define the thing desired.
Further reading: Something I want to be clear about with the roadmap for v2 and why v2 is important: It describes one of the simplest means of instantiating the net directly, as I'm sure you know.  This allows one to even build their own layers.  The layers, in other words, aren't dependent on the net, and the net on the layers.  You can shim in your own, or use what is there.

In couple of weeks I plan to start to work on vulkan compute module on top of NAPI for nodejs

Why not become familiar with gpu.js, the underlying engine with brain.js, and provide an architecture layer that does this, then offer it to the world?  That would be awesome!  There are already 3 backends, with one coming (Headless WebGL) here: https://github.com/gpujs/gpu.js/tree/develop/src/backend
Simple js in, GPU computation out.  Wicked fast, wicked easy.

Coding NN directly in SPIR-V should give performance gains even over OpenCL.

OpenCL in GPU.js was seen as simply another backend, or driver.  That isn't to say you that if we had an OpenCL module, everything node side would have to go through OpenCL.  It is simply a driver.   If another backend was provided, we'd simply use that, like Vulken.  All the hard stuff is worked out, like type detection, javascript transpilation to whatever language you like, etc.

So once written same code could be used very efficiently on CPU and GPU.

GPU.js already does this.

For me it is not a question "if"

Very encouraging to hear!

I think the more people supporting brain.js will sign to W3 to that project, the more chances to get the standard falling into Simplicity and Performance First phrase.

I agree!  One of the biggest needs is being open to new ideas from the community.  They have some pretty fantastic ones.  Also, not to be afraid of sticking up for an idea with logic backing it.  Although simple, it has been one of the neatest things to see in practice for brain.js and open source.

Is it ok for you to place sort of appeal to participate in standard preparations in read.me of Brain.js? So users of brain.js could also share their opinion?

I welcome it.

Further reading: Something I want to be clear about with the roadmap for v2 and why v2 is important: It describes one of the simplest means of instantiating the net directly, as I'm sure you know. This allows one to even build their own layers. The layers, in other words, aren't dependent on the net, and the net on the layers. You can shim in your own, or use what is there.

We are on same page here. But if it comes to standardization that additionally should work among variety of different hardware I think it is easier to stick to JSON that can be quite easily "compiled" to work on selected hardware, than having a custom JS code that will have to run somewhere in between hardware and JS runtime. I think better to focus on JSON descriptive format to be as flexible as possible (if it comes to standardization of course). In some later phases of upgrading standard it could add some more sophisticated techniques that could use custom JS inside standardized ML.
Vulkan and deep down performance is entire big another topic :-) , so maybe lets split here and summarize what I/we expect from W3C standardized API:

Simple to use, simple to learn API (lets make a proposal draft of it or evolve my proposal: https://github.com/DanielMazurkiewicz/WebAI)
Allowing to use all available hardware and hardware accelerations simultaneously
Neural Network architecture should be described entirely in standardized JSON (keeping in mind also a flexibility, which is a possibility to mimic in JSON all sort of NN architectures: https://towardsdatascience.com/a-deeper-understanding-of-nnets-part-1-cnns-263a6e3ac61) - NN could be easily exchanged not only between javascript based runtimes, but also between other languages / ML runtimes / ML macro compillers / so on...
Safety that will guarantee JS runtime exclusive access to its NeuralNetwork object
Splitting standardization process to parts, so it could be quickly adopted and later extended:
a) Basic ML functionality/API - widely used high level well proven algorithms (keeping in mind that it is gonna be extended)
b) Extended ML functionality - extending JSON architecture capabilities, training algorithms, access to neural network internals, all highly scientific orientated API, custom JS injections...
c) Providing standardized set of ML models (and dedicated APIs) that could be widely used (like body parts recognition - face, hands, pose, gestures ...; characters recognition, voice recognition, shapes recognition - triangle, rectangle, circle, square ...; objects recognition; ... TBD)


Letters' bouding box is merged

Environment

Tesseract Version: tesseract 4.1.0
Commit Number: ---
Platform:  Darwin xxx.local 18.7.0 Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT 2019; root:xnu-4903.271.2~2/RELEASE_X86_64 x86_64

Current Behavior:
bouding box of letters are somehow mixed uptesseract picture.jpg output batch.nochop makebox

source:

interestingly, tesseract beahves different when I crop word by hand or when I first search for word, crop it and then search for letters. It also behaves slightly different than when the word is in context, like above.

Expected Behavior:
get bouding box correctly
Suggested Fix:

If you need exect bounding boxes per letter - use tesseract 3 (or try OEM_TESSERACT_ONLY or OEM_TESSERACT_LSTM_COMBINED as OcrEngineMode )

Just curious, why do you need bounding boxes of glyphs?

https://github.com/tesseract-ocr/tesseract/commit/3a5e5089343798932d9952628acfdf56f3108c43
This was committed after 4.1.0 was released, so you might want to test the latest code from the master branch.

I am also interested in getting a fix for this. @amitdo , a couple of reasons we might want the correct bounding boxes for glyphs:
For intelligent data capture applications, not just plain OCR of a document. I.e. field extraction. The classic example is invoice data extraction - an application that finds specific fields on an invoice to automate data entry into some finance application. Like, total amount, invoice number, purchase order number, invoice date. We may have code that gets all glyphs in a rectangular region of the page. This would only be accurate if the glyph's bounding boxes were accurate.
Another reason is that we might want to display OCR results for human verification. For example, ask the user to verify glyphs that have a low OCR certainty, under some threshold. We may want to highlight the character/glyph that requires verification on the document image, using the bounding box coordinates.

I am afraid that if you need best bounding boxes for glyphs you have to use legacy engine for this:

But current tesseract (5.0.0-alpha-20201224-4-g924d1) in OEM_LSTM_ONLY provide already much better result than originally reported:

I am afraid that if you need best bounding boxes for glyphs you have to use legacy engine for this:

But current tesseract (5.0.0-alpha-20201224-4-g924d1) in OEM_LSTM_ONLY provide already much better result than originally reported: