Php 8.3 compatibility issue

Monolog 3.6.0
While checking the php compatibility via codesniffer for PHP 8.3, it is giving these errors for Monolog library
116 | ERROR | "$this" can no longer be used in a plain function or method since PHP 7.1.
121 | ERROR | "$this" can no longer be used in a plain function or method since PHP 7.1.
126 | ERROR | "$this" can no longer be used in a plain function or method since PHP 7.1.
138 | ERROR | "$this" can no longer be used in a plain function or method since PHP 7.1.
157 | ERROR | "$this" can no longer be used in a plain function or method since PHP 7.1.
176 | ERROR | "$this" can no longer be used in a plain function or method since PHP 7.1.

In which file?!

Ah that looks to be in Monolog\Level.. I guess this is a bug in php-codesniffer because it's an enum and not a class? Anyway nothing wrong here IMO

See  https://github.com/squizlabs/PHP_CodeSniffer/issues/3827 - make sure you update your tooling if you are using the latest PHP version.

@Seldaek I'm having trouble understanding the situation. Has the issue been resolved, or is there still a problem? Are you implying that there might be an issue with the configuration? Please provide a clear solution or a well-constructed response before closing tickets.

There is no problem at all. Please read the linked issue before complaining. This was never a problem with Monolog.
I'll paste it here though:

I happen to know this has already been fixed in the PHPCompatibility develop branch.

This bug was fixed over a year ago in the PHPCompatibility project. So you need to update that.


Fix typos

I'm not entirely sure about the second patch in here, please confirm that this is correct.

Looks good to me, thanks!
bors r+


Missing documentation for Custom InputFormatters

This article clearly documents building a custom OutputFormatter, but no information at all is provided for building a custom input formatter.

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

ID: e2207f37-0cf3-3ab9-add8-c50c36409b53
Version Independent ID: df89a2fd-501e-c917-2962-e4d8d124e852
Content: Custom formatters in ASP.NET Core Web API
Content Source: aspnetcore/web-api/advanced/custom-formatters.md
Product: asp.net-core
GitHub Login: @Rick-Anderson
Microsoft Alias: tdykstra

[Off topic, but amazing work everyone to make this so seamlessly integrated for documentation efforts]

@danroth27  do we want to add Custom InputFormatters  to this topic?

Hmm, I'm not sure what is missing here. The doc talks about does talk about how to implement input formatters and the sample has an example implementation of a vCard input formatter.

Override ReadRequestBodyAsync/WriteResponseBodyAsync methods

public class VcardOutputFormatter : TextOutputFormatter

Inherited from OutputFormatter.
Where's the implementation of ReadRequestBodyAsync?
There's 3 usages of ReadRequestBodyAsync on the page and none of the 3 usages is a code sample.
https://imgur.com/a/cFgjrpu

@dotnetchris  you have to expand the #region to see it. It's there.


[BUG] Challenge required Error

Hey,
I followed the tutorial and uploaded some Reels to my instagram without any problem until I got a challenge exception
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\private.py", line 298, in _send_private_request
  File "C:\python3.9.4\lib\site-packages\requests\models.py", line 953, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://i.instagram.com/api/v1/media/configure_to_clips/?video=1

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "F:\Tiktok\script\Instagram\TikTokMain.py", line 78, in <module>
    media = cl.clip_upload(
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\clip.py", line 191, in clip_upload
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\clip.py", line 171, in clip_upload
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\clip.py", line 267, in clip_configure
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\private.py", line 449, in private_request
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\private.py", line 330, in _send_private_request
instagrapi.exceptions.ChallengeRequired: challenge_required

After reading some user guids I added these to my code
from instagrapi.exceptions import (
    BadPassword, ReloginAttemptExceeded, ChallengeRequired,
    SelectContactPointRecoveryForm, RecaptchaChallengeForm,
    FeedbackRequired, PleaseWaitFewMinutes, LoginRequired
)

def handle_exception(client, e):
    if isinstance(e, BadPassword):
        client.logger.exception(e)
        client.set_proxy(self.next_proxy().href)
        if client.relogin_attempt > 0:
            self.freeze(str(e), days=7)
            raise ReloginAttemptExceeded(e)
        client.settings = self.rebuild_client_settings()
        return self.update_client_settings(client.get_settings())
    elif isinstance(e, LoginRequired):
        client.logger.exception(e)
        client.relogin()
        return self.update_client_settings(client.get_settings())
    elif isinstance(e, ChallengeRequired):
        api_path = json_value(client.last_json, "challenge", "api_path")
        if api_path == "/challenge/":
            client.set_proxy(self.next_proxy().href)
            client.settings = self.rebuild_client_settings()
        else:
            try:
                client.challenge_resolve(client.last_json)
            except ChallengeRequired as e:
                self.freeze('Manual Challenge Required', days=2)
                raise e
            except (ChallengeRequired, SelectContactPointRecoveryForm, RecaptchaChallengeForm) as e:
                self.freeze(str(e), days=4)
                raise e
            self.update_client_settings(client.get_settings())
        return True
    elif isinstance(e, FeedbackRequired):
        message = client.last_json["feedback_message"]
        if "This action was blocked. Please try again later" in message:
            self.freeze(message, hours=12)
            # client.settings = self.rebuild_client_settings()
            # return self.update_client_settings(client.get_settings())
        elif "We restrict certain activity to protect our community" in message:
            # 6 hours is not enough
            self.freeze(message, hours=12)
        elif "Your account has been temporarily blocked" in message:
            """
            Based on previous use of this feature, your account has been temporarily
            blocked from taking this action.
            This block will expire on 2020-03-27.
            """
            self.freeze(message)
    elif isinstance(e, PleaseWaitFewMinutes):
        self.freeze(str(e), hours=1)
    raise e

def get_code_from_email(username):
    mail = imaplib.IMAP4_SSL("outlook.office365.com")
    mail.login("email", "pass")
    mail.select("inbox")
    result, data = mail.search(None, "(UNSEEN)")
    assert result == "OK", "Error1 during get_code_from_email: %s" % result
    ids = data.pop().split()
    for num in reversed(ids):
        mail.store(num, "+FLAGS", "\\Seen")  # mark as read
        result, data = mail.fetch(num, "(RFC822)")
        assert result == "OK", "Error2 during get_code_from_email: %s" % result
        msg = email.message_from_string(data[0][1].decode())
        payloads = msg.get_payload()
        if not isinstance(payloads, list):
            payloads = [msg]
        code = None
        for payload in payloads:
            body = payload.get_payload(decode=True).decode()
            if "<div" not in body:
                continue
            match = re.search(">([^>]*?({u})[^<]*?)<".format(u=username), body)
            if not match:
                continue
            print("Match from email:", match.group(1))
            match = re.search(r">(\d{6})<", body)
            if not match:
                print('Skip this email, "code" not found')
                continue
            code = match.group(1)
            if code:
                return code
    return False

def challenge_code_handler(username, choice):
    if choice == ChallengeChoice.SMS:
        return get_code_from_sms(username)
    elif choice == ChallengeChoice.EMAIL:
        return get_code_from_email(username)
    return False

def change_password_handler(username):
    # Simple way to generate a random string
    chars = list("abcdefghijklmnopqrstuvwxyz1234567890!&£@#")
    password = "".join(random.sample(chars, 8))
    return password

cl.handle_exception = handle_exception
cl.challenge_code_handler = challenge_code_handler
cl.change_password_handler = change_password_handler

And now i am getting this
Status 429: Too many requests
Traceback (most recent call last):
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\private.py", line 298, in _send_private_request
  File "C:\python3.9.4\lib\site-packages\requests\models.py", line 953, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 429 Client Error: Too Many Requests for url: https://i.instagram.com/api/v1/accounts/contact_point_prefill/

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\private.py", line 432, in private_request
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\private.py", line 365, in _send_private_request
instagrapi.exceptions.PleaseWaitFewMinutes: Please wait a few minutes before you try again.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "F:\Tiktok\script\Instagram\TikTokMain.py", line 122, in <module>
    cl.login("email", "pass")
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\auth.py", line 374, in login
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\auth.py", line 47, in pre_login_flow
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\auth.py", line 147, in set_contact_point_prefill
  File "C:\python3.9.4\lib\site-packages\instagrapi-1.16.7-py3.9.egg\instagrapi\mixins\private.py", line 441, in private_request
  File "F:\Tiktok\script\Instagram\TikTokMain.py", line 95, in handle_exception
    self.freeze(str(e), hours=1)
NameError: name 'self' is not defined

My main Objective is to solve the "instagrapi.exceptions.ChallengeRequired: challenge_required"
and continue uploading to my reels.
Though that adding challenge handler and exception handler would solve that for me but it did not ):
Anything helps !
Desktop (please complete the following information):

OS: [Win 10]
Python version [e.g. 3.9.4]
instagrapi version [1.9.3]

Just FYI although you have removed your password from the post, anyone who subscribed to the library still received it by email.
Suggest you change your outlook.com password.


Webhooks no longer work.

Hello Scott,
I got a weird issue with my Gitlab and your Webhooks plugin.
For some reason if I commit to my repo, the main page doesn't get update anymore.
Alltough the application (gitlab) and the webhooks-plugin are telling me that the hook was successfully executed?!
Thats the Repo: https://sc.knight-industries.org/knight-industries-org/www-knight-industries-org
The Gitlab Webhook Setting:

And the Webhooks Setting in the OctoberCMS Backend:

The Git Project is set to trigger on build success. Then it should push the code to the page.
It worked before but I'm not sure whats wrong.
Log of the Webhook:

And the newest logs are somehow empty :s

Do you have nay advice?

Hmm, I wonder what has changed that caused it to stop working. Did you modify the webhook script, or anything with the git repository? I haven't pushed any changes to the webhooks plugin in a while, when was the last time this hook was working?

Hmm about a week or so. Only thing I changed was the auth method form password to SHA-2 certificate.  But I don't think that this is the problem.
Last Day it worked was

And I didn't changed the Webhooks code.

Hmm, well since the last successful pull was about a week ago, and around that time was also when you changed the auth method, I'm willing to be that's your problem. Try setting it back to the way it was and see if that fixes it, then we'll know for sure.

Nope that didn't worked out. Even a newly created Webhook for a different project doesn't work.

Try changing the script to this and see if the hook is even triggering anything
echo "foo"

 Yes that worked

Ok so the webhook is definitely being hit, it's a problem with the git pull. I don't have time to try and recreate it right now, but I'm still inclined to believe it's some kind of authentication issue with GitLab.

