eBPF: Remove the use of deprecated bpf_map_def

struct bpf_map_def is deprecated in libbpf. So let's use BTF enabled definitions instead.
Fixes #294
Signed-off-by: Vaishali Thakkar<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

https://github.com/parca-dev/parca-agent/pull/329#pullrequestreview-935416564
No, this one is your first!

LGTM 100
Do we need to update the minimum required Kernel version? I thought we need to use BTF definitions for this, no?

Yes, that's true. We should update the minimum required kernel version.
Also, it'll be nice if you can already commit your updated local-dev script which has a link to that custom minikube image.

Ok, so I think we can update it to 4.19 as that's the last LTS kernel for which BTF support has been backported.

Also, it'll be nice if you can already commit your updated local-dev script which has a link to that custom minikube image.

I'm gonna create a PR to add those scripts and where we bump the minimum required version, so that it'd be easier to trace it back.

Sounds good. Thanks.
For the kernel version thing, I couldn't find a single source to get the exact idea of the version until which it is backported. Maybe we should also add a command/link for users to check if the distro they're running has a BTF support. I'll check more on that and will come back the details. :)

I have checked this with our current minikube setup which uses 4.19 and it works without warnings.
Linux parca-agent 4.19.202 #1 SMP Tue Feb 8 19:13:02 UTC 2022 x86_64 GNU/Linux
:tada: Happy to merge it.

Sounds good. Thanks for checking that. :)


Remove deprecated ARIA attributes from language data.

aria-describedat and aria-kbdshortcuts have been deprecated in ARIA 1.1 and should no longer be used. Source
aria-describedat has been made obsolete by aria-details.
aria-kbdshortcuts has been replaced by aria-keyshortcuts.
Soon ARIA will be upgrading to 1.2 as well, so removing these two attributes would be prudent.

Thanks @yihjenku. I made the change.
webCustomData.ts is generated from @vscode/web-custom-data, so I made the change there.

Great, thanks so much!


Consider using parentheses in toString() of finite types with more than one value

Description:
$title, basically for unions of values similar to BUnionType - https://github.com/ballerina-platform/ballerina-lang/blob/v2.0.0-beta.2.1/compiler/ballerina-lang/src/main/java/org/wso2/ballerinalang/compiler/semantics/model/types/BUnionType.java#L484.
Describe your problem(s) and solution(s)
public function main() {
    (int|string)[] a = [];
    int b = a; // compilation error: incompatible types: expected 'int', found '(int|string)[]'

    (1|2)[] c = [];
    int d = c; // compilation error: incompatible types: expected 'int', found '1|2[]'
}

Similar to the union case, the second error should probably be
expected 'int', found '(1|2)[]'

The current error could be misinterpreted as 1|(2[]) (which is how it is parsed).
This also leads in a misleading error for
public function main() {
    1|2[] a = [];
    (1|2)[] b = a; // incompatible types: expected '1|2[]', found '(2[]|1)'
}

Let's use https://github.com/ballerina-platform/ballerina-lang/issues/35018 to track this.


Slide splash page on iPhone 7+ is off center

http://imgur.com/a/tWjVP

Offset for me too in both portrait and landscape. iPad Pro 9.7 iOS 11

I'm experiencing a similar issue on iPhone SE. The splash screen us offset to the lower right.

In my iPad it’s to the top left instead of right

On my iPad it’s in the top left

Fixed on all devices


Ability to set source precedence

Is your feature request related to a problem? Please describe.
Completion-nvim allows to sort completion items by length, alphabetical order, but it doesn't allow sort items by their source.
Describe the solution you'd like
Deoplete has this feature which allows to set each source precedence.
https://github.com/Shougo/deoplete.nvim/blob/master/doc/deoplete.txt#L1196
Describe alternatives you've considered
I tried to put sources into diffeerent complete_items in g:complete_chain_complete_list. But it's uncomfortable to constantly switch sources.

@kuator there's already precedence opt by source. I can't find a document for it. But it exists.
  let g:completion_items_priority = {
          \ 'Field': 5,
          \ 'Function': 7,
          \ 'Variables': 7,
          \ 'Method': 10,
          \ 'Interfaces': 5,
          \ 'Constant': 5,
          \ 'Class': 5,
          \ 'Keyword': 4,
          \ 'UltiSnips' : 1,
          \ 'vim-vsnip' : 0,
          \ 'Buffers' : 1,
          \ 'TabNine' : 0,
          \ 'File' : 0,
          \}

But I also think precedence by sources would be great. There's should be more discussion about it, as It might get really complicated.

@aca this is epic, thanks!


Question regarding RegTree::FVec::Fill

xgboost version used: v0.7 (2017.12.30)
If installing from source, please provide

The commit hash (git rev-parse HEAD)<PHONE_NUMBER>4a774c4226c4432a642bb1da446d24

In tree_mode.h
inline void RegTree::FVec::Fill(const RowBatch::Inst& inst) {
  for (bst_uint i = 0; i < inst.length; ++i) {
    if (inst[i].index >= data.size()) continue;
    data[inst[i].index].fvalue = inst[i].fvalue;
  }
}

When setting the fvalue of data in FVec, should we also set the flag(indicating missing value) to 0? Because both Init and Drop set the flag to -1. And I didn't see other code set the flag to non -1.
One use case is in CPUPredictor::PredValue
    p_feats->Fill(inst);
    for (size_t i = tree_begin; i < tree_end; ++i) {
      if (tree_info[i] == bst_group) {
        int tid = trees[i]->GetLeafIndex(*p_feats, root_index);
        psum += (*trees[i])[tid].leaf_value();
      }
    }
    p_feats->Drop(inst);

Seems all flags in p_feats will be -1.
Did I miss something?

Missing values are literally "missing" (sparse elements) in those sparse vectors.

Indeed, for features that do not exist, default missing is the correct behavior. But the real question is, for the non missing values set in the Fill function, shall we also set the flag to non missing?

FVec::Entry is a union, so setting the fvalue is the same as setting the flag. The assumption is that we would never encounter the float value with the same representation as integer -1.

I see, that makes sense given the assumption. Thanks for the explanation!


Improve Checks on Dynamic Introspection

At this time, if you add an interface that is already in introspection, it is replaced by the new one, without checking the major  or minor version.
We should improve checking in case of :

