SONARMSBRU-165: *.secured properties should be considered sensitive

SonarQube analysis reported 3 issues:

 3 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Initialize all static fields in 'ArgumentProcessor' when those fields are declared and remove the explicit static constructor.
 Change 'ProcessRunnerArguments.GetCommandLineArgsLogText()' to a property if appropriate.
 Change 'ProcessRunnerArguments.GetQuotedCommandLineArgs()' to a property if appropriate.

SonarQube analysis reported 3 issues:

 3 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Initialize all static fields in 'ArgumentProcessor' when those fields are declared and remove the explicit static constructor.
 Change 'ProcessRunnerArguments.GetCommandLineArgsLogText()' to a property if appropriate.
 Change 'ProcessRunnerArguments.GetQuotedCommandLineArgs()' to a property if appropriate.

SonarQube analysis reported 3 issues:

 3 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Initialize all static fields in 'ArgumentProcessor' when those fields are declared and remove the explicit static constructor.
 Change 'ProcessRunnerArguments.GetQuotedCommandLineArgs()' to a property if appropriate.
 Change 'ProcessRunnerArguments.GetCommandLineArgsLogText()' to a property if appropriate.

SonarQube analysis reported 3 issues:

 3 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Initialize all static fields in 'ArgumentProcessor' when those fields are declared and remove the explicit static constructor.
 Change 'ProcessRunnerArguments.GetQuotedCommandLineArgs()' to a property if appropriate.
 Change 'ProcessRunnerArguments.GetCommandLineArgsLogText()' to a property if appropriate.

Looks good (apart from the failing tests)

Looks like this is already merged into the master, I'm closing the PR - cc @bgavrilMS


Add unit tests that cover xarray interface

See #45 for details.
Add unit tests that cover the xarray interface for release 0.7.0.

Great thanks to Christopher Polster's pull request which covers this. 👍


Improve "Guided Tour" page in the docs

TLDR:
We should update “Guided Tour” page in docs to:

Better explain the situation in which you are left after creating your open-saas app on the disk.
Provide good ideas (possibly a checklist) on how to proceed from here.
Warn them that once they change the name of their app, they will have to restart wasp db start.

Below follows my thinking and ideas on what this could look like.

After creating my open-saas app on my disk, flow is a bit broken by taking me to the live demo app, and then explaining features → there is no moment where I go to the app I just created locally on the disk and do something with it. Also, the situation in which I am in right now is not explained, and that is that the app on my disk is not really working because it is missing API keys, and I should want to provide those. It should somehow warn me about this → hey, Stripe payment will fail, and this and that will fail, your app is only partially working / set up.
“Guided Tour” page ends with “And that concludes our guided tour! For next steps, we recommend …” → feels like something is missing? What are the next steps really? This is maybe the right place to warn them about missing API keys again (what I mention above). Also, give them some idea how to proceed: they can go through the codebase of the app they just created on their disk, delete some parts, modify some parts, check out the guides when not sure about something, … . Should they read all the guides first? Maybe make it clear (again?) they should check Wasp framework docs for any details on stuff? What I can expect from my app now that I go for it → what will not work or will work? I am again thinking about those API keys I think. We need to explain what is missing, and how much work each step will be: configure Email Sender (30 min, requires domain, free), configuring Stripe (45 min, requires bank account), get a domain, … . Yeah, so I am thinking about similar things here like in the paragraph above.
What we could do is give them a checklist of ideas on what they could do from this point on:

[ ]  Go through main.wasp and make it yours: rename stuff, remove parts you know you won’t need (e.g. redundant auths).

[ ]  Change the name of your app (when they do this, wasp start will crash though: TODO: let them know they need to restart wasp start db for stuff to work again (because db is given a name based on app name, and app looks for that name)).

[ ]  Go to demo-app part and start coding your core business logic there.
[ ]  Take a look at src/client/App → that is your root component, you might want to edit that.
[ ]  Maybe point them how they can do some global styling.
[ ]  Fill in the API keys in the env vars, those that you need. Delete those you don’t need.
[ ]  Let them know again that Wasp is the framework and they can check the docs if they need to learn anything.

I made this checklist up quickly, I am sure we can do it quite better.
Here is a list that I made for myself, for Work Diary, so maybe we could have something like this, but adapted:

[ ]  Make a new Github repo with this project.
[ ]  Decide if I am going with email & pass or with Google login, or both.
[ ]  Go through main.wasp file and delete redundant stuff + rename stuff.

piece of advice: even if you don’t have a custom domain yet, in some places it will be needed. Just put some domain you would like to have, it won’t be an issue to get started with development.
[ ]  update meta tags in head
[ ]  change app name (+ restart wasp start db after that)
[ ]  remove auth methods I won’t use, configure ones I will.
[ ]  adjust the info in [emailSender.defaultFrom](http://emailSender.defaultFrom.name) , if you are using emailSender.
[ ]  Delete “PasswordReset” routes and also “EmailVerification” route if you are not using email + pass auth.

[ ]  Go through .env.server and delete redundant stuff. Also delete it in .env.server.example.
[ ]  Delete S3 logic, Plausible logic.
[ ]  Update Landing Page.
[ ]  Reorganize files by functionality.

[ ]  Btw it might be worth it to port this back to actual open-saas codebase.

[ ]  Implement some MVP logic in DemoApp.
[ ]  Implement summarization of day / week / month via GPT. Comparison of this week vs usual weeks. Possibly also answering some questions about the work, like how much time did I spend doing this kind of work, what was unusual, ... .
[ ]  Set up Google Analytics.
[ ]  Buy the domain.
[ ]  Deploy the app.
[ ]  Get e2e tests going.
[ ]  Look into unit tests.
[ ]  Get CI working.

TODO: We should also, at the early enough point in the Guided Tour, warn user about Dummy provider and how it sends emails into the console. We should do this before they try to sign up, which should probably be quite early, certainly before we tell them to start tinkering with the app, or before they might decide to do so on their own. Which is likely very early, because they might be like "ok this is a lot of text, I just want to try it a bit first, let's see I will sign up here and bam".


fix(tests): use chrome for unit tests instead of phantomjs

This PR

Removes phantomjs from package.json
Installs chrome in the CI docker container

Perhaps its outside of the scope of this PR in particular, but there are only 3 unit tests actually being run at the moment [0], because the markdown component [1] has fit instead of it on each of it's 3 unit tests .. would you be able to change them and see how the rest of the tests perform? Unless the fit was an intentional addition of https://github.com/fabric8-ui/ngx-widgets/pull/148 ? @sanbornsen
[0] https://ci.centos.org/job/devtools-ngx-widgets/220/console
[1] https://github.com/fabric8-ui/ngx-widgets/blob/master/src/app/markdown/markdown.component.spec.ts

@jarifibrahim This commit https://github.com/fabric8-ui/ngx-widgets/pull/150/commits/083bbd378f4cf278dbb1cf8cbbe80f7cc1b68c8e adds the same functionality i think. Its part of this PR - https://github.com/fabric8-ui/ngx-widgets/pull/150 which is going to be merged soon.

@rohitkrai03 Good find! Yeah, it looks there are existing plans to use headless chrome and re-enable the unit tests.

@aptmac Since the all the unit tests will be enabled in https://github.com/fabric8-ui/ngx-widgets/pull/150, should we merge this PR?

I think a lot of this will be fixed by the upgrade.

I think a lot of this will be fixed by the upgrade.

@joshuawilson so should we merge this PR?

working on it now, should be ready today or tomorrow at the latest

oh, sorry, this PR not the upgrade. Let me finish the upgrade and then anything left over you can submit a PR for that. The upgrade and this would be a hard rebase.

This was taken care of in the upgrade. If you still want to remove references to PhantomJS, please submit a new PR.


Make it possible to receive audio data from other scripts #11

In some cases MonoBehaviour.OnAudioFilterRead() doesn't work. This change provides the alternative way for such cases.
But honestly I'm not sure that this change is align with principles of overall architecture design. It may make testing complex.
If you don't like it, it's okay to reject it for any reasons.👍

I'm surprised it even works with WebGL! Thanks for the great work.

Thank you for merging PR lightning fast!🙌


training unpaired but with ID matching?

Hi,
I'm doing a project to convert images taken from camera A to look like camera B, and to tackle it like the horse-zebra problem, but with the exception that I have images of facial parts taken from both cameras but they are not aligned with each other (shooting angle and posing), and I would like to verify it learns features from the same person and not from a random image, to apply to old image without matching B image.
would there be a training sequence with cyclegan-turbo that will bring the best solution to transform from A to B

Hi, I'm doing a project to convert images taken from camera A to look like camera B, and to tackle it like the horse-zebra problem, but with the exception that i have images of facial parts taken from both cameras but they are not aligned with each other (shooting angle and posing), and i would like to verify it learn features each from the same person and not from random image, to apply to image without B image. would there be a training sequence with cyclegan-turbo that will bring best solution to transform from A to B

Hello, I have a similar request. Have you managed to achieve it?

Hi @omriio
I think I did not understand your task description. Could you provide some visual examples of the setting that you are describing?
-Gaurav

hi @GaParmar, thank you for responding!
as I understand there are 2 scenarios:
pairs: two sets with images paired perfectly - > use img2img
unpairs: two sets with the same "concepts" (horse-zebras) - > use cyclegan
my case:  corresponding pairs of images for each "object", they're not aligned (angle, lighting, posing, etc'), but I need to keep as many as possible characters from the same specific object (textures, shape) to be able to identify it. in this example with full faces its obvious who the characteristics are, but if I want lips-only or forehead-only images, what technique would make the right transformation but keep the object the same?
I did try modifying the "class UnpairedDataset" always to choose couples from the same id and I am quite happy with the results, but would like to hear your input, and thank you for sharing your research


encoding issues

If there is a Chinese in the file, it will be garbled

Probably recaf just gotta use utf-16 for decompiled text.

use java -jar -Dfile.encoding=UTF-8 recaf.jar to run

If there is a Chinese in the file, it will be garbled

Reason
Your Windows system defaults to GBK, you double-click recaf.jar or use java -jar recaf.jar Run, the default characters inside the Java are GBK, the Recaf processing text is use  new String(Byte[]), this when the default encoding is gbk, it will cause garbled.

we can close this issue?

we can close this issue?

Yep, thank you for your PR.


Link Sprite Sheet

