Is there a phpstorm version?

I use both Sublime Text and PHPStorm and am wondering if you know of a phpstorm version of your theme?
Either way, thanks for the great theme.

Thanks :wink: Sorry, I have no plans to create themes for other editors.


Security Notice & Bug Bounty - Remote Code Execution - huntr.dev

This issue has been generated on-behalf of Mik317 (https://huntr.dev/app/users/Mik317)
Vulnerability Description
The issue occurs because a user input is formatted inside a command that will be executed without any check. The issue arises here: https://github.com/aichbauer/node-git-commit-range/blob/master/index.js#L32
POC
// poc.js
const gitCommitRange = require('git-commit-range');

gitCommitRange();

gitCommitRange({
  path: '.; uname > poc',
  from: '15be93c31ad87c9ced03ba0b60fc2fb55c977c5c',
  to: '32b940b014322834966d79b109d2d7adec8e3ea3',
  include: false,
});

Impact
RCE on git-commit-range via insecure command formatting
Bug Bounty
We have opened up a bounty for this issue on our bug bounty platform. Want to solve this vulnerability and get rewarded 💰? Go to https://huntr.dev/
We will submit a pull request directly to your repository with the fix as soon as possible. Want to learn more? Go to https://github.com/418sec/huntr 📚
Automatically generated by @huntr-helper...

🛠 A fix has been provided for this issue. Please reference: https://github.com/418sec/node-git-commit-range/pull/1
🔥 This fix has been provided through the https://huntr.dev/ bug bounty platform.

Bug Bounty
We have opened up a bounty for this issue on our bug bounty platform. Want to solve this vulnerability? Go to https://huntr.dev/
We will submit a pull request directly to your repository with the fix as soon as possible. Want to learn more? Go to https://github.com/418sec/huntr 📚
Automatically generated by @huntr-helper...


API access open

🐛 Describe the Bug
https://bmdb.jpl.nasa.gov/apis is open; should be closed
🕵♀ Extra Details
See message from @dcrichto in Message-ID<EMAIL_ADDRESS>

It's not open anymore


blocks itunes from receiving keys

hey! the addon is great, just what I was looking for, I have just one small problem, in situation when I'm not using spotify player, but itunes, the keys stop working, would it be possible to check if spotify is opened, and if not, then propagate the keys back to the os?

Sure! I hope to be able to provide a patch ASAP

:+1: thanks man!

Well I think I have a bad news for you. I tryed to investigate the situation, and the thing is working in a totally different way.
Chrome is automatically binding to the mediakeys everytime you open it. This will be later catched as an event in my extension, which will be fired up by Chrome everytime you press a Media key.
When pressed, I'll be told which one was and I'll delegate it to spotify. So technically, if you have any Spotify tab you should no longer be able to forward actions to the spotify tab, but the play key will be anyway catched by Chrome.
You can see various blog posts or articles talking about this. So I guess I've got nothing in my own hands to fix this.
Sorry :)

Well, that's a shame, thanks for the explanation!


Corrupted frames when reading from file (mp4) on x86

Hi!
I'm trying to use this nice repo both on a jetson and on a laptop with an x86 CPU and Nvidia GPU.
However, the very first try gives me broken/corrupted frames even in the simplest example with mp4.
Here is what I do:

#include <iostream>

#include "jetson-utils/videoOutput.h"
#include "jetson-utils/videoSource.h"

#include "jetson-utils/cudaMappedMemory.h"

int main() {

  uchar3 *frame = nullptr;
  uchar3 *frame_resized = nullptr;

  videoOptions disp_options;
  disp_options.zeroCopy = false;
//  auto disp = videoOutput::Create("display://0", disp_options);
  auto disp = videoOutput::Create("file:///home/user/out.mp4", disp_options);

//  std::string file_path = "/home/nvidia/Downloads/DellePR1.4.mp4";
  std::string file_path = "/home/user/Downloads/DellePR1.4.mp4";

  std::string jutil_path = "file://" + file_path;

  videoOptions src_options;
  src_options.zeroCopy = false;
  auto cap = videoSource::Create(jutil_path.c_str(), src_options);
  auto new_options = cap->GetOptions();
  new_options.Print();

  uint32_t width = cap->GetWidth();
  uint32_t height = cap->GetHeight();

  if (!cudaAllocMapped((void **) &frame, width, height, imageFormat::IMAGE_RGB8)) {
    fprintf(stderr, "[POOL] -- cudaAllocMapped failed\n");
    exit(1);
  }

  bool res = true;

  while (res) {

    cudaDeviceSynchronize();
    res = cap->Capture(&frame);
    cudaDeviceSynchronize();

    if (res) {
      auto res_render = disp->Render(frame, width, height);
      while (!res_render) {
        std::cout << "retry render\n";
        res_render = disp->Render(frame_resized, width, height);
      }
    }
  }

  disp->Close();

  return 0;
}

This snippet works nicely on a jetson, but on x86 I got this:

Apparently, old memory is used. I've tried to use cudaDeviceSynchronize() but that does not help. I've also used a queue, and an artificial pause and the mapped memory is disabled but the result is the same all the time.
Best regards,
Alex

Hi @slovak194, as a first test to debug, does this work for you?
video-viewer /home/user/Downloads/DellePR1.4.mp4 /home/user/out.mp4

What GPU card to you have in your x86 system?   Are you running the jetson-inference x86 container?
Note that x86 support in this repo is pretty much a beta feature (and container only), as I don't personally have the resources to test jetson-inference across the multitude of system/OS configurations that exist for x86.

Hi, @dusty-nv
Thank you for getting back!

does this work for you?

No, same issue. Frames are corrupted both in out.mp4 file and in a viewer.

What GPU card to you have in your x86 system?

NVIDIA GeForce 940MX
Driver Version: 530.30.02
Driver Version: 530.30.02    CUDA Version: 12.1
CUDA Toolkit 11.8

Are you running the jetson-inference x86 container?
I use bare jetson-utils installation from the latest master without inference and containers.

Note that x86 support in this repo is pretty much a beta feature (and container only), as I don't personally have the resources to test jetson-inference across the multitude of system/OS configurations that exist for x86.

We can do (already doing) that.
I run very common Ubuntu LTS 20.04 and the environment is pretty much the same as on our jetsons. But of course, even the smallest hint from your side will be very useful. I suspect the issue is in how memory is used somewhere in the buffer manager.
We would love to have it working as it gives the possibility to easily move load/algorithms between jetson and PC.
Thank you in advance!
Best regards,
Alex


Implicit allow connections from localhost not documented

This is a Bug Report

Problem:
An Ingress NetworkPolicy implicitly allows all connections from the pod's host.  See https://github.com/kubernetes/api/blob/v0.18.2/networking/v1/types.go#L64-L65 .  The documentation at https://kubernetes.io/docs/concepts/services-networking/network-policies/ omits this fact.
Proposed Solution:
Make that doc describe the implicit allow.
Page to Update:
https://kubernetes.io/docs/concepts/services-networking/network-policies/

1.18, and older

Dup of #20897


Expose an interface to ImageResizer and PowerRename that can be called by Power Automate Desktop, a Windows service or other desktop task scheduler

Description of the new feature / enhancement
To expose ImageResizer and PowerRename through APIs that can be called by Power Automate Desktop, a Windows service or some other desktop task scheduler.
In the latest version of Power Automate Desktop:

there is no way, other than "Run VBScript", to resize images.
file renaming doesn't support regular expressions.

PowerToys is great at these tasks so its a missed opportunity for the Power suite not to be able to leverage them.
Scenario when this would be used?
The most useful current scenario for us would be to rename and resize photos automatically when they appear in a project particular folder.
For example, currently we have many engineers that take project related photographs and save them into the relevant project folders exactly as they are - they don't have the time to rename them or resize them so we have an administrator that has to rename the photos and resize them into 3 different dimension sets.  Even using PowerToys, this process takes some time given the number of project folders and daily frequency that these tasks must be undertaken.
The rules for renaming are based on regular expressions and are repeatable but have to be carried out manually in each of 40-50 project folders several times a day.  The resizing is also formulaic and repeatable but also carried out manually.
Instead of these manual processes, we could like to use Power Automate Desktop running on the admin's machine to parse project folders and undertake all renaming and resizing tasks automictically by calling into PowerToys.
Supporting information
No response

Hi @robsaddler, I split this request into two separate issues.
One for ImageResizer: https://github.com/microsoft/PowerToys/issues/32722
Another one for PowerRename: https://github.com/microsoft/PowerToys/issues/32723
Feel free to add any additional details in those issues I made :)


Missing font weight Semibold (600)

We're missing the font weight Semibold = 600

@freenow/wave version: latest

Relevant code
fontWeight: 'semibold' // not working

What was expected to happen?
It should render the text with font-weight:600 but is bolder than it should
Expected size:

Reproduction
If you can, try to reproduce the issue in a Codesandbox. You can fork the one
here: https://codesandbox.io/s/wave-playground-forked-fcv8tm?file=/src/App.js

I'm considering whether we should include the font in the library 🤔 What is your opinion? I think it removes the whole class of font-loading optimisations which are project-specific

I'm considering whether we should include the font in the library 🤔 What is your opinion? I think it removes the whole class of font-loading optimisations which are project-specific

Including the font within the library may limit the ability to optimize font loading, if the font is to be included in the library, can we provide options for users to choose whether to include it or not?


[Feature] onError in Streaming Mode Does Not Support Returning Custom Exceptions

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Apache Dubbo Component
Java SDK (apache/dubbo)
Descriptions
It is expected that the onError method of StreamObserver can pass through custom exceptions to the Consumer. Currently, the onError method in ServerCallToObserverAdapter is as follows:
public void onError(Throwable throwable) {
    final TriRpcStatus status = TriRpcStatus.getStatus(throwable);
    onCompleted(status);
}

This ultimately returns a StatusRpcException.
Related issues
No response
Are you willing to submit a pull request to fix on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@EarthChen PTAL

Yeah, we haven't figured out how to support it yet

@nikbobo I have great interest about this issue, could you assign it to me?

@nikbobo I have great interest about this issue, could you assign it to me?

I cannot assign to anyone, please contact project admin.

@elysium-w Sure, but please write a brief proposal before coding.

