fix issue with indentation of code/markup following directive blocks ...

… related to #4358
Summary of the changes
added a test that demonstrates the problem scenario:
@section Foo {

}

@section Bar {
    @{
        var test = 1;
    }
}

gets incorrectly formatted like
@section Foo {

}

    @section Bar {
    @{
        var test = 1;
    }
    }

added a block of code that fixes the problem
removed a previously added block of code that only partially resolved the issue

Looks like you need to merge from main perhaps, lots of unrelated changes it seems. Also could you update the description, or log a new issue that this PR can "fix", to demonstrate what the formatting engine used to do in this situation? Thanks!

changes made since initial review by @davidwengier:

moved context.CodeDocument.GetSyntaxTree().Root outside of all loops
target the expected structure of the section directive specifically to save overhead of calling FirstAncestorOrSelf

i thought of a scenario that breaks the 'just look at parents' approach: c# code within a section creates new indentation scopes, and when they are picked they may not match the simple 3-nodes-up configuration. so something like FirstAncestorOrSelf must be used. updated the new test to cover the scenario.

Correct me if I'm wrong, but given a situation like below:
@section Goo {
}

For each line with a P tag, above, we'll look all the way up through the whole document, looking all the way up through the tree every time, right? This is where my perf concerns come in.
So even if we could rely on a predictable tree structure, it's still not a great experience, and means formatting is going to be, for some documents, an exponentially slower process as the document grows.

first, a big LOL at @section Goo
i know, it's not great... but improper formatting is a greater evil, imo. trying to think of a better means of determining the indentation. perhaps the relationship between the indentation scopes and their containing directives should be computed before this whole loop, and then during the loop some quicker checks could be made to determine when we are past a directive and remove its scopes from the list of candidates to look back towards.

but improper formatting is a greater evil, imo

