isDirectory() not compatible with z/OS

    isDirectory() {
      return (this.mode & 61440) === 16384;
    }

On z/OS the isDirectory() function always returns false.
The inode masks S_IFMT and S_IFDIR (61440 and 16384) do not seem to be compatible since S_IFMT on z/OS is defined as  0xFF000000
External source: https://www.nntp.perl.org/group/perl.mvs/2014/11/msg1875.html
A potential fix I have tested is to use the constants from the fs module instead of a hardcoded value.  eg: fsNode.constants.S_IFMT

@joliss @stefanpenner
Hi, I've made a PR that fixes https://github.com/joliss/node-walk-sync/pull/72
It would be awesome if we could get it reviewed.  Thanks!


retry bad_wifi after an hour

If a wifi network is marked bad and we're still connected via BT, retry the wifi after an hour to see if the issue was transient.
Also, bash-completion for easier TAB TAB.

Tested successfully; ready to merge if no objections.


Adding plugin support

This will support a plugins configuration item that contains paths to directories containing "plugins". A plugin is a python module that contains an __init__.py. Then you can specify package_name.MyTask in this format in the tasks object:
{
    // ...
    "tasks": [
      {
        "type": "package_name.MyTask"
      }
    ],
    // ...
}

It won't actually be possible to create plugins until we move BaseTask out of the main repo, or come up with some other way to handle extending a core piece.

👍

@douglascamata Please review this even though it has been merged.


feat: add haxe-language-server

@mason-org/triage notify(Reason::NewPullRequest)
This comment ensures a notification is delivered to maintainers.
 🤖 This is an automated comment.    📖 Source code


new resolve #501

I would like to add my profile (I had not seen that the original project had inserted a new user, so I had to redo my request).
Resolve #501

Hey @CarlosViniMSouza the Readme.md file is not necessary in the PR, just the changes in data.json is enough

@CarlosViniMSouza ove added resolve #501 to your pr description thus if the PR get merged the issue will automatically get closed.
And one other thing you don't have to close an old pr of there are changes in the original repo. Just pull the latest changes of the origin, probably resolve some merge conflicts and you are free to go 🚀

OK

I removed my profile from README.md

I'll be clear here @CarlosViniMSouza , so when you opened the issue and uploaded the image there you might have seen some url like this in the picture right? you have to copy that url and paste it in this file. see this example

@starlightknown, my problem is that i don't know how to generate this link!

@starlightknown, my problem is that i don't know how to generate this link!

you don't have to generate it, it generates automatically. could you open an issue and upload a picture and check?

@starlightknown, my problem is that I don't know how to generate this link!

Ok! I will try to explain what @starlightknown is trying to tell. Open the issue and drag and drop the image into the issue section (Where you type the content of the issue). It will upload and look like a link shown by @starlightknown. Hope that helped :smiley:

@starlightknown I figured out how to generate this image! I apologize for the inconvenience!😅 ~ I will correct the url-image part of my PR

@starlightknown I figured out how to generate this image! I apologize for the inconvenience!sweat_smile ~ I will correct the url-image part of my PR

great, no worries. we are here to help :)

I should only have done the following: Take screenshot of the screen, open the issue, and in the issue field, do Crtl + V !! 🖖

@starlightknown I figured out how to generate this image! I apologize for the inconvenience! ~ I will correct the url-image part of my PR

No worries. Hubbers are always ready for help

Thanks @starlightknown and @kaiwalyakoparkar for the great collaboration 🚀

Guys, could you tell me what 'LGTM' means? (I don't know many English acronyms! 🧐)

Guys, could you tell me what 'LGTM' means? (I don't know many English acronyms! )

Nothing much it means "Looks good to me"

LGTM - looks good to me

Thanks for information! ✌🙂


⚠ HawkeUno has degraded performance

In 38b70f9, HawkeUno ($UNO) experienced degraded performance:

HTTP code: 403
Response time: 1697 ms

Resolved: HawkeUno performance has improved in 3adc72a after 7 minutes.


xlights: init at 2023.11

Description of changes
closes https://github.com/NixOS/nixpkgs/issues/247402
xLights is a sequencer for Lights.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 247444 run on x86_64-linux 1

1 package built:

xlights

I would update this to 2023.12 which is the latest (released after this PR) however there's an upstream issue where while the tag/release name in github is equal to the version in the package, the actual build name has not been updated to reflect what the version actually is so you get a situation where the version is 2023.12 which matches the tag, however the build name for the download isn't updated and is still 2023.11.


Fix #999: Prefix CATTLE_HOST_LABELS= to machine labels

@cjellick @will-chan @sonchang

Fixes https://github.com/rancherio/rancher/issues/999

@cjellick updated the PR based on your comments

Waiting for build to go green (currently queued up) and to see the accompanying validation-test PR.


Add integration test for display cutouts on Android screen rotation

Test that the position of a cutout as reported by the Android engine repositions based on screen orientation.
Related to https://github.com/flutter/engine/pull/55992
Part of https://github.com/flutter/flutter/issues/155658
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Until https://github.com/flutter/engine/pull/55992 lands, the newly added test should fail.

https://github.com/flutter/engine/pull/55992 landed, but the CI will fail until the engine changes roll into framework.

Looks like something isn't working as nicely on CI as it is locally... I'll need to check that out

I think you are hitting a timeout I am not sure of the default but assembly which appears to include downloading dependencies is taking  34 seconds "Running Gradle task 'assembleDebug'...                             34.2s

Ah, probably should have rebased on master after the engine roll landed before worrying about the test failure. Let's see now.

The problem was that, with API<35, the default behavior of an app is to make cutouts opaque to the application unless it requests otherwise, so we need to request that in order to access and test the cutouts.
Now, the problem is that FlutterDriver.waitFor never completes, even after its timeout, but only the first time the test is driven after starting an emulator (which is a problem for CI). This only happens when adb is called in setupAll to enable an overlay, which actually simulates the cutout that we then test.
If I could figure out a way to set up the display cutout before the test app launches, this would be easily resolved. But doing it in setupAll is a problem.

I've moved the emulator setup out of setupAll into the dart script that runs the corresponding shard of tests. This way, adb enables the cutout before flutter drive is actually run, and the driver is not broken. I have also added comments to document the requirements of a device/emulator to run these new tests, as well as how and why those requirements are satisfied by our CI.

@matanlurey I started looking at the native_driver tests and their screen rotation methods after your comment on the engine PR about rotation tests. But it will still be possible to relocate it.

I am taking over this pr. Newly added test can be run with flutter drive.

Replaced by https://github.com/flutter/flutter/pull/160255


Altlabel uses JSON not TSV

When the students refactored, they turned the crk.altlabels file into a JSON. I either need to do the same for all the other altlabel files in order to make it work properly, or change if back to using .tsv. I'm going to try the .tsv method first.

Turns out it's much easier to convert a .tsv to a JSON file, so I did that instead. This still needs to be implemented for all the other languages.


install doesn't honor .env

Issue description
pipenv install doesn't use PIPENV_PIPFILE when specified in .env. However,  install uses PIPENV_PIPFILE when it is defined as a normal shell variable.
Expected result
pipenv install doesn't create new Pipfile
Actual result
$ cd /tmp/test
$ find .
.
./.env
./requirements
./requirements/Pipfile
./requirements/Pipfile.lock

$ cat .env
PIPENV_PIPFILE=${PWD}/requirements/Pipfile

$ pipenv install
Creating a Pipfile for this project...
Pipfile.lock not found, creating...

$ export PIPENV_PIPFILE=/tmp/test/requirements/Pipfile

$ pipenv install
Creating a virtualenv for this project...
Pipfile: /tmp/test/requirements/Pipfile

Steps to replicate
$ pipenv --support
Pipenv version: '2018.7.1'
Pipenv location: '/usr/local/Cellar/pipenv/2018.7.1/libexec/lib/python3.7/site-packages/pipenv'
Python location: '/usr/local/Cellar/pipenv/2018.7.1/libexec/bin/python3.7'
Other Python installations in PATH:

2.6: /usr/bin/python2.6

2.6: /usr/bin/python2.6

2.7: /usr/local/bin/python2.7

2.7: /usr/local/bin/python2.7

2.7: /usr/bin/python2.7

3.7: /usr/local/bin/python3.7m

3.7: /usr/local/bin/python3.7

2.7.15: /usr/local/bin/python

2.7.10: /usr/bin/python

2.7.15: /usr/local/bin/python2

3.7.0: /usr/local/bin/python3

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.7.0',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '16.7.0',
 'platform_system': 'Darwin',
 'platform_version': 'Darwin Kernel Version 16.7.0: Tue Jan 30 11:27:06 PST '
                     '2018; root:xnu-3789.73.11~1/RELEASE_X86_64',
 'python_full_version': '3.7.0',
 'python_version': '3.7',
 'sys_platform': 'darwin'}

System environment variables:

PATH
GREP_COLOR
NNTPSERVER
LESS_TERMCAP_mb
TERM_PROGRAM
NODEUP_BUCKET
LESS_TERMCAP_md
LESS_TERMCAP_me
TERM
SHELL
TMPDIR
Apple_PubSub_Socket_Render
TERM_PROGRAM_VERSION
CLUSTER_NAME
LESS_TERMCAP_ue
TERM_SESSION_ID
LC_ALL
USER
LS_COLORS
GREP_COLORS
COMMAND_MODE
SSH_AUTH_SOCK
__CF_USER_TEXT_ENCODING
LSCOLORS
LESS_TERMCAP_us
_
PWD
LANG
ITERM_PROFILE
CLOUDSDK_PYTHON
XPC_FLAGS
HOST_IP
XPC_SERVICE_NAME
GPG_TTY
KOPS_STATE_STORE
SHLVL
HOME
COLORFGBG
IMAGE
ITERM_SESSION_ID
LOGNAME
LESS_TERMCAP_so
LC_CTYPE
FZF_DEFAULT_COMMAND
WEECHAT_HOME
GOPATH
SECURITYSESSIONID
LESS_TERMCAP_se
COLORTERM
PYTHONDONTWRITEBYTECODE
PIP_PYTHON_PATH

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: /usr/local/Cellar/pipenv/2018.7.1/libexec/tools:/Users/pradipcaulagi/.yarn/bin:/Users/pradipcaulagi/.config/yarn/global/node_modules/.bin:/Users/pradipcaulagi/go/bin:/Users/pradipcaulagi/Downloads/google-cloud-sdk/bin:/Users/pradipcaulagi/.cargo/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/go/bin:/Applications/Wireshark.app/Contents/MacOS
SHELL: /bin/zsh
LANG: en_US.UTF-8
PWD: /tmp/test

Contents of Pipfile ('/private/tmp/test/Pipfile'):
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]