I think this requirement is unreasonable. Dubbo's stream is against gRPC. In gRPC, if StreamObserver.onError is called within a stream, the error information is written back to the client via trailers, and the Throwable is not passed. then the stream is closed. A point we can optimization is provide extension that allow users to write additional error information back to the client.


requestProTeamList down

Issue for posterity
Writing tests, CMsgDOTAProTeamListRequest no longer gets a response from GC.
Adding comment to README

That was a quick turn around! :smile:

Yeah I immediately closed it since it's no use leaving it open. I just created it so there'd be a trace in the issues in case someone shows up asking questions :) I'm preparing a pull with some README modifications indicating the stuff that no longer works and the stuff for which the proto's have disappeared. I'm commenting all functions for which the protos are gone and adding comments to all those that still have protos but don't receive responses anymore. I'll also be adding some more tests and elaborating the existing ones so we'll be able to detect this stuff more easily in the future

I'm just joshing you. :P


Weird behavior with offset and buildline after update to 0.4

I experience some weird behavior regarding the offset function in combination with a sketch created with BuildLine.
Problem
(see code snippet below)

GIVEN
a sketch is created with  BuildSketch, Buildline, Polyline and make_face.

WHEN
the sketch is used in another BuildPart  + BuildSketch context by using add

AND
an offset call is executed

THEN
sketch is flipped <-- issue

AND
sketch is bigger by offset amount

Expectations

THEN
sketch stays in same location

AND
bigger by offset amount.

Information
Build123d version: 0.4.0
Happens in earlier version? Not in 0.3.0
Before offset call:

After offset call:

When using the commented out Triangle call which creates exactly the same sketch the behavior is as expected.
My intention with this construction is a little bit more complex and involves multiple objects, but this is the minimal code snippet I could reproduce the issue with.
This issue currently stops me from using 0.4.0
Code snippet.
from build123d import *

with BuildSketch() as profile_a:
    # Triangle(a=2, b=2, C=90, align=Align.MIN)
    with BuildLine():
        Polyline(
            (0, 0),
            (2, 2),
            (2, 0),
            close=True,
        )
    make_face()

with BuildPart() as part, BuildSketch(Plane.XZ) as profile, Locations((10, 0)):
    add(profile_a)
    offset(amount=1, kind=Kind.INTERSECTION)

I am able to reproduce the above on build123d release 0.4, but not able to reproduce on a newer version -- 0.4.1.dev39+g659ce3a nor 0.4.1.dev50+g8632e40 (latest commit). I used this slightly modified snippet to avoid empty buildpart warnings:
with BuildSketch() as profile_a:
    # Triangle(a=2, b=2, C=90, align=Align.MIN)
    with BuildLine():
        Polyline(
            (0, 0),
            (2, 2),
            (2, 0),
            close=True,
        )
    make_face()

with BuildPart() as part:
    with BuildSketch(Plane.XZ) as profile:
        with Locations((10, 0)):
            add(profile_a)
            offset(amount=1, kind=Kind.INTERSECTION)
    Box(.1,.1,.1)

I can can confirm the issue does not exist on the latest commit. I'll wait for a new release. Thanks for the help.


Contact form ignores theme

If you bring in the dark background from new-theme, the contact form questions are black on black. (Each input is green on black when given focus.)
I haven't yet worked out how to fix this. E.g. I tried adding this in the singleLineInput styles block:
color: theme.main.colors.text,

but it makes no difference.

Thank you @DarrenCook
First, there is a bug in theme.js (in new-theme.js also)
  palette: {
    primary: {
      main: colors.accent
    }
  },
  typography: {
    fontFamily: `Arial, sans-serif`,
    fontSize: 16
  },
  pallete: {
    action: {
      hover: "rgba(0, 0, 0, 0.01)"
    }
  }

the pallete property is a misspelled palette so it should be removed and its inner properties have to be moved to the palette, like below
  palette: {
    primary: {
      main: colors.accent
    },
    action: {
      hover: "rgba(0, 0, 0, 0.01)"
    }
  },
  typography: {
    fontFamily: `Arial, sans-serif`,
    fontSize: 16
  }

and now to fix the form's colors for the dark new-theme.js we have to add a new inner property to palette type: "dark", the object should look like below.
  palette: {
    primary: {
      main: "#709425"
    },
    type: "dark",
    action: {
      hover: "rgba(0, 0, 0, 0.01)"
    }
  },

with that the form will look as it should, more about that in Material UI docs https://material-ui-next.com/customization/themes/
I will update the starter's code and the article (https://forgatsby.greglobinski.com/customize-personal-blog-starter/) within a few days.

These changes to the new-theme.js &/or theme.js make the contact form inputs hidden.

@doakheggeness
What do you mean by "make the contact form inputs hidden"?

The form was blank. except for the 'Send' button on load. If you clicked in an area, for instance where the 'name' should be displaying, then it would act normally ( name minimizing, etc.) and be visible. It appears the style for the inputs must be #fff, matching the background?? When I changed the two stylesheets back to the previous state, everything displayed as normal. I am a newby to React, so I am slowing getting the hang of of.

@doakheggeness
I suggest reading the Material UI v. 1.0 docs about theming.  https://material-ui-next.com/customization/themes/

Fixed with version 1.1.0


⚠ ClamAV has degraded performance

In 6c97412, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in b7b0071.


nix: logically split packaging to not be flake-dependent

Nix cleanup!

I added a runtime check that makes the program panic if the helper path does not exist as it needs it anyways.


You can resolve it by calling properties on the Amount object, for example, if str(Amount) is 100.001 STEEM, then Amount().amount will be 100.001 and Amount().asset will be STEEM.
Try it:
steem.get_balances("clayop")['balance'].amount

Thanks!

.amount is a float


Update Microbuild version

This is a temporary fix to enable us to continue building in Microbuild
This is becoming a constant issue. I will look into other ways so that we dont depend on the version number. I think Roslyn does it but it is a little complicated.
Tagging @srivatsn @davkean for review

Yes please let's find a way to avoid this. I'd like to understand why this occurs.
:+1:


Use native embedded backend for Windows

add the download of it at build time
change limitations/prerequisites in readme
enable windows in test matrix

using native executable from today build got from Github wrokflow artifact, I have this error in debug console log when playing tests and also when trying manually:
[22928:1129/132848.724:INFO:CONSOLE(0)] "The FetchEvent for "http://localhost:8081/v1/capabilities?namespace=default" resulted in a network error response: the promise was rejected.", source:  (0)
[22928:1129/132848.724:INFO:CONSOLE(1)] "Uncaught (in promise) TypeError: Failed to fetch", source: vscode-webview://0disj3dn98e46hg58c504jgu64gapd88jbb73uod3k9f3dkfg25d/service-worker.js?v=4&vscode-resource-base-authority=vscode-resource.vscode-cdn.net&remoteAuthority= (1)
[22928:1129/132853.823:INFO:CONSOLE(0)] "The FetchEvent for "http://localhost:8081/v1/capabilities?namespace=default" resulted in a network error response: the promise was rejected.", source:  (0)
[22928:1129/132853.823:INFO:CONSOLE(1)] "Uncaught (in promise) TypeError: Failed to fetch", source: vscode-webview://0disj3dn98e46hg58c504jgu64gapd88jbb73uod3k9f3dkfg25d/service-worker.js?v=4&vscode-resource-base-authority=vscode-resource.vscode-cdn.net&remoteAuthority= (1)
[22928:1129/132858.919:INFO:CONSOLE(0)] "The FetchEvent for "http://localhost:8081/v1/capabilities?namespace=default" resulted in a network error response: the promise was rejected.", source:  (0)
[22928:1129/132858.919:INFO:CONSOLE(1)] "Uncaught (in promise) TypeError: Failed to fetch", source: vscode-webview://0disj3dn98e46hg58c504jgu64gapd88jbb73uod3k9f3dkfg25d/service-worker.js?v=4&vscode-resource-base-authority=vscode-resource.vscode-cdn.net&remoteAuthority= (1)
[22928:1129/132903.976:INFO:CONSOLE(0)] "The FetchEvent for "http://localhost:8081/v1/capabilities?namespace=default" resulted in a network error response: the promise was rejected.", source:  (0)
[22928:1129/132903.976:INFO:CONSOLE(1)] "Uncaught (in promise) TypeError: Failed to fetch", source: vscode-webview://0disj3dn98e46hg58c504jgu64gapd88jbb73uod3k9f3dkfg25d/service-worker.js?v=4&vscode-resource-base-authority=vscode-resource.vscode-cdn.net&remoteAuthority= (1)
[22928:1129/132909.068:INFO:CONSOLE(0)] "The FetchEvent for "http://localhost:8081/v1/capabilities?namespace=default" resulted in a network error response: the promise was rejected.", source:  (0)
[22928:1129/132909.068:INFO:CONSOLE(1)] "Uncaught (in promise) TypeError: Failed to fetch", source: vscode-webview://0disj3dn98e46hg58c504jgu64gapd88jbb73uod3k9f3dkfg25d/service-worker.js?v=4&vscode-resource-base-authority=vscode-resource.vscode-cdn.net&remoteAuthority= (1)
[22928:1129/132914.158:INFO:CONSOLE(0)] "The FetchEvent for "http://localhost:8081/v1/capabilities?namespace=default" resulted in a network error response: the promise was rejected.", source:  (0)
[22928:1129/132914.159:INFO:CONSOLE(1)] "Uncaught (in promise) TypeError: Failed to fetch", source: vscode-webview://0disj3dn98e46hg58c504jgu64gapd88jbb73uod3k9f3dkfg25d/service-worker.js?v=4&vscode-resource-base-authority=vscode-resource.vscode-cdn.net&remoteAuthority= (1)

the problem was between the computer and the chair...
The test is now working locally on Windows too!

Is it still in Blocked / hold state?

still blocked.
it requires https://github.com/KaotoIO/kaoto-backend/pull/348 to be merged and then a release of kaoto-backend to move forward on VS Code Kaoto side


Feature Request:  Discord Notifications for Unmatched Assets

It would be super helpful to be able to get a discord notification for unmatched assets rather than have to regularly check the log.  Since it's regularly a small list for most, being able to quickly reference the notification to stay on top of them would be very beneficial.

The reason this was never added was because not everybody has a small list and this could be thousands of entries posted to discord (mines for example). Even if this was an option to add and to make it make sense for all involved. What would be your idea for this?