This is true in an ideal world, but unfortunately falls down in reality. It's one of the most frustrating things about working on formatting, and developer tooling in general, and I'm sorry you're about to discover it :)
Let's pretend a user has a 1000 file razor file, and they currently are happy with what the formatter does, because they don't use @section. If we were to improve formatting for @section (which this PR definitely does), but also make that format operation take 5 seconds (which, to be clear, this PR probably doesn't do), then what now happens in reality is that the user formats their document, waits a second or two, but sees nothing happening, so they think its a bug, and they keep working. The editor meanwhile gets the formatting results back from Razor, but sees that the document state has changed, and so has no choice but to abandon the formatting edits that Razor produced. Formatting is now producing better results, but the users experience is notably worse.
It's a tricky balancing act, and we have to consider all sorts of things, and unfortunately in past decisions, @section blocks have been somewhat disadvantaged, because their usage is perhaps not as high as other features, or because most users put them at the end of the document, therefore avoiding these issues etc. If you're a user who is affected, this PR is great, but if you're the user from my above scenario, this PR is bad.

perhaps the relationship between the indentation scopes and their containing directives should be computed before this whole loop

I think this is definitely on the right track, and I like how you've avoided doing the tree traversal as often. A couple of things come to mind though, that might be worth trying. I've commented in the PR itself.

Ping @NTaylorMullen @ryanbrandenburg @allisonchou for review.

Thank you @tuespetre for kicking this all off, and the awesome contribution!

thanks to you, crew 🙌


Sharepoint Webpart Documentation out of date

There is an issue with this .tsx code block it is outdated and doesn't work
also the picture in the next step does not represent the code correctly.
I also added my code when i paste it in vs code

[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 71e8ee66-fe7c-a46e-eeec-fb199962042c
Version Independent ID: f78aef17-5aa2-8807-73d4-509faecbd153
Content: Connect your client-side web part to SharePoint (Hello World part 2)
Content Source: docs/spfx/web-parts/get-started/connect-to-sharepoint.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

This was indeed completely wrong - sorry for that and huge thanks for flagging things. We have now updated the web part tutorial series to be up to date and to be a bit more simplified getting started journey.


The website certificate is valid for myscale.com, but not www.myscale.com.

The link in the readme goes to www.myscale.com
It makes Firefox to argue about the certificate.

Fixed with https://github.com/myscale/myscaledb/commit/4490b4f67ee2aade562174379cffd7a48fddcf28


[PDI-19286] XML output : The ">" special character is not escaped whe…

…n "Namespace" is set with this special character.

:white_check_mark: Build finished in 6m 36s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/xml/core

:ok_hand: All tests passed!
Tests run: 80, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


CVE-2021-23337 and CVE-2020-28500 in Lodash (prior to 4.17.21)

Version
3.1.4
Reproduction link
https://www.cvedetails.com/vulnerability-list/vendor_id-20100/product_id-57083/year-2021/Lodash-Lodash.html
Steps to reproduce
Dependency Lodash  4.17.19 is Vulnerable Library
CVE-2021-23337:
Lodash versions prior to 4.17.21 are vulnerable to Regular Expression Denial of Service (ReDoS) via the toNumber, trim and trimEnd functions.
CVE-2020-28500:
Lodash versions prior to 4.17.21 are vulnerable to Command Injection via the template function.
CVE Details:
https://www.cvedetails.com/vulnerability-list/vendor_id-20100/product_id-57083/year-2021/Lodash-Lodash.html
Solution: Upgrade to version lodash-4.17.21
What is expected?
Security Vulnerabilities
What is actually happening?
Security Vulnerabilities

Lodash is a dev dependency, it's not exposed in the final package


8.5.3 Patch Release

Lots of little things have been fixed up and changed from user feedback. Thus, I want to get this seemingly small patch out and available now.
Release Notes

Added support for playing with champred's patch that changes all moves to use the Physical/Special split

View the online release notes for a patch download link: Download v1

OHKO move accuracy is now automatically calculated and shown
Unlearnable Gym TMs in the Log Viewer are now more subtle. You can disable these from the Log Viewer options tab.
[Ruby/Sapphire/Emerald] An icon showing that your Pokémon has PokéRus now displays on the Tracker screen
Added a 1 second cooldown when using the New Run feature, to prevent it from occurring twice in a row
Fixed an issue where the Tracker could not detect a battle had ended when playing on a French FireRed ROM
Fixed some typos, as well as cleaned up several buttons and texts for popup windows
For a full list of new features, changes and bug fixes, check out the Version Changelog

One small note for the physical/special split is that it is based off the most recent generation, so if any move categories changed it may not match up with what people are used to from the gen 4/5 tracker. May be good to mention, but it's probably not a big deal.

One small note for the physical/special split is that it is based off the most recent generation, so if any move categories changed it may not match up with what people are used to from the gen 4/5 tracker. May be good to mention, but it's probably not a big deal.

from what i can see seems like there were only a few moves that changed category and they're later gen changes - https://bulbapedia.bulbagarden.net/wiki/List_of_modified_moves#Damage_category_changes

spikes seems to be the only applicable one

Good to know that doesn't change anything then. I double checked the code to be sure and it's still status.


Question: React dependency not a peer dependency?

Hello!
My team is considering using this library.  One thing that we were wondering about is why React and React-DOM is only listed in package.json as a devDependency and not as a peerDependency also?
One thing that we're trying to determine is what versions of React would work with this library.  For example, we have a situation where we need to use an older version of React (15.x).  Since React is not listed as a peer dependency in this library's package.json, we are not able to determine if something in this library would break with an older version of React.
Thanks for your help!

React is listed as a full dependency rather than a dev dependency in each of the individual packages (e.g. packages/card/package.json). Does that need to change also?

I came here due to the following error in my console and following the information from the link in the error:

Uncaught Invariant Violation: Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:

You might have mismatching versions of React and the renderer (such as React DOM)
You might be breaking the Rules of Hooks
You might have more than one copy of React in the same app
See https://fb.me/react-invalid-hook-call for tips about how to debug and fix this problem.

It turns out number 3 is causing the error in my applications which leaded me to this issue here.
Should'nt each package here (ex. @material/react-text-field, @material/react-checkbox,...) have 'react' as a peer dependency as suggested in the documentation listed above?

Correct, as stated in the previous comments, we agreed react should be changed to a peerDep from dep.

I've gone through all package.json files and changed react and react-dom to peer dependencies.
Is that all that must happen? #940

Thanks @thdk!


Motion: Let sized deallocator warnings be fixed properly

When currently building with Gcc 5.3 (on Cygwin), using Automake, there are gazillions of compatibility warnings.
When I build with my local IDE, these lead to errors, because -Werror is enabled regardless.
This is ugly, to say the least.
Why not remove these functions again, set -fno-sized-deallocation, which will work on all flavors (including std=C==14), and only add them much later, when we need them or get problems otherwise (should not happen before 2019, judging by how long it took for C++11 to become (almost) mainstream...

As an aside, why is my local build done without -Werror?

If you want the code to stay this way, we should use  -Wno-c++14-compat to silence the warning. Also, as far as I know, -Werror should be enabled for all builds?

Duplicate of #911 - sorry


Pass errors callback function

Please consider to pass errors to error callback function.

Getting problem for some numbers like this "400005498004". Does not give output

EAN13 barcodes need a number with 13 digits. Your number only has 12.

Hi,
I checked the source code, hope the following code at line 14 in common.coffee which is already included should create the check digit.
if @number.length == 12
checkDigit = @generateCheckDigit(@number)
@number+=checkDigit

And its working for my other 12 digit keys (Have a look at my actual code below)
var barcodeOptions = {color:"#333",prefix:true,number:true,};
new EAN13(document.querySelector('#ticket1barcode'), '445003517032', barcodeOptions);
new EAN13(document.querySelector('#ticket2barcode'), '445003518032', barcodeOptions);
new EAN13(document.querySelector('#ticket3barcode'), '445003519032', barcodeOptions);
new EAN13(document.querySelector('#ticket4barcode'), '445003520032', barcodeOptions);
new EAN13(document.querySelector('#ticket5barcode'), '445003521032', barcodeOptions);
new EAN13(document.querySelector('#ticket6barcode'), '445003522032', barcodeOptions);
new EAN13(document.querySelector('#ticket7barcode'), '445003523032', barcodeOptions);
new EAN13(document.querySelector('#ticket8barcode'), '445003524032', barcodeOptions);
new EAN13(document.querySelector('#ticket9barcode'), '445003525032', barcodeOptions);
new EAN13(document.querySelector('#ticket10barcode'), '445003526032', barcodeOptions);

And here I don't get output only for<PHONE_NUMBER>32.
It seems its not working when check digit is zero.

Sorry, you're right. I forgot about this feature myself. ;)
44c83f642f should fix it for you.

Thanns Johannes. The commit worked for me :)

Hi,
commit 42c7e226cd87788e10ac60173c214522d138be30 is not resembling in distribution files.
Please check line number 25 in ean13.js and line number 23 in jquery-ean13.js

I now moved the release files to the release area. I had problems with this before. Thank you!


skip notebook smoke tests for now

Notebook smoke test is still timing out after increasing the timeout. Skipping test for now while this is worked offline.

Pull Request Test Coverage Report for Build 316524559

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

Totals

Change from base Build 316283623:
0.0%

Covered Lines:
20347

Relevant Lines:
47302

💛  - Coveralls

Failure on product build is unrelated (occurred against multiple branches), so merging in


DecompressionViaStream fails on .NET Core 2.0 and above.

DecompressViaStream test currently fails on .NET Core 2.x/3.x but passes on .NET Core 1.x.
Message: System.Exception : Decompress failed with for mapsdatazrh.compressed
----> System.ArgumentException : Offset and length were out of bounds for the array or count is greater than the number of elements from index to the end of the source collection.

Fixed in https://github.com/master131/BrotliSharpLib/commit/04ab69990c5d400b531bfeccaaadade61e7a3246.


Move interactive.open call back into NativeInteractiveWindow ctor

This reverses course on a previous recommendation, but is necessary in order to prevent races instantiating an interactive window. Basically we need to create our NativeInteractiveWindow wrapper without awaiting on anything (added a comment to that effect in the provider). This means moving getInteractiveController and executeCommand('interactive.open', ...) into the NativeInteractiveWindow ctor, and also means notebookDocument can now be undefined and we have to await on this.loadPromise before we try to access this.notebookDocument

Codecov Report

Merging #6725 (e8f7978) into main (2e52e4f) will increase coverage by 0%.
The diff coverage is 2%.

:exclamation: Current head e8f7978 differs from pull request most recent head 19903e8. Consider uploading reports for the commit 19903e8 to get more accurate results

@@          Coverage Diff           @@
##            main   #6725    +/-   ##
======================================
  Coverage     68%     69%
======================================
  Files        412     412
  Lines      28599   28609    +10
  Branches    4269    4272     +3
======================================
+ Hits       19722   19823   +101
+ Misses      7193    7094    -99
- Partials    1684    1692     +8

Impacted Files
Coverage Δ

...e-window/nativeInteractiveWindowCommandListener.ts
9% <0%> (ø)

...ence/interactive-window/nativeInteractiveWindow.ts
5% <1%> (-1%)
:arrow_down:

...eractive-window/nativeInteractiveWindowProvider.ts
17% <7%> (+2%)
:arrow_up:

src/client/datascience/kernel-launcher/types.ts
48% <0%> (-40%)
:arrow_down:

src/client/common/process/baseDaemon.ts
53% <0%> (-5%)
:arrow_down:

src/client/common/errors/index.ts
69% <0%> (-4%)
:arrow_down:

...lient/datascience/kernel-launcher/kernelProcess.ts
72% <0%> (-3%)
:arrow_down:

...ience/kernel-launcher/localKernelSpecFinderBase.ts
79% <0%> (-3%)
:arrow_down:

.../datascience/notebook/notebookControllerManager.ts
76% <0%> (-3%)
:arrow_down:

src/client/datascience/raw-kernel/rawSocket.ts
84% <0%> (-2%)
:arrow_down:

... and 19 more


Update docusaurus.config.js to fix dev docs link

Fixed broken link  from: "/docs/developers/intro",
to: "/docs/category/develop-on-nova-evm-",

I also wonder where the trailing "-" at the link "/docs/category/develop-on-nova-evm-" is coming from, it looks awful. You didn't introduce it, of course, but would you be able to have a look and fix it, so that the link to the page is "/docs/category/develop-on-nova-evm" @ryyeung ?


[v3.2] Update healthcheck port from 10248 to 10256

With changes to gravity/kubernetes in version 3.x, port 10248 is now only internally accessible and cannot be used for Load Balancer health/liveness checks. I suggest changing to the kube-proxy 10256 port that still has an externally accessible /healthz endpoint that provides an HTTP:200 response and json date/timestamp.
@IsaacEldridge - Created this separate PR for v3.2 of changes in #293 in v3.1 branch
Writer's Quality Checklist
Before merging your PR, did you:

[ ] Run spell checker
[ ] Run link checker to check for broken xrefs
[ ] Check for orphan files
[ ] Perform a local build and do a final visual check of your content, including checking for:

Broken images
Dead links
Correct rendering of partials if they are used in your content
Formatting issues, such as:

Misnumbered ordered lists (steps) or incorrectly nested unordered lists
Messed up tables
Proper indentation
Correct header levels

[ ] Receive final review and signoff from:

Technical SME
Product Manager
Editor or peer reviewer
Reporter, if this content is in response to a reported issue (internal or external feedback)

[ ] If applicable, verify that the software actually got released

Thanks @zeelewis


Metrics related to gax

Question
This is a general question regarding how to get metrics around gax. (Or, maybe this is a feature request?)
Recently we realized that gax (transitive dependencies introduced by java-bigtable client library) tends to create a fairly large thread pool of many system threads to handle IO with Bigtable server. These threads are named with prefix Gax- so we think it is probably related to this gax-java library according to this line.
We're not sure if the size of the thread pool is the culprit of causing regression of our service performance when the traffic goes high. Thus we'd like gain some observability into the gax threads first. However, I cannot find a good way to export the metrics such as "the number of gax threads" from time to time. For example, an option to register this metric to prometheus-metric client would be great.
We also noticed that Gax has a channel pool resizing behavior over the time. It would also be nice to get metrics of channel pool size over the time and watch how the pool size get resized under different traffic patterns.

@rfan-debug We'll need to learn prometheus-metrics client. Do you have metrics from other Google-related classes that are already working for prometheus-metrics client, so that we can learn from it?

@blakeli0 Considering that we have an observability effort, do you want to take note of the metrics asked for here and close the issue, or keep it open?

@blakeli0 Considering that we have an observability effort, do you want to take note of the metrics asked for here and close the issue, or keep it open?

Yes the metrics asked for are already being noted in internal docs. I was planning to reply here once we have something more concrete, does that sounds good to you?


BMM150.h is incompatible with Arduino Cloud

My sketch ('compass.ino' example) stops when I try to access the BMM150 libraries in Arduino Cloud. All works as expected outside the cloud. Yes, I have implemented the recommended sketch structure for I2C devices in Setup. Any attempt to access the BMM150 within the sketch in Arduino Cloud causes the program to halt. If I comment out the statements, the sketch continues. Sketch installed on a generic ESP32-wroom-32.

Is your I2C initialization completed? On the other hand, are the resources occupied by Arduino Cloud conflicting with other resources?

Hello,
I'm going to close this issue, feel free to re-open it if you have any other questions.
[Autocommentary]


core and spec library should support bioimage.io ids with version

since this is the new default way it is presented on the website.
Currently does not work:
import bioimageio.core
bioimageio.core.load_resource_description("10.5281/zenodo.5817052/5906839")

(This model: https://bioimage.io/#/?id=10.5281%2Fzenodo.5817052%2F5906839)
fails with the following error:
RuntimeError                              Traceback (most recent call last)
<ipython-input-2-d7955696f13e> in <module>
----> 1 bioimageio.core.load_resource_description("10.5281/zenodo.5817052/5906839")

~/Work/bioimageio/core-bioimage-io-python/bioimageio/core/resource_io/io_.py in load_resource_description(source, weights_priority_order)
     39         return source
     40
---> 41     raw_rd = load_raw_resource_description(source, update_to_format="latest")
     42
     43     if raw_rd.type == "model" and weights_priority_order is not None:

~/Work/bioimageio/spec-bioimage-io/bioimageio/spec/io_.py in load_raw_resource_description(source, update_to_format)
    167             return source
    168
--> 169     data, source_name, _root, type_ = resolve_rdf_source_and_type(source)
    170     if root is None:
    171         root = _root

~/Work/bioimageio/spec-bioimage-io/bioimageio/spec/shared/_resolve_source.py in resolve_rdf_source_and_type(source)
    154     source: typing.Union[os.PathLike, str, dict, raw_nodes.URI]
    155 ) -> typing.Tuple[dict, str, typing.Union[pathlib.Path, raw_nodes.URI, bytes], str]:
--> 156     data, source_name, root = resolve_rdf_source(source)
    157
    158     type_ = data.get("type", "rdf")

~/Work/bioimageio/spec-bioimage-io/bioimageio/spec/shared/_resolve_source.py in resolve_rdf_source(source)
     83                 response = requests.get(f"{zenodo_record_api}/{record_id}")
     84                 if not response.ok:
---> 85                     raise RuntimeError(response.status_code)
     86
     87                 zenodo_record = response.json()

RuntimeError: 404

good catch, I'll fix asap


Define _PyCFunctionFastWithKeywords() on CPython 3.7+

https://docs.python.org/3/c-api/structures.html#c._PyCFunctionFastWithKeywords

Regarding the 3.7 cfg, this was introduced in cpython commit 6969eaf4682beb01bc95eeb14f5ce6c01312e297 and first tagged in v3.7.0a1.

Regarding the 3.7 cfg, this was introduced in cpython commit 6969eaf4682beb01bc95eeb14f5ce6c01312e297 and first tagged in v3.7.0a1.

we incorrectly use args: *mut *mut PyObject in _PyCFunctionFast when it should also be *const *mut PyObject like you have used here.

This is actually correct for 3.6:
typedef PyObject *(*_PyCFunctionFast) (PyObject *self, PyObject **args,
                                       Py_ssize_t nargs, PyObject *kwnames);

we incorrectly use args: *mut *mut PyObject in _PyCFunctionFast when it should also be *const *mut PyObject like you have used here.

This is actually correct for 3.6:
typedef PyObject *(*_PyCFunctionFast) (PyObject *self, PyObject **args,
                                       Py_ssize_t nargs, PyObject *kwnames);

👍 thanks for the review @nw0 - in which case I'll merge this now and we can leave further corrections for #1289

👍 thanks for the review @nw0 - in which case I'll merge this now and we can leave further corrections for #1289

I see you committed the rewording, thanks.

I see you committed the rewording, thanks.


Clang format

Add a clang format file to enforce code style rules.

You might consider using a script similar to this: https://github.com/smaeul/uncrustify-auto
That one, uses uncrustify, but it's probably not too hard to convert it to clang format.

I have a cmake clang-format script sitting around. It is currently a little crusty (relies on bad cmake style), but if I clean it up I would consider donating it to the cause :)


Pagination with table layout and hardcoded "showing X to X of X entries"

[x] I have searched the Issues to see if this bug has already been reported
[x] I have tested the latest version

Steps to reproduce

Create a Pagination element with layout='table', a fixed totalPages (like 23) and a variable currentPage
Notice that there is this behaviour while navigating:

currentPage = 1 => "Showing 1 to 5 of 23 Entries"
currentPage = 2 => "Showing 1 to 5 of 23 Entries"
currentPage = 3 => "Showing 1 to 5 of 23 Entries"
currentPage = 4 => "Showing 2 to 6 of 23 Entries"
currentPage = 5 => "Showing 3 to 7 of 23 Entries"
(this behaviour goes on)

Current behavior
The "Showing" label doesn't seem to actually display information related to the page.
Expected behavior
Showing the actual range of items displayed, like "Showing 1 to 10 of 230 Entries", "Showing 11 to 20 of 230 Entries" and so on. This, though, would require the pagination element to take parameters regarding this range, since it depends on the actual results displayed in a table and could be variable (e.g.: last page could contain less elements than the full page).
Other notes
I have noticed in the code that this info is extracted from:
  const lastPage = Math.min(Math.max(layout === 'pagination' ? currentPage + 2 : currentPage + 4, 5), totalPages);
  const firstPage = Math.max(1, lastPage - 4);

Which seems to be used both for the default "pagination" layout, where it probably makes sense (I haven't checked), but also for the table layout, where the label doesn't (or shouldn't? or doesn't appear to?) refer to pages but rather to single items.

Yeah, I've noticed this as well. This is a hybrid of bugfix and new feature. We have weird behavior in some environments where the first couple of clicks don't change the active page, and also, there should definitely be a property to set the number of items per page.

I think it either needs the number of items per page + total number of items, or starting item index + last item index, otherwise there isn't enough information to correctly compute the indices of the last page.
E.g., if we have 35 elements and 10 elements per page, we will have the ranges 1-10, 11-20, and then the last page is 21-35, which is not the same as the component could compute on its own just with the page size (which would result in 21-40).
If I find some time I can try to make a pull request for this issue (more likely during the upcoming winter holidays rather than right now).


Add BLE operations visualization tool

Summary
A Stetho like module for debugging real BLE connections on Android devices.

#377


Keep history

...or show it, because now, on every app start I'm asked to search for a repo.
I should see those that I edited in the past (and that are theoretical on my device already) at start and make a "Translate another app" button maybe.

If you start writing "ht…" on the URL field, it will suggest previous repositories. What other way to show them do you suggest? Maybe a 2-tabs layout, previous repositories and add repositories?

Or that.

Alright. My new idea is to have "Load repository — Add new repository — Discover" as three tabs in the main activity. What do you think?

You mean Local or Load ? Something that you point to a folder say on the SD card?

I mean Load. Load a local "repository" (the repository itself is not cloned, only the strings.xml files are downloaded), which you previously added and thus saved on the internal storage.

I hope you like the new layout, any feedback about it is appreciated!

Maybe some icons would make evrrything even prettier.

Yeah I thought that too but that would involve scanning the GitHub repository of the application (I can't depend on the F-Droid repository; the application you're translating might not be there).
Also, although the application icon is usually stored as ic_launcher.png, it might not be necessarily the case. Furthermore the GitHub API doesn't have an option to search for a file (when I look for the strings.xml files, I actually cheat a bit and use the search in the file contents because the word resources is always on the strings.xml file).
Maybe I could sneak the AndroidManifest.xml and see the android:icon="@mipmap/ic_launcher"… Anyway, not a priority.

@OffifialMITX, remember this?:

Maybe some icons would make evrrything even prettier.

Well, since I clone the repository I can now save that. Done on commit#848cba.

I don't understand, what do you mean?

I don't understand, what do you mean?

I mean I've added icons on the home screen, just like you wanted @OffifialMITX:

Oh cool!

I don't think that the ".com" after GitHub and GitLab is actually needed.

It's the full host. An application may be hosted at an .org or .net host. Anyway, the .com is a useful as the GitHub part (nothing at all), so I don't mind it being there.

You're actually right.
Am 15. Januar 2017 17:06:05 MEZ, schrieb Lonami<EMAIL_ADDRESS>

I don't think that the ".com" after GitHub and GitLab is actually
needed.

It's the full host. An application may be hosted at an .org or .net
host. Anyway, the .com is a useful as the GitHub part (nothing at
all), so I don't mind it being there.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/LonamiWebs/Stringlate/issues/3#issuecomment-272704499

--
Diese Nachricht wurde von meinem Android-Gerät mit K-9/p≡p gesendet.

Can you please upload a screenshot with a larger resolution so I have something to work with? 480x854 would be enough.

@OffifialMITX

What font is actually used?

Here are the first two, the third one belongs to the UI improvement and comes a bit later:

I did think about both options. However:

In the first one, I would still like to see the host somewhere (GitHub, GirLab, etc.).
In the second one, perhaps it would look a bit crazy since all the repositories would show some background at probably different % each.

Also, maybe the selected locale in the top right corner of each item (like for example "(es)") would look good.

Good arguments but:

In my opinion it's not necessary to know which host an application belongs to for it's translating. I will try how a vertical bar looks like in the next few days.
You're actually right, that's what I thought either, but maybe this is how it could look like in the desktop application when the mouse focuses on it.

Yes, :+1: for the selected locale
I still have some things on my might, containing mainly progress % view, start menu UI and the design of the translator (I have a basic Idea how it could be optimized for larger strings but I have to think about it a little more) and I'll try to put everything on pictures until the end of the next week.


Add mousewheel support

It is great feature.
should leave this issue open in case anyone wants to discuss it. :+1:

I'll close this PR as we've stopped working on our fork.


writeContractDeposit

This would have the same API as viem's writeContract but under the hood use writeSendMessage

Why did you delete the one I commited lol?

done in #91


YouTube notification when Lewis posts a new video

Todo:

[x] Use the Discord tasks
[x] Get the YouTube secret from the .env file
[x] Check if its a new upload

todo:
[ ] Check if its a new upload

Gonna merge it then


fix: update go tutorial removing deprecated openfeature sdk

This PR

This pull request removes the deprecated go package github.com/open-feature/go-sdk/pkg/openfeature and includes minor documentation improvements. The changes include:

Importing github.com/open-feature/go-sdk/openfeature due to github.com/open-feature/go-sdk/pkg/openfeature is deprecated from SDK
Running go get github.com/open-feature/go-sdk/openfeature to fetch the SDK.
Adding missing imports
Minor go improvements

Related Issues
Notes

Follow-up Tasks

How to test

Files Modified

openfeature.dev/docs/tutorials/getting-started/go.mdx

How to Test

Pull down this branch.
Compile and run locally
Follow the instructions and steps to generate go implementation

Thanks @jbovet! I'll review this tomorrow.


.NET 5: NullReferenceException thrown from WinRT.Runtime.dll calling GattCharacteristic.WriteValueAsync

Describe the bug
NullReferenceException thrown from WinRT.Runtime.dll using . TFM net5-windows10.0.19041.0 .
I have code that connects to a BluetoothLE device using Windows APIs and which writes to a GattCharacteristic. This code works in a .NET 4.7.2 console app, where the Windows SDK is referenced by adding a reference to the Microsoft.Windows.SDK.Contracts NuGet package.
Moving to .NET 5 and with the exact same code, the call to GattCharacteristic.WriteValueAsync now fails with a NullReferenceException thrown from WinRT.Runtime.dll. In this .NET 5 console app, the Windows SDK is referenced by setting the TargetFramework in the csproj file to net5-windows10.0.19041.0.
To Reproduce

Use the attached sample: SDKBugRepro.zip
This sample connects to the Microsoft Service you can setup using the Microsoft Bluetooth LE Explorer sample app: download from the Windows Store.
i). Run the app, select "Virtual Peripheral" from the menu. Select Microsoft Service from the dropdown, then click Add Service. In the service configuration, select "Advertise as connectable", "Include service in advertisement", "Publish advertisement".
Now run the two projects in the supplied sample code - the .NET 4.7.2 app works and can write a string to the Microsoft Write characteristic of the virtual peripheral service. The .NET 5 sample - exact same code - throws the exception.

Expected behavior
The .NET 5 version should work the same as the version built for .NET Framework 4.7.2
Version Info
TFM net5-windows10.0.19041.0
Additional context

@AndyCW is this different from #975 that you previously opened?  We expect for bunch of the Bluetooth APIs to be affected by the bug that we duped #975 to and expect those to be resolved once our latest fixes go out and are available in the .NET SDK.

@manodasanW Apologies! Lost track of what I'd opened.
This is a dup. I will close it.


Suggestion: Allow static members to reference type parameters if they have default types

Search Terms
default type, default type parameters
Suggestion
This is related to #32211 and especially, #24018. I would have commented on the latter, but it's limited to collaborators. @andy-ms said:

Without inheritance that wouldn't make much sense:
class Super<T> {
    static m(x: T): void;
}
Super.m(); // What's `T`?

I see what you mean though, when a subclass is introduced:
class Sub extends Super<number> {}
Sub.m(); // T is number

My proposal is to allow type parameters in static members only if those parameters have default values. For example:
class Super<T> {
  // Error: Static members cannot reference class type parameters.
  // Proposition: Static members can only reference class type parameters with default type.
  static value: T
}

let val = Super.value // What's T

But if you specify a default value, everything makes sense:
class Super<T = number> {
  static value: T
}

let val = Super.value // number

class Sub extends Super { }
class Sub2 extends Super<string> {}

let subval = Sub.value // number
let subval2 = Sub2.value // string

Use Cases
In my personal use case, I have an instance property options that merges input with data from a static defaults property:
class Component<O> {
  static defaults: O // Error: Static members cannot reference class type parameters.
  options: O

  constructor (settings: Partial<O>) {
    let ctor = this.constructor as typeof Component
    this.options = { ...ctor.defaults, ...settings }
  }
}

If I then extend the class, I get no type-checking for defaults:
interface SubOptions {
  foo: string
}

class SubComponent extends Component<SubOptions> {
  static defaults = {
    foo: 42 // not type-checked
  }
}

Examples
With my proposal, you could define the base class like this:
interface DefaultOptions {
  element: HTMLElement,
  value: number
}

class Component<O = DefaultOptions> {
  static defaults: O
  options: O

  constructor (settings: Partial<O>) {
    let ctor = this.constructor as typeof Component
    this.options = { ...ctor.defaults, ...settings }
  }
}

The descendant like this:
interface SubOptions {
  foo: string
}

class SubComponent extends Component<SubOptions> {
  static defaults = {
    foo: 42 // Error: Type '42' is not assignable to type 'string'.
  }
}

And the final result would be:
let def = Component.defaults // DefaultOptions
let foodef = SubComponent.defaults // SubOptions

Everything is explicitly defined and typed.
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

This would be a massive soundness hole. Nothing about any particular instantiation of e.g. O is at all constrained by the default; literally any type might come out of that spread expression with no relation to the instantiated type.

@RyanCavanaugh I don't think I understand.

literally any type might come out of that spread expression

I'm combining O and Partial<O>, so the return type must be O? I'm fairly new to TS, but I checked this in the playground and it's correct:
interface O {
    test: string
    len: number
}

let defaults: O = {
    test: 'hello',
    len: 42
}

let input: Partial<O> = {
    test: 'world'
}

// typeof merge is O
let merge = { ...defaults, ...input }

Could you explain and perhaps give an example? I want to understand.


further spells

guy from reddit

Good to see you here, merging your request !!
Will update rest files and the changes will be visible by tomorrow morning


Add feels_like to lovelace card

As stated in my request on the integration, it would be great to see this data added to the lovelace card. For example, my dark sky integration that I am moving away from shows the apparent temp in the following location of  a custom card

Ok, feels like added in the new released version.


beekeeper-studio: init at 1.10.2

Motivation for this change
Add the open source SQL editor and database manager beekeeper-studio! It's agnostic of
which database you use so you can connect to MySQL, MariaDB, PostgreSQL and many others
in a cool GUI app.
Things done

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

[x] 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"
[x] 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)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks for the contribution! :) PTAL at the outstanding review comments.

