how to disable additional section totally.

environment
unrelated
actual behavior
"additional section" in statusline takes too much spaces
expected behavior
hide/disable "additional section" totally.

what additional sections?

@chrisbra  the last section in statusline
+-----------------------------------------------------------------------------+
|~                                                                            |
|~                                                                            |
|~                     VIM - Vi IMproved                                      |
|~                                                                            |
|~                       version 8.0                                          |
|~                    by Bram Moolenaar et al.                                |
|~           Vim is open source and freely distributable                      |
|~                                                                            |
|~           type :h :q<Enter>          to exit                               |
|~           type :help<Enter> or <F1>  for on-line help                      |
|~           type :help version8<Enter> for version info                      |
|~                                                                            |
|~                                                                            |
+-----------------------------------------------------------------------------+
| A | B |                     C                            X | Y | Z |  [...] |
+-----------------------------------------------------------------------------+

 section|meaning (example)
-------|------------------
  A    | displays the mode + additional flags like crypt/spell/paste (INSERT)
  B    | VCS information (branch, hunk summary) (master)
  C    | filename + read-only flag (~/.vim/vimrc RO)
  X    | filetype  (vim)
  Y    | file encoding[fileformat] (utf-8[unix])
  Z    | current position in the file
 [...] | additional sections (warning/errors/statistics) from external plugins (e.g. YCM, syntastic, ...)

I can customize other sections like bellow:
let g:airline_section_x = ''
let g:airline_section_z = '%3p%% %3l/%L:%3v'

but don't know how to customize/disalbe the additional sections.
i even tried customized layout without the additional sections:
function! AirlineInit()
    let g:airline#extensions#default#layout = ['a', 'b', 'c', 'x', 'y', 'z']
endfunction
autocmd VimEnter * call AirlineInit()

still with no luck.
Is there any configuration vars control its behavior ?  thanks.

You need to disable the extensions, that fill that section. Most likely like the whitespace extensions, ale, ycmd, syntastic, neomake and languageclient (depends on what vim plugins you have installed in addition to vim-airline, if you need to disable each extension. Check the output of the :AirlineExtension command)

So these are other extensions doing the trick. Thank you, :AirlineExtension is a good start point.

@jieteki  Did you finished this problem? how to do that when I see the doc of airlineExtention.

@Adolf-L I just did this today, you can run the :AirlineExtension command to see what extensions are enabled, and then disable them one by one until you find the offender.. in my case it was the whitespace extension that was annoying me

You need to disable the extensions, that fill that section. Most likely like the whitespace extensions, ale, ycmd, syntastic, neomake and languageclient (depends on what vim plugins you have installed in addition to vim-airline, if you need to disable each extension. Check the output of the :AirlineExtension command)

how about you making an option to disable that additional sections at all? wouldn't that be more simpler, instead of me iterating through every plugin that i have loaded using :AirlineExtension and check manually if that one was doing something in the statusline additional sections section. huh?

I found that command in the official doc:
" an empty list disables all extensions
  let g:airline_extensions = []

Works for me.

work for me. After search from the list of AirlineExtension. I found the whitespace extension take up too much space on my situation. So I just copy the extension list from AirlineExtension to the let g:airline_extensions = [  ... ] to enjoy all other good things and disable the annoying whitespace

Hello, everyone.
Can you give me an example?
let g:airline_extensions = [ whitespace]
I get the following error "E121: Undefined variable: whitespace"

You need to quote the whitespace Am 27.01.2024 um 19:57 schrieb Jorysm @.***>:
Hello, everyone.
Can you give me an example?
let g:airline_extensions = [ whitespace]
I get the following error "E121: Undefined variable: whitespace"
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Simply use the command.
:AirlineToggleWhitespace
The trailing whitespace checker component of the status line will be removed.


configure ciphers for rabbitmq with enabled ssl

Proposed solution for #183

@jaredledvina I just added the changelog entry. Is the entry ok for you?


Exclude --layers from the common bug cli flags

Other applications like podman would like to switch the the default value
for --layers.  Because we re-use this code, we just need to pop layers out
of the budflags.
Signed-off-by: baude<EMAIL_ADDRESS>

LGTM

@rh-atomic-bot r+
@TomSweeneyRedHat He is going to change the default of --layers on podman.


Add a simple static method to strip non-XML responses from debug log

We could add other types (like JSON) in the future, or if I missed any in the CR I can fix them up, it's a simple fix.
The more complicated approaches, that I abandoned, were to try and grab some chunk of the file and find a safe way to encode it to utf8 for either file or stdout streams -- but that didn't seem worth it :)
Fixes #273

:rocket:


Added functional Foothill scraper

@jodizzle Wow. Scrapy is amazing.  I didn't realize it could handle weird forms like that.  Looks a lot better now.

Looks good.  OK to merge.


Complex Encoder-Decoder System for Titanic like Data

Hi,
I was wondering if there is a way to use complex encoder system for titanic-like binary classification? What would you recommend to enhance performance on this data?
Thank you.

Hi @GaganaB , I'm not really sure what do you mean by "complex encoder system". My best guess is that it means a combination of parameters for the encoders of each feature that can squeeze more performance on that dataset, am I correct?
I don't have an easy answer here, because i haven't spent much time optimizing for that dataset specifically (as it is kinda pointless anyway, given the toy nature of the problem), but there are a few things I can point out that could be useful:

change some of the preprocessing parameters for numerical features, for instance use z-score or minmax normalization
play around with the combiner parameters, specifically the number and size of layers. If you make them too big, the model will likely overfit, but you can compensate by increasing regularization and dropout
play around with the threshold of the binary output feature, as the best f1 my come from either increasing or decreasing it to either improve accuracy or recall

Moreover, we are considering adding additional preprocessing for numerical features, binning and binarization in particular, as it has been shown in some cases that by doing so NN models can outperform tree based models also on tabular data, see keep an eye out for when we are going to add it to Ludwig!

Thanks a lot. I think this works. Is there a way in Ludwig to try model ensembling?

@GaganaB you are welocme. And no there's no ensembling mechanism within Ludwig, you can easily build your own as training of Ludwig models in the API can be done in 2 lines of code, you can train a bunch, collect their predictions and probabilities and then train another one on top of those. Not sure doing it for the Titanic dataset makes sense though :)


Ensure navigator unlocks scrolling when navigating away

Bug/issue #, if applicable:  closes #293
Summary
Fixes a bug where the navigator would not unlock scrolling on mobile devices.
Dependencies
NA
Testing
You need an archive with Tutorials, like sloth creator
Steps:

From a doc page open the navigator on a mobile device
Click on a link, that leads to a tutorial
Assert scrolling is possible again

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

@swift-ci test

@swift-ci test


[AIRFLOW-1383] Add no_dag_reset option to airflow clear command

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title.

https://issues.apache.org/jira/browse/AIRFLOW-1383

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

The CLI command airflow clear does not expose any way to set the reset_dag_runs parameter of the underlying DAG.clear() method.
This means that airflow clear will always cause affected DAG runs to be set to state=RUNNING, and immediately try and re-run the cleared tasks. In some circumstances it might not be desirable to immediately re-run everything.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

A simple test for the CLI syntax is added to tests/core.py
Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2420 into master will decrease coverage by 0.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2420      +/-   ##
==========================================
- Coverage   69.25%   68.94%   -0.32%
==========================================
  Files         146      146
  Lines       11230    11230
==========================================
- Hits         7777     7742      -35
- Misses       3453     3488      +35

Impacted Files
Coverage Δ

airflow/bin/cli.py
51.86% <ø> (ø)
:arrow_up:

airflow/executors/dask_executor.py
30.23% <0%> (-51.17%)
:arrow_down:

airflow/executors/base_executor.py
91.07% <0%> (-3.58%)
:arrow_down:

airflow/settings.py
86.41% <0%> (-2.47%)
:arrow_down:

airflow/executors/__init__.py
58.53% <0%> (-2.44%)
:arrow_down:

airflow/utils/dag_processing.py
85.36% <0%> (-0.82%)
:arrow_down:

airflow/utils/helpers.py
55.49% <0%> (-0.58%)
:arrow_down:

airflow/jobs.py
74.84% <0%> (-0.21%)
:arrow_down:

airflow/www/views.py
67.48% <0%> (-0.15%)
:arrow_down:

airflow/models.py
87.26% <0%> (-0.05%)
:arrow_down:

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 d231dce...4a62b7e. Read the comment docs.

The CI build succeeds against sqlite but hangs against postgres/mysql - any suggestions why?

I'm unsure why the landscape check reports reduced code quality. The report there appears to reference code not touched by this pull request.

@ashb I've extended the test to assert the state of a created DAG run before and after clearing, which should cover the intended functionality.
I haven't changed the CLI option - as per the comment above, the current airflow/bin/cli.py:CLIFactory does not provide an option to rename the output variable from the CLI flag; wider changes there are probably out-of-scope. The not args.foo idiom is used a number of other times in any case.
One of the kubernetes tests failed for reasons that look unrelated and probably transitory. All the other tests passed.


🚀 verification 방법 변경

This PR removes the Google Analytics plugin and related configurations from Gatsby and deletes an obsolete ads.txt file. Additionally, it adds 'google-site-verification' HTML file for site verification.

#7  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mingi3314 and the rest of your teammates on  Graphite


gen_pose_file.py: rewrote the file

Argument checking up is invalid and not user friendly
Don't use print to write file
Rename pcd_file to captical letter since it is iteral, myabe I would refactor a couple of files to consolidate this kind of stuff to another generic file in the future.
Rename main() to meaningful method name.

Please submit a pull request for the changes you would like to make. Currently closing this issue. Thanks!


Using the wrong tooltip property for the "Finish" button.

https://github.com/CodeGator/CG.Blazor/blob/ecf4f3c291fd8a0eb89ae6f96a65dcc7322cf9ff/src/CG.Blazor/Components/Wizard/Wizard.razor#L119-L121
Hi!
Just tested your wizard component and found a little bug.
You've used the "NextButtonTooltip" for the Finish button instead of "FinishButtonTooltip".
Have a nice day!

Great catch! Thanks for letting know. Fixed in the next build.


Optimization suggestions for search

I hope the search can ignore case and support reverse order, for example, searching for "load image" includes:“
"IMAGE load XXXX"