Yes it was a permission issue. Had to change the key of my webuser to 700, issue therefore closed!

Sweet, glad you got it straightened out :)


OD unit test passing turend on with TF

Get rid of MXNET from OD to make it testable.

Fix batch size issue for OD (pad each data batch to full batch_size).
fix: ObjectDetectorTest::test_dict_annotations, ObjectDetectorTest::test_single_image
Fix the crash when an image doesn't have annotations.
fix: ObjectDetectorTest::test_dict_annotations
Reset tf graph when create model
fix: ObjectDetectorTest::test_export_coreml, ObjectDetectorTest::test_export_coreml_with_non_maximum_suppression

Now the OD should run without any crashes and failures.


Vertical scrollbar doesn't use available space

VSCode Version: 1.2.0
OS Version: OS X el capitan

Steps to Reproduce:

scroll to the bottom of an editor, so that the last line is aligned with the bottom of the editor
notice that the scroll bar position indicator is not at the bottom of the scroll bar

Expected behavior:

the scroll bar position is mapped to the full length of the file without the overflow that allows scrolling further.
the dark scroll position that indicates the beginning and the end of the page shrinks as you scroll beyond the end of the file

Screenshots to clarify:

Current situation

Desired situation

The dark horizontal line shows where the cursor is. The scrollbar slider is artificially enlarged to be at least 20px high I think (to allow for clicking on it). The scrollbar is mapped to the full height of the file (which includes the extra page at the bottom). You can remove this via editor.scrollBeyondLastLine.

@alexandrudima I think you are misunderstanding me.
Basically, I would like the scrollbar to be mapped to the full height of the file without the extra page at the bottom.
That way, there is more space available to show markings.
Right now, the last page-equivalent of the scrollbar is basically unused. I really like scrolling beyond the last line, so I'd rather not give that up.

An example of the same behavior is when you do overscroll in e.g. Chrome. The size of the scrollbar handle is constant throughout the page, because it reflects page size compared to the full document, and while you overscroll, it shrinks.

To keep the number of issues in our inbox at a manageable level, we're closing issues that have been on the backlog for a long time but haven't gained traction: we look at the number of votes the issue has received and the number of duplicate issues filed. Thank you for your time and understanding.
P.S. If you disagree and feel that this issue is crucial: we're happy to listen and to reconsider.


Smart equal ignore

Changes in this PR:
Changed the SMART Response EqualIgnore function to receive a list of fields to ignore as a variable. Updated SMART Response EqualIgnore tests to make sure the function works as expected when given list of ignored fields.
Pull requests into this repository require the following checks to be completed by the submitter and reviewers.
Submitter:

[x] Administrative tasks are complete.

This pull request describes why these changes were made.
The JIRA ticket links to this PR.
The JIRA ticket for this PR is: Lantern 311 JIRA Ticket

[x] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

[x] Code review has been performed.

Code diff has been done and checked to ensure only expected code is being committed.
A linter for the language has been run.

Primary Reviewer:

[x] Code works.

Code has been run locally and the basic functionality affected by the PR has been checked.
Edge cases have been checked to ensure behavior is as expected.

[x] Code review has been performed.

Code is maintainable, reusable, efficient, and correct.
Code accomplishes the tasks purpose.
Code follows style guidance appropriate for the language, including passing linter checks.
Code is well commented (which does not mean verbosely commented).

[x] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

Secondary Reviewer:

[ ] Code works.

Code has been run locally and the basic functionality affected by the PR has been checked.
Edge cases have been checked to ensure behavior is as expected.

[ ] Code review has been performed.

Code accomplishes the tasks purpose.

[ ] Tests are complete.

Tests have been run locally and pass.

Merger:
Prior to merging, ensure that all checks have been completed. Delete the branch after the merge
is complete.
Create a new branch <branch name>_gomodupdate. Run make update_mods to update the go.mod
and go.sum files to point to master. Create a PR. Require one sanity check reviewer.

This branch has that same small issue with shiny dashboard that the other smart response merge request had, so i assume it'll be resolved once you rebase this branch!

🤔  I'm getting this error when building and populating database from scratch.

🤔 I'm getting this error when building and populating database from scratch.

This is an error from the SMART Response branch, Emily addressed it in the other PR but this hasn't been updated with those changes yet!

I know you didn't touch this but I figure it could be easily addressed in this PR since I just noticed it: I don't think the query for the FHIR Version/Capability/Endpoints table is correct. I think the get_smart_response_capabilities query in the endpoints.R should be formatted using LEFT JOINs like in the get_well_known_endpoints_tbl query right below it. Let me know if that doesn't work or you have any questions!


Remove honojs/hono-minimal

What is the feature you are proposing?
We have to remove the honojs/hono-minimal project. It's no longer maintained and it will confuse the beginner for Hono.

I've made it as private. Nobody can use it now.


MSVC clz64 for x86

clz64 function for MSVC in cutils.h and tests/test_conv.c uses _BitScanReverse64 which is not defined for x86. I suggest to use the following implementation:
static inline int clz64(uint64_t a)
{
#if defined(_MSC_VER) && !defined(__clang__)
#if INTPTR_MAX == INT64_MAX
    unsigned long index;
    _BitScanReverse64(&index, a);
    return 63 - index;
#else
    if (a >> 32)
        return clz32((unsigned)(a >> 32));
    else
        return clz32((unsigned)a) + 32;
#endif
#else
    return __builtin_clzll(a);
#endif
}

Similar fix: videolan project

Sounds good, can you send a PR?

i send a fix.https://github.com/quickjs-ng/quickjs/pull/435


Date of birth infinitive scrolling IOS

When entered from left to right MM-DD-YYYY (right to left works)

Thanks for feedback, github project are not the best place to provide feedback so I will close this now.


Bump bci-base to 15.6

related to harvester/harvester#6568

@mergify backport v0.3.x

@mergify backport v0.2.x


Card number formatting breaking with copy and paste.

Expected behavior
When you copy and paste a card number into a formatCardNumber-applied element, then entering a CVC into a formatCardCVC-applied element, the card number should retain it's formatting.
Actual behavior
The formatting on the card number is lost upon entering your CVC number. To get the formatting back the user has to tab back to the card number field and delete a single number.
Chrome Version 55.0.2883.95 (64-bit)
jquery.payment v3.0.0
Steps to reproduce
http://d.pr/v/FLlA -- Video displaying the behavior.

There shouldn't be any interaction between the cvc and card fields. It looks like this might be an integration issue. If it's still a problem, can you please re-open with a minimal jsfiddle or equivalent?


Views

Hi there 👋,
In this particular project, I :

Implement the design from the sneak peek wireframes.

Styling is nice to have but focus on making sure everything functions first. (Note: I ignore login logic, I am using Devise in coming milestone)

Use methods that I have created in the Blog App - processing data in models.
Hint: In order to see some posts or comments, you might need to add some data to your database as described here.

Kidnly try to use this link in order to give a good image. photo =
https://user-images.githubusercontent.com/108806646/216281530-fa5c3020-197a-498e-9024-339268ba1959.jpg


fix: missing jwt secret

From the latest commit there seems to be a breaking change when running kurtosis run github.com/kurtosis-tech/ethereum-package '{"mev_type": "mock"}'
There was an error interpreting Starlark code
Evaluation error: struct has no .jwt_secret attribute
        at [github.com/kurtosis-tech/ethereum-package/main.star:131:47]: run

Error encountered running Starlark code.

This PR fixes the error.

Can you please run kurtosis lint --format to fix the linting issue?

Closing in favor of #312


Unable to find the tar.gz package of the documentation

I spent a good amount of time looking for it to no avail.
Finally i found them in the release section of the GitHub repository.
I think the links do download the links should be included with each release.

In #55 I've added dropdowns to access the download URLs to the documentation. Please have a look and see if it is what you had in mind.

It looks really good.
But it is kind of hidden, inside the "View documentation".
I think below VIEW DOCUMENTATION we could have DOWNLOAD DOCUMENTATION, and if you click on the button or on the arrow, it will show the drop´-down that we show now.


[r6.0.0] cyber_recorder record improvements

There seems to be something wrong with Apollo CI. This PR has already been merged into the master branch. Please help to make CI pass on other branches. @lx18233184051

Tested locally, works the same as with r5.5.0
[martin@in-dev-docker:/apollo]$ cyber_recorder info /media/storage/data/bag/2020-10-30-23-23-57/20201030232358.record.00000
record_file:    /media/storage/data/bag/2020-10-30-23-23-57/20201030232358.record.00000
version:        1.0
duration:       43.499768 Seconds
begin_time:     2020-10-30-23:23:58
end_time:       2020-10-30-23:24:41
size:           2094653 Bytes (1.997617 MB)
is_complete:    true
message_number: 8864
channel_number: 17
channel_info:
                /apollo/monitor                                         87 messages: apollo.common.monitor.MonitorMessage
                /tf                                                      0 messages: apollo.transform.TransformStampeds
                /apollo/localization/pose                             4351 messages: apollo.localization.LocalizationEstimate
                /apollo/localization/msf_lidar                           0 messages: apollo.localization.LocalizationEstimate
                /apollo/localization/msf_gnss                            0 messages: apollo.localization.LocalizationEstimate
                /apollo/localization/msf_status                          0 messages: apollo.localization.LocalizationStatus
                /apollo/routing_response                                 0 messages: apollo.routing.RoutingResponse
                /apollo/routing_response_history                         0 messages: apollo.routing.RoutingResponse
                /apollo/monitor/system_status                           33 messages: apollo.monitor.SystemStatus
                /apollo/canbus/chassis                                4351 messages: apollo.canbus.Chassis
                /apollo/prediction                                       0 messages: apollo.prediction.PredictionObstacles
                /apollo/navigation                                       0 messages: apollo.relative_map.NavigationInfo
                /apollo/routing_request                                  0 messages: apollo.routing.RoutingRequest
                /apollo/hmi/status                                      42 messages: apollo.dreamview.HMIStatus
                /apollo/control/pad                                      0 messages: apollo.control.PadMessage
                /apollo/audio_event                                      0 messages: apollo.audio.AudioEvent
                /apollo/drive_event                                      0 messages: apollo.common.DriveEvent


Several changes

Update build config
Use new server data format

Are you OK with switching to Kotlin by the way?

Are you OK with switching to Kotlin by the way?

