[FEATURE] Add more flexibilty in construction of the SelfInstructTask template

Is your feature request related to a problem? Please describe.
When using Distilabel recently, I wanted to modify the  # Criteria for Queries part of the task template. Since this isn't currently exposed via the SelfInstructTask constructor I ended up patching  self-instruct.jinja2 directly.
Describe the solution you'd like
It might be nice to also expose this part of the template for the user to modify?
Describe alternatives you've considered
At some point, the tasks might be modified so that they don't have much similarity to the original paper. I don't think this is such a problem but another possible approach would be to to add more functionality or documentation for constructing custom tasks.

@davanstrien, we will wait for @alvarobartt and @gabrielmbmb to return from holidays to discuss the more structural change. For now, we will add some documentation to provide this overview.
https://github.com/argilla-io/distilabel/issues/256


Write a C program to calculate compound interest

Description
Write a C program to calculate compound interest
Compound interest = principal amount * (1 + rate of interest) ^ time
Principal : 1000
Rate      : 10
Time      : 5

Compound interest : 1610.51

How to contribute

Save the solution in CalculateCompoundInterest.c file
Add CalculateCompoundInterest.c file in calculate-compound-interest folder

!assign


Add reusable GHA workflow to allow for CI checks

Feature Request

Q
A

New Feature
yes

RFC
no

BC Break
no

Summary

In order to allow renovate to properly rely on CI, we need to add the reusable GHA workflow to this repository

Related: https://github.com/laminas/laminas-cache-storage-adapter-zend-server/issues/12


Suppress Warnings from LSP Platform Changes

Putting this up to unblock builds. Alternatives for the types made obsolete are under discussion and not yet available.
Revert Issue: https://github.com/dotnet/aspnetcore/issues/31133

/azp run


Update Issue - Automate_undefined_2777

URL tested: https://accounts.google.com/v3/signin/identifier?continue=https%3A%2F%2Fmail.google.com%2Fmail%2Fu%2F0%2F&emr=1&followup=https%3A%2F%2Fmail.google.com%2Fmail%2Fu%2F0%2F&ifkv=AcMMx-dPygYL6_C58NSq9HLs0FWaXxysLwHMbP8PudPdjNHPURpSAUxAfAeOTodYAMKBwlkVkzSFsA&osid=1&passive=1209600&service=mail&flowName=GlifWebSignIn&flowEntry=ServiceLogin&dsh=S-855624162%3A1732575447506143&ddm=1
Open URL on Browserstack
default description

Property
Value

Browser
Chrome 131.0

Operating System
OS X Sonoma

Resolution
1688x784

Screenshot Attached
Screenshot URL

Updated Screenshot Attached


Rust collector can't produce SBOMs

Steps to reproduce:
❯ git clone  https://github.com/cloudflare/quiche
Cloning into 'quiche'...
remote: Enumerating objects: 15771, done.
remote: Counting objects: 100% (259/259), done.
remote: Compressing objects: 100% (150/150), done.
remote: Total 15771 (delta 168), reused 165 (delta 102), pack-reused 15512
Receiving objects: 100% (15771/15771), 17.27 MiB | 4.82 MiB/s, done.
Resolving deltas: 100% (8888/8888), done.

/tmp via 🐘 took 5s
❯ cd quiche

quiche on  master via 🦀
❯ ls
CODEOWNERS   COPYING      Cargo.toml   Dockerfile   Makefile     README.md    apps         clippy.toml  fuzz         nginx        octets       qlog         quiche       quiche.svg   rustfmt.toml tools

quiche on  master via 🦀
❯ cdxgen --t rust
Unable to produce BOM for .
Try running the command with -t <type> or -r argument

@UgniusV if you use -r for recursive it works fine. The project is organized into workspaces with multiple sub-directories.

Thanks ❤


Issue copying and pasting coordinates

If I input a location any way and then input a new location by typing or pasting the first location that is flashed is the last location. You must hit enter one more time and then the new location is highlighted.
This is a new issues since yesterday:
@nfeuerstein @elinz

For example, I click on Coordinate Conversion tool.

My map is currently centered on the US.
In the Input box, type the coordinates 45N 120E and hit return. The map is not panned to the coordinates 45N 120E.  I have to click back into the Input textbox and hit return again for the map to to pan and center on this coordinates and the flash and the point are drawn on the map.
Now type the coordinates 45S 120W in the Input textbox and hit return. The flash and point are drawn on the map, but it's located on the previous entered coordinate of 45N 120E. You have to click in the Input textbox and hit return again for the map to pan and the flash and point to draw on the map on coordinate that's currently in the Input textbox.

It does not matter if the point is on the screen or not it always flashes the last location

The point is you always have to hit return twice in the Input textbox for the flash/point to be drawn in the correct location.

@elinz correct. I have also tested this at the single add-in level and the component level.

@topowright Added a fix for this issue in this PR. Can you confirm the fix when you get a chance?

@kgonzago you are awesome. I have verified this fix.


Meta Bug - Communiation

Problem
This concerns the issue #13350 and to a lesser extend #1734 and the communication between users (me in this case) and project members (@epage) but it is a distinct issue from both my comment on the original technical issue and the subsequent issue I opened. Please bare with me for a moment. I'm not trying to escalate a conflict or to get the last word.
I fully expect this issue to be locked as were the two others and I would not be surprised if as a result of posting this issue here, I might be banned from the project or github, even though I don't really know the mechanism. So whatever happens to this issue and my account, I will not open any more tickets in this project or interact with it, unless or until this issue here is resolved amicably.
I see my part or my fault in this procedure at least insofar as I could have chosen a more conciliatory tone from the start. I don't believe that anything I said is abusive or inappropriate though. I chose that strong language I used because the road of polite conversation has been traveled and did not lead anywhere. If I am misjudging the situation, I am perfectly willing to correct my view and sincerely apologize. However, I don't see that yet.
So far, I disagree both with the necessity and the proportionality of locking the original issue #1734, but this is just an opinion and mine is no better than that of @epage, and this is your project, not mine.
When I opened the followup issue, I selected "feature request" as an issue type. That means that I request something I see as an improvement. It's not a demand. It was a suggestion. And that was not about the original technical issue - ~/.cargo it was about a much broader issue that is social in nature. I do not expect you to agree with me, to be convinced or to argue with me. I offered a suggestion, no more than that. This was not an attempt to start a war, I wanted to explain where I am coming from, what I believe the implications of the original issue are and why I believe this is important.

I locked it for social reasons, not technical, as that conversation was going in the wrong direction.

That raises the question what exactly "wrong" and "right" means in this context. On one hand, there is your project with it's agenda, the interests of rust developers, project members, employees, on the other hand there are people affected by your decisions. And then there are transitive effects that your decision have, such as "if even major tools like cargo don't respect XDG conventions, they can savely be ignored". Can you really so easily classify that direction as wrong?

As this violates the spirit of that locking

In my original comment I said that cargo violates user trust to which this seems to be a response. I said more than just a phrase, I wrote pages of text explaining exactly what I meant by that violation of trust. I think I made a good point, but that's usually what people think about their own points.
But I cannot violate the spirit of locking, because I never agreed to subscribing to a spirit of censorship. And I have a hard time interpreting this locking as anything else, because I really don't see that I crossed a line with anything I said until that point.

I am going to lock this as well until we feel it is time to open things back up.

This happened seconds after I posted a very long issue, that does not contain strong language at all. There was definitely no time for @epage to read that text, not even speed reading it. It's hard to take that as anything but personal.

I understand you don't see yourself as a Rust user and so its surprising when Rust related things get in your way.

I know for a fact that I am not a Rust user. This is not a matter of perspective.

This will be a continued problem we need to work on improving

You understand that the effect of "Trust us" is limited, when I already mentioned that I perceive your attitude as a violation of trust, do you? This happens in a context of an issue that was opened nine years ago. So asking for patience and trust sounds like politics here.

I don't want to say too much more because I don't want this viewed as me "getting in the last word" before locking