The reason this was never added was because not everybody has a small list and this could be thousands of entries posted to discord (mines for example). Even if this was an option to add and to make it make sense for all involved. What would be your idea for this?

My idea would be to have a discord notification similar to what I get from poster_renamerr.  Wouldn't discord just turn the message into a downloadable file when it's over the character limit?

Unfortunately when dealing with discord and it's webhooks if the message is over the character limit it will simply just kick back an error and do nothing.

Unfortunately when dealing with discord and it's webhooks if the message is over the character limit it will simply just kick back an error and do nothing.

In there anyway to integrate with a pastebin type of service so it just posts a link?

Not that I'm aware of. However if somebody wants to submit a PR in order to make this addition I'm not opposed to it especially if it was a configurable option.

Maybe something worth looking into when I get the time. My life is very hectic right now I haven't even opened my laptop in quite a while. But I'm for sure not opposed to it.


317 refactorization of mapieregressor and conformityscore

Description
The ConformityScore class called by MapieRegressor does not allow the scores to be calculated from X (only from y_pred).
To solve this problem a refactorization of MapieRegressor has been done : MapieRegressor now calls the EnsembleEstimator (defined in regression/estimator.py) that handle the training and usage of the estimator. The ConformityScore class is also modified : it now takes X and EnsembleEstimator in parameters.
Finally, the quantile calculation is intuitive according to the method (plus, base, minmax).
Fixes #317
Type of change
Please remove options that are irrelevant.

New feature (non-breaking change which adds functionality)
This change requires a documentation update

How Has This Been Tested?

[x] Unit testing of regression and conformity_scores
[x] Check the classes depending on MapieRegressor (time_series, quantile_regression)

Checklist

[x] I have read the contributing guidelines
[ ] I have updated the HISTORY.rst and AUTHORS.rst files
[x] Linting passes successfully : make lint
[x] Typing passes successfully : make type-check
[x] Unit tests pass successfully : make tests
[x] Coverage is 100% : make coverage
[ ] Documentation builds successfully : make doc

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (ed5c431) 100.00% compared to head (3f42406) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #325   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           32        33    +1
  Lines         3774      3860   +86
  Branches       406       410    +4
=========================================
+ Hits          3774      3860   +86

Impacted Files
Coverage Δ

mapie/conformity_scores/conformity_scores.py
100.00% <100.00%> (ø)

...ie/conformity_scores/residual_conformity_scores.py
100.00% <100.00%> (ø)

mapie/regression/__init__.py
100.00% <100.00%> (ø)

mapie/regression/estimator.py
100.00% <100.00%> (ø)

mapie/regression/regression.py
100.00% <100.00%> (ø)

mapie/regression/time_series_regression.py
100.00% <100.00%> (ø)

mapie/tests/test_common.py
100.00% <100.00%> (ø)

mapie/tests/test_conformity_scores.py
100.00% <100.00%> (ø)

mapie/tests/test_regression.py
100.00% <100.00%> (ø)

mapie/tests/test_time_series_regression.py
100.00% <100.00%> (ø)

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


SDKMAN is able to install on Java on ARM

We require a conversation to take place in the appropriate Gitter Room before raising a new Github Issue. Please note that issues will be closed immediately if prior discussion did not take place. We need to do this to help manage the quality and validity of Issues raised on this project.
Please tick one:

[ ] this is a new Feature Request, a conversation was started in the Gitter CLI Development Room.
[x] this is a new Bug Report or Issue, a conversation was started in Gitter User Issues Room.

Please explain the Issue / Feature Request here:
Currently, I am able to use sdkman and install java for the wrong architecture on a raspberry pi.  It looks like it isn't doing some detection for the raspberry pi to prevent the install and is downloading an x86 64 bit version of java.
See error below:
pi@raspberrypi:~ $ sdk install java

Downloading: java 8.0.163-zulu

In progress...

######################################################################## 100.0%

Repackaging Java 8.0.163-zulu...

Done repackaging...

Installing: java 8.0.163-zulu
Done installing!

Do you want java 8.0.163-zulu to be set as default? (Y/n): Y

Setting java 8.0.163-zulu as default.
pi@raspberrypi:~ $ java -version
-bash: /home/pi/.sdkman/candidates/java/current/bin/java: cannot execute binary file: Exec format error
pi@raspberrypi:~ $ which java
/home/pi/.sdkman/candidates/java/current/bin/java
pi@raspberrypi:~ $ file
Usage: file [-bcEhikLlNnprsvzZ0] [--apple] [--extension] [--mime-encoding] [--mime-type]
            [-e testname] [-F separator] [-f namefile] [-m magicfiles] file ...
       file -C [-m magicfiles]
       file [--help]
pi@raspberrypi:~ $ file /home/pi/.sdkman/candidates/java/current/bin/java
/home/pi/.sdkman/candidates/java/current/bin/java: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 2.6.18, BuildID[sha1]=f1ac5b47218b2ffcda98cb143d7f9fc431f3574e, not stripped

specific version of sdkman cli: 'SDKMAN 5.6.3+299'

I think the main problem here is that we are currently interpreting the output of uname incorrectly. Would you be able to share the output of uname on your pi please? We also don't have any Java versions available for ARM right now, but let's solve that after this is resolved.

@marc0der I intended to do that last night, I'll make sure to get that output for you when I get home this evening.

Bump for armv6 for Raspberry Pi Zero W.
pi@raspberrypi:~ $ uname -a
Linux raspberrypi 4.14.71+ #1145 Fri Sep 21 15:06:38 BST 2018 armv6l GNU/Linux

@marc0der, would a pr help get this done sooner?  If so I'd be happy to assist.

Since we don't support the arm architecture, this will have wide reaching effects in in the sdkman-hooks project as well as in db migrations. But sure, feel free to raise PRs.

need support
/home/pi/.sdkman/candidates/groovyserv/current/bin/groovyclient: cannot execute binary file: Exec format error

Can't this issue be closed because of PR #769 ?


upload svg

@Ameypatil07 @Zein-Zeus Need some more work before we can merge. Let's connect on call.


SwitchCondition with number throws an exception when compared against a string

Fixes #4071
Description
SwitchCondition treated errors as exceptions when all that was happening was an comparison which was false. In addition, it was always assuming that a quoted number should be compared as a number.
Specific Changes

Fix switch to not treat expression errors as exceptions but instead as false condition
Fix switch to treat ambiguity in string case definition more robustly by comparing for both value type and string of value type. (23 OR "23")

Testing
updated unit test cases to cover this.

I created some corners cases for the parsing approach here https://github.com/xieofxie/botbuilder-dotnet/commit/3f0228fac7d17c8498ac7c429cb3825f762b3bff

case 21 will capture 21.0, so the exact match case 21.0 will be skipped
case 'true   ' will capture true, so the exact match 'true' will be skipped

I am in favor of use object for Case schema and keep its original data type.
I also notice that we don't parse double, so case 21.0 will not work.
Thanks

I've kicked off the fail builds again.


Increased memory limits for requests

We need to increase memory limits to have working apps
in ephemeral environment which is used by pr checks.
Current memory limits in ephemeral are causing OOM kills of the workers.
Solution is to not remove apps resources in bonfire deploy command.
See https://github.com/RedHatInsights/historical-system-profiles-backend/pull/293
and https://github.com/RedHatInsights/system-baseline-backend/pull/316
Unfortunatelly these PRs means that hsps' non API services
does not comply with the MaxLimitRequestRatio in ephemeral env.
This will solve that :-)
Secure Coding Practices Checklist Link

https://github.com/RedHatInsights/secure-coding-checklist

Secure Coding Checklist

[ ] Input Validation
[ ] Output Encoding
[ ] Authentication and Password Management
[ ] Session Management
[ ] Access Control
[ ] Cryptographic Practices
[ ] Error Handling and Logging
[ ] Data Protection
[ ] Communication Security
[ ] System Configuration
[ ] Database Security
[ ] File Management
[ ] Memory Management
[x] General Coding Practices

/retest

/retest


Question on CloudNative-PG Upgrade

When you have a moment, I tried to upgrade CNPG from 0.18.2 (1.20.2 / PG 14.10) to either 0.19.1 or 0.20.1, while the coordinator upgrades just fine, the cluster itself does not.  I would get messages such as:
invalid: spec.imageName: Invalid value: "ghcr.io/cloudnative-pg/postgresql:16.0": can't upgrade between ghcr.io/cloudnative-pg/postgresql:14.10 and ghcr.io/cloudnative-pg/postgresql:16.0

So I changes the various name: postgres to name: postgres16.  I have barmanObjectStore configured much like yours to MinIO running on TrueNAS backing up to postgres14-v1 and my cluster16.yaml was set to use a new postgres16-v1.
Upon trying to create the postgres16 cluster the Job postgres16-1-full-recovery reports good things like:
"msg":"Recovering from external cluster"
"msg":"Target backup found"
"msg":"Starting barman-cloud-restore"
"msg":"Restore completed"
....
"msg":"Starting up instance"

But fails with  The data directory was initialized by PostgreSQL version 14, which is not compatible with this version 16.1:
16:23:57.467 UTC [38] FATAL:  database files are incompatible with server","pipe":"stdout","logging_pod":"postgres16-1-full-recovery"}
{"level":"info","ts":"2024-02-05T16:23:57Z","logger":"pg_ctl","msg":"2024-02-05 16:23:57.467 UTC [38] DETAIL:  The data directory was initialized by PostgreSQL version 14, which is not compatible with this version 16.1 (Debian 16.1-1.pgdg110+1).","pipe":"stdout","logging_pod":"postgres16-1-full-recovery"}

I went through the various release notes and upgrade docs and it seems pretty straight forward, so don't understand why this happens (I'm not PostgreSQL expert),  not seeing something specific in the upgrade steps needed.
Anyway, I put the names back and went back to cluster14.yaml and the restore from MinIO via barmanObjectStore worked perfect, now using postgres14-v2.  I did notice none of the restored accounts had incorrect passwords, had to manually fix that and then my applications were back on-line.
I tried to review your git history on your PG cluster files for clues on how you handled this, didn't find it.  I also found this thread where you previously had a restore failure (different error) https://github.com/cloudnative-pg/cloudnative-pg/discussions/1163
I'm wondering if the barmanObjectStore cannot be used for version upgrades and you have to some kind of manual pg_dump backup and restore with cloudnative-pg?