[dev-packages]

[requires]
python_version = "3.7"

Contents of Pipfile.lock ('/private/tmp/test/Pipfile.lock'):
{
    "_meta": {
        "hash": {
            "sha256": "7e7ef69da7248742e869378f8421880cf8f0017f96d94d086813baa518a65489"
        },
        "pipfile-spec": 6,
        "requires": {
            "python_version": "3.7"
        },
        "sources": [
            {
                "name": "pypi",
                "url": "https://pypi.org/simple",
                "verify_ssl": true
            }
        ]
    },
    "default": {},
    "develop": {}
}

No it does not, ahdn that is by design. .env is only evaluated by run and shell. The documentation has some suggestion on this issue:
https://docs.pipenv.org/advanced/#configuration-with-environment-variables

@uranusjr thanks! I has missed direnv. However, it seems confusing to use .env for some sub-commands but .envrc for others. Just out of curiosity, why are environment variables via .env not support for all sub-commands?

.env files are loaded by pipenv only for the purpose of and in the context of running your code inside your virtualenv (e.g. running an application in a virtualenv).  This is a context switch at which time pipenv is modifying your environment in other ways (which python you use, your path search location, etc) and therefore it also may alter the context of your environment variables.  We have had a few extensive discussions on this in past issues, feel free to search for those if you'd like to know more.


Add styles for Select component

Closes IBM/carbon-components-angular#
add styles for Select component, for issue #163
Changelog
Changed

light and inline styles as per Carbon design:
http://www.carbondesignsystem.com/components/select/code

Inline:

Light:

Only light styles is necessary as there is already an existing PR for inline styles

@youda97 OK. Thanks for the update. I'll update this PR once the other one is in then, instead of changing now and then having to resolve conflict later.

@cal-smith ready for another round of review :-)


Adding related entity callbacks

In this PR:

Added callbacks to entities that have links to related entities:

Briefcase.getOwner
Changeset.getCreator
NamedVersion.getCreator
NamedVersion.getChangeset

Refactored IModelsClient from @itwin/imodels-client-authoring package to extend the IModelsClient from @itwin/imodels-client-management package. This surfaces all operations from management client to authoring without having to duplicate operations. It also allows for the authoring client to be passed to functions that require management client which is relevant for next item in this list.
Refactored specific operations entity operation classes that require handles to other entity operations, for example NamedVersionOperations that require ChangesetOperations. The constructor was changed to accept whole IModelsClient instead of specific operation instances. This approach has two benefits:

It avoids circular dependencies. We have a scenario where NamedVersionOperations require ChangesetOperations and ChangesetOperations require NamedVersionOperations. This is a circular dependency which is avoided  when both operation classes accept a parent class (IModelsClient).
It does not require to change the constructor when new operations are required.

Added tests and refactored assertions to check the callbacks.

/azp run


legacy_api_password auth provider confusion

Home Assistant release with the issue:
0.101.0
Last working Home Assistant release (if known):
0.100.3
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io on Raspberry Pi 4
Integration:
Core/auth
Description of problem:
I migrated my config some months ago and removed the legacy password entry from the http: component. I didn't get any deprecation warning and it worked as it's supposed to.
However, after upgrading to 0.101.0 I always get a 401: Unauthorized error.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
homeassistant:
  name: Home
  latitude: !secret home_latitude
  longitude: !secret home_longitude
  customize: !include customize.yaml
  customize_glob: !include customize_glob.yaml
  auth_providers:
  - type: legacy_api_password
    api_password:<PHONE_NUMBER>
  - type: homeassistant

Additional information:
A downgrade to 0.100.3 worked fine without having to change anything of the config.
Is there anything I didn't understand correctly? From the release blog post it's a bit hard to understand what has been removed (http: stuff) and what's still there (legacy_api_password auth provider) but as far as I understood from the current docs, the config code above should work fine.

If somebody wonders, this is the sections which are conflicting in my eyes. At least I have no idea which one tells the current truth:
"It is no longer possible to make authenticated requests using trusted networks or by appending ?api_password=X to the URL.
[...]
The support of configuring the auth providers for API Password and Trusted Networks via the HTTP configuration is also removed. It now needs to be configured in the auth provider section (docs)."
I'd also really love to remove api_password but the mystrom component requires this feature to be there.

The auth providers are still there and supported. With auth providers you can get an access token via the authorize flow. It won't allow you to authenticate by passing some data as part of the url.

I didn't get the explanation from @balloob

How can we re-install the api_password auth provider?
Also, the docs haven't been updated. They still say api_password can be used.

I'm pretty sure this is going to get closed. I've started trying to figure out a way to get it back.
https://community.home-assistant.io/t/anyone-working-on-a-way-to-bring-back-api-password/146085?u=bbrendon

Auth providers are used to log in to Home Assistant. So the legacy API password provider will allow you to login to Home Assistant using an API password:

It will not, however, allow you to mark a request as authorized by adding the password to the url.

@balloob thanks for clarification! This indeed makes sense and I've started to migrate away from "api_password" GET parameter now.
For most things, I could simply use webhook automations.
However, some things also required that I could trigger a Node Red node from a webhook which was not that straight forward.
For those who are interested, this is how I call a rest_command from a webhook, which forwards all incoming data to Node Red: https://community.home-assistant.io/t/ha-webhook-node-red-integration/146019


map_delimited_values can edit itself

There's a potential compound-edit bug in map_delimited_values, where subsequent values can be wrongly matched as the end of other values. For example 67; will match the last two characters in 8232967;.
This would come up very rarely I expect. We already match relying on a prepended ;, but we need to boost this to match on enclosing ;. So, instead of matching 67; we need to match ;67;.

Looks like the solution is:

def map_delimited_values(y, value_map, col_name):
    """
    Map the delimited values using the given mapper, dropping unknown responses.
    """

    msg = (
        "Unknown category id '{}' for '{}' found in the ddf."
        " The data for this category id will not be converted "
        "because there is no corresponding metadata.").format

    y = ';{}'.format(y)
    for value in y.split(';')[1:-1]:
        if value in value_map:
            seek = ';{};'.format
            p = re.compile(seek(value))
            y = p.sub(seek(value_map[value]), y)
        else:
            warnings.warn(msg(value, col_name))
            y = y.replace(value+';', '')

    if len(y) > 1: y = y[1:]

    return y

This has been resolved.


want to fetch only first media of user

Please follow the guide below

Issues submitted without this template format will be ignored.
You will be asked some questions and requested to provide some information, please read them carefully and answer completely.
Put an x into all the boxes [ ] relevant to your issue (like so [x]).
Use the Preview tab to see how your issue will actually look like.

Before submitting an issue, make sure you have:

[x] Updated to the latest version
[x] Read the README and docs
[x] Searched the bugtracker for similar issues including closed ones

Purpose of your issue?

[ ] Bug report (encountered problems/errors)
[x] Feature request (request for a new functionality)
[x] Question
[ ] Other
i have user list stored in userlist.txt
which then is used to fetch the user media (upto 12-18)
and then filter and all
i was trying to get user first media
where i used place temp = media_items[0]
and also tried to replace media_items[0] in every function but it didnt workout
i a newbie to json and python
and i am not getting the this what does
self.LastJson("items") return

Paste the output of python -V here:
Python 3.4.3 (v3.4.3:9b73f1c3e601, Feb 24 2015, 22:43:06) [MSC v.1600 32 bit (Intel)] on win32
Code:
# Example code that will produce the error reported
def filter_medias(self, media_items, filtration=True, quiet=False, is_comment=False):
    if filtration:
        temp = media_item[0]
        media_item = temp
        if not quiet:
            self.logger.info("Received %d medias." % len(media_items))
        if not is_comment:
            media_items = _filter_medias_not_liked(media_items)
            if self.max_likes_to_like:
                media_items = _filter_medias_nlikes(
                    media_items, self.max_likes_to_like)
        else:
            media_items = _filter_medias_not_commented(self, media_items)
        if not quiet:
            self.logger.info("After filtration %d medias left." %
                             len(media_items))
    return _get_media_ids(media_items)

Error/Debug Log:
2018-01-02 21:11:49,070 - INFO - Received 12 medias.
2018-01-02 21:11:49,074 - INFO - After filtration 0 medias left.
2018-01-02 21:11:49,078 - INFO - Going to comment 0 medias.
0it [00:00, ?it/s]
2018-01-02 21:11:49,118 - INFO - DONE: Total commented on 0 medias.
2018-01-02 21:11:52,965 - INFO - Received 18 medias.
2018-01-02 21:11:52,972 - INFO - After filtration 4 medias left.
2018-01-02 21:11:52,980 - INFO - Going to comment 4 medias.
  0%|                                                    | 0/4 [00:00<?, ?it/s]2
018-01-02 21:11:54,385 - INFO - Commented with text: wow
 25%|###########                                 | 1/4 [00:20<01:02, 20.84s/it]2
018-01-02 21:12:15,678 - INFO - Commented with text: wow
 50%|######################                      | 2/4 [00:35<00:37, 18.99s/it]2
018-01-02 21:12:30,745 - INFO - Commented with text: haha

Describe your issue
as you can see that after filter
it fetch 4 medias
but what i want it should return only the latest or first media in items ( only one media per user)
so can any one tell me how to alter the media or get_media_id or filter function to get the it done

@sudoguy , @Killerherts
How would you implement this in a code like below (from like_and_follow_last_uers_media_likers.py)
for username in users:     medias = bot.get_user_medias(username, filtration=False)     if len(medias):         likers = bot.get_media_likers(medias[0])         for liker in tqdm(likers):             bot.like_user(liker, amount=2)             bot.follow(liker)
 likers = bot.get_media_likers(medias[1:]) and  likers = bot.get_media_likers(medias[0])[1:] both gives an error.

@MrMdR
likers = bot.get_media_likers(medias[0])

likers = likers[0] if likers or []


Suggestion: do something about replies loading bit by bit with "more tweets" at the bottom