new major > old major :  error should be raise (can't upgrade an interface major at runtime) ?
same major : error should be raise (Interface already present in mapping) ? or simple warning ?
same minor : error should be raise (Interface already present in mapping) ? or simple warning ?
new minor < old minor : error should be raise (can't downgrade an interface at runtime)
new minor > old minor : the new interface replaces the old one

As a suggestion while resolving this issue evaluate if the struct Interface should be present in the public API of the SDK.


Implement spin up from an OCI registry

#1014 implements running a Spin application from an OCI registry through the spin oci run command while we validate the implementation.
Ideally, we want to integrate this functionality with spin up.
This issue tracks updating spin up to accept a reference from the registry and start the application.

closed by #1146


Which firewall ports to which FQDNs/URLs need to opened for azcopy and how the proxy settings have to be configured

Which version of the AzCopy was used?
10.13.0
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
azcopy login
What problem was encountered?
INFO: Proxy detected: https://azcopyvnextrelease.blob.core.windows.net -> //...:8000
INFO: Proxy detected: https://login.microsoftonline.com -> //-> //...**:8000
adal: Refresh request failed. Status Code = '401'. Response body: {"error":"invalid_client","error_description":"AADSTS700027
: The certificate with identifier used to sign the client assertion is not registered on application. [Reason - The key was not found.
I am using azcopy with a service principal and a certificate and Azure AD Authentication. I get the error above on an on prem Windows server. On an Azure Windows Server VM azcopy works fine with the same service principal and certificate. I believe this is not an azcopy issue, but a matter of opening the necessary firewall ports to the relevant Azure/Microsoft FQDN/URLs and having the correct proxy configuration.  Could you please advise which target FQDN/URL/port pairs have to accessible and how the proxy configuration has to be done for azcopy on prem to function without any connectivity related issues.
I intend to do some tests with Storage Explorer as well, so if any additional FQDN/URL/Port and proxy configuration is needed for Storage Explorer could you please add the relevant details too.
How can we reproduce the problem in the simplest way?
Not sure.
Have you found a mitigation/solution?
No

Hi there, @sezginr!
It sounds like the client certificate is possibly getting overwritten by your proxy. I'm not sure about the configuration of your particular proxy unfortunately.
Regards,
Adele

Hi Adele,
Thanks for your reply.
We have made some progress, now azcopy version 10.13.0 authenticates (explicit login and autologin), but the latest version 10.16.0 does not and gives the error below. The only difference is the azcopy version, everything else is the same - same server,same proxy settings, same azcopy service principal and certificate.
Failed to perform login command:
adal: Failed to execute the refresh request. Error = 'Post "https://login.microsoftonline.com/18ed93f5-e470-4996-b0ef-9554af9
85d50/oauth2/token?api-version=1.0": dial tcp <IP_ADDRESS>:443: connectex: No connection could be made because the target ma
chine actively refused it.'
Regards,
Sezgin

Interesting to hear. I'm not certain we made any specific changes to the login process between 10.13.0 and 10.16.0. Do any other versions in between work? It'd be useful to know where the break begins, so we might be able to track it down.

Unfortunately I don't have any versions between 10.13.0 and 10.16.0. I am not sure if anything is wrong/broken with version 10.16.0, it could be that it has different requirements for allowed URLs/FQDNs and we have not met them.

Let me get you some download URLs for those.
https://azcopyvnextrelease.blob.core.windows.net/release20220228/azcopy_windows_amd64_10.14.0.zip
https://azcopyvnextrelease.blob.core.windows.net/release20220315/azcopy_windows_amd64_10.14.1.zip
https://azcopyvnextrelease.blob.core.windows.net/release20220511/azcopy_windows_amd64_10.15.0.zip
Could you test each of these and tell me when the issue starts to present?

Thanks.
I tried all 3 versions in Powershell on a new server where v.10.13.0 works and got the same error below with all:
"INFO: If you set an environment variable by using the command line, that variable will be readable in your command line history. Consider clearing variables that contain credentials from your
command line history.  To keep variables from appearing in your history, you can use a script to prompt the user for their credentials, and to set the environment variable.
Failed to perform login command:
adal: Failed to execute the refresh request. Error = 'Post "https://login.microsoftonline.com/18ed93f5-e470-4996-b0ef-9554af985d50/oauth2/token?api-version=1.0": dial tcp <IP_ADDRESS>:443: i/o timeout' "
No mention of proxy. v.10.13.0 which works detects the system proxy(without any $env:HTTPS_PROXY entry in the user Powershell profile) :
INFO: Proxy detected: https://login.microsoftonline.com -> http://**************:8888
INFO: SPN Auth via cert succeeded.
With our network configuration azcopy needs to authenticate to Azure AD using the proxy, but for authorization to the storage account and file transfer it needs to bypass the proxy and go over ExpressRoute, the storage account network access settings prohibit access from public networks (public network access is set to Disabled, there is a private endpoint configured).
azcopy v.10.13.0 detects the system proxy without the Powershell HTTPS_PROXY environment variable being defined($env:HTTPS_PROXY). In the system proxy config we have instructions to bypass the proxy for blob storage account targets, so the authorization and azcopy file transfer works.
azcopy versions v.10.14.0-10.16.0 seem to not detect the system proxy. If I declare the proxy explicitly in the user Powershell profile using  $env:HTTPS_PROXY then all azcopy versions detect the system proxy and authenticate, but they fail to get authorized because I guess they don't honour the system proxy bypass settings and the storage account network access is only allowed over ExpressRoute (via the private endpoint) and not via the proxy. I get the error below:
User-Agent: [AzCopy/10.16.0 Azure-Storage/0.15 (go1.17.9; Windows_NT)]
X-Ms-Client-Request-Id: [7758c2d1-9e48-4913-699a-3ad461e23019]
X-Ms-Version: [2020-10-02]
RESPONSE Status: 403 This request is not authorized to perform this operation.
Content-Length: [246]
Content-Type: [application/xml]
Date: [Tue, 13 Sep 2022 15:56:59 GMT]
Server: [Microsoft-HTTPAPI/2.0]
X-Ms-Client-Request-Id: [7758c2d1-9e48-4913-699a-3ad461e23019]
X-Ms-Error-Code: [AuthorizationFailure]

Finally all azcopy versions from v.10.13.0 up to v.10.16.0 authenticate and get authorised so file transfer works (in Powershell).
The trick was to define the 2 Powershell environment variables with the values below:
$env:HTTPS_PROXY="http://ourproxyurlhere:proxyporthere"
$env:NO_PROXY="*.blob.core.windows.net"
Thank you very much for your help.

Awesome. I'm glad that all worked out for you. Have a nice day!


Totally pimp the Image Slicer (to be a  image iterator, to slice many images), and add unit tests for the slicer

This new code allows to write:
[ im_2d for im_2d in Slicer(im_3d) ]

as well as:
[ dice(x, y) for (x,y) in Slicer((im3d_x, im3d_y)) ]

illustrating a suggestion in https://github.com/neuropoly/spinalcordtoolbox/issues/1621#issuecomment-406765731

Except that a range is not a list in Python 3, so range is the proper term ...

... but still I can rephrase later, saying that it provides “indices for all the slices along the desired axis”.

(rephrased)


ListView initialListSize and pageSize does not work

hi, i'm trying to set the ListView initialListSize  and pageSize but it does not work, i'm still having all my datasource rendered, can someone help please ? thx ^^

+1

Bump

You probably have your ListView nested inside a ScrollView or something similar. You can either move it out or give your ListView a height (stylewise).
<ListView
  style={{height:400}}
  initialListSize={10}
  // ...
/>

Hi @aliouchka, this is an old issue. I'm hoping you've been able to resolve this issue on your own.

@ericnakagawa can You link this issue!


feat: use psr3-bridge

Pull Request Test Coverage Report for Build 693052924

-9 of 13 (30.77%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-7.1%) to 67.901%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Logger.php
2
11
18.18%

Totals

Change from base Build 688674143:
-7.1%

Covered Lines:
55

Relevant Lines:
81

💛  - Coveralls

Pull Request Test Coverage Report for Build 693052924

4 of 13 (30.77%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-7.1%) to 67.901%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Logger.php
2
11
18.18%

Totals

Change from base Build 688674143:
-7.1%

Covered Lines:
55

Relevant Lines:
81

💛  - Coveralls

@techoner @gaobinzhan plz review.

@basakest I found that easyswoole does not follow the PSR-3 specification, so there is no need to use psr3-bridge, just implement Casbin\Log\Logger directly.

@basakest Please add EasySwoole\Permission\Logger  to config, and then set logger during initialization.

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix(fhir.core): Use "createdAt" for point-in-time resource document...

...queries instead of "created"'s timestamp portion for backwards compatibility in datasets where no FHIR resource metadata is stored on version documents.

Codecov Report
Base: 64.23% // Head: 64.24% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (4031e11) compared to base (830f966).
Patch coverage: 33.33% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##                8.x    #1086   +/-   ##
=========================================
  Coverage     64.23%   64.24%
- Complexity    12467    12469    +2
=========================================
  Files          1734     1734
  Lines         58217    58219    +2
  Branches       5372     5373    +1
=========================================
+ Hits          37396    37400    +4
  Misses        18506    18506
+ Partials       2315     2313    -2

Impacted Files
Coverage Δ

...l/fhir/core/request/FhirResourceSearchRequest.java
76.23% <33.33%> (-0.27%)
:arrow_down:

.../snomed/core/ecl/SnomedEclRefinementEvaluator.java
85.27% <0.00%> (+0.25%)
:arrow_up:

...index/src/com/b2international/index/es/EsNode.java
82.29% <0.00%> (+1.04%)
:arrow_up:

...onal/snowowl/snomed/core/tree/TerminologyTree.java
87.23% <0.00%> (+2.12%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Refactor] Common place for types/interfaces/constants/helpers shared between client and server side

Move types/interfaces/constants/helpers shared between client and server side as many as possible into single place, to avoid duplicate codes for better maintenance.

As @yizheliu-amazon has mentioned in #329, we will want to move any shared constants on the server side, to prevent the issue of missing constants when the user builds and runs the artifact, since the client-side source files are ignored when building (details here)


Backend load failure on MacOS

I have a student reporting failure with these images after a clean install. I'll try to find out more.

I think this must be covered by other issues so I'll close it.


fix #1891 - fix ft.create type

Codecov Report

Merging #1901 (e984bfd) into master (e265c52) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Current head e984bfd differs from pull request most recent head 31f95f2. Consider uploading reports for the commit 31f95f2 to get more accurate results

@@           Coverage Diff           @@
##           master    #1901   +/-   ##
=======================================
  Coverage   95.05%   95.05%
=======================================
  Files         358      358
  Lines        3337     3337
  Branches      405      405
=======================================
  Hits         3172     3172
  Misses         83       83
  Partials       82       82

Impacted Files
Coverage Δ

packages/client/lib/client/index.ts
87.38% <0.00%> (ø)

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 e265c52...31f95f2. Read the comment docs.


Add .global.secretStore.backend in the clustergroup schema

It is currently not there even though we mention it in the values files.

Looks good!
Would it be good to also update the doc to show how turn off the vault with values-global.yaml?

global:
pattern: multicloud-gitops
secretStore:
backend:

/lgtm
Thanks @mbaldessari!

Agreed - lgtm


pattern based metadata through metadata spreadsheet

in many cases metadata follows a pattern for example appending an extension (eg. / or .html) to a canonical or appending a | Company Name to a title.
it would be great to have the ability to specify something like
{{title}}  | Adobe Blog for a title
or
{{url}}.html or {{url}}/ if the customer CDN provides a URL rewrite.

i think the pattern syntax should be aligned with what we would be using here: https://github.com/adobe/helix-html-pipeline/issues/461

can you come up with a list of properties that are valid replacements? or are they just all of the metadata available?
eg.

can I have a my-title in my metadata.json, which I can then use in the page-metadata?
or a my-title in the 1st row in the page-metadata, that I can then use in the 2nd row?
is url the canonncial url?

Wilson would love to have this feature


Drivers should report progress on the current action, (copying/downloading a file)

Author Name: Kiran Ranjane (Kiran Ranjane)
Original Redmine Issue: 2640, https://dev.opennebula.org/issues/2640
Original Date: 2014-01-13

There is no indication in Sunstone while the image is downloading which really makes us think how much time it would take to download the image, Sure we can check it from backend but it would be better to implement it in Sunstone.

Original Redmine Comment
Author Name: Ruben S. Montero (@rsmontero)
Original Date: 2014-01-17T08:51:56Z

Moving temporarily it to Request track

Original Redmine Comment
Author Name: Ruben S. Montero (@rsmontero)
Original Date: 2014-02-04T14:35:04Z

This turned to be something difficult to implement for this release, as it needs support from oned. Moving this to the backlog. Thanks for your feedback


fixes to make it work at least

Fixed the flow that never seemed to work

I'm sorry that I only now saw your PR because I've been extremely busy lately. I will review the code within the next two days and merge it.

You're right, those parts of the code are only minimally adequate for working conditions. But I think the people using these codes should have the ability to change them to the form they want, so there should be no problem merging them.


Bug: no output

Version: 0.0.6-beta3
Markshell: https://github.com/waldekmastykarz/cli-microsoft365/blob/552d9fd2b83f8c3487e1d8b274b4965b6275ec78/docs/manual/docs/cmd/spo/site/site-list.md
Expected: markshelled output
Actual: no output

The no output is caused by the following line:

command
spo site list

And fails with the following error.

command support is added in next drop. 👍

command missing is fixed in 📦<EMAIL_ADDRESS>

I've just got a similar issue there:

@alexlemaire you don't have any language specified that is why you get this issue.
Good Catch - The only change I can address this does not highlight the code at all.

Alright I see now, thanks for your reactivity!

Good Catch - The only change I can address this does not highlight the code at all.

In my specific case having markshell not highlight the code if nothing is specified would fit perfectly.
Or/and a global setting for markshell to specify a preferred default language. But that may be too restrictive/error prone...

It won't work that way because what is the preferred language if you have HTML, CSS, SASS and JavaScript on the same page.
In fact, GitHub doesn't highlight that code either, why put it in a code block anyway in the first place, when there is no benefit on specifying the language?

I'd agree with treating code block without language as plain-text without any highlighting.

That is one and the same ;)

Don't throw any exception any more and smooth highlight the code like this now:

First one is without any language.
Second one javascript
Third typescript

Fixed in - 0.0.6-beta6

I've just tried the new package version. No error anymore but when using markshell.toConsole(pathToMyMarkdown) it now outputs null into the console.
I cross-checked with fs.readFileSync using the same path and the file prints out. markshell.toRawContent works also fine.

@alexlemaire added now your readme file to my test. I will let you know where it is broken.

@alexlemaire Thanks for your readme - found issue and fixed it in<EMAIL_ADDRESS>

Awesome!
Another small thing I noticed: it looks like in code blocks and inline code, all _ characters are getting removed.
Here are 2 screenshots to see the difference between the raw markdown and the one getting printed:
raw

printed

I'm not sure whether this should still continue in this issue or if you would like me to open another one maybe? (since it's not anymore linked to a no output issue)

Another update is out - that should fix this issue too: version 0.0.8

Thanks a lot for the follow-up!
I think this might be related to the new fix: when using underscore _ to create italic text in markdown, the underscores are also printed out (examples below).
raw

printed

0.0.9 should fix this. Thanks for finding this small issues @alexlemaire


SQL Server 2014 T-SQL Reference Redirects to SQL Server 2017 T-SQL Reference

Describe the bug
I am looking for the T-SQL reference for SQL Server 2014.  Whenever I try to filter the content to the 2014 version, it redirects me to a 2017 page.
To Reproduce
Go to a link like the following:
https://docs.microsoft.com/en-us/sql/t-sql/functions/date-and-time-data-types-and-functions-transact-sql?view=sql-server-2017
From the drop down menu for SQL Server 2017, select the 2014 version.

This returns the following error:

Expected behavior
I would expect this to show the SQL Server 2014 version of the documentation.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop:

OS: Windows 10 Enterprise, version 1803, build 17134.1069.
Browser: Chrome, version 77.0.3865.90 (Official Build) (64-bit)

Additional context
We still have SQL Server 2015 systems in production, so I need to T-SQL reference as part of the support and development process.

@bwcassidy Hello Bruce, I believe this odd situation is known, and was the best solution that was available to us (odd situation).  There is some question as to whether certain SQL 2014 documentation should redirect to SQL 2016, or to SQL latest version.
I will look into this Issue # '2017' this week.
Thanks.


Fix broken links in documentation

As I was browsing around documentation I found some broken links!
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

Tugboat has finished building the preview for this pull request!
Link:

https://pr4418-2tz5om1yq56gjjsehndszjpivyqgpvxz.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6020f01698552d112c59347d

Tugboat has finished building the preview for this pull request!
Link:

https://pr4418-2tz5om1yq56gjjsehndszjpivyqgpvxz.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6020f01698552d112c59347d

Tugboat has finished building the preview for this pull request!
Link:

https://pr4418-2tz5om1yq56gjjsehndszjpivyqgpvxz.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6020f01698552d112c59347d


Added Chapter 01, Question 21. Added a simpler solution in Chapter 07, Question 06.

I noticed that Question 21 from Chapter 01 was missing, so I added that. Also, in Chapter 07, Question 06 strlen function was being used. The problem could be solved without it, so I added a simpler solution under the original one.
@farhana1i @pysaqib @ubaidmalik113 please review this commit.

@Nabeegh-Ahmed checked and verified. @farhana1i review the second file if we should do pusha popa or just stick with the existing push and pop for limited registers.

@Nabeegh-Ahmed checked and verified. @farhana1i review the second file if we should do pusha popa or just stick with the existing push and pop for limited registers.

@Nabeegh-Ahmed checked and verified. @farhana1i review the second file if we should do pusha popa or just stick with the existing push and pop for limited registers.

@PySaqib you are right. But I think for small codes, push those registers only which are used by the subroutine.

@Nabeegh-Ahmed checked and verified. @farhana1i review the second file if we should do pusha popa or just stick with the existing push and pop for limited registers.

@PySaqib you are right. But I think for small codes, push those registers only which are used by the subroutine.

@harismuneer I think this branch is ready to be merged, I have reviewed the code one last time and it appears to be fine. @PySaqib and @farhana1i if you could also look at it one more time, that will be really helpful.

@harismuneer I think this branch is ready to be merged, I have reviewed the code one last time and it appears to be fine. @PySaqib and @farhana1i if you could also look at it one more time, that will be really helpful.

@Nabeegh-Ahmed yes I think its better if its reviewed by more people :)

@Nabeegh-Ahmed yes I think its better if its reviewed by more people :)

