Add endpoint to get a random country on request.

Requirement

add a new /random endpoint that returns a random country object on request.
It is not yet clear what details of the country would be needed, but the basics for now.

name
iso2/3
dial code

@MartinsOnuoha I would like to work on this

I can work on this feature.

I have created a PR in response to this issue : https://github.com/MartinsOnuoha/countriesNowAPI/pull/37

@deepank120896 @ryzbaka
Please be sure to go through the contribution guideline.  here
Great! I'll be assigning this issue to @deepank120896
@ryzbaka there's another issue here, would you like to work on it?

Is this still opened? If not, I would like to take a shot at it.

@deepank120896 @ryzbaka

Is this still opened? If not, I would like to take a shot at it.

@xarrijorge this issue is closed now, @ryzbaka raised a PR for it


Outdated risk calculation diagram

Where to find the issue
https://github.com/corona-warn-app/cwa-documentation/blob/master/images/risk_calculation/risk_calculation_enf_v2_overview.pdf
Describe the issue
According to https://www.coronawarn.app/en/blog/2021-04-16-corona-warn-app-risk-calculation-further-improved/ dated April 16, 2021 the risk calculation has been changed:
"For app users, this means: Previously, an encounter between two people had to last at least 13 minutes in order to be considered for an encounter with an increased risk (red tile). Now, the app considers encounters of at least 9 minutes. Thus, encounters that represent an actual relevant contact with COVID-19 infected persons can be detected even more accurately and represented accordingly in the app as a warning about an encounter with increased risk"
https://github.com/corona-warn-app/cwa-documentation/blob/master/images/risk_calculation/risk_calculation_enf_v2_overview.pdf  dated 17.03.2021 is therefore now out of date.
Suggested change

Change the 13 minute limit to 9 minute limit in https://github.com/corona-warn-app/cwa-documentation/blob/master/images/risk_calculation/risk_calculation_enf_v2_overview.pdf.

Also monitor the implementation of the documentation update from https://github.com/corona-warn-app/cwa-documentation/issues/453.

Revise the FAQ entry https://www.coronawarn.app/en/faq/#encounter_19_calc by linking to any updated and correct documentation and diagrams, should they become available.

Internal Tracking ID: EXPOSUREAPP-6547

@MikeMcC399 Thanks. Internal Tracking ID: EXPOSUREAPP-6547

Corona-Warn-App Open Source Team

The PDF diagram images/risk_calculation/risk_calculation_enf_v2_overview.pdf  carries the date/timestamp 17.03.2021, 14:25.
The diagram images/solution_architecture/risk_calculation.svg carries a date/timestamp 23.08.2021, 10:30.
The later diagram has been partially updated to use 9 minutes instead of 13 minutes.

On the left-hand side it still shows a 13 minute threshold though:

The diagrams in svg and PDF format should both be corrected and re-published.

The contents of both the following files has been corrected and they now carry a timestamp 25.01.2022, 18:00:

PDF diagram images/risk_calculation/risk_calculation_enf_v2_overview.pdf

SVG format diagram images/solution_architecture/risk_calculation.svg


Reorder parameters within functions

Fixes #141

Duplicate of #194

Great. I wasn't implying that you immediately change branch from master. I was just suggesting something to follow in future.
Again, I'll be reviewing it within a day or two.
Thanks @alpeshjamgade 🙂

Hello @alpeshjamgade! Thanks for updating this PR.  The crux is add G=constant.G as non-default parameter wherever required as user might want to give different values of G.
Really great efforts!!
Thanks @alpeshjamgade

It would be great if you could update the Jupyter notebook Visualizing event horizon and ergosphere of Kerr black hole.rst in docs/source as the change in API would break it!

In schwarzschild_radius() function could you use parameters mass, c=constant.c, G=constant.G and then handle them internally as you did with mass variable using isinstance().

In schwarzschild_radius() function could you use parameters mass, c=constant.c, G=constant.G and then handle them internally as you did with mass variable using isinstance().

ok

Only on test case is failing. I'll look at the issue! 😃

i dont understand i did isort and black but it is still showing failed.

@alpeshjamgade Somehow your dev environment got messed up I guess and so isort is not working properly. However I created a pull request to your experimental branch. Merge that, and it may get fixed.
After this you might wanna reset your dev environment.

@alpeshjamgade Somehow your dev environment got messed up I guess and so isort is not working properly. However I created a pull request to your experimental branch. Merge that, and it may get fixed.
After this you might wanna reset your dev environment.

I've updated my PR as you force-pushed in between 🤣

Finally the PR is complete AFAIK. @alpeshjamgade Really tremendous efforts from your side and thanks for your patience.

Finally the PR is complete AFAIK. @alpeshjamgade Really tremendous efforts from your side and thanks for your patience.

thank you for your help.

@shreyasbapat I've reviewed the PR from my side thoroughly. See to it and I believe this is ready to be MERGED !

@shreyasbapat I've reviewed the PR from my side thoroughly. See to it and I believe this is ready to be MERGED !

On it !

Really tremendous efforts from your side and thanks for your patience.

Thanks @alpeshjamgade  :) . I will review it once. This might be the last time you are making any changes to it.

This will need a rebase

Thanks a lot for these efforts @alpeshjamgade :)
This contribution was very essential for the project :)


cleanup9, deleted perf_counters.h/.cpp

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] others

Which issues of this PR fixes ：

Fixes #3281
Problem Summary(Required) ：

Removed unused files
be/src/util/perf_counters.h
be/src/util/perf_counters.cpp

@staman96 Thanks for your contribution.

@imay Thanks for the opportunity to get some basic experience of the procedure.


Add SRI

Fixes #1049

If it is an internal app, that makes it an even juicier target :-)
But, yes, the hashes should be automatically generated.


Cow: Blacklist gigde.com

Cow requests the blacklist of the website gigde\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

gigde\.com has been seen in 6 true positives, 0 false positives, and 0 NAAs.

Approved by double-beep in Charcoal HQ


Document how to grant permissions for CiviForm & Program Admins

Seattle had a hard time finding this documentation, its something around adding to the ADFS admin group and adding emails to the file itd_civiform_admins
Also, if we figure it out with Seattle we should figure out how to groups to the Program Admin list.

We've had somewhat similar issue with Bloomington recently: not being able to grant account CiviForm admin permissions. I'll update docs.


Importing browser_client.dart from http package results in odd "Not found: dart:html" compiler error

Steps to Reproduce

adding http: "^0.11.3+16" as dependency
importing 'package:http/browser_client.dart';
try to compile
just removing the import makes it compile again

Logs
Launching lib/main.dart on ONEPLUS A3003 in debug mode...
Initializing gradle...                                       0,9s
Resolving dependencies...                                    1,1s
Running 'gradlew assembleDebug'...
compiler message: dart:html: Error: Not found: dart:html.
Compiler terminated unexpectedly on /Users/boni/Documents/Programmieren/Dart/uffl2/lib/main.dart

FAILURE: Build failed with an exception.

* Where:
Script '/Users/boni/Documents/Programmieren/Dart/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 446

* What went wrong:
Execution failed for task ':app:flutterDependenciesDebug'.
> Process 'command '/Users/boni/Documents/Programmieren/Dart/flutter/bin/flutter'' finished with non-zero exit value 1

