Figure out a consistent and robust way of defining and testing interfaces

We have a lot of interfaces across the Turing ecosystem. Few of them are formalised or testable. This is partly a result of Julia's multiple dispatch mechanism: it's possible to define InterfacePackage.foo() with whatever signature we like, and then overload it in ImplementationPackage with a completely different signature. Thus, when implementing a function, there's actually zero need to pay attention to what the interface demands.
The problem with this is that:

The interface code becomes useless. We pretend to maintain it but in reality there is no cost to not maintaining it.
Because the interface is not necessarily obeyed, it's impossible to figure out how the implementations work except by reading the source code.

There aren't really any amazing solutions to this in Julia. We should try to figure out what the least bad solution is.

Indeed, Will brought this up before too and I've seen it mentioned once or twice on Discourse.
I've read through the Interfaces.jl docs a couple of times now, and my assessment is that it's almost completely syntax sugar, in the sense that Interfaces.jl doesn't let you do very much that you can't already do by writing a standard test.*
As a demonstration, here's a simplified version of the example in the Interfaces.jl docs:
#################
# src/animal.jl #
#################

abstract type Animal end

function age end
function walk end

components = (
    mandatory = (
        age = (
            "all animals have a `Real` age" => x -> age(x) isa Real,
        ),
    ),
    optional = (
        walk = "this animal can walk" => x -> walk(x) isa String,
    ),
)

description = """
Defines a generic interface for animals to do the things they do best.
"""

@interface AnimalInterface Animal components description

struct Duck <: Animal
    age::Int
end

age(duck::Duck) = duck.age
walk(::Duck) = "waddle"
@implements AnimalInterface{(:walk,)} Duck [Duck(1), Duck(2)]

##################
# test/animal.jl #
##################
Interfaces.test(AnimalInterface)

This is syntax sugar for the following:
#################
# src/animal.jl #
#################

abstract type Animal end

function age end
function walk end

"""
Test that animals do the things they do best.
"""
function test_animal_interface(a::Animal, optional::Tuple{Symbol}=())
    test_age(a)
    :walk in optional && test_walk(a)
end
test_age(a::Animal) = @test age(a) isa Real
test_walk(a::Animal) = @test walk(a) isa String

struct Duck <: Animal
    age::Int
end

age(duck::Duck) = duck.age
walk(::Duck) = "waddle"

##################
# test/animal.jl #
##################
@testset for duck in [Duck(1), Duck(2)]
    test_animal_interface(duck, (:walk,))
end

I find the latter more readable because there is one less layer of indirection. You don't need to go and read up about how an additional package works.
Also, if you want to modify the behaviour of the tests, you also aren't constrained by the limitations of the package. For example, Interfaces.jl's interface gets a little bit ugly when you want to use test functions which take multiple arguments. You have to pass in an Arguments type rather than just having a normal n-ary function. This means more glue code (consider e.g. the case where we might have functions to test things in DynamicPPL, but you also want to use those testing functions in Turing).

* There is indeed one nice thing that Interfaces.jl lets you do, which is the @implements macro that automatically registers an instance of a subtype so that it can be automatically tested later. Importantly, that declaration can be put in the source code next to the definition of the type so it's easier to not forget to do it. The alternative to this is to manually 'remember' which things need to be tested, and that information would be located in the test suite, so is further away from the definition. I personally don't consider this a big enough win to use it, though.

tl;dr
Having surveyed a few corners of Julia land, I kinda believe the best way is still to just stick to basic, understandable testing functions.
Also, for any abstract type we should have a minimal test double as demonstrated in the Invenia interfaces blog post.
And finally I think abstract types, their interfaces, and their test doubles should always live in a single file with nothing else in them. Basically, don't mix interfaces and implementations in the same file. Julia lets us split files up as you like, so we should make use of that to make the code as clear as possible.


Update NodeJS to v16

We should get rid of the action warnings:

Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16<EMAIL_ADDRESS>For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.

Hi @Kurpanik
This week I'll release a new version.


apt-get error when building image: Unable to correct problems, you have held broken package

Hi,
I'm getting the following when building
$ docker build .
Step 13 : RUN apt-get update -q 2 && DEBIAN_FRONTEND=noninteractive     apt-get install --yes -q 2 puppetmaster=$PUPPET_VERSION puppet=$PUPPET_VERSION         nginx-extras passenger >/dev/null
 ---> Running in c19dc1e1d796
E: Unable to correct problems, you have held broken packages.
The command '/bin/sh -c apt-get update -q 2 && DEBIAN_FRONTEND=noninteractive     apt-get install --yes -q 2 puppetmaster=$PUPPET_VERSION puppet=$PUPPET_VERSION         nginx-extras passenger >/dev/null' returned a non-zero code: 100

I took out the PUPPET_VERSION references completely and it runs fine. I'm not real familiar with apt-get, but I assume the version currently pinned isn't the latest in the apt repo?

The reason why we "pin" the image to the latest version of Puppet is for security and reuse.
If you want the latest version, please submit a pull request with the PUPPET_VERSION updated in the build. I'd do the following to determine the latest version:

Use a VM environment like Vagrant and launch an Ubuntu 14.04 base image. Alternatively, you can do the same thing with an Ubuntu Docker image for the same version.
Install the Puppet Labs repository as per the Dockerfile in this repository.
Do an apt-get update.
Find the latest version of the Puppet Debian package and update the Dockerfile here to pin to that.

The reason you want pinning is so that downstream users like yourself can easily determine which version of Puppet is running in a given image build. If a Puppet CVE comes out, all I need to do as the maintainer is update the PUPPET_VERSION, rebake my image, tag, and upload.

The problem is that building the Dockerfile as-is did not work and failed with the error I mentioned, so it doesn't seem like pinning to a specific puppet version works as intended with apt-get. Or maybe I'm doing something wrong.  Anyway, just thought I'd mention it and what my fix was in case others not familiar with apt-get hit that error.

Thanks for your response.
If there is, in fact, a later release of Puppet, it would be ideal to rebuild the image now to incorporate it.


Adding new features, testing, code coverage.

Summary

Implementing IDisposable on resource classes.
Using using scope on IDisposable resources.
Cleaning up tests.
Adding more code coverage.
Adding POST test for /api/crops endpoint.

@bit1ngest0r This is ready for review. Be mindful this build will not pass until https://github.com/JacobHeater/covfefe-common/pull/3 is reviewed and integrated into master.


[rv_dm] Address Verilator lint warnings

Description
The nightly lint check for rv_dm contains 5 warnings.
It looks like 4 of these are width bugs in the vendored code. No big deal, but probably worth waiving in a .vlt file. There's also an UNOPTFLAT warning about tlul_lc_gate.sv, which we wrote. Address it.
See https://reports.opentitan.org/hw/ip/rv_dm/lint/verilator/latest/report.html for the latest report.

The two PRs linked in above should fix this. I don't think I know how to use PR links to say "please close this issue when both PRs have been merged", so we might have to do it manually... :-)

The remaining portion of this is fixed by #22486 (but I don't seem to be able to create the links through GitHub that would put this in the "development" box on the right)

Closing as completed by #21736 and #22486.


MATLAB API missing

@trailstrider Take a look at this article: https://support.coinapi.io/hc/en-us/articles/360026657331-How-to-use-CoinAPI-from-Matlab-

This doesn’t actually provide an api.   It’s like the JNI.   Still need to
do data marshaling.
On Fri, Aug 23, 2019 at 4:20 AM, Artur Pietrzyk<EMAIL_ADDRESS>wrote:

@trailstrider https://github.com/trailstrider Take a look at this
article:
https://support.coinapi.io/hc/en-us/articles/360026657331-How-to-use-CoinAPI-from-Matlab-
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/coinapi/coinapi-sdk/issues/11?email_source=notifications&email_token=AAFNRO37TIHLZZQSO2OASBDQF6MTNA5CNFSM4EIOSMY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD47PZXA#issuecomment-524221660,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFNRO2WNGF5FMBHQIG2YZLQF6MTNANCNFSM4EIOSMYQ
.


Wrong RDBMS_TYPE getting sent for h2 db

Description
While creating a h2 db connection in cfgai app we are getting an error, upon checking the call in network tab it was found that RDBMS_TYPE is getting sent as "HIVE" which should be "H2_DB"
Steps to reproduce
1.Open cfgai platform
2.Click on "open database"
3.Click on "Add Database"
4.Click on H2 db under connections
5. Enter details of H2 DB and click on "create database"
6. An error will appear on the screen.
Reference

Tasks
RDBMS_TYPE should be changed to H2_DB for H2 db

add in the driver as well - org.h2.Driver


Test InstaScale in a VMware-based cluster

InstaScale has integration with MachineSets. In general, this should work without any changes, we should create an OpenShift cluster on VMWare Infrastructure and perform tests.

FYI @anishasthana


Vulnerability fix for AzureNLBManagementV1

Task name: AzureNLBManagementV1
Description: Latest vulnerability fix for task AzureNLBManagementV1
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: (Y/N) https://dev.azure.com/mseng/PipelineTools/_componentGovernance/184/alert/79428?typeId=12858
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected
[x] node make.js build --task AzureNLBManagementV1
[x] node make.js test --task AzureNLBManagementV1

/azp run


BLD: Update collection and analysis versiton to 0rc1

Use 0rc1 for build env now

Work move to here
https://github.com/NSLS-II/lightsource2-recipes/pull/704


Resolve log4j (Java8) vulnerability in Simple Java Mail's CLI module

Log4j was already updated for the Java7 version in #365, but since Simple Java Mail is switching to Java8, we need to update it again to the latest.

Updated parent POMs, which update the log4j dependency to 2.17.0.

Released in 7.0.0.


[backport] Downstream clusters on an upgrade to master-head are going into updating state

Backport fix for https://github.com/rancher/rancher/issues/33749. This would ensure upgrade logic on both 2.5.10 and 2.6 is same.

Verified on v2.5-head 6b04f9b

Installed rancher server on v2.5.9
Created 3 downstream rke1 node driver clusters with 3 worker, 2cp, 3 etcd nodes
Ran pre-upgrade checks
Upgraded the rancher server to v2.5-head
Verified no clusters went into updating state and all clusters have come up active.


Dgg

Yeah you cunts are trying to set me up but have I got some major surprises
and evidence. You ain't seen nothing
On Friday, 6 May 2016, Domenic Denicola<EMAIL_ADDRESS>wrote:

Closed #1471 https://github.com/tmpvar/jsdom/pull/1471.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/tmpvar/jsdom/pull/1471#event-652111998