If you don't want to be seen as wanting the last word, you might want to reconsider the communication tools you are using here. This is the rhetorical equivalent of a sledge hammer.
I would not have opened the second ticket (#13350), if you had a means of private communication published that would not require me to register an account on a service I would not otherwise use. If the original issue would not have been locked, I would have clarified a few points and I would have apologized to you personally and the team as a whole for the roughness of my language, not because I believe that anything I said is factually wrong, but simply because it was unnecessarily rough. I could have and better should have tampered down the tone and said what I wanted to say.

but I want to make sure you understand that I understand the problem

You achieved the opposite. You made sure that I understand that you misunderstood my real issue. I understood from your early responses that you are advocating for XDG compliance in the team. I understood that, before I wrote my comment and before your replies and the locking. I mentioned you personally, because what you said over emphasized the 9 year period. I tried to make clear that I do not criticize you by saying something like "what epage said is reasonable", that was not clear enough though.
My problem is, that if a tool like cargo refuses to comply for XDG for 9 years, this has a damaging effect far beyond the annoyance of me having to delete a .cargo folder. That is not and never was the point.

even if not the degree you feel about it.

It's a qualitative not a quantitative issue. Is it my home directory or is it yours? Referencing the issue you didn't read before you locked it: Is it my privacy or <insert your favorite coorp/project>'s?
It's just not up to you to decide where and what to write in peoples home directories, unless they can reasonably expect you to do that.
Summary
I find the locking habits both counterproductive and abusive, knowing that this is your project and you can choose whatever policy you like. But is this really whom you want to be?
As promised, unless you invite another response from me, I'm off.
Steps
No response
Possible Solution(s)
No response
Notes
No response
Version
No response

I'm closing this to reduce visibility, because this is not really about publicity. Just wanted you to see that response, @epage.


bump version to 13-latest

The AST changed between 10 and 13, so this would be a breaking change.

Hi Christopher,
Thanks for this PR.
This is an interesting case for semantic versioning. I'm wondering if it'd make sense to switch this crate to a system that mirrors the Pg version and libpg_query version that it's linking against. Of course, the libpg_query version doesn't fit in semantic versioning, since it's currently 13-2.1.0. Alternatively, we could use the relevant Pg version (13.2) and use the patchlevel to indicate something about this crate or the libpg_query version.
Do you have thoughts about this? @tdbgamer, if you have any ideas on this I'd love to get your input too!
Cheers,
Dave

Hmm I think generally in this scenario, most libraries I've seen tend to do their own semantic versioning independently and just throw a table in the README mapping versions of the wrapper lib to the underlying lib.
Overall I'm not super opinionated on how this is done. This is the sort of thing where I expect most consumers would understand they need to build an abstraction layer over this to protect themselves from changes made by postgresql anyways.

Hmm I think generally in this scenario, most libraries I've seen tend to do their own semantic versioning independently and just throw a table in the README mapping versions of the wrapper lib to the underlying lib.
Overall I'm not super opinionated on how this is done. This is the sort of thing where I expect most consumers would understand they need to build an abstraction layer over this to protect themselves from changes made by postgresql anyways.

I think in this case it'd make sense to bump the version to 1.0.0 for this change, and then bump the major version whenever the associated Pg major version changes. And for any other releases we'd bump the minor or patchlevel versions and update the table in the README.md as needed if there's an associated Pg version bump.

I think in this case it'd make sense to bump the version to 1.0.0 for this change, and then bump the major version whenever the associated Pg major version changes. And for any other releases we'd bump the minor or patchlevel versions and update the table in the README.md as needed if there's an associated Pg version bump.

Actually, I think I'll just make this version 0.2.0. Version 0 is unstable anyway, and I'm a bit leary of bumping this to 1.0.0 without knowing if it's actually stable.

Thanks for this! I merged it with some tweaks from the command line.


[ALLUXIO-1600] update docs for the exceptions

https://tachyon.atlassian.net/browse/ALLUXIO-1600

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8980/
Test PASSed.

Thanks for the PR @shimingfei, I've left a comment.

@shimingfei any update on this PR?

@yupeng9
Sorry for late reply, I have updated the PR, Thanks!

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9069/
Test PASSed.

LGTM


Sass -> Scss

How can I switch to scss easly?
Thanks.

You should be able to just start using .scss immediately. It's configured to handle both extensions! 🕺


Add a circleci config for testing

@brianhelba This will need someone with admin access to enable CI integraiton.

Closes #2


Enable creation of arbitrary contexts (alternative module IDs)

I'm submitting a feature request
Webpack version:
2.1
Expected/desired behavior:
Currently it's possible to either load modules by their ID's (e.g. __webpack_require__(123) or create a custom context (require.context(...)) which maps paths to module IDs.
It would be great to have a way to create arbitrary "contexts" of modules which were previously in the bundle. What I mean by that is creating a simple mapping to resolve arbitrary names (set by the plugin) to actual module IDs, without adding any extra ones.
I've managed to hack together something based on the ContextReplacementPlugin, but it's not a good solution, because the modules need to be added manually, plus it duplicates the modules instead of referencing the ones which are already there. I want to add new modules, just be able to access them using an "alternative module ID", stored in the context map.
I'd be happy to implement this myself, as a plugin, but I'm not sure what's the best way - the ContextModuleFactory is quite complex. It seems to me, I'd need to duplicate that and just make it much simpler -- unless there's a better way?

What is the motivation / use case for changing the behavior?
Some applications need to use arbitrary, dynamically generated names for importing modules. This would enable such use cases, given that the names are predictable (e.g. file-paths).

const ctxRequire = require.customContext('my-context-name');
const someModule = ctxRequire('my-custom-reference');
const someOtherModule = ctxRequire(dynamicallyGeneratedName);

In other words it's like alternative module IDs. I could just hook 'before-module-ids' and change the module.id of each module and then use __webpack_require__ directly, but that way I'm limiting ourselves to a single set of custom references, while it might make sense to keep multiple aliases for the same module, e.g. with and without the extension, similarly to how the ContextReplacementPlugin works:
Given the following Context Map:
{
    "my-custom-reference": 123,
    "my-custom-reference.js": 123
}

Usage:
const ctxRequire = require.customContext('my-context-name');
const someModule = ctxRequire('my-custom-reference');    // without extension
const sameModule = ctxRequire('my-custom-reference.js'); // with extension

For adding names I'd imagine the consumer of such plugin could either:

create a simple loader that doesn't modify the source, but adds the array of aliases for a given context
use a Compilation plugin hook like 'normal-module-loader' to create and add the array of aliases given each module

This still would be nice to have. Perhaps with a different (rethought) API to the one originally proposed.


Write a Kotlin program to print pattern 1

Description
Write a Kotlin program to print pattern 1
Input  : 5

Output :
1
1 2
1 2 3
1 2 3 4
1 2 3 4 5

Contributed by - @PraaneshSelvaraj

How to contribute

Save the solution in program/PrintPattern1.kt file
Add PrintPattern1.kt file in print-pattern-1 folder

!assign


use cchardet to autodetect text encodings

ref: https://github.com/PyYoshi/cChardet
While we're at it, it probably makes sense to change the default output encoding to utf-8, since this seems to be what players like VLC expect.

Would this solve issues where currently Swedish å is written as "├Ñ", ä as "├ñ" and ö as "├╢"?

Hi @sfkpmr, I believe this would solve this issue but it's hard to say for sure. Do you have a test file that I could use to verify with by any chance?

Sure > https://pastebin.com/BVzA1eLD

Thank you @sfkpmr! Actually what you posted looks like utf-8, which should be handled correctly by most software (including subsync). What are the circumstances under which you see the malformed characters?

I simply ran per the instructions, subsync video.mp4 -i unsynchronized.srt > synchronized.srt, and the output looks like this:
https://pastebin.com/2M3Q8z3e

@sfkpmr gotcha. I'm having trouble reproducing this -- what version are you running? You can find out by running subsync --version

@sfkpmr actually I have a different hypothesis as to what is going on -- probably the original srt file you have is not encoded in utf8, but my guess is that it gets converted to utf8 on your system clipboard when you copy the contents of the file and paste to pastebin. If so, sorry for the wild goose chase -- I'm guessing I would need the original file (not copied+pasted to pastebin) in order to figure out whether this is an encoding issue as I suspect.

That's odd, Windows' save dialog suggests that it is using UTF-8.
I uploaded the file here.

@sfkpmr thank you so much for continuing to help troubleshoot this. As you said, it is UTF-8, so something else must be going on.
In the most recent release of subsync (which can be installed via pip install --upgrade git+https://github.com/smacke/subsync@STABLE), there is a flag --make-test-case which creates a compressed archive of all files involved for help with later debugging. (It simply fingerprints the movie to avoid copyright issues.) Example usage is:
subsync reference.mp4 -i unsynced.srt -o synced.srt --make-test-case
and it will create a file reference.mp4.tar.gz.
Would you be willing to do this for me and upload the tarball so I can take a closer look at what's going on?
Also, it's possible that the latest version doesn't have this issue anymore.

Hi @sfkpmr I'm going to go ahead and close this issue since I don't seem to see the encoding corruption issues with the latest version when running your test file. Please feel free to reopen if the latest v doesn't fix your issue (can grab with pip install --upgrade git+https://github.com/smacke/subsync@STABLE).


Question: I'm not seeing the Closing event in the Window component.

Testing Project Reunion 0.5, I'm not able to stop the closing of my application because there is not a Closing event. May be i'm missing something or is just this Window event is not yet available?
<Window
    x:Class="WinUI3Desktop.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:local="using:WinUI3Desktop"
    xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    Activated="Window_Activated"
    Closing="Window_Closing"  <--- Not available
    Closed="Window_Closed"
    VisibilityChanged="Window_VisibilityChanged"
    mc:Ignorable="d">

The event is not available in WinUI 3, but you can implement it by yourself using the Win32 APIs. For example:
https://github.com/marb2000/DesktopWindow

Thank you for the tip.!
Just curious, in future WinUI3 Window implementations, we will have to implement this event as an extension, as you suggested?

@marb2000 do you have thoughts on germanfortiz's question above. I'm moving a WPF app to WinUI3 and I'm missing this Closing event that WPF had. Can we expect one in the future?

@marb2000
Thanks for writing the DesktopWindow extension code.
WinUI 3 is now 1.0.2 and there is still no Closing event.
So, referring to DesktopWindow, I tried to intercept the window message and implement the close event directly, but there is a problem in the basic structure.
(It works normally in Packaged.)
When unpackaged, it does not work with the following exception.

The code below is the reproduced code.
https://github.com/dimohy/csharp-check/tree/main/WinUI3/WinUI3UnpackagedWinProcTest
If you are still working on this and know the cause, please help.

Definitally would like to see this implemented!

There is AppWindow.Closing Event

But not Window.Closing...
Window Events

But not Window.Closing... Window Events

It is the same thing : you can intercept it to prevent closing

Thanks for the solution, it runs fine!
In App.OnLaunched() add:
m_window.Closed += Window_Closed;
in Window_Closed(object sender, WindowEventArgs args)
to stop the closing of the window:
args.Handled = true;
-Exit the application from a window or a page:
Application.Current.Exit();
will execute the Window_Closed callback.

With the help of @castorix and @yvlawy, I finally solved this issue:
In the App.OnLaunched event, I added this:
  protected override void OnLaunched(Microsoft.UI.Xaml.LaunchActivatedEventArgs args)
  {
        mainWindow = new MainWindow()
        {
            Title = this.GetVersionDescription(),
        };

        mainWindow.Activate();

        // Get the handle of the created window.
        this.windowhandle = PInvoke.User32.GetActiveWindow();

        // Retrieve the WindowId that corresponds to hWnd.
        Microsoft.UI.WindowId windowId =
            Microsoft.UI.Win32Interop.GetWindowIdFromWindow(this.windowhandle);

        // Lastly, retrieve the AppWindow for the current (XAML) WinUI 3 window.
        Microsoft.UI.Windowing.AppWindow appWindow =
            Microsoft.UI.Windowing.AppWindow.GetFromWindowId(windowId);

        if (appWindow != null)
        {
            appWindow.Closing += this.AppWindow_Closing;
        }
  }

Finally, to stop the closing event, I added the following in the AppWindow_Closing function:
  private void AppWindow_Closing(Microsoft.UI.Windowing.AppWindow sender, Microsoft.UI.Windowing.AppWindowClosingEventArgs args)
  {
        if (somethingIsRunningAndTheUserCanNotCloseTheApp())
        {
              args.Cancel = true; // This prevents the application to close.
        }
  }

Thanks!


Failure to deploy Helm3 App: apiVersion v2 invalid, must be v1

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Click Cluster > Apps > Launch
Choose an app from a Helm3 repository and launch

Result:
Fails to deploy app, saying wait helm template failed. Error: apiVersion 'v2' is not valid. The value must be "v1"
Other details that may be helpful:
The log when deleting the app looks like the following, which looks like it's trying to use Helm2/Tiller to control the app
2020/03/31 21:06:39 [ERROR] AppController p-p6c5p/ocm [helm-controller] failed with : wait helm template failed. Error: apiVersion 'v2' is not valid. The value must be "v1": exit status 1
---- repeats about once a second until I delete the app -----
2020/03/31 21:06:40 [INFO] Deleting chart using helm version: rancher-helm
[main] 2020/03/31 21:06:40 Starting Tiller v2.16.3-rancher1 (tls=false)
[main] 2020/03/31 21:06:40 GRPC listening on :47294
[main] 2020/03/31 21:06:40 Probes listening on :39590
[main] 2020/03/31 21:06:40 Storage driver is ConfigMap
[main] 2020/03/31 21:06:40 Max history per release is 10
[storage] 2020/03/31 21:06:41 getting release history for "ocm"

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.4.0
Installation option (single install/HA): single install

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory): VM 4core 4GB ram
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.3", GitCommit:"06ad960bfd03b39c8310aaf92d1e7c12ce618213", GitTreeState:"clean", BuildDate:"2020-02-11T18:14:22Z", GoVersion:"go1.13.6", Comp
iler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.3", GitCommit:"06ad960bfd03b39c8310aaf92d1e7c12ce618213", GitTreeState:"clean", BuildDate:"2020-02-11T18:07:13Z", GoVersion:"go1.13.6", Comp
iler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client: Docker Engine - Community
 Version:           19.03.8
 API version:       1.40
 Go version:        go1.12.17
 Git commit:        afacb8b
 Built:             Wed Mar 11 01:27:04 2020
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.8
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.17
  Git commit:       afacb8b
  Built:            Wed Mar 11 01:25:42 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.2.13
  GitCommit:        7ad184331fa3e55e52b890ea95e65ba581ae3429
 runc:
  Version:          1.0.0-rc10
  GitCommit:        dc9208a3303feef5b3839f4323d9beb36df0a9dd
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

https://rancher.com/docs/rancher/v2.x/en/catalog/#catalog-helm-deployment-versions
"When you create a custom catalog, you will have to configure the catalog to use either Helm 2 or Helm 3. This version cannot be changed later. If the catalog is added with the wrong Helm version, it will need to be deleted and re-added."

Yeah, I deleted my old custom catalog and re-added it as Helm3, then deployed the app. After filing this, I tried nuking the catalog (which was labeled Helm3) and recreating it, and it seemed to work. I'll try to investigate a little more, having some other issues with my test cluster right now so it might be a little slow

I encountered this on an upgrade from 2.3.6 -> 2.4.2, where my helm catalog repo was migrated to a helm 2 repo (as expected) but I could no longer install new revisions of my app.

@Oats87 isn't that the opposite problem of the original issue ? OP was trying to install helm3 chart and got helm2 error. Sounds like you have helm2 chart and got helm2 error, was it same error ?

We figured it out. Real issue is that rancher's 2.3.6 helm version (2.14) didn't have the api validation, it was introduced in helm 2.15. And then Rancher 2.4's helm2 version is 2.16, so the validation is there. So the effect is that you can add a helm3 (api-v2) chart in rancher 2.3.6 and deploy with helm2, but you can't do the same in 2.4.
The current fix is to either:
a) downgrade your chart to helm2 (apiv1)
b) upgrade the catalog and its charts to helm3 (apiv2)
We don't want to remove the check, as it is the standard (upstream) helm behavior. In an ideal world it should have always been there and the 2.3.x deploy should have failed.

