Horizontal scaling support on (Meteor) Galaxy Server

Hello,
We are working on a project (just like an chatting app) based on meteor and hosted on Galaxy server, and using "rocketchat:streamer" to establish communication b/w multiple devices.
It works fine if the app is deployed on single container but if we scale the app horizontally i.e. if we create more than one container it breaks the functionality, a device connected to one container can't communicate with device connected to other container.
We are not using database for any reason, just emits events with a uniqe string and perform tasks
if a perticular events matches,
Following is code...
const streamer = new Meteor.Streamer('streamer_name');
streamer.emit("unique_pin", {
event: 'anyEventName',
...
});
streamer.on("unique_pin", (data) => {
const { event } = data;
if (event === 'anyEventName') {
// Do some task if event matches
}
}
Thanks

take a look at #26 😉

Thanks for your reply
We tried this but the problem is not resolved, there is no change.
But It creates db which I think it is not being used.
import { Meteor } from 'meteor/meteor';
import sync from 'meteor-sync-eventemitter';
@sync({ collectionName: 'stream'})
class SyncedStream extends Meteor.Streamer {
constructor() {
super('game');
}
}
const streamer = new SyncedStream();
streamer.emit(pin, {
event: 'userJoin'
});
It works only for single container, are we missing something?

You haven't configured meteor-sync-eventemitter for rocketchat:streamer.
It should work if you change the decorator to
@sync({ collectionName: 'stream', emitFn: '_emit' })

Take a look at https://github.com/maxnowack/meteor-sync-eventemitter#rocketchatstreamer

Unfortunately this package doesn't scale horizontally automatically. On Rocket.Chat we've done a "stream matrix" manually, which uses a database to list all running servers and then connect each of them to emit changes.

It’s so crazy that there still doesn’t seem to be an easy way to connect DDP server to server in a multi-container environment (especially Galaxy).
Resorting to a database sync seems so crude. We use redos-oplog vent. Which basically puts redis as the sync versus Mongo. Was hunting for ways to remove this round trip and just go straight from server to server.

Hi @evolross,
We had tried many different things and finally we have replaced the streamer with PubNub https://www.pubnub.com for our client's app https://www.playfactile.com

Right... that's basically what Redis does for us. It's a 3rd party server they all sync with. Wish it could be more direct.
I'm going to experiment with somehow figuring out the Galaxy URL and param programatticaly and see if DDP.connect works with that to directly connect to a container. Then it will just be a matter of keeping the container URLs in the database and heartbeat'ing them to make sure they're still alive and update when one gets replaced, crashes, etc.  That just seems cleaner, doesn't put another server/service in the middle, and should reduce lag.


BUG: Vetext-Incoming-Forwarder-Lambda response triggers error in Twilio

Describe the bug
The Vetext-Incoming-Forwarder-Lambda works as expected but when it returns a response to Twilio, it returns an object that Twilio considers an error.
To Reproduce
Steps to reproduce the behavior:

Send an inbound SMS to Vetext-Incoming-Forwarder-Lambda endpoint
Look at Twilio error logs for the SMS number used to send the message
Verify error exists

Expected behavior
A success response should be logged in Twilio when processing is successful.  An error response should be logged in Twilio when processing fails.
Additional context
The fix for this issue is to update the "create_twilio_response" method so that the response only contains the "statusCode" key in the response.
def create_twilio_response(status_code: int = 200):
    response = {
        "statusCode": status_code
    }

    return response

This change will require changing the associated test suite.  Namely:

Remove body assert in test_request_makes_vetext_call(mocker, all_path_env_param_set, event) method
Remove body assert in test_failed_alb_invocation_call_throws_general_exception_goes_to_dead_letter_sqs method
Remove body assert in test_unexpected_event_received method

@jakehova do you have a copy of the error message you could add to this ticket?
@cris-oddball @k-macmillan for your review

@jakehova please fill in a section for AC, both positive and negative testing. The Expected Behavior is not really sufficient.

Hey team! Please add your planning poker estimate with Zenhub @cris-oddball @EvanParish @ianperera @jakehova @justaskdavidb2 @k-macmillan @kalbfled @ldraney

I believe this ticket is also related to this issue: https://github.com/department-of-veterans-affairs/notification-api/issues/939

created draft PR with the change: https://github.com/department-of-veterans-affairs/notification-api/pull/1117
I believe this fix may also resolve #939 .  I will confirm during my testing.

I have confirmed that this change resolves the 502 and does not log any errors in twilio.  HOWEVER, it does log a warning for invalid content-type.  I'm going to update the content type to be application/json to see if it resolves the warning as well.

Had to update the response object to follow the schema provided by Twilio's docs .
I updated the response object to match and now there are no errors or warnings.

code scan/dependency scan/unit tests are failing due to something that changed in the repo.  The results of those steps point to files that were not touched in my branch.  I'm not sure any PR will pass testing unless we go through all the results and resolve the issues brought up in the scans.

After the above issue is resolved... @jakehova or @k-macmillan whomever tests this can you please validate that the error is no longer thrown in VEText's twilio account and a screenshot is provided here for confirmation? @cris-oddball if you want to peek over Jacob/Kyle's shoulder while they are in there please coordinate with them. I'll ask VEText if they can add you to the account for future validation.

Asked Paras to add @cris-oddball to the test and enterprise Twilio accounts. Cris, keep an eye on your email and spam folder today/tomorrow. If you don't see anything by then let me know.

Reviewed the code associated with my comment on the PR. No changes were necessary. Merged master into the branch and requested review.

Code deployed to Perf.

changed twilio number for +19703005680 to use the Perf webhook at https://sandbox-api.va.gov/vanotify/twoway/vettext
had to update ALB to accept either public or private IF path is perf-api.va.gov OR sandbox-api.va.gov PTH is twoway/vetext THEN forward to perf-vetext-incoming-tg 1
twilio logs only showing errors for all the bad tests we sent.
cloudwatch logs indicate message received and processed, but throw an error, see below

Why are we throwing a cert error and why is it reporting the staging endpoint?
START RequestId: 8a8c88d2-c393-4201-bc0c-173688e7c988 Version: $LATEST

[INFO]   2023-03-16T15:01:54.712Z          8a8c88d2-c393-4201-bc0c-173688e7c988   alb invocation

[INFO]   2023-03-16T15:01:54.712Z          8a8c88d2-c393-4201-bc0c-173688e7c988   Successfully processed event to event_bodies

[INFO]   2023-03-16T15:01:54.713Z          8a8c88d2-c393-4201-bc0c-173688e7c988   Retrieved AuthToken from SSM

[INFO]   2023-03-16T15:01:54.713Z          8a8c88d2-c393-4201-bc0c-173688e7c988   Making POST Request to VeText using: https://staging.api.vetext.va.gov/api/vetext/pub/inbound-message/twilio

/opt/python/lib/python3.8/site-packages/urllib3/connectionpool.py:1045: InsecureRequestWarning: Unverified HTTPS request is being made to host 'staging.api.vetext.va.gov'. Adding certificate verification is strongly advised.


Machine mac-41 is Quarantined state due to error.

Is there an existing issue for this?

[X] I have searched the existing infra issues

Type of Request
infra task
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
Machine mac-41 is Quarantined state due to error.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

It looks like you're dealing with an issue related to a machine named "mac-41" in your infrastructure environment, which is in a Quarantined state due to an error in the device doctor tool. Specifically, the error message mentions "Unhandled exception: device_id not found."
To find information or report this issue, you should check your internal documentation, issue tracking system, or communicate with your infrastructure or devops team. If you've already searched for existing issues, and there isn't one addressing this specific problem, you may want to create a new issue for it and provide all the relevant details, including the error message, steps to reproduce, and any relevant system information. This will help your team investigate and resolve the issue effectively.

FYI: flutter-devicelab-mac-45 is having same issue: https://github.com/flutter/flutter/issues/136549

@keyonghan  Hi Keyong,  Can you take a look machine mac-41? It's in Quarantined state with error "Quarantined: Device doctor tool error: b'Unhandled exception:\nidevice_id not found". Everything is running fine from machine side.
Thanks so much.

I'll get this one this is probably caused by the reinstallation of homebrew.

@godofredoc Same error is starting on machine mac-4 as well.

https://chromium-swarm.appspot.com/bot?id=flutter-devicelab-mac-5 is having same issue

Machine mac-5 is getting same error.

Adding mac-2 to the listing.

Adding mac-6 to the listing.

Adding mac-39 to the listing.

Fixed adding the arm64 brew path to the swarming service path configuration file.


Simplified reference methods.

This PR changes the reference and referenceSet search methods to accept single filter arguments rather than lists. The complex semantics of searching accessions is also reduced to simple string matching. This is part of a wider effort to simplify the schema so that it can be easily implemented and the semantics of each request is clear.
Performance arguments for allowing a client search for several references or referenceSets at once seem difficult to sustain. References are not something that a client needs to search for very often, so a few more server calls will make no difference in practice.
The semantics for searching over accessions depends on the server parsing accession numbers held within a string field. If this sort of functionality is required, then a formal model of the accession number object should be made, so that string parsing/construction is not required by either the client of the server.

+1  We really need to drive such optimization by compelling use cases.
Jerome Kelleher<EMAIL_ADDRESS>writes:

This PR changes the reference and referenceSet search methods to accept single
filter arguments rather than lists. The complex semantics of searching
accessions is also reduced to simple string matching. This is part of a wider
effort to simplify the schema so that it can be easily implemented and the
semantics of each request is clear.
Performance arguments for allowing a client search for several references or
referenceSets at once seem difficult to sustain. References are not something
that a client needs to search for very often, so a few more server calls will
make no difference in practice.
The semantics for searching over accessions depends on the server parsing
accession numbers held within a string field. If this sort of functionality is
required, then a formal model of the accession number object should be made, so
that string parsing/construction is not required by either the client of the
server.
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
You can view, comment on, or merge this pull request online at:
https://github.com/ga4gh/schemas/pull/377
Commit Summary
• Simplified reference methods.
File Changes
• M src/main/resources/avro/referencemethods.avdl (30)
Patch Links:
• https://github.com/ga4gh/schemas/pull/377.patch
• https://github.com/ga4gh/schemas/pull/377.diff
—
Reply to this email directly or view it on GitHub.*

Thanks for clarifying the comments @jeromekelleher. Adding back two fyi's that got lost because I put them on the (now outdated) diffs:

I was curious about the history of this part of the query interface -- it looks like it's unchanged since @richarddurbin first submitted it in #66
the current Google implementation does a "STARTS_WITH" for accessions, which is a simple way to support our understanding of the original desired functionality.

After today's DWG call, I'd like to classify this as a bug (which means we want to resolve it before the next release). The consensus is we'd like to hear from @richarddurbin before anything else is done.

+1

@dglazer, @calbach, any thoughts on this? I've updated the PR to squash the commits so it's ready to merge. This is a breaking change, but I hope one that will make the references API easier to use.

+1, albeit reluctantly on account of the breaking changes. Agreed that it removes superfluous functionality.

+0
I'm fine with this simplification, since @richarddurbin is, but otherwise am not close enough to the use cases to assert that we're not removing important functionality.

Looks like we have the requisite +1s --- anyone care to merge?


xsnow: 3.4.4 -> 3.7.9, use wrapGAppsHook3

Update to 3.7.9, use gappshook
closes #360757
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

[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/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

I've been using this pull request in my flake for about a week without any problems.


slideSwitcherView become transparent？

let view = UIView()
view.backgroundColor = UIColor.red
slideSwitcherView.superview?.insertSubview(view, belowSubview: slideSwitcherView)
now, slideSwitcherView become transparent？

Thanks for feedback!

let view = UIView()
view.backgroundColor = UIColor.red
slideSwitcherView.superview?.insertSubview(view, belowSubview: slideSwitcherView)
now, slideSwitcherView become transparent？

It's not recommended.

segementSlideSwitcherView.layer.zPosition = -1

This is to make the drop-down refresh control work properly.

A version of V3 is in development, and there is a high probability that the development and release will be completed this week, when the Switcher control will be pulled out to make it easier to use custom controls. 😉


Overall statistics

So far we have addressed the statistics one item at a time.
But how about looking at the statistics as a whole?
Analyze the activities in the same chart, to highlight efficiency, competition, anomalies, etc. between the different activities carried out.

I didn't want to make this app overly 'heavy' (e.g. with multiple dimensions of statistics) in the initial design. However, having some simple blended trend charts doesn't seem excessive. I can consider designing and implementing them in future iterations.


Error getting variable in PUAE core running CD32.

Using the puae_libretro core, when attempting to build a session which loads a CD32 game, the following error is issued:
Exception ignored on calling ctypes callback function: <bound method DictEnvironmentDriver.environment of <libretro.drivers.environment.composite.CompositeEnvironmentDriver object at 0x0000025A936C6CF0>>
Traceback (most recent call last):
  File "C:\Users\jsens\AppData\Roaming\Python\Python312\site-packages\libretro\drivers\environment\dict.py", line 44, in environment
    return self._envcalls[envcall](data)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\jsens\AppData\Roaming\Python\Python312\site-packages\libretro\drivers\environment\default.py", line 97, in <lambda>
    EnvironmentCall.GET_VARIABLE: lambda data: self._get_variable(
                                               ^^^^^^^^^^^^^^^^^^^
  File "C:\Users\jsens\AppData\Roaming\Python\Python312\site-packages\libretro\drivers\environment\composite.py", line 527, in _get_variable
    result = self._options.get_variable(string_at(var.key) if var.key else None)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\jsens\AppData\Roaming\Python\Python312\site-packages\libretro\drivers\options\dict.py", line 75, in get_variable
    value = string_at(self._options_us[key].default_value)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Program Files\Python312\Lib\ctypes\__init__.py", line 525, in string_at
    return _string_at(ptr, size)
           ^^^^^^^^^^^^^^^^^^^^^
OSError: exception: access violation reading 0x0000000000000000

The system appears to run fine after the error.
The variable (key) is  b'puae_cart_file'. It is in self._options_us but not in self._variables.
self._options_us[key].default_value is set to None, which means it's essentially trying to get string_at(None). There probably should be a check for this case, but I don't know off-hand what a reasonable return value would be (b'', None, or even b'None'?).

The 3DO core opera_libretro has the same issue when getting b'opera_bios' and b'opera_font'. It could be preventing the core from finding a default BIOS file. As such, this core might be better for testing a potential fix.


add a thor based generator

Doesn't this belong in conduit core?
On Friday, October 14, 2016, Corey Purcell<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/conduit/conduit-atom/pull/52
Commit Summary

add a thor based generator

File Changes

M Rakefile
https://github.com/conduit/conduit-atom/pull/52/files#diff-0 (43)
A bin/generate_action
https://github.com/conduit/conduit-atom/pull/52/files#diff-1 (5)
M conduit-atom.gemspec
https://github.com/conduit/conduit-atom/pull/52/files#diff-2 (1)
A lib/conduit/atom/generators/action.rb
https://github.com/conduit/conduit-atom/pull/52/files#diff-3 (63)
A lib/conduit/atom/generators/blueprints/action.tt
https://github.com/conduit/conduit-atom/pull/52/files#diff-4 (9)
A lib/conduit/atom/generators/blueprints/decorator.tt
https://github.com/conduit/conduit-atom/pull/52/files#diff-5 (9)
A lib/conduit/atom/generators/blueprints/mock.tt
https://github.com/conduit/conduit-atom/pull/52/files#diff-6 (8)
A lib/conduit/atom/generators/blueprints/parser.tt
https://github.com/conduit/conduit-atom/pull/52/files#diff-7 (13)
A lib/conduit/atom/generators/blueprints/rspec_action.tt
https://github.com/conduit/conduit-atom/pull/52/files#diff-8 (5)
A lib/conduit/atom/generators/blueprints/view.tt
https://github.com/conduit/conduit-atom/pull/52/files#diff-9 (1)

Patch Links:

https://github.com/conduit/conduit-atom/pull/52.patch
https://github.com/conduit/conduit-atom/pull/52.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/conduit/conduit-atom/pull/52, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAN6-ALEQQXZGODlm47Q5iZcARcNt3fMks5qz-jngaJpZM4KXa48
.

--
Pedro Martinez
Director, Product Development
Hello Labs
561-451-6630

I can try moving it to core, ideally it's available and easy to add to. It might be a little tricky. What say you @pedromartinez

Might be worth considering at some point. If it's a tricky we can always revisit. Your call.

after reviewing how conduit base does it's generate_driver. I think we can easily use that to do a generate_action_generator or something in base that adds these files without generating a full driver. Is that good enough or should we switch to making base classes that we can inherit from?

Not sure I fully understand. Don't we already have an action base class?

no our generator/action.rb class inherits from Thor. I'm referring to the generator itself.

Oh, that would be pretty cool if I'm understanding things correctly.


Rio doesn't work in eks 1.13

Install Rio v0.5.0-rc1 in EKS
rio systemlogs will show the following error
E0926 17:10:04.138471    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:04.384964    4161 controller.go:117] error syncing 'rio-system/registry': handler ingress-app-object-controller: the server could not find the requested resource, handler routing-serviceset-object-controller: the server could not find the requested resource, requeuing
E0926 17:10:05.289905    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:06.446585    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:07.615544    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:08.782817    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:09.971205    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:11.129773    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:12.289673    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:13.446004    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:13.452752    4161 controller.go:117] error syncing 'rio-system/build-controller': handler ingress-app-object-controller: the server could not find the requested resource, handler routing-serviceset-object-controller: the server could not find the requested resource, requeuing
E0926 17:10:14.605047    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:15.763087    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource
E0926 17:10:16.926080    4161 reflector.go:126] pkg/mod/k8s.io/client-go@v11.0.1-0.20190409021438-1a26190bd76a+incompatible/tools/cache/reflector.go:94: Failed to list *v1beta1.Ingress: the server could not find the requested resource

