git-cliff forgot to read remote config from pyproject.toml

Is there an existing issue for this?

[X] I have searched the existing issues

Description of the bug
[tool.git-cliff.remote.github]
owner = "atticuszz"
repo = "python-uv"

not work in pyproject.toml
but git cliff --github-repo "atticuszz/python-uv" works perfectly
Steps To Reproduce

commit a example message with git and push it to remote
use the template

[tool.git-cliff.remote.github]
owner = "atticuszz"
repo = "python-uv"

[tool.git-cliff.changelog]
# template for the changelog header
header = """
# Changelog\n
All notable changes to this project will be documented in this file.\n
"""
# template for the changelog body
# https://keats.github.io/tera/docs/#introduction
body = """
{% if version %}\
    ## {{ version | trim_start_matches(pat="v") }} - {{ timestamp | date(format="%Y-%m-%d") }}
{% else %}\
    ## unreleased
{% endif %}\
{% for group, commits in commits | group_by(attribute="group") %}
    ### {{ group | striptags | trim | upper_first }}
    {% for commit in commits| unique(attribute="message") %}
        - {% if commit.scope %}*({{ commit.scope }})* {% endif %}\
            {% if commit.breaking %}[**breaking**] {% endif %}\
            {{ commit.message | upper_first }}\
            {% if commit.remote.username %} by @{{ commit.remote.username }}{%- endif %}\
            {% if commit.remote.pr_number %} in #{{ commit.remote.pr_number }}{%- endif %}
    {% endfor %}
{% endfor %}\n
"""
# template for the changelog footer
footer = """
<!-- generated by git-cliff -->
"""
# remove the leading and trailings
trim = true
# postprocessors
# postprocessors = [
#   { pattern = '<REPO>', replace = "https://github.com/atticuszz/python-uv" }, # replace repository URL
# ]
# render body even when there are no releases to process
render_always = true
# output file path
output = "CHANGELOG.md"

[tool.git-cliff.git]
# parse the commits based on https://www.conventionalcommits.org
conventional_commits = true
# filter out the commits that are not conventional
filter_unconventional = true
# process each line of a commit as an individual commit
split_commits = false
# regex for preprocessing the commit messages
commit_preprocessors = [
  # Replace issue numbers
  # { pattern = '\((\w+\s)?#([0-9]+)\)', replace = "([#${2}](<REPO>/issues/${2}))"},
    # If the spelling is incorrect, it will be automatically fixed.
    { pattern = '.*', replace_command = 'typos --write-changes -' }
]
# regex for parsing and grouping commits
commit_parsers = [
  { message = "^feat", group = "<!-- 0 -->🚀 Features" },
  { message = "^fix", group = "<!-- 1 -->🐛 Bug Fixes" },
  { message = "^doc", group = "<!-- 3 -->📚 Documentation" },
  { message = "^perf", group = "<!-- 4 -->⚡ Performance" },
  { message = "^refactor", group = "<!-- 2 -->🚜 Refactor" },
  { message = "^style", group = "<!-- 5 -->🎨 Styling" },
  { message = "^test", group = "<!-- 6 -->🧪 Testing" },
  { message = "^chore\\(release\\)", skip = true },
  { message = "^chore\\(deps.*\\)", skip = true },
  { message = "^chore\\(pr\\)", skip = true },
  { message = "^chore\\(pull\\)", skip = true },
  { message = "^chore|^ci", group = "<!-- 7 -->⚙ Miscellaneous Tasks" },
  { body = ".*security", group = "<!-- 8 -->🛡 Security" },
  { message = "^revert", group = "<!-- 9 -->◀ Revert" },
]
# filter out the commits that are not matched by commit parsers
filter_commits = false
# sort the tags topologically
topo_order = false
# sort the commits inside sections by oldest/newest order
sort_commits = "oldest"

run git-cliff to check the example commit message in CHANGELOG.md ,i won't appear @by_user, but run git cliff --github-repo "atticuszz/python-uv"

Expected behavior
so i guess that git-cliff forgot to read remote config from pyproject.toml
Screenshots / Logs
No response
Software information

Operating system:ubuntu 24.4
Rust version:no rust,is python
Project version:"git-cliff>=2.6.1"

Additional context
No response

I'm unable to reproduce. This should work out of the box:
https://github.com/orhun/git-cliff/blob/99b78b52f2d81484fa4e5ea7a66de6e9bea2b3df/git-cliff-core/src/config.rs#L41-L47
Is your configuration file is named pyproject.toml? Do you get any additional information when you run git-cliff with -v or -vv flag?

so sorry to bother u to test for it:)
it fixed for some reason i don't know
thank you so much!


Update HtmlAgilityPack

https://htmlagilitypack.codeplex.com/SourceControl/changeset/108454
AgilityPack has now been updated to resolve the stackoverflow issue. We no longer need the stackoverflow issue workaround.

updated HtmlAgilityPack to <IP_ADDRESS>
remove redundant HtmlAgilityPack binary
updated some tests to reflect change

The StackOverflowException discussion for reference:
https://code.google.com/archive/p/abot/issues/77

@andy-williams your pull request shows that this issue was not fixed with that version. See the bottom of this page which is the build result from your pull request. That is what happens when a stackoverflow exception happens due to the hap problem.

Perhaps I misread the commit in HAP. I see that this is quite historic though - did you raise this with the author of HAP previously? If not, I'll try to submit a patch for the HAP project.

I did try to get this fixed but HAP appears to be unmaintained and there have been several unofficial forks. Also it appears that many of HAPs internals rely on stacks so there are many ways to make it blow up. I just happened to find 1 and my patch fixed it but maybe not other ways. Others have fixed some of the other ways but not my problem, etc.. etc...


Dev build pipeline additions

Added basic github action to build angular app
added cypress e2e testing
added commitlint to validate git commit messages

Looks good to me


fix ynEigenAdapter referral deposit

Context:
Number 1:
ynEigenDepositAdapter bug -
for all assets except stETH and oETH
the ynEigenDepositAdapter.deposit function does simply:
        return ynEigen.deposit(asset, amount, receiver);

        instead of:

        asset.safeTransferFrom(msg.sender, address(this), amount);
        asset.forceApprove(address(ynEigen), amount);
        return ynEigen.deposit(asset, amount, receiver);

        Which does not actually transfer tokens over.

This fix fixes this function and its caller.
Additionally added a previewDeposit that handles all assets for convenience.
Number 2:
the asset parameter was left out from the deposit event in ynEigen, which makes it difficult to capture the info easily of what deposit was made.
Context still exists from the asset that was transferred but it's not contained properly in the event

@xhad  thanks for looking at this earlier, i added to bug fix in the same one since we'll just upgrade both things.


Lazy load Monaco Editor to unblock SSR process

I'd really love the ability to include Solid REPL in Solid Site's SSR process. Unfortunately Monaco introduces side-effects when I attempt to do so. You can experience this on https://github.com/solidjs/solid-site/tree/solid-start-migration. A solution discussed on Discord is to lazy load Monaco itself but that might apparently pose some trouble for the top-level await.
Either way it would be really nice to get support for this. I'm hoping someone has some spare time to sink their teeth into it.
Thanks!

include Solid REPL in Solid Site's SSR process

What would this do? Like render the initial HTML of the editor with the payload?

I think we can probably close this once https://github.com/solidjs/solid-site/pull/261 gets merged, as an example of using solid-repl within SSR

One example of doing this is here: https://github.com/solidjs/solid-docs-next/pull/70, but there seems to be a vite issue blocking us and causing an error about bundling two workers at the same time: https://app.netlify.com/sites/solid-docs/deploys/62b60ea52f4e28000aeb0792
Unexpected early exit. This happens when Promises returned by plugins cannot resolve. Unfinished hook action(s) on exit:


This youtube clip consistently fails to download but works fine on the webpage

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[johjo50ubuntu:~/usr/mnt/r../KarlBirgerBlomdahl]$ youtube-dl -i  --write-auto-sub --sub-format srt --restrict-filenames --socket-timeout 5 --no-check-certificate  -o '%(title)s.%(ext)s' 'https://www.youtube.com/watch?v=09yk-oVkpLk&t=483s'
[youtube] 09yk-oVkpLk: Downloading webpage
[youtube] 09yk-oVkpLk: Downloading player 11e3a4ec
[youtube] 09yk-oVkpLk: Downloading MPD manifest
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Exit Code 1
[johjo50ubuntu:~/usr/mnt/r../KarlBirgerBlomdahl]$ youtube-dl -i  --write-auto-sub --sub-format srt --restrict-filenames --socket-timeout 5 --no-check-certificate  -o '%(title)s.%(ext)s' 'https://www.youtube.com/watch?v=09yk-oVkpLk&t=483s' --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-i', u'--write-auto-sub', u'--sub-format', u'srt', u'--restrict-filenames', u'--socket-timeout', u'5', u'--no-check-certificate', u'-o', u'%(title)s.%(ext)s', u'https://www.youtube.com/watch?v=09yk-oVkpLk&t=483s', u'--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 2.7.17 (CPython) - Linux-5.4.0-139-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.11, ffprobe 3.4.11, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] 09yk-oVkpLk: Downloading webpage
[youtube] 09yk-oVkpLk: Downloading MPD manifest
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Exit Code 1
[johjo50ubuntu:~/usr/mnt/r../KarlBirgerBlomdahl]$                                                                                ❮17:52❯

Description

This download consistently fails. I can watch the video on youtube.

[x] I've searched the bugtracker for similar issues including closed ones

Really? Duplicate of #31530.


Revert "Add check memory leak for windows"

Reverts webmachinelearning/webnn-native#201 because new PR will be failed.
@BruceDai @wangli69087 Could you please check the memory leak size to detect regression not the key words "memory leaks", i think we should enable the CI until fixing all memory leaks if we can't check memory leak size.
@huningxin WDYT?

@huningxin WDYT?

I agree with @fujunwei . If there is a memory leak, please file an issue and fix that before we enable the CI for that.