The search is already ignoring case, for the reverse order is not a really easy thing to do, but i'll try my best

They're basically asking for a tokenized search where you split the input string based on whitespace and/or non-"word" characters, perform the search for each token, then merge the results.
Note: this could be annoying in some cases where a user already knows what they want.. e.g. IPAdapter Apply (yes I know that was changed recently).. and then it ends up returning every single node related to IPAdapter and Apply (which is less than ideal).

They're basically asking for a tokenized search where you split the input string based on whitespace and/or non-"word" characters, perform the search for each token, then merge the results.
Note: this could be annoying in some cases where a user already knows what they want.. e.g. IPAdapter Apply (yes I know that was changed recently).. and then it ends up returning every single node related to IPAdapter and Apply (which is less than ideal).
If they truly mean "reverse" because that's the way they're used to speaking (e.g. subject-verb vs verb-subject) then I can't see any non-trivial way that could be done predictably and I wouldn't even try.

True enough, but this could be easily overcome by merging a 'standard' search and a tokenized search and sorting the results with the 'standard'  search first.  Or simply put a checkbox/switch for preference.

Sooo, I got the fuzzy search working shortly after the first request to improve the search in the Sidebar, I used fuse.js and it worked decently then I found another implementation that seemed to work better so I decided to go with that one, but I stopped because of this reverse thing, it's not there I don't have half an idea how to implement it but I just don't want to make it too slow and imprecise
And yes, 100% I will make an option in the menu to switch from standard to fuzzy which I will probably enable by default at that point!

Ok, I've implemented the fuzzy search, which is working even in reverse. You can disable it in the settings if you still want to use the default one!


git clone issue

Hi,
i would like to clone your reply but get this error:
git clone https://github.com/carbocation/go-instagram /go/src/github.com/carbocation/go-instagram
Cloning into '/go/src/github.com/carbocation/go-instagram'...
fatal: unable to access 'https://github.com/carbocation/go-instagram/': Failed to connect to github.com port 443: Connection timed out
git clone https://g...content-available-to-author-only...b.com/carbocation/go-instagram /go/src/github.com/carbocation/go-instagram
Thanks,
Gerald

I cannot reproduce this issue. It may be a github issue or an issue with connectivity, perhaps?

i guess a github connectivity issue, its gone now... thanks for checking
2015-04-19 0:02 GMT+02:00 carbocation<EMAIL_ADDRESS>
Closed #8 https://github.com/carbocation/go-instagram/issues/8.
—
Reply to this email directly or view it on GitHub
https://github.com/carbocation/go-instagram/issues/8#event-284560344.


ci/github: Minor fix for untrusted code

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

pushing through to fix ci


In-app message was closed automatically without the user interaction while app launch.

Recently i have tried in firebase inapp messaging in our iOS application.
In-app message was closed automatically without the user interaction while app launch.
In-app message works well app in logout mode. Issue will happens only on home screen. Please suggest me a solution for this problem.
Thanks in advance

Thanks for reporting, @kumareshsaran. Please provide the details from the issue template to have a better understanding of the issue.


Manifest file is missing or unreadable Could not load manifest.

When I added a folder with an extension in the browser.
Manifest file is missing or unreadable
Could not load manifest.
But manifest.json is in the extension folder

me too

same here

Got the same issue. Solved it. It's not missing. Just change the folder to ....chatgpt-chrome-extension\extension. It's not working because the actual .json file is not in the folder specified, but within a folder in there. Just pick the folder called Extension instead and it will work.


[Question]can react-datasheet work with Redux?

I start on boarding an existing application and learn  react and Redux. I have a application using Redux  and it uses axios to retrieve remote service url. The return dispatch a action to update state to call the parent component's render to show a data sheet to make user input form. The sheet data is a promise or not available when the render firstly called. I see the log, first time the render called, data in is a empty object, on following calls,  real data comes.
I see your examples that data are given as property of component in react render function.  Redux enables react app to update state by using action, and render being automatically called to refresh display.
My question, can react-datasheet working with Redux? or if underline data in state changes, what to do to let the sheet to refresh display with new data?
Any advice on my case?
Thanks,
Chen

We're happily using it with redux. We the react-redux useSelector hook and then just set the data props on the table from that and it will update automatically since it is 'declarative' not 'imperative'


Update TensorFlow module to Swift 5.

Done while investigating TF-326, which showed a note related to Swift version.
<unknown>:0: error: fatal error encountered while reading from module 'TensorFlow'; please file a bug report with your project and the crash log
<unknown>:0: note: compiling as Swift 5.0, with 'TensorFlow' built as Swift 4.1.50 (this is supported but may expose additional compiler issues)

*** DESERIALIZATION FAILURE (please include this section in any bug report) ***

Updating to Swift 5 didn't fix the issue, but it's a desirable standalone change.

@swift-ci Please test tensorflow


cannot get fields while inside the form.onPart()

I have the following front-end
               var form = document.getElementById('uploadfile');                 var formData = new FormData(form);                 formData.append('path',filepath);                 uploadFile(formData);
and i am running the following back-end:
`form.parse(req);
            form.onPart = function(part) {

                if (!part.filename) {
                    // let formidable handle all non-file parts
                    form.handlePart(part);
                    return;
                }

                // once pause is implemented, the part will be able to throttle the speed
                // of the incoming request
                part.pause = function() {
                  form.pause();
                };

                // resume is the counterpart to pause, and will fire after the `put` emits
                // "drain", letting us know that it's ok to start emitting "data" again
                part.resume = function() {
                  form.resume();
                };

                console.log(req.body);
                var append = filesystemClient.fileSystem.append(accountName, pathToEnumerate, part, function(err, result, request, response) {
                    if (err) {
                        console.log("error");
                    }else {
                        console.log("success");
                    }
                })
                append.on('progress', function(variable){
                    console.log(variable);
                });
            };

`
I want to achieve to upload a file to the azure data lake and that is working great with this set-up,
However i want to get the PATH parameter that i appended to the form in the front-end so i know where the user want's to store this file on ADLS and i can give it with the function.
Can you please help me out on how to do this because i can't seem to get it.
It looks like he first does the file and uploads it and once that is done i can get the path parameter but this needs to go the other way around.
Thanks!

Solved it, closing topic

@Liamdelee , how do you solved it?


chore: conditionally choose default component for AWSTimestamp

Issue #, if available:
Description of changes:
We don't want a drift between customer's experience in Studio and CLI. We will roll in the transition to NumberField with relationship support, riding on that feature flag.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #909 (b6a0ffe) into main (820094c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main     #909   +/-   ##
=======================================
  Coverage   93.00%   93.01%
=======================================
  Files         118      118
  Lines        5091     5093    +2
  Branches     1500     1501    +1
=======================================
+ Hits         4735     4737    +2
  Misses        335      335
  Partials       21       21

Impacted Files
Coverage Δ

...te-form-definition/helpers/model-fields-configs.ts
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 820094c...b6a0ffe. Read the comment docs.


x/mobile: Gomobile does not support ARMv5 builds for Android

When building an apk with gomobile build, the resulting apk only contains libraries for the following architectures: arm64-v8a, armeabi-v7a, x86, x86_64. I also tried the following commands:
gomobile build -o basic.apk -target=android/arm golang.org/x/mobile/example/basic
GOARCH=arm GOARM=5 gomobile build -target android -o basic.apk golang.org/x/mobile/example/basic
gomobile build -o o.apk -target=android/armeabi golang.org/x/mobile/example/basic

None of them resulted in an armeabi library, always armeabi-v7a. The same problem exists with the gomobile gradle plugin.
What version of Go are you using (go version)?
go version go1.9 linux/amd64
gomobile version +e095d65 Sun Sep 10 23:20:06 2017
gradle plugin: "gradle.plugin.org.golang.mobile.bind2:gobindPlugin:0.2.5"
Original forum thread

As far as I know, the minimum Android version gomobile supports is 4.0 (API 15), which requires ARMv7 already.

Is there a source for this? I found this stackoverflow answer, but it's not clear if the ARMv7 requirement starts at Android 4.0 or 4.4.
Besides, these are the official specs, as mentioned in the answer, it would be possible to modify newer Android versions to run on ARMv5 (eg custom roms or hardware released without Google Play).

Gomobile requires ARMv7. It may be possible that there exist android devices that are not ARMv7 compatible, but gomobile does not support them.


Move default values to config.mk