Flutter Doctor
[✓] Flutter (Channel dev, v0.2.11, on Mac OS X 10.13.4 17E199, locale de-DE)
    • Flutter version 0.2.11 at /Users/boni/Documents/Programmieren/Dart/flutter
    • Framework revision f408bb06f9 (2 days ago), 2018-04-10 14:27:53 -0700
    • Engine revision 6474647f52
    • Dart version 2.0.0-dev.46.0.flutter-ef926f98f5

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/boni/Library/Android/sdk
    • Android NDK at /Users/boni/Library/Android/sdk/ndk-bundle
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /Applications/Android Studio 3.1 Preview.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.3, Build version 9E145
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.0

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio 3.1 Preview.app/Contents
    • Flutter plugin version 23.2.2
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 22.2.1
    • Dart plugin version 171.4424
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] VS Code (version 1.22.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Dart Code extension version 2.11.2

[✓] Connected devices (1 available)
    • ONEPLUS A3003 • 75aef492 • android-arm64 • Android 8.0.0 (API 26)

The dart:html package requires a browser to run, it won't work in flutter or on the Dart VM.

If that's the case then the following problems arise:

https://pub.dartlang.org/packages/html dart:html is marked as compatible with flutter
just importing a file from a different package (http) and failing due to incompatible html plugin and not knowing the cause would be super frustrating

Can you enlighten me where exactly dart:html has a browser requirement? I went through the code but didn't notice something like that.
I think you mean that using browser_client.dart would require a browser to run, which would make sense.

That link is referring to package:html, not the confusingly named dart:html.  dart:html is the library for run-time bindings to the browser window, DOM, and such.

This issue seems to be solved.


Compatible with 2020.3?

Now I switch my Clion to 2020.3 EAP, and any plan to build a new version to be Compatible with 2020.3?

@linux-china I've uploaded the new version. Sorry for the long waiting


Testability of examples

We now use sed command to change parameters in examples for test: https://github.com/pfnet/chainer-test/blob/master/test_example.sh#L20
It is bad idea. We need more testable examples.

Add options for parameters
Add test mode to examples

This issue is fixed via #922


[FEA]: Make the Project Automation workflows reusable

Is this a new feature, an improvement, or a change to existing functionality?
Improvement
How would you describe the priority of this feature request
Medium
Please provide a clear description of problem this feature solves
Right now only the nv-morpheus/morpheus repo benefits from the project automations. Rather than copy-pasting the same automations into each repo and needing to maintain them separately, we should create a reusable workflow and then call it in each repo.
Describe your ideal solution
The nv-morpheus/utilities repo gets the reusable workflows and each repo calls them

nv-morpheus Org

morpheus

calls workflows

utilities

houses workflows
calls workflows

mrc

calls workflows

morpheus-experimental

calls workflows

morpheus-viz

calls workflows

Additional context
The nice thing is there's no logic change between the workflows at all, so we can make the existing env variables into default input variables and not even need to supply them in the caller workflows.
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have searched the open feature requests and have found no duplicates for this feature request

Suggestion from @cwharris - open a new repo for reusable workflows to prevent utilities from getting too cluttered and having a focused location for these, and potential future reusable workflows.
I like this approach for his reasons, and because it does feel awk having utilities be the only repo that both houses and calls the workflows.


Adding files to virtual file system (VFS) fails in 2.3.0

Problem statement
Hi
I moved from MuJoCo 2.1.1 to 2.3.0 and am now experiencing a strange behavior with MuJoCo's virtual file system.
As it seems, calls to mj_addFileVFS do no longer add files to the VFS. Everything worked as expected before.
I'm on an Ubuntu 20.04 x86-64.
To reproduce
Here are two minimal examples to reproduce this. Both examples try adding a file to the VFS and check the number of files later.
Get the two MuJoCo versions
mkdir -p $HOME/debugging && cd "$_"
wget https://github.com/deepmind/mujoco/releases/download/2.1.1/mujoco-2.1.1-linux-x86_64.tar.gz
wget https://github.com/deepmind/mujoco/releases/download/2.3.0/mujoco-2.3.0-linux-x86_64.tar.gz
tar -xf mujoco-2.1.1-linux-x86_64.tar.gz
tar -xf mujoco-2.3.0-linux-x86_64.tar.gz
mkdir files
touch files/file.stl
touch mujoco_old.cpp
touch mujoco_new.cpp

Now, put the following into mujoco_old.cpp:
#include "mujoco.h"
#include <string>
#include <memory>
#include <iostream>
#include <cstdlib>

// MuJoCo 2.1.1
int main()
{
  auto mj_vfs = std::make_unique<mjVFS>();
  mj_defaultVFS(mj_vfs.get());
  std::string dir = std::getenv("HOME") + std::string("/debugging/files/");
  std::string file = "file.stl";

  mj_addFileVFS(mj_vfs.get(), dir.c_str(), file.c_str());

  std::cout << mj_vfs->nfile << std::endl;  // Should print 1
  return 0;
}

Build and run it with
g++ mujoco_old.cpp -g -I/$HOME/debugging/mujoco-2.1.1/include -L/$HOME/debugging/mujoco-2.1.1/lib -lmujoco_nogl -o mujoco_old
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$HOME/debugging/mujoco-2.1.1/lib/
./mujoco_old

Output
scherzin@ids-opel:~/debugging$ ./mujoco_old
1

The same with MuJoCo 2.3.0 fails:
Put the following into mujoco_new.cpp:
#include "mujoco/mujoco.h"
#include <string>
#include <memory>
#include <iostream>
#include <cstdlib>

// MuJoCo 2.3.0
int main()
{
  auto mj_vfs = std::make_unique<mjVFS>();
  mj_defaultVFS(mj_vfs.get());
  std::string dir = std::getenv("HOME") + std::string("/debugging/files/");
  std::string file = "file.stl";

  mj_addFileVFS(mj_vfs.get(), dir.c_str(), file.c_str());

  std::cout << mj_vfs->nfile << std::endl;  // Should print 1
  return 0;
}

Build and run it with:
g++ mujoco_new.cpp -g -I/$HOME/mujoco-2.3.0/include -L/$HOME/mujoco-2.3.0/lib -lmujoco -o mujoco_new
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$HOME/mujoco-2.3.0/lib/
./mujoco_new

Output
scherzin@ids-opel:~/debugging$ ./mujoco_new
0

Wow, that was fast! Thanks a lot.


Loosen @doc typings.

This change is

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


feat: settings dialog and moved theme settings to dialog from sidebar

feat:

settings dialog
save/load theme setting (moved from sidebar).

@mckaywrigley
If this PR gets merged, I would like to add a default temperature to this settings screen.

love this!


Fix missing parentheses in shorthand hash syntax as argument calls

Previously, Style/HashSyntax would autocorrect the code:
raise Errors.new 'No!', foo: foo

to 👇:
raise Errors.new 'No!', foo:

While it may lead to valid code in this case, parentheses around Error.new are required in edge cases like condition that we can inline to a modifier form after we apply the shorthand hash syntax.
In this example:
if condition?
  raise Error.new 'A long error message...', foo: foo
end

we would autocorrect to invalid Ruby code:
raise Error.new 'A long error message...', foo: if condition?

Putting parentheses in method calls that return value as an argument in another call is the safest autocorrection in this case. I'm starting to wonder whether we always put the parentheses in them method calls as it's generally safer to autocorrect.

Putting parentheses in method calls that return value as an argument in another call is the safest autocorrection in this case. I'm starting to wonder whether we always put the parentheses in them method calls as it's generally safer to autocorrect.

I recall in the past one idea was to check if the version without paren is valid or different (at the AST level) from the one with parens to decide whether those are needed or not. That'd be pretty slow, though. No easy solutions...

My idea is to always put the parens when autocorrecting shorthand values and letting the user decide afterwards whether to keep them or not. The idea being that there are other cops that can autocorrect the same code afterward. I have run into other autocorrections that left me with broken code – inlining conditions as modifier forms, being one.
Anyway, let's see how many other edge cases we will find. I am trying to enforce a big codebase into EnforcedShorthandSyntax: always style and will get back with the issues I find.

My idea is to always put the parens when autocorrecting shorthand values and letting the user decide afterwards whether to keep them or not. The idea being that there are other cops that can autocorrect the same code afterward. I have run into other autocorrections that left me with broken code – inlining conditions as modifier forms, being one.

Yeah, that might reasonable as well, and probably it's the simplest to implement.


Compiling on x64 for arm64

Is there a list available of possible values for the --arch option?
I am trying to compile for arm64 and cannot seem to find the right key for that architecture...
I am compiling on x64, but need to compile for a Raspberry Pi.
I tried this, but it doesn't work:
node-gyp clean configure build --arch=arm
The only values that do something for --arch are 'ia32', everything else gets compiled to x86-64.

https://nodejs.org/api/os.html#osarch


Android: Arrow images are different for previous and next image

The next arrow and previous arrow on Android have two different images and I can't find a way to replace the images being used.
This is not an issue on iOS.

I don't see how this is possible. That next arrow is not included in the plugin. I bet you have an arrow in your app named Next.png


Added: What is the subset of C# supported by the NEO compiler and NEO VM

Added to Developer Documentation: What is the subset of C# supported by the NEO compiler and NEO VM

I would not put this first in the list, but perhaps last. This list is general Developer Documentation, and yours is quite specific.

Edit 2: Should we perhaps start to add subsections to the Developer Documentation?
I think that's a good idea ...thought it for the PR but I'm glad you brought it up.  I'm accumulating a fair bit of C# content that could easily command it's own subsection.
Going back to your comment about "yours is quite specific", "What is the subset of C# supported by the NEO compiler and NEO VM" is in fact the #1 topic for C#.NEO developers.  It should be more accessible.

👍
About it being specific, I meant that a lot of developers are looking at this, also those not not interested in C# specifics, but perhaps just general docs, or python, or JS for instance.

Please re-order your addition to the back of the list. Btw, usually this repo is listing the things in alphabetical manner.

Closing this PR.  I've added the link here: https://github.com/mwherman2000/neo-csharpcoe#neo-csharpcoe-top-10-list-of-resources-for-cneo-developers


Hashtag Post Default Tab

In the period of hashtag, the default tab is share tab. If the hashtag date is past the default tab is archive tab. So far is good.
However, for past hashtags, when the user switches to the share tab, the default tab should remain as share until the user switches back to the archive tab.

It is done with permanent Hashtag tab


Tiling problem with release 1.13.0

I just downloaded the new update and opened the default map.  I flew down a bit from the mountain and this is what I see.

I guess it's the Specular thing. Try changing specular values. Anyway, I am going to do another fix.
So, we don't need to touch that again.
Let me know, if this bug still exists.

Actually, this issue exists for a while. The landscape is not smooth too. If you sculpt it a bit with a smooth brush, this will go away.
The new light angle & specular values make things look bad.

Is this something that can be fixed without having to smooth the landscape?

We can we have tweak some normals or some clever specular management. I'm on it. Will tackle on that.

I just tested the TextureVariation to fix tiling in my UDS integration and it works well as far as I can tell.


Update swaggers for logs and metrics queries

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Hi @srnagar , can you split your PR into 2? one for ARM, one for  dataplane. as we have different PR reviewer for them

@akning-ms I have created a separate PR for ARM changes - https://github.com/Azure/azure-rest-api-specs/pull/14609

Pipelines stuck in queued status. Closing and re-opening the PR to trigger the pipelines.


Alert js closing event name, doesn't fit the rest naming convensions

I just noticed that Alert.js is the only of our components that emits an event named closed.bs.alert .
All the others use the hidden.bs.{name}
However the alert finally removes its self.
Are we ok with it or maybe should emit disposed or hidden?
The reality that it is disposing its self, and at least it could be streamlined with the rest.
I am just saying, that may it  would be the right moment to rename the event before stable
PS1: after this, all disposals could emit an event,
\CC @twbs/js-review

True but it's a breaking change, and currently there is no issue/request to emit the event for disposal of plugins? :slightly_smiling_face:
/CC @XhmikosR

Yup, agreed it's a BC for sure. You should make an issue or add this in the v6.0.0 project so that we make the change then :)


Merge Sort in JAVA

Merge sort algo in Java

Merge sort already exists in #65 , please try to add a different algorithm or improve the existing algo


Update docs for Animation.track_set_path and AnimationMixer.root_motion_track

Updated the usage description of [method Animation.track_set_path] and [member Animation.root_motion_track] to identify the parent is the root_node identified in animation player, and that the format is the same for each.

Thanks! And congrats for your first merged Godot contribution :tada:


[QUESTION] Failed to read entry point info from invalid 'package.json'

Warning: Failed to read entry point info from invalid 'package.json' file: /opt/atlassian/pipelines/agent/build/node_modules/ngx-translate-lint/dist/package.json
Hello @svoboda-rabstvo , it's me again, I have this warning during a build in AOT production mode, have you ever had this feedback ?

Hi @NolanRoose,
This warning only on version 1.20.0 ?

Hello @svoboda-rabstvo , yes

thanks you

@NolanRoose Hi, I fixed a problem with the version flag, which included parsing package.json file. Could you pls check if this case reproduces on the last version of the release (1.20.2) ?

@svoboda-rabstvo Hello ! Its good ! The warning disappeared on ^1.20.3 ! Thanks you !

@NolanRoose you are welcome, and have a nice day


New test for wait_trial_completion_when_stopping param of Tuner

Issue #, if available: https://github.com/awslabs/syne-tune/issues/392
Description of changes: Added a new test for the tuner to check whether the workers continue runnig with the flag on and are stopped with the flag off. Closes #392
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Please open an issue for this, putting in as much info as possible.

Of course. Will do


Change documentation link protocol from http to https.

Some place uses https while most other place use http.
Make it unified.

@piotrek1543 Thank you for pointing it out！


Demo not working

The demo is not opening. If this project is managed properly, this will be a hit in no time. I opened this project just because it says "Alternative to octobercms"

@ugintl thank you, I'll fix it soon.
any further contribution is highly appreciated.

@ugintl Fixed, please try again


[BEAM-53] Pub/sub client with gRPC implementation

Support pub/sub via gRPC.
Will later use for pure Java pub/sub source/sink.

R: @kennknowles

Looks like checkstyle dislikes your copyright headers.

Could do with a squash but this addresses your comments and supports some quirks needed later by the dataflow runner.

Looking at the misc support code in PubsubIO thinks would be much clearer if we also had a PubsubApiaryClient and all the regex and other misc handling in PubsubIO was moved into PubsubClient.
BUT I'd like to make steady progress, so how about we take that as a TODO?

LGTM. Merging.

I think this PR might not have received the attention it deserves:

New dependencies; what's the impact on shading and leakages? Impact on dependency tree? What are transitive dependencies?
New public APIs in the io package? This sounds something that better fits util.
No tests.
Importing repackaged Preconditions?
Etc.

@mshields822, can I ask you to work with @dhalperi to get these things fixed up?

Package-private would be better than util/, and it does seem possible for now at least.

No worries:

Flink can't handle gRPC without special magic (openSSL dep problems), so that's the last straw for me with gRCP. I'm going to add a PubsubApiaryClient and see what I can do to bring the gRPC deps back under control.
Only reason any of this is public is so the Nexmark driver can piggy back on it for all its pubsub create/list/delete stuff. Davor suggested util/ is a happy place for this sort of code, will follow that.
Repackaged slipped through an auto-complete. Sorry about that.
Hard to unit test this and I didn't want to go to town on integration or service tests, especially since it is much easier to test at the pipeline level than the raw client level.
Stand by.

Also, the extra deps (beyond grpc-pubsub-v1) are all needed by the gRPC client. They are also included by the grpc-pubsub-v1 dep, but our transitive dependency check requires them to be included directly. You can find the same deps inside the bigtable-client-core pom. So the only new dep here is on grpc-pubsub-v1 itself.

This work continues in
https://github.com/apache/incubator-beam/pull/213


Add modifier = Modifier parameter to all composable function in Search

Issue

close #323

Overview (Required)

add modifier parameters to all composable function
pass modifier to the root composable function
chore: add trailing comma for function signatures

Screenshot
Omitted because no effects to appearance of any UI.

@mikanIchinose Thank you for your contribution! 🎉  Hold on a second please for the review🙏

Thank you for your cooperation!


Missing module '.config/db'

🐛 Bug Report
When I was trying to set up, I ran into an error due to a missing module .config/db in server.js

🤔 Expected Behavior
Imports all files without error.
😯 Current Behavior

💁 Possible Solution
I looked through the entire repository if the module was misplaced.
It turns out that the config folder is missing in your repo. @ashwinkumar1211

Purposefully removed the config module from project structure and that's causing the missing module error. Check the commit status if you want.(2)  And more importantly the config module had "authentication key" which can not be publically portrayed. I'll be glad to upload the config structure though. Closing this thread with a hope you got it.

Yeah I checked the previous commit. It looks like the db.js file contains some important code that sets up the mongodb connection. Your project needs to establish a connection with the database to work. I'd suggest that you keep those files and just replace the authentication keys with some template strings.
Also change your secret keys. You may have removed them in the latest commit, but anyone can go through your previous commit and steal them.
@ashwinkumar1211

@ashish-puttaa Thanks for pointing it. I found that .gitignore was messing around from the beginning. So removed the branch which obviously deleted the history (anyway changed the secret :P) and added a proper config folder with db.js ( you can check it out now ).


Cleanup DockerFile to eliminate warnings

Motivation:
When building our docker image a lot of warnings showed up.
Modification:

Ensure we not try to start services on installation
Redirect STDERR
Remove some commented lines

Result:
Less warnings during building the docker image.

Related to #114

@swift-nio-bot test this please


AMBARI-23263. hdfs-site.xml is not getting updated after config change and restart via Ambari.

What changes were proposed in this pull request?
Now cluster desired configs are retrieved directly from DB during agent configs update creation.
How was this patch tested?
Manual testing.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1310/
Test FAILed.
Test FAILured.


Liquid Exception related to datetime validation

Trying to get my own instance of notebook running and I can't get past the build stage:
➜  notebook git:(master) jekyll serve
Configuration file: ~/dev/todrobbins/notebook/_config.yml
            Source: source
       Destination: public
 Incremental build: disabled. Enable with --incremental
      Generating...
  Liquid Exception: Invalid Date: '"source/about.md"' is not a valid datetime. in /_layouts/page.html
             ERROR: YOUR SITE COULD NOT BE BUILT:
                    ------------------------------------
                    Invalid Date: '"source/about.md"' is not a valid datetime.

Looks like it might be related to how git_mod.rb is parsing the value produced by {{ page.path | prepend: 'source/' | git_mod | date_to_string }} within page.html.

Appears to be a problem with the latest version of Jekyll. I'll look into it.

Converted notebook to Hugo; closing this issue.


mat-select items move while scrolling on iphone and Ipad

Bug, feature request, or proposal:
Mat-select option list moves while scrolleing on iphone and ipad
What is the expected behavior?
it should be closed when scrolling

Closing as this is working as intended, the select list is expected to stay open during scrolling.


page size and column selector doesn't work in jquery bootgrid

Hi
thanks for your nice job.
i used the basic example code in an html page.
pagination and search box are working nice but column selector and page size selector don't work. they dont come down to select.
i tried moving css and js files below and up of each other it didn't work. i am using bootstrap-rtl. i also used standard bootstrap.cc but that also didn't work.
what is the issue?
here is my html:

Demo

<link href="Content/bootstrap-rtl.css" rel="stylesheet">
<link href="Content/jquery.bootgrid.css" rel="stylesheet">

ID
Sender
IP

            10238
<EMAIL_ADDRESS>            14.10.2013

            10239
<EMAIL_ADDRESS>            14.10.2013

            10240
<EMAIL_ADDRESS>            14.10.2013

            10241
<EMAIL_ADDRESS>            14.10.2013

            10242
<EMAIL_ADDRESS>            14.10.2013

            10243
<EMAIL_ADDRESS>            14.10.2013

            10244
<EMAIL_ADDRESS>            14.10.2013

            10245
<EMAIL_ADDRESS>            14.10.2013

            10246
<EMAIL_ADDRESS>            14.10.2013

            10247
<EMAIL_ADDRESS>            14.10.2013

            10248
<EMAIL_ADDRESS>            14.10.2013

beginning of table has dropped in previous comment but in my code is write.
I forgot to mention that i have in my Downloads.js:
$(function () {
$("#grid").bootgrid();
});

Not working for me either. Just copying the Basic example verbatim.The demos do, however, work. Trying to figure out what the difference is.

Figured it out - you need to include bootstrap.js. Not sure why I didn't see this initially.


Running on arria10sx dev kit

Hi @doonny thanks for a great work. I have a question, can I run this demo on Arria10sx development kit?  In case of DE5a-Net, it has a pcie but arria10sx doesn't have pcie interface. Do you think I can make it run on Arria10sx_devkit?
Thank you,
-Sunil

It works on a10-soc as long as you got the BSP for the board.


Update 14_01B_01_CIEL14B_1E.txt

Delete 'for' from sentence

'lack for nothing' is an expression, I believe the current is correct
https://www.merriam-webster.com/dictionary/lack%2Fwant for nothing

See #1376


Feat: Get In Touch section to be improved.

Describe the feature
There is no background behind the contact form. Interface looking too simple. Visually needs improvement.
Add ScreenShots

This is how it is looking.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Its-Aman-Yadav please assign this task to me.

@Its-Aman-Yadav sir I updated the contact form section. Please check.


Sorting in nullable relation.

Bug description
I am using a function that sorts null values by first/last.
{ orderBy: { item: { sort: 'asc', nulls: 'last' }}}

This feature only works on nullable fields. This is okay. I like it.
However, a problem arises when the field is non-nullable but the relationship itself is nullable.
In the example below, I want to sort user using relation values.
There are both nullable and non-nullable values in a relation.
However, because the relation itself is nullable, all relation values are nullable from the user's perspective.
So in this case, I need to be able to specify first/last even when sorting using the notNullData values.
Currently Invalid value provided. I get the error Expected SortOrder, provided Object.
How to reproduce
console.log(await prisma.user.findMany({ orderBy: { relation: { notNullData: { sort: 'asc', nulls: 'first' } }} }));

Argument `notNullData`: Invalid value provided. Expected SortOrder, provided Object.

Expected behavior
No response
Prisma information
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

model User {
  id       Int       @id @default(autoincrement())
  email    String    @unique
  name     String?
  relation relation?
}

model relation {
  id           Int     @id
  user         User    @relation(fields: [id], references: [id])
  nullableData String?
  notNullData  String
}

const prisma = new PrismaClient();

const { id } = await prisma.user.findFirst() || await prisma.user.create({data: { email: 'test@test', name: 'TEST' }});

if(!await prisma.relation.findFirst()) {
  await prisma.relation.create({data: { id, nullableData: 'nullableData', notNullData: 'notNullData' }});
}

console.log(await prisma.user.findMany({ orderBy: { relation: { notNullData: 'asc' }} }));
console.log(await prisma.user.findMany({ orderBy: { relation: { nullableData: { sort: 'asc', nulls: 'first' } }} }));

Environment & setup

OS: Windows
Database: MySQL
Node.js version: 18.16.0

Prisma Version
5.4.2

Reproducible on 5.9.1.

I need this too, or is there a workaround for now?

Did you find any solution?

Any update here?

Any update?

As far as I know, there is no update about this. I am working around it by giving up some nice features of prisma. Even with a very weird hack.
I added some comments to schema.prisma file for the fields I want. And every time I do db push, I change the contents of the file to not null with a regex, and then do db push, and then change them back to nullable and do db generate. This way, the actual DB is not null, but prisma behaves as nullable.
However, prisma originally warns me when I make a mistake involving null, but does not work on that field.
For example:
model test {
  id         Int     @id
  targetData String? @db.Text //NOT-NULL-IN-REAL-DB
}

(await fs.promises.readFile('prisma/schema.prisma')).toString().replace(new RegExp('(\n *[^ \n]+ +[^ \n]+)\\?(.+) //NOT-NULL-IN-REAL-DB\n', 'g'), '$1$2\n')


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/34256

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/58860


zathuraPkgs: turn into scope, and other small cleanups

Description of changes
This makes it significantly easier to customize zathura.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

CC @globin, since the bot didn't pick it up.


Posslble documentation issue. Im using Basic Auth with drupal-client version but seeing a {"message":"Invalid preview secret."} on node page.

Prerequisites: Ive followed all of the instructions and my Next JS site works fine and I can see new nodes on page refresh on my next site.
I've followed setup instructions in Basic Auth with drupal-client version but seeing a {"message":"Invalid preview secret."} on node page. I think its because the documentation is incomplete for configuring the code to process the DRUPAL_PREVIEW_SECRET.
"Set a previewSecret on the DrupalClient. This is the same secret used when creating your site on Drupal."
The documented code to do this is the unmodified default code in the next js starter template and makes no reference to DrupalClient nor DRUPAL_PREVIEW_SECRET
pages/api/preview.ts
import { NextApiRequest, NextApiResponse } from "next"

import { drupal } from "lib/drupal"

export default async function handler(
  request: NextApiRequest,
  response: NextApiResponse
) {
  return await drupal.preview(request, response)
}
}