https://twitter.com/grishka11/status/1438608729281339408
This is apparently new and it drives me absolutely nuts. Would be really nice to have a way to undo this and make that page show replies, all of them at once like before, and nothing else.

I found out that adding ?s=20 to the URL reverts the frustration-free behavior of there being no "more tweets" section, all replies being visible and them getting loaded on scroll. So it should be easier than I initially thought.

More Tweets seems to come and go, the hardest thing with trying to deal with it is being able to consistently get it to appear.
I don't want to add any auto-clicking of stuff in tweet timelines because of the potential of Twitter changes which break them or make the auto-clicking do something the user doesn't expxect


Minor bug fix in TrainJob API

What this PR does / why we need it:
Fixing a bug where set used instead of an array.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
NA
Release note:

This change is

/lgtm
/approve


Add stub for ARRAY[]

pg_dump turns expressions like any(1, 2) into any(array[1,2]). This PR counteracts that, and could also function as a stub for full-fledged array support.

Current coverage is 66.95% (diff: 100%)

Merging #731 into develop will increase coverage by <.01%

@@            develop       #731   diff @@
==========================================
  Files             1          1
  Lines          9712       9717     +5
  Methods        1021       1021
  Messages          0          0
  Branches       2895       2897     +2
==========================================
+ Hits           6502       6506     +4
- Misses         3210       3211     +1
  Partials          0          0

Powered by Codecov. Last update 2de4866...6926606

Thank you!


CLI is unable to bundle spec V3 documents using bundle command

Describe the bug
ayncapi bundle command does not support bundling spec v3 documents, running the command on v3 documents gives error
Error: One of the files you tried to bundle is AsyncAPI v3 format, the bundle command does not support it yet, please checkout https://github.com/asyncapi/bundler/issues/133

How to Reproduce
To reproduce this error all you have to do is try to bundle an v3 spec using CLI.
Run the following command to reproduce this error
asyncapi bundle file-name

Expected behavior
asyncapi bundle command should be able to bundle Asyncapi V3 spec files.

+1.

Any news on this ?

Mostly done by @Souvikns, he just needs to update some tests in https://github.com/asyncapi/cli/pull/1138


Add separate summarization for burn vs. transactions to clearly distinguish burns in the console summary output.

Currently, the text summary doesn't show burn separate from staker rewards.
The original impl on PR #104 did separate out the burn, but with the changes in pr #107 it was harder to calculate burn. It would be much better to split out the burn.
For example:
------------------- Summary & Totals ------------------
Calculations based on entire pool
Net Coinbase Received: 7202.642800018
Total Amounts Due To Stakers / To Burn: 6842.57039671
Net MINA to Pool Operator (after send transaction fees):<PHONE_NUMBER>08
Total Coin Base Generated: 7200
User Transaction Fees Generated (net of snark fees): 2.642800018
Total Snark Fees Paid From Coinbase: 0
Total Payout Transaction Fees: 0.016
------------------- Summary & Totals ------------------
Should instead be:
Total Amount Due to Stakers: 123
Total Amount to Burn:  23

Implemented in PR #129


[FLINK-21466][table] Add SQL Client default mode value embedded

1.Add SQL Client default mode value embedded.
2.Modify sqlClient md,support default mode value embedded.
3.Test sql client default mode embedded.only left a test which uses embedded to verify it still work.

I have appended a commit to remove useless embedded and also added a test for SqlClient.
Would be great if @fsk119 can have a look again.

Merging...


Error: length < 0 in createTextSpan during getApplicableRefactors

tsserver version: 2.6.1
Error: length < 0
at createTextSpan (tsserver.js:8817:19)
at Object.createTextSpanFromBounds (tsserver.js:8823:16)
at getErrorSpanForNode (tsserver.js:6264:19)
at createDiagnosticForNodeInSourceFile (tsserver.js:6195:20)
at Object.createDiagnosticForNode (tsserver.js:6191:16)
at checkNode (tsserver.js:77699:36)
at getRangeToExtract (tsserver.js:77655:54)
at Object.getAvailableActions (tsserver.js:77509:38)
at tsserver.js:75017:128
at Object.flatMapIter (tsserver.js:1570:23)
at Object.getApplicableRefactors (tsserver.js:75016:23)
at Object.getApplicableRefactors (tsserver.js:80232:32)
at IOSession.Session.getApplicableRefactors (tsserver.js:86474:53)
at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (tsserver.js:85436:61)
at tsserver.js:86669:88
at IOSession.Session.executeWithRequestId (tsserver.js:86660:28)
at IOSession.Session.executeCommand (tsserver.js:86669:33)
at IOSession.Session.onMessage (tsserver.js:86689:35)
at Interface. (tsserver.js:87881:27)
at emitOne (events.js:96:13)
at Interface.emit (events.js:191:7)
at Interface._onLine (readline.js:241:10)
at Interface._normalWrite (readline.js:384:12)
at Socket.ondata (readline.js:101:10)
at emitOne (events.js:96:13)
at Socket.emit (events.js:191:7)
at readableAddChunk (_stream_readable.js:178:18)
at Socket.Readable.push (_stream_readable.js:136:10)
at Pipe.onread (net.js:560:20)

Waiting for asserts to be hit (not yet shipped).

We had a telemetry hit in VS Code 1.28.1 with TS 3.1.1:
Error processing request. Debug Failure. False expression: This failure could trigger https://github.com/Microsoft/TypeScript/issues/20809
Error: Debug Failure. False expression: This failure could trigger https://github.com/Microsoft/TypeScript/issues/20809
    at getErrorSpanForNode (tsserver.js:8839:22)
    at createDiagnosticForNodeInSourceFile (tsserver.js:8752:20)
    at Object.createDiagnosticForNode (tsserver.js:8743:16)
    at checkNode (tsserver.js:111533:36)
    at getRangeToExtract (tsserver.js:111452:53)
    at Object.getAvailableActions (tsserver.js:111260:38)
    at tsserver.js:106434:128
    at getIterator (tsserver.js:535:23)
    at Object.next (tsserver.js:530:35)
    at Object.arrayFrom (tsserver.js:1135:32)
    at Object.getApplicableRefactors (tsserver.js:106433:23)
    at Object.getApplicableRefactors (tsserver.js:115353:32)
    at IOSession.Session.getApplicableRefactors (tsserver.js:122853:53)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (tsserver.js:121787:61)
    at tsserver.js:123061:88
    at IOSession.Session.executeWithRequestId (tsserver.js:123052:28)
    at IOSession.Session.executeCommand (tsserver.js:123061:33)
    at IOSession.Session.onMessage (tsserver.js:123081:35)
    at Interface.<anonymous> (tsserver.js:124344:27)
    at emitOne (events.js:116:13)
    at Interface.emit (events.js:211:7)
    at Interface._onLine (readline.js:282:10)
    at Interface._normalWrite (readline.js:424:12)
    at Socket.ondata (readline.js:141:10)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at Pipe.onread (net.js:594:20)

Closing in the absence of a concrete repro


[Helm] Clean up RayCluster Helm chart ahead of KubeRay 0.4.0 release

Why are these changes needed?

This PR cleans up the ray-cluster Helm chart.

Removes unnecessary and deprecated fields.
Adds some comments
Renames initArgs to rayStartParams in a backwards-compatible way.

Related issue number