If there is a memory leak, please file an issue and fix that before we enable the CI for that.

@BruceDai , does the plan sounds good to you?

@BruceDai , does the plan sound good to you?

Yes, I'm OK to revert this CI, thanks.

There's already an issue https://github.com/otcshare/webnn-native/issues/874 for memory leak to track.

Thanks @BruceDai .
https://github.com/webmachinelearning/webnn-native/pull/238 seems to fix that issue. As it has been merged, I'll re-run the bot before I revert the check in CI.

Re-run memory leak check CI of https://github.com/webmachinelearning/webnn-native/pull/239 still failed. I'll merge this PR to unblock other PRs.


[lighthouse] add moar audit references

@paullewis @paulirish @addyosmani @brendankenny here's the last of the reference docs for the currently-active Lighthouse audits
cc @jpmedley in case you want to review

see #3693


Add disk vendor, model and serial in "list -d"

Reads disk info from sysfs, requires some "massaging" of the vendor and model output as it contains non printable characters that mess with the table output.
Example output:

Hello @herver,
Nice patch, thanks for the contribution! And definitely, agree with you about "massaging", and the way how you deal with bytes is the best way.
I would like to suggest only one modification in your code if you don't mind. Although I also use unwrap() in some parts of the code, unwrap() can make the software panic if the value doesn't exist.
Replace the unwrap() all over the place is something I would like to do, but didn't find the time yet.
I will comment on one part of your code with an example (not tested), but I think it should work. (feel free to let me know your opinion about the change I'm proposing)
Once again, thank you for your contribution.

Cool, thanks a lot for the contribution once again :)


Objects overlap

When opening a link to a board on mobile, the footer covers the "load more" button.
Issue does not seem to occur when opening a website and then navigating to a board.

thank you for this bug report.
fixed in this commit: https://github.com/sepehr-safari/pinstr/commit/4c9ede28baebff5d4ccc3de4568ea8c2d7917c3c


Python Script for generating file lists

This pull request adds a python script to generate a list of files in all packages, it also changes the azure pipeline to generate those files at the end of the build sequence.
Describe the pull request

What does your PR fix? Fixes #

Which triplets are supported/not supported? Have you updated the CI baseline?

Does your PR follow the maintainer guide?

Just saying: <vcpkg_root>/installed/vcpkg/info has a list file for each port with all installed files for that port. It just needs to collect these and bundle them as an artifact.


Added Grid styles for search modal

Modal Grid style, #19

Thank you for the pull request! Looks really good. I will merge it and close #19


Updated Readme.md with badges

🛠 Issue (Number)

Issue no #698
👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[✔] My code follows the code style of this project.
[✔] This PR does not contain plagiarized content.
[✔] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

@Kumar-laxmi
All the changes are approved by reviewers @siddhi-244 @zugzwang03
So, could you please merge this.


Ci/fix-discord-notify

This PR fixes the issue with the notify workflow regarding secrets format (following #512).
Failed run: https://github.com/okp4/ui/actions/runs/3016543228

Codecov Report

Merging #515 (92b7b74) into main (78f064e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #515   +/-   ##
=======================================
  Coverage   96.34%   96.34%
=======================================
  Files          70       70
  Lines         847      847
  Branches      124      124
=======================================
  Hits          816      816
  Misses         29       29
  Partials        2        2

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

:tada: This PR is included in version 1.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Integrate with Neo Blockchain Toolkit to build generated Python smart contracts using neo3-boa

Implemented it on this commit:
https://github.com/ngdenterprise/neo3-visual-tracker/pull/118/commits/1f4df6a1f3bc914463287e98be700ee2446450d2


LUN-296: Jira releases

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword LUN.

Tested with this release: https://issues.tuerantuer.org/projects/LUN/versions/12610


[Secure Consul Ph. 2] Consul tokens generated by Vault mgmt tokens expired after 1 hour

🚀 Feature Request
Relevant Package

This feature request is for security-bootstrapper or security-consul-bootstrapper
Description

The issue is located in: all EdgeX services and application services using Consul tokens generated by Vault mgmt token through Vault's /consul/creds/ API are expired after 1 hour and thus all calls to consul will be failed after tokens are expired.  You will see 403 Status Forbidden when that happens.
This is kind of newly emergent bug once we integrated the Consul token with Vault's mgmt token generation.
Currently, the TTL or period of Vault management token is set to 1h (1 hour) and hence the consul tokens generated by this Vault token are inherited the TTL behavior from Vault as Vault is managing the life cycle of these tokens including consul tokens generated by it.
We should renew the Vault management token so that the expiration would not happen and thus consul tokens would not be revoked by Vault.
If not doing the renewal of Vault management token, once consul tokens using by EdgeX services expired, we start to see the 403 status code from those services calling to consul shown as follows:

edgex-core-consul        |     2021-04-02T17:34:15.110Z [ERROR] agent.http: Request error: method=GET url=/v1/kv/edgex/appservices/1.0/AppService-rules-engine/Writable/?index=357&recurse=&wait=1800000ms from=<IP_ADDRESS>:38122 error="ACL not found"
edgex-app-service-configurable-rules | level=ERROR ts=2021-04-02T17:34:15.111195933Z app=AppService-rules-engine source=config.go:421 msg="Unexpected response code: 403"
edgex-core-consul        |     2021-04-02T17:34:16.108Z [ERROR] agent.http: Request error: method=GET url=/v1/kv/edgex/core/1.0/edgex-support-scheduler/Writable/?index=178&recurse=&wait=1800000ms from=<IP_ADDRESS>:43482 error="ACL not found"
edgex-support-scheduler  | level=ERROR ts=2021-04-02T17:34:16.109513144Z app=edgex-support-scheduler source=config.go:421 msg="Unexpected response code: 403"
edgex-core-consul        |     2021-04-02T17:34:16.259Z [ERROR] agent.http: Request error: method=GET url=/v1/kv/edgex/core/1.0/edgex-core-data/Writable/?index=74&recurse=&wait=1800000ms from=<IP_ADDRESS>:32952 error="ACL not found"
edgex-core-data          | level=ERROR ts=2021-04-02T17:34:16.260686548Z app=edgex-core-data source=config.go:421 msg="Unexpected response code: 403"
edgex-core-metadata      | level=ERROR ts=2021-04-02T17:34:16.701787779Z app=edgex-core-metadata source=config.go:421 msg="Unexpected response code: 403"
edgex-core-consul        |     2021-04-02T17:34:16.700Z [ERROR] agent.http: Request error: method=GET url=/v1/kv/edgex/core/1.0/edgex-core-metadata/Writable/?index=169&recurse=&wait=1800000ms from=<IP_ADDRESS>:44804 error="ACL not found"
edgex-core-consul        |     2021-04-02T17:34:17.470Z [ERROR] agent.http: Request error: method=GET url=/v1/kv/edgex/core/1.0/edgex-core-command/Writable/?index=179&recurse=&wait=1800000ms from=<IP_ADDRESS>:60862 error="ACL not found"
edgex-core-command       | level=ERROR ts=2021-04-02T17:34:17.470762537Z app=edgex-core-command source=config.go:421 msg="Unexpected response code: 403"
edgex-core-consul        |     2021-04-02T17:34:18.309Z [ERROR] agent.http: Request error: method=GET url=/v1/kv/edgex/core/1.0/edgex-sys-mgmt-agent/Writable/?index=406&recurse=&wait=1800000ms from=<IP_ADDRESS>:48780 error="ACL not found"
edgex-sys-mgmt-agent     | level=ERROR ts=2021-04-02T17:34:18.310163222Z app=edgex-sys-mgmt-agent source=config.go:421 msg="Unexpected response code: 403"
edgex-core-consul        |     2021-04-02T17:34:22.775Z [ERROR] agent.http: Request error: method=GET url=/v1/kv/edgex/core/1.0/edgex-core-command/Writable/?index=179&recurse=&wait=1800000ms from=<IP_ADDRESS>:32778 error="ACL not found"
edgex-core-command       | level=ERROR ts=2021-04-02T17:34:22.775455801Z app=edgex-core-command source=config.go:421 msg="Unexpected response code: 403"

as the Vault revokes or cleans out all consul tokens together generated from the management token since that token was expired:
edgex-vault              | 2021-04-02T17:23:57.903Z [INFO]  expiration: revoked lease: lease_id=auth/token/create/hfbe9ebcca18e1f664d6237aa433c65ad6d04be34536c85ccd16980e1031d95af
edgex-vault              | 2021-04-02T17:23:57.929Z [INFO]  expiration: revoked lease: lease_id=consul/creds/appservice-rules-engine/p1kpp0HFOi79BR32ZONNqo1D
edgex-vault              | 2021-04-02T17:23:57.933Z [INFO]  expiration: revoked lease: lease_id=consul/creds/edgex-sys-mgmt-agent/KmHREML1aD2cMyI1pQ248bpa
edgex-vault              | 2021-04-02T17:23:57.934Z [INFO]  expiration: revoked lease: lease_id=consul/creds/edgex-core-metadata/6DrfEvlPejpHsUVRvrQOfbPe
edgex-vault              | 2021-04-02T17:23:57.934Z [INFO]  expiration: revoked lease: lease_id=consul/creds/edgex-support-notifications/xuLl9NMtBmGKAdJ1TrB8GLcq
edgex-vault              | 2021-04-02T17:23:57.935Z [INFO]  expiration: revoked lease: lease_id=consul/creds/edgex-core-data/BOK2pe279nDxVOcsgIeYLgjB
edgex-vault              | 2021-04-02T17:23:57.935Z [INFO]  expiration: revoked lease: lease_id=consul/creds/edgex-support-scheduler/gzIcx5ivTVW8ukeUZ1xctmV1
edgex-vault              | 2021-04-02T17:23:57.936Z [INFO]  expiration: revoked lease: lease_id=consul/creds/edgex-core-command/aTYjaJHfppWDpgAypda2l5zr

Renew the Vault management token will prevent this from happening.
Describe the solution you'd like

In the Consul's entrypoint scirpt of security-bootstrapper, we should fork a background long running process to renew the Vault's management token so that it won't expire and hence consul tokens will continue to work.
Describe alternatives you've considered
 Have you considered any alternative solutions or workarounds?

different approach using each service's token, will not be an issue any more


feat(native-filters): Adjust filter components for horizontal mode

SUMMARY
This PR adjusts native filter components for horizontal filter bar.
Noteworthy changes:

Value filter:
a) Introduce a oneLine mode to Select component - if enabled, only 1 tag will be displayed at once and the rest of selected tags will be collapsed to an overflow tag. When user opens the dropdown, only the overflow tag is displayed. All tags are always displayed in a single line.
b) If tag is truncated, display a tooltip with full name
c) Customize maxTagPlaceholder, so that the overflow tag doesn't display 3 dots at the end, which takes precious space
(e.g. instead of the default + 1 ... we show +1 when 1 item is overflowing)
Range filter:
a) Adjust the top and bottom margin to make sure that the slider is centered in horizontal mode
b) Decrease font weight and line height so that the marks fit well in the horizontal filter bar
Time range filter: add styling to make sure that the pill is centered in horizontal mode

