Window Maximize Doesn't Properly Fit Screen

Describe the bug
When clicking the maximize button, my client only takes a portion of the screen. I have tried reinstalling runelite and deleting related files, changing display settings, and restarting my computer but nothing has helped. I tried with the official OSRS client and it works just fine.
To Reproduce
Steps to reproduce the behavior:

Open the client as normal
Login or stay logged out, doesn't matter
Click maximize button and then be sad :(

Expected behavior
Client is expected to take up the whole screen.
Screenshots

Environment (please complete the following information):

OS: Windows 10
RuneLite version: <IP_ADDRESS>
Launcher version: 2.0

Additional context
None

If you set DPI scaling override to System does that do anything?

damn you're my hero thanks for the speedy response!

This is probably still a bug I was just wondering if that would fix it or not.

well i'll leave open and let someone else decide then, ty

I am also having the same problem as RSNKitt3n. I'm sorry if me commenting this is useless/annoying, I'm not really sure how issues like this are brought up on Github as I've never used it, but I just wanted to let you know that I'm also having this problem. I was installing Runelite on a new laptop, and the same issue occurred. I tried the DPI override you suggested with no success.

If you use https://sigterm.info/runelite/RuneLiteSetup32.exe - does that fix it?

Am I supposed to uninstall and use that setup to reinstall Runelite?

What do you mean by "crashing"? I can run it and it works okay.

Crashing as in when I use that specific setup to install Runelite, the game does not make it to the log-in screen. Using the installation setup from runelite.net, the game installs properly and works fine aside from the issue stated in this thread.

Does the 32bit installer from runelite.net work?

Yes.

@RSNKitt3n are you available to test it instead?

This worked for a user in discord.

Upon rechecking my Runelite version, the 32 bit installer does not work for me, but the 64 bit installer does. Sorry for the confusion.

all of this is true for me as well, but the 32bit version runs very poorly for my pc is 64 bit. There seems to only be a link on runelite.net for a previous version in 32bit. getting like 15 fps where before I had at least 40 or 50 on the 64bit client

i might just stick with the fcked up maximize client just to get better fps, over this previous 32bit version provided on runelite.net

If you use https://sigterm.info/runelite/RuneLiteSetup32.exe - does that fix it?

This fixed my resolution issue. Main monitor is 1440p, side monitor is 1080p. Issue was launcher was using the 1920x1080 resolution of the side monitor rather than the 2,560 x 1,440 resolution of the main monitor.

Fixed with launcher version 2.0.2.


remove in-place swap on host for sharded in FD

This is what we talked about @arakhmati @cfjchu
Now this test works : pytest models/demos/falcon40b/tests/test_falcon_mlp.py::test_FalconMLP_inference[BFLOAT16-SHARDED-0.9987-tiiuae/falcon-40b-instruct-prefill-1-64-8chips]
without having to make a separate host copy in python.
CC: @TT-BrianLiu

Something to consider is that this implementation currently duplicates the memory usage of the entire tensor when writing to device, which could be expensive for large tensors.
An optimization that would reduce memory usage but shouldn't affect perf is to potentially only remap the given chunk we are writing at a time, this way we can still do a contiguous write for the chunk while reducing total memory usage at a given time.


Activate a user

The estimated time to resolve the issue is 30 min

Time spent to resolve this issue: 20 min


release?

download site no work

Hi, as noted in the main readme of this project, it is no longer maintained for... reasons. However a better alternative is available which has roughly the same features as this project.
i'm copy-pasting the section of the readme:

You may want to check Stava, which is roughly the same concept but a bit more advanced with a few new concepts. This repository was created as an experiment and is no longer maintained, sorry for the wasted clicks.

I'm also closing the issue and updating the description of the project to mention this is no longer maintained while also linking to the alternative. Thanks for your interest in the project.


Various interface fixes

Fixes I wrote while playing around with interfaces introduced in #6213.
The first patch makes sure that the context name is fixed to ctx regardless of what the interface specifies it as - this is because the context is special, and the code in the defined function below requires it to be set to ctx.
The second patch is a rework of https://github.com/dagger/dagger/pull/6282, to ensure that it also works with interfaces (there's a weird case we can get where we generate the function cases, but don't do the struct codegen so we reference but don't actually create the concrete impl). Essentially, we should traverse the tree starting only at the top-level and work down from there. See https://github.com/dagger/dagger/pull/6282#discussion_r1446200476 for why the previous patch is insufficient.

Added a test in cc88419


[Question]: Can I use a version of the Bash task that's just before 3.226.2?

Task name
Bash
Task version
3.226.2
Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[X] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Ubuntu 20.04 / Ubuntu 14.04 32-bit container
Question
The latest update for the default bash command (bash@3) has broken my pipeline build:
/__w/_tasks/Bash_6c731c3c-3c68-459a-a5c9-bde6e6595b5b/3.226.2/bash.js:9
async function runBashPwd(bashPath, directoryPath) {
      ^^^^^^^^

SyntaxError: Unexpected token function
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:549:28)
    at Object.Module._extensions..js (module.js:586:10)
    at Module.load (module.js:494:32)
    at tryModuleLoad (module.js:453:12)
    at Function.Module._load (module.js:445:3)
    at Module.runMain (module.js:611:10)
    at run (bootstrap_node.js:394:7)
    at startup (bootstrap_node.js:160:9)

This is because I'm using an old version of node (the newest I have available) on a 32-bit container of Ubuntu. I installed node into the container by using the following lines:
# Getting node v6 because there's no newer precompiled version in trusty-i386
RUN curl -s https://deb.nodesource.com/gpgkey/nodesource.gpg.key | apt-key add - \
    && curl -sL https://deb.nodesource.com/setup_6.x | bash - \
    && apt-get update && apt-get install -y nodejs

LABEL "com.azure.dev.pipelines.agent.handler.node.path"="/usr/bin/nodejs"

Unfortunately, my target application must run on Ubuntu 14.04 32-bit, which is why I'm doing the build inside a container. Is there any way to use the previous revision of the bash task that doesn't require node 10 or newer? Or, can somebody direct me how to build a more modern version of node that I can run inside my container, so I can still do my builds? Any help would be greatly appreciated.

Hi @jovere, thank you for the question. Unfortunately, I can't help you with running node10+ on ubuntu 14.04, but, you can pin the task version by using<EMAIL_ADDRESS>or any version which is available for your organization.
But I'm afraid that the Bash task started to use node10 a long time ago, as I see the migrated PR was 3 years ago and the last version which supported node6 handler was 3.178.0.
In case your organization doesn't contain the last node6 version of the task, you could build it by yourself, upload to your organization using tfx-cli and use NodeTaskRunnerInstaller to install node6 runner to the agent in some cases, because some agents could not have node6 binaries inside it (better to check is you agent was built with node6 or not, all node version which agent uses placed in externals folder).

@DmitriiBobreshev Thank you very much for your help! I knew I could use top level versioning, but it didn't occur to me that I could get that granular. You have saved me a lot of stress.
I'm also looking into getting the latest node build for this system, but it's going to take a lot of compiling intermediate tools.
Thanks again,
Jeremy


Facebook Login or Sign up Clone Web Page

Have you read the Contributing Guidelines ?
yes
Description
In this project i have made a Facebook clone page using HTML and CSS
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
In this project i have made a Facebook clone page using HTML and CSS

@Moksh45 Please mention the Issue number it fixes.

@Moksh45  Do the requires changes ASAP else we will close this .

Have you read the Contributing Guidelines ?
yes
Description
In this project i have made a Facebook clone page using HTML and CSS
Fixes #issue_no
Zero (Because I am adding  a new folder in Web Development\Basic so  I am not Solve any Issue but Add a new Folder)
Checklist

[x]  I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x]  I've edited the README.md and link to my code.

Related Issues or Pull Requests
I am adding  a new folder in Web Development\Basic so  I am not Solve any Issue but Add a new Folder

done

done

What you have done , @

done

What you have done he @photon149 he said mention the issue number that you have been fixes but you didn't mentioned yet . It's been long time so we gonna close this PR .


Add the Native Target Framework

