Docs should explain how [patch] behaves for sub-dependencies (dependency of a dependency)

Problem
My current project has a dependency A. A has a further dependency on B. If I add the following to my Cargo.toml:
[patch.crates-io]
B = { git = 'https://...' }

It's unclear from the docs what the behavior is going to be. In my testing, If B @ the patch is a major version ahead of version in A's Cargo.toml, B continues to be fetched from crates.io rather than from my specified URL. That doesn't feel like the right behavior at all but at the very least, it should be documented.
Steps
No response
Possible Solution(s)

Update docs describing expected behavior for sub-dependencies and when how it interacts with the multiple-major-versions condition.
Refuse to build if the overridden path doesn't include a version that's compatible with the main dependency (A in this case)

Notes
No response
Version
% cargo version --verbose
cargo 1.60.0-nightly (358e79fe5 2022-01-04)
release: 1.60.0-nightly
commit-hash: 358e79fe56fe374649275ca7aebaafd57ade0e8d
commit-date: 2022-01-04
host: aarch64-apple-darwin
libgit2: 1.3.0 (sys:0.13.23 vendored)
libcurl: 7.77.0 (sys:0.4.51+curl-7.80.0 system ssl:(SecureTransport) LibreSSL/2.8.3)
os: Mac OS 12.1.0 [64-bit]

It's also worth noting that [patch] applies transitively. Let's say you use my-library...

In this paragraph, it mentions that [patch] applies on transitive dependencies as well, but yeah, the doc is always welcome to polish!
By the way, cargo should emit warnings for unused patches. Have you seen this warning in your build?
warning: Patch `foo v0.1.0 (/home/xxx/xxx)` was not used in the crate graph.
Check that the patched package version and available features are compatible
with the dependency requirements. If the patch has a different version from
what is locked in the Cargo.lock file, run `cargo update` to use the new
version. This may also occur with an optional dependency that is not enabled."

On the tangential note, looks like there's no good way to solve the underlying problem I'm encountering here: the ability to override a transitive dependency from the root crate. Is there?

As the warning says, the patch should meet the original dependency requirement in Cargo.toml. If you can access or fork the patch package, it might work that adjusting pacakge.version in accordingly in Cargo.toml

Of relevance: https://stackoverflow.com/questions/72240671/patch-a-transient-git-dependency-to-a-specific-rev


Handle recovery bytes starting at 0

Close #18
Depends on https://github.com/spruceid/siwe/pull/95

How exactly this fixes it?
tnks guys.

How exactly this fixes it?

There is a discrepancy between the standard ECDSA recovery byte format and Ethereum’s -- basically it tries to encode more data in it, whilst keeping the same functionality. I believe they can include the chain ID now, so we’ll probably have to revisit this in the future.


Authentication failures using Spanner with appengine:run locally for App Engine Standard J8

Auth errors using both plugins in
mvn appengine:run : http://mvnrepository.com/artifact/com.google.cloud.tools/appengine-maven-plugin/1.3.1
mvn appengine:devserver : https://mvnrepository.com/artifact/com.google.appengine/appengine-maven-plugin/1.9.53
IAM Errors on creating, querying :
- com.google.cloud.spanner.SpannerException: PERMISSION_DENIED: io.grpc.StatusRuntimeException: PERMISSION_DENIED: Access denied. Missing IAM permission: spanner.databases.create.

com.google.cloud.spanner.SpannerException: PERMISSION_DENIED: io.grpc.StatusRuntimeException: PERMISSION_DENIED: Access denied. Missing IAM permission: spanner.sessions.create.

I've verified within the servlet that GOOGLE_APPLICATION_CREDENTIALS exists and is valid.
Same code works fine when using the Jetty Plugin mvn jetty:run

@ludoch @lesv FYI

@jabubake is there any chance this is related to https://github.com/GoogleCloudPlatform/google-cloud-java/issues/2209 ?

I tested some spanner sample code on GAE 8 today and with right IAM it should work now.


Upgrade spring-cloud-dependencies version to 2021.0.0

Changelog:

Upgrade spring-cloud-dependencies version to 2021.0.0
Add new version tags in dependencies file

/azp run java - spring - tests

Hi @chenrujun , please help review, thanks!

@moarychan
Temporarily fix compilation errors, this class will be removed in other PR.

@moarychan , could you please paste the link of that PR?

https://github.com/Azure/azure-sdk-for-java/pull/26170


Possible bug in self._collected_trainable_weights with effects on GAN training

Hi,
I was going over the source code to try to understand why changing a model's Trainable property failed to update the model parameters after calling fit() (even though they would appear in model.trainable_weights). I found that the property self._collected_trainable_weights is set only once in Model's compile:
https://github.com/fchollet/keras/blob/master/keras/engine/training.py#L995
and never updates afterwards, even if self.trainable_weights changes.
Then, when we call fit() for the first time, _make_train_function is called, which uses the old (and out of date) self._collected_trainable_weights.
This might be caused from a code addition of trainable_weights.sort in compile, which makes a local variable instead of setting self._collected_trainable_weights to be a reference to  self.trainable_weights.
Is this intended behavior or a bug with updating model Trainable flag?
Many thanks,
Yarin

This is intended: you always need to compile your model if you make any changes to it. A model is considered "finalized" at compile time and any further changes will not be taken into account by the forward/backward functions until compile is called again.
The fact that the backward/forward functions are generated one the fly when they are required is actually an implementation detail, due to performance reasons. They were originally generated in compile and it is reasonable to still think of them that way.

In short, this is due to the fact that Keras models are graph. There's one point where the graph must be finalized, and that's compiled. This will no longer be the case in the imperative version of Keras, where models will be editable at any time.

Thanks!


Cancel return request

In "order return request details" page, there is "cancel" button, so that customer can cancel this return request.

Tested on dev10. The only issue right now is that the status remains "cancelling". Therefore, the user cannot resubmit a return request for the same item after that. Bill is aware of this, and is looking into it.


Limitation Clarification: A corrupted dataset may require deletion and cascade delete all dependent reports

Just a warning that although you may not plan to delete a dataset, if something happens to the root dataset you may be forced to delete it which will essentially force you to rebuild your entire workspace/linked reports/etc. We were very excited to create a shared dataset and link various reports to the shared power bi dataset service instance.  However, one day our scheduled refresh started failing and the error was tracked down to a data table field name change.  The dataset was successfully refreshing locally in power bi desktop but the instance in the power bi service was somehow corrupted and the only fix was to delete the cloud service dataset....which cascade deletes all your dependencies, etc. and means you need to touch every dependent report, re-connect, re-publish...This is more of a warning to beware getting carried away with your linked reports, it was very frustrating for us to have to rebuild everything.

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

ID: a1cce347-2064-d2b3-d7dd-008ecdb36013
Version Independent ID: 6abb84b5-1657-c8b1-62ef-641883c0e4b9
Content: Connect to datasets in the Power BI service from Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-report-lifecycle-datasets.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for the feedback @tx5westmt, and sorry to hear about the issues you faced with the dataset and the dependent reports. Given that this is product feedback rather than doc feedback, I'm going to close this out. It will still show up at the bottom of the doc page under closed feedback.


pod file: works with 5.5.1, fails with 6.0.1-rc.1

when including cocoa pods with existing swift file.
my pod file:
use_frameworks!
pod 'RNSVG', :path => '../js/node_modules/react-native-svg'
pod 'React', :path => '../js/node_modules/react-native', :subspecs => [
'Core',
'CxxBridge', # Include this for RN >= 0.47
'DevSupport', # Include this to enable In-App Devmenu if RN >= 0.43
'RCTText',
'RCTNetwork',
'RCTAnimation',
'RCTImage',
'RCTWebSocket', # needed for debugging
# Add any other subspecs you want to use in your project
]
pod "yoga", :path => "../js/node_modules/react-native/ReactCommon/yoga"
pod 'DoubleConversion', :podspec => '../js/node_modules/react-native/third-party-podspecs/DoubleConversion.podspec'
pod 'GLog', :podspec => '../js/node_modules/react-native/third-party-podspecs/GLog.podspec'
pod 'Folly', :podspec => '../js/node_modules/react-native/third-party-podspecs/Folly.podspec'

The latest version doesn't have the duplicate symbol issue anymore, I recommend upgrading :)


Lockscreen shortcuts appears on notification center

Hi and thank you for this tweak!
I have spotted a bug for lockscreen shortcuts (flashlight and camera). They appears in the Notification Center when you scroll it from the top of the iPhone.
This happens for me on iPhone 8 iOS 13.5

Sincerely

I may be misunderstanding your bug report, but the lockscreen toggles are supposed to be there in the notification center. Them being there is not a bug.


Improve proxy support

Hi, I ran into some problems when using this package in Kubernetes using a HTTP proxy that requires authentication, so I made some improvements to the HTTP proxy implementation.
Improvements:

Added optional username and password proxy options. When specified this will set the Proxy-Authorization header with basic authentication.
var options = {
  token: {
    key: "path/to/APNsAuthKey_XXXXXXXXXX.p8",
    keyId: "key-id",
    teamId: "developer-team-id"
  },
  proxy: {
    host: "<IP_ADDRESS>",
    port: 8080,
    username: "secretUsername",
    password: "secretPassword"
  },
  production: false
};

var apnProvider = new apn.Provider(options);

Added support for retrieving HTTP proxy config from environment variables, instead of needing to specify it in options.
This is useful in environments where HTTP proxy is configured automatically through Kubernetes for example. This also follows a normal convention across tools and other request libraries, and is normally the expected behaviour (reference).
When one of the following environment variables is set we're using this value to connect (in this order):

apn_proxy
npm_config_[http/https]_proxy (npm config variable for http or https proxy)
[http/https]_proxy
all_proxy
npm_config_proxy (npm config variable for general proxy)
proxy