https://github.com/NixOS/nixpkgs/pull/140700 solves this package!


Minor Feature Request: URI to Open Plugin/Theme Browser + Update All

I noticed after a while, that there are three things in the settings that I open very regularly, so speeding up the process via URI might be useful. In particular:

trigger the "Update All Community Plugins" Function
open the Community Theme Browser
open the Community Plugin Browser

Right now, via the settingid URI, I can only access the community plugin tab, and the appearance tab, creating an unnecessary extra step. This is only a super minor feature request, so no hurry at all! :)

(I just noticed my update all plugins action does not work, I try to fix it)

Now it's hopefully really works. I have to wait 10 after checking for update, because the internal functions in Obsidian doesn't expose a Promise, so I don't know when it's finished with checking for updates.

Great, just tested it, works perfectly!
However, what happens when the process of checking for updates takes longer than 10 seconds, e.g. because of many plugins/bad internet? 🤔

btw, the "updated all plugins" URI is not yet mentioned at all in the readme. (I got the URI by looking through the main.ts.)

@chrisgrieser I know it's not a good solution, but the only way I could think of. It just presses the button again, so it can't break anything. I just won't work then.
It's documented under others


Feature: Upgrade the cluster without loss

Which command or component
I am the maintainer of a production cluster and I have some questions about the maintenance of the production cluster.
Detailed description of the feature

