Update ankushtripathii.json - Fixing location and profile

Description
I have found a bug that the profile for  Ankush tripathi Location is not set, also the view profile button is not working
Related Issues
#346
Changes Proposed
I have replace the your location with the users current location according to his GitHub Profile (Maharashtra , India)
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

@Bashamega Thanks for the contribution.


Latest chart does not support apiKey with parameter store

I recently set about upgrading our old scalyr chart which we had manually downloaded and modified because at the time serviceAccount annotations were not supported.
While serviceAccount annotations are now working, obtaining the apiKey from parameter store is not. We are used to our old manual deployment having this environment variable value:
    - name: SCALYR_API_KEY
      value: arn:aws:ssm:us-east-1:<account-id-redacted>:parameter/<path-to-key-redacted>

Then a mutating webhook for kube-secrets-init changes this to the actual parameter store value. However, there appears to be no way to do this in practice. The templates are hardcoded to use a secret, even though the README talks about using parameter store instead of a secret. I don't see a way to do this with the current chart.

@Kami Could you help look at this? This should be supported per instruction on README.
It looks like the serviceAccount was added here from v0.2.13, but the exact syntax to apply the apiKey from the parameter store isn't documented.

@cpetestewart Did you happen to have a chance yet to test the changes in #64?

I will be looking into this shortly. Apologies, I got pulled onto other projects.

@cpetestewart Great, thanks for confirming.
In the mean time I already had a chance to add some basic tests and docs to that PR so hopefully it doesn't need much more work besides a review to get it merged.

@cpetestewart https://github.com/scalyr/helm-scalyr/pull/64 has been merged and a new chart version with those change (v0.2.34) is now available - https://scalyr.github.io/helm-scalyr/.


OpenQASM3 paper refers to OpenQASM2 code while using indexed classical registers in conditional

(Not sure if this is the way to report an issue in the OpenQASM3 paper https://arxiv.org/abs/2104.14722 Feel free to close or move)
In page 20 of the paper OpenQASM 3: A broader and deeper quantum assembly language says:

As an example, consider an inverse quantum Fourier transform circuit as written in OpenQASM 2
<code>

In the code that follows, in line 14, it says:

if (c[0] == 1) p(-π/2) q[1];

This is not valid OpenQASM2. The same happens in lines 18, 19, and 23-25.
That created some confusion in the readers, like in https://quantumcomputing.stackexchange.com/questions/17901/if-statement-in-openqasm2-0-on-ibm-quantum-experience-error

Why is that not valid OpenQASM2? Seems more like an error in Qiskit's OpenQASM2 parser.

The if statement conditionally executes a quantum operation based on the value of  a  classical  register.   This allows  measurement  outcomes  to  determine  future  quantum operations.  We choose to have one decision register for simplicity.  This register is interpreted as  an  integer,  using  the  bit  at  index  zero  as  the  low  order  bit.   The  quantum  operation executes only if the register has the given integer value.

Open Quantum Assembly Language pg. 7 (emphasis added)

I see. Seems like the paper should reference 4 separate cregs to conform to OpenQASM2 syntax, then.

That was 4 years ago. We're all smarter in hindsight since then :)

Oh, wait, I see. You mean the current OpenQASM 3 paper, @blakejohnson


Error editing commands.json entry with allowParams missing

triggercmd edit
...
? Select a command: python test
? Type the command name: python test
? Type the command: python /root/test.py
? Select a command ground: background
? Type the command voice: python
Error: 'allowParams'
This is my commands.json entry.  Notice it doesn't have an allowParams value.
{"trigger":"python test","command":"python /root/test.py","ground":"background","voice":""},
The agent is ok with a missing allowParams value.

Hi @rvmey, thanks for your comment. This is a really problem. I will fix this and release again.


⚠ Bofrost.lu has degraded performance

In 636b0ab, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1103 ms

Resolved: Bofrost.lu performance has improved in 4d6d290 after 1 hour, 37 minutes.


Translate Price toggle control added in admin panel on channel setting

Not all stores want their prices to be translated into different locales, this option allows admin to choose if they want to keep the prices as digits or translate them into the chosen locale.
Issue Reference
#7278
Description
The website admin is not able to disable price translation.
How To Test This?
Enable multiple locales for a channel.
Switch the locale from the website front.
The price will get translated into the chosen locale.
for example:
For English, the price will look like AED 100.00
But if the customer switches to Arabic, it looks like ١٠٠٫٠٠ AED
It is clearly undesirable because no eCommerce store in the middle east wants to show prices like this.

Hi there,
Thanks for your contribution.

This feature should be implemented in the currency section rather the directly added to channel.

@tahiryasin
Please resolve the conflicts.

I am closing this, as this is not in our roadmap.


Quit doesn't quit

Current Behaviour (for problems)

open data curator
quit
OK quit
Application remains open

Expected Behaviour

open data curator
quit
OK quit
Application quits

Your Environment
.

Data Curator version: 0.13.0
Operating System and version: macOS High Sierra 10.13.3

+1 for this.  It's annoying when I have to go into Task Manager to close the application (when I close Data Curator and then try to open again via the icon, I see a flash and then it doesn't open... it only works again when I kill it in Task Manager)

@pripley123 this should be fixed in the next release later this week

Great to hear it's getting fix.  Thanks for letting me know.

From: Stephen Gates<EMAIL_ADDRESS>Sent: Wednesday, April 11, 2018 12:25:04 PM
To: ODIQueensland/data-curator
Cc: Paul Ripley; Mention
Subject: Re: [ODIQueensland/data-curator] Quit doesn't quit (#616)
@pripley123https://github.com/pripley123 this should be fixed in the next release later this week
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ODIQueensland/data-curator/issues/616#issuecomment-380567635, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Af-qF8ncQG1m5T_XQ3KEoHGMDHVXMSAnks5tnliQgaJpZM4S_Sno.

Fixed in release 0.14.0.

Works for me now!


azure.core.exceptions.ServiceRequestError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:1091)

Hi,
I am trying to authenticate to a secure Azure ML workspace using an SPN. And the module is failing for a few team members while it is working for some of them.
Error Message -
Traceback (most recent call last):
File "C:\Users\user_name\Anaconda3\envs\env_name\lib\site-packages\utils\auth\azkvsecret.py", line 46, in get_az_kv_secret
secret_value = client.get_secret(name)
File "C:\Users\user_name\Anaconda3\envs\env_name\lib\site-packages\azure\core\tracing\decorator.py", line 83, in wrapper_use_tracer
return func(*args, **kwargs)
File "C:\Users\user_name\Anaconda3\envs\env_name\lib\site-packages\azure\keyvault\secrets_client.py",
line 77, in get_secret
**kwargs
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\keyvault\secrets_generated_operations_mixin.py", line 1574, in get_secret
return mixin_instance.get_secret(vault_base_url, secret_name, secret_version, **kwargs)
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\tracing\decorator.py", line 83, in wrapper_use_tracer
return func(*args, **kwargs)
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\keyvault\secrets_generated\v7_3\operations_key_vault_client_operations.py", line 697, in get_secret
**kwargs
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline_base.py", line
211, in run
return first_node.send(pipeline_request)  # type: ignore
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline_base.py", line
71, in send
response = self.next.send(request)
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline_base.py", line
71, in send
response = self.next.send(request)
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline_base.py", line
71, in send
response = self.next.send(request)
[Previous line repeated 2 more times]
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline\policies_redirect.py", line 158, in send
response = self.next.send(request)
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline\policies_retry.py", line 467, in send
raise err
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline\policies_retry.py", line 445, in send
response = self.next.send(request)
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline\policies_authentication.py", line 119, in send
response = self.next.send(request)
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline_base.py", line
71, in send
response = self.next.send(request)
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline_base.py", line
71, in send
response = self.next.send(request)
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline_base.py", line
71, in send
response = self.next.send(request)
[Previous line repeated 1 more time]
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline_base.py", line
103, in send
self._sender.send(request.http_request, **request.context.options),
File "C:\Users\username\Anaconda3\envs\env_name\lib\site-packages\azure\core\pipeline\transport_requests_basic.py", line 361, in send
raise error
azure.core.exceptions.ServiceRequestError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:1091)
## get_az_kv_secret function defined in utils
from azure.keyvault.secrets import SecretClient
from azure.identity import DefaultAzureCredential
from azure.identity import ClientSecretCredential
def get_az_kv_secret(
az_kv_name: str='kvname',
secret_name: list=['keyname']
) -> dict:
tenant_id = "TENANT_ID"
client_id = "CLIENT_ID"
client_secret = "CLIENT SECRET"

secret_dict = {}
kv_uri = f"https://{az_kv_name}.vault.azure.net"
credential = ClientSecretCredential(tenant_id=tenant_id, client_id=client_id, client_secret=client_secret)
client = SecretClient(vault_url=kv_uri, credential=credential)

for name in secret_name:
    **secret_value = client.get_secret(name)** ##Failing at this step
    secret_dict[name] = secret_value.value  # type: ignore

return secret_dict  # type: ignore

While this is working fine for a few team members, it is failing for some of them when they run it from their local conda environments.
Below are the environment details

python==3.7.13
azure-identity==1.7.0
azure-keyvault-secrets==4.6.0
azure-core==1.22.1
azureml-core==1.42.0.post1

Some of the packages are installed as dependencies for azure sdk.
Following the spn authentication mechanism from here for AML workspaces - Authentication in AzureML

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Identity:0.41671595,KeyVault:0.33573103,Storage:0.038466632'

Thanks for the feedback, we’ll investigate asap.

Hi @xiangyan99 @mccoyp any updates on this?

Hi @pankajkumar002, sorry about the delayed response. I was out of office last week, but I'll look into this and provide an update as soon as possible.

Hi again @pankajkumar002 -- after looking into this more, I'm pretty confident that this isn't particular to Key Vault. My guess would be that requests to different endpoints might fail with the same error, but correct me if I'm wrong.
I wouldn't expect that the Key Vault service is providing an invalid, self-signed certificate, so I think this should be a local configuration issue. Online searches suggest that installing the pip-system-certs (conda-forge link) package (which replaces python-certifi-win32) may fix this problem.
Synchronous KV requests use requests under the hood, so the sections discussing certificates (starting here) in their docs may also shed some light on what may be the issue.
Let me know if installing pip-system-certs in problematic environments helps! You mentioned conda, so you may need to use
conda install -c conda-forge pip-system-certs