This was discussed in our discord
https://discord.com/channels/673534664354430999/1159134548274188390/1172918476839604435
https://github.com/Rodent1/home-ops/blob/0cfe2c9ede87f182b5362c177a1b25d1edbe1516/kubernetes/apps/database/cloudnative-pg/cluster/cluster16.yaml I used this thing
Specifically these parts

```yaml
  bootstrap:
    initdb:
      import:
        type: monolith
        databases: ["*"]
        roles: ["*"]
        source:
          externalCluster: cnpg-6

and
  externalClusters:
    - name: cnpg-6
      connectionParameters:
        # Use the correct IP or host name for the source database
        host: postgres-rw.database.svc.cluster.local
        user: postgres
        dbname: postgres
        sslmode: require
      password:
        name: cloudnative-pg-secret
        key: password

While the pg14 cluster was still running
Ref: https://cloudnative-pg.io/documentation/current/database_import/

You basically want to spin up a new cluster side by side the old one and then have cnpg bootstrap the new one from the old one. Make sure the old database is not being used by any workloads first.


help-center-stage-2: Add support for relative links.

We want to have similar functionality to the relative links in help_relative_links.py for our new astro documentation.
It is recommended to discuss implementation plan before working on this issue.
Parent issue: #31247
It might make more sense for whoever's working on this to work on #31254 and vice versa. Either can be worked on first.

Syntax discussion ongoing in https://chat.zulip.org/#narrow/stream/19-documentation/topic/New.20syntax.20for.20settings.20and.20relative.20links.20for.20help-beta/near/1914323


Reset histogram widget when user clicks over CLEAR button

There is a problem with the CLEAR button, and the steps for replicating it are in #258 ticket. We have created a function to reset the state of the histogram widget (origin of the problem). How does it sound @javierarce?
CR: @javierarce.
cc @javisantana
Fixes #258.

:+1:  LGTM


More i18n to gauth and mfa register device views

Make more strings translateable in gauth registration and mfa register device views.

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

What kind of test should I write for this kind of change?


"Syntax error: import not found" in dev build only

Describe the bug
After installing leaflet and vue2-leaflet (and trying to use the component), npm run build works fine but the hmr dev server does not load the page correctly, and the following message is shown in the browser console:
Uncaught SyntaxError: import not found: setOptions utils.js:1:9
For reference, this is the offending line in utils.js:
import { setOptions } from '/node_modules/leaflet/dist/leaflet-src.js';
I've tried checking for duplicates but as I'm not an expert in the area of bundling it's hard to tell for sure if it's a duplicate of something else or not.
Reproduction

Start off with the vue2 template from here: https://www.mathew-paul.nz/posts/how-to-use-vue2-with-vite/
npm i leaflet vue2-leaflet

Change app.vue to:
<template>
<div>
	<l-map style="height: 500px; display: block"></l-map>
</div>
</template>

<script>
import { LMap } from 'vue2-leaflet';

export default {
components: {LMap}
};
</script>

System Info
System:
    OS: Linux 5.11 Ubuntu 20.04.3 LTS (Focal Fossa)
    CPU: (16) x64 AMD Ryzen 7 1700 Eight-Core Processor
    Memory: 5.95 GB / 15.58 GB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Node: 16.13.0 - /usr/bin/node
    npm: 8.1.0 - /usr/bin/npm
  Browsers:
    Firefox: 94.0
  npmPackages:
    @vitejs/plugin-vue: ^1.9.3 => 1.9.4
    vite: ^2.6.4 => 2.6.14

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I can't reproduce by following your steps. My dev server works fine.

I can't reproduce it either now. Sorry for the noise.


'Request access' form UX

The form to request access doesn't provide adequate feedback ('invite code sent', 'check your email' or similar). The 'Sign in' screen mentions a waitlist, so if I'm being put on one I'd expect to be notified of it when/after submitting the form.
Why do I believe this to be an issue? Not having received an email after submitting the form and the feedback being a bit vague, I tired a couple times more before realizing there's a waitlist. Aside from the UX implications, this sounds like it could impact invite code distribution as well.

If users are not receiving any email or notifications after submitting the form or being directed to a waitlist, it would cause confusion that there is some error at their side. This would lead them to submit multiple forms which could negatively impact the invite code distribution.
To address this issue, it may be helpful to review the feedback provided on the form. Consider providing messages that the request has been received and make them aware of the waitlist. Additionally, providing an estimated wait time could also prevent users from multiple submissions..


Remove Java and JS builds and intergration tests from workflow

Which area this feature is related to?
/area test-automation
Issue details
We need to have more clear workflows without any tests and builds for the two deprecated implementations - Javascript and Java.
Target date: 16 May 2023

pr https://github.com/redhat-developer/alizer/pull/192 will close issues #172 and #173


Always initialize variable.

Some compilers complain about potentially uninitialized usage otherwise.

Closing because Kurento 7 drops need for this custom fork.


tooltips to explain some of the attribute names

Some attribute names in the data sets are not very clear. There should be a way to provide the definition/description of some attributes.
This feature is requested by GPCR, but it should be useful for all projects.

@robes this task seems to be well contained as well, if you want to assign this to students?

right, this could be another good student task.

Do you have any student that could work on this? This feature will be used across all projects though, and might need close guidance from @robes.

Anoop interviewed some and has someone he thinks is good.
Carl
Sent from my iPhone
On Mar 4, 2016, at 4:53 PM, Hongsuda <notifications<EMAIL_ADDRESS>wrote:
Do you have any student that could work on this? This feature will be used across all projects though, and might need close guidance from @robeshttps://github.com/robes.

Reply to this email directly or view it on GitHubhttps://github.com/informatics-isi-edu/chaise/issues/219#issuecomment-192540021.

Here is Rajbharath's resume.   He was close second for the software
developer position.
On Fri, Mar 4, 2016 at 5:41 PM, Carl Kesselman<EMAIL_ADDRESS>wrote:

Anoop interviewed some and has someone he thinks is good.
Carl
Sent from my iPhone
On Mar 4, 2016, at 4:53 PM, Hongsuda<EMAIL_ADDRESS>wrote:
Do you have any student that could work on this? This feature will be used
across all projects though, and might need close guidance from @robes
https://github.com/robes.
—
Reply to this email directly or view it on GitHub
https://github.com/informatics-isi-edu/chaise/issues/219#issuecomment-192540021
.[image: Web Bug from
https://github.com/notifications/beacon/ADbjXtcNevC3-k1h4bTmu76KKeeh3_hHks5pqNPogaJpZM4HnHi9.gif]

Thanks.
I would say that we look at him, and if he looks good, that we consider bringing onboard ASAP to do things like logging dashboard, tooltips (I have some more ideas about how we should to this), and other discrete tasks.
Carl

Dr. Carl Kesselman
Dean’s Professor, Epstein Department of Industrial and Systems Engineering
Fellow, Information Sciences Institute
Viterbi School of Engineering
Professor, Preventive Medicine
Keck School of Medicine
University of Southern California
4676 Admiralty Way, Suite 1001, Marina del Rey, CA  90292-6695
Phone:  +1<PHONE_NUMBER>
Email:  carl<EMAIL_ADDRESS>Web: http://www.isi.edu/~carl
On Mar 5, 2016, at 10:24 AM, Anoop Kumar <anoopk<EMAIL_ADDRESS>wrote:
Here is Rajbharath's resume.   He was close second for the software developer position.
On Fri, Mar 4, 2016 at 5:41 PM, Carl Kesselman <carl<EMAIL_ADDRESS>wrote:
Anoop interviewed some and has someone he thinks is good.
Carl
Sent from my iPhone
On Mar 4, 2016, at 4:53 PM, Hongsuda <notifications<EMAIL_ADDRESS>wrote:
Do you have any student that could work on this? This feature will be used across all projects though, and might need close guidance from @robeshttps://github.com/robes.
—
Reply to this email directly or view it on GitHubhttps://github.com/informatics-isi-edu/chaise/issues/219#issuecomment-192540021.
<Rajbharath_Rajendran.pdf>

I've updated the issue summary.  Should we generalize this whole issue to having tooltips drive by the ERMrest schema comments?  Or should we keep it specific to column display for now?

I think general is a good idea.
One think that I’ve been thinking of is that should we be considering column hames to be like a controlled vocabulary which has a description field as well as synonyms.  The same tooltip type behavior (and search behavior for that matter) you want on therms, you probably want for columns too.
Carl

Dr. Carl Kesselman
Dean’s Professor, Epstein Department of Industrial and Systems Engineering
Fellow, Information Sciences Institute
Viterbi School of Engineering
Professor, Preventive Medicine
Keck School of Medicine
University of Southern California
4676 Admiralty Way, Suite 1001, Marina del Rey, CA  90292-6695
Phone:  +1<PHONE_NUMBER>
Email:  carl<EMAIL_ADDRESS>Web: http://www.isi.edu/~carl
On Mar 7, 2016, at 2:25 PM, Karl Czajkowski <notifications<EMAIL_ADDRESS>wrote:
I've updated the issue summary. Should we generalize this whole issue to having tooltips drive by the ERMrest schema comments? Or should we keep it specific to column display for now?
—
Reply to this email directly or view it on GitHubhttps://github.com/informatics-isi-edu/chaise/issues/219#issuecomment-193483488.

Please clarify. Whatever is in the comment of the column definition is the tooltip?

Yes, I think the comment of the column definition is the tooltip for instances of the column name in table, list, or facet views.
@robes, @hongsuda It could also serve as the default tooltip for values of that column, if there is not a better value-specific tooltip source available...?

@shinyichen right
@karlcz i suppose we could consider that as long as it doesn't inadvertently introduce more confusion

Can this get reviewed, I'm sure this was done a long time ago.

Right.. This is done in record app. I am closing the issue.


Potential Problem with CC: ActionMailer Parameter

Hi, I am using sparkpost_rails 1.4.0 with rails 4.2.8
When I try to include an Ruby Array of Emails for the "cc" parameter for  Action Mailer, I receive this error from SparkPost during delivery:
{"message"=>"invalid data format/type", "description"=>"content.headers json dictionary value has type 'json_array' when expecting type 'string'", "code"=>"1300"}

However, when I use the same exact array for a "bcc" parameter, the mail successfully goes through and the BCC work as I would expect.
Could there be a problem in how these headers are constructed?
Thanks in advance.

Any news about this issue? I am having the same problem here using the recipient_interceptor gem, because it puts a empty array on CC parameter for mail message.

I've got the same issue. Bcc works, but cc doesn't


Add Hincrby command

Related to #157

Add tests.
Add a server command for hincrby.

Please review when you have time!

Sorry for missing these small details lately.

Don't worry about it!
Sorry I wasn't paying attention for the naming convention of HIncrBy.
There were others, so I fixed them.
Thank you for your review:smile:


Create collections_seeds.csv file

The rake process for importing webvtt files requires a collection be defined on transcripts in the webvtt import rake task. These are imported using collections_seeds.csv. Required attributes are uid and title. uid should match the collection on the transcripts_seeds.csv file.

[ ] Done When a collections_seeds.csv file is created for our initial import.

Note: In loading the Collections we found that the Collections dropdown on the site doesn't scroll. Adding an additional done when for addressing the UI issue.

@sroosa - does this look OK for Collections scroll?

@jasoncorum yes!


Update danish.nsi

Codecov Report

Merging #4 into topic/refactor-installer will decrease coverage by <.01%.
The diff coverage is n/a.

@@                     Coverage Diff                      @@
##           topic/refactor-installer       #4      +/-   ##
============================================================
- Coverage                     77.37%   77.36%   -0.01%
============================================================
  Files                            99       99
  Lines                        139350   139350
============================================================
- Hits                         107826   107814      -12
- Misses                        31524    31536      +12

Impacted Files
Coverage Δ

src/gui.c
57.08% <0%> (-0.52%)
:arrow_down:

src/gui_gtk_x11.c
48.01% <0%> (-0.5%)
:arrow_down:

src/channel.c
83.14% <0%> (-0.12%)
:arrow_down:

src/os_unix.c
57.34% <0%> (-0.09%)
:arrow_down:

src/screen.c
79.17% <0%> (-0.07%)
:arrow_down:

src/term.c
60.36% <0%> (-0.06%)
:arrow_down:

src/syntax.c
78.42% <0%> (-0.03%)
:arrow_down:

src/ex_cmds2.c
84.54% <0%> (+0.09%)
:arrow_up:

src/window.c
83.58% <0%> (+0.09%)
:arrow_up:

src/if_xcmdsrv.c
84.17% <0%> (+0.35%)
:arrow_up:

... and 2 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 7019075...bfd3eb6. Read the comment docs.

How about now?

Thank you. LGTM.


translateY isn't working in react-native-web

Bug
Warning: React does not recognize the `translateY` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `translatey` instead. If you accidentally passed it from a parent component, remove it from the DOM element.
    at svg
    at Svg (webpack://mobile/./node_modules/react-native-svg/lib/module/ReactNativeSVG.web.js?:1:17591)

React native info output:
System:
  OS: macOS 14.2.1
  CPU: (8) arm64 Apple M1
  Memory: 96.83 MB / 8.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 21.4.0
    path: /opt/homebrew/bin/node
  Yarn:
    version: 1.22.21
    path: /opt/homebrew/bin/yarn
  npm:
    version: 10.2.4
    path: /opt/homebrew/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.14.3
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK:
    API Levels:
      - "28"
      - "30"
      - "31"
      - "33"
      - "33"
      - "34"
    Build Tools:
      - 30.0.3
      - 31.0.0
      - 33.0.0
      - 33.0.1
      - 33.0.2
      - 34.0.0
    System Images:
      - android-30 | Google Play Intel x86 Atom
      - android-31 | Google Play ARM 64 v8a
      - android-34 | Google Play ARM 64 v8a
    Android NDK: Not Found
IDEs:
  Android Studio: 2022.3 AI-223.8836.35.2231.10406996
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.1
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.2
    wanted: 0.73.2
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

Library version: 14.1.0

Hello @gavrilikhin-d,
Can you provide an example to reproduce that issue?
Thank you.


push packagecloud versions for linuxmint/{rebecca,rafaela,rosa}

This is a request for releng to push packages to packagecloud for at least linuxmint/rebecca, linuxmint/rafaela and linuxmint/rosa, the current linuxmint LTS releases.  Those are also the push-string specs required.  I imagine that these packages would be identical to their ubuntu/trusty upstream equivalents.
The current git-lfs/script.deb.sh will create repo specs such as:
deb https://packagecloud.io/github/git-lfs/linuxmint/ rebecca main
deb-src https://packagecloud.io/github/git-lfs/linuxmint/ rebecca main

Currently, these have to be changed to:
deb https://packagecloud.io/github/git-lfs/ubuntu/ trusty main
deb-src https://packagecloud.io/github/git-lfs/ubuntu/ trusty main

... in order for installation to be able to proceed.
linuxmint remains the most popular distribution, so it makes sense to push debs for it too.

AFAIK the same Ubuntu packages should work on Mint.

@andyneff Correct.

I'm going to open a conversation with packagecloud about how the same package might be able to be shared across distro and perhaps distro-versions without duplicating the uploads, and still being able to use their nice install repo-setup script.

@javabrett That sounds great. :metal:

Just to be clear, the motivation here is: packagecloud host the installer package repos, and provide neat scripts to help those on Linux who are less familiar with their (apt, rpm/Yum) package-manager and repo URLs.  These scripts are script.deb.sh, script.rpm.sh etc.
The problem is that these resolve specifically the installer's exact distribution and version (scripts call these os and dist respectively, somewhat confusingly to me anyway), so on say LinuxMint, these will automatically resolve to e.g. os=linuxmint, dist=rebecca.  If we don't upload such a package, the script succeeds, creates an invalid deb URL as above, then the install fails.  Presumably the same applies for ol ~= el,, centos ~= el etc.
I contacted packagecloud.  If we don't want to upload duplicate packages for compatible releases (per my original proposal, now seeming less-good to me), then the best workaround at the moment is to either not use the script and add the deb manually, then manually do the remaining required steps such as installing the signing-key etc., or if you want to use the script, you can pass it a compatible (upstream) distribution-name (as os), e.g. on LinuxMint/Rebecca curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.deb.sh | os=ubuntu dist=trusty sudo -E bash.
I guess for now we could upload each package once only, named for the upstream-most distribution+version it is compatible-with, then name in install-documentation all of the compatible downstream distributions/versions that should use that as an override, and provide the above command.  There might be exceptions - for example, even if for a release debian==downstream-ubuntu, Ubuntu might be considered popular/important-enough to create a proper upload for.  LinuxMint is pretty popular too though.
So to whoever is cutting/uploading the next release: do you have the map of number of package-versions, and which distribution-version combinations that are compatible, or suspected-compatible with each?  That would then help everyone on e.g. LinuxMint to get the command right and pick-up either the Ubuntu or Debian repos.
If that makes sense and with the information above, I can propose install-doc changes and close #1076.

do you have the map of number of package-versions, and which distribution-version combinations that are compatible, or suspected-compatible with each?

It's tracked here:
https://github.com/github/git-lfs/blob/9c7155dc780b47b704d21309c12eceef797fc893/script/packagecloud.rb#L21
I don't mind uploading the same packages multiple times. But, it sounds like people can install the PackageCloud repo by running the deb script with hard coded os/dist values. Once installed, future package updates through apt or yum will just work for Fedora, Linux Mint, and Ubuntu, even though the original packages are uploaded to PackageCloud once, for Enterprise Linux and Debian.
That sounds good to me :+1:

It's tracked here:
https://github.com/github/git-lfs/blob/9c7155dc780b47b704d21309c12eceef797fc893/script/packagecloud.rb#L21

Yep, I've got that map, and so originally proposed the additions to it with https://github.com/github/git-lfs/pull/1076/files#diff-b75c3a11a4c6f4186a87ff378b668101R41 .  The current maps supposes that Ubuntu is common-enough that it is worth the duplicated upload, 'fair nuff.
The current rough-edge is that if you assume that many folks installing on Linux arrive at https://git-lfs.github.com/ , see the enormous packagecloud install button offering a package-based install (which most folks will prefer as you say for the reason of auto-upgrades etc,), they arrive at https://packagecloud.io/github/git-lfs/install a fairly plain page but which offers the helpful curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.deb.sh | sudo bash for quick-install.
There's a couple of rough-edges with that:

It doesn't actually install git-lfs, it only installs the repo.  You have to follow-up with sudo apt-get install git-lfs to install the LFS client.  The packagecloud script alerts you to this with The repository is setup! You can now install packages., but not every person will immediately realise that they haven't in-fact installed git-lfs.
Unless you are on one of the exact dists/releases (os/dist) mentioned in the upload-map, or you twig to the fact that you are on LinuxMint and read the script and realise that you are going to end-up with a bad deb line unless you run something like curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.deb.sh | os=ubuntu dist=trusty sudo -E bash, adding the os=ubuntu dist=trusty ... -E, you will end-up with an invalid repository.  Doesn't feel like a lot of folks who haven't used packagecloud before will twig to this first-go.

I guess what I'm saying is that given the sparsity of the instructions on the packagecloud install page (and assuming that doesn't change), to improve the install experience and likelihood of success, we might want to place a gh-page in-between highlighting some of these subtleties.

A separate page sounds great. Would you be able to submit a md document for the git-lfs repo? I can port that to git-lfs.github.com too. I have an internal issue to open source the gh-pages repo, but I probably won't do that before LFS v1.2 ships.

I've been thinking about this, and almost asked another user what they thought. But, I keep coming back to the fact that we're offloading a bit of extra work on the user just because we don't want to upload the same package multiple times.
Better docs is definitely good, though. Even if we upload dup packages, describing how to customize the packagecloud scripts for unmapped distros could help people figure out a workaround.

Yeah I have flip-flopped on uploading duplicates a few times. That task originally prompted me to chase-down the packagecloud ruby gem issue preventing same.
Accepting that bandwidth and storage are unlikely to be an issue at the current size of Git LFS artifacts, the only downside is coverage. There will always be another fringe distro that could use a compatible package but can't because of how the packagecloud helper-script process detects it.  Of course that user is entitled to work out the compatible upstream and update their apt list to reference that deb repo URL.
If we covered all the LTS versions of the top few rpm and apt packaged distros, very few users would need to refer to the "if you are not running RHEL (releases), Fedora (releases), Debian (releases), Ubuntu (Releases) or LinuxMint (Releases), please read this to learn how to install a compatible package".  It's a matter of settling on the reasonable list of compatible duplicates and updating that packagecloud.rb map.

See #1130 for a page I roughed-out.
During this process I was reminded of the consequences of fan-out on distro/release and any subsequent failure to publish those repos or dropping maintenance for them.
When I originally corrected my deb line /linuxmint/ rebecca main -> /ubuntu/ trusty main, I then installed 1.1.1.  I find that I still have that version as my packaged-install, because when 1.1.2 was pushed to packagecloud it only pushed a few releases, either intentionally or unnoticed as a consequence of #1102 or #1103.  There was no ubuntu repo pushed to packagecloud for 1.1.2, only debian made it, so I've not been offered an upgrade since.
Fot that reason, https://github.com/javabrett/git-lfs/blob/installing-doc/INSTALLING.md currently mentions only debian and not ubuntu.

This is now fixed with the release of 1.2.0, thanks!


Question regarding image rendering

In my main application model, I have an property like this: selectedKeys :: Keys - whereas the Keys are only known, once the application gets booted (f.e. reading the keys file from disk). Of course I could make it Maybe Keys instead of providing some hard coded initial keys value, but I don't think that's the problem I have.
When the UI renders first time, I expect it to show a profile image for some dummy initial keys for a fraction of a second.
As soon as the keys are loaded, the correct image should be displayed. However the wrong image gets displayed (the old one).
What's even more strange is, that after the keys are loaded, I grab some messages and display them, there also a profile image should get rendered, but instead I do see nothing at all instead.

Again some video upload (seeing is better than explaining I guess):
https://user-images.githubusercontent.com/394428/174465250-94bf44b9-67bf-4a86-bb40-a574b5a11fa9.mp4
I have some Debug.Trace output where I render the profile image, this is the output:
"https://robohash.org/134bdeaf23fe7078d94b2836dcb748e762073d4bc274a2c188a44a3fc29df31c.png?size=40x40"
"https://robohash.org/18755d9063629e12cfd5a1c03ed58d9f6891297b6577d658373ebe8e1ad590c4.png?size=40x40"
trying to connect to ws://localhost:2700 ...
"https://robohash.org/18755d9063629e12cfd5a1c03ed58d9f6891297b6577d658373ebe8e1ad590c4.png?size=40x40"
Connected to ws://localhost:2700
"https://robohash.org/18755d9063629e12cfd5a1c03ed58d9f6891297b6577d658373ebe8e1ad590c4.png?size=40x40"
"https://robohash.org/18755d9063629e12cfd5a1c03ed58d9f6891297b6577d658373ebe8e1ad590c4.png?size=40x40"
"https://robohash.org/18755d9063629e12cfd5a1c03ed58d9f6891297b6577d658373ebe8e1ad590c4.png?size=40x40"

So what happens here, first time it renders a different image, which is expected, as there is a placeholder, but as soon as the app ist booted, there should be a different image, and from the debug output you see exactly that. The correct image should get rendered a fraction of a second later.
Just in case the problem is within my profile image helper, I'll post the code for it as well:
data ImageSize
  = Big
  | Small

profileImage :: WidgetEvent e => Maybe Text -> XOnlyPubKey -> ImageSize -> WidgetNode s e
profileImage picture xo size = image_ (trace (show path) path) [ fitEither ]
  where
  path = case picture of
    Just p ->
      if p == "" then fallback else p
    Nothing -> fallback
  fallback = do
    let baseUrl = T.pack $ "https://robohash.org/<xo>.png" ++ imageSize size
    let imgUrl x = T.replace "<xo>" (T.pack $ exportXOnlyPubKey x) baseUrl
    imgUrl xo

imageSize :: ImageSize -> String
imageSize Big = "?size=300x300"
imageSize Small = "?size=40x40"

It depends on how this is used. The main thing is making sure the model of the Composite that contains your buttons is updated. If xo is not somehow part of the model, you will have to use mergeRequired to account for it.
Could you point me to the location in your project where you use this?

https://github.com/prolic/futr/blob/master/src/UI.hs#L99 here

I just tried your suggestion and added this to the box:
box_
  [ mergeRequired keysChanged
  , onClick $ if model ^. waitingForConns then NoOp else EditProfile
  ] $ tooltip "Edit Account" $ profileImage pImage xo Small
  `styleBasic` imageButtonStyling
where
    keysChanged wenv old new = old ^. selectedKeys /= new ^. selectedKeys

However nothing changes, same weird behavior.

I tried running the application but I get this:

Is there any way I can force it to start? Maybe by faking the content of the keys file?

Ah sorry, no it expects a localhost relay server. I just pushed a code change to master that enables one that is production, so you don't need to setup anything.

How should I test this?

I created an account and posted something.
Restarted the application, but profile information was not loaded so I input it again.
Pasted an image link, loaded it and then saved it.
The image in the right-most button (Edit Account) was updated correctly.
The image in the previously created post was not updated.

Should the image in the post be updated too?

I don't think you need mergeRequired for these buttons. Maybe you want to have one box that wraps a complete section so you can avoid re-creating it unless the model that section is interested in has changed.
Another thing to consider is that merge is hierarchical. If a parent widget has a box with mergeRequired and its condition returns false, no children will have its merged function called.

I did some refactoring and the problem is gone. Probably some seemingly unrelated bug in how the model received updates. No further investigation is needed.
Thank you very much for your support.


Hide workflow submission page sections when complete

Automatically collapse sections as the user completes them (with option to reopen)
Maybe we should only allow 1 to be open at a time?

Implemented for input/output sections

Closing unless demonstrated necessary


fix bug with failed trade modifying market state

can we use the package-level elfpy.check_non_zero()? I don't see why we'd want duplicates of this function

There was a bug with that one due to minor differences between the wallet and market state, which was fixed in an earlier PR. The fixed moved us over to a different version of check_non_zero in wallet and market. However, now that I'm looking, it seems like those two versions have already diverged, which is a good argument for keeping it in one place. I'll make the switch.

can we use the package-level elfpy.check_non_zero()? I don't see why we'd want duplicates of this function

There was a bug with the original elfpy.check_non_zero() due to minor differences between the wallet and market state, which was fixed in an earlier PR by making two separate functions. However, now that I'm looking, it seems like those two versions have already diverged, which is a good argument for keeping at least part of the check in one place. The recursive nature of it makes this hard. If it lived in elfpy.__init__.py then we would need to import a lot of types (e.g. right now Checkpoint from markets and Quantity from types). This is a bad pattern; we do not want to be importing a bunch of packages into elfpy core. I think for now we should keep them separate and create an issue for unifying the checks. I'll push up an example.

sounds like that's in the direction of the "more localized checks" mentioned here https://github.com/delvtech/elf-simulations/blob/738f4f53ab0782387ba1232344bc615ed1b3fd7b/elfpy/init.py#L201
    # TODO: issue #146
    # this is an imperfect solution to rounding errors
    # and should be removed in favor of localized checks

when I created that check in elfpy, I think we had two identical functions. now we're going forward with more specific function-level checks. sounds good.


Closing atom 1.19 produces many instances node.exe on Windows 8.1 x64

Prerequisites

[ ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Closing atom 1.19 on Windows 8.1 x64 produces many instances node.exe running and consuming all CPU.
In atom --safe mode all is OK.

Steps to Reproduce

Start atom
Close atom

Expected behavior:  closes in accepted time
Actual behavior: it takes 30 secs to  close
Reproduces how often: always since 1.19 version update
Versions
Atom    : 1.19.2
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0
apm  1.18.4
npm  3.10.10
node 6.9.5 x64
python
git
visual studio 2015
Additional Information
Packages list attached
packages.txt

Thanks for the report!

In atom --safe mode all is OK.

If this is the case, then it's likely that the issue is caused by a community package. I don't have any guesses based on your package list but if you disable all your community packages and then re-enable them 1 at a time, that should help pinpoint which package is causing the error.


perf: Talk maxing out CPU

Noticed that my battery drained rapidly. Not the first time I've seen this, or that it's been reported #1617.

This time I was able to record a performance profile in devtools, uploaded here

Saw this today as well, but it went away when I changed to the tab, so I wasn't able to get a performance profile

Just saw it again. Seems to be happening after a network disconnect / extended idle

I think we fixed most instances of this let us know if it crops up


genai: add WithAttribution client option

Add an option that lets Google products identify which calls
are made with this client.

Not objecting to this PR, but I do wonder if this can be emulated with a WithHTTPClient instead of adding new options?

It's theoretically possible but first, this should work for gRPC as well as HTTP (not that we need that now, but maybe someday) and more importantly, if you use WithHTTPClient it replaces all the setup that the normal client does. You'd have to do your own auth, for example. So it's challenging.


Feature request: create PSBTs from ord client

Example:
ord wallet psbt --sell {inscriptionID} 0.1btc

Tagging to follow development.
Interested in seeing which of the multiple PSBT structures for selling an inscription is chosen.

Do you have any preference to which one should be used?

Creating the PSBT to sell for a certain price is quite straightforward (once you know which address you want the payment sent to):

For a sell order, the PSBT should have a single input and single output and signed with SIGHASH_SINGLE | ANYONECANPAY

https://github.com/orenyomtov/openordex/blob/main/NIP.md#verification
Taking that PSBT and creating the final purchase TX as the buyer is the more complicated part

'--psbt' might not be the best name for the flag. I think a descriptive name like 'ord wallet {{inscriptionID}} --sell' is easier to understand.

Subcommands of buy/sell would make the most sense unless we foresee a large number of PSBT templates being added. Even in such a case those subcommands could be converted to proxy the PSBT subcommand.
ord wallet sell {inscription ID} {price}
ord wallet buy {psbt B64} [optional expected price]
Optional expected price can be compared to finalized PSBT before sending as a 'checksum'


Roll engine a38839817cda..f91934073ee1 (1 commits)

git@github.com:flutter/engine.git/compare/a38839817cda...f91934073ee1
git log a38839817cda..f91934073ee1 --no-merges --oneline
2019-11-21<EMAIL_ADDRESS>Roll src/third_party/dart aa2b3a5eaa..521b168441 (15 commits) (flutter/engine#13958)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Auto-roller completed checks. About to merge.


Unexpected errors from handlers does not get logged including DB errors

If any handler returns an error which results into 500 error from handler, the error is not getting logged at any level.
For example, if handler has following as return statement:
return h.store.Create(ctx, cust)
and the Create function has an error returned, it is expected to get it logged.

@vikash do we need an error log or a debug? Error log prints the stack trace but that isn't all helpful for the user as the trace would consist entirely of gofr calls!


graphviz: 9.0.0 -> 10.0.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/graphviz/versions.
meta.description for graphviz is: Graph visualization tools
meta.homepage for graphviz is: https://graphviz.org
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 10.0.1 with grep in /nix/store/95qy8cs07mx7mwx9qwikcp7zrxkx5hm7-graphviz-10.0.1
found 10.0.1 in filename of file in /nix/store/95qy8cs07mx7mwx9qwikcp7zrxkx5hm7-graphviz-10.0.1

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

23112 total rebuild path(s)

23112 package rebuild(s)

First fifty rebuilds by attrpath

BeatSaberModManager
CHOWTapeModel
ChowCentaur
ChowKick
ChowPhaser
CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcprime
CuboCore.libcsys
DisnixWebService
EmptyEpsilon
Fabric
HentaiAtHome
LibreArp
LibreArp-lv2
MIDIVisualizer
MMA
OSCAR
OVMF
OVMFFull
R
SDL
SDL2
SDL2_Pango
SDL2_gfx
SDL2_image
SDL2_image_2_0
SDL2_image_2_6
SDL2_mixer
SDL2_mixer_2_0
SDL2_net

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/95qy8cs07mx7mwx9qwikcp7zrxkx5hm7-graphviz-10.0.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/c55e9622e989f537616c32d9d0ea16c9c0522751#graphviz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/95qy8cs07mx7mwx9qwikcp7zrxkx5hm7-graphviz-10.0.1
ls -la /nix/store/95qy8cs07mx7mwx9qwikcp7zrxkx5hm7-graphviz-10.0.1/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @bjornfor @7c6f434c for testing.

@ofborg eval
(after 0e123540e5870dff8f8a823d26f9d0abe14ac91b)

Fails to build for me, macos 12 x86_64:
In file included from GVTextLayout.m:19:
In file included from ./gvplugin_quartz.h:18:
In file included from /nix/store/ns33cjy120wm6l5g27hsgg0iyc4cb0ds-apple-framework-ApplicationServices/Library/Frameworks/ApplicationServices.framework/Headers/ApplicationServices.h:23:
In file included from /nix/store/j74y27x0rd9xip9kp5skrqc8458my284-apple-framework-CoreServices/Library/Frameworks/CoreServices.framework/Headers/CoreServices.h:19:
In file included from /nix/store/35arz20x269wb9fzx4gfmf8h5m5c61pw-apple-framework-CoreFoundation/Library/Frameworks/CoreFoundation.framework/Headers/CoreFoundation.h:60:
In file included from /nix/store/35arz20x269wb9fzx4gfmf8h5m5c61pw-apple-framework-CoreFoundation/Library/Frameworks/CoreFoundation.framework/Headers/CFPropertyList.h:13:
In file included from /nix/store/35arz20x269wb9fzx4gfmf8h5m5c61pw-apple-framework-CoreFoundation/Library/Frameworks/CoreFoundation.framework/Headers/CFStream.h:15:
In file included from /nix/store/bj09naxfr523dj1a9namyy39x6ql94cj-Libsystem-1238.60.2/include/dispatch/dispatch.h:48:
/nix/store/bj09naxfr523dj1a9namyy39x6ql94cj-Libsystem-1238.60.2/include/os/object.h:75:9: fatal error: 'objc/NSObject.h' file not found
#import <objc/NSObject.h>
        ^~~~~~~~~~~~~~~~~

That seems to do the trick.

aarch64-darwin failure looks unrelated to me


Feature: Adding the ability to set line height on ElementNodes(Paragraph, List, Heading)

Setting line height is very common in all text editors and I think making it easily settable within lexical would be beneficial. Currently there is not a clean way to achieve the desired behavior.
This is an example within google docs
https://user-images.githubusercontent.com/69976884/221614006-2723041e-2b22-49a3-8264-3dbccafdc1f5.mov

You can add this value to an ElementNode via the node replacement API, but I don't know if it's common enough of a use case that we want to add it to core.


Spike: Optimize listing pipeline objects

As platform operator I want to keep Cartographer from doing unnecessary work, such as iterating 1000s of objects on every reconcile.
Problem

Objects stamped by the pipeline are created with generateName. As such, we cannot use the client Get method, but must instead use the List method. These objects are iterated over to determine if any have the same spec as the object that could be stamped.

When reporting outputs of objects, all objects created by the pipeline-runTemplate pair are retrieved with a List. These objects are iterated over to determine the most recently submitted successful object.

As the number of objects grows, this work could become non-trivial.
Notes
Garbage Collection

If garbage collection has already been implemented for Pipeline Service, this story may be obviated.
Garbage collection may be an implementation to satisfy this story.

Note from card: ENHANCEMENT - Explore if we really need to list runnables.
Currently in EnsureObjectExistsOnCluster we make a call to ListUnstructured. Before the implementation of Runnable, this was simply a Get.
With the addition of Runnable, we thought we did not have the name of the created object, only the generate name. After we Create the object, we should get the name back and save that successfully to the cache.
If we are able to check to see if the object exists in cache by name, that would simplify the looping in the cache.


Value Error Exception

If I get the exception on value error, can I just try to download again until it is fine?

More detail? Where do you get it? @lamhoson


[FEATURE] - Guest Registration Screen restrict user from pushing Next when entered Guest Info is not Registered

This is a similar request to issue 502 which helped by turning the +Register Guest button blue once the user enters guest information. We still have concerns that our registrants miss the +Register Guest button and click the Next button without pressing the +Register Guest button.
I propose that if someone types anything in the First Name and/or Last name that the Next button turns white and is disabled until the registrant clicks the + Register Guest button. At that time the Next button turns blue again. If the user enters another first name and/or last name, the color of those two buttons switch again.
Let me know if you need any additional/clearer details on what I'm suggesting here. We will use customizable text on this screen to help instruct the user to push the +Register Guest button prior to pushing the Next button as a temporary solution. Our registrants are forgetting to push the +Register Guest button with the current configuration.
Thanks for your consideration of this enhancement!

Hi @mcke0100,
I currently have in a branch a solution that might help with this, but I would love some feedback. What happens in my branch is that if information is filled in on the guest form and you hit next a dialog modal pops up and asks if you want to continue:

I would love feedback. First if you think this is a good solutions and then on the default language in the dialog. I will be adding the ability to customize the dialog language, but getting a good default is always good for the bulk of users.
What do you think?

My preference would be “You have unsaved Guest information entered.   Select Cancel to finish adding your guests”.
Then I would have the Cancel button blue, as that is what we’re suggesting to do.
Sue Kolles | University of St. Thomas
Advancement Systems |<PHONE_NUMBER>
Mobile |<PHONE_NUMBER>
From: McKenna, Amy M. @.>
Sent: Monday, August 28, 2023 9:07 AM
To: SFDO-Community/Summit-Events-App @.>; SFDO-Community/Summit-Events-App @.>
Cc: Mention @.>; Kolles, Susan A. @.***>
Subject: RE: [External] Re: [SFDO-Community/Summit-Events-App] [FEATURE] - Guest Registration Screen restrict user from pushing Next when entered Guest Info is not Registered (Issue #514)
Thad.
I like this functionality a lot. I included Sue on this discussion as she may have an opinion. I think the message is good for a default. I really like the first sentence “You have unsaved guest information entered.” I am good with the second sentence but I am wondering if more information may help. “Do you wish to continue without Saving? Select Cancel to go back and add the current guest to your registration.”
Amy
From: Thaddaeus Dahlberg<EMAIL_ADDRESS>Sent: Saturday, August 26, 2023 3:34 PM
To: SFDO-Community/Summit-Events-App<EMAIL_ADDRESS>Cc: McKenna, Amy M<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [External] Re: [SFDO-Community/Summit-Events-App] [FEATURE] - Guest Registration Screen restrict user from pushing Next when entered Guest Info is not Registered (Issue #514)
You don't often get email from<EMAIL_ADDRESS>Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Hi @mcke0100https://github.com/mcke0100,
I currently have in a branch a solution that might help with this, but I would love some feedback. What happens in my branch is that if information is filled in on the guest form and you hit next a dialog modal pops up and asks if you want to continue:
[image]https://user-images.githubusercontent.com/16785863/263496080-87dac9a7-ce39-4799-b4c9-847a03eae8cb.png
I would love feedback. First if you think this is a good solutions and then on the default language in the dialog. I will be adding the ability to customize the dialog language, but getting a good default is always good for the bulk of users.
What do you think?
—
Reply to this email directly, view it on GitHubhttps://github.com/SFDO-Community/Summit-Events-App/issues/514#issuecomment-1694496845, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCEQ33XKDBHVOTANIOJVLCTXXJMR5ANCNFSM6AAAAAA36TTG24.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

This is great. Thanks Thad.
Amy
From: Thaddaeus Dahlberg @.>
Sent: Monday, August 28, 2023 10:44 AM
To: SFDO-Community/Summit-Events-App @.>
Cc: McKenna, Amy M. @.>; Mention @.>
Subject: [External] Re: [SFDO-Community/Summit-Events-App] [FEATURE] - Guest Registration Screen restrict user from pushing Next when entered Guest Info is not Registered (Issue #514)
You don't often get email from<EMAIL_ADDRESS>Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Here you go:
[image]https://user-images.githubusercontent.com/16785863/263752783-3b164511-b5bf-4990-95be-cbe18829a7b9.png
I also made it so you can customize the message in the modal as well as the button text via the event setup record. The default is what is appearing in the screenshot now.
—
Reply to this email directly, view it on GitHubhttps://github.com/SFDO-Community/Summit-Events-App/issues/514#issuecomment-1695924899, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCEQ33XXYUDX2GQP4ON7IBLXXS4FLANCNFSM6AAAAAA36TTG24.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Tested in the Branch and works great!

Sue Kolles and I confimred it looked good through Metecho.

Included in beta release <IP_ADDRESS>

Included in production release <IP_ADDRESS>


Fix onActivityResult implementation

Resolves issue #92.
I've also changed request code to "more random" value and added clearing the result variable to prevent multiple result submission issue from occurring ever again.

@incrediblezayed please take a look at this PR. This issues is probably causing crashes in quite a few apps

Hello guys,
Can we please merge this? Our apps is crashing for some users because of it.

Hello guys,
Can we please merge this? Our app is crashing for some users because of it. @incrediblezayed

This weekend, promise, will deploy the latest version on pub.dev


CASMTRIAGE-6053 Update CSM_ARTI_DIR and ceph mount

Description

Checklist

[x] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[x] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

@bo-quan Thanks for the review. I have updated accordingly. Kindly review to ensure it meets the specs.

/backport release/1.4 release/1.6


Feature Request: Keep view scrolled to end

It would be cool if new input from the UART would display the new text instead of jumping back to the top of the text.  I have to constantly scroll down to see new text and each time it takes longer to scroll down so I usually quit the app and relaunch it.

@crazypill Thank you. I think it will be useful, so in today's commit I added this feature.

Thank you for adding this!


[done] Chinese docs

waiting for review of https://github.com/Jinjiang/vue-cli/pull/14 https://github.com/Jinjiang/vue-cli/pull/17 https://github.com/Jinjiang/vue-cli/pull/18 https://github.com/Jinjiang/vue-cli/pull/19 https://github.com/Jinjiang/vue-cli/pull/20 https://github.com/Jinjiang/vue-cli/pull/21 https://github.com/Jinjiang/vue-cli/pull/22
@Justineo 🙏

html-and-static-assets.md 第81行英文原文没删掉


cherry-pick: shared source data loss (#19443) and rate limit (#19445, #19466)

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
cherry-pick: shared source data loss (#19443) and rate limit (#19445, #19466)

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

cc @kwannoel can I cherry-pick #19085 together?
meet a e2e test issue without that

cc @kwannoel can I cherry-pick #19085 together?
meet a e2e test issue without that

SGTM


trivia_ios_DEVICES_DEMO

![trivia_ios_DEVICES](https://github.com/user-attachments/assets/76dc0a09-e1d6-4a13-925d-722ee941f44e)


Adds support for FQDN in route configuration

With Pretender v0.9.0 now supporting fully-qualified URLs, this update adds the ability to use FQDNs in your route configurations. It also adds a new urlPrefix parameter that allows you to configure Mirage to prepend a FQDN to all of your route configurations automatically.

sweet, great work dude!


Cleanup all danging unicode references.

Fixes #168

Codecov Report

Merging #169 (5fff4ad) into main (87d37a3) will increase coverage by 0.07%.
The diff coverage is 92.00%.

@@            Coverage Diff             @@
##             main     #169      +/-   ##
==========================================
+ Coverage   88.13%   88.21%   +0.07%
==========================================
  Files          12       12
  Lines        2166     2180      +14
==========================================
+ Hits         1909     1923      +14
  Misses        257      257

Files
Coverage Δ

src/picologging/logger.cxx
85.29% <100.00%> (+0.15%)
:arrow_up:

src/picologging/formatter.cxx
86.31% <89.47%> (+0.76%)
:arrow_up:


Update to correct writing convention of Node.js

Nice project, thank you for building it ❤
This PR suggests a correction to the proper way of the written form of Node.js

Thanks a lot for the correction :100:


Multispeaker Config

Hi @yl4579 thanks so much for your work.
To train multispeaker, do we just need to generate train|val_list.txt and change multispeaker to True in the config?
Any chance you could share (for example) your VCTK training script?

You also need to change the decoder type to HiFiGAN and make sure the data file contains the speaker ids for the second stage training. I’m currently testing the code on LibriTTS and it seems to work for the first stage, though it’s not done yet.
The code is exactly same as single speakers, the only difference is that multispeaker models use HiFiGAN decoder and the style diffuser takes one extra speaker embedding as the input.

I found a few bugs in the multispeaker settings. I have fixed them, but there might still be some other problems. I'm still testing everything so it may take another one to two weeks.

I found a few other bugs in the second stage as I have finished the first stage training in evaluations. Hopefully I’ll finish the LibriTTS model by next week.

say i gave it data with speaker_id 0,1,2,..n in its training. Now how would I access these speakers when making an inference


cmdr 1.12.0-ts.0

@osyrisrblx bumping this; forgot this pull request existed but it still good

Thanks for your PR! I'm now maintaining this repo and I have merged in my PR to upgrade to 1.12.0, and therefore I will close this.


fix(botocore.py): support sqs send message batch

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


Cannot use a file with spaces in public folder

[x] Read the docs.
[x] Use Vite >=2.0. (1.x is no longer supported)
[x] If the issue is related to 1.x -> 2.0 upgrade, read the Migration Guide first.

Describe the bug
Cannot use a file with spaces in public folder.
Reproduction
Try show an image with a space in it. The url will request a %20 encoded space correctly, but a 404 will be returned.
System Info

vite version: 2.0.1
Operating System: MacOS 11.2.1
Node version: 14.13.1
Package manager (npm/yarn/pnpm) and version: npm 6.14.11

Duplicate of #2195

@matias-capeletto thanks - I searched for space but I guess I missed that one...


Unable to build from source - gradle files in .gitignore.

Or at least I think this is a reason they are not downloading when I click Download ZIP on the Code tab.

The .gradle folder is not supposed to be included in the repository since it is generated upon running gradlew build.
I just tested this by downloading the zip from MASTER and running it, builds like expected.

I'm not talking about gradle folder which btw downloads fine
Wait what I downloaded it again and now the file size is 1,42 MB and also it contains more files like the mentioned gradlew. Guess my download was interrupted or something but how then Chrome marked the download as completed?


editorconfig: Enforce a 110 chars max line length.

Make supported IDEs and text editors keep a maximum line length of 110 characters for most code files.

Testing Plan: verified that it works fine in vim with the EditorConfig plugin for vim.

I merged this as c7a97e34fb0797efcdd32409f2801c0e073e024a, after tweaking it to limit the 110 limit to Python code and 120 for JS code.  There are various reasons some other file formats might require very long lines.


Add Michael Sprenger as maintainer

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


question about semantic segmentation

Thanks for the code! If I only want to train semantic segmentation, Can I use this yaml configs/Misc/Base-Semantic-DeepLab.yaml, Do I need to modify it? @bowenc0221

Please modify configs/panoptic_deeplab_R50_os32_cityscapes.yaml according to configs/Misc/Base-Semantic-DeepLab.yaml.
configs/Misc/Base-Semantic-DeepLab.yaml won't give good results for semantic segmentation.


Antispam

Adding a basic antispam system through an hidden field

surprised pikachu face
we'll fix the history later dw


mini_mime without dropping legacy Ruby support

#1059

In tandem with https://github.com/discourse/mini_mime/pull/1

Merged @ 634d91145cb65a85dff9da61eddc9e2de0609f8b

What prevents it get to 2.7.0 release? I mean there is only RC (since may), but no final version of gem.
I have implemented micro_mime, which is the same as mini_mime, but consumes even less memory and faster. The question is how to trick mail gem to use micro_mime instead of mini_mime? Something like multi_json, but for mime gems

@stereobooster, track the 2.7.0 milestone at https://github.com/mikel/mail/milestone/5
re. mini_mime vs micro_mime, my comment on the earlier pull request:

I think most users would prefer to have an extension bundled with the library itself rather than needing to know to add an additional optional dependency. That'll also tempt other library authors to depend on both mini_mime and micro_mime with awkward/mismatched version constraints.

This is exactly the situation I—and other library authors, I wager!—would like to avoid. We dropped mime-types for mini_mime rather than introduce some mime lookup plugin interface, after all.

My opinion is that mail depending on ANY mime type library is an anti-pattern. Make it a lambda function you provide as part of the configuration, then the user can supply whatever code they prefer to do mime-type lookup.

How about "mail-core" gem, which will not have this dependency and mail gem which will depend on  mini_mime and "mail-core"

@stereobooster The simplest route is to have users require 'micro_mime' after requiring mini_mime. Then you can replace the MiniMime constant with your MicroMime implementation.


[FR] Altitude percent include nadir

Is your feature request related to a problem? Please describe.
altitudepercent only get positive values (noon altitude), it can use negative values for nadir instead 0.
Describe the solution you'd like
Get negative values in altitudepercent for nadir.

Describe alternatives you've considered
Use https://github.com/claytonjn/hass-circadian_lighting to get the values.

will be in the next release (2.0.0 beta-3 or later)


Run _CheckForInvalidConfigurationAndPlatform in design-time builds

Should fix the VS side of dotnet/sdk#798
The targets which currently get run as part of _CheckForInvalidConfigurationAndPlatform are:

_CheckForUnsupportedTargetFramework
CheckForDuplicateItems
_CheckForInvalidConfigurationAndPlatform
CheckForImplicitPackageReferenceOverrides

I'm not convinced their change to drop this target from being included in InitialTargets was a wise one, I think we've probably just regressed legacy.

Yes, after testing this I'm not sure whether we should try fixing this by changing the design time targets.  The targets that are run by the project system when I try it out are:

CollectResolvedSDKReferencesDesignTime
CollectPackageReferences
ResolveComReferencesDesignTime
ResolveProjectReferencesDesignTime
ResolveAssemblyReferencesDesignTime
CollectSDKReferencesDesignTime
ResolvePackageDependenciesDesignTime
CompileDesignTime

Rather than make all of these targets depend on _CheckForInvalidConfigurationAndPlatform, a simpler solution is to modify the _CheckForUnsupportedTargetFramework target to run before GetReferenceAssemblyPaths.

I think the better change is to switch it back to being a initial target.

Going to close this PR - ping me if you still think this is the right approach, instead of adding it back to initial targets.

Side note, this still isn't all the design-time targets - any target can be a design-time target when using the MSBuild single file generator.


Fix minor problem when using token_dim with PromptTuning

Closes #474

Hello @wistuba, this PR is a duplicate of #459

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Hello @wistuba, as @thomas-schillaci mentioned this PR is a duplicate of the prior PR #459. Therefore, closing this. Thank you for your effort. 😄