It turns out that ingress is not in networking.k8s.io/v1beta1 group in eks 1.13. Maybe we should update docs to support cluster up to 1.14 since we are using networking.k8s.io/v1beta1 api group.

using rio v0.5.0-rc2.
EKS cluster v1.13.10-eks-5ac0f1
EKS cluster v1.14.6-eks-5047ed
GKE cluster v1.14.6-gke.1
k3s v0.7.0 (smoke pass local)

Rio install - DONE.
Spot checking the main functionality - DONE
Public domain test - DONE.


[FEATURE] Add Zod for validating server action input

Description
Server actions should be treated as regular endpoints, which means we need to check for auth, validate input, etc... Good practice is to use zod for it 👍
Screenshots
No response
Additional information
No response

This is implemented already, closing it as a already there 👍


Substitute md5 for sha1sum

Electra's Core Utilities package doesn't include md5 on iOS 11.3.1, but it includes sha1sum. This simple fix should still work on previous versions of Electra or LiberiOS.

Should've checked opened PRs before creating this one. Closing this PR in favour or https://github.com/BishopFox/bfinject/pull/43


Accessing Image Manifest

Hello all,
This may be a rather elementary question/issue, but I'm having problems accessing a test image manfest. The image in question is here: https://digital-anon-facsimiles.s3.eu-west-2.amazonaws.com/m48-1-test.tiff but I cannot seem to access a manifest for it using the suggested pattern in the Source Images section of the readme. I've been trying variants on the following: https://digital-anon-facsimiles.s3.eu-west-2.amazonaws.com/iiif/2/m48-1-test/info.json but haven't been having much luck.
Also, when I use the CloudFormation endpoint in place of the S3 bucket URI (i.e., https://dm2vkd8jzjuqr.cloudfront.net/iiif/2/m48-1-test/info.json) I get the following error message: InvalidARN: ARN accountID does not match regex "[0-9]{12}".
I'm probably missing something very basic but am new to AWS...
Thanks in advance!
Josh

Hello @JoshuaAPhillips,
The default S3 key pattern for TIFF files is %s.tif (note the single f on the end of the file extension). If you rename your test file to m48-1-test.tif and make sure that your serverless-iiif stack was created using digital-anon-facsimiles as the SourceBucket parameter, you should then be able to load your image via the CloudFront endpoint (i.e., https://dm2vkd8jzjuqr.cloudfront.net/iiif/2/m48-1-test/info.json).

Hello @mbklein, thanks for this - it's worked perfectly for this one test image. I've been able to access a manifest which is accessible through the Digital Bodleian's IIIF manifest editor tool. However, I've tried a couple of other test image. While the server generates manifests for both that are accessible through the URI, when I put them into the same tool they don't resolve.
The URIs are:
https://dm2vkd8jzjuqr.cloudfront.net/iiif/2/m48-2-test/info.json
https://dm2vkd8jzjuqr.cloudfront.net/iiif/2/m48-3-test/info.json
Is this something you might be able to help with?

There's at least one and possibly two things going on here – one is a software issue, while the other is an issue of terminology.
If you're trying to submit https://dm2vkd8jzjuqr.cloudfront.net/iiif/2/m48-2-test/info.json to the IIIF manifest editor as a manifest to open, that's not going to work. The info.json file isn't a presentation manifest; it's an image information document.
If you're trying to create a new manifest or add this image to an existing manifest, then there's a separate issue going on – your web browser is refusing to load the image information because it's failing its CORS preflight check. It's a lot to explain if you're not already familiar with CORS, but because of the type of request the manifest editor is making, it's expecting a response from the server that isn't matching. This is definitely something we can (and should) fix in the IIIF server's code, but I'm going to have to consider the best way to make the (minimal) security implications clear to those installing the server.
Thanks for bringing this to my attention, and please keep an eye out for a fix.

Thanks for this reply. I've been doing the latter: creating a new manifest and adding the info.json link as a canvas in that manifest. Curiously, this seems to work from my laptop, but not from my desktop and from what you've said I suspect it's because I modified CORS on my laptop's browser a while back.
I'd be very grateful indeed if you could keep me posted about any possible changes or fixes.
All best,
Joshua

Thanks for letting me know! I'll keep an eye out for the upgraded AWS app.
Best,
J

serverless-iiif v4.3.1 has been released. When you apply the update, you're going to want to make sure you set two of the new parameters:

CorsAllowCredentials: true
CorsAllowOrigin: REFLECT_ORIGIN

I'm not sure if you'll need to override CorsAllowHeaders or not. I'd try it without and only change it to authorization, cookie if the problem isn't solved without it.

Hi @mbklein, just playing around with the re-deployed the AWS app and it seems to be working perfectly now: thank you so much for your help!
Thanks again, and all best,
J

Sure thing! I noticed a small bug in the deploy template after release, and it's been patched and published now. No immediate need to update if it's working for you, but you might want to make sure you're on at least v4.3.2 at some point just to stay current.


Tabs are not working in current build

Using the current build (https://github.com/twbs/bootstrap/commit/721d948d949f783d3213f383d1ef97ff909a2249) the tabs seems broken for me. You can see a demo of this issue here:
https://plnkr.co/edit/VBjPURJoPwCeQWuyt3OB?p=preview
I've included the JS (see bootstrap-721d948.min.js) from the master branch.
Clicking the tabs throws a Uncaught TypeError (Cannot read property 'nodeName' of undefined), looks like a regression from #23087 at this line https://github.com/twbs/bootstrap/pull/23087/commits/2eb1e687bd74d147145000b3971821cc890d7652#diff-02a6d2619ddbbd65ac148de3377674eeR154.

Update:
Looks like when applying the base .nav class on the HTML
 tag:
--- index.html	Tue Aug 22 17:29:03 2017
+++ index.html	Tue Aug 22 17:28:39 2017
@@ -18,7 +18,7 @@
       </button>

-

+

             Over mij

(https://plnkr.co/edit/lhm70q5YxdsHg10kYjjA?p=preview)
That the error is gone.
To reduce confusion, should this class also be added on the starter-template? Or is it better that the class navbar-nav is added on Tab.js nav selector? Note that everything was working fine with alpha 6 (so without the base .nav class).

I don't see the use of our Tabs plugin in the starter-template so I don't think we should change something here.
About the .nav class it's writtin in our docs that every Nav must have a .nav class see : https://getbootstrap.com/docs/4.0/components/navs/

Just ran into this after upgrading from bootstrap4 beta1 ->2. Not sure why it just popped up now and not before, but these issues that depend on a parent css class to work are extremely difficult to debug. Would be nice if there weren't these arbitrarily-required parent classes whose only purpose is for jquery to point at.

Would be nice if there weren't these arbitrarily-required parent classes whose only purpose is for jquery to point at.

Yes as this means we can't just leverage the Tabs JS with custom markup, but must conform to the Bootstrap Way™


CI: Multicluster / Cluster mesh provisioning fails with The network "default" does not have available private IP space

CI failure
A (ephemeral?) provisioning failure which can occur in the CI Multicluster (maybe also ConformanceGKE) test:
zone: 'us-west2-a'>] finished with error: The network "default" does not have available private IP space in <IP_ADDRESS>/9 to reserve a /14 block for pods for cluster {Zone=us-west2-a, ProjectNum=185287498374, ProjectName=***, ClusterName=cilium-cilium-2547825658-mesh-2, ClusterHash=49b8b652c99f4a2499e61bb405d765399a0c4832836647f99dd4ee06e0e5f328}.

Observed in: https://github.com/cilium/cilium/runs/7019814122  -- logs_510456.zip

FWIW, this is also occurring in cilium-cli CI: https://github.com/cilium/cilium-cli/issues/940

Also observed on https://github.com/cilium/cilium/pull/21212 for GKE, multicluster and external workloads workflows, e.g. https://github.com/cilium/cilium/runs/8207578433?check_suite_focus=true


Script editor code folding color is influenced by main theme base color

Godot version
3.4.2 stable
System information
Windows 10
Issue description
Code folding color is influenced by main theme color. It's hard to see difference when change the code folding color. But when you change the main theme color, the arrow color changes.
Use a  main dark color

Use a main light color

Change Code Folding Color doesn't do anything

Steps to reproduce
Use a light theme and a dark theme code editor theme. Change code folding color settings in editor settings.
Minimal reproduction project
No response

Tested 45eaa2daf1e3d2571a9f31a3421911262ec82f51 (3.4.2-stable) on Ubuntu 21.10
I'm able to reproduce the issue if I change the Text Editor -> Theme -> Color Theme setting from 'Adaptive' to 'Custom'. This prevents the code editor from changing its background to match the rest of the application.
The color actually does change very slightly, but the color itself is incorrect.

I'll try fixing this if I get the go ahead from one of the maintainers.

I'll try fixing this if I get the go ahead from one of the maintainers.

Sure, feel free to use a fixed color instead. Make sure it still adapts to the automatic light/dark theme switch in the Adaptive theme (called Default in the master branch).


can't download dataset?

Hi. help pls. thx
sudo python download.py /opt/kinetics/train_split/data/kinetics_train.csv /opt/kinetics/train_split
Traceback (most recent call last):
File "download.py", line 10, in
from joblib import delayed
ImportError: No module named joblib
conda install joblib
Collecting package metadata: done
Solving environment: done
All requested packages already installed.
pip install joblib
Requirement already satisfied: joblib in /home/admin/.pyenv/versions/anaconda3-2018.12/envs/kinetics/lib/python2.7/site-packages (0.13.2)

I am facing the same issue. Did you found the solution?

Python 2.7 expired. Perhaps it's related to that. Good luck :)


Readable error messages for parsers

Parsers now output the following error message:
"Expressions are not supposed to be destroyed."

Someone should implement destructors/garbage collection on expressions.

think it will be solved by replacing ltl2ba: see e.g. #15


test: mark stdlib/StringSwitch as executable_test

This is a runtime test, ensure that is marked as an execution test.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test


Registered ID's for uniquely identifying servers.

It would be nice if server owners could register a unique server id for each game server they want to host.
This way an individual server, eg: laundmo's fastball randomiser, could be uniquely identified between restarts, allowing for stuff like favoriting servers.
I'm not exactly sure yet what would be the best way of registering and authenticating these servers, but perhaps a short key pair between game and master servers would suffice.

Relates to #41

Closed by https://github.com/R2Northstar/NorthstarMasterServer/pull/41


Get predicate from source sheet

Hi.
For my use case, I need to read the Predicate linking two entities (two "subjects") from a source sheet.
It was a bit tricky to do that directly with YARRRML but I managed to make it work.
I wonder if it is something Mapeathor could do ?

Hi,
In Mapeathor two subjects can be linked in the Predicate_Object Sheet, not in the Source sheet. The Subject sheet's purpose is only to declare the subject, what's it's the IRI and class. Linking two subjects falls into creating triples, that is why it is the Predicate_Object Sheet. Two subjects previously declared in the Subject sheet can be linked there, using the columns ID (subject1), predicate, referenceID (subject2), InnerRef and OurterRef (fields in data that have to be equal to perform the link), just as it is shown in the example in the wiki. There is another option to link two subjects "manually" without using these columns, that is writing the IRI of the new subject in the Object column, with 'IRI datatype'.
Hope this answers your question,
Ana

Well, actually what I need to do is to read the predicate field of the Predicate_Object sheet from my input csv file.
My data are interspecies interactions records, and my input CSV file looks like that :

ID
SUBJECT
INTERACTION
OBJECT

0
species1
obo:eats
species2

With Mapeathor I can create two entities in the Subject sheet, one for species 1, one for species 2.
What I's like to have in the Predicate_Object sheet is something like this :

ID
Predicate
Object
DataType
ReferenceID
InnerRef
OuterRef

SUBJECT
{INTERACTION}

OBJECT
{ID}
{ID}

so that the Predicate field (here the 'eats' interaction) is read from my input file.

Ok I see. For now, the tool cannot interpret predicates as rml:reference, it can for rr:template (obo:{INTERACTION}) and constant (obo:eats). I'll incorporate it, thanks for letting me know :)

Done


A few longer administrative strings tagged for translation are not actually translated

I got a report from one of our translators that two of our settings pages, /#organization/filter-settings and the bottom of /#settings/your-account have large sections that don't seem to be translated properly:

What's weird is that the blocks are tagged for translation with tr this and have translation data in the locale/ directory, which would suggest that the bug is in the handlebars+i18next integration or how we're using it (we're extracting the strings correctly for translator, but not actually doing the substitution correctly at runtime).  Debugging is required.
If I had to guess, I suspect the problem is that these are the places where we're using tr this with a multi-line block of text, and we're stripping whitespace in only one of "when we construct the strings for translation" and "when we try to substitute translated strings".  If you look in the `translations.files, e.g., there's no leading whitespace on those lines:
locale/zh_Hans/translations.json:  "An API key can be used to programmatically access a Zulip account. Anyone with access to your API key has the ability to read your messages, send messages on your behalf, and otherwise impersonate you on Zulip, so you should guard your API key as carefully as you guard your password.
 We recommend creating bots and using the bots' accounts and API keys to access the Zulip API, unless the task requires access to your account.": "API Key用于编程开发访问你的Zulip账户，任何人使用你的Key都可以：读取你的信息，使用你的身份发送信息，所以请不要公开API Key，像保护你的银行卡密码一样保护好它
我们建议创建机器人账户权限的API Key来访问Zulip API，除非某些特定任务需要

which would suggest maybe it's that the runtime whitespace handling is not stripping whitespace whereas the string-extracting time code is?
@andersk do you have any ideas for where the whitespace stripping is happening incorrectly?

Here's something that I found.
Whenever we use {{#tr this}}  we start a new line with leading whitespace with multiple lines. This text is sent to the helper function Handlebars.registerHelper('tr', function (context, options)  in static/js/templates.js.
The code looks something like this
    const result = i18n.t(options.fn(context), context);
    return new Handlebars.SafeString(result);

The const result that's returned is no different than the string between tr this tags.
Here's something that I tried but didn't work. I changed the above code to this.
    let teststring = options.fn(context);
    teststring.trim();
    teststring.replace(" ","");
    teststring.replace("\t","");
    teststring.replace("\n","");
    const result = i18n.t(teststring, context);
    console.log(teststring);
    console.log(result);
    return new Handlebars.SafeString(result);

Here's console output

What might be the reason for the above code to fail? I would like to work on this issue once it's clear what can be done.

Ultimately, this issue is a debugging challenge.  The next thing I'd try is manually finding the actual translation data in the i18n data sent from the backend to the frontend and compare the strings against what you have there for equality.


More robust handling of namespaces in pipe completion

This is a more robust implementation of https://github.com/rescript-lang/rescript-vscode/pull/843 that should cover all cases instead of just a few. Here, instead of checking for namespaces when filtering the completion items, we flatten any namespace items to their accessible paths before filtering. This allows the regular filter mechanism to just do its work like it's used to, and the filtering will work for namespaced items.

@cristianoc it would be nice to do the same in the printer, we can still get namespaced type names there:


Update with html and css file

I have  added in to my repo a html file and Css file  some other file for linters

Hi, @basitali111
I've pushed some new updates to the update -name branch. Please review the latest changes.


openvpn: 2.5.0 -> 2.5.2, openvpn_24: 2.4.9 -> 2.4.11

Motivation for this change
Fixes CVE-2020-15078.
https://community.openvpn.net/openvpn/wiki/CVE-2020-15078
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

2 packages marked as broken and skipped:

libsForQt512.plasma-nm
libsForQt514.plasma-nm

1 package failed to build:

openvpn-auth-ldap

11 packages built:

connman
connman-gtk
connman-ncurses
connmanFull
connman_dmenu
crowbar
networkmanager-openvpn (gnome.networkmanager-openvpn ,networkmanager_openvpn)
plasma-nm (libsForQt5.plasma-nm)
openvpn
protonvpn-cli
protonvpn-gui

The build failure for openvpn-auth-ldap is not introduced by this change.

Result of nixpkgs-review pr 124154 at e2df9554 run on aarch64-linux 1

2 packages marked as broken and skipped:

libsForQt512.plasma-nm
libsForQt514.plasma-nm

1 package failed to build:

openvpn-auth-ldap

12 packages built successfully:

connman
connman-gtk
connman-ncurses
connmanFull
connman_dmenu
crowbar
networkmanager-openvpn (gnome.networkmanager-openvpn)
libsForQt5.plasma-nm (libsForQt515.plasma-nm ,plasma5Packages.plasma-nm)
openvpn
openvpn_24
protonvpn-cli
protonvpn-gui

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

nixpkgs-review happy for me, nixos x86_64, nixosTests.initrd-network-openvpn appears (already) broken for me - can anyone confirm?
ovpnclient # %GOptions error: Unrecognized option or missing or extra parameter(s) in /dev/stdin:8: iproute (2.5.0)
ovpnclient # Use --help for more information.

can anyone confirm?

Yes, the test seems to be broken on master.


Make build-init plugin use JUnit Jupiter by default

Context
Make JUnit Jupiter the default choice when generating a Java library project and use it when generating a Gradle plugin project in Java. Please see individual commits for details about additional cleanup steps.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[x] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.


⚠ Bofrost.es has degraded performance

In f90a190, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1379 ms

Resolved: Bofrost.es performance has improved in a82146e after 5 minutes.


Version Packages (beta)

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to beta, this PR will be updated.
⚠⚠⚠⚠⚠⚠
beta is currently in pre mode so this branch has prereleases rather than normal releases. If you want to exit prereleases, run changeset pre exit on beta.
⚠⚠⚠⚠⚠⚠
Releases
<EMAIL_ADDRESS>Minor Changes

#325 095248e Thanks @mrcjhicks! - Added semantic aliases used by in-line alert.
Token Diff
Tokens added (5):

informative-subtle-background-color-default
negative-subtle-background-color-default
neutral-subtle-background-color-default
notice-subtle-background-color-default
positive-subtle-background-color-default

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [095248e]:

<EMAIL_ADDRESS>

Run report for 3c443fff
Total time: 25.3s | Comparison time: 29.9s | Estimated savings: 4.7s (15.6% faster)

Action
Time
Status
Info

🟩
SyncWorkspace
0ms
Passed

⬛
SetupNodeTool(~20.12)
1.2s
Skipped

🟩
InstallNodeDeps(~20.12)
6.5s
Passed

🟩
SyncNodeProject(tokens)
0.3ms
Passed

🟩
SyncNodeProject(root)
0.1ms
Passed

🟩
SyncNodeProject(csvGenerator)
19ms
Passed

🟩
RunTask(tokens:buildTokens)
14.7s
Passed

🟩
RunTask(tokens:buildManifest)
684ms
Passed

🟩
RunTask(root:test)
1.7s
Passed

🟩
RunTask(csvGenerator:generateCSV)
2.2s
Passed

🟩
RunTask(csvGenerator:test)
2.9s
Passed

🟩
RunTask(tokens:build)
2s
Passed

🟩
RunTask(tokens:generateDiffResult)
2s
Passed

Touched files
.changeset/pre.json
packages/tokens/CHANGELOG.md
packages/tokens/package.json
tools/token-csv-generator/CHANGELOG.md
tools/token-csv-generator/package.json


Improved Hacker news scraper to add more information

Description

The hackerNews scraper is improved to retrieve the articles along with their score, author, author url, time, comment count and link.

Resolves: [#429]
Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[x] Have you adhered to the repository's defined coding convention rules?
[x] Have you updated the 'documentation.md' file with the method/function documentation?
[x] Have you sent a message along with the result or response?
[x] Have you used the try-catch technique?
[x] Has the method/class been added to the documentation (md file)?

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

Great work @Mihan786Chistie !!
Do not forget to

Share your contribution on socials(twitter mainly), tag us and Clueless 🫂
Give this repository a star ✨
Follow Clueless Community on GitHub ✅


Doc issue: How to specify "default" in APIs

How do you specify the "default" value for a property in the APIs?
For example, the RequestSpotInstance API specifies that the "default" SpotPrice is the On-Demand price, but it doesn't say how to specify default. The SpotPrice property of the RequestSpotInstancesRequest class is a string. Set it to null? string.Empty?
Some of the fields are non-nullable, for example int32. In the same api, the "default" for the number of instances is 1. The property is an int. So....do I request 0 instances in order to get the default of 1? I would expect a Nullable instead to allow for a default.
(Sorry if this is the wrong place for this kind of question but it seems like a general doc issue)

Searched the code - null for default.
CANNOT actually specify "default" for value types, so the docs are wrong/misleading. Internal class members in the marshaller are nullable, but they are not exposed that way in the public class, no way to pass null.


Incorrect has_group verifier

has group did not test if a group exists, id -g prints group info for a user.

already request in for this, sorry


fails to import directory structure (OPML import)

it appears the 'outline title' tag is ignored - other readers import this ok, but SmartRSS doesn't - in the sample below, the 'News, General' folder is never created
partial sample of opml file...
<?xml version="1.0" encoding="UTF-8"?>
<opml version="1.0">
<head>
<title>Feed Subscriptions</title>
</head>
<body>
  <outline title="News, General" text="News, General">
      <outline text="21st Century Wire" title="21st Century Wire" type="rss"
            xmlUrl="http://21stcenturywire.com/feed/" htmlUrl="https://21stcenturywire.com"/>

This is a display issue, reloading reader view causes missing directories to appear. Looks like Sources tree is not rerendered properly on Folders collection change, I'll look into this

ah, and here i kept dumping the data and re-importing without thinking to simply refresh :)
thanks


[HELP NEEDED] Responsive Mobile Layout

I've made a few efforts to achieve a relatively Mobile-friendly layout without compromising the normal layout.
However most of my solutions are suboptimal at best. I'm kindly asking for some help to iron out bugs or optimize certain things.
Remaining issues Include, but are not limited to: Centering of the burger menu, making the gallery not cover parts of the ParametersPanel when too many Accordion Items are open and better centering of the Tab icons.

I would not approach it this way. ChakraUI has responsive built in and we should be using that.
You can set breakpoints in theme.ts like this to any that suit us.
  breakpoints: {
    sm: '30em', // 480px
    md: '48em', // 768px
    lg: '62em', // 992px
    xl: '80em', // 1280px
    '2xl': '96em', // 1536px
  },

Then you are free to use breakpoint specific styling across any component in the app.
For example, doing  ...
<Flex gap={4} flexDir={['column', 'column', 'row', 'row', 'row']}>

... would assign column flex direction for mobiles and go to row for larger screens. This can be done for any parameter. It'll be quite a bit of work to get a completely responsive layout because we'll need to make a ton of parameter updates but this would be the easier way to do it.
Once you setup the breakpoints, I'd focus setting size specific values to components that set max height at the moment and go from there. That way you get components rolling down as they shouid.
I gave this setup a quick test and it works as expected. Jus that it is quite a bit of work to make sure everything works properly.

I can make a rough draft PR for you if you want. You can build on that.

Sure thing!

https://user-images.githubusercontent.com/54517381/232291292-670de5ac-c972-4c8b-b162-2c7657e690bd.mp4
I did a quick test to see if it works at all. It does. You'll run into trouble when dealing with resizable but we'll work that out when you get to it.

What about the gallery?

What about the gallery?

I was just testing it. So I didn't add it up. It can go anywhere really. That's a design decision. Not a technical one. Was just testing the technical aspect of things right now.

I can make a rough draft PR for you if you want. You can build on that.

 Because I can contribute to this PR but you won't be able to contribute to mine without looping through my own personal repo. This might be simpler.
I can replace all of the changes in this PR with the base responsive setup and then you can take it from there? Let me know if that is okay.

 I updated this PR instead of making a new one. I also added documentation in the main post for your reference and anyone else. I also created a new thread on Discord for this so we can keep all Responsive Layout discussions there.
Hit me up with any queries you might have. Thanks again for taking this up.

Hey, we just pushed an update to the UI. Can you sync up again? We'll merge it in after. If you're up for it, please keep track of all changes we make to the UI so if anything breaks responsive, let us know.


Code Start

I've just been working on this for a little bit:

takes input from user

creates an array of randomly generated boolean values

checks to see if they are the same before the first spin

function to display the array as heads or tails

I have a placeholder for spinning the wheel, but I thought I would upload what I have so far.

Approved!


Add support for Chalice

I have added a decorator for Chalice that will enable writing LTI applications in AWS Lambda. Chalice is similar to Flask for serverless applications.

Coverage decreased (-19.0%) to 79.455% when pulling 9f2cc97b1fa21d188f4e7235dd60ef72896a39f1 on brianjbeach:master into 5088b1d3a0e7525a32a759397d244c84ae332b73 on mitodl:master.

Coverage decreased (-19.0%) to 79.455% when pulling 9f2cc97b1fa21d188f4e7235dd60ef72896a39f1 on brianjbeach:master into 5088b1d3a0e7525a32a759397d244c84ae332b73 on mitodl:master.

Coverage decreased (-19.0%) to 79.455% when pulling 9f2cc97b1fa21d188f4e7235dd60ef72896a39f1 on brianjbeach:master into 5088b1d3a0e7525a32a759397d244c84ae332b73 on mitodl:master.

Coverage decreased (-19.0%) to 79.455% when pulling 9f2cc97b1fa21d188f4e7235dd60ef72896a39f1 on brianjbeach:master into 5088b1d3a0e7525a32a759397d244c84ae332b73 on mitodl:master.

I fixed all of Travis' nagging since the last version. I have added a decorator for Chalice that will enable writing LTI applications in AWS Lambda. Chalice is similar to Flask for serverless applications.

can you add unit tests?

I found an issue in Chalice's local mode when writing test cases. That has been fixed and I can move forward again. See https://github.com/aws/chalice/issues/765 for details.
In addition to adding test cases, I have created a common base class that the flask and chalice decorators share in common.py.  the base class includes all methods that are identical across implementations.
Finally, I have merged the two commits that happened while I was waiting foe the Chalice team to incorporate my PR to fix local mode.

Hi Amir. I'm just checking in. Hoping to get this PR accepted for a blog post I am working on. If you cannot accept it please let me know so I can go another route.

please rebase it.

please also add instructions to test it manually.

Squashed changes and rebased my branch. I also created a simple hello world project for testing. You can find that here. https://github.com/brianjbeach/lti-chalice-sample

Hi. Just pinging this thread again hoping to move this forward.

@brianjbeach I started taking a deeper look at this and it's a much larger set of changes than I was anticipating. How much of this is strictly necessary for Chalice support, as opposed to the specific test implementation?
I ask because we are thinking about making this library more neutral about implementation and moving all the Flask details in our other repo. If/when we do that, we might pull out the Chalice stuff as well into it's own repo.
We really appreciate your interest, so I'm inclined to merge this, as long as you understand we may go in a slightly different direction in the future.

@brianjbeach i am back from vacation. any thoughts on https://github.com/mitodl/pylti/pull/84#issuecomment-397275649. Can i test it locally? Even debug is true it is now showing me trace.

Amir, I finally got to look at this today and I was able to reproduce your error. I need some time to figure it out. I have never tested in local mode, but the unit tests leverage local mode and they are passing. I'm not sure why it's failing when running local mode from the command line. I should have more time to debug it this week.

i was trying but get error.
["There was an LTI communication error", 500]

can you give me time today where we can sit together on hangout and test it?

Amir, I updated the instructions here https://github.com/brianjbeach/lti-chalice-sample. I have ran through that a few times now. What client are using for testing? I don't see the request body in the screenshot so it is difficult for me to debug it.

Amir, I added an call to log the original exception with a stack trace whenever we return the default_error "There was an LTI communication error".  This should help you see why you are failing. Note that default_error is common so Flask and other implementations will also log when this error is thrown.

I realize I am adding lots of little changes again. Let me know if you want me to squash them again.

@amir-qayyum-khan I think @blarghmatey was able to run this successfully?

This is great. Thanks for all of you help. Have you pushed to pypi?

Thank you for the reminder. I just pushed up v0.7.0 to PyPI as an sdist and a wheel


[WIP] tests: Don't consider MiningChain in tests

What was wrong?
Tests were being parametrised for both [Chain, MiningChain], only for MiningChain to be marked skip. pytest has to set up the fixtures (and therefore tear them down afterwards) so that they can be marked.
Identified in PR comment https://github.com/ethereum/trinity/pull/608#issuecomment-493433855.
TBH, I don't know why there's a distinction, or if MiningChain is used somewhere. @carver, do you?..
EDIT: Looks like in integration tests and plugin txpool tests; these use the class directly AFAIU, they don't need the parametrised chain fixture?..
How was it fixed?
Don't bother parametrising the fixture.
Before PR, for Byzantium jobs 2142, 2177:
============ 4691 passed, 4732 skipped, 1 xfailed in 372.96 seconds ============
============ 4691 passed, 4732 skipped, 1 xfailed in 387.18 seconds ============

With PR, for Byzantium jobs 2209, 2229:
============= 4691 passed, 20 skipped, 1 xfailed in 241.98 seconds =============
============= 4691 passed, 20 skipped, 1 xfailed in 227.80 seconds =============

There is, of course, much variance.
To-Do

[ ] Clean up commit history

[ ] Add entry to the release notes PR

Cute Animal Picture

Source: Tomas Ivašauskas @ efoto.lt

Waiting for first CI run (no-change from master) to finish to get a public reading; meanwhile, local run (for Byzantium) shows a drop to 141 seconds from 167 (~15% reduction).

After the last change (split across a587e72 and 057a998), pytest fixture chain_without_block_validation is not used anywhere. :/

After the last change (split across a587e72 and 057a998), pytest fixture chain_without_block_validation is not used anywhere. :/

Yeah, this is one of those things that looks like it hasn't been touched at all since the split, so not a huge surprise that things can be cleaned up a lot. Testing both APIs is important in py-evm, of course, but at first glance I am :+1: to drop MiningChain tests from trinity.

I'll open a separate issue, so that clean-up/refactoring does not get mixed with optimisations. (Tried to do it here, got out of control fast.)
Unmarking WIP; my hope is this will have a notable impact on PR #608.


[camera] Move camera to camera/camera

Description
This pull request prepares the camera plugin for federation by simply moving all current logic into it's own "camera" folder. A suggested by @ditman in the PR #3253 (see https://github.com/flutter/plugins/pull/3253#issuecomment-723355867).
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.

I believe this is not necessary since this change doesn't require a release to pub.dev

[ ] I updated CHANGELOG.md to add a description of the change.

I believe this is not necessary since this change doesn't require a release to pub.dev

[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@ditman, thank you for reviewing. I have processed your feedback and would appreciate it if you could give another review.


Pairing ok, but Siri doesn't find any of my KNX lights

Hi,
I'm using OpenHAB with KNX.
I'm able to pair the bridge in HMCatalog and iDevices , but Siri doesn't find any lights.
If I say "Couch Licht einschalten" it reports "Ich habe diesmal keine Lampen gefunden". What am I doing wrong?
Sitemap:
sitemap demo label="HomeKit" {
Switch item=Couch
}
Items
Switch Couch	"Couch"	(EG_Wohnen, Licht)	{ knx="1/0/7" }

Would I see the configured items in HMCatalog? How can I verify that all items are detected?

This is the REST response:
{ name: 'homekit',
label: 'HomeKit',
link: 'http://localhost:8081/rest/sitemaps/homekit',
homepage:
{ id: 'demo',
title: 'HomeKit',
link: 'http://localhost:8081/rest/sitemaps/homekit/demo',
leaf: 'false',
widget:
{ widgetId: 'demo_0',
type: 'Switch',
label: 'Couch',
icon: 'switch',
item: [Object] } } }
It gets parsed as an empty array.

Ok, got it. The bug is, that if there is only one item in the sitemap, then the widgets.length condition in ItemProvider returns undefined, because it is no array.


[WebAPI] MediaRecorder > Exceptions

Acceptance criteria

[x] The listed features are documented sufficiently on MDN

[x] BCD is updated

[x] Interactive example and data repos are updated if appropriate

[x] The content has been reviewed as needed

Features to document

Exceptions thrown by the MediaRecorder API (not just the MediaRecorder interface necessarily)

Related Gecko bugs

[ ] Fire InvalidModificationError on track set changes — pretty simple, just need to add documentation for the Exceptions thrown by the various parts of the MediaRecorder interface.

For folks helping with Firefox-related documentation features — make sure above AC have been done, but also:

Set bugs to dev-doc-complete
Add entry to Firefox release notes if feature is enabled in release
Add entry to Firefox experimental features page if feature is not yet enabled in release

Exception is already here: https://developer.mozilla.org/en-US/docs/Web/API/MediaRecorder/start - in fact all exceptions here are correct.
Updated https://wiki.developer.mozilla.org/en-US/docs/Web/API/MediaRecorderErrorEvent/error
Need to check BDC for MediaRecorderErrorEvent - the tests aren't found if I use the mdn-bcd-collector app

Happy to see that others are using the mdn-bcd-collector project!  😄
At the moment, the collector project has disabled testing event APIs, since most of the interfaces aren't exposed on their own.  I opened https://github.com/foolip/mdn-bcd-collector/issues/799 for further discussion regarding adding tests for event interfaces.

Oh this is brill @vinyldarkscratch thank you so much, I was going to drop you an email to ask!
The app is great btw... and in regards to events I'll remember that for the future, so thank you! 🙏
@chrisdavidmills I'm putting this in for review - I am still unable to change the bug to dev-doc-complete however, apologies!

@Rumyra reviewed, looks great thanks! I've also set the bug to dev-doc-completed.
We were waiting for any BCD to merge, or shall we close this?

BCD has been merged :) All is good @chrisdavidmills happy to close this 👍


Feature Request - whereis alias for Get-Command

Summary of the new feature/enhancement
As a part-time Linux user, I am used to using whereis.  I would like an alias for Get-Command to whereis in PowerShell. Currently, no whereis alias is available.

Proposed technical implementation details
Considering I use about 10 commands in Linux and most of them have already been aliased in PowerShell since the early days, I would like to create a PR to add this alias to PowerShell itself.
Oh, that's not very technical, but basically I'd read the source code for other Linux-based aliases, and modify for whereis to Get-Command and assume I must have agreement first.

Any reason you can't just add a set-alias in your profile?

While there's been quite a bit of opposition to adding some aliases that collide with Linux in the past... I think this makes a lot of sense.
Get-Command can and will find both native utilities as well as PowerShell commands, so despite overshadowing the builtin whereis alias, I think this would largely end up being an enhancement of it, rather than a detractor.

IMHO it's so easy to ship and sync your user profile around that this should stay in the realm of a profile alias. You can also always ship a module that adds tons of aliases and completers, but I think keeping the surface area of the core of powershell small and lean on existing available extensibility is the right move here.

There are many Powershell modules and short names (aliases) can conflict. Current intention is to delegate aliases to users - they can create aliases in their profiles as they prefer.

FWIW, here's what I personally have in my toolbox:

whatis.ps1
whence.ps1
dirname.ps1

Unix-isms for PowerShell.zip
This allows constructs such as:

code (whence foo)
code (dirname (whence bar))
ii (dirname (whence baz))

Very crude, but does the job.
(I used MKS Toolkit / Korn Shell as my standard shell / scripting environment on Windows, both at home and at work, for more than 25 years... @BrucePay will remember that was a great environment...)

Working at MKS was great. A very cool company.

A nice company to work with, too. I was a beta tester for quite some time. My personal and professional relationship with them started to degrade when product activation was introduced. Then someday they didn't want me as a tester anymore... So I had to bite the bullet and convert all my ksh workloads to... PowerShell ;-)


Add ACME/Let’s Encrypt integration tests

Tests are based on a boulder image I pushed on my personal docker hub account.
Thx @gwallet for the help.

@trecloux If you don't mind, could you use this image instead:
https://hub.docker.com/r/containous/boulder automatically build from https://github.com/containous/boulder ?

@emilevauge I updated the image. Unfortunately the test looks instable on travis

@emilevauge Thanks for the tip, fixed it.
@vdemeester Ok for you ?

@vdemeester I rebased the branch, should I squash the commits ?

@trecloux

should I squash the commits ?

In this case yeah, we don't need to keep Copy paste is bad :-) commits in history ;)


【test】Replenish the UT for spring registry plugin

What type of PR is this?
/test
Which issue(s) this PR fixes:
Fixes #737
Add UT for registry plugin Only
Use case description:
Replenish the UT for spring registry plugin
Does this PR introduce a user-facing change?
No
Additional documentation e.g., usage docs, etc.:
No

Codecov Report

Merging #736 (d7eff2d) into develop (c0ea065) will increase coverage by 3.64%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop     #736      +/-   ##
=============================================
+ Coverage      26.42%   30.06%   +3.64%
- Complexity      1574     1790     +216
=============================================
  Files            617      617
  Lines          13438    13436       -2
  Branches        1482     1482
=============================================
+ Hits            3551     4040     +489
+ Misses          9440     8874     -566
- Partials         447      522      +75

Flag
Coverage Δ

unittests
30.06% <0.00%> (+3.64%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...y/interceptors/health/ConsulHealthInterceptor.java
76.31% <0.00%> (+76.31%)
:arrow_up:

...wei/registry/service/register/RegisterManager.java
56.75% <ø> (+10.41%)
:arrow_up:

...ant/core/service/tracing/sender/TracingSender.java
68.25% <0.00%> (+4.76%)
:arrow_up:

...awei/registry/auto/sc/ServiceCombRegistration.java
70.00% <0.00%> (+10.00%)
:arrow_up:

...huawei/registry/support/RegisterSwitchSupport.java
66.66% <0.00%> (+11.11%)
:arrow_up:

...try/config/RegistryConfigSubscribeServiceImpl.java
18.18% <0.00%> (+18.18%)
:arrow_up:

...m/huawei/registry/service/register/ScRegister.java
26.66% <0.00%> (+18.33%)
:arrow_up:

...main/java/com/huawei/registry/utils/ZoneUtils.java
100.00% <0.00%> (+27.27%)
:arrow_up:

.../huawei/registry/config/RegisterDynamicConfig.java
100.00% <0.00%> (+28.57%)
:arrow_up:

... and 37 more

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


Feature/update to duende

Claims are now strings in Duende, so I removed the obsolete call to ClaimConverter in RedisUserStore.cs.
Upgraded projects to .net 6
Replaced Newtonsoft with System.Text.Json

The check fails because the build agent is set to build up to .Net 5 and the project was compiled with .net 6 for LTS.
Please be so kind and update the build agent. Locally, the tests are green and the quickstart solution works.


VSTS Artifacts upload from PowerShell

Hello All,
I need a solution/ alternative ways to upload a file from PowerShell to VSTS Artifacts.
is there anyway,
your information will be highly appreciated.

This issue is stale because it has been open for a year with no activity. Remove the stale label or comment on the issue otherwise this will be closed in 5 days


do not change the indents of bullet lists

I also created a PR in https://github.com/vim/vim/pull/15426

Thank you for the PR!
Though, maybe I'm missing something. When I'm writing bullet lists, it looks to me that it already works correctly. What exact problem does this PR address?

Thank you for the PR!
Though, maybe I'm missing something. When I'm writing bullet lists, it looks to me that it already works correctly. What exact problem does this PR address?

When I write
...
- Banana
  - Banana1
  - Banana2
- Cherry
...

and run gg=G to format the whole file, I don't want to change the indents of bullet lists, but actually it becomes
...
- Banana
- Banana1
- Banana2
- Cherry
...

OK, I see. However, your fix does not seem to do that, at least on VIM 9.1

OK, I see. However, your fix does not seem to do that, at least on VIM 9.1

I have force-pushed a new commit to fix the bug. I think now it should work.

Sorry for the slow response. Thanks for the fix!!


BNavBar type="dark" doesn't appear to work

Describe the bug
When in the light theme, setting the type property to "dark" doesn't seem to work; I would expect the text to change to be lighter, but it stays black, making the navbar difficult to read. This is visible right on the docs:

Reproduction
https://bootstrap-vue-next.github.io/bootstrap-vue-next/docs/components/navbar.html
System Info
System:
    OS: macOS 13.5.2
    CPU: (10) arm64 Apple M1 Max
    Memory: 20.27 GB / 64.00 GB
    Shell: 3.5.1 - /opt/homebrew/bin/fish
  Binaries:
    Node: 18.16.1 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.5.1 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 117.0.5938.92
    Safari: 16.6

Used Package Manager
npm
Validations

[X] Have tested with the latest version. This is still alpha version and sometime things change rapidly.
[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

I first noticed this in my project, but going to the docs, it was trivial to just change the theme to light and all the navbars show the same problem.

Whether or not you like it, that's the default behavior when you use variants.

Whether or not you like it, that is Bootstraps default behavior.
However, you can however solve this issue by using color modes

https://getbootstrap.com/docs/5.3/components/navbar/#color-schemes

Technically :eyes:
...
You could
...
Add
class="navbar-dark"

Whether or not you like it, that is Bootstraps default behavior.

Well, yes, but in BootstrapVue (and the docs for BootstrapVueNext, currently) the type property is supposed to change the text color from light to dark. From the current BootstrapVueNext docs:

Control the text color by setting type prop to light for use with light background color variants, or dark for dark background color variants.

(https://bootstrap-vue-next.github.io/bootstrap-vue-next/docs/components/navbar.html#color-schemes)
So while you're right that color modes could be used to fix it, that's both not documented, and a deviation from BootstrapVue, which is why I assumed this was a bug and not an intended breaking change.
Personally, I've never been the biggest fan of the type property on navbars and would like to simply set the text variant I want; so I'm fine if that's the new direction. I just also don't want to do my own thing if the intent was to keep this and make it work in the future.

I didn't even know the type property existed in the docs. (Totally have read them, I promise. Jk, totally haven't). But the type property doesn't exist in the lib. Docs are usually out of date, mostly because I don't update them when I make changes.

I didn't even know the type property existed in the docs. (Totally have read them, I promise. Jk, totally haven't). But the type property doesn't exist in the lib. Docs are usually out of date, mostly because I don't update them when I make changes. It's on the todo list. Alternatively, you could fix it 👀

Haha, fair enough, I'm down to fix it. I can check the BootstrapVue code, but I believe it was literally just swapping between the navbar-light and navbar-dark classes. Is that implementation good enough, or do you want something more in depth?

Neither navbar-light nor navbar-dark classes should be referenced. They are deprecated classes. We are going to direct people to use color modes only. type props should be removed from the documentation. The files are navbar.md and navbar.data.ts. A section will likely need to be made called Color Schemes or something. It should briefly talk about how if a global scheme is used, it may not be needed, but if a static variant navbar is used, then you may want to override the global scheme.


I want to make a card.

Do you know how to make a card similar to this in WinUI3 for Python?

I also want it to do the following：
1.Cards are automatically added based on a list when a refresh button is clicked
2.The cards are all in a scrolling area and use a fluid layout
3.When you click a button in a card, it automatically gets the text of the Textblock in the card where the button is located
4.Cards have a fixed template, and the content of the template is automatically modified when the card is added
I love WinUI3 so much that I didn't expect to be able to use it in Python one day

Sorry, I have no time to implement the all features. This is the minimal sample code showing the cards.
from win32more._winrt import unbox_value
from win32more.xaml import XamlApplication
from win32more.Microsoft.UI.Xaml import Window
from win32more.Microsoft.UI.Xaml.Controls import GridView, Border, TextBlock
from win32more.Microsoft.UI.Xaml.Media import SolidColorBrush
from win32more.Microsoft.UI.Xaml.Shapes import Rectangle
from win32more.Windows.UI import Colors
from win32more.Windows.Foundation import PropertyValue

class App(XamlApplication):
    def OnLaunched(self, args):
        win = Window()
        grid = GridView()
        grid.IsItemClickEnabled = True
        grid.add_ItemClick(self.ItemClicked)
        for i in range(10):
            border = Border()
            border.Background = SolidColorBrush(Colors.DarkGray)
            border.Width = 200
            border.Height = 150
            name = "Card" + str(i)
            border.Tag = PropertyValue.CreateString(name)
            text = TextBlock()
            text.Text = name
            border.Child = text
            grid.Items.Append(border)
        win.Content = grid
        win.Activate()

    def ItemClicked(self, sender, args):
        tag = args.ClickedItem.as_(Border).Tag
        print(unbox_value(tag))

XamlApplication.Start(App)

Sorry, I have no time to implement the all features. This is the minimal sample code showing the cards.
from win32more._winrt import unbox_value
from win32more.xaml import XamlApplication
from win32more.Microsoft.UI.Xaml import Window
from win32more.Microsoft.UI.Xaml.Controls import GridView, Border, TextBlock
from win32more.Microsoft.UI.Xaml.Media import SolidColorBrush
from win32more.Windows.UI import Colors
from win32more.Windows.Foundation import PropertyValue

class App(XamlApplication):
    def OnLaunched(self, args):
        win = Window()
        grid = GridView()
        grid.IsItemClickEnabled = True
        grid.add_ItemClick(self.ItemClicked)
        for i in range(10):
            border = Border()
            border.Background = SolidColorBrush(Colors.DarkGray)
            border.Width = 200
            border.Height = 150
            name = "Card" + str(i)
            border.Tag = PropertyValue.CreateString(name)
            text = TextBlock()
            text.Text = name
            border.Child = text
            grid.Items.Append(border)
        win.Content = grid
        win.Activate()

    def ItemClicked(self, sender, args):
        border = args.ClickedItem.as_(Border)
        tag = border.Tag
        textblock = border.Child.as_(TextBlock)
       textblock.Text = textblock.Text.split(' ', 1)[0] + " was clicked!"

XamlApplication.Start(App)

I want to create a page and GridView using xaml and use this method, and the background of the card you made looks different than I wanted.


Issue running detekt

Hi all, I have an issue with running gradle detekt in kotlin project.
I added detekt to list of plugins of module id("io.gitlab.arturbosch.detekt").version("1.22.0")
id("io.gitlab.arturbosch.detekt").version("1.22.0")
And I have next versions of:
Java - openjdk version "17.0.5" 2022-10-18
Kotlin - Kotlin version 1.7.0-release-281 (JRE 17.0.5+8)
Gradle - 7.3.3
But when I run gradle detekt next error appear:

FAILURE: Build failed with an exception.
What went wrong:
Execution failed for task ':report-service:detekt'.
Analyzing .../src/main/kotlin/.../SomeRepository.kt led to an exception.
Location: io.gitlab.arturbosch.detekt.rules.bugs.EqualsWithHashCodeExist.visitClassOrObject(EqualsWithHashCodeExist.kt:72)
The original exception message was: 'boolean org.jetbrains.kotlin.psi.KtClassOrObject.isData()'
Running detekt '1.22.0' on Java '17.0.5+8' on OS 'Mac OS X'
If the exception message does not help, please feel free to create an issue on our GitHub page.

It was because of an incompatible version. This setup works with version '1.19.0'


CI: Address all outstanding deprecation warnings

All deprecation warnings generated by our CI system should be addressed via code refactoring, formatted code TODO comments, or being marked as wontfix where appropriate.
For a Ruby deprecation warning...

Refactor the Ruby code if doing so addresses the warning and does not break compatibility with our oldest supported Ruby version
Use a formatted TODO comment where the refactor needs to be applied if the refactor would break compatibility with an older Ruby version

For a gem deprecation warning, follow the same logic as the Ruby deprecation warnings depending on the impact to the oldest supported version of the gem.

[ ] Rubocop
[ ] 2.2 Unit tests
[ ] 2.3 Unit tests
[ ] 2.4 Unit tests
[ ] 2.5 Unit tests
[ ] 2.6 Unit tests
[ ] 2.7 Unit tests
[ ] 3.0 Unit tests
[ ] 3.1 Unit tests
[ ] 3.2 Unit tests
[ ] JRuby Unit tests
[ ] 2.2 Multiverse agent tests
[ ] 2.3 Multiverse agent tests
[ ] 2.4 Multiverse agent tests
[ ] 2.5 Multiverse agent tests
[ ] 2.6 Multiverse agent tests
[ ] 2.7 Multiverse agent tests
[ ] 3.0 Multiverse agent tests
[ ] 3.1 Multiverse agent tests
[ ] 3.2 Multiverse agent tests
[ ] JRuby Multiverse agent tests
[ ] 2.2 Multiverse background tests
[ ] 2.3 Multiverse background tests
[ ] 2.4 Multiverse background tests
[ ] 2.5 Multiverse background tests
[ ] 2.6 Multiverse background tests
[ ] 2.7 Multiverse background tests
[ ] 3.0 Multiverse background tests
[ ] 3.1 Multiverse background tests
[ ] 3.2 Multiverse background tests
[ ] JRuby Multiverse background tests
[ ] 2.2 Multiverse database tests
[ ] 2.3 Multiverse database tests
[ ] 2.4 Multiverse database tests
[ ] 2.5 Multiverse database tests
[ ] 2.6 Multiverse database tests
[ ] 2.7 Multiverse database tests
[ ] 3.0 Multiverse database tests
[ ] 3.1 Multiverse database tests
[ ] 3.2 Multiverse database tests
[ ] JRuby Multiverse database tests
[ ] 2.2 Multiverse framework tests
[ ] 2.3 Multiverse framework tests
[ ] 2.4 Multiverse framework tests
[ ] 2.5 Multiverse framework tests
[ ] 2.6 Multiverse framework tests
[ ] 2.7 Multiverse framework tests
[ ] 3.0 Multiverse framework tests
[ ] 3.1 Multiverse framework tests
[ ] 3.2 Multiverse framework tests
[ ] JRuby Multiverse framework tests
[ ] 2.2 Multiverse httpclients tests
[ ] 2.3 Multiverse httpclients tests
[ ] 2.4 Multiverse httpclients tests
[ ] 2.5 Multiverse httpclients tests
[ ] 2.6 Multiverse httpclients tests
[ ] 2.7 Multiverse httpclients tests
[ ] 3.0 Multiverse httpclients tests
[ ] 3.1 Multiverse httpclients tests
[ ] 3.2 Multiverse httpclients tests
[ ] JRuby Multiverse httpclients tests
[ ] 2.2 Multiverse httpclients_2 tests
[ ] 2.3 Multiverse httpclients_2 tests
[ ] 2.4 Multiverse httpclients_2 tests
[ ] 2.5 Multiverse httpclients_2 tests
[ ] 2.6 Multiverse httpclients_2 tests
[ ] 2.7 Multiverse httpclients_2 tests
[ ] 3.0 Multiverse httpclients_2 tests
[ ] 3.1 Multiverse httpclients_2 tests
[ ] 3.2 Multiverse httpclients_2 tests
[ ] JRuby Multiverse httpclients_2 tests
[ ] 2.2 Multiverse rails tests
[ ] 2.3 Multiverse rails tests
[ ] 2.4 Multiverse rails tests
[ ] 2.5 Multiverse rails tests
[ ] 2.6 Multiverse rails tests
[ ] 2.7 Multiverse rails tests
[ ] 3.0 Multiverse rails tests
[ ] 3.1 Multiverse rails tests
[ ] 3.2 Multiverse rails tests
[ ] JRuby Multiverse rails tests
[ ] 2.2 Multiverse "rest" tests
[ ] 2.3 Multiverse "rest" tests
[ ] 2.4 Multiverse "rest" tests
[ ] 2.5 Multiverse "rest" tests
[ ] 2.6 Multiverse "rest" tests
[ ] 2.7 Multiverse "rest" tests
[ ] 3.0 Multiverse "rest" tests
[ ] 3.1 Multiverse "rest" tests
[ ] 3.2 Multiverse "rest" tests
[ ] JRuby Multiverse "rest" tests
[ ] 2.5 Infinite tracing tests
[ ] 2.6 Infinite tracing tests
[ ] 2.7 Infinite tracing tests
[ ] 3.0 Infinite tracing tests
[ ] 3.1 Infinite tracing tests
[ ] 3.2 Infinite tracing tests
[ ] JRuby Infinite tracing tests

This one shows up a lot without being attached to a specific file:
net/http: warning: Content-Type did not set; using application/x-www-form-urlencoded


Initializes an object from braced-init-list

Since the library for C++11 will be good add constructor for an object with initializer list. Now need use C-array or temporary std::vector:
gml::mat<int,2,2> test({1,1,1,1},false); //<-compile-time error
gml::mat<int,4,1> result(std::vector<int>{0,0,1,1}.data(),true);//<- work, but unconvenient

Closing this old issue as initializer list would mess up constructor overloads.


Update note

closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/10032

@CarolynRowe
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Fix severity issue

Hi! In several tests I have done with your exporter (thanks for sharing, BTW!) I've observed that the severity field is not filled in properly. You also have mentioned in the code that some changes might be needed.
In my PR I check for the same field cvss2Severity but at different locations. You might also check tidwall/gjson#268 for the solution (it was related to gjson, yes).
I've also added support for Go modules by adding go.mod and go.sum in the src/ folder. Now you'll have to cd src/ and run go build ... in order to build the binary.
I will also add a new PR for building a Docker image.
Thanks for your time.
Greetings from Berlin,
Victor

Hello Victor! Thanks for your contribution, I really appreciate it.
The changes look good to me. Can you please let me know which version of Vuls are you using? I'd like to upgrade my setup and test it to make sure that everything works as expected.

Hello @dorneanu.
I've upgraded my local Vuls setup and tested out your changes. Everything worked as expected!
I am going to build and publish a new release now.
Thanks again for the contribution 🖤

Hi @iakovmarkov,
thanks for taking care and sharing this exporter at all. Have a nice day!
Greets

Hi @iakovmarkov I'm not seeing the last release after this change.  I'll try and build the code myself, but if it's not too hard for you do you mind pushing a release for others?
Thanks!

Hi @zhsc! I have created new release now: https://github.com/iakovmarkov/prometheus-vuls-exporter/releases

Thank you so much for the quick turnaround!


Setuptools support

Factor out the logic to build extensions from support.py, and add an official setuptools way to build hpy extensions.
Once this branch is merged, we should also merge the corresponding branch in PyPy, which is hpy-setuptools-extension

(this branch also renames listbuilder.c into ctx_listbuilder.c, to avoid having to special case this file inside hpy.devel


Use temp dirs for downloads in progress (fixes #12)

This is not very DRY, but it passes existing tests. Any suggestions for cleaning it up a bit, or adding new tests to exercise the partial download protection? Is this approach to fixing #12 ok, or do you have a different idea of how to address it?

Updated with greater dry-ness.

lgtm!

Rebased. Waiting for the built to run and then will merge if it is green.


Resolve Faraday deprecation notice

The Faraday gem introduced a new error class structure in v0.9.0. Let's reference this new structure instead of the backward compatibility enabling alias.
Since Faraday version 0.17.1 a deprecation warning is printed to STDERR. This resolves it.
As this new error structure was introduced in Faraday version 0.9.0 and the zendesk_api gem requires version 0.9.0 as a minimum, this is a safe change to make.
https://github.com/zendesk/zendesk_api_client_rb/blob/7fe691d0c34f818c13b1b20d1c921b4701e99e73/zendesk_api.gemspec#L29
Fixes #408.

Thank you very much for your contribution @orien ! 🙇


Database navigation list items padding lost after update

Description
I just updated to 23.0 and the list items on the left are comprised and do not have any soft of padding:
I think this is related to the last bug report I made: #19152

DBeaver Version
Version <IP_ADDRESS>303040621
Operating System
macOS Ventura 13.2.1 (22D68)
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

@jmarcher Hi. This allowed making the interface uniform for Windows, Linux, and macOS. Works as expected


cmake: 3.12.1 -> 3.12.2

Motivation for this change
Trying to use cudatoolkit-10 with magma and running into a bug solved by upgrading cmake. Reference: https://github.com/clab/dynet/issues/1457#issuecomment-423931508
Things done

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

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

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

I'm not 100% sure how to test this locally. I tried checking nix path-info -S and nix-shell -p nox --run "nox-review wip". nox returns No uncommit changes. Did you mean to use the "--against" option? and I'm a bit too new at nix to figure it out. I figure this will impact most packages and it might be easier to let CI do this. I have verified the binary on Ubuntu and NixOS, and have passed the current blocker I am facing with magma+cuda-10.

@stites Could you please re-make this pull request against the staging branch, because it would cause a mass-rebuild?

Will do, I'll also bump to 3.13.4 which is the latest stable cmake release (see #55060). Sorry about the delay!

Submitted to staging, see #56059


Cmdline feature autosave results to csv

Two new options are added to command line:

r :  ResultsAutoSaveFileName : When specified, results are automatically saved (in conjunction with the previous changes actually appended) to the specified file.
d : DbServer : When specified, overrides the existing setting's database server address. This is useful for running the same saved benchmark in different environments from command line without modifying the saved settings file.
Examples:
SqlQueryStress -c saved.SqlStress -u -t 1 -d . -r results.local.csv
SqlQueryStress -c saved.SqlStress -u -t 32 -d sqldb.perf.mycompany.com -r results.perf.csv

Thanks!!


Add GIL metric

cc @ntabris for visibility
Closes https://github.com/coiled/benchmarks/issues/809

Also cc @milesgranger

Chatted with Nat offline and he mentioned we have GIL contention prometheus data for this CI build

Just noting that I like that this went from request to ready-to-merge sub-day 👍


Screen goes blank when I highlight rte content and clear with backspace key

Notice theres an issue when I enter some data into the rte component, highlight everything and try to clear with backspace key, my app went blank. Does someone know how to handle this? I have tried adding if statements to event and event.getCurrentContent() that comes from my onRTEChange function but still throwing this error below. i have also tried console logged event and event.getCurrentContent() but i cant pick out how to handle this error. Anyone any ideas how to handle this issue?

const [value, setValue] = useState(values.comment || '')

const onRTEChange = event => {
    const content = JSON.stringify(convertToRaw(event.getCurrentContent()))
    setGuidelines(content)
}

<MUIRichTextEditor
  readOnly={disabled}
  id="rte-comment"
  label="Comment"
  controls={['numberList']}
  value={value}
  onChange={onRTEChange}
/>

Sorry I could not reproduce this problem. Are you using the latest version? Will close for now.


import numpy as np

Always import numpy as np.

Thanks.
I was thinking about this convention.
In Chainer, there are both numpy and np, but I don't think ChainerCV needs to follow this style.
I personally prefer consistency, so thumbs up for only using np.

LGTM


🐛 Fix permission issues and wrong control plane image

Xref: #115
Signed-off-by: Chris Hein<EMAIL_ADDRESS>

/kind bug
/assign @weiling61

/lgtm


Increase network timeout for router requests under heavy load (WIP)

Codecov Report

Merging #2166 (15dffd9) into master (67e9698) will decrease coverage by 58.40%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #2166       +/-   ##
=============================================
- Coverage     73.21%   14.81%   -58.41%
+ Complexity    10231     2285     -7946
=============================================
  Files           735      736        +1
  Lines         55973    56021       +48
  Branches       6896     6897        +1
=============================================
- Hits          40981     8299    -32682
- Misses        12810    47240    +34430
+ Partials       2182      482     -1700

Impacted Files
Coverage Δ

...ava/com/github/ambry/config/Http2ClientConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ain/java/com/github/ambry/config/RouterConfig.java
0.00% <0.00%> (-98.37%)
:arrow_down:

...ain/java/com/github/ambry/network/RequestInfo.java
0.00% <0.00%> (-96.78%)
:arrow_down:

...github/ambry/network/http2/Http2ClientMetrics.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...github/ambry/network/http2/Http2NetworkClient.java
0.00% <0.00%> (-64.89%)
:arrow_down:

...thub/ambry/network/http2/RequestsStatsHandler.java
0.00% <0.00%> (ø)

.../java/com/github/ambry/router/DeleteOperation.java
0.00% <0.00%> (-92.90%)
:arrow_down:

.../com/github/ambry/router/GetBlobInfoOperation.java
0.00% <0.00%> (-90.31%)
:arrow_down:

...java/com/github/ambry/router/GetBlobOperation.java
0.00% <0.00%> (-88.83%)
:arrow_down:

...ain/java/com/github/ambry/router/PutOperation.java
0.00% <0.00%> (-91.65%)
:arrow_down:

... and 584 more

Continue to review full report at Codecov.

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


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1778

Questions ?
Please refer to the Backport tool documentation


Send-MailKitMessage docs

Hello,
I couldn't find any info on using the Send-MailKitMessage cmdlet usage??

Hi Glenn,
You are right, I have not put any documentation up yet. I really should.
I created a gist demonstrating the commandlet usage here:
https://gist.github.com/austineric/0d04bae34bcc74243ac77c76ae8af6ef
Mostly I created the module for my environment at work where there is an
smtp server without authentication, just the server address and port are
needed to send email. If you have other needs let me know and maybe I can
enhance its functionality.
Eric
On Tue, Sep 8, 2020 at 10:51 PM Glenn<EMAIL_ADDRESS>wrote:

Hello,
I couldn't find any info on using the Send-MailKitMessage cmdlet usage??
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/austineric/Send-MailKitMessage/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AICOVTYBNH27MJLENTSM723SE33U7ANCNFSM4RA6AW5A
.

Thank you for the explanation... your module with the added Credential for authentication and SecureSocketOptions to match the SMTP server would allow it to be used everywhere!

Hi @user8446,
Version 3.0.0 supports credentials and SecureSocketOptions.
Documentation was also added in the ReadMe.


Fix vendor comments in 29 casks

Another portion of fixes for https://github.com/caskroom/homebrew-cask/issues/19450.

LGTM


Snowflake COMMENT ON support

Support for https://docs.snowflake.com/en/sql-reference/sql/comment, for example:
COMMENT ON my_schema.my_table IS 'hello world'

Currently, parsing fails:
Expected: an SQL statement, found: COMMENT

Hi @psukys
I can submit a PR to resolve this issue if you're planning to do it.

@git-hulk please do, I lack knowledge on internals to pick it up immediatelly


[modify] apply ckeditor's default sytles

Coverage decreased (-9.2%) to 75.523% when pulling 0d401037ff2747871d87fc0dffbadc1a82ea1e57 on sunny4381:fixes/cms-notice-style into d89912b6b11d83cd12ddc78fb771da19b22b0289 on shirasagi:master.


[Workflow] Android full CI runs out of space in github CI

Workflow issues
Running full android CI gets out of space:

System.IO.IOException: No space left on device : '/home/runner/runners/2.296.1/_diag/Worker_20220908-144851-utc.log'
   at System.IO.RandomAccess.WriteAtOffset(SafeFileHandle handle, ReadOnlySpan`1 buffer, Int64 fileOffset)
   at System.IO.Strategies.BufferedFileStreamStrategy.FlushWrite()
   at System.IO.StreamWriter.Flush(Boolean flushStream, Boolean flushEncoder)
   at System.Diagnostics.TextWriterTraceListener.Flush()
   at GitHub.Runner.Common.HostTraceListener.WriteHeader(String source, TraceEventType eventType, Int32 id)
   at GitHub.Runner.Common.HostTraceListener.TraceEvent(TraceEventCache eventCache, String source, TraceEventType eventType, Int32 id, String message)
   at System.Diagnostics.TraceSource.TraceEvent(TraceEventType eventType, Int32 id, String message)
   at GitHub.Runner.Worker.Worker.RunAsync(String pipeIn, String pipeOut)
   at GitHub.Runner.Worker.Program.MainAsync(IHostContext context, String[] args)
System.IO.IOException: No space left on device : '/home/runner/runners/2.296.1/_diag/Worker_20220908-144851-utc.log'

see
https://github.com/project-chip/connectedhomeip/actions/runs/3016038239/attempts/1

### Platform

android

### Anything else?

Smoke tests do pass, so we likely need to split out builds and cleanup after builds are made.

Looking through android yaml it also seems we do 'submodules' on checkout which is not needed if we use checkout_submodules.py