Trace a specific node

When troubleshooting I want to be able to trace a specific node.
Would it be possible to allow to trace just individual nodes instead of ALL nodes?

@WimVerhaert With current approach, which is not possible. I will take this as feature requirement and implement it.
I believe this ticket relevant to this one #157

I assume this means filtering Resource Logs by sensor. That would be a great feature to add.

https://github.com/mycontroller-org/mycontroller/issues/473 may address part of the issue. we are not going to support fully in 1.x


[feature/swift4] Migrate to swift 4

@serejahh please review

@MortMerr Sorry, I don't work for Yalantis anymore 😃

hey, @MortMerr!
sorry for the delay and thank you for your contribution!


Create file opus-tools-0.1.6.rb

The version 0.1.6 is the last that supports the "--uncoupled" flag for libopus. Works fine with current version of libopus 1.1.2

A number of audit failures (FYI you can check these locally with brew audit --strict --online <formula>):
== /usr/local/Library/Taps/homebrew/homebrew-versions/opus-tools.rb ==
C:  1:  9: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
C:  4: 12: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
C:  5:  7: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
C:  6:  8: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
C:  8:  8: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
C: 15: 14: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
C: 16: 14: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
C: 17: 14: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

1 file inspected, 8 offenses detected
Error: 8 problems in 1 formula

==> audit problems
homebrew/versions/opus-tools:
 * Formula name conflicts with existing core formula.
 * A `test do` test block should be added
 * Stable: SHA1 checksums are deprecated, please use SHA256
 * Formula should have a desc (Description).
 * `require 'formula'` is now unnecessary
 * :automake is deprecated. Usage should be "automake"
 * :libtool is deprecated. Usage should be "libtool"
 * Use `system "make", "install"` instead of `system "make install"`

Closing due to inactivity.


config option to make /explore/users private

The /explore/users page may contain information that is unwanted to be publicly visible or scrapable.
There should be an easy option to remove or at least hide that endpoint.
#1475 touches on this, but was closed due to staleness

I agree this would be very useful in many use cases!

This is a duplicate of #2908


FROST with Secp256k1

Is it possible to use FROST tss with secp256k1 curve? I saw in the code that it requires just a generic curve implementation, but it also requires Challenge Deriver when creating a signer and is labeled under ted25519.

Yes it is. FROST generates a Schnorr signature so the only difference that is required is the challenge deriver or how the c value is computed. Bitcoin taproot requires using SHA256 with certain tag prefixes. Ed25519 uses SHA512. Mina uses Poseidon. If you want to have a basic proof of concept, you could just use SHA256 for the three inputs.

Yes, you can define a challenge deriver using secp256k1 curve abstraction to do this, which should be similar the existing ed25519 challenge deriver.


The extractor does not work in my server but works well in my localhost

Checklist

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

Verbose log
Description
I created a Django project to extract video URL from pornhub.
The extracted video URL I get directly from locahost works well, but after I deploy my codes, the url I get from server does not work. When I open the url, it shows "403 - Forbidden"
Here are codes:
       ydl_opts = {         'format': 'best',         'nooverwrites': True,         'no_warnings': False,         'ignoreerrors': True,     }
with youtube_dl.YoutubeDL(ydl_opts) as ydl:
    url = "https://www.pornhub.com/view_video.php?viewkey={}".format(
        video_id)
    res = ydl.extract_info(url, download=False)
    logging.warning("res", res)
print(res[url])

Example:
Website url: http://<IP_ADDRESS>:8000/tiktok/video/videoURL/ph5c77e0b836386
Result from localhost:
https://dv-h.phncdn.com/hls/videos/201902/28/210403291/,480P_2000K,240P_400K,_210403291.mp4.urlset/index-f1-v1-a1.m3u8?ttl=1587561021&l=0&hash=70e4f679d4da55a5812adca6aa110890
Result from deployed machine:
https://c1v-h.phncdn.com/hls/videos/201902/28/210403291/,480P_2000K,240P_400K,_210403291.mp4.urlset/index-f1-v1-a1.m3u8?5WCQ17D5_pWVJZoQc9oNNsR0OhHEAvpzClTwnnORebXS_6kAEWeQmrNhI1ncVcUwlSfiXkg2fM5iwD7Mb8ROEiiUzf9SJkSGx_ZwAqqMi90QBA6YxgTJrymbi04k-yDyiEb7b4Qall4YwtAyjnlGSCSLnF-38HunXs5isZhFu_GCb71qryJOcxxj7FicSlK1yMElDpRDnKrYWUvD4mzd
Does anyone has ideas about this case? Is this related IP address? Thanks a lot!

@dstftw , Sorry but could you please reopen my case? Actually the video URL can be extracted, but it just can not be opened in browser correctly, so no obvious error log.

Hi Sergey,
Thanks for checking my issue, do you mind helping me understand why the
issue is closed? Actually I really need help on this issue.
Thanks!
On Wed, Apr 22, 2020 at 8:32 PM Sergey M<EMAIL_ADDRESS>wrote:

Closed #24943 https://github.com/ytdl-org/youtube-dl/issues/24943.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ytdl-org/youtube-dl/issues/24943#event-3260365598,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO5DDENKSH5DWYQWIIIQMOTRN3PWXANCNFSM4MOD3ZOA
.


docs(ionic): add guide for setting up with ionic

Checklist

Issue number for this PR: #nnn (required)
#1003
Docs included?: (yes/no; required for all API/functional changes)
Yes
Test units included?: (yes/no; required)
No, N/A
e2e tests included?: (yes/no; required for multi-function/multi-class changes)
No, N/A
In a clean directory, npm install, npm run build, and npm test run successfully? (yes/no; required)
Yes

Description

Fix for #1003. Added section an link to a specific Ionic/Ionic CLI installation and setup.
Code sample
N/A

Ionic's CLI does not generate a environment.ts by default, so users would have to make that themselves.  I can add that though.

thanks a lot for the updated docs ! there is something missing thought .
What if I want to implement firebase as a provider not just inside home.ts .


Provides Alternative to Amazon S3 (w/ minio lib)