Closes https://github.com/ray-project/kuberay/issues/739
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests: I deployed a Ray cluster using the default chart.
[ ] This PR is not tested :(

This PR deletes a lot of redundant environment variables (I am not sure how to check whether they are redundant or not). An environment variable may be used by KubeRay or Ray. Is there any document about environment variables for Ray? Others look good to me.

I can certify that none of the environment variables are used by recent versions of Ray or KubeRay.
I did a similar cleanup of RayCluster CRs just before the 0.3.0 release.


Using disruptor as an pipeliner.

Hello, I'd like to have a ringbuffer like structure that does the following job,

That is, to use a ringbuffer to merge a bunch of sequenced records (1, 2, 3 ...) into one sequence.

You can build this already with two instances of the Disruptor.  The first fans out to 3 parallel handlers, then each of those publisher sends events to a 2nd instance of the Disruptor.  The single event handler on the 2nd Disruptor would take care of the reordering of events.

The single event handler on the 2nd Disruptor would take care of the reordering of events.

do you mean I have to manually sort the incoming events? what if there is one missing while the ringbuffer becomes full?

Actually there is an easier way.  If you have 4 consumers where 3 of them are processing the data in parallel.  A hash on the sequence number is the simplest solution.  Then the 4th consumer can gate on the other 3.

Hmm, I don't follow. If there are two disruptors, which sequence should the 4th consumer gate on? Could you give an simple code snippet of this task. Thanks!

You would only need a single Disruptor.

I'll look at a code snippet tomorrow.

I have add a new method to MultiProducerSequencer,
    @Override
    public void seq(long seq) {
        if (seq < 0)
        {
            throw new IllegalArgumentException("seq must be >= 0");
        }

        long current;
        do
        {
            current = cursor.get();
            if (seq < current) {
                return;
            }
            long wrapPoint = seq - bufferSize;
            long cachedGatingSequence = gatingSequenceCache.get();

            if (wrapPoint > cachedGatingSequence || cachedGatingSequence > current)
            {
                long gatingSequence = Util.getMinimumSequence(gatingSequences, current);

                if (wrapPoint > gatingSequence)
                {
                    waitStrategy.signalAllWhenBlocking();
                    LockSupport.parkNanos(1); // TODO, should we spin based on the wait strategy?
                    continue;
                }

                gatingSequenceCache.set(gatingSequence);
            }
            else if (cursor.compareAndSet(current, seq))
            {
                break;
            }
        }
        while (true);
    }

However this makes all blocking wait strategy useless. Since they only rely on the cursor position. I tried to find a multi-cursor implementation but failed. Is it the correct way to do this? Can we have a multi-cursor, single gating sequence implementation?

You can do this without modifying the Disruptor.  I'll post a code snippet later today.

This a way that you can get the behaviour that you want.

Thanks!  But this seems not be able to fully parallel. Will all the parallel handlers be waiting when the join handler is working? And I cannot do pipeline partitioner like this way,

The parallel handlers won't be waiting while the join handler is running.  The join handler will wait until each  has finished processing a given sequence.  However only one of the handler will actually do the work, the other 2 will skip it.  So the processing is not perfectly parallel, but can work well enough if the work done on each item is higher than the overhead of the Disruptor and reasonably even for each event.
The layout you mention is possible, you would need 3 join handler instances.
        disruptor.handleEventsWith(
            new ParallelHandler(0, 3),
            new ParallelHandler(1, 3),
            new ParallelHandler(2, 3)
        ).then(
            new JoiningHandler(1),
            new JoiningHandler(2),
            new JoiningHandler(3));

cool! if some JoiningHandler hangs, will the other two keep going?

Hmm, isn't that going to have a tons of temporary items passing from ParallelHandler to JoiningHandler?

Depends on how you implement the result.  You could use a preallocated object and fill it from within the Parallel handler and the Disruptor will reuse the event later on.
The other JoiningHandlers will run in parallel.  However if the JoiningHandler hangs indefinitely then eventually the ring buffer will fill up and block.

ok. Can't quite tell the overhead though. I may have up to 128 partitions and this would lead to 128 JoiningHandlers, which will try to sequence up unnecessary 127 times for every parallel event. And I may need a very large ring buffer...  However, it's illuminating :smile: t

If the tasks are not doing I/O I would keep the total number of handlers less than the number of available cores.  Obviously benchmarking will determine the most appropriate configuration.  Also if your number of handlers is a power of 2, you can use a mask as the modulo operation.

Well, hmm, isn't there a inter dependency between paralleler and joiner ? If I have 16 logical cores, how about using 16 paralleler and 16 joiner?

Assuming that you have sufficient data as input you could have all 32 handlers running at the same time.  The interplay between the parallel and joiner handlers is an asynchronous notification, so the parallel handler can move onto processing more events without waiting for the joiner.  Everything is bounded by the size of the buffer though.

hmm, there is a problem. Those joiners may start processing 1:2 before 1:1 which is unacceptable.

Hmm, this architecture turns out to be very slow in real applications,

Jprofiler shows the dependency is too complex to be handled.  It turns out that Sleep strategy is the best for tangled dependency in disruptor, but still quite slow.

JProfiler is not suitable for this type of profiling. You need to use perf-map-agent, Flight Recorder, or Linux perf.

Thanks. I'll carry some more profilings. But this is indeed running much slower than manually splitted queues. I think small fine-grained waiting barriers is better for this partitioning pipeliner. Or is there a better way to do this in disruptor?

Just did a flamegraph  https://la.wentropy.com/5mvJ   .   This shows nearly 65% of time is consumed by some waiting events.

Further testing shows that single parallel handler's performance is the same with eight parallel handler.

Are you testing with your real workload or the example that I created?

real workload

oh, the flamegraph is from your example

Regarding the 8 threads vs 1 thread.  This is not uncommon.  If the amount of work to do per event is quite small, then the overhead of parallelising the workload means there is no overall benefit.  The other common situation is that the workload is memory bound so when running multiple instances in parallel they end up contending on L3 cache and no really reaping the benefit of having additional CPU cores.
Have you tried using an alternative framework for your world load (e.g. Fork/Join)?

yes, a simple fork join with a concurrent linked queue is three times faster. I have tried tuning the task size, varying from 4k to 1M. There isn't a difference though.

and an interestibg fact is, sleeping wait strategy is 80% faster than all the others.

Not really surprised that Fork/Join is faster.  If your workload works nicely with Fork/Join it is probably not a good fit for the Disruptor.  The Disruptor works best for asynchronous message based services.
The SleepingWaitStrategy is fine when you are hammering it flat out, but if you have no input for a short period of time and the handler needs to idle, you have a long wake up time before it starts processing events again (~60µs).  There is a trade off here between latency and throughput.

Ok. Um, so what's the suggested library that can implement this partitioned pipeliner efficiently?

Assuming that this is a large batch operation, I would be looking at something using a model similar to fork/join or parallel streams.  In addition to the libraries shipping with the JDK, you also may want to look at the Goldman Sach's collections which has an alternative implementation of parallel streams.


Ticket/2.7.x/10940 deprecate apply for catalog

Without this patch, the puppet apply command uses the --apply option
to reference a Puppet catalog. This can cause a bit of confusion as both
the subcommand and option have the same name.
This patch improves the usability of the puppet apply command by
adding a new --catalog option. This patch also adds a deprecation
warning whenever the --apply option is used, which advises end-users
to use the --catalog option instead.
This patch updates the in-line help documentation by adding help output
for the new --catalog option, and adding a deprecation warning for the
--apply option.
The plan is to deprecate the --apply option in the next 2.7.x release,
and removal in the future. External documentation should be updated to
reflect the new preference towards using the --catalog option in-place
of --apply.
Preferred method of applying a Puppet catalog using puppet apply:
$ puppet apply --catalog <catalog>

This patch also includes the spec tests covering the changes in
behavior.

Reviewed by Carl Caum<EMAIL_ADDRESS>


(HTTP code 500) server error - Get https://registry-1.docker.io/v2/library/elasticsearch/manifests/latest: Get https://auth.docker.io/token?account=lijieking&scope=repository%3Alibrary%2Felasticsearch%3Apull&service=registry.docker.io: net/http: request canceled (Client.Timeout exceeded while awaiting headers)

Expected behavior
Actual behavior
Information about the Issue
(HTTP code 500) server error - Get https://registry-1.docker.io/v2/library/elasticsearch/manifests/latest: Get https://auth.docker.io/token?account=lijieking&scope=repository%3Alibrary%2Felasticsearch%3Apull&service=registry.docker.io: net/http: request canceled (Client.Timeout exceeded while awaiting headers)
Steps to reproduce the behavior

...
...

Having the same issue

Looks like the elasticsearch repository (https://hub.docker.com/_/elasticsearch/) does not have a :latest tag, so you will have to specify a specific tag to use; here's the steps to do so:

Click on the ... button to view the image's details

Click on the Selected tag: latest to open the "tag selection" window

Click the tag / version of the image you want to run

Click the (X) to close the tag selector window

Click on the Create button to pull the image and start the container

Closing this ticket, because this is not a bug, but feel free to continue the conversation


[Go Live][June 21]

Docs Build status updates of commit e814466:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/post-publish/teams-store-ranking-parameters.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/deploy-and-publish/appsource/publish.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/browser-device-permissions.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/device-capabilities-overview.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/location-capability.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/media-capabilities.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/native-device-permissions.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/qr-barcode-scanner-capability.md
:white_check_mark:Succeeded
View

msteams-platform/graph-api/import-messages/import-external-messages-to-teams.md
:white_check_mark:Succeeded
View

msteams-platform/graph-api/proactive-bots-and-messages/graph-proactive-bots-and-messages.md
:white_check_mark:Succeeded
View

msteams-platform/graph-api/rsc/resource-specific-consent.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/overview.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/prerequisites.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/add-authentication.md
:white_check_mark:Succeeded
View

msteams-platform/overview-solution.md
:white_check_mark:Succeeded
View

msteams-platform/resources/bot-v3/bots-test.md
:white_check_mark:Succeeded
View

msteams-platform/resources/bot-v3/bots-with-tabs.md
:white_check_mark:Succeeded
View

msteams-platform/sbs-meeting-token-generator.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-meetings-sidepanel.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-meetings-stage-view.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-send-proactive.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-tab-device-permissions.yml
:white_check_mark:Succeeded
View

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


CDC should validate request's cluster ID

Bug Report

TiKV does not validate CDC request's cluster-ID. TiKV should send ClusterID Mismatch error instead of RegionNotFound error.
There is no CDC nodes.

TiKV keeps logging CDC RegionNotFound error.

What version of TiKV are you using?

5.0.0

@JinLingChristopher Could you take a look?

I will work on this issue

@divyaguduru Are you still looking at this problem? I'd like to try to fix it.

/assign

@overvenus Can you help provide more guidance please? At the moment I'm not quite sure how we should validate it? It looks like we can get the cluster id from the header, but how do I verify it? Also, how can I reproduce it?

It looks like we can get the cluster id from the header, but how do I verify it?

Yes, cluster id is attached in the header.

At the moment I'm not quite sure how we should validate it?

You can add a new field (cluster_id) in Service. TiKV validates a request when TiKV receives it in event_feed gRPC handler.
https://github.com/tikv/tikv/blob/3f0c72a389b4987c27b3dc42d8931a7ecaff9638/components/cdc/src/service.rs#L275-L280

Also, how can I reproduce it?

I think adding a unit test that covers it is enough. It is not necessary to reproduce it in a real cluster.


Webcam not appearing in pose-detection-new branch

When I open the Squats page on the pose-detection-new branch, the webcam is blacked out.

Fixed in commit: https://github.com/AI-Coach-PT/ExerSights/commit/eb19b9f8ecc9c3126f5b0970d2423bfda20e791c
In Webcam component, I changed style from "display:none" to "visibility:hidden, position:absolute"


DRAFT - KNN refactor, but passing distance as function, not precomputed - DO NOT MERGE

An experimental version of https://github.com/alan-turing-institute/sktime/pull/1998, where the distance is passed to sklearn as a function (lazy evaluation), as opposed to the PR https://github.com/alan-turing-institute/sktime/pull/1998 which precomputes the distance matrix.
This is for completess of the record only, DO NOT MERGE.
The runtime behaviour of passing the distance as a function is much worse than precomputing, as it appears.

With the better solution in https://github.com/alan-turing-institute/sktime/pull/1998, I think we can happily put this attempt of mine on the trash heap.


Install documentation

updates to the README following discussions about how easy it is to follow

It's good that you're making these installation instructions easier to follow. Can you also add some Jenkins job to build these (e.g., using LXC containers)? Let us know if you need some help with LXC, as Yiyi and Tim have been using it for their testing.

@darko-marinov I'm not sure what you mean? Build what?
As in run the installation instructions from a fresh install?

Yes, I meant run installation in some Jenkins job.

Jenkins: test this please

Jenkins: test this please

@shd101wyy it's a problem in the Makefile, working on fix now

@ehildenb I am migrating evm-semantics-pull-request job to run on rvwork-2-big-jenkins-container node. Seems like jenkins job failed again. Please let me know If I need to do anything. Thank you.

@shd101wyy I think it is an issue with your migration, because my buildbot server is running the tests fine. Can you take a look at the error messages? Let me know if you have questions. Seems to be something about not being able to find prelude.k.

Jenkins: test this please

@ehildenb It's working now. Something started kserver, which caused that error.

Jenkins: test this please

Jenkins: test this please

Jenkins: test this please

Jenkins: test this please

looks okay still but I have a question I'd like answered before we merge

@dwightguth changed as requested


eric to create prototype for new structure

Three main organizational sections: payload (comes from the schema), description (may just extend the current notes stuff, and certainly contains notes), and examples.
Editors to review at 2 April editor's working meeting

Completed


Review category

Coverage decreased (-0.2%) to 86.142% when pulling db239829a7a5c9b0632c7fc6d6f3353469859f48 on review-category into a01c249da506bef52cb909d24759918b9a01ebfe on master.


fix(chat): fix attachment download path & name (Issue #2008)

Description:

fix link path and file name in nested attachments

Issues:

Issue #2008

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


fixed: AssemblyContentProvider can return None if some of entities th…

…rows on MembersFunctionsAndValues property getter
streamline and fix ResolveUnopenedNamespaceTagger logic
I'm surprised it's been working (sort of) all this time :(

The exception MembersFunctionsAndValues raised was like this:

I thought we know when FCS throws exceptions, but apparently we don't.


Fix a few typos in tutorials

Please confirm that this PR has done the following:

[ ] Tests Added
[ ] Documentation Added
[x] Name of contributors Added to AUTHORS.rst
[x] Description in RELEASE_NOTES.md Added

Description of PR
I've just fixed a few typos in the tutorials.

Thank you @danielhuppmann for making those great tutorial notebooks publicly available for free!


Contributor Role Blueprint NotActions change

The contributor NotActions relevant to blueprints have a wrong description. You can create, edit and delete blueprint artifacts with Contributor role, but you can't assign or delete assignments for blue print artifacts.
Current description (wrong) for NotActions:
Microsoft.Blueprint/blueprintAssignments/write	Create or update any blueprint artifacts
Microsoft.Blueprint/blueprintAssignments/delete	Delete any blueprint artifacts
Description suggested for NotActions:
Microsoft.Blueprint/blueprintAssignments/write	Create or update any blueprint assignment
Microsoft.Blueprint/blueprintAssignments/delete	Delete any blueprint assignment
Or maybe another suggestion for NotActions:
Microsoft.Blueprint/blueprintAssignments/*	Create and manage blueprint assignments.

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

ID: 99581afb-7950-6daa-7227-055d04446b7e
Version Independent ID: f1a5bf89-9e54-aede-3a9a-70788a95927b
Content: Built-in roles for Azure resources
Content Source: articles/role-based-access-control/built-in-roles.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

To clarify, I'm talking about Contributor role description in https://docs.microsoft.com/en-us/azure/role-based-access-control/built-in-roles#contributor

@chirurico Thanks for the comment. We are actively investigating on the issue and will get back to you shortly with an update.

@chirurico Thanks for catching this. A PR has been successfully initiated to resolve this issue. The changes will be live - in roughly 24 hrs.
We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Fix cannot mount PVC in driver (#174)

Issue #, if available:
174
Description of changes:
Since it's unclear why the driver cannot read /host/proc/mounts (with Karpenter) at the first time, retry to read /host/proc/mounts will success.
It is necessary to keep investigating the root cause, but this issue currently affects many Karpenter (and possibly other) users. To prevent users from having to manually restart s3-csi-node-xxx every time, it is better to solve the issue by retrying the read in the code.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Passed locally test


Experiment: ResultHandler class

replacing createResultHandler with new ResultHandler()

getPositiveResponse —> just positive;
getNegativeResponse —> just negative;
both accept either:

Zod Schema,
ApiResponse configuration,
An Array of such configurations,
A function returning an above mentioned entity:

for positive that function accepts output: IOSchema,
for negative the function has no arguments.

almost there, need some more polishing


PDBFile image_base should be unsigned long long

In pdb_file.h
PDBFile::initialize() aswell as PDBFile::parse_sections() should take the image_base as unsigned long long....
idk why it's passed as an int 🙈 RIP 64bit on 32bit archs

Fixed in b6442ef57dbd8322dc4a4af56f816cf143e026f9, but...

I also changed most other addresses for uint64_t.
Some structures containing addresses look like taken from some 32-bit specifications and are used in a way that expects 4-byte address entry, these cannot be suddenly changed for 8 byte int. I hope none of those I changed are used in this way, because that woudl cause problems.
This entire library (pdbparser) is really outdated and not worth maintaining, we should use something else (LLVM?) instead, because any larger changes or improvements are pretty much put of the question.


handle LocalizationConfusion memory consuption and upgrade min weasyprint version

This PR:

update references to avoid that machines with less memory will be killed while computing the metric (workaround to close #1042 )
bump weasyprint to >=55.0 (html to pdf bugfixes >=54.0 new pango version)

Any feedback is welcome :hugs:

failing test related to #1063 (will work after this is merged - i have updated the dummy model on hf hub)

@frgfm now better ? :)


Verbose unicode expansion of Lean code

When writing Lean, which heavily uses unicode, my git diffs with difftastic look like this:

Whereas, e.g., delta looks like this:

Is there any way to tell difft to not escape unicode?

Could you share a sample file?

You should get a similar diff between this commit https://github.com/Seppel3210/batteries/commit/e3ffef5301b0c391097368dc3b4d372934381418 and its parent

I guess a file that's a bit smaller would be this https://github.com/Seppel3210/batteries/blob/main/Batteries/Data/Fin/Lemmas.lean

I can't reproduce I'm afraid:

Could you confirm your difftastic version, your operating system, and your terminal emulator?

FYI I'm on a trip without my laptop, so I'll take a couple days to respond

this wasn't an issue with delta, as it uses itself as the pager


Added Persian Language

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

Thanks @ajahangard, but AFAIK Persian uses fa-IR Am I right?

Thanks @ajahangard, but AFAIK Persian uses fa-IR Am I right?

fa is the general code for Persian and fa-IR is for Persian(Iran), like ar and ar-SA.
if it is a requirement I can change file names.

Ya, I know it's general ;) but I'm not sure if any other than Iran use fa

If it's YES, please revert your last commit
Thanks

If it's YES, please revert your last commit
Thanks

I checked and I chose to use fa-IR rather than fa. so I changed the files and committed the changes. Should I so anything else?

I reverted my commits and recommitted the correct culture files.


Create soil temperature model

Crop planting dates depend heavily on soil temperatures. This data is not included in the weathergen files, being site specific, thus an appropriate model linking soil temperature and the variables from weathergen is required.

Research trail leads to: Gupta, S. C., J. K. Radke, and W. E. Larson. 1981. Predicting Temperatures of Bare and Residue Covered Soils With and Without a Corn Crop1. Soil Sci. Soc. Am. J. 45:405-412. doi:10.2136/sssaj1981.03615995004500020034x
however this may be hard to access.


Update common-deployment-errors.md

OperationNotAllowed
We get same error while resizing data disk for classic VM SKU: Standard_D4_v3.
This is not supported when VM is powered on. Post deallocating the VM we are able to do so.
Reason of failure: When we are using a classic VM SKU, it might not support expanding disks without downtime.
https://learn.microsoft.com/en-us/azure/virtual-machines/linux/expand-disks?tabs=ubuntu#expanding-without-downtime-classic-vm-sku-support.
I have tested the same in lab with Standard_D4_v3 size VM. Initially I also got same error as expected when VM was powered on:

@ManishaSinha0410 - thanks for this update!
#sign-off


Redo squeak detail item reply thread display

The Material UI "timeline" component for the selected squeak can have just he persons name in the TimelineContent section.
Then the actual content of the squeak can be below the timeline, and use the full width.
Something like this:

The main squeak detail component and the replies should be separate from the Material UI timeline.

https://github.com/yzernik/squeaknode/pull/964
https://github.com/yzernik/squeaknode/pull/965


[AWS SDK Instrumentation] Trace does not report HTTP status in attributes

Description
I was using the opentelemetry-ruby SDK to instrument my call to S3 using the opentelemetry-instrumentation-aws_sdk gem. I noticed that the populated attributes are as follows:
#<struct OpenTelemetry::SDK::Trace::SpanData name="S3.ListBuckets",
kind=:client,
status=
#<OpenTelemetry::Trace::Status:0x00007f9b14253508 @code=1, @description="">,
parent_span_id="\xA1\xA7\x11\xB0\xAEGp\xC5",
total_recorded_attributes=4,
total_recorded_events=0,
total_recorded_links=0,
start_timestamp=1642709302898094000,
end_timestamp=1642709303152641000,
attributes=
 {"aws.region"=>"us-west-2",
  "rpc.system"=>"aws-api",
  "rpc.method"=>"ListBuckets",
  "rpc.service"=>"S3"}, # <- no http.status

I noticed that the attributes did not have the http.status attribute set. In the opentelemetry-instrumentation-aws_lambda gem we should be able to popoulate this attribute using meatadata from the AWS SDK response here:
https://github.com/open-telemetry/opentelemetry-ruby/blob/f0593e7d97c62b10862c382d14f533dbc49be580/instrumentation/aws_sdk/lib/opentelemetry/instrumentation/aws_sdk/handler.rb#L40-L42
We do something similar in the OTel Python Instrumentation of the AWS SDK:
try:
    result = original_func(*args, **kwargs)
    _apply_response_attributes(span, result)

...

metadata = result.get("ResponseMetadata")

...

status_code = metadata.get("HTTPStatusCode")

if status_code is not None:
    span.set_attribute(SpanAttributes.HTTP_STATUS_CODE, status_code)

If this makes sense I can add a PR to populate this attribute 🙂 There might some small differences in AWS SDK Ruby, but should still be accessible.
Thanks.

If this makes sense I can add a PR to populate this attribute 🙂 There might some small differences in AWS SDK Ruby, but should still be accessible.

@NathanielRN contributions are welcome!

Is this present on the underlying HTTP instrumentation client span? Maybe the issue here is that the aws-sdk instrumentation doesn't enrich the downstream http span, but instead generates another layer.

@ericmustin

Is this present on the underlying HTTP instrumentation client span? Maybe the issue here is that the aws-sdk instrumentation doesn't enrich the downstream http span, but instead generates another layer.

I can post the entire console log output here for your reference! Not sure about "enriching" a downstream span... but the information is available from a "AWS Ruby SDK" viewpoint and we should probably take advantage of asking the AWS SDK what it thinks the response code should be when setting it on what is supposed to be the "AWS SDK span". I see an http.status_code = 200 but that's for my Rails server request. Let me know if that makes sense! I posted a PR that hopefully solves this issue.
Processing by ApplicationController#aws_sdk_call as */*
#<struct OpenTelemetry::SDK::Trace::SpanData
 name="S3.ListBuckets",
 kind=:client,
 status=
  #<OpenTelemetry::Trace::Status:0x00007fcd342afc70 @code=1, @description="">,
 parent_span_id="7\xCC7)v\xE5#m",
 total_recorded_attributes=4,
 total_recorded_events=0,
 total_recorded_links=0,
 start_timestamp=1642723862714061000,
 end_timestamp=1642723862968598000,
 attributes=
  {"aws.region"=>"us-west-2",
   "rpc.system"=>"aws-api",
   "rpc.method"=>"ListBuckets",
   "rpc.service"=>"S3"},
 links=nil,
 events=nil,
 resource=
  #<OpenTelemetry::SDK::Resources::Resource:0x00007fcd31fc1858
   @attributes=
    {"service.name"=>"aws-otel-manual-rails-sample",
     "process.pid"=>20334,
     "process.command"=>"bin/rails",
     "process.runtime.name"=>"ruby",
     "process.runtime.version"=>"2.7.5",
     "process.runtime.description"=>
      "ruby 2.7.5p203 (2021-11-24 revision f69aeb8314) [x86_64-darwin20]",
     "telemetry.sdk.name"=>"opentelemetry",
     "telemetry.sdk.language"=>"ruby",
     "telemetry.sdk.version"=>"1.0.2"}>,
 instrumentation_library=
  #<struct OpenTelemetry::SDK::InstrumentationLibrary
   name="OpenTelemetry::Instrumentation::AwsSdk",
   version="0.1.0">,
 span_id="\xCD\xCF\xD6\xFCOr\x19\xF8",
 trace_id="a\xE9\xFA\x16\xA1>\xCD\"vB\xA3\xF27a\xDD\xB6",
 trace_flags=#<OpenTelemetry::Trace::TraceFlags:0x00007fcd354b56e0 @flags=1>,
 tracestate=#<OpenTelemetry::Trace::Tracestate:0x00007fcd354ac798 @hash={}>>
Completed 200 OK in 280ms (Views: 0.2ms | Allocations: 32415)

#<struct OpenTelemetry::SDK::Trace::SpanData
 name="ApplicationController#aws_sdk_call",
 kind=:server,
 status=
  #<OpenTelemetry::Trace::Status:0x00007fcd342afc70 @code=1, @description="">,
 parent_span_id="\x00\x00\x00\x00\x00\x00\x00\x00",
 total_recorded_attributes=6,
 total_recorded_events=0,
 total_recorded_links=0,
 start_timestamp=1642723862535508000,
 end_timestamp=1642723862973336000,
 attributes=
  {"http.method"=>"GET",
   "http.host"=>"<IP_ADDRESS>:8080",
   "http.scheme"=>"http",
   "http.target"=>"/aws-sdk-call",
   "http.user_agent"=>"curl/7.64.1",
   "http.status_code"=>200},
 links=nil,
 events=nil,
 resource=
  #<OpenTelemetry::SDK::Resources::Resource:0x00007fcd31fc1858
   @attributes=
    {"service.name"=>"aws-otel-manual-rails-sample",
     "process.pid"=>20334,
     "process.command"=>"bin/rails",
     "process.runtime.name"=>"ruby",
     "process.runtime.version"=>"2.7.5",
     "process.runtime.description"=>
      "ruby 2.7.5p203 (2021-11-24 revision f69aeb8314) [x86_64-darwin20]",
     "telemetry.sdk.name"=>"opentelemetry",
     "telemetry.sdk.language"=>"ruby",
     "telemetry.sdk.version"=>"1.0.2"}>,
 instrumentation_library=
  #<struct OpenTelemetry::SDK::InstrumentationLibrary
   name="OpenTelemetry::Instrumentation::Rack",
   version="0.20.1">,
 span_id="7\xCC7)v\xE5#m",
 trace_id="a\xE9\xFA\x16\xA1>\xCD\"vB\xA3\xF27a\xDD\xB6",
 trace_flags=#<OpenTelemetry::Trace::TraceFlags:0x00007fcd354b56e0 @flags=1>,
 tracestate=#<OpenTelemetry::Trace::Tracestate:0x00007fcd354ac798 @hash={}>>

@NathanielRN thanks for creating the PR with the requested change.
IMO http.status attribute belongs to the child HTTP span. From the SDK client point of view, the request can success or fail, and if I remember correctly the http status code is not returned from the SDK library when using it.
On the other hand, I can see how this attribute might be meaningful in some cases and I think we can add it also for the AWS SDK span.

@YanivD thanks for your feedback!
I can totally see this being useful to the child HTTP span, but on the other hand, users seem to treat AWS SDK as the terminal node in tracing. Even the OTel Ruby Docs for the AWS SDK Instrumentation suggest you use the instrumentation with suppress_instrumentation: true. This would eliminate the HTTP child span if I understand correctly?
In our service, we’ve found this attribute valuable. The HTTP status code can be used a lot by the backend to customize the visualization. At AWS, we have CI tests to make sure this value shows up across several OTel language which is what prompted the change. We were thinking of making an exception in https://github.com/aws-observability/aws-otel-test-framework/pull/522 but thought instead we could just contribute the change upstream right away 🙂
I can also see it being useful for filtering  through AWS SDK spans in the backend by status code.
Finally, I think this span is the right place for it because at the end of the day it is still metadata populated by the AWS SDK itself, not necessarily by the underlying HTTP client used. In this way our instrumentation will be enhanced because it provides insight into what AWS SDK itself thinks about the result and makes that information available to the user in the trace which they normally would have to jump through hoops to go access.
Please let me know what you think! Happy to discuss whatever we can to move this along 🙂

Even the OTel Ruby Docs for the AWS SDK Instrumentation suggest you use the instrumentation with suppress_instrumentation: true. This would eliminate the HTTP child span if I understand correctly?

You understand it correctly.
I thing we can move on with this change and add http.status 🙂

@NathanielRN
I'm adding http.status_code also for JS instrumentation 🥳
https://github.com/open-telemetry/opentelemetry-js-contrib/pull/844


Fee categories when sending

We have few categories of 10 min, 30 mins and 60 mins. I don’t think that is standard or a good option

Completed implemented in Design


Add paulrouget to the list of reviewers

This is mostly to be able to do @bors-servo r=XXX. Often I get a simple r=me, without the @bors-servo snippet.
Also, happy to review anything about embedding and ports.

This change is

:pushpin: Commit f886db7 has been approved by jdm

@bors-servo r+

:hourglass: Testing commit f886db70d8b83dce31dfe0a57d555b32c5ca9ae5 with merge c0828c62519d8270bc8adb05a49b364283199625...

:sunny: Test successful - status-travis
Approved by: jdm
Pushing c0828c62519d8270bc8adb05a49b364283199625 to master...

Thank you.


Fix --dirname-pattern {target} placeholder (#580)

This PR fixes issue #580
The issue is that when downloading highlights using --dirname-pattern, the {target} placeholder was being expanded to {profile}/<highlight-title>, with no way to separate these two components from the command line.
The changes in this PR make it so that {target} is replaced only with <highlight-title>, while changing the default --dirname-pattern to {profile}/{target} for backward compatibility.

This PR fixes issue #580

You cannot 'fix' a question.
We cannot merge this PR, it introduces backwards incompatible changes.


Feature/no found matches

I wonder, if the problem is only with bulk search, why add "default" empty address on empty result to the repository instead of adding this if to the bulkQuery method ( here https://github.com/ONSdigital/address-index-api/blob/develop/server/app/uk/gov/ons/addressIndex/server/controllers/AddressController.scala#L213 )?
This way we localize the change to the relevant functionality and avoid any regressions in other parts of the application.

@Mironor agree - changing now


⚠ Thumbnails API Endpoint has degraded performance

In d36695f, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1052 ms

Resolved: Thumbnails API Endpoint performance has improved in 129fc26 after 9 minutes.


Pomodoro timer extension

The Pomodoro Technique is a time management method developed by Francesco Cirillo in the late 1980s.
It uses a timer to break work into intervals, typically 25 minutes in length, separated by short breaks.
I'd like to make a chrome extension that manages time using the Pomodoro technique.
I'd like to work on this issue under GSSOC'22, kindly assign it to me.

looks good, @michaelehab you can start working on it :))

closing this since PR is merged :))


Improve the consumability of the data returned by serviceA (and serviceB)

Currently serviceA and serviceB both return a common ServiceData object which is serialized as json.
There are two problems with this;

ServiceA and ServiceB should actually return different data; they are different services
Even when formatted, the resulting nested json object is hard to read and understand

So the two services should return different data and then that data should be presented in a more readable way.
This issue is not to produce a flashy user interface, just to make the existing data readable (probably with some simple HTML).

Yes, return some HTML for service A to better show who is saying what and when.


Rounding errors in vcd simulator time

As a consequence of #535, there's rounding errors in simulation time. I'm guessing this isn't a huge deal if your timescale is small enough and you are just plotting the result but I was trying to analyze simulation output and I was seeing the clock frequency move around a little bit during the simulation which was confusing.
To reproduce
I was working through the example here
sim = Simulator(dut)
sim.add_clock(1e-6) # 1 MHz

result
$comment Generated by nMigen $end
$date 2020-12-12 10:12:39.508725 $end
$timescale 100 ps $end
$scope module top $end
$var wire 1 0 ovf $end
$var wire 16 1 count $end
$var wire 1 2 clk $end
$var wire 1 3 rst $end
$var wire 1 4 en $end
$upscope $end
$enddefinitions $end
#0
$dumpvars
00
b0 1
02
03
04
$end
#5000
12
#10000
02
#15000
12
#20000
02
#24999
12
#29999
02
#34999
12
#40000
02
#45000

Isn't this issue just a duplicate of #535? The rounding errors are the exact problem that issue exists for,

Fair enough.


Error with non-ASCII file names on python2.x

Problem while trying to use with python2.7:
Traceback (most recent call last):
File ".../miniconda2/bin/resync", line 281, in <module>
main()
File ".../miniconda2/bin/resync", line 252, in main
dump=args.resourcedump)
File ".../miniconda2/lib/python2.7/site-packages/resync/client.py", line 637, in write_resource_list
rl.write(basename=outfile)
File ".../miniconda2/lib/python2.7/site-packages/resync/list_base_with_index.py", line 305, in write
s.resources_as_xml(chunk, fh=f)
File ".../miniconda2/lib/python2.7/site-packages/resync/sitemap.py", line 114, in resources_as_xml
tree.write(fh,encoding='UTF-8',xml_declaration=True,method='xml')
File ".../miniconda2/lib/python2.7/xml/etree/ElementTree.py", line 820, in write
serialize(write, self._root, encoding, qnames, namespaces)
File ".../miniconda2/lib/python2.7/xml/etree/ElementTree.py", line 939, in _serialize_xml
_serialize_xml(write, e, encoding, qnames, None)
File ".../miniconda2/lib/python2.7/xml/etree/ElementTree.py", line 939, in _serialize_xml
_serialize_xml(write, e, encoding, qnames, None)
File ".../miniconda2/lib/python2.7/xml/etree/ElementTree.py", line 937, in _serialize_xml
write(_escape_cdata(text, encoding))
File ".../miniconda2/lib/python2.7/xml/etree/ElementTree.py", line 1073, in _escape_cdata
return text.encode(encoding, "xmlcharrefreplace")
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 119: ordinal not in range(128)

With test data can now replicate issue on python 2.7, not a problem on python 3.5:
(py2)simeon@RottenApple resync (non_ascii_uris)> python --version
Python 2.7.12 :: Continuum Analytics, Inc.
(py2)simeon@RottenApple resync (non_ascii_uris)> resync -r x: tests/testdata/odd_file_names
Traceback (most recent call last):
  File "/Users/simeon/miniconda3/envs/py2/bin/resync", line 4, in <module>
    __import__('pkg_resources').run_script('resync==1.0.2', 'resync')
  File "/Users/simeon/miniconda3/envs/py2/lib/python2.7/site-packages/pkg_resources/__init__.py", line 719, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/Users/simeon/miniconda3/envs/py2/lib/python2.7/site-packages/pkg_resources/__init__.py", line 1511, in run_script
    exec(script_code, namespace, namespace)
  File "/Users/simeon/miniconda3/envs/py2/lib/python2.7/site-packages/resync-1.0.2-py2.7.egg/EGG-INFO/scripts/resync", line 280, in <module>

  File "/Users/simeon/miniconda3/envs/py2/lib/python2.7/site-packages/resync-1.0.2-py2.7.egg/EGG-INFO/scripts/resync", line 248, in main

  File "build/bdist.macosx-10.6-x86_64/egg/resync/client.py", line 536, in write_resource_list
  File "build/bdist.macosx-10.6-x86_64/egg/resync/list_base_with_index.py", line 208, in as_xml
  File "build/bdist.macosx-10.6-x86_64/egg/resync/list_base.py", line 133, in as_xml
  File "build/bdist.macosx-10.6-x86_64/egg/resync/sitemap.py", line 127, in resources_as_xml
  File "/Users/simeon/miniconda3/envs/py2/lib/python2.7/xml/etree/ElementTree.py", line 820, in write
    serialize(write, self._root, encoding, qnames, namespaces)
  File "/Users/simeon/miniconda3/envs/py2/lib/python2.7/xml/etree/ElementTree.py", line 939, in _serialize_xml
    _serialize_xml(write, e, encoding, qnames, None)
  File "/Users/simeon/miniconda3/envs/py2/lib/python2.7/xml/etree/ElementTree.py", line 939, in _serialize_xml
    _serialize_xml(write, e, encoding, qnames, None)
  File "/Users/simeon/miniconda3/envs/py2/lib/python2.7/xml/etree/ElementTree.py", line 937, in _serialize_xml
    write(_escape_cdata(text, encoding))
  File "/Users/simeon/miniconda3/envs/py2/lib/python2.7/xml/etree/ElementTree.py", line 1073, in _escape_cdata
    return text.encode(encoding, "xmlcharrefreplace")
UnicodeDecodeError: 'ascii' codec can't decode byte 0xcc in position 6: ordinal not in range(128)
(py2)simeon@RottenApple resync (non_ascii_uris)> py3
discarding /Users/simeon/miniconda3/envs/py2/bin from PATH
prepending /Users/simeon/miniconda3/envs/py3/bin to PATH
(py3)simeon@RottenApple resync (non_ascii_uris)> resync -r x: tests/testdata/odd_file_names | wc -l
      10
(py3)simeon@RottenApple resync (non_ascii_uris)>

Have merged into master 46e154375

Fixed as v1.0.5 and pushed to pypi https://pypi.python.org/pypi?:action=display&name=resync&version=1.0.5


Handle rapid deletion and recreation of subgraphs more gracefully

In development and similar settings, often a subgraph gets deleted, completely removed and then immediately redeployed. This usually fails because of some internal caches that graph-node keeps. This PR removes these failures on indexing nodes. If there are dedicated query nodes, they will still not pick up the deployment until their internal caches get updated (by default in at most 5 minutes) Installations that use a single node, or when the main concern is the index node, will no longer be affected by this cache issue.

This also fixes #4005


added code for creation of the SQLContext object for Spark <= 1.6.1

LGTM - very nice @romeokienzler

thanks a lot @mboehm7

Thank you @romeokienzler. I'll merge.

I thank you @deroneriksson


After migrating from gogs push doesn't work

The upgrade from gogs docs are outdated
The option Rewrite all update hook of repositories  is now called  Resync pre-receive, update and post-receive hooks of all repositories.
Second: all pre-/post/update hook files are now called gitea. So when you run this command it creates new hooks instead of replacing the old Gogs hooks. When somebody pushes to the repo the old Gogs hook results in an error.

User should remove that script manually, auto remove them is dangerous.

Then we should change the docs :)
But what is exactly "dangerous" about it?
If it's dangerous we should warn the user, just like we do when a user removes a repo.