What this PR does / why we need it:
(this PR complements #337 )
move the definition of some default values (directory paths) to config.mk
Which issue(s) this PR fixes:
these default value are set in several places, this is a first pass to centralize these configuration parameters

I also think moving this little commits to PR #337 would it make easier to review.

@bvavala What is the strategy here? Should we block this PR until #337 is merged? In that case redundant commits should be cleaned out.

correct

commits ported to #337


Crashing on opening the Inventory Screen

https://github.com/mezz/JustEnoughItems/issues/759
We have NEI <IP_ADDRESS> and JEI <IP_ADDRESS>5
CCC <IP_ADDRESS>
Code Chicken Lib <IP_ADDRESS>

crash-2017-03-06_22.47.06-client.txt

https://github.com/TheCBProject/NotEnoughItems/commit/25648caa0c11752693ed5840b2ac5e1ced7379ef


What are people using for crash reporting?

Hi,
Before WinUI 3 I was using AppCenter to gather crash reports and get some basic analytic insights into the usage of my app.
AppCenter does not work with WinUI 3.
So, any recommendations? What are people using for this?
(My app is sideloaded, so the store infrastructure is not a replacement)

Transferring this to a discussion (excellent question/discussion!)


Insufficient privileges to complete the operation

We are trying to run an assessment
microsoft365-assessment.exe start --mode AddInsACS --authmode application ...
with application API permissions set according the required ones documented here: https://pnp.github.io/pnpassessment/addinsacs/requirements.html
Here application permissions in Azure AD

Yet getting Insufficient priviliges errors in job logfile.
2024-03-28 12:10:03.082 +00:00 [ERR] Code in PreScanningAsync failed. This can happen when the first enumerated site collection is locked
HttpResponseCode: 403
Code: Authorization_RequestDenied
Message: Insufficient privileges to complete the operation.

Missing permissions spotted


Add 'download-atom-shell-chromedriver' task

Looks good, mind mentioning it in the README.md?

Good idea. I also removed a chunk of the README where it showed the same code snippet in both coffee-script and plain javascript; seemed unnecessary in this context. Seem ok?

seemed unnecessary in this context. Seem ok?

I think having both is nice just so people can easily copy/pasta for either Gruntfile language and get going.

:ok_hand:


[XCM delivery fees] Add missing SetFeesMode instruction in xtokens

Motivation
At Moonbeam, we are on process of enabling XCM delivery fees in our runtimes. While testing this functionality through xtokens pallet, some of our tests were failing.
The reason for this is that if XCM delivery fees are enabled on the origin chain (the chain that in this case is building the message though xtokens) and the SetFeesMode instruction is not appended to the final message, the local XCM execution prior to send the message will fail, as the XCM executor will try to deduct the delivery fees from the holding register (usually the origin chain's native asset) and they won't be there.
More context on: https://github.com/paritytech/polkadot-sdk/blob/master/polkadot/xcm/xcm-executor/src/lib.rs#L459-L467
Similar issue on polkadot-sdk
There was also a similar problem on polkadot-sdk. In that case, SetFeesMode was missing in some part of the code that transfer_assets function of pallet-xcm executes.
Related PR: https://github.com/paritytech/polkadot-sdk/pull/3792

SetFeesMode was a mistake and it will be removed in XCM v5 so I don't really want to touch it. However if it is a blocking issue for you, I am fine with this change.
Can you add some tests?

SetFeesMode was a mistake and it will be removed in XCM v5 so I don't really want to touch it. However if it is a blocking issue for you, I am fine with this change.
Can you add some tests?

Thanks! Yes I could add some tests. Although, I don't see a good way to test this, as if we would like to test the addition of SetFeesMode specifically, we would need to enable delivery fees in the mocked runtime(s), and I think that's out of the scope for this PR as that change in mocks would break other existing tests.
The addition of SetFeesMode doesn't break any existing compatibility, as if XCM delivery fees are not enabled in the runtime, the behavior is the same as always, so I would say the best way to test SetFeesMode is by ensuring that none of the existing tests and scenarios are failing.
What do you think? Do you have any suggestions? Thanks in advance :)

I see. Ideally, we will want another mock runtime to test the code with different config. But I can merge this as it is fully backward compatible (i.e. all existing tests passes).


Upgrade DownloadGitHubNpmPackageV1 to NodeJS16

Task name: DownloadGitHubNpmPackageV1
Description: Task was upgraded to Node 16
Documentation changes required: N
Added unit tests: N
Attached related issue: https://github.com/microsoft/build-task-team/issues/3718
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

Task was tested on macos-latest, windows-latest and ubuntu-latest images with node10 and node16 handlers

/azp run


Syntax error in BluEye.h

I was trying to do a very basic setup with a c++ Actor class, here is the code:
http://pastebin.com/yQq76fbv
And when I built the project, Visual Studio tells me that there are 27 errors inside BluEye.h:
http://pastebin.com/qh2f93V3
I installed BLUI just as you told in the wiki, but as I'm very new to UE4 it would be very likely that the mistake is on my side.
Ty in advance

Hello!
place this additional include above BluEye.h
#include "RenderHandler.h"

Thank you, it works fine now!
Is there a way for me to know about missing headers errors? I'd feel bad if I had to pull a github issue every time I encounter an issue like this one!

@LoveMetal-StrangeGames
Sorry for the really late reply, started working again!
I'd hit up the forum on the Unreal Engine official forums site. Under released products.


Updated rnn / lstm code

Thank you for the PR!  Minor comment: perhaps consider writing commit messages with some insight ("updated code" is a bit generic ;-).


Connecting To Remote Jupyter Server Fails

When I connect to a remote Jupyter server, I keep getting the message forever:

The console logs don't show anything, just shows that it is started to try to connect and then there are no new messages.
Note: The server I forwarded to is "localhost:8888" because I used SSH tunneling, and the remote server is setup to not require a token

We need the jupyter server logs.

Unable to reproduce this in the latest insiders build/reuglar build now


 Could you please update?

@przemos -- Przemek, thank you for your feedback; we appreciate it. To make sure we have all the information needed to triage your issue, please let us know which topic you’re referencing, or provide more context around the general task.


Wrong colors on a few Rider specific keys

Rider does have a few more code highlight elements than the other Jetbrains IDEs, unfortunately some are incorrectly colored. I've referenced them bellow:

Hi @Tom1206 :wave:, thanks again for your contribution :+1:
Yeah, would be nice if there would be some kind of a list of all the theme keys used across all IDEs, but it's more like a try'n error to get support for all of these.
I've added it as a new feature to the backlog.


Add Tree::get_name_bytes to handle non-UTF-8 entry names

Could get_name be reimplemented in terms of this method?

@alexcrichton Good idea, will do.


The Frontend (FE) component of StarRocks in a Kubernetes environment fails to start properly. The container is in a ContainerNotReady state, and the logs indicate an error due to an existing FE instance with the same host name.

Describe the bug
The Frontend (FE) component of StarRocks in a Kubernetes environment fails to start properly. The container is in a ContainerNotReady state, and the logs indicate an error due to an existing FE instance with the same host name.
To Reproduce
Steps to reproduce the behavior:

Deploy the StarRocks FE component in a Kubernetes cluster.
Ensure that the deployment includes readiness, liveness, and startup probes set to:

Initial delay: 0 seconds
Timeout: 1 second
HTTP request: GET /api/health on port 8030

Observe the container status and logs for any errors.

Expected behavior
The FE component should start successfully, pass the health probes, and join the StarRocks cluster without any hostname conflicts, becoming ready for operation.

[Sat Jul 20 08:23:42 UTC 2024] first start fe with meta not exist.
[Sat Jul 20 08:23:42 UTC 2024] FE service is alive, check if has leader ...
[Sat Jul 20 08:23:42 UTC 2024] Find leader: starrocks-cluster-fe-0.starrocks-cluster-fe-search.starrocks.svc.cluster.local!
[Sat Jul 20 08:23:42 UTC 2024] Add myself(starrocks-cluster-fe-1.starrocks-cluster-fe-search.starrocks.svc.cluster.local:9010) to leader as follower ...
ERROR 1064 (HY000) at line 1: Unexpected exception: FE with the same host: starrocks-cluster-fe-1.starrocks-cluster-fe-search.starrocks.svc.cluster.local already exists
[Sat Jul 20 08:23:42 UTC 2024] first start with no meta run start_fe.sh with additional options: ' --host_type FQDN --helper starrocks-cluster-fe-0.starrocks-cluster-fe-search.starrocks.svc.cluster.local:9010'

Root Cause
The error message indicates that there is already an FE instance with the same hostname (starrocks-cluster-fe-1.starrocks-cluster-fe-search.starrocks.svc.cluster.local) in the cluster, which prevents the new instance from joining.
Resolution Steps

Verify the existing FE instances in the cluster to identify and resolve any duplicates.
Ensure that each FE instance has a unique hostname.
Adjust the deployment configuration if necessary to prevent hostname conflicts.
Restart the FE instance and monitor its status to ensure it joins the cluster successfully.

apiVersion: starrocks.com/v1
kind: StarRocksCluster
metadata:
name: starrocks-cluster
namespace: starrocks
spec:
starRocksFeSpec:
image: starrocks/fe-ubuntu:latest
replicas: 3
starRocksBeSpec:
image: starrocks/be-ubuntu:latest
replicas: 3

I also encountered this problem. I deployed multiple environments in the same way. Two environments were normal, but one environment outputted the error: ERROR 1064 (HY000) at line 1: Unexpected excption: FE with the same host, which made it impossible to add a new FE instance to the cluster.

I guess it is the influence of k8s version. can check the root cause and see how to avoid this problem.

After FE Pod startup, it will always attempt to join the StarRocks Cluster, so it is normal for this error to occur after the Pod is rebuilt. This error is not the root cause of FE Pod restart.
ERROR 1064 (HY000) at line 1: Unexpected exception: FE with the same host: starrocks-cluster-fe-1.starrocks-cluster-fe-search.starrocks.svc.cluster.local already exists

See: https://github.com/StarRocks/starrocks/blob/ad51b419363f0ed97987e68630765188ea1cee6a/docker/dockerfiles/fe/fe_entrypoint.sh#L221-L226
From the code context, It seems you did not use PVC to persist the FE meta data. Please see https://github.com/StarRocks/starrocks-kubernetes-operator/blob/main/doc/mount_persistent_volume_howto.md for more information.

Thanks to yandongxiao for the tip，I found the real error message from the fe.log file in the pvc directory:
Caused by: com.sleepycat.je.EnvironmentFailureException: Environment invalid because of previous exception: (JE 18.3.16) kube-starrocks-fe-1.kube-starrocks-fe.search.starrocks.svc.cluster.local_9010_1725331459701(-1):/opt/starrocks/fe/meta/bdb Clock delta: 339929 ms. between Feeder: kube-starrocks-fe-0.kube-starrocks-fe-search.starrocks.svc.cluster.local_9010_1725331350969 and this Replica exceeds max permissible delta: 5000 ms. HANDSHAKE_EEROR: Error during the handshake between two nodes. Somse validity or compatibility check failed,preventing further communication between the nodes. Environment is invalid and must be closed. Originally thrown by HA thread: RepNode kube-starrocks-fe-1.kube-starrocks-fe-search.starrocks.svc.cluster.local_9010_1725331459701(-1)
I encountered this problem because the time between nodes was not synchronized.


Add nfalco to xunit plugin

Description
Add @nfalco to https://github.com/jenkinsci/xunit-plugin - picking it up for (co-)adoption as described in google group request approved by the actual mantainer @abayer
Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] All newly added users have logged in to Artifactory at least once

Could anyone give me access to github repository (github username is nfalco79) please?

@nfalco79 done. You should have permissions now


Tracking: support Apache Superset without plugin

See motivations at #11145.

[x] #9939 #11145
[x] #7556
[x] #11567
[x] #11573
[x] #11575
[x] #11613
[x] #11621
[x] #11624

After all of the above is done, we can connect to Apache Superset without the plugin and create datasets (simple ones I have only tested) successfully.