First I tried using the current AWS S3 SDK to connect to my minio server (using an example from Minio's docs site)

However I kept seeing it try connect to amazonaws.com. I'm sure I missed something.
If anyone knows any gotchas here, please let me know.
After avoiding it, I went ahead and tried the Minio GO SDK (...looked similar enough to AWS' GO API).
It seems to compile at the moment, tho it just won't give any console output/errors.
I was hoping a more experienced Go master could take a look.
Regarding Ticket: https://github.com/rancher/rancher/issues/2229#issuecomment-259874326
More useful reading
https://docs.minio.io/docs/how-to-run-minio-in-docker
Minio SDK,
Server,
Client.

NB: This is my first OSS contribution for Go!
I know I probably haven't done anything the "Go way."
Again, feedback is welcome!

The build feature and Rancher Compose CLI in general are in the process of being deprecated. As a result, we're planning to shift towards just fixing bugs and not adding any new features.
If this PR was pretty well tested and known to be backwards compatible then I think we'd be open to merging it. It looks like it completely replaces the S3 library though, and that could cause some issues that require testing and then further development.

Thx @joshwget
On the Compose CLI - is there a roadmap or blog post I can read up?
Is the rancher-compose.yml format still viable? 😰
Most of my systems run w/o, however they are only cluster aware with that metadata...

@justsml The file formats will remain the same. It's only the rancher-compose binary that will be deprecated. Instead of running rancher-compose up you'd just run rancher up from the CLI.

Thanks for the update @joshwget


Add ACE interaction items for breaching

I didn't really expect you to be active here ^^ but now that i know you are:
https://ace3mod.com/wiki/framework/interactionMenu-framework.html

I'm not sure what your goal here is. Everything is run through ACE Explosives. Try it out or check the video linked in Steam/BI Forums.

Yeah, i saw that but i think it would be easier if there was a simple Breach item in the interaction menu that would plant the charge, play "lean on wall next to door" animation and boom. This would also incorporate well with my next suggestion

Additional Resources:

https://github.com/acemod/ACE3/blob/master/addons/interaction/functions/fnc_openDoor.sqf
https://github.com/acemod/ACE3/blob/master/addons/interaction/functions/fnc_getDoor.sqf

The ACE Explosives approach allows for more control, and multiple simultaneous breaches using clacker or synced timers. I don't think taking over player movement would be a good idea.

You're probably right c:


Support a validation library instead of custom modelling

Like

https://github.com/alecthomas/voluptuous
https://github.com/keleshev/schema
https://github.com/schematics/schematics

Schematics might be the closest one.
Now I need to duplicate of record definitions.

I have a quick draft for schematics if anyone interested
from sanic_openapi.doc import summary, description, tag
from sanic_openapi.doc import route_specs

from schematics.types import StringType, IntType, DateTimeType, BooleanType, EmailType
from schematics.models import Model

from datetime import date, datetime

def to_native(field):
    if isinstance(field, list):
        return list(map(to_native, field))
    elif isinstance(field, type):
        if issubclass(field, Model):
            return {k: to_native(v) for k, v in field._fields.items()}
    elif isinstance(field, IntType):
        return int
    elif isinstance(field, BooleanType):
        return bool
    elif isinstance(field, DateTimeType):
        return datetime
    elif isinstance(field, StringType):
        return str
    else:
        raise ValueError('ANYAD')

# TODO support more args instead of just the first one
def consumes(*args, content_type=None):
    def inner(func):
        if args:
            route_specs[func].consumes = to_native(args[0])
            route_specs[func].consumes_content_type = content_type
        return func
    return inner

def produces(*args, content_type=None):
    def inner(func):
        if args:
            route_specs[func].produces = to_native(args[0])
            route_specs[func].produces_content_type = content_type
        return func
    return inner

"jsonschema" would also be useful

There's also webargs: https://github.com/sloria/webargs

It seems like Flask-Restful was considering migrating to it (although I think Flask-Restful is inactive now): https://github.com/flask-restful/flask-restful/issues/335
Flask-Restplus, that has Swagger integration and started as a fork of Flask-Restful, was considering webargs too: https://github.com/noirbizarre/flask-restplus/issues/95#issuecomment-160314463 , https://github.com/noirbizarre/flask-restplus/issues/9
Flask-apispec uses webargs for parsing / validation and Marshmallow for serialization / deserialization: https://github.com/jmcarp/flask-apispec
What seems one of the most active Flask-Restplus users / devs added some modifications to support webargs but still just suggested using Flask-apispec because it uses webargs and Marshmallow: https://github.com/frol/flask-restplus-server-example/issues/36#issuecomment-251098426

This would be great to do. Would want to pick one thats most commonly used to start with but could potentially expand out to support more. Any ideas on which one? Maybe add a poll?

My 2 cents of what I've seen:
In comparison to the ones listed above, the closest related one would be Marshmallow which is just a bit more popular. It is a serializer / deserializer with validations.
I know that Webargs is based on the same Marshmallow fields and schemas.
But `Webargs is specifically made to read parameters from requests (from JSON body, query strings, headers, etc) and Marshmallow is what does the work of serializing / deserializing.
I know that Apispec, Marshmallow and Webargs seem to be maintained by the same guys: @jmcarp and @sloria .
Apispec is the one I know that supports OpenAPI / Swagger.
Flask-Restful was considering migrating all the marshalling to Marshmallow but that project seems kind of abandoned now, I think...
It seems like Flask-restplus supports JSON Schema.
The maintainer of Flask-restplus, which seems to be the successor of Flask-Restful, didn't like that Swagger didn't support JSON schema

Asynchronous orms are good candidates too: [aio]peewee, sqlalchemy,
asyncorm.
On Aug 27, 2017 4:40 PM, "Sebastián Ramírez"<EMAIL_ADDRESS>wrote:
My 2 cents of what I've seen:
In comparison to the ones listed above, the closest related one would be
Marshmallow https://github.com/marshmallow-code/marshmallow which is just
a bit more popular. It is a serializer / deserializer with validations.
I know that Webargs is based on the same Marshmallow fields and schemas.
But `Webargs is specifically made to read parameters from requests (from
JSON body, query strings, headers, etc) and Marshmallow is what does the
work of serializing / deserializing.
I know that Apispec, Marshmallow and Webargs seem to be maintained by the
same guys: @jmcarp https://github.com/jmcarp and @sloria
https://github.com/sloria .
Apispec is the one I know that supports OpenAPI / Swagger.
Flask-Restful was considering migrating all the marshalling to Marshmallow
https://github.com/flask-restful/flask-restful/issues/335 but that
project seems kind of abandoned now, I think...
It seems like Flask-restplus supports JSON Schema
http://flask-restplus.readthedocs.io/en/stable/marshalling.html#define-model-using-json-schema
.
The maintainer of Flask-restplus, which seems to be the successor of
Flask-Restful, didn't like that Swagger didn't support JSON schema
https://github.com/noirbizarre/flask-restplus/issues/95#issuecomment-160419446
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/channelcat/sanic-openapi/issues/6#issuecomment-325202489,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6s06CoTvw_ZrP_vIBhLM3PXMM20gimks5scX_mgaJpZM4Mmozu
.

IMO, the best investement is to go directly for Marshmallow. It's the way people is handling (un)marshalling, it has a really stable API, and won't probably go down soon, as opposed to most of the other projects.
Finally, the fact of using sanic doesn't mean you will have an ORM below, and by supporting directly an ORM you are effectively doing vendor lock-in, and leaving out anyone not using an opinionated stack


[XLA] Remove unused dlfcn.h and implement sincos[f] for MSVC

Split from #15310.
#15213

Can one of the admins verify this patch?

Jenkins, test this please.


skoda connect not working for new skoda

hi,
when trying to connect my skoda enyaq with home assistant via skoda connect i am not successful.
ha latest docker version.
error in skoda-connect-integration:
2024-08-18 21:19:00.470 WARNING (SyncWorker_14) [homeassistant.loader] We found a custom integration skodaconnect which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant 2024-08-18 21:19:54.424 ERROR (MainThread) [custom_components.skodaconnect.config_flow] Fetch vehicles failed with error: Invalid token "None" 2024-08-18 21:19:54.424 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved (None) Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 366, in async_configure     result = await self._async_configure(flow_id, user_input)              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 413, in _async_configure     result = await self._async_handle_step(              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 579, in _async_handle_step     self._async_remove_flow_progress(flow.flow_id)   File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 497, in _async_remove_flow_progress     raise UnknownFlow homeassistant.data_entry_flow.UnknownFlow 2024-08-18 21:19:54.678 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved (None) Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 549, in call_configure     await self._async_configure(flow.flow_id)   File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 437, in _async_configure     raise ValueError( ValueError: Show progress can only transition to show progress or show progress done. 2024-08-18 21:20:39.655 ERROR (MainThread) [custom_components.skodaconnect.config_flow] Fetch vehicles failed with error:
i assume it is the same issue as for other 3rd party products (ioboker, evcc). evcc was able to solve it.
the used password is correct.
regards,
andre

Are you able to use the old Skoda Essentials app, or only the new MySkoda app?

This integration currently only supports the old Skoda API compatible with the old Skoda Essentials App.
If you are unable to use the Skoda Essentials account the integration will not work for you. This is the case for new vehicles and/or Skoda accounts.
We are aware the Skoda Essentials app is being deprecated and are working on updating the integration to be compatible with the new API / MySkoda app, but this is a major piece of development work.
Please do not open new issues for this.

no, i can only use the new one. that was the reason why i wasn't able to use evcc as well. after fixing the api in evcc i am able to use it there. so i am sure that the api needs to be fixed. with july 1st 2025 it will be the case for the "old users" as well, because the old api will be switched off.


Describe Release Process with changes

The release process in the Handbook is missing the release candidate page that is central for pointing people to what metrics need reviewing for the release and seeing what the metrics would look like after the release.
This Pull Request details the steps required to release CHAOSS metrics as I understand it from conversations we had.
Please add any steps and details I may have missed.
Please suggest improvements to make it simpler for everyone.
The goal of this PR is to ensure we have a defined process that anyone in the CHAOSS project can follow. There may be details missing on how to execute steps and I'm happy to provide more details.

I think we need to walk through this whole process, make sure everyone understands what the release looks like, and wire-frame the requested pages before making any edits to the handbook.

What is the earliest we can achieve this? We should start the review period and I think we need the Release Candidate page for that. -- We can also push back the release to later in August if we need more time to prepare the RC.

I don't know how soon we can achieve this because I wasn't part of the discussion and I am not certain what the release process looks like with these changes.
When we moved to the continuous release process I was very careful to ask questions about what we wanted the process to look like. Many of the things that are in this new process change were things that were discussed and decided against when we moved to continuous release. Some of them were actually things I wanted to do but the consensus was against (Just saying I am not opposed to the change).
We documented the process we wanted and discussed it in several community meetings.
The process we have now is far from perfect and I am certainly open to change it, but I am a little frustrated by the impulsiveness of this change, that the decision was made right before the regular release, and made in a common WG meeting rather than the community meeting.
I am not opposed to the change but I want to make sure we all understand what this looks like, what work will need to be done, and that we aren't going to be constantly changing our release process in the future.

Option 1: We follow the documented process and see if what we decided in the past is working as we had thought.
Option 2: We discuss the changes on the mailing list and come to a consensus in two weeks at the next community meeting. We push back the release by 2 weeks to Mid-August.
Option 3: We decide to make the changes now and start the Review Period as planned.
I’d be okay with option 2 or 3 above.
I fully understand the frustration with this last-minute.
I also think that it’s not fair to put all the work on you (@klumb), so maybe we can divide and conquer somehow.

Do you want to chat tomorrow? I am free anytime before 10am.
I am leaning towards 3 but I think we need to hold off on documenting the process until we have done it.
Further, I think we should probably reassess the continuous release process - We are introducing inconsistency and redundancy to the overall release process.

I have an opening 8:30-9am or after 3pm tomorrow

Hi @GeorgLink just a note well, this should be done within the handbook documentation where I have already proposed this in my GSoD proposal. So it would be great if this can be left to the aspirant working on the handbook project. What do you think?

Your help is always welcome.

@klumb please have a look. I think the process is cleaner now and reflects what we discussed.

This looks good. I think we should merge it and revisit it after the release.

Thank you :)


Add EntityIndexingMessage::addSkip to add skipped contents more easily

1. Why is this change necessary?
Because of https://github.com/shopware/platform/pull/2082
2. What does this change do, exactly?
It adds addSkip to allow easier changing changing without having to call setSkip/getSkip/array_merge yourself.
3. Describe each step to reproduce the issue or behaviour.
To do https://github.com/shopware/platform/pull/2082
4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-17695
Please use this issue to track the state of your pull request.

@JoshuaBehrens @NiklasLimberg
shouldn't this be changed first in this PR? https://github.com/shopware/platform/blob/trunk/src/Core/Content/Product/DataAbstractionLayer/ProductIndexer.php#L133

@mitelg I don't care the order. I can refactor the other PR later on. I don't care which one I edit but eventually I wasn't sure whether both changes are cool so I decided to separate them

I just wanted to give the hint 😉

Aaah I see. The other is merged, the let me update my code here

that's what I wanted to say 😅👍
sorry 😇

@mitelg there we go

I rebased and amend code to the feedback

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-18662
Please use this issue to track the state of your pull request.

Hi @JoshuaBehrens,
that was my first time using the new importer tool built by @shyim, didn't expect it to create a new ticket but whatever let's use the new ticket to make it easier for the new tool.

I don't care the ticket code :) go for it anyway you like. Or shall I set the code in the changelog file?

You're welcome :) thank you for the merge


[Enhancement] Mallctl of jemalloc support setting the memory of retain pages to not dump

Why I'm doing:
Life cycle of memory in Jemalloc: free -> dirty -> muzzy -> retain -> os.
Currently, in jemalloc, each CPU will have 4 Arenas in per-cpu mode, and the virtual memory of each Arena is unlimited and will not be released if conf retain=true.  (The reason for not returning virtual mem to os is to prevent the creation of a large number of virtual memory holes, which will lead to a decrease in virtual memory allocation performance.)
In theory, the maximum virtual memory limit of BE is the number of cores × 4 × (physical memory), so when execute core dump, the generated core file is relatively large.
This is the current status of the process(starrocks_be) in a user's production environment.

Physical mem of machine is 256G, physical mem of be is 127G, virtual memory of be is 830G, so it will generate on core file of 830G, it's crazy.
Retain  pages takes up 700G of memory here
starrocks_be_jemalloc_active_bytes<PHONE_NUMBER>88
starrocks_be_jemalloc_allocated_bytes<PHONE_NUMBER>44
starrocks_be_jemalloc_mapped_bytes<PHONE_NUMBER>32
starrocks_be_jemalloc_metadata_bytes<PHONE_NUMBER>
starrocks_be_jemalloc_metadata_thp 0
starrocks_be_jemalloc_resident_bytes<PHONE_NUMBER>24
starrocks_be_jemalloc_retained_bytes<PHONE_NUMBER>16 (700G)

So we add an interface to jemalloc and set all retain pages to DONTDUMP using madvise before execute core dump.
This can reduce the file size and prevent core dump from stuck long time to output core file.
Test:
Core: 8
Mem: 64G
Test Set: Tpch-100G
Test case 1: After running all the 22 test case of tcph in a single thread and then trigger core.
Test case 2: Running the sql select count(b.l_orderkey), count(repeat(b.l_shipmode, 10)), count(b.l_suppkey), count(b.l_comment), count(b.l_quantity) from lineitem a join [shuffle] lineitem b on a.l_orderkey=b.l_orderkey; multi times and then trigger core dump.
Before the pr:

Core file size of test case 1: 28G
Core file size of test case 2: 77G

After the pr:

Core file size of test case 1: 5G
Core file size of test case 2: 4G

Usage example

Set retain pages of all arena to nodump:

        std::string str = "arena."
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

btw, how to enable this feature in starrocks?  to run this code in initialization stage?
        std::string str = "arena." + std::to_string(MALLCTL_ARENAS_ALL) + ".dontdump";
        je_mallctl(str.c_str(), NULL, NULL, NULL, 0);

btw, how to enable this feature in starrocks? to run this code in initialization stage?
        std::string str = "arena." + std::to_string(MALLCTL_ARENAS_ALL) + ".dontdump";
        je_mallctl(str.c_str(), NULL, NULL, NULL, 0);

Put it in the core dump handler of BE, i will add in next pr

btw, how to enable this feature in starrocks? to run this code in initialization stage?
        std::string str = "arena." + std::to_string(MALLCTL_ARENAS_ALL) + ".dontdump";
        je_mallctl(str.c_str(), NULL, NULL, NULL, 0);

Put it in the core dump handler of BE, i will add in next pr

static void failure_writer(const char* data, int size) {
    madvise_dontdump();
    dump_trace_info();

    [[maybe_unused]] auto wt = write(STDERR_FILENO, data, size);
}

@Mergifyio backport branch-3.3

https://github.com/Mergifyio backport branch-3.2


Set WeekFirstDay not working version 5.1.0

        var cal = Calendar.current
        cal.firstWeekday = 2 // 2 represents Monday
        cal.timeZone = Zones.gmt.toTimezone()
        cal.locale = Locales.englishUnitedStates.toLocale()

I would like to set the week first day as Monday but the above which worked in swift 4 is not working after migrating to swift 4.2. So, Kindly can you check it once.

I'm not sure what you would to accomplish.
Can you post

Hi @malcommac, also would like to know the answer to this.
How can we set the first day of week when using SwiftDate? since firstDayOfWeek is only a get property.

I need to set the week start date based on user preference. For example: if the user selected Tuesday as the week start date. I want to make sure all computations for the week happen based on that.  I am using dateAtStartOf(.week) and dateAtEndOf(.week) throughout my project.
Currently, i tried setting the firstWeekDay in this way
var calendar = Calendar.autoupdatingCurrent   calendar.firstWeekday = 3   let region = Region(calendar: calendar, zone: userTimezone, locale: Locale.autoupdatingCurrent)   SwiftDate.defaultRegion = region
But it seems to fail, since the default calendar is returned in any instance of Date.
Could you please provide a way to set a week start day? It would be really helpful

I need to set the week start date based on user preference. For example: if the user selected Tuesday as the week start date. I want to make sure all computations for the week happen based on that. I am using dateAtStartOf(.week) and dateAtEndOf(.week) throughout my project.
Currently, i tried setting the firstWeekDay in this way

var calendar = Calendar.autoupdatingCurrent
'calendar.firstWeekday = 3'
'let region = Region(calendar: calendar, zone: userTimezone, locale: Locale.autoupdatingCurrent)'
'SwiftDate.defaultRegion = region`

But it seems to fail, since the default calendar is returned in any instance of Date.
Currently in v6.1.0
@malcommac Could you please provide a way to set a week start day? It would be really helpful

I have the same problem. I really need to let the user change the first weekday, but passing  the configured "Calendar" to "Region" initializer always returns "firstWeekday" based on locale

@Vignesh-Thangamariappan did you manage to solve this? I'm having the same challenge.
Thanks

I have the same problem.
I can't modify firstWeekday and minimumDaysInFirstWeek because of Region's initializer.
There we have:
	public init(calendar: CalendarConvertible = SwiftDate.defaultRegion.calendar,
				zone: ZoneConvertible = SwiftDate.defaultRegion.timeZone,
				locale: LocaleConvertible = SwiftDate.defaultRegion.locale) {
		self.calendar = Calendar.newCalendar(calendar, configure: {
      print($0.firstWeekday, $0.minimumDaysInFirstWeek) // Correct values

			$0.timeZone = zone.toTimezone()
			$0.locale = locale.toLocale()

      print($0.firstWeekday, $0.minimumDaysInFirstWeek) // Incorrect values

      // Manually set the values
      $0.firstWeekday = calendar.toCalendar().firstWeekday
      $0.minimumDaysInFirstWeek = calendar.toCalendar().minimumDaysInFirstWeek

      print($0.firstWeekday, $0.minimumDaysInFirstWeek) // Correct values
		})
	}

It seems to me that after any change to Calendar's properties in the configure closure, all of my settings are reset.
But if I change the values after that, everything works as expected.

Thanks @TimGrell. This looks to be a good fix. Can you please make these changes in a separate branch and push it to the repo. Once they merge your changes, It would be a great addition in the next update of Swift Date.

Thanks @TimGrell. This looks to be a good fix. Can you please make these changes in a separate branch and push it to the repo. Once they merge your changes, It would be a great addition in the next update of Swift Date.

Is there a way to change firstWeekday in current version of SwiftDate? I have the same problem about it.

Is there a way to change firstWeekday in current version of SwiftDate? I have the same problem about it.

me too

Is there a way to change firstWeekday in current version of SwiftDate? I have the same problem about it.

me too

I've figured it out myself by setting Calendars.gregorian or the other to make it change programmatically, hope this help you somehow.

Сan you give an example of how you overcame this bug ?

Сan you give an example of how you overcame this bug ?

var calendar = Calendar.init(identifier: .gregorian)
calendar.firstWeekday = 2
SwiftDate.defaultRegion = Region(calendar: calendar,
zone: TimeZone.autoupdatingCurrent,
locale: Locale.autoupdatingCurrent)
// print SwiftDate.defaultRegion.calendar.firstWeekday then got 1

There used to be a convenient method for that (https://github.com/malcommac/SwiftDate/compare/4.5.1...5.0.0#diff-4ddb0e769834923d6d637cfe9c9974c5d6d2c0e17c1703ac9ec3b7eaae5f68a9L89-L96). Maybe we can bring it back as a fix?

@BagmetDenis @Elshir @suwenclub  You can temporarily use my forked version in which the firstWeekDay works as expected. Here is a link for reference: https://github.com/Vignesh-Thangamariappan/SwiftDate


method to stop a bundle added.

public void stopBundle(String symbolicName)

Thanks for the contribuition @AndreeaDieaconu  . Can you also add a Sling jira issue at https://issues.apache.org/jira/browse/SLING ?


Project Find Results: matched strings are not jumped to correct lines

How to reproduced:

Search something in a project
Click on one result which the file is not opened in Atom
Always jumped to the end of the file

But if the file is already opened, everything works.
Demo:

cc @maxbrunsfeld @as-cii

Duplicate of sindresorhus/atom-editorconfig#51, please subscribe there for updates.


Implement skimming deposit collection mechanism based on wallet max_balance (closes #1653)

closes #1653

Cherry picked https://github.com/rubykube/peatio/pull/1651


Additional tests for "future" ops with compatDetails (stress tests, e2e tests)

Description
Additional test coverage for #16364.  Fixes AB#5316

Add periodic future ops (with compat behavior set to "ignore") to Stress Test to get coverage of them being processed under the various configurations and conditions the stress test covers.
e2e test to round-trip a future op through compression/chunking/grouping
unit test to cover future op going through resubmit (should not actually be resubmitted)

Reviewer Guidance
For the stress tests, I followed the lead of the "large ops" addition, where periodically an additional op is sent on top of the typical counter increment.  I ran a pipeline with these changes and it operated as expected based on the logs.

For reference, I wrote some 1-off tests in TypeScript Playground for the op period calculation logic in this PR


Using the GraphQL API codegens in the backend

** Which Category is your question related to? **
Functions
** What AWS Services are you utilizing? **
AWS Lambda
** Provide additional details e.g. code snippets **
Is there a standard workflow to consume the GraphQL API codegens in a lambda function? If not, what would be a good approach?

Hey @ericclemmons , can you help me on this?

@andrestone You can invoke a GraphQL API from your Lambda function using these steps -  https://aws-amplify.github.io/docs/cli-toolchain/quickstart#graphql-from-lambda and then you can have a build hook ( https://aws-amplify.github.io/docs/cli-toolchain/usage#build-options) to your Lambda function to copy over the generate code from the graphql/ folder to the lambda function folder

Thanks for the comment., @kaustavghosh06.


Soft Reboot doesn't load specific parameter changes

Pixhawk 1 (FMU_v3)
1.11 Stable
When updating PWM_MAIN_MINx and PWM_MAIN_MAXx settings which require a reboot to take effect, they do not get pulled in from a soft reboot (from the tools/reboot vehicle button). Only with a hard power cycle.
Tested this multiple times.
To recreate:
Change PWM_MAIN_MINx to something different (and PWM_MAIN_MAINx) and hit reboot, it doesn't actually institute the change even though the parameter is changed and lists as changed after the reboot.
Photos:
Parameter changed and a soft Reboot (twice for good measure). Look at servo 6 The old minimum for Main 6 was 1400.
Still at 1400 us (when at min):

Minimum specific output value is:

Overall minimum PWM output:

After a hard reboot (pulled the battery):
Goes to 1000 us

That's weird, either way the same portion of init is executed regardless of reboot.
Can you inspect the system a bit more directly through the console after the first reboot?
pwm info
param show PWM*
listener actuator_outputs

Strange. It is working now but I saw this twice when writing today and pretty repeatedly on the test field. These are different aircraft. Testing had to stop to open the hatch for a power cycle 2 times throughout the day. Let me play with this more to recreate and capture it.
I'll keep looking at it or will close.

Here is it working after a soft reboot:

Ok, it's doing it again.
Did one soft reboot and here is what shows up. It didn't pull the min/max or trim back to 1500:

Just did another soft reboot and still the same.

I don't think I'm following what you're doing exactly. Where is channel 6 getting min 1100 us and max 1700 us?
Although without following those details the difference is pretty clearly something with the px4io.

Guess I didn't hit comment... Retyping:
I am setting min and max manually to 1100/1700 just for that output channel. Then hard rebooting and all works. Then when setting those back to default (-1) so 1000 and 2000 min/max and soft rebooting (and after the soft reboot getting the console output above for you), nothing takes and it is holding the numbers from the previous state (hence the 1100/1700). I did another soft reboot on the last test as well after getting the output and it still stayed at 1100/1700. If I then hard reboot everything works as intended.
What is also strange is that this is not all the time. Most of the time but not all the time. Several different systems though have shown the same functionality (ruling out hardware itself). I don't recall seeing this on Pix4 but I couldn't say definitively since that setting was not altered as much during those flights so this is being done on a Pix1.


rubocops/lines: recommend on_os/OS.os? based on context.

Recommend the use of on_macos and on_linux unless we're in def install, def post_install or test do in which case recommend OS.mac? and OS.linux? instead.
TODO:

[x] add autocorrection, tests
[x] fix homebrew/core

Fixes https://github.com/Homebrew/brew/issues/11687

Review period will end on 2021-09-02 at 17:25:26 UTC.

Review period skipped due to critical label.

Two requests here. I think what made the on_os blocks easy to read was that they had to be blocks. If we can, I'd like to enforce that here too, maybe with the exception of ternary operators.
And a second, less important, request: I'd like to avoid unless in core since it's very ruby specific and makes formula harder to understand for the general public.

Some background discussion: #11687

I think what made the on_os blocks easy to read was that they had to be blocks.

It depends on where they're used, in my opinion. They're nice for expressing dependencies (i.e. depends_on), and for fencing operations that need to be done on specific platforms. They're really bad for conditionally defining variables because they lead to patterns like
foo = nil

on_macos do
  foo = "foo"
end

on_linux do
  foo = "bar"
end

or
foo = "foo"
on_linux do
  foo = "bar"
end

This is because of how variable scoping works inside blocks. I guess allowing them for ternary operators or ternary-operator-like expressions would help, but that seems a lot harder to enforce than "allow if statements in install, test, and post_install, and blocks everywhere else.
I had previously contemplated turning on_macos and on_linux into methods that optionally take blocks, and return booleans otherwise as a way to avoid being forced into the patterns above (along with possibly defining aliases on_macos? and on_linux?). However, that doesn't get at what you (@SMillerDev) want here either. I suspect doing this makes them harder to standardise with Rubocop too.
CC @Homebrew/core for thoughts.

My fear in this change is mostly going to be things like the following that IMHO would hurt readability of formulae.
./configure
system "make", "Darwin" if OS.mac?
system "make", "linux" if OS.linux?

Requiring blocks seemed like the best way to avoid that. If people have other ideas I'm sure I can be persuaded though.

The benefit to using a block-style if is that we can get the best of both worlds, right? Meaning, the easier-to-read block syntax without the problems described above. For example:
foo = if OS.mac?
  "foo"
else
  "bar"
end

Did I misunderstand the concern?

Two requests here. I think what made the on_os blocks easy to read was that they had to be blocks. If we can, I'd like to enforce that here too, maybe with the exception of ternary operators.
And a second, less important, request: I'd like to avoid unless in core since it's very ruby specific and makes formula harder to understand for the general public.

I’m fine with both of these as guidance. I’d like to punt on making them required for this PR to be merged, though (because it’s going to delay it further).

My fear in this change is mostly going to be things like the following that IMHO would hurt readability of formulae.
system "./configure"
system "make", "Darwin" if OS.mac?
system "make", "linux" if OS.linux?

Requiring blocks seemed like the best way to avoid that. If people have other ideas I'm sure I can be persuaded though.

I think blocks didn’t look nice here either. Again a future extension for this case would be to require use of an if/else here; we shouldn’t be allowing or assuming a case which isn’t Mac or Linux.

Some tests would be good here

Agreed, WIP locally.

and probably not too difficult to write

Disagreed, WIP locally 😂.

and probably not too difficult to write

Disagreed, WIP locally 😂.

😭 Which part is giving you trouble? I thought it would be a simple rubocop expect_offense/expect_correction kind of thing?

Thanks for all the help here!


FE-1179 Rename info banner action show and url

Why?
Rename info banner action show and url
How?

info_banner_url -> info_banner_action_url`
info_button_show -> info_banner_action_show

Testing
Make sure everything looks OK in the info banner.

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

Changelist by Bito
This pull request implements the following key changes.

Key Change
Files Impacted

Feature Improvement - Rename Info Banner Action Keys

- RemoteBannersDataSource.kt - Updated constant names and references for info banner action URL and show button

- RemoteBannersDataSourceTest.kt - Updated test cases to use new constant names for info banner action URL and show button


These lines don't test what they seem to.

This is testing a new instance of the component....Not necessary.
https://github.com/stewie1570/leaf-validator/blob/1f292b654dbae0c37f6c34ee44ec14bce172512b/src/lib/hooks/useMountedOnlyState.test.tsx#L71-L72
This is testing a whole new instance of the component that use mounted-only-state.

v3.2.1


HERO 4 Get the filename of the last photo

Is it possible in any way to take the filename of the last taken photo in HERO 4?
I'd highly like to avoid fetching the big JSON file from http://<IP_ADDRESS>:8080/gp/gpMediaList

You could parse http://<IP_ADDRESS>:8080/videos/DCIM/[%d]GOPRO/ (HTML cherokee frontend web server site) but I recommend /gp/gpMediaList since its in JSON and easy to parse. You do need to cope with the download time for gpMediaList

Yeah, that's the problem. After each photo taken, I need to store it's filename and take next one after a less than 1sec (cams connected to the odometer).
Thanks anyway for the quick response!
Good job with all this reverse engineered stuff!

If you need take photo with interval less than 1sec it is not possible with the classic mode of this code. You have to use MultiShotMode. Are you working on an MMS if you work with an odometer ?
Personnally, i have modified a bit the source code.
def download_del_MultiShot(self, path=""):
	sequence = []
	ligne = []
	if path == "":
		arr = json.loads(self.listMedia())
		folder = ""
		for i in arr['media']:
			for i2 in i['fs']:
				folder = i['d']
		filename = arr["media"][0]["fs"][len(arr["media"][0]["fs"])-1]["n"]
		lower_bound = arr["media"][0]["fs"][len(arr["media"][0]["fs"])-1]["b"]
		high_bound = arr["media"][0]["fs"][len(arr["media"][0]["fs"])-1]["l"]
		for i in range(int(high_bound) - int(lower_bound)+1):
			f = filename[:4] + str(int(lower_bound) + i) + ".JPG"
			ligne.append(str(f))
			self.downloadMedia(folder, f)
			self.deleteFile(folder,f)
	return ligne

I supposed you try to synchronise data of the odometer and the GoPro ?


Improved area search

I think the code is pretty horrible btw! 🙈

If we decide to implement the full postcode matching feature I mentioned it would ideally be enabled by a config flag to preserve normal functionality for other uses of Covince.

What do you think about whether the top option in the dropdown should be highlighted by default to indicate if you press enter it will be selected?

this is done, didn't add it because it wasn't in the API of the library we're using, but a css hack does just as well. Also reminded me to apply the colour palette as well, thanks!

Could we find a way to not fail for full postcodes? Suggested logic: if we detect a number, we assume it's a postcode. At that point, we remove anything after a space when matching?
The only thing that probably does need to be addressed: searches with spaces seem to fail even if they match local authorities.

I think these two are the same issue, just an artifact of the tokenisation process and could be handled by e.g. matching against the longest string. Agree that we don't want anything specific to postcodes. We could make an issue to switch to a proper fuzzy matcher as an enhancement, just need to be able to represent it on the UI as well.

Looking great to me!

Everything is looking fantastic!!


Migrate firewall startup/shutdown job to bicep based deployment.

Type of PR

[x] Documentation changes
[x] Code changes
[ ] Test changes
[ ] CI-CD changes
[ ] GitHub Template changes

PR Checklist

[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My code follows the code style of this project.
[x] I ran lint checks locally prior to submission.
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Purpose of PR

Does this introduce a breaking change

[ ] YES
[x] NO

Validation

[ ] Unit tests updated and ran successfully
[ ] Update documentation or issue referenced above

Issues Closed or Referenced

Closes: https://github.com/retaildevcrews/wcnp/issues/1159
References: https://github.com/retaildevcrews/wcnp/issues/1151

can we also as part of this PR remove deprecated powershell scripts for clusters, gateway, and firewalls?

can we also as part of this PR remove deprecated powershell scripts for clusters, gateway, and firewalls?

this will be done in later PRs

Manually deployed the bicep firewall deployment file from this PR
Manually disabled the current powershell automation firewall runbooks under RG "rb-wcnp-firewall-automation-dev"


DA header optimizations

Originally, specs aimed to optimize DA header size, but it turned out that this can be done on the implementation side without sacrificing simplicity in the calculation of the Merkle root, as described here.

@Wondertan @nashqueue can we update this issue?

I completely forgot the context behind this issue.

I think the optimization here is that you don't have to gossip tail padding rows .

I think the optimization here is that you don't have to gossip tail padding rows .

yeah exactly. probably not a huge deal, but could add up for larger squares

closing because we have an issue in app for this, and all the logic lives there now https://github.com/celestiaorg/celestia-app/issues/826


Retry when sock.recv() raises EINTR

I'm surprised by test for 767c47e was succeeded.
It fails on my Mac.

Is the test working correctly on your Mac now? If so, I can merge this.

Is the test working correctly on your Mac now? If so, I can merge this.

Yes.

Thanks!


Fix remaining annotations

Motivation
We have recently enforced local control of method signature for the automated setting of source schemas for ecephys interfaces - this primarily extends it to the ophys interfaces as well removes a few random references to unspecified PathTypes that were previously missed.
All references to PathType across the repo should now be properly specified as FilePathType, OptionalFilePathType, or FolderPathType and import these directly from the utils rather than redefining them every time.
How to test the behavior?
No changes to actual tests, but they are quite useful in checking that nothing gets broken with the update.
pytest

Checklist

[X] Have you checked our Contributing document?
[X] Have you ensured the PR description clearly describes the problem and solutions?
[X] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?

Cool, I'll merge once the full tests finish making sure nothing is broken.


feat: initialize azure-net repository content

Ref: https://github.com/jenkins-infra/helpdesk/issues/2924

I've started a Scan on infra.ci.

All good, ready to 🚀


Run DNSTwist from P&E reports module (POC)

🐛 Summary
Update the dnstwist scan to run P&E's scripts in the pe-reports repository.
Expected behavior
The scan will run on all of P&E's stakeholders and results will be stored in the P&E database.

@aloftus23 , @Matthew-Grayson  is this still a thing?  I ask because the long-term (18 mths+) product roadmap had the entire reporting interface being replaced by the existing back-office CyHy reporting API eventually.
I'm not certain what DNSTwist is, but I would like to review this before this proceeds any further.
cc @schmelz21


Update Fission logo on readme

Summary
Problem
Logo on the readme has been refreshed
Solution
Replace with one of the selection in https://drive.google.com/drive/u/1/folders/1GoDU4-hlMBlEjGV4LME1syk2EXGgc-Gm
Detail
Describe the bug
A clear and concise description of what the bug is.

@chadkoh we should just move to the homestar logo right?


Can't exclude composite build tasks

Expected Behavior
We should be able to exclude a composite build task, as stated in the docs
Current Behavior
Using gradle's sample project (without touching anything):
❯ ./gradlew :server-application:build -x :server-application:test

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':domain-model:release'.
> A problem occurred configuring project ':server-application:app'.
   > This build's work graph is currently in use by another thread.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 5s

Context

In my own project I can't do this either, either because of the above error or because of:
> Expected vintage state of build 'spring-app' to be in state SettingsLoaded or later but is in state SettingsLoaded and transitioning to Configured.

I stopped all gradle daemons but it didn't help.
Steps to Reproduce

Download gradle's sample project: https://docs.gradle.org/current/samples/zips/sample_structuring_software_projects-kotlin-dsl.zip
And run ./gradlew :server-application:build -x :server-application:test
Your Environment

Gradle: 7.5.1
Kotlin: 1.6.21
OS: MacOS Monterey 12.5 (M1 Pro)

Fixed via https://github.com/gradle/gradle/pull/21835


add stage group finish replay node

I'm still wondering about this, should we add some of the other stage stuff? Or maybe a "define title screen" node? since thats pretty much possible if we just overload the stage_init of the game

added with last commit


Disable watchman on Travis

Green tests are better than red tests

This fixes Travis tests, going to pull in.


No matching version found for<EMAIL_ADDRESS>

npm version 5.6.0
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.
npm ERR! notarget
npm ERR! notarget It was specified as a dependency of 'error-stack-parser'

me too

me too

This is likely a CDN issue. Please use v1.3.4 if you encounter StackFrame is not a constructor


CORS check in React Native

https://github.com/GetStream/stream-js/blob/96e2187f1cd9eaca265b1e481aaf048ba2e12fc9/dist/js/getstream.js#L1022
This fails in RN, though it does support CORS. Here's an example of another project with the same issue.
Is it possible to special-case this check? Happy to submit the (small) PR!

makes sense to have a check for RN. PR is very welcome!

I've chased it down the stack: https://github.com/matthisk/xmlhttp-request/issues/2 . Happy to close or block this if you want to wait for that to land?

I am trying to get this working in react-native. I am running the sample code but am not seeing any new data in my dashboard. Also, no errors are being thrown in the app.
Is this related?

Nevermind, it looks like the promise was throwing an error:
{"error":{"code":null,"detail":"url signature missing or invalid","duration":"4ms","exception":"AuthenticationFailed","status_code":403},"response":{"UNSENT":0,"OPENED":1,"HEADERS_RECEIVED":2,"LOADING":3,"DONE":4,"readyState":4,"status":403,"timeout":0,"upload":{},"_aborted":false,"_hasError":false,"_method":"POST","_response":"{\"code\": null, \"detail\": \"url signature missing or invalid\", \"duration\": \"4ms\", \"exception\": \"AuthenticationFailed\", \"status_code\": 403}","_url":"https://api.getstream.io/api/v1.0/feed/timeline/jack/following/?api_key=8vr8tcg5p2mq&location=unspecified","_timedOut":false,"_trackingName":"unknown","_incrementalEvents":false,"responseHeaders":{"X-Amz-Cf-Id":"RyjHGTWZH62sWZziy0kv5J2kjFeDPTd8an7Q7OArgvIj6R0z1Q1GSQ==","Via":"1.1 173748bfac8713c88182cd1b8118e99a.cloudfront.net (CloudFront)","X-Cache":"Error from cloudfront","Vary":"Accept","Server":"nginx","Date":"Thu, 16 Mar 2017 05:10:01 GMT","Allow":"GET, POST, HEAD, OPTIONS","Connection":"keep-alive","Content-Type":"application/json"},"_requestId":null,"_headers":{"stream-auth-type":"simple","authorization":"timelinejack r7EMMqII7Tf8nd8JROY1PG_JIQk","x-stream-client":"stream-javascript-client-browser-unknown","accept":"application/json","content-type":"application/json"},"_responseType":"","_sent":true,"_lowerCaseResponseHeaders":{"x-amz-cf-id":"RyjHGTWZH62sWZziy0kv5J2kjFeDPTd8an7Q7OArgvIj6R0z1Q1GSQ==","via":"1.1 173748bfac8713c88182cd1b8118e99a.cloudfront.net (CloudFront)","x-cache":"Error from cloudfront","vary":"Accept","server":"nginx","date":"Thu, 16 Mar 2017 05:10:01 GMT","allow":"GET, POST, HEAD, OPTIONS","connection":"keep-alive","content-type":"application/json"},"_subscriptions":[],"seq_id":2,"id":"2: POST https://api.getstream.io/api/v1.0/feed/timeline/jack/following/?api_key=8vr8tcg5p2mq&location=unspecified","_id":"2: POST https://api.getstream.io/api/v1.0/feed/timeline/jack/following/?api_key=8vr8tcg5p2mq&location=unspecified","timeoutTimer":4,"statusCode":403,"responseURL":"https://api.getstream.io/api/v1.0/feed/timeline/jack/following/?api_key=8vr8tcg5p2mq&location=unspecified","body":{"code":null,"detail":"url signature missing or invalid","duration":"4ms","exception":"AuthenticationFailed","status_code":403}},"line":154028,"column":32,"sourceURL":"http://packager.s7-p23.chapinkapa.exponent-app.exp.direct/main.bundle?platform=android&dev=true&hot=false&minify=false"}

This looks to be the same error as this post:
http://stackoverflow.com/questions/34878589/url-signature-missing-or-invalid-getstream
But I am using a token on the client-side...

@tbarbugli This is bug in xmlhttp-request dependency. May be use another dependency?

FYI matthisk/xmlhttp-request#2 was closed 16 days ago


UCP 7220: Print panic info into tikv_stderr.log

Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: close #7220
Problem Summary:
What is changed and how it works?
What's Changed:
redirect panic log to a file, which default is tikv_stderr.log and can be setted in config panic-log-file
Related changes
Check List
Tests

No code

Side effects
I think it seems no side effect
Release note

@skyitachi please take a look again if you are free.

 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<EMAIL_ADDRESS>seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


How to use in a storyboard

How can I get this to show up in a storyboard? I attempted to mark the RangeSlider class as IBDesignable and assign it to a UIView however at runtime the control doesn't render.

@blkbam did you get this working? I would also like to use it on storyboard but have not been able to do so. I guess something needs to be implemented in here:
required init?(coder: NSCoder) {
    super.init(coder: coder)
}

to actually set it up and make it render

@digitalturtle I abandoned it for https://github.com/TomThorpe/TTRangeSlider

@blkbam @digitalturtle Hi guys,
Thanks for having given RangeSlider a try.
I just made it IBDesignable, so it shows up in storyboards and is rendered properly at runtime.


[report-converter] Support ESLint

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity increasing per file
==============================
- tools/report-converter/tests/unit/eslint_output_test_files/files/index.js  1
- tools/report-converter/codechecker_report_converter/eslint/analyzer_result.py  2
- tools/report-converter/tests/unit/test_eslint_parser.py  3
- tools/report-converter/codechecker_report_converter/eslint/output_parser.py  8

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity increasing per file
==============================
- tools/report-converter/tests/unit/test_eslint_parser.py  3
- tools/report-converter/codechecker_report_converter/eslint/output_parser.py  8
- tools/report-converter/tests/unit/eslint_output_test_files/files/index.js  1
- tools/report-converter/codechecker_report_converter/eslint/analyzer_result.py  2

See the complete overview on Codacy


Cannot have custom message for length?

I'm trying to add a custom error message for the field length, but for some reason govalidator is not recognising it and always replacing it with the default message.
The usage:
Country string json:"country" valid:"required~Country code is required,len(2|2)~Country code must be two-letter code"`

The result:

Am I doing something wrong?

Hi @tutabeier
You have an error. There is not len validator. You can use length or runelength.
Or even better to use ISO3166Alpha2 to validate country code.

Hello guys!
I forked this package cause owner disappeared. Hope, he will be back, but it would be easier to merge these changes back if he is back
Link to my repo: create issue there and we'll discuss it.


feat: Assert presence of acknowledgments

Description, Motivation, and Context
A baby step toward channel packet acknowledgments. We only assert that acks exist.
Known Limitations, Trade-offs, Tech Debt

I don't think we can assert a specific ack count given the parallel nature of the test suite. It could be racy.
I explored replacing WaitForBlocks with WaitForAcks but got an order of operations error: failed to start dest chain: rpc error: code = Unavailable desc = connection closed before server preface received which leads me to think the node is not ready yet. Therefore, I punted on that refactor (for a future PR). The cosmos chain sometimes makes cli commands and sometimes makes rpc/grpc calls.

FYI, searching a transaction events yielded no results. So I need to educate myself more on the relationship between a tx and packets. I know relayers must search the events to find the ack data.
The example does NOT contain any ack events that would help us to assert the ack data.
(types.Event) type:"coin_spent" attributes:<key:"spender" value:"cosmos137l3z0jzucfdt0xctvjeg3k2us0ty7jjyafesh" index:true > attributes:<key:"amount" value:"2000uatom" index:true > ,
 (types.Event) type:"coin_received" attributes:<key:"receiver" value:"cosmos17xpfvakm2amg962yls6f84z3kell8c5lserqta" index:true > attributes:<key:"amount" value:"2000uatom" index:true > ,
 (types.Event) type:"transfer" attributes:<key:"recipient" value:"cosmos17xpfvakm2amg962yls6f84z3kell8c5lserqta" index:true > attributes:<key:"sender" value:"cosmos137l3z0jzucfdt0xctvjeg3k2us0ty7jjyafesh" index:true > attributes:<key:"amount" value:"2000uatom" index:true > ,
 (types.Event) type:"message" attributes:<key:"sender" value:"cosmos137l3z0jzucfdt0xctvjeg3k2us0ty7jjyafesh" index:true > ,
 (types.Event) type:"tx" attributes:<key:"fee" value:"2000uatom" index:true > ,
 (types.Event) type:"tx" attributes:<key:"acc_seq" value:"cosmos137l3z0jzucfdt0xctvjeg3k2us0ty7jjyafesh/0" index:true > ,
 (types.Event) type:"tx" attributes:<key:"signature" value:"k5AkKBh9hyu4b1RDi8SFphDQUa8+M/+BhscBzdgnx+558bEMYJkQ+4V967azCwlvId+5q2tnyoks9ZhUBajlkw==" index:true > ,
 (types.Event) type:"message" attributes:<key:"action" value:"/ibc.applications.transfer.v1.MsgTransfer" index:true > ,
 (types.Event) type:"coin_spent" attributes:<key:"spender" value:"cosmos137l3z0jzucfdt0xctvjeg3k2us0ty7jjyafesh" index:true > attributes:<key:"amount" value:"1000000uatom" index:true > ,
 (types.Event) type:"coin_received" attributes:<key:"receiver" value:"cosmos1a53udazy8ayufvy0s434pfwjcedzqv34kvz9tw" index:true > attributes:<key:"amount" value:"1000000uatom" index:true > ,
 (types.Event) type:"transfer" attributes:<key:"recipient" value:"cosmos1a53udazy8ayufvy0s434pfwjcedzqv34kvz9tw" index:true > attributes:<key:"sender" value:"cosmos137l3z0jzucfdt0xctvjeg3k2us0ty7jjyafesh" index:true > attributes:<key:"amount" value:"1000000uatom" index:true > ,
 (types.Event) type:"message" attributes:<key:"sender" value:"cosmos137l3z0jzucfdt0xctvjeg3k2us0ty7jjyafesh" index:true > ,
 (types.Event) type:"send_packet" attributes:<key:"packet_data" value:"{\"amount\":\"1000000\",\"denom\":\"uatom\",\"receiver\":\"osmo137l3z0jzucfdt0xctvjeg3k2us0ty7jjvx6fx9\",\"sender\":\"cosmos137l3z0jzucfdt0xctvjeg3k2us0ty7jjyafesh\"}" index:true > attributes:<key:"packet_data_hex" value:"7b22616d6f756e74223a2231303030303030222c2264656e6f6d223a227561746f6d222c227265636569766572223a226f736d6f3133376c337a306a7a756366647430786374766a6567336b327573307479376a6a767836667839222c2273656e646572223a22636f736d6f733133376c337a306a7a756366647430786374766a6567336b327573307479376a6a796166657368227d" index:true > attributes:<key:"packet_timeout_height" value:"1001-1041" index:true > attributes:<key:"packet_timeout_timestamp" value:"1651793306109575301" index:true > attributes:<key:"packet_sequence" value:"1" index:true > attributes:<key:"packet_src_port" value:"transfer" index:true > attributes:<key:"packet_src_channel" value:"channel-0" index:true > attributes:<key:"packet_dst_port" value:"transfer" index:true > attributes:<key:"packet_dst_channel" value:"channel-0" index:true > attributes:<key:"packet_channel_ordering" value:"ORDER_UNORDERED" index:true > attributes:<key:"packet_connection" value:"connection-0" index:true > ,
 (types.Event) type:"message" attributes:<key:"module" value:"ibc_channel" index:true > ,
 (types.Event) type:"ibc_transfer" attributes:<key:"sender" value:"cosmos137l3z0jzucfdt0xctvjeg3k2us0ty7jjyafesh" index:true > attributes:<key:"receiver" value:"osmo137l3z0jzucfdt0xctvjeg3k2us0ty7jjvx6fx9" index:true > ,
 (types.Event) type:"message" attributes:<key:"module" value:"transfer" index:true >

Looking into the CI test failures which all have to do with penumbra.
penumbra_chain_test.go:29:
[260](https://github.com/strangelove-ventures/ibc-test-framework/runs/6325297694?check_suite_focus=true#step:4:260)
        	Error Trace:	penumbra_chain_test.go:29
[261](https://github.com/strangelove-ventures/ibc-test-framework/runs/6325297694?check_suite_focus=true#step:4:261)
        	Error:      	Received unexpected error:
[262](https://github.com/strangelove-ventures/ibc-test-framework/runs/6325297694?check_suite_focus=true#step:4:262)
        	            	error generating wallet on penumbra node: container returned non-zero error code: 1
[263](https://github.com/strangelove-ventures/ibc-test-framework/runs/6325297694?check_suite_focus=true#step:4:263)
        	Test:       	TestPenumbraChainStart
[264](https://github.com/strangelove-ventures/ibc-test-framework/runs/6325297694?check_suite_focus=true#step:4:264)
        	Messages:   	failed to start penumbra chain

{pd-1-penumbra-1-TestPenumbraChainStart-CreateKey-oef} - stderr:
227
Error: transport error
228

229
Caused by:
230
    0: error trying to connect: tcp connect error: Connection refused (os error 111)
231
    1: tcp connect error: Connection refused (os error 111)
232
    2: Connection refused (os error 111)

Got some great feedback from @agouin in DMs.
Going to close this PR and submit another. TLDR; I probably have to look at txs. The GRPC to get acks from 04-channel module does not give me back enough information. The sequence is not enough.


Cannot access to VSO extensions after joining partner program

I've already join to vspp but I don't have link to extensions in control panel

Fix acer computer start up

I have to ask microsoft workers to enable early adoption program for you vso account.


run tesseract with multiple languages at one time C#

How does tesseract work with multiple languages text?
I installed Tesseract 4.1.1 by Charles weld, from NuGet package manager, but i can run the engine over one language file
Here is my code:
var img = new Bitmap(Open_Image_File.FileName);      var ocr = new TesseractEngine("./tessdata", "eng", EngineMode.LstmOnly);                    var page = ocr.Process(img);               txtres.Text = page.GetText();
I am wondering if someone could assist to use two ore three languages at the same time, for example (English and Arabic) together?

I’ve never used multiple languages, but the source code for the C++ tesseract project states the following syntax.  Contact tesseract-ocr on Google Groups for more information.
// Parse a string of the form [~][+[~]]*.
// Langs with no prefix get appended to to_load, provided they
// are not in there already.
// Langs with ~ prefix get appended to not_to_load, provided they are not in
// there already.

I’ve never used multiple languages, but the source code for the C++ tesseract project states the following syntax. Contact tesseract-ocr on Google Groups for more information. // Parse a string of the form [~][+[~]]*. // Langs with no prefix get appended to to_load, provided they // are not in there already. // Langs with ~ prefix get appended to not_to_load, provided they are not in // there already.

Im also trying to do so and this doesnt seem to work for me. Im creating a engine like this:
new TesseractEngine("someFolder", "rus+eng", EngineMode.LstmOnly);

This results in only russian characters being read.
Using "eng+rus" results in only english characters being read.

I’ve never used multiple languages, but the source code for the C++ tesseract project states the following syntax. Contact tesseract-ocr on Google Groups for more information. // Parse a string of the form [~][+[~]]*. // Langs with no prefix get appended to to_load, provided they // are not in there already. // Langs with ~ prefix get appended to not_to_load, provided they are not in // there already.

Im also trying to do so and this doesnt seem to work for me. Im creating a engine like this:
new TesseractEngine("someFolder", "rus+eng", EngineMode.LstmOnly);

This results in only russian characters being read. Using "eng+rus" results in only english characters being read.

For me the issue was that I was using models from tesdata_fast. Using models from tesdata_best solved the issue.


Support more keystore types - PEM

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I am part of the SAP BTP's Unified customer landscape integrations team. In short our service allows customer's to automate their connectivity between a BTP account and their SuccessFactors/S4Hana tenant. For some authentications however we still do not have automatisation but we have provided customers with a documentation of how they can setup the connectivity manually, e.g. https://help.sap.com/docs/btp/sap-business-technology-platform/using-mutual-transport-layer-security-mtls?version=Cloud#loioe90cafcd92a54d1cba7d7fa049f674fa. In this page we describe mTLS authentication to SuccessFactors and we have documented it with PEM type keystore from Destination service.
A client of ours followed that but then tried to consume the destination through a JavaScript application developed with Cloud SDK and got the error "The format of the provided certificate 'SF_Integration.pem' is not supported. Supported formats are: p12, pfx."
I was able to find this in your implementation at https://github.com/SAP/cloud-sdk-js/blob/4cd9019aa366d19ba6617910828efcacb40f90de/packages/connectivity/src/http-agent/http-agent.ts#L205. Is it possible to allow further keystore types like PEM and JKS or if not the atleast provide a better error message for PEM type similar to how you have done this for JKS keystores - https://github.com/SAP/cloud-sdk-js/blob/4cd9019aa366d19ba6617910828efcacb40f90de/packages/connectivity/src/http-agent/http-agent.ts#L235
Describe the solution you'd like
Support for PEM type keystore
Describe alternatives you've considered
Clear error message with conversion instructions like with JKS
Impact / Priority

We will take steps to update our documentation with instructions for P12, as well as the existing ones for PEM, but it would be great for customers if they could use PEM directly, because even if they use P12 they will still need to extract the certificate from it in PEM format to then be able to allowlist it in their SuccessFactors tenant.
Additional context
Add any other context or screenshots about the feature request here.

Hi @DobrinTs , we've discussed this and put it into our backlog. We're planning to work on it soon.

Hey @DobrinTs, this feature has been implemented here and will be released with the next version of SAP Cloud SDK for JS. Feel free to try out the latest canary version until then.


Update NodeClockNotSynchronising.md

Want to add the RedHat OCP Bug#1822211 solution
Taint a node with NoExecute effect causes oc debug node to fail in OCP 4

Taint a node with NoExecute  effect causes oc debug node to fail in OCP 4
Bug#1822211


Fail to use libvips/build-win64-mxe for custom libvips

Question about an existing feature
What are you trying to achieve?
I want to use a custom libvips with *magick on Windows 11. I try to overide the default libvips refer to this comment #2377. But it can not work.

An initial project install sharp with version 0.33.5.
Download vips-dev-w64-all-8.15.3 from build-win64-mxe.
Put the files from folder bin into node_modules\@img\sharp-win32-x64\lib or node_modules\sharp\build\Release.
Test reading image metadata.
Failed with messages :

D:\MyData\Studio\Development\test\test-sharp\node_modules\sharp\lib\sharp.js:113
  throw new Error(help.join('\n'));
  ^

Error: Could not load the "sharp" module using the win32-x64 runtime
ERR_DLOPEN_FAILED: The specified module could not be found.
<EMAIL_ADDRESS>Possible solutions:
- Ensure optional dependencies can be installed:
    npm install --include=optional sharp
- Ensure your package manager supports multi-platform installation:
    See https://sharp.pixelplumbing.com/install#cross-platform
- Add platform-specific dependencies:
    npm install --os=win32 --cpu=x64 sharp
- Consult the installation documentation:
    See https://sharp.pixelplumbing.com/install
    at Object.<anonymous> (D:\MyData\Studio\Development\test\test-sharp\node_modules\sharp\lib\sharp.js:113:9)
    at Module._compile (node:internal/modules/cjs/loader:1358:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)
    at Module.load (node:internal/modules/cjs/loader:1208:32)
    at Module._load (node:internal/modules/cjs/loader:1024:12)
    at Module.require (node:internal/modules/cjs/loader:1233:19)
    at require (node:internal/modules/helpers:179:18)
    at Object.<anonymous> (D:\MyData\Studio\Development\test\test-sharp\node_modules\sharp\lib\constructor.js:10:1)
    at Module._compile (node:internal/modules/cjs/loader:1358:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)

Node.js v20.13.1

When you searched for similar issues, what did you find that might be related?
https://github.com/lovell/sharp/issues/2377#issuecomment-696686581

Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this question

    let imgPath = path.join(process.cwd(), "file", "3.jpg");
    let origImg = sharp(imgPath);
    console.log(await origImg.metadata());

Please provide sample image(s) that help explain this question

https://sharp.pixelplumbing.com/install#custom-libvips

The use of a globally-installed libvips is unsupported on Windows

Sorry, I cannot help with any questions relating to the use of custom libvips on Windows - it might work, but you're on your own - good luck!

https://sharp.pixelplumbing.com/install#custom-libvips

The use of a globally-installed libvips is unsupported on Windows

Sorry, I cannot help with any questions relating to the use of custom libvips on Windows - it might work, but you're on your own - good luck!

OK, thank you all the same.


train model with gpu but gpu not busy

when i train model use gpu on my own datasets, the training step is very slow, and use command 'nvidia-smi' see the gpu information ,it's show gpu not be used？who can help me,thank you

os check the os.environ["Visible Devices"]..

when i train model use gpu on my own datasets, the training step is very slow, and use command 'nvidia-smi' see the gpu information ,it's show gpu not be used？who can help me,thank you

Hello! Have you solved this issue yet? I met the same issue : i have put the CUDA_VISIBLE_DEVICES=0 before python train_pixel_link.py in scripts/train.sh . But i found that this program was running in CPU with 500%CPU, and the gpu was not busy.

when i train model use gpu on my own datasets, the training step is very slow, and use command 'nvidia-smi' see the gpu information ,it's show gpu not be used？who can help me,thank you

if you have sloved this problem, could you give me some tips about this issue? thanks!

@yjfncu

Had you solved this problem? My CPU boost up 5600% @@!. That's crazy

when i train model use gpu on my own datasets, the training step is very slow, and use command 'nvidia-smi' see the gpu information ,it's show gpu not be used？who can help me,thank you

if you have sloved this problem, could you give me some tips about this issue? thanks!

hello,  Had you solved this problem?

when i train model use gpu on my own datasets, the training step is very slow, and use command 'nvidia-smi' see the gpu information ,it's show gpu not be used？who can help me,thank you

if you have sloved this problem, could you give me some tips about this issue? thanks!

hello, Had you solved this problem?

In my case, I remember that i check the LD_LIBRARY_PATH and i also export the PYTHONPATH again.


Please provide a cargo install option to not write .crates.toml

When installing a crate as part of a package build process (with --root pointing to a directory that will get packaged as /usr), I'd like to avoid writing the .crates.toml file (which would become /usr/.crates.toml).  Right now, I run cargo install and then remove that file afterward.  Please consider providing an option to not write .crates.toml at all.

Sounds like a good option to me!

Yes 👍 on this as well. For Gentoo I have to remove .crates.toml after the fact as well. (ref for me gentoo/gentoo#2918)

@alexcrichton
I was giving this a try since it looked relatively straight forward. Just a few quick questions.
I tried putting this block in an if statement after adding a --no-metadata flag. However, it looks like the file is implicitly created elsewhere when trying to acquire a file lock. Perhaps just deleting the file at the very end of the install() function would work best?
While I'm at it, does this look fine for the updated --help?
...
    -q, --quiet               Less output printed to stdout
    --color WHEN              Coloring: auto, always, never
    --frozen                  Require Cargo.lock and cache are up to date
    --locked                  Require Cargo.lock is up to date
    --no-metadata             Do not write the .crates.toml file in the root

That's what I wrote before discovering the implicit creation. Should I specify it could delete an existing .crates.toml or should we check for the existence of the file before acquiring a lock and then only deleting the file if we created it?

@JustAPerson oh I think it's also ok to avoid the lock if --no-metadata is passed (avoiding the file being created at all) because in theory that's covered by whomever's calling Cargo at that point. Also yeah help looks good to me!

The --no-track flag was added as part of the install-upgrade feature which implements this request. Closing this in favor of the tracking issue at #6797.


Bare bones widget inspector support.

Sent out for review without tests in expectation of significant changes based on code review feedback.
Toggle the widget inspector from the flutter tool by pressing 'i'.
When the widget inspector is select mode:
Pointer down to to inspect a widget.
Pointer click to finalize selection of a widget. You can now interact
with the application as you normally would but with the inspected widget
highlighted.
Click the inspect icon in bottom left corner of screen to reactivate
select mode.

That error from the CLA bot typically means the email address you set up for git (and got used in the git commit) is not associated with your github account.

looks like you just want to set git config --global user.email

The general approach seems fine. One major problem is that there's no way to know when the item you are watching is moving, so if you inspect something that's animating, your border won't move with it. To fix that you'll need something like https://github.com/flutter/flutter/issues/3861 (though I don't really know how you'd handle it even with that exactly). You could work around it by somehow requesting that you paint every frame that paints, but the system basically spends all its effort making that impossible because it'd be a performance disaster, so there's no good API to hook into to do that right now either.

It sounds like what you want to do is actually hook into the pipeline at a pretty low level. If you're ok with requiring that people add a line of code to their main.dart and import a separate package, you could provide a new binding that overrides the FlutterWidgetsBinding to provide the relevant hooks, in much the same way that the test framework does that. (In your case, you'd probably need to create a new BuildOwner so that you could override the handling of dirty widgets so that any time you have a dirty widget, you also mark the inspector as potentially needing an update, as well as creating a new PipelineOwner so you could catch any changes that result in a paint update, in case any of the render objects are repainting independent of any widget updates.)
The alternative would be to hook onDrawFrame to just always mark the inspector dirty, passing through every call to the underlying binding. That would be really sketchy though. For example, it would totally break down if you had multiple inspector widgets at once. It would also be less efficient, though honestly even the binding override won't be great in that respect.

@jacob314 Should we close this PR for now?

@jacob314 I'm going to close this PR since my understanding is that you're not awaiting review and do not plan to land this exact PR, and so we should take it off the review queue. Don't hesitate to reopen it or open a new one when you're ready for another round of review!

@jacob314 You reopened the PR, does that mean you want review again?

I'm going through the comments again now. I need to address a few more of them before it is ready to be reviewed again.

Using a layer appears to work great for determining when to redraw.
Pushed a new version which should address all comments.
Ready for a full review.

This seems pretty solid. Only a few more tweaks. :-)

Given how expensive this approach is, we should make sure that this widget cannot run outside of debug mode. Can you add some code that will throw an exception in profile and release builds, saying that this widget isn't available outside debug mode? Also, please add a detailed comment to the Layer subclass explaining that this approach is horrifically for performance and is only used here because this is limited to debug mode and that nobody should think of using this in production code. :-)

I'm not sure whether it is feasible, but here's a design that might be more efficient:

Move the widget tree that is under inspection to an off-screen pipeline owner.
When the off-screen pipeline owner requests a visual update, request a visual update in the on-screen pipeline owner.
In a persistent frame callback, flush the off-screen pipeline owner.
Read the relevant information from the off-screen pipeline owner and generate writes into a normal on-screen render tree.
Let the on-screen pipeline owner flush the on-screen render tree as usual.

This approach will let the inspector use all the machinery of the whole system to work efficiently instead of attempting to do its own layout and paint phase during the addToScene walk of the composite phase.
Essentially, using a separate pipeline owner separates the work from the inspector's pipeline and the tree under inspection so that they can run in series instead of in parallel.
If it is too disruptive to move the tree that is under inspection off-screen, another approach would be for the inspector to build a render tree in an off-screen pipeline owner and then flush the inspector's pipeline during the composite phase.  This approach is less elegant in the sense that we'd still be running a mini version of the whole system inside the composite phase of the real system, but at least you'd have all the machinery of the whole system available to you so you wouldn't need to do grossly inefficient things like re-creating TextPainter objects for every animation frame.

Addressed all comments.
@abarth interesting ideas for how to improve the performance. I'd like to investigate the first option once the initial CL lands. Having a custom PipelineOwner could be interesting for other reasons as wel.
Keep in mind the current design at feels like it has fine performance for the applications I've tested with likely because the UI I need to render in the overlay is extremely simple. Note that the TextPainter is now cached almost all times when the selected RenderObject does not change. The current UI is as complex as the UI rendered within the screen area for Chrome Devtools so I expect there is a good change we do not need to render anything more sophisticated on the device screen to provide a good developer experience.

Please don't take my comments as blocking in any way.  It's just an idea for an approach that might be more scalable (assuming it works at all).

the UI I need to render in the overlay is extremely simple

These tools tend to get more sophisticated over time.  There's some value in choosing an approach with a solid foundation to be ready for the feature to grow over time.

@jacob314 Looks like you didn't push your recent changes (or at least, the first comment, which you say is done, doesn't seem to have been done).

Fixed the typo in the git push command. Ready for review now.

Removed the direct dependency on material by moving the tooltip code to /rendering and injecting a builder to create the button to re-enter inspect mode. If no builder is specified, the inspector always stays in select mode which is fine as toggling back and forth between select and view modes is an experiment which we could easily remove.

Note that the hit testing code was also simplified to eliminate all dependence on RenderBox hit testing as RenderBox hit testing started giving undesirable results for sample applications due to an added gesture recognizer. The new heuristic is the render object with the smaller bounding rectangle wins if multiple hit test match occurs and the match is not on the edge of one of the rectangles.

Are there any tests for the inspector itself?

Added tests to the inspector. Made  _WidgetInspectorState and _InspectorSelection public for easier testing.

Ready for next round of review. Add tests for the inspector itself, addressed all other outstanding comments.

Modulo the minor comments:


after installed by npm ,how to use in alfted

emmm

Would you be more specific, please?
FYI: I update this workflow, so you can try installing new version

Would you be more specific, please?
FYI: I updated this workflow today, so you can try installing the new version

Hi. After update the workflow does not work again 😅


Process Clair notifications in background.

Clair's webhook will block and mark notification delivery as failed if it doesn't return fast enough. clair-sqs should receive the notification call and then process it in the background, else the notification will be delivered until its expired.

Fixing that bug fixed that behaviour already; closed for now.


make callouts have angle='auto' for the default

should be 30 degrees unless callout segment will go outside of plotting area, then use best angle selection (give 4 options around the callout location)

also for arrows?

Nope. Arrows is a base R function and should act like one.

:+1: gotcha

Fixed by #218


Visitor inefficiencies

In its current form the visitation is still rather inefficient:
Regardless of whether a program actually overrides a given fn visit_* method the given node will be cloned and passed to the visitor. Every single node in the crate's source code. Every single one. In a large code base this might end up in hundreds of thousands of clones.
This is happening due to the eager node.clone() in
match self.kind() {
    $(SyntaxKind::$kind => visitor.$method(
        &ast::$node::cast(node.clone()).expect(INTERNAL_ERR),
        semantics,
    ),)*
    _ => {},
};

One possible way to improve this is to change
$(fn $method(&mut self, _node: &ast::$node) {})*

to
$(fn $method<T>(&mut self, _node: &SyntaxNode, _materialize: T)
where
    T: FnOnce(&SyntaxNode) -> ast::$node
{})*

and
$(SyntaxKind::$kind => visitor.$method(
    &ast::$node::cast(node.clone()).expect(INTERNAL_ERR),
),)*

to
$(SyntaxKind::$kind => visitor.$method(
    self,
    |node| ast::$node::cast(node.clone()).expect(INTERNAL_ERR),
),)*

Which effectively moves the .clone() and corresponding ::cast(...) into a lazily evaluated closure.
As such a method like this
fn visit_struct(&mut self, _node: &Struct) {}

would become
fn visit_struct<T>(&mut self, _node: &SyntaxNode, _materialize: T)
where
    T: FnOnce(&SyntaxNode) -> ast::Struct
{}

The user would then have to explicitly materialize the node like so:
fn visit_struct<T>(&mut self, node: &SyntaxNode, materialize: T)
where
    T: FnOnce(&SyntaxNode) -> ast::Struct
{
    let struct_node = materialize(node);
    // ...
}

This way unless the user explicitly materializes the node as a typed variant no cloning is performed.
This also has the additional benefit of passing ownership to the method, avoiding more clones, if the the user needs to keep the object.
This of course comes at the cost of a slightly more complex API. I'd argue though that it's worth the

What do you think about this, @pksunkara?

Another alternative would be to remove the typed fn visit_* altogether, as they are causing the issue to begin with. But then not much would be left of the trait/crate. 🤔

I too was afraid of the clone when I started working on this. Then, I investigated and found this little definition in rowan/src/cursor.rs:
#[derive(Debug, Clone)]
pub struct SyntaxNode(Rc<NodeData>);

And all the ast nodes are just structs on rowan::SyntaxNode on this. So, the cloning was really just cloning a pointer at the end.
Were you noticing any performance issues? Do you think we should still address this?

Apart from the .clone() the ::cast(&node) was another possible issue.
I say was, as it is apparently just as efficient as the clone. This is how basically all cast implementations look like:
impl AstNode for Struct {
    fn can_cast(kind: SyntaxKind) -> bool {
        kind == STRUCT
    }

    fn cast(syntax: SyntaxNode) -> Option<Self> {
        if Self::can_cast(syntax.kind()) {
            Some(Self { syntax })
        } else {
            None
        }
    }

    fn syntax(&self) -> &SyntaxNode {
        &self.syntax
    }
}

A quick benchmark confirms this:
// With clone + cast:
// Time elapsed: 23.810835ms
// Number of nodes is: 36992

// Without clone + cast:
// Time elapsed: 23.331632ms
// Number of nodes is: 36974

So yeah, closing this for obvious reasons.

Thanks for confirming it. I haven't benchmarked it before. 🙏

Also just to clarify the difference in node count during benchmarks:
I was running the visitor on the project itself, hence the difference between versions.


[Bug]: 403 errors on github api

Occurred when viewing PR merge activity over time. Went away after hard refresh and haven't been able to reproduce since

The front end should not be accessing the GitHub API at all. Rather, all of the GitHub API accesses should only be done when building the cache, and the front end should exclusively read from the cache.


Always getting 403

Hi.
Trying to use the connector getting Bearer token retrieved but the API always gives me 403 Forbidden.
Using base url:
https://generic.wg.spotify.com/podcasters/v0
Any idea what could be?
Thank you

The base URL looks fine.
Can you double-check that your podcast ID is correct?
Also, can you set
export LOGURU_LEVEL=TRACE

in your terminal, and then try again? The output should then contain the API response. (Careful, it might contain sensitive data.)

@UlTriX do you use Spotify for Creators (previously Anchor) as host as well or just for the statistics?

@mre I am already running with
export LOGURU_LEVEL=TRACE
2024-11-26 18:33:21.550 | TRACE    | spotifyconnector.connector:_request:253 - request - https://generic.wg.spotify.com/podcasters/v0/shows/1aYO5HvnAl9Kd2XuVX0Mmw/listeners?start=2019-12-04&end=2019-12-04
2024-11-26 18:33:21.688 | ERROR    | spotifyconnector.connector:_request:273 - Error in API:
2024-11-26 18:33:21.688 | INFO     | spotifyconnector.connector:_request:274 - 403
2024-11-26 18:33:21.689 | INFO     | spotifyconnector.connector:_request:275 - {'content-type': 'text/plain; charset=utf-8', 'cache-control': 'private, max-age=0', 'access-control-allow-origin': '*', 'access-control-allow-headers': 'Accept, App-Platform, Authorization, client-token, content-access-token, Content-Type, Origin, Retry-After, SPA-Preferred-Publisher, Spotify-App, Spotify-App-Version, spotify-org-uri, X-ClientAttribute-Version, X-Client-Id, x-cloud-trace-context, X-Cloud-Trace-Context, X-Geo-Country, X-Installation-Id, x-permission-grant-token, X-Spotify-Additional-Idp, X-Spotify-Connection-Id, X-Spotify-Quicksilver-Uri, x-twitch-jwt', 'access-control-allow-methods': 'POST, GET, OPTIONS, PUT, HEAD, DELETE, PATCH', 'access-control-allow-credentials': 'true', 'access-control-max-age': '604800', 'Content-Length': '0', 'strict-transport-security': 'max-age=31536000', 'x-content-type-options': 'nosniff', 'date': 'Tue, 26 Nov 2024 18:33:21 GMT', 'server': 'envoy', 'Via': 'HTTP/2 edgeproxy, 1.1 google', 'Alt-Svc': 'h3=":443"; ma=2592000,h3-29=":443"; ma=2592000'}
2024-11-26 18:33:21.689 | INFO     | spotifyconnector.connector:_request:276 -
Show is 1aYO5HvnAl9Kd2XuVX0Mmw grabbed from the spotify url.
@woolfg  I was invited as an admin to the Podcast in question. I access Spotify for Creators to see the statistics.
I believe Spotify for Creators is being used as host. How can I double check that ?
Thank you for all the help.

Hey @UlTriX, the connector does not work for Anchor hosted podcasts as Spotify uses different APIs for that. Those APIs also use a different authentication method. We have built this already for one of our clients in our hosted SaaS solution which can be found on openpodcast.app (sorry, currently the website is German only, but feel free to contact us at<EMAIL_ADDRESS>) but this specific part is not open sourced.


icon color not working Form item floating labels

icon color not working Form item floating labels
I have checked in mobile inspector, it is (icon) showing color #000
`import {Container, Badge,Text, Button, Content, Form, Item, Input , Label,Icon} from 'native-base';`

<View style={[ styles.bg]}>
    <Form>
        <Item floatingLabel>
            <Icon style={{color: '#fff'}} name='ios-person-outline' />
            Username
            <Input />
        </Item>
        <Item floatingLabel>
            <Icon style={{color: '#fff'}} name='ios-lock-outline' />
            Password
            <Input />
        </Item>
    </Form>
</View>

non floating form item  working fine, only floating variable not working
<Form>
    <Item>
        <Icon style={{color: '#fff'}} name='ios-person-outline' />
        <Input placeholder='Username' />
    </Item>
    <Item>
        <Icon style={{color: '#fff'}} name='ios-lock-outline' />
        <Input placeholder='Password' />
        <Input />
    </Item>
</Form>

Thanks, for reporting. Will be looked into.
cc @shivrajkumar

I get this error. :)

@iambanh2 @iiitmahesh what you guys can do is to add
  '.floatingLabel': {
        'NativeBase.Input': {
          height: 60,
          top: 8,
        },
        'NativeBase.Icon': {
          color:'blue',
        },

        'NativeBase.Label': {
          top: 8,
        },
      },

in the native-base-theme/components/Item.js
It worked for me .Give it a try .

@shivrajkumar I am still getting this issue. Icon not appearing when using floating label. my configuration is as follows:
"native-base": "2.2.0",
"react": "16.0.0-alpha.12",
"react-native": "0.45.1",

@GeekRishabh your solution also didn't work for me. I am new to react-native. Can someone please help me with this issue. @shivrajkumar can you reopen this issue?

Same issue for me. color prop is not working in floating label. Is there any solution?

@crazydev71 for now I am writing Icon component outside the Item component and that is working fine.


Re-organizing the imports in the tests.

So we can switch to explicit registration, instead of having modules
register themselves, which is not very pythonic.
@dumbunny @aaijazi

LGTM, but I think you should get @dumbunny's sign off before merging.

LGTM. After the merge / import, make sure there are no BUILD issues with test/environment.py due to the dynamic imports.


How to obtain scaled imputed expression values

Hello,
The procedure provided in the tutorial to obtain imputed values returns a matrix without cell scale correction. How would it be possible to get the scale-normalized data matrix?
Thank you for your great work on the python package.

Hi, thank you for raising your concern, I opened a PR to make it simpler to retrieve the cell scale normalized values reconstructed by our model and explained how in the basic tutorial. This will be included in the next release of the package which is coming soon


nginx log format

Hello!
I have tried everything but can't make it work with this nginx log format:
$remote_addr - $remote_user [$time_local] "$request" $status $body_bytes_sent "$http_referer" "$http_user_agent" "$http_x_forwarded_for"
Example:
<IP_ADDRESS> - - [23/Aug/2013:14:01:26 +0100] "GET /sites/xxx.pt/files/imagecache/64x64/avatar/picture-156409.jpg HTTP/1.1" 200 1768 "http://xxx.pt/forum/pr-saldos-80" "Mozilla/5.0 (Windows NT 5.1; rv:22.0; Avant TriCore) Gecko/20130630 Firefox/22.0" "<IP_ADDRESS>, <IP_ADDRESS>"
I'm using version 0.6. I need the %h to be the http_x_forwarded_for
Thanks!

Have you solved this problem ? I have the same problem.
@introfini

@gxhllj are you having issues with the log format?

yes, and i have replied you in another issues.
maybe i need email you.
@allinurl


x/pkgsite: build issues on AIX and Plan 9

We've disabled the AIX and Plan 9 builders for pkgsite because of two dependencies that break on those platforms respectively:

github.com/go-git/go-billy/v5 and github.com/go-git/go-git/v5 are broken on AIX (These are the same packages causing #58308)
github.com/lib/pq is broken on Plan 9 (I've sent pull request https://github.com/lib/pq/pull/1133) but it seems like pull requests are not frequently reviewed.

Once these fixed either upstream or through removing a dependency, we can remove the code in x/build/dashboard/builders.go that disables these on the Plan9 and AIX builders and their misc-compile builders.

@matloob anything left for this issue?

I was thinking that this issue could track the work to eventually reenable the plan9 and aix builders.