To completely disable proxies and ignore environment variables set { proxy: false }. This is similar to how axios is handling proxy config.

Added support for the no_proxy environment variable that makes sure urls that shouldn't be proxied isn't.  (Reference).

@florianreinhart I'm hoping you could review my pull request when you have a moment. If possible, I'd love to see this included in an upcoming release soon. :)


Wai Ha Lee: Watch handonboard.com

Wai Ha Lee requests the watch of the watch_keyword handonboard\.com. See the MS search here and the Stack Exchange search here.

handonboard\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Coverage remained the same at 64.618% when pulling 4784da0f0215798e22f698743569083a86a80dea on auto-blacklist-1539860820 into 96151a6da2c4f5b16b32adec529a05e8bd16cf9c on master.

!!/approve


alerts from grafana

[ ] pods not running
[ ] increase in resource usage

Apparently learning PromQL well enough to make good alerts is quite difficult... So far I have alerts set up for:

If a pod fails
If ingest-core memory usage is coming close to the maximum
If we are coming close to the number of pods allowed on a node (110), threshold for alert set at 90

Hi @jacobwindsor is this done?

Yes but there is a PR still pending https://github.com/ebi-ait/ingest-kube-deployment/pull/25


Add datasource in multiple servers.

https://bugzilla.redhat.com/show_bug.cgi?id=1383606
@miq-bot add_label providers/hawkular wip

@miq-bot add_label providers/hawkular, wip

Travis failures does not seem related to this PR
  1) ManageIQ::Providers::Openstack::InfraManager::Refresher will verify maintenance mode
     Failure/Error: yield if block_given?

This works well.
@miq-bot assign @bronaghs

@miq-bot assign @dclarizio
CC @h-kataria @martinpovolny

@rubenvp8510 can you add some screenshots to the first comment please?  It's hard for me to see what's actually being added here. Thx, Dan
@martinpovolny @h-kataria please review

@rubenvp8510 can you add some screenshots to the first comment please? It's hard for me to see what's actually being added here. Thx, Dan

bump ^^

@dclarizio - Are the screenshots in the first comment what you are looking for? Ready for review?

@rubenvp8510 The message on the first screenshot doesn't seem to make sense:
Not adding new datasource to Hawkular server the provider itself
Can you fix that so it means something? 😄
Also, since Datasource and Provider are primary objects, please capitalize them in the messages.

Please re-create PR in https://github.com/ManageIQ/manageiq-ui-classic


Update scala-js-cli to support Scala.js 1.13.0

Seems we have to build the Linux binaries of scala-js-cli from docker, like we do for Scala CLI, to address the various glibc-related errors seen on the CI.

Suspended by https://github.com/VirtusLab/scala-cli/pull/1912.


fix(product landing page): allow header or children to be fixed

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #46021

This PR should allow either the header or its children to be fixed which will also make our example project code pass again. As an aside, the test we have in place now doesn't test if the nav is fixed at all, it only looks at the initial position.
Now technically allowing either the header or its children to be fixed does allow for neither the header nor the nav to be fixed and still pass the test. Say if you have a logo inside the header and that is the only element you set to position fixed, the test would still pass. I'm not sure if that is really an edge case we want to worry about and it's the same in the test script we have been using forever.
I change the looping code compared to the way it is done in the test script let me know if you see any issues with it.

Not sure why the Firefox test keeps failing. And have re-run it twice now.

Not your fault @lasjorg - we added a fix for something, but it doesn't seem to quite fix it. I just reverted that.

And clicked the update branch - they should pass now.

@moT01 It's still failing it seems.


Proposal. Rough draft of lala; deprecate old draft.

Introduced binder space and value space; three computation with their notation and underlying data structure; a pattern language; a type system; and a draft idea of a package manager.

LGTM.


Gitea: integration with Draw.io

Feature Description
Hi there!
I have an idea for integrating Gitea services and Draw.io. In my opinion, this can be a pretty useful thing!
"The user attaches the .drawio file to the messages in the issues. When clicking on the attached file, the user is redirected to the draw.io, at the same time, the contents of the file [XML in base64 or the like] are sent to the payload of the POST request. Draw.io receiving the file content and expands it into a schema. Apparently, the issue is complex and requires collaboration with the guys from Gitea. Let's try?" (с)
https://github.com/jgraph/drawio/issues/4128
Screenshots
No response

I wonder if this could be achieved with an external renderer via the config so nothing new needs to be added to Gitea

I wonder if this could be achieved with an external renderer via the config so nothing new needs to be added to Gitea

I don't think it's possible to be as an external renderer. Because it needs to read/save the files. It should be more like the markdown editor of course for .drawio files but .md files.

According to the statement on their README.

draw.io is not suitable as a framework for building other products from. For this try either Tldraw or Excalidraw.

Tldraw is non-commercial license. Looks like https://github.com/excalidraw/excalidraw is the only possible solution.

As I understand it, the only official interface is the full-blown editor UI. What we need would be a viewer-only UI. Maybe there are third-party made renderes available somewhere.

Seems confluence has done some integration with a viewer.min.js file from the draw.io repo, so it may be possible:
https://www.drawio.com/doc/faq/configure-javascript-viewer-drawio-confluence-server

gitlab supports viewing, editing draw.io files. seems to work offline from draw.io some links:

some code in master: https://gitlab.com/gitlab-org/gitlab/-/tree/master/app/assets/javascripts/drawio?ref_type=heads
and merge request that looks like mvp: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/86206/diffs#7981eac67e1980a41d95e5bfc2ccea7350ce53ae

i think editing is a very nice feature.

Seems they are embedding a iframe from https://embed.diagrams.net, not an ideal solution but could at least work with self-hosted renderer if that URL is made configurable.

Why not just use drawio JS as lib?

Why not just use drawio JS as lib?

There is no lib as far as I know, it's only available integrated with their backend.


V0.5 brehmer final

Coverage increased (+0.1%) to 85.444% when pulling d5a3ace4661c8490bd7d3dbc777f97d43dc477d7 on V0.5_Brehmer_FINAL into 065348b37ff657ad4bd766192decee0e3d28dd8b on V0.5_FINAL_WEIJIE.


Support for OpenSSL 1.1.1d

This PR adds support for OpenSSL 1.1.1d. It should fix part of #1087.

I tested this PR locally. In addition to some build warnings I also see the tests get stuck in a similar place as bugfix/pki-selection/1.10.x
test output
test 6
      Start  6: SimpleCommunicationSecureBestEffort
6: Test command: /usr/bin/python3 "/home/steven/osrf/Fast-RTPS/ws/build/fastrtps/test/communication/simple_communication.py"
6: Environment variables:
6:  SIMPLE_COMMUNICATION_PUBLISHER_BIN=/home/steven/osrf/Fast-RTPS/ws/build/fastrtps/test/communication/SimpleCommunicationPublisher
6:  SIMPLE_COMMUNICATION_SUBSCRIBER_BIN=/home/steven/osrf/Fast-RTPS/ws/build/fastrtps/test/communication/SimpleCommunicationSubscriber
6:  XML_FILE_PUB=simple_secure_besteffort_pub.xml
6:  XML_FILE_SUB=simple_secure_besteffort_sub.xml
6:  EXTRA_PUB_ARG=--wait 2
6: Test timeout computed to be: 1500
6: Subscriber participant  discovered participant a6.b1.a0.7e.9c.49.a4.36.78.eb.58.4f|0.0.1.c1
6: Subscriber participant  discovered participant a6.b1.a0.7e.9c.49.95.af.c8.f3.98.22|0.0.1.c1
6: Subscriber participant  discovered participant eb.ba.3f.10.a7.26.9a.d<IP_ADDRESS>e.58|0.0.1.c1
6: Subscriber participant  discovered participant eb.ba.3f.10.a7.26.9a.d<IP_ADDRESS>e.58|0.0.1.c1
6: Publisher participant  discovered participant a6.b1.a0.7e.9c.49.95.af.c8.f3.98.22|0.0.1.c1
6: Publisher participant  discovered participant a6.b1.a0.7e.9c.49.a4.36.78.eb.58.4f|0.0.1.c1
6: Subscriber participant  authorized participant eb.ba.3f.10.a7.26.9a.d<IP_ADDRESS>e.58|0.0.1.c1
6: Publisher participant eb.ba.3f.10.a7.26.9a.d<IP_ADDRESS>e.58|0.0.1.c1 authorized participant a6.b1.a0.7e.9c.49.a4.36.78.eb.58.4f|0.0.1.c1
6: Subscriber participant  authorized participant eb.ba.3f.10.a7.26.9a.d<IP_ADDRESS>e.58|0.0.1.c1
6: Publisher participant eb.ba.3f.10.a7.26.9a.d<IP_ADDRESS>e.58|0.0.1.c1 authorized participant a6.b1.a0.7e.9c.49.95.af.c8.f3.98.22|0.0.1.c1
6: Subscriber participant  authorized participant a6.b1.a0.7e.9c.49.a4.36.78.eb.58.4f|0.0.1.c1
6: Subscriber participant  authorized participant a6.b1.a0.7e.9c.49.95.af.c8.f3.98.22|0.0.1.c1

Build status:

Linux
Mac
Windows


gs1.1 adaptive gossip

Implements https://github.com/libp2p/specs/blob/master/pubsub/gossipsub/gossipsub-v1.1.md#adaptive-gossip-dissemination

Closely follows the go-libp2p-pubsub implementation
Adaptive gossip dissemination in _emitGossip
fixes up the fanout peer handling in heartbeat

includes bugfix in the heartbeat where we were deleting fanout peers who are still in the topic (yikes)

Add hasGossipProtocol utility function, replace where appropriate

Codecov Report

Merging #94 into gsv1.1 will increase coverage by 1.81%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           gsv1.1      #94      +/-   ##
==========================================
+ Coverage   80.00%   81.81%   +1.81%
==========================================
  Files           1        1
  Lines          10       11       +1