I mean archer sprite sheet... Put together enemy archer sprite sheet.

Implemented by #64.


Use Neurodocker

@PeerHerholz Do you think you've got time to try getting this working with neurodocker?

Sure thing, at it.

A first draft is up in #55, where I requested your review. Local build worked fine on my end. Could you maybe try it also on your machine?

Resolved by #55


Epic: Stateful Service Insertion

Provide an API that allows stateful services to be injected in to the datapath between one or more containers.
This may be unsupported on some types of driver so it may be necessary for some form of capabilities exchange to be added to the Driver API.
An implementation on Linux might be possible using NFQUEUE
There are many use cases for this type of functionality:

Injecting a tap to duplicate network traffic to a probe for central analysis
Injecting a probe to sample network traffic and export using sFlow or IPFIX
Injecting an IDS/IPS device
Injecting a stateful firewall that provides DPI

While all of the above use cases could be addressed via specialist drivers, I think it makes sense to avoid having 12 drivers that all do the same thing in a slightly different way.

@dave-tucker It has been detected that this issue has not received any activity in over 6 months. Can you please let us know if it is still relevant:

For a bug: do you still experience the issue with the latest version?
For a feature request: was your request appropriately answered in a later version?

Thank you!
This issue will be automatically closed in 1 week unless it is commented on.
For more information please refer to https://github.com/docker/libnetwork/issues/1926


Use UTC timezone for ts consistently

Previously, slog would try to use the local timezone and fallback to UTC
in multi-threaded programs. The check for is_single_threaded takes an
non-negligible amount of time when there's lots of logging. This uses
UTC unconditionally instead, which both makes the format consistent across
programs and also speeds up logging slightly.
Fixes https://github.com/slog-rs/slog/issues/308.

Sorry for the long wait!
I was worried about backwards compatibility breaking assumptions bout timezone.
However, it appears that the RFC specifies UTC is the default and the old code had an explicit timezone. That means this should be good to go and not break conformant parser :)

Wow, thanks for the review and quick release! No worries about the wait, I've seen much worse :sweat_smile:


ruby: 3.1.5 -> 3.1.6, 3.3.2 -> 3.3.3

Description of changes

https://www.ruby-lang.org/en/news/2024/06/12/ruby-3-3-3-released/
https://www.ruby-lang.org/en/news/2024/05/29/ruby-3-1-6-released/

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I'm pretty confident you're going to want to rebase this PR against staging (helpful instructions on the cleanest way to do this for this repo here).

Thank you for the heads up! I just rebased.

Built and tested (briefly) on x86_64-darwin.

@pbsds do you mind helping out with a merge if this looks right to you?

When is this PR going to be merged into unstable?

https://nixpk.gs/pr-tracker.html?pr=319933
the site is a bit slow due to being bombarded by people checking for the recent ssh cve patches


Adding Focus mode extension to block other sites

Is your feature request related to a problem? Please describe.
Whenever we are doing a productive work on a specific website we often get distracted by other websites.
Describe the solution you'd like
A chrome extension when enables on a website would allow the user to use only that specific website for a specific amount of time until turned off.
What problem is this feature trying to solve?
Focus while on a specific task
@Sulagna-Dutta-Roy Can you please assign this to me under GSSoC'24?
Thank You!

Closing this as no update received


Getting polyscope parent window size/dimensions for callback

I'm creating custom UI elements since the default callback seems to only support a single additional window and also has a hardcoded name of "Command UI".  In the C++ source, I noticed the windows are being docked by continually updating the current window heights of the parent window and adjusting the positions accordingly.
I'm trying to do this in the Python implementation with the imgui bindings, but the functions GetWindowHeight() and GetWindowWidth() seem to return completely static values when I call them within my callback, and do not change if I move/resize the parent window or the actual callback GUI element. It's unclear what it's doing. I'm seeing the same trouble with GetWindowContentRegionMax() and GetContentRegionMax().
Not sure if this is a ImGui specific question but it seems related to PolyScope. For that matter, it would be nice if the main polyscope instance could return the window sizing params for this purpose if the above bindings don't actually have the parent window context. Otherwise, I'm not sure how to get GUI elements to dock to different sides of the window, etc. It would also be nice if we could rename the default callback GUI, as well as enable/disable the Structure, Picker, and Polyscope GUI elements from Python like that can be done in C++. Additionally, further customization of those default elements would be highly appreciated since without it, users who want to customize the UI will have to turn off the boilerplate and re-write it which defeats the purpose in my opinion.

Since this post hasn't gotten any attention I would assume issues posted on the python repo are low-priority as the C++ one gets a little more traction. To deal with the functionality I want I went ahead and made the changes on the C++ source side and recompiled and rebuilt the polyscope whl... A bit awkward but I need to stay in Python for this application. Still would like some UI customization to be passed from Python, maybe I'll do a PR eventually.


Can I use g++ or mpic++ to link the hipcc object?

Before I want to use hipcc to link object together but I meet the LLVM problem #979 . So I change the ld to mpic++ and including the hip library (my program is MPI program). Through this, the LLVM problem solved and can link successfully, but when I running the executable file, the following error appears:
terminate called after throwing an instance of 'std::runtime_error'
  what():  No device code available for function: _Z10hip_normalv
Aborted

and I add a simple test program to show the error:
#include<stdio.h>
#include <hip/hip_runtime.h>
#define BLOCK_SIZE 64
__global__  void hip_normal()
{

}
int main(int argc, char *argv[])
{
    hipLaunchKernel(hip_normal,10,10,0,0);
    return 0;
}

I use hipcc, the running result is normal:
hipcc -c main.cpp -o main.o  [compile]
hipcc main.o -o Main [link]
./Main  [running]

But I use g++ to link:
hipcc -c main.cpp -o main.o   [compile]
g++ -L/opt/rocm/lib  -lhip_hcc main.o -o Main [link]
./Main [running]

Running error appeared :
terminate called after throwing an instance of 'std::runtime_error'
  what():  No device code available for function: _Z10hip_normalv
Aborted

So I want to ask the behavior using mpic++ or g++ to link is wrong or not even if I add the hip_hcc library?
Thank you very much!

Add -fno-gpu-rdc to hipcc when you compile main.cpp and link with g++ o mpic++.

@louishp , thank you for your reply, but when I add -fno-gpu-rdc to hipcc the error appears:
clang-7: error: unknown argument: '-fno-gpu-rdc'
and I use hipcc --help do not find the option in the list. The fllowing is my hipcc version info.
$ hipcc --version
HIP version: 1.5.18353
HCC clang version 7.0.0 (...) (...) (based on HCC 1.2.18354-ec91fed-e2b51bf-009cb63 )
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /opt/rocm/hcc/bin

The feature is in ROCm 2.1+ if I understand correctly. You might want to upgrade your ROCm stack to have newer HCC with that flag.

I tested with  a sample code(example.cpp) with hipcc and mixed compile & link(Hipcc and g++)
Case1:
The sample code compile and link with hipcc and the time taken kernel call(add) is "18371NS"
Compile & Link:-hipcc
hipcc -c example.cpp.cpp -o main.o  [compile]
hipcc main.o -o Main [link]
./Main  [running]
rocprof :
Name						Calls	TotalDurationNs	AverageNs	Percentage
void add(int*, int*)	1			18371		18371		100
Case2:
The sample code compile hipcc and link with g++ and the time taken  kernel call (add) is "18520Ns"
Compile(hipcc) & link(g++)
hipcc -c -fno-gpu-rdc example.cpp -o main.o   [compile]
g++ -L/opt/rocm/lib  -lhip_hcc main.o -o Main [link]
./Main [running]
rocprof :
Name	                  Calls	TotalDurationNs	AverageNs	Percentage
void add(int*, int*)	1	18520	        18520	      100
Add -fno-gpu-rdc to hipcc is  support an early finalization of device code.
In both cases(case1 and case2) the time taken Kernel  call difference is 149Ns.
Is -fno-gpu-rdc  impact on performance ? If yes is any optimize solution available?
I would like to know  is any an alternate solution(on below issue) there compile example.cpp with hipcc(without Add -fno-gpu-rdc) and link with g++.
Is any proper document on cross compile and link.
Issue:
$ hipcc -c example.cpp -o main.o
$ g++ -L/opt/rocm/lib  -lhip_hcc main.o -o Main
$ ./Main
terminate called after throwing an instance of 'std::runtime_error'
what():  Missing metadata for global function: Z3addIiEvPT_S1
Aborted (core dumped)
example.zip

Is the 149ns performance drop repeatable?  149ns difference out of 18371ns is 0.8%.  I would be surprised if this was a consistent repeatable measurement given how little work is being done on the device in your example code; scaling an array of length 1000 by 2.0.

149ns performance drop not repeatable.
I cross checked the behavior in the  same machine(AMD GPU -Vega10).
In Case 2 the kernel call time taken less compare to Case 1.
void add(int*, int*)	1	24890	"24890"	100 (Case 1)
void add(int*, int*)	1	18816	"18816"	100 (Case 2)
void add(int*, int*)	1	25186	"25186"	100 (Case 1)
void add(int*, int*)	1	18075	"18075"	100 (Case 2)
void add(int*, int*)	1	21038	"21038"	100 (Case1)
void add(int*, int*)	1	18371	"18371"	100 (Case2)
finalize GPU kernels at compile-time with -fno-gpu-rdc.
I have question using "-fno-gpu-rdc " impact on performance in run time
Example:
if device code compile with "hipcc -fno-gpu-rdc" and host code compile g++.
Both device code(object files) and host code(Object files) linked to execute.
Is  any Impact on GPU performance(Kernel run time)?

I have question on using "-fno-gpu-rdc " impact on performance GPU?