Known issues:

The time range pill's width is dynamic depending on selected time range ("No filter" pill is smaller than "2022-01-01 : 2022-12-01" pill). However, the filter component itself always has a fixed width, which means that if the pill is short, there might be some empty space between the time range filter and the filter next to it. We are planning to replace it soon with a new component, stay tuned!

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

Enable HORIZONTAL_FILTER_BAR ff
Set horizontal mode for the native filters bar
Add various types of filters
Make sure that nothing breaks

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

CC @codyml for review

/testenv up FEATURE_HORIZONTAL_FILTER_BAR=true

@michael-s-molina @geido @codyml I implemented the Select changes based on our yesterday discussion. Can you take a look?

/testenv up FEATURE_HORIZONTAL_FILTER_BAR=true

/testenv up FEATURE_HORIZONTAL_FILTER_BAR=true

LGTM! One thing I noticed that can probably wait for a future PR (or left as-is) is that if there's a time filter in the overflow dropdown, clicking either of the popover buttons also closes the dropdown, which is probably not what we want:

Good point @codyml. Let's handle that in a follow-up.


Response can contain invalid UTF-8 sequences

curl http://:8123 --data "SELECT '\xF8' FROM system.one" -v | hexdump

* Rebuilt URL to: http://:8123/
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying ::1...
* TCP_NODELAY set
* Connected to  (::1) port 8123 (#0)
> POST / HTTP/1.1
> Host: :8123
> User-Agent: curl/7.54.0
> Accept: */*
> Content-Length: 19
> Content-Type: application/x-www-form-urlencoded
>
} [19 bytes data]
* upload completely sent off: 19 out of 19 bytes
< HTTP/1.1 200 OK
< Date: Tue, 28 Aug 2018 07:33:43 GMT
< Connection: Keep-Alive
< Content-Type: text/tab-separated-values; charset=UTF-8
< X-ClickHouse-Server-Display-Name: f5cef54f26d7
< Transfer-Encoding: chunked
< Keep-Alive: timeout=3
<
{ [7 bytes data]
100    21    0     2  100    19    206   1961 --:--:-- --:--:-- --:--:--  2111
* Connection #0 to host  left intact
0000000 f8 0a
0000002

curl http://:8123 --data "SELECT '\xF8' FROM system.one" | iconv -f UTF-8

iconv: (stdin):1:0: incomplete character or shift sequence

Am I understanding right that Content-Type: text/tab-separated-values; charset=UTF-8 means that response should have C3 B8 instead of F8?

I think the only way to make header consistent with response encoding is to change header, as making response body UTF-8 will most likely double encode it.

Yes, charset=UTF-8 is technically wrong.
Strings in ClickHouse can contain arbitary binary data,
that is typically assumed to be UTF-8 for text strings,
but the data type and the server itself is charset-agnostic.
We definitely should not recode this binary data in neither way. If we will write C3 B8 instead of F8 we assume that our data is in latin1 encoding or something like that, that is even more wrong. (This is an antipattern that has very large widespread.)
But if we simply remove charset=UTF-8, then browsers will not display our data correctly if our strings are actually in UTF-8. And assumption about UTF-8 is a reasonable default.


mne.filter.filter_data : All picks must be < n_channels (19), got 19

Describe the bug
I get this value while trying to get a high pass filter
ValueError: All picks must be < n_channels (19), got 19
Steps to reproduce
fname = mne.datasets.sample.data_path() + '/MEG/sample/sample_audvis_raw.fif'
raw = mne.io.read_raw_fif(fname)
raw.info
mne.filter.filter_data(raw,h_freq=0.1,l_freq=None,sfreq=600.61)
Expected results
a successful execution of high pass filter
Actual results
ValueError: All picks must be < n_channels (376), got 376
Additional information
I have tried this with my dataset too, same error
I have tried using picks too like below:
picks=['Fp1',"Fp2","F3","F4","Fz","T5","T6","T3","T4","C3","C4","Cz"]
or
picks=[ 0  1  2  3  4  5  6  7  8  9 10 11 12 17]
or picks=[0,1,2,3,4,5,6,7,8,9,10,11,12,17]
but same error
Platform:      Linux-5.4.104+-x86_64-with-Ubuntu-18.04-bionic
Python:        3.7.10 (default, May  3 2021, 02:48:31)  [GCC 7.5.0]
Executable:    /usr/bin/python3
CPU:           x86_64: 2 cores
Memory:        12.7 GB
mne:           0.23.0
numpy:         1.19.5 {blas=openblas, lapack=openblas}
scipy:         1.4.1
matplotlib:    3.2.2 {backend=module://ipykernel.pylab.backend_inline}
sklearn:       0.22.2.post1
numba:         0.51.2
nibabel:       3.0.2
nilearn:       Not found
dipy:          Not found
cupy:          Not found
pandas:        1.1.5
mayavi:        Not found
pyvista:       Not found
vtk:           Not found

There are several issue with your approach:

mne.filter.filter_data expects an np.array as its data argument, not a raw object. Use the filter method instead, i.e. raw.filter().
If you want to highpass-filter the data, you need to specify the l_freq argument (lower edge frequency); set h_freq to None if you don't want a lowpass filter.
Data needs to be loaded for the filter operation to work; either pass preload=True when loading the data, or run raw.load_data() right before filtering.

Thanks for Clearing the doubt!
I tried all the changes and it works completely fine.


Support Dry-Run RDS Configuration

Background
We currently use a static YAML configuration to bootstrap Envoy. With the recent growth in the number of services, it's becoming quite unmanageable to have everything in a single YAML file. We have started to look at RDS and CDS to sync the routes and clusters. Before we turn on RDS in production, we'd like to dry-run it for some time to make sure the routes that we are getting back from RDS SotW is same as what we have today as part of the route_config. HCM currently allows only one of the rds and route_config to be supplied.
Feature Request
Would it be possible to add two new attributes to the RDS for supplying an initial configuration and other for indicating whether to actually update the routes received from the RDS? This would make our migration journey a lot easier.
For example,
{
  "config_source": "{...}",
  "route_config_name": "...",
  "initial_route_config": "...",
  "dry_run": "..."
}

Both the initial_route_config and dry_run here would be optional fields.
In addition, Envoy could publish two new stats counter comparing the hashes of the initial routes with what it receives from the RDS when running in the dry-run mode. For example, rds_dry_run_config_mismatch for the hash mismatches and rds_dry_run_config_match for when the two hashes matches exactly.

How does the dry_run work? If mismatch, it will use initial route config and if not, does it update? If so, how about the update in the third time?

@daixiang0 In dry-run it would only emit the stats on whether the config received from RDS was a match to the initial config or not. It'd always use the initial config supplied. We could setup some alerts on the stats to notify us if there are any mismatches but we don't ever expect Envoy to use the config received from RDS in the dry-run mode.

In general I'm inclined to push back on implementing this. This feature seems very specific to your migration and I think it could be implemented in other ways. For example, canary and then comparisons using config_dump or something like that. It's also non-trivial to implement. cc @envoyproxy/api-shepherds for thoughts.

@mattklein123 Thanks for the feedback. I wasn't aware of the config-dump thing. I was thinking it could be useful for big migrations in general.
I did a POC to try it out but it is for version v1.21 and it seems that some things moved in v1.22:
https://github.com/agrawroh/envoy/commit/29f5d8ab5c3fd819a73ce5a5d456e21a3576d2d6

You could also, for this case, consider using a secondary filter chain or listener with both new/old configuration, you should be able to config dump or send traffic across both to compare.


unified-signatures incorrectly reports errors for defaulted paramters

Bug Report

TSLint version: 5.11.0
TypeScript version: 2.9.2
Running TSLint via: CLI

TypeScript code being linted
function foo(num: number);
function foo(num: number, num1: number);
function foo(num: string);
function foo(num: number | string, num1: number = 0) {
    console.log(num);
}

with tslint.json configuration:
{
    "rules": {
        "unified-signatures": true
    }
}

Actual behavior
ERROR: index.ts[2, 27]: This overload and the one on line 1 can be combined into one signature with an optional parameter.
ERROR: index.ts[3, 14]: This overload and the one on line 1 can be combined into one signature taking `number | string`.

Expected behavior
No error.

The point of this rule is to create the minimal set of overloads. Because
function foo(num: number);
function foo(num: number, num1: number);

is equivalent to
function foo(num: number, num1?: number);

you'll get an error here, which is expected.


fix: change graphql-compose to wtrocki/graphql-compose

Created to have it ready while we are off with @craicoverflow

Superseded by https://github.com/aerogear/graphback/pull/1810. Closing.


Strange behavior on Debian linux with any terminal except xterm-color

Describe the bug
Barely see anything absolutely no colors, sometimes something flashes up, otherwise mostly black screen. Menu "works", if you can figure out where you are.
To Reproduce
Steps to reproduce the behavior:

Install the latest Debian on a Hyper-V virtual machine. Only install SSH.
Install the latest dotnet 6.0 runtime on the machine.
Install libncursesw6, because it will be necessary.
Copy a Release build of the Example to the virtual machine.
Try to run it thru SSH with Putty
Try to run it thru Hyper-V Virtual Machine Connection.

Expected behavior
Works exacly like on Windows.
Screenshots

Exactly the same problem with the latest official Microsoft Dotnet Docker image, and with the Docker Desktop terminal.

Hi, thanks for getting in touch.  Looks like the colors are not being recognized by the curses driver.  What is your $TERM variable?  Try setting it to xterm-256color
export TERM=xterm-256color

Another option is to use Application.UseSystemConsole = true (must be set before Init).
If you are using a remoting technology e.g. PuTTY then you might need to set this TERM variable in that too, see: https://github.com/gui-cs/Terminal.Gui/discussions/1455

Hi, thank you for you answer.
I played a little with the $TERM variable values. Default is "linux" in debian. In case of Putty, it is "xterm".
On the system console, if i set it to anything else (xterm-16color, xterm-256color, konsole-16color, konsole-256color) the folowing happens:

Only highlighted colors, the menu cursor cannot be seen. Also Midnight Commander has only higlighted colors with these settings, so this spoils another applications.
However, xterm-16color and xterm-256color are working in Putty.
Just by a lucky educated guess, i tried on the system console the linux-16colors parameter. This looks the best solution. No highlighted colors only problem, and Midnight Commander also works well.

The Application.UseSystemConsole = true solution works on Windows, and in Putty. In linux sytem console forcibly results the highlighted color problem.
Somehow Midninght Commander uses the right colors even if the seting is linux or xterm. Maybe they are using some heuristics in this case?
Best Regards,

Interesting.
In Docker Desktop on Windows 10, the best solution is Application.UseSystemConsole = true
The next best solutions are xterm-16color and xterm-256color. However these, results strange menu borders.
Fascinating.

Strange.  If Midnight Commander works ok out of the box then it should be possible to improve things.  At the very least we could look for TERM variables that we know are not going to work and override them to ncurses (hopefully).

Two things relative to this:

The coming ConfigurationManager feature (#2284) will enable forcing the use of NetDriver (aka System Console) via an external config file for cases where that's the only way to make Terminal.Gui apps work.
Should we consider adding a check to CursesDriver for TERM=xterm-256color and either setting it ourselves, or at least providing a Console.Writeline in Shutdown that highlights that it's not set?

Should we consider adding a check to CursesDriver for TERM=xterm-256color and either setting it ourselves, or at least providing a Console.Writeline in Shutdown that highlights that it's not set?

I think this is a great idea!
SetEnvironmentVariable(String, String) makes changes for the current process only.  So there is no harm in forcing xterm-256color in situations where we know the value the user currently has wont work (e.g. it is blank or linux or one of the other values curses does not understand)


How to deal with transactions in domain layer

I have a question about handling transactions between data and domain layer.
Think about a usecase where two entities should be persisted into a local database in one transaction.
Should the domain layer know something about "transactions" when calling into the data layer and what about "nested transaction"? Who is responsible to decide whether two or more operations should handled atomic as the domain layer doesn't know something about where and how the data is persisted.
Another point is also how to handle if there is a foreign key relationship between entities.
Thank you for any advice.

Those are all implementation details, and should therefore all be contained in the data layer.

but the data layer is not aware of which operations on data should be handled atomic, i think thats a detail to the usecase and even to the presentation. But i dont know how to provide the domain layer an appropriate interface to the data layer where multiple operations can be handled transactional, including support for relationship between entities.

The data layer should be the only layer aware of transactions with a database, or anything that has to do with getting/persisting data.
In the article describing that data layer...
"The idea behind all this is that the data origin is transparent for the client, which does not care if the data is coming from memory, disk or the cloud, the only truth is that the data will arrive and will be got."
The presentation layer displays data when it is provided by a UseCase. The domain layer implements the UseCases which describe the business logic of the application. The UseCases should not require any knowledge of the data layer.

I already read the article , but i think i couldn't describe my problem in a clear way.
As i understood the domain layer should instruct the data layer to fetch or persist data (whereever the data comes or goes to doesn't matter).
Consider following usercase:
The presentation layer presents a view where the user can create new entries (for example 10000 entries) with one action (like button click).
There would be a use case in the domain layer, which will get the request from the presentation layer for creation and maybe validation. The use case will forward the creation and persistence to the data layer. In this scenario i would expect from the users perspective of view that either all entries will be created or no one, for example if the application process is terminated during write operations to the database.
The data layer could create each entry one by one in own transactions or it could create all entries in one transaction and following data consistency.
From my point of view only the domain layer (or the presentation layer) should determine whether all these new entries should be created in an atomic way.
You should of course abstract the technical implementation detail for transactions away from the domain layer, but it is for me not clear how to abstract this logical detail.
Am i thinking wrong?

"You should of course abstract the technical implementation detail for transactions away from the domain layer"

That is correct, but what you said just before it contradicts it...

"From my point of view only the domain layer (or the presentation layer) should determine whether all these new entries should be created in an atomic way"

Whether they are created in an atomic manner or not is an implementation detail of the data layer.
The domain layer could add a method to the Repository class such as
public abstract Observable<TrunkOfThings> updateTrunkOfThings(Thing newThing);

Now the data layer can handle the implementation details of what exactly "updating" entails, and it can hand back an updated version of the TrunkOfThings to the domain layer, which can then convert it to a UseCase, and hand it off to the presentation layer.

I dont think that there is a contradiction, one thing is the implementation detail in the data layer like using a SqtLiteTransaction or whatever, the other thing is to pack logical operations that belongs to the domain in an abstraction (for example a set of Runnables) and pass them to the data layer and force them to run in one transaction.
otherwise i dont know how you would do operations like update and delete entries from different database tables in one commit without moving business logic into the data layer.

Sorry for the late reply guys but I'm with @caseykulm a priori. Although @misrakli can you give us an specific example of what you are trying to do?

Imagin following use case.
You have a flow of screen with tasks where the user can modify (insert, update, delete) data from different domain entities. At the last screen you have to save the modifications and do one commit because all captured data should be saved in an atomic way (in this case we have a local database on the device, no cloud, no webservice, etc.).
The domain layer of course should not know how the data will be handled atomic, the data layer could use a database or shared preferences or whatever, but the domain layer i think must have the chance to say to the data layer "please do all this operations on the entities together or fail and rollback if there is a error".

@misrakli I do not see any problem here. The repository in the data layer executes the operation atomically and just in case it fails, you could throw an exception and catch it on the domain layer and do something else or just propagates it until you treat the error at a view level.
Always encapsulate the exception in an ErrorBundle: https://github.com/android10/Android-CleanArchitecture/blob/master/domain%2Fsrc%2Fmain%2Fjava%2Fcom%2Ffernandocejas%2Fandroid10%2Fsample%2Fdomain%2Fexception%2FErrorBundle.java

@android10 well the question now is how could the domain layer do this transactional operation.
in my approach there is a call(Callable c) in the interface to the data layer and i dont know if there is a better way to do that.

@Aetheus Nice to hear that there is still interest on this party :-)
The only 'clean solution' we found for us is really to provide a run(Runnable r) in the data layer and let the use case decide which operations to pack into that runnable. You can also do it with a "unit of work", but at last you need a way to put atomic operations from the point of the use case together.
The transaction costs in our case I think are not as expensive as inconsistent data are (by the way we have an offline first approach)
In one of our use cases we have to decide in an atomic manner whether we can perform a specific operation on an entity or not, i.e. from the point of the use case this means

query the entity
check state
modify the entiy
All this three steps have to be performed within one db transaction, otherwise inconsistent data could be the result.
Maybe there are better solutions for such use cases but we ended up with the described way as we also didn't find any other approach within a reasonable time span.

Personally, I have a dao for every table in data layer. I use them in order to save/delete/retrieve data from a single table.
Then i have my repositories which combine daos in order to pass data to and from domain layer (domain and data layer communicate through interfaces). As a result i handle transactions inside repositories. The code looks like that:
Completable.wrap(         			//call the daos                 )).                 doOnSubscribe(disposable -> db.beginTransaction()).                 doOnComplete(() -> db.commitTransaction()).                 doOnError(throwable -> db.endTransaction()).                  doOnTerminate(() -> db.endTransaction());

@Aetheus have you managed to derive at least a clean solution to the problem? A google search shows you posted on few sites about the scenario.

Hi @mwei0210 - in my search for an answer, I've come to 3 broad options:

Create another layer (a "Unit of Work") that will be responsible for instantiating the Repositories. During instantiation, it will inject the transaction context into the repositories. You can then "commit" and "rollback" this "unit of work" as you would a normal repository. tl;dr: reinvent transactions in a wrapping layer

Ditch the domain layer completely. Use the "Clean Architecture" if you really want to, but just use an off-the-shelf ORM. You'll feel dirty, but at least things like transactions and mocking for tests are probably already accounted for.

Ditch transactions completely. Use a retry mechanism like queues. Let every repository action be standalone. So using my example of an "Order" and "Credit" repository, the rough flow would be:
a) send a request to POST /purchase endpoint
b) that endpoint then puts a "deduct credit" message into the "credit queue" and a "make order" message into the "order queue"
c) the "deduct credit" message in the "credit queue" will be retried until it succeeds. Likewise for the "make order" message in the "order queue". So now it wouldn't matter if Repositories can't rollback - you've essentially said "there are no rollbacks - we'll retry until we succeed".
Of course, now you'd be in the realm of eventual consistency - whether or not this is acceptable for your use case is up to you.


Wraps up signup flow with connection to backend

Tracking Info
Resolves #35
Make sure your branch name conforms to: <feature|staging|bugfix|...>/<username>/<3-4 word description separated by dashes>. Otherwise, please rename your branch and create a new PR.
Changes
I created the signUpConfirmation page for mentee and mentor. This included connecting the backend to the frontend through a POST request.
NOT 100% DONE, NOT SUPPOSED TO MERGE YET

Still need to connect this view to the rest of the signup flow once other views are completed.

Testing
How did you confirm your changes worked?
I began the server on my computer and attempted clicking the submit button on a built simulator version of the app. This let me test if the post request was working and see exactly what was posted.
Confirmation of Change
Upload a screenshot, if possible. Otherwise, please provide instructions on how to see the change.

@YashRavipati1 I converted this to draft PR since @HarshGurnani is currently working on it

Working Signup through Phone -
https://user-images.githubusercontent.com/56745367/229822256-ef15debb-61a5-4967-8fdd-c439484922e6.mov

Most of the changes looked good, just a few really small things that might need to be changed. When inputting the graduation year, a comma is added to the year. Also, when navigating back to the previous steps, the progress bar isnt highlighted all the way, even though we have gotten to the last step and are just going back (the progress bar on the figma is highlighted all the way and circles on the steps that are completed, including the ones after the step we are on, are also filled in). I attached a couple of screenshots highlighting these issues. Honestly the progress bar might not even need to be changed but it's mostly a design choice.

Only action item from my end is resolving merge conflicts.

@HarshGurnani can you also address the changes pointed out by @YashRavipati1?
Also, I noticed that the Grade Radio Button selector for Mentee Info Screen has some sizing issues where the digits 1 and 0 in 10 are in separate lines. Can you fix that as well?


[Feature]: Allow extending functionality of toEqual

🚀 Feature Proposal
I'd really like the ability to extend the toEqual matcher. In my case, I would like to call .equals on each of my objects (if it exists), and otherwise resort to the default functionality. I saw this issue (#7351), but it kinda looked like it just stalled...
Motivation
I may have mentioned some of this above, but I want the ability to add extra functionality to the toEqual matcher, without rewriting the entire thing.
Example
// Test setup somewhere (very rough on implementation)
expect.equalityTester((objectA, objectB) => {
  // Do custom .equals checking, otherwise resort to default functionality
})

// In a test
it('correctly checks equality', () => {
  const measurement1 = new Measurement(1, 'ft')
  const measurement2 = new Measurement(12, 'in')

  expect(measurement1).toEqual(measurement2) // Should pass, currently doesn't
})

Pitch
This allows users to extend equality checks as necessary, without large changes

Did you try using expect.extend() to implement something like expect(measurement1).toEqualMeasurement(measurement2)?

Got you. So if you're able to specify your special equality, you can still define your catch-it-all toEqualEx() and use it blindly everywhere instead of toEqual.

Any update/thoughts here?

We really need something like this because sometimes our connection pool ends up in an toEqual and it OOMs/blows up.
I.e. an assertion like:
expect(book.authors).toEqual([a1]);

When it fails will try to "kind of toJSON-but-not-really" a1, end up crawling into the connection pool/various very cyclical data structures, and blow up.
We've made our own custom matcher that works around this, expect(...).toMatchEntity(a1), and that's fine, but this approach requires constantly fixing/changing assertions that looks like "they should just work" (toEqual([a1])) over to our non-buggy alternative.
Kind of funny, for us its only a problem when the assertion fails and Jest tries to do the nifty diff of expected/actual; if the assertion passes, everything is just fine. Which makes it even easier for users to not realize they "should not use toEqual here", b/c it will only blow up/OOME Jest during some future refactoring that happens to fail the expect.

@ittays ☝ ?

Got you. So if you're able to specify your special equality, you can still define your catch-it-all toEqualEx() and use it blindly everywhere instead of toEqual.

While this feature ask specifically for toEqual, I'd like to stress out this: https://github.com/facebook/jest/pull/7352#issuecomment-508080125
.extend does not solve a) code duplication if it should be applied to toEqual, toContainEqual, toEqualStrict, toMatchObject ... and all spyMatchers!
I am aware of a very burning use-cases for this = if you use (immutable) value-objects with non-trivial equality semantics (typically implemented using .compare / .equal functions, e.g. bignumber.js or some Temporals) are used a lot in the code under test.
Synthetic example:
const testDate = Temporal.Instant.from("2019-11-14T00:55:31.820Z")
expect(someMockedApi.calculateCorrectedDate)
.toHaveBeenCalledWith(testDate);

would work if you would register .equal-based equality tester in jest (qualified using instanceof, let's say)
Without it, all developers have to do something like this in all tests:
expect(someMockedApi.calculateCorrectedDate)
.toHaveBeenCalledWith({asymmetricMatch(val) { return testDate.equal(val);}});

@stephenpassero I have a rough and semi-working idea of how this could be solved using an enhancement. It allows you to register custom equality testers to the internal jasmine equal machinery. The tester first check for applicability (does object have equals function) and if it's applicable it returns true/false.
Downside is, my current version is crude and affects all tests, might be worth looking for help making it configurable even per-test or per test-file.
Would you be willing to provide some review / alfa testing of this?

Hello 👋 I've opened #13654 in an attempt to address this issue. Let me know what you think! Cheers!

Thanks a lot @andrewiggins, this uses the same approach I attempted for my experimental hack!


Rust's impl are not highlighted quite correctly.

I noticed this earlier today, but my regex skills are not up to solving it on my own:

For the first impl, the type should be identified (as it is when following struct). For the second, both type (MyObject) and trait (Clone) should be identified, and "for" should not be confused as the control structure keyword for.

You're right. This is because the rule that matches impl statements is pretty basic right now. It's hard to match all cases because of the generic parameters. I tried a few times but didn't find a good solution either yet.


[Sage-491] interaction pattern

Description

SAGE-491
Previously closed original ticket for comment context: SAGE-554
Previously closed followup ticket for comment context: SAGE-580

[x] - expandable card -> update display property, it's currently full width
[x] - add tabindex to StatusIcon for keyboard focus
[x] create focus styles for links that aren't buttons
[x] create focus to links in TransactionCard
[x] create focus to links in StatBox
[x] alert - update focus state for action buttons
[x] hero - resolve focus state on hero artwork
[x] catalog item - resolve focus state on title
[x] button / link truncation is not functioning properly
[x] resolve react helpLink shadow sizing

Screenshots

Buttons

Before
After

Labels

Before
After

Catalog Item Interaction

Before
After

Text links

Before
After

Stat Box link

Before
After

Alert action buttons

Before
After

Testing in sage-lib

Sage 3 specs -> Elements
The best way to test is to visit all pages and tab through the docs site
Testing in kajabi-products

#613 (LOW) Interaction pattern(focus state) adjusted to latest specs. Interaction pattern, namely focus states changes will occur as a result in kajabi-products but no adverse effects expected.

Related

This changes look really great. I don't have any major concerns here. However, I have noticed on some elements, specifically Buttons, that sometimes the focus outline does not always render evenly. It does not consistently render this way either, so I'm not exactly sure what is causing it, but something I've noticed in the past; not related to this work. I initially thought it may be related to the transition, but disabling it still causes the issue. Could be from calc? Not sure.
I've enclosed a video that captures what I've seen:
https://user-images.githubusercontent.com/7331038/124981870-10b94380-e004-11eb-853b-13a2773b54bd.mov

Ah yes, the infamous subpixel rendering issue! I'll see if I can nab this finally.


The value of each object DatePoint is added up on chart

Hi, i'm trying to send my list of DataPoint to BezierLine, but it seems to sum entire xAxis value.
My list is
List dataPoint = data.map((e) =>
DataPoint(
value: double.parse(e[1]),
xAxis: DateTime(
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).year,
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).month,
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).day)))
.toList();
But the result on phone is the sum of xAxis value in list.
Can help me, please?

Add a simple code sample that we could use to reproduce the issue.
On Tue, Mar 17, 2020, 5:12 PM Antonio<EMAIL_ADDRESS>wrote:

Hi everyone, i'm trying to send my list of DataPoint to BezierLine, but it
seems to sum entire xAxis value.
My list is
List dataPoint = data
.map((e) => DataPoint(
value: double.parse(e[1]),
xAxis: DateTime(
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).year,
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).month,
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).day)))
.toList();
But the result on phone is the sum of xAxis value in list.
Can anyone help me, please?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aeyrium/bezier-chart/issues/72, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFL3UF6HFKKOEHADQBU4J3RH7YTFANCNFSM4LN46UKA
.

Hi @diegoveloper, thanks.
In a Stateful Widget i have
List dataPo = [];
List dataPoint = data.map((e) =>
DataPoint(
value: double.parse(e[1]),
xAxis: DateTime(
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).year,
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).month,
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).day)))
.toList();
....
A ResponsiveGridCol that contain GraphWidget
ResponsiveGridCol(
lg: 18,
md: 18,
sm: 28,
xs: 28,
child: Center(
child: new GraphWidget(dataPoint: dataPo),
),
),
In GraphWidget class i have
List dataPoint = [];
GraphWidget({this.dataPoint});
....
series: [
BezierLine(
dataPointFillColor: Colors.black,
lineStrokeWidth: 4.0,
lineColor: Colors.green,
label: "Duty",
onMissingValue: (dateTime) {
return 44.5;
},
data: dataPoint
),
],
The results is equals to the attached image on first post.
But if i change data property (who actually contain dataPoint) with the same code of example
data: [
DataPoint(value: 45.5,xAxis: DateTime(2019, 5, 26)),
DataPoint(value: 48.5, xAxis: DateTime(2019, 5, 30)),
]
the point on chart are shown correctly.
Sorry for my bad english, i'm Italian.

Try using a single class main.dart that I could just copy and paste :).
Small simple code.
On Tue, Mar 17, 2020, 5:29 PM Antonio<EMAIL_ADDRESS>wrote:

Hi @diegoveloper https://github.com/diegoveloper, thanks.
In a Stateful Widget i have
List dataPo = [];
List dataPoint = data.map((e) =>
DataPoint(
value: double.parse(e[1]),
xAxis: DateTime(
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).year,
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).month,
DateTime.fromMillisecondsSinceEpoch((1000 * e[0]).toInt()).day)))
.toList();
....
A ResponsiveGridCol that contain GraphWidget
ResponsiveGridCol(
lg: 18,
md: 18,
sm: 28,
xs: 28,
child: Center(
child: new GraphWidget(dataPoint: dataPo),
),
),
In GraphWidget class i have
List dataPoint = [];
GraphWidget({this.dataPoint});
....
series: [
BezierLine(
dataPointFillColor: Colors.black,
lineStrokeWidth: 4.0,
lineColor: Colors.green,
label: "Duty",
onMissingValue: (dateTime) {
return 44.5;
},
data: dataPoint
),
],
The results is equals to the attached image on first post.
But if i change data property (who actually contain dataPoint) with the
same code of example
data: [
DataPoint(value: 45.5,xAxis: DateTime(2019, 5, 26)),
DataPoint(value: 48.5, xAxis: DateTime(2019, 5, 30)),
]
the point on chart are shown correctly.
Sorry for my bad english, i'm Italian.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aeyrium/bezier-chart/issues/72#issuecomment-600331038,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFL3UBHSJN2FFU37IBLN73RH72VBANCNFSM4LN46UKA
.

Hi @diegoveloper, i solved with this property: bezierChartAggregation: BezierChartAggregation.AVERAGE. I think by default the chart does the sum of the data.
However is there any property that can make me set the number of decimals that the AVERAGE shows up?
Thank you.

no , sorry there isn't at this moment, but you can send a PR if you want.
On Wed, Mar 18, 2020 at 4:22 PM Antonio<EMAIL_ADDRESS>wrote:

Hi @diegoveloper https://github.com/diegoveloper, i solved with this
property: bezierChartAggregation: BezierChartAggregation.AVERAGE. I
think by default the chart does the sum of the data.
However is there any property that can make me set the number of decimals
that the AVERAGE shows up?
Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aeyrium/bezier-chart/issues/72#issuecomment-600865568,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFL3UHIVAQZ4FPQZWLJ6Z3RIE3TFANCNFSM4LN46UKA
.


Webpack step doesn't work

I have followed the instructions so far, but when I get to the part where we run webpack for the first time I get the following:
C:\Users\mike\MySource\Project\Project>node_modules\.bin\webpack app.tsx webpack-config.js
Hash: b7174e5b44ef07e6c639
Version: webpack 4.42.1
Time: 580ms
Built at: 2021-02-05 17:57:40
 1 asset
Entrypoint main = main.js
[0] multi ./app.tsx ./webpack-config.js 40 bytes {0} [built]
[1] ./app.tsx 292 bytes {0} [built] [failed] [1 error]
[2] ./webpack-config.js 730 bytes {0} [built]

WARNING in configuration
The 'mode' option has not been set, webpack will fallback to 'production' for this value. Set 'mode' option to 'development' or 'production' to enable defaults for each environment.
You can also set it to 'none' to disable any default behavior. Learn more: https://webpack.js.org/configuration/mode/

ERROR in ./app.tsx 1:8
Module parse failed: Unexpected token (1:8)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
> declare var require: any
|
| var React = require('react');
 @ multi ./app.tsx ./webpack-config.js main[0]

I've tried a bunch of things but I can't seem to figure it out. This is my first experience with webpack and typescript really so I don't know where to look.

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

ID: c4b0ea28-4a31-a3ea-fa5f-c32d5c72d4bd
Version Independent ID: 31bd064b-51eb-5505-986e-2c7148784085
Content: Create a Node.js and React app - Visual Studio
Content Source: docs/javascript/tutorial-nodejs-with-react-and-jsx.md
Product: visual-studio-windows
Technology: vs-javascript
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

I figured it out. I left out the --config before the webpack-config.js argument.


Memory consumption is very high, possible memory leak with io.netty.channel.DefaultChannelHandlerContext

Hello,
Our service uses Redisson client library to work with Azure Redis. Pods memory consumption is very high, it constantly grows. Heap dump analysis shows that io.netty.channel.DefaultChannelHandlerContext is suspected for memory leak, it is the top consumer. Could you please advise why this issue can be observed? Maybe there is something wrong with the config or anything else.

We are using the RMapCache structures.
Thanks in advance.

Can you share a memory dump?

mrniko

Yes, please find it in the private repo here - https://github.com/Natalia588/data/releases/tag/heapdump
I've invited you to join it

Thanks. Seems like it's a classloader leak

Many thanks. What would you recommend? Is it a Netty issue?

18K classes. I don't think that they belong to Redisson or netty

Ok, thank you

@Natalia588  how did you solve this? We are facing the same issue

@techwizardg
Can you share the memory dump?

Issue Summary
We’re experiencing a memory leak when connecting to Redis/Valkey using the Redisson client 3.30.0.
Details
•	Context: The application has 32 default Netty threads.
•	Behavior: Memory usage continuously increases, eventually causing the application to be OOM killed. The application publishes to Redis/Valkey on specific events.
•	Observations: This issue occurs only with Valkey and not with Redis.
•	Diagnostics Attempted:
•	Heap Dump Analysis: No conclusive information was found; only the Netty threads appeared to hold approximately 4MB each.
•	System Properties Used: Tried adjusting Netty’s memory usage by setting:

-Dio.netty.noPreferDirect=true
-Dio.netty.maxDirectMemory=0
-Dio.netty.allocator.type=unpooled
-Dio.netty.leakDetectionLevel=advanced
•	Result: Despite these settings, memory still increases continuously over time.
Expected Behavior
The application should maintain stable memory usage over time without a continuous increase when connected to Valkey, similar to its behavior with Redis.
Additional Information
Any guidance on configurations or memory management specific to Redisson with Valkey would be appreciated.

It's hard to find a cause without the memory dump.


SEO: Meta (parameters) & Canonical link on Header Section

Suggestion: SEO
I see many people nowadays care more about SEO for their content to be visible on Search Engines.
I read it here, it is possible to allow users to include text to the header section of a webpage.
How about adding Personal Syntax  or Additional Text-Input Box on eSteem editor that allows authors to specify Metadata Parameters and Canonical-link to the header section of a webpage.
Many people will get interested in using eSteem if this feature is implemented on Mobile, Surfer, and Web.
Code in Header Section
*<link rel="canonical" href="http://example.com/">
*<meta name="keyword" content="">

Metadata - KeyWords for more visibility of content on Search Engines.
Canonical - To specify the original source of the article if already posted elsewhere like website or other Steem cloned (scorum, whaleshare…). This improves SEO ranking for double posting.

the pr you referred doesn't explains clearly usage of this case. lets wait until this type of usage matured.


Fix torch breakage for convert_to_tensor([])

Broke with https://github.com/keras-team/keras/pull/18752
Alternately we could handle this case in torch.convert_to_tensor, if we want to preserve the result_type error.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f1b5858) 78.96% compared to head (271fe6e) 56.23%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #18773       +/-   ##
===========================================
- Coverage   78.96%   56.23%   -22.74%
===========================================
  Files         336      336
  Lines       34051    34051
  Branches     6671     6671
===========================================
- Hits        26890    19147     -7743
- Misses       5582    13452     +7870
+ Partials     1579     1452      -127

Flag
Coverage Δ

keras
56.23% <100.00%> (-22.64%)
:arrow_down:

keras-jax
?

keras-numpy
56.23% <100.00%> (-0.01%)
:arrow_down:

keras-tensorflow
?

keras-torch
?

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

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


Highlight of closing brackets and curly brackets.

Wouldn't it be nice if jsBin have "highlight" of closed and opened bracket,

So if cursor was before or after closing "bracket"  it would highlight the current bracket and the opening one.
(and vice versa also)
I think such a function will really help readability.

This also seems like it should be closed...


No multi-gene mapper

Hello!
I'm trying to run this tool.
But, I have some questions about No multi-gene mapper.
Could you explain more about No multi-gene mapper?
Is it a different mode for the tool?

I don't think T1K has the warning message about "No multi-gene mapper". Could you please show me the command you used? Thank you.

Sorry, I don't explain clearly.
I want to know what  Fig1a in the paper writing No multi-gene mapper.
I find  that  the program  has one line in T1K_manuscript_evaluation: Fig1/Analysis.ipynb, which writes # No multi-gene mapper.
Therefore, I assume No multi-gene mapper mean another mode.
I am curious about what is No multi-gene mapper  and how to use it.

Thank you for the clarification.
Yes, "No multi-gene mapper" is another mode, where I added the code to ignore the reads mapped to more than one KIR gene.  The purpose is to demonstrate that ignoring multi-gene mapped reads is bad. Ignoring multiple gene mapped reads is adopted in several HLA genotypers, such as arcasHLA. It is too hard to modify arcasHLA to work with KIRs, so we add this test to mimic the behavior of arcasHLA. This mode is not really present in T1K.
Just out of curiosity, why do you need this mode?

Thank you very much!
To be honest, I don't need this mode.
However, I saw this mode on the paper and T1K_manuscript_evaluation: Fig1/Analysis.ipynb.
Then, I thought No multi-gene mapper was a mode and couldn't find something in the helper message.
I can't suppress my curiosity and create an issue. ><

Haha, then I shall close this issue for now. If you have more questions, please feel free to reopen this one or create a new issue.


Add more introduction in readme

Now the readme is mess.

Might be nice to use docker compose to pull a mariadb container and configure it for use so that with one command the entire environment can be brought up and tested.

Thanks for your reporting. The compose file will be added in https://github.com/ArchCI/demo .
BTW, we're using MySQL by default. It would be okay to use MariaDB but I don't know everybody likes it.

Yeah mysql is fine.
Yeah a compose file will improve it.


readme: fix broken link

Also added EOL at the end of README.md.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1461

Relevant Lines:
1554

💛  - Coveralls


control-service: Add helm chart entries for Vault Configuation

Add Vault configuration entries to the Control Service helm chart.

LGTM!


Implement refresh auth

What's frustrating? 😳
Currently authorization work till access token expires. After that, you need to authorize again.
Solution you'd like 🥰
Implement support of refresh tokens

That's work, but not so fast and maybe it doesn't look as good as it could. But nevertheless it works!


implement health check in docker cluster, event based

Only the last commit is for health check.
Listening on an event queue to monitor container's status. Previously, Cluster doesn't have event method, SubscribeToEvent is introduced to implement adapter.events.

Can I send this PR to docker master? Or we should hold it for the further discussion?

Please send it. There might be more work to make it work end-to-end. But let's get feedbacks.

@dongluochen Thanks for the review.


[PR][CLOSED] Removed unnecessary argument from output address link

 Issue by geunsik-lim
Tuesday Jun 12, 2018 at 03:57 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/pull/16

This PR is to remove unnecessary path from output webaddress
for easy debugging.
Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>
geunsik-lim included the following code: https://github.sec.samsung.net/STAR/TAOS-CI/pull/16/commits

:octocat: cibot: Thank you for posting issue #33. The person in charge will reply soon.


remove "=" after arguments

It looks like in the .wdl file some lines retained gatk3 arguments instead of those for gatk4. Say in line 269, I think gatk4 uses -I exome.fixed.bed but not -I=exome.fixed.bed. After making the changes, it worked locally for me. I use gatk-<IP_ADDRESS> on a Ubuntu machine.

Please accept this PR.

Test passed on Terra


Serving adjustments don't work with fractions

I have a recipe that uses fractions

However they are not affected by changing the number of servings

I think it should also work with fractions.
App version v0.18.0+40

Hey @Vortex-TH, you're using special characters as fractions. If you use regular characters, e.g. 1/2 instead of ½, fractions are already supported. This should be the same behaviour as in the web app.

You're right, but I think it would be good to support these special characters as well (since they were imported like this from a recipe website). Or at least convert them to another format automatically.


[Bug]: Image is not appearing

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Image is not appearing of testimonial
Expected behavior
the image of person is not visible
Add ScreenShots

On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSOC'24 contributor
[X] I have starred the repository

please assign me for this issue

@PATILYASHH  i wolud like to solve this issue

@apu52  plz assign me this issue

please assign me for this issue.

@PATILYASHH this issue has already been solved . sorry to close this issue . but u can still make new issues on which u want to work on


Investigate current xpub behaviour of creating new user if no profile

Currently in xpub, if a user logs in but has not profile, then a new user is created:
https://github.com/elifesciences/elife-xpub/blob/cd0bd9fef9c838c543897e041694741d27d20086/packages/component-model-user/entities/user/index.js#L37
A new uuid is created (via objection... i think) and used as the userId.
We would need to replicate that behaviour to maintain parity

Writing a document here.... https://docs.google.com/document/d/1oEMhSxCicNjNZFa5TAB7kIsvFQT_zWLGLhlYJtOSCwQ/edit?usp=sharing

conversation started, closing


Dockerize project

Local environment building and testing using docker.
Runs postgres and bot as two separate processes under the same container.

Didn't use docker-compose, everything runs smoothly


custom Acceleration curve

So you can see a curve and you can just edit the curve for prefect customization
just a thought tho
like imagine a 2d wavetable synth were you can edit the curve directly

Like CursorSense?

I'm not sure if the curve graph would help. For me, I have to move the cursor to see if it fits me. :)


TIM-109 fix(feed): sort newsfeed by newest first

TL;DR
This PR adds an order by clause to the getNewsfeed query to fetch posts in descending order of creation.
What changed?
The getNewsfeed function was modified to order the newsfeed items by their 'created_at' timestamp in a descending order. This ensures the latest posts are fetched first.
How to test?
Test by checking if the returned list of posts on the newsfeed are ordered from newest to oldest.
Why make this change?
This change was needed to ensure a consistent order of posts shown on the user's newsfeed. With this change, the user will always see the most recent posts at the top of their newsfeed.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#49  👈
#48
#47
#46
#45
#44
#43
#42
#41
#40
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @xdjeremy and the rest of your teammates on  Graphite

Merge activity

May 8, 7:08 AM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.


Update code for doc.rs generation

Currently, very little documentation is produced.

Use docstring comments /// instead of normal comments //


Use explicit tags on .gitmodules

Description
This is a test to do a workaround for konflux using an older version of libbpf. However, this should also help us use renovate for updating dependencies.
Checklist

[ ] Investigated and inspected CI test results
[ ] Updated documentation accordingly

Automated testing

[ ] Added unit tests
[ ] Added integration tests
[ ] Added regression tests

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why you did not do so. (Valid reasons include "CI is sufficient" or "No testable changes")
In addition to reviewing your code, reviewers must also review your testing instructions, and make sure they are sufficient.
For more details, ref the Confluence page about this section.

This didn't fix the konflux build, closing.


azcopy copy to Azure storage container with SAS token, works but reports Failed if read permission not set

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
AzCopy 10.4.0
Which platform are you using? (ex: Windows, Mac, Linux)
Linux
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
azcopy copy localfile "https://storage-container-path?sas-token"
What problem was encountered?
We have a "dropbox" storage container that we only want our client system to copy files to. I thought that creating the SAS token to only include create permission would do this (upload only once, and don't allow listing or download of any of the files).
This worked, mostly - the file is copied to the storage container, no errors are reported, but the job summary reports the transfer as failed and status code are set to 1 (error)
How can we reproduce the problem in the simplest way?
Using Azure PowerShell, create a SAS access token for a storage container:
$context = (Get-AzStorageAccount -ResourceGroupName <name> -AccountName <storage account name>).context
New-AzStorageContainerSASToken -Context $context -Name <container name> -Protocol HttpsOnly -Permissions c -ExpiryTime '2120-04-22 18:00:00' -FullUri
Use the generated URI for the azcopy command destination.
Have you found a mitigation/solution?
Add read permission when creating the SAS token

The error probably comes from the length check that we do at the end, as a sanity check of the transfer.  You can put --check-length=false on the command line if you don't want that check, and then it won't fail.
We may make it smarter about this in the write-only case in a future release.  I'm going to close this issue now, but let us know if the check length setting isn't a suitable solution for you.


Error with generate map

When I run the command python generate_map.py <image_file>
and this error happened, my tensorflow version is 1.1.0.
/home/shuai/.local/lib/python2.7/site-packages/skimage/transform/_warps.py:84: UserWarning: The default mode, 'constant', will be changed to 'reflect' in skimage 0.15.
  warn("The default mode, 'constant', will be changed to 'reflect' in "
Traceback (most recent call last):
  File "generate_map.py", line 32, in <module>
    cnn.load_vgg_weights()
  File "/home/shuai/image-compression-cnn/model.py", line 33, in load_vgg_weights
    with open(hyper.vgg_weights) as f:
IOError: [Errno 2] No such file or directory: './data/caffe_layers_value.pickle'

Hi @yuanshuai220
Just change
self.fine_tuning = True
to
self.fine_tuning = False
On this file:
https://github.com/iamaaditya/image-compression-cnn/blob/master/params.py#L12


a little bug about time

thank u for your work!

i run the code with cli ,it extract subtitle accurately ,but it don,t show time I don,t know why,if u can explain why.thank u.look forward to your reply

but it seems cool when i extract another file.

只有个别视频会出现这个问题对吧，你把视频发我看下

已经修复


Unity silently fails to BuildUp in UWP app built with Visual Studio 2019

Description
Within an UWP app referencing Microsoft.NETCore.UniversalWindowsPlatform v6.2.2, Unity v5.9.5 will fail to build up an instance without throwing an exception. With Unity < v5.9, the instance will be built up as expected.
To Reproduce
Create a new UWP application in Visual Studio 2019, add Unity as a nuget package and try to BuildUp instances. The following unit test will succeed with Visual Studio 2017 or Unity < 5.9 and fail with Visual Studio 2019 and Unity >= 5.9.
internal interface ICalculator
{
    int Add(int one, int two);
}

internal class Calculator : ICalculator
{
    public int Add(int one, int two) => one + two;
}

internal class CalculatingObject
{
    [Dependency]
    internal ICalculator Calc { get; set; }
}

[TestClass]
public class UnityTest
{

    [TestMethod]
    public void TestBuildUp()
    {
        var container = new UnityContainer();
        container.RegisterType<ICalculator, Calculator>();

        // works
        Assert.IsNotNull(container.Resolve<ICalculator>());

        var theCalculator = new CalculatingObject();
        container.BuildUp(theCalculator);

        // fails
        Assert.IsNotNull(theCalculator.Calc);
    }

}

Additional context
Sample project: https://genes.pics/s/isTjYisJZSa9H5D

Thank you very much for the quick fix! Can confirm the issue has been resolved with 5.9.6.


creating account

is there a feature for creating account

Hi! Yes, sure https://github.com/seregazhuk/php-pinterest-bot/blob/master/docs/account.md#registration

Great, thanks a lot !


Update forecasting.py

Updated docstrings for
check_step_length
check_cutoffs
check_scoring
https://github.com/alan-turing-institute/sktime/issues/484

Reference Issues/PRs
#484
What does this implement/fix? Explain your changes.
Added docstrings check_step_length, check_cutoffs, check_scoring
Does your contribution introduce a new dependency? If yes, which one?
No new dependencies.
PR checklist

For all contributions

[ ] I've added myself to the list of contributors.

Any other comments?

@raishubham1 thanks for the PR! I'll look through it later!

Thanks Markus,
I would like to continue with more contributions, once i have a bit of hang
I'll move to advanced issue's.
Best Regards,
Shubham
On Sun, 15 Nov 2020, 9:16 pm Markus Löning<EMAIL_ADDRESS>wrote:

Merged #487 https://github.com/alan-turing-institute/sktime/pull/487
into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/487#event-3997531077,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG77DAOSABI4ZV2XA6F6PLLSP7ZXBANCNFSM4TU7HDOA
.

Check out the series-to-series transformer issue #483 - I think that would be a good next step to try. Chat with us on Gitter or here if you need help! Would love to see you stay involved! 👍

Sure, I'll look over the issue and will let you know if any help is
required.
Best Regards,
Shubham
On Sun, Nov 15, 2020 at 9:39 PM Markus Löning<EMAIL_ADDRESS>wrote:

Check out the series-to-series transformer issue #483
https://github.com/alan-turing-institute/sktime/issues/483 - I think
that would be a good next step to try. Chat with us on Gitter or here if
you need help! Would love to see you stay involved! 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/487#issuecomment-727594233,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG77DAJ4Q2KR2546MD3X6RLSP74L7ANCNFSM4TU7HDOA
.

--
Thanks and Regards
Shubham
+91-9179572201


steel or aluminium plate replacement (check alignment if it fixes)

As part of this task,
The Aluminum plates that are used to join front legs of rover, the edges are sharp and is not safety standard. Also, these plates are not holding the legs rigidly as expected.
Must replace with plates with rigid material plates like plastic, fabric or fine metal plate.
Additionally, test and check the alignment and stability error when moving or at stationary.

references for metal sheet:
https://robu.in/product/easymech-3h-joining-plate-for-2020-series-aluminium-profile-2-pcs/?gclid=EAIaIQobChMI3paSxvb59QIVHJNmAh1dIAzDEAQYASABEgJJ8fD_BwE


cancelAnimationFrame

Hi guys,
First of all, congratulations for the amazing job done.
I'd like to know as requestAnimationFrame is already available if it is possible to add cancelAnimationFrame.
Thank you very much

@jdanyow We don't actually have this in our polyfills do we?

Hello Rob,
I'm taking the liberty to speak for Jeremy as he asked me to create the issue ;).
Thank you again

@EisenbergEffect we don't have rAF or cAF in the aurelia polyfills- folks supporting IE9 pull in a separate polyfill like https://www.npmjs.com/package/raf

@jdanyow Would you be willing to update the docs on bootstrapping older browser to include information on how to configure this?


Generalized wall restraints

This PR generalizes restrain.plane to also support other native WallData geometries like cylinders and spheres. This is achieved by templating the class on the wall object and using the built-in methods to compute normal vectors. The result is two new methods restrain.cylinder and restrain.sphere.

Thanks! The current design of the restrain module does not restrict how many restraints can be applied to the same particle. This is for two reasons: (1) there is no central structure registering which groups are attached to which restraints (as in the Integrator) so this is not technically possible with the current design, and (2) there are some types of restraints that can be intended to operate on the same group (e.g., position + orientation for doing TI).
As ForceComputes, there is no technical reason that multiple restraints cannot safely operate on the same group (since the forces are additive), but the physics could become nonsensical. Or, they could be interesting, for example, combining a cylinder with a plane could confine a particle to a ring.
afaik, the wall potentials are also completely additive? md.wall.group is converted to a struct that packs a bunch of walls together, and then the potential evaluator loops over them. The main difference is that the walls act on all particles in the system by type, rather than on a group.
Basically, is any of this helpful to put in the documentation, or is it more confusing than clarifying?

Let's add something to the python documentation along the lines: "Restrains/Force computes are additive and multiple restrains and force computes can be used in the same simulation. The user is responsible for ensuring that the system is defined and initialized properly."

Docs updated. I've also used this branch in production simulations on TACC GPUs, so once the unit tests clear again (and I make sure the production output isn't nonsense) this will be ready to merge.


replacing nodes leaves the name of the previous node

So i have kinematicbody for example. and I replace it with a rigid body.
it changes correctly and I can see the properties of the rigid body. but the name stays the same.
not sure if a bug. but it certainly confused me.

Not a bug, IMO, because if I had a custom name on a node (say, spawn_team_red), and decided that it should be a Position2D, and not a Node2D, after which it changed name to Position2D, all my current code would stop working.
Maybe... you want the name to change only if it was default?

yeah I guess it would be a very nice feature

I don't think the name should be changed, even if it were default. If you add scripts referring to "KinematicBody" and change the node time to RigidBody, you don't want your scripts to break - and we don't want to do advanced refactoring that should find the script references to "KinematicBody" and change them just to cater for this case IMO.

Maybe this could be optionally enabled in the editor settings? I would use this feature, after all if I have to change the type of a node that has existing functionality, I allready expect my scripts to break.

Just to make it clear, I don't think that we should change the name in any case, but I'm not against it anyway. Thus, I agree with @Hinsbart that this should go into settings.

IMO adding editor settings for such corner cases is a bit overkill... It would have to apply only when nodes have their default name (e.g. "KinematicBody"), and not when they have a more specific name (like "my_cool_player"). I don't think that we change node types often enough that it's worth adding an editor parameter for such a corner case.
Actually most of the times when I change a node type, it's for a node with a custom name, like changing my "level" node from Node2D to YSort. All in all the code change would likely not be invasive, but we have to remember that having tons of editor settings means have tons of various workflows, and some might end up having bugs that will be hard to notice if only a handful of users ever noticed the opt-in feature when browsing through hundreds of editor parameters :)

This does not make sense to me. Replacing node types is a rare action IMO, and you usually give proper names instead of using the default.
Changing the name might cause scripts to break, as stated, and then you'd have to change all the names in the script anyway, so editing one more name (of the actual node) won't be much trouble.
I think I said that in other issue: what we need is proper refactoring tools. And a general design for those tools before adding functionalities like the one requested here.

Closing as there is no consensus for implementing this. Also, this is quite old. :)


Figure out how to connect to IBM translations pipeline

Part of https://github.com/Qiskit/documentation/issues/476. We must start using IBM-powered translations for at least some languages to improve our coverage amount. The IBM translations pipeline cannot use open source crowdin.com and we instead need to adapt to the tooling they already have.
Ideally we can keep this infrastructure in open source. I suspect the end product will look similar with having the translations/ folder as a full copy of the MD/MDX/IPYNB files from English - only the process to get it will differ.

Closed in favor of a closed source issue.


why FLAGS_log_file_header is not exist in release version 0.6.0 ?

FLAGS_log_file_header flag and related codes exist in the master branch, but not in the release version (0.6.0).
Is it a bug or is there a reason to got rid of it?

The flag was not removed but added after the 0.6.0 release.


ValidationProblemDetails vs ProblemDetails

I posted a lengthy detailed comment about the problem over at NSwag repo but the same problem applies when I tried using Swashbuckle.AspNetCore.   I've been working with both...
https://github.com/RicoSuter/NSwag/issues/2571

I appreciate the OpenAPI spec is platform agnostic which brings up the interesting dilemma of why even include ProblemDetails if it can't be used in any concrete implementations without being changed.  The problematic part of course being Extensions as no vendor that I know of supports that naming convention directly.  Perhaps this isn't an issue for the spec to address but is more a downstream problem and how client generates deal with Extensions???

If I'm understanding your issue correctly, you seem to be saying that Swashbuckle auto-generates "error" response descriptions based on the ProblemDetails type as opposed to the ValidationProblemDetails type. I'm a little confused by this because in fact, Swashbuckle doesn't auto-generate any "error" response descriptions. If you want to include error responses, you need to explicitly decorate your action with ProducesResponseType attributes, and indicate the response type yourself. For example:
[ProducesResponseType(typeof(ValidationProblemDetails), 400)]
public IActionResult CreateOrder([FromBody]Order order)

How embarrassing.  After many hours of looking through documentation and searching the web I did not find this documented anywhere but in retrospect it should have been obvious.  Thanks you!


Support pages under app/pages

Hello,
I'm trying next-translate for a multilingual site using BlitzJS. Although, Blitzjs is based on Nextjs, I couldn't get it to work because in Blitz pages can be created in more places not just in the default top level folders /pages or /src/pages:
BlitzJs default folders
export const topLevelFoldersThatMayContainPages = [   'pages',   'src',   'app',   'integrations', ]
It would be useful if we could have an option to add top level folders that may contain pages. For BlitzJs devs, they would just simply define folders they use for pages in the i18n.json file. Is that possible?

Wasn't this discussed in https://github.com/vinissimus/next-translate/issues/499 and fixed in https://github.com/vinissimus/next-translate/pull/505?

@kumosal If the fix we made works then I will close the issue. Comment if it doesn't work and report it and then we will reopen it. Thanks, @iDavidB to remember it!

still it doesn't work like in nextjs. it works only if I change this line to:
const pagesPath = path.join(dir, arePagesInsideSrc ? '/src/pages' : '/pages')
to:
const pagesPath = path.join(dir, arePagesInsideSrc ? '/src/pages' : '/app/pages')
now next-translate can translate pages under "/app/pages",

now next-translate can translate pages under "/app/pages"? Ok, I didn't know it. So in this case we need to fix it. Thanks to report it.

@kumosal did you try NEXT_TRANSLATE_PATH='./app' yarn dev?
`

Already fixed in 1.1.1-canary.2, it should work without any change. It looks first src then app and then integrations. Also, I add the config variable pagesInDir to support cases as defined on https://github.com/vinissimus/next-translate/issues/682

@aralroca this doesn't fully fix Blitz support, because blitz supports multiple different pages folders throughout the codebase (not just a single pages folder). I see you need pages folder for reading contents of _app. Do you need to manually read pages for anything else?

@aralroca Ye

@flybayer you can pass the pagesInDir in the i18n.js configuration file
https://github.com/vinissimus/next-translate/blob/master/src/plugin/index.ts#L18
And is only need it to know if exist an _app.js or not, and if this _app.js has a getInitialProps. Because it changes the behavior of the loader.

Yes but in Blitz we can have multiple /pages folders, not only one. pagesInDir should be an array, not a string right?


[ArPow Stage 4] Remove any prebuilts and enable regression block for linker

See ArPow stage descriptions for details.

This issue is superseded by https://github.com/dotnet/source-build/issues/3017.