Are you asking whether I am familiar to rewrite this repo in Kotlin or whether I am fine with okhttp upgrading to Kotlin? I don't know Kotlin enough to make an informed decision. (I do not like Java much but it seems to me Java > 8 has added many goodies making life more pleasant.)
WRT to okhttp, my understanding from https://github.com/square/okhttp/issues/4723 is that there will be a backwards compatible Java API and that it should not make the app much more bloated.
So, this actually seems fine to me.

Kotlin and Java are very nicely interoperable.   What is key is that the language (Kotlin vs Java) that dependencies are written is not important at all -- we can use the current OkHttp version actually.  Re Java 8, 9, 10, etc... The issue with Android is that the Java version is baked into the ROM, and you can't use new language features that can't be backported back to old Java versions.  This is one key motivation behind embracing Kotlin on Android.  You can use all the new features since the Kotlin runtime is bundled into the app.
In next PR I will switch to Kotlin + Gson.


[Dropdown Menu] Glitches in Safari

This glitch happens because it's animating all. Animating only opacity, height and padding fixes it. I think it's better for Safari to fix this bug rather than rewriting the material-ui to animate only changed properties :pensive:

This doesn't happen anymore with Popover


multiple file loaded callback

Hello ,
I have some problem for loading multiple and trigger callback .
 PIXI.loader
    .add('loading1', '../common/images/loading/loading_logo1.json')
    .add('loading2', '../common/images/loading/loading_logo2.json')
    .add('loading3', '../common/images/loading/loading_num.fnt', () => {
        // loading1 , loading2 not loaded first than loading3 every time
        loading.showLoading();
    });

// or

PIXI.loader
    .add([
        {url: '../common/images/loading/loading_logo1.json'},
        {url: '../common/images/loading/loading_logo2.json'},
        {url: '../common/images/loading/loading_num.fnt', onComplete(){
            // loading1 , loading2 not loaded first than loading3 every time
            loading.showLoading();
        }}
    ]) ;

so I use these code ....
PIXI.loader
    .add('loading_logo1.json', () => {
        PIXI.loader.add('loading_logo2.json', () => {
            PIXI.loader.add('loading_num.fnt', () => {
                // it works fine
                loading.showLoading();
            })
        })
    });

Does it has any better way ?

Hello!
PIXI.loader is actually https://github.com/englercj/resource-loader so you probably wanted to
 PIXI.loader
    .add('loading1', '../common/images/loading/loading_logo1.json')
    .add('loading2', '../common/images/loading/loading_logo2.json')
    .add('loading3', '../common/images/loading/loading_num.fnt')
    .load((loader, resources) => {
        loading.showLoading();
    });

@sktt Thanks
Sorry for my English ,
But on your way I have the same problem ,
loading1 , loading2 , loading3, is loaded asynchronous ,
I can't confirm when loading 3's callback is trigger and loading1 and loading 2 is loaded or not .
It seems to that loading 3 and loading1 , loading2 not loaded in the queue , and loaded complete at different time .
so I want to ask that , does any way can load three source together and trigger a callback ?
This is my code , It will load loading1 2 3 in the first , when loading complete play a animation .
Animation will stay about 5 sec , so after .load() trigger and play 2.2 sec , I will load another source to do init function .
 PIXI.loader
    .add('loading1', '../common/images/loading/loading_logo1.json')
    .add('loading2', '../common/images/loading/loading_logo2.json')
    .add('loading3', '../common/images/loading/loading_num.fnt')
    .load((loader, resources) => {
        loading.showLoading();

       PIXI.loader.reset();
        setTimeout(()=>{
            PIXI.loader
                .add(sourceArr)
                .on('progress', (data) => {
                    loading.progressBar(data.progress | 0);
                })
                .load(() => {
                    init();
                });
        }, 2200);
    });

All files are loaded in parrellel, no order is guarenteed. If you only wanted to load a single file at a time and do stuff after each one you can change the concurrency value of the loader. To do so, you will need to create your own loader:
var loader = new PIXI.Loader('', 1);

That loader will only load 1 resource at a time, meaning each time your 'load' or 'progress' event is called, a single resource was loaded.

let loader = new PIXI.loaders.Loader();

loader.add([
        '../common/images/loading/loading_logo1.json',
        '../common/images/loading/loading_logo2.json',
        '../common/images/loading/loading_num.fnt'
    ])
    .load(() => {
        loading.showLoading();

        setTimeout(()=>{
            PIXI.loader
                .add(sourceArr)
                .on('progress', (data) => {
                    loading.progressBar(data.progress | 0);
                })
                .load(() => {
                    init();
                });
        }, 2200);
    })

It works fine .
Thanks a lot .


Propagation of certificates after acme-* renewal?

With setting nginx.enableAcme = true; on the day when initially discovered certificates expire, a configured ldap provider cannot be queried any longer because of outdated certificates.
In the admin GUI, we had to manually:

delete the discovered certificate (/admin/#/crypto/certificates)
run the certificate_discovery task (/admin/#/administration/system-tasks)
reassign the certificate to the tenant (/admin/#/core/tenants -> edit -> Other global settings)
reassign the certificate to the provider (/admin/#/core/providers -> edit -> Protocol settings)

This hit us by surprise, by any chance, do you see a way to automate this process?
At least i guess it would be worth of a warning in the README, right?

Was the expired certificate managed by authentik or imported manually at some point? For the latter, I'd expect this behavior.
And for the former, I cannot see the same behavior for a Let's Encrypt certificate which was recently renewed and automatically discovered by authentik:

(note "Managed by authentik (Discovered)")
See also https://goauthentik.io/docs/core/certificates#external-certificates

Files are checked every 5 minutes, and will trigger an Outpost refresh if the files differ.

Mysterious, the one that wasn't changed is a discovered one as well.
There is still a longer living self-signed certificate beneath, and it might be that authentik wasn't restarted after the certificate was refreshed, beside after detecting the outdate causing ldap service disruption. Even the restart wasn't effective although we didn't wait for 5 minutes.
I will keep an eye on the next change cycle to see if it will happen again.

In the current cycle authentik correctly noticed the replacement of the certificate. The reason why it didn't remains unresolved.


Drop kube-proxy/kubelet skew requirement

Discussed in SIG Network today; the docs claim that kubelet and kube-proxy must be the same version, but we had actually forgotten that, and the last time this was actually even theoretically true was 1.19. SIG Network will be making sure there are no "kubelet and kube-proxy must be the same version" problems in the future. (https://github.com/kubernetes/kubernetes/pull/117307 adds some clarification about that.)
(KEP-3178 involves an n-2 skew constraint between kube-proxy and kubelet, but that already exists transitively via apiserver skew constraints without needing a specific kube-proxy/kubelet skew rule: kube-proxy 1.27 doesn't work with kubelet 1.24, but (a) kube-proxy 1.27 requires kube-apiserver >= 1.27, and (b) kubelet 1.24 only supports kube-apiserver <= 1.26.)
cc @thockin @aojea

/approve

👏


kaggle colab does not work

I get this error everytime
<ipython-input-2-4604c83e7d7e> in <module>
      1 import dill as pickle
----> 2 from yuno.search import pipelines,base,config,utils
      3 import torch
      4 import torch.nn.functional as F
      5 import faiss

ModuleNotFoundError: No module named 'yuno'

@IAmPara0x I have no experience on kaggle but I just clicked to copy and edit and I just clicked run all on run section
I get this everytime
<ipython-input-1-cfd3f8ca369a> in <module>
      5
      6 import dill as pickle
----> 7 from yuno.search import pipelines,base,config,utils
      8 import torch
      9 import torch.nn.functional as F

ModuleNotFoundError: No module named 'yuno'

@IAmPara0x ah yeah did it as you said now its working thanks! I just wonder what does increasing or decreasing curiosity

do NOT use latest environment


Async connect failure with unix socket. Regression from merge pull request #578

It broke my 'eredis' client library so I investigated. #578 @mnunberg
The new redisCheckConnectDone() is only managing tcp socket.
With unix socket, c->saddr is NULL and the connect fails, of course invariably, with an errno EINVAL => REDIS_ERR.
To keep the logic, for the unix socket, redisContextConnectUnix() need to store also its sockaddr_un in the redis context and redisCheckConnectDone() need to use it.

Thanks for the bug report. Indeed, I didn't test it with Unix sockets when I implemented it, but the fix seems simple enough.

Am I alone who sees a clear memory leak in 088d146?

@vstakhov this should be freed when the object is freed as well, see https://github.com/redis/hiredis/blob/master/hiredis.c#L692

Ok, it now makes some sense indeed as it is also assigned to c->saddr, thanks. The return result from malloc should be checked for NULL however.


The waymo v2 bucket - cannot find associations between vehicels bbox and lidar

Lately I granted access to waymo v2 via the bucket: waymo_open_dataset_v_2_0_0
There is a file called 'camera_to_lidar_box_association' that include the associations between camera and lidar obejcts.
The associations looks really great but the only problem is that it doesn't include any vehicle associations, only pedastrians.
Is it a known issue? Are you planning to upload it separatly?

Hi,
Thanks for interested in the Waymo Open Dataset. Currently, there is no plan to provide additional association data. I would recommend to use the projected_lidar_labels, which has id in the format of {laser_object_id}_{Camera_Name}. Note that the projected labels is not as high quality of the 2D annotated labels.
Thanks,
Wayne


App Doesn't Open After Exiting... Without Task Manager or Restart

I can open the app only once—if I close it, I can't reopen it without opening task manager and ending all Scout processes or restarting my computer.
I've experienced this issue on two different machines, which is why I'm finally reporting it. Both computers were Dells running Windows 10. More details available upon request... not sure what is needed.

Did you check if it was in the system tray by the clock. There is a setting to turn that on/off?

Wow... I never knew. Thank you, sorry to waste your time.


UioHookEvent.Time should be ulong instead of ushort

No idea how I made that mistake, but here we are - the timestamp of the event gets truncated, because it's defined as 16-bit when in fact it should be 64-bit.

Turns out that the Time field of the event is not a timestamp, but a synthetic value which can be used only to compare the times of two events.

Nevertheless, the issue with the field's size is fixed in 2.0.0.


Delegate / Witness Resignation

Can they resign, should they be able to, and if so how?

Seems fairly important that they be able to resign if they want. Do we need it for launch -- not so sure.

I definitely want witnesses to be able to resign. The logical way to do this is to set their block producing key to null. It would also be nice if they could un-resign by setting it again. If a witness's owner decides to go on vacation or something, he might not want to/be able to maintain his node, so it would be desirable if he could disable his node while he's away, then re-enable it when he returns. I'd like to have this working for launch.
It seems less necessary that delegates be able to resign, but we might as well allow it. I don't see as clear a reason delegates should be able to un-resign, though.
Since we're on the topic, do we allow workers to resign? If we want to allow all three to resign, it might be worth implementing this at the voting level, allowing the owner of a votable entity to withdraw that entity from contests, and later re-enter it.

a7d517eabcf22835b89e15aff144a0ea5abf535e
bba6f78101ed2866c06801fd840cdf955184e5e6
I made these commits in my forked repo. I wasn't super positive in terms of where to start in terms of setting up an outline of where to attack, so I made two and hoped that one of them might be a useful place. Let me know if they're helpful and I'll create a PR. (I'm aware that there is a bit of a nasty tab in the witness_object.hpp file, that was fixed in the next commit).

Please consider this: https://bitsharestalk.org/index.php/topic,17308.0.html

Nathan wrote: " If a witness's owner decides to go on vacation or something, he might not want to/be able to maintain his node, so it would be desirable if he could disable his node while he's away, then re-enable it when he returns. I'd like to have this working for launch."
What is meant by "witness's owner" ? Also, allowing witnesses to come and go may lead to instability of network. Would "the next standby" be signing block in his own delegate name or in the name of the delegate his substitutes for? This discussion should be brought into the forums for broader perspective. As a current delegate, I think this isn't good for the ecosystem, but do recognize the convenience it provides. Just don't believe that's worth the risks.

As to resigning, I think some automatic measures should be built in, like firing a witness if they miss X number of blocks. Translate blocks to time, the limit should not be more than a day or 2.
Obviously if a worker / witness / delegate no longer desires or is able to perform their duties, the network should provide a way to facilitate a replacement as fast and efficiently as possible. In the case of delegates I see no urgency, and they can be replaced through voting schedule / mechanism. Workers and witnesses require more care, and shareholders definitely need to be involved. For witnesses the issue is the most urgent, but again with the numbers involved (who knows how many will be required in 2.0, and how that will be determined as a function of growth and network security), perhaps not all that urgent either.

@ThomasFreedman A witness is a server which connects to the network and produces blocks. The person running that server/controlling the private keys is the witness's owner.
The point of having protocol support for disabling a witness without sacrificing its approval rating is to prevent instability. A witness can be disabled, at which point it will not be eligible for election at the next maintenance interval, and the next runner up will be elected instead. If the witness is re-enabled, and still has sufficient approval to be elected, he will be elected again at the next maintenance interval.
The problem is that runner-ups may not have nodes online when they get elected. We could either regard this as worst practice and recommend stakeholders withdraw approval of this witness, or we could require unelected candidates to submit a ticket proving their nodes are online prior to the maintenance interval at which they may be elected. If no ticket is submitted, they will not be eligible for election. I do not think this feature is important enough to delay launch, but it's something to consider for the future.

This issue was moved to bitshares/bitshares-2#38


Uncaught TypeError: Cannot read property '0' of undefined

[Enter steps to reproduce below:]
See #308 - I got this error as well.
Atom Version: 0.199.0
System: Microsoft Windows 10 Enterprise Insider Preview
Thrown From: omnisharp-atom package, v0.4.33
Stack Trace
Uncaught TypeError: Cannot read property '0' of undefined
At C:\Users\Igal\.atom\packages\omnisharp-atom\lib\omni-sharp-server\client-manager.js:80

TypeError: Cannot read property '0' of undefined
  at ClientManager.getClientForEditor (C:\Users\Igal\.atom\packages\omnisharp-atom\lib\omni-sharp-server\client-manager.js:80:75)
  at ClientManager.getClientForActiveEditor (C:\Users\Igal\.atom\packages\omnisharp-atom\lib\omni-sharp-server\client-manager.js:67:25)
  at Function.Object.defineProperty.get [as client] (C:\Users\Igal\.atom\packages\omnisharp-atom\lib\omni-sharp-server\omni.js:31:28)
  at GoToDefinition.underlineIfNavigable (C:\Users\Igal\.atom\packages\omnisharp-atom\lib\omnisharp-atom\features\go-to-definition.ts:91:17)
  at C:\Users\Igal\.atom\packages\omnisharp-atom\lib\omnisharp-atom\features\go-to-definition.ts:47:79

Commands
  4x -1:28.4.0 markdown-preview:toggle (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "monokai"
    ]
  }
}

Installed Packages
# User
atom-yeoman, v0.2.0
autoclose-html, v0.16.0
browser-refresh, v0.8.3
git-plus, vundefined
language-spec, v0.1.0
linter, v0.12.2
monokai, v0.14.0
omnisharp-atom, v0.4.33
open-in-browser, vundefined
spec-maker, v0.4.0

# Dev
settings-view, v0.185.0

Should be fixed in v0.5.0. Please re-open if it isn't!

I'm getting the same error with 0.6.2.
[Enter steps to reproduce below:]

Open a valid C# file that will kick off OmniSharp
Create a new file.
Paste some random (incomplete) C# code that you'd like to have syntax highlighting for
Eg: ((model.Type == "A") || (model.Type == "B"))
Change the grammar to C#.
BAM! Uncaught TypeError: Cannot read property '0' of undefined.

Atom Version: 0.200.0
System: Microsoft Windows 8.1 Enterprise
Thrown From: omnisharp-atom package, v0.6.2
Stack Trace
Uncaught TypeError: Cannot read property '0' of undefined
At C:\Users\william\.atom\packages\omnisharp-atom\lib\omni-sharp-server\client-manager.js:129

TypeError: Cannot read property '0' of undefined
  at ClientManager.getClientForEditor (C:\Users\william\.atom\packages\omnisharp-atom\lib\omni-sharp-server\client-manager.js:129:88)
  at LinterCSharp.lintFile (C:\Users\william\.atom\packages\omnisharp-atom\lib\omnisharp-atom\linter.js:43:14)
  at C:\Users\william\.atom\packages\linter\lib\linter-view.coffee:163:18
  at Array.forEach (native)
  at C:\Users\william\.atom\packages\linter\lib\linter-view.coffee:162:18
  at FSReqWrap.oncomplete (fs.js:77:15)

Commands
     -0:35.5.0 editor:outdent-selected-rows (atom-text-editor.editor.is-focused)
     -0:22.4.0 omnisharp-atom:toggle-output (atom-workspace.workspace.scrollbars-visible-always.theme-monokai.theme-one-dark-ui)
  2x -0:09.5.0 editor:move-to-first-character-of-line (atom-text-editor.editor.is-focused)
     -0:08.3.0 core:move-up (atom-text-editor.editor.is-focused)
     -0:05.8.0 snippets:previous-tab-stop (atom-text-editor.editor.is-focused)
     -0:05.8.0 editor:outdent-selected-rows (atom-text-editor.editor.is-focused)
     -0:05.5.0 snippets:previous-tab-stop (atom-text-editor.editor.is-focused)
     -0:05.5.0 editor:outdent-selected-rows (atom-text-editor.editor.is-focused)
     -0:05.3.0 snippets:previous-tab-stop (atom-text-editor.editor.is-focused)
     -0:05.3.0 editor:outdent-selected-rows (atom-text-editor.editor.is-focused)
     -0:05.0 snippets:previous-tab-stop (atom-text-editor.editor.is-focused)
     -0:05.0 editor:outdent-selected-rows (atom-text-editor.editor.is-focused)
     -0:04.8.0 snippets:previous-tab-stop (atom-text-editor.editor.is-focused)
     -0:04.8.0 editor:outdent-selected-rows (atom-text-editor.editor.is-focused)
     -0:03.8.0 core:delete (atom-text-editor.editor.is-focused)
     -0:00.9.0 core:undo (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "ignoredNames": [
      ".suo",
      ".vsmdi",
      ".vscc",
      ".dll"
    ],
    "disabledPackages": [
      "data-atom",
      "emmet",
      "wrap-guide",
      "js-refactor",
      "refactor"
    ],
    "destroyEmptyPanes": false,
    "themes": [
      "one-dark-ui",
      "monokai"
    ],
    "audioBeep": false
  }
}

Installed Packages
# User
atom-yeoman, v0.2.0
autoclose-html, v0.16.0
highlight-selected, v0.9.3
language-aspx, v0.4.0
language-csharp, v0.5.0
language-cshtml, v0.1.1
language-powershell, v2.0.1
language-sln, v0.1.0
language-vbdotnet, v0.1.0
linter, v0.12.6
monokai, v0.14.0
omnisharp-atom, v0.6.2
tabs-to-spaces, v0.10.0

# Dev
No dev packages

@david-driscoll this is an interesting issue. We can't deal with files that don't have a filename as we don't know which project/client to associate them with.

My gut feeling is that we should somehow force the user to select a file name for new files.

https://atom.io/packages/fancy-new-file seems like a good candidate for implementing new file behaviour.  When creating new files in both vim and emacs, the filename is specified up front.

Looks like it's complaining about this line: https://github.com/OmniSharp/omnisharp-atom/blob/v0.4.33/lib/omni-sharp-server/client-manager.ts#L109
I just tried to get this to happen locally, no dice (And I already had markdown preview installed!).  There have been a bunch of new guards added in the latest version, @hmemcpy would you have a chance to try it again with the latest and let us know if it's still happening for you?


Further Simplification API

Summary
Further simplify the API with a single allowable POST method. This method only allows a string and returns an object with a positiveRating, negativeRating, and neutralRating.
Closes #47
Verification

[ ] Read through title and description of POST request
[ ] Confirm string parameter is required in body of request
[ ] Confirm returned parameters include positiveRating, negativeRating, and neutralRating
[ ] Validate that AI response can be obtained from available methods

Effort

Task
Time Est (hrs)
Time Actual (hrs)
Description

Review Feedback
0.25
0.5
Review feedback obtained in class and review NLTK feedback

Write Update
0.5
0.5
Update API documentation to match desired functionality

Documentation
0.25
0.25
Update documentation for pull request

Time Est (hrs): 1
Time Actual (hrs): 1.25

Add data model for feedback and text - should be submitted as a json object.

I looked at the code and I did not see any post path.I saw only get path , did you mean your. lambda handle get request?.You maybe missing the word post , and I am not suer if it is important .

@TurkiBK I thought I wasn't seeing it initially, as well, but I clicked on the blue expand icon in the file, and it had been condensed. It is in fact there!


fixes outputs

return [] so we can easily test in an workflow step
if: ${{ steps.promote.outputs.images-updated != '[]' }}

updates jq statement so that the file is json parsable, allowing us to use fromJson in workflows

@whereismyjetpack if you can fix the linting, i'll get approved


[c#] Swap in a custom list for SynchronizedCollection

SynchronizedCollection is not present in .NET Standard 1.6, among others. It
was only being used to demonstrate how collection type aliasing works, so it
has been replaced with SomeCustomList which just wraps List to
demonstrate how this would work.

Rebased atop latest Mono fixes. Re-running CI.

The CI failures seem to be real failures related to this change.

Yup. I broke something somewhere during the rebase. Looking...

Fixed it for .NET. Letting CI run to check Mono.
Also opened "Custom collections need to implement ICollection and not just ICollection" to fix a discrepancy between the documentation and the implementation around custom containers.

AppVeyor failure is a C++ timeout. PR #256 should help with these. Going ahead with the merge.


Map has stopped updating

As can be seen in the image below, the map has not updated for the past two days.
This could be related to commit https://github.com/IBMIXN/EmotionalMapProject/commit/e897d227fd3f7d649628e919410ad4fb5b14342f.

This happened because I did not check the code to ensure it worked how I wanted it to. I've created a PR to help others avoid the same mistake: https://github.com/node-schedule/node-schedule/pull/598


Undefined index: ELEMENT when using findElement() functions

What are you trying to achieve? (Expected behavior)

I'm trying to $driver->findElement(), findElements(), or RemoteWebElement->findElement().
All throw an exception
What do you get instead? (Actual behavior)
I get an error "Undefined index: ELEMENT" on lines that use $raw_element['ELEMENT']
var_dump($raw_element) outputs the following structure
array(1) {
  'element-6066-11e4-a52e-4f735466cecf' =>
  string(36) "1ba86d2e-0292-9242-b5b3-d90149516cef"
}

to fix this I temporary replaced all usages of $raw_element['ELEMENT'] with \array_values($raw_element)[0] and findElement() works now for me.
Details

Php-webdriver version: 1.7.1
PHP version: 7.3
Selenium server version: 3.141.59
Operating system: Mac OS X 10.14.6
Browser used + version: Firefox 70.0.1

See #469 - this fix is merged to community branch and will be part of release 1.8.0.
For now you can use dev version of the branch, see https://github.com/facebook/php-webdriver/issues/469#issuecomment-554979855


[arrayfire] New Port

Adds a new port for ArrayFire. Ports all backends -- CUDA, OpenCL, and CPU.
cc @9prady9 @umar456 to join the discussion here.
cc @padentomasello, @tlikhomanenko, @vineelpratap, @syhw, @andresy
Some things to note:

Depends on CUDA, cuDNN (soft) and MKL -- uses the corresponding vcpkg ports. Not sure how this affects CI here.
ArrayFire relies on several submodules.
a) Some of these (e.g. those for testing) aren't required and aren't present in this port.
b) Most of the submodules that are required don't have ports and probably won't (these are ArrayFire specific projects).
c) Some things do have ports, e.g. NVIDIA Cub. The solution here is probably to ultimately change the ArrayFire build to look for Cub on the existing system rather than downloading it to prevent potential conflicts with other downstream dependencies that require Cub, although in my experience, this hasn't been an issue.
d) As a stopgap measure to solve these issues, the vcpkg config uses vcpkg_from_github to clone submodules as needed them moves them into their expected directories via CMake so that they can be built from source normally with calls to add_subdirectory from ArrayFire CMake things.
This is a port of 3.7.1, a recent stable release. We can update the port to 3.7.3/3.8 once those releases are out.
Backends are features, as in ./vcpkg install arrayfire[cuda]. Multiple backends/features can be installed at once.
Only installs libafcuda, libafcpu, or libafopencl, as specified. Doesn't install examples, tests, or other things, although this might be useful down the road.
Doesn't build with forge/AF graphics features. This is probably something that can be added or changed later given that most of the needed ports are here and the forge submodule is manually cloned and moved to the right place.

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