All the -fno-gpu-rdc option does is instruct the compiler to generate device code at compile time rather than at link time.  To do that, any device code that calls other device code must be in the same translation unit (that's why it is called no-rdc).
Exactly the same ISA is executing on the device in both cases:
$ /opt/rocm/bin/hipcc -c example.cpp -o main.o

$ /opt/rocm/bin/hipcc main.o -o main

$ /opt/rocm/bin/extractkernel -i main

$ /opt/rocm/bin/hipcc -fno-gpu-rdc -c example.cpp -o main_no_rdc.o

$ g++ -L/opt/rocm/lib main_no_rdc.o -o main_no_rdc -lhip_hcc

$ /opt/rocm/bin/extractkernel -i main_no_rdc

$ diff main-000-gfx906.isa main_no_rdc-000-gfx906.isa
2c2
< main-000-gfx906.hsaco:        file format ELF64-amdgpu
---
> main_no_rdc-000-gfx906.hsaco: file format ELF64-amdgpu

The diff between the two kernel ISA files shows no changes in the device code.  There is no difference at all what happens on the device between the two cases.
I guess I am confused about your question.  What problem are you trying to solve?  I am not convinced 149ns is worth worrying about in the example you gave because it is such a small example.  Of course, if you're just asking because you're curious about what the compiler option does then fair enough.

@holyprince Do you still need assistance with this ticket? If not, please close ticket. Thanks!

@anilbommareddy  it is generally not recommended to compile hip program with g++ (link with g++), since HIP api changes quickly.
In the recent programs, if you want to compile with g++, you need to link object files agiant "-lhiprtc" instead of old "-lhip_hcc".
Here is an example:
# link with g++ or clang++;
vcopy: vcopy.o
	$(CXX) $(LIBS) -lhiprtc vcopy-dir/vcopy.o -o vcopy

To generate asm or object files with clang++:
hipcc $(CXXFLAGS) $(HIPXXFLAGS) $(LIBS) -O2 -S --offload-arch=gfx942 ../vcopy.cpp -o vcopy.S

or
clang++ --offload-arch=gfx942 --driver-mode=g++ --hip-link  -fPIC -std=c++17 $(LIBS) -Wall -O2 -S -x hip vcopy.cpp -o "vcopy.S"


Setup Backend Server

Description
Setup the backend server using the MVC layout. Configure HTTP route and setup endpoints with dummy tests.
This task was mostly collaboratively completed by the team during meetings.

Related Issue

Solves #13

Type of change

[x] New feature (enhancement)

How Has This Been Tested?
This has been tested with an implemented automated testing suite using mocha and supertest.
Run the test suite with
npm test

[x] Automated testing

Checklist:

[x] Does a similar (open or closed) pull request not already exist?
[x] Is the pull request head repository a fork repository?
[x] Is the pull request compare branch a development branch?
[x] Is the code documented, particularly in hard-to-understand areas?
[x] Does the code build without new warnings?
[x] Has testing been performed that proves changes are effective and work?
[x] Has a self- and/or peer-review of the code been performed?
[x] Does all new and existing automated testing pass?
[x] Is the person responsible for the repository assigned to the pull request?
[x] Is the pull request linked to a project?
[x] Is the pull request linked to a milestone?

For more information, refer to the Contributing Guidelines document.

Do we also need to set up database schema/model for this issue? Or create a new issue beacause I can not find any issues relate to the schema

Do we also need to set up database schema/model for this issue? Or create a new issue beacause I can not find any issues relate to the schema

There is a file created in the PR for the database model - it is currently empty. The existing Issues created earlier today will involve communicating with the database, such as creating a new document for a new user/blog, for example.
Regarding setting up the database, we created one yesterday. There is another Issue for connecting to that database #14 that I have assigned myself to not too long ago and am going to complete it very soon I suspect.


Problem with building subproject

When other subprojects are not installed in standard path it does not work.

fixed


[FEATURE] Code Linting

Detailed description
The Codebase is pretty Raw at this point.
To make onboarding and contributing easier, lint rules and maybe a prettier setup are recommended
Context
easiier to contribute.
no need for maintainers to force specific code styles in commentary -> can be handled by repo
Possible implementation
esLint, Prettier
maybe tsconfig?
Additional information
No response
Code of Conduct

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

hey @kabir0x23 do you think this a good idea?
if yes i could set up some rules.

hey @kabir0x23 do you think this is a good idea?
if yes I could set up some rules.

Yes, this is a great idea.
I have also tried linting once on one of my project, linter and prettier on commit.
Yes, go ahead 🚀.

this is ready, but please merge #10 before this.
it would be recommended to maybe create another branch, to base feature branches off.
maybe:
main for production ready
develop for developing and to base feature branches off?

Okay nice,
Got it
Like
create a develop branch only for developing purposes.
and main for developing
can you elaborate on, to base feature branches off?

this is ready, but please merge #10 before this.
it would be recommended to maybe create another branch, to base feature branches off.
maybe:
main for production-ready development for developing and to base feature branches off?

first merging PRs in dev branch (linters will their work)
and then we will merge it into the main branch

exactly, the only branch that merges into master is develop all other branches go into develop first.

exactly, the only branch that merges into master is develop all other branches go into develop first.

Nice Nice

Okay got it.

exactly, the only branch that merges into master is develop all other branches go into develop first.
for example: checkout develop -> pull -> checkout -b some-feature -> PR to develop
I will add an action to check code in CI tomorrow, but you can still merge this, i will do all the actions on another branch.

I will a create a development branch


Delete ISBServiceRollout should validate no more pipeline is referring to a ISBService before deleting

Summary
Delete ISBServiceRollout should validate no more pipeline is referring to a ISBService before deleting
Use Cases
To avoid cause a healthy pipeline to go down because of missing ISBService.

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

This may be revisited to see if cases like user wants to explicitly delete the ISBService even though there is a pipeline relates to it.


[Words / Favorite home] Exercise 1,15,29,43. Extraneous sound when pronouncing a word "Дом"]

Precondition:
Log in with an existing user. Open: Groups / Speech exercises / Words / Favorite home
http://brainup.site/groups/2/series/1/subgroup/2/exercise/29/task/29
https://brainup.site/groups/2/series/1/subgroup/2/exercise/43/task/43
https://brainup.site/groups/2/series/1/subgroup/2/exercise/57/task/57
https://brainup.site/groups/2/series/1/subgroup/2/exercise/71/task/71
Steps:
1.Select exercise 1/15/29/43
2.


Apple M1 Build Updates & Instructions

Problem
Building and running tests as outline in CONTRIBUTING.MD on a M1 based MacBook is not possible (or easy). This is due to various containers we rely on not having supporting the appropriate architecture. This results in strange crashes, hangs, etc when trying to build and test the project.
Solution
Update docker-compose and Docker files to address this. Included a README to describe how to run the m1 based docker-compose.
Checklist
@getquill/maintainers

Thanks @tdrozdowski!!!


MacOS amd64 CI backend test failures