Hello, I am experiencing the same error. Has it been confirmed installing pip-systems-certs resolves the issue.
If not how could it set SSL verify to false through the Azure Python SDK?

Hi @jdickson-li, I'm unfortunately not sure if pip-system-certs solves the issue since we haven't heard back on the original issue. I encourage you to give it a try and let us know if it helps, though!
As for disabling SSL certificate verification, that can be done by providing a custom transport to a client. The client library developer guide in azure-core describes how to do this: https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/core/azure-core/CLIENT_LIBRARY_DEVELOPER.md#transport
By default, Key Vault clients currently use a RequestsTransport from azure.core.pipeline.transport. This transport accepts a connection_verify keyword argument, as described in the guide above, which is True by default but can be set to False. This will disable SSL certificate verification, but you should only disable this if you're sure about the security of your connection. You could use this with a SecretClient like so:
from azure.core.pipeline.transport import RequestsTransport
from azure.keyvault.secrets import SecretClient

transport = RequestsTransport(connection_verify=False)
client = SecretClient(<vault_url>, <credential>, transport=transport)

Hey, @mccoyp Thank you for your response. The pip-system-certs library did not work for me so I ended up using the request library and calling the REST API directly, which worked.
Thanks for sharing the doc details that will be useful to know for the future.

It seems that the error occurs due to the unstable of the connection.
I introduced the mechanism of retrying inside the prepdocs.py and works fine for now. When the certificate verify failed, it will pass in the second trial.
Here is what I have modified. args.start_index is set once the progress is shut down, I can continue the progress from specific index.
def process_file(filename):
    if args.verbose: print(f"Processing '{filename}'")
    if args.remove:
        remove_blobs(filename)
        remove_from_index(filename)
    elif args.removeall:
        remove_blobs(None)
        remove_from_index(None)
    else:
        if not args.skipblobs:
            upload_blobs(filename)
        print(f"Processing '{filename}'")
        page_map = get_document_text(filename)
        sections = create_sections(os.path.basename(filename), page_map)
        index_sections(os.path.basename(filename), sections)

if args.removeall:
    remove_blobs(None)
    remove_from_index(None)
else:
    if not args.remove:
        create_search_index()

    print(f"Processing files...")
    # Recursively find all files in the directory
    filenames = glob.glob(args.files, recursive=True)

    # Filter out directories
    filenames = [f for f in filenames if os.path.isfile(f)]

    # Maximum number of retry attempts
    max_retry_attempts = 3
    args.start_index = 0

    start_index = args.start_index if hasattr(args, 'start_index') and args.start_index < len(filenames) else 0

    for i in tqdm.tqdm(range(start_index, len(filenames)), desc="Processing files", unit="file"):
        filename = filenames[i]
        if os.path.isfile(filename):
            retry_attempts = 0
            while retry_attempts < max_retry_attempts:
                try:
                    process_file(filename)
                    break  # Break the while loop if processing was successful
                except Exception as e:
                    print(f"Error processing file {filename}. Error: {str(e)}. Attempt {retry_attempts+1} of {max_retry_attempts}")
                    retry_attempts += 1


docs(forms): switch to aria-describedby

Description
aria-labelledby is wrongly used here since it overrides existing label: either we add label's id as the first value in aria-labelledby, or we switch to aria-describedby as done in this PR.
In both cases, I think the callout before should mention this difference between the two attributes.
Friendly ping @patrickhlauke as you may have another recommendation. :)
Type of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Refactoring (non-breaking change)
[ ] Breaking change (fix or feature that would change existing functionality)

Checklist

[x] I have read the contributing guidelines
[x] My code follows the code style of the project (using npm run lint)
[x] My change introduces changes to the documentation
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

Live previews

https://deploy-preview-{your_pr_number}--twbs-bootstrap.netlify.app/

I checked the forms page without luck but didn't check examples or cheatsheet.

ah, now i remember where that came from... https://github.com/twbs/bootstrap/pull/37587
I was actually not convinced by that PR myself, but it was merged anyway. I would still say that the assertion at the basis of that PR is...debatable. I would suggest just wholesale reverting that PR, to be honest.

i.e. also remove the paragraph that says authors should use aria-labelledby ...

Friendly ping @hannahiss and @aniort. Might be interested in the topic.

I would propose to add label's id as the first value, since using aria-labelledby for mandatory information could be a good idea...?

While I understand the "could be a good idea" part, I personally think this ends up being massively overengineered for what it actually is/does (particularly for a generic example in our documentation). It's been generally accepted practice to use aria-describedby to link additional info and instructions for a form field. The nuance of "if this is mandatory, it should be in the accessible label, not in the accessible description" is one that I'd say should be left up to authors to decide. Any more information on the original assertion that aria-labelledby is better supported than aria-describedby?

Please don't revert totally https://github.com/twbs/bootstrap/pull/37587 because the first part (in input-group) is still needed.

ah yes, good point. agree, that part is non-controversial and correct

Agreed, if it's that much important it should probably be in <label>, not passed as a potentially wrong label override.

I just reverted the callout changes from #37587, and also improved labels in select sizing examples—which previously mentioned class names…


Update WebRTC build

New version is at commit 1a35fbd9c38cbf2f454b4808d28e2ca0e93853cf (current master).
https://chromium.googlesource.com/external/webrtc/+/1a35fbd9c38cbf2f454b4808d28e2ca0e93853cf

Yes

Updated to Release 71 (https://groups.google.com/forum/#!topic/discuss-webrtc/HUpIxlDlkSE).
Tested briefly. Works fine so far. (Requires some changes on the user side.)


mcevents: fix the size of the fields, add missed semicolons

Size of the fields msg and label аre mixed up.
Truncate label size to 32((EDAC_MC_LABEL_LEN + 1)), increase MSG size to 64.
Add missed semicolons.

@AlexZzz do you mind taking a look?

32 bytes is really enough for the label field.
msg field's size is a driver-specific one. In most cases messages are quite short, so maybe it's better to keep it 32 bytes long?

I checked again. Yes, you seem to be right. The original msg size is returned.

Thanks, looks good.


Is this project in maintenance mode? What's the future?

What is the situation with this extension? Is this functionality going to be incorporated into Windows App SDK and Visual Studio?
I'm asking for two reasons:

There is an issue with it not providing a way of creating runtime classes clearly. The only way I see is to use the terribly named and even more terribly described ViewModel template. So should we look here for a fix or in the Windows App SDK or Visual Studio?

There is an issue with the lack of an automated configuration system in VS projects, which means this is an extra dependency that needs to be communicated to a recipient of a VS project, as they need to manually install it. That is a huge friction point for some scenarios, such as education settings, where we can't expect markers to have this extension installed, and can't expect them to install it. So again, knowing the fate of this extension helps understand how this can be fixed. Will this issue be fixed as a natural consequence of it's functionality being integrated into WindowsAppSDK / Visual Studio? Or is this extension going to be maintained going forward, making the existence of this extension a blocking issue for using C++/WinRT in education settings.

The Windows App SDK VSIX extension provides templates (including a runtime class one) for WASDK apps. C++/WinRT's templates are for UWP.
The C++/WinRT VSIX extension is not needed for building and running C++/WinRT UWP or WA SDK-based apps apps. The extension only provides templates for UWP projects. All the build support is in the NuGet package, which does get automatically acquired and installed when building a project.
There is work to integrate the C++/WinRT extension into Visual Studio being done, but my guess is that it's being done for LTS, not any future improvement. All the investment is in WA SDK now.

OK, thank you. I had some pretty serious issues with trying to use a UWP app for a Uni project. Which I talked about over on Discord and in a forum. My Uni doesn't have any configuration or workloads installed for UWP, and I think I really pissed off my teacher with the installs and configuration needed - like him having to install UWP workload on his surface tablet to debug my project. And he was even annoyed by having to set Dev mode in Windows. I don't see the fuss myself, but I'm just the messenger here. It seems to be difficult to expect people to perform these manual setups just to debug a project.
I'm happy to hear that integration for the Windows App SDK scenario is going on. I will probably be using that eventually. Just waiting for it to be ready really.


[fix] [client-consumer] NotifyPendingReceivedCallback when zeroConsumerImpl received batch message firstly , then close consumer.

Motivation
NotifyPendingReceivedCallback when zeroConsumerImpl received batch messsage firstly , then close consumer.
Documentation

[x] no-need-doc

@BewareMyPower @michaeljmarshall cc

Currently, zero queue consumer cannot receive batched messages. It is intended, otherwise it should be a consumer whose queue size is 1.
The ZeroQueueConsumerImpl#receiveIndividualMessagesFromBatch implementation is just for this case. See ZeroQueueSizeTest#testFailedZeroQueueSizeBatchMessage.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

@RobertIndie could you help to check it again.

/pulsarbot run-failure-checks

Since we will start the RC version of 3.0.0 on 2023-04-11, I will change the label/milestone of PR who have not been merged.

The PR of type feature is deferred to 3.1.0
The PR of type fix is deferred to 3.0.1

So drag this PR to 3.0.1


Remove doubled ActorSystem

We've got two ActorSystem delivered by actorSystemLayer and actorSystem layers.

I'll work on this


Preferred video quality settings

Describe your suggested feature
It will be really helpful to add a setting by which it will be pre defined to open any anime in needed quality.
If using external player than it open it in maximum quality all the time.
Right now in inbuilt player it open it in maximum quality than i have to select needed quality.
Having a setting for preferred quality settings will save time and data.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to the latest version.
[X] I will fill out all of the requested information in this form.

Pretty much all the extensions have "Preferred Quality" option now.
See jmir1/aniyomi-extensions#79


Use requirements.txt to install Binder Python dependencies

Binder recognizes a variety of configuration files to prepare the runtime of the Binder for the user during the build so that no additional actions need to be taken. To avoid having a Binder user having to do any work to install libraries with Jupyter magics in the notebook this PR uses the Binder requirements.txt config file to install any Python libraries not included in the stdlib. This also harmonizes the notebook between running locally and in Binder, which is preferable.
For an example, see the Binder running on this branch on my fork:
I realize that this repo was started to do work and try things out, so the number of libraries in the requirements.txt might need to expand in the future, but I thought it might be helpful to start using it now rather than when this exploitative project is done.

@cranmer this is ready for review. Let me know if you have any suggestions or requested changes.


[BUG] Custom console url stripping characters

Describe the bug
We use Royal TS to manage and connect to remote devices/services. The app supports uri handler for rtsx:// When we add the uri of rtsx://vnc%3a%2f%2fUser@server-x.domain.local?property_Port=1234&property_Description=Server%20X as a console URL for a host, the colon in the uri gets stripped out when clicking on the console button from the UI. The browser opens a new tab trying to connect to rtsx//vnc%3a%2f%2fUser@server-x.domain.local?property_Port=1234&property_Description=Server%20X
To Reproduce
Steps to reproduce the behavior:

Edit a host
Add URI as described above as Console URL and save.
Click on Console button for same host.

Expected behavior
Browser opens unmodified url.
Environment

Harvester ISO version: v1.2.2
Underlying Infrastructure: Dell PowerEdge R640

Confirmed, this reproduces on v1.3.1(-rc1) as well.
This commit introduced code that enforces that console URLs must always start with http:// or https:// or else http:// will be prepended to the console URL:
https://github.com/rancher/dashboard/pull/6806/commits/bc5523bed18a7b11668d4082069b1aca3e2d8a34
    if (!url.startsWith('http://') && !url.startsWith('https://')) {
      return `http://${ url }`;
    }

I'm not sure why this check was introduced. I suppose support for custom protocol schemas isn't well established between different browsers and versions?
There is an RFC describing a possible vnc:// URL schema as well, so to me it makes little sense to limit the URL schemas to http:// and https:// only. While the vnc:// URL schema is recognized by IANA, the rtsx:// schema is not. Nevertheless, limiting the possible URLs to those with http:// or https:// schemas is overly restrictive IMO.

If they really wanted to verify it was a URL with a schema, so to avoid doing something like /some/path/in/harvester; it could just verify that ^[a-z]+:\/\/ matches the url string.


Morning session M2 D2

Hi @L2oukmane  :wave:
Great job on implementing this awesome project with all the colors
The project looks really good and everything is working normally.

To be honest, we didn't find any issue with your project. Everything is okay.

The code is highly clean and organized.

Features are working properly.

GOOD JOB once again and happy coding :smile:

Hey @L2oukmane Great Work this project looks awesome and am hear to confirm your clean code writing skill


docker: push purge images

skip skydive-compile-tests
skip skydive-functional-tests-backend-elasticsearch
skip skydive-functional-tests-backend-orientdb
skip skydive-go-fmt
skip skydive-k8s-tests
skip skydive-scale-tests
skip skydive-selenium-tests
skip skydive-unit-tests

run skydive-ppc64le-tests


numeric values should be "trimmed" before conversion when importing CSV

trying to load a file and I get the error:
Error: failed: line 41203: column 'duration_sec': strconv.ParseFloat: parsing "2631600  ": invalid syntax
csv files are pretty messy, so we might just want to trim the string values before trying to convert them to a numeric type.

Can you pls also share the CSV data? I could take it as an example for a unit test.

Sure!
influx write -f /Users/rsavage/Downloads/ufo-sightings/error_on_line_31.csv --skipHeader=1 --header "#constant measurement,ufos" --header "t|dateTime:1/2/2006 15:04,city|tag,state|tag,country|tag,shape|tag,duration_sec|double,duration_str|string,comments|string,date_posted|ignored,latitude|double,longitude|double" -b ufo_data
error_on_line_31.csv.zip

The code that originally fixed the issue (4f1d20b) is now part of pkg/csv2lp library, the functionality is validated by a unit test in https://github.com/influxdata/influxdb/blob/master/pkg/csv2lp/data_conversion_test.go#L255 .


Fixes typo: general initializer => global initializer.

That's what happens when you write documentation too fast and late at night ;)

Squeaked it into the release by 7 minutes. :)