So now Superset can connect to RW just as PostgreSQL without installing the plugin first, we close this issue.
i.e. successfully create a chart of RW's table/MVs.

It is possible that there is something else missing, but let's create separate issue later.


adds 👍

@skaterdav85 Please review. This adds a simple emoji to the log for better readability.

thanks!


Updating VSTS Task Lib in PowerShellV2 task to eliminate environment block size error

Updated the version of VstsTaskSdk (powershell) from 0.9.0 to 0.11.0.
Updated the version of vsts-task-lib from 2.0.5 to 2.6.0.
The reason: environment variable block size issue was fixed in July 2017 in vsts-task-lib. See here: Microsoft/vsts-task-lib#228
VstsTaskSdk 0.9.0 was released in June 2017.
vsts-task-lib 2.0.5 was released in April 2017.
Updating the dependencies should fix the issue: Microsoft/vsts-tasks#7824.

 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.:x: jkanczler sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@ericsciple or @stephenmichaelf, could you review my changes?

Thanks for the contribution! Can you update the two 137 values to 140? It matches our current sprint. Thanks

Version is updated to correctly represent the current sprint.

We should take this


Dist packaged linux binaries seem to include two binaries

Running a dist compile seems to result in two binaries being created, both the name and the regular one. Please see the screenshots below:
Binaries are doubled (leading to huge tar file size):

Desktop files are also double:

Icons too:

I am sorry that I didn't catch this in my earlier testing, I was way too focused on getting my MacOS binary to work :-1:

It looks like something related to old build output...
Could you please remove the fyne-cross folder and try again ?
If the error still persist, could you please paste the command you are running?

You are correct. Looks like something weird happened on my end...


Issue Updating and Bulding from Source

Required information

Distribution: Ubuntu
Distribution version: 16.04
The output of "lxc info" or if that fails:

Kernel version: 4.4.0-148-lowlatency
LXC version: 3.1.0
LXD version: 3.13
Storage backend in use: zfs

Issue description
Building from source it shows this information
➜  lxd git:(master) make
go get -t -v -d ./...
CC=cc go install -v -tags "libsqlite3"  ./...
github.com/CanonicalLtd/raft-test/internal/network
# github.com/CanonicalLtd/raft-test/internal/network
/media/willismar/Dados/go/src/github.com/CanonicalLtd/raft-test/internal/network/network.go:53:2: cannot use transport (type *eventTransport) as type raft.Transport in return argument:
	*eventTransport does not implement raft.Transport (missing TimeoutNow method)
Makefile:18: recipe for target 'default' failed
make: *** [default] Error 2

I tried to reach the URL bellow but it states as 'Not Found ':
https://github.com/CanonicalLtd/raft-test/internal/network/network.go
But it can be found at:
https://github.com/CanonicalLtd/raft-test/blob/master/internal/network/network.go
Steps to reproduce

make update
make deps
make

Information to attach

[ ] Any relevant kernel output (dmesg)
[ ] Container log (lxc info NAME --show-log)
[ ] Container configuration (lxc config show NAME --expanded)
[ ] Main daemon log (at /var/log/lxd/lxd.log or /var/snap/lxd/common/lxd/logs/lxd.log)
[ ] Output of the client with --debug
[ ] Output of the daemon with --debug (alternatively output of lxc monitor while reproducing the issue)

Run git pull in your go/src/github.com/CanonicalLtd/raft-test directory and that will pull in the fix needed for this.
It's something we fixed a couple weeks back when raft upstream changed their API.

@stgraber I'm afraid they may have broken it again, I'm seeing the same error on a freshly built system
ubuntu@lantea:~/go/src/github.com/CanonicalLtd/raft-test$ go get github.com/CanonicalLtd/raft-test
# github.com/CanonicalLtd/raft-test/internal/network
internal/network/network.go:53:2: cannot use transport (type *eventTransport) as type raft.Transport in return argument:
	*eventTransport does not implement raft.Transport (missing TimeoutNow method)

Also https://github.com/hashicorp/raft/commit/eba83432bf24fbc71177da3718314fe166633ad0#diff-7a3d2ecae642058c45b6a1f699ee053dR175

@cyberwillis until we fix the raft-test library to comply with the change in interface that the raft folks have made, you can build LXD by reverting the raft library to the previous version:
cd $GOAPTH/src/github.com/hashicorp/raft
git checkout 2c551690b5c0eb05ef7f4ad72ed01f7f6ce3fcb6

Then resume the original build instructions:
cd $GOPATH/src/github.com/lxc/lxd
make

Fixed here https://github.com/CanonicalLtd/raft-test/commit/ecdf110973a40c4c94fa1423b6fb6e37df90f2c5.


4.2 frontend records for single species

This is ready to merge. See commit messages for full lowdown of what's in this PR.

@charlesroper do you think we need/can do anything about the table on phone-size screens?

Yes, need to hide the non essential columns same as the search page. But that can be in the next PR.

Returns from where? Another page?

Yes, whatever page they link to from this one.

@jdrever got it. Could use cookies, but probably easier to use localStorage or sessionStorage (see here).
Current thinking:

Don't need to store if Map tab is selected because it is default
Store state when Data is selected
Remove state when Map is selected
On page load check tab state and trigger if it exists

Yep, but don't worry too much about it at this stage, don't think we need to deliver it by end of the month, I think it's a refinement.