Will add note about this to 2.4 release and docs.

After filing this, I tried nuking the catalog (which was labeled Helm3) and recreating it, and it seemed to work.

I did that a couple of times, it seemed to work some of the time, but around half the instances of my apps i try to create in different projects fail with the v1/v2 error again because rancher apparently tries to use helm 2 instead of helm 3.
Are there known cache issues? Can it have to do with re-using names for namespaces / projects / app combinateions where previously a helm 2 app existed?
I also made sure that the v3 catalog has a different name than the v2 catalog that existed before, still Rancher tries to deploy apps with helm2 and then fails with the invalid apiVersion v2 error..
What is a good way to help diagnose this issue?

@dasMulli the apps will continue to use the helm version they were deployed with. If you delete the app and catalog then recreate both (ensuring you select helm3 from dropdown when creating catalog) it should work. If you're still seeing it at that point, check the rancher pod logs for [Installing/Deleting] chart using helm version [X]. After the install you can do kubectl get apps myApp  -n namespace and check status.helmVersion to see what it's set to.
If you're still seeing issues can you create a minimal sample repo/chart that you can reproduce with ? Then we can take a look.

Thanks, i'll try that next week.

The catalog I have was always Helm 3 based, never Helm 2. Created a fresh Helm v3 App with a Helm v3 binary and yet Rancher complains about the Chart not being v1.
Is this confirmed to be fixed in Rancher 2.4.6?

@theAkito It should be resolved in 2.4.6, but let me go ahead and test it out to see if I am seeing the same issues.
Was it a fresh install of Rancher 2.4.6 or did you upgrade from a previous rancher version?

@brendarearden
Sorry for the misunderstanding. I cannot update my cluster just for fun, so I want to only update, if it really works. I am on Rancher 2.4.5.

@theAkito the workaround is to create an app with an old version of the chart that was present in the catalog when you created it initially, then update the app to a recent version.

@dasMulli
So now I understand why it worked, when I upgraded the main app from Helm v2 to Helm v3. I basically issued the workaround without knowing it. So I'm kind of lost here with the brand new chart, that was never v2 to begin with... 🐕 💨

Due to this and other issues, I was forced to upgrade to a newer version, so I chose v2.5.0-alpha2 to be safe.
Yet, the version information says, that it is still using the exact same Helm version, as before. The error coming from this issue still persists and nothing changed...

Guys actually this issue still exists. Rancher 2.5.2 and 2.5.3
Tried to upgrade rook-ceph:
from
v1.4.7: (uses v1) https://charts.rook.io/release/rook-ceph-v1.4.7.tgz
to
v.1.5.0-1: (uses v2) https://charts.rook.io/release/rook-ceph-v1.5.1.tgz
with the same error
handler helm-controller: wait helm template failed. Error: apiVersion 'v2' is not valid. The value must be "v1"
: exit status 1, requeuing
Everywhere is Helm 3 defined but still - Tiller
2020/12/03 08:12:35 [INFO] Installing chart using helm version: rancher-helm
[main] 2020/12/03 08:12:38 Starting Tiller v2.16.8-rancher1 (tls=false)
[main] 2020/12/03 08:12:38 GRPC listening on :56147
[main] 2020/12/03 08:12:38 Probes listening on :49897
[main] 2020/12/03 08:12:38 Storage driver is ConfigMap
[main] 2020/12/03 08:12:38 Max history per release is 10
[tiller] 2020/12/03 08:12:40 getting history for release rook-ceph
[storage] 2020/12/03 08:12:40 getting release history for "rook-ceph"
[tiller] 2020/12/03 08:12:41 preparing update for rook-ceph
[storage] 2020/12/03 08:12:41 getting deployed releases from "rook-ceph" history
[storage] 2020/12/03 08:12:41 getting last revision of "rook-ceph"
[storage] 2020/12/03 08:12:41 getting release history for "rook-ceph"
[tiller] 2020/12/03 08:12:42 rendering rook-ceph chart using values
[tiller] 2020/12/03 08:12:47 creating updated release for rook-ceph
[storage] 2020/12/03 08:12:47 creating release "rook-ceph.v14"
[storage] 2020/12/03 08:12:47 getting release history for "rook-ceph"
[storage] 2020/12/03 08:12:48 getting deployed releases from "rook-ceph" history
[storage] 2020/12/03 08:12:48 deleting release "rook-ceph.v4"
[storage] 2020/12/03 08:12:48 Pruned 1 record(s) from rook-ceph with 0 error(s)
...

apiVersion: management.cattle.io/v3
kind: CatalogTemplateVersion
metadata:
  creationTimestamp: "2020-11-28T21:30:58Z"
  generation: 1
  labels:
    catalog.cattle.io/template_name: rook-release-rook-ceph
    cattle.io/creator: norman
  managedFields:
  - apiVersion: management.cattle.io/v3
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:labels:
          .: {}
          f:catalog.cattle.io/template_name: {}
          f:cattle.io/creator: {}
        f:ownerReferences: {}
      f:spec:
        .: {}
        f:digest: {}
        f:externalId: {}
        f:version: {}
        f:versionName: {}
        f:versionUrls: {}
      f:status:
        .: {}
        f:helmVersion: {}
    manager: Go-http-client
    operation: Update
    time: "2020-11-28T21:30:58Z"
  name: rook-release-rook-ceph-v1.4.7
  namespace: cattle-global-data
  ownerReferences:
  - apiVersion: management.cattle.io/v3
    kind: CatalogTemplate
    name: rook-release-rook-ceph
    uid: bbdecfdc-0492-44a0-81aa-86e8bc81013b
  resourceVersion: "82758340"
  selfLink: /apis/management.cattle.io/v3/namespaces/cattle-global-data/catalogtemplateversions/rook-release-rook-ceph-v1.4.7
  uid: 97686d7e-9f20-4117-bd9a-af1bcede7656
spec:
  digest: 108f3648608ed1407d5c8d3ed13c72b8c8f4bdae2b7708be067e3fd3bbf740cd
  externalId: catalog://?catalog=rook-release&template=rook-ceph&version=v1.4.7
  version: v1.4.7
  versionName: rook-ceph
  versionUrls:
  - https://charts.rook.io/release/rook-ceph-v1.4.7.tgz
status:
  helmVersion: helm_v3

apiVersion: management.cattle.io/v3
kind: CatalogTemplate
metadata:
  creationTimestamp: "2020-11-28T21:30:58Z"
  generation: 1
  labels:
    catalog.cattle.io/name: rook-release
    cattle.io/creator: norman
  managedFields:
  - apiVersion: management.cattle.io/v3
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:labels:
          .: {}
          f:catalog.cattle.io/name: {}
          f:cattle.io/creator: {}
      f:spec:
        .: {}
        f:catalogId: {}
        f:defaultVersion: {}
        f:description: {}
        f:displayName: {}
        f:folderName: {}
        f:icon: {}
        f:projectURL: {}
        f:versions: {}
      f:status:
        .: {}
        f:helmVersion: {}
    manager: Go-http-client
    operation: Update
    time: "2020-11-28T21:30:58Z"
  name: rook-release-rook-ceph
  namespace: cattle-global-data
  resourceVersion: "82758331"
  selfLink: /apis/management.cattle.io/v3/namespaces/cattle-global-data/catalogtemplates/rook-release-rook-ceph
  uid: bbdecfdc-0492-44a0-81aa-86e8bc81013b
spec:
  catalogId: rook-release
  defaultVersion: v1.5.1
  description: File, Block, and Object Storage Services for your Cloud-Native Environment
  displayName: rook-ceph
  folderName: rook-ceph
  icon: https://rook.io/images/rook-logo.svg
  projectURL: https://github.com/rook/rook
  versions:
  - digest: 993846d3fa7d1e25fb0e120627ec3b85b20db80a1c10d6cec30a10840760230f
    externalId: catalog://?catalog=rook-release&template=rook-ceph&version=v1.5.1
    version: v1.5.1
    versionName: rook-ceph
    versionUrls:
    - https://charts.rook.io/release/rook-ceph-v1.5.1.tgz
  - digest: 7c06929d8e5d8776fabd321c10f9aa6a7498b672c3dc8c7f34245fc1b981d548
    externalId: catalog://?catalog=rook-release&template=rook-ceph&version=v1.5.0
    version: v1.5.0
    versionName: rook-ceph
    versionUrls:
    - https://charts.rook.io/release/rook-ceph-v1.5.0.tgz
  - digest: 108f3648608ed1407d5c8d3ed13c72b8c8f4bdae2b7708be067e3fd3bbf740cd
    externalId: catalog://?catalog=rook-release&template=rook-ceph&version=v1.4.7
    version: v1.4.7
    versionName: rook-ceph
    versionUrls:
    - https://charts.rook.io/release/rook-ceph-v1.4.7.tgz