closing this PR for now until further progress.


Balancer: Wildcard host behavior wrong

In 1.1.x, *.host.com correctly turned into hdr_end(.host.com).  In 1.2 it turns into hdr_end(host.com), which matches other things like otherhost.com.
https://github.com/rancher/lb-controller/blob/master/controller/rancher/rancher.go#L223-L231
Also we should add a way to do host.com + *.host.com in one rule, like **.host.com.

@vincent99 for 1.4, I'm going to address the first part of the issue. We can figure out the format for the host.com + *host.com in the future release separately.

Tested with rancher-server version - v1.4.0-rc5
Created LB with following port Rule:

haproxy config gets configured with  hdr_end(host.com)
frontend 8090
bind *:8090
mode http
acl 8090_host_com__host hdr_end(host) -i .host.com
acl 8090_host_com__host hdr_end(host) -i .host.com:8090
use_backend 8090_host_com_ if 8090_host_com__host

When accessing the LB with host set to "abchost.com" , it gets 503 Service Unavailable as expected.
Sangeethas-MBP:testwget sangeethahariharan1$ wget  --header="host: abchost.com" http://<IP_ADDRESS>:8090/name.html
--2017-01-30 14:47:02--  http://<IP_ADDRESS>:8090/name.html
Connecting to <IP_ADDRESS>:8090... connected.
HTTP request sent, awaiting response... 503 Service Unavailable
2017-01-30 14:47:02 ERROR 503: Service Unavailable.

Sangeethas-MBP:testwget sangeethahariharan1$ wget  --header="host: abc.host.com" http://<IP_ADDRESS>:8090/name.html
--2017-01-30 14:47:07--  http://<IP_ADDRESS>:8090/name.html
Connecting to <IP_ADDRESS>:8090... connected.
HTTP request sent, awaiting response... 200 OK
Length: 13 [text/html]
Saving to: ‘name.html’

100%[==================================================================================================================================================================>] 13          --.-K/s   in 0s

2017-01-30 14:47:07 (1.03 MB/s) - ‘name.html’ saved [13/13]


Unknown Error when adding component

I get the above error when trying to add the component and input the latitude and longitude.

Anything in your logs? What version of HA are you running? Running latest version of the component? v0.5

Just a thought.. did you restart HA after you added the component?

Logger: custom_components.bureau_of_meteorology.config_flow
Source: custom_components/bureau_of_meteorology/PyBoM/collector.py:66
Integration: Bureau of Meteorology (documentation, issues)
First occurred: 13:48:21 (1 occurrences)
Last logged: 13:48:21
Unexpected exception

Traceback (most recent call last):
  File "/config/custom_components/bureau_of_meteorology/config_flow.py", line 36, in async_step_user
    result = await collector.get_observations_data()
  File "/config/custom_components/bureau_of_meteorology/PyBoM/collector.py", line 47, in get_observations_data
    await self.flatten_data()
  File "/config/custom_components/bureau_of_meteorology/PyBoM/collector.py", line 66, in flatten_data
    for sub_observation in self.observations_data["data"][observation]:
TypeError: 'NoneType' object is not iterable

The above is in my logs. I am running 0.117 and 0.5.
I have restarted a couple of times.

Can you please share what co-ordinates you are entering? Send them via email (or even modify them slightly) if you have privacy concerns, moran.brendan@outlook,com