@Bwko we don't know that hook scripts are from Gogs or user customize scripts.

Had a similar issue, removing hooks from Gogs fixed it.
Solution: https://github.com/go-gitea/gitea/issues/3558#issuecomment-372429183


restructure project files

@rajarajanpsj has asked that the contoller, api and main packages be moved out of the pkg sub directory. This will require some changes to the Makefiles.

I think this is a good move.

https://github.com/fidelity/kraan/issues/12

This is builing ok, ready to merge if happy


toml-rs can't read its own output if the root level object is a list

Sending in a list of integers, e.g. toml::Value::from(vec![toml::Value::from(1), toml::Value::from(2)]) results in a serialized output like [1, 2].
But when trying to deserialize that same output, one gets the error Error: expected a right bracket, found a comma at line 1 column 3.
I don't know if the toml file format allows the root node to be a list or not, so I don't know if this is a bug in the serializer or the deserializer, but I think it's pretty clear that any output that toml-rs spits out, it should be able to read back in.
It should also be noted that if the root object is a list of tables, one gets even trippier output, like
[[]]
file = ".git"
modified = "2020-04-15T17:46:22.776979796+02:00"
size = 4096
type = "directory"
user = "liljencrantz"

[[]]
file = "todo"
modified = "2020-04-14T16:19:16.269954425+02:00"
size = 2020
type = "file"
user = "liljencrantz"