In the Blog Post (https://devblogs.microsoft.com/nuget/native-support/#native-target-framework) it is precisely stated that you should specify the "native" target framework, when developing C++ Win32 applications.

@dotnet-policy-service agree


Merge to live!

DisplayFormat new example
New Project Object Model deprecation notices

Docs Build status updates of commit 47c881a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Access.AcTextTransferType.md
:white_check_mark:Succeeded
View

api/Excel.Range.DisplayFormat.md
:white_check_mark:Succeeded
View

api/Excel.Range.Text.md
:white_check_mark:Succeeded
View

api/PowerPoint.Comment.AuthorIndex.md
:white_check_mark:Succeeded
View

api/PowerPoint.Comment.Parent.md
:white_check_mark:Succeeded
View

api/PowerPoint.Comments.Parent.md
:white_check_mark:Succeeded
View

api/PowerPoint.comment.collapsed.md
:white_check_mark:Succeeded
View

api/PowerPoint.comment.timezonebias.md
:white_check_mark:Succeeded
View

api/Visio.viscellindices.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add GitHub and Deployment URL at Every Project

[x] add GitHub repo link
[x] add deploy link
[x] cross check that, the link are working fine

all done as per task


RFC: adds bigquery projects

Hey guys,
We have an API for requesting bigquery projects, i.e.:
$bigQuery = $service->bigQuery();
$projects = $bigQuery->projects();

This returns an array because there is no generic Projects object AFAIK.

This looks good. Would it be possible to a little test coverage as well?

ping @bshaffer

:+1:
Could you also add ['listProjects'] to this array: https://github.com/bshaffer/gcloud-php/blob/add-bigquery-projects/tests/BigQuery/Connection/RestTest.php#L69

Due to the fact that BigQueryClient requires a Project ID, and this method doesnt require one, I don't think this is going to work. Here is what I have working:
use Google\Auth\CredentialsLoader;
use Google\Cloud\BigQuery\BigQueryClient;
use Google\Cloud\BigQuery\Connection\Rest;

function list_projects()
{
    $keyFile = CredentialsLoader::fromWellKnownFile();
    $scopes = BigQueryClient::SCOPE;
    $connection = new Rest([
        'scopes' => $scopes,
        'keyFile' => $keyFile,
    ]);
    $result = $connection->send('projects', 'list');
    foreach ($result['projects'] as $project) {
        printf($project['id'] . PHP_EOL);
    }
}

Unless there is a precedent set by other APIs, this one "Project ID-less" method is going to have to be omitted.

That's a great point.
My vote goes for leaving this out of the BigQueryClient and making sure we implement the resource manager API which appears to include the same functionality (and quite a bit more) with projects.list.

I was getting slightly different results from gcloud list projects than from bigquery.projects.list. But I'd need to look into it more to be sure.


WinverUWP has a different language even though my OS is English

This issue has been fixed upstream, but the version on Store hasn't been fixed yet.

Store should be updated now


set to make ctrl+k remember last buffer not all

it's quite annoying when want to paste last line but got all deleted lines by ctrl+k ,
or make ctrl+x works the way only remember last, it's a waste to make these two identical.
there maybe some keybind or options, and I generally read buildin guide and check the issues, It bothers me a few monthes until now I decide to ask some help.

Hi,
The CutLine command is actually appending each line to the clipboard.
If you want to replace the clipboard with the current line try binding CTRL+K like this:
    "Ctrlk": "SelectLine,Copy,Delete",

Kind Regards Gavin Holt

ok ,it works, paste one line should be the default setting.


Fix handling of nested fields/forms in getFieldFromState

I was trying to use the <Errors /> component on a form that wasn't stored at the root of state and found that it didn't work. This was because the utils/getFieldFromState helper assumed that model paths were always prefixed with a single path segment, and the tests only made sure such a scenario didn't cause errors to be thrown.

Great catch! Thanks for this.


Email is not fetched on first Sequence login

If user is not logged in Sequence, the askForEmail: true param is ignored and we don't collect the email until the user reconnects.
On new login the popup with email info collection is presented:

@Riglanto this is fixed now so closing


Add description label to ArgoCDSyncAlert

What type of PR is this?
Added "description" label to ArgoCDSyncAlert
JIRA - https://issues.redhat.com/browse/GITOPS-2639

/ok-to-test

/test unit

/test v4.12-kuttl-parallel

/test unit

@Rizwana777 it appears the unit tests need to be updated, primarily due to switching from message to summary/`description.

/test v4.14-kuttl-parallel

/test v4.14-kuttl-parallel

/retest


feat: do not kill ec when ecm restart  part-1 (#4184)

What is the purpose of the change

When the ECM restarts, choose not to kill the engine.
StatusUpdate message,  pid update message , yarn appid update message  of engine server(EM) directly send to linkis-manager service(LM), Make engine conn manager(ECM) service stateless .

Related issues/PRs
Related issues: #4184
Related pr:#4184
Brief change log

Replace the function of DefaultEngineConnListService class in Ecm with DefaultEngineConnManageService, DefaultEngineConnStatusCallbackService , DefaultEngineConnPidCallbackService.
(StatusUpdate message,  pidUpdate message , yarnAppidUpdate message  of engine conn(EM) service directly send to linkis-manager service(LM))
Modify EMRestfulApi.executeECMOperation method add logDirSuffix parameter.
Modify DefaultEMEngineService.stopEngine method parameter, adding  engineType, logDirSuffix.

Checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] I have explained the need for this PR and the problem it solves
[x] I have explained the changes or the new features added to this PR
[x] I have added tests corresponding to this change
[x] I have updated the documentation to reflect this change
[x] I have verified that this change is backward compatible (If not, please discuss on the Linkis mailing list first)
[x] If this is a code change: I have written unit tests to fully verify the new behavior.

Replace the function of DefaultEngineConnListService class in Ecm with DefaultEngineConnManageService, DefaultEngineConnStatusCallbackService , DefaultEngineConnPidCallbackService.
(StatusUpdate message, pidUpdate message , yarnAppidUpdate message of engine conn(EM) service directly send to linkis-manager service(LM))

These new codes are not pushed? @CharlieYan24

These new codes are not pushed? @CharlieYan24

I was trapped in building test environment and test the above code those few days. And fixed the serialize problem  in /userResources  and /getApplicationList api in RMMonitorRest class.


how to share url?

how to share url? such as web page，intenet imge

@qxwei You can share url  just like sharing text.  For sharing web image, you can download it to local storage and share it.

but，it call the exception as：PlatformException(error, file not exists, null)
ShareExtend.share("share text", "这是个网址，分享出去会是什么样子？https://www.twovs.com/");


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Formatted text into new json message

The json format requires, one line, no tab, no newline, a single string. No multiline strings.
This needs some processing on the -n "in data string", when piping a larger formatted file/information.
Make it possible to digest formated in-data to be stored in the json format.

Fixed for textfiles, cvs, and code-files, in release 0.4.1.


Cannot remove c:/ from find file path input on Windows. -> Cannot connect to remote via tramp.

What did you expect to happen?
After SPC f f I expect to be able to delete the current path and and enter something like /plinkx:session:~/afile.
What actually happened?
After SPC f f the root of the path, which is c:/ cannot be deleted and thus I cannot enter the correct path to connect to a remote machine.
Additional details:
Running emacs on windows, using putty to connect. With vanilla emacs this works.
Steps to reproduce:

SPC f f
Try to delete the whole path, including c:/.

 Doom 2.0.9, Emacs 26.3 on Windows 10 1903.

523e945a3ae2 may have indirectly resolved this issue. Could you confirm?

The issue still persists on the newest doom version, it is not possible to delete the drive root path.

Twice is not enough. For me, 4 times is needed. Finally, you will enter something like

Find file: //plink:session:~/afile.

plinkx doesn't work for me, so i have to use plink.

I tested this with Spacemacs, and 4 times is also needed. This is more likely a tramp issue.

Turns out this is an issue with how ivy deals with Windows paths. Seems like a few issues and PRs, and I believe they "fixed" it by making it require 3 or 4 slashes to get to the "true" root on Windows systems. May be best to bring it up with them if you want a better solution.
Or perhaps try
(after! ivy
  (setq ivy-magic-slash-non-match-action 'ivy-magic-slash-non-match-cd-selected))

Doom sets this to nil by default, which may be why you need 4 slashes. I'll probably revert this change soon.

I have the same problem but on a Mac. In Linux pressing C-x f f removes autocompletion. Not in Mac so I cannot use Tramp. It will leave // there.


Add CMS version question to GitHub issue template.

- Summary
It is often helpful to know the CMS version when a user is reporting an issue, especially not that we are releasing Beta versions.
- Test plan
N/A
- Description for the changelog
Add CMS version question to GitHub issue template.
- A picture of a cute animal (not mandatory but encouraged)

@Benaiah Done.

@tech4him1 great, thanks!

OK, squash done, should be ready to merge.


Update man page for energy efficient agent to describe changes for short running regions

estimate 1

Resolved with the removal of the agent for the time being.


Patch/SetType relations

After a long debugging session I figured out that the checker.Check modifies the ast state by setting the nodeType.
In my patcher I don't want my users to have to pass the context argument to the functions we call as that isn't ergonomic or necessary for the scripters to have to know if the function call requires an context argument or not.
From my example above our users would input: TestMe("1.33").GetFloat() but the actual script that the patcher generates will be TestMe(ctx, "1.33").GetFloat()
So when the checker runs at https://github.com/antonmedv/expr/blob/939aca18e66514ca0727c309f8df8c510a0e2fc1/expr.go#L182C1-L182C8
it will set the return value of the TestMe function to the invalid value interface{} instead of the correct value *scripting.TestStruct as the script is syntactically correct but fails to the checker before we run the patcher.
The change to checker.CallNode introduced in https://github.com/antonmedv/expr/commit/fcf2b55decfdb74892e3f30e28c73f45cd24ec15
in combination with
https://github.com/antonmedv/expr/blob/939aca18e66514ca0727c309f8df8c510a0e2fc1/checker/checker.go#L135
causes the issue I have.
Maybe what I need is to have another visitor that runs before performing the typecheck to modify the AST for my usecase?
Originally posted by @tw1nk in https://github.com/antonmedv/expr/discussions/473#discussioncomment-7643119

Fixed.


 I got logs from container as follow:
lxc-start: cgroup_rmdir: failed to open /sys/fs/cgroup/blkio/lxc/336807cf502eb942728ab9d2fcb421770099dbe9ef66cab8d8fc50c0e36bc25c
lxc-start: cgroup_rmdir: failed to open /sys/fs/cgroup/devices/lxc/336807cf502eb942728ab9d2fcb421770099dbe9ef66cab8d8fc50c0e36bc25c
lxc-start: cgroup_rmdir: failed to open /sys/fs/cgroup/memory/lxc/336807cf502eb942728ab9d2fcb421770099dbe9ef66cab8d8fc50c0e36bc25c
And docker log:
Jan 13 12:08:00 a1-ops-docker240031.hz docker[2586]: [d8de7c22] +job log(die, 336807cf502eb942728ab9d2fcb421770099dbe9ef66cab8d8fc50c0e36bc25c, 5843b2
Jan 13 12:08:00 a1-ops-docker240031.hz docker[2586]: [d8de7c22] -job log(die, 336807cf502eb942728ab9d2fcb421770099dbe9ef66cab8d8fc50c0e36bc25c, 5843b2
Jan 13 12:08:00 a1-ops-docker240031.hz docker[2586]: [d8de7c22] +job release_interface(336807cf502eb942728ab9d2fcb421770099dbe9ef66cab8d8fc50c0e36bc25
Jan 13 12:08:00 a1-ops-docker240031.hz docker[2586]: [d8de7c22] -job release_interface(336807cf502eb942728ab9d2fcb421770099dbe9ef66cab8d8fc50c0e36bc25
Jan 13 12:08:10 a1-ops-docker240031.hz docker[2586]: [error] deviceset.go:782 Warning: error waiting for device 336807cf502eb942728ab9d2fcb421770099db
Jan 13 12:08:20 a1-ops-docker240031.hz docker[2586]: [error] driver.go:141 Warning: error unmounting device 336807cf502eb942728ab9d2fcb421770099dbe9ef
my docker environment info:
docker info
Containers: 42
Images: 208
Storage Driver: devicemapper
Pool Name: docker-253:0-9570893-pool
Pool Blocksize: 64 Kb
Data file: /data/docker/devicemapper/devicemapper/data
Metadata file: /data/docker/devicemapper/devicemapper/metadata
Data Space Used: 54516.5 Mb
Data Space Total: 102400.0 Mb
Metadata Space Used: 39.5 Mb
Metadata Space Total: 2048.0 Mb
Execution Driver: lxc-1.0.5
Kernel Version: 3.10.0-123.el7.x86_64
Operating System: CentOS Linux 7 (Core)
The other containers is working in normal. only this one has problem. I don't know what triggered this problem. How to handle this problem?
Thanks

we recommend lxc-1.0.7, also have you tried with the natic execdriver

OK. I will try  lxc-1.0.7, thanks.
I must be use some lxc feature, so I can not change to execdriver.

let us know how it goes

@holmes86 how did it go

closing as stale, @holmes86 let me know if you are still seeing on 1.0.7

@jfrazelle I'm getting this with Docker 1.5 and the native-0.2 driver. I'm on CoreOS alpha.


argus.client.base.ArgusClientError: 'API Error encountered using endpoint on submit_results

scylla-enterprise-perf-regression-predefined-throughput-steps-vnodes test runs 3 load stages.
write and read stages are passed:
read
write
But mixed stage failed with :
10:08:27  2024-10-27 08:06:16.817: (TestFrameworkEvent Severity.ERROR) period_type=one-time event_id=e8de7474-09f3-4770-bbb4-2b78159ce95b, source=PerformanceRegressionPredefinedStepsTest.test_mixed_gradual_increase_load (performance_regression_gradual_grow_throughput.PerformanceRegressionPredefinedStepsTest)() message=Traceback (most recent call last):
10:08:27  File "/home/ubuntu/scylla-cluster-tests/performance_regression_gradual_grow_throughput.py", line 72, in test_mixed_gradual_increase_load
10:08:27  self._base_test_workflow(workload=workload,
10:08:27  File "/home/ubuntu/scylla-cluster-tests/performance_regression_gradual_grow_throughput.py", line 127, in _base_test_workflow
10:08:27  self.run_gradual_increase_load(workload=workload,
10:08:27  File "/home/ubuntu/scylla-cluster-tests/performance_regression_gradual_grow_throughput.py", line 209, in run_gradual_increase_load
10:08:27  results = run_step(stress_cmds=workload.cs_cmd_tmpl, current_throttle=current_throttle,
10:08:27  File "/home/ubuntu/scylla-cluster-tests/sdcm/utils/decorators.py", line 272, in wrapped
10:08:27  send_result_to_argus(
10:08:27  File "/home/ubuntu/scylla-cluster-tests/sdcm/argus_results.py", line 165, in send_result_to_argus
10:08:27  argus_client.submit_results(result_table)
10:08:27  File "/usr/local/lib/python3.10/site-packages/argus/client/base.py", line 224, in submit_results
10:08:27  self.check_response(response)
10:08:27  File "/usr/local/lib/python3.10/site-packages/argus/client/base.py", line 68, in check_response
10:08:27  raise ArgusClientError(
10:08:27  argus.client.base.ArgusClientError: ('API Error encountered using endpoint: POST /api/v1/client/testrun/scylla-cluster-tests/7cb52991-e1ae-4e3e-8351-856c6b216b82/submit_results', '7cb52991-e1ae-4e3e-8351-856c6b216b82')

https://jenkins.scylladb.com/job/scylla-enterprise/job/perf-regression/job/scylla-enterprise-perf-regression-predefined-throughput-steps-vnodes/20/

problem is that due some reason this run was not created in Argus at all:
< t:2024-10-27 04:14:54,161 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': ['Run not found', '7cb52991-e1ae-4e3e-8351-856c6b216b82'], 'exception': 'SCTServiceException'}, 'status': 'error'}
< t:2024-10-27 04:14:54,729 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': 'Created', 'status': 'ok'}

I don't know yet the root cause.

problem is that due some reason this run was not created in Argus at all:
< t:2024-10-27 04:14:54,161 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': ['Run not found', '7cb52991-e1ae-4e3e-8351-856c6b216b82'], 'exception': 'SCTServiceException'}, 'status': 'error'}
< t:2024-10-27 04:14:54,729 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': 'Created', 'status': 'ok'}

I don't know yet the root cause.

This could mean that test failed creation on a separate stage, but then got re-created by the sct itself, hence the "run not found" exception. Next response indicates that it got created, but something maybe dropped it later. Maybe id re-use or a consistency issue. Need to check all stages that interacted with argus

problem is that due some reason this run was not created in Argus at all:
< t:2024-10-27 04:14:54,161 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': ['Run not found', '7cb52991-e1ae-4e3e-8351-856c6b216b82'], 'exception': 'SCTServiceException'}, 'status': 'error'}
< t:2024-10-27 04:14:54,729 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': 'Created', 'status': 'ok'}

I don't know yet the root cause.

This could mean that test failed creation on a separate stage, but then got re-created by the sct itself, hence the "run not found" exception. Next response indicates that it got created, but something maybe dropped it later. Maybe id re-use or a consistency issue. Need to check all stages that interacted with argus

It could also mean that another test (read/write) did this, but mixed didn't.

@k0machi what's up with this one ?
seems like we don't have "Create Argus Test Run" in the pefRegressionParallelPipeline.groovy
regardless we don't expect creation to fail, and we need to be able to lookup the logs for this failure, to understand it (on argus end)

Without the explicit stage the run is created inside "Run SCT Stages" stage, specifically during ClusterTester init, so the cause for failure should be visible in the logs for that particular SCT run.

Looking into this, I see a weird issue:
So here's the write test, initializing succesfuly:
< t:2024-10-27 04:13:51,469 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': ['Run not found', '70fa1e8d-a52e-41ec-aeb0-9da8ef711ec6'], 'exception': 'SCTServiceException'}, 'status': 'error'}
< t:2024-10-27 04:13:52,035 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': 'Created', 'status': 'ok'}
< t:2024-10-27 04:13:52,387 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': 'updated', 'status': 'ok'}
< t:2024-10-27 04:13:52,743 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': 'added', 'status': 'ok'}

And here's the mixed test's argus.log:
< t:2024-10-27 04:14:54,161 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': ['Run not found', '7cb52991-e1ae-4e3e-8351-856c6b216b82'], 'exception': 'SCTServiceException'}, 'status': 'error'}
< t:2024-10-27 04:14:54,729 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': 'Created', 'status': 'ok'}
< t:2024-10-27 04:14:55,080 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': ['Run not found', '7cb52991-e1ae-4e3e-8351-856c6b216b82'], 'exception': 'SCTServiceException'}, 'status': 'error'}
< t:2024-10-27 04:14:55,430 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': [], 'exception': 'DoesNotExist'}, 'status': 'error'}
< t:2024-10-27 04:14:55,431 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': ['Run not found', '7cb52991-e1ae-4e3e-8351-856c6b216b82'], 'exception': 'SCTServiceException'}, 'status': 'error'}
< t:2024-10-27 04:15:18,778 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': ['Run not found', '7cb52991-e1ae-4e3e-8351-856c6b216b82'], 'exception': 'SCTServiceException'}, 'status': 'error'}
< t:2024-10-27 04:15:25,812 f:base.py         l:65   c:argus.client.base    p:DEBUG > API Response: {'response': {'arguments': [], 'exception': 'DoesNotExist'}, 'status': 'error'}

So this is weird - API responded that it had succesfully saved the run, yet it does not exist for subsequent calls. Manually querying the run doesn't help - it indeed doesn't exist:

I suspect something happened with scylla at that moment - unfortunately the logs for this are not available anymore (and I suspect the application logs won't show much, since the API did respond with success - which means something happened on scylla's end).

That is very disturbing.
Did you check on each node separately?
It's not the first time we run into such cases, last time it was during upgrade of Scylla or node replacement
What more information we can log to help figure out the next time
(We should collect logs, as soon as issues are reported, or archive logs periodically to s3 to something like that)

That is very disturbing.
Did you check on each node separately?
It's not the first time we run into such cases, last time it was during upgrade of Scylla or node replacement
What more information we can log to help figure out the next time
(We should collect logs, as soon as issues are reported, or archive logs periodically to s3 to something like that)

I haven't checked individual nodes yet, I'll do that. We should collect logs periodically, maybe have a github action that would do a snapshot of last N hours of production each time an issue is reported?

I suspect MV scylla issue - see both id and test_id are indexed columns - so when MV fails to update (which is not ensured by cql insert/update request) we may hit issue with insert being correct and querying for it not. @k0machi can you confirm this ID exists without using indexed column in query?

I suspect MV scylla issue - see both id and test_id are indexed columns - so when MV fails to update (which is not ensured by cql insert/update request) we may hit issue with insert being correct and querying for it not. @k0machi can you confirm this ID exists without using indexed column in query?

No, the id doesn't exist at all - only two runs that have #20 as their build number exist for this one:

@k0machi
what next ? what can we do to capture more data when it's gonna happen again ?

@k0machi
what next ? what can we do to capture more data when it's gonna happen again ?

I will add context dump to the exceptions happening in the submit requests (or we could just dump them wholly on every error to improve readability - session data, request body, etc.

We can't identify the root cause, adding new logging for identify those


Fix a deadlock between the crawler and dialer, and other hangs

Motivation
In #1905, Zebra gradually loses peer connections, until there are none left. Then it stops making sync progress.
When this issue happens, a large number of peers are stuck in the attempt_pending state, despite the handshake timeout.
This issue also impacted the peer::Connection refactor in #1817, and a bunch of upcoming zebra-network security fixes.
Solution

Stop ignoring inbound message errors
Add timeouts during handshake setup
Add a timeout to the candidate set update
Use the select! macro in the crawler to avoid starvation
Spawn a separate task for each handshake
Increase the crawl fanout to improve address diversity
Avoid starvation when using the select function
Fix priority of multiple ready futures in the select function
Add correctness documentation

The code in this pull request has:

[x] Documentation Comments
[x] Existing Unit Tests and Integration Tests

Review
@dconnolly or @oxarbitrage can review - whoever is available.
I'd like to get this merged in the next few days. It doesn't directly conflict with the other security fixes, but it will make them harder to test.
Related Issues
Closes #1905 (based on 3 mainnet and 3 testnet nodes x 3 days of testing).
Closes #1941 (based on 3 testnet nodes x 3 days of testing).
Possibly closes some other related hang or reliability issues.
Deals with part of the #1892 and #1657 refactors.

I'm going to put this PR in draft until I've tested a few full syncs over the weekend.

Coverage is failing due to type errors in a dependency, maybe the most recent nightly has a bug?
I'll open a PR on Monday if no-one else gets to it first.

I added a small change that avoids starvation when using the future::select function, and prioritises multiple ready futures correctly.

@dconnolly or @oxarbitrage this fix is now ready for review.
Let me know if you want to do a walkthrough.
(I tried to split this PR, but unfortunately these changes all depend on each other and modify similar files.)

The pull-request (zealous-zebra) failure is:

failed to load source for dependency redjubjub

which should be fixed by a rebase/merge.

Squashed fixup commits to make the review easier.
Rebased to fix build errors.


Logrotate doesn't work in a docker image

The logrotate solution put into the pureelk image relies on cron. Cron doesn't run right in an ubuntu docker image. See here for more info: http://phusion.github.io/baseimage-docker/
Long story short, upstart doesn't work and cron requires upstart.
We need another solution for the logrotate stuff as these log files still get gigantic and fill the disk after a month of running.

Glad to see the update for the kibana version and the docker level logging. Hoping to get some acknowledgement of this issue eventually....

Still looking for an acknowledgement of this issue...

Sorry for the delay. This issue is now fixed. We are now setting the maximum log size in the container. Thanks for reporting this!

So to be clear, I'm referring to these files:
/var/log/pureelk/array-tasks.log
/var/log/pureelk/scheduler.log
Sorry if that wasn't clear.
Those files get to many many GB in size in just a week or two.

Actually you are right. We only set max-size. Per Docker documentation, we should also include max-file.

The log driver stuff doesn't affect all files being written by your container. That's only for docker logging. You'll need another solution to deal with these files than the options you're referring to here.

We merged in a change that adds logrotate to the log files.


Improve korean translations

Fixes #.
Description of the changes:

Improve korean translations
fix wrong battery unit comment

How changes were validated:

manual

@microsoft-github-policy-service agree

Unfortunately, we're not able to merge translation suggestions in this repo because of limitations in our tooling. I have filed an issue internally to track this, and our localization team will review and apply these changes. Thank you for your contribution!


Support test code coverage calculation for declaritive

Is your feature request related to a problem? Please describe.
Hope test code coverage number can be provided for declarative bot test project so that declarative bot quality can be measured.
Describe the solution you'd like
For 1) declarative steps 2) expression 3) LG they both have condition branch. Is there a way to provide an api to return test coverage at least for declarative steps?
[enhancement]

@DingmaomaoBJTU , could you please add more detail to your request? We would like to understand exactly what you are hoping to get here.

@DingmaomaoBJTU thank you for opening this issue. We are not able to take this at the moment.


RTC-S1 SAS Beta Acceptance Test

ADT provided a delivery of the beta version of the RTC-S1 SAS on 12/15/2022. This ticket is to track the acceptance testing for the image. As per usual, a wiki page that covers the testing procedure and validation of expected results should be the byproduct of this ticket.
The delivered images and documentation for the release can be found in Artifactory-FN under general/gov/nasa/jpl/opera/adt/rtc_s1/r2/beta/
Any issues with the test should be communicated back to ADT via email ASAP.

Wiki instructions are now available here: https://github.com/nasa/opera-sds-pge/wiki/RTC-S1-Beta-Acceptance-Testing-Instructions


pass Notification to barStyleFactory

my calculation of bottom depends on Notification.key

Hello @achtan
Can you provide more details of your use case?

if I set Notification.key = 'timer' then I want to show this notification on left top corner, but if Notification.key is any other value than should by shown in left bottom corner

Sounds like you have two different stacks, is that correct?

hmm, yes that's true

It might be easier to use two StackNotification components. That way each can have their own lifecycles and styles.

Yes, you would want to use two notification stack components, @achtan.


Welcome to submit Dio's third-party plugins and related libraries here

After searching for a while for a caching solution, I created my own interceptor. But, maybe another solution exists and I missed it ...
It would be great to have a list of third party interceptors and transformers in the README. Having conventions for plugin would be very appreciated too (like naming, API, usage, etc ...).
Here two packages of mine :

dio_cache
dio_retry

After I saw your plugins, I was about to come here and suggest either an "awesome-dio" or a section plugin authors can collaborate and update on that wouldn't affect the current lifecycle.

This would be great, I have a small gist on built_value integration with dio.

It would be useful to have this kind of list, and I have a cache lib too :

dio-http-cache :  a simple cache library for Dio like Rxcache in Android

Ok? I think we still should?

We have some specifications for third-party libraries：

All plugins and libraries should support Dio 3.0.x
Having detailed documentation and examples

@wendux
Please consider adding this one to your ducomention.
PrettyDioLogger

@wendux
Please consider adding this one to your ducomention.
dcache_flutter
Dcache_flutter is a library both for Dio 3.x and 2.x. Dcache_flutter uses sqflite as disk cache. It is used in our product. It is fully tested and stable. Maybe others will like it too.

Hello I created a Lib to make Certificate Pinning on Flutter with a Dio interceptor to uses in Dio:
http_certificate_pinning
I think that is interesting add into Dio documentation.

I created an oauth 2.0 authentication plugin for dio I think it would be interesting to add it in the documentation.
https://pub.dev/packages/oauth_dio

Here is a new package for retry with Dio. It supports dio 4.0 and null safety: https://pub.dev/packages/dio_smart_retry
This is a next generation of an abandoned dio_retry package.
By default, the request will be retried only for appropriate retryable http statuses.
Also, it supports dynamic delay between retries.

Here's an interceptor we created to transform your dio requests in curl requests: https://pub.dev/packages/curl_logger_dio_interceptor

An interceptor that pretty prints requests/responses with ability to customize the logger. https://pub.dev/packages/dio_http_formatter

Instead of using interceptors, I wrap dio adapters with golang's http.RoundTripper pattern.
Pub: https://pub.dev/packages/roundtripper
Github: https://github.com/morlay/roundtripper
For impletementing atom processings and compose them in required.
Case Like:

Requests Loging with cost time https://github.com/morlay/roundtripper/blob/main/lib/roundtripbuilders/request_log.dart
RequstBody encoding https://github.com/morlay/roundtripper/blob/main/lib/roundtripbuilders/request_body_convert.dart

As the package is now away from maintaining, we encourage you great author to consider migrate to the hard-fork diox. And see https://github.com/cfug/diox/issues/29 for more details.

Hello I created a Lib to make api gen like retorfit in dio.
cinch
I think that is interesting add into Dio documentation.

dio Listening tool (Android Studio&IDEA)
https://github.com/mdddj/dd_flutter_idea_plugin/blob/master/dio.md

Easy to use cookie handler: https://github.com/hurelhuyag/dio_cookie_cached_handler

Hi!
I created a simple request library with type transforming based on dio which my team using in production at our own projects!
You can check it here: https://pub.dev/packages/tio

That looks pretty cool!


CB-15410 Create CCM Upgrade flow skeleton

See detailed description in the commit message.

I have finally gone through the whole PR. :smile:

Is this still required?

of course it is required


Interleaved log records using StreamHandler

Monolog version 1.24.0
PHP version 7.0.33
We get interleaved log records while using StreamHandler. The under-laying PHP stream writer implements a chunked write with a default chunk size of 8192 bytes. In an application with many concurrent processes writing to a single log file this leads to interleaved log records.
Example (if the chunk size would be 4):
my log line 1
my lmy log line 3
og line 2
my log line 4

This issue could be fix in a simular way as in #1129 by calling stream_set_chunk_size.

Minimal reproducible example
index.php
<?php
require_once(__DIR__ . '/vendor/autoload.php');

use Monolog\Handler\StreamHandler;
use Monolog\Logger;

$logger = new Logger('channel-name');
$logger->pushHandler(new StreamHandler(__DIR__ . '/app.log'));
$logger->info('This is a log!' . str_repeat('.', 8192));

Run it once
> php index.php
> wc --max-line-length app.log
8266 app.log

Run it in parallel (output shortened)
> parallel php index.php ::: {1..1000}
> wc --lines --max-line-length app.log
1000 16458 app.log
> awk '{ if (length > 8266) print $0 }' app.log
[2021-05-07T11:01:21.395883+02:00] channel-name.INFO: This is a log!.....[2021-05-07T11:01:21.395883+02:00] channel-name.INFO: This is a log!.....[] []


Update scalafmt-core to 3.2.0

Updates org.scalameta:scalafmt-core from 3.0.8 to 3.2.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, early-semver-minor, semver-spec-minor

Superseded by https://github.com/sangria-graphql/sangria-marshalling-api/pull/101


The bot stops a broadcasts and crashes (with recover) if user in the mailing list does not exist

No testing was made


Attempting to enable federation of an already-enabled type should fail fast

Attempting to enable federation of an already-enabled type with a non-default group will create the crd for the federated type and then fail in attempting to create the FederatedTypeConfig (since it already exists).   kubefed2 enable should check for the presence of the FTC for the type and fail immediately if it already exists, since a type can be enabled at most once for a control plane.

/kind bug

Is this issue still exists? I checked out the code in enable.go file and found out the cmd will update the FederatedTypeConfig if it already exists.
	concreteTypeConfig := resources.TypeConfig.(*fedv1a1.FederatedTypeConfig)
	concreteTypeConfig.Namespace = namespace
	existingTypeConfig := &fedv1a1.FederatedTypeConfig{}
	err = client.Get(context.TODO(), existingTypeConfig, namespace, concreteTypeConfig.Name)
	createdOrUpdated := "created"
	if err != nil {
		if !apierrors.IsNotFound(err) {
			return errors.Wrapf(err, "Error retrieving FederatedTypeConfig %q", concreteTypeConfig.Name)
		}
		err = client.Create(context.TODO(), concreteTypeConfig)
		if err != nil {
			return errors.Wrapf(err, "Error creating FederatedTypeConfig %q", concreteTypeConfig.Name)
		}
	} else {
		existingTypeConfig.Spec = concreteTypeConfig.Spec
		err = client.Update(context.TODO(), existingTypeConfig)
		if err != nil {
			return errors.Wrapf(err, "Error updating FederatedTypeConfig %q", concreteTypeConfig.Name)
		}
		createdOrUpdated = "updated"
	}

This issue has been superseded by #720. I would close this but considering this issue was created after #720, I'll defer to @marun.

@font This issue was not been superseded by #720.  If you inspect the code, you will notice that CRD creation is performed before checking if an FTC exists.  This allows creation of a duplicate CRD with non-default group/version.  A check for the FTC should instead be performed in advance of CRD creation, and enabling should fail if the group and version of the CRD that will be created don't match an existing FTC.

@marun Apologies I misread your original comment. The important keyword was non-default group.


Test Currency Source

Coverage increased (+1.7%) to 87.879% when pulling 48ffc3defd8cec5afbe0bcd4cf172f74cec396ff on feature/test_source into 0fc996bf5bf69b88b6185dde5be0b0d57ab0a773 on master.

Coverage increased (+1.7%) to 87.879% when pulling 7ea5d752df1ee1e2985824070d8ae349f0c3ff70 on feature/test_source into 0fc996bf5bf69b88b6185dde5be0b0d57ab0a773 on master.


appointer/swaggervel This package is abandoned and no longer maintained. No replacement package was suggested.

**appointer/swaggervel   package is no more maintained or abondoned.

Just added support for https://github.com/DarkaOnLine/L5-Swagger. Soon docs will be updated on https://infyom.com/open-source/laravelgenerator/docs/generator-options#swagger


Fall back to a default GOPATH

This avoids the need to have GOPATH set and is in line with how Go itself
behaves, see https://github.com/golang/go/issues/17262.

@sschuberth Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@sschuberth Thank you for signing the Contributor License Agreement!


Fixes #2075 and #1949

When deleting a menu, canceling it and opening another menu again. The action has two id's like this: .../2/1. This pull requests fixes that.
This will also fix #2075 and fix #1949

Can you make sure this applies to all types? This behavior likely exists in all Voyager BREAD views

@fletch3555 Sure, I will check every delete modal and update the PR.

This is essentially the same as #2670.  One uses __id, and the other uses the index route and appends the id.  Can you make them function the same (I prefer using the correct route, not assuming the delete route is the same as appending the id to the index route).

Fixed all the other instances I could find. Some have different routes and so we have to append the id to the end and not replace the __id.

Sorry, just saw your comment about needing to append.  What do you mean they "have different routes"?  If it's an issue where some are ".delete" and some are ".destroy", then I think that's a different issue that should be changed as well.

Yes, we generate all the routes for the datatypes. But we set the routes for database and setting our self. So they differ from the generated ones. I could make that a different pull request, where we make the routes the same?

Yes, please make them all the same (bread ones should be resource routes, so they're all destroy). We did this same thing a while back, so we must've just missed some.  Go ahead and test that under the 1.x-dev branch to make sure it wasn't just merged there for some reason

Yes, please make them all the same (bread ones should be resource routes, so they're all destroy). We did this same thing a while back, so we must've just missed some.  Go ahead and test that under the 1.x-dev branch to make sure it wasn't just merged there for some reason

I made them all append (so we have some unity, like the resource routes). Are there any other delete modals that I missed?

Something went wrong here
You set the form-action to database.destroy when someone trys to delete a menu.
Good thing is that table is not defined, so this just doesn't work.
Otherwise it would just delete a whole table

Wow! Good catch! Should I make a pr to fix this?

No, I did already: https://github.com/the-control-group/voyager/pull/2892


refactor file structure for improved code quality around cli testing

For example we have a tests folder and test/data folder under forc-client, we should probably make the tests folder just any other project under test/data for consistency.

For example we have a tests folder and test/data folder under forc-client, we should probably make the tests folder just any other project under test/data for consistency.

hey can you provide the link


Is there some way we can contribute/donate/support?

@rstrouse I've seen a couple issues/threads where folks have wanted to contribute/donate/support in some way. Each time, you respond with something along the lines of "just glad you found it useful." It's an admirable response. But, is there any thing we can do to show our gratitude? Donate money? https://www.buymeacoffee.com/? Patreon? GitHub Sponsor? Donate hardware? Spread the word? Arbitrary praise? Convince HomeAssistant that 100% is closed, not open 😅? Something else?
It's fine if the response is, "nah, I'm good." But I feel like ESPSomfy-RTS is one of the better put-together projects around:

It's well documented.
The reasons for decisions are explained well.
The UI provides a good UX

It's easy to do the right thing.
It's hard to do the wrong thing.
We're warned asked to confirm before dangerous stuff.

You're responsive and helpful.
Perhaps more importantly, you're not condescending (from what I've seen anyway).
It works better than commercial options.
It provides more features than both the actual remote and commercial options.
etc.

And if there's something I can do to show that (besides tell you in this GitHub issue), I'd like to do it.

But, is there any thing we can do to show our gratitude?

You just did thanks!

Convince HomeAssistant that 100% is closed, not open 😅?

Is that not the most illogical thing you ever heard?  What is funny is how the response to simply pointing out how silly it is, can be so nasty.  I think it is even weirder when the concept is just explained as a matter of fact.  Like you swallowed the wrong pill and reality is somewhere outside your grasp.  But then again I am pretty sure rabbits don't poop hard boiled eggs.  And if there is a big guy in a red suit standing next to my fireplace in the middle of the night... he's probably there to rob me.
Thanks for the feedback and Happy New Year!

Convince HomeAssistant that 100% is closed, not open 😅?

Is that not the most illogical thing you ever heard? What is funny is how the response to simply pointing out how silly it is, can be so nasty. I think it is even weirder when the concept is just explained as a matter of fact.

It definitely seems like a weird hill for them to die on. It's the opposite of how a lot of people think (it's basically a glass half-empty/half-full situation). And in a case like mine–where I've only automated horizontal blinds with a tilt-only mechanism–percentage is the last thing I would expect. The angle of the blinds is more useful than what percentage of open the "cover" is.
I can't think of any advantage there is to keep this tech debt in Home Assistant, instead of updating the model to reflect a better understanding of what a "cover" is. But then again, I'm not a core contributor, so maybe keeping the tech debt makes sense.

But, is there any thing we can do to show our gratitude?

You just did thanks!
Thanks for the feedback and Happy New Year!

Fair enough! Well thanks again. If you change your mind, and want/need something, let me know.


Omni-chain WorldID: Support on different L1s and L2s

Description
WorldID will support Ethereum mainnet, Optimism, and Polygon PoS really soon. However, the landscape of different L2s on Ethereum is greatly expanding and other L1s should also have access to the Ethereum state for integrating WorldID in order to make our protocol credibly neutral and accessible to as many developers as possible (chain-agnostic).
Requirements for bridging to work

Proving the state inside of world-id-contracts which are deployed on Ethereum mainnet on different networks
Correct implementation of the Semaphore verifier, or any other future iteration of the verifier for WorldID proofs. The implementation will be different for different execution environments (EVM - Ethereum, Scroll, zkSync, Polygon ZKEVM, Optimism, Arbitrum, ...; Cairo - Starknet, Sway - Fuel; Sealevel - Solana; Noir - Aztec; Substrate - Polkadot; CosmWasm - CosmosSDK/IBC; RiscV - RiscZero; Leo - Aleo; SnarkyJS - Mina and more to come in the future). The development of these implementations can be supported by the Worldcoin Foundation in the form of grants in the future.
Relaying the proofs from Ethereum to other networks

EOAs sending data through calldata to the target network. EOA is controlled by an external relayer service that pushes proofs to the target network WorldID contract.
Oracles/Re: e.g. Chainlink, Empiric/Astraly
Other: Succinct, Herodotus, Axiom

Solutions
Interesting candidates for proving and verifying L1 state on different networks to be able to do omni-chain WorldID integrations:

Herodotus (L1<>L2 comms, SNARKified storage proofs)
Axiom (L1<>L2 comms)
Succinct (Telepathy L1<>L1 comms)

0x20beaa5f85392b57b008c5785fc71af61115a51d

O


Stop Visual Studio Code to watch for file changes

VSCode Version: all
OS Version: Lubuntu based at Ubuntu 18.04

Steps to Reproduce:

Does this issue occur when all extensions are disabled?: Yes/No
Why is not possible to disable the function of watch for file changes?
I would like to have this option to set at VSCode config file, or at least limit it for the actual folder.

We have a setting "files.watcherExclude" to exclude folders from file watching.


Adding tanzu code 54

@pilloPl, just confirming that the date is supposed to be December 15, 2021? I only ask because 53 is on the same date, and that didn't seem to happen with other videos. Otherwise looks fine to me.

Adding for good record keeping. Response from Slack:
Jakub Pilimon  9:58 AM
hi
9:58
sorry, it was 2 days ago :slightly_smiling_face: 1/19


Allow root node to be extended

If the root_node method is called multiple times, extend the existing
root node instead of overwriting it.

Why in the world are the tests passing...
https://github.com/fotinakis/swagger-blocks/blob/ed779b7d136e9fc2ebabdd00b37dd33e193f009d/spec/lib/swagger_v2_blocks_spec.rb#L285
Also, would love to see a test for this behavior, including merging of inline keys.

Yeah, just noticed that myself when I read your comment. I'll do some more investigating. My guess is that it's because swagger_root is being called twice in the same class.

For inline keys, I noticed that they are currently being ignored for successive swagger_schema and swagger_path calls, so I didn't implement them for swagger_root either. Those need to be fixed up as well, but I may do that as a separate PR.

Hmm, I investigated a bit more, and this is going to be harder than I originally thought. Right now, the methods on RootNode check the swagger version immediately when called. That means that if the version is defined in one class, and I'm calling the response method from another class, it throws a NotSupportedError, because it doesn't know the version. The obvious solution would be to defer validation until build time, but that will require more work. I'll see what I can do.

Or, we could have a PR before this one that entirely removes support for v1.2. Then, do this PR. Then tag everything as 2.0.

+1 for 2.0 version

Let's remove all the 1.2 support code and then revisit this PR. Anybody feel like making a PR to remove 1.2 features and files?

@fotinakis When you say remove 1.2 support code, do you mean remove all code relating to 1.2? (i.e. this gem will cease to work with Swagger 1.2)

Yes @TylerRockwell — at that point we would release v2.0.0 of this gem and make sure people know it's only compatible with Swagger 2.0.


How to save a pdf blob content

How to save a pdf blob content. I am using type: application/pdf, but wrong format gets downloaded. The blob works well with older approaches like : window.URL.createObjectURL(blob) or window.navigator.msSaveOrOpenBlob(blob, fileName)

same problem here, what was your solution?

@feitzi Use below code:
req.onload = function (event, header) {
var blob = req.response;
try{
window.navigator.msSaveOrOpenBlob(blob, fileName);
}
catch(ex){
var data = new Blob([blob],{type:'application/octet-stream'});
FileSaver.saveAs(data, fileName);
}
};
This will work on all the broswers. The only limitation on Mac Safari is that the file will get downloaded as "Unknown", and will have no extension. You will have to put the extension and give it some file name. And also update to the latest version of file saver.

My problem was that i load the file from the server with a wronge responseType.
Setting the responseType in httpService to arraybuffer and everything works smoothly.

httpService.get(this.scope.fileUrl, { responseType: "arraybuffer" })

@feitzi Awsome.
In my case I was expecting response type of any format i.e. pdf, zip, doc etc. So, req.responseType = "blob"; worked for me.


How to enable locale on production for mozilla.org (fixes #45)

@peiying2
Review for this is on you, since you need it to fix bug 1338759

@peiying2
When you are on the main page of the pull request, you should see a link "Add your review" since your review was explicitly requested. Click it and select "Approve changes".
In the future, you can use the same process to request updates. When you comment, select "Start a review". Then you can use the button on the top corner to complete your review, and choose if it's just comments, approved or require changes.


Create deno.yml

name: Setup .NET Core SDK uses<EMAIL_ADDRESS>with: # Optional SDK version(s) to use. If not provided, will install global.json version when available. Examples: 2.2.104, 3.1, 3.1.x, 3.x, 6.0.2xx dotnet-version: # optional # Optional quality of the build. The possible values are: daily, signed, validated, preview, ga. dotnet-quality: # optional # Optional global.json location, if your global.json isn't located in the root of the repo. global-json-file: # optional # Optional package source for which to set up authentication. Will consult any existing NuGet.config in the root of the repo and provide a temporary NuGet.config using the NUGET_AUTH_TOKEN environment variable as a ClearTextPassword source-url: # optional # Optional OWNER for using packages from GitHub Package Registry organizations/users other than the current repository's owner. Only used if a GPR URL is also provided in source-url owner: # optional # Optional NuGet.config location, if your NuGet.config isn't located in the root of the repo. config-file: # optional # Optional input to enable caching of the NuGet global-packages folder cache: # optional # Used to specify the path to a dependency file: packages.lock.json. Supports wildcards or a list of file names for caching multiple dependencies. cache-dependency-path: # optional

Description:
Describe your changes.
Related issue:
Add link to the related issue.
Check list:

[ ] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

Dear Team, kindly review my contribution and merge. Thanks

Will have to close this PR as it looks like spam


[BUG] Cluster mongo is not supported

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

Expected Behavior
No response
Steps To Reproduce
No response
Environment
HertzBeat version(s):

Debug logs
No response
Anything else?
No response

not support mongodb cluster now, we will do it.

The reason for this bug is that the MongoDB collector starts a multi-document transaction through a session to read the MongoDB status. According to the official MongoDB documentation, the MongoDB version deployed on a single machine only supports this function when it is 3.6 or above, so the error "Sessions are not supported" will be reported. You can now solve this problem by upgrading the MongoDB version deployed on a single machine to above 3.6 or converting to cluster deployment. We are in the process of compatibility with the old version and will notify you in this issue once it is compatible.


Adjustments for Password

Removed Forgot Password option in login
Made passwords hidden when user enters then in Account Settings page

looks good!


We need to create a unit/regression test

Hi,
In every PR that is currently in the pipeline the question "does this affect the precision" is, understandably, asked.
I think we should implement a unit-test or regression-test. This will greatly speed up the merging of new features / refactoring.
@TixiaoShan and @YoshuaNava, any suggestion?
personally, i like what Cartographer does, that is allowing to launch directly the application in "rosbag mode".
This means that the rosbag C++ API is used instead of "rosbag play".
This allows results to be more consistent, no matter how fast is your PC.

Can be use for example the rosbag related to this to perform the regression test?
https://github.com/RobustFieldAutonomyLab/LeGO-LOAM/blob/master/LeGO-LOAM/launch/demo.gif ?

And I think we should also address/discuss the fact that results are NOT deterministic, as I discussed here #53 and here #105

I believe that the non-deterministic behavior comes from having a multi-threaded application with non-blocking queues. Since the odometry and mapping are separate, the mapping might compute its solution with different amounts of odometry inputs from one run to another. To ensure deterministic and proper comparisons you would need to either bring all the code into a single thread or put everything in the same process with blocking queues between them.

I can easily take care of the refactoring myself, using proper multi-threading and refactoring to improve separation of concerns.
But before doing it I would like to know if maintainers have time to review and merge PRs, since there are few already in the pipeline.

@facontidavide This is an awesome idea.
Would we aim to test full SLAM performance from the start? Should we do so with datasets, or simulated environments?
I agree that the non-deterministic behavior should be ameliorated. In his PhD thesis, Ji Zhang mentions running LOAM on KITTI at around 10% the speed, if I remember well, to obtain good results. His method was great, but people should not need to resort to this.
Depending on the answer from @TixiaoShan we could continue developing on this repo, or maybe start a new one? From the start I think it would be better to continue working here, but it also depends on the time that he can dedicate to review PRs.

Dataset for sure, not simulation. As I said, Cartographer solved the problem of real-time quite elegantly. https://google-cartographer-ros.readthedocs.io/en/latest/demos.html

Either we use a dataset with ground truth (Kitti?) to test quantitatively the accuracy (recommended) or any dataset to test that at least we don't have regressions.

In terms of moving to another repository, I do want to contribute to the original repo, since @TixiaoShan deserve all the credit for this work.
But I also want to work considerably faster and all the PR I have been sent so far has been collecting dust.
I will probably wait until middle of September, once I am back from my vacation. If things are still moving too slowly, I will create  a "friendly" fork, where you @YoshuaNava is welcome to collaborate.
By "friendly" I mean that I will always give my availability to work together with the original contributors to merge my changes upstream.

I agree. And I also think that regardless of the decision taken, we should credit the authors for their work appropriately.
I'll be glad to contribute :+1:

I agree that the non-deterministic behavior should be ameliorated. In his PhD thesis, Ji Zhang mentions running LOAM on KITTI at around 10% the speed, if I remember well, to obtain really good results. His method was great, but people should not need to resort to this. At least we should know how much this really influences results.

I believe the reason is that the odometry to mapping ratio is something like 10 to 1 when running in real time. E.g. For every one scan to map solution you have time to compute 10 scan to scan (odometry) solutions. For maximum accuracy, you would run things with a 1 to 1 ratio, i.e. a single scan to scan solution to initialize the scan to map solution.

Aaaaand... done!
It was a lot of work, but i refactored (heavily) the software to solve this issue:
https://github.com/facontidavide/LeGO-LOAM/tree/single_process
This new version use multiple threads in a single process and synchronize them in a deterministic way. As a consequence:

Given the same rosbag, it will always return the same result.
The level of parallelism is the same as the original version (number of used threads)
No changes that may affect the precision have been added.
We don't need to use rosbag play anymore. If the name of the rosbag is passed as an argument, the application load the rosbag and process it at the maximum speed allowed by the CPU.

This also address issue #112


Merge rc5 into master so that rest server supports rc4 and rc5 (aka v1.0)

[x] merge
[x] replace rc5 with v1.0
[x] release rest-server-1.0

Done via #170


helm list should print chart and chart version separate

Feature request: The command helm list at the moment prints Chart and its version in one string.
NAME                   	NAMESPACE         	REVISION	UPDATED                                	STATUS  	CHART                                   	APP VERSION
cert-manager           	foo               	6       	<TIMESTAMP>   	deployed	cert-manager-v1.5.3                     	v1.5.3

Like APP Version there also should be a Chart version. So that there are 2 columns for the Chart name cert-manager and for Chart version 1.5.3.
That also should be adjusted for the yaml or json output of course.
Doing that it will be much easier to use that in Ansible Plays or whatever.. so it is not necessary to get the version via regex..
Output of helm version:
version.BuildInfo{Version:"v3.8.0", GitCommit:"d14138609b01886f544b2025f5000351c9eb092e", GitTreeState:"clean", GoVersion:"go1.17.6"}

.. and the Chart column should also show the repo or how it is possible to get the repository info of that helm chart?

What happens if you use the --output json flag? Does that split out the fields as you'd expect?

Keep in mind that the default output (--output table) is meant to be human-readable. Machine-readable output is meant to be provided by alternate --output functions.

@elwood218 need more info.

@yxxhero which info you need??
At the moment the output shows helm chart name and version in one string.. But this should be separated.. So what info you else need???

What happens if you use the --output json flag? Does that split out the fields as you'd expect?

@elwood218

@bacongobbler you didn't understand the request.

@elwood218 welcome to create a PR for this feature. Thanks very much.

I already checked all different output!
    "chart": "generic-0.1.0",

Chart key is always the same and the value is always <chart-name>-<version>. But it should be like separated keys:
   "chart": "generic",
   "version": "0.1.0"

Wow.. @yxxhero yeah after your questions I see that you are not the one who could solve that

Why is this request closed? I have the same problem :(. I want to get the version of the chart of a installed release. It's not easily possible to extract the version from "chart" because semver allows - and the chartname allows numbers.
current state of helm list --filter='monitoring' -n monitoring -o json:
[{
"name":"monitoring",
"namespace":"monitoring",
"revision":"6",
"updated":"2022-11-09 15:57:10.30892234 +0100 CET",
"status":"deployed",
"chart":"kube-prometheus-stack-40.1.2",
"app_version":"0.59.1"
}]

my proposal (to be downwards-compatible)
[{
"name":"monitoring",
"namespace":"monitoring",
"revision":"6",
"updated":"2022-11-09 15:57:10.30892234 +0100 CET",
"status":"deployed",
"chart":"kube-prometheus-stack-40.1.2",
"chart_name":"kube-prometheus-stack",
"chart_version":"40.1.2",
"app_version":"0.59.1"
}]

Why is this closed as complete? It doesn't appear in helm ls
[
   {
      "app_version" : "",
      "chart" : "templated_docker-compose-1.0.0",
      "name" : "example",
      "namespace" : "default",
      "revision" : "1",
      "status" : "deployed",
      "updated" : "2024-08-12 23:11:02.065325133 +0000 UTC"
   }
]


Dynamic Event Calendar

We would like to implement a Dynamic Event Calendar feature on the GDG RCCIIT website. This calendar will showcase all upcoming events and allow users to filter events by type (e.g., workshop, talk, hackathon). Users should also have the ability to RSVP directly from the calendar interface.

Display a calendar view of upcoming events.

Workshops
Talks
Hackathons

Please assign me under GSSoC'24 ext and hacktoberfest labels

assigned


Issue/133/extended magerr

@yanzastro great work! I cleaned up some of the documentation and fixed a few tests so that their outputs are deleted at the end of the test.
Two questions:

You committed a lot of files in the examples/estimation directory that look like they're outputs from the example? i.e. NZDir_NZ.hdf5, NZDir_samples.hdf5, NaiveStack_NZ.hdf5, Naive_samples.hdf5, demo_knn.pkl, nzdir_model.pkl, output_KNN.hdf5, point_NZ.hdf5, point_samples.hdf5, sampletest.hdf5, and varinf_NZ.hdf5. Do these need to be saved and version controlled in the repo? Or are they just the outputs of the examples? I removed them from the repo because I think they're the latter, but if they're the former, I can restore them!
The extended errors require that the galaxy dataframe includes columns called "major" and "minor". I noted this in the docstring. But are these the semi-major and minor axes? Or the full major and minor axes? It looks like you are using them on the same footing as the PSF FWHM in the "auto" method, so I think they must be the full axes?
If these are the full axes, I think the GAaP calculation might be missing a factor of 2? There you convert the PSF FWHM to a Gaussian sigma, and multiply the "major" and "minor" values by the factor that converts a half-IQR to a Gaussian sigma.

Basically, to summarize 2 and 3 above: in the "auto" method, it looks like you are treating the "major" and "minor" values like they are diameters, but in the "gaap" method, you are treating them like they are radii.
Does that make sense? Maybe I am misunderstanding the methods.
But other than these few points, great work! We'll get this merged very soon!

Hi @jfcrenshaw, thank you for the report! Let me reply to each of them:

those output files are generated during a previous test. They are not needed but I forgot to delete them and thank you for pointing it out!
2,3. yes I did mixed up some axes and semi-axes. The cosmoDC2 catalog provides half-light radius so everything in the "auto" case is radius or semi-axes; for the "GAAP" case, the maximum aperture size A_max and A_min are given as FWHM, so I convert them into sigma first in line 388 and 389. I converted HL radius to sigma of semi-major and semi-minor radius in line 383 (a fact of 2 was missing here); I also found another error in line 392.

Great! I think we're almost there. Just a few more questions:

https://github.com/LSSTDESC/RAIL/blob/a1a3fe199f0cd7a879ec0de20c48d55609d21fa1/rail/creation/degradation/lsst_error_model.py#L383
What is the origin of the 1/1.35 factor here? It looks like the factor that converts a full IQR to a Gaussian sigma? If that's what it is, we should change the comment next to it. But it also makes me worry that we are mixing up radii and diameters again. Since the half-light radius is obviously a radius, shouldn't we still have the half-IQR to Gaussian factor, and not the full-IQR to Gaussian factor? (But I may be totally misinterpreting this factor!)

In the "auto" method, we are still using the PSF size and the majors/minors on the same footing. But the PSF size is the FWHM which is a diameter, and the majors/minors are radii. Should we divide the PSF size by 2 to put them on the same footing? Or is it correct as-is?

Again, sorry if my comments above are off base, since I don't know the specifics of these models!

@jfcrenshaw you are absolutely correct! I confused myself on "half" "semi-" "radius" etc.
For 1, the conversion should be 1/0.6745 (see https://en.wikipedia.org/wiki/Interquartile_range). It converts the half IQR (i.e. the HL radius from the catalog to my understanding) to Gaussian sigma;
For 2, yes I forgot to convert PSF to radii (sigma). Now I update it by multiplying with the factor fwhm_to_sigma = 1 / 2.355

@jfcrenshaw great! thanks so much!


Metrics when a pod has no requested resources

Description
This PR handles the situations where a pod (or one of its containers) has no requested resources specified. That means that even when there is a metrics server that can tell how many resources that pod is consuming, there is no way to transform that number in percentage (because there are no upper limits). So, while always showing how much resources a pod is consuming, the reserved will be 0. Also, to notify the user that a pod has no requested resources specified, a little "!" is placed next to the pod's name, and hovering with the mouse over it will let the user know what's the problem.

Pull Request Test Coverage Report for Build 243197491

29 of 29 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 92.183%

Totals

Change from base Build 243154695:
0.05%

Covered Lines:
1523

Relevant Lines:
1603

💛  - Coveralls

/rebase


500 You need to use a client supporting PRET (PRE Transfer) to use PASV

FTP OS: Unix
FTP Server: ProFTPD
Computer OS: Windows
I'm just trying to download a file and I get this error, idk... As an extra info,  HasFeature(FtpCapability.PRET) gives me TRUE but Idk. Check
Capabilities does return this:
MLSD, PRET
With DownloadFileAsync it does work but it doesn't with DownloadFile. I do also get that error listing a directory so I'm not sure anymore
Logs :
# GetListing("/test/subdir/", Auto)
Command:  TYPE I
Response: 200 Command okay

# OpenPassiveDataStream(AutoPassive, "MLSD /test/subdir", 0)
Command:  EPSV
Response: 502 Command not implemented.

# OpenPassiveDataStream(PASV, "MLSD /test/subdir", 0)
Command:  PASV
Response: 500 You need to use a client supporting PRET (PRE Transfer) to use PASV

Using PRET commands manually with .Execute() let me do LIST, RETR etc.

Can you paste the code that does not work, and the code that works?

Not working:
FtpClient client = new FtpClient("host", 12998, "T", "passwd");
client.DownloadFile("C:\\Users\\Hidend\\Documents\\file.txt", "/main/file.txt");;
client.Disconnect();
Console.ReadKey();

# Connect()
Status:   Connecting to ***:12998
Response: 220 ProFTPD 1.2.11 Server (GNU/Linux)
Status:   Detected FTP server: ProFTPD
Command:  USER ***
Response: 331 Password required for T
Command:  PASS ***
Response: 230 user logged in successfully.
Response: 230-
Response: 230-
Response: 230-
Command:  FEAT
Response: 211 End
Response: 211-Extensions supported:
Response: MLST type*,x.crc32*,size*,modify*,unix.owner*,unix.group*,x.slaves*,x.xfertime*
Response: PRET
Response: CPSV
Response: NOOP
Response: CLNT
Response: AUTH TLS
Response: PBSZ
Response: SITE XDUPE
Response: SSCN
Status:   Text encoding: System.Text.ASCIIEncoding
Command:  SYST
Response: 215 UNIX system type.
Status:   Auto-detected UNIX listing parser

# DownloadFile("C:\Users\Hidend\Documents\file.txt", "/main/file.txt", Overwrite, None)

# OpenRead("/main/file.txt", Binary, 0)
Command:  TYPE I
Response: 200 Command okay

# OpenPassiveDataStream(AutoPassive, "RETR /main/file.txt", 0)
Command:  EPSV
Response: 502 Command not implemented.

# OpenPassiveDataStream(PASV, "RETR /main/file.txt", 0)
Command:  PASV
Response: 500 You need to use a client supporting PRET (PRE Transfer) to use PASV

Working:
FtpClient client = new FtpClient("host", 12998, "T", "passwd");
client.Execute("PRET RETR /main/file.txt");
client.DownloadFile("C:\\Users\\Hidend\\Documents\\file.txt", "/main/file.txt"); ;
client.Disconnect();
Console.ReadKey();

# Connect()
Status:   Connecting to ***:12998
Response: 220 ProFTPD 1.2.11 Server (GNU/Linux)
Status:   Detected FTP server: ProFTPD
Command:  USER ***
Response: 331 Password required for T.
Command:  PASS ***
Response: 230 user logged in successfully.
Response: 230-
Response: 230-
Response: 230-
Command:  FEAT
Response: 211 End
Response: 211-Extensions supported:
Response: MLST type*,x.crc32*,size*,modify*,unix.owner*,unix.group*,x.slaves*,x.xfertime*
Response: PRET
Response: CPSV
Response: NOOP
Response: CLNT
Response: AUTH TLS
Response: PBSZ
Response: SITE XDUPE
Response: SSCN
Status:   Text encoding: System.Text.ASCIIEncoding
Command:  SYST
Response: 215 UNIX system type.
Status:   Auto-detected UNIX listing parser
Command:  PRET RETR /main/file.txt
Response: 200 OK, planning for upcoming download

# DownloadFile("C:\Users\Hidend\Documents\file.txt", "/main/file.txt", Overwrite, None)

# OpenRead("/main/file.txt", Binary, 0)
Command:  TYPE I
Response: 200 Command okay

# OpenPassiveDataStream(AutoPassive, "RETR /main/file.txt", 0)
Command:  EPSV
Response: 502 Command not implemented.

# OpenPassiveDataStream(PASV, "RETR /main/file.txt", 0)
Command:  PASV
Response: 227 Entering Passive Mode (54,38,178,128,56,18).
Response: 227- Using SLAVE1 for upcoming transfer
Status:   Connecting to ***:14354
Command:  RETR /main/file.txt
Response: 150 File status okay; about to open data connection from SLAVE1.
Status:   Disposing FtpSocketStream...
Response: 226 Transfer complete, 68B in 0.0 seconds (0B/s)
Response: 226- Checksum from transfer: 860215e3
Command:  QUIT
Response: 221 Goodbye!
Status:   Disposing FtpSocketStream...

I'll try but there is this post on the Filezilla Client forum where they say PRET is weird and has no proper spec. Still, I'll try.

Since it says:
Supports DrFTPD's PRET command and in FTP Support was listed and said "Automatic", I thought it was supported haha.
Ok thanks dude!

That was written by the original author years ago. He apparently didn't add any support for it :(

Please try this...
If the server supports PRET we send it before transferring data...
https://we.tl/t-A1RiuFqlzF

Downloading works, thank you!

Released in 26.0.0.
https://www.nuget.org/packages/FluentFTP/26.0.0


Unable to resolve service for type 'WindowsServiceWorker.JokeService'

[Enter feedback here]
Cannot get this sample running, it thrown "Unable to resolve service for type 'WindowsServiceWorker.JokeService' while attempting to activate 'WindowsServiceWorker.WindowsBackgroundService'." error until I implemented IHostedService for JokeService and add services.AddTransient<JokeService>() in main

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

ID: ed148088-4550-2788-0756-390eab15e3f6
Version Independent ID: 64d5cdf8-e50f-0c89-7f16-82645ba6383e
Content: Create a Windows Service using BackgroundService
Content Source: docs/core/extensions/windows-service.md
Product: dotnet-fundamentals
GitHub Login: @IEvangelist
Microsoft Alias: dapine

duplicated


Add support for multi touch operation of multiple sliders at the same time

I have embedded several sliders into the same page and they are supposed to be used on a huge multi touch screen. However, the arrow and bullet navigators seem to fire only on click events (see https://github.com/jssor/slider/blob/19.0.1/js/jssor.slider.js#L3208 and
https://github.com/jssor/slider/blob/19.0.1/js/jssor.slider.js#L3364). On all multi touch screens I am aware of, only the first touch will eventually turn into a click event after the finger has been released. I don't know much about mobile, though ...
If the following actions are performed one after another, not a single click event will be fired (at least in Chrome):

touch a navigator item of slider 1 with finger 1
touch a navigator item of slider 2 with finger 2
release the navigator item of slider 1
release the navigator item of slider 2

If you have multiple people using the multi touch screen at once, this results in very strange and unexpected behavior. If the number of users and the number of sliders is high, it get even more confusing (in my particular case, the screen has 40 simultaneous touch points).
I suggest to add support for the ontouchstart, ontouchendand ontouchend events as well, similar to https://github.com/jssor/slider/blob/19.0.1/js/jssor.js#L1947

I will investigate this behavior.
Thanks!

Thanks for your prompt reply. Do you think you can investigate it soon? Otherwise I would need to hack something for my multi touch application.
BTW: The same problem occurs with Firefox on my Android phone, i.e. it looks like you could actually use phones or tablets for testing. But you would need to disable certain gestures for zooming and scrolling.

I can't get it done shortly. I think you can make some changes.

Looks like you removed the original jssor slider source code: https://github.com/jssor/slider/tree/master/js (no jssor.js, jssor.slider.js anymore ...)

The source code is available in the release of version 19.0.1.
https://github.com/jssor/slider/releases/tag/19.0.1


fix(colors): update colors to align with system changes

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] clarity.design website / infrastructure changes
[x] Other... Please describe:

What is the current behavior?
Updates the design system variables slightly to improve accessibility and align to work conducted during token migration.
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Update our light theme library back to the original color (700 from 500). Please do so in the code as well.


style(README): alert markdown GooseAI link

Formatting alert markdown on Note, Warning, and Tip section.
From unintended relative link to GooseAI link.

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


DS-3153 : Move local.cfg.EXAMPLE to [src]/dspace/config/ directory.

Per discussion in comments of DS-3153:
https://jira.duraspace.org/browse/DS-3153
This PR simply moves the local.cfg.EXAMPLE file into the [src]/dspace/config/ directory, alongside all other configuration files.  This has the added benefit that the local.cfg.EXAMPLE ends up in the installation directory (as a starting point for creating your local.cfg).
I also made minor updates to the comments in local.cfg.EXAMPLE and tweaked the build assembly, as it no longer needs to ensure the local.cfg.EXAMPLE file is referenced individually...it'll be copied into the release packages alongside all other configs.
This really just needs code review / approval from others. There's nothing to "test" as the local.cfg.EXAMPLE file doesn't do anything by default.

+1

+1 from me, see my comment on https://jira.duraspace.org/browse/DS-3153


ci: Add Documentation Deployment

#__
Checklist

[ ] I didn't over-scope my PR
[ ] My PR title matches the commit convention
[ ] I did not include breaking changes
[ ] I included unit tests that cover my changes
[ ] I added/updated the documentation about my changes
[ ] I made my own code-review before requesting one

Description of the changes

Technical highlight/advice

Codecov Report

Merging #112 (a65fdb5) into beta (28587b6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             beta     #112   +/-   ##
=======================================
  Coverage   72.00%   72.00%
=======================================
  Files          38       38
  Lines         911      911
=======================================
  Hits          656      656
  Misses        255      255

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:tada: This PR is included in version 1.0.0-beta.47 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to point to a specific java install during install.py execution

Hi,
I have multiple installations of Java on my computer. The default one being a Java8 install. All are in the Path variable, though.
How do I specify that I want to target the JDK-13 install?
Thanks.

Hi @diyoyo, you can find the kernel.json for the java kernel (jupyter kernelspec list) and change the first string in argv to be the full path to the java command in the bin folder for your jdk-13 install.
e.g. on windows possibly something like:
"argv": [
   "C:\\Program Files\\Java\\jdk-13\\bin\\java.exe",
   "-jar",
   ...

Whichever jvm is running the kernel, is also the jvm executing the user code.

@SpencerPark What about on MacOS? I added the path that I get from  /usr/libexec/java_home -V but that does not work as it seems.
What I got from above command is,
14.0.1, x86_64:	"OpenJDK 14.0.1"	/Library/Java/JavaVirtualMachines/openjdk-14.0.1.jdk/Contents/Home
So I updated the kernel.json like below.
{
    "argv": [
        "/Library/Java/JavaVirtualMachines/openjdk-14.0.1.jdk/Contents/Home/bin/java",
        "-jar",
        "/my/path/share/jupyter/kernels/java/ijava-1.2.0.jar",
        "{connection_file}"
    ],
    "display_name": "Java",
    "env": {},
    "interrupt_mode": "message",
    "language": "java"
}

Which did not work for me. Also I tried /Library/Java/JavaVirtualMachines/openjdk-14.0.1.jdk/Contents/Home and /Library/Java/JavaVirtualMachines/openjdk-14.0.1.jdk/Contents/Home/bin.java.exec.

Hey @samitha9125, what you've put into that kernel.json looks good to me... are you getting any errors in the terminal that is running the jupyter server or just still not picking up the newer version?
Also if you wouldn't mind testing, could you try instead setting the env to explicitly define the JAVA_HOME?
{
    "argv": [
        "java",
        "-jar",
        "/my/path/share/jupyter/kernels/java/ijava-1.2.0.jar",
        "{connection_file}"
    ],
    "display_name": "Java",
    "env": {
        "JAVA_HOME": "/Library/Java/JavaVirtualMachines/openjdk-14.0.1.jdk/Contents/Home"
    },
    "interrupt_mode": "message",
    "language": "java"
}

@SpencerPark That did not work either. I have installed Java 14 and Java 8. I cannot uninstall Java 8 because I use it for other program what doesn't support Java 9 and up. So I need to explicitly tell IJAVA to use JDK 14.
When I execute jupyter console --kernel=java even after your suggestion, it shows this error,
Error: A JNI error has occurred, please check your installation and try again
Exception in thread "main" java.lang.UnsupportedClassVersionError: io/github/spencerpark/ijava/IJava has been compiled by a more recent version of the Java Runtime (class file version 53.0), this version of the Java Runtime only recognizes class file versions up to 52.0
	at java.lang.ClassLoader.defineClass1(Native Method)
	at java.lang.ClassLoader.defineClass(ClassLoader.java:763)
	at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
	at java.net.URLClassLoader.defineClass(URLClassLoader.java:467)
	at java.net.URLClassLoader.access$100(URLClassLoader.java:73)
	at java.net.URLClassLoader$1.run(URLClassLoader.java:368)
	at java.net.URLClassLoader$1.run(URLClassLoader.java:362)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.net.URLClassLoader.findClass(URLClassLoader.java:361)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:335)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
	at sun.launcher.LauncherHelper.checkAndLoadMain(LauncherHelper.java:495)
Traceback (most recent call last):
  File "/usr/local/bin/jupyter-console", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/site-packages/jupyter_core/application.py", line 270, in launch_instance
    return super(JupyterApp, cls).launch_instance(argv=argv, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/traitlets/config/application.py", line 663, in launch_instance
    app.initialize(argv)
  File "<decorator-gen-114>", line 2, in initialize
  File "/usr/local/lib/python2.7/site-packages/traitlets/config/application.py", line 87, in catch_config_error
    return method(app, *args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/jupyter_console/app.py", line 141, in initialize
    self.init_shell()
  File "/usr/local/lib/python2.7/site-packages/jupyter_console/app.py", line 114, in init_shell
    client=self.kernel_client,
  File "/usr/local/lib/python2.7/site-packages/traitlets/config/configurable.py", line 412, in instance
    inst = cls(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/jupyter_console/ptshell.py", line 272, in __init__
    self.init_kernel_info()
  File "/usr/local/lib/python2.7/site-packages/jupyter_console/ptshell.py", line 326, in init_kernel_info
    raise RuntimeError("Kernel didn't respond to kernel_info_request")
RuntimeError: Kernel didn't respond to kernel_info_request

Or if I run the Jupyter Notebook, it shows Kernel is restarting and shows errors in the console.

@samitha9125 I just tested on a mac with jdk 8, 11, and 13 installed. Using the both the first or second method I provided worked on that machine.
The default was java 8 in the shell that launched the kernel. If I don't modify the kernel.json then the kernel fails to start with the same exception you've reported. After modifying the kernel spec via either changing the argv or env the kernel successfully starts with the specified jdk as the compiler.
The only potential difference I can see is that my machine was using adoptopenjdk instead of openjdk but I hope that is not a problem.
What I think is a more likely problem is that the kernel may be installed in more than one place and the kernelspec being edited is not the one being used by the kernel that is running.
Could you show the output from jupyter kernelspec list?
Could you also look for [ZMQTerminalPythonApp] Starting kernel: ['... when running juptyer console --kernel=java --debug to make sure it has the path to the ijava-1.3.0.jar in the folder that contains the kernel.json you are editing? (to make sure editing the one that is actually running)
Thanks

@SpencerPark Hi. Thank you so much for your unbelievable support. I really appreciate it. I found the issue after debugging using your methods. When I created my virtualEnv, for some reason it has used system python rather than the python version I installed. I was foolish not to run a which python after installing it. Anyways which all the time used the system python and jupyter has been installed in system and not in the virtualEnv (Because pip command also was from the system).
So I've used system python and jupyter installed in the system but I changed a kernel.json from my virtualEnv (No idea how it was end up there). Anyways after removing jupyter from system and removing my virtualEnv, recreated a virtual environment with the correct python path and installed both jupyter and IJava. This helped me to fix my issue after following steps that you have defined.
Thank you so much for your help.


Select interaction default styling function can now handle geometry less features

This fix make sure the feature passed to the default styling function of the select interaction don't crash when trying to return the style of a feature without geometry.

When would a feature without a geometry get selected? Are you adding it manually to the collection of the select interaction? Can't you protect it there?

In our application, a map is shown next to a table containing a list of feature. In this list some have geometry and some don't. In order to keep the map and table in sync, we push selected element in table to the select interaction. We could filter the geometry less feature at this point, but since all our modification tool assume that the modified feature are all in the select interaction, it break a lot of thing if they're not pushed in the interaction.
I know it's a pretty specific use case, but since it's possible to add a geometry less feature to the select interaction I feel like it's a good thing to be able to handle it cleanly.

I agree with the reasoning here. Do we have a CLA @fblackburn? This is good to be merged once we have one.

CLA confirmed.

Thanks @ahocevar for the confirmation and to @fblackburn for the fix.

@fblackburn, this needs to be rebased, I fear. Can you do it? If not just tell us and we'll sort it out together.

I'll work on it later today. Thanks!

The branch has been rebased

Cool thanks again!


Move Emulator Wiki pages to docs.microsoft.com

Is your feature request related to a problem? Please describe.
Inconsistent docs experience for developers. There is already a doc on docs.microsoft.com that talks about debugging with the Emulator, but there is this new doc @JonathanFingold told me about that lives on the Wiki pages. This new doc on Wiki pages is great but not easily discoverable.
Describe the solution you'd like
Keeping it consistent with the rest of the docs for Bot Service/Bot Framework, move all the emulator docs over to docs.microsoft.com with either a section for Emulator under "Debug" section or something similar.
Additional context
See the discussion thread from this issue on bot-docs. Copying over the suggestions from that issue to here:
Some suggestions on updating the content
Get up and running:

update the doc on the instructions link  with a better language such as "Download the Emulator for your platform from the GitHub releases page"

Connecting to bot using a .bot file
Bullet point 2, sub bullet 1 -> Make a note about AAD/Corp credentials not supported at the moment (unless that is fixed already)
Lastly, foot notes should link people off to "Azure Bot Services" (which already exists) and also a link to deploying the bot (context being that they've tested their bot locally as explained in this doc)

@Kaiqb is this making it in 4.4?

ping @Kaiqb

This will make it for 4.4

Applicable content that was in the wiki is now in the emulator article within docs. Please close this issue.


add paging to ldap search

on larger directories, a single-query can exceed the size limit of a
single search, giving the errorr (SIZE_EXCEEDED).
This commit changes the ldap search to use paging, whereas multiple
search requests of a default size of 1000 results are made.
This was created with inspiration from the following sources:

https://gist.github.com/mattfahrner/c228ead9c516fc322d3a
https://bitbucket.org/jaraco/python-ldap/src/f208b6338a28/Demo/paged_search_ext_s.py?fileviewer=file-view-default

Verified working with three different Active Directories.

So the LDAP version is a CLI option but paging (and its options) aren't.  Thoughts?

You have a point that these might be related, and i did add the LDAP version CLI addition, mainly because of your comments.
The paging ldap functionality is part of ldap v3, and dates to 1999. Its RFC is here RFC2696. So it would seem quite old.
I don't really know the users of this project very well, so I really cannot say whether this change would be a breaking change.
Otherwise one would have to look into adding functionality, which would make the script able to detect wether paging is possible, and fall back to the old logic if it isn't. But thats fairly more complex code.