How to upgrade the version without loss, such as upgrading from 1.23.x to 1.24.x ？
In a production cluster, we wan to optionally add additional SANs (hostnames or IPs)  to the API server PKI certificate,how to update ?

you can use Clusterfile sealos to support all the kubeadm config in Clusterfile

In the feature, just change the image name in Clusterfile and reapply it, sealos will upgrade the cluster (in developing)

you can use Clusterfile sealos to support all the kubeadm config in Clusterfile
In the feature, just change the image name in Clusterfile and reapply it, sealos will upgrade the cluster (in developing)

@fanux
I'm not talking about the first installation, I just use SANs as an example, how to adjust the parameters after installation in a production cluster.

@myselfghost sealos cert command will support add SANS


Sydney can't log into Whatsapp Web

Well, since yesterday afternoon I've been trying to connect Sydney to my whatsapp and I haven't been successful in any of the times. Tried to reinstall multiple times, changed bing cookie multiple times too and nothing changed.
Basically it doesn't show any apparent error, but it keeps trying to connect forever in the terminal.
I'll leave here the steps to try to reproduce this problem, although I'm still not sure if it's something personal or global:
1- Download and install Sydney as per the steps in this repository
2- Scan the qr code that will appear
3- Wait to see if Sydney can make it past the "Logging in..."

Thank you for your attention and sorry for the excessive issues I've been doing recently.

I am also suffering from this, will try to see what is wrong later