Not sure if that's how it's supposed to look.

I believe this is a bug in the serialization here, really only toml tables can be considered a document to parse later, othrewise serialization is just fragments of a document and may not make sense. There's no way to deserialize a top-level array from a toml document.

Maintenance of this crate has moved to the https://github.com/toml-rs/toml repo.  As a heads up, we plan to move toml to be on top of toml_edit, see https://github.com/toml-rs/toml/issues/340.
Closing this out.  If this is still a problem, feel free to recreate this issue in the new repo.


Disables automatic string deduplication

Disables automatic string deduplication as it can break data type evolution.

Codecov Report

Merging #28 into master will increase coverage by 0.16%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   83.70%   83.86%   +0.16%
==========================================
  Files          28       28
  Lines         804      812       +8
  Branches       27       24       -3
==========================================
+ Hits          673      681       +8
  Misses        131      131

Impacted Files
Coverage Δ

...la/io/github/vigoo/desert/GenericBinaryCodec.scala
92.28% <100.00%> (ø)

...src/main/scala/io/github/vigoo/desert/codecs.scala
84.27% <100.00%> (+0.83%)
:arrow_up:

Continue to review full report at Codecov.

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


Integration with Gitlab in addition to GitHub

DotCI works out of the box with GitHub. Are there any plans to integrate with Gitlab as well? What would it take to support that git host in addition to GitHub? This wouldn't have any tie-in with GitlabCI, but rather treat Gitlab only as a repo host, just like GitHub.