From: Charles Roper @.>
Sent: Tuesday, April 6, 2021 6:27:54 PM
To: jdrever/ShropBotSoc @.>
Cc: James Drever @.>; Mention @.>
Subject: Re: [jdrever/ShropBotSoc] 4.2 frontend records for single species (#54)
@jdreverhttps://github.com/jdrever got it. Could use cookies, but probably easier to use localStorage or sessionStorage (see herehttps://vanillajstoolkit.com/reference/#Browser-Storage).
Current thinking:

Don't need to store if Map tab is selected because it is default
Store state when Data is selected
Remove state when Map is selected
On page load check tab state and trigger if it exists

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/jdrever/ShropBotSoc/pull/54#issuecomment-814298254, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAPKYJM5CRIJWDMLZCDGLUDTHNABVANCNFSM42NCNOKA.


Cannot upgrade provider from v0.27 to v0.29

What happened?
When changing the package version fron v0.27 to v0.29, I have the following issue:
  Warning  SyncPackage        5m8s (x2 over 7m14s)  packages/providerrevision.pkg.crossplane.io  cannot establish control of object: domains.cloudsearch.aws.upbound.io is already controlled by ProviderRevision provider-aws-b52ad4c74cbf (UID 85da3c9a-851a-49fc-9804-09a73f8272d9)
  Warning  SyncPackage        2m9s                  packages/providerrevision.pkg.crossplane.io  cannot establish control of object: domainmailfroms.ses.aws.upbound.io is already controlled by ProviderRevision provider-aws-b52ad4c74cbf (UID 85da3c9a-851a-49fc-9804-09a73f8272d9)

How can we reproduce it?

Install a provider v0.27.0 using ArgoCD:

apiVersion: pkg.crossplane.io/v1
kind: Provider
metadata:
  name: provider-aws
  annotations:
    argocd.argoproj.io/sync-wave: "30"
    argocd.argoproj.io/sync-options: SkipDryRunOnMissingResource=true
spec:
  package: xpkg.upbound.io/upbound/provider-aws:v0.27.0
  controllerConfigRef:
    name: irsa-controllerconfig

Update the package to v0.29.0

What environment did it happen in?

Universal Crossplane Version: v1.11.0
Provider Version: v0.29.0
Kubernetes version: v1.23
Kubernetes distribution: EKS

this issue is realted to crossplane-core
check if CPU and Memory is enough from crossplane pod, remove the old ownerReference from the CRDs by hand and wait die next reconcile loop

thanks @haarchri , changing the CPU on crossplane deployment solved the issue


chore: update test annotations, spec badge

Update the test annotations with new spec verbiage, and update the spec compliance badge. The functional changes have already been merged/released.

Codecov Report

Merging #265 (b83ff62) into main (229cd61) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##               main     #265   +/-   ##
=========================================
  Coverage     91.60%   91.60%
  Complexity      187      187
=========================================
  Files            20       20
  Lines           417      417
  Branches         22       22
=========================================
  Hits            382      382
  Misses           23       23
  Partials         12       12

Flag
Coverage Δ

unittests
91.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Feature/cli

Command line interface added to the api.
Why this?
Everyone is not programmer in this world thus this helps other users to easiy use ncellapp features via cli.
PS
I am a linux user so i have not tested this in windows environment please test it once.
Usage

Options:
  -pn, --phonenumber INTEGER  Pass your phone number.
  -t, --token TEXT            Pass token you got from register.
  --help                      Show this message and exit.

Commands:
  getbalance       Get your account balance
  recharge         Recharge your account
  rechargehistory  Get your recharge history
  rechargeothers   Recharge others
  register         Gives you token to use this app features.
  sendfreesms      Send a free sms
  sendpaidsms      Send a paid sms
  transferbalance  Transfer balance to others
  view-profile     List your profile data

Examples :
ncellapp -pn [yournumber] register
ncellapp -t [token from register] getbalance
ncellapp -t [token from register] view-profile

Hi @wisdomrider,
Thank you so much for your interest in this project and for the time you spent on this PR. But, this wrapper is intended for developers only and a new version of the NcellApp wrapper with a different structure is going to be released soon which will break the current 'NcellApp CLI' you created.
Our team checked the CLI you created and it needs some improvements. Please, add me on discord, and let's discuss it there with the team.

It's okay.I worked on this PR because that day i was free.I am sorry but i cannot contribute continuously so you can edit this branch and use it in future for cli (If you need it).It's always good to see nepalese people working on these kind of projects.I had also made a esewa's unofficial api before 6 months but i am not good at writing documentation so its not public :smile:


Document creating multiple records

fixes #57

Coverage remained the same at 100.0% when pulling 990dcdea70e415db20973c96eec66280d599ec43 on samrocketman:doc-create-multiple into 0d85cbcdcfa4825f2556f33ca00a6df53c5b1dc8 on rbw0:master.

Correction, after some testing I found that it only created a single record.


feat: Optimize unit tests

Closes #
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

Codecov Report
Patch coverage is 100.00% of modified lines.

Files Changed
Coverage

view.go
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


UHF-3810 TODOs

UHF-3810

What was done

Went through all the TODOs in the HDBT-theme and created tickets of the ones that didn't have.
Removed some old code that was found when checking the TODOs.

How to install

Make sure your instance is up and running on latest dev branch.

git pull origin dev
make fresh

Update the HDBT theme

composer require drupal/hdbt:dev-UHF-3810_todos

Run make drush-cr

How to test

[x] Check the TODOs make sense and are written in similar format to help when searching them.
[x] Check that the service page with service units listed on the bottom works fine if you add some paragraphs to the lower content.
[x] Check that the unit page with accessibility information accordion on the bottom works fine if you add some paragraphs to the lower content.
[x] Check that if you add <nolink> link to main menu first level it doesn't break to layout like it did before.
[x] Check that the map paragraph and external video paragraph and image-placeholder still work correctly after the aspect-ratio code change. You can for example add them to a standard page and scale the page to make sure they still scale nicely.
[x] Check that code follows our standards

Designers review

[x] This PR does not need designers review
[ ] This PR has been visually reviewed by a designer (Name of the designer)

Approved, but github mobile has hidden the approve action too well 🙈


Update ballerina version

Purpose
Update ballerina version

Codecov Report

Merging #15204 into master will decrease coverage by 0.12%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #15204      +/-   ##
============================================
- Coverage     23.25%   23.12%   -0.13%
  Complexity     4996     4996
============================================
  Files          1304     1347      +43
  Lines         66017    67302    +1285
  Branches       8921     9133     +212
============================================
+ Hits          15350    15562     +212
- Misses        48979    50039    +1060
- Partials       1688     1701      +13

Impacted Files
Coverage Δ
Complexity Δ

tool-plugins/vscode/src/utils/logger.ts
52.63% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/test-runner/runner.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/debugger/index.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/debugger/manager.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/test-runner/activator.ts
0% <0%> (ø)
0% <0%> (?)

...lugins/vscode/src/core/extended-language-client.ts
12.5% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/debugger/session.ts
0% <0%> (ø)
0% <0%> (?)

composer/packages/tracing/src/index.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/extension.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/bbe/renderer.ts
0% <0%> (ø)
0% <0%> (?)

... and 33 more

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 44b8713...2367848. Read the comment docs.


Ctrl + A does not work on WASM

Cannot reproduce on linux x11 chrome, what's your environment?

ctrl + a works inconsistently on mac chrome. Try to type something, then remove one char, then ctrl + a.

Works fine on my end with deleting and reselecting. Unfortunately I'm fully mac-less so I'll unassign for now

Sorry I'm on Linux Mint 21.1 chrome

Hi, with the bevy and related dependency updates this may have been fixed, could anyone retest on main please?

https://github.com/StaffEngineer/bevy_cosmic_edit/assets/111751109/2e2abffd-a308-4bf5-9012-af328f314539
(after each word I press "ctrl + a" but it's still flaky)

strange! i still can't reproduce, are you still on osx? I can't see any pattern to when it does work and when it doesn't in that video.. Could be in the command cross platform compatibility stuff, system ordering, or something further up the input pipeline, perhaps the canvas is struggling to get exclusive KB access?

perhaps the canvas is struggling to get exclusive KB access

yep, have the same feeling


multi_gpu_model

First of all, I would like thank you about this great work.
I notice that the "multi_gpu_model" does not exploit the maximum of GPU computing available why?

best regards.

Probably PCIe bandwidth. Could you please share your code that is resulting this workload? Also the header of this report since I can't reproduce it. If you change between GPUs when you have a large number of parameters (like after a main dense layer) than data transfer to other GPUs can become a bottleneck. You have 4 GPUs but you can be way much slower than a single GPU with a bad network design.

I use examples of models keras like VGG16, InceptionV3 ... I also notice that for the first and second time, the execution took 4 hours for each but for the third time 14 hours!

I use examples of keras models like VGG16, InceptionV3 ... I notice also that for the first and second epochs, the execution time took 4 hours for each one but for the third one it will take 14 hours!

When using multi_gpu_model batch is actually split by number of GPUs, therefore for fair perf comparison batch size should be 4 times bigger. I had similar issue and increasing batch size have helped.


workflow update to mail error log w/signoff

Notes for Reviewers
Update the workflow to send error logs whenever an error occurs while running mesheryctl registry generate. Additionally, create a new file to store only the errors.
Local run of the workflow.
Signed commits

[ ] Yes, I signed my commits.

Does this new email replace the existing email for this workflow or will it be a duplicate?

This will replace the component generator failure email in these scenarios:

If the component generator job fails, the old failure email will be used.

If the component generator is successful but the error log is not empty, a new email regarding the error logs will be sent, and the workflow will be marked as successful.

For the scenario if the generator works and updater fails.

We will have two emails: one for the failure of the updater and one for the error logs.


Offline buildpacks

Deployment type

[ ] Cloud Foundry Application (cf push)
[ ] Kubernetes, using a helm chart
[ ] Docker, using docker compose
[ ] Docker, single container deploying all components
[x ] Other (please provide below)

Is there a support for Packaging Dependencies for Offline Buildpacks? If not, are you planning to do it in the future ? Any information you can provide me would be greatly appreciated, thank you.
Expected behaviour
Actual behaviour
Steps to reproduce the behavior
Log output covering before error and any error statements
Insert log here

We currently have no support for this. It is something we'd be interested in but it's not on our current road map. If this is something that you work on your side let us know and we can see about getting those changes to master. Thanks!

@richard-cox
One of our colleague (@ArthurHlt ) worked on an offline packager to support offline CF deployments
https://github.com/orange-cloudfoundry/stratos-ui-cf-packager
Please let us know if this could be useful.

thanks @gberche-orange to push me to talk about this, I should talk about it.
I did the work https://github.com/orange-cloudfoundry/stratos-ui-cf-packager to package stratos-ui for cloud foundry without internet connection and without stratos-buildpack. It packages stratos-ui through travis and build binaries inside it.
Maybe it could be on the main repo because stratos-ui only need nodejs and gulp for building and package assets and one of the good part of the project use golang as backend. On golang, this project use the plugin experimental concept from go which only works on linux (neither mac and windows and like we can see here this is not a priority).
I have no special bad feelings about it on stratos because how it is used make things really better and highly pluggable.
So, as it is tied to linux I don't think there is a high value to be able to build stratos when staging on Cloud Foundry. I'm maybe wrong but the only thing I can see is security issues from golang which can be handled by bumping the version when packaging in a CI. I can do a pull request if there is an interest to package in this way
In https://github.com/orange-cloudfoundry/stratos-ui-cf-packager there is another component inside which is a plugin, it was made to be able to register multiple endpoints inside stratos-ui by only use user provided service. It uses gautocloud which give the ability to do the same for kubernetes or cloud foundry. This project can also maybe leverage some issues on stratos-ui to be cloud agnostic (I'm thinking about populate databases, pre-configuration and so on).
BTW, You have made a great work, stratos-ui is already awesome and when I see that it's still in beta, i can't imagine how it will be when it be the first release. Thank you to fill this huge gap in cloud foundry open source.

Sorry @gberche-orange and @ArthurHlt , I managed to miss these two comments. We'll take a look and get back to you. I know @nwmac has been looking in to building the back end on different OSs and currently has an open PR https://github.com/SUSE/stratos-ui/pull/1366 which might help you out.

just update that we are able to build the binary using https://github.com/cloudfoundry-incubator/stratos/tree/2.0.1


⚠ Adacta-fintech.com has degraded performance

In d2b07c1, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 1109 ms

Resolved: Adacta-fintech.com performance has improved in b4098f6.


Add scopt to the community build

@olafurpg Any idea what is the issue here. Travis passed here dotty-staging/scopt#1

I was able to reproduce the problem locally. It seems the scopt build is missing some metadata for sbt-git because it's cloned as a submodule, I suspect a bug in sbt-git which is pulled in via sbt-ghpages. The following diff to the build fixes the problem for me locally
diff --git a/build.sbt b/build.sbt
index 4a306af..f636bf3 100644
--- a/build.sbt
+++ b/build.sbt
@@ -29,8 +29,8 @@ lazy val scopt = (crossProject(JSPlatform, JVMPlatform, NativePlatform) in file(
     // to push, ghpages-push-site
     site.settings,
     site.includeScaladoc(s"$v/api"),
-    ghpages.settings,
-    git.remoteRepo := "git@github.com:scopt/scopt.git",
+    // ghpages.settings,
+    // git.remoteRepo := "git@github.com:scopt/scopt.git",
     description := """a command line options parsing library""",
     libraryDependencies += specs2 % Test,
     scalacOptions ++= Seq("-language:existentials"),
diff --git a/project/plugins.sbt b/project/plugins.sbt
index 59565a2..0e56294 100644
--- a/project/plugins.sbt
+++ b/project/plugins.sbt
@@ -2,7 +2,7 @@ resolvers += "jgit-repo" at "http://download.eclipse.org/jgit/maven"

 addSbtPlugin("com.typesafe.sbt" % "sbt-site" % "0.7.1")

-addSbtPlugin("com.typesafe.sbt" % "sbt-ghpages" % "0.5.2")
+// addSbtPlugin("com.typesafe.sbt" % "sbt-ghpages" % "0.5.2")

 addSbtPlugin("ch.epfl.lamp" % "sbt-dotty" % "0.1.5")

Thanks @olafurpg, scopt now passes the community build


Remove "@" from popup placeholder

Went to use the extension, and was stumped trying to run the tool.
Looks like my username without a leading @ worked, and I was led astray on how to enter my username due to this placeholder.
Either that, or you support with @ and without, and my app password just took a minute to become valid?

Thank you for your contribution! Originally, I was sanitizing with the code below, but in the current released version, we mistakenly excluded this process..🥲
https://github.com/kawamataryo/sky-follower-bridge/pull/76/files#diff-471f82d59d92b828b59743ed8e57b61bd6b438d1216771d5a5557fd950d62dc7L130-L132
I think your changes are a good idea. LGTM！


Fix infer_schema_length: nil option

Currently, setting infer_schema_length: nil when creating a DataFrame from CSV has no effect, because there is a redundant fallback to the default value of 1000 rows if the max_rows option is not set (or set to nil).
Effectively, it is not possible to use all rows for schema inference without setting infer_schema_length (or max_rows) to a large enough "guess". I believe this does not match the documented behavior:

:max_rows - Maximum number of lines to read. (default: nil)
:infer_schema_length Maximum number of rows read for schema inference. Setting this to nil will do a full table scan and will be slow (default: 1000).

This PR removes the fallback to the default.
I'm not sure if the changes here are actually preferable to changing the docs, as scanning all rows will be slow for larger files (as stated), but ran into an issue myself with the following situation:

the files' line count is not known upfront (guessing a large enough infer_schema_length / max_rows value felt wrong)
first Nk lines contain 0, later lines contain float values. (an issue with the source of the files, but fixing that is not possible)

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


mosquitto python not currently exist

The Mosquitto Python module does not currently exist, can change the script to paho mqtt client.
https://eclipse.org/paho/clients/python/

This seems to still be a problem, any way possible that this could be fixed?
I have tried changing the code but im no programmer.
Got this far, but then im out of luck.
Traceback (most recent call last):
File "./mqtt-zabbix.py", line 245, in
connect()
File "./mqtt-zabbix.py", line 214, in connect
result = mqttc.connect(MQTT_HOST, MQTT_PORT, 60, True)
File "/usr/local/lib/python2.7/dist-packages/paho/mqtt/client.py", line 686, in connect
return self.reconnect()
File "/usr/local/lib/python2.7/dist-packages/paho/mqtt/client.py", line 808, in reconnect
sock = socket.create_connection((self._host, self._port), source_address=(self._bind_address, 0))
File "/usr/lib/python2.7/socket.py", line 565, in create_connection
sock.bind(source_address)
File "/usr/lib/python2.7/socket.py", line 228, in meth
return getattr(self._sock,name)(*args)
TypeError: coercing to Unicode: need string or buffer, bool found

@jamestutton kindly submitted an improvement that also includes the use of Paho. I don't use this project any more, but if anyone fancies testing it and letting me know if it works, we might be able to close this issue!

you need separately install "pip install paho-mqtt" and launch it.
*sign, al this uses Python2


Project directories now can be added dynamically

I think it makes sense to make projects parser options to load dynamically .
Otherwise we would have to add changes to the parser-utils every time module is added and that would increase chances of breaking something while doing so.
Also maybe at some point help msgs from new modules would probably be better to migrate to project.yml rather than editing text_utils for the same reasons.
I know modules are not added often, but we gotta hope for the best right?

LGTM, thanks :heart:


Negative step size causes infinite loop

This should probably be an error if it's not supported, but it also doesn't say anywhere that it isn't. If you attempt to use a negative step size it results in an infinite loop. Here's a workable example:
(0..10).step(-1).each { |i| puts i }

Ideally this should throw an exception, or ideally, actually work by stepping through the range in reverse.

(0..10).step(-1) should be a noop because you can't reach 10 from 0 with -1 steps.

Btw I would expect this to work: (10..0).step(-1).to_a # => [] (https://carc.in/#/r/86bf)
But maybe it's another issue?

Btw I would expect this to work: (10..0).step(-1).to_a # => [] (https://carc.in/#/r/86bf)
But maybe it's another issue?

I believe that negative step should raise ArgementError.
@straight-shoota @watzon what do you think?

If it's not going to be supported then I'd say it should. Ideally it would be nice to have reverse ranges supported at least.

I can take care of this. @straight-shoota can you assign this to me please?

It should raise on negative step, and zero step. Like in Ruby.

Negative steps in large-to-small range sounds perfectly valid to me ((10..0).step(-1)).

For the record Ruby does this
p (10..0).step(-1).to_a # => [10, 9, 8, 7, 6, 5, 4, 3, 2, 1, 0]
p (0..10).step(-1).to_a # => []
p (10..0).step(0).to_a  # => infinite loop
p (0..10).step(0).to_a  # => infinite loop

Ok, I will try to cover all cases.

What should we do with Float step? Currently it is not supported too:
icr(0.33.0) > (0..10).step(0.1).to_a
Showing last frame. Use --error-trace for full trace.

In /usr/local/Cellar/crystal/0.33.0/src/range.cr:435:15

 435 | @step.times { @current = @current.succ }

But it works in Ruby 2.6.5 and 2.7.0:
jupiter:~/workspace/crystal (master)$ irb
irb(main):001:0> (0..10).step(0.1).to_a
=> [0.0, 0.1, 0.2, 0.30000000000000004, 0.4, 0.5, 0.6000000000000001, 0.7000000000000001, 0.8, 0.9, 1.0, 1.1, 1.2000000000000002, 1.3, 1.4000000000000001, 1.5, 1.6, 1.7000000000000002, 1.8, 1.9000000000000001, 2.0, 2.1, 2.2, 2.3000000000000003, 2.4000000000000004, 2.5, 2.6, 2.7, 2.8000000000000003, 2.9000000000000004, 3.0, 3.1, 3.2, 3.3000000000000003, 3.4000000000000004, 3.5, 3.6, 3.7, 3.8000000000000003, 3.9000000000000004, 4.0, 4.1000000000000005, 4.2, 4.3, 4.4, 4.5, 4.6000000000000005, 4.7, 4.800000000000001, 4.9, 5.0, 5.1000000000000005, 5.2, 5.300000000000001, 5.4, 5.5, 5.6000000000000005, 5.7, 5.800000000000001, 5.9, 6.0, 6.1000000000000005, 6.2, 6.300000000000001, 6.4, 6.5, 6.6000000000000005, 6.7, 6.800000000000001, 6.9, 7.0, 7.1000000000000005, 7.2, 7.300000000000001, 7.4, 7.5, 7.6000000000000005, 7.7, 7.800000000000001, 7.9, 8.0, 8.1, 8.200000000000001, 8.3, 8.4, 8.5, 8.6, 8.700000000000001, 8.8, 8.9, 9.0, 9.1, 9.200000000000001, 9.3, 9.4, 9.5, 9.600000000000001, 9.700000000000001, 9.8, 9.9, 10.0]

I think it's fine if we make it work with numbers. These are little things that provide value, even if they are inconsistent with how ranges work in general.

I added iteration over Range with negative step: https://github.com/crystal-lang/crystal/commit/3fd4a32382e24c81df7b891a8da5ee1143fc78af.
It is still a WIP.

The following code:
p (0..10).step(-1).to_a
p (10..0).step(-1).to_a
p (10..0).step(0).to_a

Gives the following output using Crystal 1.0.0:
[]
[10, 9, 8, 7, 6, 5, 4, 3, 2, 1, 0]
Unhandled exception: Zero step size (ArgumentError)
  from /usr/share/crystal/src/number.cr:201:5 in 'step:to:by:exclusive'
  from /usr/share/crystal/src/range.cr:266:7 in 'step'
  from test4.cr:3:3 in '__crystal_main'
  from /usr/share/crystal/src/crystal/main.cr:110:5 in 'main_user_code'
  from /usr/share/crystal/src/crystal/main.cr:96:7 in 'main'
  from /usr/share/crystal/src/crystal/main.cr:119:3 in 'main'
  from __libc_start_main
  from ../sysdeps/x86_64/start.S:122:0 in '_start'
  from ???

I think this can be closed.

Yes, zero step size directly errors since #10130.


Scroll drift in Chrome when zoomed in

A fun one! Thanks to this Chromium bug, in combination with these parseInts, you can end up in a situation where ember-collection slowly scrolls upwards with no user input.
To reproduce, take a look at the scroll position demo in Chrome, zoom in to 110%, and scroll down a bit.
If you land on anything other than a nice round number, you'll end up in a feedback loop between the parseInt truncation in the addon code and the rounding glitch in the browser that inexorably drags you back to the top of the collection.

Any updates on this? Seems like the fix is to remove the parseInts..

The problem with removing the parseInts is right now I think that would break the layouts since layout-bin-packer doesn't deal w/ floating point numbers... but I guess it could. This needs more investigation.

This problem still happen in Microsoft Edge browser when page is zoomed in or zoomed out. Is there any plan to fix it?


Provide Lsp for neovim-lspconfig

What is your idea? Provide a use case.
Open a pull request in neovim-lspconfig adding a file for aiken lsp
Why is it a good idea?
To have lsp working with ease in neovim
What is the current alternative and why is it not good enough?
A plugin for neovim exists but doesn't provide lsp and users have to locally modify the neovim-lspconfig repo to make it work with neovim.
This is the file I'm using right now (inspired by pylsp file):
local util = require 'lspconfig.util'

return {
  default_config = {
    cmd = { 'aiken', 'lsp' },
    filetypes = { 'aiken' },
    root_dir = function(fname)
      local root_files = {
        'aiken.toml'
      }
      return util.root_pattern(unpack(root_files))(fname) or util.find_git_ancestor(fname)
    end,
  },
  docs = {
    description = [[
    Aiken language server for neovim, you are free to do whatever you want with it
    but I'm now liable for anything you do.
    ]],
  },
}

It has to be put under nvim-lspconfig/lua/lspconfig/server_configurations/aiken.lua to make it work.
There are other options (like formatting) that can be put there but I don't need more, so someone with more time working with aiken should complete the file.

cool, I would also like it work with mason, does that involve something else similar to lspconfig?


/register docs claim that password is required

... which is untrue if kind == guest

This got fixed in PR #402


Implement Superchain icon modal + tooltip interaction

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #107

PR #108

PR #113

PR #119   👈

This stack of pull requests is managed by Graphite.


added user profile

What type of PR is this? (check all applicable)

[x] 🚀 Added Details
[x] 🌟 stared the repo
[x] 🐛 Grammatical Error
[x] 📝 Documentation Update
[x] 🚩 Other

Description
Add Link of GitHub Profile

pls review


net: sockets: Fix TLS_HOSTNAME option length inconsistency

The TLS_HOSTNAME socket option expects a NULL terminated string and doesn't really care about the optlen provided. However, as the option expects that the string is NULL terminated, the optlen value should take NULL character into account, for consistency across the codebase.
Signed-off-by: Robert Lubos<EMAIL_ADDRESS>

Will you add this to the release notes, @rlubos ?

@mbolivar-nordic I don't think this change merits an entry in the changelog, we're not changing any API here.

Additionally out of two possibilities:

NULL terminated string, without length
string (without NULL termination), with length

we have a mix of above two.

Ok, it might be a bit of redundant to provide the string size along with a C-string (or rather option value size, look at this from the setsockopt perspective), but I don't think the alternatives are any better to be honest:

NULL terminated string, without length

Here we break the setsockopt() convention to provide the actual argument size. Plus it breaks the aforementioned userspace argument copying.

string (without NULL termination), with length

For me, considering a string w/o NULL termination is even more counterintuitive. Plus this adds additional overhead for options processing, as mbed TLS expects NULL terminated string.

that is not really intuitive for C programmers, because it requires both NULL termination and length, so passing setsockopt(..., hostname, strlen(hostname)); is something application code will quite likely do, meaning it is very error prone

I agree this may be error prone, but I don't really see a better alternative, considering we're in constraints implied by setsockopt() API format

with CONFIG_USERSPACE=y zsock_setsockopt() syscall makes sure that full string with NULL termination is copied (or verified?), but then no driver is actually checking for NULL termination, which can potentially break kernel
even without userspace, there are missing checks about this hostname being NULL terminated

Ok, it's true there's no argument validation in this case. Feel free to open an issue for that.

NULL terminated string, without length

Here we break the setsockopt() convention to provide the actual argument size. Plus it breaks the aforementioned userspace argument copying.

That is right, it is not an option for setsockopt() API. I just pointed it out as standard way to pass strings with C APIs.

string (without NULL termination), with length

For me, considering a string w/o NULL termination is even more counterintuitive.

Interesting. Which part is counterintuitive for you? Is it in general for any C API or when taking into accout setsockopt() context?
If we just ignore setsockopt() context for a while, then there are either C APIs that consume NULL terminated strings (which is like 80-90% of the cases) or there is a string passed (which does not need and most likely is not NULL terminated) with explicit length. There are many examples of the latter, like struct mqtt_utf8 (in this case length is not passed as function argument, but used in structure) or zcbor_tstr_encode_ptr(zcbor_state_t *state, const uint8_t *ptr, size_t len) in ZCBOR.
If we take setsockopt() context into account, then... I don't know, it just doesn't change my thinking about it. But I am not strong setsockopt() user myself.
I also looked at the man 7 socket about SO_BINDTODEVICE, which is the only string that I can think of (or that I have used so far), that is passed with setsockopt() API. In that case man pages say about NULL termination being part of the data. However, looking at the Linux kernel implementation, passing string without NULL termination is handled just fine.

Plus this adds additional overhead for options processing, as mbed TLS expects NULL terminated string.

That is right, unfortunately. I am not sure we should make API decisions based on external projects like mbedTLS. Of course it is good to consider those, but it should not be the only driving factor. Internally mbedTLS calls strlen() and allocates buffer for hostname on its internal heap. Maybe mbedTLS introduces an API to which we pass hostname explicitly. That way one strlen() invocation could be removed and then NULL termination would not be required anymore.
There are also modem drivers that can respect TLS_HOSTNAME option. There, we can potentially use %.*s format specifier for snprintk() when preparing AT command request, so that NULL termination is not required. In current codebase there is actually just one driver that uses TLS_HOSTNAME, which is drivers/wifi/simplelink/. If the documentation there is right (maybe there is same mistake you have fixed in this PR), then NULL termination is not required: https://github.com/zephyrproject-rtos/hal_ti/blob/5e7d5cd584047699c9fd279923120cb25ba3dda7/simplelink/source/ti/drivers/net/wifi/sl_socket.h#L1222-L1225

I don't believe we're having a debate about NULL termination...

Interesting. Which part is counterintuitive for you?

The TLS_HOSTNAME option description clearly says it expects a string. And NULL-terminated string is a standard in C. I don't really have much more to add here.

If we just ignore setsockopt() context for a while...

Please, let's not ignore setsockopt() as it's the key here. Of course there are APIs that take benefits of NULL termination, there also those which don't. But setsockopt() is neither - it's not a string API, it a generic API accepting any data type. All it expects is that the binary size of the option is provided along with the option value. And as the TLS_HOSTNAME expects a string, the binary size of the string should be provided, regardless of how redundant this is.

I also looked at the man 7 socket about SO_BINDTODEVICE, which is the only string that I can think of (or that I have used so far), that is passed with setsockopt() API. In that case man pages say about NULL termination being part of the data. However, looking at the Linux kernel implementation, passing string without NULL termination is handled just fine.

And we of course could do the same with TLS_HOSTNAME, the question is whether we want to. We could verify if the string provided has a proper NULL termination, allocate an extra buffer if it doesn't, copy the data. But we need to remember we're in embedded environment, and instead of bloating the codebase, we could simply expect the application to provide data in correct format.

That is right, unfortunately. I am not sure we should make API decisions based on external projects like mbedTLS.

That's the reality we're in. mbed TLS so far is the only crypto library we have and whole TLS socket idea is build on top of it. I'm pretty sure that if there were some controversial API decisions not fitting the general design of the TLS sockets, we would seek for alternatives/workarounds. But accepting C-string as a parameter? Common.


Link to Ultimate Edition

Hi,
could you fix the link to the ultimate edition:
URL: https://portapps.github.io/app/intellij-idea-community-portable/
Given link there (wrong): https://portapps.github.io/app/intellij-ultimate-portable
Correct link: https://portapps.github.io/app/intellij-idea-ultimate-portable/

Done thanks!


No templates included in default installation

The readme implies templates are included, and the app_loader template loader will retrieve them, but there are no templates in the repository that I can see.
Trying this out now. Will let you know if I find any other issues. :)

the issue is the MAINFEST.in file and the setup.py file need to be setup to include the non python template files, so i issued pull #6 to fix


Rationalise the GitHub and Buildkite permissions

I've rationalised the list of teams:

Buildkite reviewers is gone; we trust all devs to manage Buildkite permissions. There was an idea at one point that all Buildkite changes would go through a select group of reviewers, but in practice that was more a hindrance than helpful.
I've moved Identity to be a subteam of Developers to simplify the top-level view.

I've also reviewed all read-only permissions on repos:
Archives I’m going to leave as is.

archivematica = I’ve given all devs write access, even though most of them should be going away soon.
archivematica-docs = deleted. It was a fork we created for the OIDC work which is now merged upstream.
aws_utils = archived, we haven’t touched it in ages
identity = I’ve given all devs write access, because all JS/TS reviewers have write access and in practice that’s most of the team
prismic-remapper python-gearman tzitzit = all devs get write access
wellcomeplayer* – tbh these should probably all be public archives

This is all the low-hanging fruit, but happy to use my admin powers to make other changes people suggest – I'm not sure what else to do.

Closing as won't do.


Remove token management middleware?

I'm not sure that having a token store built into Apollo Client is useful. I'm referring to this class: https://github.com/apollostack/apollo-client/blob/e8f22bd4b085637360eccb4745951987b11f0d2f/src/middleware.ts#L12
Just a sanity check here:

In Meteor, you would get the token from Meteor directly
When using cookies, you just store it in the browser
Even when using custom tokens, you would want to cache them in local storage

Is the built in token storage thing that ships as part of the network interface useful?
Are you using it @jbaxleyiii @johnthepink?

@stubailo you're right. It's not really that useful. It made sense with the initial way we intended to do our implementation. But I'm great with removing it now.

Sounds good - should be an easy issue to pick up for anyone interested!

[PR #254](https://github.com/apollostack/apollo-client/pull/254_

I think it is safe to close since the changes have been merged.


Implemented buffer/stream constructors for CanvasImage

Implemented feature to allow CanvasImage construction from an in memory buffer/stream. Added demonstration code to Canvas example.
As discussed here

 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.

 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.

@impsnldavid Merged! Thanks for your contribution. Much appreciated!

@impsnldavid Merged! Thanks for your contribution. Much appreciated!


Upload files

Added the possibility to upload files to Gitlab, so that Issues or other places using markdown can be enriched with attachments, by using the Markdown property provided by the upload response.

Codecov Report

Merging #27 into master will decrease coverage by <.01%.
The diff coverage is 10.71%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
- Coverage   23.51%   23.51%   -0.01%
==========================================
  Files          77       80       +3
  Lines        1216     1229      +13
  Branches      108       95      -13
==========================================
+ Hits          286      289       +3
- Misses        856      881      +25
+ Partials       74       59      -15

Impacted Files
Coverage Δ

...ent/Models/Uploads/Requests/CreateUploadRequest.cs
0% <0%> (ø)

...itLabApiClient/Internal/Http/GitlabApiRequestor.cs
8.88% <0%> (-1.37%)
:arrow_down:

.../GitLabApiClient/Internal/Http/GitLabHttpFacade.cs
41.66% <0%> (-1.82%)
:arrow_down:

...GitLabApiClient/Models/Uploads/Responses/Upload.cs
0% <0%> (ø)

src/GitLabApiClient/GitLabClient.cs
76.92% <50%> (-1.46%)
:arrow_down:

src/GitLabApiClient/UploadsClient.cs
66.66% <66.66%> (ø)

...ApiClient/Internal/Queries/ProjectsQueryBuilder.cs
41.02% <0%> (+2%)
:arrow_up:

... and 7 more

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 19456a7...f809105. Read the comment docs.

Thank you! Looks nice.

 Hope to see 1.0.5 on NuGet soon :)

By the way, this could close issue https://github.com/nmklotas/GitLabApiClient/issues/20.

Thanks, yes I've created 1.0.5 ~22 hours ago

Awesome work 👍


Inventory Packet Validation

Describe the problem related to your feature request.
At the moment, we trust the client a little too much when it comes to modifying inventories. For example, a client can send a click slot packet that just adds any item to any slot out of thin air.
What solution would you like?
There are 2 types of assertions we need to make on incoming modifications:

plain invalid info that shouldn't be processed ever
making sure items aren't being duplicated/spawned/deleted, aka conservation of mass

Type 1 validations:

[x] slot ids are in-bounds
[x] item stack amounts are in-bounds
[ ] some slots are "outbound only" eg. crafting result slots decided not to do this because it's not that useful and pretty hard to assert for

Type 2 validations:

[x] while the cursor is holding an item

[x] and clicking an empty slot, the item should go into the clicked slot
[x] and clicking a filled slot,

[x] if the item is the same type, and same nbt data, an amount in the cursor item's stack be transferred to the slot's stack such that the resulting slot's stack reaches 64 (maybe make this configurable? some items can only stack to 1)
[x] otherwise, the slot and cursor item should swap

[x] and the user clicks and drags, the resulting differences in item stacks should add up to the correct amount

example: user is holding 5 diamonds, slot 1 is holding 1 diamond, slot 2 is empty. user clicks and drags the 5 item stack across slot 1 and 2. now the user should be holding 1 diamond, slot 1 should have 3 diamonds, slot 2 should have 2 diamonds

[x] while the cursor is not holding an item

[x] and clicking an empty slot, nothing should happen
[x] and clicking a filled slot

[x] and pressing shift, using left click, the slot's stack should go directly to another slot (it doesn't matter which one)
[x] using right click, half the stack should be picked up (which slot gets the remainder if the amount in the stack is an odd number)
[x] otherwise, the slot's stack should be transferred to the cursor

The type 2 validations may be expensive, so ideally we should allow the user to disable it.
There's probably some more that I missed, but that should be enough to catch most cases.
What alternative(s) have you considered?
We could also not do this, and defer this to end users, but I think that could end up being pretty error prone.
Additional context
For #288
https://wiki.vg/Protocol#Click_Container

some slots are "outbound only" eg. crafting result slots decided not to do this because it's not that useful and pretty hard to assert for

What happens if an item is moved to the outbound slot and it gets overwritten by the result of a crafting recipe? (Does the client predict crafting recipe results? Not really sure how that all works).
Being able to move items to the output of, say, a furnace seems like a potential source of bugs for future code that updates furnaces.
Clients effectively have an extra inventory slot.

Seems like something we should address at some point.


Use the path to the database instead of the db object.

Fixes #5

I'm manually apply this fix, but it's inconvenient.
C:\logstash-7.11.1\vendor\bundle\jruby\2.5.0\gems\logstash-input-sqlite-3.0.4\lib\logstash\inputs\sqlite.rb
Please accept MR.

any reason not to commit this? see: https://github.com/logstash-plugins/logstash-input-sqlite/pull/6/commits/98498dadf0ef8dd192dc790f80e4911b7dc39048

Works for me as well.


logging: do not swallow subsystem logs

commit #16861 introduced a normalization of error handling into the
daemon/cmd package.
by doing so we swallowed useful error logs.
this commit adds the error logs back and adds a few additional
fmt.Errorf wrappers where logging is not adequate
Signed-off-by: Louis DeLosSantos<EMAIL_ADDRESS>Removes any log swallowing that was occuring on daemon/cmd init

Given the potential improvements to debuggability, I'm nominating this for 1.11 backport.

/test


chore(localize): updated translation from Crowdin

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


F401 removes import required by typing annotations

F401 is a rule that removes unused imports.
The issue is that F401 removes imports that are necessary for typing annotations expressed as comments, see below.
Repro Case
from typing import Any  # <-- this import is removed by ruff
x = 1  # type: Any

Current Behavior
The import is eliminated erroneously:
x = 1  # type: Any

Desired Behavior
The code is unchanged.
from typing import Any  # <-- this import is removed by ruff
x = 1  # type: Any

I believe this is a duplicate of https://github.com/astral-sh/ruff/issues/1619 (Ruff doesn't parse comment annotations right now).


[hi] Localise Event-streaming in Hindi

Signed-off-by: Yash Pimple<EMAIL_ADDRESS>Related issue- https://github.com/cncf/glossary/issues/1511
Checklist before opening this PR (put x in the checkboxes)
This PR does not contain plagiarism
don’t copy other people’s work unless you are quoting and contributing it to them.
I have signed off on all commits
signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Thank you @YashPimple. Sorry for the long time this PR took to merge :)


First call to watch on appliedtogroups results in duplicate AppliedToGroup objects

Describe the bug
First time when 'kubectl get appliedtogroups -A -w' is executed, it dumps Appliedtogroups info twice, it should have similar bugs for which resource call the same Watch()
To Reproduce

Add appliedtogroups  object and set up watch. Appliedtogroups  will be dumped twice in the output
 kubectl get appliedtogroups -A -w
NAMESPACE   NAME                                   CREATED AT
            7b58a7bd-f1c4-5f76-8e0d-8910d9d9e407   0001-01-01T00:00:00Z
            7b58a7bd-f1c4-5f76-8e0d-8910d9d9e407   0001-01-01T00:00:00Z
                                             0001-01-01T00:00:00Z

Expected

It should be dumped only once
Additional context

I can tell the reason here:
In Antrea codebase pkg/apiserver/storage/ram/watch.go, I see in the process()  will sendWatchEvent  for what Antrea has already have, through s.storage.List()
However, kubectl itself has already done the similar thing in its pkg/cmd/get/get.go , from line 665-678
var objsToPrint []runtime.Object
    if isList {
        objsToPrint, _ = meta.ExtractList(obj)
    } else {
        objsToPrint = append(objsToPrint, obj)
    }
    for _, objToPrint := range objsToPrint {
        if o.OutputWatchEvents {
            objToPrint = &metav1.WatchEvent{Type: string(watch.Added), Object: runtime.RawExtension{Object: objToPrint}}
        }
        if err := printer.PrintObj(objToPrint, writer); err != nil {
            return fmt.Errorf("unable to output the provided object: %v", err)
        }
    }

I think Antrea should be consistency with Kubernetes

APIs under controlplane API group are not supposed to be consumed by user but internal components. They are optimized for efficiency and don't follow same behavior as K8s's native APIs. The code you pasted is not the only place that doesn't match normal K8s API behavior. Is there any reason you would expect those APIs to act like kube-apiserver?

@tnqn Hi Quan, thanks for reply, the reason is that we are using this part in Nephe and Saas, we want to watch this on kubectl.


ubuntu vcpkg::zlib depedence failure

Xmake Version
2.6.4
Operating System Version and Architecture
ubuntu
Describe Bug
ubuntu 20
xmake v2.6.4
xmake.lua
add_requires("vcpkg::zlib", "vcpkg::pcre2")
xmake -y
=> install vcpkg::zlib latest ..
failed fetch vcpkg::zlib-latest failed! if you want to get more verbose errors, please see: -> /home/wdidada/CLionProjects/rest_description_xmake/build/.packages/v/vcpkg_zlib/latest/cache/installdir.failed/logs/install.txt error: install failed!
Expected Behavior
build success
Project Configuration
No response
Additional Information and Error Logs
No response

xmake f -c -vD 贴下详细 logs

I set VCPKG_ROOT=~/vcpkg and get it


[FEATURE REQ] Improve the MS doc configuration of credential info to avoid using any secerts

Context
Delete the password configuration since Spring Cloud Azure supports the default credential. This's also reported by MS doc issue: It is using service principal which is security risk in Azure
Goal
Use a simple way to introduce the Spring Cloud Azure usage, it should cover all the Spring Cloud Azure docs in MS doc and lots of readme doc, but the reference doc should introduce client secret usage.

It will be created in Azure SDK java repo.


Option to exclude files

A functionality to ignore some folders and files, such as the hooks directory might be really useful.
I am trying to use this action to copy one whole repo to another repo, but I don't want the hooks to create an infinite loop/ fail when the PR is merged.

This would be incredibly useful - were you able to find a work around?

No, unfortunately not, I had to make my own (very janky) action.


The wrong type of certificate is mentioned in point 7

Point 7 indicates that the 'Certificate (Base64)' must be used, but the image indicates that 'PEM certificate download' must be used. If the Base64 certificate is used in Github Enterprise, you will receive an error message and if the PEM certificate is used, it works. Please adjust in the description that the PEM certificate must be used.

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

ID: 7b718882-4334-9eab-a83a-76023d0ec2b2
Version Independent ID: b8b7ca3b-8373-382d-00f1-06ca3a9c60fa
Content: Tutorial: Microsoft Entra single sign-on (SSO) integration with GitHub Enterprise Managed User - Microsoft Entra ID
Content Source: docs/identity/saas-apps/github-enterprise-managed-user-tutorial.md
Service: entra-id
Sub-service: saas-apps
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@roelrulezz
Thanks for your feedback! We will investigate and update as appropriate.

@roelrulezz
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@roelrulezz, we're creating a separate PR for your changes in our private repo, you may expect the updated doc within 2-3 working days.
#please-close


Missing OEL8/RHEL8/Fedora installation prerequisites

I was trying to build an image based on something that is not Debian.
I found that there has to be a lot of software being installed before installing the FFmpeg.
It would be good to have a doc updated with necessary bits of software that have to be installed before executing the install script.

I'll post a solution here:
RUN microdnf update --nodocs -y && \
    microdnf clean all && \
    microdnf update --nodocs -y && \
    microdnf install gcc gcc-c++ make curl git python3 rust cargo tar gzip findutils diffutils file xz perl perl-Data-Dumper -y && \
    microdnf clean all

maybe someone will be faster than I to submit the PR.


Extend lambda cdk template

In light of https://www.serverless.com/blog/serverless-framework-v4-a-new-model
We could probably upgrade this template to include an equivalent canary deployment. I believe there is also a datadog plugin for CDK too that we could use

https://github.com/DataDog/datadog-cdk-constructs/issues/232#issue-2010819696
Some questions to answer

dd trace has issues with bundlers:
https://github.com/DataDog/dd-trace-js?tab=readme-ov-file#bundling

https://github.com/DataDog/dd-trace-js/issues/2160#issuecomment-1175553622
We can use external in the meantime

aws CDK uses the cli esbuild so we can't pass in plugins
https://github.com/aws/aws-cdk/issues/18470


Add dictionary support to Avro converters

Reporter: Ryan Blue / @rdblue
Note: This issue was originally created as PARQUET-288. Please see the migration documentation for further details.

Matt Massie / @massie:
https://issues.apache.org/jira/browse/PARQUET-123
There is already dictionary support in AvroIndexedRecordReader. Which converter are referencing here?

Ryan Blue / @rdblue:
@massie, I noticed that only the String converter had dictionary support while working on PARQUET-286 and extended that to other binary readers in PR #201. This issue covers adding support in the other primitive type converters that can also benefit.