[SPARK-13314][SQL] Fixes malformed WholeStageCodegen tree string

When the query plan contains binary operator(s), WholeStageCodegen tree string can be malformed. This PR fixes this issue.
Before:
val a = sqlContext range 5
val b = sqlContext range 2
a select ('id as 'a) unionAll (b select ('id as 'a)) explain true
...
== Physical Plan ==
Union
:- WholeStageCodegen
:  :  +- Project [id#3L AS a#6L]
:  :     +- Range 0, 1, 8, 5, [id#3L]
+- WholeStageCodegen
   :  +- Project [id#4L AS a#7L]
   :     +- Range 0, 1, 8, 2, [id#4L]

After:
== Physical Plan ==
Union
:- WholeStageCodegen
:     +- Project [id#0L AS a#2L]
:        +- Range 0, 1, 8, 5, [id#0L]
+- WholeStageCodegen
      +- Project [id#1L AS a#3L]
         +- Range 0, 1, 8, 2, [id#1L]

cc @davies

Test build #51267 has started for PR 11200 at commit 79a4cb4.

Test build #51267 has finished for PR 11200 at commit 79a4cb4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51267/
Test PASSed.

Merged build finished. Test PASSed.

I think both are wrong. We want something like
== Physical Plan ==
Union
:- WholeStageCodegen
   :  +- Project [id#3L AS a#6L]
   :     +- Range 0, 1, 8, 5, [id#3L]
+- WholeStageCodegen
   :  +- Project [id#4L AS a#7L]
   :     +- Range 0, 1, 8, 2, [id#4L]

The ":" indicates the scope of whole stage codegen.

At least there should be a vertical tree line connecting two WholeStageCodegen nodes to indicate that they are both children of Union? The : in plan tree string used to indicate that.
Are you worrying about that users may mistake sub-trees under WholeStageCodegen for actual offspring nodes?

I'm really confused. what were we doing before whole-stage codegen?

Oh, I didn't realize that those extra vertical lines are used to indicate scopes of codegen'd stages, then the original tree string makes sense.  I thought they were dangling connecting lines.
Before whole stage codege:
Union
:- Project [id#3L AS a#6L]
:  +- Range 0, 1, 8, 5, [id#3L]
+- Project [id#4L AS a#7L]
   +- Range 0, 1, 8, 2, [id#4L]

After whole stage codege (before this PR):
Union
:- WholeStageCodegen
:  :  +- Project [id#3L AS a#6L]
:  :     +- Range 0, 1, 8, 5, [id#3L]
+- WholeStageCodegen
   :  +- Project [id#4L AS a#7L]
   :     +- Range 0, 1, 8, 2, [id#4L]

I'm closing this PR then.


AC fix

Changed every instance of "ac" to "AC" in conversationResponse.js so that the animation would occur when the user prompts for the AC.

not sure how old commits snuck onto this branch.  Spoke to Ashley and she is going to create a fresh PR


Intermittent Failure in Fetching Emails Using googleapiclient.discovery in Gmail API

We are experiencing intermittent failures when attempting to fetch emails using the googleapiclient.discovery library with the Gmail API. Despite the presence of the emails in the inbox that match the query criteria, the API sometimes fails to retrieve them.

Please note that this is an intermittent issue but recently I have been facing it a lot.
I have this code running on 2 different ec2 instances with 2 different email IDs and the both of them face the issue intermittently

Steps to reproduce

Use the googleapiclient.discovery library to fetch emails from the Gmail inbox.
Query for unread emails from an email address and subject criteria.
Observe the intermittent failures in fetching the emails

Version of libraries
Python 3.10.12

google-api-core==2.17.1
google-api-python-client==2.118.0
google-auth==2.28.0
google-auth-httplib2==0.2.0
google-auth-oauthlib==1.2.0
googleapis-common-protos==1.62.0

Code example
Here is the code I have been using for a year, and it fails sometime
from google.auth.transport.requests import Request
from google.oauth2.credentials import Credentials
from google_auth_oauthlib.flow import InstalledAppFlow
from googleapiclient.discovery import build

logger = logging.getLogger(__name__)
logger.setLevel(logging.INFO)

def get_gmail_service():
    """
    The file token.json stores the user's access and refresh tokens, and is
    created automatically when the authorization flow completes for the first time.
    """
    scopes = ['https://mail.google.com/']
    creds = None
    if os.path.exists('token.json'):
        creds = Credentials.from_authorized_user_file('token.json', scopes)
    # If there are no (valid) credentials available, let the user log in.
    if not creds or not creds.valid:
        if creds and creds.expired and creds.refresh_token:
            creds.refresh(Request())
        else:
            flow = InstalledAppFlow.from_client_secrets_file(
                'credentials.json', scopes)
            creds = flow.run_local_server(port=0)
        # Save the credentials for the next run
        with open('token.json', 'w') as token:
            token.write(creds.to_json())

    service = build('gmail', 'v1', credentials=creds, cache_discovery=False)
    return service

def delete_email():
    service = get_gmail_service()
    results = service.users().messages().list(
        userId='me',
        labelIds=['INBOX'],
<EMAIL_ADDRESS>subject:Transaction OTP"
    ).execute()
    message = results.get('messages', [])[0]

    if not message:
        logger.info('[delete_email] No messages from CDSL found to be deleted')
    else:
        logger.info("[delete_email] Found message from CDSL, deleting it...")
        service.users().messages().delete(userId='me', id=message['id']).execute()
        logger.info("[delete_email] Successfully delete message from CDSL")

def get_otp_from_email(tries=0) -> str:
    service = get_gmail_service()
    try:
        results = service.users().messages().list(
            userId='me',
            maxResults=1,
            labelIds=['INBOX'],
<EMAIL_ADDRESS>is:unread subject:Transaction OTP"
        ).execute()
        message = results.get('messages', [])[0]

        if not message:
            logger.warning('[get_otp] No messages from CDSL found.')
        else:
            msg = service.users().messages().get(userId='me', id=message['id']).execute()
            email_data = msg['payload']['headers']
            for values in email_data:
                name = values["name"]
                if name == "From":
                    from_name = values["value"]
                    match_opt = re.search('(\d{6})', str(msg['snippet']))
                    return match_opt.group()
    except Exception as exp:
        logger.warning("[get_otp] No messages from CDSL found to read", exp)

    if tries >= 5:
        logger.error("[get_otp] No email from edis even after %s tries", tries)
        exit()

    # If email is not received, wait for 15 seconds and try again
    while tries < 5:
        time.sleep(15)
        get_otp_from_email(tries=tries + 1)

Stack trace
Here is the truncated log of the day it failed

09:09:44.619
2024-07-25 03:39:44,619—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

09:09:45.083
2024-07-25 03:39:45,083—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages/190e7f906d9a91b0?alt=json

09:10:00.423
2024-07-25 03:40:00,423—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

09:10:00.937
2024-07-25 03:40:00,937—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages/190e7f906d9a91b0?alt=json

09:10:16.278
2024-07-25 03:40:16,278—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

09:10:16.817
2024-07-25 03:40:16,793—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages/190e7f906d9a91b0?alt=json

09:10:32.134
2024-07-25 03:40:32,133—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

09:10:32.618
2024-07-25 03:40:32,618—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages/190e7f906d9a91b0?alt=json

09:10:47.940
2024-07-25 03:40:47,940—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

09:10:48.557
2024-07-25 03:40:48,557—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages/190e7f906d9a91b0?alt=json

09:11:03.944
2024-07-25 03:41:03,943—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

09:11:04.464
2024-07-25 03:41:04,464—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages/190e7f906d9a91b0?alt=json

Thanks!

Hi @ohmayr , did you had a chance to check the issue? It is causing some real pain

Hi @shikharvaish28, thanks for reporting this and sharing the reproduction code. I see that you've also shared some logs for the requested requested URL.
Can you share the error / response that you're getting for a failed request?

Hi @ohmayr, since I use the python library I don't have the response. Instead, I did add some more logging and here is what it got:

Aug 5, 2024 09:08:13.233
2024-08-05 03:38:13,233—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

Aug 5, 2024 09:08:13.636
2024-08-05 03:38:13,635—ERROR-edis.edis_utils — [get_otp] No messages from CDSL found. Message = []

Aug 5, 2024 09:08:13.787
2024-08-05 03:38:13,786—DEBUG-urllib3.connectionpool — https://o154087.ingest.sentry.io:443 "POST /api/XXXXXXXXXXXXXXXX/store/ HTTP/1.1" 200 41

Aug 5, 2024 09:08:24.414
2024-08-05 03:38:24,413—DEBUG-google.auth.transport.requests — Making request: POST https://oauth2.googleapis.com/token

Aug 5, 2024 09:08:24.414
2024-08-05 03:38:24,414—DEBUG-urllib3.connectionpool — Starting new HTTPS connection (1): oauth2.googleapis.com:443

Aug 5, 2024 09:08:24.639
2024-08-05 03:38:24,639—DEBUG-urllib3.connectionpool — https://oauth2.googleapis.com:443 "POST /token HTTP/1.1" 200 None

Aug 5, 2024 09:08:24.816
2024-08-05 03:38:24,655—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

Aug 5, 2024 09:08:25.141
2024-08-05 03:38:25,141—ERROR-edis.edis_utils — [get_otp] No messages from CDSL found. Message = []

Aug 5, 2024 09:08:25.151
2024-08-05 03:38:25,150—DEBUG-urllib3.connectionpool — Starting new HTTPS connection (1): o154087.ingest.sentry.io:443

Aug 5, 2024 09:08:25.371
2024-08-05 03:38:25,371—DEBUG-urllib3.connectionpool — https://o154087.ingest.sentry.io:443 "POST /api/XXXXXXXXXXXXXXXX/store/ HTTP/1.1" 200 41

Aug 5, 2024 09:08:28.659
2024-08-05 03:38:28,659—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

Aug 5, 2024 09:08:29.183
2024-08-05 03:38:29,054—ERROR-edis.edis_utils — [get_otp] No messages from CDSL found. Message = []

Aug 5, 2024 09:08:29.216
2024-08-05 03:38:29,216—DEBUG-urllib3.connectionpool — https://o154087.ingest.sentry.io:443 "POST /api/XXXXXXXXXXXXXXXX/store/ HTTP/1.1" 200 41

Aug 5, 2024 09:08:40.167
2024-08-05 03:38:40,167—DEBUG-googleapiclient.discovery — URL being requested: GET https://gmail.googleapis.com/gmail/v1/users/me/messages?maxResults=1&labelIds=INBOX&q=from%3Aedis%40cdslindia.co.in+is%3Aunread+subject%3ATransaction+OTP&alt=json

Aug 5, 2024 09:08:40.813
2024-08-05 03:38:40,782—ERROR-edis.edis_utils — [get_otp] No messages from CDSL found. Message = []

def get_otp_from_email(tries=0) -> str:
    service = get_gmail_service()
    try:
        results = service.users().messages().list(
            userId='me',
            maxResults=1,
            labelIds=['INBOX'],
<EMAIL_ADDRESS>is:unread subject:Transaction OTP"
        ).execute()
        message = results.get('messages', [])[0]

        if not message:
            logger.warning('[get_otp] No messages from CDSL found.')
        else:
            msg = service.users().messages().get(userId='me', id=message['id']).execute()
            email_data = msg['payload']['headers']
            for values in email_data:
                name = values["name"]
                if name == "From":
                    from_name = values["value"]
                    match_opt = re.search('(\d{6})', str(msg['snippet']))
                    return match_opt.group()
    except Exception as exp:
        logger.warning("[get_otp] No messages from CDSL found to read", exp)

    if tries >= 5:
        logger.error("[get_otp] No email from edis even after %s tries", tries)
        exit()

    # If email is not received, wait for 15 seconds and try again
    while tries < 5:
        time.sleep(15)
        get_otp_from_email(tries=tries + 1)

Please suggest if I need to add something to get the response

@shikharvaish28 I ran the provided script with my own query parameters and i'm able to successfully get the message.
I can't say for sure why it isn't working in your case since you mentioned that it only happens sometimes. Try tinkering with the query parameters and see if it works?
Would help if you have a concrete example to reproduce this.

@ohmayr so that's precisely the problem - the code fails sometimes, but not consistently. This happens across two tenants with different accounts. The code runs simultaneously on both instances, but usually, only one fails while the other succeeds.
Can you help me add debug logs to capture and share the response when the message fetch fails?


Add GermanTechJobs and DevITJobs with remote jobs

These look like jobs not freelancing gigs. I recommend this repo https://github.com/lukasz-madon/awesome-remote-job

Hey @andychase, on the job boards are both full time and freelance roles (you can use filters to find them)
That's why we added them in the "Jobs" section, next to No Fluff Jobs or Jobhunt.ai which are also not purely freelance gigs :)
So maybe you could reconsider the PR and merge it as well?

Oh yeah you are right I forgot there is that jobs section. Let me think on this further!

These seem like they are regional so I added them to the regional section


create project contains a lot of bugs and todos

Someone needs to work on this page

fixed in #437


Throw error when building a parse tree of a non-existent revision.

Cleans up error handling so that an error is thrown when trying to parse a missing revision.  This fixes https://github.com/wiki-ai/revscoring/issues/135.

Sitting for 25 days.  @Ladsgroup @ToAruShiroiNeko @he7d3r, got a minute?


Update BaseYii.php

Q
A

Is bugfix?
yes/no

New feature?
yes/no

Breaks BC?
yes/no

Tests pass?
yes/no

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

It's good as is.


Draft: Preserve laziness via macro

Looks quite unsound but works

Very nice work!

It's still a draft, but I don't have any complaints with this.

Generated Scaladoc looks ok:

Scala 2

Scala 3


More autorest cleanup in Eventgrid

This is another big batch of changes. Autorest on an unmodified tree now produces no diffs.
Because previously hand written files are now generated there are lines that have moved around but are unchanged.
Major changes:

SystemEventNames is now produced automatically. SystemEventMappingNames was entirely unused, so it is gone.
1.a) We're now using the comment from the swagger for all javadoc here, where previously it was inconsistently the swagger comment or not.
Removed all attribute modifications
Took updates to EventGridPublisherClientImplBuilder that came with autorest update
Added aeg-channel-name argument in EventGridPublisherClientImpl

There are two classes of API change:

Added @Deprecated to two events that should have been removed from GA.
Many previously @Immutable types are now correctly @Fluent.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-messaging-eventgrid

/azp run java - eventgrid - ci


Create Dark-and-Easy

A dark theme I made a couple of years ago.... so easy on the eyes.

You'd better read the readme.md file about how to add a theme. Closing this for now

Whatever... thanks. No worries.


Welcome page customise links text is cut when 3 extensions are already installed

While verifying #22098

VSCode Version: 1.13

Steps to Reproduce:

Install JavaScript, TypeScript and Python extensions from the Welcome page
Observe that customisation message is cut at "and":

Should we adapt to the text length somehow, or maybe shorten (installed) to a check mark (✔) or something similar as a quick fix?

Tracked as #27510


GDScript: Fix built-in script find_class bugs

built-in scripts may have a "::" in their path, confusing find_class()
also changed is_subclass() to a more useful has_class() to match the ones in GDScriptParser
theres sadly no test suite for built-in scripts, so theres not much i can do about adding test cases
fixes https://github.com/godotengine/godot/issues/70184

Thanks!


XTextBox starting row

Issue filed by neoengineer on ImpulseAdventure website:
GUIslice is a great library - Thank you! I ma having a issue with the behavior of the text box created with gslc_ElemXTextboxCreate. The text always seesm to start in the middle of the box, even after a call to text box reset. I am using an adafruit feather M0 and an adafruit 480x320 display. Eveything else appears to work. You can see this behavior using gslc_ex10_ard. I experimented by reducing the TBOX_ROWS value to force "a no scroll" and the text then starts at the top of the text box, but then wraps in the middle of the display. any help would be greatly appreciated.

Textbox control and scrollbar updates have been completed and out for beta test.

Fixes have been made to the textbox to make the operation more intuitive. New content is added from the top and will eventually continue below the visible window portion of the buffer. Scrolling to the bottom of the buffer enables the user to monitor the latest rows that have been added.


Kute not working in iOS v7.x

Platform:iPhone5s iOS 7.0 Safari
It seems to not support window.performance in this version. :(

For legacy browsers you need polyfills, check my minifill.


Use a custom luajit base64 implementation

This implementation of base64 should be about 40 times as fast as the previous one. It uses luajit's bit module, which neovim provides even on platforms without luajit.

Thank you :)


(No. 21) New Filter: By Specific Owner [ Medium ]

Would require new fields in the PAD input.

potentially helpful clarification: "Support filtering by agency"

Anne added the name of the structure owner (i.e, California Department of Transportation, etc.) so it can be leveraged in the application. Anne sent an excel file (path below) which includes this new field (StructOwner).
G:\projects\FishPass_PSMFC_2018\Data\Source\PADPlusTracingForFISHPassV3.xls

National Forests, CDFW Regions, Caltrans Districts, BLM/State Boundaries were listed in their request, but these don't map directly to the field Anne shared. I requested her to compile a list of owners as they appear in that field for us to use.

List from Anne on 2/22 that should be direct from the text:
California Department of Transportation
U.S. Forest Service
Sonoma County Department of Transportation and Public Works
U.S. Forest Service Los Padres National Forest
Santa Barbara County Flood Control District
Santa Clara Valley Water District
Humboldt County
County of Santa Cruz
Santa Barbara County Public Works-Roads Division
Mendocino   County


XPath on xml response returns json

When accept is 'application/xml', response is having XML. However, upon applying XPath on this response, resulting value is in JSON, which is also entire content instead of the filtered/matched XPath.
Enclosing target folder which has the print statements for both xml response, response with XPath.
target_xpathOnxmlresponseTojson.zip

please read this section of the doc carefully and you still have issues, you can re-open this ticket: https://github.com/intuit/karate#type-conversion


Update mentorship process

Closes https://github.com/nodejs/mentorship/issues/114
New process:

Will it require mentees to start the process again? like filling form?

@ankibalyan For the upcoming round we will ask mentees to fill in a new form. Mainly because of the process change; the form is a little different this time. We are going to give applied mentees from the previous round a higher priority by  announcing the start of the cohort first to them, and then to the rest of the world!

Looking forward to it.


Add Ruby 3.0 support and replace Travis CI with GitHub Actions

Close https://github.com/influxdata/influxdb-ruby/issues/245
Close https://github.com/influxdata/influxdb-ruby/pull/246
Close https://github.com/influxdata/influxdb-ruby/pull/247 (included in this PR, but didn't work on Ruby < 2.7)

I think you need to click some button to enable GitHub Actions for this repo 🤔
You can see that the run, https://github.com/dentarg/influxdb-ruby/actions/runs/535074223, completed successfully in my fork


Easier use of Benchmark.measure

When using Benchmark.measure it is not easy to assign benchmarked values to variables outside the block scope. Say, I want to benchmark a method that returns a value and I want to use that value later. One way to do achieve this is to declare the variable beforehand and make in uninitialized:
foo = uninitialized Foo
time_spent = Benchmark.measure do
  foo = create_foo
end
foo.do_something

While this block provided to Benchmark.measure should always be executed, the usage of uninitialized is still unsafe. Maybe some refactoring may shuffle things around and it might no longer be guaranteed that foo will be initialized. The compiler won't complain.
An alternative would be to initialize the variable with nil and use not_nil! afterwards, but it makes no real difference and is just a bit more complicated.
Both ways require do declare variable(s) beforehand, which is tedious and the great power of type inference is not used.
My first intention was to solve this with macros, but a macro with block would still require the variable to be at least declared outside the block (though it would be guaranteed to be initialized by the inlined block scope), and the benchmarking code as argument for the macro is not sufficient if there are multiple statements involved. As discussed in Gitter yesterday, the minimal benefit would probably not be worth the macro overhead.
But there is an easy way to make this better: Extract the code before and after the yield in Benchmark.measure into two methods Benchmark.start and Benchmark.stop. Then these two methods can be used anywhere and the code to be benchmarked can be in the same lexical scope as the rest (measures could also be overlapping each other).
The above example would look this way:
benchmark = Benchmark.start
foo = create_foo
time = Benchmark.stop(benchmark)
foo.do_something

The stop line could also be time = benchmark.stop but I see little benefit from this, start and stop are nice and easy to remember.
Any ideas on this proposal?

Would this mean that multiple Benchmarks could be active at the same time? For example:
benchmark1 = Benchmark.start
# do something
benchmark2 = Benchmark.start
# do more things
time1 = Benchmark.stop(benchmark1)
time2 = Benchmarl.stop(benchmark2)

Also, would it make more sense for Benchmark.start to return an instance that has a stop method instead of having to pass the returned value back to Benchmark.stop? More like this:
benchmark1 = Benchmark.start
# do something
benchmark2 = Benchmark.start
# do more things
time1 = benchmark1.stop
time2 = benchmark2.stop

Yes that would be possible. Start time is stored in local variables, and they can be used overlapping or even conditional. You can use it freely.
I've also considered your second example, but I think the other approach makes more sense, because start/stop are equivalent methods on Benchmark and the scope of operation keeps the same. But I would be fine with this, too.
Another nice treat: You can use the same starting point for multiple measurements:
benchmark = Benchmark.start
# do something
time1 = benchmark.stop benchmark, "done something"
# do something else
time2 = benchmark.stop benchmark, "done something and something else"

Closing since there has been no further interest in this matter.


CHIP_FEATURE_EMB_PSRAM / CHIP_FEATURE_EMB_FLASH (IDFGH-12344)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hi,
does this 2 CHIP_FEATURE_EMB_PSRAM / CHIP_FEATURE_EMB_FLASH always be "external" ?

https://github.com/espressif/esp-idf/issues/10175


html-pdf fails with Debian 10 due to new version of OpenSSL

After upgrading to Debian 10, html-pdf stopped working. That's because phantomjs, used by html-pdf fails when parsing /etc/ssl/openssl.cnf
This is the error when trying to run the example test that will generate the businesscard.pdf:
TAP version 13
# allows custom html and css
not ok 1 Error: html-pdf: Received the exit code '1' Auto configuration failed<PHONE_NUMBER>01120:error:25066067:DSO support routines:DLFCN_LOAD:could not load the shared library:dso_dlfcn.c:185:filename(libssl_conf.so): libssl_conf.so: cannot open shared object file: No such file or directory<PHONE_NUMBER>01120:error:25070067:DSO support routines:DSO_load:could not load the shared library:dso_lib.c:244:<PHONE_NUMBER>01120:error:0E07506E:configuration file routines:MODULE_LOAD_DSO:error loading dso:conf_mod.c:285:module=ssl_conf, path=ssl_conf<PHONE_NUMBER>01120:error:0E076071:configuration file routines:MODULE_RUN:unknown module name:conf_mod.c:222:module=ssl_conf
  ---
    operator: error
    expected: |-
      undefined
    actual: |-
      [Error: html-pdf: Received the exit code '1'
Auto configuration failed
139625719701120:error:25066067:DSO support routines:DLFCN_LOAD:could not load the shared library:dso_dlfcn.c:185:filename(libssl_conf.so): libssl_conf.so: cannot open shared object file: No such file or directory
139625719701120:error:25070067:DSO support routines:DSO_load:could not load the shared library:dso_lib.c:244:
139625719701120:error:0E07506E:configuration file routines:MODULE_LOAD_DSO:error loading dso:conf_mod.c:285:module=ssl_conf, path=ssl_conf
139625719701120:error:0E076071:configuration file routines:MODULE_RUN:unknown module name:conf_mod.c:222:module=ssl_conf
]
    at: <anonymous> (/tmp/examples/businesscard/test.js:24:7)
    stack: |-
      Error: html-pdf: Received the exit code '1'
      Auto configuration failed
<PHONE_NUMBER>01120:error:25066067:DSO support routines:DLFCN_LOAD:could not load the shared library:dso_dlfcn.c:185:filename(libssl_conf.so): libssl_conf.so: cannot open shared object file: No such file or directory
<PHONE_NUMBER>01120:error:25070067:DSO support routines:DSO_load:could not load the shared library:dso_lib.c:244:
<PHONE_NUMBER>01120:error:0E07506E:configuration file routines:MODULE_LOAD_DSO:error loading dso:conf_mod.c:285:module=ssl_conf, path=ssl_conf
<PHONE_NUMBER>01120:error:0E076071:configuration file routines:MODULE_RUN:unknown module name:conf_mod.c:222:module=ssl_conf

          at ChildProcess.respond (/tmp/examples/businesscard/node_modules/html-pdf/lib/pdf.js:121:31)
          at ChildProcess.emit (events.js:197:13)
          at Process.ChildProcess._handle.onexit (internal/child_process.js:254:12)
  ...
/tmp/examples/businesscard/test.js:25
    t.assert(pdf.filename, 'Returns the filename')
                 ^

TypeError: Cannot read property 'filename' of null
    at /tmp/examples/businesscard/test.js:25:18
    at ChildProcess.respond (/tmp/examples/businesscard/node_modules/html-pdf/lib/pdf.js:126:14)
    at ChildProcess.emit (events.js:197:13)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:254:12)

Setting that config file to an empty file solves the problem, but this is obviously not a long-term solution:
TAP version 13
# allows custom html and css
ok 1 null
ok 2 Returns the filename
ok 3 Saves the file to the desired destination

1..3
# tests 3
# pass  3

# ok

The default /etc/ssl/openssl.cnf file with debian 10 is:
#
# OpenSSL example configuration file.
# This is mostly being used for generation of certificate requests.
#

# Note that you can include other files from the main configuration
# file using the .include directive.
#.include filename

# This definition stops the following lines choking if HOME isn't
# defined.
HOME                    = .

# Extra OBJECT IDENTIFIER info:
#oid_file               = $ENV::HOME/.oid
oid_section             = new_oids

# System default
openssl_conf = default_conf

# To use this configuration file with the "-extfile" option of the
# "openssl x509" utility, name here the section containing the
# X.509v3 extensions to use:
# extensions            =
# (Alternatively, use a configuration file that has only
# X.509v3 extensions in its main [= default] section.)

[ new_oids ]

# We can add new OIDs in here for use by 'ca', 'req' and 'ts'.
# Add a simple OID like this:
# testoid1=<IP_ADDRESS>
# Or use config file substitution like this:
# testoid2=${testoid1}.5.6

# Policies used by the TSA examples.
tsa_policy1 = <IP_ADDRESS>.1
tsa_policy2 = <IP_ADDRESS>.5.6
tsa_policy3 = <IP_ADDRESS>.5.7

####################################################################
[ ca ]
default_ca      = CA_default            # The default ca section

####################################################################
[ CA_default ]

dir             = ./demoCA              # Where everything is kept
certs           = $dir/certs            # Where the issued certs are kept
crl_dir         = $dir/crl              # Where the issued crl are kept
database        = $dir/index.txt        # database index file.
#unique_subject = no                    # Set to 'no' to allow creation of
                                        # several certs with same subject.
new_certs_dir   = $dir/newcerts         # default place for new certs.

certificate     = $dir/cacert.pem       # The CA certificate
serial          = $dir/serial           # The current serial number
crlnumber       = $dir/crlnumber        # the current crl number
crl             = $dir/crl.pem          # The current CRL
private_key     = $dir/private/cakey.pem# The private key

x509_extensions = usr_cert              # The extensions to add to the cert

# Comment out the following two lines for the "traditional"
# (and highly broken) format.
name_opt        = ca_default            # Subject Name options
cert_opt        = ca_default            # Certificate field options

# Extension copying option: use with caution.
# copy_extensions = copy

# Extensions to add to a CRL. Note: Netscape communicator chokes on V2 CRLs
# so this is commented out by default to leave a V1 CRL.
# crlnumber must also be commented out to leave a V1 CRL.
# crl_extensions        = crl_ext

default_days    = 365                   # how long to certify for
default_crl_days= 30                    # how long before next CRL
default_md      = default               # use public key default MD
preserve        = no                    # keep passed DN ordering

# A few difference way of specifying how similar the request should look
# For type CA, the listed attributes must be the same, and the optional
# and supplied fields are just that :-)
policy          = policy_match

# For the CA policy
[ policy_match ]
countryName             = match
stateOrProvinceName     = match
organizationName        = match
organizationalUnitName  = optional
commonName              = supplied
emailAddress            = optional

# For the 'anything' policy
# At this point in time, you must list all acceptable 'object'
# types.
[ policy_anything ]
countryName             = optional
stateOrProvinceName     = optional
localityName            = optional
organizationName        = optional
organizationalUnitName  = optional
commonName              = supplied
emailAddress            = optional

####################################################################
[ req ]
default_bits            = 2048
default_keyfile         = privkey.pem
distinguished_name      = req_distinguished_name
attributes              = req_attributes
x509_extensions = v3_ca # The extensions to add to the self signed cert

# Passwords for private keys if not present they will be prompted for
# input_password = secret
# output_password = secret

# This sets a mask for permitted string types. There are several options.
# default: PrintableString, T61String, BMPString.
# pkix   : PrintableString, BMPString (PKIX recommendation before 2004)
# utf8only: only UTF8Strings (PKIX recommendation after 2004).
# nombstr : PrintableString, T61String (no BMPStrings or UTF8Strings).
# MASK:XXXX a literal mask value.
# WARNING: ancient versions of Netscape crash on BMPStrings or UTF8Strings.
string_mask = utf8only

# req_extensions = v3_req # The extensions to add to a certificate request

[ req_distinguished_name ]
countryName                     = Country Name (2 letter code)
countryName_default             = AU
countryName_min                 = 2
countryName_max                 = 2

stateOrProvinceName             = State or Province Name (full name)
stateOrProvinceName_default     = Some-State

localityName                    = Locality Name (eg, city)

0.organizationName              = Organization Name (eg, company)
0.organizationName_default      = Internet Widgits Pty Ltd

# we can do this but it is not needed normally :-)
#1.organizationName             = Second Organization Name (eg, company)
#1.organizationName_default     = World Wide Web Pty Ltd

organizationalUnitName          = Organizational Unit Name (eg, section)
#organizationalUnitName_default =

commonName                      = Common Name (e.g. server FQDN or YOUR name)
commonName_max                  = 64

emailAddress                    = Email Address
emailAddress_max                = 64

# SET-ex3                       = SET extension number 3

[ req_attributes ]
challengePassword               = A challenge password
challengePassword_min           = 4
challengePassword_max           = 20

unstructuredName                = An optional company name

[ usr_cert ]

# These extensions are added when 'ca' signs a request.

# This goes against PKIX guidelines but some CAs do it and some software
# requires this to avoid interpreting an end user certificate as a CA.

basicConstraints=CA:FALSE

# Here are some examples of the usage of nsCertType. If it is omitted
# the certificate can be used for anything *except* object signing.

# This is OK for an SSL server.
# nsCertType                    = server

# For an object signing certificate this would be used.
# nsCertType = objsign

# For normal client use this is typical
# nsCertType = client, email

# and for everything including object signing:
# nsCertType = client, email, objsign

# This is typical in keyUsage for a client certificate.
# keyUsage = nonRepudiation, digitalSignature, keyEncipherment

# This will be displayed in Netscape's comment listbox.
nsComment                       = "OpenSSL Generated Certificate"

# PKIX recommendations harmless if included in all certificates.
subjectKeyIdentifier=hash
authorityKeyIdentifier=keyid,issuer

# This stuff is for subjectAltName and issuerAltname.
# Import the email address.
# subjectAltName=email:copy
# An alternative to produce certificates that aren't
# deprecated according to PKIX.
# subjectAltName=email:move

# Copy subject details
# issuerAltName=issuer:copy

#nsCaRevocationUrl              = http://www.domain.dom/ca-crl.pem
#nsBaseUrl
#nsRevocationUrl
#nsRenewalUrl
#nsCaPolicyUrl
#nsSslServerName

# This is required for TSA certificates.
# extendedKeyUsage = critical,timeStamping

[ v3_req ]

# Extensions to add to a certificate request
basicConstraints = CA:FALSE
keyUsage = nonRepudiation, digitalSignature, keyEncipherment

[ v3_ca ]

# Extensions for a typical CA

# PKIX recommendation.

subjectKeyIdentifier=hash

authorityKeyIdentifier=keyid:always,issuer

basicConstraints = critical,CA:true

# Key usage: this is typical for a CA certificate. However since it will
# prevent it being used as an test self-signed certificate it is best
# left out by default.
# keyUsage = cRLSign, keyCertSign

# Some might want this also
# nsCertType = sslCA, emailCA

# Include email address in subject alt name: another PKIX recommendation
# subjectAltName=email:copy
# Copy issuer details
# issuerAltName=issuer:copy

# DER hex encoding of an extension: beware experts only!
# obj=DER:02:03
# Where 'obj' is a standard or added object
# You can even override a supported extension:
# basicConstraints= critical, DER:30:03:01:01:FF

[ crl_ext ]

# CRL extensions.
# Only issuerAltName and authorityKeyIdentifier make any sense in a CRL.

# issuerAltName=issuer:copy
authorityKeyIdentifier=keyid:always

[ proxy_cert_ext ]
# These extensions should be added when creating a proxy certificate

# This goes against PKIX guidelines but some CAs do it and some software
# requires this to avoid interpreting an end user certificate as a CA.

basicConstraints=CA:FALSE
# Here are some examples of the usage of nsCertType. If it is omitted
# the certificate can be used for anything *except* object signing.

# This is OK for an SSL server.
# nsCertType                    = server

# For an object signing certificate this would be used.
# nsCertType = objsign

# For normal client use this is typical
# nsCertType = client, email

# and for everything including object signing:
# nsCertType = client, email, objsign

# This is typical in keyUsage for a client certificate.
# keyUsage = nonRepudiation, digitalSignature, keyEncipherment

# This will be displayed in Netscape's comment listbox.
nsComment                       = "OpenSSL Generated Certificate"

# PKIX recommendations harmless if included in all certificates.
subjectKeyIdentifier=hash
authorityKeyIdentifier=keyid,issuer

# This stuff is for subjectAltName and issuerAltname.
# Import the email address.
# subjectAltName=email:copy
# An alternative to produce certificates that aren't
# deprecated according to PKIX.
# subjectAltName=email:move

# Copy subject details
# issuerAltName=issuer:copy

#nsCaRevocationUrl              = http://www.domain.dom/ca-crl.pem
#nsBaseUrl
#nsRevocationUrl
#nsRenewalUrl
#nsCaPolicyUrl
#nsSslServerName

# This really needs to be in place for it to be a proxy certificate.
proxyCertInfo=critical,language:id-ppl-anyLanguage,pathlen:3,policy:foo

####################################################################
[ tsa ]

default_tsa = tsa_config1       # the default TSA section

[ tsa_config1 ]

# These are used by the TSA reply generation only.
dir             = ./demoCA              # TSA root directory
serial          = $dir/tsaserial        # The current serial number (mandatory)
crypto_device   = builtin               # OpenSSL engine to use for signing
signer_cert     = $dir/tsacert.pem      # The TSA signing certificate
                                    # (optional)
certs           = $dir/cacert.pem       # Certificate chain to include in reply
                                        # (optional)
signer_key      = $dir/private/tsakey.pem # The TSA private key (optional)
signer_digest  = sha256                 # Signing digest to use. (Optional)
default_policy  = tsa_policy1           # Policy if request did not specify it
                                        # (optional)
other_policies  = tsa_policy2, tsa_policy3      # acceptable policies (optional)
digests     = sha1, sha256, sha384, sha512  # Acceptable message digests (mandatory)
accuracy        = secs:1, millisecs:500, microsecs:100  # (optional)
clock_precision_digits  = 0     # number of digits after dot. (optional)
ordering                = yes   # Is ordering defined for timestamps?
                                # (optional, default: no)
tsa_name                = yes   # Must the TSA name be included in the reply?
                                # (optional, default: no)
ess_cert_id_chain       = no    # Must the ESS cert id chain be included?
                                # (optional, default: no)
ess_cert_id_alg         = sha1  # algorithm to compute certificate
                                # identifier (optional, default: sha1)
[default_conf]
ssl_conf = ssl_sect

[ssl_sect]
system_default = system_default_sect

[system_default_sect]
MinProtocol = TLSv1.2
CipherString = DEFAULT@SECLEVEL=2

More info: it seems all the problem is in this line. Commenting it makes the error go away:
# System default
openssl_conf = default_conf

More info: it seems all the problem is in this line. Commenting it makes the error go away:
# System default
openssl_conf = default_conf

Same issue for me when migrating on Debian10.
Your patch works fine !

Thank you !

More info: it seems all the problem is in this line. Commenting it makes the error go away:
# System default
openssl_conf = default_conf

Thank you, saved my day !

Hi, I have this same issue running on this environment :
the running environment is
os : Ubuntu 18.04.3 LTS
node : v8.10.0
html-pdf : latest
but I don't have this line inside /etc/ssl/openssl.cnf file
# System default
openssl_conf = default_conf

Same problem here.
If you don't want (or can't) override your OpenSSL system configuration, you can use OPENSSL_CONF env var to load a custom config file without openssl_conf parameter
cp  /etc/ssl/openssl.cnf /tmp/html-pdf-openssl.cnf
sed -i '/openssl_conf/d' /tmp/html-pdf-openssl.cnf
OPENSSL_CONF=/tmp/html-pdf-openssl.cnf html-pdf test/businesscard.html businesscard.pdf

Any news on this ? I don't want to override my OPENSSL_CONF

This issue persists on Ubuntu 21.04 and like the other person mentioned...
# System default
openssl_conf = default_conf

...does not exist.

Two years later, the problem persists in Debian 11. Exact same error. Is this package being maintained?

The issue still persists. Any drop-in replacement available for html-pdf ?

I guess we need to move on from this lib to another solution to generate PDF. Any thought? Do you know a better approach to tackle this issue - Generating pdf from html in NodeJS - with less pain @cypx @DigitalLeaves @pigeonvictor @Franckapik @jscul
I am even thinking about using other languages to do that for me. Thanks.

Solução sem DevOps

problema de SSL corrigido: simular vazamentos de phantomjs de configuração vazios do sandbox  bazelbuild/rules_closure#351 (comentário)
html-pdf assept opções para childprocessas quais serão passadas em phantomJs
node_modules/html-pdf/lib/pdf.js:89 (childProcessOptions)
https://nodejs.org/api/child_process.html
adicionado objeto env a childProcessOptions para substituir OPENSSL_CONF

pdf.create(html, {
      childProcessOptions: {
        env: {
          OPENSSL_CONF: '/dev/null',
        },
      }
    });

This worked for me!

Solução sem DevOps

problema de SSL corrigido: simular vazamentos de phantomjs de configuração vazios do sandbox  bazelbuild/rules_closure#351 (comentário)
html-pdf assept opções para childprocessas quais serão passadas em phantomJs
node_modules/html-pdf/lib/pdf.js:89 (childProcessOptions)
https://nodejs.org/api/child_process.html
adicionado objeto env a childProcessOptions para substituir OPENSSL_CONF

pdf.create(html, {
      childProcessOptions: {
        env: {
          OPENSSL_CONF: '/dev/null',
        },
      }
    });

This worked for me!

Hey, are there any side effects using this option?


crystal: 0.32.1 -> 0.33.0

Motivation for this change
ZHF: #80379
Can't be backported to 20.03 because of feature freeze (0.32.1 build for 20.03 handled in #80389)
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This change is

@GrahamcOfBorg build crystal

@GrahamcOfBorg build crystal

Sorry, accidentally committed a (now revoked) secret.
@GrahamcOfBorg build crystal

@GrahamcOfBorg build crystal

Will merge on Feb 25 12 pm UTC unless anyone minds otherwise.

@GrahamcOfBorg build crystal_0_27 crystal_0_29 crystal_0_30 crystal_0_31 crystal_0_32 crystal_0_33

@GrahamcOfBorg build crystal_0_32

Unfortunately, looks like changes to generic break crystal_0_32_0 build :(

Superseded by #85432


Fixing mistake in image class

Looks good, but could you squash the two commits?

Squashed and merged manually as 024c4f14a.

Nope, you haven't, second commit isn't included)

Hm indeed, I must have messed up somehow :(

Now it should be good with 6aa331e58. Thanks for checking!


Precompiled macros #ifndef/#if/#elif/#else/#endif within the inactive region cannot be syntactically highlighted

The latest [clangd-17](https://github.com/llvm/llvm-project)) already supports syntax highlighting in inactive region, but not in contiguous inactive region, as shown in the figure below,

The desired display is as follows,

BWT, this feature in the [vscode - cpptools] (https://github.com/Microsoft/vscode-cpptools) has been the support,  clangd planned to support this feature?

The same behaviour was also suggested by @daiyousei-qz in #1181, and the draft patch at https://github.com/llvm/llvm-project/compare/main...daiyousei-qz:avoid-mark-conditional-preprocessor-line-inactive illustrates a possible implementation strategy.

When will clangd-17 be released?

According to https://llvm.org/docs/HowToReleaseLLVM.html#annual-release-schedule, sometime in early September.
Note that weekly snapshots of trunk are available at https://github.com/clangd/clangd/releases.


Fixed #33114 -- Defined default output_field of StringAgg.

fix and a unit test for this bug.

@charettes Thank you for your review. I fixed them.

@AliSayyah Thanks for this patch :+1: Welcome aboard :boat:

It's a pleasure to be here @felixxm :pray:


Any new release planned?

Hey guys,
I noticed some active work in the last months here with merging PRs, improving docs and infrastructure. That is great!
Is this project being revived and moving forward (again)? Any release planned in the near future?
Would be glad to get some insights on the plans here.
Thanks a lot.
Cheers

oof, I forgot about releasing it, otw
at the other end, we don't have a release schedule though...
wdyt @materializecss/maintainers? Should we release new version for every non documentation changes? Or any other ideas?

Should we release new version for every non documentation changes?

Not needed since GHA and the GH Pages are nostly independent. Releases should only be done for the changes in the code of materialize.


add filter to tree

Signed-off-by: Lea Carlier<EMAIL_ADDRESS>

Problem in question: when a parent node is collapsed (loop0 here) and the child is in the filter (read), if we check the parent, should the child also be checked or not?

@PatrickTasse could you please check this out?

From some examples I have seen, the behavior of a checkbox is the same whether the item is collapsed or not. I don't know if that's industry standard.

I guess I was wrong... at least from what Trace Compass does.

Behavior-wise, it's ok now

Looks good, except when checking the root element after filter, grand-parent selection icon is checked while parent is half checked. From the Trace Compass screencast behavior, if all filtered children are checked, the parent should be checked and become partially checked when removing the filter.

We could accept this patch as is, then write the unit test that will detect this error and fix it, or fix it now.

I guess I was wrong... at least from what Trace Compass does.

I wonder if we'd like to do the auto-expansion of the tree when typing a filter like in Trace Compass? Could be done afterwards separately. Pls, let me know.
Otherwise it looks good to me.


Compilation when a project isn't initialized

In a directory with a contract at contracts/Foo.sol, oz compile says "No contracts found to compile". It only finds the contract if there is an .openzeppelin directory or a package.json file.
We should make it so that oz compile works when neither of these are present, so that usage is as simple as creating a contracts directory and putting source files in it.

Superceded by https://github.com/OpenZeppelin/openzeppelin-sdk/issues/1378 (which has a more detailed explanation).


Remove allowBack and supportsRtl

See<EMAIL_ADDRESS>

Thanks @dozoisch for the PR and Effort. I have merged the same. Today I will push the release.
Thanks
</ Pranav >

@prscX thanks :)!


Keep the Composer Cache

Composer's cache is very reliable. This also speeds up the overall build times +40% to +60%.

Closing and reopening this PR lets us see the results of the performance difference with the caching enabled.
Initial Travis (before closing this pr) https://travis-ci.org/laravel/framework/builds/168797367
After Travis (after closing this pr) https://travis-ci.org/laravel/framework/builds/168797939

Thanks!

@taylorotwell are the Composer cache bugs fixed : https://github.com/laravel/framework/pull/12698 ?

Hmm, that's a good question.


Arguments.scanRestOfString() does not work at all on Linux

Ubuntu 20.04.1 LTS
Swift 5.3.2-RELEASE-ubuntu20.04
SDK Version: Latest
Arguments.scanRestOfString() will always return empty string on Linux due to the differences in SDK usage. scanOfTheRestOfString() will call Scanner.scanUpToString() on Linux with the parameter "" (empty string), however see this line, where empty string as a parameter will always return nil.
Calling context.args.scanner.scanUpToCharacters(from: .newLines) instead, fixes this issue.

it seems they don't support it, maybe you try my package for Vapor https://github.com/nerzh/telegram-vapor-bot

@cleanbit Thank you for reporting the issue! This should now be fixed by https://github.com/rapierorg/telegram-bot-swift/pull/122


added support for AMZN 2 to use systemd

This should resolve #908

Pull request acceptance prerequisites:

[x] Signed the CLA (if not already signed)
[x] Rebased/up-to-date with base branch
[ ] Tests pass
[ ] Updated CHANGELOG.md with patch notes (if necessary)
[ ] Updated CONTRIBUTORS (if attribution is requested)

Thanks for the PR, @vchan2002. I pulled your branch into a branch of mine that adds acceptance tests for Amazon Linux so we get some coverage - specinfra (one of the underlying Beaker libraries) actually doesn't support Amazon Linux 2 yet, so I'll have to leave those out of Travis for now, but my spot checks verify that everything else works as expected, so I'll pull these changes into my branch and we can add Amazon Linux 2 to CI later once specinfra is updated. Cheers!


⚠ Digital Ocean - https://status.digitalocean.com/ has degraded performance

In e6bd6da, Digital Ocean - https://status.digitalocean.com/ (https://status.digitalocean.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 294 ms

Resolved: Digital Ocean - https://status.digitalocean.com/ performance has improved in 1591047.


Auth0audit

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

Raising new PR as old 10415 got closed

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Hi @rsingh134 ,
Please check for failure validations and also accept Contributor License Agreement. Thanks

Hi @rsingh134,
Please check for failure validations. Thanks

Hi @rsingh134, Please check for failure validations. Thanks

I am not much aware of these errors and also files doesn't seems relevant.

Hi @rsingh134,
Please provide me a contributor access to your branch, so I will check for these failures. Thanks

Hi @rsingh134,
Please respond. Thanks

Hi @rsingh134,
We are waiting for your reply. Thanks

@v-atulyadav I am not sure, how to add you as contributor. I am having some issues, will ping you on teams to sort this.

Hi @rsingh134,
I will check this. Thanks

Hi @rsingh134,
As per our discussion please check with branch's original author for contributor access. And also get latest of master and push into your branch. Thanks

Hi @rsingh134,
Any update on above? Thanks

Hi @rsingh134,
Any update on contributor access? Thanks

Hi @rsingh134, We wanted to check on the status of PR #10441. PR is pending for more than 60 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation.

Since we have not received a response in the last 7 days, we are closing your PR #10441 per our standard operating procedures. If you still need support for this issue, you can re-open the PR at any time.
If you do re-open, we simply request that you ensure the PR has response to the last request. Thank you for your cooperation.


use proxyStatus in module when possible

Coverage remained the same at 40.283% when pulling 5ef40afd277dd85ff8b387b926800f20e1c7acca on ele828:fix-module-status into 8da340bc9571409fe3e8debecc646ba6b3a24ad7 on ringcentral:master.

Coverage remained the same at 40.283% when pulling f5f5d31ca3aec9c17a13adcb3da7481b941c7849 on ele828:fix-module-status into 8da340bc9571409fe3e8debecc646ba6b3a24ad7 on ringcentral:master.


Update pricing to reinclude legacy

Summary
update pricing to reinclude legacy info for enterprise

[x ] The documentation style guide has been adhered to.
[x ] If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
[ x] Files which have changed name or location have been allocated redirects.

Mind also updating the links on this page that point to unbound/bundled: https://developers.cloudflare.com/workers/platform/limits/

Going to merge this?

Hey @nevikashah / @tanushree-sharma - is this good to merge?


Implemented menu option to select different sounds, added 2 sounds

This is my attempt to fix https://github.com/michaelvillar/timer-app/issues/49 by adding an option to the context-menu.
The selection is not persisted between launches, and is selected per timer, so if you have multiple timers running you can select different sounds for them.
Please note that I have only very limited Swift/Cocoa knowledge, so if it looks 'off' somehow that is totally expected ;-)

@simongroenewolt good stuff so far. Can you fix the swiftlint issues and then we'll review?

@simongroenewolt good stuff so far. Can you fix the swiftlint issues and then we'll review?

@karbassi I'll fix the swiftlint stuff and clean up a little more.

@karbassi I'll fix the swiftlint stuff and clean up a little more.

@karbassi the code is ready for review now. I've added an additional sound option: 'no sound'.

@karbassi the code is ready for review now. I've added an additional sound option: 'no sound'.


Number parsing issue : Poland

I am using a parse method of PhoneNumberUtil with a Poland phone number e.g.(+48 (0) 22 512 XXXX). here, i got result as "Country Code: 48 National Number: 22512XXX Leading Zero(s): true" whereas i am assuming Zero will be truncated from the result.
When i try the same use case with UK phone number e.g.(+44 (0) 22 512 XXXX), it truncates the Zero from it and gives result as expected like "Country Code: 44 National Number: 22512XXXX".
After debugging the lipphonenumber sdk, i found that internally these values are coming from a resource file(PhoneNumberMetadataProto_**) which is an encrypted file.
So, my understanding is that number parsing logic must be same for all countries,

Why it is different for poland and UK?
Does this require any fix from my end, as i am expecting result without leading the zero value?

Thanks for the clarification.
However, phone number that we get are beyond our control and would like to fix this for our customers in Poland. Can you please suggest what is the best way to override this xml configuration for Poland in our project?

You can't easily, because we don't consider this a valid use-case.
If a number is bad, you want to reflect that. This number is not diallable and doesn't work. Why do you want to try and fix it? Parsing it will retain the number you gave it, which is what it's designed to do.
This isn't a bug with the library. Please consider joining the discussion group if you'd like to discuss this further.


Can not set background color for default selection

I want to set background color on  default selected rows, But it does not work. I can only set background color with click event.
.i.e. In the example below,
when "selected" is used, then  "bgColor" does NOT work. Any idea how I can fix it and if there is a workaround ? Thanks
selectRow = {
mode: 'checkbox',
clickToSelect: true,
onSelect: this.handleOnSelect,
hideSelectColumn: true,
//selected: this.state.selected,
clickToExpand: true,
bgColor: (row, rowIndex) => {
return '#00BFFF';  // return a color code
},
clickToEdit: true  // Click to edit cell also
};

@mpeng it's work for me, would you please give me more information. I think you can write a example online in the codesandbox so that i can reproduce it, thanks

@AllenFang , It works for me too. I must miss something earlier. My bad. Thanks


Detailed test output seems to be missing

When some tests are run using npm run test, I only get the end summary of the points, and not the details (e.g. why did some test fail). Here is what I have:
$ npm run test

><EMAIL_ADDRESS>test {project path}
> karma start

------------       RESULT       ------------

Summary:
  Sample group 1: 1/1 point
    ✔ Group1 > should work: ok
  Sample group 2: 0/1 point
    ✖ Group2 > should also work: failed

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>test: `karma start`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/...

Is there something that I need to enable in order to have the full output?
You can see how I tried to set up the environment on the private repo.

I just saw that the footer message talking about the detailed output is explicitly disabled in Reporter. So is it just that karma-helpers doesn’t support showing the detailed output, and competitors should always use the HTML file?
In that case, the header image in the README file should probably not have the footer line.

I just saw that the footer message talking about the detailed output is explicitly disabled in Reporter. So is it just that karma-helpers doesn’t support showing the detailed output, and competitors should always use the HTML file?
In that case, the header image in the README file should probably not have the footer line.

Yes, the detailed output is currently not implemented as the goal was to provide an initial version for the automated run only and competitors should use the HTML file. So that's why the footer message is disabled until it gets implemented.
But I should have time to look at this again on the weekend, it shouldn't actually be too complicated to implement I think.
(and yes, the image from the readme was copied from another package out of laziness ;) but it should be correct again once the feature has been added)

Yes, the detailed output is currently not implemented as the goal was to provide an initial version for the automated run only and competitors should use the HTML file. So that's why the footer message is disabled until it gets implemented.
But I should have time to look at this again on the weekend, it shouldn't actually be too complicated to implement I think.
(and yes, the image from the readme was copied from another package out of laziness ;) but it should be correct again once the feature has been added)

This feature has now been added in v1.1.0 and is enabled by default for the non-json reporter, meaning the same mocha messages that would be visible in the browser are now also logged in the terminal. Additionally, the browser console is now forwarded to the terminal, so console.log statements etc are now also visible.

This feature has now been added in v1.1.0 and is enabled by default for the non-json reporter, meaning the same mocha messages that would be visible in the browser are now also logged in the terminal. Additionally, the browser console is now forwarded to the terminal, so console.log statements etc are now also visible.


ciao: examples: ansible-galaxy install fails in clearlinux

When running ansible-galaxy install -r requirements.txt behind a proxy in clearlinux it fails as follows
# sudo -E ansible-galaxy install -r requirements.yml  -f
 [WARNING]: - clearlinux.docker was NOT installed successfully: The API server (https://galaxy.ansible.com/api/) is not responding, please
try again later

ERROR! - you can use --ignore-errors to skip failed roles and finish processing the list.

I get the same error on Ubuntu 16.04 desktop.

Need to identify where is the issue, it seems like the issue is on ClearLinux @mrkz could you please  share your finding here?

This isn't a bug in the documentation or the playbooks. But we stilll need to figure out whats wrong.

Since ClearLinux 10900 there is /etc/ssl/certs symlink to /var/cache/ca-certs which fixes the issue.
For ubuntu, the system might have missing certificates

I am unable to reproduce on ubuntu 16.04
@markdryan are you still having issues? do you have ca-certificates package installed?

It's working for me now as well, although I'm pretty sure it wasn't working last week.  Anyway we can close this issue I guess.


Update upstream prometheus

Merge upstream Prometheus from https://github.com/prometheus/prometheus/commit/44fcf876caad9a5d28b92a728d2c98c34015d377 into mimir-prometheus.
TO BE MERGED WITH MERGE COMMIT.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 23 committers have signed the CLA.:white_check_mark: pstibrany:white_check_mark: jesusvazquez:x: dgl:x: mmorel-35:x: hdost:x: brancz:x: prymitive:x: bboreham:x: fpetkovski:x: metalmatze:x: roidelapluie:x: mtfoley:x: steveazz:x: shenxn:x: lixin963:x: damnever:x: attachmentgenie:x: a-gavin:x: RichiH:x: zamariola:x: Harkishen-Singh:x: wangrzneu:x: NexucisYou have signed the CLA already but the status is still pending? Let us recheck it.


Macro doesn't exand list since 3.1.0

Hello,
I use the following template (templ.j2) to create a file with Ansible:
---
{%- macro expand_capa(_capa_list) -%}
  {#- use: writes the capa_list object as json list with double ticks -#}
  {#- params: _capa_list is list object of capability strings -#}
  [
  {%- for c in _capa_list -%}
    "{{ c }}"{% if not loop.last %},{% endif %}
  {%- endfor -%}
  ]
{%- endmacro %}

path "x" {
  capabilities = {{ expand_capa(capabilities) }}
}

With Jinja2 3.0.3 it creates the following file:
---
path "x" {
  capabilities = ["read","write","update","delete"]
}

But with Jinja2 3.1.0 it creates a different file:
---
path "x" {
  capabilities = ['read', 'write', 'update', 'delete']
}

It's similar to the output generated by capabilities = {{ capabilities }}. And it looks like the macro gets ignored.
For me, it looks like a bug, because the behavior has changed between two minor versions. I would expect that Jinja2 3.1.0 generates the same file as Jinja2 3.0.3
Environment:

Python version: 3.10.5
Jinja version: 3.1.0 and 3.0.3
Ansible version: 2.13.1

Ansible Playbook:
---
- hosts: localhost
  gather_facts: false

  tasks:
    - name: write template
      template:
        src: templ.j2
        dest: output
      vars:
        capabilities:
          - read
          - write
          - update
          - delete

This is not an issue in Jinja but in Ansible.
Duplicate of https://github.com/ansible/ansible/issues/78141.


Add support for transitive reference

The props/targets files are not imported automatically when the package is referenced transitively. Copying the props/targets files to buildTransitive/ should be enough to support transitive reference.
The main reason for this change is to allow creating meta-packages. These packages can reference Dotnet.ReproducibleBuilds and other packages (such as Roslyn Analyzers). Then, you can reference a single package to your projects.
https://github.com/NuGet/Home/wiki/Allow-package--authors-to-define-build-assets-transitive-behavior

I agree with @baronfel; we can do this but lets reduce duplication.


some days are invisible

some days are behave like another months day and being invisible

Could you try to reproduce this with the sample project?

I tried but there is no change. it still goes on.

Could you try to reproduce this with the sample project?

I tried but there is no change. it still goes on.

okey ı did it my bad


Source code Scraping subdomain

While doing full recon+attacks it always get stucked at this step. Its been 3hrs and its still not finished is it normal?

Depends completely on the target size, can't help on that. What you can do is increase gospider threads on cfg file.

The execution time for source-code scraping mainly depends on how large the target is. Gospider is used for source code scraping.
What u can do check:

If htop  command shows gospider running, that's means the scan is running.
Do ls -alpts in the reconftw/Recon/target.com/.tmp directory, now check the time of the file at the top. If you see a huge difference between the current time and the last time the file was updated that means, gospider isn't running.

Moreover, if you want u can disable "source code scraping" from the config file by setting SUBSCRAPING=false  on line 56


How to formulate problem with State is a combination of multiple factors?

Hi, shangtong
I am reading the book and following your implementation code.
In chapter 9, the random walk example, the book mentioned V(s, w) = W*S, where in random walk the state is 1 dimension, in its simplest form it could be like V(s, w) = w_1*s + w_0, but I wonder what if the state has 2 or more components?
say, S = S(position, time), how to rewrite the above function? Is it V(s, w) = w_1*(position + time) + w_0?

function approximation with features, e.g., tile coding


merging perf.shared projects

Fix Issue #1767
Changes

merging projects
creating folder based on namespace
renaming folder with incorrect spelling

Checklist

[ ] I ran Unit Tests locally.
[ ] CHANGELOG.md updated with one line description of the fix, and a link to the original issue if available.

For significant contributions please make sure you have completed the following items:

[ ] Design discussion issue #
[ ] Changes in public surface reviewed

The PR will trigger build, unit tests, and functional tests automatically. Please follow these instructions to build and test locally.
Notes for authors:

FxCop and other analyzers will fail the build. To see these errors yourself, compile localy using the Release configuration.

Notes for reviewers:

We support comment build triggers

/AzurePipelines run will queue all builds
/AzurePipelines run <pipeline-name> will queue a specific build

Do me a favor, on this branch please open the AspNetCore.sln.
This solution compiles Perf as a dependency and needs to update these references as well.
Thanks,

@TimothyMothra , just updated all slns to contemplate the new projects


Timeout Option added

fixes #124

@tcort do you know what fails on the "Docker / Build & Publish (pull_request)" job?

the error was Unable to find the password. Did you set with.password?

Yes but what's the cause as other PR are working...

I think I found the issue. You created your PR before the Docker workflow was added. So you don't have the config files in it. But Github still waits for the workflow to complete.
Can you please update your fork branch by merging the upstream master in it?

I merged it anyway crossing my finger that it was just an issue with your PR and that it will work on master... and it did...

Thank you! We used your action on https://github.com/academic/awesome-datascience and it helps a lot!


[#200] Expose current streaming WAL filename to Prometheus interface

Added new field current_wal_filename to struct server, which is then exposed to the prometheus metrics.
Closes #200.

Made the requested changes.
Should I squash and force-push all commits every time I make an update in a PR? That is, should one PR be only one commit?

Yes, all pull requests are one commit

You need to add your name to AUTHORS as well.
Then squash and force push.
https://www.git-tower.com/learn/git/faq/git-squash
https://github.com/progit/progit2/releases/tag/2.1.421

@jesperpedersen Done.

Merged.
Thanks for your contribution !


Feedback: test compile errors could give more useful info

Example:
$ godog .
# command-line-arguments
/tmp/1461229988472036489godog.go:70: no new variables on left side of :=
$

Would be nicer with:
my_test.go:50: no new variables on left side of :=

Perhaps check how go test does it.

yes, that would be a good enhancement

it is sad that go does not expose the build tools. in order to build a package with _test.go files is a little too complicated. gb has replicated the go build tool as a library, but I would not like to have it as a dependency. (thinking about having all dependencies inlined in the same package). It could be tried though.
To prevent such errors, it probably would make sense to test the package against errors while building and print the correct error message instead. Also, it could keep the generated go file when the exit code is different. Probably this should be resolved first, while looking for more sophisticated options like build tooling.

while giving another shot at go 1.4 TestMain function, was able to produce wanted behavior. This seems to be more pragmatic go approach in handling similar requirements - running a custom test suite with standard go build toolchain. Though, go versions until 1.4 won't be supported, which is not so dramatic. The branch is still in progress, but should be available in a week or two for release 0.4.


:seedling: Add catalog exploration and extension rbac/installation scripts

Description
The lack of tooling can make finding and installing extensions hard. This PR adds some hack/ scripts to help smooth over this rough edge until the proper tools are ready
Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

Is it worth adding some of these scripts and having to maintain them until we have proper tooling in place? Is there things that we've learned from writing these scripts that we can write up documentation on how to manually do all these things until the proper tooling is in place?

Is it worth adding some of these scripts and having to maintain them until we have proper tooling in place? Is there things that we've learned from writing these scripts that we can write up documentation on how to manually do all these things until the proper tooling is in place?

If you tell me we'll have the tools next week, then agree. Otherwise, in my opinion, yes. I don't think I'd use a package manager that requires me to read/understand documentation and go through a whole bunch of manual including manually finding/pulling/unpacking the bundle, and inspecting its content. I think it makes sense to show the minimum amount of effort towards tooling to help facilitate user's lives. This script is in hack, the documentation calls it out as a not-guaranteed-to-work helper script to smooth things out until we have proper tools. How much maintenance do you foresee happening here? Will it be that much of a time sync? Is it really not worth it?

@everettraven I think this is ready for merge ^^


Github web adress

Where can I find my github web adress please ?

Hello aydenmrt, your github address is https://github.com/aydenmrt - it is the link to your personal account.
Enjoy :)

Hello, thank you , I thought that, but I always have an error message :
The operation couldn’t be completed. (Pipeliner.ApiError error 0.)


shedlock-provider-jdbc and autocommit

In the executeCommand method of JdbcStorageAccessor class (shedlock-provider-jdbc module), the connection is forced to autocommit before applying the statement without restoring its previous value any after, and this could be indesiderable, specially if the connection is released to a pool and then consumed by other threads.
Perhaps the correct behaviour would be to save the previous autocommit value of the connection and restore it in a finally block.

Hi, thanks for feedback. Is this a theoretical concern or does it causes you any issues?

Hi Lukas, actually I have an application relying on a connection pool with autocommit set to false. So, after the JdbcStorageAccessor calls the executeCommand it releases the connection to the pool with a non-default (for the pool) state and it causes problems on subsequent calls that, expecting a non autocommit connection, try to manage the boundaries of the transaction.

Released as 4.32.0


An exception has occurred, use %tb to see the full traceback.  SystemExit: 2

This template is for miscellaneous issues not covered by the other issue categories.
For questions on how to work with TensorFlow, or support for problems that are not verified bugs in TensorFlow, please go to StackOverflow.
If you are reporting a vulnerability, please use the dedicated reporting process.
For high-level discussions about TensorFlow, please post to<EMAIL_ADDRESS>for questions about the development or internal workings of TensorFlow, or if you would like to know how to contribute to TensorFlow, please post to<EMAIL_ADDRESS>

@isayasfeyera,
Please fill the template to analyze the issue.
System information
Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
You can collect some of this information using our environment capture script You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
Describe the expected behavior
Code to reproduce the issue Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.


layers: Fix issue when sub-passes have diff attachment count.

In core_validation, if the attachment count between a primary
and secondary sub-pass was different, then it should treat
the shorter one as UNUSED.  So, if the longer one has the
attachment flags set to UNUSED, it shouldn't be an error.
Change-Id: I3647403551962ef522e7e2ba7ab10a5cb7466c52

+1. Only minor comments.

Somehow fixes and pull/push process grabbed a bunch more changes.  Will try to resolve.


Bugfix/sim player auth issue

Description
Sim client scripts run in parallel could fail due to conflicting permission requests. This change implements a simple lock to ensure only one script does the initial user modifications and fixes https://improbableio.atlassian.net/browse/UNR-3280
Tests
https://buildkite.com/improbable/product-research-benchmarks-run/builds/9124
Primary reviewers
@aleximprobable @improbable-danny

I think I may have been requested as a mistake instead of @improbable-danny?


How to setup laravel-echo-server

Hi to all,
i'm a bit confused on how setup this server with Laravel.
Right now i have a working configuration with Pusher, and i would like to test this server.
laravel-echo-server can use sqlite and redis. In .env the BROADCAST_DRIVER should be redis in both case ?
I have setup all, when i visit the page with server up the connection is established (i notice that because if i stop the server, Chrome Dev tools log errors), but when i try to send messages.. that are not dispatched. And, with pusher the configuration works fine (i'm just trying to change the server).
Can someone tell me how to configure server and how to setup Laravel ?
Thanks a lot

1- the broadcast driver is redis in both cases yes
2- the database used to store data that should persist, like presence channel members , so using sqlite or redis is up to you
3- to send msgs , first try to use the public channel in the event check https://laravel.com/docs/5.3/broadcasting#concept-overview
4- make sure u restart the queue to pick up the new event , i had the same issue and this step fixed it for me.
5- if u got issue with echo check this https://github.com/tlaverdure/laravel-echo-server/issues/40

There have been some improvements made, please try again with the CLI tool.


refactor: change CustomHint error usage into VirtualMachineError::Other(anyhow::Error)

This change is

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (0a5416b) 70.43% compared to head (feb74b0) 70.43%.

:exclamation: Current head feb74b0 differs from pull request most recent head 51373ec. Consider uploading reports for the commit 51373ec to get more accurate results

Files
Patch %
Lines

...lockifier/src/execution/syscalls/hint_processor.rs
25.00%
3 Missing :warning:

Additional details and impacted files
@@               Coverage Diff                @@
##           main-v0.13.1    #1397      +/-   ##
================================================
- Coverage         70.43%   70.43%   -0.01%
================================================
  Files                59       59
  Lines              7713     7712       -1
  Branches           7713     7712       -1
================================================
- Hits               5433     5432       -1
  Misses             1853     1853
  Partials            427      427

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

crates/blockifier/src/execution/syscalls/hint_processor.rs line 217 at r3 (raw file):

    ) -> HintExecutionResult {
        let StarknetHint::SystemCall { system: syscall } = hint else {
            return Err(HintError::Internal(VirtualMachineError::Other(anyhow::anyhow!(

actually, is this better with just the anyhow! macro or should we keep the internal CustomHint error?


Httpauth digest response

Could you also please add a link to the relevant RFC ?

These are the two RFCs, the last, this, and the upcoming PRs are based on.
HTTP Digest Access Authentication: https://www.rfc-editor.org/rfc/rfc7616
The Session Initiation Protocol (SIP) Digest Access Authentication Scheme: https://www.rfc-editor.org/rfc/rfc8760

These are the two RFCs, the last, this, and the upcoming PRs are based on. HTTP Digest Access Authentication: https://www.rfc-editor.org/rfc/rfc7616 The Session Initiation Protocol (SIP) Digest Access Authentication Scheme: https://www.rfc-editor.org/rfc/rfc8760

could you also add RFC7617 to the README ?
I had one more comment, apart from that it looks ready to merge.

The next release (3.6.0) is scheduled for Wed 18. October.
Should we merge this before or after the release ?

Sry for the late response. I can make the changes only after the 18th. So i would suggest to wait for the release and my last fixes. Then its ready to go for 3.7

Okay, it is scheduled for after 3.6.0 release
If you dont have time to work on fixing review comments, please consider to close the PR
and reopen it when you have time...

I'm not sure what happens in the FreeBSD test. But i looks like the test is hanging in an bootloop. I checked it and it gets the same output over and over again. Its running for over 3h right now. There is no other output than the repeating boot messages.

I've just disabled the FreeBSD workflow for now (until its more stable):
https://github.com/vmactions/freebsd-vm/issues/74


Optimize ReadPropertyMultiple method to get the response in a reasonable time

The method performs a HTTP call for each point, having a response time of 17s for 6 elements. Current Version of Metasys 10.1 API does not support multiple values reading on the Server. However, the response time is linear on the number of elements. Our SDK handles properly asynchronous calls, but probably Metasys API Server has a limit on Maximum concurrent connections.

We could cut down the time caching the GetObjectIdentifier results (12s for 6 elements), but to address the issue a verification is needed on how concurrent calls are handled in Metasys API Server, also to understand how many concurrent users a Web Application on top of Metasys can supports. @mgwelch

In the first timeline we have the current @SavannahEvans implementation that properly handles Task in parallel, whilst in the second timeline we have a modified version of ReadPropertyMultiple that wait for each call. What is relevant for troubleshooting here is that when the Metasys API server have to handle 6 concurrent requests the response time of each one grow up to 8 s. However, when the server handles 1 request at a time the response get back in 3 s.

Issue addressed in https://github.com/metasys-server/basic-services-dotnet/commit/84e051ea9f27c34b88d1e01c382b41b1d2349498. Now reading only the requested attributes in multiple calls instead of the whole object in one call. This results in much better perfornances, see below the timeline for 10 points requests.


Does libmaxminddb require geoip liabrary

Does libmaxminddb require geoip liabrary when we are compiling with nginx modsecurity

If you are referring to libGeoIP, no, it does not. That library was used to read from the legacy dat database format. libmaxminddb is used for reading from the newer mmdb format.


Unexpected identifier in import moment-jalaali

i see this bug in ssr mode
/Project/node_modules/react-datepicker2/dist/index.js:1
import moment$2 from 'moment-jalaali';
       ^^^^^^^^

SyntaxError: Unexpected identifier

@mehdiraized
I think It's something about the last changesets in the building method.
I'll check it soon.

@mehdiraized Can you test the latest changesets (v3.0.11)?
It seems OK after this commit https://github.com/mberneti/react-datepicker2/commit/5a68ee9a80ba1f214b362156299342bb21c7aef8


Unneeded  tags

http://forums.halite.io/t/unneeded-br-tags-in-visualizer/536/5

Here's a bit more information. The extra tags were added with commit 498882c28e36cc47d514700099e71838078f8979 which added a footer include to that and a bunch of other pages. The actual footer include was then pulled back out with commit 88ab4efc7d68ae4f1ca8c1b2d1e63cd2b50fc2ed but the extra tags got left.

Thanks @Janzert. Hadn't looked into it at all yet.


⚠ Aither has degraded performance

In 9bb54e9, Aither ($AIH) experienced degraded performance:

HTTP code: 200
Response time: 2532 ms

Resolved: Aither performance has improved in 65bd314 after 6 minutes.


⚠ Bofrost.be has degraded performance

In bddcae7, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1198 ms

Resolved: Bofrost.be performance has improved in 5a1cdf2 after 13 minutes.


Occult salt_user and salt_password options to clients in rundeck jobs

When using salt-step to execute jobs in rundeck via saltstack it is necessary to configure the salt_user and salt_password options in jobs.
Is the any way to occult to a client these two parameters from the jobs configured?
Until now, these parameters are visible to them and it is not necessary.
Thanks in advance.

Good morning,
on this subject, the only way that i found until now is to do 2 jobs,
one with salt_user and salt_password options, and another one (offered to the final users), that calls the first, so on this way to users dont see the options.
Do you know another option more easily to do this?
Any information would be appreciated.
Thanks.

@albertmfb how to config this?can you describe it clearly? thank you

+1, can anybody describe where the salt_user and salt_passwd would be written?
thx so much!

This screen shot might help you to store the username and password.

@openstackAnalyst  key storage is good,but every job use salt will define option about salt, how we can only define those in one job,and other job call it to get the salt options?


During test all, FAIL: metastore.tests.TestApi.test_show_tables

Describe the bug:
During ./build/env/bin/hue test all, metastore.tests.TestApi.test_show_tables fails like below.
======================================================================
FAIL: metastore.tests.TestApi.test_show_tables
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/deploy/hadoopeng/hue/build/env/lib/python2.7/site-packages/nose-1.3.7-py2.7.egg/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/deploy/hadoopeng/hue/apps/metastore/src/metastore/tests.py", line 96, in test_show_tables
    get.assert_called()
  File "/home/deploy/hadoopeng/hue/build/env/lib/python2.7/site-packages/mock-3.0.5-py2.7.egg/mock/mock.py", line 908, in assert_called
    raise AssertionError(msg)
AssertionError: Expected 'get' to have been called.

Steps to reproduce it?
./build/env/bin/hue test all

These tests fails only during integration test.
Those succeed when it triggered by test specific
Hue version or source? (e.g. open source 4.5, CDH 5.16, CDP 1.0...). System info (e.g. OS, Browser...).
open source 4.7.0
CentOS Linux release 7.7.1908
OpenJDK Runtime Environment (AdoptOpenJDK)(build 1.8.0_242-b08)

I debugged it.
The test user need to have permission to access the Metastore application.
Content-Length: 159
x-xss-protection: 1; mode=block
Content-Security-Policy: script-src 'self' 'unsafe-inline' 'unsafe-eval' *.google-analytics.com *.doubleclick.net data:;img-src 'self' *.google-analytics.com *.doubleclick.net http://*.tile.osm.org *.tile.osm.org *.gstatic.com data:;style-src 'self' 'unsafe-inline' fonts.googleapis.com;connect-src 'self';frame-src *;child-src 'self' data: *.vimeo.com;object-src 'none'
x-content-type-options: nosniff
Content-Language: en-us
Vary: Accept-Language, Cookie
X-Frame-Options: SAMEORIGIN
Content-Type: text/javascript

{
"message": "You do not have permission to access the Metastore application.",
"traceback": [],
"detail": null,
"is_embeddable": false,
"title": "Error"
}


Support for Hierarchical injection

This PR introduces support for hierarchical injection.  Key details

Can create as many scopes as needed

Scopes inherit their parents registrations

Scoped Injector and Root Injector have identical API.  (No specialization)

Support for disposal of hierarchy

Support for local scope cache

Support for local scope metrics

Support for passing the current injection scope to the external resolvers

Reduced dependency on global variables.

@bingenito If you fancy reviewing also appreciated.


Reproducible builds

https://github.com/sigstore/cosign/blob/8c9c02526c785a5acca63ad1957e60d06b6da24e/Makefile#L21
This makes it harder to have reproducible builds, is there a specific reason to have a date in the ldflags? I understand Git SHA.

+1 on taking this out!


GetFiles() using Glob curly braces gives empty result

What You Are Seeing?
Trying to get files using glob with curly braces with
GetFiles("Directory.Name/{package.json,package-lock.json,tsconfig.json,.npmrc}");
results an empty collection. Even a simple selection like
GetFiles("Directory.Name/{package.json}");
is empty. However executing a GetFiles() with each file like
GetFiles("Directory.Name/package.json");
works perfectly but is less convenient for many files and advanced filtering.
What is Expected?
Files should be filtered using curly braces. Curly braces should be supported since https://github.com/cake-build/cake/issues/2201
What version of Cake are you using?
0.35.0
Are you running on a 32 or 64 bit system?
64
What environment are you running on?  Windows? Linux? Mac?
Windows
Are you running on a CI Server?  If so, which one?
No
How Did You Get This To Happen? (Steps to Reproduce)
See above
Output Log
Logs show nothing

@cake-build/cake-team I can take a look at this tonight. Assigning it to myself.


6001 error with DRM protected DASH stream on Tizen 5.5 TV

What version of Shaka Player are you using?
4.3.8 and reproduced on 4.3.6
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Not tried.
Are you using the demo app or your own custom app?
Custom
If custom app, can you reproduce the issue using our demo app?
Using the demo app on desktop, I can't reproduce. The problem is specific to the samsung environment.
What browser and OS are you using?
Mozilla/5.0 (SMART-TV; LINUX; Tizen 5.5) AppleWebKit/537.36 (KHTML, like Gecko) 69.0.3497.106/5.5 TV Safari/537.36
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
Samsung The Frame QE50LS03T QLED 4K UHD Smart TV 50’’ 2020
Firmware T-NKMDEUC-2401.1
What are the manifest and license server URIs?
Sending through e-mail
What configuration are you using?  What is the output of player.getConfiguration()?
Sending through e-mail
What did you do?
I configured a DRM license server and I'm trying to play a protected DASH Live stream.
The same code base manages to run DASH unencrypted streams on my Samsung TV.
Also, the same code base manages to run DRM protected streams on my desktop (Chrome 114).
What did you expect to happen?
The stream should play.
What actually happened?
Shaka raises an error 6001 - "DRM.REQUESTED_KEY_SYSTEM_CONFIG_UNAVAILABLE".
I was assuming we had polyfills for older Chrome versions but they don't seem to be working here.

I did some additional testing with previous versions and the bug seems to appear from v3.2.0.
3.1.8 is able to play the stream as expected.

Hi ! I'm still facing the issue, could someone have a look? Thanks!

I see some differences between v3.1.8 and v3.2.0 in two significant areas:

a "content workarounds" class, which is used in particular to work around firmware issues on Tizen
the use of the MediaCapabilities API instead of the direct querying of EME for DRM content, which was optional and off by default for v3.1, and required in v3.2+

One other detail I see is that we don't trust the MediaCapabilities implementation on Tizen, and force Shaka Player to use our polyfill.  Is the application correctly installing Shaka Player's polyfills before instantiating the player?  (shaka.polyfill.installAll())
The MediaCapabilities polyfill wasn't forced into use for Tizen until v3.2.10, v3.3.9, v4.0.5, v4.1.3, and v4.2.0.  So any version before these would not be representative of what we are doing on Tizen today.

I'm unable to test on Tizen 5.5, since I don't have a device, so I'll need you to give us some feedback on these:

Does the issue reproduce on v3.2.6 (released same day as v3.1.8, with the same fixes)?  Because both branches were being maintained at the same time, v3.2.0 predates v3.1.8 by 9 months.
Does the issue reproduce on v3.2.21 (the final release of v3.2)?  This would have every available fix applied until the date when v3.2 was discontinued, and v3.2 had more maintenance releases than any other branch, over a period of 21 months.
Does the issue reproduce on v3.3.19 (final release of v3.3)?

Moving back to the latest code from main, please try these things:
In lib/polyfill/media_capabilities.js, there is a section particular to Tizen that looks like this:
        } else if (shaka.util.Platform.isTizen()) {
          let extendedType = videoConfig.contentType;
          if (videoConfig.width && videoConfig.height) {
            extendedType += `; width=${videoConfig.width}`;
            extendedType += `; height=${videoConfig.height}`;
          }
          if (videoConfig.framerate) {
            extendedType += `; framerate=${videoConfig.framerate}`;
          }
          if (videoConfig.bitrate) {
            extendedType += `; bitrate=${videoConfig.bitrate}`;
          }
          isSupported = Capabilities.isTypeSupported(extendedType);

If you disable this by changing:
        } else if (shaka.util.Platform.isTizen()) {

to:
        } else if (shaka.util.Platform.isTizen() && false) {

Does this change your results on Tizen 5.5?  If so, that would point to the platform rejecting some extended parameter of your content, such as resolution, framerate, or bitrate, and might make the failure a legitimate one.  That check didn't exist until v3.2.20, v3.3.18, v4.2.9, and v4.3.5.
There is one other thing specific to Tizen you could change.  If you change this section of the same file:
    let canUseNativeMCap = true;
    if (shaka.util.Platform.isApple() ||
        shaka.util.Platform.isPS5() ||
        shaka.util.Platform.isPS4() ||
        shaka.util.Platform.isWebOS() ||
        shaka.util.Platform.isTizen() ||
        shaka.util.Platform.isChromecast() ||
        shaka.util.Platform.isEOS() ||
        shaka.util.Platform.isHisense()) {
      canUseNativeMCap = false;
    }

by commenting out the line with shaka.util.Platform.isTizen(), does this change your results on Tizen 5.5?  If so, that would indicate some unknown bug in our polyfill, or that the polyfill isn't needed any more on Tizen 5+.

Thanks for looking into this Joey. I'll try these changes and report back in the next few days.

Hi Joey,
Installing polyfills with shaka.polyfill.installAll(); did the trick, I'm now able to play the DRM stream with v3.2 and the latest (v4.3.9). Given that, are the remaining tests required ?
Thomas


feat: add watermark to UI

Let us decide if we like the placement here:

And will not be displayed if isWatermark enabled is not true

Can we show the full date-timestamp rather than just time?

The placement is perfect. Can we show the full date-timestamp rather than just time?

The rate unit is xx/sec not xx/min.

The rate unit is xx/sec not xx/min.

Changed


[wip] support --json flag for pages publish

Fixes #2437
What this PR solves / how to test:
Adds a --json flag to pages publish command, that outputs the URL to the deployment.
This is still WIP but would be great to get some feedback on the approach here. The idea is to put the logger into a mode where it will not use stdout, and then any log messages that would have gone to stdout will be redirected to stderr instead, just leaving stdout for the json.

I wonder if the logger should become instance based and passed through to things that need it?
Or should we have a global --json flag that constructs the logger in a mode with stdout disabled to prevent any chance of something getting through before it's configured?
Should we stop using stdout and reserve that just for machine readable messages?

I noticed there's one other place already that already respects a --json flag in the cli.
Associated docs issue(s)/PR(s):

[insert associated docs issue(s)/PR(s)]

Author has included the following, where applicable:

[ ] Tests
[ ] Changeset (Changeset guidelines)

Reviewer has performed the following, where applicable:

[ ] Checked for inclusion of relevant tests
[ ] Checked for inclusion of a relevant changeset
[ ] Checked for creation of associated docs updates
[ ] Manually pulled down the changes and spot-tested

Thanks for this PR! Apologies for the delay here, but in the meantime we implemented a different way to do what this PR proposes. Setting the WRANGLER_OUTPUT_FILE_PATH env var to a file path will trigger Wrangler to write JSON-NL metadata about what it's doing to that file, including the Pages deployment URL. Because of that I'm going to close this PR for now, but feel free to open a new issue if that doesn't fully address your use-case.


Make cu_calculator a switch control

Feature Request
This function is not enabled when cu_calculator is closed, which can save a little resource, if necessary.

related link:
https://github.com/apache/incubator-pegasus/pull/318


Any help on custom dataset?

Hello,
I am absolutely struggling with how, to begin with, image inpainting and this seems like a good algorithm to work on, any help whatsoever would be appreciated.
Also, I tried to run the code but got errors.

You can post the error information

Before that, how should I prepare my dataset?

Put all your training images to the custom path, then change dataroot  to your custom path in https://github.com/Zhaoyi-Yan/Shift-Net_pytorch/blob/master/options/base_options.py.
I have updated Readme here:
https://github.com/Zhaoyi-Yan/Shift-Net_pytorch#train-models

Is it necessary for the training images to have mask or the mast is randomly generated?