pages/api/exit-preview.ts
import { NextApiResponse } from "next"

export default function exit(_, response: NextApiResponse) {
  response.clearPreviewData()
  response.writeHead(307, { Location: "/" })
  response.end()
}

I am guessing that this code is the source of my error.
Thanks

You're right. The documentation should mention to update the DrupalClient.
import { Experiment_DrupalClient as DrupalClient } from "next-drupal"

export const drupal = new DrupalClient(
  process.env.NEXT_PUBLIC_DRUPAL_BASE_URL,
  {
    previewSecret: process.env.DRUPAL_PREVIEW_SECRET,
  }
)

I'll update it.
Thank you.

I have modified pages/api/preview.ts
from
import { NextApiRequest, NextApiResponse } from "next"

import { drupal } from "lib/drupal"

export default async function handler(
  request: NextApiRequest,
  response: NextApiResponse
) {
  return await drupal.preview(request, response)
}
}

to
import { NextApiRequest, NextApiResponse } from "next"
import { drupal as libDrupal } from "lib/drupal"
import { Experiment_DrupalClient as DrupalClient } from "next-drupal"

export const drupal = new DrupalClient(
  process.env.NEXT_PUBLIC_DRUPAL_BASE_URL,
  {
    previewSecret: process.env.DRUPAL_PREVIEW_SECRET,
  }
)