-33.8xxxxxxxxxxxx44
151.1xxxxxxxxx77
Hope that isn't too filtered.

I can see what the issue is, your weather station, "Sydney - Observatory Hill" is missing gust data. I will work on a new release that will work with missing gust data.
{
   "metadata":{
      "response_timestamp":"2020-10-29T03:07:30Z",
      "issue_time":"2020-10-29T03:01:02Z"
   },
   "data":{
      "temp":22,
      "temp_feels_like":20.5,
      "wind":{
         "speed_kilometre":17,
         "speed_knot":9,
         "direction":"SSE"
      },
      "gust":null,
      "rain_since_9am":0,
      "humidity":66,
      "station":{
         "bom_id":"066214",
         "name":"Sydney - Observatory Hill",
         "distance":4651
      }
   }
}

Just released v0.6 where it should be resolved.

Resolved.
Many thanks.

So Brendan... can the same fix be done for day 6? Also last night around 10 the UV/Fire data disappeared and the minimum goes to null as well. Can null just return unknown maybe or n/a?

So Brendan... can the same fix be done for day 6? Also last night around 10 the UV/Fire data disappeared and the minimum goes to null as well. Can null just return unknown maybe or n/a?

When they disappear, they become "None" in Python, which translates to a state of "unknown" in HA.
I'm not sure what the best/most elegent way is to handle each scenario when data disappears temporarily. I could probably code it in so it retains it's existing state.?

I could probably code it in so it retains it's existing state.?

Sounds like a good idea...


feat: support nested plugins

close #3437, supersedes #3528

Currently I simply flatten all nested plugins. @sxzz How should we handle nested plugins with circular references? How does vite handle that case?

Could you please make an example for me?

Ugh. For example:
const plugin = []

plugin.push(plugin)

@so1ve Vite and rollup won't handle this case. It's not a valid usage.

Thanks!


FIX #133 - Enable to specify runner and copy_mode for KedroPipelineModel.predict to decrease inference time

Description
Fix #133
Development notes

KedroPipelineModel.load_context now loads all the artifacts in MemoryDatasets
It is possible to specify the copy_mode for each dataset in KedroPipelineModel, PipelineML and pipeline_ml_factory (either by specifying a string to make all the datasets have the same mode, or a dictionary to eventually specify different copy_mode for each dataset).
It is possible to specify the Kedro AbstractRunner in KedroPipelineModel, PipelineML and pipeline_ml_factory

Checklist

[X] Read the contributing guidelines
[X] Open this PR as a 'Draft Pull Request' if it is work-in-progress
[X] Update the documentation to reflect the code changes
[X] Add a description of this change and add your name to the list of supporting contributions in the CHANGELOG.md file. Please respect Keep a Changelog guidelines.
[X] Add tests to cover your changes

Notice

[X] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Codecov Report

Merging #152 (82e8231) into develop (232efe0) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           develop      #152   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           25        25
  Lines          757       771   +14
=========================================
+ Hits           757       771   +14

Impacted Files
Coverage Δ

kedro_mlflow/framework/hooks/pipeline_hook.py
100.00% <ø> (ø)

kedro_mlflow/pipeline/pipeline_ml_factory.py
100.00% <ø> (ø)

kedro_mlflow/mlflow/kedro_pipeline_model.py
100.00% <100.00%> (ø)

kedro_mlflow/pipeline/pipeline_ml.py
100.00% <100.00%> (ø)

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 232efe0...82e8231. Read the comment docs.

Codecov Report

Merging #152 (82e8231) into develop (232efe0) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           develop      #152   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           25        25
  Lines          757       771   +14
=========================================
+ Hits           757       771   +14

Impacted Files
Coverage Δ

kedro_mlflow/framework/hooks/pipeline_hook.py
100.00% <ø> (ø)

kedro_mlflow/pipeline/pipeline_ml_factory.py
100.00% <ø> (ø)

kedro_mlflow/mlflow/kedro_pipeline_model.py
100.00% <100.00%> (ø)

kedro_mlflow/pipeline/pipeline_ml.py
100.00% <100.00%> (ø)

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 232efe0...82e8231. Read the comment docs.


Gradle 6.0 resolves wildcarded maven dependencies differently than Gradle 5.X

Expected Behavior
I have a working project that has a dependency on a maven artifact produced by a peer component like this:
repositories {
   ivy {
       url "../cnf/local"
   }
}

configurations {
  ejbTools
}

dependencies {
  ejbTools 'test:com.ibm.ws.ejbcontainer.fat_tools:1.+'
}

The complete build.gradle file can be found here: https://github.com/OpenLiberty/open-liberty/blob/master/dev/com.ibm.ws.ejbcontainer.async_fat/build.gradle
Current Behavior
The dependency test:com.ibm.ws.ejbcontainer.fat_tools:1.+ fails to resolve with Gradle 6.0 with the following error:
> Task :com.ibm.ws.ejbcontainer.async_fat:addEJBTools FAILED

FAILURE: Build failed with an exception.

* Where:
Build file '/Users/aguibert/dev/git/open-liberty/dev/com.ibm.ws.ejbcontainer.async_fat/build.gradle' line: 32

* What went wrong:
Execution failed for task ':com.ibm.ws.ejbcontainer.async_fat:addEJBTools'.
> Could not resolve all files for configuration ':com.ibm.ws.ejbcontainer.async_fat:ejbTools'.
   > Could not find any matches for test:com.ibm.ws.ejbcontainer.fat_tools:1.+ as no versions of test:com.ibm.ws.ejbcontainer.fat_tools are available.
     Searched in the following locations:
       - https://repo.maven.apache.org/maven2/test/com.ibm.ws.ejbcontainer.fat_tools/maven-metadata.xml
       - http://public.dhe.ibm.com/ibmdl/export/pub/software/olrepo/test/com.ibm.ws.ejbcontainer.fat_tools/maven-metadata.xml
       - file:/Users/aguibert/dev/git/open-liberty/dev/cnf/local/test/com.ibm.ws.ejbcontainer.fat_tools/
       - file:/Users/aguibert/dev/git/open-liberty/dev/cnf/local/test/com.ibm.ws.ejbcontainer.fat_tools/<IP_ADDRESS>909241016/ivy-<IP_ADDRESS>909241016.xml
     Required by:
         project :com.ibm.ws.ejbcontainer.async_fat

Context
Currently my project is using Gradle 5.5 and can be built with Java 8, 11, or 12. I am trying to get it working with Java 13 too so I am trying to upgrade to Gradle 6.0.
It seems that there is a general behavior change with the way wildcarded dependencies work in Gradle now (for example com.foo:bar:1.+). My configuration may have been invalid but tolerated in Gradle 5.X, so I'm willing to accept a pointer to a Gradle issue/pr/doc page that describes the behavior change and ideally the steps needed to get around this.
Steps to Reproduce
This will take a while, but you can do:

clone my OpenLiberty repo: https://github.com/aguibert/open-liberty/tree/master
Checkout the java13-gradle-bug-report branch
Run ./gradlew releaseNeeded and it will eventually fail

Your Environment
$ ./gradlew --version

------------------------------------------------------------
Gradle 6.0-20190923093257+0000
------------------------------------------------------------

Build time:   2019-09-23 09:32:57 UTC
Revision:     5b1d0543e4b76fc51aafb5d2fc757dcf086b0e4c

Kotlin:       1.3.50
Groovy:       2.5.8
Ant:          Apache Ant(TM) version 1.9.14 compiled on March 12 2019
JVM:          13 (Eclipse OpenJ9 openj9-0.16.0)
OS:           Mac OS X 10.14.6 x86_64

Thanks for trying Gradle 6.0!
This is caused by a breaking change in Gradle 6.0: by default, Gradle no longer resolves dependencies for which the metadata is missing. For a single dependency version, that means no *.pom or ivy.xml. For a dependency with a range, this fails also when the maven-metadata.xml is missing.
The simple solution is to configure the repository to tell it you want to search for artifacts without metadata. See the documentation on how to do that.

thanks for the reply @ljacomet, that is a really useful doc page. I found what you are describing in this section:
https://docs.gradle.org/nightly/userguide/upgrading_version_5.html#maven_or_ivy_repositories_are_no_longer_queried_for_artifacts_without_metadata_by_default
But I'm not quite clear on what combination of metadata source need to be added to get the old behavior. Before I had this:
repositories {
    maven {
        url "http://repo.mycompany.com/repo"
    }
}

And according to the doc it sounds like I only need to add the artifact() metadata source like this?
repositories {
    maven {
        url "http://repo.mycompany.com/repo"
        metadataSources {
            artifact()
        }
    }
}

However, now my build fails earlier, so I also added the mavenPom() source too, but I'm not sure if that's correct.
I tried consulting the relevant docs but it just says that metadataSources are an incubating feature in Gradle 5.6, but not what their default values are or what the possible inputs are besides artifact() or mavenPom()

Thanks for the feedback, will look into improving the documentation.
For Gradle < 6.0, the default was:
repositories {
    maven {
        url "http://repo.mycompany.com/repo"
        metadataSources {
            mavenPom()
            artifact()
        }
    }
    ivy {
        url "http://repo.mycompany.com/repo"
        metadataSources {
            ivyDescriptor()
            artifact()
        }
    }
}

and for Gradle >= 6.0 it becomes:
repositories {
    maven {
        url "http://repo.mycompany.com/repo"
        metadataSources {
            mavenPom()
        }
    }
    ivy {
        url "http://repo.mycompany.com/repo"
        metadataSources {
            ivyDescriptor()
        }
    }
}