Have tested x64-linux. Might want some help testing the others.

Does your PR follow the maintainer guide?

Yes

@jacobkahn, could you format vcpkg.json file firstly?
See github.com/microsoft/vcpkg/blob/master/docs/maintainers/maintainer-guide.md#manifest for solution.

@JackBoosY @PhoebeHui — I've removed all of the submodule dependencies as mentioned (besides of arrayfire/threads, which is a standalone library with dependencies and has no vcpkg port).
A patch is needed for now to update the build config to search for these libraries rather than relying on their existing as submodules. Hopefully there can be more flexibility with the build in future ArrayFire releases such that we don't need the patch, cc @umar456 @9prady9.

@jacobkahn, here is CI test failure:
x64-windows and x64-windows-static:
-- Performing post-build validation
The following cmake files were found outside /share/arrayfire. Please place cmake files in /share/arrayfire.

    D:/packages/arrayfire_x64-windows/cmake/ArrayFireConfig.cmake
    D:/packages/arrayfire_x64-windows/cmake/ArrayFireConfigVersion.cmake
    D:/packages/arrayfire_x64-windows/cmake/ArrayFireUnifiedTargets-release.cmake
    D:/packages/arrayfire_x64-windows/cmake/ArrayFireUnifiedTargets.cmake
    D:/packages/arrayfire_x64-windows/debug/cmake/ArrayFireConfig.cmake
    D:/packages/arrayfire_x64-windows/debug/cmake/ArrayFireConfigVersion.cmake
    D:/packages/arrayfire_x64-windows/debug/cmake/ArrayFireUnifiedTargets-debug.cmake
    D:/packages/arrayfire_x64-windows/debug/cmake/ArrayFireUnifiedTargets.cmake

The following dlls were found in /lib or /debug/lib. Please move them to /bin or /debug/bin, respectively.
    D:/packages/arrayfire_x64-windows/lib/af.dll
The following dlls were found in /lib or /debug/lib. Please move them to /bin or /debug/bin, respectively.
    D:/packages/arrayfire_x64-windows/debug/lib/af.dll
Found 3 error(s). Please correct the portfile:
    C:\a\1\s\ports\arrayfire\portfile.cmake

x64-osx:
In file included from /Users/vagrant/Data/installed/x64-osx/include/boost/stacktrace/safe_dump_to.hpp:217:
/Users/vagrant/Data/installed/x64-osx/include/boost/stacktrace/detail/collect_unwind.ipp:33:2: error: "Boost.Stacktrace requires `_Unwind_Backtrace` function. Define `_GNU_SOURCE` macro or `BOOST_STACKTRACE_GNU_SOURCE_NOT_REQUIRED` if _Unwind_Backtrace is available without `_GNU_SOURCE`."
#error "Boost.Stacktrace requires `_Unwind_Backtrace` function. Define `_GNU_SOURCE` macro or `BOOST_STACKTRACE_GNU_SOURCE_NOT_REQUIRED` if _Unwind_Backtrace is available without `_GNU_SOURCE`."
 ^
1 error generated.

failure logs for x64-osx (13).zip

For the post build check failrues, you can use the vcpkg_fixup_cmake_targets(CONFIG_PATH cmake) to fix the cmake targets issue.
For the af.dll issue, you need to patch the CMakeLists file to install the

The latest CI test failures:
x64-windows: you need to check why it doesn't generate for debug configuration.
-- Performing post-build validation
Mismatching number of debug and release binaries. Found 0 for debug but 1 for release.
Debug binaries
Release binaries
    D:/packages/arrayfire_x64-windows/bin/af.dll
Debug binaries were not found
Found 1 error(s). Please correct the portfile

x64-windows-static: you need to check why it generated in a static build, and patch the CMakeLists.txt file to fix this issue, rather that removing the bin folder.
DLLs should not be present in a static build, but the following DLLs were found:
    D:/packages/arrayfire_x64-windows-static/bin/af.dll

There should be no bin\ directory in a static build, but D:\packages\arrayfire_x64-windows-static\bin is present.
If the creation of bin\ and/or debug\bin\ cannot be disabled, use this in the portfile to remove them
    if(VCPKG_LIBRARY_LINKAGE STREQUAL "static")
        file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/bin" "${CURRENT_PACKAGES_DIR}/debug/bin")
    endif()
Found 2 error(s). Please correct the portfile

@PhoebeHui @JackBoosY — tests are passing - I think this is ready for a final review.

@PhoebeHui @JackBoosY bump

@PhoebeHui @JackBoosY bump

Please let us know if there is anything else that needs to be addressed here. Thank you.

Since your changes is huge, does the official accept them?

Since your changes is huge, does the official accept them?

Do you mean does arrayfire approve of the patch that is submitted in this port ?

@BillyONeal, the arrayfire port haven't been test in CI system from build https://dev.azure.com/vcpkg/public/_build/results?buildId=44791&view=results, could you help have a look?
arrayfire:x64-windows:    cascade: 5cb12df794ddbae1efd99ab6fb0b459048019f8d
I checked the arrayfire dependency port, all are passed. I'm not sure why it cascade.

Since your changes is huge, does the official accept them?

Do you mean does arrayfire approve of the patch that is submitted in this port ?

I saw these changes have nothing to do with implementation, so ignore my comment. That's okay.

@jacobkahn, hello,
I tested the features of jacobkahn [cpu], jacobkahn [cuda] and jacobkahn [opencl], and jacobkahn [unified], none of which can be built on Linux. Can you solve them? Below is their failure log：
-- Found Boost: F:/test7/vcpkg/installed/x64-windows/include (found version "1.73.0")
CMake Error at CMakeModules/InternalUtils.cmake:10 (message):
  FFTW not found
Call Stack (most recent call first):
  src/backend/cpu/CMakeLists.txt:312 (dependency_check)

CMake Error at CMakeModules/InternalUtils.cmake:10 (message):
  CBLAS not found
Call Stack (most recent call first):
  src/backend/cpu/CMakeLists.txt:313 (dependency_check)

-- No git. Setting hash to default
-- Configuring incomplete, errors occurred!
See also "F:/test7/vcpkg/buildtrees/arrayfire/x64-windows-rel/CMakeFiles/CMakeError.log".

F:/test7/vcpkg/buildtrees/arrayfire/x64-windows-rel/CMakeFiles/CMakeError.log:
Determining if the include file pthread.h exists failed with the following output:
Change Dir: F:/test7/vcpkg/buildtrees/arrayfire/x64-windows-rel/CMakeFiles/CMakeTmp