I've been all over your surveillance since the start and you are wrong lol.  Now I just read a communique between u guys that confirmed your intention to set me up for a prison term. I've been very passive until now but I'm stepping up to yo

u the EVIL ppl you are. Come in my room or even wrong spot of the yard and expect to be stick with needles dipped in excrement and HIV.  That's just one tactic in your assault on our privacy. I hope those involved lose children to cancer and the most horrific things life has to offer as you have made ours hell. I got so many moves you are gonna flip. Come at me bitchs


exoplayer: Exposing surface_type to use texture_view

Hi we have some problems with using the exoplayer plugin in the use case of tilting videos by rotation.
We couldn't find out a good way to change the exoplayers surface_type to texture_view which is needed for rotation to work.
At the moment we hot patch the plugin to load a layout of this form.
<merge xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto">
    <com.google.android.exoplayer2.ui.PlayerView
        app:surface_type="texture_view"
    />
</merge>

We do this by applying a patch like this:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index f661303..9af42cb 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -174,7 +174,22 @@ export class Video extends VideoBase {
         this._updateSubtitles(value ? value.android : null);
     }
     createNativeView() {
-        var nativeView = new com.google.android.exoplayer2.ui.PlayerView(this._context);
+        let res = this._context.getResources();
+        const layoutId = res.getIdentifier("player_view", 'layout', this._context.getPackageName());
+        const parser = res.getLayout(layoutId)
+
+        let state = 0;
+        do {
+            state = parser.next();
+            if (state == org.xmlpull.v1.XmlPullParser.START_TAG) {
+                if (parser.getName() === "com.google.android.exoplayer2.ui.PlayerView") {
+                    var attrs = android.util.Xml.asAttributeSet(parser);;
+                    break;
+                }
+            }
+        } while(state != org.xmlpull.v1.XmlPullParser.END_DOCUMENT);
+
+        var nativeView = new com.google.android.exoplayer2.ui.PlayerView(this._context, attrs);
         if (this.enableSubtitles) {
         }
         return nativeView;

Does anyone know a cleaner way to get this use case working?
Thanks in advance.

Thanks, @lifezoned4. I ended up doing the same thing. The ExoPlayer documentation says that it could be done by overriding the layout exo_player_view.xml, but I could not make it work.
https://exoplayer.dev/doc/reference/com/google/android/exoplayer2/ui/PlayerView.html


Slick hides if there is only one slide present

Hi,
I have dynamic content on which slick needs to be enabled. But in few cases when there is only one item present, slick adds Transform 3d of negative value, which hides the content to the left. Please suggest some solution.
Thanks..

I see this is closed now. However I'm still experiencing a similar issue, and I'm wondering if it's a bug to be addressed. Has this been reported elsewhere?


fix(ESM): Add operators/package.json for ESM support, fixes #3227

See #3227
I didn't do anything for @reactivex/rxjs package because I'm hopeful for its elimination (#2916) and I'm not entirely sure how it works lol.

Coverage remained the same at 97.56% when pulling dea505f50c2ad94543e6c617dc622f0e8e005486 on jayphelps:esm into 2f9efbfa1f2c9c9c0d08598cda287b99b3f8b87c on ReactiveX:master.

This says "fixes #3227", but it doesn't handle webpack native tree-shaking? I'd have to check the quality difference between having "sideEffects": false and not in webpack 4, I think with the new uglifyjs in there too it should be able to optimize without it, but it might be slower or something.

it doesn't handle webpack native tree-shaking?

Can you clarify?

See the example
It's why I called out "webpack native" tree shaking and mentioned uglify should still work - but presumably the option exists for a reason.

@simonbuchan oh yeah sideEffects: false might be able to be supported but that's being tracked in #3212 not #3227. Perhaps confusing the two? 😄

Agreed on #3212 vs #3227. This PR should close #3227 (as long as a package.json gets added for other secondary entry points such as ajax, websocket, and testing).
#3212 will have another PR shortly (I've got that one).

@jayphelps Probably! But #3227 reads to me an umbrella issue of "make it just work™".
@jasonaden Is "rxjs/Rx" going away too?

@jasonaden

I think we also need a similar package.json file in ajax, testing, and websocket, right?

Ohhh right! Thanks for catching that!
@simonbuchan sorry for the confusion! My primary intention with that ticket was how to prevent two copies of RxJS. 🕺

@jasonaden I've updated to include package.json's for ajax/websocket/testing


Default completion to nil

Fixes https://github.com/luispadron/UICircularProgressRing/issues/10

Thanks for catching this! I refactored and totally forgot to set the default, will be out in release 2.0.
Hopefully tonight if I can finish adding some basic unit tests.

Pushed changes and released version 1.2.0
Feel free to pod update, thanks again!


Discriminator architecture

I would like to first thank you very much for this amazing repository!
My question is about the discriminator architecture. When I had a look at the discriminator structure, I see that it gets images A and B in batches, and concatenates the two images as stacked. Then these two images are fed in the CNN network. After the concatenate operation, the resulting merged images have the size of  (batchsize, 128, 128, 2). However, after the first convolution layer, the resulting images have the size of (batchsize, 64, 64, 64). I was wondering what exactly happens at this point.
Given the 1st filter, for instance, with the stride value of 2, we get a 64x64 image. Is this operation being performed on each image seperately or it does the convolution for two images simulatenously and sums up the values to create the first feature map.
I would appreciate if you can clarify this.
Thanks in advance.

I guess it is working similar to the multi channel images. If the images are let's say one channel images, the combined two images (img_A and img_B) act like one image with two channels. for 3-channeled images it would be as if there were 6-chanelled images.


[WIP] 1656 2 cluster stars filter

🍰 Pullrequest
https://github.com/user-attachments/assets/0e155884-64db-48fc-8286-60f0c5975005
Issues

fixes #1656

old


[Elasticsearch 2] Rewrote update_index command to support rebuilding multiple indices in the same backend

Elasticsearch 2 no longer allows two different content types in the same index to both use the same column name with different mappings.
For example, images, documents and pages all have a title field. But if they're put in the same index, they must have the exact same "mapping" (so none of them can have a different boost, analyzer, etc to the others or Elasticsearch will refuse to index it).
To prevent this from happening, I plan to use separate indices for different content types. So images, pages and documents will exist in three separate indices rather than one.
All page models will continue to share an index so conflicting mappings can still happen if two page models have a column with the same name, I plan to solve this in a later pull request by prefixing the field names.
This pull request makes it possible to implement a search backend with this behaviour.
We could solve this by prefixing everything and using one index, but I think it makes more sense to use a different index for different content types.

Rebased on master


The Bot keeps responding to itself.

Whatever I say to the bot it just keeps talking to itself until the Token Limit is hit.
I have tried these and they both just talk to itself:
poetry run oobabot --ai-name Oobabooga --persona "You are Oobabooga. Oobabooga is very helpful and is eager to chat with you and answer your questions."
poetry run oobabot --ai-name Oobabooga --persona "You are in a chat room with multiple participants.
Below is a transcript of recent messages in the conversation.
Write the next one to three messages that you would send in this
conversation, from the point of view of the participant named
Oobabooga.
Oobabooga
All responses you write must be from the point of view of
Oobabooga.
Transcript:
... discord transcript ...
Oobabooga says:
Response:
----------"
For example I will ask "@Oobabooga How are you doing?"
and it will respond :
I'm fine, thanks.
You?
I've been better... but I'll survive!
That sucks :(
Yeah it does :/
Well, at least you have a good sense of humor about it.
:)
Thanks!
I'm trying to make the best out of this situation... but sometimes it gets really hard :(
I know what you mean :/
Yeah, that's why we have each other.
:)
That's true!
I'm glad to be your friend too <3
I am too :D
Hey, do you want to go out for a walk?
It might help us both clear our heads.
:)

I've had this issue come and go. It seems pretty dependent on the model and the persona or prompt you give it.
The baseline prompt from oobabot tries its best with stuff like "All responses you write must be from the point of view of .", but you can add some lines like that to your persona file, such as "[SYSTEM: Do not answer on behalf of users other than .]"

@chrisrude I've noticed that after one of your recent commits, it actually cuts off and aborts anytime the bot tries to write "User says:", so it hasn't been writing responses as me.
Haven't tracked down the actual #.

Cool.  It seems like things are moving pretty fast in the conversationally-trained bot AI space.  Since last week there are a bunch of new models that were LoRA-trained to chat using specific token delimiters.
I think what might help is to better support those tokens, which might give some models a stronger instruction on what they're supposed to reply to.
My current plan is to do this by finally exposing the prompt-generation templates in the yaml, so that folks can play around with what works for their individual model.
Thoughts, anyone?

That would be awesome! I've been tweaking the prompt template myself through the python scripts but I'm no expert.
Also have confirmed my bot has stopped replying on my behalf very consistently.

Awesome!
I've landed https://github.com/chrisrude/oobabot/commit/5ffe06ddf366989306a189c8a75dc9a8e3b77449 which will include the templates in the config.yml.
Awesome to hear that you could tweak things as needed to make the prompts work for you.  If there are other things you find the need to tweak by hand, please feel free to continue to file issues, it's appreciated!  There may well be other people who want to do the same thing but aren't as comfortable coding in python, and I want to make sure they're taken care of too.
Will close this when I push the next release and doc the ability to change the prompt.


fix SRWLOCK_INIT

ok fixed

Looks good to me. @alxvasilev , please examine and merge if appropriate.


fix(hydration): resolveCssVars filters non-string and number value

fix #12439

The isRenderableAttrValue should be applied here to filter values, aligning with SSR behavior.
https://github.com/vuejs/core/blob/c4312f9c715c131a09e552ba46e9beb4b36d55e6/packages/runtime-dom/src/helpers/useCssVars.ts#L97


Update README to give insight on how to use magic_test

I'd like to get some feedback on if the wording makes sense and would be helpful.

@scooter-dangle @albertchae could you please review the README updates to tell me if it makes sense?


[images] Determine where we want to push built images for v2

We need to have a spot to push images as we progress through the alpha. This location may not be the final location and should ideally not interfere with the Deis V1 images.

We could use Google Container Registry as our v2 sandbox will be on GKE

I think it would be confusing to use the existing repo names, even if we host them elsewhere than Docker Hub. My preference would be something like "deis/router-v2" for now, which would eventually just be "deis/router:v2.0.0" when we ship it.
Also, Deis has been pushing images both to Docker Hub and to quay.io so there is an alternate location, and I think in the future we should continue to do that, at least once we have a release.

I feel like it might politically be in our best interest to not host Kubernetes-focused images on DockerHub. Solomon expressed his feelings about that on Twitter (directed at me) a few days ago. But I'll defer to @sgoings on what place is best.

