please disable initial request when dashboard load

dear Mikhail Volkov,
As of now we are using your plugin version 2.8.0 and it is very comfortable for us.
But in that can we have one change that when dashboard reloads it should not execute initial request function or OnClick it should not perform initial request function. anything is ok.
{options.reset.variant !== ButtonVariant.HIDDEN && (
<Button
className={cx(styles.margin)}
variant={options.reset.variant as any}
icon={options.reset.icon}
style={
options.reset.variant === ButtonVariant.CUSTOM
? {
background: 'none',
border: 'none',
backgroundColor: theme.visualization.getColorByName(options.reset.backgroundColor),
color: theme.visualization.getColorByName(options.reset.foregroundColor),
}
: {}
}
disabled={loading}
onClick={initialRequest}  *** wants to disable this functionality* it should return some boolean true
size={options.buttonGroup.size}
>
{options.reset.text}
please do the needful as it is very convenient to use this version

@vnvasireddy We added an option to disable synch with dashboard reload in the upcoming version.
You can test from the CI artifact while we are working on the new version.


Fix: Flash of Dark Theme

Note: This PR is a temporary fix until the issue with setTheme is fixed.
Current Status:

On first load or when refreshing the browser, setTheme isn't setting the theme value.

This leads to theme being 'undefined', causing the .theme-dark class to be added.

This PR doesn't solve the setTheme problem directly. Instead, it checks to ensure the theme is set to either "dark" or "light".

What kind of changes does this PR include?

Something else:

Minor tweak to prevent the flash of the dark theme.

Description:

Added a check to ensure the theme value is 'light' or 'dark', providing a temporary solution to the setTheme issue in useEffect.

Before / After

Did you change something visual

Before:
https://github.com/withastro/docs/assets/20273871/9006143e-c79f-40f5-822d-8308e706551e
https://github.com/withastro/docs/assets/20273871/a77536e8-7661-48d4-bdde-1161aa5807fa
https://github.com/withastro/docs/assets/20273871/41b08978-ab47-49b0-a02a-a96e7cfdab54
https://github.com/withastro/docs/assets/20273871/26235901-6412-48e4-9b48-2d363d9f0a05
https://github.com/withastro/docs/assets/20273871/65f64f15-21a1-40ed-9bdc-2774907182a0
After:
https://github.com/withastro/docs/assets/20273871/627849e7-7d14-4511-aaab-7dcdb8181c1d

Looks good in Firefox (light & dark)

Make sure to clone and build this locally as well. In the preview in our original PR, this wasn't happening at all IIRC

@ElianCodes Confirmed locally (including with/without throttling)

Hey @ElianCodes and @TheOtterlord ! If we think this fixes the problem, then heads up either one of you could have merged this to get the quick fix in! Is there a reason we're not merging this yet?

@sarah11918, go ahead and merge, I'm not available for the rest of today ☹


Version 1.0.0-beta.15 not compatible with "customerio-reactnative": "4.0.1" on IOS

Hello !
I wanted to upgrade my app expo with "customerio-reactnative": "4.0.1".
But it seems that the current version of customerio-expo-plugin is not compatible with "customerio-reactnative": "4.0.1" on IOS
Here the error
[RUN_FASTLANE] ▸ ** BUILD FAILED **
[RUN_FASTLANE] ▸ The following build commands failed:
[RUN_FASTLANE] ▸        SwiftEmitModule normal x86_64 Emitting\ module\ for\ NotificationService (in target 'NotificationService' from project 'XXXXX')
[RUN_FASTLANE] ▸        SwiftCompile normal x86_64 /var/folders/jr/vzf29l0n5sl24_9mjl93p7pm0000gn/T/eas-build-local-nodejs/4ba72d39-d211-4bb6-9d12-c7f32cf6fd09/build/apps/XXXXX/ios/NotificationService/NotificationService.swift (in target 'NotificationService' from project 'XXXXX')
[RUN_FASTLANE] ▸        SwiftCompile normal x86_64 Compiling\ NotificationService.swift /var/folders/jr/vzf29l0n5sl24_9mjl93p7pm0000gn/T/eas-build-local-nodejs/4ba72d39-d211-4bb6-9d12-c7f32cf6fd09/build/apps/XXXXX/ios/NotificationService/NotificationService.swift (in target 'NotificationService' from project 'XXXXX')
[RUN_FASTLANE] ▸        SwiftCompile normal x86_64 Compiling\ Env.swift /var/folders/jr/vzf29l0n5sl24_9mjl93p7pm0000gn/T/eas-build-local-nodejs/4ba72d39-d211-4bb6-9d12-c7f32cf6fd09/build/apps/XXXXX/ios/NotificationService/Env.swift (in target 'NotificationService' from project 'XXXXX')
[RUN_FASTLANE] ▸        SwiftCompile normal x86_64 /var/folders/jr/vzf29l0n5sl24_9mjl93p7pm0000gn/T/eas-build-local-nodejs/4ba72d39-d211-4bb6-9d12-c7f32cf6fd09/build/apps/XXXXX/ios/NotificationService/Env.swift (in target 'NotificationService' from project 'XXXXX')
[RUN_FASTLANE] ▸ (5 failures)
[RUN_FASTLANE] Exit status: 65

The build is working well with the version 3.9.0

Hey there !
Thank you for your interest in adopting the latest React Native version, which now supports Customer.io Data Pipelines (CDP). Unfortunately, we do not currently support customerio-reactnative version 4.0.1 with our Expo plugin. You can find more information about this in our documentation.
We are actively working on adding support for CDP in Expo, but we do not have a specific timeline to share at the moment. Rest assured, we will notify you once CDP support is available in our Expo plugin. In the meantime, we recommend continuing to use customerio-reactnative version 3.9.1 or earlier.
I will go ahead and close this issue for now but feel free to reopen or drop a comment if you have any queries.


kvcoord: merge budgets for in-flight writes and lock spans

PR: https://github.com/cockroachdb/cockroach/pull/66915
From release notes:

The setting kv.transaction.write_pipelining_max_outstanding_size is now a no-op. Its function is folded into the kv.transaction.max_intents_bytes setting. [#66915][#66915] {% comment %}doc{% endcomment %}

I don't think we need to document this no-op.
@rmloveland Please re-open if you disagree from a KV perspective.

Makes sense to me - the write pipelining cluster setting is only mentioned in a couple of very old release notes.  It does not appear to have even been in the public settings available via SHOW CLUSTER SETTINGS.


Added long description to pypi setup

This PR adds project description to https://pypi.org/project/py-expression-eval/

https://packaging.python.org/tutorials/packaging-projects/#creating-setup-py

Coverage remained the same at ?% when pulling 59dbf1bfb78761eae9d11e3c44717b8aa76ca92b on feature/add-long-pypi-description into 1aad274aef93cd23d6a49a8b1d94dd1b634f0e51 on master.


H1 tag not responsive for max-width: 768px

Hello there guys. On my iPhone 6 Plus, if I use an h1 tag in portrait mode, and it's too long of a title, the h1 will bleed outside of its div container and mess up the overall appearance - using jumbotron theme. H1 tag is not responsive. It somehow escaped QA :sweat_smile:

Hey @aicm thanks for the report. Could you post a quick JSBin demonstrating the problem? Makes it easier for us to check that kinda stuff :smiley:

JSBin: http://jsbin.com/dawogaxizo/1/
Unable to duplicate via the iOS Simulator for the iPhone 6 Plus.

Hi @kkirsche!
You appear to have posted a live example (http://jsbin.com/dawogaxizo/1/edit), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 10, column 1 thru line 10, column 32: Attribute cz-shortcut-listen not allowed on element body at this point.
line 43, column 7 thru line 43, column 12: Stray end tag div.
line 50, column 5 thru line 50, column 10: Stray end tag div.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Updated JSBin cleaning up leftover tags from copying off the example source code: http://jsbin.com/zixuxefolo/1/

Hi @kkirsche!
You appear to have posted a live example (http://jsbin.com/zixuxefolo/1/edit), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

W002: <head> is missing X-UA-Compatible <meta> tag that disables old IE compatibility modes
W003: <head> is missing viewport <meta> tag that enables responsiveness

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

http://jsbin.com/nulanuzupa/1/ Updated again...
Latest screen shot

@kkirsche confirming in both Safari and Chrome on an actual iPhone 6 Plus, I see what you see.

Thanks for looking on a device @StevenBlack. Simulator is good but not a replacement for the actual hardware. :+1:

If a particular string of text is too large or too long, that's up to you to account for I imagine. We don't have a responsive type scale in Bootstrap 3.

 My apology, I didn't do a good job reporting this. @mdo has a point, however. It's precisely when the string is too long that the h1 tag pokes outside of its containers. And bootstrap is probably not the one that needs to account for user's content.
Here's my FF on OS X which basically shows the same way it does on my iPhone when I resize the viewport:

@aicm
h1 { word-wrap: break-word; }

@kukac7 genius. Thank you!

Closing per @mdo's comment and ff., i.e.: this problem should be addressed on an individual basis, not on the Bootstrap level.


EC2 Service discovery not working behind proxy

What did you do?
Good question. It all began when we tried to set up prometheus in a private subnet in AWS. We require a proxy to get access to the outside world, but using the ec2 sd does not respect  the environment variables set up for this. Can you imagine!
What did you expect to see?
A list of ec2 instances appearing in the targets page in prometheus.
What did you see instead? Under which circumstances?
Nothing!!! Checking the logs, we get a timeout when querying ec2. It seems as though it's not finding it. Sad face.
Environment

System information:
Linux 3.13.0-93-generic x86_64

Prometheus version:
 prometheus, version 1.0.1 (branch: master, revision: be40190)
 build user:       root@e881b289ce76
 build date:       20160722-20:00:12
 go version:       go1.6.2

Prometheus configuration file:

global:
  scrape_interval: "15s"     # By default, scrape targets every 15 seconds.
  evaluation_interval: "15s" # By default, evaluate rules every 15 seconds.

scrape_configs:
- job_name: "prometheus"
  scrape_interval: "15s"
  metrics_path: "/metrics"
  static_configs:
    - targets: ["localhost:9090"]
- job_name: "node_exporter"
  scrape_interval: "15s"
  metrics_path: "/metrics"
  honor_labels: true
  ec2_sd_configs:
  - region: us-west-2
    port: 9100

Logs:

ubuntu@ip-10-161-71-70:~$ tail -f /var/log/prometheus/prometheus.log
time="2016-09-23T01:55:42Z" level=info msg="Done checkpointing fingerprint mappings in 2.021242ms." source="persistence.go:1394"
time="2016-09-23T01:55:42Z" level=info msg="Local storage stopped." source="storage.go:406"
time="2016-09-23T01:55:43Z" level=info msg="Starting prometheus (version=1.1.0, branch=master, revision=5ee84a96db6190d4fcdaf4eff74a09b52824a9aa)" source="main.go:73"
time="2016-09-23T01:55:43Z" level=info msg="Build context (go=go1.6.3, user=root@54c6975115bb, date=20160903-19:01:53)" source="main.go:74"
time="2016-09-23T01:55:43Z" level=info msg="Loading configuration file /opt/prometheus/prometheus.yml" source="main.go:221"
time="2016-09-23T01:55:43Z" level=info msg="Loading series map and head chunks..." source="storage.go:358"
time="2016-09-23T01:55:43Z" level=info msg="418 series loaded." source="storage.go:363"
time="2016-09-23T01:55:43Z" level=warning msg="No AlertManagers configured, not dispatching any alerts" source="notifier.go:176"
time="2016-09-23T01:55:43Z" level=info msg="Listening on :9090" source="web.go:233"
time="2016-09-23T01:55:43Z" level=info msg="Starting target manager..." source="targetmanager.go:75"
0x451c90
0x451c90
0x451c90
time="2016-09-23T01:57:43Z" level=error msg="could not describe instances: RequestError: send request failed\ncaused by: Post https://ec2.us-west-2.amazonaws.com/: dial tcp <IP_ADDRESS>:443: i/o timeout" source="ec2.go:80"
0x451c90
0x451c90
0x451c90
time="2016-09-23T01:59:43Z" level=error msg="could not describe instances: RequestError: send request failed\ncaused by: Post https://ec2.us-west-2.amazonaws.com/: dial tcp <IP_ADDRESS>:443: i/o timeout" source="ec2.go:90"

Addressed in #2027


feat: prettier format alby go

Please do not merge this. We have some bigger PRs open that I would want to merge first. :sweat_smile:

@pavanjoshi914 I think now would be a great time to run it as we almost have no open PRs. Can you please update the PR?

Closing in favour of #169


Unbreak the build for GHC 8.6.5

In cardano-ledger-alonzo.cabal, the lower bound on base needlessly excluded
the version of base used by GHC 8.6.5.

Ah, I must admit, I had been assuming that by the time of Alonzo we would be firmly in 8.10.x land. I was hoping to make use of the ability to define multiple libraries in one package to define the tests in a nicer way.

Ah, I must admit, I had been assuming that by the time of Alonzo we would be firmly in 8.10.x land.

Apparently that's now :slightly_smiling_face:. But I don't know whether we're fully in 8.10.x land. As long as we have CI building with 8.6.5, we should try not to break it.

I was hoping to make use of the ability to define multiple libraries in one package to define the tests in a nicer way.
Isn't that unrelated to the GHC version? Isn't that a Cabal feature? The network team has been doing this for a while already.

Sure, but an alternative would be to simply not build to Alonzo package in 8.6.5

Sure, but an alternative would be to simply not build to Alonzo package in 8.6.5

Hmm, with something like the following?
  if impl(ghc < 8.10)
    buildable: False

Ah, that's nicer than my plan, which had been to alter the CI script!

Ah, that's nicer than my plan, which had been to alter the CI script!

Yeah, I want to stay as far away from Nix stuff as I can :grin:

This wouldn't be nix, would be the Github actions script

Closed in favour of #2027.


Next Steps links always point to Classic versions

The Zone pivot and links were missing for the new Standard/Premium instructions

@duongau

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@MatthewSteeples @duongau - we'll need to move these changes to the private repo (we can't merge zone pivot changes in the public repo because there's no way to preview the content and verify that everything renders correctly.
Can you move the commits to the private repo, @duongau ?  Let us know if you need assistance.

#hold-off

@MatthewSteeples @duongau,
We went ahead and staged these changes in https://github.com/MicrosoftDocs/azure-docs-pr/pull/230088. They render as expected, so we'll merge this PR now.
Thanks.


test(client): Sanitize error snapshots

Introduce custom jest matchers that sanitize output of
printError function. I opted for custom matcher instead of
adding this functionality to default snapshot serializer because there
might be some cases, where we actually want to see the line numbers in
the snapshots (#14265 is one example). This won't be possible if line
numbers are always sanitized.
Close #14321

Should this wait until your referred PR has been merged so we know we have coverage for the line numbers?

That's a good idea, i'll try to find time for reviving that one soon.


PHPDOC align for all tags

E.g.
<?php
/**
 * A description.
 *
 * @package package/subpackage
 * @author SomeGuy
 * @copyright Copyright (c) SomeGuy, SomeYear
 * @license SomeLicense
*/

should be aligned to
<?php
/**
 * A description.
 *
 * @package   package/subpackage
 * @author    SomeGuy
 * @copyright Copyright (c) SomeGuy, SomeYear
 * @license   SomeLicense
*/

I think that's very similar to what phpdoc_params currently does - it's just not (only) about @param.

all

We can't really do it for "all", but I think it would be a good idea to align anything that's a tag defined in https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/1.11/Symfony/CS/DocBlock/Tag.php#L26. That way we wouldn't be aligning custom annotations by mistake.

We'd need to be really careful to get the annotation signatures right though, because each have different sets of arguments.

Yes, maybe also only the first align (as shown in my example) would be enough.
I mean in my example I only used tags with one argument.
The only other (important) tag I know of which would need a similar handling as @param is @return.
Of course all this things may be split into different fixers.

phpdoc_params

That fixer is renamed to phpdoc_align in 2.x though.

Of course all this things may be split into different fixers.

Meh, maybe the best thing to do would be to keep in one fixer and allow the tags to align to be configured in a simelar way to how the phpunit fixers work

That fixer is renamed to phpdoc_align in 2.x though.

Ah okay, so then I would expect it to fix more than just @param (based of the name).

Ah okay, so then I would expect it to fix more than just @param (based of the name).

Yeh, the name sucks, but it was only changed on the 2.x line for BC reasons.


Add the ability to accept multiple cert tuples like the description says

This checks to make sure that the certs exist when passed into push.
This wasn't merged but needs to be.

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


Create Transaction Builder Form

@7helles do we have the designs ready for this app?

@earth2travis When Jord and I talked about this a couple weeks ago, my impression was it would probably fit in a form builder app that he has been mulling over. However, since that may get pushed out an indeterminate amount of time, there are designs that could be used as a standalone proposal form version of multicall transactions: https://www.figma.com/file/9lCwprCNaoAmUDmiMQpFEA/DAOhaus-v3?node-id=7711%3A192742

@7helles do you need additional input from developers to start designs on a standalone app?

@earth2travis for something that fits into a bigger form builder app, yes. We should probably have a meeting with @jordanlesich and hash out details for the project and how it fits into budget - he has a much better grasp of it than I do. If it's just going to be a proposal type accessed via the proposals modal, I think those designs could be used.

Closing as duplicate of https://github.com/HausDAO/daohaus-monorepo/issues/161


setting node name breaks tree linkage

# a simple tree
root = DataTree(name='root')
child = DataTree(name='child', parent=root)
grandchild = DataTree(name='grandchild', parent=child)

# changing the name of a child node does not correctly update the dict key in it's parent's children
child.name = 'childish'
print(root)  # this appears to be fine
print(list(root.children))  # however, the keys in root.children have not been updated
print(root['childish'])  # so this fails

Simple fix seems to be wherever the name property is being set it needs to also ensure that the keys in self.parent.children are updated as needed. Not sure if there is anywhere else that is storing these keys that also needs updating.

Thank you for reporting this! The offending setter is here
https://github.com/xarray-contrib/datatree/blob/0afaa6cc1d6800987d8b9c37a604dc0a8c68aeaa/datatree/treenode.py#L597
This should update the key it is stored under in it's parent.
This should be a pretty simple fix if you (or perhaps @etienneschalk ?) are interested in going in? (If not then no worries)

Hello @TomNicholas
In the context of merging datatree into xarray, should new developments continue to be made on this repo, or in the xarray repo? Or is there a code freeze until datatree can be worked with from inside the xarray repo? Or simply, new developments happening here will be integrated into xarray with some git wizardry?

In the context of merging datatree into xarray, should new developments continue to be made on this repo, or in the xarray repo? Or is there a code freeze until datatree can be worked with from inside the xarray repo? Or simply, new developments happening here will be integrated into xarray with some git wizardry?

I think we accept bug fixes here, but not new features. And whilst those bugfixes will be moved to xarray, you won't necessarily get full attribution for them (i.e.

But we should fix the bug here! Because people will still be using this repository for a while yet (as this is what is uploaded to pypi/conda as xarray-datatree)

I'm happy to tackle the fix, but will be traveling for a conference that runs through most of next week, so probably wouldn't get to it until after that. If someone else wants to fix it before then, by all means ;)

What should be the expected behaviour when renaming a child node to None?
I had a look at how xarray behaves when renaming a DataArray inside of a Dataset. It seems that the renaming is just ignored when trying to change the name property of the DataArray directly:
import xarray as xr

https://docs.xarray.dev/en/stable/generated/xarray.DataArray.name.html
xds = xr.Dataset({"a": xr.DataArray([1])})
print(xds)

<xarray.Dataset>
Dimensions:  (dim_0: 1)
Dimensions without coordinates: dim_0
Data variables:
    a        (dim_0) int64 1

print(xds["a"])

<xarray.DataArray 'a' (dim_0: 1)>
array([1])
Dimensions without coordinates: dim_0

xds["a"].name = "toto"

print(xds["a"])

<xarray.DataArray 'a' (dim_0: 1)>
array([1])
Dimensions without coordinates: dim_0

xda = xds["a"]
xda.name = "toto"
print(xda)

<xarray.DataArray 'toto' (dim_0: 1)>
array([1])
Dimensions without coordinates: dim_0

print(xds)

<xarray.Dataset>
Dimensions:  (dim_0: 1)
Dimensions without coordinates: dim_0
Data variables:
    a        (dim_0) int64 1

@etienneschalk, I find that to be very counterintuitive behavior. My naive expectation would be that the variable should be renamed as desired and the dataset updated to reflect that, and if there was any issue (like renaming to None or to the name of another variable) an exception would be raised. Of course, this is an xarray issue.


Add page level comments on docs.github.com

What is the current behavior?
Users are directed to the general github.community or Support contact page

What changes are you suggesting?
Add page-level commenting on individual pages for docs.github.com articles.
Additional information
On the docs.microsoft.com site, there are sections on each page/docset that has this feature enabled that renders something like

I believe this is based of a modified version of https://github.com/utterance/utterances/

Hi @nschonni, sorry about the delay, this one must have slipped through my notifications 💛

Add page-level commenting on individual pages

Can you explain a little more about what you're looking for? Are you looking for something like the automatically included information added to an issue when you click This page?

Sorry, it used to have in-page rendering of the issues created from those buttons, but I guess they might have disabled it sometime over the last few months.
You can see the metadata on an issue like https://github.com/MicrosoftDocs/azure-docs/issues/65249 that it pre-fills in the issue. That used to be combined with their version of utterances to also have an in-page multi issue threaded comment section that was backed by those issues.
Benifit of that, is that users can see the open/closed discussions on a particular page to see if someone already opened an issue with their same problem and what the discussions were. This can be a nicer flow that opening an issue (after searching) and maybe not finding the previous ones.
You can see how the previous setup worked over here https://docs.microsoft.com/en-ca/teamblog/a-new-feedback-system-is-coming-to-docs

What is the current behavior?
Users are directed to the general github.community or Support contact page

What changes are you suggesting?
Add page-level commenting on individual pages for docs.github.com articles.
Additional information
On the docs.microsoft.com site, there are sections on each page/docset that has this feature enabled that renders something like

I believe this is based of a modified version of https://github.com/utterance/utterances/

magnetikonline:fix-restrictions-access-cache-workflow

Thanks so much for this suggestion, we've discussed it more recently with our internal teams and have decided not to go in this direction 💛
We appreciate your interest and passion in the GitHub docs!


"Excavator"

[x] Implement actual action
[x] Support env = { SPECIAL_SNOWFLAKES = 'curl,brotli' }
[x] Support env = { SKIP_UPDATED = true }
[ ] Resolve how to post log into some issue (#20)

Or how to properly expose log for all users

6ccb6c3b2503647c8acd61f3537f82854fae1f9f 4d591b18ed3bca1bbc413e55856df9aa68345892


remove onInit, build it into addFilter

actually, maybe don't remove it since t.svg et al don't become available until init


Advanced dictionary picker

Allow the user to pick which kana to include in the quiz based on a chart

This does exactly that!


Simplify tilemask logic

The 'tile in tilemask' logic does not follow 100% the unoffical spec. Modify/simplify the logic, deprecate 'PARTIAL' status

Merged to develop branch.

Part of SDK 4.1.0RC1


Fix 2x image processing

Fix: Fix -2x image processing
Description
Previously, only 1x images were being processed. The path we were passing to processDefinedImage was not taking -2x images into account, so sharp would only look for 1x versions and subsequently error because we were only giving it -2x versions.
Solution
Update the fullPath variable to be the result of path + filename + extension, where filename contains the original/raw -2x suffix.
Additional Notes
This could probably be cleaner, I was in a hurry to get this out by end of day — please feel free to suggest a more elegant way to pass this information down.

Thanks!
Just to confirm, It still generates the 1x from the 2x right?

Yezzir


[add] New AniList input plugin

Motivation for changes:
Migrated my personal list from MyAnimeList to AniList
Detailed changes:

Based off the existing MyAnimeList plugin since already had some experience with it and AniList works similarly.

Implemented feature requests:

Feathub #80.

Config usage if relevant (new plugin or updated schema):
anilist:
      username: <value>
      status:
        - <current|planning|completed|dropped|paused|repeating>
        - <current|planning|completed|dropped|paused|repeating>
        ...
      release_status:
        - <all|finished|releasing|not_yet_released|cancelled>
        - <finished|releasing|not_yet_released|cancelled>
        ...
      format:
        - <all|tv|tv_short|movie|special|ova|ona>
        - <tv|tv_short|movie|special|ova|ona>
        ...

Can someone from @FlexGet shed some light on why these tests are failing, please?
I couldn't find any script lines referenced in their log messages.

I think this was a problem with our required dependency versions. Try merging latest develop back in to your branch, I believe we fixed this already.

This should be a seamless merge now. Wiki page will be ready by the time this goes live

Ahh, nice. I think the issue with your last attempt was you rebased, or squashed the commits from develop on top of your branch, rather than doing a regular merge.

You got it, a bad commit caused those conflicts. That should teach me to not be lazy using GitHub Web to make changes.
Just swapped the type to str and included the dict check\conversion as suggested

Cool, looks good to me, thanks!


ReST export plugin should accept --template option

The option was accepted by template plugin only, but it absolutely makes sense for -h rst to accept it as well. There is a default story template, and no other template can be used...

I don't like the confusion with saying 'rst|template' only for Plans and Test exports as there is no 'rst' nor 'template' how. However we shouldnt have --template option there at all.
Thus it keeps current status quo.

Good point, I didn't notice there's another bug hiding in plain sight, template export format was not enabled for all classes, just stories.
I tried to improve things in eb2baece: --how template should now be available for all exportable objects, and only tmt story export will mention rst|template in --template help text. Does this make things better?

@happz Much better, thanks. Tests were expecting old help message thus fixed in 7de7b1f9
LGTM

Rawhide failure in  /plans/install/pip/full ::
ERROR: Could not build wheels for aiohttp, which is required to install pyproject.toml-based project
Problem already reported in aiohttp issue tracker - https://github.com/aio-libs/aiohttp/issues/7229
IMO we have no other way than ignore this failure (what uses it btw?)

Seems this is pulled in by mrack:
Collecting aiohttp>=3 (from asyncopenstackclient>=0.8.1->mrack>=1.12.1->tmt==1.25.0)


Update the game over message.

Update the game over message so people know how to play again!


New Abstract for restricting arguments in functions

I started working on something for theme sniffs but realized this would fit better ustream. https://github.com/WPTRT/WordPress-Coding-Standards/pull/80
As we have an abstract to restrict function it would make sense to have one to restrict arguments.
A few examples for this would be

get_bloginfo( 'home' ) where home is deprecated
making sure the correct argument is used e.g. date_i18n( get_option( 'date_format' ) ) #739
requiring that a function has a certain argument get_bloginfo( 'name', 'display' )
if a certain hook should not be used e.g add_filter( 'mime_types', '' ); as it is plugin territory
unneeded checking for backwards compatibility with function_exist( 'add_action' )

Related issues https://github.com/WPTRT/WordPress-Coding-Standards/issues/93 https://github.com/WPTRT/WordPress-Coding-Standards/issues/42 https://github.com/WPTRT/WordPress-Coding-Standards/issues/27 https://github.com/WPTRT/WordPress-Coding-Standards/issues/1 https://github.com/WPTRT/WordPress-Coding-Standards/issues/96
The current array is as below.
array(
'get_bloginfo' => array(
	1 => array(
		'home' => array(
			'alt'     => 'home_url()',
			'version' => '2.2'
		),
		'siteurl' => array(
			'alt'     => 'home_url()',
			'version' => '2.2'
		),
	),
),
);

First we have the function name, the position of the argument, the argument it should match. Alt is the alternative function for the deprecated parameter and version is when it was deprecated.
What I unsure about is how I can differentiate when a argument should not be used and when a argument is required.

May I suggest having an abstract class and then three concrete class - one for deprecated arguments, one for required arguments, one for checking correct usage of an argument ? Keeps the different logic isolated in different sniffs.
Does that help or did you mean something else ?

Thank you @jrfnl Would such a structure make sense:

abstract class WordPress_AbstractArgsRestrictionsSniff implements PHP_CodeSniffer_Sniff

class WordPress_RestrictedArgsSniff extends WordPress_AbstractArgsRestrictionsSnifff

class WordPress_Sniffs_Theme_DeprecatedArgsSniff extends WordPress_RestrictedArgsSniff

class WordPress_RequiredArgsSniff extends WordPress_AbstractArgsRestrictionsSnifff

class WordPress_Sniffs_XSS_BloginfoArgsSniff extends WordPress_RequiredArgsSniff

class WordPress_ControlArgsSniff extends WordPress_AbstractArgsRestrictionsSnifff

class WordPress_Sniffs_WP_DateArgsSniff extends WordPress_ControlArgsSniff

The array for WordPress_RestrictedArgsSnif would be
'get_bloginfo' => array(
	2 => array(
		'home' => array(
			'alt'     => 'home_url()',
		),
		'siteurl' => array(
			'alt'     => 'home_url()',
			'version' => '2.2'
		),
	),
),

The array for WordPress_RequiredArgsSniff would be
'get_bloginfo' => array(
	2 => array(
		'display',
	),
),

The array for WordPress_ControlArgsSniff would be
'apply_filters' => array(
	1 => 'widget_title',
	2 => true,
	3 => true,
	4 => true,
),


Android Auto custom apps, how does it work?

Hi!
After patching my unit with the M.I.B tool for carplay and lifetime updates, I installed the toolbox (v.4.1a) and performed a full backup. After this I started the script "Android Auto custom apps patch", and at the end it was written being all successful. I did a reset, plugged in the phone, but it was all as before. Just the few apps like maps, spotify and whatapp were available.
MHI2_ER_AU37x_P5089
That´s my gal.json file after running the script.
gal.json.txt
Any ideas what´s the problem here?

You need to have an older version of Android Auto to be able to use this feature, because Google disabled custom apps after people started running YouTube clients in their cars. And even then, you need apps that are AA enabled, it doesn't give you access to all the apps on your Android Auto.
However, using AAAD (Android Auto App Downloader) probably still allows you to run custom apps.


Change spelling of Github to GitHub

On the createcwu page

https://github.com/BCDevExchange/devex/commit/4857287844b1f4718873c3939cab3614fee702a9


Support for Franca IDL and Franca deployment models.

I created language support for the two languages of the Franca project:

"franca": Franca IDL
"fdepl": Franca Deployment models

Franca is an open-source project which provides an Interface Definition Language (IDL) and tools around this IDL. It is used heavily in the Automotive domain (e.g., for building car navigation systems), but also in other domains. Here is Franca's github page for more information.
I introduced a new category "idl". I didn't find a category with the same semantics, so I suppose it should be added. There are more IDLs, e.g. Thrift, Protobuf, OMG IDL. Probably someone will add them in future and finds the category useful.
Disclaimer: I am quite new to the highlight.js business :-). I used the "java" language support as a starting point. The highlighting for Franca seems to work here, but there might be some room to improve implementation-wise. Please feel free to suggest improvements.

It seems the auto detection mixes up the existing "stylus" language and my new "fdepl" language. How can this be avoided? The language snippet of "stylus" looks quite different from "fdepl" files...

You could leave detection problems as is for now, we'll look into it when time allows. Thanks!

@kbirken
First of all, truly sorry for the long delay in anyone responding to this PR.
Thank you so much for desiring to contribute and writing a language grammar.  Unfortunately, we currently are no longer accepting new languages into the core repository.  The existing maintainers simply don't have time to maintain new languages (fixing bugs, dealing with issues, etc).  We'd still love to host your language as a 3rd party module though - if you're willing to help maintain it.
It's easy to turn your language into a module that anyone who likes can easily integrate with Highlight.js using a single line of code:
<script src="/path/to/highlight.min.js"></script>
<!-- they'd simply add your language modules -->
<script src="/path/to/highlightjs-franca.js"></script>
<script src="/path/to/highlightjs-fdepl.js"></script>
<script>hljs.initHighlightingOnLoad();</script>

It's also possible for users to rebuild the highlight.js library to include specific 3rd party modules, all in a single file (vs loading a bunch of 3rd party languages one at a time).
If you'd like to become the maintainer for this language then we're happy to create a new repository in the highlightjs organization and host the repository there.  That assumes you'd be open to reviewing issues and fixing bugs as they arise.  You're also welcome to host the repo yourself and we'll link to it from our README.
There isn't much in the way of how to docs on this yet, but there are sample repositories and it's honestly pretty easy.  We're still fleshing out an official "spec" for 3rd party modules, but even after that's done it should require minimal changes from the examples we have already.  IE, if you wanted to get started now there isn't any big reason to wait.
My own robots.txt grammar repository is a great, simple example:
https://github.com/highlightjs/highlightjs-robots-txt
If you have any questions feel free to ask.

Closing for inactivity; (also finally cleaning all these old PRs out for the new year).
As mentioned above if you're interested in becoming the maintainer of a 3rd party module, please respond [whenever you find time] and we'd be happy to help you get started.


Custom modules.paths.app_folder value not working

When the config/modules.php value for modules.paths.app_folder is changed (from the default app/ to src/ for instance), the module will be generated using the old namespace format with the app\ path in lowercase.

The module composer.json didn't reflect the custom app_folder, it's still using app/.

The Service Provider and every other class were created with the wrong namespace. All the namespaces have the initial app\ segment and are in lowercase.
The custom app_folder 'src/' wasn't generated, instead the 'app/' is used.
Is there any workaround for this, please?
@nWidart @dcblogdev @alissn
Thanks for the love.

Please, I'd like you to reopen this issue as it hasn't been addressed. Sorry for the late response; I got carried away with work.
The modules.paths.app value is not reflected in the namespace, creating confusion between default and custom namespace.
Expected results

'app/'
This is the default Laravel value and it's great we maintain it. However, the value should be reflected in the autoload.psr-4 namespace (key), and its corresponding path (value).
Right now, the modules.paths.app is being forcefully removed from the namespace (ltrim(config('modules.paths.generator.model.path', 'app/Models'), config('modules.paths.app_folder', ''))), which makes all the options autoload.psr-4 keys look-alike ("Modules\\Example\\": "...",), that we can all agree it's a bad practice and opinionated way of coding an open source!
Forcefully removing the modules.paths.app value from the namespace, and forcing all the options autoload.psr-4 keys look-alike will create confusion, and make it difficult to reverse engineer (converting a "namespace" to "path" and vice versa).
"Modules\\Example\\App\\": "app/",

'src/'
Here, the custom value (src) should be reflected in the autoload.psr-4 namespace, and the corresponding path.
"Modules\\Example\\Src\\": "app/",

'/'
Here, the custom value (/) indicates no app/ path, the module's root path will be used instead.
"Modules\\Example\\": "/",

''
Here, the modules.paths.app value is empty (``), and will therefore fall back to the default value (app/).
"Modules\\Example\\App\\": "app/",

Thanks for your continuous dedication, contribution and support for this package.
@nWidart  @dcblogdev  @alissn


Issue templates are misbehaving

URL to the relevant documentation
Bug or New content issue template
Please describe the UI problem.
The issue templates (this has happened on Bug and New content) are making you fill out non-required fields before you can submit the issue.  Here's a snippet from the New content template.  You can see that the URL one is not marked required.

type: textarea
attributes:
label: URL, if applicable
description: Provide a URL and section name/paragraph where the new content belongs, if applicable.
validations:
required: false

type: textarea
attributes:
label: Describe the new content you are requesting.
description: >
Please give a clear and concise description. What value does the new content bring?
validations:
required: true

Hm. I tried both types of issues and I was able to submit without filling in the URL field. But if I didn't have the title field filled out, the submit button remained unclickable. Had you added a title to the issues you tried out @beckykd ?

Can this be closed?

Here's something odd.  If I fill out just the required information, then scroll up and back down, I can submit it.  But otherwise, I can't:

Maybe we just can't do anything about that.

Weird.

Maybe we just can't do anything about that.

I think the best we can do is file a bug report. A lot of times devs don't know about bugs like this, so it's really helpful for them to submit, if you're willing https://support.github.com/contact/bug-report
I'm going to close this for our repo as not fixable by us.


Co by area and date

Add "Collection objects by area and date" task to application and hub tasks list.
No database changes required,
One javascript library (RangePicker) added to .../src/.

Coverage increased (+0.2%) to 89.747% when pulling 30d41c3529c89a2dcbe7b35ef7ec9c0f82f1f1f6 on CO_by_area_and_date into 172193663a54e4ee4047d03e140e64eeddd195f2 on development.

Coverage increased (+0.1%) to 89.695% when pulling 775bdae33870df6d212a31ea6213138efeb0476e on CO_by_area_and_date into 172193663a54e4ee4047d03e140e64eeddd195f2 on development.

Coverage increased (+0.1%) to 89.695% when pulling 775bdae33870df6d212a31ea6213138efeb0476e on CO_by_area_and_date into 172193663a54e4ee4047d03e140e64eeddd195f2 on development.

@jrflood @TuckerJD
Thanks for the code, came together pretty well.  Before we pull can you do a couple things (just keep committing and pushing to this branch, the pull request will be updated automatically).

rebase this against development (again), (deterministic) tests should all be passing now
Add some detail to the pull request comment, and minimum give us a bullet list of 3-5 items that describes the functionality added; describe what the javascript library does, or link to it in the description.  It's added to /src/vendor, not /src/?  What license is this code under?  Can we add some comments at the top of the library to indicate origin/license etc.
There are significant map related changes here- what are they?  Describe them in the pull request.
Address the comments in the code review.

Coverage increased (+0.1%) to 89.571% when pulling 79885ef3334060c71f9fce46d7c8b3a185f5c938 on CO_by_area_and_date into 90cf15cdb6d124dc4d353cf7da5ef93cadf7c32e on development.


fix: activity item click area too small

Try out this version of the Hiro Wallet - download extension builds.

extends the click area of activity transaction item.
cc/ @aulneau @kyranjamie @fbwoolf

@kyranjamie can you please review ?


1.8 - 1.18 Compatibility

I honestly don't know why people still use it but as people keep asking me to support it, I'd be thankful for someone to add 1.8 - 1.18 compatibility this is in no need priority at all.
Taks needing done:

[ ]  Module up the Plugin with the Compatibility module, WE6 & WE7
[ ] Add support for WorldEdit 6 schematics and resetting, etc.
[ ] Add support for World Edit 7 schematics and resetting, etc.
[ ] Tested the pasting and resetting works correctly
[ ] Release the Jar.

There is no rush and no ETA on when this'll be done, it's just here.

Done in another repository.


[Feature] Add Island Name Banner

Add island name banner

I was thinking of a generator. Anyways I found a GitHub project and currently modifying it to be an app in the HID tool. Project is at https://github.com/FlynnFarrow/ACNHIslandNameGenerator

Here's how it looks currently.


Fails to run Local Klaytn Network

When I run following command after pulling master branch, installing required packages, and running ganache without any configuration changes, it prints out an error.
➜  klaytn-contracts git:(master) ✗ npm run run:klaytn

><EMAIL_ADDRESS>run:klaytn /Users/kakao/work/klaytn-contracts
> ./test/prepare.sh

~/work/klaytn-contracts/test ~/work/klaytn-contracts
~/work/klaytn-contracts ~/work/klaytn-contracts/test
./init_accounts.sh: line 7: jq: command not found
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (7) Failed to connect to localhost port 8551: Connection refused
FUND_ADDR =
curl: (7) Failed to connect to localhost port 8551: Connection refused
(node:56739) UnhandledPromiseRejectionWarning: TypeError: Cannot read property '0' of undefined
    at /Users/kakao/work/klaytn-contracts/test/init_accounts.js:9:26
    at Object._fireError (/Users/kakao/work/klaytn-contracts/node_modules/caver-js/packages/caver-utils/src/index.js:63:9)
    at /Users/kakao/work/klaytn-contracts/node_modules/caver-js/packages/caver-core-method/src/index.js:255:22
    at /Users/kakao/work/klaytn-contracts/node_modules/caver-js/packages/caver-core-requestmanager/src/index.js:151:28
    at XMLHttpRequest.request.onreadystatechange (/Users/kakao/work/klaytn-contracts/node_modules/caver-js/packages/caver-core-requestmanager/caver-providers-http/src/index.js:122:13)
    at XMLHttpRequestEventTarget.dispatchEvent (/Users/kakao/work/klaytn-contracts/node_modules/xhr2-cookies/dist/xml-http-request-event-target.js:34:22)
    at XMLHttpRequest._setReadyState (/Users/kakao/work/klaytn-contracts/node_modules/xhr2-cookies/dist/xml-http-request.js:208:14)
    at XMLHttpRequest._onHttpRequestError (/Users/kakao/work/klaytn-contracts/node_modules/xhr2-cookies/dist/xml-http-request.js:349:14)
    at ClientRequest.<anonymous> (/Users/kakao/work/klaytn-contracts/node_modules/xhr2-cookies/dist/xml-http-request.js:252:61)
    at ClientRequest.emit (events.js:400:28)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:56739) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:56739) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
~/work/klaytn-contracts/test
~/work/klaytn-contracts

But, all tests just work well with $ npm run test:ganache.

Resolved after installing jq


⚠ ClamAV has degraded performance

In 1d5b4d9, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 91 ms

Resolved: ClamAV performance has improved in dd802b6 after 5 minutes.


Status E fault function does not work

Describe the bug
I had the opportunity to test my SmartEVSE with a test device and found the following issues:
Status C+E: -12V on the CP pin becomes smaller (diode error detection) Charging does not stop
Upload your config
{"version":"v3.6.10","serialnr":6622,"mode":"SMART","mode_id":3,"car_connected":false,"wifi":{"status":"WL_CONNECTED","ssid":"iot","rssi":-52,"bssid":"6A:D7:9A:24:A1:8D"},"evse":{"temp":28,"temp_max":65,"connected":false,"access":true,"mode":1,"loadbl":0,"pwm":1024,"solar_stop_timer":0,"state":"Ready to Charge","state_id":0,"error":"None","error_id":0,"rfid":"Not Installed"},"settings":{"charge_current":0,"override_current":0,"current_min":6,"current_max":20,"current_main":50,"current_max_circuit":25,"current_max_sum_mains":600,"max_sum_mains_time":0,"solar_max_import":0,"solar_start_current":6,"solar_stop_time":1,"enable_C2":"Solar Off","mains_meter":"API","starttime":0,"stoptime":0,"repeat":0},"mqtt":{"host":"<IP_ADDRESS>","port":1883,"topic_prefix":"SmartEVSE","username":"Lehm-WG","password_set":true,"status":"Connected"},"home_battery":{"current":0,"last_update":0},"ev_meter":{"description":"Eastron3P","address":11,"import_active_power":0,"total_kwh":734.0999756,"charged_kwh":0,"currents":{"TOTAL":0,"L1":0,"L2":0,"L3":0},"import_active_energy":734.0999756,"export_active_energy":0},"mains_meter":{"import_active_energy":0,"export_active_energy":0},"phase_currents":{"TOTAL":6,"L1":-13,"L2":6,"L3":13,"last_data_update":1728319380,"original_data":{"TOTAL":6,"L1":-13,"L2":6,"L3":13}},"backlight":{"timer":113,"status":"ON"}}
Upload a debug log
(I) (EVSEStates)(C1) Cable limit: 20A  Max: 20A
(D) (IsCurrentAvailable)(C1) Current available checkpoint D. ActiveEVSE increased by one=1, TotalCurrent=0.0A, StartCurrent=6A, Isum=1.0A, ImportCurrent=0A.
(C1) 19:20:46 STATE A -> B
(C1) Switching Contactor1 OFF.
(C1) Switching Contactor2 OFF.
(C1) Diode OK
(D) (Timer100ms)(C1) ModbusRequest 4: Request Energy Node 0
(D) (Timer100ms)(C1) ModbusRequest 20: Request EVMeter Current Measurement Node 0
(D) (CalcBalancedCurrent)(C1) Checkpoint c: Resetting SolarStopTimer, MaxSumMainsTimer, IsetBalanced=25.0A, saveActiveEVSE=0.
(I) (printStatus)(C1) STATE: B Error: 0 StartCurrent: -6 ChargeDelay: 0 SolarStopTimer: 0 NoCurrent: 0 Imeasured: 1.7 A IsetBalanced: 25.0 A, MainsMeter.Timeout=3, EVMeter.Timeout=64.

(D) (CalcBalancedCurrent)(C1) Checkpoint c: Resetting SolarStopTimer, MaxSumMainsTimer, IsetBalanced=25.0A, saveActiveEVSE=0.
(C1) 19:20:48 STATE B -> C
(C1) Switching Contactor1 ON.
(C1) Switching Contactor2 ON.
(D) (Timer100ms)(C1) ModbusRequest 4: Request Energy Node 0
(D) (Timer100ms)(C1) ModbusRequest 20: Request EVMeter Current Measurement Node 0
(I) (printStatus)(C1) STATE: C Error: 0 StartCurrent: -6 ChargeDelay: 0 SolarStopTimer: 0 NoCurrent: 0 Imeasured: 2.4 A IsetBalanced: 25.0 A, MainsMeter.Timeout=10, EVMeter.Timeout=64.
(I) (printStatus)(C1) L1: -1.8 A L2: 0.6 A L3: 2.4 A Isum: 1.2 A
(D) (Timer100ms)(C1) ModbusRequest 4: Request Energy Node 0
(D) (Timer100ms)(C1) ModbusRequest 20: Request EVMeter Current Measurement Node 0
(D) (Timer100ms)(C1) ModbusRequest 20: Request EVMeter Current Measurement Node 0
(I) (printStatus)(C1) STATE: C Error: 0 StartCurrent: -6 ChargeDelay: 0 SolarStopTimer: 0 NoCurrent: 0 Imeasured: 2.4 A IsetBalanced: 25.0 A, MainsMeter.Timeout=8, EVMeter.Timeout=64.
(I) (printStatus)(C1) L1: -1.8 A L2: 0.6 A L3: 2.4 A Isum: 1.2 A
(D) (Timer100ms)(C1) ModbusRequest 4: Request Energy Node 0
(D) (Timer100ms)(C1) ModbusRequest 20: Request EVMeter Current Measurement Node 0
(D) (Timer100ms)(C1) ModbusRequest 20: Request EVMeter Current Measurement Node 0
(I) (printStatus)(C1) STATE: C Error: 0 StartCurrent: -6 ChargeDelay: 0 SolarStopTimer: 0 NoCurrent: 0 Imeasured: 2.4 A IsetBalanced: 25.0 A, MainsMeter.Timeout=6, EVMeter.Timeout=64.

(C1) Switching Contactor2 OFF.
Status E was triggert 19:20:50
To Reproduce
Steps to reproduce the behavior:

What Mode are you in?  Smart
What steps do you take so the problem arises?
Simulation with the tester:
Switch 1: Vehicle plugged in = status B
Switch 2: Request charge = status C
Switch 3: Short circuit of the diode = status E
===> charging process is not interrupted!

Expected behavior
If a status E occurs, I expect the charging to be switched off, as if there was an RCM error.

Yes, it seems to be twice as much, but we Germans seem to interpret the examination guidelines more strictly. Please give me until the weekend to check whether the DGUV complies with the IEC 61851-1 standard or whether something has changed.
If everything is OK, I will then close the issue.

I bought the EN IEC 61851-1: The current one is from 2019 and has been corrected twice (last correction april 2024).
In Table A.4, Detected by the electric vehicle power supply device
System states  the footnote in state C2/D2 is:
The electric vehicle power supply device shall support the low level of the PWM signal of −12 V, the presence
the diode, before closing the supply switching device on the power supply device for electric vehicles at least
check once.

There is more detail in the A4 test procedure area
A4.9 Pilot Voltage Short Circuit Test states:
This test verifies compliance with the timing requirements in Table A.6, Procedure 12
checked.
The test begins with the nominal values of the resistors R2test and R3test. If condition C or D
lasted at least 5 s, an additional resistor of 80 Ω is used as a connection between the
Pilot conductor and protective conductor are connected.

Diagramm from Picture of Tabel A6. Procedure 12
Curve 1: PWM signal; Curve 2: Signal at the contactor; Curve 3: Auto charging requirements; Curve 4: Power consumption from the car
(Sorry picture in German)

If there is a disturbance in the PWM signal, the SmartEVSE must switch off the contactor c1 within 3 seconds
In my opinion, there is an issue with SmartEVSE

Hello @Elektrofix-OL
The SmartEVSE checks before closing the contactor, once if the low level of the PWM signal is there. (so it checks for the diode) As required by the standard.
But you are right, once it's charging it should also detect if the signal is disturbed somehow (shorted to ground for example).
I will add the check for a shorted or otherwise disturbed signal to the code.
Thanks for bringing this to our attention!

I tested it: failed
Test 1:
Switch 1: Vehicle plugged in = status B
Switch 2: Request charge = status C
Switch 3: Diode short circuit = status E
===> Charging process is not interrupted!
Switch 2: OFF = status B&E
Delay 2 sec.
Switch 2: Request charge
===> No Charging = OK
Test 2:
Switch 3: Diode short circuit = status E
Switch 1: Vehicle plugged in = status B
Switch 2: Request charge = status C
===> No Charging = OK
Test 3:
Switch 1: Vehicle plugged in = status B
Switch 3: Diode short circuit = status E
Switch 2: Request charge = status C
===> Charging starts even though diode is short-circuited!
Short circuit 0 ohms between CP and PE
==> Loading stops
Short circuit removed
===> Charging starts automatically! Dangerus!!
If such an error occurs, it would be good if this error were displayed and had to be confirmed as with the RCM error. Automatic connection is dangerous.
I only had a normal multimeter for this test, so unfortunately I can't see what the voltages were. I checked my tester after the test, diode and resistance values were OK.
I hope my results help you improve your software.

@Elektrofix-OL
Diode short circuit is not the same as state E.
State E is 0V so the pilot signal at GND level or floating (not connected).
Test 1:
The diode test is performed immediately after entering state B, so if you short the diode while in State C it will not be able to detect it. This is however still according to spec, as the test was performed before entering State C.
Test 2:
As your second test shows, if you start with a shorted diode it will not be possible to switch to state C.
Test 3:
See above, the diode test was already performed before you short circuited it. So it was able to enter State C.
When you place a short between CP and PE, it will stop, and switch back to State B
When you remove the short, it will immediately do a diode test again, and if the diode is detected switch to State C.
But if you leave the diode shorted, it will not switch to State C, but remain in State B.
The short circuit test mentioned in A.4.9 was not detected previously. My IEC_61851-1 uses a resistor of 120 Ohm between PE and CP, i have tested 0 / 80 / 120 Ohm and all of these values now successfully interrupt charging.
I don't think we should stop with an error that requires user interaction.
As the diode test is always performed, it's safe to resume charging.

Diode short circuit is not the same as state E. State E is 0V so the pilot signal at GND level or floating (not connected).

Yes, I understand status E = CP -1V to 1V and status F -13V to -11V, they are different. It is not stated in EN 61851 that it has anything to do with the diode. I just realized that.

Test 1: The diode test is performed immediately after entering state B, so if you short the diode while in State C it will not be able to detect it. This is however still according to spec, as the test was performed before entering State C.

@mstegen  I'm not entirely convinced. It would be better to test the diode immediately before changing from status B to C and not so much "minutes" before. A car that balances the cells stays in B and later switches back to C to recharge.

Test 2: As your second test shows, if you start with a shorted diode it will not be possible to switch to state C.

That's how it should be

Test 3: See above, the diode test was already performed before you short circuited it. So it was able to enter State C. When you place a short between CP and PE, it will stop, and switch back to State B When you remove the short, it will immediately do a diode test again, and if the diode is detected switch to State C. But if you leave the diode shorted, it will not switch to State C, but remain in State B.

I did my test 3: three times in a row this afternoon and the error was always there: switching on despite a diode short circuit.  Then I briefly installed V3.6.10 and then again the one from yesterday. Now I can't recreate the error in test 3.
Confirmed: After the short circuit the diode check is there and only when I remove the diode short circuit does it switch on again.

I don't think we should stop with an error that requires user interaction. As the diode test is always performed, it's safe to resume charging.

I agree with you that the diode check shouldn't necessarily lead to a user action, but the short circuit between CP and PE should. In addition, a short circuit between CP and PE always indicates a defective charging cable or defective electronics in the car. Ideally, this shouldn't happen, but my experience shows that people are not careful with cables and tend to drive over them and damage the wires. Forklift drivers are particularly pain-free.

I'll see if i can move the diode check to just before switching to State C. or add a second diode check.
Again thanks for bringing the short circuit detection between CP and PE to my attention.


Sidebar Category Title / Non-collapsable primary categories.

🚀 Feature
I'd love to see a title or non collapsable category for sidebar items, in the style that gitbook does https://docs.gitbook.com/
This helps break up ideas without removing all context. And it just looks cleaner than having dropdown arrows pointing all over the place in a non collapsed landing page

You can already make sidebar categories non-collapsible:
https://docusaurus.io/docs/sidebar#collapsible-categories

This feature makes every category non-collapsable. The feature I'm talking about makes only the top-level directory non-collapsible, so the user can see a few collapsed categories inside of several top-level directories
In the below picture, the "Guides" "Reference" and "Concepts" sections would be non-collapsable, while the internal directories would be collapsed.

I see, so using a sidebar category.collapsible = false attribute make sense to override the default global setting with more granularity?

Exactly, I think it helps communicate the context of the top-level categories without adding too much visual complexity

@Yorkemartin I implemented this with #5203. Do you think it would be necessary for us to also have a "category title" item, just like in gitbook, or is a non-collapsable category good enough?


src layout with package not in name of project

Hi and thanks for the great package manager.
If I have a src folder with three modules: t1, t2 and t3, and I name my project "a_project" in pyproject.toml, I have an error: ValueError: No file/folder found for module a_project. Is that intended behavior?
For now the only solution I found is to have an empty folder named a_project with just a __init__.py file in it to be able to install all modules.

That is intended behaviour, I'm afraid, and your solution probably won't work as well as it seems. Flit is designed to package a single top-level package, so in that scenario you should put t1, t2, t3 inside your a_project folder and import them them from a_project import t1 (or from a_project.t1 import something). Within the package, you can also use relative imports (from . import t1).
Your package (the a_project folder) is allowed to have a different name from the distribution (what you pip install) - see https://flit.pypa.io/en/latest/pyproject_toml.html#module-section for that.


[Swashbuckle 5.1.4] Angle brackets in CDATA are encoded in Swagger

Hi,
I added the following comments for my method. When I check my Swagger file, the angle brackets are encoded. However I don't want the "<" to be encoded to be <. What can I do?
Code:
    /// <remarks>
    /// <![CDATA[Create user. This is longer explanation how to use the
    /// API. This description can contain rich text formatting.]]>
    /// </remarks>
    public void Post(User user) { }

The Swagger generated by Swashbuckle 5.1.4:
"description":"Create user. This is longer explanation how to use the \r\n            API. This description can contain <strong>rich</strong> <code>text</code> formatting."
Thanks,
Peter

Correct me if I'm wrong but I think this is really a duplicate of #258 which is really just asking for better support to add richer markup in the description fields

Shouldn't this be written as markdown? e.g.
/// Create user. This is longer explanation how to use the
/// API. This description can contain **rich** `text` formatting.


In Query Manager, add feature to copy content of a certain cell instead copying text of a query

Sometimes user wants to copy the error message (if it is very long), not the sql query from Query Manager panel.
Now it always copies the sql query.
As a good example, copy the certain cell works nice in Metadata panel.
It copies the cell the user right clicked to and chose Copy in a context menu.
I suggest to use the same behavior here, in Query Manager.
https://github.com/user-attachments/assets/542f7de0-a1c2-4b2f-8b5d-7dd3ef7308f8

Thank you for idea


Add pt-online-schema-change to help run migrations on large tables

What happened?
For apps with very large datasets, pt-online-schema-change is useful for running migrations. It would be great if the stack included it or if there were a buildpack to install it.

Hey @genevieve, the use-case for this package is a little to specific for us to add it into the stacks but it could be a good fit for a buildpack. Feel free to open an RFC here.

Hey @genevieve, the use-case for this package is a little to specific for us to add it into the stacks but it could be a good fit for a buildpack. Feel free to open an RFC here.


beginner's tutorial

could you please add some beginner's tutorial on how to use this template for developers like me, who is new to chrome extensions. thanks

to be more specific, say that, I'm going to build an extension, in which metamask login involved. so I think i should open newTab to login, after that, all pages should share the login state. there's already a lib to wrap metamask login rainbowkit. but in a normal react app, if you wanna use rainbowkit, the <App /> should be wrapped by a rainbowkit provider. but for a chrome extension using your template, i didn't find a entrypoint like in react app. so i don't quite clear where should i use this provider so that i could use the lib.
please help me with this, thanks.

@Azleal That's the problem with the lack of knowledge of 'how chrome extension works', that's not related to this template.
But nonetheless after latest big refactor, with turborepo, the complication with beggining of developing template increase.
Either I as a contributor need to spend some times to know how the turborepo work and how to continue developing.
@Jonghakseo The more specific README should be write in the near future.
I'm close this, but let's watch this tutorials to know sth about extensions overall and react with that:
https://www.youtube.com/watch?v=0-5mPIJuwJ8&list=PLBS1L3Ug2VVrTlexfI5i9OB0KpNfIjeeN
https://www.youtube.com/watch?v=rAZXWkVhCgg&list=PLBS1L3Ug2VVods9GnWbJc__STt9VnrJ9Z


deploy with the robot tests

Codecov Report

Merging #16 (1378c5a) into main (7b604c3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #16   +/-   ##
=======================================
  Coverage   17.85%   17.85%
=======================================
  Files           3        3
  Lines          28       28
  Branches        3        3
=======================================
  Hits            5        5
  Misses         23       23

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 7b604c3...1378c5a. Read the comment docs.


Kubeadm with unstable deb repo

Following #140, my findings:
Context:

3 RPI 3, 2 RPI2
All are on HypriotOS 1.1.1
Master is a RPI3

$ sudo kubeadm init --use-kubernetes-version=v1.4.5
Running pre-flight checks
<master/tokens> generated token: "4ed849.29f2eee78f75d062"
<master/pki> generated Certificate Authority key and certificate:
Issuer: CN=kubernetes | Subject: CN=kubernetes | CA: true
Not before: 2016-10-30 20:47:52 +0000 UTC Not After: 2026-10-28 20:47:52 +0000 UTC
Public: /etc/kubernetes/pki/ca-pub.pem
Private: /etc/kubernetes/pki/ca-key.pem
Cert: /etc/kubernetes/pki/ca.pem
<master/pki> generated API Server key and certificate:
Issuer: CN=kubernetes | Subject: CN=kube-apiserver | CA: false
Not before: 2016-10-30 20:47:52 +0000 UTC Not After: 2017-10-30 20:48:12 +0000 UTC
Alternate Names: [<IP_ADDRESS> <IP_ADDRESS> kubernetes kubernetes.default kubernetes.default.svc kubernetes.default.svc.cluster.local]
Public: /etc/kubernetes/pki/apiserver-pub.pem
Private: /etc/kubernetes/pki/apiserver-key.pem
Cert: /etc/kubernetes/pki/apiserver.pem
<master/pki> generated Service Account Signing keys:
Public: /etc/kubernetes/pki/sa-pub.pem
Private: /etc/kubernetes/pki/sa-key.pem
<master/pki> created keys and certificates in "/etc/kubernetes/pki"
<util/kubeconfig> created "/etc/kubernetes/admin.conf"
<util/kubeconfig> created "/etc/kubernetes/kubelet.conf"
<master/apiclient> created API client configuration
<master/apiclient> created API client, waiting for the control plane to become ready
<master/apiclient> all control plane components are healthy after 167.263559 seconds
<master/apiclient> waiting for at least one node to register and become ready
<master/apiclient> first node is ready after 1.542673 seconds
<master/apiclient> attempting a test deployment
<master/apiclient> test deployment succeeded
<master/discovery> created essential addon: kube-discovery, waiting for it to become ready
<master/discovery> kube-discovery is ready after 18.030144 seconds
<master/addons> created essential addon: kube-proxy
<master/addons> created essential addon: kube-dns

Kubernetes master initialised successfully!

You can now join any number of machines by running the following on each node:

kubeadm join --token=4ed849.29f2eee78f75d062 <IP_ADDRESS>

$ kubectl get ns
NAME          STATUS    AGE
default       Active    5m
kube-system   Active    5m

$ kubectl get cs
NAME                 STATUS      MESSAGE                                                                                ERROR
controller-manager   Unhealthy   Get http://<IP_ADDRESS>:10252/healthz: dial tcp <IP_ADDRESS>:10252: getsockopt: connection refused
scheduler            Healthy     ok
etcd-0               Unhealthy   HTTP probe failed with statuscode: 503

On a worker node:
$ sudo kubeadm join --token=4ed849.29f2eee78f75d062 <IP_ADDRESS>
Running pre-flight checks
<util/tokens> validating provided token
<node/discovery> created cluster info discovery client, requesting info from "http://<IP_ADDRESS>:9898/cluster-info/v1/?token-id=4ed849"
<node/discovery> cluster info object received, verifying signature using given token
<node/discovery> cluster info signature and contents are valid, will use API endpoints [https://<IP_ADDRESS>:6443]
<node/bootstrap> trying to connect to endpoint https://<IP_ADDRESS>:6443
<node/bootstrap> detected server version v1.4.5
<node/bootstrap> successfully established connection with endpoint https://<IP_ADDRESS>:6443
<node/csr> created API client to obtain unique certificate for this node, generating keys and certificate signing request
<node/csr> received signed certificate from the API server:
Issuer: CN=kubernetes | Subject: CN=system:node:pico-worker1 | CA: false
Not before: 2016-10-30 20:56:00 +0000 UTC Not After: 2017-10-30 20:56:00 +0000 UTC
<node/csr> generating kubelet configuration
<util/kubeconfig> created "/etc/kubernetes/kubelet.conf"

Node join complete:
* Certificate signing request sent to master and response
  received.
* Kubelet informed of new secure connection details.

Run 'kubectl get nodes' on the master to see this machine join.

And when I create the weave-kube (after I joined a first node btw) :
$ sudo kubectl create -f https://raw.githubusercontent.com/kodbasen/weave-kube-arm/master/weave-daemonset.yaml
daemonset "weave-net" created

And then:
$ kubectl get nodes
NAME           STATUS    AGE
pico-master    Ready     13m
pico-worker1   Ready     2m
HypriotOS/armv7: pirate@pico-master in ~
$ kubectl get cs
NAME                 STATUS    MESSAGE              ERROR
scheduler            Healthy   ok
controller-manager   Healthy   ok
etcd-0               Healthy   {"health": "true"}
HypriotOS/armv7: pirate@pico-master in ~
$ kubectl get ns
NAME          STATUS    AGE
default       Active    13m
kube-system   Active    13m
HypriotOS/armv7: pirate@pico-master in ~

So maybe I didn"t wait enough for etcd at the first output above ?
Let me know if you need something else.

With the 4 RPi jointed into the cluster, load average is between 5 and 8.

Interesting, which component is taking the most load?

Seems it starts to lower slowly...
From htop, in terms of CPU (with no special order)

kubelet
dockerd
kube-apiserver

I'm now back to a load average of 2.11... just passed to 1.18... still lowering. So seems the init phase was quite ressource heavy. Will let the cluster as is to see till where it goes before doing any new action.

... but it increases or lowers... workers were quite calm so far, most are still (Load average  0 < LA < 2 for 1 RPI3 and 2 RPI2), but one among them is quite active with LA up to 4.45 (RPI3)

Some findings ; if you want output of special commands, just gave them to me :)
$ kubectl get pods --all-namespaces
NAMESPACE     NAME                                  READY     STATUS             RESTARTS   AGE
kube-system   dummy-2501624643-ox4bu                1/1       Running            0          34m
kube-system   etcd-pico-master                      1/1       Running            1          34m
kube-system   kube-apiserver-pico-master            1/1       Running            2          34m
kube-system   kube-controller-manager-pico-master   0/1       CrashLoopBackOff   7          34m
kube-system   kube-discovery-2202902116-6d9c4       1/1       Running            0          33m
kube-system   kube-dns-2334855451-kio06             3/3       Running            0          33m
kube-system   kube-proxy-3cw9z                      1/1       Running            0          33m
kube-system   kube-proxy-3efr7                      1/1       Running            0          13m
kube-system   kube-proxy-6yhmv                      1/1       Running            0          13m
kube-system   kube-proxy-l1gy0                      1/1       Running            0          13m
kube-system   kube-proxy-oygdb                      1/1       Running            0          24m
kube-system   kube-scheduler-pico-master            0/1       CrashLoopBackOff   7          33m
kube-system   weave-net-6kro4                       2/2       Running            0          23m
kube-system   weave-net-aab9k                       2/2       Running            0          23m
kube-system   weave-net-b1ib6                       2/2       Running            0          13m
kube-system   weave-net-qraay                       2/2       Running            0          13m
kube-system   weave-net-rswrd                       2/2       Running            1          16m

Seems there are still some etcd issues:
HypriotOS/armv7: pirate@pico-master in ~
$ sudo kubectl create namespace sock-shop
Error from server: client: etcd member http://<IP_ADDRESS>:2379 has no leader

After 1 day with no activities on it:

Master is right now with 3 < LA  < 5
Workers looks calm with LA  < 1
On master; my top 3 CPU process are kubelet, dockerd and kubescheduler
and:

HypriotOS/armv7: pirate@pico-master in ~
$ kubectl cluster-info
The connection to the server localhost:8080 was refused - did you specify the right host or port?
HypriotOS/armv7: pirate@pico-master in ~
$ kubectl get cs
The connection to the server localhost:8080 was refused - did you specify the right host or port?

Hi,
Till a later release, I'll use my 5 RPi to build a simple docker-swarm cluster and my 2 cubietrucks with HD as a glusterFS endpoint.
Aside of it, I'll test K8S on 64 bits infra to be more famillair with first and then come back to ARM version ;-)


Arch_Online installer discards download after being blocked

P.S. This isn't a bug, but it's some basic missing functionality that would be very helpful.
I was using the ArchOnline installer for v<IP_ADDRESS>, and it was blocked by Microsoft Defender after downloading the entire file and then attempting to install, resulting in MS Defender killing the terminal process.

That's fine, I thought, but the installer doesn't cache the download, forcing me to redownload it. It shouldn't be as big a deal as it is, but low bandwidth issues are causing more of a hassle than it should be, especially for an error as common as forgetting to disable MS Defender before running the executable. From my (in)experience a simple fix would be to cache the download after completion, so that should an error stop the process, it's easy to try the installation again, which would be especially useful in a scenario where many different errors could pop up from improper setup.
Enviroment:

Windows build number: 10.0.19045.4291
Security Software: MS Defender with Real-time protection enabled
WSL version 1/2: WSL2
ArchWSL version <IP_ADDRESS>
ArchWSL Installer type zip/appx/other clean/update: Arch_Online.zip

Defender is blocking because it is (rightly or wrongly) detecting Trojan:Win32/Vigorf.A in the rootfs tar.

The zip version is being flag by defender on my system to.

Defender is blocking because it is (rightly or wrongly) detecting Trojan:Win32/Vigorf.A in the rootfs tar.

Yes, it's an issue with the installer, but thats not the purpose of this issue. This issue exists to suggest caching functionality for someone familiar with it to implement. If you want to, you can open a seperate issue about it.

Should I implement caching?
Admittedly, it can be a bit annoying to have to re-download the necessary files after a failed installation.
(I figured that would only happen a little bit, so I didn't worry about it.)
I don't think that in itself is difficult.
PS: I created a new build <IP_ADDRESS> and the file is no longer being false positiveed by defender.


Mod Conflict with EpicLoot Bounty/TreaureMap pins created by the Merchant on accepting Bounties/TreasureMaps

When installed with EpicLoot the Icon and Name strings are hidden from the Bounty/TreasureMap Indicators on the map. It doesn't break the actual Bounty or Treaure Map just the map feature.

Confirmed the issue is happening on Epic Loot version 0.9.37 and Better Cartography Table version 0.2.0. If you need assistance with fixing mod compatibility please reach out to me as I am currently helping maintain Epic Loot.

@OrianaVenture Thanks! I will have some time to look into this over the next week. I don't personally use Epic Loot, any tips on how to easily trigger / reproduce this situation?

Hi, you will have to locate the merchant first. Then accept a bounty or buy a treasure map. It's consistent every time.

any chance we'll see a fix for this? thanks

Yeah I actually started changing how stuff works internally so that's it's less likely that it clashes with other mods (including this specific case here), however I don't have any ETA to give because I'm currently quite busy dealing with stuff happening over at the Hades II modding scene now that the game came out😅

No worries, probably doesn't make sense to finalize the changes until they're verified against the Ashlands release. have fun with Hades II :D

This should be fixed in version 0.4.0.


No update radio

Hi, some radio don't work, because no radio link update, do you accept donation ? Thomas.

Hi there @uratom962,
I'm right in the middle of removing dead stations and adding new ones. Very soon pmrp will have all stations up and running.
Support and donations are always welcome. You can do so via,
coindrop
Thanks & Cheers!!!


I am using kafka tool to see the offset in zookeeper.

You should be able to view the offsets saved in zookeeper if you're using HighLevelConsumer.

I am using HighLevelConsumer. But even the consumer listen to a new message the the offset in the zookeper remains the same. And if i start the consumer again it start reading from the specific offset where it is stuck and not increasing. I remove the consumer group from kafka and restart the process and then it start working normally.

I do not know if it is a problem with kafka of the library itself but I had another issue with the library. It stopped reading the data from kafka. I also solved it by removing the consumer group from kafka.

That's strange behavior. How do you go about remove the consumer group from kafka?

bin/kafka-consumer-groups.sh --zookeeper localhost:2181 --delete --group consumer_group_name
This is the command to delete the consumer group from kafka.

are you able to use a later version of kafka?

Have not tried that is the cluster we have and that is were the problem happened. The issue is really rare not able to reproduce using any use case. I will try to see if i can reproduce it with later kafka version.


Implement Crumbs API for new AST

This task is automatically imported from the old Task Issue Board and it was originally created by Keziah Wesley.
Original issue is here.

Implement the same "Crumb" API as the old parser, for the new parser's types.
Notes
This will require use of Rc instead of Box in Tree to support structure-sharing.
This can be implemented with a derive macro, or using reflect.

Transition completed in #4113


Spaces/Tabs don't show in code blocks after editing

Please ensure:

[X] The bug is caused by Refined GitHub. It doesn't happen if I disable the extension.
[X] The bug happens after clearing extension cache. The "Clear cache" button below can also be found at the bottom of the options page.

Example URLs
https://github.com/os-js/osjs-server/pull/58#issuecomment-1166689579
Extension version
22.6.25
Browser(s) used
Brave
Description
When you post a block of code as part of a comment on a PR (and likely also on gists/issues), this extension shows the tabs and/or spaces that are included in the code block, indicated by either dots or arrows.
When you edit a block of code then save your edits, these dots and arrows are not shown until the user manually refreshes the page.
Include in this issue:

[ ] Screenshots/video/gif demonstrating the bug, if it’s visual
[ ] Console errors, if any

This has been discussed before but I can’t find it, it was probably for another feature.
There are some situations where GitHub updates comments and our features are subsequently lost. It would be great to catch these instances but I worry that it’d become too heavy to track.
We recently made some changes around that though:

https://github.com/refined-github/refined-github/issues/2852


[5.8] Refactoring the unless method with when in BuildsQueries class

The tests for "unless" are now failing.

WARNING : Don't merge this PR because of the inconsistency of the correct behavior
This is a breaking change because the value from the condition is altered by $this->when(! $value, ...).
I think this is inconsistent with Illuminate\Support\Collection::unless
So, what is the correct behavior ?

If this is a breaking change then please send it to master.


Deprecate --host and --port in favour of --bind-addr

So exactly what changed?

@antofthy If you run a prerelease, you can run --help to see the --bind-addr option and its description. It just combines --host and --port as described in the original issue. When we create a new release, we'll be sure to mention this option in the notes.

Thank you.


Issue importing './llms' submodule

I'm getting the following error when trying to run some of the examples locally.
Error: Package subpath './llms' is not defined by "exports" in /Users/jameszammit/Github/stuff/llm/node_modules/langchain/package.json

@zammitjames how did you resolve this issue?


Release/1.0.1

Fix tstamp parameter in track_self_describing_event()

coverage: 95.946%. remained the same when pulling ff56fcaca9dbb66ab93453d94e032c9641170ab6 on release/1.0.1 into f722a96d43917e855a17557cf896e7dc47b8e587 on master.


Warning: MIDI API is still unusable.

Since the initial release of the new MIDI API back in 2014 during the early days of Windows 8.x, this API has had a serious device enumeration flaw, and a crippling inability to transmit SysEx messages.  Microsoft has repeatedly promised to fix this.  It's 2018, 4 years out from when these flaws were first identified, and 3 years out from when these flaws became common knowledge and we still wait for a fix.  Please either remove this broken API, or fix it.  Presently, only the old Win32 Midi APIs are viable on Windows.  So for Windows 10 S users, our kids based on Microsoft's educational emphasis, there is no music.

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

ID: 90265caf-b1af-f015-674a-a015e3a35d65
Version Independent ID: bf193eb6-cd09-7538-12ef-da3cd0188e8d
Content: MIDI - UWP app developer
Content Source: windows-apps-src/audio-video-camera/midi.md
Product: windows
GitHub Login: @drewbatgit
Microsoft Alias: drewbat

There are so many bugs and glitches that anyone who writes music applications are disappointed. I already regret that I climbed entirely UWP. :-( It's no wonder that there are almost no musical applications - only toys.

I hope that a lot of activity in writing problems will cause Microsoft to stir. :-)

This issue is in a hopelessly neglected state.  All the devs that should be working on this within Microsoft are either purposely disregarding this issue (have directly contacted all of them) given the dozens of related postings since 2015, or Microsoft management has decided Music composition/creation under UWP is irrelevant.
Either way, it's clear this problem will never be resolved.  I for one have given up given how many bigger problems remain to be resolved with UWP.  Use the following search terms in combination to see all the related postings: UWP MIDI API BROKEN
The press is also a problem here.  They have been contacted and taken no action to look into this.  So much for oversight by the press.

or Microsoft management has decided Music composition/creation under UWP is irrelevant.

I think this is most likely. :-(

Hello, GitHub issues in this repository are for feedback specifically about the documentation. It seems like your issue is a developer support request. Here are all of the developer support resources:

The Feedback Hub app is the right place to lobby for a change, a feature, or to make a suggestion.
For community assistance with development questions, there are our forums, and StackOverflow.
The Windows developer support site explains other support options.
The Visual Studio developer community is available, and responsive. The former User Voice feature is now there on the Suggest a feature tab, if you want to lobby for a change, a feature, or to make a suggestion.
For developer issues, or suspected product bugs, there's Microsoft Support; a support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.

If you receive assistance via any support resource, and you believe that information would help us improve the documentation, then please come back and let us know.


File name isn't fully visible in Preview page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Lunch the app
Log in with any account
Select any user and tap on Plus button
Tap add attachment
Select from Galary

Expected Result:
File name fully visible in Preview page
Actual Result:
File name isn't fully visible in Preview page.
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Issue found by executing PR: https://github.com/Expensify/App/pull/6850
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Triggered auto assignment to @marcaaron (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Not sure if this is a deploy blocker since it seems fairly minor and don't exist on production because the feature is not implemented there and not because of a regression.
@mananjadhav will maybe want to fix this as a follow up to https://github.com/Expensify/App/pull/6850

@marcaaron I can add a fix for it.
@kbecciv I couldn’t understand the issue. When you say it isn’t fully visible ? Is it getting trimmed or you mean the color visibility?

@mananjadhav Yes, the file name is trimmed only, no issue with a color.

@marcaaron @shawnborton Can you help me get the intended behavior here? If the file name is too long this is how it looks.

Are we looking to trim it? At the moment I am reusing the subtitle of Header component. Also, are we still considering it a DeployBlocker?

No, not a deploy blocker.
It sounds like you are saying that anything with a "long subtitle" and using the Header component would cause this behavior and in that case probably does not need to be fixed before we can close out the linked issue.
As for a solution, probably would be best to loop in @Expensify/design. The two options seem to be:

extend the height of the header to accommodate the large text
use ellipsis to truncate the text

Triggered auto assignment to @mallenexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

I say we truncate. We might want to truncate the middle part so the file
extension is always visible too.
On Wed, Dec 22, 2021 at 4:05 PM MelvinBot @.***> wrote:

Triggered auto assignment to @mallenexpensify
https://github.com/mallenexpensify (External), see
https://stackoverflow.com/c/expensify/questions/8582 for more details.
—
Reply to this email directly, view it on GitHub
https://github.com/Expensify/App/issues/6873#issuecomment-999879810, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AARWH5X5AMG2GIYKFSGQQTTUSI4S7ANCNFSM5KROOD4A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

We might want to truncate the middle part so the file

Well, I thought about this. But as I mentioned earlier, we're reusing the Header component and the subtitle might not necessarily be a filename (have an extension). So are we sure to truncate the middle part? or you'd want to keep it multiline. We can have an assumption that the subtitle won't be that big enough but can't be sure for mobile devices.
I was thinking more on the lines of having a subtitleTruncateMode (none, middle, tail) in the Header. Would that be overkill?

Good points. I don’t think the subheader should be more than 1 line since
we don’t want to modify the height.
Perhaps we can pass through an optional prop to truncate in the middle?
Curious what Marc thinks there.
On Wed, Dec 22, 2021 at 4:12 PM Manan @.***> wrote:

We might want to truncate the middle part so the file
Well, I thought about this. But as I mentioned earlier, we're reusing the
Header component and the subtitle might not necessarily be a filename
(have an extension). So are we sure to truncate the middle part? or you'd
want to keep it multiline. We can have an assumption that the subtitle
won't be that big enough but can't be sure for mobile devices.
I was thinking more on the lines of having a subtitleTruncateMode (none,
middle, tail) in the Header. Would that be overkill?
—
Reply to this email directly, view it on GitHub
https://github.com/Expensify/App/issues/6873#issuecomment-999882764, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AARWH5SGKUMMZTHJTJJR24TUSI5L7ANCNFSM5KROOD4A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (Exported)

Triggered auto assignment to @Julesssss (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@kbecciv I see in the OP  Issue found by executing PR: #6850 but I don't see a specific comment there. Do you have more context on why this issue got created?  Was it from this convo in #expensify-open-source?
Upwork job posted https://www.upwork.com/jobs/~01dec90796e410ac17

@mallenexpensify The linked PR we added the filename handling to the existing modal and it turns out that if the file name is very long it doesn't render properly. Hence this issue is created, but apparently, the issue exists in the current Header component itself.

It sounds like you are saying that anything with a "long subtitle" and using the Header component would cause this behavior and in that case probably does not need to be fixed before we can close out the linked issue.

Hence better if we treat this as a separate issue. I can submit a proposal based on Shawn's comment if we agree.

Yes, definitely agree it's a separate issue.

Proposal
In the Header component,

I'll implement a prop hasFileNameInSubtitle, which would either be Boolean.

If the prop is set then I'll implement what we've done in ReportTypingIndicator. Two Views split by .. The first View and Text will have ellipsis with flex-shrink: 1. The second View with flex-shrink: 0, which shall have the extension.

Quick highlights,

Based on https://github.com/Expensify/App/issues/6873#issuecomment-999886023,  we're sure we won't have multi-line subtitles
Because ellipsisMode="middle" only works on iOS and Android, I've replicated what we've done in ReportingTypingIndicator. Hence I've added the prop, hasFileNameInSubtitle and not a generic prop shouldShowEllipsis.

I see that you are going to borrow a lot of logic from ReportingTypingIndicator. It indicates that we need to dry this up. I would suggest creating a small component that does that for us and reusing it.

I'll implement a prop hasFileNameInSubtitle, which would either be Boolean.

The header is supposed to be generic and hasFileNameInSubtitle sounds like a very specific case. so I have a second thought.
Why don't we pass a component in the subtitle prop? Thus subtitle would accept a component or string.
@Julesssss  What do you say?

Why don't we pass a component in the subtitle prop? Thus subtitle would accept a component or string.

Like that idea @parasharrajat. Yeah I can create a new component that replicates the logic and replace at both the places.

@mananjadhav Would you like to revise your proposal?

Hi @parasharrajat, here you go.
Updated Proposal

I'll create a component TextWithEllipses (or suggest a better name) which shall take two props leadingText and  trailingText.
It'll have two Views split by .... The first View and Text will have ellipsis with flex-shrink: 1. The second View with flex-shrink: 0.
In HeaderWithCloseButton, we'll add the prop as subtitle: oneOf(PropTypes.func, PropTypes.string). Based on the type we'll either show the passed string or load the component.
In AttachmentModal, we'll add a function to render TextWithEllipses component by splitting the text with .

Sounds good. We can look into the naming during the PR review.
@mananjadhav 's proposal looks good.
cc: @Julesssss
:ribbon: :eyes: :ribbon:  C+ reviewed

I don't quite understand the solution. Why are we specifying strings parts to do the truncating? Shouldn't it be determined by the length of the passed string and width of the window?

https://reactnative.dev/docs/text#ellipsizemode

The problem with ellipsizeMode is that it doesn't work on Web, Desktop, and Mobile Web. I proposed a solution that we already are using in another component.

Oh right, because middle truncation is what we want to do. Ok, looks like people have been debating whether to add this to the CSS standard for a long time.

Oh right, because middle truncation is what we want to do.

That's right. Are you fine with the proposed solution https://github.com/Expensify/App/issues/6873#issuecomment-1004158725 ?

Gonna leave this one for @Julesssss

Caught up with the conversations and the proposal makes sense.

📣 @mananjadhav You have been assigned to this job by @Julesssss!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

@mananjadhav can you please apply here and I'll hire ya?
https://www.upwork.com/jobs/~01dec90796e410ac17

Done @mallenexpensify

@parasharrajat and @mananjadhav are  making progress on the PR https://github.com/Expensify/App/pull/7097

PR is complete. There's one issue with react-native-web for which Rajat has raised a PR. Once merged we'll merge this (might have to rollback my extra commits)

Depency PR is merged today. @mananjadhav you can proceed.

Thanks @parasharrajat I'll push the revert today.

Issue not reproducible during KI retests. (First week)

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.1.32-0 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/7097

If no regressions arise, payment will be issued on 2022-02-01. :confetti_ball:

Issue not reproducible during KI retests. (Second week)

@mallenexpensify Reminder for Upwork.

Thanks for the ping @mananjadhav , you're paid now.
@parasharrajat , I hired you, let me know when you accept and I'll pay for C+

Paid @parasharrajat !


Is there a way to read the Omni script on blockchain.info?

Hi,
I'm writing a blockchain book, and want to explain how Omnilayer works.  @dexX7 would you be available to do a quick 30 min chat over skype?
It would be very helpful if I can show how a property transaction works through a standard blockchain explorer.  I'd like to understand better how omnicore writes/reads data on the blockchain.
I also want to verify that I have a correct understanding of how the network works.
If you are open to doing a quick chat, please email me at<EMAIL_ADDRESS>

I'm now at the point where I get the Omnilayer data from the OP_Return field, but how do I convert the data into a json object?  I see information on how to decode from a class B Omnilayer transaction, but there is no information on how to decode from a Class C transaction.
Are class C transactions what Omnilayer is using now?
Thanks for your help,
Lorne


Controller Config conflict

In the ES_input.cfg (userdata/system/configs/emulationstation)
Have two Xbox controllers that show up as the same GUID & Device Name. But they have a different button set up, by one value (+/-) is there a way to differentiate between the two? Like a MAC address or ?
Picture attached is an Example of what I want to do.
deviceMacAddress=“xxxxxxxx”
Or is there another file I can edit that has them stored/configured that I can edit the button layout?

The problem I am having is in the es_input.cfg they are stored under one GUID / one entry. So every time I config one. It changes the button layout of the one I just config before.

Oops

This is what it shows up as (over writes every time I config one of the controllers)


CamelCase- HackerRank

Ques.
There is a sequence of words in CamelCase as a string of letters, , having the following properties:
It is a concatenation of one or more words consisting of English letters.
All letters in the first word are lowercase.
For each of the subsequent words, the first letter is uppercase and rest of the letters are lowercase.
Given s, determine the number of words in s.
Example
s= oneTwoThree
There are 3 words in the string: 'one', 'Two', 'Three'.
Details:

Programming Language

[x] C++

@ravikr126 @sameeksharl please assign this issue to me under LGMSOC

Please assign me this in java language.

hey, @sameeksharl what about my pull request ?