Run Build Command(s):F:/test7/vcpkg/downloads/tools/ninja/1.10.1-windows/ninja.exe cmTC_d3a8a && [1/2] Building C object CMakeFiles\cmTC_d3a8a.dir\CheckIncludeFile.c.obj

FAILED: CMakeFiles/cmTC_d3a8a.dir/CheckIncludeFile.c.obj

C:\PROGRA~2\MICROS~1\2019\PROFES~1\VC\Tools\MSVC\1424~1.283\bin\Hostx64\x64\cl.exe  /nologo   /nologo /DWIN32 /D_WINDOWS /W3 /utf-8 /MP   /MDd /Zi /Ob0 /Od /RTC1 /showIncludes /FoCMakeFiles\cmTC_d3a8a.dir\CheckIncludeFile.c.obj /FdCMakeFiles\cmTC_d3a8a.dir\ /FS -c CheckIncludeFile.c

CheckIncludeFile.c(1): fatal error C1083: Cannot open include file: 'pthread.h': No such file or directory
ninja: build stopped: subcommand failed.

Hi @jacobkahn, I tested the features of jacobkahn [cpu], jacobkahn [cuda] and jacobkahn [opencl], and jacobkahn [unified], none of which can be built on Windows. They all failed with same issue, could you please take a look? Below is their failure log：
-- Found Boost: F:/test7/vcpkg/installed/x64-windows/include (found version "1.73.0")
CMake Error at CMakeModules/InternalUtils.cmake:10 (message):
  FFTW not found
Call Stack (most recent call first):
  src/backend/cpu/CMakeLists.txt:312 (dependency_check)

CMake Error at CMakeModules/InternalUtils.cmake:10 (message):
  CBLAS not found
Call Stack (most recent call first):
  src/backend/cpu/CMakeLists.txt:313 (dependency_check)

-- No git. Setting hash to default
-- Configuring incomplete, errors occurred!
See also "F:/test7/vcpkg/buildtrees/arrayfire/x64-windows-rel/CMakeFiles/CMakeError.log".

config-x64-windows-out.log

hi @JonLiu1993  Is this with Intel-MKL installed on Windows ?

Hi @9prady9 ,Yes, I have installed Intel-MKL on windows

Hi @9prady9 ,Yes, I have installed Intel-MKL on windows

Ok, @jacobkahn Looks like the setting USE_CPU_MKL and USE_OPENCL_MKL to ON on relevant features on vcpkg port is missing. Can you please add those.
@JonLiu1993 I don't quite yet understand why CUDA would fail. The dependency FFTW is looked for but only used/checked for in CPU backend builds. Does trying to install arrayfire[cuda] install default features cpu | unified too ?

@9prady9, yes, it would install the default feature 'cpu' and 'unified'.

@9prady9, yes, it would install the default feature 'cpu' and 'unified'.

Ok that makes sense. Once @jacobkahn adds the missing USE* flags in, this error should go away.

@JonLiu1993 — I've added those flags, cc @9prady9. Thanks both of you for catching that. Can you try things again @JonLiu1993 so we can see if we still have an FFTW issue?

@jacobkahn , I tested the default and other functions of arrayfire again, but it still failed. Below is the failure log. could you please take a look?
install-x64-windows-dbg-out.log

@JonLiu1993 this looks like events.hpp in arrayfire/threads was missing a #pragma once in the submodule commit for which 3.7.1 is compiled which caused those definitions to be duplicated.
I added a patch to fix it. I unfortunately don't have an x64 Windows env with which to test the fix - please let me know if there are any errors! Thanks again for helping test.

@jacobkahn ,I tested it again and it still seems that there are some problems, can you take a look?
install-x64-windows-dbg-out.log

@JonLiu1993 — just added a fix for that issue — a header in the threads library was conflicting with a Windows std C++ header. Reverted part of the patch (which removes a necessary infix) to resolve it. Thanks to @9prady9 for helping me debug.
Can you try once more? Thanks!

@jacobkahn ,I tested it again, and now the DEBUG version can be compiled, but an error occurred when compiling the RELEASE version. I checked the log and found that some problems occurred during the installation of RELEASE. Some dlls of RELEASE were installed. In the debug directory, this problem exists in every feature of arrayfire. Hope this discovery can help you solve the problem
Mismatching number of debug and release binaries. Found 3 for debug but 0 for release.
Debug binaries

    F:/test7/vcpkg/packages/arrayfire_x64-windows/debug/bin/af.dll
    F:/test7/vcpkg/packages/arrayfire_x64-windows/debug/bin/afcpu.dll
    F:/test7/vcpkg/packages/arrayfire_x64-windows/debug/bin/afopencl.dll

Release binaries

Release binaries were not found

Found 1 error(s). Please correct the portfile:
    F:\test7\vcpkg\ports\arrayfire\portfile.cmake
-- Performing post-build validation done
Error: Building package arrayfire:x64-windows failed with: POST_BUILD_CHECKS_FAILED

install-x64-windows-rel-out.log

@JonLiu1993 Any idea why would VCPKG_BUILD_TYPE be empty when vcpkg configures the project ? At which point is this variable set in the workflow so that I can use it appropriately ?
The reason I ask this is, although @jacobkahn is checking this (as shown below) in the portfile
# bin/dll directory for Windows non-static builds for the unified backend dll
if(VCPKG_TARGET_IS_WINDOWS AND NOT VCPKG_LIBRARY_LINKAGE STREQUAL "static")
  if (NOT DEFINED VCPKG_BUILD_TYPE OR VCPKG_BUILD_TYPE STREQUAL "release")
    set(AF_BIN_DIR ${CURRENT_PACKAGES_DIR}/bin)
  endif()
  if (NOT DEFINED VCPKG_BUILD_TYPE OR VCPKG_BUILD_TYPE STREQUAL "debug")
    set(AF_BIN_DIR ${CURRENT_PACKAGES_DIR}/debug/bin)
  endif()
  file(MAKE_DIRECTORY ${AF_BIN_DIR})
  set(AF_DEFAULT_VCPKG_CMAKE_FLAGS ${AF_DEFAULT_VCPKG_CMAKE_FLAGS} -DAF_INSTALL_BIN_DIR:PATH=${AF_BIN_DIR})
endif()

when I print the value of VCPKG_BUILD_TYPE, it is coming out as empty on command line output when the install command runs. Hence, the error of both debug and release dlls being copied over into same folder. I confirmed this from the value of AF_INSTALL_BIN_DIR in the CMakeCache.txt from debug and release config folders.

@JonLiu1993 Any idea why would VCPKG_BUILD_TYPE be empty when vcpkg configures the project ? At which point is this variable set in the workflow so that I can use it appropriately ?

That is set by a triplet, if it wants to. Most of the time it will be empty, which means a port needs to mint both release and debug bits. Note examples which all do:
if (NOT VCPKG_BUILD_TYPE OR VCPKG_BUILD_TYPE STREQUAL "release")
# release bits
endif()
if (NOT VCPKG_BUILD_TYPE OR VCPKG_BUILD_TYPE STREQUAL "debug")
#debug bits
endif()

@BillyONeal I have seen the other port files that are using similar pattern. Hence, my confusion on what is @jacobkahn missing in the code that isn't setting the build configuration. I don't quite understand how triplet will indicate build configuration/type ? vcpkg install arrayfire:x64-windows command doesn't have a build configuration, but both debug and release are built by that command. So, I would expect the build type to be set appropriately when that particular configuration is built. Is there something in the port file code that one needs to add to ensure that ?

Hence, my confusion on what is @jacobkahn missing in the code that isn't setting the build configuration.

The default is "not set", meaning a port must produce both release and debug bits. The option can only be set by a triplet author. None of our in-box triplets set it.

Hence, my confusion on what is @jacobkahn missing in the code that isn't setting the build configuration.

The default is "not set", meaning a port must produce both release and debug bits. The option can only be set by a triplet author. None of our in-box triplets set it.

what is the suggested work around for the problem we are having here where the INSTALL cmake variable is being set to same path in both debug and release configurations because build type is not set ?

@JonLiu1993 — I've just sent what I think should be a catch-all fix for all of the DLLs generated by the build. Sorry this took so long - it's extremely slow to experiment and iterate in my environment. Things work with x64-windows for me — hopefully they do for you too now and we can get this merged!
@BillyONeal I think we were getting hung up on a weird Ninja multiconfig scenario where cache variables weren't being changed across configs - either way, a patch was required to get the behavior we need. Thanks for those pointers.
That said, the idea that a DLL path would have to be determined via a CMAKE_BUILD_TYPE check inside a project's CMake seems odd to me, so I'd be more interested in seeing what other projects do to resolve this. It seems like most projects have a bunch of other cruft that move DLLs around (or remove them entirely) because of how VCPKG_BUILD_TYPE might not be defined and a path can't be reliably passed. If you have examples of projects that do that in the recommended way, that would be helpful to understand for future iterations of this port.

if NOT defined VCPKG_BUILD_TYPE is causing the second if to become true always irrespective of what config is being built.

portfile.cmake does not have a config; it is only built once. portfile.cmake is obligated to invoke the library in question's build system such that it produces both release and debug bits. For example, vcpkg_configure_cmake invokes a port's cmake-based build system twice; once for release, and once for debug: https://github.com/microsoft/vcpkg/blob/10c38371cb264932530b7e175b34712f48c15264/scripts/cmake/vcpkg_configure_cmake.cmake#L299-L307

@BillyONeal I think we were getting hung up on a weird Ninja multiconfig scenario where cache variables weren't being changed across configs - either way, a patch was required to get the behavior we need.

vcpkg doesn't inject any ninja multiconfig stuff.

the idea that a DLL path would have to be determined via a CMAKE_BUILD_TYPE check inside a project's CMake seems odd to me, so I'd be more interested in seeing what other projects do to resolve this.