I am also suffering from this, will try to see what is wrong later

Ok thanks a lot sir 👍✨

Fixed in #150. Problem was with whatsapp web js, you need to git pull and yarn install.

Unfortunately, this problem still exists


Add in _skaffold-files _skaffold-package-goals

Built in skaffold support goals:

_skaffold-files: a list of files to watch
_skaffold-package-goals: a list of jib goals bound to the
package phase.

fixes #958

Still need to add in a snapshot dependency. But code can be reviewed till then. This also fixes #962

We don't treat the build files from the the build dependencies, and just recompute every cycle
e.g. Dockerfile changes - need to recompute dependencies
e.g. Dockerfile instruction COPY . .
file is added to the project, that changes the glob pattern so the build file needs to be recomputed anyways.

@r2d4, just to clarify, if src/.../myfile.java changes would you recompute the build?

How the dependencies are parsed for each artifact
Main switch
https://github.com/GoogleContainerTools/skaffold/blob/24aefd21060f3c7e32f8c82ac959e67835c6c997/pkg/skaffold/runner/runner.go#L359-L363
Docker artifact implementation
https://github.com/GoogleContainerTools/skaffold/blob/abe80329cf875d78257c654d177f297d1a9cf15d/pkg/skaffold/docker/parse.go#L222
Bazel artifact implementation
https://github.com/GoogleContainerTools/skaffold/blob/426e56e1a6b70c7942e734d76db42240d7cc7c04/pkg/skaffold/bazel/bazel.go#L39
Jib implemention
func GetDependencies(workspace string, artifact *v1alpha2.JibArtifact) []string {
   // jib artifact might have a field that take in the a list of globs that are dependencies
   // it could just expand those globs and return the list of files
   return filepath.Match(artifact.JibFiles)
}

Thanks @r2d4, I think we should be good to go then?


Review first draft of compliance matrix alignments

We have created a Compliance Crosswalk Matrix(1) - to help see where our baseline criteria aligns with global frameworks, standards, and regulations.  I started us off with looking at the EU CRA, NIST's SSDF, NIST's CSF, Security Insights and I'm working through BP Badges.  Eventually we'll need to do things like NIST 800-53, EU DORA, EU NIS/NIS2, ISO 2700x, etc.
I'd like feedback on the initial alignments from the community to ensure or correct my perspective.  Did I account for and align things appropriately?

The "Crosswalk" tab is where everything ultimately is aggregated and  will be displayed (until we debate the best ways to show our data....a future conversation).  This is where the meat of what I am looking for feedback on.  Here I am keying off the OSPS criteria and showing "If you fulfill OSPS-AC-01. then you also get credit for SSDF requirement PO3.2 or CRA Annex 1.2d, etc.)
Each framework/standard/reg has it's own tab where I key off that standard to the OSPS.  As we pull in more regs/frameworks, a tab such as this is where the mapping will start and ultimately it will get added back into the "Crosswalk" tab.  The data should be identical to the alignments in the 1st tab.

I'd love it if folks could look at this and ponder it, and then we can coordinate a live working session to work through the feedback together.  I propose doing this in early January '25.
Thanks for your time, expertise, and collaboration on this!
(1) - https://docs.google.com/spreadsheets/d/1an5mx3rayoz3JRFUepD56zgprpwXBXBG70fVZvIMCpA/edit?gid=769418932#gid=769418932

Let's hash out the ultimate display method/location in this issue: https://github.com/ossf/security-baseline/issues/99

I've shared this with a number of folks hoping to get more eyes on this. Thank you so much @SecurityCRob and others for your efforts putting together this very large cross walk.


[bug] Unscoped commits should use all scope

Describe the bug
when doing commits without the optional scope feat: fixed something they aren't picked up by auto-gen.
To Reproduce
commit something with feat: fixed something and see that it doesn't appear.
Expected behavior
commit something with feat: fixed something and see that it does appear.
Environment
all

PR is getting pushed soonish.


TicDatFactory and PanDatFactory should be able to read from the same formats by default

Should be fairly straightforward to make this work, unless there is something odd in the table names like "columns", "data" being the actual column names for every table.

Clean up the doc string a bit and then merge through.

Fixed with above commit


Env variables dumped to jenkins-env should be double quoted

This is related to PR #184.

Also related to https://github.com/openshiftio/openshiftio-cico-jobs/pull/186

Don't forget escaping. That's why I wouldn't close #186 that easily. It has a better method of saving env vars.

Fixed by https://github.com/openshiftio/openshiftio-cico-jobs/pull/690


Contrib modules getting built perfectly fine, even when CMAKE says their dependencies are not available

Just grabbed latest master from both main and contrib repository and tried to build all the contrib modules also. What baffled me is the following.
The Cmake procedure clearly looks for dependencies and states that for example the following dependencies are not found
Caffe:   NO
Tesseract:   NO

And there comes the catch, it then states, ow lets build the modules depending on it
To be built:                 cudev core cudaarithm flann imgproc ml reg surface_matching video cudabgsegm cudafilters cudaimgproc cudawarping dnn fuzzy imgcodecs photo shape videoio cudacodec highgui objdetect plot ts xobjdetect xphoto bgsegm bioinspired dpm face features2d line_descriptor saliency text calib3d ccalib cudafeatures2d cudalegacy cudaobjdetect cudaoptflow cudastereo datasets rgbd stereo structured_light superres tracking videostab xfeatures2d ximgproc aruco optflow stitching matlab python2
    Disabled:                    world contrib_world
    Disabled by dependency:      -
    Unavailable:                 java python3 viz cvv hdf sfm

Which means in my opinion that the CMake is wrongly configured.
So my suggestions

If dependencies are not found on the system, do not build the modules itself. That will force people to install the dependencies first and ensure that they know what they are doing.

A small notice is that the Tesseract:   NO is being projected twice in the output, so I am guessing it is checked twice also, which is redundant.

It is just a report of checks. It is not error report.
Most part of OpenCV dependencies is optional.

If dependencies are not found on the system, do not build the modules itself

It works if dependency is required for module, for example, see "hdf" or "viz" module (or set of cuda modules).

Tesseract:   NO

Agree, double check is redundant.
"text" module is available in this case, but it is not fully functional (it is not disabled totally because there was an assumption that there are many detection backends and "tesseract" is only one of them)

I know they are checks. So if I understand it correctly, even though torch and caffe are for example not on my system, i am still able to completely use the dnn module?