export default async function handler(
  request: NextApiRequest,
  response: NextApiResponse
) {
  return await libDrupal.preview(request, response)
}

please can you confirm that this is correct?
also note that I aliased the drupal import to overcome naming conflict

I have updated lib/drupal.ts as below so as not to remove previous configuration. Is this correct?
export const drupal = new DrupalClient(
  process.env.NEXT_PUBLIC_DRUPAL_BASE_URL,
  {
    previewSecret: process.env.DRUPAL_PREVIEW_SECRET,
  },
  {
    auth: () =>
      "Basic " +
      Buffer.from(
        `${process.env.BASIC_AUTH_USERNAME}:${process.env.BASIC_AUTH_PASSWORD}`
      ).toString("base64"),
  }
)

I assume this needs to go into /lib/drupal.ts which looks like this (for basic auth)
import { Experiment_DrupalClient as DrupalClient } from "next-drupal"

export const drupal = new DrupalClient(
  process.env.NEXT_PUBLIC_DRUPAL_BASE_URL,
  {
    previewSecret: process.env.DRUPAL_PREVIEW_SECRET,
    auth: () =>
      "Basic " +
      Buffer.from(
        `${process.env.BASIC_AUTH_USERNAME}:${process.env.BASIC_AUTH_PASSWORD}`
      ).toString("base64"),
  }
)

This results in an error on the node preview iframe:
Server Error
Error: The path attribute is missing for entity with slug /blog/out-box-tooling-best-developer-experience

This error happened while generating the page. Any console logs will be displayed in the terminal window.
Source
(api)/pages/api/preview.tsx (9:9) @ async handler

   7 |   response: NextApiResponse
   8 | ) {
>  9 |   return await drupal.preview(request, response)
     |         ^
  10 | }
  11 |

where preview.ts is unchanged from the default installed code:
import { NextApiRequest, NextApiResponse } from "next"

import { drupal } from "lib/drupal"

export default async function handler(
  request: NextApiRequest,
  response: NextApiResponse
) {
  return await drupal.preview(request, response)
}


DG

Transfer DG to GitHub (NFR, user stories etc.)
Update UML according to last week's feedback. (We should use consistent colours for different components )
Write the section on testing.

Can you help edit respective sections (@chekju model, @eugenelim97 storage @liushiru ui). You can follow the color scheme  used in architecture section.

Do I request PR as usual or can I commit changes directly for this?

Do I request PR as usual or can I commit changes directly for this?

Both can I think


Extending Group::canView() Doesn't Follow the 'Tri-State' Response Pattern

Affected Version
4.5.3
Description
Rather than allow for a 'tri-state' style response of true/false/null on extensions of canView, canEdit() etc. Group only allows extensions of these methods to count towards forcing the failing of these checks, never forcing the success like Member and other classes allow for.
Use Case
I want to provide a list of Groups the current member belongs to, via a GraphQL query, to my front-end. This is to facilitate for some basic display logic (e.g. are they in the clients group, or an admin etc).
Currently this is not possible as members cannot 'view' any groups, even if they are an active member of the group, unless they have the CMS_ACCESS_SecurityAdmin permission, which end-users should not have.
Steps to Reproduce

Extend Group via a DataExtension and add a canView() method that returns a 'tri-state' of true/false/null (or just return true; for simplicity)
Test the output of Group::canView() on a member who doesn't have the CMS_ACCESS_SecurityAdmin permission (e.g. an end user)
It will return false (even though our extended canView() gets called and returns a blanket true response)

Expected Behaviour

Testing Group::canView() should return true as per the blanket value of true which we are returning from our extension.

This is how Member for example (and most other classes I think) deals with extending canView() and is generally a much more useful and flexible implementation
PR

https://github.com/silverstripe/silverstripe-framework/pull/10751
https://github.com/silverstripe/developer-docs/pull/262

This also affects the viewer and editor group dropdowns on pages too.
For instance if you are in a group that doesn't have permission for CMS_ACCESS_SecurityAdmin then no groups will be returned.

any updates on this?

All PRs were merged. Close issue.


selector of the directive

Bug Report

TSLint version: 5.2.0
TypeScript version:
Running TSLint via: Phpstorm

TypeScript code being linted
// @Directive({ selector: '[shiftHeight]' })

with tslint.json configuration:

Actual behavior
TS2515:The selector of the directive "JobHeightDirective" should have one of the prefixes "nga, ngd, ngt" (https://goo.gl/uacjKR) (directive-selector
Expected behavior
https://angular.io/guide/styleguide#!#02-08 Based on the style guide it seems to be right how I wrote it cause its in the component shift. Cant find anything about nga, ngd, ngt in the style guide

That's not a tslint core rule. This issue should be opened over there https://github.com/mgechev/codelyzer


clblast: init

Description of changes
https://github.com/CNugteren/CLBlast
CLBlast is a lightweight, performant and tunable OpenCL BLAS library written in C++11. It is designed to leverage the full performance potential of a wide variety of OpenCL devices from different vendors, including desktop and laptop GPUs, embedded GPUs, and other accelerators. CLBlast implements BLAS routines: basic linear algebra subprograms operating on vectors and matrices.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2288


No data recorded also with 0.23.7

Hi there,
I pulled the latest update, but still I do not get any update from my VW ID3. The data has not been recorded for some days. There is a huge time difference regarding update from the server and update from the car.

The logs do not show significant errors (login and all this seems to work), except for this one:
vwsfriend_1   | Traceback (most recent call last):
vwsfriend_1   |   File "/opt/venv/bin/vwsfriend", line 8, in <module>
vwsfriend_1   |     sys.exit(main())
vwsfriend_1   |              ^^^^^^
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/vwsfriend/vwsfriend_base.py", line 516, in main
vwsfriend_1   |     weConnect.update(updateCapabilities=True, updatePictures=True, force=True)
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/weconnect/weconnect.py", line 193, in update
vwsfriend_1   |     self.updateVehicles(updateCapabilities=updateCapabilities, updatePictures=updatePictures, force=force, selective=selective)
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/weconnect/weconnect.py", line 218, in updateVehicles
vwsfriend_1   |     self.__vehicles[vin].update(fromDict=vehicleDict, updateCapabilities=updateCapabilities, updatePictures=updatePictures,
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/weconnect/elements/vehicle.py", line 220, in update
vwsfriend_1   |     self.updateStatus(updateCapabilities=updateCapabilities, force=force, selective=selective)
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/weconnect/elements/vehicle.py", line 330, in updateStatus
vwsfriend_1   |     self.domains[domain.value][key].update(fromDict=data[domain.value][key])
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/weconnect/elements/climatization_timer.py", line 51, in update
vwsfriend_1   |     super().update(fromDict=fromDict, ignoreAttributes=(ignoreAttributes + ['timers', 'timeInCar']))
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/weconnect/elements/generic_status.py", line 106, in update
vwsfriend_1   |     newRequest = GenericStatus.Request(localAddress=key, parent=self.requests, fromDict=request)
vwsfriend_1   |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/weconnect/elements/generic_status.py", line 150, in __init__
vwsfriend_1   |     self.update(fromDict)
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/weconnect/elements/generic_status.py", line 155, in update
vwsfriend_1   |     self.operation.fromDict(fromDict, 'operation')
vwsfriend_1   |   File "/opt/venv/lib/python3.11/site-packages/weconnect/addressable.py", line 317, in fromDict
vwsfriend_1   |     self.setValueWithCarTime(self.valueType.UNKNOWN, lastUpdateFromCar=None, fromServer=True)
vwsfriend_1   |                              ^^^^^^^^^^^^^^^^^^^^^^
vwsfriend_1   |   File "/usr/lib/python3.11/enum.py", line 787, in __getattr__
vwsfriend_1   |     raise AttributeError(name) from None
vwsfriend_1   | AttributeError: UNKNOWN

Installation is running on raspberry pi3.
Any hint or fix is appreciated. Thank you.

Please try 0.23.8. this should fix the error

Thanks. Worked. Awesome! 😸


Set up staging deployment for 1upwebapp

CircleCI should initiate a deployment on a successful 1upwebapp build (master branch).
Set up AWS Serverless deployment so that it’s possible to log in to 1upwebapp from a public URL.

Started this work on https://github.com/1uphealth/1upwebapp/compare/10-aws-deployment


⚠ GitLab Server has degraded performance

In 3e21e89, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 9361 ms

Resolved: GitLab Server performance has improved in 35cb6f9.


Add embedded option

Hello guys,
Following https://github.com/jhipster/generator-jhipster/issues/7944 and https://github.com/jhipster/generator-jhipster/pull/9341 for supporting rich document models on document databases (MongoDB and Couchbase), I added a new embedded option that enables subdocuments as explained here : https://github.com/jhipster/generator-jhipster/issues/7944#issuecomment-419755147
Please make sure the below checklist is followed for Pull Requests.

[ ] Checks are green
[ ] Tests are added where necessary
[ ] Documentation is added/updated where necessary
[ ] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

By the way, is the embedded option in the entity JSON file mandatory? What happens when the option isn't there? I guess the default value is false right?

Yes, it is not mandatory, the default value will be false as you said


CR only: remove unnecessary locks in List/ListPartition

remove unneeded locks in listPartition(). also changed trace to get better ideas on the Get() and transform() delays
sonyali@sonyadev4:~/logs/perf-test/gce-15000/arktos/arktos-062221-15k/arktos/kubemark_tp_1_master/log$ grep "168993596" kube-apiserver.log
I0622 20:35:04.364395       1 trace.go:81] Trace[168993596]: "listPartition etcd3: key=/registry/pods/" (started: 2021-06-22 20:34:47.210768417 +0000 UTC m=+13485.778327502) (total time: 17.153596435s):
Trace[168993596]: [8.704617084s] [8.704617084s] Acquiring ListAppendMux listPartition
Trace[168993596]: [17.153593818s] [8.448976144s] Took items from the response

close the PR. local test didn't show issues around the lock and 15k run didn't hit the panic which means in arktos only one partition is used.


Crashing after selecting Bluetooth Device, file, and settings

Describe the bug
After device and file are chosen and settings are selected, the app crashes after hitting start. The error is "Objective-C exception thrown.  Name: NSInvalidArgumentException Reason: -[iOSDFULibrary.DFUFirmware initWithUrlToZipFile:]: unrecognized selector sent to instance 0x282f4f020
To Reproduce
Steps to reproduce the behavior:

Use Version 1.20.15
Implement bindings into application
Click start update
See error

Expected behavior
Zip file is sent to device and firmware is updated
Additional context
I have the sample application in my GitHub. It is for business operations, so I cannot make public, but am willing to share to resolve this issue

https://github.com/ISSPRO-Eng/DFU_Sample_App Here is a public repo of the DFU part. I can make this public and hopefully get this part working

fyi use bt.Identifer not bt.UUID (it is obsolete)


Bundling slower and more jagged in recent commits

One of these three commits introduced some weirdness in the edge bundling:
https://github.com/bokeh/datashader/pull/480
https://github.com/bokeh/datashader/pull/473
https://github.com/bokeh/datashader/pull/472
Visually comparing the bundling, you can see that it was smooth before:

And is quite jagged now:

Additionally it is quite a bit slower:
PRE: CPU times: user 54.3 s, sys: 4.1 s, total: 58.4 s
POST: CPU times: user 2min 24s, sys: 8.41 s, total: 2min 33s
Here I'm comparing 400e9f00d9a08cca8916df7294dc19d964ecb3e2 (PRE) to https://github.com/bokeh/datashader/pull/480 (POST)

@philippjfr, thanks for narrowing this down. After rebasing the commits on a separate branch, I narrowed the issue down to PR #458 and #480. I'm still investigating.

I found the source of the jaggedness. It was the same issue found in the segfault issue (#477).


Had to uninstall; without normal password entry simultaneous to face recognition it's not worth it.

Please describe the issue in as much detail as possible, including any errors and traces
Please solve https://github.com/boltgolt/howdy/issues/9 so reliability can improve independent from productivity.

I've searched for similar issues already, and my issue has not been reported yet.
Linux distribution (if applicable):
Howdy version:

Please do not make threads like this, it doesn't help anybody.


[FEATURE] arm64 build instructions (or even binary release)

Describe the feature
I'm trying to build greenlight for my Raspberry Pi 4B (which is arm64). I'm having trouble getting the package to build/install. Any insight would be much appreciated!
Describe the use case
No response
Additional information
I've tried to yarn build the v2.1.1 release of greenlight-beta and it seems to get pretty far in the process. However, when I get to packaging I get this:
  • electron-builder  version=24.9.1 os=6.1.0-rpi8-rpi-v8
  • loaded configuration  file=/home/pi/greenlight/electron-builder.yml
  • writing effective config  file=dist/builder-effective-config.yaml
  • packaging       platform=linux arch=arm64 electron=27.1.3 appOutDir=dist/linux-arm64-unpacked
  • building        target=AppImage arch=arm64 file=dist/Greenlight-2.1.1-arm64.AppImage
  ⨯ exec: "opj_decompress": executable file not found in $PATH
github.com/develar/app-builder/pkg/icons.ConvertIcnsToPngUsingOpenJpeg
        /Volumes/data/Documents/app-builder/pkg/icons/icnsToPngUsingOpenJpeg.go:158
github.com/develar/app-builder/pkg/icons.ConvertIcnsToPng
        /Volumes/data/Documents/app-builder/pkg/icons/icns-to-png.go:47
github.com/develar/app-builder/pkg/icons.doConvertIcon
        /Volumes/data/Documents/app-builder/pkg/icons/icon-converter.go:226
github.com/develar/app-builder/pkg/icons.ConvertIcon
        /Volumes/data/Documents/app-builder/pkg/icons/icon-converter.go:56
github.com/develar/app-builder/pkg/icons.ConfigureCommand.func1
        /Volumes/data/Documents/app-builder/pkg/icons/icon-converter.go:33
github.com/alecthomas/kingpin.(*actionMixin).applyActions
        /Volumes/data/go/pkg/mod/github.com/alecthomas/kingpin@v2.2.6+incompatible/actions.go:28
github.com/alecthomas/kingpin.(*Application).applyActions
        /Volumes/data/go/pkg/mod/github.com/alecthomas/kingpin@v2.2.6+incompatible/app.go:557
github.com/alecthomas/kingpin.(*Application).execute
        /Volumes/data/go/pkg/mod/github.com/alecthomas/kingpin@v2.2.6+incompatible/app.go:390
github.com/alecthomas/kingpin.(*Application).Parse
        /Volumes/data/go/pkg/mod/github.com/alecthomas/kingpin@v2.2.6+incompatible/app.go:222
main.main
        /Volumes/data/Documents/app-builder/main.go:90
runtime.main
        /usr/local/Cellar/go/1.17/libexec/src/runtime/proc.go:255
runtime.goexit
        /usr/local/Cellar/go/1.17/libexec/src/runtime/asm_arm64.s:1133
  ⨯ /home/pi/greenlight/node_modules/app-builder-bin/linux/arm64/app-builder process failed ERR_ELECTRON_BUILDER_CANNOT_EXECUTE
Exit code:
1  failedTask=build stackTrace=Error: /home/pi/greenlight/node_modules/app-builder-bin/linux/arm64/app-builder process failed ERR_ELECTRON_BUILDER_CANNOT_EXECUTE
Exit code:
1
    at ChildProcess.<anonymous> (/home/pi/greenlight/node_modules/builder-util/src/util.ts:251:14)
    at Object.onceWrapper (node:events:632:26)
    at ChildProcess.emit (node:events:517:28)
    at maybeClose (node:internal/child_process:1098:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:303:5)

Cannot build electron packages:
Error: Command failed with exit code 1: electron-builder

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Any ideas? I'm very comfortable with a handful of programming and scripting languages, but JavaScrypt and TypeScript are not on that list! As soon as we start yarning I have no idea what we're actually doing...

It looks like you are missing the dependency opj_decompress on your system which makes the build fail. Can you try to install that? (More info here: https://manpages.ubuntu.com/manpages/focal/man1/opj_decompress.1.html)
Yarn is the package manager for node. It's a replacement of npm. The yarn command installs all dependencies and puts the files in the node_modules/ folder.
yarn build command does execute the election builder and compiles the app for you. You can also run yarn dev to run the application in debug mode. (Exact command are defines in packages.json.)
The binary should also be available on flathub, although I don't have a raspberry pi myself to test the install with, so I'm not sure if it runs 100% correctly or if it is even findable via the discovery app (but happy to help out if it doesn't :))

You are correct about the dependency of obj_decompress missing! I installed libopenjp2-tools and that seemed to fix it.
I appreciate your patience and explanations!
I completely missed the aarch64 package on flathub! That should get me fully squared away! However, I've already started down this path of building, so I continued trying to build from source as a somewhat academic exercise (perhaps useful to anyone else that finds this issue?). I had to solve one more snag:

[nextron] Packaging - please wait a moment
  • electron-builder  version=24.9.1 os=6.1.0-rpi8-rpi-v8
  • loaded configuration  file=/home/pi/greenlight/electron-builder.yml
  • writing effective config  file=dist/builder-effective-config.yaml
  • packaging       platform=linux arch=arm64 electron=27.1.3 appOutDir=dist/linux-arm64-unpacked
  • building        target=AppImage arch=arm64 file=dist/Greenlight-2.1.1-arm64.AppImage
  • building        target=deb arch=arm64 file=dist/greenlight_2.1.1_arm64.deb
  • adding autoupdate files for: deb. (Beta feature)  resourceDir=/home/pi/greenlight/dist/linux-arm64-unpacked/resources
  • downloading     url=https://github.com/electron-userland/electron-builder-binaries/releases/download/fpm-1.9.3-2.3.1-linux-x86/fpm-1.9.3-2.3.1-linux-x86.7z size=4.6 MB parts=1
  • downloaded      url=https://github.com/electron-userland/electron-builder-binaries/releases/download/fpm-1.9.3-2.3.1-linux-x86/fpm-1.9.3-2.3.1-linux-x86.7z duration=793ms
  ⨯ cannot execute  cause=exit status 1
                    errorOut=/home/pi/.cache/electron-builder/fpm/fpm-1.9.3-2.3.1-linux-x86/lib/ruby/bin/ruby: line 6: /home/pi/.cache/electron-builder/fpm/fpm-1.9.3-2.3.1-linux-x86/lib/ruby/bin.real/ruby: cannot execute binary file: Exec format error
    /home/pi/.cache/electron-builder/fpm/fpm-1.9.3-2.3.1-linux-x86/lib/ruby/bin/ruby: line 6: /home/pi/.cache/electron-builder/fpm/fpm-1.9.3-2.3.1-linux-x86/lib/ruby/bin.real/ruby: Success

                    command=/home/pi/.cache/electron-builder/fpm/fpm-1.9.3-2.3.1-linux-x86/fpm -s dir --force -t deb -d libgtk-3-0 -d libnotify4 -d libnss3 -d libxss1 -d libxtst6 -d xdg-utils -d libatspi2.0-0 -d libuuid1 -d libsecret-1-0 --deb-recommends libappindicator3-1 --deb-compression xz --architecture arm64 --after-install /tmp/t-yBgLDd/0-after-install --after-remove /tmp/t-yBgLDd/1-after-remove --description '
     Greenlight is a streaming application for Xbox and xCloud' --version 2.1.1 --package /home/pi/greenlight/dist/greenlight_2.1.1_arm64.deb --name greenlight --maintainer 'UnknownSKL<EMAIL_ADDRESS>--url 'https://github.com/unknownskl/greenlight#readme' --vendor 'UnknownSKL<EMAIL_ADDRESS>--deb-priority optional /home/pi/greenlight/dist/linux-arm64-unpacked/=/opt/Greenlight /home/pi/greenlight/dist/.icon-set/icon_16x16.png=/usr/share/icons/hicolor/16x16/apps/greenlight.png /home/pi/greenlight/dist/.icon-set/icon_32x32.png=/usr/share/icons/hicolor/32x32/apps/greenlight.png /home/pi/greenlight/dist/.icon-set/icon_48x48.png=/usr/share/icons/hicolor/48x48/apps/greenlight.png /home/pi/greenlight/dist/.icon-set/icon_64x64.png=/usr/share/icons/hicolor/64x64/apps/greenlight.png /home/pi/greenlight/dist/.icon-set/icon_128x128.png=/usr/share/icons/hicolor/128x128/apps/greenlight.png /home/pi/greenlight/dist/.icon-set/icon_256x256.png=/usr/share/icons/hicolor/256x256/apps/greenlight.png /home/pi/greenlight/dist/.icon-set/icon_512.png=/usr/share/icons/hicolor/512x512/apps/greenlight.png /tmp/t-yBgLDd/2-Greenlight.desktop=/usr/share/applications/greenlight.desktop
                    workingDir=

electron-builder wanted to download/use a particular fpm binary, which was x86. I had to install fpm separately by installing ruby, doing sudo gem install fpm, then set USE_SYSTEM_FPM to true with export USE_SYSTEM_FPM="true" before I did yarn build. I followed the workflow described in this github issue from electron-builder.
And then everything went fine! I got my .deb package and installed that with apt. Thanks again for your help!


[external-lb] Is there a way to deploy external load-balancer providers for development purposes?

Hi.
I'm looking at writing an external-lb provider for our cloud's load-balancer facility; I'm fine with writing the provider (the code for existing providers seems pretty simple), but I'm not sure of the best way to go about deploying it. Does it have to be built into the official image, or is there a way to tell Rancher to use a custom image so I can test my changes without impacting anyone else?

I've done some digging and it looks to me like you have to deploy the container yourself, so it looks like we can just deploy an image with a custom build it'll automatically watch the cluster and call our provider as required.
Is this correct?

Yes, it will retrieve the data from metadata. Please see an example here for f5: https://github.com/rancher/community-catalog/tree/master/templates/f5/0
Let me know if you have any other questions.


Functional tests against one website

Currently the Functional tests are against many different websites. Ideally, we should be able to write tests against a single website.
http://the-internet.herokuapp.com/ website captures prominent and ugly functionality found on the web and serves as a good testing platform.
Create tests against each of the available examples in http://the-internet.herokuapp.com/

The functional tests now runs on a local version of the internet express app.
https://github.com/getgauge/taiko/blob/919f776de73185ab4c497916d4244448d9e3d715/test/functional-tests/specs/pageActions.spec#L4


include audio player for audio recordings (where it is possible / allowed)

Sample texts: Audio file in the sourceDesc
Unmonitored: Assets path in the corpus header + per-utterance file in the utterance


Automated assets download

Proposed changes
Support download of assets depending configuration
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

Codecov Report

Merging #269 (1ca9b74) into master (0dc7271) will increase coverage by 1.11%.
The diff coverage is 68.18%.

@@            Coverage Diff             @@
##           master     #269      +/-   ##
==========================================
+ Coverage   32.62%   33.73%   +1.11%
==========================================
  Files          33       33
  Lines        2011     2072      +61
==========================================
+ Hits          656      699      +43
- Misses       1242     1252      +10
- Partials      113      121       +8

Impacted Files
Coverage Δ

internal/config/config.go
24.00% <ø> (ø)

internal/cypress/config.go
24.13% <ø> (ø)

internal/saucecloud/playwright.go
6.81% <0.00%> (ø)

internal/saucecloud/testcafe.go
7.31% <0.00%> (ø)

internal/testcafe/config.go
19.51% <ø> (ø)

internal/saucecloud/cloud.go
59.80% <65.51%> (+0.94%)
:arrow_up:

internal/resto/client.go
53.89% <72.72%> (+5.53%)
:arrow_up:

internal/saucecloud/cypress.go
78.72% <100.00%> (ø)

internal/saucecloud/espresso.go
86.36% <100.00%> (ø)

Continue to review full report at Codecov.

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


NVS / Save Data Transfer Tools

I would love to see or create tools to transfer save data between swadges, whether the tools exist on the swadges themselves, on a PC, or both.

esptool has a read_flash option so it is possible to read everything off one swadge and dump it to another.


Investigate flaky hello-pepr-finalize test

Environment
Device and OS: CI
App version: main
Kubernetes distro being used: k3d
Steps to reproduce

Examine failed workflow tests for hello-pepr-finalize

Expected result
Test passes
Actual Result
Test occasionally fails
Visual Proof (screenshots, videos, text, etc)
3 hello-pepr-finalize tests have failed on main within the last 200 runs of E2E tests as of 21 Oct 2024.
Severity/Priority
Low
Additional Context
Generate a list of recently-failed E2E tests in CI with this command:
 gh run list --workflow "E2E - Pepr Excellent Examples" --limit 200 --json workflowName,headBranch,conclusion,databaseId --jq '.[] | select(.conclusion == "failure" and .headBranch == "main") | .databaseId' | xargs -I {} gh run view {} --json jobs --jq '.jobs[] | select(.conclusion == "failure") | .name' > ci-failures.log && sort ci-failures.log | uniq -c | sort -r

Related to https://github.com/defenseunicorns/pepr-excellent-examples/pull/182


Fix stderr logging for journald and syslog

logger.PutMessage, added in #28762 (v17.04.0-ce), clears msg.Source. So journald
and syslog were treating stderr messages as if they were stdout.
Signed-off-by: David Glasser<EMAIL_ADDRESS>
- What I did
Backported https://github.com/moby/moby/pull/33832 to projectatomic/docker
- Description for the changelog

fixes Red Hat bz1720363.

LGTM

LGTM

Opened against the wrong branch entirely, shouldn't be master, should be docker-1.13.1-rhel.  That one is present here: https://github.com/projectatomic/docker/pull/346


Made module Python 3-compatible

also added requirements file

Sorry I didn't see this when you submitted! Was away. Just merging in changes now. Also pushing in some of my own. Thanks!

Merged in along with some of my own to Python3.

Thanks again for this pull request, very embarrassed I didn't see it, particularly since I just threw myself into doing the same thing! I've made a new pip release, but I'm a bit nervous that I've done it prematurely! It's on my todo list to get some tests in for this package ... let me know if you have any problems with the new release.


Custom tooltip event in Accoridon panel

When I am outside accordion custom event in tooltip fires like it should, but when I am inside accordion problem emerge.
Plunker:
http://plnkr.co/edit/dtYO32GM2fc9WKy46fka?p=preview
Outside tooltip hides when mouse leave area, but it doesn't in accordion.
Is this bug, or am I doing something wrong?

@Jebedo, please do not use the issues forum for support related requests.  Rather, please follow the instructions here.
There were a lot of issues with your plunker.  Here's the working one:

You're including two different versions of the Bootstrap CSS
You're including Font Awesome
You're including two different versions of the Angular UI Bootstrap library and neither are the latest (0.14.1 is the latest).
You're including jQuery after you include core Angular

In the future, for us to be able to help you, you need to create a MINIMAL plunker that shows the problem.  Adding extra stuff just makes our job that much harder and you're left wondering why no one is helping you.

You are right, my Plunker was terrible, I am new with Plunker :)
But the "bug" is still there even with your, repaired, code and new ui-bootstrap 0.14 version:
http://plnkr.co/edit/dtYO32GM2fc9WKy46fka?p=preview

This is a result of the prototypal inheritance gotcha as described here and in the FAQ section under 'My input / select element doesn't work as expected inside the tab / accordion directive'.  Updated working plunk here

Nice! It worked. Thank you very much @icfantv  and @RobJacobs.
Now to go back to study on scope and inheritance ;)


MSA developments and entity type filter

This PR adds a new feature which allows filtering query results by the type of "other" entities (for instance, return only Statements in which the subject is a kinase for a ToTarget query). In conjunction with this, the PR makes the following changes:

Add a new ent_type argument to StatementQuery driving the filter described above
Split get_key into get_agent_grounding and get_query_key to expose get_agent_grounding
Change get_other_agents to use grounding instead of name and refactor to expose single Statement level method to get other agents
Refactor get_summary to expose get_summary_stmts that can be called from outside if we want to get Statements instead of NL directly

Looks good!


feat: Implement 78 Sine_Wave in Java

Ported 78 Sine_Wave to Java.

Excellent we needed more Java ports!!

Excellent we needed more Java ports!!


fix(browser-detection): fix isFirefox check

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

[X] The commit message follows Conventional Commits
[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?

Closes #156
What is the new behavior?
isFirefox check now works as expected
Does this PR introduce a breaking change?

[ ] Yes
[X] No

Other information

@WynieCronje thank you for the PR! Could you please go over other functions in that folder and add lowercasing too? Also, scope for commit message is the package. So in this case it would be fix(cdk): fix isFirefox check ;)

@WynieCronje thank you for the PR! Could you please go over other functions in that folder and add lowercasing too? Also, scope for commit message is the package. So in this case it would be fix(cdk): fix isFirefox check ;)

Do you prefer another commit?
Or should I squash the commits into one?
Or is this something that happens when PR get accepted?

Do you prefer another commit?
Or should I squash the commits into one?
Or is this something that happens when PR get accepted?

Don't worry, we squash when we merge PRs

Don't worry, we squash when we merge PRs


Scheduler options not honored in Torque

Describe the bug
If a scheduler_option that conflicts with a default request in the standard template is made via the TorqueProvider, it's currently ignored. It turns out that Torque honors the first line rather than the last in case of a conflicting option.
For eg:
#PBS -l nodes=1:ppn=1
#PBS -l nodes=1:ppn=16:xk   # <--- from scheduler_options, will be ignored.

This seems NOT to be the case on all clusters. For me, prepending instead of appending breaks stuff as later (non-configurable) options override my custom options.


Allow direct lib path imports

Copied from changelog:

Importing keystore-idb/lib/* directly should now work as intended. This allows bundlers to use the "real" import paths (eg. import "keystore-idb/lib/utils.js") in addition to the "proxy" import paths (eg. import "keystore-idb/utils.js"). One reason to do this could be that you want your library to support both new and old bundlers, ie. bundlers with or without exports support in their package.json file.

If only we could stick with one bundler 😅


Incorrect parse errors issued when linebreak is in mapped type as clause

Bug Report
Today, putting a linebreak before the extends keyword is a syntax error, but not a line break after the extends keyword.
Especially in the context of mapped types which use conditionals (like Omit, if you expand it), it can be very surprising if you aren't aware of this.  I can see an argument for leaving it alone in the case of X5 (below), hopefully the case of Omit3 demonstrates where this can catch users by surprise.
Perhaps if nothing else, there is some way to improve the error message in this situation (Omit3 below), which is:

Mapped object type implicitly has an 'any' template type. (7039)
']' expected.(1005) (if you happen hover in just the right place over the extends)
';' expected.(1005) (if you happen hover in just the right place over the truthy part of the condition)

Which doesn't help as much as the error message in X5 below:

Declaration or statement expected. (1128) (if you hover over the extends)
'number' only refers to a type, but is being used as a value here.(2693) (if you hover over the thing right of the extends)

🔎 Search Terms
line break, linebreak, extends, newline, new line, extends keyword
🕗 Version & Regression Information
This is present since at least TypeScript 4.1 when key remapping was introduced.

This is the behavior in every version I tried, and I reviewed the FAQ for entries about key remapping

⏯ Playground Link

Playground link with relevant code
💻 Code

// ✅ BASELINE: normal syntax
type X1<T> = T extends number ? true : false;

type X2 = X1<1>;
//   ^? X2 = true

// ✅ works fine to have a new line after `extends`
type X3<T> = T extends
number ? true : false;

type X4 = X3<1>;
//   ^? X4 = true

// ❌ Placing a new line _before_ the `extends` is a syntax error
type X5<T> = T
extends number ? true : false;
// ^ Error: 'number' only refers to a type, but is being used as a value here. (2693)
//         ^ Error: 'number' only refers to a type, but is being used as a value here. (2693)

type X6 = X5<1>;
//   ^? X6 = 1

// ✅ This works fine in some other contexts that are less ambiguous:
type Omit2<T, K
extends keyof any> = Pick<T, Exclude<keyof T, K>>;

type Y1 = Omit<{ a: 1, b: 2 }, 'a'>;
//   ^? Y1 = { b: 2 }

// ❌ But if you use it in a remapping, it fails
type Omit3<T, K extends keyof any> = {
//                                   ^ Error: Mapped object type implicitly has an 'any' template type. (7039)
  [P in keyof T as P
    extends K ? never : P]: T[P];
//  ^ Error: ']' expected.(1005)
//              ^ Error: ';' expected.(1005)
}

type Y2 = Omit3<{ a: 1, b: 2 }, 'a'>;
//   ^? type Y3 = { a: any; b: any; }

// ✅ But if you use it in a remapping, it fails
type Omit4<T, K extends keyof any> = {
  [P in keyof T as P extends
  K ? never : P]: T[P];
}

type Y3 = Omit4<{ a: 1, b: 2 }, 'a'>;
//   ^? type Y3 = { b: 2; }

🙁 Actual behavior

X4 is not a syntax error: you can linebreak after an extends keyword in simple conditional types
X5 is a syntax error: you cannot linebreak before an extends keyword in simple conditional types
Omit3 is a syntax error: you cannot linebreak before an extends keyword in key remapping
Omit4 is not a syntax error: you can linebreak after an extends keyword in key remapping

🙂 Expected behavior
None should be syntax errors.
Omit3 is the one that I care about the most because it's the one that comes up the most when writing complex types.
here are some more real-world examples where this kind of thing is likely to come up:

here are some more real-world examples where this kind of thing is likely to come up
in these situations, it's pretty reasonable to want to put the extends on the next line.
type GreaterThanDigit<
  First extends Numeric,
  Second extends Numeric
> =
  `0123456789` extends // here
    `${string}${Second}${string}${First}${string}`
  ? true
  : false

type UnionToFnInsertion<T> =
  (T extends T ? (arg: () => T) => unknown : never) extends // here
    (arg: infer R) => unknown
  ? R
  : never

type Equal<X, Y> =
  (<T>() => T extends X ? 1 : 2) extends // here
  (<T>() => T extends Y ? 1 : 2)
  ? true
  : false

@fatcerberus thanks!  could you please help me understand what's different about
type Omit2<T, K
extends keyof any> = Pick<T, Exclude<keyof T, K>>;

such that it works?

right, so wouldn't that same thing be true for the square bracket starting the map key here:?
type Omit3<T, K extends keyof any> = {
//                                   ^ Error: Mapped object type implicitly has an 'any' template type. (7039)
  [P in keyof T as P

Some of this is working as intended (X5 is a working-as-intended ASI for example) but this one is a bug
type M<T> = {
    [K in keyof T as K
    extends "" ? K : K]: any;
}

I assume these should be valid too? (A few random places where parseType got called)
type X1<A> = `${A
extends A ? A : A}`;

type X2<A> = X1<A
extends A ? A : A>;

type X3<A, K extends A
extends A ? A : A> = any;

type X4<A, K = A
extends A ? A : A> = any;

type X5<A> = {
  foo: A
}

What about these?
type X5<A> = {
  foo: A
  extends A ? A : A;
};

declare function foo<A, K>(a: K
extends A ? A : A): any;


combine "normalizeProtocol" and "normalizeHttps"

if "normalizeProtocol" is a string, use it as the protocol to normalize,
if it is a true, use "http"
if it is falsy, don't normalize the protocol at all

never mind, my mistake...


Style changes to blockheight header

Some opinions on header styling,
changed so only the number pulses
With this commit:

before commit

I've also added a commit that removes "Topics" from the header. It doesn't have much use IMO and clutters the top header navigation

Nice!


feat(connector): [Authorizedotnet] use connector_response_reference_id as reference to merchant

Type of Change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Implement connector_response_reference_id as reference for Authorizedotnet
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

Resolves #2323
How did you test it?

Manually
Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

@jurijsb Thanks for the PR

Hey @jurijsb  👋,
Thanks a bunch for all your contributions! We've got a little something for you to show our appreciation.
Just take a moment to fill out this form, and get ready for some awesome swag coming your way.
Thanks!


Fix exception in case no user did fit the criteria

Coverage remained the same at 90.46% when pulling b345b645cb67384f5bec12e3406143f6ad8ccfad on fix_find_users into dbdcc1f2eb6e033e377898a8bf198282f82813c1 on master.


Broccoli build returns 0 on build failure

When plugins fail throwing errors, broccoli build still returns exit code 0, despite the fact that it has not built successfully, and has not created the build target directory.

Hmmm, this is quite surprising. Does someone have time to make a small reproduction?

@rwjblue Are you saying that sometimes build errors in plugin code are caught and cause non-zero exits? At the most basic plugin API level, are plugins meant to catch their own errors?
I don't know how much help I can be. It's been a whole year and I don't even remember what I was trying to do when I encountered this. Hazards of leaving bugs sit for ages.

fix https://github.com/broccolijs/broccoli/pull/456

fixed with v3.5.1 🎉 release thanks to @chiragpat


Update gRPC API ports in README

Motivation

gRPC API ports are outdated in the README.md file

Changes

Public port 10092 has been updated to 9092
Private port 10093  has been updated to 9093

bors merge


Backup and restore apps may fail when moving between formats

Describe the bug
See title. Flags backed up from a zarr ms cannot be restored to a conventional ms - they end up as appended rows.
To Reproduce
Backup flags from a zarr ms and attempt to restore them to a conventional ms.
Expected behavior
The above should work as expected (provided ordering and selection are consistent).
Version
main

I was just bitten by this again. It seems that it's not just the column that get's restored that is affected, all the columns seem to have additional rows in the final scan. I guess ROWID gets appended to. Any ideas on how to recover from this? cc @sjperkins

Trying to access data in the affected scan with
In [99]: xds[-1].DATA.values.shape

results in a long error culminating in
RuntimeError: IPosition::getFirst(n); n is too high

WIthout ROWID present, dask-ms will append https://dask-ms.readthedocs.io/en/latest/tutorial/writes.html#updating-appending-rows.
Its not immediately obvious to me what the correct thing to do here is. ROWID is needed to map the zarr format back to the original ms, but it is discarded during conversion to zarr. Possibly we could retain the ROWID column when writing to zarr.

Yeah just keeping ROWID makes the most sense to me. I have nuked the old MS, splitting again. We should probably prioritize this as it can corrupt data

Ah, yeah. I had forgotten about this one. This may be another point to add to the desired features list on dask-ms. If we retained ROWID on the zarr datasets, we would have substantially more power in xarray-land too, as it would be possible to resolve selections i.e. read from zarr, manipulate with xarray and write back to ms. It would be expensive in the worst-case scenario, but probably worth it overall.

Any thoughts on how difficult this would be to change @sjperkins?


Is there a way to run validation through command line on entire directory to get a report or from within the vscode

Hi
Is there a way to run spell validation through command line on entire directory or from within VsCode to get a report for all files at ones. Right now this extension shows errors only for the open files.
Regards

Not directly from the extension.
One option is to:

Install cspell, which is the library this spell checker is based upon.
Invoke cspell from the Integrated Terminal (View -> Integrated Terminal)
Example to check all go files recursively excluding the vendor directory:

cspell -e "vendor/**" "**/*.go"

If you have a cspell.json file in your project directory, it will automatically pick it up. Or you can tell it where to find it via the -c option.

I'm going to close this. If you have any more questions, please feel free to open again or create a new issue.


Dynamic Slide with two columns don't work very well

I've trying to do a dynamic slide with two columns separate each one, but the main div size render small, when I click on slide or resize the window the div become with your normal size.
Use inspector to analyze, i realize that class .VueCarousel-inner begin with this value "flex-basis: 810px"  and after resize widow it change to "flex-basis: 1140px".
Here is my code:
<carousel v-if="projects.length"
            :perPage="1"
            paginationActiveColor="#4d2df5"
            paginationColor="#b2ebd1"
            :scrollPerPage="false"
            >
              <template v-for="(project, index) in projects">
                <slide>

                       <router-link :to="{ name: 'details', params: { slug: project.slug }}" >   {{ project.title.rendered }}</router-link>

                    {{ tags.name }}

               </slide>
               </template>

          </carousel>

I think that problem happen because :perPage="1" not identify suddenly.
Since now, Thanks for help me!!!

Hey @agtoncerqueira, are rendering these carousels on the server or the client? I've seen a similar issue with SSR but never with regular rendering

Hey @quinnlangille I'm rendering from the server, but i don't know what happened everything is working well again...
The Config is the same, I just added ":paginationSize="12"" but this it's not too important. My project back normal without explanation I hope that keep like this in the future on production!
Sorry me for take a long time to answer!!!

Hey @agtoncerqueira, we've been doing quite a bit of development in the past few weeks, so it's likely this issue was resolves as a side-effect from one of the releases. I'll close the issue, thanks for responding!


Integrate firebase authentication + Home page with tabs

This PR has the following UIs:

Sign in page with the logo and google login + firebase authentication
On successful login the user is redirected to the homepage with custom tab bar
4 views on the custom tab bar - home, prompt, log and user profile

Addresses issues #1 and #4

Looks good to me!

Fantastic job, Shivanee!


Error: Could not load file or assembly 'System.Linq.Expressions' using .NET 8 in-proc when debugging

This is on Apple M2, so could very well be related to ARM support.
Repro steps:

Create new Functions project, select C#, and .NET 8 (not isolated)
Run the build task using this tree item

Start debugging with

It started debugging with no issues
4. Making a request the Function trigger fails with an error:

Executed 'GetJoke' (Failed, Id=0a654229-8734-490b-ab13-0d83f022b6a8, Duration=68ms) [2024-06-27T18:59:26.333Z] System.Private.CoreLib: Exception while executing function: GetJoke. jokes-api: Could not load file or assembly 'System.Linq.Expressions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified.

Hi, Any updates on this error so far?

@ejizba Do you know if there's any update on this?

Sorry, I'm not familiar with this problem. I'd suggest creating a bug in the core tools repo and/or reach out to the core tools folks offline if it's a big blocker
cc @mattchenderson @kshyju

I solved this issue upgrading Microsoft.NET.Sdk.Functions  to version 4.4.0.


Graphics card: Microsoft Basic Render Driver is blacklisted.

Dear Sir
i get this
Graphics card: Microsoft Basic Render Driver is blacklisted.
at  ComposeSnake:desktop:run
THX

Hello @CMingTseng, I suggest to raise this issue in https://github.com/JetBrains/compose-jb, it looks like related to Compose for Desktop.


File path info in chunkFilename function when optimization.chunkIds is deterministic

Feature request

What is the expected behavior?
I am using the output.chunkFilename function to generate friendly readable chunk filenames based on their source path.  This works well if optimization.chunkIds is set to named.  However, this leaves big long ids in the source code references (e.g. "src_web_app_components_home_index_js").  Ideally, I'd like to leave optimization.chunkIds set to deterministic (the default in a production build), so the chunk ids are small numbers. The issue is that the chunk info provided to the chunkFilename function no longer has any identifiable information about the file paths to generate the filenames anymore.
What is motivation or use case for adding/changing the behavior?
Automatic readable filenames with smaller code.
How should this be implemented in your opinion?
Include more information to the chunkFilename regardless of the optimization.chunkIds setting.
Are you willing to work on this yourself?
yes

You can use /* webpackChunkName: "my-chunk-name" */

Also you can implement own plugin for naming, example https://github.com/webpack/webpack/blob/main/lib/ids/DeterministicChunkIdsPlugin.js

I want the file naming to be automatic based on the source path without having to add webpackChunkName comments everywhere. I also just want the default DeterministicChunkIdsPlugin id generation.
I guess i'm looking for where output.chunkFilename gets called in the source code.  All of the name & files info gets excluded from the chunk in deterministic mode.

I guess i'm looking for where output.chunkFilename gets called in the source code. All of the name & files info gets excluded from the chunk in deterministic mode.

it is not easy and not possible in some cases

I'm having some success simply applying both plugins. Not sure of any unintended side-effects yet:
const { DeterministicChunkIdsPlugin, NamedChunkIdsPlugin } = require('webpack').ids

class NamedDeterministicChunkIdsPlugin {
  apply(compiler) {
    // Apply named chunks plugin (assigns name to chunk.id)
    new NamedChunkIdsPlugin().apply(compiler)

    compiler.hooks.compilation.tap('NamedDeterministicChunkIdsPlugin', (compilation) => {
      compilation.hooks.chunkIds.tap('NamedDeterministicChunkIdsPlugin', (chunks) => {
        chunks.forEach((chunk) => {
          chunk.name = chunk.id // Store the name on another field
          chunk.id = null // null out id so the next plugin will reassign it
        })
      })
    })

    // Apply deterministic chunks plugin (reassigns chunk.id to a short numeric id)
    new DeterministicChunkIdsPlugin().apply(compiler)
  }
}

Interesting solutions :smile:  but should work without problems :smile:

The example I posted breaks in watch mode.  Ended up just making a simpler plugin to assign a chunk.name based on the root module file path.  Ideally webpack could leave all the chunk info in deterministic mode.

Ended up just making a simpler plugin to assign a chunk.name based on the root module file path.

Would you mind posting that or the most salient pieces? I'm currently struggling because /* webpackChunkName "[request]" */ is resulting in conflicts when I have, e.g. import('path/one/to/bar.js') and import('path/two/to/bar.js').

@jamesarosen  This was my final solution. As-is, it would give you bar.js for both, but you should have all the pieces there to adjust it.
class AutoChunkNamePlugin {
  apply(compiler) {
    compiler.hooks.compilation.tap('AutoChunkNamePlugin', (compilation) => {
      compilation.hooks.chunkIds.tap('AutoChunkNamePlugin', (chunks) => {
        chunks.forEach((chunk) => {
          if (!chunk.name) {
            const chunkModule = compilation.chunkGraph.getChunkRootModules(chunk)[0]
            const rootModule = (chunkModule && chunkModule.rootModule) || chunkModule
            const rootPath = rootModule && rootModule.userRequest
            const name = rootPath && rootPath.split(path.sep).slice(-2)[0]
            if (name) chunk.name = name
          }
        })
      })
    })
  }
}


Negative Resin

If you click > to preview the resin amount tomorrow, then click any of the buttons to subtract resin, and then click < to return to your current time, you'll have negative resin that shows as 0.
Clicking > again won't fix the issue, the preview will act like usual and add hours to simulate regeneration without touching the actual number.

This is working as intended as far as I'm concerned. If you modify resin value and go back in time, the extrapolated value will be negative, but it's not possible to have negative resins, hence it's clamped in the range [0, 160].
The same behavior can be observed when marking tasks as done while previewing the next few days. I think this is a logical behavior.
The only "fix" for this might be to disallow editing things altogether while in preview.

As logical as it is, I don't think anybody intends to have negative resin.
If you look at the times, you'll see that even thought it shows as 0, it's still being calculated as if it were a negative number.
Disallowing editing things while in the preview is a good fix, but you could alternatively make it not touch your resin in the present since resin you edit in the "future" shouldn't affect the resin you have in the present.

you could alternatively make it not touch your resin in the present since resin you edit in the "future" shouldn't affect the resin you have in the present.

It's definitely possible, but I think the complexity required code-wise to address this edge case makes it not worth pursuing. This will require us to clone another instance of the data store for previewing, but the current code isn't written with this in mind.
I think disabling edits during preview is a practical way to go. Other issues should be prioritized, but I'll keep this open in case someone else wants to tackle it.


Fix script properties being lost and prevent node reference corruption upon scene reimport

Closes #92278
Closes #92277
Expands and depends upon the previous PR (https://github.com/godotengine/godot/pull/92182) to fix further bugs in the hot scene reloader. It now prevents script properties on reimported nodes from getting lost, and prevents node reference properties referencing nodes which get replaced by the reimport system from being retained, preventing several cases which can cause editor crashes.

@lyuma I expect this to solve your common case of "click on the skeleton" and click reimport to see Godot Engine crash bug.

Thanks @fire, I was a bit confused how the CoreStringNames API was meant to be used.

Thanks!


gophercloud: unbounded recursion on auth failure in ProviderClient.Request()

If AllowReauth is enabled when creating a service client, authentication failures will trigger a recursive path. If authentication fails, the Request method will call ReauthFunc, then call itself recursively. This handles 401->201 transitions, but becomes unbounded if the response is never successful, e.g. incorrect credentials.
A potential solution is to remove the client's ReauthFunc when recursing, so that the second invocation of Request will not recurse further. It remains to be determined whether this is safe, however.

@bwarden please check github.com/gophercloud to see if the issue exists and file there if it does.

https://github.com/gophercloud/gophercloud/issues/187

Upstream bug rejected, referring to https://github.com/gophercloud/gophercloud/blob/master/auth_options.go#L46, which suggests we should add a custom HTTP agent to our implementation, e.g. https://github.com/rackspace/rack/blob/1.0.0/auth/clients.go#L311

@rbradford Do we even use this code any more?

We don't use this code anymore.