Squeaked it into the release by 7 minutes. :)


Fix non-compile error

Added CMakeLists.txt for compilation with CMake and any IDE that supports CMake.
Removed const qualifier from function parameter where function modifiers parameter.

This fixes #23

Could be an idea to split out the compilation error fix as a separate PR which will be a no-brainer to accept?

This should be covered by PR #24


Incorporate Google's CityHash successor: FarmHash

@nickbabcock has a .NET implementation at
https://github.com/nickbabcock/Farmhash.Sharp

Thanks for the shout out. I won't speak for brandon, but I believe our two libraries may have a different focus. Farmhash.Sharp emphasizes speed whereas Data.HashFunction emphasizes completeness, safety, and ergonomics. This is apparent in benchmarks where Farmhash.Sharp exhibits 10x throughput over 64bit Cityhash from Data.HashFunction*. I've included a heatmap of relative throughput with Farmhash.Sharp and Data.HashFunction highlighted.
While I certainly encourage Data.HashFunction to incorporate an implementation of Farmhash, if your sole purpose for a non-cryptographic hash function is throughput, I'm skeptical a safe implementation in Data.HashFunction will match performance.

*For 64bit RYU JIT platform -- arguably the most common platform

Thanks @nickbabcock, great info here. As an aside, I wonder how much of that 10x throughput advantage of FarmHash over CityHash comes from differences in the underlying algorithm.
Back on topic though, after taking a peek into this libraries' code I see totally managed implementation. So I agree with you on the different approaches. Alas, if either of us come across a managed FarmHash implementation let's meet back here, deal?

Deal 😄
Farmhash.Sharp shouldn't be 10x faster than an equivalent .NET Cityhash implementation as both are very similar. Farmhash (by google) was designed to take advantage of modern hardware (SSE 4.2 instructions). I'm banking on .NET to eventually implement these intrinsics -- then you'll see Farmhash.Sharp throughput dramatically increase for large payloads.

@nickbabcock is exactly right about the differences between the overall focus between the two libraries.
I'd love to add an implementation of FarmHash to Data.HashFunction, but in general if one is looking for the absolute fastest implementation of a given hash function, they will likely be able to find a more specialized library that could provide better performance.
Keeping this ticket open to track progress on said implementation.

@nickbabcock @NickStrupat I was able to implement the Fingerprint32, Fingerprint64, and Fingerprint128 methods of FarmHash.
@nickbabcock I went to create a binding implementation for FarmHash.Sharp but found that you're not strong name signing your assemblies -- I don't want to start the signing versus non-signing debate that's been done many times.  I personally have chosen to sign Data.HashFunction with a project-specific private key that has been checked in publicly (and therefore is no longer private).
I can create my binding implementation as non-signed, but you might consider signing your assemblies, even if it is just with a unprotected signing key. See https://docs.microsoft.com/en-us/dotnet/framework/app-domains/strong-named-assemblies , for benefits and their OSS recommendation (literally the last sentence).

I'm not well versed in the strong name signing debate, but a quick search corroborates an unencrypted, checked-in private key fixes most issues. I've created an issue for myself to track and complete!


Clarification question changes for DOS4

Trello: https://trello.com/c/3U1ORXqu/568-dos4-clarification-question-content-improvements
We're removing the post-clarification-deadline 'followup' question functionality. Instead we are directing suppliers to the Help page where they can contact support. This new process has been agreed with CCS.

Stefan has also supplied some new content that should help reduce the number of questions we receive (see ticket).

Functional tests here: https://github.com/alphagov/digitalmarketplace-functional-tests/pull/661


add functions to create random numbers

rand creates pseudo-random numbers, while secure_rand32/64 produce cryptographicly secure random numbers

bors ping

bors try

bors r+

bors r+

I assume that @josephlr like the patch an accept it

Does this patch actually generate random numbers?

It should generate random numbers. The function calls implicitly generate_random_number64, which uses Intel's rdrand instruction to generate random numbers. See also _rdrand32_step.

okay great I have getrandom working on x86_unknown_hermit. Thanks for the pointer, although is there a solution for aarch64?

Currently not, but we are working on it.

Okay so my patch to getrandom will only support x86_64-unknown-hermit.
Thanks for your assistance!


Regenerating credits to cap paid API usage

When using paid APIs, will need to control for API usage. One way to do this is:

Have some kind of user registration based on email and confirmation of emails.
Track usage to limit API calls over some period of time.. e.g. 24 hours, 72 hours, or something similar. Have the credit regenerate of that window of time.
Indicate the user credits remaining in the UI
Optionally, allow users to provide their own API key for unlimited usage.

Code might currently require adding your own API keys. I think we can add one of our own OpenAI keys and cap the spend on the account settings.

Still to be done. Should be able to track the token count through API calls, but then will need to add the UI to display to the user.

Added capability in apps/ai_tutor


Enter API key for Premium

Hello,
I noticed that you have listed ways to use the public, and enterprise keys.
' Domain and URL for Google API
Public Const gstrGeocodingDomain = "https://maps.googleapis.com"
Public Const gstrGeocodingURL = "/maps/api/geocode/xml?"
' TODO: Add support for Data Science Toolkit, Bing Geospatial, etc.
' set gblnBusiness = 1 to use the Enterprise Geocoder (requires clientID and key)
Public Const gblnBusiness = 0
' clientID and key from Enterprise Geocoder
Public Const gstrClientID = ""
Public Const gstrKey = ""
Do you know a way to use a premium api key?

@mkeller3 trying to look into it this weekend

@mkeller3 it looks like i may need to try and create a premium api key setup to get this working, so i will need to investigate a little bit more - thank you for letting me know

Ok thanks!

Any luck? If you create a free api key that would have the same output.

I signed up for a Free API Key and did some testing. I just committed a bit of functionality on the .bas file to switch between the Enterprise Geocoder (client ID + key), the Premium plan (key only) and the free geocoder.

Great!


test: create Demo page similar to simple Müsli page

To test and see how the widgets and the database connection is working, we want to add such a page for demo

We did create the demo page at the meeting at 04.07. successfully.