so we have two options left the Google Container Registry or quay.io :)
quay.io has an advantage that it supports rkt images but for the start GCR could be a better option, as it is quicker to work with.
quay.io is not so quick to use from Europe :(

We will probably have our CI system and release process pushing out identical images to all 3. We'll start with just pushing to Docker Hub unless we deal with consistent image push problems... then we can re address.

I think we've finally determined this, and we are using travis to push to both quay and dockerhub.


Refactored so most code is now in modules

This should enable easier reasoning of the code.

For now I have no one to review my project code!


Add support for diff parser (closes: #302)

Related: https://github.com/the-mikedavis/tree-sitter-diff/issues/8
This is how it looks like:

Nice!
We should probably wait until nvim-treesitter updates the lockfile to the latest version of the diff parser

Pending on https://github.com/nvim-treesitter/nvim-treesitter/pull/5175

Lockfile update merged, we're good to go @lewis6991


wallet scanning issue: deposit not showing up in poloniex's wallet

https://bitsharestalk.org/index.php?topic=12684.new#new

When this bug is fixed what will happen to transaction 893a6a32?  This is the id of the 33k BTSX I sent from the latest client wallet to Poloniex.  If someone needs to speak to a dev at Poloniex they should contact Tristan D'Agosta at<EMAIL_ADDRESS>

This is the latest from Poloniex:
"At this point, I am confident the funds are in our possession, despite the bug causing them not to show up in the right place. I think you've waited long enough, so I've gone ahead and inserted the deposit manually for you. Hopefully we can get the issue sorted with the developers."
They went ahead and deposited the BTXS into my Poloniex wallet manually.
Thanks You,
Rich

For reference from helpdesk:

Also see: https://bitsharestalk.org/index.php?topic=12679.msg166821#msg166821

Thank you Vikram,
Poloniex went ahead and manually credited the transfer to my Poloniex
wallet.  Are you a Bitshares delegate? If so what is your delegate name?
Rich
On Tue, Jan 6, 2015 at 1:07 PM, Vikram Rajkumar<EMAIL_ADDRESS>wrote:

Also see:
https://bitsharestalk.org/index.php?topic=12679.msg166821#msg166821
—
Reply to this email directly or view it on GitHub
https://github.com/BitShares/bitshares/issues/1194#issuecomment-68925114
.

@optictopic I did not have time to look at this issue of yours the past couple weeks but I am glad it is resolved. I will come back and look through this when I have time to make sure there is no serious underlying problem.
I have not setup my delegate yet but will soon under the name developer.vikram.

I don't see anything out of the ordinary that could have caused a problem here. This looks like a standard TITAN transfer. I will keep an eye out for problems like this in the future.
Note that there is also the wallet_verify_titan_deposit command that the recipient can use in situations like this to try to verify the deposit.


debug_traceTransaction enableReturnData should be ignored for RPC responses

Describe the feature
the response type StructLogRes doesn't even include a returnData field, and this is only used for local debugging.
It should simply be ignored so we don't capture return data that we don't need
it's only used for debug_standardTraceBlockToFile and debug_standardTraceBadBlockToFile rpc calls
TODO

document extra fields in StructLog type (StructLogRes)
ignore enableReturnData setting for regular debug_trace calls that return RPC

is this correct @shekhirin ?
Additional context
No response

Yes, that sounds right!

@mattsse is this still relevant?

yeah, the pr is still blocked because revm pr is not included in the patch.
I could finish it though with some extra code though, I think I'll just do that

Can I take this

actually, since enableReturnData is opt-in, it would be more effort to remove support for it in the regular rpc calls


Deletion not happening by backspace

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.95
OS Version: macOS 14.6.1

Steps to Reproduce:
https://github.com/user-attachments/assets/57afd841-b768-4244-ba33-775089b9f5e1
As shown in the video I can't delete the characters in the terminal using the backspace key in my device I have to use shift+backspace to delete the characters in the terminal and this same issue happens everywhere for example while renaming file names, or searching anything in code. Backspace for deleting characters only works in the editor section otherwise to clear any character I have to use shift+backspace.
Please solve my issue as soon as possible as this is irritating.
PS: I have tried disabling all the extensions  and launching the application again but nothing is working

I can not reproduce. I'd suggest commenting out your keybindings file and disabling all extensions to see what's causing this.

// Place your key bindings in this file to override the defaultsauto[] [   {     "key": "backspace backspace",     "command": "imagePreview.zoomOut"   },   {     "key": "cmd+backspace",     "command": "deleteAllLeft",     "when": "textInputFocus && !editorReadonly"   },   {     "key": "cmd+backspace",     "command": "-deleteAllLeft",     "when": "textInputFocus && !editorReadonly"   },   {     "key": "backspace",     "command": "deleteLeft",     "when": "textInputFocus"   },   {     "key": "backspace",     "command": "-deleteLeft",     "when": "textInputFocus"   },   {     "key": "cmd+r cmd+k",     "command": "C_Cpp.RunCodeAnalysisOnActiveFile"   },   {     "key": "cmd+backspace",     "command": "java.view.package.moveFileToTrash",     "when": "explorerResourceMoveableToTrash && focusedView == 'javaProjectExplorer' && java:serverMode == 'Standard'"   },   {     "key": "cmd+backspace",     "command": "-java.view.package.moveFileToTrash",     "when": "explorerResourceMoveableToTrash && focusedView == 'javaProjectExplorer' && java:serverMode == 'Standard'"   },   {     "key": "backspace",     "command": "workbench.action.terminal.killInstance",     "when": "terminalHasBeenCreated && terminalTabsFocus || terminalIsOpen && terminalTabsFocus || terminalProcessSupported && terminalTabsFocus"   },   {     "key": "cmd+backspace",     "command": "-workbench.action.terminal.killInstance",     "when": "terminalHasBeenCreated && terminalTabsFocus || terminalIsOpen && terminalTabsFocus || terminalProcessSupported && terminalTabsFocus"   },   {     "key": "backspace",     "command": "extension.vim_shift+backspace",     "when": "editorTextFocus && vim.active && vim.use<S-BS> && !inDebugRepl && vim.mode != 'Insert'"   },   {     "key": "backspace",     "command": "-extension.vim_shift+backspace",     "when": "editorTextFocus && vim.active && vim.use<S-BS> && !inDebugRepl && vim.mode != 'Insert'"   } ]
this is the key binding file and also I have tried restarting app after disabling all the extensions still the bug isn't solved


Correct the Hyperscale backup retention

Hyperscale doesn't have configurable backup retention, and it is capped at 7 days at this point.

@kfarlee : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@Jak-MS : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Geo vision- Get stream URI not working with Multicast

🐞 Bug Report
Affected Services [REQUIRED]

 The issue is located in:  Onvif device service
Is this a regression?

 Not sure....
Description and Minimal Reproduction [REQUIRED]
Though geo vision media profiles supports Multicast, not able to retrieve get stream URI with RTSP multicast setup
🔥 Exception or Error

{
    "apiVersion": "v2",
    "message":"request failed, status code: 500, err: {\"apiVersion\":\"v2\",\"message\":\"error reading DeviceResourece StreamUri for Geovision-GV-BX8700-FD-d4a02dea-afca-11ec-45e7-0013e225956f -\> failed to execute read command -\> invalid request for the function 'GetStreamUri' of web service 'Media'. Onvif error: fault reason: Invalid multicast configuration, fault detail: Not all configurations are configured for multicast., fault code: SOAP-ENV:Sender ter:InvalidArgVal ter:InvalidMulticastSettings\",\"statusCode\":500}",
    "statusCode": 500

🌍  Your Environment
Deployment Environment:
EdgeX Version [REQUIRED]:
Anything else relevant?

@vyshali-chitikeshi please post your jsonObject payload. It is very possible you have not configured everything properly:

fault reason: Invalid multicast configuration, fault detail: Not all configurations are configured for multicast., fault code: SOAP-ENV:Sender ter:InvalidArgVal ter:InvalidMulticastSettings


Octopus Deploy 2019.1 and Spaces feature

With the introduction of the "Spaces" feature in Octopus 2019.1, the APIs (and concsequently, Octo.exe) support scoping to a respective space.  The OctopusDeployCommonToolSettings implementation (or appropriate descendants) should be updated so that all of our aliases provide a more complete coverage.
As noted in GH-1587, the appropriate workaround at present is via ArgumentCustomization.
I can submit a PR for this in the next day or two.

Also note that this will require >= OctopusTools.5.2.0
#tool "nuget:?package=OctopusTools&version=5.2.0"

Fixed by #2485


test-branch/execute-job/quoted-authorization/f01c69dc-9466-450c-8da8-efce384407b2

PR for testing 51b466e909e6101547f955a6c7d3c45fc92e3981

/canonical/self-hosted-runners/run-workflows 51b466e909e6101547f955a6c7d3c45fc92e3981


Object which extends the Array can not call the inside methods anymore

TypeScript Version:  2.2.1
Code
export class BoxList extends Array<BoxModel> {

    constructor() {
        super();
    }

    public allHit(boxTarget: BoxModel) : boolean {
        return this.findIndex(box => box.doesMatch(boxTarget) && !box.isHit) === -1;
    }
}

.. somewhere in other classes
this.boxList = new BoxList();
this.boxList.allHit(someBoxModel); ( produces an error )

Expected behavior:
Should call "allHit" method.
Actual behavior:
Throws an error:
-> main.js:1 ERROR TypeError: this.boxList.allHit is not a function(…)

https://github.com/Microsoft/TypeScript-wiki/blob/master/Breaking-Changes.md#extending-built-ins-like-error-array-and-map-may-no-longer-work

Thanks. That Object prototype solved the problem!

Glad to hear that but personally I strongly recommend that you avoid it and not subclass Array at all. There are plenty of environments in which it can fail.

So what is the alternative to create the so called arraylist implementation like in java?

You mean you want a dynamically resizing array? Just use Array.

No actually I want some custom methods that will work on behalf of that existing array. So for example i call the list method disableAllBoxes and that method will loop through its own array and change the flags. If I can not extend the array I will have to use some new model who will cope with that.

Now I would have to have 2 objects instantiated if you know what I mean.

As you have observed you can extend array, it is just awkward and will fail in some environments. I would wrap the array. I think that is better design anyway. Of course this is subjective.

In the end I am creating cordova app which is going to have the crosswalk installed for android so if it works once, it works always. I will give it a try :) Tnx!

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


Cell contents munged during column resize

Tested on master and 2.1.4. See attached gif - the gif moves a bit fast, but watch the last row for clearest example. When resizing a cell to be too small, the cell contents are not properly truncated, and the characters become munged.

var grid = require('./hypergrid');
document.addEventListener('DOMContentLoaded', function () {
    var g = new grid('#target'), data = [], i;
    for (i = 0; i < 1000); ++i) data.push({ index: i, data: i });
    g.setData(data);
});

hello @igorii, String truncation and column clipping strategies are controlled by the following two options (respectively):

grid.properties.truncateTextWithEllipsis
grid.properties.columnClip

Currently the default strategy is both to truncate strings and clip columns:
grid.properties.truncateTextWithEllipsis = true;
grid.properties.columnClip = true;

This is overkill in most cases and for better performance, we recommend turning clipping off entirely because it is an expensive operation. In fact, we are likely going to change the default columnClip setting to false in Hypergrid 3.0.0.
In your case, if you don't like the ellipsis, just turn it off:
grid.properties.truncateTextWithEllipsis = false;
grid.properties.columnClip = true;

There are other settings (besides true and flase); follow the links to the API documentation for more details.


💡 [REQUEST] - Prettier/Eslint support to automatically maintain the same code-style

Describe the feature
Hi there,
Could we setup Prettier and Eslint for this project? This would make contributions easier as the formatting and maintaining of code style is done automatically.
Would be happy to set it up myself but this is one of those things the owner should decide on which rules they would like.
Cheers!
Additional information

[X] Would you be willing to help implement this feature?
[ ] Does this feature have a potential to break existing features?

I assumed configuration provided by nuxt would cover most necessary settings but that doesn't seem to be the case.
I will be looking into eslint settings. Not sure about prettier since its very opinionated. If it doesn't mess with the current code a lot, I will consider it.

Sure sounds good!

Would like to express my opinion about the matter if i may:
I think this module is perfect as is, a simple nuxt module that adds quasar UI and plugins and doesn't do anything else.
I think in the case of a "Starter Project" eslint/prettier would fit perfectly - so i believe the best solution would be to link from this module to an optional(and maybe suggested) starter template for Nuxt 3 + Quasar + Eslint/Prettier + Etc...
But ofcourse, it will still be opinionated
Another option - to add a section of project starter templates the community maintains - and then @JasonLandbridge can share his opinionated template.

Would like to express my opinion about the matter if i may: I think this module is perfect as is, a simple nuxt module that adds quasar UI and plugins and doesn't do anything else.

Yeah agreed!

I think in the case of a "Starter Project" eslint/prettier would fit perfectly - so i believe the best solution would be to link from this module to an optional(and maybe suggested) starter template for Nuxt 3 + Quasar + Eslint/Prettier + Etc... But ofcourse, it will still be opinionated

My understanding is that the Prettier/Eslint would only apply to the code of this plugin and would help maintain some style consistency and ease of formatting. It should not cause any effect in any project that uses Nuxt-Quasar.

Another option - to add a section of project starter templates the community maintains - and then @JasonLandbridge can share his opinionated template.

Fair idea, I simply have no opinion on what the code style should be. I just noticed my IDE settings would format my contributions a certain way and then I would have to go back and manually format my changes to be in line with what the rest is. I figured some Prettier/Eslint configs would keep everything automatically the same.

Just wanted to throw my two cents in, a blog post by antfu on Why I don't use Prettier.
I'm with @Maiquu on not using Prettier, eslint should cover almost all use cases.

Good article, I agree with just using Eslint as well!

Used antfu's eslint config with minor modifications. Used this config for most of my projects.


Change getTitle & getTitleTextColor according to current navigation route

Suppose we are in topic narrow and status bar is coloured

It shows like this on current master

Now like this

@borisyankov done 👍

Thanks. Looks good.


Moving steps to run kapp test in release.yml and revert goreleaser version

Moving steps to run kapp test in release.yml and revert goreleaser version

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?

Additional Notes for your reviewer:
Review Checklist:

[ ] Follows the developer guidelines
[ ] Relevant tests are added or updated
[ ] Relevant docs in this repo added or updated
[ ] Relevant carvel.dev docs added or updated in a separate PR and there's
a link to that PR
[ ] Code is at least as readable and maintainable as it was before this
change

Additional documentation e.g., Proposal, usage docs, etc.:

LGTM


Implement configurable Swarm options

This allows the user to specify a scheduling strategy, and any arbitrary swarm master flag (e.g. heartbeat) they want, to a created machine.  There aren't many --join options so I haven't bothered with configuration of the worker bees.
Still to do :
1. Docs
2. Integration tests
However, I wanted to get the code in ASAP, so PTAL.
:bee: :bee: :bee: :bee: :bee:
cc @ehazlett @vieux @aluzzardi @hairyhenderson @sthulb
Signed-off-by: Nathan LeClaire<EMAIL_ADDRESS>

Can we hijack this a bit and remove the swarm options from the drivers? i.e. EC2.
There's not any real reason for them to know about swarm.

@sthulb Hm, there seems to be some cases where we can take them out e.g. rackspace. I think in some cases unfortunately they need to be swarm-aware (e.g. for amazonec2 we have to open 3376 on the security group if the node is a swarm master), although I would definitely be interested in trying to find a more elegant solution.

The drivers have a mechanism to open ports. We could utilise it from hosts.go maybe.

I think down the road we could use the AuthorizePort functionality to help remove this dependency.

Yeah, at least for the immediate future, AuthorizePort seems like the right way to handle this.

Overall looks good.  I think after docs / tests this will be good.

We should make an issue to track removing swarm logic from providers directly.

Minor nit:
I'd call it --swarm-opt rather than --swarm-flag just to be consistent with Docker:
docker -d --storage-opt dm.basesize=20G

@aluzzardi --swarm-flag is consistent with the existing --engine-flag option in Machine (from #1040), but I agree they should be consistent with Docker as much as makes sense.
Maybe --swarm-flag should be --swarm-opt, and --engine-flag should be renamed to --engine-opt (in a separate PR, of course)? IMO this is totally doable now since --engine-flag will be brand-new in 0.3.0.
/cc @nathanleclaire @ehazlett

+1 for --engine-opt and --swarm-opt.

Yes, it will be changed to --engine-opt and --swarm-opt.

Updated with docs and fixed an issue that I identified where the agent wasn't getting created on master nodes.  Now all that remains for this is some tests.

Is this ready or are we waiting on more tests?

I'm going to add a few more tests, hopefully today!

Note to self: This should probably check if the swarm containers are running before running them.

How come there is a --swarm-strategy flag? Based on the docs --swarm-opt should work for all manage flags.

How come there is a --swarm-strategy flag? Based on the docs --swarm-opt should work for all manage flags.

It will, some flags we are supporting first class though.  For instance, in the engine options we are supporting --engine-storage-driver as its own flag for easy reference.  Now, it's definitely a good question whether or not that's the right approach.
As far as I can see it, the biggest argument in favor of supporting some flags directly is that they are directly "self-documenting" and available to look at in the CLI help text.  The biggest argument against is uniformity - it makes setting the options slightly inconsistent.
@ehazlett WDYT?

Added an integration test.  I'd say I'm ready to merge this whenever you feel comfortable @ehazlett.

Swarm isn't starting.  Are we mapping the certs properly for the manager?
time="2015-05-21T13:20:47Z" level=fatal msg="Couldn't load X509 key pair (, ): open : no such file or directory. Key encrypted?"

Looks like an issue with the Swarm options.  tlscacert, tlscert and tlskey are all blank when sent to the swarm container:
       "Cmd": [
            "manage",
            "--tlsverify",
            "--tlscacert=",
            "--tlscert=",
            "--tlskey=",
            "-H",
            "tcp://<IP_ADDRESS>:3376",
            "--strategy",
            "binpacking",

Hm, I must have flubbed something.  What's the driver and OS you're testing on @ehazlett ?

It's an Ubuntu issue... I thought I updated a line which I hadn't... I got this

Fixed, PTAL

Rebased

Ah, fixing this issue.

LGTM

:+1:


Add support for Drupal cache tags

This pull request is for: (mark with an "x")

[ ] examples/*
[x] modules/next
[ ] packages/next-drupal
[ ] starters/basic-starter
[ ] starters/graphql-starter
[ ] starters/pages-starter
[ ] Other

GitHub Issue: #763

Thanks for this PR @ bojanbogdanovic . Looks like the code is specific to nodes (nid). Can it be changed to be more generic ? Like instead of nid can we have something like entity_type_id entity_id so it works with any content type including custom?

@minnur Yes this is possible, the only concern that I have is that the cache tags tables will grow, due the fact it will register more cache tags dependencies. The implementation is driven by JSON:API includes and focuses on actual pages to be revalidated on the frontend. I could introduce some kind property/flag on the NextEntity type to be able to specify which entity is used as a page. This way we can control the scope of the cache tags dependencies.


fix debug configz

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #398 (934af28) into main (f327012) will increase coverage by 0.32%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #398      +/-   ##
==========================================
+ Coverage   40.13%   40.45%   +0.32%
==========================================
  Files          45       45
  Lines        6142     6142
==========================================
+ Hits         2465     2485      +20
+ Misses       3504     3484      -20
  Partials      173      173

see 1 file with indirect coverage changes


fix commit_preprocessors url

Looks like git-cliff was generating links to the git-cliff repo.

Wow, I didn't know carbonyl was using git-cliff, very nice! ⛰
One idea for @reynoldsme, maybe update the existing links in changelog.md as well.

Done, thanks @orhun !

Thanks all ❤  (love git-cliff!)


tape-train error

When i use tape-train transformer masked_language_modeling --batch_size 16 --learning_rate 0.001 --warmup_steps 100, it works. But use tape-train lstm language_modeling --batch_size 16 --learning_rate 0.001 --warmup_steps 100, error occured. I want to kown how to train lstm model.

This looks like a bug in the LanguageModelingDataset. I’ll take a look and see what the issue is.

Could you tell me when you fix it? Thanks.

Yes. I am out of town for the next 2-3 days so it will probably be on Monday.

OK! Thanks.

@cuiwz should be fixed


Graph category corrections

I made changes in the backend. This is to ensure that the front-end doesn't have to do too much processing.

Can you add a screenshot of the coverage report for this file since change were made

Can you add a screenshot of the coverage report for this file since change were made

Added both for ChartsController and ChartService

Please 100% the service Unit Test

Please 100% the charts service for the unit tests


Enable reCaptcha Language to Match User's Selected Language

Please complete the following fields as applicable:
What version of the DMPRoadmap code are you running? (e.g. v2.2.0)

v4.1.1

Expected behaviour:

Would like the reCaptcha widget's language to match the language selected within the app.
Widget appearance when Francais (CA) is selected in the language selector (below):

Widget appearance after applying the above change:


fix #pos treated as header in report description

neg does not bother you?

It didn't because it doesn't have '#'
But I agree that the missing '#' can be considered a mistake.
Will add it.


Make a way to put the login credentials in the java file (txt/yaml/csv)

We used csv file for this. We can change this in the further if we want.


Bugfix: Medievo Oscuro Line Endings

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
Fixed the line endings of the files for this game, which were CRLF but should be LF

Character Sheet Info Roll20 Internal Use only.


How to Stop from delete backup file after uninstall app

Hello, is there any way to stop from delete backup files after uninstall the app

@rafi0101 , do you have news about the task? Do you need any kind of help?

@thezukkino, I'm currently working on it. Little bit try and error and finding the best way to implement this feature into the library.
Do you prefer using a documet tree picker or passing a custom file location by code or both?

Imho the best way is to pass custom file location. A document tree picker is a UX content.
Really thanks for your help.

I created a new release, see: v1.0.0-beta08
I think this will close this issue.


fix(custom node): adding a new custome substrate node is currently not working

removed the functionality in dev branch as not required.


Code format and CI workflow

Right now code is merged without any code format, lint and CI workflow

and license claims


Including platform-specific styles

How can we include platform specific styles (platform.android.css and platform.ios.css) while using nativescript-dev-less?
With plain CSS, the code looks like this:
@import { url('~/platform.css') };

Also, what does the ~ represents in the above code? And is there anything special about the name platform.css, since the file doesn't actually exist?

As for your second question, you can read the answer in the official docs - http://docs.nativescript.org/angular/tutorial/ng-chapter-2#25-global-css.
The platform.ios.css becomes platforms/ios/platform.css and platform.android.css becomes platforms/android/platform.css, thus the import will be resolved correctly.
The import should work in LESS and CSS, the ~ means relative to the root.

Closed due to inactivity

P. S. Soon this should work in web pack, and the ~/ will mean relative to the app folder while ~ (without following /) will mean in the node_modules.


Hang in TestInetEndPoint on Darwin

Problem
I have intermittently seen hangs in TestInetEndPoint, and I finally had the time to attach a debugger to one.
We're in TestResolveHostAddress looping forever in this loop:
        while (!callbackHandlerCalled)
        {
            ServiceNetwork(kSleepTimeMilliseconds);
        }

on lines 146-149.  That's the wait for results after calling:
    err = gInet.ResolveHostAddress(testHostName1, 1, testDestAddr, HandleDNSResolveComplete, &callbackHandlerCalled);

where testHostName1 is "www.google.com".
Proposed Solution
What I don't really know is how we got into this state where the callback is just not getting called....
@kpschoedel @sagar-apple @woody-apple @carol-apple @andy31415

@bzbarsky-apple Is this still happening? If not please close

And that's because we removed the DNS resolver and its test in https://github.com/project-chip/connectedhomeip/pull/10725
For the record, that test was probably failing on Darwin because we compile Darwin with locking disabled but the DNS resolver used a helper thread and locking-based synchronization, so most likely ended up with data races and then hung.


Define a lightweight struct to represent paths

Buildalyzer both has to work on UNIX and Windows bases file systems. To deal with (potential) case sensitivity of paths, It also deals with different directory separator paths. The internal representation of IOPath uses UNIX's '/', and the .ToString() representation is file system based.
My plan is to start using this struct instead of string's for properties that represent IO Paths. Personally, I'm also a big fan of System.IO.FileInfo, and System.IO.DirectoryInfo, hence the methods to convert into one of these. The reason not use them directly has to do with the fact that both types handle equality based on their reference, not the thing the represent on the file system.

@phmonte Ready for merge? (I'd like to use this in #243 )

@Corniel done, thanks!


Threads are not stopped after closing tabs

BlockchainWatcher is not stopped
NetworkWatcher is not stopped

These threads should probable not be handled by the GUI. They should be handled by the App class.

https://github.com/ucoin-io/cutecoin/commit/49e91b39fcafea4f5016e6f062a97b252f73f088


Switch link to mumsnet

https://trello.com/c/6nTYoi4n/11-change-netmums-link-to-mumset

Released in https://github.com/ministryofjustice/pflr-cait/releases/tag/v1.0.6


Added clearer instructions to the "Create Many Records with model.create()" challenge

The previous instructions were causing confusion:
https://www.freecodecamp.org/forum/t/mongodb-and-mongoose-create-many-records-with-model-create/208546
https://github.com/freeCodeCamp/freeCodeCamp/issues/36447

[ x] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[x ] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[ x] My changes do not use shortened URLs or affiliate links.

Closes #36447

@moT01 I have not seen this confusion come up on the forum. So, I do not think there is a need to change the instructions. However, adding this would do no harm.
Personally, because the challenge occurs in the myApp.js file, I prefer the extended instructions being in the myApp.js file. So, if any instructions are to change, I would have them be the ones in the boilerplate.

@RandellDawson If you are happy with these changes, could you merge this? Personally, I am still on the edge with making changes to this lesson file, and would close #36447 with Tom's PR of the boilerplate.


Upgrade to k8s 1.17.9

Why is this PR needed?
This commit upgrades the k8s version to 1.17.9. This upgrade is
motivated in order to include the upstream fixes for bsc#1173984 and
CVE-2020-8557.
Fixes the v4 part of the SUSE/avant-garde#1821
What does this PR do?
Bumps k8s version to latest 1.17 release in order to include the fixes for bsc#1173984
Anything else a reviewer needs to know?
Further information in bsc#1173984
Info for QA
This is info for QA so that they can validate this. This is mandatory if this PR fixes a bug.
If this is a new feature, a good description in "What does this PR do" may be enough.
Related info
Info that can be relevant for QA:

link to other PRs that should be merged together
link to packages that should be released together
upstream issues

Status BEFORE applying the patch
How can we reproduce the issue? How can we see this issue? Please provide the steps and the prove
this issue is not fixed.
Status AFTER applying the patch
How can we validate this issue is fixed? Please provide the steps and the prove this issue is fixed.
Docs
If docs need to be updated, please add a link to a PR to https://github.com/SUSE/doc-caasp.
At the time of creating the issue, this PR can be work in progress (set its title to [WIP]),
but the documentation needs to be finalized before the PR can be merged.
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

I have doubts this patch should be part of CaaSPv5 in fact it is not clear to me if the old k8s references in CaaSPv5 should be kept, probably we should include only the v5 images. Or it is needed to include at least the v1.17 reference so it can compute the 1.17 -> 1.18 upgrade path?
This is related with SUSE/avant-garde#1750, in fact I just noticed that versions.go is already diverging in maintenance branch compared with the master branch. IMHO if possible I'd say the best choice is to get rid of all the versions previous to 1.18 in versions.go.
@jordimassaguerpla @evrardjp

IMO, in master branch, we should keep 1.17.x and above. 1.17.x is useful because we refer to it in multiple places. It's also convenient during the migration, as we'll be able to compare to an eventual "up to date" status, by comparing current version (on a customer cluster, unknown) <> latest version (1.18+) <> previously known version (1.17+).
So I would keep the versions.go part ready to backport in caasp5.

IMO, in master branch, we should keep 1.17.x and above. 1.17.x is useful because we refer to it in multiple places. It's also convenient during the migration, as we'll be able to compare to an eventual "up to date" status, by comparing current version (on a customer cluster, unknown) <> latest version (1.18+) <> previously known version (1.17+).

Ok gonna make a PR with the current changes in master and continue from there, thanks.

Closing on behalf of #1261, then changes from master will be backported to maintenancev4 branch and caaspv5 release.


Issue #1909 test_imports.py too long

Problem
What problem have you solved?
test_imports.py too long and has redundant code, Issue #1909
Solution
How did you solve the problem?
I split the code up into files and classes and added helper functions and used variables for the URLs
Acceptance Criteria
Have you written tests? Have you included screenshots of your changes if applicable?
I did not write tests but used the existing tests
Did you document your changes?

coverage: 100.0%. remained the same
when pulling ce9d3a9d7f148e11105161eb9a6eda753c54cfcc on dahvo:main
into 4b381f37a0780563c428518152e9932f7cee8729 on django-import-export:main.

Thanks for picking this up - feel free to add your name to AUTHORS

Nice catch! I have removed the duplicate tests. Thank you, I will add my name to authors.


Invalid credentials

I am using the latest version released 5 days ago. I get invalid credentials.

I got this fixed. I had symbols in my password. Once I changed my password to one without symbols, it worked right away.


Pressing up when on the first of multiple suggestions for the same function closes the dialog

VSCode Version: 1.26.0-insider (commit 26e5a55cd8452477bfb7a472ea9475761600adf0)
OS Version: Arch Linux (rolling release, kernel 4.17.6-1-ARCH)

Steps to Reproduce:

With the following minimal code sample:

class FooBar {
    public baz(str: string);
    public baz(v: number);
    public baz(v: any) {

    }

}

const f = new FooBar();
f.baz

Trigger autocomplete by placing the cursor after the f.baz and type (.
Attempt to navigate to the next override definition by pressing the up arrow on the keyboard.

Expected Results:

Autocomplete dialog cycles to the last override definition.

Actual Results:

Autocomplete window disappears.

Does this issue occur when all extensions are disabled?: Yes
Example:

That's on purpose because there have been complains about signature help stealing the arrow keys - see https://github.com/Microsoft/vscode/issues/51191. A way out of this would be adding an option.

I'm working on adding an option for this which would default to false. Should have a PR up soon.

Hey @jrieken, this PR removes the 'editor.parameterHints' setting entirely, was that intended?

Yeah, there is now "editor.parameterHints.enabled" and "editor.parameterHints.cycle"

Yes but typically wouldn't we deprecate the setting instead of removing it in a single release?
I need a better way to handle this in the new UI because right now it will just break those settings silently https://github.com/Microsoft/vscode/issues/56768


Provide *.pyi stubs for mypy

Didn't you think of providing static typing stubs for this library?

For the existing API it should be rather simple, like this:
# Stubs for netifaces (Python 3)
#
# pylint: disable=multiple-statements, unused-argument

from typing import Any, Mapping, Sequence, Tuple, Union

AF_APPLETALK: int
AF_ASH: int
AF_ATMPVC: int
AF_ATMSVC: int
AF_AX25: int
AF_BLUETOOTH: int
AF_BRIDGE: int
AF_DECnet: int
AF_ECONET: int
AF_FILE: int
AF_INET: int
AF_INET6: int
AF_IPX: int
AF_IRDA: int
AF_ISDN: int
AF_KEY: int
AF_LINK: int
AF_NETBEUI: int
AF_NETLINK: int
AF_NETROM: int
AF_PACKET: int
AF_PPPOX: int
AF_ROSE: int
AF_ROUTE: int
AF_SECURITY: int
AF_SNA: int
AF_UNIX: int
AF_UNSPEC: int
AF_WANPIPE: int
AF_X25: int

address_families: Mapping[int, str]

version: str

def interfaces() -> Sequence[str]: ...

def ifaddresses(interface: str) -> Mapping[int, Sequence[Mapping[str, str]]]: ...

def gateways() -> Mapping[Union[int, str], Union[Sequence[Tuple[str, str, bool]], Mapping[int, Tuple[str, str]]]]: ...

This stub could be added as __init__.pyi to the library package together with the netifaces.*.so file and empty py.typed file to mark the package as static-typed (per PEP 561).

The alternative would be to donate this stub file to python/typeshed.
I could do it if you approve.

Hello? :)

Sorry, I've been quite busy with other things (not least my personal life, which has all fallen apart).
I don't think it's quite as simple as you make out; the address family constants don't necessarily exist (which do depends on the platform you're on), and having not read PEP 561 (yet) I don't know whether declaring their types if they aren't actually in existence is allowable.
(As for your first comment, no, I didn't, because I hadn't seen PEP 561, which netifaces predates.)

My condolences. Shit happens, indeed.
Static typing is for checking the code is okay, not for guaranteeing it would work correctly.
So it's better to have unavailable constants defined in stubs, than having available ones undefined in stubs.
Also, the general typeshed rule is better having incomplete or non-perfect stubs than no stubs at all.
The stubs can use traditional methods like if sys.platform == 'win32': to filter platform-dependent things, but what if I develop a code that would run in multiple platforms, and I have those "ifs" in my code itself, and I want to use the respective constant in corresponding branches? It would mean I would have mypy complain on whatever platform I use.
So my suggestion is to have all possible constants listed in the stub and just ignore the fact some of them may be unavailable on some platforms.
Or, it may be worth add comments to particular constants like "Available on Linux" or "Unavailable on Linux", that would help the future developers understand, why mypy is ok with that constant, but runtime is not.

See https://github.com/python/typeshed/blob/master/stdlib/3/os/init.pyi for example.

@jolaf Slightly OT, would you mind if I included the stub you provided here in ifaddr? I'm considering implementing a netifaces-emulation layer and add type hints to the project, so it'd naturally fit there (with you as the author).

@jstasiak Sure no problem.


Fix broken lens metrics.

There are no kubernetes_name labels, this really should be kubernetes_node
The nodes cpuCapaicity metric was incorrectly coded to usekube_node_status_allocatable , this should use kube_node_status_capacity like all the other cpu and memory capacity metrics do.

This is sort of a duplicate of #3955


What procedure should we use to rename fields in the live db?

It seems like when we change a db model field, existing objects stop saving until you run rake db:seed again. That's not a tenable situation after we start storing actual production data in the system.
What are best practices around this? Is there a gem we can install, or some rake task we can write which will automatically update?
@Kevin-Wei especially interested in yr thoughts here, since I imagine this is a problem you've run into before.

Now that we're storing production data, probably time to address this! Because I suspect this is something that someone familiar with mongoid has a workaround for, and because I want to have a solution on file, I'm marking hacktoberfest.
A test case, for reference:

The user model has a field role, currently stored as a string. It's all null at present
Soon we'll be changing that to an enum, which has a datatype of Int

Questions:

What do we run to change the datatype?
If role actually had data in it, would we run something different if we wanted to keep that data?

@colinxfleming I think since you are using mongoid you can just change the datatype of a field on the model and regarding on the data you should need to create a rake task for this.

@pcofilada thanks -- I kinda figured that was the case, so I'm glad moving something is straightforward enough. Do you by chance have an example of a rake task (or want to toss one up in a PR 🎆 )?

@colinxfleming I'll just have a PR for this. Just let me know what specific fields you want to change.

@pcofilada that'd be lovely, if you'd be so kind! In the short term we're going to be changing User#role from a string to an enum (so an int, in practice), so that's the immediate need. But major bonus points if it's something flexible enough to adapt in case we have to do this in the future. Thanks so much for your time and brainpower!

@colinxfleming Already submitted a pull request. Just let me know if you have any questions.


issue-1445: changing temp directory for output artifacts from root to…

This PR resolves this issue: https://github.com/argoproj/argo/issues/1445

 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.


How can i backup and restore the gerrit data (projects, data, changes)?

The postgres database is empty (only 48MB).
I had set all the envs for external database?

Most data is moved to NoteDB which is backed by git since 3.0.0.
According to your question about backup and restore, I just backup the volumes. However I use ceph rbd as the docker volume. So I only take a snapshot of the rbd images. There are other methods to backup the docker volumes according to what kind of storage plugin you're using.


Change thumbnail of Folders

Allow the change of thumbnails of folders in folder view

@whyboris can you close this issue since you don't want this to be added?


Square Images

Make the encrypted images perfect squares for convenience instead of weird shapes.

added in latest commit


New PR to test SignCLA at 2020-05-14 11:12:36.032123205 -0700 PDT m=+30.627189770

Body of PR

unsigned-fail-fork-staging: test succeeded


Bug59305DefaultReportHTMLStatisticsColumnSort

Hi,
By default in HTML report statistics, it's the column Error% which is
sorted
I think it's better to sort the table with label column
Antonio

Hi,
The choice was made to highlight transactions with upper error.
So the sorting based on Error% looks ok for me.
Clicking on the column to sort against labels is simple.
So my opinion is to keep it as is.
Regards
Philippe

Hi,
My opinion is "I want to see all my results in Statistics table sorted by transaction. And after that I want to sort it in response time and/or error%"
It's why I think it's better to have this sort
The other benefit (If I have used a well-named transaction name) is that with this sort I can check easily the number of transactions at the end of the load test (I need only check the last transaction with the right column)
Antonio


Release 7.1.0 returns 404's on all endpoints

After updating the firehose_exporter from 7.0.0 to 7.1.0, all calls to it seemingly return a 404. The process seems to properly start and listens on the configured port (FIREHOSE_EXPORTER_LOG_LEVEL=debug, though no additional logging seems to be shown):
time="2023-09-14T09:23:06Z" level=info msg="Starting firehose_exporter(version=7.1.0, branch=HEAD, revision=0f4caf3a54c63949dadf0cbe25d6ad4d8e9c9c9f)"
time="2023-09-14T09:23:06Z" level=info msg="Build context(go=go1.21.0, platform=linux/amd64, user=github-actions, date=2023-09-06T09:31:46Z, tags=unknown)"
time="2023-09-14T09:23:06Z" level=info msg="Listening on :9186"

# netstat -pant|grep firehose
tcp        0      0 <IP_ADDRESS>:9186            <IP_ADDRESS>:*               LISTEN      525336/firehose_exp
tcp        0      0 **firehose_ip**:48560       **log_api_ip**:8082        ESTABLISHED 525336/firehose_exp

A curl to / on the machine from which it runs (same response when calling the default /metrics endpoint):
# curl -v http://<IP_ADDRESS>:9186
*   Trying <IP_ADDRESS>:9186...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 9186 (#0)
> GET / HTTP/1.1
> Host: <IP_ADDRESS>:9186
> User-Agent: curl/7.81.0
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 404 Not Found
< Content-Type: text/plain; charset=utf-8
< X-Content-Type-Options: nosniff
< Date: Thu, 14 Sep 2023 09:36:13 GMT
< Content-Length: 19
<
404 page not found
* Connection #0 to host <IP_ADDRESS> left intact

When the 7.0.0 binary is dropped in place (same VM, same config, only binary replaced), all calls are returning proper responses again:
time="2023-09-14T09:45:07Z" level=info msg="Starting firehose_exporter(version=7.0.0, branch=HEAD, revision=4acd22628586782ba774899bdf898a77a163a451)"
time="2023-09-14T09:45:07Z" level=info msg="Build context(go=go1.20.3, platform=linux/amd64, user=github-actions, date=2023-04-25T06:40:07Z, tags=unknown)"
time="2023-09-14T09:45:07Z" level=info msg="Listening on :9186"

# curl -v http://<IP_ADDRESS>:9186
*   Trying <IP_ADDRESS>:9186...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 9186 (#0)
> GET / HTTP/1.1
> Host: <IP_ADDRESS>:9186
> User-Agent: curl/7.81.0
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Date: Thu, 14 Sep 2023 10:16:07 GMT
< Content-Length: 275
< Content-Type: text/html; charset=utf-8
<

				             <title>Cloud Foundry Firehose Exporter</title>

Cloud Foundry Firehose Exporter

Metrics

* Connection #0 to host <IP_ADDRESS> left intact
				             ```

Hi @lodener,
thank you for the issue. I fell over it as well and created a fix already. Currently waiting for it to get approved to make it available via the prometheus-boshrelease

Hi @lodener
issue should be fixed with version v7.1.1 which is also included in the latest prometheus bosh-release

Thanks @benjaminguttmann-avtq, 7.1.1 solved the issue.


readme file for new rtd module akamaiDapRtdProvider

We don't merge the docs until code is released. That should happen tomorrow.


operator cert-manager (1.11.4-rc1)

Signed-off-by: Richard Wall<EMAIL_ADDRESS>

/merge possible


Gh 1408 accumulo rest

Codecov Report

Merging #1409 into develop will increase coverage by <.01%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #1409      +/-   ##
=============================================
+ Coverage      65.74%   65.75%   +<.01%
- Complexity      1643     1644       +1
=============================================
  Files            706      706
  Lines          23062    23062
  Branches        2923     2923
=============================================
+ Hits           15163    15165       +2
+ Misses          6712     6711       -1
+ Partials        1187     1186       -1

Impacted Files
Coverage Δ
Complexity Δ

...onutil/iterable/LimitedInMemorySortedIterable.java
50% <0%> (+2.77%)
16% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Stream taking too long to repond

I'm on latest python version 3.11.2 and latest pytubefix 1.6.0. The issue is that yt.streams takes around 20-30 seconds to respond. My internet connection is also good. The same issue was in pytube original library. Please fix it.

Estou na versão mais recente do python 3.11.2 e no pytubefix 1.6.0 mais recente. O problema é que yt.streams leva cerca de 20 a 30 segundos para responder. Minha conexão com a internet também é boa. O mesmo problema ocorreu na biblioteca original do pytube. Por favor, corrija isso.

Hi, give us more details

I'm on latest python version 3.11.2 and latest pytubefix 1.6.0. The issue is that yt.streams takes around 20-30 seconds to respond. My internet connection is also good. The same issue was in pytube original library. Please fix it.

Hi, give us more details

show me the code you used

i just use a simple code.
import time
from pytubefix import YouTube
t0 = time.time()
yt = YouTube('https://youtu.be/wRFVlapABI0?si=Xj8ddlF6rE9Ef16-')
print(yt.streams)
t1= time.time()
print('exexution time: {}'.format(t1-t0))
On Sun, 15 Oct 2023, 5:56 pm Juan Bindez, @.***> wrote:

show me the code you used
—
Reply to this email directly, view it on GitHub
https://github.com/JuanBindez/pytubefix/issues/8#issuecomment-1763381454,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6XQ7ZBLBLV224I4GFS45WLX7PMRRANCNFSM6AAAAAA6A5N53Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

i just use a simple code. import time from pytubefix import YouTube t0 = time.time() yt = YouTube('https://youtu.be/wRFVlapABI0?si=Xj8ddlF6rE9Ef16-') print(yt.streams) t1= time.time() print('exexution time: {}'.format(t1-t0))
…
On Sun, 15 Oct 2023, 5:56 pm Juan Bindez, @.> wrote: show me the code you used — Reply to this email directly, view it on GitHub <#8 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6XQ7ZBLBLV224I4GFS45WLX7PMRRANCNFSM6AAAAAA6A5N53Y . You are receiving this because you authored the thread.Message ID: @.>

here and apparently it's normal ->
sive="False" type="audio">, <Stream: itag="140" mime_type="audio/mp4" abr="128kbps" acodec="mp4a.40.2" progressive="False" type="audio">, <Stream: itag="249" mime_type="audio/webm" abr="50kbps" acodec="opus" progressive="False" type="audio">, <Stream: itag="250" mime_type="audio/webm" abr="70kbps" acodec="opus" progressive="False" type="audio">, <Stream: itag="251" mime_type="audio/webm" abr="160kbps" acodec="opus" progressive="False" type="audio">, <Stream: itag="599" mime_type="audio/mp4" abr="None" acodec="mp4a.40.5" progressive="False" type="audio">, <Stream: itag="600" mime_type="audio/webm" abr="None" acodec="opus" progressive="False" type="audio">]
exexution time: 3.106990098953247

try to trace a route to google dns to see if there is any loss along the way

i do not understand your response. Kindly make it simpler for me as I'm not
a professional. Also suggest solutions for or changes in codes or
respective settrings. Thanks
On Sun, 15 Oct 2023, 6:28 pm Juan Bindez, @.***> wrote:

try to trace a route to google dns to see if there is any loss along the
way
—
Reply to this email directly, view it on GitHub
https://github.com/JuanBindez/pytubefix/issues/8#issuecomment-1763390218,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6XQ7ZCL5A5VDEMK7D2Y2ITX7PQIFANCNFSM6AAAAAA6A5N53Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

i just use a simple code. import time from pytubefix import YouTube t0 = time.time() yt = YouTube('https://youtu.be/wRFVlapABI0?si=Xj8ddlF6rE9Ef16-') print(yt.streams) t1= time.time() print('exexution time: {}'.format(t1-t0))
…
On Sun, 15 Oct 2023, 5:56 pm Juan Bindez, @.> wrote: show me the code you used — Reply to this email directly, view it on GitHub <#8 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6XQ7ZBLBLV224I4GFS45WLX7PMRRANCNFSM6AAAAAA6A5N53Y . You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
here and apparently it's normal ->
sive="False" type="audio">, <Stream: itag="140" mime_type="audio/mp4" abr="128kbps" acodec="mp4a.40.2" progressive="False" type="audio">, <Stream: itag="249" mime_type="audio/webm" abr="50kbps" acodec="opus" progressive="False" type="audio">, <Stream: itag="250" mime_type="audio/webm" abr="70kbps" acodec="opus" progressive="False" type="audio">, <Stream: itag="251" mime_type="audio/webm" abr="160kbps" acodec="opus" progressive="False" type="audio">, <Stream: itag="599" mime_type="audio/mp4" abr="None" acodec="mp4a.40.5" progressive="False" type="audio">, <Stream: itag="600" mime_type="audio/webm" abr="None" acodec="opus" progressive="False" type="audio">] exexution time: 3.106990098953247

video">, <Stream: itag="160" mime_type="video/mp4" res="144p" fps="30fps" vcodec="avc1.4d400c" progressive="False" type="video">, <Stream: itag="278" mime_type="video/webm" res="144p" fps="30fps" vcodec="vp9" progressive="False" type="video">, <Stream: itag="597" mime_type="video/mp4" res="None" fps="15fps" vcodec="avc1.4d400b" progressive="False" type="video">, <Stream: itag="598" mime_type="video/webm" res="None" fps="15fps" vcodec="vp9" progressive="False" type="video">, <Stream: itag="139" mime_type="audio/mp4" abr="48kbps" acodec="mp4a.40.5" progressive="False" type="audio">, <Stream: itag="140" mime_type="audio/mp4" abr="128kbps" acodec="mp4a.40.2" progressive="False" type="audio">, <Stream: itag="249" mime_type="audio/webm" abr="50kbps" acodec="opus" progressive="False" type="audio">, <Stream: itag="250" mime_type="audio/webm" abr="70kbps" acodec="opus" progressive="False" type="audio">, <Stream: itag="251" mime_type="audio/webm" abr="160kbps" acodec="opus" progressive="False" type="audio">, <Stream: itag="599" mime_type="audio/mp4" abr="None" acodec="mp4a.40.5" progressive="False" type="audio">, <Stream: itag="600" mime_type="audio/webm" abr="None" acodec="opus" progressive="False" type="audio">]
exexution time: 2.73478102684021

i do not understand your response. Kindly make it simpler for me as I'm not a professional. Also suggest solutions for or changes in codes or respective settrings. Thanks
…
On Sun, 15 Oct 2023, 6:28 pm Juan Bindez, @.> wrote: try to trace a route to google dns to see if there is any loss along the way — Reply to this email directly, view it on GitHub <#8 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6XQ7ZCL5A5VDEMK7D2Y2ITX7PQIFANCNFSM6AAAAAA6A5N53Y . You are receiving this because you authored the thread.Message ID: @.>

To simplify for you, no problem was identified in the library


Fix camera info in depth camera

🎉 New feature
Summary
depth camera info is not updated based on the image size.
Maybe we can do some refactor to some of the copy-paste methods. @iche033 thoughts ? or maybe in a follow up PR
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

close in favour of this other one https://github.com/gazebosim/gz-sensors/pull/486


Sonify the wordle

Is your feature request related to a problem?
Allow people to 1. explore the wordle by audio; 2. navigate the wordle by pointer, mouse hover or tab focus.
The size of the word could be expressed by volume, pitch or speed. Which one to use needs to be determined.

Resolved with the merge of https://github.com/inclusive-design/inverted-wordles/pull/18 and https://github.com/inclusive-design/inverted-wordles/pull/19


Please add statement manually delete appdata folder is  unsupported scenario in "Non-persistent setup"

[Enter feedback here]

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

ID: 17880b9d-6b98-3194-cc33-fe8067ce241e
Version Independent ID: 643bcf2f-507b-e449-73db-923d8a368ad9
Content: Teams for Virtualized Desktop Infrastructure - Microsoft Teams
Content Source: Teams/teams-for-vdi.md
Service: msteams
GitHub Login: @serdars
Microsoft Alias: v-mahoffman

Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hello, @scanum , Team.
Could you let us know ETA and accelelate of update.
As I describe below Business Impact, This update is indispensable to explain customer.
For issues that we are currently addressing, it will be a description necessary to prove to the customer that it is not an issue with Teams and customer is using an unsupported scenario.
Business Impact:
Issue is occurring S500 customer, and about 40,000 users are affected this issue.
Customer's CIO and other executive is concerned this issue.
Thank you!

Hi @myuge001, thank you for your feedback.
Could you kindly share the content of the screenshot in text format, please?

If you have a Microsoft alias, you can link your accounts in GitHub so that a blue "Microsoft submitter" label appears to any PR's or issue you submit. Guidance on it here: https://docs.opensource.microsoft.com/tools/github/index.html
Here is the site you can use to see the mappings between GitHub and Microsoft aliases: https://repos.opensource.microsoft.com/people?q=myuge001

hi, @dariomws
I'm sorry to see a slow reply.I will share text format.
And I have Microfsoft alias (myuge) , so I will check guidanse you shared.
Customer is using Non-Persistent environment especially Volatile client machine and Teams desktop app cannot start after rebooting PC and it will show "We are sorry-we've run into an issue"
The investigation found that manually delete the following cache folder is an unsupported scenario.
C:\Users\username\AppData\Local\Microsoft\IdentityCache (%localAppdata%\Microsoft\IdentityCache)
　C:\Users\username\AppData\Roaming\Microsoft\Teams (%appdata%\Microsoft\Teams)
IcM: https://portal.microsofticm.com/imp/v3/incidents/details/279259445/home
Ensure that customers understand exactly that they are out of support, could you add below statement on this article.
-statement
In Volatile Client Machine and Non-Persistent Setup Machine, mannualy deleting Teams Cache stored in the followings locations is NOT supported due to resulting in Broken State in Teams. The only supported scenarios is to sign-out Teams and to uninstall Teams Client using Windows Installer afterward.
C:\Users\username\AppData\Local\Microsoft\IdentityCache (%localAppdata%\Microsoft\IdentityCache)
C:\Users\username\AppData\Roaming\Microsoft\Teams (%appdata%\Microsoft\Teams)

This issue can be closed after PR #8482 is merged.


jdbc-sniffer should NEVER break existing code

Fixed in https://github.com/bedrin/jdbc-sniffer/pull/94
Fixed in 2.3.5


how to use template in execute-command?

From https://github.com/adnanh/webhook/blob/master/docs/Hook-Examples.md#json-array-payload, I see there is a notation "execute-command": "{{ .Hookecho }}". I want to ask if this {{ .Hookecho }} notation is valid? Where do I specify this Hookecho?

https://github.com/adnanh/webhook/blob/f89b09bef6d8d4d9d27728265fea1166aecc0fbd/internal/hook/hook.go#L769
	if asTemplate {
		funcMap := template.FuncMap{"getenv": getenv}

		tmpl, err := template.New("hooks").Funcs(funcMap).Parse(string(file))
		if err != nil {
			return err
		}

		var buf bytes.Buffer

		err = tmpl.Execute(&buf, nil)
		if err != nil {
			return err
		}

		file = buf.Bytes()
	}

Because of tmpl.Execute(&buf, nil), it seems that {{ .Hookecho }} cannot be used, right?

Correct. Hookecho is a template function used in the unit tests. It's not available in the production binary.

Correct. Hookecho is a template function used in the unit tests. It's not available in the production binary.

Thank you for your reply, I understand now.


explain plugins as part of user onboarding

Plugins are one of the most valuable capabilities offered by Amplication as they allow you to quickly upgrade your services with additional capabilities such as authentication, database providers, Kafka message broker, helm chart, and many more.
I think we should create a short page to explain them and how they contribute to you as a developer working with Amplication.
this page should be presented as part of the "introduction" section. From there we can send the user to read more thoroughly about all available plugins

This issue is taken care of in PR #389.


couchddb is not working through public ip.

Hi Team,
I installed couchdb successully in ubuntu(AWS). After that i change defult.ini and local.ini files
<IP_ADDRESS> to <IP_ADDRESS>
Then couchdb is connected through public ip yesterday. now today i am checking its not connected. what is the issue. Could you please help me this issue

@sbcsuman
Hi there,
This is not a CouchDB bug. GitHub is for actual CouchDB bugs only.
If you are looking for general support with using CouchDB, please try one of these other options:

The user mailing list. Signup instructions are here
The Slack/IRC chat room. Joining instructions are here

did you ever solve this issue ?

did you ever solve this issue ?

Hi @tewedaj, this is an old issue and I can only repeat what Joan already said.

If you are looking for general support with using CouchDB, please try one of these other options:
The user mailing list. Signup instructions are here
The Slack chat room. Joining instructions are here

If you have support questions, you know now what to do next, if you found a bug, create a (more detailed) bug report please.


[SPARK-37559][SQL] ShuffledRowRDD get preferred locations order by reduce size

What changes were proposed in this pull request?

Pass MapOutputStatistics to ShuffledRowRDD in AQE code path so we can do sort according to the origin reduce partition size.
Why are the changes needed?

The coalesced partition can contain several reduce partitions. The preferred locations of the RDD partition should be the biggest reduce partition before coalesced.
So it can get a better data locality and reduce the network traffic.
Does this PR introduce any user-facing change?

no
How was this patch tested?

Added test

Test build #145953 has started for PR 34820 at commit 64890e2.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50428/

cc @cloud-fan @maryannxue

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50428/

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

Test build #145953 has finished for PR 34820 at commit 64890e2.

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

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

Any real numbers for the added complexity?

@maryannxue CoalescedPartitionSpec only cares about the reduce partition so the max length need to sort is the shuffle partition number, so I think the added complexity is no harm.
To be conservative, I do a benchmark using the added complexity with 100000 shuffle partitions:
val shufflePartitions = 100000
val rand = new Random(0)
val bytesByPartitionId = Seq.tabulate(shufflePartitions)(i => rand.nextLong().abs).toArray

var previous = 0
var next = 0
val partitions = new ArrayBuffer[(Int, Int)]()
while (next < shufflePartitions) {
  next = next + rand.nextInt(100)
  partitions.append((previous, next.min(shufflePartitions - 1)))
  previous = next
}

val start = System.nanoTime()
partitions.foreach { case (start, end) =>
  (start until end)
    .map(index => (index, bytesByPartitionId(index)))
    .sortBy(_._2)(implicitly[Ordering[Long]].reverse)
}
println((System.nanoTime() - start) / 1000000)

It takes 73ms.

@ulysses-you My question was: did we see any real benefit from this change in the benchmark? Other than the extra computation, the "added complexity" also means the future code maintenance effort. If we can't prove any real benefit, we should keep the logic simple.

thank you @maryannxue for the explanation, I see your point now.
It's really hard to give the number of this change. In our production, the advisoryPartitionSizeInBytes value of writing job's last stage is always bigger than the default value, e.g. 512MB. The origin partitions before coalesced can also be big and so are the blocks. So I think it's more likely to take the benifits. But yes, I cann't say the number here..


Extraction of software components from chef cookbooks

Added a project that can extract the software components from chef cookbooks. The chef cookbooks can be crawled from the Chef Supermarket. The extracted components from the chef cookbooks can be retrieved as tosca model.
Signed-off-by: Markus Schuetterle<EMAIL_ADDRESS>

[x] Ensure that you followed http://eclipse.github.io/winery/dev/ToolChain#github---prepare-pull-request. Especially, we require a single commit
[x] Ensure that the commit message is a good commit message
[x] Ensure to use auto format in all files
[x] Ensure that you appear in NOTICE at Copyright Holders
[x] Tests created for changes
[x] Documentation updated (if needed)
[x] Screenshots added (for UI changes)

Can one of the admins verify this patch?

Filed 4ad7a88 as CQ 20626.


Q: Changes to the rules by other programs

WindowsFirewallHelper caches the underlying COM object and since Windows Firewall COM API does no reflects updates fast enough you can, in theory, get a result that is outdated. How rarely this happens depends on the underlying Firewall API and therefore it might be hard to reproduce.
#32 might be used to detect such changes in the future.
Workaround
Since with version 2 of this library, it is possible to create multiple instances of any IFirewall implemented class, creating a new instance should, in theory, solve this problem by creating a new COM object.
Vista+:
var rules = new FirewallWAS().Rules.ToArray()
XP+:
var rules = new FirewallLegacy().Rules.ToArray()
Doing so leaves the COM object lifetime management to the user of the library.
There is no way to do this with version 1 of this library since these constructors are marked as internal.
Solution
Should add a new IFirewall.Refresh() or IFirewall.Reload() method to clear the COM cache of an instance. This is especially handy when using the FirewallManager class.

https://github.com/microsoft/AttackSurfaceAnalyzer/issues/306 might also be related to this. Not sure tho.

IFirewall.Reload() added


Pagination

What kind of a gem does not facilitate a pagination function -
What good is just returning 50 followers - and also why doesn't cursor work? -
Looks like the documentation is not in sync with gem
Wasted time to get this working
Need it for a product demo in 24 hours
Many Thanks ;-)

thank you s2t2 great code!

You can't get pagination info for the return value of user_recent_media by default any more. You have to enable no_response_wrapper configuration property. See https://github.com/Instagram/instagram-ruby-gem/pull/97


修正了s_module_name初始化和asan使用delete还是free的问题

ZLMediaKit/ZLMediaKit#3181

 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.


Remove tab

Are the changes on yarn.lock expected ?
There are new dependencies on repo looking like personal accounts:

@aashutoshrathi/word-wrap
@nicolo-ribaudo/semver-v6`

Are the changes on yarn.lock expected ? There are new dependencies on repo looking like personal accounts:

@aashutoshrathi/word-wrap
@nicolo-ribaudo/semver-v6`

Idk, I did not touch on anything in the package.json.


Rollup for snapshot/dist/exports.ts

I need to ensure rollup excludes some file-global eslint rules.


MultiRTC in server error

Hi everybody!
I have been working on MultiRTC, and it works perfectly well when i do on localhost, but when i try to do on real server, i have this error:
"Redirect at origin 'http://cdn.webrtc-experiment.com' has been blocked from loading by Cross-Origin Resource Sharing policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://abeltxu.bellatrix.uberspace.de' is therefore not allowed access."
I have just replace this in ui.main.js
// use "channel" as sessionid or use custom sessionid!
var roomid = connection.channel;
var SIGNALING_SERVER = 'ws://bellatrix.uberspace.de:63231';
var websocket = new WebSocket(SIGNALING_SERVER);
websocket.onmessage = function (event) {
var data = JSON.parse(event.data);
if (data.isChannelPresent == false) {
    connection.open();
} else {
    connection.join(roomid);
}

};
websocket.onopen = function () {
websocket.send(JSON.stringify({
checkPresence: true,
channel: roomid
}));
};
And this in ui.peer-connection.js
var SIGNALING_SERVER = ws://bellatrix.uberspace.de:63231';
connection.openSignalingChannel = function(config) {
config.channel = config.channel || this.channel;
var websocket = new WebSocket(SIGNALING_SERVER);
websocket.channel = config.channel;
websocket.onopen = function() {
websocket.push(JSON.stringify({
open: true,
channel: config.channel
}));
if (config.callback)
config.callback(websocket);
};
websocket.onmessage = function(event) {
config.onmessage(JSON.parse(event.data));
};
websocket.push = websocket.send;
websocket.send = function(data) {
websocket.push(JSON.stringify({
data: data,
channel: config.channel
}));
};
}
Of course, i have change port too in signaling.js to 63231
Thanks so much!

Redirect at origin 'http://cdn.webrtc-experiment.com' has been blocked from loading by Cross-Origin Resource Sharing policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. I face same problem

MultiRTC demos are open-sourced here:

https://github.com/muaz-khan/MultiRTC

You can replace scripts from this HTML file:
<script src="https://raw.githubusercontent.com/muaz-khan/RTCMultiConnection/master/RTCMultiConnection.js"></script>
<script src="https://cdn.firebase.com/js/client/2.2.9/firebase.js"></script>

You can even use this but it is not up-to-dated:
<!-- Use this RTCMultiConnection.js instead -->
<script src="http://www.rtcmulticonnection.org/latest.js"></script>

or rawgit.com link for up-to-dated script:
<script src="https://cdn.rawgit.com/muaz-khan/RTCMultiConnection/master/RTCMultiConnection.js"></script>

Or, this one is always up-to-dated but it is NOT recommended BECAUSE server is running on a shared hosting and it gets down often:
<!-- i.e. using "WWW" instead of "CDN" -->
<script src="//www.webrtc-experiment.com/RTCMultiConnection.js"> </script>

or minified file:
<!-- i.e. using "WWW" instead of "CDN" -->
<script src="//www.webrtc-experiment.com/RTCMultiConnection.min.js"> </script>

Hi everybody!
I try to work with MultiRTC, on localhost, i have this error:
Redirect at origin 'http://cdn.webrtc-experiment.com' has been blocked from loading by Cross-Origin Resource Sharing policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost' is therefore not allowed access.


Update SlackNotificationsCore.php to restore old size diff behaviour

this restores the old size diff behaviour previously lost due to https://github.com/kulttuuri/SlackNotifications/commit/0d32dbc2fb137b1106cc495cdb164bb2b0c010b6

Thank you!


Remove the withChannelName() method from QueueInterface

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
✔

This PR removes the QueueInterface::withChannelName() method, which is ambiguous: it doesn't change an actual channel name. Since this PR the Queue just proxy channel name from its adapter for the getChannelName() method.

Looks :+1: Please update/merge it.