So to restore pre 6.0 behaviour, you need to copy the old declaration. Note that if your repository never has metadata, you can also simply tell Gradle to look for artifact(), saving you a useless attempt at finding metadata.

thanks for the further explanation @ljacomet! I'll go ahead and close this issue since I understand what needs to be changed.


load method doesn't exist anymore

In the objC project, we used to initialize the handler with the load class method which was called at run time and doesn't exist anymore in Swift. The current workaround is to initialize our handlers directly in the AppDelegate, but it would be really better if we could allow an automatic initialization of the handlers, depending on which ones are selected on the handlers.json file.

solved with this pull request : https://github.com/fifty-five/Cargo-iOS-Swift/pull/9
Now handlers which are in the handlers.json are initialized


vg answer weights for vqa finetuning?

Hi,
I noticed that when you finetune on vqa downstream tasks, you set the answer weight for vg to 0.2, I wonder whether it's empirical or due to other reasons? Thanks

Hi, it is empirical. Thanks!

Ok, thanks a lot for your reply!


github-merge: Add option to overwrite user email

The merge script claims authorship of the merge commit, which is confusing:

The maintainer didn't author the merged code, nor the merge commit. Everything is done by the script.
In case of breach of a git hosting platform, such as GitHub, or the account of a maintainer, the hosting platform would allow a signed commit (signed by the hosting platform) to be pushed in the name of the maintainer. Thus, ...
... the important thing to check for is the commit signature or the actual code changes, not the name or email of the maintainer.
Most modern software projects attribute their merges to a "merge" account, regardless of who triggered the merge. (For example, rust-lang)

Solve this issue by:

Setting the git user name (committer and author) to "merge-script".
Allowing the maintainer to provide an email to use for the merge commit.

Just as before, the merge commit is still required to be signed by the maintainer.
Just as before, the git hosting platform will still denote which account was used to push the changes.

Reopen from https://github.com/bitcoin-core/bitcoin-maintainer-tools/pull/112

Recent merge commits, for example, https://github.com/bitcoin/bitcoin/commit/141115a0604001b8cce848804798dc174770a994, point at bitcoin-core-merge-script GitHub user. Is it required?

No, it is not required. I think it is up to the project that is using this script what to set the optional setting to, or leave it at the default value.
At least on GitHub, it seems to be possible to claim commits of unclaimed email addresses. Unclaimed means that the email is not tied to any GitHub account. While there shouldn't be any risk, projects on GitHub may prefer to pick an email that is claimed.
The account you are referring has an email tied to it on GitHub (90386131+bitcoin-core-merge-script@users.noreply.github.com), which can not be re-tied to another account? Also, the account was registered with an email account that is now deleted, so logging in to it shouldn't be possible either. If someone still manages to log in to it, there doesn't seem to be any risk?

See also the example output: https://github.com/bitcoin-core/bitcoin-maintainer-tools/pull/112#issuecomment-915969747

cc @achow101 @glozow

what does github do for those who see this

Concept NACK, and I've reverted this in my own version of the merge script.

The maintainer didn't author the merged code, nor the merge commit. Everything is done by the script.

This argument does not make any sense to me. That's like saying git is the one authoring merge commits when you use git merge, rather than the user who called it. While the tool does the heavy lifting, the maintainer is still the author as they took the explicit action to perform the merge. If you really want to be pedantic, then the real "author" is the person who wrote the boilerplate text that the script outputs into the merge commit message, but it would be insane for every merge to be "authored" by that person.

Most modern software projects attribute their merges to a "merge" account, regardless of who triggered the merge. (For example, rust-lang)

I think that's kind of ridiculous for us since we do not do automated merging. Furthermore, I think that paradigm is a bit ridiculous in general since maintainers should be held accountable for their actions, and part of doing that requires knowing who did what action. Obscuring who made the decision to merge something reduces maintainer accountability.
I understand that the actual merger identity is still preserved in the PGP signature attached to the commit, but git hides that information by default. Obviously verification of the signature is required to be sure of the identity, but it is easy to run the verify commits script in a cronjob and therefore looking at the log wouldn't require signature verification at that time, so having the author be the merger is safe and useful.
Although even with the verify commits script malicious maintainers could misattribute the author field by setting the user and email to a different maintainer (as the script does now), but that is easy to mitigate by having the script check that the name and email exists as a UID on the PGP key.


⚠ Bofrost.de has degraded performance

In bfbbb82, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1647 ms

Resolved: Bofrost.de performance has improved in 9f7ffbc after 2 minutes.


Deleted package detected

I'm a Cyber Security researcher and developer of PackjGuard [1] to address open-source software supply chain attacks.
Issue
During my research, I detected a deleted package in this repository.
Details
Specifically, the package libconfig-py mentioned in file README at line 12 does not exist on the public PyPI registry. A bad actor can hijack this package to propagate malicious code.
Impact
Not only your apps/services using https://github.com/Amitminer/LibConfig-PY repo code are vulnerable to this attack, but the users of your open-source Github repo could also fall victim.
You could read more about such attacks here<EMAIL_ADDRESS>Remediation
Please highlight this in file README and register a placeholder package for libconfig-py on public PyPI soon to remediate.
To automatically fix such issues in future, please install PackjGuard Github app [1].
Thanks!

PackjGuard is a Github app that monitors your repos 24x7, detects vulnerable/malicious/risky open-source dependencies, and creates pull requests for auto remediation: https://github.com/marketplace/packjguard

Thanks!


Improve gitignore

At the gitignore file, adding some comments can result in a better structure and readability.
I've added some files from some IDE I use (intelliJ and Vim).
Feel free to make any edition or addition to this PR :)

I like the idea, since I don't use any of those editors I didn't think about it but it's really useful. Thank you!


Error when uploading CSV file