...SKIPPED a lot of old versions
status:
  helmVersion: helm_v3

Seems rancher wrongly detect what helm binary to use, still tries to use rancher-helm, which is, I beleive, Helm2.

Tried removing and re-adding rook.io catalog - no effect

@cbron This issue should be reopened, as this problem persists. I am dead in the water right now trying to spin up a new cluster because a critical infrastructure service (bitnami/rabbitmq) refuses to deploy. Tried deploying both Helm 2 and Helm 3 versions, the error is the same. There are other instances of this service running just fine in other clusters, but it refuses to deploy in the new one.
I will not remove and re-add the catalog because so many things deployed to production rely on the catalog and I can't afford any downtime if removing the catalog causes rancher to freak out. What else is there that we can do? Since apparently upgrading to the most recent version is not even enough to resolve the issue.

@rex yes, same issue trying to deploy the Ghost Blogging platform on a test cluster via the Bitnami helm chart. I can confim the issue still persists. I am using Rancher v2.5.7 with K3S (Kubernetes 1.20.5)

Also exiting here when trying to deploy extrenal-dns from Bitnami helm v3
Rancher 2.5.7

Issue still persists on v2.5.9 stable.
Is it fixed in v2.6.0?

Also facing this still

1 month, no response. So much for an enterprise kubernetes management that can't even deploy an app without breaking.

You're commenting on an issue closed 18 months ago that nobody is regularly monitoring.
2.6 (and the explorer UI in 2.5, which is now the only option) has a new "Apps & Marketplace" system that is much more directly Helm 3.  The Rancher "App" abstraction as its own type is no longer needed to provide reasonable RBAC, which native Helm 2 was totally missing.  You should be using that at this point, there is no reason to use the old apps other than maintaining existing installs.


Add highlighted list for what phase of turn it is on right of client

Currently, it is not clear what phase of the turn it is.
It would be nice if on the right side of the screen, it showed what phase of the game it was.
A list of all phases, and a box/highlight around the current phase would show the player what phase it is.

Currently, we only have a "Mulligan" phase that is used before the game begins, and then we have a "Main" phase where you can do anything. The server does not yet support multiple phases such as MtG and other games have.

But yes, it would be nice to have something like this :)

This should be reasonably easy to implement in Vue.js client.

Some turn indicator and phase name could be fixed if the server would send information about this.


Upgrade JDK17 for Sonar

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #14726 (2c1dc27) into dev (7c37a7f) will increase coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head 2c1dc27 differs from pull request most recent head ede9170. Consider uploading reports for the commit ede9170 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14726      +/-   ##
============================================
+ Coverage     38.69%   38.72%   +0.02%
+ Complexity     4625     4624       -1
============================================
  Files          1285     1285
  Lines         43957    43911      -46
  Branches       4852     4824      -28
============================================
- Hits          17011    17003       -8
+ Misses        25065    25030      -35
+ Partials       1881     1878       -3

see 32 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Support local time 8760

Time format used by NSRDB, SAM, ResStock, ComStock, dGen. Lixi and Ashreeta have been translating to project time external to dsgrid as a work around.

Industry dataset needs this now, should support:

Local time hours 0,...,8759 or 1,...,8760
Adjusting for daylight savings time where appropriate

Should there be user options for how to do this?

Rolling data so all data is time synchronized

@lixiliu, please coordinate with @sjsmith11 and @smurthy0211 on designing an initial implementation that meets their needs.

#296 #298


Missing ctrl+a (select all) from shortcut

I don't see shortcut definition in terminal/src/cascadia/TerminalApp/ShortcutActionDispatch.cpp but function exist in terminal/src/host/selection.cpp. I didnt find where this function is called and how we can add it to the shortcut.

src/host/selection.cpp is a component in conhost, the Windows Console Host.
You're probably looking for /dup #3723

Sorry, I meant /dup #1469.


Upgrade text editor to use a rich text editor

I would like to upgrade your notes app to use an open-source rich text editor Quill.
If it is okay with you for me to go ahead on this, I will keep the page style as is and upgrade the text area and all functions that interact with the notepad (i.e. save, clear, etc.)

yes, go ahead please. @RyanMarzec


Refresh data

This PR is aimed at including some auto-pulls using setIntervals in two parts of the application. The first is in the player stats per map, which should allow us to poll data and view the scoreboard without having to refresh the page. After five seconds, users are allowed to force a refresh for the data, since we would like to have a bit of a wait time so spamming is not an option.
The second point of this PR will be to try and attempt to refresh vetoes as well. However, there's not an easy way of determining if a match is complete, or has vetoes without making more API calls. So the next step is to determine if we can find a way to ensure a match is running, and has a veto process, and check to see if they are in that state. One way of doing this is to maybe create an API end point that pings the match and gets the status of the server, and the other is just using different API calls to determine if we have vetoes or not. Still need to think on this a bit more.
Closes #118 #119

since the last update, the order of the matches and everything is getting really messed up.

this is after I click on the match id to sort them, default is even worse

This is most likely due to the synchronous loading from the data into the table. I'm no front-end developer so this will take some time to fix :) I've been looking at it, but not sure how to adjust it.

I think an easier way to fix this will to be create some new routes that just give the data that it's trying to assemble on the front end, that may help with ordering values... May look into that as a next avenue of trying to fix this.


refactor: contract integration test should streamline the error for better testing

cc: @Daanvdplas
fn decoded<T: Decode>(result: ExecReturnValue) -> T {
	match <T>::decode(&mut &result.data[2..]) {
		Ok(value) => value,
		Err(_) => panic!("\nTest failed by trying to decode `{:?}` into `T`\n", result),
	}
}

At the moment, any method failed will end up at line 13:39 which is inside the decoded method. This is very hard to debug. My recommendation is to refractor the decoded method to return Result<T> so it can streamline the error back to the main function. Something like:
fn decoded<T: Decode>(result: ExecReturnValue) -> Result<T> {
	<T>::decode(&mut &result.data[2..]).map_err("\nTest failed by trying to decode `{:?}` into `T`\n", result)
}

Yes this was something that could be improved, great suggestion!

Fixed by #130


Update/data api

Changed things:

New SpongeAPI version
Code updated for the Data API and the SpongeAPI
Removed unused errors

You need to fix the identation in some of the files. You can use one of our code styles in the Common repo to reformat your code

The Travis build is failing at the moment, so compile errors will need to be fixed before we can approve this.

The build still fails.

I am working on it:)

Sorry to be nitpickey, I just want to keep the codebase as uniform as possible.

Hahaha, no problem:)

Travis build still fails. You may want to run the Gradle wrapper locally before pushing.

Another nitpick: any chance you could revert the spaces added to the beginning of the TODO comments by the formatter, so the branch diff is kept to a minimum?

I know it still fails, but this due to some other files, that I haven't edited. I am still working on them. I will revert the spaces before the TODO's:)

Alright, thanks. Again, sorry to be a pain.

Hahahaha, It's alright I don't mind:)

Looks good. Is there anything else you want to change, or is it ready to be merged?

Well there are still a few little things I would like to change.

Now it is ready to be merged.

Merged via 70cd6d7. :)


The throughput is lower (worse) when run the test with 3.17.0 cassandra-stress version compared to 3.13.0 version

Before start using 3.17.0 cassandra-stress version in performance test, we ran the throughput/latency tests (mixed and read load) the few times and compared its results with results of the same test that ran with same Scylla version but 3.13.0 c-s version
Scylla version 2024.3.0~dev-20241106
Every test ran 3 untrottled cycles on the same cluster
Tablets disabled

Max throughput in mixed load:

c-s version
cycle 1
cycle 2
cycle 3
link

3.13.0
561599
561712
557011
https://argus.scylladb.com/tests/scylla-cluster-tests/516bfdc7-215b-4bdd-bd1c-be795c389134

3.17.0
533610
550041
553213
https://argus.scylladb.com/tests/scylla-cluster-tests/f667b4e3-bb37-43cf-aa40-edb6930480aa

Max throughput in read load:

c-s version
cycle 1
cycle 2
cycle 3
link

3.13.0
899256
885138
881635
https://argus.scylladb.com/tests/scylla-cluster-tests/0d7b9ac8-c847-456a-ae47-1e8e312cdac8

3.17.0
873856
854535
875792
https://argus.scylladb.com/tests/scylla-cluster-tests/f1e28be1-e166-423e-bde5-86852f5a3f02

Tablets enabled

Max throughput in mixed load:

c-s version
cycle 1
cycle 2
cycle 3
link

3.13.0
543966
542706
543258
https://argus.scylladb.com/tests/scylla-cluster-tests/aacd976a-21b8-4c95-a00c-cdeea51d7ba8

3.17.0
525962
523390
525102
https://argus.scylladb.com/tests/scylla-cluster-tests/b449b191-8816-445f-8f48-86ff6451835f

Max throughput in read load:

c-s version
cycle 1
cycle 2
cycle 3
link

3.13.0
780703
786261
782665
https://argus.scylladb.com/tests/scylla-cluster-tests/076805d0-3cd8-4b12-8697-49599cafc43d

3.17.0
644307
639954
640790
https://argus.scylladb.com/tests/scylla-cluster-tests/54f0e5d0-af63-46cf-ba97-4b71057a1f16

Conclusion:
throughput is lower (worse) when run the test with 3.17.0 cassandra-stress version

those are the changes between the versions:
https://github.com/scylladb/cassandra-stress/compare/v3.13.0...v3.17.0
I'll guess this is one of the following:

https://github.com/scylladb/cassandra-stress/commit/c09fd080b1022d0580574f8b2af17251de197cdf
https://github.com/scylladb/cassandra-stress/commit/f87206a212c1bd6f7611f4abbcab30f70e4811bf

@juliayakovlev , @fruch , is there a way to see CPU and memory load on loaders ?

@dkropachev @Bouncheck
do we expect ReplicaOrdering.RANDOM to affect results like that ?

@juliayakovlev , @fruch , is there a way to see CPU and memory load on loaders ?

in the os metrics dashboard, filter them by ip address (cross change with argus resources tab)
example from the last test from the table above:

@dkropachev @Bouncheck
do we expect ReplicaOrdering.RANDOM to affect results like that ?

From what I see loaders fail to generate requests with same speed as before, so I would say it could be, but only if it starved of CPU.


Add Python binding for online punctuation models