hi @tim-kretschmer-c2fo it is not our roadmap atm because we use ghe mainly.
One option is to make scm generic and support adding new backends via plugins.
I am going to keep this issue open, till I research how feasible that is.


Feature: Row Manipulation Functionality

Is your feature request related to a problem? Please describe.
Users currently lack the ability to add, modify, or delete rows within the system/application.
Additional context
Add any other context or screenshots about the feature request here.
This feature enhancement would greatly improve the usability and functionality of the system/application, providing users with greater control over their data management tasks.

#29 implements this feature

Closing as this is fixed.


how to enable will topic

while the mqtt client connect to server, maybe it needs to add will topic for the connect telegram, and how to enable will topic ??

Currently it is not supported.


Need some help with interrupts

Hello everyone,
I'm trying to test some interrupts in a stm32f302 nucleo board, but I'm having some trouble with the Mutex. When I define one, I get this error:
"calls in statics are limited to constant functions, tuple structs and tuple variants rustc (E0015)"
here's the code I'm trying to test:
`
#![no_main]
#![no_std]
#[allow(unused_extern_crates)]
extern crate panic_halt;
extern crate stm32f30x;
use core::cell::RefCell;
use cortex_m_rt::entry;
// use cortex_m::interrupt::{Mutex, self};
use stm32f30x::{interrupt,Interrupt};
static MY_TIM2: cortex_m::interrupt::Mutex<RefCell<Option<stm32f30x::TIM2>>> = cortex_m::interrupt::Mutex::new(RefCell::new(None)); // <----error
static mut TOOGLE: bool = false;

#[entry]
fn main() -> ! {
let p = cortex_m::Peripherals::take().unwrap();
let cp = stm32f30x::Peripherals::take().unwrap();
let rcc = &cp.RCC;
let gpiob = &cp.GPIOB;
let tim2 = &cp.TIM2;
rcc.apb1enr.write(|w| w.tim2en().set_bit()); // tim2 clock
rcc.ahbenr.write(|w| w.iopben().set_bit()); // pb2 clock

gpiob.moder.write(|w| w.moder13().output());
gpiob.odr.write(|w| w.odr13().set_bit());

tim2.psc.write(|w| unsafe {w.psc().bits(23_999)}); // 24k prescaler
tim2.arr.write(|w| unsafe {w.arrl().bits(1000)}); // auto reload value
tim2.dier.write(|w| w.uie().set_bit());
tim2.cr1.write(|w| w.cen().set_bit());

// interrupt::free(|cs| MY_GPIO.borrow(cs).replace(Some(cp.GPIOB)));
cortex_m::interrupt::free(|cs| MY_TIM2.borrow(cs).replace(Some(cp.TIM2)));

let mut nvic = p.NVIC;
nvic.enable(Interrupt::TIM2);

loop {

}

}
#[interrupt]
fn TIM2() {
cortex_m::interrupt::free(|cs| {
let tim2 = MY_TIM2.borrow(cs).borrow();
if tim2.as_ref().unwrap().sr.read().uif().bit_is_set() {
unsafe { TOOGLE = !TOOGLE };
tim2.as_ref().unwrap().sr.write(|w| w.uif().clear_bit());
}
});
}
`
I followed the example from the Rust embedded book in the concurrency section.
thanks in advance