==========================================
+ Hits            8        9       +1
  Misses          2        2

Impacted Files
Coverage Δ

test/utils/index.js
81.81% <0.00%> (+1.81%)
:arrow_up:

Continue to review full report at Codecov.

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


Separate simulation to stand alone library.

It looks like a good idea to extract simulation to stand alone library which one can use also in other libraries and tools. We should discuss this.

Solved by PR #30


docs: style resetting

Describe the problem
I can't find style resetting guidance in docs
Describe the proposed solution
Maybe, you can add this in docs: https://github.com/unocss/unocss#style-resetting
Alternatives considered

Importance
i cannot use Stylify CSS without it

Hi,
Stylify doesn't have any own reset and preset by default, so you can use any CSS reset library you want (as you already mentioned in the description).
I recommend using for example:

Those that are listed in the UNO docs (provided by you in the description)
https://csstools.github.io/sanitize.css/
https://github.com/sindresorhus/modern-normalize
https://necolas.github.io/normalize.css/

I will add this into the docs. I havent though, that somebody may want to know this. Thanks!


Fail taskrun when results validation fails

Changes
This commit adds the ReasonFailedValidation into taskrun status and fail
the taskrun when results validation fails. Before this commit users can
only see error messages in logs but the taskrun are completed like no
errors happen.
/kind bug

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

@ywluogg

/retest

/retest

/test

/test

/retest

/retest

/lgtm


[Marker.io] background height is little less as compared to figma

Steps to reproduce:
1)go to start gifting today section
2)notice the background
3)compare its height with figma
Actual result:it is slightly less as compared to figma
Expected result:if current behaviour is correct, we can ignore this

Reported by: vinod<EMAIL_ADDRESS>- Contact via Marker.io
Source URL: https://na13-prezzee.webflow.io/contact
Issue details: Open in Marker.io
Device typedesktopBrowserChrome <IP_ADDRESS>Screen Size1366 x 768OSWindows 10 (10.0.0)Viewport Size1366 x 641Pixel Ratio@1x

🤖 Marker.io bot - This issue was marked as ✅ Resolved in Marker.io


request: make article content collapsed by default or give the choice

Hi,
Thank you very much for adding article content sync into a child block!
But is it possible to make the content and highlight child blocks collapsed by default? Or togglable via a UI checkbox?
Have a nice day!

Hey @thiswillbeyourgithub, yeah this is a good one.
I will make both blocks collapsed by default ~


http-interop/http-middleware is abandoned

is there any plan for migrating to http-interop/http-server-middleware ?

@agosong  See #528 and our thread on the forum:
https://discourse.zendframework.com/t/psr-15-compatibility-issues-and-proposal/378


Create card component

Added cards in a masonry layout (fixed column widths but variable row widths)
Has pinning functionality (resides in the Post component state for now)

TODO: Have it update actual data

Renders text differently from Youtube videos

TODO: When clicking on Youtube thumbnail, open a react-youtube component (perhaps as a popup)
TODO: Account for github posts

Added SCSS

TODO: Once @roeeyn 's Github login gets merged, I will need to migrate the posts to Home.js or a separate container component
Part of issue #8

LGTM!


Path grouping strategy

Users can write function to generate path grouping.
For example, I can write path with /myPath?queryParam1?queryParam2, so I will not have SwaggerGeneratorException (Conflicting method/path combination x for actions (...)) when mapping old .NET Framework API with Route("/api/[controller]").
Maybe related to https://github.com/domaindrivendev/Swashbuckle.AspNetCore/issues/2354, but I don't know if ApiDescription has information about ConsumesAttribute.

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

Project coverage is 83.23%. Comparing base (8ea0461) to head (e27a6c9).

Files with missing lines
Patch %
Lines

...erGen/SwaggerGenerator/ApiDescriptionExtensions.cs
0.00%
2 Missing :warning:

...ggerGenerator/ApiParameterDescriptionExtensions.cs
75.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3152      +/-   ##
==========================================
- Coverage   83.25%   83.23%   -0.03%
==========================================
  Files          76       76
  Lines        3142     3144       +2
  Branches      526      526
==========================================
+ Hits         2616     2617       +1
- Misses        526      527       +1

Flag
Coverage Δ

Linux
83.23% <70.00%> (-0.03%)
:arrow_down:

Windows
?

macOS
83.23% <70.00%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Wait. I will change exception message

I was just about to enable auto-merge on this - what are you changing?


Resolving label height clipping in larger text mode for TableViewCell

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
Binary change:

File
Before
After
Delta

libFluentUI.a
40,446,736 bytes
40,447,720 bytes
+984 bytes

TableViewCell.o
1,433,712 bytes
1,434,696 bytes
+984 bytes

When an NSAttributedString (or even a normal String) has no specified font, the system font is used as default. There was a specific scenario where an NSAttributedString without a .font attribute would cause the cell's label to be clipped in larger text mode. It seems that the boundingRect does not redraw when the system font adjusts to the larger text. This change will take the max height of our .preferredSize() (used for strings) and .preferredLabelSize (used for attributed strings) methods to accommodate for this scenario.
Also removed some outdated comments in .preferredLabelSize().
Verification

Before
After

Pull request checklist
This PR has considered:

[ ] Light and Dark appearances
[ ] iOS supported versions (all major versions greater than or equal current target deployment version)
[x] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[ ] iPad Pointer interaction
[ ] SwiftUI consumption (validation or new demo scenarios needed)
[ ] Objective-C exposure (provide it only if needed)

Microsoft Reviewers: Open in CodeFlow

should be bring this changes to .10 branch?


Center welcome picture and don't repeat it

usually the (great) Image Of The Day pictures have their best part in the middle: let's show it in the dashboard welcome page (and avoid repeating it).
===
Turns this:

Into this:

Nice, this has been bugging me forever

Yes, me too I spent a lot of nights awake thinking about this huge issue :wink:


Another valid modules fails validation

test.wasm.gz
(module
  (type (func))
  (type (func (result f64 i32)))
  (func (type 1)
    loop
      br 0
      call 0
      br_if 0
      i32.trunc_f64_u
      unreachable
    end
    unreachable
  )
)

Haven't started investigating yet, but it looks pretty similar to the last one: unreachable code (following the unconditional br) and a br_if instruction.
cc @yurydelendik

I cannot reproduce atm. It requires --enable-multi-value with "wasm-validate-rs" and looks like it is passing in wabt's "wasm-validate".

Ah woops, I translated the example to wat by hand wrong, here is the correct stuff:
test.wasm.gz
(module
  (type (;0;) (func))
  (type (;1;) (func (result f64 i32)))
  (func (;0;) (type 1) (result f64 i32)
    loop (result f64 i32)  ;; label = @1
      br 0 (;@1;)
      call 0
      br_if 0 (;@1;)
      i32.trunc_f64_u
      unreachable
    end
    unreachable))

I get
Error: stack operand type mismatch (at offset 84)

Sorry for uploading the wrong wat!

(with --enable-multi-value)


destructuring-assignment in class properties

First I'd like to just say that I like the new destructuring-assignment rule because I think it enforces a lot more clarity and explicitness. However I'm having problems with it throwing errors inside of class properties.
class MyComponent extends React.Component {
  state = {
    value: this.props.defaultValue,
  };

  ...
}

IMO this should be an exception and be totally valid or at least have the option to exclude it just from this use-case. I know that I could move it inside a constructor and destructure it there but I'd rather not do that.

Duplicate of #1800.
An option would be great.

This isn't a duplicate, #1800 is about it erroring in files that have nothing to do with react. I have the same issue, complains about destructuring in class properties

Fair point; I'll reopen.


ARROW-12959: [C++] Option for is_null(NaN) to evaluate to true

Add Python tests that make use of the nan_is_null option, see https://github.com/apache/arrow/blob/master/python/pyarrow/tests/test_compute.py#L1302
import math
arr = pa.array([1, 2, 3, None, math.nan])
arr.is_null()                  # expected = pa.array([False, False, False, True, False])
arr.is_null(nan_is_null=True)  # expected = pa.array([False, False, False, True, True])

@ianmcook  could you provide me some help for R bindings in order to use this new option.

@Christian8491 could you please confirm that it is safe to set nan_is_null to true when the datum has a non-float type, and it will have no effect or performance cost in that case?

@Christian8491 could you please confirm that it is safe to set nan_is_null to true when the datum has a non-float type, and it will have no effect or performance cost in that case?

Yes @ianmcook as the logic demands a first check for floating type cc @edponce

I pushed 9a4f39a to update the R bindings to use this option. Thanks again @Christian8491 for doing this!


NPE in OidcAuthenticationContextWebflowEventEventResolver

Working with 5.1.0 SNAPSHOT built from latest in master.
The exception is raised when logging into a new service with SSO and entering credentials are bypassed.
line 61 in OidcAuthenticationContextWebflowEventEventResolver.java attempts to get the FlowExecutionURL.  With this being a SSO login no FlowExecution key is present when this is executed.
The behavior is that the login page is presented after the exception occurs.
I fixed the issue on my end for now by doing this:
   final URIBuilder builderContext = new URIBuilder(StringUtils.trimToEmpty(context.getFlowExecutionUrl()));

Thanks @tsschmidt I applied the suggested change.


Improve documentation on MDLC, MDC, NDLC and NDC

Could the documentation possibly be included to explain what is the purpose of NDLC and NDC and maybe how it differs from MDLC and MDC?
With NLog 4.5 structured logging MDLC seems to add metadata properties that are part of the structured data being logged but not part of the message rendered, correct?
What does NDLC do then? append the pushed value to the rendered message or prepended, does it also support structured data, etc...?

And what is the difference for mapped and nested? Like I described in my
question?
søn. 8. apr. 2018 kl. 01:18 skrev Rolf Kristensen<EMAIL_ADDRESS>
:

The "C" in all of them translates to "Context" (As one can see in the
Wiki), so they all provide a context (or scope) which can be captured by
log-messages created within the scope.
MDLC and NDLC are the new ones that supports async context from tasks (As
one can see in the Wiki).
MDLC and MDC allows one to configure a setting within a context/scope (As
one can see in the Wiki)
NDLC and NDC allows one to assign a name to the context/scope (As one can
see in the Wiki).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NLog/NLog/issues/2652#issuecomment-379506034, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAva1yYs3kE9JgK5U5otxoY9RHMMx7Q9ks5tmUkvgaJpZM4TLMMA
.

Mapped works like a dictionary where each scope can add new values
Nested works like a stack allowing one to rename the scope on each stack push. Used for BeginScope in Microsoft Extension Logging
Sent from my Samsung device
-------- Original message --------
From: Sander Struijk<EMAIL_ADDRESS>Date: 08/04/2018 15:13 (GMT+01:00)
To: NLog/NLog<EMAIL_ADDRESS>Cc: Rolf Kristensen<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [NLog/NLog] Improve documentation on MDLC, MDC, NDLC and NDC   (#2652)
And what is the difference for mapped and nested? Like I described in my
question?
søn. 8. apr. 2018 kl. 01:18 skrev Rolf Kristensen<EMAIL_ADDRESS>
:

The "C" in all of them translates to "Context" (As one can see in the
Wiki), so they all provide a context (or scope) which can be captured by
log-messages created within the scope.
MDLC and NDLC are the new ones that supports async context from tasks (As
one can see in the Wiki).
MDLC and MDC allows one to configure a setting within a context/scope (As
one can see in the Wiki)
NDLC and NDC allows one to assign a name to the context/scope (As one can
see in the Wiki).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NLog/NLog/issues/2652#issuecomment-379506034, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAva1yYs3kE9JgK5U5otxoY9RHMMx7Q9ks5tmUkvgaJpZM4TLMMA
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/issues/2652#issuecomment-379549661, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK-fnPyRPZ2P0Vz79X_hnPCVxC-mh0xmks5tmg0UgaJpZM4TLMMA.

So nested works like described here? https://nblumhardt.com/2016/11/ilogger-beginscope/

Serilog have transformed BeginScope into something that works like a combination of MDLC and NDLC
Sent from my Samsung device
-------- Original message --------
From: Sander Struijk<EMAIL_ADDRESS>Date: 08/04/2018 19:03 (GMT+01:00)
To: NLog/NLog<EMAIL_ADDRESS>Cc: Rolf Kristensen<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [NLog/NLog] Improve documentation on MDLC, MDC, NDLC and NDC   (#2652)
So nested works like described here? https://nblumhardt.com/2016/11/ilogger-beginscope/
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/issues/2652#issuecomment-379565419, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK-fnK1QTjbiRcrz2vuOsM9fuy6N83Sgks5tmkL4gaJpZM4TLMMA.

Maybe this wiki-page covers your questions:
https://github.com/NLog/NLog/wiki/Gdc-layout-renderer

I've added https://github.com/NLog/NLog/wiki/Contexts
Please your input and feel free to improve/edit the page

@304NotModified Have added more details to the page:
https://github.com/NLog/NLog/wiki/LogEvent-Context-Information

Hi @snakefoot
I see you did a complete rewrite and removed a lot I have written. Do you think it was unclear? e.g. this is now dropped

Advise

When you need the context for all threads, use the GDC.
When you need the context for one thread, use the MDC.
When you work with async, use the MDLC.
When tracking scoped work, use NDC or NDLC
If the context is different for every message, use Structured logging or the Log event properties

IMO that's the question I get from others and was unclear to me when I started with Nlog.
Also do you think we have to ignore the MDC and NDC in favor of the MDLC and NDLC?

IMO that's the question I get from others and was unclear to me when I started with Nlog.

I have now made a headline with example code for each of your advice lines. You are welcome to add more details for each headlines. Or make an index of the headlines.

Also do you think we have to ignore the MDC and NDC in favor of the MDLC and NDLC?

Yes. Too confusing.

Yes. Too confusing

OK, but maybe we should add something about it, as currently users are using the MDC and NDC

Already done since it says Replaces ${mdc} and Replaces ${ndc}.
And have also updated:
https://github.com/NLog/NLog/wiki/NDC-Layout-Renderer
https://github.com/NLog/NLog/wiki/mdc-Layout-Renderer

And have also updated:
https://github.com/NLog/NLog/wiki/NDC-Layout-Renderer
https://github.com/NLog/NLog/wiki/mdc-Layout-Renderer

👍

@snakefoot I'm busy combining both documents,
but I'm search for an example: when should we use NestedDiagnosticsLogicalContext.Push over MappedDiagnosticsLogicalContext.SetScoped?

Ndlc should be used when wanting to output the entire scope-stack.
Sent from my Sony Xperia
---- Julian Verdurmen wrote ----
@snakefoothttps://github.com/snakefoot I'm busy combining both documents,
but I'm search for an example: when should we use NestedDiagnosticsLogicalContext.Push over MappedDiagnosticsLogicalContext.SetScoped?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/issues/2652#issuecomment-487716970, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACXZ7HGGD4B6Y4L5YZPVUB3PS5F37ANCNFSM4EZMYMAA.

So we need an example with 2 pushes I guess

Have updated these wiki-packages to better explain when they are useful:

https://github.com/NLog/NLog/wiki/NDLC-Layout-Renderer
https://github.com/NLog/NLog/wiki/MDLC-Layout-Renderer

Thanks!


JS API: debug setting not respected

It seems that using the debug setting doesn't work properly with the JS API, as Puppeteer is closed immediately after the iterations end.
Here's my fuite.mjs file:
import { defaultScenario, findLeaks } from 'fuite'
import { createTests, setup, iteration } from './conversationsToSettings.scenario.mjs'

const myScenario = {
  ...defaultScenario,
  createTests,
  setup,
  iteration
}

let log = []

for await (const result of findLeaks('https://localhost:8888', {
  scenario: myScenario,
  iterations: 7,
  heapsnapshot: false,
  debug: process.env.DEBUG || false,
  progress: true,
})) {
  const entry = JSON.stringify(result, null, 2)
  // console.log(entry)
  log.push(entry)
}

And I'm running it like this
DEBUG=true node fuite.mjs

Hm, I think maybe you are missing the node --inspect-brk? You're definitely passing in debug correctly – the CLI does the same thing:
https://github.com/nolanlawson/fuite/blob/4e57dd973d458d93af5d62802c679ed25617dd43/src/cli.js#L83-L91


self.use_transactional_tests = false for all test cases

Active Record test case depends on database savepoint feature to restore the database status by default as self.use_transactional_tests = true
As of TiDB 5.2, it does not support savepoint feature yet then
changing self.use_transactional_tests = false for all test cases until it is
implemented.
Refer
https://github.com/pingcap/tidb/issues/6840

Follow up https://github.com/pingcap/activerecord-tidb-adapter/pull/22#issuecomment-914921528

Opened as a draft pull request since I am not sure this code is enough yet.

Once this pull request is green and merged, this line can be reverted https://github.com/pingcap/rails/blob/1f68df160a11fd7ed7e52eed898cb47bff3b3751/activerecord/test/cases/test_case.rb#L23

Since TiDB 6.2 supports savepoint that does not require setting self.use_transactional_tests = false, I'm going to close this pull request.


Added knob support to set task variables

Added linkage between knobs and task variables. This is done with a builder pattern as most knobs probably will not need to publish task variables.

Technically, this looks good to me - I have concerns about whether we should be doing this though (see https://github.com/microsoft/azure-pipelines-agent/pull/2862#discussion_r395042775 where I think we should continue the conversation)

Abandoning since we decided this is more over kill than needed right now.


Compat for Django 1.9 and 1.10

Hi there!
I'm the package maintainer of python-jingo in Debian. I maintain it, because it's a build-dependency of python-django-compressor, which is itself used by OpenStack Horizon (the OpenStack dashboard), and I maintain all packages for OpenStack in Debian.
As we're going to ship Debian 9 (aka: Stretch, which we will freeze a the end of the year) with Django 1.10, we are already extensively testing and making sure that all of our packages are Django 1.10 ready. Therefore, we're testing them against the beta1 release of Django (which is currently in Debian Experimental).
I have therefore written a few patches for Jingo to support Django 1.10. As much as I can tell, you guys aren't even supporting Django 1.9, which is already old (more than 6 months). I have rebuilt the package in both Django 1.9 and 1.10 env, and all unit tests are passing. Well, all but:

test_render_django
test_render_django_no_toplevel_override
test_render_django_toplevel_override

I haven't figured out yet how to fix these in Django 1.9 / 1.10, and I'm not much interested in it, because it looks like only fixing the unit tests themselves. However, if you can come with a patch, I'd love to use it.
Best would be if you could release a new version of Jingo with this pull request, plus the fix of the 3 unit tests above which are failing. In such case, please let me know (I'm zigo on Freenode and OFTC), and I'll make sure your new upstream release ends up in Debian Stretch.
Hoping this pull request helps,
Thomas Goirand (zigo)

@thomasgoirand as the only remaining maintainer of jingo, I've EOLed the project, as noted in the README. I've moved my projects to django-jinja and recommend others do the same. I won't be doing any additional releases of jingo.
If you're interested in taking over maintenance of the project let me know, I can give you access to PyPI and we can direct people to your fork. Alternatively, you can always fork the project under a new name.
If you do want to pick up maintenance, I'd recommend only supporting currently-supported Django versions, i.e. 1.8–1.10. It helps a lot.


MQTT Update interval

Hey @syssi,
sorry for bugging you again... but I have a problem that I don't have a clue on how to solve.
My PowerSensor is sensing and sending the required power every 5s via MQTT.... However the esphome-soyosource-gtn-virtual-meter does only read a new value via MQTT every 60sec or so....
Is there a way to speed up the MQTT and process the incomming MQTT Messages faster?

Thanks for your help!

Could you provide your yaml configuration? It looks like you are throttling the incoming MQTT messages. Something like this:
  - id: powermeter
    internal: true
    platform: mqtt_subscribe
    name: "${name} instantaneous power consumption"
    topic: "smartmeter/sensor/groundfloor/obis/1-0:16.7.0/255/value"
    accuracy_decimals: 2
    unit_of_measurement: W
    device_class: power
    filters:
      - throttle_average: 60s

The ESPHome node does receive all subscribed topics immediately per default. Please be careful if you play around with these parameters. You will create oscillations if you update the power demand too often.

My yaml configuration is the same as yours.
I tried leaving out the "filters" option altogether and also tried with various "filters:
- throttle_average: [X]s" Settings where [X] was within: 1s, 5s, 10s, 15s
However the current solution processes new MQTT messages only exactly every 60s. You can also see that on the log Timecode in the above screenshot.
I would be happy if I could reduce that Intervall to about 5-10s to be more "real-time"
Cheers!

Could you provide a plaintext log of around 10 minutes? We are talking about the updates of the instantaneous power consumption sensor, correct?

In my case the instantaneous power consumption is published (and received by the ESPHome node) every second. Throttling the updates / incoming messages to 15s leads to a log message every 15 seconds:
[20:31:45][D][sensor:125]: 'groundfloor_inverter smartmeter instantaneous power': Sending state 697.32001 W with 2 decimals of accuracy
[...]
[20:32:00][D][sensor:125]: 'groundfloor_inverter smartmeter instantaneous power': Sending state 701.06000 W with 2 decimals of accuracy

If you see this log message only once per minute and you don't apply any filters please double check your MQTT traffic (f.e. subscribe to the topic using mosquitto_sub manually). I assume there is just one message per minute published via MQTT.

Hi Sebastian @syssi,
here is a 10min debug log:
[19:42:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 0 watts per inverter (0 in total)
[19:42:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 0 watts per inverter (0 in total)
==>[19:42:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -655.75031 W with 2 decimals of accuracy**
[19:42:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -655
[19:42:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 655 / last demand: 0
[19:42:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -655.75031 W with 2 decimals of accuracy
[19:42:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -655
[19:42:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 655 / last demand: 0
[19:42:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 655 watts per inverter (655 in total)
[19:43:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 655 watts per inverter (655 in total)
[19:43:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 655 watts per inverter (655 in total)
==>[19:43:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -655.20691 W with 2 decimals of accuracy
[19:43:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -655
[19:43:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 655 / last demand: 655
[19:43:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 655 watts per inverter (655 in total)
[19:43:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -655.20691 W with 2 decimals of accuracy
[19:43:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -655
[19:43:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 655 / last demand: 655
[19:44:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 655 watts per inverter (655 in total)
[19:44:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 655 watts per inverter (655 in total)
[19:44:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 655 watts per inverter (655 in total)
==>[19:44:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -654.32379 W with 2 decimals of accuracy
[19:44:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -654
[19:44:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 654 / last demand: 655
[19:44:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -654.32379 W with 2 decimals of accuracy
[19:44:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -654
[19:44:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 654 / last demand: 655
[19:45:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 654 watts per inverter (654 in total)
[19:45:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 654 watts per inverter (654 in total)
[19:45:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 654 watts per inverter (654 in total)
==>[19:45:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -654.81165 W with 2 decimals of accuracy
[19:45:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -654
[19:45:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 654 / last demand: 654
[19:45:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -654.81165 W with 2 decimals of accuracy
[19:45:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -654
[19:45:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 654 / last demand: 654
[19:46:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 654 watts per inverter (654 in total)
[19:46:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 654 watts per inverter (654 in total)
[19:46:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 654 watts per inverter (654 in total)
==>[19:46:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -656.82721 W with 2 decimals of accuracy
[19:46:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -656
[19:46:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 656 / last demand: 654
[19:46:54][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -656.82721 W with 2 decimals of accuracy
[19:46:54][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -656
[19:46:54][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 656 / last demand: 654
[19:46:54][I][ota:105]: Boot seems successful, resetting boot loop counter.
[19:46:54][D][esp32.preferences:113]: Saving 1 preferences to flash...
[19:46:54][D][esp32.preferences:142]: Saving 1 preferences to flash: 0 cached, 1 written, 0 failed
[19:47:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 656 watts per inverter (656 in total)
[19:47:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 656 watts per inverter (656 in total)
[19:47:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 656 watts per inverter (656 in total)
==>[19:47:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -658.75085 W with 2 decimals of accuracy
[19:47:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -658
[19:47:55][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 658 / last demand: 656
[19:47:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -658.75085 W with 2 decimals of accuracy
[19:47:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -658
[19:47:55][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 658 / last demand: 656
[19:48:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 658 watts per inverter (658 in total)
[19:48:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 658 watts per inverter (658 in total)
[19:48:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 658 watts per inverter (658 in total)
==>[19:48:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -658.20691 W with 2 decimals of accuracy
[19:48:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -658
[19:48:55][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 658 / last demand: 658
[19:48:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -658.20691 W with 2 decimals of accuracy
[19:48:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -658
[19:48:55][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 658 / last demand: 658
[19:49:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 658 watts per inverter (658 in total)
[19:49:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 658 watts per inverter (658 in total)
[19:49:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 658 watts per inverter (658 in total)
==>[19:49:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -657.66315 W with 2 decimals of accuracy
[19:49:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -657
[19:49:55][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 657 / last demand: 658
[19:49:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -657.66315 W with 2 decimals of accuracy
[19:49:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -657
[19:49:55][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 657 / last demand: 658
[19:50:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 657 watts per inverter (657 in total)
[19:50:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 657 watts per inverter (657 in total)
[19:50:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 657 watts per inverter (657 in total)
==>[19:50:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -657.79785 W with 2 decimals of accuracy
[19:50:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -657
[19:50:55][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 657 / last demand: 657
[19:50:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -657.79785 W with 2 decimals of accuracy
[19:50:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -657
[19:50:55][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 657 / last demand: 657
[19:51:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 657 watts per inverter (657 in total)
[19:51:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 657 watts per inverter (657 in total)
[19:51:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 657 watts per inverter (657 in total)
==>[19:51:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -658.35089 W with 2 decimals of accuracy
[19:51:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -658
[19:51:55][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 658 / last demand: 657
[19:51:55][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -658.35089 W with 2 decimals of accuracy
[19:51:55][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -658
[19:51:56][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 658 / last demand: 657
[19:52:14][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 658 watts per inverter (658 in total)
[19:52:34][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 658 watts per inverter (658 in total)
[19:52:54][D][soyosource_virtual_meter:073]: 'unnamed': Setting the limiter to 658 watts per inverter (658 in total)
==>[19:52:56][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -657.43427 W with 2 decimals of accuracy
[19:52:56][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -657
[19:52:56][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 657 / last demand: 658
[19:52:56][D][sensor:127]: 'soyosource-gtn-display instantaneous power consumption': Sending state -657.43427 W with 2 decimals of accuracy
[19:52:56][D][soyosource_virtual_meter:124]: 'unnamed': Using the passthrough method: -657
[19:52:56][D][soyosource_virtual_meter:020]: 'unnamed': New calculated demand: 657 / last demand: 658

I denoted the times a MQTT message gets processed via adding a  "==>" prefix in the above log.
You can clearly see that processing the next  MQTT message only appears to happen each 60s.
My configuration is:
  # mqtt subscribe example
  - id: powermeter0
    internal: true
    platform: mqtt_subscribe
    name: "${name} instantaneous power consumption"
    topic: "instantaneous_PowerRequirement_Inverter1"
    accuracy_decimals: 2
    unit_of_measurement: W
    device_class: power
#    filters:
#      - throttle_average: 15s

Likewise noted: I tried with the filter setting active already and with various settings "throttle_average:[x]s" (with [x]= 1,2,5,10,15s) but to no avail
Any thoughts?

Please double check your MQTT traffic. Please use an independent MQTT client (like mosquitto_sub) and subscribe to instantaneous_PowerRequirement_Inverter1. Please provide a capture of these messages. If the measurement (message) doesn't change ESPHome doesn't perform a sensor update.
The MQTT subscription component you are using is a core feature of ESPHome. It's an "on_receive" callback which triggers the log message on every received message. Which ESPHome version do you use? I cannot reproduce your issue with 2022.6.3.

Do you have a spare ESP8266/ESP32? I would like to provide a configuration example to debug the MQTT component / traffic.

Hey Sebastian,
sorry for not comming back to you in time.... I was busy with my job the last few days. Yes I have a spare ESP32 board at hand. Hoewever I have solved the MQTT issue. In fact my MQTT broker (Mosquitto) was somehow missbehaving. Once restarted everthing worked liked expected. So please excuse my dumb question. However this is the last thing you take into consideration.
However I have analysed the power_demand_calculation methods a bit and I think that there is room for some improvement. I don't know if you are open to talk about that? If yes I suggest we open a new "issue" to talk about it and not to muddy the waters in this thread?

Perfect! Let's close this issue and feel free to create a new one to discuss improvements. I'm happy about your feedback & suggestions to improve this project.


Update Share Permissions not working

I pulled down the latest version of the update share permissions script and created a CSV to go with it.  I have only included one line to validate that everything is working but when we run the script nothing happens.  It runs and drops back to the prompt without any updates to the UI or the NMC.  I had support check and the NMC isn't even showing an attempt to use the API.  If I set the address of the NMC wrong, we still don't get any updates in the UI.  It just drops back to the prompt the same as before.  I have attempted to add better reporting to the script but that didn't seem to do anything either.

Can you please share the one-line CSV that you're using? It's fine if you want to change any of the share properties to anonymize it—all I need to troubleshoot is all the columns in the CSV. The same columns should be there and populated, but the data in the cells can be changed.
Thank you,
Coan

From: dracodoom @.>
Sent: Wednesday, May 31, 2023 3:12 PM
To: nasuni-labs/nasuni-api-utilities @.>
Cc: Subscribed @.***>
Subject: [EXTERNAL] [nasuni-labs/nasuni-api-utilities] Update Share Permissions not working (Issue #15)
I pulled down the latest version of the update share permissions script and created a CSV to go with it. I have only included one line to validate that everything is working but when we run the script nothing happens. It runs and drops back to the prompt without any updates to the UI or the NMC. I had support check and the NMC isn't even showing an attempt to use the API. If I set the address of the NMC wrong, we still don't get any updates in the UI. It just drops back to the prompt the same as before. I have attempted to add better reporting to the script but that didn't seem to do anything either.
—
Reply to this email directly, view it on GitHubhttps://github.com/nasuni-labs/nasuni-api-utilities/issues/15, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVPCGAVMDJSXT5CDB6HQKU3XI6QZZANCNFSM6AAAAAAYV6XMNE.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
This e-mail message and all attachments transmitted with it may contain privileged and/or confidential information intended solely for the use of the addressee(s). If the reader of this message is not the intended recipient, you are hereby notified that any reading, dissemination, distribution, copying, forwarding or other use of this message or its attachments is strictly prohibited. If you have received this message in error, please notify the sender immediately and delete this message, all attachments and all copies and backups thereof.

Thanks,
Charles Kearney
REVGroup IT Systems Manager
225-806-8056
From: Coan Dillahunty @.>
Sent: Wednesday, May 31, 2023 3:25 PM
To: nasuni-labs/nasuni-api-utilities @.>
Cc: Charles Kearney @.>; Author @.>
Subject: [EXTERNAL] Re: [nasuni-labs/nasuni-api-utilities] Update Share Permissions not working (Issue #15)
You don't often get email from<EMAIL_ADDRESS>Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Can you please share the one-line CSV that you're using? It's fine if you want to change any of the share properties to anonymize it-all I need to troubleshoot is all the columns in the CSV. The same columns should be there and populated, but the data in the cells can be changed.
Thank you,
Coan

From: dracodoom<EMAIL_ADDRESS>Sent: Wednesday, May 31, 2023 3:12 PM
To: nasuni-labs/nasuni-api-utilities<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [EXTERNAL] [nasuni-labs/nasuni-api-utilities] Update Share Permissions not working (Issue #15)
I pulled down the latest version of the update share permissions script and created a CSV to go with it. I have only included one line to validate that everything is working but when we run the script nothing happens. It runs and drops back to the prompt without any updates to the UI or the NMC. I had support check and the NMC isn't even showing an attempt to use the API. If I set the address of the NMC wrong, we still don't get any updates in the UI. It just drops back to the prompt the same as before. I have attempted to add better reporting to the script but that didn't seem to do anything either.

Reply to this email directly, view it on GitHubhttps://github.com/nasuni-labs/nasuni-api-utilities/issues/15, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVPCGAVMDJSXT5CDB6HQKU3XI6QZZANCNFSM6AAAAAAYV6XMNE.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>This e-mail message and all attachments transmitted with it may contain privileged and/or confidential information intended solely for the use of the addressee(s). If the reader of this message is not the intended recipient, you are hereby notified that any reading, dissemination, distribution, copying, forwarding or other use of this message or its attachments is strictly prohibited. If you have received this message in error, please notify the sender immediately and delete this message, all attachments and all copies and backups thereof.

Reply to this email directly, view it on GitHubhttps://github.com/nasuni-labs/nasuni-api-utilities/issues/15#issuecomment-1570898334, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BAHIQEZOZEVOGJSSU4OJ56LXI6SLDANCNFSM6AAAAAAYV6XMNE.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

There was a problem with the CSV formatting in Nasuni labs. Updated scripts and examples to correct the problem


Duplicate call extendRoutes

Version
v2.5.1
Reproduction link
Not really needed
Steps to reproduce

Create nuxt app with npx create-nuxt-app <project-name>
Create module like https://pastebin.com/gSkQUiKg
Run in dev mode
Update nuxt.config.js

What is expected ?
Just call one time the extendRoutes
What is actually happening?
Call multiple time extendRoutes

This bug report is available on Nuxt community (#c8937)

wrap that in a nuxt hook like build:before to control execution more fine-grained.

@manniL I don't know how i can make that, i have tried with just
this.nuxt.hook('buid:before', () => {
  this.extendRoutes(routes => {
    console.log('ok')
  })
})

But this is not work.

you are missing an l in build

@manniL yes i have missing only on copy but it is right in the code ;)


fix(json-mapper): discriminator type deserialization

Information
Fix type determined when deserializing an object having a discriminator in the parent

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

v7.36.7
GitHub release

Your semantic-release bot :package::rocket:


Fix/components in appjs

fixes #37
made all the components in the app.js

Hey @hs309123 great work cleaning up the code.
Just one issue  the filter for contests doesn't seem to work when in mobile view, an you recheck please ?

Great work @hs309123 . The codebase looks much cleaner now.
Merging the PR.

@all-contributors please add hs309123 for code


i18n(es): add typescript page

What kind of changes does this PR include?

[ ] Minor content fixes (broken links, typos, etc.)
[ ] New or updated content
[x] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description

adds guides/typescript.md page in Spanish
page is updated with changes in #761

LGTM! 🚀

Thanks @kevinzunigacuellar, @agustinmulet, great work!!


Canvas elements in the html are not properly converted in to the result canvas.

Polygon in the map is drawn using canvas. I am trying to convert into canvas. but i  unable to convert it properly.
Actual result:

Expected Output should be like this

If you are working on localhost, test your code on Google Chrome, otherwise it will fail. If you can't test on Chrome, try uploading your files in a server.


& got removed from subject

Hi,
If subject contains & than it gets stripped out. does anyone has this issue?

This issue is two years old and I was unable to reproduce using the latest code from master. If you're continuing to have this issue with the latest version, please open a new issue.


[UI] emptying a table in the UI get their headers uncentered

Version: master-head (v2.3) (8/15/19)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Locate a table that has at least one row in the Rancher UI
Or populate one like the clusters with one cluster.
Empty the table by removing all the rows.

Result:

the table headers get uncentered to the left.

Environment information

rancher v2.3 head-master
single install

Not going to fix in the 2.4 timeframe.  Shouldn't be an issue for redesign.


Medical icons

Just found out about feathers and the I was inclined to check lucide. I always wanted to find a library like this ( very very nice icons ), And like every other developer with specific needs at specific times I would like to propose adding some medical icons to the library. Here are some

Icon name: medical icons
Use case: medical related applications
Screenshots of similar icons:

Added some screenshots from font-awesome-medic-icons

with specific needs at specific times

Not the purpose of Lucide, IMO.

@locness3 I totally agree. I was just saying those are some icons that I would love to have in lucide so I don't import multiple icons libraries

@ericfennis great, looking forward to those 2

What do you guys think about these?
syringe

<svg
  width="24"
  height="24"
  xmlns="http://www.w3.org/2000/svg"
  viewBox="0 0 24 24"
  fill="none"
  stroke="currentColor"
  stroke-width="1"
  stroke-linecap="round"
  stroke-linejoin="round"
>
  <line x1="18" y1="2" x2="22" y2="6" />
  <line x1="17" y1="7" x2="20" y2="4" />
  <path d="M7 21l12-12l-4-4l-12 12z" />
  <path d="M6 14l2 2zm3-3l2 2zm3-3l2 2z" />
  <line x1="5" y1="19" x2="2" y2="22" />
</svg>

syringe-2

<svg
  width="24"
  height="24"
  xmlns="http://www.w3.org/2000/svg"
  viewBox="0 0 24 24"
  fill="none"
  stroke="currentColor"
  stroke-width="1"
  stroke-linecap="round"
  stroke-linejoin="round"
>
  <line x1="18" y1="2" x2="22" y2="6" />
  <line x1="17" y1="7" x2="20" y2="4" />
  <path d="M9 19l10 -10l-4-4l-10 10l-0.5 4.5z" />
  <path d="M6.5 13.5l2 2zm3-3l2 2zm3-3l2 2z" />
  <line x1="4.5" y1="19.5" x2="2" y2="22" />
</svg>

@wojtekmaj The base looks good but.. When setting the stroke-width to 2 it looks a bit too tight.
With that it is not matching the guidelines.

I tried to work on the stethoscope, but I'm not a designer. Feedback is welcome.

the stethoscope with geometrical shapes

@delnyn nice!

thanks :)

@wojtekmaj: While we could adjust the markings on the syringes to comply with the guides, I'd propose to get rid of them altogether:

<svg
  xmlns="http://www.w3.org/2000/svg"
  width="24"
  height="24"
  viewBox="0 0 24 24"
  fill="none"
  stroke="currentColor"
  stroke-width="2"
  stroke-linecap="round"
  stroke-linejoin="round"
>
  <path d="M18 2L22 6"/>
  <path d="M17 7L20 4"/>
  <path d="M9 19L19 9L15 5L5 15V19H9Z"/>
  <path d="M8 12L12 16"/>
  <path d="M5 19L2 22"/>
</svg>

<svg
  xmlns="http://www.w3.org/2000/svg"
  width="24"
  height="24"
  viewBox="0 0 24 24"
  fill="none"
  stroke="currentColor"
  stroke-width="2"
  stroke-linecap="round"
  stroke-linejoin="round"
>
  <path d="M18 2L22 6"/>
  <path d="M17 7L20 4"/>
  <path d="M19 9L9 19H5V15L15 5"/>
  <path d="M8 12L12 16"/>
  <path d="M5 19L2 22"/>
  <path d="M14 4L20 10"/>
</svg>

last one is the best imo

Yeah, I second that, I think the notches are definitely a bit too much, and the third one maybe looks a bit more like a bicycle pump than it does a syringe.

@karsa-mistmere @delnyn I agree last one looks the best.
Feel free to open a PR!

Will do!


MP Wong Chen still listed as Parliament Candidate

MP Wong Chen has only Posts "Parliament Candidate" attached to him.  The error already exist in the old MySociety PopIt DB.
Data below; along with Membership ID for your convenience:
[
  {
    "mid": "55b2ffdc1965c6b14cace682",
    "mrange": [
      "2013",
      null
    ],
    "mrole": "Parliamentary Candidate"
  },
  {
    "mid": "55b3000c1965c6b14cace992",
    "mrange": [
      "2013",
      null
    ],
    "mrole": "Parliamentary Candidate"
  },
  {
    "mid": "55b31d5d4a4cab6f4ca15313",
    "mrange": [
      "2013",
      null
    ],
    "mrole": "Parliamentary Candidate"
  },
  {
    "mid": "55b31d60e81cea364b06d62a",
    "mrange": [
      "2013",
      null
    ],
    "mrole": "Parliamentary Candidate"
  }
]

AQL used:
FOR p IN persons
    FILTER LIKE(LOWER(p.data.name),'%wong chen%')
    COLLECT m = p.data.memberships[*]
    FOR n IN m
        RETURN {
            mid: n.id,
            mrange: [n.start_date, n.end_date],
            mrole: n.role
        }

@leowmjw Added http://api.popit.sinarproject.org/en/memberships/c880c9e2afb14185a89b6018f519371d


Fix a bug where not uploading a new file would lead to an error.

See also as in the following backtrace:
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python2.7/site-packages/flask_admin/base.py", line 68, in inner
    return self._run_view(f, *args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/flask_admin/base.py", line 359, in _run_view
    return fn(self, *args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/flask_admin/model/base.py", line 1633, in edit_view
    if self.update_model(form, model):
  File "/usr/local/lib/python2.7/site-packages/flask_admin/contrib/sqla/view.py", line 1010, in update_model
    if not self.handle_view_exception(ex):
  File "/usr/local/lib/python2.7/site-packages/flask_admin/contrib/sqla/view.py", line 967, in handle_view_exception
    return super(ModelView, self).handle_view_exception(exc)
  File "/usr/local/lib/python2.7/site-packages/flask_admin/contrib/sqla/view.py", line 1006, in update_model
    form.populate_obj(model)
  File "/usr/local/lib/python2.7/site-packages/wtforms/form.py", line 96, in populate_obj
    field.populate_obj(obj, name)
  File "/app/cbg/admin_upload_s3.py", line 67, in populate_obj
    if self.data and self.data.filename and isinstance(self.data, FileStorage):
AttributeError: 'unicode' object has no attribute 'filename'

Thanks @alexex! Merged, included in new release 0.1.2.


TypeError with current version of Naja

I am trying to use contributte/datagrid with the current version of naja and I get following error:
Uncaught TypeError: naja.addEventListener(...).bind is not a function.
The error is thrown in every naja extension of datagrid, both in production and development.
Environment:
naja: 1.7.0
datagrid: 6.2.5
webpack: 4.41.2
Since I am using webpack as  module bundler and datagrid assets are not modules and expect naja to be globally available, my naja setup looks as follows:
naja.ts
import naja from 'naja';

// extensions registered

window.naja = naja;
document.addEventListener('DOMContentLoaded', naja.initialize.bind(naja));

export default naja;

index.ts
import './naja';
import 'bootstrap-datepicker';
import 'bootstrap-select';
import 'ublaboo-datagrid/assets/datagrid';
import 'ublaboo-datagrid/assets/datagrid-instant-url-refresh';
import 'ublaboo-datagrid/assets/datagrid-spinners';

Any suggestions?

I also tried including naja directly via script tag as an UMD and it does not work either. Same error.

@rozsival it's because naja is loaded after datagrid. Try loading naja before.

@dakorpar No, it's not. You can see my naja.ts entry is the top-most loaded module included before datagrid assets. Also, if naja is not present when datagrid extension are registered, an exception is thrown clearly stating that naja or nette.ajax.js must be loaded first. That is not the issue. The addEventListener function is executed, the bind function on it causes the TypeError.

@dakorpar Ok, maybe you meant the DOMContentLoaded event listener?

@dakorpar Geeeez, so naja must be included and initialized too 🙈 That is so wrong as it does not correspond with naja docs. Extensions should be registered before naja init. It would be much better if datagrid assets were exposed as a module so a developer can register them on per-project basis. But in this case I understand the main aim of naja support was to really make it work instead of rewriting the assets so it was done as some kind of a wrapper around previously used jQuery extensions. Thanks for the help!

@rozsival Writing datagrid as npm module is planned. :)

@paveljanda Thanks for the reply and your effort! You've done an amazing job with this project. Maybe I could help with the assets being rewritten as a module?

@rozsival Oh that would be amazing! I started with library happy (those nice checkboxes - https://github.com/paveljanda/happy). That one is required by datagrid and already is written using modules. I am not a frontend programmer so I hope I did a decent job. :D
I would appreciate some help as I am new to JS modules. :)
Are you able to send a full PR with datagrid rewritten as module? Will it break any backwards compatibility?

I would go to TypeScript if we decide to full-ground-rewrite.

@f3l1x I am not strictly against it. I am just voting for simplicity and usability. RIght now, it's no problem just to include the assets in html and datagrid just works out of the box. As far as I know, mentioned library happy works also in browser out of the box because it's in JavaScript (but still a module).
What advantages does TypeScript give us over plain JavaScript?

@paveljanda I suppose I would be able to send a full PR, but I'm afraid I'm not able to answer your question about the BC at the moment. I'm able to get into it in  mid-January/February, so I'll get back  to you on this topic.
@f3l1x For sure! I finally migrated from Flow to TypeScript and there's no way back for me now 😄

@paveljanda It's the type checking what makes TS much better over plain JS. The TS compiler is able to discover so many weak spots and potential bugs in the code base, it also guides developers towards writing much more strict and predictable code, not to mention such cool features as code completion, type guards and many more.

For my point of view it's easy to do codereview and faster accept changes without worries about break all stuff. :-) Just like PHPStan and Nette\Tester.

@rozsival Good point. Also January/Fabruary is absolutely fine. It's opensource, right? :D
Okey, let's go for it. How will it work with some dist assets? Will be there some dist code generated it the repo?
How does it work for users who want to contribute to the project?

User will fork the repo
User will make appropriate changes to the TS code
User will have to generate also the dist code? Or will it be done somehow automatically?

Sorry for being noob in this case. :)

I just place it here, awk.
https://www.pika.dev/

I am not familiar with the Pika Registry. On my projects, there are mostly two ways of doing it.

Dist assets are automatically built while publishing to npm registry, so only the source assets are included in the repo. However, in case of such repositories as datagrid, where code base might be distributed through multiple package managers (composer and npm), this might be a disadvantage as some developers might want to include dist assets from vendor dir if their project does not use npm modules. So there is the second approach.

Dist assets are built and included in the repo. The npm publish process just uses those "pre-built" assets. The build process might be done in a CI (Travis or other) or in a hook being executed on certain operations (merge, push etc.). It really depends on what suits the project and its collaborators the most.

@paveljanda Hey there! I think it's about time to rewrite datagrid assets. Honestly, while using it in my TypeScript / Naja based projects, I am facing more or less frustrating issues that can be resolved, but I am not happy at all about how they are being resolved currently.
So to start with rewriting, I would first like to know, if I could be provided with some full-featured datagrid demo project where I could map all features that need to be included in the new assets version. The current version of datagrid.js is kinda hard to read, so I don't want to miss out anything.
To quickly sum up the plan:

TypeScript will be used
All features will be divided into separated modules for better maintenance and further development (no single file of hundreds of lines of code)
No more jquery and nette.ajax.js, everything will be powered by Naja
There will be a top-level module that will be exported as an extension to be registered with Naja, this top-level module will bind and run all separated modules as needed
Everything will be bundled with webpack into two files:

an ES module with the resulting Naja extension (without naja itself, developers will use this in their own naja setup)
a single UMD file (including naja with datagrid registered, intended to be used by developers who do not use module bundlers or do not want to setup naja themselves)


Add compatibility with compression-webpack-plugin 10.0.0

Hi,
I don't know if this can make for 6.3.0, but compression-webpack-plugin 10.0 has just been released and now there is a warning if you try to use it
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>

Will be fixed by #117

Closed by https://github.com/shakacode/shakapacker/pull/117 Will be released in the next version


Add a custom filter chip option to accept avatar

This PR will support #1010 .

Added a custom option FormBuilderFilterChipsOption, to pass to FormBuilderFilterChip options instead of passing the generic FormBuilderFieldOption to it.
This will give us the ability to pass a custom Widget? avatar.

Thanks for your helpful PR @ShadyZekry.
Can I however request that we have FormBuilderFilterChipsOption  renamed to FormBuilderChipOption.
This way we can reuse the same class for both FormBuilderFilterChip and FormBuilderFilterChip as the latter could also benefit from the avatar option.


Jenkins - removed extra space from hardcoded password templating

This caused the secret to be updated if you use a generated password in the values.yaml to perform an upgrade. It's just basic cleanup.

lgtm thanks @lachie83!


Windows 11 Widgets

Any plans to Windows 11 Widgets removal?

Same question.
Would be great!

Well, technically, this is the Windows10Debloater.
I'm personally not sure about Windows 11 myself though. Maybe look for a different script.


Missing Strings on Crowdin

Environment
Docker
Version
0.11.5
Describe the problem
In some widgets and in the application some Strings are missing to be translated, so I have gone to Crowdin to modify them, but I have found that they are not specified, so they are not internationalized. Here are some screenshots of these Strings.
As a clarification, the screenshots have been obtained when Homarr had another language than English configured.
Logs
NO LOGS NEEDED
Context

Please tick the boxes

[X] I confirm that I attached the proper logs
[X] I've read the docs
[X] I've checked for duplicate issues
[X] I've tried to debug myself

Thank you for opening this issue ! The calendar / date can't have localized strings as they require the dayjs language file to be loaded manually in the code and if we want to import all supported languages it would add a LOT of content to load on the page
The other ones should be fairly easy to implement or fix for someone with time to spare ;)

@AmadeusGraves hello, if you are still here, can you try to start homarr in developer mode with NODE_ENV=development
And then in the settings there will be a developer menu. There you can enable a setting to see the keys of translations, you can then show us what translations are still missing.
Concerning the calendar it still won't be translated because we'd need to load dynamically the locale of the user and we cannot do that. It would add too much space to each page load.

Fixed in #1257

Oh thanks for those, I haven't added all of them to #1257.
I have fixed the clock in #1236 and will implement the same thing in the calendar.
I'll take a look for the other ones. #1257 main points were on the widgets.


Route Permission not working, but console is showing correctly, also one try catch error is shown.

Can you give me more informations, strapi version, node version ... i will investige on this issue

I'm trying to use this on strapi 4.6
Node 16.. to 18..
I think it's not working in strapi 4.5 either.
On Sat, 11 Feb, 2023, 9:04 am PaulRichez, @.***> wrote:

Can you give me more informations, strapi version, node version ... i will
investige on this issue
—
Reply to this email directly, view it on GitHub
https://github.com/PaulRichez/strapi4-plugin-route-permission/issues/4#issuecomment-1426593690,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALVZ4YKL7NEGPIOUURHZ6YTWW4CERANCNFSM6AAAAAAUWHX7BA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Hello, any update on this?? im getting route permission updated message in console. but not updated.

Hello, new version will be soon and fix this

@areghunath Try the 2.0.3


Taking RBAC settings from Alerting plugin default to false

Description

Taking RBAC settings from Alerting plugin default to false
Legacy settings removed because all relevant settings are removed or taken from Alerting plugin
Poll API removed as it is not used

[Tests]
All existing tests passed
Signed-off-by: @akbhatta
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #165 (1b08bff) into main (dc55f62) will increase coverage by 10.87%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #165       +/-   ##
=============================================
+ Coverage     62.42%   73.30%   +10.87%
=============================================
  Files           101       32       -69
  Lines          4484     2030     -2454
  Branches        681      404      -277
=============================================
- Hits           2799     1488     -1311
+ Misses         1517      539      -978
+ Partials        168        3      -165

Flag
Coverage Δ

dashboards-reports
73.30% <ø> (ø)

reports-scheduler
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...sscheduler/action/GetAllReportDefinitionsAction.kt

...heduler/action/UpdateReportInstanceStatusAction.kt

...search/reportsscheduler/action/PluginBaseAction.kt

...ortsscheduler/action/OnDemandReportCreateAction.kt

...in/org/opensearch/reportsscheduler/util/Helpers.kt

...portsscheduler/model/ReportDefinitionDetailsDoc.kt

...rtsscheduler/model/InContextReportCreateRequest.kt

.../reportsscheduler/resthandler/PluginBaseHandler.kt

...pensearch/reportsscheduler/model/ReportInstance.kt

...sscheduler/model/UpdateReportDefinitionResponse.kt

... and 46 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 dc55f62...1b08bff. Read the comment docs.

lgtm
Could you create an issue for removing the polling related code in dashboard-reporting plugin and assign to me?

#166


Storing session?

while using global shortcuts i've noticed every time create_session is called (and shortcuts are bound to it) it creates a new section in KDE's shortcut settings.
i wish to avoid this. is there a way to store the session and use it later? is this even a reasonable way to do what i want to do? i couldn't find anything related to this in the x-d-p documentation

I am not familiar with that portal, you should ask people who wrote the portal.

Or some KDE developers to explain where those sections come from

while using global shortcuts i've noticed every time create_session is called (and shortcuts are bound to it) it creates a new section in KDE's shortcut settings.

This sentence give the impression that you do not understood how the portal flow works, so it may not work like you expect. I may be wrong.
The paradigm of the portal expects you to gave a set of bindings in advance, after that the application has no control besides closing the session.
Note, that you can create multiple session but the user will get a prompt for each session.

i wish to avoid this. is there a way to store the session and use it later?

You can't avoid this, this is how the portal is designed. The user only way to change a binding it through the DE settings.

is this even a reasonable way to do what i want to do? i couldn't find anything related to this in the x-d-p documentation

You did not clearly explain what you want to do.

my apologies for not being clear. by that i meant i have tested multiple create_sessions at the same time so i could find out if the new set of bindings in the DE settings are caused by that.
my issue right now is every time my app starts i have to call create session and that causes a whole new set of keybinds to be created - but they aren't even enabled because the ones from last session are still enabled and are blocking the new ones to take effect. i assumed by loading the session from last time i could avoid that. but there is no way to do that (and i don't really have any evidence that might work in the first place, so i thought to ask)

my issue right now is every time my app starts i have to call create session and that causes a whole new set of keybinds to be created - but they aren't even enabled because the ones from last session are still enabled and are blocking the new ones to take effect. i assumed by loading the session from last time i could avoid that. but there is no way to do that (and i don't really have any evidence that might work in the first place, so i thought to ask)

that looks like an issue on the KDE implementation


cognalys.com - two factor authentication / mobile number verification for free

I've committed about https://www.cognalys.com . But its not updated on website
New method of Mobile number verification / Two factor authentication for free

I see no pull requests related to cognalys...where/when did you commit this change?

I think its 3 weeks before .
i've followed
https://github.com/jdavis/twofactorauth/blob/master/CONTRIBUTING.md

Could you link to your pull request?

Just created new
https://github.com/jdavis/twofactorauth/pull/946

and i dont have right access to upload logo
https://www.cognalys.com/static/logo/favi.png


refactor(@angular-devkit/build-angular): update webpack typings

@clydin you beat me to it, I was gonna do it a couple of hrs ago.

Ah.  I missed one.  Cast removed.


actual hover using mouse_entered and mouse_exited control signals

godot 4.2.beta6 issues was fixed:

https://github.com/godotengine/godot/pull/84547

HYPE! I'll test this tomorrow, really hoping it will function entirely properly.

Nice! I'll edit the PR slightly to resolve a conflict from my last PR and merge it right away.

Thanks!

Nice! I'll edit the PR slightly to resolve a conflict from my last PR and merge it right away.

all good with me, should i make changes to the readme in this commit for the Godot version

This PR seems to improve the performance of rebuilding the inspector by 5-10%. Cool!


Error on Message Terraria.MessageBuffer

Description
When loading up a Linux server of Terraria using tModLoader (latest version (v<IP_ADDRESS>)) using mono, with and without mods, the error message "Error on Message Terraria.MessageBuffer" happens after playing the game for ~5 minutes.
This message at first happens infrequently, where it eventually escalates to consistent error logging, to eventually disconnecting everyone from the server ("Connection timed out"). Vanilla without tModLoader works fine.
Steps to Reproduce

Install Terraria from Steam on Linux (latest Debian Stretch in my case)
Copy over the Steam files over tModLoader
Make sure mono is installed by installing package mono-complete
Make sure the required files have the rights to be executed (chmod u+w tModLoaderServer.exe / Terraria.exe)
run mono tModLoaderServer.exe
Any world and configuration will do (with and without mods enabled).
Connect to the server as a client and start exploring a cave or something similar.
After a few minutes of gameplay, this error will pop up and will disconnect everyone connected to the server

Reproduction frequency
12/12 so far.
Versions
Latest version of Terraria on Steam for Linux on this date.
Latest version of tModLoader on this date (v<IP_ADDRESS>, directly downloaded from https://forums.terraria.org/index.php?threads/1-3-tmodloader-a-modding-api.23726/ )
Mods
With and without mods.
Additional Information
Running on an external VPS with the latest Debian Stretch, 2GB RAM and an i7 processor.
Screenshot of it happening:

No other information could be fetched from this issue, unless I'm forgetting a log file. There were no logs found in my ~/.local/share/Terraria/ModLoader/Logs folder.

I know many mods cause MessageBuffer errors, so just confirm again that this is with 0 mods for me if you can.
I'm not sure what is going on, but using a system install of mono probably puts you in a configuration we can't account for. Can you try launching the server the normal way and seeing how it goes? -> just run ./tModLoaderServer

Hi, thanks for replying. :)
I can confirm that this is with 0 mods enabled. My friends and I have tried that at least 3 times.
About your second case, this are the results I get:

root@debian:~/TerrariaSrv# ./tModLoaderServer
monoconfig not found!
root@debian:~/TerrariaSrv# ./tModLoaderServer.exe
run-detectors: unable to find an interpreter for ./tModLoaderServer.exe

The only way I can run it is by using mono tModLoaderServer.exe .

hm, that monoconfig file is in both the official dedicated server files and the regular steam install that I downloaded. Not sure why it is saying it can't be found. Not sure how it could be missing or why it would be a fatal error for launching.

Someone else had the monoconfig not found problem. Turned out they were trying to install tModLoader over TShock instead of the vanilla server software: http://terraria.org/server/terraria-server-1353.zip
Make sure to install tModLoader over vanilla not tShock

Having this error pop up even on Windows. 100% sure that this is vanilla server software.

Along with this, I keep getting booted from "Invalid operation at this state". One of my friends was also experiencing it too.