For #1227.
I renamed the parameter for OnlinePunctuationConfig to model_config, let me know if that's ok, or whether I should follow #762 and keep it as model.

Thank you for your contribution!


Broken link in Zebra documentation Config section

Description

In the zebrad config section of the documentation the ConsensusSection link gives a 404 not found error.
Steps to Reproduce
Go to https://doc.zebra.zfnd.org/zebrad/config/struct.ZebradConfig.html and click on ConsensusSection

The link points to  https://doc.zebra.zfnd.org/zebra_consensus/zebra_consensus/config/struct.Config.html but maybe it should point to: https://doc.zebra.zfnd.org/zebra_consensus/config/struct.Config.html instead?

In a quick look this is weird. In the internal docs the link is not broken https://doc-internal.zebra.zfnd.org/zebrad/config/struct.ZebradConfig.html
and i was not able to find anything in the code that can be causing it. I saw that zebra-consensus dependency is called with a trailing / in zebrad: https://github.com/ZcashFoundation/zebra/blob/main/zebrad/Cargo.toml#L72 while all the others do not trail.
I don't think this is the cause here but we can try.

Unfortunately, the refactor we made in https://github.com/ZcashFoundation/zebra/pull/5460 do NOT fix the link issue: https://doc.zebra.zfnd.org/zebrad/config/struct.ZebradConfig.html

This doesn't seem to be happening anymore...


How to enter volume number on Publications Details page

Help guide says to enter the volume number in the "Title of the Work", but this field implies that the title of the journal article should be entered here. We need a way for inputter to add volume number from a picklist of numbers below the "Journal of Publication Name" after that information is entered

At today's meeting it was agreed that:

[ ] We keep the current Resource Model mechanism to manage the Journal/Volume names
[ ] The Publication Type would be the first node in the card
[ ] The Journal / Publication Name be renamed (Currently using "Journal or Volume the Publication was published in" can be changed
[ ] The Other Journal or Publication Name label be changed (Currently using "Other Journal or Volume"
[ ] The Journal or Volume and Other Journal or Volume be presented side-by-side

A more robust widget or workflow will be used to hide the complexity of this section sometime after MVP.
We will work together to enhance the help text on that page.


CreatePatternAsync

Do you have any examples of this being implemented?
I'm not having any luck trying to port my java version to use your blazor extensions to create a drafting hatch.
EJ_Draw.zip
EJ Sketch.zip

EJ_Draw (2).zip
Updated. All functions work except createpatternasync


BUG: Support builds with JupyterLab 1.0

JupyterLab 1.0 requires an update the the @jupyter-widgets/base package:
https://github.com/jupyter-widgets/ipywidgets/blob/master/docs/source/changelog.md#updates-for-widget-maintainers
Closes #155

Just verifying that this resolved the issue! :heart: :tada:


Linreg

Simple tool for performing regularized linear regression.

Here's where I'm coming from on the formula: http://stackoverflow.com/questions/27476933/numpy-linear-regression-with-regularization

I'm pretty confident that the formula is correct. Here's my result using sklearn:
>>> from sklearn.linear_model import Ridge
>>> clf = Ridge(alpha=3., solver='svd')
>>> clf.fit(features['train'], labels['train'])
Ridge(alpha=3.0, copy_X=True, fit_intercept=True, max_iter=None,
   normalize=False, random_state=None, solver='svd', tol=0.001)
>>> clf.coef_
array([ 0.        ,  6.52208421,  3.82487688,  3.61985418,  2.18396316,
        2.10604481,  1.2807778 ,  1.28789997,  0.72492414])
>>> clf.intercept_
11.217589325366376
>>> predicted = clf.predict(features['test'])
>>> pn.learn.mse(predicted, labels['test'])
array([[ 7.14405231]])

i mostly have some minor comments. other than that it looks good to go

some replies on your comments. other than that good to go.


Add kernel independent logging

This moves https://github.com/jupyter/notebook/pull/2252 to jupyter_server. I think it is useful to have the ability to log which code has been executed (for auditability reasons).
The method of specifying a python logger is clunky and I'm looking for suggestions on ways to improve that (specifying the logger in a traitlets config file?).
Pinging @Zsailer

Thanks, @gclen! Pinging @kevin-bates for his thoughts, too.

Thanks for starting this @gclen.
I like the idea and think this could be generally useful - especially in larger installations.  It feels like a great fit as an event source for the Jupyter Telemetry project and makes sense to leverage for this.
I would imagine we'd include more data about the kernel "session" (user, kernel-id, kernel-name, provider, etc.) that could then be used to differentiate activity, either via cross-referencing (via kernel-id) or directly.

Sorry—I didn't have time to look at this closely yesterday. I thought this was directed at developers for debugging purposes, not at operators/administrators auditing the server.
I agree with Kevin. This should be handled by telemetry instead.
I'll draft a PR that adds telemetry to the server this afternoon.

See #233 for a draft of telemetry/event logging in Jupyter server.

I'm going to close this PR in favor of #233.
That said, @gclen it would be great to work with you on developing an event schema for the kernels API. As someone who is interested in logging/auditing kernels, you have super valuable insight there.

Checkout #238 for initial discussion on drafting a kernels schema.


URL not update when user click on app pages

@osiset
In my app, I have multiple vue js pages. In my local environment whenever I click on the page link, the URL is updated in the address bar. but on Shopify only the app name is visible. there is some way to update the URL on Shopify as well?

This is not a package issue, please use discussions thanks!


Avoid Indexoutofbounds when getting enum typename

Fixes https://circleci.com/gh/googleapis/artman/8422#artifacts/containers/0 and the issue raised by ArrayOutOfBounds.
Tested by running artman --local generate java_gapic on some of the failing jobs in https://circleci.com/gh/googleapis/artman/8422#artifacts/containers/0, e.g. google/cloud/language/artman_language_v1beta2.yaml

Codecov Report

Merging #2441 into master will decrease coverage by <.01%.
The diff coverage is 50%.

@@             Coverage Diff              @@
##             master    #2441      +/-   ##
============================================
- Coverage     86.44%   86.43%   -0.01%
  Complexity     5278     5278
============================================
  Files           458      458
  Lines         21047    21050       +3
  Branches       2286     2287       +1
============================================
+ Hits          18194    18195       +1
- Misses         2036     2037       +1
- Partials        817      818       +1

Impacted Files
Coverage Δ
Complexity Δ

...n/transformer/java/JavaModelTypeNameConverter.java
86.01% <50%> (-1.13%)
38 <0> (ø)

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 f0f0990...0e394ac. Read the comment docs.

PTAL

Might be the cause of #2429?


⚠ Bofrost.at has degraded performance

In 91944b9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1048 ms

Resolved: Bofrost.at performance has improved in bd7a226 after 8 minutes.


For one retry, length of list of fixed intervals passed to NewSimpleBackoff should be 2

Which version of Elastic are you using?
[ ] elastic.v5 (for Elasticsearch 5.x)
Please describe the expected behaviour
NewSimpleBackoff(500) should enable one retry after 500ms.
Please describe the actual behaviour
NewSimpleBackoff(0, 500) enables one retry after 500ms. This is because of below code in Next method implementation of SimpleBackoff:
if retry >= len(b.ticks) {
return 0, false
}
This means if retry = 1 and length of the list is also 1, there will not be any retries.
Any steps to reproduce the behavior?
Create SimpleBackoff by calling NewSimpleBackoff(500). Stop elasticsearch. Periodic healthcheck marks all nodes as dead. Start elasticsearch before the next periodic healthcheck. Fire an elasticsearch query (using PerformRequestWithOptions) before next periodic healthcheck. Since all connections are dead, it will invoke healthcheck. Call still fails because no retries happen, even though SimpleBackoff was created with one fixed interval value.

Can you make a test similar to this one that fails?

Sure, I will add a test. Thanks!

In PerformRequestWithOptions, n is declared but not initialised

In PerformRequestWithOptions, n is declared but not initialised. If there are no connections available, n is incremented. This means that while retrying, for first retry 1 is passed not 0. In the test that you have mentioned, retries start with 0.

I agree, that seems to be wrong. It should be -1 by default, or incremented after retrying.

This will need to be backported to 6.x and 5.x.


Doesn't support Termux on Android

Version: v1.4.2 aarch64 prebuilt on GitHub
The server does start, but then crash with Bad system call once I visit that URL.

Solved. ttyd exists in termux package reposity. Just try pkg install ttyd!


fix email unsubscription inconsistency after unenrollment

Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
fixes #445
What's this PR do?
fix email unsubscription inconsistency after un-erollment
How should this be manually tested?

Just unsubscribe to emails before unsubscribe
enroll back in should be subscribed back
that should have deleted the popout entry in edX against email subscription

Codecov Report

Merging #475 (edde6a7) into main (5b8fce9) will increase coverage by 3.94%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #475      +/-   ##
==========================================
+ Coverage   90.11%   94.06%   +3.94%
==========================================
  Files         241      137     -104
  Lines        7802     3319    -4483
  Branches      700        0     -700
==========================================
- Hits         7031     3122    -3909
+ Misses        667      197     -470
+ Partials      104        0     -104

Impacted Files
Coverage Δ

courses/api.py

courses/views/v1/__init__.py

users/serializers.py

users/management/commands/unblock_users.py

cms/blocks.py

openedx/views.py

mail/views.py

ecommerce/api.py

openedx/models.py

cms/conftest.py

... and 94 more

Continue to review full report at Codecov.

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


No custom TemplateFailureListener found

I'm not sure if this is a deadbolt-2 issue, but I have no idea why it's occurring. When I run my Play app (Play v2.4.3) on my laptop, everything is fine, but on a remote server, I get this cryptic message and the play-authenticate auth does not work. Is there a plausible reason why this error would occur on a cloud server and not on my laptop?
2015-09-15 15:18:59 +0000 [INFO] from play.api.Play in main - Application started (Prod)
2015-09-15 15:19:00 +0000 [INFO] from play.core.server.NettyServer in main - Listening for HTTP on /0:0:0:0:0:0:0:0:9000
2015-09-15 15:19:00 +0000 [INFO] from play.core.server.NettyServer in main - Listening for HTTP on /0:0:0:0:0:0:0:0:9000
2015-09-15 15:19:09 +0000 [INFO] from be.objectify.deadbolt.java.TemplateFailureListenerProvider in application-akka.actor.default-dispatcher-2 - No custom TemplateFailureListener found, falling back to no-op implementation
2015-09-15 15:19:09 +0000 [INFO] from be.objectify.deadbolt.java.ViewSupport in application-akka.actor.default-dispatcher-2 - Default timeout period for blocking views is [130000]ms

I tried changing my deadbolt.conf file to this, but it didn't make any difference:
deadbolt {
  java {
    blocking = true # JPA doesn't support non-blocking
    cache-user = true # For the rest of the request
    handler = auth.MyDeadboltHandler
    view-timeout=130000 # Default 1000
    blocking-timeout=2000
  }
}

My build.sbt file has these two lines:
 "be.objectify"  %% "deadbolt-java"     % "<IP_ADDRESS>",
  "com.feth" %%  "play-authenticate" % "0.7.0-SNAPSHOT",

This isn't an issue, but it looks like I should document it better.
Background:
Because constraints are handled in a blocking manner, the promises used are completed with a limit defined by deadbolt.java.view-timeout, as you have in your config.
If the promises time out and you have a TemplateFailureListener implementation, you'll get notifications of the failure; because the timeout value is given to the template wrapped in a Supplier, it is then possible to dynamically alter the value based on the failure information.
If you don't have a TemplateFailureListener defined, NoOpTemplateFailureListener is automatically used, and this is the notification you're seeing.

Just figured this out. Thanks for the clarification.
Are these log entries related?
2015-09-16 06:21:14 +0000 [INFO] from be.objectify.deadbolt.java.ExecutionContextProvider in application-akka.ac
tor.default-dispatcher-15 - No custom execution context found.
2015-09-16 06:21:33 +0000 [INFO] from be.objectify.deadbolt.java.ExecutionContextProvider in application-akka.ac
tor.default-dispatcher-14 - No custom execution context found.
2015-09-16 06:21:33 +0000 [INFO] from be.objectify.deadbolt.java.ExecutionContextProvider in application-akka.ac
tor.default-dispatcher-11 - No custom execution context found.
2015-09-16 06:21:40 +0000 [INFO] from be.objectify.deadbolt.java.ExecutionContextProvider in application-akka.ac
tor.default-dispatcher-19 - No custom execution context found.
2015-09-16 06:21:44 +0000 [INFO] from be.objectify.deadbolt.java.ExecutionContextProvider in application-akka.ac
tor.default-dispatcher-14 - No custom execution context found.
2015-09-16 06:22:06 +0000 [INFO] from be.objectify.deadbolt.java.ExecutionContextProvider in application-akka.ac
tor.default-dispatcher-20 - No custom execution context found.
2015-09-16 06:22:08 +0000 [INFO] from be.objectify.deadbolt.java.ExecutionContextProvider in application-akka.ac
tor.default-dispatcher-23 - No custom execution context found.
2015-09-16 06:22:10 +0000 [INFO] from be.objectify.deadbolt.java.ExecutionContextProvider in application-akka.ac
tor.default-dispatcher-23 - No custom execution context found.

This is another new feature I haven't got around to documenting yet; you can - you can provide an execution context for Deadbolt to run async code in, instead of the default context provided by Play.
It's mentioned at the bottom of the Scala version's quickstart guide - I'll update the Java version asap.
I also think I'll change that log level to debug!

Thanks Robert!

Robert?

@slimandslam if you change your version to 2.4.3 your logs should be quite a bit cleaner.

Steve!  :-)


Harvesting Map Services

This maybe a newbie question but I can't find documentation on how to harvest map services.  We have a number of map services hosted on a local server with ArcGIS REST Services that I would like to harvest.  What is the best way to go about doing this?  Is this feature even implemented yet?

Recent version of the harvester has the ability to harvest ArcGIS server services through a dedicated broker. If you are using default UI, the easiest way is to add input broker "ArcGIS server services", give it some meaningful name and URL pointing to your local "rest/services" end point, for example:
https://localhost:6080/arcgis/  (that forward slash at the end of this URL might me meaningful, so it better to keep it that way).
When you create a task with such a broker and run it, it will browse the end point and create Dublin Core metadata for each recognized artifact like map service or image service.
NOTE! If you attempt to harvest just deployed private instance of the ArcGIS server, harvester will fail to harvest such a server since that server has only self-signed SSL certificate installed. In order to get it working you have to either purchase a commercial certificate or trick Tomcat where the harvester is installed to think that this particular certificate is a valid certificate. It could be done by inserting that self-signed certificate into the 'cacerts' keystore of the Java JRE (or JDK) your Tomcat is running on, using standard Java tool called 'keytool'.

Thanks.  I was using an older version that only had CSW and WAF.  Just compiled the new version, looks promising.

I tried the new version and am now getting an error: Error executing task when trying to run the Task.  I am running the same tasks as I did in a earlier version of Harvester (Which had no issues).  Any ideas whats going on? See log below
<IP_ADDRESS> - - [27/Sep/2016:15:12:38 -0600] "POST /geoportal-harvester-war-2.0.0/rest/harvester/tasks/7d80a058-f840-4603-b33d-6153385553e1/execute HTTP/1.1" 500 -
<IP_ADDRESS> - - [27/Sep/2016:15:13:10 -0600] "POST /geoportal-harvester-war-2.0.0/rest/harvester/tasks/94917bfb-930d-42da-8072-1091acbe02ab/execute HTTP/1.1" 500 -
<IP_ADDRESS> - - [27/Sep/2016:15:13:11 -0600] "POST /geoportal-harvester-war-2.0.0/rest/harvester/tasks/7d80a058-f840-4603-b33d-6153385553e1/execute HTTP/1.1" 500 -

Code 500 indicates internal server error. So, where does that log come from? I would be looking for more information, like exception in hrv..log, which may tell more specific info. Also, is this properly deployed, ie. if you open you browser and type http://:/geoportal-harvester-war-2.0.0/rest/harvester/tasks, does it give you any JSON-kind of response (even empty array)?

That log comes from localhost_access_log.  The hrv log doesnt really stamp any errors when I click run task (See below).  And http://:/geoportal-harvester-war-2.0.0/rest/harvester/tasks does give me a working JSON return.  I will try redeploying the war file.
Sep 27, 2016 3:51:21 PM com.esri.geoportal.commons.csw.client.impl.ProfilesLoader load
INFO: CSW profiles loaded.

Sep 27, 2016 3:53:26 PM com.esri.geoportal.commons.csw.client.impl.ProfilesLoader load
INFO: Loading CSW profiles
Sep 27, 2016 3:53:26 PM com.esri.geoportal.commons.csw.client.impl.ProfilesLoader load
INFO: CSW profiles loaded.

What is the content of the task definition? You can check it by invoking 'export' (last screenshot).

There is one more thing I would do: I would delete (or move somewhere else) database files. Database is an H2 embeded databse and it's data is stored in the home folder of the user under Tomcat (or web server in general) is running. There are two database files: harvester.mv.db and harvester.trace.db. If you remove these files and restart harvester, it will create new and empty database. You will lose old definitions and you will need to define new ones, but the deployment will be really fresh.

removing the db files and redeploying the war file has gotten rid of the error!  Though it is still not harvesting, it is atleast trying.  Here is the hrv log from trying to broker Map Service -> Geoportal.
INFO: SUBMITTING: TASK :: PROCESSOR: DEFAULT[], SOURCE: AGS[ags-host-url=http://ngismap:6080/arcgis/rest/services/Canada/base_cache/MapServer/, cred-username=xxx, cred-password=xxx], DESTINATIONS: [GPT[gpt-host-url=http://cal8783:8088/geoportal/, cred-username=gptadmin, cred-password=gptadmin, gpt-cleanup=false]], INCREMENTAL: false Sep 28, 2016 10:07:28 AM com.esri.geoportal.harvester.engine.defaults.DefaultProcessor$DefaultProcess lambda$new$1 INFO: Started harvest: com.esri.geoportal.harvester.ags.AgsBroker@5f9909ca --> [GPT [http://cal8783:8088/geoportal//]] Sep 28, 2016 10:07:28 AM com.esri.geoportal.harvester.support.ReportLogger started INFO: Started processing task: PROCESS:: status: working, title: com.esri.geoportal.harvester.ags.AgsBroker@5f9909ca --> [GPT [http://cal8783:8088/geoportal//]] Sep 28, 2016 10:07:28 AM com.esri.geoportal.harvester.support.ReportStatistics started INFO: Harvesting of PROCESS:: status: working, title: com.esri.geoportal.harvester.ags.AgsBroker@5f9909ca --> [GPT [http://cal8783:8088/geoportal//]] started at Wed Sep 28 10:07:28 MDT 2016 Sep 28, 2016 10:07:28 AM com.esri.geoportal.harvester.support.ReportLogger completed INFO: Completed processing task: PROCESS:: status: completed, title: com.esri.geoportal.harvester.ags.AgsBroker@5f9909ca --> [GPT [http://cal8783:8088/geoportal//]] Sep 28, 2016 10:07:28 AM com.esri.geoportal.harvester.support.ReportStatistics completed INFO: Harvesting of PROCESS:: status: completed, title: com.esri.geoportal.harvester.ags.AgsBroker@5f9909ca --> [GPT [http://cal8783:8088/geoportal//]] completed at Wed Sep 28 10:07:28 MDT 2016. No. succeded: 0, no. failed: 0
So it is showing as 0 Succeeded and 0 Failed, and when checking Geoportal Catalog there is no new entry.

Instead of http://ngismap:6080/arcgis/rest/services/ I would register just http://ngismap:6080/arcgis/
NOTE! Once you modify the broker, a task created based on the old broker definition won't be updated. So, you have to create a new task then run it.
Also, it turn's out it doesn't have to be run through HTTPS, at least in my environment doesn't.

Ok just tried that and its resulting in a new error and failed task:
`
Sep 28, 2016 10:52:47 AM com.esri.geoportal.harvester.support.ReportLogger error
SEVERE: Error processing task: PROCESS:: status: working, title: com.esri.geoportal.harvester.ags.AgsBroker@6fa45356 --> [GPT [http://cal8783:8088/geoportal//]]
com.esri.geoportal.harvester.api.ex.DataInputException: Error listing server content.
at com.esri.geoportal.harvester.ags.AgsBroker.iterator(AgsBroker.java:115)
at com.esri.geoportal.harvester.engine.defaults.DefaultProcessor$DefaultProcess.lambda$new$1(DefaultProcessor.java:126)
at java.lang.Thread.run(Thread.java:745)
Caused by: com.fasterxml.jackson.databind.exc.InvalidFormatException: Can not construct instance of java.lang.Long from String value ("NaN"): not a valid Long value
at [Source: {"currentVersion":10.22,"serviceDescription":"","mapName":"Layers","description":"","copyrightText":"","supportsDynamicLayers":false,"layers":[],"tables":[],"spatialReference":{"wkid":4326,"latestWkid":4326},"singleFusedMapCache":false,"initialExtent":{"xmin":-2.2117354231454307,"ymin":49.48528420902821,"xmax":-0.04197360553260565,"ymax":52.42996096150276,"spatialReference":{"wkid":4326,"latestWkid":4326}},"fullExtent":{"xmin":"NaN","ymin":"NaN","xmax":"NaN","ymax":"NaN","spatialReference":{"wkid":4326,"latestWkid":4326}},"minScale":0,"maxScale":0,"units":"esriDecimalDegrees","supportedImageFormatTypes":"PNG32,PNG24,PNG,JPG,DIB,TIFF,EMF,PS,PDF,GIF,SVG,SVGZ,BMP","documentInfo":{"Title":"","Author":"","Comments":"","Subject":"","Category":"","AntialiasingMode":"None","TextAntialiasingMode":"Force","Keywords":""},"capabilities":"Map,Query,Data","supportedQueryFormats":"JSON, AMF","exportTilesAllowed":false,"maxRecordCount":1000,"maxImageHeight":2048,"maxImageWidth":2048,"supportedExtensions":""}; line: 1, column: 432] (through reference chain: com.esri.geoportal.commons.ags.client.ServerResponse["fullExtent"]->com.esri.geoportal.commons.ags.client.ExtentInfo["xmin"])
at com.fasterxml.jackson.databind.exc.InvalidFormatException.from(InvalidFormatException.java:74)
at com.fasterxml.jackson.databind.DeserializationContext.weirdStringException(DeserializationContext.java:923)
at com.fasterxml.jackson.databind.deser.std.StdDeserializer._parseLong(StdDeserializer.java:488)
at com.fasterxml.jackson.databind.deser.std.NumberDeserializers$LongDeserializer.deserialize(NumberDeserializers.java:353)
at com.fasterxml.jackson.databind.deser.std.NumberDeserializers$LongDeserializer.deserialize(NumberDeserializers.java:331)
at com.fasterxml.jackson.databind.deser.SettableBeanProperty.deserialize(SettableBeanProperty.java:490)
at com.fasterxml.jackson.databind.deser.impl.FieldProperty.deserializeAndSet(FieldProperty.java:101)
at com.fasterxml.jackson.databind.deser.BeanDeserializer.vanillaDeserialize(BeanDeserializer.java:260)
at com.fasterxml.jackson.databind.deser.BeanDeserializer.deserialize(BeanDeserializer.java:125)
at com.fasterxml.jackson.databind.deser.SettableBeanProperty.deserialize(SettableBeanProperty.java:490)
at com.fasterxml.jackson.databind.deser.impl.FieldProperty.deserializeAndSet(FieldProperty.java:101)
at com.fasterxml.jackson.databind.deser.BeanDeserializer.vanillaDeserialize(BeanDeserializer.java:260)
at com.fasterxml.jackson.databind.deser.BeanDeserializer.deserialize(BeanDeserializer.java:125)
at com.fasterxml.jackson.databind.ObjectMapper._readMapAndClose(ObjectMapper.java:3789)
at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:2779)
at com.esri.geoportal.commons.ags.client.AgsClient.readServiceInformation(AgsClient.java:149)
at com.esri.geoportal.harvester.ags.AgsBroker.listResponses(AgsBroker.java:125)
at com.esri.geoportal.harvester.ags.AgsBroker.listResponses(AgsBroker.java:132)
at com.esri.geoportal.harvester.ags.AgsBroker.iterator(AgsBroker.java:112)
... 2 more
Sep 28, 2016 10:52:47 AM com.esri.geoportal.harvester.support.ReportLogger completed
INFO: Completed processing task: PROCESS:: status: completed, title: com.esri.geoportal.harvester.ags.AgsBroker@6fa45356 --> [GPT [http://cal8783:8088/geoportal//]]
Sep 28, 2016 10:52:47 AM com.esri.geoportal.harvester.support.ReportStatistics completed
INFO: Harvesting of PROCESS:: status: completed, title: com.esri.geoportal.harvester.ags.AgsBroker@6fa45356 --> [GPT [http://cal8783:8088/geoportal//]] completed at Wed Sep 28 10:52:47 MDT 2016. No. succeded: 0, no. failed: 1
`

This is a bug and a good one, but I think I got it fixed. Why don't you get the latest code and give it another try? (You won;t need to delete database).

Awesome it appears the bug is fixed.  I was able to successfully harvest.

Thank you for making Geoportal Server 2.0 better.


add test cases for GC

In this PR, I added test cases for GC and disabled GC for metadata tables. I will port more complicated test cases tomorrow.

Coverage increased (+0.1%) to 65.247% when pulling 7497e97ca93183d32a3d8943ca411abf633177f4 on yingjunwu:gc into de09751d1312a58b2bcce719cf2c22ef711fc652 on cmu-db:master.

Coverage increased (+1.07%) to 66.181% when pulling c78baecea3c3d3b9b9bd013260127a534b4106cd on yingjunwu:gc into de09751d1312a58b2bcce719cf2c22ef711fc652 on cmu-db:master.

@yingjunwu Rebase and I'll merge this.


token-js: Expose calculateFee for mints with transfer fees

Problem
The rust side provides a nifty way to calculate a transfer fee using calculate_fee https://github.com/solana-labs/solana-program-library/blob/108e91e441e6633f046168536be571068e90169b/token/program-2022/src/extension/transfer_fee/mod.rs#L50, but wallets or JS clients can't use this.
Solution
Add an implementation for calculateFee in token-js.

I would like to try this issue

Yep that's pretty much it. And it would also be good to add a helper like https://github.com/solana-labs/solana-program-library/blob/108e91e441e6633f046168536be571068e90169b/token/program-2022/src/extension/transfer_fee/mod.rs#L133 on TransferFeeConfig

I will try solve this issue then, if there is something i don't understand then please help me, since I'm beginner in Rust.

@dhruvdabhi101 have you worked on it?
I would like to join as contributor as well :D

Yep that's correct, https://github.com/solana-labs/solana-program-library/pull/4594 resolved it. Closing the issue


Make L1 block headers available in the execution context

It would be useful it Optimism provided access to L1 block headers similar to L1 Block Number. https://github.com/ethereum-optimism/optimism/blob/f4cf99e34b9d06f20335eece5dc84e10fdbeb8cc/packages/contracts/contracts/L2/predeploys/iOVM_L1BlockNumber.sol#L12
In particular, making the last n state roots available on optimism would enable contracts to easily prove and operate on layer 1 state without any interaction from the base chain.
So in contrast to getL1BlockNumber(), an optimal solution would probably support getL1Header(uint256 n) so it is possible for a client to generate a storage proof that interacts with a particular recent layer 1 block header.

We've been thinking about adding a history accumulator to merkleize all past L1 data together that was posted to L2. See discussion https://github.com/ethereum-optimism/optimistic-specs/discussions/235
That would enable an efficient proof to L1 activity that was previously confirmed on L2.
This is an advanced feature that may come only after initial bedrock deployment though. Block-hash will definitely be available, and latest L1 state-root probably too (if not already the case).


[swift-4.0-branch] IRGen: Use link_once instead of external for private decls

• Explanation: We can get duplicate symbols for accessors IRGen creates per IGM instance e.g
for lazy protocol witness table accessors. This results in linkage errors.

Scope: This can happen on wmo compilation when private protocols are use.

Risk: Low, changing the linkage from external to linkonce_odr should not cause issues.

*Reviewed: Erik

Testing this has been verified on the breaking software.

rdar://31988578

@swift-ci Please test

@swift-ci Please test linux

Please add a test

Also this seems overly broad because for instance private methods will never be emitted multiple times

Yes. And. Yes.
See commit message "We can get duplicate symbols for accessors IRGen creates per IGM instance e.g
for lazy protocol witness table accessors.
(A better but more risky fix is to hoist the caching of the accessors to
IRGenerator so that we only generate them once per module to begin with)
Yes I owe a test case here ..."


feat: Non-blocking polling for confluent kafka driver.

Description
This request pool is strongly inspired by an issue from the confluent-kafka-python project: https://github.com/confluentinc/confluent-kafka-python/issues/612. I'm trying to stop using the call_or_wait function for the main message consuming flow.
Fixes #1904
Type of change
Please delete options that are not relevant.

[ ] Documentation (typos, code examples, or any documentation updates)
[x] Bug fix (a non-breaking change that resolves an issue)
[x] New feature (a non-breaking change that adds functionality)
[ ] Breaking change (a fix or feature that would disrupt existing functionality)
[ ] This change requires a documentation update

Checklist

[x] My code adheres to the style guidelines of this project (scripts/lint.sh shows no errors)
[x] I have conducted a self-review of my own code
[ ] I have made the necessary changes to the documentation
[x] My changes do not generate any new warnings
[ ] I have added tests to validate the effectiveness of my fix or the functionality of my new feature
[ ] Both new and existing unit tests pass successfully on my local environment by running scripts/test-cov.sh
[ ] I have ensured that static analysis tests are passing by running scripts/static-analysis.sh
[ ] I have included code examples to illustrate the modifications

Closed in prior to #2003


Documentation issues

Please don't close and use this to report your documentation issues:
e.g.
https://splunk.github.io/splunk-connect-for-syslog/1566/gettingstarted/quickstart_guide/

I think this information is not reflected in the documentation but helps tremendously:
https://splunk-usergroups.slack.com/archives/CNV918JCQ/p1648658161334779?thread_ts=1647621045.564949&cid=CNV918JCQ
for any vendor/product source
_nix_syslog can be appended to route the fall through to the proper index
I used this in splunk_metadata.csv for routing to a different index
   vmware_vsphere_nix_syslog,index,virtualization

code block for timezone is broken:
https://splunk.github.io/splunk-connect-for-syslog/main/troubleshooting/troubleshoot_resources/#fix-timezone

code block for timezone is broken: https://splunk.github.io/splunk-connect-for-syslog/main/troubleshooting/troubleshoot_resources/#fix-timezone

fixed in https://github.com/splunk/splunk-connect-for-syslog/commit/d421c385e6c41204cc346bc7c671f3da07559cdf

I think this information is not reflected in the documentation but helps tremendously: https://splunk-usergroups.slack.com/archives/CNV918JCQ/p1648658161334779?thread_ts=1647621045.564949&cid=CNV918JCQ
for any vendor/product source _nix_syslog can be appended to route the fall through to the proper index
I used this in splunk_metadata.csv for routing to a different index  vmware_vsphere_nix_syslog,index,virtualization

True, it can be useful when current parsers don't process new types of messages from a given vendor. In this situation one can add to /opt/sc4s/env_file for example:
SC4S_LISTEN_TESTING_TEST_UDP_PORT=1515
and then events coming through 1515/udp will get vendor=testing and product=test. If these are rfc compliant messages and no parser catches them, they will be also parsed as class=nix_syslog.  And because of that records like testing_test_nix_syslog,index,main` will correctly redirect them like described here.
However, it's rather a hack than a solution to document. For situations when vendor messages are not parsed as documented please contact Splunk support or open a fix/feat request so we can update parsers.


style(bee-q): promise executor functions should not be async

This fixes the bugs spotted by SonarLint here:
https://sonarcloud.io/project/issues?resolved=false&types=BUG&id=Endava_bee-q

☁ Nx Cloud Report
We didn't find any information for the current pull request with the commit 34920fc071456f2ba4952d5e8d5f659c3a783507.
Please verify you are running the latest version of the NxCloud runner.
Check the Nx Cloud Github Integration documentation for more information.
Alternatively, you can contact us at<EMAIL_ADDRESS>
Sent with 💌 from NxCloud.


Tags (categories) support

Add tag support. (Tags are located near the date)

By tag support I mean the new property tags (array of tag interface, where tag has a label and a link). And categories. Also add get articlea by categories support as well.


Removed hardcode of $pathAttribute from moveNode()

Thank you. I did not even notice that I have here hardcoding)


HDWallpapers - latest wallpapers [1920x1200] failed with error 0

Error message: Call to a member function text() on null
Query string: action=display&bridge=HDWallpapers&c=latest_wallpapers&m=&r=1920x1200&format=Html
Version: dev.2019-03-17

Should be fixed now !


[SPARK-43315][CONNECT][PYTHON][SS] Migrate remaining errors from DataFrame(Reader|Writer) into error class

What changes were proposed in this pull request?
This PR proposes to migrate all remaining errors from DataFrame(Reader|Writer) into error class
Why are the changes needed?
To improve PySpark error
Does this PR introduce any user-facing change?
No API changes, only error improvement.
How was this patch tested?
This existing CI should pass.

Merged to master.


Don't loop infinitely on a malformed request

closes #104

@sector-f can you try the issue_104 branch and tell me if you're still having the same problem

Yeah, this does seem to fix it as far as I can tell

All right. More test cases are always welcome but I'll go ahead and release this as 0.15.2


Added linecount check to conftest bats test

What is this PR About?
Improving testing generally
Check list

[x] Have you created a test case in _test/conftest-unittests.sh

cc: @redhat-cop/rego-policies

/approve

/approve

/lgtm

/lgtm


Define GraphQL queries for MVP

About
This is a rough draft of the queries/mutations/subscriptions the frontend probably would need for an MVP
Subscriptions can be either long poll or websockets.
Patients

QUERY: get all patients
SUBSCRIPTION: get newly added patients (may not be an MVP feature)
MUTATION: add single patient (may not be an MVP feature)
MUTATION: edit single patient (not MVP required)
MUTATION: add multiple

Organization

QUERY: get org data
SUBSCRIPTION: subscribe to changes (not sure if this is needed -- org info should be pretty static)
MUTATION: update org information

TestResults

QUERY: get all test results
SUBSCRIPTION: get newly added test results (may NOT be an MVP feature)
MUTATION: add a test result for a single patient

Queue:

QUERY: get all patients in the queue
SUBSCRIPTION: get patients that are added or removed from the queue (use the same subscription but with different mutation types)
MUTATION: add patient to the Queue
MUTATION: remove patient from the queue

CSV Export History:

MUTATION: save timestamp and the csv of last upload to Prime Hub

User:

QUERY: get current user

Devices:

QUERY; get all devices
SUBSCRIPTION: get all Devices (not sure if this is needed. It should be pretty static, but clients still need to sync if it changes)
MUTATION: set all devices (not update)

Auth:

not sure. Don't know if auth endpoints need/should go through graphql, or if we can just create the callback urls in a more traditional manner

A couple points on these:

I think there are two things overlapping here somewhat dangerously: Organization (an entity that we probably need some users to be able to list, but most users belong to one and don't know about the others), and Configuration (the choices made by the organization). We should probably have graphql endpoints for querying and updating organizations, because we should eat our own dogfood, but mostly people should be doing get my organization and get the configuration for my organization. (So actually they would probably do get the current user and the associated organization now that I stop to think about it.)
Devices are the perfect example: we need a global list of valid devices, but non-admin users don't need that list, they need the list that is configured for their site.
Queue entries have patients and a bunch of other data which may be different by site: the ask-on-entry questions need to be in there, at least, and arguably we should be able to select which machine a test is going to run on and have that be saved even if we haven't gotten the result back yet
Auth should just be auth and should not be part of graphql

More broadly: I don't know how graphql is typically structured, but in the sample it looks like everything is either a "method" of Query or Mutation. Is that a requirement, a convention, or just a convenience for introducing people to things? I think we might want to have some organization beyond the flat single level of named queries, but maybe that's the old REST fan in me... 😜

More broadly: I don't know how graphql is typically structured, but in the sample it looks like everything is either a "method" of Query or Mutation.

I think we have to define the types and resolves for queries (roughly, reads) and mutations (roughly, create-update-delete) separately. They're entirely different things that are both necessary.
You can see examples here: https://github.com/CDCgov/prime-data-input-client/blob/main/backend-graphql-dev/src/server.js

Answering my own question: that is either a sufficiently strong convention that it might as well be a requirement or a requirement, so if we want to organize the schema logically we'll have to pretend it's 1985 and we just discovered long identifier names.


add alert to Module dependences of contacts

Coverage remained the same at 41.768% when pulling 64f30599bdf0ef2126df1e5885caead1b5212ca4 on embbnux:bugfix/contacts-dependences into b1f27dd9cae55e1011052cb662f127ced26eb0fe on ringcentral:master.


Glitches in Brackets's UI with only Custom Working extension enabled.

With the Brackets default theme active, and Custom Working being the only enabled extension, Bracket's editor windows are positioned to the right of the menu (instead of below the menu). With the menu taking up space from top to bottom.
Also, at the bottom of the 2 editor windows, some whitespace is added.
Have you any idea what might cause this?
OS: Arch Linux
OS version: Linux 4.8.13-1-ARCH #1 SMP PREEMPT Fri Dec 9 07:24:34 CET 2016 x86_64 GNU/Linux
Brackets version:  Release 1.8 experimental build 1.8.0-0
The page of the package : https://aur.archlinux.org/packages/brackets/

I apologize for the delay, I was offline.
Please comment if the problem persists

This is the internet. No problems.
I'm not using Brackets anymore. Uninstalled long ago.
Visual Studio Code is my main editor now, and somewhat Atom.

👍


Refactor histogram to gauge and more test examples

@w-vi still tests failing, looks as problem with order, but I'm not sure. Pls look if find something more

@w-vi fixed


fix compilation errors

Fix compilation errors at Rust version 1.78.
Description of changes:

I think rusttype is mainly to introduce the default font, but in fact to pass is ab_glyph::Font, so I use ab_glyph::FontArc.

draw_text_mut method needs ab_glyph::PxScale, so use ab_glyph::PxScale instead of rusttype::Scale, there is no place to use rusttype, we can consider to remove it.

fixed the problem of From, directly use core::convert::Into on the line, so we can remove conv lib

It now works well with Rust 1.78 after these changes.

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!

Hi @Moon1102
Please sign your commits with DCO. Ref: https://www.secondstate.io/articles/dco/

@Moon1102  Thanks for your PR! I think this is a perfect solution to  #6, it does not change the interface and also removes the redundancy dependencies.


Fix scaling loop by conditional setting of replicaCount.

Helm3 replaces the replica count set by the HPA repeatedly, resulting in a up- and down-scaling loop.
Because of that it is necessary to update the helm charts.
helm/helm#7090

Thanks for your contribution. Can you please increase the version of the charts in the Chart.yaml?
Everything else looks good.


Can not build successfully

When I try $ yarn build.
It just returns
yarn run v1.22.4
error Couldn't find a package.json file in "/Users/[username]"
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
So do I need to git clone the whole project first?

fixed by first git clone git-path, and continue inside the folder path


Change Request: Clarify AZURE_NO_PROXY formate for all languages

The Basics

Which design guideline is affected?

https://azure.github.io/azure-sdk/general_azurecore.html#environment-variables

Which target languages are affected?

All languages

Describe the change:

Clarify the expected format for NO_PROXY.
Reasoning
Please provide any additional detail or reasoning for the request.
Currently, NO_PROXY only states that CIDR annotation must be supported, it doesn't clarify an expected format for the value stored in the environment variable.
Should this be a language specific format? For example Java has a corresponding http.nonProxyHosts (documentation) environment variable which is expected to be a | delimited list of host string, which may be a regex string.

Scheduled for 4/12

Link to recording (MSFT INTERNAL): https://msit.microsoftstream.com/video/2240a1ff-0400-86e9-7be5-f1ea7e981f89
AZURE_NO_PROXY format:

MUST be comma-separated
MUST support IP addresses, CIDR ranges and hostnames
-- Do hostnames typically support wildcards? (This could be fairly common)
-- Domain extensions are supported by wget, which are similar to wildcards
For idiomacy, restrict to what the language supports

Q: Support for NO_PROXY or not, or support for varying levels of NO_PROXY?

Java example: Azure-specified NO_PROXY takes precendence over Java's NO_PROXY. Proxy selector could parse out and handle Azure-specified NO_PROXY. This wouldn't be considered lowest common denominator.
JS/TS core doesn't support NO_PROXY yet because the transport doesn't support it

Q: What does partial support look like? Ie, you have NO_PROXY variable under our guidelines, but there is a proxy available?

Best effort to support the max level, and otherwise document lack of support.

ACTION @kyle-patterson to submit PR updating general guidelines


Change the type of cmd from String to Cmd

This PR fixes https://github.com/AtelierArith/Replay.jl/issues/62.

https://github.com/AtelierArith/Replay.jl/pull/75 should resolve CI / Julia nightly
Good to go to merge this PR


Warning: Cask 'movist-pro' is unreadable: undefined method `method_missing_message' for Utils:Module

Please note that we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[x] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[ ] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[ ] ran brew update and can still reproduce the problem?
[ ] ran brew doctor, fixed all issues and can still reproduce the problem?
[ ] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)

What happened (include command output)

Command output

Warning: Cask 'movist-pro' is unreadable: undefined method `method_missing_message' for Utils:Module

What you expected to happen

Step-by-step reproduction instructions (by running brew commands)
$ brew search util-linux

From the issue template:

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

If you add the necessary information to this issue (don't create a new one, please) then this issue may be reopened.


Asks for confirmation if file not found. Added related --safe/-s and --f...

Asks for confirmation if file not found. Added related --safe/-s and --force/-f options.
I noticed that if you accidentally misspelled a file name when using retain, you were screwed and that file was gone forever. That seemed terribly unsafe, so I made the program ask for confirmation if it found that a file did not exist.
Then, to allow the program to still be useful in scripts where interaction would not be possible, I added --force and --safe. --force causes the program to run as it would have before these changes--ignoring any files that do not exist, and --safe terminates the program if any of the files are not found.
Exiting via --safe or the user opting not to continue both cause an exit status of 1 and raise an exception.
Since this change isn't backwards compatible with the previous version of the script, I updated the version to 2.0.0 (as per semantic versioning). I'm not sure if that carries over in the pull request or not though.

Forgot to update some documentation. Getting to that now.


Automatic build of virtlet image on docker hub

Should be done in the manner similar to AppController, but using build/cmd.sh

We have this already: https://cloud.docker.com/app/mirantis/repository/docker/mirantis/virtlet/builds