Hi @josephpenafiel
In brief, it should be enough to enable const-fn feature in cortex-m crate:
[dependencies.cortex-m]
version="0.6.0"
features=["const-fn"]

This feature is going to be enabled by default some time in the future: see #153 .  As for the embedded-rust book, the appropriate update is on review at the moment: see PR#196.
Regards,
Sergey

thank you that worked, but now I have a new problem. I want to do something simple. Toggle a led everytime a timer interrupt occurs.
I made a Mutex for the timer, since I need to access it in the interrupt handler to clear the interrupt flag. But since all the peripherals moved into the Mutex, I can't access the GPIO anymore.
The compiler tells me this:
 let gpiob = &cp.GPIOB;

|                 --------- borrow of cp.GPIOB occurs here
...
44 |     cortex_m::interrupt::free(|cs| MY_TIM2.borrow(cs).replace(Some(cp.TIM2)));
|                               ^^^^ move out of cp occurs here    -- move occurs due to use in closure
...
57 |                 gpiob.odr.write(|w| w.odr13().clear_bit());
|                 ----- borrow later used here
what's the correct way of doing what I want?

I would suggest to try the following approach: make both TIM2 and GPIO (or LED pin) global shared resources. Then use them in TIM2 handler under interrupt::free mutex.

when i do that I get a similar error
 --> src/main.rs:47:31    | 27 |     let cp = stm32f30x::Peripherals::take().unwrap();    |         -- move occurs becausecphas typestm32f30x::Peripherals, which does not implement the Copytrait ... 46 |     cortex_m::interrupt::free(|cs| MY_GPIOB.borrow(cs).replace(Some(cp.GPIOB)));    |                               ---- value moved into closure here    -- variable moved due to use in closure 47 |     cortex_m::interrupt::free(|cs| MY_TIM2.borrow(cs).replace(Some(cp.TIM2)));    |                               ^^^^ value used here after move      -- use occurs due to use in closure

Hi @josephpenafiel
Here is an example for BluePill that converts both TIM2 and PC13 into shared resources and uses them in a safe way: safe LED blink from TIM2 irq.
Regards,
Sergey

not helpful


cleanup kdtree impl

Nothing special happened here. Just split of tree.h to tree.h and tree.cpp.

Coverage decreased (-3.0%) to 42.192% when pulling 1ee1cbf37eaa71aaae3a527cec40065171c6b1d4 on dima/cleanup-kdtree-impl into 64c0beed2818d8b45965be6247ea4a358659ca36 on master.

Coverage decreased (-3.0%) to 42.192% when pulling 1cb98da83da4fd60da6eb01ed879b49780f60077 on dima/cleanup-kdtree-impl into 64c0beed2818d8b45965be6247ea4a358659ca36 on master.


little bug for log out account

Users can't log out accounts.  In the second view, there has a Login button in the navigation bar, when I click it, it will jump to login view but soon it will jump back to the second view. The user can't log out the account through click Login button in CreateAnnotationViewController this view.

Good catch! I think I only made the login view. The only real logout is a hard reset. This can be the ticket to make the logout view.


Writing documentation for several languages

Available languages until now:

en_US -> 100%
pt_BR -> 100%

100%


Relation is not working with ObjectId as reference

Collection One User:
{
"_id": ObjectId("52b6bd7ad6861b6101000003"),
"name": "John Doe",
"updated_at": ISODate("2013-12-22T10:22:50.180Z"),
"created_at": ISODate("2013-12-22T10:22:50.180Z")
}
Collection Two Role:
{
"_id": ObjectId("52b6bd7ad6861b6101000006"),
"type": "sword",
"user_id": ObjectId("52b6bd7ad6861b6101000003"),
"updated_at": ISODate("2013-12-22T10:22:50.189Z"),
"created_at": ISODate("2013-12-22T10:22:50.189Z")
}
If user saved as string on Role collection then all relation is working fine. But its not working when reference id is saved as ObjectId. Is there any way to make it work?

Any workaround for this?

For the application where data is produced by Nodejs or any other framework, it stores relationship as ObjectId('related model id'),
In my case I am using database where Related model id are stored as MongoDb\BSON\ObjectID().
NON of Moloquent relationship working

@talukdar here is your solution,
$userFiles = UserFile::whereIn('directory_id',$userDirectories->pluck('directory_id')->map(function($row){
return new \MongoDB\BSON\ObjectID($row);
}))->get();
However you'll not be able to use any Eloquent/Moloquent Relationships in these type of case as described by @Chintan7027


tour times out and fails to load additional examples

I've noticed this a few times already - if you run through step 1 and leave the page open for a while and then try to go to another step, or re-run the same one, the page will hang and require a refresh

This occurs (...or, doesn't) because the WebSocket connection closes. I'll see if I can keep it open.

This even works across (local) server restarts. Interesting.

Closed via https://github.com/lbryio/lbry.tech/commit/e1cf4d960391a74c9f483f5e975c1f92ad25ce49


Removal of starknet js deployment scripts

Hi @barretodavid My name is Cyndie.
So I have been integrating your tutorials on Account Abstraction in the starknetbook, and right  now I'm on the starknet js
chapter.
One of my assignments is to integrate the aa deployment script, which I've been meaning to test it out,
but I have noticed you have deleted the scripts directory.
So does it mean that its not working as you expected, or you'll be updating it?
Thank you for your time, and I learnt a lot on aa following your tutorials.

Aaah never mind found it. Thanks.


Linux (Basic)Physics support

How difficult would it be to bring the ancient OpenSim basicphysics module up to match the current code?  If it's not difficult that could provide at least a stop-gap measure while the much more useful PhysX module is being worked on.
If it's not too complex, for instance doesn't take a complete API rework, just a matter of time and testing just point me in the right direction and I'll give it a go.

FTIW: I've got the code up to the point it attempts to load the physics module and it crashes.  So I'm unable to make further progress without it.

I think I got it working: https://github.com/InWorldz/halcyon/pull/20

It is done.


Swagger Contract Model fields are displayed as requestModel.

The Contract Model fields getting displayed as requestModel.. Why this appears?

Hi, looks like this is an issue for swagger-ui or whatever tooling you use to generate your integration.  Please open your issue there--this repo is just for the specification.


test(*): mock all api endpoints in cypress tests

Why
We had some leaky E2E tests that would hit the actual endpoints with bad results sometimes, and we needed a way to make sure that all requests were mocked, or else the test suite would fail.
How

This required adding dotenv for environment variable management in cypress. this was already used in vite, this just extends the ability so that we can add it to the config.env object during test running.
Next I added this snippet which handles checking for non-mocked routes that match the current VITE_PORTAL_API_URL.
Updated all tests that were failing to mock values, taking care to make sure tests continue to pass.

It appears it's revealed some race conditions as well, so I'll look into this!


Update hdinsight-ubuntu-1804-qa.md

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

#label:"aq-pr-triaged"

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


Please use /MD instead of /MT runtime library

Honestly this isn't a bug per se, but it's problematic enough that from my perspective it might as well be one.
webrtc.lib is compiled with static runtime linking (/MT) instead of dynamic (/MD). The problem with this is that all the .libs I'm importing into my project must match in order for it to successfully link.
Conventionally, /MD is preferred over /MT because /MT is prone to linker conflicts that are difficult to resolve (when multiple imported .libs use the same runtime libraries) and tends to bloat the size of the output executable.
In any case, I've got other libraries I'm using that are compiled with /MD, and I need a build of webrtc.lib that is as well. Thank you!
Evan

webrtc.lib is owned by Google, please ask them. We have no control over what they do. They always use static library for Chromium and WebRTC because as far as I know it avoids another entire class of errors. But I cannot speak for them. I don't think there's a simple way to switch to dynamic CRT.


Suggestion to only show METS differences between CSIP and DIP

The whole CS IP table is unnecessary. It would be clearer to just list the elements and attributes where the DIP differs from CS IP or AIP.

We have tried it both ways before releasing the review version.
We have decided to list all the METS element for the Common Specification Information Package (CSIP) as well as the DIP (Dissemination Information Package) in order to get a coherent specifition for the DIP.
For future versions of the CSIP, SIP, AIP and DIP specifications we will consider to make lists showing all elements and lists showing just the differences.


Add map 3d

I talked yesterday with some people who would like to have the possibility to visualize in 3D. I don't use it personally but I saw that it was possible with mapbox that you already use.
https://docs.mapbox.com/mapbox-gl-js/example/add-terrain/

Thank you for your suggestion, I have re-implemented the website from scratch and added that to the future version.
You can check it out here: https://gpx.studio/gpx.studio/ and provide feedback on this new repository: https://github.com/gpxstudio/gpx.studio.