The port builds as normal and uses install( to install the resulting binaries where that's been configured. vcpkg passes CMAKE_INSTALL_PREFIX to make sure install( drops the bits in the right place:
https://github.com/microsoft/vcpkg/blob/6ba00186b11754db7a9a83a0e53cd921dfbd5ca8/scripts/cmake/vcpkg_configure_cmake.cmake#L268-L277

@BillyONeal thanks for taking the time to write all that up. That's really helpful.

The port builds as normal and uses install( to install the resulting binaries where that's been configured. vcpkg passes CMAKE_INSTALL_PREFIX to make sure install( drops the bits in the right place:

I see that the prefix has the debug infix... so it should have been prepended to RUNTIME DESTINATION for the DLLs dropped into place by install(. I suppose I still have absolutely no idea why that didn't work - using CMAKE_BUILD_TYPE as a proxy for whether or not to add debug to the path appears necessary on x64-windows with my particular config. Probably something wrong with my environment or the project's config, but I have no clue. Seems like that repro for @JonLiu1993 as well. In any event, this works for now...

@jacobkahn ,I just tested the default arrayfire and its features. Except for the cuda features, everything else is successful. The version of cuda installed in my test environment is v11，The attachment is its complete log，could you please take a look?
nvcc fatal   : Unsupported gpu architecture 'compute_30'
CMake Error at cuda_compile_4_generated_thrust_sort_by_key_impl_double_1.cu.obj.Debug.cmake:220 (message):
  Error generating

install-x64-windows-dbg-out.log

@JonLiu1993 — good catch and thanks for testing - just patched the build to work for CUDA 11. A future version of this port with a newer ArrayFire version will have this updated code and won't require a patch for those nvcc flags, but this should work fine for now.
One more try would be great (I don't have a GPU accessible from my Windows VM). Thanks again!

@jacobkahn ,I just re-tested the cuda feature and it failed. The problem seems to have not been resolved through the log. The following is the complete log. Please take a look
install-x64-windows-dbg-out.log

@JonLiu1993 fixed. Also reprod the issue - pushed a fix I've confirmed works on Linux. Can you give it one last try?

@JonLiu1993 — turns out the ArrayFire commit the port is checked out too doesn't work too well with CUDA 11. I'm going to try a different commit and re-work the patch. I'll push that up and let you know when it's done.

@JonLiu1993 — just updated the port's commit. Tested with CUDA 11 on Linux and everything builds. Let's test on Windows once more.

@jacobkahn ,I just updated and tested the latest submission. It seems that there are still some problems. Can you take a look?
install-x64-windows-dbg-out.log

@JonLiu1993 turns out there are difficulties compiling with MSVC 2019, but not with 2017, cc @9prady9 @umar456. I think a fix for these is in the works, but may not be ready for some time, and we'd like to land the port as is. Is it possible to detect the MSVC compiler version in the port file with some vcpkg variable? cc @PhoebeHui @JackBoosY

Waiting is fine, but is it possible to detect the MSVC compiler version in the port file with some vcpkg variable?

Not with a vcpkg variable but the library's build system has that ability.

@JonLiu1993 — everything should work with MSVC 2019 now as per those fixes from @9prady9 @umar456. Can you test one more time on your machine? I've re-tested on Linux, and things look good.

@jacobkahn, the libarchive issue has been fixed, it doesn't relate to this change, please ignore it, I have rerun the CI testing.

@PhoebeHui Thanks. We'll wait for one more test from @JonLiu1993.

@jacobkahn ,I tested the default triplet of arraryfire and x64-windows of all features, they all built successfully. Good job！

Looks like the x64_osx is stuck @PhoebeHui — once that's fixed, I think we're ready to merge?

@jacobkahn, I checked, the CI testing with x64-osx is in progress. Let's wait for the results.

@PhoebeHui it looks like the macOS CI is stuck again...

/azp run

Thanks all for making this possible!
@strega-nil, could you please help review and merge this PR?
All features test passed with x64-windows, tested by @JonLiu1993
All features test passed with x64-linux, tested by @jacobkahn

@jacobkahn could you respond to whether the endif();if() pair should be an elseif()? then I'll merge :)

@strega-nil — just responded. Thanks for the review and those fixes!

@strega-nil — it looks like the elseif change means one other tweak is necessary (need to update the patch) — sending a fix in a moment.

@jacobkahn alright, cool, thanks!

/azp run

Alright, this looks great :) Thanks @jacobkahn :)

when i install arrayfire with vcpkg ;
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:145 (message):
Command failed: vcpkg/downloads/tools/cmake-3.18.4-linux/cmake-3.18.4-Linux-x86_64/bin/cmake --build . --config Debug --target install -- -v -j41
Working Directory:cpkg/buildtrees/arrayfire/x64-linux-dbg
See logs for more information:
vcpkg/buildtrees/arrayfire/install-x64-linux-dbg-out.log
i check the install-x64-linux-dbg-out.log ，it appear "ninja: build stopped: subcommand failed" end of this file

@jinggaizi Can you open a new issue to report that?
Note you also should provide the full error logs.
Thanks.

@jinggaizi Can you open a new issue to report that?
Note you also should provide the full error logs.
Thanks.

thanks for your reply , i use gcc version 6.3.1 and fix the problem


Builtin signature check?

Currently cask seems to do its own hash check, and it is not available for all packages. However, most applications for OS X already come with digital signatures, signed by a developer certificate that is in turn signed by Apple. For pkg files, we can verify with pkgutil --check-signature. For .app bundles, we can verify with codesign -v. Will homebrew cask support these builtin security checks in the future?

There are no plans for it. There are so many more things that need to be done first, and we have such a lack of contributions to the core, it’s unlikely to be done soon.
I’m not opposed to it in the slightest, but we need to keep in mind that some apps are not signed for legitimate reasons (like not wanting to pay to do it).

Seems like there isn’t much interest in this for now. With so much more stuff with higher priorities, I’ll close it. We can always revisit this later.


Project setup

In this pull request, I have implemented the following changes:

[x] Install rails app with webpack
[x] Install required gems.
[x] Install required node packages
[x] Create database & greetings table
[x] Add controller methods to render random messages from the database & respond in JSON
[x] Add react components to display messages
[x] Add professional readme.md

@Omar-Muhamad and @OyePriscilla, Thanks for your kind review and feedback.


Correcting Windows virtual-key code translation

Kmonad uses a map to associate Windows virtual-key codes with KMonad KeyCodes. When translating the other direction, the logic would flip the entries of this map and go from KeyCode to virtual-key code. However, where more than one virtual-key code mapped to the same KMonad KeyCode, this created a conflict where only one would be able to be sent to the sink, and often not the expected code.
This commit fixes that issue by maintaining two separate maps, one for each direction. In this way, multiple virtual-key codes may be interpreted as the same KMonad KeyCode, and multiple KeyCodes may be sent as the same virtual-key code as necessary.
This is a fix for #326.
Previously requested in #350. I switched branches so I had to open a new PR. I've updated based on the previous comments, and at least one other user has confirmed that this fixes the issue.

Ah, I must have missed @Veanvi confirming that this works for them.  In that case I don't see a reason to hold out any longer; thanks!


CAMEL-18912: Reuse client from consumer

Before this change, autowired clients would fail to be detected.
Also small optimization added by only requesting a single queue when listing the queues.
Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[x] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I formatted the code using mvn -Pformat,fastinstall install && mvn -Psourcecheck

@simon-ras I wonder if you have time to check the other health checks in the AWS as they may also benefit from reusing client

I can give it a shot - though it might take a few days due to limited spare time.

Also, what is the best procedure for backporting this - should I make available PR with cherry-picked commit for each version, or better let you take care of this when merging the main PR?

1 per component as if you do not get time to complete them all then others can take over later

we will backport

Don't worry about the backport operations, we'll do it. Thanks for your time and for reporting.

Forgot to format prior to pushing. Will reformat and push again.

Yes, for all health checks. There is still a few components which does not have health checks, they are listed here https://issues.apache.org/jira/browse/CAMEL-18131. They continue to use ComponentVerifierExtension.
I just discovered that there are some test which fail though, so this PR is not quite ready yet...

I believe AWS2S3ConsumerHealthCustomClientTest - which is the failing test - is invalid. These tests all use the client which is set on the configuration and since the healthcheck now uses the client of the endpoint, then such case cannot be tested.
Do you mind @oscerd do you agree?

It's important to make it works in all the cases. I think your fix will work only in case of client set on endpoint.. what happens if I specify keys secret and access? It should work too..

Ah right. You're right Simon.

@simon-ras can we cherry pick this commit to camel-3.x or would you be able to do another PR for that branch so we get this into 3.x releases too

@simon-ras can we cherry pick this commit to camel-3.x or would you be able to do another PR for that branch so we get this into 3.x releases too

Sure, please see https://github.com/apache/camel/pull/9105


horovod in powerai 1.6.2,  error occurs with hvd.init()

Hi,
Try to install Horovod with PyTorch based on powerai image following  official tutorial, error occurs when try to hvd.init(). Is there any material on how to set up DDL_OPTIONS to use ddl? Thanks

code

import horovod.torch as hvd
hvd.init()
[2020-06-11 03:01:22.297540: W horovod/common/ops/ddl_operations.cc:105] DDL backend has been deprecated. Please, start using the NCCL backend by building Horovod with 'HOROVOD_GPU_ALLREDUCE=NCCL HOROVOD_GPU_BROADCAST=NCCL'.
terminate called after throwing an instance of 'std::logic_error'
what():  DDL_OPTIONS env variable needs to be set to use DDL.
Aborted (core dumped)`

I also tried to set up env variable for DDL_OPTIONS with export DDL_OPTIONS=DDL it produce another error
`

import horovod.torch as hvd
hvd.init()
[2020-06-11 03:14:33.814715: W horovod/common/ops/ddl_operations.cc:105] DDL backend has been deprecated. Please, start using the NCCL backend by building Horovod with 'HOROVOD_GPU_ALLREDUCE=NCCL HOROVOD_GPU_BROADCAST=NCCL'.
E 03:14:33.814   634 652 DDL:86  ] ill-formated option :[DDL]
terminate called after throwing an instance of 'std::system_error'
what():  Resource deadlock avoided
Aborted (core dumped)
`

You will need to launch the job using the ddlrun tool. This will setup the appropriate environment variables. For more information on ddlrun, see https://www.ibm.com/support/knowledgecenter/SS5SF7_1.6.2/navigation/wmlce_ddlrun.html.

@bnemanich thanks very much. Does that mean I can use horovod in script, and use ddlrun to run?


set firewalld default zone fails when default_zone not set

set firewalld default zone fails when default_zone not set because of the conditional:
when: defaultzone.stdout != default_zone
I suggest removing the conditional altogether, as well as task "get actual firewalld default zone", since removing these would produce the same effect:

If default zone isn't 'public' it would be changed to public;
If default zone is 'public', it won't do anything.

@goniomdq Thank you for this issue,I'll check and fix it.


Folders are not deletable if vowel mutations are in the name

From @Steffov on February 18, 2015 22:47
I notices that it's not possible to delete a folder, which is created under files, when they contains vowel mutations like ü,ä or ö. The vowel mutations are not displayed too. I was able to delete the folder on the site, after changing the ü to "ue" directly in the database.
Copied from original issue: friendica/red#903

From @Steffov on March 29, 2015 9:18

Did you create this folder with mutated vowels in the browser or did you use a WebDAV client?

Yes, I create the folder in the browser. Today I tried this again. The result was the same.
Here are the steps to reproduce:

Select Files (cloud)
Create new folder
Name the folder: Übersicht

Effects:

Try to open this new Folder
It will apear a xml error like this:
<d:error><s:exception>Sabre\DAV\Exception\NotFound</s:exception><s:message>The file with name: bersicht could not be found.</s:message><s:sabredav-version>1.8.10</s:sabredav-version></d:error>
Folder isn't deletable. In additional the folder name isn't correct.
It's "bersicht". The correct spelling would be "Übersicht".

You have other problems with ä,ö,ü in your hub, or only with folder in files?

By now I couldn't find any other issues like that in my hub.

From @dawnbreak on March 7, 2015 23:26
Could not reproduce this behaviour. Did you create this folder with mutated vowels in the browser or did you use a WebDAV client? Can you open such folder? You have other problems with ä,ö,ü in your hub, or only with folder in files?

From @friendica on March 30, 2015 4:28
Able to duplicate exactly as described. Did find an issue creating directories in general which has now been fixed. This remains elusive. Trace statements show that we do hand the correct name to the Sabre library. It comes back mangled. It also seems to be dependent on the position of the accented character in the string. Creating blÜbersicht (probably not a real word, but I was testing this hypothesis) worked fine and was selectable. Übersicht was chopped at the first char. Trace statements inside the Sabre code haven't yet provided a clue where it is getting mangled.

@Steffov This issue is closed because Redmatrix evolved into Hubzilla. When this is still an issue in the most recent version of Hubzilla, please recreate this issue over there. We're sorry for any inconvenience this will maybe give you.


Integral disappear when you scale too much down

If you scale too much down at the end there should be a straight line

@lpatiny
it's not a bug because the scale of integral become zero.

Still the integral should stay visible. Please try to find a way so that it does not disappear.


#186 - support ACL perms via tags

#186  - this is required when supporting acl controls that dont occur on the path. For example .some.domain._public.events can then be expressed as
.some.domain.events
and the _public grant is configured in the spec->channels->channel->public[tags -> grant-access: _public]

lgtm


Provide completion for odo catalog describe service

[kind/Enhancement]
Which functionality do you think we should update/improve?
See $SUBJECT
Why is this needed?
Most other commands provide completion,  but this one doesn't see to

We can add this for consideration in sprint 163 @geoand


Logo briefly jumps when opening any page from the home page

I enabled Jamfin CSS with all modules except change-logo.css, so I use the default Jellyfin logo.
On the home page, if I try to click on any item (e.g. a media library), the Jellyfin logo in the top-left corner of the screen briefly jumps to the right during the transition, before quickly vanishing. This looks ugly and distracting.
Tested on Jellyfin's native Linux client.

I'm unable to replicate this on either JMP or the browser.

Further testing shows this is happening on my machine even with the Jellyfin default CSS, so this isn't a Jamfin issue after all.


Split person/entity field sets more practically

Spawned from https://github.com/citation-file-format/citation-file-format/pull/155#issuecomment-842456834.
Currently, fields that are used in both person and entity definitions - a lot of them simple strings - each have their own definition, which isn't very nice and unnecessarily blows up the file.
Instead, we could group them into three sets, e.g., uniqueToEntity, uniqueToPerson, and shared (refs #103), and then the definition could be
"entity": {
    "allOf": [
        {"$ref": "#/definitions/uniqueToEntity"},
        {"$ref": "#/definitions/shared"}
    ]
},
"person": {
    "allOf": [
        {"$ref": "#/definitions/uniqueToPerson"},
        {"$ref": "#/definitions/shared"}
    ]
}

In this example, shared was the original suggestion in the comment, although I'd like to make it more semantically unambiguous by naming the definition personOrEntity or similar.

So, this won't work with draft 7, as allOf cannot be used for real inheritance in combination with "additionalProperties": false and/or "required": [].
The only way one can make this work is by introducing empty property entries for all properties in both person and entity, which sort of defies the reason for splitting those up in the first place.
I'll have a look at using the if-else syntax to make this work in a sensible manner, but am not hopeful.

@jspaaks If you know of or can find another way of implementing this, please let me know. Otherwise I'd vote to close this.


[Tosa] Add support for legalizing scatter_nd

This commit legalizes tf(l).scatter_nd to tosa.scatter in a similar approach to the existing gather_nd support. Specifically, inputs are rewritten to the expected formats by tosa.scatter and finally the output of tosa.scatter is reshaped to the expected output of scatter_nd.
tosa.scatter does not support duplicated indices while TF does. Therefore, we restrict legalization of scatter_nd to a constant indices tensor only and check that the provided indices are unique.

Hello @rdzhabarov , Can you please review this PR? Thank you !

Hi @rdzhabarov , Can you please review this PR? Thank you !

Hi @rdzhabarov , Can you please review this PR? Thank you !

Hi @rdzhabarov , Can you please review this PR? Thank you !

Hi @rdzhabarov , Can you please review this PR? Thank you !

Hi @rdzhabarov , Can you please review this PR? Thank you !

Hi @rdzhabarov , Can you please review this PR? Thank you !

Hi @rdzhabarov , Can you please review this PR? Thank you !

Hi @rdzhabarov , Can you please review this PR? Thank you !


document about jsonschema for dvc.yaml

I'll add a small note later on mentioning that jsonschema is available for dvc.yaml, after 1.0 docs is complete. I'll assign this to myself.
Related: https://github.com/iterative/dvc/pull/4028, https://github.com/iterative/dvc/issues/4027

Cool, should we also publish the JSON schema as an independent file in dvc.org and link to it from our note?

@jorgeorpinel, I'm thinking of adding to the catalog of schemastore. This way, many ide will download that by automatically based on filepath match (eg: when you open dvc.yaml), and user won't need to configure anything at all.
Jetbrains supports this out of the box, VSCode and Atom needs yaml extension for this support (they support json already by default).
So, maybe, I will create a separate repo for the schema, where I'll add some tests and that will hopefully get mirrored to the schemastore.

Please consider https://pydantic-docs.helpmanual.io/

@shachibista, yes we are using pydantic. In fact, waiting for a release there.

@shachibista, this is still a WIP though. And, we have too many schemas already.


How add mixed effects to a cox model?

I want to add random effects to my Cox model formulation, and I have noticed that
cox1 = coxph(@formula(event ~ A + B + C + (1|D)), data)

produces the following error:

ERROR: MethodError: no method matching |(::Int64, ::String)
Closest candidates are:
|(::Any, ::Any, ::Any, ::Any...) at operators.jl:591
|(::T, ::T) where T<:Union{Int128, Int16, Int32, Int64, Int8, UInt128, UInt16, UInt32, UInt64, UInt8} at int.jl:365
|(::T, ::CEnum.Cenum{S}) where {T<:Integer, S<:Integer} at ~/.julia/packages/CEnum/Bqafi/src/operators.jl:13

but this syntax is allowed in @formula (i.e., the following works fine):
@formula(event ~ A + B + C + (1|D))

I assume mixed effects aren't supported in coxph?
For reference, random effects in survival analysis are often referred to as 'frailty terms' (in fact, I think in R they are added as frailty(D) as opposed to 1|D, but I'm not sure.
Is this something that is supported but I'm just doing it wrong?

further reference, in case it's helpful:
https://stats.oarc.ucla.edu/r/dae/mixed-effects-cox-regression/

Hi @casasgomezuribarri, apologies for the delayed response. Mixed effects are not currently supported by this package though it's something I'd like to incorporate eventually.


App bundle

Do we have to submit signed apk bundle??

@Amit588986 , Yes
Please let me know if you have any problem submitting the Apk Bundle

@Uyadav207 Nope


compilepkg: fix paths in compile errors

A previous change (18a0557abfe2e1029fa12e35a46e846eb2564292) resulted
in full paths being shown when there are compile errors. The paths
are in a temporary directory and they can be problematic with tools
that interpret the build output (e.g. vim's :make).
This change fixes this by switching the current working directory
before calling out to compile.
For example - before:
/tmp/rules_go_work-3549239678/github.com/cockroachdb/cockroach/pkg/sql/scan.go:91:20: undefined: descpbx

After:
github.com/cockroachdb/cockroach/pkg/sql/scan.go:91:20: undefined: descpbx

I don't know, it feels like I'm moving further in the wrong direction. Maybe there's a better way to solve #64379.. I'll see if I can try this idea: https://github.com/cockroachdb/cockroach/issues/64379#issuecomment-1012694957

Closing in favor of #3.


permission node for specific warp

Hey I need to know the permission node so that someone can use for example /warp plots

simplewarp.warp.plots


DSA_generate_parameters_ex: use the old method for all small keys

Fixes #14733

Still not sure we should be allowing people to keep generating weak keys..

Still not sure we should be allowing people to keep generating weak keys..

It can be useful to use openssl to test other systems in production.

you mean - other systems that are insecure :)

Still not sure we should be allowing people to keep generating weak keys..

This is the legacy API so I would not bother making it incompatible with old tests and similar stuff in this way.
But yeah in EVP API we should at some point stop allowing weak keys (at least via some configuration). IMO Post 3.0.

you mean - other systems that are insecure :)

How do you regression test that your firewall rejects weak keys unless you can generate such keys?

you mean - other systems that are insecure :)

How do you regression test that your firewall rejects weak keys unless you can generate such keys?

I would hack the code and remove the condition - generate some data and then use that data from then on for your testing.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master. Thank you for the review and other comments.


refactored environment variables

@nzikalala check out this nuget package which is already being used in the project Microsoft.Extensions.Configuration.UserSecrets.
Perhaps you can fix the secrets issue in this PR? This addition will remove the need for setting the environment on Windows, and only set them in user secrets in VS. README to be updated once this is tried and test on both server and locally.
Check this line here.
 // Retrieve the values from secure storage or environment variables
 ValidAudience = authSettings["Issuer"] ?? Environment.GetEnvironmentVariable("AuthManagement__Issuer"),

The server's sys. env vars still need to be set though. @EduanO