A clear and concise description of what the bug is.
Expected results
CSV file uploaded into database
Actual results
Got error when uploading CSV file
Screenshots
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 2464, in __call__
    return self.wsgi_app(environ, start_response)
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 2450, in wsgi_app
    response = self.handle_exception(e)
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1867, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.6/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.6/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python3.6/site-packages/flask_appbuilder/security/decorators.py", line 109, in wraps
    return f(self, *args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/flask_appbuilder/views.py", line 94, in this_form_post
    response = self.form_post(form)
  File "/app/superset/views/database/views.py", line 141, in form_post
    delete=False,
  File "/usr/local/lib/python3.6/tempfile.py", line 551, in NamedTemporaryFile
    (fd, name) = _mkstemp_inner(dir, prefix, suffix, flags, output_type)
  File "/usr/local/lib/python3.6/tempfile.py", line 262, in _mkstemp_inner
    fd = _os.open(file, flags, 0o600)
FileNotFoundError: [Errno 2] No such file or directory: '/app/superset/app/static/uploads/tmpgzxvi893.csv'

How to reproduce the bug

Enable CSV upload
Upload CSV file

Environment
(please complete the following information):

First installation using Docker
superset version: master
python version: based on master branch
node.js version: based on master branch
npm version: based on master branch

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

Looks like your file is not present at the location from where you are trying to pick it up.
FileNotFoundError: [Errno 2] No such file or directory: '/app/superset/app/static/uploads/tmpgzxvi893.csv'

I tested by uploading the same file on other superset version which is 0.34.0 and it is uploaded

I tested by uploading the same file on other superset version which is 0.34.0 in same environment and it is uploaded. What does it mean by your statement here?

Looks like your file is not present at the location from where you are trying to pick it up.

This issue occurs in the default (master)

Cause:  Insufficient privilege, which doesn't allow a file creation in the Docker container environment.
Fix: giving it a root privilege
Please add user: root:root in your docker-compose.yml to fix the issue.
superset:
env_file: docker/.env
build: *superset-build
container_name: superset_app
command: ["flask", "run", "-p", "8088", "--with-threads", "--reload", "--debugger", "--host=<IP_ADDRESS>"]
restart: unless-stopped
ports:
- 8088:8088
user: root:root  # make this change
depends_on: *superset-depends-on
volumes: *superset-volumes

@reenarosid Okay got it. Thank you

It should be noted that the solution recommended by @reenarosid should only be used in development environments - if you're using the same dockerfile or docker-compose setup in a hosted environment, it's a questionable idea to give the application root privileges. We can likely add read/write permissions on the appropriate folder in the docker build to avoid the necessity of running as root. I'll take a look into that when I get a few minutes.

@willbarrett What is the status of this?

I haven't gotten a few minutes yet :)

I managed to get past this issue by creating a new folder in my dockerfile (RUN mkdir /csvs) then updating superset_config.py
# superset_config.py
UPLOAD_FOLDER = "/csvs/"


Question: Group-Object group order

Summary of the new feature / enhancement
Group-Object might be used to update and object list with an other list (See e.g. Stackoverflow: Compare CSV file with extra Column data):
$csv1 = ConvertFrom-Csv @'
Alert_ID, Server, Type, Timestamp,  Severity
    1234, Srv1,   WIN,  2023-03-10, Critical
    1235, Srv1,   Net,  2023-03-9,  Critical
    1236, Srv2,   WIN,  2023-03-8,  Critical
    1237, Srv3,   WIN,  2023-03-10, Critical
'@

$csv2 = ConvertFrom-Csv @'
Alert_ID, Server, Type, Timestamp,  Severity, Ticket
    1234, Srv1,   WIN,  2023-03-10, Critical, INC1
    1236, Srv2,   WIN,  2023-03-8,  Critical, INC3
'@

$csv2 + $csv1 |Group-Object -Property Alert_ID |Foreach-Object { $_.Group[0] } |Format-Table

Alert_ID Server Type Timestamp  Severity Ticket
-------- ------ ---- ---------  -------- ------
1234     Srv1   WIN  2023-03-10 Critical INC1
1235     Srv1   Net  2023-03-9  Critical
1236     Srv2   WIN  2023-03-8  Critical INC3
1237     Srv3   WIN  2023-03-10 Critical

But this highly depends on the order of the grouped objects
Questions:

(Not using any additional switches as -AsHashTable`) Is it safe to presume that the order of the grouped object list is the same as the order of the input list?
Is this part of the integration tests (or can it be integration tested, so that it is guarantied in the future)?

This feature (if existing at all) is not documented (or I missed it).
If it is safe to rely on the order, I will do a documentation update request

It's actually the (ascending) sort order of the input objects / grouping properties that determines the order of output groups, not the input order (in the code in the initial post the input objects happen to be sorted already), as the following examples demonstrate
 10..1 + 1..10 | Group-Object

Get-ChildItem -File  | Group-Object Length

But, yes, it's fair to assume that existing script rely on this behavior, and it definitely should be documented.

@mklement0 ,

It's actually the (ascending) sort order

Sorry, apparently there are 2 sort orders in play and I wasn't clear on that.
(I actually meant the inner grouped sort order.
$x = ConvertFrom-Csv @'
@,#
b,4
a,3
b,2
a,1
'@

As you defined , the outer sort order appears to be ascending:
"$(($x |Group-Object '@').Name)" |Should -be 'a b'

But about the inner group order{s}:
"$(($x |Group-Object '@')[0].Group.'#')" |Should -be '3 1'

Will they stay in order?
In other words, In the code in the initial post, I take the first occurrence in the grouped list ($_.Group[0]).
Can I safely presume that this is an object from the $csv2 list if the concerned property (Alert_ID) exists in both ($csv1 and $csv2) lists?

Thanks, good to know - I misunderstood the question.

Related document: https://github.com/MicrosoftDocs/PowerShell-Docs/pull/9892


tmt process even disabled objects

The following plan should rather not find anything than failing on unhashable type: 'DependencyFmfId'
/test:
    enabled: false
    require:
      - name: /library
        type: library
        path: ../tests
    duration: 5m
    test: ./runtest.sh
/plan:
    discover:
      - how: fmf
    execute:
        how: tmt

$ tmt run discover
/var/tmp/tmt/run-049

/plan
    discover
        how: fmf
        directory: /home/dapospis/test/git/SP_components/a

plan failed

The exception was caused by 1 earlier exceptions

Cause number 1:

    unhashable type: 'DependencyFmfId'

I would expect tmt not to process disabled objects.

Hm, leaving the enabled aside, using unhashable types like DependencyFmfId dataclass in set() is a nasty bug...

Trying to fix the error message with https://github.com/teemtee/tmt/pull/2300

I'm afraid there is no way of not processing disabled objects as processing is the 'fmf' (file format) responsibility and disable/enable comes from tmt (metadata specification).


add implementation of giving it live pv data

Pull Request
Description

Add live pv data to v1 model
added an example
TODO - run this with evaluation, perhaps with 1 hour of live pv data

Fixes #36
How Has This Been Tested?
Added an example and TODO run evaluation

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Hi @peterdudfield ,what does this pr do? Is this similar to #106 that was built on #66

Hi @peterdudfield ,what does this pr do? Is this similar to #106 that was built on #66

Can this be merged?

Yea, I let this sit there for a bit. I think what needs to happen is

resovle conficts
run (and update) evaluation code to see how much better the model is with this live PV data.

Do you fancy doing that @aryanbhosale

Hi @peterdudfield ,what does this pr do? Is this similar to #106 that was built on #66

Its realted to #66, but this mean users can enter their live PV data into the model. And yea, this could be done using the enphase API, or other methods.
Wanted to also run the evaluation to show how much better the model is by using this data

Oh that's great, are there any changes needed here ?

Hi @peterdudfield ,what does this pr do? Is this similar to #106 that was built on #66

Its realted to #66, but this mean users can enter their live PV data into the model. And yea, this could be done using the enphase API, or other methods.
Wanted to also run the evaluation to show how much better the model is by using this data

Oh that's great, are there any changes needed here ?

Totally up to you. You could build off this branch, or start a fresh, totally up to you.

Can this be closed @peterdudfield , since we've added live integration of 4 inverters


UIview and adding shapes

Hi, am trying to find the location where I can edit the UIview for several reasons, it would be a great help if you can help.

I need to shift the UIView upwards, how is this possible? I tried tracking down the code but no matter what values I changed it seemed the same
I want to test a few coordinates on the View, however, when I add a rectangle, I can't update it, and I believe this is related to the original problem of not being able to move the UIView up. I know this might be a simple issue I'm overlooking but if you can help, I would greatly appreciate it.
Thanks

I'm not exactly sure what you're asking... Which UIView do you want to shift?
Note that there are a couple of things going on in the app:

There is the camera preview layer. You can change how this looks by setting the videoGravity option.
There are the bounding boxes. These are translated to fit onto the preview layer. You may need to change the maths for this.

I mean mostly the main problem is not being able to add points to the UI View, if I add points it's not showing, even if I add to the sublayer, or am I adding to the wrong sublayer?

(points, rectangles and etc ^. Thats what I meant by points)

Also I'm trying to find the coordinates of the bounding boxes, but the values that come out are a little weird. Any tips on this as well?


gtk-doc: support python3

Motivation for this change
Make libbytesize (which depends on gtk-doc) build when using Python3 as default Python.
Things done

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

[ ] NixOS
[ ] macOS
[x] other Linux distributions

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

Success on aarch64-linux (full log)
Attempted: gtk-doc
Partial log (click to expand)
copying path '/nix/store/018a1qjj98pb08w8hflr0fymsrwr0pas-libxslt-1.1.32' from 'https://cache.nixos.org'...
copying path '/nix/store/asf5qq9nk50m17pbs1w12xjw78sy4znr-db-5.3.28' from 'https://cache.nixos.org'...
copying path '/nix/store/5lngsma59wrh8bk0yc469shar788ii5i-readline-6.3p08' from 'https://cache.nixos.org'...
copying path '/nix/store/2wxf06pcnlzj3xp2nfkakbnkbnmc1ap6-libxslt-1.1.32-bin' from 'https://cache.nixos.org'...
copying path '/nix/store/0bxsz2pw8n2f11lwwxwwbb8cdzaw2l9j-python-2.7.15' from 'https://cache.nixos.org'...
copying path '/nix/store/m374c4gjq562fpkszjlwnx76hqrn8kwa-dblatex-0.3.10' from 'https://cache.nixos.org'...
copying path '/nix/store/8fpglg263b1bfwha2zbk3pb20k6lsdzp-python2.7-setuptools-39.2.0' from 'https://cache.nixos.org'...
copying path '/nix/store/8gqgcz4p9frb3ccqmmfsnqaa3914bjdj-python2.7-six-1.11.0' from 'https://cache.nixos.org'...
copying path '/nix/store/4kp1pbzhpb050hhc1rkjpwh9nsmnifz8-gtk-doc-1.28' from 'https://cache.nixos.org'...
/nix/store/4kp1pbzhpb050hhc1rkjpwh9nsmnifz8-gtk-doc-1.28

Success on x86_64-darwin (full log)
Attempted: gtk-doc
Partial log (click to expand)
/nix/store/2n0hjylzh67mj7g15allhmjqzl52q68q-gtk-doc-1.28

Success on x86_64-linux (full log)
Attempted: gtk-doc
Partial log (click to expand)
these paths will be fetched (1.40 MiB download, 8.25 MiB unpacked):
  /nix/store/2lswm5had1cvnhdii36nagfip6aniph6-gtk-doc-1.28
  /nix/store/395s3hpsyh268g8za3k1pp0nywphdq14-dblatex-0.3.10
copying path '/nix/store/395s3hpsyh268g8za3k1pp0nywphdq14-dblatex-0.3.10' from 'https://cache.nixos.org'...
copying path '/nix/store/2lswm5had1cvnhdii36nagfip6aniph6-gtk-doc-1.28' from 'https://cache.nixos.org'...
/nix/store/2lswm5had1cvnhdii36nagfip6aniph6-gtk-doc-1.28

Success on x86_64-linux (full log)
Attempted: gtk-doc
Partial log (click to expand)
these paths will be fetched (1.40 MiB download, 8.25 MiB unpacked):
  /nix/store/2lswm5had1cvnhdii36nagfip6aniph6-gtk-doc-1.28
  /nix/store/395s3hpsyh268g8za3k1pp0nywphdq14-dblatex-0.3.10
copying path '/nix/store/395s3hpsyh268g8za3k1pp0nywphdq14-dblatex-0.3.10' from 'https://cache.nixos.org'...
copying path '/nix/store/2lswm5had1cvnhdii36nagfip6aniph6-gtk-doc-1.28' from 'https://cache.nixos.org'...
/nix/store/2lswm5had1cvnhdii36nagfip6aniph6-gtk-doc-1.28


Remove trailing spaces from generated Gopkg.toml

Sometimes IDE/editor removes trailing spaces automatically. It causes ridiculous git diff...

That got missed 😮 Thanks a lot! 👍


Add Ballerina forum sample

Purpose

$Subject

Closing this since another PR will be sent with the complete code


Upgrade chosen from 0.9.8 to 1.5.1

This is a BC breaking change, but in general, the equivalent of s/chzn|liszt/chosen/g will upgrade dependant code.

Looks like behat-extension is already using the new chosen notation for events, in parallel to the old liszt: https://github.com/silverstripe-labs/silverstripe-behat-extension/blob/805a0e69ed859f85daf325a687dd7a2532dcab5b/src/SilverStripe/BehatExtension/Context/SilverStripeContext.php#L496

needs rebasing already :/

see #4284

Yeah, almost all PRs that touch javascript will end up needing rebasing before merging now, side-effect of committing the compiled code.
Hadn't seen #4284, but having read through it, my feeling is rise-up isn't important enough to fork dependancy. It's only used in preview, and can be implemented in CSS if needed (as this PR does). I also wouldn't change to select2 unless there's a better reason than "it might be better?" - there's not a big BC break here, and select2 probably has it's own unfortunate issues too.

I'm testing and I'm finding lots of places where dropdowns are no longer being chosen-ified, although some places where they are.

The problem is that the shrinkwrap file doesn't include the chosen lib, so npm install won't install it, and rebuilding the libraries breaks distribution files.
As far as I can see, installing chosen with --dev is nigh impossible, because of the sheer amount of modules. My local machine ran out of memory about 4 times trying to install all deps.
If I could find out a way to update the shrinkwrap without install --dev then that would be fine, but no go.

I suggest we put this story off, and upgrade npm to 3, later.

I've fixed these issues, rebased, and pushed up.
CMS fix at https://github.com/silverstripe/silverstripe-cms/pull/1454.
Will merge once tests pass.


java-microprofile fails RH certification

The kabanerobeta/java-microprofile container fails RH certification on the following points....
Requires a summary label
Requires a vendor label
Requires a name label
Requires a description label

The labels need adding to the Dockerfile-stack (see nodejs for an example).

Fixed (master): https://github.com/kabanero-io/collections/pull/290
Fixed (release-0.6): https://github.com/kabanero-io/collections/pull/292

Latest stack changes had resolved the failure.


Chrome does not ask to save login credentials

Using your authorization system, I am asked to save my username/pwd on FF. However, on Chrome, this is not the case.
Popular solutions to this problem for ajax-based logins utilize a hidden iframe (e.g., https://gist.github.com/kostiklv/968927) but that does not prevent the page from being reloaded, which defeats the whole purpose of an SPA.
I would be very interested to hear your thoughts on how to address this issue - it seems very odd to me that Chrome, developed by the same company that made AngularJS, doesn't play well with SPAs in this regard.

This has finally been fixed in Chrome 46! :tada: No iframe hacks anymore!
Check out this example with descriptions.
If you are interested there are further examples in this repo. You can run it with node: node server.js.
Maybe also see the comments from this commit: https://github.com/mkurz/ajax-login/commit/c0d9503c1d2a6a3a052f337b8cad2259033b1a58)

@fnakstad I think this issue can be closed.

Yeah works for me now


Top line of tooltip not visible

RHV 4.2.1 VM Portal > VM details > Console tooltip
Top line of tooltip is not visible. Scrolling or resizing window does not help.

@mareklibra  - there's no severity or target milestone label on this.

Already exists issue for that https://github.com/oVirt/ovirt-web-ui/issues/568


Fixes a bug where not all of the similar types gets returned.

You can merge this right away

I don't like the removal of line breaks in the function signatures. I think only src/Tokens.cc should be commited. Maybe also the file_path refactor.

I have trouble reading the multiline functions when looking over lots of source code, but its just a personal preference. If you want, you can merge it and change it back. I do not have access to a computer for a few days

Well I quess I have the same argument. It's personal preference, however when we wrote the initial code our styleguide said no lines over 80 chars.

Thats from the old days when coding in a text terminal with fixed char width. When writing in juci this makes no sense;)


Github CI: Use dedicate job for rpm building

 * Use `actions/upload-artifact` and `actions/download-artifact` to share
   the rpm builds.

 * The rpm build will wait after the jobs of rust lint and python lint
   are all passed.

 * The integration test cases will wait rpm build job.

 * Merged all CI configurations into single one for this rpm sharing and
   test chaining.

Force merging as this is just test related.


Fixes/modifications to xgb schemas.

@hirzel , I also ended up removing some code from import_from_sklearn_pipeline that you might have added for round tripping of customize_schema. It caught validation errors on wrong values of hyperparams and created a planned operator instead. For multiple examples from the product, it ended up creating a TrainablePipeline and raised the "please fit before predict" error. So I thought it may be clearer to propagate the ValidationError error instead.


[question] Aeotec Nanomote Quad zwave events being blocked?

First, thank you for your work on this project. It's so much snappier than the old z-wave implementation.
I have an Aeotec Nanomote Quad z-wave device. When I press one of the keys, the logs show the following:
2021-11-25 05:05:30.077 DEBUG ZWAVE: Node 16: mapping value(s) of property 'level' (level) from CC 128 (Battery) to node property 'minBatteryLevel
2021-11-25 05:05:30.078 DEBUG ZWAVE: Node 16: mapping value(s) of property 'level' (level) from CC 128 (Battery) to node property 'batteryLevels
2021-11-25 05:05:30.078 INFO ZWAVE: Node 16: value updated: 128-0-level 28 => 0
2021-11-25 05:05:30.078 INFO ZWAVE: Node 16: value updated: 128-0-isLow false => true
2021-11-25 05:05:30.133 DEBUG GATEWAY: Skipping send of stateless value 16-91-0-scene-004: it's from cache
2021-11-25 05:05:30.134 INFO ZWAVE: Node 16: value notification: 91-0-scene-004 0
2021-11-25 05:05:31.136 DEBUG GATEWAY: Skipping send of stateless value 16-91-0-scene-004: it's from cache

In HomeAssistant, monitoring for * events in the developer tool section produces no result.
What can I do so that these value changes are not skipped? Any help is much appreciated!

@tboyko Are you using mqtt or zwavejs integration?

@robertsLando In the settings section I have Z-wave and HomeAssistant enabled. I've disabled the MQTT gateway by toggling the "Disable MQTT" gateway option.

In the settings section I have Z-wave and HomeAssistant enabled

If you mean you have zwave js server enabled it means this is handled on hass side and not by me. You should open an issue there. Dunno if @AlCalzone can help in this too but you can ignore those logs as them are on z2m side

@robertsLando I may be misspeaking as I'm not totally familiar with how this project works. My understanding is that it bridges z-wave interactions to MQTT, but that it can also bridge straight to HomeAssistant using some other sort of non-MQTT protocol. I'm not positive on how I have it set up, either, which is why I shared my settings in qwavejs2mqtt, above. I believe I'm doing a z-wave (js) to native HomeAssistant integration but could be wrong.
I also believe that the issue lies in this project, as the debug message I'm receiving Skipping send of stateless value is from the codebase. I assume that this project is not forwarding the event on to HomeAssistant.

This project can actually be used as control panel, mqtt gateway (if you have mqtt enabled, that's not your case), and could also init zwavejs server that can be used paired with hass zwavejs integration to control your devices directly from hass. If the problem is that

In HomeAssistant, monitoring for * events in the developer tool section produces no result.

It means it is something related to the zwavejs server that is not forwarding events to the hass integration and I have no control of that, I just init the server but the logic inside it is out of z2m control. On my side based on the logs you shown me it seems like your device is not marked as ready for some reason BTW I would need a full log file to check why this is happening

Ok, I see what you are saying, but still a bit confused on why "the logic inside it is out of z2m control" since it's in the codebase, here. But, since I already shared that, I'm guessing that I'm missing something.
In the event that it is out of this project's control, should I be contacting the Home Assistant project?
I'm happy to provide additional logs. What is helpful?

I re-paired the device with my z-wave network and the issue seems to have resolved itself. 🤦 Apologies for the trouble!

"the logic inside it is out of z2m control" since it's in the codebase

You right in fact that comes from the codebase, but the events I manage on my side are also managed by zwavejs server: https://github.com/zwave-js/zwave-js-server
On startup of zwave client on my side if zwavejs server option is enabled I start it passing zwavejs driver as argument then everything they send to hass is managed by them, not z2m :) Glad you fixed your problem BTW, hope I have been more clear now about why I said that. If you have some problems with control panel or else let me know :)


Code generator should run as part of interop tests

Currently the Python interop tests don't use generated code. Not only should they but the generated code should be generated as part of the test (rather than checked in as is the case with some of the other languages).

@kpayson64: While you're in the neighborhood, if this looks doable, please feel invited to take it on. :-)

I believe this work has been completed:
We are already using an auto-generated stub,
https://github.com/grpc/grpc/blob/master/src/python/grpcio/tests/interop/client.py#L104
As of #6229, all generated generated python code gets built out of the shared src/proto/... as part of the test.
Feel free to reopen if I'm missing something.


[apptools-android] Add tests to test fullscreen apps for apptools android

Impacted tests(approved): new 1, update 0, delete 0
Unit test platform: [Android]
Unit test result summary: pass 1, fail 0, block 0
https://crosswalk-project.org/jira/browse/XWALK-4680

LGTM

Please resolve the conflict.


Switch to jax.random.key

TODOs:

[x] update docs

closes #581

LGTM! are PRNGKey(0) and key(0) the same? did anything change in the test' values?

They are the same:
>>> import jax
>>> jax.random.key(0)
Array((), dtype=key<fry>) overlaying:
[0 0]
>>> jax.random.PRNGKey(0)
Array([0, 0], dtype=uint32)


Inconsistency between --lsa via NTLM and Kerberos

It looks like there is some inconsistency between the result of the --lsa option when connecting via NTLM and Kerberos:
Below is the result when dumping lsa via a NTLM authentication:

And here is the result when dumping via Kerberos (-k):

The AES and DES keys for the DC$ account are not dumped.
I'll take a look at why ASAP.

Maybe something is wrong in impacket. 🧐

@Dfte This is a Really nice patch, I think this should be made a PR in fortra's upstream

I'll PR this in the afternoon yeah!

PR's done, we'll have to wait for it to be merged :)
https://github.com/fortra/impacket/pull/1628

Hello @Dfte can you do the same pr on https://github.com/Pennyw0rth/impacket/tree/gkdi ? :)

Closed as fixed in https://github.com/Pennyw0rth/impacket/pull/3

Reverted, PR on impacket has not been tested once i think:

there is space on tab mix on the PR
if you fix the space / tab problem, you have a


Find certificate with selector - part 2

Add the fix also for AKS and GKE.

Successfully manually test on AKS.


Add link to xfsharp gitter room

I've just created a Gitter room for this repository at: https://gitter.im/exercism/xfsharp We should add this link to the documentation.

@ErikSchierboom Added the PR here

@tushartyagi I think it would also make sense to add the link to the docs/resources.md file, which contents are displayed on exercism.io. Would you be willing to also send a pr for that?

@ErikSchierboom created PR #318 for ^^

Perfect.


Tools: add auto fingerprinting tool

enter a route and platform (or no platform if the platform can be determined automatically with fuzzy fingerprinting), and it will auto place firmware versions to the correct spot
TODO:

[ ] preserve comments next to existing fw versions?
[ ] run the formatting script on existing fingerprints to normalize them

my thought was this could be run in a CI job, if someone requested it in an issue or PR, and it could automatically create a PR
!fingerprint '8ae405c83e79aec8|2023-09-03--17-00-45' 'SUBARU IMPREZA SPORT 2020'

@jnewb1 Please make sure to delete your branches after closing the PR


Uncaught Error: ENOENT: no such file or directory, lstat '~/AppData\LocalLow\1CGS\Caliber\SystemC...

[Enter steps to reproduce:]

...
...

Atom: 1.44.0 x64
Electron: 4.2.7
OS: Майкрософт Windows 10 Pro
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT: no such file or directory, lstat 'C:\Users\Homeb\AppData\LocalLow\1CGS\Caliber\SystemCfg_v2\ папки и интернет-хранилище-Библиотеки'
At fs.js:115

Error: ENOENT: no such file or directory, lstat 'C:\Users\Homeb\AppData\LocalLow\1CGS\Caliber\SystemCfg_v2\ папки и интернет-хранилище-Библиотеки'
    at Proxy.realpathSync (fs.js:1435:7)
    at Proxy.fs.realpathSync (ELECTRON_ASAR.js:306:40)
    at TreeView.moveEntry (~/AppData/Local/atom/app-1.44.0/resources/app/static/<embedded>:14:1005063)
    at TreeView.onDrop (~/AppData/Local/atom/app-1.44.0/resources/app/static/<embedded>:14:1011992)
    at HTMLDivElement.<anonymous> (~/AppData/Local/atom/app-1.44.0/resources/app/static/<embedded>:14:985623)

Commands

Non-Core Packages
teletype 0.13.4

Thanks for reporting this!
This has already been fixed in https://github.com/atom/tree-view/pull/1358. It is currently scheduled for release when v1.53.0 reaches Stable.


fix(develop): HTTP/2 error: Stream error #486

New Pull Request Checklist

[x] I have read the Documentation
[x] I have searched for a similar pull request in the project and found none
[x] I have updated this branch with the latest develop to avoid conflicts (via merge from master or rebase)
[x] I have added the required tests to prove the fix/feature I am adding
[x] I have updated the documentation (if necessary)
[x] I have run the tests and they pass

This merge request fixes / refers to the following issues: ...
Pull Request Description
Fix DioError [DioErrorType.DEFAULT]: HTTP/2 error: Stream error: Stream was terminated by peer (errorCode: 1)
Ref
https://github.com/dart-lang/http2/issues/49
https://github.com/flutterchina/dio/issues/486

@ueman @kuhnroyal can you please review this and merge it?

By the way, some unit tests should be added for sure.

This PR does not resolve the issue from what I can see.

Hi everyone! We've made our hardfork repo public and published a new version of dio, named diox.
The new package contains the PR of the fix.
Please refer to https://pub.dev/packages/diox/versions/5.0.0-dev.1 to use the fork.
You can also see why we're working for a hardfork at https://github.com/cfug/diox/issues/29 and https://github.com/flutterchina/dio/issues/1607.


Is there an upper limit to the length of the "ask" method response string?

When we create a Index using "GPTSimpleVectorIndex" the index becomes a class. So we can use the method "ask" of this class. Then we have a "response" as a result.
Is there an upper limit to the length of the ask method response string?
It seems to me that in some cases the response string is cut, when the sting response is big.

@BrunoAPazetti the response string can be cut off, usually because the default max_tokens from a single OpenAI call is 256
See this page here for extending that: https://gpt-index.readthedocs.io/en/latest/how_to/custom_llms.html#example-changing-the-number-of-output-tokens-for-openai-cohere-ai21

@logan-markewich's answer is right on. Closing issue for now.

Thank you!

@BrunoAPazetti the response string can be cut off, usually because the default max_tokens from a single OpenAI call is 256
See this page here for extending that: https://gpt-index.readthedocs.io/en/latest/how_to/custom_llms.html#example-changing-the-number-of-output-tokens-for-openai-cohere-ai21

Seems like this link no longer works. Is there another resource that describes how to fix this issue?

@jmcrook
https://gpt-index.readthedocs.io/en/latest/how_to/customization/custom_llms.html#example-changing-the-number-of-output-tokens-for-openai-cohere-ai21


Raise error rather than default fallback for model selection

In the following code a ValueError or something should be raised if no value have been matched rather than silent fallback to completion model.
As well, since select_model_type is only ever called by get_model it is impossible for the default kwarg to ever be used, perhaps worth removing
def select_model_type(model_interface: str='gRPC', task: str='complete'):
    if model_interface == 'gRPC':
        if task == 'embed':
            return RpcEmbeddingLanguageModel
        return RpcCompletionLanguageModel
    return RpcCompletionLanguageModel

def get_model(model_id: str, metadata: dict=MODELS_ZOO, task: str='complete'):
    if model_id in metadata.keys():
        model_interface = metadata.get(model_id).get('network', dict())
        model_url = model_interface.get('url', None)
        model_interface = model_interface.get('type', None)
        return select_model_type(model_interface, task)(name=model_id, url=model_url)
    else:
        return None

refer #3 for further discussion around this. that fix is no longer included in any branch

Forgot to close it but been addressed in #32


Chromium on Ubuntu 22.04 LTS not recognized

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
Uptime Kuma doesn't recognize /snap/bin/chromium as a valid path for chromium. This path is the default install path for Ubuntu 22.04 LTS when using canonical repositories and should probably be considered valid.
I believe getting it allowed is as simple as adding /snap/bin/chromium and /snap/bin/chromium-browser to server/monitor-types/real-browser-monitor-type.js at line 45. The only other step is ensuring this path is auto-detected properly.
👟 Reproduction steps

Install Uptime Kuma on a fresh Ubuntu 22.04 LTS install using the Manual method
Install chromium-browser via apt install chromium-browser
Notice that apt redirected to snap, so chromium is located at /snap/bin/chromium
Attempt to set the chrome executable to this snap location

👀 Expected behavior
Expected /snap/bin to be an acceptable location for Uptime Kuma
😓 Actual Behavior
Was prompted to set the environment variable to allow chrome to exist anywhere
🐻 Uptime-Kuma Version
1.23.6
💻 Operating System and Arch
Ubuntu 22.04 LTS
🌐 Browser
N/A
🐋 Docker Version
N/A
🟩 NodeJS Version
20.8.0
📝 Relevant log output
Location of log output other than docker is not documented

Given that you already have debugged where this needs to be added, I think this would be a prime target for a first PR. ^^
Here is our contribution guide: https://github.com/louislam/uptime-kuma/blob/1550a5f79270f39007086d8d23c180ba9f63096b/CONTRIBUTING.md

Well, that's the fun part. I've already tested adding the path to that array. Trouble is, the chromium path is supposed to be autodetected using paths in that array. Typing /snap/bin/chromium manually does work, but something is still broken here.
The command-exists dependency correctly checks a given path and the findChrome function looks like it iterates over the array and either returns the path correctly or some other function has a bug. The text of the error in the browser comes from browserType.launch, a function that probably exists in a dependency that I can't see at the moment.
Further travel down this rabbit hole would be detrimental to my class work, so I have to return to that - thus, this issue instead of a pull request.

I always thought Ubuntu and Debian are using the same chromium package, but they are not.
Will try to add /snap/bin/chromium


Sum and Difference of Two Numbers

Problem Link - https://www.hackerrank.com/challenges/sum-numbers-c/problem

Please verify the problem and assign it to me. I will do it in C.

I will do it in java

@Nivedita967 i would like to contribute in python

@Abhichal18 Are you working?

I would like to code on this issue in C++.Kindly Assign me

I can code it in C++

@jayantjethi Go for it :)