see e.g. this CI failure
======================================================================
ERROR: test_abs_cpu (__main__.OnnxBackendNodeModelTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/runner/hostedtoolcache/Python/3.10.9/x64/lib/python3.10/site-packages/onnx/backend/test/runner/__init__.py", line 259, in device_test_func
    return test_func(*args, device=device, **kwargs)
  File "/Users/runner/hostedtoolcache/Python/3.10.9/x64/lib/python3.10/site-packages/onnx/backend/test/runner/__init__.py", line 315, in run
    self.assert_similar_outputs(ref_outputs, outputs,
  File "/Users/runner/work/onnx-mlir/onnx-mlir/build/test/backend/Debug/inference_backend.py", line 1173, in assert_similar_outputs
    super(InferenceBackendTest, cls).assert_similar_outputs(ref_outputs, outputs, rtol, atol)
  File "/Users/runner/hostedtoolcache/Python/3.10.9/x64/lib/python3.10/site-packages/onnx/backend/test/runner/__init__.py", line 181, in assert_similar_outputs
    if ref_outputs[i].dtype == np.object:
  File "/Users/runner/hostedtoolcache/Python/3.10.9/x64/lib/python3.10/site-packages/numpy/__init__.py", line 284, in __getattr__
    raise AttributeError("module {!r} has no attribute "
AttributeError: module 'numpy' has no attribute 'object'

...
======================================================================
ERROR: test_vgg19_cpu (__main__.OnnxBackendRealModelTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/runner/hostedtoolcache/Python/3.10.9/x64/lib/python3.10/site-packages/onnx/backend/test/runner/__init__.py", line 259, in device_test_func
    return test_func(*args, device=device, **kwargs)
  File "/Users/runner/hostedtoolcache/Python/3.10.9/x64/lib/python3.10/site-packages/onnx/backend/test/runner/__init__.py", line 299, in run
    self.assert_similar_outputs(ref_outputs, outputs,
  File "/Users/runner/work/onnx-mlir/onnx-mlir/build/test/backend/Debug/inference_backend.py", line 1173, in assert_similar_outputs
    super(InferenceBackendTest, cls).assert_similar_outputs(ref_outputs, outputs, rtol, atol)
  File "/Users/runner/hostedtoolcache/Python/3.10.9/x64/lib/python3.10/site-packages/onnx/backend/test/runner/__init__.py", line 181, in assert_similar_outputs
    if ref_outputs[i].dtype == np.object:
  File "/Users/runner/hostedtoolcache/Python/3.10.9/x64/lib/python3.10/site-packages/numpy/__init__.py", line 284, in __getattr__
    raise AttributeError("module {!r} has no attribute "
AttributeError: module 'numpy' has no attribute 'object'

----------------------------------------------------------------------
Ran 2162 tests in 825.400s

FAILED (errors=459, skipped=1703)

For a successful builld:
Collecting numpy>=1.16.6
  Downloading numpy-1.23.5-cp310-cp310-macosx_10_9_x86_64.whl (18.1 MB)

For a failed build:
Collecting numpy>=1.16.6
  Downloading numpy-1.24.0-cp310-cp310-macosx_10_9_x86_64.whl (19.8 MB)

Sound like latest numpy has incompatible changes.

We could preinstall numpy 1.23 so onnx wouldn't try to install 1.24. The MacOS github action build is very cumbersome to manage because we can't get onto the VM on the server to try things out (there might be some hacky way to get on the VM but I never had the cycles to actually try). Everything is done through modifying the action script in a trial and error fashion, which is very tedious and time consuming.

resolved?


Disable printing warning logs when test-mode is enabled.

Question
As is explained in the document, test-mode is used for writing Junit-tests. When I am writing codes in other modules, the console keeps printing the following messages. Could this be avoided?
2021-01-13 16:14:11.653  [] [] [] [WARN] 16644 --- [oms-akka.actor.default-dispatcher-78] c.g.k.p.w.actors.TroubleshootingActor    TroubleshootingActor.java:23: [TroubleshootingActor] receive DeadLetter: DeadLetter(WorkerHeartbeat(workerAddress=<IP_ADDRESS>:28888, appName=service-command-communication, appId=11, heartbeatTime=1610525631633, containerInfos=[], version=3.4.0, extra=null, systemMetrics=SystemMetrics(cpuProcessors=4, cpuLoad=-1.0, jvmUsedMemory=0.2376, jvmMaxMemory=3.4863, jvmMemoryUsage=0.0681, diskUsed=174.5065, diskTotal=930.716, diskUsage=0.1875, score=7)),Actor[akka://oms/deadLetters],Actor[akka://oms/deadLetters])

2021-01-13 16:14:11.656  [] [] [] [WARN] 16644 --- [oms-akka.actor.default-dispatcher-78] c.g.k.p.w.actors.TroubleshootingActor    TroubleshootingActor.java:23: [TroubleshootingActor] receive DeadLetter: DeadLetter(WorkerHeartbeat(workerAddress=<IP_ADDRESS>:28888, appName=service-command-communication, appId=11, heartbeatTime=1610525646633, containerInfos=[], version=3.4.0, extra=null, systemMetrics=SystemMetrics(cpuProcessors=4, cpuLoad=-1.0, jvmUsedMemory=0.2411, jvmMaxMemory=3.4863, jvmMemoryUsage=0.0692, diskUsed=174.5065, diskTotal=930.716, diskUsage=0.1875, score=7)),Actor[akka://oms/deadLetters],Actor[akka://oms/deadLetters])

2021-01-13 16:14:11.658  [] [] [] [WARN] 16644 --- [oms-akka.actor.default-dispatcher-78] akka.stream.Materializer                 Slf4jLogger.scala:89: [outbound connection to [akka://oms-server@<IP_ADDRESS>:10086], message stream] Upstream failed, cause: StreamTcpException: The connection has been aborted

2021-01-13 16:14:11.659  [] [] [] [WARN] 16644 --- [oms-akka.actor.default-dispatcher-78] akka.remote.artery.Association           Slf4jLogger.scala:89: Outbound message stream to [akka://oms-server@<IP_ADDRESS>:10086] failed. Restarting it. akka.stream.StreamTcpException: The connection has been aborted

emmm, It seems can't be turn off.

emmm, It seems can't be turn off.


Hover Effect and size of cards now become same

Kind of change this PR introduce: On Hover, it will Transform the About-us Cards and also box-shadow effects shows at hovering. Similarly, the red border also adds in order to Resemble the theme of red related to Blood-Buddy.
Issue it resolves: Under the About-us Cards are not of the same Height. It solves the Height of About-us Cards.
Screenshots & GitHub Page Link: https://bhavyastar.github.io/Blood-Buddy/
Screenshot of Problem:

Solution of problem:

Kind of change this PR introduce: On Hover, it will Transform the About-us Cards and also box-shadow effects shows at hovering. Similarly, the red border also adds in order to Resemble the theme of red related to Blood-Buddy.

Issue it resolves: Under the About-us Cards are not of the same Height. It solves the Height of About-us Cards.

Screenshots & GitHub Page Link: https://bhavyastar.github.io/Blood-Buddy/
Screenshot of Problem:

Solution of problem:

Please mention the issue it resolves , not the sentences but link the issue using #

Kind of change this PR introduce: On Hover, it will Transform the About-us Cards and also box-shadow effects shows at hovering. Similarly, the red border also adds in order to Resemble the theme of red related to Blood-Buddy.

Issue it resolves:  #64
Screenshots & GitHub Page Link: https://bhavyastar.github.io/Blood-Buddy/
Screenshot of Problem:

Solution of problem:

Please mention the issue it resolves , not the sentences but link the issue using #

Connect it with the issue, not the PR. @bhavyastar  I think this issue was never assigned to you because it doesn't exist even.


"argument named region not expected here"

Problem
In line 79 of setup.sh, this build command:
gcloud builds submit ./terraform --project="$OPS_PROJECT" \
    80      --config=./ops/terraform.cloudbuild.yaml \
    81      --substitutions=_ENV="ops",_STATE_GCS_BUCKET_NAME=$STATE_GCS_BUCKET_NAME,_TF_SERVICE_ACCT=$TERRAFORM_SERVICE_ACCOUNT

is failing with the following TF errors:
Starting Step #2 - "tf plan"
Step #2 - "tf plan": Already have image: hashicorp/terraform:1.0.0
Step #2 - "tf plan": ╷
Step #2 - "tf plan": │ Error: Unsupported argument
Step #2 - "tf plan": │
Step #2 - "tf plan": │   on main.tf line 4, in module "quizrd_ops":
Step #2 - "tf plan": │    4:   region          = var.region
Step #2 - "tf plan": │
Step #2 - "tf plan": │ An argument named "region" is not expected here.
Step #2 - "tf plan": ╵
Step #2 - "tf plan": ╷
Step #2 - "tf plan": │ Error: Unsupported argument
Step #2 - "tf plan": │
Step #2 - "tf plan": │   on main.tf line 5, in module "quizrd_ops":
Step #2 - "tf plan": │    5:   setup_cd_system = var.setup_cd_system
Step #2 - "tf plan": │
Step #2 - "tf plan": │ An argument named "setup_cd_system" is not expected here.
Step #2 - "tf plan": ╵
Finished Step #2 - "tf plan"
ERROR
ERROR: build step 2 "hashicorp/terraform:1.0.0" failed: step exited with non-zero status: 1
--------------------------------------------------------------------------------

BUILD FAILURE: Build step failure: build step 2 "hashicorp/terraform:1.0.0" failed: step exited with non-zero status: 1
ERROR: (gcloud.builds.submit) build 8795dfc3-914a-4482-8e32-54a3f1ed84c7 completed with status "FAILURE"

Steps to Reproduce
I forked emblem and followed the README.md.
Expected behavior
I expected it to complete without errors.
Additional context
Screenshots
If applicable, add screenshots to help explain your problem.
User environment:

OS: MacOS
Browser: Version 112.0.5615.39 (Official Build) beta (x86_64)
Emblem Project Version: 0.1.0

Sorry, you can ignore this, it's actually working fine. I was seeing this error on a modified fork so I decided to make sure the vanilla repo works but I inadvertently switched tabs at some point and was looking back at the old errors.


Pretty printing for functors and other nested structures.

We define functors (in the category theory library) to extend Prefunctor, which has data fields obj and map. When F is a functor, we routinely use F.obj as a spelling for F.toPrefunctor.obj. However, this originally had the unintended consequence that F.obj X would pretty print as Prefunctor.obj F.toPrefunctor X, which is far from ideal (the map field had the analogous issue). In #2545 we introduced some custom unexpanders to address this.
However, this issue is certainly not isolated to functors in the category theory library. One good approach is to add a global pp option that would pretty print nested projections through extends with the correct (aka concise) dot notation. See also the associated zulip discussion. If/when such a global option is added, the unexpanders introduced in the PR above should be removed.
Here is a minimal example illustrating the issue:
structure Foo where
  f : Nat → Nat

structure Bar extends Foo where
  hf : f 0 = 0

example (F G : Bar) (m n : Nat) (h : m = n) : F.f m = F.f n := by
  sorry -- bad pretty printing here...

@[app_unexpander Foo.f] def
  unexpandFoo : Lean.PrettyPrinter.Unexpander
  | `($_ $(F).toFoo $n) => set_option hygiene false in `($(F).f $n)
  | _ => throw ()

example (F G : Bar) (m n : Nat) (h : m = n) : F.f m = F.f n := by
  sorry -- good pretty printing here...

This is closed by #3811
Now you can do pp_extended_field_notation Foo.f to automatically construct this unexpander.
structure Foo where
  f : Nat → Nat

structure Bar extends Foo where
  hf : f 0 = 0

example (F G : Bar) (m n : Nat) (h : m = n) : F.f m = F.f n := by
  sorry -- bad pretty printing here...
  -- Foo.f F.toFoo m = Foo.f F.toFoo n

pp_extended_field_notation Foo.f

example (F G : Bar) (m n : Nat) (h : m = n) : F.f m = F.f n := by
  sorry -- good pretty printing here...
  -- F.f m = F.f n


[ENHANCEMENT] Support Podman

Support Podman as a container technology for ECP.

@koelslaw can you update the README.md to include instructions on how to install Podman?
I don't see it in my Rocky builds. I can't check right now, but I feel like its as simple as dnf install podman or apt install podman? If so, that'd be easier for Linux installations all around.
Can you check that and LMK what you recommend?

Podman/podman-compose is completely ignoring health checks at the moment and tries to stand up fleet server immediately. This wouldn't necessarily be an issue but It also ignore the restart flag after a certain amount of time or attempts (not sure which). Fleet containers stops restarting and your left without it. If you run shell or start the container individually it takes right off. 🤷 More work to try and figure this out.

We'll reopen if this makes progress.


Integer overflow when running command

I get the following error when trying to use this tool. I’m using macOS 12.0.1 on an Apple M1 Pro, using node v17.3.1.

 (node:91811) TimeoutOverflowWarning:<PHONE_NUMBER>000 does not fit into a 32-bit signed integer.
Timer duration was truncated to<PHONE_NUMBER>.
(Use `node --trace-warnings ...` to show where the warning was created)
(node:91811) TimeoutOverflowWarning:<PHONE_NUMBER>000 does not fit into a 32-bit signed integer.
Timer duration was truncated to<PHONE_NUMBER>.
Error: Not Found
    at BufferList.afterCreateAuthResponse [as _callback] (/Users/Mark.Crossfield/.nvm/versions/node/v17.3.1/lib/node_modules/org-labels/node_modules/ghauth/ghauth.js:39:19)
    at BufferList.end (/Users/Mark.Crossfield/.nvm/versions/node/v17.3.1/lib/node_modules/org-labels/node_modules/bl/bl.js:75:10)
    at Stream.onend (node:internal/streams/legacy:47:10)
    at Stream.emit (node:events:402:35)
    at Stream.handleEnd (/Users/Mark.Crossfield/.nvm/versions/node/v17.3.1/lib/node_modules/org-labels/node_modules/duplexer/index.js:81:21)
    at Stream.emit (node:events:402:35)
    at drain (/Users/Mark.Crossfield/.nvm/versions/node/v17.3.1/lib/node_modules/org-labels/node_modules/through/index.js:33:23)
    at Stream.stream.queue.stream.push (/Users/Mark.Crossfield/.nvm/versions/node/v17.3.1/lib/node_modules/org-labels/node_modules/through/index.js:41:5)
    at Stream.end (/Users/Mark.Crossfield/.nvm/versions/node/v17.3.1/lib/node_modules/org-labels/node_modules/through/index.js:17:35)
    at _end (/Users/Mark.Crossfield/.nvm/versions/node/v17.3.1/lib/node_modules/org-labels/node_modules/through/index.js:61:9)
    at Stream.stream.end (/Users/Mark.Crossfield/.nvm/versions/node/v17.3.1/lib/node_modules/org-labels/node_modules/through/index.js:70:5)
    at IncomingMessage.onend (node:internal/streams/readable:716:10)
    at Object.onceWrapper (node:events:509:28)
    at IncomingMessage.emit (node:events:402:35)
    at endReadableNT (node:internal/streams/readable:1343:12)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)

I think this is a bug in one the libraries in the stack traces.
Also I am unsubscribing for this, feel free to fix it yourself if it's a bug in this module.

I’m not sure I would know where to start fixing it myself, or working out whether the bug is with this module or one of the libraries.
I’d happily raise an issue with one of the libraries if anyone could point which one is at fault. I’m guessing that you were unable to reproduce the problem yourself?


Fixed column disturbing table row layout

Guidelines
Describe the bug
I have a simple table where I want to make first column as fixed , after making column as fixed if we will try to adjust the resolution of browser table row is getting messed up.
To Reproduce
Steps to reproduce the behavior:

Go to sandBox Env Link
Adjust browser Resolution
observe the ROW its getting messed up

Well that's not good.
Has anybody looked under the hood yet? I don't mind taking a look in a bit but this seems more aesthetic than a bug in code so I'm going to change the tags.
Thank you!

@oze4  because of this data is getting messed up and its looking too bad as for each fixed row other columns are getting unequal alignment.

This is most likely connected to this, but I am not sure on how this would affect the rest of the table. @oze4 any ideas?

@oze4  any update on this ?

No unfortunately i had no time to look into it and it seems like a complex problem. Maybe next week.

Any update on this?

@dcods22 this fork is meant to be a community repo. We wanted to help keep this project alive.
Please, do not rely on us for a fix. If you need something resolved urgently, we are actively accepting and merging pull requests. This does seem like an edge case or "rare" issue to encounter, meaning for such a complex rare issue you will most likely have to resolve this yourself.
@Domino987 and myself have been super busy lately, so thank you for understanding.
Please let us know if you have any questions.

Not stale.

@fagom Can you check with the latest version?


Unable to switch to use KMIP as Encryption Key Vault

Hello
I have just procured a new Synology NAS using DSM 7.2.2-72806, with KMIP config using the kmip-server-dsm according to the guide. Yet, I have experienced a strange problem. The NAS could able to make a KMIP connection under the "Control Panel > Security > KMIP" section in DSM, but unable to select KMIP under the "Storage Manager > Global Settings > Reset Encryption Key Vault" section.
Success Import Client Cert:

Success KMIP server connection:

Not able to switch to KMIP for Encryption Key Vault:

Below is the pykmip config.sh  and  server.conf
config-sh.txt
server-conf.txt
And I could not find any errors in the server.log, below is the extract:
extract-server.log
Are there any issues in my settings or configuration which lead to such strange problem?
Thanks in advance. :)

Thank you for reporting this. It seems that breaking change was introduced in https://github.com/rnurgaliyev/kmip-server-dsm/pull/16. I've rolled back pykmip to v0.10.0 for now. You should be able to setup remote encryption vault now, if you pull latest release and rebuild the docker container.
I will have more time in October to have a deeper look at the issue and make further updates of pykmip possible. cc @oittaa

Hi rnurgaliyev, thank you for your help.
I have tried to rebuild the container based on v2024.9.2, and the strange problem has been resolved.
Thanks. :)


Auto merge

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[x] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

@csmengwan - There are no files to review. I see that you did a merge commit. Did you manually create the PR or it was a side effect of something.

I manually created this.

But there are no files to review. Should I close this?

I think this is due to branch change (auto rest => master).
Can you merge the pull request so I don't have to clean and checkout the repo again?

nope. Please sync with upstream master and you should be good.

Whenever you are working on something, please create a feature branch on your fork, so that you dont end up in this situation.


[nit] Apply default value for ChatOpenAI client

This is a small convenience PR to give a default to the client attribute of ChatOpenAI, avoiding a warning from type checkers about it being missing when initializing the class. The attribute appears to be set during validation already using a default value.
@baskaryan

Closing since this is a tiny nit and I opened it from master on my fork thinking it'd close quickly. I'll create a branch and reopen.


Indicating persistent transport type in the emulator's document

Summary of the changes (Less than 80 chars)

SignalR Emulator document updated to reflect the lack of persistent transport type

Thanks for contributing. I happened to add another PR #1556 to mention that emulator not working for persistent mode, but I failed to link it to the issue.

Content added.


fix(snack-bar): snack bar not animating in if no positions are passed in

Adds some better handling for the case where no positions are passed into a snack bar. Currently the snack bar attempts to animate in to an invalid animation state.
Fixes #11197.

I think a better way to tackle this would be to set ensure the defaults are set in the open method in snack-bar.ts
We can do:
open(message: string, action: string = '', config?: MatSnackBarConfig):
    MatSnackBarRef<SimpleSnackBar> {
  const _config = {...new MatSnackBarConfig(), ...this._defaultConfig, ...config};

  // Since the user doesn't have access to the component, we can
  // override the data to pass in our own message and action.
  _config.data = {message, action};
  _config.announcementMessage = message;

  return this.openFromComponent(SimpleSnackBar, _config);
}

This will have our object as the default, while being overridden by whatever the user provides either by provider or at the time open is called.

That's what I was thinking initially. My reasoning for doing it this way is that the purpose of using the injection token in the first place is to be able to change the default values, but by creating a new MatSnackBarConfig, you're already setting some defaults that come with the class. That being said, we can go with the simpler solution and see if people end up being confused about it.

I see specifying the default options as overriding the defaults for the ones you provide; anything left unspecified would just be the original default.

The feedback has been addressed.


Revisit default Git clone depth

ods-start performs a shallow clone (depth=1). This leads to the following warning in SonarQube:

Shallow clone detected during the analysis. Some files will miss SCM information. This will affect features like auto-assignment of issues. Please configure your build to disable shallow clone.

Should we default to a non-shallow clone?

#685 introduces an easy way to change the clone depth:
params:
  name: start.depth
  value: "0"

With that the ticket here may be repurposed to discuss:

Is depth a good name for the parameter? Maybe clone-depth would be better.
Is 1 the best default? Maybe 0 would be better.

Just realised this isn't true. Setting depth=0 leads to an error, instead of disabling depth. We could accept 0 on the task param level, and then not render add the depth flag to the git command to make it work. Otherwise users would need to find an arbitrary high number, which isn't ideal.

Will change the default to 0 and disable the flag in that case.


fix: use hardcoded running version in User-Agent header

:tada: This PR is included in version 3.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix the cron-job, which does not seem to be working.

Data is now at Nov 23 and that is more than a week ago.
& the website is still displaying the Oct 26 - Nov 2 data.
Why is it not updating?

Fixed.

@vivekrai what was the issue?
On 6 Dec 2015 11:48 a.m., "Vivek Rai"<EMAIL_ADDRESS>wrote:

Closed #89.
—
Reply to this email directly or view it on GitHub.

The cron job was run by root which didn't know the location of 'nikola`. I
just added a path variable to the cron script.
--
Vivek Rai
https://vivekiitkgp.github.io
On Mon, Dec 7, 2015 at 8:22 AM, Max Klein<EMAIL_ADDRESS>wrote:

@vivekrai what was the issue?
On 6 Dec 2015 11:48 a.m., "Vivek Rai"<EMAIL_ADDRESS>wrote:

Closed #89.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/hargup/WIGI-website/issues/89#issuecomment-162395395.


[CSDiag] Account for expression sanitizer mutating AST

After calling get{Possible}Type{s}WithoutApplying types have to be
re-cached afterwards because sanitizer (run as part of the constraint
generator) can mutate AST to e.g. re-introduce member references.
Resolves: rdar://problem/46497155

@swift-ci please test


GUI option to start DataLayer services at app launch

Add a new "Data Layer" tab to Settings
Options to:
"Enable Data Layer" --> Starts the DL RPC service
"Enable Data Propagation Server" (subordinate to the enable DL checkbox)

This merged - at least the single option. The Propagation server piece is delayed due to delays in the under laying service - Creating tracker for that


Add support for syndication_links plugin

Broken out from #4.
This adds support for @dshanske's syndication_links plugin - https://github.com/dshanske/syndication-links

I added a bit different version... what do you think?

Thanks a lot for your merge-request!

👍  looks good


Feature Request: Hover over the table name when you add a table or widget

When we want to add a page based on another table and the table's name is long, sometimes we can't know which table we are selecting.
For Example:

As you can see in the image, we have for example three tables shown as DATA_StorONE_...

Hi @gmouawad.
A tooltip should appear when you hover over a long table name. Is it not appearing?

Hello @georgegevoian
No it is not appearing in our docker local installation.

When was the last time you pulled the image? The tooltip was added about 6 months ago, so it might be a case of the image being old.

is it included in the self managed version of Grist? we didn't update grist in a while. the image used is gristlabs/grist-ee
I see this github is core, is there any grist-ee github page?
The image was taken the moment I created this issue @georgegevoian

@gmouawad I've updated the grist-ee docker image.


Ctrl-Z Does Not Suspend ov Properly, Instead Launches a Subshell

Expected Behavior:
When pressing Ctrl-Z, the ov process should be suspended, allowing the user to return to the shell that launched ov. In this shell, using the fg command should resume the ov process. This behavior should also apply when ov is used in a pipeline, such as in cat hoge | ov, or with commands like man or bat.
Actual Behavior:
Pressing Ctrl-Z launches a subshell specified by the SHELL environment variable as a child process of the ov process. To return to ov, the user must exit this new subshell. However, this results in the loss of command history in the subshell, and no jobs are shared with the parent shell. Additionally, when ov is used within a pipeline, it cannot be properly suspended and instead resumes immediately.
Suggestion:
Please refer to the implementation of the suspend function in less for guidance on achieving the expected behavior: less signal.c implementation.
Tested on:
Ubuntu 22.04
go version go1.21.2 linux/amd64
<EMAIL_ADDRESS>

Thank you for Issue.
I thought there was a problem when I sent signal, but it worked now.
I will change it.

The default has been changed to not use subshell on platforms other than Windows.

Thank you for your response. I have confirmed that it is using suspend instead of a subshell. I verified that on the master branch, ov suspends as expected when used alone.
However, when used in a pipeline, the upstream process does not suspend and seems to get stuck. I'm not very familiar with the details, but I think it might be necessary to unmask SIGTSTP and send SIGTSTP instead of SIGSTOP. It seems that less handles it this way.
I tested on bash, zsh, fish and tcsh on Ubuntu22.04.

Thank you so much.
I fixed it in #639.

Thank you. It worked as expected on the send-process-group branch. I can now use it in my regular workflow.

Thank you. It worked as expected on the send-process-group branch. I can now use it in my regular workflow.

Thank you for the wonderful report!

I will Close because it was released. thank you.


Replace mitmproxy dockerfile with the one provided by the developer

I figured this must be more robust. it's based on Alpine as well.

oh wow cool


User overloaded types for createRequest

Expected this to just work. Looks like TypeScript won't currently infer overloaded types when providing a union type source


environment.noXlibs: disable gnome3 support for pinentry

Motivation for this change
environment.noXlibs already disables gtk2 and qt support when noXlibs is set -- disable gcr (gnome3) as well to avoid dragging in GTK3 and many more heavyweight dependencies.
Things done

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

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

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

[x] Determined the impact on package closure size (by running nix path-info -S before and after)

cough I want to see numbers!

/nix/store/vq4848x45ci7djh5bakj5qwrh1yf2r2h-pinentry-1.1.0        284260848
/nix/store/kwba74cjc607bxwszcf3j3y9yfcbqgxw-pinentry-1.1.0         50182848

And for nixos-config = { environment.noXlibs = true; } looking at full system closure size:
/nix/store/wzsvnkqrd73w1wxk3ygphh6pna443k38-nixos-system-nixos-19.09.git.d521db7                 860.0M
/nix/store/g236lfra3j79rbgm13sxrky8i821fysi-nixos-system-nixos-19.09.git.4e0af5c                 755.0M

Very nice, thanks!


DP-9370 - use cc-service-bot to manage Semaphore project

Background
Currently, some existing Semaphore projects were created manually via the Semaphore UI.
cc-service-bot is a tool for managing Semaphore projects via code (amongst other things).
Refer to the user guide for more information.
The DevProd team is migrating all Semaphore projects, which are not currently managed by cc-service-bot, to be managed by cc-service-bot.
Migration Changes
This automated pull request changes

Service YAML file, service.yml

Rationales for argument values
The configuration here was selected based upon the settings of the existing Semaphore project. This configuration matches the existing project where possible. If desired, you can change the configuration to match your team's other repos. It was decided that it was more likely that you would like to maintain the existing settings for a project than conform to other projects you may own. It was impossible to have a one size fits all approach that would work for the migration tooling because different teams want different things.

lang and lang_version are set to unknown because they are required by cc-service-bot. However, they are not used in this use case. They would only be used if you had pipeline_enable: true (or omitted it). In that case, these arguments would be passed to sem-version in the .semaphore/semaphore.yml file.
git.enable is set to true because it results in the generated Semaphore project configuration to be committed to the repo under .semaphore/project.yml. Whether this is enabled or not, the same project gets generated. Additionally, it implies to auto-commit changes rather than leaving them stashed for engineers to commit manually. This basically separates the various plugins' updates into logical commits instead of lumping them together into a single commit to parse. Docs:

https://confluentinc.atlassian.net/wiki/spaces/Foundations/pages/2870740796/Add+Git+and+Git+Hooks
https://github.com/confluentinc/cc-service-bot/blob/master/plugin_git.py#L4

pipeline_enable is set to false because if it is set to true, then cc-service-bot will modify the .semaphore/semaphore.yml file to match the standard implementation. Since the existing project was implemented manually, it is assumed that this differs from the cc-service-bot design, and we wouldn't want to overwrite it.
triggers is set based upon your existing project's settings. The default behavior for the cc-service-bot is ['branches', 'tags', 'pull_requests']. If that is what you would like for this project, then you should delete the triggers parameter.
branches is set based upon your existing project's settings. The default behavior for the cc-service-bot is ['master', 'main', '/^v\\d+\\.\\d+\\.x$/']. If that is what you would like for this project, then you should delete the branches parameter. Note that [] means workflows are triggered for all branches.

Actions

If you are happy with the changes, please merge them.
If you have concerns about the changes, either modify the pull request directly or comment on this pull request, tagging the Developer Productivity team (@confluentinc/tools).

@cchristous as discussed, closing this PR since this repository doesn't accept any PRs.


board suggestion: solenoid controller

For an odour stimulator project, I created a board that delivers a "spike and hold" signal. This was based on the schematic offered by the company that makes the solenoid valves that we are using for that project.
I would like to suggest to adapt the created board so that it works with BeeHive.

Right now the board takes in 24V to drive  12V solenoid valve. If we want the BeeHive version of the board to have the same specs, we would need to add a step up regulator to the board, so that it would take the standard 12V from beehive, convert it to 24V for the spike period of the signal.
Other than that, we would just need to change the connectors in the current board, as the board, the way it is, already has inputs to drive 2 solenoids.

Hi @amchagas ,
I was quite puzzled with the statement "the board takes 24V to drive 12V solenoid". I looked into schematics that is provided by
the manufacturer and that is present in your repository. Am I right in an assumption that solenoids in this system work the way described by the manufacturer in the first passage? (fig. 1)

Fig. 1
I think that this is it because otherwise it is quite hard to understand why we need to step up 12V to 24V to then step it down again to 12V.
Regarding the implementation, yes, I think your idea is right. I will look into how to implement step up regulator and change connectors

Hi Ihor,
The figure/explanation you shared is why the board needs 24v to drive a 12v solenoid. From what I understand this could be applied to any solenoid, and not just the one from this specific manufacturer?
Therefore we could add this circuit to step up the voltage, and the odour stimulator then becomes beehive compatible/driveable.. (I wonder if there are any ICs that allow for either step up/down depending on a combination of resistors? - this would make it quite flexible, no?)
The other option would be to skip the step up and make sure we use lower rated solenoids.
Thanks for looking into this!

You can also see below how the signal driving the solenoid looks like..

Hi @amchagas ,
thank you for the information provided. I have been looking into this and I too think that the recommendations from the manufacturer could be applied to all solenoids, regardless of their origin (except those that explicitly mention that their solenoids cannot be used under these conditions).
Regarding the step up regulator, I would like to ask quite a simple question: what is the best outcome for us? My opinion is that if we need 12V solenoids and "spike and hold" signal then we will try to reach this goal rather than changing it to omit implementing drastic changes.

Hi Ihor,
the advantage that I see is that by using 12V solenoids we can add this odour stimulator to the "portfolio" of tools developed with BeeHive. Also as far as I know, a lot of solenoids use 12V by default... (but this is from observing, and not actually researching things).

Hi @amchagas ,
based on the information you provided, I think we should stick to 12V solenoids. I will look into how to make a step-up regulator or what can be done with this issue. Right now, I am thinking about making a step up regulator as a discrete module and finding one that can be bought. What do you think? Are there any other things in the system that may require 24V (like motors, servos)? If not, than a step-up regulator on a PCB with solenoid driver seems as a rational idea

Hi Ihor,
thanks for looking into this. It is hard to tell if we ever going to need 24V for other applications. I think we should aim for not needing that, but hard to say with 100% of confidence.
At the same time, a step up as a discrete module is a good general solution, but I wonder how much current we would be able to get from the step-up circuit (as far as I understand in step up applications you end up with a higher voltage having to "sacrifice" current?) to make it useful for other applications? (I really do not know how much motors, servos that are running at those voltages would need in terms of current?). For the solenoids we are using, I know that they are fairly low current...
Also, for the solenoid board, having an isolated step up would mean we take the 12V from the main board, step up to 24V (for the spike phase of the signal on the board) then step it down again to 12V (for the remaining of the signal) on the solenoid board, maybe that is a bit wasteful? But than again if it works, it should be fine, since having a generic step up solution might be best?
Another argument on the solenoid board, is that having an external step up would allow us to use the solenoid board with different types of valves... I mean, if we use a 6V rated valve, we do not need the external step up, and can simply use the step down on the board. this would make the signal be 12V in spike phase, than 6V for the rest of the signal (I hope this makes sense?)
I have no preference relating to this, and would go with what solution you think it is best!

Hi Andre,
thanks for sharing a detailed opinion on this! Truly, I cannot seem to find a middle ground here, though I am gravitating towards a discrete regulator more. Yes, regulators are all about converting one component of power, voltage or current, into another, but power will remain constant in every case (or rather, it can only change via decreasing).

Hi @amchagas ,
take a look at this step-up converter. In essence, it is what we need: it is relatively cheap, adjustable and is proven to work.
Regarding our own implementation of something along these lines, it is still complicated. I spend time investigating possible solutions. I am still unsure what approach to take so that will take time. Regardless, I think the module that I mentioned above is worth checking out.

Hi @amchagas ,
take a look at this step-up converter. In essence, it is what we need: it is relatively cheap, adjustable and is proven to work.
Regarding our own implementation of something along these lines, it is still complicated. I spent time investigating possible solutions, though I am still unsure what approach to take so that will take a while. Regardless, I think the module that I mentioned above is worth checking out.

Hi @isobianin , I think the module looks good.
maybe an approach would be to buy some of them, test them out with the system and make decisions from there?
I think having some hands on experience with it could help more informed decision?

Hi @amchagas ,
well, it is certainly a good decision, provided there is a budget for it. Honestly, I think that they will come in handy in all sorts of projects

Hi @amchagas ,
I have a question regarding coils that we are using. In the solenoid controller board you used coils that are much smaller than those that had been used in FlyPi board and later adopted by BeeHive. From both footprints I can deduce that they are not interchangeable. The question is, what inductor are we going to end up using? It would be convenient to have only one type of inductor across the system

Hi @isobianin,
the boards look good! and great idea implementing the boost with the parts we are already using!!
About the inductor, we should stick with the same model and footprint we are already using. The different one on the other solenoid board is likely an oversight from my side...

Hi Andre,
splendid. I changed the footprint, should be fine now.

Hi @amchagas ,
I made some changes to the discrete module, mainly, I added another connector that can output 24V just in case there are multiple low amp 24V modules (fig. 1)

Fig. 1 - Two 24V output connectors

Boards are arriving this week for testing!
In the meantime, I was thinking about the input signal pins.
Right now, they are 2X 2pin, with input and ground. To easy compatibility with the rest of the system, we could simply replace them for 1X 4 pins that is compatible with the connectors leaving the central hub.
I made the necessary changes and uploaded them as a pull request: https://github.com/BeeHive-org/BeeHive/pull/32

Hi @amchagas ,
have you looked into this thoroughly? I remember that we thought through a convention regarding high ampere signals, in particular, that where high amperes present, two connectors are used. On the other hand, not all applications require a lot of amperes, so we settled to have two connectors "just in case" but commonly use only one. If you are using boards right now and it is more comfortable to have 1X 4pins connectors then there should be a lot more changes across the whole system.
As an example, the main board should have two connectors for 2 independent 12V5A lines. Thus, it is possible to power more electronic devices rather than just using 1X 4pin conn. But again, it depends on how the board is actually being used so if 1x 4pinn is more convenient so be it

Oh sorry I probably didn't express myself properly.. the connectors I am referring to are the digital signals that control the turning on/off of the rest of the circuit.. so they are normally low power 3.3v to 5v Volts

I see, no worries! I have just got a little confused over this


Is there any way to achieve a "colspan" feature?

I need to group two columns in one, do you know if I can try something to achieve this?
It would be amazing to have a colspan feature :)
Thanks!

Hi @pedrobrost, We are using a forked version of js-xlsx library and for the moment it does not support colspan How to write merged Cells #92,
But, with our forked version, you can get the merged styled look by applying style to it (eg, white background and/or no borders),
Hope this will help, re-open this issue if you need further help.


Editor does not register keyboard input anymore after un-focussing the windows and refocussing

Godot version
4.0.1
System information
Windows 11, Vulkan, ryzen 9 5900X, rx 6700xt
Issue description
After tabbing out of the editor and then going back to it, the editor does not register keyboard input (of any kind, from any keyboard I have plugged in).
I have verified that my keyboard is working and the editor is focused. This is not specific to the project I am working on, happens regardless of the project,
Note that this does not always happen, but always sometimes. Though I don't know what causes the issue/ when exactly it happens.
Steps to reproduce
Create a new project in Godot 4.0.1
Tab in and out of the project a few times (maybe run the project a few times as well, maybe lock the account (Win + L))
Try to use any keyboard shortcut in the Godot editor
Minimal reproduction project
See "Steps to reproduce"

This is a duplicate of another issue, but I can't find it right now.

This is a duplicate of another issue, but I can't find it right now.

I think you're looking for https://github.com/godotengine/godot/issues/74957 and https://github.com/godotengine/godot/pull/75172

Seeing this happen almost 100% of the time after alt+tabbing in windows. Pressing the win key resolves it but it comes back after a while. Super annoying!

This should be fixed in the master branch, but the fix hasn't been cherry-picked for 4.0.x yet. Can someone try building master for Windows and check if the issue still occurs there?

This is indeed a duplicate of the linked issue. If the error is still reproduceable even after the fix, we should reopen the original one.
The fix will be available in 4.0.2.

I'm seeing this issue (or another with the same effect) on 4.2.2 on Ubuntu 20.04. Not on every Alt+Tab but fairly regularly (every 5-15 minutes). Keyboard shortcuts work but I can't type into text entry fields.

@domdfcoding This is likely an unrelated issue, so please open a new issue.


Added GL_HALF_FLOAT_OES token

Added the missing GL_HALF_FLOAT_OES token for the GL_OES_vertex_half_float extension. in GLES2 where GL_HALF_FLOAT is not available (only available in GLES3+) , this token needs to be used instead when the GL_OES_vertex_half_float extension is available.
References:
https://www.khronos.org/registry/OpenGL/extensions/OES/OES_vertex_half_float.txt
https://www.khronos.org/registry/OpenGL-Refpages/es2.0/xhtml/glVertexAttribPointer.xml
https://www.khronos.org/registry/OpenGL-Refpages/es3.0/html/glVertexAttribPointer.xhtml

the other half of the work would be in renderer_gl.cpp. Would something like this be okay for you @bkaradzic?
	static const GLenum s_attribType[] =
	{
		GL_UNSIGNED_BYTE,            // Uint8
		GL_UNSIGNED_INT_10_10_10_2,  // Uint10
		GL_SHORT,                    // Int16
#if BGFX_CONFIG_RENDERER_OPENGLES == 20
        GL_HALF_FLOAT_OES,           // Half
#else
		GL_HALF_FLOAT,               // Half
#endif
		GL_FLOAT,                    // Float
	};

It should be done this way:
https://github.com/bkaradzic/bgfx/blob/71e1dd66664153d5f69d85af5aa759dfc0c90f55/src/renderer_gl.h#L100

The header you modified is part of GL, not GLES2, and GLES header has define you need:
https://github.com/bkaradzic/bgfx/blob/71e1dd66664153d5f69d85af5aa759dfc0c90f55/3rdparty/khronos/GLES2/gl2ext.h#L695

woops. you are right. I edited the wrong file. sorry. will try to do this another time.


docs: add note about minimal pydantic version

see https://pydanticlogfire.slack.com/archives/C06EDRBSAH3/p1714923558748049

I'm not sure what the minimum supported version is, but it's definitely less than 2.7.0.

I think it's 2.6.2 - need to check.

I can only say that I haven't worked with 2.4.2 but with 2.7.0.
Best would be Samuels solution and adding a console statement, I just filed this PR so there is at least some information about it.
But I'm fine to close it if you don't think it's appropriate.

2.5.0 is the minimum version.


wip: OCPBUGS-16032: Remove target directory in backup function of upgrade-recovery script

On a failed copy attempt, a retry will result in the source directory being copied under a partially copied target directory. This results in a nested directory structure in which the nested directory holds the most recent copied contents.
This PR removes the target directory specification from the copy command cp -Ra.
/cc @donpenney @jc-rh

/jira refresh

/jira refresh

/jira refresh

/lgtm

/approve

/cherrypick release-4.13


Update LICENSE

Including svelte-i18n license and some of the shape file licenses in the notice file.

I'm not sure if the LICENSE file is the appropriate place to add these attributions.  I will reach out to discuss.

I had an discussion with Tomer, we're going to take a different approach.  Current plan is to have generate a notices file separately from LICENSE and reference that in the front end so users can review dependencies.
I'm going to close this PR.


code-server heartbeat logic

Description
I am running code-server with multi-tenancy and would like to know when to kill the users containers after a period of inactivity. I am using the heartbeat file that code-server touch's every minute for this.
I would like to know on what basis does code-server decides to touch the heartbeat, i.e what are the conditions that code-server decide as activity and inactivity.

A little dig on #1115 will let you know how it works.

@sr229 I had a look at this. However I am not able to gather under what conditions is the heartbeat function called.
ex scenarious:

no websocket activity
code-server view not in focus (code-server tab in background_

We don't have the ability to send that serverside. Even then, @Microsoft is more than concerned for this rather than us.

If there have been any HTTP requests in the past minute or if there are
any active web sockets then the user is considered active.
An active web socket is one that is connected even if there is no
activity on it. So this includes idle users that just have the tab open
(even if it's in the background).
If the user closes the tab or their network connection dies then the
heartbeat will stop until they connect again.
There's currently no way to detect if the user is idle. I was thinking
about capturing idle data on the client-side (probably a timer based on
mouse and keyboard activity) then sending that through the socket to the
server but it's not implemented yet. Maybe we could show an "are you
still there?" message as part of that.

Hi @code-asher - are you still looking for a feature for the "are you still there message"? (since we use it today to clean up resources - pods in kubernetes) This would be really useful to us and if the community also wants it I would be happy to work on this.

Yeah, I'd be open to merging a feature like that. I think we initially decided not to because users will usually disconnect naturally (close the tab, close the browser, computer goes to sleep, I think even in some cases if the browser tab isn't focused it'll close the websocket), but I'm not sure how reliable that all is so a dedicated feature might be helpful.

On start up I usually just try a curl request, and see if I get a 500 or a login redirection, looping until I get a response before reporting to the user they can proceed.

@code-asher any suggestions on approach. Here's what I am thinking:

If we already have a function that's triggered on some user activity in the IDE I can hook to that with a timeout of say 'x' minutes and raise a "reconecting.." like popup ("Are you still there" - Yes/No). If "No"/ response timeout then possible touch a file that indicates user doesn't seem to be active. The popup will still be active and the user can choose to click yes at any point in that case the file will be deleted.
If we do not have the above, then raise this popup every 'x' mins to the user for the same functionality as above.

Any other suggestions/approach that you can share?

I don't think we have anything specifically for user activity so we'd need to add something. My basic idea is:

Hook up event handlers to the body, probably keydown, mousemove, mousedown, scroll, touchstart.
Whenever those events fire update the last active time.
Add an interval that shows the popup if the last active time is greater than x minutes.
If the user does not answer the popup within a set timeframe disable automatic reconnection and disconnect the web sockets.
If the user comes back and answers the popup reconnect the web sockets and re-enable automatic reconnection.

The first three steps can all be done in lib/vscode/src/vs/server/browser/client.ts. The last two will involve modifying the existing reconnection logic in VS Code since right now it just always automatically reconnects and we need to be able to toggle that on/off.

@code-asher For user idle detection, there's a Idle Detection API in chrome:
https://web.dev/idle-detection/
Maybe you can looking at some polyfills etc.

Thanks for the tip! This looks like exactly what we'd need.

If there have been any HTTP requests in the past minute or if there are any active web sockets then the user is considered active. An active web socket is one that is connected even if there is no activity on it. So this includes idle users that just have the tab open (even if it's in the background). If the user closes the tab or their network connection dies then the heartbeat will stop until they connect again. There's currently no way to detect if the user is idle. I was thinking about capturing idle data on the client-side (probably a timer based on mouse and keyboard activity) then sending that through the socket to the server but it's not implemented yet. Maybe we could show an "are you still there?" message as part of that.

Is there any chance that you are exposing the api which updates the heartbeat file? I want to see the timestamp of heartbeat file for my usecase. Do you have any api for this?

Basically I have a shell script which loops over each of the users currently running a docker instance with code-server.
This script collects the time stamp (how old the file is) of the code-server heartbeat, as stored in the users docker mounts home directory.  It also collects another time stamp of when the user started their instance, so as to know if the docker instance is only just starting up, and code-server may not be running or the user hasn't connected to it yet.
Basically I get a list of users, and how long they have been 'idle', as well as other information, like on which 'node' there docker instance is running, and what pre-pared image they are running
From this I do the following.

Unless 'queit' (running from cron) report a table of running users and there idleness
kill off any user that has been 'idle' too long (> hour)
log counts of, total user instances, if they are 'idle' (> 10 minutes), counts of images, and how many on each docker node.

These stats are logged into system logs (remote) so that I can now generate some usage graphs (splunk), or run other scripts to report on numbers of unique users using the system per day.
So while the scripts actions are fairly simple (loop of logged in users reported 'idleness' from heartbeat files),  it does quite a lot with that information.
Getting a time from a heartbeat file (perl gets file times in seconds since time perl was run)...
# Constant
INFINITE=99999999

file_timestamp() {     # get the timestamp of a file.
  local log="$1"
  if [ -e "$log" ]; then
    perl -e '
        $last=(stat( "'"$log"'" ))[9];
        print $^T - $last, " ", $last, "\n";
      '
  else
    echo $INFINITE 0
  fi
}

SO I call file_timestamp() for each running users heartbeat file, and a file touched when they logged in, and take the smaller.   This works even when the heartbeat file is 'yet to exist' for first time users.
I hope this helps...

thanks for this @antofthy. Can you explain this code snippet in detail? I am not familiar with Perl, when I am trying to run this, I get a compilation error. I just want to see the last updated time of heartbeat file of vscode instance programmatically.
manually, I would go to the cluster->pod->deployment of code-server vscode and open a shell to that. Then I simply go to the path /root/.local/share/code-server and run stat heartbeat. This gives me modified time etc. Any idea on how can I do this programmatically?


Update localization.rst

Summary
Howto for adding a new WIP language.
Ticket Link

Thanks, @metanerd, this is super-helpful. :)


fix(select): fix select icon bug

Checklist

[x] Fix linting errors
[x] Tests have been added / updated (or snapshots)

Change information

:tada: This PR is included in version 1.0.0-alpha.31 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Reduce away intersections that aren't assignable to constituents in CFA

Fixes #37659.

@ahejelsberg shouldn't A & B be assignable to A in this case? A & B by definition has all of A's private fields, no?

@weswigham Well, yes, but A and B also have a conflicting private member x and we consider it an error to access conflicting members (introduced years ago in #13990). The access happens as we're checking the properties from the whole intersection against the properties of each of A and B, a check that was introduced recently in #37195.

@weswigham We're basically trying to have it both ways. It would seem reasonable for A & B to be assignable to A and B even with the conflicting private properties, yet we rely on an error being issued in the link below as an early warning that a mixin produced an invalid type.
https://github.com/microsoft/TypeScript/pull/13990/files#diff-4b0cabe62740b5899b9a7850e502007aR95

Superceded by #37762.


Fix python 3.7

It's not that pretty but does pass tests in all python versions (and so does Malcolm4).
Perhaps when we drop Pythons < 3.7 we can review and simplify annotypes.

Codecov Report

Merging #3 into master will increase coverage by 1.26%.
The diff coverage is 96.77%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   97.21%   98.47%   +1.26%
==========================================
  Files          25       25
  Lines         898      722     -176
==========================================
- Hits          873      711     -162
+ Misses         25       11      -14

Impacted Files
Coverage Δ

annotypes/version.py
0% <0%> (ø)
:arrow_up:

annotypes/_calltypes.py
100% <100%> (ø)
:arrow_up:

annotypes/_stackinfo.py
100% <100%> (ø)

annotypes/_array.py
98.21% <100%> (+0.1%)
:arrow_up:

annotypes/_anno.py
98.79% <100%> (ø)
:arrow_up:

annotypes/_typing.py
69.23% <100%> (-16.49%)
:arrow_down:

Continue to review full report at Codecov.

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

All suggestions completed.


refactor(AlgebraicGeometry/EllipticCurve/*): refactor base change for ring homomorphisms

Refactor Weierstrass curves and nonsingular rational points to allow for base changes over an arbitrary ring homomorphism φ : K →+* L. This generalises the natural map algebraMap K L and removes the necessity for Algebra K L, but also gives an easy way to define DistribMulAction for the action of L ≃ₐ[K] L on the nonsingular rational points over L, since L ≃ₐ[K] L restricts to L →ₐ[K] L that restricts to L →+* L. The old notation W⟮L⟯ is preserved for base change over algebraMap K L.
Also remove some unnecessary coercions in the Weierstrass file.

They should not need to write algebraMap R A in most cases, as the notation W⟮A⟯ still exists for a Weierstrass curve W over R and an algebra A over R. I think it is worth it, as in the following use case:
variable {R : Type} [CommRing R] (W : Affine R) {A : Type} [Field A] [Algebra R A]

instance : DistribMulAction (A ≃ₐ[R] A) (W⟮A⟯) where
  smul := fun σ => Point.ofBaseChange W σ.toAlgHom
  one_smul := fun P => by cases P <;> rfl
  mul_smul := fun _ _ P => by cases P <;> rfl
  smul_add := fun _ => map_add _
  smul_zero := fun _ => rfl

Here, the proof for smul_add follows trivially from map_add' in ofBaseChange. In the old definition, I can't consider an arbitrary automorphism of A that fixes R since there is only algebraMap R A, and would have to rewrite the same proof.

If there is no need to write algebraMap I am happy with the modifications, thanks!
bors d+

bors r+


Reactivity of charts to data

I am trying to update the chart based on changes to the data, especially values such as:
title
label
data
But it doesn't seem to be working. Any idea why?

Fixed this issue in<EMAIL_ADDRESS>


Use uniform representation of schemas

There should be a consistent data type for schemas. Right now, we have type-level schemas and value-level schemas. With the upcoming changes to the julia compiler, this should no longer be necessary. A schema should then be something that looks like the following.
(obs = (:E,:V), attrtypes=(:Weight), homs = (src=(:E,:V), tgt=(:E,:V)), attrs = (weight=(:E,:Weight)))

It has to be a namedtuple instead of a struct because then it can be stuck inside a type. There can also be a "dynamic" version, where obs and attrtypes are vectors and homs and attrs are dicts. Then we can get rid of the whole SchemaDesc, SchemaDescType, SchemaDescTypeType garbage.

For reference, this feature will be enabled by this PR https://github.com/JuliaLang/julia/pull/46300, which should land in Julia v1.9.


The key was not present in the dictonary when joining channel, but channel exists.

http://image.prntscr.com/image/d3b99382983c48c9937199413a896ef8.png
However, when using HexChat I can join the channel fine.
I'm trying to connect to the osu IRC. (https://osu.ppy.sh/wiki/Internet_Relay_Chat)

Nothin'?

Still an issue, and haven't found a way around this.

I also randomly encounter this issue as well. Has anyone found a fix?

I've also recently started having this issue, but it only started when i tried to build for x64.

How do you reproduce this bug?

Join a channel, it throws the error, even when the channel exists. Followed the sample code.

osu IRC is probably not standards conformant.

ppy won't be happy to hear that.
I am pretty sure it also happened on freenode as well.

@Arefu > Bancho uses a custom implementation of the IRC protocol, and is in no way a full-feature implementation; therefore, do not expect all features of your IRC client to function properly.
Anyways, if you do happen to have a log or the exception stack trace, it could help. It's probably related to #54.

Alright, I'll dig up my code when I get home if I still have it lying around.
I figured it would be standard for it to be able to broadcast what channels exist, even if I knew it wasn't standard compllient. Anyway, expect me to reply when I get back from work

I have an event set up so it outputs a line when the channel list is received. That line does not show up.


React/RCTBridgeModule.h not found

This error just popped out of the blue. Has anyone run into this issue?

I'm trying with a new generated app via ignite:
ignite --version
> 1.13.0

Error:
In file included from /Users/Matt/Work/ed/scratch/node_modules/react-native-device-info/RNDeviceInfo/RNDeviceInfo.m:9:
/Users/Matt/Work/ed/scratch/node_modules/react-native-device-info/RNDeviceInfo/RNDeviceInfo.h:12:9: fatal error: 'React/RCTBridgeModule.h' file not found
#import <React/RCTBridgeModule.h>
        ^
1 error generated.

The way I got around this error for now was keeping the version at 0.9.7. If you have a carrot in your package.json ( "^0.9.7"), just remove the carrot (^) and it should be fine for now.
My RN project is using 0.34, so maybe that's the cause of this problem.

Hmm - possibly related to https://github.com/rebeccahughes/react-native-device-info/issues/140 then?

OK, so running react-native upgrade then react-native upgrade again (accepting all changes both times) and then react-native run-ios worked. It moved the app back to react-native 0.38.

This issue was caused by an error in the semantic versioning, which was fixed about 20 hours ago. Now, when running 'npm i' or 'yarn' with "react-native-device-info": "^0.9.6" in your package.json, version 0.9.9 will be installed, which effectively restores the iOS native header to the version that was in 0.9.7.

Running react-native link causes the original error to reoccur, but even after building successfully, I see
undefined is not an object (evaluating 'RNDeviceInfo.systemManufacturer')

OK, so starting afresh worked perfectly. Thanks @derekgreenberg 👍

@MattHall Sorry - should have mentioned that this fix is for those who are starting afresh, such as Ignite 1.13.0 users who are generating a new project.

Yep - sorry. Forgot this wasn't my issue!

If you version is 0.10.0, then this is no longer compatible with RN < 0.40, see #140 for detailed conversation (it is actually possible to support both RN < 0.40.0 and RN >= 0.40.0, it's just more work

Product -> Schemes -> Manage schemes... -> (your project) -> tab build -> (uncheck checkbox "Parallelize Build")

@Laconty there is no build tab anymore.

Just had to react-native run-ios instead of trying to run from Xcode. That put everything together right I guess.