In general, it depends on module and its design.
For example, dnn module can read caffe models and execute forward pass processing without using of anything from caffe (dnn module doesn't require caffe at all).

:+1: okay then I guess simply providing a small fix for the redundant check would be enough for closing this down!

Actually, I just did a grep -R "Tesseract" * inside the modules folder of the contrib repository and it returned me
datasets/samples/tr_icdar_benchmark.cpp:        Ptr<OCRTesseract> ocr = OCRTesseract::create();
text/text_config.hpp.in:// HAVE OCR Tesseract
text/CMakeLists.txt:find_package(Tesseract)
text/CMakeLists.txt:if(Tesseract_FOUND)
text/CMakeLists.txt:    message(STATUS "Tesseract:   YES")
text/CMakeLists.txt:    message(STATUS "Tesseract:   NO")
text/CMakeLists.txt:if(${Tesseract_FOUND})
text/CMakeLists.txt:include_directories(${Tesseract_INCLUDE_DIR})
text/CMakeLists.txt:if(${Tesseract_FOUND})
text/CMakeLists.txt:  target_link_libraries(opencv_text ${Tesseract_LIBS})
text/README.md:The recognition part currently uses open-source Tesseract OCR (https://code.google.com/p/tesseract-ocr/). If Tesseract OCR is not installed on your system, the corresponding part of the functionality will be unavailable.
text/README.md:Here are instructions on how to install Tesseract on your machine (Linux or Mac; Windows users should look for precompiled binaries or try to adopt the instructions below):
text/README.md:Tesseract installation instruction (Linux, Mac)
text/README.md:1. Google announced that they close code.google.com, so at some moment in the future you may have to find Tesseract 3.03rc1 or later.
text/README.md:2. Tesseract configure script may fail to detect leptonica, so you may have to edit the configure script - comment off some if's around this message and retain only "then" branch.
text/samples/segmented_word_recognition.cpp:    Ptr<OCRTesseract>  ocrTes = OCRTesseract::create();
text/samples/segmented_word_recognition.cpp:    cout << " OCR_Tesseract  output \"" << output << "\". Done in "
text/samples/webcam_demo.cpp:    cout << "  Press 'o' to switch between OCRTesseract/OCRHMMDecoder recognition." << endl;
text/samples/webcam_demo.cpp:    char *recognitions_str[2] = {const_cast<char *>("Tesseract"), const_cast<char *>("NM_chain_features + KNN")};
text/samples/webcam_demo.cpp:    vector< Ptr<OCRTesseract> > ocrs;
text/samples/webcam_demo.cpp:      ocrs.push_back(OCRTesseract::create());
text/samples/webcam_demo.cpp:              parallel_for_(r, Parallel_OCR<OCRTesseract>(detections, outputs, boxes, words, confidences, ocrs));
text/samples/end_to_end_recognition.cpp: * Shows the use of the Tesseract OCR API with the Extremal Region Filter algorithm described in:
text/samples/end_to_end_recognition.cpp:    cout << "Shows the use of the Tesseract OCR API with the Extremal Region Filter algorithm described in:" << endl;
text/samples/end_to_end_recognition.cpp:    Ptr<OCRTesseract> ocr = OCRTesseract::create();
text/include/opencv2/text/ocr.hpp:/** @brief OCRTesseract class provides an interface with the tesseract-ocr API (v3.02.02) in C++.
text/include/opencv2/text/ocr.hpp:   -   (C++) An example of OCRTesseract recognition combined with scene text detection can be found
text/include/opencv2/text/ocr.hpp:    -   (C++) Another example of OCRTesseract recognition combined with scene text detection can be
text/include/opencv2/text/ocr.hpp:class CV_EXPORTS_W OCRTesseract : public BaseOCR
text/include/opencv2/text/ocr.hpp:    /** @brief Creates an instance of the OCRTesseract class. Initializes Tesseract.
text/include/opencv2/text/ocr.hpp:    CV_WRAP static Ptr<OCRTesseract> create(const char* datapath=NULL, const char* language=NULL,
text/FindTesseract.cmake:# Tesseract OCR
text/FindTesseract.cmake:unset(Tesseract_FOUND)
text/FindTesseract.cmake:find_path(Tesseract_INCLUDE_DIR tesseract/baseapi.h
text/FindTesseract.cmake:find_library(Tesseract_LIBRARY NAMES tesseract
text/FindTesseract.cmake:set(Tesseract_LIBS ${Tesseract_LIBRARY} ${Lept_LIBRARY})
text/FindTesseract.cmake:if(Tesseract_LIBS AND Tesseract_INCLUDE_DIR)
text/FindTesseract.cmake:    set(Tesseract_FOUND 1)
text/src/ocr_tesseract.cpp:void OCRTesseract::run(Mat& image, string& output_text, vector<Rect>* component_rects,
text/src/ocr_tesseract.cpp:void OCRTesseract::run(Mat& image, Mat& mask, string& output_text, vector<Rect>* component_rects,
text/src/ocr_tesseract.cpp:CV_WRAP String OCRTesseract::run(InputArray image, int min_confidence, int component_level)
text/src/ocr_tesseract.cpp:CV_WRAP String OCRTesseract::run(InputArray image, InputArray mask, int min_confidence, int component_level)
text/src/ocr_tesseract.cpp:class OCRTesseractImpl : public OCRTesseract
text/src/ocr_tesseract.cpp:    OCRTesseractImpl(const char* datapath, const char* language, const char* char_whitelist, int oemode, int psmode)
text/src/ocr_tesseract.cpp:            cout << "OCRTesseract: Could not initialize tesseract." << endl;
text/src/ocr_tesseract.cpp:        //cout << "OCRTesseract: tesseract version " << tess.Version() << endl;
text/src/ocr_tesseract.cpp:        cout << "OCRTesseract("<<oemode<<psmode<<"): Tesseract not found." << endl;
text/src/ocr_tesseract.cpp:    ~OCRTesseractImpl()
text/src/ocr_tesseract.cpp:        cout << "OCRTesseract(" << component_level << image.type() <<"): Tesseract not found." << endl;
text/src/ocr_tesseract.cpp:Ptr<OCRTesseract> OCRTesseract::create(const char* datapath, const char* language,
text/src/ocr_tesseract.cpp:    return makePtr<OCRTesseractImpl>(datapath, language, char_whitelist, oem, psmode);

You can clearly see that only text/CMakeLists.txt generates the output, which gets me wondering why this would be generated twice?

So my guess is that somehow the CMakeLists.txt of the text module is called twice instead of simply once.

BTW, inside the CMake config files of dnn I found  #take into account OpenCV two-pass module addition scheme which could explain why also other stuff is double printed... Care to explain what is meant by this?

The ocv_glob_modules macro is used to process all modules, it makes two passes indicated by OPENCV_INITIAL_PASS variable. ocv_add_module macro is used to init basic module variables and compute dependencies, during first pass it stops file processing by calling return(). During second pass it does not stop processing and other commands (ocv_glob_module_sources, ocv_module_include_directories, ocv_create_module, ...) will be called.
The text module is trying to detect Tesseract before ocv_add_module call (hidden inside ocv_define_module), that is why the detection is made twice.

Okay thank you! That is exactly what I needed to know!

Could you check if this is correct?
https://github.com/StevenPuttemans/opencv_contrib/commit/3824b1d3b575805c9bfc746d1fe088af75eb4ecf

Looks good to me

@alalek well I am going to do the same for some of the other modules out there who are skipping this rule of first defining the ocv_define_module before pushing it. It seems to me it is better if the modules all have same behaviour.

I believe CMakeLists.txt for modules should follow this scheme:
// quiet detection (no messages), find_package() with QUIET option

if(required dependencies are missing)
  message(missing required dependencies list) // optional
  ocv_module_disable(...)
endif()

ocv_define_module(...)

// report status of some important optional 3rdparty dependencies, like "Tesseract: YES/NO".

// compiler flags, etc

Yeah I kind of agree :) I guess they follow it now?

It is fine now. Thanks!

:+1:


Reorder categories so that marked and categories from similar articles can appear earlier

As-is
The category dialog sorts the category using arbitrary order (same order the API has returned)

To-be
Per discussed in 20210217, we should

Move categories that has been added to the front
Then list out categories that is also marked in similar articles
Then the rest of the categories.

Move categories that has been added to the front
Then list out categories that is also marked in similar articles
Then the rest of the categories.

how to find the Then list out categories that is also marked in similar articles ?
@MrOrz

Hi @kelvin2go ,
Thanks for taking a look! It can be retrieved by modifying the current GraphQL query to include categories.
query LoadArticlePage($id: String!) {
  GetArticle(id: $id) {
    # ... Other fields ...
    similarArticles: relatedArticles {
      edges {
        node {
          # ... Other fields for the similar article ...
          articleCategories {
            categoryId
          }
        }
      }
    }
    # ... Other fields ...
  }
}


Intellisense in not showing up when trying to add setting for "eslint.trace.server" and "tslint.trace.server"

VSCode Version: Latest alpha
OS Version: Windows10

Steps to Reproduce:

Launch app and open any folder which has .ts or .js file.
Update settings.json file for below properties (for Typescript add tslint and for JavaScript add eslint)
"tslint.trace.server":"messages" ,
or
"eslint.trace.server":"messages"
Build solution or launch output window View->Toggle output.
Verify in the drop down you will get option for eslint or tslint.

Actual: The final output is coming properly but not getting any intellisense when trying to add property in settings.json file.

https://github.com/Microsoft/vscode/issues/6697


unused parameters in module 4

containerRegistryId and keyVaultId are not used by module 4 of bicep while the readme passes them as parameters.
the readme should be updated to reflect that.

thanks - fixed!


feat: remove spendFromUser() func in Spender.sol

The spendFromUser() function in the Spender.sol contract was replaced by the new spendFromUserToWithPermit() function, so the spendFromUser() was removed and the associated Foundry Test was also removed.

Verify command:

DEPLOYED=false forge test -vvv --match-path 'contracts/test/Spender.t.sol'

Let's merge to feature branch first.


No pypcap binaries for python 2.6

There is no pycapc windows binaries for python 2.6

I tried to compile it myself : look like winpcap sources are needed
I tried to compile winpcap myself : look like Windows "driver SDK" is needed
For what I remember, "driver SDK" is not free, and I not willing to use it
illegally nor buy it.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 31 Dec 2008 at 10:19

Hi.
You would like to try Npcap: https://github.com/nmap/npcap.
It's another fork of WinPcap with multiple new features.


Style module sections separately

As far as I can tell, there is no way to style the modules-left, modules-center and modules-right sections separately.
I would like this functionality to be able to set widths for the sections and have transparent areas between them, as if there were three separate bars.

issue should be closed as this is implemented in #880

Brilliant, thank you.


Add new payload types

Description

Type: Feature
ID: 863h1uqj2
Priority: High

As we don't include typedefs in the GameMaker plugin, this task is not applicable.


feat: hashmap based caching

Before this PR, pacquet install was installing packages by "generations": All direct dependencies are installed first, then dependencies of direct dependencies, then dependencies of dependencies of dependencies, and so on. This was not optimal, but it was done to avoid race condition where some packages would be fetched twice should they be invoked in the window of time where its duplicated has begun fetching but not yet saved to local disk.
This PR aims to change this by using an in-memory hashmap cache to keep track of all fetched packages, then re-enable full parallelization.

Could you add a description what this PR is about?

Could you add a description what this PR is about?

done


Bookmark refuses to be deleted

I have an error where a bookmark of Acts 5 or something popped up and now won't delete. I didn't create it and no matter how many times I try to delete it, it won't. When I tried to follow the wiki and delete the entire bookmark database, I can't find its location.
Thanks for any help,

Hi there.
Looks like a bug. I can reproduce the deletion of the bookmark.
But I'm not sure what you mean with "I didn't create it". Who else did?

I didn't create the bookmark It just happened to be there when i opened the program. More than one person uses the computer though so someone might have done it accidentally.
Thank you for helping.


Fix lazy generics in docs

Hey, I noticed that in docs BinaryTreeSchema is parametrized with only input T
that causes type erasure for some types, notable for branded strings, such as:

const NonEmptyStringSchema = pipe(
  string(),
  minLength(1),
  brand('NonEmptyString')
);

type NonEmptyString = InferOutput<typeof NonEmptyStringSchema>;

const FileSystemCommonSchema = object({
  name: NonEmptyStringSchema,
});

type FileSystem = (
  | {
  readonly type: 'directory';
  readonly children: readonly FileSystem[];
}
  | {
  readonly type: 'file';
}
  ) & {
  readonly name: NonEmptyString;
};

const FileSystemDirectorySchema: GenericSchema<FileSystem & { type: 'directory' }> = intersect([
  FileSystemCommonSchema,
  object({
    type: literal('directory'),
    children: array(lazy(() => FileSystemSchema)),
  }),
]);

const FileSystemFileSchema = intersect([
  FileSystemCommonSchema,
  object({
    type: literal('file'),
  }),
]);

const FileSystemSchema = union([
  FileSystemDirectorySchema,
  FileSystemFileSchema,
]);

would cause type error because (I assume) FileSystem's name NonEmptyString being downcasted to string as an input and then assigned to the output as a default <---- this is only an assumption, I just went with my intuition and put GenericSchema<unknown, FileSystem & { type: 'directory' }> there and it works

This looks wrong to me. You get the TS error because your schema does not match the input type of FileSystem & { type: 'directory' }. Assigning unknown is not a good solution as it simply removes type safety.
Also, have a look at this guide. Often it is better to merge objects instead of using intersect.

Thank you for checking it!
It will work when I change NonEmptyString to string though:
const NonEmptyStringSchema = pipe(
  string(),
  minLength(1),
  brand('NonEmptyString')
);

type NonEmptyString = InferOutput<typeof NonEmptyStringSchema>;

const FileSystemCommonSchema = object({
  name: string(),
});

type FileSystem = (
  | {
      readonly type: 'directory';
      readonly children: readonly FileSystem[];
    }
  | {
      readonly type: 'file';
    }
) & {
  readonly name: string;
};

const FileSystemDirectorySchema: GenericSchema<
  FileSystem & { type: 'directory' }
> = intersect([
  FileSystemCommonSchema,
  pipe(
    object({
      type: literal('directory'),
      children: array(lazy(() => FileSystemSchema)),
    }),
    check(
      (v) => new Set(v.children.map((c) => c.name)).size === v.children.length,
      'Expected unique names in the children'
    )
  ),
]);

const FileSystemFileSchema = intersect([
  FileSystemCommonSchema,
  object({
    type: literal('file'),
  }),
]);

const FileSystemSchema = union([
  FileSystemDirectorySchema,
  FileSystemFileSchema,
]);

Notice that I removed unknown here and changed every NonEmptyString to string
It will also work if I simply change unknown to Omit<FileSystem, 'name'> & { type: 'directory', name: string },
To me, the current example work only for types with no transformations; if transformations are involved it could be misleading, assigning input type to output type.
I wonder if it can be improved to work with support for transformed types.

Side note, I digressed to intersect instead of merge because merge loses partialChecks (and I assume checks), not warning me about it. When .entries used, all partialChecks wrappers are silently lost; I don't want to track it manually and choosing between merge and intersect and ideally would like type checked do it for me!
example
const TemporalConcernUnsortedSchema = object({
  createdAt: DatetimeSchema,
  updatedAt: DatetimeSchema,
});

const TemporalConcernSchema = pipe(
  TemporalConcernUnsortedSchema,
  forward(
    partialCheck(
      [['createdAt'], ['updatedAt']],
      (input) => input.createdAt <= input.updatedAt,
      'createdAt must be less or equal than updatedAt'
    ),
    ['updatedAt']
  )
);

const UserSchema = object({
...TemporalConcernSchema.entries // partialCheck code ignored since
});

I wonder if it can be improved to work with support for transformed types.

The first generic of GenericSchema defines the input type and the second one the output type.

Side note, I digressed to intersect instead of merge because merge loses partialChecks...

Yes, that's true. In this case it is necessary to apply the pipeline afterwards. I understand if intersect is easier to work with in this case.


Parameterized onExit handler for templates

Summary
Hello! I am trying to use exit-handlers within a template and was wondering how to pass parameters to the exit-container.
Given an exit-template as follows:
   "name": "send-template-update"
    "inputs":
      "parameters":
      - "name": "podName"
      - "name": "podStatus"
    "container":
      "image": "ubuntu"
      "command":
      - "/bin/bash"
      - "-c"
      "args":
      - "echo \"{{inputs.parameters.podName}}{{inputs.parameters.podStatus}}\""

And a task-template as follows
name: example-template
onExit: send-template-update
...

And an instantiation of that template within a DAG
name: example-task
template: example-template
...

how would I pass the arguments of podName and podStatus to the exit-container.
Motivation
I am trying to automate collection of per-node statuses.

Exithandler doesn't support passing input parameters.  you can use global parameters and here the list of global variables
https://github.com/argoproj/argo/blob/f8053ae379a8244b53a8da6787fe6d9769158cbe/docs/variables.md


What is Aero attachment folder

What is Aero attachment folder?

For others wondering it means that the attachment folder is grayed out in the file tree.


CORE-2299 - Add capability to ignore missing or empty folder with includeAll

CORE-2299 - Add capability to ignore missing or empty folder with includeAll - Add new 'ignoreMissingOrEmptyDirectory' attribute to 'includeAll'.  Default false.

Thanks!
I merged to master rather than 3.3.x due to the xsd change. I updated the change to use the 3.4 XSD.
I also changed the flag to be errorIfMissingOrEmpty with a default of true


💃 Add dnf integration tests

Run the dnf integration tests with every pull request.

Looks like Mr Sander lied to us:
_hawkey.RuntimeException: Failed to resolve: Encountered two streams with matching NSVCA testmodule:master:3020190304173846:a5b0195c:aarch64 but differing content

I have no immediate idea what this could be. I'll have a look later.


Translate intro in the basics section

Done issue #41

@thatdaniboy are you still going to translate this topic?


Add My Password Generator Project

Issue Type: Hey admin! I want to add a Password Generator to your site. I am contributor at GSSoC'22. Please assign this issue to me.

[ ] Bug Report
[x] Feature Request
[ ] Documentaion

Describe the bug

A clear and concise description of what the bug is.

Possible solution
I want to add this here.

Repository Link:- https://github.com/Susmita-Dey/Password-Generator
Live demo:- https://susmita-dey.github.io/Password-Generator/

Screenshots

If applicable, add screenshots to help explain your problem.

create on issue on  https://github.com/arpit456jain/Amazing-Js-Projects

create on issue on https://github.com/arpit456jain/Amazing-Js-Projects

Okay, creating.


Ligth-client concurrency model

Place to discuss "how we want to run" the light-client.
Currently it seems like we use a single thread, and then use a Tokio basic-scheduler to block on results for futures via
https://github.com/interchainio/tendermint-rs/blob/5c3aff6ab1835f0e4db6215c9deaad6a12c10062/tendermint-lite/src/main.rs#L22
Discussion started at https://github.com/interchainio/tendermint-rs/pull/125#pullrequestreview-369307277

ref #172, #171, #173

It looks like a single-threaded tokio runtime is a good idea for now? Then we can use a threaded runtime later if there is a need for parallellism somewhere, and spawn additional tasks, as opposed to using await in the "current task" everywhere?

Sounds reasonable to me. There might be some need for parallellism in what requests headers and what verifies those. These could be separate tasks.

I have some suggestions coming by weeks end sorry for the delay! 🙏

It seems like there's a lot of potential for parallelization. If you'd like some inspiration, check out how Zebrad implements batch verification for Ed25519 signatures.
Here's a plot of the performance improvements they saw:

For the initial version of this ala https://github.com/informalsystems/tendermint-rs/milestone/1 we will do everything synchronously and revisit concurrency and batch verification down the road.

Will close for now and create issues for specific component concurrency concerns.


Design "my bookings" after login (homepage)

The homepage contains all the bookings (current and past) so let's make them nice... :)
All bookings are already with offline support.

I'll take this one

Ok, I am reassigning this issue to me. It's taking too long.

Alright, GraphQL API fixed, but not merged yet. Waiting for:

[ ] chatbot/kiwi-chatbot-infrastructure!528
[ ] chatbot/kiwi-chatbot-infrastructure!522


Replace smart quote character U+2019 with straight quote

The apostrophe in both of these messages is the unicode "right single quote" character.
This replaces them with a more standard apostrophe.
Signed-off-by: Pete Lumbis<EMAIL_ADDRESS>

Thanks!


Minimum players filter

Hi there! First off, absolutely excellent job on the website I can't believe I lived so long without having known about it haha.
Would it be possible to include a minimum players/reviews filter for the sales page? There are tons of games, but I (and, I'm hoping others) only like to play the ones that have been tested thoroughly by gamers before me haha, so only popular ones.
Thanks!

Filtering by amount of players only works for popular multiplayer games. Singleplayer games will have a fast drop off of players making this somewhat useless.
Filtering by rating and amount of reviews should accomplish this nicely.

Just saw the feature live, thank you so much!


Write an optional request logging interceptor that spits out httpd-format access logs

Lots of tools process apache httpd2 access logs, and it's handy for debugging. Would be useful to have an interceptor that emits this log file.

from @ryanhall07 :
I don’t think the default should be log everything truncated
something like log all errors (up to a rate limiter) and maybe a random sample of success requests (edited)
also it should be easy to quickly enable/disable these features in real time
if i’m actively debugging an issue
and “error” should really be anything not ideal, like an error code or a long request
eventually we’d probably want all those configs/overrides backed by feature flags
--
something like default % sample success requests, and rate limit errors, then have overrides per endpoint

/cc @jmuia


Postgres Wire Protocol: Add section about Simple Query protocol mode

@joemoe Did you have a chance to take a look?

retest this please


Lite version Timeline for Catalina

📒 Description
As we agreed that we would implement the Lite version of Timeline in macOS 10.15 until we officially supported the Screen Recording permission.
This PR introduces the Lite version:

OS 10.15+ -> Get Window title and no things more
<= OS 10.14 -> Work as usual.

🕶 Types of changes
Breaking change (fix or feature that would cause existing functionality to change)

Please make sure the OS <= 10.14 will work as usual.

🤯 List of changes

[x] Logic to detect OS version on library
[x] If it's 10.15+, set title as filename

👫 Relationships
Closes #3427
Closes #3422
🔎 Review hints
Check on OS 10.15 Catalina

Start the staging app and turn on Timeline Recording
Play around the app for 15 mins.
Check the Staging local database (/Users//Library/Application Support/Kopsik/development/kopsik.db) to make sure the title is same as the filename

Open Timeline in Web, make sure it presents correctly

Check on OS 10.14

Start the staging app and turn on Timeline Recording
Play around the app for 15 mins.
Check the Staging local database (/Users//Library/Application Support/Kopsik/development/kopsik.db) to make sure the app is able to record the WIndow title

Open Timeline Web, and see if it is as usual

I think for the timeline view in the desktop app we should take account if the title and filename are the same. In that case, we only show it once not twice. From the web timeline ui screenshot, it would only show the darker text of Safari and would not show the lighter Safari text duplicate under it.

Thank for the valid point. I'm looking on it 👍


DWARFv5 supplementary file support

Most DWARFv5 files will parse perfectly fine using our current parser, with the notable exception of supplementary files (section 7.3.6 in the spec). The TL;DR on supplementary files are that they're shared objects for debug information; instead of every binary containing duplicate definitions for types like FILE (or even just the builtin types), the theory is that the cstdio could have its own supplementary file (containing its own type definitions) which our binary then references.
This will require a significant refactor in our plugin, but the version of it in my head will also be faster and more accurate in the end; We need to treat DWARF more like source code and do proper AST stuff to resolve types -- since that'll require completely virtualizing the DWARF graph, we can take the opportunity to add additional verification/optimization on the graph (and eliminate the incredible amount of duplicated information while we're loading it)...like a compiler. I realize it sounds overkill, but in hindsight it should've been obvious that a portable, platform/language-agnostic debug info system is effectively LLVM but exclusively for type and function info...except for the fact that it tries really hard to pretend that it's not.
It's also worth mentioning that while gimli supports DWARFv5, including supplementary files, it currently only supports a single supplementary file at a time. Even if they were to start supporting multiple supplementary files, I believe my described refactor would still result in a net improvement in speed and accuracy. It will also allow us to support Components.
I will note here, in protest to the powers the be, that the proclaimed goals to "reduce link time and executable size" with the introduction of supplementary files has not yet made its way into any files I've seen. In fact, I've found that using supplementary files tends to increase the number of stubs (which tend to contain more information than they need to), resulting in more duplicated information.

Support for loading supplementary files is added in 5.2.5753-dev


fix: the moduleIds should be deterministic if the env is production and format is mf

Summary
When format is mf , and env is production , the moduleIds should be deterministic for better cache and smaller size.
And the named moduleIds may cause domain name leaked if the build env path has sensitive infomation
Related Links

Checklist

[X] Tests updated (or not required).
[X] Documentation updated (or not required).

Can we add this configuration here instead of another tools function? https://github.com/web-infra-dev/rslib/blob/5c8a73b9f1a77c3e9a2213060700a35733d60c0d/packages/core/src/config.ts#L583


Multi-User data access with different permissions than owner

Is this feature request related to a new or existing Amplify category?
auth, api
Is this related to another service?
No response
Describe the feature you'd like to request
According to the Auth documentation it is possible to have a list of users as owner of a data set in the api schema. This is a pretty neat feature but not working for my use case because I want the owner field to be fix and the owner to have more permission than a set of users. I want to do something like this:
type Todo @model @auth(rules: [
  { allow: owner },
  { allow: customGroups, groupField: 'readers', operations: [read] },
]) {
  content: String
  readers: [String]
  owner: String @auth(rules: [{ allow: owner, operations: [read, delete] }]))
}

The owner should be allowed to do everything except updating the owner field. The readers should be allowed to see the Todo but they shouldn't be allowed to do anything else with it. I want the owner to be able to specify the set of users who can see the Todo.
Describe the solution you'd like
As described above, something like this could work:
type Todo @model @auth(rules: [
  { allow: owner },
  { allow: customGroups, groupField: 'readers', operations: [read] },
]) {
  content: String
  readers: [String]
  owner: String @auth(rules: [{ allow: owner, operations: [read, delete] }]))
}

Describe alternatives you've considered
It might also be possible to implement this using a custom resolver or a lambda function but I'd rather want to avoid this because of the additional resources that would need to be created manually and I think this might be a rather common use case regarding data sovereignty laws, bacause you can't create a cognito group for every subset of users you want to allow because of the Cognito Quotas
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request

Would this feature include a breaking change?

[ ] ⚠ This feature might incur a breaking change

Hey @jo2 :wave: thanks for raising this! I'm going to transfer this over to our API repo for tracking purposes. For this use case I believe you will want to use an additional owner auth rule as described in the linked docs Multi-user data access
type Todo @model @auth(rules: [
  { allow: owner },
  { allow: owner, ownerField: 'readers', operations: [read] },
]) {
  content: String
  readers: [String]
  owner: String @auth(rules: [{ allow: owner, operations: [read, delete] }]))
}


added blog

added in nav blog page
BlogEntry component
added images to use as banners
Added gray-matter package to read the YAML of markdown, separe and don't render that part in the marked page

apparently i had to remove that "cm" to be able to use fmt because it told me it couldn't find any folder or file named like that. just in case it is a windows/unix incompatibility, i returned it


Chrometana extension infected

Your extension on the Chrome store is infected! Please fix ASAP!

Duplicated #27

Hey guys! Fun fact.
I am in contact with Google and will be putting up an official press release tonight. This is absolutely maddening and I'm so sorry guys.

Update's been shipped, check #27 for more info


Remove some redundant code line from uart.c

This PR removes some lines of code which are redundant. Just to save some valuable bytes in flash...

Hi @hwmaier,
Thanks for considering our precious flash size, and thanks for sending this change.
Unfortunately these changes don't make a significant difference to compiled size. In debug mode, the difference is 4 bytes (because of trying to keep the compiled code in line with the source for debugging, I think). In release mode, the difference is zero. Modern C compilers are good at recognising things like redundant writes and identical branches, and merging them.
So the question becomes, do the changes improve readability? The UART driver can certainly benefit from more readability! The first change, I think, does not - having the explicit case there (and no fallthroughs) is easier to see what's going on. The second change does though.
I've cherry-picked the second change into our internal review & merge queue.
Angus

Cherry-picked in part, thanks!


Write out packed pp files

Enhance Iris so that you can write out packed PP files.

Closing due to inactivity.


Change the screenshot of PBI Desktop download!

The screenshot showing which version of Desktop to select is very misleading because it has an arrow pointing to the 64-bit version when what you are recommending on this solution is the 32-bit version. People might not read all your instructions including the disclaimer that the screenshot shows the 64-bit version is selected, but that they should opt for the 32-bit.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 680485c7-5823-2c7f-df5d-b8c8a8205759
Version Independent ID: 5a217642-7723-6980-28ae-e26559c24201
Content: Resolve Access and .XLS import issues in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-access-database-errors.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hello @sjlarkin and thanks for your post. Note that in the article, it directs you to use the version that matches your version of Desktop. Customers can experience this in either direction (32 vs 64, and 64 vs 32). The text above also clearly explains which is selected, and how to make the correct selection.
Sorry for any confusion.