#3 Possible memory leak because of Ticker

PR for Issue #3
Added the Ticker Stop()
Go test results:
~/goexpect$ go test
E0821 14:02:26.983666   11558 expect_test.go:135] Accept failed: accept tcp [::]:32844: use of closed network connection
E0821 14:02:27.002522   11558 expect_test.go:203] PTY cols/rows: 240/22 want: 120/40
E0821 14:02:27.002759   11558 expect_test.go:135] Accept failed: accept tcp [::]:41063: use of closed network connection
PASS
ok  	_/home/marek/goexpect	11.224s

I signed it!


fix(setting): create a real SC for the setting "default-longhorn-static-storage-class"

Which issue(s) this PR fixes:

Issue # longhorn/longhorn#6446
What this PR does / why we need it:
Special notes for your reviewer:
Additional documentation or context

@mergify backport v1.6.x v1.5.x


use group name instead of sid for current_resource in the group resource on windows

Using the SID makes things look not so nice from a reporting perspective
Fixes #3411

ping @chef/client-windows for review please!

My concern would be that you're changing the semantics of the field and maybe introducing issues comparing between group names and sids between new_resource and current_resource.  Generally we follow the pattern that if they can be different types that the current_resource must load the same type as the new_resource specified so that the comparison worked.

@lamont-granquist if you take a look at the issue, it seems that new_resource is using a human readable name, and current_resource is using the sids. Before, the compairson happened by converting new_resource to sids. Now, we keeps the name and sid for current_resource, but still compare everything with sids. (I think, it's been a while)

Okay, LGTM :+1:

@chef/client-core @chef/client-windows needs another :+1:

So I'm not sure this is safe. If you convert the sid to a name, it's not clear to me that it will be unique. does the name include the domain? And even across forests, you could have the same name, so unless the name format is UPN, you could hit issues when there's a cross-forest trust.
If this is just reporting, can't we just change what reporting outputs? Alternatively, could we use something that contains both the username AND the sid? Then it's human readable, but we won't hit name collisions in the idempotence logic.

Closing due to old age.


Add events for video, audio, and data.

This allows writing a relay without using ffmpeg (by using a client publisher to directly write to an RTMP endpoint by relaying this packet).
Example code for what this looks like:
const config = {
  rtmp: {
    port: 1935,
    chunk_size: 60000,
    gop_cache: true,
    ping: 30,
    ping_timeout: 60
  },
  http: {port: 8000, allow_origin: '*'},
  logType: 3
};

var nms = new NodeMediaServer(config);
nms.run();

// This is another NMS or RTMP server.
let rp = new NodeRtmpClient('rtmp://localhost:1936/live/stream');
rp.startPush();

nms.on('video', (id, response) => {
  console.log(
      'got packet for ', id, 'with timestamp', response.timestamp)
  rp.pushVideo(response.payload, response.timestamp);
});

@nsthorat you should not commit with message "save", instead you should write a description of the changes.
In 15 years of git, your the first person i see doing this, and god know i have see lame commit messages.

We have squash and merge for a reason, @hthetiot.


Question about max files/folders

Greetings,
I am working with a hard drive that has approximately 700,000 files and 12,000 folders. (4TB)
Should DeltaWalker be able to handle those quanities without freezing up? Are there settings I need to check on my Windows 11 machine?
Kind regards,
Dennis

your issue is unrelated to Nextra

Just so you know, their website:
https://www.deltawalker.com/documentation
has a link to this:
https://github.com/shuding/nextra/issues/new?title=Feedback for “Welcome!”&labels=feedback

Contact them via other way


event request for modify the email body in approval & cancel notification emails

Hi everyone
during development, I need an event to modify the details of email body

Report Name : 1320 "Notification Email"
function name: SetReportFieldPlaceholders
event location : in the last of this procedure where i  can modify the details (variables ) like Field1Label,Field1Value, Field2Label, Field2Value etc.....

Please always put the full detailed signature of the requested events.
You help us implementing the requests faster when your request contains AL code copied from VSCode instead of screenshots.

@StanPesotskiy  here is the signature of event

You help us implementing the requests faster when your request contains AL code copied from VSCode instead of screenshots.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

The problem I see is that the event publisher OnSetReportFieldPlaceholders() passes Field3Label, Field3Value as var to subscribers, allowing us to set those fields to our own values - but then the case "Notification Entry".Type will override them in most cases, i.e. when Type is Approval.
A better fix might be to add a var Handled: Boolean argument to OnSetReportFieldPlaceholders(), and only enter the case if we have not Handled setting the fields in a subscriber. But I guess I'll wait and see if this change lands and is sufficient for us.

@dtkb
This request was closed but you added a new comment. We do not track the closed requests so your comment may be not registered and answered. If you want to continue discussion please create a new request and put a link to the closed one.

@dtkb
This request was closed but you added a new comment. We do not track the closed requests so your comment may be not registered and answered. If you want to continue discussion please create a new request and put a link to the closed one.

@StanPesotskiy done; see link in mention above.

@StanPesotskiy done; see link in mention above.


ideas and suggestions.

Hi. please add more functionality like adjustability, add/remove/adjust effects, change/toggle/on-off audio/video devices. i tried to install the pre-release but don't work. i like this plugin. is super useful but need more polish and functions. please consider every option like Elgato Camera Hub plugin as example.
Thank you for your great work, will help a lot much people include me.

Hi Danitza,
This is complete and is currently submitted to Elgato marketplace. It should be available for you to use via marketplace update in a few days depending on their review. When I made the first version, I wasn't sure if anyone else would use it, but I'm happy to hear that you like it!
If you want to try the pre-release, copy the contents of the zip into
%APPDATA%\Elgato\StreamDeck\Plugins\com.zaphop.nvidiabroadcast.sdPlugin
And then restart StreamDeck (right click on taskbar StreamDeck icon, click Quit), or reboot your computer.
Thank you for your suggestions and kind words!

it worked, but, still being bad integration, and still don't like it, needs too much job for now.
what you have to do, is set 2 slots for each category tab on the broadcast app, so in that way, we can have bottons to change the effects of each slot, and see it reflected on the slot buttons into the Stream Deck, and press each slot button to toggle on/off. the idea is when a effect is asigned to a slot (i will refer to the effects button toggle like this), there's text to see which effect is asigned and if it's on or off.
You can add a button type/category on the plugin, where in his options, we can assign what we want, i mean, a drop down bar which contains the Nvidia Broadcast specific tab, Camera/Microphone/Speakers separately, to choose one of them, and another bar, which controls which effect we wish, and to which specific slot we want it. also the hability to assign a button to adjust them in + or - that is we need. i'm spanish ¿it's understandable?. you can text me through discord with same username without ''#''.  i dunno how much explain me better but i will cooperate to convert this into something awesome and useful. i'm very interested and i will be using it so much. kisses.

i mean something like this and so on.

long press on Toggle to access to effect adjustability of slot 1 and/or 2. or just simply assign another page or moar buttons if the deck is more bigger to fit them. this is my setup for audio and cameras. i use voicemeeter banana with the deck.

i hope you get some inspiration from this.

Can we talk on discord? text me. same name without #.


Issue when using BufferedImage#getRGB throwing IndexOutOfBounds

So I'm a friend of the user Hydos. First off he is apologizing being harsh. I'm just posting his issue with the requested stacktrace. He really does need this issue fixed regardless of your prior interactions. His issue report starts after the line break.

First of all, here is the repo  before I leave any information out. The usage of JXLatte is in MaterialUploader.java You can use GitHub's "Go to File" Feature to find this. Here is the complete stack trace from my program

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index -144 out of bounds for length 65536
at java.desktop/java.awt.image.ComponentColorModel.getRGBComponent(ComponentColorModel.java:905)
at java.desktop/java.awt.image.ComponentColorModel.getRed(ComponentColorModel.java:946)
at java.desktop/java.awt.image.ComponentColorModel.getRGB(ComponentColorModel.java:1137)
at java.desktop/java.awt.image.BufferedImage.getRGB(BufferedImage.java:918)
at com.thepokecraftmod.rks.test.load.MaterialUploader.mergeAndLoad(MaterialUploader.java:74)
at com.thepokecraftmod.rks.test.load.MaterialUploader.lambda$new$0(MaterialUploader.java:41)
at com.thepokecraftmod.rks.test.load.MaterialUploader.upload(MaterialUploader.java:158)
at com.thepokecraftmod.rks.test.tests.PokemonTest.main(PokemonTest.java:43)

https://cdn.discordapp.com/attachments/1080332031914950716/1097396030779166820/body_a_mtl.jxl
Here is an image file as well which was being opened at the time to cause the issue.
If you need any more info I request you ask for it.

It looks like this file has out-of-range "colors" (I say "colors" because it is grayscale), which is causing issues with the built-in linear grayscale profile's LUT. If you call JXLImage.fillColor() before calling asBufferedImage() and then you draw it on another sRGB BufferedImage (which is the default), it should clamp these values. That being said, I probably should add a clamp() method to JXLImage for convenience, or have it automatically clamp before calling asBufferedImage().

Closing as JXLImage no longer uses BufferedImage.


[proposal] add Add(context.Context, task Task) in PlatformRuntime interface

Proposal
The runtime v1/v2 can reload and manage existing tasks after containerd restart. These tasks will be loaded into Runtime Plugin. If the custom plugin which tries to load 0.2.x shim task and insert it into the existing io.containerd.runtime.v1.linux major plugin, there is no way to do that right now.
If the PlatformRuntime interface can provide the Add functionality, the developer can build more valuable custom plugin in their own production.
Interface Diff
diff --git a/runtime/runtime.go b/runtime/runtime.go
index 1b3f87c..c3c1a77 100644
--- a/runtime/runtime.go
+++ b/runtime/runtime.go
@@ -69,4 +69,6 @@ type PlatformRuntime interface {
        // Tasks returns all the current tasks for the runtime.
        // Any container runs at most one task at a time.
        Tasks(context.Context, bool) ([]Task, error)
+       // Add existing task.
+       Add(context.Context, Task) error
 }

V1/V2 Implementation
Export Add(context.Context, Task) error to Runtime

Is the usecase for this to migrate off of 0.2.x workloads?

Is the usecase for this to migrate off of 0.2.x workloads?

Yes. custom plugin can load existing from other state folders. For example,
moby daemon runs at the /bar/run/docker/libcontainerd/ something like that.
Migratory plugin can load the task into v1 runtime plugin. Therefore, the
containerd can take over the existing 0.2.x workloads.
It sounds crazy. But it seems it also can allow developers to migrate from
one state folder into other folder, not just for the 0.2.x workloads. Does
it make senses?

Is this just an internal/plugin API, no GRPC changes?

LGTM

This breaks shim logic for determining where to place Tasks right? It seems like we would need a command:
shim place id -> address. Then connect address -> Add()

This breaks shim logic for determining where to place Tasks right?

not fully understand this question. But I think this proposal doesn't break existing shim logic. The PlatformRuntime is used to manage the workloads. The Add interface allows custom plugins to add workload into the PlatformRuntime. For example,
func init() {
        plugin.Register(&plugin.Registration{
                ID:   "legacy-0.2.x-shim",
                Type: plugin.InternalPlugin,
                Register: []plugin.Type{
                        plugin.RuntimePlugin,
                },
        })
}

func New(ic *plugin.InitContext) (interface{}, error) {
        v1runtime, err := loadV1Runtime(ic)
        if err != nil {
                return nil, err
        }

        tasks, err := loadExistingTasks(ic)
        if err != nil {
                return nil, err
        }

        for _, t := range task {
                if err := v1runtime.Add(ic.Context, t); err != nil {
                        return
                }
        }
        ...
}

we don't need the command. just follow the plugin design. does it make senses to you?

For v1 this is fine. What about for v2 as your proposal mentions?

For v1 this is fine. What about for v2 as your proposal mentions?

The TaskManager also implements PlatformRuntime interface. In this proposal, I will extend this like v1. Just export Add method. Is there any risk or concern, here? 😄

Hi, @crosbymichael
I checked the #2434 and found that the Delete method has been moved from PlatformRuntime.
The Task will hold the TaskList. When caller calls the Task.Delete method, Task will call the TaskList.Delete to remove it from PlatformRuntime.
It seems reasonable if TaskService removes task from PlatformRuntime. Just curiosity, why remove the Task in the Task? 😄


Updating CognitiveServices/TextAnalytics SDK

Description

New SDK with a new /entities endpoint
Moving to Language/TextAnalytics folder
Namespace update to Microsoft.Azure.CognitiveServices.Language.TextAnalytics - Note that this is probably a breaking change.

Spec PR

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] Please add REST spec PR link to the SDK PR
[x] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Closing for now. Will split this into two PRs according to feedback.


az batch pool | example links need to be escaped in reference content

Describe the bug
Please see similar issue for Core command subgroup account: Azure/azure-cli GH issue #29622

Describe the bug
Please escape these example links in source code so they are corrected in MicrosoftDocs/azure-docs-cli autogenerated reference content.
Example links are needed in reference examples, but they should to be escaped so that GitHub does not treat them like live URLs.
List of Extension links that need escaped:

File name
Build error

latest/docs-ref-autogen/batch/pool.yml
Link 'https://learn.microsoft.com/azure/virtual-' points to a learn site page that doesn't exist. Check the path or URL and update the link.

Related command
az batch pool
Related command
az batch pool
Errors
n/a
Issue script & Debug output
n/a
Expected behavior
n/a
Environment Summary
n/a
Additional context
No response

Thank you for opening this issue, we will look into it.


⚠ deces.matchid.io search API has degraded performance

In 7aead30, deces.matchid.io search API (https://deces.matchid.io/deces/api/v1/search?q=pompidou) experienced degraded performance:

HTTP code: 200
Response time: 5026 ms

Resolved: deces.matchid.io search API performance has improved in 6e4e392.


Update url.py

Fix url.py to collect old tweet
original issue =>https://github.com/twintproject/twint/issues/1260

Nice 👍


cleanup ProfilerConfig

refactoring configuration

Current coverage is 40.28% (diff: 28.84%)

Merging #2133 into master will decrease coverage by 0.04%

@@             master      #2133   diff @@
==========================================
  Files          1578       1580     +2
  Lines         54617      54541    -76
  Methods       10759      10743    -16
  Messages          0          0
  Branches       7466       7472     +6
==========================================
- Hits          22027      21972    -55
+ Misses        32590      32569    -21
  Partials          0          0

Powered by Codecov. Last update 81466af...fdb406d


Not working on Ionic Framework

I'm having the same issue as here:
https://github.com/baruchvlz/angular-font-awesome/issues/11
You have closed the issue but we are very interested on this, at least 5+. It's a great package, but if we can't use it, is useless.

I've made a temporal patch to make it work. I have only modified tsconfig.json, to add map, and correct .ts names. Later i compiled the .ts using the command tsc, info here:
https://code.visualstudio.com/docs/languages/typescript
For anyone who wants to use it on ionic, you can download the files of my transpiled fork:
https://github.com/CarlosBC/angular-font-awesome
And put the needed files on your project /node_modules/angular-font-awesome.
I'm not an expert, this is only a temporal fix, i hope that baruchvlz fix it's package.
I'm going to make a pull request in case you want some file.

Thanks for taking the time to take a look at this.
I think that a better way to do to this would be to remove the files from the .npmignore, this way we can keep the GitHub repository with just the essential files for development. I will be doing these changes and publishing the package soon.

I don't know how it works nor how it should be done. But ionic needs the .js and the map file to work. Do it as you think it's better.

@CarlosBC I just published the NPM package with the new JS files. Should be fine now. (fingers crossed)

Not working, js files are ok, but missing js.map file.

Ok, the package as it stands should generate .js, .d.ts, .js.map files after being installed.

Now installation fails, seems like is missing jasmine and node. But maybe also needs tsc, because now i have tsc installed and can't fail, but yesterday i hadn't.
`

<EMAIL_ADDRESS>postinstall C:\xxxx\node_modules\angular-font-awesome
npm run start

npm WARN invalid config loglevel="notice"

<EMAIL_ADDRESS>start C:\xxxx\node_modules\angular-font-awesome
tsc

error TS2688: Cannot find type definition file for 'jasmine'.
error TS2688: Cannot find type definition file for 'node'.
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>start: tsc
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?
npm ERR!      C:\Users\xxxx\AppData\Roaming\npm-cache_logs\2017-10-12T12_06_25_637Z-debug.log
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>postinstall: npm run start
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\xxxx\AppData\Roaming\npm-cache_logs\2017-10-12T12_06_30_803Z-debug.log`

That's dumb... why wouldn't Ionic have those packages in their dependencies is beyond me. Those two packages are basic type definitions for Angular... I'll adjust it.

Now it works. 👏👏

Thanks for the help. @CarlosBC


Update README.md

Add go report card to proxy

Build is failing because we need a new key for pushpin which James added here - https://github.com/harness/ff-proxy/pull/145/files#diff-dd2c0eb6ea5cfc6c4bd4eac30934e2d5746747af48fef6da689e85b752f39557R33 . We can merge this anyway because it's just a readme change and can't break anything


Undefined method error when trying to define ManagedBookmarks via cpe_chrome on Windows

I'm getting the following error when trying to use cpe_chrome to set ManagedBookmarks on a Windows system:
NoMethodError: undefined method 'encode' for ["name", "My Bookmark"]:Array
Here's how I'm setting the node attribute:
node.default['cpe_chrome']['profile']['ManagedBookmarks'] = [
  {
    'name' => 'My Bookmark',
    'url' => 'https://google.com'
  },
  {
    'name' => 'Another Bookmark',
    'url' => 'https://google.com'
  }
]

Looks like changing ManagedBookmarks from :multi_string to :string prevents the error.  However, bookmarks are not getting created properly. Here's some notes from me trying to debug why:

The ManagedBookmarks key can accept a few different formats (see https://www.chromium.org/administrators/complex-policies-on-windows). I'm assuming the desired output would be a single JSON string value for the ManagedBookmarks key.
When multiple bookmarks are defined, separately indexed keys are getting created - one for each bookmark - due to the if data_has_mulitple_entries? logic in to_chef_reg_provider. Assuming the desired output is a single JSON string, this logic should be modified to exclude ManagedBookmarks (as well as any other potential keys that require JSON strings for values).
The registry_key resource call in cpe_chrome_win.rb uses the full path for the key, e.g. HKLM\\Software\\Policies\\Google\\Chrome\\ManagedBookmarks. This additional folder/subkey is not required and should be at the root (HKLM\\Software\\Policies\\Google\\Chrome). Changing the registry_key resource to use setting.path instead of setting.fullpath solves this but may break other keys. The reference registry file linked in the source should be reviewed again to match the hierarchy (https://dl.google.com/dl/edgedl/chrome/policy/policy_templates.zip)
Unsurprisingly, using Ruby hash rockets syntax in the value of the ManagedBookmarks key results in an invalid JSON string, so bookmarks don't get created. The bookmarks do get created after changing the hash rockets to colons/proper JSON.

I will try to start a PR to address some of these things, but would appreciate some feedback on the above so I know that I'm taking the right approach.

Thanks for the bug report. I have a patch internally for this specific setting that should be up shortly.

Thanks @svmastersamurai, let me know how I can help with a PR.

Oh, I see you committed a patch yesterday: https://github.com/facebook/IT-CPE/commit/435f87c73888cc2f8bc9adabf910ce75cc65d79b
Thanks! Will test soon.

Works great! Thanks for the patch.


OnlyOffice Server Running, OwnCloud can't connect

I have my ownCloud server running, and on a separate server I have OnlyOffice. When I access the OnlyOffice server by IP it says it is running. I added the IP address to my owncloud server for the address for the OnlyOffice server and it saved successfully. But when I try and open a document with Owncloud/OnlyOffice I get
"ONLYOFFICE not reached. Please contact admin"
Is it possibly because my onlyonly office is not using SSL, while my OwnCloud Server has a SSL from lets encrypt?

Hello @axxofreak!
You are right, when ownCloud instance is running over https it won't work with ONLYOFFICE which is not using https. To enable SSL for Document Server installed please follow the instruction in our helpcenter.

Hello @axxofreak
Please let us know if the issue is solved.

Hy,
Not use nginx and disabled ssl but not work!

Hi @usr-h
If you aren't @axxofreak and not related with him, would you submit another issue and describe you problem there? This would simplify the issue tracking.

Hello @axxofreak, please specify if you have enabled SSL and everything works or there are any issue.
Hello @usr-h, please specify if you have submitted a new issue and if the issue still exists. If not this ticket will be closed.

This issue was closed due to no response.


System

you try to use arm setup for your intel platform, no wonder it doesnt work. built it your self or use x86 setup from latest github action for example: https://github.com/awawa-dev/HyperHDR/actions/runs/8514125239 (Apple_macOS_x64_DMG_installer artifact)


Exception The object identifier does not represent a valid object

[EROR] Exception The object identifier does not represent a valid object. (Excep
tion from HRESULT: 0x800710D8)

Could you post some more information on what you were doing and what other messages got logged before this one?

Hello,
I'm French, Sorry for the Bad English.
I have the same problem on a Windows Server 2012 R2 (VPS OVH)
I have a running Webservice in http (80). The webservice is started and functional from the web.
When I execute Let's Encrypt-Win-Simple-V1.9.5-Beta-1, for the first time, it ask me a mail ("Enter an email adress ...") , and valid by Yes the conditions ("Do you agree to https://let...."), and after that, I have this message :
"
[EROR] Exception L'identificateur d'objet ne représente pas un objet valide. (Exception de HRESULT : 0x800710D8)
Would you like to start again? (y/n) :
"
After the first time I use the tool, I have directly the same message (after the message for present the tool ("[INFO] Let's ..... ACME Server: https://acme-v01.api.letsencypt.org/")
I don't understand, I searched on the web for resolve it, but no way. I searched during two full days.
I had tried the V<IP_ADDRESS> too, and I had the choise menu ("Webdav,..., Manual"), but my website isn't on the list (there is no message but my site is not listed)
I had verified the binding (Host name = "localhost"), and I had tried others Host name, no way too.
Do you have an idea for resolve this ? Do you think it's a Bug or a bad server config ?
Thank you in advance for try to help me.

Capture of the problem :
First use of the tool :

Others use

I will go to use the V.1.9.3. In it, My Website is listed :

Hi Pitapam, could you perhaps try v1.9.5 beta 2 with the --verbose switch to see if you still get the issue that you had with beta 1?

Closing this due to lack of activity and the fact that we've moved forwards a lot with the code.


Valgrind memory errors reports

Running cu_tester under valgrind reports many "Conditional jump or move depends on uninitialised value(s)" errors.
An extract:
==19461== Command: ./cu_tester
==19461==
==19461== Conditional jump or move depends on uninitialised value(s)
==19461==    at 0x4C2E78E: __strncpy_sse2_unaligned (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==19461==    by 0x40A929: ght_node_to_nodelist (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x405230: test_ght_node_unbuild_tree (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x4E3AC99: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3AF27: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3B2A5: CU_run_all_tests (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4070D9: main (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==
...
==19461== Conditional jump or move depends on uninitialised value(s)
==19461==    at 0x4C2E0E9: strlen (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==19461==    by 0x40928E: ght_strdup (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x409A30: ght_node_new_from_hash (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x40AA0C: ght_node_to_nodelist (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x40A9C9: ght_node_to_nodelist (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x40A9C9: ght_node_to_nodelist (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x40A9C9: ght_node_to_nodelist (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x405230: test_ght_node_unbuild_tree (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x4E3AC99: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3AF27: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3B2A5: CU_run_all_tests (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4070D9: main (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==
==19461== Conditional jump or move depends on uninitialised value(s)
==19461==    at 0x4C2E0F8: strlen (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==19461==    by 0x40928E: ght_strdup (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x409A30: ght_node_new_from_hash (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x40AA0C: ght_node_to_nodelist (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x40A9C9: ght_node_to_nodelist (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x40A9C9: ght_node_to_nodelist (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x40A9C9: ght_node_to_nodelist (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x405230: test_ght_node_unbuild_tree (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x4E3AC99: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3AF27: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3B2A5: CU_run_all_tests (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4070D9: main (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==
==19461== Conditional jump or move depends on uninitialised value(s)
==19461==    at 0x4C2F1BC: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==19461==    by 0x40524B: test_ght_node_unbuild_tree (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x4E3AC99: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3AF27: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3B2A5: CU_run_all_tests (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4070D9: main (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==
==19461== Conditional jump or move depends on uninitialised value(s)
==19461==    at 0x4E3AFD7: CU_assertImplementation (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x405275: test_ght_node_unbuild_tree (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x4E3AC99: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3AF27: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3B2A5: CU_run_all_tests (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4070D9: main (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==
==19461== Conditional jump or move depends on uninitialised value(s)
==19461==    at 0x4C2F1BC: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==19461==    by 0x405294: test_ght_node_unbuild_tree (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x4E3AC99: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3AF27: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3B2A5: CU_run_all_tests (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4070D9: main (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==
==19461== Conditional jump or move depends on uninitialised value(s)
==19461==    at 0x4E3AFD7: CU_assertImplementation (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4052BE: test_ght_node_unbuild_tree (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x4E3AC99: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3AF27: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3B2A5: CU_run_all_tests (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4070D9: main (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==
==19461== Conditional jump or move depends on uninitialised value(s)
==19461==    at 0x4C2F1BC: strcmp (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==19461==    by 0x4052DD: test_ght_node_unbuild_tree (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==    by 0x4E3AC99: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3AF27: ??? (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4E3B2A5: CU_run_all_tests (in /usr/lib/libcunit.so.1.0.1)
==19461==    by 0x4070D9: main (in /usr/src/pointcloud/libght/libght/b/test/cu_tester)
==19461==
...
==19461== ERROR SUMMARY: 242 errors from 30 contexts (suppressed: 0 from 0)

The very first report is an off-by-one error.
The tester file (cu_ght_core.c) allocates an array of GHT_MAX_HASH_LENGTH bytes, then passes this allocated buffer to ght_node_to_nodelist which writes into GHT_MAX_HASH_LENGTH+1 bytes instead.

There are more weird things in ght_node_to_nodelist, namely that the destination GhtHash "h" is written to twice, if node->has is not null, once copying the provided "hash" parameter, and a second time copying the node->hash parameter.
Is libght supposed to be working / worth looking at ? (/me starts wondering)

Sorry, now I get it's not a duplication, is appending (strcat) to the initial string. But there cannot possibly be space for all if the initial hash could be also MAX_HASH_LENGTH in size, could it ?

This pull request fixes the valgrind reported errors: https://github.com/pramsey/libght/pull/14

WRT whether libght is supposed to be working or not, it is supposed to work, but I found it of mixed utility in actually handling data in a data set with modest tiling sizes, and moving to large tiling sizes introduces blocking in the TOAST tables, which seem to work against the potential efficiencies of the GHT scheme. So it hasn't received a lot of love since initial implementation.


[SPARK-27672[SQL] Add since info to string expressions

What changes were proposed in this pull request?
This PR adds since information to the all string expressions below:
SPARK-8241 ConcatWs
SPARK-16276 Elt
SPARK-1995 Upper / Lower
SPARK-20750 StringReplace
SPARK-8266 StringTranslate
SPARK-8244 FindInSet
SPARK-8253 StringTrimLeft
SPARK-8260 StringTrimRight
SPARK-8267 StringTrim
SPARK-8247 StringInstr
SPARK-8264 SubstringIndex
SPARK-8249 StringLocate
SPARK-8252 StringLPad
SPARK-8259 StringRPad
SPARK-16281 ParseUrl
SPARK-9154 FormatString
SPARK-8269 Initcap
SPARK-8257 StringRepeat
SPARK-8261 StringSpace
SPARK-8263 Substring
SPARK-21007 Right
SPARK-21007 Left
SPARK-8248 Length
SPARK-20749 BitLength
SPARK-20749 OctetLength
SPARK-8270 Levenshtein
SPARK-8271 SoundEx
SPARK-8238 Ascii
SPARK-20748 Chr
SPARK-8239 Base64
SPARK-8268 UnBase64
SPARK-8242 Decode
SPARK-8243 Encode
SPARK-8245 format_number
SPARK-16285 Sentences
How was this patch tested?
N/A

Test build #105305 has started for PR 24578 at commit 33c289a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10580/
Test PASSed.

Test build #105305 has finished for PR 24578 at commit 33c289a.

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

Merged build finished. Test PASSed.

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

Thanks!


Compile errors with clang10 and libc++

When trying to use your library with clang10 and libc++ and c++2a I get some compile errors for missing function-like macro span_HAVE. It works with clang9 and libc++ and c++2a though.
Godbolt showcasing the issue (against 0.7.0, but errors also relevant on master branch): https://godbolt.org/z/yWWR6Z
Compile errors:

x86-64 clang 10.0.0 (Editor #1, Compiler #1) C++
x86-64 clang 10.0.0
-stdlib=libc++ -std=c++2a
1
<Compilation failed>
x86-64 clang 10.0.0 - 1422ms
#1 with x86-64 clang 10.0.0
nonstd/span.hpp:1523:61: error: function-like macro 'span_HAVE' is not defined

#if span_CPP11_OR_GREATER && span_FEATURE( BYTE_SPAN ) && ( span_HAVE( BYTE ) || span_HAVE( NONSTD_BYTE ) )

                                                            ^

nonstd/span.hpp:1553:5: error: function-like macro 'span_HAVE' is not defined

#if span_HAVE( STRUCT_BINDING )

Thanks @AndWass ,
The problem is in the handling of the case where the C++20 standard header <span> is available as the following does compile:
-Dspan_CONFIG_SELECT_SPAN=span_SPAN_NONSTD -stdlib=libc++ -std=c++2a

The error only appears with libc++, because only that library provides header <span> apparently.
I'll look into it further.

@martinmoene I noticed the same error using clang(d) 10.0.0
[clang pp_expr_bad_token_lparen] [E] In included file: function-like macro 'span_HAVE' is not defined
My simple fix was to move line 281:
#define span_HAVE( feature )  ( span_HAVE_##feature )
to precede what I believe is its first use on line 32
# define span_CONFIG_SELECT_SPAN  ( span_HAVE_STD_SPAN ? span_SPAN_STD : span_SPAN_NONSTD )
This gets rid of the error in clangd and also compiles without any issue with
clang++ -stdlib=libc++ -std=c++2a -Wall sp_test.cpp
that contains #include <nonstd/span.hpp>

Can confirm that this fixed the issue for me as well

@AndWass thanks for bringing this to my attention, and @Arghnews much appreciated your solution which is now in head. Sorry for my delayed reaction.

Cheers, and thanks for the awesome library.

Thanks for the fix! Much appreciated :)


Update README.md correct markdown formatting

Coverage remained the same at 99.256% when pulling f48b64d40213d487bc97e937bff5a2698cd0ae90 on juliocanares:patch-1 into d7c64e4b0393f076c353828e124c77d6764c056f on optimizely:master.

Thanks for your contribution! You opened a PR directly into master. Please close and open a PR against our devel branch. Also please take a look at our CONTRIBUTING.MD doc and sign our CLA!

This change has been made in https://github.com/optimizely/node-sdk/pull/60


Add Github Actions reporter

See #330
First and second commit moves a few functions from individual reporters into the base one for reuse. The third commit is the actual reporter. I didn't want to duplicate the functionality once again.
Here's a screenshot of how it looks like in Githubs UI:

This looks great! Would love to see it merged and released.

Can we have an update here ?


[1.1.2] [Tiny] Make SqlServerDatabaseModelFactoryTest use scratch database …

…to avoid getting in a dirty state, since RelationalDatabaseCleaner uses SqlServerDatabaseModelFactory

@AndriySvyryd Did you check performance after this change? Admittedly a lot has gone on since then and DB initialization may well have changed, but I seem to remember that at the time there were some DB-heavy tasks that were done to initially set up the DB. If those are being repeated more than they were before it might make those tests slow.

@lajones I didn't measure as they definitely will be slower. But I think slow tests are better than flaky tests

@AndriySvyryd - sort of - if they become too slow then people just don't run them. As long as you've run them and the time it takes to run them is still acceptable then that's fine.

@lajones On my machine running them all goes up from 4.3s to 6.6s

@AndriySvyryd No worries then.


InjectManifest Plugin doesn't compile the serviceworker with webpack

Library Affected:
workbox-webpack-plugin
Browser & Platform:
all browsers
Issue or Feature Request Description:
Ideally, the SW would go through the normal webpack compilation, so that the code gets transformed. Right now, the code seems to not be transformed by it, and as a result it creates a divide in our project as the transforms that apply are different. It also makes it difficult to use something like TypeScript or Flow in the ServiceWorker.

I have the same problem, in my custom sw I want to import some files from node_modules but webpack do not compile the swSrc.
Does anyone know how to do that?

If I understand the problem correctly, one way to solve this would be to first build the service worker file only and then build the rest of the app and specify swSrc to be the previously built service worker file.
To achieve this simply export multiple configs from webpack config file. In the first config, specify the service worker body file as the single entry point and do any transformations you want with it in the rest of the build. In the second build just use the file.

That’s one way to do it, but I think that’s working around the plugin, vs having the plugin work in the more ideal way.

@edgaraskazlauskas If you could provide an example that would be very helpful. I've never run through a webpack build process twice.

I'd like to to use InejctManifestPlugin as follows, without the two-pass hack suggested above:
// webpack.config.js
module.exports = {
  entry : {
    sw: './src/sw.js'
  },
  output: {
    filename: '[name].js'
  },
  plugins: [
    new InjectManifestPlugin()
  ]
}

The way it currently works is not intuitive at all.

If you allow templating in the swSrc parameter, you could allow it to be defined as swSrc: './dist/js/js.[contenthash].js -- right?

If I am reading this thread correctly an example still has not been provided and it is still a problem. I think I am running into the same issue expressed by the OP. I have a service worker with module imports. I wanted to use the workbox Webpack plugin to inject a precache manifest how the benefit of the injected dynamic manifest comes at the cost of modules not being transpiled.
// sw.js (original)
import {precacheAndRoute} from 'workbox-precaching/precacheAndRoute.mjs'

precacheAndRoute(self.__precacheManifest)

// webpack.config.js
const {InjectManifest} = require('workbox-webpack-plugin')
const path = require('path')

module.exports = {
	entry: {
		sw: './src/sw.js',
	},
	plugins: [
		new InjectManifest({
			swSrc: path.resolve(__dirname, 'src/sw.js'),
			importWorkboxFrom: 'disabled',
		}),
	],
}

// sw.js (bundled... not)
importScripts("precache-manifest.47565e861a50e4b271a0ce24cc9d8095.js");

import {precacheAndRoute} from 'workbox-precaching/precacheAndRoute.mjs'

precacheAndRoute(self.__precacheManifest)

As you can the bundled sw.js file does not include the webpackBootstrap and the module import is remains unchanged.
The inability to run the service worker through Webpacks transpiler pretty much renders the plugin useless for me.
Hoping to find some solutions.

Does the new guidance at https://developers.google.com/web/tools/workbox/guides/using-bundlers help any?
CC: @philipwalton

Does the new guidance at https://developers.google.com/web/tools/workbox/guides/using-bundlers help any?

Not much. I have the same struggles. I write everything in TypeScript and Webpack is using ts-loader to combile the TypeScript code. But the plugin totally ignores this. If I set swSrc to be a TypeScript file and swDest to be a JavaScript file, the plugin simply copies the TypeScript file and renames it, but doesn't run it through the defined webpack chain.

Does the new guidance at https://developers.google.com/web/tools/workbox/guides/using-bundlers help any?

I'm not sure which bits in particular were 'new' (I've only just started looking at this!)
I can see:

If you also want to use webpack to generate your service worker file (as described in this article), beware that the workbox-webpack-plugin's injectManifest config accepts a swSrc option that it will update with your precache manifest.
This can be problematic if you're using the same webpack configuration to generate your service worker file, since it may not exist by the time the workbox-webpack-plugin code runs.

This does acknowledge the problem, but doesn't seem to suggest a workaround.
I'm trying to migrate from the old sw-precache module, and have now had to generate my own root serviceworker.js file (rather than just additional files to include), which requires me to use calls to things like registerRoute directly, rather than it being defined via the webpack configuration. I'm trying to use imports so that my IDE actually picks up what exists, as I was getting lots of  'workbox' is not defined errors in my linter.
I tried to do a two-pass method where one pass will build from my serviceworker code to a transpiled one, and then run the injectManifest on the transpiled class, but it has issues with the filenames. Now some part of the manifest is trying to load https://app/my_main_app_file.js as in the manifest generated it has an additional slash at the front of the URL.
Is there a way to do this with workbox + webpack, or should I just stick with the sw-precache plugin instead which worked, but is deprecated?

I copy handleMake from serviceworker-webpack-plugin. And it's work for me. I'm new to webpack so It's maybe not perfectly solve this issue.
The code:
https://github.com/ALiangLiang/workbox/blob/feat/%231513/packages/workbox-webpack-plugin/src/inject-manifest.js
  plugins: [
    ...,
    new workboxPlugin.InjectManifest({
      swSrc: 'sw.js',
      entry: path.join(__dirname, '..', 'src/sw.js'),
    })
  ]

@ALiangLiang I can't quite get your workaround to work - In your case what does swSrc refer to? If I both let entry and swSrc refer to the same TypeScript file, the raw TS Code gets copied over to the Service Worker, if it is an empty dummy JS file, the service worker is empty. (in addition to the importScripts(...))
I'd even argue that a swSrc is unneeded in combination with entry. I want to write the service worker by myself, since there is a lot of stuff workbox cannot possibly handle for me(session data, live updates, ...).
But I don't want to do this in a JS file that just gets copied over, but instead write it in proper TypeScript.
The only thing I need is to (quite literally) have the generated manifest injected into the compiled service worker so that I don't have to manually hunt down static assets.

@ALiangLiang I can't quite get your workaround to work - In your case what does swSrc refer to? If I both let entry and swSrc refer to the same TypeScript file, the raw TS Code gets copied over to the Service Worker, if it is an empty dummy JS file the service worker is empty.

I see your point. Maybe it is a typo. Did you try swDest: 'sw.js' ? That would make much more sense..

@ALiangLiang  Why not propose a PR ?

@pungggi
As I said, I am a new to webpack. I just copied and tried this workaround. The mechanism is still not very clear. I can make a PR, but I don't think it's suitable for using it directly in the project. I need somebody help me to adjust the interface(options).

I can confirm ALiangLiang solution works:

set up output as usual and set it also under swSrc
set where to find your workbox template
set the final worker filename

what is the final solution for this?

https://stackoverflow.com/questions/56710951/how-to-get-workboxplugin-injectmanifest-to-work
I also got stuck, please update the doc and provide a solution?  Thanks!

I'm also having the exact same problem mentioned above by @henrylearn2rock in his stackoverflow post. I would've thought this to have been a standard thing that the injectManifest plugin would take care of. Makes the whole thing kinda pointless to not transform this code through webpack first 🤔
@jeffposnick please help ☺  (I've read through the entire guide that you posted and there's nothing in there about this and none of the steps resolve it)

We are actively working on a rewrite for the InjectManifest plugin that will perform a webpack child compilation on swSrc, in addition to populate the precache manifest.
It's still too early to use, but you can see a sneak-preview of the changes at https://github.com/GoogleChrome/workbox/compare/v5...wepback-v5?expand=1
We'll be rolling out the first alphas of v5 with this functionality following some additional tests and cleanup work—likely in the order of a couple of weeks from now.

(I'm not 100% sure what the ergonomics of this will be if the swSrc is a TypeScript file, so I'm hoping for feedback from folks with that use case once we release our first alphas.)

This should be addressed by the current Workbox v5.0.0 alpha.

I'm trying to figure out how to make webpack and workbox plugin play together (compile and inject in one flow), but a bit confused about the correct setup.
Is this already supported in v5 ? do you have some examples ?
Thanks!

In v5, if you list a service worker file as the swSrc in InjectManifest mode, webpack will automatically perform a (child) compilation on that file, in addition to replacing self.__WB_MANIFEST with the precache manifest that was derived from the main compilation's assets.
That was not the case in v4, so if you need that functionality, please try out the v5 beta.

Thanks! Finally it worked for me with plugin v4.3.1 and customized sw-src.js (with "import-from" statements instead of importScripts()),

entry: path.resolve('sw-src.js'),
output: {
path: SOME_OUTPUT_PATH,
filename: 'bundle.js',
},
new InjectManifest({
swSrc: 'bundle.js',
swDest: ${SOME_OUTPUT_PATH}/sw.js,
importWorkboxFrom: 'local',
globDirectory: SOME_OUTPUT_PATH,
globPatterns: [ '**/*.{html,js,css}' ],

@jeffposnick , I'm trying the beta and was wondering if you plan on dropping the importsDirectory and precacheManifestFilename params when its no longer in beta

@ray-kim-12 yes, neither of those options should be supported in the current beta 1 release, as they are no longer relevant.

I've just tried v5.beta-1 because I also wanted a webpack/typescript build on my worker file. I got exactly that and saw the typical webpack generated code without any single config modification (I am using InjectPlugin). However when moving from v4 to v5 I noticed the 'workbox' global no longer exists and there is no mention in the changelogs.
Is there doc/example about how to access 'workbox' in v5?

workbox corresponded to an instance of the workbox-sw module, which is effectively a proxy that mapped a namespace of other Workbox modules with code to import the corresponding files from the CDN.
Because you're using a local bundle of all the Workbox code that you actually need now, the idea is that workbox-sw (and therefore the workbox global symbol) should no longer be needed.
If you're trying to accomplish something specific with the workbox symbol that you can't figure out how to do in v5, could you open a new GitHub issue (or Stack Overflow question) with the details? This is a closed GitHub issue and it would be best to move the discussion elsewhere.

Tried to use v5.0.0-beta.1, but can't get it to compile. I get an error: Can't find self.__WB_MANIFEST in your SW source.
// service-worker.js
precacheAndRoute(self.__WB_MANIFEST);

// webpack.config.js
...
config.plugins.push(
  new WorkboxWebpackPlugin.InjectManifest({
    // Use custom service worker implementation
    swSrc: path.resolve(projectConfig.srcPath, 'service-worker.js'),
    // Don't precache sourcemaps (they're large) and build asset manifest:
    exclude: [/\.map$/, /asset-manifest\.json$/],
  })
);

Oh, found the problem, it was because I was importing another file named service-worker.js from another directory. Renaming the file fixed the problem.

I have the same problem, in my custom sw I want to import some files from node_modules but webpack do not compile the swSrc.
Does anyone know how to do that?

Did you solve this issue?

@novaknole the workbox-webpack-plugin is not intended to be used if your webpack entry point is your service worker file itself (e.g. in your case, ./src/js/service-worker.js).
Instead, it's intended to be used where your application's JavaScript code is the entry point, and then it will also create a service worker file that will precache the JavaScript chunks generated by webpack after bundling your app's main JavaScript code. Does that make sense?
If you're looking specifically for how to use webpack to bundle a service worker file, I'd recommending taking a look at this guide: Using Bundlers (webpack/Rollup) with Workbox

ServiceWorkers are not constrained to just caching assets. It is quite handy to be able to import ES6 modules into a ServiceWorker without worrying about Webpack configuration.
There are a lot of repos that have workbox-webpack-plugin as a dependency. Many projects that have the workbox-webpack-plugin as a dependency just blackbox all the Webpack plugins they use.
If the defaults remain like this many people will have to spend tons of time just on creating alternative Webpack configurations.
if you are saying that ES5 should be a default in 2020 I cannot agree less.

@philipwalton Thanks man. I made it work.

@novaknole How did you make it work :(

@changyeamoon still a problem or need help?

@novaknole It is resolved, turned out to be a different kind of issue. Thanks for checking up 😄

It looks like back in 2019 a temporary package, wrkbx was published to test compiling service workers. Although the original comment stated that it shouldn't be used in production, I've got a client who was using this package as part of their build and never moved over to workbox-webpack-plugin. It looks like this package is being downloaded hundreds of times weekly:

To help folks discover that they should move back over to the official package, it would be nice to mark this package as deprecated on NPM (docs).
The command would look something like:

npm deprecate wrkbx "This package should not be used in production. Please switch to the official workbox packages. See https://github.com/GoogleChrome/workbox for instructions"

cc @pungggi who it looks like the package belongs to - can you deprecate the package?


Question: accept an invitation

I'm building in a simple invitations system to allow users to invite other users to a group they are part of via a has_many_through relationship.
I've overridden the devise registrations and session controller to check for the invitation and handle the user & group relationship creation accordingly. I'm stumped though on what to do if the user is already signed in. That doesn't seem to hit registration#new or session#new.
Can anyone point me in the direction of what I would need to intercept a currently signed in user and make the change to the record (ie mark the invitation used and add him to the appropriate group)?

Please use the mailing list for questions.

Sorry, missed that.


New data for R2

@pstradio FYI I already added a few of the new datasets

closes #54

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1560

Relevant Lines:
1820

💛  - Coveralls


Show helpful message in Debug Welcome View if no debuggers are available

While investigating https://github.com/microsoft/vscode-remote-release/issues/5794:
If VS Code's built-in debugger "js-debug" becomes disabled (for whatever reason), the contents of the "Run and Debug" viewlet is not really helpful:

I suggest that we show the debug Welcome View with a message along the lines of:

"The built-in JavaScript debugger is disabled.
You can enable it or install another debugger from the Marketplace"

If there are additional installed but disabled debuggers, the message could reflect that.

How about this?

Yep, sounds good - ship it,!

Verified:


DO NOT MERGE: travis sh check

Does travis /bin/sh script fail if given a bash-specific feature?

No check necessary. Travis runs Ubuntu containers. On Ubuntu sh is replaced by a sim link to bash per default.
It will get problematic when we test Mac instances though. There ah is sim linked to dash afaik.

On Ubuntu sh is replaced by a sim link to bash per default.

The undesired result. I wanted Travis CI to fail.

Fails as desired.
/home/travis/build/exercism/xhaskell/bin/fetch-configlet: 5: /home/travis/build/exercism/xhaskell/bin/fetch-configlet: Syntax error: Bad for loop variable

So as I hoped, on travis /bin/sh is not bash.

Note for posterity: if [[ "a" == "a" ]]; then echo "yes"; fi does cause a [[: command not found, but didn't fail the Travis build for some reason.

I think that, when sh is a link to bash, the runtime detects it was called as sh and changes behavior accordingly:
https://www.gnu.org/software/bash/manual/bashref.html#Bash-POSIX-Mode
http://unix.stackexchange.com/questions/44836/when-sh-is-a-symlink-to-bash-or-dash-bash-limits-itself-to-posix-compliance-so
If I remember correctly from my times playing with Linux from Scratch, a few other compiled program and scripts share the same behavior, so you cannot expect - as a rule - that linked executables behave as their targets.

As an aside (not relevant to Haskell track), that is a useful way to introduce extra behaviour into a family of executables, or simply have one executable act as many executables. Busybox takes this approach: https://busybox.net/FAQ.html

If the Busybox executable is renamed to one of the commands it supports, it will act as that command automatically

I'll make one more check, but basically what I hear here is that the only safe thing to assume about /bin/sh is that it implements http://pubs.opengroup.org/onlinepubs/9699919799/ (is that right?), and scripts with /bin/sh shebang should be written accordingly.

dash.


feature/minimal apis

feat(minimal-apis): add endpoint definitions for minimal apis
ci(minimal-apis): publish endpoint definitions package

Codecov Report

Merging #11 (9a28e9e) into main (90b7e96) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #11   +/-   ##
=======================================
  Coverage   91.79%   91.79%
=======================================
  Files          21       21
  Lines         268      268
  Branches       27       27
=======================================
  Hits          246      246
  Misses         12       12
  Partials       10       10

Continue to review full report at Codecov.

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


Http Basic Auth interfering with Gitea auth.

Gitea version (or commit ref):
Git version: 1.8.1 built with go1.12.5 : bindata, sqlite, sqlite_unlock_notify
Operating system: Debian 9 Strech - Up to date
Database (use [x]):

[ ] PostgreSQL
[X] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[X] No
[ ] Not relevant

Log gist:
Gitea.log - https://gist.github.com/soniok/2c44c06967ef3b9d450f74fe4bc1f95a

Description
I'm automatically logged into a gitea account if i'm also using HttpBasicAuth with a account username that has the same name has the account in gitea.
I can view pages and configuration/settings but I cannot update or edit.
My solution is to not use the same name in both gitea and HttpBasicAuth in apache2.
Please note, Gitea is behind an Apache2 reverse-proxie and sub directory, both based on the docs
Cordially,

I get the following error message when I try to edit a gitea account via the web interface and being in this "auto logged" mode:
Invalid csrf token.


Flaky Test: istio.io/istio/tests/integration/pilot/locality/failover.TestLocalityFailover

https://prow.k8s.io/view/gcs/istio-prow/logs/istio-integ-k8s-tests/7620
@liamawhite I believe you wrote the tests?

These have only failed once during their lifetime in the post-submit. Is there any info on how frequent the failure is here?

They fail fairly frequently: https://k8s-testgrid.appspot.com/istio-postsubmits-master#integration-k8s-tests

Ok, will investigate.

Root cause: race condition at https://github.com/istio/istio/blob/555ec2a9b4f1c58b309ed5e4431d52ed8ca0e051/pilot/pkg/proxy/envoy/v2/ads.go#L638
if we have no instances (delay in propagation) then we won't use the override locality labels

This has improved, but we still do see some flakes, so I will reopen and add the flaky label
https://testgrid.k8s.io/istio-presubmits-master#integ-pilot-k8s-presubmit-tests


Dual channel polarimetry intro

Merging dual channel polarimetry intro

@becca9808 can you remove all of the .pyc files from the commit?

nvm I think I got it